ListMultipartParts adds parameters to header instead of URL string

The function ListMultipartParts adds the limit and marker parameters to the header, instead of the URL string. So, the request doesn't recognize them properly.
I fixed it and my tests ensure that pagination now works. I added the code to the pull request in #11. I suspect that ListMultipartUploads and any other GET requests suffer from the same problem, but I don't use them so I haven't fixed them (perhaps you can take a look this weekend).
By the way, I have a pretty thorough multipart-uploading test suite I would like to add (takes 77s to run). It requires AWS credentials so I'll add them in a separate pull request, but I'm not sure what the best way to structure it would be (simply don't run the tests if the env variables aren't set?)

Thanks for fixing the header versus query string issue. I took a look at other methods, ListMultipartUploads gets it wrong too but ListJobs gets it right. Overall there isn't a consistent method to build the URL so I'm going to go through and probably clean them all up.
I'd love to add the tests. I'm not sure of a good way to structure it either. I often do small changes, test, repeat, and I wouldn't want them running every time. An environment variable seems okay. My only preference would have the tests disabled by default.

I added a parameter type (basically a wrapper around url.Values) in bc3edc23cfb61d8883df280355754057c58059fd and used it to build up parameters in a consistent manner everywhere.

Awesome, thanks!
2015-03-20 18:37 GMT-07:00 Robert Williamson<EMAIL_ADDRESS>
I added a parameter type (basically a wrapper around url.Values) in
bc3edc2
https://github.com/rdwilliamson/aws/commit/bc3edc23cfb61d8883df280355754057c58059fd
and used it to build up parameters in a consistent manner everywhere.
—
Reply to this email directly or view it on GitHub
https://github.com/rdwilliamson/aws/issues/14#issuecomment-84216344.


Feature Request: Update Cloudflare DNS Record Comment to Include Timestamp

Hello,
First of all, thanks for the great product, it's been working fabulously. I was wondering if it might be possible to add an environmental variable similar to RECORD_COMMENT that would include a date/time/timezone stamp in the DNS record comment. Bonus points if it is a configurable timestamp, but UTC ISO 8601 / RFC 3339 is more than sufficient.
e.g. something of the format [YYYY-MM-DDThh:mm:ssZ] Created by cloudflare-ddns... or whatever format suits your fancy @favonia.

@trjohnson19 Thanks for the suggestion. I think the difficulty is really to have a clean UI to configure the comments. On the other hand, for your use case, is it sufficient to use shoutrrr to tell you the change events? May I know the motivation about the timestamps? Knowing the motivations helps me a lot.

The use case is mostly similar to that of using shoutrrr for notifications. I’ve just not yet gotten around to setting up shoutrrr.
Basically, I had the issue where I was away from my house and my docker went down, so I checked Cloudflare to see the latest DDNS entry. It would have been quite clear at that point if I had seen the latest update hadn’t been for ~3–4 days that my stack had gone down.

@trjohnson19 Got it. What you need is probably a dead man's switch. One possible choice is https://healthchecks.io/. The idea is that the DDNS will constantly ping the website "hey I'm alive!" if it's working fine. When the website does not hear the "hey I'm alive!" for some time, it can notify you. Please read the section "Logging, Healthchecks, Uptime Kuma, and shoutrrr" about how to configure the feature.
If this seems to address your need perfectly, I might close this issue because it might not be easy to design an UI to configure the feature you requested.

That's fair, I can understand not wanting to add unneeded complexity when other options exist. Thanks for the recommendations!
Closing.

@trjohnson19 Well, I actually think the Healthchecks might address your need better! Let me know if it doesn't address all your need so that we can brainstorm something.


Why i don't have split thin lines between windows like the screenshot?

Can you share your config?

Closing due to inactivity


To connect with aws

what are the files that needs to be changed to connect with aws.
I am facing issues to ingest data to aws s3.
Can anyone please help?

@binodyadav6119 Can you add some more context on what exactly your issue is? I'm not sure what "files that needs to be changes to connect with aws" refers to exactly.
Generally speaking...
You need to specify an AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY environment variable. You can get these as defined in AWS documentation. AWS auth for delta-io/kafka-delta-ingest is defined by delta-io/delta-rs.


Modify python scripts so they can be called directly rather than downloaded.

Problem Description
The Python scripts for end repair are installed by downloading them from GitHub.
rule install_internal_scripts:
    output:
        end_repair=os.path.join(config.get("db_dir"), "rotary-" + VERSION, "scripts", "flye_end_repair.sh"),
        end_repair_utils=os.path.join(config.get("db_dir"), "rotary-" + VERSION, "scripts", "flye_end_repair_utils.py"),
        install_finished=os.path.join(config.get("db_dir"), "checkpoints", "internal_scripts_" + VERSION)
    log:
        "logs/download/install_internal_scripts.log"
    benchmark:
        "benchmarks/download/install_internal_scripts.txt"
    params:
        db_dir=config.get("db_dir"),
        url="https://github.com/jmtsuji/rotary/archive/refs/tags/" + VERSION + ".tar.gz"
    shell:
        """
        mkdir -p {params.db_dir}
        wget -O - {params.url} 2> {log} | tar -C {params.db_dir} -xzf - >> {log} 2>&1
        touch {output.install_finished}
        """

Because the scripts are already included in the repo, they should be installed during installation rather than downloaded from Github.
Problem Solution
Because all the scripts are being converted to Python, we can install them alongside rotary.py using our pyproject.toml file. However, it should be noted that the argsparse code must be placed in its function as I did in rotary.py. When installing via pyproject.toml, the resulting CLI scripts call functions in the Python files directly rather than the overall Python file.
[project.scripts]
rotary = "rotary.rotary:main"

See: https://github.com/jmtsuji/rotary/blob/99209d8158a6df68949fecd21180787562ab35b2/rotary/rotary.py#L34

@jmtsuji This was weird and confusing for me. So basically, when pip installs via pyproject.toml, it creates a script that calls the main() function in your Python code. Because of this, other overall Python script is not called, so you must include the args parsing code as a dedicated function called by main().
For an example, See: https://github.com/jmtsuji/rotary/blob/develop_lee/rotary/rotary.py

Yes, agreed, we should properly install repair.py rather than downloading it from Github. Thanks for the advice... I'll try to format the script like you mentioned, with a dedicated parser function called by main().

@LeeBergstrand Quick question about file organization:
Considering that we will probably only have Python code in the scripts folder now, would it be better to rename this folder something like modules? Or remove this folder entirely and move the Python code files directly into the main rotary folder, like is the case for utils.py etc.? As discussed in #23, my long-term hope is that the code in what is now the scripts folder will be directly runnable as modules via the rotary CLI. (I'm not that familiar with Python package management and the best way to arrange the files for that to happen.) Thanks!

@LeeBergstrand Quick question about file organization:
Considering that we will probably only have Python code in the scripts folder now, would it be better to rename this folder something like modules? Or remove this folder entirely and move the Python code files directly into the main rotary folder, like is the case for utils.py etc.? As discussed in #23, my long-term hope is that the code in what is now the scripts folder will be directly runnable as modules via the rotary CLI. (I'm not that familiar with Python package management and the best way to arrange the files for that to happen.) Thanks!

@jmtsuji I would move the Python code files directly into the main rotary folder for now.

@LeeBergstrand The repair.py module is now working (pending some further edge case handling and refactoring) in the end_repair branch. I'd like to add it to rotary's CLI so that it can be called through rotary in the snakemake rules. This allows me to call the repair module like this in the snakefile in rule assembly_end_repair:
Lines 352-355
rotary repair -l {input.qc_long_reads} -a {input.assembly} -i {input.info} -o {params.output_dir} \
          -f {params.flye_input_mode} -F {params.flye_read_error} -I {params.min_id} \
          -L {params.min_length} -e {params.ref_end} -E {params.reassemble_end} -t {threads} -m {resources.mem} \
          > {log} 2>&1

(Until the repair module is callable through the CLI, it's difficult to actually test if the overall workflow is actually working)
What do you think would be a suitable way to add repair.py to the CLI? This might need to be moved to a separate issue. Thanks so much.

@jmtsuji Adding it as a sub-command might be tricky as it would require calling repair.py from rotary.py. To support this change, I would likely refactor rotary.py so that the if else block for the subcommands is at the top and the args parsing is in a function for each command. Seems like a lot of work. We would have to do this refactoring to support https://github.com/jmtsuji/rotary/issues/23.
To get things working, we can add rotary repair directly as a script. For example, we can name it rotary-repair.
https://github.com/jmtsuji/rotary/blob/24f032885acc133f59422b87e17ff70ec1e69e65/pyproject.toml#L30
In the pyproject.toml you can add a new line to the scripts parameter.
rotary-repair = "rotary.repair:main"

Then, you can run pip install --editable . into the project directory, which should add a rotary-repair command to the current environment.

This is a workaround until the sub-commands can be appropriately implemented.

Addressed in https://github.com/jmtsuji/rotary/commit/1207cc02caa0c277acf33373a87ce2846546094a

rotary-repair = "rotary.repair:main"

name-of-script-in-cli = "package_name.script_name:function_to_call_in_script"
Thats how the above works. So main() is an CLI enterypoint.
If you look at the code inside rotary-repair it looks like this:
#!/Users/lee/miniforge3/envs/rotary/bin/python3.9
# -*- coding: utf-8 -*-
import re
import sys
from rotary.repair import main
if __name__ == '__main__':
    sys.argv[0] = re.sub(r'(-script\.pyw|\.exe)?$', '', sys.argv[0])
    sys.exit(main())

Great, this is also working on my end now -- let's use this as a temporary workaround. I think it's reasonable to close this issue now.


Fail to parse SSL certificate after upgrading to 0.10

kong version: 0.10 RC4
Operating System: centos

The SSL plugin has been removed and dynamic SSL capabilities have been added to Kong core, and are configurable via new properties on the API entity.

I tried to configure a certificate for an api using:
curl -X POST http://localhost:8001/certificates \
  -d "cert=/opt/App/kong/example.pem" \
  -d "key=/opt/App/kong/example.key" \
  -d "snis=example.com"

But it seems ssl.parse_pem_cert failed to parse my certificate:
2017/02/28 11:03:50 [error] 20690#0: *613 [lua] certificate.lua:15: log(): [ssl] could not parse PEM certificate: PEM_read_bio_X509_AUX() failed, context: ssl_certificate_by_lua*, client: <IP_ADDRESS>, server: <IP_ADDRESS>:443
2017/02/28 11:03:50 [crit] 20690#0: *612 SSL_do_handshake() failed (SSL: error:1408A179:SSL routines:ssl3_get_client_hello:cert cb error) while SSL handshaking, client: <IP_ADDRESS>, server: <IP_ADDRESS>:443

Before 0.10 it worked well using SSL plugins with the same certificate, and I've verified my certificate using openssl command:openssl req -text -noout -verify -in example.pem without any complain.

Either your certificate is invalid, either the value given to ssl.parse_pem_cert() is not what we expect it to be. Could you check what the configured cert is from the Admin API? And if possible, give us steps to reproduce this.

@thibaultcha Thanks for replay.
Certificate API shows as follow:
[
{
cert: "/opt/App/kong/example.pem",
id: "4b247df5-bef9-4b7a-bf0a-e8116d079f23",
snis: [
"example.com"
],
key: "/opt/App/kong/example.key",
created_at:<PHONE_NUMBER>000
}
]

Snis API shows as follow:
{
data: [
{
name: "example.com",
created_at:<PHONE_NUMBER>000,
ssl_certificate_id: "4b247df5-bef9-4b7a-bf0a-e8116d079f23"
}
],
total: 1
}

I'm pretty sure my certificate works well before upgreading to 0.10.

 You need to send the actual content of the file, aka the -F "file=@path" curl argument. Please let us know.

Maybe the Admin API could display that such certificates are invalid before insertion. Do you feel like contributing to such a feature?

@thibaultcha Thanks very much, it did solve my problem.
I would try to dig into it, hope I could help to contribute some day.

@suclogger   嗨, we also encountered the same problem with you, please let your is how to deal with, my email address is<PHONE_NUMBER> @qq.com, thank you

@fancy512 As @thibaultcha mentioned above, cert should be upload while requesting the certificate api rather than specific a path point to cert position.


Update Lucene to 8.3

Checklist

[x] User relevant changes are recorded in CHANGES.txt
[x] Touched code is covered by tests
[x] Documentation has been updated if necessary
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

@seut had to push a fixup


Doesn't scroll

I tried using it and pressed start and unchecked the fullscreen, it doesn't scroll when the video finishes

Been awhile!! I have updated the extension and have posted it on the chrome web store (should take around 3 days to be published)


Dump the effective config file

Suggestion
I'm using Typedoc which has very similar options to tsc and book keeping and converting these options is tedious and error prone. It would be nice if tsc could dump its effective compiler options (considering the default values, extends rules, etc.) as a json file. This can be particularly useful when debugging the configurations to see what config is effectively being used for the compilation process.
See: https://github.com/TypeStrong/typedoc/issues/480
TypeScript Version:  2.2.1 / nightly (2.2.0-dev.201xxxxx)
Expected behavior:
tsc --effectiveConfig
# Prints out a json object on STDOUT that is ready to be redirected to a JSON file
{
    "compilerOptions": {
        "module": "commonjs",
        "sourceMap": true,
        "target": "es2016",
        "noEmitOnError": true,
        "alwaysStrict": true,
        "importHelpers": true,
        "lib": [
            "es2015"
        ]
        ...
    }
}

I was looking for an option on how to output the effective config when I found this. I'm stuck since an hour trying to find out how extends actually extends stuff. I could only find in the docs:

files, include and exclude from the inheriting config file overwrite those from the base config file.
All relative paths found in the configuration file will be resolved relative to the configuration file they originated in.

But nothing about what happens to deeper arrays such as types or typeRoots, ... are they merged? how?
This is why I'm totally 👍 for something like Jest has with jest --showConfig, but for tsc.


Swap dokku to ubuntu

since it was never actually supported on rhel, and this is partially what led to a lot of the problems.
yes i am aware this will delete what is there.
then we can switch to ansible dokku

it should not replace the volumes, right?

No, just everything on root disk (ssh keys, etc.)

Oh there is no playbook for it, that was the point right?
Love the branch name btw :D

Please do not merge this. My changes need to be merged 1st. I will manually roll out this this evening and update you later. Converting this to a draft for now.

I rolled it out and then realized that many of our VMs, which were deployed with the regular cloud flavors, are currently not working due to this issue and unfortunately now this VM is also not reachable due to the same errors. I will update you as soon as the VM is online. However, a new VM has been spawned with Ubuntu 22.04 and the volume that was previously attached is also made available,
resource "openstack_compute_volume_attach_v2" "dokku-va" {
  instance_id = openstack_compute_instance_v2.dokku.id
  volume_id   = "6cbce9f7-1ffd-4848-9ec8-0a2ccfd52225"
  device      = "/dev/vdb"
}

I will close this PR and push my changes to my PR along with the state file updates.

Change commit: https://github.com/usegalaxy-eu/infrastructure/pull/195/commits/589cc317cb54ba2fd4bd8cb1a6ba13af52bce1b0

Your VM is up and running. The username is ubuntu. I see that no SSH keys are present, and only our cloud key is added. Please send me your SSH key so that I can add it to the VM directly.


Fix pod selecotr on prometheus rules

The pod selector is a regex and should use a regex matches.
The '=' operator matches strings literally where '=~' matches regex.
Signed-off-by: Daniel Belenky<EMAIL_ADDRESS>-->
Fix pod matchers in Prometheus rules

/lgtm

/retest

/test pull-kubevirt-e2e-kind-k8s-sriov-1.14.2

/test pull-kubevirt-e2e-k8s-1.15-ceph


Relation to customgpt.ai ?

Sorry for the confusion, is this project similar to customgpt.ai. Or related in any way?
Is this completely free? I red something about 90usd per year, what is that for?

This is not related to the project you shared. It is free to self-host but if you wish to use the cloud hosted version it is 9$/month or 90$/year


Please document that you need admin rights for everything

When I tried bootstrapping on my normal user shell, I ran into the following error:
> .\bootstrap-vcpkg.bat
Microsoft (R) Build Engine version <IP_ADDRESS>428 for .NET Framework
Copyright (C) Microsoft Corporation. All rights reserved.

Build started 05/03/2018 17:01:09.
     1>Project "C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\dirs.proj" on node 1 (default targets).
     1>Project "C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\dirs.proj" (1) is building "C:\UsersLocal\nikolaus.waxweile
       r\Development\vcpkg\toolsrc\vcpkg\vcpkg.vcxproj" (2:2) on node 1 (Build target(s)).
   2:2>Project "C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\vcpkg\vcpkg.vcxproj" (2:2) is building "C:\UsersLocal\nikol
       aus.waxweiler\Development\vcpkg\toolsrc\vcpkglib\vcpkglib.vcxproj" (3:4) on node 1 (default targets).
     3>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\CodeAnalysis\Microsoft.CodeA
       nalysis.targets(189,5): error : The source file for this compilation can be found at: "C:\windows\TEMP\4575a8c5-e28f-4288-b73b-
       f2b618e02b14.txt" [C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\vcpkglib\vcpkglib.vcxproj]
     3>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\CodeAnalysis\Microsoft.CodeA
       nalysis.targets(189,5): error MSB3758: An error has occurred during compilation. error CS2001: Source file 'C:\windows\TEMP\xuc
       uj4ii.0.cs' could not be found [C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\vcpkglib\vcpkglib.vcxproj]
     3>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\CodeAnalysis\Microsoft.CodeA
       nalysis.targets(189,5): error MSB3758: An error has occurred during compilation. warning CS2008: No source files specified [C:\
       UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\vcpkglib\vcpkglib.vcxproj]
     3>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\CodeAnalysis\Microsoft.CodeA
       nalysis.targets(189,5): error MSB4175: The task factory "CodeTaskFactory" could not be loaded from the assembly "C:\Program Fil
       es (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\Microsoft.Build.Tasks.Core.dll". The task factory must return
       a value for the "TaskType" property. [C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\vcpkglib\vcpkglib.vcxproj]
     3>Done Building Project "C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\vcpkglib\vcpkglib.vcxproj" (default targets)
       -- FAILED.
     2>Done Building Project "C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\vcpkg\vcpkg.vcxproj" (Build target(s)) -- FAI
       LED.
     1>Done Building Project "C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\dirs.proj" (default targets) -- FAILED.

Build FAILED.

       "C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\dirs.proj" (default target) (1) ->
       "C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\vcpkg\vcpkg.vcxproj" (Build target) (2:2) ->
       "C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\vcpkglib\vcpkglib.vcxproj" (default target) (3:4) ->
       (SetCABuildNativeEnvironmentVariables target) ->
         C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\CodeAnalysis\Microsoft.Cod
       eAnalysis.targets(189,5): error : The source file for this compilation can be found at: "C:\windows\TEMP\4575a8c5-e28f-4288-b73
       b-f2b618e02b14.txt" [C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\vcpkglib\vcpkglib.vcxproj]
         C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\CodeAnalysis\Microsoft.Cod
       eAnalysis.targets(189,5): error MSB3758: An error has occurred during compilation. error CS2001: Source file 'C:\windows\TEMP\x
       ucuj4ii.0.cs' could not be found [C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\vcpkglib\vcpkglib.vcxproj]
         C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\CodeAnalysis\Microsoft.Cod
       eAnalysis.targets(189,5): error MSB3758: An error has occurred during compilation. warning CS2008: No source files specified [C
       :\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\vcpkglib\vcpkglib.vcxproj]
         C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\CodeAnalysis\Microsoft.Cod
       eAnalysis.targets(189,5): error MSB4175: The task factory "CodeTaskFactory" could not be loaded from the assembly "C:\Program F
       iles (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\Microsoft.Build.Tasks.Core.dll". The task factory must retur
       n a value for the "TaskType" property. [C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\toolsrc\vcpkglib\vcpkglib.vcxproj]

    0 Warning(s)
    4 Error(s)

Time Elapsed 00:00:00.90
C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\scripts\bootstrap.ps1 : Building vcpkg.exe failed. Please ensure you have
installed Visual Studio with the Desktop C++ workload and the Windows SDK for Desktop C++.
At line:1 char:4
+ & {& 'C:\UsersLocal\nikolaus.waxweiler\Development\vcpkg\scripts\boot ...
+    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,bootstrap.ps1

Running it from an admin shell worked. Then I found that even running vcpkg requires admin rights. Is this intended? If so, please document it.

Are you sure? That was my first thought too, but the path in the kids looks like the user's home directory.

Huh? This is my home folder, I do all my development in a user shell there 😕

@madig: Have you maybe performed the initial clone from a admin shell? In any case, it would probably not hurt to recheck the permissions. Usually vcpkg doesn't require admin rights.

I'm not sure, but this is the reason when I met this problem.

The checkout was done in a user shell. I did a re-checkout in my user shell into the Development subfolder of my home directory, same problem. I noticed that the error message says "The source file for this compilation can be found at: "C:\windows\TEMP..." -- a folder I can't access without admin rights. Does vcpkg manage to put its' own files outside its' reach? 😀

Files in %TEMP% is generated by Visual Studio. So may build with visual studio in your environment needs admin privilige?

That's weird. I installed VS Community 2017 though the downloading utility you get from the site and elevated to admin rights when prompted. How did you do it?

It seems to work in my user shell after the latest VSC2017 update?! Alright then, closing this...


Retrieving a number field from an object does not convert 1.0 to 1

for this input:
{
"x" : 1.0
}
in try jsonata:
jsonata(1.0) -> 1
jsonata(x) -> 1
in java:
jsonata(1.0) -> 1
jsonata(x) -> 1.0 (it retains)

whenever we perform some calculations, this methods is applied:
com.dashjoin.jsonata.Utils.convertNumber(Number)
Which converts the number to int or long.
In this test case, the value is simply passed through. Note that using Json.parseJson("{"x":1.0}") instead of Map.of() also works, since the parser does the conversion from 1.0 to 1

Could be addressed in com.dashjoin.jsonata.Functions.validateInput(Object). The only option would be to error out, which seems like it's overkill in this case.
Alternatively, we could change the value. This has some performance implication though, since the input is immutable.


err="etherbase must be explicitly specified"

My base node, using snapshot settings, has also managed to read snapshot blocks.
I use command "docker compose up --build -d" over 6 hours ago, and it still hasn't started syncing!
I'm getting the following error:
"Served eth_coinbase conn=<IP_ADDRESS>:33410 reqid=2 duration="76.346µs" err="etherbase must be explicitly specified"
Any experienced folks out there who understand this, please help me out with an answer or a solution.

My other base node, without using snapshots,
is syncing blocks directly, which is very slow,
the synchronization is in progress.
Any experienced folks out there who know how to fix this using snapshot nodes...Tks!

i have the same issue after restart docker-compose

@ifsayyes Hi there! It looks like the issue here is that OP_GETH_SYNCMODE is set to snap and the rest of the snap-sync related parameters are commented out. I would recommend you are on the latest version (the head of this repo) and also using the current .env.mainnet link.
Note, if your node is for production purposes snap-sync mode most likely won't be stable enough as the feature is new and experimental.
If you're still experiencing an issue after making the above changes, please join the Base Discord and open a ticket in the #developer-support channel.
We can take a closer look their to see what the issue might be.
@ifsayyes Because the snap sync feature is new and experimental, there are not enough peers at present which is most likely what is leading to the latency on your sync.
Hope that helps clarify, you all! Please definitely drop by Base Discord and LMK (wbnns) if you all are still having a problem. We can help you there.


Support qtvirtualkeyboard module

i add module name in qt-modules.txt and sucessfully install that and now can run example code on my rpi
but that is support only English for supporting other language i should use "qmake-rpi" with this argument "CONFIG+=lang-all" ( base on answer in this link link )how do this? should i change ./init-qtrpi-full.sh code?

Thanks for your feedback. It's a good news that the qtvirtualkeyboard module works.
Currently all Qt modules from qt-modules.txt are built in the same way without any arguments.
Check the file compile-qt-modules.sh:

The for loop (end of the file) iterates on all Qt modules (on the text file) and calls the function build_qtbase if the module is qtbase otherwise it calls  build_module
The function build_module uses the function qmake_cmd (defined in common.sh)
The function qmake_cmd performs the "real" qmake:

$ROOT/raspi/qt5/bin/qmake -r |& tee $ROOT/logs/$LOG_FILE.log

Based on your link, I guess you want to do something like this only for the "qtvirtualkeyboard" module:
$ROOT/raspi/qt5/bin/qmake "CONFIG+=lang-all" -r |& tee $ROOT/logs/$LOG_FILE.log

So I guess you can update qtrpi in multiple ways to achieve this. For example you can check if the module name is "qtvirtualkeyboard" in the file compile-qt-modules.sh and do your specifc qmake command with the arguments.
If you are proud of your enhancement, you can create a Pull Request, after a review time we will merge it on the develop branch.


train code

Could you provide training code?

@hhds233, this work is a funded project, so we have only released evaluation and inference scripts for now. We shall release the training script in the future. However, for now, you can follow the implementation details for training from the paper. As we have provided the class for GDIP-Yolo, importing it along with the hyperparameters mentioned in the paper should work.
Let me know if you need any help with that. Closing the issue as of now.


Make schema name for the CTA queries and limit configurable

Description
This PR implements an ability to configure the logic that generates the schema name for the create table as queries based on the database, user, used schema and executed SQL.
There are 2 new parameters in the config, both are off by default.
# Function accepts database object, user object, schema name and sql that will be run.
SQLLAB_CTA_SCHEMA_NAME_FUNC = (
    None
)  # type: Optional[Callable[["Database", "User", str, str], str]]
# Flag that controls if limit should be enforced on the CTA (create table as queries).
SQLLAB_CTA_NO_LIMIT = False

Use cases

automatically put sensitive data into the sensitive schema and that data is accessed
put user CTA results into the user personal schema, it can be extended to the team schema's as well

Other changes

get_query_with_new_limit function is modified no to apply limit if the user limit is lower, e.g. if user runs SELECT * FROM user LIMIT 1 - it would return 1 row, not 1000
fixed bug for the Postgres database where CTA results were not committed and table was not created e.g. https://github.com/apache/incubator-superset/blob/df2ee5cbcb8bc6e5cd310b9b9015509db744a256/tests/celery_tests.py#L175
created additional schemas in travis for testing purposes
removed unused SQL_SELECT_AS_CTA field from the test config.

Tests
[x] unit test + integration tests
[x] tested locally on mysql & postgres
[x] is used in production @ dropbox for over a month

@mistercrunch big thanks for the comments. Sorry for putting this PR prematurely, I wanted to use CI to figure out how to deal with postgres & mysql schema creation and investigating some bugs. I was a bit lazy to set up it on my fork :)  I've resolved all the comments and will do a bit more testing & deploy @ dropbox prior to the next review round to make sure it will be more polished & ready to be used.
Promise to publish more mature PRs going forward.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #8867   +/-   ##
=======================================
  Coverage   58.97%   58.97%
=======================================
  Files         359      359
  Lines       11333    11333
  Branches     2787     2787
=======================================
  Hits         6684     6684
  Misses       4471     4471
  Partials      178      178

Continue to review full report at Codecov.

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

@bkyryliuk this needs a rebase

@villebro  - done, thanks !

@villebro & @willbarrett  - resolved the comments & rebased

As this PR includes a db migration, it would be nice to merge soon if there are no objections. I'll leave this open for a few more days, then merging if no further comments surface.


Question about taggedPosts

The docs say to get tagged photos:
client.taggedPosts(tag, options, callback);
My questions:

What are the option parameters we are able to feed into options? That wasn't entirely clear from the docs.
I need to see what info the photos in callback will contain. In my case I need to search for multiple tags so get all photos with 1 tag, then in the callback can I see all the tags a photo has per photo?
Is there any example of a callback somewhere?

The options are documented at https://www.tumblr.com/docs/en/api/v2#tagged-method

The response of the callback is a list of posts.
The example below will perform a search for the tag #dog limited to 10 posts and output the blog name, post id and all tags for each post that is a photo post:

var tumblr = require('tumblr.js');
var client = tumblr.createClient({ consumer_key: '<enter your key here>' });

client.taggedPosts('dog', {limit: 10}, function(err, data) {
    for (var i = 0; i < data.length; ++i) {
        var post = data[i];

        if (post.type === 'photo') {
            console.log(post.blog.name, post.id, post.tags.join(', '));
        }
    }
});

@jasonpenny Wow! Great support! Thanks! If you don't hear back from me it means everything worked perfectly!
Have a great day!


Read credentials from aws config

Changed the s3 url signing to read the credentials from the aws prescribed shared location and moved remaining settings out of secrets.

Coverage increased (+0.001%) to 95.708% when pulling a32a07e3d3db681f6d276c3da88ae0b17247d89d on DEC-1171-Read-from-s3-config into 679c966bec26524e48da1c52c309b38fefc56519 on master.

👍


Missing details about custom query params.

This is related to Send authentication requests section.
I had a use case where, client application pass a custom param and access that value via custom policies. This SO question  is about that. It turns out AAD B2C support, passing custom query parameters along with authorize request.
I think it would be better to have that info in this article. Can we include that piece of details? 🤔

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

ID: c54adceb-9510-aeb5-0c61-c0d052459cda
Version Independent ID: 0ff6b6b2-2802-9534-8963-796b9c4a2a76
Content: Web sign-in with OpenID Connect - Azure Active Directory B2C
Content Source: articles/active-directory-b2c/openid-connect.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@SampathDilhan Thank you for your query. We will investigate and update the thread.

Thanks for you feedback, good point. I'll update the document with the relevant information.

@SampathDilhan You can pass custom query string by using InputClaims in a given Technical Profile. In below example, I am passing login_hint and hsu parameters in the request for External Azure AD tenant added as IDP.
         </CryptographicKeys>
         <InputClaims>
               <InputClaim ClaimTypeReferenceId="login_hint"<EMAIL_ADDRESS>/>
	       <InputClaim ClaimTypeReferenceId="hsu" DefaultValue="1" />
	 </InputClaims>
         <OutputClaims>


Rewrite in Go

Rewrite the entire project in Go.
Markdown parser: https://github.com/russross/blackfriday
For pygments, find alternatives or drop shell to Python like https://github.com/pksunkara/pygments/blob/master/pygments.go

Perhaps something like https://github.com/terryh/gopdf/blob/master/wkhtmltopdf.go for PDF?


Improving handling of string templates in the Java syntax.

This patch ports:
https://github.com/apache/netbeans/pull/6745
into this extension.

Looks like the PR is merged in NB 21

This can be closed once NB 21 PR is merged


Allow custom build task to specify a custom metadata file path.

This accomodates for cases where the name of the repository does not match up with the metadata file

shipped in 0.0.27


ctrl + r  opens recent instead of refreshing reload menu

Version: 1.35.1
OS: Linux x64 5.0.0-19-generic

Steps to Reproduce:

ctrl + shift + p and search for reload window ( Note: On the right hand side, the keybinding assigned for reload window is ctrl + r)
try ctrl + r . this opens recent window, instead of refreshing workspace.

Video demo here.

Does this issue occur when all extensions are disabled?: Yes

You can change the keybinding to Reload Window command from the keybindings editor.


V1.0.0

Current coverage is 88.88% (diff: 88.80%)

No coverage report found for master at 4b9553f.

Powered by Codecov. Last update 4b9553f...42a30b8


Aarch64/Arm64 Linux version

It would be nice to have an aarch64/arm64 Linux binary as part of the releases. Then people with Raspberry Pis or users of Asahi Linux could also use the program.

As far as I can tell, only this line would have to be edited:
https://github.com/bee-san/Ares/blob/71973ddddb50074ca405f09431458c3ffadc6d27/Cargo.toml#L74
Changed to: targets = ["x86_64-unknown-linux-gnu", "aarch64-unknown-linux-gnu", "x86_64-apple-darwin", "x86_64-pc-windows-msvc", "aarch64-apple-darwin"]
I'm not an expert, though.

Linux aarch64 isn't supported by cargo-dist. The four target platforms in Cargo.toml are all the supported ones by cargo-dist.
https://opensource.axo.dev/cargo-dist/book/config.html?highlight=targets#targets

I don't want to maintain a separate workflow just for Linux aarch64. You should make an issue on cargo-dist's repo. I'm closing this as wontfix


iroh-net: Concurrent connect does not work

I wanted to write some code that dials by peer id. Before putting in a long term solution I wanted to just dial all current region ids at the same time (we only got 1 and 2).
But it seems that dialing concurrently causes the dialing to not work at all.
The dialing code is below. If I set concurrency to some value >1 I don't get any successful dials. See log.
Code:
    fn dial_by_peer<'a>(
        &'a self,
        peer: &'a PublicKey,
        alpn: &'a [u8],
    ) -> BoxFuture<'a, anyhow::Result<quinn::Connection>> {
        println!("dialing {} in regions {:?}", peer, self.regions);
        async move {
            use futures::stream::StreamExt;
            let mut attempts = futures::stream::iter(self.regions.iter().cloned())
            .map(|region| {
                let addr = PeerAddr {
                    peer_id: *peer,
                    info: AddrInfo {
                        derp_region: Some(region),
                        direct_addresses: Default::default(),
                    },
                };
                self.endpoint.connect(addr, alpn)
            })
            .buffer_unordered(4);
            let mut err = None;
            while let Some(conn) = attempts.next().await
            {
                match conn {
                    Ok(conn) => return Ok(conn),
                    Err(e) => err = Some(e),
                }
            }
            Err(match err {
                Some(err) => err,
                None => anyhow::anyhow!("no regions to connect to"),
            })
        }
        .inspect(|res| {
            println!("dial result {:?}", res.as_ref().map(|_| ()));
        })
        .boxed()
    }

Log (concurrent dials):
❯ cargo run --example tracker server
   Compiling iroh v0.7.0 (/Users/rklaehn/projects_git/iroh/iroh)
    Finished dev [unoptimized + debuginfo] target(s) in 2.53s
     Running `target/debug/examples/tracker server`
tracker starting using /Users/rklaehn/Library/Application Support/iroh_tracker
listening on PeerAddr { peer_id: PublicKey(dwxjty7awokmscy5), info: AddrInfo { derp_region: Some(2), direct_addresses: {<IP_ADDRESS>:44204, <IP_ADDRESS>:44204} } }
peer addr: dwxjty7awokmscy5v6pnyb26nw54ftzddcxukfqfc3kbbdskrk4a
dialing gbyrelvzrmp4r32tl2maeo5pqwv226qfcivznnyraircnrjh7sbq in regions [1, 2]
dial result Err(failed connecting to provider

Caused by:
    timed out)
2023-10-17T08:59:33.447592Z ERROR tracker: error dialing peer gbyrelvzrmp4r32tl2maeo5pqwv226qfcivznnyraircnrjh7sbq: failed connecting to provider
dialing gbyrelvzrmp4r32tl2maeo5pqwv226qfcivznnyraircnrjh7sbq in regions [1, 2]
dial result Err(failed connecting to provider

Caused by:
    timed out)

log (sequential dials):
❯ cargo run --example tracker server
   Compiling iroh v0.7.0 (/Users/rklaehn/projects_git/iroh/iroh)
    Finished dev [unoptimized + debuginfo] target(s) in 2.31s
     Running `target/debug/examples/tracker server`
tracker starting using /Users/rklaehn/Library/Application Support/iroh_tracker
listening on PeerAddr { peer_id: PublicKey(dwxjty7awokmscy5), info: AddrInfo { derp_region: Some(2), direct_addresses: {<IP_ADDRESS>:44204, <IP_ADDRESS>:44204} } }
peer addr: dwxjty7awokmscy5v6pnyb26nw54ftzddcxukfqfc3kbbdskrk4a
dialing gbyrelvzrmp4r32tl2maeo5pqwv226qfcivznnyraircnrjh7sbq in regions [1]
dial result Ok(())

Not so sure if this is really about concurrency. If I change the code to sequentially dial, and then put region 2 first (the node is in region 1), it also does not work.
It almost seems as if once you connect in one region you can not dial in another region anymore.

Could you PR a test for this? should be pretty straightforward to use two local derpers in iroh-net and connect to them maybe twice one after the other

side note: using FuturesUnordered for sth like this is exactly what got us into problems like #1646

I don't think this is about FuturesUnordered. It seems that it is not really possible to dial two regions at the same time due to the fact that the PeerMap always has 1 region per peer.
If you dial concurrently with [2, 1] to a node in region 1, things work. The region gets set to 2 and then immediately get set to 1 again. Then some stuff happens, and you are connected.
If you dial concurrently with [1,2] it does not work. See log below. It's basically last writer wins for which derp region you are in, and then of course things go wrong since the last region we are in is not where the target peer is.
iroh on  tracker [$!?] via 🦀 v1.72.0
❯ RUST_LOG=info cargo run --example tracker server
   Compiling iroh v0.7.0 (/Users/rklaehn/projects_git/iroh/iroh)
    Finished dev [unoptimized + debuginfo] target(s) in 2.32s
     Running `target/debug/examples/tracker server`
tracker starting using /Users/rklaehn/Library/Application Support/iroh_tracker
2023-10-17T10:02:32.801824Z  INFO magicsock{me=dwxjty7awokmscy5}:actor: iroh_net::magicsock: home is now derp-2 (default-2)
2023-10-17T10:02:32.815069Z  INFO magicsock{me=dwxjty7awokmscy5}:derp.actor: iroh_net::magicsock::derp_actor: adding connection to derp-2 for home-keep-alive
listening on PeerAddr { peer_id: PublicKey(dwxjty7awokmscy5), info: AddrInfo { derp_region: Some(2), direct_addresses: {<IP_ADDRESS>:44204, <IP_ADDRESS>:44204} } }
peer addr: dwxjty7awokmscy5v6pnyb26nw54ftzddcxukfqfc3kbbdskrk4a
dialing gbyrelvzrmp4r32tl2maeo5pqwv226qfcivznnyraircnrjh7sbq in regions [1, 2]
dialing gbyrelvzrmp4r32tl2maeo5pqwv226qfcivznnyraircnrjh7sbq in region 1
dialing gbyrelvzrmp4r32tl2maeo5pqwv226qfcivznnyraircnrjh7sbq in region 2
add_peer_addr: PeerAddr { peer_id: PublicKey(gbyrelvzrmp4r32t), info: AddrInfo { derp_region: Some(1), direct_addresses: {} } }
2023-10-17T10:02:32.816223Z  INFO magicsock{me=dwxjty7awokmscy5}:actor:add_known_addr: iroh_net::magicsock::endpoint: inserting peer's endpoint in PeerMap peer_id=gbyrelvzrmp4r32tl2maeo5pqwv226qfcivznnyraircnrjh7sbq
add_peer_addr: PeerAddr { peer_id: PublicKey(gbyrelvzrmp4r32t), info: AddrInfo { derp_region: Some(2), direct_addresses: {} } }

This is probably still not working, but given that we now have derp urls the original reason for doing this is kinda gone, so closing this.


Have video not extend further than width

@jaydenseric Made a pull request for this but it was somehow overwritten so it is not there anymore. I also saw that you made some changes. I recreated it. This is what I got and it works. I also added the YouTube privacy version of the url.
(function ( window, document, undefined ) {

	/*
	 * Grab all iframes on the page or return
	 */
	var iframes = document.getElementsByTagName( 'iframe' );

	/*
	 * Loop through the iframes array
	 */
	for ( var i = 0; i < iframes.length; i++ ) {

		var iframe = iframes[i],

		/*
	     * RegExp, extend this if you need more players
	     */
		players = /www.youtube.com|www.youtube-nocookie.com|player.vimeo.com/;

		/*
		 * If the RegExp pattern exists within the current iframe
		 */
		if ( iframe.src.search( players ) > 0 ) {

			/*
			 * Calculate the video ratio based on the iframe's w/h dimensions
			 */
			var width             = iframe.width;
			var height            = iframe.height;
			var videoRatio        = ( height / width ) * 100;

			/*
			 * Replace the iframe's dimensions and position
			 * the iframe absolute, this is the trick to emulate
			 * the video ratio
			 */
			iframe.style.position = 'absolute';
			iframe.style.top      = '0';
			iframe.style.left     = '0';
			iframe.width          = '100%';
			iframe.height         = '100%';

			/*
			 * Wrap the iframe in a new
 which uses a
			 * dynamically fetched padding-top property based
			 * on the video's w/h dimensions
			 */
			var wrap              = document.createElement( 'div' );
			wrap.className        = 'video-wrap';
			wrap.style.width      = '100%';
			wrap.style.position   = 'relative';
			wrap.style.paddingTop = videoRatio + '%';

			/*
			 * Add the iframe inside our newly created

			 */
			var iframeParent      = iframe.parentNode;
			iframeParent.insertBefore( wrap, iframe );
			wrap.appendChild( iframe );

			/*
			 * Wrap the div in a new
 which is used
			 * to set a max width accordin to the video's dimensions
			 */
			var outerWrap        = document.createElement( 'div' );
			outerWrap.className  = 'outer-video-wrap';
			outerWrap.style.maxWidth   = width + 'px';

			/*
			 * Add the div wrap inside our newly created

			 */
			var wrapParent       = wrap.parentNode;
			wrapParent.insertBefore( outerWrap, wrap );
			outerWrap.appendChild( wrap );

		}

	}

})( window, document );

It blows my mind that this problem still persists on the interwebs. Too many plugin developers out there calling in this script redundantly without taking a look at the code. All I know is, Fitvids.js & Fluidvid.js is bloating my websites while at the same time adding unnecessary padding to my videos. The sad fact is that these scripts just dont work. Eventually the vendors & browsers will handle this, and @jaydenseric 's methods are simple yet superior to the other solutions online.


Selection API - calling .get() after selection returns the original data, having to call .copy() before .get() is easy to miss

const level8 = await ArxMap.fromOriginalLevel(8, settings)
const selection = new Box3(new Vector3(0,0,0), new Vector3(10, 10, 10))
const polygons = $(level8.polygons).selectWithinBox(selection).get()

console.log(polygons.length) // this returns 1277, all polygons from level 8 are copied

const level8 = await ArxMap.fromOriginalLevel(8, settings)
const selection = new Box3(new Vector3(0,0,0), new Vector3(10, 10, 10))
const polygons = $(level8.polygons).selectWithinBox(selection).copy().get() // <---- .copy() called before .get()

console.log(polygons.length) // this returns 0 correctly

This is super easy to miss!

Also: some methods work without calling .selectAll() beforehand, like .move(). This needs to be normalized.

Every method now has a jsdoc block and every method seem to work in a consistent way, so I'm closing this


Add Blueprints to collection view section

Project URL
https://github.com/zenangst/Blueprints
Category
Table View / Collection View
Description
Add Blueprints to collection view section
Why it should be included to awesome-ios (optional)
Thought it would be a nice addition to the list as collection views can be hard to get right and this framework tries to fix that problem.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 4 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          2 Warnings

:warning:
Too many changes (when adding, please keep it to one project per Pull Request)

:warning:
Found 6 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

254
301
https://github.com/alfa-laboratory/YARCH-Examples redirects tohttps://github.com/alfa-laboratory/YARCH

761
301
https://github.com/ReactiveKit/Bond redirects tohttps://github.com/DeclarativeHub/Bond

788
301
https://github.com/AbsurdAbstractions/Layoutless redirects tohttps://github.com/DeclarativeHub/Layoutless

917
301
https://github.com/bahlo/SwiftGif redirects tohttps://github.com/swiftgif/SwiftGif

1335
301
https://github.com/ReactiveKit/ReactiveKit redirects tohttps://github.com/DeclarativeHub/ReactiveKit

2725
301
https://github.com/JetpackSwift/FrameworkTemplate redirects tohttps://github.com/RahulKatariya/FrameworkTemplate

  Generated by :no_entry_sign: Danger


Fix button alignment in ThemeSwitch component

Fix button alignment in ThemeSwitch component
This PR fixes the alignment issue of the button within the ThemeSwitch component. The button was not properly centered within its container, causing it to appear misaligned in the navigation bar. This fix ensures that the button is vertically centered.
Changes Made:

Applied flex layout and alignment classes to the div containing Menu.Button in ThemeSwitch.tsx.

Before:

After:

Thanks for the fix! In the future, do run the code through prettier before committing.


[Summer 2021] Create models for Server Side Template Injection

Server side template injection is a very impactful vulnerability which can lead to Remote Code Execution (https://portswigger.net/research/server-side-template-injection).
Currently we have defined some of them (jinja) in the general.pysa file. We should move them to a specific file e.g. server_side_template_injection_sinks.pysa  and cover other libraries like

django.template https://docs.djangoproject.com/en/3.2/topics/templates/
mako https://github.com/sqlalchemy/mako

We should make sure to have pysa models for the functions which can allow arbitrary input in the template definition leading to RCE. Documentation at https://pyre-check.org/docs/pysa-basics

@PulkitMishra  Heads up someone internally worked on the django.template recently (sorry didn't saw that coming). But we can cover https://chameleon.readthedocs.io/en/latest/ instead :)

oh @r0rshark was almost done with the diff:/
Nvm. can you please block this one. Will send a PR on chameleon in some time?

@PulkitMishra  had a chat internally with the owner of the task and it was not finished and just created a draft so feel free to implement the django.template models as well :)

FYI there are a bunch of other templating engines that could be worth covering:
https://youtu.be/R1KrPc1IOeE?t=1213

@gbleaney thanks. I guess from that list it's the following that we need to cover:

Chameleon
Cheetah
Genshi
Trender
Chevron
Airspeed

I will check this and get back. Nice talk btw!

Thanks for sharing @gbleaney . Will send a PR over in some time or if you suggest can continue adding commits to this as well https://github.com/facebook/pyre-check/pull/458
@r0rshark can you please let me know in case any of these are being worked on internally?

@PulkitMishra  none internally is working on those. Feel free to add them to your Pull request :)


Deleting SetDisplayNameAndIcon from test and demo apps

During AppNotification registration for unpackaged apps, we don't retrieve icon/displayName from window anymore. Hence, in this PR we remove the sample code used to set icon and display name using Shell APIs.

/azp run

I presume this isn't needed anymore because the functionality is now part of the WIndowsAppSDK or the UDK, right? What about the Preview2 fork, does it still need this code?

It's not needed because the functionality to support icons using Shell property store is gone, and the code we have now in place will provide at least the default icon.


[EXP]: collect research town logs

Description

idea
paper
review

Additional Information
No response

supported by 0.0.1


sql where conditions are uncertain， how to deal with

like this
there is some uncertain conditions, you can search use : name, id ,or other fields
eg: select * from user where {name = "jack " or id = 1 ..}
can i custom sqlc

Hi @nobody05,
I answered a similar question in #1062. Can you have a look at it and tell me if it answers your problem?

thanks


Use non-serif font

Serif fonts look ugly when the text size is relatively small, IMO it would be much better to use something less distracting and detailed.

I'd suggest a Google webfont.


ERROR: Service 'nginx' failed to build: The command '/bin/sh -c apt-get update --fix-missing && apt-get install -y cron' returned a non-zero code: 100

When I run 'docker-compose up --build' it provides me
'Service 'nginx' failed to build: The command '/bin/sh -c apt-get update --fix-missing && apt-get install -y cron' returned a non-zero code: 100'
My host is mac os 14.XX.
How do I solve that？

same issue here. Tried with debian, ubuntu, nothing works

I got the same error!... any updates on this?

I solved it by changing the nginx version to nginx:1.15.12 in nginx/Dockerfile

i have already proposed a solution (#146 ) for this

Closed by #146.


Xception model training error

hey i m new in transfer learning and currently trying to use Xception model …heres what i m doing …
_ZooModel zooModel = Xception.builder()
.workspaceMode(WorkspaceMode.ENABLED)
.cacheMode(CacheMode.DEVICE)
.numClasses(2) .seed(randomSeed) .build();
zooModel.setInputShape(new int[][] {{3,150,150}});
zooModel.init();
DL4JResources.setBaseDownloadURL("https://dl4jdata.blob.core.windows.net/");
ComputationGraph pretrainedNet = (ComputationGraph) zooModel.initPretrained();
FineTuneConfiguration fineTuneConf = new FineTuneConfiguration.Builder() .optimizationAlgo(OptimizationAlgorithm.STOCHASTIC_GRADIENT_DESCENT)
.updater(new Nesterovs(5e-5)) .seed(123) .build();
ComputationGraph xception = new TransferLearning.GraphBuilder(pretrainedNet)
.fineTuneConfiguration(fineTuneConf) .setFeatureExtractor("avg_pool")
.removeVertexKeepConnections("predictions")
.addLayer("predictions", new OutputLayer.Builder(LossFunctions.LossFunction.NEGATIVELOGLIKELIHOOD)
.nOut(2) .weightInit(WeightInit.XAVIER)
.activation(Activation.SOFTMAX).build(), "avg_pool")
.setOutputs("predictions")
.setInputTypes(InputType.convolutional(150, 150, 3)) .build();
System.out.println(xception.summary());_
heres my Training phase:
TransferLearningHelper transferLearningHelper = new TransferLearningHelper(xception); transferLearningHelper.fitFeaturized(dsi);
on training i m getting this error :
Exception in thread "main" java.lang.IllegalStateException: Invalid input, does not match configuration: expected [minibatch, numChannels=2048, inputHeight=1, inputWidth=1] but got input array of shape [100, 3, 150, 150]
what should i do now …There's no documentation on Xception …i used Xception.summary() function to track my vertices .I have :
output classes=2
Img input dimension are (150,150 ,3)
Num. of training data =3000
Num of batchSize=150 …guide me please?
I Have also ping this issue on Deeplearning4j forum but NO response .  Suggest Me ?

Closing this. Support questions should be on the forums: https://community.konduit.ai/


php8 error

hello auther:
building in php version:8.0.0
Problem 1
- Root composer.json requires php ^7.1.3 but your php version (8.0.0) does not satisfy that requirement.
Problem 2
- laravel/framework[v5.8.0, ..., 5.8.x-dev] require php ^7.1.3 -> your php version (8.0.0) does not satisfy that requirement.
- Root composer.json requires laravel/framework 5.8.* -> satisfiable by laravel/framework[v5.8.0, ..., 5.8.x-dev].

Upgrade laravel to v8.0


Add Contributor Covenant 1.4

As every prime open source project even this shining specimen needs to have rules how the thriving community around it should behave. The only right set of code of conduct rules is obviously Contributor Covenant.

This seems to conflict with some of the wording in the License (https://github.com/TvdW/ruler-of-exchange/blob/master/LICENSE). Once I get a lawyer to look at it I will choose whether to accept or deny this pull request.

Wof


fix holdout test vs. control explanation

🚀 Deployed on https://62d97d5c0f695c028c8e4bd9--cozy-fox-0defec.netlify.app


App Crash

the app crashes when I try to load my character file.
Here's the SaveFile:
FE44DC5E4DF893F262CDF3A0FD249DD5.zip

same i cant load my savefile

Latest version has now been published here on github. https://github.com/CutFlame/MCDSaveEdit/releases
I was able to load your file with the latest version of MCDSaveEdit


add helm upgrade test to ci

Describe the solution you'd like
after #108 is merged,
test that installs helm chart using a specific version then upgrades to latest chart is manifest_staging
example: https://github.com/open-policy-agent/gatekeeper/blob/master/.github/workflows/upgrade.yaml

@sozercan Should this run e2e tests, or just deploy the Helm chart then upgrade? Right now the e2e tests looked to be pretty reliant on a YAML deployment of Eraser.
If the upgrade is successful, should this workflow promote the helm chart from staging to the charts dir? Or will that be a manual step?

Also, you can assign me or @joheec

@lee0c i also opened #199 for a regular helm test

@sozercan Has a chart been published to the github pages repo yet? To fully build this out that is required.

no, this will require a new release to be cut. #199 might be easier one to get started with


broken link in MinnowBoard documentation

Describe the bug
The link is broken for the website to update the MinnowBoard firmware.
To Reproduce
Click the link referenced in Zephyr Docs (https://docs.zephyrproject.org/latest/boards/x86/minnowboard/doc/index.html#booting-zephyr-on-the-minnowboard):
MinnowBoard firmware website - https://minnowboard.org/tutorials/updating-the-firmware
Expected behavior
Active resource, such as https://software.intel.com/en-us/articles/minnowboard-maxturbot-uefi-firmware
Impact
Need to search the web for the correct repository and documentation.

Looking at the suggested link in Expected Behavior above, perhaps more appropriate would be:
https://github.com/tianocore/edk2-platforms/blob/master/Platform/Intel/Vlv2TbltDevicePkg/Readme.md


Check openTelemetry deployment files and create a deployment tutorial for these files

The helm-otel folder of this repository contains files/templates for openTelemetry deployments. We need to check if they still work and we should create documentation on how to deploy these files.
Additionally, we can then link this documentation in the monitoring guide of the architecture browser

See also https://github.com/devonfw/devon4j/issues/494


Build fails on FreeBSD as of latest commit

Describe the bug
The build fails on FreeBSD as of the latest commit (updating from 918ce1dd674f99a41545c50272507a275ca53baf to b2dee07b5e513058c58ad608a8e6880179f6759c)
To Reproduce
Steps to reproduce the behavior:

git pull --ff
cargo build --release

Expected behavior
Successful build
Logs
...
   Compiling segment v0.2.2
The following warnings were emitted during compilation:

warning: In file included from c_src/mimalloc/src/static.c:37:
warning: In file included from c_src/mimalloc/src/prim/prim.c:22:
warning: c_src/mimalloc/src/prim/unix/prim.c:172:184: error: use of undeclared identifier 'hint'; did you mean 'link'?
warning:         _mi_warning_message("unable to directly request aligned OS memory (error: %d (0x%x), size: 0x%zx bytes, alignment: 0x%zx, hint address: %p)\n", err, err, size, try_alignment, hint);
warning:                                                                                ^~~~
warning:                                                                                link
warning: /usr/include/unistd.h:351:6: note: 'link' declared here
warning: int      link(const char *, const char *);
warning:          ^
warning: 1 error generated.

error: failed to run custom build command for `libmimalloc-sys v0.1.32`

Caused by:
  process didn't exit successfully: `/root/meilisearch/target/release/build/libmimalloc-sys-4080080ea4344f77/build-script-build` (exit status: 1)
  --- stdout
  OPT_LEVEL = Some("3")
  TARGET = Some("x86_64-unknown-freebsd")
  HOST = Some("x86_64-unknown-freebsd")
  cargo:rerun-if-env-changed=CC_x86_64-unknown-freebsd
  CC_x86_64-unknown-freebsd = None
  cargo:rerun-if-env-changed=CC_x86_64_unknown_freebsd
  CC_x86_64_unknown_freebsd = None
  cargo:rerun-if-env-changed=HOST_CC
  HOST_CC = None
  cargo:rerun-if-env-changed=CC
  CC = None
  cargo:rerun-if-env-changed=CFLAGS_x86_64-unknown-freebsd
  CFLAGS_x86_64-unknown-freebsd = None
  cargo:rerun-if-env-changed=CFLAGS_x86_64_unknown_freebsd
  CFLAGS_x86_64_unknown_freebsd = None
  cargo:rerun-if-env-changed=HOST_CFLAGS
  HOST_CFLAGS = None
  cargo:rerun-if-env-changed=CFLAGS
  CFLAGS = None
  cargo:rerun-if-env-changed=CRATE_CC_NO_DEFAULTS
  CRATE_CC_NO_DEFAULTS = None
  DEBUG = Some("false")
  CARGO_CFG_TARGET_FEATURE = Some("fxsr,sse,sse2")
  cargo:rerun-if-env-changed=CC_x86_64-unknown-freebsd
  CC_x86_64-unknown-freebsd = None
  cargo:rerun-if-env-changed=CC_x86_64_unknown_freebsd
  CC_x86_64_unknown_freebsd = None
  cargo:rerun-if-env-changed=HOST_CC
  HOST_CC = None
  cargo:rerun-if-env-changed=CC
  CC = None
  cargo:rerun-if-env-changed=CFLAGS_x86_64-unknown-freebsd
  CFLAGS_x86_64-unknown-freebsd = None
  cargo:rerun-if-env-changed=CFLAGS_x86_64_unknown_freebsd
  CFLAGS_x86_64_unknown_freebsd = None
  cargo:rerun-if-env-changed=HOST_CFLAGS
  HOST_CFLAGS = None
  cargo:rerun-if-env-changed=CFLAGS
  CFLAGS = None
  cargo:rerun-if-env-changed=CRATE_CC_NO_DEFAULTS
  CRATE_CC_NO_DEFAULTS = None
  CARGO_CFG_TARGET_FEATURE = Some("fxsr,sse,sse2")
  running: "cc" "-O3" "-ffunction-sections" "-fdata-sections" "-fPIC" "-m64" "-I" "c_src/mimalloc/include" "-I" "c_src/mimalloc/src" "-Wall" "-Wextra" "-ftls-model=initial-exec" "-DMI_DEBUG=0" "-o" "/root/meilisearch/target/release/build/libmimalloc-sys-0fb61fc61da7cd89/out/c_src/mimalloc/src/static.o" "-c" "c_src/mimalloc/src/static.c"
  cargo:warning=In file included from c_src/mimalloc/src/static.c:37:
  cargo:warning=In file included from c_src/mimalloc/src/prim/prim.c:22:
  cargo:warning=c_src/mimalloc/src/prim/unix/prim.c:172:184: error: use of undeclared identifier 'hint'; did you mean 'link'?
  cargo:warning=        _mi_warning_message("unable to directly request aligned OS memory (error: %d (0x%x), size: 0x%zx bytes, alignment: 0x%zx, hint address: %p)\n", err, err, size, try_alignment, hint);
  cargo:warning=                                                                                ^~~~
  cargo:warning=                                                                                link
  cargo:warning=/usr/include/unistd.h:351:6: note: 'link' declared here
  cargo:warning=int      link(const char *, const char *);
  cargo:warning=         ^
  cargo:warning=1 error generated.
  exit status: 1

  --- stderr

  error occurred: Command "cc" "-O3" "-ffunction-sections" "-fdata-sections" "-fPIC" "-m64" "-I" "c_src/mimalloc/include" "-I" "c_src/mimalloc/src" "-Wall" "-Wextra" "-ftls-model=initial-exec" "-DMI_DEBUG=0" "-o" "/root/meilisearch/target/release/build/libmimalloc-sys-0fb61fc61da7cd89/out/c_src/mimalloc/src/static.o" "-c" "c_src/mimalloc/src/static.c" with args "cc" did not execute successfully (status code exit status: 1).

warning: build failed, waiting for other jobs to finish...

Meilisearch version:
b2dee07b5e513058c58ad608a8e6880179f6759c
Additional context
Additional information that may be relevant to the issue.
FreeBSD, x86_64, Rust 1.68.2

Hey, could you try to build this project to see if you still have the issue: https://github.com/irevoire/mimalloc-rust

Builds fine
 Downloaded mimalloc v0.1.37
  Downloaded libc v0.2.145
  Downloaded libmimalloc-sys v0.1.33
  Downloaded 3 crates (1.9 MB) in 0.54s (largest was `libmimalloc-sys` at 1.2 MB)
   Compiling libc v0.2.145
   Compiling cc v1.0.79
   Compiling libmimalloc-sys v0.1.33
   Compiling mimalloc v0.1.37
   Compiling mimalloc-rust v0.1.0 (/root/mimalloc-rust)
    Finished release [optimized] target(s) in 7m 23s
➜  mimalloc-rust git:(main) ✗

Still able to reproduce after a fresh clone of Meilisearch... :(

Hello @ThatOneCalculator
Sorry for the inconvenience
@oluademola, can you take care of this please? Because @irevoire has been absent for an undetermined time 😢

Thank you for the offer, but I already have a search index built up on my own VPS.

Thank you for the offer, but I already have a search index built up on my own VPS.

This is the fix
https://github.com/microsoft/mimalloc/commit/bcbcc343ec658382bc6ab59615590c1cdb75ef6e

Oh! Is there a reason that's not upstreamed into meilisearch?

@ThatOneCalculator and @feld sorry for the delay, I had a challenge spinning up the FreeBSD vm to test this. Thank you for letting us know about the fix. @curquiza or @irevoire, what do you think about it being upstreamed to main? Thanks.

Hey @ThatOneCalculator,

Oh! Is there a reason that's not upstreamed into meilisearch?

That’s because in meilisearch we’re using the rust wrapper over mimalloc: https://github.com/purpleprotocol/mimalloc_rust
It would be nice if you could open an issue on their side explaining where is the fix and once they release it we’ll be able to integrate it to meilisearch!
Thanks for your investigation.

Oh! Is there a reason that's not upstreamed into meilisearch?

not sure, but as a quick fix find that file in your ~/.cargo and patch it by hand, then build again and you can get your system back online

It would be nice if you could open an issue on their side explaining where is the fix and once they release it we’ll be able to integrate it to meilisearch!

I opened this https://github.com/purpleprotocol/mimalloc_rust/issues/95

The mimalloc_rust crate hasn't been touched in a couple months. Would it be a good idea for Meilisearch to fork it to keep the base up to date?

Severity: low severity, FreeBSD is not in the list of officially supported systems
Extra small to small: just replace with the default allocator on FreeBSD or wait for the commit on mimalloc to land, or update mimalloc-rust

@ThatOneCalculator Meilisearch uses mimalloc v0.1.36 while mimalloc v0.1.37 contains the fixes.


cups-driverd list consumes lots of RAM

Describe the bug
/usr/lib/cups/daemon/cups-driverd list 1 0 requested-attributes='ppd-make' call consumes substantial amount of RAM (150+ MiB) to enumerate the PPDs and generate 1.5KB of output with the list of printer manufacturers.
To Reproduce
Steps to reproduce the behavior:

Open CUPS admin web interface: http://localhost:631/admin/
Click on 'Add Printer'
Select your printer or try to add a dummy network printer
Navigate up until "Name:, Description:, Location:" dialog
Press "Continue"
Watch /usr/lib/cups/daemon/cups-driverd list 1 0 requested-attributes='ppd-make' process consumes 150+ MiB of RAM just to generate the list of available printer driver manufacturers.
After selecting printer manufacturer and pressing continue button, watch /usr/lib/cups/daemon/cups-driverd list 1 0 requested-attributes=all ppd-make='PRINTER_MANUFACTURER' process consume 150+ MiB once again to generate the list of available drivers of a selected manufacturer.

Expected behavior
Sane memory consumption.
System Information:

OS and its version: Debian 12
CUPS version 2.4.2-3+deb12u4

Additional context
"Add Printer" dialog is a challenge due to this issue for a memory-constrained systems like the SBC with 256 MiB of RAM I run CUPS on.

There's no memory leak per se:
==793242== HEAP SUMMARY:
==793242==     in use at exit: 142,200,527 bytes in 33,028 blocks
==793242==   total heap usage: 842,942 allocs, 809,914 frees, 643,089,997 bytes allocated
==793242==
==793242== LEAK SUMMARY:
==793242==    definitely lost: 0 bytes in 0 blocks
==793242==    indirectly lost: 0 bytes in 0 blocks
==793242==      possibly lost: 0 bytes in 0 blocks
==793242==    still reachable: 142,200,527 bytes in 33,028 blocks
==793242==         suppressed: 0 bytes in 0 blocks
==793242== Reachable blocks (those to which a pointer was found) are not shown.
==793242== To see them, rerun with: --leak-check=full --show-leak-kinds=all
==793242==
==793242== For lists of detected and suppressed errors, rerun with: -s
==793242== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

Does the driver removal work for you?

Yes, with less drivers it consumes less memory. However I believe it should not consume that much anyway, there's a bug somewhere. There's just nothing which could consume hundred of megabytes for this task.

The line numbers correspond to git master commit 082c5ae12a72889312452716efed3cf8833e3518 version.
$ valgrind --leak-check=full --show-leak-kinds=all ./cups-driverd list 1 0 requested-attributes='ppd-make'

…
==798094== 9,155,712 bytes in 1,356 blocks are still reachable in loss record 54 of 57
==798094==    at 0x484782C: calloc (vg_replace_malloc.c:1554)
==798094==    by 0x10C974: add_ppd(char const*, char const*, char const*, char const*, char const*, char const*, char const*, char const*, long, unsigned long, int, int, char const*) (cups-driverd.cxx:234)
==798094==    by 0x10DB48: load_ppd(char const*, char const*, char const*, stat*, ppd_info_t*, _cups_file_s*, long) [clone .constprop.0] (cups-driverd.cxx:2271)
==798094==    by 0x10ED62: load_ppds(char const*, char const*, int) (cups-driverd.cxx:2531)
==798094==    by 0x10EB4F: load_ppds(char const*, char const*, int) (cups-driverd.cxx:2443)
==798094==    by 0x10F4F3: list_ppds(int, int, char const*) (cups-driverd.cxx:1087)
==798094==    by 0x10C793: main (cups-driverd.cxx:194)
==798094==
==798094== 11,228,576 bytes in 1,663 blocks are still reachable in loss record 55 of 57
==798094==    at 0x484782C: calloc (vg_replace_malloc.c:1554)
==798094==    by 0x10C974: add_ppd(char const*, char const*, char const*, char const*, char const*, char const*, char const*, char const*, long, unsigned long, int, int, char const*) (cups-driverd.cxx:234)
==798094==    by 0x10F113: load_drv (cups-driverd.cxx:1671)
==798094==    by 0x10F113: load_ppds(char const*, char const*, int) (cups-driverd.cxx:2546)
==798094==    by 0x10EB4F: load_ppds(char const*, char const*, int) (cups-driverd.cxx:2443)
==798094==    by 0x10F4A1: list_ppds(int, int, char const*) (cups-driverd.cxx:1059)
==798094==    by 0x10C793: main (cups-driverd.cxx:194)
==798094==
==798094== 38,277,088 bytes in 5,669 blocks are still reachable in loss record 56 of 57
==798094==    at 0x484782C: calloc (vg_replace_malloc.c:1554)
==798094==    by 0x10C974: add_ppd(char const*, char const*, char const*, char const*, char const*, char const*, char const*, char const*, long, unsigned long, int, int, char const*) (cups-driverd.cxx:234)
==798094==    by 0x10DB48: load_ppd(char const*, char const*, char const*, stat*, ppd_info_t*, _cups_file_s*, long) [clone .constprop.0] (cups-driverd.cxx:2271)
==798094==    by 0x10ED62: load_ppds(char const*, char const*, int) (cups-driverd.cxx:2531)
==798094==    by 0x10EB4F: load_ppds(char const*, char const*, int) (cups-driverd.cxx:2443)
==798094==    by 0x10EB4F: load_ppds(char const*, char const*, int) (cups-driverd.cxx:2443)
==798094==    by 0x10EB4F: load_ppds(char const*, char const*, int) (cups-driverd.cxx:2443)
==798094==    by 0x10F46E: list_ppds(int, int, char const*) (cups-driverd.cxx:1056)
==798094==    by 0x10C793: main (cups-driverd.cxx:194)
==798094==
==798094== 77,533,216 bytes in 11,483 blocks are still reachable in loss record 57 of 57
==798094==    at 0x484782C: calloc (vg_replace_malloc.c:1554)
==798094==    by 0x10C974: add_ppd(char const*, char const*, char const*, char const*, char const*, char const*, char const*, char const*, long, unsigned long, int, int, char const*) (cups-driverd.cxx:234)
==798094==    by 0x10FB7D: load_drivers (cups-driverd.cxx:1887)
==798094==    by 0x10FB7D: list_ppds(int, int, char const*) (cups-driverd.cxx:1161)
==798094==    by 0x10C793: main (cups-driverd.cxx:194)
==798094==
==798094== LEAK SUMMARY:
==798094==    definitely lost: 0 bytes in 0 blocks
==798094==    indirectly lost: 0 bytes in 0 blocks
==798094==      possibly lost: 0 bytes in 0 blocks
==798094==    still reachable: 142,200,503 bytes in 33,028 blocks
==798094==         suppressed: 0 bytes in 0 blocks
==798094==
==798094== For lists of detected and suppressed errors, rerun with: -s
==798094== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

Currently each PPD take 6744 bytes (assuming 64-bit time_t and off_t types) of memory to hold all of the information needed. We could realize some savings by dynamically allocating most of the string fields, but at the expense of processing time, which is already substantial... We could also look at limiting the sizes of the ppd_rec_t structure (the 6744 byte one) by examining current drivers and deriving smaller limits for these field, although I don't know whether that would yield substantial savings (maybe reduce the total size to ~5k?)

Yes, just added debug printf to add_ppd and see this:
!!! add_ppd (called 20939 time), ppd_info_t size: 6752, already allocated: 141380128

Is there any reason to get all the information about all the ppds first, and only then process it? A lot of memory could be saved if the loading and processing is performed in sequence, this way we don't need to store each ppd_info_t in memory.

@ValdikSS Since we cache the PPD information, we at least need to store all of it. Search results (from the "list" action) need to be sorted, which is best/most efficiently done in memory. And any time you list all available drivers ("lpinfo -m") you'll need to sort all 20k+ drivers on a typical distro...
Concatenating a particular PPD could be done with a simple linear search (no sorting), at least.
That said, we probably are not going to spend a huge amount of time optimizing this since printer drivers are going away in CUPS 3.0 and you can locally optimize this by eliminating unnecessary PPD files.

I agree with Mike - it would be great if PPD driver listing was better, but our current focus lies elsewhere and PPD are going away from CUPS, so we won't fix it at the moment. But in any case, if you find time, look into it and prepare patch, we will review it.
I'll close the issue for the moment - feel free to reopen once there is a patch.


Updating via ARM errors out once SP is rotated

Hi, once we rotate the SP, and update the ARM template used originally to create the AKS resource with the new SP credentials,, subsequent ARM updates fail complaining that the SP fields are immutable and cannot be changed. Even if we don't change the SP field, ARM still complains. What's the recommended way here?

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

ID: 0b842222-f6ac-5266-227f-7e400610e07b
Version Independent ID: 429cd3bf-2691-f633-2e0d-e70e5d8d0e03
Content: Reset the credentials for an Azure Kubernetes Service (AKS) cluster
Content Source: articles/aks/update-credentials.md
Service: container-service
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

Thanks for the feedback! We are currently investigating and will update you shortly.

@sayash After updating the service principal credentials or changing the service principal, You need to update the AKS to make use of the new service principal.
You can use this command or the API to do that.
Once that update is successful, You should be able to use the ARM template with updated or new service principal.
I created  a new cluster and ran through these steps and its working as expected.
Try this out and let me know.

@sayash is there any update

@jakaruna-MSFT Hi, I'm going to test and update with results today. Thanks for checking in!

@jakaruna-MSFT Thanks, confirming it works! The ARM template has to have the new SP.

@sayash Good to know that it worked.
I am closing this issue for now. If you need additional help or if you face this issue again, Please comment here and we can continue the discussion.


docs(readme): fix the link to tutorial URL

Fix the link to the tutorial URL.

Thanks for your help on that


Ability to get property alias name of menu

How do we get the property alias name of the given Meganav menu in Razor?  So that we can set that as a CSS name of the menu when the page is rendered in the DOM.
So that the rendered menu may look something like this...

item 1

item 2

item 3

or

item 1

item 2

item 3

Or is this not possible with current version of Meganav?
The only thing I know how to do with this package is the following...

  @Html.Partial("ExampleNavigation", Model.Content.GetPropertyValue<IEnumerable<MeganavItem>>("my-property-alias-name"))

Hello, thanks for raising this question!
I would recommend that you duplicate the "ExampleNavigation" partial that ships with the package and use this as a template. This way you can pass in the property value relevant to it's use and use different HTML for each navigation.
If you're looking for a solution with less duplication (but inherently more complexity) you could create a razor helper that takes a string parameter of "cssClass" and render the navigation that way.
`@helper RenderNavigation(IEnumerable items, string cssClass = "")
{
if (items.Any())
{

@foreach (var item in items)
{

@item.Title (Level: @item.Level)

                @RenderNavigation(item.Children)

        }

}

}`
For the above example I have duplicated the "RenderNavigation" helper from the "ExampleNavigation" Razor partial that ships with the package but added an optional parameter of "cssClass".
You could call it like this in your Razor View:
@RenderNavigation(Model.Content.GetPropertyValue<IEnumerable<MeganavItem>>("my-property-alias-name"), "my-custom-css-class")
If you want to reuse the above Razor helper in multiple Razor Views across your project you can create a file for it within the "App_Code" folder.
Hope this helps!

Awesome. This was actually leading me to my second question and the "bigger" picture.  With mega nav, I would really like to reference my menu on all templates, not just the home template.  I will have 7 child templates under the master template in Umbraco.  With your examples above, does the razor helper code allow me to use meganav on multiple templates?

I looked over the documenation on Implementing Razor, and tried the following...
@{

    foreach (var item in Model.GetPropertyValue<IEnumerable<MegaNavItem>>("fFatopNav"))
    {

do stuff

    }
}

But Visual Studio is complaining and saying:

The type of namespace name 'MegaNavItem' could not be found

What should I look over to correct this issue?
Thanks

In regards to your first message, you can definitely achieve that by using the Razor helpers! Though if you want the navigation to appear on all templates I'm unsure why you wouldn't simply reference the navigation partial in your Master template. Maybe I misunderstood...
The recent issue you have reported could be a number of things. Firstly, the model that Meganav returns is called "MeganavItem" whereas in your code you have "MegaNavItem" (note the difference in capital N). Alternatively it is because you don't have "@using Cogworks.Meganav.Models" in the references at the top of your Razor Partial.
Hope this helps!

In regards to the recent issue, it was my fault - I forgot the using statement at the top of my master template.
In regards to my first message, yes!  I simply want to reference my menu onto the master template :)
I want to (1) take your example above....
@RenderNavigation(Model.GetPropertyValue<IEnumerable<MeganavItem>>("FFatopNav"), "mFFAtopnav");
and (2) use it with the following...
if (items.Any())
{

...

}

How do we combine these 2 together?
Thanks!

Glad you sorted your issue!
I'm not quite sure what you're trying to do with the 2nd part though... Could you clarify and I'll try to help!

When you say second part, what do you mean...

Ahh really sorry, I must have replied while you were editing your original post as I didn't see the code sample you've provided above.
Really glad I could help you out, and thanks again for using the package! :-) #h5yr

Hello,
when I wrote for in the children items  .Where(x => x.Content.IsVisible())
I get an error message.
Server Error in '/' Application.
Object reference not set to an instance of an object.
Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.
Exception Details: System.NullReferenceException: Object reference not set to an instance of an object.
Source Error:
Line 23:                     {
Line 24:
Line 25:                             @foreach (var subMenu in item.Children.Where(x => x.Content.IsVisible()))
Line 26:                             {
Line 27:
Source File: e:\Projects\Microsoft\devISKUR v7.8.1\umbISKUR-dev(v7.8.1)\Views\Partials\USNNavigation\USNMainNavigation.cshtml    Line: 25
Where am I doing wrong? Can you help me? please.

@inherits Umbraco.Web.Mvc.UmbracoTemplatePage
@using Umbraco.Web;
@using Umbraco.Web.Models;
@using Cogworks.Meganav.Models;
@{
var navigationNodeID = Umbraco.TypedContent(1075);
var test = navigationNodeID.GetPropertyValue("USNMainNavigation");

        @foreach (var item in navigationNodeID.GetPropertyValue<IEnumerable<MeganavItem>>("USNMainNavigation"))
        {

                <a<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>class="nav-link"  data-toggle="dropdown" role="button" aria-haspopup="true"<EMAIL_ADDRESS>

                @*@if (item.Children.Where(x => x.Content.GetProperty("isVisible")).Any())*@

                @if (item.Children.Any())
                {

                        @foreach (var subMenu in item.Children.Where(x => x.Content.IsVisible()))
                        {

                                <a<EMAIL_ADDRESS><EMAIL_ADDRESS>

                        }

                }

        }

}


Sprint 30

Review done for Sprint 30. I don't see anything major.

One quick thing: I forgot to update the changelog for my PR.
https://github.com/cfpb/cfgov-refresh/pull/693


Use pod's termination grace period if defined and greater than config…

…ured grace period
Fixes #118

Need to add tests


Adds configs logging_add_custom_attributes, logging_drop_attribute_fb_input

Support for more compact logging metadata in newrelic-infra.yml
# Adds custom_attributes to fluent-bit forwarder tags.
logging_add_custom_attributes: true

# Drop fb.input from fluent-bit forwarder tags.
logging_drop_attribute_fb_input: true

Use in conjunction with enrichmentDisabled in logging pipeline

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


WIP Locale Comparison Issue #174

Case insensitive Locale comparison across all fields for simplicity. Addresses issue #174 . If locale needs to be more focused, can do

@cdcabrera Thanks, can you squash your commits and npm run build to update the dist folder?

almost there, resolving conflicts

If the conflict is in dist, should be able to rebase and npm run build again

reopened under PR#191


[BUG] command does not work

command does not work. node funi.js --auth  o node funi --auth none of them work for me
C:\Users\Usuario\Downloads\funimation-downloader-nx-master>node funi --auth
node:internal/modules/cjs/loader:936
throw err;
^
Error: Cannot find module 'C:\Users\Usuario\Downloads\funimation-downloader-nx-master\funi'
←[90m    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)←[39m
←[90m    at Function.Module._load (node:internal/modules/cjs/loader:778:27)←[39m
←[90m    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)←[39m
←[90m    at node:internal/main/run_main_module:17:47←[39m {
code: ←[32m'MODULE_NOT_FOUND'←[39m,
requireStack: []
}
C:\Users\Usuario\Downloads\funimation-downloader-nx-master>

Please follow the template guidelines:
Will take a look at it tomorrow.

Since we merged both downloaders, your issue may be invalid. Please check and reopen if this issue persists.


Create DNS Flow skeleton like Certs Flow

We've got two very similar, and related, processes to manage:

DNS
Certificates

Both now have an API:

DNS: https://github.com/DevelopingSpace/starchart/blob/main/app/lib/dns.server.ts and
Certificates: https://github.com/DevelopingSpace/starchart/blob/main/app/lib/lets-encrypt.server.ts

The Certificates have an asynchronous flow, made-up of various sub-jobs: https://github.com/DevelopingSpace/starchart/tree/main/app/queues/certificate.
We need the same sort of thing for DNS.  There is some work in https://github.com/DevelopingSpace/starchart/blob/main/app/lib/domains.server.ts that might help with this, but what we really need to do is break down the process of working with DNS (Route53, MySQL, Notifications, etc) as a series of async jobs that can be performed in the background, and linked together as a flow.
Without this, we're going to waste a lot of time trying to hook everything together, so I'd like to prioritize the work, and get it started as soon as possible.
@dadolhay has the most experience thinking in terms of distinct, async jobs, so perhaps he can throw some thoughts in here.
@Genne23v are you interested in working on this?
Whoever does work on it, I'd highly recommend we do this in stages like @dadolhay did with the certs flow: build the skeleton, then build the individual workers to connect back to our existing APIs.  I'd like to avoid having someone go off for a month and try to build this without feedback.
To get the ball rolling, here's a draft of the process:

addDnsRecord() (or update, delete) takes record details and creates a new flow. A record is immediately put into MySQL with the state indicating that it's in process, so the front-end can begin to query for that data (and state)
A call to Route53 is made
We ask Route53 when it's done using the SDK API (might take 2 or 3 tries)
When the record is done, we update the MySQL data with the new status (could be error or ok)
We send the user whatever notification(s) are necessary to let them know what has happened

What am I missing?

@humphd I should've let you know that I'm working on this. It took much longer than I thought as I struggled with some errors. I just pushed #274 to resolve this issue. Although notification is missing in my code.

Thanks @Genne23v.


Update Readme.md

According to #882,these errors I had can be ignored so here.

Doesn't sounds too useful to have this in the readme.


Fixes #7181

The problem was that ET terms were being applied within the atoms that were in the coordmap. This disables that and adds tighter distance constraints between coordMap atoms when doing the ETK minimization.

@sriniker do you have time to take a look at this?


Web activity changes for IR

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[x] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

@mfrintu Can you update an example to use the new property or point me to the operations that would use this definition?  Its hard to track across the various swagger specs and polymorphic types.

I’m adding  a new optional property to the Web Activity that is derived from Activity and any operation that is using pipeline will have it.
Thanks,
Monica
From: Chris Eggert<EMAIL_ADDRESS>Sent: Monday, October 28, 2019 3:20 PM
To: Azure/azure-rest-api-specs<EMAIL_ADDRESS>Cc: Monica Frintu<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-rest-api-specs] Web activity changes for IR (#7605)
@mfrintuhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmfrintu&data=02|01|mfrintu%40microsoft.com|0a37916d85384e8088b408d75bf4f2ed|72f988bf86f141af91ab2d7cd011db47|1|0|637078979917153550&sdata=VoXg%2FCwHExsFjar6OrEC3O3sc997XVeEttylozOniZA%3D&reserved=0 Can you update an example to use the new property or point me to the operations that would use this definition? Its hard to track across the various swagger specs and polymorphic types.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fpull%2F7605%3Femail_source%3Dnotifications%26email_token%3DAJQ6RAGL7SI6COILCSS24LTQQ5QQLA5CNFSM4JEJQMSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECOSN5A%23issuecomment-547170036&data=02|01|mfrintu%40microsoft.com|0a37916d85384e8088b408d75bf4f2ed|72f988bf86f141af91ab2d7cd011db47|1|0|637078979917163504&sdata=fbbAjisToOZ5sHtVzgHUYEPTeCTj4TKTsWLi983aRkw%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJQ6RAA3Z7AKXXWU7PKUXFLQQ5QQLANCNFSM4JEJQMSA&data=02|01|mfrintu%40microsoft.com|0a37916d85384e8088b408d75bf4f2ed|72f988bf86f141af91ab2d7cd011db47|1|0|637078979917163504&sdata=V1xMWV542AKqyzmYCGXpFR1IOtVUeuCo8gy2H3wkK2I%3D&reserved=0.

Automation for azure-sdk-for-python


Re-order hbase env attribute loading to fix PR 668

This PR should fix both the node.set issue, and the attribute precedence issues that pull #668 sought to remedy.
I moved most attributes back into attribute files, and deleted duplicate settings.  We were setting the hbase env settings for JMX and Kerberos twice, in two different recipes.  Lev's PR caught one of two, and the second ended up being a nil access without the first one happening.

This is no longer needed, as we've located a corrected set of changes.


Posts query endpoint implementation

Description
Implemented the posts query endpoint. Closes #55
Checklist

[x] Targeted PR against correct branch
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Wrote tests
[x] Re-reviewed Files changed in the Github PR explorer

Codecov Report

Merging #57 into master will decrease coverage by 2.72%.
The diff coverage is 47.91%.

@@            Coverage Diff             @@
##           master      #57      +/-   ##
==========================================
- Coverage   82.98%   80.26%   -2.73%
==========================================
  Files          14       15       +1
  Lines         570      613      +43
==========================================
+ Hits          473      492      +19
- Misses         85      109      +24
  Partials       12       12

Impacted Files
Coverage Δ

x/posts/internal/types/querier.go
0% <0%> (ø)

x/posts/internal/keeper/querier.go
0% <0%> (ø)
:arrow_up:

x/posts/internal/types/post.go
56.96% <0%> (-4.69%)
:arrow_down:

x/posts/internal/keeper/keeper.go
98.03% <100%> (+0.44%)
:arrow_up:

x/posts/internal/types/msgs.go
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Waiting #59 to be merged before

There are conflicts after #59 is merged.

Should be fixed and ready to be merged


Service Request by Emilia Camp about MONITORING data entry for a SMART Strategy

[x] Qualtrics Service Request Form was received from User
[x] e-mail acknowledging that User's Qualtrics Service Request Form was sent
[ ] Problem NOT RESOLVED
[ ] Problem PARTIALLY RESOLVED
[ ] Problem FULLY RESOLVED and Issue is Closed
[ ] Qualtrics Post-Service Request Survey was sent to User

Use comments feature (below) to document:

A clear and concise description of the solution.

A clear and concise description of any alternative solutions or features you've considered.

Any additional context, such as screenshots about the problem.

From: David Lounsbury<EMAIL_ADDRESS>Sent: Thursday, September 16, 2021 2:14 PM
To: david lounsbury<EMAIL_ADDRESS>Subject: IPA Tracker Support Request Form Submitted
**IPA SERVICE REQUEST SUMMARY

CONTACT INFORMATION:

Emilia  Camp
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER>

COUNTY

Suffolk

WHICH FEATURE(S):

MONITORING: Creating, updated or deleting information about an implemented SMART Strategy

BRIEF PROBLEM DESCRIPTION:

I am having an issue with updating the IPA tracking. When I clicked on measure history, the drop down (measure) doesn't link to the measure on the form below (measure monitoring/edit-delete). When I update the measure, it will update ALL selections.

URGENCY:

50

ANYTHING ELSE TO SHARE:**

From: david lounsbury
Sent: Thursday, September 16, 2021 3:09 PM
To<EMAIL_ADDRESS>Cc: Donrie Purcell<EMAIL_ADDRESS>Nicholas Bove<EMAIL_ADDRESS>Subject: FW: IPA Tracker Support Request Form Submitted
Hi Emilia:
We have received your request.
A short Zoom meeting to see exactly what you are describing may be useful.
[Donrie or Nick, please follow up with Emilia]
Best,
David

Hi All,
I've just met with Emilia. It seems as if within IPA MONITORING, when entering measures data, it was not generating under measure monitoring history. Further, she was unable to update demographics. I've just opened up the IPA we were reviewing (OEND Fast Track) and I seem to be able to update the monitoring section without any issues. Could it perhaps be an admin vs Suffolk user issue?

Hi Nick,
Great work, I was awaiting a 4 o clock meeting. No worries & thanks for all your efforts.
Donrie

No problem Donrie. Yes, it's not resolved yet. I had Emilia share her screen and it was clear that she was not able to update data in "Measure Monitoring History"

Interesting. We will await Jeff's review of the matter.

Interesting. We will await Jeff's review of the matter.

@jbazinet1111 Please have a look. @NBOVE1 thinks it may be a user access issue.
DL

There is no differentiation between admin vs user in this scenario.
Can you document the steps she is taking? (w/ screen shot?)

@NBOVE1 When you have a moment, can you provide more information about this issue?
Ask Emilia about the browser she is using. Could that be the issue?
Also, should we try to delete her username and password, and then re-add it? Could that be the issue?

Hi David,
Yes, I'm working around Emilia's schedule today to touch base and further work on this issue. I'll check the browser and re-add her username. I'll report back with an update right after our meeting.
Thanks!
Nick

Just some feedback - its not going to be tied to the user. Its going to be the manner in which the data is filled in and what data is present.

Met with User to continue troubleshooting issue. User was attempting to enter data for multiple measures in MONITORING. User was not utilizing the dropdown in Measures Monitoring History. Team reviewed the user guide with user and completed entering data for one measure in Strategy C2. User then entered data for another measure on her own. User felt competent in using the IPA tracker moving forward.


(question) possible to reduce size of package + dependencies?

First of all great work on this SDK! It's super easy, works great. I have set it up in an AWS Lambda serverless function, which also works (yay for serverless blockchain communication!), but the package size is quite big: almost 50MB zipped. The AWS Lambda limit is exactly 50MB zipped, so it barely fits inside a Lambda function. However, when you need a couple more dependencies you already have a problem. This is probably due to gRPC and its many many subdependencies.
Would it be possible to create a minimal version, check if all dependencies are really needed or something so the package size can be decreased?
Thanks!

There seems to be some decent wins that can be done by separating out a few of the agora-api dependencies into dev only dependencies.
The two biggest culprits seem to be grpc and sodium-native. The former I think we're somewhat stuck using for now. The latter, at least taking a peak locally, seems to be largely driven by the intermediate build objects. sodium-native is currently used (transitively) by stellar-base.
We can poke around a bit with the npm dependency / build system to try and make sodium-native more manageable.

Perhaps as mentioned in https://github.com/kinecosystem/kin-node/issues/2#issuecomment-696424338, we can just remove sodium-native as a dependency, and let consumers decide whether or not they want to pull the dependency in.

The agora-api changes have been pulled in to<EMAIL_ADDRESS>(https://github.com/kinecosystem/kin-node/commit/a3f83142845c643b83ffc813cd17231151acf3d3).
As for sodium-native, a comment has been added to the readme indicating it can be safely removed (at the cost of some performance). Running the following steps now produces a zipped node_modules that is about ~24 MiB:

npm i --production
rm -rf node_modules/sodium-native
zip modules.zip node_modules

Let us know if this is sufficient for you use cases.

Thanks a lot, this works! The package size now decreased to around 35MB instead of 49MB. This gives plenty of room for other stuff. It's still quite big, but I guess gRPC is quite heavy as well.
By the way, if you use serverless and the serverless-webpack plugin, you can get it working easily by adding this in serverless.yml:
custom:
  webpack:
    packager: 'yarn'
    packagerOptions:
      scripts:
        - rm -rf node_modules/grpc/src/node/extension_binary/* # Remove grpc for other systems
        - rm -rf node_modules/sodium-native
        - npm rebuild grpc --target=12.0.0 --target_arch=x64 --target_platform=linux --target_libc=glibc # add grpc for node lambda env
    includeModules:
      forceExclude:
        - aws-sdk