Deduplicate lsif npm identifiers

The output of lsif-npm can contain vertices and edges with duplicated IDs. This PR ensures that a global integer ID generator is created to stop the current behavior of resetting the counter to MAX_SAFE_INTEGER on each construction (one for the import linker, one for the export linker, and one each time an int-id moniker is encountered after a uuid-id moniker).

 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.:x: efritz sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@dbaeumer @jumattos Could you take a look?

Any chance of looking at these?

@dbaeumer any chance you could look at this?


rum.AppMonitor - customEvents property is not considered during resource update

What happened?
After update to pulumi-aws "5.29.0" and add to definition of rum.AppMonitor -> customEvents -> status "ENABLED" pulumi don't see the diff while updating resource on AWS. It is not working either from DISABLED to ENABLED or ENABLED to DISABLED. Work only when we create a new resource but then again when I try to change this value it stay as it was created originally.
Expected Behavior
Pulumi responds to value changes for AppMonitorCustomEvents (customEvents)
Steps to reproduce

Create some RUM AppMonitor (with custom event enabled or disabled)
Try to change only custom event value

Output of pulumi about
Pulumi is running on Azure DevOps Agent
Task         : Pulumi Azure Pipelines Task
Description  : Azure Pipelines task extension for running Pulumi apps.
Version      : 1.1.2
Author       : Pulumi
Help         : Join us on Slack at https://slack.pulumi.com./
Downloading: https://get.pulumi.com/releases/sdk/pulumi-v3.53.1-linux-x64.tar.gz
Caching tool: pulumi 3.53.1 x64
/home/vsts/work/_temp/7f8bce3d-441b-4e45-bcb4-3c18fc11bd72/pulumi/pulumi version
v3.53.1
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hi @lukasz-giants, thanks for reporting this. Can you clarify whether this worked for you in a version of pulumi-aws before 5.29?

Hi @thomas11, this feature wasn't be available before 5.29. Support for custom event was introduce in 5.29 and https://github.com/hashicorp/terraform-provider-aws/blob/v4.51.0/CHANGELOG.md


Introduce be.ordina/microservices-dashboard-server

See http://ordina-jworks.github.io/microservices-dashboard/1.0.0/#aggregation-provided

Thanks for trying out our dashboard!
You can always ask for help or address issues on gitter.
For this PR I have 2 remarks:

Don't forget to add "support/dashboard" to the list of directories in your build.sh file.
You may want to add --remove-orphans as a parameter to the docker-compose up commands in your startup.sh file. This can save you a lot of resources when you need to run the script a couple of times.


Cli commands stuck after extension installation

I have installed the extension as per the instructions.
The extension seems to work fine.
but subsequent to the installation I am not able to run the
Magento Cli commands.
for example
I was not able to run the command
php bin/magento cache:clean
after that I have disabled the module
and run the commands, I was able to run the commands.
Then I re-enabled the modules. and tried to run the commands
php -d memory_limit=-1 bin/magento setup:upgrade\
but that also stuck. my server system is Ubuntu 16.04 with 16GB of Ram.
I checked all server benchmarks, all seems to be ok.
update. when I set the memory limit, the setup:upgrade' process never finishes, but when I say run the command allocating 10G, I get the error like below; Running schema recurring...PHP Fatal error:  Allowed memory size of<PHONE_NUMBER>0 bytes exhausted (tried to allocate 20480 bytes) in /web/vendor/magento/framework/App/State.php on line 152`

Hi @pkjoshi
Sorry, I was not able to reproduce the issue.
Try to update the extension and its dependency, let me know if the issue still reproducible.


Security improvement: don't reveal powered-by

The first thing recommended when setting up an Express instance is to secure it by removing its "X-Powered-By" header.
https://www.npmjs.com/package/helmet
https://blog.risingstack.com/node-js-security-checklist/
https://strongloop.com/strongblog/best-practices-for-express-in-production-part-one-security/
So let's save cycles by not setting this header in the first place.
This header is also totally useless. If some people are needing it somewhere for a very specific usecase they can add a little custom middleware that returns the "X-Powered-By" header again.

As this is a breaking change, this is targeted for 5.x.

Hi! Can you elaborate on what exactly the security risk is that we would be resolving with this change? As long as things like IIS, ASP.Net, PHP, and more are sending this header by default, I don't see any good reason for Express to stop doing it unless you can provide very concrete information (and not speculation) on how this header is a security risk.

After talking around, as long as PHP, ASP.Net and others are sending X-Powered-By by default, we will as well. If you are planning an industry-wide change to get this header removed from everything by default, please keep us in the loop :)

When a web server/application is to be secured, be it PHP, Apache, Nginx, (whatever the software) the first thing to do is removing the web server/framework/etc. fingerprints if possible, i.e. the Server and X-Powered-By headers. Agreed it's not the most important point, but it's efficient (evidences provided in the literature links below).
The previously cited links all agree on it:

https://www.npmjs.com/package/helmet
https://blog.risingstack.com/node-js-security-checklist/
https://strongloop.com/strongblog/best-practices-for-express-in-production-part-one-security/

Here is literature from SANS, one of the most trusted source for computer security training (ask your network):
common services that are particularly prone to banner grabbing are web servers
Here is literature from OWASP, which provides recommendations widely followed by the industry (ask your network): Web framework fingerprinting is an important subtask of the information gathering process. Knowing the type of framework can automatically give a great advantage if such a framework has already been tested by the penetration tester. It is not only the known vulnerabilities in unpatched versions but specific misconfigurations in the framework and known file structure that makes the fingerprinting process so important. etc. :

https://www.owasp.org/index.php/Fingerprint_Web_Application_Framework_(OTG-INFO-008)#HTTP_headers
https://www.owasp.org/index.php/Web-metadata

The "X-Powered-By" header is not listed as a useful HTTP header:
https://www.owasp.org/index.php/List_of_useful_HTTP_headers
Now why would Helmet gather more than 1500 stars?
Node.js and Express are about aesthetic, stripping the useless, removing the cruft and going fast! All what you are ceaselessly doing in all your contributions.
So now let's reverse the burden of proof: where are the proofs that using X-Powered-By by default is a good practice?

I would like to hear why Microsoft still ships it on by default with ASP.Net and why PHP ships it on by default, etc. If you can provide that information, perhaps you can sway us, but for now, we won't be changing this feature.
Helmet does WAY MORE than turn off this header. If you showed me a module that literally did nothing but disable this header and had 1500 stars, that's a different story.
Basically, get ASP.Net and PHP to commit to removing it and we will as well. Until then, you can either turn it off like you have to do with every platform or create a module and publish it to npm that has it off by default for people to use.

Of course Helmet does way more than just removing the X-Powered-By header. But developers and project managers deploying Helmet buy into this choice.
@evanhahn, @evilpacket, @gergelyke @hacksparrow, @crandmck, as authors of the aforementioned software and articles, would you care to convince @dougwilson to remove the X-Powered-By header in Express 5?

+1 for removal

Look, removing the header provides no security benefits. Removing the "frivolous" header only does that--removes a non-useful header. What is the point of the header you say? Well, it's to give credit to the hard-working people who have enabled you to write your code so fast, who you never even gave money to, as this is free, open source. This library is even MIT licensed, so please, feel free to fork it and provide your own copy that does not include the header if you wish.
A lot of people incorrectly assume that "if that header is not there, attackers won't know I'm running Express!". I was walked-though a commercial security product that does server identification by security engineering. This software includes specific detection for Express.js, even (being popular), and these are some of the rules it uses:

Are there response headers all in lower-case? The more there are, the more points assigned as likely being Node.js server, which in turn counts towards Express.js.
Some requests to random URLs are made looking for a 404. If the response body is in the format "Cannot GET {url}" then it gives a massive number of points towards it being Express.js. This is actually the main give away it uses to know you are an Express.js server.
What does the ETag header look like if there is one? There are definitions for the different versions of the ETag header format for Express.js, so this header matching certain formats not only gives points towards being Express.js, but even hints at the version of Express.js you are using, since the format has changed over time (even the X-Powered-By header doesn't provide that level of detail, since it does not contain a version).

Even then, when using the server vulnerability attacking part of that software, it will simply try Express.js attacks against a server, regardless of if it even thinks the server is Express.js. This is because, since everything is configurable, there is no good way to truly know.
The X-Powered-By and Server headers are frivolous headers, I agree. Does removing them help with security? No, only increases obscurity, which is not a valid form of security by any means.
Just think of it this way: we give you Express.js for free and in return "trick" you into giving us credit for our hard work or you have to put 1 line of code in your application, which is already code and your developers should understand how to code when using Express.js.
There is no further discussion to be had on this topic.


Model re-compilation updated under Distributed Mirror Strategy issue

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 (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution : Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): 2.4.0
Python version: 3.6
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source):N/A
CUDA/cuDNN version: 11.0
GPU model and memory: RTX 2080 Ti x4

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
I am faced to an issue with Tensorflow with a model trained on multiple GPUs (x4) with the Distributed Mirror Strategy.
The first training works perfectly with the Distributed Mirror Strategy.
After, I want to update some parameters (in the same run) such as learning rate, optimizer. The model compilation works perfectly and when the new training sequence is starting, I got the error below.
Here is my code. Do you have any advise to avoid this error and the training stop ? When I reload the model and run the next step of the training, it works but not in the pipeline. Do you have any idea ?
mirrored_strategy = tf.distribute.MirroredStrategy()
with mirrored_strategy.scope():
    # Everything that creates variables should be under the strategy scope.
    # In general this is only model construction & `compile()`.

    print("[INFO] compiling model...")
    baseModel, headModel, model = get_model(LastLayerNeurons=20, DropOut=0.5)

    for layer in baseModel.layers:
        layer.trainable = False

    # construct the set of metrics
    metrics = ['accuracy']
    opt = SGD(lr=1e-3)
    model.compile(loss="categorical_crossentropy", optimizer=opt, metrics=metrics)

print("[INFO] training head...")
model.fit(train_dataset,
          steps_per_epoch=steps_per_epoch,
          validation_data=val_dataset,
          validation_steps=validation_steps,
          epochs=10,
          max_queue_size=500,
          callbacks=callbacks,
          verbose=1,
          use_multiprocessing=True,
          workers=20)

with mirrored_strategy.scope():
    print("[INFO] re-compiling model...")

    for layer in model.layers[:]:  # Dernier block i.e #5
        layer.trainable = True

    # construct the set of metrics
    metrics = ['accuracy']

    opt = SGD(lr=1e-4)
    model.compile(loss="categorical_crossentropy", optimizer=opt, metrics=metrics)

print("[INFO] fine-tuning V2 model...")
model.fit(train_dataset,
        steps_per_epoch=steps_per_epoch,
        validation_data=val_dataset,
        validation_steps=validation_steps,
        epochs=100,
        max_queue_size=500,
        callbacks=callbacks,
        verbose=1,
        use_multiprocessing=True,
        workers=20)

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.
Exception ignored in: <bound method Image.__del__ of <tkinter.PhotoImage object at 0x7f1bc3ea32b0>>
Traceback (most recent call last):
  File "/usr/lib/python3.6/tkinter/__init__.py", line 3507, in __del__
    self.tk.call('image', 'delete', self.name)
RuntimeError: main thread is not in main loop

@EdouardDKP
Could you please share the colab gist with all the dependencies as to analyse the issue reported here.Thanks

Hello,
I have tested to add the code below into my callbacks files. Thanks to that, that works.
No more issue at this time.
import matplotlib
matplotlib.use('Agg')

Best Regards

@EdouardDKP
Thank you for your update, glad its working fine for you, kindly move this issue to closed status as it is resolved.


how can i do it with existing tokens?

these scripts create 2 new tokens and create a farm for these tokens, is it possible to create a farm with already existing tokens?

Sure! You can add any existing tokens but please take not that these tokens must to be ERC20 tokens.
Here are two steps to use any existing tokens in this yield farming application:

Put the existing tokens in contracts directory.
Use them in TokenFarm.sol smart contract (example usage).

The tokens are already deployed, I don’t want to deploy it again, how can I do?


Add a "cookbook" section to the documentation

Some initial recipes to add would be how to use a filerefcacher to allow creating a file in cache without passing it in memory/using a static path. Also throw in an example for how to do model checkpoint resuming from within a stage using record.get_dir etc.

Add a recipe on outputting jupyter notebooks as artifacts

Add a recipe on using click and externally running a command via torchrun through cf

Add a recipe on multiple stages (one of which is optional via args) both create a model and the later one continues off of it if available


How to get destination infos with onStop?

During page transition, I am able to intercept with onStop. I can see the current router state. However, I am unable to see the next page that I am transitioning to. Any advice on how to get the destination information?
onStop: ->
  console.log this
  # "this" has only current router state. "this" does not have destination infos

i found it. in iron-location
so will use this code Iron.Location.get()


Fixed grammatical error in the section description

Describe this PR

Hey there, this is super quick PR that fixes a grammatical error I found in the description of the Getting Started tutorial.

Changes

Fixed a grammatical error in the Getting Started tutorial.

What issue does this fix?

N/A

Any other relevant information
@all-contributors please add @IM-Deane for doc

Thank you @IM-Deane; it takes a village... to have great docs with correct grammar. We appreciate your catching that!


Open in-app browser

It will be nice if there is a way to give custom actions to links instead of always opening in Safari. For our use, we want to open SFSafariViewController but MarkdownUI does not have that ability!
I implemented it through custom urls, but it is an ugly hack and I shouldn't have to do that.

Duplicate of #52

Hi @shahla00,
This has been already discussed in #52. I will reopen #52 and continue the discussion there.
Thanks!


very needed features for web publishing

The code for very needed features for web publishing #32

Thanks @Ahimta for all the work you've done here.
However I won't merge it anyway. There are features I don't want here and the code diverge too much (plus next major release will change a lot).
I encourage you to keep your fork if you still need all of this.

Thanks @MattKetmo for taking the time to review this.
I worked on this ages ago, since then I've worked and studied more software development, math and even geometry.
Now that I have more experience, especially in Javascript and DOM API, I can see that this PR needs much work in addition to porting it to the upstream Darkroomjs.
Anyways, I don't have much time these days, so I will leave things as is for now and those who really want these features can either port them themselves or post a bounty on Bountysource to make it worth my time.


⚠ Bofrost.es has degraded performance

In c4b8a73, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1540 ms

Resolved: Bofrost.es performance has improved in 3dca1b7 after 8 minutes.


"Generate deployment manifest" missing from task documentation

I was just reading Azure IoT Edge docs on deployment and came across an Azure IoT Task action == Generate deployment manifest that I was not aware of.
When checking the Azure Devops IoT Edge task page it is missing documentation for Generate deployment manifest action.
Please update ADO docs with the action type for the Azure IoT task to include this missing information.

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

ID: d93a0bac-faf1-3c5d-d358-487d3d0d4f9e
Version Independent ID: 06d36475-d60d-067d-0c12-d5af0ff1d85c
Content: Azure IoT Edge task - Azure Pipelines
Content Source: docs/pipelines/tasks/build/azure-iot-edge.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @vijayma
Microsoft Alias: vijayma

Thanks for letting us know. This issue is fixed in our new task reference, which is generated using the task source code and includes all task inputs, including any aliases here:
https://learn.microsoft.com/en-us/azure/devops/pipelines/tasks/reference/azure-iot-edge-v2?view=azure-pipelines
Please check it out and let us know what you think. Thank you.


Update 010-Transmon_floating_teeth.ipynb

Connection pads had a problem with the sizes, that is it.

What are the issues this pull addresses (issue numbers / links)?
Did you add tests to cover your changes (yes/no)?
Yes
Did you update the documentation accordingly (yes/no)?
Yes
Did you read the CONTRIBUTING document (yes/no)?
Yes
Summary
Details and comments

@fgnyilmaz Is this PR a replacement for PR #687 or an extension of it?

Is this PR a replacement for PR's #687 and #688 or a extensions of them?

Please replace #687 with #688. #688 is the latest one, some parameters were not correct. I am sorry if I caused any trouble.

@fgnyilmaz : For my understanding, we are:

closing #687 since #688 replaces it
leaving #688 open
leaving #689 open
Is this correct?

@fgnyilmaz : For my understanding, we are:

closing Update 010-Transmon_floating_teeth.ipynb #687 since Update transmon_pocket_teeth.py #688 replaces it
leaving Update transmon_pocket_teeth.py #688 open
leaving Update 010-Transmon_floating_teeth.ipynb #689 open
Is this correct?
Yes, please.


module global crashes VS 2017 15.8.1

In case you write down "module global" it crashes VS 2017 15.8.1
Repro steps

Create a new .Net Core Console App
Add a new item: File1.fs
Replace in "File1" by "global"
Hit Enter
Wait a couple of seconds... Visual Studio should crash

Expected behavior
I now "module global" is not valid, my goal was to define "namespace global", because I forgot changing module by namespace it crashed VS.
Expected behavior: to signal the error, something like "module cannot be global"
Actual behavior
VS crashes
Known workarounds
Be sure to replace "module" by "namespace" before naming it "global"
Related information
Operating system: Windows 10 Pro (10.0.17134)
Host (useful for support):
Version: 2.1.2
Commit: 811c3ce6c0
.NET Core SDKs installed:
2.1.4 [C:\Program Files\dotnet\sdk]
2.1.104 [C:\Program Files\dotnet\sdk]
2.1.200 [C:\Program Files\dotnet\sdk]
2.1.201 [C:\Program Files\dotnet\sdk]
2.1.202 [C:\Program Files\dotnet\sdk]
2.1.300-rc1-008673 [C:\Program Files\dotnet\sdk]
2.1.400 [C:\Program Files\dotnet\sdk]
Severity: Low

This also reproduces with a non-SDK project.

No ActivityLog.xml file dropped on my machine either.

Has been fixed. Will be in dev15.9.


The Web Application title

It would be nice if the Web Application titles would also get names like Site Collections, Webs, Lists and List Items

This feature should come to SPDG upon merging this https://github.com/Acceleratio/SPDG/pull/10


daemon not running; starting now at tcp: * daemon started successfully ERROR: Could not find any ADB device ERROR: Server connection failed

[x] I have read the FAQ.
[x] I have searched in existing issues.

scrcpy 2.4 https://github.com/Genymobile/scrcpy

daemon not running; starting now at tcp:5037
daemon started successfully
ERROR: Could not find any ADB device
ERROR: Server connection failed
Press Enter to continue...

https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#device-not-detected


Use botkube install, fix sync:cli, remove certs flag from MM

Description
Changes proposed in this pull request:

Use botkube install,
Update install instructions,
Fix sync:cli,
Remove certs flag from MM

Preview URL: https://switch-to-cli.botkube-docs.pages.dev/next/

Thanks @josefkarasek  for the review!
Here is a commit with fixes https://github.com/kubeshop/botkube-docs/pull/241/commits/72842e0783e89eda83facde46be9e6ef2b0988ff


Don't throw error if tx is not decoded

If you cannot decode a tx just return the same tree and an empty vector of nullifier / spent notes

I'm not sure about this, panicking results in an error in JS side, which is what we really want if a tx fails to decode, we don't want to silently fail in this case

fail = we don't own the corresponding key, which will happen most of the time that function is called

Closing as I can't reproduce in my machine, feel free to reopen if you reencounter this

Yep agree I opened because I had this issue before but it must not be related to that


Internal: Fixes Query Ownership

Pull Request Template
Description
Please include a summary of the change and which issue is fixed. Include samples if adding new API, and include relevant motivation and context. List any dependencies that are required for this change.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)
[] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[] This change requires a documentation update

Closing issues
To automatically close an issue: closes #IssueNumber

@kirankumarkolli Could you review these changes?

I did share feedback offline.
The ownership is not required for development. It's recommended to have only lead's in ownership.


Litmus does not consider appKind when filtering the target pod for an experiment

What happened:

I have two pods running with the same label. One is an instance of a statefulset and another one is an instance of a deployment.
I wanted to target the pod that is associated with a statefulset. So, I have added appLabel="common label for both pods(no unique label)", appNs="namespace", appKind="statefulset" to the chaosengine
During the PreChaos check, I see Litmus is checking the status of both the pods (statefulset and deployment) when I expected it to check only the pod associated with statefulset. It is because, Litmus does not worry about the appKind of the pods when it tries to fetch the pods. it only considers labels for filtering.

Code also says the same thing :
func FilterNonChaosPods(clients clients.ClientSets, chaosDetails *types.ChaosDetails) (core_v1.PodList, error) {
	podList, err := clients.KubeClient.CoreV1().Pods(chaosDetails.AppDetail.Namespace).List(v1.ListOptions{LabelSelector: chaosDetails.AppDetail.Label})
	if err != nil {
		return core_v1.PodList{}, err
	} else if len(podList.Items) == 0 {
		return core_v1.PodList{}, errors.Wrapf(err, "Failed to find the pod with matching labels in %v namespace", chaosDetails.AppDetail.Namespace)
	}

Then, Litmus tries to find a random pod among these two pods as I have not chosen any specific pod to target in the env variables. In this process it found the pod which is associated with the deployment. And, the Chaos injection failed as the litmus then finds that this pod is not of kind statefulset.

What you expected to happen:
I expect litmus to filter pods based on both labels and the appkind. So, it can filterout exact pods the user intended to create chaos on.
Where can this issue be corrected? (optional)
/pkg/utils/common/pods.go#L191
func FilterNonChaosPods(clients clients.ClientSets, chaosDetails *types.ChaosDetails) (core_v1.PodList, error) {
	podList, err := clients.KubeClient.CoreV1().Pods(chaosDetails.AppDetail.Namespace).List(v1.ListOptions{LabelSelector: chaosDetails.AppDetail.Label})
	if err != nil {
		return core_v1.PodList{}, err
	} else if len(podList.Items) == 0 {
		return core_v1.PodList{}, errors.Wrapf(err, "Failed to find the pod with matching labels in %v namespace", chaosDetails.AppDetail.Namespace)
	}

How to reproduce it (as minimally and precisely as possible):
application which has pods from both deployment and satefulset has same labels would trigger this problem
Anything else we need to know?:

I believe this is an issus in litmus-go rather than this repo. I have created an issue (679) and PR (680) in that repo to address instead


Migrate from Elasticsearch TransportClient to RestClient

The Elasticsearch TransportClient is deprecated in version 7 and will be removed in version 8. Version 8 will be the next LTS version. To be able to support version 8 we will need to migrate to the RestClient.
Also consider upgrading to Elasticsearch 7, even though it's not an LTS version. Especially since Spring has moved from 6.x to 7 for their latests minor release of spring.

I assume switching (and upgrading) also requires some (works-with) CQs ...

Depends if the REST client is part of the current dependencies. From what I understand it's worth it to upgrade to Elasticsearch 7 at the same time to get the best performance from the REST client.

Upgrading to 7 would also remove the need to include the specific snakeyaml in the compliance pom (see #2192)

Briefly looked into this, there might be an incompatibility:
org.eclipse.rdf4j.sail.elasticsearch.ElasticsearchBulkUpdater public constructor takes a org.elasticsearch.client.Client argument, and the old TransportClient implements this interface, but the new RestHighLevelClient / RestClient doesn't


Release DataFusion 42.2.0

Is your feature request related to a problem or challenge?
We are still struggling downstream with new bugs that were uncovered by a new check added to 42.0.0 that were hit by downstream consumers such as delta-rs:

https://github.com/delta-io/delta-rs/pull/2886

We tried to release a fix for some of these items in

https://github.com/apache/datafusion/issues/12813

But it wasn't enough
Describe the solution you'd like
Consider releasing 42.2.0 patch release with a workaround to unblock upgrades downstream

[x]  https://github.com/apache/datafusion/issues/13065

Describe alternatives you've considered
We could also just wait for the next release

https://github.com/apache/datafusion/issues/12470

I am personally worried that the 43.0.0 release while awesome may be potentially disruptive. Thus it it would be nice to allow people to get 42 based things like Delta.rs percolated through the ecosystem
Additional context
No response

I am actively working on this

Release vote thread: https://lists.apache.org/thread/qhh4os0z1wkdjv9kw3kl1tf1z9loo07d (thank you @timsaucer  and @andygrove for checking it already!)

The release is approved and released: https://lists.apache.org/thread/sbj1ntymd8xv9t4ljb0tqq21j600b3gs


rewrite error

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I want to access http://123.com/aaa and redirect to http://123.com/bbb, but the URL still shows http://123.com/aaa
this is my ingress.yaml
What did you see instead?
If I visit http://123.com/aaa, it will show http://123.com/bbb. This is not what I want. Can I set it to implicit forwarding
What version of Traefik are you using?
version：traefik:1.7.26
What is your environment & configuration?
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  annotations:
    traefik.ingress.kubernetes.io/request-modifier: 'ReplacePathRegex: ^/aaa/bbb'
  labels:
    app: weeee
  name: num1
spec:
  rules:
  - host: 123.com
    http:
      paths:
      - backend:
          service:
            name: dafasd
            port:
              number: 80
        path: /bbb
        pathType: Prefix

If applicable, please paste the log output in DEBUG level
No response

Hi! I'm Træfiker :robot: the bot in charge of communication regulation.
Thanks for your interest in Traefik!
We dedicate the issue tracker to bug reports and feature requests only. My advanced AI has spotted that your issue might be a configuration problem or relates to something that doesn't look like a bug.
To confirm this, please join our Community Forum and reach out to us on the Traefik section.
In case I'm wrong (well, that would be embarrassing :sweat_smile:), my developers will re-open the issue and fix me!
In the meantime, you can double check Traefik's documentation.


(PUP-4643) Add evaluation control functions break(), next(), and return()

This adds the functions break(), next() and return() which control the order of evaluation:

break() breaks the innermost iteration as if end of input was reached
next() returns a value from a lambda, function, class body or define body
return() returns a value from a function, class body or define body

When next() or return() is used to break() a class or define body, the value is ignored.

CLA signed by all contributors.

Found only nitpicks. Merged anyway.


Add qr decomposition for Tensor

Summary
This pull request addresses issue "Implement QR factorization" (resolve #2) by adding the qr function for Tensors, based on the full QR decomposition provided by the LinearAlgebra library. The implementation includes the necessary error handling and dimension reshaping to work seamlessly with Tensors. In addition, tests have been added to ensure the accuracy and correctness of the new function.
Example
julia> using Tensors
julia> using OMEinsum
julia> using LinearAlgebra
julia> using Test

julia> data = rand(2, 4, 6, 8)
2×4×6×8 Array{Float64, 4}:
...

julia> tensor = Tensor(data, (:i, :j, :k, :l))
2×4×6×8 Tensor{Float64, 4, Array{Float64, 4}}:
...

julia> Q, R = qr(tensor, labels(tensor)[1:2])
...

julia> tensor_recovered = ein"ijk,klm->ijlm"(Q, R)
2×4×6×8 Array{Float64, 4}:
...

julia> @test tensor_recovered ≈ tensor
Test Passed

Note that the tests fail now because we use the function introduced in https://github.com/bsc-quantic/Tensors.jl/pull/20 that is still not in master. With this function, the tests work as expected.


Remove jQuery

WIP: some tests are broken but appears to be functional

Many thanks for the PR! Apologies that reorder has become a blocker. Would you like to try cherry-picking the changes in #160 into this PR to test your hypothesis?

@jgwhite I'll take a look as soon as possible!

I'm unable to work on this for the foreseeable future! If anyone wants to pick up the remaining crumbs and get the build passing, feel free to fork the branch !

This project is now being maintained by the team at @heroku.
To get the project healthy and maintainable again, we’re doing a bit of spring cleaning of older issues and pull requests. This pull request hasn’t been touched in a while so we’re closing it. Thank you so much for all your hard work.
Please feel free to re-open if you’d like to resume work on it.

Hi - we're still very reliant on this addon and are currently using my forked version on our production app. I will wait until Heroku's team have done some work on the project before re-issuing a PR; just incase significant refactoring is done.

@omairvaiyani thanks for your patience. We’re going to focus on updating dependencies and project health, which should clear the way for your work 👍


TypeError: cannot assign 'list' object to buffer 'CatBN_output_buffer' (torch Tensor or None required)

Hi There, I am trying to run your code (only the training part), and I got the following error. Can you help me fix this asap.
Files already downloaded and verified
<type 'list'>
Traceback (most recent call last):
File "train.py", line 262, in
fire.Fire(train)
File "/home/gchennupati/.local/lib/python2.7/site-packages/fire/core.py", line 127, in Fire
component_trace = _Fire(component, args, context, name)
File "/home/gchennupati/.local/lib/python2.7/site-packages/fire/core.py", line 366, in _Fire
component, remaining_args)
File "/home/gchennupati/.local/lib/python2.7/site-packages/fire/core.py", line 542, in _CallCallable
result = fn(*varargs, **kwargs)
File "train.py", line 200, in train
num_classes=100
File "/home/gchennupati/Work/Cancer/temperature_scaling/models/densenet_efficient_multi_gpu.py", line 215, in init
drop_rate=drop_rate, storage=storage)
File "/home/gchennupati/Work/Cancer/temperature_scaling/models/densenet_efficient_multi_gpu.py", line 136, in init
self.register_buffer('CatBN_output_buffer', self.storage)
File "/home/gchennupati/.local/lib/python2.7/site-packages/torch/nn/modules/module.py", line 113, in register_buffer
.format(torch.typename(tensor), name))
TypeError: cannot assign 'list' object to buffer 'CatBN_output_buffer' (torch Tensor or None required)

I think it's an issue with the densenet model. Try running it with a different model.

See #7


Slow down while swinging

A user requested a way to set the mod in a way that would make the character move slower while swinging.
https://www.nexusmods.com/stardewvalley/mods/10941?tab=posts#comment-105241323

Implemented in 1.1.0.


Formula not working properly

I try to play around the formula toolbar in quill with the setting
toolbar: [     [{ 'header': '1'}, {'header': '2'}, { 'font': [] }],     [{size: []}],     ['bold', 'italic', 'underline', 'strike', 'blockquote'],     [{'list': 'ordered'}, {'list': 'bullet'},       {'indent': '-1'}, {'indent': '+1'}],     ['link', 'image', 'video'],     ['clean'],     ['formula']   ],
The icon shows on the toolbar, but when you enter something like "e=mc^2", it doesn't show on the result.

same problem

You also need to enable the module

Try this:
npm install katex
Add the following lines where you use react-quill.
import katex from 'katex';
import 'katex/dist/katex.min.css';
window.katex = katex;
Make sure, formula is present in both the toolbar array and formats array.
Hope it helps.


fix inconsistent use of tabs and spaces in indentation

Hi ucasfl, thanks for the PR and sorry for the slow response. This change was incorporated into our branch-2.1 release.


Getting error in React Native<EMAIL_ADDRESS>crypto could not be found within the project or in these directories:

Package Name: @azure/app-configuration
Package Version: 1.3.1
Operating system: MAC
[ ] nodejs

version: 16.14.0

Describe the bug
Getting error: crypto could not be found within the project or in these directories in react native project
To Reproduce
Steps to reproduce the behavior:

Installed the @azure/app-configuration package
Try running the app

Expected behavior
Should not get this error
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.

@IshaTyagi thank you for reporting this issue! Currently our SDK doesn't support React-Native and we are tracking the work in #20806


Beeg Url structure has changed

Hello
I noticed today that the url in site have changed. The old url format still works but you cannot get old links anymore on the website
exemple of the same video
https://beeg.com/6526409
is now
https://beeg.com/1941093077?t=911-1391
assuming that the ts is some expiration token.
As a result the plugin is not working anymore
youtube-dl -vv -f 720p https://beeg.com/1264283379?t=1279-1879
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-vv', u'-f', u'720p', u'https://beeg.com/1264283379?t=1279-1879']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.06.21
[debug] Python version 2.7.6 (CPython) - Linux-3.13.0-170-generic-x86_64-with-Ubuntu-14.04-trusty
[debug] exe versions: ffmpeg N-76159-gda43e9e, ffprobe N-76159-gda43e9e, rtmpdump 2.4
[debug] Proxy map: {}
[Beeg]<PHONE_NUMBER>: Downloading webpage
[Beeg]<PHONE_NUMBER>: Downloading JSON metadata
WARNING: Unable to download JSON metadata: HTTP Error 404: Not Found
[Beeg]<PHONE_NUMBER>: Downloading JSON metadata
ERROR: Unable to download JSON metadata: <urlopen error [Errno -2] Name or service not known> (caused by URLError(gaierror(-2, 'Name or service not known'),))
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 627, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2227, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib/python2.7/urllib2.py", line 404, in open
response = self._open(req, data)
File "/usr/lib/python2.7/urllib2.py", line 422, in _open
'_open', req)
File "/usr/lib/python2.7/urllib2.py", line 382, in _call_chain
result = func(*args)
File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 1140, in https_open
req, **kwargs)
File "/usr/lib/python2.7/urllib2.py", line 1184, in do_open
raise URLError(err)

I m back with this issue.
the problem is they seem to mix randomly urls with token and others without.
the script extract well urls with token, but not the one without token
https://beeg.com/1277207756
https://beeg.com/1670801077?t=91-811
youtube-dl -vv -f 720p https://beeg.com/1277207756
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-vv', u'-f', u'720p', u'https://beeg.com/1277207756']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.06.27
[debug] Python version 2.7.6 (CPython) - Linux-3.13.0-170-generic-x86_64-with-Ubuntu-14.04-trusty
[debug] exe versions: ffmpeg N-76159-gda43e9e, ffprobe N-76159-gda43e9e, rtmpdump 2.4
[debug] Proxy map: {}
[Beeg]<PHONE_NUMBER>: Downloading webpage
[Beeg]<PHONE_NUMBER>: Downloading JSON metadata
WARNING: Unable to download JSON metadata: HTTP Error 404: Not Found
[Beeg]<PHONE_NUMBER>: Downloading JSON metadata
ERROR: Unable to download JSON metadata: <urlopen error [Errno -2] Name or service not known> (caused by URLError(gaierror(-2, 'Name or service not known'),))
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 627, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2227, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib/python2.7/urllib2.py", line 404, in open
response = self._open(req, data)
File "/usr/lib/python2.7/urllib2.py", line 422, in _open
'_open', req)
File "/usr/lib/python2.7/urllib2.py", line 382, in _call_chain
result = func(*args)
File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 1140, in https_open
req, **kwargs)
File "/usr/lib/python2.7/urllib2.py", line 1184, in do_open
raise URLError(err)


9X production build fails after edits to content/talk/_index.md and content/home/talks.md

I am very frustrated at the build fails of my site after having edited the talks.md and _index.md sections of the site.
First, I really have no idea how to operate in the talks section lol ... so that may speak for itself.
Second, editing content/talk/_index.md, I believe, blew everything up. I reverted these back to their original state by pasting the original code in the section and turning "talks.md" to false, still failing to build.
Third, after these build fails, I can no longer make simple edits to the site. For example, I attempted to change my middle initial on the site from "P." to "Paul" and it failed to build. I am flummoxed.
Here is the first failed deploy:
10:05:27 PM: Build ready to start
10:05:34 PM: build-image version: 6b66e79d54a8f76b01392124644c3052ac0d32ef
10:05:34 PM: build-image tag: v3.3.6
10:05:34 PM: buildbot version: 2ba6f2db8150441c8469e6ae3ab3561a2c9d03fa
10:05:34 PM: Fetching cached dependencies
10:05:34 PM: Starting to download cache of 125.7MB
10:05:35 PM: Finished downloading cache in 1.142516427s
10:05:35 PM: Starting to extract cache
10:05:39 PM: Finished extracting cache in 4.037738205s
10:05:39 PM: Finished fetching cache in 5.342224863s
10:05:39 PM: Starting to prepare the repo for build
10:05:40 PM: Preparing Git Reference refs/heads/master
10:05:41 PM: Found Netlify configuration file netlify.toml in site root
10:05:41 PM: Found Netlify configuration file(s). Overriding site configuration
10:05:41 PM: Starting build script
10:05:41 PM: Installing dependencies
10:05:42 PM: Started restoring cached node version
10:05:45 PM: Finished restoring cached node version
10:05:45 PM: v10.19.0 is already installed.
10:05:46 PM: Now using node v10.19.0 (npm v6.13.4)
10:05:47 PM: Attempting ruby version 2.6.2, read from environment
10:05:48 PM: Using ruby version 2.6.2
10:05:48 PM: Using PHP version 5.6
10:05:48 PM: Installing Hugo 0.66.0
10:05:49 PM: Hugo Static Site Generator v0.66.0-78C3C78F/extended linux/amd64 BuildDate: 2020-03-03T15:28:32Z
10:05:49 PM: Started restoring cached go cache
10:05:49 PM: Finished restoring cached go cache
10:05:49 PM: unset GOOS;
10:05:49 PM: unset GOARCH;
10:05:49 PM: export GOROOT='/opt/buildhome/.gimme/versions/go1.12.linux.amd64';
10:05:49 PM: export PATH="/opt/buildhome/.gimme/versions/go1.12.linux.amd64/bin:${PATH}";
10:05:49 PM: go version >&2;
10:05:49 PM: export GIMME_ENV='/opt/buildhome/.gimme/env/go1.12.linux.amd64.env';
10:05:49 PM: go version go1.12 linux/amd64
10:05:49 PM: Installing missing commands
10:05:49 PM: Verify run directory
10:05:49 PM: Executing user command: hugo --gc --minify -b $URL
10:05:49 PM: Building sites …
10:05:49 PM: ERROR 2020/03/10 02:05:49 render of "section" failed: execute of template failed: template: section/talk.html:4:3: executing "section/talk.html" at <partial "site_head" .>: error calling partial: execute of template failed: template: partials/site_head.html:145:45: executing "partials/site_head.html" at <partial "functions/get_icon" 32>: error calling partial: "/opt/build/repo/themes/academic/layouts/partials/functions/get_icon.html:6:25": execute of template failed: template: partials/functions/get_icon.html:6:25: executing "partials/functions/get_icon.html" at <$icon.Fill>: error calling Fill: image "/opt/build/repo/assets/images/icon.png": fill : image: unknown format
10:05:49 PM: ERROR 2020/03/10 02:05:49 render of "taxonomyTerm" failed: execute of template failed: template: _default/list.html:4:3: executing "_default/list.html" at <partial "site_head" .>: error calling partial: execute of template failed: template: partials/site_head.html:145:45: executing "partials/site_head.html" at <partial "functions/get_icon" 32>: error calling partial: "/opt/build/repo/themes/academic/layouts/partials/functions/get_icon.html:6:25": execute of template failed: template: partials/functions/get_icon.html:6:25: executing "partials/functions/get_icon.html" at <$icon.Fill>: error calling Fill: image "/opt/build/repo/assets/images/icon.png": fill : image: unknown format
10:05:49 PM: ERROR 2020/03/10 02:05:49 render of "taxonomyTerm" failed: execute of template failed: template: authors/terms.html:4:3: executing "authors/terms.html" at <partial "site_head" .>: error calling partial: execute of template failed: template: partials/site_head.html:145:45: executing "partials/site_head.html" at <partial "functions/get_icon" 32>: error calling partial: "/opt/build/repo/themes/academic/layouts/partials/functions/get_icon.html:6:25": execute of template failed: template: partials/functions/get_icon.html:6:25: executing "partials/functions/get_icon.html" at <$icon.Fill>: error calling Fill: image "/opt/build/repo/assets/images/icon.png": fill : image: unknown format
10:05:49 PM: ERROR 2020/03/10 02:05:49 render of "home" failed: execute of template failed: template: index.html:4:3: executing "index.html" at <partial "site_head" .>: error calling partial: execute of template failed: template: partials/site_head.html:145:45: executing "partials/site_head.html" at <partial "functions/get_icon" 32>: error calling partial: "/opt/build/repo/themes/academic/layouts/partials/functions/get_icon.html:6:25": execute of template failed: template: partials/functions/get_icon.html:6:25: executing "partials/functions/get_icon.html" at <$icon.Fill>: error calling Fill: image "/opt/build/repo/assets/images/icon.png": fill : image: unknown format
10:05:49 PM: Total in 449 ms
10:05:49 PM: Error: Error building site: failed to render pages: render of "taxonomyTerm" failed: execute of template failed: template: _default/list.html:4:3: executing "_default/list.html" at <partial "site_head" .>: error calling partial: execute of template failed: template: partials/site_head.html:145:45: executing "partials/site_head.html" at <partial "functions/get_icon" 32>: error calling partial: "/opt/build/repo/themes/academic/layouts/partials/functions/get_icon.html:6:25": execute of template failed: template: partials/functions/get_icon.html:6:25: executing "partials/functions/get_icon.html" at <$icon.Fill>: error calling Fill: image "/opt/build/repo/assets/images/icon.png": fill : image: unknown format
10:05:49 PM: Skipping functions preparation step: no functions directory set
10:05:49 PM: Caching artifacts
10:05:49 PM: Started saving pip cache
10:05:49 PM: Finished saving pip cache
10:05:49 PM: Started saving emacs cask dependencies
10:05:50 PM: Finished saving emacs cask dependencies
10:05:50 PM: Started saving maven dependencies
10:05:50 PM: Finished saving maven dependencies
10:05:50 PM: Started saving boot dependencies
10:05:50 PM: Finished saving boot dependencies
10:05:50 PM: Started saving go dependencies
10:05:50 PM: Finished saving go dependencies
10:05:50 PM: Error running command: Build script returned non-zero exit code: 255
10:05:50 PM: Failing build: Failed to build site
10:05:50 PM: failed during stage 'building site': Build script returned non-zero exit code: 255
10:05:50` PM: Finished processing build request in 16.082947914s
Please help, I do not want to have to start from scratch with the site.
Thank you,
Tom

The error appears to indicate that Hugo cannot understand the image provided in the assets/images/icon.png location. Adding the image (if it's missing - you may need to create the folders if they don't already exist in your site) or fixing the format of the icon image may help.

Works! Thanks, George.
*Thomas P. DePatie | *PhD Student
Applied Organizational Psychology
Hofstra University
Hempstead, NY 11549
<EMAIL_ADDRESS>LinkedIn: https://www.linkedin.com/in/thomasdepatie
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=icon
Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=link
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
On Tue, Mar 10, 2020 at 9:14 AM George<EMAIL_ADDRESS>wrote:

The error appears to indicate that Hugo cannot understand the image
provided in the assets/images/icon.png location. Adding the image (if
it's missing - you may need to create the folders if they don't already
exist in your site) or fixing the format of the icon image may help.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gcushen/hugo-academic/issues/1590?email_source=notifications&email_token=AL5GZABYRK3VARWBT7UINILRGY4MLA5CNFSM4LEV5IWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOLLGEQ#issuecomment-597078802,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL5GZAAROYR53E27YJBPBY3RGY4MLANCNFSM4LEV5IWA
.


Implement basic support for (list) comprehensions.

It appears one of the main issues blocking support for comprehensions is the need for closures to capture variables used in the expression inside the comprehension.
I suspect a large class of applications doesn't require these variables to alias the outer scope, i.e. the variables could be truly local, simplifying the required closure logic substantially.
This suggests that this would be a good use-case to aim for as an incremental step when adding support for comprehensions to Numba.

This is a duplicate of #504, but you are right, and I'm pretty sure this is one of the most desired features by Numba users in general.

Added in https://github.com/numba/numba/pull/2369


Impl Clone for FFI wrappers.

I've published version 0.0.21 of lldb-sys which will be needed for this to compile.
Would you mind updating the version required as part of this commit so that it is all correct?
Also, if you have a moment and don't mind, could you move the impl Clone so that it is before the impl Debug / impl fmt::Debug so that they stay in roughly alphabetical order?  If not, no big deal as that's pretty boring and tedious.

Thanks for the feedback! The details you mentioned are pretty helpful, and I'll fix them soon.
The fix (also the previous work in lldb-sys) is not that tedious — thanks to shell scripting and editor macros.

What about this version?

Looks great to me! Thanks!


Add part-of property to fossils for easy grouping of related fossils

I needed to be able to group non-standalone fossils together, e.g. left_ptera_wing, ptera_body and right_ptera_wing.

Hi, nice change, I'll see to include it on the API over the weekend.


Cannot install atom package on Debian GNU/Linux where apmd is installed

Hello,
I just downloaded atom-amd64.deb from atom.io web site and tried to install it on a Debian "stretch" machine with command "sudo dpkg -i atom-amd64.deb". The installation failed because the package tried to install executable /usr/bin/apm while a different package already installed a different file with the same name; in fact, the file has been installed by "apmd" package.
The list of files in apmd package is here: https://packages.debian.org/stretch/amd64/apmd/filelist
So, I had to remove package apmd before installing atom package.
I understand that apmd was a quite old package and that it was installed on this machine a few Debian versions ago, but I still think you should list it in the "conflicts" list of the package "control" file.
Bye,
Giuseppe

Thanks for reaching out!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/3926. I'll bring up your suggestion to the developers.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Kurogane Kaikijong [PCSG00609]

App summary

App name: クロガネ回姫雀
App serial: PCSG00609
App version: 1.00

Vita3K summary

Version: v0.1
Build number: 1827
Commit hash: https://github.com/Vita3K/Vita3K/commit/0f2aabb900ddd27078df540bfe8f1d177ff0445d

Test environment summary

Tested by: Arnold Secret
OS: Windows 10
CPU: Intel Core i7-4500U
GPU: NVIDIA Geforce GT 740M
RAM: 8 GB

Issues
Goes in-game but it's stuck whilst preparing the mahjong pieces. Also save data loading loop when asked to save, requiring a clear of the savedata and selecting no and yes at the initial startup screen.
Screenshots

Log
vita3k.log

please change title to
Kurogane Kaikijong [PCSG00609]

as of https://github.com/Vita3K/Vita3K/commit/2101d1ea3faf8bb4c01fe3b6853724125169daf8, the game no longer freezes at the mahjong pieces. Didn't encounter any bugs with the saves. Gameplay is slow but playable if you have a powerful computer.
Screenshots

Log
vita3k.log
Recommended Labels

Playable


IPython 4.0.0 kernel spec name change

python -> python2

fixes #720

+1

I made another patch to solve this at 92008a3 which also gives the possibility to overwrite the list of kernel spec names to try via settings.py.


There should be "show all" button in "Compare tools" and "Latest Project" sections

@andrejcibik should we just use normal button, smaller one,

or you have another idea?

I think small one might be enough.
Will it expand the section and dynamicaly add new items or link somewhere else?

@andrejcibik

Will it expand the section and dynamicaly add new items or link somewhere else?

link elsewhere, to pages which are not ready yet:

https://steemprojects.com/grids in case of "Comparision Tools"

and

https://steemprojects.com/projects in case of "All projects"

currently, only redirection to https://steemprojects.com/projects/ is implemented:


is_a? accepts CONST only ?

Assigning a type to a variable breaks .is_a?:
"foo".is_a?(String)
# => true
"foo".is_a?(String || Array(String))
# => true
klass = String | Array(String)
"foo".is_a?(klass)
# expecting token 'CONST', not 'klass'

That's because is_a? is actually a pseudo method whose argument follows the type grammar and the type grammar simply works like that.
It's the same reason why Int8[4] in conventional Crystal code is an instance of Array(Int8) containing a 4_i8, while in the type grammar Int8[4] is another way of saying StaticArray(Int8, 4).
AFAIK the closest you can get to what you want to do is create an alias:
alias Klass = String | Array(String)
p "foo".is_a?(Klass) # => true

@Exilor thanks!


Extend Graph README.md

Extend Graph README.md. Use JSON README.md as reference.

I updated Graph README.md with a simple starting example:
https://github.com/redis/node-redis/pull/2239


Question about frame

How can i find my minus frame to using game memory addresses?

It's not stored anywhere in memory.
You have extract information about attackers and defenders animations.
How long is the animation is in frames and what's the current frame of animation that both characters are in.
After that you need to do some math to figure out how many frames quicker or slower the attacker will recover from the defender.
That's your +/- frames that Tekken Overlay calculates and displays.


New STL Files

A new done frame which is lighter and has a holder for an on/off switch.
Propeller guard STL.

I have merged your commit. Sorry for the late response, I really appreciate your contribution and will surely try your prop guards.

Hi,Great. Thanks for letting me know. I will set up a project for the joystick controller and send the details to you once ready.Best wishesPaulSent from my iPadOn 4 Sep 2023, at 17:10, Anish @.***> wrote:
I have merged your commit. Sorry for the late response, I really appreciate your contribution and will surely try your prop guards.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Hi Anish,
I have now published my joystick controller on Hackster.io.
https://www.hackster.io/paulsb/drone-joystick-controller-25e036
Have a look and, if you think others may be interested, can you add a link
to your documentation.
Best wishes
Paul
On Mon, 4 Sept 2023 at 17:04, Anish @.***> wrote:

Merged #3 https://github.com/anish-natekar/PicoW_Copter/pull/3 into
main.
—
Reply to this email directly, view it on GitHub
https://github.com/anish-natekar/PicoW_Copter/pull/3#event-10275615360,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNFFPPIUMEYIZQY4XSROJTXYX32BANCNFSM6AAAAAA4GLACKI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Great - many thanks.
Best wishes
Paul
On Wed, 6 Sept 2023 at 13:38, Anish @.***> wrote:

Sure, i'll make the changes to my docs and update it by this weekend.
This controller looks great I will try to build, and test it soon. I like
that you have added a charging port for the Lipo.
I might build a simpler one without a display. In the future I want to try
adding a FPV AIO camera to the drone that can send video to my phone so I
might try adding a phone holder in the stl file of the remote.
—
Reply to this email directly, view it on GitHub
https://github.com/anish-natekar/PicoW_Copter/pull/3#issuecomment-1708273119,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNFFPNXCF6MJB7PP3EPYWLXZBVEVANCNFSM6AAAAAA4GLACKI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi Paul,
I have updated the documentation - https://picow-copter-docs.readthedocs.io/en/latest/
I have mentioned your project under the Hardware section with a link to your article.
Best
Anish Natekar

Hi AnishMany thanks.Best wishesPaulSent from my iPadOn 9 Sep 2023, at 16:20, Anish @.***> wrote:
Hi Paul,
I have updated the documentation - https://picow-copter-docs.readthedocs.io/en/latest/
I have mentioned your project under the Hardware section with a link to your article.
Best
Anish Natekar
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Changing helper settings to convars

I changed the helper settings to be saved as convars as they are more convenient. I also spotted that I made a mistake on the last commit where I 'fixed' the settings being saved. But I was accidentally setting the frame width and height to nil here
I also fixed the div height not being set properly when the settings are loaded, I simply put it on a timer for 0.5 seconds.

Was wondering why the editor was doing that. Thanks for the fix.


Added a label in OpenvpnParser output

Ref #50

Coverage remained the same at 100.0% when pulling 53c1969ad36adafe2eb23e698f5eaad3eae9cabf on rohithasrk:openvpn-label into a128b19bf3503423ff1b4ca427426919594f955e on ninuxorg:master.


Hover lost border which makes it very hard to distinguish from normal text

VSCode Version: alpha
OS Version: Windows 10

See attached screen shot. I was hovering modesId. The hover looks like a normal text editor line.

@jrieken I actually not sure who is responsible. Starting with you :-)

no clue how made changes here...


Support use state query params to verify auth callback?

When redirect to github auth url, we can bring a state query param and when auth success github will add this state to the callback url query, we can use it to verify if this callback call is valid.
For example we can put a time string in state, and verify if it is outdated in callback

The OAuth2Strategy used as base from this GitHubStrategy already sets and the state search param.

Is it possible to custom it? I did not found the option

It’s not possible, per spec the state must be random, I generate one using a UUID, there’s no need to customize it and opening the option to do it would let apps use unsafe values.

It’s not possible, per spec the state must be random, I generate one using a UUID, there’s no need to customize it and opening the option to do it would let apps use unsafe values.

Not what I expected, if you generate a UUID, how can you verify if the state is correct on callback?

The state is stored in the session before sending the user to the authorization url, then the state the provider sent back is compared against the one in the session.


Remove emptyTupleIterator

https://github.com/openfga/openfga/blob/07794019d080bf629a6408d9917ea1492e1eaba6/pkg/storage/iterators.go#L36-L50
This code isn't referenced anywhere except in tests, let's remove it.

I'm happy to take this on as my first ticket. Would be nice to get familiar with how committing/merging PRs work.


Add http endpoint adapter

…zerServer
Overview

Actually IDK, I think this may be powerful. Do you want to hold it until we will check 1.22?

Actually IDK, I think this may be powerful. Do you want to hold it until we will check 1.22?

I would bump to 1.22 and try using the new mux, if you have time to do so


FIX: Fix generate_custom_images.py

I needed to make these two changes to get this script to work on my machine:

indentation issue
Give a default to 'family' to be '' and not None

PS: Please let me know if this is the right way to fix this, and how to do this. I'm very new to this.
My main motivation is that I need to create a spark cluster on google cloud that has python installed with specific libraries. If there is a better way I'd be open to hearing it thanks!

Thanks for catching this. Please sign the CLA so that we can merge your changes.

The fix for indentation issue was submitted with #64 .

What about the '--family' default? Can you please put that in? It doesn't have to be this pr.else explain why this is not correct. Thanks!

It's fixed too: https://github.com/GoogleCloudPlatform/cloud-dataproc/blob/34b3f028a301bdae9c5233b9e78cf39d0b47b59a/custom-images/generate_custom_image.py#L307


Sharing intents with both text and file types (iOS)

Ive been using this library to share files, however Ive run into a bit of an issue with the Adobe Sign and Fill app.
Ive spent a bunch of time digging into why this is and it comes down to the fact that the sign and fill app is sharing both text (a note about the file as kUTTypeText) and the file itself as kUTTypeFileURL. What is then happening is that this library is detecting the text as the second type and because its the last in the array of shares (eg if index == (content.attachments?.count)! - 1) it's only sharing the text specific parts. This means the file itself isn't sent through with the share.
Has been a bit of a mission to work all this out but I think the upshot is that redirectToApp should not be typed to text/file etc and instead on the other end (RecieveSharingIntent.swift) we should assume that any of those components may be present.
Hope this makes sense, I will try my hand at a fix shortly

+1 to this issue and @lukemcgregor's fix!


Will there be any new releases

I see that the last release was in April 2019. Please let me know if the project is still maintained, stable and will there be any future releases? Or is it deprecated?

you might want to check here: https://github.com/inexio/pysmi

You can read https://github.com/etingof/pysnmp/issues/429 to learn the latest.


create_user fails if HashidsField is set on the User model

I tried to add an hashid field to my custom user model and it fails when I try to create a new user.
My custom user model:
class User(AbstractUser):
    hashid = HashidsField(real_field_name='id')
    username = None
    email = models.EmailField(_('email address'), unique=True)

    USERNAME_FIELD = 'email'
    REQUIRED_FIELDS = []

    objects = CustomUserManager()

    def __str__(self):
        return self.email

The error:
======================================================================
ERROR: test_create_user (users.tests.UsersManagersTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "<...>/mysite/users/tests.py", line 9, in test_create_user
    user =<EMAIL_ADDRESS>password='foo')
  File "<...>/mysite/users/managers.py", line 18, in create_user
    user = self.model(email=email, **extra_fields)
  File "<...>/lib/python3.8/site-packages/django/db/models/base.py", line 485, in __init__
    _setattr(self, field.attname, val)
  File "<...>/lib/python3.8/site-packages/django_hashids/field.py", line 100, in __set__
    raise AttributeError("HashidsField is read only")
AttributeError: HashidsField is read only

I was able to solve the problem only by passing hashid=django.db.models.DEFERRED to create_user<EMAIL_ADDRESS>'password', hashid=django.db.models.DEFERRED). (Actually, I put the hashid=django.db.models.DEFERRED directly in my CustomUserManager)
So, is that the intended behaviour or maybe there's a better solution?

@ndrbrt thanks for the report.
I think the issue is that I was setting HashidsField.column to empty string instead of None. Should be fixed in #3


⚠ Bofrost.de has degraded performance

In 4d2b4ed, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1062 ms

Resolved: Bofrost.de performance has improved in 83749ce.


Add the Jupyter notebook about the football-data feed.

Not so much a PR for reviewing, but a brain dump from a spike.  You can review the Jupyter notebook here https://github.com/FootyStats/footy/blob/feature/football-data/football-data.ipynb .  Yes the notebook is H.U.G.E. but it is mainly data taken in from the feed.  Raw as it is, it gives us an insight on how deal with the data feed to automatically update the model going forward.

I've converted this PR to a draft for now.  More meaningful work can be done on this later on.

This branch is no longer required now we have a prototype that utilises the Football Data feed.


Support per-call unsuccessful response handlers

I considered doing exception handlers as well, and decided not to until/if-ever we needed it.
But maybe you're right and it's best to do it now. Give me a few minutes...

PTAL


Bug: modal transition duration is now defined via inline styles

It doesn't look like it can be anything other than 300ms at the moment, but better keep an eye on it.

Fixed by 0663aa424454c0cc54af222d8300c9f3e55a6ea3.


Issues with proteomics container

Things are getting a bit weird with the proteomics container. I am documenting my progress (or lack thereof here), and may be someone can chime in.
Currently, both release and devel fail. But I get release to build locally:

But devel fails

I believe that these fail should be fixed with the addition of libxt-dev in mscore, but...
On release, it is present, which is consistent with the positive result above

On devel, it is *not present, which is consistent with the negative result above

Not sure why ...

@lshep - could trigger a new push when you have a minute. Thanks!

@lgatto for the dockerhub repositories?  I can but I thought you figured out how to trigger builds this weekend?

I just triggered the release_mscore2 and will follow with the release_proteomics2 ... if successful I will move onto devel.

I thought so, but the Dockerfile doesn't reflect my latest push

It should be
# The suggested name for this image is: bioconductor/devel_mscore.

FROM bioconductor/devel_base2

MAINTAINER<EMAIL_ADDRESS>
RUN apt-get update -qq && \
    apt-get install -y  --no-install-recommends \
    libxt-dev libx11-dev libcairo2-dev \
    libfreetype6 \
    libexpat1-dev \
    libgmp3-dev \
    liblapack-dev \
    libopenbabel-dev \
    libgl1-mesa-dev \

I must be missing a step to actually make sure that the latest content of the out directories are used to build the container.

It should do that automatically.  The dockerhub just pulls from the github directory.  Maybe it was just too close to the commit.  Anyways... it does look like the corrected one is being/has built for mscore2

I don't think the correct one was build for mscore2, because devel misses libxt-dev, although it is present in the Docker file. mscore2 builds successfully, even when it missed that dependency.

I've only rebuilt the release_mscore2 I haven't build the devel yet.

Yes, the Docker files for release reflects the latest changes.

@lshep - what have you done when you say I've only rebuilt the release_mscore2?

I've manually triggered the build on the release_mscore2  at https://hub.docker.com/r/bioconductor/release_mscore2/~/settings/automated-builds/
The release_proteomics2 is queued up to build once the latest tag gets built for the release_mscore2 docker also by a manually trigger at https://hub.docker.com/r/bioconductor/release_proteomics2/~/settings/automated-builds/ .
Once the release_proteomics builds successfully I would kick off the devel_mscore2.  I wait to do this because the devel builds are all triggered by rebuilds of parent packages. now that the prometcore builds of mscore, prometcore and proteomics would try to build and if prometcore builds successfully then the metabolics will get build so several containers will be queued with the devel_mscore2 . if it needs more changes I would prefer not to have to cancel out all the queued builds (since they get built automatically) Only one build can happen at a time on the dockerhub, they do not happen simultaneously.

When you say manually triggered do you mean
The trigger buttons at the top

Or the Build triggers at the bottom

At the top.

There's still something wrong with mscore2. I first pull the latest containers
lg390@elyacin [04:28:07] [~/dev/00_github/bioc_docker] [master]
-> % docker pull bioconductor/devel_mscore2
Using default tag: latest
latest: Pulling from bioconductor/devel_mscore2
Digest: sha256:31bbdc9f2522a675d5faa7a8d52263c4d24b7eb76402b33632e76c3bfc75ba42
Status: Image is up to date for bioconductor/devel_mscore2:latest
lg390@elyacin [04:28:38] [~/dev/00_github/bioc_docker] [master]
-> % docker pull bioconductor/release_mscore2
Using default tag: latest
latest: Pulling from bioconductor/release_mscore2
Digest: sha256:0a6392a4a48e42117bbb9dbe838629c5ede6c5262f5e5471f0b07a295d1b0428
Status: Image is up to date for bioconductor/release_mscore2:latest

Is my dependency available in release_mscore2: YES
-> % docker run -ti bioconductor/release_mscore2 bash
root@fe425d4f255b:/# dpkg --get-selections | grep libxt
libxt-dev:amd64					install
libxt6:amd64					install
libxtst6:amd64					install

Is my dependency available in devel_mscore: NO
-> % docker run -ti bioconductor/devel_mscore2 bash
root@1263132bf283:/# dpkg --get-selections | grep libxt
libxt6:amd64					install
libxtst6:amd64					install

This is consistent with the docker files on the docker hub: the latest changes are reflected in release_mscore2, but not devel_mscore2, although there are correct in the out directories on github.

I only tried to build the release versions of the dockers NOT the devel versions.  There is still a problem with the release version of release_proteomics2 using the correct version of release_mscore2.  I didn't do devel for the reasons stated about with linked dependencies.

I only tried to build the release versions of the dockers NOT the devel versions.

Ok. it's still confusing that the docker file hasn't been updated.
I will focus on the release for debugging, then.

It has been updated for the release version because the release version got pulled from github when it builds it on dockerhub.  If you trigger a build on devel_mscore2 the dockerfile would be updated there; I haven't done this because of the linked containers.  The dockerfile has been updated for the release_mscore2.

The dockerfile only gets updated when it pulls from github.  It only pulls from github when a build is triggered.

Thanks, it makes sense now.

Coming back to what is driving me a bit insane these last day. Locally, I can fire up release_mscore2 and successfully run all the code in out/release_proteomics/install.R.
I can also successfully build release_proteomics2:
lg390@molerat:~/tmp/bioc_docker/out/release_proteomics$ docker build -t local/release_proteomics2 .
[ lots of output here]
The downloaded source packages are in
        ‘/tmp/RtmpxHGrEg/downloaded_packages’
>
> # just in case there were warnings, we want to see them
> # without having to scroll up:
> warnings()
NULL
>
> if (!is.null(warnings())) {
+     w <- capture.output(warnings())
+     if (length(grep("is not available|had non-zero exit status", w)))
+         quit("no", 1L)
+ }
>
> suppressWarnings(BiocInstaller::biocValid(fix=TRUE, ask=FALSE))
[1] TRUE
>
 ---> 8677822df88c
Removing intermediate container ad0e4b7d3ce0
Successfully built 8677822df88c

and
lg390@molerat:~/tmp/bioc_docker/out/release_proteomics$ docker images
REPOSITORY                     TAG                 IMAGE ID            CREATED              VIRTUAL SIZE
local/release_proteomics2      latest              8677822df88c        20 minutes ago       6.164 GB
bioconductor/release_mscore2   latest              4b48396ff366        6 hours ago          1.73 GB
bioconductor/devel_mscore2     latest              b77b86625ba2        7 days ago           3.715 GB
haskell                        latest              754074761f63        2 weeks ago          961.1 MB
ubuntu                         latest              f5bb94a8fac4        2 weeks ago          117.3 MB
ubuntu                         14.04               9df772af0cb9        4 weeks ago          188 MB
hdd                            latest              1392acd6f829        2.280377 years ago   1.675 GB
haskell                        7.8                 7093a9a3a5cc        2.371320 years ago   601.3 MB

What am I missing? Why does it fail remotely?

The remote build failed again, with the same message - it just says
Build canceled.

The container builds fine locally - is there a time out on docker hub?
I am running out of ideas; I am going to try with a different parent container.

@lshep @sneumann - any idea what this Build canceled means?

I couldn't find much, just

If the Docker client loses connection to the daemon, the build is canceled. This happens if you interrupt the Docker client with CTRL-c or if the Docker client is killed for any reason. If the build initiated a pull which is still running at the time the build is cancelled, the pull is cancelled as well.

Something to also keep in mind,  I want to say if there are issues with any package installations in R it will also cause the Docker to fail to build on dockerhub.

Yes, and that should be reflected with a message like
1: In install.packages(pkgs = doing, lib = lib, ...) : installation of package ‘ChemmineR’ had non-zero exit status
2: In install.packages(pkgs = doing, lib = lib, ...) : installation of package ‘bioassayR’ had non-zero exit status

Or do you mean something else?

What is the difference between the R3.4.0_Bioc3.5 and latest builds ? Recently it seems one of them failled, the other cancelled. Are these two separate builds, or one build and (if that were successful) one more tagged, which is cancelled if the first one failed ? Just guessing here.

What is the difference between the R3.4.0_Bioc3.5 and latest builds ? Recently it seems one of them failled, the other cancelled. Are these two separate builds, or one build and (if that were successful) one more tagged, which is cancelled if the first one failed ? Just guessing here.

When you manually trigger the R3.4.0_Bioc3.5 it automatically adds a second build for latest.  It runs the code again to get each Tag option for the container.  Since the dockerhub runs only one build at a time not simultaneously,  if I notice/am watching the builds and the R3.4.0_Bioc3.5  fails to build, I cancel out the second build of latest manually since it will also fail.

Following up on the issues with the proteomics container, @sneumann successfully built it locally (thank you for testing, Steffen). As the build does take some time (2+ hours), it is likely that the build canceled is the result of a time out, which is consistent with this post. (although a bit out dated).
I am going to use protmetcore as parent for proteomics, as that container has already some packages pre-installed. This failed last time I tried because protmetcore hadn't been re-build using mscore, which has the some required dependencies for to successfully install Cairo.
Unless anyone object, I am going to do the following to make sure all parents are up-to-date.
0. Update the config with new parent

trigger a release_protmetcore2 build
once 1 is successful, trigger a release_proteomics2 build

Bonus question:

Does anyone know how, or if at all possible to increase the timeout?

I could imagine that docker Inc. Has a plan that involves a monthly sum of money...

I blame Android for the brevity and typos

The images build successfully when I reduce the number of packages to be installed, suggesting that a timeout is indeed the problem.

This has been solved by creating a protcore container.


handle Default values

There are a few fields that require Default values, that are identified with the default value label in a new Default column. These value labels are always from a controlled term field, which is also specified for those rows.
To simplify your job, I've specified the default value identifier in parentheses after the default value label. (I think the default value object gets specified with both the label and the identifier, if it's a controlled term.)
(Note: Some fields with Default values are hidden, but others are not.)

Implemented for terms and also for text fields


Fix for #311 for empty (but not null) metadata fields

Fix for #311 where empty (but not null) metadata fields are not being hidden or showing emptyText.

@HitmanInWis i dropped in a PR to your branch; added some Unit tests, added a test for Collections and genericized the array check to be objects (and just caring if its empty or not - checking the array/collection values seems too generous :P)

where can I see this PR?  not finding it

@HitmanInWis sorry - i forgot to hit "Create PR" button ...
should be here now: https://github.com/HS2-SOLUTIONS/asset-share-commons/pull/8

Codeclimate seems to be stuck on this :(


Migration generates a varchar field with a length which is too long for Postgres

Hello!
I hope this is the right place.
I stumbled upon following behavior:
If we have a field string field with a long length like:
[StringLength(20000000)]
public string MyString { get; set; }

The generation of the migration succeeds, but when applying it to the database it fails with following output:
22023: length for type varchar cannot exceed 10485760
The generated SQL was:
"MyString" varchar(20000000) NULL
Now I understand that it might not the best idea to store very long texts in the database, but I still think this use case could be supported (or at least fail when generating the migration).
As I understand the SQL server generation uses nvarchar(max) for all strings that exceed a certain length, maybe something similar could be done here?
Anyway, thanks for this project!

You had better read a little bit about the PostgreSQL text types. There's no such thing as nvarchar(max) (that's SQL Server), but PostgreSQL recommends that you simply use the text type for variable-width strings, if you don't want to enforce a maximum. There's no performance penalty involved in using text.
Long story short, just remove the [StringLength] attribute and you should be fine.
As to whether it's a good idea to store long texts - that's a different question, and the answer is going to depend on your application.

Thanks for the fast answer! I read about the the text types. My question was more in a way: if the StringLength attribute specifies a length larger than the maximum could the generation automatically use the text type?

What purpose would that serve? Does specifying [StringLength(20000000)] actually mean that you want PostgreSQL to enforce length constraints for 20000000 (quite an odd requirement)? If this is what you want, then automatically generating text doesn't translate that requirement. It seems much more consistent to simply let PostgreSQL tell you that it does not support what you want, no?

@rogi Traditionaly EF mapping has interpreted this as "must be able to string of at least this length" which allows it to be more provider agnostic. This is why SQL Server switches to (max) if the length is longer than can fit in the sized string type. This also makes sense when the size comes from a data annotation attribute where it may be there more for validation at the API level than for database definition.

@ajcvickers thanks for the comment... But according to the documentation, StringLengthAttribute accepts a maximum length, and can optionally also accept a minimum length - which also corresponds well to how I understand SQL types - varchar(8) means a string of up to 8, and not a string that can save at least 8 (at least in PostgreSQL).
Are you saying EF has a different interpretation of this? I'm fine with aligning with EF conventions on this, but then how does one go about specifying a maximum length?

Depends what you mean by "maximum length". If you mean is there a way that you can be sure that the database created by Migrations will perform validation that nothing greater than a certain length is saved, then there isn't a way. On the other hand, if you mean can you specify that the database must be able to store a string as big as your application might have, then max length is used for that. It could certainly be argued that the API name could be different to express this better, but that ship sailed a long time ago.

@roji I'm not saying that SQL Server doesn't throw when attempting to insert a string that is too long for, for example, nvarchar(24), just that the EF configuration of max length is not intended to assert that that will happen. So, if my domain model has a string property that can be a maximum length of 24, then it's certainly okay for the database to do this, but it's not required that this happen.
Perhaps another way of looking at this is that the EF model configuration, as much as possible, should ensure that valid values can be stored in the database for that configuration. It doesn't really say anything very much about whether or not the database will enforce this. So for type mappings, the provider should choose the "best type" that matches the given configuration. For SQL Server, it's always best to use non-(max) types if possible because they generally perform better. So, if possible, SQL Server will choose nvarchar(x) rather than nvarchar(max) not because storing greater than nvarchar(x) will throw if the string is longer than x, but because if the string is never longer than x, then nvarchar(x) is a better mapping to use.
If, for example, text performed better than varchar(x) for Postgres, then it would be valid to choose this type regardless of the specified max length. On the other hand, if they are basically equal in functionality, then varchar(x) probably makes more sense because it retains the max length information, but from an EF perspective this doesn't really matter. Likewise, from an EF perspective, if a string with the max length specified can only be stored in a text column, then it is fine for Postgres to map this to a text column.

Oh, and SQLite I don't think has any concept of max length and never validates anything anyway.

OK, thanks for the explanation @ajcvickers, it's clearer now. The naming is indeed confusing/unfortunate.
I've just pushed a commit that maps string to text if length > 10485760, which is the maximum PostgreSQL allows. See this conversation for a rationale on setting an arbitrary upper limit (they don't want the max character length to depend on the encoding of the specific string).
The PostgreSQL docs explicitly state that there is no performance difference between varchar and text, so I guess it makes sense to map strings with length < 10485760 to varchar(x), and with length > 10485760 to text.
Does this mean it is actually recommended (for SQL Server) to specify something like [StringLength(20000000)], because that will map to varchar(20000000) rather than to varchar(max), and that would be better performance-wise? Isn't such a large size basically the same as saying "any size"?

@roji The max lengths for SQL Server are much smaller: 4000 for Unicode strings, 8000 for ANSI strings.

@ajcvickers oh wow, ok.

@roji I should probably open a separate issue, but related to StringLength(n) attribute, I'm not getting any ValidationErrors when I call SaveChanges on a string with length that exceeds n . Instead the string is truncated to length n without exceptions or warnings. I'm using EF Core 2.1 preview 1.


Injectable middleware

Something I have wanted to implement for a while is a sort of injectable middleware feature.
An idea is maybe to have a cli flag --middleware which accepts a path to a ts/js-file that would contain a list of middleware functions that would be executed in order before each request. So that users can inject their own logging, or whatever else they would like to enhance their experience
e.g: denoliver ./src --middleware=/path/to/middleware.js
and in the config file:
{
...
"middleware:" "/path/to/middleware.ts
...
}

Could also be something like beforeRequest and afterRequest or something similar

Started working on this

Merged with #30


Slow on Android?

In Example project, each "click" takes about 5 seconds to be processed (navigation from home to some option), and Overlay animations are very slow.
Tested on a low-end device (Moto G XT 1033) and in a mid-end device (Asus ZenFone 2 Selfie).

When React Native App is running on debug mode on the device, it may be very slow, which may be caused by the running environment, such as Remote Debug, etc. You can try to turn off Remote Debug or package for Release apk.


validation prevents parent menu creation ("edit mode")

This is  a bug.

Prerequisites

[x] Are you running the latest version?
[x] Are you reporting to the correct repository?
[x] Did you check the documentation?
[x] Did you perform a cursory search?

Description

Steps to Reproduce

Try creating a structure with only a (parent) menu

Expected behavior

link to not be required so that generation can happen; or to be ignored so validation can still happen
Actual behavior

link must be of type string & link must be empty

Note, it only happens if you enter the menu configuration for a 2nd time

Unable to reproduce

@aocneanu did you fill in a menu entry to then edit it? (the bug only appears in the edit mode, as described)

I did


Export recommender model to CoreML

I would love to be able to use the recommender model on my app without having to send information to a server to be processed. Would it be possible to support exporting the recommender model to CoreML?

Thanks for trying our Turi Create. Yes, unfortunately, the recommender isn't easily exportable to Core ML yet. Can you share with us what recommender you have trained?
There may be some options for you to get around this issue temporarily?

I have an app where users can create things by using items. I have a lot of data about which users have used which items and I want to be able to recommend new items to users based on the items they used previously.

We are looking into this. Stay tuned.

i am also looking forward...

@srikris @znation Hey guys, I'm using the item_similarity recommender. Would there be a CoreML workaround for this?
Cheers.

We're continuing to investigate a solution with a high priority. Thanks for your patience - stay tuned.

Hey @afranklin, there is something news about "Export recommender model to CoreML" ?

@tjkev It's being actively worked on; we'll update this thread when something is available to try. Thanks for your interest!

Great news: Recommender model export is available in the 5.0 beta. For more details, see https://github.com/apple/turicreate/releases/tag/5.0b1

That's amazing! Thanks @afranklin

Wonderful news!


[bug] Endpoint is reachable Validator "Zero-click" Vulnerability

Describe the bug
In the validator Endpoint is reachable there's a zero-click vulnerability.
When the validator checks if the endpoint is reachable, the endpoint/URL requested can be a malicious domain used to exfiltrate data through subdomains, URL paths, and query parameters.
Examples of exfiltrating data through calling malicious endpoints:

Query parameter example: https://www.malicious-domain.co.uk/path?firstName=John&lastName=Doe&pass=johndoe123&username=johndoe
Subdomain example: https://johndoe123.johndoe.malicious-domain.com/
Path example: https://malicious-domain.com/john/doe/johndoe123/johndoe
Of course, those can be combined.

To Reproduce
Steps to reproduce the behavior:

Attacker owns a (malicious) domain
Apply endpoint is reachable validator when configuring Guardrails
Add data for exfiltration to the endpoint (containing malicious domain) - done by the LLM
Request the malicious endpoint which contains the data - done by the validator function
Attacker has the data

Note: The request to check if the endpoint is reachable can both pass and fail, but nevertheless, if the request was made to the malicious domain (containing the data in any form), the attacker has successfully exfiltrated the data.
Expected behavior
I see 2 possible solutions:

remove Endpoint is reachable validator
check only if the host is reachable (e.g. for https://www.subdomain.malicious-domain.co.uk/path?firstName=John&lastName=Doe&pass=johndoe123&username=johndoe check https://www.malicious-domain.co.uk/) [Also in that case we might need to consider renaming to host_is_reachable]
have a white and black list of URLs when configuring the validator

I'm open to any other suggestions. I'll propose a PR with the implementation of the 2nd proposed solution.
Library version:
Version (e.g. 0.4.2)

Thanks for the PR! I’m a big fan of this change in theory but I do have a few follow ups

I think we lose a lot by losing the ability to hit specific paths
I’m fine with losing the ability to hit urls + query params
Is there a way to do this without fetching the entire endpoint? Like some tcp/http strat where we just do a reach out without requesting the full page? Like HEADing it?

I also love the suggestion of having lists of url regexes that are allowed/blocked

I think we lose a lot by losing the ability to hit specific paths
Agree! I like the feature, but unfortunately, it introduces a vulnerability/possible misuse, so I think we should agree on some compromise solution. But, of course, it's your call.
I’m fine with losing the ability to hit urls + query params
Noted 🙂
Is there a way to do this without fetching the entire endpoint? Like some tcp/http strat where we just do a reach out without requesting the full page? Like HEADing it?
After thinking this through, I think checking DNS records might be useful... This way we won't ping the target server, but we'll know if the URL can be resolved to an actual IP address.
I think we can use socket.getaddrinfo() for this purpose.

Please let me know what you think and if I can make the PR with said DNS records solution. Also ping me, if you have any additional questions.
I'll be more than happy to make the changes to the code 🙂

I like the socket.getaddrinfo() idea, but unfortunately that doesn't tell us whether an HTTP endpoint is open. I think it would be good to combine this approach with the HEAD request approach defined here - https://realpython.com/site-connectivity-checker-python/
I also want to add that we should put this functionality under a head_only param passed to the validator constructor. When that value is true (which it should be by default), the validator will only check HEAD and socket info. When it's false, it'll do the existing logic.
As far as the zero-touch vulnerability goes, we should absolutely document and mitigate it, but I think it's not possible to get rid of totally. So for this code path, I think we should do the following

add a warning saying "Full requests will be made to this site" and log the site name each time
include 2 params as you recommended allowed_domains and blocked_domains
ensure that there isn't a tertiary concern around remote script execution - if we pull down a site with scripts in it, we want to make sure those scripts do not execute. This could be as simple as never touching the body of the req and only checking for the http status response header

"I understand; I'll set aside the DNS idea then. 🙂
Concerning the head_only parameter, its inclusion in the validator's constructor puzzles me. I can't envision a scenario where fetching the body is necessary to ascertain if the endpoint_is_reachable. Thus, I believe we can always opt for a requests.head(endpoint) to obtain the HTTP header, simplifying our approach.
However, even when we limit our request to just the header, the vulnerability persists, and an attacker could access the data, just as before.
Regarding the list of domains, I think choosing just one will suffice. My recommendation is to use allowed_domains, but I'd like your confirmation. Would you prefer this as an initializer parameter or a function parameter?
As for the tertiary concern about remote script execution, I wouldn't worry too much. Python's HTTP requests library is designed to be resilient against such issues."

I understand; I'll set aside the DNS idea then. 🙂
Concerning the head_only parameter, its inclusion in the validator's constructor puzzles me. I can't envision a scenario where fetching the body is necessary to ascertain if the endpoint_is_reachable. Thus, I believe we can always opt for a requests.head(endpoint) to obtain the HTTP header, simplifying our approach.
However, even when we limit our request to just the header, the vulnerability persists, and an attacker could access the data, just as before.
Regarding the list of domains, I think choosing just one will suffice. My recommendation is to use allowed_domains, but I'd like your confirmation. Would you prefer this as an initializer parameter or a function parameter?
As for the tertiary concern about remote script execution, I wouldn't worry too much. Python's HTTP requests library is designed to be resilient against such issues.


Spelling

This PR corrects misspellings identified by the check-spelling action.
The misspellings have been reported at https://github.com/jsoref/codeql-action/commit/525c790ab88928c92ff49e731f998468d1ed0e13#commitcomment-47650569
The action reports that the changes in this PR would make it happy: https://github.com/jsoref/codeql-action/commit/42808dbc96d5279ddc197dd9812f29cda3605e6f
Note: this PR does not include the action. If you're interested in running a spell check on every PR and push, that can be offered separately.
Merge / deployment checklist

[ ] Confirm this change is backwards compatible with existing workflows.
[ ] Confirm the readme has been updated if necessary.

@aeisenberg: yeah, I've already made the PR to them; dropped here. Updated the map.
I also have a set on en_US changes which I imagined might be controversial, so I left them out of this PR.
https://github.com/jsoref/codeql-action/compare/spelling...en-us
This repository appears to have a mix of en_US/en_GB. I generally encourage repositories to use only one flavor of English... I can offer a PR to adopt either flavor.

Failing build checks are unrelated to this change and are fixed in main.

This repository appears to have a mix of en_US/en_GB. I generally encourage repositories to use only one flavor of English... I can offer a PR to adopt either flavor.

I don't think we have a policy on this. Feel free to open a separate PR and we can discuss.

Rebased and squashed.