Looking for a way to automatically test the built bundle

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for an issue that matches the one I want to file, without success.

Please describe your issue:
Looking for a way to run some sanity tests on the built bundle produced by electron-packager. Some sort of automated test that I could run on CI to ensure that the bundle built at least pulls in the correct files, etc.
I haven't been able to find any way to automate this however. The current process is the manually check the bundle before committing and pushing any changes to master. This is cumbersome and misses some edge cases on occasions. To this end, it'd be nice if I could run some system or E2E tests on the bundle.

Console output when you run electron-packager with the environment variable DEBUG=electron-packager. Please include the stack trace if one exists.
Not applicable
What command line arguments are you passing? Alternatively, if you are using the API, what
parameters are you passing to the packager() function?
Not applicable
Please provide either a failing minimal testcase (with a link to the code) or detailed steps to
reproduce your problem. Using electron-quick-start
is a good starting point.
Not applicable

The best advice I can give you is to adapt some of Electron Packager's own tests into your testsuite, since it checks that various files exist and have certain state.


When we cancel installation of P2P, CC shows not proper message.

When we cancel installation of P2P via CC on Mac, CC shows: "The P2P Daemon has been installed and will start running in a few seconds".  But actually P2P was not installed.
How to reproduce:

Click Install P2P via CC.
Proceed it.
When OS will ask to type your password to allow this, cancel.

Fixed in Master.


Manual triggers for CI and CD

This PR includes two manual trigger workflows for CI and CD, respectively.
For CI_manual, we include a Python version matrix for versions 3.9 and 3.10 compatible with the latest release of WF-PSF. Python versions 3.11 and 3.12 requires upgrades to WF-PSF TensorFlow and TensorFlow Add-on packages.
For CD.yml and CD_manual.yml, we also bumped up the Python version to 3.10.11.
Due to CI failures with pytest==8.1.x (https://github.com/CosmoStat/wf-psf/issues/125) and deprecated pytest-plugins (https://github.com/CosmoStat/wf-psf/issues/118), I locked `pytest>=7.0.0,<8.1' and removed deprecated pytest-plugins.
Solves #118 and #125.
Ran CI and CD tests in forked repo: https://github.com/jeipollack/wf-psf/actions

@sfarrens I am ready for you to review this PR.

@sfarrens ready for you to re-review the latest changes.

Hi Sam, I was wondering if we could redirect this PR to main branch instead of develop in order to make immediate use of the manual trigger workflows. let me know what you think.

This PR can be merged now


update Dev#B2 copy

 [QAE18IMP] Review and update copy for Sustainable Development award (6)

Good job :indeed-parrot:


Some improvements to pythonlib

This is in view of #3928, specifically some essential things required for the first task, example/pythonlib/basic/.
You can look at commits individually if you like. The gist of this PR is to improve the scaffolding we currently have to unblock the previously mentioned tasks. It can be summarized by the following points (roughly each corresponding to one commit):

[x] make run task the default, following the same convention used in ScalaModule and JavaModule
[x] add a command to run an interactive REPL. This is named console, again following the conventions of scalalib
[x] rework the way source files are handled:

support multiple source directories, again following similar conventions of scalalib
instead of aggreting python scripts in one syntetic directory, keep them where they are and instead manipulate PYTHONPATH or pass in parameters to various directories where appropriate

[x] define a module for unit tests, similar to the TestModule of scalalib

@jodersky TBH this is what i was looking for, Great !!!
Also i want to add something like the way i used Repl i think that is also good See #3986
And the way of testing is now professional and matching with other examples (i didn't liked my way of testing)
So i think i should wait till this PR gets Merged and after wards i will work further on #3986 till then let me copy your changes and work around with the same pattern
Thanks...

@jodersky ya that's great we will work together to bring the professional python first class support to mill
Team Work is the Dream Work !!!

./mill mill.scalalib.scalafmt.ScalafmtModule/
./mill mill.javalib.palantirformat.PalantirFormatModule/
./mill mill.kotlinlib.ktlint.KtlintModule/
@jodersky  these commands will fix the lint error from the root

Alright, I think this is ready for review now.

Looks pretty good overall. Left some comments. @jodersky you mentioned you looked at how Bazel handles Python, if in that research you found anything relevant to this PR can summarize whatever you learned that's relevant in the PR description

@jodersky i think we should work on linting before deps coz deps don't require much logical and backend code but linting is far harder and more complicated and in that case, experience is more required 😉
Rest is You's Choice...

@jodersky i think we should work on linting before deps coz deps don't require much logical and backend code but linting is far harder and more complicated and in that case, experience is more required 😉

I'm not sure. I think dependencies are a very important feature, and they aren't all that easy when you consider that we'll need more than just building a pip command line.

@jodersky i think we should work on linting before deps coz deps don't require much logical and backend code but linting is far harder and more complicated and in that case, experience is more required 😉

I'm not sure. I think dependencies are a very important feature, and they aren't all that easy when you consider that we'll need more than just building a pip command line.
For example, we'll need to deal with things like installing wheels. Also, there are a couple of "quality-of-life" features that I'd like to include:

ability to specifcy dependencies via an existing requirements.txt file (this is inspired by pants' python support)
on dependency conflicts, mention which modules requested the conflicting version (right now, since we're only building a pip command line, it's really hard to find out what module is the culprit)

Ok that's great afterall this all wil be available for mill 😊

@lihaoyi, I think I addressed all your comments. Would you like to go over it once more? Regarding the python example, it will all be reworked by @himanshumahajan138.

@jodersky looks great, I'll merge it and we can continue iterating on it after


Backlog 34960

Adding Azure Sql Database, SQL data sources types to PDI.
We have not added support for Azure Active Directory - Integrated authentication mode as we have an open issue going on with Microsoft regarding that.

:white_check_mark: Build finished in 43m 3s
Build command:
mvn clean verify -B -e -Daudit -amd

:ok_hand: All tests passed!
Tests run: 5065, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 23m 40s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,dbdialog

:ok_hand: All tests passed!
Tests run: 5053, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 23m 9s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,dbdialog

:ok_hand: All tests passed!
Tests run: 5053, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

:warning: Build finished in 24m 44s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,dbdialog

:no_entry: Failed Tests
:cloud_with_lightning_and_rain: 1 test(s) failed:
org.pentaho.di.trans.step.mqtt.MQTTStreamSourceTest.testMqttStreamSingleTopic (click to expand)
${result.errorDetails}

Tests run: 5053, Failures: 1, Skipped: 7    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 25m 12s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,dbdialog

:ok_hand: All tests passed!
Tests run: 5061, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

:warning: Build finished in 24m 1s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,dbdialog

:no_entry: Failed Tests
:cloud_with_lightning_and_rain: 1 test(s) failed:
org.pentaho.di.trans.step.mqtt.MQTTStreamSourceTest.testMqttStreamSingleTopic (click to expand)
${result.errorDetails}

Tests run: 5061, Failures: 1, Skipped: 7    Test Results

:information_source:  This is an automatic message

:warning: Build finished in 23m 46s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,dbdialog

:no_entry: Failed Tests
:cloud_with_lightning_and_rain: 18 test(s) failed:
org.pentaho.di.trans.StepWithMappingMetaTest.loadMappingMeta (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.trans.StepWithMappingMetaTest.loadMappingMetaTest (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.trans.StepWithMappingMetaTest.testFileNameAsVariable (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.trans.TransMetaTest.testLoadXml (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.trans.TransMetaTest.testSetInternalEntryCurrentDirectoryWithRepository (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.trans.TransMetaTest.testTransWithOneStepIsConsideredUsed (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.trans.TransTest.testLoggingObjectIsNotLeakInTrans (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.www.ExecuteJobServletTest.testExecuteJobServletTest (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.www.ExecuteJobServletTest.testExecuteJobServletTestCantFindDirectory (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.www.ExecuteJobServletTest.testExecuteJobServletTestJobNotFoundInDirectory (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.www.ExecuteJobServletTest.testExecuteJobServletTestCantFindRepository (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.www.ExecuteJobServletTest.testExecuteJobServletTestWithUnauthorizedUser (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.www.ExecuteTransServletTest.doGetRepositoryAuthenticationFailTest (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.www.ExecuteTransServletTest.doGetTransformationNotFoundTest (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.www.ExecuteTransServletTest.doGetTransformationFailsTest (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.www.ExecuteTransServletTest.doGetTransformationSucceedsTest (click to expand)
javassist.CannotCompileException: by java.util.zip.ZipException: invalid distance too far back

org.pentaho.di.trans.steps.jsoninput.analyzer.JsonInputAnalyzerTest.testJsonInputExternalResourceConsumer (click to expand)
org/pentaho/platform/engine/core/system/PentahoSystem

org.pentaho.di.trans.steps.jsonoutput.analyzer.JsonOutputAnalyzerTest.testJsonOutputExternalResourceConsumer (click to expand)
Could not initialize class org.pentaho.metaverse.api.analyzer.kettle.KettleAnalyzerUtil

Tests run: 5061, Failures: 18, Skipped: 7    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 23m 10s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,dbdialog

:ok_hand: All tests passed!
Tests run: 5061, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message


Group size and restrictions: documentation and implementation contradict each other

According to readme:
group_size (int): no restrictions as long as weight.numel() is divisible by the group_size however group_size should be divisible by 8
(Zamba2 1.2B has torch.Size([8512, 2048]) linears. With readme in mind, I intentionally tried to make group size = 38 (64 was to imprecise, 32 didn't decrease memory too much).
So readme probably should has this restriction for divisibility or there should be no assert in code.

The readme is correct,

Then assert is incorrect?
As if "no restrictions as long as weight.numel() is divisible by the group_size"  is correct, I must be able to use group size 38 for [8512, 2048]: 17432576 / 38 = 458752

Why would you like to quantize a 1B model?

Proof of concept to see what parts can be quantized making model smaller without making model too bad.
1B  loads  instantly.  7B takes quite a while, barely fits in 16GB VRAM, and takes time to run compared to 1B.

Using a group-size of 38 is an edge-case which is incompatible with the fast backends. As a rule of thumb, all matrix/vector shapes should be ideally powers of 2, so try to choose from 32, 64, 128, etc. I can change the readme to only include powers of 2 / out_features size + mention the fast backends to avoid any confusion :+1:
7B like a Llama2-7B should take ~5GB of VRAM and should run at 170 tokens/sec on a 4090. Would you mind sharing your code to quantize and run? I can assist you to run properly quantize it and run it fast, here's an example: https://github.com/mobiusml/hqq/blob/master/examples/backends/hqq_lib_demo.py

to only include powers of 2 / out_features size + mention the fast backends to avoid any confusion 👍

Yeah, that would help avoid confusion

Would you mind sharing your code to quantize and run?

It actually would be helpful.
In general non transformers documentation would be helpful as it's where hqq can shine the most. Where exllama and gguf can't be used easily if it all, with HQQ it's just possible to replace nn.Linears after loading the model without finetuning it.
I'm targeting  zamba2 adapted to HF transformers instead of their fork. Zamba2 7B version  has 81 layers of mamba2 and mamba2 has two linears that make majority of weights.
Also the  culprit of the slowness was found: most backends(aten, torch, torch_compile).
At least on my laptops's 3080 with 16GB vram only torchao_int4 runs as fast as bf16. On 7B model to generate ~1K tokens ATEN takes ~5 minutes. AO4 and BF16 both take around a minute.
I didn't test backends pytorch/pytorch_compile on 7B, but on 1.2B they already were too slow, to the point if in each mamba block instead of quantizing in_proj and out_proj, I quantizied just in_proj, the speed doubled.
Also I didn't see from hqq.utils.generation_hf import HFGenerator before that hqq_lib_demo, but it seems it's tied to usual architecture: Zamba uses very own custom cache, so when I use HFGenerator/  static cache doesn't work.

HQQLinear.set_backend will not speed-up inference much, that's the backend for the dequantization op, mainly used as a fallback solution for inference, but mostly useful for training.
What you want is the backend for faster inference, which is enabled via  prepare_for_inference(model, backend=infer_backend, verbose=True) . Valid infer_backend here would be torchao_int4 or bitblas, on top of that you need torch.compile with static cache via HFGenerator, otherwise you'll not see that 3-4 speed-up.
As you said, HFGenerator is for transformers so it wouldn't work. Instead try to compile the forward pass of the model:
model.forward = torch.compile(model.forward, mode="reduce-overhead", fullgraph=True )

You'll have to run it for a couple of iterations to warm-up. I saw there are some people trying to refactor Mamba for torch.compile, so maybe that work would be useful.


Turning hwid to variable outside gethwid function

Im pretty stupid but how would i turn the id into a variable that i can use outside the gethwid function

I'm not sure I understand the question

im not familiar with anonymous functions and have the same question.
getHWID().then(id => {
// use ID however you want
})
im having a hard time using 'id' outside of this anonymous functions scope, id like to return it or assign it something I can use elsewhere in the program. thanks.

getHWID().then(id => {
// use ID however you want
console.log('your hardware ID is:', id)
})
or
const otherFunction =(hwid)=>{
// what ever you want to do with the id
}
getHWID().then(id => {
otherFunction(id)
})

thanks for the tip, appreciate it!

getHWID() runs asynchronously and as such returns a Promise. You can use this promise directly or you can use it as part of async/await code.


Add support for Irish

KeyboardKit should have support for Irish.

This can be tested in the master branch once the demo uses the new version.


chore(dep-bump): upgraded ava to 4.0.0

:tada: This PR is included in version 3.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


build(project): add build dependency to chain-init

Self explanatory. Fixes make clean chain-init call.

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


HTML strings rejected

Dot allows html rather than text labels, delimited by angle brackets, e.g:
digraph D {

  node [shape=plaintext]

  FOO [ label = <The foo,
 the bar and
 the baz> ];

  FOO -> {BAR, BAZ};

}

HTML node label.dot.zip

This would be tricky to add and maintain backwards compatibility, we need to know whether the string was a normal string/"quoted string" or an .
Idea 1. Store the html string with the normal strings but have a method on Edge isHtml(string) that checks if the string was an html string, stored, possibly if F# supports it, a new HashMap(ReferenceEqualityComparer.Instance) where the string is matched by reference equals, checking it is the same string, rather than a string with the same value.
Idea 2. (not backwards compatible) store a DotString or a HtmlString both derived from DotValue so the reader can check which type of string it is. HtmlString could have a method getString() to extract just the text and convert  to '\n'.

One can not derived a class from string as .net string is sealed.


Update Katas to 0.25.2207 release

The set of files updated looks good. I assume you'll be updating it to the release version instead of the beta next?

The set of files updated looks good. I assume you'll be updating it to the release version instead of the beta next?

Thanks. Yes, now the version has been updated to the released one.


New Publisher Function Request on Page 193 OnAfterLoadDataFromRecord

Please can we have a new publisher function as above that has the 'MainRecordRef' as a parameter, and is called at the end of the "LoadDataFromRecord" function? That will allow developers to subscribe to the Publisher function and access the 'MainRecordRef' variable from the subscriber function.
I need this as I have written a 'Drag and Drop' ControlAddIn extension to select files using Drag and Drop (instead of the File Dialogue), and need to be able to dynamically access the record to which the attachments needs to be attached.
Much appreciated!
Clinton

You should post EventRequests in this Repo: https://github.com/Microsoft/ALAppExtensions/issues

Thank you for the response and advice. I have posted the request there.

Closing this issue as it is related to ALAppExtensions


ATRIA score migrated

Changes:
(1) Event time, outp->inp
(2) CurrentDateTime without value
(3) I guess the age calculations were not completely correct, in some cases two rules could fire for example. I corrected this in a new file, 2 test cases show this problem around the end of the test files.

Atria assessment added


Example code is not correctly indented

https://mido.readthedocs.io/en/stable/meta_message_types.html#implementing-new-or-custom-meta-messages

Thanks for reporting. It's a silly mistake from when I reorganized the docs. My bad. Fix incoming.


fs.rename not allowed when allowing fs:scope

What I've done

made Tauri v2(beta) project
npm run tauri add fs
added src-tauri/capabilities/main.json

{
  "$schema": "../gen/schemas/desktop-schema.json",
  "identifier": "main-capability",
  "description": "Capability for the main window",
  "windows": ["main"],
  "permissions": [{"identifier": "fs:scope", "allow": [{"path": "$DOCUMENT/"}, {"path": "$DOCUMENT/**"}]}]
}

wrote typescript code using rename

import { BaseDirectory, rename } from "@tauri-apps/plugin-fs";

try {
  await rename("sample.txt", "renamed.txt", {
    oldPathBaseDir: BaseDirectory.Document,
    newPathBaseDir: BaseDirectory.Document
  });
} catch(e) {
  console.log(e);
}

placed sample.txt (target to rename) on document directory
npm run tauri dev

expected behavior
"sample.txt" is renamed to "renamed.txt"
actual behavior
An error occurs. error message says "fs.rename not allowed." but permissions in the message include "fs:allow-rename".
fs.rename not allowed. Permissions associated with this command: fs:allow-app-write, fs:allow-app-write-recursive, fs:allow-appcache-write, fs:allow-appcache-write-recursive, fs:allow-appconfig-write, fs:allow-appconfig-write-recursive, fs:allow-appdata-write, fs:allow-appdata-write-recursive, fs:allow-applocaldata-write, fs:allow-applocaldata-write-recursive, fs:allow-applog-write, fs:allow-applog-write-recursive, fs:allow-audio-write, fs:allow-audio-write-recursive, fs:allow-cache-write, fs:allow-cache-write-recursive, fs:allow-config-write, fs:allow-config-write-recursive, fs:allow-data-write, fs:allow-data-write-recursive, fs:allow-desktop-write, fs:allow-desktop-write-recursive, fs:allow-document-write, fs:allow-document-write-recursive, fs:allow-download-write, fs:allow-download-write-recursive, fs:allow-exe-write, fs:allow-exe-write-recursive, fs:allow-font-write, fs:allow-font-write-recursive, fs:allow-home-write, fs:allow-home-write-recursive, fs:allow-localdata-write, fs:allow-localdata-write-recursive, fs:allow-log-write, fs:allow-log-write-recursive, fs:allow-picture-write, fs:allow-picture-write-recursive, fs:allow-public-write, fs:allow-public-write-recursive, fs:allow-rename, fs:allow-resource-write, fs:allow-resource-write-recursive, fs:allow-runtime-write, fs:allow-runtime-write-recursive, fs:allow-temp-write, fs:allow-temp-write-recursive, fs:allow-template-write, fs:allow-template-write-recursive, fs:allow-video-write, fs:allow-video-write-recursive, fs:write-all, fs:write-files

fs:scope only modifies the scope, i.e which files you're allowed to access, you still need to enable the operation itself, so you need to add fs:allow-rename in your permissions list.

adding "identifier": "fs:allow-rename" made rename done.
But in my project, other operations (readDir, readTextFile, exists, readFile) works without specific allowing. This seems to be inconsistent.

you probably have fs:default in you capability, which enables
    "read_dir",
    "read_file",
    "read_text_file",
    "read_text_file_lines",
    "read_text_file_lines_next",
    "exists",


cuda_array_interface not working as expected with Pytorch tensors

[x] I am using the latest released version of Numba (most recent is visible in
the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[x] I have included below a minimal working reproducer

Not sure if this is a stupid issue, but I had hoped that properly adding an ____cuda_array_interface__ dict to a Pytorch Tensor would allow me to convert between numpy, pytorch and numba arrays. However, this is not true.
import unittest
import numpy as np
np.random.seed(0)
import torch
import numba.cuda as cuda

def typestr(tensor):
    import sys
    endianness = '<' if sys.byteorder == 'little' else '>'
    types = {
        torch.float32: 'f4',
        torch.float: 'f4',
        torch.float64: 'f8',
        torch.double: 'f8',
        torch.float16: 'f2',
        torch.half: 'f2',
        torch.uint8: 'u1',
        torch.int8: 'i1',
        torch.int16: 'i2',
        torch.short: 'i2',
        torch.int32: 'i4',
        torch.int: 'i4',
        torch.int64: 'i8',
        torch.long: 'i8'
    }
    return endianness + types[tensor.dtype]

class TestPTNumba(unittest.TestCase):

    def test_conversion(self):
        array = np.random.rand(5, 5)
        tensor = torch.tensor(array).cuda()
        cai_dict = {
            'shape': tuple(tensor.shape),
            'data': (tensor.data_ptr(), True),
            'typestr': typestr(tensor),
            'version': 0,
            'strides': tensor.stride(),
            'descr': [('', typestr(tensor))]
        }
        setattr(tensor, '__cuda_array_interface__', cai_dict)
        device_array = cuda.as_cuda_array(tensor)
        self.assertTrue(np.allclose(device_array.copy_to_host(None), array))

if __name__ == '__main__':
    unittest.main()

It can be seen from inspecting the device_array that some data is properly accessible, with a lot of zeros in between or wrong ordering. Is this expected behaviour?

Turns out tensor.strides() is not in bytes but in unit of element_size.

This might be a mistake on our part in trying to copy the spec.  @sklam: can you check this?

OK, confirmed that the stride should be in units of bytes.  Sorry about the false alarm.  Closing...


Ubuntu snap package for Telestion Core

Closing this as it is based on potentially outdated assumptions. Please feel free to re-open (with an updated description) based on the current structure.
Considerations for closing this

inactivity with no clear use-case definition
no documented advantages over docker based distribution


Implement AutoAuth transaction handling

What would you like to be added:
In order to perform a Dftx, an auth input is required (A UTXO specifically for the account that requires authorization). The full node currently searches for auth inputs and if none are found it creates a AutoAuth tx, which is a regular UTXO send with dust outputs. The C++ implementation is here:
 // at last, create additional tx for missed
    if (!notFoundYet.empty()) {
        try {
            optAuthTx = CreateAuthTx(pwallet, notFoundYet, txVersion); // success or throw
        } catch (const UniValue& objError) {
            throw JSONRPCError(objError["code"].get_int(), "Add-on auth TX failed: " + objError["message"].getValStr());
        }
        // if we are here - we've got signed optional auth tx - add all of the outputs into inputs (to do not miss any coins for sure)
        for (size_t i = 0; i < optAuthTx->vout.size(); ++i) {
            if (!optAuthTx->vout[i].scriptPubKey.IsUnspendable())
                result.push_back(CTxIn(optAuthTx->GetHash(), i));
        }
    }

Why is this needed:
It's needed otherwise the sending account might not have a free UTXO to use as Auth Input for that specific account that is triggering the action.

This issue might be sidestepped by sending the change outputs in all Dftx back to the address requiring authorization (Which should in most cases be the same address paying the fee, otherwise the original Dftx will also fail).

According to further investigation, it is possible to use the last Dftx unconfirmed output as in input for the next Dftx, allowing you to chain the auth inputs until you theoretically run out of fees.
Ideal shortterm solution here would be for jellyfish-transaction-builder to cache unconfirmed outputs and select them if there are no other outputs available, this should work for most cases, although a custom Auth tx will be still be required sometimes (i.e. if the UTXOs were used up in some unrelated scenario).

Closing, implementation is prevout chain agnostic, it's up to the provider to implement this


Update Min/Max logic

✅ No public API change.

10

How are we still passing this test?

Refers to: src/tests/Microsoft.PowerFx.Core.Tests/ExpressionTestCases/MinMax.txt:178 in ea01cbf. [](commit_id = ea01cbf6c313dacbcfa61e5d8803536e48aaf76d, deletion_comment = False)

2

We still need to check these tests.

Refers to: src/tests/Microsoft.PowerFx.Core.Tests/ExpressionTestCases/MinMax.txt:142 in ea01cbf. [](commit_id = ea01cbf6c313dacbcfa61e5d8803536e48aaf76d, deletion_comment = False)

✅ No public API change.

✅ No public API change.

✅ No public API change.


It would be convenient if there was a TODO light

@EugeneDar can you give an example please/

@yegor256 the usual comments in the code are not highlighted in any way, but most development tools for most languages highlight the word TODO. This helps to make a noticeable comment in the code.
An example of what this might look like

:

@yasamprom this does make sense, let's implement this; please help


ci: use reusable workflow

:tada: This PR is included in version 1.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


using firefox instead of chrome during dev builds

Hi, during development i use pnpm dev command which opens up a new chrome instance for testing the extension. How would i configure it to open up firefox instead of chrome? i tried searching through the docs but couldn't find any thing on changing the browser. any help would be appreciated

The plugin accepts a browser option:
https://github.com/aklinker1/vite-plugin-web-extension/blob/9343f14f122b8a0a0b89176af0f83c8936177d67/packages/vite-plugin-web-extension/src/options.ts#L52-L57
I could have sworn this was documented somewhere... but I guess not? I couldn't find it in the docs either.
// vite.config.ts
export default defineConfig({
  plugins: [
    WebExtension({
      browser: process.env.TARGET_BROWSER,
    }),
  ],
});

Then run:
$ TARGET_BROWSER=firefox vite

// vite.config.ts
export default defineConfig({
  plugins: [
    WebExtension({
      browser: process.env.TARGET_BROWSER,
    }),
  ],
});

that really helps. thanks a lot


Not working for me

When I try to start the service, nothing seems to happen (I guess that's normal).
When I run systemctl status<EMAIL_ADDRESS>I get this:
<EMAIL_ADDRESS>Unit entered failed state.
<EMAIL_ADDRESS>Failed with result 'exit-code

Do you have any idea what I could be doing wrong?

Which Minecraft Server Version do you use?
Please poste the output of these commands:

systemctl status<EMAIL_ADDRESS>which java
java -version
which screen
screen -version
tree -L 3 /opt/
tail -n 20 /opt/minecraft/vanilla/logs/latest.log

I got the same issue.
Herre is the output of the commands:
systemctl status
<EMAIL_ADDRESS>- Minecraft Server vanilla
   Loaded: loaded<EMAIL_ADDRESS>enabled; vendor preset: enabled)
   Active: activating (auto-restart) (Result: exit-code) since Thu 2017-07-27 11:30:04 CEST; 27s ago
  Process: 8287 ExecStop=/usr/bin/screen -p 0 -S mc-%i -X eval stuff "say SERVER SHUTTING DOWN. Saving map..."\015 (code=exited, status=1/FAILURE)
  Process: 8276 ExecStart=/bin/sh -c /usr/bin/screen -DmS mc-%i /usr/bin/java -server -Xms512M -Xmx2048M -XX:+UseG1GC -XX:+CMSIncrementalPacing -XX:+CMSClassUnloadingEnabled -XX:ParallelGCThreads=2 -XX:MinH
 Main PID: 8276 (code=exited, status=0/SUCCESS)

Jul 27 11:30:04 XXXXX.XX.net systemd[1]<EMAIL_ADDRESS>Control process exited, code=exited status=1
Jul 27 11:30:04 XXXXX.XX.net systemd[1]<EMAIL_ADDRESS>Unit entered failed state.
Jul 27 11:30:04 XXXXX.XX.net systemd[1]<EMAIL_ADDRESS>Failed with result 'exit-code'.

which java
/usr/bin/java
java -version
openjdk version "9-internal"
OpenJDK Runtime Environment (build 9-internal+0-2016-04-14-195246.buildd.src)
OpenJDK 64-Bit Server VM (build 9-internal+0-2016-04-14-195246.buildd.src, mixed mode)

which screen
/usr/bin/screen
screen -version
Screen version 4.03.01 (GNU) 28-Jun-15
tree -L 3 /opt/
/opt/
`-- minecraft
    `-- vanilla
        |-- banned-ips.json
        |-- banned-players.json
        |-- eula.txt
        |-- logs
        |-- minecraft_server.1.12.jar.bak
        |-- minecraft_server.jar
        |-- ops.json
        |-- server.properties
        |-- usercache.json
        |-- whitelist.json
        `-- world

tail -n 20 /opt/minecraft/vanilla/logs/latest.log
This is from the server being run with the command line before using the systemdunit
[20:12:31] [Server thread/INFO]: Preparing spawn area: 12%
[20:12:32] [Server thread/INFO]: Preparing spawn area: 15%
[20:12:33] [Server thread/INFO]: Preparing spawn area: 19%
[20:12:34] [Server thread/INFO]: Preparing spawn area: 24%
[20:12:35] [Server thread/INFO]: Preparing spawn area: 30%
[20:12:36] [Server thread/INFO]: Preparing spawn area: 36%
[20:12:37] [Server thread/INFO]: Preparing spawn area: 42%
[20:12:38] [Server thread/INFO]: Preparing spawn area: 49%
[20:12:39] [Server thread/INFO]: Preparing spawn area: 58%
[20:12:40] [Server thread/INFO]: Preparing spawn area: 68%
[20:12:41] [Server thread/INFO]: Preparing spawn area: 77%
[20:12:42] [Server thread/INFO]: Preparing spawn area: 87%
[20:12:43] [Server thread/INFO]: Preparing spawn area: 99%
[20:12:43] [Server thread/INFO]: Done (16.861s)! For help, type "help" or "?"
[20:14:23] [User Authenticator #1/INFO]: UUID of player DerDegu is 9c88bd7c-a767-4399-b93f-095dc2c161d6
[20:14:23] [Server thread/INFO]: $name[/XXX.XXX.XXX.XXX:63735] logged in with entity id 2452 at (-116.5, 67.0, 254.5)
[20:14:23] [Server thread/INFO]: $name joined the game
[20:14:26] [Server thread/WARN]: Can't keep up! Did the system time change, or is the server overloaded? Running 2088ms behind, skipping 41 tick(s)
[20:26:26] [Server thread/INFO]: $name lost connection: Disconnected
[20:26:26] [Server thread/INFO]: $name left the game

syslog
Jul 27 11:42:07 vmdXXXX systemd[1]<EMAIL_ADDRESS>Service hold-off time over, scheduling restart.
Jul 27 11:42:07 vmdXXXX systemd[1]: Stopped Minecraft Server vanilla.
Jul 27 11:42:07 vmdXXXX systemd[1]: Started Minecraft Server vanilla.
Jul 27 11:42:07 vmdXXXX screen[8555]: No screen session found.
Jul 27 11:42:07 vmdXXXX systemd[1]<EMAIL_ADDRESS>Control process exited, code=exited status=1
Jul 27 11:42:07 vmdXXXX systemd[1]<EMAIL_ADDRESS>Unit entered failed state.
Jul 27 11:42:07 vmdXXXX systemd[1]<EMAIL_ADDRESS>Failed with result 'exit-code'.

Might the problem be resolved if I used openjd-8? or is it screen related?

@TheSquidly can you please add " -L -Logfile /tmp/screen.log" to the unit file right after "screen" in line 19.
After that run "systemctl daemon-reload && systemctl start<EMAIL_ADDRESS>That will generate a capture of everything happening within the screen session. Either java or screen aren't behaving right for you. Also I want to note, that openjdk 9 is a pre-release, the final version will be released next month.
Please also post the output of "ls -la /opt/minecraft/vanialla".


Supersede shlab.imgct in favor of class-prefixed itrackvalr

Supersede shlab.imgct in favor of class-prefixed itrackvalr
Problem
We are developing multiple packages when we should instead consolidate into one package. At this point, this means superseding shlab.imgct into itrackvalr. How can we do this, and follow best practices for R packages and target audience usability?
Solution
I’m thinking the right strategy is to build shlab.imgct into itrackvalr as a self-contained internal toolkit. Then we work within a single package that is namespaced as in itrackvalr::func() for an exported function func().
As an additional step with big dividends, we implement prefixed function class, sort of like a collection of related methods (relation typically dependent on data types, so description fits well here). This might make it itrackvalr::ic_func(), where ic is our class for image categorization. Similarly, we could include iv as our class for image valuation, where this is for methods related to the emotion rating data. Both in case of a future iteration of the task that uses different images, which seems very possible. That then leaves et for eye tracking, br for behavioral responses, and qs for questionnaires/self-report.
Additional Notes
For the most part, I consider the above classes to be most useful as a strategy to modularize and make simpler the pre-processing stage, where much of the work is related to the class-specific data sets. We might find that we want other class prefixes as we move toward analysis, visualization, etc. A good rule of thumb is that if we have more than three functions related to the same unique concept, we might want to consider adding a class.
Advanced option
See R6 classes: https://adv-r.hadley.nz/r6.html

Probably not going to do this because image categorization is separately useful

I'm going to go ahead and close this issue per our earlier conversation.


Execute generated script in Cloud

Research how to use Google Colab API
Figure out the best automation workflow to send and execute the script
Figure out how to access the output (if better visuals)

Little information were found by Calo and I with two possibilities. One solution is to use Google Colab API with google drive(not figured out yet), the other is to write an automation program that pastes the code in Google Colab, press run, and copy the output somehow.


support terminology protocols

terminology and your app are very similar so it might be a good idea to try to collab with then and or just support some of there protocols like tycat

also, it would be cool to add terminology

now that i think about you support playing just about every image format so i could just write an extraterm shell (if i wasn't lazy)

I'm probably not going to support terminology's protocols directly. But I do intend to help out and implement protocols and extensions defined at the Terminal Working Group at Free Desktop: https://gitlab.freedesktop.org/terminal-wg


What is the appsmith equivalent to nuBuilder lookup?

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Please check the gif screenshot. Click the paly button in the upper right corner of the gif to play.
Why should this be worked on?

@m2acgi you should be able to drag a modal widget and populate that with the values that you need


Decompose React UI library into abstract components (implement function) and concrete UI (implement default styles)

Problem
#649 brought into focus something that's been a problem with the React UI library for a long time. Because this library is so prescriptive about style and structure, it makes basic things like styling i18n next to impossible or wildly inefficient.
For instance, this good-faith effort to add translations to the library both:

Increased the size of the UI library, and
Forced everyone to download every language set, no matter which one they use.

Proposed solution
Dismantle this library into style-and-structure agnostic pieces:

a set of abstract components that implement the default functionality of the controls. Use Next.js' <Link> or Radix as inspiration.
a set of concrete UI controls that you can use as defaults, or supply your own.

In all cases, strings – for instance – should be taken in as props, so that everyone can use their own i18n infra.
This will be a breaking change, and will require a major version bump.
Blocks #649.

I would love to suggest a solution while Steve's "Proposed solution" is being build / taken care of:

Accept the merge of #650 for the time being.
Once the library is dismantled, improve the translation.

Seem to me the problem here is not well understood ==> NOBODY wants to use an app that displays exotic language he/she does not understand. It should be top priority to solve this, right now!
Steve's "Proposed solution" will taking months to implement... During this time, users are not onboarding.
Let's measure objectively the Benefits Versus Risks. I feel it's a no-brainer.
I'm open to any suggestions.

I believe that making it available in other languages is essential for more people to join the network.
unfortunately it is not true that most people in the world know English

Actually, 1 in 8 people in the world speak English. An i18n wallet feature can immediately make possible to onboard 8x more people to the Solana ecosystem. I believe the downside of a non-optimal architecture (that is completely usable) now is compensated by the more users using the Wallet.

I received a very moving message from @steveluscher yesterday.
Please guys, no more brigading (same tweet repeated)... I understand it's a way to show that's a feature we really need, but Steve got our point and is trying his best.
Let's try to be constructive and help him achieve our common goal by providing some ideas:

Create a fork of the wallet_adapter called wallet_adapter_intl on the Solana labs' repository (or on other repo if linked from the Solana labs' repo)  ==> English users have the current light version loading only english, Non-English users have the "heavy" version with all the languages (but those selecting it choose language over fast loading) ==> solving both problems!
Lazy load only one language file as the end user need one language at a time ==> solving one of the 2 problems!

Please, let me know if this can be done for the time being (while waiting for React UI library to be decomposed)

From what I saw, the library size today is 172kb
Every new language added to the dictionary would required ~350 bytes
So having the top 10 spoken languages to the dictionary would add 3.5kb to the total library size.
I understand that this solution is not for all languages, and not scalable, but it could be accepted on the short term to give access to another 4 billion people, while people can work on the long term version that will be more flexible and smaller to accept any language.
The average page size for websites is ~2.2MB - adding another 4-10kb to this accounting doesn't change too much.
We are (still) not Google, that needs to optimize every single byte for the end-user. We want billions of people using Solana.
Let's remember Donald Knuth words:
"The real problem is that programmers have spent far too much time worrying about efficiency in the wrong places and at the wrong times; premature optimization is the root of all evil (or at least most of it) in programming"

@jordansexton @steveluscher any comments on our proposals?

🏓

Meanwhile ... on CT, @jordansexton is asking :

One of the things Ethereum has done a great job of is cultivating open source contributions.
What can Solana Labs and Foundation devs do to make contributing to Solana easier?
What are we not doing well at right now? Please tell me about your experiences (positive and negative)

https://twitter.com/jordaaash/status/1608900210973106177

I think there's enough context here. Steven and I have been away for the holidays, and have multiple initiatives we're focused on. We get Github notifications just like everyone else, and we have to prioritize. Please keep the discussion additive and on topic.
Steven has already closed https://github.com/solana-labs/wallet-adapter/pull/650 and I agree with his rationale. He opened this tracking issue to describe a path forward, and stated that there isn't a timeline for this. Rather than attempt to do everything in react-ui, material-ui, and other packages, we should provide a library of components that are easier to customize at the application level.
I'm sympathetic to the fact that this doesn't point to a near-term solution, and I'm happy to promote alternative approaches without taking on the burden of maintenance for them. If you have an implementation of these components in mind that handles i18n in a particular way, please publish it, and let's link it from here for now: https://github.com/solana-labs/wallet-adapter/blob/master/PACKAGES.md#ui-components

Thank you so much for the great work done !
I'm really glad. Congrats :tada:

For sure! I'll shipit tomorrow.


Basis Cash Improvement Proposal #10 - Change Calculation of Bond Conversion Limits to 24hours Epochs instead of 1 hour

This is a suggestion that works on two points with regards to the _updateConversionLimit() function which is used to decide how many bonds get issued on bond purchase.
Motivation:
As per the current implementation, say if the protocol is off by 5%, and say a supply of 10,000 BAC; then a debt cap of 500 BAB is made and now the protocol is allowed to issue debt by 5%.
However, this cap although reset every hour, can only be fulfilled after the 24-hour epoch and if the protocol is in its expansion phase. This means in our example it is theoretically possible to have 5% of the BAC circulation converted into debt (BAB), 24 times (every hour the cap resets)!
Solution: Reset the debt cap once per 24hr epoch. Not every hour.
For reference: existing elastic stablecoin protocols such as ESD, issue debt only once per epoch.
Further, a second point that can possibly arise is when we look at the Epoch function. As per the code, it is possible to change the epoch duration from 24 hours to anything that the governance decides. However, epochs are not saved as integer counters but rather as timestamps (Epoch.lastExecutedAt), whereas the condition used in the _updateConversionLimit() is tracking epochs as integer counters.
Hypothetical scenario: If currently the epoch is at 10 and we are counting 24 hours epochs. Say that the governance decides that we need to start counting 48-hour epochs instead; which means the protocol starts counting from epoch 5 henceforth. However, this would mean that _updateConversionLimit() will not execute for another 10 days because it looking for epoch 11 to continue.
Solution: In this PR I've made changes to improve the _updateConversionLimit() function by removing t he epoch conditions and calling it with the allocateSeigniorage() function. Have yet to write unit tests for this change.

Thank you for propose this PR. The problem of the cap is resetting every hour, so it can be allow unlimited bond purchases. Actually, this problem comes from my misunderstanding of BIP-6. So fixed PR is now live #40. I believe this fix can resolves your concern.

Thank you for propose this PR. The problem of the cap is resetting every hour, so it can be allow unlimited bond purchases. Actually, this problem comes from my misunderstanding of BIP-6. So fixed PR is now live #40. I believe this fix can resolves your concern.

Thank you for propose this PR. The problem of the bond cap is being reset every hour, so it can be allow unlimited bond purchases. Actually, this problem comes from my misunderstanding of BIP-6. So fixed PR is now live #40. I believe this fix can resolves your concern.

Yes thanks for the change; however there is one more minor fix that needs to be added which is how the _updateConversionLimit() checks epochs. The function should ideally compare epoch timestamps and not epoch number as any change to the epoch period will cause the protocol to not trigger the fn.

Thank you for propose this PR. The problem of the bond cap is being reset every hour, so it can be allow unlimited bond purchases. Actually, this problem comes from my misunderstanding of BIP-6. So fixed PR is now live #40. I believe this fix can resolves your concern.

Yes thanks for the change; however there is one more minor fix that needs to be added which is how the _updateConversionLimit() checks epochs. The function should ideally compare epoch timestamps and not epoch number as any change to the epoch period will cause the protocol to not trigger the fn.


App refresh bug fix

White Spaces Now Allowed Before and After Email Input
This was a simple fix. I added an optional trim param to the input forms. This is set false by default, so all other input forms not using this will remain virtually the same.
All data from the Redux Store is now persists
I added a new dependency called redux-persists which takes care of the hard work of storing all the data from the redux-store into local/async storage. Now apps should be able to refresh and not have a state clear, they should simply 'rehydrate' to the previous state.
Mobile Refreshes Upon Updating
This involves more redux. I needed to store appliances of the selected property into the redux store to get proper behavior of refreshing. I moved some more files around in order to get this to work. All of the modals that mutated properties are now Smart Components (those that connect to the redux store). They use a dispatch method to refresh updates retrieved from the backend. This way, we only need to call dispatch methods to refresh when want too.
***The modals that edit a property have now been moved into a subdirectory of modals called property-modals. It even has its own index files. The base components, types, and functions should still be available for export but I do not include them in the index file.

It honestly could, I would hold off on merging the branch until I've gotten a chance to test this with the merged code.


rclc fails to initialize a support object

Issue template

Hardware description: Arduino Nano Rp2040
RTOS:
Installation type:
Version or commit hash:  galactic v2.0.3

Steps to reproduce the issue
void setup()
{
    Serial.begin(115200);
    set_microros_transports();

    Serial.print("Running\n");
    digitalWrite(LED_BUILTIN, LOW);
    allocator = rcl_get_default_allocator();
    Serial.print("Done allocating\n");

    rc = rclc_support_init(&support, 0, NULL, &allocator);
    if (rc != RCL_RET_OK)
    {
        Serial.print("Could not initialize node support\n");
        Serial.print(rc);
        Serial.print("\n");
        error_loop();
    }

Expected behavior
The code above should initialize the rclc_support_t
Actual behavior
The init function fails and return error code 1
Additional information
The additional print statements are there to check where the code fails. It's used on an Arduino with micro ros arduino.

Hello, please reopen this issue in the micro-ROS for Arduino repository and explain your hardware configuration, transport, etc.

Closed in favor of https://github.com/micro-ROS/micro_ros_arduino/issues/804


Add readme rustdoc tests to CI

Motivation
Ensure the readme code compiles and is up to date.
Solution
I think we can call rustdoc directly and test the readme in its own CI stage.

Alternative Idea: Use cargo-readme to auto-generate / check the readme.
This way the README file would be in sync with the tonic docs.rs page. Code examples would be tested automatically via cargo test because the live in a .rs file.
Bonus: You have the same nice "introduction information" no mater if new users land on the docs.rs page or this GitHub repository.

@senden9 the issue with that is we can't have fully-fledged examples in the main readme because it requires codegen and hooking into the rustdocs build.rs. I am considering just dropping examples from the readme since this will be hard and just point people to the examples crate.

@LucioFranco Is there currently any testing around the examples to ensure they are working? If not, is this something we should think about writing and adding into the CI?

So I am going to remove all examples from the readme. We will instead point to the examples crate which is checked via CI already. All doc examples are also checked via CI and cargo test.

Sounds good!

This can be closed now that we've removed examples from the readme, thanks for all the help!


docs: Add upgrading page

docs: Add upgrading page

Currently just borrowed upstream Kayobe upgrade page, needs more info.

Cancelled CI jobs to save SMS usage


Extend CI test matrix

See comments in https://github.com/Toblerity/Fiona/pull/1218#pullrequestreview-1350899043

@sgillies  Fiona does not have any numpy dependency that I'm aware of.  I assume this is a copy-paste artifact from rasterio?

@sgillies It looks as if there are no appveyor checks showing up, but the configuration is still there. Is this unexpected or should the appveyor config be removed?

The AppVeyor builds are running here, but I guess some GitHub authorizations are needed to signal the check result back to the PR.

@mwtoews  Thanks for checking. It looks as if they are broken.

Let's remove the AppVeyor config, but in a different PR.

@sgillies I had a quick look at the new CI config and one thing led to another:

Extending the Linux CI to all supported Python versions is a personal preference.  Most of Fiona's code should not really depend on the Python version, although dependencies might (and also e.g. type annotations support changed over time)

As the conda tests are comparably slow I thought it would be more "interesting" to test on Windows instead of on Linux. For the same reason also to reduce the tests to just one per platform. Maybe it would make sense to just test the newest supported Python version to detect issues like the DLL loading on Windows? Or test the oldest and newest supported Python version for each platform?

A while ago there was an issue that the rstcheck check broke due to unresolvable dependencies.  Pip-compile takes it to the next level by pinning all the dependencies to ensure the same environment with each test run. Enabling dependabot for the rstcheck dependencies ensures that rstcheck is not forgotten. It is configured to run once a month, this should keep the maintenance overhead minimal.

In case you don't like any of these changes I can revert them or you can just pick the changes you like.

Numpy dropped support for Python 3.7, we could also drop the tests for Python 3.7.

I suggest to keep the Python versions in this PR as they are now, and drop Python 3.7 in a separate PR against master (as it could have other non-CI changes). It would be best to keep Python 3.7 for maint-1.9. I can create a PR if you want.


Fail to upload one blob with the same name if there is a cloned rehydrated blob got deleted

Storage Explorer Version: 1.22.0-dev
Build Number: 20211008.1
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Big Sur 11.5
Architecture: ia32\x64
How Found: Ad-hoc testing
Regression From: Not a regression
Steps to Reproduce

Expand one non-ADLS Gen2 storage account -> Blob Containers.
Create a blob container -> Upload two blobs -> Change one blob's access tier to 'Archive'.
Right click the archived blob -> Click 'Clone and Rehydrate'.
Input another existing blob name -> Click 'Apply'.
Delete the cloned rehydrated blob.
Try to upload a blob which name is the same as the cloned rehydrated blob.
Check whether succeed to upload.

Expected Experience
Succeed to upload.
Actual Experience
Fail to upload.

Additional Context

After deleting, wait five minutes then try to upload, it still reproduces.
Here is the Azcopy Log:
cfdcf97d-932e-7f40-6612-d0954b4fa72b.log

The backend has some protection mechanism to prevent corrupting data that is being rehydrated. Let's wait for a day or two and see if it allows re-upload the data.

This issue also reproduces in the following steps:

Expand one storage account -> Blob Containers. (Make sure the soft delete is enabled)
Create a blob container -> Upload one blob.
Chang its access tier to 'Archive' -> Change its access tier to 'Cool'/'Hot'.
Delete the blob -> Upload a blob whose name is same as the deleted blob.
Check succeeds to upload the blob.

Actual Experience
Fail to upload the blob.

Verified this issue on the build 20230516.9. Not allow to use an existing name to clone and rehydrate. So we are going to close it.
Refer to 6655


Image Customisation

Hi,
Is there any plan to introduce custom icons? It would be great to be able to replace the terminal image with something more appropriate to each program.
Really nice tools!
Thanks,
Matt

Hey, I somehow totally missed this issue.
Having multiple icons sets or themes was always something I wanted from the beginning. However, I lack the artistic skills to come up with additional icons. I've had a Call for Artists at the top of the github README for a little over a year at this point, but alas, no takers so far.  :(
If you've got a set of icons or images that you want to use, you could drop them into the images/ directory in Gooey and overwrite the existing media. Outside of that, there's not really any established machinery in the code for supplying custom image paths.

Would it be better to maybe have an option for the developer to provide a different set of icons?
E.g. I use Gooey for some astronomy software, and making some stars as an icon will not be very popular (I guess). But if I can implement it as an option when calling Gooey, everything would be fixed.

@DanielAndreasen, I've added a few features to make supplying your own images a little easier.
The @Gooey decorator now accepts a image_dir parameter with which you can supply a directory containing your custom images.
I went with an approach kinda along the lines of a Django admin. Namely, the defaults are always loaded, but you can overwrite specific images by supplying a new path. So, you don't need to supply all images, just those that you want to overwrite. They're implicitly loaded based on filename. So, just putting an image in your images directory called config_icon.png would be pulled in and used in place of the default config_icon.png.
example use:
@Gooey(image_dir='/path/to/my/image/dir')
def main():
   ...

Available icon names:

program_icon.ico
success_icon.png
running_icon.png
loading_icon.gif
config_icon.png
error_icon.png

It is not surprising that this works too :smile:

Yay!


Order names

Currently when we identify surname or birthdates, the order of the surnames is not correct. We should fix that

First resolution of this error in d4dd18cf4802eec3bf094169563719a051c48fc2
However, we should now implement some line segmentation before sorting the words in the line


select nextval() goes to read replica

Describe the bug
Hi the nextval() increase the value in the psql sequence. But the thing is, eve within transaction, built in query parser handle the select nextval() and forward it to read replica. Can we make query whitelist/blacklist like pgpool, or can we fix the nextval should be redirect to write replica under read replica?
If I want to add additional logic in query parser, which file should I look into?
Thank you
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Take a look at the query router. It parses the SQL into an AST using the sqlparser crate. You should be able to identify the called functions and adjust the desired server role accordingly.
Having a list of queries is not going to work long term since just adding a new query to the app will break the behavior. I'm open to other ideas though.
If you want a quick fix, open a transaction before executing your primary-only query. All transactions are routed to the primary automatically.

Something weird is definitely going on. Do you think you could submit a PR with a simple Java app that I can use to reproduce this? We can use that as part of our test suite to fix this and also to make sure we don't hit this bug in the future in case we change something.

We can but this functionally doesn't exist, yet.

i notice that pgcat has something like this
/* shard_id: 5 */ SELECT * FROM foo WHERE id = 1234;
does the "with comments" feature support to select query redirection based on comments?
/* server: primary */ SELECT * FROM foo WHERE id = 1234;

Not at the moment, but we can add that in if needed: https://github.com/postgresml/pgcat/blob/main/src/query_router.rs#L149

Regarding the initial issue (SELECT nextval() being routed to a standby), note the pgpool checks if a function that is called in a SELECT query is has Volatility=volatile or not and decides whether to route the query to the primary or a standby based on that:
https://git.postgresql.org/gitweb/?p=pgpool2.git;a=blob;f=src/utils/pool_select_walker.c;h=27e745ba473d19a50f91f92aff17b4eb5c2aa94c;hb=HEAD#l385
https://pgpool.net/mediawiki/index.php/Pgpool-II_4.2_development
https://www.pgpool.net/pipermail/pgpool-hackers/2019-September/003438.html

After did some quick debugging, it seems the query_router.infer() seems never been called in client.rs.

This won't help for the nextval(), but if queries aren't getting "infered", make sure you have the query_parser_read_write_splitting = true config entry in your pgcat.toml file.


bundle_size_analyzer

don't forget to fix prettier

don't forget to fix prettier

of course, thanks !


Improved API for discord

Changes:

Simplified AutoTx and web3 configurations
Added support for subsidized networks in API mode with Alchemy
Updating task 'messages' and 'updated_at' fields when a message is added
Fixed issues with API auto-submitting transactions
Implemented supported networks endpoint
Implemented provide_feedback endpoint
Implemented send transactions endpoint

ALCHEMY_API_KEY env variable is required to run the API (in order for the subsidized networks to work)

/workflows/benchmarks agents/token 3

/workflows/benchmarks agents/token 3

/workflows/benchmarks agents/token/research 5

/workflows/benchmarks agents/token/research 5


Fix the color of the upload button's spotify icon

This pull request fixes the color of the play icon which shows on the upload button when Spotify is playing.

Isn't the Spotify color a separate color?

Isn't the Spotify color a separate color?

Sorry if I wasn't clear enough-- I meant this icon right here which appears when you're playing Spotify to let you know that you can create a listening party

Ohh, I thought you meant the Spotify icon for the menu; sorry.


feat: add QGB query attestation CLI command

Overview

Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

How would I verify that attestation 2+ work? Do I need to run something else to get more attestations added to state
Is that error expected for an attestation that doesn't exist yet?

At chain start, there is only 1 attestation. And after 400 blocks, you will see another one.
For the tests, I will check if I can write something. The thing is it's just calling a query,  it's not doing anything fancy. Thus, I thought that testing it would only be testing if cobra works well, then if the query works fine (and that's already tested in the keepers)

Codecov Report

Merging #1892 (c348b44) into main (bb00764) will decrease coverage by 0.63%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1892      +/-   ##
==========================================
- Coverage   21.85%   21.22%   -0.63%
==========================================
  Files         117      120       +3
  Lines       13304    13696     +392
==========================================
  Hits         2907     2907
- Misses      10107    10499     +392
  Partials      290      290

Impacted Files
Coverage Δ

x/qgb/client/query.go
0.00% <0.00%> (ø)

x/qgb/keeper/query_attestation.go
0.00% <0.00%> (ø)

x/qgb/module.go
7.27% <0.00%> (ø)

... and 2 files with indirect coverage changes


Add many test codes and distribute case operation ( of switch ) to some switch statements

Reduce encode_optype.go code size
Reduce many optype ( removed indent or escaped optype )
Add multiple VM for indent or escaped optype ( distribute switch-case to reduce memory usage at compile time )
Add many test cases for improvement coverage

Codecov Report

Merging #81 (9c5f6ec) into master (bf18ed0) will increase coverage by 22.70%.
The diff coverage is 90.69%.

@@             Coverage Diff             @@
##           master      #81       +/-   ##
===========================================
+ Coverage   27.88%   50.58%   +22.70%
===========================================
  Files          35       38        +3
  Lines       16201    22921     +6720
===========================================
+ Hits         4517    11595     +7078
+ Misses      11320    10934      -386
- Partials      364      392       +28

Impacted Files
Coverage Δ

encode_opcode.go
62.34% <ø> (-1.78%)
:arrow_down:

encode_vm.go
36.37% <ø> (+22.38%)
:arrow_up:

encode_vm_escaped.go
54.65% <ø> (ø)

encode_vm_escaped_indent.go
51.26% <ø> (ø)

encode_vm_indent.go
42.42% <ø> (ø)

encode.go
91.66% <86.66%> (+1.10%)
:arrow_up:

encode_compile.go
83.23% <92.85%> (+5.68%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #81 (9c5f6ec) into master (bf18ed0) will increase coverage by 22.70%.
The diff coverage is 90.69%.

@@             Coverage Diff             @@
##           master      #81       +/-   ##
===========================================
+ Coverage   27.88%   50.58%   +22.70%
===========================================
  Files          35       38        +3
  Lines       16201    22921     +6720
===========================================
+ Hits         4517    11595     +7078
+ Misses      11320    10934      -386
- Partials      364      392       +28

Impacted Files
Coverage Δ

encode_opcode.go
62.34% <ø> (-1.78%)
:arrow_down:

encode_vm.go
36.37% <ø> (+22.38%)
:arrow_up:

encode_vm_escaped.go
54.65% <ø> (ø)

encode_vm_escaped_indent.go
51.26% <ø> (ø)

encode_vm_indent.go
42.42% <ø> (ø)

encode.go
91.66% <86.66%> (+1.10%)
:arrow_up:

encode_compile.go
83.23% <92.85%> (+5.68%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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


Cannot send variables to a template

Hi there,
I'm struggling to pass variables from my rails app to my Sendgrid template.
Everything else works fine.
Here is the code I'm using:
    mail(
      to: @user,
      subject: 'email subject',
      body: 'email body',
      template_id: 'xxx',
      dynamic_template_data:{
        variable_1: 'foo',
        variable_2: 'bar'
      }
    )

and I tried all of the following in my template:
:variable_1
-variable_1-
%variable_1%
[variable_1]
variable1
:variable1
-variable1-
%variable1%
[variable1]
Variable1
:Variable1
-Variable1-
%Variable1%
[Variable1]

Any idea?
I also tried unique_args && sub

I have the same issue too

OMG, got it! Inside template need use double curly brackets {{variable_1}}

Yeah, Sendgrid uses handlebars for template variables.  I've updated the Readme to reflect that fact.  Thanks for raising this issue.

Awesome! The only one I didn't test haha. Thanks for the feedback @DKarp and for the update @tyrauber


Mapping the temporal and spatial dynamics of the human endometrium in vivo and in vitro

Publication:
Mapping the temporal and spatial dynamics of the human endometrium in vivo and in vitro
Priority:
1

This dataset has already been submitted to DCP


Error adding Spring Boot Parameters to App Configuration via ARM Template

I'm trying to create a App Configuration instance via an ARM template and populate the key value pairs via the template.  According to the Spring Boot Azure App Configuration documentation, all keyName need to have the prefix of '/application/' for Azure Spring Cloud Config to retrieve these values.  But including the prefix while creating the keyName violates the segment length constraint for the naming convention.
I get the following error when trying to run the template.
ERROR: {"error":{"code":"InvalidTemplate","message":"Deployment template validation failed: 'The template resource 'app_config_sandbox/application/spring.datasource.driver-class-name$development' for type 'Microsoft.AppConfiguration/configurationStores/keyValues' at line '56' and column '72' has incorrect segment lengths. A nested resource type must have identical number of segments as its resource name. A root resource type must have segment length one greater than its resource name. Please see https://aka.ms/arm-template/#resources for usage details.'.","additionalInfo":[{"type":"TemplateViolation","info":{"lineNumber":56,"linePosition":72,"path":"properties.template.resources[1].type"}}]}}
The following template generates this error:
{
  "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
  "contentVersion": "<IP_ADDRESS>",
  "parameters": {
    "environment": {
      "type": "String",
      "allowedValues": [
        "sandbox"
      ]
    }
  },
  "variables": {
    "environmentSettings": {
      "sandbox": {
        "appConfigInstanceName": "app_config_sandbox",
        "region": "centralus"
      }
    }
  },
  "resources": [
    {
      "type": "Microsoft.AppConfiguration/configurationStores",
      "apiVersion": "2021-03-01-preview",
      "name": "[variables('environmentSettings')[parameters('environment')].appConfigInstanceName]",
      "location": "[variables('environmentSettings')[parameters('environment')].region]",
      "sku": {
        "name": "standard"
      },
      "resources": [
        {
          "name": "application/spring.datasource.driver-class-name$sandbox",
          "type": "keyValues",
          "apiVersion": "2021-03-01-preview",
          "dependsOn": [
            "[variables('environmentSettings')[parameters('environment')].appConfigInstanceName]"
          ],
          "properties": {
            "value": "test"
          }
        }
      ]
    }
  ]
}

az deployment group validate --parameters environment=sandbox --template-file template_2.json --resource-group testRg
If I remove the 'application/' prefix from the key name, ARM is able to process the template correctly and create the App Configuration instance, but the keyname is then not in the correct format and the spring-cloud-azure-appconfiguration-config SDK is not able to pick up the parameters from the configuration.
https://docs.microsoft.com/en-us/java/api/overview/azure/spring-cloud-starter-azure-appconfiguration-config-readme?view=azure-java-stable#supported-properties
Is this not a possibility to add key value pairs via an ARM template for Spring Boot apps?  Or am I missing something?

Hi @cmoyer4
The key and label specified in the ARM template need to be encoded due to the restriction of characters in the ARM template. In your case application/ should be encoded as application~2F. Please check out the Tip section towards the end of this document for more details: https://docs.microsoft.com/en-us/azure/azure-app-configuration/quickstart-resource-manager

@cmoyer4 please let us know if you have other questions. Closing for now.


Add support for SVG icons

This defaults to using SVG if it's available, then falls back to assuming PNG.
This still puts the burden of finding the default Exercism icon on the caller, which we should eventually fix (serving a default icon if no track icon can be found). I'd like to use an svg for that, though, so I've not done it yet (I have a file with SVG of the logo in it, but I don't have the software to open it).
See exercism/exercism.io#2910
/cc @Insti

OK, I'm happy with this now.

Thanks for the review, @Insti!


⚠ Bofrost.fr has degraded performance

In 7e0aec6, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1843 ms

Resolved: Bofrost.fr performance has improved in c563669 after 6 minutes.


Ambiguous ClusterPoint

Hello, we are running:

Pinpoint 2.0.4 agent on JBoss 7 with OpenJDK 11.
Pinpoint 2.0.4 collector and web.

During the weekend some networking issues caused communication problems between agents and collector. When the networking issue has been resolved, many of our Pinpoint-monitored applications were in this state:

What looks strange to me is that some of our applications were running more than one Pinpoint-monitored JVM on the same host, having some of the JVMs experiencing the error while others not.
It seems like, for some reason, the collector has been able to get back on track only some of the agents after the networking failure.
Agent logs are clean: after the errors reported during the networking issues, no ERROR or WARN entries.
Collector logs, instead, report this problem upon opening MyApp on Pinpoint web:
2021-09-29 11:31:32 [WARN ](c.n.p.c.c.r.StreamRouteHandler     ) Ambiguous ClusterPoint MyApp, host.name,<PHONE_NUMBER>783 (Valid Agent list=[com.navercorp.pinpoint.collector.cluster.GrpcAgentConnection@66649590, com.navercorp.pinpoint.collector.cluster.GrpcAgentConnection@66649590]).
2021-09-29 11:31:32 [WARN ](c.n.p.c.c.ClusterPointRouter       ) handleStreamRouteCreate failed. command:TCmdActiveThreadCount(), routeResult:NOT_FOUND

MyApp and host.name are replacements for out real application names and host names. This WARN appears only with regard to the failing host.names so i think this is a strong clue on what is causing the issue.
Is there a bug or something preventing all agents to get back to normal monitoring after a networking error occurs? Maybe some idle or timeout configuration do be tuned?

Adding some info: the agents reporting the error in the "active threads" view are perfectly working with respect to all the other functionalities (scatter diagram, traces, inspector...).

Hi @davide-parini
If possible, please update pinpoint version to 2.3.0.
There are some issues that is related with cluster at version 2.0.x because we changed transport layer to grpc since 2.0.0.
Thanks :)

Thank you for your feedback @koo-taejin, we are going to upgrade all our applications to the latest release in the next few months.


DRILL-8092: Add Auto Pagination to HTTP Storage Plugin

DRILL-8092: Add Auto Pagination to HTTP Storage Plugin
Description
This PR adds the ability for Drill to access APIs that have some sort of pagination.  In a nutshell, let's say an API limits you to 100 records per page.  This improvement allows Drill to execute multiple HTTP requests to retrieve the complete dataset.
This update works in two ways: with a limit and without.   In the event a limit is pushed down, the new paginator object will generate the correct number of URLs and BatchReaders, execute the queries and return the results.  Currently, this is executed in series, but in future work this could be parallelized.
In the event a limit is not pushed down, the reader will keep generating URLs and retrieving data until the row count of data returned is less than the page size, or the API returns zero results, or non-success code.
In the event that a limit is included in the query, it is possible that pieces of this could be parallelized, however in this PR, the requests are executed in series.  Each request gets its own batch reader.
Documentation
(From README)
Remote APIs frequently implement some sort of pagination as a way of limiting results.  However, if you are performing bulk data analysis, it is necessary to reassemble the data into one larger dataset.  Drill's auto-pagination features allow this to happen in the background, so that the user will get clean data back.
To use a paginator, you simply have to configure the paginator in the connection for the particular API.
Offset Pagination
Offset Pagination uses commands similar to SQL which has a LIMIT and an OFFSET.  With an offset paginator, let's say you want 200 records and the maximum page size is 50
records, the offset paginator will break up your query into 4 requests as shown below:

myapi.com?limit=50&offset=0
myapi.com?limit=50?offset=50
myapi.com?limit=50&offset=100
myapi.com?limit=50&offset=150

Configuring Offset Pagination
To configure an offset paginator, simply add the following to the configuration for your connection.
"paginator": {
   "limitField": "<limit>",
   "offsetField": "<offset>",
   "maxPageSize": 100,
   "method": "OFFSET"
}

Page Pagination
Page pagination is very similar to offset pagination except instead of using an OFFSET it uses a page number.
 "paginator": {
        "pageField": "page",
        "pageSizeField": "per_page",
        "maxPageSize": 100,
        "method": "PAGE"
      }

Testing
Added unit tests and tested manually.

@jnturton Thanks for your feedback.  I addressed all your comments and responded to the last two.  Thanks again!


What is the tabNavigator click event? I need to pass a parameter in the item component when I click a picture under tabNavigator. How do I do that?

Is this a bug report?
(write your answer here)

Have you read the Contributing Guidelines?
(Write your answer here.)
Environment

react-native -v:
node -v:
npm -v:
yarn --version:

Then, specify:

Target Platform:

Development Operating System:

Build tools:

Steps to Reproduce

(Write your steps here:)

Expected Behavior

(Write what you thought would happen.)
Actual Behavior

(Write what happened. Add screenshots!)
Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

I've used react-navigation's TabNavigator, tab pages, A, B, and C, all using StatusBar,
When the A tab is changed, StatusBar does not work because it is covered by the last C tab StatusBar,
How can you separate StatusBar on each tab page?

@facebook-github-bot stack-overflow


Sign In using Sign In card

Hi all,
I am using node.js to communicate with my bot. I want to know how can we authenticate using sign-in card in my bot framework.  Will it revert some token or something else after sign in on my website or I have to do some other thing?

Thank you for your submission. The Microsoft Bot Framework team prefers that bug fixes and feature requests were opened as new issues on the official Bot Framework Github repo.  The Stack Overflow botframework tag is a place to ask how to questions and get support from the community.
Before posting to Stack Overflow, please feel free to take a look at this tutorial. Hopefully it will answer some of your questions.


Question: switch to registry.k8s.io

Detailed description of the question.
k8s.gcr.io Image Registry Will Be Frozen From the 3rd of April 2023
We need to switch k8s cluster image to registry.k8s.io ?
Ref to: https://kubernetes.io/blog/2023/02/06/k8s-gcr-io-freeze-announcement/
Some reference materials you see.
No response

We don't need to care about these, we don't care about any mirror warehouse image-shim will handle it for you.


Fix $_ type inference crash

PR Summary
Fixes an infinite loop when trying to infer $_ inside an incomplete switch statement when $_ is inside the condition.

PR Context
Fixes https://github.com/PowerShell/PowerShell/issues/21221 which was caused by a regression added with https://github.com/PowerShell/PowerShell/pull/17716

PR Checklist

[X] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[X] Summarized changes
[X] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[X] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[X] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[X] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[X] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[X] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

@daxian-dbw Would you have a look at this PR please?

Approving for 7.4.x as this fixes a crash and only affects interactive scenarios.


Enable no-unsafe eslint rules

Eslint v7 has 4 new rules very useful to guard against using any types inadvertently.

Disallows assigning any to variables and properties (no-unsafe-assignment)
Disallows calling an any type value (no-unsafe-call)
Disallows member access on any typed variables (no-unsafe-member-access)
Disallows returning any from a function (no-unsafe-return)

The purpose of this PR is gauge interest by Lodestar maintainers before actually working on the issues they raise. I've counted 800 lint errors of no-unsafe-* rules, mostly in tests accessing un-typed stubs

Imo this is a great idea +1 from me

good stuff!  i think we should do this

👍🏻

added in the changes to apply these rules.  there are some files that have an eslint-disable at the top as well as the same for some lines here and there, but most do not.

@3xtr4t3rr3str14l Whenever you see fit, find opportunities to de-duplicate boilerplate code. For example in the test utils stubs you can define this helper function
// eslint-disable-next-line @typescript-eslint/ban-types
type StubbableType<TType> = Function & {prototype: TType};
function createStubInstance<TType>(constructor: StubbableType<TType>): SinonStubbedInstance<TType> & TType {
  return sinon.createStubInstance(constructor) as SinonStubbedInstance<TType> & TType;
}

and initialize the class properties with just
this.depositEvent = createStubInstance(DepositEventRepository);

@3xtr4t3rr3str14l Whenever you see fit, find opportunities to de-duplicate boilerplate code. For example in the test utils stubs you can define this helper function
// eslint-disable-next-line @typescript-eslint/ban-types
type StubbableType<TType> = Function & {prototype: TType};
function createStubInstance<TType>(constructor: StubbableType<TType>): SinonStubbedInstance<TType> & TType {
  return sinon.createStubInstance(constructor) as SinonStubbedInstance<TType> & TType;
}

and initialize the class properties with just
this.depositEvent = createStubInstance(DepositEventRepository);

@dapplion definitely.  i saw a lot of duplicates in the process.  in beacon-state-transition too, memorably.


🐛:  Tasks for CellML models do not contain language abbreviation field

Contact Details
No response
Location
BioSimulations API
What happened?
When getting project summaries, the task summaries do not contain the "acronym" field for cell-ml models
What made it happen?
This is potentially happening either in the API endpoint for the summaries or is due to combine-API not returning the information. I don't recall exactly where this information is set.
Version
Production (Deployed on a .org domain)
What browsers are you seeing the problem on?
No response
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

This should be fixed by #4524.
Rather than loading the data from JSON file which is going to of type "any" , it might be useful to more precisely define the types and data in a regular typescript file.  The code also uses "as" quite a bit which can be removed to make it more robust.
@jonrkarr is this information generated by hand or does it update with the ontology updates? If it is not automatically updated, its trivial to simply define it in typescript instead.

CellML doesn't have a longer name.
This information is hard-coded. At least for now, it can't be read from EDAM.

Yup, the reason for the acronym field not being present makes sense, but I was just using it as default to display as labels on the chart.  I thought the table component was also doing this, but it must have accounted for this since it displays "CellML" correctly.

You should display the acronym || name.

:tada: This issue has been resolved in version 9.17.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Screenreader not picking up on account dropdown

When using VoiceOver or NVDA, the screenreader is not "seeing" the account dropdown menu (the user's name and carrot, and corresponding menu).

Playing around with https://chrome.google.com/webstore/detail/chromevox/kgejglhpjiefppelpmljglcjbhoiplfn I was able to recreate.

Backlogged as MUMUP-2508.

Fixed in the 2.7.0 release.


[video_player] web support

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
I get an XFile video from the camera package and I'd like to show it to my user before they upload it. Right now the only way is to turn it into a regular File, which you cannot do on the web.
Proposal
A VideoPlayerController.xFile function

Try this:
import 'dart:io';

import 'package:cross_file/cross_file.dart';
import 'package:video_player/video_player.dart';

final XFile pickedVideo = ...;
final VideoPlayerController controller;

if (kIsWeb) {
  controller = VideoPlayerController.networkUrl(Uri.parse(pickedVideo.path));
} else {
  controller = VideoPlayerController.file(File(pickedVideo.path));
}

It works!
It doesn't work for me on safari though

I think this answers the question


[QR Code Install] Display scanned URL

When scanning a QR Code, you could display the scanned URL when you're in the confirmation dialog

A QR code can contain multiple URLs, which could go off of the screen.


[Interface Procedure Request] Codeunit 9855 - Permission Set Relation - AddNewPermissionSet

Dear ALAppExtensions team,
Through this issue we would like to request Interface Procedure AddNewPermissionSet to be added to Codeunit 9855 Permission Set Relation the following signature:
    [Scope('Cloud')]
    procedure AddNewPermissionSet(CurrAppId: Guid; CurrRoleId: Code[30]; CurrScope: Option System,Tenant; RelatedAppId: Guid; RelatedRoleId: Code[30]; RelatedScope: Option System,Tenant; PermissionType: Option Include,Exclude) Success: Boolean
    var
        PermissionSetRelationImpl: Codeunit "Permission Set Relation Impl.";
    begin
        Exit(PermissionSetRelationImpl.AddNewPermissionSet(CurrAppId, CurrRoleId, CurrScope, RelatedAppId, RelatedRoleId,  RelatedScope, PermissionType);
        //Note: to make this work scope local must be removed from procedure AddNewPermissionSet in Internal Codeunit 9856 "Permission Set Relation Impl."
    end;

We need this to be able to write to/control the "Tenant Permission Set Rel." table from code (in a responsible way).
This is already possible for the "Access Control" and the "User Group Permission Set" tables, but unfortunately not for the "Tenant Permission Set Rel." table.

Created a PR:
[microsoft/ALAppExtensions] Added new procedure "Permission Set Relation".AddNewPermissionSetRelation (PR #21720)