Compare strings with equals() instead of ==

Previous PR https://github.com/opensearch-project/OpenSearch/pull/16415

New PR https://github.com/opensearch-project/OpenSearch/pull/16714


run-vcpkg@v11 requires manually setting environment variables for caching to work

Using vcpkg in manifest mode, our workflow which worked on v10 emits this output and doesn't cache.
         "D:\a\Cataclysm-DDA\b\vcpkg\vcpkg.exe" install "--host-triplet=x64-windows-static" --x-wait-for-lock --triplet "x64-windows-static" --vcpkg-root "D:\a\Cataclysm-DDA\b\vcpkg\\" "--x-manifest-root=D:\a\Cataclysm-DDA\Cataclysm-DDA\msvc-full-features\\" "--x-install-root=D:\a\Cataclysm-DDA\Cataclysm-DDA\msvc-full-features\vcpkg_installed\x64-windows-static\\" --clean-after-build
         "D:\a\Cataclysm-DDA\b\vcpkg\vcpkg.exe" install "--host-triplet=x64-windows-static" --x-wait-for-lock --triplet "x64-windows-static" --vcpkg-root "D:\a\Cataclysm-DDA\b\vcpkg\\" "--x-manifest-root=D:\a\Cataclysm-DDA\Cataclysm-DDA\msvc-full-features\\" "--x-install-root=D:\a\Cataclysm-DDA\Cataclysm-DDA\msvc-full-features\vcpkg_installed\x64-windows-static\\" --clean-after-build
     3>D:\a\Cataclysm-DDA\b\vcpkg\scripts\buildsystems\msbuild\vcpkg.targets(183,5): warning : warning: Embedding `vcpkg-configuration` in a manifest file is an EXPERIMENTAL feature. [D:\a\Cataclysm-DDA\Cataclysm-DDA\msvc-full-features\Cataclysm-lib-vcpkg-static.vcxproj]
         The GHA binary source requires the ACTIONS_RUNTIME_TOKEN and ACTIONS_CACHE_URL environment variables to be set. See https://learn.microsoft.com/en-us/vcpkg/users/binarycaching#gha for details.
     3>D:\a\Cataclysm-DDA\b\vcpkg\scripts\buildsystems\msbuild\vcpkg.targets(183,5): error MSB3073: The command ""D:\a\Cataclysm-DDA\b\vcpkg\vcpkg.exe" install "--host-triplet=x64-windows-static" --x-wait-for-lock --triplet "x64-windows-static" --vcpkg-root "D:\a\Cataclysm-DDA\b\vcpkg\\" "--x-manifest-root=D:\a\Cataclysm-DDA\Cataclysm-DDA\msvc-full-features\\" "--x-install-root=D:\a\Cataclysm-DDA\Cataclysm-DDA\msvc-full-features\vcpkg_installed\x64-windows-static\\" --clean-after-build" exited with code 1. [D:\a\Cataclysm-DDA\Cataclysm-DDA\msvc-full-features\Cataclysm-lib-vcpkg-static.vcxproj]

@akrieger good question, I need to add explicitly this info to the run-vcpkg@v11 documentation.
I imagine you have a workflow that does this:
     uses: run-vcpkg@v11. # <= Succeeds
     run: vcpkg.exe install <rest of cmd line> # <= Fails with missing env var error

When running vcpkg yourself (i.e. when it is not run by the action), you need to add the following variables, like it is done in the C++ project template:
      - uses: actions/github-script@v6
        with:
          script: |
            core.exportVariable('ACTIONS_CACHE_URL', process.env.ACTIONS_CACHE_URL || '');
            core.exportVariable('ACTIONS_RUNTIME_TOKEN', process.env.ACTIONS_RUNTIME_TOKEN || '');

I see two fixes here:

run-vcpkg@v11 must define those two variables, so the user does not need to do it manually. Note that in the workflow linked above the run-vcpkg action is not used at all, hence the definition of the env var must be done explicitly in the workflow.
add docs to run-vcpkg@v11 about these variables.

Our workflow file uses run-vcpkg and then manually invokes vcpkg via manifest mode.
    - name: Restore artifacts, or run vcpkg, build and cache artifacts
      uses: lukka/run-vcpkg@v11
      id: runvcpkg
      with:
        # run-vcpkg tries to hash vcpkg.json but complans if it finds more than one.
        # That said, we also have our custom vcpkg_triplets to hash, so we keep everything the same.
        appendedCacheKey: ${{ hashFiles( 'msvc-full-features/vcpkg.json', 'msvc-object_creator/vcpkg.json', '.github/vcpkg_triplets/**', '.github/vckg_ports/**' ) }}
        # Rev this value to drop cache without changing vcpkg commit
        prependedCacheKey: v1-x64-full
        vcpkgDirectory: '${{ runner.workspace }}/b/vcpkg'
        # Caching happens as a post-action which runs at the end of the whole workflow
        vcpkgGitCommitId: '5b1214315250939257ef5d62ecdcbca18cf4fb1c'

    - name: Integrate vcpkg
      run: |
        vcpkg integrate install --vcpkg-root '${{ runner.workspace }}\b\vcpkg'

        ...

    - name: Build <-- vcokg invoked by msbuild here
      run: |
          msbuild -m -p:Configuration=Release -p:Platform=x64 "-target:Cataclysm-vcpkg-static;Cataclysm-test-vcpkg-static;JsonFormatter-vcpkg-static" msvc-full-features/Cataclysm-vcpkg-static.sln
          msbuild -m -p:Configuration=Release -p:Platform=x64 "-target:ObjectCreator-vcpkg-static" msvc-object_creator/ObjectCreator-vcpkg-static.sln

Setting the env vars seemed to suppress that output. I found the job failing is caused by a changed portfile, so I'll edit the issue title appropriately.

@akrieger fixed with release https://github.com/lukka/run-vcpkg/releases/tag/v11.1, let me know!

     Attempting to fetch 19 package(s) from GHA
     ...
     Restored 19 package(s) from GHA in 9.5 s. Use --debug to see more details.

Seems like it worked! I removed the logic which set the env vars myself, vcpkg seemed to restore from its own cache correctly.

@akrieger let me know if anything else needs to be fixed, thanks!


allow number keys to skip to a percentage of the video

This PR allows number keys to skip to a percentage of the video, like youtube.

Good addition, thank you.


V10 content changes

Updates for Early-Career Payments journey (v10 content changes)

Update 'Did you take out a Postgraduate Masters Loan?' (ECP-913)
Update 'Did you take out a Postgraduate Doctoral Loan?' (ECP-914)
Update 'How many higher education courses did you take a student loan out for?' (ECP-911)
Update 'When did all of your higher education courses start?' (ECP-912)
Update 'Application complete' screen (ECP-915)
Fix No Validation on Personal Detail: Email address page (ECP-932)

Closing as delivered separately under respective JIRA's


Adding folder and filetype icons using new phantom API

Using the inline layout of the phantom we can render a small html file which renders just an image as the file/folder icon.

We need to find expanded directories when refreshing/renaming, currently it is just view.find
Can we access phantoms of a view and content of a phantom? Does not seem so, there is undocumented view.query_phantom(pid), but it returns region.
We could keep expanded paths in settings just like index, mmm… that would change some logic, plus I’m not ready to drop ST2 support — so all in all it is pretty complex feature, on the other side, having proper icons for different files is very tempting.

I totally agree. it's just an idea and prospect for future. I still want to use/see phantoms for smaller features on smaller plugins.


Ubuntu 19.10 Updates Crashes

This is what I get ANYTIME I try to update my system (Ubuntu 19.10). Please I need a solution from the community...
OUTPUT:
natus@etApi-CCXC:~$ sudo apt update
Get:1 http://us.archive.ubuntu.com/ubuntu eoan InRelease [2,821 B]
Err:1 http://us.archive.ubuntu.com/ubuntu eoan InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Get:2 http://us.archive.ubuntu.com/ubuntu eoan-updates InRelease [2,829 B]
Err:2 http://us.archive.ubuntu.com/ubuntu eoan-updates InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Get:3 http://ppa.launchpad.net/gezakovacs/ppa/ubuntu eoan InRelease [2,832 B]
Err:3 http://ppa.launchpad.net/gezakovacs/ppa/ubuntu eoan InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Get:4 http://us.archive.ubuntu.com/ubuntu eoan-backports InRelease [2,831 B]
Err:4 http://us.archive.ubuntu.com/ubuntu eoan-backports InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Get:5 http://ppa.launchpad.net/linuxgndu/sqlitebrowser/ubuntu eoan InRelease [2,841 B]
Err:5 http://ppa.launchpad.net/linuxgndu/sqlitebrowser/ubuntu eoan InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Get:7 http://us.archive.ubuntu.com/ubuntu eoan-security InRelease [2,830 B]
Err:7 http://us.archive.ubuntu.com/ubuntu eoan-security InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Hit:8 https://brave-browser-apt-release.s3.brave.com stable InRelease
Get:9 http://us.archive.ubuntu.com/ubuntu eoan-proposed InRelease [2,830 B]
Err:9 http://us.archive.ubuntu.com/ubuntu eoan-proposed InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Hit:6 https://packagecloud.io/AtomEditor/atom/any any InRelease
Reading package lists... Done
N: See apt-secure(8) manpage for repository creation and user configuration details.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
E: The repository 'http://us.archive.ubuntu.com/ubuntu eoan InRelease' is not signed.
E: Failed to fetch http://us.archive.ubuntu.com/ubuntu/dists/eoan/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
N: See apt-secure(8) manpage for repository creation and user configuration details.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
E: The repository 'http://us.archive.ubuntu.com/ubuntu eoan-updates InRelease' is not signed.
E: Failed to fetch http://us.archive.ubuntu.com/ubuntu/dists/eoan-updates/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
N: See apt-secure(8) manpage for repository creation and user configuration details.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
E: The repository 'http://ppa.launchpad.net/gezakovacs/ppa/ubuntu eoan InRelease' is not signed.
E: Failed to fetch http://ppa.launchpad.net/gezakovacs/ppa/ubuntu/dists/eoan/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
N: See apt-secure(8) manpage for repository creation and user configuration details.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
E: The repository 'http://us.archive.ubuntu.com/ubuntu eoan-backports InRelease' is not signed.
E: Failed to fetch http://us.archive.ubuntu.com/ubuntu/dists/eoan-backports/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
N: See apt-secure(8) manpage for repository creation and user configuration details.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
E: The repository 'http://ppa.launchpad.net/linuxgndu/sqlitebrowser/ubuntu eoan InRelease' is not signed.
E: Failed to fetch http://ppa.launchpad.net/linuxgndu/sqlitebrowser/ubuntu/dists/eoan/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
E: Failed to fetch http://us.archive.ubuntu.com/ubuntu/dists/eoan-security/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
E: The repository 'http://us.archive.ubuntu.com/ubuntu eoan-security InRelease' is not signed.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.
E: Failed to fetch http://us.archive.ubuntu.com/ubuntu/dists/eoan-proposed/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
E: The repository 'http://us.archive.ubuntu.com/ubuntu eoan-proposed InRelease' is not signed.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.

Hello there,
I don't think Ubuntu 19.0 is supported for SQL Server on Linux. According to Microsoft docs, only 16.04 is supported. You can see the details here.
Ubuntu 18.04 is supported after making these modifications.
Cheers.

Closing this issue as SQL Server is not supported on Ubuntu 19.0.

Alright thanks very much for the insights


Enable to specify guest-space-id for downloading/uploading file

opts should be passed to bare/api-call for guest space support

Thanks :)


Issue with container using flexbox

The same issue I faced with idangerous swiper as described here https://github.com/kenwheeler/slick/issues/2378
If the container has display:flex, then on resize the width is calculated incorrect. When i reload the page everything is fine.
How can I fix it ?
example https://jsfiddle.net/ValentinaSmirnova/bu5kb0c6/

Maybe you could reinitialize the Swiper when resizing?
eg.:
`
setTimeout(function() {
initSwiper();
}, 1000);
window.initSwiper = function() {
var swiper = [];

$('.swiper-container').each(function(index){

    var $el = $(this);

    swiper[index] = $el.swiper({
      //options
    });

    $(window).resize(function() {
        updateSwipers();
    });
});

}
window.updateSwipers = function() {
$('.swiper-container').each(function() {
this.swiper.update();
//this.swiper.slideTo();
//this.swiper.startAutoplay();
});
}
`
Not sure if this would help... Important is the swiper.update() function when resizing the page!

Not the Swiper issue, this is how CSS flex works


REALMC-8281: Support not having environments folder

The original bug was that not having environments would cause the import to fail because it tries to unmarshal environments directory. The original removal of the other environment directory in testdata/new_app confirmed that.
So effectively not having that directory seems to work well as a test.


Show project name in single user action table

looks good, but does the project name display later than the other fields?

looks good, but does the project name display later than the other fields?

This has been fixed now


Make open chat the default state

Defaulting to yes increases options. Consider making all seamails "open" and removing the option to opt in/out entirely.

I've been thinking about this for a few days. Defaulting to "open", great idea, we should do that. I'd also like to see an additional setting, "allow any participant to add new members", which can be toggled on and off by the chat owner.
But I think there is value in keeping the option to create a seamail thread which is permanently "closed".


Nami wallet lost or missing NFt digital pic

I bought 3 nft from CNFT market place. Then It went to my nami wallet so it was ok. Then today I log back in it was 1 is missing I dont know what happen but I submitted a complain to  CNFT to have a ticket but no Acknowledgement my submission.Even a rob return acknowledgement to my report. And Nami have no help link either.

Check your wallet address account using pool.pm to make sure that you are not just facing a display/sync issue.
Make sure that the NFT is neither under the asset tab, nor under the collectible tab.
Go in "send" screen, click on the "asset" button and make sure that the asset isn't displayed there either. (The send page load a fresh copy of your assets)

If you can't find the assets, you will need to dig your transaction history to find out what happened. You can check on Nami history tab if the NFT was ever received, and if it as been sent away later on. You can also check your wallet history on cardanoscan as a second way to verify the data.

THANK YOU IT IS NOW RESOLVED. ALL IS NOW FINE AND MORE POWER CNFT.IO


IDP - Add release notes - 0.7.0

Need 2 action items before merging

[x] https://github.com/harness/developer-hub/pull/2635 should be merged. @deepakbolangady
[x] Docs for the new actions are added. @bhavya-sinha (On-hold for next release)

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


TFLu wrong predictions for optimized model

@tensorflow/micro
System information

Host OS Platform and Distribution: macOS 10.14.16 and Ubuntu 18.04.4 LTS
TensorFlow installed from: source
Tensorflow version: TF v2.2
Target platform: ARM Mbed OS (bare-metal) on STM32L4
MCU Compiler: GCC ARM (GNU Arm Embedded Toolchain 9-2020-q2-update)

Describe the problem
Running an optimized model (see code snippet for generation below) on the MCU results in wrong predictions. Those predictions differ depending on the build profile (see more below).
Running the model locally with the TF Lite Interpreter works flawlessly.
Please provide the exact sequence of commands/steps when you ran into the problem
I'm currently benchmarking multiple models on the STM32L4 and want to verify the accuracy on the MCU itself by sending the test data and the results back and forth via UART.
This works flawlessly for the converted model without any optimizations:
converter = tf.lite.TFLiteConverter.from_keras_model(tf_model)
tflite_model = converter.convert()
open('./TFLite-model/LeNet-MNIST.tflite', 'wb').write(tflite_model)

This also works flawlessly for the model with full int8 quantization:
converter = tf.lite.TFLiteConverter.from_keras_model(tf_model)
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS_INT8]
converter.representative_dataset = yield_representative_dataset
tflite_model = converter.convert()
open('./TFLite-model/LeNET-MNIST_int8ops.tflite', 'wb').write(tflite_model)

However, the predictions are wrong when only using [tf.lite.Optimize.DEFAULT] - where only some of the weights are quantized.
converter = tf.lite.TFLiteConverter.from_keras_model(tf_model)
converter.optimizations = [tf.lite.Optimize.DEFAULT]
tflite_model = converter.convert()
open('./TFLite-model/LeNet-MNIST_optimized.tflite', 'wb').write(tflite_model)

Everything works flawlessly on the local TFLite Interpreter, and the problem only occurs with the mentioned model with and without cmsis-nn on the MCU.
Building with the release.json profile (mainly -Os) always results in the following prediction independent from the input:
[0.0 0.0 0.0 0.0 1.0 0.0 0.0 0.0 0.0 0.0]

However, building with the debug.json profile (-Og) always results in different wrong outputs but reproducible with the same input e.g.:
[0.1012007445, 0.1061911806, 0.0979489163, 0.1032304987, 0.1272625774, 0.0807816312, 0.0818515792, 0.0830543563, 0.0847685188, 0.1337099075]

Things I already tried

the model works flawlessly on the TF Lite interpreter on my host machines
the model is correctly ported to the MCU

no TFLu runtime errors
verified the input and output tensors size and datatypes (as expected float32)

there is ongoing computation on the MCU and I can even benchmark each layer individually

I currently don't have a second MCU at hand to test this.

Given that all the other models run flawlessly on the MCU and that the output depends on the compiler optimizations, I'm a bit clueless and expect some problem with TFLu.
The only difference from the optimized to the non-optimized model are the quantized weights in the fully-connected layers, I expect a problem somewhere there.
Furthermore the fully int8 quantized model runs flawlessly.
So I would restricte the error to kernels which use the int8 quantized weights, but still do the computation in float32 -- this is the only spot where I do see a difference of the model architectures.
Thankful for any feedback and pointers.
Attachements
GDrive Folder

model architecture
the original keras model
the converted optimized model which gives wrong predictions

Thanks for the report, and sorry you're hitting problems!
This definitely seems like a bug, but the behavior that I would expect here is that the kernels would report an error because they don't support the combination of float calculations and eight-bit weights. This isn't well documented, but for code size reasons we focus on either pure eight-bit activations and weights, or float activations and weights, and don't support the "hybrid" combination of float activations and eight-bit weights in TFL Micro. This combination is supported in the mobile version of TFL, which is why it works in the Python interpreter, since that's using the mobile version.
We should report an error here rather than letting the calculations continue, we should figure out which kernel is failing silently and allowing uninitialized data to flow through the graph.
To solve your problem, can we help you figure out how to convert your graph to fully quantized weights and activations? Are you seeing accuracy issues with that approach right now?

Thanks for getting back to me @petewarden.
This makes sense and confirms my observations. The pure models with int8 or either float32 activations and weights do work flawlessly.
I pasted my code snippet for a graph with fully quantized weights and activations above:
converter = tf.lite.TFLiteConverter.from_keras_model(tf_model)
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS_INT8]
converter.representative_dataset = yield_representative_dataset
tflite_model = converter.convert()
open('./TFLite-model/LeNET-MNIST_int8ops.tflite', 'wb').write(tflite_model)

The resulting model works and performs well, especially using cmsis-nn. 😊
I'm fine with either closing the issue or, as you mentioned, keep it open to track the work on an error report for the kernel. In my case the layer is probably the FULLY_CONNECTED layer.

And for anyone running TF 1.15 the tweaked conversion script below (derived from TinyML by @petewarden) also fully quantizes your h5 model...
converter = tf.lite.TFLiteConverter.from_keras_model_file('LeNet-MNIST.h5')
converter.inference_input_type = tf.lite.constants.INT8
converter.inference_output_type = tf.lite.constants.INT8
converter.optimizations = [tf.lite.Optimize.DEFAULT]

@lheim It looks like you are using an older Version of Tensorflow . Many bugs have been fixed in the latest version. Could you please execute your code using Latest Version 2.4.1 or 2.5 and let us know if the issue still persists? Thanks!

@sushreebarsa I'm not working on this project anymore, so I can't test if the problem still exists. Nonetheless, feel free to close the issue if you think it's fixed now.
Cheers!

Can someone from the community can clarify if this issue is resolved so that we can close it ? Thanks!


Support more regex types

Findutils offers

[ ] findutils-default
[ ] awk
[ ] egrep
[ ] ed
[ ] emacs
[ ] gnu-awk
[ ] grep
[ ] posix-awk
[x] posix-basic
[ ] posix-egrep
[x] posix-extended
[ ] posix-minimal-basic
[ ] sed

I think that posix-extended is the best of those, and I am glad that that's supported.
One option that's not in that list, that even find does not have, is PCRE. Can you please add support for that? (grep supports PCRE with the -P switch.)

Ok.
What do you prefer?

to use system library
keep it in this repository (personally I'd prefer it)

The first small step: #81

Probably PEG can be added to your list. E.g. using awesome https://github.com/soasme/PeppaPEG.
It would be a unique feature!

I'm a fan of PEGs (and that project name is a laugh) but I'm not sure what the utility is for bfs.  I don't expect people to search for files like bfs -peg 'name = num "." ext; num = [0-9]+; ext = "jpg" | "png";'

bfs -fpeg <filename> and users can keep PEGs somewhere.
Another great idea: fuzzy match.
Something like Reverse Engineering Sublime Text’s Fuzzy Match.

@data-man Any idea why the new test from https://github.com/tavianator/bfs/commit/bfa15314e92b298aed219788f93e41f0c7edc38f fails when building with Oniguruma?

Oh wait, I see.  Turns out that \(thing\|other\) is not actually part of POSIX basic regular expressions.  But glibc supports it as an extension.  Will revert that.


Support markdown quotes (>) in the Feedback form

When logging a piece of feedback about a page, I want to quote the sentence(s) I'm confused about. I do this in the traditional way
> quote
which should produce

quote

But instead the form encodes my input into > quote.
Describe the solution you'd like
Do not HTML-encode a > that is on its own at the start of a line.
Describe alternatives you've considered
I usually click through to the underlying Github issue and edit the > back to a > for the quote to render correctly back on the page.
Additional context
First comment on https://github.com/MicrosoftDocs/azure-docs/issues/49818 and its edit history.

@ishepherd When you refer to "the feedback form", are you referring to the Github form that opens when you click on the button at the bottom of the doc linked in this issue?

@rootsmusic No, I meant the feedback form that was on every page.
It's since been replaced with a link to github.com. That's one way to solve it :-D


JWT - Creating Routes

When you run the generator, it may start installing the packages even though you haven't responded yes to fix the conflict.. (replacing routes.js with the one for authentication).  Done() is being called Async, that is why.. need to figure out how to make it not do that.

I think this is not an issue anymore.


'NeuralNet' object has no attribute 'layers_'

I use NeuralNet 's function : save_params_to() ,but it it wrong with message:AttributeError: 'NeuralNet' object has no attribute 'layers_' . In the this class, I find the attribute 'layers' but no find 'layers' .
So Is it some thing wrong ?

i know why.
first use the NeuralNet function initalize_layers()


Restore index, foreign key and sequence constraint names

The naming can get weird or look weird when you pgosc has run multiple times on a table. It will always prefix the the prefix_key and additional run into name length limit issues.
This also applies to sequence names.
Can happen as part of cleanup.

I would like to express my heartfelt gratitude to you for your unwavering commitment to maintaining this invaluable utility. Your dedication to ensuring its continued functionality has not gone unnoticed, and I am truly appreciative of the effort you invest in this endeavor.
With the utmost respect and appreciation, I kindly inquire if there have been any recent developments or updates with regard to the previously closed pull request (PR). Your guidance and insights on this matter would be greatly valued, as your expertise plays a pivotal role in shaping the utility's performance and efficiency.
Once again, thank you ever so much for your tireless work in maintaining this utility, and I eagerly await any information you can share regarding the aforementioned PR. Your contributions are instrumental in enhancing our workflow, and your commitment to excellence is truly commendable.

So, given that we use including all to copy the original schema structure and  indexes here, PG itself chooses and ensures that the indexes are unique across table names. And, so the index name ends being pgosc_st_$TABLE_NAME_$ID_$COLUMN_key. This is because the name of the new/shadow table itself is pgosc_st_$TABLE_NAME_$ID.
I think restoring to the original index name is going to be hard because the low level id (OID) of the table changes too (since we fork a new table and default to the new table as the primary table at the end of the swap!).
However, we could at least remove pgosc_ bits from the name and make it so it is $TABLE_NAME_$COLUMN_key. So, pgosc_st_books_123abc_last_login_key -> books_last_login_key.
As noted above, we will lose the original naming, but at least the name isn't littered with the shadow table naming convention. We can do the same for sequences too.
Curious if folks have thoughts here (cc @jfrost). Tagging explicitly based on the emoji reaction
@brycethornton @sasharevzin @Shohou @sasharevzin

I'd vote for renaming the original table's indexes after swap, then rename the new table's indexes to match the original names, as @jfrost mentioned. This would work even if the user does pass the --drop flag. Is an index rename a fast operation even on a very large table?

Is an index rename a fast operation even on a very large table?
They are generally a few milliseconds as long as you don't run afoul of a locking issue. We can utilize lock_timeout with exception catching for extra safety.

I think the tricky bit is creating a map of old index names and the new index names. Since a column can have multiple indexes, and a user could've use custom naming for index, I wonder if there is a reliable way of mapping a column (or a composite/set of columns) against an index name. And accordingly derive indexes based on column names, so we can accordingly rename post swap.
Let me know in case it doesn't make sense. I will give it a bit more thought later and get back.

@shayonj I'm not sure what you're asking exactly. We have all the index names and index definitions for each table available in pg_indexes. https://www.postgresql.org/docs/current/view-pg-indexes.html
We're already grabbing that info here: https://github.com/shayonj/pg-osc/blob/aa29495e178a18fb491268d590f5a7a4bfb853cd/lib/pg_online_schema_change/query.rb#L124-L135
Are you thinking that we need to know the column names in the index in order to rename them?
We don't, we can just cache the index names and then rename them all with something like:
ALTER INDEX foo_bar_idx RENAME TO foo_bar_idx_old;

That's obviously oversimplified, but it seems doable at first glance.

Sorry for the late reply - Yeah i was talking about this line: https://github.com/shayonj/pg-osc/blob/b70ff3d6625109536ed015919e9f606061e1bbb9/lib/pg_online_schema_change/functions.rb#L48
Basically, when we create the shadow table with including all, we lose the original naming, so it will now be hard to map them.
But yes, I think what we can do is take control of the index building/adding process and that way we can build the in-memory map of the old index names and rename them again post cutover. Which should work like you mentioned.
Its a bit of work, but I think def achievable for indexes.
We will then have to do the same for sequences.


#3223 - Create history/journal tables for auditing purposes (revert)

Revert history tables creation to allow its recreation.
After the deployment to DEV the offerings history failed due to differences in the column's order that could potentially happen in the future. To resolve it a new history table creation SQL will be used to ensure the history tables will follow the same order from the env it is deployed.
The decision to revert these commits was to allow a clean history of the history table creation.

All DB migrations manually reverted on DEV.


docs: fix inline code markup

Replace markdown code markup with HTML as it can't be recognized when interleaved with the latter.
Also changed git rebase -i to git rebase --interactive to be more clear and consistent with the content above (git log --graph).

Hi @kidonng, thanks for the fixes! 😄


Add createauxblock RPC command to 1.14.x version to faciliate minining pools' adoption

Feature Request
User https://twitter.com/realalirez has been contacting mining pools in the past days. Several mining pools are not upgrading to newer builds because they do not include the createauxblock RPC command, which, in the past, it was discussed might have been added in Dogecoin Core 1.17 (see @rnicoll Post in 2019: https://rnicoll.name/dogecoin/1-17-mining/index.html). They could instead use getauxblock, but I think they are running some automated scripts across several AuxPow coins' pools and this would facilitate the process.
Describe the Feature Request
Add createauxblock/submitnewblock RPC commands, as seen in Namecoin 1.17 to make mining pools happy.
Describe Preferred Solution
Grab the changes from Namecoin 1.17+ and port them into the 1.14.x builds.
Describe Alternatives
Ask the miners to just use the old commands!
Related Code
https://github.com/namecoin/namecoin-core/blob/ca8028799cd6a8dfab63600f089f0d10a9c82d41/src/rpc/mining.cpp#L1263

I think that the current share any miner has is irrelevant to the decision whether to include this or not. We knew that it had gone missing for a little over a month now, and we hadn't prioritized it because we had no indication that anyone needed it right now. Now we do, so we should act accordingly: the request is there, let's not procrastinate.

I think the update might not be trivial

It's relatively straightforward because we don't need to change existing logic to bring these in. The functionality can be isolated now and if we want to consolidate later, we can do that at a time that we can be more confident about the quality of the code. We can tag it "experimental".

might require a lot of non trivial and prolonged testing

It will need a good rpc test and it needs to be compared with what btccom@46c85c5 does - either to follow that as close as possible, or maybe it should just be derived from that. Prolonged testing, always, but waiting with the release of a feature like this will not help testing at all - it will just delay testing and create barriers. My requirement was to depreciating getauxblock, not so much to re-introducing these methods.
TLDR; Rocket-science detectors are sitting at a hard zero. Let's do this.

On the two quirks

About the first quirk: like I said it's a feature that I think makes sense and it doesn't hurt anyone that uses a single address, the only overhead introduced for those that don't use it is vectorization overhead, and a 3-line patch.

I don't think it's crazy to ask them to support adding a _ in if needed/remove if not needed.

Re-reading bullet point 1 on the v1.14.2 release notes and your work in dbfb33ab shows a bit of a different story: the underscore made it into 1.14.0 and then there were issues (I was told, also by you, that the issues were raised directly by miners) and it was reverted back without the underscore, i.e. following Dogecoin API instead of Namecoin API. Then if I add to that the fact that some miners run a separately patched/maintained Dogecoin Core just because those Namecoin RPC methods were removed, I feel that it shows that not being inclusive may lead to difficulties when you want to get something adapted - like, say, a fee policy change (to just make up a crazy example on the spot.)
If then we're going to take a "hard stand" and make an issue about diverging APIs between 2 coins, the thing that - based on the past outcome - will likely happen is that someone will just fork and change the API to what they need it to be. Kind of like what was done now, because there is no rule that says you have to run the exact version that is offered here. I personally don't do that either.
So the question then becomes: is facilitating optional compatibility with Namecoin's API worth the effort? I think the effort is low, especially when we stop doing these endless series of rewrites - which we're discussing in #2273 - and just have a stable
base which we actually maintain. We're talking 30 lines of code and about the same quantity of qa test code, if that. If that prevents creating contention with/between miners and making their upgrades easier, then I think that's a reasonable thing to do. But that's just my take...
I guess what it comes down to is: do developers dictate or enable? Facilitate or rule? Do we draw a line around the majority and say "that's it", or do we try to make the majority as large as possible?

@michilumin Since this was starting to look stalled, I have pulled in your work from michilumin@1c5b9b3, added an extensive RPC test and fixed the issue with the address cache (retaining your commit plus some fixes). Will open PR shortly.

This is brought back now and #2654 should make it even easier for pools that were using the btccom version to migrate to 1.14.5 - I'm closing this.


[Event Request] Codeunit 5940 SevContractManagement

About 1 year ago, I requested serval events in codeunit 5940 ServContractManagement (Link) Now I discovered, that the parameters are not correct based on the request. The request itselv was inconsitent. The two events (10 OnBeforeServLedgEntryToServiceLine /11 OnServLedgEntryToServiceLineOnBeforeServLineInsert) do not pass the right Service Ledger Entry. Actually, the global Service Ledger Entry ServLedgEntry is passed. But what we need is the local Service Ledger Entry ServiceLedgerEntry - my request has mixed those two values.
To fix the issue, I see two options.
Option 1 - Fix
Rate the problem as a bug and replace the variable ServLedgEntry with ServiceLedgerEntry in both function calls.
Optioion 2 - Extend
Rate the problem as an additional request and extend both events with an additional Service Ledger Entry parameter.
local procedure ServLedgEntryToServiceLine(var TotalServLine: Record "Service Line"; var TotalServLineLCY: Record "Service Line"; ServHeader: Record "Service Header"; ServiceLedgerEntry: Record "Service Ledger Entry"; ContractNo: Code[20]; InvFrom: Date; InvTo: Date)
var
    StdText: Record "Standard Text";
    IsHandled: Boolean;
begin
>>> Existing Code
    OnBeforeServLedgEntryToServiceLine(TotalServLine, TotalServLineLCY, ServHeader, ServLedgEntry, IsHandled);
>>> New Code Option 1
    OnBeforeServLedgEntryToServiceLine(TotalServLine, TotalServLineLCY, ServHeader, ServiceLedgerEntry, IsHandled);
>>> New Code Option 2
    OnBeforeServLedgEntryToServiceLine(TotalServLine, TotalServLineLCY, ServHeader, ServLedgEntry,  ServiceLedgerEntry, IsHandled);

    if IsHandled then
        exit;
.
.
.
.
        "Shortcut Dimension 1 Code" := ServiceLedgerEntry."Global Dimension 1 Code";
        "Shortcut Dimension 2 Code" := ServiceLedgerEntry."Global Dimension 2 Code";
        "Dimension Set ID" := ServiceLedgerEntry."Dimension Set ID";

>>> Existing Code
        OnServLedgEntryToServiceLineOnBeforeServLineInsert(ServLine, TotalServLine, TotalServLineLCY, ServHeader, ServLedgEntry);
>>> New Code Option 1
        OnServLedgEntryToServiceLineOnBeforeServLineInsert(ServLine, TotalServLine, TotalServLineLCY, ServHeader, ServiceLedgerEntry);
>>> New Code Option 2
        OnServLedgEntryToServiceLineOnBeforeServLineInsert(ServLine, TotalServLine, TotalServLineLCY, ServHeader, ServLedgEntry, ServiceLedgerEntry);

        Insert;
        CreateDim(
            DimMgt.TypeToTableID5(Type), "No.",
            DATABASE::Job, "Job No.",
            DATABASE::"Responsibility Center", "Responsibility Center");
    end;
end;

Thank you for your collaboration.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Append to the imported struct field cause a panic

The following program sample.go triggers a panic:
If there is an imported struct in a script that has a slice field - you can't append to that slice because it causes a panic
package main

import (
	"fmt"
	"reflect"

	"github.com/traefik/yaegi/interp"
)

type BarParams struct {
	Arr []string
}

const script = `
package bar

import "foo"

func Bar(p foo.BarParams) string {
	p.Arr = append(p.Arr, "4")
	return "ok"
}
`

func main() {
	i := interp.New(interp.Options{})

	custom := make(map[string]map[string]reflect.Value)
	custom["foo"] = make(map[string]reflect.Value)
	custom["foo"]["BarParams"] = reflect.ValueOf((*BarParams)(nil))
	i.Use(custom)

	_, err := i.Eval(script)
	handle(err)

	v, err := i.Eval("bar.Bar")
	handle(err)

	bar := v.Interface().(func(BarParams) string)

	result := bar(BarParams{Arr: []string{"1", "2", "3"}})
	fmt.Println(result)
}

func handle(err error) {
	if err != nil {
		panic(err)
	}
}

Expected result:
ok

Got:
panic: runtime error: invalid memory address or nil pointer dereference

goroutine 1 [running]:
main.handle(...)
	/home/evgen/go/src/elopatkin/main.go:47
main.main()
	/home/evgen/go/src/elopatkin/main.go:34 +0x4cd
exit status 2

Also slice fields can't be used with a spread operator. Change the script from prev example:
const script = `
package bar

import "foo"

func Bar(p foo.BarParams) string {
	_ = append([]string{}, p.Arr...)
	return "ok"
}
`

Got:
7:13: panic
panic: reflect.Set: value of type []string is not assignable to type string [recovered]
	panic: reflect.Set: value of type []string is not assignable to type string

goroutine 1 [running]:
elopatkin/vendor/github.com/traefik/yaegi/interp.runCfg.func1(0xc000112460, 0xc000119900)
	/home/evgen/go/src/elopatkin/vendor/github.com/traefik/yaegi/interp/run.go:124 +0x20c
panic(0x670a80, 0xc000010bc0)
	/usr/local/go/src/runtime/panic.go:969 +0x166
reflect.Value.assignTo(0x66d620, 0xc00000c880, 0x197, 0x6ced8b, 0xb, 0x670a80, 0x0, 0x97, 0xc00000c940, 0x66d620)
	/usr/local/go/src/reflect/value.go:2403 +0x426
reflect.Value.Set(0x670a80, 0xc000010bb0, 0x198, 0x66d620, 0xc00000c880, 0x197)
	/usr/local/go/src/reflect/value.go:1532 +0xbd
reflect.Append(0x66d620, 0xc00000c8a0, 0x197, 0xc000149a70, 0x1, 0x1, 0x66d620, 0xc00000c880, 0x197)
	/usr/local/go/src/reflect/value.go:2028 +0xe8
elopatkin/vendor/github.com/traefik/yaegi/interp._append.func2(0xc000112460, 0xc00005ecc0)
	/home/evgen/go/src/elopatkin/vendor/github.com/traefik/yaegi/interp/run.go:2469 +0x14c
elopatkin/vendor/github.com/traefik/yaegi/interp.runCfg(0xc000119900, 0xc000112460)
	/home/evgen/go/src/elopatkin/vendor/github.com/traefik/yaegi/interp/run.go:130 +0x6d
elopatkin/vendor/github.com/traefik/yaegi/interp.genFunctionWrapper.func2.1(0xc00000c840, 0x1, 0x1, 0xc00000c840, 0xc00005f050, 0x13)
	/home/evgen/go/src/elopatkin/vendor/github.com/traefik/yaegi/interp/run.go:716 +0x3cb
reflect.callReflect(0xc00005f050, 0xc000149dc8, 0xc000149db0)
	/usr/local/go/src/reflect/value.go:549 +0x322
reflect.makeFuncStub(0xc00005f080, 0x3, 0x3, 0x1, 0x67a120, 0xc00005f050, 0x67a120, 0xc00005f050, 0x13, 0x0, ...)
	/usr/local/go/src/reflect/asm_amd64.s:20 +0x42
main.main()
	/home/evgen/go/src/elopatkin/main.go:41 +0x3a9
exit status 2

I don't know yet what exactly you're doing wrong, but I don't believe that a yaegi bug is involved for now, as one can successfully do something equivalent, such as:
package main

import (
	"fmt"
	"log"

	"github.com/traefik/yaegi/interp"
	"github.com/traefik/yaegi/stdlib"
)

const fooSrc = `package foo

type BarParams struct {
	Arr []string
}
`

const barSrc = `package bar

import (
	"fmt"
	"foo"
)

func Bar(p *foo.BarParams) string {
	p.Arr = append(p.Arr, "4")
	return "ok"
}
`

func main() {
	i := interp.New(interp.Options{})
	i.Use(stdlib.Symbols)

	if _, err := i.Eval(fooSrc); err != nil {
		log.Fatal(err)
	}
	// fmt.Println(i.Symbols("foo"))

	if _, err := i.Eval(barSrc); err != nil {
		log.Fatal(err)
	}

	v, err := i.Eval(`bar.Bar`)
	if err != nil {
		log.Fatal(err)
	}

	bar := v.Interface().(func(*struct { Arr []string }) string)

	result := bar(&struct { Arr []string }{Arr: []string{"1", "2", "3"}})
	fmt.Println(result)
}

In my example i export a BarParams struct from the golang code. That is a difference. To reproduce you must declare some struct with a slice field and then import it inside yaegi script using a Use method. Then append something to that slice field inside yaegi.

In my example i export a BarParams struct from the golang code. That is a difference. To reproduce you must declare some struct with a slice field in your golang code and then import it inside yaegi script using a Use method. Then append something to that slice field inside yaegi.
Not
_, err := i.Eval(stringWithBarDefinition)

But
custom["foo"]["BarParams"] = reflect.ValueOf((*BarParams)(nil))
i.Use(custom)

Sure, but that's what I meant: i'm not yet convinced that what you're doing here is enough/correct/supported etc. It might not be enough to just manually add foo.BarParams as you did to the symbols before you can use it. It might be the wrong scope, etc. I don't know yet.

Alright, just to confirm that it can be done, here's a working example closer to what you were trying to achieve:
package main

import (
	"fmt"
	"log"
	"reflect"

	"github.com/traefik/yaegi/interp"
	"github.com/traefik/yaegi/stdlib"
)

var Bar string

const barSrc = `package bar

import (
	"foo"
)

func Bar() string {
	foo.Bar = "Hello"
	return foo.Bar
}
`

func main() {
	i := interp.New(interp.Options{})
	i.Use(stdlib.Symbols)

	exports := interp.Exports{"foo": map[string]reflect.Value{"Bar": reflect.ValueOf(&Bar).Elem()}}
	i.Use(exports)

	if _, err := i.Eval(barSrc); err != nil {
		log.Fatal(err)
	}

	v, err := i.Eval(`bar.Bar`)
	if err != nil {
		log.Fatal(err)
	}

	bar := v.Interface().(func() string)
	result := bar()
	fmt.Println(result)
}

So we're marking that bug as a question for now. That said:

We acknowledge that it is a pain to achieve, and we're going to work on an easier way to do it. Specifically, something that will probably replace that part:

	exports := interp.Exports{"foo": map[string]reflect.Value{"Bar": reflect.ValueOf(&Bar).Elem()}}
	i.Use(exports)

We acknowledge there might be bugs for more complicated cases such as yours (e.g. with types, and composite literals, etc), and we'll investigate them while we're doing 1) (or after it's done).

Sorry, but i can't understand your prev. example. It do something different.
To reproduce a bug you should specify struct in your golang code and then Use it in you yaegi code. This struct must contain a slice field. When you append to this slice inside yaegi a panic will cause.
Yaegi has an API to import golang functions and structs, but it has a bug. In your example there are no:

struct inporting
slice field appending

So it is completely wrong. And bug still exist.

Sorry, but i can't understand your prev. example. It do something different.

Yes it does, we never claimed it was identical.

To reproduce a bug you should specify struct in your golang code and then Use it in you yaegi code. This struct must contain a slice field. When you append to this slice inside yaegi a panic will cause.

And I will reiterate that I am not yet convinced that you are properly "importing" your BarParams type into the symbols of yaegi, which would be the reason why you are seeing this bug.
And which is why we said in my previous reply that we are going to work on a way to facilitate that and to make sure it is done properly.

Yaegi has an API to import golang functions and structs, but it has a bug. In your example there are no:

struct importing
slice field appending

Yes, I know.

So your previous example is completely wrong.

No, it is not wrong, it is correct, but it is different.

And a bug still exists.

Maybe. We already acknowledged that in the previous message.


Document transform()-based test

Including two documentation-related points in https://github.com/xspec/xspec/pull/892#pullrequestreview-417664807

Added a Wiki page.


update documents again.

Hi, I solved problem, so I again send pr!

@lamberta  I did it!

Hi @seo-inyoung ,
Thanks for contributing.
Please aggregate commits.
Because it notify all members on this repo every commit. :)

Hi @seo-inyoung
Please check @jaketae's comment. You don't have to remove the disclaimer header.

Thank you @seo-inyoung
As mentioned on the mailing list, we're directing the community to work alongside vendors in our GitLocalize project. It's the only way to make sure the translations are in sync with the updated source material. The community can submit pull requests through GitLocalize and provide reviews.
Can you add you changes to these files? These two look pretty in-sync:

https://gitlocalize.com/repo/4592/ko/site/en-snapshot/tutorials/load_data/images.ipynb
https://gitlocalize.com/repo/4592/ko/site/en-snapshot/tutorials/load_data/text.ipynb

But this one needs some work:

https://gitlocalize.com/repo/4592/ko/site/en-snapshot/tutorials/keras/text_classification.ipynb

Closing this PR but please let me know how the GitLocalize contribution experience is. Thanks


Hard drive backups.

 I'm using RPI-clone on RPI3 every day and  you've put me off using other boards that may not work with this, it is so useful. I maintain a distrust of SDs even though I can now easily back them up (when I'm there). I spend 6 months in the UK and 6 months in Spain and I need my kit to work when I'm not there. I use:
sudo pi-clone -U sda
I'm backing up from sd to sd in a usb adaptor and it works every time.
Q1: If I change to booting and running from a hard drive (USB2, assuming I can find split leads that will let this happen power wise..) is the process the same for backing up from that to another USB hard drive?
Also is there an option to change IP address as part of the process?
Thanks for your work on this sterling software.
Pete

I continue to sing the praises of rpi-clone  as I use it daily on my RPI3 SD setups. I did not manage to get a response however on HD use. As I'm moving to RPI4, can anyone advise. I currently use "sudo rpi-clone -U sda" unless it's a new SD in which case I use "sudo rpi-clone -f -U sda"
Imagine a scenario where FIRSTLY one might move the entire installation to hard disk - and THEN  looking ahead, copy disk to disk.  What is the least change to my current routine to achieve this?

Did you find a solution to this problem? I'm in the same boat. Thanks in advance.


Remove extra space from doc comment in Blob.mo

There is a rendering issue on https://mops.one/base/docs/Blob due to this extra space.

Thanks!


Remove ClearML and Prefect as core services

This is a meta issue to track work for 0.4.2 Milestone

ClearML is simple to test but Prefect is much more challenging.

this needs to be checked as well #878

Clearml seems to need GCC compiler when installing the daemon agent in a conda-store env, also the inbuilt qhub agent called services is failing during init of a job -- needs to debug (see an error bellow), the access and dashboard are working as expected.
Running Task 682367ffa7ac4fa9adec0c134d62be21 inside default docker:  arguments: []
2022-05-13 17:40:21
Executing: ['docker', 'run', '-t', '-l', 'clearml-worker-id=clearml-services:service:682367ffa7ac4fa9adec0c134d62be21', '-l', 'clearml-parent-worker-id=clearml-services', '-e', 'NVIDIA_VISIBLE_DEVICES=none', '-e', 'CLEARML_WORKER_ID=clearml-services:service:682367ffa7ac4fa9adec0c134d62be21', '-e', 'CLEARML_DOCKER_IMAGE=', '-e', 'CLEARML_TASK_ID=682367ffa7ac4fa9adec0c134d62be21', '-v', '/tmp/.clearml_agent.8j40rjqv.cfg:/root/clearml.conf', '-v', '/root/.clearml/apt-cache:/var/cache/apt/archives', '-v', '/root/.clearml/pip-cache:/root/.cache/pip', '-v', '/root/.clearml/pip-download-cache:/root/.clearml/pip-download-cache', '-v', '/root/.clearml/cache:/clearml_agent_cache', '-v', '/root/.clearml/vcs-cache:/root/.clearml/vcs-cache', '--rm', '', 'bash', '-c', 'echo \'Binary::apt::APT::Keep-Downloaded-Packages "true";\' > /etc/apt/apt.conf.d/docker-clean ; chown -R root /root/.cache/pip ; export DEBIAN_FRONTEND=noninteractive ; export CLEARML_APT_INSTALL="$CLEARML_APT_INSTALL libsm6 libxext6 libxrender-dev libglib2.0-0" ; [ ! -z $(which git) ] || export CLEARML_APT_INSTALL="$CLEARML_APT_INSTALL git" ; declare LOCAL_PYTHON ; [ ! -z $LOCAL_PYTHON ] || for i in {15..5}; do which python3.$i && python3.$i -m pip --version && export LOCAL_PYTHON=$(which python3.$i) && break ; done ; [ ! -z $LOCAL_PYTHON ] || export CLEARML_APT_INSTALL="$CLEARML_APT_INSTALL python3-pip" ; [ -z "$CLEARML_APT_INSTALL" ] || (apt-get update -y ; apt-get install -y $CLEARML_APT_INSTALL) ; [ ! -z $LOCAL_PYTHON ] || export LOCAL_PYTHON=python3 ; $LOCAL_PYTHON -m pip install -U "pip<20.2" ; $LOCAL_PYTHON -m pip install -U clearml-agent ; cp /root/clearml.conf /root/default_clearml.conf ; NVIDIA_VISIBLE_DEVICES=none $LOCAL_PYTHON -u -m clearml_agent execute --full-monitoring  --id 682367ffa7ac4fa9adec0c134d62be21']
2022-05-13 17:40:26
docker: invalid reference format.
See 'docker run --help'.

At this point, many on the team believe it is time to remove these integrations from Nebari. They can still be used if properly included in the helm_extension section of the nebari-config.yaml.
Scheduled for removal in the March release.

@iameskild - Can we close this issue as complete?


window.cordova.InAppBrowser.open undefined

Hey there,
I'm trying to use this plugin but I'm getting the error:
undefined is not an object (evaluating 'window.cordova.InAppBrowser.open')

Any ideas? I have the inappbrowser plugin installed -- it works via window.open and ngCordova's $cordovaInAppBrowser method
A note: I'm definitely using this library and not ngCordova's version
Thanks!

What provider are you trying to use?

stripe, but I believe it will throw this error on every provider

Try a different one just for the heck of it.  A more popular one like Facebook or Google.  Trying to narrow things down.

Just tried with facebook -- same deal -- if it helps I get the same error when trying to use ngCordova's version of the oauth plugin

What version of Apache Cordova are you using and are you using Android or iOS?

Cordova version 5.4.1 -- iOS

It appears this was caused by trying to access the ng-cordova-oauth functionality before the deviceReady event had been called. Everything appears to work -- sorry for the mistake

I wrapped up the call within deviceready, yet it doesn't seem to work. I tried to log InAppBrowser object but it shows as undefined. However, isInAppBrowserInstalled() function returns true! Not really sure what's the issue? :/

Strange, try to remove and install again the inAppBrowserPlugin and try your code again.

Been there. Done that. No help. I guess my InAppBrowser plugin has got corrupted. Anyways, removing the platform and adding it again helped. This might help someone else too. Also, I had updated my Cordova before doing this so if re-adding platform doesn't help, update Cordova as well!

:+1:

I had the same issue and upgrading the cca to version 0.8.1 solved the issue!
Thanks @cseer90 for the help... Spent a whole day without luck until I found this solution!

It was the same for me. I removed the ios platform and then I added it and it came back to work.
First I tried to remove and add the plugin but it was not helped.

I'm facing same issue, In my app we used iOS platform version 4.1.0. As per your above suggestion I removed and added the platform still having the same issue, kindly someone help below issue for Cordova iOS InAppBrowser.
typeerror undefined is not an object(evaluating.cordova.inappbrowser.open)


Unable to modify/view certain taks

Describe the bug

I've encountered a weird bug with v.2.3.2. I'm unable to edit or open some of the tasks shown in the list. They are visible, and there is nothing special about them, but when i use arrow keys to select this task and try to open using ENTER, nothing happens. None of the keyboard shortcuts work either.
Currently i have ~20 tasks in the list. And 4 of them have this issue.
To Reproduce

I have no clue how to reproduce this. If I add 10 new tasks, maybe 2 of them will have this issue. the affected ones will be forever in this state until i manually delete them using task cli.
Here is one such task,
❯ task 20

Name          Value
ID            20
Description   psuite auth
Status        Pending
Project       homelab
Entered       2023-11-18 09:48:14 (4w)
Last modified 2023-11-18 09:55:33 (4w)
Virtual tags  PENDING PROJECT READY UDA UNBLOCKED
UUID          5a19c75b-64b8-4cf6-b800-e829a2d58c93
Urgency       1.159
folder        someday

    project      1 *    1 =      1
    age      0.079 *    2 =  0.159
                            ------
                             1.159

Date                Modification
2023-11-18 09:55:33 Project set to 'homelab'.
                    Folder changed from 'in' to 'someday'.

is there something special about this task which prevents it from being edited in VIT?
This is a fresh installation, and config.ini is empty
Expected behavior

I can modify/view/delete any tasks in vit
Test case
If reproducing your issue requires any TaskWarrior setup, please produce a test case script.

Unfortunately I cannot fix bugs that I cannot reproduce.
There is nothing about the task that would indicate an issue, and I've never experienced this issue before.
If you can use https://github.com/vit-project/vit/blob/2.x/scripts/generate-dummy-install.sh to create a reproducible bug, then I can investigate.
The only other suggestion I have is a completely fresh install on another machine, as a way to determine if it's something in your environment that is causing an issue, such as overriding key press events.

Thanks for pointing to the script. That helped. I was able to find the issue by starting with a fresh setup using the script and copying my config&data incrementally.
It looks like my Syncthing somehow messed up syncing the files. Certain tasks ended up in both completed.data and pending.data files. Task cli was able to manage it without any issues. I assume it doesn't look up the uuid in completed.data unless specifically asked to.
It is easy to reproduce the issue. Just run the dummy install script and copy pending.data to completed.data. Open VIT and you won't be able to interact with the tasks anymore. Running task command will work fine.
Anyway I was able to fix mine by deleting the problematic entries from completed.data manually. I wouldn't consider this a "bug" in VIT since the data is not supposed to be that way. But maybe it is interesting for you to check the logic on why it happened though. I'll let you decide and close the issue..
Thanks!


Add Comprehensive Contributing Section to README

This pull request introduces a detailed "Contributing" section to the README file of the project. The aim of this update is to provide clear and accessible guidance for potential contributors, encouraging more community involvement and streamlined contribution processes.
Key Points of the Contribution:
Reporting Bugs: Guidelines on how to report issues effectively.
Submitting Enhancements: Instructions on how to suggest project improvements.
Pull Requests: Steps for creating and submitting pull requests.
Code Review and Community Engagement: Encouragement for active participation in the community and code review processes.
Staying Updated: Tips on keeping forks up-to-date with the main repository.
Objective:
The addition of this Contributing section is designed to foster a welcoming and productive environment, making it easier for contributors to understand how they can participate in the development and maintenance of the project. This is essential for growing the project's community and ensuring high-quality contributions.

https://github.com/primer/react/blob/main/contributor-docs/CONTRIBUTING.md


Readme Update

This PR fixes the issue #808
Added the brief about following Usescases :
Use Case 1: Autotune - Config Recommendation for a User Provided Performance Objective
Use Case 2: Hyper Parameter Optimization as a Service - Performance
Use Case 3: Monitor a deployment over a long term and provide recommendations to help reduce cost (Production scenario with no experiment trials)
Remote (ROS)
Local (ACM)
Use Case 4: Provide tooling for automating performance regressions instead of manual testing (A/B/n testing as part of the build pipeline)

Closing stale PRs


Go to definition and other functionality doesn't work in Rust files

Here is the relevant part of my .kakrc:
plug "ul/kak-lsp" do %{
    cargo build --release --locked
    cargo install --force --path .
} config %{
    set global lsp_diagnostic_line_error_sign '║'
    set global lsp_diagnostic_line_warning_sign '┊'

    define-command ne -docstring 'go to next error/warning from lsp' %{ lsp-find-error --include-warnings }
    define-command pe -docstring 'go to previous error/warning from lsp' %{ lsp-find-error --previous --include-warnings }
    define-command ee -docstring 'go to current error/warning from lsp' %{ lsp-find-error --include-warnings; lsp-find-error --previous --include-warnings }

    define-command lsp-restart -docstring 'restart lsp server' %{ lsp-stop; lsp-start }
    hook global WinSetOption filetype=(c|cpp|cc|rust|javascript|typescript) %{
        map window user "l" ": enter-user-mode lsp<ret>" -docstring "LSP mode"
        set-option window lsp_auto_highlight_references true
        set-option window lsp_hover_anchor false
        lsp-auto-hover-enable
        echo -debug "Enabling LSP for filtetype %opt{filetype}"
        lsp-enable-window
    }

    hook global WinSetOption filetype=(rust) %{
        set window lsp_server_configuration rust.clippy_preference="on"

        hook window BufWritePre .* lsp-formatting-sync

        hook window -group rust-inlay-hints BufReload .* rust-analyzer-inlay-hints
        hook window -group rust-inlay-hints NormalIdle .* rust-analyzer-inlay-hints
        hook window -group rust-inlay-hints InsertIdle .* rust-analyzer-inlay-hints
        hook -once -always window WinSetOption filetype=.* %{
            remove-hooks window rust-inlay-hints
        }
    }

    hook global KakEnd .* lsp-exit
}

The readme has the startup code as
eval %sh{kak-lsp --kakoune -s $kak_session}
hook global WinSetOption filetype=(rust|python|go|javascript|typescript|c|cpp) %{
    lsp-enable-window
},

but when I add the eval line, I get an error message about the lsp-start command already being defined, and the linked wiki page about configuration with plug.kak says that lsp-enable-window takes care of it anyway.
When I open a Rust file with this configuration, I can tell that the language server is working in some capacity because I get the warning and error markings in the left margin and I can do things like list all errors and lsp capabilities. But I get no inlay hints or hover functionality, “go to definition” doesn’t work, and “show code actions” shows nothing. I have been completely unable to diagnose the problem.

What you have in the first listing should be enough as, iirc, plug.kak sources lsp.kak automatically, when you do eval you essentially request it again from kak-lsp. I believe @andreyorst can provide a better explanation of how to configure kak-lsp with plug.kak.

correct, plug,kak loads all scripts automatically, so you don't need to use
eval part
пн, 15 июн. 2020 г., 07:35 Ruslan Prokopchuk<EMAIL_ADDRESS>
What you have in the first listing should be enough as, iirc, plug.kak
sources lsp.kak automatically, when you do eval you essentially request
it again from kak-lsp. I believe @andreyorst
https://github.com/andreyorst can provide a better explanation of how
to configure kak-lsp with plug.kak.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ul/kak-lsp/issues/341#issuecomment-643896972, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEUROT5WOKNJ6WJ2OROHK6LRWWQHHANCNFSM4NHA6PMQ
.

here's my config. Note that I start lsp with lsp-enable-window command in a filetype hook inside config block

here's my config. Note that I start lsp with lsp-enable-window command in a filetype hook inside config block

I get the same result with this config.

There might be a problem with rls, maybe a log can help:
set global lsp_cmd "kak-lsp -s %val{session} -vvv --log /tmp/kak-lsp.log".
If you are usin rustup then you probably want to use rustup which rls; this is the current default config:
[language.rust]
filetypes = ["rust"]
roots = ["Cargo.toml"]
command = "sh"
args = ["-c", "if command -v rustup >/dev/null; then $(rustup which rls); else rls; fi"]

This seems to have gone away upon reinstalling rls.


Revoked access does not publish to SNS queue

Describe the bug
I've built a wrapper for the SNS notifications, and revocation messages appear to be vanishing. I've invoked the teamNotifications-main Lambda with the payload from TEAM-Revoke-SM-main - Notify Requester Session Ended, and the lambda reports success, but the message never reaches SNS.
To Reproduce
Steps to reproduce the behavior:

Request access
Approve access
Wait for access to be assigned
Revoke active access

Expected behavior
An SNS message should be published to the topic TeamNotifications-main, with the details of the revocation.
Additional context
It appears that the switch case in the lambda itself is missing the "revoked" status:
https://github.com/aws-samples/iam-identity-center-team/blob/main/amplify/backend/function/teamNotifications/src/index.py
It would be helpful to add logging to this function as well

@robbycuenot looking into this.  I am also interested in the wrapper you have built for the SNS notification. How are you using it today ?


Wrong command name in README and bug on diagram generation

Hi,
First there's a typo in the README, the command is not solidity-code-gen but solidity-diagram-gen.
Plus, when I run solidity-diagram-gen ./contract on my contract folder (the folder I use is the contract folder of this repo) I get the following bug
Generating diagram...

/usr/local/lib/node_modules/solidity-diagram-gen/node_modules/solidity-parser/index.js:39
      throw e;
      ^
SyntaxError: Expected "!", "!=", "(", "+", "++", "-", "--", "0", "<", "<=", "==", ">", ">=", "[", "^", "delete", "false", "hex", "mapping", "new", "this", "true", "v", "~", [1-9], comment, end of line, identifier, number, string, or whitespace but "," found. Line: 55, Column: 6
    at peg$buildStructuredError (/usr/local/lib/node_modules/solidity-diagram-gen/node_modules/solidity-parser/build/parser.js:1306:12)
    at Object.peg$parse [as parse] (/usr/local/lib/node_modules/solidity-diagram-gen/node_modules/solidity-parser/build/parser.js:14485:11)
    at Object.parse (/usr/local/lib/node_modules/solidity-diagram-gen/node_modules/solidity-parser/index.js:34:23)
    at Object.parseFile (/usr/local/lib/node_modules/solidity-diagram-gen/node_modules/solidity-parser/index.js:45:17)
    at /usr/local/lib/node_modules/solidity-diagram-gen/src/solidity-diagram-gen.js:22:42
    at Array.forEach (<anonymous>)
    at processSolFiles (/usr/local/lib/node_modules/solidity-diagram-gen/src/solidity-diagram-gen.js:21:15)
    at Walker.klaw.on.on (/usr/local/lib/node_modules/solidity-diagram-gen/src/solidity-diagram-gen.js:407:30)
    at Walker.emit (events.js:182:13)
    at endReadableNT (_stream_readable.js:1092:12)

I found that the line responsible for this bug is the following: (,,,,,,,,,kittyDna) = kittyContract.getKitty(_kittyId); which is a valid solidity syntax.

Hello, @nkthiebaut
Thanks for noticing the typo.  #6 fixed this.
The bug reported occurs inside the solidity-diagram-gen npm module. I'll attempt to solve it.

Receiving the same error when I try to run it.  Is there a fix for this ?  Would love to try this out

Name nguyentiensy .cảm ơn các ban🥰

Hi,
First there's a typo in the README, the command is not solidity-code-gen but solidity-diagram-gen.
Plus, when I run solidity-diagram-gen ./contract on my contract folder (the folder I use is the contract folder of this repo) I get the following bug
Generating diagram...

/usr/local/lib/node_modules/solidity-diagram-gen/node_modules/solidity-parser/index.js:39
      throw e;
      ^
SyntaxError: Expected "!", "!=", "(", "+", "++", "-", "--", "0", "<", "<=", "==", ">", ">=", "[", "^", "delete", "false", "hex", "mapping", "new", "this", "true", "v", "~", [1-9], comment, end of line, identifier, number, string, or whitespace but "," found. Line: 55, Column: 6
    at peg$buildStructuredError (/usr/local/lib/node_modules/solidity-diagram-gen/node_modules/solidity-parser/build/parser.js:1306:12)
    at Object.peg$parse [as parse] (/usr/local/lib/node_modules/solidity-diagram-gen/node_modules/solidity-parser/build/parser.js:14485:11)
    at Object.parse (/usr/local/lib/node_modules/solidity-diagram-gen/node_modules/solidity-parser/index.js:34:23)
    at Object.parseFile (/usr/local/lib/node_modules/solidity-diagram-gen/node_modules/solidity-parser/index.js:45:17)
    at /usr/local/lib/node_modules/solidity-diagram-gen/src/solidity-diagram-gen.js:22:42
    at Array.forEach (<anonymous>)
    at processSolFiles (/usr/local/lib/node_modules/solidity-diagram-gen/src/solidity-diagram-gen.js:21:15)
    at Walker.klaw.on.on (/usr/local/lib/node_modules/solidity-diagram-gen/src/solidity-diagram-gen.js:407:30)
    at Walker.emit (events.js:182:13)
    at endReadableNT (_stream_readable.js:1092:12)


Updated elastic query to throw Exception if request to elastic was pr…

…ocessed successful but returned not 200

PR on hold


AttributeError

Assalam and hi,
I just try to write code from the malaya documentation prepared for segmentation
Model = malaya.segmentation.transformer(model = 'small')
But its appear the error :
AttributeError: module 'malaya' has no attribute 'segmentation'
The error also occur for verterbi algorithm
Thanks alot for your response

Tested with latest version, make sure you installed latest version,
import malaya
malaya.__version__
-> '<IP_ADDRESS>'

malaya.segmentation.transformer
-> <function transformer at 0x7f4e779fcf28>

Tested with latest version, make sure you installed latest version,
import malaya
malaya.__version__
-> '<IP_ADDRESS>'

malaya.segmentation.transformer
-> <function transformer at 0x7f4e779fcf28>

Ok thanks, problem solved
On Sun, 17 Jan 2021 at 13:35, HUSEIN ZOLKEPLI<EMAIL_ADDRESS>wrote:

Tested with latest version, make sure you installed latest version,
import malayamalaya.version-> '<IP_ADDRESS>'
malaya.segmentation.transformer-> <function transformer at 0x7f4e779fcf28>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/huseinzol05/malaya/issues/74#issuecomment-761737851,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARQNL7EWMSRW2DZERUS3ASLS2JZLZANCNFSM4WFZU65Q
.

Ok thanks, problem solved
On Sun, 17 Jan 2021 at 13:35, HUSEIN ZOLKEPLI<EMAIL_ADDRESS>wrote:

Tested with latest version, make sure you installed latest version,
import malayamalaya.version-> '<IP_ADDRESS>'
malaya.segmentation.transformer-> <function transformer at 0x7f4e779fcf28>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/huseinzol05/malaya/issues/74#issuecomment-761737851,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARQNL7EWMSRW2DZERUS3ASLS2JZLZANCNFSM4WFZU65Q
.


fix: rebranding to apl

Linked PRs:
https://github.com/linode/apl-api/pull/555
https://github.com/linode/apl-console/pull/433


instantiate classes, add test case

The tests will now properly instantiate classes instead of calling PigLatin::translate(); directly. This allows the usage of member variables etc. in solutions. I've also added a test case as proposed in another pull request to x-common to check for proper implementation of the consonant+qu rule.

Hi @petemcfarlane, I thought it would be better because you might want to break your function up into multiple functions. If you do so, you would also have to call PigLatin::myFunction() from within the class instead of calling $this->myFunction(). That's why I changed it. If you choose to use a class I think you should use it as you normally would.
However, a plain function would be fine as well in this case. Do you want me to change it?

Hi @woylie. I think I see what you mean about breaking methods up into multiple, smaller functions, but I disagree that calling the private static methods would be something you wouldn't normally do. Using private static methods like this is perfectly valid PHP, especially on an object that doesn't have state, only has pure functions or is immutable.
I think to make the exercise as approachable as possible, and not have to worry about classes, static or not-static methods, private/public methods, we should probably just make it a function.
Are you sure you wouldn't mind updating the PR? I'm happy to help if you don't have the time.
Thanks!

Hi @petemcfarlane, I just made the change.


grafana: 7.3.7 -> 7.4.0

Motivation for this change
https://github.com/grafana/grafana/releases/tag/v7.4.0
https://github.com/grafana/grafana/releases/tag/v7.4.0-beta1
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Thanks! Ported to stable as 1a5249ecfe788ff2fdd208e84f35e3c68e9a31c6.


win,fs: remove trailing slash in junctions

I've investigated why this trailing slash was added originally, but I couldn't find any reason.
Fixes: https://github.com/libuv/libuv/issues/3329

This is breaking the Node.js test suite: https://ci.nodejs.org/job/node-test-binary-windows-js-suites/30902/RUN_SUBSET=3,nodes=win11-arm64-COMPILED_BY-vs2022-arm64/testReport/junit/(root)/parallel/test_fs_symlink_dir_junction/. I'm not sure whether it's a legit issue or the node.js test is not correct.

It is appears to be correctly failing


Show tooltip on QuickPick item's description

In our extension, we use QuickPick API to collect information. In some case, the description is long, which exceed the width of QuickPick. It is better that show the entire description as tooltip when hover on it.

duplicate of https://github.com/microsoft/vscode/issues/99301


chore: dragonfly updates version to v2.0.5

Signed-off-by: Gaius<EMAIL_ADDRESS>
Description

Dragonfly updates version to v2.0.5.

Related Issue

Motivation and Context

Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.

Codecov Report

Merging #1498 (06b56e6) into main (ada1571) will decrease coverage by 0.04%.
The diff coverage is n/a.

:exclamation: Current head 06b56e6 differs from pull request most recent head 34e1f0f. Consider uploading reports for the commit 34e1f0f to get more accurate results

@@            Coverage Diff             @@
##             main    #1498      +/-   ##
==========================================
- Coverage   43.49%   43.45%   -0.05%
==========================================
  Files         116      116
  Lines       10681    10681
==========================================
- Hits         4646     4641       -5
- Misses       5552     5555       +3
- Partials      483      485       +2

Flag
Coverage Δ

Object-compatibility-e2etests
?

e2etests
?

unittests
43.45% <ø> (-0.05%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

client/daemon/peer/peertask_conductor.go
56.62% <0.00%> (-0.50%)
:arrow_down:


Whitescreen when cached form data differs from expected data structure

When rendering the OpenAPI Designer, it will whitescreen if the browser has a cached form version with different structure than expected. E.g. if a new version of OpenAPI designer is deployed with backwards incompatible changes, users visiting the designer with cached form data might see a white screen.
Solution
One solution is to manually clear the browser cache by entering the following command in their browser console:
delete localStorage.cachedForm

Goal
Provide a graceful solution to resolve cached form data incompatibilities without requiring the user to clear cached data from their browser console.
Console error
Unhandled promise rejection Error: Error invoking App. Check the inner error for details.
------------------------------------------------
Inner Error:
Message: can't assign to properties of (new String("scope")): not an object
Inner Error Stack:
setValue@http://localhost:9000/scripts/app-bundle.js:1483:11
setValue@http://localhost:9000/scripts/app-bundle.js:1485:9
setValue@http://localhost:9000/scripts/app-bundle.js:2088:11
setValue@http://localhost:9000/scripts/app-bundle.js:2088:11
App@http://localhost:9000/scripts/app-bundle.js:53:9
invoke@http://localhost:9000/scripts/vendor-bundle.js:18795:14
invoke@http://localhost:9000/scripts/vendor-bundle.js:18772:155
invoke@http://localhost:9000/scripts/vendor-bundle.js:19036:16
get@http://localhost:9000/scripts/vendor-bundle.js:18489:27
get@http://localhost:9000/scripts/vendor-bundle.js:18975:14
ensureViewModel/<@http://localhost:9000/scripts/vendor-bundle.js:29883:58
run@http://localhost:9000/scripts/vendor-bundle.js:9835:22
notify/<@http://localhost:9000/scripts/vendor-bundle.js:9848:28
flush@http://localhost:9000/scripts/vendor-bundle.js:7133:9

End Inner Error Stack
------------------------------------------------

Stack trace:
AggregateError@http://localhost:9000/scripts/vendor-bundle.js:21130:13
invoke@http://localhost:9000/scripts/vendor-bundle.js:19038:15
get@http://localhost:9000/scripts/vendor-bundle.js:18489:27
get@http://localhost:9000/scripts/vendor-bundle.js:18975:14
ensureViewModel/<@http://localhost:9000/scripts/vendor-bundle.js:29883:58
run@http://localhost:9000/scripts/vendor-bundle.js:9835:22
notify/<@http://localhost:9000/scripts/vendor-bundle.js:9848:28
flush@http://localhost:9000/scripts/vendor-bundle.js:7133:9

This issue was discovered during review of PR #160.

It needs a specific kind of incompatibility to crash like that, usually the data just disappears. Anyway, as I said in #160, the best solution is probably to add a try catch somewhere to catch all major errors.
Not requiring the user to clear cached data in errors like this may not be possible, but we can provide an easy way to clear the data.


Sending default read concern to the server

From what I read in MongoDB Driver Specifications, driver must omit default read concern, if specified. As far as I understand, the default read concern in PHP can be obtained as follows:
$readConcern = new \MongoDB\Driver\ReadConcern();
null === $readConcern->getLevel(); // true

// or:

$readConcern = (new \MongoDB\Driver\Manager())->getReadConcern();
null === $readConcern->getLevel(); // true

But from what I see in a code of this library , the default read concern can be sent to the server by the library.
For example:
// \MongoDB\Driver\Manager instance is created internally in Client's constructor
$client = new \MongoDB\Client();

// By default, Database instance will inherit read concern from \MongoDB\Driver\Manager instance,
// constructed in Client's constructor. Collection instance will inherit readConcern, writeConcern
// and readPreference from Database. This means that $collection->readConcern for now contains
// default read concern.
$collection = $client->selectDatabase('test')->selectCollection('test');

// Find operation will inherit "readConcern" option from Collection. Thus, default read concern
// will not be omitted and will be sent to the server.
$cursor = $collection->find([]);

So I have a question whether this is an expected behavior or should this behavior be fixed.
Does the low-level driver remove default read concern and write concern before sending a query to the MongoDB server?
It worth noting, that the same logic applies to the write concern and writing commands, i.e. default write concern can be sent to the server with writing commands.

Does the low-level driver remove default read concern and write concern before sending a query to the MongoDB server?

libmongoc omits applying a default read concern (e.g. { }) to queries (see: https://github.com/mongodb/mongo-c-driver/commit/3e34af30faca57e014044e0a3b7ca8f3460991df and CDRIVER-1527).

It worth noting, that the same logic applies to the write concern and writing commands, i.e. default write concern can be sent to the server with writing commands.

This does appear to be an issue with write concerns, despite CDRIVER-1322 having a function to detect default write concerns. The logic in https://github.com/mongodb/mongo-c-driver/commit/3e34af30faca57e014044e0a3b7ca8f3460991df#diff-f056549fad972eba9c93b7e0dcd9542bR1414 (also for CDRIVER-1527) appears to include a default write concern so long as the option is specified. Since PHPLIB always provides the option, we do see it sent across the wire:
2017-01-17T16:40:59.966-0500 I COMMAND  [conn1] command test.foo command: insert { insert: "foo", writeConcern: {}, ordered: true, documents: [ { x: 1, _id: ObjectId('587e8f6b6118fd4a0c5acbf2') } ] } ninserted:1 keysInserted:1 numYields:0 reslen:44 locks:{ Global: { acquireCount: { r: 3, w: 3 } }, Database: { acquireCount: { w: 2, W: 1 } }, Collection: { acquireCount: { w: 2 } } } protocol:op_query 3ms
...
2017-01-17T16:45:33.064-0500 I COMMAND  [conn2] command test.foo command: findAndModify { findAndModify: "foo", new: false, upsert: false, query: { x: 1 }, update: { $set: { y: 1 } }, writeConcern: {} } planSummary: COLLSCAN update: { $set: { y: 1 } } keysExamined:0 docsExamined:1 nMatched:1 nModified:1 numYields:0 reslen:120 locks:{ Global: { acquireCount: { r: 1, w: 1 } }, Database: { acquireCount: { w: 1 } }, Collection: { acquireCount: { w: 1 } } } protocol:op_query 0ms

I'll look into this further and create related JIRA issues as needed. Thanks for the report!

@jmikola, thanks for the answer!
Do I understand correctly, that the issue with the write concern will be fixed in low-level driver, not in this library?

Do I understand correctly, that the issue with the write concern will be fixed in low-level driver, not in this library?

I would like to address this in both places if possible. If the library can have a notion of an unset read concern, read preference, and write concern, that would allow us to not pass that around the selected objects (e.g. Database, Collection, Bucket) and inject it into each operation.
That said, if the write concern is still considered to be in a "default" state by libmongoc, which I expect it may be if ultimately sourced from a vanilla client object, it may be a unintentionally including it in its operations. That would be a bug we could address.

I have problem with readConcern.
I tried upgrade mongodb replica set from 3.0 to 3.2.
When I was upgrade secondary server, sometimes I catched exception "Read concern is not supported by the server executing this command", but I don't use readConcern.
My environment
Debian 8
PHP 5.6.29, php-fpm
mongodb/mongodb: 1.1.1
ext-mongodb: 1.2.5

Mongodb versions on replica set
primary: 3.0.14
secondary-1: 3.0.14
secondary-2: 3.2.12

I've found a bug on Collection:findOne(). We have double selectServer after https://github.com/mongodb/mongo-php-library/commit/4c61d3c5aecfb0ae8a4958806d64db1d66bca05e#diff-810cdcfdd8a6b9e1fc0d1e96d7786874R462
    public function findOne($filter = [], array $options = [])
    {
        if ( ! isset($options['readPreference'])) {
            $options['readPreference'] = $this->readPreference;
        }
        $server = $this->manager->selectServer($options['readPreference']);
        if ( ! isset($options['readConcern']) && \MongoDB\server_supports_feature($server, self::$wireVersionForReadConcern)) {
            $options['readConcern'] = $this->readConcern;
        }
        if ( ! isset($options['typeMap'])) {
            $options['typeMap'] = $this->typeMap;
        }
        $operation = new FindOne($this->databaseName, $this->collectionName, $filter, $options);
        $server = $this->manager->selectServer($options['readPreference']);
        return $operation->execute($server);
    }

It

@marliotto: Thanks for pointing out that bug. I've opened PHPLIB-260 as a tracking ticket and will hope to get that fixed soon.
@petr-buchin: I realized that I never created a JIRA issue for the aforementioned idea of tracking unset read concern, read preference, and write concern options. I've just done so in PHPLIB-261 and hope to address that in 1.2.0.

@marliotto: 1.0.5 and 1.1.2 have been released with a fix.

Closing this as all related PHPLIB tickets have been resolved and released.


Can't select correct active tab when list of dynamic tabs changes

Bug description:
I am using a tabset that has 1 static tab then loops through a list of dynamic tabs. The problem I'm running into is that some actions trigger a new set of tabs to be loaded and then set one of those new tabs as the active tab. When I do this the scope variable I'm using for the active tab index gets set to null and either no tab or the wrong tab gets selected.
Before upgrading from ubis 0.13.4 to 1.2.5 I accomplished this by setting the bool "active" property on each tab after they were reloaded. After the upgrade and now having active on the tabset pointing to an index I am unable to get this working. The linked plunker demonstrates this. I've tried a couple variations and am unable to get it working in every case which makes me think it's a bug.
Link to minimally-working plunker that reproduces the issue:
http://plnkr.co/edit/vIt6EZI15s1Q2wGY7qkq
Version of Angular, UIBS, and Bootstrap
Angular: 1.4.5
UIBS: 1.2.5
Bootstrap: 3.3.4

This is likely a timing/digest issue as wrapping the setting of activeTabIndex after updating tabs in a timeout fixes the issue.
See this updated plunker
I'm not sure this is something that can be fixed in our library since I think the issue is with ng-repeat not being finished rendering the new tabs into the dom yet?
@wesleycho, thoughts?

@bryanbak, please be respetcful of our time by searching both open and closed issues prior to entering anything (also per the text you deleted when you entered this issue).  As explained in several other threads, we are not going to support dynamic tabs indices (and @deeg gave you a work around).
Closing as won't support.

@icfantv not trying to be disrespectful of anyones time. I'm sure I wasted much more of my time than any of yours :)
Apparently my search skills weren't strong this morning cause I searched the docs, stackoverflow, and open/closed bugs before posting this.
It might be good to clarify that dynamic tabs aren't supported on the docs page. I thought it would be fine since my example is very similar to the first example in the docs ng-repeating over a list of tabs. And this same thing was working prior to upgrading uibs.


Live TV streams broken

Sunday 18 or Monday 19 July something changed in the VRT live streams that NUPlayer cannot handle.
Either the video activity crashes or video stops for a few seconds while audio continues making watching Live TV impossible.
Feel free to comment on this issue if you have any relevant feedback.

I think it might be due to this: https://github.com/add-ons/plugin.video.vrt.nu/pull/893

VRT removed the HLS AES livestreams: https://www.vrt.be/heb-je-een-vraag/s/article/Waarom-werkt-sinds-19-juli-2021-de-stream-van-de-VRT-niet-meer-op-mijn-toestel
You can now only use the MPEG-DASH live streams with Widevine DRM.

Thanks for the info guys, much appreciated.
NUPlayer was always using the DASH stream already and DRM is (always) added when required - it checks for drm tag in f vualto-video-aggregator-web/rest/external/v2/videos result.
For NUPlayer two problems popped up since they changed the live stream:
- Video playback is aborted and you return automatically to the home screen
The DASH manifest is sometimes missing the 720p stream while it is still "announced" somewhere else - you can see this on the website as well, try refreshing a few times on the live stream, sometimes 720p is available, sometimes it's not.
This is causing an IndexOutOfBoundsException which will crash the player. I hope VRT will fix this at some point in time.
E/LoadTask: Unexpected exception handling load completed
      java.lang.IndexOutOfBoundsException: Index: 6, Size: 6
        at java.util.ArrayList.get(ArrayList.java:437)
        at com.google.android.exoplayer2.source.dash.DefaultDashChunkSource.updateManifest(DefaultDashChunkSource.java:246)
        at com.google.android.exoplayer2.source.dash.DashMediaPeriod.updateManifest(DashMediaPeriod.java:161)
        at com.google.android.exoplayer2.source.dash.DashMediaSource.processManifest(DashMediaSource.java:898)
        at com.google.android.exoplayer2.source.dash.DashMediaSource.onManifestLoadCompleted(DashMediaSource.java:741)
        at com.google.android.exoplayer2.source.dash.DashMediaSource$ManifestCallback.onLoadCompleted(DashMediaSource.java:1376)
        at com.google.android.exoplayer2.source.dash.DashMediaSource$ManifestCallback.onLoadCompleted(DashMediaSource.java:1371)
        at com.google.android.exoplayer2.upstream.Loader$LoadTask.handleMessage(Loader.java:483)
        at android.os.Handler.dispatchMessage(Handler.java:106)
        at android.os.Looper.loop(Looper.java:193)
        at android.os.HandlerThread.run(HandlerThread.java:65)

- Video playback is choppy
It looks like 1080p / 50 fps without DRM works great  but with DRM / Widevine there seems to be performance issues and a lot of dropped frames are observed - See issue 6348 in ExoPlayer repo. Looks like they have some experimental code ready to improve this, but this should be investigated which needs time.
I've worked around both issues temporarily by setting the max video size to 540p. This is far from ideal but better than a crashing or choppy player I guess....

@davidvde72 @bmesuere @michaelarnauts I could use some testing validation on this one.
As you might have noticed the 1080p streams have disappeared from latest release (1.3.5) due to the problems described in this issue and everything plays at 540p max at the moment.
I've now released 1.3.6 in the beta channel that allows you to select the video quality from the video player via the gear icon.
From my testing it seems that:

Canvas and Ketnet streams both work flawlessly at 1080p (and 720p)
Een stream has any of the following issues:

Is missing the 720p stream from time to time
When 720p is available the playback is choppy (audio continues but video doesn't)
Selecting 720p sometimes results in the 1080p stream (see Info overlay)
1080p always crashes after 2nd media chunk (20-40 seconds of playback time approximately) and you return to the Home screen

Could you test around a bit with the different video qualities on the different streams and let me know if you observe anything different ? I'm assuming your results will be similar but it would be good to check on different hardware and different internet connection etc.
For info; for those testing purposes the stream info overlay is highly useful, enable developer mode in Settings first, then you can toggle the stream info overlay from the player with the "Info" icon.
Many thanks,

Sorry, I was on vacation for the last week. I just did some tests on my Chromecast with Google TV:

een: 720p not listed, 1080p plays but video often stops (audio continues) it doesn't crash like you reported.
canvas: all streams listed, everything seems to work
Ketnet: all streams listed, everything seems to work

Ok great, thanks for testing! Looks like VRT fixed the "één" stream and now it also works correctly again in NUPlayer. I'll try to push out an update that selects 1080p automatically as soon as possible.


"Checking if there is a matched certificate with Azure MFA" will fail if there is only 1 matched certificate

The "Checking if there is a matched certificate with Azure MFA" will fail (and appear twice in the report) if there is only 1 matched certificate:

When the $MatchedCert only has 1 item, it will be considered as a string. And the $MatchedCert[0] will be the first character, instead of the whole string (certificate thumbprint):

The issue can be resolved by modifying the line from:
$MatchedCert = $TP | Where {$localCert -Contains $_}
To:
$MatchedCert = @($TP | Where {$localCert -Contains $_})

@jessie-pang thank you! this is fixed.


Create test suite and support travis CI prior to modifications

This PR provides:

Linting based on foodcritic and cookstyle
Kitchen tests using dokken to ensure functionality of the resource (currently only testing positive cases, negative cases are tricky).
Kitchen tests run on Ubuntu 16.04, 18.04 and CentOS 7, with Chef 12, 13 and 14.
Make foodcritic happy: This includes putting a LICENSE file in the repository. We put the Apache 2.0 file here, because the metadata.rb had Apache 2.0 set. I hope that’s okay.
Fixes for Chef 13+: Use new_resource. to access properties in resource DSL

@aklyachkin Sorry for the noise. The PR should be ready for review now.

Vielen Dank!


MiOpen causing the training to fail

Full error message when I am trying to train a deep learning model.
MIOpen(HIP): Warning [SQLiteBase] Unable to read system database file:/opt/rocm/miopen/share/miopen/db/gfx906_60.kdb Performance may degrade
terminate called after throwing an instance of 'boost::filesystem::filesystem_error'`
what():  boost::filesystem::permissions: No such file or directory: "/home/ssml/singh/.config/miopen//gfx906_60.HIP.2_11_0_993628deb.ufdb.txt"

@Griffintaur We need to reproduce the problem in order to fix it. Please provide full reproduce info.

@Griffintaur close issue for now for inactivity, please reopen and attach full info so that we can reproduce the issue.


Allow showing errors to readers

I am currently starting with tutorials for Manifolds.jl using Pluto and PlutoStaticHTML.
In the notebook (preview) at
https://juliamanifolds.github.io/Manifolds.jl/previews/PR534/tutorials/getstarted.html
I have a few cells where I would like to actually have an error to illustrate what error to expect.
Maybe this is also something for Pluto to have a way to nicely expect/display errors, but
For example the cell before 3. The Sphere is quite tricky just to display an error. Could this maybe be made nicer?
Also rendered markdown code and input cells look exactly the same, is there a way to change their appearance?

For example the cell before 3. The Sphere is quite tricky just to display an error. Could this maybe be made nicer?

Ah yes good question. See https://github.com/rikhuijzer/PlutoStaticHTML.jl/issues/73. If you have the time, then a PR to update the docs here at PlutoStaticHTML could be nice.
Let's leave this issue open until one of us updates the docs

Maybe this would even be something for PlutoUI – something to nicely display an expected error (and maybe even really throw an error if there is no error to display). Then PlutoStaticHTML could just render that PlutoUI-element “reasonably”?
I find my current solution a little hand-crafted and the error display also does not really look nice.

I still have to check how this renders in PlutoStaticHMML but the solution I ended up with (adapted from the linked Pluto issue) is
macro expect_error(code, error=:DomainError)
	quote
		try
			$(esc(code))
+                       error(string("Error of type ", $(esc(error)), " was not thrown."))
		catch e
			if e isa $(esc(error))
				pretty_error(e)
			else
				rethrow()
			end
		end
	end
end;

together with
function pretty_error(err)
    Markdown.parse("""
!!! info "This is how the Error we expect here looks like"
    ```
    $(replace(sprint(showerror, err), "\n" => "\n        "))
    ```

""")
end;

That's a bug. I'll see whether I can write a test and patch it. Probably done before tomorrow 12:00

Cool (because I would have had no idea where to start).
Don't worry its not urgent, its already nice that we found a way to display errors that fast. I have to write quite some part of the tutorial anyways, still. So if it takes longer – no problem :)

Sorry to bother again – in general the result looks great!

I just noticed the e> (which also only appears for one of the two displays, the other one is fine). I looked at the code and its

DomainError with 0.1:
    The vector [0.1, 1.0, 1.0] is not a tangent vector to [1, 0, 0] on Sphere(2, ℝ), since it is not orthogonal in the embedding.</pr

e>

( did not change any of the line breaks.
Any idea how this might have happened that the pre is so strangely split?

Any idea how this might have happened that the pre is so strangely split?

Yeah because I'm trying to parse HTML with basic string operations and regexes which are not a good idea, see https://stackoverflow.com/a/1732454/5056635.
I should have switched to a XML parser 😅

Then – just maybe – don't regexp 😃
Ok, but then I know where that's from and that its not me ;)

Then – just maybe – don't regexp :smiley:

Very helpful. Thanks :+1:

Sorry to bother again – in general the result looks great!

I just noticed the e> (which also only appears for one of the two displays, the other one is fine). I looked at the code and its

DomainError with 0.1:
    The vector [0.1, 1.0, 1.0] is not a tangent vector to [1, 0, 0] on Sphere(2, ℝ), since it is not orthogonal in the embedding.</pr

e>

( did not change any of the line breaks.
Any idea how this might have happened that the pre is so strangely split?

@kellertuer Is the issue fixed now in v6.0.4?

yes, it looks perfect locally :) Thanks!


[Access Request] table 5050 Contact procedure UpdateQuotes

We need access to the procedure UpdateQuotes of table 5050 Contact.
Can you please make it available for us?
// >>>>>>>>>>>
 //local procedure UpdateQuotes(Customer: Record Customer; CustomerTemplate: Code[10])
 procedure UpdateQuotes(Customer: Record Customer; CustomerTemplate: Code[10])
 // <<<<<<<<<<<

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Update JuliaCon.jl and pass timezone

This is mostly working although I just hit a bug https://github.com/JuliaCon/JuliaCon.jl/issues/22
Just creating a Draft PR for review first.

I have tested the different commands. Everything seems fine (iff the server time zone is set to UTC)

So now, @rmsrosa made a good remark about a week ago. Maybe for people requesting a specific TZ, the bot could answer with a DM instead of a reply in the general chat.
What do you think @tk3369 ?

Ok, this is all ready to go. I have updated the Manifest and switched to JuliaCon v2021.2.0.

This branch has been used throughout JuliaCon. Just merged now.