Hide the Nintendo Drivers from Other Programs

This is something that causes issues in some games, basically since the (bad) Nintendo supplied drivers are still there, some games that don't use controller-selectors and just pick up the first thing they see (Dark Souls looking at you) will pick up one of the Joy-Cons and get inputs from them. This is a small issue, and not likely easy to solve, however squashing it would be very nice.
On a similar note, built in direct SCP support would be nice, to eliminate the vJoy -> XInput layer, optionally of course. Again, for games like Dark Souls, where having extra controllers results in problems.

I noticed this in a game the other day, this isn't what I'm working on now but I'll look into it further / how to resolve it when I have time. You might be able to fix it by setting the default gamepad in windows (to the vJoy controller) although that probably won't work for all games and a more permanent fix would be nice.

Eliminating the vJoy->XInput layer would require a lot of work, and I would have to add a button mapping feature, so for now I prefer to leave that to programs like x360ce, but maybe in the future ;P

Yeah neither of these things are all that short term. As for button mapping, since I'm looking at remapping vJoy anyway, I may look into that part myself as well. I'll at least get something going through the command line arguments.

well one thing that although would be nice would be to hide the actual joycons and only have the vjoy there, because programs which can work with multiple controllers see both the real joycons and the vjoy, which isnt too funny.

Yeah, My1 that's exactly what I said, the Nintendo Drivers are the other things that are seen in joy.cpl.
Also, I believe WiinUSoft has found a solution to this for the Wii U Pro, which would likely transfer over, however that solution doesn't work on recent versions of Windows 10, and the code isn't open source (yet, he is going to open source it when it's done apparently). At the very least, this means it's possible.

in joy.cpl you can set the preferred controller, though, which may help for some games in the mean time.

I digged arround with ghosting issue for these few days, and found that it is not just ghosting but 'dual' ghosting ((wireless controller + vjoy) + xinput). For example, in resident evil 4 pc version's controll setting, you will see three gamepad device(wireless controller/vjoy/xinput) with joycon driver + xoutput on background. And it ghosting. and 'mame' emulators input setting also detect 3 input for one push button for binding.
Almost emulator(dolphin...etc) was OK with ghosting problem because there is option to choose specific control device and other control layer will be muted, But almost steam game has not such option.
PS. Some Steam game works very rarely with dinput blocker for this problem, but seems like there is no stable solution we can choose right now.

looking into this further, it looks like this problem is a bit beyond the scope of what I'm capable of in any reasonable amount of time, fortunately, it looks like this project was built for this exact problem: https://github.com/nefarius/ViGEm/tree/master/Sys/HidGuardian
It looks like it's still a work in progress though, but you can test it here:
https://vigem.org/hidguardian-v2-secret-alpha-test-area/
There are also other solutions out there as @jabujabu mentioned, this one might not be the best solution, it also looks a bit technical for the average user, so post what works for you / for which games

Wow. I'll look for it. This could be big change for me. Thanks for taking
care of it. Have a nice day.
On Tue, Apr 3, 2018, 12:25 PM Matthew Fosse<EMAIL_ADDRESS>wrote:

looking into this further, it looks like this problem is a bit beyond the
scope of what I'm capable of in any reasonable amount of time, fortunately,
it looks like this project was built for this exact problem:
https://github.com/nefarius/ViGEm/tree/master/Sys/HidGuardian
It looks like it's still a work in progress though, but you can test it
here:
https://vigem.org/hidguardian-v2-secret-alpha-test-area/
There are also other solutions out there as @jabujabu
https://github.com/jabujabu mentioned, this one might not be the best
solution, it also looks a bit technical for the average user, so post what
works for you / for which games
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mfosse/JoyCon-Driver/issues/11#issuecomment-378116712,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOKDNbAwLbl1RyztUP_oICu0UPOHGJrRks5tkuu5gaJpZM4N9Zhj
.

I'm also having this issue. It can only be the duplicate controllers interfering with each other. Trying to get the Switch Pro Controller working with Minecraft Bedrock and I'm having no luck. When I start the game the controls go crazy and it selects menu options back and forth and I'm unable to even get into the game :(
Without JoyCon-Driver the controls in Minecraft work mostly... everything apart from sprinting for some reason... which is why I'm here, looking for an alternative in using JoyCon-Driver.
Having said that, the rest of the utility works quite well! I love the little touch of playing the mario tune using the vibration of the controller :D pretty neat
If HidGuardian was a one-click utility I would look into it, unfortunately there's too many steps to make it worthwhile for me just to play Minecraft. I'd rather just use my SF30Pro in the meantime.


Fix Django warnings about index_together with new migration

This is triggering a Django warning

RemovedInDjango51Warning: 'index_together' is deprecated. Use 'Meta.indexes' in 'taggit.TaggedItem' instead.

This updates the index_together use to resolve this, hopefully painlessly.

something to check, whether index renaming is cheap or not in the major SQL vendors (locking etc)

also whether the index renaming is actually safe? we want to make sure that deployments including this don't cause downtime.

Things to do on this one:

set things up to make RemovedInXWarnings instead be errors for testing on djmain.
concretise the existing index names when setting up the IndexTogether relations, making sure to not generate any actual SQL operations

Getting this error while running tests:
django.utils.deprecation.RemovedInDjango51Warning: 'index_together' is deprecated. Use 'Meta.indexes' in 'taggit.TaggedItem' instead.

Versions:
Django = "4.2.2"
django-taggit = "4.0.0"

Why is this not merged yet?

Addressed in https://github.com/jazzband/django-taggit/pull/869


[Feature] Check if a bundle is confirm or not

Component
transports
Describe the feature you would like
I use flashbot to send a bundle to blockchain and get a bundle hash, but I don't know if this bundle confirm or not. It would be great if Alloy could provide a functionality to check the transaction.
Here is a repo that could be referred: https://github.com/onbjerg/ethers-flashbots/blob/master/src/pending_bundle.rs
Additional context
No response

Hi @chen4903 thanks for your suggestion, the correct place to implement this would indeed be https://github.com/leruaa/alloy-mev/issues/11
Assuming this doesn't require implementation changes on Alloy's side, marking as duplicate.
cc @leruaa


Maintained ?

Hi,
i was wondering if this was still maintained,
and if not are you guys willing to pass the maintainership to someone / some group ?
not really sure i'm willing myself, but let's at least be clear  :)
Btw, i use this in one of my tool, and it's working great 👍  good job and thanks for that

ping @danomatic @davidstelter @jaimeburnap

Not really. I'm busy with many other things, and mostly work in React now. However I would like to see continued bug fixes and performance updates.  If someone wants to take it over, let me know.


Fix the eta-build test

Currently, the test action of eta-build is broken. It should be refactored to use epm instead because it can take care of setting the classpath.

Testing will be done with a separate test suite, see #24.


Check for rackspace-novaclient

(Carrying over from #85)
Adding rackspace-novaclient as a required package may not be the best idea, but we should have a way to alert users that the package isn't installed if they're actively trying to talk to Rackspace's cloud.

Would this be a good use case for setuptools' extras?
    extras_require = {
        'rackspace':  ['rackspace-novaclient']
    }

This would allow users to run pip install supernova[rackspace] and get the desired setup (I think).

That's not a bad idea. Want to put in a PR? :)

Tug #98 submitted.


[QUESTION] Enriching the auto-generated OpenAPI-Spec

Description
Hi there, new to FastAPI so sorry if this question has been asked elsewhere:
What is the best way to enrich the auto-generated OpenAPI-Spec generated by FastAPI. It currently seems to support only a few things (like changing the title or description), but  if I would want to add tags to group my endpoints or do sth more fancy like adding a logo (which is supported by ReDoc via x-logo) it seems to me that I would want to use FastAPI to generate the base specification and then add some scripts that will enrich this spec in a structured way?
Has anyone encountered this issue before?
Additional context
I already looked into the code a little bit and it seems that the ReDoc that gets shown when serving the API is generated on the fly (as is the OpenAPI schema) => this seems to make a custom processing of the specification to make use of cool ReDoc features harder

tags: https://fastapi.tiangolo.com/tutorial/path-operation-configuration/#tags
logo: not possible currently, but there's a PR to host static file in the work

and yes it's generated on the fly. the documentation is very well written, you should take a look :hibiscus:

great thanks for the pointer :)

it seems to me that I would want to use FastAPI to generate the base specification and then add some scripts that will enrich this spec in a structured way?

re-reading you it seems like you somehow missed the part with pydantic models (https://fastapi.tiangolo.com/tutorial/body-nested-models/)
the models you define will be documented in the schema if you prefer

no, where my question came from is that there are all these cool Redoc vendor extensions (like x-logo) that I currently have in my OpenAPI spec (with flask as a backend) => I would ideally like to move this to FastAPI soon, but I don't really expect you guys to support all of those vendor extensions out of the box and was thinking I would need to somehow write my own scripts that enrich the generated OpenAPI spec with the vendor extensions I want and wasn't sure how to best integrate this with FastAPI. Does this make sense?
All the path operation configuration you pointed me to definitely already help (but does not fix the problem with the vendor extensions) :)

maybe it would make sense to add support for vendor extensions in general somehow? (s.t. that I can enrich the OpenAPI spec with whatever I like and if ReDoc happens to use that vendor extension: lucky me!)

Seems like a post-processing hook for creating the openapi spec would be handy for this.  That way custom processing could happen after the fastAPI spec generation happens and before it's presented to Swagger UI or ReDoc.

Thanks for the help here @euri10  and @wshayes !
So, I just ended up adding it to the docs. Exactly with your ReDoc logo example :smile: https://fastapi.tiangolo.com/tutorial/extending-openapi/ :tada:

Awesome, thanks!

Great! Thanks for coming back and closing it.

Thanks for the help here @euri10 and @wshayes !
So, I just ended up adding it to the docs. Exactly with your ReDoc logo example 😄 https://fastapi.tiangolo.com/tutorial/extending-openapi/ 🎉

https://fastapi.tiangolo.com/advanced/extending-openapi/


Phase 2 takes a very long time, compared to other phases...

Plot Name: plot-k32-2021-07-16-15-37-c2eeb44dcc3399b848191dbffde5730678b65160763dc5c7332a02ee658d4d14
[P1] Table 1 took 12.7015 sec
[P1] Table 2 took 78.3049 sec, found<PHONE_NUMBER> matches
[P1] Table 3 took 98.9804 sec, found<PHONE_NUMBER> matches
[P1] Table 4 took 150.923 sec, found<PHONE_NUMBER> matches
[P1] Table 5 took 177.029 sec, found<PHONE_NUMBER> matches
[P1] Table 6 took 139.75 sec, found<PHONE_NUMBER> matches
[P1] Table 7 took 84.459 sec, found<PHONE_NUMBER> matches
Phase 1 took 742.191 sec
[P2] max_table_size =<PHONE_NUMBER>
[P2] Table 7 scan took 16.7837 sec
[P2] Table 7 rewrite took 31.74 sec, dropped 0 entries (0 %)
[P2] Table 6 scan took 145.283 sec
[P2] Table 6 rewrite took 47.2912 sec, dropped 581249660 entries (13.534 %)
[P2] Table 5 scan took 42.3078 sec
[P2] Table 5 rewrite took 79.2062 sec, dropped 761965690 entries (17.7415 %)
[P2] Table 4 scan took 111.832 sec
[P2] Table 4 rewrite took 103.39 sec, dropped 828823492 entries (19.2979 %)
[P2] Table 3 scan took 112.607 sec
[P2] Table 3 rewrite took 44.8614 sec, dropped 855070753 entries (19.9087 %)
[P2] Table 2 scan took 51.7597 sec
[P2] Table 2 rewrite took 161.97 sec, dropped 865544182 entries (20.1524 %)
Phase 2 took 954.95 sec
I am using a 5950x with stock settings intel p4600 and 970evo.I oberseved a significant drop in usage of cpu during phase 2 ...Using  -r 16 -u 256...anyone has a solution?

p2 table 7 uses the -2 temp. the others use the -t directory i'm pretty sure, so that looks like the issue whatever drive you have set for that is slow.

p2 table 7 uses the -2 temp. the others use the -t directory i'm pretty sure, so that looks like the issue whatever drive you have set for that is slow.

thx man, just switched to 970evo...and it works ok...

Please use example -t D:/ -2 E:/ and -r 30 set it
Total plotting time 30 minutes ok

Please use example -t D:/ -2 E:/ and -r 30 set it
Total plotting time 30 minutes ok

I am on linux btw


Show how to prevent teleporting on top of objects

By default the teleporter will always look for the nearest floor to stand the player on, so a tutorial showing how to prevent teleporting on certain objects would be helpful.

https://www.youtube.com/watch?v=6JmXiK_hkg8


[WIP] retrieve list of portal ids and filter input lists with it

working on a solution for #52. i'm not sure i like this approach, but it is progress.
first, i queried the catalog to get a current list of ids with portal pages. (this could probably be done with fewer lines of code.
# retrieve list of ids with portal pages
url = "https://app.nih-cfde.org/ermrest/catalog/1/attribute/CFDE:gene/id@sort(id)"
response = urlopen(url)
data_json = json.loads(response.read())
portal_pages = pd.json_normalize(data_json)
portal_page_ids = portal_pages["id"].to_numpy()

then, i created a id_list2 which filters id_list and use that for making markdown pages.
# load up each ID in id_list file - does it have a portal page?
 id_list2 = set()
with open(args.id_list, 'rt') as fp:
    for line in fp:
        line = line.strip()
        if line:
            if line not in portal_page_ids:
                print(f"ERROR: requested input id {line} not found in portal_page_ids", file=sys.stderr)
                print(f"skipping!", file=sys.stderr)
                continue
                #sys.exit(-1)
            id_list2.add(line)

print(f"Loaded {len(id_list2)} IDs contained in both the ref list and the portal page list.",
      file=sys.stderr)

template_name = 'alias_tables'
for cv_id in sorted(id_list2):
....

technically this is working, because the output looks like this:
Running with term: gene
Using output dir output_pieces_gene/00-alias for pieces.
Loaded 24620 reference IDs from data/validate/ensembl_genes.tsv
ERROR: requested input id ENSG00000000001 not found in ref_id_list
Loaded 19972 IDs from data/inputs/STAGING_PORTAL__available_genes__2022-08-19.txt
ERROR: requested input id ENSG00000204616 not found in portal_page_ids
skipping!
ERROR: requested input id ENSG00000000001 not found in portal_page_ids
skipping!
ERROR: requested input id ENSG00000262302 not found in portal_page_ids
skipping!
ERROR: requested input id ENSG00000275778 not found in portal_page_ids
skipping!
ERROR: requested input id ENSG00000278992 not found in portal_page_ids
skipping!
ERROR: requested input id ENSG00000279846 not found in portal_page_ids
skipping!
ERROR: requested input id ENSG00000281994 not found in portal_page_ids
skipping!
ERROR: requested input id ENSG00000282232 not found in portal_page_ids
skipping!
ERROR: requested input id ENSG00000288373 not found in portal_page_ids
skipping!
ERROR: requested input id ENSG00000288708 not found in portal_page_ids
skipping!
Loaded 19962 IDs contained in both the ref list and the portal page list.

however, something like this would have to be added to every script. i wonder if there is a way to do this as like a common script...

i move a chunk of the code to cfde_common and made it a function
def get_portal_page_ids(term):
    # get list of ids with portal pages from json
    url = f'https://app.nih-cfde.org/ermrest/catalog/1/attribute/CFDE:{term}/id@sort(id)'
    response = urlopen(url)
    data_json = json.loads(response.read())
    df = pd.json_normalize(data_json)
    ids = df["id"].to_numpy()
    print(f"Loaded {len(ids)} {term} IDs in the CFDE Portal from {url}")
    return(ids)

Then, I added these 3 lines of code to the python scripts and used id_list_filtered as the input for the make markdown function.
    # filter by ids with a page in the portal
    id_pages = cfde_common.get_portal_page_ids(term)
    id_list_filtered = [value for value in id_list if value in id_pages]
    print(f"Using  {len(id_list_filtered)} {term} IDs.")

Will see if I can make this work for the other vocabularies next.

working now for all inputs. especially useful for anatomy and compound

okay, i have a snakemake rule that is working okay. when you run bash scripts/retrieve-ids.sh form the the command line, it will download only missing files. however, snakemake always re-dowloads all five.
rule retrieve:
    message:
        f"retrieve list of ids in the registry"
    output:
        "data/validate/anatomy.csv",
        "data/validate/disease.csv",
        "data/validate/compound.csv",
        "data/validate/gene.csv",
        "data/validate/protein.csv",
    shell: """
        bash scripts/retrieve-ids.sh
    """

retrieve-ids.sh looks somethign like this:
if [ ! -f data/validate/anatomy.csv ]
then
    echo "Downloading csv of ids for anatomy."
    curl -L "https://app.nih-cfde.org/ermrest/catalog/1/attribute/CFDE:anatomy/id@sort(id)?accept=csv" -o data/validate/anatomy.csv
else
    echo "csv with ids found for anatomy."
fi

Then, I added this to common_cfde.py  and replaced most instance of REF_FILES with ID_FILES to use these csv files for filtering input lists.
ID_FILES = {
    'anatomy': 'data/validate/anatomy.csv',
    'compound': 'data/validate/compound.csv',
    'disease': 'data/validate/disease.csv',
    'gene': 'data/validate/gene.csv',
    'protein': 'data/validate/protein.csv',
    }

In cases where the .tsv version of the file was used as both the reference file and the alias file, i kept them as is and commented them out of the snakefile. Mostly KG related things.

might be interested in: http://ivory.idyll.org/blog/2020-snakemake-hacks-collections-files.html

check out https://github.com/nih-cfde/update-content-registry/pull/71 - no need to merge, but I think it will result in less surprises down the road (plus it's a cute fun hack)

I added a little bit chuck of code to aggregate-markdown-pieces.py that makes a csv file with the total number of markdown chunks per widget!
    # print results
    jsonCounter = len(glob.glob1(dirpath,"*.json"))
    f = open("logs/chunks.csv", "a")
    f.write(f"{dirpath},{jsonCounter}\n")
    f.close()

This plot shows what I suspected which is that there are few instances where I have filtered my input ID list to 0. So said. Will work on a fix later.

Found a script for a lincs widget without a rule, so I added that 👍
Need to find a way to filter the input lists that use the .tsv files as alias files by the .csv reference file 😞
Made a rule to make plots that you can run with make plots 😄

suggest merging this sooner rather than later. it's getting big. and I approved the changes as of 3 days ago... and now you've made a bunch more ;)

its not done yet! at least, i feel like the big picture isn't done yet. but i do see how maybe could have been/be broken up into smaller pieces

k ;)

the yak i am shaving is that there are four types of python scripts and i can't seem to find a single solution that will allow me to add a filter by ID_FILES because how the processes an alias file affects when and how i filter by a ref file.

no alias file (build-anatomy-links, build-markdown-pieces)
alias file is data/input/*txt (build-protein-refseq)
alias file is data/input/*tsv (build-markdown-pieces-ucsc-genome-browser-widget)
alias file is REF_FILES (build-markdown-pieces-gene-kg)

i currently have different solutions for each type of those scripts. the first is working great, the second, not so much...

replace REF_FILES with ID_FILES in code
copy/paste/edit new block of code to read and filter

hence the new graphs for QC because i like visuals. although this probably address #57

its getting bigger 😆 but also more better

TLDR
new common function for getting list of portal ids for validation
def get_validation_ids(term):
    # get list of validation retrieved form portal pages
    validation_file = ID_FILES.get(term)
    if validation_file is None:
        print(f"ERROR: no validation file. Run `make retrieve`.", file=sys.stderr)
        sys.exit(-1)

    # load validation; ID is first column
    validation_ids = set()
    with open(validation_file, 'r', newline='') as fp:
        r = csv.DictReader(fp, delimiter=',')
        for row in r:
            validation_id = row['id']
            validation_ids.add(validation_id)

    print(f"Loaded {len(validation_ids)} IDs from {validation_file}.",
          file=sys.stderr)

    return(validation_ids)

validate and skip
# validate ids
validation_ids = cfde_common.get_validation_ids(term)

skipped_list = set()
id_list = set()
with open(args.id_list, 'rt') as fp:
    for line in fp:
        line = line.strip()
        if line:
            if line in validation_ids:
                id_list.add(line)

            if line not in validation_ids:

                skipped_list.add(line)

                f = open("logs/skipped.csv", "a")
                f.write(f"{args.widget_name},{term},{line},ref\n")
                f.close()

print(f"Validated {len(id_list)} IDs from {args.id_list}.\nSkipped {len(skipped_list)} IDs not found in validation file.",
      file=sys.stderr)

check in alias file and skip, something like this but variable
# validate that ID list is contained within actual IDs in database
ref_file = cfde_common.REF_FILES.get(term)
if ref_file is None:
    print(f"ERROR: no ref file for term. Dying terribly.", file=sys.stderr)
    sys.exit(-1)

# load in ref file; ID is first column
ref_id_list = set()
ref_id_to_name = {}
with open(ref_file, 'r', newline='') as fp:
    r = csv.DictReader(fp, delimiter='\t')
    for row in r:
        ref_id = row['id']
        ref_id_to_name[ref_id] = row['name']
        ref_id_list.add(ref_id)

print(f"Loaded {len(ref_id_list)} reference IDs from {ref_file}",
      file=sys.stderr)

# load in id list
id_list = set()
skipped_list = set()
with open(args.id_list, 'rt') as fp:
    for line in fp:
        line = line.strip()
        if line:
            if line in ref_id_list:
                id_list.add(line)
            if line not in ref_id_list:
                skipped_list.add(line)

                f = open("logs/skipped.csv", "a")
                f.write(f"{args.widget_name},{term},{line},alias\n")
                f.close()

print(f"Skipped {len(skipped_list)} IDs not found in {ref_file}.",  file=sys.stderr)

added counter for input
# print length of input list
with open(args.id_list, 'r') as fp:
    x = len(fp.readlines())
print(f"Loaded {x} IDs from {args.id_list}.", file=sys.stderr)

added counter for output
# summarize output
print(f"Wrote {len(id_list) } .json files to {output_dir}.",
      file=sys.stderr)

and also in script/aggregate-markdown-pieces
    # print results
    jsonCounter = len(glob.glob1(dirpath,"*.json"))
    f = open("logs/chunks.csv", "a")
    f.write(f"{dirpath},{jsonCounter}\n")
    f.close()

some examples outputs
Running with term: gene
Using output dir output_pieces_gene/05-MetGene for pieces.
Loaded 1274 IDs from data/inputs/gene_IDs_for_MetGene.txt.
Loaded 19975 IDs from data/validate/gene.csv.
Validated 1202 IDs from data/inputs/gene_IDs_for_MetGene.txt.
Skipped 72 IDs not found in validation file.
Wrote 1202 .json files to output_pieces_gene/05-MetGene.

Running with term: gene
Using output dir output_pieces_gene/00-alias for pieces.
Loaded 19971 IDs from data/inputs/gene_IDs_for_alias_tables.txt.
Loaded 19975 IDs from data/validate/gene.csv.
Validated 19962 IDs from data/inputs/gene_IDs_for_alias_tables.txt.
Skipped 9 IDs not found in validation file.
Skipped 136 IDs not found in data/inputs/Homo_sapiens.gene_info_20220304.txt_conv_wNCBI_AC.txt.
Wrote 19826 .json files to output_pieces_gene/00-alias.

Running with term: compound
Using output dir output_pieces_compound/04-drugcentral for pieces.
Loaded 3612 IDs from data/inputs/compound_IDs_DrugCentral.txt.
Loaded 73498 IDs from data/validate/compound.csv.
Validated 3607 IDs from data/inputs/compound_IDs_DrugCentral.txt.
Skipped 0 IDs not found in validation file.
Skipped 5 IDs not found in alias file.
Wrote 3602 .json files to output_pieces_compound/04-drugcentral.

I pushed to staging to test how things were working. I expected very few resources to refresh... The numbers are higher than expected, so I will look into figure out what is different.
2022-09-16 11:31:02,150 - INFO - Refreshed 1/366 resource_markdown values for 'anatomy' (353 in registry)
2022-09-16 11:31:11,843 - INFO - Refreshed 478/1901 resource_markdown values for 'disease' (1872 in registry)
2022-09-16 11:34:16,276 - INFO - Refreshed 13448/73500 resource_markdown values for 'compound' (59341 in registry)
2022-09-16 11:38:54,617 - INFO - Refreshed 12213/64149 resource_markdown values for 'protein' (64147 in registry)
2022-09-16 11:42:29,798 - INFO - Refreshed 0/19984 resource_markdown values for 'gene' (19971 in registry)
Resource markdown refreshed on release

I have a few 0s in my summary table of files created, but I think that is a math problem because the files are being created locally, they just aren't being counted. sigh. will investigate.
See https://github.com/nih-cfde/update-content-registry/blob/retrieve-pages/logs/README.md
Note: the last column is the one that is super important. this is the number of IDs that doesn't exist in the portal. these would normally cause the make update to fail if they were left in the workflow, but i removed them so the workflow runs successfully.

let me know if you want to work through this together at all!

Okay, this is my new favorite report that tells me how many annotations were written or skipped. The one’s with 0s in the written column worry me. https://github.com/nih-cfde/update-content-registry/blob/retrieve-pages/logs/README.md
So, two relevant scripts to check for potential errors are build-markdown-pieces-gene-kg.py ( makes all the kg_widgets) and  build-markdown-pieces-gene-translate.py(makes the alias_table widget).
See also the new code in the aggregate-markdown-pieces.py which does the counting


[SPARK-17805][PYSPARK] Fix in sqlContext.read.text when pass in list of paths

What changes were proposed in this pull request?
If given a list of paths, pyspark.sql.readwriter.text will attempt to use an undefined variable paths.  This change checks if the param paths is a basestring and then converts it to a list, so that the same variable paths can be used for both cases
How was this patch tested?
Added unit test for reading list of files

Test build #66455 has started for PR 15379 at commit ade9823.

Test build #66455 has finished for PR 15379 at commit ade9823.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66455/
Test PASSed.

Merged build finished. Test PASSed.

ping @davies @yanboliang

+1 for this PR.

Thanks - the argument renaming issue is largely orthogonal and I don't think we can break it now. I'm going to merge this in master/2.0.

Thanks @rxin, @HyukjinKwon and @holdenk for reviewing!


No header when using fluidNavigator

Hi why the header is gone???
only replace createFluidNavigator with createStackNavigator the header now works.
But with createFluidNavigator it doesn't!
const MainNavigator = createFluidNavigator({
    Test:{screen : NotificationScreen,
        navigationOptions :  {
            headerStyle: {
                backgroundColor: Platform.OS === 'android' ? Colors.primary : ''
            },
            headerTitleStyle : {
                fontFamily : 'open-sans-bold'
            },
            headerBackTitleStyle : {
                fontFamily : 'open-sans'
            },
            headerTintColor: Platform.OS === 'android' ? 'white' : Colors.primary,
        }},
}, {
    navigationOptions : {
        drawerIcon : drawerConfig => <Ionicons name={Platform.OS === 'android' ? 'md-cart' : 'ios-cart'} size={23} color={drawerConfig.tintColor}/>,
    }
});

Hi @Stevemoretz,
I am facing the same issue, did you have any luck with the header? Why the header is disappearing when we are using FluidNavigator?
Thanks

Hi @kabaljit Nope there's no reasonable way to fix this.It never appears to be disappeared that's the problem.No one can help but the author.Wish he actually responded.But he isn't.

Please consider this issue.

Up


[Bug] upgrade to hive-exec 2.3.4 due to CVE

Search before asking

[X] I searched the issues and found no similar issues.

Linkis Component
linkis-public-enhancements
What happened + What you expected to happen
https://github.com/advisories/GHSA-rrfq-g5fq-fc9c
hive-exec is used a few places in linkis
Relevent platform
all
Reproduction script
n/a
Anything else
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

This issue can be fixed by search and replace maven dependencies, please assign it to me. @peacewong


netbsd small changes: fix datetime.d

fix datetime issues on NetBSD

I moved timezone tests as you suggest

This needs a bug report for a changelog entry. Other than that, LGTM

I don't really care whether there's a changelog entry or not, but this just affects a unit test, so I don't see much point.

It's hard to tell where the code actually is, since github doesn't show me the whole file.
I assumed that at least the part where gettimeofday is added as an import is a bug fix. If not, feel free to ignore me.

No. This is purely in unit tests. The problem is that there's a bug in FreeBSD and NetBSD with how the global variable tzname is filled in (it uses the wrong variable in iterating one of the loops IIRC), and apparently, on NetBSD, with its current TZ databsae files, America/New_York gives the wrong value (whereas America/Los_Angeles gives the wrong value on both FreeBSD and NetBSD and has for a while). The bug was reported to the FreeBSD guys by Martin quite a while ago, but it's gone under the radar for some reason, and until it's fixed in all of the version we support, the tests can't test that functionality on those systems. :(
I really should find the time to figure out how to push it through on at least the FreeBSD side (especially since that's what I've been using for my primary OS), but it's one of many items on my todo list that I'm not getting to in a timely enough manner.

No. This is purely in unit tests.

It appears the line at 465 that was amended was in the function currStdTime, so this probably is a bug fix too. But probably only caused a deprecation warning. In any case, very minor, and no major need for a bug report.


Dataset.transpose() fails if there are repeated headers

If the same value appears multiple times in the headers of a Dataset, then when you transpose() it the values from the first time that header occurs are copied to all instances of it in the new Dataset. Although normally we wouldn't expect to have repeated headers, in the case of transpose() it seems quite likely, given that the initial header row is actually the first column of the Dataset you want.
I would expect Dataset.transpose().transpose() to give me the original Dataset back. However:
import tablib
data = tablib.Dataset()
data.headers = ("first_name", "last_name", "age")
data.append(('John', 'Adams', 90))
data.append(('George', 'Washington', 67))
data.append(('John', 'Tyler', 71))
data.transpose().transpose().dict == data.dict

Which gives:
False

If we investigate in more detail:
data.transpose().transpose()['first_name']
data.transpose().transpose()['last_name']
data.transpose().transpose()['age']

We get:
['John', 'George', 'John']
['Adams', 'Washington', 'Adams']
[90, 67, 90]

The broken bit of transpose() is:
        for column in self.headers:

            if column == self.headers[0]:
                # It's in the headers, so skip it
                continue

            # Adding the column name as now they're a regular column
            row_data = [column] + self[column]
            row_data = Row(row_data)
            _dset.append(row=row_data)

If you replace that with:
        for index, column in enumerate(self.headers):

            if column == self.headers[0]:
                # It's in the headers, so skip it
                continue

            # Adding the column name as now they're a regular column
            # Use `get_col(index)` in case there are repeated values
            row_data = [column] + self.get_col(index)
            row_data = Row(row_data)
            _dset.append(row=row_data)

then it works as expected.

Thank you very much! This has now been integrated.

:sparkles: :cake: :sparkles:

I found a related bug to this, though this kind of data may not occur frequently. The bug was found using @HypothesisWorks.
data = tablib.Dataset(('fname', 'Doe', '10'), headers=['fname', 'lname', 'age'])
print(data, data.transpose().transpose(), sep='\n\n')

fname|lname|age
-----|-----|---
fname|Doe  |10

fname|lname|age
-----|-----|---


Add unpublishFeedGen script

This PR is based on a feature request from #76,
and I am modifying my own script to applying it in the main repository.

Hey just wondering, who reviews PRs in this repo? Have had one outstanding for a while and would like to get it merged or rejected


no blinking characters in ncurses apps

Tested ncurses app and detected,
that the attribute A_BLINK has no effect.
There is a definition in terminfo for blinking:
blink=\E[5m
For what?
It would be fine, when characters are able to blink.
Thank you
Erhie

@Erhie probably an un-implemented ANSI escape sequence.

would be nice, if someone implements also this escape sequence for blinking characters
Erhie

@Erhie It's probably coming sooner or later.  The coming year will allegedly be the year of big major console improvements, according to the console team.  See #111 posts by miniksa.

Yea we definitely don't support blinking characters ATM. But it's on the backlog :)

Thanks for the discussion.
Closing this issue since:

We currently don't support blinking characters in the Console. It is on the backlog though and we'll consider for a future release
This is the WSL issues repo, but this is an issue in Console which has its own Console GitHub Repo
GitHub doesn't allow issues to be moved between repos, preserving posters' identity :(

If you have further asks/issues, please file new issues on our Console GitHub Repo.


[bug]: RegisterBlockEpochNtfn is sending the same event twice

Background
RegisterBlockEpochNtfn is sending 2 events (same data) for block updates
Your environment

version of lnd: 0.16.2-beta
operating system: Linux
version of bitcoind: v24.0.1

Steps to reproduce
use code provided here https://lightning.engineering/api-docs/api/lnd/chain-notifier/register-block-epoch-ntfn or use Alex Library https://github.com/alexbosworth/ln-service#subscribetoblocks
Expected behaviour
Send only 1 event per block
Actual behaviour
Send 2 events per block (same data)

Do you see this happening all the time, or only occasionally?

Do you see this happening all the time, or only occasionally?

it is happening all the time

i don't see this on regtest
https://github.com/alexbosworth/ln-service/blob/master/test/chainrpc-integration/test_subscribe_to_blocks.js
block events only emit once for me

You're calling the RPC with height. Do you only get repeated blocks for that height or for all subsequent blocks?

@alexbosworth @guggero this a test project where you can see the behavior https://github.com/dolcalmi/lnd-subscribe-to-blocks-test (please check the readme.. although is just run yarn install and then make start)

did you see the behavior on the above test in ln-service? just need to do npm install on the repo and then node filename to run the file, as long as you have docker installed
can log out blocks array to see which block events were generated

did you see the behavior on the above test in ln-service? just need to do npm install on the repo and then node filename to run the file, as long as you have docker installed
can log out blocks array to see which block events were generated

did you see the behavior on the above test in ln-service?
no, but we had to debounce in galoy backend https://github.com/GaloyMoney/galoy/blob/main/src/servers/trigger.ts#L322 (same is happening  in signet and mainnet)
Not sure to underestand this can log out blocks array but here is a log https://github.com/dolcalmi/lnd-subscribe-to-blocks-test/blob/main/logs.txt#L2058 of lnd

like if you run the subscribe to blocks file and you do console.log on blocks at the end of the test or in the foreach at the end, does it show repetition for you?

like if you run the subscribe to blocks file and you do console.log on blocks at the end of the test or in the foreach at the end, does it show repetition for you?

yes, same behavior https://github.com/dolcalmi/lnd-subscribe-to-blocks-test/blob/main/test-blocks.js#L51
From grpc client -> Height: 201 - 09d6c1e3515a71a63534a6b0de09970cabaedc0d904d104693cd837bb975bda0
from lnService -> Height: 201 - 09d6c1e3515a71a63534a6b0de09970cabaedc0d904d104693cd837bb975bda0
From grpc client -> Height: 201 - 09d6c1e3515a71a63534a6b0de09970cabaedc0d904d104693cd837bb975bda0
from lnService -> Height: 201 - 09d6c1e3515a71a63534a6b0de09970cabaedc0d904d104693cd837bb975bda0
from lnService -> Height: 202 - 40664ee2ef52654fd0b9ba65205f0f184a8764e68902d29a0dbc805569861293
From grpc client -> Height: 202 - 40664ee2ef52654fd0b9ba65205f0f184a8764e68902d29a0dbc805569861293
from lnService -> Height: 202 - 40664ee2ef52654fd0b9ba65205f0f184a8764e68902d29a0dbc805569861293
From grpc client -> Height: 202 - 40664ee2ef52654fd0b9ba65205f0f184a8764e68902d29a0dbc805569861293

sorry i mean if you run https://github.com/alexbosworth/ln-service/blob/master/test/chainrpc-integration/test_subscribe_to_blocks.js
and modify the end of the test to do console.log(blocks) does it show duplicates

sorry i mean if you run https://github.com/alexbosworth/ln-service/blob/master/test/chainrpc-integration/test_subscribe_to_blocks.js
and modify the end of the test to do console.log(blocks) does it show duplicates

:thinking: it does not show duplicates... I did some tests and I think I found it... it is happening with this setting zmqpubrawblock
bitcoind config: https://github.com/dolcalmi/lnd-subscribe-to-blocks-test/blob/main/dev/bitcoind/bitcoin.conf#L9
lnd config: https://github.com/dolcalmi/lnd-subscribe-to-blocks-test/blob/main/dev/lnd/lnd.conf#L34

I just took a quick look at the log file in your test and see this:
Possible reorg at block: height=188, hash=1596dbacc5e72da3ce0a38f1fbd47e4258bb49206bc034481690f5563a516560
...
Disconnecting block: height=188, hash=1596dbacc5e72da3ce0a38f1fbd47e4258bb49206bc034481690f5563a516560

So it looks like your test is doing a re-org, in which case it would be expected to be notified about a block multiple times. So I'd say something is up with the test logic itself.

Thank you for the test project. I now took a look at it and can say with a large degree of certainty that the multiple notifications happen because lnd thinks there is a re-org (the log lines I posted above).
I changed to another bitcoind docker image and the problem was gone:
  bitcoind:
    image: ruimarinho/bitcoin-core:23
    ports:
      - "18443:18443"
    environment:
      - HOME=/home/bitcoin
    volumes:
      - ${PWD}/dev/bitcoind/bitcoin.conf:/home/bitcoin/.bitcoin/bitcoin.conf

So I'm pretty sure something in how ZMQ works is broken or different in your lncm variant of the bitcoind docker image.

Closing this as I'm pretty sure this is an issue of the specific fork/build of bitcoind and not lnd.
@dolcalmi I looked at the Dockerfile for lncm/bitcoind and it looks like the ZMQ parameters are supplied as command line arguments: https://github.com/lncm/docker-bitcoind/blob/master/24.0/Dockerfile#L256
Maybe adding the same values in the config file actually creates two ZMQ endpoints under the same address which ends up pushing the events twice? No idea how ZMQ works under the hood, but if that is the case then you should be fine with just removing those two lines from your bitcoin.conf and things should work normally.


Language support EN

my printer returns these states as READY:
status: { hrDeviceStatus: 3, status1: " Replace Toner", status2: "", status3: "", status4: "" }
status: { hrDeviceStatus: 3, status1: " Ready to Copy", status2: "", status3: "", status4: "" }

Is it possible to add these strings?
Originally posted by @OMdra in https://github.com/nielstron/pysyncthru/issues/19#issuecomment-663820358

Is it possible to add these strings?

Sure

status: { hrDeviceStatus: 3, status1: " Replace Toner", status2: "", status3: "", status4: "" }

I suggest to make this the state "Low on Toner".  Or is it redundant with the percentage information about the toner? @OMdra

I suggest to make this the state "Low on Toner". Or is it redundant with the percentage information about the toner?

These two (Ready to copy and Replace toner) statuses change again and again after 3 seconds. I don't know if it's a good idea.
Toner information will be probably a better solution.
toner_black: {opt: 1, status: 0, remaining: 22, cnt: 707},
toner_cyan: {opt: 1, status: 0, remaining: 18, cnt: 651},
toner_magenta: {opt: 1, status: 0, remaining: 15, cnt: 652},
toner_yellow: {opt: 1, status: 2, remaining: 0, cnt: 670}

These two (Ready to copy and Replace toner) statuses change again and again after 3 seconds.

I see, agree that they should map to ready then

Toner information will be probably a better solution.

But this is already displayed, right?

@OMdra can you try the current dev branch?

But this is already displayed, right?
Yes, toner information works.

can you try the current dev branch?
Not a problem, but I'll try it in the evening.

I think the problem is still with capital letters.
You have in language.py Ready To Copy but printer returns Ready to Copy.
On the other side Replace Toner works.

Thanks for the hint! Please try again with the latest commits

@nielstron now it's great! Everything works for me. Thanks


Improving Calibrators plot

Describe the bug
The calibrator Plot needs some improvments.
To Reproduce

Open the HPLC_UV_Standards example
Click on Action > run workflow
Click on "View" > Calibrators
Select 5-HTP in the transitions explorer

Expected behavior

Not all the points (5HTP-pts, the blue points in the screenshot) are displayed. only the ones that have not been excluded by the calibration curve algorithm are displayed. we would like to have all the points displayed. excluded points and non excluded points may be displayed with a different shape.
Having the point behind the line is not convenient. On another side, having them on the top can lead to confusion. maybe they can be the same color, but with a different shape.
Add the possibility to hide the points, and/or the curve.
Add some margin on top and right
Add a zoomer item as other plots do
Add some details about the curve: the parameters used and the one deducted: transformation_model, transformation_model_param_y_weight, ...

Version information
1.18.

A few other points to potentially add to the Expected Behavior

Show raw points even if there is no quantitation method or line of best fit
Filled points could be used for "fitted" points and unfilled points could be used for "unfitted" points


Add .phps extension for PHP

.phps stands for "PHP Source"
Useful to show examples of PHP code. A properly-configured server will output a .phps file as is, with color-formated source code instead of the HTML that would normally be generated.
Usage in GitHub
Source of example file

Nice pull request! :+1:

:+1: thanks @fbonzon. This will be live in the next week or so when we cut a new release of Linguist.


[Performance] Memory and CPU usage issues on idling

Environment
React Native Environment Info:
System:
OS: Windows 10
CPU: x64 Intel(R) Core(TM) i7-2600K CPU @ 3.40GHz
Memory: 11.62 GB / 15.90 GB
Binaries:
Yarn: 1.12.3 - D:\Program Files (x86)\Yarn\bin\yarn.CMD
npm: 6.1.0 - C:\Program Files\nodejs\npm.CMD
react 16.6.0
react-native 0.57.4
Description
I'm having some weird issues with the memory and CPU usage, especially when the app is idling ("minimized" in background, not killed).
I'm using Android Studio's profiler and here's what I have:

The vertical separator is the moment when I put my app in the background (openning another app like GMail or whatever).
Am I mistaken or the Memory usage should have go down at this point? I'm not sure about this.
What I don't understand is why do I have a peak in CPU usage and power on a regular basis?
Even if it's like 2-6%, it's still more than I would expect, as the App is currently not running?
Any clue on what is going on?

IIRC, Android apps don't release memory when enters background mode. But you can do it voluntarily using AppState events and release some memory.


Dedicated ignore file

Hi there!
ripgrep uses an .ignore file as well as a ripgrep-specific ignore file .rgignore in addition to the .gitignore for filtering out files to focus on. I'm currently using the ignore functionality but I would like to encode this as part of projects so others can benefit from the ignore but can't easily add it to the .gitignore since we need to commit a portion of the ignored files to version control. How hard would this be to add?
Thanks for the great tool!

Probably pretty easy. Would this be only respecting .ignore as well as .gitignore, or would we also want a .cargowatchignore (or .cwignore or something)?

I feel like .ignore has some history with sharing across other tooling like fd and ripgrep which might be helpful if you want all tools to share them? Otherwise, the advantage of having a dedicated .cargowatchignore or .cwignore is chiefly that we can explicitly ensure that the watch doesn't loop because of temporary files generated by a test, for example (my case, actually).
I could try to make the patch if you'd like the hand?

If you can, that would be ace :)

Sure thing, any guidance on how to get started specifically? I'll try cloning and looking around sometime next week in my spare time.

This will need upstream support, so you'll want to look at watchexec (where I'm also a maintainer!) first. Look at the watch() function in run.rs, that's where the .gitignore is loaded. The current interface only has a boolean (true to load .gitignore), that should probably be changed to some other structure to allow more than just that to be loaded. One idea would be a Vec<OsString> and just load the file by that name, but whatever makes sense.
If you want to add support to the watchexec CLI interface, you'll need to edit the args and pick some argument scheme; don't feel obligated to, though. In any case, you'll then need to open a small PR here adding support for your new API in watchexec.

Hi there! :wave: Sorry for the delay in getting up a PR.
The attached PR supports upstream by basically mimicking the .gitignore but making it distinct. This way you could specify a .ignore that would work across a family of tooling that isn't tied to what files you don't want checked into version control.
Let me know what you think! Happy to make tweaks when the time presents itself.

Haven't forgotten about this, just trying to find some time to get back to it. Last time I gave a go I didn't see a release upstream I could use at watchexec that included it, but I see there's a 1.11.1 release I might give a try.


Update request: github-desktop 3.1.1 → 3.2.1

Package name: github-desktop
Latest released version:

Current version on the unstable channel: 3.1.1
Current version on the stable/release channel: 3.0.6

[X] Checked the nixpkgs pull requests

Notify maintainers

@dan4ik605743 @LostAttractor

Note for maintainers: Please tag this issue in your PR.

https://github.com/desktop/desktop/releases/tag/release-3.2.1

#225611


a problem of '--debug' option

When I built my bundle using --debug options, I got messy code with Chinese characters in source maps, like this:

All my js files are UTF-8. Here is my command:
browserify --debug -r ./main.js > bundle.js
But it is all normal in the bundle.js:

This problem only happens when I look into the separated source files.
How can I fix it?

https://github.com/thlorenz/inline-source-map/issues/6#issuecomment-96078718

update combine-source-map to 0.7.1

upgate combine-source-map to 0.7.1 in browser-pack since inline-source-map fixed it on 0.6.0


CAPEX view

[ ] Only merged PRs should be included in CAPEX, not merged issues.
[ ] CAPEX should only include PRs with the status 'Done'.
[ ] 'Won't do' and 'duplicate' status should be ignored by CAPEX.
[ ] Github Actions should assign closed PRs to the current sprint.
[ ] Only closed and merged PRs should be added to the CAPEX board. PRs that are closed but not merged are not counted.
[ ] PR merged with no story point / PREQ (product request) set, should create issue for that deficiency.

PREQs can be expressed by issues
merged and closed PRs should be linked to PREQs issues in order to be included in CAPEX

problem is if there are multiple PRs linked to an issue, any of them closing will close the issue as well
we can instead, only mention PREQs issues from child issues as breaking PREQ to dedicated issues is obviously a right pattern

Github Actions should assign closed PRs to the current sprint.
Only closed and merged PRs should be added to the CAPEX board. PRs that are closed but not merged are not counted.
PRs without story points should fail.

are epics needed in the process?


82 wallet   single account template updates

Description
Adds the following to the secondary detail section of the account template:

Transactions
Tokens
Keys
Children (child accounts created by this account)

Note that the row expansion function of the transaction table (requested in issue #55) has been make the current styling possible.
Fixes #82
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Looks really solid :)

Solid Design!
Did various checks on accounts with and without complex key / permission structures and it seems to sum up correctly for treshold, weight, linkauth and parent/children relations.
Transaction Table is good looking.
Children: Awesome Feature, tested on both domain and standard accounts.
Tokens seems to be shown and design is eye candy. Missing logos though.
What standard are we using for kilo and bytes - ( kb is technically 1000 bits ) vs KB ( 1024 Bytes ) afaik ?

Thank you for the feedback @DestinyDevTeam.
Yes, I haven't added the token logos yet, but I think it would be best if we open a new issue for that. We still need to decide what token list to use (tokenmanager, T-Swaps or a new token list).
I think it would be more correct to switch to KB instead of kb. From the code it is clear that we read the values from the chain and just multiply by factors of 10 to get kilo/micro units. We don't check for bytes vs bits and I believe the values are stored on chain as bytes.


ZeroDownTime mode is providing an incorrect value when not specified

When using the Deploy nuget package (Optimizely DXP) task and not selecting a ZeroDownTime the argument provided to the powershell command is "-ZeroDowntimeMode Not specified" which causes an error as "Not specified" is not a valid value.

Hi @nickginis
Thanks for your report. Do you have some more information about which type of agent you are using? Is there any chance that you have a log that I can look at? That would help to find the issue.

Hey, We are using agent "windows-latest". Also, this appears to occur when you've first selected a value for ZeroDownTime such as "ReadOnly" and then change it back to a blank value. This causes the value to become "Not specified". I have attached a screenshot of what appears after changing the value back to blank.

@nickginis Thanks! I will look in to it ASAP. I have a new version coming out soon. I will try to fix it in next version. I will give you a note when it is live

Hi @nickginis
v2.2.12 should solve your issue. Please try and get back to me if you still have any problem.
Regards
Ove

Hey Ove,
The issue has been resolved. Thanks for the quick response!
Nick

Nice! I will close this issue then.


Feature request - marking types as deprecated.

Greetings,
I'd like ask you for a possibility to mark type as deprecated, so the flow will throw warning on places where given type is used.
eg.
// $FlowDeprecated
type SomeDeprecatedType = {
}

I'd love this too. And marking certain fields in an object type as deprecated. Flow could just warn in this case. Very useful for incremental refactoring.

I think it is better to have a $Deprecated utility class. As you can apply it either to the whole type (type Foo = $Deprecated<{field1: ..., field2: ...}> or only to a certain fields in the type (type Bar = {oldProp: $Deprecated<number>, newProp: number | string}).
It would be nice to have an additional message field, but that's optional and can be a part of a separate story. I mean this: type Bar = {oldProp: $Deprecated<number, "Use 'newProp' instead">, newProp: number | string}

+1
this would greatly improve the developer experience of our library

I :heart: idea, question is how this can be implemented?

@langpavel
type someType = {
  newName: string,
  oldName: $Deprecated<string>
}

@gre This actually works?

@gre Thanks, but it's different use case. We want throw warning when another developer is trying to use type which is deprecated.
In your case it's about marking deprecated attributes, by developer who decides or identifies or does kind of refactoring, where he decides to mark given type or type usage as deprecated.

@rostislav-simonik Right, point is that if flow find only path which use type marked as deprecated it should warn, so fix to my example must be then:
type someType = {
  newName?: string,
  oldName?: $Deprecated<string, "use the newName property instead">
}

And now I think that my original example was better. (mark entire type)
But in case that shape differs in many ways, marking properties may be more useful:
type Person = {
  name?: $Deprecated<string, "use fullName">,
  fullName?: string,
  age?: $Deprecated<number, "use bornYear">,
  bornYear?: number,
};

@langpavel yeah I was just refactoring your proposal to answer your question. why not, I was wondering if asking for a "reason" is a bit too much for Flow, I mean that feels a bit hacking the type system, but why not.
@rostislav-simonik I agree it's a bit different use-case but I don't think it's not incompatible. both could live at the same time. It is like in Scala language for instance, they have the @deprecated annotation and you can place it either on the whole class, or on a property of that class.
obviously deprecated a class means not using it at all, where deprecated a field means not using just the field of that class, that remain legit.
2 different usecases but same idea to me, just the implementation might be trickier for the property one.

@gre Hmm, thinking about property one, but it should be same in result, because you actually need to know if deprecated code path is used,
so the flag must be present on every possible flow type...
You can for example deprecate 2 or "END" in enum...

@gre Also, think about this:
type Person = {
  name?: $Deprecated<string, "use fullName">,
  fullName: string | $Deprecated<undefined, "fullName will be required in next release">,
  // ...
};

@gre Yes, both use cases are life capable and can be covered. I have just written to differentiate cases.

@rostislav-simonik yeah make sense. and i would totally understand if you want to split the thing into 2 separate issues &/or if flow team would like to give a go for the first simpler one.

@gre I'm not really convinced that this are two issues.
It may look as but I believe that it should be implemented as one feature with one code path,
I'm not really convinced that it can be splitted to more code paths in flow source…
Byt may be I'm just wrong with my mental model of flow internals :smiling_imp:

@langpavel two issues, two use cases, two implementations are 3 different things. I'm happy with any configuration if use case presented by us, will be covered. ;)

Would also love this feature

+1, would be great

As we have deprecated types for e.g. in react-native ( WebView.android.js ) with this feature it will be possible to migrate it to flow.

Has something like this been added yet? I can see this issue is still open 🤔
If someone can point me in the right direction, I'm happy to try add an implementation myself.

I'm looking for this feature -- is there any update?


No link

Towards the end of the article in Step 9 Verify the result, in step 1 it says "Add the geo to the selected attribute on a user. The list of available geos can be found in this table." but there is no table there. Probably you referring to table at the beginning of the article which has Geo and preferred data location value.
May be remove "The list of available geos can be found in this table." or provide the link
thanks
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b285f539-0487-0de5-24ef-89f773a1081d
Version Independent ID: b4f26f73-6b46-9703-692b-abe1f0d5f825
Content: Azure AD Connect: Configure preferred data location for Microsoft 365 resources
Content Source: articles/active-directory/hybrid/how-to-connect-sync-feature-preferreddatalocation.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

Hello @GarySurati, Thank you for your feedback! We will review and update as appropriate.

@GarySurati
Thank you for pointing this out! I've created a PR to update the documentation and it's currently waiting for the author to review and sign off.
I will now close out this issue. Please allow some time for the author to review and the changes to reflect.
Thank you for your time and patience!


Support resolving documentation links to content in other DocC archives

Bug/issue #, if applicable: rdar://114731067
Summary
This adds experimental support for resolving documentation links and symbol links to content in other DocC archives.
At a high level this external link support works in two steps:

While building the documentation for the first module, DocC emits two files, one that describe the link hierarchy and its possible disambiguations and one that contains minimal amount of content for every linkable page.
Later, while building the documentation for the second module, DocC is passed the documentation archive for the first module and reads these file and uses them to resolve links to content in the first archive.

[!IMPORTANT]
There is more work needed to support this end-to-end. If you run this now you'll notice that the external links are 404s in the browser unless you host both archives on the same server.

Dependencies
None
Testing
To test this you need a project with two documentable targets.

Build documentation for one of the targets and pass --enable-experimental-external-link-support to enable linking to this documentation.

Verify that the output directory contains a "linkable-entities.json" file and a "link-hierarchy.json" file.

In the other target, add links to symbols in the first target starting with a leading slash and the name of the first module, for example /FirstModule/MyClass/myFunction()

Build documentation for the other target and pass --dependency /path/to/first-target.doccarchive.

The link should resolve to display the documentation comment for myFunction() from the first target.

Change the external documentation link to introduce a typo. The external link should raise an error with a fix-it, same as it would for a local link.

Remove the "linkable-entities.json" a "link-hierarchy.json" files from the first archive and build documentation for the second target again.

There should be an error about missing files in the dependency.

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[x] Updated documentation if necessary

@swift-ci please test

I've tried to document how the link resolver gets encoded and decoded but I know that it's not as comprehensive as I would like it to be. Please let me know what pieces are most confusing and I'll try to add additional comments there.

@swift-ci please test

I realized that I forgot to call out one noteworthy thing about this implementation.
You may notice that this new external resolver doesn't conform to ExternalReferenceResolver or ExternalSymbolResolver and that this new external resolver doesn't return DocumentationNode values for its entities. From trying to use those protocols for this functionality—now and in the past—I've discovered that they cause significant problems, mostly because they return DocumentationNode values.
The issue with that is that DocumentationNode holds not-yet-rendered markup that needs to be rendered again in the local context. This means that links, formatting, and images in the original external content needs to be transformed into markup and be rendered again meaning that external references and external media need to be added to local context which introduce a risk for collisions between local content and external content. https://github.com/apple/swift-docc/issues/468 goes into more detail about these issues.
Since those are public protocols it would be a breaking change to modify their requirements. Instead I opted to create concrete types that are internal only. With the exception of LinkResolver.dependencyArchives I've kept all the new API internal only so that we are free to iterate on it without public breaking changes.
In the future we should deprecate OutOfProcessReferenceResolver, ExternalReferenceResolver, and ExternalSymbolResolver.

@swift-ci please test


BOMD Works on It's Own, But Not With Mods Like Ars

Mod Name
Bosses of Mass Destruction & Ars Nouveau
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/bosses-of-mass-destruction https://www.curseforge.com/minecraft/mc-mods/ars-nouveau
Describe the bug
I tried to load Bosses of Mass Destruction with Ars Nouveau and it crashes.
Steps to reproduce

Load BOMD with all required mod along with Ars Nouveau
It will crash

Logs
https://gist.github.com/ChrisLTheNerd/5c94d949e5b461e1a67ffdef8c05da45
Additional context
There's no crashes when BOMD is used along side mods like L_Ender 's Cataclysm [Forge], Alex's Mobs [Forge], Better Villages [Forge], Traverse [Fabric], Terrestria [Fabric], Sandwichable [Fabric], Better Archeology [Fabric], ext.
Ars uses GeckoLib internally, so I think that's the issue. At the time of this edit I see it might be a similar issue as #231, but I'm reporting it just in case.
Using "Connector-1.0.0-beta.13+1.20.1-full", Forgified Fabric API "resourcefulconfig-fabric-1.20.1-2.1.0", and Forge 47.1.3.

Same situation as #231 (ars bundling forge geckolib, which crashes when initialized from fabric mods), I'll keep track of it there


Left view of a binary tree in Python

💥 Proposal
Given a binary tree, print its left view using Python
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


:memo: Docs: Document actions returning errors as values

Description
Add documentation for errors as values
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #923

PR #924   👈

PR #817

This stack of pull requests is managed by Graphite.


Export XDG_ACTIVATION_TOKEN for apps launched by Hyprland

Currently firefox supports using this variable to activate itself. Without this, one has to manually switch to the workspace firefox is in when opening an url. While in theory this should be exported by the app launching firefox, currently there are many apps that don't do this. If the compositor supports this, one can easily workaround that by editing desktop files to use hyprctl dispatch exec.
This is implemented by sway in https://github.com/swaywm/sway/pull/6639, though I personally don't care much about workspace matching.

doesnt it already work with misc:focus_on_activate?

doesnt it already work with misc:focus_on_activate?

This only works if the app tries to activate itself, i.e. only works when XDG_ACTIVATION_TOKEN is set. If you launch firefox using foot's url-mode it will work, but it won't work when you just do firefox "https://google.com" in a terminal.

Of course, with misc:focus_on_activate = false none of the above will work, but that's not the case I'm talking about here.

the reason I keep it off and generally not work on it is that it was causing me more issues than switching workspaces to firefox. Maybe it's my shady impl, maybe not, but some apps would repeatedly spam activation a kjhillion times a second (e.g. obs)

but some apps would repeatedly spam activation a kjhillion times a second (e.g. obs)

How do I reproduce that?

open obs and start recording iirc

Yeah that's probably a qtbug or something, but it doesn't seem to cause any actual issue even with misc:focus_on_activate = true, and imo should not block implementing this.

it doesnt block it, but the 129 other issues don't make this a priority either. You're free and welcome to try making a patch yourself and making a MR.

but some apps would repeatedly spam activation a kjhillion times a second (e.g. obs)

I believe this no longer happens with qt 6.5.

doesnt it already work with misc:focus_on_activate?

This only works if the app tries to activate itself, i.e. only works when XDG_ACTIVATION_TOKEN is set. If you launch firefox using foot's url-mode it will work, but it won't work when you just do firefox "https://google.com" in a terminal.

with
misc:focus_on_activate = true
on firefox-nightly it jumps to the workspace where firefox is located if I call firefox-nightly "https://google.com" in a terminal

Still, telegram-desktop will get focused when a notification is received. Not great to be interrupted like that.

@fufexan @vaxerski Ilya Fedin, developer of telegram-desktop was surprised when I told him that Hyprland switches workspaces and focusing telegram-desktop every time notification arrives. They use QWindow::alert, here is a quote from https://github.com/telegramdesktop/tdesktop/issues/26343 not really related to this issue thread:

Apparently that's how your compositor reacts to QWindow::alert? It blinks in taskbar/dock on KDE (both X11/Wayland), Windows and macOS.

@q234rty

I believe this no longer happens with qt 6.5.

Today I migrated my system from qt5 to qt6 and stumbled upon this issue, when telegram grabs focus and switches workspaces every. damn. message.
I have dev-qt/qtbase and dev-qt/qtwayland v6.6.0 both.

Maybe we can reopen https://github.com/hyprwm/Hyprland/issues/2732 and migrate discussion related to telegram-desktop and obs there? They both qt and I don't think they do something wrong about this, maybe we treat their signals wrong? Just disabling misc:focus_on_activate is not a solution to the OP issue in my opinion. I still want to use this feature flawlessly in my daily work, for example, when I open links from terminal or my code editor, as #1350 OP

Today I migrated my system from qt5 to qt6 and stumbled upon this issue, when telegram grabs focus and switches workspaces every. damn. message. It was all good in qt5 since afaik it did not support focus request in wayland.

I was talking about the obs issue where activation requests where literally spammed, which was AFAIK fixed in https://invent.kde.org/qt/qt/qtwayland/-/commit/2fa2a5184dcdf54a7868d73418ed141713587969
Now, for telegram-desktop the easy workaround would be to disable "Draw attention to the window" in its notifications settings. Though if there would be activation spams from other apps, making focus_on_activate into a window rule could be a consideration.
The original issue about firefox mentioned here was "fixed" by gtk, which had an wrong implementation of xdg-activation causing wlroots-based compositors to reject it's request. See e.g. https://github.com/swaywm/sway/issues/4155 and https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/5293. Exporting XDG_ACTIVATION_TOKEN would still be useful for workspace matching if the application supports this environment variable.

Now, for telegram-desktop the easy workaround would be to disable "Draw attention to the window" in its notifications settings.

Oh, that's nice. I have enabled focus_on_activate back and disabled this feature in telegram, thank you :)


[Bugfix] Minor typo fix on Exercise Detail Page

Checklist

[ ] I tested all changes and all related features with different users (student, tutor, editor, instructor, admin)
[x] Client: I translated all newly inserted strings into English and German.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.

Motivation and Context

Solves an issue that I found during testing: #4023
Description

Steps for Testing

Log in to Artemis
Navigate to Exercise Detail page
Check for English and German if label for statistics is shown correctly

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

@jhainzlschmid thanks, so stupid 🤦🏽♀


Fix problems with new Electra structs

This PR fixes the issues discovered by testing with the Electra reference structures.

Add electra.AggregationAndProof structure.
Add electra.SignedAggregationAndProof structure.
Add electra.AttesterSlashing structure.
Add VersionedAttesterSlashing structure.
Add VersionedIndexedAttestation structure.
Fix ssz-size for CommitteeBits (it's 8 bytes not 1 bit vector, confusing).
Fix typo when unmarshaling consolidations.
In SignedConsolidation, make message a pointer.
Remove ExitsRoot from ExecutionPayloadHeader.
Use yaml (not json) struct tag for Consolidations.
Fix input to MerkleizeWithMixin for ExecutionPayload.

One of the tests is still failing, but I'm pretty sure the ref test is wrong here.
I sent this message to hww:

Extra data of 0x is an empty value.  In YAML strings don't have to be quoted, as far as I'm aware.  You will find that consensus and execution nodes are a little lax when it comes to formatting, so it's likely that you will have to handle that situation.

@mcdee I'm not entirely sure how to fix this 0x issue. I can fix our marshaled output, but the test code will marshal it to a string, which has a different value. It will marshal it to 0 (not 0x) there.
func testYAMLFormat(input []byte) string {
	val := make(map[string]any)
	if err := yaml.UnmarshalWithOptions(input, &val, yaml.UseOrderedMap()); err != nil {
		panic(err)
	}
        ...

For example, the following. The top line is the expected output, the bottom line is the actual output:

Any advice here?

Ah I found that you did this for Capella. I'll copy this:

Pushed a commit which handles that edge case. Also, Hsiao-Wei has acknowledged the issue. I'm going to make a PR to the consensus-specs to get that fixed. The fix might not make it into the next release though.


Mirror fallback timeout with TCP probing

cc/ @indirect
This PR implements #4128
Feature: support probing a mirror with a fallback timeout, if the probe fails, bundler drops down to the source url instead of using the mirror.
There are basically 2 ways of declaring a mirror:

bundler config mirror.all uri -> this will set a mirror for any uri provided, matching the new bundler behaviour of sending the upstream as a header to the mirror.
bundler config mirror.escaped_uri uri -> old style, setting up a mirror for a specific uri.

On top of this, there is a new option which allows declaring a fallback_timeout for any mirror:

bundler config mirror.all.fallback_timeout number|boolean
bundler config mirror.escaped_uri.fallback_timeout number|boolean

If a mirror has a fallback timeout set, bundler will probe a tcp connection to it, if the connection succeeds, bundler will use the mirror, otherwise bundler will call the original url skipping the mirror completely.
The fallback timeout supports both a number (float) and a boolean, if true is specified it will use the default timeout of 0.1 seconds, in case false is specified it will disable probing completely.
This maintains the old behaviour in the case probing is not setup and the mirror is not available, it will just fail as it was before.
There are a few caveats: this does not support partial configuration
For example: setting up many mirrors for specific urls and then trying to setup a fallback timeout for all will not apply the all timeout for the different specific urls.
All is treated as a specific uri that has priority over any other specific configuration. So these configurations may be ignored.
When a given mirror returns more than one address (ipv4+ipv6 for ex) all of them will be probed, this means that if a given mirror corresponds to 6 addresses, and all of them fail, there may be 6 probe timeouts, which means that the time it will take will be 0.6 seconds in the case that it is setup with the default timeout.
This was tested briefly (as it is basically a RFC for now) like this:
~$ bundler config mirror.all http://localhost:9292
~$ cat .bundle/config
---
BUNDLE_MIRROR__ALL: http://localhost:9292

...
make-me-an-artist(master)$ bundle
Fetching source index from http://localhost:9292/
Retrying fetcher due to error (2/4): Bundler::HTTPError Could not fetch specs from http://localhost:9292/
Retrying fetcher due to error (3/4): Bundler::HTTPError Could not fetch specs from http://localhost:9292/
Retrying fetcher due to error (4/4): Bundler::HTTPError Could not fetch specs from http://localhost:9292/
Could not fetch specs from http://localhost:9292/
...
~$ bundler config mirror.all.fallback_timeout true
~$ cat .bundle/config
---
BUNDLE_MIRROR__ALL: http://localhost:9292
BUNDLE_MIRROR__ALL__FALLBACK_TIMEOUT: 'true'
...
make-me-an-artist(master)$ bundle
Fetching gem metadata from https://rubygems.org/.........
Fetching version metadata from https://rubygems.org/..
Installing arrayfields 4.9.2
...

I'd love to see some YARD documentation for the new classes ;)

IMHO this also requires a integration-style spec that exercises the fallback path during a bundle install

I've added documentation for the new classes, hopefully improved rubocop feelings.
I only have a question regarding the integration test, where can I find a sample of how are they implemented in bundler?

Any of the specs that do install_gemfile and use an artifice option -- those are the ones that will let you  mock the HTTP server.

Right, I see how.
Will figure something out for the integration test, I have a few ideas already.

We might need to do a realworld test that boots a rack server -- unfortunately, artifice intercepts network requests in a way that doesn't allow testing some headers, so I think it won't allow testing sockets either. There's an example of booting a rack server in the realworld test for the X-Gemserver header.

Yeah, exactly what I was thinking.
Thanks

I've added a few real world end to end specs that setup a server using rack and a mirror (which is not there) just to assert that when there is no fallback timeout defined, bundler will fail, and when it is, bundler will reach the original url.
Both using the configuration specific for one mirror, or using the configuration for all

Great, sounds like exactly what we were looking for. I'm ready to merge this once the build is green. :)

Yay!
There is something odd with the TCPServer, apparently it can return a port that is still in use by other socket.
I restored the linear way of looking for an unused port.

@homu r+

:pushpin: Commit 51180f1 has been approved by indirect

:zap: Test exempted - status


Upgrade Go deps and fix go test error

Issue #, if available:
N/a
Description of changes:
- Adds missing argument to `Printf` in seccomp/utils.go which fixes
  error during `go test ./...`

Signed-off-by: John McBride<EMAIL_ADDRESS>
Builds / tests locally.
More in depth bottlerocket testing incoming!
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Patched this change into my local bottlerocket. Built a custom AMI and deployed to ECS. Then set the settings:
$ apiclient set settings.oci-hooks.log4j-hotpatch-enabled=true

Finally, deployed 2 ECS tasks that simulate a non-patched Java:

For non-root user:

ash-5.1# cat /var/lib/docker/containers/5327ac204ac955ee4d191301aefa7cabb88f7c717c70b0808538cb091f3ff25d/mounts/shm/hotdog.log
2023/01/10 22:01:46 Starting hotpatch
2023/01/10 22:01:46 Reducing capabilities to: "cap_chown,cap_dac_override,cap_dac_read_search,cap_fowner,cap_fsetid,cap_kill,cap_setgid,cap_setuid,cap_setpcap,cap_net_bind_service,cap_net_raw,cap_sys_chroot,cap_sys_ptrace,cap_mknod,cap_audit_write,cap_setfcap=ep"
2023/01/10 22:01:46 Constraining file descriptors
2023/01/10 22:01:46 Starting hotpatch after 0s delay
2023/01/10 22:01:47 Starting hotpatch after 1s delay
2023/01/10 22:01:47 Found java: 1
2023/01/10 22:01:47 dropping all capabilities and switching UID to 1000 for /usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>.1-1.fc35.aarch64/bin/java
2023/01/10 22:01:47 dropping all capabilities and switching UID to 1000 for /usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>.1-1.fc35.aarch64/bin/java
2023/01/10 22:01:47 Patch exited 0: ""

For root user:

ash-5.1# cat /var/lib/docker/containers/d5f802246f2c6c6e481feec80c8ecc2088f0a16309317d86e915b9aa2b20c54e/mounts/shm/hotdog.log
2023/01/10 22:05:25 Starting hotpatch
2023/01/10 22:05:25 Reducing capabilities to: "cap_chown,cap_dac_override,cap_dac_read_search,cap_fowner,cap_fsetid,cap_kill,cap_setgid,cap_setuid,cap_setpcap,cap_net_bind_service,cap_net_raw,cap_sys_chroot,cap_sys_ptrace,cap_mknod,cap_audit_write,cap_setfcap=ep"
2023/01/10 22:05:25 Constraining file descriptors
2023/01/10 22:05:25 Starting hotpatch after 0s delay
2023/01/10 22:05:26 Starting hotpatch after 1s delay
2023/01/10 22:05:26 Found java: 1
2023/01/10 22:05:26 setting caps to "cap_chown,cap_dac_override,cap_fowner,cap_fsetid,cap_kill,cap_setgid,cap_setuid,cap_setpcap,cap_net_bind_service,cap_net_raw,cap_sys_chroot,cap_mknod,cap_audit_write,cap_setfcap=ep" for "/usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>.1-1.fc35.aarch64/bin/java"
2023/01/10 22:05:26 setting IAB to "!cap_dac_read_search,!cap_linux_immutable,!cap_net_broadcast,!cap_net_admin,!cap_ipc_lock,!cap_ipc_owner,!cap_sys_module,!cap_sys_rawio,!cap_sys_ptrace,!cap_sys_pacct,!cap_sys_admin,!cap_sys_boot,!cap_sys_nice,!cap_sys_resource,!cap_sys_time,!cap_sys_tty_config,!cap_lease,!cap_audit_control,!cap_mac_override,!cap_mac_admin,!cap_syslog,!cap_wake_alarm,!cap_block_suspend,!cap_audit_read,!cap_perfmon,!cap_bpf,!cap_checkpoint_restore" for "/usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>.1-1.fc35.aarch64/bin/java"
2023/01/10 22:05:26 setting caps to "cap_chown,cap_dac_override,cap_fowner,cap_fsetid,cap_kill,cap_setgid,cap_setuid,cap_setpcap,cap_net_bind_service,cap_net_raw,cap_sys_chroot,cap_mknod,cap_audit_write,cap_setfcap=ep" for "/usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>.1-1.fc35.aarch64/bin/java"
2023/01/10 22:05:26 setting IAB to "!cap_dac_read_search,!cap_linux_immutable,!cap_net_broadcast,!cap_net_admin,!cap_ipc_lock,!cap_ipc_owner,!cap_sys_module,!cap_sys_rawio,!cap_sys_ptrace,!cap_sys_pacct,!cap_sys_admin,!cap_sys_boot,!cap_sys_nice,!cap_sys_resource,!cap_sys_time,!cap_sys_tty_config,!cap_lease,!cap_audit_control,!cap_mac_override,!cap_mac_admin,!cap_syslog,!cap_wake_alarm,!cap_block_suspend,!cap_audit_read,!cap_perfmon,!cap_bpf,!cap_checkpoint_restore" for "/usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>.1-1.fc35.aarch64/bin/java"
2023/01/10 22:05:27 Patch exited 0: ""

👍🏼


Test Bug: Isolated Service template will fail if the "name" variable is updated

Description
When testing the Isolated Service Single Region template, we modified the infra/templates/az-isolated-service-single-region/terraform.tfvars "name" value. This caused the unit_test.go tests to fail because they are hard-coded to expect the original "isolated-service" name value.
Acceptance Criteria
Reference: [Done-Done Checklist] (https://github.com/Microsoft/code-with-engineering-playbook/blob/master/Engineering/BestPractices/DoneDone.md)

[x] Should be able to change the Isolated Service terraform.tfvars name value without breaking unit tests

Resources

Related to #254

Tasks
Stories are intended to be completed in a single sprint; if task breakdown creates addition work then team should discuss promoting the Story to an Epic.
Reference: [Minimal Valuable Slices] (https://github.com/Microsoft/code-with-engineering-playbook/blob/master/Engineering/BestPractices/MinimalSlices.md)
Reference: [How to Write Better Tasks] (http://agilebutpragmatic.blogspot.com/2012/04/splitting-story-into-tasks-how-to-write.html)
Assignee should break down work into tasks here

Resolved as a part of larger work item #254


bugfixGet the file format by file name

Proposed changes
When file.format is not specified for the paimon table, the default file format will be used. In 0.8, the default format was orc, and in 0.9, it was changed to parquet.
Therefore, when there was no file.format and the paimon table was in version 0.9, Doris mistakenly used the orc reader to read the parquet file.
Now modify it to directly determine the file format based on the file name, and then use the corresponding reader.

run buildall

run buildall


Supporting another UI framework (i.e. SemanticUI)

Prerequisites

[x] I have read the documentation.

Description
I want to use this lib in a project that is built with SemanticUI instead of Bootstrap.
The documentation is pretty extensive in how to customize Froms to create aditional widgets, fields, themes, and so on. But the core components are (I guess) tight to Bootstrap HTML/classnames.
How should I proceed to use another UI library like semantic UI.
Can/should I for instance change the default widgets used for strings/dates/booleans/etc...

As far as I can see, SemanticUI relies on jQuery and mutable observers, which worries me at the highest level and is something we don't plan to support in any way in the future.
If you're only speaking of CSS here, and admitted that SemanticUI supports a pure CSS mode, the plan would indeed be to craft custom components (SchemaField, fields and/or widgets) to reflect the expected HTML semantics.

Hi. I am only interested in html and css part for now.
I will follow your recommendation. Tanks.

I think my question is related to this post.
In my case I'd like to use another UI lib, the https://ant.design/. How can I replace Bootstrap?

I'm looking at https://github.com/nilportugues/react-jsonschema-form-semanticui .. will try to circle back around and share what i find

@zenflow curious what your findings are


[Issue]: Helm Chart unavailable

Problem Description
The helm chart is not available anymore. Please add a note with the new helm repo url to the README and update the artifacthub links.
See: https://radeonopencompute.github.io/k8s-device-plugin/
I suppose its because of the switch from https://github.com/RadeonOpenCompute to https://github.com/ROCm
Operating System
Helm
CPU

GPU
AMD Radeon VII
ROCm Version
ROCm 6.0.0
ROCm Component
No response
Steps to Reproduce
Try to pull the helm chart
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

@y2kenny-amd Was there any announcement made about the changing of the url? It looks like it was just silently updated without notice to your customers/consumers, which is really bad UX.

@y2kenny-amd Was there any announcement made about the changing of the url? It looks like it was just silently updated without notice to your customers/consumers, which is really bad UX.

It was an organization level thing outside of my control.  I raised alarm about the potential issues but the guys thought the redirect from old org would work (it didn't.)


Create Palindromic Numbers.cpp

This is my C++ solution for problem Palindromic Numbers ( https://lightoj.com/problem/palindromic-numbers ).
If you have any query related solution contact me on<EMAIL_ADDRESS>I will try to help you.

I appreciate your effort. But please read the contribution guidelines at the repo readme.


Bug in npm run

This code:
        nct run 'Run arbitrary package scripts' {
            nct {
                param($wordToComplete, $commandAst)

                $scripts = (Get-Content .\package.json | ConvertFrom-Json).scripts
                $scripts | Get-Member -MemberType NoteProperty |
                    ForEach {
                        $target = $scripts.($_.Name)
                        New-CompletionResult $_.Name "$($target)"
                    }
            }
        }

needs to use $wordToComplete:
        nct run 'Run arbitrary package scripts' {
            nct {
                param($wordToComplete, $commandAst)

                $scripts = (Get-Content .\package.json | ConvertFrom-Json).scripts
                $scripts | Get-Member -MemberType NoteProperty |
                    Where-Object { $_.Name -like "${wordToComplete}*" } |
                    ForEach {
                        $target = $scripts.($_.Name)
                        New-CompletionResult $_.Name "$($target)"
                    }
            }
        }

Otherwise, if I type in something like npm run ele<tab> it ignores the ele and starts completion with whatever the first script is.

Fixed https://github.com/dfinke/NPMTabCompletion/commit/9ee6523a8fd3540e60d4419e0f1d7b9136712ad0

Thanks!

Thank you!


feat(V2): change the style of a  live codeblock

Motivation

change to

Have you read the Contributing Guidelines on pull requests?
yes i read
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

This will look more beautiful. Can it be modified like this?
@lex111 @slorber
This is not a complete implementation, I just wrote an example.  If there is no problem with this style, I will finish it

@jhoneybee thanks for your efforts, but the new design is very non-obvious to the end-user; I mean, most likely the user will not be able to understand there is actually a live editor. Therefore, hiding live editor is a bad idea.

@lex111 Thank you very much for your reply.
hiding live editor is a bad idea. but lt will reduce layout space.
I will think of other ways to solve. it won't take much time

@lex111

```jsx live expand

if have expand , show code editor .

@lex111  Is there any progress?

@lex111
this did not make you like this design, I am very sorry.  can you tell me the shortcomings of the design?  I will avoid this design

@jhoneybee I have provided example from Statex (see link in my previous comment). It seems to me that this is a more suitable option without significantly changing our existing design (which has already become familiar to many people).

@jhoneybee I have provided example from Statex (see link in my previous comment). It seems to me that this is a more suitable option without significantly changing our existing design (which has already become familiar to many people).

I understand.  thank you very much.


Renderer doesn't consider when pdf CropBox != MediaBox

Describe the bug
If you have a CropBox that is different from the overall page size it will crop part of the pdf within the editor/form/viewer (which is intended by the PDF spec).
However, schemas are placed according to these cropped positions, but when rendered on the PDF they will be misplaced.
PDF with CropBox different from MediaBox (white margin not shown in the editor):

Rendered:

To Reproduce

Upload a PDF with a custom CropBox
Add schemas
Render a preview
schemas will be misplaced.

Expected behavior
Schema's should be positioned as per your designs, taking into account the box differences.
Your Environment
- pdfme version: v5

Your Error Log
-

Additional context
No response

Maybe, related code is here: https://github.com/pdfme/pdfme/blob/eb80a1eb83678125b907eccbffa1b770be94f426/packages/generator/src/helper.ts#L135

ref:

https://github.com/Hopding/pdf-lib/issues/818
https://pdf-lib.js.org/docs/api/classes/pdfpage#getcropbox
https://pdf-lib.js.org/docs/api/classes/pdfpage#setcropbox


[zh-cn] Update replicaset.md

fix[replicaset.md]: Chinese translation is not accurate enough

Thanks.
/lgtm
/approve


Remove sharp dependency

sharp is not being used and it is related to 2 vulnerability issues. To keep code clean and solve this security issues it's better to just remove it from the dependencies list.

https://github.com/josemiguel-alvarez/tailwind-nextjs-starter-blog/security/dependabot/6
https://github.com/josemiguel-alvarez/tailwind-nextjs-starter-blog/security/dependabot/2

This PR also updates the lockFileVersion to v2. v2 is the currently supported version in the current Node LTS version. v2 is backwards compatible with v1.

sharp is recommended if you are hosting your own nextjs application. It's what vercel uses in production build as well. I might remove it for other reason - just to keep the download size smaller and those who want to self-host can install it themselves, but that will probably be for the next version release under pliny. I will probably not touch most of the dependencies in the current version.


Refactor Git metadata collection

Git metadata collection is refactored in order to:

always return some sort of data, even if an error occurs
remove fallback strategy attempting to resolve the last parent of a merge commit to a branch name (use the --branch or -b argument instead)

Also:

clean up the error message when no branch flag is provided or resolvable from Git

When uploading AppMaps from a directory that does not contain Git metadata, the following error occurs:
➜  tmp ~/source/appland/appland-cli/appland-cli upload ./rspec/DAO_User_authentication_is_accepted_with_a_valid_password.appmap.json
warn: Could not collect git metadata (repository does not exist)
  50% |████████████████████                    |  [0s:0s]panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x28 pc=0x1541a18]

goroutine 1 [running]:
github.com/applandinc/appland-cli/cmd.init.3.func1(0xc000152840, 0xc000126a70, 0x1, 0x1)
	/Users/kgilpin/source/appland/appland-cli/cmd/upload.go:83 +0x548
github.com/spf13/cobra.(*Command).execute(0xc000152840, 0xc000126a50, 0x1, 0x1, 0xc000152840, 0xc000126a50)
	/Users/kgilpin/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:846 +0x2aa
github.com/spf13/cobra.(*Command).ExecuteC(0x1b4e460, 0x0, 0x0, 0x0)
	/Users/kgilpin/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:950 +0x349
github.com/spf13/cobra.(*Command).Execute(...)
	/Users/kgilpin/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:887
github.com/applandinc/appland-cli/cmd.Execute()
	/Users/kgilpin/source/appland/appland-cli/cmd/root.go:51 +0x2d
main.main()
	/Users/kgilpin/source/appland/appland-cli/main.go:8 +0x20

So, I'd suggest that Git metadata be optional. And if there is no Git metadata, the Git options (url, branch, commit) are blank. The server will accept scenarios without Git metadata. The -branch option can be used to specify the branch, but it should not be an error if there is no Git metadata and no -b option is provided.

I've changed it so that
a.) we won't attempt to patch Git metadata into scenarios if the data is unavailable
b.) both commit and branch must both be either present or omitted. Presence of only one will cause an error.
I've also added the ability to override the app name via an --app/-a flag.


Please make sure that your uploaded "msms.txt" is also named "msms.txt"

Hello,
I submitted an msms.txt for CE calibration and received the above error for task 32B7C77BC3289AFD8C193081E9D2795C. I made sure the file had that name. Do you know what the error might be caused by? The msms.txt contains some PSMs from other raw files (I searched multiple raw files together but could only submit one to CE calibration), but I have done this before with no error
Thanks,
Kevin

Hi,
I'm having the same issue on the rescoring pipeline with the same error message. My msms.txt was definitely only searched from one raw file.
Thanks,
John.

Can you please add your task ID will check both tasks and let you know what is the issue.

I had two tasks with the same error:
7F1FA0DB0D03CBF1B9451A8EC94283FB
5B9619C6113CF8DEC809F82898468302

My Task ID was: 9BB2212CEBE769F01DC0543133B15DC8

I have checked both tasks for both the issue is that we assume for Cysteine that Carbamidomethylation is a fixed modification. For yangkl96 in the msms there was (Cysteinyl) after Cysteine amino acid in the peptide sequence not sure what this means for MQ output but we couldn't handle this. For jamc1996 there was (ca) after Cysteines I believe this is Carbamidomethylation and since we handle Carbamidomethylation as a fixed modification we assume that the C in the peptide sequence wouldn't have a modification in it.

That's great, thank you for your help!


README: Update plugins' list + descripions

Looks good! easy to read and the links direct where they need


Fix typo

@MikeWasson,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Thanks @MikeWasson
I'll be back in Seattle June 5th.

@MikeWasson,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


Restarting Pod on Config (state) change

In what area(s)?

/area runtime
Ask your question here
Currently, our team uses consul-template to pull config from consul and secrets from vault. Once consul-template detects a changed value, a SIGTERM is sent to the exec'ed application process and the new config is loaded.
Is it possible to do something similar with DAPR? There are certain config options in our application that only get read during application start. Additionally, by loading config at runtime, we don't have to spam an API (e.g., HTTP) to fetch config state for options that are used often.

If you are sending a SIGTERM to your application, you can send one to the daprd container as well. Or you can use the Dapr shutdown API to restart the daprd sidecar and cause a reload (assuming that your hosting platform will automatically restart the daprd container):
https://docs.dapr.io/operations/hosting/kubernetes/kubernetes-job/
Does that cover your use case?
Also, for more immediate community support, head on over to our community discord:
https://aka.ms/dapr-discord

well, consul-template was sending the sigterm - in this case, wouldn't dapr be taking over my consul-template-like operations and need to be aware to do this?

This is not stale - no clean solution exists currently

@adawalli can this now be done with the new Configuration API? your app can get notified whenever a config item changes.
https://docs.dapr.io/developing-applications/building-blocks/configuration/configuration-api-overview/