Fails to render if PHP module open_ssl is not present.

PHP Fatal error:  Call to undefined function visualCaptcha\openssl_random_pseudo_bytes() in D:\Documents\UniServerZ\www\visualCaptcha\vendor\emotionloop\visualcaptcha\src\visualCaptcha\Captcha.php on line 223, referer: http://localhost/visualCaptcha/public/
Perhaps some kind of warning could be added?

I think I'd prefer adding it as a requirement in the README than having to deal with it not existing, etc.
If you think that makes sense, I'd gladly accept a PR for that! :D

ReadMe works too.
Or there's this:
if (!extension_loaded('openssl'))
{
    echo "openssl extension is required.";
}


Build(deps-dev): Bump eslint from 8.5.0 to 8.6.0 (#2309)

Pull request opened by the merge tool on behalf of #2309

Merged in ef33f6ea364b7d303595a03c74dca39f70cf1340


Expose Agent#id to Liquid (and to JavaScriptAgent)

It would be handy if, for example, a JavaScriptAgent could identify the source of an incoming event by ID and store per-source information in the memory.

Thanks for the quick review!


Function add keycode for SmartTV

Is your feature request related to a problem? Please describe.
I'm working on a project on SmartTV using React-Hotkeys. Back key on webOS does not receive
Describe the solution you'd like
Add method to add KeyCode like mousetrap
Describe alternatives you've considered
Just open more self-defined uses of mouestrap as v1 version.
Additional context
The control keys on the remote have different codes. Example: webos: "461:" BackTV ", tizen:" 10009: "BackTV"

Hi @tconns, thanks for filing your issue.
I have no experience working with webOS, so you're going to have to help me out here. Could you please confirm you're referring to the addKeycodes method on mousetrap?
Could you also confirm in your examples that the KeyboardEvents have a code or keyCode of 461, and a key of "BackTV"?

Sorry for the late!
In v1, I used addKeycodes method on mousetrap, it works normally.
But in v2 version, addKeycode doesn't work, looks like mousetrap can't override listKeycode in library.
You can try the example of replacing the 'numlock' key name with the 'BackTV' key. Then call 'BackTV' to the handlers.

(Tizen) Mousetrap.addKeycodes({
10009: "BackTV",
10252: "PlayPauseTV",
412: "PrevTV",
415: "PlayTV",
417: "NextTV",
19: "PauseTV",
413: "StopTV"
});
(WebOS)Mousetrap.addKeycodes({
461: "BackTV",
412: "PrevTV",
415: "PlayTV",
417: "NextTV",
19: "PauseTV"
});
Can you add a method to add these keycode? Right now I am adding directly to the library's DictionaryKey but it still has some errors.

This feature is now available in v2.0.0-pre8.


Add python_executable param to launch_experiment

If the path is not specified, it will default to the interpreter from
the sys module if available, otherwise "python". This better supports
non virtualenv/conda environments.
Fixes #130

Looks great, thanks!


Updating main with dev

Removing buttons from mode bar

Will re-open this pull request when dev is updated with csv removals


Add IOPS and encrypted columns to CloudVolume

Amazon EBS provides information about volume's IOPS and encryption
status as part of the cloud volume model so this patch extends the
existing cloud model to include these two attributes.
@miq-bot add_label storage, sql migration

@gberginc - Given that this PR requires a schema change and the schema freeze took effect on 3/27, I expect this will not be merged until after the release.
cc @blomquisg

@bronaghs yeah, even though we've been constantly discussing the freeze, I honestly missed this constraint.
Is there any other way to squeeze this into E release? For example, by using custom attributes where iops and encryption would be stored for ebs volumes?

This will probably not be the case, as cloud volumes doesn't seem to have a reference to custom attributes either.

@gberginc custom attribute would be one way (the reference is on the CustomAttribute side as a polymorphic relation :resource). You would need to do a Fine specific PR though and if we would want this format in master, we should have a data migration, that would delete the custom attributes and put the data in the actual table attributes.

looks great 👍

@Ladas @bronaghs we've updated the inventory in the Amazon provider so that the tests pass in our environment with these migrations applied.
Is there anything else needed for this to be merged? I am asking mainly because this UI PR depends on a similar change to the snapshots (adding encryption information) and would like to know if it's ok to do it like this.

@Fryguy looks good to me, can we merge?


Risklist

If anyone would like to read and come with improvements, that would be greatly appreciated
Also added work hour lists for Sebastian and Daniel

.


UTF-16 compression seems to be broken on Safari

Hi there! First of all, this library is amazing. Thanks. The results were great.
I started using this library few days ago and have been testing it mainly on Chrome/Windows. I was using the UTF-16 compression method. It worked great. However, on latest Safari 10/Sierra, the method seems to be broken, though no errors are thrown. So I've tested the compress method and it also fails in my app. The only way I could get it working was to use the base64 method. Since then, it's been working on all browsers I could test.
I'm not sure what's the problem as I've not done any deeper investigation.

Hey! We had a similar problem. In fact, what's happening is that UTF-16 gets "converted" to UTF-8, which keeps the "string" as is, but will break some of the actual bytes once transferred by HTTP.
There's no clean solution but to give up on UTF16 encoding, and instead use Base64 or EncodedURIComponent.

Provide a test case here:
source string
{"uids":["ph0190"],"type":"userBasic"}
when compressedToUTF16 in safari and decompressed in other browser ie. Chrome, it comes out
{"uids":["ph019i"u],"type":"userBasic"}

auromota, was there a point in the docs where you expected to see this caveat where it was not mentioned? If so we'll keep the issue open until that is fixed, otherwise I guess we can close this, right?

Well, compressToUTF16 is supposed to work on all browsers. At least according to http://pieroxy.net/blog/pages/lz-string/index.html#inline_menu_6

Same here, decompressFromUTF16 and compressToUTF16 do not work on safari

To anyone coming here from Google: if you're sending compressed data (doesn't matter if you use compress or compressToUTF16) via XMLHTTPRequest, you have to pass the data to XMLHTTPRequest.send as a Uint8Array and it will work as expected - it's basically like an asynchronous file upload which works in all browsers.
Using a simple Express middleware in Node.js as a backend, the difference is easily spotable:
Sending data as a Uint8Array:
[ <Buffer 00 04 00 40 0c 00 20 0a 00 60 07 00 04 01 20 05 00 68 00 c0 16 00 70 03 c0 00 80 22 00 48 03 20 02 80 2a 00 68 03 a0 01 80 26 00 58 03 60 03 80 2e 00 ... > ]

Sending data as a string, browsers will remove all non-printable characters (e.g. \u0000) from the body, effectively corrupting the data:
[ <Buffer 04 40 e0 b0 80 e2 80 8a 60 dc 80 d0 81 e2 80 85 68 c3 80 e1 98 80 e7 80 83 ec 80 80 e8 80 a2 48 cc a0 ca 80 e2 a8 80 e6 a0 83 ea 80 81 e8 80 a6 58 cd ... > ]

Any workarround for this? Facing the same issue, but I can't send UInt8Arrays since I am sending the encoded strings as JSON. Maybe an utf16 version without non-printable characters?

From my testing only sending data as binary worked consistently well. You could use base64 encoding but this may rip the benefits of compression.

You are probably using the wrong library if you want your results in a UInt8Array. The whole point of LZ-String is to produce "packed" string, since that's what you store in localStorage, and browsers store it as UTF-16. In order to put a compressed string in JSON, you may use compressToBase64. That is guaranteed to stay untouched by any conversion of ASCII, Unicode or anything else. Of course, the compression ratio is much worse.

Same here, decompressFromUTF16 and compressToUTF16 do not work on safari or IE 11

Can you provide a working page that demonstrate the issue?
Most of the time the issue will be conversions between UTF16 to UTF8 and back to UTF16. This is not a binary-conservative conversion. In other words, you will get back a string that is not the same than the one you began with. Obviously LZ-String cannot decompress it successfully.
LZ-String is meant to compress strings into other strings to be stored in locatStorage, which uses UTF-16, so there is no conversion.

@pieroxy sorry it was 4 years ago, I do not have access to the project anymore

There are now support functions for binary safe conversion to/from uint8array (useful for network transport as well) - have a look at https://github.com/pieroxy/lz-string/blob/master/src/Uint8Array/utils.ts (bear in mind that v2 is still in development and not released to npm yet!)


Sample UTs do not follow UT template

The samples pointed out in
https://github.com/PowerShell/DscResources/blob/master/TestsGuidelines.md

do not follow the test template (e.g. they wrap the entire test within InModuleScope)

Thanks.
Could you file an issue about that in xNetworking repo where these samples come from (we need to update them)?
Also, do you know of any real world implementation of the updated samples so that we could point to those instead?

These templates are only examples and do not need to be strictly adhered to.
@mbreakey3 is updating the test templates to make them more clear.
If there is an issue with xNetworking tests please open an issue under that repository.

Norberto explained this offline.
Re-opening since sample links do need to be changed until this is fixed.

I'd like to fix the xNetworking ones as well anyway: @narrieta  - are you going to log this against xNetworking? I'll get onto fixing xNetworking as soon as I can.

Done: https://github.com/PowerShell/xNetworking/issues/132

I think this one is solved. To me it looks like the links to samples do link to correct sample that follow the templates. I suggest we close this issue.


[Bug]: LIKE realm query string operator returning no results

What happened?
Evening, hope you are all well.
We attempted a query using the LIKE string operator (https://www.mongodb.com/docs/realm/realm-query-language/#string-operators) and no results were returned. A query with the CONTAINS string operator is working as expected.
LIKE is particularity of interest due to the more fuzzy search nature of the query - if not supported yet can we raise a feature request please?
If only some of the query string operators are supported at https://www.mongodb.com/docs/realm/realm-query-language/#string-operators, is it possible to know which ones are not supported?
Thank you again in advance!
Repro steps
These 2 examples return 0 x results (using LIKE):
realm.all<T>().query("name LIKE 'Dig'");
realm.all<T>().query("name LIKE[c] 'Dig'");

Yet the below 2 examples return many results (using CONTAINS):
realm.all<T>().query("name CONTAINS[c] 'Dog'");
realm.all<T>().query("name CONTAINS[c] 'D'");

Version
0.6.0+beta
What Realm SDK flavor are you using?
MongoDB Atlas (i.e. Sync, auth, functions)
What type of application is this?
Flutter Application
Client OS and version
macOS 12.6
Code snippets
No response
Stacktrace of the exception/crash you're getting
n/a

Relevant log output
n/a

Hey, for LIKE to behave like CONTAINS, you need to add wildcards at the beginning and the end of the search phrase. For example, these two queries are equivalent:
realm.all<Person>("name CONTAINS[c] 'a'");
realm.all<Person>("name LIKE[c] '*a*'");

If you don't have wildcards, then LIKE will try to match exactly, meaning for your example, only objects where name == 'Dig' will be returned.

Thanks for the incredible fast help @nirinchev, very very appreciated and hope you are having a good week so far.
For the examples below, Person.name is "Dog".
This returns 0 x results:
realm.all<Person>("name LIKE[c] '*dig*'");
This returns 1 x results - so perhaps LIKE is 'exact matching' which sounds like something is not right?
realm.all<Person>("name LIKE[c] '*dog*'");

Seems I have misunderstood the documentation (https://www.mongodb.com/docs/realm/realm-query-language/#string-operators), apologies.
I can now see that the wildcard string "d?g" matches "dog", "dig", and "dug", but not "ding", "dg", or "a dog".
Please feel free to close this if the above is my mistake and thank you again for your help on this!

Ah, I see. Yes, unfortunately, LIKE doesn't do fuzzy matching like a true full text search would do. It's instead more similar to a very limited regex search, where you provide a pattern and we compare each entry against that pattern.
We do have some very early POCs of full text search, but it's not something we're actively working on right now and as such, we don't have a clear timeline for when it might land.


The installation of plugins doesn't work if .SHELLFLAGS are too fancy…

… in the local MAKEFILE

I keep finding stuff


Kinetic model of ionisation

We should develop a kinetic model of ionisation rates I = I_i(p), satisfying

where I_i(T) is the ADAS ionisation rate coefficient for ion i and f_M a Maxwellian distribution at temperature T, which should be approximately satisfied at all temperatures T for which ADAS has data. The kinetic ionisation rate I_i(p) should depend on ion species (atomic number Z and charge Z0) but not temperature. Here we neglect the density effect that is included in the ADAS data, and we should therefore match the ADAS coefficients in the limit of zero density (OR let the kinetic ionisation rate formula be density dependent, but that seems significantly more complicated).
This would allow us to replace fluid ionisation rates by this kinetic description of ionisation, except for when the bulk is not resolved (i.e. collfreq_mode SUPERTHERMAL or hottail grid disabled in which case the fluid contribution should be added).
We would still use fluid recombination, since I am not aware of a good kinetic model for it, and the recombination cross section would decay rapidly with energy and thus be uninteresting for fast electrons.

The idea I have for the implementation is as follows (partly outlined in doc/notes/theory):
The seminal Burgess & Chidichimo paper (which for example is used in the FLY Los Alamos atomic physics code) gives the electron impact ionization cross section as

where Delta I_Z and N_a are the ionisation energy and occupation number of an atomic subshell, respectively, and C an undetermined order-unity constant (with recommended value ~2), and the expression should be summed over the subshells of the ion (typically the dominant contribution arising from one or two shells). The problem is that the ionisation energy is quite complicated, because sometimes it should be taken as an excitation energy needed for secondary (Auger) ionization and sometimes the binding potential of the ion. The parameter beta^star can be modelled as prescribed in the paper and describes a near-threshold modification to the cross section. Given these parameters, the Garland model describes how the validity of the formula can be extended to relativistic energies.
Rather than figuring out exactly how these parameters should be picked for each ion, and hoping that the result will match ADAS, I propose that we fit coefficients in the formula to ADAS data. Then, we could modify get_adas.py such that for each ion that it downloads temperature data for, it also performs a non-linear least-squares fit of the above formula (integrated over a maxwellian at each temperature) and saves the resulting coefficients.

I have investigated the Burgess & Chidichimo formula above (extended according to Garland), and took C*N_a, Delta I_Z and beta^star as 3 free parameters that were fitted to ADAS data after integrating the cross-section over a Maxwellian. This model seems to work really well (typical errors at most around tens of %, spanning tens of orders of magnitude in ionization rates), even without summing over multiple shells.
Therefore, I think that we should move forward with this simple model, as it appears to adequately cover the entire ADAS range and would give a fully kinetic description valid at all energies.
I will start the implementation of the equation term in the DREAM kernel and work out a data structure for these fit-coefficents. Using hardcoded values for argon from my test script I will try to explore the consequences of using kinetic rather than fluid ionization; hoping that this may stabilize some scenarios that wouldn't converge before.

I have finally implemented the source, but using placeholder hardcoded coefficients for only H, Ne and Ar. I use the full f_hot jacobian, which turns out to be ridiculously expensive.
I propose an approximation where we use the ADAS IonRateEquation jacobian, but the kinetic vectorelements (possibly with an additional approximate nhot jacobian)

This now works as intended as far as I know. It remains to add the corresponding binding-energy heating term into the T_cold equation, in the form of an update to the IonizationHeatingTerm which was supposed to fill this purpose. This is annoyingly tricky, because the ADAS part of the ionization coefficient should not be added (since it is accounted for via the RadiatedPowerTerm). We therefore probably wish to subtract the ADAS rates from the net kinetic rates in this term.

Fixed the approximate jacobian. Only remains to implement updated kinetic rate coefficients to achieve higher accuracy and with more elements.

Fixed the approximate jacobian. Only remains to implement updated kinetic rate coefficients to achieve higher accuracy and with more elements.

(Closing, as this has been available in DREAM for quite a while now)


integration-cli: add info to fatal log

Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

LGTM

LGTM


Add fancier CI badges showing number of tests and/or test coverage

We could add badges for each test suite showing the number of tests, like those at https://github.com/onnx/onnx-mlir?tab=readme-ov-file#onnx-mlir

These projects could help:

https://github.com/marketplace/actions/ci-badges (hooks up to coverage reports, generates badges, uploads to gists, provides URLs that can be used with badge endpoints to render SVGs)
https://pypi.org/project/pytest-local-badge/ (generates local svgs)

I'd see how other projects use pytest-local-badge. Maybe we can push the svg files to a branch automatically.

Not finding too many resources for CMake/CTest badges. Here are a few though, focused on CDash (dashboards):

https://cmake.org/cmake/help/latest/guide/tutorial/Adding Support for a Testing Dashboard.html
https://cmake.org/cmake/help/book/mastering-cmake/chapter/CDash.html
https://github.com/jcfr/CBadge


Change of license? GPLv3+ → Apache 2.0

Dear all,
Trafilatura is now widely used and it has become apparent that the GPL license is not prevalent in Python packages, its potential implications are also not easily understood. To better align with community standards and promote ongoing development, I plan to transition Trafilatura's software license to Apache 2.0.
As you have contributed lines still currently in use it would be best to get your agreement in order to move forward on this. For the sake of simplicity I use the list provided by Github and you may be mentioned below even if your contributions focus on documentation or evaluation files.
For reference, other packages of mine concerned by similar changes:

https://github.com/adbar/courlan/issues/80
https://github.com/adbar/htmldate/issues/131

If you agree please add a corresponding message to this thread. If not please contact me or open a public discussion.

@felipehertzer: pending
@vbarbaresi: pending
@knit-bee: :heavy_check_mark:
@LukasBBAW: pending
@phongtnit: pending
@andremacola: :heavy_check_mark:
@idoshamun: :heavy_check_mark:
@edkrueger: pending
@EllieLockhart: pending
@tonyyanga: pending
@immortal-autumn: pending
@glacierck: pending
@deedy5: :heavy_check_mark:
@k-sareen: pending
@dsgibbons: pending
@mrienstra: pending
@LaundroMat: pending
@sdondley: pending
@Korben00: pending
@awwitecki: :heavy_check_mark:
@feltcat: pending
@muellermartin: :heavy_check_mark:
@marksmayo: pending
@sashkab: pending
@HeLehm: :heavy_check_mark:
@tomwojcik: :heavy_check_mark:
@Yomguithereal: :heavy_check_mark:
@Ashikpaul: :heavy_check_mark:
@Maddesea: pending
@jaesivsm: pending
@dmoklaf: :heavy_check_mark:

I agree.

I agree.

I agree

Hello Adrien,
of course this looks great.
And thanks again for your outstanding work ! I use Trafilatura with great pleasure every day.
Regards,
Raphael Geronimi
+33777947835

Le 21 févr. 2024 à 15:10, Adrien Barbaresi @.***> a écrit :
Dear all,
Trafilatura is now widely used and it has become apparent that the GPL license is not prevalent in Python packages, its potential implications are also not easily understood. To better align with community standards and promote ongoing development, I plan to transition Trafilatura's software license to Apache 2.0.
As you have contributed lines still currently in use it would be best to get your agreement in order to move forward on this. For the sake of simplicity I use the list provided by Github and you may be mentioned below even if your contributions focus on documentation or evaluation files.
For reference, other packages of mine concerned by similar changes:
adbar/courlan#80 https://github.com/adbar/courlan/issues/80
adbar/htmldate#131 https://github.com/adbar/htmldate/issues/131
If you agree please add a corresponding message to this thread. If not please contact me or open a public discussion.
@felipehertzer https://github.com/felipehertzer: pending
@vbarbaresi https://github.com/vbarbaresi: pending
@knit-bee https://github.com/knit-bee: pending
@LukasBBAW https://github.com/LukasBBAW: pending
@phongtnit https://github.com/phongtnit: pending
@andremacola https://github.com/andremacola: pending
@idoshamun https://github.com/idoshamun: pending
@edkrueger https://github.com/edkrueger: pending
@EllieLockhart https://github.com/EllieLockhart: pending
@tonyyanga https://github.com/tonyyanga: pending
@immortal-autumn https://github.com/immortal-autumn: pending
@glacierck https://github.com/glacierck: pending
@deedy5 https://github.com/deedy5: pending
@k-sareen https://github.com/k-sareen: pending
@dsgibbons https://github.com/dsgibbons: pending
@mrienstra https://github.com/mrienstra: pending
@LaundroMat https://github.com/LaundroMat: pending
@sdondley https://github.com/sdondley: pending
@Korben00 https://github.com/Korben00: pending
@awwitecki https://github.com/awwitecki: pending
@feltcat https://github.com/feltcat: pending
@muellermartin https://github.com/muellermartin: pending
@marksmayo https://github.com/marksmayo: pending
@sashkab https://github.com/sashkab: pending
@HeLehm https://github.com/HeLehm: pending
@tomwojcik https://github.com/tomwojcik: pending
@Yomguithereal https://github.com/Yomguithereal: pending
@Ashikpaul https://github.com/Ashikpaul: pending
@Maddesea https://github.com/Maddesea: pending
@jaesivsm https://github.com/jaesivsm: pending
@dmoklaf https://github.com/dmoklaf: pending
—
Reply to this email directly, view it on GitHub https://github.com/adbar/trafilatura/issues/512, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXAU6Q6IEUAZBXD52HYBHLYUX55RAVCNFSM6AAAAABDTD5ZRCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE2DMOBVGI2DKMI.
You are receiving this because you were mentioned.

My tool is GPLv3. I don't really know if this is compatible with apache 2.0, the subject seems complicated, but I personally don't really care and don't have an issue with you migrating to apache 2.0.

I agree
On Wed, 21 Feb 2024 at 15:10, Adrien Barbaresi @.***>
wrote:

Dear all,
Trafilatura is now widely used and it has become apparent that the GPL
license is not prevalent in Python packages, its potential implications are
also not easily understood. To better align with community standards and
promote ongoing development, I plan to transition Trafilatura's software
license to Apache 2.0.
As you have contributed lines still currently in use it would be best to
get your agreement in order to move forward on this. For the sake of
simplicity I use the list provided by Github and you may be mentioned below
even if your contributions focus on documentation or evaluation files.
For reference, other packages of mine concerned by similar changes:

adbar/courlan#80 https://github.com/adbar/courlan/issues/80
adbar/htmldate#131 https://github.com/adbar/htmldate/issues/131

If you agree please add a corresponding message to this thread. If not
please contact me or open a public discussion.

@felipehertzer https://github.com/felipehertzer: pending
@vbarbaresi https://github.com/vbarbaresi: pending
@knit-bee https://github.com/knit-bee: pending
@LukasBBAW https://github.com/LukasBBAW: pending
@phongtnit https://github.com/phongtnit: pending
@andremacola https://github.com/andremacola: pending
@idoshamun https://github.com/idoshamun: pending
@edkrueger https://github.com/edkrueger: pending
@EllieLockhart https://github.com/EllieLockhart: pending
@tonyyanga https://github.com/tonyyanga: pending
@immortal-autumn https://github.com/immortal-autumn: pending
@glacierck https://github.com/glacierck: pending
@deedy5 https://github.com/deedy5: pending
@k-sareen https://github.com/k-sareen: pending
@dsgibbons https://github.com/dsgibbons: pending
@mrienstra https://github.com/mrienstra: pending
@LaundroMat https://github.com/LaundroMat: pending
@sdondley https://github.com/sdondley: pending
@Korben00 https://github.com/Korben00: pending
@awwitecki https://github.com/awwitecki: pending
@feltcat https://github.com/feltcat: pending
@muellermartin https://github.com/muellermartin: pending
@marksmayo https://github.com/marksmayo: pending
@sashkab https://github.com/sashkab: pending
@HeLehm https://github.com/HeLehm: pending
@tomwojcik https://github.com/tomwojcik: pending
@Yomguithereal https://github.com/Yomguithereal: pending
@Ashikpaul https://github.com/Ashikpaul: pending
@Maddesea https://github.com/Maddesea: pending
@jaesivsm https://github.com/jaesivsm: pending
@dmoklaf https://github.com/dmoklaf: pending

—
Reply to this email directly, view it on GitHub
https://github.com/adbar/trafilatura/issues/512, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZXBEZW7YA56QQP2RZ77VTYUX55RAVCNFSM6AAAAABDTD5ZRCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE2DMOBVGI2DKMI
.
You are receiving this because you were mentioned.Message ID:
@.***>

I don't mind

I agree

I agree

@Yomguithereal You can certainly include a package under an OSI-approved license in a package under GPL. The other way round is unclear and the answer can depend on who you're asking and which software community you're in (e.g. in R the GPL is much more common).
See the discussion in the datatrove issue above if you're interested.

I agre.

I agree.

Agree

No objections from me (same as earlier).

Fine by me!
On Wed, Feb 21, 2024 at 2:12 PM feltcat @.***> wrote:

No objections from me (same as earlier).
—
Reply to this email directly, view it on GitHub
https://github.com/adbar/trafilatura/issues/512#issuecomment-1958103805,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFDNDWPK3U247JGNWP3OFDYUZWOJAVCNFSM6AAAAABDTD5ZRCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJYGEYDGOBQGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

I agree.

I agree.

I agree
On Wed, 21 Feb 2024, 15:10 Adrien Barbaresi, @.***>
wrote:

Dear all,
Trafilatura is now widely used and it has become apparent that the GPL
license is not prevalent in Python packages, its potential implications are
also not easily understood. To better align with community standards and
promote ongoing development, I plan to transition Trafilatura's software
license to Apache 2.0.
As you have contributed lines still currently in use it would be best to
get your agreement in order to move forward on this. For the sake of
simplicity I use the list provided by Github and you may be mentioned below
even if your contributions focus on documentation or evaluation files.
For reference, other packages of mine concerned by similar changes:

adbar/courlan#80 https://github.com/adbar/courlan/issues/80
adbar/htmldate#131 https://github.com/adbar/htmldate/issues/131

If you agree please add a corresponding message to this thread. If not
please contact me or open a public discussion.

@felipehertzer https://github.com/felipehertzer: pending
@vbarbaresi https://github.com/vbarbaresi: pending
@knit-bee https://github.com/knit-bee: pending
@LukasBBAW https://github.com/LukasBBAW: pending
@phongtnit https://github.com/phongtnit: pending
@andremacola https://github.com/andremacola: pending
@idoshamun https://github.com/idoshamun: pending
@edkrueger https://github.com/edkrueger: pending
@EllieLockhart https://github.com/EllieLockhart: pending
@tonyyanga https://github.com/tonyyanga: pending
@immortal-autumn https://github.com/immortal-autumn: pending
@glacierck https://github.com/glacierck: pending
@deedy5 https://github.com/deedy5: pending
@k-sareen https://github.com/k-sareen: pending
@dsgibbons https://github.com/dsgibbons: pending
@mrienstra https://github.com/mrienstra: pending
@LaundroMat https://github.com/LaundroMat: pending
@sdondley https://github.com/sdondley: pending
@Korben00 https://github.com/Korben00: pending
@awwitecki https://github.com/awwitecki: pending
@feltcat https://github.com/feltcat: pending
@muellermartin https://github.com/muellermartin: pending
@marksmayo https://github.com/marksmayo: pending
@sashkab https://github.com/sashkab: pending
@HeLehm https://github.com/HeLehm: pending
@tomwojcik https://github.com/tomwojcik: pending
@Yomguithereal https://github.com/Yomguithereal: pending
@Ashikpaul https://github.com/Ashikpaul: pending
@Maddesea https://github.com/Maddesea: pending
@jaesivsm https://github.com/jaesivsm: pending
@dmoklaf https://github.com/dmoklaf: pending

—
Reply to this email directly, view it on GitHub
https://github.com/adbar/trafilatura/issues/512, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAKKUFKPHGQWWEXVXVXJKDYUX55PAVCNFSM6AAAAABDTD5ZRCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE2DMOBVGI2DKMI
.
You are receiving this because you were mentioned.Message ID:
@.***>

I agree

I agree

Agreed

From: Adrien Barbaresi @.>
Sent: Wednesday, February 21, 2024 8:10:35 AM
To: adbar/trafilatura @.>
Cc: Madden, Sean A @.>; Mention @.>
Subject: [adbar/trafilatura] Change of license? GPLv3+ → Apache 2.0 (Issue #512)
[This email originated from outside of OSU. Use caution with links and attachments.]
Dear all,
Trafilatura is now widely used and it has become apparent that the GPL license is not prevalent in Python packages, its potential implications are also not easily understood. To better align with community standards and promote ongoing development, I plan to transition Trafilatura's software license to Apache 2.0.
As you have contributed lines still currently in use it would be best to get your agreement in order to move forward on this. For the sake of simplicity I use the list provided by Github and you may be mentioned below even if your contributions focus on documentation or evaluation files.
For reference, other packages of mine concerned by similar changes:

adbar/courlan#80https://github.com/adbar/courlan/issues/80
adbar/htmldate#131https://github.com/adbar/htmldate/issues/131

If you agree please add a corresponding message to this thread. If not please contact me or open a public discussion.

@felipehertzerhttps://github.com/felipehertzer: pending
@vbarbaresihttps://github.com/vbarbaresi: pending
@knit-beehttps://github.com/knit-bee: pending
@LukasBBAWhttps://github.com/LukasBBAW: pending
@phongtnithttps://github.com/phongtnit: pending
@andremacolahttps://github.com/andremacola: pending
@idoshamunhttps://github.com/idoshamun: pending
@edkruegerhttps://github.com/edkrueger: pending
@EllieLockharthttps://github.com/EllieLockhart: pending
@tonyyangahttps://github.com/tonyyanga: pending
@immortal-autumnhttps://github.com/immortal-autumn: pending
@glacierckhttps://github.com/glacierck: pending
@deedy5https://github.com/deedy5: pending
@k-sareenhttps://github.com/k-sareen: pending
@dsgibbonshttps://github.com/dsgibbons: pending
@mrienstrahttps://github.com/mrienstra: pending
@LaundroMathttps://github.com/LaundroMat: pending
@sdondleyhttps://github.com/sdondley: pending
@Korben00https://github.com/Korben00: pending
@awwiteckihttps://github.com/awwitecki: pending
@feltcathttps://github.com/feltcat: pending
@muellermartinhttps://github.com/muellermartin: pending
@marksmayohttps://github.com/marksmayo: pending
@sashkabhttps://github.com/sashkab: pending
@HeLehmhttps://github.com/HeLehm: pending
@tomwojcikhttps://github.com/tomwojcik: pending
@Yomguitherealhttps://github.com/Yomguithereal: pending
@Ashikpaulhttps://github.com/Ashikpaul: pending
@Maddeseahttps://github.com/Maddesea: pending
@jaesivsmhttps://github.com/jaesivsm: pending
@dmoklafhttps://github.com/dmoklaf: pending

—
Reply to this email directly, view it on GitHubhttps://github.com/adbar/trafilatura/issues/512, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYN66G7EKJEMFBQL5MQQEHDYUX55XAVCNFSM6AAAAABDTD5ZRCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE2DMOBVGI2DKMI.
You are receiving this because you were mentioned.Message ID: @.***>

For the record: I waited about a month, all answers are in favor, I'm moving forward with the change.


Zero width and height when creating CircleIndicator programmatically.

Creating a CircleIndicator without an AttributeSet does not properly set the width, height, and margin of the indicators:
        CircleIndicator indicator = new CircleIndicator(getActivity());

Because handleTypedArray() only sets default values when mIndicatorWidth/mIndicatorHeight/mIndicatorMargin < 0 but they're not set in any constructor.

I guess the same goes for all the other properties but I never got so far as to see the indicators on-screen.

Sorry, I forgot this case…
I will fix it soon.


Unresponsive Swirl Logo in Login Page

On login page:

When I inspect
Zoom in/out
Resize window
Swirl logo gets disappeared

OS - Windows 10
Browser - Chrome
screen-capture (6).webm

Please assign me, I'll fix it up

PLease assign me

Thanks, @s-vamshi, for pointing this out. The team will fix it up in the coming versions.

In Release 3.0, out now, the Galaxy UI login page now handles window resizing more gracefully.
Thank you for letting us know!  Closing this Issue.


Rename QuestionAnswering template to QuestionAnsweringExtractive

Following the discussion with @thomwolf  in #2255, this PR renames the QA template to distinguish extractive vs abstractive QA. The abstractive template will be added in a future PR.

I like having "extractive" in the name to make things explicit. However this creates an inconsistency with transformers.
See
https://huggingface.co/transformers/task_summary.html#extractive-question-answering
But this is minor IMO and I'm ok with this renaming

yes i chose this convention because it allows us to match the QuestionAnsweringXxx naming and i think it's better to have task_name-subtask_name should auto-complete ever become part of the Hub :)


Vickel: Watch merchant-accounts.ca

Vickel requests the watch of the watch_keyword merchant-accounts\.ca. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Spevacus in Charcoal HQ


NO-JIRA: Relax cel 4.16

What this PR does / why we need it:
this breaks the cost budget for < 4.17. We should figure why and enable it back. If not fixable, we should consider imposing a minimum version on the management cluster or an opt-out solution.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/lgtm

/jira refresh


hotkey

Hi,
Can you add an option to assign a keyboard shortcut to the add-on(for example CTRL+ALT+V will open all URLs in the clipboard)?

Short answer: I will consider implementing it.
Long answer: Setting a shortcut key statically is technically very easy. However, I think that it is better for users to be able to customize the shortcut keys themselves. It seems that it became possible with Firefox 60(https://bugzilla.mozilla.org/show_bug.cgi?id=1421811). However, it takes a while because I never implemented setting screen of WebExtensions.

This is status update! I investigated the option page of the add-on and setting preservation. And I am asking the community about what I can not understand.

@nirilan I've released a new version. please check it!


Allow unregistration of 0th function in functionList

The first (0th) item cannot be unregistered.
Click the body and see that the event fires even after deregistering: https://jsfiddle.net/dLdubjnm/

@michaelcpuckett thanks for catching this and submitting a fix!


Allow override a Pulumi token by using SetToken of infer.Annotator

Closed as part of https://github.com/pulumi/pulumi-go-provider/pull/129.


releasenotes: updated documentation changes

Included additional documentation changes for this release
Signed-off-by: David B. Kinder<EMAIL_ADDRESS>

Codecov Report

Merging #8186 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8186      +/-   ##
==========================================
- Coverage   64.53%   64.53%   -0.01%
==========================================
  Files         420      420
  Lines       40101    40101
  Branches     6762     6762
==========================================
- Hits        25881    25879       -2
- Misses      11103    11104       +1
- Partials     3117     3118       +1

Impacted Files
Coverage Δ

boards/posix/native_posix/timer_model.c
98.21% <0%> (-1.79%)
:arrow_down:

samples/philosophers/src/main.c
96.87% <0%> (-1.57%)
:arrow_down:

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 265f502...b3d7737. Read the comment docs.


question about http reverse proxy usecase

Hi,
i have a question is envoy able to mirror the traffic for the following usecase?
For each incoming request, it clones the request into 2 requests, forwards them to 2 servers.
The results from server A are returned as usual, but the results from server B (qa_server) are ignored.
teeproxy handles GET, POST, and all other http methods.
cc: https://github.com/chrislusf/teeproxy
is thats already possible could you show me the configuration i need to set and how to run it?

Yes, see https://www.envoyproxy.io/docs/envoy/latest/api-v2/api/v2/route/route.proto.html#route-routeaction-requestmirrorpolicy

hi any response? :)


RELEASE.CHANGELOG.md is missing version 3.0.0

Describe the issue
It seems that when version 3.0.0 was released, @normj updated the RELEASE.CHANGELOG.md by copy pasting the header of the previous 2.1.0 release and changing the contents. However, they forgot to change the version number of the release so now there are two 2.1.0 releases in the RELEASE.CHANGELOG.md while the latest should be 3.0.0
Should be a simple fix and helps with confusion for people looking to upgrade to 3.0.0 and wanting to know what has changed. I, for one, had trouble finding the breaking changes :sweat_smile:
Links
https://github.com/aws/aws-ssm-data-protection-provider-for-aspnet/blob/master/RELEASE.CHANGELOG.md

Appears to be a simple change after looking at the RELEASE.CHANGELOG.md.

Fixed in latest commit.


Movement with modulus

I'm trying to have multiple ellipses moving in opposite directions, bound within the frame of the canvas.  I cannot figure out why the second ellipse only moves from the right side to the left once, and then keeps flashing up in the left corner..

Never mind, figured out how to get the translation right!


Update persistent compilation cache doc

This is a follow-up of the PR https://github.com/google/jax/pull/21819.

LGTM, but I recommend asking Skye to do a pass as well.

I want to merge this first and request Syke to review a follow-up PR about LRU cache eviction.


Ack the messages

Needs to be indented with spaces, but yeah, that's correct.
We had this at one point, how did it not make it into this branch before?

Fixed.


window size & states not working properly on windows 11

when frameless title bar is on, the window position in config moves around 30 pixels up every time i close it, and even though the config says the last state was maximize, it never run on maximize, instead it just open on normal, with the title bar outside of the screen because of the drift so i cant even reach the title bar. when i turn the frameless title bar off, the window always start on maximize, no matter what. i have no idea what is going on

aha! it's the custom title bar!! the showNormal() is kicking them from maximize and fullscreen, and for me who only uses the title bar, i think the workaround might just be having some conditionals to that, if the config says maximized, then showMaximized instead of showNormal, imma go try that

for the shift problem, when the window is not close to the top of the screen, it shifts consistently 38 pixels up and 9 pixels to the left, the workaround for me maybe to just manually correct that values when saving the geometry to the config

also figured out why window always start on maximize when the frameless is off, the load_config just says if the config for that passes as a boolean, showMaximized. i think there's supposed to be a maxim.toBool() in there

oh my god okay, removing the showMaximized altogether fixes everything, this is because showMaximized was loaded before the title bar (which reloaded the geometry), reload geometry after show will results in shift up left to the size of the non frameless title bar. those shifting up could be a workaround for x11's window shift issue, but in windows it just caused a shifting instead of correction, more about that here: https://doc.qt.io/archives/4.3/geometry.html

but removing showMaximized results in nvui not launching in maximized even if the config says it should HAHA

how to fix, working workaround for me: modify windows.cpp as follows

on load_config, instead of showMaximized(), use maximized = (maxim.toBool())
on enable_frameless_window, instead of showNormal();, use if (maximized) showMaximized(); else showNormal();
maximized is already a boolean available in the header files so why not just use that
on save_state, instead of geometry(), use normalGeometry() so it saves size and position when not maximized

bonus, add save_state() on changeEvent so it saves not only when you close
disclaimer: i don't use fullscreen and only use frameless window so that's all i need to modify, also i don't know programming at all this is all trial by fire if someone experiencing the same issue and have a better solution i would appreciate the feedback


Arbled characters when using snapshot_download and hf_hub_download to download files with cn/jp/kr names

Describe the bug
When use snapshot_download or hf_hub_download to download files with chinese (both simpifiled and traditional), japanese or korean names, the tqdm will show arbled characters.
Run the following code
from huggingface_hub import HfApi

hf_client = HfApi()
hf_client.snapshot_download(
    repo_id='narugo/test_rp',
    repo_type='dataset',
    local_dir='test_langs',
    local_dir_use_symlinks=False,
)

print('这是简体中文，输出是正常的')
print('This is simplified chinese, nothing wrong with the environment')

The result is

The chinese text in print statement will be printed normally, while text in snapshot_download will not.
The multi-lang repository for test we are using: https://huggingface.co/datasets/narugo/test_rp
Reproduction
from huggingface_hub import HfApi
hf_client = HfApi()
hf_client.snapshot_download(
repo_id='narugo/test_rp',
repo_type='dataset',
local_dir='test_langs',
local_dir_use_symlinks=False,
)
print('这是简体中文，输出是正常的')
print('This is simplified chinese, nothing wrong with the environment')
Logs
No response
System info
Copy-and-paste the text below in your GitHub issue.

- huggingface_hub version: 0.20.1
- Platform: Linux-5.10.198-187.748.amzn2.x86_64-x86_64-with-glibc2.2.5
- Python version: 3.8.1
- Running in iPython ?: No
- Running in notebook ?: No
- Running in Google Colab ?: No
- Token path ?: /home/user/.cache/huggingface/token
- Has saved token ?: True
- Who am I ?: narugo
- Configured git credential helpers:
- FastAI: N/A
- Tensorflow: N/A
- Torch: 2.1.0
- Jinja2: 3.1.2
- Graphviz: N/A
- Pydot: N/A
- Pillow: 9.5.0
- hf_transfer: N/A
- gradio: N/A
- tensorboard: N/A
- numpy: 1.24.4
- pydantic: N/A
- aiohttp: 3.8.6
- ENDPOINT: https://huggingface.co
- HF_HUB_CACHE: /home/user/.cache/huggingface/hub
- HF_ASSETS_CACHE: /home/user/.cache/huggingface/assets
- HF_TOKEN_PATH: /home/user/.cache/huggingface/token
- HF_HUB_OFFLINE: False
- HF_HUB_DISABLE_TELEMETRY: False
- HF_HUB_DISABLE_PROGRESS_BARS: None
- HF_HUB_DISABLE_SYMLINKS_WARNING: False
- HF_HUB_DISABLE_EXPERIMENTAL_WARNING: False
- HF_HUB_DISABLE_IMPLICIT_TOKEN: False
- HF_HUB_ENABLE_HF_TRANSFER: False
- HF_HUB_ETAG_TIMEOUT: 10
- HF_HUB_DOWNLOAD_TIMEOUT: 10

Hi @narugo1992, thanks for reporting. I've investigated a bit but not sure we can do much about it. The problem comes from the header returned by cloudfront in which the filename is encoded. Here is a standalone script to reproduce it:
import requests
from huggingface_hub import hf_hub_url

url = hf_hub_url(repo_id="narugo/test_rp", filename="日本語/でこんにちは.jpg", repo_type="dataset")
response = requests.head(url=url, allow_redirects=True) # 1 redirect to cloudfront CDN
print(response.headers.get("content-disposition"))

which outputs
inline; filename*=UTF-8''%E3%81%A7%E3%81%93%E3%82%93%E3%81%AB%E3%81%A1%E3%81%AF.jpg; filename="ã§ããã«ã¡ã¯.jpg";

Filename is "badly" decoded by requests and since we're using this value in the tqdm line, I don't think we can do much :confused:

FIY, for the url of huggingface site resources, maybe extracting the file name from the url and using it for the desc parameter of tqdm is a solution.
e.g. https://huggingface.co/datasets/narugo/test_rp/resolve/main/日本語/でこ %E3%82%93%E3%81%AB%E3%81%A1%E3%81%AF.jpg, just extract the file name through urlsplit, and obtain the decoded file name through unquote.

Thanks for your input @narugo1992. I just opened a PR to fix this (see https://github.com/huggingface/huggingface_hub/pull/1954). I went for the simplest solution which is to provide the filename explicitly -no encoding/decoding involved-. Relying on the URL directly was not possible as it wouldn't work for files hosted on a CDN (that's why we have read the content-disposition header which is error-prone).


update packages

:white_check_mark: Build TheIdServer 4.3.2-fix-dependencies1-0008 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/278b25b963 by @)


Watch-admin crash browser

Describe the bug
After i run watch-admin, when i try switch element in template (Shopping Experience) the browser are hangs
To Reproduce
Steps to reproduce the behavior:

Create new plugin with shopping experience element
Run make watch-admin
In browser go to localhost:8888/<IP_ADDRESS>:8888
Go to Shopping Experience -> edit template -> switch element
Browser crash

Desktop (please complete the following information):

OS: macOS Catalina 10.15.7 ( i tried it on Windows 10 and works fine)
Browser: chrome and safari
Version: lates

Hi
thank you for posting this
i'm not quite sure if this is still a thing, I just tried it and it worked for me
also, I'm not quite sure if this is either a shopware related thing, or just something on your machine
i just tried it with dockware/dev:<IP_ADDRESS>
does this work for you?
if not maybe you can provide a small video?

i havent heard back since June,
and it shuld actually work so i would now close this issue for now


{"error":"unauthorized_client"}

since yesterday the adapter do not start anymore.
The error in the log is:
"error":"unauthorized_client"
Login failed
AxiosError: Request failed with status code 400

any ideas?
add; I have added the line number in the error and it comes up with line 168, so it look like an issue with the authorization. What is with a redirect url?

Login flow needs to adopted
Only working implement
https://github.com/Dielee/volvo2mqtt

Login flow needs to adopted
Only working implement
@TA2k
I'm not sure, how I can handle this, I don't have a docker installation. You can adopt the login flow?

Same problem with the login. Any new rules at the Volvo server?

any news here? Can someone help to solve the login issue also in this adapter?


MEDIUM: watcher: fix race condition & plumbing stop for test

this fix is related to: https://github.com/haproxytech/haproxy-consul-connect/issues/10

This will not stop the goroutines from running once the Watcher is stopped.
One way to do that is to add a check in the infinite loops to stop them when the Watcher is stopped:
for {
  select {
    case <-w.shutdownCh:
      return
    default:
  }

  // work
}

Additionally, I'd prefer exposing a Stop() method rather than the chan directly.

I applied your recommendations.

Please recheck and give a thumbs up for mergeing.


Add < lenaarenot > as contributor

I create a tutorial as my bachelor project and so I need to be added to this file

Hi @lenaarenot! Could you also add your name to the tutorial you contributed to? Then you will get credit on the tutorial page as well.
(Let me know if you need help finding the right file)

ah, are you referring to https://github.com/galaxyproject/training-material/pull/4968? In that case, the easiest way is to add this change to that PR, then it will all get merged together once it is ready :)


False positive in US when cities =T

Hi. With cities=T, some texts with short sentences tend to favor United States:
loc_data <- data.frame(location=c("I live here, in Honduras", "Paris"), ID=c("a1", "a2")); detect_geographies(data=loc_data,  id=ID, input=location, output="country", cities =T)
returns "United States|Honduras" for "I live here, in Honduras"
Same result with the shorter text below (abridged to try and understand the error):
loc_data <- data.frame(location=c(", in Honduras", "Paris"),                        ID=c("a1", "a2") ); detect_geographies(data=loc_data,                    id=ID, input=location, output="country", cities =T)
Result is OK with cities=F

@brandonleekramer Would you have hints on how to fix this incorrect country detection?


RFP - cmocka

Checklist

[x] The package I am requesting does not already exist on https://community.chocolatey.org/packages;
[x] There is no open issue for this package;
[x] The issue title starts with 'RFP - ';
[ ] The download URL is public and not locked behind a paywall / login;

Package Details
Software project URL :
Direct download URL for the software / installer :
Software summary / short description:

Not sure on the URL or the format required to submit.
Do you require binaries?

You haven't ticked that the software URL is public? If the software is not public then a package can't be created.

The software is public. I just don't know if I am supposed to reach out to the cmocka team to request they create a binary.

How do you install the software if there is no binary? What is the location of the software?

I have reached out to cmocka to see if they are willing to provide a binary url.
Thanks for your help.

I'm going to go ahead and close this. Once you have everything required we can either reopen or create a new issue.


Removed MS apps

Done?

You can see the status very clearly and if there are errors on the travis-ci website. For now, it's building:
https://travis-ci.org/SwiftKitz/Appz

OK

Done 😭 Sorry for bothering you.

:8ball: score!

😊

It is probably just too late in Kuwait, make sure you take a break as well!

Ok thanks 😊


.molang files are missing auto completion and are having color highlighting issues.

Describe the bug
.molang files are missing any form of auto completion, color highlighting is present but is having issues as well.
To Reproduce

Create a new .molang file.
Open the file.
Try to type start of a common function like math.
There is no auto completion and color highlighting is messed up and malfunctioning.

Expected behavior
When trying to type a name of a common function like Molang queries/math functions and more, there should be an auto completion.
Screenshots
Expected auto completion behavior, working within feature json file:

Actual auto completion behavior within molang file and messed up color highlighting:

Coming to version 2.1.55


Aranovskiy filter alternative

Consider another frequency tracking filter
such as KalmANF
https://github.com/randyaliased/KalmANF
Based on
R. Ali, T. van Waterschoot, "A frequency tracker based on a Kalman filter update of a single parameter adaptive notch filter", accepted for Proceedings of the 26th International Conference on Digital Audio Effects (DAFx23), Copenhagen, Denmark, September 2023.

Somehow that implementation of KalmANF filter doesn't converge for me in presence of gaussian noise even though it's model assumes that the noise is present.
See:
https://github.com/randyaliased/KalmANF/issues/2

There are many other options
pitch tracking algorithms like YIN, autocorrelation
Algorithm need to be able to tolerate noise and presence of multiple harmonics.
Some links:
https://github.com/alesgenova/pitch-detection
Todo: more links


Fixed Arabic align problem

The two buttons are not align for Arabic. It looks a little bit weird with different screen size.

Thanks for the fix! LGTM. Merged.

fixed #197


Add Embed Mode

I have implemented an embed mode which automatically engages when the web app detects to be inside an iframe.
This is a slightly different approach than mentioned in #21, because I think this is more responsive and convenient for the users to just use the same link for their embeds as they see in their browsers.
This is how the result looks like.

[This merge request fixes #21.]

e m b e d e d


common eventbus

rxjs - https://rxjs.dev/guide/overview
mitt -  https://github.com/developit/mitt
ionic - https://ionicframework.com/docs/core-concepts/fundamentals#events
vue - DOM events https://vuejs.org/guide/essentials/event-handling.html

CustomEvent extends DOM Event,

https://vueuse.org/shared/createEventHook/


Enforce PodSecurityStandards since psp disappeared in 1.25

Pod Security Policies were removed in kubernetes 1.25 and the default chart fails to install on a modern cluster running 1.29.2
Checklist

[ ] Update changelog in CHANGELOG.md.
[x] Make sure values.yaml and values.schema.json are valid.
[ ] (Giant Swarm) If creating a release, bump the version and appVersion in Chart.yaml.

Hey @ebCrypto, thanks for the PR!
I totally agree we should move to disabling PSPs by default.
We still have some users in the process of migrating away from PSPs and for whom it would be a breaking change to invert the logic here without a major version release, so we have to hold off just a bit longer.
Feel free to leave this PR open if you like, and I'll merge it when we're past any potential breakage. In the meantime, PSPs can of course be toggled off through the Helm values.

Well that took a lot longer than I hoped on our side :sweat_smile: thanks!


Build in an opt-in auto run feature

As @Evertt pointed out in #5 an auto-run feature would be nice, but only opt-in. This way you don't accidentally run a delete query without where statements.

Catually, not going to do this. I think running backend code should be done on actual intent and adding this feature just will do harm somewhere per accident.


Remove MBEDTLS_CHECK_PARAMS option

Description
Remove MBEDTLS_CHECK_PARAMS option
Work items for 3.0

Remove MBEDTLS_CHECK_PARAMS, MBEDTLS_CHECK_PARAMS_ASSERT and MBEDTLS_PARAM_FAILED from config.h.
Make it a compile-time error to enable MBEDTLS_CHECK_PARAMS.
Remove component_test_check_params* from all.sh.
A minimum amount of changes so that the CI passes.
Tweak the API for RSA context initialization to have better handling for an invalid padding mode - separate issue #4523

Fixes #4313
Status
READY
Requires Backporting
NO
Migrations
NO
Todos

[x] Tests
[x] Documentation
[x] Changelog updated

I do not think that starting with a merge commit is a good habit. At least that's not the way we usually do it. We rather fetch locally the latest development and base our work on it.

Should I re-create this PR?

You don't have to. You can just re-base your two commits on top of development in your "Remove__CHECK_PARAMS_option" local branch and then force-push it: "git push  --force-with-leave Remove__CHECK_PARAMS_option".
Otherwise, if this is more convenient for you, you can just close this one and create another one. This hasn't been reviewed yet thus this is not a problem.

two places in PSA where Ronald has pointed out that we need to find a replacement.

These are test assertions, not something we rely on for security, so it's ok to file them as a high-priority 3.x follow-up, not a blocker for 3.0.

There are two conflicts with development thus please add a last merge commit with development as you have done before, with a commit message explaining the resolution of the conflicts.

@mpg are you ok with the last version ?


Fish shell is supported?

Hello, your project is very nice. However it does not work in the Fish shell. Please add support for Fish shell.

No, please read the description before creating an issue. Thanks.


[Feature] Track total time spent

I'm using:

Device: Precision 5520
OS: Windows 10 22H2
Browser: Microsoft Edge

I want to suggest this feature:
A way to track how much time you've spent playing a game on XCloud.

There is already a playtime badge in the stream menu (three dot menu):

I think it will reset everytime you start a game, so it shows your session time with that game.

I'm planning to make a similar feature to this


Rainbow Heart

❤

This is lovely and I'm happy for it to remain here! But it might be better placed on the cardioid page? https://thecodingtrain.com/CodingChallenges/133-times-tables-cardioid.html


Ring buffer refactor

@yatli refactord the MIDI ring buffer code to use membank1.
We now have 23371Bytes of RAM available.
You might have a better solution to modifying the template to use ptr, instead of creating an array of size N (buf[n])
My implementation is a bit of a hack. I create ring buffers of size 0. then assign pointer and length in the MidiUart constructors.

Looks like there's a problem with buffer sizes.
I miscalculated. earlier the original buffers were of type int16_t
so double the size.

I'm thinking about this. Now we have quite a few if (ptr == NULL) { .. do with unknown ptr .. } else { .. do with bank1 } branches. These can be unified, and, their behavior can be statically deduced because we assign them at compile time anyway.

This pull request is still not stable. Getting crash after some time of use.

@jmamma anything using a ringbuf where sizeof(C) != 1?

@jmamma anything using a ringbuf where sizeof(C) != 1?

Yes.
See : GUI/Events.hh

I'll get back my free time after 24th (and recover from the burnout).
If time permits, I'll take this one?

I merged master, and now the compiler reports:
Sketch uses 20796 bytes (8%) of program storage space. Maximum is 253952 bytes.
Global variables use 3201 bytes (5%) of dynamic memory, leaving 53630 bytes for local variables.

I've downloaded this branch to MC and played for a few minutes.
Looks like there's still some problem: at a time point the connection seems to be broken. after that, MC responses slowly, and chain operations fail. It cannot download patterns from MD, not even switching to channel 16 to leverage the TI.

if it works fine on master (I haven't tried yet) then it must be related to the changes in this PR. I'll investigate this tonight.

Yes, this PR is unstable.
Need to investigate what the cause is. Perhaps the buffers are unexpectedly filling up, and causing weird behaviour, You should be able to check this by adding some if statements in get and put.
The other potential cause is an issue with bank switching, or some type of memory overflow ?

great. I'll go over the code more carefully. there're also clues left by the 1 month younger me.

It appears txRb.isFull()) condition is being triggered?
216 void MidiUartClass::m_putc(uint8_t c) {
217   //#ifdef BLAH
218   if (c == 0xF0) {
219     uart_block = 1;
220   }
221   if (c == 0xF7) {
222     uart_block = 0;
223   }
224 again:
225
226   if (txRb.isFull()) {
227     while (txRb.isFull()) {
228       if (IN_IRQ()) {
229         // if we are in an irq, we need to do the sending ourselves as the TX
230         // irq is blocked
231         setLed2();

trying this led trick

@jmamma which one is Led2, on the left?

think i found the problem...

led2 is right

in MidiUart.h
 75
 76   volatile RingBuffer<0, RX_BUF_TYPE> rxRb;
 77
 78 #ifdef TX_IRQ
 79   volatile RingBuffer<0, TX_BUF_TYPE> txRb;
 80 #endif

I'm setting N = 0, to indicate that we're using membank2.
However the ringbuffer code uses N, in stead of len:
For example:
@@ -129,7 +129,7 @@ template <class C, int N, class T>
   if (ptr == NULL) { ret = buf[rd]; }
   else { ret = get_byte_bank1(ptr + rd); }
   rd++;
-  if (rd == N) { rd = 0; }
+  if (rd == len) { rd = 0; }
   return ret;
 }

still crashing.

I'm still trying to figure out what I meant with the comments left. Trying to sync up :)

Did you clean before make ?

I'm testing new patch. Crashed at first, but I realised RingBuffer.h was not recompiled.

The behavior differs from the original.
ptr is volatile uint8_t*, but buf is C[N].
&buf[wr] is buf + sizeof(C) * wr.

ok I got what I meant. Fixing this one.

whoa, there was a copy/paste error -- should be rd but we put wr there

Ha, I was looking at that as well.

whoa, there was a copy/paste error -- should be rd but we put wr there

where?

@@ -141,7 +141,7 @@ template <class C, int N, class T>
   memcpy(dst, (void *)&buf[rd], sizeof(C));
   }
   else {
-  memcpy_bank1(dst, ptr + wr, sizeof(C));
+  memcpy_bank1(dst, (void*)&static_cast<volatile C*>(ptr)[rd], sizeof(C));
   }

git blame me.

in general I think we should use volatile T* ptr, not uint8_t

in general I think we should use volatile T* ptr, not uint8_t

That's fine by me.
MidiUart.cpp will need to modified.
 12 MidiUartClass MidiUart((volatile uint8_t *)BANK1_UART1_RX_BUFFER_START,UART1_RX_BUFFER_LEN,(volatile uint8_t *)BANK1_UART1_TX_BUFFER_START,UART1_TX_BUFFER_LEN);
 13 MidiUartClass2 MidiUart2((volatile uint8_t *)BANK1_UART2_RX_BUFFER_START,UART2_RX_BUFFER_LEN,(volatile uint8_t *)BANK1_UART2_TX_BUFFER_START,UART2_TX_BUFFER_LEN);

Replace  (uint8_t *) with RX_BUF_TYPE/TX_BUF_TYPE
I should think

I have to head off for a few hours.
My limited testing is showing the stability as resolved.

If they were using uint8_t then no need to change. I'm wrapping up compatible code.

have a nice one. I'll have some pressure test tonight.


Reduce memory leak

Run project in command line godot --verbose -e for more information.
Use singleton instead of static class

Closing this in favor of #10


Odd code?

I'm not sure this condition for the else will ever be met. Wont newRegistrationId  ALWAYS be null? Nothing else is changing it.
if (newRegistrationId == null)
{
newRegistrationId = registration.RegistrationId;
}
else
{
await hub.DeleteRegistrationAsync(registration);
}

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

ID: e474fcd1-4479-be1b-8711-d709a16ecca1
Version Independent ID: 6d8e4db8-89e1-0dbc-a25c-877c0865bf48
Content: Azure Notification Hubs Notify Users with .NET backend
Content Source: articles/notification-hubs/notification-hubs-aspnet-backend-windows-dotnet-wns-notification.md
Service: notification-hubs
GitHub Login: @ysxu
Microsoft Alias: yuaxu

@nezoic Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@BryanTrach-MSFT please close


Installed extensions discovered once only after vscode reinstall

Type: Bug
I recently upgraded my pc and copied my C:\Users\USERNAME\.vscode folder across to my new drive because it has all my extensions and settings.
I freshly installed vscode and when it opened up all my extensions were there, it all just worked. Then once I restarted vscode all these exensions disappeared. The extensions explorer reports 0 installed extensions.
The only way to bring them back is to re-install vscode, and then it also will only load them the very first time after install. I have done this a few times now and am wondering what I can do differently.
VS Code version: Code 1.74.2 (e8a3071ea4344d9d48ef8a4df2c097372b0c5161, 2022-12-20T10:29:14.590Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: No

I've kind of fixed it, didn't find any satisfying solution.
I had to move my .vscode folder to some new location, install my extensions from the vscode extension explorer page and then I was able to switch out the metadata folder manually for my backup copy. I did this for some of my more important extensions and was able to recover my configurations.
My guess is there was some extension among my 20-something that was causing this error, but I don't really know which one it was.


Make Color of Embed Optional

Is your feature request related to a problem? Please describe.
I want to test the Bot if it returns the right message. but if dont set the color it fails
Describe the solution you'd like
Make color optional
Describe alternatives you've considered
RANDOM doesnt work unfortunately
Additional context
https://github.com/cordejs/corde/discussions/721

Hi @kaaax0815 just to be aware, I'm working on a test function to solve your problem. Still this week I will release a version with this function for you

Hi, @kaaax0815. I just released version 4.1.0. It contains a function called toEmbedMatch. This function checks partially a embed message, So if you do not pass the property color, your tests will woks.

@lucasgmagalhaes thanks I'll check it out asap


Fix building on RedHat Enterprise Linux

Fix #7485
PR Summary
When trying to build on RHEL, the environment is not recognized as RedHatFamily.
Added detection logic so that appropriate symbolic links are generated.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[x] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[x] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

@TravisEz13 Added another property called IsRedHat to future proof the logic.
@dantraMSFT  Added a throw when none of the environments are set.

@TravisEz13  Ready to merge?


uninitialized constant Rake (NameError) in lib/rake/hooks.rb:1

rake-hooks-1.2.3/lib/rake/hooks.rb:1:in `<top (required)>': uninitialized constant Rake (NameError)
        from ~/.rvm/gems/ruby-1.9.3-p125@global/gems/bundler-1.1.3/lib/bundler/runtime.rb:74:in `require'

@cheef +1
Another fix is to add require 'rake' just above Bundler.require(:default, Rails.env) in application.rb, but I like your fix better.


Unit Test -- Core-Metadata: Get Command by Id

Related to: https://github.com/edgexfoundry/edgex-go/projects/11#card-23264363

Could you please assign it to me


Quick Open leaks memory on each invocation

Add a console.log() in the "focusin click" listener in jquery.smart_autocomplete.js
Open and immediately close Quick Open 5 times
Click in the editor area

Result: 5 console logs per click
Expected: 0, or at most 1, log per click
I think this is a pretty small leak (though it's impossible to quantify because the "heap snapshot" developer tools feature appears to be broken in CEF 3).  But looking at what's actually referenced in the closure, I don't think a whole lot is actually retained.

Fixed in 1.3


Replace apache commons Base64 w/ JDK

Why are the changes needed?

Minor code clean-up, prefer to use the JDK-provided tools.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Thanks, merging to master


TPC-C gets super slow when customer index is enabled

When we enable the customer index, TPC-C throughput drops dramatically and become not scalable at all.

@jessesleeping In the email you said you tried to allocate the key tuple’s varlen in the executor context varlen pool, instead of the index’s one. But the number is still really bad. I don't quite understand why the number is still really bad in this case. There's no contention here, since one executor context can only be used by a single execution thread at one time, right?

When using executor context's varlen pool, the number is bad but scalable. The problem is varlen pool itself is much slower than the plain malloc.

Jiexi -- If we use a latch free per-connection pool, would the performance
improve?
Or do you think that a new implementation is needed? If so, what are the
key differences in its design with respect to the current one?
On Dec 27, 2016 5:57 AM, "Jiexi Lin"<EMAIL_ADDRESS>wrote:
When using executor context's varlen pool, the number is bad but scalable.
The problem is varlen pool itself is much slower than the plain malloc.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cmu-db/peloton/issues/426#issuecomment-269251486, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFRCN8q0eZ2woexgSNDzKEKwjmSWRM2iks5rMFtUgaJpZM4LUsw_
.

I think per connection varlen pool or per thread varlen pool can help improve our performance in other cases (e.g. allocating varlen for inserted titles), but not the problem stated here.
A lock free implementation may help when background gc is freeing memory in varlen pool while the worker thread is allocating memory in its local pool.
Small things we can do is to optimize the usage of std::list and bitmap in current implementation. I think @MattPerron and I have the feeling that they can be more efficient. But I think what we really need to do is to re-consider what we want from the varlen pool and how can we distinguish it from general purpose malloc implementation?

"When using executor context's varlen pool, the number is bad but scalable. The problem is varlen pool itself is much slower than the plain malloc."
Then we should just use pure malloc in these cases.
"But I think what we really need to do is to re-consider what we want from the varlen pool and how can we distinguish it from general purpose malloc implementation?"
If we are going to re-implement Varlen Pool we should do it right from the ground up and fix all the problems we have found so far.
Maybe I'm missing something but In my understanding the only thing Varlen pool gives us is the ability to separate memory that we allocate for temporary use (in the executor context) and memory that we allocate for the table itself. I'm not sure if there is a performance benefit to doing this, but presumably it would help the hardware prefetcher in cases like a sequential scan. If our implementation is always slower than JEMalloc, we should just be using malloc all the time.
If we choose to keep the varlen pool, and not simply use jemalloc. I think it makes the most sense for the Table (not the tile or tilegroup) to have a pool per core. We would then remove all the synchronization from varlen pool (note that it would require that worker threads be pinned to cores). varlen pool also should be profiled in isolation to achieve the best performance. I suggest having a pools attached to the table because it then allows us to use @jessesleeping 's reference count optimization when copying tuples without worrying about potential future garbage collection issues when tilegroups are cleaned up.
As @jessesleeping suggested, we seem to be re-implementing malloc, maybe there is a way to implement varlen pool by taking advantage of JEMalloc's non-standard API to specify the "Arena" to allocate memory, and leaving the details of implementation to JEMalloc. An argument against this is if we change to another allocator in the future, we would have to reimplement the varlen pool. It may also be worth it to consider NUMA-awareness. JEMalloc's lack of NUMA-awareness is challenging to circumvent.
What are everyone's thoughts about this?

This probably deserves a separate issue.

Fixed with #461 + #456


Added new example following the "Clean Architecture"

Hi Guys!
When building large-scale applications one of the big problems is understanding how to design the code. I have created this really simple example of "Clean Architecture" implementation.
I hope it could be useful to be merged into the master.
This is my first PR so please let me know if I need to add something.

I'm having a lot of problems with ESLint, seems that I cannot use classes in this version, could someone help me, please? :)

Currently, AppVeyor is failing up to node 3.3 in one test but in the other versions the same test works correctly (and also in my local env). Does someone know how to fix it?

Hi @lokk3d sorry about that! It looks like it also happns on those versions in Github Actions as well, so seems like there is something incompatible with Node.js < 4 in either your example or tests. Have you tried it locally with one of the failing versions to see? Alternatively you're welcome to contribute against Express 5 instead of 4, which doesn't support those Node.js versions.

Hi, @dougwilson thanks for your comment! Probably in this case contributing to version 5 is the best choice. I have just to close this PR and open the new PR in branch 5.0, right? (or is more correct in 5.x?)
Thanks a lot for your help :)

No problem. The base of the PR can always be changed by you or us to make it point to 5, but I see you already closed this and opened a new one instead.


Generic groups and enemies

Possible implementation
const enemies: Phaser.GameObjects.Group<Enemy> = [];
enemies = this.add.group<Enemy>({
  classType: Enemy,
  runChildUpdate: true,
});

and then with iterator:
enemies.children.iterate((enemy)=>{
  // enemy is of type GameObject<Enemy>
});

Make sure that type is optional
group<T=any>(children?: Phaser.GameObjects.GameObject<T>[] | Phaser.Types....

and game object
class GameObject<T=any> extends Phaser.Events.EventEmitter {

The recommendation was discovered here:
Currently with 3.60.beta10 I get this issue:

It is solved, if I return back boolean, but boolean here makes no sense to me:

iterate expects the callback to provide a boolean. If it returns false the Set iteration is aborted immediately and no further children are processed. If it returns true it carries on.


Help

Hi, you could contact me I have something to ask you in private, that doesn't concern hosts.extras
mi addres<EMAIL_ADDRESS>

Asking here.


Unable to install plugins

macOS version: 10.14.6
Kap version: 3.1.0
Steps to reproduce

Launch Kap
Open prefs (cmd-,)
Click "Plugins" tab
Scroll down
Starting at "do-not-disturb", the toggle for it and all following plugins is disabled

Current behaviour
The toggle to install certain plugins is disabled, looks like:

Expected behaviour
Be able to click the toggle and install the plugin.
Workaround
I attempted to work around by navigating to ~/Library/Application Support/Kap/plugins and adding the plugin to the dependencies in package.json (and installing), but even after that the plugin does not seem to show up in the "Plugins -> Installed" tab in Kap preferences.

Btw, I love Kap! Thanks for creating it.

Thank you @bantic! Please see https://github.com/wulkano/kap/issues/811#issuecomment-600258113 for more information.

Thank you! Apologies for not having found that duplicate issue. I'll give the test build a shot!


Add 'openssl req' option to specify extension values on command line

The idea is to be able to add extension value lines directly on the
command line instead of through the config file, for example:
openssl req -new -extension 'subjectAltName = DNS:dom.ain, DNS:oth.er' \
                 -extension 'certificatePolicies = <IP_ADDRESS>'

Fixes #3311
Thank you Jacob Hoffman-Andrews for the inspiration

This is an alternative to #4971

Commit amended with a thank you note

Also took this out of WIP

A couple of usability thoughts:

The help text should make it more clear what the difference between -extension and -extensions is. A naive guess would be that one simply takes multiple arguments, while the other takes a single one, but they're actually quite different: One takes a config section name and one takes a key/value.

The man page should include an example usage, ideally using a very common extension like subjectAltName.

I pushed an extra commit that amends the documentation (both manual and -help).  Does that look better?

The help text should make it more clear what the difference between -extension and -extensions is.

Maybe you could use -ext instead of -extension to increase the Hamming distance to the -extensions option?

Maybe you could use -ext instead of -extension to increase the Hamming distance to the -extensions option?

Or -raw_ext, or -verbatim_ext, or -inline_ext, ... Otherwise I'd too say that -ext would do better than -extension...

-ext is a good idea, except it's already taken in openssl x509, and I'd rather not have that confusion...

-rawext?

I feared but wasn't sure that -ext was already in use. How about -setext? But in the end, it's up to you.

-setext is not bad, but since this option actually adds extensions, I think I'll finally go for -addext
(it's made so it combines with -extensions, if someone has a config file with basic extensions they like, and then just want to add an extra SAN or something...)

Just drop the second half of the sentence.

Well, yeahok, that's also an option...

Squashed and merged.  bfa470a4f64313651a35571883e235d3335054eb
Thank you!


Add repr() to Parameter for nicer docs

Currently, the documentation page for Range doesn't look nice. One
reason is that the parmeters are documented like this:
 start = <luigi.parameter.DateParameter object>

This patch makes it look like this:
 now = IntParameter (defaults to None): set to override current time. In seconds since epoch

Also see the screenshots in the upcoming PR.

So

Becomes

I do not care for this patch.  In my development process, I regularly use repr as a mechanism to enable debugging within REPL console, and this will seriously screw up my
I have a patch in progress to modifies repr to follow what repr is commonly used for: https://docs.python.org/2/library/functions.html#func-repr

Yea. I'm fine with nomerge on this. @demoray, do you happen to know what rules sphinx use to create the documentation? I was surprised that it used __repr__ and not __str__ as I've heard str is for supposed to be a human readable version (sensible for documentation).
Please share your patch where you define repr in the way it's supposed to be used.

@demoray, can you review again? Does it add value for you now?

Looks ok.  I'll submit a pull request once I'm done with my repr updates.

:+1:

Wow, surprised about involvement of repr in this, but this is awesome.

Yea, I tried to google if there's anything else sphinx uses before doing repr(), but didn't find anything. I checked and it doesn't do str() either.


Upgrade rand-token and remove specific test scripts

Fix #151

@diegohaz  Hi Diego, let me to check it in my local env.
Regards

Tested in my pc.
Works.


RuntimeError: CUDA Error encountered in <function CompiledLib.bn_mean_var_cuda at 0x7fd1b85757b8>

When I run run_local.sh, I get the problem as followed. How can I solve it? Thanks.
Traceback (most recent call last):   File "train.py", line 249, in <module>     main()   File "train.py", line 212, in main     preds = model(images, args.recurrence)   File "/home/ym/anaconda3/envs/py3.5_pytorch0.4.0/lib/python3.5/site-packages/torch/nn/modules/module.py", line 491, in __call__     result = self.forward(*input, **kwargs)   File "/home/ym/anaconda3/envs/py3.5_pytorch0.4.0/lib/python3.5/site-packages/torch/nn/parallel/data_parallel.py", line 114, in forward     outputs = self.parallel_apply(replicas, inputs, kwargs)   File "/home/ym/anaconda3/envs/py3.5_pytorch0.4.0/lib/python3.5/site-packages/torch/nn/parallel/data_parallel.py", line 124, in parallel_apply     return parallel_apply(replicas, inputs, kwargs, self.device_ids[:len(replicas)])   File "/home/ym/anaconda3/envs/py3.5_pytorch0.4.0/lib/python3.5/site-packages/torch/nn/parallel/parallel_apply.py", line 65, in parallel_apply     raise output   File "/home/ym/anaconda3/envs/py3.5_pytorch0.4.0/lib/python3.5/site-packages/torch/nn/parallel/parallel_apply.py", line 41, in _worker     output = module(*input, **kwargs)   File "/home/ym/anaconda3/envs/py3.5_pytorch0.4.0/lib/python3.5/site-packages/torch/nn/modules/module.py", line 491, in __call__     result = self.forward(*input, **kwargs)   File "/home/ym/CCNet/networks/ccnet.py", line 196, in forward     x = self.relu1(self.bn1(self.conv1(x)))   File "/home/ym/anaconda3/envs/py3.5_pytorch0.4.0/lib/python3.5/site-packages/torch/nn/modules/module.py", line 491, in __call__     result = self.forward(*input, **kwargs)   File "/home/ym/CCNet/libs/bn.py", line 184, in forward     self.activation, self.slope)   File "/home/ym/CCNet/libs/functions.py", line 183, in forward     _check(_ext.bn_mean_var_cuda, x, mean, var)   File "/home/ym/CCNet/libs/functions.py", line 16, in _check     raise RuntimeError("CUDA Error encountered in {}".format(fn)) RuntimeError: CUDA Error encountered in <function CompiledLib.bn_mean_var_cuda at 0x7fd1b85757b8>

I only want to use 1 or 2 gpus, is that because of gpu number?

It's ok to just use 1 or 2 gpus.
Have you compiled Inplace-abn or criss-cross attention with the correct arch. Please refer to here.


Update Cargo.toml

I think this is a typo, because it doesn't match the optional fontconfig version. It's stopping me building because I have a freetype static lib clash.

I believe you need to update the servo-fontconfig dependency to 0.5 as well.

@bors-servo r+

:pushpin: Commit a5b70ad has been approved by jdm

:hourglass: Testing commit a5b70ad62ee855d3f3688c501792cf61dd24d17f with merge d79a924fb7ac4e5c509a983ebad484f2d2ae513b...

:sunny: Test successful - checks-travis
Approved by: jdm
Pushing d79a924fb7ac4e5c509a983ebad484f2d2ae513b to master...

Should a new release be published for this?

A new release would be awesome - then I can go back to using crates.io rather than my own copy.

@jdm would it make sense to yank the 8.0 release? I'm not sure how Cargo works with multiple versions of the same dependency and it sounds like that release has issues with it.

I don't see a good reason to yank 0.8.

@derekdreery 0.9.0 is out now.


Instagram bridge delivers wrong URI

Hi folks,
some commit in the last few days (not more than roughly three or four days, because I pull from master nearly once or twice a week) must have had a regression in it that has broken (at least) the instagram bridge.
I'm currently getting a URI with an additional / in it, like:

https://www.instagram.com//p/BJ-3XnhDYO6/

instead of

https://www.instagram.com/p/BJ-3XnhDYO6/

If I can provide any further information to help debugging, don't hesitate to get back to me! :)
Cheers
Thomas

fixed in upcoming pull request

Thanks, mate! :+1:

@TwizzyDizzy The request pull #389 was merged. Can you check whether it works for you or not?

Fixed in #389


chore: add DB backup on repairnator

Done. Some docs are pushed on doc directory.

in French, original!

For the record, we have backups here: https://github.com/Spirals-Team/repairnator/tree/master/resources/data/mongodb-dump

https://owncloud.tutafey.eu/index.php/s/jbOnAIehvsqalEG Is the one last made by Simon before we moved the Repairnator instance.

We push the backups on https://github.com/repairnator/mongodb-backups in a compressed way (good compression, xz, 7z).


Updated constraint could not find existing matching constraint to update?

New Issue Checklist
🚫 If this template is not filled out your issue will be closed with no comment. 🚫

[x] I have looked at the Documentation
[x] I have read the F.A.Q.
[x] I have filled out this issue template.

Issue Info

Info
Value

Platform
osx

Platform Version
10.11

SnapKit Version
3.2.0

Integration Method
cocoapods

Issue Description
⚠ I have a controller containing a tabview and I make a constraint:
self.tabView.snp.makeConstraints { (maker) in
maker.top.equalTo(self.view).offset(34)
}
I like to adjust above offset value during running time when users click some menu item:
tabView.snp.updateConstraints { (maker) in
maker.top.equalTo(self.view).offset(60)
}
But I got  fatal Error("Updated constraint could not find existing matching constraint to update: ...
From other similar posts, I know that I can't not create a new constraints if it exists already, but constants such as offset here can be updated. But why this way still doesn't work?
Then I tried to remove the constraint (and there is only one constraint for this tabview) and re-made it:
tabView.snp.removeConstraints(). // first remove only one constraint for this view
ctrl.tabView.snp.remakeConstraints {(maker) in
maker.top.equalTo(self.view).offset(60)
}
This time, there was no error, but it ended with that my tab was not visible.
I took reference  issue #361 and #295 but I can't figure it out. Any suggestions?
thanks

@xiaoyunw this should work fine, can you share more of your view controller code that is handling this? It's hard to tell from the limited example.

Closing due to inactivity


Merge v0.11 into main

Version 0.11 can be merged into main once a certain level of support has been achieved in explainaboard_web.

It looks not trivial changes are pushed to non-main branch? could you merge it directly to main?


are you going to work on it in future

FIRST I get so excited that someone has listen to us,
then i download it and found it buggy,
my all expectations got killed at first sight
cheers to you for making this effort.
[i was not able to pull request thats why i posted here]

Hi nitin,
Nice that you like the project, appreciate that 😀
Let me say that I did not expect this project to catch a lot of interest for someone else, and therefore only wrote it for me and only tested it there so I am not very surprised that it's buggy ;-) But I think I can fix some; what bugs exactly did you encounter?

Are you still interested in bug fixes?

I will close this for inactivity now, feel free to open a new issue if you got any other requests.


Charts not showing on Safari

Hi all,
I used nvd3 linechart, and d3 pie, arc, and they work on Chrome and Opera.
However thw graphs are not showing on Safari.
Anyone has any idea with this issue?

It should work with the latest version.


CI Inverse diff problem

Currently we use git diff -R --exit-code to check that crate don't contains LGPL docs and that regeneration done right way.
It often cause misunderstanding. Ex. in https://travis-ci.org/gtk-rs/gtk/jobs/478784167#L751-L773 it shows that LGPL docs was accidentally added to code,
but even my first reaction was that docs purge not works :(
There was same misunderstanding for regen check in past.
Maybe we need remove -R options?

@bilelmoussaoui That's not a problem with the new CI or is it?

There's a WIP branch that's blocked by the gir and cairo-sys but it's part of the CI improvement issue


Update scala-library, scala-reflect to 2.13.14

About this PR
📦 Updates

org.scala-lang:scala-library
org.scala-lang:scala-reflect

from 2.12.19 to 2.13.14
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
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!

💡 Applied Scalafix Migrations

org.scala-lang:{scala-library,scala-compiler,scala-reflect,scalap}:2.13.0

dependency:Collection213Upgrade@org.scala-lang.modules:scala-collection-migrations:2.8.1
Documentation: https://github.com/scala/scala-collection-compat#collection213upgrade

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-lang" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, scalafix-migrations, commit-count:n:2

2.13 requires significant changes. See DAFFODIL-2152. Closing.


Can't make project with SystemJS (Angular 1 or Angular 2) to work

I followed all the instructions and when i try to create a project with systemjs it doesnt work.
First the installation throw this error:

I had to uninstall jspm and reinstall it to make it work. Then i run npm install and jspm install. And when i run gulp serve it throws this error:

Webpacks projects runs just fine, but i can't manage to make a project with system/jspm to work.
Thanks in advance.

IMO duplicate of https://github.com/FountainJS/generator-fountain-systemjs/issues/34
Already fixed. Waiting to release

Well in that issue they only talk about the jspm. I said that you can change the version of jspm and the project will install. But I still have another problem after that (second picture) and when I run gulp serve it throws another error. So I don't think it's a duplicate

We have release a new version 0.5.1 who should fix your issue.

@zckrs thanks it is working now, thou i must say (and i know it's probably because you guys are still developing it) the Angular 1.X with SystemJS comes without any router. This is not a BIG issue but just saying hehe.
If this is not an issue just say it and close the issue.

We don't address router libs in Fountain yet (for any framework). It's planned for the future.


[📑 Docs]: Enhance Accessibility: Add 'How to Contribute to AsyncAPI Docs' Section in README

What Dev Docs changes are you proposing?
As part of our community documentation efforts, a quick guide for new contributors needs to be added to the README for:

Providing concise steps for contributing to the documentation.
Including a direct link to the detailed documentation for comprehensive guidance.

As a result, new contributors can effortlessly discover how to get involved.
Code of Conduct

[x] I agree to follow this project's Code of Conduct

We're handling this in the Comunity content bucket via the new Onboarding Guide. :)
Closing.


Adaptation for multiclass classification

Working through the tutorial while trying to implement the same general architecture for a multiclass classifier. Is there anything specific that needs to be changed to make it work other than setting the numClasses variable and setting the label vectors appropriately? Mainly wondering if the value = tf.transpose(value, [1, 0, 2]) operation after setting the weight matrix and bias needs to be changed in any way? Thanks in advance.

I believe those two changes should be enough. The tf.transpose operation shouldn't be affected. The purpose of that operation is to turn the volume from [batch_size, time_step, lstm_output_dim] into [time_step, batch_size, lstm_output_dim], and then to get the last hidden state vector in the following line, so the number of classes won't affect that.
You can also check out this post for more info on Tensorflow LSTMs and related operations.


Additional Limitations

In addition to above listed Considerations and Limitations:

It is important to note that once an external participant has been invited in PowerBI B2B, the invitation cannot be withdrawn. In other words, if you elect to remove a guest from access a workspace, the guest is still able to access the workspace even though they are no longer authorised to do so.  (BTW.  This has been reported as a security flaw to MS, in that the behaviour does not align with any other M365 products that are B2B enabled).

There is also a limitation with Conditional Access.  To access the workspace the Guest needs a license. If they already have one in their tenant that should be able to be used to grant access along with the authorisation to the workspace.  There is a bug in Power BI B2B where the license of the guest user from their home tenant is not recognised.

For multinational organisations operating under single M365 Tenant, once PowerBI B2B has been activated - it is activated for all. (ie. Once it is turned on, all workspace owners will have the right to leverage B2B).

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

ID: 725c3bff-b250-c882-6f2b-1f2cb7f00341
Version Independent ID: f48832e7-8fc5-498a-8522-435a8ce42129
Content: Distribute content to external guest users with Azure AD B2B - Power BI
Content Source: powerbi-docs/admin/service-admin-azure-ad-b2b.md
Service: powerbi
Sub-service: powerbi-admin
GitHub Login: @kfollis
Microsoft Alias: kfollis

Thanks for your comment, @[github person who posted].
I've assigned this issue to the doc author for review.

@mibruhje - can you please review this feedback and suggest an article update?

I started an email thread for this.


Need to check IAM Roles work

Check if IAM roles work and if not make them then document it

IAM roles work. I was able to create an IAM role with the following policy and get it working with uploads.
{     "Version": "2012-10-17",     "Statement": [         {             "Sid": "Stmt1234567890",             "Effect": "Allow",             "Action": [                 "s3:*"             ],             "Resource": [                 "arn:aws:s3:::bucketname/*"             ]         }     ] }

Thanks :)


Crash on cycle Setup => Reset => Setup

Hey we noticed a bug with the app, that isn't a deal breaker, but is a PITA.
You can try on https://btcpay669124.lndyn.com/apps/4MwgTm6pb2VqQduyX9Sq5w5NVVeC/boltcardfactory

In the BTCPay factory page, you click "Setup". The app open, you present the card.
You swipe back to BTCPay, click "Reset" in BTCPay. Go to the app, present the card.
You swipe back to BTCPay, click "Setup" in BTCPay, present the card => Fail.

The workaround is to close the boltcard app and click on Setup in BTCPay again.
Version: 0.3.2

Hey I've just tested this and I can't see it happening on my device. Did you submit a crash report to the app store?

It is not a crash, this is Error: transceive fail
I am not switching app, I am swiping back

As you, just switch app doesn't trigger the issue.

https://github.com/user-attachments/assets/a2800cf4-2bdb-49e4-8d2d-21abfce12f6d

I got the error on wiping and not setting up after I tried the 3rd card with multiple setup/reset rounds. After clicking "wipe again" it worked and I could also do a setup again.
Using Pixel 7 on latest GrapheneOS and app version 0.3.2 downloaded from github


Edge troubleshooting

Docs Build status updates of commit 2e2488e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/concepts/browsers-used-by-office-web-add-ins.md
:warning:Warning

Details

docs/concepts/browsers-used-by-office-web-add-ins.md

Line 60: [Warning] Invalid file link:(~/../troubleshoot/error-messages/cannot-open-add-in-from-localhost).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 309759c:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/concepts/browsers-used-by-office-web-add-ins.md
:warning:Warning

Details

docs/concepts/browsers-used-by-office-web-add-ins.md

Line 60: [Warning] Invalid file link:(~/../../troubleshoot/error-messages/cannot-open-add-in-from-localhost).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit fe14777:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/concepts/browsers-used-by-office-web-add-ins.md
:warning:Warning

Details

docs/concepts/browsers-used-by-office-web-add-ins.md

Line 60: [Warning] Invalid file link:(~/../../troubleshoot/error-messages/cannot-open-add-in-from-localhost).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 8d3fb17:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/browsers-used-by-office-web-add-ins.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


non-decorated Windows Resize broken on Linux

@kekekeks  "mouse capture is not reset
we kinda need to change the api"
@jmacato "iv seen this before"

Sometimes it does that randomly when resized. Definitely a pointer capture issue since when locked to resizing mode you can't click anything else inside the app.

We basically need to pass PointerEventArgs to Begin(Move/Resize)Drag so it will reset the pointer capture. I think it's also required to pass the event id to start dragging on Wayland, so it will be required anyway.