Disable resolve.symlinks.

Hi all,
I have a non-monorepo use case where I'd like to import ES6 files from outside of the src folder via symlink, as recommended by a CRA error message.
Importing through a symlink gives me this issue, where the file is imported but not processed with the rest of the code under src/, as one would expect. My PR seems to solve it by treating symlinked files as if they were under src/, according to the well-received solution here.
Demo repo, with screenshots. The solution works at least for files under the project root.
Fixes #3547.

Bump ^ This is ridiculous... 5 Months with an approval, no conflicts. Likely needs to re-run this test....

5 months now....

any updates ？

My use case: I am working on an application that uses create-react-app, and we do not want to eject -- for one because there are strong warnings against it, and for a second reason is that we would never be able to replicate the knowledge that goes into it.
But, we also want to provide some of the components we create as open-source, which means there is times where I need to work with sources in both the app repository and the component(s) director(y|ies).
Right  now my workflow to achieve that is painful:

Build the component and run npm link, then remove the node_modules directory.
Link the component into the app, then do bits and pieces in the app. Try-out smaller changes directly by modifying the generated .js code.
Eventually switch back to the component: Run npm install to restore the node_modules directory. At this the app won't work anymore because of conflicts in react versions etc that lead to fun such as "invalid hook calls".
Build tests and storybook stories for all the things discovered in step 2, then implement them.
Move to step 1 to start a new cycle.

As long as the app anyways depends on all the stuff that the component depends that is going to work. It's going to stop working when the component has own dependencies, which one would have to install explicitly into the app's node_modules at that point.
I tried now to do something else: Create a dedicated "exposed" component directory which only has the generated files as well as package.json and the production dependencies, and npm link that instead.
In shell-script:
#!/bin/sh

exposed_directory=${PWD}/exposed

# Clean out the exposed directory and recreate it
rm -rf "${exposed_directory}" && mkdir -p "${exposed_directory}"

# Link the various pieces we need to make it look like our package
for f in src $(node -p '(require("./package.json").files || []).join("\n")'); do
	ln -s "../${f}" "${exposed_directory}/${f}"
done
# Hardlink files such as package.json to prevent smart tools trying to bork things.
for f in package.json; do
	ln "${f}" "${exposed_directory}/${f}"
done
(cd "${exposed_directory}" && NODE_ENV=production npm link)

npm install
npm run build -- --watch

This works ... IFF I apply the changes from this PR.
TL;DR: Please, pretty please, merge this? :)

Please consider merging this 🙏

What is this waiting for?

What is this waiting for?

Hi there, I was discussing this today with @iansu and we were wondering if this was actually the right solution to the problem.
We are also unsure of what side-effects this change might have (we haven't tested).
Instead of this change, I think we could instead improve where we check for folders outside of the src directory so that it allows for symlinked folders.
How does that sound?

@mrmckeb My use case is always about node_modules.nosync. On top of import and require on .[jt]sx? files, asset loaders, such as postcss-loader, translates the resulting css imports into real paths which also triggers the src jail.

Excuse my ignorance @vicary, do you mean you're using .nosync to stop node_modules from syncing to iCloud? That's what I found in Google.
If so, can you add some more detail around what how this change affects that?

Excuse my ignorance @vicary, do you mean you're using .nosync to stop node_modules from syncing to iCloud? That's what I found in Google.

The usage usually involves keeping both node_modules as a symlink, and node_modules.nosync as a directory in project root. Some modules/loaders always try to resolve the require path such that node_modules always become node_modules.nosync.
Say you import bootstrap in App.js,
import 'bootstrap/scss/bootstrap.scss';

It'll results in the following error,
Failed to compile.

./src/index.css (./node_modules.nosync/css-loader/dist/cjs.js??ref--5-oneOf-4-1!./node_modules.nosync/postcss-loader/src??postcss!./src/index.css)
Module not found: You attempted to import ../node_modules.nosync/css-loader/dist/runtime/api.js which falls outside of the project src/ directory. Relative imports outside of src/ are not supported.

If so, can you add some more detail around what how this change affects that?

So it's either asking all related projects to not resolve symlinks into original paths, or CRA somehow loosen the src jail a bit when the same path is actually resolvable in the allowed paths, aka node_modules, src... etc.

I've commented here but I'll reiterate here because I think this should not be merged. The symlinks: false is just a workaround for the real issue: that some source directories aren't processed by webpack when they should. The proper solution is to ensure those directories can be added into the include of the appropriate loaders, not to disable symlink resolution.
Disabling symlink resolution also has the side-effect of breaking pnpm because it uses a single package store for a machine and symlinks required packages into node_modules as appropriate (which is great if you want to avoid wasting 400mb of disk in node_modules for every CRA project you work on). Enabling symlinks: false means enabling a module resolution behavior that is not nodejs-compatible, and webpack should aim to be 100% compatible with nodejs with regards to module resolution rules.

I agree with some of the above comments—this is not the solution to the problem, so I’ve closed the PR. Reopen if you will but I’m sure we can solve the issue without breaking a package manager.


Loses characters on Internet Explorer 11

Description
Letters of input show up and then rapidly disappear at random when typing quickly on IE 11.
Steps to Reproduce
When using Internet explorer 11, locally using modern.ie VM and interacting with the first text box on https://mozilla-services.github.io/react-jsonschema-form/ .  A client initially ran into the issue running IE 11 on her desktop.
Quickly typed in: This is steven thompson
Expected behavior
Should have gotten This is steven thompson in the text box.
Actual behavior

### Version
Current

Have confirmed that it's a race condition between receiving the next keypress and the response from onChange() on upstream components propagating back down as formData.  Going to take a stab at unidirectional data flow by propagating the events upstream to the validator and rejecting at the validation stage while keeping the values local inside the components and still using formData to populate initial state.

Verified that it's a race condition getting data through the onChange propagators out to ; then propagating everything back up through the chain.  Profiling shows it's creating around 15k new React elements in my case - seems a little much.  Going to move those out of the render functions and re-render as needed - holler if you'd like a different approach.

Same issue here in meteor and browsing with chrome.
I just realized that the entry point of the package was "lib/index.js" which was not optimized.
Using import Form from 'react-jsonschema-form/dist/react-jsonschemaform'; instead solve my issue.

which was not optimized.

What do you mean, not optimized?

@israelshirk is the problem occuring with liveValidate set to false (which should be dthe default)?

AFK, I'll check this afternoon.  Thanks, Israel

On Oct 18, 2016, at 11:25 AM, Nicolas Perriault<EMAIL_ADDRESS>wrote:
@israelshirk is the problem occuring with liveValidate set to false (which should be dthe default)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

There were 2 build script according to package.json
"build:lib": "rimraf lib && NODE_ENV=production babel -d lib/ src/"
"build:dist": "rimraf dist && NODE_ENV=production webpack --config webpack.config.dist.js --optimize-minimize"
If I understand correctlly, the 'lib' one was just build ES6 into plain javascript and it's still human readable while the 'dist' one will bundle and optimize the code which brought a better performance.
However,
"main": "lib/index.js"
indecated that the entry point of the package was not the optimized one which was 'dist'.

@n1k0 The issue does seem worse with liveValidate turned on; I've been running into the issue with liveValidate = false.

There were 2 build script according to package.json

So the build:lib command builds the module files to be published for the npm package, as native es6 is not yet an option. The build:dist will indeed generate an optimized bundle, though that's not something you should import, really.
The typical supported workflows are:

To load the lib as a <script> import in some webpage;
Importing the npm package module in your project code, then generate your own optimized bundle.

The issue does seem worse with liveValidate turned on

This is expected as liveVaidate will validate the whole tree on each atomic change; we may want to move to atomic node validation in the future though.
For the initial issue, it seems that bidirectional event propagation between input and state gets in the way. We may want to consider buffering the event stream and chunking propagated change event data, but I'm not sure just yet. I'll think about it.

Hi,
I experiments the same issue with Firefox 50.1.0 .
With knilink solution I resolved the problem, but n1k0 seems to discourage this.
If I try to set onChange on the component (which I didn't set before), with function that does nothing, the behaviour seems to be less frequent.
For an input form on production this behaviour is not really acceptable.
How to work around the problem ?
Thanks,

Maybe we ned a way to do live validation only if nothing happen during the last 1 seconds because if the fields wait for the update before accepting any new characters it can cause this kind of problems.

Or maybe make it customizable

I tried noValidate property with the same result. Characters are loses when typing quickly. So perhaps the validation is not the cause of this issue.

So I updated node to v6. performed a total module re-installation. Isolated the Component and pass schema to via props. And I got the same behaviour. I have two form nested in another. I loses character in each form's first field. Moreover if I try my schema on playground it works perfectly... So the problem come surely from my code but I don't understand how. This my schema:
{
    "required": [
        "address",
        "client"
    ],
    "properties": {
        "address": {
            "properties": {
                "adresse": {
                    "type": "string",
                    "title": "Adresse"
                },
                "code_p": {
                    "type": "string",
                    "title": "Code postal"
                },
                "ville": {
                    "type": "string",
                    "title": "Ville"
                },
                "tel": {
                    "type": "string",
                    "title": "T\u00e9l\u00e9phone"
                }
            },
            "type": "object",
            "required": [
                "adresse",
                "code_p",
                "ville",
                "tel"
            ]
        },
        "client": {
            "properties": {
                "nom": {
                    "type": "string",
                    "description": ""
                },
                "code_externe": {
                    "type": "string",
                    "title": "code facturation"
                },
                "societe": {
                    "type": "string",
                    "title": "Soci\u00e9t\u00e9"
                },
                "email": {
                    "type": "string",
                    "title": "Email"
                },
                "siret": {
                    "type": "string",
                    "title": "SIRET"
                },
                "id_type_paiement": {
                    "enumNames": [
                        "Ch\u00e8que",
                        "Prelev",
                        "Virt"
                    ],
                    "type": "number",
                    "title": "Type paiement",
                    "enum": [
                        1,
                        2,
                        3
                    ]
                },
                "naf": {
                    "type": "number",
                    "title": "Code NAF"
                },
                "force": {
                    "type": "boolean",
                    "title": "force"
                }
            },
            "type": "object",
            "title": "Client",
            "required": [
                "nom",
                "societe",
                "email",
                "siret",
                "code_externe",
                "id_type_paiement"
            ]
        }
    },
    "description": "Cr\u00e9ation d'un client dans Admin2",
    "title": "Nouveau client",
    "type": "object"
}

I fetch this Schema with calling OPTIONS on url.
class CustomerSchemaFetcher extends React.Component {

    constructor(props) {
        super(props);
        this.state = {
            schema: null,
            error: null
        };
    }

    componentDidMount(props) {
        if (!this.state.schema)
            this.fetchOption()
    }

    fetchOption() {
        fetch(
            this.props.customer_url,
            {
                 credentials: 'same-origin',
                 method: 'OPTIONS'
             })
            .then(checkStatus)
            .then((response) => {
                const address = response.json()
                    .then((data) => {
                        this.setState({schema: data.methods.POST})})
            })
            .catch((error) => {
                this.setState({error: error.message});
                console.error(error);
            });
    }

    render(){
        if (this.state.schema)
            return <CustomerForm schema={this.state.schema}
                                 update={this.props.update}
                                 url={this.props.customer_url} />;
        else if (this.state.error)
            return <ErrorMsg msg={this.state.error} focusIt={true} />;
       else return null;
    }
}

class CustomerForm extends React.Component {

    constructor(props) {
        super(props);
        this.state = {
            error: null,
            data: {
                client: {
                    force: true}}};
        this.createCustomer = this.createCustomer.bind(this);
        this.change = this.change.bind(this);
    }

    change(form) {
        console.info("Change");
        console.log(form.formData);
    }

    createCustomer(form) {
        console.info("create new customer");
        console.log(form.formData);
        fetch(
            this.props.url,
            {
                 credentials: 'same-origin',
                 method: 'POST',
                 headers: {
                     'Content-Type': 'application/json',
                     'X-CSRFToken': getCookies("csrftoken")
                 },
                 body: JSON.stringify(form.formData)
             })
            .then(checkStatus)
            .then((response) => {
                const address = response.json()
                    .then((data) => {
                        this.props.update(data)})
            })
            .catch((error) => {
                console.error(error.response);
                error.response.json()
                    .then((data) => {
                        console.info('error is serializable');
                        console.log(data);
                        this.setState({
                            data: form.formData,
                            error: data.details.errors});
                    })
                    .catch((ex) => this.setState({
                            data: form.formData,
                            error: error.message}));
            });
    }

    render(){
        console.info("render with:");
        console.log(this.state.data);
        return (
            <span>
                { this.state.error &&
                <ErrorMsg msg={this.state.error} focusIt={true} />}
                <Form schema={ this.props.schema }
                      uiSchema={{
                          client: {
                              force: {
                                  "ui:widget": "hidden"
                              }
                          }
                      }}
                      ref="form"
                      liveValidate={false}
                      noValidate={true}
                      formData={this.state.data}
                      onSubmit={ this.createCustomer }
                      onChange= { this.change } />
            </span>);
    }
}

I put some logging outputs for understand but I have the same behaviour without.

In my case, another workaround was to return false in shouldComponentUpdate.
So I guess it would be great if uncontrolled formData is supported such as
<Form schema={...} onChange={...}/> without formData.
Or
`

I'm having the same problem with IE11, but interestingly, when I set it to emulate IE 10 everything is ok.

I am currently facing a similar issue with all my forms in all browsers. I can see multiple rendering of the input widget when I type quickly, but form onChange happen just after the multiple rendering. It definitely looks like a race condition. Continuing investigation as it's really a problem in my case (If I type "azeazeazeaze", I get "aaaaaaze" most of the time - in a form that have only 5 text fields). Pretty sure this can be related to custom widget performance somehow.

Pretty sure this can be related to custom widget performance somehow.

@MoOx dunno if it could be that, but there's a performance trick when it comes to passing function as props and bind them in the render function directly, then the functions are always treated as different, triggering rerenders.
A common trick is to bind them in the constructor, or to use this syntax:
class Foo extends Component {
  onChange = (event) => {
    this.whatever()
  }
  render() {
    return <input onChange={this.onChange}/>;
  }
}

But it could be a million other things. I'm happy you're investigating this as it's been reported a few times already.

@MoOx side question, does what you describe also happens with the playground? because it's using the trick mentioned above to circumvent unecessary rerenders. I can't reproduce the missing input events everybody seems to experiment with the playground on my box.

Was not able to reproduce with online demos in https://mozilla-services.github.io/react-jsonschema-form/
I already use the arrow method class property syntax, so this is probably something. Stay tuned ;)

@MoOx now I think more about it, there's also this trick with setImmediate https://github.com/mozilla-services/react-jsonschema-form/blob/master/playground/app.js#L132-L137

Why don't you
this.setState(
  {[name]: parseFloat(event.target.value)},
  () => this.props.onChange(this.state))
)

so you are sure this.state is up to date? This reminds me this https://github.com/mozilla-services/react-jsonschema-form/issues/315#issuecomment-256930949

Mostly because that would block the UI until the render is entirely finished, which would do exactly what we're trying to fight here... The safeRenderCompletion option of the setState helper does exactly this, which is useful in a test environment where you want to check things after they've been actually been rendered.

TBH it's been a while I didn't touch this area of the code, feel free to (re)experiment with all this, i'm pretty sure I got something wrong.

#480 may just have solved this; could anyone test latest master to see if it fixes the issue on their end? Thanks.

Going to try this today.

This fixes it for my IE11 VirtualMachine

Ok I'm closing this as this seems to be gone now tanks to #480.


refactor pattern match app

Refactored in an attempt to make it easier to understand.
If it's still too complex, it could be made into two apps:
1 Simplify this app so it has no delete button on each card.
2 Add a minimal app like this one from the docs, but with a delete button for each filter

Merging for now - will turn in into 2 apps in a future PR


fetch sites

@jiwalker-usgs want to do a quick review? I'll start on the process script.

going to switch this out to use the discharge data


DFBUGS-461: [release-4.16-compatibility] Fix security issue: Backtracking regular expressions

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2311005

it looks like, it needs bipul/sanjal approval to merge

/jira refresh

/lgtm


Should have in-app routing

Current behaviour
App is using anchor tag which makes the page reload when clicking in app link
Expected Behavior
App should use in-app routing so that page will not refresh
Addition Information
@hsluoyz please let me know, I would like to work on it

@shakti97 I have already created a PR referencing this similar issue.
#12


redundancy  on gene page hide redundant terms

here, cyclin dependent protein kinase activity can be hidden in default view because it has no unique extensions.

Is this the same as #45?

yep dupe


Issue Related to Badge and notification bar - Notification title and text

Expected Behaviour
Push notification without sound on foreground and in notification bar if app is not running with message and application name with sound. Like -
App  Name
you have push Notification
Badge should get clear when app will open.
Actual Behavior
Foreground - Push notification is appearing as expected without sound
Background - I'm getting two types of notification - Either with application name or with the text which I'm pushing from server.(Please see the image)
When Notification is coming with app name at that time sound is also coming.But when it is coming with message , Phone is only vibrating sound is not coming some times it is not vibrating also.
When it is coming with application name it is appending in notification bar (as In Image) but when it is coming with message it is overwriting the previous one.
Badge - It is appearing properly on application Icon as per my requirement but it is not getting clear same as -> Badge is shown on iOS even though it was not configured #1141
Reproduce Scenario (including but not limited to)
I have followed the following tutorial and used the same coding as in this tutorial.
http://phonegappro.com/tutorials/apache-cordova-phonegap-push-notification-tutorial-part-2/
Steps to Reproduce
cordova create test com.test.testapp testapp
cordova plugin add phonegap-plugin-push --variable SENDER_ID="721393051486(I used my Sender Id)"
cordova platform add android.

Import project in android studio, and added required coding.
the following code I have added at a particular user action (eq: - After successful login) -
document.addEventListener("deviceready",onDeviceReady,false);
function onDeviceReady(){
console.log("Device Ready")
var push = PushNotification.init({ "android": {"senderID": "721393051486"},
"ios": {}, "windows": {} } );

push.on('registration', function(data) {
console.log(data.registrationId);
});

push.on('notification', function(data) {
alert(data.title+" Message: " +data.message);
});

push.on('error', function(e) {
console.log(e.message);
});
}

Platform and Version (Android 4.2 )
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Samsung Tab
Cordova CLI version and cordova platform version
cordova --version - 6.3.0
cordova platform version android # e.g. 5.2.1
Plugin version
cordova plugin version | grep phonegap-plugin-push   - . 1.8.0
Sample Push Data Payload
I'm sending push notification from POSTMAN  using SAP HCPms Connection URL and Connection ID.
Header in Postman  (POST CALL)-
authorization: Basic YXNsdfsdasdfsaTAdfajc2dfa==
X-SMP-GCM-DATA: TEST NOTIFICATOIN
Body -
{
 "alert": "test push notification Pending approval",
  "badge": 5,
 "data": "testData",
 "sound": "soundval"
}

Response-
Stautus - 201 - Created.
(And some other information)
Sample Code that illustrates the problem
Logs taken while reproducing problem

@bhuvneshg21 wow, you have a conflagration of issues here which I will attempt to address. First, I believe most of the problems you are running into are because the push data you are sending is not in the correct format for Android. I extensively document all of the various payload options. You should read that over and you'll see the basic Android push payload should look like:
{
    "data": {
        "title": "Large Icon",
        "message": "Loaded from drawables folder"
    }
}

That is how you will get a title and body in your notification shade.
If you don't want the last push to be replaced you need to provide a notification ID.
Your badge isn't being cleared as that option is default by false.
So, please read over the docs I've linked to and let me know if it resolves your issues.

@macdonst Thanks a lot for your immediate response.
I have gone through with all the links provided in your reply. It was realy helpful.
I want to make few points here.
As per my Understanding of the following documnet provided by SAP. We have very only one option for payload.
https://help.hana.ondemand.com/hana_cloud_platform_mobile_services/frameset.htm?aaec2dbe78ec4fc08ef0a605a899e3dd.html
To resolve the Issue, I have done nothing I just changed the header and body(Payload) in POSTMAN.
Now I'm sending -
**-  HEADER -
content-type :application/json,
cache-control: no-cache,
authorization :Basic YXNoa2FyY

BODY -
{
"alert": "You have a pending Approval1",
"data": "Data",
"sound": "default",
"badge": "1"
}**

The Issue still  I'm facing is,  the very first notification is coming with application name (means with no message). Afterwards it is coming fine(With Title,message,sound).
Could you please put some light, What may be the issue.

@bhuvneshg21 well you are sending the push through SAP so I'm not too sure what they could be doing to the payload and I don't have time to test something like that. Seems like something SAP support could help with.
That being said try the following two thing, 1) use one of the node-gcm scripts I have provided in the PAYLOAD.md section to test the behaviour and 2) run adb logcat | grep PushPlugin to gather some logs while testing via SAP and let me see what actually shows up on the device.

Dear @macdonst , I got stuck with a high priority issue, that's why not getting time to test or capturing logs. But definitely I'll do . After that I'll open this issue again.
Thanks for your valuable help.


Template for Metrics don't run with metrics 1.1 (Microprofile >= 1.3)

use case :

generate demo application with MP Metrics
start demo application
call on /metric/increment. It's OK
call on /metric/increment. It's KO

For second call, i obtain

Exception émise par la classe d'application 'com.ibm.ws.microprofile.metrics11.cdi.MetricResolver11.checkReusable:104'

In fact, MP Metrics 1.1,

Annotations and Metadata can now have a flag reusable that indicates that the metric name
can be registered more than once. Default is false as in Metrics 1.0

This problem is usage of gauge and scope RequestScoped

If this behaviour is required, then it is possible to mark all such instances as reusable by passing the
respective flag via Metadata or field in the Annotation. Gauges are not reusable

.

Hi @lilian-benoit
I think there are 2 different types of 'reusing' a metric.

Define the same metric on 2 different methods
Define the metric only once but the scope of the resource makes that registration happens multiple times.

The latter is not a reuse and can be detected to keep track where the metric is initially found and thus registered. The scenario you describe does not fail for all implementations. So there is a different interpretation of the current spec based on the implementations.
Each JAX-RS resource is by default 'request scoped' and thus forcing developers to use ApplicationScoped for this is against the nature of JAX-RS.
I created also the following issue in the repo https://github.com/eclipse/microprofile-metrics/issues/364

@Gauge must indeed be in an @ApplicationScoped bean.


Cursor Paginator $paginator->onFirstPage() return false on the Homepage

Laravel Version: 10.4.1
PHP Version: 8.2.3
Database Driver & Version: MySQL

Description:
I am using Cursor Pagination on my application with the default pagination view. It normally works when I click the button Next  on the first page (Homepage), and then I click the button Previous back to the first page, but I still can click the Previous button one more click.
Steps To Reproduce:
I try to debug the value of  $paginator->onFirstPage(). The first visit homepage returns true, and I click on the Next button. It returns false, and then I click Previous to go back to Homepage, it returns false, and I click Previous again, it returns true.

Hey there,
Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to open up a new issue with a link to the original one and we'll gladly help you out.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!


create Archaisms.yml

source: https://github.com/errata-ai/proselint/blob/master/proselint/Archaisms.yml

Linting with vale failure on empty input is a know bug: see https://github.com/errata-ai/vale-action/issues/39.


Internal server error when trying to follow followed event

When I try to refollow the event,  endpoint gives me 500 internal server error. Can we fix this issue? Also, I know that this refollow request is not a valid one.

I'm aware of this problem. I'll inform you from this issue when it's fixed.

After #156 is merged, this issue will be fixed. When you try to follow an already followed event, API will return the existing follow record in the db.

I've just released the fix.


csv / tsv export - only one table is exported

Hi Jonathan,
the example file examples/biochemical_models/data.xlsx contains three data tables, but when I convert it to csv or tsv, e.g. using
obj-tables convert schema.csv data.xlsx data.csv
the output file contains only a singel table (Reaction), which is the last table in the original xlsx file. I think by default, all three tables should be exported as one multi-table csv / tsv file. Ideally, the tables should appear in the same order as in the original file, but that's only for convenience - if it's difficult to implement this, the order may also change.
Thank you!
Best,
Wolf

The data is output as a collection of files. The output file pattern has to have a glob pattern (*):
obj-tables convert schema.csv data.xlsx data-*.csv

Perfect, thank you!
One more thing: now the table of contents is also exported. Since this table exists rather for users' convenience in excel and does not contain additional relevant information, can you suppress its export?
Thank you!

The Python methods already have an argument which controls this. I can expose this to the command lien program and REST API in #103.


compton as a service causes gray screen of death

Issue description
I have a system where I use xomand+none. And had compton on as a service in configuration.nix. Everything worked fine until I added terminator to my systemPackages. After doing so, and rebooting, my system was a noninteractive gray screen of death after login. Removing terminator restored the function of the system.
you can see the configuration.nix file here: http://lpaste.net/6822723740615311360
After some help from clever on IRC, and poking around the journal. I moved compton from being a service to being a package, and now terminator causes no problems.
Weirdly, if I installed terminator ad hoc, it worked, until I reboot (giving the gray screen of death). Forcing me to

login as root via xterm
login as myself via su - isaac
remove the ad-hoc package
reboot
login as me again
reinstall ad hoc so I can use terminator
Oy!

Here is the journal for one such frustrating session: http://lpaste.net/5291960017220534272
Steps to reproduce

Build a system with the above configuration.nix.
Login as isaac.
Witness gray screen of death.
Logout with Mod+shift+q
Login as root with xterm
edit the configuration.nix and remove terminator
run nixos rebuild switch
reboot
Login as isaac
Witness that things work.
edit configuration.nix so that compton is a package instead of a service, and put back terminator
run nixos rebuild switch
reboot
login as isaac
Witness that things work.

Technical details

System: nixos 17.03
Nix version: 1.11.8
Nixpkgs version: "17.03.873.0c041520c3"

Any news on this issue?


Rename soft shadow quality project settings for easier searching

rendering/quality/shadows is now rendering/quality/point_shadows to explicitly denote that the settings only affect point light shadows (OmniLight and SpotLight), not directional light shadows.
Soft shadow quality settings were renamed to contain the word "filter". This makes the settings appear when searching for "filter" in the project settings dialog, like in Godot 3.x.
See https://github.com/godotengine/godot/issues/54161#issuecomment-1094361249.
Preview

Is the naming "Point Light" commonly used anywhere else in the engine? I seem to remember there being a great amount of disagreement over calling OmniLights and SpotLights "Point" lights because they can have size, so neither are truly point lights.

Is the naming "Point Light" commonly used anywhere else in the engine? I seem to remember there being a great amount of disagreement over calling OmniLights and SpotLights "Point" lights because they can have size, so neither are truly point lights.

Light2D was renamed to PointLight2D when DirectionalLight2D was added. While it's not 100% correct, I feel it's a decent enough approximation to refer to both omni and spot lights at the same time. ("Punctual lights" might be more common, but it also implies that the light is an infinitely small point if you read the wording as-is.)

Light2D was renamed to PointLight2D when DirectionalLight2D was added. While it's not 100% correct, I feel it's a decent enough approximation to refer to both omni and spot lights at the same time. ("Punctual lights" might be more common, but it also implies that the light is an infinitely small point if you read the wording as-is.)

Maybe we should call them "positional" lights to contrast with directional lights? I don't like introducing a nomenclature in the project settings that we have rejected elsewhere.

Maybe we should call them "positional" lights to contrast with directional lights? I don't like introducing a nomenclature in the project settings that we have rejected elsewhere.

"Positional" sounds good to me :slightly_smiling_face:

I think this PR probably also rename the RenderingServer APIs that deal with ShadowAtlas and rename most things there as PositionalShadowAtlas/positional_shadow_atlas naming.

Looks mostly good. But I question whether users are more likely to search for the word "filter" or "soft shadow" when they want to adjust the quality of the soft shadows.
In 3.x we only ever used percentage closer filtering. We didn't have proper soft shadows, so the name shadow_filter was more appropriate.
I lean towards including soft_shadows in the name of the soft shadow settings as the move to a new version is a good opportunity to move towards more accurate and easy to understand naming.

Looks mostly good. But I question whether users are more likely to search for the word "filter" or "soft shadow" when they want to adjust the quality of the soft shadows.
In 3.x we only ever used percentage closer filtering. We didn't have proper soft shadows, so the name shadow_filter was more appropriate.
I lean towards including soft_shadows in the name of the soft shadow settings as the move to a new version is a good opportunity to move towards more accurate and easy to understand naming.

Can we settle on Soft Shadow Filter Quality? :stuck_out_tongue:

I renamed the shadow quality settings to "Soft Shadow Filter Quality" in the end.

Thanks!


Report "Dolby Multichannel PCM 5.1" input format

This PR adds the "Dolby Multichannel PCM 5.1" to the list of input formats SoCo can report on.
Tested on a Beam Gen 2, input source was the Disney Plus.

LGTM -- thanks @ericbriffa.


fix path syscall-args search

Description/Motivation/Screenshots
fix syscall-args path search
How Has This Been Tested ?
"Tested" indicates that the PR works and the unit test (i.e. make test) run passes without issue.

[x] x86-32
[x] x86-64
[ ] ARM
[ ] AARCH64
[ ] MIPS
[ ] POWERPC
[ ] SPARC
[ ] RISC-V

Checklist

[x] My code follows the code style of this project.
[x] My change includes a change to the documentation, if required.
[x] If my change adds new code,
adequate tests have been added.
[x] I have read and agree to the
CONTRIBUTING document.

From @therealdreg
Repro:
root@fr33project:~# bash -c "$(curl -fsSL https://gef.blah.cat/sh)"
root@fr33project:~# wget -q -O- https://github.com/hugsy/gef/raw/main/scripts/gef-extras.sh | sh

Re-posting here for tracking:

I could reproduce your issue. there is a problem, but your PR doesn't solve it, it 's just a side effect of it. the problem is that the setting is overwritten by its default value when it shouldn't be
basically all values are overwritten by their default
tbh that may be tricky to solve quickly, so what I'd suggest you is simply make a gef restore to force reloading the .gef.rc

This PR fixes the visible side-effect rather than the root cause of the issue. One possible way to address this would be to force gef restore once the setting gef.extra_plugins_dir is changed to a different value. This will fix the issue this PR tries to fix, because the setting is originally correctly set by the gef-extras.sh install script.

thx for your research @hugsy


audio open error

Hello, so i download the package and added your example, but i get the following error. Any idea on how to reslove this. I'm unable to figure the arecord command that is getting launched in this package.
Got SIGNAL startComplete
Received Info: ALSA lib pcm_hw.c:1700:(_snd_pcm_hw_open) Invalid value for card
arecord: main:722: audio open error: No such file or directory
recording audioProcess has exited with code = 1

If you are on Linux, what is the output of this?
$ arecord temp.wav

Please see the README file carefully:
This module depends on your machine having an installation of sox (Mac/Windows Users) OR ALSA tools for Linux. You can use this library to record from any microphone device. Before installing and experimenting with the mic module, you need to ensure that you are able to capture audio via the command line:
$ arecord temp.wav

To get ALSA tools on Raspberry Pi running raspbian, try the following:
$ sudo apt-get update
$ sudo apt-get upgrade
$ sudo apt-get install alsa-base alsa-utils

After the above is tested and validated, you can proceed to install the module using:
$ npm install mic

@ashishbajaj99 Yes, i have th libs installed.

I had a similar issue. In my case I found that I needed to specify the correct device. I'm using a USB audio capture device.
To list available devices run this,
arecord --list-pcm

The value I used was, "plughw:CARD=system,DEV=0" which you can pass to the mic() constructor like so,
var micInstance = mic({
    device: "plughw:CARD=system,DEV=0",
    rate: '16000',
    channels: '1',
    debug: true,
    exitOnSilence: 6
});


dlb: optimize auto detect logic

Signed-off-by: Loong<EMAIL_ADDRESS>The default of type uint32 in protobuf is 0, so the else field can not run into.
Fix the above issue and add tests to cover some cases.
DLB mock functions are in development, so only test the config logic for now.
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@adisuissa @wrowe could you help review?

@adisuissa I have refactored it and added new tests, please review and merge if possible.


Add a composition helper

Sometimes things get super nested, and it's hard to keep track. Would be nice if we had compose() (just using Lodash flowRight()) that allows us to rewrite:
header('user-agent', startsWith(not(i('Google'))))
as:
compose(
	header('user-agent'),
	startsWith,
	not,
	i('Google')
)

or:
compose(
	header('user-agent'),
	startsWith
	i,
	not('Google')
)

Nah. Declining to do this for now.


Do not try to unmarshall response when http response code is 204

I'm not completely sure if it's correct semantics in my PR, but I had such bug: some GH API endpoints (contributors collection endpoint in my case) returns 204 No Content when there's no content, which leads to an EOF error. Probably it would be better to not to try unmarshall data to avoid error.
Code to reproduce:
package main

import (
	"fmt"

	"github.com/google/go-github/github"
)

func main() {
	client := github.NewClient(nil)
	_, _, err := client.Repositories.ListContributors("cloudfoundry", "em-posix-spawn", nil)
	if err != nil {
		fmt.Println("An error has occured:", err.Error())
	}
}

$ go run reproduce.go
An error has occured: EOF

LGTM.

so the real problem here isn't just when it's a 204 response, but rather when there is an empty response body, right?  For example, if I change the test case to return a 202 Accepted, it still causes an error, albeit a different one than the 204 response:
github_test.go:447: Do returned unexpected error: invalid character 'N' looking for beginning of value

So I think a better fix would be not to specifically look at the status code, but rather to verify that the body is non-empty, check the error returned from Decode, or something similar.  I'll play around with it and see what I can come up with.

err, I just noticed that you're using http.Error to return the status, so you're not actually returning an empty response body in the test... in your case, you're returning the body "No Content".  So that explains the invalid character 'N' message I was getting.
Though it still seems that this method is failing on 204 responses like you originally reported.

okay, I change this to specifically check for io.EOF errors coming from the json decoding call.  Merged as c9c9f3beeac50d26ec5440b8da68efeb9506d25e.  Thanks!


Nnvm

@javelinjs ccsgd is removed and succeeded by sgd_mom_update style single step optimizers. I removed ccsgd in scala. Please confirm the change or add it back as alias of sgd for backward compatibility

LGTM


Cloudwatch fails when setting interval

Any number but 900 will result in this
Config
cloudwatch {
namespace => "AWS/EC2"
metrics => [ "CPUUtilization" ]
interval => 400
filters => { "tag:Monitoring" => "Yes" }
region => "us-east-1"
}
Error
{:timestamp=>"2016-06-18T22:04:57.214000+0000", :message=>"Pipeline aborted due to error", :exception=>#<RuntimeError: Interval must be divisible by peruid>, :backtrace=>["/opt/logstash/vendor/bundle/jruby/1.9/gems/logstash-input-cloudwatch-1.1.3/lib/logstash/inputs/cloudwatch.rb:126:in `register'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/logstash-core-2.3.3-java/lib/logstash/pipeline.rb:330:in `start_inputs'", "org/jruby/RubyArray.java:1613:in `each'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/logstash-core-2.3.3-java/lib/logstash/pipeline.rb:329:in `start_inputs'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/logstash-core-2.3.3-java/lib/logstash/pipeline.rb:180:in `start_workers'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/logstash-core-2.3.3-java/lib/logstash/pipeline.rb:136:in `run'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/logstash-core-2.3.3-java/lib/logstash/agent.rb:473:in `start_pipeline'"], :level=>:error}

@jrgns

@aabed Hey, for some reason the error message is not being shown. It reads:
Interval must be divisible by period
If the interval is not divisible by the period, the metrics will be calculated for overlapping periods. The following will work:
  cloudwatch {
    namespace => "AWS/EC2"
    metrics => [ "CPUUtilization" ]
    interval => 400
    period => 200
    filters => { "tag:Monitoring" => "Yes" }
    region => "us-east-1"
  }

I see , thanks


Compiler: warn on deprecated macros

Depends on #9341
Ref: #9272
It forced me to go deeper into how the location in macros are traced. I added a Location#user_location which I thought it was the same as Location#original_location, but I was wrong. This will come handy on the implementation tool also. 🎉
The specs are ported from the codegen/warnings_spec that applies to methods.

Rebased on master with #9341 merged


-fix #238 #250 reinstancing is not performed on method (UK2Node_CallFunction or UK2Node_CSAsyncAction) nodes.

-properly reconstruct methods and async methods on c# recompile reload. this will also now pick up any changes to signatures and prevent blueprints from breaking since they reference their old types.

will fix #238 and #250

General comments to follow UE's guidelines:
const Type - not Type const
Don't use auto
PascalCase
Otherwise it looks good :D

Thank you for the PR! Great fix ⭐


What happens on a restart?

What happens when a krakend pod with auth/revoker setup. Gets restarted.
To my understanding as bloomfilter is in memory only. All data will get purged.
How do i ensure that on rollout/restart, blocked token doesn't become valid again?

You must emulate the Enterprise edition's behavior and push the bloomfilter again to all instances when they start. This component only takes care of the bloomfilter, and not any of the orchestration part.


bug: The WebChat container's role should not always be 'complementary' [screen reader accessibility]

Hi,

Relates to :~ #3655.

Screenshots

Version

WebChat.js version :~ 4.10.1;
JS :~ https://cdn.botframework.com/botframework-webchat/4.10.1/webchat.js;

Describe the bug
Sometimes a Chat-bot will be additional content on the Web-page, set apart from the main content. However, other times a Chat-bot will be the only or main focus of the Web-page.
In these cases role=complementary is not appropriate, and will confuse screen reader users, who may rely on landmark roles for navigation.

Steps to reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Give implementors / developers the option to modify the container-role via defaultStyleOptions.js:
   containerRole: 'complementary', // WAI-ARIA landmark role. Suggestions: 'complementary' or 'region'. Defaults to 'complementary'.

The value would be used in :~ https://github.com/microsoft/BotFramework-WebChat/blob/v4.11.0/packages/component/src/BasicWebChat.js#L71;
For example, role={containerRole}:
  return (
    <AccessKeySinkSurface ... ... >
      <TypeFocusSinkBox
        className={sinkClassName}
        disabled={disabled}
        ref={transcriptFocusRef}
        role={containerRole}
        sendFocusRef={sendBoxFocusRef}
      >
        // ...
      </TypeFocusSinkBox>
    </AccessKeySinkSurface>
  );

Please let me know what you think of this proposal.
Yours,
Nick

Additional context

[Bug]

Hi @compulim, can you review this issue?

Hi @compulim, can you review this issue?

Hi @compulim,
I'm working on a fix based on my proposal above ...  hopefully have a pull-request to follow ;).
Nick

Hi @compulim,
I'm working on a fix based on my proposal above ...  hopefully have a pull-request to follow ;).
Nick

Assigning to @nfreear. In the future, please provide repro steps as requested in the template.

Assigning to @nfreear. In the future, please provide repro steps as requested in the template.

I think it is better to put role attribute on the <ReactWebChat>, for example:
<ReactWebChat directLine={...} role="region" />

My arguments:

We do have dir attribute too, it is not using style options
It seems more natural and intuitive

For composition version of Web Chat, it will be on the <BasicWebChat> component.
<Composer directLine={...}>
  <BasicWebChat role="region" />
</Composer>

I think it is better to put role attribute on the <ReactWebChat>, for example:
<ReactWebChat directLine={...} role="region" />

My arguments:

We do have dir attribute too, it is not using style options
It seems more natural and intuitive

For composition version of Web Chat, it will be on the <BasicWebChat> component.
<Composer directLine={...}>
  <BasicWebChat role="region" />
</Composer>

Hi @compulim,
Thanks for the suggestion -- accepted.
Can I check my understanding? You're saying use the role in ReactWebChat.js:
  <BasicWebChat className={className} role={containerRole} />

Is that correct?
Thanks,
Nick

Hi @compulim,
Thanks for the suggestion -- accepted.
Can I check my understanding? You're saying use the role in ReactWebChat.js:
  <BasicWebChat className={className} role={containerRole} />

Is that correct?
Thanks,
Nick

Hi @compulim & @corinagum,
I have a question in my previous comment ... @compulim Are you able to answer so I can complete PR #3669, and get it reviewed please?
Thanks,
Nick

Hi @compulim & @corinagum,
I have a question in my previous comment ... @compulim Are you able to answer so I can complete PR #3669, and get it reviewed please?
Thanks,
Nick

It will become:
const ReactWebChat = ({ className, role, ...composerProps }) => (
  <Composer {...composerProps}>
    <BasicWebChat className={className} role={role} />
  </Composer>
);

When end-developer render Web Chat, their call will looks like:
ReactDOM.render(
  <ReactWebChat
    className="my-chat-ui"
    directLine={...}
    role="region"
  />
);

It will become:
const ReactWebChat = ({ className, role, ...composerProps }) => (
  <Composer {...composerProps}>
    <BasicWebChat className={className} role={role} />
  </Composer>
);

When end-developer render Web Chat, their call will looks like:
ReactDOM.render(
  <ReactWebChat
    className="my-chat-ui"
    directLine={...}
    role="region"
  />
);


Removing Chardet dependency (or switch to chardetng) to not use LGPL3

Hi,
I'm doing an "off the clock"/weekend project that I am hoping will become commercial (but it's far from it right now) and I want to use rc-zip because it works really really well for the zip files I'm using. However it's dependent on chardet which is LGPL (probably because chardet is LGPL) which means I can't use it in a commercial project (given how rust links the libraries).
Is there any way to make rc-zip not use chardet? For example using https://crates.io/crates/chardetng (not sure if it covers the right things)
To put my money where my mouth is I'm more than happy to donate 100USD for this effort via PayPal (or donate somewhere if you prefer). I know it's not much (especially to change a dependency, create a new version and upload to crates.io) but given that my project is miles away from generating any revenue at all it's all I can afford right now really :) I guess it's more a token of appreciation.
I know the right way is to submit a PR but frankly my rust skills are probably too limited to do this right.
Regards,
Niklas

I just looked into chardetng but it looks like it detects some Codepage 437 text as being Shift-JIS (to be fair, it doesn't look for CP437 at all):
running 1 test
test.zip: should be utf-8

                                             👇
ng detected Encoding { Shift_JIS } (confident? true), old detected ISO-8859-1 with confidence 0.73
cp-437.zip: should be cp-437

ng detected Encoding { Shift_JIS } (confident? true), old detected SHIFT_JIS with confidence 1
shift-jis.zip: should be shift-jis

test tests::real_world_files ... ok

So it seems like a no-go for now, sorry :(


Fix favicon link

I have no experience with "apple-touch-icon-precomposed", but just some notes:

According to https://mathiasbynens.be/notes/touch-icons the biggest icon should be 192x192, while we provide a 512 × 512 image
Do we need a touch icon?

We may want to support some icons for users who want to add a link to their mobile desktops. I don't really like it but it is kind of standard bad practice.
The question for me is: why only for Apple and specifically apple-touch-icon-precomposed?
These are all the icons required to support all platforms:
<link rel="shortcut icon" href="/favicon/favicon.ico">
<link rel="icon" sizes="16x16 32x32 64x64" href="/favicon/favicon.ico">
<link rel="icon" type="image/png" sizes="16x16" href="/favicon/favicon-16.png">
<link rel="icon" type="image/png" sizes="32x32" href="/favicon/favicon-32.png">
<link rel="icon" type="image/png" sizes="64x64" href="/favicon/favicon-64.png">
<link rel="icon" type="image/png" sizes="96x96" href="/favicon/favicon-96.png">
<link rel="icon" type="image/png" sizes="160x160" href="/favicon/favicon-160.png">
<link rel="icon" type="image/png" sizes="192x192" href="/favicon/favicon-192.png">
<link rel="apple-touch-icon" sizes="57x57" href="/favicon/favicon-57.png">
<link rel="apple-touch-icon" sizes="60x60" href="/favicon/favicon-60.png">
<link rel="apple-touch-icon" sizes="72x72" href="/favicon/favicon-72.png">
<link rel="apple-touch-icon" sizes="76x76" href="/favicon/favicon-76.png">
<link rel="apple-touch-icon" sizes="114x114" href="/favicon/favicon-114.png">
<link rel="apple-touch-icon" sizes="120x120" href="/favicon/favicon-120.png">
<link rel="apple-touch-icon" sizes="144x144" href="/favicon/favicon-144.png">
<link rel="apple-touch-icon" sizes="152x152" href="/favicon/favicon-152.png">
<link rel="apple-touch-icon" sizes="180x180" href="/favicon/favicon-180.png">
<link rel="manifest" href="/favicon/manifest.json">
<meta name="msapplication-TileColor" content="#FFFFFF">
<meta name="msapplication-TileImage" content="/favicon/favicon-144.png">
<meta name="msapplication-config" content="/favicon/browserconfig.xml">
<meta name="theme-color" content="#ffffff">

In my opinion, I would remove the OS specific favicons and use the standard.

This article is really interesting about favicons: http://www.jonathantneal.com/blog/understand-the-favicon/ and it has a deep explanation (and links) about it.
The article recommends something like this (but it is worth reading to adapt it):
<link rel="apple-touch-icon" href="path/to/touchicon.png">
<link rel="icon" href="path/to/favicon.png">
<!--[if IE]><link rel="shortcut icon" href="path/to/favicon.ico"><![endif]-->
<!-- or, set /favicon.ico for IE10 win -->
<meta name="msapplication-TileColor" content="#D83434">
<meta name="msapplication-TileImage" content="path/to/tileicon.png">
<link rel="apple-touch-icon-precomposed" href="apple-touch-icon-152x152-precomposed.png">

Thanks for the links @Xaviju - that seems a bit more manageable than all the different sizes in your previous comment.

Ok – let’s improve these further things in a follow-up. The thing is that now the link is simply broken cause of file renaming, which we should fix. :) Enhancements can be made but shouldn’t block a fix.


Multi-line hash comments

fixes #51 ! Adds a new option that enables escaped newlines at the end of hash comments to continue them.

Awesome! Thank you!

My only suggestion would be to note this change in the CHANGELOG. But that can be done in a follow up.


[ProjectManager][HaxeProject] Fixed compile error when output path of project ends with '' or '/'. fixes #1469

Checking this one.

Sadly this is not 100% the proper fix. OutputPathAbsolute is used in several places, some of them doing Path.GetDirectoryName() which returns "C:\parentPath" in the case of "C:\parentPath\path".
I guess you already noticed this being somewhat wrong because you put that Directory.Exists() check, and in this case FlashDevelop should create it.
The thing is that if the directory doesn't exist this fix doesn't work and Haxe will still fail.
Looking at the code I would say this has to be fixed in HaxeProjectBuilder.DoBuild().

I put this check, because there may be a file

Thanks, fixed

I'm curious, have you tested this with Haxe 2+? I'm wondering if there is any Haxe version that requires the ending slash.

Haxe 2? No ;)

I'm wondering if there is any Haxe version that requires the ending slash.
I do not know, but I know that because of her there are problems:
https://github.com/HaxeFoundation/haxe/issues/6768
https://github.com/HaxeFoundation/haxe/issues/6212


update ANIXE's description

r? @nikomatsakis
(rust_highfive has picked a reviewer for you, use r? to override)


More info on Read-efficient

The two points below seem the same to me, which says to use both PartitionKey and RowKey. Please corret me if I am wrong.
"
Specify both PartitionKey and RowKey in your queries. Point queries such as these are the most efficient table service queries.
Use compound key values. The only keys you have are PartitionKey and RowKey. For example, use compound key values to enable alternate keyed access paths to entities.
"
Please elaborate on this on how it improves efficiency?
Consider storing duplicate copies of entities. Table storage is cheap so consider storing the same entity multiple times (with different keys) to enable more efficient queries.

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

ID: aee7c9f3-3906-182a-307b-f7ce3ef04fbe
Version Independent ID: 73003433-aaa7-0638-972f-e597a7d17aaa
Content: Guidelines for Azure storage table design
Content Source: articles/storage/tables/table-storage-design-guidelines.md
Service: storage
Sub-service: tables
GitHub Login: @SnehaGunda
Microsoft Alias: sngun

@PingPongSet Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

(Not the content author, or have anything to do with the official streams) - AFAIK, the Table Storage system automatically indexes your data using the Partition and Row keys. Other columns in your table are not indexed. So, if you query the table using at least one or both of your Partition/Row keys, the Storage system can locate the required row(s) more efficiently.
There are two caveats to this:

Partition and Row keys are STRINGS. Indexing and locating indexed strings is always more "costly" performance-wise.

You may see comparable or better performance if you query other columns (without using the P/R keys) that are of numeric types, since numeric types can typically be searched through faster than strings.

The difference in performance is not really "human-visible" unless you have tens of thousands of rows stored and you are hitting the store with multiple lookups.

#reassign: @tamram

@PingPongSet - Thanks for your feedback. The explanation provided by @sujayvsarma is accurate. We're closing this one out now - please let us know if you have further questions.
#please-close


[CORE] Drop redundant partial sort which has pre-project when offload sortAgg

What changes were proposed in this pull request?
As titile.
How was this patch tested?
UT

Run Gluten Clickhouse CI


Update CHANGELOG and versions.

This is a release commit before creating release tag.
The test in authentication updated according to the new hash, based
on the query string with sentWith param with updated version.
Resolves: OLPEDGE-2365
Signed-off-by: Oleksii Zubko<EMAIL_ADDRESS>

Codecov Report

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

@@             Coverage Diff             @@
##             master       #394   +/-   ##
===========================================
  Coverage   88.1377%   88.1377%
===========================================
  Files            72         72
  Lines          2470       2470
  Branches        271        271
===========================================
  Hits           2177       2177
  Misses          162        162
  Partials        131        131

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 17c3e4a...23bf3ee. Read the comment docs.


Optimize polyfit

Closes #5629

Use Variable instead of DataArray
Use reshape_blockwise when possible following https://github.com/pydata/xarray/issues/5629#issuecomment-960133879

[x] User visible changes (including notable bug fixes) are documented in whats-new.rst

cc @phofl

@aulemahal can you check that xclim's test suite passes with this please

Nice!

@dcherian I can confirm the test suite of xclim passes successfully when tested against this branch!


Student suggested that we leave the O in AIOs blank and develop them in the meeting

They felt that they often had to rework them completely after the meeting.

not gonna adopt this


add sol_arena_readonly (or sol_arena_stringshare or sol_stringshare)

Similar to sol_arena, this variant would keep single-instance of strings if they are read-only, saving memory.
This could be done as a new handle exclusive to read-only strings or change sol_arena from sol_ptr_vector to a sol_vector of sol_str_slice, returning the slice or its data (the data is duplicated if didn't exist in the stringshare, otherwise an existing reference is used).
In EFL we used this a lot and proved to be useful for stuff that keeps similar strings around (like key values used in our http), but the stringshare wasn't a local handle, but a global one. Shared strings were refcounted, otherwise they would never be deleted (arenas are often deleted in short time).
We use lots of read-only string in Soletta, be in HTTP parameters, be in string packets. Once they are created, they are not allowed to be modified. If we change string packets to use stringshare, for instance, we could save memory when forwarding packets (now we do get_string() -> new_string() -> packet_del(old), this would increase refcount, then decrease, keeping it at 1).

easy to implement, but I want to see what others think about it. @bdilly, @cmarcelo


Rename this project

Following on #1148 and #1204 I ask you to rename this project.
Bot comes from Robot.
And Robot means "forced laborer": https://en.wikipedia.org/wiki/Robot#Origin_of_the_term_'robot'
This would make this project complies with the most recent changes:
https://www.bbc.com/news/technology-53050955
https://www.zdnet.com/article/github-to-replace-master-with-alternative-term-to-avoid-slavery-references/
https://www.cnet.com/news/microsofts-github-is-removing-coding-terms-like-master-and-slave/
https://github.com/phpmyadmin/phpmyadmin/issues/16197
https://github.com/ibm-cloud-docs/RegistryImages/issues/51
Among many other.
I don't have a suggestion though.

when we change the terms master or slave, we change because there are communities that are offended by those terms. I don't know anyone who is offended by the term BOT.

@g13ydson the term "bot" literally means "forced laborer". "Forced laborer" is a "pretty" name for slave.
I'm offended by using a tool named "factory slave".

@davividal But the term is attributed to a humanoid. this forced labor is done by a humanoid, not a human.

The term master/slave is applied on a software. A software is a (state) machine. It is way more abstract than an humanoid slave ("robot"). Still, both are machines.
Still you are saying that I can't say that I'm going to setup a "master/slave mysql", but I can use humanoid slaves.
Forced labor is not ok. It does not matter who is making it.
What about "factory collaborator"?

Please read our code of conduct before contributing to this project. Our reasons for the previous rename are well documented, and we will not tolerate issues opened not in good faith.


Wrong analyzer error for identical constant instantiated tearoffs

The followin code produces an error in analyzer and no issues in CFE
class CheckIdentical {
  const CheckIdentical(Object? o1, Object? o2) : assert(identical(o1, o2));
}

X topLevelFunction<X>(X x) => x;

const c1 = topLevelFunction<int>;
const int Function(int) c2 = topLevelFunction;

main() {
  const CheckIdentical(c1, c2); // Analyzer error. Evaluation of this constant expression throws an exception.
}

Both of those function objects are obtained as a generic function instantiation with the constant type expression int as the type argument, based on two occurrences of the identifier topLevelFunction, and they should both evaluate to the same function object, a closurization of said function declaration, cf. https://github.com/dart-lang/language/blob/3e1d7084ecba96bc88a9db49532c2bba1ff2da15/specification/dartLangSpec.tex#L13107.
Tested on ``Dart SDK version: 2.15.0-150.0.dev (dev) (Fri Sep 24 21:55:32 2021 -0700) on "windows_x64"

This code works with the bleeding edge analyzer. I suspect that @srawlins fixed it.


Push master new

merge master branch with release/2.0

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.5 out of 6 committers have signed the CLA.:white_check_mark: lvjian0706:white_check_mark: w5688414:white_check_mark: lovejing0306:white_check_mark: LDOUBLEV:white_check_mark:<PHONE_NUMBER>:x: wugaoshengwugaosheng seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


My computer keeps saying this file contains a virus how do you fix this?

I need help if someone can help me out.

Its not a total problem that you can fix this injector lolskin code contains 3rd party program stuff thats why that error always pops up i dont think there is anything you can do to fix.

Windows Defender > Folder exclusion.


*: fix release-8.5 darwin build

What problem does this PR solve?
Issue Number: close #9647
Problem Summary:
What is changed and how it works?

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/hold
wait for my tests.

/unhold
builds for macOS passed.


Expose stats via prometheus

Hi, is it possible to expose the stats via prometheus

The monitor option just shows on console

Right now have done it this way
    const puppeteer = addExtra(vanillaPuppeteer)

    const cluster = await Cluster.launch()
    const clusterSize = cluster.jobQueue.size()
    clusterSizeMetric.set(clusterSize)

    const availableWorkers = cluster.workersAvail.length
    const busyWorkers = cluster.workersBusy.length
    const totalWorkers = cluster.workers.length
    crawlerWorkerQueueMetric.labels('available').set(availableWorkers)
    crawlerWorkerQueueMetric.labels('busy').set(busyWorkers)
    crawlerWorkerQueueMetric.labels('total').set(totalWorkers)

    const cpuUsage = cluster.systemMonitor.cpuUsage
    const memory = cluster.systemMonitor.memoryUsage
    crawlerClusterTelemetryMetric.labels('cpu_usage').set(cpuUsage)
    crawlerClusterTelemetryMetric.labels('memory_usage').set(memory)

https://www.npmjs.com/package/@devgo/puppeteer-cluster?activeTab=readme#clustergetmonitorobject
I've released it with what's in the PR and a few features I need, so check it out and do with it what you will.


Need to implement Follow feature from Suggestion components

completed this in https://github.com/Roman-Ojha/Social-Application/commit/eadee0b4d2d85361005d468329f7dbd8f8a04179


Shrine: top-right price on homescreen is missing background color, too close to app bar

From the spec:

From the polymer implementation:

Gallery:

(notice that the price should have a pink-salmon background color, and has more padding from the bottom of the app bar)

The real problem here isn't the location of the price. It's that the app bar (wrong background color) should  be rendered at elevation 0, with a one pixel border at the bottom.
The price is highlighted when the item ends up in the shopping cart.


chore(Dependencies) : update zio-logging-slf4j from 0.5.14 to 2.1.7

Updates dev.zio:zio-logging-slf4j from 0.5.14 to 2.1.7.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-logging-slf4j" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "dev.zio", artifactId = "zio-logging-slf4j" }
}]

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #561.


"Unknown template object" error with Handlebars 2.0

When using the prebuilt release version 0.16.1 in an app based on Ember App Kit, I see the following error:
Uncaught Error: Unknown template object: function               handlebars.js:455
 template                                                       handlebars.js:455
 hb.template                                                    handlebars.js:644
 (anonymous function)                                  liquid-fire-0.16.1.js:1858
 requireModule                                           liquid-fire-0.16.1.js:71
 __exports__.initialize                                liquid-fire-0.16.1.js:2488
 window.Ember.Application.initializer.initialize       liquid-fire-0.16.1.js:2550
 (anonymous function)                                               ember.js:3079
 Visit                                                              ember.js:2002
 DAG.topsort                                                        ember.js:2114
 Namespace.extend.runInitializers                                   ember.js:3076
 Namespace.extend._initialize                                       ember.js:2961
 Backburner.run                                                      ember.js:194
 run                                                               ember.js:17905
 runInitialize                                                      ember.js:2758
 jQuery.Callbacks.fire                                             jquery.js:3143
 jQuery.Callbacks.self.fireWith                                    jquery.js:3255
 jQuery.extend.ready                                               jquery.js:3467
 completed                                                         jquery.js:3498

We are using Ember 1.9.1 and Handlebars 2.0.
Are there any known issues when using Handlebars 2.0? I couldn't find anything in the open or closed issues.

Unfortunately, we can't do a single prebuilt version anymore that will work with any Ember version, because you need to choose the right template compiler to go with your Ember & Handlebars.
To make this work, we would need to produce separate builds for each Ember release. I'm happy to take a PR that achieves that, and I may eventually find time to do it myself, but I can't prioritize it right this moment.


Create BZ checklist for all issues with new feature label.

We should be creating regression tests for all issues, no matter whether they are a bug or a new feature. Accordingly, let's ensure that the BZ checklist for new feature even though we aren't today.
This is an internal issue though creating in App for visibility.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01632f7d35826f4e94
Upwork Job ID:<PHONE_NUMBER>829371392
Last Price Increase: 2023-09-19

Handling in the Expensify/Expensify issue.

Was just testing 👍

E/E issue is where we're working on this. This issue is open for testing. Will close it when the E/E issue closes.


Support for uncompressed and compressed fastq/fasta

Description of the bug
Passing uncompressed fastq in mikrokondo causes failures.
Also, would like to have support for uncompressed fasta files.
Command used and terminal output
No response
Relevant files
No response
System information
No response

fixed in: https://github.com/phac-nml/mikrokondo/releases/tag/0.5.0


Should call mapTileRequestCompleted instead

Should call mapTileRequestCompleted otherwise it will just keep expiring and requesting tiles.

Makes sense, so MapTileApproximater is designed to return a temporary "better than nothing" scaled image, not to provide raster scaling capability for tiles out of native zoom levels.
New issue here
https://github.com/osmdroid/osmdroid/issues/1519


chore: restrict vpn_ha version to not pick up 2.3.2

The release introduced with 2.3.2 is picked up by this module.
This PR ensures we don't pick up 2.3.2 as it's breaking and restricts tf version as well as introduces new variables.

@gtsorbo this fixes the release tag introduced by @g-awmalik here

/gcbrun


Add All Ledgers Endpoint to CLI

Expected Behavior
stellar ledgers

This should also take cursor, order, and page-limit as arguments.
Ledger details to display
id
hash

https://www.stellar.org/developers/horizon/reference/endpoints/ledgers-all.html

Interested in taking a stab at this -- aka since I am not an official member of this project, this is my way of saying "ASSIGN TO ME"
Also, I think it should stay this way. Will look better for when y'all apply for the next Stellar Build Challenge 😉

@robertDurst you can feel free to grab any ticket that hasn't been previously assigned.  But I'm happy to throw you on this one.  You are 100% a crucial contributor to the project!

@correlator id == hash 😆

Here is what I propose:
hash
sequence
transaction_count
operation_count
closed_at

Here is a preview:


🐛  Biome is configured to extend a nested config file, it doesn't work as expected

Environment information
CLI:
  Version:                      1.9.1
  Color support:                true

Platform:
  CPU Architecture:             x86_64
  OS:                           macos

Environment:
  BIOME_LOG_PATH:               unset
  BIOME_LOG_PREFIX_NAME:        unset
  BIOME_CONFIG_PATH:            unset
  NO_COLOR:                     unset
  TERM:                         "xterm-256color"
  JS_RUNTIME_VERSION:           "v20.16.0"
  JS_RUNTIME_NAME:              "node"
  NODE_PACKAGE_MANAGER:         "npm/10.8.1"

Biome Configuration:
  Status:                       Loaded successfully
  Formatter disabled:           false
  Linter disabled:              false
  Organize imports disabled:    false
  VCS disabled:                 true

Workspace:
  Open Documents:

What happened?
REPO for reproduce: https://github.com/rkovalov/biome-issue
Steps to reproduce

Clone this repository
Run npm install
Run npm run format:lint

###Actual
Biome doesnt extend the nested config file and run default config.
Expected result
Biome should support extending nested configuration files and automatically ignore tsconfig files defined in ./biome-config/base.json
Code of Conduct

[X] I agree to follow Biome's Code of Conduct

Duplicated: https://github.com/biomejs/biome/issues/1867


<EMAIL_ADDRESS>Fix URL

This PR fixes the download URL for the altair windows executable which was renamed to include _x64 in version 4.0.10.

Version
Filename

<=4.0.9
altair_4.0.9_win.exe

>=4.0.10
altair_4.0.10_x64_win.exe

In addition, the repository has been moved from https://github.com/imolorhe to https://github.com/altair-graphql.
Fixes #6882

/verify


it's breaking my pages

when i switch from my products page to another page everything breaks down
Images:
before

after

Hello @bckd00r, does the project use tailwind or was it installed on a non tailwind application ?
Can you share a link to test this out ?

Hello @bckd00r, does the project use tailwind or was it installed on a non tailwind application ?
Can you share a link to test this out ?

Yes im using with tailwind

Did you use this
import 'react-tailwind-table/dist/index.css';

?

@bckd00r since you already have tailwind installed on your project, you really would not be needing that line.
You can remove it.

not working


Puny encoded URLs used when creating a connection

When a connection is made to a channel with an IDN domain the connection editor (${baseurl}/conedit) are using puny encoded URLs instead of IDN encoded ones.
This is valid URLs in both "This connection's primary address is..." and "Available locations: ..."
Cheers,
Erik

As it seems the very same box is used in the top of channel settings (my own settings) with the very same problem. Let me know if I shall write a separate issue for that one.

We may need both forms showing on the settings page. This is the address you will give to others and they should be able to use this to connect with you. The issue is that other networks may or in many cases will not handle utf-8 hostnames or usernames (experimental support for punycoded usernames went in a few weeks ago). We can use this issue to track both, but I'm documenting via this comment that the needs may be slightly different.


KOLKOL by dexter21767

Addon Manifest URL
https://2ecbbd610840-kolkol.baby-beamup.club/manifest.json
Addon Description
stream everything over https with multiple subtitles
Language of Content
Multilingual (Default)
Choose Labels

[X] anime
[X] asian drama
[X] bollywood
[X] cartoons
[ ] debrid support
[X] http streams
[ ] live tv
[X] metadata
[X] movies
[ ] misc
[ ] music
[ ] nsfw
[ ] podcasts
[ ] radios
[ ] subtitles
[ ] torrents
[X] tv shows
[ ] video games

Why is it not working? It keeps loading all the time.

This addon doesn't really work.


Neural Network Extension, HIP GPU backend - add support for layers that use MIOpen

This PR adds support for NN layers that call MIOpen. These include convolution, deconvolution, fully_connected,
local_response_normalization, normalization, pooling, reshape, scale, softmax, tensor_add, tensor_max, tensor_min, tensor_multiply,
and tensor_subtract layers.

This PR has partial support for tensor_matrix_multiply. We would need to use rocBLAS for doing GEMM which will be added in the next PR for this layer.

@kiritigowda @asalmanp @rrawther Which version of MIOpen is needed to run everything in this PR?

@kiritigowda @asalmanp @rrawther Which version of MIOpen is needed to run everything in this PR?

Can you use the V2.11.0, this is what we are providing with current setup script

@kiritigowda @asalmanp @rrawther Which version of MIOpen is needed to run everything in this PR?

Can you use the V2.11.0, this is what we are providing with current setup script

@paveltc you need to run the MIVisionX-setup.py --backend=hip to install the miopen with the hip backend before building the mivisionx with the hip backend for testing this PR.

@asalmanp @kiritigowda I sent an email on Friday where I listed the errors that I got while running the setup script. Any updates on that?

@paveltc can this PR be merged?

@kiritigowda Working on running the tests now.

Unit test results
Batch Normalization: PASS
Local Response Normalization: PASS
Softmax: PASS
Tensor Add: PASS
Tensor Multiply: PASS
Tensor Subtract: PASS
Pooling: PASS
Scale: PASS
Convolution: Small mismatches between HIP and OCL layer results
Fully Connected: Small mismatches between HIP and OCL layer results
Deconvolution: Segmentation fault with OCL on PR619 branch, no segmentation fault on TOT. Mismatches between TOT OCL and HIP PR619.
PASS = OpenCL and HIP results match

@asalmanp
Tests update:
Reshape: PASS
Convolution: still small differences between HIP and OCL
Deconvolution: small differences between HIP and OCL. No more segmentation fault
Fully Connected: still small differences between HIP and OCL

Per @asalmanp request, adding a sampling of the differences for the layers mentioned.
Deconvolution
First bytes
OCL
0000000       -0.819574      -1.2741065      -1.5055298      -1.7213209
HIP
0000000       -0.819574      -1.2741066      -1.5055298      -1.7213209
Last bytes
OCL
0164260    -0.010193704     0.002807483    -0.013146199  -0.00017771152
HIP
0164260    -0.010193703    0.0028074845      -0.0131462  -0.00017771141
The rest of the examples only show first bytes of outputs
Convolution
OCL
0000000     -0.14978679     -0.14978679     -0.14978679     -0.14978679
HIP
0000000     -0.14978677     -0.14978677     -0.14978677     -0.14978677
Fully Connected
OCL
0000000      -0.2192885      0.44633734      -1.2363861      -0.6913732
HIP
0000000     -0.21928841      0.44633722      -1.2363861      -0.6913732

Per @asalmanp request, adding a sampling of the differences for the layers mentioned.
Deconvolution
First bytes
OCL
0000000 -0.819574 -1.2741065 -1.5055298 -1.7213209
HIP
0000000 -0.819574 -1.2741066 -1.5055298 -1.7213209
Last bytes
OCL
0164260 -0.010193704 0.002807483 -0.013146199 -0.00017771152
HIP
0164260 -0.010193703 0.0028074845 -0.0131462 -0.00017771141
The rest of the examples only show first bytes of outputs
Convolution
OCL
0000000 -0.14978679 -0.14978679 -0.14978679 -0.14978679
HIP
0000000 -0.14978677 -0.14978677 -0.14978677 -0.14978677
Fully Connected
OCL
0000000 -0.2192885 0.44633734 -1.2363861 -0.6913732
HIP
0000000 -0.21928841 0.44633722 -1.2363861 -0.6913732

@paveltc Thanks for providing the result. As discussed, the differences are in the range of 1e-7 or less. This should be fine.


RAbD data missing

Hi,
The file rabd_test.txt contains 60 pdb entries and that count is also reported in the paper. However, the corresponding file test_data.jsonl for RAbD contains 58 entries only. After looking through the pdb entries, I found that pdb entries '1w72' and '3h3b' are missing. Can you kindly provide the data for these two pdb entires as well.

Hi,
Sorry for the late reply. These two entries were missing because I
encountered errors when processing their PDB files into json format. I
don’t think I can provide these two
Wengong
On Sun, Apr 2, 2023 at 3:06 AM Tanuj Sur @.***> wrote:

Hi,
The file rabd_test.txt contains 60 pdb entries and that count is also
reported in the paper. However, the corresponding file test_data.jsonl for
RAbD contains 58 entries only. After looking through the pdb entries, I
found that pdb entries '1w72' and '3h3b' are missing. Can you kindly
provide the data for these two pdb entires as well.
—
Reply to this email directly, view it on GitHub
https://github.com/wengong-jin/abdockgen/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATGYLDCZSORXUXMI37BGI3W7EQPVANCNFSM6AAAAAAWQD45MM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Callback missing features in Step by Step

As it was reported in https://github.com/locaweb/locawebstyle/issues/1634

@gnxdev  thank you for Pull Request.
Please update your branch with our master, because I sent fix to travis.
Look the fix: #1638

@gnxdev Can you update your branch with our master? :point_up:

@gnxdev this PR still important? Can you update your branch?

Closed.


Autotranslation doesn't work on java

We can see this on javaTutorial.ipynb notebook.

thx.  this is dup of https://github.com/twosigma/beakerx/issues/5487 and https://github.com/twosigma/beakerx/issues/5039


Document how to use #[serde(bound="...")] to deal with common recursive cases

@killercup asked about this in IRC. It previously came up in #336.

From IRC:
[11:55:35]  <dtolnay>	try sticking #[serde(bound="")] on the recursive field
[11:55:54]  <dtolnay>	this overrides the generic type bounds that we put on the impl
[11:56:22]  <dtolnay>	it was probably generating something like "impl Serialize for Diagnostic where Vec<Diagnostic>: Serialize" which is recursive

I'll try to add this the docs later.


Why we can't have this in golint? vetshadow warning?

Why we can't have this in golint?
https://github.com/Microsoft/vscode-go/issues/1455#issuecomment-360305829

Thanks for the Issue.
This appears to be out of scope and so is being closed. If you feel this is in error please feel free to re-open.
Please note this is not the forum to discuss edits to CodeReviewComments. If you wish to propose changes to it please send a mail to golang-nuts.


Backup Camera Roll stuck at synchronisation

Describe the bug
After enabling camera roll backup in the settings, it starts to upload the pictures to the server and is able to create a respective folder on the server but gets stuck during upload and never finishes. No pictures are getting uploaded.
To Reproduce
Steps to reproduce the behavior:

I freshly installed the App and connected to Nextcloud and used a new Test folder to contain the lespas folder
I went into settings and enabled the camera roll backup slider and confirmed to upload the pictures (I tested it with different images)
In synchronisation status,  it gets stuck at uploading the image, no process at all for one picture after one hour
I checked  the nextcloud server site and the  DCIM folder gets created successfully but no images inside
I can still upload images manually to new albums in the App and they all sync fine even though sync status is still stuck at the camera roll backup process
The manual created albums also appear on the server site correctly

I also tried to reinstall the app again and use another clean lespas folder and used different images for upload or first having no images and enabling camera backup and then adding images afterwards but nothing helped and it always got stuck with the first upload for camera backup.
However,  everything else works perfectly and manually adding albums is all fine an  syncs instantly.
Expected behavior
The camera backup runs successfully.
Screenshots

Smartphone (please complete the following information):

App version 2.7.3
Device: FP4
OS: /e/os 1.8
Version: Android 12
Nextcloud AIO docker Setup version 25.0.4

Additional context
I can provide more information and logs if necessary :)

Also with single very small image no progress:

Frederik,
Thank you for reporting. If you can access your web server's access log, try grep 'LesPas' in it, find any line with response code 4xx and 5xx

Thank you very much for the fast response!
I have the Nextcloud AIO containers running and a caddy reverse proxy setup as recommended for Nextcloud AIO.
In the caddy logs, I only found this line, even though there must have been many more requests failing, so not sure if this is a relevant one, at least it is about the upload to the DCIM folder, which never worked:
{"level":"error","ts":1678270934.170098,"logger":"http.log.error.log1","msg":"readfrom tcp <IP_ADDRESS>:54644-><IP_ADDRESS>:11000: client disconnected","request":{"remote_ip":"<IP_ADDRESS>","remote_port":"58950","proto":"HTTP/2.0","method":"PUT","host":"cloud.bergrunde.net","uri":"/remote.php/dav/files/Freddy/Test/DCIM/fairphone_fp4/OpenCamera/Screenshot_20230308-103700_Lawnchair.png","headers":{"Authorization":[],"Content-Type":["image/png"],"Content-Length":["1449836"],"Accept-Encoding":["gzip"],"User-Agent":["LesPas_v2.7.3"]},"tls":{"resumed":false,"version":772,"cipher_suite":4865,"proto":"h2","server_name":"cloud.bergrunde.net"}},"duration":0.420701395,"status":502,"err_id":"hjahaz4p1","err_trace":"reverseproxy.statusError (reverseproxy.go:1281)"}

I also tested the upload function with an entirely different android smartphone running Android 8, and same problem here with my Nextcloud server.

Hm, now the app got stuck in this sync local changes mode, even though camera roll is disabled. Force stop and clearing cache did not help.

http response code 502 means Back Gateway, you should somehow check your web server's configuration. Can I access your server remotely? It's more easier to debug in this way.

Sorry, I guess I do not feel comfortable giving someone access to my server, so no problem if it is too tiresome for you to debug this issue, but thank you anyway :)
However if you feel like debugging, I have just analyzed the "real" caddy log (previously it was via docker log) and now I can see all the requests and errors that have occurred regarding lespas. I did a new fresh installation of lespas and removed every trace of lespas from my server and started the logging with the new first launch of the app on my phone. I mailed you the complete web server log containing only all entries related to lespas from this session.
I got the logs via tail -f cloud.log | grep -ie "lespas" | jq > lespas.log.
I tested the following:

First, in the app, I uploaded a picture manually to a new album "test" and this worked
Then, I took a photo with my camera and enabled in the settings the camera upload function and started a sync - gets stuck
I went back to main view and uploaded another image to the "test" album, worked again
I disabled the backup function again

Here are some errors, which might be interesting with status 404 (not found) and 409 (conflict)
Click to expand
{
  "level": "error",
  "ts":<PHONE_NUMBER>.119089,
  "logger": "http.log.access.log1",
  "msg": "handled request",
  "request": {
    "remote_ip": "<IP_ADDRESS>",
    "remote_port": "50830",
    "proto": "HTTP/2.0",
    "method": "PROPFIND",
    "host": "cloud.bergrunde.net",
    "uri": "/remote.php/dav/files/Freddy/Test/DCIM/fairphone_fp4",
    "headers": {
      "Cache-Control": [
        "no-cache"
      ],
      "Depth": [
        "0"
      ],
      "Authorization": [],
      "Accept-Encoding": [
        "gzip"
      ],
      "User-Agent": [
        "LesPas_v2.7.3"
      ]
    },
    "tls": {
      "resumed": false,
      "version": 772,
      "cipher_suite": 4865,
      "proto": "h2",
      "server_name": "cloud.bergrunde.net"
    }
  },
  "user_id": "",
  "duration": 0.126614073,
  "size": 252,
  "status": 404,
  "resp_headers": {
    "Vary": [
      "Brief,Prefer"
    ],
    "X-Xss-Protection": [
      "1; mode=block"
    ],
    "Cache-Control": [
      "no-store, no-cache, must-revalidate"
    ],
    "X-Powered-By": [
      "PHP/8.0.28"
    ],
    "X-Content-Type-Options": [
      "nosniff"
    ],
    "Alt-Svc": [
      "h3=\":443\"; ma=2592000"
    ],
    "Pragma": [
      "no-cache"
    ],
    "Set-Cookie": [],
    "Strict-Transport-Security": [
      "max-age=31536000;",
      "max-age=31536000;"
    ],
    "Content-Security-Policy": [
      "default-src 'none';"
    ],
    "Expires": [
      "Thu, 19 Nov 1981 08:52:00 GMT"
    ],
    "X-Robots-Tag": [
      "none"
    ],
    "X-Frame-Options": [
      "SAMEORIGIN"
    ],
    "Referrer-Policy": [
      "no-referrer"
    ],
    "Server": [
      "Caddy",
      "Caddy",
      "Apache/2.4.55 (Unix)"
    ],
    "Date": [
      "Wed, 08 Mar 2023 15:45:00 GMT"
    ],
    "Dav": [
      "1, 3, extended-mkcol, access-control, calendarserver-principal-property-search, nextcloud-checksum-update, nc-calendar-search, nc-enable-birthday-calendar"
    ],
    "Content-Type": [
      "application/xml; charset=utf-8"
    ],
    "X-Permitted-Cross-Domain-Policies": [
      "none"
    ]
  }
}
{
  "level": "error",
  "ts":<PHONE_NUMBER>.7107995,
  "logger": "http.log.access.log1",
  "msg": "handled request",
  "request": {
    "remote_ip": "<IP_ADDRESS>",
    "remote_port": "50830",
    "proto": "HTTP/2.0",
    "method": "PUT",
    "host": "cloud.bergrunde.net",
    "uri": "/remote.php/dav/files/Freddy/Test/DCIM/fairphone_fp4/OpenCamera/IMG_20230308_164241.jpg",
    "headers": {
      "Authorization": [],
      "Content-Type": [
        "image/jpeg"
      ],
      "Content-Length": [
        "581825"
      ],
      "Accept-Encoding": [
        "gzip"
      ],
      "User-Agent": [
        "LesPas_v2.7.3"
      ]
    },
    "tls": {
      "resumed": false,
      "version": 772,
      "cipher_suite": 4865,
      "proto": "h2",
      "server_name": "cloud.bergrunde.net"
    }
  },
  "user_id": "",
  "duration": 0.317427158,
  "size": 181,
  "status": 409,
  "resp_headers": {
    "Server": [
      "Caddy",
      "Caddy",
      "Apache/2.4.55 (Unix)"
    ],
    "X-Xss-Protection": [
      "1; mode=block"
    ],
    "Referrer-Policy": [
      "no-referrer"
    ],
    "X-Frame-Options": [
      "SAMEORIGIN"
    ],
    "Strict-Transport-Security": [
      "max-age=31536000;",
      "max-age=31536000;"
    ],
    "Pragma": [
      "no-cache"
    ],
    "Content-Type": [
      "application/xml; charset=utf-8"
    ],
    "X-Robots-Tag": [
      "none"
    ],
    "X-Powered-By": [
      "PHP/8.0.28"
    ],
    "X-Content-Type-Options": [
      "nosniff"
    ],
    "Cache-Control": [
      "no-store, no-cache, must-revalidate"
    ],
    "Expires": [
      "Thu, 19 Nov 1981 08:52:00 GMT"
    ],
    "Alt-Svc": [
      "h3=\":443\"; ma=2592000"
    ],
    "Set-Cookie": [],
    "Date": [
      "Wed, 08 Mar 2023 15:45:01 GMT"
    ],
    "X-Permitted-Cross-Domain-Policies": [
      "none"
    ],
    "Content-Security-Policy": [
      "default-src 'none';"
    ]
  }
}

And I am wondering, why lespas is trying to access this folder, which is not in my lespas folder and failing to access a .json file here which is not existent?

Click to expand
{
  "level": "error",
  "ts":<PHONE_NUMBER>.6095674,
  "logger": "http.log.access.log1",
  "msg": "handled request",
  "request": {
    "remote_ip": "<IP_ADDRESS>",
    "remote_port": "32928",
    "proto": "HTTP/2.0",
    "method": "GET",
    "host": "cloud.bergrunde.net",
    "uri": "/remote.php/dav/files/Freddy/Pictures/00_Handy/2023/2023_00_Freunde_N/791225.json",
    "headers": {
      "Cache-Control": [
        "no-cache"
      ],
      "Authorization": [],
      "Accept-Encoding": [
        "gzip"
      ],
      "User-Agent": [
        "LesPas_v2.7.3"
      ]
    },
    "tls": {
      "resumed": false,
      "version": 772,
      "cipher_suite": 4865,
      "proto": "h2",
      "server_name": "cloud.bergrunde.net"
    }
  },
  "user_id": "",
  "duration": 0.146007781,
  "size": 281,
  "status": 404,
  "resp_headers": {
    "Expires": [
      "Thu, 19 Nov 1981 08:52:00 GMT"
    ],
    "X-Xss-Protection": [
      "1; mode=block"
    ],
    "X-Content-Type-Options": [
      "nosniff"
    ],
    "Referrer-Policy": [
      "no-referrer"
    ],
    "Date": [
      "Wed, 08 Mar 2023 15:39:14 GMT"
    ],
    "X-Powered-By": [
      "PHP/8.0.28"
    ],
    "X-Frame-Options": [
      "SAMEORIGIN"
    ],
    "Content-Type": [
      "application/xml; charset=utf-8"
    ],
    "Alt-Svc": [
      "h3=\":443\"; ma=2592000"
    ],
    "Set-Cookie": [],
    "Cache-Control": [
      "no-store, no-cache, must-revalidate"
    ],
    "Content-Security-Policy": [
      "default-src 'none';"
    ],
    "Strict-Transport-Security": [
      "max-age=31536000;",
      "max-age=31536000;"
    ],
    "X-Permitted-Cross-Domain-Policies": [
      "none"
    ],
    "X-Robots-Tag": [
      "none"
    ],
    "Pragma": [
      "no-cache"
    ],
    "Server": [
      "Caddy",
      "Caddy",
      "Apache/2.4.55 (Unix)"
    ]
  }
}

As recommended by Nextcloud, I use the following reverse proxy setup with caddy:
<IP_ADDRESS> is my docker network gateway, and port 11000 is the one from the automatically created Nextcloud AIO Apache container.
cloud.bergrunde.net {
	tls {$EMAIL}

	header Strict-Transport-Security max-age=31536000;

	reverse_proxy <IP_ADDRESS>:11000

	log {
		output file /var/log/caddy/cloud.log {
		}
	}

}

With that much of detail you provided, a direct access won't be necessary.
I read the log you sent me. Found interesting difference how Caddy response to webdav PUT calls as to the other web servers, When Les Pas uploading a new file IMG_20230308_164241.jpg to a not existing folder .../Test/DCIM/fairphone_fp4/OpenCamera/, instead of a response code 404 like Apache does, it spills 409. That's why the camera roll backup process stuck.
Debug build with the fix is attached here. Could you please give it a try?
As for why Les Pas was peeking outside of lespas/ folder, it's because those are folders you've shared with others or shares you received from others, Les Pas was figuring out whether the share was published by itself or not by looking for a json file in it.

You are awesome, it is fixed with this new version! I did the same procedure as last time and, just for completeness, mailed you the logs again.
And thanks for your explanation with the "peeking", that makes sense now :)
Btw: Does this mean, that if I share a folder via Les Pas and on the receipts site, I move it to a different location on the server side, Les Pas will still find it and progress it normally. That is actually quite nice, such that one is not obligated to keep all the pictures within the shared_with_me folder :)
Btw2: It looks like the F-Droid version is behind the play store version. Do you know the reason? Probably due to F-Droid checking the new releases, right?
I will close the issue than, if you need more logs for debugging, let me know. Thanks a lot!

Looks lile #106 is related and might also be fixed with this.

No, the folder should stay in share_with_me, otherwise, things will break.
F-Droid pulls repo and builds on their own machine to make sure there's no modification to the source. That's why it's always several days behind.
You were using r2.7.3. r2.7.4 fixed #106, the latest is actually r.2.7.5 and fix to this issue will be included in upcoming r2.7.6


THRIFT-2941

https://issues.apache.org/jira/browse/THRIFT-2941

@jfarrell this was closed incomplete a while back.  The PR can be closed.


Integrated a Jekyll blog into our Rails app (Issue #51)

Ok guys, here's our Jekyll blog! I welcome any and all feedback so I can learn more.
Main points:

Guidance - I used this guide as my source for merging Jekyll into an already-existing Rails app. It was fantastic.
Documentation - I wrote a lengthy BLOG_POSTING_GUIDE.md that should cover how to use our new blog for new users. I assumed the audience would be familiar with our rails app but not with Jekyll.
Navigation links - I "hard-coded" some URLs as a work-around to merging Jekyll inside our existing app. All covered in the BLOG_POSTING_GUIDE.md.

No major show-stoppers, my only current issues as listed in the posting guide are:

the new rake jekyll task completes correctly (rebuilds the blog and published the HTML pages) but shows an error of rake aborted! Don't know how to build task 'jekyll'
for now, we may need to host images from other sites like Imgur. if anyone has experience on where to store and how to link to media inside our rails app, please let me know and I'll update the blog posting guide.

Thanks Chris (@ckib16)! Merged.


Video Description

is it possible to get the youtube video description?

You need to use the YouTube Data API for that.

how can I go about doing that? could you please explain to me becases as far as I know, we have to have the youtube player API for that.