ValueError probably caused from some missing parallel results

I'm processing an SDF file that fails with the following error:
"""
Traceback (most recent call last):
  File "/efs/home/muammar.elkhatib/miniconda3/envs/ml/lib/python3.11/site-packages/joblib/_utils.py", line 72, in __call__
    return self.func(**kwargs)
           ^^^^^^^^^^^^^^^^^^^
  File "/efs/home/muammar.elkhatib/miniconda3/envs/ml/lib/python3.11/site-packages/joblib/parallel.py", line 598, in __call__
    return [func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/efs/home/muammar.elkhatib/miniconda3/envs/ml/lib/python3.11/site-packages/joblib/parallel.py", line 598, in <listcomp>
    return [func(*args, **kwargs)
            ^^^^^^^^^^^^^^^^^^^^^
  File "/efs/home/muammar.elkhatib/miniconda3/envs/ml/lib/python3.11/site-packages/datamol/utils/jobs.py", line 83, in _run
    return fn(args, **fn_kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/efs/home/muammar.elkhatib/miniconda3/envs/ml/lib/python3.11/site-packages/medchem/structural/lilly_demerits/_demerits.py", line 324, in _score
    results["mol"] = mols
    ~~~~~~~^^^^^^^
  File "/efs/home/muammar.elkhatib/miniconda3/envs/ml/lib/python3.11/site-packages/pandas/core/frame.py", line 4311, in __setitem__
    self._set_item(key, value)
  File "/efs/home/muammar.elkhatib/miniconda3/envs/ml/lib/python3.11/site-packages/pandas/core/frame.py", line 4524, in _set_item
    value, refs = self._sanitize_column(value)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/efs/home/muammar.elkhatib/miniconda3/envs/ml/lib/python3.11/site-packages/pandas/core/frame.py", line 5266, in _sanitize_column
    com.require_length_match(value, self.index)
  File "/efs/home/muammar.elkhatib/miniconda3/envs/ml/lib/python3.11/site-packages/pandas/core/common.py", line 573, in require_length_match
    raise ValueError(
ValueError: Length of values (4666) does not match length of index (4658)

Based on the traceback error, some molecules are not returning results when computed with the parallel backend. Because there is a mismatch between the original pd.DataFrame and the one with results, pandas cannot proceed.  I will try to understand what molecules are failing but I think it would be good if at least the library could catch the error and populate the result with np.nan.  Would you have any suggestions?
Best,

Good catch, can you check whether all your molecules parses correctly without any issues ? (use the datamol to_mol  function for that).
Ideally, you have to filter out all invalid molecules because of how the lilly code handles them.

Good catch, can you check whether all your molecules parses correctly without any issues ? (use the datamol to_mol function for that).
Ideally, you have to filter out all invalid molecules because of how the lilly code handles them.

They all parse without issues. I used this code:
from rdkit import Chem
import datamol as dm

suppl = Chem.SDMolSupplier(datasets[4][1])

for mol in tqdm(suppl, total=len(suppl)):
    dm.to_mol(mol)

Thanks for your fast reply 😄

dm.to_mol can return None. Can you check if any of the molecule is None ? Also it helps to standardize the list of molecules so that the smiles are canonical.

dm.to_mol can return None. Can you check if any of the molecule is None ? Also it helps to standardize the list of molecules so that the smiles are canonical.

I passed the Mol objects to rdMolStandardize.Cleanup and created a pandas Series to count all nans. Ther are zero. Let me know if you would require more information.
Thanks.

Ok, this is indeed weird. If you are able to share your SDF for me to debug, that would be nice. Otherwise if you have an alternative SDF, it will be helpful.

Ok, this is indeed weird. If you are able to share your SDF for me to debug, that would be nice. Otherwise if you have an alternative SDF, it will be helpful.

Thank you for your fast responses. I will check if I can share the SDF file for you to debug. I don't have another SDF that could be used to reproduce this error.

@muammar, any updates on this to share ?

@muammar, any updates on this to share ?

I had to use the ruby implementation of the rules to get the ball rolling. The library presenting the problem is the Maybridge HitCreator. I'm unsure whether I can share the SDF, but they have a request form. Thank you for your fast responses, @maclandrol

hi, I have the same issue and can provide the smiles list for debugging
smiles_list = [
    "C[n+]1ccc2c(c1)[nH]c1ccc(O)c(Br)c12",
    "O=C1OC2(c3ccccc31)c1ccc(O)c(Br)c1Oc1c2ccc(O)c1Br",
    "O=c1cc(COBr)oc(Br)c1OBr",
    "C=C(C)C(=O)Oc1c(Br)c(Br)c(Br)c(Br)c1Br",
    "CC(=O)Oc1c(Br)cc(C#N)cc1Br",
    "CC(=O)Oc1c(Br)cc(Br)cc1Br",
    "O=C(Oc1c(Br)cc(Br)cc1Br)c1ccccc1O",
    "C=CC(=O)Oc1c(Br)cc(Br)cc1Br",
    "O=C(C=CC(=O)Oc1c(Br)c(Br)c(Br)c(Br)c1Br)Oc1c(Br)c(Br)c(Br)c(Br)c1Br",
    "O=C(C=CC(=O)Oc1c(Br)cc(Br)cc1Br)Oc1c(Br)cc(Br)cc1Br",
    "CCCC(=O)Oc1c(Br)cc(C#N)cc1Br",
    "CCCCCCC(=O)Oc1c(Br)cc(C#N)cc1Br",
    "CCCCCCCC(=O)Oc1c(Br)cc(C#N)cc1Br",
    "COC1=C(Br)C(O)C(O)(CC#N)C=C1Br",
    "COc1cc(Cc2cnc(N)nc2N)cc(OCCCC(CCC(=O)O)C(=O)O)c1Br",
    "COc1cc(OC)c(C=C(C#N)C(N)=S)cc1Br",
    "Brc1c(Br)c(Br)c(Oc2c(Br)c(Br)c(Br)c(Br)c2Br)c(Br)c1Br",
    "Oc1c(Br)cc(Br)cc1Oc1ccc(Br)cc1Br",
    "Brc1cc(Br)c(Oc2c(Br)c(Br)c(Br)c(Br)c2Br)cc1Br",
    "CCOc1cc(OCC)c([N+](=O)[O-])nc1Br",
    "CCOc1ccc([N+](=O)[O-])nc1Br",
    "C=CCOc1c(Br)c(Br)c(Br)c(Br)c1Br",
    "Brc1cnc(OCCCCCN2CCSC2)c(Br)c1",
    "Brc1cccnc1OCCCCCN1CCSC1",
    "NOCc1ccc(Br)c(OP(=O)(O)O)c1",
    "CC(C)(C)C1=C(Br)C(OC(=O)C(F)(F)F)(C(C)(C)C)OP1(=O)c1ccccc1",
    "NS(=O)(=O)c1cc2c(cc1Br)NC=NS2(=O)=O",
    "CC(=NS(=O)(=O)c1cc(Br)ccc1Br)OC(C)C",
    "CCc1sc(C=C(C#N)C(N)=S)cc1Br",
    "O=C(Sc1ccc(Br)cc1Br)c1ccc([N+](=O)[O-])cc1",
    "Cc1cc2c(Br)c(C(C)C)c(Br)c(Br)c(C)c-2c1[C+]1C(=O)C(c2c(C)cc3c(Br)c(C(C)C)c(Br)c(Br)c(C)c2-3)C1=O",
    "CCOC(=O)c1nn(-c2ccc(C)cc2)c(=O)c2c(N)sc(Br)c12",
    "Oc1ccc(Br)c2cccnc12",
    "COc1ccc(Br)c2c1C(=O)C=C2",
    "C=CCN1CCc2c(cc(O)c(O)c2Br)C(c2ccccc2)C1",
    "Brc1cc2c(Br)cc(Br)c(Br)c2c(Br)c1Br",
    "O=[N+]([O-])c1cccc2c(Br)[nH]nc12",
    "F[B-](F)(F)F.N#[N+]c1ccc(Br)c2c1CCCC2",
    "O=C(OCCN1C(=O)c2cccc3c(Br)ccc(c23)C1=O)OCc1ccccc1",
    "CN1C(=O)c2cccc3c(Br)ccc(c23)C1=O",
    "COc1ccc(CCN2C(=O)c3cccc4c(Br)ccc(c34)C2=O)cc1",
    "O=C1NC(=O)c2ccc(Br)c3cccc1c23",
    "O=C1c2cc(Br)ccc2-c2ccc(Br)c3cccc1c23",
    "O=C1c2cc(S(=O)(=O)O)ccc2-c2ccc(Br)c3cccc1c23",
    "O=C1c2ccccc2-c2ccc(Br)c3cccc1c23",
    "O=C1OC(=O)c2ccc(Br)c3cccc1c23",
    "CN1C(=O)N(C)c2ccc(Br)c3cccc1c23",
    "O=c1c2cc(Br)c3cccc4c(=O)c5cc(Br)c6cccc1c6c5-c2c34",
    "CC(C)CC1C(=O)N2CCCC2C2(O)OC(NC(=O)C3C=C4c5cccc6[nH]c(Br)c(c56)CC4N(C)C3)(C(C)C)C(=O)N12",
    "Brc1c2c(c(Br)c3nsnc13)N=S=N2",
    "O=C1OC2(c3cc(Br)c(O)c(Br)c3Oc3c2cc(Br)c(O)c3Br)c2c(Cl)ccc(Cl)c21",
    "O=C1OC2(c3ccccc31)c1cc(Br)c(O)c(Br)c1Oc1c2cc(Br)c(O)c1Br",
    "O=C(O)c1ccccc1-c1c2cc(Br)c(=O)c(Br)c-2oc2c(Br)c(O)c(Br)cc12",
    "COc1ccc2nc3cc(OC)c(=O)c(Br)c-3sc2c1",
    "O=C(O)C1Nc2c(O)ccc(Br)c2C2C=CCC12",
    "CCn1nc(-c2cc(Br)ccc2F)nc2c(=O)n(C)c(=O)nc1-2",
    "Cc1cc(Br)ccc1-n1c(CF)nc2ccc(N)cc2c1=O",
    "COc1cc(Br)cnc1[N+](=O)[O-]",
    "Cc1cc(Br)ccc1N=NC(C#N)C#N",
    "Cc1cc(Br)cc(C(=O)NC(O)C(Cl)(Cl)Cl)c1O",
    "O=C(C(=O)c1cc(Br)ccc1O)c1cc(Br)ccc1O",
    "O=C(Nc1ccc(Br)cc1)c1cc(Br)ccc1O",
    "O=C(O)c1cc(Br)ccc1O",
    "CC(=O)Oc1ccc(Br)cc1C(=O)O",
    "CCn1nc(-c2cc(Br)ccc2OC)nc2c(=O)n(C)c(=O)nc1-2",
    "CCN(CC)CCN(CCN(CC)CC)c1cc(OC)c(OC)cc1Br",
    "COC(=O)COc1ccc(Br)cc1CSC(=N)N",
    "CCOC(=O)COc1ccc(Br)cc1CSC(=N)N",
    "CS(=O)(=O)c1ccc(-c2cc(Br)sc2-c2ccc(F)cc2)cc1",
    "Cc1cc(Br)ccc1SCNC(=O)Oc1cccc2c1OC(C)(C)C2",
    "CCOC(=O)C(C(=O)OCC)c1cc(Br)c2noc3c2c1C(=O)c1ccccc1-3",
    "O=C1c2ccccc2-c2onc3c(Br)cc(O)c1c23",
    "Brc1cc(CN2CCCNCCNCCCNCC2)cc(CN2CCCNCCNCCCNCC2)c1",
    "Nc1c(Br)cc([N+](=O)[O-])cc1[N+](=O)[O-]",
    "CC(C)(C)c1cc(C=C(C#N)C#N)cc(Br)c1O",
    "CCc1oc2ccccc2c1C(=O)c1cc(Br)c(O)c(Br)c1",
    "O=C1OC(c2cc(Br)c(O)c(Br)c2)(c2cc(Br)c(O)c(Br)c2)c2ccccc21",
    "O=S1(=O)OC(c2cc(Br)c(O)c(Br)c2)(c2cc(Br)c(O)c(Br)c2)c2ccccc21",
    "O=C1C(c2cc(Br)c(O)c(Br)c2)=C(c2cc(Br)c(O)c(Br)c2)C(=O)N1CCc1ccc(O)cc1",
    "O=[N+]([O-])c1cc(Br)c(O)c(Br)c1",
    "COc1cc2ncnc(Nc3cc(Br)c(O)c(Br)c3)c2cc1OC",
    "C=CC(=O)Oc1c(Br)cc(C(C)(C)c2cc(Br)c(OC(=O)C=C)c(Br)c2)cc1Br",
    "COc1ccc(CCN2C(=O)C(c3cc(Br)c(OC)c(Br)c3)=C(c3cc(Br)c(OC)c(Br)c3)C2=O)cc1",
    "CN(C)S(=O)(=O)n1c(C#N)nc2c(Br)cc(C(F)(F)F)cc21",
    "O=C1C(Br)=CC(=NCl)C=C1Br",
    "O=S(=O)(N=C1C=CC(Br)=CC1=NS(=O)(=O)c1ccccc1)c1ccccc1",
    "O=C1C(Br)=CC(=NS(=O)(=O)c2ccccc2)C=C1Br",
    "O=c1cc(Br)c(=O)[nH][nH]1",
    "Nc1c(Br)cc(Br)c2c1C(=O)c1ccccc1C2=O",
    "Cc1ccc2c(Br)cc(Br)c(O)c2n1",
    "Oc1c(Br)cc(Br)c2cccnc12",
    "O=C(COc1c(Br)cc(Br)c2cccnc12)Nc1ccc2c(c1)OC(F)(F)O2",
    "O=C(Nc1ccc(Br)cc1)c1cc(Br)cc(Br)c1O",
    "O=Cc1cc(Br)cc(Br)c1O",
    "O=P(O)(O)Oc1c(Br)cc(Br)cc1-c1cc(Br)cc(Br)c1O",
    "COc1ccc(N=C2SCC(=O)N2Cn2cnc3c(Br)cc(Br)cc3c2=O)cc1",
    "Cc1ccccc1N=C1SCC(=O)N1Cn1cnc2c(Br)cc(Br)cc2c1=O",
    "O=C1CSC(=Nc2ccccc2)N1Cn1cnc2c(Br)cc(Br)cc2c1=O",
    "S=c1[nH]c2c(Br)cc(Br)cc2c(=S)s1",
    "CC(=O)n1oc2c(Br)cc(Br)cc2c1=O",
    "COC(=O)n1oc2c(Br)cc(Br)cc2c1=O",
    "CCOC(=O)n1oc2c(Br)cc(Br)cc2c1=O",
    "CCCSP(=O)(OCC)Oc1ccc(Br)cc1Cl",
    "ON1C(c2ccc(Br)cc2F)=NNc2cc(Cl)c(Cl)cc21",
    "O=c1ccc(=O)n(-c2ccc(Br)cc2F)[nH]1",
    "NC(=O)CC1(O)C=C(Br)C(=O)C(Br)=C1",
    "CCOC(=O)CC1(O)C=C(Br)C(=O)C(Br)=C1",
    "CC(=O)OC1(CC#N)C=C(Br)C(=O)C(Br)=C1",
    "COC1(OC)C=CC(=O)C(Br)=C1",
    "CC(=O)OCC(=O)C1(OC(C)=O)CCC2C3CC(F)C4=CC(=O)C(Br)=CC4(C)C3(F)C(O)CC21C",
    "CCCCCCCN(CCCCCCC)CC(O)c1cc2ccccc2c2cc(Br)ccc12",
    "O=C1NC(=O)C(c2c[nH]c3cc(Br)ccc23)=C1c1cn(C(c2ccccc2)(c2ccccc2)c2ccccc2)cn1",
    "O=C1NC(=O)C(c2c[nH]c3cc(Br)ccc23)=C1c1cnc[nH]1",
    "O=C(NC=Cc1c[nH]c2cc(Br)ccc12)C(=O)c1c[nH]c2ccccc12",
    "O=C1C(NC2CCCCC2)=C(c2ccccc2)c2cc(Br)ccc21",
    "O=C1Nc2cc(Br)ccc2C1=O",
    "O=C1c2ccccc2-c2ccc3c4ccc5c6c(ccc(c7ccc1c2c37)c64)-c1cc(Br)ccc1C5=O",
    "O=C1NC(=O)c2cc(Br)ccc21",
    "CC1(C)C(=O)N(CCc2ccc(S(=O)(=O)NC(=O)NC34CC5CC(CC(C5)C3)C4)cc2)C(=O)c2cc(Br)ccc21",
]

Hi! I got a same issue while using Lilly demerits filter. I was able to proceed by dropping rows corresponding to missing molecules supplied by the user.
If this is an acceptable behavior in general, I guess we can implement as such in LillyDemeritsFilter. I did not made a pull request yet as I am unsure whether this should be done this way, or I am misunderstanding something.


Error in /pitch-a-talk

Error in quantsquills.com
Error in /pitch-a-talk
Bad Gateway
View on Bugsnag
Stacktrace
src/components/TalkPitchForm.js:54 - v/l</<
src/components/TalkPitchForm.js:48 - promise callback*v/l<
src/components/useForm.js:78 - BoRJ/t.a/f<
.cache/production-app.js:181 - UxWs/</</<
.cache/production-app.js:179 - UxWs/</<
.cache/production-app.js:153 - promise callback*UxWs/<
.cache/production-app.js:39 - promise callback*UxWs

View full stacktrace
Created automatically via Bugsnag

Sendgrid appeared to have deleted our API key for some reason. New one created, site re-built, all working fine.

Sendgrid appeared to have deleted our API key for some reason. New one created, site re-built, all working fine.


Cleaned up

All cleaned up

The code changes make sense. I'm hoping to have time this weekend to build and test it locally, then I'll merge and publish a new version to Steam. Thanks again for the translations and the fix!!


How to install configuration and generate an interactive platform？

How do I install the configuration and generate an interactive platform like the one described in the article "An interactive visualization platform for deep symbolic regression"?

Unfortunately we weren't able to get the interactive platform approved for public release :(


app.rb

This definitely looks legit - @jamiemtdwyer any reason this hasn't been merged?

Actually, on further inspection - I can't tell if this is needed. Will defer to Jamie on this.

I believe the PR author was confused because the tutorial on Shopify was not up to date with the most recent code here – this has been fixed, so this should be closed.


add_requiredStatement not accepting entry with included language

Hi Giacomo, not sure if you plan on maintaining this package with Prezi3 in the works, but I'm trying to add required statement via entry and getting
AssertionError: Language must be a valid BCP47 language tag or none. Please read https://git.io/JoQty.

I'm using pt-BR, which is included by iiifpapi3.LANGUAGES = ["pt-BR", "en"], in pretty much all other fields without a problem.
If I use the label, value, language_v, language_l form it accepts pt-BR, but ends up overwriting the en block in the final manifest. Is this the desired behavior? All other add_ methods work multiple times. Maybe it should be set_requiredstatement?

iiifpapi3.LANGUAGES = ["pt-BR", "en"]

manifest.add_label("pt-BR", item["Title"]) # works fine

required = {
        "label": {"en": ["Hosting"], "pt-BR": ["Hospedagem"]},
        "value": {
            "en": ["Hosted by imagineRio"],
            "pt-BR": ["Hospedado por imagineRio"],
        },
    }

manifest.add_requiredStatement(entry=required) # raises AssertionError

    manifest.add_requiredStatement( # works fine
        label="Hosting",
        value="Hosted by imagineRio",
        language_l="en",
        language_v="en",
    )

    manifest.add_requiredStatement( # also works, but overwrites en entry
        label="Hospedagem",
        value="Hospedado por imagineRio",
        language_l="pt-BR",
        language_v="pt-BR",
    )

Thanks Martim, I think should be added only once so you are right that it should be a set method. I will have a look to the the error that you have in the next hours.
Thanks for your precious feedback!

I have checked the spec:
https://github.com/IIIF/presentation-validator/blob/90197c88d8676c652cf475909bf34ab689fb6ea0/schema/iiif_3_0.json#L367
Only one required statement is allowed, but rememeber that add_requiredStatement return a Multilanguage object if you leave it empty so you can do something like this for your purpose:
reqst = manifest.add_requiredStatement()
reqst.add_label('Hosting','en')
reqst.add_value('hosted by imagineRio','en')
reqst.add_label('Hospedagem','pt-BR')
reqst.add_value('Hospedado per imagineRio','pt-BR')

Got it. However, the entry form not accepting pt-BR is a bug?

Yes, I have just pushed a patch for this bug:
https://github.com/giacomomarchioro/pyIIIFpres/commit/71b5d26853b9433629e20f24312cc5459ae95fd7
Thanks a lot for that.
I suggest however to try to use entry only as a last resort because usually, it should not perform any check when you entry data directly as a dict.  Thanks for your feedbacks!


Chat - Onboarding modal opens briefly, Concierge chat is with red dot in LHN if open public room

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.6-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4706407
Issue reported by: Applause - Internal Team
Action Performed:

Go to https://staging.new.expensify.com/
Log out
Navigate via public room link (my link https://staging.new.expensify.com/r/1353086947735283
)

Expected Result:
Concierge chat is without red dot in LHN
Actual Result:
Onboarding modal opens for a moment and Concierge chat is with red dot in LHN if open public room as anon user
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/7acf5ecb-0319-4905-aa05-8ab42b96ba00
View all open jobs on GitHub

@arosiclair  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-vsp

cc @filip-solecki @allgandalf @adamgrzybowski I believe this might be related to or change here https://github.com/Expensify/App/pull/44536

Hey, I'm developer from SWM. I'd like to take care of this ticket

@BrtqKr Thank you! Do you think you can raise the PR for the fix today?

I'll look into it in a moment, but I haven't researched it yet, so I'm not sure. I'll try to handle it until the end of the day, but I'll give you an update later if that's ok

The onboarding modal should not show for anonymous accounts and Concierge chat loads without red dot in LHN

Proposal
Please re-state the problem that we are trying to solve in this issue.
The onboarding modal opens briefly and the Concierge chats with a red dot in LHN if we open a public room
What is the root cause of that problem?
On Navigation root, we redirect the user to Onboarding when the user has not  yet completed
https://github.com/Expensify/App/blob/a91b7f8526dd3798deb247922be88cbff503f16a/src/libs/Navigation/NavigationRoot.tsx#L90-L93
But we didn't check here if the user is anonymous and inside the hasCompletedGuidedSetupFlowSelector function we didn't check if the onboarding value is undefined.
Because when onboarding data has not yet been applied or is still in the process the value is undefined, so if we check if it's an array it will return false and will return the user has not yet completed onboarding
https://github.com/Expensify/App/blob/a91b7f8526dd3798deb247922be88cbff503f16a/src/libs/hasCompletedGuidedSetupFlowSelector.ts#L4-L10
What changes do you think we should make in order to solve the problem?
We can check if the user is anonymous inside this condition
https://github.com/Expensify/App/blob/a91b7f8526dd3798deb247922be88cbff503f16a/src/libs/Navigation/NavigationRoot.tsx#L90-L93
And we can use this logic for the hasCompletedGuidedSetupFlowSelector function
https://github.com/Expensify/App/blob/a91b7f8526dd3798deb247922be88cbff503f16a/src/libs/actions/Welcome.ts#L50-L52
And by doing this, it also resolves the Concierge red dot
What alternative solutions did you explore? (Optional)

@allgandalf Please kindly review my proposal

Regarding the onboarding it was enough to apply the condition for authenticated as in this commit from yesterday.
I wanted to verify the red dot though and I think we shouldn't handle it on the front-end at all, because it's coming from the errorFields on the back-end side

I believe we have reverted the offending PR and the fix will be handled in the original scope of that change


TypeError: init() missing 1 required positional argument: 'save_root

In examples folder, while running video_test.py throws the following error.
abhishek@abhishek-ubuntu:~/Desktop/marlgrid-master$ /usr/bin/python3 /home/abhishek/Desktop/marlgrid-master/examples/video_test.py
/usr/local/lib/python3.8/dist-packages/gym_minigrid-1.0.2-py3.8.egg/gym_minigrid/roomgrid.py:302: SyntaxWarning: "is" with a literal. Did you mean "=="?
  if front_cell is None or front_cell.type is 'wall':
Traceback (most recent call last):
  File "/home/abhishek/Desktop/marlgrid-master/examples/video_test.py", line 7, in <module>
    env = GridRecorder(env, render_kwargs={"tile_size": 11})
TypeError: __init__() missing 1 required positional argument: 'save_root'

Problem solved using pycharm
I copied the steps from your video:
pip install git+git://github.com/kandouss/marlgrid.git

import gym, marlgrid.envs
env = gym.make('Goalcycle-demo-solo-v0')

Now, when I am trying to run the "video_test.py"(this file did not get downloaded using the first command), it throws the error.

any updates??


on_fit_batch_start() method not called in the beginning of the fit_batch() method

Describe the bug
Hello !
I would first like to thank you for the useful tools that Speechbrain provides us !
I noticed that the method on_fit_batch_start() is never called anywhere in the Brain class.
Expected behaviour
Shouldn't we call the self.on_fit_batch_start() method on the fit_batch() method ?
Somewhere around this line of code : https://github.com/speechbrain/speechbrain/blob/develop/speechbrain/core.py#L1218
Thank you very much :)
To Reproduce
No response
Environment Details
No response
Relevant Log Output
No response
Additional Context
No response

You are correct! @tiatariene I will fix this ASAP. Thanks a lot :)

Nice ! Many thanks


dbt init --adapter default to the available adapter

Describe the bug
In v0.18+, dbt init takes an additional argument --adapter so as to generate profiles.yml according to the adapter type specified.
Steps To Reproduce
Fresh install of a non-core plugin:
pip install dbt-spark

Initialize a project:
$ dbt init my_project
Running with dbt=0.18.0
Creating dbt configuration folder at /Users/jerco/.dbt
With sample profiles.yml for redshift
Encountered an error:
Runtime Error
  Could not find adapter type redshift!

Expected behavior
The --adapter argument should check the adapters available in the context
System information
Which database are you using dbt with?

[ ] postgres
[ ] redshift
[ ] bigquery
[ ] snowflake
[x] other (any community-contrib plugin)

The output of dbt --version:
dbt --version
installed version: 0.18.0
   latest version: 0.18.0

Up to date!

Plugins:
  - spark: 0.18.0

Copying from #3434:

If the --adapter argument is not passed, I think our options are:

init checks the adapters available, rather than falling back to redshift as the default
init falls back to a more reasonable, generic default (not redshift) that ships inside of core


⚠ Jenkins Plugins Artifactory has degraded performance

In 6899c16, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1289 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 55ea09b after 12 minutes.


Images are not respecting the width and height

Description
Images are not respecting the width and height, the final size is the original and not the one defined in the properties.
React Native version:
System:
OS: Linux 5.5 Arch Linux undefined
CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
Memory: 1.51 GB / 15.35 GB
Shell: 5.8 - /usr/bin/zsh
Binaries:
Node: 10.16.0 - ~/.nvm/versions/node/v10.16.0/bin/node
Yarn: 1.21.1 - ~/.nvm/versions/node/v10.16.0/bin/yarn
npm: 6.13.6 - ~/.nvm/versions/node/v10.16.0/bin/npm
Watchman: 4.9.0 - /usr/bin/watchman
SDKs:
Android SDK:
API Levels: 24, 26, 28, 29
Build Tools: 27.0.3, 28.0.3, 29.0.1, 29.0.2
System Images: android-28 | Google Play Intel x86 Atom, android-29 | Google Play Intel x86 Atom
IDEs:
Android Studio: 3.5 AI-191.8<IP_ADDRESS>10548
npmPackages:
@react-native-community/cli: ^2.9.0 => 2.10.0
react: 16.11.0 => 16.11.0
react-native: 0.62.0 => 0.62.0
npmGlobalPackages:
react-native: 0.61.3
Steps To Reproduce

npx react-native init newProject;
Add the image component, with a large image and try to define the width and height;
The defined width and height will not be respected.

Expected Results
According to the documentation and like the editable example in it, the expected result is this:

However the result achieved, using the same code is this:

The code used:
import React from 'react';
import {Image, View, StyleSheet} from 'react-native';

import logo from './logo.png';

const styles = StyleSheet.create({
  container: {
    paddingTop: 50,
  },
  logo: {
    width: 66,
    height: 58,
  },
});

const App = () => {
  return (
    <View style={styles.container}>
      <Image source={logo} styles={styles.logo} />
    </View>
  );
};

export default App;

What happens if you try "requiring" the image?
i.e
<View style={styles.container}>
      <Image source={require('./logo.png')} styles={styles.logo} />
    </View>

@marcosdourado the problem is your code, instead of styles={styles.logo} use style={styles.logo} without (s)

Thanks @mydesweb , it was a big vacuum on my part, it was drawn by an S, who never? Thank you very much.


sensors fix auto usage

Merging immediately as this is potentially nasty if the primary gyro changes.
@bkueng please still take a look.

Yep looks good.


Is there a way to restore an old .wallet file?

Hello i found an old .wallet file on my old USB-Drive.
I don't have any .info file or backup file, just the .wallet file.
Importing into the multidoge wallet doesn't work. Is there a way to find the Private key or the adress to even check if i have ballance on this Wallet?

I just did this. You can download Dogecoin-qt (Dogecoin core), copy the wallet.dat file to the location used by Dogecoin-qt, then start the app.
It should show the unconfirmed balance and then you can go to the "Much receive" tab, open up Help > Debug window > Console and type dumpprivkey <receive address> for each address. Put all the outputs in a doge-private-keys.key file (each key separated by a new line) like this:
key1_goes_here
key2_goes_here

Then import this key file in MultiDoge. It will recognize all the keys.
Unfortunately, synchonizing is going to be a significant amount of download.

Thank you for your answer! But i don't have a "wallet.dat" file, i only have a "multidoge.wallet" file from Multidoge. Is there a way to convert it to an .dat file, so i can try this?

i have the same issue, trying to open a old .wallet file errors out with com.google.dogecoin.store.UnreadableWalletException Could not read wallet

If you have a backup of .wallet you don't use the import tool. Did you try File-> Open Wallet and then choose the .wallet file you want to restore?

Yes i tried this and got an error.
The Error sayes: "com.google.dogecoin.store.UnreadableWalletException Could not read wallet"

Could your wallet backup file have gotten damaged? That's kind of what it sounds like to me.

Could be yeah. Its an 6 year old Wallet and got moved through 3 PCs over the time. Sadly i only have the ".Wallet" file and not more.

 I bet it doesn't know how to restore with only that one piece, It might be worth holding onto. If you think you have a bunch of coins maybe you would eventually find someone who had the knowledge to reverse engineer enough to see if the private keys are in that file. I don't know enough to be able to tell.

Thanks for your answers! I will definitely hold on to it :)

I found this article about .wallet files:
https://fileinfo.com/extension/wallet
It looks like there is a tool for extracting info from a Multibit wallet which uses the .wallet extension as well so there's a chance it might work. It does take some technical knowledge, but if you are really determined, or have a friend who can help, this might be a way to get at your keys:
https://github.com/Multibit-Legacy/read-multibit-wallet-file
Good Luck!

Thanks for your answers! I will definitely hold on to it :)

you wallet file is corrupted maybe. putting it on the USD stick is easy get damage. we may help decrypt it if you have password of it. but chance is low.
www.btc2doge.com

i have the same issue, trying to open a old .wallet file errors out with com.google.dogecoin.store.UnreadableWalletException Could not read wallet. i dont have the .info file

conact us. we may help decrypt the wallet if you have password. although the chance is low. how much doge u hold on that wallet?
www.btc2doge.com

i have the same issue, trying to open a old .wallet file errors out with com.google.dogecoin.store.UnreadableWalletException Could not read wallet. i dont have the .info file

conact us. we may help decrypt the wallet if you have password. although the chance is low. how much doge u hold on that wallet?
www.btc2doge.com

nice scam, how many people fall for it?

i have the same issue, trying to open a old .wallet file errors out with com.google.dogecoin.store.UnreadableWalletException Could not read wallet. i dont have the .info file

conact us. we may help decrypt the wallet if you have password. although the chance is low. how much doge u hold on that wallet?
www.btc2doge.com

nice scam, how many people fall for it?

yes, many. millions people i guess. i planned all things, i developed multidoge and let all you guys locked and lose coins. yeah. i'm the breaking bad. 🥇 so please be scammed by me ! i beg you. 1 doge is ok. lol

i have the same issue, trying to open a old .wallet file errors out with com.google.dogecoin.store.UnreadableWalletException Could not read wallet. i dont have the .info file

conact us. we may help decrypt the wallet if you have password. although the chance is low. how much doge u hold on that wallet?
www.btc2doge.com

nice scam, how many people fall for it?

yes, many. millions people i guess. i planned all things, i developed multidoge and let all you guys locked and lose coins. yeah. i'm the breaking bad. 🥇 so please be scammed by me ! i beg you. 1 doge is ok. lol

pretending to have developed multidoge now? cmon, your join date and everything is visible on your GitHub profile.

 But when I do "npm install -g mbexport" I get the following...
npm ERR! code EEXIST
npm ERR! path C:\Users\xxxxx\AppData\Roaming\npm\node_modules\mbexport\build\mbexport
npm ERR! dest C:\Users\xxxxx\AppData\Roaming\npm\mbexport.cmd
npm ERR! EEXIST: file already exists, cmd shim 'C:\Users\xxxxxs\AppData\Roaming\npm\node_modules\mbexport\build\mbexport' -> 'C:\Users\xxxxx\AppData\Roaming\npm\mbexport.cmd'
npm ERR! File exists: C:\Users\xxxxx\AppData\Roaming\npm\mbexport.cmd
npm ERR! Remove the existing file and try again, or run npm
npm ERR! with --force to overwrite files recklessly.
npm ERR!      C:\Users\xxxxx\AppData\Roaming\npm-cache_logs\2021-07-14T21_07_50_089Z-debug.log
and when I just go to a command prompt and try mbexport I get....
internal/modules/cjs/loader.js:905
throw err;
^
Error: Cannot find module 'C:\Users\xxxxx\AppData\Roaming\npm\node_modules\mbexport\build\mbexport'
←[90m    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:902:15)←[39m
←[90m    at Function.Module._load (internal/modules/cjs/loader.js:746:27)←[39m
←[90m    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:76:12)←[39m
←[90m    at internal/main/run_main_module.js:17:47←[39m {
code: ←[32m'MODULE_NOT_FOUND'←[39m,
requireStack: []
I noticed that it's trying to find the module at C:\Users\xxxxx\AppData\Roaming\npm\node_modules\mbexport\build\mbexport and my directories seem to end at C:\Users\xxxxx\AppData\Roaming\npm\

in your command prompt try typing:

npm install mbexport

try uninstalling mbexport and then installing it again

npm uninstall -g mbexport

you could also try forcing an npm update in case npm is having problems

npm install -g npm@latest --force

@Evo-Crypto Did you resolve your issue?

@Evo-Crypto Unfortunately I didn't

@ReverseControl sadly no. The wallet extractor works quite well for newer wallets I testet with. But with my old Dogecoin wallet it gave an error and asks for my password. Maybe the .wallet file is broken or I forgot my password.

If you guys eventually find a solution please post it here. I will suggest we close this issue for the time being.
@patricklodder @rnicoll @michilumin I suggest we close this one.

I have powerful machine to wallet file of all versions of bitcoin wallet.
I have recovered many bitcoin wallets. and some of the had corrupt file. for fast response text me in telegram or gmail
Telegram: @walletrecover
<EMAIL_ADDRESS>

i have the same issue, trying to open a old .wallet file errors out with com.google.dogecoin.store.UnreadableWalletException Could not read wallet. i dont have the .info file
do you fixed it? you may reach BTC recovery service on twitter. he helped me on my doge wallet year ago. good luck

I couldn't figure out my situation either. I just gave up after trying to use mbexport and not getting past the error messages I was getting

do you fixed it? you may reach BTC recovery service on twitter. he helped me on my doge wallet year ago. good luck

@ReverseControl sadly no. The wallet extractor works quite well for newer wallets I testet with. But with my old Dogecoin wallet it gave an error and asks for my password. Maybe the .wallet file is broken or I forgot my password.

you need the password to access to it, if you have rough idea of the password, https://www.btc2doge.com/ can help

I couldn't figure out my situation either. I just gave up after trying to use mbexport and not getting past the error messages I was getting

if you still have the file, can contact https://www.btc2doge.com/ for help


Fix the problem caused by Switch Feild on Phone Feild

Fix z-index on Phone field
WHY
field switch was shown above phone field dropdown list
BEFORE - What was wrong? What was happening before this PR?

AFTER - What is happening after this PR?

HOW
How did you achieve that, in technical terms?
by adjust z-index on phone dropdown list to 3 instead of 2
Is it a breaking change?
no
How can we test the before & after?
add switch field next to phone field .

Hey @EGYWEB-Mohamed
Thank you very much for raising the issue and providing the solution.
There is already some css on the field, and I think this should also there with the other css.
I will ping @jorgetwgroup here, so that he can test and provide the fix in the PRO repo.
All the best wishes 🎅

Good luck :)

Hello there @EGYWEB-Mohamed 100% true, this have this visual problem about z-index.
Thanks a lot for this PR, this is made directly in field in this PR: https://github.com/Laravel-Backpack/PRO/pull/111
@pxpm is ready to check and close this issue.
Cheers.

I've just merge the PR in PRO, so we can close this. Had a problem releasing the tag, but once that'll be solved, we're good. Thanks @EGYWEB-Mohamed !


Add property MapPublicIpOnLaunch to subnet

Fixes #51.

The documentation lists MapPublicIpOnLaunch as a Boolean parameter. Is there a reason you chose to pass it as a String instead?

Oh, I will fix that.
On Tue, Jan 19, 2016 at 9:03 AM, T.J. Corrigan<EMAIL_ADDRESS>wrote:

The documentation lists MapPublicIpOnLaunch as a Boolean parameter. Is
there a reason you chose to pass it as a String instead?
—
Reply to this email directly or view it on GitHub
https://github.com/MonsantoCo/cloudformation-template-generator/pull/52#issuecomment-172918770
.

Done. PTAL.


Would it be possible to compare subtitles with the video name instead?

I was wondering if it would be possible to compare the subtitles with the video name so that the correct sub for the episode could be used automatically. For example, E02 in the track title could be matched with the E02 subtitle if all the subs are loaded.

Why would one load all subtitles instead of letting mpv automatically load external subtitles that match the playing file?

Usually the subtitle file names and the video file names don't match up with a tv series for example, so MPV wouldn't be able to load the correct subtitle unless I rename each and every subtitle file to match the video file. Currently I just load all the subtitles and cycle to the one for the episode since its a lot easier than renaming all the subtitle files, but was hoping if there was an easier solution.

This is really outside the scope of this project. The aim of this script is to choose what subtitle to use for the given audio, it is expected that all subtitles apply to the current video. Choosing to load subtitles that actually apply to that file is the job of mpv, the user, or another script. I don't want to over-complicate this script for such a niche use-case. Another script would be ideal, but I don't know of any that fit this specific use-case, and I'm too busy to write one myself.
You may want to look into the fuzzy sub-auto option, which may be enough to fix the mismatch. There are also lots of scripts which allow you to download subtitles for the video online, which might bypass this problem altogether.


Thank you very much for your contribution

I am developing a set of children's popular science content based on mobile VR, and I need a stable Cardboard SDK. Thank you very much for your contribution, and hope to continue to optimize and improve.

Thanks, this means a lot to us. Feel free to create ticket if any issue occurs.


Add missing and facet value code

What

Add missing spec for ElasticsearchPresenter.  …
Managed to miss adding code to this class when trying to add
and_facet_values.
Realised this is because there are no tests around the #document
method. This commit adds a spec to cover the code I added when
implementing facet_groups and facet_values

Add and_facet_values to ElasticsearchPresenter
This field was missing and therefore not getting indexed.

Update spec I got wrong earlier.  …
and_facet_values will never exist in expanded links for the documents.
It should take it's data from facet_values.

Remove redundant method ExpandedLinksPresenter#and_facet_values

Following on from https://github.com/alphagov/rummager/pull/1501
Trello: https://trello.com/c/ovMEU0aL/283-end-to-end-testing-on-integration-of-new-links-based-approach-for-tagging

 End-to-end testing on integration of new links-based approach for tagging


LoadAverage is displayed incorrectly

Page Nodes.
It is necessary to ensure that LoadAverage is calculated as
GetLoadAverage(0) * 100 / sysInfo.GetNumberOfCpus()
Currently, the sorting value is displayed as
GetLoadAverage(0)
ticket to ydb-embedded-ui https://github.com/ydb-platform/ydb-embedded-ui/issues/963

https://github.com/ydb-platform/ydb/pull/6153


mps2_an521: no input to shell from Windows qemu host

Describe the bug
Trying to run a sample with shell functionality and have it work in an emulated ARM machine on a Windows host.
The sample runs and prints the shell prompt, but no input from keyboard is possible.
To Reproduce
Run in Windows command prompt (cmd):
west init
west update
set "GNUARMEMB_TOOLCHAIN_PATH=%ProgramFiles(x86)%\GNU Arm Embedded Toolchain\9 2020-q2-update"
set ZEPHYR_TOOLCHAIN_VARIANT=gnuarmemb
cd zephyr
west build -b mps2_an521 -t run samples\subsys\shell\shell_module -- -DQEMU="%ProgramFiles%\qemu\qemu-system-arm.exe"

Expected behavior
I expect to be able to enter shell commands and have them executed.
Impact
Prevents manual use of shell for testing/dev purposes
Logs and console output
C:\work\hydra\qemu\zephyr>west build -b mps2_an521 -t run samples\subsys\shell\shell_module -- -DQEMU="%ProgramFiles%\qemu\qemu-system-arm.exe"
-- west build: generating a build system
Including boilerplate (Zephyr base (cached)): C:/work/hydra/qemu/zephyr/cmake/app/boilerplate.cmake
-- Application: C:/work/hydra/qemu/zephyr/samples/subsys/shell/shell_module
-- Zephyr version: 2.4.99 (C:/work/hydra/qemu/zephyr)
-- Found west (found suitable version "0.8.0", minimum required is "0.7.1")
-- Board: mps2_an521
-- Cache files will be written to: C:\Users\walter\AppData\Local/.cache/zephyr
-- Found dtc: C:/ProgramData/chocolatey/bin/dtc.exe (found suitable version "1.4.7", minimum required is "1.4.6")
-- Found toolchain: gnuarmemb (C:/Program Files (x86)/GNU Arm Embedded Toolchain/9 2020-q2-update)
-- Found BOARD.dts: C:/work/hydra/qemu/zephyr/boards/arm/mps2_an521/mps2_an521.dts
-- Generated zephyr.dts: C:/work/hydra/qemu/zephyr/build/zephyr/zephyr.dts
-- Generated devicetree_unfixed.h: C:/work/hydra/qemu/zephyr/build/zephyr/include/generated/devicetree_unfixed.h
Parsing C:/work/hydra/qemu/zephyr/samples/subsys/shell/shell_module/Kconfig
Loaded configuration 'C:/work/hydra/qemu/zephyr/build/zephyr/.config'
No change to configuration in 'C:/work/hydra/qemu/zephyr/build/zephyr/.config'
No change to Kconfig header in 'C:/work/hydra/qemu/zephyr/build/zephyr/include/generated/autoconf.h'
-- Configuring done
-- Generating done
-- Build files have been written to: C:/work/hydra/qemu/zephyr/build
-- west build: running target run
[121/126] Linking C executable zephyr\zephyr_prebuilt.elf
Memory region         Used Size  Region Size  %age Used
           FLASH:       54520 B       224 MB      0.02%
            SRAM:       11536 B        16 MB      0.07%
        IDT_LIST:         200 B         2 KB      9.77%
[125/126] To exit from QEMU enter: 'CTRL+a, x'[QEMU] CPU: cortex-m33
C:\Program Files\qemu\qemu-system-arm.exe: warning: nic lan9118.0 has no peer

uart:~$

Environment (please complete the following information):
Windows 10
QEMU version 5.1.92 (v5.2.0-rc2-11843-gf571c4ffb5-dirty)
Additional context
What's the correct way of specifying the Qemu path? Setting the QEMU_BIN_PATH environment variable doesn't seem to work.
We are using the board mps2_an521 because that seems to be the only emulated board with a Cortex-M33 processor. We would really like to emulate an nRF5340.

cmake/emu/qemu.cmake uses find_program to find QEMU, but doesn't specify the .exe suffix. I suspect that is why setting the QEMU_BIN_PATH env variable doesn't work.
After changing line 17 as follows, setting QEMU_BIN_PATH will work:
  NAMES qemu-system-${QEMU_binary_suffix} qemu-system-${QEMU_binary_suffix}.exe

Serial input is working with board qemu_cortex_m3, so this may be a problem with the uart_cmsdk_apb driver not working properly in qemu

FFIW this is not related to Windows.

FFIW this is not related to Windows.


Update deploy.yml

Add class so providers will bid

Ah thank you @Zblocker64! Not sure how that was missed. Is beta2 or beta3 preferable here?
We should add this to _templates/deploy.yml and run ./document.sh which will update the deploy.yml files for all chains. I'm happy to sort this unless you'd like to do it?

Ah thank you @Zblocker64! Not sure how that was missed. Is beta2 or beta3 preferable here?
We should add this to _templates/deploy.yml and run ./document.sh which will update the deploy.yml files for all chains. I'm happy to sort this unless you'd like to do it?

Beta3. That’s the most common type. Needs to be all chains. I’ll let you do it. Thanks!

Closed in favour of #929 - thanks again!


[TASK] Make code open source

Description
Make source code public.
Acceptance Criteria
Final checks completed and source code made public.

Completed!
CC: @bingzhang


Timeline Slider Labels Should Match the Date

The timeline slider label numbers seem to be arbitrary. My screen reader (Voiceover) read a series of large numbers ("18396.17, etc.") rather than the date.

@SteveChab can you please take a look at https://deploy-preview-514--cmu-delphi-covidcast.netlify.app/ I tried to fix it but somehow I cannot access with the windows narrator screenreader the individual tabs, so I don't know how to test it.

Terrific!


Checkpoints

While network is unstable, I'll post some checkpoints, here.
To verify this hashes, type in the wallet-console, the following command:
getblockhash block_number

UPD (02.01.2022): All this checkpoints, was been added in KevaCoin-blockchain, and available there: https://kevacoin-project.github.io/keva_ws/
Namespace id is Safechain (SFC) checkpoints @64775841 or NWJepNRAdk7NSBRZtk8N8rMTi5MDuUEb4b.

Number of mining pools is decreasing. https://zpool.ca removed from this list:

Seems, like dev planning double-spending, after sales of pre-mine:

Last checkpoints from zergpool.com:

getblockhash 400575
--><PHONE_NUMBER>452f80ea9ae30c9e0284da19024dfedb049910c47494ae0791aaaa

getblockhash 406927
--><PHONE_NUMBER>88801cc3c6e38cf73544b1098c71ed22b6edbe9c15ddb8a36da5a0

getblockhash 406927
--><PHONE_NUMBER>88801cc3c6e38cf73544b1098c71ed22b6edbe9c15ddb8a36da5a0

getblockhash 408729
--><PHONE_NUMBER>5747697cdd2456d03a22f3f3b0914ce527ebdafd97d5ca6aa9de16

getblockhash 411216
--><PHONE_NUMBER>19c2fefe98f8154a3f793a2737df357648913ad73ccec17371f8c4

solopools.net enabled safechain, but no any nodes connected, to sync.
Latest checkpoint from solo-pools, before delisting, is available on the safechain-exploere, there, now:

Last checkpoints from https://solopools.net

getblockhash 423407
--> 1256efc573b7dc5d5a04b549c7c8e5f2de86d229dc6ec4c32628de3de8c608bc

Latest checkpoints from https://fpool.eu/site/block?id=804

getblockhash 430455
-><PHONE_NUMBER>9f109a042828c3f27b017ca597657f4b148e6fec4eece84e603cd1

Withdrawals working on https://exbitron.com/, on https://fpool.eu/ chain.
100 SFC sucessfully sent, and included in block 433767 (block hash a580bf48e880cfb58c6468f7f1e26581149ed4fe6b02337d943cb17e1b9600bb). txid: e313482822e40865b43c37ff93cdd003d14a2ca5a266b23477c42600c6db5237: https://fpool.eu/explorer/SFC?txid=e313482822e40865b43c37ff93cdd003d14a2ca5a266b23477c42600c6db5237
Checkpoint:
getblockhash 433767
-> a580bf48e880cfb58c6468f7f1e26581149ed4fe6b02337d943cb17e1b9600bb

Deposits on exibitron.com are working too, with fpool.eu chain. I just sent 200 millions SFC, by 4 transfers, and sucessfully accepted this. Proof: https://username1565.github.io/brainwallet.github.io/#verify?vrAddr=B795acXNPU2U1H7JwMZtSGQBZJPdvkofmG&vrMsg=Deposits on exbitron.com working too%2C with fpool.eu chain.
200 millions was been sent on exbitron.com%2C by 4 transactions%3A

50%2C000%2C000 SFC%3A https%3A%2F%2Ffpool.eu%2Fexplorer%2FSFC%3Ftxid%3Dea1014a9c76b9d8be084777c97a2a10834fe47a3cecd340abe4c4b7c0d59f81d
Block height%3A 433779%2C block hash%3A 5ed08d274becb80faa8b0d2f93e0738a42edecca5eea772039cd4cad66db5c59

50%2C000%2C000 SFC%3A https%3A%2F%2Ffpool.eu%2Fexplorer%2FSFC%3Ftxid%3Dd14fd24346bf0821857c7098c5ef1595f701f215538492026f96b1d3a175c3e3
Block height%3A 433782%2C block hash%3A eeef1894258f7e676134377a651d90dc308ed9574c7d007a4db9c6d84ae868a8

50%2C000%2C000 SFC%3A https%3A%2F%2Ffpool.eu%2Fexplorer%2FSFC%3Ftxid%3D93188bfb24a6eacf090b21d6c432918aad0f1c6b2013af0c98a4fecade1cd1f4
Block height%3A 433783%2C block hash%3A 7db1ea2e012351cb5d2b1a97a7c14023991fa7a5b3d8d16423af11f69100a78e

50%2C000%2C000 SFC%3A https%3A%2F%2Ffpool.eu%2Fexplorer%2FSFC%3Ftxid%3Dbf213a9ddce5783934d597f5d92bb2aa3d24981f9f9af2dfa3d41a6052b2e02a
Block height%3A 433787%2C block hash%3A adf2a9e162d13f5a7d2f97212db9684b4728fe09a2d528ca973ba34d46887961&vrSig=INKickGlb0woBEpwM4EcPLxMnHuv1BWPXYK%2BZYN7BbxyK75ci4NCpnoY0IfZSKwNRlfU8Rp9h7iJ6taIHNytKKo%3D&vrstrMessageMagic=safechain Signed Message%3A

To let anyone verify my digital signature,
I added Safechain (SFC) to my fork of brainwallet.github.io, with this commit: https://github.com/username1565/brainwallet.github.io/commit/50f7e65785ba89347a8b81ec9c5c4f30ac7dc181 Now Safechain afailable there: https://username1565.github.io/brainwallet.github.io

Last checkpoints from fpool.eu:

getblockhash 433834
-> 692cf781e75db621a08ba3af3c760c977931fded5b29260c8ff6d8c0b2851a32

Last checkpoints from fpool.eu:

getblockhash 441531
-> 0000000a936d7fc939c368338114c72c03cc99803b22167e7ddecb61b1375aa4

9 active connections:

Last checkpoints from fpool.eu:

getblockhash 475281
-><PHONE_NUMBER>7547a9c8f6f195b870b37bc652a18f62d288b56d3d5cd77653359d

last checkpoints from fpool.eu

getblockhash 484758
-> 3df9275c556f09bb7e30a24beb240a9ccfc97f5c293c47a114a003aa4b250f25

last checkpoints from fpool.eu:

getblockhash 517188
-> 761efd43f78a4c2e72f3a810fd4f2f21a0b8a9ede7e057057dd652205992fd55

last checkpoints from fpool.eu:

getblockhash 694653
-> 4d50371c6706e546bc534c445f3bac575d00aeef211a6d50c42b1542273c045d


Deployment's yaml contains wrong environment variable name for the SA's password

The deployment has wrong name for the SA's password environment variable.
It is using:

name: MSSQL_SA_PASSWORD

however, the 2017 image expects the variable to be named SA_PASSWORD. Without this change the server terminates with the error:
ERROR: Unable to set system administrator password: Password validation failed. The password does not meet SQL Server password policy requirements because it is too short. The password must be at least 8 characters..

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

ID: 04922c04-a2da-186c-5946-1cc4d9a7447b
Version Independent ID: 6e490885-cdd7-079a-0b3a-4688616b0954
Content: Deploy a SQL Server container with Azure Kubernetes Services (AKS) - SQL Server
Content Source: docs/linux/tutorial-sql-server-containers-kubernetes.md
Product: sql
Technology: linux
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@mstavrev -- thank you for your feedback.  I am looking into this document.

@MikeRayMSFT -- Mike, please review the above-mentioned PR 13894.


Add support for command COMMAND GETKEYSANDFLAGS

Add support for the COMMAND GETKEYSANDFLAGS command in DiceDB similar to the COMMAND GETKEYSANDFLAGS command in Redis. Please refer to the following commit in Redis to understand the implementation specifics - source.
Write unit and integration tests for the command referring to the tests written in the Redis codebase 7.2.5. For integration tests, you can refer to the tests folder. Note: they have used TCL for the test suite, and we need to port that to our way of writing integration tests using the relevant helper methods. Please refer to our tests directory.
For the command, benchmark the code and measure the time taken and memory allocs using benchmem and try to keep them to the bare minimum.

Hi , Can I give it a try ?

Hi @arpitbbhayani , I am excited to work on this issue. Can you please assigned it to me?

Hi @arpitbbhayani, could you please assign this to me?

Hi @arpitbbhayani I am also interested to work on this issue.

Hi , Can I give it a try ?

Assigned

hi @YuvrajGosain, are you still working on this issue? If not, Can I pick this up?

Hi @anchalsinghrajput , yes i am working on this, will be raising the PR in next 2-3 days.

Hello @YuvrajGosain,
There has been no activity on this issue for the past 5 days.
It would be awesome if you keep posting updates to this issue so that we know you are actively working on it.
We are really eager to close this issue at the earliest, hence if we continue to see the inactivity, we will have to reassign the issue to someone else. We are doing this to ensure that the project maintains its momentum and others are not blocked on this work.
Just drop a comment with the current status of the work or share any issues you are facing. We can always chip in to help you out.
Thanks again.

Hi @arpitbbhayani , pending to write test cases, will raise the PR by tomorrow.

#1035 , Here is the PR this . @JyotinderSingh Can you please help review.

Hi @JyotinderSingh, @AshwinKul28 , @apoorvyadav1111 , can someone pls help review this PR.

Hello @YuvrajGosain,
There has been no activity on this issue for the past 5 days.
It would be awesome if you keep posting updates to this issue so that we know you are actively working on it.
We are really eager to close this issue at the earliest, hence if we continue to see the inactivity, we will have to reassign the issue to someone else. We are doing this to ensure that the project maintains its momentum and others are not blocked on this work.
Just drop a comment with the current status of the work or share any issues you are facing. We can always chip in to help you out.
Thanks again.

@YuvrajGosain are you still working on this issue?


[Web Portal] change command section's default comments to placeholder

change command sections's default command to placeholder
add shell syntax highlighting
change task role section's scroll area
tweak command section's default height

Coverage increased (+9.4%) to 60.653% when pulling c12b2b50fe273594fe0a6bb70197d082c07bc181 on qinsu/portal/placeholder into 6757ab86c3f3a9245f2a5fc7f5a1d50a97684453 on pai-0.14.y.

Coverage increased (+9.4%) to 60.653% when pulling c12b2b50fe273594fe0a6bb70197d082c07bc181 on qinsu/portal/placeholder into 6757ab86c3f3a9245f2a5fc7f5a1d50a97684453 on pai-0.14.y.


Can't set liveness-ttl flag using JETSTREAM_LIVENESS_TTL environment variable

Version: https://github.com/bluesky-social/jetstream/pkgs/container/jetstream/307057313?tag=sha-2aacb5885aa704f70e79dc1d09066e67271d5363
Steps to reproduce:

Set JETSTREAM_LIVENESS_TTL=30s in your environment
Run jetstream against an event stream that doesn't send any events (eg. a PDS running locally)

Expected behaviour:
Jetstream shuts down after 30 seconds.
Actual behaviour:
Jetstream shuts down after 15s and logs the message:
no new events in last 15 seconds, shutting down for docker to restart me

Additional context
This appears to be expected behaviour of urfave/cli. The default value used to be able to be overridden by an env var but that was "fixed" in https://github.com/urfave/cli/pull/1528.

Oh, actually maybe it's not a bug with jetstream actually. the option isn't available on the version of jetstream I'm using but it's the latest dockerfile that has been published.
Is ghcr not up to date?


Categorizing receipts

I quickly went through the criteria for reimbursements requests and I thought that congress people may request reimbursements of a given nature but actually present a receipt of another.
I think it could be of some help to categorize the receipts and then check against the information they provided to verify whether they match or not with the criteria.

Closed in favor of #170


Rendering error

When running DQN with --display option getting the following error
Traceback (most recent call last):
File "main.py", line 66, in
tf.app.run()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 43, in run
sys.exit(main(sys.argv[:1] + flags_passthrough))
File "main.py", line 61, in main
agent.train()
File "/home/savvai/Documents/DQN-tensorflow/dqn/agent.py", line 40, in train
screen, reward, action, terminal = self.env.new_random_game()
File "/home/savvai/Documents/DQN-tensorflow/dqn/environment.py", line 28, in new_random_game
self.new_game(True)
File "/home/savvai/Documents/DQN-tensorflow/dqn/environment.py", line 24, in new_game
self.render()
File "/home/savvai/Documents/DQN-tensorflow/dqn/environment.py", line 60, in render
self.env.render()
File "/usr/local/lib/python2.7/dist-packages/gym/core.py", line 174, in render
return self._render(mode=mode, close=close)
File "/usr/local/lib/python2.7/dist-packages/gym/envs/atari/atari_env.py", line 119, in _render
from gym.envs.classic_control import rendering
File "/usr/local/lib/python2.7/dist-packages/gym/envs/classic_control/rendering.py", line 23, in
from pyglet.gl import *
File "/usr/local/lib/python2.7/dist-packages/pyglet/gl/init.py", line 236, in
import pyglet.window
File "/usr/local/lib/python2.7/dist-packages/pyglet/window/init.py", line 1817, in
gl._create_shadow_window()
File "/usr/local/lib/python2.7/dist-packages/pyglet/gl/init.py", line 205, in _create_shadow_window
_shadow_window = Window(width=1, height=1, visible=False)
File "/usr/local/lib/python2.7/dist-packages/pyglet/window/xlib/init.py", line 163, in init
super(XlibWindow, self).init(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/pyglet/window/init.py", line 505, in init
config = screen.get_best_config(template_config)
File "/usr/local/lib/python2.7/dist-packages/pyglet/canvas/base.py", line 161, in get_best_config
configs = self.get_matching_configs(template)
File "/usr/local/lib/python2.7/dist-packages/pyglet/canvas/xlib.py", line 179, in get_matching_configs
configs = template.match(canvas)
File "/usr/local/lib/python2.7/dist-packages/pyglet/gl/xlib.py", line 29, in match
have_13 = info.have_version(1, 3)
File "/usr/local/lib/python2.7/dist-packages/pyglet/gl/glx_info.py", line 89, in have_version
client = [int(i) for i in client_version.split('.')]
ValueError: invalid literal for int() with base 10: 'None'

Hey, I have the same problem. Digging a bit into the code, I discovered that for some reason glXGetClientVersion returns None when trying to render AIGym's world. I don't know exactly why this happens, but my guess is that it has something to do with tf session initialization.
While fiddling with the code I also found a (very dirty) hotfix for this problem - you just need to initialize a dummy environment before you initialize the tf session. The simplest way to this is to edit main.py a little bit:
add
import gym
at the top, and, inside main, create a new g_env variable:
def main(_):
  gpu_options = tf.GPUOptions(
      per_process_gpu_memory_fraction=calc_gpu_fraction(FLAGS.gpu_fraction))

  g_env = gym.make('Breakout-v0')
  g_env.render()

  with tf.Session(config=tf.ConfigProto(gpu_options=gpu_options)) as sess:

I know this is really weird and does not solve the issue, but it might get you going until devs look into this.

For me, just add:
import gym
g_env = gym.make('Breakout-v0')
g_env.render()
at the very top of main.py, before import tensorflow as tf. I believe there is some conflict between glx with NVidia.


[UIE-160] Check for missing cache files in CI

Every once in a while, a PR updates dependencies without updating the Yarn cache. For example, most recently, #4680 (and the associated cleanup, #4704). This causes confusion for developers when they later run yarn install and find untracked files and often results in noise in PRs if the cache files are committed along with other unrelated changes.
This adds to the build CI job a check for missing cache files. This will prevent merging a PR unless the Yarn cache is up to date.
Note, that we used to effectively have this check by running yarn install --immutable-cache in CI. However, we had to remove that when we added Pact because the cache file for one of the Pact packages exceeds GitHub's file size limit (see #4138).

Note, build is expected to fail before #4704 is merged.

@nawatts how will this not fail in the pact case?

how will this not fail in the pact case?

Since the cache file for @pact-foundation/pact-core is ignored in .gitignore, it won't show up as an untracked file.
https://github.com/DataBiosphere/terra-ui/blob/3215fec9d52a9481e9d60ff5e0afef475e0e3ff5/.gitignore#L53-L54

@nawatts the missing cache files were added with https://github.com/DataBiosphere/terra-ui/pull/4706, so you should be able to update this branch and proceed.

Looks like #4706 only added two out of the three missing cache files. Running yarn install still generates an untracked file<EMAIL_ADDRESS>

Added the remaining cache file to this PR in https://github.com/DataBiosphere/terra-ui/pull/4708/commits/52ee50e56e7f7626f36ba82f57e0526978bcca07.

Looks like #4706 only added two out of the three missing cache files. Running yarn install still generates an untracked file<EMAIL_ADDRESS>
I thought that one was the pact file that we intentionally didn't check in. If not, perhaps just add it with this PR.

The one that we intentionally omit is for @pact-foundation/pact-core, not @pact-foundation/pact. Since the pact-core cache file is ignored, it never appears as an untracked file.


GEODE-47: Add CLI option to run locators in foreground

add a command line option to start locators in the foreground. The gfsh command will not return until the locator stops running.

add unit tests for new CLI option

add distributed unit tests for new CLI option

add unit test for new Builder options

add CLI Strings for foreground option

add jmh compile dependency to fix intelliJ build in case it isn't inherited from main

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

My preference would be to NOT pollute stdout with all the log data and keep
the logs going to the file they normally would. Otherwise, I will end up
having to redirect stdout to somewhere and can't see what it says without
tailing that file. It would also potentially make it more difficult to
actually gather logs and stats files via the gfsh command for doing so
because we won't know where they are.
--
Mike Stolz
Principal Engineer, GemFire Product Lead
Mobile: +1-631-835-4771
On Wed, Nov 28, 2018 at 6:54 PM Jens Deppe<EMAIL_ADDRESS>wrote:

@jdeppe-pivotal requested changes on this pull request.
I have a couple of points on the PR.

Ctrl-C should terminate the process - currently it does not.
I would strongly argue that if the process is started in the
foreground that all logs should then also be written to stdout. I believe
that the original intent was for this to run in a container and keep the
container 'up' (when processes get launched in the background the container
would simply shut down immediately). It doesn't help that logs get written
to a separate file. Essentially, with foreground the process should
act like any other CLI utility and produce relevant output on stdout.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/geode/pull/2914#pullrequestreview-179610843,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASblIBT2_udiuf-RDC2PVawj-kBSUaxKks5uz0xdgaJpZM4Y4ndy
.

I agree with @jdeppe-pivotal that a process started in the foreground should be logging to stdout.  The intent of the command is to make it easier to run geode in containers and allow the "platform" to collect and manage logs.  For example, see the docker logs command:  https://docs.docker.com/engine/reference/commandline/logs/#usage.
Adding to the Jen's comment on ctrl-c, the process to be stopped is the geode locator or server, not the gfsh process.

Closing this until we have further discussions about the requirements and implementation


Location is more accurate when other apps are open

🐛 Bug Report
For my app (a golf app that measures shot distances) I'm using LocationAccuracy.best however the accuracy seems to be around 20m accuracy and doesn't update that frequently (~every 5 seconds).
However, when I open another golf app that also requests location permissions (e.g. The Grint), all of a sudden the location updates in my app are super accurate (within 1m) and update every second.
As soon as I close the other app, the updates in my app go back to being ~20m accuracy and ~5s updates.
How is it that other apps are able to request such high precision updates?
Note that this is only on Android, on iOS it's very accurate even without other apps open.
Expected behavior
Reproduction steps

Open an app on Android and request an location stream with LocationAccuracy.best.
Notice that updates are ~20m accuracy and jump around.
Open an app like The Grint and start GPS tracking.
Go back to the original app while The Grint is still requesting updates in the background and notice that the location updates in the app are super accurate ~1m accuracy.
Close The Grint app.
Go back to the original app and notice that location updates are back to ~20m.

Configuration
This is the configuration for the Android location settings:
locationSettings = AndroidSettings(
          accuracy: LocationAccuracy.best,
          distanceFilter: 1,
          forceLocationManager: true,
          intervalDuration: const Duration(seconds: 1),
          //(Optional) Set foreground notification config to keep the app alive
          //when going to the background
          foregroundNotificationConfig: const ForegroundNotificationConfig(
            notificationText: "Your location will continue to update.",
            notificationTitle: "Location updating in background",
            enableWakeLock: true,
          )
      );

Setting up the stream:
_positionStream = Geolocator.getPositionStream(locationSettings: locationSettings)
        .listen((Position? position) {
          print("Position updated: ${position?.latitude}, ${position?.longitude}");
          if (position != null) _myPosition = LatLng(position.latitude, position.longitude);
          else position = null;
          setState(() {});
        });

Version: 9.0.2
Platform:

[ ] :iphone: iOS
[x] :robot: Android

I tried another package and it is much more accurate https://pub.dev/packages/location

Dear @odiggity25,
If you set forceLocationManager to true you basically force the app to use the Legacy LocationManager for position updates.  This is based either on GSM network for coarse positioning) and GPS based (fine) location. However, if you are for example testing inside, it might be hard to find an accurate position. If you set forceLocationManager to false, the app basically uses network and GPS and it combines them. In addition it uses internet (based on ISP data) and WiFi (if supported by the router) positioning as well. Therefore, inside you might expect a more accurate position. Why TheGrind influences your app is that it basically works together, if one app has a "better" location, other apps might profit from it.
I'll close this issue since I suspect that this is expected behaviour. Feel free to reopen when needed.

@TimHoogstrate ohhh, gotcha. Sorry, I misread the documentation when I investigated this parameter. I only added this parameter because it is in the setup guide here https://pub.dev/packages/geolocator. I would suggest maybe removing it from the guide if it's not something most people should be using (I'm guessing most people want to use as many options as possible).

@orriesquabbit bruh did you fixed it, if yes can you please tell me the settings config that you finalized?

just remove forceLocationManager: true, from the settings. So:
locationSettings = AndroidSettings(
accuracy: LocationAccuracy.best,
distanceFilter: 1,
intervalDuration: const Duration(seconds: 1),
//(Optional) Set foreground notification config to keep the app alive
//when going to the background
foregroundNotificationConfig: const ForegroundNotificationConfig(
notificationText: "Your location will continue to update.",
notificationTitle: "Location updating in background",
enableWakeLock: true,
)
);

Okay sure.. I'll try


GPS multi support

Do not merge yet. This adds multi-module support for GPS.

Bit rotting, deleting.


How to join consumer group so Zookeeper can restart dead consumer

The examples of a consumer providing a heartbeat to Zookeeper online has been in Java.
Is there an example of how to create a C++ consumer app that can be monitored and healed by zookeeper?

It seems people took a great effort to remove the dependency towards zookeeper.
https://www.confluent.io/blog/removing-zookeeper-dependency-in-kafka/
https://blogs.apache.org/kafka/entry/what-s-new-in-apache5
Thus I don't think the underlying C library, i.e. librdkafka (https://github.com/edenhill/librdkafka) would provide any support for zookeeper related stuff.

@kenneth-jia
Thank you.
How does a consumer group get healed in this new approach?

I don't quite understand your question, -- what does "get healed" mean? The failed consumer would not be restarted (unless you add additional logic in your application to manage the process state), while the partitions (which were assigned to the failed consumers) would be rebalanced to the other consumers (within the same consumer.group).


Reflect is not defined

After adding the plugin I have the error in terminal
 ReferenceError: Reflect is not defined
....

I'm using this plugin for correct extending Error class, and when I try to throw an error I have error "Reflect is not defined"
After a quick investigation I've noticed that you are using Reflect
any ideas how to fix it?

Currently the expectation is that you'd be loading babel-polyfill or using babel-plugin-transform-runtime. It's possible that the usage of Reflect may change in the future but hard to say.


Functional tests witness messages follow up

Fixes #1032, Follow up to #1029
Removes the with_witness named argument from CBlock.serialize(). p2p_segwit requires non-witness serialization for computing the proper block weight, which is why an explicit serialize_without_witness has been added (this is the same as existing design on CTransaction).
Drops msg_witness_blocktxn in favor of msg_blocktxn, which now uses serialization with witness by default and unconditionally.

utACK f2cbccb
So we can not completely get rid of serialize_without_witness and alike... So sad


Support Request: Homebridge to Home Assistant to HomeKit

Describe Your Problem
I can see all the sensors in home assistant no problem but for some reason when using homekit to my iphone all the sensors show no response. I am not sure if I am doing something wrong or missing a step. I do nothing different with my other homebridge to home asssistant to homekit apps. Otherwsie everything is working perfectly~!
Relevant log output
No response
Config for homebridge-switchbot
No response
Screenshots
No response
Device & Model
outdoor tempature sensor
Node.js Version
v20.9.0
NPM Version
1.7
Homebridge Version
1.7
Homebridge SwitchBot Plugin Version
@switchbot/homebridge-switchbot v2.10.1
Homebridge Config UI X Plugin Version
No response
Operating System
Ubuntu Docker

Wait so are you using home assistant or this plugin to push to HomeKit?
And if it is this plugin, which devices? Can you got to the plugins setting's device tab and can you see the devices there?

So i use this plugin in homebridge
I connect the homebridge to home assistant via the homebridge kit plugin

The sensors work no problem in HA

But when using on my home app on the phone I get no response.

I do the same workflow with homebridge ring setup, so i think the issue is at this app level.

How are you pushing them to HomeKit (Home App) if you have the bridge in HomeAssistant?
Why not just use the SwitchBot BLE or Cloud integration in HomeAssistant? And put this plugin directly into HomeKit (Home App)

switchbot Cloud HA plugin doesn't have many supported devices it seems.

switchbot Cloud HA plugin doesn't have many supported devices it seems.

Still doesn't answer this question:

How are you pushing them to HomeKit (Home App) if you have the bridge in HomeAssistant?

The built in homekit homebridge apps

Well then this isn't an issue with the plugin, but with how home assistant is handling the services & characteristics that this plugin is sending over.
If you add this plugin into HomeKit directly and are still having issues, I can look into it further.
I think best route is to close this issue, open an issue with HomeAssitant, requesting support for other device. Directing them to this plugin if they need help.

Will do thank you for taking a look.


Cache hash.File result

About ~5% of CPU time of Parca Agent are spent in hashing object files to prepare for the debuginfo Exists gRPC call. This is even though files are extremely unlikely to change. I think we should only rehash if the mtime of the file changes. Yes, it's possible to modify the mtime, but that would mean that a user is shooting themselves in the foot by doing so, so I think that tradeoff is worth it.

Ah! Great timing :D Was thinking about this yesterday for a similar problem we are going to face for the DWARF unwinder. The current line of thought I have is to use the inode of the file. It should be unique per FS and machine in Linux. So we could maybe use this in the Agent to avoid the rehashing here?
Was going to use this for the unwinder, but I think that here it could be slightly better than the mtime in case there are clock drifts or as you say if it gets modified by users or some other problem

inode does not seem robust to me, it does not change if the file is modified and if a file is deleted, the next created could re-use the inode

Oh, you are right! Thanks for bringing this up. I think I got this wrong because btrfs, the filesystems I use, does not recycle them, but this is an implementation detail for each filesystem!

I think we have a caching mechanism now
https://github.com/parca-dev/parca-agent/blob/ba2df5bc9b97b6434a5509dfa586cbbadf7d8cc3/pkg/debuginfo/manager.go#L321-L356
I am closing this. Please don't hesitate to re-open it if it doesn't what you expected.


cmake in the most recent version give errors

I am not an expert in cmake, but here is the error message:
zhouke@europa[~/src/bamtools]$ mkdir build
zhouke@europa[~/src/bamtools]$ cd build
zhouke@europa[~/src/bamtools/build]$ cmake ..
CMake Error at CMakeLists.txt:20 (message):
BamTools requires an out of source build.
$ mkdir build
$ cd build
$ cmake ..
$ make

(or the Windows equivalent)
Call Stack (most recent call first):
CMakeLists.txt:24 (ensure_out_of_source_build)
-- Configuring incomplete, errors occurred!

I have the same problem and the solutions proposed don't work.
/verona/nobackup/biostat/software/bamtools-master/B$ cmake ..
CMake Error: Could not open file for write in copy operation /src/toolkit/bamtools_version.h.tmp
CMake Error: : System Error: No such file or directory
CMake Error at toolkit/CMakeLists.txt:38 (configure_file):
  configure_file Problem configuring file

Could the install process be made more robust ? Is it a requirement to use git clone ? I simply clicked the Download ZIP button, and unzipped the archive.
/verona/nobackup/biostat/software/bamtools-master$ cat /etc/debian_version
7.8

This still doesn't work, I keep getting Could not open file for write in copy operation /src/....
This is on a virgin git clone, using cmake <IP_ADDRESS> on CentOS 6.6:

$ git clone https://github.com/pezmaster31/bamtools.git
Initialized empty Git repository in ~/src/bamtools/.git/
remote: Counting objects: 4059, done.
remote: Total 4059 (delta 0), reused 0 (delta 0), pack-reused 4059
Receiving objects: 100% (4059/4059), 2.77 MiB | 2.21 MiB/s, done.
Resolving deltas: 100% (2527/2527), done.
$ cd bamtools/
$ git show
commit 02c2be8866f9e43b1b5f0f80e4daae5ff352c06b
Merge: 71392a2 df972a0
Author: Derek Barnett
Date:   Wed Jun 17 23:07:16 2015 -0400
Merge pull request #105 from mjhsieh/patch-1

one extra unicode string in the beginning of file

$ cd src/
$ mkdir build
$ cd build
$ cmake ..
-- The C compiler identification is GNU 4.4.7
-- The CXX compiler identification is GNU 4.4.7
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
CMake Error: Could not open file for write in copy operation /src/toolkit/bamtools_version.h.tmp
CMake Error: : System Error: No such file or directory
CMake Error at toolkit/CMakeLists.txt:38 (configure_file):
configure_file Problem configuring file
CMake Warning (dev) in CMakeLists.txt:
No cmake_minimum_required command is present.  A line of code such as
cmake_minimum_required(VERSION 2.8)

should be added at the top of the file.  The version specified may be lower
if you wish to support older CMake versions for this project.  For more
information run "cmake --help-policy CMP0000".
This warning is for project developers.  Use -Wno-dev to suppress it.
-- Configuring incomplete, errors occurred!
See also "~/src/bamtools/src/build/CMakeFiles/CMakeOutput.log".

To my experince, you should use the command "cmake ..bamtools/" please remember, do not use "cmake ..bamtools/src". That is, NO scr sub-directory is needed. Then, everything is going good!

I'm hoping to build bamtools with cmake version 3.3.1 on centos-release-6-7.el6.centos.12.3.x86_64  but received the error message below.
'CMake Error at CMakeLists.txt:40 (message):
....
CMakeLists.txt:44 (ensure_out_of_source_build)'
I've tried placing the build folder at the same level as the bamtools clone but received the same error. I tried replacing the text in the CMakeLists.txt file with
"if( ${CMAKE_SOURCE_DIR} STREQUAL ${CMAKE_BINARY_DIR} )
message( FATAL_ERROR "Out of source check failed." )
endif()"
but ended up with the same issue.
cmake -DCMAKE_INSTALL_PREFIX=../bamtools
-- The C compiler identification is GNU 7.2.0
-- The CXX compiler identification is GNU 7.2.0
-- Check for working C compiler: ../miniconda3/bin/cc
-- Check for working C compiler: ../miniconda3/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: ../miniconda3/bin/c++
-- Check for working CXX compiler: ../miniconda3/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at CMakeLists.txt:32 (message):
Out of source check failed.
-- Configuring incomplete, errors occurred!
See also "/home/devealeb/bamtools/build/CMakeFiles/CMakeOutput.log".
Any advice on how to resolve this would be sincerely appreciated. Thanks, Brian

For those having this issue, i tried the solution above placing build folder at the same level as the bamtools clone but kept getting the same error UNTIL I also deleted CMakeCache.txt from the bamtools clone folder. After deleting CMakeCache.txt i could build and install bamtools.

Thanks. Deleting the CMakeCache.txt has just worked for me


Download artifacts from S3 only if required

Previously the brkt-cli would always try to download the Metavisor
artifacts from S3 everytime an encrypt/update command is run, even
if the command fails subsequently do to issues with the vCenter/ESX
server and/or the specified parameters. With this change, we:

Check if artifacts were already downloaded
Check if the VMDK hash matches the manifest file
Only when one or more of the files are not found or the manifest hash
does not match the VMDK, do we try to download the artifacts from S3.

Additionally a hidden --no-cleanup option has been provided which can
be used in case the user does not want the downloaded artifacts to be
deleted after the encryption/update completes successfully. This is
especially useful in the case where the same artifact is required to
encrypt/update multiple images and eliminates the need to download
the artifacts from S3 for each of the encrypt/update operations.

Please take  a look again

LGTM


BoostrapTable doesn't re-render using<EMAIL_ADDRESS> but does using 0.1.15

Hi guys,
Following the related issue https://github.com/react-bootstrap-table/react-bootstrap-table2/issues/444
Example code https://github.com/react-bootstrap-table/react-bootstrap-table2/issues/444#issuecomment-410495815 (copy below) works with react-bootstrap-table-next at version 0.1.15, but not with 3.0.0
class Table extends React.Component {
  constructor(props) {
    super(props);
    this.state = { products };

    this.columns = [{
      dataField: 'id',
      text: 'Product ID'
    }, {
      dataField: 'name',
      text: 'Product Name',
      filter: textFilter(),
      formatter: cell => `% ${cell} %`
    }, {
      dataField: 'price',
      text: 'Product Price',
      formatter: (cell, row) => {
        return (
          <button onClick={ this.handleTagClick(row.id) }>Click</button>
        );
      }
    }];
  }
  /* eslint no-param-reassign: 0 */
  handleTagClick = rowId => () => {
    const newProducts = this.state.products.map((row) => {
      if (row.id === rowId) {
        row.name += row.name;
      }
      return row;
    });
    this.setState({ products: newProducts });
  }

  render() {
    return (

        <BootstrapTable
          keyField="id"
          data={ this.state.products }
          columns={ this.columns }
          filter={ filterFactory() }
        />
        { sourceCode }

    );
  }
}

Am I missing something here ?
Thank you

I setup an example here: https://codesandbox.io/s/8km0ly74m0
Working:

0.1.15

Not working:

= 1.0.0

@zbitouzakaria please follow this to update/mutate data in correctly or not you will make react-bootstrap-table recognize your data is not change.  Too many people ask the same question. So please I will ask developer to read above doc and figure out how shouldcomponentupdate work and why we need immutable.js sometime.
ping me if you still dont know how to mutate data, thanks

Got it, thanks! Here is a working example for those interested:
https://codesandbox.io/s/8km0ly74m0


Using astro-i18n with new content folder

Hi, first great job, I'm enjoying using the library and would be potentially keen to contribute.
I have successfully published a website using it, but now that I want to add a blog section, I wonder how to combine this lib with the new Astro content folder.
if simply using it naively with a [slug].astro page and my md files in the content folder we will endup having duplicated route in each language.
What I'm expecting to have:

/blog/article-1-in-english. // default language
/fr/blog/article-1-in-french // second language

But what is being generated let you access:

/blog/article-1-in-english.
/blog/article-1-in-french.
/fr/blog/article-1-in-english
/fr/blog/article-1-in-french

Which is normal considering how both systems works. So I'm wondering, at which level could we make sure this does not happen ? I'm not an Astro expert yet but if something could make it happen I'd be happy to try and implement it.
Maybe by giving the possibility to ignore some page in the i18n.config it could helps 🤔

Hey,
I ran into the same issue, so I created a repo to demonstrate it
https://github.com/oliviergraziano/astro-i18n-content-incompatibility
Happy to help debugging this if needed :)

Hey @agonist, I made my website using Astro content folder and this very plugin for translating everything (french and english).
You can check out how I did it here.
The logic behind single post translation is here.
I'm not saying this is the proper way but it works so far :)

@narduin sounds great ! I will take a look thanks.
didn't had time to work on it again but if I'm still running into the issue I'll post a repo.


chore: bump windows cni version to v<IP_ADDRESS> for podsubnet and overlay

What type of PR is this?
/kind feature
What this PR does / why we need it:
update windows cni version to v<IP_ADDRESS> for podsubnet and overlay
Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 79.119%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2228

Relevant Lines:
2816

💛  - Coveralls


Update Composer dependencies (2022-09-08-00-07)

Visual regression test passed!
View the visual regression test report


[BUG]: Drizzle SQLITE magic sql operator not mapping params

What version of drizzle-orm are you using?
0.35.3
What version of drizzle-kit are you using?
0.26.2
Describe the Bug
Hello Drizzle team,
We have an issue with our sql operator. We have some complex queries for a local database and have found a problem with the sql operator.
This problem forces us to use sql.raw(), since this does not escape strings we have problems with filepaths with a single quote in them e.g. _Users/MyName/directory/Boris'file_path.txt
Using the sql operator for SQLITE does not map query params correctly and fails to run.
Our call looks like this:
const queryStr = sql`INSERT INTO ${tableName} (${colsString}) VALUES (${valuesString})`;

try {
	result = db.all(queryStr);
	console.log(result);
} catch (err) {
	console.log('Error in executing insertRow:');
	console.error(err);
}

Our output:
Output of error varies from something abstract like:
_SqliteError: near "?": syntax error_

To this:
_Failed to run the query 'INSERT INTO ? (?) VALUES (?)'_

Please let us know if this is something on our end or yours, we need to make decision about jumping ship to another solution while we still can without too much work.
Expected behavior
We expect the query to run without error.
Environment & setup
Electron, SQLITE, Vite & Electron Forge.

There's a couple of things you need to change about your query to make it work:

tableName should instead be the instance of the SQLite table. Alternatively, you can do sql.raw(tableName) as well.
colsString should be replaced with an array of columns mapped with sql. So something like:
sql.join(columns.map((column) => sql.raw(column.name), sql.raw(', '))).

Also, is there any particular reason not to use the db.insert() syntax instead?

Also, is there any particular reason not to use the db.insert() syntax instead?

We are arbitrarily adding columns and values, syntactically its easier to go pure SQL and generate a string query rather than db.insert for this although we can try that out. We have more complex queries where it is more difficult to use the ORM than db.insert or db.delete and this issue will still be prvalent there.
But since it is supposed to be just an sql string, should it not accept straight up SQL?


VolumeSurface contour display style

As mentioned in #236. Added a contour option to volume surfaces that produces sparser meshes. I've also removed setAttribute method from the ContourBuffer class - I think this is redundant now (I've tested mol surfaces and volume surfaces with this removed, is there anything else that would use it?)

thank you


Prevent editor sticky scroll from being set in workspace

Related: https://github.com/microsoft/vscode/issues/199394
Repro:
Set "editor.stickyScroll.enabled": false (and related settings) in .vscode/settings.json, it should not work as it's purely a user preference. A project should not be able to override it.

We have many settings that could be categorized as 'user preferences'. Auto save, font size, color theme, showing stick scroll in the terminal (currently enabled in the vscode repo)
We don't need to restrict users here. If a workspace wants to set such a setting, it should be able to. If the users of that workspace don't like that setting, then they can file an issue against that workspace. But maybe that workspace has only one user.

@aeschli I have a very different opinion here, just because some project does not like or does like a feature, they should not restrict you from using it or configure how it's used. The same applies to ones you listed, although color theme is arguable. A perfect example of this is that I would be unhappy if autosave was enabled in VS Code's repo and I would end up having a permanent local change in my .vscode/settings.json that I need to avoid committing.

If the users of that workspace don't like that setting, then they can file an issue against that workspace

Clearly the repos may not accept such a change and there's nothing the user can do about it but the frustrating workaround above.

showing stick scroll in the terminal (currently enabled in the vscode repo)

This was always going to be turned off with and was just a hack to help self hosting https://github.com/microsoft/vscode/issues/197734


CSRF Route Not Found

better-auth v0.3.3
I have setup the app based on the docs, with the following on a Hono server:
export const auth = betterAuth({
	baseUrl: 'http://localhost:3000',
	basePath: '/api/v1/auth',
	database: kyselyDialect,
	emailAndPassword: {
		enabled: true,
		password: {
			hash: (password) => Bun.password.hash(password),
			verify: (hash, password) => Bun.password.verify(password, hash),
		},
	},
});

app.on(['POST', 'GET'], '/api/v1/auth/**', (c) => auth.handler(c.req.raw));

On the client side:
import { createAuthClient } from 'better-auth/client';

export const authClient = createAuthClient({
	baseURL: 'http://localhost:3000/api/v1/auth',
});

But getting the following error on the client:
BetterAuthError: CSRF route not found. Make sure the server is running and the base URL is correct and includes the path (e.g. http://localhost:3000/api/auth).
      at /xxx/node_modules/better-auth/dist/client.js:7:5

and the following log from the server:
<-- GET /api/v1/auth/csrf
--> GET /api/v1/auth/csrf 404 1ms

Please let me know where I might have gone wrong?

I haven't used Hono before but have you tried using?
app.on(['POST', 'GET'], '/api/v1/auth/', (c) => auth.handler(c.req.raw));

The way to create wildcard routes in Hono is thru * pattern
'app.on(['POST', 'GET'], '/api/v1/auth/*', (c) => auth.handler(c.req.raw));' is correct
https://hono.dev/docs/api/routing

Looks like app.on(['POST, 'GET'], ....) does not work with wildcards on Hono.
I used
app.get('/api/auth/*', (c) => auth.handler(c.req.raw));
app.post('/api/auth/*', (c) => auth.handler(c.req.raw));

And it worked!
But it still isn't working with custom paths. Below does not work and gives a 404
app.get('/api/v1/auth/*', (c) => auth.handler(c.req.raw));
app.post('/api/v1/auth/*', (c) => auth.handler(c.req.raw));

If this is the case then should the docs be updated to change the current example to use the separate HTTP method approach (app.get(...) & app.post(...)) instead?

Ideally yes, but the custom paths not working yet. Only /api/auth seems to be hitting

Ideally yes, but the custom paths not working yet. Only /api/auth seems to be hitting

it should work since yesterday's release.

If this is the case then should the docs be updated to change the current example to use the separate HTTP method approach (app.get(...) & app.post(...)) instead?

It should work with on, but I believe the matcher needs to be doubled.
app.on(["POST", "GET"], "/api/auth/**", (c) => {
	return auth.handler(c.req.raw);
});

This will be updated in the docs.

app.on(['POST', 'GET'], '/api/auth/**', (c) => auth.handler(c.req.raw)); with double matcher does not work, still have to do
app.get('/api/v1/auth/*', (c) => auth.handler(c.req.raw));
app.post('/api/v1/auth/*', (c) => auth.handler(c.req.raw));

But with v0.4.2 the custom routes (/api/v1/*) are working.


storage-initializer: support AWS_CA_BUNDLE

/kind feature
Describe the solution you'd like
While following this tutorial about how to configure the storage-initializer to fetch from an S3 endpoint I found it difficult to find how to add a custom value to the AWS_CA_BUNDLE environment variable. The main motivation for me is to avoid TLS validation issues while trying to contact an internal S3 endpoint (we have Openstack Swift exposing an S3 API internally, that uses a TLS certificate signed by our internal CA). I tried first via standard container ENV variables but due to how the storage-initializer is configured (via a ConfigMap IIUC) I didn't find a way to do it easily, so I had to add it to the Docker image itself (we have our own images in our internal registry, very close to official kfserving ones). This worked nicely but it would be way better if there was a more friendly way, maybe adding an annotation to s3_secret.go? Something like s3-ca-bundle or similar, that in turn adds the AWS_CA_BUNDLE environment variable to storage-initializer.
If there is another way please let me know!
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]

@elukey Hi, did you try setting serving.kubeflow.org/s3-verifyssl: "0" in your secrets? Or do you have to verify the connection?

@DontStopMeNow I tried as test (and it didn't work), but yes I have to verify the connection :)

For anyone, who will try to use
    serving.kubeflow.org/s3-verifyssl: "0"
It doesn't work, but what help me to achieve the same result is to set
    serving.kserve.io/s3-cabundle: ""
in the same secret.


[FP]: Wrongly reporting vulnerability CVE-2022-25760 on core.access.impl

Package URl
cpe:2.3:a:service_project:service:3.17.0:::::::*
CPE
cpe:2.3:a:accesslog_project:accesslog:*:*:*:*:*:*:*:*
CVE
CVE-2022-25760
ODC Integration
{"label"=>"Maven Plugin"}
ODC Version
8.4.0
Description
This CVE-2022-25760 impacts "carlos8f/node-accesslog" javascript nodejs library. But tools reporting CVE-2022-25760 on<EMAIL_ADDRESS>jar file which is wrong.
From Dependency Check tool team, we need confirmation on these false positives. Could you please validate and confirm?

@prabutdr I cannot find a core.access.impl package so it appears to me that your package is a company private package, which would make this FP fall under items you need to resolve by yourself. The partial name matching is an expected and documented source of false-positives. For public open source projects we work on validating them and then avoiding reporting of those FPs. For private packages that is left to the users themselves.


Defer cell metadata based manipulations to a transform

Currently, there are a number of places in the parser/renderer where we manipulate the docutils AST based on cell metadata

https://github.com/ExecutableBookProject/MyST-NB/blob/ab4ba1d0964a7fe0a6cd516143ccc0a472b63570/myst_nb/parser.py#L56-L63
https://github.com/ExecutableBookProject/MyST-NB/blob/ab4ba1d0964a7fe0a6cd516143ccc0a472b63570/myst_nb/parser.py#L207
https://github.com/ExecutableBookProject/MyST-NB/blob/ab4ba1d0964a7fe0a6cd516143ccc0a472b63570/myst_nb/parser.py#L218

It would probably be better, for future development/extensibility, to handle these in a seperate transform, rather than 'hard-baking' them into the parser.
The full cell metadata should also be added to the CellNode, rather than just tags:
https://github.com/ExecutableBookProject/MyST-NB/blob/ab4ba1d0964a7fe0a6cd516143ccc0a472b63570/myst_nb/parser.py#L203-L205

That seems reasonable to me. That way all of the cell metadata is always present until the very end, and then at the last second a transform passes through all CellNodes and does what it needs to do based on what is inside?


[Frontend] Update TTL in metadata cache path

This patch updates the ttl of the received metadata so that it can
be compared with the cached metadata. Apparently, the received metadata
isn't up-to-date and must be fixed upon receipt.

Codecov Report

Merging #2163 (2c6e642) into master (8c1350e) will decrease coverage by 58.41%.
The diff coverage is 0.00%.

:exclamation: Current head 2c6e642 differs from pull request most recent head c9b7cb6. Consider uploading reports for the commit c9b7cb6 to get more accurate results

@@              Coverage Diff              @@
##             master    #2163       +/-   ##
=============================================
- Coverage     73.24%   14.83%   -58.42%
+ Complexity    10238     2284     -7954
=============================================
  Files           733      733
  Lines         55896    55898        +2
  Branches       6896     6896
=============================================
- Hits          40942     8293    -32649
- Misses        12762    47122    +34360
+ Partials       2192      483     -1709

Impacted Files
Coverage Δ

...java/com/github/ambry/router/GetBlobOperation.java
0.00% <0.00%> (-88.80%)
:arrow_down:

.../src/main/java/com/github/ambry/router/Router.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/com/github/ambry/store/Message.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/named/PutResult.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaMode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaName.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/rest/RestMethod.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/config/KMSConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/config/SSLConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/router/ByteRange.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 570 more

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 8c1350e...c9b7cb6. Read the comment docs.

Got two approvals. If the maintainers can merge, I could deploy.


Update elixir version

@AlexeyKuznetzov could you please help with code style checks in bin/ci?

@dmitrytrager It looks like the new version of Credo forces setting spec types for functions. It's controversial proposition and should be discussed.

@AlexeyKuznetzov done


native ad in grid

Hi,
i have a grid (2 columns) , i want to show a native ad in it but , i want the ad to take all the width not only one column, is this possible with your library ?

@diabloo Hi,
don't think so...


Screen reader does not announce if an update is available for an installed package unless you hover over the update button

This more likely to be a usability bug than a MAS bug, but we should consider fixing it since we will be in the same part of the code while fixing other accessibility bugs.

Implemented in https://github.com/Microsoft/PTVS/pull/2630


Gravatar fallback

Gravatar should fallback to the next available option if no gravatar is available
resolves #9

Hi @seandelaney, These 404's are generated by the browser itself when a request fails, regardless of how the resource is requested. There is no way for us to check if the file is going to exist without generating these errors.
However, we do cache failure. This means that once an image fails it is not requested again until it is pushed out of our cache.

@seandelaney, the default cache is currently limited to a fixed number of 20 cache entries. Could it be that you have more avatars than that on a single page?
I'll see if I can make the limit a bit more flexible over the weekend.

Hmmm. That would make sense - I only see 20 404's listed but I have 21 avatars.
I'm using this React package on a Rota Scheduling web app project and there is an account with over 50 employees.


Skip model saving during precompilation

This is a small optimization that skips model saving during neuron_parallel_compile in order to speed up precompilation by a few minutes and avoid writing large, unusable checkpoints to disk.
Tested with Llama-2-7b-hf using run_clm.py.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Thank you very much for the pull-request. Tagging @michaelbenayoun which is the main maintainer for this code.


Mention in documentation that frame type must be >= 100?

I just spent several hours debugging this - it doesn't seem to be mentioned anywhere. Am I missing something?

From a cursory reading it looks like this isn't the case. Frame types just need to be > PTFrameTypeEndOfStream, which is 0.


Rename probation worker to user

Based on recent discussions around this we've decided to go for User over Probation Worker, so rename existing usages for consistency.

 Basic allocation of applications to assessors