Cleanup labrad-plot.

In the process of updating the plotter to use WebGL, I have reorganized labrad-plot.ts to be more in line with Google Style standards.

Made the maximum line width 80 characters.
Updated comments to start with capitals and end with full stops.
Converted all uses of var to let or const.
Made all private methods end in a suffix.
Ensured all public methods have a comment.

I also reorganized the file to make it easier to follow:

Grouped observers and listeners and renamed them appropriately.
Made functions that the UI interacts with public.
Renamed UI functions to make their use clearer.
Moved all utility functions to the end of the file.

I have tried to keep each commit as atomic to one action as I could so that review could be made easier by looking at each commit individually. If there is a better process for submitting changes like this, please let me know (chaining a bunch of dependent PRs seemed like a rebasing nightmare).

I branched this off of https://github.com/labrad/scalabrad-web/pull/201, I'm not sure how to deal with these extra commits on this PR your rebase workflow.

Thanks for breaking this branch up logically and thanks for cleaning it up. Going thru the PR now.
Firstly the grapher list doesn't render correctly for me. I get a newline after the icon:

The above comment is a problem in Pull #201 so, I'll start reviewing there?

Is there a way to review this despite the other branch being broken? They aren't even really dependent, I just rebased in the changes because it made testing quicker with the more performant list.

Most of it is just cosmetic so shouldn't be too hard to review on it's own. I like to check that the app functions before giving the go ahead though...

I have rebased and dropped the commits from the other (now abandoned) branch.

A few small comments; otherwise LGTM. Thanks for keeping the individual commits small and focused; that made the review much easier.

LGTM


Now playing doesn't work

@ayush5harma My device is Redmi 4X, which is installed lineageos 18.1, After flashing your module, I saw the option, but it is not available. And the pixel Ambient service also does not work.

Use a custom ROM that is based on AOSP . ROM's using lineage as base or lineage itself doesn't go well with google things.

But I have used PE before and it still doesn't work.
On Fri, Feb 12, 2021, 3:53 PM Ayush Sharma<EMAIL_ADDRESS>wrote:

Use a custom ROM that is based on AOSP . ROM's using lineage as base or
lineage itself doesn't go well with google things.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ayush5harma/PixelFeatureDrops/issues/29#issuecomment-778038359,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS2B57FKU25CAFQVVFTDFKDS6TM6FANCNFSM4XQHJJRQ
.

Install Pixel Ambient Services from playstore.

But I have used PE before and it still doesn't work.
…
On Fri, Feb 12, 2021, 3:53 PM Ayush Sharma @.***> wrote: Use a custom ROM that is based on AOSP . ROM's using lineage as base or lineage itself doesn't go well with google things. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#29 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS2B57FKU25CAFQVVFTDFKDS6TM6FANCNFSM4XQHJJRQ .

But I have used PE before and it still doesn't work.
…
On Fri, Feb 12, 2021, 3:53 PM Ayush Sharma @.***> wrote: Use a custom ROM that is based on AOSP . ROM's using lineage as base or lineage itself doesn't go well with google things. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#29 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS2B57FKU25CAFQVVFTDFKDS6TM6FANCNFSM4XQHJJRQ .

Install Pixel Ambient Services  from playstore

I just downloaded it from the Play Store.
On Fri, Feb 12, 2021, 4:10 PM Ayush Sharma<EMAIL_ADDRESS>wrote:

Install Pixel Ambient Services from playstore.
But I have used PE before and it still doesn't work.
… <#m_2175738016429752733_>
On Fri, Feb 12, 2021, 3:53 PM Ayush Sharma @.***> wrote: Use a custom
ROM that is based on AOSP . ROM's using lineage as base or lineage itself
doesn't go well with google things. — You are receiving this because you
authored the thread. Reply to this email directly, view it on GitHub <#29
(comment)
https://github.com/ayush5harma/PixelFeatureDrops/issues/29#issuecomment-778038359>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS2B57FKU25CAFQVVFTDFKDS6TM6FANCNFSM4XQHJJRQ
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ayush5harma/PixelFeatureDrops/issues/29#issuecomment-778045363,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS2B57GXMYSVTG52LKPFVCLS6TO5XANCNFSM4XQHJJRQ
.

I just downloaded it from the Play Store.
…
On Fri, Feb 12, 2021, 4:10 PM Ayush Sharma @.> wrote: Install Pixel Ambient Services from playstore. But I have used PE before and it still doesn't work. … <#m_2175738016429752733_> On Fri, Feb 12, 2021, 3:53 PM Ayush Sharma @.> wrote: Use a custom ROM that is based on AOSP . ROM's using lineage as base or lineage itself doesn't go well with google things. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#29 (comment) <#29 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS2B57FKU25CAFQVVFTDFKDS6TM6FANCNFSM4XQHJJRQ . — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#29 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS2B57GXMYSVTG52LKPFVCLS6TO5XANCNFSM4XQHJJRQ .

all what's mentioned is for aosp base these things are tighly coupled with google services something lineage is known to break

I just downloaded it from the Play Store.
…
On Fri, Feb 12, 2021, 4:10 PM Ayush Sharma @.**> wrote: Install Pixel Ambient Services from playstore. But I have used PE before and it still doesn't work. … <#m_2175738016429752733> On Fri, Feb 12, 2021, 3:53 PM Ayush Sharma @.**_> wrote: Use a custom ROM that is based on AOSP . ROM's using lineage as base or lineage itself doesn't go well with google things. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#29 (comment) <#29 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS2B57FKU25CAFQVVFTDFKDS6TM6FANCNFSM4XQHJJRQ . — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#29 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS2B57GXMYSVTG52LKPFVCLS6TO5XANCNFSM4XQHJJRQ .

all what's mentioned is for aosp base these things are tighly coupled with google services something lineage is known to break

If you face issues with the features mentioned in readme  being on supported custom ROM there are fixes in the sidenote if still something doesn't work  you can open an issue.Closing this for now


Improve and fix rspec

Fix some rspec related errors
Use "expect" syntax instead of "should" syntax

See https://github.com/locaweb/bpmachine/pull/2


chore(git): ignore Intellij settings

Summary
This PR adds .idea/ to .gitignore.
Test Plan
None.

Thanks for contributing!
I would highly recommend putting these in a user-level gitignore instead.
I prefer not to include things unrelated to the repository itself in its gitignore.


Purpose of checkForFix()

What is the purpose of checkForFix()? Can't the same process be incorporated into the getCoord() function?

It separates checking for a GPS fix. It can be incorporated into the getCoord() function but wasn't for organization's sake.


Freudenberg - The number of jobs posted on Peviitor and the one returned by the API isn't the same.

ENVIROMENT: production
URL: https://peviitor.ro/
Browser: Chrome
Device: laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open "https://peviitor.ro/" in the browser.
Type "Freudenberg" in the search bar.
Check the number of jobs available for Freudenberg.
Run "Freudenberg" scraper on "https://dev.laurentiumarian.ro/scraper/based_scraper_py/".
Check the number of jobs displayed.

EXPECTED RESULTS:
The number of jobs posted on "https://peviitor.ro/" and the number of jobs returned by the API should be the same.
ACTUAL RESULT:
The number of jobs posted on Peviitor website is 9 while the number of jobs returned by the API is 11.

Fixed

After retesting the number of jobs posted on peviitor and the one returned by the API are the same (15)


Is there hls support on the web?

Is there hls support on the web?

On the example I save the HLS qualities on Files using dart.io and path_provider. The 2 APIs don't have Web support

On the example I save the HLS qualities on Files using dart.io and path_provider. The 2 APIs don't have Web support

So, as far as I understand, live streaming does not work on the web. which files open on the web

So, as far as I understand, live streaming does not work on the web. which files open on the web


Hot client port seems to be ignored

Operating System: Alpine 3.7.0
Node Version: 10.6.0
NPM Version: 6.1.0
webpack Version: 4.15.1
webpack-serve Version: 2.0.2
webpack-hot-client Version: 4.1.0

This issue is for a:

[x] bug
[ ] feature request
[ ] modification request

CLI Command
  $ webpack-serve docs/webpack.config.js

serve.config.js
module.exports = {
  clipboard: false,
  devMiddleware: {
    publicPath: '/pattern-library/dist/'
  },
  hotClient: {
    allEntries: true,
    host: {
      client: '*',
      server: '<IP_ADDRESS>'
    },
    port: {
      client: 8082,
      server: 8081
    }
  },
  host: '<IP_ADDRESS>'
}

Expected Behavior
Websocket client uses client port 8082.
Actual Behavior
Websocket client try using server port 8081 and fails.

How Do We Reproduce?
Set hotClient.port as an object.

Maybe this is a webpack-hot-client issue. options.webSocket.port changes between modifyCompilercall (where it has the right value) andbeforeCompileevent (where it takes value fromport.server`).
No idea why this happens.

Yeah, that's an issue for webpack-hot-client. Please open it up over at https://github.com/webpack-contrib/webpack-hot-client

Done https://github.com/webpack-contrib/webpack-hot-client/issues/86


[SPARK-11903] Remove --skip-java-test

Per @pwendell's comments on SPARK-11903 I'm removing this dead code.
If we are concerned about preserving compatibility, I can instead leave the option in and add a warning.
For example:
echo "Warning: '--skip-java-test' is deprecated and has no effect."
;;

cc @pwendell, @srowen

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46567/
Test FAILed.

Jenkins, retest this please. This LGTM - I think it's good to simply remove it.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46570/
Test FAILed.

Jenkinstein, retest this please!

Test build #46579 has started for PR 9924 at commit edf44c1.

Test build #46579 has finished for PR 9924 at commit edf44c1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46579/
Test PASSed.

I've merged this. Thanks.


multi-threaded transport for JRT

@baldersheim please review
@bjorncs @ollivir FYI

@baldersheim please take another look


helm: verbs install/delete are confusing compared to kubectl

Coming from kubectl I expect kubectl create to create things on the cluster and kubectl delete to delete things on the cluster.
With helm today things are a bit confusing when I come from that experience. helm create creates an empty chart on disk. helm delete deletes a chart that we previously installed with helm install.
I think there are three ways to fix this:

helm delete could be renamed to helm uninstall
helm create could be renamed helm new or helm new-chart then helm install could be renamed
We move around the help text around to contain different categories

Repo Commands
  repo        add, list, or remove chart repositories
  search      search for a keyword in charts
  update      update information on available charts in the chart repositories

Local Commands:
  home        displays the location of HELM_HOME
  serve       start a local http web server
  version     print the client version information

Release Commands:
  delete      given a release name, delete the release from Kubernetes
  get         download a named release
  install     install a chart archive
  list        list releases
  upgrade     upgrade a release
  status      displays the status of the named release
  rollback    roll back a release to the previous version

Cluster Commands:
  init        initialize Helm on both client and server

Chart Commands:
  create      create a new chart with the given name
  inspect     inspect a chart
  fetch       download a chart from a repository and (optionally) unpack it in local directory
  lint        examines a chart for possible issues
  package     package a chart directory into a chart archive
  verify      verify that a chart at the given path has been signed and is valid

My personal preference would be to match kubectl habits with 2. But, would love feedback.

I tried to map these in the verb noun way so that I could get a visual of that. I also created a dev subcommand for some of the one offs but it does not map to verb noun. I may have missed some combos in the process but so that we can get an idea:
helm create release CHART_NAME
helm create repo REPO_NAME REPO_URL
helm delete repo REPO_NAME
helm delete release RELEASE_NAME
helm list charts [--repo REPO_NAME] [SUBSTRING]
helm list releases
helm describe release RELEASE_NAME
helm describe repo REPO_NAME
helm dev create-chart PATH_TO_CHART
helm dev serve PATH
helm dev package PATH_TO_CHART
helm dev lint PATH_TO_CHART
helm verify chart URL_FOR_CHART
helm rollback release RELEASE_NAME
helm update release RELEASE_NAME URL_FOR_CHART
helm update repo [REPO_NAME]

I have not really seen a tool that uses the VERB NOUN combo instead of NOUN VERB. Can you point me to an example @viglesiasce ?
For background, here are the cold commands: https://github.com/kubernetes/helm/tree/1.999.0/cmd/helm
You'll notice that we used to do things the way this ticket suggested. But based on feedback, we changed it (viz. uninstall is now delete, and we bubbled up common commands to follow helm VERB instead of helm NOUN VERB).

@technosophos kubectl is verb noun. kubectl describe service -n kube-system kube-dns

Yeah I think keeping things similar to kubectl makes sense.
We had really wanted to keep things as close as possible to package managers ({yum,apt} install) but I think that most helm users will have had some experience with kubectl already. Additionally, package managers only really have one noun (packages) which makes them simpler from a ux perspective. The repos are handled by the filesystem (/etc/yum/repos.d/ or /etc/apt/sources/list.d/). We are also baking in the package maintainer tools with helm which is split out in RPM land, for example.

I think you guys are maybe misunderstanding kubectl. There is a distinction between CMD NOUN VERB and CMD VERB ARGV....
For the most part, kubectl follows two conventions:
kubectl VERB ARGS
kubectl NOUN

Take @philips's example: kubectl describe service -n kube-system kube-dn. In that case service is not playing the role of NOUN. It's just an argument in argv.
Same with kubectl get pods. pods isn't a subcommand, it's an argument. This is like helm list foo. foo is an argument.
Then there are a few of the NOUN VERB ARGS formats, mostly in kubectl config:

kubectl config set
kubectl config use-context
...

And then a few NOUN ones:

kubectl cluster-info
kubectl api-version
kubectl logs

I can't actually find any that are of the form kubectl VERB NOUN, where NOUN is a fixed word. It's always just an arg.
Again, I'm going to appeal to well-known UNIX programs, which tend to follow the CMD VERB or CMD NOUN VERB convention. I'd honestly feel much more comfortable going to things like helm repo-add than doing helm add repo.
Now, to take things another step, what would your help text look like for the VERB NOUN format for which you are advocating? How do I find out, for example, what things I can do with Helm repos? To see the relevant commands, I'd have to do helm add -h && helm index -h && helm list -h && helm remove -h -- and that's only because I know what the verbs are. This is why things are typically grouped into NOUN subgroups. Think of it from the user's perspective. What does the user want to know? What sorts of things they can delete, or what actions they can do on a release?

For folks looking for the simplest answer to this , just run helm list, find the name of your installed chart, and delete it -  helm delete xyz


Provide integration with nuget.org to publish our packages

The idea is to publish two packages to nuget:

vivarni.domain
vivarni.domain.infrastructure

Need to figure out how this works.
What workflow do we need?
Perhaps we also need to split the current repository in two?

https://github.com/marketplace/actions/publish-nuget seems interesting

Fixed! Thx @Clivar  :-D


Cannot use for domain that uses ALIAS record

Describe the bug
Attempting to update/renew certificate on a domain that uses an ALIAS record fails, because it attempts to update the resolved ALIAS domain instead of the domain requested
To Reproduce

Set an ALIAS record for a root domain, instead of an A/AAAA record
Attempt to create/renew certificate for that domain

Expected behavior
Certificate renewal is successful
Certbot command
certbot  certonly -v --test-cert --server https://acme-staging-v02.api.letsencrypt.org/directory --authenticator dns-porkbun            --dns-porkbun-credentials /etc/letsencrypt/porkbun/porkbun.ini --cert-name test-d example.com
Versions (please complete the following version information):

certbot: 2.4.0
certbot_dns_porkbun: 0.8

Error message
2024-01-01 15:29:04,837:DEBUG:urllib3.connectionpool:https://porkbun.com:443 "POST /api/json/v3/dns/create/dyndns.org HTTP/1.1" 400 None
2024-01-01 15:29:04,843:DEBUG:certbot._internal.error_handler:Encountered exception:
Traceback (most recent call last):
  File "/opt/venv/lib/python3.11/site-packages/certbot_dns_porkbun/cert/client.py", line 109, in _perform
    self.record_ids[validation] = self._get_porkbun_client().dns_create(self._root_domain,
                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/venv/lib/python3.11/site-packages/pkb_client/client.py", line 119, in dns_create
    raise Exception("ERROR: DNS create api call was not successfully\n"
Exception: ERROR: DNS create api call was not successfully
Status code: 400
Message: Invalid domain.

Note trying to request a cert for "dyndns.org" (the root of the domain that my domain is pointing at with an ALIAS record) and not my domain

Thank you for your bug report.
ANAME/ALIAS records are treated like CNAME and DNAME records according to the specification of the DNS-01 challenge, i.e. the DNS-01 challenge domain _acme-challenge.<fqdn> is resolved and then the challenge value is set for this resolved domain.
For example, the following DNS records are equivalent in this context:
_acme-challenge.mydomain.com. 600 IN ALIAS _acme-challenge.example.com.
_acme-challenge.mydomain.com. 600 IN CNAME _acme-challenge.example.com.

The failed attempts as you describe should only occur if you use a wildcard ALIAS entry.
However, I cannot complete the debugging with your current information, so I would ask you to provide the exact DNS constellation. You can mask the domain names, but you should keep the number of subdomains.
For this reason, the following possible solutions are only general and may not apply depending on the DNS constellation.
Solutions:
Modify DNS ALIAS record
So if you use a DNS record like this one:
*.mydomain.com. 600 IN ALIAS example.com.

You have to adjust it and remove the wildcard. Because every request for the challenge _acme-challenge.mydomain.com. is forwarded to _acme-challenge.example.com. As you have specified this with the ALIAS record.
Forward challenge to other controlled domain
You can also create a DNS record that points to a domain/subdomain that does not forward and is a Porkbun domain that you control.
For example:
_acme-challenge.mydomain.com. 600 IN CNAME mydomain2.com.

Create a dummy A record
The domain for the challenge is resolved according to the DNS-01 specification, so you can also create a dummy DNS A record. In this case, the existing DNS ALIAS wildcard record would be retained and the DNS-01 challenge would still be performed successfully.
For example:
*.mydomain.com. 600 IN ALIAS _acme-challenge.example.com.
_acme-challenge.mydomain.com. 600 IN A <IP_ADDRESS>

The dummy A record seems like a reasonable option, I'll give that a try. Note that this same setup worked fine with my previous registrar (gandi) and the plugin I was using for them.
The full DNS records for the domain are:
@ 600 IN ALIAS mydomain.dyndns.org
*.mydomain.io. 600 IN ALIAS mydomain.dyndns.org
@ 600 IN MX 10 fwd1.porkbun.com
@ 600 IN MX 20 fwd2.porkbun.com
@ 300 IN TXT <SPF config>
default._domainkey.mydomain.io 300 IN TXT <DKIM config>
_dmarc.mydomain.io 300 IN TXT <DMARC config>

I've updated the wildcard to be a CNAME pointing at my root, though at the back of my mind I feel like that causes other issues when the root is an ALIAS. I'll try that once it propagates and, if that fails, I'll try the dummy A record for _acme-challenge. Thanks!

Using a dummy A record works fine. Thanks again for the help!


About is now dynamic feature

About became a dynamic feature
:about_entrance module was added as a gateway to the about feature
DynamicFeaturesComponent got necessary dependencies
Resolves #157

This PR was released with 0.22.1


Give me ur Disc for cheataway 2

It's happening. disc = kdot_227

whats a disc

whats a disc

Discord user bruh


links not being recognized

when I paste a link in the editor, it is not being recognized, and display the link as plain text.
any idea how to make it clickable automatically?
also opened on the core repo
https://github.com/codex-team/editor.js/issues/1309

Editor has an onPaste event, should implement a custom callback
Anyways I think theres a link tool, where this functionality is handled


TypeError: array() takes 1 positional argument but 2 were given

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code: yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOs Big Sur
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v2.4.0-49-g85c8b2a817f 2.4.1
Python version: 3.8.7

Describe the current behavior
I'm trying to assign the return value of get_value() to a numpy array within a method decorated using tf.function. It works only if I call the method directly. However, if I use the result of tf.numpy_function() (in case3), I get an error.
Describe the expected behavior
I don't know, but the error is not doing any good explaining why this is happening
Standalone code to reproduce the issue
Please find the notebook here which contains the following:
import numpy as np
import tensorflow as tf

class Foo:
    def __init__(self):
        self.storage = np.zeros((10, 3, 1))

    @staticmethod
    def get_value():
        return np.array([[1], [2], [3]])

    @tf.function
    def case1(self):  # works
        self.storage[0] = self.get_value()

    def case2(self):  # works
        self.storage[0] = tf.numpy_function(self.get_value, inp=[], Tout=tf.float32)

    @tf.function  # fails
    def case3(self):
        self.storage[0] = tf.numpy_function(self.get_value, inp=[], Tout=tf.float32)

if __name__ == '__main__':
    foo = Foo()
    foo.case1()
    foo.case2()
    foo.case3()

Error:
2021-02-01 23:11:35.422101: I tensorflow/compiler/jit/xla_cpu_device.cc:41] Not creating XLA devices, tf_xla_enable_xla_devices not set
2021-02-01 23:11:35.422548: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2021-02-01 23:11:35.439501: I tensorflow/compiler/mlir/mlir_graph_optimization_pass.cc:116] None of the MLIR optimization passes are enabled (registered 2)
Traceback (most recent call last):
  File "/Users/emadboctor/Library/Application Support/JetBrains/PyCharm2020.3/scratches/scratch.py", line 29, in <module>
    foo.case3()
  File "/usr/local/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py", line 828, in __call__
    result = self._call(*args, **kwds)
  File "/usr/local/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py", line 871, in _call
    self._initialize(args, kwds, add_initializers_to=initializers)
  File "/usr/local/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py", line 725, in _initialize
    self._stateful_fn._get_concrete_function_internal_garbage_collected(  # pylint: disable=protected-access
  File "/usr/local/lib/python3.8/site-packages/tensorflow/python/eager/function.py", line 2969, in _get_concrete_function_internal_garbage_collected
    graph_function, _ = self._maybe_define_function(args, kwargs)
  File "/usr/local/lib/python3.8/site-packages/tensorflow/python/eager/function.py", line 3361, in _maybe_define_function
    graph_function = self._create_graph_function(args, kwargs)
  File "/usr/local/lib/python3.8/site-packages/tensorflow/python/eager/function.py", line 3196, in _create_graph_function
    func_graph_module.func_graph_from_py_func(
  File "/usr/local/lib/python3.8/site-packages/tensorflow/python/framework/func_graph.py", line 990, in func_graph_from_py_func
    func_outputs = python_func(*func_args, **func_kwargs)
  File "/usr/local/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py", line 634, in wrapped_fn
    out = weak_wrapped_fn().__wrapped__(*args, **kwds)
  File "/usr/local/lib/python3.8/site-packages/tensorflow/python/eager/function.py", line 3887, in bound_method_wrapper
    return wrapped_fn(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/tensorflow/python/framework/func_graph.py", line 977, in wrapper
    raise e.ag_error_metadata.to_exception(e)
TypeError: in user code:

    /Users/emadboctor/Library/Application Support/JetBrains/PyCharm2020.3/scratches/scratch.py:22 case3  *
        self.storage[0] = tf.numpy_function(self.get_value, inp=[], Tout=tf.float32)

    TypeError: __array__() takes 1 positional argument but 2 were given

Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

I have tried in colab with TF version 2.4, nightly version(2.5.0-dev20210201) and was able to reproduce the issue.Please, find the gist here.Thanks!

See also #46563 for a little more detail about what causes this error in general.

@nfelt Any particular suggestions for fixing the issue?

@emadboctorx Unfortunately not really. My best guess is that something in the wrapper logic around numpy_function is attempting to convert some sort of TF tensor object into a numpy ndarray to feed as an argument into your numpy_function and at that point is hitting the same conversion error that I hit in the other issue.  However, the actual original stacktrace seems to be obscured because the error message is being re-propagated up through some intermediate layers.

Was able to reproduce the issue in TF 2.6.0-dev20210528,please find the gist here..Thanks !

All Possible solution added [Solved] TypeError: method() takes 1 positional argument but 2 were given

Able to reproduce the error for a similar case as shown below:
from PIL import Image
img_data = np.random.random(size=(100, 100, 3))
img = tf.keras.preprocessing.image.array_to_img(img_data)
array = tf.keras.preprocessing.image.img_to_array(img)
TypeError                                 Traceback (most recent call last)
 in
2 img_data = np.random.random(size=(100, 100, 3))
3 img = tf.keras.preprocessing.image.array_to_img(img_data)
----> 4 array = tf.keras.preprocessing.image.img_to_array(img)
~\anaconda3\envs\tf-latest-gpu\lib\site-packages\tensorflow\python\keras\preprocessing\image.py in img_to_array(img, data_format, dtype)
226       dtype = backend.floatx()
227     kwargs['dtype'] = dtype
--> 228   return image.img_to_array(img, data_format=data_format, **kwargs)
229
230
~\anaconda3\envs\tf-latest-gpu\lib\site-packages\keras_preprocessing\image\utils.py in img_to_array(img, data_format, dtype)
307     # or (channel, height, width)
308     # but original PIL image has format (width, height, channel)
--> 309     x = np.asarray(img, dtype=dtype)
310     if len(x.shape) == 3:
311         if data_format == 'channels_first':
TypeError: array() takes 1 positional argument but 2 were given

I found that was caused by PIL lib.
I found the numpy array function description here
 An array, any object exposing the array interface, an object whose
            __array__ method returns an array, or any (nested) sequence.

so, it will invoke the __array__ of PIL object.
I solved this problems by downgrade PIL from 8.3.0 to 8.2.0.
Maybe this will help you.

I solved this problems by downgrade PIL from 8.3.0 to 8.2.0.

Thanks! I had suspected something like this, so I had done a pip install --upgrade --force-reinstall tensorflow already. Interestingly, the pillow package was unaffected by this change, as it is not even listed in the tensorflow dependency tree.
pip install "pillow<8.3" does indeed fix this for now, and the root issue is tracked in https://github.com/python-pillow/Pillow/issues/5571.

Just came by to say that we too were seeing issues with this and a downgrade of pillow to 8.2.0 has worked around the issue for now.

I found that was caused by PIL lib.
I found the numpy array function description here
 An array, any object exposing the array interface, an object whose
            __array__ method returns an array, or any (nested) sequence.

so, it will invoke the __array__ of PIL object.
I solved this problems by downgrade PIL from 8.3.0 to 8.2.0.
Maybe this will help you.

Thanks alooooot !!

Upgrade to 8.3.1 is work too.

[already solved] TypeError: array() takes 1 positional argument but 2 were given

I have the same problem, downgrading pillow to 8.2.0 doesn't solve my problem.
Ubuntu18.10
python3.7
tensorflow2.6

＠schissmantics, Hi, have you solved this problem now, I've tried your code on tensorflow 2.5 and python 3.7 and Pillow 8.2.0(8.3.1), not solving the problem and same error.

In some newer version of numpy, an extra parameter dtype may be passed to the the __array__ attribute of an object, but in previous implementation of Tensor and some other structures, __array__ is defined like
  def __array__(self):
    raise NotImplementedError(
        "Cannot convert a symbolic Tensor ({}) to a numpy array."
        " This error may indicate that you're trying to pass a Tensor to"
        " a NumPy call, which is not supported".format(self.name))

In that case, an error is thrown before the function is executed
#51595 and fc0f0e61ca9fe3ca3b9b58f51bcf00e0643ed9e3 can solve this

The issue is NOT related to Pillow. It is clear by the description and the notebook.
It seems to be related to this issue, though.
I thought this commit would solve the issue, but it seems not to be the case.

The changes mentioned by @collinzrj does fix the problem, but it raises another error:
NotImplementedError: Cannot convert a symbolic Tensor (PyFunc:0) to a numpy array.
This error may indicate that you're trying to pass a Tensor to a NumPy call, which is not supported.

Although, I think this one is indeed expected.

@caio-davi yes, it is expected


Scrape FIPS algorithm data #276

Added scraping of additional data(type, description, version, capabilities) for each FIPS algorithm
Edit by @adamjanovsky: This closes #276

Thanks for the update 👍.
There are still two questions from my old comment that are hanging. Could you answer them please?

Also, did you get the same algorithms with this approach as you did with the alternative approach that you proposed recently? If not, what is the source of this discrepancy?

By the way, how long does execution of this method take? What's the most time demanding part? The call to FIPSAlgorithmDataset.parse_alg_data_from_html(page) could be parallelized if needed.

Next time, please contribute by creating branch directly in crocs-muni/sec-certs repository. This way, the automated tests are run against your commits. I manually run the all relevant parts of the pipeline:

pre-commit: ❌  does not pass, you have to fix a MyPy error (use np.nan instead of np.NaN in some modules unrelated to your PR, likely due to breaking change in numpy).  and apply some Ruff formatting across files.
tests: ❌
docs: all ok. ✅

The tests failed in FIPS part with
ERROR tests/fips/test_fips_algorithm_dataset.py::test_alg_dset_lookup_dict - TypeError: Dict: {'alg_number': '5331', 'algorithm_type': 'AES', 'vendor': 'Juniper Networks, Inc.', 'implementatio...
ERROR tests/fips/test_fips_algorithm_dataset.py::test_to_pandas - TypeError: Dict: {'alg_number': '5331', 'algorithm_type': 'AES', 'vendor': 'Juniper Networks, Inc.', 'implementatio...

For more info on automated pipelines and how to setup you environment to comply with our standards, please see https://sec-certs.org/docs/contributing.html#quality-assurance.


Truncated Response number or date with open-mixtral-8x7b on "la plateforme"

Hello, Could you do something for the open-mixtral-8x7b  model to fix the truncation bug on "la plateforme".
Here they explain what they did to solve it on a vllm server (with spacing between <s> and [INST]).
On api where they are in completion I solved the bug by following the issue resolution.
https://github.com/vllm-project/vllm/issues/2464

we do see this issue


Check automatic GPU loading in GhostBSD and NomadBSD

Check automatic GPU loading in GhostBSD and NomadBSD. Ideally we can boot straight into Xorg without any manual interaction.

https://github.com/nomadbsd/NomadBSD/blob/master/config/etc/rc.d/initgfx
In the meantime, consider
/usr/sbin/sysrc -f /etc/rc.conf kld_list+="sysctlinfo cuse utouch i915kms" >/dev/null 2>/dev/null

but according to https://github.com/furybsd/furybsd-livecd/issues/151#issuecomment-706781170 this is not a clean future-proof solution.

Message from drm-fbsd12.0-kmod-4.16.g20200221:
--
The drm-fbsd12.0-kmod port can be enabled for amdgpu (for AMD GPUs starting
with the HD7000 series / Tahiti) or i915kms (for Intel APUs starting with
HD3000 / Sandy Bridge) through kld_list in /etc/rc.conf. radeonkms for older
AMD GPUs can be loaded and there are some positive reports if EFI boot is NOT
enabled (similar to amdgpu).
For amdgpu: kld_list="amdgpu"
For Intel: kld_list="/boot/modules/i915kms.ko"
For radeonkms: kld_list="/boot/modules/radeonkms.ko"
Please ensure that all users requiring graphics are members of the
"video" group.
Older generations are supported by the legacy kms modules (radeonkms /
i915kms) in base or by installing graphics/drm-legacy-kmod.

So we must detect the GPU and insert the matching one at runtime into overlays/uzip/furybsd-live-settings/files/usr/local/bin/furybsd-init-helper?

Switched to using initgfx from NomadBSD.


使用过程中会造成zookeeper连接数过多

I think the code in org.smartloli.kafka.eagle.common.util.KafkaZKPoolUtils have a bug about zookeeper connection, that make a lot connection to zookeeper, even we set the pool size  to limit .
assume the pool size is 5,

at the begin we try to init  5 connection;
The we use getZkClient(clusterAlias) to get one connect, and remove one connection from zkCliPool;
If we release a connection, then we add the connect back to zkCliPool;

But , when had used all 5 connection and not realse them,  The zkCliPool will be empty;
At this time if we try to getZkClient(cluterAlias) , it will recreate 5 conneciton;
for (int i = 0; i < zkCliPoolSize; i++) {
  zkc = KafkaZkClient.apply(clusterAliass.get(clusterAlias), JaasUtils.isZkSecurityEnabled(), ZK_SESSION_TIMEOUT_MS, ZK_CONNECTION_TIMEOUT_MS, Integer.MAX_VALUE, Time.SYSTEM, METRIC_GROUP_NAME, "SessionExpireListener");
 if (zkc != null) {
	 zkCliPool.add(zkc);
	}
}

If I had miss any thing , please let me know.


Fix bug in importance sampled posterior

What does this implement/fix? Explain your changes
When running importance sampling with sir, ImportanceSamplingPosterior._sir_sample used the wrong kwarg for oversampling_factor when calling the sir function, such that this function always ran with a default oversampling_factor of 32.
Does this close any currently open issues?
Fixes # (issue)
Any relevant code examples, logs, error output, etc?
...
Any other comments?
...
Checklist
Put an x in the boxes that apply. You can also fill these out after creating
the PR. If you're unsure about any of them, don't hesitate to ask. We're here to
help! This is simply a reminder of what we are going to look for before merging
your code.

[x] I have read and understood the contribution
guidelines
[x] I agree with re-licensing my contribution from AGPLv3 to Apache-2.0.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added tests that prove my fix is effective or that my feature works => see #1080
[x] I have reported how long the new tests run and potentially marked them
with pytest.mark.slow.
[x] New and existing unit tests pass locally with my changes
[x] I performed linting and formatting as described in the contribution
guidelines
[x] I rebased on main (or there are no conflicts with main)

This PR also contains the tutorial. Shall we close #1080 then and do both in this PR?
Yes, I have close #1080.


Add serialization support for SPDX v2.2

This could be a good first issue. v2.1 was only chosen because some of the tooling available to validate SPDX files didn't support v2.2.
If anyone wants to submit a PR for this I think these would be the basic steps required:

add additional ConvertOutputFormat and BomFormat enum values for SPDX 2.1 (the current SPDX enum values should track v2.2, although maybe this should be more explicit?)
update ConvertCommand.cs to handle both enum values
update SpdxTagSerializer to accept the enum value and adjust the line that adds the SPDXVersion tag
adjust the output of the PackageChecksum tag depending on version (the v2.2 supported ones are already there, just commented out)

Available in v0.4.0


feat!: combine LanceFragment.merge_columns and LanceDataset.add_columns in one implementation

These two methods were very similar but took different arguments.  This combines the implementation of the two methods.
In addition:

The python implementation of Updater was removed as:

It is no longer used (since merge_columns now uses Fragment::add_columns)
It was internal and could potentially be a maintenance burden in the future

The python method LanceFragment.add_columns was removed since it has been deprecated for six months and could be confusing.

In a few weeks it might be good to deprecate LanceFragment.merge_columns and add LanceFragment.add_columns back with a new signature that mirrors LanceDataset.add_columns.
BREAKING CHANGE: The rust method LanceDataset.add_columns now has an optional batch_size parameter.

Codecov Report
Attention: Patch coverage is 62.29508% with 23 lines in your changes missing coverage. Please review.

Project coverage is 78.15%. Comparing base (f9024ce) to head (7b5f836).

Files with missing lines
Patch %
Lines

rust/lance/src/dataset/fragment.rs
5.55%
17 Missing :warning:

rust/lance/src/dataset/schema_evolution.rs
87.17%
3 Missing and 2 partials :warning:

rust/lance/src/dataset/updater.rs
50.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3021      +/-   ##
==========================================
- Coverage   78.19%   78.15%   -0.05%
==========================================
  Files         239      239
  Lines       76782    76825      +43
  Branches    76782    76825      +43
==========================================
+ Hits        60043    60045       +2
- Misses      13669    13693      +24
- Partials     3070     3087      +17

Flag
Coverage Δ

unittests
78.15% <62.29%> (-0.05%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

In a few weeks it might be good to deprecate LanceFragment.merge_columns and add LanceFragment.add_columns back with a new signature that mirrors LanceDataset.add_columns.

Agreed. The naming inconsistency makes me sad :(


Build failure: ecryptfs

Steps To Reproduce
Steps to reproduce the behavior:

add ecryptfs in boot.supportedFilesystems:

{ config, lib, pkgs, ... }:
{
  environment.systemPackages = with pkgs; [
    # ecryptfs
    # ecryptfs-helper
  ];
  boot.supportedFilesystems = [ "ecryptfs" ];
  # security.pam.enableEcryptfs = true;
}

Build log
building Nix...
building the system configuration...
these 3 derivations will be built:
  /nix/store/10k8pcn4fw6v33ma9y60v85kgvwb4281-etc.drv
  /nix/store/426w9k1f1hdkpwnizpn9zlha520qhsfz-initrd-linux-6.3.1.drv
  /nix/store/wzfpa9pahj5767n3prxazw6c04dcdz37-nixos-system-nix-22.11.4091.a08e061a4ee.drv
building '/nix/store/10k8pcn4fw6v33ma9y60v85kgvwb4281-etc.drv'...
building '/nix/store/426w9k1f1hdkpwnizpn9zlha520qhsfz-initrd-linux-6.3.1.drv'...
/bin -> /nix/store/km7nxvfj7ni3p8mvy7snmr145hgvxic9-initrd-bin-env/bin
/etc/crypttab -> /nix/store/csnda85fwff717ypl2rrwiqi6gsx70sv-initrd-crypttab
/etc/fstab -> /nix/store/hqzixlv44rb9zx0195p5c8r7nhfnx894-initrd-fstab
/etc/initrd-release -> /nix/store/fj5lk12rclvvb39dnsxdsacxd185jvfh-initrd-release
/etc/kbd/keymaps -> /nix/store/32kzfalq9xwjycp5jbjprifp96rhrsbl-console-env/share/keymaps
/etc/modprobe.d/debian.conf -> /nix/store/b3l9qmvisdkv0bxd5c18xma0qm88gz2v-kmod-debian-aliases.conf-22-1.1
/etc/modprobe.d/nixos.conf -> /nix/store/7jii5dqc23lmc5jsjlvw052bhj0a3an2-etc-nixos.conf
/etc/modprobe.d/systemd.conf -> /nix/store/7kp81fdwds9jks1zd1p0dql3imssrihw-systemd-stage-1-251.15/lib/modprobe.d/systemd.conf
/etc/modprobe.d/ubuntu.conf -> /nix/store/0q4cja14zv440va2jcwgpyzb71v7mi18-initrd-kmod-blacklist-ubuntu
/etc/modules-load.d/nixos.conf -> /nix/store/mdhayw921si4dkywqhvz7wcglr3wy748-initrd-nixos.conf
/etc/os-release -> /nix/store/fj5lk12rclvvb39dnsxdsacxd185jvfh-initrd-release
/etc/passwd -> /nix/store/7d5hgpsqbamxilzmnk1jhqsmjs3iybz2-fake-nss/etc/passwd
/etc/shadow -> /nix/store/kc726ddgpb6cm815mbgs971snp0a3l07-initrd-shadow
/etc/sysctl.d/nixos.conf -> /nix/store/7zr5996hbvv3shkwgpdip96pqyq5v6bw-initrd-nixos.conf
/etc/systemd/network -> /nix/store/ry2nqp54q0yll7qw4f0l7wyxw18rg615-initrd-link-units
/etc/systemd/system -> /nix/store/w076xcn3igazwd6awh0z6lmf68bncd0w-initrd-units
/etc/systemd/system.conf -> /nix/store/i6nsdk69wbkk76r5zkmqlam6kjzigdp9-initrd-system.conf
/etc/udev/rules.d -> /nix/store/nk1x9b82pbz13zkpc7shld07fdzr28m7-initrd-udev-rules
/etc/vconsole.conf -> /nix/store/6168vfb9njr0bkdargljy5fslh5ndchh-vconsole.conf
/init -> /nix/store/7kp81fdwds9jks1zd1p0dql3imssrihw-systemd-stage-1-251.15/lib/systemd/systemd
/lib/firmware -> /nix/store/4agjkgkzm2ivl1312l5vn99piybrgmgg-linux-6.3.1-modules-shrunk/lib/firmware
/lib/modules -> /nix/store/4agjkgkzm2ivl1312l5vn99piybrgmgg-linux-6.3.1-modules-shrunk/lib/modules
/sbin -> /nix/store/km7nxvfj7ni3p8mvy7snmr145hgvxic9-initrd-bin-env/sbin
Warning: Couldn't satisfy dependency libc.so.6 for "/nix/store/vnwdak3n1w2jjil119j65k8mw1z23p84-glibc-2.35-224/lib/libnss_files.so.2"
Warning: Couldn't satisfy dependency ld-linux-x86-64.so.2 for "/nix/store/vnwdak3n1w2jjil119j65k8mw1z23p84-glibc-2.35-224/lib/libc.so.6"
Warning: Couldn't satisfy dependency libsystemd-shared-251.so for "/nix/store/7kp81fdwds9jks1zd1p0dql3imssrihw-systemd-stage-1-251.15/lib/systemd/libsystemd-core-251.so"
Warning: Couldn't satisfy dependency libc.so.6 for "/nix/store/vnwdak3n1w2jjil119j65k8mw1z23p84-glibc-2.35-224/lib/libm.so.6"
Warning: Couldn't satisfy dependency ld-linux-x86-64.so.2 for "/nix/store/vnwdak3n1w2jjil119j65k8mw1z23p84-glibc-2.35-224/lib/libm.so.6"
Warning: Couldn't satisfy dependency libc.so.6 for "/nix/store/vnwdak3n1w2jjil119j65k8mw1z23p84-glibc-2.35-224/lib/libpthread.so.0"
Warning: Couldn't satisfy dependency libc.so.6 for "/nix/store/vnwdak3n1w2jjil119j65k8mw1z23p84-glibc-2.35-224/lib/libdl.so.2"
Warning: Couldn't satisfy dependency libc.so.6 for "/nix/store/vnwdak3n1w2jjil119j65k8mw1z23p84-glibc-2.35-224/lib/libgcc_s.so.1"
Warning: Couldn't satisfy dependency libc.so.6 for "/nix/store/vnwdak3n1w2jjil119j65k8mw1z23p84-glibc-2.35-224/lib/librt.so.1"
Error: Os { code: 17, kind: AlreadyExists, message: "File exists" }
error: builder for '/nix/store/426w9k1f1hdkpwnizpn9zlha520qhsfz-initrd-linux-6.3.1.drv' failed with exit code 1
error: 1 dependencies of derivation '/nix/store/wzfpa9pahj5767n3prxazw6c04dcdz37-nixos-system-nix-22.11.4091.a08e061a4ee.drv' failed to build
~/.dotfiles/nixos

Additional context
Adding ecryptfs to only environment.systemPackages doesn't fail, but as soon as it is added in boot.supportedFilesystem or security.pam.enableEcryptfs is set, I see this cryptic "AlreadyExists" error.
Is this a problem with ecryptfs or something else, looks like initrd can't be built?
Notify maintainers
@obadz
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.3.1, NixOS, 22.11 (Raccoon), 22.11.4091.a08e061a4ee`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.1`
 - channels(root): `"home-manager-22.11.tar.gz, nixos-22.11"`
 - channels(kpe): `"home-manager-22.11.tar.gz, nixos-unstable"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

There were also other problems with ecryptfs like ecryptfs-mount-private not working because it cannot find a security wrapper file (which was not created; how should it have been created).
I ended up using this (not that I know what I'm doing), which allowed me mounting a private:
{ config, lib, pkgs, ... }:
{
  environment.systemPackages = with pkgs; [
    ecryptfs
    #ecryptfs-helper
  ];
  #security.pam.enableEcryptfs = true;
  #boot.supportedFilesystems = [ "ecryptfs" ];
  security.wrappers =
    let
      mkSetuidRoot = source:
        { setuid = true;
          owner = "root";
          group = "root";
          inherit source;
        };
    in
      {
        "mount.ecryptfs_private" = mkSetuidRoot "/run/current-system/sw/bin/mount.ecryptfs_private";
        "umount.ecryptfs_private" = mkSetuidRoot "/run/current-system/sw/bin/umount.ecryptfs_private";
      };
}

ecryptfs-helper also doesn't evaluate because python2 is marked as insecure (see e.g. #148779).
@obadz Could you port it to Python3?

For others having the same problems:
gocryptfs is packaged in nixpkgs and a modern alternative to ecryptfs/EncFS.

There were also other problems with ecryptfs like ecryptfs-mount-private not working because it cannot find a security wrapper file (which was not created; how should it have been created).

I assume you're seeing the following error when running ecryptfs-mount-private:
/nix/store/d8f1k74all48dc5qsdb8wm9da1z7r06c-ecryptfs-111/bin/.ecryptfs-mount-private-wrapped: line 69: /run/wrappers/bin/mount.ecryptfs_private: No such file or directory

where the path to mount.ecryptfs_private is actually:
/run/current-system/sw/bin/mount.ecryptfs_private

ecryptfs-mount-private-wrapped

@crimeminister Yes, but I fixed it that with PR https://github.com/NixOS/nixpkgs/pull/190944 which was merged last month (requires setting programs.ecryptfs.enable = true, added by that PR).

ecryptfs-mount-private-wrapped

@crimeminister Yes, but I fixed it that with PR #190944 which was merged last month (requires setting programs.ecryptfs.enable = true, added by that PR).

Thanks for the heads up, and for doing the work to get this fixed. I'll look for this change to land in the next stable iteration.


One more test and collect coverage from all .js in src folder

Codecov Report

Merging #17 into master will decrease coverage by 19.26%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master      #17       +/-   ##
===========================================
- Coverage   66.66%   47.39%   -19.27%
===========================================
  Files           6       11        +5
  Lines         120      173       +53
  Branches       20       29        +9
===========================================
+ Hits           80       82        +2
- Misses         35       78       +43
- Partials        5       13        +8

Impacted Files
Coverage Δ

src/assert.js
0% <0%> (ø)

src/rpc/transport.js
0% <0%> (ø)

src/vault/index.js
0% <0%> (ø)

src/vault/vault.js
0% <0%> (ø)

src/index.js
0% <0%> (ø)

src/convert.js
88.46% <0%> (+7.69%)
:arrow_up:

Continue to review full report at Codecov.

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


How to assign LAG uplink to host vmnic while adding hosts to DVS

How to assign LAG uplink to host vmnic while adding hosts to DVS

https://docs.vmware.com/en/VMware-vSphere/6.5/com.vmware.vsphere.networking.doc/GUID-C627D418-C0F0-4D55-9F7D-8F54764A989C.html

Is there any possibility to achieve this using pyvmomi or ansible modules programatically?

from vmware.lib.common import vcsautils
from pyVmomi import vim
import pdb
pdb.set_trace()
data={'hostname': 'workload-vcenter.nfv.com', 'username'<EMAIL_ADDRESS>'password': 'HP1nvent@123'}
rc = vcsautils.getVcConnection(vc_fqdn=data['hostname'],vc_user=data['username'],vc_password=data['password'])
si,context = rc[0],rc[1]
content = si.RetrieveContent()
host = vcsautils._getObj(content, [vim.HostSystem], "resc-esxi-2.nfv.com")
dvs_obj = vcsautils.getDVSwitch(si, 'infra_dvswitch1')
uplink = 'vmnic5'
dvs_host_configs = list()
#uplink_port_names = 'Uplink 1'
uplink_port_names = ['Uplink 0', 'Uplink 1', 'Uplink 2', 'Uplink 3']
#uplink_port_names = 'LAG-1'
dvs_config_spec = vim.DistributedVirtualSwitch.ConfigSpec()
dvs_config_spec.uplinkPortPolicy = vim.DistributedVirtualSwitch.NameArrayUplinkPortPolicy()
dvs_config_spec.uplinkPortPolicy.uplinkPortName = uplink_port_names
dvs_config_spec.maxPorts = 60000
dvs_host_config = vim.dvs.HostMember.ConfigSpec()
#dvs_host_config.operation = vim.ConfigSpecOperation.add
dvs_host_config.operation = vim.ConfigSpecOperation.edit
dvs_host_config.host = host
pnic_specs = list()
pnic_spec = vim.dvs.HostMember.PnicSpec()
pnic_spec.pnicDevice = uplink
pnic_spec.uplinkPortKey = '5'  ===========> my dvswitch has 4 uplinks, so the keys were like 0-3. All these uplinks were added to a lag of another 4 ports (4,5,6,7)
pnic_specs.append(pnic_spec)
dvs_host_config.backing = vim.dvs.HostMember.PnicBacking()
dvs_host_config.backing.pnicSpec = pnic_specs
dvs_host_configs.append(dvs_host_config)
dvs_config_spec.host = dvs_host_configs
dvs_host_spec = dvs_config_spec
dvs_host_spec.configVersion = dvs_obj.config.configVersion
task = dvs_obj.ReconfigureDvs_Task(dvs_host_spec)

I would like to know this, too. I've been trying to understand the API for hours but can't find out how how the uplink portgroup and LAGs are connected to each other. A DistributedVirtualSwitchHostMemberPnicSpec doesn't have a lacpGroup property or something. And I don't want to guess the correct uplinkPortKey, especially not when there are several LAGs.
Neither does DVPortgroupConfigInfo has any information about related LAGs, not does VMwareDvsLacpGroupConfig tell me what (uplink) portgroup it's defined on.
It's a complete mistery to me how to do this. Any help would be highly appreciated!

@udautkarsh
Not very pretty but I think you can find the correct uplinkPortKeys like this:
# Find LAG "lag1"
for lag in dv_switch.config.lacpGroupConfig:
    if lag.name == "lag1":
        break

# Find all uplink ports on the portgroup for host
portCriteria = vim.dvs.PortCriteria()
portCriteria.host = [host]
portCriteria.portgroupKey = uplink_portgroup.key
portCriteria.uplinkPort = True
ports = dv_switch.FetchDVPorts(portCriteria)

lag_uplinks = []

count = 0
while count < len(lag.uplinkName):
    for port in ports:
        if port.config.name == lag.uplinkName[count]:
            lag_uplinks.append(port.key)
    count += 1

You need to add the host to the switch first.

@mariolenz  yeah I assumed hosts are added to dvs. I did not find any way to assign server nics to lag uplinks while adding host itself. Thanks I will improve my code.

I did not find any way to assign server nics to lag uplinks while adding host itself.

Neither did I. I'm quite sure it's just not possible. Maybe through the REST API but not through the SOAP API pyVmomi uses.

Thanks I will improve my code.

If you find find a better way, please let me know! Somehow, my code feels a bit wrong to me (if you know what I mean) but at least it seems to work :-)
I think I resent the count a bit, but wanted to make it work for ansible-collections/community.vmware#903 even if it's a bit ugly. Maybe something like for uplinkName in lag.uplinkName would be more pythonic.


Updated some of the translations

I will do it more accurately whenever I can get the game working again on this pc.
Contains
Fixes some bad translations (my own mistake!)
How to test
You really cannot unless you have someone who speaks Dutch natively. There's still weird translations however. Even though they're not actually wrong they still seem weird in some situations as I experienced a couple of months after running the game again.

Can one of the admins please verify this patch?

@GooeyHub: add to whitelist
@TimVerhaegen: welcome back to the PR list :D

Hooray Jenkins reported success with all tests good!


Profile syscall_mask and policy index

I'm curious how your figured out the 2 other fields in the profile_t ? Is there a func in the sandbox kext that validates this?
Thanks!

Maybe instead of syscall_mask it's some kind of flags field?  for the NEW protobox profiles they are ALL set to 1 ? 🤷


Easier team setup

Currently, server requires a separate bucket for each team, this makes team setup harder as someone or something needs to setup this bucket separately. Even is this functionality is added to the server, then server would need to have full admin permissions to manage buckets, which could be a risk if there is anything else hosted on the same cloud account.
To make setup easier and more secure, I would propose to just use folders in the same bucket instead, for example:
turbo-cache-bucket
team_A
asset1
asset2
team_B
asset1
asset2
...
Of course team names and asset names should be still hashed.
By using this approach, turbo cache server just needs permissions to manage content in one bucket and it is very easy to implement adding a new team. For adding a new team, server would just validate token and if it's correct, it would just accept any team name - any valid token works for any team. Later on if needed tokens could be improved to be issues and validated per team.
What do you think?

I like this idea! Sounds like a good idea to limit the need for create bucket permissions. Currently, the idea is that the server will make the bucket if it does not exist. I can imagine that this gets problematic quickly.
Regarding the hashing, it's a bit of hack I am generating a simple hash of the team id/slug that's passed from the Turbo CLI to avoid any potential not allowed characters.


docs: adding unattended upgrades

Description
Please include a summary of the change. Include the motivation for the change.
Adding infos about unattended upgrades in apt install section.
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation Fix

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] I have tested my code extensively
[ ] I have selected the develop branch as the target branch

We closed this PR because Unattended Upgrades is more in the responsibility of the node owner to keep his whole system up to date, than the node software itself.


postfix and operands for subscract and decrement

Thanks @kohtenko, but the ++ and += operators in SwiftyUserDefaults have really confusing and potentially dangerous semantics. I don't like them anymore.
Check out http://radex.io/swift/nsuserdefaults/static/ and the next branch. Swift's -- and -= just work out of the box with the new approach.


Add PingContext support for go1.8+

As title.

Codecov Report

Merging #766 into master will decrease coverage by 0.06%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #766      +/-   ##
==========================================
- Coverage   51.38%   51.32%   -0.07%
==========================================
  Files          38       39       +1
  Lines        8547     8557      +10
==========================================
  Hits         4392     4392
- Misses       3670     3680      +10
  Partials      485      485

Impacted Files
Coverage Δ

context.go
0% <0%> (ø)

xorm.go
66.19% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


How to get the gene name embedding

Thanks for your excellent work! I am littile confused about the embedding module. The input embedding is the result of value embedding plus gene name embedding. Does that means the value embedding depends on the gene expression value, however the gene name embedding is data independent? If I were correct, how can I get the gene name embedding from the pre-trained model?

Yes, you are right. You can get the gene name embeddings here: https://github.com/biomap-research/scFoundation/blob/f622db7ecd4ea3e43f6de1edf1f510b98a986d55/model/pretrainmodels/mae_autobin.py#L112
But in our application, we recommend using the gene context embeddings, which means you need to feed some data into the model and get the decoder's output embeddings as gene embeddings.

Thanks for your kindness. But I am still wandering whether this gene name embedding can be used as some universal information since it is learnable during the pretraining. And I found the pos_emb here:
weights_path = './models.ckpt'
checkpoint = torch.load(weights_path, map_location=torch.device('cpu'))
checkpoint['gene']['state_dict']['model.pos_emb.weight']
But the shape of this matrix is [19267, 768]. And the gene name you provide here

only includes 19264 genes, how can I match them?

Hi,
The first 19264 embeddings correspond to the gene name embedding. And the order of genes is here： https://github.com/biomap-research/scFoundation/blob/main/model/OS_scRNA_gene_index.19264.tsv

Thank you so much


Scala 2.13 support

It would be great to have a cross-build for Scala 2.13. circe-validation is one of the last libs in my dependency tree not available for Scala 2.13

Hi @danbim,
thank your for the PR. I'll look into it shortly and publish a new release. I wasn't aware people were using this, especially as this is a somewhat experimental exploration. In the long run you might be better off with a fork though or by copying the tiny codebase into you project.

Version 0.2.0 with your changes is on its way to maven central. In the future it will be easier to cut a new release and I will get @scala-steward involved. Thank you for the PR!

Thanks Niklas for the quick merge :)
Btw... your new microsite certificate seems to be invalid.

Thanks it appears like GitLab is still obtaining the certificate. I'll wait a little longer 🙏


adding support for functions and variable interpolation

Closes #1716
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

I'm slightly concerned about inventing our own CSS syntax here. It seems rather brittle at the moment - e.g. what happens if a variable name conflicts with a CSS keyword? What do you think about doing something like <View width="calc(100px + var(--size-250))"> instead? We would still add the prefixes for you, but at least the syntax would be CSS compatible and more obvious what's a variable to replace. What do you think?

Also, is calc the only function that is actually supported? If so, then maybe we should make the regex more specific?

https://caniuse.com/css-math-functions would indicate that min/max/clamp are supported as well as calc
I'm on the fence about adding var(), it'd make it easier to replace and using other variables would end up looking like that. I guess it's odd just because the standalone version is already not decorated with var()
width="size-200" not width="var(--size-200)"
yet it'd be
width="calc(100% - var(--size-200))" not width="calc(100% - size-200)"

On making the regex more specific:
As @snowystinger pointed there's more than calc supported I used https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_Functions#math_functions and use eg min in my grid definitions and as you can nest functions useing a regex would not work anymore. Further more I think it is an advantage that we automatically support new functions once they are available.
On using var:
I personally find the version without var much more readable - the example shows a very simple function but it could get even more complex so leaving out the var() function pays off even more. I highly doubt that the there will ever be a keywould size-\d+ but naturally there's no 100% guarantee.
One problem I see with var() is that supporting something var(--size-200, var(--size-250)) is something I don't see how it'll work. It won't work with the plain syntax either but because we advertise the useage of var() people might be tempted to believe they could use it.
In the end size-200 is in reality is not a CSS-Variable but a macro that we are expanding so marking that macro as a var feels wrong to me. Going on with the idea that we are having a macro here are some proposals:

#size-200
<size-200>
$size-200
@size-200

Anyways it's your call and I'll implement whatever you guys decide is the best for the project and from your comments I see you lean towards the use of var() which is fine for me.

@devongovett @snowystinger any thoughts on my comment - I ask because I could invest some time into this at the end of the week

@devongovett @snowystinger how do want to proceed - otherwise we should close this as wontfix

Thanks. Going to merge as is. This seems to be the most pragmatic approach. Ideally we would try to type this as well (possibly using Template literal types), and add something about this to the docs (probably here https://react-spectrum.adobe.com/react-spectrum/styling.html#dimension-values). If you would like to work on that, please feel free to send followup PRs!

Thanks. Going to merge as is. This seems to be the most pragmatic approach. Ideally we would try to type this as well (possibly using Template literal types),

I looked at that - https://github.com/adobe/react-spectrum/issues/1716#issuecomment-815302680 and template literals are not required - the problem I ran into was that this does not work in .d.ts which is used to define DimensionValue -  need to see if one can use them im .d.ts but what's the minimum version you require - template literals require a fairly new version of typescript.
Let me see what I can find out and I'll open a ticket to add typing support.


fix:OS-SWP-SUG-01

Fixing audit review, issue OS-SWP-SUG-01 Checking if address are not equal

Issue: #124
Article: Missing Address Check in Pair Contract

This might work well.
Please check: https://github.com/soroswap/core/pull/146/commits/c34ef0a3fc7cf779fe3a8e80e6a4e63e94ef9932


Generate random nick

Generate a random nick when starting the web app so that users can differentiate themselves from other without having to use the /nick command

hi this is done; doesn't look like i can use the agile board but i submit a pull request

https://github.com/status-im/js-waku/pull/94

You can use GitHub keywords by writing "Resolves #81" in the PR's description.


Issues transmitting on NodeMCU Mini and Server Issues on NodeMCU V3

Hi all,
I initially tried to configure the NodeMCU mini for use with the IR blaster, however despite getting the board to receive signals, I couldn't get them to transmit out. The components were all as per the original design with identical resistor values. I even tried lowering the TX resistor value and bringing the LED it to within inches of the device being transmitted but with no luck.
Does anyone have any tips on getting the Mini board to work with this? Perhaps different pins etc.
After not managing to get the Mini working, I grabbed a full size V3 board hoping that it might make things easier. I found that the board would now default to an IP outside of the router e.g. <IP_ADDRESS> if I left the network details blank as I had before with the Mini board.
I therefore tried adding e.g. <IP_ADDRESS> and <IP_ADDRESS> in the AP webpage instead. Doing this resulted in the board accepting the IP assignment and config and showing as OK in the serial monitor. However, the board simply won't connect to my router.
I read that the ESP8266 board manager only worked up until 2.4.0 ? and as I was using the latest 3.0.1 I rolled back to 2.4.0 as suggested.
The issue I now have is that as soon as I rolled back to 2.4.0 it will no longer compile as it stops comiling with the error:
"LittleFS.h: No such file or directory "
As soon as I update the board back to 3.0.1 it compiles again.
Turning on DHCP seems to resolve the issue of the board being assigned an address outside of the router range, however even adding a visible wavelength LED onto the D2 pin doesn't produce a flash even when transmitting.
I am literally loosing my hair over getting this running and would be eternally grateful for any help.
Best wishes,
Jay :)

The requirement for 2.4.0 is no longer active and you do not need to roll back to this version as long as your other dependencies are up to date.
If you're struggling understanding dependencies I would encourage you to just use PlatformIO to handle installing the blueprint as that will handle ensuring all versions of all dependencies are up to date and appropriate
PlatformIO also has options to specify which board you're compiling for which will automatically adjust pins for certain nonstandard boards
The default IP is <IP_ADDRESS> which is a common local subnet used on home networks similar to 192.168.xxx.xxx but you need to configure your network to match whatever subnet you use at home.
What subnet does your router/local network use? The fact that DHCP works just means your manual networking configuration was probably incorrect and inconsistent with the rest of your local network; devices need to be on the same local subnet to be accessible to each other
What command are you sending? Post the full URL
What http and serial output do you get when you send the command?