Generate TGT and "Authenticator"

Hi Steve, would your package work for the strictly client side functionality of generating a particular service TGT and the associated "Authenticator" to then send to the server service for authentication?
If so, do you mind pointing me to the calls that i would use to do it?
Thanks, Scot

For reference, here are the MIT Kerberos5 APIs that we are currently using to do it for non-.NET/non-Core current implementations, where the final call (krb5_mk_req_extended) will generate what i calling the "authenticator" bytes:
krb5_init_context((krb5_context *)&context);
krb5_sname_to_principal(context, (char *) 0, (char *) 0, KRB5_NT_SRV_HST, &server);
krb5_get_default_realm(context, &therealm);
krb5_cc_resolve(context, CCNm_, &ccache);
krb5_cc_get_principal(context, ccache, &client);
krb5_auth_con_init(context, &auth_context);
krb5_copy_principal(context, server, &creds.server);
krb5_copy_principal(context, client, &creds.client);
krb5_get_credentials(context, 0, ccache, &creds, &credsp);
krb5_mk_req_extended(context, &auth_context, auth_options, &cksum_data, credsp, &outbuf);

No sorry. There's no generation in this library. All it does is validate service tickets.

Thanks for the quick response Steve! Any plans to extend it to the client side (generation)?

This is a common question. Assume no. I would like to, but don't have the time.

Understandable.
Sorry, last question: It appears from my very quick look at your code that you are doing this all in C# (ie, not relying on any other (C) library). Is that correct? If so, how hard do you think it would be to do the work for the client side (generation all in C#)?

The language choice is not a problem here. Kerberos uses ASN.1 for all its messages, which is just a PITA to work with. This library also only has an ASN.1 parser, so someone would need to build a way to generate the messages. That's currently the biggest hurdle. There's a ton of Interop work to account for as well.

Yes, my point wasn't language, but instead whether it is an ALL MANAGED solution. Sounds like it is. Thanks for the info on the complexity.


Multiple metric label renames applied randomly

If you provide benthos with multiple label renames to apply:
metrics:
  rename:
    by_regexp:
    - pattern: "resource\\.processor\\.(.+)\\.processor\\.(\\d+)\\.(.+)"
      to_label:
        resource_type: processor
        resource_name: $1
        processor: $2
      value: $3

The assignment of label values will often be random. This is because to_label is a map, and map traversal order is random.

Fix will be in the next release.


Remove CHEBI:CHEBI:41423 from PUBCHEM.COMPOUND:2662 results

Querying PUBCHEM.COMPOUND:2662 returns CHEBI:CHEBI:41423 as an equivalent ID, which is wrong: https://nodenormalization-sri.renci.org/1.3/get_normalized_nodes?curie=PUBCHEM.COMPOUND%3A2662&conflate=true
Note also that MESH:C105934 is labeled as [OBSOLETE] celecoxib -- are we definitely identifying obsolete MESH terms correctly?

A similar issue: @cbizon noticed that in this screenshot, there is a CHEBI:CHEBI: ID but no correct CHEBI ID. I noticed that the top synonym seemed to be the UNII ID rather than a human-readable name (vitamin b12)

The CHEBI:CHEBI: bug seems to be coming from pull_drugcentral in drugcentral.py, which creates an xref file whose second column is expected to be an identifier only; however, for CHEBI (and only CHEBI) this column includes the CHEBI: prefix, and so we double-concatenate it later on. Working on a fix now.


style: remove default set from brand

There is a new PR for this https://github.com/digdir/designsystem/pull/701


Add x87 control on Linux

x87 unit can be controlled on Linux through fpu_control.h header file.
The macro it provides should be wrapped in our common functions.

Already done with the standard fenv functions.


Clearer documentation

Hi I'm trying to get the ODB to work in docker running on ubuntu. I'm still very unclear about what files I need to place where.
I know I need 12cR2 http://www.oracle.com/technetwork/database/enterprise-edition/downloads/index.html
but what flavor? Is it based on my own OS or the docker base image you guys are using?
I downloaded the linux zip and placed it in the right directory and made sure not to decompress the file but I just get a checksum error complaining about the right files.

You need to download the file for Linux, as per the .download file in the repository itself. But, you're not providing the actual error, so it's difficult to tell you how to fix it.
Does the build complete if you bypass the MD5 check?

Not sure what .download file you are talking about could you provide a link?
I downloaded the linux zip here and left it uncompressed:
`waffleiron ॐ  ~/git/oracle/OracleDatabase/dockerfiles/<IP_ADDRESS>:(1d6h18m|git@master!)
9398 ± tree                                                                                ✭
.
├── checkDBStatus.sh
├── checkSpace.sh
├── Checksum.ee
├── Checksum.se2
├── createDB.sh
├── dbca.rsp.tmpl
├── db_inst.rsp
├── Dockerfile.ee
├── Dockerfile.se2
├── installDBBinaries.sh
├── installPerl.sh
├── linuxx64_12201_database.zip
├── runOracle.sh
├── runUserScripts.sh
├── setPassword.sh
├── setupLinuxEnv.sh
└── startDB.sh
`
this is the error:
waffleiron ॐ  ~/git/oracle/OracleDatabase/dockerfiles:(1d6h18m|git@master!) 9401 ± ./buildDockerImage.sh -v <IP_ADDRESS>                                                                                ✭ Checking if required packages are present and valid... md5sum: Checksum.xe: No such file or directory MD5 for required packages to build this image did not match! Make sure to download missing files in folder <IP_ADDRESS>.
thanks for the help!

You've downloaded Oracle Database 12.2 Enterprise Edition but you're not passing -e to ./buildDockerImage.sh, so it's trying to build the default XE image.

I see, stuff is happening now, thanks a lot!


Fix table row rendering for default values of array_syntax and list_syntax

Coverage remained the same at 91.896% when pulling baff07a1d36d75b9fee8b22e1dd37c6b336c94e2 on derrabus:patch-1 into 9064f84898219d78d85263194016ab1a567199ec on FriendsOfPHP:3.0.

Thank you @derrabus.


Could not install public module

When I install the ble module (public repo), I have got different results by two PCs (both are Windows 7 and same version of tool installed such as yotta, git etc).
I got module repository error in one PC:
E:\tmp>yotta install ble
info: get versions for ble
Traceback (most recent call last):
  File "C:\Python27\Scripts\yotta-script.py", line 9, in <module>
    load_entry_point('yotta==0.8.1', 'console_scripts', 'yotta')()
  File "C:\Python27\lib\site-packages\yotta\main.py", line 239, in main
    status = args.command(args, following_args)
  File "C:\Python27\lib\site-packages\yotta\install.py", line 63, in execCommand
    return installComponent(args)
  File "C:\Python27\lib\site-packages\yotta\install.py", line 194, in installComponent
    working_directory = path
  File "C:\Python27\lib\site-packages\yotta\lib\access.py", line 352, in satisfyVersion
    return satisfyVersionByInstalling(name, version_required, working_directory, type=type)
  File "C:\Python27\lib\site-packages\yotta\lib\access.py", line 294, in satisfyVersionByInstalling
    v = latestSuitableVersion(name, version_required, _registryNamespaceForType(type))
  File "C:\Python27\lib\site-packages\yotta\lib\access.py", line 108, in latestSuitableVersion
    vers = remote_component.availableVersions()
  File "C:\Python27\lib\site-packages\yotta\lib\registry_access.py", line 461, in availableVersions
    return _listVersions(self.namespace, self.name)
  File "C:\Python27\lib\site-packages\yotta\lib\registry_access.py", line 102, in wrapped
    return fn(*args, **kwargs)
  File "C:\Python27\lib\site-packages\yotta\lib\registry_access.py", line 251, in _listVersions
    'Check that the name is correct, and that it has been published.') % (name, namespace)
yotta.lib.access_common.Unavailable: ble does not exist in the modules registry. Check that the name is correct, and that it has been published.

E:\tmp>

But, no error in another PC.
D:\tmp>yotta install ble
info: get versions for ble
info: download<EMAIL_ADDRESS>from https://yotta-private.herokuapp.com
info: get versions for frdm-k64f-gcc
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for mbed-gcc
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for mbed-drivers
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for mbed-hal
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for cmsis-core
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for ualloc
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for minar
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for core-util
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for compiler-polyfill
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for mbed-hal-freescale
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for mbed-hal-ksdk-mcu
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for uvisor-lib
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for mbed-hal-k64f
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for mbed-hal-frdm-k64f
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for cmsis-core-freescale
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for cmsis-core-k64f
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for dlmalloc
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for minar-platform
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for minar-platform-mbed
info: download<EMAIL_ADDRESS>from the public module registry

D:\tmp>

Not sure why ble module are downloaded from https://yotta-private.herokuapp.com.
Any suggestion?

I've gotten a different error where yotta cannot resolve the module name 'ble'.
Please refer to the below logs and let me get any advice :

yotta build
info: get versions for nordic-nrf51822-16k-gcc
info: download<EMAIL_ADDRESS>from the public module registry
info: get versions for ble
error: Failed to satisfy dependencies of E:\work\mbed\ble_nordic\ble-nrf51822:
error: ble does not exist in the modules registry. Check that the name is correct, and that it has been published.
error: ble not available
error: Missing dependencies prevent build. Use yotta ls to list them.

ble isn't the name of a module that currently exists – are there instructions somewhere that guided you to try to use that?

the ble module might be limited to the private registry.
There's a github/armmbed/ble-examples repo showing how to build ble examples.

Thanks!

'ble-nrf51822'  has dependency on the 'ble' module.
In the module.json file of 'ble-nrf51822' :
"dependencies": {
"ble": "~0.4.8"
},
So, I think the 'ble' must be a module, and, without it, 'ble-nrf51822' can't be built.
Is it something I'm missing?

yes, the BLE_API respository on github is the 'ble' module.


Review step ux

Unify finish button text and use "Launch" label
Add exit confirmation, when user has reached the review step and thus
filled a lot of data (previous steps will still allow canceling without confirm).

I'd probably say something like "Leave image launch wizard?" but this is on Maria.

Tagging @MariSvirik

Contacted doc team. I would run it past a native speaker.
I would personally go with the microcopy that Ondrej wrote:

PF4 uses "exit" as a verb.
@ezr-ondrej add a period at the end of the sentence.
@lzap using "wizard" in microcopy is not a good practice. Who or what is a wizard?
I would even use the blue primary button. I assume it's not a super destructive action. But that's not very important.

Example from sources

If we already decided on using Launch everywhere (and it's approved by PMs), let's use "Launch"

And add a toast after the user exit the flow successfully.
Example from c.rh.c

I agree with @MariSvirik and her comments but I don't know how often I have ever seen "Stay" as an option even though I am not against using it. "Cancel" is usually the option I see when the user decides they do not want to exit. But I see no problem using "Stay". As she said, use a period at the end of "All inputs will be discarded." otherwise this looks good and makes sense to me.

Given I've implemented all required, I'm merging, this will go through final ack of UX, so if this is still not good enough, we will improve later on :)


Add separate string for successful configuration units

[x] I have signed the Contributor License Agreement.
[x] This pull request is related to an issue.

Resolves #4382

This PR adds a new string for configuration units to give additional clarity between the configuration set being applied successfully and each individual configuration unit being applied successfully

Microsoft Reviewers: Open in CodeFlow

/azp run


Manifest v3 Update

Update this chrome extension app as per new manifest_version: 3 guidelines

Work is still in progress

Merging this PR after reviewing throughly.
Currently Classic GA is disabled. Need to integrate with modern GA4 features.


Document the Policy API concepts

/cc @dhaiducek

/hold for review from @dhaiducek

I'm pretty sure the links still won't work. I might be proved wrong, but the URLs in the actual site all have a trailing slash, so I think you'll need to update all of these:
./sibling-page --> ../sibling-page
../../concept-page --> ../../../concept-page

to get up into the parent. Alternatively, you can do this for the concept pages, which might be a bit cleaner:
../../concept-page --> /concepts/concept-page

I'm confused because the links work in markdown so I don't know why it wouldn't when generated for the website. @mikeshng are you able to provide some help here?

If you're looking for an example in the site for a link that works this way:
https://github.com/open-cluster-management-io/open-cluster-management-io.github.io/blob/d237120ac80dfe0686647e15256b0bbcc586ab6f/content/en/concepts/placement.md?plain=1#L23

Hi guys, sorry I missed the message.
You should run:
make server

and then visit the local generated website with a browser to verify your changes work properly. This website is generated using the Hugo framework. You can create pages using .MD but for links it might work differently.
More details in https://github.com/open-cluster-management-io/open-cluster-management-io.github.io/blob/main/README.md

It would be really nice if the live site and the Markdown worked consistently, but removing that trailing slash is an issue for another day since it'd probably require quite a few link updates...

Thanks @mikeshng and @dhaiducek! I fixed the links and this ready for review again.

/approve

/unhold


Make argument to select cipher

Sometimes the user already knows the cipher

This is dependent on  #104, so that we can use a nice list of ciphers


Support configuration of the xmppAddress

xmppAddress is currently defined as a constant (https://sourcegraph.com/github.com/google/go-gcm/-/blob/gcm.go#L42) - this prevents clients from specifying the pre-prod server at gcm-preprod.googleapis.com:5236, or (more importantly for my use-case) connecting to the newer fcm-xmpp.googleapis.com:5235

Note that to work with FCM, this requires separating xmppHost from xmppAddress as well (as the xmppUser should still be senderId +<EMAIL_ADDRESS>per https://firebase.google.com/docs/cloud-messaging/server)


Readme: Removes experimental from Jest snapshot

Per final comment in #372

Snapshot rendering should actually be pretty stable / useable after React 15.4.1.
See this post for more info.

Thanks!

@gaearon has this time come?

I'm open to reviewing a PR that enables snapshot testing by default.


Merge release/1.12.0 branch into main branch

What does this PR do?
This PR does the merge of release/1.12.0 branch into main branch.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #203 (ed8fa24) into main (67614bb) will decrease coverage by 0.82%.
The diff coverage is 75.00%.

@@             Coverage Diff              @@
##               main     #203      +/-   ##
============================================
- Coverage     86.87%   86.05%   -0.82%
- Complexity      178      181       +3
============================================
  Files            21       21
  Lines           579      588       +9
  Branches         76       78       +2
============================================
+ Hits            503      506       +3
- Misses           49       55       +6
  Partials         27       27

Files
Coverage Δ

.../datadog/gradle/plugin/DdExtensionConfiguration.kt
100.00% <100.00%> (ø)

...m/datadog/gradle/plugin/internal/OkHttpUploader.kt
84.03% <ø> (ø)

...com/datadog/gradle/plugin/DdAndroidGradlePlugin.kt
74.38% <88.24%> (-0.83%)
:arrow_down:

...in/kotlin/com/datadog/gradle/plugin/DdExtension.kt
50.00% <20.00%> (-35.71%)
:arrow_down:


chore(webdriver): fix version support

It's a little confusing to me to have prefix be a property now but suffix() is still an accessor method. I'd recommend making it getSuffix(). Other than that, looks good.

The accessor methods will change with #607.


Support for partitionless panels e.g. iConnect2.

Hey there.
I've added support for partitionless control panels such as the iConnect2 ref. https://github.com/home-assistant/core/issues/40290. I've tested the changes in my Home Assistant and I have no errors in the log from the Risco integration. I hope I haven't broken anything for the panels that support partitions.
Please let me know if you think anything should be done differently.
Thanks.

Thanks @OnFreund. I won’t have time to work on this for the next few weeks so if somebody else want’s to pick it up please feel free to do so. If not I’ll have a go when time permits. :-)

@SimonThoustrup Hi Simon, are there any developments in this area since February? I am still struggling with my 'Partitionless' Risco (MyElas) iConnect2 alarmsystem.
Any workarounds that you are aware of?

	Hi. Sorry but I haven’t had time to do the refactoring. The patch I made still works though so you might get away with placing the patched risco.py in site-packages/pyrisco/ in your HA install. I hope I will have some time to fix this properly sometime next year.
	Sent from my phone - likely short

Fra: Rookie-Ron @.***>Sendt: torsdag, december 30, 2021 2:28 PMTil: OnFreund/pyriscoCc: SimonThoustrup; MentionEmne: Re: [OnFreund/pyrisco] Support for partitionless panels e.g. iConnect2. (#2)
@SimonThoustrup Hi Simon, are there any developments in this area since February? I am still struggling with my 'Partitionless' Risco (MyElas) iConnect2 alarmsystem.
Any workarounds that you are aware of?
—Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.Message ID: @.***>

@SimonThoustrup Thanks for the hint, no luck however. Maybe I do not have the process clear?
First I installed pyrisco in my Python3.9 /site-packages subfolder. Thereafter I overwrote 'risco.py' by your patched file (dated February 2, 2021). After restarting Home Assistant the old error message remains: "File "/usr/local/lib/python3.9/site-packages/pyrisco/risco.py", line 134, in partitions self._partitions = {p["id"]: Partition(p) for p in self._raw["partitions"]}".
This seems strange, as the file the error points to is your patched one, in which line 134 has your updated content.
Probable I need to a different installation process? Can you help me?

@SimonThoustrup @OnFreund
Being a 'rookie' in Docker indeed (running Home Assistant core-2021.12.7 in Docker), I discovered I needed to replace the 'risco.py' within the Home Assistant container, rather than in the host OS directory... After a restart of HAss the patched 'risco.py' resulted in the proper 'alarm_control_panel' domain and panel status indication, also in Lovelace:

Some issues remain for this moment:

"inschakelen voor thuis" (= arm_home) does not function (Risco 'partial_arm'?), and results in the log:

Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 185, in handle_call_service
await hass.services.async_call(
File "/usr/src/homeassistant/homeassistant/core.py", line 1495, in async_call
task.result()
File "/usr/src/homeassistant/homeassistant/core.py", line 1530, in _execute_service
await handler.job.target(service_call)
File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 209, in handle_service
await self.hass.helpers.service.entity_service_call(
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 663, in entity_service_call
future.result()  # pop exception if have
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 896, in async_request_call
await coro
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 700, in _handle_entity_call
await result
File "/usr/src/homeassistant/homeassistant/components/risco/alarm_control_panel.py", line 148, in async_alarm_arm_home
await self._arm(STATE_ALARM_ARMED_HOME, code)
File "/usr/src/homeassistant/homeassistant/components/risco/alarm_control_panel.py", line 174, in _arm
await self._call_alarm_method(risco_state)
File "/usr/src/homeassistant/homeassistant/components/risco/alarm_control_panel.py", line 177, in _call_alarm_method
alarm = await getattr(self._risco, method)(self._partition_id, *args)
File "/usr/local/lib/python3.9/site-packages/pyrisco/risco.py", line 409, in partial_arm
return Alarm(await self._site_post(CONTROL_URL_NO_PARTITIONS, body))
File "/usr/local/lib/python3.9/site-packages/pyrisco/risco.py", line 300, in _site_post
return await self._authenticated_post(site_url, site_body)
File "/usr/local/lib/python3.9/site-packages/pyrisco/risco.py", line 287, in _authenticated_post
raise OperationError(str(json))
pyrisco.risco.OperationError: {'result': 52, 'validationErrors': None, 'errorText': None, 'errorTextCodeID': '0', 'status': 200, 'response': {'classVersion': 0, 'partitions': None, 'systemStatus': 0, 'systemReady': True, 'trouble': False, 'bellStatus': 0, 'alarmPending': False, 'batteryLow': False, 'acLost': False, 'haEnabled': False, 'bellOn': False, 'zones': None, 'haDevices': None, 'users': None, 'devCollection': None, 'exitDelayTimeout': 0, 'armNotAllowed': False, 'disarmNotAllowed': False, 'cpTime': '2022-01-02T21:05:01.9332344+00:00', 'partReadySupported': False, 'partFullReady': False, 'part1Ready': False, 'part2Ready': False, 'type': 2}}
Function "inschakelen voor vertrek" (=arm_away) does work as expected.

If I activate "inschakelen voor thuis" (= arm_home) through the alarm panel itself (and my mobile phone) it results in status 'unknown' when the alarm is (partially)armed:

For this: Comments on the Log? Is there something I should do in the mapping of 'arm_home' versus 'partial_arm'? Is 'arm_home' supported anyway?

Hi @Rookie-Ron
How did you replace the  'risco.py' within the Home Assistant container? I cant find the path. I am running HA OS on oracle virtualbox.
Can you please post a guide on how you did it?
Thank you.

Hi @Rookie-Ron
How did you replace the 'risco.py' within the Home Assistant container? I cant find the path. I am running HA OS on oracle virtualbox.
Can you please post a guide on how you did it?
Thank you.

I am running Home Assistant (core) in a Docker Container. The container has his own file structure, which you can access with command:
sudo docker exec -it homeassistant sh
When you change to the main directory with cd /, the ls command will show you the file structure.
'risco.py' is under /usr/local/lib/python3.9/site-packages/pyrisco
To replace the default file with the patched one, from the directory where you saved the patch, use:
sudo docker cp risco.py homeassistent:/usr/local/lib/python3.9/site-packages/pyrisco/risco.py
One downside: after each update of Home Assistant you need to copy the patch to replace the original file.
Good luck!

Hey there,
Are you having issues after update Homeassistant 2022.8.0?
I replaced the default file with the patched one:
sudo docker cp risco.py homeassistant:/usr/local/lib/python3.10/site-packages/pyrisco/risco.py
and I get an error: Error while setting up risco platform for alarm_control_panel
It worked flawlessly before the update.
Any ideas?
Thank you!

The API has changed with the latest version, so whatever patch you're using will have to change accordingly.

Thank you @OnFreund for the explanation.
I am using @SimonThoustrup 's patch. Is this something that can be done easily?

Don't know - you'll have to ask Simon...

There hasn't been any real activity here for over 18 months, and the library has changed quite significantly in the mean time, so I'm closing this.

Hi @SimonThoustrup are you planning to make changes to your patch?

Hi @SimonThoustrup, same issue and question from my side as @ASOUAMES. Are you planning an update to your patch to match the new API? Any hints how to do it myself?

Perhaps @SimonThoustrup is busy and cannot deal with this at the moment. Our only solution is for partitionless panels to be supported by @OnFreund. Hopefully one day.

@ASOUAMES as I mentioned multiple times - I don't have a partition-less panel and I don't have to way to develop and test for it.
I'm happy to provide assistance, but someone else will have to make it happen.

Understood, what if I gave you access to my panel? My knowledge of python is s..t.

That's not something I can develop remotely. It requires quick iterations
I don't have the API for local communication

https://github.com/lucacalcaterra/risco-mqtt-bridge/issues/52
Found a work around. Perhaps it will help.


Can we have a 0.4.2 tag?

It would be nice to have a 0.4.2 version tag: https://github.com/ros/class_loader/commit/8864be372a83921f15906ae80752be9d6a300b81
Thanks!

Oof, it's been a while. I don't know what happened here. It looks like the 0.4.2 commit was tagged with 0.5.0? @nuclearsandwich

Oof, it's been a while. I don't know what happened here.

It looks like the 0.4.2 commit was tagged with 0.5.0?

What gives you that impression? I see the commit 812744d284cd48cf575a15389d106f5d4c8903c0 tagged 0.5.0
and 8864be372a83921f15906ae80752be9d6a300b81 is untagged.
I bumped class_loader in noetic to 0.5.0 to allow patch room in melodic. It seems like I never completed a 0.4.2 release into Melodic.

Uh, yeah, I had a brainfart I think. I'll just push the tag.

Tag added.


gradient explosion in TAPT DAPT pretraining

Hi, I am trying to reproduce the results of AdaptSum and met problem when pretaing the model in the TAPT setting. It worked quite well for the science and debate datasets, where the data size is small. However, when I trained TAPT for social media, the loss function got exploded:
I run pretraining with command:
python ./src/tapt_pretraining.py -path=./dataset/'social media'/TAPT-data/train.source
-dm='social media'
-visible_gpu=1
-save_interval=1000
-recadam
-logging_Euclid_dist
and the training process witnesses the loss exploding to NaN:
(Epoch 0) LOSS: 2.291335 Euclid dist: 322.301648        13%  1999/15089  [17:55<1:47:14,    2.03it/s]
(Epoch 0) LOSS: 2.246833 Euclid dist: 959.653581        20%  2999/15089  [26:46<1:39:52,    2.01it/s]
(Epoch 0) LOSS: 9.272711 Euclid dist:<PHONE_NUMBER>718079518205927655211008.00000        33%  3999/15089  [35:40<1:46:22,    1.74it/s]
(Epoch 0) LOSS: nan Euclid dist: nan        40%  4999/15089  [44:14<1:21:29,    2.16it/s]
(Epoch 0) LOSS: nan Euclid dist: nan        46%  5999/15089  [52:34<1:10:48,    1.80it/s]
(Epoch 0) LOSS: nan Euclid dist: nan        53%  6999/15089  [1:01:15<1:14:45,    1.49it/s]
I tried to lower learning rate to 0.01 and adjust the gradient clip value, it put the time of loss explosion later, but didn't solve the problem. Am I missing something or doing it wrong? What should I do in order to control the model?

Hi Danshi,
When using RecAdam in TAPT, you can also set different  "anneal_t0" and "anneal_k", because RecAdam optimizer is very sensitive to these two parameters. In our experiments, as report in the paper, "we select the best t0 and k in {500, 600, 700, 800, 900, 1, 000} and{1e−2,1e−3,1e−4,1e−5,1e−6}". So the default number of these two parameters may cause the loss explosion problem.

Hi @TysonYu
Great work, I have a similar question regarding SDPT pretraining
Gradient is exploding so early. I was wondering what were the optimum numbers of for t0 and k in that case?


Fails to connect and hang on "LNL Relay"

Hey there,
While using the mod, noticed this bug while trying to connect to hosts that don't have the mod.

How to reproduce:

Try to connect to a session that doesn't have IPv6 capabilities
Wait for IPv6 punchthrough to fail
Wait for IPv4 punchthrough to fail
Indicator hangs on "LNL Relay" and connection to the session is never done

Troubleshooting tested:

Disabling the mod in RML settings – has no effects and bug still appears for some reason
Removing completely the mod – resolves the issue

Mod log + trace: https://gist.github.com/jae1911/24cfe16bca6be03f6b82e857e12696e4
Tested on mod version: 2.0.0
Tested on game version: 2<IP_ADDRESS> (same behavior observed on previous versions)

3.0.0 fixed this issue.


Stores resource definitions in database.

Updated the provisioning stub read resources from the database. Also if a resource is not present, the resources directory is checked to see if a resource is present. This helps in loading the existing resources

Thanks for this nice contribution!
There are some minor issues in the code, but they are only minor, so I'm going to merge the pull request and add some comments to the commit, hoping you can address the issue in a follow up pull request.
I'm also only picking up the first commit from the request as the second one seems to target another branch from your fork (looks like they both contain the same changes).

@rajkiranrbala Actually, I'm not able to pull only the first commit from the pull request. Would you mind removing commit b897d63 from the request? then I'll be able to merge it. Thanks!

I've added some comments to your commit. You can address them in this pull request if you want, or a subsequent one. Like I said, I will merge it (and Thanks for code!), just need you to discard the second 'merge into upstream' commit. Thanks!

I will make the changes based on your suggestion

I removed the commit b897d63.

OK Thanks for removing the second commit. I've merged the pull request. Please open another one when you've addressed my review comments :) Thank you!


allow exporting types using keyof typeof

In the past I have found it very useful to be able to export types using keyof typeof someObject. It had been working previously, but volar has just started complaining:

While in theory exporting types only at the top of the file makes sense, it means that you cannot create a type based on a const. Also, doing so works fine in a regular *.ts file.
Why would I want to do this? It allows you to have some arbitrarily complex configuration associated with some set of string keys, and then create a type that is only valid when using one of those keys, without having to define the list of possible values more than once.
For example, I can use it to create a list of possible icon names, with a mapping to specific Icons.
import CheckIcon from<EMAIL_ADDRESS>import CheckCircleIcon from<EMAIL_ADDRESS>// ...

const POSSIBLE_ICONS = {
  check: CheckIcon,
 'check-circle': CheckCircleIcon,
  // ...
};

export type IconNames = keyof typeof POSSIBLE_ICONS;

there are many other cases where this can be useful as well...
Note - may be somewhat related to https://github.com/johnsoncodehk/volar/issues/923 which is just complaining about exporting additional types altogether.

More digging and I've found that moving these extra exports to an additional <script> block alongside the <script setup> block seems to do the trick.
I think this may be the intended behaviour, but the vue docs are a bit lacking in specifics about TS, and the fact that it used to work but changed suddenly created the extra confusion.

One potential remaining issue is what to do if we need access to the const object in the template
<script lang="ts">
  // I need to define config here in order to export the type
  const CONFIG = {
    a: 'a',
    b: 'b',
  };

  export type ConfigKeys = keyof typeof CONFIG;
</script>

<script lang="ts" setup>
  // we do have access to CONFIG and ConfigKeys here!
  // how can I re-export CONFIG back to the template? other than some hacky renaming?
</script>

<template>
  <!-- We do not have access to CONFIG here :( -->
</template>

Closing this.
See related https://github.com/johnsoncodehk/volar/issues/940
Will open above question in vue/core

One potential remaining issue is what to do if we need access to the const object in the template
<script lang="ts">
  // I need to define config here in order to export the type
  const CONFIG = {
    a: 'a',
    b: 'b',
  };

  export type ConfigKeys = keyof typeof CONFIG;
</script>

<script lang="ts" setup>
  // we do have access to CONFIG and ConfigKeys here!
  // how can I re-export CONFIG back to the template? other than some hacky renaming?
</script>

<template>
  <!-- We do not have access to CONFIG here :( -->
</template>

This is actually working for me, but I don't that is not working in previous vue version. Have you tried update to latest vue version?

@johnsoncodehk regarding your question, I just tried with v0.33.10 and with v0.33.13 and this does not work:
<script lang="ts">
import { defineComponent } from 'vue'

export type CustomType = {}

export default defineComponent({ ... })
</script>

<template>
  <!-- type CustomType = /*unresolved*/ any -->
  <!-- Cannot find name 'CustomType'.ts(2304) -->
</template>


Feature Spec: FPGA Power Servo Integration

Feature Description
Currently, power servo is implemented only for WLAN and is performed via a software servo. Ideally, we would leverage the FPGA power servo IP from NI to result in speed improvements and open the functionality up for other code modules.
Class Definition
Given that this is a function of generation, I think that it makes the most sense for the code to exist inside of the existing SG module rather than creating a new class.
API Definition
Currently, using the power servo is something of a pain in automation as there are two possible paths that need to be followed based on whether or not power servo is selected. My thought is to simplify this by creating a single function that would perform both.
New Types
//Defines the two valid modes for power targeting, since one function performs both
public enum PowerTargetMode { Input, Output };

//Defines the parameters necessary for configuring the power servo
public struct ServoConfiguration
{
    public int MaxIterations;
    public double ServoTolerance_dB;
    //etc.
}
public static ServoConfiguration GetDefaultServoConfiguration()
{
    return new ServoConfiguration
    {
        MaxIterations = 10,
        ServoTolerance_dB = 0.05
        //etc.
    };
}
//Defines the results that will be returned from the SetPowerLevel function
public struct PowerResults
{
    public double FinalInputPower_dBm;
    public double FinalOutputPower_dBm;
    public bool ServoSuccess;
    public double[] ServoTrace;
    //etc.
}

New Methods

public static PowerResults SetPowerLevel(NIRfsg rfsgSession, double powerLevel_dBm, PowerTargetMode powerTarget = PowerTargetMode.Input,
    ServoConfiguration servoConfig = new ServoConfiguration())
{
    //C# has no method for supplying default values for a struct
    //Hence, here we evaluate if two necessary parameters of the servo configuration are default (0).
    //If so, the most likely case is that the servoConfig parameter was left as default; hence, we
    //set it to the default value.
    if (servoConfig.MaxIterations == 0 && servoConfig.ServoTolerance_dB == 0)
        servoConfig = GetDefaultServoConfiguration();

    switch (powerTarget)
    {
        case PowerTargetMode.Output:
            //Perform the FPGA-based power servo

            return new PowerResults
            {
                //Fill in the results
            };
            break;
        default:
            //Simply set RFSG power to requested power level

            //Return results corresponding to the input power, while leaving the others to be
            //initialized to null values
            return new PowerResults
            {
                FinalInputPower_dBm = powerLevel_dBm,
                ServoSuccess = true
            };
            break;
    }
}

Proposed API Usage
Since the function defaults to using input power, a developer could simply call
SetPowerLevel(rfsgSession, 3);
to set the input power level to 3. Otherwise, a call for power servo might look like this:
SetPowerLevel(rfsgSession, 3, PowerTargetMode.Output, GetDefaultServoConfiguration());

@cindyxie After giving it some thought, I had an idea of how I think this might be best implemented. Could you review the feature spec above and let me know what you think? You can access the stub code that is shown in the feature spec above in commit 87933ae4229ae6a8f73b8df86f8899fa1c866a15 in the FGPA Power Servo branch


allow CRISPRidentify to be run from anywhere :: not just archive dir

Hello,
I was trying to install CRISPRidentify using standard unix conventions
and allow our users to ru it from anywhere.
tools and dspecific data tools path are broken as theyt are hard coded path relative to the archive tree. egg
tools/vmatch/vmatch  tool hardcoded
tools/prodigal/prodigal tool hardcoded
tools/hmm_search/hmmsearch --tblout result_hmm.out tools/hmm_search/models_tandem.hmm protein_results.fa hmme and model hardcoded
tools/hmm_search/models_tandem.hmm data hardcoded
tools and data shoul be searched relative to the install directory.
regards
Eric

Hello Eric
The vmatch, prodigal and hmmsearch tools are now used with the conda env as you suggested. The binaries for these tools were removed.
I will fix the hardcoded paths to the models shortly. Thanks for the suggestion.
Best,
Alex

fixed now


Add calibratable parameters to bmi_topmodel.c

Current behavior
TOPMODEL reads in parameter values from params.dat. If you want to change parameter values, you need to edit that file.
Expected behavior
A recent pull request in the ngen repo gives Nextgen the ability to change parameter values. This means we can use the model engine to override parameter values from params.dat. To do this, we'll need to edit bmi_topmodel.c to include calibratable parameters. We should follow the updated bmi_cfe.c file as a template: https://github.com/NOAA-OWP/cfe/blob/master/src/bmi_cfe.c
Steps to perform

Identify all parameters to add
Add #define PARAM_VAR_NAME_COUNT
Add static const char *param_var_names[PARAM_VAR_NAME_COUNT] and static const char *param_var_types[PARAM_VAR_NAME_COUNT]
Update static int Get_value_ptr to include all added parameters

Parameters to include
The following vars are included in params.dat (thanks @madMatchstick for the table!)

Variable
Datatype
Limits
Units
Role
Process
Description

subcat
char
256

info_string

character title of subcatment; often same as model title

szm
double

meters
parameter_fixed
rainfall-runoff
exponential scaling parameter for the decline of transmissivity with increase in storage deficit; units of depth

t0
double

ln(meters^2)
parameter_fixed

areal average of ln(a/tanB)

td
double

hours
parameter_adjustable
rainfall-runoff
unsaturated zone time delay per unit storage deficit

chv
double

meters/hour
parameter_fixed
overland flow
average channel flow velocity

rv
double

meters/hour
parameter_fixed
overland flow
internal overland flow routing velocity

srmax
double

meters
parameter_adjustable
rainfall-runoff
maximum root zone storage deficit

Q0
double

meters/hour
state

initial subsurface flow per unit area

sr0
double

meters
state

initial root zone storage deficit below field capacity

infex
int
Boolean

option
green-ampt
set to 1 to call subroutine to do infiltration excess calcs; not usually appropriate in catchments where Topmodel is applicable (shallow highly permeable soils); default to 0

xk0
double

meters/hour
parameter_adjustable
rainfall-runoff
surface soil hydraulic conductivity

hf
double

meters
parameter_adjustable
green-ampt
wetting front suction for G&A soln.

dth
double

parameter_adjustable
green-ampt
water content change across the wetting front; dimensionless

@xfeng2021, can you look at the table above and let me know which parameters you'd like added?

I forwarded the table with the list of parameters to you. Thanks.

Also, a few more details on implementing/updating the BMI components can be found in noaa-owp/CFE#44


Add an Active Projects page

An action item for the GeoCAT-Xdev meeting was to create an active projects page on the ESDS site.
Some projects to include would be:

GeoCAT

comp
viz

Diagnostics efforts?
ESDS page?

Include the Diagnostics efforts if there is a page (i.e., link) to them and/or if they are official projects with someone behind them.
What do you mean by the "ESDS page"?
Include the Xdev efforts like intake-esm and jupyter-forward, too.  Maybe there are others?  @NCAR/xdev

Include the Diagnostics efforts if there is a page (i.e., link) to them and/or if they are official projects with someone behind them.
What do you mean by the "ESDS page"?
Include the Xdev efforts like intake-esm and jupyter-forward, too.  Maybe there are others?  @NCAR/xdev

Sounds good - by the ESDS page, I meant this repo? But we can leave that out.


Add docs for recommended patterns for multi-tenancy in DI

See https://github.com/aspnet/AspNetCore/issues/2964#issue-305746022

@Antaris would you be able to provide a draft of this doc? cc @sebastienros

@Rick-Anderson Happy to have a look at this 👍

@Rick-Anderson I know I am now half a year late on this (changing jobs, etc. :-)), but I've finally sat down to start putting some guidance together. Hope to have a draft with you by the end of this week

@Antaris  just checking in


API_DEBUG & APP_DEBUG Off Still Shows Too Much

Can anyone suggest the best way to avoid the following types of messages appearing:
{
  "message": "SQLSTATE[42703]: Undefined column: 7 ERROR:  column \"device_token\" of relation \"installations\" does not exist (SIC)",
  "code": "42703",
  "status_code": 500
}

I can catch all HTTP Exceptions but I want to let some through. Should I do a string check for SQLSTATE or is there a more elegant method?

I implemented this in the routes.php (Modified from another post):
/* Dingo still outputs exceptions in non debug mode such as db query exceptions etc, we need to disable that for security purposes*/
app(\Dingo\Api\Exception\Handler::class)->register(function (\Exception $exception) {

    // If mysql exception, then don't show details of query
	if (!env('API_DEBUG') && $exception instanceof \Illuminate\Database\QueryException) {

	// Mimic the normal API response with a different message
        return \Response::make([
            'message'     => 'Query Issue',
            'status_code' => 500,
        ], 500);
	}
});

why don't you register directly QueryException ?

Yes you should be able to directly register that QueryException to save the instanceof check. This is the best solution for this.

Awesome thanks will do.


Add estimate r on deaths

From the updated plots.rst:
Conventionally, the R number is computed on infections (here shown in plot 4 and
referred to as "R based on cases"). Assuming a fixed mortality rate, we can also
use the reported deaths to estimate R (as R depends not on the total numbers but
their change over time). This is done in plot 5, and the estimated R is shown in
yellow. See discussion of plot 4 above for details.

This computation of R based on deaths can be of interest, for example, for the
case of the `United Kingdom <html/United%20Kingdom.html>`__, where the daily new
infections remained approximately constant from about 10 April to about 10 May,
but at the same time the daily number of deaths shows a slow but persistent
reduction from day to day. A possible explanation for the situation would be
that the nearly constant number of daily infections results from a saturation of
the testing capacity, and that actually the spread is reducing (which is
reflected in the deaths, which may be more accurately captured and reported). In
that case, the R based on deaths may provide a more accurate estimate.

If deaths and infections are similarly captured and reported, both entities
should point to the same value, although the R based on deaths could be trailing
the estimate of R based on infections by several weeks; reflecting the duration
of the illness.

Plan to use this branch to compute the numbers for tomorrow, we can then see how this comes out.

Codecov Report

Merging #37 into master will increase coverage by 0.21%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #37      +/-   ##
==========================================
+ Coverage   85.90%   86.12%   +0.21%
==========================================
  Files           3        3
  Lines         589      591       +2
==========================================
+ Hits          506      509       +3
+ Misses         83       82       -1

Impacted Files
Coverage Δ

coronavirus/coronavirus.py
84.68% <100.00%> (+0.25%)
:arrow_up:

Continue to review full report at Codecov.

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

Produced reasonable results. Should consider not computing the data if the cases are too small.


Request argument to choose Port when running remotely

Is your feature request related to a problem? Please describe.
No. I would like to specify a port when running DL4MicEverywhere so that it does not use the default port.
Describe the solution you'd like
Allow to specify a port for forwarding when running DL4MicEverywhere remotely with a --port flag or argument.

Hi @nunopimpaomartins,
I’ve added the requested feature in commit 65fb2e53f8ac32cc1145036461686c27359e75f0.
When using the command line, the port number can now be forwarded with the -p argument. The program will still check if that port is available, and if it’s not, it will incrementally try the next port until it finds an available one. However, if the user specifies a port, we assume that it is available and will open on the selected port.
Thanks a lot for the feedback and contribution! 🤗


Support time start/end for BusinessHour & CustomBusinessHour

[X] Closes #396
[X] Tests added: Please use assert_type() to assert the type of any return value

Created PR here: https://github.com/pandas-dev/pandas/pull/49774


error: websocket: bad handshake with fly intercept

Any tips on how to resolve this issue when trying to fly intercept?
Thanks!

If you're using nginx or some other reverse proxy, make sure it's configured to support websockets. Local networking (i.e. enterprise VPN/etc) could also be getting in the way.

If you're using an AWS ELB, you have to make sure that the protocol forwarding to concourse:web on port 8080 is ssl and not https. On Kubernetes, this is achieved in your Service Definition with :
"annotations": {
      "service.beta.kubernetes.io/aws-load-balancer-ssl-cert": "arn:aws:iam::derpderpderp",
      "service.beta.kubernetes.io/aws-load-balancer-protocol": "ssl",
      "service.beta.kubernetes.io/aws-load-balancer-backend-protocol": "tcp",
      "service.beta.kubernetes.io/aws-load-balancer-ssl-ports": "443"
}

To expand on @ls-avanier comment, this comment to another thread clarified it even further to those who has setup ELB following the concourse installation guide.

...make sure your elb is set up with TCP or SSL listeners and not http or https ... you'll want to keep it that way so hijack can work.

And our listeners now look like this:
TCP 80 to TCP 8080
SSL 443 to TCP 4443
HTTP 222 to HTTP 2222
This setup seems to work correctly with intercept/hijack.

I actually ran into this issue but because I had an istio sidecar running alongside my concourse-web container.  Is there a good explanation of exactly how the fly intercept command works and what ports the communication goes back and forth on?   My guess is that the worker has some requirement to speak directly with concourse-web on an undisclosed port after the initial connection is established.

@jensendw There's no special port, it just goes over Garden API (port 7777 on the worker by default), though this has the same requirements - it can't be going through a buffering HTTP(S) load balancer.
(also, sorry - this repo was meant to be archived, so I'm going to re-archive it, but feel free to open an issue or ask for help in the forums)


[Feature Request] Regex-based mappings

It would be very nice if something like this was supported:
mappings: {
  "https://esm.sh/(@?[^@]+)@([^/]+)/(.*).js": {
    name: "$1",
    version: "^$2",
    subPath: "$3.js",
  },
},

More discussion in https://github.com/denoland/dnt/issues/308


feat: wip add automated tests (based on prometheus-service)

Check #47 for more details

Looks like this PR was merged automatically when I created a new PR without GHA integration tests here: https://github.com/keptn-sandbox/datadog-service/pull/49

Created a new PR for wip integration tests here: https://github.com/keptn-sandbox/datadog-service/pull/51


Read GET Request with multiple JMX MBean entries

Hi there.
I'm using jolokia jvm agent 1.3.6 and metricbeat jolokia module to parse jmx metrics of a tomcat 7.0.41 server.
When I launch a list GET request to jolokia, I can see 3 datasources listed (2 of them in the jolokia list extract json output below).
To be compliant with metricbeat jolokia module functionning, I'm trying to get the corresponding datasources attributes through a READ GET request, but I can't find the right syntax to do so.
I either get a java.lang.IndexOutOfBoundsException or a javax.management.InstanceNotFoundException.
Have you any tips to do so? Thanks for your help.
Jolokia list GET request json output extract (I'm able to parse GC entries attributes with metricbeat jolokia module but not Datasource attributes) :
"request": {
  "type": "list"
},
"status": 200,
"timestamp":<PHONE_NUMBER>,
"value": {
  "Catalina": {
    "class=javax.sql.DataSource,context=/akuiteo,host=localhost,name=\"jdbc/V9GestionDS\",type=DataSource": {
      "attr": {
        "accessToUnderlyingConnectionAllowed": {
          "desc": "Introspected attribute accessToUnderlyingConnectionAllowed",
          "rw": true,
          "type": "boolean"
        }
      },
      "class": "org.apache.tomcat.util.modeler.BaseModelMBean",
      "desc": null
    },
    "class=... ,context=/akuiteo.clients,host=...,name=\"jdbc/V9GestionDS\",type=DataSource": {
      "attr": {
        "accessToUnderlyingConnectionAllowed": {
          "desc": "Introspected attribute accessToUnderlyingConnectionAllowed",
          "rw": true,
          "type": "boolean"
         }
       }
     }
  }
  "java.lang": {
     "name=PS MarkSweep,type=GarbageCollector": {
       "attr": {
         "CollectionCount": {
           "desc": "CollectionCount",
           "rw": false,
           "type": "long"
         },
         "CollectionTime": {
           "desc": "CollectionTime",
           "rw": false,
           "type": "long"
          }
       },
       "class": "sun.management.GarbageCollectorImpl",
       "desc": "Information on the management interface of the MBean"
      },
      "name=PS Scavenge,type=GarbageCollector": {
        "attr": {
          "CollectionCount": {
            "desc": "CollectionCount",
            "rw": false,
            "type": "long"
          },
          "CollectionTime": {
            "desc": "CollectionTime",
            "rw": false,
            "type": "long"
          }
        },
        "class": "sun.management.GarbageCollectorImpl",
        "desc": "Information on the management interface of the MBean"
      }
    }
  }

What exactly did you try to read the datasources ?

I tried different syntaxes through metricbeat jolokia module conf yml file and directly from a web browser based on what I get as a result of a list GET request and what I found in the doc (https://jolokia.org/reference/html/protocol.html#get-request).
Examples through a browser :

http://hostname:8778/jolokia/read/Catalina:class=javax.sql.DataSource,context=//akuiteo,host=localhost,type=DataSource ==> IndexOutOfBoundsException

http://hostname:8778/jolokia/read/Catalina:class=javax.sql.DataSource,context=/akuiteo,host=localhost,type=DataSource ==> InstanceNotFoundException

http://hostname:8778/jolokia/read/Catalina:class=javax.sql.DataSource,context=!/akuiteo,host=localhost,type=DataSource ==> InstanceNotFoundException

http://hostname:8778/jolokia/read/java.lang:name=PS MarkSweep,type=GarbageCollector ==> OK

If you need to escape the '/' as descibed in the documentation, which you did in your third example. But there you forgot the "name=" part (like in the other examples, too)

My bad, I did try with the name but didn't notice that the exception wasn't the same :
javax.management.RuntimeOperationsException,"error":"javax.management.RuntimeOperationsException : Exception invoking method loginTimeout"
I will investigate on authentication from there.

This exception results from an error on calling the JMX operation. Its nothing Jolokia can help much as it comes from Tomcat directly.

Sure, thanks for your help and work on Jolokia!


Add irodsinfo REST API endpoint to retrieve iRODS configuration

This would enable external tools such as cubi-tk to automatically fetch a valid iRODS configuration if SODAR API access was already established. Not relying on the user manually downloading irods_environment.json into the right location improves UX during onboarding of new users/machines.
related: bihealth/sodar-cli/issues/6

The API entry point should be in the irodsinfo app, not samplesheets. Other than that, this is a good idea. I'll write up a complete spec and we can include this in v0.14.

Spec

Add REST API endpoint to irodsinfo.views_api

Name = IrodsConfigRetrieveAPIView
Method = GET
Permission class = IsAuthenticated
Return user iRODS config as JSON

Format: {'irods_environment': {irods-env-goes-here}}

Refactor config generating in IrodsConfigView into a mixin

Both views should use the same mixin to generate the config

Let's ignore the certificate part for now, as it's not currently required by the client in our iRODS server implementation
Add permission and view tests for the API view
Add irodsinfo REST API docs to manual

Done by @gromdimon.


Hey @rpj09 I tried running the project locally, but couldn't install it. I installed all the dependencies and after running FRIDAY.py the output is none. What to do?

    Hey @rpj09 I tried running the project locally, but couldn't install it. I installed all the dependencies and after running FRIDAY.py the output is none. What to do?

Originally posted by @anuragh2002 in https://github.com/rpj09/FRIDAY-virtual-assistant/issues/2#issuecomment-1272370023

Please assign me the issue @rpj09


Save Geocoded Addresses in Access

Google Geocoder limit is 2,500 requests per IP per day for server-side requests.  Currently with 1,200 sites adding future counties will create issues geocoding
Issue Fix:

Use excel geocoder from http://ramblings.mcpher.com/Home/excelquirks/snippets/steps/geocode to geocode existing addresses.
Store Lat/Lon to separate table in MS Access
Create query to pull addresses without coordinates -> run through excel -> add coordinates to Access
Use Locations access query to pull coordinates.

Table is built.
Need to build query or excel sheet to compare new v old to add new geocoded addresses and remove deleted sites.

Query built to return empty geocodes or sites with mismatch.  Locations needing Geocoding may be added to 'tblAddresses', then run frmGGCMain to geocode.  Add completed geocodes to 'ohana-geocodes' table.


MAINT: Use a doubly linked list for contract chain.

Instead of requiring the roll finder to juggle the indices into the ordered
contracts, use a doubly linked list where the nodes element is the contract
with members pointing to the previous and next contracts in the chain.
Besides improving legibility in the roll finder code, this change is on the path
to adding a predicate to exclude contracts from the chain, e.g. contracts in ED
which are not in the roll schedule.

This change could be pushed further by having methods such as get_contract_center return the ContractNode instead of the sid.

Coverage decreased (-0.1%) to 87.074% when pulling 2940feb2f3352c3f8fa56e4402a4ceb1709d7009 on use-linked-list-for-contracts into 74df429057c02572f4a758a668d967b7e81a4f75 on master.

Coverage decreased (-0.09%) to 87.129% when pulling c57e4c85c2dd6500f3c31d643de7354fef541843 on use-linked-list-for-contracts into 74df429057c02572f4a758a668d967b7e81a4f75 on master.

Coverage decreased (-0.5%) to 86.749% when pulling d6bcf313053e8cd9c1ba55eec84838ee4c43d67e on use-linked-list-for-contracts into 74df429057c02572f4a758a668d967b7e81a4f75 on master.

Coverage decreased (-0.1%) to 87.074% when pulling d14739798392af9a87890f8342e3524a295bff8f on use-linked-list-for-contracts into 74df429057c02572f4a758a668d967b7e81a4f75 on master.


Add work around for SIP restrictions on macOS

This requires https://github.com/JuliaPackaging/Yggdrasil/pull/4987.  I verified that without that PR, the new test fails, but with the PR they're successful.
CC @ericphanson @GunnarFarneback @staticfloat @mortenpi.
Fix #40.

Awesome!!

@ericphanson @GunnarFarneback @mortenpi After https://github.com/JuliaPackaging/Yggdrasil/pull/4987 makes it to the registry (in ~20 minutes or so), can you please try out this PR on your macOS systems?  I successfully ran the test on macOS and Linux (of course on Linux it was working also before this PR, but good to test anyway).

At least tests are passing on all platforms on CI :slightly_smiling_face:

Codecov is a required CI job, but it's failing because code coverage is apparently broken on the @static line.....

I removed Codecov from required. Is there an open Julia issue to track the issue of @static not being having correct code coverage reported?

This is awesome, the Documenter macOS tests pass as well with this!

Confirmed that pointing https://github.com/GunnarFarneback/LocalRegistry.jl/pull/38 to this branch passes all tests.

Is there an open Julia issue to track the issue of @static not being having correct code coverage reported?

I don't know, but I wonder if it's a problem with nested @static?  Because the outer one is detected correctly, and this one is definitely stepped over, otherwise the new test wouldn't work on macOS.


fix(typings): allow bufferCreationInterval null for bufferTime

Description:
The tests for bufferTime verifies that it is possible to set
bufferCreationInterval to null, but the typings did not allow it
when strictNullChecks were set to true.
Related issue (if exists):
Fixes #3728

Coverage decreased (-0.2%) to 96.691% when pulling 0f4dafc681352e90b7881df3f2aa08f190456b2a on mattiash:buffertime-typings into c05a945ddb23a8f08dd3963a80323d3a2c0efff6 on ReactiveX:master.

My two cents: Is this consistent with other parts of the API? Are there other places where null is used rather than undefined? I can't recall an API method to which null is passed to indicate that a parameter is not specified in a call; I can only recall those to which undefined is passed.
For example, undefined is used to to subscribe. This is okay:
of(1).subscribe(undefined);

Whereas this is not:
of(1).subscribe(null); // [ts] Argument of type 'null' is not assignable to parameter of type '((value: number) => void) | undefined'.

For consistency, I think is should be undefined, rather than null. Or all of the typings should be changed to accept either null or undefined for parameters that are to be ignored.

The tests pass in null for bufferCreationInterval: https://github.com/ReactiveX/rxjs/blob/master/spec/operators/bufferTime-spec.ts#L24
The code sets bufferCreationInterval to null if the parameter is not included:
https://github.com/ReactiveX/rxjs/blob/master/src/internal/operators/bufferTime.ts#L67
and it later checks if it is null:
https://github.com/ReactiveX/rxjs/blob/master/src/internal/operators/bufferTime.ts#L129
Changing it to undefined is a much bigger change, but I have nothing against it.

Changing it to undefined is a much bigger change ...

Yeah, you are right. It would be a bigger change and smaller changes are always more appealing.
Regarding the check against null, the check uses == which will return true for either undefined or null. That's the mechanism used throughout the Angular codebase, IIRC.
Anyway, others can decide, I just thought I'd point out the consistency issue.
Thanks for the PR, BTW. :+1:

Thanks, @mattiash!


data_kidney_KPMP_VISIUM

@Jieran-S
Yes, I already implemented it and tested it.

Awesome!! I couldn't find the folder for your data at the main branch. Could you point me to the folder for your results? Thank you!

/home/jovyan/scratch/SpaceHack2/data/KPMP_KIDNEY
download script is
/home/jovyan/scratch/SpaceHack2/data/KPMP_KIDNEY/downLoad_KPMP_data.ipynb
test scrip is
'scratch/SpaceHack2/test_scripts/test_data_wo_download_KPMP.sh'

Yes! Thank you! I might do some modifications so we can loop them on github as well --- Will discuss this with Naveed and keep you updated! Thanks for implementing them :))


webhooks: Ignore unknown events from updown.io.

The webhook now logs a message instead because adding new event types unrecognized by it would be expected.
Fixes #24721.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Hmm, this works but I think this might be the wrong fix. Given this description, I think we intend to be raising this exception:
class UnsupportedWebhookEventTypeError(WebhookError):
    """Intended as an exception for event formats that we know the
    third-party service generates but which Zulip doesn't support /
    generate a message for.

    Exceptions where we cannot parse the event type, possibly because
    the event isn't actually from the service in question, should
    raise AnomalousWebhookPayloadError.
    """

So I think instead we want this exception to just have a 200 status code so that the third-party gets a success from Zulip, even though we're flagging it as a thing to track in Sentry?

@PIG208 Should I pick this one up and complete it?

@sbansal1999 go for it!

Sorry for the delay, I have updated the PR!

Backported to 7.x as ec8a284ad5997b7f6ac86493085d9f5f2eff0792.


Converted Search to TypeScript

Changes

Converted code to TypeScript
Updated related code

Checklist

[x] have tested my change
[x] updated relevant documentation

You're very welcome! 😁
It is a great feature indeed, making clever use of grep.


HOME: show info button on non-active currencies.

"Info" should go to the link to the currency page

https://zpl.io/aMvOZAz

Duplicate of #https://github.com/horizontalsystems/bank-wallet-android-app/issues/182


Customize pools min_size

Let user decide what min_size a pool should have.
kuberbd:
  name: "kuberbd"
  size: "2"
  min_size: "1"
  pg_num: "64"
  pgp_num: "64"
  rule_name: "replicated_rack"
  type: 1
  erasure_profile: ""
  expected_num_objects: ""
  application: "rbd"

Change is for any kind of pool
openstack_pools
cephfs_data, cephfs_metadata
client.pools(clients.yml) - like kuberbd example

Fixed conflicts, please check

@radut please, squash your commits and rebase on master.
Thanks

squashed + rebased

LGTM


Normalize and expose data for draggable and droppable nodes

Yo!
Thanks for the speedy work, it should allow somewhat more complex interactions to happen, can't wait, but I think it's important to keep one thing in mind and remember why I/others have even asked about this: we wanna freely make draggables interact with droppables regardless of what context they're in.. Sorry if what I'm about to write seems overly detailed but the nuances really matter:
As it stands right now, data in these hooks or not, we're still under the same context, meaning, if we were to have a set of draggables interact with a droppable not in the same context, well, that can't happen. The current model implies that each context handles its drag events (dragEnd, dragStart, etc.), collisions, sensors individually. With the new model, the system would allow any draggable to be dropped into any droppable, then, it's up to some callbacks (be them just the accepts key or something more advanced) to decide if a droppable can indeed work with a draggable.
I just can't find a good reason not to operate in a manner where, as you pointed out in: https://github.com/clauderic/dnd-kit/issues/139#issuecomment-814178995 , every droppable decides how the collisions happen (better yet, let a central figure handle it, but I guess that's more advanced), you don't have coupling between draggables/droppables besides the newly introduced accepts argument. If we approach things this way, then there are basically no limits to what you can build.
So, then I thought "well, I really appreciate the react way of doing things, wrapping components and so is elegant" and that's nice, but take a look at the model of the createSlice method from Redux, which basically lets you create a small portion of data that you care about within the store - with the store being unique, declared at the top of your application. This means that all data flow is visible and you can, through useSelector, declare what you care about.
This flux-like model gave me inspiration - why are we locking ourselves like this? Assuming we can create a top-level, unique context, our codebase would end up looking something like this:
const Droppable = () => {
  const { attributes, listeners, setNodeRef, transform } = useDraggable({
    id: 'uniqueId',
    types: ['type1', 'type2'],
    data: {},
  });
};

const Draggable = () => {
  const { setNodeRef } = useDroppable({
    id: 'uniqueId',
    accepts: {
      type1: {
        collision: customAlgo,
        sensors: {}
      },
      type2: {
        collision: customAlgo2,
      }
    },
  });
};

And if we were to have a Redux Dev Tools-like thingie, whenever we'd have a drag, drop and any other event, everyone could see it but not everyone should react to it. If we were to re-implemen the useSortable hook, we'd simply create a "slice" inside the context that the sortable can use (so, while everything is visible, it's still encapsulated - extremely important point) and it does its own thing while we can monitor/change it from the outside.
Anyways, sorry if this got a bit longer than expected, but this was my motivation and, I assume, based on what I saw being built with DnD libraries, many others'. There has to eventually be a way to see data flow like this without needing to hack something ugly in.

we wanna freely make draggables interact with droppables regardless of what context they're in

Having draggable and droppable elements from different DndContext providers interact with one another will never happen. There are dozens of reasons why that could never work.
I think you're approaching this problem with the wrong mental model. DndContext is your central store. The goal is to get to a place where it's easier to manage multiple types of drag and drop interactions under the same central DndContext provider, not to make it so that you can have elements under different DndContext providers interact with one another.

Aye! I've been reading the source for a week now and will hopefully understand it as soon as possible so I can start contributing and helping, but I think we're talking about the same thing and I just went a bit too "broad" and bringing in Redux as a philosophy. Perhaps my wording made it seem like the "free flow" of things wouldn't be in a central store, which you already have.
Perhaps a better wording would be "operating within the same context but being able to discern between interactions", which seems to be the goal here.
An example of how having two contexts mess things up is when you have a SortableContext inside a DnDContext, without the SortableContext, a draggable is able to see an actual sortable item as a draggable as well, so, my thinking was "well, that SortableContext could just insert itself into the bigger context but obviously manage its slice of state only and not care about others from the outside". Internally, each DnDContext manages itself in a sandbox way, it's just that, since they are all descendants of a bigger context, they could still be aware of each other's presence.
I've seen the useDndMonitor hook and I already have an idea on how to tie things together. Eagerly waiting on this update to see what I can build with it.


fix build with immutable src

The source directory is generally assumed to be immutable. Place generated files into OUT_DIR instead.

OUT_DIR: https://doc.rust-lang.org/cargo/reference/environment-variables.html
Bindgen suggesting to use it: https://rust-lang.github.io/rust-bindgen/tutorial-4.html
docs.rs build failing: https://docs.rs/crate/ipset/0.7.1/builds/1213512

I put binding.rs in the src folder for IDE code assistant.

Mhh ... I don't have a great solution :(
All things considered, giving this up and generating the file in OUT_DIR is probably best. rust-analyser would still be able to handle it, text-based autocomplete not so much.
Alternatively, going back to manually generating the file in src is not great, but still better than failing to build with read-only src.


Setting up

Hello, I believe this is not the place, but is their an example, which i could donwload and see how it works?
(And be able to run local)

@402Martin
This project is the source code of veaury, and also includes the development environment. The source code of veaury is in the src directory, and the debugging projects of the development environment are in the dev-project-vue3 and dev-project-react directories.
You can clone the project, install it, and debug it.
If you want to debug veaury source code in dev-project-react, you need to ensure the following configuration:

Uncomment the veaury in alias to develop the source code of veaury in the dev-project-react/config/webpack.config.js
Set configFile to true on line 447 in the dev-project-react/config/webpack.config.js
Uncomment the overrides attribute in the dev-project-react/babel.config.js

If you want to debug veaury source code in dev-project-vue3, you need to ensure the following configuration:

Uncomment the veaury in alias to develop the source code of veaury in the dev-project-vue3/vue.config.js
Uncomment the test attribute of the veaury/babel/ReactPreset in the dev-project-vue3/babel.config.js

Setup: Run the command npm run setup:yarn or npm run setup:npm in the root directory of the main project to install the main project and two subprojects
Develop: Run the commands npm run dev:vue and npm run dev:react in the root directory of the main project for development

@devilwjp, Thanks for your answer, I am currently migrating a project from vue to react and I wasn't able to make veaury work, Is there any guide (like a hello world) that includes set up?


Improve log performance when forking

See the way logs are done here: https://github.com/ethereumjs/testrpc/pull/179
i.e., instead of making n requests to the forked chain, make 1.

Issue moved to trufflesuite/ganache-core #145 via ZenHub


Bad replace in joinURL function

Currently you do path.replace(/^\//, '').replace('/\/$/', '')
the second replace has a string as the first argument that should be a regexp
i.e. it should be path.replace(/^\//, '').replace(/\/$/, '')
or just path.replace(/^\/|\/$/g, '') for both in one replace
By the way, glad I stumbled on this library, was working on something very similar, but this is far more elegant. Thank you

Currently you do path.replace(/^\//, '').replace('/\/$/', '')
the second replace has a string as the first argument that should be a regexp
i.e. it should be path.replace(/^\//, '').replace(/\/$/, '')
or just path.replace(/^\/|\/$/g, '') for both in one replace
By the way, glad I stumbled on this library, was working on something very similar, but this is far more elegant. Thank you

Fixed.
Thank you very much!

Thank you very much! Would you like to join this project?

Let me think about that.
Just want to say though, I replaced axios in a vue project using this library by creating a "shim", so I just import this library through my shim and haven't had to change the rest of the code yet. Everything works great, and I don't need to rush to change the axios calls to this all at once.
Before you ask, the "shim" is quite specific to my app, it's not like it could be used in a generic way.


Option to choose videos

All the videos are getting downloaded. There must be a feature to select a couple of videos

Thanks for a feature request.
Currently, the project is on a halt due to my college schedule. You can always contribute to the project.


I believe it should, however I'm looking at VFIO_IOMMU_DIRTY_PAGES and it's not clear.

For my own benefit here:
The problem is that if we don't clear, then the relevant pages (and it's the entire sg's, ever used) will always be dirty, most likely making the downtime include all those pages.

I'm looking at vfio_iommu_type1_dirty_pages and my understanding is that all currently pinned pages are considered dirty ( vfio_iova_dirty_bitmap) and that they remain dirty after vfio_iommu_type1_dirty_pages returns.

Closing since it's tightly coupled with https://github.com/nutanix/libvfio-user/issues/540.


document the WorkflowTemplate workflowMetadata feature

Checklist:

[ ] My organization is added to USERS.md.

If anyone knows the first version that supported this, I'd like to add that note... I can also do some digging myself.

Found it: https://github.com/argoproj/argo-workflows/releases/tag/v2.10.2


Improve add extensions quickpick when multiple Quarkus projects exist in current workspace

If there are two Quarkus projects in the workspace: quarkus-project-maven and quarkus-project-maven2, the quickpick after calling Quarkus: Add extension to current Quarkus Project, this quickpick appears:

This enhancement idea adds the project name alonsgide the path to pom.xml/build.gradle:

To get the project name, maybe we could have the microprofile/java/projectLabels command return not only the project uri, but the project name as well?

As mentioned in https://github.com/redhat-developer/vscode-quarkus/pull/204#issuecomment-589153609, currently we have the folder name being displayed instead than the project name


Laravel 11.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 11.x.
Before merging, you need to:

Checkout the l11-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:warning: The latest version of Laravel has a dependency for phpunit/phpunit of ^10.5. If you would like to also upgrade your tests, you may run the PHPUnit 10 Shift for free.

:warning: Shift detected GitHub Actions which run jobs using a version matrix. Shift attempted to update your configuration for Laravel 11. However, you should review these changes to ensure the desired combination of versions are built for your package.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/laravel-selfupdater.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "codedge/laravel-selfupdater": "dev-l11-compatibility",
    }
}

Finally, run: composer update


Snackbar decouple timeout and onTimeout

When using the snackbar component, and only using the timeout prop, without setting the onTimeout prop, we get the following error:
onTimeout is not a function

All we want is to hide the snackbar after x milliseconds. Timeout should not be coupled with onTimeout, and should by default hide the snackbar, without having to implicitly implement it. Especially when the docs state:
timeout - Amount of time in milliseconds after the Snackbar will be automatically(<--) hidden.

I think this is fixed already


Eliminate inference on cljs.analyzer when compiling cljs/cljc

This is a more thorough change than #150, eliminating the inference on cljs.analyzer entirely. It instead infers the fact that it's compiling ClojureScript by checking (:ns &env). While this feels hacky to me, it's widely used.

Blog post explaining the approach
Google Groups post
Prismatic Schema
Encore
More

This should address both #133 and #148.

Nice.  Yeah the analyzer hack is really a stretch.  I'm glad that there's a better solution.


🌐 Add Chinese translation for docs/zh/docs/deployment/https.md

as title

📝 Docs preview for commit a96292b488cbc317f660279a2ff3dd9efa778f11 at: https://649a08ee219954094e357f72--fastapi.netlify.app

It seems this was covered in another PR, so I'll now close this one, but thanks for the effort! :rocket: :cake:


Getting "unauthorized: incorrect username or password" when pulling image since latest update

Summary
I'm trying to pull the docker image from: "https://registry-1.docker.io/v2/salesforce/cli/manifests/latest-full" using "salesforce/cli:latest-full" and until the most recent update on September 27, 2024, it was working fine. However, I noticed that today since 4:30pm (EDT) i have been receiving unauthorized: incorrect username or password error.

[!TIP]
use sf doctor --create-issue to automatically fill the required information

Expected result
image should be pulled
Actual result
unauthorized error
Additional information

PASTE_VERSION_OUTPUT_HERE

This looks like it's an issue with your docker authentication rather than the salesforce cli. See https://forums.docker.com/t/unauthorized-incorrect-username-or-password/35677


Enhanced IPV6 handling in astats

Enhanced IPV6 handling in astats

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/734/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/812/
Test PASSed.


Suggestion: support to :for

Support method :for(start; test; incremental), for instance:
@pageWidth = 1200;
@pageSpace = 40;
@pageParts = @pageWidth / @pageSpace;
:for(@pageSpan = 0; @pageSpan < @pageParts; @pageSpan++) {
.span{@pageSpan}_block {
margin-left: @pageSpan * 10;
}
}
Will make .span0_block, .span1_block, ... .spanN_block.

I commented under your :if issue but here it is again...
LESS already has conditionals via guarded mixins. This stays as close as possible to the declarative nature of CSS and very similar to functional programming.  An example of this:
.mixin (@a) when (lightness(@a) >= 50%) {
  background-color: black;
}
.mixin (@a) when (lightness(@a) < 50%) {
  background-color: white;
}

Check out the docs at lesscss.org for more.

Yeah... I found the "when" method some seconds after, but the ":for" method is different. For instance: you can set how many .spanN you want with a variable.
:for(@i=0; @i<12; @i++) {
.span[@i] { width: expression(20px * @i); }
}

Very true but this really is not a very "CSSy". You could still do this but you would have to create every possible instance that is possible...
I doubt that @cloudhead would go for a :for or something like that.
The only thing I can think of would be to have a "until" or something like that:
.mixin (@a, @i) until (@i) {
.span[@i] { width: expression(20px * @i); }
}

Unfortunately this then brings up another problem is inline variables in LESS. As of right now (1.2.2) there is no way to have ".span[@i]". That is one thing that I would really like to have and have been trying to think of a elegant way to have inline vars.

hum... it's possible create expressions? for instance: '''expression(".span" + @i) { ... }''' or something like that?
about the for, I guess that is possible make a hack, for instance:
'''
.mixin(@a, @b) when(@a < @b) {
expression('.span' + @a) { width: @a * 20px; }
.mixin(@a + 1, @b);
}
.mixin(0, 12);
'''
So, only need a method to create rules with vars.

you can use recursion + guards to create a loop, not officially supported though.

also youd have to do nth-child(@var) probably

Just thought I'd give an example of what I think cloudhead means - to make sure I understand correctly:
.mixin(@i) when (@i < 5) {
    &:nth-child(@i) {
        // do stuff
    }
    @i = @i + 1;
    .mixin(@i);
}
.mixin(@i) when (@i > 5) {
    // fallback to "break" the loop and avoid errors for no-match
}

That about right?

Tried this out of curiosity in http://winless.org/online-less-compiler (didn't think the @i = @i + 1; line would work) and to get your actual desired result, you have to have something like:
.mixin(@i) when (@i < 5) {
    &:nth-child(@i) {
        color: red;
        // do stuff
    }
    .mixin(@i+1);  // recurse
}
.mixin(@i) {
    // fallback to "break" the loop and avoid errors for no-match
}

mix {
    .mixin(0);
}

Thing is, @i = @i + 1; isn't valid less, but even using something like @i: @i + 1 doesn't work (this last one for some reason probably makes the loop infinite, as the compiler says that the "Maximum call stack size exceeded"). Calling .mixin(@i+1) easily bypasses this.
Note: only tested this on the online winless compiler which presumably uses less 1.2.1

It works very well for this case. But the problem is a method to create rules with expressions. For instance:
.mixin(@i) when (@i < 5) {
    expression(".span" + @i) {
        width: @i * 20px;
    }
    .mixin(@i+1);  // recurse
}
.mixin(@i) {
    // fallback to "break" the loop and avoid errors for no-match
}

mix {
    .mixin(0);
}

I'm not really sure why this would be more efficient than what's already available to us, or encourage symantic and readable markup?
@david-proweb: Have you taken a look at the LESS grid systems out there? Have a look at http://semantic.gs/ which doesn't have a ".span1", ".span3", etc. style. Instead, it uses parametric mixins. There are a couple advantages to this:

There is no limit to the amount of columns on a grid
You dont end up writing non-semantic markup into your html like class="span5 push1 row2"

@mindfullsilence Yeah, it's a good project. But I use bootstrap and I like of the method of .spanN rules. It's useful to use a compiled CSS, instead of run LESS everytime.

Closed to make it more specific (for exists, but need some more features): take a look about support interpolation and scape on rules.


Is there a straightforward way to build the proto files for csharp (or any other particular language)?

Dear envoy team
I'm currently trying to build the *.proto files in the API folder to be used to create configs for the envoy proxy. Sadly, it is non-trivial to build the proto files with "buf" or by any other means.
I tried to use the dotnet grpc tools, but essentially searching and including all of the dependencies has proven to be error prone work. Next I tried to use "buf" since I saw that you use this as well. Now I'm hitting the following error:
% buf generate                                                                                ~/Development/github/mse-wirepact-k8s-operator/Envoy
envoyproxy\api\contrib\envoy\extensions\filters\network\kafka_mesh\v3alpha\kafka_mesh.proto:5:8:xds/annotations/v3/status.proto: does not exist
envoyproxy\api\envoy\config\core\v3\protocol.proto:11:8:xds/annotations/v3/status.proto: does not exist
envoyproxy\api\envoy\config\route\v3\route_components.proto:20:8:xds/annotations/v3/status.proto: does not exist
envoyproxy\api\envoy\extensions\bootstrap\internal_listener\v3\internal_listener.proto:5:8:xds/annotations/v3/status.proto: does not exist
envoyproxy\api\envoy\extensions\common\async_files\v3\async_file_manager.proto:5:8:xds/annotations/v3/status.proto: does not exist
envoyproxy\api\envoy\extensions\filters\http\composite\v3\composite.proto:7:8:xds/annotations/v3/status.proto: does not exist
envoyproxy\api\envoy\extensions\filters\http\ext_proc\v3\processing_mode.proto:5:8:xds/annotations/v3/status.proto: does not exist
envoyproxy\api\envoy\extensions\filters\network\meta_protocol_proxy\matcher\v3\matcher.proto:5:8:xds/annotations/v3/status.proto: does not exist
envoyproxy\api\envoy\extensions\filters\network\meta_protocol_proxy\v3\route.proto:5:8:xds/annotations/v3/status.proto: does not exist

I try to generate c-sharp types from the envoy proto files (in particular the bootstrap.proto part) to generate "static" configs for an envoy instance.
The goal is to create a bootstrap object in code, then serialize it to json/yaml and give it to envoy.
I'd appreciate any nudge in the direction of solving my building issues ;-)
Thank you

without bazel would be hard work as that is what the protos and their deps are collected with. Even with bazel its not trivial.
the bazel target you are looking for is @envoy_api//:v3_protos
there is an example of generating documentation from the protos in tools/protodoc/protodoc.bzl. This uses "aspects" and a protobuf plugin so is not the easiest code to follow
you probs want to look at rules_proto
i have ~successfully built json schemas from the protos in the past, but it never got past r&d and was some time ago (#15100 )

Hey @phlax
Thanks for the reply!
I guess I'll have to let it be. But thank you for the idea! It would've been a cool feature for my operator to generate the envoy config (that gets injected as a sidecar) via statically typed classes, rather than a big string with interpolated ports :-)


add @ng-icons/core

Thanks for this very useful site! Just adding the ng-icons library to the list.

Thank you @ashley-hunter 🙌!

I merged it! But it looks like it was also added before 😂 So probably will have to revert.


Ros2

Feel free to comment on anything that should change or could be improved etc. :)

Several comments. I think the first big one to address is keeping the nmea_msgs in a separate package and how to minimize the changes to the directory layout.

Thanks for all the feedback, will take a look and try to fix this coming week. About the directory structure, I tried to update it to be more of a ROS2 style, but you are right probably makes more sense to keep it aligned to the ROS1 package to a higher degree, to avoid confusion.

@evenator Regarding my question above, just a clarification if you want me to make it more like the original ROS1 package before the merge ? also gentle nudge to see if my other changes looks ok.

I've responded to comments and finally got a ROS2 environment set up in docker to test this out. It looks like there's a missing dependency.

Ok changed the structure to better adhere to the Ros1. I think we need an init.py file in the scripts folder though, which is a bit unfortunate, I don't think its possible to use them for the entrypoints console scripts in setup.py file if it's not there. Fixed som formatting as well. Let me know if there is anything else left to do before merge :)

This is looking good. I need to go through and test it a bit, but I think it's almost ready to merge. In the future, I'd like to refactor the ROS1 package into a more ROS2-compatible style by replacing all of the large node scripts with thin executables, like what entrypoints generates.

@evenator is there anything else I can do to make you comfortable merging this?

There are still a few issues with dependencies and with the launch file, but I'll go ahead and merge this now,  since it's not going into master, so there are no regressions.


Fix lints.

Fixed php lint errors.

@swissspidy There are issues with our linters in travis that these were merged.


How many classifications does segmatch extract?

Hi, I find that in the random_forest_eigen_25trees.xml file it only has 2 classifications, but when I read the wiki(https://github.com/ethz-asl/segmatch/wiki) it seems has at least 4 classifications, such as tree,wall,trailer,car. I don't know why？
And I train a classifier with 5 classifications, and change the predict_prob() to predict() because I think predict_prob() only supports binary classifications,  when I run the kitti_localization.launch and change  random_forest_eigen_25trees.xml to my own .xml file, I still get the error "OpenCV Error: Bad argument (the input sample must be 1d floating-point vector with the same number of elements as the total number of variables used for training) in predict, file /home/nv/opencv-2.4.9/modules/ml/src/tree.cpp, line 3622
terminate called after throwing an instance of 'cv::Exception'
what():  /home/nv/opencv-2.4.9/modules/ml/src/tree.cpp:3622: error: (-5) the input sample must be 1d floating-point vector with the same number of elements as the total number of variables used for training in function predict"
How to use my own classification file with segmatch? Thanks a lot!

ＨＩ， @YuxiHuang1 ,thank you for your work, where can we get the train data from?

Hi @YuxiHuang1 you you for your interest in our library! We used predict_prob() to predict the probability of two segments belonging to the same object or environment part and use this information to localize. I believe that you understood that we were doing semantic classification - we are not but that might come in a future version..
If your goal is to do localization, I suggest you look into the eigenvalue based descriptor. The random forest will be deprecated in the next version of segmatch as we introduce another descriptor. Hope that helps!

Hi,@githublyk,I used my own pointcloud data to train the random forest tree.

Thanks @rdube ,I think the random forest method can be replaced by deep learning，looking forward to your next version!

Hi @YuxiHuang1 you you for your interest in our library! We used predict_prob() to predict the probability of two segments belonging to the same object or environment part and use this information to localize. I believe that you understood that we were doing semantic classification - we are not but that might come in a future version..
If your goal is to do localization, I suggest you look into the eigenvalue based descriptor. The random forest will be deprecated in the next version of segmatch as we introduce another descriptor. Hope that helps!

hi，could you plz share the segmatch code which have randon forest? i want to do comparsion in my paper ,thanks! :)


tests: rpi_pico tests fails on twister with: No rule to make target 'bootloader/boot_stage2.S

Describe the bug
rpi_pico tests don't seem to be building correctly and fail with
Scanning dependencies of target modules__hal_rpi_pico
gmake[2]: *** No rule to make target 'bootloader/boot_stage2.S', needed by 'modules/hal_rpi_pico/CMakeFiles/modules__hal_rpi_pico.dir/__/__/bootloader/boot_stage2.S.obj'.  Stop.
gmake[1]: *** [CMakeFiles/Makefile2:3343: modules/hal_rpi_pico/CMakeFiles/modules__hal_rpi_pico.dir/all] Error 2

To Reproduce
./scripts/twister -p rpi_pico -v -n -M -T samples/basic/blinky/
Test fails to build with the error above.
Expected behavior
Test passes
Impact
Blocks CI build whenever a test or sample is built against rpi_pico
Logs and console output
Scanning dependencies of target modules__hal_rpi_pico
gmake[2]: *** No rule to make target 'bootloader/boot_stage2.S', needed by 'modules/hal_rpi_pico/CMakeFiles/modules__hal_rpi_pico.dir/__/__/bootloader/boot_stage2.S.obj'.  Stop.
gmake[1]: *** [CMakeFiles/Makefile2:3343: modules/hal_rpi_pico/CMakeFiles/modules__hal_rpi_pico.dir/all] Error 2
gmake: *** [Makefile:91: all] Error 2

Environment (please complete the following information):

OS: Linux
Toolchain: Zephyr SDK
Commit SHA: d611284271

@yonsch you mentioned second_stage_bootloader on discord and that twister should not build it, I ran this while troubleshooting the assert problem and after fixing that (with an #undef of the macro) it hits this issue. I think twister is still trying to build it.

Seems to only happen with CMake using Makefiles, so for examples:
west build -p -b rpi_pico samples/basic/blinky -G"Unix Makefiles"
fails, but
west build -p -b rpi_pico samples/basic/blinky
works. The CI twister setup uses ninja by default so it should work there but both setup should work.


Change Edit XML Behavior to Open Already Opened XML

If no XML already opened, prompt for XML to edit on form load.

Addressed in v1.6


Attempting to call getTask throws exception

Every time I try to call getTask with the task id returned from upload, I see the following error:

However, the uploading does work. I just can't seem to use getTask. I did notice that if I put in a long timeout (1500ms) between when the promise from upload resolves and calling getTask, it does not throw that error. But I get null back.
getTasks seems to work, but it always returns an empty array.

It fixed by https://github.com/mybigday/react-native-s3/commit/5a045693a245726712776471bf7739d2f389256e, I'll publish v0.0.4. :)
Also, the getTasks is need type argument (see this), it should be work.

Cool, thanks!
Yeah I was passing upload to getTasks


Weather command with a time value of zero

The command supports a value of zero.
A value of zero mimics the effect of sleeping in a bed.

This allows /weather clear 0, which mimics the effects of sleeping in a bed.
c.f. bugs.mojang.com

Specifies the time in seconds for the specified weather to last. Must be between 0 and 1,000,000 (inclusive, without the commas).
c.f. Gamepedia

:tada: This issue has been resolved in version 2.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: