docs: added instructions on how to integrate ansible-lint in venv w…

Suggest for documentation updates
Updates
Added some documents to showcase "how to integrate ansible-lint inside venv with Ansible Language Extension".
Tested on major operating systems
The instruction is tested on the following environment.

Windows

Windows 10 Enterprise Evaluation
10.0.19042 Build 19042

Mac

Mac OS Catalina
10.15.7 (19H15)

Linux

Fedora 33

Comment
This PR is based on the workaround 2 in the past PR. I couldn't reopen the past PR, because I pushed to the repository with --force option. Sorry for messing up.


CreateCallList implemented

Thanks for the PR!
This is an auto-generated code, so we cannot accept your changes, but the implementation you proposed is already available in the new version of the package.
So the job is done and we are closing the PR.


Not working

Hello, just stumbled upon this after seeking a DDV player, and i must say, brilliant work! However, the build's don't work, here is the output:
[ENTER] main
SdlAudioWrapper::SetAudioSpec [I] SetAudioSpec samples: 1024 freq 44100
[ENTER] SdlAudioWrapper::Open
SdlAudioWrapper::Open [I] Trying default audio device
[EXIT]  SdlAudioWrapper::Open
[ENTER] Actions::Actions
[EXIT]  Actions::Actions
GameFileSystem::InitBasePath [I] basePath is \\InfinitySrv\massData\Temp\OddStuffz\ALIVE-0.1.0-win32\
[ENTER] Engine::InitSubSystems
OpenGLRenderer::InitGL [I] GL settings r 8 g 8 b 8 bufferSize 24 double buffer 1
[EXIT_EXCEPTION] Engine::InitSubSystems
[ENTER] Engine::~Engine
assertOnGlError [E] GL ERROR: 1282, before glDeleteProgram(mShader)
Assertion failed: 0 && "GL ERROR", file C:\alive\src\openglrenderer.cpp, line 20

And your blog is inaccessible

Not sure what the cause of that GL error is, but you can pass the command line -directx9 and see if directx works?

Hmm actually that GL error is probably just a bad path on an exception, since you seem to have a debug build try setting break on exception to see where/what exception is being thrown? I think it would be logged out but the assert prevents it getting that far due to openglrenderer destructor issue.

you mentioned path, and I was running it on a network drive, put it on my desktop and runs without issues. UNC support?

Ah yes UNC probably doesn't work. I'll try to repro when I get time.


Mixed log entities in logtrail

Hi!
i have 1 pod who generate logs.
this pod based on BusyBox and execute following shell command
i=0; while true; do echo "$i: $(date)"; i=$((i+1)); sleep 1 ; done
so when i open kubectl logs & docker logs  command the output have strong order of numbers
0: Thu Jun 20 10:13:35 UTC 2019
1: Thu Jun 20 10:13:36 UTC 2019
2: Thu Jun 20 10:13:37 UTC 2019
3: Thu Jun 20 10:13:38 UTC 2019
4: Thu Jun 20 10:13:39 UTC 2019
5: Thu Jun 20 10:13:40 UTC 2019
6: Thu Jun 20 10:13:41 UTC 2019
7: Thu Jun 20 10:13:42 UTC 2019
8: Thu Jun 20 10:13:43 UTC 2019
9: Thu Jun 20 10:13:44 UTC 2019
10: Thu Jun 20 10:13:45 UTC 2019
11 etc....

but when i see logs from my pod in kibana, the logs appears in very very mixed way
and it looks like
79: Thu Jun 20 10:14:55 UTC 2019
89: Thu Jun 20 10:15:05 UTC 2019
91: Thu Jun 20 10:15:07 UTC 2019
120: Thu Jun 20 10:15:36 UTC 2019
118: Thu Jun 20 10:15:34 UTC 2019
80: Thu Jun 20 10:14:56 UTC 2019
39: Thu Jun 20 10:14:14 UTC 2019
50: Thu Jun 20 10:14:26 UTC 2019
33: Thu Jun 20 10:14:08 UTC 2019
8: Thu Jun 20 10:13:43 UTC 2019
30: Thu Jun 20 10:14:05 UTC 2019
15: Thu Jun 20 10:13:50 UTC 2019
17: Thu Jun 20 10:13:52 UTC 2019
14: Thu Jun 20 10:13:49 UTC 2019
25: Thu Jun 20 10:14:00 UTC 2019
4: Thu Jun 20 10:13:39 UTC 2019
1: Thu Jun 20 10:13:36 UTC 2019
3: Thu Jun 20 10:13:38 UTC 2019

so u can see it mixed, and how to solve this problem? i use ElasticSearch 6.2.3 and logtrail 6.2.3
and Fluent-bit 1.1.2
my logtrail.json looks like
{
  "index_patterns" : [
    {
      "es": {
        "default_index": "logstash*",
        "allow_url_parameter": false
      },
      "tail_interval_in_seconds": 2,
      "es_index_time_offset_in_seconds": 0,
      "display_timezone": "local",
      "display_timestamp_format": "MMM DD HH:mm:ss",
      "max_buckets": 500,
      "nested_objects" : true,
      "default_time_range_in_days" : 0,
      "fields" : {
        "mapping" : {
            "timestamp" : "@timestamp",
            "display_timestamp" : "@timestamp",
            "program": "kubernetes.container_name",
            "message": "log"
        }
      }
    }
  ]
}

P.S I think this is not fluent-bit or fluentd issue because i was try both, and the problem still exist

Can you paste a sample document from your ES index?  Also, can you check the order in Kibana discover UI?


Bug with highlighting nick and @twitter name rendering

@llkats something for you to look at maybe? :dancer:

🍺🍺🍺🍺🍺🍺🍺🍺🍺🍺🍺🍺🍺

https://github.com/bigboringsystem/bigboringsystem/commit/8622798ec02ce86da277e45ffc02eb0c82902b03 fixed!


perf: create base venvs only

With control over the pip installation prefix there is no need to create venvs other than the base one. This saves a lot of memory and speeds up the task runs.
These are some numbers based on the riot test suite when run with riot run -p3.9 test
Before
.riot folder size: 184 MB
test run time: 63 s
After
.riot folder size: 27 MB
test run time: 34 s
Bonus
riot list works 🙂
❯ pipenv run riot list
test  Python Interpreter(_hint='3.6') 'pytest' 'pytest-cov' 'mock' 'typing-extensions'
test  Python Interpreter(_hint='3.7') 'pytest' 'pytest-cov' 'mock' 'typing-extensions'
test  Python Interpreter(_hint='3.8') 'pytest' 'pytest-cov' 'mock' 'typing-extensions'
test  Python Interpreter(_hint='3.9') 'pytest' 'pytest-cov' 'mock' 'typing-extensions'
black  Python Interpreter(_hint='3')
fmt  Python Interpreter(_hint='3')
flake8  Python Interpreter(_hint='3') 'flake8' 'flake8-blind-except' 'flake8-builtins' 'flake8-docstrings' 'flake8-import-order' 'flake8-logging-format' 'flake8-rst-docstrings' 'pygments'
mypy  Python Interpreter(_hint='3') 'mypy' 'pytest'
codecov  Python Interpreter(_hint='3') 'coverage'
docs  Python Interpreter(_hint='3') 'sphinx==3.3' 'sphinx-rtd-theme==0.5.0' 'sphinx-click==2.5.0' 'reno'
servedocs  Python Interpreter(_hint='3')
releasenote  Python Interpreter(_hint='3')
reno  Python Interpreter(_hint='3')

There seems to be an issue with mypy when it is installed in its own prefix: https://github.com/python/mypy/issues/10829. We could wait for a resolution on the linked issue or temporarily take out mypy from the riotfile.py and run it the traditional way so that we can benefit from this optimisation.

Thanks for digging into it @P403n1x87! Yeah I'm ok with just running mypy outside of riot for now :+1:


Feature Request -- Safe Mode (Auto Rollback Changes if connection lost or not re-established within X time)

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

I realize I am duplicating a post, however, I assumed when I created a reply it would re-open the closed issue. That appears not to be the case and I simply hoped that opening a new issue would generate more feedback than a closed one. Should the original issue be able to be re-opened I am happy to close this.
Is your feature request related to a problem? Please describe.
I have multiple sites that am having to make several changes that could break access (Network address changes, Firewall changes, Routing, Etc.)  ........ I'm scared. HAHA!
The remote sites have 0.75% technical experience. Yes, I might be able to find someone to power cycle a router. However, I've also worked as an MSP and had the resident "IT person" unplug the network cable to "reboot" which they thought worked because "the lights went off and came back on" (Link Lights) #facepalm
Needless to say, took me a bit to figure out that's what they were doing.
Describe alternatives you considered
To my knowledge OPNSense doesn't have any reasonable alternative to this particular problem. If I am wrong, my apologies for wasting time and please help direct me in a correct path.
Additional context
Originally posted by @tcsi-github in https://github.com/opnsense/core/issues/3042#issuecomment-1119297907
I also created a forum post regarding this but no reply as of yet.
https://forum.opnsense.org/index.php?topic=28238.0
@banym had a good template as well and I include his here for comparison. Personally, I feel like the simpler thing would be to take a snapshot as soon as the breaking change option is enabled. I can see the use in giving the option for a timer, however, I would think setting a "middle-road" 120 seconds would be a default.
Describe the solution you'd like
It would be nice to lock the firewall in a "major change" mode where only one session is able to do changes until the major change mode is exited. This mode should be able to define a working configuration from the backup config history or the current configuration when this specific mode is activated. It should be possible to set a timer for change commitment. Now the administrator can for example make significant changes to routing or rules that possible could lock him out of the firewall. If he does not approve that his change was successful and works as intended the firewall roles back to the defined configuration. This way the administrator can log in an "try" again or rethink his change
Describe the solution you like
I was thinking about my MikroTik days and remember that if I happened to make a incorrect change that those changes would be reverted if I failed to connect back or didn't apply them within X time. Sometimes it was a pain, but it saved my butt so so many times. Kept me from having to make calls to talk someone through a reboot and just gave me a bit of room to breathe.
I would submit something the below as a rough draft for an option.
The ability might stay disabled by default and only enabled prior to the change. This would allow the option to stay out of the way and only be used when explicitly needed.
@fichtner -- I believe this to be a worthwhile addition to the OS and would be a step in the direction of more enterprise use-cases. My development skills are limited, however, I would commit to time testing and anything within my abilities to help this become a actual feature.
I would welcome any thoughts or feedback on the suggestion.
Example Change --->  Admin has to change a WAN IP

Scenario - Successful Change

Admin logs in and enables "Breaking Changes Mode" (Or whatever better name)

The working config is immediately snapshot.
Notification alert is placed at the top bar
No other logins are allowed but the user
Working config is marked as the config to be used for restoration
Perhaps the connection is marked for monitor?
Timer Starts (120 second?)
Notification Alert shows at top (See above)

Admin makes changes just as they normally would

Complete with "Apply Changes", etc.

Tests changes and is happy with results

Connection either not broken or reconnected within timer

Admin disables "Breaking Changes Mode"

Previous snapshot config is then removed to prevent use on reboot.
System returns to normal operation
Notification removed from top bar

Now lets try a screw-up....

Scenario - Failed Change

Admin logs in and enables "Breaking Changes Mode" as before

Same as above happens to enable feature

Admin makes changes just as they normally would

Complete with "Apply Changes", etc.

Finds they have made a mistake and are unable to connect back to the GUI

Connection timer expires, or some other defined trigger

"Breaking Changes Mode" begins

Previous Snapshot is automatically restored
Router is Rebooted (if needed)
Other actions such as logging the issue, etc...

Admin wait for completion of restore
- Perhaps an email or other notification could be sent notifying all is well again.
Admin is able to start again, thankful they have not created a bigger problem requiring an on-site visit (For me, an EXPENSIVE multiple day trip)

I would love this in opnsense!  I loved using commit confirmed 10 in Juniper.

Same. I use reload in 5 often in switch or router configurations that are remote.

Thats why auto sync is disabled when using a HA setup ;)

@mimugmail , I can see that, however forgive me, are you saying that HA is the only option where this would be considered?
Apologies, sometimes I'm a bit dense. :-)

Honestly I have no idea how to implement such a huge change in an easy way, the HA setup is already here, stable, easy to understand :)

Forgive me if I come off wrong, that is not the intent. I'm not entirely sure how to answer this comment correctly.
While I agree that HA does have it's place, I feel like this is one, very unnecessary overhead you are suggesting, and two, this isn't exactly solving the same problem.
HA, to my understanding, requires at least duplicating hardware to achieve sync (I also thought it requires multiple public IPs but I'm not as sure over that). Apologies, but I'm not going to spend another $600+ for identical hardware and additional power consumption just for a misconfiguration. I would be time out of pocket but I could justify a couple of "mistake trips" and  it be cheaper as opposed to the cost of duplicate hardware.
HA solves for availability and redundancy, which granted, is covered by the solution. However, I am wanting a solution for recoverability only. If the hardware dies, fine.... in a HA setup I'd still have to drive out and replace the bad hardware, I would just still be running.
Also, I would like to understand the complexity of the request. I don't quite see how this is a huge change when the underpinnings are already there. We have the ability to backup / restore configs, I am simply suggesting a means of enabling a system that does it automatically. It could be expanded and made much more, but at the heart, that's what I think most everyone would agree they wanted. Saved point and a auto restore if there was an issue.
I do realize there is probably more to consider so I would welcome someone helping me understand more. :-)
I hope that came off in the open spirit of debate rather than rude.

There's just no generic concept of "configure and commit all pending changes" in a reliable way, which always will make such a  feature incomplete and disappointing when really needed. You can in theory offer functionality like this on a per component level, which is also what we did for the firewall api (https://docs.opnsense.org/development/api/plugins/firewall.html#concept).
It's rather simple, if one could determine the conditions reliably (which I don't think one can knowing quite some different support scenario's), one could also built a plugin for it and open a PR for discussion.
A reliable failsafe, which would cover more different scenario's, would probably be to offer snapshots with zfs and go back in time when the user asks for it during boot.  As this would also cover kernel/driver issues or software changes people forgot to act upon. Maybe that's something to look into for a future business release, you never know.

I can understand your explanation of "Configure and Commit all pending changes". I also believe you are also right in using ZFS snapshots for a more robust solution.  However, what if a more simplistic approach was considered.
As I said before, we have the ability to backup "config.xml" as well as restore it again. We even have the "opnsense-importer" to restore the config file on boot I believe.
My understanding to this point is ..... If I have router 1 that dies, I can replace it with identical router 2 and restore with the config.xml from router 1 and, again assuming they are identical hardware, be right back ready to go. If this is correct, I believe we could reasonably be able to assume that the hardware isn't changing since the option would be more for a configuration error rather than a hardware change (which I believe even a ZFS snapshot would have issues with as well?)
Working from that, let us consider Juniper's "Confirm Commit X" Command
Per their site.

To confirm a commit, enter either a commit or commit check command.
If the commit is not confirmed within the time limit, the configuration rolls back automatically to the pre-commit configuration and a broadcast message is sent to all logged-in users. To show when a rollback is scheduled, enter the show system commit command. The allowed range is 1 through 65,535 minutes, and the default is 10 minutes.

This could look like this in OPNSense

Admin enables "Rollback Option" with a timer of "5 minutes"

A backup "config.xml" is taken

Admin does whatever is needed in the time frame

Commits are not tracked in any other way than normal

Admin DOES stop timer within the given allotment

Timer is killed and no restore occurs leaving the system in the current state

Admin DOES NOT stop the timer within the given allotment

Logins are paused to prevent changes during restoration
Automatic restoration of the previous created config.xml
The effect should theoretically be the exact same as if I did it manually.

Using this approach we are not storing changes for bulk commit or really changing the commit process at all..... we are simply saying "I'm about to do something I may not come back from, let me make a backup and set a timer to restore the backup if I don't finish in time"
Again, I do agree this could be built out more with ZFS in future. I will be the first to say I'm not familiar with the deep inner workings of the OS. I believe, however, I see most of the processes in place currently to make the above process work.
As always, I welcome the feedback and do appreciate the consideration. I lack developer skills, however, I am passionate about this project and feel this feature would be most welcome based on the feedback when I have posed the thought.

I don't expect it will work, but as mentioned if it would, the plugin framework offers everything you need, so no need to keep this in core. you (or anyone else) could start working on such a plugin and open a PR there.

My apologies for the incorrect placement. I should have realized this could be a plug in.
I appreciate the discussion very much.

Chiming in to say that I would appreciate this feature too. It’d be much nicer to have a simple “wait 20 secs after lights stop blinking”, than (in my case) connecting to the same LAN as the Proxmox server virtualizing OPNsense (rather than my daily VLAN), reverting a snapshot or typing in the command line to undo the changes, and possibly reboot if needed.
Is there some recognizable way for other users to express interest without butting in the middle of implementation discussions?

My apologies for the incorrect placement. I should have realized this could be a plug in.

No problem at all, it's easy to overlook. Don't mind keeping the ticket open for now, just mentioning its (currently) not a core priority and someone wil have to do some work at some point in time in order to mature ideas. When keeping it simple, a plugin would probably be the better place anyway.

@JJGadgets Thank you for the interest! I don't believe anyone would view it as butting in. @AdSchellevis Thank you for leaving the request open. I understand there are bigger things than my request. 🙂 I will create a request in plug-ins and reference here.

@tcsi-github let me move this then


Guild Card Special Investigation Medal Icon

Have you managed to extract the aforementioned icon from the game files? The one that appears by each monster on your guild card after you complete a special investigation for them. I've been making and updating a "completionist" tracker spreadsheet on Google Sheets ever since i started playing base Rise, and i think that's one of the few icons i still need.

You can dump the file gui/80_Texture/GuildCard_MR_IAM.tex

I'm sorry about my ignorance, but neither i nor my programmer friend can figure this out, where is that file, and where do i run that command? 😰 He told me to download this repo as a zip file and run the commands you mentioned on Windows Terminal but none of the commands are recognized, nor can i find that file in the zip folder

Try this:

Because I didn't release the binary, you need to install rust and build from source (see requirement in readme)
go to the root folder of this repo (either a zip download, or a git clone)
`cargo run -- dump -p "" -n "gui/80_Texture/GuildCard_MR_IAM.tex" -o GuildCard_MR_IAM.tex
cargo run -- dump-tex -t GuildCard_MR_IAM.tex -o GuildCard_MR_IAM.png
GuildCard_MR_IAM.png file should appear and it is what you want

Thank you very much! And sorry for being a bother 😅


ParserError: Source<EMAIL_ADDRESS>not found: File not found.

Hi guys, I'm following the OpenZepellin tutorial wich works fine, but when I run the mith analyze I received that error:
ParserError: Source<EMAIL_ADDRESS>not found: File not found.
Here's the code snippet of the smart contract:
pragma solidity ^0.8.0;

// Import Ownable from the OpenZeppelin Contracts library
import<EMAIL_ADDRESS>
// Make Box inherit from the Ownable contract
contract Box is Ownable {
    uint private value;

    // Emitted  when the stored value changes
    event ValueChanged(uint256 newValue);

...

If a change my code to import the contract as below, it works!
import<EMAIL_ADDRESS>How can I use the import statement just using @ and not a relative path of the node_modules to the OpenZepellin contracts?

The solc binary does not know the location of @openzepplin, it's possible by using solc's standard-json input format to mythril, but it does not seem to be worth it in this case.

The solc binary does not know the location of @openzepplin, it's possible by using solc's standard-json input format to mythril by providing import list, but it does not seem to be worth it in this case.

Thank you @norhh!

Hello @norhh
Can I use myth if I have @openzeppelin links? because currently, I got the same error

@idirall22 , you can either use standard-json format of solc or you can just add all the imports as code into a single contract.

@norhh How would I use the standard-json format?

@0xfoobar I should add some documentation regarding this, but this directory should contain a decent example https://github.com/ConsenSys/mythril/tree/develop/tests/testdata/json_test_dir . You can check how the remapping is in the json file is done for the directory.
You just need to run ./myth a filename --standard-json <json_file>
For more information you can look into how settings are constructed in solc documentation

when I run ./myth a filename --standard-json <json_file> I get this error:
myth: error: unrecognized arguments: --standard-json testfile.json. Is the --standard-json flag still available?

Figured it out. The --solc-json flag is working

I still dont get how I should execute myth analyzefor it in order to recognize the OpenZeppelin contracts. Can someone tell me?

@devScrooge  add a json file to remap the path to node_modules folder:
{     "remappings": [         "@openzeppelin/contracts-upgradeable/utils/=../node_modules/@openzeppelin/contracts-upgradeable/utils/"     ],     "optimizer": {         "enabled": true     } }
run myth with --solc-json option: myth analyze test.sol --solc-json remapping.json

just to contribute:

{
"remappings": [
"@openzeppelin/contracts-upgradeable=../node_modules/@openzeppelin/contracts-upgradeable",
"@openzeppelin/contracts=../node_modules/@openzeppelin/contracts",
"@chainlink=../node_modules/@chainlink"
],
"optimizer": { "enabled": true }
}


Add a couple of links to the README

Coverage decreased (-0.02%) to 87.904% when pulling 908e906721f7e0c762e2101fc8a5eba4edf3d0f1 on yallop:documentation-updates into 915648a2147a0c7b5f435c2c65d3bf58fb9a4630 on ocamllabs:master.


Integrate AppMetrics latency tracking

we will either need to enfold the AppMetrics client project in the CIAPI.CS solution or publish it via NuGet.

we will either need to enfold the AppMetrics client project in the CIAPI.CS solution or publish it via NuGet.

due to the extreme abstraction and separation at the core of CIAPI a direct logging hook is not a practical approach.
reapproaching this problem setting sights on an 'after-the-fact' log output parsing strategy


Sync strategy Meta tags duplicated by device on update

Describe the bug
Currently when the device is tagging with Practitioner, Organization/Team, Location and CareTeam, it appears to duplicate the entries if they are already existent
Expected behavior
The tags should overwrite existing ones and there should be no duplicate Tags
Screenshots
N/A
Additional context
Sample Patient request with meta tags
{
  "resourceType": "Patient",
  "id": "adb6a5cf-f4a9-4898-85f0-d1b01cf33166",
  "meta": {
    "lastUpdated": "2023-06-20T18:21:32.459+03:00",
    "tag": [
      {
        "system": "https://smartregister.org/app-version",
        "code": "0.2.2",
        "display": "Application Version"
      },
      {
        "system": "https://smartregister.org/care-team-tag-id",
        "code": "Not defined",
        "display": "Practitioner CareTeam"
      },
      {
        "system": "https://smartregister.org/location-tag-id",
        "code": "86453",
        "display": "Practitioner Location"
      },
      {
        "system": "https://smartregister.org/organisation-tag-id",
        "code": "065e9e01-e95b-4762-a66e-08555e005721",
        "display": "Practitioner Organization"
      },
      {
        "system": "https://smartregister.org/practitioner-tag-id",
        "code": "825f8ed4-e66d-4670-b749-6d1dc31282cb",
        "display": "Practitioner"
      }
    ]
  },
  "identifier": [
    {
      "use": "official",
      "value": "659178",
      "period": {
        "start": "2023-06-20T18:21:28+03:00"
      }
    },
    {
      "use": "usual",
      "value": "134675",
      "period": {
        "start": "2023-06-20T18:21:28+03:00"
      }
    },
    {
      "use": "secondary",
      "value": "ef65a9c9-0e86-4ffe-8651-94561f6bb9dc",
      "period": {
        "start": "2023-06-20T18:21:28+03:00"
      }
    }
  ],
  "active": true,
  "name": [
    {
      "use": "official",
      "text": "Ella",
      "family": "Makasi",
      "given": [
        "Joyline"
      ]
    }
  ],
  "gender": "female",
  "birthDate": "1998-06-14",
  "address": [
    {
      "use": "home",
      "type": "physical"
    }
  ],
  "generalPractitioner": [
    {
      "reference": "Practitioner/825f8ed4-e66d-4670-b749-6d1dc31282cb"
    }
  ],
  "managingOrganization": {
    "reference": "Organization/065e9e01-e95b-4762-a66e-08555e005721"
  }
}

** Acceptance Criteria**

[ ] Write a unit test that calls the logic and updates multiple times. This should initially fail the uniqueness assertion
[ ] Refactor the logic to ensure the meta tag are not duplicated on update

Sample broken json with duplicates
{
    "resourceType": "Organization",
    "id": "065e9e01-e95b-4762-a66e-08555e005721",
    "meta": {
        "versionId": "2",
        "lastUpdated": "2023-06-21T13:34:45.941+00:00",
        "source": "#2219e20486360d66",
        "tag": [
            {
                "system": "https://smartregister.org/care-team-tag-id",
                "code": "Not defined",
                "display": "Practitioner CareTeam"
            },
            {
                "system": "https://smartregister.org/location-tag-id",
                "code": "Not defined",
                "display": "Practitioner Location"
            },
            {
                "system": "https://smartregister.org/organisation-tag-id",
                "code": "Not defined",
                "display": "Practitioner Organization"
            },
            {
                "system": "https://smartregister.org/practitioner-tag-id",
                "code": "Not defined",
                "display": "Practitioner"
            },
            {
                "system": "https://smartregister.org/care-team-tag-id",
                "code": "Not defined",
                "display": "Practitioner CareTeam"
            },
            {
                "system": "https://smartregister.org/location-tag-id",
                "code": "86453",
                "display": "Practitioner Location"
            },
            {
                "system": "https://smartregister.org/organisation-tag-id",
                "code": "065e9e01-e95b-4762-a66e-08555e005721",
                "display": "Practitioner Organization"
            },
            {
                "system": "https://smartregister.org/practitioner-tag-id",
                "code": "825f8ed4-e66d-4670-b749-6d1dc31282cb",
                "display": "Practitioner"
            }
        ]
    },
    "identifier": [
        {
            "use": "official",
            "value": "9edd6e27-97fc-4f9c-b677-ebd2f6e96ae4"
        }
    ],
    "active": true,
    "type": [
        {
            "coding": [
                {
                    "system": "http://terminology.hl7.org/CodeSystem/organization-type",
                    "code": "team"
                }
            ]
        }
    ],
    "name": "Devtest Team",
    "alias": [
        "DTTeam"
    ]
}

What's the value this should be unique by? I think usually it'd be code but not all of these have codes.
I guess it would be system? But that's kind of strange since system should be able to be duplicated for many tags, but in this use case I guess it isn't, and adding the sub-path seems common, https://www.hl7.org/fhir/resource.html#id

It should be unique based on the code
{
  "resourceType": "Organization",
  "id": "b284e210-930d-465b-9cf6-4be64b31fd4e",
  "meta": {
    "versionId": "4",
    "lastUpdated": "2023-06-22T10:21:20.766+00:00",
    "source": "#e16d801e4b50d218",
    "tag": [
      {
        "system": "https://smartregister.org/",
        "code": "Not defined",
        "display": "Practitioner CareTeam"
      },
      {
        "system": "https://smartregister.org/",
        "code": "eb47e6cf-6631-4d22-85db-423c109f9717",
        "display": "Practitioner CareTeam"
      },
      {
        "system": "https://smartregister.org/",
        "code": "52a6d6e5-e0cd-4239-9950-7c094296128c",
        "display": "Practitioner Location"
      },
      {
        "system": "https://smartregister.org/",
        "code": "b284e210-930d-465b-9cf6-4be64b31fd4e",
        "display": "Practitioner Organization"
      },
      {
        "system": "https://smartregister.org/",
        "code": "b3c19bc5-838c-4b53-a681-3995765e276f",
        "display": "Practitioner"
      },
      {
        "system": "https://smartregister.org/care-team-tag-id",
        "code": "Not defined",
        "display": "Practitioner CareTeam"
      },
      {
        "system": "https://smartregister.org/location-tag-id",
        "code": "Not defined",
        "display": "Practitioner Location"
      },
      {
        "system": "https://smartregister.org/organisation-tag-id",
        "code": "Not defined",
        "display": "Practitioner Organization"
      },
      {
        "system": "https://smartregister.org/practitioner-tag-id",
        "code": "Not defined",
        "display": "Practitioner"
      },
      {
        "system": "https://smartregister.org/care-team-tag-id",
        "code": "Not defined",
        "display": "Practitioner CareTeam"
      },
      {
        "system": "https://smartregister.org/location-tag-id",
        "code": "Not defined",
        "display": "Practitioner Location"
      },
      {
        "system": "https://smartregister.org/organisation-tag-id",
        "code": "Not defined",
        "display": "Practitioner Organization"
      },
      {
        "system": "https://smartregister.org/practitioner-tag-id",
        "code": "Not defined",
        "display": "Practitioner"
      }
    ]
  },
  "identifier": [
    {
      "use": "official",
      "value": "f5ca402f-8cf8-423a-a738-1edeb5f1abd6"
    }
  ],
  "active": true,
  "type": [
    {
      "coding": [
        {
          "system": "http://terminology.hl7.org/CodeSystem/organization-type",
          "code": "team"
        }
      ]
    }
  ],
  "name": "Big Fat Town"
}

Notice on this resource it has 2 that look like this
      {
        "system": "https://smartregister.org/care-team-tag-id",
        "code": "Not defined",
        "display": "Practitioner CareTeam"
      },

Thanks @dubdabasoduba, your version is the correct one cc @ellykits @pld


{Service Fabric} add sfmc tls properties

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Thank you for your contribution! We will review the pull request and get back to you soon.

please note that we will freeze the codes at 05/14/2024 10:00 UTC for the upcoming release train. Please resolve all comments by tomorrow, otherwise the release of this PR will have to be postponed to the next sprint (on 07-02)

@mwesigwaguma Since we haven't received a response to this PR for a long time, I will put it in the backlog first. After you have resolved its comments, we will consider adding it to the release milestone


[RFC]testutil: Add snake oil cert files for roles

The current test utility support passing role based snake oil
certificates for testing but only only as strings. Some of the APIs that
need to be tested require these to be passed as file paths instead so
this patch adds that capability.
The fake ca certs were generated with the following process using
ciao-cert from commit 3bb33ad64383f0712d0f68e7e25379a7be636254:
$ ./ciao-cert -directory ../testutil -host localhost -server -role
scheduler -ip <IP_ADDRESS>
$ ./ciao-cert -directory ../testutil -host localhost -role $role
-server-cert ../testutil/cert-Scheduler-localhost.pem -ip <IP_ADDRESS>
The RoleToTestCertPath function the change implements will give the path
to the CA certificate generated when creating the server certificate and
the path to the certificate matching the requested role. If there is no
certificate path matching the specified role an error will be returned
as well.

Coverage decreased (-0.04%) to 65.034% when pulling 34451dd6f48f9caa7e22d80e940520e69c50d4f5 on bryteise:test-cert-files into 38752a17b25d59e8965b677d8999b52934a914a0 on 01org:master.

@rbradford This really should have been marked RFC. It is part of an effort for adding unit tests to tracing which isn't merged yet.
Anyway the programmatic generation I didn't know about before so I explored using that. It does seem to add a fair amount of extra overhead for each use in verbosity and in run time so I'm not really sure it is a better approach (four tests used to run in ~.4s now run in 2-3s). I pushed a branch here(https://github.com/bryteise/ciao/blob/gen-certs/trace/tracer_test.go#L210) if you want to look at my implementation.

Can you make the certs in a TestMain() once and use them in all the unit tests?
I do like the idea of not having any snakeoil certs checked in, but still having programmed usable certs for coherent unit tests.
If the total decrease across ciao is that 0.4s becomes 3s, I'd take the slowness.

@tpepper I may be not quite understanding your suggestion but I'm understanding it as:

All tests needing certs calls a Generate() function that either creates the certs or verifies they exist and then returns
Have a gitignore for the snakeoil temporary certs (there isn't an obvious clean up point)

I'm a little worried about stale certs but it isn't anything a git clean can't solve so I'm okay with this approach.

@rbradford I'm actually kind of in favor of checking the certs into the tree because there is much less room for false positive test failures due to certificate generation problems. Could you or @tpepper and in some reasoning here? I do have a patch that does on the fly test certificate generation but was having trouble justifying it based on cluttering the git tree.

BTW launcher currently has some test certs checked into the tree which are used for testing.  I may remove these based on the outcome of this conversation.
I don't think we need to worry too much about people trying to use our test certs on real clusters as they will only work on local host.  The main reason I can think of to not check the certs in is that it clutters up the source tree it's easy to forget to regenerate them when the cert-tool changes, but then the tests would probably fail.
Perhaps we could add a helper function that makes it easier to generate temporary certs, say, certs.CreateTempSet.  Then all you'd need to do would be to
given
       type  TempCerts struct {
           CAPath string
           serverPath string
           clientPath string
       }

	template, err := certs.CreateCertTemplate(ssntp.SCHEDULER, "test"<EMAIL_ADDRESS>[]string{"localhost"}, []string{"<IP_ADDRESS>"})
        paths, cleanupFN, err := certs.CreateTempSet(template, false)
       defer cleanupFN()

paths is an instance of TempCerts and cleanupFN is a function that removes all three certs when invoked.

@markdryan You might want to look at https://github.com/bryteise/ciao/commit/aecff89cc443f7fdab2e17961db4ff23793d894f for my alternate approach. The commit describes my reasoning pretty well I think for why I avoided the cleanup function but it could be added. Regardless it generates all the different role certificates in addition to the CA at once.

My desires here are:

No duplicate certs checked in per component to make the component tests run.  Options are check in one comprehensive set in testutil, or modify everything to spawn certs on demand when they're needed.  The one-check-in-of-master-snakeoil-certs path would be nicer if it could include a git-hook to build/commit new certs on commits to the ciao-cert module, as @markdryan points out we humans aren't great at maintaining persistent state that matches the code expectations.  But, if we make mandatory changes to the key format, we complicate system update...new binaries gets deployed and the cluster can no longer form because old keys are present.  That may drive us in a direction where practically we don't modify how we create certs, aside from for optional additions, where if those features have unit tests which required pre-build snake-oil keys, the commiter's unit tests would likely fail until they also fixed the keys.  So practically it might not be a huge problem.

No persistent snake-oil.  Even though the keys are "localhost" only, localhost works on all hosts, so you can make a cluster with these.  But I guess you'd need to forward localhost ssntp traffic to your real (remote, not localhost) ssntp server.  So yeah @markdryan is right that this is probably a minor risk practically.  Still it's something we're liable to have to justify over and over again to a security reviewer who stopped and puked up a red flag upon seeing snake oil checked in.  It also feels kludgy for the same reason a tests-presumed set of keys in /etc/pki/ciao for is kludgy.  It feels cleaner and more complete and self-documenting if any/all component test prerequisites are established in a TestMain() and then cleaned up after m.Run().  I like https://github.com/bryteise/ciao/commit/aecff89cc443f7fdab2e17961db4ff23793d894f though I would prefer a RemoveTestCerts() helper available.  You get reuse across the component at least.

If the above seems like I'm equivocating, apologies.  I could go either way on this.  I like the idea of autogenerate and cleanup, but if it's too invasive for now to rip out the keys I can live with that.

Thanks @tpepper, that's exactly the kind of information I was looking for. Your 2 did not really seem worrying to me since these cert would sit under testutil instead of in a component's directory. It is very understandable to me to want to avoid the existence of the test keys outside of a test run however I just weighted this pretty low.
The ability to have self creating, self cleanup per component unit test is really a win and probably the most compelling reason to avoid checking in certs to me. I think only one set of certs needs to exist for any test though so I'm not sure per component generation has as much value.
I can't really speak to how often the cert layout has changed and how much cert regeneration would be needed. I guess if that is frequent it would be a problem but as you said would be git-hookable to avoid any real problems.
I'm also okay with either approach but as I said I had trouble making a good enough argument for my MakeCerts() change and wanted more feedback. Hopefully we can get to a consensus (I'll go back and add a RemoveTestCerts() and make sure that happens as part of MakeCerts() too).


bug: fixed active item on header

closes #385

LGTM but I remember that @zzacharo had some plans in general about withRouter, did you discuss?

@kprzerwa we did the refactor already ;) But we removed too much :P Thanks @topless for fixing it :)


Disable resize borders when component is not selected

Add enable/disable property to enable adding boolean show and hide property for resize borders

I finally figure out how to do this. It's very simple just use the following properties :
zoomable=""
rotatable={false}


RNCWebViewManagerImpl.kt : Type mismatch: inferred type is String? but String was expected

Bug Description:
RNCWebViewManagerImpl.kt : Type mismatch: inferred type is String? but String was expected
Issue Summary:
When attempting to run the latest version of React Native WebView, an error occurs in the RNCWebViewManagerImpl.kt file. The error message states, "Type mismatch: inferred type is String? but String was expected." This error is specifically related to code on line 337, which is responsible for loading a URL into the WebView component using the args.getString(0) method.
Steps to Reproduce:
Upgrade to the latest version of React Native WebView.
Attempt to use the WebView component to load a URL by calling webView.loadUrl(args.getString(0)).
Expected Behavior:
The URL should be loaded into the WebView without any type mismatch errors, as it did in the previous version of React Native WebView.
Actual Behavior:
An error occurs with the following message: "Type mismatch: inferred type is String? but String was expected," pointing to the line of code mentioned above.
Environment:
React Native WebView Version: ^13.5.1
its heepend is android with gradle 7.0.2

I got the same issue, any solution to this problem.

Came across this while using 13.6.2 on an older version of React Native. You can edit the source file and cast the value to String:
In node_modules/react-native-webview/android/src/main/java/com/reactnativecommunity/webview/RNCWebViewManagerImpl.kt at line 340 change
webView.loadUrl(args.getString(0)) to webView.loadUrl(args.getString(0) as String).


Add&Format by prettier, update CRA, add TS, add yarn

Add prettier
Format entire codebase by prettier
Add prettier "fmt" command
Add yarn lock
Add editorconfig
Update CRA (react-scripts)
Add TS support (just rename 2 reducers)
Add focal / rxjs deps

sorry, wrong target


Simple question about table rows

Question
With your Table component:
How can I assign a class to a row depending on its property?
From a regular VUE loop it would be very simple:
 {{ foo }}
but don't know how I can do this with your component.

It's inside the b-table-column where you specify the field it'll show:
<b-table :data="tableData">

    <template scope="props">
        <b-table-column field="id" label="ID">
            {{ props.row.id }} <-- here I'm specifying the property "id"
        </b-table-column>

        <b-table-column field="user.first_name" label="First Name">
            {{ props.row.user.first_name }} <-- and here I'm specifying the property "user.first_name"
        </b-table-column>
    </template>
</b-table>

Oops, wrong answer due to my missed code. Now it's corrected so that the question is better understood.
So I want to put a class to entire row of the table.

Hmm... I'm gonna have to add a new property that receives a function for this, something like this:
<template>
    <b-table :data="tableData" :row-class="getRowClass"></b-table>
</template>

<script>
    export default {
        methods: {
            getRowClass(row, index) {
                if (row.amount < 10) return 'is-warning'
            }
        }
    }
</script>

Yes! Just that man. Waiting for the new release 👍

It's also possible to make it one-liner with ES6:
<b-table :data="tableData" :row-class="row => row.amount < 10 && 'is-warning'">[...]</b-table>

I've tested yesterday and works fine. But if you set the table as striped, you need to add '!important' to your class to get the desired result. Would it be possible to make it less aggressive?

It's from bulma, nothing can be done :(


Typo, I think...(From Taiwan, not good at English)

Challenge http://www.freecodecamp.com/challenges/waypoint-use-bracket-notation-to-find-the-first-character-in-a-string has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
Paragraph 4 :
Use bracket notation to find the first character in "a the" firstLetterOfLastName variable.
(FreeCodeCamp is awesome!!!!!)

this issue has already been fixed, just not pushed to the site yet. thanks for reporting though!


Update deploy-control-plane.md

Identified issues while SDAF being deployed by a CX using CLI hence proposing the working changes.

The prepare_region.sh script is being deprecated we should use deploy_control plane instead.
//Kimmo
//Kimmo

From: Courtney Wales @.>
Sent: Thursday, July 13, 2023 5:20:53 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Kimmo Forss @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Update deploy-control-plane.md (PR #112126)
@KimForsshttps://github.com/KimForss
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-teamhttps://github.com/orgs/MicrosoftDocs/teams/public-repo-pr-review-team
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/112126#issuecomment-1634438216, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGCOMTBVLUFMIOQZK2YGAU3XQAG5LANCNFSM6AAAAAA2IPKKEY.
You are receiving this because you were mentioned.Message ID: @.***>

@chmarri - see @KimForss 's comment above https://github.com/MicrosoftDocs/azure-docs/pull/112126#issuecomment-1634848556 . can you update the PR accordingly?
@KimForss - please review after the changes are finalized... or make suggestions for @chmarri  to check and commit. when you are done, sign off. thanks!
#hold-off

But its not creating SAP Library if we use deploy_control.sh it went well with prepare_region.sh
Thanks,
Chandra
From: Kimmo Forss @.>
Sent: Friday, July 14, 2023 1:48 AM
To: MicrosoftDocs/azure-docs @.>
Cc: Chandra Marriboina @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Update deploy-control-plane.md (PR #112126)
The prepare_region.sh script is being deprecated we should use deploy_control plane instead.
//Kimmo
//Kimmo

From: Courtney Wales<EMAIL_ADDRESS>Sent: Thursday, July 13, 2023 5:20:53 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Kimmo Forss<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Update deploy-control-plane.md (PR #112126)
@KimForsshttps://github.com/KimForss
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-teamhttps://github.com/orgs/MicrosoftDocs/teams/public-repo-pr-review-team

Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/112126#issuecomment-1634438216, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGCOMTBVLUFMIOQZK2YGAU3XQAG5LANCNFSM6AAAAAA2IPKKEY.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/112126#issuecomment-1634848556, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3EOOIOUOBKZX73WNIYN2HDXQBJX5ANCNFSM6AAAAAA2IPKKEY.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@KimForss - see above. thanks

emailed author

Per conversation with @KimForss, these documents have just been updated to address the issues. #please-close


Crash when profiling with checkpointed GPU Regent Circuit

This different than #1418 because I'm on:

Piz Daint instead of Sapling
Regent instead of C++ Circuit
GPUs instead of CPUs

Assertion:
circuit.checkpoint: /users/eslaught/resilience/legion/runtime/legion/legion_profiling.cc:646: void Legion::Internal::LegionProfInstance::process_copy(const Legion::Internal::LegionProfInstance::ProfilingInfo*, const Realm::ProfilingResponse&, const Realm::ProfilingMeasurements::OperationMemoryUsage&): Assertion `info.num_hops == cpinfo.inst_info[idx].num_hops' failed.

The full backtrace is attached, but the relevant portion seems to be:
[1] Thread 8 (Thread 0x1555553c1840 (LWP 19292) "circuit.checkpo"):
[1] #0  0x000015554fc46157 in waitpid () from /lib64/libc.so.6
[1] #1  0x000015554fbc36af in do_system () from /lib64/libc.so.6
[1] #2  0x00001555524167f1 in gasneti_bt_gdb () from /users/eslaught/resilience/circuit.run1_prof_debug/librealm.so.1
[1] #3  0x000015555241a049 in gasneti_print_backtrace () from /users/eslaught/resilience/circuit.run1_prof_debug/librealm.so.1
[1] #4  0x0000155551bf2466 in gasneti_defaultSignalHandler () from /users/eslaught/resilience/circuit.run1_prof_debug/librealm.so.1
[1] #5  <signal handler called>
[1] #6  0x000015554fbb6360 in raise () from /lib64/libc.so.6
[1] #7  0x000015554fbb7941 in abort () from /lib64/libc.so.6
[1] #8  0x000015554fbaea5a in __assert_fail_base () from /lib64/libc.so.6
[1] #9  0x000015554fbaead2 in __assert_fail () from /lib64/libc.so.6
[1] #10 0x000015555305b7e2 in Legion::Internal::LegionProfInstance::process_copy (this=0x154ca8201480, prof_info=0x154cac039008, response=..., usage=...) at /users/eslaught/resilience/legion/runtime/legion/legion_profiling.cc:646
[1] #11 0x000015555305bcc7 in Legion::Internal::LegionProfiler::handle_profiling_response (this=0x2fef970, base=<optimized out>, response=..., orig=<optimized out>, orig_length=<optimized out>) at /users/eslaught/resilience/legion/runtime/legion/legion_profiling.cc:2316
[1] #12 0x000015555320d0d8 in Legion::Internal::Runtime::profiling_runtime_task (args=0x154cac038f50, arglen=224, userdata=<optimized out>, userlen=<optimized out>, p=...) at /users/eslaught/resilience/legion/runtime/legion/runtime.cc:32246
[1] #13 0x0000155551ec5381 in Realm::Task::execute_on_processor (this=0x154cac0382d0, p=...) at /users/eslaught/resilience/legion/runtime/realm/tasks.cc:302
[1] #14 0x0000155551ec5416 in Realm::UserThreadTaskScheduler::execute_task (this=<optimized out>, task=<optimized out>) at /users/eslaught/resilience/legion/runtime/realm/tasks.cc:1632
[1] #15 0x0000155551ec7fdc in Realm::ThreadedTaskScheduler::scheduler_loop (this=0x135cb80) at /users/eslaught/resilience/legion/runtime/realm/tasks.cc:1105
[1] #16 0x0000155551eccb97 in Realm::UserThread::uthread_entry () at /users/eslaught/resilience/legion/runtime/realm/threads.cc:1355
[1] #17 0x000015554fbcbbe0 in ?? () from /lib64/libc.so.6
[1] #18 0x0000000000000000 in ?? ()

slurm-45121868.txt

This is almost certainly a Realm bug. Legion always issues Realm copies between the same source and destination instances (we fuse across fields and index spaces though). The number of hops should be invariant for the same source and destination instances. What does gdb show when you run pvector cpinfo.inst_info? (make sure you have an STL plugin for gdb)

This appears to be related to NDEBUG. Profiling crashes with the error above when building with NDEBUG, otherwise it works. (This explains why my initial attempts to reproduce on Sapling all failed, because I was using a debug build of Legion to test it.)
This is lower priority for me now that I have a workaround, but probably somewhere in our code we have an assert() that wraps something important.

I'm confused. How could this have crashed when built with NDEBUG because we're failing on an assert? It's impossible to have built with NDEBUG and then crashed on assertion.

I have a lot of builds flying around. Let me start from a clean one and confirm exactly what's going on.

I haven't done any deep investigation, but I can at least provide an answer about NDEBUG.
Legion's CMake infrastructure explicitly removes NDEBUG from the arguments it passes:
https://github.com/StanfordLegion/legion/blob/299e636a0bc982db6b6855ed9f46d0b0a7eab7ed/CMakeLists.txt#L195-L199
However it does not set these as cache variables. As a result, if you look at CMakeCache.txt, you'll still see NDEBUG there. But you aren't actually building with NDEBUG.
This is confusing and should probably be fixed, but the result is that all our builds do not use NDEBUG regardless of build system and configuration.

I suppose this could happen with a region-to-region copy where there are multiple source/destination instances for different fields, and some of those source/destination instances are in different memories from each other. It would be very good to confirm that is actually what is going on by seeing the output of pvector cp_info.inst_info (make sure you have an STL plug-in).
If this is the case, we'll need to modify the Legion Prof logging format. Right now the CopyInfo struct contains the total timing information for the entire Realm copy, but in this case Realm has really broken down the copy into several small copies for different source-destination pairs which are reflected in the CopyInstInfo structs, but aren't reflected in the CopyInfo struct, so we'll need to find some way to propagate the "hop count" data for each of the CopyInstInfo structs.

This happens deterministically on Piz-Daint? I would expect that to be the case if the mapping is deterministic. It would probably also explain why you don't see it on sapling. I don't really think this should be dependent on timing, but it's definitely going to be dependent on mapping.
(Glad that you figured out why we were still seeing the assertion with NDEBUG.)

The most recent reproduction steps for this are at: https://github.com/StanfordLegion/legion/issues/1418#issuecomment-1517061459
(I'm switching to C++ Circuit for this now since I got that to reproduce, and because it reproduces on Sapling.)

So I don't understand how this is possible. Here is the output of cpinfo.inst_info from Realm:
(gdb) p cpinfo.inst_info
$4 = std::vector of length 2, capacity 2 = {{
    src_inst_id = {
      id =<PHONE_NUMBER>245307404 (0x400040004000000c)
    },
    dst_inst_id = {
      id =<PHONE_NUMBER>435776517 (0x4000000000800005)
    },
    num_fields = 2,
    request_type = Realm::ProfilingMeasurements::OperationCopyInfo::COPY,
    num_hops = 2
  }, {
    src_inst_id = {
      id =<PHONE_NUMBER>245307404 (0x400040004000000c)
    },
    dst_inst_id = {
      id =<PHONE_NUMBER>435776517 (0x4000000000800005)
    },
    num_fields = 24,
    request_type = Realm::ProfilingMeasurements::OperationCopyInfo::COPY,
    num_hops = 3
  }}

Both entries have the same source and destination instance pair, and yet Realm is claiming that one of those copies needs two hops and the other one needs three hops. @streichler how can two copies with the same src/dst pair have different number of hops? That should only depend on the memories where the instances are, so the number of hops between a pair of instances should be the same for all copies issued between them since it should only be a function of the graph of memories in the Realm machine model.
Here are the memories and the kinds of memory where the two instances are:
(gdb) p/x cpinfo.inst_info[0].src_inst_id.get_location()
$13 = {
  id = 0x1e00010000000000
}
(gdb) p cpinfo.inst_info[0].src_inst_id.get_location().kind()
$14 = Realm::Memory::SYSTEM_MEM
(gdb) p/x cpinfo.inst_info[0].dst_inst_id.get_location()
$15 = {
  id = 0x1e00000000000002
}
(gdb) p cpinfo.inst_info[0].dst_inst_id.get_location().kind()
$16 = Realm::Memory::FILE_MEM

Note that if we do allow hops from the same two instances to be different, we'll need to completely change both the Legion logging as well as the back-end profiler.

I could speak more authoritatively if we had logging for these copies with -level dma=2,xplan=1,xpath=1, but in general, the number of hops can depend on field size and/or instance layouts, which may be different for different fields in the same instances.  Extra hops can appear when Realm judges it to be preferable to pack data before going over a particular channel.

Logs attached. Bad instances:
(gdb) p cpinfo.inst_info
$1 = std::vector of length 2, capacity 2 = {{
    src_inst_id = {
      id =<PHONE_NUMBER>427387918
    },
    dst_inst_id = {
      id =<PHONE_NUMBER>253696004
    },
    num_fields = 2,
    request_type = Realm::ProfilingMeasurements::OperationCopyInfo::COPY,
    num_hops = 2
  }, {
    src_inst_id = {
      id =<PHONE_NUMBER>427387918
    },
    dst_inst_id = {
      id =<PHONE_NUMBER>253696004
    },
    num_fields = 24,
    request_type = Realm::ProfilingMeasurements::OperationCopyInfo::COPY,
    num_hops = 3
  }}

dma_0.log
dma_1.log

Yep, it's actually the fact that the second group has 24 fields that causes Realm to think there's value in clumping them
up and doing 1 contiguous transfer over the network rather than shipping each of the 24 fields' data separately.
[0 - 7f0fc303f840]    0.852132 {2}{xpath}: FFP: 1e00000000000000->1e00010000000002 serdez=0 redop=0 bytes=16384 frags=[2048, 2, 0]/[2048, 2, 0]
[0 - 7f0fc303f840]    0.852157 {2}{xpath}: first: 1e00000000000000->1a00000000000000 cost=3376
[0 - 7f0fc303f840]    0.852175 {2}{xpath}: first: 1e00000000000000->1a00010000000000 cost=5276
[0 - 7f0fc303f840]    0.852193 {2}{xpath}: inter: 1a00000000000000->1a00010000000000 cost=3376+4276 = 7652 <? 5276
[0 - 7f0fc303f840]    0.852209 {2}{xpath}: last: 1a00010000000000->1e00010000000002 cost=5276+1658400 = 1663676 <? 0
[0 - 7f0fc303f840]    0.852289 {2}{xpath}: FFP: 1e00000000000000->1e00010000000002 serdez=0 redop=0 bytes=98304 frags=[24576, 24, 0]/[24576, 24, 0]
[0 - 7f0fc303f840]    0.852308 {2}{xpath}: first: 1e00000000000000->1a00000000000000 cost=19760
[0 - 7f0fc303f840]    0.852322 {2}{xpath}: first: 1e00000000000000->1a00010000000000 cost=43660
[0 - 7f0fc303f840]    0.852336 {2}{xpath}: inter: 1a00000000000000->1a00010000000000 cost=19760+20660 = 40420 <? 43660
[0 - 7f0fc303f840]    0.852357 {2}{xpath}: last: 1a00010000000000->1e00010000000002 cost=40420+10070400 = 10110820 <? 0

Notice the "inter:" condition fails on the first path and succeeds on the second path.

Ok, @elliottslaughter I'm going to assign this one back to you to figure out how you want to change the profiler to show the same copy requiring different numbers of hops, and then we can change how we log stuff, and then I'll change the runtime to dump data in the new way.

Ok, let me try to summarize to make sure I understand the design constraints.
Currently, it looks like we're logging num_hops on CopyInfo:
https://github.com/StanfordLegion/legion/blob/10cf632e9f65e5511b6b07fbdffbb661af9c147a/runtime/legion/legion_profiling.h#L274
That means, for a copy currently we assume that all pairs of instances involved in the copy (i.e., all CopyInstInfos) have the same num_hops.
It sounds like not only is this not true, but it might not even be true for a given pair SRC, DST. Because different fields can be packed differently, different fields may experience different numbers of hops when transiting between those two instances within a single logical copy operation.
I've been looking at the Rust profiler code, and we currently do not actually use num_hops at all (beyond the work to parse and store it). There are a couple options I can think of:

Remove it. Then there's no issue with tracking conflicting values. Obviously prevents us from showing this to users.
Store it on CopyInstInfo. This is a per-instance and per-field data structure (see below), so that would make the num_hops unique per CopyInstInfo. It would still be TBD how to show this to the user.

https://github.com/StanfordLegion/legion/blob/10cf632e9f65e5511b6b07fbdffbb661af9c147a/runtime/legion/legion_profiling.h#L264-L265

FYI, somehow we parsed the num_hops, but not showed them, I will push a fix for it.

@elliottslaughter The merge request for this has been merged and up-streamed to control replication. Please try again and confirm that you no longer see the issue.

Confirmed.


add flush to ExtendedQueryHandler

Flush is part of https://www.postgresql.org/docs/current/protocol-flow.html#PROTOCOL-FLOW-EXT-QUERY

Thank you!


is the resource manager header intended for public consumption?

In going through the tpm2.0-tools build I'm trying to reverse engineer the dependencies it has on the TPM2.0-TSS code. One thing it's referencing directly is the resource manager header. AFAIK it's doing this to get at only a few status codes.
What's not entirely clear to me is whether or not this file is intended for public consumption. If it is we only need to decide where to install it so that's easy. But from my reading of the TAB/RM specification we reference in the README, it looks like this file may not be intended for public consumption. This would mean that the tpm2.0-tools shouldn't be touching it at all and that the code there needs to be fixed.
@wcarthur1 any thoughts on this?

Philip,
Below…
Will Arthur
Intel Corporation
Server Security Firmware
803-216-2101
From: Philip Tricca<EMAIL_ADDRESS>Sent: Tuesday, January 5, 2016 3:50 PM
To: 01org/TPM2.0-TSS<EMAIL_ADDRESS>Cc: Arthur, Will C<EMAIL_ADDRESS>Subject: [TPM2.0-TSS] is the resource manager header intended for public consumption? (#95)
In going through the tpm2.0-tools build I'm trying to reverse engineer the dependencies it has on the TPM2.0-TSS code. One thing it's referencing directly is the resource manager header. AFAIK it's doing this to get at only a few status codes.
[Arthur, Will C] What’s it doing with them?  All it can do is report them…or is it trying to report some user readable error message?
What's not entirely clear to me is whether or not this file is intended for public consumption. If it is we only need to decide where to install it so that's easy. But from my reading of the TAB/RM specification we reference in the README, it looks like this file may not be intended for public consumption.
[Arthur, Will C] Why do you think that?  What in the RM spec indicates that RM error codes aren’t for public consumption.
This would mean that the tpm2.0-tools shouldn't be touching it at all and that the code there needs to be fixed.
@wcarthur1https://github.com/wcarthur1 any thoughts on this?
—
Reply to this email directly or view it on GitHubhttps://github.com/01org/TPM2.0-TSS/issues/95.

On the first point I was confusing the error levels that the tools are getting from the SAPI headers with the return codes defined in the RM header: https://github.com/01org/tpm2.0-tools/blob/master/sapi-tools/common.cpp#L202
I just did a quick test here and the tools are including the resource manager header but not actually using anything from it. Confusing: https://github.com/01org/tpm2.0-tools/blob/master/sapi-tools/common.cpp#L65
The last bit about the header being intended for public consumption seems to be cleared up by section 4 of the TAB/RM spec. These are the error levels that the tools consume, what I initially confused with error codes. TAB/RM error codes on the other hand are considered implementation specific.
So it's expected that client applications would be consuming the RM header to decode error codes, though these codes are not defined in the spec / are implementation specific. Also the name, location and contents of the header file are all implementation specific. Hopefully I'm not missing this too badly?

Below…
Will Arthur
Intel Corporation
Server Security Firmware
803-216-2101
From: Philip Tricca<EMAIL_ADDRESS>Sent: Tuesday, January 5, 2016 5:11 PM
To: 01org/TPM2.0-TSS<EMAIL_ADDRESS>Cc: Arthur, Will C<EMAIL_ADDRESS>Subject: Re: [TPM2.0-TSS] is the resource manager header intended for public consumption? (#95)
On the first point I was confusing the error levels that the tools are getting from the SAPI headers with the return codes defined in the RM header: https://github.com/01org/tpm2.0-tools/blob/master/sapi-tools/common.cpp#L202
I just did a quick test here and the tools are including the resource manager header but not actually using anything from it. Confusing: https://github.com/01org/tpm2.0-tools/blob/master/sapi-tools/common.cpp#L65
The last bit about the header being intended for public consumption seems to be cleared up by section 4 of the TAB/RM spec. These are the error levels that the tools consume, what I initially confused with error codes. TAB/RM error codes on the other hand are considered implementation specific.
So it's expected that client applications would be consuming the RM header to decode error codes, though these codes are not defined in the spec / are implementation specific. Also the name, location and contents of the header file are all implementation specific. Hopefully I'm not missing this too badly?
[Arthur, Will C] yes, that’s true, except for  the decode part.  I would expect that apps would just say “RM error, 0xXXX”, because the codes aren’t really going to mean anything to apps and there’s no remedial actions they can take.  They are more for RM developers to debug the issue.  The RM error level is spec’d in tss2_common.h and isn’t in a RM specific file.
—
Reply to this email directly or view it on GitHubhttps://github.com/01org/TPM2.0-TSS/issues/95#issuecomment-169149970.

Great. Thanks for clearing that up. I'll be cleaning up / out the use of the RM header in the tools repo as part of this work. But back to my original question: To install the header or not?
From the data in this thread it sounds like:

There's nothing that says we can't / shouldn't.
From your last point it sounds like you believe that there's little value in doing so.

I guess I'd personally error on the side of providing too much information than not enough. Installing the header is probably a good idea just in case someone out there wants to do something with the error codes that the RM may return. Reasonable?

Below…
Will Arthur
Intel Corporation
Server Security Firmware
803-216-2101
From: Philip Tricca<EMAIL_ADDRESS>Sent: Tuesday, January 5, 2016 5:40 PM
To: 01org/TPM2.0-TSS<EMAIL_ADDRESS>Cc: Arthur, Will C<EMAIL_ADDRESS>Subject: Re: [TPM2.0-TSS] is the resource manager header intended for public consumption? (#95)
Great. Thanks for clearing that up. I'll be cleaning up / out the use of the RM header in the tools repo as part of this work. But back to my original question: To install the header or not?
From the data in this thread it sounds like:

There's nothing that says we can't / shouldn't.
From your last point it sounds like you believe that there's little value in doing so.

I guess I'd personally error on the side of providing too much information than not enough. Installing the header is probably a good idea just in case someone out there wants to do something with the error codes that the RM may return. Reasonable?
[Arthur, Will C] Works for me.  But…codes could change since they’re not specified…I’d probably not do it for that reason.
—
Reply to this email directly or view it on GitHubhttps://github.com/01org/TPM2.0-TSS/issues/95#issuecomment-169157380.

I'm happy to leave it as it is now without the header installed if that's what you would prefer. In digging through this it turns out the dependency from the tpm2.0-tools repo is easy to break since it was just including the header but not using anything from it. If there's a need in the future we can revisit this on demand.
Thanks again for the data.


Not clear with the command to run svtools

Hello,
Trying to run svtools to convert Manta generated diploidSV.vcf file, into bedpe but cannot find the command to run it.
I am using this command:
svtools vcftobedpe diploidSV.vcf
but its giving error.

thanks it worked fine


Fixed issue - valueUrl to valueUri

 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.


install simp_le from master branch + make that work, fixes #221

This has been written with some help from https://stackoverflow.com/a/19048754/1959568
I suggest to squash the commits, as the second description is misleading and should read something like update dependencies and installation flow for simp_le, avoiding virtualenv.

This had become necessary due to a weird edge case with
nginx-letsencrypt    | Creating/renewal okfn-de-test.matrix.allmende.io certificates... (okfn-de-test.matrix.allmende.io)
nginx-letsencrypt    | Traceback (most recent call last):
nginx-letsencrypt    |   File "/usr/lib/python2.7/site-packages/simp_le.py", line 1404, in main
nginx-letsencrypt    |     return main_with_exceptions(cli_args)
nginx-letsencrypt    |   File "/usr/lib/python2.7/site-packages/simp_le.py", line 1381, in main_with_exceptions
nginx-letsencrypt    |     existing_data = load_existing_data(args.ioplugins)
nginx-letsencrypt    |   File "/usr/lib/python2.7/site-packages/simp_le.py", line 1153, in load_existing_data
nginx-letsencrypt    |     all_data = IOPlugin.registered[plugin_name].load()
nginx-letsencrypt    |   File "/usr/lib/python2.7/site-packages/simp_le.py", line 371, in load
nginx-letsencrypt    |     with open(self.path, self.READ_MODE) as persist_file:
nginx-letsencrypt    | IOError: [Errno 21] Is a directory: 'key.pem'
nginx-letsencrypt    |
nginx-letsencrypt    | Unhandled error has happened, traceback is above
nginx-letsencrypt    |
nginx-letsencrypt    | Debugging tips: -v improves output verbosity. Help is available under --help.

which is now resolved, as of
nginx-letsencrypt    | Creating/renewal okfn-de-test.matrix.allmende.io certificates... (okfn-de-test.matrix.allmende.io)
nginx-letsencrypt    | 2017-06-25 00:59:31,835:INFO:simp_le:1213: Generating new account key
nginx-letsencrypt    | 2017-06-25 00:59:35,805:INFO:simp_le:1306: okfn-de-test.matrix.allmende.io was successfully self-verified
nginx-letsencrypt    | 2017-06-25 00:59:36,611:INFO:simp_le:1314: Generating new certificate private key
nginx-letsencrypt    | 2017-06-25 00:59:42,555:INFO:simp_le:393: Saving account_key.json
nginx-letsencrypt    | 2017-06-25 00:59:42,555:INFO:simp_le:393: Saving key.pem
nginx-letsencrypt    | 2017-06-25 00:59:42,556:INFO:simp_le:393: Saving chain.pem
nginx-letsencrypt    | 2017-06-25 00:59:42,556:INFO:simp_le:393: Saving fullchain.pem
nginx-letsencrypt    | 2017-06-25 00:59:42,556:INFO:simp_le:393: Saving cert.pem

Pardon me, the error occured due to another reason, as I apparently had created the folders implicitly before the client could go to work:

https://github.com/allmende/docker-matrix/blob/bd611109c17e0a7b747966337d816a78cc96bbe4/docker-compose.yml#L35-L38

Still this PR allows to build a current simp_le version. If you wanted to go for a predictable build, we could set a distinct commit for checkout.

This PR + an additional commit to add a git checkout seems like a good idea to me.


Code won t work once packaged in an executable single jar

Dear,
I have spent the day trying to implement picocli in my project but I am having a weird problem. I basically have the "sub-command" situation like (to simplify, I wrote the test code below )
@Command(name = "top", description = "coucou", subcommands = {Sub1.class, Sub2.class}, synopsisSubcommandLabel = "(sub1 | sub2)")
class Top {

	public static void main(String[] args) {
		System.exit(new CommandLine(new Top()).execute(args));
	}
}

@Command(name = "sub1", description = "blah", version = "1.0", mixinStandardHelpOptions = true)
class Sub1 implements Callable<Integer> {

	@Option(names = {"-p", "--props"}, required = true, description="Path to property file")
	protected File propFile;

    public Integer call() throws Exception {
    	System.out.println("All good, executing Sub1");
		return 0;
	}

}

@Command(name = "sub2", description = "hdsjkal", version = "1.0", mixinStandardHelpOptions = true)
class Sub2 implements Callable<Integer> {

	@Option(names = {"-p", "--props"}, required = true, description="Path to property file")
	protected File propFile;

	/**
	 * the JSON file to submit
	 */
	@Option(names = {"-j", "--json"}, required = true, description="Path to JSON file to submit")
	protected File jsonFile;

	public Integer call() throws Exception {
		 System.out.println("All good, executing Sub2");
		return 0;
	}
}

If I execute Top within eclipse, it works fine but as soon as I generate an executable jar (containing all dependencies) I always get the same result (the runexample is just a wrapper bash):
> ./runexample
Missing required subcommand
Usage: top (sub1 | sub2)
coucou
Commands:
  sub1  blah
  sub2  hdsjkal

> ./runexample sub1
Missing required subcommand
Usage: top (sub1 | sub2)
coucou
Commands:
  sub1  blah
  sub2  hdsjkal

> ./runexample sub1 -h
Missing required subcommand
Usage: top (sub1 | sub2)
coucou
Commands:
  sub1  blah
  sub2  hdsjkal

any idea what is going on ?

forgot to mention : using picocli 4.3.2 and I am using maven.

ok found it, my shell script was swallowing the params...

You already solved this, but in case anything else pops up: picocli offers a tracing feature that can be useful to troubleshoot issues.
Enjoy using picocli!


Mention that row selection can be disabled

Mention in the docs that row selection can be disabled

Preview: https://react-data-view-pr-data-view-29.surge.sh


Can't add fields to Multipart body headers

Romuald (Bug 49949):
When using the HTTPClient tool, we can attach a file or many and name it but in some occasions we would be interested in adding header aside from the usual Content-Disposition and Content-Type, Content-Transfer-Encoding.
Some applications like SAP with their Archivelink HTTP interface are adding fields there (X-compId as an example).
Therefore for such applications it is not possible to simulate a file upload with a POST request as the backend webserver is not able to manage it (doesn't receive the missing header field).
In the attached screenshot I have highlighted in yellow the piece that I would like to configure the same way we can do with the HeaderManager for example.
Hope I didn't use bugzilla by mistake as I didn't find another location to post this enhancement request. If it is the case I am more than happy to post it at the right place.
Thanks.
Created attachment Capture.JPG: Screenshot of a HTTP Request dump with MS NETMON 3.3

OS: Windows Vista

old thread but would be great feature. I'm also testing SAP content server specs. (create POST)
attached my JMeter working example (for the workaround from Sebb : send the entire body)
icc4sap-loadtest-base.jmx.txt
import is to deselect (to make sure the entire body will be send without modification)


Register InputPointerSource pointer event not work

Describe the bug
I tried moving the "DirectX 11 and XAML App (Universal Windows - C++/CX)" template project to WinUI3 c++/winrt.
I get InputPointerSource from SwapChainPanel::CreateCoreIndependentInputSource,
and use the InputPointerSource  to register the pressed/moved/released pointer event and DeviceKinds is Mouse|Touch|Pen.
But use the mouse to rotate the cube not work.
But if I register pointer event from the control SwapChainPanel itself, can work fine.
I push my test project to https://github.com/EmmettTsai/SpinningCube
register pointer event code at here:
https://github.com/EmmettTsai/SpinningCube/blob/71e8c6b982583fb7852b5f8662dc8569a047b874/SpinningCube/DirectXPage.cpp#L55
Steps to reproduce the bug

Deploy with debug/x64
Rotate the cube manually with the mouse not work.

Expected behavior
The cube can be rotated manually with the mouse.
Screenshots

NuGet package version
1.1.2
Packaging type
Packaged (MSIX)
Windows version
Windows 10 version 21H2 (19044, November 2021 Update)
IDE
Visual Studio 2022
Additional context
No response

Apparently, a dedicated DispatcherQueue thread is needed to pump the events. The documentation completely ignores this. Calling swapChainPanel.CreateCoreIndependentInputSource() on UI thread throws RPC_E_WRONG_THREAD error, while having it on a dedicated thread without dispatcher queue fails to deliver the events.
So your test code should change to:
m_dispatcherQueueController = DispatcherQueueController::CreateOnDedicatedThread();

// Register our SwapChainPanel to get independent input pointer events
m_dispatcherQueueController.DispatcherQueue().TryEnqueue([this] {
    // The CoreIndependentInputSource will raise pointer events for the specified device types on whichever thread it's created on.
    InputPointerSourceDeviceKinds deviceKind = (InputPointerSourceDeviceKinds)(
        Microsoft::UI::Input::InputPointerSourceDeviceKinds::Mouse |
        Microsoft::UI::Input::InputPointerSourceDeviceKinds::Touch |
        Microsoft::UI::Input::InputPointerSourceDeviceKinds::Pen);
    m_coreInput = swapChainPanel().CreateCoreIndependentInputSource(deviceKind);

    // Register for pointer events, which will be raised on the background thread.
    m_coreInput.PointerPressed({ this, &DirectXPage::SwapChainPanel_OnPointerPressed });
    m_coreInput.PointerMoved({ this, &DirectXPage::SwapChainPanel_OnPointerMoved });
    m_coreInput.PointerReleased({ this, &DirectXPage::SwapChainPanel_OnPointerReleased });
    });

Make sure to use the DispatcherQueueController in namespace Microsoft.UI.Dispatching, not the one in Windows.System.

@vmilea Thanks for the explanation. It work fine.

Issue has been resolved, close this issue.


[BUG] Exception thrown on auth resolution with empty  .AzureToolsForIntelliJ/AuthMethodDetails.json

Describe the bug
When using the DefaultAzureCredentialBuilder (just new DefaultAzureCredentialBuilder().build();), an exception will be thrown when there is an empty file located at  ~/.AzureToolsForIntelliJ/AuthMethodDetails.json. Unfortunately, the Azure Toolkit for Rider IDE plugin creates an empty file at the location at launch.
This means that developers using Rider are likely to have exceptions thrown when using the DefaultAzureCredentialBuilder.
Exception or Stack Trace
com.azure.core.exception.ClientAuthenticationException: DefaultAzureCredential authentication failed. ---> IntelliJCredential authentication failed. Error Details: com.fasterxml.jackson.databind.exc.MismatchedInputException: No content to map due to end-of-input
 at [Source: (File); line: 1, column: 0]
Caused by: reactor.core.Exceptions$ReactiveException: com.fasterxml.jackson.databind.exc.MismatchedInputException: No content to map due to end-of-input
 at [Source: (File); line: 1, column: 0]
	Suppressed: java.lang.Exception: #block terminated with an error
Caused by: com.fasterxml.jackson.databind.exc.MismatchedInputException: No content to map due to end-of-input
 at [Source: (File); line: 1, column: 0]

To Reproduce
Create an empty file at ~/.AzureToolsForIntelliJ/AuthMethodDetails.json or just launch Rider with the Azure Toolkit for Rider version 3.50.01664-2023.2
Example Snippet
This small kotlin snippet will reproduce the issue
val defaultCredential = DefaultAzureCredentialBuilder().build()
val ctx = with(TokenRequestContext()) {
  scopes = arrayListOf("https://database.windows.net")
  this
}
defaultCredential.getTokenSync(ctx)

Setup (please complete the following information):

OS: OS X 12.6.8
IDE: Rider 2023.2 + Azure Toolkit for Rider 3.50.0-1664-2023.2
Library/Libraries: com.azure:azure-identity:1.9.1
Java version: 11

Thanks for filing this issue, @tkent.  @g2vinay could you please follow up?
/cc @billwert

@tkent
Thank you for reporting this issue.
I am investigating it, will follow up with next steps.

A fix for this is merged via https://github.com/Azure/azure-sdk-for-java/pull/37044
It will be released in beta release this month and GA in Nov 2023 (next month)


User-reported unhelpful error message

Program: https://code.world/#PKfbAVhUWnWApDgs-Vl0EoQ
Message:
program.hs:1:9-22: error:
    Variable not in scope: drawingOfmouth

The student here wanted drawingOf(mouth) and wrote drawingOfmouth instead.  In principle, it might be great to recognize that splitting the word was what was intended, but the missing parentheses as well makes this a poor test case.  I'll close this, but would reconsider the principle if we had a more compelling test case.


Issue with already present repo?

Using the bacalhau-v1.4.0:
❯ bacalhau-1.4.0
Compute over data

Usage:
  /Users/daaronch/code/bacalhau-versions/v1.4.0/bacalhau [command]

Available Commands:
  agent       Commands to query agent information.
  completion  Generate the autocompletion script for the specified shell
  config      Interact with the bacalhau configuration system.
  devstack    Start a cluster of bacalhau nodes for testing and development
  docker      Run a docker job on the network (see run subcommand)
  exec        Execute a specific job type
  help        Help about any command
  job         Commands to submit, query and update jobs.
  node        Commands to query and update nodes information.
  serve       Start the bacalhau compute node
  version     Get the client and server version.
  wasm        Run and prepare WASM jobs on the network

Flags:
      --api-host string         The host for the client and server to communicate on (via REST).
                                Ignored if BACALHAU_API_HOST environment variable is set. (default "bootstrap.production.bacalhau.org")
      --api-port int            The port for the client and server to communicate on (via REST).
                                Ignored if BACALHAU_API_PORT environment variable is set. (default 1234)
      --cacert string           The location of a CA certificate file when self-signed certificates
                                	are used by the server
  -h, --help                    help for /Users/daaronch/code/bacalhau-versions/v1.4.0/bacalhau
      --insecure                Enables TLS but does not verify certificates
      --log-mode logging-mode   Log format: 'default','station','json','combined','event' (default default)
      --repo string             path to bacalhau repo (default "/Users/daaronch/.bacalhau")
      --tls                     Instructs the client to use TLS

Use "/Users/daaronch/code/bacalhau-versions/v1.4.0/bacalhau [command] --help" for more information about a command.

❯ bacalhau-1.4.0 version
17:09:32.354 | INF pkg/repo/fs.go:99 > Initializing repo at '/Users/daaronch/.bacalhau' for environment 'production'
Error: failed to setup repo: failed to reconcile repo: failed to initialize repo: failed to initialize repo: failed to create user key: user key file already exists at path: /Users/daaronch/.bacalhau/user_id.pem
Usage:
  /Users/daaronch/code/bacalhau-versions/v1.4.0/bacalhau version [flags]

Flags:
      --client          If true, shows client version only (no server required).
  -h, --help            help for version
      --hide-header     do not print the column headers.
      --no-style        remove all styling from table output.
      --output format   The output format for the command (one of ["table" "csv" "json" "yaml"]) (default table)
      --pretty          Pretty print the output. Only applies to json and yaml output formats.
      --wide            Print full values in the table results

Global Flags:
      --api-host string         The host for the client and server to communicate on (via REST).
                                Ignored if BACALHAU_API_HOST environment variable is set. (default "bootstrap.production.bacalhau.org")
      --api-port int            The port for the client and server to communicate on (via REST).
                                Ignored if BACALHAU_API_PORT environment variable is set. (default 1234)
      --cacert string           The location of a CA certificate file when self-signed certificates
                                	are used by the server
      --insecure                Enables TLS but does not verify certificates
      --log-mode logging-mode   Log format: 'default','station','json','combined','event' (default default)
      --repo string             path to bacalhau repo (default "/Users/daaronch/.bacalhau")
      --tls                     Instructs the client to use TLS

failed to setup repo: failed to reconcile repo: failed to initialize repo: failed to initialize repo: failed to create user key: user key file already exists at path: /Users/daaronch/.bacalhau/user_id.pem

possibly 1.4.0 only

This specific error will occur when v1.4.0 bacalhau attempts to open a v1.5.0 bacalhau repo.

Ok, but we shouldn't exit 1 and we shouldn't exit 1 on a key already existing - create a new one with a new name if you want or better reuse the key.

The issue arises because Bacalhau repos are not backward compatible. Once a repo is migrated to a newer version - like v1.5 here, older versions (v1.4) cannot read it.
Specifically, here's what's happening:
Initialization Files: In v1.4.0, the presence of ~/.bacalhau/repo.version determines if a repo has been initialized. In v1.5.0, this has been replaced with ~/.bacalhau/system_metadata.yaml as part of rethinking our configuration system (details here).
Backward Compatibility Issue: When v1.4.0 tries to open a repo initialized by v1.5.0, it doesn't find the repo.version file and assumes the repo is uninitialized. It then attempts to initialize the repo but fails because files like user_id.pem already exist, leading to the error you're seeing.

Lets make sure v1.5.0 can handle this in a more graceful way and print better error message

Proposed fix, write repo.version in 1.5.0


[type: feature] support dubbo gray relase.

// Describe your PR here; eg. Fixes #issueNo
Support dubbo gray release.

Use selector to record all service instances.
User can choose the which one to gray release.
If need to gray release, use shenyu's loadbalancer to choose one of the all gray instances.

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #2056 (ae6523b) into master (53704ef) will decrease coverage by 0.58%.
The diff coverage is 10.07%.

@@             Coverage Diff              @@
##             master    #2056      +/-   ##
============================================
- Coverage     60.83%   60.24%   -0.59%
+ Complexity     4034     4021      -13
============================================
  Files           606      610       +4
  Lines         17833    17910      +77
  Branches       1704     1714      +10
============================================
- Hits          10848    10790      -58
- Misses         6110     6236     +126
- Partials        875      884       +9

Impacted Files
Coverage Δ

...ister/AbstractShenyuClientRegisterServiceImpl.java
1.85% <0.00%> (-0.11%)
:arrow_down:

...register/ShenyuClientRegisterDubboServiceImpl.java
35.29% <0.00%> (+5.29%)
:arrow_up:

.../register/ShenyuClientRegisterGrpcServiceImpl.java
29.41% <0.00%> (ø)

...register/ShenyuClientRegisterMotanServiceImpl.java
29.41% <0.00%> (ø)

.../register/ShenyuClientRegisterSofaServiceImpl.java
23.07% <0.00%> (ø)

...er/ShenyuClientRegisterSpringCloudServiceImpl.java
12.50% <0.00%> (ø)

...ster/ShenyuClientRegisterSpringMVCServiceImpl.java
15.21% <0.00%> (ø)

.../register/ShenyuClientRegisterTarsServiceImpl.java
27.77% <0.00%> (ø)

...a/org/apache/shenyu/common/constant/Constants.java
50.00% <ø> (ø)

...ache/shenyu/common/dto/convert/DivideUpstream.java
0.00% <ø> (ø)

... and 36 more

Continue to review full report at Codecov.

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


Fix the Urchin

Urchin is supposed to be able to hit the tiles around it, not just in front of it.

Hayden updated the urchin to attack squares around it.


Fast lyapunov index calculation

Hi there,
I am very interested in the research content of this article,and I want to express my appreciation for making code open source.
I notice that this article uses FLI as a chaos indicator. When solving the FLI of the corresponding solution for an initial value, I first combine the original equation and the variational equation to solve with odeintw, where v(0) is randomly selected some vectors with module length 1 but different directions. Secondly, the exponent_v0 is denoted as lyapunov_exponent_v0 by taking the upper bound of the modular length of v(t) obtained from each initial value v(0) in the 0-T time range. Finally, with respect to different initial values v(0), the final lyapunov_exponent is obtained.
According to the above method and the threshold value in the paper, the test results on H_2, eps=0.02 are obtained as shown in the figure below.

Do you have any suggestions on how I might address this issue? Any guidance or assistance you can provide would be greatly appreciated.
Thanks again for your invaluable support!
Cheers!

Hi,
Thank you for your interest in our work! What's your question exactly? I can help you with the ML side of the paper and the usage of the code. For domain specific questions please reach out prof. Celleti, Gales or Vasile.
Best!
Victor

Hi!
Thanks for your response!
Sorry, I didn't describe my problem clearly.Let me simplify my question as follows.
I noticed that FLI was used as a chaos indicator for testing in the article. Specifically, what is the algorithm for calculating FLI? Because I wrote the code in accordance with the description in the article and the threshold Settings, I did not get the same result as in fig3(b) (my result is the image displayed on github). I don't know what the reason is.
Thanks again for your invaluable support!
Best
Hjy
------------------ 原始邮件 ------------------
发件人:                                                                                "vrodriguezf/mlchaos"                                                                                @.>;
发送时间: 2024年9月28日(星期六) 下午4:55
@.>;
<EMAIL_ADDRESS>主题: Re: [vrodriguezf/mlchaos] Fast lyapunov index calculation (Issue #2)
Hi,
Thank you for your interest in our work! What's your question exactly? I can help you with the ML side of the paper and the usage of the code. For domain specific questions please reach out prof. Celleti, Gales or Vasile.
Best!
Victor
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Update Authoring Documents

Signed-off-by: Maysun J Faisal<EMAIL_ADDRESS>Fixes https://github.com/devfile/api/issues/809

The grammatical suggestions in the CI seem to be for docs whose content were not updated(just the devfile yaml was updated).
Merging as a result.


Style dashboard

@jdhoffa

In all the tabs I kept the intro text not in a card to separate it from cards containing each of the plots. It was a conscious decision to indicate it that it is introductory information. To me it makes sense but I don't have extremely strong opinion on it. I do want to visually separate the introduction with the statistics from the plots.
I want to keep it on top because it it the choice that influences all the rest of the content. So to me it is like 'title content' therefore it should be on top. The text on the landing page will be different from the introductory text once the user made the first choice so I don't think that the visual display should be the same.

I do understand what you mean but... as it is the "sector" and "asset type" selectors just floating up there at the top, and not contained in a card does look a bit bizarre, they sort of look like they were placed there by accident

But in any case, this isn't something worth blocking this PR for.
We can discuss it with the team during the sprint review :-)


Is it some kinda misspelling in v8.1.0-SNAPSHOT?

Hi,
I was looking through the source code and IMO the code is incorrect a little bit:
public static Unbinder bind(@NonNull Object target, @NonNull Activity source) {
    return getViewBinder(target).bind(Finder.VIEW, target, source);
}

I think that the correct Finder is ACTIVITY here, isn't it?
Because later you will try to cast a source to the Finder's type (in this case you will try to cast a passed Activity to the View class and it's impossible of course).

Looks like copy/paste. Want to submit a PR?

Yeah, sure. I will send it soon.

Thanks!
On Thu, May 26, 2016 at 3:49 PM Yegor<EMAIL_ADDRESS>wrote:

Yeah, sure. I will send it soon.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/JakeWharton/butterknife/issues/595#issuecomment-221975209


Cant create FQDN pool with shareNodes: true

Environment

Application Services Version: latest
BIG-IP Version: 15.1.5.x

Summary
Cannot create a FQDN pool with shareNodes set to true. Same configuration works fine if shareNodes is set to false.
When shareNodes is set to true, AS3 does seem to actually create the pool in the Big-IP config seemingly correct, but it reports the task as failed.
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

{
    "class": "AS3",
    "action": "deploy",
    "persist": true,
    "loglevel": "debug",
    "declaration": {
        "class": "ADC",
        "schemaVersion": "3.46.0",
        "id": "123abc",
        "label": "Sample 1",
        "remark": "An HTTPS sample application",
        "controls": {
            "trace": true
        },
        "Common": {
            "class": "Tenant",
            "Shared": {
                "class": "Application",
                "template": "shared",
                "myPool": {
                    "class": "Pool",
                    "monitors": [
                        {
                            "bigip": "/Common/tcp"
                        }
                    ],
                    "members": [
                        {
                            "servicePort": 1998,
                            "addressDiscovery": "fqdn",
                            "autoPopulate": true,
                            "shareNodes": true,
                            "hostname": "example.com"
                        }
                    ]
                }
            }
        }
    }
}

Observe the following error response:

{
    "id": "23c68974-40d3-4897-9b12-ebe62602389c",
    "results": [
        {
            "code": 200,
            "message": "success",
            "lineCount": 22,
            "host": "localhost",
            "tenant": "Common",
            "runTime": 14571
        },
        {
            "code": 422,
            "message": "declaration failed",
            "response": "01020066:3: The requested Node (/Common/example.com) already exists in partition Common.",
            "host": "localhost",
            "tenant": "Common",
            "runTime": 19470
        }
    ],
    "declaration": {
        "class": "ADC",
        "schemaVersion": "3.46.0",
        "id": "123abc",
        "label": "Sample 1",
        "remark": "An HTTPS sample application",
        "controls": {
            "trace": true,
            "archiveTimestamp": "2023-09-07T18:15:58.348Z"
        },
        "updateMode": "selective"
    }
}

Expected Behavior
AS3 creates the FQDN pool as expected without providing an error message.
Actual Behavior
AS3 creates the FQDN pool but reports the declaration was unsuccessful.

Thank you for reaching out.  In order to prioritize and understand the issue better, please reach out to us at<EMAIL_ADDRESS>

Closing as this has been released in AS3 3.48.0


Error when trying to save changes

Whenever I attempt to save any changes I get this error message, I tried looking it up and the only thing that I found was that a file could be write locked or something like that. Me being not too great with this sort of thing had no idea what that meant so I tried to look that up and could not find anything that made any sort of sense to me so hopefully I can ask for help here. I have pasted the error below, thank you for your time.

Error
An exception was thrown (press Ctrl+C to copy):
System.UnauthorizedAccessException: Access to the path is denied.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.IO.FileStream.WriteCore(Byte[] buffer, Int32 offset, Int32 count)
at System.IO.FileStream.FlushWrite(Boolean calledFromFinalizer)
at System.IO.FileStream.Dispose(Boolean disposing)
at System.IO.Stream.Close()
at Gibbed.Borderlands2.SaveEdit.ShellViewModel.<>c__DisplayClass59_0.b__1() in c:\projects\gibbed-borderlands2\projects\Gibbed.Borderlands2.SaveEdit\ShellViewModel.cs:line 571
at Caliburn.Micro.Contrib.Results.DelegateResult.Execute(ActionExecutionContext context) in R:\CMContrib\src\CMContrib.SL\Results\DelegateResult.cs:line 22
OK

It could be write locked or it doesn't have permission to write to that location. You could try saving it to a new location then copying that file over to the correct one?

That doesn't seem to work either.

Just tried disabling my windows defender and that worked... Sorry for the trouble, but thank you for trying.


Update Aqara_QBKG38LM.md

Tested today in HA the single rocker version, works like a charm with no issues.

great, thanks for contributing


New EFS-File-Manager.yaml IAM requested update - Add iam:passedtoservice condition key

I am currently researching how to add this functionality.

Approved this PR request.
https://github.com/aws-samples/aws-efs-simple-file-manager/pull/109


Gantt does not unload content properly

Describe the bug
While upgrading to carbon 2.x.x I have discovered a bug with the Gantt chart where unloading tracks can mess something up, and content from one track will then bleed into another. Resizing the window fixes it, so whatever the bug is must be isolated to the unloadContent. This was not a problem on 1.x.x versions, which you can see by changing the version in the codesandbox below.
Link to CodeSandbox
https://codesandbox.io/s/goofy-blackwell-vgzh9?file=/src/graph.js
Expected behavior
The track(s) are unloaded without any bleeding or merging of other tracks
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: windows, ubuntu
Browser: chrome

@Matthematic Sorry for the inconvenience caused. Currently we are working on it and will get back to you once we are done with fixing the issue.

Duplicate of #245