wrong use of sequence_loss_by_example in ptb_word_lm.py?

Hi, I noticed "sequence_loss_by_example" with default of "average_across_timesteps":
def sequence_loss_by_example(logits, targets, weights,
average_across_timesteps=True,
softmax_loss_function=None, name=None):
but in pub_word_lm.py:
loss = tf.nn.seq2seq.sequence_loss_by_example(
[logits],
[tf.reshape(self._targets, [-1])],
[tf.ones([batch_size * num_steps])])
self._cost = cost = tf.reduce_sum(loss) / batch_size
weights sum of batch_size*num_steps was used in "sequence_loss_by_example" to normalize the score, so there is no need to divide loss by batch_size again, right? cost was used for gradient later.

@ebrevdo Eugene, can you take a look?

Lukasz knows this code better.

Hi!
I think the use of sequence_loss_by_example in ptb_word_lm in ok, because sequence_loss_by_example averages across time-steps (not batch), but ptb_word_lm uses it with just 1 time-step -- it first puts the time component into the batch dimenstion. That's why it has the reshape, but the lists are all of length 1 -- that's the time dimenstion and that's why you see "batch_size * num_steps" there in weights.
Hope that clarifies it!
Lukasz

When i run the following demo:
import tensorflow as tf
A = tf.constant([[0.1,0.2,0.3,0.4],[0.2,0.1,0.4,0.3],[0.4,0.3,0.2,0.1],[0.3,0.2,0.1,0.4],[0.1,0.4,0.3,0.2]], dtype=tf.float32)
B = tf.constant([1, 2, 1, 3, 3], dtype=tf.int32)
w_1 = tf.constant(value=[1,1,1,1,1], dtype=tf.float32)
w_2 = tf.constant(value=[1,2,3,4,5], dtype=tf.float32)
D = tf.contrib.legacy_seq2seq.sequence_loss_by_example([A], [B], [w_1])
D_1 = tf.contrib.legacy_seq2seq.sequence_loss_by_example([A], [B], [w_1], average_across_timesteps=False)
D_2 = tf.contrib.legacy_seq2seq.sequence_loss_by_example([A], [B], [w_2])
D_3 = tf.contrib.legacy_seq2seq.sequence_loss_by_example([A], [B], [w_2], average_across_timesteps=False)
with tf.Session() as sess:
print(sess.run(D))
print(sess.run(D_1))
print(sess.run(D_2))
print(sess.run(D_3))
import tensorflow as tf
A = tf.constant([[0.1,0.2,0.3,0.4],[0.2,0.1,0.4,0.3],[0.4,0.3,0.2,0.1],[0.3,0.2,0.1,0.4],[0.1,0.4,0.3,0.2]], dtype=tf.float32)
B = tf.constant([1, 2, 1, 3, 3], dtype=tf.int32)
w_1 = tf.constant(value=[1,1,1,1,1], dtype=tf.float32)
w_2 = tf.constant(value=[1,2,3,4,5], dtype=tf.float32)
D = tf.contrib.legacy_seq2seq.sequence_loss_by_example([A], [B], [w_1])
D_1 = tf.contrib.legacy_seq2seq.sequence_loss_by_example([A], [B], [w_1], average_across_timesteps=False)
with tf.Session() as sess:
print(sess.run(D))
print(sess.run(D_1))
The outputs are as following:
[1.4425355 1.2425355 1.3425356 1.2425356 1.4425356]
[1.4425355 1.2425355 1.3425356 1.2425356 1.4425356]
I found that whether the parm 'average_across_timesteps' is set as  'True' or 'False' the result is the same, why?


ScummExtract fails on windows

Describe the bug
When file is dragged to the window the event fails, throwing an exception.
To Reproduce
Steps to reproduce the behavior:

Drag file to window

Expected behavior
when you drag file to the window, it should open the file and show the result.
Screenshots
No
Desktop (please complete the following information):

OS: Windows 10 x64 Bits

Additional context
none.

to fix this problem I have done the following in file ngf/System/Event.h:
I have removed, DropEvent drop; inside the union, looking like this:

Maybe I should remove the union and leave the other variables out, I still don't know.
With this, I can now drag files to the window and it works correctly, but I have found a new problem when viewing the images, when I click on BM, it throws me the following exception:

When I compiled version 0.7.0, I got it to work for me, but I don't know where it's failing now.

I have already seen what the problem is.
The problem is that it cannot find the DISK01.LEC files, etc. In the ScummExtract.cpp file, function:
*static void readData(const ResNode &node, void data) {
I have commented the following line:
os << "/Users/scemino/Downloads/Secret Of Monkey Island Vga (1990)(Lucas Arts)/DISK0" << node.disk << ".LEC";
and I have put my own location of the files, looking like this:
os << "D:/Secret Of Monkey Island Vga/DISK0" << node.disk << ".LEC";
This can be corrected, but in a quick way I have solved it like this, I attach a screenshot to clarify it:

With this change it already works (although with bugs, as Scemino commented), the images are displayed correctly and it allows saving them:

Although when saving them, it saves them wrong ... It inverts them vertically:

To solve it, I have changed inside the file ngf/src/Graphics/Image.cpp, in the function:
void Image::saveToFile (const std::filesystem::path& filename) const {
I have modified:
stbi__flip_vertically_on_write = 1;
as follows:
#ifndef WIN32
stbi__flip_vertically_on_write = 1;
#endif
Great with this, it already saves the images in the correct way:

But .... there is a problem, this negatively affects engge, as it saves saved games vertically inverted :(

So I have to put it as it was for it to work properly in engge
I will continue investigating ...
Regards :)

You've been warned:
https://github.com/scemino/EnggeFramework/blob/main/demos/ScummExtract/README.MD

Thank you very much for making this correction:
ScummExtract: remove hard-coded directory  (https://github.com/scemino/EnggeFramework/commit/c819057e22063083a3b88e53c74d1b64f3d40f6a).

You've been warned:
https://github.com/scemino/EnggeFramework/blob/main/demos/ScummExtract/README.MD
This is provided as is, I don't plan to support it.

Sorry, I misinterpreted the read.me file. I will not mention any problem regarding any of the demos.
I proceed to close this issue

My bad it's only the demo ScummExtract that I don't support, but you can open issues on the others.
I keep this demo because it can be used by someone.

My bad it's only the demo ScummExtract that I don't support, but you can open issues on the others.
I keep this demo because it can be used by someone.

I understand your posture and I see it correct. :)
Brilliant!!! Of the other demos if I'm going to ask you. I will also open other issues. I have seen some other problem


Feat: Enhancing the Free E-Books section

Describe the feature
The options of download and add to read later are not aligned perfectly . I want to add some styles and align the buttons perfectly. @anuragverma108 please assign this issue to me under GSSOC'24. I will try my best.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I would like to work on this issue , please assign this to me.
@anuragverma108

Assigned !

I would like to work on this issue , please assign this to me.


Please upload to Hackage

Nix needs the fix for the doctest failures

@andrewthad are you currently in the middle of anything which would block a release?

There’s nothing else I’ve got pending. It’s fine to do a release. It just needs a patch release version bump and a change log mentions.

Released as 1.7.2.


Validate 2 separate lists, only one radio has to be selected of the two lists

Versions:

VueJs: 2.5.2
Vee-Validate: 2.0.5

Description:
Trying to validate 2 separate lists, but only one radio has to be selected of the two lists. I've tried giving the same name="radio_group_1" but I'm getting some strange behaviors. Currently within the codesandbox demo, it's not validating the second list options on submit. Also, the option is being selected when the radio isn't even selected (not a Vee propblem).
Steps To Reproduce:
codesandbox

issue was with the v-model data was being seperated..
purposeSelected: ""


Fix outfile path.

I think you want dirname here rather than basename

Thanks for pointing this out, this and another fix was merged https://github.com/karolis-sh/esbuild-plugin-public-directory/pull/8 and released in 0.0.4


Release on NPM

The latests version on NPM is the 1.3.7 while it seems the latest version here is the 1.3.11. Would you mind to release the latest version in NPM please?
It should be as easy as:
cd PROJECT_FOLDER
npm publish

Thank you!
PS, I'm using unpkg.com so the site breaks with the last module.exports, which doesn't appear on the current release<EMAIL_ADDRESS>

Interesting. I wasn't aware that we ever released it on npm. I'll see if it's possible for me to republish.

I am not authorized to update that package. Someone else has registered the package on NPM. I did ask to be added as a contributor but in the meantime, you can message the owner of the package.

The people at NPM are great, so if you cannot solve it with the person who registered it for any reason you can just contact NPM and they will likely put you as the maintainer. I will contact the owner of the package though.

For anyone interested, I forked this to add some extra methods, got fed up with the Coffeescript, started converting to modern JS, and got pretty carried away... long story short it's on npm and in UMD format so it should work fine with unpkg.
I extracted functions into modules to allow for individual imports of the utility functions (import 'isHiragana' etc), and thus much smaller build size in cases that don't involve conversion, fixed a bunch of old issues that were in this repo, added more extensive test cases, as well as a few extra methods/options I needed.
I didn't go for a PR since it turned out to be such a rewrite. Also I did modify the punctuation handling (which might have breaking changes for WaniKani etc).

@DJTB That's actually great news. I originally wrote it in Coffeescript because the project was spec'd as such, but having it in CS meant that I somewhat dread making changes to it 😭 . It's not totally clear whether I have the authority to make this call but I'd like to sunset the CS version into another branch and make an ES6 version the master.
I'll take a look at yours.

I am down with sunsetting the CS version for ES6.

I'm easy, all the groundwork was by the original contributors ^^
I didn't remove any of the old tests, only added more - so theoretically I guess there shouldn't really be breaking changes, just unexpected benefits like ! converting to ！ in the IME mode.
For reference, #45 and #46 are both integrated in the ES6 version.

Hey everyone, the owner of the NPM package has given me permission to publish the module. Only now, i have 3 problems.

I am having some issues with the code. I THINK it's working ok but the grunt-qunit task is not running due to an error on phantomjs. I haven't touched it in so long that I'm not confident in being able to build it anymore. Maybe someone can either build it and confirm that it's working or maybe I can publish then we check that it went ok? Or maybe we wait for point 2...
I want to move to the kanawana version asap. @DJTB I wonder if you want to make a pull request from your repo that could become Version 2.0. Is it as simple as that?
I have 0 time for this right now 😅

Should be reasonably* straightforward.
Regarding NPM, I set up kanawana to be able to publish multiple packages (was thinking of providing IME mode separate to the convenience/text methods) - but it confuses the build system slightly, especially since it only is publishing a single package right now! I'll simplify that back to just publishing the main/single package. As well as a find/replace kanawana -> wanakana.
Once that's done, I could PR into a new branch here to do final cleanup: we'll need to reconnect / create WK links for Travis & Coveralls at least. Close relevant issues & old PRs.
Would you like to make me a collaborator to assist with all this?

I have sent the invite @DJTB. Let me or @mimshwright know if you need anything else.

Started the conversion in release-2.0 branch.
Still in progress. Updated build & release scripts looking good so far.
Distribution files aren't committed - but the compiled browser package can still be copied during publish to the gh-pages branch for inclusion in demos/wanakana site.
@vietqhoang What do you want to do about docs?

Just link to docs
Just have basic usage
Include basic usage in readme / on the Wanakana website and link from there to extended docs?

The docs are currently parsed with jsdoc like so

Thanks for the hard work. I think third option is the best?

Naturally, but .
I was going to copy the current gh-pages branch to the master branch in a subdir - so it can then be pushed to gh-pages branch automatically with any updated changes from releases. However, the assets appear to have been built at some point - and I can't see how/where that was done. Is it from a separate repo? I can just hardcode the html changes if you won't want/need to rebuild the styles/js in future. Looking at gh-pages commits it seems the assets are effectively frozen as-is - and just the html is being updated for new versions.
Ideally it'd be nicer to keep the readme/site in sync since they contain approximately the same information - though if there is no longer any build for the website and changes are minimal enough (find and replace text in the html) it's probably quicker to just do it manually than to set up a new build.

The pages and assets are all there is; no preprocessing is done.
Manual is fine, but if you want to streamline it go for it.

Well, the assets include scss -> css but I won't change styling anyhow. Manual it is!
I've sent through a request to the WaniKani github org from travis and coveralls for authorization. @vietqhoang could you accept or prod someone else on the org to do so?
Should be ready to merge and publish once the CI passes and I've updated those links in the readme.

Ahh was there SCSS? Been so long sincr I looked.
Missed the Travis request. Sorry. Just approved the rest of your requests.

Aww dang. Think you'll have to log in to travis and flip the switch for me.

https://travis-ci.org/profile/WaniKani

Switch is flipped

@mimshwright @vietqhoang Do either of you want to pull down release-2.0, run npm run build and play with the dist files at all before I merge to master -> publish as 2.0.0?
After building:
dist/*.* files for direct/slim es6 imports
dist/lib/wanakana.js = main umd build
dist/lib/wanakana.min.js = minified umd build
dist/lib/wanakana.esm.js = es6 module build
When published, the npm package will reference the builds via main, browser, module fields so users can just require or import and they'll get the right one.

Ran the build and took a look. Everything looks good to me. 👍 Thanks for the hard work.

Right on! I'll publish it in the next day or so and deprecate kanawana -> point people to this package.
I'd love to take a stab at some of the remaining input issues sometime but there isn't a device lab hereabouts for me to test on. I doubt that emulation will provide valuable results, but I might see what browserstack can provide anyway.
If any general new issues crop up from the release I should have the time to fix those.

@mimshwright Can you add (or ask cuongcua90 to add) viet and I (or the entire WaniKani org?) as collaborators using npm access?
Otherwise if I try to publish I'm pretty sure it'll be rejected.

@DJTB I added you to WanaKana on NPM. You should be able to publish. If @vietqhoang  or anyone else wants to be added, please let me know your npm handle.
Also, @DJTB I just noticed you're also in Berlin. Me too. How cool!

Ah.. yes... well... あのーー... I was!
And will be again I suppose, I have some close friends there.
I've been moving about every 1-3 months for the past few years, so at any one time all my interweb accounts will say I'm in different countries. Causes trouble it does.
Perhaps I should make a script that can update all my damn accounts at once through navigator.geolocation. I'll even resort to programatically doing it with Nightmare.js if I have to haha.

:tada: :balloon: :tada:
Complete via 4e7b6ffff59ee6dccbf14250b9e902ba70cfb309

@mimshwright My npm handle is viethoang. Thanks!

@vietqhoang @DJTB Seems like if I just do import { isHiragana } from 'wanakana', the umd version of wanakana would be included, which means there's no tree shaking.
This is because the default target of webpack is 'web' and wanakana sets the default entry for browser to the umd version
  "main": "wanakana.js",
  "react-native": "wanakana.js",
  "module": "es/index.js",
  "browser": "umd/wanakana.min.js",
  "unpkg": "umd/wanakana.min.js"

I have to do import { isHiragana } from 'wanakana/es' instead to get the tree-shaking version.


docs: How do I reference $accessor inside a store's action tree?

How do I correctly reference $accessor inside a store's action tree?
This works:
store.ts
export const actions = actionTree(
  { state, getters, mutations },
  {
          this.app.$accessor.foo.bar()

    },

.. but getting type error: TS7022: 'actions' implicitly has type 'any' because it does not have a type annotation and is referenced directly or indirectly in its own initializer.
vue-shim.d.ts
declare module '@nuxt/types' {
  interface NuxtAppOptions {
    $accessor: typeof accessorType
  }
}

TS2502: '$accessor' is referenced directly or indirectly in its own type annotation.

@simondib You'll need to add an explicit return type for that action.
https://nuxt-typed-vuex.roe.dev/actions#referencing-other-modules

I'm closing this but do feel free to come back to me if that doesn't solve things for you :slightly_smiling_face:


Incorrect SSH key location (me-central-1)

Description
Newly created EC2 instance is adding SSH key to a wrong location (~root/.ssh/authorized_keys instead of ~ubuntu/.ssh/authorized_keys).

[ x ] ✋ I have searched the open/closed issues and my issue is not listed.

Versions

Module version [Required]:

Terraform version:

Terraform v1.2.8
on linux_amd64

provider registry.terraform.io/hashicorp/aws v4.28.0
provider registry.terraform.io/hashicorp/local v2.0.0

Your version of Terraform is out of date! The latest version
is 1.2.9. You can update by downloading from https://www.terraform.io/downloads.html

Provider version(s):

Terraform v1.2.8
on linux_amd64

provider registry.terraform.io/hashicorp/aws v4.28.0
provider registry.terraform.io/hashicorp/local v2.0.0

Your version of Terraform is out of date! The latest version
is 1.2.9. You can update by downloading from https://www.terraform.io/downloads.html
Reproduction Code [Required]
data "aws_ami" "ubuntu" {
  most_recent = true
  owners      = ["279184472064"]

  filter {
    name = "name"
    values = ["ubuntu-minimal/images/hvm-ssd/ubuntu-focal-20.04-amd64-minimal-*"]
  }

  filter {
    name   = "virtualization-type"
    values = ["hvm"]
  }
}

resource "aws_eip" "_" {
  for_each = local.eip_enabled_instances

  instance = module.ec2_instance[each.key].id
  vpc      = true
}

resource "aws_key_pair" "_" {
  key_name   = "${var.app_name}-${var.env}"
  public_key = var.public_key
}

module "ec2_instance" {
  for_each = var.ec2_instances

  source  = "terraform-aws-modules/ec2-instance/aws"
  version = "~> 4.1.4"

  name                        = "${var.app_name}-${var.env}-${each.key}"
  ami                         = data.aws_ami.ubuntu.id
  instance_type               = each.value.instance_type
  key_name                    = aws_key_pair._.key_name
  monitoring                  = var.detailed_monitoring
  associate_public_ip_address = each.value.public_ip
  vpc_security_group_ids      = [module.vpc.default_security_group_id, aws_security_group._[each.key].id]
  subnet_id                   = each.value.public_ip ? module.vpc.public_subnets[0] : module.vpc.private_subnets[0]

  metadata_options = {
    http_endpoint = "disabled"
  }

  root_block_device = [
    {
      encrypted   = true
      volume_type = "gp3"
      volume_size = each.value.volume_size
    },
  ]
}

Steps to reproduce the behavior:
tf init; tf apply

I tried to reproduce this issue by manually creating an EC2 machine with this AMI and SSH key - I couldn't reproduce this issue, but I was able to mount volume from module created EC2 instance and find what is causing the issue.
Expected behavior
SSH key should be in correct location.

Actual behavior
SSH key is added to the incorrect location.

Terminal Output Screenshot(s)
root@ip-10-0-101-229:/mnt# cd home/ubuntu/.ssh/
root@ip-10-0-101-229:/mnt/home/ubuntu/.ssh# ls -alh
total 8.0K
drwx------ 2 ubuntu ubuntu 4.0K Sep  8 11:31 .
drwxr-xr-x 3 ubuntu ubuntu 4.0K Sep  8 11:31 ..
-rw------- 1 ubuntu ubuntu    0 Sep  8 11:31 authorized_keys
root@ip-10-0-101-229:/mnt/home/ubuntu/.ssh# cat ^C
root@ip-10-0-101-229:/mnt/home/ubuntu/.ssh# sudo su -
root@ip-10-0-101-229:~# ls -alh
total 24K
drwx------  4 root root 4.0K Sep  8 12:29 .
drwxr-xr-x 19 root root 4.0K Sep  8 12:33 ..
-rw-r--r--  1 root root 3.1K Dec  5  2019 .bashrc
-rw-r--r--  1 root root  161 Dec  5  2019 .profile
drwx------  2 root root 4.0K Sep  8 12:29 .ssh
drwx------  3 root root 4.0K Sep  8 12:29 snap
root@ip-10-0-101-229:~# cat .ssh/authorized_keys
no-port-forwarding,no-agent-forwarding,no-X11-forwarding,command="echo 'Please login as the user \"ubuntu\" rather than the user \"root\".';echo;sleep 10;exit 142" ssh-rsa CUT_SSH_KEY

Additional context
I'm not sure if there's same problem in other regions - maybe it's region-specific, as me-central-1 is quite new.

Are you sure, though?
After all I wasn't able to replicate this issue by using same AMI, same instance type, same region and so on when I created instance manually from the console.

I am sure it is not related to the module because there are no differences in the code related to the region.
Something has to be different. Maybe in the way the new region behaves or the way AMI was built or copied.

I checked this code also in eu-central-1 and I have exactly same behavior.


⚠ Bofrost.lu has degraded performance

In 6ac5612, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1221 ms

Resolved: Bofrost.lu performance has improved in 7a957b2 after 15 minutes.


PowerShell example in the --wait section implies the wrong return type

Thank you for adding the Using --wait section at the request of the development community. However, the PowerShell example there is misleading. It assigns the result of Start-Process to a variable called $exitCode. However, what Start-Process actually returns is a System.Diagnostics.Process object, which has a member called ExitCode. So you should have something similar to
$proc = StartProcess . . .
$exitCode = $proc.ExitCode

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

ID: eedaccf1-bbb7-9a7c-21be-64217359acb5
Version Independent ID: da785eb8-7187-0831-0b3c-1b422c6f2a15
Content: Command-line parameter examples for installation - Visual Studio
Content Source: docs/install/command-line-parameter-examples.md
Product: visual-studio-windows
Technology: vs-installation
GitHub Login: @ornellaalt
Microsoft Alias: ornella

@LSpencer777 -- you're welcome, and thank you for bringing this issue up.
@ornellaalt -- please review private PR 6982.


router-link does not show href tags

Version
2.7.0
Reproduction link
https://jsfiddle.net/zysj2meg/2/
Steps to reproduce
install vue via vue-cli
What is expected?
vue init webpack-simple vue
cd vue
npm install
npm install --save vue-router
npm run dev
-- create routes, and add router-link inside html
What is actually happening?
The routes show but without anchor tag (a)

You used tag="li" to change the tag purposely, so it's normal that you get li instead with no href.


Share issues.

There are multiple share bugs.  #1990  #1748  #815  #451. Some or all may be dups.
Only track's title (without url) is shared from soundcloud to riot. The workaround is to share to clipboard and then insert it into riot. It doesn't depend on markdown setting. The full buffer from soundcloud is:
Listen to Skeewiff Ft Bam (Jungle Brothers) - Somebody's Watching Me ***FREEDL*** by Skeewiff #np on #SoundCloud
https://soundcloud.com/skeewiff/skeewiff-ft-bam-somebodys-watchin-me

I've had people tell me that this same issue also happens with Pinterest, only the title but not the URL is shared.

My idea is that riot cuts the text after the first "\n".

My guess is that the Intent has both a title/description and a message/content, and that Riot shares the former instead of the latter.

Confirmed my guess by sharing a message with my email app and by inspecting it with an intent debugger. What Riot shares is the subject instead of the text, because of matrix-org/matrix-android-sdk#284.

Fixed thanks to https://github.com/matrix-org/matrix-android-sdk/pull/285


HttpsTests consistently failed on Windows 2008

Test:
[InlineData(RuntimeFlavor.Clr, RuntimeArchitecture.x64, "https://localhost:44396/", ApplicationType.Standalone)]
public Task Https_HelloWorld(RuntimeFlavor runtimeFlavor, RuntimeArchitecture architecture, string applicationBaseUrl, ApplicationType applicationType)
{
    return HttpsHelloWorld(ServerType.IISExpress, runtimeFlavor, architecture, applicationBaseUrl, applicationType);
}

Failure:
System.Net.Http.HttpRequestException : An error occurred while sending the request.
---- System.Net.WebException : The underlying connection was closed: An unexpected error occurred on a send.
-------- System.IO.IOException : Unable to read data from the transport connection: An existing connection was forcibly closed by the remote host.
------------ System.Net.Sockets.SocketException : An existing connection was forcibly closed by the remote host
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Server.Testing.RetryHelper.<RetryRequest>d__0.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.AspNetCore.Server.Testing.RetryHelper.<RetryRequest>d__0.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Server.IISIntegration.FunctionalTests.HttpsTest.<HttpsHelloWorld>d__1.MoveNext() in D:\b\work\35116a8defcd3a21\Universe\IISIntegration\test\Microsoft.AspNetCore.Server.IISIntegration.FunctionalTests\HttpsTest.cs:line 60
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
----- Inner Stack Trace -----
   at System.Net.HttpWebRequest.EndGetResponse(IAsyncResult asyncResult)
   at System.Net.Http.HttpClientHandler.GetResponseCallback(IAsyncResult ar)
----- Inner Stack Trace -----
   at System.Net.TlsStream.EndWrite(IAsyncResult asyncResult)
   at System.Net.ConnectStream.WriteHeadersCallback(IAsyncResult ar)
----- Inner Stack Trace -----
   at System.Net.Sockets.Socket.EndReceive(IAsyncResult asyncResult)
   at System.Net.Sockets.NetworkStream.EndRead(IAsyncResult asyncResult)

Test:
[InlineData(RuntimeFlavor.CoreClr, RuntimeArchitecture.x64, "https://localhost:44397/", ApplicationType.Standalone)]
[InlineData(RuntimeFlavor.CoreClr, RuntimeArchitecture.x64, "https://localhost:44398/", ApplicationType.Portable)]
public Task Https_HelloWorld_NoClientCert(RuntimeFlavor runtimeFlavor, RuntimeArchitecture architecture, string applicationBaseUrl, ApplicationType applicationType)

Failure:
System.Net.Http.HttpRequestException : An error occurred while sending the request.
---- System.Net.WebException : The underlying connection was closed: An unexpected error occurred on a send.
-------- System.IO.IOException : Unable to read data from the transport connection: An existing connection was forcibly closed by the remote host.
------------ System.Net.Sockets.SocketException : An existing connection was forcibly closed by the remote host
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Server.Testing.RetryHelper.<RetryRequest>d__0.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.AspNetCore.Server.Testing.RetryHelper.<RetryRequest>d__0.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Server.IISIntegration.FunctionalTests.HttpsTest.<HttpsHelloWorld>d__1.MoveNext() in D:\b\work\35116a8defcd3a21\Universe\IISIntegration\test\Microsoft.AspNetCore.Server.IISIntegration.FunctionalTests\HttpsTest.cs:line 60
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
----- Inner Stack Trace -----
   at System.Net.HttpWebRequest.EndGetResponse(IAsyncResult asyncResult)
   at System.Net.Http.HttpClientHandler.GetResponseCallback(IAsyncResult ar)
----- Inner Stack Trace -----
   at System.Net.TlsStream.EndWrite(IAsyncResult asyncResult)
   at System.Net.ConnectStream.WriteHeadersCallback(IAsyncResult ar)
----- Inner Stack Trace -----
   at System.Net.Sockets.Socket.EndReceive(IAsyncResult asyncResult)
   at System.Net.Sockets.NetworkStream.EndRead(IAsyncResult asyncResult

Seems like it's an issue with outdated tooling. I updated VS on the agent and grabbed the latest tooling along with ANCM, IISExpress... and the tests are passing now. I'll turn the tests back on.

tests re-enabled, closing.

thanks, @JunTaoLuo


Add changelog to gemspec

For a nice “Changelog” link in the https://rubygems.org/gems/http-form_data sidebar.

This change is

Thank you!


How do I input custom 2D values?

I'm currently using numerical inputs as training data, but I can only input 1-dimensional values in the following format:
{x x x x x} y y
as explained in the documentation.
Is there a way to directly enter 2D data as well? Just wondering!
Thanks :)

(This applies when entering training data as text inside inputData.txt.) Just arrange your 2D input values in one long linear list. For example, if your net has 16x16 inputs, you'll have 256 numbers inside the { } curly braces.

That's wonderful. Thanks David!


DRILL-6453: Resolve deadlock when reading from build and probe sides simultaneously in HashJoin

The Problem
Originally hash join sniffed the first data holding batch in the probe and build side. Using the size statistics from both sides, memory calculations were performed in order to determine when to spill data.
The issue with this is that fetching the first data holding batch from both sides can cause a deadlock in the exchange operators. Details of how this can happen have been included by others on the Jira.
Theory of Operation
Sniffing Batches
Batch sniffing is done in three phases.

Schema sniffing is done in buildSchema()
Before executing the build phase we sniff the first data holding build side batch and use the stats to decide the number of partitions and do memory calculations.
Before executing the probe phase we sniff the first data holding probe side batch, and use the size statistics to do memory calculations that decide when to spill.

Memory Estimation
When sniffing the schema for the build and probe side, we may get lucky and get data for the probe side. If this is the case then we cause use the probe side data to estimate the optimal number of partitions to use in the join operator. If we don't have probe side data when computing the number of partitions to use we assume that the incoming probe batches will be less than or equal to the configured batch size.
Since the number of partitions must be configured upfront before we may have probe data, we may get stuck in a situation where we have too many partitions to effectively process the probe side. In order to avoid this scenario we also adjust the number of records in probe side partition batches after we receive data from the probe side.
Corner cases
While implementing this many corner cases had to be handled.

Empty build side
Empty probe side
Empty probe and build sides
Getting probe side data when retrieving the probe schema
Not getting probe side data when retrieving the probe schema

Testing
I added unit tests for all the corner cases, and have extracted logic for predicting incoming and partition batch sizes into BatchSizePredictorImpl. In unit tests various corner cases are tested by providing mock implementation of BatchSizePredictorImpl.

@Ben-Zvi I've refactored the code, should be much cleaner now. Please take another look.

@Ben-Zvi Thanks for the review and suggestions. The code is much cleaner now.

OK - tag it ready-to-commit ...


Update log4j to 2.17.1

This fixes https://nvd.nist.gov/vuln/detail/CVE-2021-44832

Thank you @loganaden !

Can you cut a release too @jeqo?
On Thu, Dec 30, 2021, 18:48 Jorge Esteban Quilcate Otoya <
@.***> wrote:

Merged #1321 https://github.com/openzipkin/brave/pull/1321 into master.
—
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/brave/pull/1321#event-5828709168, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYAXRYHKKYEMX74MKTYTUTSLPBANCNFSM5K5ILLWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Of course. Will do tomorrow

I just created 5.13.7


Fixed incorrect sample summary reference

Reads were being observed to writeSamples instead of readSamples

Created a new PR from a different branch, rather than master.


Add a callback when the MemoryManager adjusts row group size

Parquet Hive would like to increment a counter when the row group size is altered by the memory manager so that Hive can detect when there are memory problems and inform the user. I think the right way to do this is to provide a callback that will be triggered when the memory manager hits its limit.
Reporter: Ryan Blue / @rdblue
Assignee: Dong Chen / @dongc
Related issues:

Warn when parquet memory manager kicks in (is related to)

PRs and other links:

PR #120

Note: This issue was originally created as PARQUET-199. Please see the migration documentation for further details.

Ryan Blue / @rdblue:
This builds on PARQUET-164; see discussion on Dong's pull request: https://github.com/apache/incubator-parquet-mr/pull/120

Ryan Blue / @rdblue:
This was merged a few days ago, just forgot to close.


do not wait for data from timebox on connect

Hello noeRls, thank you for your work on the Timebox Evo server and integration!
When trying your code, I had an issue, that my Timebox Evo does not send  any data on connect, which would mean, that the connect request would timeout, as the Promise was never resolved.
Maybe there are differences between different firmware versions on it?  Resolving the Promise directly on connect, makes it work perfectly fine for me.

I ordered my Timebox Evo about a year ago, but will be receiving a second unit either this or next week. I'll test if they behave differently then, too.

Okay, it's working for me. In my case I still receive the data, I guess they have different implementation of the startup protocol for the timebox evo.
It just skip the confirmation step of receiving data. However it is not a big deal because the connection will fail before if there is a problem.
Thanks again for your contribution !


Use opentelemetry-instrumentation-bom-alpha

It seems opentelemetry-bom-alpha contains opentelemetry-bom so we don't need to depend on the latter.
Since opentelemetry-instrumentation-api-semconv and io.opentelemetry:opentelemetry-bom-alpha can point to conflicting versions, we should use another BOM: opentelemetry-instrumentation-bom-alpha which contains opentelemetry-bom-alpha.
This BOM has an issue though, its patch version upgrades contain minor version upgrades for opentelemetry-bom-alpha, see:

1.33.0 -> 1.35.0
1.33.1 -> 1.36.0
1.33.2 -> 1.37.0
1.33.3 -> 1.38.0
1.33.4 -> 1.39.0
1.33.5 -> 1.40.0

So maybe we should not upgrade OTel throughout our patch releases. This might not be a huge issue though since OTel rarely issues patch releases for opentelemetry-bom-alpha or opentelemetry-bom.
Closes gh-819

If we decide to go forward with this, the otelInstrumentationBomAlpha version should be 1.33.3-alpha for 1.3.x since it contains io.opentelemetry:opentelemetry-bom:1.38.0.
For main, we can stay on 1.33.3-alpha or see if we should upgrade to 2.7.0-alpha.


fix(ecmascript-runtime): relative url improvement + env setup

Description
This PR fixes 2 things:
First, put the runtime fn correctly to the corresponding environment. Previously node / dom env missed few runtime fn to resolve URLs, now those are correctly in place.
Secondly, changes relative url behavior to have base for the rendering context such as location.origin or server_addr. This is mostly bringing back old urlrewrite implementation. Primary example where we currently fail is below:
export const runtime = 'edge'

fetch(new URL('../some/asset.ttf', import.meta.url));

since we got the module path such as /_next/static/media/typewr__.9434c090.ttf, omitting base makes incorrect url construct without base. In case of webpack it have asset specific urls like blob:...ttf which can be a valid URL without base, then it can be fetched through. By rebringing old location rewrite we won't have real relative url but it'll be a valid url can be accessed.
Closes PACK-1995

Working on resolving test failure.


IllegalArgumentException on world load

Zip of world: https://mega.nz/#!GU1lnbjQ!Fp5Z5YmM91WOl8p5oRs_gE9KDzMFXq19SCMFsKW7vGc
Stacktrace:
net.minecraft.util.ReportedException: Ticking memory connection
	at net.minecraft.network.NetworkSystem.networkTick(NetworkSystem.java:210) ~[NetworkSystem.class:?]
	at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:806) ~[MinecraftServer.class:?]
	at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:687) ~[MinecraftServer.class:?]
	at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:156) ~[IntegratedServer.class:?]
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:536) [MinecraftServer.class:?]
	at java.lang.Thread.run(Thread.java:745) [?:1.8.0_65]
Caused by: java.lang.IllegalArgumentException: minY > maxY (65 > 64)
	at cubicchunks.world.cube.Cube$LightUpdateData.queueLightUpdate(Cube.java:531) ~[Cube$LightUpdateData.class:?]
	at cubicchunks.lighting.LightingManager.queueDiffuseUpdate(LightingManager.java:74) ~[LightingManager.class:?]
	at cubicchunks.lighting.LightingManager.columnSkylightUpdate(LightingManager.java:59) ~[LightingManager.class:?]
	at cubicchunks.server.chunkio.IONbtReader.readLightingInfo(IONbtReader.java:255) ~[IONbtReader.class:?]
	at cubicchunks.server.chunkio.IONbtReader.readCube(IONbtReader.java:115) ~[IONbtReader.class:?]
	at cubicchunks.server.chunkio.CubeIO.loadCubeAndAddToColumn(CubeIO.java:170) ~[CubeIO.class:?]
	at cubicchunks.server.ServerCubeCache.loadCube(ServerCubeCache.java:348) ~[ServerCubeCache.class:?]
	at cubicchunks.server.ServerCubeCache.loadCube(ServerCubeCache.java:388) ~[ServerCubeCache.class:?]
	at cubicchunks.world.dependency.DependencyManager.register(DependencyManager.java:122) ~[DependencyManager.class:?]
	at cubicchunks.worldgen.dependency.DependentCubeManager.register(DependentCubeManager.java:75) ~[DependentCubeManager.class:?]
	at cubicchunks.worldgen.WorldGenerator.generateCube(WorldGenerator.java:156) ~[WorldGenerator.class:?]
	at cubicchunks.worldgen.WorldGenerator.generateCube(WorldGenerator.java:172) ~[WorldGenerator.class:?]
	at cubicchunks.server.ServerCubeCache.loadCube(ServerCubeCache.java:369) ~[ServerCubeCache.class:?]
	at cubicchunks.server.ServerCubeCache.loadCube(ServerCubeCache.java:388) ~[ServerCubeCache.class:?]
	at cubicchunks.world.dependency.DependencyManager.register(DependencyManager.java:122) ~[DependencyManager.class:?]
	at cubicchunks.worldgen.dependency.DependentCubeManager.register(DependentCubeManager.java:75) ~[DependentCubeManager.class:?]
	at cubicchunks.worldgen.WorldGenerator.generateCube(WorldGenerator.java:156) ~[WorldGenerator.class:?]
	at cubicchunks.worldgen.WorldGenerator.generateCube(WorldGenerator.java:172) ~[WorldGenerator.class:?]
	at cubicchunks.server.ServerCubeCache.loadCube(ServerCubeCache.java:340) ~[ServerCubeCache.class:?]
	at cubicchunks.server.ServerCubeCache.loadCube(ServerCubeCache.java:384) ~[ServerCubeCache.class:?]
	at cubicchunks.server.PlayerCubeMapEntry.<init>(PlayerCubeMapEntry.java:71) ~[PlayerCubeMapEntry.class:?]
	at cubicchunks.server.PlayerCubeMap.getOrCreateCubeWatcher(PlayerCubeMap.java:304) ~[PlayerCubeMap.class:?]
	at cubicchunks.server.PlayerCubeMap.lambda$addPlayer$6(PlayerCubeMap.java:366) ~[PlayerCubeMap.class:?]
	at cubicchunks.visibility.CuboidalCubeSelector.forAllVisibleFrom(CuboidalCubeSelector.java:43) ~[CuboidalCubeSelector.class:?]
	at cubicchunks.server.PlayerCubeMap.addPlayer(PlayerCubeMap.java:362) ~[PlayerCubeMap.class:?]
	at net.minecraft.server.management.PlayerList.preparePlayer(PlayerList.java:310) ~[PlayerList.class:?]
	at net.minecraft.server.management.PlayerList.playerLoggedIn(PlayerList.java:389) ~[PlayerList.class:?]
	at net.minecraft.server.management.PlayerList.initializeConnectionToPlayer(PlayerList.java:171) ~[PlayerList.class:?]
	at net.minecraftforge.fml.common.network.handshake.NetworkDispatcher.completeServerSideConnection(NetworkDispatcher.java:263) ~[NetworkDispatcher.class:?]
	at net.minecraftforge.fml.common.network.handshake.NetworkDispatcher.access$100(NetworkDispatcher.java:73) ~[NetworkDispatcher.class:?]
	at net.minecraftforge.fml.common.network.handshake.NetworkDispatcher$1.update(NetworkDispatcher.java:212) ~[NetworkDispatcher$1.class:?]
	at net.minecraft.network.NetworkManager.processReceivedPackets(NetworkManager.java:308) ~[NetworkManager.class:?]
	at net.minecraft.network.NetworkSystem.networkTick(NetworkSystem.java:195) ~[NetworkSystem.class:?]
	... 5 more

Is this old world? Or newly generated?

Newly generated. Crashed because of an NPE, added "-XX:-OmitStackTraceInFastThrow" to my JVM args so I could get a stacktrace, now instantly crashing with this error on world load.

Do you have seed? Might be easier to debug this way.

NPE crash that is fices by this? Seems like PlayerChunkMap crash. These are bad. The first occurrence of this NPE should have fuill stacktrace.

I'll try and grab the seed, it was randomly generated. I didn't get a full stacktrace when it crashed originally.

It probably crashed started throwing them every tick. If it spammed your console with them - look for the first one in old logs. It should have full stacktrace.

The crash you are experiencing now is likely due to world corruption, but even that shouldn't happen. I didn't touch the code for a long time, but as I recall - there is no way these values could get into light update queue to be saved to NBT in the first place.

Went through the logs folder and I can't seem to find the client log, looks like it was deleted (it looks like Forge is only keeping the 3 most recent).
Seed for the world is<PHONE_NUMBER>876177193

Looks like I just got the same issue myself with my world.

If there's anything I can do to help, just let me know. Once I'm more familiar with Cubic Chunks internals I can probably be more useful.

I hope so. Now I think I should write more comments in my code... I can't figure out how the code was supposed to work.


Rename TallyHo wallet name to Taho

Description
The rename request TallyHo -> Taho came from the Tally team on a rebrand that has occurred
Updated wallet name, docs and readme-s to make note of this (I don't think creating a new package is justified at this point and would be confusing and a pain for DApps who currently support Tally/Taho)

Request from Taho team:

Checklist

[ ] The version field in package.json of the package you have made changes in is incremented following semantic versioning and using alpha release tagging
[ ] The box that allows repo maintainers to update this PR is checked
[ ] I tested locally to make sure this feature/fix works
[ ] I have run yarn file-check, yarn type-check & yarn build to confirm there are not any associated errors
[ ] This PR passes the Circle CI checks

@Adamj1232 I think they may want the package name to change? 🤔 Should we double check? It would be simple enough I suppose and then on the old tallyho package we could show it is deprecated? Or even use this?

@Adamj1232 I think they may want the package name to change? 🤔 Should we double check? It would be simple enough I suppose and then on the old tallyho package we could show it is deprecated? Or even use this?

@taylorjdawson Yeah that is a good question and we discussed it yesterday in the planning meeting. Im waiting to hear back from the Tally/Taho team on this so we will see what their response is. Their initial request was to just update the displayed wallet name, but its prob best to create a new package and deprecate Tally


error during parsing of prepared statement causes perpetual error state

Discovered by usql project. Parsing errors of prepared statements causes error state and eventual lock. This was caused by not sending the required sync() call to the backend after the error. Fixed in 0.1.7.

Fixed in 0.1.7


[AutoDiff] NFC: fix warning in DeserializeSIL.cpp.

Remove unused variable Attr3. It is no longer necessary after
https://github.com/apple/swift/pull/27919:
DifferentiabilityWitnessFunctionInst has fewer fields to deserialize.

@swift-ci Please test tensorflow


text diff tool

What's the Problem?
Sometimes I need to compare diffs between two text strings.
Solution/Idea
Feature: diff tool.
Alternatives
https://tool.chinaz.com/tools/diff
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

DevToys Version
No response
Comments
No response

Hi,
There's already a Text Diff tool :)


Sync fails when typesafe project accessors is enabled

If I turn on TYPESAFE_PROJECT_ACCESSORS it fails sync'ing
Expected Behavior
No errors
Current Behavior
If I add enableFeaturePreview("TYPESAFE_PROJECT_ACCESSORS") under settings.gradle.kts then when I sync I get the following:

org.gradle.api.internal.catalog.GeneratedClassCompilationException: Unable to compile generated sources:

File RootProjectAccessor.java, line: 29, method getTherapiRuntimeJavadoc() is already defined in class org.gradle.accessors.dm.RootProjectAccessor
Unable to compile generated sources:
- File RootProjectAccessor.java, line: 29, method getTherapiRuntimeJavadoc() is already defined in class org.gradle.accessors.dm.RootProjectAccessor

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Context
I'd like to have project accessors for settings up intermodule dependencies
Steps to Reproduce

clone this commit
sync

Your Environment
Gradle 7.2
IntelliJ IDEA 2021.3 Beta (Ultimate Edition)
Build #IU-213.5605.12, built on November 3, 2021
Openjdk 11.0.9

@elect86 did you find any solution on this? Just enabled the feature and ran into the same issue.

nope

I think I had the same issue. It seemed to be that the rootProject shared the name with one of the subprojects.
rootProject.name = 'DifferentName'

Renaming the rootProject resolved the issue.


Issue: Unable to run the example in the readme

I'm new to both gnark and Go. I created a go module and created a file cubic.go and pasted the code from the readme (attached below).
Step 1:
go mod

runs without error and installs all the packages.
Step 2:
go run .

errors:
# github.com/consensys/gnark/internal/backend/bn254/groth16
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bn254/groth16/setup.go:55:25: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bn254/groth16/setup.go:78:26: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bn254/groth16/setup.go:258:27: assignment mismatch: 2 variables but pedersen.Setup returns 3 values
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bn254/groth16/prove.go:337:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bn254/groth16/prove.go:338:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bn254/groth16/prove.go:339:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bn254/groth16/prove.go:357:32: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFTInverse
# github.com/consensys/gnark/internal/backend/bls12-381/groth16
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-381/groth16/setup.go:55:25: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-381/groth16/setup.go:78:26: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-381/groth16/setup.go:258:27: assignment mismatch: 2 variables but pedersen.Setup returns 3 values
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-381/groth16/prove.go:337:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-381/groth16/prove.go:338:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-381/groth16/prove.go:339:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-381/groth16/prove.go:357:32: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFTInverse
# github.com/consensys/gnark/internal/backend/bls24-317/groth16
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-317/groth16/setup.go:55:25: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-317/groth16/setup.go:78:26: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-317/groth16/setup.go:258:27: assignment mismatch: 2 variables but pedersen.Setup returns 3 values
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-317/groth16/prove.go:337:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-317/groth16/prove.go:338:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-317/groth16/prove.go:339:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-317/groth16/prove.go:357:32: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFTInverse
# github.com/consensys/gnark/internal/backend/bw6-633/groth16
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-633/groth16/setup.go:55:25: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-633/groth16/setup.go:78:26: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-633/groth16/setup.go:258:27: assignment mismatch: 2 variables but pedersen.Setup returns 3 values
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-633/groth16/prove.go:337:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-633/groth16/prove.go:338:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-633/groth16/prove.go:339:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-633/groth16/prove.go:357:32: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFTInverse
# github.com/consensys/gnark/internal/backend/bls12-377/groth16
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-377/groth16/setup.go:55:25: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-377/groth16/setup.go:78:26: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-377/groth16/setup.go:258:27: assignment mismatch: 2 variables but pedersen.Setup returns 3 values
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-377/groth16/prove.go:337:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-377/groth16/prove.go:338:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-377/groth16/prove.go:339:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls12-377/groth16/prove.go:357:32: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFTInverse
# github.com/consensys/gnark/internal/backend/bls24-315/groth16
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-315/groth16/setup.go:55:25: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-315/groth16/setup.go:78:26: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-315/groth16/setup.go:258:27: assignment mismatch: 2 variables but pedersen.Setup returns 3 values
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-315/groth16/prove.go:337:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-315/groth16/prove.go:338:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-315/groth16/prove.go:339:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bls24-315/groth16/prove.go:357:32: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFTInverse
# github.com/consensys/gnark/internal/backend/bw6-761/groth16
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-761/groth16/setup.go:55:25: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-761/groth16/setup.go:78:26: undefined: pedersen.Key
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-761/groth16/setup.go:258:27: assignment mismatch: 2 variables but pedersen.Setup returns 3 values
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-761/groth16/prove.go:337:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-761/groth16/prove.go:338:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-761/groth16/prove.go:339:25: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFT
../../../go/pkg/mod/github.com/consensys/gnark@v0.8.0/internal/backend/bw6-761/groth16/prove.go:357:32: cannot use true (untyped bool constant) as fft.Option value in argument to domain.FFTInverse

Environment: Windows Subsystem for Linux
package main

import (
	"github.com/consensys/gnark-crypto/ecc"
	"github.com/consensys/gnark/backend/groth16"
	"github.com/consensys/gnark/frontend"
	"github.com/consensys/gnark/frontend/cs/r1cs"
)

// CubicCircuit defines a simple circuit
// x**3 + x + 5 == y
type CubicCircuit struct {
	// struct tags on a variable is optional
	// default uses variable name and secret visibility.
	X frontend.Variable `gnark:"x"`
	Y frontend.Variable `gnark:",public"`
}

// Define declares the circuit constraints
// x**3 + x + 5 == y
func (circuit *CubicCircuit) Define(api frontend.API) error {
	x3 := api.Mul(circuit.X, circuit.X, circuit.X)
	api.AssertIsEqual(circuit.Y, api.Add(x3, circuit.X, 5))
	return nil
}

func main() {
	// compiles our circuit into a R1CS
	var circuit CubicCircuit
	ccs, _ := frontend.Compile(ecc.BN254.ScalarField(), r1cs.NewBuilder, &circuit)

	// groth16 zkSNARK: Setup
	pk, vk, _ := groth16.Setup(ccs)

	// witness definition
	assignment := CubicCircuit{X: 3, Y: 35}
	witness, _ := frontend.NewWitness(&assignment, ecc.BN254.ScalarField())
	publicWitness, _ := witness.Public()

	// groth16: Prove & Verify
	proof, _ := groth16.Prove(ccs, pk, witness)
	groth16.Verify(proof, vk, publicWitness)
}

Not sure if it's the ideal approach but downgrading the gnark-crypto dependency to v0.9.1 works.

Hi - see the solution in https://github.com/Consensys/gnark/issues/780, maybe it is similar issue? I.e. when not explicitly go getting gnark, then go mod tidy upgrades all implicit dependencies to the latest and there is mismatch between gnark and gnark-crypto.


Enable Travis cron and notifications features

As found in the recent stack image updates, the base Ubuntu docker image can sometimes change in a way that causes breakage unrelated to commits in this repo (eg fewer installed-by-default packages). Whilst this repo now has a Travis run for validating the build and installed package list, it is only triggered by pushes to master, so the most recent job can be showing as green even if a rebuild now might not be.
I was thinking it might be good to enable the Travis cron feature to trigger a extra builds on master once a {day, week, month} so problems can be identified prior to having to do an urgent release when new package CVEs are announced. This could be combined with the Travis notification system (email, IRC, slack, ...) to avoid having to remember to check the Travis status periodically.
For more details, see:
https://docs.travis-ci.com/user/cron-jobs/
https://docs.travis-ci.com/user/notifications/

Thank you for the suggestion @edmorley, it's a very good idea.
I've enabled weekly runs for the master branch (if there's been no other build in the past 24h).
As for notifications, we already have them internally, based on GitHub commit status.
Travis properly updates the commit status when running a cron job. So we'll be notified of failures.
cc @heroku/build


Update test code to use unix socket instead of tcp connection if avai…

…lable
@alainjobart
Change was approved internally last week

LGTM

LGTM


Harry todo

Mostly looks good - only final things, one of the checks is failing:
taxpose/training/pm_baselines/ompl_traj_gen.py:14: error: Cannot find implementation or library stub for module named "ompl"  [import]
taxpose/training/pm_baselines/ompl_traj_gen.py:14: note: See https://mypy.readthedocs.io/en/stable/running_mypy.html#missing-imports
taxpose/training/pm_baselines/ompl_traj_gen.py:187: error: "PMRenderEnv" has no attribute "articulate_specific_joints"  [attr-defined]
taxpose/training/pm_baselines/test_bc.py:270: note: By default the bodies of untyped functions are not checked, consider using --check-untyped-defs  [annotation-unchecked]
taxpose/training/pm_baselines/test_goal_flow.py:385: note: By default the bodies of untyped functions are not checked, consider using --check-untyped-defs  [annotation-unchecked]
taxpose/training/pm_baselines/test_goal_flow.py:388: note: By default the bodies of untyped functions are not checked, consider using --check-untyped-defs  [annotation-unchecked]
Found 2 errors in 1 file (checked 46 source files)
Error: Process completed with exit code 1.

There are only two errors here:

you need to add ompl.* to the [[tool.mypy.overrides]] section of pyproject.toml
you need to import articulate_specific_joints from here: https://github.com/r-pad/taxpose/blob/72d26d24f8c61ce568dadf8d74acdee583bba844/taxpose/training/pm_baselines/bc_dataset.py#L36 and use that instead of env.articulate_specific_joints, because that function doesn't actually exist anymore. you also need to pass env as an argument instead of calling env.articulate_specific_joints.


docs: Windows rewrite

Refactor the Windows documentation to keep it all in the same place
and not make the user go back and fourth between multiple docs.

Same PR as https://github.com/rancher/docs/pull/1453 -- against staging this time.


Update to 1.20.1 Please

It's marked for 1.20.1 on Modrinth, but it doesn't work.

Whoops, I'll push a hotfix tonight

Thanks 😊


python test failure: _crust_over_core_over_links_face_interface_test.GenericInvokerEventInvocationSynchronousEventServiceTest

https://grpc-testing.appspot.com/job/gRPC_master/15549/config=dbg,language=python,platform=macos/testReport/junit/(root)/tests/py_test__crust_over_core_over_links_face_interface_test_GenericInvokerEventInvocationSynchronousEventServiceTest/

Just a hunch, but the error on this one looks like it was a race condition on the file system wherein someone was trying to load the shared object while someone else was rewriting it.

Agreed. I think it has to do with
running build_ext
copying python_build/lib.macosx-10.10-intel-2.7/grpc/_cython/cygrpc.so -> ./src/python/grpcio/grpc/_cython

Weirder than that: those running build_ext bits are in the logs after the tests have started. They should very much be before...

Oh, haha, I got jumbled up. Didn't realize that we hadn't merged #5569 yet. That should help, yes.

I think even with #5569 we might see this problem (is there a way to disable running build_ext in the test fully?).

Not without making a custom command for testing, AFAIK. But if it's already built once the system should detect it and realize not to rebuild. I don't see any signs of the contrary on #5569.

I think basically the same problem can also show up as this:
ImportError: dlopen(/jenkins/workspace/gRPC_master/config/opt/language/python/platform/macos/src/python/grpcio/grpc/_cython/cygrpc.so, 2): no suitable image found.  Did find:
	/jenkins/workspace/gRPC_master/config/opt/language/python/platform/macos/src/python/grpcio/grpc/_cython/cygrpc.so: truncated fat file.  file length=315392, but needed slice goes to 2629896

https://grpc-testing.appspot.com/job/gRPC_master/15639/config=opt,language=python,platform=macos/testReport/junit/(root)/tests/py_test__base_interface_test_SyncEasyTest/

Also, this could exhibit as:
ImportError: /var/local/git/grpc/src/python/grpcio/grpc/_cython/cygrpc.so: cannot open shared object file: No such file or directory

Yeah, all sounds like the same problem.


Update README-hi.md on the  Improve readability and user-friendliness.  Issue#41989

Description

Issue

Closes: #

/retitle Update README-hi.md to improve readability and user-friendliness

Make sure that the README is a localized version of the English original.

/language hi


Submitting build 100 to coordinator...failed

Hi,
After upgrading to Gitlab 6.0 I added a new project to gitlab-ci and I get the above error message. It seems my runner gets the job, but cannot submit the results.
Is this a known problem, or what should I look at that causes this?
many thanks,
James

+1
However I have not tried the new official go based runner. Will give it a try and let you know.


How can I use your source files for chatgpt-advanced.

Hello.
I got a big inspiration from your project.
I am a university student who is doing a project about chat-gpt.
And your real-time chat function idea is amazing.
I'd like to vary your idea to get some simple infos from a website.
But I am not sure how can I make it based on your idea.
Can I get some informations how did you do this?

Hi, the code is MIT licensed, see https://github.com/qunash/chatgpt-advanced/blob/main/LICENSE


⚠ Fox Sports has degraded performance

In 2c57bc4, Fox Sports (https://foxsports.com) experienced degraded performance:

HTTP code: 200
Response time: 1114 ms

Resolved: Fox Sports performance has improved in 9c985f0.


Cluster Autoscaler can't scale worker nodes ASGs to zero

I have issues
I would like to enable the cluster autoscaler to support scaling down ASGs to zero nodes. According to the cluster autoscaler documentation, https://github.com/kubernetes/autoscaler/blob/master/cluster-autoscaler/FAQ.md#how-can-i-scale-a-node-group-to-0, we need to add the following tag to the ASG:
{
    "PropagateAtLaunch": true,
    "Value": "bar",
    "Key": "k8s.io/cluster-autoscaler/node-template/label/foo"
}

However, I wasn't able to find a good way of passing that extra tag to only one specific worker_group. I'd like to be able to specify a different value and key for each worker group.
I'm submitting a...

[ ] bug report
[X] feature request
[X] support request
[ ] kudos, thank you, warm fuzzy

What is the current behavior?
Current behavior does not allow passing extra tags to the ASGs that get created for each specific worker group.
What's the expected behavior?
Ideally, we'd be able to do something like this
 worker_group_count = 3

  worker_groups = [
    {
      name                 = "default"
      instance_type        = "r5.4xlarge"
      asg_desired_capacity = "8"
      asg_max_size         = "12"
      asg_min_size         = "8"
    },
    {
      name                 = "transparencyapi-green"
      instance_type        = "r5.4xlarge"
      asg_desired_capacity = "3"
      asg_max_size         = "3"
      asg_min_size         = "0"
      asg_extra_tags       = {
        key = "k8s.io/cluster-autoscaler/node-template/label/transparencyapi"
        value = "green"
        propagate_at_launch = "true"
      }

      kubelet_extra_args   = "--register-with-taints='deployment=green:NoSchedule' --node-labels='transparencyapi=green'"
    },
    {
      name                 = "transparencyapi-blue"
      instance_type        = "r5.4xlarge"
      asg_desired_capacity = "3"
      asg_max_size         = "3"
      asg_min_size         = "0"
      asg_extra_tags       = {
      key = "k8s.io/cluster-autoscaler/node-template/label/transparencyapi"
      value = "blue"
      propagate_at_launch = "true"
      }
      kubelet_extra_args   = "--register-with-taints='deployment=blue:NoSchedule' --node-labels='transparencyapi=blue'"
    },
  ]

Are you able to fix this problem and submit a PR? Link here if you have already.
I'd be more then happy to submit a PR for this if I can get some guidance. I'm having trouble figuring out the best way to enable this.
Environment details

Affected module version: master
OS: Ubuntu 18.10
Terraform version: 0.11.0

Any other relevant info

Hey @Chili-Man you are correct! And we have another issue already for this here: https://github.com/terraform-aws-modules/terraform-aws-eks/issues/89
Feel free to add your voice to that issue or better yet, make a PR 😎


Settings: Developer Mode

With Developer Mode activated, the app will display internal data (list ids, indices, isComplete, etc.).
Developer mode will be enabled/disabled with a checkbox in the settings page.

Commit: https://github.com/johneastman/todo/commit/f0736ee68850c5a4754e07c487c96751aef12fa7


Uncaught (in promise): InvalidRibbonDefinition: InvalidRibbonDefinition RichApi.Error: InvalidRibbonDefinition

Question
I am trying to disable ribbon using office js library. while running below code I am getting this error
ERROR:  Uncaught (in promise): InvalidRibbonDefinition: InvalidRibbonDefinition
RichApi.Error: InvalidRibbonDefinition
Code: Office.ribbon.requestUpdate({       tabs: [         {           id: 'TabCustom',           groups: [             {               id: 'Tab.Group2',               controls: [                 {                   id: 'Dataverse',                   enabled: false,                 },               ],             },           ],         },       ],     });

Hi @AbhishekKatariya9, thank you for submitting. We use Github issues to report bugs with the samples in this repository, and it looks like your question uses code that is not from one of the samples in this repository.
For programming questions like this, we suggest you go to Stack Overflow and see if your question has already been asked or ask a new question. Stack Overflow has an active community of developers that regularly provide feedback and guidance.
Since this isn't a bug, I will close this issue. Let me know if you have any questions!
Alison


Refs #31300 -- Add function-based virtual fields

As proposed in this forum, this forum and this ticket. Kudos to Paolo for the thorough research on backend support for this feature.
Code samples:
from django.db import models

# Basic usage
class MyModel(models.Model):
    a = models.IntegerField()
    b = models.IntegerField()
    a_plus_b = models.GeneratedField(expression=F("a") + F("b"))

# Option to customize how field is persisted
class MyModel(models.Model):
    name = models.CharField(max_length=10)
    lower_name = models.GeneratedField(
        expression=Lower("name"),
        output_field=TextField(
            db_collation="fr_FR"
        ),
    )

# Option to not store as column in database (not supported in Postgres)
class MyModel(models.Model):
    name = models.CharField(max_length=10)
    lower_name = models.GeneratedField(
        expression=Lower("name", output_field=TextField()), db_persist=False
    )

Implementation notes:

The feature is implemented for Postgres, SQLite, and MySQL/MariaDB
This feature has limitations on MySQL/MariaDB and older versions of SQLite because the generated field value won't be set on save() since INSERT...RETURNING isn't implemented for this DB backend
Postgres does not support GENERATED ALWAYS ... VIRTUAL fields
SQLite doesn't support constants being injected into the CREATE TABLE statement
It should be easy to implement this feature for Oracle, I just didn't bother quite yet since it's tedious to run Oracle locally

Please let me know what you think! This is a tentative first draft, so feedback and ideas are greatly appreciated 😄

Jeremy I'm glad to see your PR based on my proposal.
I'll try to review it in few days.
Have see the comment from Simon in the mailing list about a nulla le base field to use custom expression?
https://groups.google.com/g/django-developers/c/9Mf7YqDA4bg/m/wb07E71oAAAJ

Jeremy I'm glad to see your PR based on my proposal.
I'll try to review it in few days.
Have see the comment from Simon in the mailing list about a nulla le base field to use custom expression? https://groups.google.com/g/django-developers/c/9Mf7YqDA4bg/m/wb07E71oAAAJ

Thanks Paolo, I appreciate it! Yes, I have included Simon's suggestion to have an optional output_field, which by default is calculated using the given expression. I have added usages example to the P/R description accordingly :smile:

Hi @pauloxnet any thoughts on this? Just wanted to check-in 😄

Hey! I've been waiting for this functionality for months, and I've just seen it's being developed right now. Great work!!
I have a question about this feature: Would it support subqueries? Say I'm building a bank, and I take a Snapshot of my Users money daily. This Snapshot has a float money field and a FK to User. Would it be possible to add a new computed field delta with the value F("money") - Snapshot.objects.filter(user=F("user")).order_by("-created_date").first().money?
Thanks!

Hi @rmortes, thanks for the interest in this P/R! Sadly I suspect your use case will indeed not work. As explained in Postgres' docs, the generation expression can only use immutable functions and cannot use subqueries or reference anything other than the current row in any way. I'm not sure of the exact limitations for other DBs, but I suspect they are similar to Postgres.

Hi @pauloxnet any thoughts on this? Just wanted to check-in smile

Sorry for the late reply. I'll try to take a proper look to this PR before the weekend.

@pauloxnet (or anyone else) are you willing to review this?

Also I would prefer we used the name GeneratedField, sticking with the SQL syntax, as previously discussed on the mailing list. I don’t see why we would copy SQL Alchemy, and “computed” isn’t any clearer IMO.
And I would like to say thanks for the work so far!

Thank you for the helpful feedback @pauloxnet and @adamchainz! I believe I have addressed all your comments - please review once more and let me know if I missed anything.

As I wrote before the change looks good to me, I just think it's missing a section added in the model fields documentation, together with other fields.

Thanks @pauloxnet, missed that part. I have added field documentation now, let me know if there's any other sections I missed.

Thanks @pauloxnet, missed that part. I have added field documentation now, let me know if there's any other sections I missed.

Thanks.
I've unset the "Patch needs improvement" flag from the ticket.
Now we await the reviews of other Django developers.

... Support for virtual fields on Oracle is missing.

Reiterating my very little knowledge of Oracle, I had investigated a year ago there seemed to be support for "Virtual Columns" from Oracle Database 11g Release 1
https://oracle-base.com/articles/11g/virtual-columns-11gr1

Thanks all for the feedback! I believe I implemented addressed all your comments.
I haven't implemented Oracle because I haven't found any way to run it locally. Any tips? I'd be happy to implement the features, but I currently have no way of verifying the changes work as expected.

하늘2023. 3. 29. 12:00에 Jeremy Nauta @.***>님이 작성:
Thanks all for the feedback! I believe I implemented addressed all your comments.
I haven't implemented Oracle because I haven't found any way to run it locally. Any tips? I'd be happy to implement the features, but I currently have no way of verifying the changes work as expected.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

하늘2023. 3. 29. 12:44에 이 포도 @.>님이 작성:하늘2023. 3. 29. 12:00에 Jeremy Nauta @.>님이 작성:
Thanks all for the feedback! I believe I implemented addressed all your comments.
I haven't implemented Oracle because I haven't found any way to run it locally. Any tips? I'd be happy to implement the features, but I currently have no way of verifying the changes work as expected.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

하늘2023. 3. 29. 12:45에 이 포도 @.>님이 작성:하늘2023. 3. 29. 12:44에 이 포도 @.>님이 작성:하늘2023. 3. 29. 12:00에 Jeremy Nauta @.***>님이 작성:
Thanks all for the feedback! I believe I implemented addressed all your comments.
I haven't implemented Oracle because I haven't found any way to run it locally. Any tips? I'd be happy to implement the features, but I currently have no way of verifying the changes work as expected.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

@felixxm @pauloxnet @adamchainz any comments on this P/R, or tips for implementing Oracle?

@felixxm @pauloxnet @adamchainz any comments on this P/R, or tips for implementing Oracle?

LGTM
I've no experience with Oracle , so I can't help you on that point :-(

Personally I'm fine with us leaving out Oracle support for new features like this, if it's troublesome. I am not sure we should be chasing all of its quirks for all features at this point. Only 2% of survey respondents use it.
That said, it may be worth a try. The django-docker-box project helps you run multiple database backends - it has instructions for setting up Oracle. Try that if you have the time and energy.
Otherwise @felixxm loves Oracle (😜) and may be able to help.

Personally I'm fine with us leaving out Oracle support for new features like this, if it's troublesome.

I agree with you.

I attempted to use django-docker-box but ran into issue after issue. Seems to be a known issue. I got close to running it using truevoly/oracle-12c, only to realize that Oracle 19 is required.
Unless anyone can help implement this feature in Oracle, I'm going to consider this P/R ready to merge!
FYI @felixxm

Thanks for the info @LilyFoote! Interested in working together to implement Oracle support? If you're willing to review this P/R and QA it for Oracle, then I'd be happy to take a "best guess" at what the implementation would be. How does that sound?

@jpnauta Actually, I should have a PR with a few commits for you soon adding a first draft of Oracle support.

Hero!

Thanks @LilyFoote for implementing Oracle support! I have merged the latest from main now, I believe this is ready to go.
@felixxm @pauloxnet are we able to get this P/R merged so we can move forward towards Lily's P/R?

This PR LGTM

@jpnauta can you rebase this PR so that can be easier to review for others?

@pauloxnet done! FYI @LilyFoote you will need to rebase as well 😄

@jpnauta rebased.

I'm pretty late to the party, but I only just came across this today.

a_plus_b = models.GeneratedField(expression=F("a") + F("b"))

Will this field update within the context of a model instance if the values of a and/or b are changed?

I'm pretty late to the party, but I only just came across this today.

a_plus_b = models.GeneratedField(expression=F("a") + F("b"))

Will this field update within the context of a model instance if the values of a and/or b are changed?

Yes @schinckel, there are a few tests in this P/R that do exactly that! Regardless of when/how the model's fields are changed, the generated field will always be a plus b.

Thanks for the reply. I've looked at the tests, and it seems to me that a refresh from the database is required to trigger the value to update.
(I'm asking because I implemented https://django-shared-property.readthedocs.io/en/latest/ specifically to handle hybrid properties. I'm going to look deeper into how you implemented this to see if I can make live-updating-properties work on top of this PR).

Yeah, that's not quite what I meant.
This was more about an update to a dependent column affecting the generated column's value without having any database access required.

@schinckel gotcha - in that case, no, this feature requires a DB update for the feature to work

Coolio. I think I can still make my stuff (which already works) build on this PR when it's merged anyway. I implemented allowing related objects' fields as part of the interface, but I think it's probably "better" to restrict to only local fields.

Closing in favor or #16860.


odd bugs

$ switch --config-directory ~/.kube
Error: unknown flag: --config-path

and README says to use --kubeconfig-directory.. but thats not listed in the --help output ?

Hi, thanks for reporting. The READ.ME was outdated.
The flag is called --kubeconfig-path because it can be either in vault or on the local filesystem.
The name --config-directory is misleading, it should be set to the config file not the directory. (default "~/.kube/switch-config.yaml").  I will adjust the naming soon and put that into a new release.

Please bear in mind that setting the config file via is not required atm. It is only optional to configure the kubeconfigRediscoveryInterval for the kubeconfig index, and to configure hooks. So the chance is that you do not even need it.

Closing this for now. Please reopen if it persists.


Pinpoint can only display the first request of the project

After installing Pinpoint 1.7.3 and deploying a test project,  I visit its addresses,  such as
http://10.100.xxx.xxx:8080/test and  http://10.100.xxx.xxx:8080, but pinpoint-web can only display first request, others after that can not be monitored.
Only if I restart my test project,  pinpoint-web will display another one point.

Hope to get help, thanks, and have a nice day.

Environment:
Pinpoint 1.7.3
hbase 1.2.6
jdk 8u171_linux_x64
CentOS 7.0

please check this issue #3802


When using the atom command I get Segmentation fault (core dumped)

Recently I reinstalled ubuntu 16.04 on my laptop as well as atom and the platformio-atom-ide-terminal package since they are super awesome. I was disappointed to discover that when I used the atom command from the platformio-atom-ide-terminal terminal I got a segmentation fault (core dumped) message. I thought this was strange since I had used this package before with no issues. I originally thought that atom itself had the issue but I confirmed that I can run the command fine from the regular terminal, it only produces the error when I run it from platformio-atom-ide-terminal. I know it should work since I used it all the time before. Any help would be appreciated. Feel free to ask me for any additional information that I should provide, I'm still fairly new at using Atom.

same issues

@ivankravets please fix  it，thanks

set this environment variable can solve this problem.

Please retest with platformio-atom-ide-terminal 2.9.0 all similar reports have been fixed.


Dead link.

Second point in the pre-requisites points to bad links. They are the ones with the samples. Can we have them updated please?

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

ID: 73b78c1c-bd0f-caec-32ad-47e6dfcc5b34
Version Independent ID: 00957482-fbae-de4e-6c9c-49dae66c74af
Content: Add natural language understanding to your bot - Bot Service
Content Source: articles/v4sdk/bot-builder-howto-v4-luis.md
Service: bot-service
Sub-service: cognitive-services
GitHub Login: @ivorb
Microsoft Alias: v-ivorb

@kaushalgautam , Fixed. Thanks for point this out.


Helper Class Missing

$this->Csv->export($filePath, $data);
The above line shows error app/View/Heper/CSVHelper class missing. The file is getting created but not auto-downloading
I am using cakephp 2.x

You're going to have to show more code because I don't know why you're getting this error. I don't see any references to a helper in the code or documentation.


Add gpt-4o-mini

Add the ability to select the gpt-4o-mini llm in the top right corner.
Pricing: $0.15/$0.60.

Tenten turned this issue into this commit: https://github.com/kurama3322/ai-chatbot/commit/00ef510101437f55cb836b59e986cd695cfbc2b8


Be truly immediate/non-blocking more often

Builds on #480, really quite small once that is merged, tidies up many of the "immediate"-related annoyances.
/cc @aledsage @grkvlt @neykov @sjcorbett

@ahgittin can you rebase this against master please, now that #480 is merged

@aledsage rebased and ready for review

@aledsage Sorry for the confusion.  Thanks for long exposition.  I thought almost everyone hated the "wait" as part of "immediate", from discussions ages ago somewhere.  Waiting 200ms is smelly.  And it hurts us writing deterministic tests, and causes nasty delays when we validate (e.g. add 1000 unresolved dependents as config keys and startup will be 3m+ while it waits for them to "validate"!).  Surprised that you do like it as usually it's me being loosy goosy!
Immediate eval was not guaranteed before in any circumstances (anything can take >200ms or whatever the threshhold was) so it is a strict improvement in contract that now we do guarantee immediate eval for non-blocking code.
Yes some blocking tasks which previously succeeded in practice for immediate eval now will not.  However I think there are very few of these.  None of the old tests failed which gave confidence for this change (much of which was done in #565 hence the brevity here).
I'm investigating your new test but to me sleeping (for any length) while someone is waiting on your "immediate" value shouldn't be acceptable.  If the caller does want short-timeout behaviour that codepath is still available (and used by immediate in a few cases, e.g. shared tasks as config value).  (It might be appropriate when the REST API is doing get-all-config for instance, possibly as an API argument?)
As for external(...) is it really the case that such refs now all fail?  Or is it just those that require blocking I/O?  Either way I think we rewrite any bits that are important (local file access?  though hardly necessary) and accept that immediate isn't supported for others.  (Haven't looked at this yet -- wanted to reply conceptually before looking at code.)
I think there are some places that a suprising InterruptedException might be thrown (ISTR latches check this even when not blocking) but we can live with this.

@aledsage I think I better understand now.  It isn't the sleeps that are the offenders -- it is the DST/map interplay.  Still getting my head round it but wanted to say that.

To add some more context (without having looked at the changes) - before adding ImmediateSupplier tasks with a short timeout (the way we implemented "immediate") would get cancelled even before the thread had the chance to get scheduled. So it's not all about whether the thread itself blocks for more than what we are ready to wait for.
I think it's fine for ImmediateSupplier to take longer than ~200ms. That's just a random limit we chose. It's easy to get over it on systems under load when threads are involved - it was easily reproducible with just the clocker blueprints. What matters is it knows it will finish in a sensible amount of time and is not blocked  indefinitely by tasks it doesn't have control over.

@neykov you beat me to replying - agreed :-)
@ahgittin to give some more background, the original use-case for "immediate" stemmed from a failure in clocker: https://github.com/brooklyncentral/clocker/blob/v2.1.0/swarm/catalog/swarm/swarm.bom#L260-L270
In a Transformer enricher, the target value is evaluated each time the "trigger sensor" changes, but its value may not yet be resolvable. It uses the idea of "immediately" to avoid leaving a bunch of threads running - see https://github.com/apache/brooklyn-server/blob/rel/apache-brooklyn-0.10.0/core/src/main/java/org/apache/brooklyn/enricher/stock/Transformer.java#L81-L98. Originally this was purely time-based (cancel after 200ms).
However, sometimes on a heavily loaded machine, the "immediately" would time out after 200ms even though the value really was available. The trigger sensor would never change again, so the target sensor would never be set.
By adding ImmediateSupplier we were able to call something that would only return "absent" if it really could not be resolved yet.
It is that style of use-case / usage that makes me want to let $brooklyn:external do IO operations (rather than aborting them by interrupting the thread).
I believe this model would be simpler for a user to understand, rather than explaining to tell them that their config resolution will fail in certain situations, such as the Transformer, if it requires IO operations (e.g. if it tries to contact vault, or read from a file, etc).
Longer term, I definitely want to get rid of the ugly/brittle "wait for up to 500ms", but I'd prefer we leave that in for a while longer rather than remove support for the above use-case.

Thanks for the explanation.  I realized from your (excellent) tests that the thing we want to allow is to permit an ImmediateSupplier to block, which was permitted in my code if it was a simple key but not if it was deeper eg in a map.  I think we agree that a "dumb" non-immediate Supplier should fail if it tries to block, but we should be consistent wrt blocking behaviour in getImmediate, and I agree we should allow it (if someone writes a bad getImmediate that's on their head, GIGO).
I think we can do this by clearing an interrupt flag in BasicExecutionContext.getImmediately(...) if it is executing an ImmediateSupplier.  Does that sound reasonable?
I think we can't avoid the fact that immediate behaviour is complex.  However we can say something like:

Simplistically, immediate / non-blocking evaluation uses heuristics under the covers to make best-effort without blocking to find a value.  This applies among other places to the validation we run during initialization:  validation will not be performed where the value isn't yet available.  To permit some types of very-brief blocking operations (I/O) or any action sensitive to thread interrupts (latch checking), the action should expose an ImmediateSupplier interface.  If a caller needs fine-grained control over resolution timings, use ValueResolver.

(Should we add this to the docs?)

All tests pass for me locally with this.  And re-reading I think @aledsage we're not in disagreement; I simply overlooked the subtlety of getImmediate having explicit permission to block.  Very good catch.

Failure at https://builds.apache.org/job/brooklyn-server-pull-requests/1820/ seems due to build environment --
[INFO] Installing /home/jenkins/jenkins-slave/workspace/brooklyn-server-pull-requests/camp/camp-brooklyn/target/brooklyn-camp-0.11.0-SNAPSHOT-test-sources.jar to /home/jenkins/jenkins-slave/maven-repositories/0/org/apache/brooklyn/brooklyn-camp/0.11.0-SNAPSHOT/brooklyn-camp-0.11.0-SNAPSHOT-test-sources.jar
Build timed out (after 60 minutes). Marking the build as aborted.
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: java.lang.InterruptedException
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: java.util.concurrent.ExecutionException: Invalid object ID 36 iota=46

Doing git commit --amend to force rebuild.

This time error is
testDisablingOnFire(org.apache.brooklyn.policy.ha.ServiceFailureDetectorTest)  Time elapsed: 0.016 sec  <<< FAILURE!
java.lang.AssertionError: expected [running] but found [on-fire]
	at org.apache.brooklyn.policy.ha.ServiceFailureDetectorTest.testDisablingOnFire(ServiceFailureDetectorTest.java:223)

... unrelated non-deterministic ?

@neykov I've added a comment in ValueResolver re your immediate flip
                // TODO svet suggested at https://github.com/apache/brooklyn-server/pull/565#pullrequestreview-27124074
                // that we might flip the immediately bit if interrupted -- or maybe instead (alex's idea)
                // enter this block
                // if (allowImmediateExecution && (isEvaluatingImmediately() || Thread.isInterrupted())
                // -- feels right, and would help with some recursive immediate values but no compelling
                // use case yet and needs some deep thought which we're deferring for now

does that cover your thinking?
if not -- and/or if you're sure of the Object -> key.getType() change (your other comment) -- just open a new PR.  going to merge this as i can't figure out how to unstuck the build.  will babysit the official build.


Navbar responsive style improved

Related Issue

When we open the navbar on mobile, it covers the whole space. I fixed it in such a way that it will consume only the left half portion while making it translucent. (#331)

In comparison to the old white navbar which occupies a whole space, this looks much better (shown below).
Output Screenshots
|

Old Navbar screenshot

hey, the screenshot you shared on issue is different from the output,
please make the navbar without any scrollbar and it should be ending at the end not in the middle


Switching phones deletes two weeks worth of contact tracing

Avoid duplicates

[ ] Bug is not mentioned in the FAQ

The problem is briefly touched here but without mentioning the larger implications

[x] Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[x] Bug is not already reported in another issue

Describe the bug
Neither backup nor transferal to a new phone is possible. This means that if you had a Covid-19 encounter in the past 14 days before you switched to a new phone, the warnings will go unnoticed.
As a result, switching phones during the pandemic puts you at risk. It also puts your previous contacts at risk if you have a positive test result but the app cannot notify them anymore.
Expected behaviour
If a backup of IDs is technically not possible, at least a data transferal between the two phones should be possible, e.g. via Bluetooth. During this procedure, the "ownership" of the random IDs shall be transferred from the old to the new phone.
However, a true backup (if technically possible) would be the best solution as it also deals with the problem of lost / stolen devices.
Steps to reproduce the issue

Have a phone with the corona app installed
Setup a new phone
Install the corona warn app
now there is no way to retrieve the previous 14 days of random IDs for the newly installed app.

Technical details

Mobile device: all
Android version: all

Possible Fix
Add a button to the app which allows transferal of ids to a different phone, e.g. via bluetooth.
Additional context
--

Sorry that we forgot to mention that we already have wishlist item #8 which addresses this feature request. As a consequence this one can be closed.
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team


Wrong namespaces

I think your project was created using mongodb samples.

I think so yes, as litedb and mongo api are much the same


Signature Files are Treated as Implementation files

Steps to reproduce

Create a file named SigTest.fsi with the following implmementation:

namespace SigTest

open System

type Number

module SigTest =
  val add : int -> int -> int

Notice the errors on type Number and val add : int -> int -> int.

Additional info
While the steps above use a stand-alone file, the same behavior occurs if opening a full project.

making a git repo, committing it once you've achieved your error state, and archiving it with git archive -o myBugSituation.zip HEAD and dropping the zip in your comment is the best way to provide a repro

Can reproduce on Suave repo.

Should be fixed (at least works with Suave right now) in 2.24.0

Can confirm. This looks to be fixed now. Thanks!

🎉


implement /network/peer_count endpoint

lighthouse: https://lighthouse-book.sigmaprime.io/http_network.html#networkpeer_count
note: pretty sure the associated table has an incorrect route, but i'm confirming that.
this should be pretty simple, its basically just returning a count of the peers rather than a list of the peers in /network/peers, and just as a numeric result.

it's been confirmed that it's definitely /network/peer_count, and lighthouse raised a doc issue to fix the table.
https://github.com/sigp/lighthouse/issues/902


[Scheduling Sample] Patient profile page (details & resources)

Patient details (tab, middle ⅓)
Appointment history (tab, middle ⅓)
Encounter history (tab, middle ⅓)

@filipeximenes Slight edit:

"Appointments" tab should only show upcoming appointments (reason is that past appointments should be translated as "Encounters"


Reset link after opening files through URL?

Right now, the URL parameters to open a file persist even after the file was opened. That means the file is opened again if, for some reason, the page is loaded again. Since libertem-server retains the state between page loads this would open the dataset multiple times.
Wouldn't it be better if the parameters are reset after a dataset is loaded successfully?

https://github.com/LiberTEM/LiberTEM/pull/1518#issuecomment-1774918683
Discussed during the PR, left as-is because the behaviour was intentional.
Personally I think I lean more on the side of resetting the URL than not doing so.


Tilt animation in Swift?

Hey,
nice view, but in Swift the kBAFluidViewCMMotionUpdate doesn't work. Can you help me?
    private func addFluidIndicator() {
        if self.motionManager.deviceMotionAvailable
        {
            self.motionManager.deviceMotionUpdateInterval = 0.3
            self.motionManager.startDeviceMotionUpdatesToQueue(NSOperationQueue.mainQueue(), withHandler: { (deviceManager, error) -> Void in

                guard let deviceManager = deviceManager else { return }

                let nc = NSNotificationCenter.defaultCenter()
                let userInfo = NSDictionary(object: deviceManager, forKey: "data")
                nc.postNotificationName(kBAFluidViewCMMotionUpdate, object: self, userInfo: userInfo as [NSObject : AnyObject])
            })
        }

        let fluidView           = BAFluidView(frame: UIScreen.mainScreen().bounds, startElevation: 0.5)
        fluidView.fillColor     = UIColor.blueColor()
        fluidView.strokeColor   = UIColor.whiteColor()
        fluidView.keepStationary()
        fluidView.startAnimation()
        fluidView.startTiltAnimation()
        self.view.addSubview(fluidView)
    }

It's a timing problem. I solve this issue.

Hey @AFcgi - I gave it a quick glance over the weekend, but couldn't figure it out. Glad you found a solution though!


[YAML] Add code climate and improve test coverage

Parent: [EPIC: YAML] Add YAML & CLI. "v0.2" of Predictoor #400

Covered by linked PRs and turned off strict code climate quality checks