autodiff recover_memory segfault

From @fredRos on June 3, 2015 10:20
Bug Report
Version
stan: 822ab42, system: debian jessie, clang version 3.5.0-10
Description
segfault with autodiff library in 2nd call to recover_memory, follow up to #1475
Steps
Minimal example compiled with
clang++ -std=c++14 -Wall -pedantic -isystem ${STANHOME}/src -isystem ${STANHOME}/lib/eigen_3.2.4 segfault.cpp
#include <stan/math/prim/arr/meta.hpp>
#include <stan/math/prim/mat/meta.hpp>
#include <stan/math/prim/scal/meta.hpp>
#include <stan/math/rev/scal/meta.hpp>
#include <stan/math/prim.hpp>
#include <stan/math/rev.hpp>

int main() {
    stan::math::var x = 1.0, mu = 0.0, sigma = 1.0;

    stan::math::var z = normal_log(x, mu, sigma);
    z.grad();
    // no segfault if first recover_memory commented out
    stan::math::recover_memory();

    z = normal_log(x, mu, sigma);
    z.grad();
    // the culprit
    stan::math::recover_memory();

    return 0;
}

Output
Segmentation fault
Copied from original issue: stan-dev/stan#1478

@fredRos, I see that it seg faults. That's known behavior.
I tried printing it, but it wasn't clear that it was uninitialized. I don't think there's a way to have that be a safe behavior without a lot of checking within the internals of the Stan Math library, so I think it'll stay this way. Do you have suggestions on how to fix the documentation?
I will go through another pass once we solidify our library so it may be taken care of without any input. But, if you have a good way to explain why this is happening, then that's great.
To address what you're trying to do, put the initialization in a function.

That seg fault falls into using an uninitialized var.  After the first call to recover_memory, all the vars are uninitialized. They've all been recovered.
To verify, print x directly after the call to recover memory.
If you want to do what you are intending, you need to recreate x, mu, and sigma again after calling recover memory.
On Wednesday, June 3, 2015, Frederik Beaujean<EMAIL_ADDRESS>wrote:

Bug Report
Version
stan: 822ab42
https://github.com/stan-dev/stan/commit/822ab42d07db58e617e5682a0ec4013626ba9c87,
system: debian jessie, clang version 3.5.0-10
Description
segfault with autodiff library in 2nd call to recover_memory, follow up
to #1475 https://github.com/stan-dev/stan/issues/1475
Steps
Minimal example compiled with
clang++ -std=c++14 -Wall -pedantic -isystem ${STANHOME}/src -isystem
${STANHOME}/lib/eigen_3.2.4 segfault.cpp
#include <stan/math/prim/arr/meta.hpp>
#include <stan/math/prim/mat/meta.hpp>
#include <stan/math/prim/scal/meta.hpp>
#include <stan/math/rev/scal/meta.hpp>
#include <stan/math/prim.hpp>
#include <stan/math/rev.hpp>
int main() {
stan::math::var x = 1.0, mu = 0.0, sigma = 1.0;
stan::math::var z = normal_log(x, mu, sigma);
z.grad();
// no segfault if first recover_memory commented out
stan::math::recover_memory();

z = normal_log(x, mu, sigma);
z.grad();
// the culprit
stan::math::recover_memory();

return 0;

}
Output
Segmentation fault
—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/stan/issues/1478.

I'm closing this. It's known behavior and isn't a bug. It's chasing a pointer that it doesn't own after the first call to stan::math::recover_memory().


Polymer 1.0

There is some difference with the transition animation in neon-animated-pages and the cyclePages function but it work well.
But I have a problem with the element iron-a11y-keys maybe related to this issue
https://github.com/dart-lang/polymer_elements/issues/70
Impossible to register a callback 'on-keys-pressed', but maybe I don't understand how it work.
Icons in the left menu are not updated when an item is selected, I don't know why.

Sorry I apparently was not watching this repo so I didn't see the pull request, looking now.

Ok, I went ahead and just made some edits and sent https://github.com/lejard-h/polymer-spa-example/pull/1. Unfortunately I am going on vacation tomorrow, but we can pick this back up when I get back (10/20)

Ok thanks.
I saw this yesterday https://github.com/dart-lang/polymer-dart/issues/524
Should we continue to upgrade the SPA example or should we directly port the starter kit.
Both apps are not completely different but maybe it's a good things to have two example of creating an app in Polymer, I don't know. But I can starting the upgrade of the starter kit.

btw I ported the polymer layout to dart.
I plan to give a way to configure all those layout to create simple apps.
https://github.com/lejard-h/polymer_app_layout_templates
This could be used in the SPA example or in the starter kit, but right know it's just an idea, I don't know how I will do that.


testing: enable the Build Cop Bot

Instructions here: https://github.com/googleapis/repo-automation-bots/tree/master/packages/buildcop

I installed the bot on the repo (first step of the instructions). We still need to call the buildcop binary to publish the test logs.

https://github.com/GoogleCloudPlatform/ruby-docs-samples/pull/504


Update logo.svg

I changed the clique logo to the one that appears on their site
Description

Backward compatibility

Testing

these are different clique projects.
in the registry we have: https://www.myclique.io/
the one you are changing to is https://www.clique.tech/

ok sorry


Pubspec.yaml fixes

move logger as it is a non-dev only dependency
add override for path, so that pub get works

Why do you need the logger out of dev dependencies?

Because logger is stilled called in the production/release code, even if _LoggerClient is not setup when in release mode

I am on beta channel, and removing path does not work. However, removing the darts override works when the path override is present


onMonthChange not firing

Describe the bug
onMonthChange not firing by clicking on "day--outside-month"
To Reproduce

Configure inline datepicker ( in my case: monthsShown={3} )
Click on the outside month date
Month has changed, but "onMonthChange" did not fired

Expected behavior
Main: fire onMonthChange
Other:
1st case: Don't change month at all.
2nd case: want to show the dates, but with prevented onClick event (not just hide)
3rd case: onOutsideDayClick event
4th case: Add hooks! =)
Desktop (please complete the following information):

OS: macOs Catalina 10.15.3
Browser: chrome
Version: 80.0.3987.149

One of the solutions (prevent onClick):
.react-datepicker__day--outside-month {
    pointer-events: none;
}

This is a real bug. When you change the month using month controls, the 'onMonthChange' event fires.
But when you cause the month to advance by either clicking on a date outside the main displayed month (like March 1 when you're looking at February), or by using an arrow key to navigate to the next or previous date, the event does not fire. Seems like it should. I would like to know when the displayed month changes so I can fetch new data for it.


10.0.19045.4894 on Windows 10 22H2

I can't find any ini file info, can anyone got the same winver 10.0.19045.4894 for rdpwrap.ini

Please, attach your RDPConf screenshot and termsrv.dll.

termsrv.zip

Please, attach your RDPConf screenshot and termsrv.dll.

termsrv.zip

https://github.com/stascorp/rdpwrap/issues/2840
I replaced termsrv.dll according to the content in this article and it can be used

I think it's better to modify the same termsrv.dll file then to replace it,  will be more stabler?

I think it's better to modify the same termsrv.dll file then to replace it, will be more stabler?

I think so. but it seems like no support version 10.0.19045.4894, it's too new

#2840
I replaced termsrv.dll according to the content in this article and it can be used

Can you please give link to download ini file?


Fix routes generation on windows

Closes https://github.com/udibo/react-app/issues/89 if it works. I was getting an esbuild error locally still but I at least got it so that the _main.tsx and _main.ts build artifacts get generated correctly.

I fixed the build script on windows by doing the following:

Always use @std/path instead of @std/path/posix.
Replace \ with / in code generation functions.
Make entryPoint path relative


Compat data is not readable when the name is too long without space

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details
See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date#Browser_compatibility for example. This is how the table look like:

The end of toLocaleTimeString.locale and toLocaleTimeString.options is hidden with no way to reveal it (and for this case, this makes them indistinguishable as the last visible char is the dot).
Either these should wrap (dots could be considered as a line breaking point maybe) or the column should become bigger to make them visible.

This is a Kuma CSS issue, I would say.


Find function not working with arabic word

Attach (recommended) or Link to PDF file here:
https://drive.google.com/file/d/1ZsfWYWts2AeKPokOAPMbnAi4-9MTIa01/view?usp=sharing
Configuration:

Web browser and its version: all browser
Operating system and its version: all operating system
PDF.js version: latest version
Is a browser extension:

Steps to reproduce the problem:

write a word on the find field in arabic

What is the expected behavior? (add screenshot)
What went wrong? (add screenshot)
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Attach (recommended) or Link to PDF file here:
https://drive.google.com/file/d/1ZsfWYWts2AeKPokOAPMbnAi4-9MTIa01/view?usp=sharing

None of the PDF viewer I've tried, which includes Adobe Reader (i.e. the reference implementation), is able to successfully copy and/or search text in that PDF document.
The problem is that none of the fonts in the PDF document contain any meaningful ToUnicode data, hence this is a limitation of the PDF document itself rather than the PDF.js library and it's thus unfortunately not possible to fix that here.

Thanks for your responce
I used another pdf file writen in arabic so when i type the word in arabic with the keyboard the pdf reader can't find the word but when i copy it from the pdf  the pdf reader find the word so i thing it's a problem of encoding

The problem reported in https://github.com/mozilla/pdf.js/issues/11321#issue-521402686 isn't actionable, for the reasons explained in https://github.com/mozilla/pdf.js/issues/11321#issuecomment-552815512, and this issue is thus invalid.

What's described in https://github.com/mozilla/pdf.js/issues/11321#issuecomment-552828520 might be a totally different problem, however please note https://github.com/mozilla/pdf.js/blob/master/.github/CONTRIBUTING.md (emphasis mine):

The issues are used to track both bugs filed by users and specific work items for developers. Try to file one issue per problem observed. Please specify a valid title (e.g. "Glyph spacing is incorrect" instead of "PDF.js does not work") and provide more details about the issue: link to the PDF, location in the PDF, screenshot, browser version, operating system, PDF.js version and JavaScript console warning/error messages. Issues that do not have enough details provided will be closed as invalid/incomplete.


only update changed workspace names

I had some problems with rhythmbox (music player that displays the song title as window title) stealing my focus on track change. This fixed it and should reduce the amount of commands needing to be sent.
Seems like a simple change to me, but tell me if I didn't take something important into consideration.

Thanks for this.
It's interesting - I didn't know of anything that changed the window name like that. I actually couldn't reproduce it with rhythmbox 3.4.2 and i3 4.14.1 - what versions are you using?
I think your change is fine - I'm going to run with it for a  few days as an ad-hoc UAT test and merge unless something looks odd.

Thank you for your answer, here a more detailed explaination:

i3 version: 4.16.1
rhythmbox version: 3.4.3

Steps to reproduce:

Open rhythmbox on a workplace on a secondary monitor
Focus any other application on a workspace on the main monitor
Move your mouse away from the center (in order to see it jumping back to the center later)
Skip a couple of tracks in rhythmbox random mode via keyboard shortcuts, so that rhythmbox has to skip to a track that currently is not visible.
The focus will change to rhythmbox for a fraction of a second, and then back to the main monitor, which makes the mouse jump back to the center of the screen

Same thing sometimes happens when using firefox and opening new tabs, although I can not reproduce the issue there every time. I supsect that the different website titles are involved somehow.
After the change  both issues seem to be fixed.


Mangrobangit/hypothesis four

@MangrobanGit

Want to pull in files re: fixing player table.
see data/player_table_fix.md for instructions.
Make sure to reset to master after this merge goes through, because depends on file data/export_player_from_sqlite.txt

Make it so.


web-search.plugin.zsh: add stackoverflow searches for common languages

Not sure if desirable, but could come in handy, and show off a bit more of the usefulness of the plugin.

Hi @vitorgalvao, thank you for that, I didn't know this functionality of duckduckgo. That said, I wouldn't put all those aliases in it. Maybe just define the one stackoverflow alias and put the rest as examples in a README for the web-search plugin (which is still pending to be written).
Also, keep in mind that the frontend-search already has a stackoverflow alias but it uses the stackoverflow website directly.
Thanks again

That sounds more like the appropriate place.


Replace SDL_CreateRGBSurface by SDL_CreateRGBSurfaceWithFormat

Replace SDL_CreateRGBSurface by SDL_CreateRGBSurfaceWithFormat
so that it will be easier to move to SDL3

Yep!


[Config Linux] Clarify: App --> Container

Replaces #577
Signed-off-by: Rob Dolin (MSFT<EMAIL_ADDRESS>

LGTM

I have no idea why git-validation is looking at these ancient commits
1.

@vbatts PTAL

LGTM


Changing hypens to underscores

[x] I have added the tests to cover my changes.
[x] I have updated the documentation and CHANGELOG accordingly.
[ ] I have read the CONTRIBUTING document.

@wjcunningham7 The push actions for develop and master look good now. Please let me know if i missed something.


Add Leptos Alert

Closes #62.
Added Alert feature to Leptos and also added examples in book-example.

To make it easier to review, I have implemented the requested changes in separate commits.


bin/BrowserSystem.js includes compiler, polyfills, System, and WebPag…

Now includes compiler, polyfills, System, and WebPageTranscoder.
Read traceurOptions attribute from

This add two large files to our npm module (BrowserSystem.js and traceur.js.map), but it greatly simplifies the instructions for browser.  I think we could load the compiler dynamically into BrowserSystem, but I don't know that this complication is worthwhile to save 1MB.

LGTM
I'm not sure what's up with the cla bot?
Also, can you provide a good commit message. The current one is not as good as your comment.

I think the github UI was "improved".  The PR description is now spread over three parts, the PR title, the PR description, and the first commit.  Combined it has the same content as the commit. I'm sure someone thinks it makes sense.
I'll look into the CLA snafu, I guess its because my legacy email address confuses it.

FWIW, I gave up on pulley. Now that github can squash the commits we are
better off just cleaning up the commit message using the web ui.
On Sun, Apr 10, 2016, 10:58 johnjbarton<EMAIL_ADDRESS>wrote:

Closed #2103 https://github.com/google/traceur-compiler/pull/2103 via
412a09e
https://github.com/google/traceur-compiler/commit/412a09e2f3728e5ddf66c273ed83bac577d00b88
.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/google/traceur-compiler/pull/2103#event-621591505


dnf instead of yum?

hi,
maybe worth to update yum do dnf?

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

ID: e7895b9a-9a6a-cd98-6947-0e4a26eda9a3
Version Independent ID: a3f27295-3e2e-a36d-02c2-8897295fc443
Content: Install the Azure CLI for Linux manually
Content Source: docs-ref-conceptual/install-azure-cli-linux.md
Service: azure-cli
GitHub Login: @dbradish-microsoft
Microsoft Alias: dbradish

@AwdotiaRomanowna Thanks for the question. We're investigating this and will get back to you shortly.

@AwdotiaRomanowna Thanks for the question. We're investigating this and will get back to you shortly.

@AwdotiaRomanowna  I am further checking with my team on this. I'll keep you posted on the updates.

@AwdotiaRomanowna  I am further checking with my team on this. I'll keep you posted on the updates.

@AwdotiaRomanowna  -
Thanks for bringing this to our attention.  We are in plans to update the document to reflect dnf as a package manager to install Azure CLI.  There will be considerable time involved to go through internal reviews and finally reflect in the public documentation. Rest assured, there are steps taken towards updating document with dnf.
Having said that, we will proceed with the closure of this thread provided you don't have further questions while we continue to work on your feedback at our end.

@AwdotiaRomanowna  -
Thanks for bringing this to our attention.  We are in plans to update the document to reflect dnf as a package manager to install Azure CLI.  There will be considerable time involved to go through internal reviews and finally reflect in the public documentation. Rest assured, there are steps taken towards updating document with dnf.
Having said that, we will proceed with the closure of this thread provided you don't have further questions while we continue to work on your feedback at our end.


Put backend tests into folders

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 5.26%. Comparing base (f80aab9) to head (a0f4875).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #292       +/-   ##
==========================================
- Coverage   96.66%   5.26%   -91.40%
==========================================
  Files          72      56       -16
  Lines        3812    1955     -1857
==========================================
- Hits         3685     103     -3582
- Misses        127    1852     +1725

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


typo

https://github.com/vandit15/Class-balanced-loss-pytorch/blob/921ccb8725b1eb0903b2c22a1a752a594fcae138/class_balanced_loss.py#L28
should be alpha: A float tensor of size [num_classes]

@e-yi refer to https://github.com/fcakyon/balanced-loss for improved and maintained version


I have been able to resolve this.

          I have been able to resolve this.

Originally posted by @taiwoakinyemi in https://github.com/pedrohcgs/CS_RR/issues/1#issuecomment-1055698109

Ho did you resolve this, Taiwo? Thanks for your help.


3 Attributes bug

Hi man thx for fix but if i put 3 attributes which is common in asp.net
 [HttpGet]
 [AllowAnonymous]
 [Route("{cameraId}/readings")]
 public IActionResult GetReadingsForCamera(int cameraId, int offset, int limit, string orderBy)
 {
       var filter = $"id eq {cameraId}";
       return CamerasService.GetAllCamerasPaginated(offset, limit, orderBy, filter);
 }

It still doesnt generate good comments
generated comments:
/// <summary>
///
/// </summary>
/// <returns></returns>

As a workaround i generate them below and copy later above my attributes

Thanks. I fixed it and released v0.0.14, check it out.


rx subscription manager

Description of the Change
Add naive but useful implementation of subscription manager for rx subscriptions.
Benefits
Don't miss your subscriptions in dtor
Possible Drawbacks
Additional inheritance level. But in fact, the class is a mixin.
Usage Examples or Tests [optional]
addSubscription(observer.subcribe(...))
Alternate Designs [optional]

addSubscription could return a reference for subscription for further usage
thread safety

This PR was introduced as sugar for managing subscriptions but it seems that the maintainers want to find a more elegant solution which includes both: automatic management of subscriptions and subject/observer lifetime.


Update test_branch jobs to be prow-compatible

@stevekuznetsov some jobs include the parameters more than once ....

LGTM @kargakis can you push out?

Pushed.

 [k8sCI] Get a suite of post-submit jobs running and turn on merge queue blocking on them


Refactor group by tags

Refactored the group by tags functionality for the AWS/OCP detail pages. Ensured the group by feature works with sorting, filtering, .csv export, etc.
Notes:

Sorting via tag name is not supported
The expanding rows are disabled intentionally

Fixes https://github.com/project-koku/koku-ui/issues/582

Codecov Report

Merging #583 into master will not change coverage.
The diff coverage is 73.33%.

@@           Coverage Diff           @@
##           master     #583   +/-   ##
=======================================
  Coverage   82.51%   82.51%
=======================================
  Files         193      193
  Lines        2963     2963
  Branches      492      492
=======================================
  Hits         2445     2445
  Misses        461      461
  Partials       57       57

Impacted Files
Coverage Δ

src/utils/getComputedAwsReportItems.ts
95.34% <100%> (ø)
:arrow_up:

src/utils/getComputedOcpOnAwsReportItems.ts
28.88% <40%> (ø)
:arrow_up:

src/utils/getComputedOcpReportItems.ts
71.11% <80%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Per today's feedback, I've added expanding rows and a detailsHeader component to show a fixed total cost -- see updated screenshot.


Package auto discovery + updated dependancy

Added package auto-discovery to avoid editing config/app.php
Updated depencancy to u2flib-server

(More PR are coming, feel free to release when you want)

Thanks. Merged and tagged


Add export all history command

Adding temporary command export_all_history to mitigate the high egress costs hitting the history archives.
This combines the export for operations, trades, effects, transactions, and diagnostic events to use only a single captive core instead of each spinning up their own for data export.
TODO: This should be removed as soon as LedgerExporter is complete and a TxMeta datalake is available

How does the runtime for all history_* exports compare with running a single export command?

I didn't run this in prod but in test it adds about 10s - 20s more time (so completes in 40s compared to 20s-30s). I think the runtime ratio will be much better in prod because the part that will benefit the most is the captive core catchup portion


Input looses focus

Found your library via Stackoverflow, it's very useful for my current project, so thanks for releasing it as Open Source Software!
I've copied your userlist-example over to my build-env and it works, mostly. There's just one issue - I loose focus after every letter entered in the input-fields.
I thought the problem might be caused by the modal-overlay - but after implementing nestedLink-methods in my own project, I have the same issue with loosing focus in the input-fields. And I don't use a modal.
After every letter I enter, the cursor is gone and I have to click in the input-field again to enter the next letter.
The issue is probably caused by the linkToObject.at-method, which I'm using to autosubmit the input-value in conjunction with onChange, in the first input-field in this example:
render() {
    const linked = Link.all ( this, 'sequence', 'command', 'info' ),
          infoLink   = this.props.sortData.at( 'info')

    const setValue = (x, e) => e.target.value

    return (

        {linked.sequence.value}
          <input value={ linked.info.value } onChange={ infoLink.action( setValue ) }/>
          <input valueLink={ linked.info }/>

    );
  }

The issue doesn't appear in the second-input-field, where I'm using a "simple" valueLink. But since I don't want to add a submit-button to commit its value to the parent, I prefer the more elaborate mechanism with onChange.
I'm on:

<EMAIL_ADDRESS><EMAIL_ADDRESS>

Really weird. Let me clarify, please.
This one
<input value={ linked.info.value } onChange={ infoLink.action( setValue ) }/>

Is the one you lose focus on? Or the second one?
It looks suspicious, btw. You're taking the value from one place, but writing user's input to different place. Shouldn't it look like this?
const jointLink = Link.value( linked.info.value, x => {
    linked.info.set( x );
    infoLink.set( x );
});

Here we construct ad-hoc link wrapper around linked.info, which keeps infoLink in sync when set with a value.

Does this Link.value help? I can add link.onChange( ) hooks to make this stuff simpler. So, it will look just like:
linked.info.onChange( x => infoLink.set( x ) );

Nice idea, will do it anyway. Just let me know if it helps for your case.

Another note. Standard <input> will not support valueLink prop in the next React version, and sometimes it behaves weird.
Try to use Input component from tags.jsx instead, and check if it will help.
import { Input } from 'valuelink/tags'

Ah, the jointLink-example is a lot simpler! But it does not fix the focus-issue. But I investigated a bit more in the meantime and I guess the problem are changing keys on every re-render. I'm using this in combination with react-anything-sortable.
 linkToObject.map() breaks the behaviour of my Sortable-component and I need to add a dynamic key to it to get Items to be sortable again. This dynamic key is probably the cause for the focus-issue. I'll post some example-code in my next comment, just want to get out this information while I have you working on this.
Thanks for pointing out the upper-case Input, I didn't see the difference between the examples and my code and was wondering why I'd still get the depreciation-warning :-)

Okay. npm update, and now you can do just linked.info.onChange( x => infoLink.set( x ) ).

Yes, changing keys in the lists will cause React to recreate inputs. To fix that you probably need to add some permanent ids to you objects which you're sorting when you create objects, if you don't have them yet.
{
    _id : _.uniqueId( 'x' ),
   ...
}

or just
let _count = 0;
...
{
    _id : _count++
    ...
}

Then, use this item._id as key.

Example would be good, since problem seems to be tricky.

Give me a few more minutes...

Okay. Meanwhile, the docs for new link.onChange and link.pipe. I changed API a bit, previous implementation introduced memory leak due to the fact that links are cached.
So, that's final API.
const jointLink = linked.info.onChange( x => infoLink.set( x ) )

https://github.com/Volicon/NestedLink/#-linkonchange-callback--any--void---link

Ok, it took me a while to reproduce the error with a simplified example, here's the example-code where the sorting works but the focus issue is present (Due to an ever-changing key in <Sortable>):
import React from 'react';
import Sortable from 'react-anything-sortable';
import { sortable } from 'react-anything-sortable';
import Link from 'valuelink'
import { Input, TextArea, Select, Radio, Checkbox } from 'valuelink/tags'
import shortid from 'shortid'

@sortable
class DemoHOCItem extends React.Component {

  constructor() {
      super();
      this.state = {
        info : ''
      };
    }

 componentWillMount() {
    console.log(this.props.sortData.value)
    this.setState( this.props.sortData.value );
  }

  render() {
    const linked = Link.all( this, 'sequence', 'info', 'other' ),
      infoLink   = this.props.sortData.at( 'info')

    const jointLink = Link.value( linked.info.value, x => {
      linked.info.set( x );
      infoLink.set( x);
    });

    return (

        {linked.sequence.value}
        <Input valueLink={ jointLink }></Input>

    );
  }
}

export default DemoHOCItem;

class Test extends React.Component {
  constructor() {
    super();
    this.state = {
      items: []
    };
  }

  componentDidMount() {
    console.log('Loading Route Sequences from server')
    fetch('http://uberfl.us/sortable.json')
    .then(
      response => {
        return response.json()
      }
    )
    .then(
      items => {
        this.setState(items)
        console.log(items)
      }
    )
  };

  handleSort(sortedArray) {
    console.log('Entering handleSort')
    var newItems = []
    var i=1
    console.log(sortedArray)
    sortedArray.map(function(item) {item.value.sequence=i, ++i, newItems.push(item.value)})
    console.log(newItems)
    this.setState({
      items: newItems
    });
  }

  render() {

    const itemsLink = Link.state( this, 'items' );

    function renderWithSortable(itemLink, index) {
      return (
        <DemoHOCItem className="vertical" sortData={ itemLink } key={ ['item',index].join('_') }>
          {console.log('Rendering Items')}
          {itemLink.sequence}
        </DemoHOCItem>
      );
    }

     return (

         <Sortable className="vertical-container" onSort={::this.handleSort} key={shortid.generate()} dynamic>
          {itemsLink.map(renderWithSortable, this)}
        </Sortable>

    );
  }
};

export default Test;

It is very fickle. Without the "dynamic"-property and a dynamic key in <Sortable> the items are not rendered. With the dynamic key the focus-issue appears.
But I got it working, following your advice, by using an incrementing key:
class Test extends React.Component {
  constructor() {
    super();
    this.state = {
      items: [],
      _sortableKey: 0
    };
  }

[..]
  handleSort(sortedArray) {
    console.log('Entering handleSort')
    this.state._sortableKey++;
    var newItems = []
    var i=1
    console.log(sortedArray)
    sortedArray.map(function(item) {item.value.sequence=i, ++i, newItems.push(item.value)})
    console.log(newItems)
    this.setState({
      items: newItems
    });
  }

[..]

render() {

    const itemsLink = Link.state( this, 'items' );

       return (

         <Sortable className="vertical-container" onSort={::this.handleSort} key={ this.state._sortableKey } dynamic>
          {itemsLink.map(renderWithSortable, this)}
        </Sortable>

    );
  }
};

Wohoo!!!
Can you tell me why the incrementing _sortableKey can be tracked by react?

This Sortable component is very tricky. I can't quickly understand how it works.
What about general sorting, React needs to know that element is moved. So, every element must have persistent index generated once and used as key in react list.
like this:
const list = [{ name: 'a', _id : 0}, { name: 'b', _id: 'whatever' }, ... ];

In your code, I think it should be enough to do it when you fetch elements from the server.
Then, no matter which permutation made on the list, React knows where every particular element is located, and should be able to preserve  state.
So, you're supposed to generate list like this:
list.map( element =>
 ...

It should work in the same way with link.map as well as with regular array.map. But again, this Sortable thingy does some dark magic in rendering, so I'm not completely sure if what I'm talking about is the case.

Btw. With my recent changes, you can shorten this:
    const jointLink = Link.value( linked.info.value, x => {
      linked.info.set( x );
      infoLink.set( x);
    });

to this:
    const jointLink = linked.info.onChange( x => infoLink.set( x ) );

Thank you for your input. Hope you'll enjoy the library. Let me know if there will be some other problems.


Image no longer works

Running this
docker run -ti -e POSTGRES_USER=postgres -e POSTGRES_PASSWORD=testpwd  ghcr.io/dbsystel/postgresql-partman:15-4

results in
/opt/bitnami/scripts/libpostgresql.sh: line 395: POSTGRESQL_REPLICATION_USE_PASSFILE: unbound variable. It seems that there was some issue in underlying bitnami image. Please consider updating/pinning to version without this bug. Thanks

Please also consider pinning upstream version. Bitnami seems to tage them like 15.4.0-debian-11-r50 15.4.0-debian-11-r51 . It would be much stable if you could use those tags to pin version, run some sort of test in CI before publishing the build and setup periodic version update using f.e dependabot. This would achieve the same automatic update process, but without surprises.

Thank you for letting us know. The root cause is the following upstream change: https://github.com/bitnami/containers/issues/74788
The next build will fix this, so I expect this to be fixed in the next few hour.
I appreciate your idea about pinning, but it will take a bit or effort to do that due to the nature of the multi-version actions. Since this was the first time this happened, and it was fixed fast, we will not put any effort in pinning versions. Feel free to pin our image digest if you use it, it will shield you from problems like this going into production.
I would welcome MRs that work toward a pin though, even if it only starts with some simple tests in the action, which would also be enough to catch errors like this.

I have just retriggered a new build which fixed the issue.


feat: Add Global setTimeout API

Description

This PR introduces the setTimeout feature for the button onClick call.

Fixes https://github.com/appsmithorg/appsmith/issues/776
Type of change

New feature (non-breaking change which adds functionality)
This change requires a documentation update

How Has This Been Tested?
Testing pending
Checklist:

[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
[x] 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

Added PR changes - 1


Kelvinamoaba/common footer

Description
Added common page footer component
Testing

Reviewers
@wsedor

@AmoabaKelvin please add some testing for these changes. One example could be to test that the index page renders the expected Layout components. Please add any other applicable ones as well. Goal is to start building up our test suites so we can leverage as part of automated testing on every github PR in the frontend space. Similar concept will happen in backend as well


[Node.js] Add typings

typings in the package.json points to an unexisting typescript definitions file:
"typings": "./lib/botbuilder-cognitiveservices.d.ts"

This file should be included in the published npm module, so we can use typescript definitions without pain.

Thank you for opening an issue against the Bot Framework Cognitive Services repo. As part of the Bot Framework v4 release, we’ve moved all the Luis and QnA work to platform SDK repo. We will continue to support and offer maintenance updates via this repo.
From now on, https://github.com/Microsoft/botframework-sdk repo will be used as hub, with pointers to all the different SDK languages, tools and samples repos.
As part of this restructuring, we are closing all tickets in this repo.
For defects or feature requests, please create a new issue in the Bot Framework repo found here:
https://github.com/Microsoft/botframework-sdk/issues
For product behavior, how-to, or general understanding questions, please use Stackoverflow.
https://stackoverflow.com/search?q=bot+framework
Thank you.
The Bot Framework Team


in installing issue

Building wheels for collected packages: Polygon3
Building wheel for Polygon3 (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py bdist_wheel did not run successfully.
│ exit code: 1
╰─> [14 lines of output]
NumPy extension not found - disabling support for it!
running bdist_wheel
running build
running build_py
creating build
creating build\lib.win-amd64-cpython-310
creating build\lib.win-amd64-cpython-310\Polygon
copying Polygon\IO.py -> build\lib.win-amd64-cpython-310\Polygon
copying Polygon\Shapes.py -> build\lib.win-amd64-cpython-310\Polygon
copying Polygon\Utils.py -> build\lib.win-amd64-cpython-310\Polygon
copying Polygon_init_.py -> build\lib.win-amd64-cpython-310\Polygon
running build_ext
building 'Polygon.cPolygon' extension
error: Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools": https://visualstudio.microsoft.com/visual-cpp-build-tools/
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for Polygon3
Running setup.py clean for Polygon3
Failed to build Polygon3
ERROR: Could not build wheels for Polygon3, which is required to install pyproject.toml-based projects

this is issue while installing and dont know how to resolve it

As mentioned in the error, perhaps you need a higher version of the Microsoft Visual C++. As all my experiments are conducted on servers with linux system, I'm sorry I can't be of further assistance.

When I use this command: "pip install torch==1.13.1+cu111 torchvision==0.10.1+cu111 -f https://download.pytorch.org/whl/torch_stable.html", it reports that it cannot find the torch and torchvision versions as specified in the command. Is it possible for me to switch to different versions?


Add CCC image/thumbnail

Oh, I think you meant the repo image

Not quite. That's the preview image for if someone shares the repo but not the manuscript at https://greenelab.github.io/ccc-manuscript/. You can set the thumbnail as described in USAGE


Create one example tip in it's own domain inside TamagoTipsManagerBundle`

Details here:  http://symfony.com/doc/current/book/translation.html
Have at least one sample tip in it's own domain will ensure that Lexik loads that domain into it's grid:

We have a domain 'TamagoTipsManager' which shows up in Lexik grid:

Does this make sense?

How is this configured?

Ping

I added it manually in one of the lexik tables while creating the first dummy tip.

Thanks.  How will users of the bundle be able to configure this?  Is there any way to configure automatically?

Not yet. By default it doesn't show any domain

It won't show up on your end. I added it manually in my local.

There should be a way for us to configure in the bundle such that I don't have to configure manually.

I think what we need to do is advice users to do something like this:
https://github.com/lexik/LexikTranslationBundle/blob/master/Resources/doc/index.md#import-translations
I believe we did this when we were initially playing around with Lexik?
Note that there is a bundleName option.  So what we could do is create some example tips and then in the README advice users to import the tips for just our TamagoTipsManagerBundle into the database.

I think we will have to create our own command for that.

I'm 95% sure we can do this with the existing Lexik command..

Yes. We got this done in the last meeting! I updated the Readme too

https://github.com/tamago-db/TamagoTipsManagerBundle#import-translations

See clarifications in README


[SPARK-7711] Add a startTime property to match the corresponding one in Scala

Merged build triggered.

Merged build started.

Test build #33116 has started for   PR 6275 at commit 7a87410.

Test build #33116 has finished for   PR 6275 at commit 7a87410.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Merged build triggered.

Merged build started.

Test build #33132 has started for   PR 6275 at commit 06662dc.

Test build #33132 has finished for   PR 6275 at commit 06662dc.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class DataFrameReader(object):
class DataFrameWriter(object):

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

Merged build finished. Test FAILed.

jenkins retest this please

Merged build triggered.

Merged build started.

Test build #33149 has started for   PR 6275 at commit 06662dc.

Test build #33149 has finished for   PR 6275 at commit 06662dc.

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

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

Merged build finished. Test PASSed.

LGTM. Remembered that I had have this in PR once, but not merged by some reason (not remembered).

LGTM as well, so I'm going to merge this into master and branch-1.4 (1.4.0).  Thanks!


Hero's createRectTween

I have to set a custom createRectTween to PhotoView's Hero.
Currently, two attributes of PhotoViewImageWrapper are exclusively Hero's parameters: heroTag and transitionOnUserGestures. At this point, I think there should be a property called heroAttributes, which will lighten PhotoViewImageWrapper's parameters and also let more clear which parameters are Hero-related (I mean, createRectTween as a parameter of a PhotoView may seen completely out of place).
I can submit a PR with all of these improvements, but breaking changes are going to emerge, so I need your approval to make sure I won't waste my time.

Absolutely, go for it.

Tackle'd.


Update discord.ini to latest v10 endpoints

Modified endpoitns to latest versions based on https://discord.com/developers/docs/topics/oauth2

Thanks for the PR! A couple of quick questions:

Have you tested this change, and was the test successful?
Is it necessary or even just preferable to include the api/v10 component? I'm just wondering whether this code would be more future-proof if it didn't stipulate a specific API version if that's possible.
If the answer to both questions is "Yes", then I'll merge this immediately, though I haven't tested myself.

Thanks for the PR. As I can see the current code still works fine. We will be testing the changes and its result. But my opinion is to keep it approved for the future if it works and continue with current ini as long as its working. @lairdshaw ?

Thanks for the PR. As I can see the current code still works fine. We will be testing the changes and its result. But my opinion is to keep it approved for the future if it works and continue with current ini as long as its working. @lairdshaw ?

Tested Yes it does infact work
Using the current https://discordapp.com/ endpoints are directing to v6 endpoint which is currently labeled as deprecated and may be discontinued in the future, while V10 is the current up to date and will not be discontinued/deprecated for quite awhile
https://discord.com/developers/docs/reference

Please give us some times to test at our end and the same will be merged.
Appreciate your patience.

@lairdshaw ?

I think the decision to merge after testing on our end is sound.

This should be Isango getting ahead of v6 getting shut, whenever that happens
The changes seem to be reasonable, no difference should be noticeable.

Tested. Merged.
Thanks for your contribution @RiisDev


Added API Key fallback and print statments

Description
This PR fixes api keys not being used
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@mudler this is the pr from discord

please use the logging facilities rather than directly using fmt.Print/Println

@cryptk I did this


Create a way to "close out" publications stuck in Wrong File Version

Currently, 611 of 1196 publications in the AI OA Workflow are in the "Wrong File Version" bucket. It's turning out to be quite difficult to get publications out of this bucket. The response rate to our emails is low. There are more than 400 publications in the bucket that we've sent emails for already. #996 is a feature aimed at that problem. But we know from other emails we send (OA reminder emails, ScholarSphere curation emails) that the response rate is not going to be 100%.
We need to decide what to do with publications for which we send repeated emails and the researcher does not reply. Basically we have three options:

leave them in the bucket, perhaps with improved sorting/filtering
put them in a new bucket of their own
remove them from the workflow (and remove the files from RMD and/or Activity Insight?)

Features to consider regardless of option

Add a count of how many times we've sent a Wrong Version email about the publication. Display this on Wrong Version bucket or otherwise enable admins to sort by it.

Option 1

Enable the user to sort and filter the publications in this bucket. E.g., hide (or sort to the bottom) publications and/or researchers for which we've sent emails ever or within a certain time frame.

Option 2

Split this bucket into two buckets, one for publications that we've already reviewed and sent a "wrong version" email for and one for all other files. For the "already reviewed" bucket, send an automatic email every six months with the list of files that we have the wrong version for. Consider consolidating this into the regular OA reminder emails. (Question: If a researcher replaces the file without telling us, and it goes through automated version review, would it be possible for it to slip back into the "already reviewed" bucket? We probably wouldn't want this.)

Option 3

Based on the file upload date and/or the number of times we've sent a Wrong Version email, remove files from the workflow. (This may not be necessary if we improve the sorting on this bucket.) What this should look like is not clear -- would we want to keep the files in Activity Insight? Would we want to send the author a final email with the file attached, but delete the file from RMD and Activity Insight?
For files that we've sent a Wrong Version email about, check to see if the researcher is still at PSU. If not, wait for a set period of time to see if we get an email response, and then remove the files. Not sure if any emails are being returned as undeliverable, but we'd certainly want to do something about articles associated with those.

@anaelizabethenriquez From the development side, I think the simplest solution here is to add a separate "already reviewed" bucket (Option 2) for these to go to after an email has been sent.  Adding another bucket with one added criteria and another task for sending those emails would be pretty simple.  Adding filtering and sorting (or even search) functionality is definitely possible, though, if that's preferred.  Option 3 could still be implemented later on if we decide we want an automated way to get rid of these files.

If a researcher replaces the file without telling us, and it goes through automated version review, would it be possible for it to slip back into the "already reviewed" bucket? We probably wouldn't want this.

Since the criteria for landing in the "already reviewed" bucket would include having an email sent, it wouldn't end up in this bucket automatically.  However, it would end up there if an email is triggered by an admin, which would hopefully involve a review.  If this isn't what we'd want to happen, what would we want?
We don't track how many of these emails we've sent out right now, but we can implement something to track that number and display it.

Another approach would be to create a "Stale file" bucket where files uploaded more than X months ago can be removed from the rest of the workflow (except perhaps the deposit bucket) so that the workflow is more usable for recent files.

I need to spend some time with this issue during the week after Thanksgiving and attempt to cue something up for the second sprint. Related #1024

@ajkiessl I've updated the description here so it describes what I think we should do. Would you mind taking a look when you get a chance? I'm wondering if there should be separate issues for the tasks in the description, or if this is preferable.

@anaelizabethenriquez This looks good.  All these tasks can stay in a single issue.  Probably best to do all of this at once.


Added example of use

Created the class com.monotonic.map_visualiser.Sample to run a simple example

Thanks.


Filtered-stream

Amazing work!
I wonder if anyone has started working on simulating (somehow) the Filtered-Stream API where you could follow users/keywords etc. in real-time?

I haven't actually even heard of filtered stream on Twitter, but looking into it, it might be worth giving a shot, but I don't think it's possible. I'll check it out and confirm.

There seems to be no way of getting filtered-streams unfortunately, since it's not used on the Twitter web app.


Vulnerability roundup 101: gst-plugins-bad-1.16.3: 1 advisory [9.8]

search, files

[ ] CVE-2021-3185 CVSSv3=9.8 (nixos-20.09)

Scanned versions: nixos-20.09: c7e905b6a97.
Cc @matthewbauer

Fixed in https://github.com/NixOS/nixpkgs/pull/117993.


Implemented C# version of mono_class_is_subclass_of

Went back and had a look at what the cpp version does and returns. This should account for the discrepancy in results regarding arrays.


Deduplicate and delete some methods

This PR fixes some minor issues:

Sometimes we imported a function twice, and then it ended up in the outline twice
We had an unused method announce_text, which was an obsolete duplicate of AccessibilityNode.text
Removed the is_focused method, which I tested and determined was unneeded

Are you still working on this PR?

Sorry for the confusion. Yes—in fact I think I will merge this in a second when the tests pass. Basically, there was an error about an "unnecessary hint" but it seemed strange to me that this hint was unnecessary. It should be used! So I dug into it and in fact I had made a fairly elementary error with the handling of patches in the compiler. The error lead to the compiler skipping patches which made it not use that hint.
I fixed the error and now all the unnecessary hints are necessary again.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.7 to before v5.10.88",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=3f50f132d8400e129fc9eb68b5020167ef80a244",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=e2aad0b5f2cbf71a31d00ce7bb4dee948adff5a9"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "3f50f132d8400e129fc9eb68b5020167ef80a244",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "e2aad0b5f2cbf71a31d00ce7bb4dee948adff5a9",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "bpf: Fix signed bounds propagation after mov32\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.10.88 by commit e2aad0b5f2cbf71a31d00ce7bb4dee948adff5a9, it was introduced in version v5.7 by commit 3f50f132d8400e129fc9eb68b5020167ef80a244. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002690


Build fails with gradle 4.8

When trying to execute the documentation generation process with the newest version of Gradle (4.8), this happens:
$ gradle clean

FAILURE: Build failed with an exception.

* Where:
Build file '/home/releases/Repositories/thymeleaf/thymeleaf-docs/build.gradle' line: 18

* What went wrong:
Could not compile build file '/home/releases/Repositories/thymeleaf/thymeleaf-docs/build.gradle'.
> startup failed:
  build file '/home/releases/Repositories/thymeleaf/thymeleaf-docs/build.gradle': 18: unable to resolve class org.gradle.api.plugins.jetty.internal.Monitor
   @ line 18, column 1.
     import org.gradle.api.plugins.jetty.internal.Monitor
     ^

  1 error

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 1s

It seems something we depend on has changed location, or does not exist anymore.
I tried to update Gradle because the version we are still using (2.12) does not support Java 10.

Ooo, try use the gradle wrapper instead, gradlew (command) so that it uses the version of Gradle that's known to work with the docs.

Similar issue using JDK 11 with ./gradlew:
$ ./gradlew generateDocs

FAILURE: Build failed with an exception.

* What went wrong:
Could not determine java version from '11'.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

Am trying to sort this out this weekend.  Seems a lot of the things we relied on in Gradle 2 have been deprecated and removed, so I'm trying out alternatives without hopefully changing too much about how we go about building things.

I've updated the dev branch with some changes that should make it run on recent Gradle (updated the wrapper to use the latest version 4.10.2) and Java 11 (running OpenJDK on my Mac).  The biggest change was finding some community plugins to replace the removal of Gradle's the built-in Jetty server that work with Java 11.  The plugins are using OSS-friendly licences.
Despite all that, the document creation process is still the exact same commands as they were before 😀  So give them a try when you're able, and then we can update master and close this issue.

I did a ./gradlew clean generateDocs on the current state of the dev branch (using JDK 11) and after a while I got an error in the PDF generation phase:
[...]
> Task :generateDocsPDF
Generating PDF doc for tutorials/2.1/extendingthymeleaf (tutorials)...
Generating PDF doc for tutorials/2.1/usingthymeleaf (tutorials)...
Generating PDF doc for tutorials/2.1/thymeleafspring (tutorials)...
Generating PDF doc for tutorials/2.1/usingthymeleaf_ja (tutorials)...
Generating PDF doc for tutorials/3.0/extendingthymeleaf (tutorials)...
Generating PDF doc for tutorials/3.0/usingthymeleaf (tutorials)...
Generating PDF doc for tutorials/3.0/thymeleafspring (tutorials)...
Generating PDF doc for tutorials/3.0/usingthymeleaf_ja (tutorials)...

Error: Failed loading page http://localhost:8080/thymeleaf-docs/tutorials/3.0/usingthymeleaf_ja.html (sometimes it will work just to ignore this error with --load-error-handling ignore)
Exit with code 1 due to network error: ConnectionRefusedError

> Task :generateDocsPDF FAILED

FAILURE: Build failed with an exception.

* Where:
Build file '/home/releases/Repositories/thymeleaf/thymeleaf-docs/build.gradle' line: 292

* What went wrong:
Execution failed for task ':generateDocsPDF'.
> java.io.IOException: Stream closed

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 2m 36s
7 actionable tasks: 7 executed

It does look like some kind of network error when connecting to the webserver rendering the HTML in order to convert it to PDF… maybe some kind of thread synchronization error at the gradle side (perhaps the client connecting when the server has already stopped serving the content…)
But funny enough, I tried a second time and…
BUILD SUCCESSFUL in 2m 29s
7 actionable tasks: 7 executed

😆
Then I tried a third time, and again it worked fine. So maybe it was just a matter of very bad luck caused by some weird planetary conjunction. I can live with a few retries in this process to be honest, after all it's only executed 4-5 times per year…

Odd, but if you're OK with that, then I'm OK with that too 👍  I had that error happen to me when I started but not towards the end so I thought something I did fixed it up.

dev branch merged into master


execution commands for contributed REPLs

shift+enter should execute the input box code as it does in jupyter IWs.
consider how to switch between enter vs shift+enter

Also discussed today the need to figure out shift+enter and enter behavior when both Python extension and Jupyter extension are installed. Should be configurable, not sure what are exactly needed to make this happen though.

Need: https://github.com/microsoft/vscode/issues/212427 to properly handle executing on enter and thus allowing Python extension to figure out whether something is a complete or incomplete Python command.

We should be good once https://github.com/microsoft/vscode-python/pull/23442 is merged. Handling execute on enter like the traditional interactive REPL in terminal from Python itself closely as we can.


Feat: Design oil seeds page

Describe the feature
@RitiChandak
check this
we can add new page /oil-seeds when user click on this option they will redirected here
Add ScreenShots

we can design oil seeds something like this

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Thank you
can you please assign level 3
i will make

theme matching
beautiful design
responsive too


Links on left side of page do not point to correct anchors

The links on left side of page do not point to correct anchors in the FAQ section. Instead they all point to the main FAQ section.
Example:
This link for backups goes to faq
Third-party backup solutions
It should go to backup
Third-party backup solutions

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

ID: bfdd20d6-7b1d-73f1-f580-d9207e10c5fb
Version Independent ID: 610d9d1a-dbea-0289-01ca-c84f2b30d1ef
Content: Microsoft Azure VMware Solution FAQ - Azure VMware Solution
Content Source: articles/azure-vmware/faq.yml
Service: azure-vmware
GitHub Login: @Ajayan1008

@suzizuber It looks like all of the hrefs for the FAQ direct to faq.yml (lines 219 through 243) which is causing the behavior we are seeing.  In addition, the TOC nav mirrors the page nave on the right.  Do we have a preference for how we want to address this?

@kobulloc-MSFT given that it's a yml file, I can't add links to the section titles in the ToC. I noticed the working nav on the right side of the FAQ page with links to each section. We could remove the section titles from the ToC (listed under FAQ article name) and let viewers use the right side nav - your thoughts on this?

@suzizuber That's the same approach I was thinking of taking.  It's an unusually long FAQ and if we were starting from scratch I might do things a bit differently but I think that will give us the best result with what we have.

@kobulloc-MSFT I will put this on my work item list and let you know when I can make the update to FAQ. Thank you for your input.

@suzizuber I have a PR available for review which should resolve this issue.

@kobulloc-MSFT I'll look for the PR update, thank you for your help on this as well. Can this PR be closed out?


Ensure Brave types are unloadable

Issue #716 discusses a small leak where memory allocated to thread locals isn't "cleaned up" when brave's tracing instance is closed. There's another problem which this describes:
Thread locals that hold a reference to a Brave type will prevent those classes from being unloaded. This is a form of leak. We can mitigate this by not using Brave types in thread locals, and/or switch to an alternate thread local store like https://github.com/raphw/weak-lock-free
Here's the full story by @raphw (thanks tons for the scoop!)

When an application is undeployed, it is usually garbage collected. But if one of your classes is stuck in a thread, the class cannot be collected and neither can its class loader which again references all of its loaded classes. You have created a leak, so to speak. Tomcat does however detect this and renews its thread objects in the server's thread pool. Other application servers cannot do this, though.

To avoid this, you must reference a thread local value in one of your instances/classes that will be part of the application being undeployed. To create a thread local value, you are using the thread as a key of a map to the value which emulates thread locals behavior. Doing so, you do however now stop tomcat from renewing its threads as those threads stick in you map and cannot be collected.

Yet, now you are in control and can store the thread references weakly. To avoid an overallocation of weak references, you need to mask the keys and reuse the weak references you created previously. To avoid all the clutter, you can just use this library: https://github.com/raphw/weak-lock-free - either with inline expunction (that is how Javas weak map works) or you start a dedicated thread to clear out the weak refs.

cc also @trustin for fun and good measure

this one has most of the core work, though it still breaks.. maybe I'll figure it out next flight

going out in 5.3.2 thanks @swankjesse @raphw for tips.. we now have tests to prove things are unloadable. ex: https://github.com/openzipkin/brave/blob/master/brave-tests/src/test/java/brave/TracingClassLoaderTest.java


Enhance getBlockHeaders endpoint with optional verbose param

This PR enhances getBlockHeaders enpoint with optional verbose param, default = false, in order to get the serialized version of the headers which saves a lot of space and speeds up header download ~ 2.5x. Especially useful if downloading a lot of headers at the same time.
Also set limit default to 1 to cause less stress on resources when not set explicitly.
Cf. https://github.com/dashevo/dapi/pull/171

Please, update JS Doc

done!


Access Denied- Windows 8

I got the sim on windows 8, but when trying to run
" > gps-sdr-sim -e brdc3540.14n -l 30.286502,120.032669,100 "I get "access denied" and when running as an administrator it comes up it can't be found?

Make sure you run it from the command prompt. It doesn't work on the Windows PowerShell.

Sorry, how do you mean the command prompt?  I did it by typing "cmd" in the task manager and doing it from there.

It will open the command prompt. Did you run it from the folder where the gps-sdr-sim located?

Do you mean did I click on the exercise file? It shows up with the usage options such as  -e <gps_nav>  ,etc  in the cmd" I just can't run anything from it. I'm not used to Windows.

exercise file not exercise.

exe, hate spell correct sometimes:)

Type "cmd" in the address bar of the file explorer where gps-sdr-sim.exe is located.


The PSObject implementation does not handle classes derived from DynamicObject properly

This repro’s on 5.1 and 6. The problem is that if I have a DynamicObject that has a dynamic property “Foo”, things like $obj.Foo work, but Select-Object -ExpandProperty Foo do not.

Steps to reproduce
Add-Type @"
using System;
using System.Collections.Generic;
using System.Dynamic;

public class MyDynamicObject : DynamicObject
{
    public override IEnumerable<string> GetDynamicMemberNames()
    {
        return new string[] { "Foo" };
    }

    public override bool TryGetMember( GetMemberBinder binder, out object result )
    {
        result = null;
        bool succeeded = false;

        if( binder.Name == "Foo" )
        {
            result = 123;
            succeeded = true;
        }

        return succeeded;
    }
}
"@

$myObj = [MyDynamicObject]::new()

$myObj.Foo                                  # <-- works
$myObj | %{ $_.Foo }                        # <-- works
$myObj | Select-Object -ExpandProperty Foo  # <-- DOES NOT WORK

Expected behavior
The last three lines should all work (yield 123).

Actual behavior
The Select-Object statement throws. ("Select-Object : Property "Foo" cannot be found.")

Environment data

> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      6.1.0-preview.1
PSEdition                      Core
GitCommitId                    v6.1.0-preview.1
OS                             Microsoft Windows 10.0.16299
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

The original title was a bit confusing given that PowerShell, from V3 on, uses the DLR extensively. I've updated it to more accurately reflect what I think the problem is. @jazzdelightsme - are you ok with the updated title?

Sure. Thank you!

PSDynamicMember appears to be a mostly-useless stub: the Value getter/setters just throw, and it doesn't even hold a reference to the owning object. It seems like the solution will involve teaching this thing how to get a value... first I suppose we need to be able to determine if the "Foo" member even is a property or not (maybe it's a method). Then, if it's a property, we have to figure out if it is settable and/or gettable. @LzyBkr / @BrucePay, is that possible with some DLR-fu?
Looking at the DLR stuff is ... intimidating. It's hard to tell where to grab on first. Any hints?

PSDynamicMember exists just for Get-Member - and the results of Get-Member don't reference an owning object.
I believe this is the code that is supposed to cover your scenario. It was likely tested (I don't typically write code like that without testing) so I'm not sure why it doesn't work - but I'd start there.

Thanks @LzyBkr!
I believe that code does in fact work. That's why things like $myObj.Foo work: PSObject dynamically binds the member access, and all is well.
But for Select-Object -ExpandProperty, on the other hand... that code does not come into play.
When using wildcards (like $myObj | select -exp *Foo*), DotNetAdapter.AddAllDynamicMembers correctly finds the "Foo" member, and adds a PSDynamicMember to the collection, so the Match method, when it calls GetIntegratedMembers, has the "Foo" member in the list.
However, the Match method is instructed to only return members of type Properties or PropertySet (as specified by MshExpression.ResolveNames(PSObject, bool)), but the "Foo" member is a Dynamic, so it gets filtered out.
For the non-wildcard case (like $myObj | select -exp Foo), ResolveNames just looks for target.Members[_stringValue] (where _stringValue is "Foo"), and dynamic members don't seem to show up in that PSMemberInfoIntegratingCollection.
If dynamic members were to be exposed to ResolveNames (via Members.Match or Members[string indexer], though... perhaps things could work. But wouldn't PSDynamicMember would have to built up a bit, for that to work? And I'm not sure if PSDynamicMember can be taught to do what it needs to do.
If that's not the right approach... then what will
MshExpression.GetValue do? (if the dynamic member does not show up in Properties?)

Ah, of course. That's the problem then. The PSObject pseudo-reflection like api can't work for dynamic objects, at least not in general.
The GetDynamicMemberNames api that a dynamic object implements is not a contract. For example, a dynamic object could synthesize a property just by trying to access it - essentially giving the object infinite members. The primary use case for the api is a debugger, or in PowerShell's case, Get-Member - in other words - for a somewhat improved UI, but nothing more.
I believe the right fix is to use a dynamic site in the same way that PowerShell does for the $obj.member syntax.
This problem affects Select-Object, ForEach-Object, Sort-Object, Measure-Object, Where-Object, Format-List, Format-Table and maybe more.
It would seem this scenario is a corner case - it has never worked and this is the first report I've seen. That said, fixing it has a nice benefit - performance should improve even when not using dynamic objects.
A fix does need to be careful though - some of those commands accept wildcards for property names. It might be best to keep the existing code for any use of wildcards.

Okay, it may be beyond my ken, but I'll buy you lunch, @LzyBkr, if you are willing to talk me through what you think the "right" change would look like...

No need to buy me lunch, but I'm happy to chat.
To get you started - you can find similar code here.
The difference is - you'll use PSGetMemberBinder.Get("PropertyName"). It really does just boil down to a couple of lines of code - creating the call site with the right binder, then invoking the delegate for that site.

I came up with an PoC (see here), that seems to work well, but it only addresses Select-Object--not other things like Where-Object.
Next I'll try pushing the solution down a bit further, into the .Members collection itself.


Q: ConnectedView – animations/transitions on props changes

I'm really loving SwiftUIFlux and the direction things are going with ConnectedView.   Great job!
One question I've run into is how to handle animations when props change.  Here's a very contrived example (leaving out some bits for brevity) ...
struct SimpleExample: ConnectedView {
  struct Props {
    let isVisible: Bool
  }

  func map(state: AppState, dispatch: @escaping DispatchFunction) -> Props {
    return Props(
      isVisible: state.isVisible
    )
  }

  func body(props: Props) -> some View {
    VStack {
      Text("A simple example")

      if props.isVisible {
        Text("I'm visible!").transition(.slide).animation(.easeInOut)
      }
    }
  }
}

This all works except for the animation/transition.
Generally we'd just handle this stuff with @State .... bindings and be done but I'm trying to figure out if there's a way I can manage either the bound @State variable from the store or some other solution so the state of my UI is coming 100% from my store.
In the past in Redux, I've had situations that are more like:
  func map(state: AppState, dispatch: @escaping DispatchFunction) -> Props {
    return Props(
      isVisible: someComplicatedLogic(state)
    )
  }

isVisible is derived from the state and should update / recompute when state changes.
Any recommendations how to handle this situation in a SwiftUIFlux'y way?

There shouldn't be anything specific to do with SwiftUIFlux.
this article explains some changes in SwiftUI:
https://swiftui-lab.com/advanced-transitions/
try
      if props.isVisible {
        Text("I'm visible!").transition(
          AnyTransition.opacity.animation(.easeInOut(duration: 1.0))
        )
      }


Check yaml files get generated for all studies in GCP000726

Please confirm metadata yaml files have been corrected generated for studies in GCP000726 as described in goci#1137

I generated YAML files for the top 10 studies and I found their sample info is duplicated (example:https://www.ebi.ac.uk/gwas/ingest/api/v2/studies/GCST90292447/samples) on the ingest API. Could someone help me with it?

This is another submission id affected by this issue: 5f697cd6a54daa00013cf263. I can see from gwas-info there were “unforeseen technical issues” (in Jan 2021) but i don’t know what they were.

For GCP000726, please review the progress on #1207
'5f697cd6a54daa00013cf263', @earlEBI  Could I know more details on it? If it is not a YAML file issue, should we create a new ticket for it? Thanks!

@jiyue1214 Hi Yue, sorry this was more a note for Sajo to see when he was investigating, that there was a similar issue to look into. It's now mentioned in this ticket that Sajo hopes to resolve by reuploading the template. It's the very old submission from 2020. The yamls are affected in the same way as the GCP000726 so will need regenerating at some point.

I have recreated files for GCP000726 and '5f697cd6a54daa00013cf263'. Once Lizzy @earlEBI  reviews them, this ticket can be closed. Thanks!

Yamls for both these submissions now look good!


Project lock failure can update status of finished landing zone with rapid request spamming

I've witnessed the following on our production server:

A project has multiple active landing zones
A user rapidly spams ZoneSubmitMoveAPIView for all zones via the REST API
Most requests end up setting the zone status to FAILED due to the project being locked, as only one moving taskflow at a time can be running per project
One of the spam requests lands on zone X, there is no lock active and the move taskflow succeeds
..but in the end, the now moved zone still ends up as FAILED with a project lock error

Looking at the timeline for that zone, it seems we have received multiple requests for moving the zone after it has already finished.
It seems that when setting the failed status due to project lock, we don't look into the zone status?
No taskflow should be attempted on a finished zone to begin with and the UI/API view should catch that before the lock is even checked. But maybe there just have been so many near-simultaneous requests, that at the point where we fail the lock check, the zone status has not yet been updated anyway. This may be a tricky one. Maybe a queue solution like proposed in #1910 could help...
Current plan to tackle this:

Ensure we don't allow submitting operations to zones with a busy status
Add a check for current landing zone status when reporting lock failure

If status not in ACTIVE or FAILED, do not update

???

This should be, at least mostly, fixed. It's still possible that if there is an ongoing non-finished taskflow and near-simultaneous spam is done, we temporarily get a lock failure as the zone status. But even then, the ongoing task will eventually submit the correct status.
That would seem good enough, but if we identify cases where this still remains a problem, I'll reopen the ticket.


Pass cmd line arguments to device application

I understand that, in order to pass command line arguments to the application I need to modify the application manifest and set the CmdArgs field. How should my dockerfile look like to accomplish this? THe dockerfile needs to somehow make sure the manifest is updated to contain the value of an environment variable available in docker,

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

ID: 49d79211-71f9-ecf3-205e-ebbefc052b17
Version Independent ID: db3be8ed-4dfb-e93f-deee-d7ca4e1d091c
Content: Use containers to build Azure Sphere apps
Content Source: AzureSphere/articles/app-development/container-build.md
Product: azure-sphere
GitHub Login: @pennyo
Microsoft Alias: azurespheredocs

@Expecho
Thanks for your feedback! We will investigate and update as appropriate.

Hello @pennyo could you please suggest in this regard.

Hello @Expecho just wanted to confirm if you have tried steps 2 and 3 in the sample tutorial section called: Build the default sample app using the Dockerfile ?

@SatishBoddu-MSFT Yeah I did both steps and the app is correctly built by the Dockerfile. I am stuck on the next step in a way that I have defined a GitHub action that succesfully builds the repo but I want to pass a secret to my application. That has to be done using the CmdArgs field in the application manifest but I have no idea how that must be done using the docker approach as outlined in the docs.

Hello @Expecho Thanks for the confirmation, we are working on this.

@Expecho Good question. I've raised this with our engineering team and will respond when we have more information.

@Expecho it sounds like you are trying to pass a runtime argument to your app in the container. The app doesn't run in the container, it needs to be deployed to a device to run. You'll need to either download the imagepackage from the produced artifact and deploy to a device or publish it to a deployment group in AS3 to send it to devices OTA.
If I'm wrong and this is a parameter that is used in the build let me know.

Ok, let my try to pain the whole picture here. I have my code on GitHub, in a public repo. My apps need a connection string. I see that as a secret that I do not want to expose in my code. I have the secret stored in GitHub as an environment variable that I can access during the build phase as seen here. Now I need to somehow pass this secret to the application. So my initial thought is to pass it as an argument to the application as outlined in the docs. But I am stuck on how to proceed with this and to integrate it in the build process.

@Expecho did you consider using IoTHub to update your azure sphere device Twin property or sending a Cloud to Device Message  with the secret you need to use on your application? See also Cloud-to-device communications guidance.
Azure Sphere is already connected securely with IoTHub so I believe this can be a secure way to share your passwords using TLS encrypted connection. See how to Set up an Azure IoT Hub for Azure Sphere and let us know if you think this is not a good approach and\or you oversee security issues?
Also, as this follows a bit out of the context of this document, can we move this discussion to Microsoft Q&A so other members from the community can share their experience as well?
fyi @robotdad
Thanks

@Expecho did you consider using IoTHub to update your azure sphere device Twin property or sending a Cloud to Device Message  with the secret you need to use on your application? See also Cloud-to-device communications guidance.
Azure Sphere is already connected securely with IoTHub so I believe this can be a secure way to share your passwords using TLS encrypted connection. See how to Set up an Azure IoT Hub for Azure Sphere and let us know if you think this is not a good approach and\or you oversee security issues?
Also, as this lands slightly out of the context of this document, can we move this discussion to Microsoft Q&A so other members from the community can share their experience as well?
fyi @robotdad
Thanks

Thansk for the response. You are right, I think Q&A might be a better fit. I'll move it in a short time and close this issue. One thing I'd like to address. You say it is safe to connect to IoT Hub and I agree. However, in the examples like the one here they tell you to copy/paste the connection string to code. That means that everyone who has access to my public repo can view my connection string. I'd like to avoid that but I have no clue how to pass the connection string in a secure way to my azure sphere device.

@Expecho definitely adding the connection string to your code is not recommended at all for some production scenarios. It is a very easy way to get you started connecting to IoTHub and develop some POCs though :).
Azure Sphere authenticates using an x.509 certificate and we have an extense set of docs about securing your environement when using Azure IoT Hub. You can start here: https://docs.microsoft.com/en-us/azure/iot-fundamentals/security-recommendations
Thank you for opening a new thread in Microsoft Q&A. Feel free to close this case when ready!

Created a Q&A question for it.


Roles is not assigned in the data seed

I'll try to fix the bug and make sure that the data is successfully inserted.

Estimated Time: 60 mins


[FLINK-25399][tests] Disable changelog backend in tests

Stabilize the builds until the issue is resolved
cc: @pnowojski , @curcur

Private branch built successfully, merging.


Delete button on Edit/Detail view

We've been moving our admin backend from an old outdated version of superadmin to the most recent version of flask-admin and started customizing a few things to make life easier on the people using our admin interface. One feature I was surprised to see missing from flask-admin was a delete button on the edit and/or detail view.
A common use case for this is: someone sends you a link straight to an edit view in the admin. You now want to delete this object. You need to navigate to the list view, then look for the object again to be able to delete it.
For now I've implemented this with a macro that fires a similar event as the delete button in the list view. I'd much rather see an implementation though that has this built into the ModelView. What are your thoughts on this?
I'd be open to overhauling my implementation and build something that can be shipped to everyone. If you agree on the usefulness of this feature.

Sounds good to me, pull request is welcome.
Please also add a setting with the default set to show the button.

@mrjoes great, I'll spend some time on this in the next couple of days. I was thinking of putting the button right aligned at the bottom of the page in the same line as the save buttons but slightly offset.
Feedback on the location?

Hello,
I also need to put the delete button on the edit/details view.
What was your solution? @lucasvo

@lucasvo, were you ever able to make that PR? Would be really helpful to have this option.

@mrjoes great, I'll spend some time on this in the next couple of days. I was thinking of putting the button right aligned at the bottom of the page in the same line as the save buttons but slightly offset.

I know it's a pretty old issue but I must ask, do you have any work in progress 🤓?
If not, the idea sounds good, and implementation should be very smooth 🚀.


Sanic consider jwt token as files

Hello, if i post a file with a jwt token , i cant get this jwt token from request.form but its in request.files. Its a bug ?

To be expected. All files should be in request.files, as per https://sanic.readthedocs.io/en/latest/sanic/request_data.html

Thanks for your apply
Its my fault
jwt.encode({'some': 'payload'}, 'secret', algorithm='HS256') get bytes not str
i put this jwt.encode({'some': 'payload'}, 'secret', algorithm='HS256').decode() in my post_data, it works ok


[nextcloud] Mariadb pod with persistent storage will not start

Hello. Thanks for nice Nextcloud helm chart.
Am having problem starting mariadb that is bundled with Nextcloud chart. Mariadb starts if not using persistent storage but with 'mariadb.storage.requestedSize: 8Gi' then it fails.
PV created with openbs cStor seems fine.
'2021-01-15  9:21:47 0 [ERROR] mysqld: Got error 'Can't create file' when trying to use aria control file '/var/lib/mysql/aria_log_control'
2021-01-15  9:21:47 0 [ERROR] Plugin 'Aria' init function returned error.
2021-01-15  9:21:47 0 [ERROR] Plugin 'Aria' registration as a STORAGE ENGINE failed.
2021-01-15  9:21:47 0 [ERROR] InnoDB: Operating system error number 13 in a file operation.
2021-01-15  9:21:47 0 [ERROR] InnoDB: The error means mysqld does not have the access rights to the directory.
2021-01-15  9:21:47 0 [ERROR] InnoDB: Operating system error number 13 in a file operation.
2021-01-15  9:21:47 0 [ERROR] InnoDB: The error means mysqld does not have the access rights to the directory.
2021-01-15  9:21:47 0 [ERROR] InnoDB: Cannot open datafile './ibdata1'
2021-01-15  9:21:47 0 [ERROR] InnoDB: Could not open or create the system tablespace. If you tried to add new data files to the system tablespace, and it failed here, you should now edit innodb_data_file_path in my.cnf back to what it was, and remove the new ibdata files InnoDB created in this failed attempt. InnoDB only wrote those files full of zeros, but did not yet use them in any way. But be careful: do not remove old data files which contain your precious data!
2021-01-15  9:21:47 0 [ERROR] InnoDB: Database creation was aborted with error Cannot open a file. You may need to delete the ibdata1 file before trying to start up again.
2021-01-15  9:21:48 0 [ERROR] Plugin 'InnoDB' init function returned error.
2021-01-15  9:21:48 0 [ERROR] Plugin 'InnoDB' registration as a STORAGE ENGINE failed.
2021-01-15  9:21:48 0 [ERROR] Failed to initialize plugins.
2021-01-15  9:21:48 0 [ERROR] Aborting

Any ideas how to continue troubleshooting this?

This is related to #171.
You have to add two lines to your NextCloud values.yaml:
 mariadb:
   enabled: true
+  podSecurityContext:
+    fsGroup: 999

This is related to #171.
You have to add two lines to your NextCloud values.yaml:
 mariadb:
   enabled: true
+  podSecurityContext:
+    fsGroup: 999

Yes that worked great.

Yes that worked great.

I fixed issue #171 - Thank you for testing/help!

I fixed issue #171 - Thank you for testing/help!


MNC tag in docker-compose does not modify the FQDN in spgw_u.conf

The MNC tag in docker-compose does not modify the FQDN in spgw_u.conf of the demo-oai-spgwu-tiny container
https://github.com/OPENAIRINTERFACE/openair-epc-fed/blob/afeb9d22770864fed00242544702e22258db637a/docker-compose/magma-mme-demo/docker-compose.yml#L157-L159
The following configuration
MCC: 208
MNC: 96
MNC03: '001'

Returns wrong FQDN value
SPGW-U =
{
    FQDN = "gw1.spgw.node.epc.mnc001.mcc208.openairinterface.org";

The workaround is to create share the file with the correct settings via the volume tag.
        volumes:
            - ./spgw_u.conf:/openair-spgwu-tiny/etc/spgw_u.conf

hello, can you show your file that you did the configuration?

@saccaco I had to manually insert the FQDN into the spgw-u-tiny container configuration file.
I'll leave an example:
SPGW-U =
{
    FQDN = "gw1.spgw.node.epc.mnc92.mcc208.openairinterface.org";
    INSTANCE                       = 0;            # 0 is the default
    PID_DIRECTORY                  = "/var/run";     # /var/run is the default

    #ITTI_TASKS :
    #{
        #ITTI_TIMER_SCHED_PARAMS :
        #{
            #CPU_ID       = 1;
            #SCHED_POLICY = "SCHED_FIFO"; # Values in { SCHED_OTHER, SCHED_IDLE, SCHED_BATCH, SCHED_FIFO, SCHED_RR }
            #SCHED_PRIORITY = 85;
        #};
        #S1U_SCHED_PARAMS :
        #{
            #CPU_ID       = 1;
            #SCHED_POLICY = "SCHED_FIFO"; # Values in { SCHED_OTHER, SCHED_IDLE, SCHED_BATCH, SCHED_FIFO, SCHED_RR }
            #SCHED_PRIORITY = 84;
        #};
        #SX_SCHED_PARAMS :
        #{
            #CPU_ID       = 1;
            #SCHED_POLICY = "SCHED_FIFO"; # Values in { SCHED_OTHER, SCHED_IDLE, SCHED_BATCH, SCHED_FIFO, SCHED_RR }
            #SCHED_PRIORITY = 84;
        #};
        #ASYNC_CMD_SCHED_PARAMS :
        #{
            #CPU_ID       = 1;
            #SCHED_POLICY = "SCHED_FIFO"; # Values in { SCHED_OTHER, SCHED_IDLE, SCHED_BATCH, SCHED_FIFO, SCHED_RR }
            #SCHED_PRIORITY = 84;
        #};
    #};

    INTERFACES :
    {
        S1U_S12_S4_UP :
        {
            # S-GW binded interface for S1-U communication (GTPV1-U) can be ethernet interface, virtual ethernet interface, we don't advise wireless interfaces
            INTERFACE_NAME         = "eth0";  # STRING, interface name, YOUR NETWORK CONFIG HERE
            IPV4_ADDRESS           = "read";                                    # STRING, CIDR or "read to let app read interface configured IP address
            #PORT                   = 2152;                                     # Default is 2152
            SCHED_PARAMS :
            {
                #CPU_ID       = 2;
                SCHED_POLICY = "SCHED_FIFO"; # Values in { SCHED_OTHER, SCHED_IDLE, SCHED_BATCH, SCHED_FIFO, SCHED_RR }
                SCHED_PRIORITY = 80;
                POOL_SIZE = 8; # NUM THREADS
            };
        };
        SX :
        {
            # S/P-GW binded interface for SX communication
            INTERFACE_NAME         = "eth0"; # STRING, interface name
            IPV4_ADDRESS           = "read";                        # STRING, CIDR or "read" to let app read interface configured IP address
            #PORT                   = 8805;                         # Default is 8805
            SCHED_PARAMS :
            {
                #CPU_ID       = 1;
                SCHED_POLICY = "SCHED_FIFO"; # Values in { SCHED_OTHER, SCHED_IDLE, SCHED_BATCH, SCHED_FIFO, SCHED_RR }
                SCHED_PRIORITY = 81;
                POOL_SIZE = 1; # NUM THREADS
            };
        };
        SGI :
        {
           # No config to set, the software will set the SGi interface to the interface used for the default route.
            INTERFACE_NAME         = "eth0"; # STRING, interface name or "default_gateway"
            IPV4_ADDRESS           = "read";                         # STRING, CIDR or "read" to let app read interface configured IP address
            SCHED_PARAMS :
            {
                #CPU_ID       = 3;
                SCHED_POLICY = "SCHED_FIFO"; # Values in { SCHED_OTHER, SCHED_IDLE, SCHED_BATCH, SCHED_FIFO, SCHED_RR }
                SCHED_PRIORITY = 80;
                POOL_SIZE = 8; # NUM THREADS
            };
        };
    };

    SNAT = "yes"; # SNAT Values in {yes, no}
    PDN_NETWORK_LIST  = (
                      {NETWORK_IPV4 = "<IP_ADDRESS>/24";} # 1 ITEM SUPPORTED ONLY
                    );

    SPGW-C_LIST = (
         {IPV4_ADDRESS="<IP_ADDRESS>" ;}
    );

    NON_STANDART_FEATURES :
    {
        BYPASS_UL_PFCP_RULES = "no"; # 'no' for standart features, yes for enhancing UL throughput
    };
};

The reason for this problem is that spgw-u-tiny's entrypoint script does not correctly create the FQDN passed by docker-compose.yaml in the container configuration file.
I recommend you look for more updated versions of these EPC components (unless you're looking for something specific from these versions), there are many unresolved bugs, it's been a few years since I've worked with this code anymore.
Best regards


Exclude dependency library for custom js.

I have a custom js below just few lines of set DataTables code. And set in mix :
mix.js('resources/js/a_type.js', 'public/js/a_type.js');
the output always include jquery library so the filesize is big. How to remove jquery from output js ?
\\ a_type.js
var table = $('#table').DataTable({
    order: [1, 'desc'],
    ajax: { url: '/vm/type/data', type: 'POST' },
    columns: [
        { data: 'id', name: 'id' },
        { data: 'name', name: 'name' },
        { data: 'created_at', name: 'created_at' },
        { data: 'updated_at', name: 'updated_at' },
    ]
})

Chain .autoload({}) call in your mix config.
Currently, ProvidePlugin kinda replaces $('#table') in your code with require('jquery')('#table').

I'm not entirely sure how to interpret your question, but you can either:

Remove the jQuery references in your source code.
Use mix.extract() to move jQuery into its own vendor file.

Thanks @adiachenko @JeffreyWay
Sorry I'm new to webpack, and @adiachenko explaination is my solution. Now if I have jquery sign $ in these js files :
mix.js('resources/js/a.js', 'public/js/a.js');
mix.js('resources/js/b.js', 'public/js/b.js');

will exclude jquery code when I comment the $: 'jquery' line.
```
new webpack.ProvidePlugin(Mix.autoload || {
    jQuery: 'jquery',
    // $: 'jquery',    <== here
    jquery: 'jquery',
    'window.jQuery': 'jquery'
 }),
```

Yes. It ensures that jQuery is available anywhere, if you reference it. If you don't want that, you can do mix.autoload({}).

I excluded jQuery from webpack.config and package.json and got conflicts with some plugins and npm run can't finish compiling with some jQuery UI components without jQuery. It's terrible.
My solution is use mix.combine instead of mix.js.


Enhance deleting of HTML tags, similar to vim-surround

Is it possible to replicate vim-surround behaviour when deleting HTML tags? Simple example:

typing dst will leave 2 empty lines and incorrectly indented

, like this:

doing same thing with vim-surround:

Hi there, just wanted to give an update that I am thinking about how to go about this. I originally had something in place that used the builtin equalprg, but this caused issues like #46. Perhaps I can try something out with only calling the format command if a file type is defined.

@rorj I don't really want to hard-code an exception case for HTML files into the plugin, but I also don't really know under what circumstances to indent/dedent. I'll keep trying to think of ideas though.

Not sure if this behaviour limited to HTML (very noticeable there for sure). I think vim-surround works similarly to blocks like this too:
{|
  test,
  anotherOne
}

ds{ or dsB here would also remove empty lines and change indentation.
From user perspective I probably never want empty (whitespace only) lines upon deletion and naturally I would expect contents indented automatically in this specific case.

Hi there, I've implemented (a really bad version) the feature you've requested via add-smart-indentation. The "heuristic" I'm using here is:
If deleting the delimiter pair yields lines that only consist of whitespace, then delete those lines, and dedent the inner contents of the surround.

I'm not sure how to get other people's feedback on how to handle the "empty" lines though---should they be deleted? I'm not 100% sure. I could just commit this and wait to see who complains but that seems sub-optimal.

@rorj I've now made it so that indentation always occurs when you surround with open/closing pairs, and dedentation always happens when you delete open/closing pairs and the lines are whitespace only. Please try the branch out and see what you think.

@andrewferrier Sorry for the tag, but what do you think about the previous comment? E.g. VSb on
test

will yield
(
    test
)

and dsb will revert to the original buffer contents. I played around with vim-surround a bit, and it seems that this exact test case is a bit inconsistent; more precisely, it seems that the indentation depends on the filetype. For the previous example, the test is indented when the filetype isn't set, but isn't indented when filetype is html or lua. Is this something where we want nvim-surround's behavior to diverge from vim-surround? I think vim-surround is using the builtin equalprg setting, but it seems like it causes weird behavior sometimes (see #46 when I tried it last).
CC: @Gelio If it's not a big deal, is it alright if I tag you once in a while to ask for your opinion on certain design choices like this? I don't really have many people that use this plugin that I can ask for feedback from, but I would like to do "what feels natural" to most users.

I think this rule of "always indent/dedent" makes a conceptually simple model for the end user, but I'm unsure if there's any sort of behavior that old vim-surround users rely on that I'm unaware of. One of the things I could do is to just push these changes and see if people complain, but I'd like to keep that to a minimum. I'd also appreciate any suggestions on how I can get more community/user feedback on these sorts of design-level decisions!

I'd say you should just commit to it and collect feedback that way. It's still early days for this plugin after all :)

CC: @Gelio If it's not a big deal, is it alright if I tag you once in a while to ask for your opinion on certain design choices like this? I don't really have many people that use this plugin that I can ask for feedback from, but I would like to do "what feels natural" to most users.

Sure, no problem, I appreciate being asked for an opinion :smile:

My opinion on this feature is that it seems entirely optional. I use an automated formatter for all languages that I work on regularly. For HTML, I use prettier. From my perspective, the exact formatting that nvim-surround leaves in the buffer is going to be changed anyway by prettier after I save the file. Thus, the nvim-surround formatting is only until I save the file. That was also the way I worked with vim-surround - when the formatting from various buffer operations got hard to read, I saved the file/invoked :lua vim.lsp.buf.formatting() and the formatter got the file into a state that has consistent formatting.
So, all in all, to me this feature is entirely optional. If it complicates the code a lot, I probably would skip adding it and tell users to use an automated formatter instead :sweat_smile:

Agree with @Gelio ; don't overcomplicate this logic. I also would use an auto-formatter. And @jonatan-branting is correct also ;), none of what you're talking about here is functionally wrong, so you could commit and wait for the complaints to roll in, it's not hard to revert after all! (And eventually you could move to a release model anyway so those who want stability can pin themselves to a branch).

@rorj Is there a particular reason why you're not using a formatter? Or any particular use case that you have in mind in which the "smart indenting" would be more helpful to you here?

To be honest, I don't follow how using prettier or any other code formatter related to this. I do use it, not on every project though. There are also instances when I edit random code like this, which is not strictly HTML:
const str = `

`

Regardless, if you use prettier or just manually fix formatting yourself, both are the same in my book — unnecessary extra step.
I've tried mini.surround over the weekend and it implements this feature. If it's not something you want to add to your plugin to keep things simpler or to not break existing behaviour that is understandable.

@jonatan-branting I think formatprg is probably not the move here, as by default it seems to join lines until they hit textwidth; I'm trying to think if there are any issues with:

Always indenting when the delimiters are inserted on new lines
Always dedenting when the delimiters are deleted are on their own line

I tried using equalprg (using the = operator) before, but it had some weird issues. It's really not that much work to write my own indentation/dedentation functions, so I think I'll opt for that instead.

As a note to myself about possible "intended" behaviors:
int main() {
    cout << "Hello world!" << endl;
}

dsB -> int main() cout << "Hello world!" << endl;

Update: This issue is going to be merged into #92, but I think I've found out how to implement this properly, as :h indentexpr seems to be set for HTML files.

@kylechui I've tried add-smart-indentation and it works well. Will try add-more-keymaps later this week.

@rorj Sounds good; I don't think add-smart-indentation is a good indication of what things will look like since I basically just manually computed indent widths and "guessed" what the indentation level should be from there, whereas add-more-keymaps uses the builtin indentexpr where valid; if you have Tree-sitter installed then that will define an indentation-computing expression for you, resulting in better, more accurate indents. The only caveat (I can think of) is that things might not work as smoothly if no indentation expression is available, e.g. in a scratch buffer.


Wow

Hi @WIZZYplizzy could you please provide us with more details of the issue you've opened? Thanks!