Add links to each tool's own awesome list

While this repo might be good for things related to all, or a combination of the TALL technologies, each of them has their own awesome lists with tons of great resources related to it that are worth checking out too. Here I've just added a link to each of them to make them more discoverable.

Good idea 👍


chore: remove walletconnect dapp

Why
Remove wallet connect dapp dependency, we're not using it

Seems like there is a conflict


Example using external api

Does anyone have a working example based on this project that hits an external endpoint? I understand the thought of ripping out the api directory and pointing ApiClient to a different endpoint. What is confusing me is how the server AND client would make requests to the same external endpoints.
The way that I am learning from this project is by adding a menu item which, in my case, points to some places. This way I don't impact the working parts of the projects, ie, Widgets. Ultimately, my goal is to have a complete CRUD places example talking to my backend API.
Thank you for any input/suggestions. :)

@LarryEitel checkout this line: https://github.com/erikras/react-redux-universal-hot-example/blob/9f0a32ce36c8c2444adcecb621038ee99b3c1d68/src/ApiClient.js#L45
You have to change it to your server schema/hostname/port to make it run for external host. If you have many environments, you will also need to create config with different hostnames.


Static analyzis

Have you considered triggering static analysis on build? Gcc and clang supports that now I believe?
Just saw another reddit add cppcheck to his template project and thought it could be interesting.

Hey thanks for the request, static analysis would be a great addition to the starter!
Unless there are other suggestions, I'll look at adding StableCoder's CMake tools to the project (in the same way as current sanitiser support).


Mark Auth getStoredUser(forAccessGroup) as unavailable for tvOS.

This may be irrelevant by Firebase 9 pending updates in #8878

If the API doesn't work now for tvOS, I don't think we need to wait for a breaking change release to mark it unavailable.

@ncooke3 Should this be closed given the requirement to share on a single device?

Yes, thanks– I had not considered that case when I created this issue. Closing.


Forms in notebooks

I would like to be able to have "input renders" which configure variables inside of a code block.

Much like forms in google collab.
https://colab.research.google.com/notebooks/forms.ipynb

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

We try to keep VS Code lean and we think the functionality you're asking for is great for a VS Code extension. Maybe you can already find one that suits you in the VS Code Marketplace. Just in case, in a few simple steps you can get started writing your own extension. See also our issue reporting guidelines.
Happy Coding!


Polish Matplotlib content for consistency

This PR goes through all the Matplotlib content and makes a number of small tweaks for style and consistency, including

Rationalizing the titles and notebook file names
Consistent capitalization of the words "Matplotlib" and "Python" in the text
Adding the notebook output file animate.gif to the git ignore list
Tweaks to various links
Making the Matplotlib logo appear the same on each page

Closes #234

@brian-rose Thanks for doing this! I just saw one irregularity on the matplotlib.md page that says we've taught 3D plotting. For some reason I'm unable to suggest an edit to that line.

@brian-rose Thanks for doing this! I just saw one irregularity on the matplotlib.md page that says we've taught 3D plotting. For some reason I'm unable to suggest an edit to that line.

Thanks, good catch, I'll fix that now.


The latest translation,sorry to bother you

Contributed by @GalacitCity and me

Hopefully, it will work out this time


Server error when submitting a request through OpenAI client

What is the issue?
Using Vercel AI SDK and llama2
Request:
const content = [
    { type: 'text', text: prompt },
    { type: 'text', text: page.content },
 ]

const result = await experimental_generateObject({
    model,
    schema,
    messages: [{ role: 'user', content }],
    temperature,
  })

Response:
mish@Mishs-MBP llm-scraper % bun examples/hn.ts
12 |     statusCode === 429 || // too many requests
13 |     statusCode >= 500),
14 |     // server error
15 |     data
16 |   }) {
17 |     super(message);
         ^
AI_APICallError: json: cannot unmarshal array into Go struct field Message.messages.content of type string
 url: "http://<IP_ADDRESS>:11434/v1/chat/completions"

      at new APICallError (/Users/mish/Documents/Projects/llm-scraper/node_modules/@ai-sdk/provider/dist/index.mjs:17:5)
      at /Users/mish/Documents/Projects/llm-scraper/node_modules/@ai-sdk/provider-utils/dist/index.mjs:279:12
12 |     statusCode === 429 || // too many requests
13 |     statusCode >= 500),
14 |     // server error
15 |     data
16 |   }) {
17 |     super(message);
         ^
AI_APICallError: json: cannot unmarshal array into Go struct field Message.messages.content of type string
 url: "http://<IP_ADDRESS>:11434/v1/chat/completions"

      at new APICallError (/Users/mish/Documents/Projects/llm-scraper/node_modules/@ai-sdk/provider/dist/index.mjs:17:5)
      at /Users/mish/Documents/Projects/llm-scraper/node_modules/@ai-sdk/provider-utils/dist/index.mjs:279:12

Possibly related to #3690?
OS
macOS
GPU
Apple
CPU
Apple
Ollama version
0.1.32

Ollama's OpenAI compatibility doesn't support content arrays, only text contents, you can read about the supported request fields in the documentation: https://github.com/ollama/ollama/blob/main/docs/openai.md#supported-request-fields.
If you want to send more messages, you must send them separately.
[
    {
        "role": "system",
        "content": "You are a helpful assistant."
    },
    {
        "role": "user",
        "content": "Hello!"
    }
]

Alright, will keep that in mind 😉

I'm getting this now:
mish@Mishs-MBP llm-scraper % bun examples/hn.ts
12 |     statusCode === 429 || // too many requests
13 |     statusCode >= 500),
14 |     // server error
15 |     data
16 |   }) {
17 |     super(message);
         ^
AI_APICallError: json: cannot unmarshal array into Go struct field Message.messages.content of type string
 url: "http://<IP_ADDRESS>:11434/v1/chat/completions"

      at new APICallError (/Users/mish/Documents/Projects/llm-scraper/node_modules/@ai-sdk/provider/dist/index.mjs:17:5)
      at /Users/mish/Documents/Projects/llm-scraper/node_modules/@ai-sdk/provider-utils/dist/index.mjs:279:12
12 |     statusCode === 429 || // too many requests
13 |     statusCode >= 500),
14 |     // server error
15 |     data
16 |   }) {
17 |     super(message);
         ^
AI_APICallError: json: cannot unmarshal array into Go struct field Message.messages.content of type string
 url: "http://<IP_ADDRESS>:11434/v1/chat/completions"

      at new APICallError (/Users/mish/Documents/Projects/llm-scraper/node_modules/@ai-sdk/provider/dist/index.mjs:17:5)
      at /Users/mish/Documents/Projects/llm-scraper/node_modules/@ai-sdk/provider-utils/dist/index.mjs:279:12

From what I understand of Vercel AI SDK, it seems they convert messages.content into an array behind the scenes in the convertToLanguageModelPrompt function.
For instance, the if-statement on line 34 checks if the messages.content is a string, and if it is it'll convert it to an array.
Given that Ollama's OpenAI compatibility doesn't support that yet, Vercel's AI Core sadly isn't compatible either.

I have started an issue there too. Since this is not an issue with Ollama per se., I'm closing.
Thank you for all your help 😄

I believe https://github.com/ollama/ollama/pull/2506 would fix this, along with improving overall OpenAI compatibility.

Glad to hear that!


Upgrade Rubocop to 0.75+ to fix documentation rake tasks

This PR fixes the failure found in #827 (and master as well presumably), pinning the Rubocop version to 0.75 and updating the documentation rake task to match the changes to MessageAnnotator.

Closing as per @Darhazer's suggestion, we can do this by checking the arity rather than forcing a significant version bump.


refactor!: sequencers module

sequencer module registers the sequencer's pubkey on InitGenesis
On first begin block, it registers the proposer address for this pubkey as a complete Sequencer object

This removes the need for any manual registration on the rollapp layer (no need for seq_gentx nor MsgCreateSequencer)

@mtsitrin tests are failing


Topic subscription backlog can't be cleared

Describe the bug
Pulsar v2.6.2
Use clear-backlog command:
bin/pulsar-admin topics clear-backlog ***
Topic subscription backlog can't be cleared.
Use unload command:
bin/pulsar-admin topics unload ***
Topic subscription backlog can be cleared.
But I don't know if any messages are lost, and why the clear backlog can't be cleared. Is it a counter error?

To Reproduce
Steps to reproduce the behavior:
1.start consumer, receive msg
2.reconsumerLater msg

This problem also exists in version 2.8.1.


Instrumentation Ideas/Draft

I'm working on a Miniplex world inspector for my game:
https://user-images.githubusercontent.com/1061/193662735-a2c76ca0-5dc8-4e39-9bc5-8dae1b8f44f0.mov
It would be great to add some instrumentation features to Miniplex' World class to make it easier for tools like this to get useful instrumentation data.
Things that might be nice to measure:

[x] Total number of entities
[x] Total number of archetypes
[x] Components of individual archetypes
[ ] Archetype "pressure" (rate of entities added or removed). This can be useful to identify archetypes that aren't really used much (but still infer an indexing cost every time an entity is added/removed/modified.)
[ ] Average time needed to add/index/remove entities

Ideas:

Making the numbers flash green or red as they increase / decrease.
Making it possible to see the entities details. That might not work great for archetypes that have a lot of very quickly added/removed entities, but it would be very useful to inspect more stable ones.


Building with precompiled headers fails under MSVC if PCH is not included

.\b2.exe --version: B2 4.9-git
OS: Windows 10
toolset: msvc-14.3
Running in a Boost 1.80.0, as per cloning from boostorg/boost, initializing submodules and .\bootstrap.bat.
File structure:
libs/atest/Jamfile
libs/atest/acpp.cpp
libs/atest/apch.hpp
Content in https://gist.github.com/anarthal/221c0ee6f3d2640fc6bba892102cf3dd
Running with .\b2 .\libs\atest//anexe
Fails with
Performing configuration checks
- default address-model    : 64-bit [1]
- default architecture     : x86 [1]
- symlinks supported       : no
- junctions supported      : yes
- hardlinks supported      : yes

[1] msvc-14.3
...found 14 targets...
...updating 9 targets...
compile-c-c++-pch bin.v2\libs\atest\msvc-14.3\debug\threading-multi\apch.pch
apch.pch.cpp
compile-c-c++ bin.v2\libs\atest\msvc-14.3\debug\threading-multi\acpp.obj
acpp.cpp
libs\atest\acpp.cpp(2): fatal error C1010: unexpected end of file while looking for precompiled header. Did you forget to add '#include "apch.hpp"' to your source?
call "bin.v2\standalone\msvc\msvc-14.3\msvc-setup.bat"  >nul

cl /Zm800 -nologo "libs\atest\acpp.cpp" -c -Fo"bin.v2\libs\atest\msvc-14.3\debug\threading-multi\acpp.obj"  -FI"libs\atest\apch.hpp" -Yu"apch.hpp" -Fp"bin.v2\libs\atest\msvc-14.3\debug\threading-multi\apch.pch" -TP /wd4675 /EHs /GR /Zc:throwingNew /Z7 /Od /Ob0 /W3 /MDd /Zc:forScope /Zc:wchar_t /Zc:inline /favor:blend -DBOOST_ALL_NO_LIB=1 -DBOOST_BUILD_PCH_ENABLED "-I."
...failed compile-c-c++ bin.v2\libs\atest\msvc-14.3\debug\threading-multi\acpp.obj...
...skipped <pbin.v2\libs\atest\msvc-14.3\debug\threading-multi>anexe.exe for lack of <pbin.v2\libs\atest\msvc-14.3\debug\threading-multi>acpp.obj...
...skipped <pbin.v2\libs\atest\msvc-14.3\debug\threading-multi>anexe.pdb for lack of <pbin.v2\libs\atest\msvc-14.3\debug\threading-multi>acpp.obj...
...failed updating 1 target...
...skipped 2 targets...
...updated 6 targets...

I would appreciate it if you would try this with the Boost 1.81.0 beta release candidate, and let us know if it still fails.
https://boostorg.jfrog.io/artifactory/main/beta/1.81.0.beta1/source/

Fails with the same output.


app: api to clear data from different storage types

Fixes #1832
Since the storageimplemetation relies only on browserContext and origin , have added the api under app instead of webContents. This doesnt handle network cache, should it be added ?

[x] Add Tests

Can it clear chromium's network resources cache now?

@magicae havnt added, but if its something desired will add a new api clearCache

@deepak1556 ClearCache sounds great i think

I also agree that clear cache would be a great feature to have.

have added clearCache api.

This API should be part of the session object, like this:
win.webContents.session.clearCache();

because we are going to have multi-session support in future.

clearCache means to clear network resources cache, i think it isnt related to session?

Different sessions can have independent caches.

Totally forgot about multi-session. @zcbenz fixed, thanks!

The clearData should also be a method of session.

The documents of Session should follow WebContents, e.g.:
## Class: Session
...

### Session.clearCache()
...

Thanks!


Accuracy and dev branch

Thanks for sharing this repo! I have some questions..

Current accuracy?

I noticed some TO-DOs and nn-buy-sell-prediction is 10 commits ahead of master. Which branch should receive PRs?

Hello! What do you mean by current accuracy? Also, master is the branch currently accepting PRs if you are looking to make some

Sry, by accuracy I mean.. Is the bot profitable?

Thanks for sharing this - any back testing results ?

I do not have backtesting setup yet. If someone wants to do that, send a pull request. I only tested the bot for a short period of time so I don't really have a trustworthy number for expected returns - was meant more to be a fun 1-week side project than anything which would personally make me any money.

Thanks @aoberai amazing stuff for a quick side project -


UI - systemVM - Fix error message jobid not found when moving to another host

Description
Fixes https://github.com/apache/cloudstack/issues/5322

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan ui

@utchoang a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5323 (SL-JID-524)

@rhtyd @utchoang @shwstppr I've seen this happening when migrating VMs with volume (local storage + KVM) in 4.15.
Does it make sense to backporting this change to the 4.15 branch as well?
We could get it in time for 4.15.2 then.


Formatting fix for sql-data-discovery-and-classification.md

There was a minor issue with the markdown formatting that was fixed.

@crpietschmann : Thanks for your contribution! The author(s) have been notified to review your proposed change. @giladm


format_by_(class|column) aren't used :fire:

Many years ago their usage was removed from the generic server settings and migrated in a way to each report's settings...
commit 73ac8da10e3bc0eed2abc471ca89adce222f0700
Author: <REDACTED>
Date:   Wed Dec 16 21:20:47 2009 +0000

    -Changed args in format method to take an options hash instead of format as the last arg to allow passing additional options such as time zone.
    -Updated callers of format method.
    -Support for passing in :_default_ as format value to force default lookup of format and not look in report object.
    -Added support for date, time and datetime formats.
    BugzID: 7453

    git-svn-id: http://miq-ubuntusub.manageiq.com/svn/svnrepos/Manageiq/trunk@17795 3c68ef56-dcc3-11dc-9475-a42b84ecc76f

Seen when reviewing: https://github.com/ManageIQ/manageiq/pull/15987

Author:

😆

Author:
😆

Gotta protect the innocent!

Manageiq/trunk@17795

I just saw it's svn 17795!


CI: update go workflow

This only changes the GH actions go workflow.
Run the workflow on master instead of develop, as we don't want to follow the develop/master promotion cycle anymore.
Also:

Run go vet and linting via golangci-lint.
Run on a cron schedule, not just on push and PRs.

@ctlong Will this change fix the failing checks or they needs to be addressed first before we review the pr?

@beyhan it needs to be fixed before it's merged. I'll change it to a draft PR for the time being to indicate that it's in-progress.

@ctlong in the interim between this getting created + now, the default branch has changed to main. Can you adjust?

@geofffranks updated!


Problem with versioning and folder that contains dot in the name

Laravel Mix Version: 3.0.0
Node Version 10.9.0
NPM Version 6.2.0
OS: win 10

Description:
There is error generating versions If one copy folder that contains dot in the name with copy() or copyDirectory() .
For example:
.copyDirectory('resources/assets/js/joyride-2.1', 'public/js');

Steps To Reproduce:
webpack.mix.js
lines like:
    .copyDirectory('resources/assets/js/joyride-2.1', 'public/js');

Resulting in error:
Error: EISDIR: illegal operation on a directory, read
    at Object.readSync (fs.js:489:3)
    at tryReadSync (fs.js:328:20)
    at Object.readFileSync (fs.js:365:19)

I have tried:
.copyDirectory('resources/assets/js/joyride-2.1/*', 'public/js/joyride-2.1');

and
.copyDirectory('resources/assets/js/joyride-2.1/*', 'public/js/joyride-2.1/');

Resulting to copy the folder files in file: public/js/joyride-2.1
If destination folder has no dot then copy works:
    .copyDirectory('resources/assets/js/joyride-2.1/*', 'public/js/joyride-2');

I think the problem is here:
https://github.com/JeffreyWay/laravel-mix/blob/a53d6705ffc68fa79b46dcfbf1c205bcc8fdef7b/src/File.js#L256
https://github.com/JeffreyWay/laravel-mix/blob/a53d6705ffc68fa79b46dcfbf1c205bcc8fdef7b/src/File.js#L257

@lfelizari
Did you try with adding '/*':
.copyDirectory('node_modules/tableexport.jquery.plugin/*','public/vendor/tableexport-jquery-plugin');
This is a workaround that I found for source folder, but I didn't found workaround for destination folder.

For googlers: one of the "hack" solution for that situation:
yarn add -D <alias>@npm:<package>

Example:
yarn add -D datatables-buttons@npm:datatables.net-buttons

The same for NPM.

I am also encountering the same problem with node packages with dots in their name causing copyDirectory() to fail. martinstefano's workaround (https://github.com/JeffreyWay/laravel-mix/issues/1842#issuecomment-449549533) does the job but not ideal. I'd rather be using the original package name consistently throughout my application.
I see that an open merge request with a potential solution exists: https://github.com/JeffreyWay/laravel-mix/pull/1926. Could this be considered for merge into master?

I'm having this issue too using pdf.js-viewer!
Can copy() just copy files and copyDirectory() actually copy directories? :(


Fix multi-gpu device handling

There was a bug preventing hipSYCL from working correctly with multiple GPUs. This PR fixes this:

Fix missing assignment of updated active device in CUDA/HIP device manager
Fix missing device activation before constructing an event

With this PR, I can run the multigpu nstream from PRK on two CUDA GPUs. Result validation fails with 4 GPUs, but I believe this is because PRK's calculation of the offsets for the decomposed buffers assigned to individual GPUs seems incorrect to me - ping @jeffhammond
CC @psath - I believe this might fix your multi-GPU issue, but unfortunately I only have a single AMD GPU for testing and can only do multi-GPU testing on NVIDIA.

@hipsycl-ci-bot test this for me please :)

CC @psath - I believe this might fix your multi-GPU issue, but unfortunately I only have a single AMD GPU for testing and can only do multi-GPU testing on NVIDIA.

Thanks!, I'll give it a test after morning meetings.

I never ran that code on more than 2 devices because I wrote it on a NUC and the only two devices I had were CPU and GPU.  However, I don't see the bug.  Can you provide some details on https://github.com/ParRes/Kernels/issues/524 with what you see?  I can run it on a DGX-A100 8 GPU box but don't know when I'll have time for that.

@jeffhammond Yes, it works just fine with two devices. I have provided some more information.

CC @psath - I believe this might fix your multi-GPU issue, but unfortunately I only have a single AMD GPU for testing and can only do multi-GPU testing on NVIDIA.

Thanks!, I'll give it a test after morning meetings.

No dice, still getting crashes when I try to do an explicit copy command group to my 2nd GPU. I'll do some more diagnostics and open a separate issue for it.

@psath Might be that there are additional issues in the buffer-accessor implementation. The application I have tested with uses USM, so no buffers.


sparse matrix to RDD

Hi. I am trying to convert a sparse matrix to a RDD using SparkContext. I am getting the below error, while doing so.
vect = vectorizer.fit_transform(samp) # This returns a sparse matrix
vect_rdd = sc.parallelize(vect) # I want to do this, because I have other rdd sets, with which I want to combine vect_rdd
This is giving the below error:
File "/opt/anaconda/python2.7/lib/python2.7/site-packages/scipy/sparse/base.py", line 192, in len
raise TypeError("sparse matrix length is ambiguous; use getnnz()"
TypeError: sparse matrix length is ambiguous; use getnnz() or shape[0]
If I do something like this, the error gets resolved
vect = vectorizer.fit_transform(samp)
vect_rdd = sc.parallelize(vect..toarray()) # When sparse matrix is converted to an array, this piece is working without any issue, but because of huge size od the RDD, I am not able to print it
I think, in context.py. for sc.parallelize(), if a sparse matrix is passed, it should consider shape[0] instead of len. Please redirect if this is not an appropriate post. Thanks in advance
Scikit-learn - 0.15.2
Spark - 1.2.0
Python - 2.7
Numpy - 1.9.0
Thanks.

Thanks for sharing the experience.
I think it will be helpful post this to Apache Spark's mailing list.


feat: add constants/float32/sqrt-two-pi

Description

This RFC proposes adding the package @stdlib/constants/float32/sqrt-two-pi returning Square root of the mathematical constant π times 2 as a single-precision floating-point number.

Related Issues

Does this pull request have any related issues?

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines

@stdlib-js/reviewers

/stdlib update-copyright-years


Senseval corpus contains irregular formed entries

I'm iterating over the instances of senseval in order to collect the words but not the POS-tags. Following the how-to in http://www.nltk.org/howto/corpus.html I use train += [w for (w, _) in instance.context] .
However eventually I get an ValueError ValueError: too many values to unpack
because an instance.context contains the String FRASL instead of a tuple. A quick glance into hard.pos showed me around 34 of those. This seems  rather inconvenient because now I have to think of how to deal with non-tuple entries.
Intuitively I'd suggest turning FRASL into a tuple, however I have no clue what it is actually denoting. If you decide against changing it, it would be nice to update  the HOWTO with a working approach for parsing instances. :)

I also encountered this problem 3 years after this issue was open. I also think that using a tuple instead of FRASL would be a good make it easier to iterate over the instances of senseval.


⚠ Bofrost.at has degraded performance

In eff10f4, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1681 ms

Resolved: Bofrost.at performance has improved in b1f704c after 24 minutes.


Update protobuf-java to 3.17.1

Updates com.google.protobuf:protobuf-java from 3.13.0 to 3.17.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.protobuf", artifactId = "protobuf-java" } ]

labels: library-update, semver-minor

Superseded by #450.


wip - migrate to oedipus_lex

What problem is this PR intended to solve?
@zenspider is making the case that oedipus_lex is better, faster, and better supported than rexical. Let's try it (again).
See prior work at https://github.com/sparklemotion/nokogiri/issues/1337
To reproduce the failing test: run bundle exec rake css:clean css:generate

All greened up. DO NOT MERGE AS IS... I added a do_parse method to the tokenizer that is bogus... but makes the tests pass??? Needs to be verified.

@zenspider Thanks for your work on this. I'm back from vacation and will make time to take a look in the next few days.

no rush!

I'm going to be moving Nokogiri to a hand-written tokenizer and parser for the CSS selectors. See #2560


Handle null produced by the j.u.c.CompletionStage

I'm using zio with fdb-java with wrapper on CompletionStage and some methods from fdb can return null as expected behaviour but zio freezes with silent drop of IO.
from  j.u.c.CompletionStage#handle javadoc:

When this stage is complete, the given function is invoked with the result (or null if none) and the exception (or null if none) of this stage as arguments, and the function's result is used to complete the returned stage.

If (t !=null) ... else ...

@TimothyKlim Thank you for making this improvement, and congratulations on your first contribution to ZIO!

@jdegoes Thank you and thanks to contributors for this awesome library!


 Pretty sure I couldn't move or shoot either.

This should be fixed in https://github.com/R2Northstar/NorthstarLauncher/pull/203.


Update Byobu URL/Remove wal and add pywal.

Checklist for submitting a pull request to Terminals Are Sexy:

[x] I have carefully read and comply with the Contributing Guidelines of this repo.
[x] I have searched the PRs of this repo and believe that this is not a duplicate.

Title: Byobu
Link: https://byobu.org/
Category: Tools and Plugins
Description: Byobu is an open source text-based window manager and terminal multiplexer.

Title: pywal
Link: https://github.com/dylanaraps/pywal
Category: Tools and Plugins
Description: generate and change colorschemes on the fly. Successor of wal.

Updating the Byobu URL since I noticed it didn't resolve anymore.
Also noticed wal is depreciated, so I removed it and added its successor pywal.

Hi there @Timecrash. Please make an individual pull request for each suggestion. Thanks in advance.

Closing due to inactivity.

Hi @Timecrash. Please squash your commits into one. Thanks!


chore: Update kubeconfig-generator

@ibakshay this plugin does not have a publicly available Docker Image. Could we skip the integration test for it?

@ibakshay this plugin does not have a publicly available Docker Image. Could we skip the integration test for it?

yes, we can skip the integration test for this plugin.

@

@ibakshay this plugin does not have a publicly available Docker Image. Could we skip the integration test for it?

yes, we can skip the integration test for this plugin.

@ibakshay can you configure this?

@ibakshay this plugin does not have a publicly available Docker Image. Could we skip the integration test for it?

yes, we can skip the integration test for this plugin.

@ibakshay can you configure this?

Yes, I will work on this issue.


Fix JS NPE when viewing specific range of PR commits

This should be the easiest fix.
While other solutions might be possible that exterminate the root cause, they will not be as trivial.

NPE surely isn't the right term? JavaScript does not have pointers 😆

Technically, JS calls it TypeError.
But still, it is basically a NullPointerException, simply as that's the term that everyone knows.
Java has no explicit pointers either, and still it's a NullPointerException there… :grin:

Update: Okay, I just found another bug in the same functionality:
Whenever I select a single commit in that menu, it shows showing only changes of commit <non-existing commit SHA>.
The URL is correct, however.

@sebastian-sauer do you have any idea perhaps what might be causing ↑?

I'll try to have a look into that.

Looks like the reference to BeforeCommitID is wrong in the tmpl https://github.com/go-gitea/gitea/blob/main/templates/repo/diff/box.tmpl#L56
When only one commit is shown this should use AfterCommitID


ng2: emitDecoratorMetadata problem with deconstructing imports

there seems to be a problem with
it should be elementRef: ElementRef without @Inject
https://github.com/gdi2290/ng-vegas-angular2-d3/blob/c6e1910a080a2b85681acc017e37275655e80407/app.ts#L20
compiles down to
https://github.com/gdi2290/ng-vegas-angular2-d3/blob/c6e1910a080a2b85681acc017e37275655e80407/app.js#L58
ElementRef should be angular2_2.ElementRef
I had to use @Inject in my demo to overwrite the Metadata rather than rely on the framework to inject the correct object base on the Type (ElementRef)
repo
https://github.com/gdi2290/ng-vegas-angular2-d3

this is probably due to the type definitions

this is confirmed a problem with the type definition not being correct in tsd


MWPW-154980 - Milo advanced page publishing

As a web producer, I want to limit who can publish certain files so that files are not published prematurely before planned launches.
New Feature Includes:

Ability to fetch an optional custom configuration file from path .milo/publish-permissions-config to apply advanced publishing rules on a per page basis.
New functionality will disable the publish button in AEM sidekick if path matches an item (url pathname) in the configuration file and can also reference a group with optional allow/deny user list which then compares the list to the email of profile that is currently logged into sidekick.
Bulk Publish v2 tool will also check paths in the job list for item configuration and refuse to publish urls that have been configured unless of course the currently logged in user has permissions.

Resolves: MWPW-154980
Testing Details:

Sidekick Publishing: Go to the test url page -> open sidekick -> verify publish button is disabled.
Bulk Tool Publishing: Go to BP Tool -> Publish test page URL -> see error thrown with message.

Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/sarchibeque/video-test?martech=off
After: https://sartxi-mwpw-154980-publishing--milo--adobecom.hlx.page/drafts/sarchibeque/video-test?martech=off

I almost think if the publish button is disabled the default hover fly out changes what it says.
Maybe something like "Publishing is disabled for this page"?
Currently the message reads like I can publish the page, which is a bit confusing.

Looks good. Left one minor suggestion, take it or leave it. Is there any documentation around managing the publish-permissions-config.json file for author admins?

I am working on authoring documentation today!


mesa build error in CentOS6: No package 'glproto' found

Hello.
Although glproto is installed in my system, when building mesa, I met the error messages
configure: error: Package requirements (glproto >= 1.4.14) were not met:

No package 'glproto' found

Consider adjusting the PKG_CONFIG_PATH environment variable if you
installed software in a non-standard prefix.

Alternatively, you may set the environment variables GLPROTO_CFLAGS
and GLPROTO_LIBS to avoid the need to call pkg-config.
See the pkg-config man page for more details.

Thank you.
https://gist.github.com/091234dc707b47899d6fda3f19b8f2bf

[x] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[x] have a problem with brew install (or upgrade, reinstall) a single, official formula (not cask)? If it's a general brew problem please file this issue at Homebrew/brew: https://github.com/Homebrew/brew/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
[x] ran brew update and can still reproduce the problem?
[x] ran brew doctor, fixed all issues and can still reproduce the problem?
[x] ran brew gist-logs <formula> (where <formula> is the name of the formula that failed) and included the output link?

To help us debug your issue please explain:

What you were trying to do (and why)
What happened (include command output)
What you expected to happen
Step-by-step reproduction instructions (by running brew install commands)

mesa doesn't build at all. It's missing several dependencies. Currently, I've added
diff --git a/Formula/mesa.rb b/Formula/mesa.rb
index c0e705b..62543f6 100644
--- a/Formula/mesa.rb
+++ b/Formula/mesa.rb
@@ -18,6 +18,9 @@ class Mesa < Formula
   depends_on "bison" => :build
   depends_on "libtool" => :build

+  depends_on "linuxbrew/xorg/damageproto"
+  depends_on "linuxbrew/xorg/dri2proto"
+  depends_on "linuxbrew/xorg/glproto"
   depends_on "linuxbrew/xorg/libdrm"
   depends_on "systemd" # provides libudev <= needed by "gbm"
   depends_on "linuxbrew/xorg/libsha1"

but the build still fails with
Last 15 lines from /home/mgi.polymtl.ca/doorba/.cache/Homebrew/Logs/mesa/03.make:
libtool: link: ranlib .libs/libgalliumvl.a
libtool: link: ( cd ".libs" && rm -f "libgalliumvl.la" && ln -s "../libgalliumvl.la" "libgalliumvl.la" )
gallivm/lp_bld_misc.cpp: In function 'LLVMOpaqueBuilder* lp_create_builder(LLVMContextRef, lp_float_mode)':
gallivm/lp_bld_misc.cpp:833:13: error: 'class llvm::FastMathFlags' has no member named 'setUnsafeAlgebra'
       flags.setUnsafeAlgebra();
             ^
make[4]: *** [gallivm/lp_bld_misc.lo] Error 1
make[4]: Leaving directory `/tmp/mesa-20180605-9622-1m28ai0/mesa-17.2.3/src/gallium/auxiliary'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory `/tmp/mesa-20180605-9622-1m28ai0/mesa-17.2.3/src/gallium'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/tmp/mesa-20180605-9622-1m28ai0/mesa-17.2.3/src'
make[1]: *** [all] Error 2
make[1]: Leaving directory `/tmp/mesa-20180605-9622-1m28ai0/mesa-17.2.3/src'
make: *** [all-recursive] Error 1

#351 makes some updates but it seems to have been abandoned.

Yes. I met the same error message regarding setUnsafeAlgebra().
But when I tried to find the definition of setUnsafeAlgebra(), there was no definition of setUnsafeAlgebra() anywhere. It seemed to be an error of mesa.

My build of mesa in Ubuntu fails with a different message:
ImportError: No module named setuptools
https://gist.github.com/1a74473593d7c0bd6d675c43f0cd9822

@nir-krakauer Did you try to install setuptools using brewed pip?

pip3 install setuptools returns:
Requirement already satisfied: setuptools in ./.linuxbrew/lib/python3.6/site-packages (39.2.0)

@nir-krakauer Could you please run not pip3 but pip2?

Thanks, brew reinstall python@2 did fix the problem.

I found setUnsafeAlgebra() as member function of class llvm::FastMathFlags in llvm-3.9. And this member function is not member function in llvm-6.0. So this seems to be a problem of llvm version.

If that's the case, you should be able to declare something like
depends_on<EMAIL_ADDRESS>

good catch @Gwang-Jin!

In<EMAIL_ADDRESS>llvm@4, and llvm@5, setUnsafeAlgebra() is defined as a member function of class llvm::FastMathFlags. Only in llvm@4 it is deleted.

I succeeded with llvm@4 at installing mesa.

Could you please post your diff (or submit a PR)?

Ok. I submitted #375.

This issue has been resolved in Linuxbrew/homebrew-xorg#375


Remove all: false from stats.toJson options to fix HMR in certain setups

This PR contains a:

[x] bugfix
[ ] new feature
[ ] code refactor
[ ] test update
[ ] typo fix
[ ] metadata update

Motivation / Use-Case
HMR stopped working with our project starting with v2.23.0.  I did some digging and found that this one particular stats.toJson option, all: false, is causing the issue and removing it fixes HMR for our use case.  Unless there is a specific reason for keeping it, I'd love to see it removed incase it is breaking other projects as well.
Breaking Changes
n/a
Additional Info
package.json (shortened to relevant bits)
"devDependencies": {
  "@babel/cli": "^7.1.2",
  "@babel/core": "^7.1.2",
  "@babel/plugin-proposal-class-properties": "^7.1.0",
  "@babel/plugin-proposal-decorators": "^7.1.2",
  "@babel/plugin-proposal-object-rest-spread": "^7.0.0",
  "@babel/preset-env": "^7.1.0",
  "@babel/preset-react": "^7.0.0",
  "@babel/preset-typescript": "^7.1.0",
  "babel-loader": "^8.0.4",
  "babel-preset-modern-browsers": "^12.0.0",
  "browser-sync": "2.23.7",
  "gulp": "^4.0.0",
  "react": "^16.4.2",
  "react-dom": "^16.4.2",
  "react-hot-loader": "^4.3.4",
  "typescript": "2.8.4",
  "webpack": "^4.20.2",
  "webpack-dev-middleware": "^3.4.0",
  "webpack-hot-middleware": "2.23.0"
}

gulpfile.js (shortened to relevant bits)
const webpack = require('webpack');
const webpackDevMiddleware = require('webpack-dev-middleware');
const webpackHotMiddleware = require('webpack-hot-middleware');
const browserSync = require('browser-sync');
const bs = browserSync.create();

gulp.task('serve', server);

function server() {
  const statsConfig = {
    colors: true, hash: false, version: false, assets: true, chunks: false,
    chunkModules: false, modules: false, children: false, reasons: false,
    source: false, errorDetails: false, chunkOrigins: false
  };

  const bundler = webpack(multiEntryConfig);

  bs.init({
    proxy: {
      target: 'example.com:8080',
      middleware: [
        webpackDevMiddleware(bundler, {
          stats: statsConfig,
          publicPath: '/s/',
          writeToDisk: true
        }),
        webpackHotMiddleware(bundler)
      ]
    }
  });
}

@glenjamin @okcoker I left a comment on the original commit that introduced the all: false stats.toJson option, but I'm pinging you here as well just incase that didn't make it through. Thanks for taking a look and helping unblock our local HMR with the latest version of webpack-hot-middleware!

@chiplay Instead of removing all: false does adding assets: true also fix the issue?

And actually looking at https://github.com/webpack/webpack/blob/9fe42e7c4027d0a74addaa3352973f6bb6d20689/lib/HotModuleReplacementPlugin.js#L276,
We might need hash: true as well.

We'll definitely need a test for this to make sure the output stats object has the expected keys in order to make the middleware and hmr plugin work.

So far all of the tests use a fake webpack instance, I suspect these sorts of changes are now hitting the limits of what confidence that type of testing provides.

@glenjamin @okcoker PTAL. I was able to go through and test all the various stats config options and found the guilty party (for me at least), which was cached: true. Adding that back in and leaving all: false gets HMR working again for us, and I'm guessing that's because we are using cacheDirectory: true in our babel-loader... but I'm not positive.
I took a look at the testing setup, but it's unfortunately beyond my abilities to spin up a real webpack instance to expand the testing coverage.

For future readers, here's the stats config documentation:
https://webpack.js.org/configuration/stats/
And what it says about cache: true

// Add information about cached (not built) modules
cached: true,

So I think it makes sense to include this.

Released in 2.24.3


allow email usernames in login_with_username

looks good @karabijavad. thanks for your commit.
i will wait till #81 is closed and then merge your PR.


Run as Windows Service

Is there any esy way to accoplish this?
TopShelf does not support .NET core.

Windows service can be made with Net Core (which is cross platform).
I think best way to achieve this is used NSSM (https://nssm.cc/) or other product like this.

I'm not sure about this one, but I'd say @Crazyht might have a good suggestion there!


Added types for SVG tags

Added most of the SVG tags on https://developer.mozilla.org/en-US/docs/Web/SVG/Element, excluding deprecated ones

Huge thanks 😍

Thank you @tpguy825 !!!
Any idea when you'll release a new version with this change @arthurfiorette? Thanks in advance!

I'm implementing async/suspense components. It will be released as soon as it's ready.


Issue when rapidly creating sandbox after changing from invalid wbs to valid wbs

Temporarely fix:
Either show the loading screen after Study save, or perhaps disable the "Create sandbox" and "Create dataset" buttons immediately after Study is saved, and then enable it again after request returns.

Create sandbox with invalid wbs
Change to valid wbs
Quickly create a sandbox and you get an error

This is done on the web app for sepes as a temporary fix


fix stack corruption and length calculations

Made as few changes as possible to fix the stack corruption and the wrong length calculations.
I'm not sure if the memory for FILE_RENAME_INFO could be released earlier, but this version (free after CloseHandle) works for sure.
fixes #2

Hi, sorry for getting back to you late - I'll merge these now. Thanks!


Add support for serializing list of variant-sized lists

Will fix #1129

@maartenbreddels @SylvainCorlay

So we discussed this approach in person with @martinRenou and I am happy with it.
I think we should probably discuss it with @maartenbreddels before merging.

Yes, I also want to add more JavaScript tests. All Notebooks seem to still work fine though.

We may also want to support arrays of arrays, I could push this:
diff --git a/bqplot/traits.py b/bqplot/traits.py
index 0181d2e0..a35571a3 100644
--- a/bqplot/traits.py
+++ b/bqplot/traits.py
@@ -158,7 +158,7 @@ def array_to_json(ar, obj=None, force_contiguous=True):
     if ar.dtype.kind == 'O':
         # Try to serialize the array of objects
         isstring = np.vectorize(lambda x: isinstance(x, six.string_types))
-        islist = np.vectorize(lambda x: isinstance(x, list))
+        islist_or_array = np.vectorize(lambda x: isinstance(x, (list, np.ndarray)))
         istimestamp = np.vectorize(lambda x: isinstance(x, pd.Timestamp))

         if np.all(istimestamp(ar)):
@@ -166,7 +166,7 @@ def array_to_json(ar, obj=None, force_contiguous=True):
             array_type = 'date'
         elif np.all(isstring(ar)):
             ar = ar.astype('U')
-        elif np.all(islist(ar)):
+        elif np.all(islist_or_array(ar)):
             return [array_to_json(np.array(row), obj, force_contiguous) for row in ar]
         else:
             raise ValueError("Unsupported dtype object")
diff --git a/tests/binary_serialization_test.py b/tests/binary_serialization_test.py
index c78e12b1..35af1273 100644
--- a/tests/binary_serialization_test.py
+++ b/tests/binary_serialization_test.py
@@ -105,7 +105,7 @@ def test_nested_list():

     data = np.array([
         [0, 1, 2, 3, 4, 5, 6],
-        [0, 1, 2, 2, 3],
+        np.array([0, 1, 2, 2, 3]),
         ['foo', 'foobar', 'bar'],
     ])

Thanks :smile:


EXC_BAD_ACCESS Error (on iOS) when using Stream

I am using stream in my weex project, the apps crash everytime I try to run it on iOS simulator. It works fine on android though.
If I comment out the stream code (viz commenting the code where I am using stream and import is still uncommented) the application works fine on both the platform.
Considering that stream code is working fine on android concludes that there is nothing wrong with my Vue code (maybe I am not doing some iOS specific thingy).
Here is the code:
       stream.fetch({
            method: 'GET',
            type: 'json',
            url: 'https://example.com/someUrl.json'
        }, res => {
            if(res.ok){
                this.isLoading = false;
                this.isError = false;

                this.number = res.data.number;
                this.name = res.data.name;
            }else{
                this.isLoading = false;
                this.isError = true;
            }
        });

The error which I get is on the following line:
        callbackRsp setObject:@{ @"OPENED": @1 } forKey:@"readyState"];
        progressCallback(callbackRsp, TRUE);

The error is:
        Thread 1: EXC_BAD_ACCESS (code=1, address=0x10)

Occuring on progressCallback line.
What I can think of is that it could be because of some package which I need to install or maybe version number of something is messed up because on playground it is working absolutely fine.
Any help would greatly appreciated.

This PR/issue doesn't received response since Weex migrated to Apache two years ago and this repository is no longer active yet.
Feel free to ask question in new repository and thanks for your contribution.

This issue is outdated for a long time and will be closed now. You can create a new one if you still have questions.


[PAGOPA-825] remove bundles by cdi

depends on PAGOPA-825 on afm-utils

List of Changes

added call to afm utils API to remove bundles by cdi

Motivation and Context

remove also the bundles when a cdi is deleted

How Has This Been Tested?

Manual and JUnit test

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 not work as expected)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

update_code

update_code

update_code

update_code

update_code


Apply deprecated eval_strategy

This PR applies the deprecated evaluation_strategy from https://github.com/huggingface/transformers/pull/30190.
TL;DR: evaluation_strategy -> eval_strategy
(Should go live in v4.41.0, but opening now so it's here when ready!)

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@BenjaminBossan will go live in v4.41.0 and stay until 4.46.0 as detailed in the thread  (6 months)

I'm still withholding review to wait for

will go live in v4.41.0 and stay until 4.46.0 as detailed in the thread (6 months)

@muellerzr Has the time come now?

@BenjaminBossan yes, time has come :)


[tpmtool] Adding documentation for new functionality

Adding a list of possible files that are generated when running tpmtool gatherlogs
Add links to TPM error codes and information flag so that users understand what they mean
Add documentation for tpmtool drivertracing
Add documentation for tpmtool parsetcglogs

@RonaldAi

#sign-off

Hi @JasonGerend , this PR is ready for review.  Let me know if you're not the right person to ping to get this reviewed and merged!

#sign-off


ncc Version 0.7.0 from npm show version as 0.6.0

Hello,
There is a problem with shown version.
$ ncc version
0.6.0

$ npm list -g | grep ncc
├──<EMAIL_ADDRESS>

Fixed


Replaced drop+create with CREATE IF NOT EXISTS

resolves #32
Description
This commit removes DROP operation and uses CREATE IF NOT EXISTS to create a new table
Checklist

[X] I have run this code in development and it appears to resolve the stated issue
[X] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the README.md file

Codecov Report

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

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             develop      #35   +/-   ##
==========================================
  Coverage           ?   44.42%
==========================================
  Files              ?       22
  Lines              ?      493
  Branches           ?       28
==========================================
  Hits               ?      219
  Misses             ?      274
  Partials           ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Single start and restart command (start if not running, restart if running)

I might have overlooked something, but is there a way to restart a script if it's already running, and start it otherwise?
I'm having this need in a deploy script I wrote for Meteor.
Thanks!

As a forever user, I'm going to be looking at solving this with systemd as a process manager in the future. systemd features three levels of off to help address is. In addition to stopping a service, you can also "disable" it.  Further, systemd offers a try-restart command, which is explicitly designed to only restart a service that's already running.
If forever adds this feature, I recommend that's also called try-restart to follow the same pattern.

For potential reference, this works fine:
forever restart $SCRIPT || forever start $SCRIPT


New mails in maildir

Is it possible to add the possibility to check whether there are new emails in a maildir/mbox/etc.?

In principle it wouldn't be hard, but I'm not aware of any libraries handling watching a maildir.  Maybe it wouldn't be too hard to cook up using inotify or something, but I'm not familiar with the mechanics of maildirs.

Yes, xmobar uses inotify. Thanks for your answer in any case!


Update Gemma to reflect upstream HF changes

Description
Fixes #594.
Since Gemma was merged in March 14th, there have been a number of upstream HuggingFace changes that mean our activations/logits no longer matched this implementation well. This PR implements the changes described here, changes include:

Changing the activation from standard GeLU to the tanh-approximated GeLU.
Keeping RMSNorm weights in float32
Setting the dtype of the embedding scale to match that in the HookedTransformer config.

A demo showing this agreement can be found here and summarised below. Note: There is currently a bug in various HF/transformer models where the default attention implementation used is not causally masking patterns, I opened an issue here but it is already being addressed here. For this reason it is important when comparing activations/logits to specify attn_implementation="eager" when using HF/transformers' from_pretrained method.
Tolerances for both 2b and 7b models when running in float32 can be found below:

2b
7b

Logits
5e-04
5e-03

Resid Pre Cache
5e-05
5e-04

Next Token Loss
1e-06
5e-05

Tolerances when running float16 are much worse but I think this is commonly the case when adding models to TL with internal upcasting/downcasting, if this is not the case though and these tolerances should also be similarly close I can investigate this further.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have not rewritten tests relating to key interfaces which would affect backward compatibility

Thanks Chris! looks good.


Allow locking/unlocking output mutex from outside the library.

This makes it possible to implement prints with multiple cmd_printf/cmd_vprintf calls
while guaranteeing no other thread can acquire the lock between them.
For example:
   cmd_mutex_lock();
   for (i = 0; i < 10; i++) {
       cmd_printf("%02x ", i);
   }
   // without locking a print from another thread could happen here
   cmd_printf("\r\n);
   cmd_mutex_unlock();

@jupe Could you please review this.

test this please

Jenkins doesn't seem to be reporting back the results.
All builds passed: https://jenkins-internal.mbed.com/job/mbed-client-cli/139/
x86-linux-native build marked as unstable: https://jenkins-internal.mbed.com/job/mbed-client-cli/label=linux,target=x86-linux-native/139
12:41:04 Code coverage enforcement failed for the following metrics:
12:41:04     Conditionals's stability is 60.23 and set mininum stability is 60.29.

What would be the best way to resolve this? For the coverage difference I'd suggest updating the limits...

all CI jobs are green, jenkins didn't update statuses unfortunately...


Beta LS Channel

There's no setting to set the channel for beta. @brettcannon In the issue you mention   "python.analysis.downloadChannel": "beta" should work, but it doesnt

It's not documented as it's not officially supported.
It should work, just that there won't be any intellisense in settings.json


functional tests (DaprStateStore/KV volume/MongoDB) are broken

The below tests are constantly failing.
=== CONT Test_DaprStateStoreTableStorage
 cli.go:325: [rad] Error: {
 cli.go:325: [rad] "code": "Internal",
 cli.go:325: [rad] "message": "the resource was not found"
 cli.go:325: [rad] }
 cli.go:325: [rad]
 corerptest.go:297:
 Error Trace: /home/vsts/work/1/s/test/functional/corerp/resources/corerptest.go:297
 Error: Received unexpected error:
 command 'rad application delete --yes -a corerp-resources-dapr-statestore-tablestorage' had non-zero exit code: exit status 1
 Test: Test_DaprStateStoreTableStorage
 Messages: failed to delete corerp-resources-dapr-statestore-tablestorage
=== CONT Test_MongoDB_Recipe

 corerptest.go:297:
 Error Trace: /home/vsts/work/1/s/test/functional/corerp/resources/corerptest.go:297
 Error: Received unexpected error:
 GET https://<IP_ADDRESS>:46307/apis/api.ucp.dev/v1alpha3/planes/radius/local/providers/Applications.Link/locations/global/operationStatuses/0055f6f4-253f-41ca-9faa-bf64058f982f
 --------------------------------------------------------------------------------
 RESPONSE 200: 200 OK
 ERROR CODE: Internal
 --------------------------------------------------------------------------------
 {
 "id": "/planes/radius/local/providers/Applications.Link/locations/global/operationstatuses/0055f6f4-253f-41ca-9faa-bf64058f982f",
 "name": "0055f6f4-253f-41ca-9faa-bf64058f982f",
 "status": "Failed",
 "startTime": "2023-01-30T08:37:09.444913383Z",
 "endTime": "2023-01-30T08:45:38.412337627Z",
 "error": {
 "code": "Internal",
 "message": "failed to delete resource \"/subscriptions/85716382-7362-45c3-ae03-2126e459a123/resourceGroups/radius-1675066911247905582/providers/Microsoft.DocumentDB/databaseAccounts/accnt-2c1fbb44-93b3-5878-bc1c-60bf9e5448e3/mongodbDatabases/mdb-2c1fbb44-93b3-5878-bc1c-60bf9e5448e3\": DELETE https://management.azure.com/subscriptions/85716382-7362-45c3-ae03-2126e459a123/resourceGroups/radius-1675066911247905582/providers/Microsoft.DocumentDB/databaseAccounts/accnt-2c1fbb44-93b3-5878-bc1c-60bf9e5448e3/mongodbDatabases/mdb-2c1fbb44-93b3-5878-bc1c-60bf9e5448e3\n--------------------------------------------------------------------------------\nRESPONSE 400: 400 Bad Request\nERROR CODE: BadRequest\n--------------------------------------------------------------------------------\n{\n \"code\": \"BadRequest\",\n \"message\": \"The requested operation cannot be performed because the database account accnt-2c1fbb44-93b3-5878-bc1c-60bf9e5448e3 state is not Online.\\r\\nActivityId: f9468031-838a-4bf0-96e8-633aadc6dddd, Microsoft.Azure.Documents.Common/2.14.0\"\n}\n--------------------------------------------------------------------------------\n"
 }
 }
 --------------------------------------------------------------------------------
 Test: Test_MongoDB_Recipe
 Messages: failed to delete corerp-resources-environment-recipes-env

corerptest.go:297:
 Error Trace: /home/vsts/work/1/s/test/functional/corerp/resources/corerptest.go:297
 Error: Received unexpected error:
 DELETE https://<IP_ADDRESS>:46307/apis/api.ucp.dev/v1alpha3/planes/radius/local/resourcegroups/kind-radius/providers/Applications.Core/containers/azstorage-ctnr
 --------------------------------------------------------------------------------
 RESPONSE 409: 409 Conflict
 ERROR CODE: Conflict
 --------------------------------------------------------------------------------
 {
 "error": {
 "code": "Conflict",
 "message": "The target resource is in progress state: Updating."
 }
 }
 --------------------------------------------------------------------------------
 Test: Test_PersistentVolume
 Messages: failed to delete corerp-azure-workload-env

AB#5888

Just did a quick scan of the tests that were marked skipped in the linked PR https://github.com/radius-project/radius/pull/5054/files, and remove flag no longer exists on any of these tests. Closing this issue.


Add send_unpublish_message method to email-alert-api

When called the method sends a request to the unpublishing-messages
endpoint in the email-alert-api. This endpoint starts the process of
notifying a user that their email subscription is no longer active.

I always forget this, so don't feel bad, but could you add a note to the CHANGELOG.

Sure 👍

Sure +1

Perfect :star:


Static landing page and footer links

This incorporates the changes requested by WebMOG to make the HTML websites publicly available.
It adds the static landing page that I have added to the Site root and which is currently displayed when visiting the non-language specific base url https://sites.ecmwf.int/docs/ifs-arpege-coding-standards . I figured it's useful to track this file as part of the repository even though an automatic deployment mechanism is likely unnecessary considering it's not intended to change.
I've used one of the web apps as a template, stripped java script & anything that looked unnecessary and filled-in the content. Please suggest changes to the appearance or content if required, and I will update the PR as well as the deployed version.
Furthermore, this PR adds links to terms of use, privacy statement and contact page that are placed in the footer of the generated Sphinx documentation.
Lastly, I removed the _static entry from the config files as this generated a build warning due to it's non-existence.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @reuterbal - please can you add a caveat to the link to the Python standard to indicate it isn't ready for use yet? (Or comment out that link till it is?) That's just a placeholder atm.


Release notes for Gradle 7.6-rc1 are contradictory to the documentation for convention plugins location.

In the release notes for 7.6 RC1 it mentions migrating from build-src to build-logic

Gradle now places convention plugins under the build-logic directory instead of in buildSrc.

However the sharing build logic between projects documentation still references using buildSrc.

We recommend putting source code and tests for the convention plugins in the special buildSrc directory in the root directory of the project. For more information about buildSrc, consult Using buildSrc to organize build logic.

Also the sample projects should be updated as well.
Expected Behavior
The documentation is updated to reference the new preferred location of build-Logic instead of the old buildSrc location.
Current Behavior
The documentation references the buildSrc as being the preferred method for the convention plugins, where as the 7.6 says the preferred location is build-logic.

Thank you for your interest in Gradle!
What's covered in the release notes is only applicable if you opt-in to the init command to generate builds that use incubating features. The overall documentation still covers the current stable recommended way.


update ipipe_py3.sh

完善CI任务：

增加cuda11+py38的测试环境
增加case失败输出pipeline.log
paddle版本更新为2.1.0

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zhangyulong04 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Fixed extraneous output from xclip on Linux when clipboard empty

Fixed an extra line of output on Linux when using xclip as the backend and trying to do a paste() and the clipboard is empty.
Also replaced 'utf-8' string with an ENCODING constant throughout.
This closes #97

Apparently this duplicates PR #96

Sorry for the delay. Thanks!


Investigate removal of url dependency

Since the http crate provides a Uri type which is used as the actual request url, it probably doesn't make sense to keep using the url crate in addition. It might only have been used because when we added it as a dependency, percent-encoding wasn't a separate crate yet.

Done in cbdbc0c.


Add a proper changelog

Right now, we do ad-hoc changelogs in GitHub release notes:
https://github.com/Qiskit/qiskit_sphinx_theme/blob/29277ce055a93a0debea8b627a196199957d3287/CONTRIBUTING.md?plain=1#L56-L59
It would be better to also have a proper changelog file in the repo. A main benefit is centralizing all the notes in one single place.
--
We probably do not need to use Reno for changelog generation. This repo is small and simple enough that manually maintaining a Markdown file is reasonable. Likely we should have PR authors update the changelog when making a change.
Relates to https://github.com/Qiskit/qiskit_sphinx_theme/issues/214 - we need to figure out how to backport the changelog when backporting bug fixes.

Going to close as low priority since this repo gets little traffic and few releases. Still a good idea to implement though if anyone wants to take it on.


Bump version to 0.2.140

Let me merge #3037 first.

:pushpin: Commit 0194551f4278b39932d50d60a20dc43490145070 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 0194551f4278b39932d50d60a20dc43490145070 with merge 51e3826642c5c53cb2ecad12efc644108daef9ec...

@bors retry

:hourglass: Testing commit 0194551f4278b39932d50d60a20dc43490145070 with merge ad7bbf4b5d4f61d2b05b6dad1bd5880e40fc22f7...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing ad7bbf4b5d4f61d2b05b6dad1bd5880e40fc22f7 to master...

Published on crates.io: https://crates.io/crates/libc/0.2.140


[BUG] - iOS cursor when long-pressing space key not working as expected when document is scrolled

Package Version
super_editor main branch (commit https://github.com/superlistapp/super_editor/commit/5d57d893890a740e080dd357305839ce0b0582aa).
User Info
Superlist
To Reproduce
Steps to reproduce the behavior:

Go to SuperEditor demo
Scroll the document (e.g. middle of document )
Long-press space key to move caret

Actual behavior
The red cursor is not seen and it's unclear how the position of the grey caret changes.
Expected behavior
Red cursor and grey carets are not in sync.
Platform
iOS
Flutter version
master
Screenshots
https://github.com/user-attachments/assets/0a7fa687-06e1-4449-8436-b8583d81d16e

Cc @matthew-carroll @angelosilvestre

FYI @knopp in case this ends up being a sliver issue

Seems like a potential regression. Will take a look.

This is a sliver issue. The floating point cursor is in viewport coordinates, which is not something that we can do in sliver world. We need to use document coordinates.
I was wondering how I missed this so I reverted to before the sliver PR and noticed that it was also subtly broken before, where in sliver scenario it was actually in document coordinates (that's why I missed it), but parts of the code, i.e. autoscroll, assumed it was viewport coordinates and did not work correctly. I'll look into this.


Could not find a version that satisfies the requirement pandas!=1.3.0

On the Pip install given in the readme
ERROR: Could not find a version that satisfies the requirement pandas!=1.3.0,>=1.2.0 (from brainio)

To replicate:
pip install git+https://github.com/brain-score/brain-score

What version of Python are you using?  I cannot reproduce on 3.8:
(base) jjpr-mit:brain-score jjpr$ docker run -it python:3.8 bash
Unable to find image 'python:3.8' locally
3.8: Pulling from library/python
23858da423a6: Already exists
...
root@42ce6b57e254:/# pip install git+https://github.com/brain-score/brain-score
Collecting git+https://github.com/brain-score/brain-score
  Cloning https://github.com/brain-score/brain-score to /tmp/pip-req-build-d9zkk24z
  Running command git clone --filter=blob:none --quiet https://github.com/brain-score/brain-score /tmp/pip-req-build-d9zkk24z
...
Successfully built brain-score brainio fire peewee result_caching
Installing collected packages: webencodings, wcwidth, tensorboard-plugin-wit, Send2Trash, pytz,
...
Successfully installed PyYAML-6.0 Send2Trash-1.8.0 absl-py-1.2.0 argon2-cffi-21.3.0 argon2-cffi-bindings-21.2.0 asttokens-2.0.8 astunparse-1.6.3 attrs-22.1.0 backcall-0.2.0 beautifulsoup4-4.11.1 bleach-5.0.1 boto3-1.24.77 botocore-1.27.77 brain-score-1.3 brainio-0.1.0 cachetools-5.2.0
...
root@42ce6b57e254:/# pip list
Package                      Version
---------------------------- -----------
absl-py                      1.2.0
...
botocore                     1.27.77
brain-score                  1.3
brainio                      0.1.0
cachetools                   5.2.0
...

I can try it under 3.7 if that's what you're using.

Ah, I fixed this by upgrading my python version once again to 3.10, and just stating that pandas was greater than 1.2.0. Sorry, I forgot to close the issue.


Create way for users to submit false positives / negatives

We should allow users to submit false positives or negatives easily. So that if they spot a hole in our system we can fix it quickly. We require PhotoDNA for this. This is not an optional step since we can have illegal photography on our servers and stored permanently. If someone wants to do this please reach out to<EMAIL_ADDRESS>first so we can discuss a solution and talk with Microsoft beforehand.

Information about PhotoDNA can be found here: https://www.microsoft.com/en-us/photodna


Adds checks into the Makefile for 32-bit platforms

Developers don't commonly think about 32-bit platforms and in one instance we broke a release due to this (quickly fixed in pre.7). This ensures arm and 386, used by dapr and trivy respectively are tested. It also helps prove these are in use to avoid a later developer removing support by thinking they aren't.
Don't read into these results, as they are new releases. This is just an example that shows the output of the commands:
$ gh release view -R aquasecurity/trivy --json assets --jq 'first(.assets[] | select(.name| test("Linux-32bit.*tar.gz")) | {url, downloadCount})'
{"downloadCount":1,"url":"https://github.com/aquasecurity/trivy/releases/download/v0.36.0/trivy_0.36.0_Linux-32bit.tar.gz"
$ gh release view -R dapr/dapr --json assets --jq 'first(.assets[] | select(.name = "daprd_linux_arm.tar.gz") | {url, downloadCount})'
{"downloadCount":16,"url":"https://github.com/dapr/dapr/releases/download/v1.9.5/dapr-operator.yaml"}

apologies to @knqyf263 and @berndverst

I think this check is valid for arm, but would love @berndverst to ack, just to make sure I didn't miss something. Since this runs on every PR, I'm hoping we can use it to trust contributors change wrt 32-bit compilation, without a lot of nervousness.

why this passes without any change while still failing to build on arm/linux?

long answer
pre.7 fixed platform code which didn't fall back to unsupported properly. The following is the fallback that's correct, basically the opposite of what compiler supports. so pre.7 already should work on arm, just this PR explicitly verifies it, at the cost of a longer make check
//go:build !(amd64 || arm64) || !(darwin || linux || freebsd || windows)

aside: we aren't currently publishing the wazero CLI on release, but doing so would also catch that.. at least building it would. I don't expect we want to release the CLI until the syntax stabilizes in the next month or two
(reminder 1.0 target slipped from end of feb to end of march, so that's when all api drift and cli drift should be done)

I'll merge this as I think this was the only thing impacting 32-bit, and I also want to have verification as new syscall support are added. If we are missing something else, easy PR!


FOOTER: Fix link to SDK

at "Developer Resources"
please link to
https://manual.threefold.io/#/
instead

fixed


Optimize Unest and implement skip_nulls=true if specified

Which issue does this PR close?

Closes #7087.
Closes #6961
Rationale for this change

This implements the new preserve_nulls option in unnest exec.
I've also adjusted the code to avoid copying and rely only on the take kernel as needed.
Todo

[ ] Set preserve_nulls to false -> If we really want align with clickhouse etc.

What changes are included in this PR?

The code will now respect the flag preserve_nulls  in UnnestOptions.
This way we can also make the sql api consistent with Clickhouse/duckdb while allowing dataframe users to choose between implementations.
Are these changes tested?
Yes, added some new tests as well.

Are there any user-facing changes?
No

No

@alamb Thanks for the review!
Do we want to set preserve_nulls = false anywhere in the code? I recall the idea was to keep the dataframe api use preserve_nulls = true but when using sql unnest would drop the nulls

Do we want to set preserve_nulls = false anywhere in the code? I recall the idea was to keep the dataframe api use preserve_nulls = true but when using sql unnest would drop the nulls

Setting preserve_null = false for the dataframe api would be a breaking change for users as data pipelines that use dataframe unnest  would now produce different results.

Do we want to set preserve_nulls = false anywhere in the code? I recall the idea was to keep the dataframe api use preserve_nulls = true but when using sql unnest would drop the nulls

I recommend that we merge this PR in without any changes to the default behavior and then we can propose a follow on PR with just the default behavior change (for SQL) by itself for discussion. That will make it easier to follow what exactly the proposed change in behavior is.


Test analysis mode

Not sure yet where we'll collect this data, but there should be a representative sample of privacy flags collected.
My initial idea is just to install the extension on my regular web browser to collect enough real data via regular browsing.
(we can't do this until I finish the data logging system, privacy-tech-lab/optmeowt-browser-extension#155 )

This is taken care of in #219.


Remove heartbeat timers

In GitLab by @nwalczak on Jul 29, 2019, 15:46

Service invoked too many times in a short time: properties

This message is popping up.  This is caused by the session heartbeat timers.  I will try increasing the time.  It seems like sometimes the function takes a lot longer to return, and this can lead to hitting the properties multiple times in a short window.  If increasing the timeouts doesn't help, we may have to scrap this feature.

In GitLab by @nwalczak on Jul 31, 2019, 11:09
I think we're going to have to remove the heartbeat timers.  I don't think they play well with the quotas imposed on google app scripts.  There is a limit on how many times a function can execute in a given time, and I think regardless of how long the heartbeat timer is, we will eventually have the right combination of invocations to hit this limit.  It would appear that there isn't a good way to keep track of active sessions.

In GitLab by @nwalczak on Jul 31, 2019, 11:10
changed title from {-Service invoked too many times in a short time: propertie-}s to {+Remove heartbeat timer+}s

In GitLab by @nwalczak on Jul 31, 2019, 11:10
changed the description

In GitLab by @nwalczak on Jul 31, 2019, 11:10
I've updated the title of this ticket to reflect the fact that we will have to remove the heartbeat timers.

In GitLab by @nwalczak on Jul 31, 2019, 13:10
Another example of problems that can be caused by the heartbeat timers... There's a limited amount of execution time per day that is allowed for each function.  If a user doesn't close the sidebar, the heartbeat timer doesn't stop calling the heartbeat function.  if the tab is left open in a browser, this timer could keep firing indefinitely, eventually running into the quota.

In GitLab by @nwalczak on Jul 31, 2019, 13:16
created branch 81-remove-heartbeat-timers

In GitLab by @nwalczak on Jul 31, 2019, 13:16
mentioned in merge request !33

In GitLab by @nwalczak on Jul 31, 2019, 13:36
closed via merge request !33

In GitLab by @nwalczak on Jul 31, 2019, 13:36
closed via commit fe53506e153f54a98a828078da2f9715de699236

In GitLab by @nwalczak on Sep 16, 2019, 11:47
mentioned in commit fe53506e153f54a98a828078da2f9715de699236


Andorid example don't work

When i use this https://github.com/chariotsolutions/phonegap-nfc/blob/master/doc/GettingStartedCLI.md
example and compile, install it dosn't work.
I compiled, installed and got information that put nfc into the device but when I put it nothing happens

What phone are you using? What version of Android? Is NFC enabled?
Are there any errors from adb logcat or javascript errors using chrome://inspect?
What type of NFC tag are you using? Is there an NDEF message on the tag?
Try reading the tag with https://play.google.com/store/apps/details?id=com.nxp.nfc.tagwriter. If that works write a new Text message back to the tag and try reading again with your app.

http://prntscr.com/h147wl
Here is tag
Nexus 7 tablet android 6

The problem is your device doesn't support that tag. See the message "NXP Mifare Classic 1k - Not Supported" in your screenshot.
The 2nd generation Nexus 7 switched to a Broadcom NFC chip which doesn't support Mifare Classic NFC tags. The Nexus 7 should work find with any Type 1 through Type 4 NFC tags.
You can read the UID from the tag if you'd like. Add a Tag Discovered listener to index.js. Replace your onDeviceReady function with this one:
onDeviceReady: function() {
    app.receivedEvent('deviceready');

    // Read NDEF formatted NFC Tags
    nfc.addNdefListener (
        function (nfcEvent) {
            var tag = nfcEvent.tag,
                ndefMessage = tag.ndefMessage;

            // dump the raw json of the message
            // note: real code will need to decode
            // the payload from each record
            alert(JSON.stringify(ndefMessage));

            // assuming the first record in the message has
            // a payload that can be converted to a string.
            alert(nfc.bytesToString(ndefMessage[0].payload).substring(3));
        },
        function () { // success callback
            alert("Waiting for NDEF tag");
        },
        function (error) { // error callback
            alert("Error adding NDEF listener " + JSON.stringify(error));
        }
    );

    // Handle non-NDEF tags and Mifare Classic tags with Broadcom readers
    nfc.addTagDiscoveredListener (
        function (nfcEvent) {
            var tag = nfcEvent.tag;
            alert(JSON.stringify(ndefMessage));
        },
        function () { // success callback
            alert("Waiting for non-NDEF tags");
        },
        function (error) { // error callback
            alert("Error adding tag listener " + JSON.stringify(error));
        }
    );

},


fixed websocket threading error onclose

Should fix the threading error,
checked it locally on my machine and it seems to be gone, behavior should be unchanged.
Should fix issue #116
not sure how to link it...

It will, but running on mainthread introduces again a really high ping, and the throughput of packages is also kind of low.

Thank you so much for taking your time for this PR @AJ92, I did merge https://github.com/colyseus/colyseus-unity3d/pull/119/ instead though which I think it's going to be easier to maintain


[Feature] Add discord link on front page (README.md)

Search before asking

[X] I searched the issues and found no similar issues.

Component
Other
Feature
recommending adding discord link in README.md ... something like this
Get Support
Have questions or feedback?  Join us on Discord server.
Data Prep Kit Discord Channel
invite link
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

i see the link on resources.md.  I am not sure if a user might think to check the resources for support community.  I am thinking we should highlight the support section on README.


App works only in one subnet that devices use (not server subnet)

I have created a server and app that allows me to use voice chat and text chat, but when i tried to use it with my friend that was in his house and me in mine, but when i am in one subnet on one ip address everything is okey, You can check everything on https://projectex.pl/coowork/voice_chat/comm.html?room=5575498
Is there any solution for this problem?
--
Edit #1 I want to add that i am using WSS protocol

Closed for inactivity. Also, needs more detail/explanation of what diagnostic steps have already been taken.


Material Tabs Bug

Describe the bug
If you have many tabs, you can easily slide onto hidden tabs to the right however you cannot go back to see previous tabs unless the name it's pretty long.
To Reproduce
Have tabs with small names, go all the way to the right, you can't go back
Expected behavior
Be able to return
Screenshots
First Example of Material Tabs on MetroDocs
Desktop (please complete the following information):

OS: [Windows]
Browser [chrome]
Metro Version [4.3.4]

use codepen.io to create example

Right now I can't do the codepen snippet, but I leave you here an example of what I mean, I can't go back

Hold the left button and move mouse

I already tried that, look at this gif:

added mouse wheel support for scrolling tabs, thanks for feedback


Problems connecting to Power BI Service, host unknown (login.microsoftonline.com:443)

I have a problem connecting to the Power BI service which is probably related to restrictions in out company:

Situation on workplace system within the company:
Deployment Wizard

powerbi://api.powerbi.com/v1.0/myorg/xxxxx
Integrated
---> error "host unknown (login.microsoftonline.com:443)"

Generally, browsing login.microsoftonline.com works fine and leads to an automatic connection.
Connection from SSMS with Authentication "Azure Active Directory - Password" works fine. (there is no browser Window popping up that would sign me in with my company account-it just works).

Situation on unrestricted PC: everything fine, SSMS works, TE3 works (Browser Window popping up to sign me in).
So I don't think a setting in Power BI Premium is missing.

Is this information sufficient so that you can recognize what the problem is?
I probably need to ask IT to change the restrictions but I don't know what to tell them.
Thanks & Regards
Michael

Which version of Tabular Editor are you using? Also, have you tried signing in with a username and password, instead of choosing integrated login? This should be equivalent to choosing "Azure Active Directory - Password" in SSMS:

I downloaded the latest version today, so it shows Build 3.3.8284.15691.
I also tried Username/Password, and I tried a different "situation", Load model from database (like in your screenshot), with both methods, integrated and username/pwd) -> always the same problem.
Do you think that additional ports are involved? I know that ports are restricted, but of course the one shown in the error message (443 / HTTPS) is not restricted.

Could be a proxy issue as well. Could you try switching between "System" and "None" under Tools > Preferences > Proxy settings. Also, if you're aware of the proxy details, try to use the "Custom" option:

Great - I used a custom proxy configuration and this solved the problem. Let's see whether TE also stops asking me for manual activation every 30 days :-)
Thank you very much!

Let us know if the manual activation prompt appears again, as this should also use the proxy settings.

ok, of course.


Inline storage view Instruction

Withdrawn in favor of having two separate instructions for pointer offsets and lensing into records.


Or operator inside value

Out of the other rule packages, I found this the best one for my needs.  So thank you for developing it.
I have one use-case which I'm sure complicates things further.
I needed the ability to match multiple values within a rule.
For example.  I may have the requirement of:
Expected -
People: Elon Musk OR Bill Gates OR Marissa Mayer
AND
Gender: Male
Actual -
People: Elon Musk
Gender: Male
I modified the compare method in ruler.go.
case "eq":
	if strings.Contains(expected.(string), ",") {
		sExpected := strings.Split(expected.(string), ",")

		for _, item := range sExpected {
			if strings.EqualFold(actual.(string), item) {
				return true
			}
		}

		return false
	}

	return actual == expected

Here is an example:
package main

import "github.com/hopkinsth/go-ruler"
import "fmt"

func main() {
	// can also build rules programmatically
	engine := ruler.NewRuler(nil)

	engine.Rule("name").Eq("Elon Musk,Bill Gates,Marissa Mayer")
	engine.Rule("gender").Eq("male")

	context := map[string]interface{}{
		"name":   "Bill Gates",
		"gender": "male",
	}

	result := engine.Test(context)

	fmt.Println(result == true)
}

Or perhaps you want to do something else?  But maybe you'll come up with something better and that all other compare methods can use.
Thanks again!

Are you mostly building rules programmatically, or are you using the JSON input? Or both?
This would be a good feature to have, but I would prefer to avoid overloading the value field in this way. I have existing uses in production that rely on being able to compare strings with commas in them as the whole string rather than as a list of alternates.
I would prefer an API that looks something like this:
engine := ruler.NewRuler(nil)
engine.Rule("name").
  .Eq("Elon Musk")
  .Eq("Bill Gates")
  .Eq("Marissa Mayer")

Ideally this would also change or build on top of the structure of the rule JSON. I propose the latter, so in addition to JSON that looks like this:
{"comparator": "eq", "path": "name", "value": "Bill Gates"}

you could also do something like this...
{
  "require": "(all | any)",
  "rules": [{
    "comparator": "eq",
    "path": "name",
    "value": "Bill Gates"
  }, {
    "comparator": "eq",
    "path": "name",
    "value": "Elon Musk"
  }, {
    "comparator": "eq",
    "path": "name",
    "value": "Marissa Mayer"
  }]
}

While this may be more verbose than simply using an array of possible values, I prefer it because it lets this library have nested rulesets (with different ANY vs. ALL conditions) easily as a result.
The only tricky thing about implementing that is that this library still needs to support parsing a JSON array of conditions and turning them into a single ruleset for evaluation. I have existing things that rely on that behavior. It's probably quick enough to compare the first non-nil, non-whitespace byte of JSON input against the [ or { character to know how to decode it.
Anyway, I'm kinda thinking out loud there. I'll work on this sometime soon.

Thanks for getting back.
I am building this programmatically from a database.
I'm looping through a bunch of rules and the criteria multiples exist within that:
for _, rrule := range ruleset.Rule {
	engine := ruler.NewRuler(nil)

	for _, criteria := range rrule.Criteria {
		values := strings.Join(criteria.Values, ",")

		if criteria.Condition == "equalto" {
			engine.Rule(criteria.Parameter).Eq(values)
		} else if criteria.Condition == "notequalto" {
			engine.Rule(criteria.Parameter).Neq(values)
		} else if criteria.Condition == "contains" {
			engine.Rule(criteria.Parameter).Matches(values)
		}
	}

	result := engine.Test(context)

	if result {
		....

	}
}

As you can see, chaining doesn't work for me.  The comma separated values work because that's what I get from the DB.
I don't use JSON.  No need to retrieve from the db and then put into JSON and then loop through it.  I see that as an extra step.
What would be nice for me, is what you propose but also when not wanting to use JSON or chaining:
engine.Require("all | any")
engine.Rule("name").Eq("Bill Gates")
engine.Rule("name").Eq("Elon Musk")
engine.Rule("name").Eq("Marissa Mayer")

But I am not sure if this is possible.

I think that could work but may require restructuring some things – I'll need to work through this. The model for how this would work could be something closer to https://github.com/rfink/verdict.js, which would give an API like this...
r1 := ruler.NewRuler(nil)
r1.Any()
r1.Rule("name").Eq("Bill Gates")
r1.Rule("name").Eq("Marissa Mayer")

r2 := ruler.NewRuler(nil)
r2.All() // this is the default
r2.Rule("name").Eq("Elon Musk")

main := ruler.NewRuler(nil)
main.All(r1, r2)
main.Test()

// also,
main = ruler.NewRuler(r1, r2)
main.Test()

I think that should give you what you need here.

I think this is fine!  Take your time on the changes.  The modification I made is fine for now.  But as soon as you do the changes, I'll re-implement and give feedback!


[Feature Request]: Warn user directly when custom loss is not differentiable

System information.
TensorFlow version (you are using): TF 2.11
Are you willing to contribute it (Yes/No) : Yes
Describe the feature and the current behavior/state.
Some people write custom loss functions from scratch that is not being differentiable. Then this leads getting None gradients while training. Warning users directly could be a useful feature instead of saying no gradients are provided.
There are way too many Stackoverflow posts about that, I'll include some of them:
https://stackoverflow.com/questions/63874265/keras-custom-loss-function-error-no-gradients-provided
https://stackoverflow.com/questions/73197501/raise-valueerror-no-gradients-provided-for-any-variables-custom-loss-function
https://stackoverflow.com/questions/59292992/tensorflow-2-custom-loss-no-gradients-provided-for-any-variable-error
https://stackoverflow.com/questions/65619581/no-gradients-provided-for-any-variable-for-custom-loss-function
https://stackoverflow.com/questions/70537503/custom-loss-function-error-valueerror-no-gradients-provided-for-any-variable
https://datascience.stackexchange.com/questions/116645/custom-loss-function-for-binary-classificatio-in-keras-gets-error-no-gradients
https://stackoverflow.com/questions/74074934/error-no-gradients-provided-for-any-variable-while-using-custom-loss
https://stackoverflow.com/questions/75738678/gradienttape-returning-none-with-custom-csi-loss-function
https://stackoverflow.com/questions/72259489/valueerror-no-gradients-provided-for-any-variable-custom-loss-function
...
Will this change the current api? How?
This will change the current API by adding some checks on loss function before starting to training, an error/warning can be thrown.
Contributing

Do you want to contribute a PR? (yes/no): Yes
Briefly describe your candidate solution(if contributing): Probably add some checks using GradientTape in: https://github.com/keras-team/keras/blob/17af3fcb1d21f950fff097e0534a6ae56bd25a46/keras/engine/compile_utils.py#L104

@Frightera,
I tried to execute the code which was gist provided on the related PR https://github.com/keras-team/keras/pull/17753 on tf-nightly and the keras 3.0 version and it was executed without any error/issue. Kindly find the gist of it here. Thank you!


[Refactor] Mark utf8_len inline to suppress -Wunused-function

What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[X] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Problem Summary(Required) ：

Suppress -Wunused-function due to utf8_len.
/build/starrocks/be/src/util/utf8.h:107:12: error: 'int starrocks::vectorized::utf8_len(const char*, const char*)' defined but not used [-Werror=unused-function]
static int utf8_len(const char* begin, const char* end) {
              ^~~~~~~~
cc1plus: all warnings being treated as errors

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[X] I have checked the version labels which the pr will be auto backported to target branch

[X] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_admit_test

@Mergifyio backport branch-2.5


Check if item exists in keychain without Touch ID auth

Is this possible? Is it possible to see if a value exists at a given key in a keychain secured with Touch ID without prompting the user for their finger print?
If not, is there a way to check if the keychain, as a whole, exists without prompting for authentication?
I was torn as to whether I should ask this on here or on StackOverflow. I hope you don't mind me asking here? :innocent: :sweat_smile:. I figured since I'm using your framework and couldn't find anything about my question in the README, I'd be coming back to you with any answer I got on SO to see how I do it via KeychainAccess.
Many thanks!

@kylejm I've tried a various way of that, there is no way to check whether the value exists or not, I think :cry: . If there is a possibility that the value would be protected with Touch ID, you should always execute keychain method asynchronously.

Ah that's lame. No worries, I've worked around the problem by storing a flag to mark that the credential is stored.  Thanks for the reply :)

Apparently you can now supply the kSecUseAuthenticationUI key to control when the authentication prompt is used, default is to show it, but you can get it to fail or skip silently:
https://developer.apple.com/reference/security/ksecuseauthenticationui

@kishikawakatsumi would something like this be acceptable? Keeping in mind the below code is just a generic keychain query for example purposes, also the code below is specific to something we know is locked behind TouchID in the keychain. We currently use this to solve that problem. I'd def like to be using this as part of the framework.
public func contains(_ key: String) -> Bool {
        // We spcify kSecUseAuthenticationUIFail so that the error
        // errSecInteractionNotAllowed will be returned if an item needs
        // to authenticate with UI and the authentication UI will not be presented.
        let keychainQuery: [AnyHashable: Any] = [
            kSecClass as AnyHashable: kSecClassGenericPassword,
            kSecAttrService as AnyHashable: Bundle.main.bundleIdentifier!,
            kSecAttrAccount as AnyHashable: key,
            kSecUseAuthenticationUI as AnyHashable: kSecUseAuthenticationUIFail
        ]

        var result: AnyObject?
        let status = SecItemCopyMatching(keychainQuery as CFDictionary, &result)

        // If that status is errSecInteractionNotAllowed, then
        // we know that the key is present, but you cannot interact with
        // it without authentication. Otherwise, we assume the key is not present.
        return status == errSecInteractionNotAllowed
    }

Maybe something like this in Keychain
public func contains(_ key: String) throws -> Bool {
        var query = options.query()
        query[AttributeAccount] = key
        query[UseAuthenticationUI] = UseAuthenticationUIFail

        let status = SecItemCopyMatching(query as CFDictionary, nil)
        switch status {
        case errSecSuccess, errSecInteractionNotAllowed:
            return true
        case errSecItemNotFound:
            return false
        default:
            throw securityError(status: status)
        }
    }

I am encountered a strange issue, SecItemCopyMatching  returns errSecSuccess,  however,  both SecItemAdd and SecItemUpdate failed with something like,

// SecItemAdd status: -25299 error: The specified item already exists in the keychain
// SecItemUpdate status: -25303 error: The specified attribute does not exist.

@ChristopherCarranza Great thinking! This was an amazing workaround!

Noting that UseAuthenticationUIFail is deprecated as of iOS 14.
I'm using something like the following...
    func has(service: String, account: String, options: [Options] = []) -> Bool {
        var query = self.query(service: service, account: account, options: options)
        query[kSecUseAuthenticationContext as String] = internalContext

        var secItemResult: CFTypeRef?
        status = SecItemCopyMatching(query as CFDictionary, &secItemResult)

        if status == errSecSuccess || status == errSecInteractionNotAllowed {
            return true
        }

        return false
    }

    private var internalContext: LAContext? = {
        let context = LAContext()
        context.interactionNotAllowed = true
        return context
    }()

@hmlongco
Thanks for your example.
I have this comment above query setup.
        // specify kSecUseAuthenticationUIFail so that the error
        // errSecInteractionNotAllowed will be returned if an item needs
        // to authenticate with UI and the authentication UI will not be presented.

Will this behavior persist if I change query to LAContext-based query with context.interactionNotAllowed = true?

@hmlongco
Thanks for your example.
I have this comment above query setup.
        // specify kSecUseAuthenticationUIFail so that the error
        // errSecInteractionNotAllowed will be returned if an item needs
        // to authenticate with UI and the authentication UI will not be presented.

Will this behavior persist if I change query to LAContext-based query with context.interactionNotAllowed = true?

After playing around with it a lot I noticed some inconsistent behaviour of the SecItemCopyMatching status response.
Sometimes, it returns 0,  meaning errSecSuccess. So to truly check if an item is there, you should also check for that code.


Core js splitting - second part

@ichim-david @sneridagh pr is ready for review.
Unfortunatly a cypress test for 'locking' fails, but i don't understand why... no code related to locking is touched by this pr.. do you have some idea?

@giuliaghisini maybe it's a flaky test that passes after a re run or increase of wait timer. I will have a look at this next week when I'm back to work, right now I'm waiting for the flight delay to pass :)
Thank you for the work so far, I cannot wait to test it :)

@ichim-david yeah I was looking at this right now and I don't think there will be time to polish this before thursday. The fact that most acceptance runs break makes me think something needs to be fixed and it's not just flakiness


Fix typos: add missing apostrophes

From the Pull Request Template:

** Please don't send typo fixes! **.
Please don't send a PR solely for the purpose of fixing a typo, unless that typo truly hurts understanding of the text. Each PR represents work for the maintainers, and that work should provide commensurate value.


Improving the documentation

We should take a look at the Mercurius Auth documentation to make sure it is super clear for new and existing users. We should also make sure it provides everything a developer needs to use and understand Mercurius Auth (without having to look closer at the underlying code/tests for example). Before starting an implementation, it would be great to understand the following:

Does the docs structure make sense? What could be improved?
Is the README sufficient? Is it missing any key information?
Do we want a best practices/examples section?
Anything else that is missing or think should be included?

Keen to hear everyone's thoughts on this!

To share some real user thoughts I'm currently hitting while implementing.  Where I can't figure out when applyPolicy is even called. I was thinking it would be called anytime a query with the @auth directive on it existed, but I can't seem to get it to fire at all.

Thanks for the feedback, that's super helpful! :)
To summarise your key points:

Better description on the difference between external and directive options
Make sure the examples better match the intended schema
Add some examples on the best way to access the directive value (note, this is currently the best way as we should have everything from the AST). We could maybe expose some helpers for this perhaps? In general, I think we could provide some more usage example here!

Would you be interested in helping refine the documentation?

Would you be interested in helping refine the documentation?

I'd love to take a stab at it.

Awesome! Feel free to drop a draft PR in the repo and we can work on it together :)

@stolinski I was thinking of starting this work later this week, would you be still interested in helping out?