text:
DEP Remove unused league/flysystem requirement

A bug in Flysystem was breaking the prefer-lowest build of silverstripe/assets, which led us to discover that the package isn't even used in framework. I've introduced a direct dependency on it in https://github.com/silverstripe/silverstripe-assets/pull/424.

Why is it a dependency of this repo and not assets?

I can only find one reference to Flysystem in the source for Framework, and it's an unused use statement 😂
That said, would shifting the dependency be considered a breaking change? For 99.999% of projects it's probably a non-issue, so maybe it's not even worth debating (since any Framework-only project should be defining the Flysystem requirement directly anyway.)

I've changed this PR to instead drop Flysystem from Framework. The only risk I can see is that a project could update to silverstripe/framework 4.7.0 without also updating to silverstripe/assets 1.7.0, leaving them without Flysystem. Two options:

Introduce a conflict with silverstripe/assets <1.7
Add a note about it to the 4.7 release notes, and assume developers will be able to get themselves out of the situation very easily

I think we used the library once in a project code to handle something project-specific since it came with the framework, but a note in the release notes would make us re-test it and include the dependency separately as a solution, so "2. add a note to 4.7 changelog" would suffice from our perspective.

An upgrade note is probably since very few people will upgrade framework but not assets. If for some reason this use-case comes up I'd probably be more inclined to release another 1.6.x tag of assets with the composer dependency included there.

One thing to note, @Cheddam - ideally we could include that upgrade note with this PR. As it stands, we don't generate those docs until the next release. In other projects people will sometimes have an "unreleased" document/section where these things can be collated as we go.
Is it worth raising with the people on the tools for releases whether this would be a good idea?

The 'process' for this at the moment is that the first person wanting to add something to the changelog creates it by hand, (hopefully) copying the template that Cow would use itself. This could obviously be more consistent, and we run the risk of missing details in the changelog when we leave writing it until release.
So, I've just talked to the team, and we're going to tweak our process to start creating a provisional changelog for the next minor immediately following the stable release of the previous one, so that we can ensure the template is used correctly, and that it's always available for contributors to add to when they land a change.
I've raised a separate PR to add one for 4.7.0, and I'll update this PR with notes once that's merged.

Yeah it'll be added to 1.7 but if you install framework 1.7 and assets 1.6 it won't pull the dep in. Not impossible but a very strange setup. I'd wait until someone complains before patching assets 1.6 to support that case.

If assets can be patched in 1.6 line, adding the deps, why not do it and wait for 1.7? Why wait for a complain?

@michalkleiner It's overhead we're better off not incurring if possible, and we'd encourage developers to resolve the issue by updating the assets module to 1.7.x instead. Running inconsistent minors isn't a recommended practice.
@sminnee Changelog notes are up 👍

Weird edge case incoming ;)
{
    "name": "test/test",
    "require": {
        "silverstripe/framework": "^4.7"
    },
    "minimum-stability": "dev",
    "prefer-stable": true
}

composer --version
Composer version 2.0.7 2020-11-13 17:31:06

composer install

composer info | egrep '(assets|framework)'
silverstripe/assets                   1.6.1     SilverStripe Assets component
silverstripe/framework                4.7.0-rc1 The SilverStripe framework

composer why silverstripe/assets
silverstripe/framework  4.7.0-rc1  requires  silverstripe/assets (^1@dev)

Yep I saw a community member hit the same thing recently. The good news is that it will be resolved when we tag 4.7.0 stable in just a couple of weeks 🙂 Definitely lends some credence to making sure we keep core requirements in lock-step at the minor level, since if we'd bumped the assets requirement to ^1.7@dev this wouldn't be occurring.

authors: Cheddam chillu michalkleiner sminnee


text:
[convert] Multiple edf files vs. multi-frame stack

silx convert *edf will by default write a scan_0 group and merge all data into it.
Maybe option --create-root-group should be set by default for multiple EDF files ?
Anyway, silx convert *edf will probably most often be used to create a multi-frame file. Should we try to detect cases when the edf files are a stack of images, and do a cat *edf > multiframe.edf prior to running convert? Or add an explicit --edf-stack option to do this?

My experience trying that command under windows is an error message even after solving #1118
I would say that unless directives are provided indicating a merge, each file should correspond to an individual entry in the final file. Probably not what the user wants but it is sure to work.
That should be already the current implementation so that at least it does not crash.
We could offer to detect the pattern ROOT_SCANNUMBER_DETECTORNUMBER_INDEXNUMBER.edf  that should be easy to couple with any SPEC file provided the user requests for it (or a SPEC file is provided)

Let me pout it clear. We have to provide a tool able to take a set of EDF (or any other image acquisition format files) and associate them to a particular scan of a SPEC file. In other words, we have to be able to provide a merging tool. So, I am not asking for a lot of cases. Just two:

The user has explicitely requested to take all input files as members of a stack. Therefore all the frames have to have the same shape.

The user has followed the convention most widely accepted at the ESRF concerning the naming.

The simplest solution is of course to implement just the first one giving the possibility to specify the output path in the HDF5 file. That would allow to make the things simpler by taking a two steps approach. For instance, the SPEC file is converted and then the associated images are addded to a particular entry explicitely asking to be merged and under what path. A URI would do the job. So basically the option:

The user has explicitely requested to take all input files as members of a stack. Therefore all the frames have to have the same shape.

Plus the possibility to specify -o outputfile.h5::/path/to/place would be enough.
The example below looks to me pretty obvious to understand:
silx convert --stack *.edf -o destination_file:::/entry_0/instrument/detector_x

source: gharchive/issue

authors: PiRK vasole

repo: silx-kit/silx

license: MIT

license_type: permissive

license_source: github-api


text:
[plot] Changing Marker style breaks the mouse drag

It looks like changing the marker style creating a new marker.
As result interaction code based on drag of objects are cancelled (weakref object became None)
Here i display the item hold by draggedItemRef and it's object id.
Here is a log:
beginDrag
drag <silx.gui.plot.items.marker.XMarker object at 0x7f04e059b820> 139658920572960
# Style is changed here
drag None 94658314290016
endDrag
# New drag of the same object
beginDrag
drag <silx.gui.plot.items.marker.XMarker object at 0x7f04e05b6160> 139658920681824
endDrag

Any idea how to solve the problem?

When it became None, ROI.createShapeItems is triggered which is why the marker is recreated.
That's an implementation issue from the ROIs. I will take a look.


text:
Unfreed memory in some clFFT tests ...

Warning:  Program terminating, but clFFT resources not freed.
Please consider explicitly calling clfftTeardown( ).

Pierre, I suspect this is in some of your code...

Apparently, this is fixed in recent version of gpyfft ... I suggest close it.

OK, I'm closing it then.


text:
h5todict asarray=True argument

This PR adds an asarray argument to h5todict which is True by default and keeps the legacy behavior, but when set to False provides the behavior that for me should have been the only one: scalars are converted to scalars in the output dict rather than arrays.
closes #2689

LGTM, Thanks for documentation of the related issue by the way

source: gharchive/pull-request


text:
silx.gui.plot.PlotWidget: Fixed OpenGL rendering

This PR changes the way anchors are rendered which was causing issues with some GPUs.
It is now passing vertex attributes directly from memory (the old way) as the other rendering do rather than creating/deleting a vertex buffer object.
closes #3681

There is an error in AppVeyor that might be relevant.

Not sure, it is one of the recurrent one.
It would be good to fix all those CI random issues... but it requires time to devote to it.


text:
🛑 Lyson NZ is down

In 2ad26dc, Lyson NZ (https://$LZ_SITE1/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Lyson NZ is back up in 60ef276 after 5 minutes.


text:
🛑 simonramos.com is down

In 6414220, simonramos.com (https://simonramos.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: simonramos.com is back up in 9cfc814.

repo: simonramosb/status


text:
SOLikeT unable to install mflike when installing on NERSC

Hi,
I'm trying to install SOLikeT on NERSC, following the directions on the readme, but when I get to the pip install -e . step, it fails with the following error message:
`ERROR: Command errored out with exit status 1:
command: /global/common/cori/software/python/3.6-anaconda-5.2/bin/python -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-fnwdimo9/mflike_3088dc3315ff48079bb30c55e1b06eb9/setup.py'"'"'; file='"'"'/tmp/pip-install-fnwdimo9/mflike_3088dc3315ff48079bb30c55e1b06eb9/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(file) if os.path.exists(file) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-pj858uq6
cwd: /tmp/pip-install-fnwdimo9/mflike_3088dc3315ff48079bb30c55e1b06eb9/
Complete output (7 lines):
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-install-fnwdimo9/mflike_3088dc3315ff48079bb30c55e1b06eb9/setup.py", line 6, in
readme = readme_file.read()
File "/global/common/cori/software/python/3.6-anaconda-5.2/lib/python3.6/encodings/ascii.py", line 26, in decode
return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc2 in position 2057: ordinal not in range(128)
WARNING: Discarding git+https://github.com/simonsobs/lat_mflike. Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.`
Then, at the end of the output, it says:
ERROR: Could not find a version that satisfies the requirement mflike (unavailable) (from soliket) (from versions: none) ERROR: No matching distribution found for mflike (unavailable)
It seems like SOLikeT is unable to find and install the necessary mflike module, which is supposed to be installed automatically. Do you have any suggestions for how to fix this problem?
Thank you so much,
Harrison Winch

Ciao Harrison, which version of pip are you using? The easiest way I can replicate this issue is by using an old pip version (pip 9.0.1). If I upgrade to a more recent version (e.g., pip 21), I can locate and clone all required packages, including mflike. Let me know if the upgrade of pip works.

Issue solved (pip must be up-to-date, see message above). Closing.

repo: simonsobs/SOLikeT


text:
Is there a bug in GifEncoder.cs Line 103?

Line 103 uses transIndex. I expect this should be index.
    indexedPixels [i] = (byte)transIndex;
vs
   indexedPixels [i] = (byte)index;
Though, I have some doubts (output file ballooned in size after this change).

Actually I think I'm just doing something wrong.
If I set a transparent color (which I don't want), then my file size for my sample clip is around 300kb, if I specify no transparent color it balloons to 12MB.
      var ge = new GIFEncoder();         ge.useGlobalColorTable = true;         ge.repeat = 0;         ge.FPS = fps;         ge.transparent =  new Color32(255, 0, 255, 255);         ge.dispose = 1;
With my sample input will give around 300kb output.
Using:
      var ge = new GIFEncoder();         ge.useGlobalColorTable = true;         ge.repeat = 0;         ge.FPS = fps;         ge.transparent =  null;         ge.dispose = 1;
Will yield an output of 12MB for the same input bytes.
Would this be expected?

Yep, ignore me...
I see this is an optimisation technique.

repo: simonwittber/uGIF

license: mit

license_source: bigquery


text:
mda string in SPUD XML IFF IRIS/Earthscope data?

@laura-iris The way that the IRIS/Earthscope station and network ids were extracted from SPUD XML involved parsing a grep on the string mda (metadata aggregator).
I surveyed the 7156 XML files that you provided uids for and found that:
There are 1864 /7156 files with mda string
There are 5292 /7156 files without mda string
Can we assume the following relationship:
A SPUD TF is assocated with IRIS data if and only if the mda string is there?
i.e. are there 1864 archived XML for which data can be sourced from IRIS and the rest are from non-IRIS sources? Or is that not a fair assumption? If that is not a fair assumption, can you think of a test for the remaining 5292 files that would return True/False that the data are hosted by IRIS/Earthscope?

I asked our SPUD experts here, and there was no real answer to this question since it wasn't generated in house. We may want to ask Anna about it, she may know the answer.

Apologies for forgetting to respond here, but I talked to Anna at the USGS and she confirmed our assumption that any TFs that used EarthScope data will have the MDA link.

authors: kkappler laura-iris

repo: simpeg/aurora


text:
ember-cli-babel 5.x has been deprecated

DEPRECATION: ember-cli-babel 5.x has been deprecated. Please upgrade to at least ember-cli-babel 6.6. Version 5.2.8 located: forecast -> ember-cli-deploy-webhooks -> ember-cli-babel

closed via #28

authors: Pezmc marcoow

repo: simplabs/ember-cli-deploy-webhooks


text:
Bumped Petitparser Version

Description
Bumps the version of a dependency, petitparser
Related issues & PRs
Fix to https://github.com/simpleclub/math_keyboard/issues/35
Checklist

[x] I have made myself familiar with the CaTeX
contributing guide.
[x] I added a PR description.
[x] I linked all related issues and PRs I could find (no links if there are none).
[x] If this PR changes anything about the main math_keyboard or example package
(also README etc.), I created an entry in CHANGELOG.md (## UPCOMING RELEASE if the change
on its own is not worth an update).
[x] All required checks pass.

@garv-shah Ah can you open a new PR with signed commits? Or you want me to create it?😄

oh, sorry, sure, i'll try!

Does this work or would I have to create a new PR? Sorry haha, I have no clue how signed commits work

@garv-shah Should work! You just need to upload your gpg key on GitHub I think🙂

oh alright!

done!

Thank you!

authors: edhom garv-shah


text:
New object metadata

Hi,
I am setting up a custom environment following instructions given here: https://github.com/simpler-env/SimplerEnv/blob/main/ADDING_NEW_ENVS_ROBOTS.md.
On bullet 7 of adding new environments, it says:
"Our environments load metadata json files for the object assets (in ManiSkill2_real2sim/data/custom/info_*.json). Based on your environment implementation, fill in the metadata for each new object asset in existing json files or create new json files."
An example of these info.json files has object descriptions of their bounding box, scale, and density:
"bridge_spoon_generated_modified": {     "bbox": {       "min": [         -0.07,         -0.0173,         -0.013021       ],       "max": [         0.07,         0.0173,         0.013021       ]     },     "scales": [       1.0     ],     "density": 1200   },
If I should add a new object, where do I get the bbox characteristics from? Is there a tool that can easily provide this?
Thanks!

Hi, you can measure them using blender (press N to toggle object dimension properties), or get the bbox using trimesh.

repo: simpler-env/SimplerEnv


text:
fix for non-scalar fields during extra updates, test case for #111

test case showing that #111 indeed works
fix when we save some non-scalar values during extra updates (e.g. datetime)

Is it really cache dir should be here?

Sorry, my bad. Added ignore files.
simplethings guys, can you somehow react on the pr's or it's better if we fork this package and then merge our changes in a bulk sometime later?

authors: aivus andrewtch

repo: simplethings/EntityAudit


text:
Add support for null => true in CreateSchemaListener::postGenerateSchemaTable()

Q
A

Branch
master

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Tests pass?
yes

Fixed tickets
#318

License
MIT

Doc PR
n/a

TODO:

[x] Add tests.

Fixes #318.

Changed base to 1.0 since it should be treated as a bugfix IMO.

great, thanks!

@tolry, thank you for merging!
Release 1.0.9 states that this PR was merged in it, but it isn't.

hmm, lgtm https://github.com/simplethings/EntityAuditBundle/blob/v1.0.9/composer.json#L26 or am I missing something?

@phansys d'oh - you used short array syntax in your test  :smile: https://travis-ci.org/simplethings/EntityAuditBundle/jobs/364493226

authors: phansys tolry

repo: simplethings/EntityAuditBundle


text:
Handles version file name change

If the version file name has been changed, the old version will not be detected. This change ensures that the check will pass because otherwise the check has to be disabled in order to merge the change.
Also adds test for case demonstrating what happens when version file name is misconfigured.

Honestly this is such an edge case and the workaround of just disabling and re-enabling the github check is so easy that I'm not convinced we actually need to add any extra complexity, but if you feel strongly about it, then the implementation itself looks good to me!

It is an edge case, however, when it goes wrong, it then depends on having the correct rights and a number of other things which get in the way of doing the merge (such as forgetting to turn it on).
BTW: Not eating it's own dog food?

repo: simplybusiness/version-forget-me-not


text:
North American Phone number format

A clear and concise description of what you want to happen.
Please add support for North American Phone number format.
North American phone numbers are usually written in one of the following formats:
(123) 234-5678
123-234-5678
Other less commonly used formats are:
123 234 5678
123.234.5678
The format being mandated right now is +1-123-456-7890, which nobody would write in North America.
In general too, the country code would be less likely to be written on a resume since one would usually have a local number in the geography they're applying for.

Thank you.
I have added a new option to the data model: locale_catalog.phone_number_format
It's set to national by default, which formats the phone number exactly how you would like. It looks at the country code and formats accordingly, using the Python port of Google's libphonenumber, phonenumbers.

Thank you. I tested it out and it looks great!

The new functionality is released with v1.13.

authors: prabhjotsbhatia-ca sinaatalay

repo: sinaatalay/rendercv


text:
Unexpected token error

I got unexpected token error in fix-path code.
This is the content of fix-path:
'use strict';
module.exports = () => {
	const env = process.env;

	if (process.platform === 'darwin') {
		return env.SHELL || '/bin/bash';
	}

	if (process.platform === 'win32') {
		return env.COMSPEC || 'cmd.exe';
	}

	return env.SHELL || '/bin/sh';
}();

When I removed last () , it didn't give invalid syntax error.
What could be wrong?
I installed nodejs via brew
v5.10.1```

Not sure how this is related to this module, but you can't invoke a function declaration, you need to wrap it to create an expression, e.g. (() => 1)();

@yask123 Fixed in https://github.com/sindresorhus/default-shell/commit/d3e441382f0b95515dd208ca228ace817a3535a2. Just reinstall this module.

Ah, the issue was in another module 👍 I wonder how that slipped through CI.

@silverwind Somehow that syntax worked fine in Node.js 4.

@sindresorhus yeah, looks like a v8 bug. Must've been fixed in 6.0.0 or earlier.

Related: http://stackoverflow.com/questions/34589488/es6-immediately-invoked-arrow-function

That was quick! Thanks! 👍

authors: silverwind sindresorhus yask123

repo: sindresorhus/fix-path


text:
Add test to do mkdirp on an existing path

Add a (failing) test, asserting that it is OK to call ftp.mkdirp(path) on a path that already exists.

Closing in favor of #9.

repo: sindresorhus/jsftp-mkdirp


text:
I changed how to add proxy in KY

I currently need to use KY on a machine with high confidentiality. This machine only has an intranet. I checked many online tutorials, but I didn't find the way to add an agent. Can you give me an example?

When used in the browser, it should respect the system proxy settings. When used in Node.js, you could use https://github.com/gajus/global-agent

authors: mycodeku sindresorhus

repo: sindresorhus/ky-universal


text:
Also export .every()?

https://github.com/kevva/p-every is different and outdated, is it worth just exporting both functions from this package since they're a similar operation? Or maybe create a dedicated package

It can just be exported as a named export from this package.

authors: fregante sindresorhus

repo: sindresorhus/p-some


text:
thumbnails on ubuntu 14.04 with pageres and some other issue in mac

Hi,
I want to take snapshot of webpage...but when i load in Ubuntu its not loaded at all...it stops in the middle.
    url = 'https://sapui5.netweaver.ondemand.com/test-resources/sap/m/demokit/cart/index.html?responderOn=true#/category/AC';
 var options = {delay:2};
var pageres = new Pageres(options)
    .src(url, ['1280x1024'], {crop: true})
    .run(function(error, items){
        console.log(items);

        items[0].on('data', function (data) {
            bufferData.push(data);
        });

        items[0].on('end', function () {
            deferred.resolve(Buffer.concat(bufferData));
        });
    });

pageres.on('warn', function(error){
    console.log(error);
});

but in Mac its different issue, but eventually i can generate in mac or windows.
even on ubuntu i can generate snapshot of other pages like google.com or any other.
can someone help in understanding why the phantom is not loaded in ubuntu but it loads in mac??
Thanks
Moyeen

Can you share any error/output messages?

Error: found in negative cache: 'sap/ui/core/Core.js' from undefined/sap/ui/core/Core.js: The given language does not adhere to BCP-47. sapui5.netweaver.ondemand.com/resources/sap-ui-core.js:80 in a1 sapui5.netweaver.ondemand.com/resources/sap-ui-core.js:80 sapui5.netweaver.ondemand.com/resources/sap-ui-core.js:144 TypeError: 'undefined' is not an object (evaluating 'new sap.m.Shell') sapui5.netweaver.ondemand.com/test-resources/sap/m/demokit/cart/…

The same is working on mac...

can someone help??

It has something to do with the website. I don't think PhantomJS is capeable of rendering it correctly. Not sure why though.
var Pageres = require('pageres');

var url = 'https://sapui5.netweaver.ondemand.com/test-resources/sap/m/demokit/cart/index.html?responderOn=true#/category/AC';

new Pageres({delay: 30})
	.src(url, ['1280x1024'], {crop: true})
	.dest('public')
	.run();

It generates a screenshot but the screenshot is just plain white.
I used a delay of 30 seconds just to test it was not caused because of some delay.

Maybe this has something to do with it: https://github.com/SAP/openui5/issues/557
If I'm not mistaken, Pageres uses PhantomJS 1.9.

repo: sindresorhus/pageres


text:
pathIsAbsolute('C:/Users/'); is false

The README has the example:
var pathIsAbsolute = require('path-is-absolute');

// Windows
pathIsAbsolute('C:/Users/');
//=> true

When I try that, it is false. I have to use the pathIsAbsolute.win32 method to get it to return true.
I am using path-is-absolute 1.0.0 with node v4 on linux.
Is that method only supposed to work if my actual environment is windows? The documentation is unclear.

So I found this
https://github.com/sindresorhus/path-is-absolute/blob/master/index.js#L18
which means that it is only supposed to work if I am running windows. So this is not a bug, but the README should make it clear that the example is not for generally validating windows paths, but for validating paths when running in a windows environment.

This is just a polyfill for the native method. Any doc improvements should be requested on the Node.js issue tracker.

I'm not suggesting a change in behavior, just a clarification in the documentation.

authors: gabegattis sindresorhus

repo: sindresorhus/path-is-absolute


text:
Don't see Pushed branches or created branches in feed

Is this on purpose?

Yes

Any way I can enable it?

https://github.com/sindresorhus/refined-github#customization

Glad I found a bug :)
I think you should put this in the list of highlights no? (that you take this out)

repo: sindresorhus/refined-github


text:
Add 'More' dropdown for GHE

Currently GHE does not have the 'Insights' dropdown. A lot of our logic
relies on this dropdown existing, so create one when it doesn't exist.
Fixes #609

Would you say this would be removed with the next release of GHE including the drop down?

It could be removed then yes, once GHE adds the Insights dropdown then it'll start renaming it to More and never get to this code.

What else needs to happen to get this merged in? Should have zero effect on GitHub users. Currently Refined GitHub does nothing on GHE due to #609.

authors: busches hkdobrev


text:
chore: clarify that IATA is for Air and UIC for Rail

Thank you! I've added a changelog entry. Ready to merge, from my perspective.

repo: sine-fdn/ileap-extension


text:
fix: add gitlab MR labels at time of creation

This refactor means that createPr now takes labels as a new parameter so that the GitLab API can include them with the MR creation request, so save one API call. This also fixes #985 as a result.

@Autre31415 could you take a look and let me know if you think it looks ok?

Would it be possible to also append labels after the fact in the case of PRs that already exist but don't have labels already?

@Autre31415 currently we do not add labels "after the fact". The reasoning is that: (a) we only have one way to configure labels and they will be added at PR time, and (b) if someone removes a label we added, they surely don't want us to be adding it back again. Unless this workflow changes in future, I don't see a need for us to support adding labels to pre-existing PRs.

@Autre31415 I have now removed addLabels from platform exports to make it clearer that ad-hoc addition of labels is not supported. addLabels wasn't called from anywhere else in the code anyway.

@rarkins Looks good to me. Now only the merge conflicts stand in the way.

I'll apply the same changes to v10 soon

repo: singapore/renovate


text:
remove youtube_player_flutter and flutter_inappwebview to fix common related issues in flutter_quill_extensions

We're having many issue and reports to issues that caused by flutter_inappwebview which is required and needed by youtube_player_flutter which is a dependency in flutter_quill_extensions
Embedding platform Webview in the editor causes performance issues, rebuilding the platform widget consistently makes the video reload many times on some platforms, making the editor unstable even when not interacting with the platform widget, increasing the bundle size, requiring more transitive dependencies and more manual set up for web.
We did plan on introducing this change in the next major release or by introducing a separate package (#2276), or by using the download URL to load the video without the iframe (#1916), directly using a normal video player widget (which violates the terms of service of YouTube) and some other solutions by it seems that we're getting more reports outside of GitHub, and I did have some issues while working on an outdated branch.
Removing those dependencies was something we planned for a while ago, pubspec by default use the latest compatible version of any dependencies as it assumes it confirms to Semantic Versioning and they have introduced macOS and Windows support in a minor version, which causes to have build failure for the users and for maintainers working in outdated branches, we do find it a bit confusing since we didn't manually updated the version of any related dependency.
It seems that flutter_inappwebview_windows and flutter_inappwebview_macos was introduced as dependencies of flutter_inappwebview 6.1.0, where they were experimental and causing build failure even on Android (on Windows machine). It seems that they working hard to introduce fixes as fast as possible, see
flutter_inappwebview #2294 and flutter_inappwebview #2288.
I'm grateful for their efforts to the open source community and the support for macOS and Windows even if it introduce many issues, however, I do think those dependencies should be included in the user project and not as part of this project, especially we don't heavily depend on it, if this was a payment package to a web service (that doesn't have public web API) or something related then it would be critical to have it, the solution we have is not ideal for many users, and more users detaching flutter_quill_extensions and eventually flutter_quill.
We're currently trying to focus on the current issues we have, we prefer to not have more on the surface for more issues at the moment.
We have four choices:

Do nothing for now and wait for issues to be fixed, then remove it once we're ready for a major breaking change which we are not and should not introduce a breaking change release for this change for many reasons we have discussed across multiple issues, see #2259
Introduce a breaking change in non-minor version 10.8.0 and include notes for users in CHANGELOG.md, provide a quick experimental solution to allow users to introduce their own solution, currently, we can't plan for a stable solution as we have more many issues that need time and introducing a good solution is not a priority, we also have discussed about this across multiple issues, see #2270
Separate the version of flutter_quill_extensions right away and only introduce a breaking change for the extensions package. Introducing a breaking change for flutter_quill is currently not an option due to the reasons we have discussed in many issues (we don't have a specific issue to keep track of those discussions due to the lack of time), this will break our current workflow which also have issues, I prefer to not use such solutions as we still busy with other issues and need better planning
Use a different package or load the video using the download URL, possibly violate YouTube's terms of service, but not necessarily, previously I saw some apps that use the download URL without violating the terms of service, I'm not sure if there are use cases or if what they meant that we should not download videos to the user device using the download URL or if we're allowed to only play the video in video player indicating that this is a youtube video with the youtube video URL (not the download URL) and with some indicators to make it clear this is not our video, for web this can be solved by simply using iframe with platform-specific check which still cause issues even on Flutter web.

If fixing this issue with a breaking change in a minor issue is okay for most users, then we might introduce it, it shouldn't cause any issues for libraries and packages in pub.dev (see packages that depend on flutter_quill_extensions), we also could simply make it so that it would revert the video as a video link, allowing the user to press the link and play it on Youtube app as they expect, also it's might not very common that users copy video URL that's YouTube specific and then insert it in the editor.
See #1888, #2276, #1916, #2280, #989 for related issues.

@singerdmx @CatHood0 @AtlasAutocode Need to know which method you prefer to handle this situation before introducing any change.

repo: singerdmx/flutter-quill


text:
Pd/add output on failure

Issue and/or context:
I spent some time debugging the failure in #1685 and as near as I can tell it was ephemeral. All of the test runs on my fork's repo have been passing since I tweaked the version script to get it to build the first time.
This change just includes @johnkerl's branch from #1685 and an option for ctest to output logs on failures. Regardless of whether this change discovers some wild difference between upstream and my fork's testing environments, it'll still be helpful for ctest failures going forward.
Changes:
Add --output-on-failure when invoking ctest.
Notes for Reviewer:
Narps.

Codecov Report
Patch has no changes to coverable lines.

:exclamation: Current head ccbdc19 differs from pull request most recent head 8e2115c. Consider uploading reports for the commit 8e2115c to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:loudspeaker: Thoughts on this report? Let us know!.

May want to remove the added macos-12 from the minimal setup though.

Not today. Context:
https://github.com/single-cell-data/TileDB-SOMA/pull/1685#issue-1899149417

authors: codecov-commenter davisp johnkerl

repo: single-cell-data/TileDB-SOMA


text:
React Support

We should provide support for using Sugar with React. This would include:

[ ]  Creating React components for each atomic element of Sugar.
[ ]  Creating React components for each molecular element of Sugar.
[ ]  Bundling the CSS.
[ ]  Deciding on name on npm.
[ ]  Publishing it to npm for use.

With tailwind, I don't think bundling css is necessary. We will need to include instructions about how to configure tailwind to include the components when it purges styles.

repo: singlestone/sugar


text:
storage: refine scan interface of Keyspace

The purpose of Keyspace is to only expose the records with the specific prefix, so we hardly use the current scan interface, which does not strip the prefix.
We may remove the public interface of scan and directly expose scan_strip_prefix as scan to users, similar to what we did in #1969. Ditto for scan_with_start_key.
https://github.com/singularity-data/risingwave/blob/8ca6a1efbe866407f8f0a4719d6e4d8d59552bba/src/storage/src/keyspace.rs#L120-L162

I'd like to take this issue. Could you please assign it to me?

authors: BugenZhao ClearloveWei

repo: singularity-data/risingwave


text:
Maix Dock Wi-Fi

The documentation is horrible to say the least.  I am trying to learn how to get the WIFI working and even though I tried the scripts they do not work.

The documentation is horrible to say the least. I am trying to learn how to get the WIFI working and even though I tried the scripts they do not work.

Same for me.
Please let me know if you get any progress.

authors: nicolaivalenti versamodule


text:
Expand Reference Documentation

There are still a lot of things that don't have docblocks and this gets reflected in the Doxygen output.  I'll leave this one open as an ongoing issue until they're finally all done.

Added docblocks to all the top-level controllers.  I'll add more in the next release.


text:
SCHEUDULED INCIDENT FUTURE

Hello,MultilineEmpty spaceIndentationhello blockquote [start_datetime:1706016480] [end_datetime:1706016540]

This is resolved status

repo: sivadurga-web/kener


text:
Does this gem works in production enviroment ?

Does this gem works in production enviroment ?

It's been designed for development use, but you could probably use it in production if you really wished. Just be in mind that it uses an in-memory store.


text:
OXS throws an xargs error when using the pre-commit hook

When trying to commit on OSX (after properly installing the pre-commit hooks and .gitlint.yaml file), I receive the following error:
 xargs: illegal option -- -
I believe this is caused by this line as OSX xargs don't seem to support --null and --no-run-if-empty. (Also seen in the hg file).
I'd like to note that the actual git lint command works fine. I seem to only get this when trying to commit.

Hi Michael, thanks for reporting this.
In OSX the --null option is called -0 which also works in GNU versions, so we can easily replace that part. However the --no-run-if-empty does not exist, but could be simulated by assigning the result to a variable and checking if the variable is not empty.
Feel free to send a PR.

Sent - #89

repo: sk-/git-lint


text:
ci(github): Basic Gradle workflow to build on Github

Contains
A Github workflow to build, test, and analyze the code.
Uses Gradle Build Scans for reporting, see https://docs.gradle.com/enterprise/gradle-plugin/#connecting_to_scans_gradle_com (automatically accepted for CI runs, but no for local runs).
How to test

[ ] ensure that gradlew distForLauncher --scan still prompts for user agreement
[ ] ensure that the jobs look reasonable and work correctly (for instance, :x: when build should be failing)

Additional Tasks

[ ] configure mandatory checks to use Github jobs instead of Jenkins jobs for now

Whoop, https://github.com/skaldarnar/Terasology/actions/runs/6066061989/job/16456225384?pr=1 produced a Build Scan which can be accessed here: https://scans.gradle.com/s/ikpmdxyh6lk4a

repo: skaldarnar/Terasology


text:
Note browser plugins for auto-skewering some URLs

I generally use the bookmarklet when I want to skewer a page, but when
I'm coding, it's annoying to re-skewer after refreshing.
I don't like loading skewer directly in a codebase's markup/templates,
because then I have to remember not to commit that tweak.
Since there's a simple way to have dev-friendly browsers skewer dev
environments automatically, it seemed worth documenting.

This is essentially the same as the Greasemonkey userscript, but more options here isn't a bad thing.

yep, that's what I thought. I personally like not having to click the 'skewer' button after a reload - it's a tiny thing, but it can be more convenient.
Thanks for merging.

authors: NateEag skeeto

repo: skeeto/skewer-mode

license: unlicense


text:
Custom predictors are inefficient for window features

The way that custom predictors are calculated in ForecasterAutoregCustom is potentially inefficient.
See the relevant code snippet.
The features are computed in a for loop and appended to a list:
        X_train  = []
        y_train  = []

        for i in range(len(y) - self.window_size):

            train_index = np.arange(i, self.window_size + i)
            test_index  = self.window_size + i

            X_train.append(self.fun_predictors(y=y_values[train_index]))
            y_train.append(y_values[test_index])

Whilst this is very generic, it is inefficient for the most common use case of window features (e.g., using the rolling mean, standard deviation, etc. at various different window sizes). There are optimised ways for computing these rolling statistics which take the whole series as input and outputs all the features across time (not just for one time period as required by the current implementation of ForecasterAutoregCustom).
Pandas and Numpy have efficient implementations of rolling statistics. They can also be further speeded up with Numba (see the window ops stuff here).
Some options I think that would be helpful for skforecast  could be

offer an additional way to derive features from the target variable so a user can supply a more efficient way of calculating window features
integrate window features in the API itself in a similar way that lag features are so that a more efficient way of calculating them can be included under the hood

On a sidenote, in my opinion, it is counterintuitive that ForecasterAutoreg supports a lags argument but ForecasterAutoregCustom requires the user to create the lag features in a custom function that the user must provide.
Best wishes,
Kishan

Hi @KishManani
I agree. We tried to implement the less restricted approach so that users could use whatever function to create predictors, given a window of values. However, in most cases, users may only need rolling stats, which are much more efficiently implemented when applied to the whole series instead of by window slices.
An important point to consider is that the function has to be applied at two levels:

When creating the training matrices:

X_train  = []
y_train  = []

for i in range(len(y) - self.window_size):

    train_index = np.arange(i, self.window_size + i)
    test_index  = self.window_size + i

    X_train.append(self.fun_predictors(y=y_values[train_index]))
    y_train.append(y_values[test_index])

In  the prediction process (inside the recursive loop) :

for i in range(steps):
  X = self.fun_predictors(y=last_window).reshape(1, -1)
  if np.isnan(X).any():
      raise ValueError(
          "`fun_predictors()` is returning `NaN` values."
      )
  if exog is not None:
      X = np.column_stack((X, exog[i, ].reshape(1, -1)))

  with warnings.catch_warnings():
      # Suppress scikit-learn warning: "X does not have valid feature names,
      # but NoOpTransformer was fitted with feature names".
      warnings.simplefilter("ignore")
      prediction = self.regressor.predict(X)
      predictions[i] = prediction.ravel()[0]

  # Update `last_window` values. The first position is discarded and
  # the new prediction is added at the end.
  last_window = np.append(last_window[1:], prediction)

I am going to do some benchmarking analysis with a function that has a dual behavior when applied to a data frame (case 1) and a NumPy array (case 2).

I've just started working with your package and I'm already impressed. I prefer it over others like sktime and mlforecast. However, I suggest redesigning certain parts of the API, as the absence of window features is a significant drawback. You might consider looking at how mlforecast has integrated this functionality into their API.

Hello @slavakx,
Thanks for your comment. Could you please provide a schema or a reproducible example with another library to better understand what you are expecting?
Thanks!

Hi @slavakx
Thanks for your feedback. We are always trying to include useful functionalities in the library, so we will add windows features to the roadmap.

Hello @slavakx,
Thanks for your comment. Could you please provide a schema or a reproducible example with another library to better understand what you are expecting?
Thanks!

Hi @JoaquinAmatRodrigo
The mlflorecast code looks like this and it supports different time features (see below my own implementation)
fcst = MLForecast(
    models=models,
    freq='D',
    lags=[7, 14],
    lag_transforms={
        1: [ExpandingMean()],
        7: [RollingMean(window_size=28)]
    },
    date_features=['dayofweek'],
    target_transforms=[Differences([1])],
)

Here is my own code for lag and window features that work pretty fast:
Lag
LAG_DAYS = list(range(1, 3))
for lag in LAG_DAYS:
    temp_df[f"temp_lag_{lag}"] = temp_df.groupby(lambda x: True )["temp"].transform(lambda x: x.shift(lag))

Moving Average (Rolling Mean)
moving_average_days = [2]
day_lags = [1,2]
for i in moving_average_days:
    for day_lag in day_lags:
        print(f"Moving average period: {i} with shift day: {day_lag}")
        temp_df[f"moving_average_{i}_lag_{day_lag}"] = temp_df.groupby(lambda x: True)["temp"].transform(lambda x: x.shift(day_lag).rolling(i).mean())

Expanding Mean
day_lags = [1,2]
for day_lag in day_lags:
    print(f"Expaning mean period with shift day: {day_lag}")
    temp_df[f"expanding_mean_lag_{day_lag}"] = temp_df.groupby(lambda x: True)["temp"].transform(lambda x: x.shift(day_lag).expanding().mean())

Exponentially Weighted Moving Everage
alpha = 0.5

day_lags = [1,2]
for day_lag in day_lags:
    print(f"EWMA with shift day: {day_lag}")
    temp_df[f"ewma_lag_{day_lag}"] = temp_df.groupby(lambda x: True)["temp"].transform(lambda x: x.shift(day_lag).ewm(alpha=alpha, adjust=False).mean())

Seasonal Rolling Mean
import pandas as pd
date_range = pd.date_range(start='2023-01-01', end='2023-02-01', freq='D')
df = pd.DataFrame({'value': range(len(date_range))}, index=date_range)
df["dayofweek"] = df.index.dayofweek
df.head()

seasonal_average_days = [2]
day_lags = [1,2]
for i in seasonal_average_days:
    for day_lag in day_lags:
        print(f"Seasonal moving average period: {i} with shift day: {day_lag}")
        df[f"seasonal_moving_average_{i}_lag_{day_lag}"] = df.groupby("dayofweek")["value"].transform(lambda x: x.shift(day_lag).rolling(i).mean())

Thanks @slavakx  for the examples.
Some of these windows features can be used in skforecast with the ForecasterAutoregCustom, for example here:

# Custom function to create predictors
# ==============================================================================
def create_predictors(y):
    """
    Create first 10 lags of a time series.
    Calculate moving average with window 20.
    """

    lags = y[-1:-11:-1]     # window size needed = 10
    mean = np.mean(y[-20:]) # window size needed = 20
    predictors = np.hstack([lags, mean])

    return predictors

# Create and fit forecaster
# ==============================================================================
forecaster = ForecasterAutoregCustom(
                 regressor       = RandomForestRegressor(random_state=123),
                 fun_predictors  = create_predictors,
                 name_predictors = [f'lag {i}' for i in range(1, 11)] + ['moving_avg_20'],
                 window_size     = 20 # window_size needed by the mean is the most restrictive one
             )

forecaster.fit(y=data_train)
forecaster

We are currently working on improving the flexibility and speed.
More detailed information here: https://skforecast.org/0.12.1/user_guides/custom-predictors

Hi all,
Creating window and custom features has been optimized in skforecast 0.14.0. Give it a try!
Best,

authors: JavierEscobarOrtiz JoaquinAmatRodrigo KishManani slavakx

repo: skforecast/skforecast


text:
Create devcontainer.json

Summary

Changes

Closes:
Task list

[ ] For workflow changes, I have verified the Actions workflows function as expected.
[ ] For content changes, I have reviewed the style guide.

Create new file to install copilot in COdespace

repo: skills/copilot-codespaces-vscode


text:
[Feature]

Summary
A clear and concise description of what the problem is, and what you solution you want.
Changes
What changes this feature would introduce. Any risks these changes involve. Any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Summary
A clear and concise description of what the problem is, and what you solution you want.
Changes
What changes this feature would introduce. Any risks these changes involve. Any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

         Several comments have been deleted. All were relevant. Why the deletions?

repo: skills/introduction-to-github


text:
When deleting rc, pod does not get deleted

I need to check how kubectl does this, but usually when you delete a rc, it also deletes the pods that this RC created.
Might be two api calls wrapper in kubectl delete ...

yes, so kubectl has a reaper to delete related resources:
https://github.com/kubernetes/kubernetes/blob/master/pkg/kubectl/cmd/delete.go#L141

Yes you're right. Kubectl automatically turns on cascade=true. I'm figuring out how to add this flag to API request in Kompose. If can not do that, I will add api call for deleting the corresponding pod.

When using the REST API or go client library, you need to do the steps explicitly (scale replicas to 0, wait for pod deletions, then delete the replication controller).

You might want to wait for K8s 1.3. We're going to support server-side cascading deletion.
https://github.com/kubernetes/kubernetes/blob/master/docs/proposals/garbage-collection.md
cc @janetkuo

@bgrant0607 thanks for the info, we will keep an eye on it.

authors: bgrant0607 ngtuna runseb

repo: skippbox/kompose


text:
[BACKUP] seek offset 超過 latest offset 會 reset 至 earliest

在 sink task 中如果透過 context seek offset 超過 partition 的最新 offset，consumer 會自己 reset offset 至 earliest。
以下是模擬 partition 中 latest offset 為 999，使用者如果設定 from 為 1200時會出現的 info log
INFO [local-test3|task-0] [Consumer clientId=connector-consumer-local-test3-0, groupId=connect-local-test3] Seeking to offset 1200 for partition test3-0 (org.apache.kafka.clients.consumer.KafkaConsumer:1595)

INFO [local-test3|task-0] [Consumer clientId=connector-consumer-local-test3-0, groupId=connect-local-test3] Fetch position FetchPosition{offset=1200, offsetEpoch=Optional.empty, currentLeader=LeaderAndEpoch{leader=Optional[192.168.103.64:15371 (id: 1001 rack: null)], epoch=0}} is out of range for partition test3-0, resetting offset (org.apache.kafka.clients.consumer.internals.Fetcher:1403)

INFO [local-test3|task-0] [Consumer clientId=connector-consumer-local-test3-0, groupId=connect-local-test3] Resetting offset for partition test3-0 to position FetchPosition{offset=0, offsetEpoch=Optional.empty, currentLeader=LeaderAndEpoch{leader=Optional[192.168.103.64:15371 (id: 1001 rack: null)], epoch=0}}. (org.apache.kafka.clients.consumer.internals.SubscriptionState:399)

seek offset 方式是透過 taskContext 先呼叫 requestCommit 再透過 offset 方法設定要前往的 offset。
目前我推測會這樣的原因是因為 Fetcher 紀錄完  resetting offset 後會呼叫 subscriptions.requestOffsetReset(topicPartition) 來 reset 此 topicPartition，在 requestOffsetReset 中是使用 defaultResetStrategy 來確定要將 topicPartition reset 至哪個 offset。
而 KafkaConsumer 的 Subscriptions 中建立時所吃的 defaultResetStrategy 就是 AUTO_OFFSET_RESET_CONFIG，因此 Worker 所建立的 baseConsumerConfigs 中 AUTO_OFFSET_RESET_CONFIG 是設定為 earliest。
不知以上的 trace 是否正確

欸 描述好像是空白的

欸 描述好像是空白的

抱歉，剛剛在打標題時不小心就發出來了，因此描述的部分是用更新補上

@Haser0305 感謝補充說明，可否試著“條列”重現的方式？

好的，以下是我如何重現這問題

創建一個 單一 partition 的 topic 裡面先寫入 999 筆資料
使用 #1709 的 branch 並且在裡面插入一些顯示字串方便觀察後創建 shadowJar file，
新建一個 exporter 並且設定這個 topic partition 的 from 為 1200

以上是我這次測試的步驟，在第二點的字串中可以觀察到每次 put 所得到的 records 都是 offset 從 0 到 194。
之後會一直重複seek，並且消費 0 -> 194 這些資料

@Haser0305 please take a look at #1804

repo: skiptests/astraea


text:
Use limit in hyperexpand

This alone could solve summation problem in skirpichev/omg#236, without using (not a bad idea anyway) gosper_sum first.

@seblabbe, you may be interested in this patch, as the problem was reported in
https://groups.google.com/forum/?fromgroups=#!topic/sympy/ZI_D-eG1UfE
Sorry for off-list answer, but I can't post this solution to sympy list.

Great, thanks for the fix!

authors: seblabbe skirpichev

repo: skirpichev/omg


text:
Use Promethus for the Pod CPU in the cluster page

We need to use Promethus data for the Pod CPU in the cluster page. This tickets only needs to update the reserved CPU.
Please refer to #207. Most codes should be the same. In the issue, you need to find the correct Promethus query and replace the existing pie chart. The query could be sum(kube_pod_container_resource_requests_cpu_cores).
Current:

Expected:

After finished, merge into the promethus branch.

@tianni4104. I would like to work on this issue.


text:
Issue using cv2.getStructuringElement

Expected behaviour
Return array of given dimensions
Actual behaviour
TypeError: an integer is required (got type _NoValueType)
Steps to reproduce

cv2.getStructuringElement(cv2.MORPH_RECT, (5, 5))
Windows 10
architecture (e.g. x86)
opencv-python version

Cannot reproduce, please provide full code. This looks like a generic usage question, not an actual issue.
>>> import cv2
>>> cv2.getStructuringElement(cv2.MORPH_RECT, (5, 5))
array([[1, 1, 1, 1, 1],
       [1, 1, 1, 1, 1],
       [1, 1, 1, 1, 1],
       [1, 1, 1, 1, 1],
       [1, 1, 1, 1, 1]], dtype=uint8)

authors: skvark straumboi

repo: skvark/opencv-python


text:
Iban in an included country raise UnknownCountryCodeResult

An IBAN from an included country raises an UnknownCountryCodeResult error.
For example Wallis & Futuna (WF) included in France (FR) WF9120041010050500013M02606 is not valid with IbanNet but is valid on https://bank-codes.fr/iban/validate/

Thanks for reporting, this is definitely a bug. Will have a look asap.

After further investigation, I do not believe that site is correct. I have reviewed different sources, all of those mention that included countries use the country code of their parent country, which in this case would be FR.
https://www.ecbs.org/iban/france-bank-account-number.html
https://www.swift.com/standards/data-standards/iban-international-bank-account-number

French Guiana, French Polynesia, French Southern Territories, Guadeloupe, Martinique, Mayotte, New Caledonia, Réunion, Saint Barthélemy, Saint Martin (French part), Saint Pierre and Miquelon, and Wallis and Futuna Islands have their own ISO country code but use "FR" as their IBAN country code.

If you have different sources, I would be happy to review, but for the time being I will not make any chances with the confidence that this library implements the SWIFT registry correctly.

repo: skwasjer/IbanNet


text:
[Koin] No need for createdAtStart

I think you got it wrong. By default Koin will behave lazily, so no need to set createdAtStart = false on each definition you don't wanna create during startup. I've also noticed some definitions don't have this property set, so I wonder if you thought not setting it would imply eager instantiation...
For example, here's Koin single DSL method. Notice createdAtStart:
    inline fun <reified T> single(
            qualifier: Qualifier? = null,
            createdAtStart: Boolean = false,
            override: Boolean = false,
            noinline definition: Definition<T>
    ): BeanDefinition<T>

@mradzinski
Oh, I thought the Koin will behave instantly instead of lazily,
so I set createdAtStart = false on each definition.
Thank you for your nice issue!

Very welcome 😊

authors: mradzinski skydoves


text:
Swift3

Hey @esten, please review. Probably best to just pull it and look it through.
The example project and unit tests have failures I could use a hand with

Errors were due to missing update to iOS 10 in the example project. Info.plist updated and all tests run and app runs

repo: skylib/SnapImagePicker


text:
[bug] saving project error

@skylot While testing this bug I noticed that the way JadxGUI handles Exceptions has in my opinion two problems:

Exceptions are usually only shown in log file instead of showing an error message
When en exception in MainWindow.closeWindow happens Jadx stays and can't be closed at all.

Regarding 1: What do you think of instead of catching and logging errors in JadxGUI define a general Thread.setDefaultUncaughtExceptionHandler that shows an general (or specific) error message plus may be exception name, message and stack trace and copy everything to clipboard.
That would also make it easier for Jadx users to create an issue here on Github (and for us as this would reduce the chance that people post a screen shot instead of a text containing the stack trace.

define a general Thread.setDefaultUncaughtExceptionHandler

@jpstotz sure, that will be very helpful
I'm also thinking about showing error window if error logged from jadx.gui package (added in LogCollector class). But it can cause many windows to appear at same time if there are several errors has been recorded.
Another way is just popup log viewer window on errors from jadx.gui package, so it will behave like error dialog :)

When en exception in MainWindow.closeWindow happens Jadx stays and can't be closed at all.

Not sure how to handle that, because failing to save project should not be ignored, an error message should appear at least. But at the same time, not closing on request is also bad.

authors: jpstotz skylot zoky2017

repo: skylot/jadx


text:
[UI] java.lang.IndexOutOfBoundsException: len=-1016

I pressed "Reload Files" button after analyzing an APK renaming several methods having several tabs of classes open
Java: java-19-openjdk
Linux Kernel: 6.0.5-arch1-1

Jadx version: dev
Java version: 19.0.1
Java VM: Oracle Corporation OpenJDK 64-Bit Server VM
Platform: Linux (6.0.5-arch1-1 amd64)
Max heap size: 5372 MB
Program args: -Xms128M -XX:MaxRAMPercentage=70.0 -XX:+UseG1GC -Dawt.useSystemAAFontSettings=lcd -Dswing.aatext=true -Djava.util.Arrays.useLegacyMergeSort=true

java.lang.IndexOutOfBoundsException: len=-1016
	at java.desktop/sun.font.FontDesignMetrics.charsWidth(FontDesignMetrics.java:499)
	at org.fife.ui.rsyntaxtextarea.TokenImpl.getWidthUpTo(TokenImpl.java:601)
	at org.fife.ui.rsyntaxtextarea.TokenImpl.getWidth(TokenImpl.java:570)
	at org.fife.ui.rsyntaxtextarea.TokenUtils.getSubTokenList(TokenUtils.java:122)
	at org.fife.ui.rsyntaxtextarea.WrappedSyntaxView$WrappedLine.calculateLineCount(WrappedSyntaxView.java:1193)
	at org.fife.ui.rsyntaxtextarea.WrappedSyntaxView$WrappedLine.getPreferredSpan(WrappedSyntaxView.java:1241)
	at java.desktop/javax.swing.text.View.getMinimumSpan(View.java:248)
	at java.desktop/javax.swing.text.BoxView.calculateMajorAxisRequirements(BoxView.java:872)
	at java.desktop/javax.swing.text.BoxView.checkRequests(BoxView.java:931)
	at java.desktop/javax.swing.text.BoxView.setSpanOnAxis(BoxView.java:330)
	at java.desktop/javax.swing.text.BoxView.layout(BoxView.java:710)
	at java.desktop/javax.swing.text.BoxView.setSize(BoxView.java:398)
	at org.fife.ui.rsyntaxtextarea.WrappedSyntaxView.setSize(WrappedSyntaxView.java:1023)
	at java.desktop/javax.swing.plaf.basic.BasicTextUI$RootView.setSize(BasicTextUI.java:1823)
	at java.desktop/javax.swing.plaf.basic.BasicTextUI.getPreferredSize(BasicTextUI.java:943)
	at java.desktop/javax.swing.plaf.basic.BasicTextAreaUI.getPreferredSize(BasicTextAreaUI.java:121)
	at java.desktop/javax.swing.JComponent.getPreferredSize(JComponent.java:1734)
	at java.desktop/javax.swing.JTextArea.getPreferredSize(JTextArea.java:623)
	at java.desktop/javax.swing.ScrollPaneLayout.layoutContainer(ScrollPaneLayout.java:799)
	at java.desktop/java.awt.Container.layout(Container.java:1541)
	at java.desktop/java.awt.Container.doLayout(Container.java:1530)
	at java.desktop/java.awt.Container.validateTree(Container.java:1725)
	at java.desktop/java.awt.Container.validate(Container.java:1660)
	at java.desktop/javax.swing.RepaintManager$3.run(RepaintManager.java:757)
	at java.desktop/javax.swing.RepaintManager$3.run(RepaintManager.java:755)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:399)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:86)
	at java.desktop/javax.swing.RepaintManager.validateInvalidComponents(RepaintManager.java:754)
	at java.desktop/javax.swing.RepaintManager$ProcessingRunnable.run(RepaintManager.java:1896)
	at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:318)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:773)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:720)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:714)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:399)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:86)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:742)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

Possible duplicate of #1668. Have you changed the font to be used in Jadx?

I commit a workaround to handle exception. Not sure if this will be helpful through, because I don't know how to reproduce this issue. Anyway, it shouldn't spam with error windows anymore, like in #1658 :sweat_smile:
@xmpf if possible, please check latest unstable build.

authors: jpstotz skylot xmpf


text:
Language Missing

Hi,
Love this tool.. Currently i'm trying to decry-pt an app but Jadx wasn't able to reverse Hindi Language,
Just Square Boxes..

@dbzdivik can you try to change font to one which supports these characters?
Look like default font used by jadx not support unicode at all :(

@skylot Will Check btw Android Studio was able to de-compile correct Text..
Also One More Issue in with Preference Menu

@dbzdivik can you check latest unstable build?
I have reset default font to system font as it was before, hope it will help. Anyway you can always change font to one which supports your language.

authors: dbzdivik skylot


text:
🛑 PT Site Home is down

In c2ecbb4, PT Site Home (https://qbit.rexsky-blog.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: PT Site Home is back up in 432b344 after 11 minutes.

repo: skywalkerRex/RexSky-Blog-Uptime


text:
Custom softbreak problem

Hello! Just upgraded in my project from Quill 1.3.7 to 2.0.2 and noticed, there's a problem when putting a softbreak at the end of a block.
Steps for Reproduction
Here's a fiddle about the bug: https://jsfiddle.net/PascalAkh/0jat739b/7/
Expected behavior:
When pressing the first button in the fiddle it should simply create one list-item with a softbreak both texts.
Actual behavior:
It creates two list-items. I've created a matcher in the fiddle that you can uncomment. After that, it creates only one list-item with a softbreak inbetween. But now when you press the right button, it completly breaks when there is a  at the end of a block. (
test...

) (Most likely because there is no \n at the end)
Platforms:

Windows
Mac

Version:
Quill version: 2.0.2
Thanks for reading! I hope we can find a fix for this.

Same here. I've been trying to restore soft line breaks for couple weeks with no success. I tried to adopt every other solution I found for previous version with no success. I got to the point, when I am able to apply soft brek (
).
I've been testing this only in regular paragraphs. Quick test showed that it works also for list items.
Unfortunately after saving data with BRs and loading same content to new instance of quill, soft breaks are changed to

. I still can't resolve this.
My code is:
    import Quill from 'quill';
    import { EmbedBlot } from 'parchment';

    class SoftBreak extends EmbedBlot {
      static create(value) {
        let node = super.create(value);
        return node;
      }

      length() {
        return 1;
      }

      value() {
        return '\n';
      }

      static value(node) {
        return '\n';
      }
    }

    SoftBreak.blotName = 'softbreak';
    SoftBreak.tagName = 'BR';

    Quill.register(SoftBreak);

    function brMatcher(node, delta) {
      let newDelta = new Delta();
      newDelta.insert({softbreak: true});
      return newDelta;
  }

I have problems with softbreak everywhere. As far as I get is working softbreak in editor but when loading new quill instance with initial content containing softbreak, they are converted into paragraphs :/

authors: PAkhidue lkalinowski

repo: slab/quill


text:
Fix selected_option type hint

Summary
Simply fix type hint for selected_option parameter
Category (place an x in each of the [ ])

[ ] slack_sdk.web.WebClient (sync/async) (Web API client)
[ ] slack_sdk.webhook.WebhookClient (sync/async) (Incoming Webhook, response_url sender)
[ ] slack_sdk.socket_mode (Socket Mode client)
[ ] slack_sdk.signature (Request Signature Verifier)
[ ] slack_sdk.oauth (OAuth Flow Utilities)
[X] slack_sdk.models (UI component builders)
[ ] slack_sdk.scim (SCIM API client)
[ ] slack_sdk.audit_logs (Audit Logs API client)
[ ] slack_sdk.rtm_v2 (RTM client)
[ ] /docs-src (Documents, have you run ./scripts/docs.sh?)
[ ] /docs-src-v2 (Documents, have you run ./scripts/docs-v2.sh?)
[ ] /tutorial (PythOnBoardingBot tutorial)
[ ] tests/integration_tests (Automated tests for this library)

Requirements (place an x in each [ ])

[X] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[X] I've read and agree to the Code of Conduct.
[X] I've run python3 -m venv .venv && source .venv/bin/activate && ./scripts/run_validation.sh after making the changes.

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

authors: CLAassistant taekop

repo: slackapi/python-slack-sdk


text:
New surface: sidebar

Description
I'd like to suggest the creation of a new surface: sidebar.
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[x] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Idea
The idea is to be able to customize the sidebar of a specific channel.
Example of usage:

Each product on my company has a channel, in the sidebar we'd like to display info like changelog, latested version.
Each POD has a channel: we'd like to display relevant info about each POD: logo, roles, projects we are in
An app related to gamification could use the sidebar to show the ranking.
An app for poll could show the history and open polls.
A Github app could display open PR/issues

The sidebar could request new content every time it's opened.
The app could also push new content (like the incoming webhook).
It could also display buttons/actions
Please, let me know what do you think,

Yes this would be very useful for an in channel app.
The modal is too intrusive and blocking as a surface.
The message is too transient as a surface.
What is needed is a non transient sidebar surface that an app can use that might be shared among the participants of a conversation as a share document is.

authors: Falci sanielfishawy

repo: slackapi/slack-api-specs


text:
added tailwindcss to deps, including yarn.lock

Project was missing tailwindcss dependency so it wouldn't start. I also included yarn.lock as there was no lockfile present at all. Let me know if that was intentional and I can remove that and resubmit the PR.

Hi
My local set up only needed astro/tailwind in devDependencies and seems a few other people are fine also. But I am happy to add tailwindcss to dependencies if that makes it smoother for everyone to use the project.
However, I prefer not to commit lockfile in the git repo, if that is okay for you to resubmit.
Thanks a bunch!

I'll check again and see if adding tailwind into devDeps works and submit it there if so if that's your preference. It wasn't in devDeps in the repo either, so I think it needs to be somewhere, devDeps or deps, whichever makes sense.
I'll also remove the lock file and resubmit. I'll try to get that done in the next couple of days.

It is in the devDeps. See https://github.com/slamethendry/TinyRocket/blob/main/package.json, line 15.
"@astrojs/tailwind": "^5.0.0",
But I do not use yarn at all so I cannot check with yarn. And I don't know enough why it did not start for you.
Cheers.

I think that devdep is only for the Astro integration with tailwind, it doesn't include tailwind itself, which is why I didn't start for me. When I stamped out this template it wouldn't start until I added Tailwind as a dep. I'll double-check, though, before I resubmit the PR.

Just for completeness' sake, here's the error I see after running yarn install and yarn dev on the repo's main branch:
❯ yarn dev
yarn run v1.22.19
$ astro dev
10:00:06 AM [vite] Error when evaluating SSR module /Users/mitch/Dev/reference-only/astro-blog-template-personal-eval/tiny-rocket/astro.config.mjs: failed to import "@astrojs/tailwind"
|- Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'tailwindcss' imported from /Users/mitch/Dev/reference-only/astro-blog-template-personal-eval/tiny-rocket/node_modules/@astrojs/tailwind/dist/index.js
    at new NodeError (node:internal/errors:405:5)
    at packageResolve (node:internal/modules/esm/resolve:887:9)
    at moduleResolve (node:internal/modules/esm/resolve:936:20)
    at defaultResolve (node:internal/modules/esm/resolve:1129:11)
    at nextResolve (node:internal/modules/esm/loader:163:28)
    at ESMLoader.resolve (node:internal/modules/esm/loader:835:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:77:40)
    at link (node:internal/modules/esm/module_job:76:36)

[astro] Unable to load your Astro config

node:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^

[Failed to load PostCSS config: Failed to load PostCSS config (searchPath: /Users/mitch/Dev/reference-only/astro-blog-template-personal-eval/tiny-rocket): [Error] Loading PostCSS Plugin failed: Cannot find module 'tailwindcss'
Require stack:
- /Users/mitch/Dev/reference-only/astro-blog-template-personal-eval/tiny-rocket/postcss.config.cjs

(@/Users/mitch/Dev/reference-only/astro-blog-template-personal-eval/tiny-rocket/postcss.config.cjs)
Error: Loading PostCSS Plugin failed: Cannot find module 'tailwindcss'
Require stack:
- /Users/mitch/Dev/reference-only/astro-blog-template-personal-eval/tiny-rocket/postcss.config.cjs

(@/Users/mitch/Dev/reference-only/astro-blog-template-personal-eval/tiny-rocket/postcss.config.cjs)
    at load (file:///Users/mitch/Dev/reference-only/astro-blog-template-personal-eval/tiny-rocket/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:36429:11)
    at file:///Users/mitch/Dev/reference-only/astro-blog-template-personal-eval/tiny-rocket/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:36454:16
    at Array.map (<anonymous>)
    at plugins (file:///Users/mitch/Dev/reference-only/astro-blog-template-personal-eval/tiny-rocket/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:36453:8)
    at processResult (file:///Users/mitch/Dev/reference-only/astro-blog-template-personal-eval/tiny-rocket/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:36523:14)
    at file:///Users/mitch/Dev/reference-only/astro-blog-template-personal-eval/tiny-rocket/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:36647:14]

Node.js v18.17.1
error Command failed with exit code 1.

It's possible (IIRC) that if you're managing this repo on your machine and any parent directory has Tailwind installed that it could be satisfying the dependency from that parent directory and you won't see this error on your side. I keep my repos entirely separate from others so when I stamped this repo out from the template, it comes down "clean". I keep it outside any other repo in its directory hierarchy which might be allowing this error to come through.
Note that other Astro templates that use tailwind do include the tailwind dep in their package.json, so you might have something "interesting" going on with your setup.

This is really interesting. I tested with npm instead of yarn just to make sure the problem was present on both sides and npm installs tailwind as a dependency of @astro/tailwind so the problem doesn't show up at all there. That's likely why you're not seeing this issue, given that you don't use yarn. I've never seen this kind of discrepancy between package managers before. Let me dig a touch more. Since you don't include package-lock.json in your repo, yarn can't use that as a basis to migrate to yarn and build its yarn.lock file, so it's missing that dependency on its own. Very strange.

I just modified the upstream branch and removed yarn.lock, rebased and force-pushed the branch.

authors: newtmitch slamethendry

repo: slamethendry/TinyRocket


text:
Collections and cleanup advriskmin

Reminder : Links in moodle will be broken after we merge this. We can maybe merge it after the exam.

 Collections SL


text:
fix(README.md): Minor typo fixed

inkscape-figure -> inkscape-figure“s” in the terminal.
--
Appreciate your great work and the meticulous instruction :)

Ah, yes, thanks for this good catch!

authors: hatute sleepymalc

repo: sleepymalc/VSCode-LaTeX-Inkscape


text:
Using $this when not in object context

Hello!
I've tried to find a solution for my error but couldn't figure it out for hours.
I'm trying to install the twig extension and followed this tutorial:
http://www.slimframework.com/docs/v3/features/templates.html
After I get an error message saying:
Fatal error: Uncaught Error: Using $this when not in object context in D:\xampp\htdocs\...\public\index.php:31

My code is (which is basicly a straight copy from the tutorial):
<?php
use Psr\Http\Message\ResponseInterface as Response;
use Psr\Http\Message\ServerRequestInterface as Request;
use Slim\Factory\AppFactory;
use Slim\Views\Twig;
use Slim\Views\TwigMiddleware;

require __DIR__ . '/../vendor/autoload.php';

$app = AppFactory::create();

// Get container
$container = $app->getContainer();

// Register component on container
$container['view'] = function ($container) {
    $view = new \Slim\Views\Twig('../templates', [
        'cache' => '../cache'
    ]);

    // Instantiate and add Slim specific extension
    $router = $container->get('router');
    $uri = \Slim\Http\Uri::createFromEnvironment(new \Slim\Http\Environment($_SERVER));
    $view->addExtension(new \Slim\Views\TwigExtension($router, $uri));

    return $view;
};

$app->get('/',function($request, $response, $args) {

    return $this->view->render($response, 'index.html.twig', [ <--- this is the 31st line, the problem is here
        'name' => $args['name']
    ]);

});

$app->run();

I've tried to look for such errors and questions but these didn't seem to help.
https://steemit.com/utopian-io/@alfarisi94/how-to-use-twig-templating-in-slim
https://stackoverflow.com/questions/31363145/slim-php-fatal-error-using-this-when-not-in-object-context.
The basic hello world example in the slim installation worked.
So now I'm stumped. Can someone help me and tell me how can I fix this?
Thank you very much!

You’re using Slim 4 and referring to v3 documentation.
Please refer to the Twig-View example on the README here

Thank you, now it works!

repo: slimphp/Slim


text:
discuss allowing runs to be deleted

A recurring question in SlipStream tutorials is "How do I delete a run?".  This comes up primarily because people want to keep their dashboard clear of runs that they are no longer interested in.
The minimal solution would be to add a flag to the run that marks a run as deleted and would not be returned in lists of runs by default.  This would hide the run, but retain all of the information associated with it (including the log files).
A more involved solution would allow the run to actually be deleted, removing the information from the database and deleting any log files associated with it.  Such an implementation would require a check that all of the allocated resources have been freed before deleting the run.  It would also require us to distinguish the "terminate" action from the "delete" action in the API, possibly changing the current semantics of the API.
This issue should start with a discussion of the desirability of this feature and its scope.  An issue should then be created for the implementation, which possibly could be treated as part of the switch from v2 to v3 of SlipStream.

The minimal solution would be to add a flag to the run that marks a run as deleted and would not be returned in lists of runs by default. This would hide the run, but retain all of the information associated with it (including the log files).

For reference, there is an open discussion about add filtering features in the dashboard tables, see slipstream/SlipStreamUI#252. One of the filters would be running/terminated runs.

On a heavily used SS instance the size of the SS DB eventually grows and this creates different issues (https://github.com/slipstream/SlipStreamServer/issues/725, https://github.com/SixSq/tasklist/issues/777) on running the DBMS (hsqldb in our case).
We need to introduce retirement (cleanup/deletion) of the old runs.  Maybe a simple cleanup of the values of the RTPs on the “old” runs can already reduce the size of the db.
Here is the case of the DB on https://bb1.hnx.helix-nebula.eu where the size of DB is almost 1GB now
-rw-r--r-- 1 slipstream slipstream 301K May 24 16:01 slipstreamdb.log
-rw-r--r-- 1 slipstream slipstream 953M May 24 11:10 slipstreamdb.script
[root@bluebox SlipStreamDB]#

NB! I had to set Xmx to 5GB on BB, otherwise slipstream DB wouldn’t start with OOM.

authors: konstan loomis rbf

repo: slipstream/SlipStreamServer


text:
[e2e]: go schedule main noldflags-multi-uses slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/4060461855
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.noldflags-multi-uses.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.schedule.main.noldflags-multi-uses.slsa3.yml
Trigger: schedule
Branch: main
Date: Wed Feb  1 03:04:26 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/4070758746
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.noldflags-multi-uses.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.schedule.main.noldflags-multi-uses.slsa3.yml
Trigger: schedule
Branch: main
Date: Thu Feb  2 02:53:45 UTC 2023
Tests are passing now. Closing this issue.

repo: slsa-framework/slsa-github-generator


text:
[feature] Add a security policy file

I'm not sure this should be handled via an OSSF-org policy or if individual projects needs their own.
@david-a-wheeler can you advise?

Related #547

Shall we close this issue then?

We should have a SECURITY.md that at least points to the Open-SSF policy I think.

The security policy should also identify the "security team" of members that are knowledgeable about security and will address security issues in order to better comply with OpenSSF security best practices.

authors: ianlewis laurentsimon


text:
docs: Document that actions/download-artifact@v3 must be used to download provenance

Fixes https://github.com/slsa-framework/slsa-github-generator/issues/3066

Could you add this under the "Known Issues" section of the READMEs? (If there isn't one then you can add it)

I've moved the notes and updated the tocs.

Thanks!

authors: ianlewis jkreileder


text:
model missing predict_proba

Hey @slundberg
Does it make sense that we can interpret a model without predict_proba() function using KernelSHap.
Let's take the example of kmeans cluster on scikitlearn. By running it, I expect that I have to get an error  but everything was running well! I debug the code and I remark that you're using the class Id (the ouput of kmens.predict) which does not make sense (unless I missed something). Could you clarify!

I think your question (if I understand it correctly) relates to predictions on unsupervised models, which has a good answer here: #1052

authors: jimbudarz rafikg

repo: slundberg/shap


text:
pervert "metalsmash117" making porn of my oc

ban him.

he may rename himself to metalsmash118 or metalsmash119, metalsmash120, metalsmash116, metalsmash115, etc.

what the fuck

This is not an issue that pertains to the game's workings. This is a different issue that should be handled differently and not by us.

Yeah

Moderation tools are being implemented

license: WTFPL


text:
HELP - Calling Stored Procedures only return bool(false)

@smadeira @codeflaretech
I am calling a stored procedure that doesn't need any parameters  like so...
<?php

require_once __DIR__ . '/vendor/autoload.php';

use MinistryPlatformAPI\MinistryPlatformProcAPI as PROC;

// Get environment variables
$dotenv = Dotenv\Dotenv::create(__DIR__);
$dotenv->load();

// Attempt to authenticate to the MP API

$mp = new PROC();
$mp->authenticate();

$events = $mp->proc('api_PCPC_EventsWeb')
             ->exec();

var_dump($events);

and it returns a bool(false)
When I try it with a required parameter of @DomainID, I get the same result.
<?php

require_once __DIR__ . '/vendor/autoload.php';

use MinistryPlatformAPI\MinistryPlatformProcAPI as PROC;

// Get environment variables
$dotenv = Dotenv\Dotenv::create(__DIR__);
$dotenv->load();

// Attempt to authenticate to the MP API

$mp = new PROC();
$mp->authenticate();

$input = ['@DomainID' => 1];
$events = $mp->proc('api_PCPC_CurrentEventsWithImages')
             ->procInput($input)
             ->exec();

var_dump($events);

When I execute these stored procedures from the SQL Server Studio, they run with no problem.
Any thoughts of what could be causing this? Could it be permissions issues? If so, what security roles do I need in MP to run stored procedures from the PHP API Wrapper?

It was a permissions issue. After digging around, I ran across Faith Chapel's MP Rest API documentation which explained it perfectly and helped me remedy the issue.
https://github.com/faithchapel/mp-rest-documentation#stored-procedure-access
Stored Procedure Access
To use a stored procedure two things need to happen. First, there needs to be a record referencing the stored procedure in the System Setup/API Procedures page. Secondly, the user needs a security role applied to it with the API Procedure linked to the security role. This is done through the API Procedures sub page on the Security Roles page.

Just a thought...
You might want to add this NOTE to the README for Stored Procedures.

repo: smadeira/ministry-platform-api


text:
Usage documentation

Add README content for importing docs, rendering with the ContentBlock component, and enabling syntax highlighting.

Usage is awkward currently, so holding off until it's refactored.

repo: smallstep/hello-mtls


text:
Wraps all webpack loader rules into a single oneOf rule

https://github.com/smapiot/piral/blob/a6f52016d1e8720feeed91d2fa5234ca72d3fbba/src/tooling/piral-cli-webpack/src/webpack/common.ts#L98
Would you agree to wraps all webpack loader rules into a single oneOf rule?

"oneOf" will traverse all following loaders until one will match the requirements. When no loader matches it will fall back to the "file" loader at the end of the loader list. create-react-app

This eliminates the need to enumerate other unknown file types, such as xslx.

Sounds good to me. I'm just not sure if / how it will affect "changing" that default config. Will definitely be more complicated, but should be doable (see https://github.com/smapiot/piral/blob/develop/src/tooling/piral-cli-webpack5/extend-config.js).

extend-config.js is under control.
But it's a break change for users who modify the rules of config in webpack.config.js.

But it's a break change for users who modify the rules of config in webpack.config.js.

Yeah that's true. My feeling is that this item would be something for 0.15 - and would be part of the breaking changes. As long as we can keep the API of extend-config.js stable I feel good about this.

authors: FlorianRappl carvinlo

repo: smapiot/piral


text:
Fix some parts of webpage by making it responsive

Here image and the content in the card are overflowing on mobile view. I would like to fix this problem. Please assign me

@SNEHAL311998

Nice work figuring out the bug 🌟
Whenever you report a responsiveness issue, please be specific about the screen size or the default model type. For example, I can see that this issue occurs in the iPhone xe/5 models.
You are assigned this with a deadline of 3 days from today

Thank you for assigning.Sorry for the late reply since I was not well for two days.

Will you be working on this?  @SNEHAL311998

No @tusharnankani you can go ahead. If you need help do let me know

I want to work on the responsive

I want to work on the responsive

Raise a new issue for this and start working on that


text:
Responsive switch and color changed

Fix #456

@smaranjitghose Please check :)

Great Work!


text:
:bug: added the dates in the timeline

before

after

@anushbhatia have a look.!


text:
better bulk_upload handling for pip installed cumulus library

When we are dealing with an installed version, we should assume that export should write to someplace outside of the python lib folder. We'll then need the bulk uploader to target that directory explicitly.

addressing via #44

repo: smart-on-fhir/cumulus-library-core