[website_docs] Questions about "Getting started", and OTel website 404s for Java docs

Where's the "Getting started" page?
We're aiming for every language SIG to have a Getting started page. Java used to have one (see open-telemetry/opentelemetry-java#3070), but then open-telemetry/opentelemetry-java#3081 dropped it and another page:

website_docs/getting_started.md was deleted
website_docs/automatic_instrumentation.md was deleted

I'm seeing that QUICKSTART.md, now just a placeholder, directs readers to https://opentelemetry.io/docs/java/manual_instrumentation. Might manual_instrumentation be a suitable candidate for a Getting started page? If not, what do you suggest we do to get a Getting started page for OTel Java?
/cc @theletterf -- who is looking into the topic of Getting started pages.
Addressing top OTel website 404s for Java docs
As you can see from https://github.com/open-telemetry/opentelemetry.io/issues/635, the top Java 404s are for the two files mentioned above that were deleted by open-telemetry/opentelemetry-java#3081. When a page is deleted, a redirect target needs to be chosen so that readers who still have a link to the page don't get a 404, page not found error.
So my question here is, what should the redirect target be for the following OTel website paths:

/docs/java/automatic_instrumentation/?
/docs/java/getting_started/?
Note that if we recreate a Getting started page, then we won't need a redirect target.

/cc @austinlparker

@chalin is there still something that needs to be done for this? thx


Add Series.shift() (with minor cleanups)

This PR add shift() in Series too.

Codecov Report

Merging #563 into master will decrease coverage by 2.9%.
The diff coverage is 94.73%.

@@            Coverage Diff             @@
##           master     #563      +/-   ##
==========================================
- Coverage   93.79%   90.88%   -2.91%
==========================================
  Files          31       31
  Lines        4432     4434       +2
==========================================
- Hits         4157     4030     -127
- Misses        275      404     +129

Impacted Files
Coverage Δ

databricks/koalas/missing/indexes.py
100% <ø> (ø)
:arrow_up:

databricks/koalas/missing/series.py
100% <ø> (ø)
:arrow_up:

databricks/koalas/frame.py
92.29% <100%> (-2.04%)
:arrow_down:

databricks/koalas/base.py
91.66% <92.3%> (ø)
:arrow_up:

databricks/koalas/usage_logging/__init__.py
23.14% <0%> (-74.08%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
76.31% <0%> (-7.9%)
:arrow_down:

databricks/conftest.py
90.69% <0%> (-6.98%)
:arrow_down:

databricks/koalas/utils.py
97.29% <0%> (-2.71%)
:arrow_down:

... and 4 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 86ca288...ec0b0b3. Read the comment docs.

Thanks, @ueshin!


Throw on non-200 status codes

@jacoscaz I would adjust it a bit ... I send commit for review

I added a new Error class to better detect a rate limit error


⚠ Bofrost.de has degraded performance

In c24a3f3, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1903 ms

Resolved: Bofrost.de performance has improved in bafdede after 27 minutes.


Fix possible segfault and missing FAIL(...) call in unit test "Insert error"

The unit test "Insert error" contains an incorrect break condition in the for loop. Checking for the validity of the pointer address always results in a truthy value, possibly walking the whole names array. When trying to assign the referenced NULL value to a std::string, this could result in a segfault.
Also, a call to FAIL(...) was missing in case the expected exception did not occur in the test.

Thanks for the fixes, will merge in a moment!


Add TAU logo as favicon

404s in the dev console are annoying.  Add the TAU logo as a favicon to make the 404 go away.

Fixed in d2552f8


Fix #52. Systemd booting logs issue.

it may be cool to make this user settable, so it is by default off, but if a user wants they can set and env var and then the systemd logs will be shown...just an idea

@shaunmulligan, I don't think we should let user do that since these logs are not useful for users. They can retrieve logs for each service using journalctl

They may be useful, one could see if devices are being plugged in correctly or if a service is starting up and failing etc. But i guess it is possible through journalctl, i guess maybe @petrosagg might have some feelings on this

I don't mind either way. SGTM


portfolio-setup Review

add  main folders
add  index.html file
add image and styles.css folders

Hello, Adnan @AdnanOlarmmi  I fully appreciate your review.  I made the changes needed, but the request review is still pending "REVIEW PENDING".

Hi @ANTOINE1128 😊,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request 🆘💀
You have submitted a project, that for the Portfolio: finish mobile version but this link is for the Portfolio: setup and mobile first  So kindly implemented the required project requirement for Portfolio: finish mobile version and submit the new pull request with all the project requirement.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.


After flutter create cannot build macOS Xcode project until first flutter run: "Unable to load contents of file list"

Steps to Reproduce

cd integration_test/flavors
flutter create --platforms=macos .
xed macos/Runner.xcodeproj
Hit the play button in XCode.

Expected results:
The project should build just fine.
Actual results:
The project fails to build.
The project will run with flutter run, just not from XCode.

Hi @a-wallen
From my side, I'm hitting this issue #72329 when executing the command flutter create --platforms=macos .:
➜  flavors git:(stable) ✗ flutter create --platforms=macos .

The Xcode project defines schemes: free, paid
You must specify a --flavor option to select one of the available schemes.

So I use below command instead:
flutter create --org free --platforms=macos .

And following remaining steps (3 and 4) will lead to below error:
error build: Unable to load contents of file list: '/Users/huynq/Documents/GitHub/flutter/dev/integration_tests/flavors/macos/Flutter/ephemeral/FlutterInputs.xcfilelist'

error build: Unable to load contents of file list: '/Users/huynq/Documents/GitHub/flutter/dev/integration_tests/flavors/macos/Flutter/ephemeral/FlutterOutputs.xcfilelist'

Did you get the error messages same as above?

@huycozy sorry that I didn't include my doctor output. I'm on master, this patch fixed the issue that you're mentioning.

The project fails to build.

@a-wallen what was the original failure?   If you are using plugins then you need to open macos/Runner.xcworkspace.

Same error even with opening macos/Runner.xcworkspace.

@jmagman haha 😅  here's the full build log.

Showing Recent Issues

Build target Flutter Assemble of project Runner with configuration Debug
error: Unable to load contents of file list: '/Users/stephenwallen/Development/flutter/flutter/dev/integration_tests/flavors/macos/Flutter/ephemeral/FlutterInputs.xcfilelist' (in target 'Flutter Assemble' from project 'Runner')
error: Unable to load contents of file list: '/Users/stephenwallen/Development/flutter/flutter/dev/integration_tests/flavors/macos/Flutter/ephemeral/FlutterOutputs.xcfilelist' (in target 'Flutter Assemble' from project 'Runner')
warning: Run script build phase 'Run Script' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'Flutter Assemble' from project 'Runner')

Unable to load contents of file list: '/Users/stephenwallen/Development/flutter/flutter/dev/integration_tests/flavors/macos/Flutter/ephemeral/FlutterInputs.xcfilelist'

Unable to load contents of file list: '/Users/stephenwallen/Development/flutter/flutter/dev/integration_tests/flavors/macos/Flutter/ephemeral/FlutterOutputs.xcfilelist'

Run script build phase 'Run Script' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase.

That has nothing to do with flavors, it's that flutter create and even flutter pub get aren't creating those xcfilelists.  If you flutter run once then you can build in Xcode.
They need to be created during ensureReadyForPlatformSpecificTooling, not buildMacOS.

@jmagman This also happens when following the steps outlined in https://docs.flutter.dev/deployment/cd#xcode-cloud. The ci_post_clone.sh does not contain a flutter build command. What would be the appropriate way to prepare the project so the Xcode build works out of the box?

I used the CI-post-clone script from the Flutter documentation and added flutter build macos which generated the missing files and solved the problem.
This is the full script for reference
#!/bin/sh

# Fail this script if any subcommand fails.
set -e

# The default execution directory of this script is the ci_scripts directory.
cd $CI_PRIMARY_REPOSITORY_PATH # change working directory to the root of your cloned repo.

# Install Flutter using git.
git clone https://github.com/flutter/flutter.git --depth 1 -b stable $HOME/flutter
export PATH="$PATH:$HOME/flutter/bin"

# Install Flutter artifacts for iOS (--ios), or macOS (--macos) platforms.
flutter precache --macos

# Install Flutter dependencies.
# I use a monorepo in github which contains many projects other that Flutter.
# Edit the next line to cd into the right Flutter project dir
cd $CI_PRIMARY_REPOSITORY_PATH/<your child proj dir>
flutter pub get

# Install CocoaPods using Homebrew.
HOMEBREW_NO_AUTO_UPDATE=1 # disable homebrew's automatic updates.
brew install cocoapods

flutter build macos

# Not sure if this step is needed anymore since I called flutter build above.
# Install CocoaPods dependencies.
cd macos && pod install # run `pod install` in the `macos` directory.

exit 0


implement data generation pipeline via pinto

implement data generation pipeline via pinto

Was able to get the pipeline running end to end with a converging training loss!
I might have incorporated a few too many things into this one pull request. I'll try to summarize below:

Created pinto pipeline connecting all data generation projects and training
Implemented optional validation into the trainer library, as well as the train project
Incorporated boolean for turning on and off mixed precision training


Reset password link broken for expo

Feature request
Is your feature request related to a problem? Please describe.
When sending a reset password email:
const { data, error } =<EMAIL_ADDRESS>
the returned link looks like:
https://jwpgcjpssmjfggggeegqlz.supabase.co/auth/v1/verify?token=KqjDGb-JA1xpLbOpI-jyZQ&type=recovery&redirect_to=mymobileapp://resetpassword

This  correctly opens the mobile app with path mymobileapp://resetpassword.
However, after the app is opened  there is no way to access the query params token and type.
I'm using the following code to listen to the url:
function urlRedirect(url) {
    if(!url) return;
    let { path, queryParams } = Linking.parse(url);
}
Linking.addEventListener('url', event => {
    urlRedirect(event.url);
});

path returns 'resetpassword',
queryParams returns null.
Describe the solution you'd like
Send a link in the following format:
mymobileapp://resetpassword?token=KqjDGb-JA1xpLbOpI-jyZQ&type=recovery

@kiwicopple thanks.
I'm going to try to submit a pull request.
I really want to use supabase in my react native app.

Did this ever get solved @10000multiplier?

Did this ever get solved @10000multiplier?

@scarsam No I'm still redirecting to my website and manually transforming the url there.

I am OK with that
function _handleOpenUrl(event) {
    const url = event.url.replace("#", "?");   // replace # with ?  to use parse
    const obj = Linking.parse(url);              //parse url
    const str = JSON.stringify(obj);            // JSON stringify to store in context storage
    setItem(str)                                          //store in context storage ( in my case I store in asyncstorage )
  }

  useEffect(() => {
    Linking.addEventListener("url", _handleOpenUrl);
    return () => {
      Linking.removeEventListener("url", _handleOpenUrl);
    };
  }, []);

Why do you need access to the token? @nainglynndw
(Given that this issue has been inactive for a while I'll close it but feel free to continue discussing or re-open it.)

I have this issue now, do you think this could be solved?


Refactor: split platform texture

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Refactor.
What is the current behavior? (You can also link to an open issue here)
Use Webgl context in engine texture.
What is the new behavior (if this is a feature change)?
Use platform context in engine texture.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No.
Other information:
link issue: #38

In texture relate calsses has many readonly, you should clean it


Remove operator designated types once CI nodes have been updated to more recent Xcode

Once the CI nodes no longer use an Xcode that uses operator designated types in their swift interfaces, we should be able to remove designated types: https://github.com/apple/swift-syntax/pull/1266

rdar://104684250

We decided that we want to support parsing of older Swift code for now.


chore: splitting the role name to avoid showing workspace name next to it

Description

Splitting the role name to avoid showing the workspace name next to the role name in the invite modal dropdown

Fixes #18831
Type of change

Refactor

How Has This Been Tested?

Tested locally that the role names in the invite modal dropdown is not showing the workspace name anymore. Also, same thing is working on members page.

Manual

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=03ee489

/ok-to-test sha=433848e

/ok-to-test sha=433848e


bug fix to prevent accessing Name on empty host

The controller crashes on deletion of baremetalmachine objects

/cc kashifest

/lgtm


fix: rename guide folders [russian]

This renames guide folders to align with lesson titles by changing the dashes(-) back to periods(.) where they have been previously replaced for the russian guide.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Relates to issue #35155
Reviewers, make sure there are no other PR's to these files before merging. Check the contributor tools.  Use the Filename Search option and search for the old filename.

LGTM - I first checked there are currently no open PRs making changes to the old filename. I also manually validated, that all the new filenames of the Guide articles match the same naming convention of the the current master branch curriculum filename.


added doubly linked list

please merge this PR doubly linked list. As this does not exist in your project.


prefix is required in toml file however it does not interpolate env variables

Im namespacing config behind an id, so my execution looks like this:
confd -verbose -onetime -node "http://$ETCD_SERVER" -confdir /opt/server/confd/ -prefix="/_$SERVER_ID/config/

Lets say there is a value at /_$SEVER_ID/config/mykey = myvalue
If I want to write a template
key {{ .mykey }}

my toml file needs to look like this:
[template]
src = "server.cfg.tmpl"
dest = "/opt/server/cfg/server.cfg"
keys = [
  "/_$SERVER_ID/config/mykey"
]

Ideally I could just place "mykey" into the keys array, however, I could not get that to work with the prefix.
The problem is that the toml file does not interpolate env variables, so I'll have do sed replace $SERVER_ID in the file before using it :(

:+1:
I'm looking for the exact same thing.
We have multiple version of our app, and I want each to have its own set of keys. Without resorting to very complex template scripting.
Yea sed is easy enough, be having this baked into confd seems like clear solution maybe a .toml.tmpl file that only works with getenv.


Do not require nightly

I've recently been told that setting the environment variable RUSTC_BOOTSTRAP=1 makes a standard rustc build able to build nightly-only crates. Maybe this could be used to not require a nightly build for compiling libcore?

See my comments in https://github.com/japaric/xargo/pull/275#issuecomment-586899567: I don't think we should do this, not unless the Rust Compiler team explicitly blesses us to do so. And I honestly doubt that will happen -- so, I am closing the issue as (likely) not actionable.


Problem running Jupyter Notebook Code

Issue Type: Bug
During the past two or three Insider preview updates, I started having trouble with Visual Studio Code.  First, I could not edit Jupyter Notebooks properly.  If I copy a block of code from one cell and pasted it into another Cell, the pasted content would get duplicated, sometimes I would get a double content or tipple of the pasted content into the new cell.  so, I investigated and was asked to use the Visual Studio Code Insiders Version.  That version worked and I was able to get back to work with it.  then, after the latest version of Windows Insider Preview got installed, I am now getting crashes or a total PC Lock that requires a hard reboot to get back on, when I am running TensorFlow code in my Jupyter Notebooks.  I then tried to install Visual Studio Code (latest version as of November 11, 2020,) I am not able to edit the notebooks.  So, I am afraid of running Jupyter notebooks and have to use other programs or applications to run my code.


[$250] Migrate index.js to function component

Class Component Migration
Filenames

[ ] src/components/SignInPageForm/index.js

type: React.Component
has state values: false
has refs: true
has context: false
lifecycle methods: componentDidMount,componentWillUnmount

Task

We currently have some class components in our codebase that we would like to refactor to a function component.
Here's a link with some general advice on how to refactor a class component to a function component: https://react.dev/reference/react/Component#alternatives
If you need additional guidance, please ask in #expensify-open-source
Test for any regressions and verify that there are no breaking changes

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0139e2c951f46681da
Upwork Job ID:<PHONE_NUMBER>042664960
Last Price Increase: 2023-07-29
Automatic offers:

rayane-djouah | Contributor | 25889648

I am ready to work on this ticket. :)

I would like to work on this issue.

I can work on this issue.

I'm able to work on this if needed!

Proposal
Please re-state the problem that we are trying to solve in this issue.
Migrate SignInPageForm/index.js to function component
What is the root cause of that problem?
need to change class based component to functional
What changes do you think we should make in order to solve the problem?

component state and props from class to function syntax
render method from class to function syntax
lifecycle methods to hooks(if any)
update callbacks from class to function syntax (possibly with useCallback hook)
update refs from class to function syntax
any other class component logic to function component
and ensure no regressions occur.

What alternative solutions did you explore? (Optional)
N/A

I would love to work on this issue

Can I work this one?

Proposal
Please re-state the problem that we are trying to solve in this issue.
Migration of components/SignInPageForm/index.js to the function component.
What is the root cause of that problem?
No root cause, just need to migrate class component to function component.
What changes do you think we should make in order to solve the problem?

Replace the class component SignInPageForm with a function component.
Use the useRef hook to create the formRef and replace the ref used in the class component.
Replace the instance method preventFormDefault with a regular function within the function component.
Utilize the useEffect hook to handle the setup and cleanup of event listeners.
Finally make sure there is no regression.

What alternative solutions did you explore? (Optional)
N/A

Proporsal
export const Form = (props) => {
  const formRef = useRef();

  const preventFormDefault = useCallback((event) => {
    event.preventDefault();
  }, []);
  useEffect(() => {
   const form = formRef.current;
    if (form) {
        form.setAttribute('method', 'post');
        form.setAttribute('action', '/');

        form.addEventListener('submit', preventFormDefault);

    }
  }, [prventFormDefault])

          return (
            <FormElement
                ref={formRef }
                {...props}
            />
        );
}

Would love to work on this

Hey everyone, please note that we will not assign an issue to you just because you ask to work on it - you'll need to submit a proposal and we will select the best proposal. Please review https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md for our contributor guide to see how to work with Expensify and get selected for jobs.

@rayane-djouah thanks for that, I missed that announcement. You're right - for this particular set of issues, we're taking volunteers without proposals. We're only considering people who volunteered after the hold was taken off.
@reinesarchisorte was the first one to volunteer that isn't already assigned another issue, but @rayane-djouah was the very first one to volunteer and has already submitted a PR for their other assigned issue. @0xmiroslav I'll let you decide who we should assign this issue to!

Looks like @0xmiroslav reviewed and requested changes a few days ago.

Really sorry for the delay! Not sure why this wasn't switched back to daily.
Summarizing payouts for this issue:
Contributor: @rayane-djouah $250 (hired and paid via Upwork)
Contributor+: @0xmiroslav $250 (payable via Manual Request after BZ checklist is completed)
@0xmiroslav I'm going to close the issue, please request payment via NewDot manual request.


Merge branch 'develop' into 'main'

Codecov Report

Merging #22 (7ebcc32) into main (057cb81) will not change coverage.
Report is 7 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #22   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            2         2
=========================================
  Hits             2         2

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


orientationChanged sent to deallocated instance of RCTCamera

My app keeps crashing with this error:
*** -[RCTCamera orientationChanged:]: message sent to deallocated instance 0x1246732e0
Any thoughts? iPad mini 2 FWIW

Has the camera view been removed at this point? (ie. you've changed pages/scenes/states that makes the camera disappear)

No. Originally I was doing that, but I wondered if that might be the problem, so i switched to keeping the component on the page the whole time.
I'm handling onBarCodeRead. I'll post the code here tomorrow if you'd like.

Yeah sure, and I'll see if I can find some time to take a look at this tonight.

  render() {
    var scanButton = this.state.isScanning ? (
        null
      ) : (
        <TouchableHighlight style={styles.button}
            onPress={this.onScanPressed.bind(this)}
            underlayColor='#99d9f4'>
          <Text style={styles.buttonText}>Scan Now</Text>
        </TouchableHighlight>
      );

    return (
      <View style={styles.container}>
        <Image source={require('image!logo')} style={styles.image}/>
        <Text style={styles.description}>
          Scan your barcode below
        </Text>
        {scanButton}
        <Camera
          ref="cam"
          style={styles.container}
          onBarCodeRead={this.onBarCodeRead.bind(this)}
          type={this.state.cameraType}/>
        <Text style={styles.description}>{this.state.message}</Text>
      </View>
    );

and
  onBarCodeRead(e) {
    if (!this.state.isScanning) return;
    this.setState({
      isScanning: false,
      message: 'Submitting Code'
    });
    this.store.storeAndSend(e.data, function (response) {
      this.setState({ message: response });
    }.bind(this));
  }

Seems innocuous enough.
However, everything works fine for a few scans, but then I get the -[RCTCamera orientationChanged:]: message sent to deallocated instance error and the app crashes.

I am running into this issue as well. Here as simple component that reproduces the problem:
var Crash = React.createClass({
  getInitialState: function() {
    return {page: 'a'}
  },

  handleToggle: function() {
    if (this.state.page == 'a') {
      this.setState({page: 'b'});
    } else {
      this.setState({page: 'a'});
    }
  },

  render: function() {
    if (this.state.page == 'a') {
      return (
        <View style={styles.container}>
          <Text>hello</Text>
          <Button onPress={this.handleToggle}>Toggle</Button>
        </View>
      );
    } else {
      return (
        <Camera ref="cam" style={styles.container}>
          <Text style={styles.instructions}>Camera</Text>
          <Button onPress={this.handleToggle}>Toggle</Button>
        </Camera>
      );
    }
  }
});

To reproduce:

Run app on iPhone
Click "Toggle" (goes to camera)
Click "Toggle" again (goes to plain text page)
Rotate iPhone
Observe crash

You are adding an observer of NSNotificationCenter in RCTCamera setOrientation():
- (void)setOrientation:(NSInteger)orientation
{
    if (orientation == RCTCameraOrientationAuto) {
        [self.manager changeOrientation:[UIApplication sharedApplication].statusBarOrientation];
        [[NSNotificationCenter defaultCenter] addObserver:self  selector:@selector(orientationChanged:)    name:UIDeviceOrientationDidChangeNotification  object:nil];
    }
    else {
        [[NSNotificationCenter defaultCenter]removeObserver:self name:UIDeviceOrientationDidChangeNotification object:nil];
        [self.manager changeOrientation:orientation];
    }
}

I think that maybe when the RCTCamera instance is deallocated you should remove the observer?
I was able to go ahead and use react-native-camera.in my project by actually just removing that observer as my app is fixed in portrait mode anyway.


Multi-threaded gateway

Portal needs to return an IP range for tunnel instead of just a single IP, like a /30 or something. Maybe we log-in with the number of shards we want?
Gateway needs to shared state and TUN device by slicing  CG-NAT range into N slices
Run each shard in one thread, with separate UDP sockets
Broadcast all portal messages to all shareds, dropping irrelevant ones in each shard

Probably not worth it, we are getting pretty good performance with multi-threaded TUN device: #7449.


Add a basics file

Add a file that includes the basic functionalities used for the tasks (crossover, mutation,...) that are necessary for each implementation. Or add to main.
Just to avoid conflicts when merging and making sure those basic functionalities are working.
Maybe it is even possible to include a fitness function that is more general and can be specified for different tasks with new parameters?

If none of this makes sense, just ignore it :)

Added separate files for every modular part of the package.


Correct Arduino's Color, Swap MAXScript's

Correct Arduino's color to it's signature blue; Swap Arduino's old purple color to MAXScript whose color is too close. Arduino is a much more commonly used programming language than MAXScript so the color will be correct more of the time.

MAXScript should probably be changed to a lighter/darker teal variant which its heavily associated with, instead of randomly becoming purple.

Edit: Ardiuno's colour is #00979D. #056d6d could work for MS if its still too similar

👎  from me. MAXScript should stay teal, because that's the colour that's always been associated with 3DS Max. Having it purple would look random and weird.

Although MAXScript has always been teal, and for longer than Arduino (which has also always been a teal color), it would satisfy more users to have Arduino's color be correct.
A simple google search indicates the vast difference in popular use. 3DS Max returns 21,300,000 results, MAXScript returns 462,000 results, Arduino returns 62,000,000 results. Having Arduino be purple "would look random and weird" to more people than MAXScript.

Having Arduino be purple "would look random and weird" to more people than MAXScript.

You see, there's a problem with that logic: you're assuming there aren't Arduino users who are used to recognising the language by that specific shade of purple.
Sorry, but nothing about this PR feels right to me.

So the general consensus is to not change the colour of Arduino and MAXscript so I'm closing this PR.


feat: add inclusion check l1->l2

Fixes https://github.com/AztecProtocol/aztec-packages/issues/1383
Should be extended with a negative test that is faking oracle data.

Benchmark results
Metrics with a significant change:

batch_insert_into_append_only_tree_16_depth_ms (8): 21.3 (+54%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit de6e2edc and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,380
179,492
716,036

l1_rollup_calldata_gas
221,180
866,432
3,446,132

l1_rollup_execution_gas
928,952
3,900,246
24,032,603

l2_block_processing_time_in_ms
1,428
5,340 (-3%)
22,007 (-12%)

note_successful_decrypting_time_in_ms
309 (-1%)
941 (+3%)
3,417 (-6%)

note_trial_decrypting_time_in_ms
19.4 (-3%)
70.3 (+54%)
144 (-11%)

l2_block_building_time_in_ms
16,161
64,163 (+1%)
257,116 (-5%)

l2_block_rollup_simulation_time_in_ms
11,751
46,711 (+1%)
187,605 (-6%)

l2_block_public_tx_process_time_in_ms
4,374
17,362 (+1%)
69,208 (-2%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
15,939
30,194

note_history_successful_decrypting_time_in_ms
2,171 (+1%)
4,265 (+1%)

note_history_trial_decrypting_time_in_ms
119 (-27%)
181 (-1%)

node_database_size_in_bytes
3,583,989
3,651,973

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
212 (-1%)
44,681
25,561

private-kernel-ordering
166
43,753
14,809

base-rollup
1,196 (-2%)
128,338
881

root-rollup
79.0 (-2%)
4,088
677

private-kernel-inner
293
71,208
25,561

public-kernel-private-input
206
31,863
25,561

public-kernel-non-first-iteration
204
31,905
25,561

merge-rollup
7.65 (-3%)
2,608
881

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
15,455
32,219

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
377
1,087

Metric
1 public data writes

tx_sequencer_processing_time_ms
537 (-1%)

Figured that the root that is included in the oracle response is not needed anymore since we are computing it now (pedersen matches 🎉).
Because the error messages have changed, more tests needed to be updated
Added explicit tests for failure in all the assertions we got in the process_l1_to_l2_message


Added support for catching functions that will be deprecated once php7 updates

Functions include ereg, eregi, ereg_replace, and split

cc @sboisvert

Are these deprecated functions restricted by VIP?

Yup we just finished removing them from all the repos. I'll add it to the documentation early next week.

This PR raises two questions for me:

Should cross-PHP version compatibility issues be a concern which should be addressed by WPCS ?
And if so, wouldn't it make more sense to incorporate the far more comprehensive  PHP Compatibility sniffs rather than reinvent the wheel ?

Should cross-PHP version compatibility issues be a concern which should be addressed by WPCS ?
I can't speak for the project as a whole but VIP is moving to requiring PHP7 compatibility, these functions are the ones we've seen used multiple times so it was a sort of quick win.

And if so, wouldn't it make more sense to incorporate the far more comprehensive PHP Compatibility sniffs rather than reinvent the wheel ?

If the project's answer to 1 is yes then I would agree with 2.


Session API GET /sessions responds with duplicated entries for federated users

Describe the issue:
For the federated users, session-related information is persisted under two user ids (improvement of https://github.com/wso2/product-is/issues/12169). So, it'll return duplicate results (same session-id with two different user ids) when we try to retrieve the sessions (GET /users/v1/sessions) [1].
How to reproduce:

Get updated IS-5.10.0 pack (update level 65)
Configure an identity provider, service provider and add Federated Authentication for the local and outbound authentication (with jit provisioning enabled).
Login to the service provider using federated login.
Get all the active sessions using session management API [1].
request: GET /users/v1/sessions

Expected behavior:
It should not return duplicated results.
Environment information :

Product Version: [ IS 5.10.0]
Database: [H2]
Userstore: [LDAP]

[1] - https://app.swaggerhub.com/apis/jcgarcia4/wso-2_identity_server_user_session_management_api_definition/v1

Trying out with the changes proposed in https://github.com/wso2/carbon-identity-framework/pull/3959, https://github.com/wso2/identity-api-user/pull/143;
wasn't able to reproduce this in public according to the steps.

This is a regression caused by the original fix.
Not reproducible with https://github.com/wso2/carbon-identity-framework/pull/4185 or https://github.com/wso2/identity-api-user/pull/155 (yet to merge)


tab completion tooltip broken with a large font and dual monitors

With certain large font-sizes and dual monitors, the tab-completion tool-tip will show up on the wrong monitor.

Fixed in 2.3.2661.


ImageDetIter uses type list as inside making as_in_context break

Using ImageDetIter and as_in_context with batch.data[0] results in TypeError: copyto does not support type <class 'list'>

as_in_context don't support list of context.

@zhreshold  is this intended behavior?

as_in_context is used to bring data from one device to another for computation.
I am closing this issue, For general "how-to" questions, our user forum (and Chinese version) is a good place to get help.


VScode folder pane shows "Folder contains emphasized items" with marking the folder as error (red in color) but no errors on files/code

VSCode Version: 1.45.0
OS Version: Windows 7 x64

Steps to Reproduce:

Create a folder, say "models"
Create a file with hyphen in it. in my case, i wanted to create a model class file in angular with the camelcasing.
commit/push to git
remove few files and add few others like the above naming format.

ng g class doc-status --type model
so i would have my class name as
export class DocStatus {
}

And then, after i rename them back to normal class names in typescript and change, i was able to get rid of the red error like marker on that folder.
History:
I had few files and removed them later. And added few more additional files. Since then, i noticed this behavior. Renaming to normal and again restoring back, removes the folder view from error like red colored to normal folder view (yellow for modified).
Concern:
Why is this inconsistency? is this due to pending git commit/push? Ideally it should detect the local changes and react to the valid behavior. as there is no error(s) on my code/files, i would not like to see that folder colored in red if its all the pending git commit/push to reflect.

Does this issue occur when all extensions are disabled?: Yes

I'm not sure if this is a problem with the decorations in the explorer or with the diagnostics, I'm confused.

I am confused by your step, something simpler to reproduce would be helpful.
Anyways, when this happens what does the Problems panel show (F1 > Focus on problems view)? Are there error or is it empty?

Hi,
The steps could be as follows.

create a simple angular project (or any other for that trial)
Ensure to enable git source of your choice to this project.
Add or restructure the folders and files for the project.
In my case, i added a folder called "models", added "ng g class Users --type model". It will by default create the model and spec file.
commit the changes to the git, (no need to push to origin).
Now, go back to the models folder, remove the spec file.
you can see the the models folder turns to red color as if there's some error.

Please note -

there would not be any error or note in the problems tab.
I have disabled all the extensions in VS, restarted and then attempted the above steps.

Screen shots:

pending git changes shows the deleted file:

problems tab:

Output tab:

The folder view and hovering over shows the below:

I hope this helps.
Other than this msg on hovering over, saying "contains emphasized items", i dont see any reason or details for the folder to look like some error. I am assuming that the difference is only the uncommitted changes of the deleted file.
Is this is detected as some error by default? Why would vs code would determine this as some concerned item? I havent built or ran any command indeed.

Before I am diving into angular, pls try this

F1 > Log Level > Trace
F1 > Show Log > Windows
Reproduce the issue
attach the log file or all messages that start with DecorationsService#getDecoration#getOrRetrieve
share the folder name in which you have wrong highlights.

[2020-05-19 11:04:52.911] [renderer4] [trace] telemetry/activityBarAction {"viewletId":"workbench.view.explorer","action":"show"}
[2020-05-19 11:04:52.975] [renderer4] [trace] DecorationsService#getDecoration#getOrRetrieve vscode.git {"weight":3,"bubble":false,"color":"gitDecoration.ignoredResourceForeground","tooltip":null,"letter":null} false {"$mid":1,"fsPath":"c:\code\Fun\AngularDCore\ClientApp.Angular\ClientApp\dist","_sep":1,"external":"file:///c%3A/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/dist","path":"/c:/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/dist","scheme":"file"}
[2020-05-19 11:04:52.980] [renderer4] [trace] DecorationsService#getDecoration#getOrRetrieve vscode.git {"weight":3,"bubble":false,"color":"gitDecoration.ignoredResourceForeground","tooltip":null,"letter":null} false {"$mid":1,"fsPath":"c:\code\Fun\AngularDCore\ClientApp.Angular\ClientApp\node_modules","_sep":1,"external":"file:///c%3A/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/node_modules","path":"/c:/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/node_modules","scheme":"file"}
[2020-05-19 11:04:52.980] [renderer4] [trace] DecorationsService#getDecoration#getOrRetrieve vscode.git {"weight":3,"bubble":false,"color":"gitDecoration.ignoredResourceForeground","tooltip":null,"letter":null} false {"$mid":1,"fsPath":"c:\code\Fun\AngularDCore\ClientApp.Angular\ClientApp\out-tsc","_sep":1,"external":"file:///c%3A/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/out-tsc","path":"/c:/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/out-tsc","scheme":"file"}
[2020-05-19 11:05:08.639] [renderer4] [trace] CommandService#executeCommand moveFileToTrash
[2020-05-19 11:05:08.640] [renderer4] [trace] telemetry/workbenchActionExecuted {"id":"moveFileToTrash","from":"contextMenu"}
[2020-05-19 11:05:08.643] [renderer4] [trace] DialogService#confirm Are you sure you want to delete 'health-check.model.spec.ts'?
[2020-05-19 11:05:11.637] [renderer4] [trace] [text file model] load() - enter file:///c:/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/src/app/root/app.component.html
[2020-05-19 11:05:11.909] [renderer4] [trace] [File Watcher (C#)] [DELETED] c:\code\Fun\AngularDCore\ClientApp.Angular\ClientApp\src\models\health-check.model.spec.ts
[2020-05-19 11:05:11.998] [renderer4] [trace] [File Watcher (C#)]  >> normalized [DELETED] c:\code\Fun\AngularDCore\ClientApp.Angular\ClientApp\src\models\health-check.model.spec.ts
[2020-05-19 11:05:13.651] [renderer4] [trace] [File Watcher (C#)] [CHANGED] c:\code\Fun\AngularDCore\ClientApp.Angular\ClientApp\src\models
[2020-05-19 11:05:13.717] [renderer4] [trace] [File Watcher (C#)]  >> normalized [CHANGED] c:\code\Fun\AngularDCore\ClientApp.Angular\ClientApp\src\models
[2020-05-19 11:05:17.051] [renderer4] [trace] DecorationsService#getDecoration#getOrRetrieve vscode.git {"weight":1,"bubble":true,"color":"gitDecoration.deletedResourceForeground","tooltip":"Deleted","letter":"D"} true {"$mid":1,"fsPath":"c:\code\Fun\AngularDCore\ClientApp.Angular\ClientApp\src","_sep":1,"external":"file:///c%3A/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/src","path":"/c:/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/src","scheme":"file"}
[2020-05-19 11:05:17.056] [renderer4] [trace] DecorationsService#getDecoration#getOrRetrieve vscode.git {"weight":1,"bubble":true,"color":"gitDecoration.deletedResourceForeground","tooltip":"Deleted","letter":"D"} true {"$mid":1,"fsPath":"c:\code\Fun\AngularDCore\ClientApp.Angular\ClientApp\src\models","_sep":1,"external":"file:///c%3A/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/src/models","path":"/c:/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/src/models","scheme":"file"}
[2020-05-19 11:05:17.057] [renderer4] [trace] DecorationsService#getDecoration#getOrRetrieve vscode.git {"weight":1,"bubble":true,"color":"gitDecoration.deletedResourceForeground","tooltip":"Deleted","letter":"D"} true {"$mid":1,"fsPath":"c:\code\Fun\AngularDCore\ClientApp.Angular\ClientApp\src","_sep":1,"external":"file:///c%3A/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/src","path":"/c:/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/src","scheme":"file"}
[2020-05-19 11:05:17.371] [renderer4] [trace] MainThreadTimeline#emitChangeEvent: id=git-history, uri=undefined
[2020-05-19 11:05:32.846] [renderer4] [trace] DecorationsService#getDecoration#getOrRetrieve vscode.git {"weight":1,"bubble":true,"color":"gitDecoration.deletedResourceForeground","tooltip":"Deleted","letter":"D"} true {"$mid":1,"fsPath":"c:\code\Fun\AngularDCore\ClientApp.Angular\ClientApp\src\models","_sep":1,"external":"file:///c%3A/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/src/models","path":"/c:/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/src/models","scheme":"file"}
[2020-05-19 11:05:39.027] [renderer4] [trace] DecorationsService#getDecoration#getOrRetrieve vscode.git {"weight":1,"bubble":true,"color":"gitDecoration.deletedResourceForeground","tooltip":"Deleted","letter":"D"} true {"$mid":1,"fsPath":"c:\code\Fun\AngularDCore\ClientApp.Angular\ClientApp\src\models","_sep":1,"external":"file:///c%3A/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/src/models","path":"/c:/code/Fun/AngularDCore/ClientApp.Angular/ClientApp/src/models","scheme":"file"}
[2020-05-19 11:05:39.041] [renderer4] [trace] telemetry/workbenchActionExecuted {"id":"workbench.files.openFile","from":"explorer"}
[2020-05-19 11:05:39.042] [renderer4] [trace] telemetry/editorOpened {"typeId":"workbench.editors.files.fileEditorInput","resource":{"mimeType":"text/x-typescript, text/plain","scheme":"file","ext":".ts","path":1820148836}}
[2020-05-19 11:05:39.043] [renderer4] [trace] telemetry/workbenchActionExecuted {"id":"workbench.files.openFile","from":"explorer"}

I'll leave to the experts to comment on, but i think the color coding of the folder is just showing us that, there is something deleted content out there for git (like yellow for modified / M). Am i right?

Thanks. This is helpful!

It seems that deleting a file shouldn't bubble up to parent folders because it doesn't explain itself well enough

using these steps (also to verify)

have a folder foo
create two files bar and delme
commit changes so that the git viewlet is clean
in the terminal (!) type rm foo/delme
:bug: the foo-folder shows that it has emphasised items

I tested on the insiders version and it works as expected. Thanks for the quick fix. :) A great tool and a great team! 👍


setacq fails on Inova after installing OpenVnmrJ

Describe the bug
After the install script runs setacq (or manually running setacq) the console won't bootup after pressing the reset button.
To Reproduce
Inova 300 MHz spectrometer has been running VnmrJ 4.2A (latest patch) for years.
Installed OpenVnmrJ 2.1 (Jan 2020 release), told it to configure as a spectrometer.
When the install script reaches setacq (or if manually running setacq) the console starts to reboot, but never finishes (the status light on the magnet regulation board remains red).
Because of this, OVJ claims it's a datastation and won't operate the spectrometer.
Work-around:
If I run /home/vnmrj_4.2_A/bin/setacq then everything works as expected and the console is able to boot up.
OVJ appears to work fine after using the old setacq.  I don't know if this is dangerous or not recommended.
Expected behavior
setacq should work as it normally does.  After being instructed to reboot the spectrometer, the spectrometer should boot up all the way until showstat can successfully show the spectrometer status.
Version of OpenVnmrJ you are using.

Downloaded from github in Aug.  Release date in the package says Jan 2020
Unzipped, ran load.nmr script

Environment (please complete the following information):

RedHat 6.10

Additional context
Note: It appears to work fine on our DD 500 MHz instrument (same OS, same VnmrJ, but DD version, etc.)

It is not dangerous to use the VJ42 setacq. The console software in VJ42 + 110 patch is the same as that in OVJ.
Can you tell what is different when you use the two setacqs. Do they both select the same NIC card (eth0 vs eth1)? Do they both select the same IP address (10.0.0.x vs 172.16.0.x)? Do they both use the same tftpboot directory (/tftpboot vs /var/lib/tftpboot)? Are the permissions of the files in the tftpboot directory the same? It is /etc/bootptab file the same? Does the OVJ version of setacq ever give the message "NMR Console software installation complete"?

Dan,
When running the two different setacqs, the only difference I see in the text output is the following:

original setacq) It gives the option of doing the blink test (which I skip, because it's set correctly).
openVJ setacq) Doesn't do the blink test, but appears to look at the settings of eth0 and eth1 to decide which is correct.  It correctly chooses eth1 on my setup.
Yes, they both report using <IP_ADDRESS> with eth1.

They both appear to be using /var/lib/tftpboot

original setacq) It looks for the existence of a /var/lib/tftpboot dir, which exists on this instrument.
openVJ setacq) It doesn't appear to look for the existence of /var/lib/tftpboot, but it does look at /etc/xinet.d/tftp and that file has server_args = -s /var/lib/tftpboot
There does not appear to be a environment variable setting $tftpdir (at least echo $tftpdir doesn't return anything), so it looks like they're both pointing to /var/lib/tftpboot.
Everything in /var/lib/tftpboot is owned by root:root, most of it just symbolic links.  The dir vxBoot is 755
Within vxBoot, vxWorks and vxWorksPPC are owned by vnmr1:nmr, but vxWorks.auto is owned by root:root.  They're all 755
/etc/bootptab is own by root:root and is 644

Yes, the openVJ reports "NMR Console software installation complete".  Note, however, with the original setacq, it says this before the data transfer lights start blinking on the NVME board.  With the openVJ setacq, no data transfer lights are seen after this message appears (see below).

The real difference that I see is watching the status lights on the NVME and sample regulation boar after the reset + enter step.

original setacq) After a couple seconds (i.e. after the "installation complete" message) I can see lots of fast blinking of the lights on the VNME board corresponding to data transfer, after which the magnet regulation board status light goes from red to green followed by the lightshow.
openVJ setacq) There are never any signs of data transfer on the NVME board, even after several minutes. The magnet regulation board status remains at red and no lightshow happens.
Note: Dennis Edwards suggested doing a full power cycle to ensure there wasn't any corrupt files in the spectrometer messing things up (apparently he's seen that on the Inova consoles before).  However, after power cycling and then running the new setacq I still see the same behavior, whereas the original setacq transfers the files and then things work.

Again, this is only my Inova 300 MHz doing this.  The DirectDrive 500 MHz worked fine.

-Scott

Dr. Scott Burt
Teaching Professor
NMR Facility Manager
Department of Chemistry and Biochemistry
Brigham Young University
C414 BNSN / C008A BNSN
Provo, Utah  84602-5700
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
chembio.byu.edu/nmr-facility https://chembio.byu.edu/nmr-facility
email<EMAIL_ADDRESS>
On Sep 18, 2020, at 9:32 AM, Dan Iverson<EMAIL_ADDRESS>wrote:
It is not dangerous to use the VJ42 setacq. The console software in VJ42 + 110 patch is the same as that in OVJ.
Can you tell what is different when you use the two setacqs. Do they both select the same NIC card (eth0 vs eth1)? Do they both select the same IP address (10.0.0.x vs 172.16.0.x)? Do they both use the same tftpboot directory (/tftpboot vs /var/lib/tftpboot)? Are the permissions of the files in the tftpboot directory the same? It is /etc/bootptab file the same? Does the OVJ version of setacq ever give the message "NMR Console software installation complete"?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/OpenVnmrJ/OpenVnmrJ/issues/419#issuecomment-694936142, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGI4R6IE2BVZGLKFPLFKB3LSGN4PFANCNFSM4RQWRA2A.

All that the setacqs do is setup Linux system files to handle the console software download, so there must be a difference somewhere. With a functioning system, could you make a copy of /etc/bootptab. Then run setNIC, which will have the option for a blink test. Then run OVJ setacq. Are the bootptab files before and after the OVJ setacq identical? Finally, could you do the following.
script log
su
tcpdump -i eth1
Then reboot the console. After a minute or so, go back the the terminal and cntrl-c to exit the tcpdump command,
cntrl-d to exit root account, and cntrl-d to quit the script command. Then send me the log file.

Here's the results:

Running setNIC correctly blinks eth0 (which is connected to the wall), but when it goes to blink eth1 it throws an error saying it couldn't identify the NIC.

Running diff on copies of /etc/bootptab before and after setacq show the files are identical.

I ran script log twice:
-after running the old VnmrJ setacq so the spectrometer is working before logging and pressing reset: log_oldsetacq
-after running the new VnmrJ setacq so the spectrometer is not working before logging and pressing reset: log_newsetacq

-Scott

Dr. Scott Burt
Teaching Professor
NMR Facility Manager
Department of Chemistry and Biochemistry
Brigham Young University
C414 BNSN / C008A BNSN
Provo, Utah  84602-5700
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
chembio.byu.edu/nmr-facility https://chembio.byu.edu/nmr-facility
email<EMAIL_ADDRESS>
On Sep 18, 2020, at 10:56 AM, Dan Iverson<EMAIL_ADDRESS>wrote:
All that the setacqs do is setup Linux system files to handle the console software download, so there must be a difference somewhere. With a functioning system, could you make a copy of /etc/bootptab. Then run setNIC, which will have the option for a blink test. Then run OVJ setacq. Are the bootptab files before and after the OVJ setacq identical? Finally, could you do the following.
script log
su
tcpdump -i eth1
Then reboot the console. After a minute or so, go back the the terminal and cntrl-c to exit the tcpdump command,
cntrl-d to exit root account, and cntrl-d to quit the script command. Then send me the log file.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/OpenVnmrJ/OpenVnmrJ/issues/419#issuecomment-694976848, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGI4R6KJICKVVULHUF2QF53SGOGM7ANCNFSM4RQWRA2A.

There are no log files attached. Also, can you compare /etc/sysconfig/network-scripts/ifcfg-eth1 files for a working and non-working system. Also, compare results of running  /sbin/ifconfig -a for working and non-working system.

That's odd, my email software claims they were attached...  Trying again.  If this doesn't work, I guess I can try adding an extension to the files?

-Scott

On Sep 21, 2020, at 11:06 AM, Dan Iverson<EMAIL_ADDRESS>wrote:
There are no log files attached. Also, can you compare /etc/sysconfig/network-scripts/ifcfg-eth1 files for a working and non-working system. Also, compare results of running /sbin/ifconfig -a for working and non-working system.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/OpenVnmrJ/OpenVnmrJ/issues/419#issuecomment-696246588, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGI4R6NUTNTSDMDWFUFPQH3SG6BZ7ANCNFSM4RQWRA2A.

Still no attachment. You can send them to me at<EMAIL_ADDRESS>


Possible to watch src/images to trigger when images added/removed?

Curious if it is possible to add a watch task to watch the src/images directory and trigger gulp images when changes occur (add, edit, delete image)

Absolutely. It's fixed in the 2.0 branch using the gulp-watch plugin. There's a whole slew of changes/improvements in there. It's starting to settle, and should be merged in the next couple weeks.

Hmm, I thought I was using the 2.0 branch already, but maybe not. I will have a closer look tomorrow.
Lots of good stuff in there, promoted me to refactor my existing workflow to follow the basic construct you have
Also, like to "trickery" of the gulpfile.js folder with all the core stuff in the folder.

:+1: Yeah that gulpfile.js folder is fun. Definitely let me know if new files aren't being picked up. The should be.

Working for me in the latest (merging soon)


Custom Kamelet Performance Issue Fix #731

Fix for #731

We were facing performance issues when working with Custom Kamelet files, so we changed few things to improve the performance
Added a separate component called KameletInput to centralize general code changes and simplified the code.
Changed the save action from onChange to onBlur

Issue #731 affecting any Route Diagrams, not only Kamelets
Kamelet definition performance makes no impact on that performance degradation.

What we noticed is slowness and inconsistency in capturing the key inputs, not sure whether its not noticeable when the machine capacity is better.
Before fix (while typing key inputs are missed because of the lag and frequent server calls)

After fix


cub apis do not support large tensor

I was working on https://github.com/apache/incubator-mxnet/pull/19015 and have fixed the cpu version of the op. However the fix for the gpu version is blocked by some cub apis which use int in the function declaration.
One example is: https://github.com/NVlabs/cub/blob/1.8.0/cub/device/device_scan.cuh#L328
This is used here in the nonzero op:https://github.com/apache/incubator-mxnet/blob/master/src/operator/numpy/np_nonzero_op.cu#L71 and also a dozen of other operators.
Just by looking at the code there are also other int usage in other cub apis:
https://github.com/NVlabs/cub/blob/0158fa19f28619886232defd412433974af89611/cub/device/device_select.cuh#L335
https://github.com/NVlabs/cub/blob/0158fa19f28619886232defd412433974af89611/cub/device/device_run_length_encode.cuh#L155
under this folder https://github.com/NVlabs/cub/tree/0158fa19f28619886232defd412433974af89611/cub/device

@sandeep-krishnamurthy @access2rohit For your guys' reference


problem in download starter project

hello
i have problem for download starter project
my request is
404: Not Found

That's not a great experience, where did you access the link from?

We've found the source of the 404 and it should now be fixed!


Fix TSAN tests compilation

This is a workaround to tackle issue https://bugs.launchpad.net/ubuntu/+source/gcc-9/+bug/2029910. It consists of installing gcc-10 only to take an object file missing in gcc-9 (due to the mentioned bug), plus setting some cmake macros required for linking against libpthread.
Migrating to ubuntu 22 or using a higher gcc version was not a possibility as TSAN reports too many false positives for our code.
NOTE: to be reverted once the issue is solved.

Codecov Report
Patch has no changes to coverable lines.
:loudspeaker: Thoughts on this report? Let us know!.

Closed in favor of https://github.com/eProsima/dev-utils/pull/100


Systemcommand: Add debug option

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

It's sometimes necessary to control debug when using SystemCommand in certain contexts (e.g., livecheck) and this PR allows us to do this. This is also true for methods that call system_command internally (e.g., Curl#curl_with_workarounds) and we'll have to modify those to support debug, where appropriate.
This was originally part of #9535 but I'm pulling it out into a separate PR, as this change is necessary in #9529 for the same reasons (avoiding debug output from curl).  This PR also includes the commit from #9535 that updates #curl_with_workarounds to include a named debug parameter (which is passed to the system_command call).
This change may also allow us to migrate the Git livecheck strategy to system_command, instead of using Open#capture3. The technical limitations mentioned in the explanatory comment in that strategy are partly related to not being able to control debug when using system_command and this PR addresses that.
Once this is merged, I'll rebase #9535 onto master (dropping/editing the commits that we're handling in this PR instead). I also have the changes worked out to migrate the Git strategy to system_command, so I'll create a follow-up PR once I've properly tested everything.

Review period will end on 2020-12-22 at 00:00:00 UTC.

Review period skipped due to critical label.

Since this is moving forward toward merging, I'll modify this really quick to also include the commit from #9535 that updates #curl_with_workarounds to allow for a named debug parameter, as we need that in #9529, too.

CI is failing because brew audit --skip-style --tap homebrew/cask is encountering an error in a specific cask:
audit for macpilot: failed
 - Use `sha256 :no_check` when URL is unversioned.

This appears to be in relation to the else block in that cask, which is using url "https://www.koingosw.com/products/macpilot/download/macpilot.dmg".

We could also use the curl retry_request parameter from #9535 in #9529, so I'm going to drop the curl commit from this PR and handle both of these curl changes in a follow-up PR after this is merged.


MINOR: Set session timeout back to 10s for Streams system tests

We increased the default session timeout to 30s in KIP-735:
https://cwiki.apache.org/confluence/display/KAFKA/KIP-735%3A+Increase+default+consumer+session+timeout
Since then, we are observing sporadic system test failures due to rebalances taking
longer than the test timeout. Rather than increase the test wait times, we can
just override the session timeout to a value more appropriate in the testing domain.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

system test run: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/4657/

System test results: http://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2021-08-19--001.system-test-kafka-branch-builder--1629415396--vvcephei--MINOR-decrease-session-timeout-streams-systest--4947e6f130/report.html
None of the streams tests failed.

Thanks, @ableegoldman !
I agree it might be overkill, and I wouldn't fault the conservative approach.
The reason I went for a blanket approach is that it seems like 30 seconds is an awfully long time to wait for a session timeout in these scenarios, when we expect the full test run to complete within a couple of minutes. It seems like network glitches are pretty common when we run these, so I'm afraid we'll just be playing whack-a-mole by fixing individual tests as they time out due to this setting change.
Thanks for the reference to your other PR!
I just verified that we are now passing in the decreased session interval to that class via the properties file, so it should no longer be necessary to set it in the java source. I've reverted that commit in this PR, and am re-running the build just to be on the safe side. Once it passes, I'll go ahead and merge it.


Unable install on mac OS

node-pre-gyp ERR! Tried to download(404): https://node-inspector.s3.amazonaws.com/debug/v0.7.7/node-v51-darwin-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v51 ABI) (falling back to source compile with node-gyp)
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
CXX(target) Release/obj.target/debug/src/debug.o
../src/debug.cc:38:27: error: too few arguments to function call, expected at least 2, have 1
v8::Debug::Call(fn);
~~~~~~~~~~~~~~~   ^
/Users/mambig/.node-gyp/7.3.0/include/node/v8-debug.h:198:3: note: 'Call' declared here
static MaybeLocal Call(Local context,
^
../src/debug.cc:46:61: error: too few arguments to function call, single argument 'isolate' was
not specified
Isolate* debug_isolate = v8::Debug::GetDebugContext()->GetIsolate();
~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
/Users/mambig/.node-gyp/7.3.0/include/node/v8-debug.h:248:3: note: 'GetDebugContext' declared here
static Local GetDebugContext(Isolate* isolate);
^
../src/debug.cc:61:67: error: too few arguments to function call, single argument 'isolate' was
not specified
Local debug_context = v8::Debug::GetDebugContext();
~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
/Users/mambig/.node-gyp/7.3.0/include/node/v8-debug.h:248:3: note: 'GetDebugContext' declared here
static Local GetDebugContext(Isolate* isolate);
^
../src/debug.cc:66:54: error: too few arguments to function call, single argument 'isolate' was
not specified
debug_context = v8::Debug::GetDebugContext();
~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
/Users/mambig/.node-gyp/7.3.0/include/node/v8-debug.h:248:3: note: 'GetDebugContext' declared here
static Local GetDebugContext(Isolate* isolate);
^
4 errors generated.

Same error here: macOS Sierra 10.12.2 (16C67) with Node version v7.4.0, NPM v4.0.5 & 4.1.1

anyone found a way around this issue yet ?

I later realized that this project is no longer maintained by the guys who started it. Sounds like people are using other solutions now.

I have the same problem on OSX 10.11.6

Try downgrade node version to 6x
n lts

that's work for me

My workaround is to use Visual Studio Code, built in debugger works flawlessly.
Tutorial - https://hansrwindhoff.wordpress.com/2015/05/05/debugging-task-runner-tasks-like-gulp-with-visual-studio-code-editordebugger/
Just "program" path should be from $(which grunt)

Try<EMAIL_ADDRESS>please.

I can confirm that this now installs on Mac without specifying the version. Thanks guys
npm install -g node-inspector


Free the "hf" filesystem protocol for hffs

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Image render improperly in List-Rendering

I'm new here. I don't know if vue can support it.
I use v-repeat to render list. When the list shifts a new item, vue reuses dom, and rebind the data. However, image will show the old one until loaded. It is uncomfortable when the network is slow on mobile phone. I struggled to choose which way to resolve it. To develop a directive of togglling visibility around the loaded time of the image. Or vue can work around.

After shift a new item, before the new image is loaded.

Provide minimal reproduction code using JSFiddle/JSBin/Codepen. You could start with this template that already includes the latest version of Vue.

Please use the forum for such questions. IIRC, there was a thread about this exact topic.

@znck see http://jsfiddle.net/df4Lnuw6/115/. Thanks.

@cecilpeng Use key with v-for. Fixed your fiddle http://jsfiddle.net/znck/df4Lnuw6/119/.
Use forum for such questions.


K8SPS-361 update port

CHANGE DESCRIPTION
Problem:
Short explanation of the problem.
Cause:
Short explanation of the root cause of the issue if applicable.
Solution:
Short explanation of the solution we are providing with this PR.
CHECKLIST
Jira

[ ] Is the Jira ticket created and referenced properly?
[ ] Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
[ ] Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

[ ] Is an E2E test/test case added for the new feature/change?
[ ] Are unit tests added where appropriate?

Config/Logging/Testability

[ ] Are all needed new/changed options added to default YAML files?
[ ] Did we add proper logging messages for operator actions?
[ ] Did we ensure compatibility with the previous version or cluster upgrade process?
[ ] Does the change support oldest and newest supported PS version?
[ ] Does the change support oldest and newest supported Kubernetes version?

Test name
Status

version-service
passed

async-ignore-annotations
passed

auto-config
passed

config
passed

config-router
passed

demand-backup
passed

gr-demand-backup
passed

gr-demand-backup-haproxy
passed

gr-finalizer
passed

gr-haproxy
passed

gr-ignore-annotations
passed

gr-init-deploy
failure

gr-one-pod
passed

gr-recreate
passed

gr-scaling
failure

gr-security-context
passed

gr-self-healing
passed

gr-tls-cert-manager
passed

gr-users
passed

haproxy
passed

init-deploy
passed

limits
passed

monitoring
passed

one-pod
passed

operator-self-healing
passed

recreate
passed

scaling
passed

service-per-pod
passed

sidecars
passed

smart-update
passed

tls-cert-manager
passed

users
passed

We run 32 out of 32

commit: https://github.com/percona/percona-server-mysql-operator/pull/718/commits/bf2c46a6b5b6bfeecf4d3b32cbf317bf0e083f7c
image: perconalab/percona-server-mysql-operator:PR-718-bf2c46a6

Test name
Status

version-service
passed

async-ignore-annotations
passed

auto-config
passed

config
passed

config-router
passed

demand-backup
failure

gr-demand-backup
passed

gr-demand-backup-haproxy
passed

gr-finalizer
passed

gr-haproxy
passed

gr-ignore-annotations
passed

gr-init-deploy
failure

gr-one-pod
passed

gr-recreate
passed

gr-scaling
failure

gr-security-context
passed

gr-self-healing
passed

gr-tls-cert-manager
passed

gr-users
passed

haproxy
passed

init-deploy
passed

limits
passed

monitoring
passed

one-pod
passed

operator-self-healing
passed

recreate
passed

scaling
passed

service-per-pod
passed

sidecars
passed

smart-update
passed

tls-cert-manager
passed

users
passed

We run 32 out of 32

commit: https://github.com/percona/percona-server-mysql-operator/pull/718/commits/ceb376b307e7a646da8af1263352cc6be2176c3b
image: perconalab/percona-server-mysql-operator:PR-718-ceb376b3

Test name
Status

version-service
passed

async-ignore-annotations
passed

auto-config
passed

config
passed

config-router
passed

demand-backup
passed

gr-demand-backup
passed

gr-demand-backup-haproxy
passed

gr-finalizer
passed

gr-haproxy
passed

gr-ignore-annotations
passed

gr-init-deploy
passed

gr-one-pod
passed

gr-recreate
passed

gr-scaling
passed

gr-security-context
passed

gr-self-healing
passed

gr-tls-cert-manager
passed

gr-users
passed

haproxy
passed

init-deploy
passed

limits
passed

monitoring
passed

one-pod
passed

operator-self-healing
passed

recreate
passed

scaling
passed

service-per-pod
passed

sidecars
passed

smart-update
passed

tls-cert-manager
passed

users
passed

We run 32 out of 32

commit: https://github.com/percona/percona-server-mysql-operator/pull/718/commits/78614ed502a2ea84afa939a83292a2ec7e1e4e20
image: perconalab/percona-server-mysql-operator:PR-718-78614ed5


Fix typo in diagram: endpoint

📑 Description
Corrects a small typo in one of the endpoint images. Before this, one of the boxes (second from the bottom) was labeled as "Endponit" instead of "Endpoint."

@slagesse-epic, can you review this PR? (Haven't found the option to directly request review from you)


fix: re-introduce preview command enablement

fix: re-introduce preview command enablement
It looks like preview (and legacy preview) command enablement was removed from the plugin in this commit
This PR

re-adds the removed enablement clause, and
relaxes the clause so that it will work even when a markdown file is not currently open.

Pull Request Checklist
You can go to dendron pull requests to see full details for items in this checklist.
General
Quality Assurance

[~] Add a test for the new feature
[x] Make sure all the existing tests pass
[x] Do a spot check by running your feature with our test workspace
[~] If your tests involve running [[manual Testing|dendron://dendron.dendron-site/dendron.dev.qa.test#manual-testing]], make sure to update [[Test Workspace|dendron://dendron.dendron-site/dendron.dev.ref.test-workspace]] with any necessary notes/additions to perform manual test ^9jtWc6ov340p
[ ] After you submit your pull request, check the output of our integration test and make sure all tests pass

NOTE: If you running MacOS or Linux, pay special attention to the Windows output and vice versa if you are developing on Windows

Special Cases

[~] If your tests changes an existing snapshot, make sure that snapshots are updated
[~] If you are adding a new language feature (graphically visible in VS Code/preview/publishing), make sure that it is included in test-workspace. We use this to manually inspect new changes and for auto regression testing

Docs

[x] Make sure that the PR title follows our commit style
[x] Please summarize the feature or impact in 1-2 lines in the PR description
[~] If your change reflects documentation changes, also submit a PR to dendron-site and mention the doc PR link in your current PR

Special Cases
Git

[x] Make sure your branch names adhere to our commit style [[#^6zdCscSXs1MM]]

All PRs should start with [feat|fix|enhance|]/[{description-of-pr-in-kebab-case}]

eg. feat/add-thisthing

Analytics

[~] If you are adding analytics related changes, make sure the Telemetry docs are updated

Github Issue

[~] If this pull request is addressing an existing issue, make sure to link this PR to the issue that it is resolving.
[~] If the resolution comes with a document update, link the docs PR to the issue as well.

@SeriousBug Just making sure if this was intentional since I remember talking with @kevinslin about just relaxing the clause to work even when a markdown file isn't open. If so I may have not gotten the note about removing the enablement altogether 😄

What I think probably happened is that these changes weren't made in the constants file: https://github.com/dendronhq/dendron/blob/fix/reintroduce-preview-enablement/packages/plugin-core/src/constants.ts
This means the next time someone runs the gen:config task, these changes to package.json will get overwritten. The changes should be applied to the constants file first, then the package.json should be regenerated with the task.

What I think probably happened is that these changes weren't made in the constants file: https://github.com/dendronhq/dendron/blob/fix/reintroduce-preview-enablement/packages/plugin-core/src/constants.ts
This means the next time someone runs the gen:config task, these changes to package.json will get overwritten. The changes should be applied to the constants file first, then the package.json should be regenerated with the task.

Ah good catch. I'll fix that

converting to draft for the time being


text color highlights unreadable

Hello!
amazing plugin!
was wondering what would the best way be to fix unreadable text in some color themes?
In this example I'm using monokai-pro, you can see black text on the right is unreadable.

fixed:
changed highlight to comment
                highlights = { -- highlight groups
                    quickfix = "Comment",
                    refactor = "Comment",
                    source = "Comment",
                    combined = "Comment",
                    label = "Comment",
                },


Feat/issue11

Added loader when submitting a form
Added loader when fetching data for events

@robertwt7

event.id key prop added
fixed 'p' nesting issue for MyEventsPanel Status
Component Loader now centered on parent's div
Test for ComponentLoader and MyEventsPanel updated

fixes #11


Problem occurred while integrating the chatsdk with my own app

The problem occurred while integrating with my own app. The integration is done according to the steps which mentioned in chatsdk documentation. But it shows nullpiontererror, means the app crash soon after it opens and gave error (null pointer). I added all repositories and libraries which mentioned in the documentation

@nirmalbaby1113 can you provide a stack trace? And ideally fill out the issue template?

@nirmalbaby1113 we have made a large number of changes to the App in the last few months. Please can you update to the newest version and check if the problem still persists. If it does then please re-open this issue.


(sourcetree) Update license URL

Description
Update license URL
Motivation and Context
Fixes #1605
How Has this Been Tested?
URL tested in Browser. Otherwise not tested
Screenshot (if appropriate, usually isn't needed):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[ ] All files are up to date with the latest Contributing Guidelines
[ ] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

:white_check_mark: Package verification completed without issues. PR is now pending human review

:white_check_mark: Package verification completed without issues. PR is now pending human review

@pascalberger your changes have been merged, thanks for your contribution 👍
Thank you for reaching out to the support, and getting this updated so quickly

@pascalberger your changes have been merged, thanks for your contribution 👍
Thank you for reaching out to the support, and getting this updated so quickly


Increase dashboad coherence

Following the discussion in #808, this is a proposal as to how a more coherent version of the dashboard and management pages could look.
Things that were adjusted:

Dashboard item descriptions (I tried to find a compromise between the discussed suggestions)
"Manage pages?" text was removed (there's still the question if and where that text should be shown instead. @oas777  and @dagraf suggested moving this to the user page instead and I think this makes sense - perhaps we could add this to the info tooltip)
The navigation for the "dashboard", "my videos" and "upload" pages were removed. This is a little controversial but I do agree with the points from #808 (i.e. it's somewhat confusing to not have every item from the actual dashboard page present, and navigation is redundant as there are breadcrumbs as well as the user menu which basically fill the same function)

Still missing but not forgotton:

updated icon for the "my page" links.

This all is of course still subject to change, but maybe this PR-deployment can give us a better base for further discussions.

Thanks, Ole, this is a great improvement. Some remaining smaller issues:

With the navigation gone, the page is more aligned to the left than others - can we prevent this?
Three elements in the first line, one in the second line, this still bugs me. Anything we can do?

Nicely done. I like the new version.

With the navigation gone, the page is more aligned to the left than others - can we prevent this?
I would suggest to left align the title and the buttons for every wideness of the browser window. At the moment, the content of the dashboard is centered when the window is wide, and it gets kind of left aligned when the window gets narrower.

Three elements in the first line, one in the second line, this still bugs me. Anything we can do?
This depends on the wideness of your browser window. If you make it very wide all elements appear on the same line. Said that, I do not have problems with the elements spread on different line. Also because in the near future there will be additional elements, I guess (i.e., "My series").

"Manage pages?" text was removed (there's still the question if and where that text should be shown instead. @oas777  and @dagraf suggested moving this to the user page instead and I think this makes sense - perhaps we could add this to the info tooltip)

I have no strong opinion here. I would suggest, you develop it as describeed above and we can then give feedback on how it looks and feels. Ok?

Seeing as both of you have a different opinion about the alignment of the items, I made some screenshots of the suggested versions to compare, assuming I understood them correctly (keep in mind that the layouts are also applied to the "Upload" and "My videos" page, I've added smaller screenshots of these as well, you can click on them to see them in large):
(1) Olaf's suggestion: Having the content aligned as if there was an invisible navigation to the left, and using an even number of items for each row. This makes it look nicely centered and coherent with other pages that use a navigation box on the left, but

(a) as David also mentioned: there will be additional items in the future, so we can't guarantee that there will always be an even number of items
(b) While this works for the "Dashboard" page, it looks rather odd on the "Upload" and "My videos" pages, since their content is considerably wider, making them right-aligned with a large empty space on the left.

(2) David's suggestion (aligning the content left): This would actually work better than my version for the "Upload" and "My videos" pages, as they would simply fill the available width (up to a certain point at least).
I don't see any issues here, apart from the one that Olaf also noticed on my suggestion (With the navigation gone, the page is more aligned to the left than others).

If we have some time left, maybe we can talk about these and agree on one version this afternoon. @LukasKalbertodt your opinion is of course also appreciated and should be taken into account.

David, we didn't decide this, but there was a discussion about

whether the left-hand navigation should be "deleted" only in the "Dashboard" or in its sub-pages as well ("My videos" e. g.) and
the width of the pages in general and in particular ("My videos" and "Upload video").

We'll continue here later...

The problem with "pretending there is an invisible navigation" is that the navigation side bar has a fixed width, while the "main content" can stretch very close to the right screen boundary. Meaning: on some screen widths it might look good as it looks center-aligned, but on others it looks really weird.
I would also strongly suggest that we shouldn't remove the navigation on at least the sub pages. Switching from "my videos" to "my series" (in the future) or "upload video" is something fairly common I would think.

Given these two points, I'd suggest going in the different direction: always show the navigation but add the entries "my page" and "Record video" to it. I know, I argued against this before, as those are two "different kind of items":

"my page" is not technically part of the "managment area" and thus shows a different navigation:

"Record video" opens Studio in a new tab.

But thinking about it again, I think it's fine. After all, we gain more consistency from that. For the "Record video" entry I would also add the "external link" icon to make it less surprising.
Additionally, I would actually remove basically all of the content from the "main" area of the management dashboard, i.e. remove those four boxes. They are completely redundant, as we would have everything in the navigation. I think we will find some interesting data to fill this content with, but even if it's completely empty it's fine IMO.
Please compare that to the YouTube Studio, Youtube's equivalent of our "management area":

This is the start page of their "management area", i.e. the page we are talking about here. They show the main menu to jump between the most important "management sub areas". And on the main part of the start page they show... random stuff. This terrible "creator insider" videos is completely annoying, of course. Other than that they show analytics and latest comments, which is useful. I could imagine something similar for Tobira. Quick links to the three last uploaded videos for example. Maybe some analytics/statistical data in the future, if we have some. I'm sure we can come up with a few things. But until we do, I think a basically completely empty dashboard is fine too. Users will quickly learn that the navigation is the way to get to actually useful pages.
I.e. something like this for now:

+1 for adding "My page" and "Record video" to the navigation
+1 for always showing the navigation
-1 regarding empty main area. Let's revisit this once the above two have been implemented and we have some ideas what to present in this "wasteland".

I have updated this according to @LukasKalbertodt's suggestion.
A few additional notes:

While the tiles were removed for now, I have re-added the "Manage pages?" text. Even though I've also said previously that I could imagine moving this to the user page, I reconsidered and came to the conclusion that not every user that needs to "manage pages" will have their own page. The text provides valuable information and having this on the "Manage" page is pretty much the only place where it makes sense, in my opinion.
The single remaining page that doesn't have a navigation on the left is the page a user gets directed to when they visit "My Page" and don't have their own page yet. I would argue that for the sake of coherence we might also want to add the navigation there.

+1 for re-adding this. I wanted to -1 this when I read it, but it actually looks good on that "wasteland". Maybe remove the "?"?
+1
What about "adding "My page" and "Record video" to the navigation"? Later?

Maybe remove the "?"?

Makes sense, I think the questionmark is not really necessary.

What about "adding "My page" and "Record video" to the navigation"? Later?

Sorry, I didn't specifically mention this, but I've added these as well.
For reference, you can check the deployment at https://pr896.tobira.opencast.org/~manage with a signed-in user (I would suggest admin, as not every test-user is allowed to record videos).

Looks good! David?

Looks good. But we should come up with more reasonable content to fill the 'wasteland'.

Looks good. But we should come up with more reasonable content to fill the 'wasteland'.

What do you think about my suggestion of showing the latest videos of that user? I.e. the first 5 videos from the "My Videos" page. Not in the same table as "My Videos", but just showing the title and a small thumbnail (and making them link to the "manage single video" page).

PS: Or we table the discussion in line with https://github.com/elan-ev/tobira/issues/807 until we have more features to be included. I hear neither "My series" nor "My collections" actually exist, Matomo has not been integrated yet and access cannot be granted either, so nothing I said would be feasible currently.

PS: Or we table the discussion in line with #807 until we have more features to be included.

I think this would be useful. So I'm in favor of merging this as is, with only the "edit pages" note in the main area. At least we gain consistency from that. And later we can decide about what to do in the main area.

Oh well I just realized that I misunderstood your second comment. The icon did show up in the burger menu but the link wasn't styled correctly, so at first I thought that's what you meant... But I believe you meant the external link in the mobile userbox menu since that's where it is missing the icon.

Ole, is this visible at https://tobira.opencast.org/~manage already?

Yes, it is.

Thanks, Ole. I think I need some time to get used to that emptiness... but I looks calmingly clean.
One minor issue: When coming back to the dashboard tab from Studio, both "Record video" and the last other function I used are highlighted:

I think only "Record video" should be.

You are right, thanks for bringing this up.
The page the user is currently on is always highlighted, so when someone is on the "My videos" page for example, and then clicks the "Record video" link, they are technically still on the "My videos" page, so that keeps being highlighted.
I would probably change it so that "Record video" won't be highlighted when that link is clicked.

Should be fixed with https://pr925.tobira.opencast.org/~manage.


SOCKS5 proxy: Always getting root URL instead of requested URL

I am trying to use a SOCKS5 proxy. It basically works but it seems I am always getting the root website instead of the URL I requested.
So when requesting https://example.com/page1 I am getting a response as if I had requested https://example.com.
Code to reproduce:
scraper = cloudscraper.create_scraper(debug=True)
r = scraper.get('https://www.japscan.to/manga/tales-of-demons-and-gods/')

This delivers correctly that:
<!DOCTYPE html>\n<html lang="fr">\n\n\t<head>\n\t\t<title>Manhua Tales Of Demons And Gods | JapScan</title>\n\t\t<meta charset="utf-8" />\n\t\t<meta name="referrer" content="no-referrer" />\n\t\t<meta name="viewport" content="width=device-width, initial-scale=1" />\n\n\t\t<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0/css/bootstrap.min.css" integrity="sha384-Gn5384xqQ1aoWXA+058RXPxPg6fy4IWvTNh0E263XmFcJlSAwiGgFAW/dAiS6JXm" crossorigin="anonymous">\n\t\t<link rel="stylesheet" href="/css/main.css" />\n\t\t<link rel="stylesheet" href="/css/mobile.css" />\n\n\t\t<link rel="stylesheet" href="https://use.fontawesome.com/releases/v5.0.8/css/solid.css" integrity="sha384-v2Tw72dyUXeU3y4aM2Y0tBJQkGfplr39mxZqlTBDUZAb9BGoC40+rdFCG0m10lXk" crossorigin="anonymous">\n\t\t<link rel="stylesheet" href="https://use.fontawesome.com/releases/v5.0.8/css/fontawesome.css"
...

But when I add a proxy:
scraper = cloudscraper.create_scraper(debug=True)
scraper.proxies = {'https': 'socks5://<IP_ADDRESS>:6667'}
r = scraper.get('https://www.japscan.to/manga/tales-of-demons-and-gods/')

Then I get this:
<!DOCTYPE html>\n<html lang="fr">\n\t<head>\n\t\t<title>Lecture en ligne des meilleurs mangas japonais en ligne | Japscan</title>\n\t\t<meta charset="utf-8" />\n\t\t<meta name="referrer" content="no-referrer" />\n\t\t<meta name="viewport" content="width=device-width, initial-scale=1" />\n\t\t\n\t\t<link rel="stylesheet" href="https://use.fontawesome.com/releases/v5.0.8/css/solid.css" integrity="sha384-v2Tw72dyUXeU3y4aM2Y0tBJQkGfplr39mxZqlTBDUZAb9BGoC40+rdFCG0m10lXk" crossorigin="anonymous">\n\t\t<link rel="stylesheet" href="https://use.fontawesome.com/releases/v5.0.8/css/fontawesome.css"
...

Which is not the requested URL but rather the root website https://www.japscan.to.
Am doing something wrong or is this some sort of bug? Thanks for help.

@VeNoMouS
Interesting. Yes, seems to be working for you.
This is my log without proxy:
https://pastebin.com/fTe6tJ9J
And this is with proxy:
https://pastebin.com/mQGPmRPd

Yea so on the 302..
 CONNECT https://www.japscan.to/cdn-cgi/l/chk_jschl?s=24380fd58ef9e750137b903c3db34649879f6ba7-1560232791-1800-AR6%2FTlJiTURq%2F6K3zw9k9q3EunU7Gf89f2AaWFsot3LRYj5h2kuRp9SIBW3ICgrrIWo7CTe19I%2FlnobV4unELam%2Btk4lbWv7ttYH8OlN0gcs%2BKwJfRqMYeCI3JXeZyPpQyWo0tYBlJ04RPhh7YLg5KXNXrP2hUccCGQ9IDL%2BdI%2By&jschl_vc=9dea4f398effa765bc932e736ef5989c&pass=1560232795.792-RFf4%2B3Ta2v&jschl_answer=-8.9616786750 HTTP/1.1
< Host: www.japscan.to
< User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:55.0) Gecko/20100101 Firefox/55.0
< Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
< Accept-Language: en-US,en;q=0.5
< Accept-Encoding: gzip, deflate, br
< Connection: keep-alive
< Upgrade-Insecure-Requests: 1
< Cookie: __cfduid=d208469907b7d5fd56ae8ddd54475cd671560232791
<

> HTTP/1.1 302 Moved Temporarily
> Date: Tue, 11 Jun 2019 05:59:59 GMT
> Content-Type: text/html
> Content-Length: 159
> Connection: keep-alive
> Location: /
> Server: cloudflare
> CF-RAY: 4e515cb2bb5ac761-AMS
> X-Frame-Options: SAMEORIGIN
>

it's changing the Location on you to /

I'll see if i replicate what is going on and try to work out why its redirecting you away...

Sorry, I just noticed that you wanted the logs to be emailed. I hope the pastebin way was ok also.

Yea paste is fine, most people seem to want to keep their requests private (not sure why) so its just easier to say hey email... otherwise you never get them

@verybadsoldier can you show me a couple raw debugs with fresh create_scraper()'s

You mean running this demo program a couple of times?
scraper = cloudscraper.create_scraper(debug=True)
scraper.proxies = {'https': 'socks5://<IP_ADDRESS>:6667'}
r = scraper.get('https://www.japscan.to/manga/tales-of-demons-and-gods/')

yea , i just wanna see if its mozilla user-agent causing it, or if its a random thing going on.

Just to make sure that I don't misunderstand you:
I should run the exact same code that I ran before to provide my latest log, yes?
If yes then, tbh, I do not expect a different outcome and would not expect a change in the user-agent header. But sure, I am not the expert and I will provider whatever is helpful to you.
Sorry if I got something wrong.

@verybadsoldier what I think is happening ... and tbh its not my code this is cloudflare, is due to you failing the first challenge response... the second challenge takes the failed solve as the the base url , technically you shouldnt fail the first not sure why you did... have you tried a different interpreter instead of js2py?
Anyway... I guess a resolve could be.. in place of https://github.com/VeNoMouS/cloudscraper/blob/master/cloudscraper/init.py#L235
        redirect_location = urlparse(resp.url)
        if not redirect_location.netloc:
            redirect_url = urlunparse(
                (
                    parsed_url.scheme,
                    domain,
                    redirect_location.path,
                    redirect_location.params,
                    redirect_location.query,
                    redirect_location.fragment
                )
            )
            return self.request(resp.request.method, redirect_url, **original_kwargs)

        return self.request(resp.request.method, resp.url, **original_kwargs)

And use the force the code back onto the original requests URL before the challenge was issued, while not ideal.. I can understand what has happened

@verybadsoldier how'd you go?

Anyway... I guess a resolve/workaround could be.. in place of https://github.com/VeNoMouS/cloudscraper/blob/master/cloudscraper/init.py#L235-L249
        redirect_location = urlparse(resp.url)
        if not redirect_location.netloc:
            redirect_url = urlunparse(
                (
                    parsed_url.scheme,
                    domain,
                    redirect_location.path,
                    redirect_location.params,
                    redirect_location.query,
                    redirect_location.fragment
                )
            )
            return self.request(resp.request.method, redirect_url, **original_kwargs)

        return self.request(resp.request.method, resp.url, **original_kwargs)

And force the code back onto the original requested URL before the challenge was issued and not use CF's 302 redirect Location header, while not ideal.. I can understand what has happened.

Yay, I changed the code and this is working nicely, thanks!!

Technically you shouldn't fail in a normal browser... have you tried a different interpreter instead of js2py? more information regarding different supported javascript interpreters can be found at https://github.com/venomous/cloudscraper/#javascript-interpreters

No, I did not try yet. But looks interesting.
So, is this a code change you would consider to keep that way or was it more like a workaround? I would be happy to test further stuff when you need something (e.g. the javascript interpreter?).
Thanks!

I'll get my test group to test it more as, as i said i dont think theres any real side affects but we are basicly ignoring the 302 ... so ..... yeaaaaa


Feature/home page text

merge

good

 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.:x: tylerdesjardins sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


riscv/common: add CURRENT_REGS declare in RV32

Summary
riscv/common: add CURRENT_REGS declare in RV32
Impact
N/A
Testing
CI-Check is enough

The change is very simple, let's ignore the false CI failure.


Changed from chris-lea ppa repo to using nodesource which replaced it.

Apparently Chris Lea has gone to work at nodesource and that has changed the way to install node.js on Ubuntu. See second paragraph for Debian and Ubuntu on [[Installing Node.js via package manager|https://github.com/joyent/node/wiki/Installing-Node.js-via-package-manager#debian-and-ubuntu-based-linux-distributions]]. Further instructions under the [[nodesource/distributions|https://github.com/nodesource/distributions]] repo.
So I changed the Dockerfile to install curl and then use the new instructions. I was able to create a new TW and browse to it.

And sorry for the links in TW instead of Markdown format - the dangers of working with both! :)

Thx! Will do some cleanup, too.