How can we help?
I would like someone else to build the manifest.
Please read and ensure the following

[X] The installer meets the above requirements

Please provide the following information
Download Page Url: http://www.w1hkj.com
Publisher: W1HKJ Software
Package Name: Flrig
Description: Transceiver control program designed to be used either stand alone or as an adjunct to Fldigi.
Package Version: 2.0.02
Installer URL: http://www.w1hkj.com/files/flrig/flrig-2.0.02_setup.exe

@AsciiWolf
Please send the package you wanted to my email
<EMAIL_ADDRESS>
I would handle it later

I've also noticed that this software has published new version, you could add a issue to update it later.


Upsert doesn't work in case backend endpoint is changed

After an API is created (via "sls deploy"), if we change apidefs.[].backend.http.baseUrl in configuration, WSO2 API configuration isn't changed (the previous version is kept) when doing "sls deploy" for upserting.
Steps to reproduce

Deploy an API via sls deploy
Check the endpoint url on WSO2 console
Change baseUrl in plugin configuration to something else
Deploy API again via sls deploy
Change endpoint url on WSO2 console (should be the same as in the first deploy)

The workaround that I use is to delete the API and re-create again with "sls deploy"

@flaviostutz - can you test if it still persists with the latest version? It helps to prioritise this issue. Thanks!


[plugin:unplugin-vue2-script-setup] Unexpected token, expected "," (1:16)

Hey i have the weirdest issue:
here's the component:
<script setup name="OnThisVoyage">
import { useOrganisationIdProp } from '../../composables/useRouteHelpers.js';
import { provideBaseCardState } from '../../features/useBaseCardState.js';
import Service from '../../classes/Service.js';

import CargoRow from './CargoRow.vue';

const props = defineProps({
  ...useOrganisationIdProp(),
  shipmentId: {
    type: [Number, String],
    required: true,
  },
});

const {
  setStateLoading,
  setStateIdle,
  setStateError,
  setStateNoData,
} = provideBaseCardState();

const cargoes = ref([]);

const fetchCargoes = () => {
  Service.organisation(props.organisationId)
    .shipment(props.shipmentId)
    .voyage()
    .index()
    .onStart(setStateLoading)
    .onSuccess(({ data }) => {
      cargoes.value = data.filter(
        (cargo) => cargo.shipping_id !== props.shipmentId
      );
      setStateIdle();
    })
    .onNoData(setStateNoData)
    .onError(setStateError);
};

onMounted(() => {
  fetchCargoes();
});
</script>

<template>
  <BaseCardWithState
    title="On this voyage"
    no-data-title="No other cargoes on this voyage."
    @retry="fetchCargoes"
  >
    <CargoRow
      v-for="{ cargo_key, shipping_id, commodity } in cargoes"
      :key="cargo_key"
      :shipment-id="shipping_id"
      :commodity="commodity"
      :organisation-id="organisationId"
    />
  </BaseCardWithState>
</template>

When trying to render this component vite throws an error:
[plugin:unplugin-vue2-script-setup] Unexpected token, expected "," (1:16)
/Users/mykolas/Documents/Projects/myg2-frontend/src/components/OrganisationShipments/OnThisVoyage.vue:1:16
1  |  <script setup name="OnThisVoyage">
   |                  ^
2  |  import { useOrganisationIdProp } from '../../composables/useRouteHelpers.js';
3  |  import { provideBaseCardState } from '../../features/useBaseCardState.js';

Does anything come to mind? I'm trying to disable things on by one and see what causes the issue

ok so the issue was  v-for="{ cargo_key, shipping_id, commodity } in cargoes"
changed it to ` v-for="item in cargoes"
and it works now....

Noticed something weird, something is broken the way this plugin works with v-for
you can't have the word link anywher in the variable,
e.g. link prefixLink linkAppend
Threse will all break with the same issue as mentioned above.
Also if (not fully confirmed) if the name of the item starts with the same text v-for="booking in bookings" although it migh just be that i've missed some link in another variable.
<div v-for="(link, index) in links" :key="index" class="py-4 space-y-2">

I could most likely try and do a pr if you point me in the right direction.

Thanks, I will have a look on this

Hello. Im getting the same token error when using
unplugin-vue2-script-setup 0.6.4 version
if i remove v-for or v-if from template works even as pug.
if template is html works


Strip signature from server and client side metadata

For now, pybind11 is our primary binding vehicle, and it doesn't populate signatures in a way that we can export cleanly. They are just the first line of the DocString, and that's it. Maybe Nanobind does (or will do) a better job of metadata transport.
On the client side, parsing these signatures back into signature objects is hard to do without creating vulnerabilities or corner cases. I think we would need to teach the server to export a JSONSchema-like thing rather than expecting the client side to parse a signature string. Inspect does not currently work with annotations in __signature__ or __text_signature__.
In both cases, we don't really need signatures for anything and are relying on a big-ball-of-text __doc__  to transport DocStrings anyhow. May as well tuck signatures back into the top of this.
I would love to revisit this when the binding side (pybind11/nanobind) and the client side (inspect) firm up a little.

As we discussed offline, I think it's still useful to keep basic signature inspection in place, so that anyone using a binding framework that can produce valid signatures (e.g. pure python or nanobind) is able to pass those signatures along to the client.  This also makes the tuber server a bit more future-proof, when/if pybind11 or boost python are able to generate valid signatures.
I agree that the signature compilation hack I put in for pybind11 signatures is fragile and should be removed, though.


Fix startup file pathnames in CMake version of the sample configurations

The cmake install currently does not create an extra /apps/ subdirectory to install the files.  This makes it incompatible with the current platform config file that specifies the startup file to be:
{{{/cf/apps/cfe_es_startup.scr}}}
With the current installation script it should simply be:
{{{/cf/cfe_es_startup.scr}}}
This change only affects the sample configurations, nothing in the runtime code.

Imported from trac issue 60.  Created by jphickey on 2015-05-28T15:45:46, last modified: 2019-03-05T14:57:55

Trac comment by jphickey on 2015-05-29 11:19:09:
Pushed commit [changeset:5cea8b6] for review
This is a trivial fix to change the pathnames, and only affects the cmake sample configuration files, no runtime code.

Trac comment by glimes on 2015-07-27 12:40:22:
Approve.
Implies we need to check that this change flows into the
test configurations in cfs_test as well.

Trac comment by glimes on 2015-09-08 12:57:44:
... somehow was not closed after the 07-28 integration was complete.

Trac comment by glimes on 2015-07-30 15:17:47:
[changeset:5cea8b6 trac-60-fix-startup-file-paths] is included in [changeset:6ea42ba ic-2015-07-28]

Trac comment by jhageman on 2019-03-05 14:57:55:
Milestone renamed

Trac comment by glimes on 2016-02-25 10:17:32:
these will be fixed in CFE 6.5


NV Upgrade Units JP

NV Upgrade Units in the "My Units" section on JP can't be upgraded to 7* and can't have their STMR added.

Still an issue.
Kinda needs to be fixed soon as GL gets their first NV (Cloud) and NV upgrade (Tifa) on Aug 20th.

this is an odd one. i think the parser is grabbing the unit id.
this is what is currently being parsed to item data.
{"id":"407004700", "name":"King Rain's Armor", "jpname":"レイン王の鎧", "type":"heavyArmor", "hp%":10, "atk":28, "def":45, "lbDamage":30, "tmrUnit":"700000089", "access":["TMR-7*"], "icon":"item_20343.png", "sortId":1350},
looking at the unit's file, this is the data that should line up with that file. the only difference is the tmr unit id.
{"id":"407004700", "name":"King Rain's Armor", "jpname":"レイン王の鎧", "type":"heavyArmor", "hp%":10, "atk":28, "def":45, "lbDamage":30, "tmrUnit":"100023205", "access":["TMR-7*"], "icon":"item_20343.png", "sortId":1350},
reason being, is that there is only one instance of Aldore King Rain in the units file.
"100023205": {
		"name":"Aldore King Rain",
		"jpname":"アルドール王レイン",
		"id":"100023205",

going through aenigma's raw data, i can find 4 separate entries for Aldore King Rain/アルドール王レイン.
"100023205": {
        "rarity_min": 5,
        "rarity_max": 7,
        "base_id": null,
        "name": "アルドール王レイン",

    "700000079": {
        "rarity_min": 7,
        "rarity_max": 7,
        "base_id": null,
        "name": "アルドール王レイン",

    "700000084": {
        "rarity_min": 7,
        "rarity_max": 7,
        "base_id": null,
        "name": "アルドール王レイン",

    "700000089": {
        "rarity_min": 7,
        "rarity_max": 7,
        "base_id": null,
        "name": "アルドール王レイン",

one files is grabbing the id from the first instance, and the other from the last instance, causing them to not line up with each other.


Fix a couple places where utf-8 was not specified explicitly for Path.read_text/write_text

Without this it would use locale.getpreferredencoding(), which at least in some cases is cp1252 in Windows.
Also includes a fix for #1514 to make cci error info show the full traceback by default.
Critical Changes
Changes

The cci error info command now defaults to showing the entire traceback when it is more than 30 lines.

Issues Closed

Fixed a bug where cci error gist could throw a UnicodeDecodeError (fixes #1977)

@jstvz I found https://pypi.org/project/flake8-file-encoding/ but it only checks open, not the Path methods.

Windows is such a trouble-maker!!


support contributor deployments

add ability to deploy as a contributor.

note we do need role assignments; so need to investigate graceful solutions

worth testing if config0 (one that uses pool allocation mode as BatchService) is adequate.


[AutoPR resources/resource-manager] Update Azure Governance Policy api-version to 2018-03-01

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3032

(message created by the CI based on PR content)

Codecov Report

Merging #2553 into restapi_auto_resources/resource-manager will decrease coverage by <.01%.
The diff coverage is 48.48%.

@@                             Coverage Diff                             @@
##           restapi_auto_resources/resource-manager    #2553      +/-   ##
===========================================================================
- Coverage                                    55.12%   55.11%   -0.01%
===========================================================================
  Files                                         6369     6369
  Lines                                       141229   141238       +9
===========================================================================
  Hits                                         77849    77849
- Misses                                       63380    63389       +9

Impacted Files
Coverage Δ

...source/azure/mgmt/resource/policy/policy_client.py
61.44% <0%> (-7.48%)
:arrow_down:

...mt/resource/features/v2015_12_01/feature_client.py
35.71% <66.66%> (ø)
:arrow_up:

.../mgmt/resource/policy/v2016_12_01/policy_client.py
38.7% <66.66%> (ø)
:arrow_up:

...resource/managedapplications/application_client.py
36.66% <66.66%> (ø)
:arrow_up:

...e/subscriptions/v2016_06_01/subscription_client.py
40.74% <66.66%> (ø)
:arrow_up:

...esources/v2018_02_01/resource_management_client.py
39.47% <66.66%> (ø)
:arrow_up:

...source/links/v2016_09_01/management_link_client.py
35.71% <66.66%> (ø)
:arrow_up:

...esources/v2017_05_10/resource_management_client.py
39.47% <66.66%> (ø)
:arrow_up:

...source/locks/v2016_09_01/management_lock_client.py
35.71% <66.66%> (ø)
: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 eb5207e...7de7624. Read the comment docs.

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2570


[ Feature Request ] Add Quick Look

Description of the new feature / enhancement
Preview a file by pressing space like in macOS
Scenario when this would be used?

When uploading images or files that you want to check if it's the correct image /file to upload
When viewing a gallery of non-image files, such PDF and PSD, and you want to quickly preview it.

Supporting information
Reference Open-Source applications that enables this one on Windows

https://github.com/QL-Win/QuickLook
https://github.com/shibayan/WinQuickLook

Duplicate of #80

Thanks for your suggestion! This issue is a duplicate.
I'm linking this issue to #80 to centralize the discussion thread, please go to that thread for discussion.
Thanks for your help to make PowerToys a better piece of software!


Implement TBT APIs

The proxy is currently missing the following APIs in the spec and need to be implemented:

AlertManeuver (and response)
UpdateTurnList (and response)
ShowConstantTBT (and response)

We need to reopen this issue as none of the tbt RPC responses have been handled in SdlProxyBase.java.
An example of handling the RPC responses can be found here as seen in SdlProxyBase.java DialNumber and SendLocation here:
https://github.com/smartdevicelink/sdl_android/commit/d8a8088e0fb676e8b60c0ca4d38d35eb257c0882


Is there a way to detect whether a epub file is encrypted?

Some users are trying to drop encrypted book to our app. Readium-js console.log the error.
We want to warn user do not use encrypted book, how to do it?
Is there a way to detect whether a epub file is encrypted ?
Thanks

You could look in the encryption.xml where the type of encryption is required to be listed.   But you need to ignore the obfuscation "encryption" setting, i.e. Readium will automatically decrypt obfuscated fonts.  But if you find other encryption info you could conclude that at least part of the book is encrypted.
For example, here is the encryption info for the two types of obfuscation (Adobe's original, and the revised method adoption in EPUB 2.x)
  <EncryptedData xmlns="http://www.w3.org/2001/04/xmlenc#">
     <EncryptionMethod Algorithm="http://ns.adobe.com/pdf/enc#RC"/>
     <CipherData>
        <CipherReference URI="fonts/sandome.adb.ttf"/>
    </CipherData>
</EncryptedData>
<EncryptedData xmlns="http://www.w3.org/2001/04/xmlenc#">
    <EncryptionMethod Algorithm="http://www.idpf.org/2008/embedding"/>
    <CipherData>
        <CipherReference URI="fonts/sandome.obf.ttf"/>
    </CipherData>
</EncryptedData>

Note that there will be an  entry for each encrypted resource, so there could be a lot of entries.  Offhand, I don't know an easier method.
@danielweck  Comment?

You probably need to add this functionality, but there are existing build blocks:
PublicationFetcher has an internal / private _encryptionHandler member:
https://github.com/readium/readium-js/blob/develop/js/epub-fetch/publication_fetcher.js#L31
...and the EncryptionHandler.isEncryptionSpecified() function returns true when the encryption.xml maps at least one resource with an algorithm.
https://github.com/readium/readium-js/blob/develop/js/epub-fetch/encryption_handler.js#L88
https://github.com/readium/readium-js/blob/develop/js/epub-fetch/encryption_handler.js#L159
In which case, in order to determine whether there are encrypted resources other than obfuscated fonts, you could iterate through manifest items and use getEncryptionMethodForRelativePath() to check for undefined return value:
https://github.com/readium/readium-js/blob/develop/js/epub-fetch/encryption_handler.js#L93
...which indicates that a resource is encrypted with algorithms other than IDPF / Adobe:
https://github.com/readium/readium-js/blob/develop/js/epub-fetch/encryption_handler.js#L19
I hope this helps. Dan

This issue can be closed, right?


FR: AI-based completion of sentences?

Hi! I am actually a bit surprised that I am apparently the first to request this, given the hype around Generative AI recently 😋
So as opposed to the on-demand suggestions offered by plugins like Text Generator, AI-generated text could also be suggested automatically via the EditorSuggester. From a few weeks of trying out various AI plugins in the code editor, I found them to be actually more useful than manually prompting GPT.
As such a feature would require a high number of API calls, it should probably be restricted to a lower number of tokens per call. In my view, completing a sentence probably makes most sense, take this example from using OpenAI in nvim:

I also stumbled upon this post at Hackernews, showing that an API call to GPT seems to be pretty straightforward. What I lack is the experience with setting up completions in Obsidian, which, well, this plugin already does? 😊

Just so you know, I made a similar FR at the text generator plugin, which has implemented it in an experimental stage.
From my side, this can be closed, but leaving this open in case you might wanna implement it in a different manner

Hi, @chrisgrieser :)
Verious Complements is focused on the speed at which completions appear. It is not on the order of seconds, but on the level of tens of milliseconds. We do not intend to implement this in the near future, as the current speed proposed by the AI is far below that.
Nevertheless, day-to-day evolution is fast... We should leave this challenge behind 👍


Use constantize instead of Kernel.const_get to lookup Toolbar class

If the GenericObjectDefinition model is auto-loaded by Rails before ApplicationHelper::Toolbar::GenericObjectDefinition then Kernel.const_get will return the model not the toolbar when called with Kernel.const_get(“ApplicationHelper::Toolbar::GenericObjectDefinition”) in other words:
[1] pry(main)> GenericObjectDefinition
=> GenericObjectDefinition(id: integer, name: string, description: string, properties: text, created_at: datetime, updated_at: datetime, region_number: integer, region_description: string)
[2] pry(main)> Kernel.const_get("ApplicationHelper::Toolbar::GenericObjectDefinition")
=> GenericObjectDefinition(id: integer, name: string, description: string, properties: text, created_at: datetime, updated_at: datetime, region_number: integer, region_description: string)

This causes a couple of test failures:
NoMethodError:
       undefined method `definition' for #<Class:0x007fb406e8bdb0>
     # ./app/helpers/application_helper/toolbar_builder.rb:21:in `build_toolbar'
     # ./spec/helpers/application_helper/toolbar_builder_spec.rb:2071:in `block (4 levels) in <top (required)>'
     # ...
     # ./spec/helpers/application_helper/toolbar_builder_spec.rb:2081:in `block (4 levels) in <top (required)>'

Making this a blocker since tests will often fail on euwe without it.  It's possible for this to fail in normal code although it's very much order of operation dependent.

🎉

Euwe Backport details:
$ git log -1
commit 3f0dfc8ee747979e37d6884ca5f4168703d0b48d
Author: Joe Rafaniello<EMAIL_ADDRESS>Date:   Fri Nov 4 17:42:14 2016 -0400

    Merge pull request #12451 from hayesr/replace_const_get

    Use constantize instead of Kernel.const_get to lookup Toolbar class
    (cherry picked from commit a3e4523b822f8cc0effaf58ff5296f77f2bd158f)


Fix setuptools develop installs

Fixes #53.

Commit 2472559c51349522784fe5c25fee36c1c6f8be56 in this PR breaks my install.
I'm installing like this:

pip install ../psi-j-python/

which I think should be an acceptable thing to do.
I get this error with the local executor called from my parsl provider prototype:
job ff93e4ad-aeb4-4b0a-b360-e09cb8314567 status JobStatus[FAILED, time=1636382664.678896, message=/bin/bash: /home/benc/parsl/exaworks/virtualenv-3.9/lib/python3.9/site-packages/psij/launchers/scripts/single_launch.sh: No such file or directory, exit_code=127]

I haven't really dug into what's missing/causing this yet, but I'm adding this comment in case it's obvious to someone else right away.

pip install -e ../psi-j-python works for me so this isn't blocking ongoing work for me, but it may be of more relevance when I want to have a user do an install from a network source.

So these launcher scripts combined with python's silly install workflow may have been the single most annoying thing in this project. And, to make matters worse, the current solution is pretty bad if running on a system in which the installed package is not on a shared fs and with a stable absolute path.
So I'm thinking maybe:

We should move these launcher scripts to the loosely defined work dir (the one used by queuing system executors to write submit scripts and output files)
Build said scripts lazily. This may need some file-based locking or we can do per-process unique scripts.

Thoughts?

Can you give more detail about the shared fs / stable absolute path issues? I don't understand. Although I agree that the installation can get pretty annoying, this particular problem was just a (somewhat subtle) bug I introduced by being careless.

Can you give more detail about the shared fs / stable absolute path issues?

There is no guarantee that the location where PSI/J is installed on the head node is accessible from service/compute nodes, which run the meat of the submit script, including the line that invokes the launcher. It is also possible, but unlikely, that, if that path is accessible from the CNs, it may be mounted in a different location.
The point is, if we want a robust solution that is a turn-key solution to most clusters, we may want to put those launcher scripts in a location that is more likely to be accessible to both head node and compute nodes.

I think I made the proposal in the past can't find the discussion about it, so can't remember how it was resolved.  Sorry for the repetition thus :-/  :
I would suggest to not use the preprocessor on the shell scripts.  The only purpose if to resolve the #include <lib.sh> statements, but the shell already has the dot operator to do the very same.  So lib.sh could be installed (preferably with a more unique name like psij_utils.sh in $PYTHON_PREFIX/bin/, and the submission scripts could begin with a . psij_utils.sh directive (the dot operator searches $PATH for include candidates).
We use that mechanism in our own code without any problems.

I like that idea @andre-merzky . (We have a bit of a parallel discussion going on in #79 so maybe we are spreading the discussion across too many issues.) I assume there is a nice way to do that with setuptools that works with develop installs?

@jameshcorbett : yeah, seems to work ok with develop installs:

(ve_test)  $ which test.sh
(ve_test)  $ cat bin/test.sh
#!/bin/sh

export FOO=FOO
. test_lib.sh
echo "FOO=$FOO ($(which test_lib.sh))"

(ve_test)  $ cat bin/test_lib.sh

export FOO=BAR

(ve_test)  $ grep -C 1 test.sh setup.py
    'scripts'            : [
                            'bin/test.sh',
                            'bin/test_lib.sh',

(ve_test)  $ pip install -e .
[...]
  Running setup.py develop for radical.pilot
Successfully installed radical.pilot

(ve_test)  $ test.sh
FOO=BAR (/mnt/home/merzky/radical/radical.pilot.devel/ve_test/bin/test_lib.sh)

We have a bit of a parallel discussion going on in #79 so maybe we are spreading the discussion across too many issues.

Yeah, that would have fit better there, thanks for the cross-link!


Toggle Ftrack upload in StandalonePublisher

Closes https://github.com/pypeclub/client/issues/86
SP previously added 'ftrack' family to all instances implicitly. It wasn't possible to decide when to send or not send to FTrack.
This new collector allows configuration by hosts/family/task names (regex) combination.
By default 'render' and 'image' instances will be marked to send to FTrack for 'standalonepublisher' host.
Requires updating pype-config with
https://github.com/pypeclub/pype-config/pull/109

Updated after review's comments.

Found a issue (not only) with editorial publishing. Defined profiles in collector are enough for most of cases but not for editorial where main family is not enough but we have to be able check for families on instance. E.g we have to be able catch if "plate" has "review" in families or not because if does not have "review" then "ftrack" should not be added.
We found out 2 possible modifications to be able do that (both do the same thing in different way):
Option n 1:
Add to each profile additional_filtering with it's add_ftrack_family value and families filter as list.

all families from additional filter must be on instance else value from profile is used
if more additional filters match then filter with more families is used
if there are 2 or more matching additional filters with same amount of families then first in settings is used

Option n 2:
Add subfamilies to mail profile. That would require to modify profile filtering as it can't handle list2list comparisons and have to able solve point system for list comparisons. In the end would work the same way as option n1 but is defined in single item.

Option n 3:
Ignore "somehow" the plugin for editorial publishing. It can't be done using families as at the time of CollectFtrackFamily there are already "plate" families which should be skipped on editorial but can't be skipped when "plate" is published without editorial.
If anybody has any idea, opinion or other way how to solve this issue please came up @mkolar @jezscha @antirotor .
Conclusion is that current state would cause that almost everything will be published to ftrack which is not expected.

I basically agree with option 1 but maybe better terminology or description will make it more intuitive. Subfamilies - although I understand it - it is by my opinion very confusing - there are no subfamilies as each family is on the same level as others. Also additional filtering - even if it provide placeholder for future expansion, it is to abstract. We want to add ftrack family to the list of families specified - its like mapping. So:
Host names: [ ]
Families: [plate]
Task names: [ ]
----------------
Add `ftrack` if additional families are present:
+/- [review]

Subfamilies - although I understand it - it is by my opinion very confusing

I agree that subfamilies are not right. But don't have better name and using Families again but for different "families" would be even more confusing.

there are no subfamilies as each family is on the same level as others

Yes and no. There is always one main family. Main family is used for profile filtering (Families label).

We want to add ftrack family to the list of families specified - its like mapping.

How would you display, filter and give ability to add filtering like:

Maybe overcomplicated but not far from reality.

I would go with Option 1, with same relabeling:

first 'families' to 'Main family'
'Additional families' instead of 'Sub-families'
Ondra's 'Add ftrack if additional families are present:'

I think its a bit clearer than Option 2. And when we will use it next time, labels might be different, for the use case.

Ondra's 'Add ftrack if additional families are present:'

But that would make the option Add ftrack if additional families are present: required, or?

I added this labels, hope it makes it a bit clearer, open for comments.

Closing 2.x version without merging, as it is new feature and no one asked for 2.x version. (agreed on Standup)


Update CAPI

To-do

[ ] Implement CAPI Calls

[ ] Last State (waiting for changes upstream in CADET-Core)
[ ] Coordinates  (waiting for changes upstream in CADET-Core)

[ ] Read solution

[x] Implement all solution types
[ ] Some errors still come up when e.g. there is no solid phase in a unit. Need to investigate more

[ ] Formalize test in pipeline

Considerations for Interface v2

Implement method to run single section
Allow modifying parameters during sections

I overhauled the CADETDll class and added missing methods. I also tried adding some tests which test different configurations that split components and/or ports.
@sleweke: Could we add an option in createLWE that allows to set multiple particle types?
@Immudzen: There is an issue in https://github.com/modsim/CADET-Python/blob/ac294c2dfa02a3a42fdfd7666deb548800360d5d/cadet/cadet_dll_utils.py#L208 for one of the examples.
I get
Exception ignored on calling ctypes callback function: <function param_provider_get_string_array_item at 0x7f3d54930b80>
Traceback (most recent call last):
  File "/home/jo/code/CADET-Python/examples/../cadet/cadet_dll_utils.py", line 208, in param_provider_get_string_array_item
    reader.buffer = bytes_val
UnboundLocalError: local variable 'bytes_val' referenced before assignment

I assume it's because there is no else for the if-clauses.
In this case, it breaks when reading sensitivites. o is array([b'COL_POROSITY'], dtype=object). Any idea how to fix this?

As a quick fix, I did the following:
    if n in c:
        o = c[n]
        if index == 0:
            if hasattr(o, 'encode'):
                bytes_val = o.encode('utf-8')
            elif hasattr(o, 'decode'):
                bytes_val = o
            elif hasattr(o[index], 'decode'):
                bytes_val = o[index]
        else:
            if hasattr(o[index], 'encode'):
                bytes_val = o[index].encode('utf-8')
            elif hasattr(o[index], 'decode'):
                bytes_val = o[index]

It gets the job done but it doesn't make me very happy since it's more of a patch. I think I don't fully understand what's happening here so maybe we could give it some more structure?

@sleweke-bayer I worked a bit on the interface and I believe we can now read everything required for feature parity with the CLI. However, when writing tests, I realized that there are still some small issues regarding the dimensionality of the parameters (see description above). Once that's dealt with, I think we're ready to clean up and merge. :nerd_face:

Some mismatch with solution_bulk detected (CLI returns shape=(nTime, nCol, nRad, nComp); CAPI returns shape=(nTime, nRad, nCol, nComp))

Expanding on this a little, I was again a bit confused what convention we finally used for structuring our multi-dimensional data. The only information I could find was in the Forum but this never seems to have been applied. From my understanding, we (mostly) use this structure:
'nTime', ('nPort',) ('nAxialCells',) ('nRadialCells',) ('nParTypes',) ('nParShells',) 'nComp', ('nBound',)

At least with this, I was able to read get consistent data shapes when comparing CLI and CAPI results. However, the flux was not consistent and would need to be restructured in CADET-Core (see 184).
If this is indeed correct, we should also update the forum post and publish this in other places s.t. we can easily access / find it.

https://github.com/modsim/CADET-Python/pull/14 is merged, I've rebased dll onto master to remove merge conflicts and pushed that branch to https://github.com/modsim/CADET-Python/tree/dll_rebased_onto_master to avoid force-pushes. Before merging dll into master, I suggest we merge it onto that branch and then merge from there.

Once this is merged we should merge https://github.com/fau-advanced-separations/CADET-Process/pull/169 to maintain compatibility.

@hannahlanzrath Would you be up to adding the MCT in here?

I'm getting this error:
Traceback (most recent call last):
  File "C:\Users\ronal\PycharmProjects\CADET-Python\cadet\cadet_dll.py", line 1697, in __del__
    self._api.deleteDriver(self._driver)
OSError: exception: access violation reading 0xFFFFFFFFFFFFFFFF

When

    def __del__(self) -> None:
        """
        Clean up the CADET driver on object deletion.
        """
        self._api.deleteDriver(self._driver)

is called.
Everything else works.

I might prefer moving the last commit ( b3f7996 ) to another branch to make it easier to continue work on this PR. We just need to remember to merge it before releasing the next CADET-Python Version


[eas-cli] [ENG-11310] Don't require expo on fresh react-native project

Why
https://linear.app/expo/issue/ENG-11310/fix-warnings-errors-that-come-up-when-adding-eas-to-a-react-native-cli
How
When Expo config error is thrown while initiating a project only a warning message is printed instead of throwing an error and failing the command
Test Plan
Automatic tests
All tests pass
Manual tests
Before
eas init

After
eas init

eas build -p ios --auto-submit

/changelog-entry bug-fix Don't require expo on fresh react-native project


Picking refactor

Fixes #7247
Moved most of the picking functions and their helper functions out of Scene.js and into a new file called Picking.js, a transfer of about 800 lines of code. Since Scene.js exposes the public api for picking, Scene.js is simply a passthrough for all picking functions. Tests, linter, and sandcastle demos are all working. Hopefully nothing slipped through the cracks. No new tests or public facing changes were made.
Some thoughts:

Scene.js documentation lists the kinds of errors each function can trigger, but the actual handling happens in Picking.js. This could be good if other areas of code besides Scene.js call picking functions, but it makes the Scene.js documentation a bit confusing.
updateMostDetailedRayPick and getPickDepth are the only two functions in Picking.js that are called from Scene.js but are not public api functions. Should these kinds of functions stay in Picking.js? Also, should updateMostDetailedRayPicks go in Scene.js or Picking.js?
I was tempted to put all scratch variables in the same area of code in Picking.js, but decided against it because Scene.js split them up to be near the functions they are used. The thing is, some of these scratch variables are used in more than one function so their placement in code may appear random at times.

Thanks for the pull request @IanLilleyT!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

@lilleyse ready for review

The updated code looks good. All the Sandcastle examples that are related to picking are still working normally.


Fix to issue #17 limits cmd_merge to be single-threaded

Hi,
it looks like the fix for issue #17, which puts some limits on the number of threads in cmd_merge, is a bit too aggressive, resulting in only using a single thread even for big workloads:
https://github.com/google-research/deduplicate-text-datasets/blob/ad86c7f65ac626581fe3a4277106309bc6b50c23/src/main.rs#L1020-L1023
texts.len() is equal to nn (the number of input parts), I think you want something like
    let num_threads = std::cmp::min(num_threads, std::cmp::max((texts_len.iter().sum::<usize>() as i64 - 1024)/10, 1));

instead.

Oops. Yeah that's bad. Let me fix this.

I just added the great suggestion by @kleinj as a PR: https://github.com/google-research/deduplicate-text-datasets/pull/22/files.
I was running the code on a 10's of gigabyte dataset and this fix makes the code only run in about 2 hours with a lot of CPUs (it would've taken 100's of hours without this fix). Let me know what you think @carlini!

This should have been fixed.


TST: test against pytest pre-releases

I noticed that pytest 8.0.0rc1 breaks AMICAL's tests. I'll be working on fixing this here.

pytest 8.0.0 is now released, so general CI is also impacted.


Why does NLog wrap the filename in parentheses

I'm looking at changing over to use NLog and I want to get ${callsite} to give me the full path-name only. Why do you force it to include parentheses and a line number? If I wanted to have those, I could add them to the configuration myself.
Surely it is a mistake to add things that are not required and, even if you must do so, it is a worse mistake to prevent the user from removing them with the qualifiers?

We will fix this in the future: https://github.com/NLog/NLog/issues/2583

Sorry for the lack of response. I got the notification of the closure but no notification of your previous comments. Your proposed solution would work fine for me, thank you.

Ok thanks for the feedback!


[Feature] Provide a way to reserve subsystem functions to a single master

When multiple components are shared resources in a system, it might be critical to limit certain functionalities to a master caller during certain states.
An example would be where a pump is being controlled (Start-Stop) by a system and interlocks are to be put in place to disallow a remote call from stopping the pump while the owner system is in a critical state where pump should never be turned off. The system would reserve the component and be the sole caller which can change the state of this component. It could then release the component when it itself transitions to a maintenance mode, allowing for engineering/manual/test UIs to take control of the pump without causing a conflict during critical state.

Release 1.1.17


Firefox Issue when zoomed in

Expected Behavior

No gap/spaces between text when zoomed in.
Chrome Browser :

Current Behavior

There are gap/spaces between text when zoomed in.
Firefox Browser :

Possible Solution

Steps to Reproduce (for bugs)

Create any chart
Open it on Firefox Browser
Zoom in to see the text are having gaps

Context

I'm trying to accomplish what the chrome browser can do whit the chart labels. If you can suggest any configuration I will appreciate it so much.
Environment

Chart.js version: 2.8
Browser name and version: Mozilla Firefox 72.0.2 (64-bit)

I would recommend filing this to Firefox. We don’t change our text rendering based on zoom. We always use the drawText method to draw the entire word at a time.

It has something to do with the devicePixelRatio and the scale (releases) or [setTransform] (in master) (https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/setTransform) used to handle the zoom.
If you set devicePixelRation: 1 in the chart options, spaces remain equal, but it gets pixelated when zoomed.
Anyway, that should narrow it down a bit if filing to Firefox.


[Bug]:If the first login method fails then when the user-tries to login with the second login method the server goes down

Is there an existing issue for this?

[X] I have searched the existing issues

Description
If the first login method fails then when the user-tries to login with the second login method the server goes down
https://ca9sxzlrgy.vmaker.com/record/MTvPotKAHK34jQZR
Steps To Reproduce

set up both OIDC and SAML login
first login with saml the user will be logged in successfully
Now try to Login with OIDC the Login fails
Now again try to login with SAML the server goes down
The 503 error will be displayed

Public Sample App
No response
Issue video log
No response
Version
localhost

Okay few things failed togather here.

The JWK Set URL for OIDC on this server is incorrect. It should be /keys in the end, not /key.
After this failed OIDC attempt, the SESSION cookie is not removed. It sticks around. This cookie is what appears to be confusing Keycloak, and it doesn't know how to proceed.

Check the video for full details.
https://user-images.githubusercontent.com/120119/209755910-e8030fe1-1dbb-438e-a8ec-4e02951ee258.mp4

On further tests though, it looks like a SAML-only issue. I can reproduce this with just login with SAML, logout, and try to login with SAML again, and I see this error.


jdk-http-client

see https://www.selenium.dev/blog/2022/using-java11-httpclient/

Not needed after upgrading to Selenium 4.14.0


Implement Modal Component that renders notices according to config, and saves preferences

The overlay needs to be triggered by banner or link click
We can probably use the same Preact component from https://github.com/ethyca/fides/issues/3162

Mostly done here https://github.com/ethyca/fides/pull/3291
But isn't integrated with any endpoints yet, blocked by https://github.com/ethyca/fides/issues/3159

moving to done


No clusters but no error messages when previously selected kubeconfig file doesn't exist.

After moving from a mac to a linux machine, and having vs.code configured for Settings Sync, my User config for vs-kubernetes looked like this:
    "vs-kubernetes": {
        // .... other settings ...
        "vs-kubernetes.knownKubeconfigs": [
            "/Users/my-user/.kube/eksconfig"
        ],
        "vs-kubernetes.kubeconfig": "/Users/my-user/.kube/eksconfig"
    },

This meant that none of the configured clusters in the configuration files were being shown, and no error message was displayed, even though the KUBECONFIG environment variable was set to ~/.kube/config:~/.kube/eksconfig (i.e. /home/my-user/.kube/config:/home/my-user/.kube/eksconfig, notice /home instead of /Users).
This was a serious usability issue for me, in that I spent hours trying to figure out why no clusters would display, even though kubectl was behaving correctly on the command line.
No error message was shown in either the devtools console or the Output window for Kubernetes. The CLUSTERS sidebar views just silently did not display any cluster.
It was only after clicking the ... button by the CLUSTERS sidebar view, selecting Set Kubeconfig by the umpteenth time, and finally realizing the selection /Users/my-user/.kube/eksconfig had a capital U that I understood the issue.
Please consider adding either a dialog or, even better, a message in the CLUSTERS sidebar view, notifying that the selected Kubeconfig file does not exist.

By the way, the fix was to go into the JSON settings and remove the settings mentioned above. This showed me all clusters from all kubeconfigs in the KUBECONFIG environment variable.
Another fix was to select the ... menu in CLUSTERS, then Set Kubeconfig, then Add new kubeconfig and navigate to a properly existing kubeconfig, although the non-existing kubeconfig was still selectable.
Also note the inconsistent capitalization of Kubeconfig between Set Kubeconfig and Add new kubeconfig.
I also think its a good idea to go into User Settings, searching for vs-kubernetes and clicking the cog-wheel settings button and disabling Sync This Setting. I suggest this should be the default, since we're not syncing the actual kubeconfig, so there is no guarantee that we'll be able to find the kubeconfig between vs.code installs of the user of the same synced config.


Download any file with Python requests library

I have a python code that reads an environment variable and downloads a file from the cloud storage:
import os
import requests

model_name = os.environ.get('MODEL')

if model_name:
    url = "https://viso-public-storage.s3.eu-central-1.amazonaws.com/ObjDet/CPU/{}.tar.xz".format(model_name)
    print(f"Downloading - {url}")
    with open(f'/data/{model_name}.tar.gz', 'wb') as f:
        r = requests.get(url)
        f.write(r.content)
print("Finished!")

If MODEL environment variable exists, it downloads from our S3 bucket.
But once slimified, it doesn't work saying that the device or resource is busy:
ubuntu@ubuntu:~/test_docker_slim$ docker run -v /data:/data --rm --network host -e MODEL='ssd_mobilenet_v1_coco_2018_01_28' ts.slim
Downloading - https://viso-public-storage.s3.eu-central-1.amazonaws.com/ObjDet/CPU/ssd_mobilenet_v1_coco_2018_01_28.tar.xz
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/urllib3/connection.py", line 170, in _new_conn
    (self._dns_host, self.port), self.timeout, **extra_kw
  File "/usr/local/lib/python3.7/site-packages/urllib3/util/connection.py", line 73, in create_connection
    for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
  File "/usr/local/lib/python3.7/socket.py", line 752, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
OSError: [Errno 16] Device or resource busy

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/urllib3/connectionpool.py", line 706, in urlopen
    chunked=chunked,
  File "/usr/local/lib/python3.7/site-packages/urllib3/connectionpool.py", line 382, in _make_request
    self._validate_conn(conn)
  File "/usr/local/lib/python3.7/site-packages/urllib3/connectionpool.py", line 1010, in _validate_conn
    conn.connect()
  File "/usr/local/lib/python3.7/site-packages/urllib3/connection.py", line 353, in connect
    conn = self._new_conn()
  File "/usr/local/lib/python3.7/site-packages/urllib3/connection.py", line 182, in _new_conn
    self, "Failed to establish a new connection: %s" % e
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x7fce707c1210>: Failed to establish a new connection: [Errno 16] Device or resource busy

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/requests/adapters.py", line 449, in send
    timeout=timeout
  File "/usr/local/lib/python3.7/site-packages/urllib3/connectionpool.py", line 756, in urlopen
    method, url, error=e, _pool=self, _stacktrace=sys.exc_info()[2]
  File "/usr/local/lib/python3.7/site-packages/urllib3/util/retry.py", line 573, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='viso-public-storage.s3.eu-central-1.amazonaws.com', port=443): Max retries exceeded with url: /ObjDet/CPU/ssd_mobilenet_v1_coco_2018_01_28.tar.xz (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fce707c1210>: Failed to establish a new connection: [Errno 16] Device or resource busy'))

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "main.py", line 10, in <module>
    r = requests.get(url)
  File "/usr/local/lib/python3.7/site-packages/requests/api.py", line 76, in get
    return request('get', url, params=params, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/requests/api.py", line 61, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/requests/sessions.py", line 542, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.7/site-packages/requests/sessions.py", line 655, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/requests/adapters.py", line 516, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='viso-public-storage.s3.eu-central-1.amazonaws.com', port=443): Max retries exceeded with url: /ObjDet/CPU/ssd_mobilenet_v1_coco_2018_01_28.tar.xz (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fce707c1210>: Failed to establish a new connection: [Errno 16] Device or resource busy'))

Any idea why docker-slim doesn't suport random file download to the mounted volume?
You can easily test on your end by cloning this repo - https://github.com/openedhardware/test_docker_slim
Cheers!

Thanks for the repro project and the repro instructions @openedhardware ! I think this failure is due to a missing env var when you run docker-slim. Adding --env MODEL='ssd_mobilenet_v1_coco_2018_01_28' to the build command did the trick for me. Without that env var the url value in the app ends up being invalid, so the requests.get(url) call in the app fails when docker-slim executes the application in its temporary container.
Here's what the build command that worked for me: docker-slim build --http-probe=false --env MODEL='ssd_mobilenet_v1_coco_2018_01_28' --mount $(pwd)/data:/data --include-path /usr/local/lib/python3.7/site-packages/certifi --show-clogs ts

@kcq
Yeah, it worked with a constant variable when docker-slim build.
But that environment variable will be changed depending on the condition.
How can we support various variables?
Thanks for your effort!
Cheers.

There a few options... The value of MODEL needs to reference something you have. For example, it can be a test model file name that's always there, so setting it to anything as long the generated URL points something on S3 should do the trick. Option two is to use an environment variable to set the MODEL env var for docker-slim (e.g., docker-slim build --http-probe=false --env MODEL=$YOUR_CURRENT_MODEL --mount $(pwd)/data:/data --include-path /usr/local/lib/python3.7/site-packages/certifi --show-clogs ts) where $YOUR_CURRENT_MODEL is set according to your conditions. Option 3 is to have the whole url in the app settable through the env vars (a few different ways to do it... including using two env vars, one for the format string and another for the model name).

I had a similar situation.
In my case, I give hostname as environment variable and if it is not given, it will be localhost.
When I ran docker-slim build, I didn't give the environment variable, so the app accessed localhost, which should have resulted in connection refused.
In this case, would it work if I had set the environmental variable to hostname that would return an HTTP response?


[fix] CurrencyUtils colombia

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@455ad51). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master   #1265   +/-   ##
========================================
  Coverage          ?   2.53%
  Complexity        ?     125
========================================
  Files             ?     376
  Lines             ?   15251
  Branches          ?    1307
========================================
  Hits              ?     387
  Misses            ?   14839
  Partials          ?      25

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 455ad51...5d99c75. Read the comment docs.


docs: Update link to AKS docs for OIDC Issuer

Reason for Change:
The link to the Azure docs is semi-broken due to the change in the header name. It is also region specific instead of generalized, and uses the old docs subdomain instead of learn.

Remove preview suffix
Remove region-specific path
Switch to new learn platform instead of docs

Requirements

[x] squashed commits
[x] included documentation
[x] added unit tests and e2e tests (if applicable).

Issue Fixed:

Please answer the following questions with yes/no:
Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?

[ ] yes
[x] no

Notes for Reviewers:

/azp run Azure Workload Identity CI


Quick Fix

Maybe try something like this.

Thanks for the correction. I am pushing the suggested changes live now.


tar_embed_list dim seems wrong

tar_embed_list's dim seems larger than 8, maybe soldier_2_concat_result insdead of fc3_soldier_result be added into tar_embed_list?

The tar_embed_list consists of [none_target, self, enemy, 4 nearest soldiers, nearest tower], which matches the dimensions in target head in action space. You can refer to https://aiarena.tencent.com/hok/doc/environments/index.html#action-spaces for details.


Added support for listhandles accessrights

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-storage-file-share


🐛 BUG: Wrangler dev fails to fetch()

Which Cloudflare product(s) does this pertain to?
Wrangler
What version of Wrangler are you using?
3.0.0
What operating system are you using?
Windows 10
Describe the Bug
After updating to the latest version, fetch API calls started to fail in wrangler dev. Fails on any website, even the mighty google.com
Rolled back to v2.19.0 - everything works fine again.
A bit more of the context: I'm using fetch to call WIX'es REST API, via https, as expected. The 3.0 version seems much faster, so I assume that some serious changes were applied under the hood. Sometimes I can see mention of WSA in this error messages, and WSA is a part of Windows networking APIs, somewhat suspicions to me to see it here.
Well, I haven't rebooted my system after updating, we're not in 1999, shouldn't be the cause of the problem, huh 😁
Screenshot:

Wanted to also add this popus up during try/catch
workerd/jsg/util.c++:276: error: e = kj/compat/tls.c++:215: failed: TLS peer's certificate is not trusted; reason = unable to get local issuer certificate
stack: 7ff76b8e8a6f 7ff76b92c9dd 0 0 0 0 0 0 0 0 0 0; sentryErrorContext = jsgInternalError

but when the whole worker response itself wraps up, it also emits
workerd/jsg/util.c++:276: error: e = kj/async-io-win32.c++:294: failed: ConnectEx(): #1225 The remote computer refused the network connection.
stack: 7ff76b8e8950 0 0 0 0 0 0 0 0 0 0 0 0 0 7ff76b8f6234; sentryErrorContext = jsgInternalError

Same here, unable to use plain "wrangler dev" after uprading to 3.0.0. Using Windows 11.
My app also uses fetch() to external APIs.
Using "wrangler dev --remote" appears to work with 3.0.0.
workerd/jsg/util.c++:276: error: e = kj/compat/tls.c++:215: failed: TLS peer's certificate is not trusted; reason = self signed certificate in certificate chain
stack: 7ff769b98a6f 7ff769bdc9dd 0 0 0 0 0 0 0 0 0 0; sentryErrorContext = jsgInternalError

im using cloudlflare queues, and --emote does not support queues yet :(
==> https://github.com/cloudflare/workers-sdk/issues/2478
catch22 ==> im dead in the water here.... please help

Same issue here:
workerd/jsg/util.c++:276: error: e = kj/compat/tls.c++:215: failed: TLS peer's certificate is not trusted; reason = unable to get local issuer certificate
stack: 7ff683b18a6f 7ff683b5c9dd 0 0 0 0 0 0 0 0 0 0 0 0 7ff683b26234; sentryErrorContext = jsgInternalError

(the URL being fetched here is https://www.patreon.com/api/oauth2/token .

We probably need the "+1" button for issues at this point 😅

im using cloudlflare queues, and --remote does not support queues yet :( ==> #2478
catch22 ==> im dead in the water here.... please help

I've managed to work around this by downgrading to Wrangler v2.20.0 and running wrangler dev --local --persist

Update on Wrangler v3.0.1
Test code:
export default {
    async fetch(request: Request, env: Env, ctx: ExecutionContext): Promise<Response> {

        const result = await (await fetch('https://google.com')).text()
        console.log(result)

        return new Response('Hello World!')
    }
}

Results in console output:
⎔ Starting local server...
[mf:wrn] The latest compatibility date supported by the installed Cloudflare Workers Runtime is "2023-05-18",
but you've requested "2023-05-31". Falling back to "2023-05-18"...
[mf:inf] Ready on http://<IP_ADDRESS>:8787/
workerd/jsg/jsg.c++:133: error: took recursive isolate lock; kj::getStackTrace() = 7ff65d9fefbe 7ff65d67aa74 7ff65d87a236 7ff65d8b332a 7ff65d8b3272 7ff65d428a05 7ff65d424748 7ff65d56b3a4 7ff65d5a7d2c 7ff65d567304 7ff65d563a51 7ff65d65283c 7ff65d563a51 7ff65d6662fc 7ff65d563a51 7ff65c7cc23c 7ff65d569a1c 7ff65d563a51 7ff65d65283c 7ff65d663d1f 7ff65d663060 7ff65d4263c0 7ff65db58095 7ff65d892fd8 7ff65d8deeee 7ff65db5968f 7ff65c91bc8b 7ff65c91b6ba 7ff65c91ad9b 7ff65e1394f8
[mf:inf] GET / 200 OK (1679ms)
...
[google.com's HTML goes here, not gonna include it bc it's irrelevant]

So, it kinda fails, but also works at the same time 🤯

Reading between the lines of the linked PR #3352, I was able to work around this by adding the following to my package.json:
  "resolutions": {
    "miniflare": "3.0.1"
  }

Probably not the best idea to pin it like this long term, but it allowed me to get back to work.

is there an update to wrangler 3.x to fix this yet?

@mrbbot appreciate the update, unfortunately even the latest miniflare (3.0.2) doesn't seem to solve the issue either.
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>

if its any clue - it seems exacerbated when I attempt to run workers in the background

For the error failed: TLS peer's certificate is not trusted; reason = unable to get local issuer certificate I found that you can export SSL_CERT_FILE to get around this problem. In my case I run export SSL_CERT_FILE=/etc/ssl/certs/ca-certificates.crt.
I use NixOS, and have created a reproduction at scottwillmoore/cloudflare-workers-with-nix.

Same issue here - can't connect to remote PG database. Running locally I get the OP's error; running with --remote I get a PG error saying connection terminated unexpectedly. I've updated to Wrangler 3.0.1.

For the error failed: TLS peer's certificate is not trusted; reason = unable to get local issuer certificate I found that you can export SSL_CERT_FILE to get around this problem. In my case I run export SSL_CERT_FILE=/etc/ssl/certs/ca-certificates.crt.
I use NixOS, and have created a reproduction at scottwillmoore/cloudflare-workers-with-nix.
EDIT: My reproduction uses<EMAIL_ADDRESS>
Question... are you using wrangler from nixpkgs, or wrangler from node_modules?
I'm using NixOS as well, but my wrangler is from node_modules (it's installed from npm).
I'm hitting this error as well. However also setting SSL_CERT_FILE=/etc/ssl/certs/ca-certificates.crt wrangler dev works as well.
That implies that my current wrangler (3.18) doesn't have built in certificates.
└─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>
Either wrangler should default https://github.com/cloudflare/workers-sdk/commit/74c8b85becad15f112ae5a7438784f0422e3a8e5 but it also makes sense that in NixOS, it should refer to the OS certs.


POST /messages/flags/narrow: Filter by is:unread if client is asking to add read flag

Applying this filter has been seen to reduce processing time by multiple seconds: zulip/zulip-flutter#377. In that issue, Greg said:

It'd probably also be good for the server to notice when we're adding the "read" flag, and add the is:unread filter automatically.

It's easy enough for clients to request the filter, but it's possible to forget.
This came up in #production help: https://chat.zulip.org/#narrow/stream/31-production-help/topic/intermittent.20high.20latency/near/1707853

This doesn't work, sadly -- because it changes the meaning of num_before and num_after.  Assume a post of /messages/flags/narrow with:
{
  "anchor": 42,
  "num_before": 2,
  "num_after": 2,
  "narrow": nil,
  "op": "add",
  "flag": "read"
}

This will mark message 42, and the two messages immediately before and after it as read (even if they already were).  If we auto-add is:unread to the narrow, it will instead skip over the two read messages before 42, and find two unread messages somewhere before 42 and mark them read.
If zulip/zulip-flutter#377 is auto-adding is:unread in the client, then I think its behaviour is incorrect.

For example, this diff fails its new test:
diff --git zerver/tests/test_message_flags.py zerver/tests/test_message_flags.py
index bc906e2c07..f05180081f 100644
--- zerver/tests/test_message_flags.py
+++ zerver/tests/test_message_flags.py
@@ -268,6 +268,22 @@ class UnreadCountTests(ZulipTestCase):
             message_ids[3:8],
         )

+        # Re-posting the same request should nominally generate no updates
+        response = self.assert_json_success(
+            self.client_post(
+                "/json/messages/flags/narrow",
+                {
+                    "anchor": message_ids[5],
+                    "num_before": 2,
+                    "num_after": 2,
+                    "narrow": "[]",
+                    "op": "add",
+                    "flag": "read",
+                },
+            )
+        )
+        self.assertEqual(response["updated_count"], 0)
+
         response = self.assert_json_success(
             self.client_post(
                 "/json/messages/flags/narrow",
diff --git zerver/views/message_flags.py zerver/views/message_flags.py
index 808f05b8f0..9f50b4ef6e 100644
--- zerver/views/message_flags.py
+++ zerver/views/message_flags.py
@@ -90,6 +90,9 @@ def update_message_flags_for_narrow(
     )
     num_after = min(num_after, MAX_MESSAGES_PER_UPDATE - num_before)

+    # This is an invalid optimization
+    if flag == "read" and operation == "add":
+        narrow = (narrow or []) + [{"operator": "is", "operand": "unread", "negated": False}]
     query_info = fetch_messages(
         narrow=narrow,
         user_profile=user_profile,

This doesn't work, sadly -- because it changes the meaning of num_before and num_after.

Hmmm good point. So there may indeed be nothing to be done here, and clients should just be sure to add this filter themselves when doing bulk mark-as-read operations. (As Zulip web and now zulip-flutter indeed do.)

If zulip/zulip-flutter#377 is auto-adding is:unread in the client, then I think its behaviour is incorrect.

No, I think the code we have there is correct. The relevant commit is https://github.com/zulip/zulip-flutter/commit/0c4653d4f4d8b98f4b12da0bd38d87a24eff30c9 ; see the resulting code at these lines. In relevant part:
Future<void> markNarrowAsRead(BuildContext context, Narrow narrow) async {
  // …
  final apiNarrow = narrow.apiEncode()..add(ApiNarrowIsUnread());

  while (true) {
    final result = await updateMessageFlagsForNarrow(connection,
      anchor: anchor,
      // …
      numBefore: 0,
      numAfter: 1000,
      narrow: apiNarrow,
      op: UpdateMessageFlagsOp.add,
      flag: MessageFlag.read);
    // …
    anchor = NumericAnchor(result.lastProcessedId!);
    // …
  }
}

So we add the is:unread filter, and it does change the meaning of that numAfter: 1000. But that's fine — the numAfter: 1000 is just a batch size, and we're going to loop through with successive batches until we get to the end anyway.

(That updateMessageFlagsForNarrow call is hopefully straightforwardly readable with no knowledge of Dart or the zulip-flutter codebase; in general our API bindings are designed to be thin and faithfully reflect the underlying API, so it corresponds straightforwardly to what's at https://zulip.com/api/update-message-flags-for-narrow .)

Yeah -- if the semantic meaning the client is trying to convey is "find 1000 messages to mark as unread" then it is making the num_after mean the right thing by adding is:unread.  But we cannot, server-side, divine if the client is trying to make sure 1000 messages after this one are read, or that we want 1000 messages to become read.
Closing as invalid, then.

(I agree that if we were auto-adding is:unread unconditionally inside our API binding layer, then that would be incorrect for the same reasons as if it were added in the server. Though it'd probably only ever be a latent bug, in that our codebase probably never will have a call to that API endpoint that isn't of this nature.)

Given that the server won't be handling this for the reason discussed above, I think there is a documentation change that should be made: https://zulip.com/api/update-message-flags-for-narrow should mention that the server has a database index for unread messages and that if you're adding the "read" flag you should therefore consider adding an is:unread filter to your narrow.
That's an important optimization that basically any full-featured client is going to need to make, and it isn't otherwise obvious. So it'd be good for that point to be made clear.


Styled component doesn't accept custom boolean properties

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example:
export const StyledComponent = styled(Box)<{customprop: boolean}>(({ theme, customprop }) => ({
...
}));

Current behavior 😯
Warning in the console:
Received true for a non-boolean attribute customprop
Expected behavior 🤔
No warnings
Context 🔦
No response
Your environment 🌎

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

By default, the styled utility forwards the custom props to the underlying DOM element. You can use the shouldForwardProp option to not forward the prop. Here is how you can do it:
const StyledComponent = styled(Box, { shouldForwardProp: (prop) => prop !== 'customprop' })<{
  customprop?: boolean;
}>(({ theme, customprop }) => ({
  backgroundColor: customprop ? 'yellow' : 'red',
  width: 300,
  height: 300,
}));

export default function App() {
  return <StyledComponent customprop />;
}

It is mentioned in the docs https://mui.com/system/styled/#what-problems-does-it-solve. By default it forwards the custom props.

It adds by default the shouldForwardProp option (that can be overridden), taking into account all props used internally in the MUI components: ownerState, theme, sx, and as.


Sign UP confirmation e-mail problem

Hi,
I install the last rtd on an ubuntu 14.04 : it is working but when i sign in, i don't receive any email confirmation to go further.
Postfix is well configured and i don't find any information about this problem : coul you help me please ?

Confirmation e-mail sent to xxx.
Verify Your E-mail Address

We have sent an e-mail to you for verification. Follow the link provided to finalize the signup process.
You're receiving this e-mail because user xavier at example.com has given yours as an e-mail address to connect their account.
To confirm this is correct, go to http://XXXX.org/accounts/confirm-email/j1oadumgej8sm51hjf6mlaubuuv2gc6y2zdvegrcbij8mungysrpdtyuqw665g0e/
Thank you from example.com!
example.com
Regards,
Xavier

You need to configure an SMTP backend. Closing as this isn't an issue with RTD.


SASL extension Callback required for Confluent Cloud?

https://docs.confluent.io/cloud/current/access-management/authenticate/oauth/configure-clients-oauth.html#client-login-callback-handler-top-level-configuration-options
Note the SASL extension configuration for extension_logicalCluster and extension_identityPoolId. During the OAuth token retrieval step, extensions are ignored, but will be passed to the broker using the SASL extension mechanism from KIP-342.

Implemented


feat: sidekick menu

[x] Add Sidekick Menu (Hamburger Icon)
[x] Allow adding and removing project from Sidekick menu
[x] Link to documentation from Sidekick menu
[x] Organize spectrum2 css file
[x] Fix flash of invalid profile picture when using microsoft idp (logged #162)
[x] Fix broken styling in config picker
[x] Tests

:tada: This PR is included in version 1.25.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Removes cookiecutter dependency

Tracking issue
Closes https://github.com/flyteorg/flytekit/pull/1665
Related to https://github.com/flyteorg/flyte/issues/4418
Why are the changes needed?

This PR removes the cookiecutter dependency from flytekit.
What changes were proposed in this pull request?

This PR replaces the cookiecutter code with a HTTP request, extracting the files from the template, and replacing {{cookiecutter.project_name}} in the contents and file name.
How was this patch tested?

I reran pyflyte init for every template and checked that the behavior is the same as before.

This is great!
Can you also remove the mentions to cookiecutter from the description in https://github.com/flyteorg/flytekit-python-template?


dts: stm32l4: Remove aes node

aes node was present in stm32l4 .dtsi files but existing stm32 crypto driver is not compatible with stm32l4.
Removing aes nodes for now, waiting for the driver to be adapted to this series.
Fixes #78482

Set to DNM, waiting to see if https://github.com/zephyrproject-rtos/zephyr/pull/78669 could resolve this by actually adding missing driver support.

Closing in favor of https://github.com/zephyrproject-rtos/zephyr/pull/78669


Resetting namespace in nested group namespace?

Laravel Version: 5.4
PHP Version: 5.6
Database Driver & Version: mySQL

Description:
I'm creating an API and the API is in the http/controllers/api namespace. however I need to use the Passport::routes(); function. When I do route list, it comes up with:
[ReflectionException]
Class App\Http\Controllers\Laravel\Passport\Http\Controllers\AuthorizationController does not exist

Steps To Reproduce:

How can I stop the namespace from repeating?

And I don't want to enter Passport::routes() in the AuthServiceProvider.

Please ask on the forums.


Store in variable (inside a struct) the elements of scene descriptive files

Store in variable (inside a struct) the elements of scene descriptive files
Check if the values RGB of identifiers (F,C) are between 0 and 255.
@pedperei
@abaiao-r

Solved


Web - Chat- Loading spinner displayed on login in conversation screen

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

Expected Result:
Last opened chat displayed
Actual Result:
Continuous loading spinner displayed until user opens a chat
Action Performed:

Launch the URL and login
Verify the convo screen

Workaround:
Clicking in a chat dismisses the infinite loading icon.
Platform:
Where is this issue occurring?
Web ✔
iOS
Android
Desktop App
Mobile Web
Version Number: 1.0.18-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

https://user-images.githubusercontent.com/44479856/114239770-8144bb80-9954-11eb-8998-35bccbc43a73.mp4
Not sure if related to this PR https://github.com/Expensify/Expensify.cash/pull/2221 since the issue is reproducible in production.
Expensify/Expensify Issue URL:

Triggered auto assignment to @lschurr (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Not sure if related to this PR #2221 since the issue is reproducible in production.

Yes it's related. The PR handling the regression is here: https://github.com/Expensify/Expensify.cash/pull/2322

This is still happening; reopening.

@marcaaron reassigned to you and added Daily label.  Please adjust as needed (just wanting to ensure this gets looked at and addressed internally)

Looks like none of the fixes are available on the prod environment, as even if the bug is still happening there at least the loading background would have been a white color, and not gray as above. That color was changed at least 2 weeks ago

The way PR are stetup, they would close the relevant issue automatically when merged, but it seems the code is not always published to production, if you want to keep the ticket open until the issue is addressed on prod maybe this needs to be handled differently and the ticket is left open until the fix is confirmed

@kidroca is correct here. This issue has been resolved and will go out with the next production deploy.


Syntax Error when using commonjs require

I'm currently seeing an issue when requiring this module in Node 12.16.1.
Using require seems to be resolving to the esm file which causes a syntax error due to the presence of import statements (SyntaxError: Cannot use import statement outside a module)
This issue may be due to the exports value in the package.json pointing to esm.js. One solution may be to use conditional exports.

Sorry! I never got notified about this issue. I’ll look into it, thanks for the report.

@lfarrel6 released as v0.2.1. Please let me know if this fixes the issue for you


Fix multiple metadatas issue

Should fix #134 and #131.
I couldn't recreate there being multiple metadata files, but I know I have experienced having multiple before and it causing the sync status to show the older metadata last modified.
This PR should fix it. I split up parseRemoteStates() so that getting the metadata file can be consistent through the whole plugin. I also added a check for multiple metadata files and deletes them leaving the newest.

Thank you!! merged.


ENH: stats: add Cantor distribution under new infrastructure

Is your feature request related to a problem? Please describe.
In https://github.com/scipy/scipy/pull/21700 there is discussion about factoring a base class for statistical distributions under the new framework. For such a base class to be fully generic, it would need to cover (mixtures of) continuous, discrete, and singular distributions.
The Cantor distribution is a curious example of a singular distribution. Having such a class would help to test/verify the “completeness” of the infrastructure.
Describe the solution you'd like.
Add a class, scipy.stats.Cantor, that represents the Cantor distribution. The class would be derived from a base class to be formed.
Describe alternatives you've considered.
No response
Additional context (e.g. screenshots, GIFs)
Less curious singular distributions exist in higher dimensions. For example, a bivariate normal distribution constructed from two perfectly correlated normal variables.
I’m not aware of any statistical application involving the Cantor distribution. So this enhancement proposal is more for fun. Or, it might be used in school lectures, who knows!

I believe the proposal to add CantorDistribution class is a strong idea and it has some interesting characteristics that differ from typical distributions, which will also provides the users, access to a wider variety of statistical tools. And indeed can be fun for educational purposes.
import numpy as np
from scipy import stats
class CantorDistribution(stats.rv_continuous):
"""Cantor distribution."""
def _pdf(self, x):
    """Probability density function of the Cantor distribution."""
    if (x < 0) or (x > 1):
        return 0

    # The Cantor distribution is defined on [0, 1]
    # Density is concentrated at points in the Cantor set
    density = 0
    if x in [0.5**k for k in range(1, 11)]:
        density = 1 / (np.log(2) * (2 ** (10)))  # Normalized density at Cantor points
    return density

def _cdf(self, x):
    """Cumulative distribution function of the Cantor distribution."""
    if x < 0:
        return 0
    if x >= 1:
        return 1

    # Count contributions from Cantor set points
    cdf = 0
    for k in range(1, 11):
        if (0.5 ** k) <= x:
            cdf += (1 / 2) ** k
    return cdf

def _rvs(self, size=1):
    """Random variates of the Cantor distribution."""
    # Randomly select points from the Cantor set
    points = []
    for _ in range(size):
        num = np.random.rand()
        point = 0
        for k in range(10):  # Number of iterations
            if num < 1/3:
                point += 0  # Take the left segment
            elif num < 2/3:
                point += 0.5 ** (k + 1)  # Take the middle segment
            num = (num - 1/3) * 3  # Rescale for the next iteration
        points.append(point)
    return np.array(points)

def mean(self):
    """Mean of the Cantor distribution."""
    return 0.5

def variance(self):
    """Variance of the Cantor distribution."""
    return 1 / 12

Instantiate the Cantor distribution
cantor_dist = CantorDistribution(name='cantor')
Let me know if this is what you want.

import numpy as np
from scipy import stats
class CantorDistribution(stats.rv_continuous):
#Cantor distribution
def _pdf(self, x):
    """Probability density function of the Cantor distribution."""
    if (x < 0) or (x > 1):
        return 0

    # The Cantor distribution is defined on [0, 1]
    # Density is concentrated at points in the Cantor set
    density = 0
    if x in [0.5**k for k in range(1, 11)]:
        density = 1 / (np.log(2) * (2 ** (10)))  # Normalized density at Cantor points
    return density

def _cdf(self, x):
    """Cumulative distribution function of the Cantor distribution."""
    if x < 0:
        return 0
    if x >= 1:
        return 1

    # Count contributions from Cantor set points
    cdf = 0
    for k in range(1, 11):
        if (0.5 ** k) <= x:
            cdf += (1 / 2) ** k
    return cdf

def _rvs(self, size=1):
    """Random variates of the Cantor distribution."""
    # Randomly select points from the Cantor set
    points = []
    for _ in range(size):
        num = np.random.rand()
        point = 0
        for k in range(10):  # Number of iterations
            if num < 1/3:
                point += 0  # Take the left segment
            elif num < 2/3:
                point += 0.5 ** (k + 1)  # Take the middle segment
            num = (num - 1/3) * 3  # Rescale for the next iteration
        points.append(point)
    return np.array(points)

def mean(self):
    """Mean of the Cantor distribution."""
    return 0.5

def variance(self):
    """Variance of the Cantor distribution."""
    return 1 / 12

Instantiate the Cantor distribution
cantor_dist = CantorDistribution(name='cantor')

Let me know if thats what you want as class:
def _pdf(self, x):
    """Probability density function of the Cantor distribution."""
    if (x < 0) or (x > 1):
        return 0

    # The Cantor distribution is defined on [0, 1]
    # Density is concentrated at points in the Cantor set
    density = 0
    if x in [0.5**k for k in range(1, 11)]:
        density = 1 / (np.log(2) * (2 ** (10)))  # Normalized density at Cantor points
    return density

def _cdf(self, x):
    """Cumulative distribution function of the Cantor distribution."""
    if x < 0:
        return 0
    if x >= 1:
        return 1

    # Count contributions from Cantor set points
    cdf = 0
    for k in range(1, 11):
        if (0.5 ** k) <= x:
            cdf += (1 / 2) ** k
    return cdf

def _rvs(self, size=1):
    """Random variates of the Cantor distribution."""
    # Randomly select points from the Cantor set
    points = []
    for _ in range(size):
        num = np.random.rand()
        point = 0
        for k in range(10):  # Number of iterations
            if num < 1/3:
                point += 0  # Take the left segment
            elif num < 2/3:
                point += 0.5 ** (k + 1)  # Take the middle segment
            num = (num - 1/3) * 3  # Rescale for the next iteration
        points.append(point)
    return np.array(points)

def mean(self):
    """Mean of the Cantor distribution."""
    return 0.5

def variance(self):
    """Variance of the Cantor distribution."""
    return 1 / 12

cantor_dist = CantorDistribution(name='cantor') # Instantiate the Cantor distribution
`

I'm -1 for adding this to SciPy. IMO, there are a lot of things which should be given priority over adding distributions which are only interesting as mathematical curiosities. There was talk about having a scikit-stats package that could contain third-party distribution implementations using SciPy's new infrastructure, and this could be a good fit there though.

Thanks for the interest @kumarpreetilata. I think your code illustrates the idea well, though there can be details to polish.
One particular point of interest is that your code intends to assign non-zero density to points where the CDF "jumps". I would assign the density to zero. What makes the choice interesting is that it's not an isolated case -- for example in gh-21700 the OP asks the question:

When there are discrete distributions in the mixture [with continuous distributions], I suppose the PDF at the support values of the discrete distributions will be infinite? And perhaps this class should also have a PMF method, and the PMF at values that are not in the support of a discrete distribution would be zero?

The choice might appear arbitrary (it could return infinity, zero, ValueError, nan, etc), but I think zero is a well-defined choice based on the fact that any probability distribution can be decomposed into the mixture of a continuous, a discrete, and a singular distribution. The PDF and PMF would correspond to the continuous and discrete component, respectively.
How is this related to the Cantor distribution in this thread? I'll elaborate in the next comment.

I agree with @steppi
It does not behave like any "usual" distribution, and I don't see any real life application for it.
It will only confuse users who try out distributions without checking the math, and if they do they will just ignore it.
(AFAIR, the only time I learned about Cantor set was as counter example to some probability or measure theory definitions.)
How would you define and compute the expectation of a function w.r.t. the Cantor distribution, or draw a random sample
AFAICS, @kumarpreetilata functions truncate the construction of cantor points. I doubt that that has any useful applications either.

Interesting points! I’d say those are the fun part and in fact quite enlightening.

It does not behave like any "usual" distribution, and I don't see any real life application for it. It will only confuse users who try out distributions without checking the math, and if they do they will just ignore it.

I agree there’s probably little practical use of the distribution. But I don’t think it would cause much confusion — the CDF plot of Cantor distribution looks just like any empirical distribution, so I don’t foresee anything special in using it. (Though there is clearly something special in writing it.)

(AFAIR, the only time I learned about Cantor set was as counter example to some probability or measure theory definitions.)

Quite the opposite — it is a counter example of heuristic reasoning about probability distributions. It is well treated within the framework of probability theory, and that’s why I propose to use it as a “test case” to verify the “completeness” of our new distribution framework — if the framework is compatible with Cantor (I think it is or at least almost is), we can be fairly confident that the framework will be able to handle any probability distribution to ever occur.

How would you define and compute the expectation of a function w.r.t. the Cantor distribution

The mathematical definition is standard and is the same as any other distribution:
$$
\mathbb{E}[g(X)] := \int_{[0,1]} g(x) \mathrm{d} \mu
$$
where $\mu$ is the probability measure and the integral is understood to be a Lebesgue integral.
How it can be evaluated numerically is not more difficult than quadrature, except that where one would plug in the density function for quadrature, one would instead plug in the “average” density (i.e. difference of CDF over  bucket length).

or draw a random sample

This is a very interesting question and there’s an answer for it on Cross Validated. The algorithm is surprisingly simple (and I think simpler than computing the CDF).

AFAICS, @kumarpreetilata functions truncate the construction of cantor points. I doubt that that has any useful applications either.

Since floating point numbers are rational approximations to real numbers, some truncation needs to happen anyway. (This is much like truncating the Taylor series to evaluate sin(x) numerically.) Using merely ten terms is probably not enough for accuracy, and those are the sort of numerical details to polish.

I agree there’s probably little practical use of the distribution.

Then it doesn't belong in SciPy.

See also the section deciding on new features in the core developer guide.

The method is applicable in many fields and “generally agreed” to be useful,

It fits the topic of the submodule, and does not require extensive support frameworks to operate,

The implementation looks sound and unlikely to need much tweaking in the future (e.g., limited expected maintenance burden),

Someone wants to contribute it, and

Someone wants to review it.

Thanks for your comments @steppi and @mdhaber.
My intent is less about adding the Cantor distribution itself but more about making sure that the new statistical distribution infrastructure has enough flexibility to support a wide range of custom distributions. Of course such “generality” needs to carefully defined — not too specific to exclude valid use cases, and not too generic to offer no aid to the user. (The old rv_generic framework is too “specific” in this sense since it’s not possible to implement mixed discrete-continuous distribution without hackish “monkey patching”.)
I really like the initiative of the new infrastructure and I’m confident it will enable new possibilities that make scipy.stats better. I took the liberty to investigate the “boundary” of the new infrastructure — what kind of distributions are possible to implement under the new framework, without hacking the framework itself? (It seems mixed discrete-continuous distributions are already under the radar of @mdhaber, which is great.)
After much research I arrived at the conclusion that the new infrastructure is really close to, if not yet, supporting arbitrary (univariate) probability distributions. I think this is an important point because it implies we will never have to invent a new new framework ever again due to the new framework lacking some feature that cannot be added in a backward compatible way.
How to ensure that generality is not compromised while defining/implementing methods of the new base class (say _ProbabilityDistribution)? I think the easiest way is to actually have a distribution that relies on the generality being available. If you don’t have a mixed discrete-continuous distribution, you won’t find out something doesn’t work for mixed discrete-continuous distribution.
An alternative to this generality is to state that the new infrastructure only supports mixture of discrete and continuous distributions. I think that will satisfy most practical applications, so is a viable choice. However, I’d still like to advocate for full generality because there is little extra coding effort to spend (if any at all) to go from mixed discrete-continuous to fully general, so why not? If the design is fully general, then it will never have to be revisited again. This probably saves resource in the long run.
I understand the above might be too “forward looking” and lacks traction probably because there’re not yet concrete examples to illustrate the loss of generality (apart from  mixed discrete-continuous distributions which are on the radar). It could have been more intuitively explained if concrete examples are gathered.
Therefore I’m closing this issue right now and we could possibly revisit when specific issues arise. Anyway I hope you could give a thought about singular distribution support when making specifications/implementations.
Thanks for your input!


Issue in downloaded data

Hi GJ,
I've been working on generating my databases and when i was running
../crabs --pairwise-global-alignment --input merged_filter_derep_10k.txt --amplicons P5_insilico_derep_4aln.txt --output P5_aligned.txt --forward GGCATCACCATACTACTAACAGACCG --reverse GGATTAGGATGTAGACTTCTGGGTG --percent-identity 0.9 --coverage 0.9
I ended up with the following error:
 |            Function | Retrieve amplicons without primer-binding regions |         Import data | ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:00 0:00:00 |  Transform to fasta | ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:00 0:00:00 |  Pairwise alignment | ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━   0% -:--:-- 0:00:01 |Parse alignment data | ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╸ 100% 0:00:01 0:00:00 Traceback (most recent call last):   File "/scale_wlg_nobackup/filesets/nobackup/XXXX/reference_database_creator/./crabs", line 847, in <module>     crabs()   File "/opt/nesi/CS400_centos7_bdw/Python/3.11.3-gimkl-2022a/lib/python3.11/site-packages/click/core.py", line 1130, in __call__     return self.main(*args, **kwargs)            ^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/home/wpea186/.local/Python-3.11-gimkl-2022a/lib/python3.11/site-packages/rich_click/rich_command.py", line 152, in main     rv = self.invoke(ctx)          ^^^^^^^^^^^^^^^^   File "/opt/nesi/CS400_centos7_bdw/Python/3.11.3-gimkl-2022a/lib/python3.11/site-packages/click/core.py", line 1404, in invoke     return ctx.invoke(self.callback, **ctx.params)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/opt/nesi/CS400_centos7_bdw/Python/3.11.3-gimkl-2022a/lib/python3.11/site-packages/click/core.py", line 760, in invoke     return __callback(*args, **kwargs)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/scale_wlg_nobackup/filesets/nobackup/XXXX/reference_database_creator/./crabs", line 640, in crabs     amplicon_fasta_dict = extract_alignment_results(console, columns, align_temp_path, amplicon_fasta_dict, include_all_start_positions_, coverage_, forward_, reverse_, raw_fasta_dict)                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/scale_wlg_nobackup/filesets/nobackup/XXXX/reference_database_creator/function/crabs_functions.py", line 1256, in extract_alignment_results     sequence = raw_fasta_dict[query]['sequence']                ~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^
I tracked it down to a line in my merged database which contained "Pending (#8627) Acanthurus albimento" is the first column.
I checked, and this seems to originate from the bold database where many sequences are formatted as follows:

NTKAT1115-20|Knipowitschia mermere|COI-5P|Pending (#9276)

I'm going to have a go at removing all text that contains "Pending" from the fasta file - so it reformats the names to something like "NTKAT1115-20|Knipowitschia mermere|COI-5P" - and will let you know how that works, but thought it might be helpful to let you know here too.
Incidentally, i'm doing alignments to a P5 primer set - so it makes sense why i've only encountered this error here thus far
Cheers,
William

Hello William,
Thanks for letting us know!
This error arises, because CRABS expects an accession number at the end of the header when the BOLD header contains 3 |. However, this Pending (#####) seems to be an exception. This in turn seems to run into issues when invoking VSEARCH during --pairwise-global-alignment, probably because of the presence of (). In the new version CRABS v 1.0.6, I've updated the BOLD parsing during --import. It will now check if the information is an actual accession number and provide a proper sequence ID if this is not the case.
You will need to re-import your original files that you downloaded from BOLD, but it should all work now :)
Please let me know if the issue persists.
Thanks,
Gert-Jan


Confirmation Before Delete

Clicking the delete icon should provide a confirmation before deleting the transaction.

Closing issues related to things that still need to be more formally designed.


Add note about Go and Spring SDKs

Considering that some 3rd-party tools such as Terraform use Go SDK and it's part of our doc in here and here. We should have a highlighted note mentioning that we don't support Go SDK, as it is built outside of our SDK team, same as Spring Framework. We have Microsoft Community support when customer can get help over GitHub.
Both of them are not part of our release cycle, so we don't have parity of features there....

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

ID: fcba3146-83d6-fea9-78ca-d75df31a7d8a
Version Independent ID: 34472edb-c5bf-d751-e5f0-70f7ab2a6c02
Content: Azure Key Vault developer's guide
Content Source: articles/key-vault/general/developers-guide.md
Service: key-vault
Sub-service: general
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@FabianGonzalez-MSFT
Thanks for your feedback! We will investigate and update as appropriate.

@FabianGonzalez-MSFT
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly

@msmbaldwin
Can you please check and add your comments on this doc update request as applicable.

Merged.
#please-close


Error in trying to import from Goodreads

I am getting this error:

This has been fixed in 0.1.0-beta.42 version

Yes. Tested with the unstable image. Thanks!


can we have a nerdctld(daemon process) to support a restful api for other client to manage container?

What is the problem you're trying to solve
nerdctl doesn't have api
Describe the solution you'd like
we  should have a nerdctld(daemon process) to support a restful api for other client to manage container
Additional context
No response

@afbjorklund has been experimenting this in his repo
https://github.com/afbjorklund/nerdctld


Documents the release process for Pyrsia

Description
Starts work on pyrsia/pyrsia #1159
This PR documents the release process so that we can use this as a reference and keep updating it as we release and add features.
PR Checklist

[x] I've read the contributing guidelines.
[x] I've read "What is a Good PR?"
[x] I've included a good title and brief description along with how to review them.
[x] I've linked any associated an issue.

@erwin1 Once this is complete - please link to your test plan doc here.

@erwin1 I updated based on your comments and addressed the overall concern in your comments instead of taking them as it is. I added some headers to clarify what we are doing at each step. Also I kept tagging as the activity of just putting the tag. Other steps I collected under a different header.

I also addressed @fishseabowl and @ihcomega56 comments. So much good feedback.


fix typo of healthcheck #14985

fix type of healtcheck to healthcheck.

/assign @sftim

hello, @icheikhrouhou. Thanks for this PR. I think a good portion of these files are generated. The updates may need to be made on the kubeadm source files, not the generated output files.

@kbhawkey thanks, I'm still new, I dont know much about these, Can you please explain to me how these files are generated ?

@icheikhrouhou, it would be good to open a new issue to address the typos.
Checking with @neolit123.

hello, thanks for finding these typos.
they should be fixed in the kubernetes/kubernetes reporsitory.
you can search for the word with the typo healtcheck under /cmd/kubeadm/app and send a PR for that same repository.
please, make sure you mention me on the new PR.
thank you.

@neolit123 @kbhawkey thanks for the help,
I will do that.


Add Section component to Lexicon as a first-class element

I've added the Section component as requested in LEXI-1101.  in order to improve its findability for designers, and the development team.

Discarding as it is duplicated

Discarding as it is duplicated


Added support for httpd auth-api service for containers.

Enhanced fetching user attributes via dbus to use the httpd auth-api service for containers.
Enhanced fetching user groups via dbus to use the httpd auth-api service for containers.
Updated rspecs

Related PRs:

ManageIQ/manageiq-pods PR# 204
ManageIQ/container-httpd PR# 15
ManageIQ/manageiq PR# 15881

WIP - do not merge.
TODO:

tire kicking with latest containers
more specs

I have no idea what the lib/services directory is supposed to be used for. Is that the best place for this? @Fryguy ?

If it's going into services, it should be namespaced properly for autoload, so it should be Services::AuthApiService, which I don't like.  I think it should just be in the lib directory directly.

With it not being in lib/services, maybe it should also be renamed to HttpdAuthApi. Thoughts ?

I'm good with HttpdAuthApi


Redundant code

The code in def resume_contest_remotely seems to match that of def run_contest_remotely.
We need to integrate them, too messy and error prone to have both!

@nirlipo this is the code that I found almost copies, I think the resume is a copy of the run. Is that correct?

Mostly addressed in 312bbd6 as part of #69. There is still a little bit of duplicated control logic but it would take a lot more effort to clean that up without making a bigger mess. I propose we remove the critical label since the worst is fixed, but keep the issue around as a reminder that run_contest_jobs and resume_contest_jobs should be further unified.

OK good suggestion. Done.


RSMP version selected during negotiation and version format

During RSMP version negotiation,
for the equipment instead of sending all its RSMP versions "supported", could we limit to only some versions "wanted" ?
And when the supervision (after equipment) has also sended it's RSMP version supported list, the latest "common" version is implicitely selected.
Perhaps the supervision instead of sending all its RSMP versions supported, could only send in return the normally latest version. To see in the supervision reply string the version could be intesting to confirm the version really selected and used.

About the fomat of version string :
The version format could be more precisely and strictly defined as :
"MAJOR.MINOR.REVIEW."
with REVISION part that can be optionnal.
It could perhaps in the software help to use some numeric calculations to determine common latest instead of using string lists order.
And as suggested in issue #100 we could perhaps ignore REVIEW part if no codes modifications needed for a specification review.

What are the use-cases for equipment that want to use an older version, rather than the latest it supports? I can think of some debugging situations, but in these cases I think it would be better to control it from the supervisor side?
But I think we could think of changing the handshake so the supervisor explicitly respond with the version to use.
It could also make sense to negioate the SXL version, like we core version, see #65.

Is the intention to be able to choose a version that's not the latest? Or is it to be more explicit about what version is actually picked?
Note: According to the semver specification, versions have the format MAJOR.MINOR.PATCH, see https://semver.org. So it's called a patch version, not review or revision.

At least, it would be more interesting if supervision only responded with the version it had selected for future exchanges with the equipment.


Social Icons in footer section of dark theme

I have updated the social icons in the footer section of the dark theme.

I have updated the social icons in the footer section of the dark theme.

Could you please render the same changes to the footer social media icons for the light mode


Quotas

Brick-level xfs project quotas don't interact well with heals when users are up against their quota limit. We sometimes end up in situations where there are pending heals that never complete because the affected subtree is at its quota limit. This change:

Disables quota enforcement while servers are being upgraded
Adds playbooks to enable/disable enforcement for a set of servers

FYI @ShyamsundarR


Solve basic programming model for entities

Changes

Define how the entity is instantiated
include the Programming Model for Entity Data in the Jakarta Data session
Define the basic type
Define Domain-Relation
Define recursion

solves https://github.com/jakartaee/data/issues/246
⚠ I hate the "Domain-Relation" to define the cardinality among entities, but I did not find another best name.

@njr-11, please, let me know what you think


Rename grpc proto file

Changed the name of the grpc proto file to formicd instead of oortd

There were conflicts with the other changes that I made, so I just made the change manually.