Pure PHP BigInteger support through phpseclib/bcmath_compat

This PR implements a pure PHP fallback when neither the GMP nor the BCmath extensions are installed. It simply uses phpseclib/bcmath_compat to provide a pure-PHP bcmath implementation. Essentially, I'm addressing the TODO item in FG\Utility\BigInteger.
When either the GMP or BCmath extensions are loaded they are of course still being used which is admittedly much faster, by more than an order of magnitude.
All tests pass with this PR. I have also tested this in a real world context (WebAuthn) successfully.
My motivation for this PR is that if it's included and a new version is released I can update the WebAuthn implementation's dependencies in the Joomla 4 CMS to no longer require GMP or BCmatch.
Thank you in advance for your time and thank you very much for the effort you've put into this library! It's very much appreciated :)

Hey @nikosdion thanks for the contribution :)
Generally the change makes sense to me. However, given we are adding a new mandatory dependency to PHPASN1, I would like to have a second pair of eyes that is still more active in the PHP ecosystem just as a sanity check.
@afk11 , any concerns with pulling in phpseclib/bcmath_compat here?

Hi all! Thanks for the contribution @nikosdion and @fgrosse for the ping :)
I've got another project that uses PHPASN1, it's tests all passed with this PR which is great.
function + invocation count in phpecc full tests
 337700 bcadd
 130065 bccomp
    301 bcdiv
    319 bcmod
 337700 bcmul

function + invocation count in phpasn1 tests
      8 bcadd
     35 bccomp
      2 bcdiv
      2 bcmod
      4 bcmul
      6 bcpow
     10 bcsub

I do wonder if we should just change extension_loaded('bcmath') to function_exists('badd') (maybe test a few functions as well to be sure :) ) in the BigInteger implementation detection so the library is more 'polyfill aware'.. Because if looks like the polyfill pulls in all phpseclib to do it's job, which may cause conflicts between PHPASN1 and other software, and adds to download times.
https://github.com/phpseclib/bcmath_compat/blob/89cbb63742a32730b7187773a60b6b12b9db4479/composer.json#L26

I can update the WebAuthn implementation's dependencies in the Joomla 4 CMS to no longer require GMP or BCmatch.

It seems if we made the library polyfill aware, you could add bcmath_compat to the WebAuthn implementations dependencies and remove the GMP / bcmath requirement?
I'd see this as a better outcome because not every one needs the polyfill + it's dependencies, but we don't want people that use earlier versions of phpseclib to suddenly be unable to use PHPASN1

Just scanned over phpseclib, it doesn't pull in many dependencies - the only conflict issue would be someone using phpseclib <2.0.19 in their project, which is declining https://packagist.org/packages/phpseclib/phpseclib/stats#2.0.19 the 1.0 releases are also relatively low in usage
Maybe it's not the worst thing to just add it here? @fgrosse what do you think?

@afk11 Thank you for the input! I'd much rather have PHPASN1 be polyfill-aware than polyfill-dependent. Not so much because of dependency conflicts but because it doesn't make sense to pull in unnecessary dependencies.
I think there is a way to cause minimal disruption. I could put the function_exists('bcadd') check as the third check in the if-block (after the two extension_loaded) and demote the phpseclib/bcmath_compat as a suggests in composer.json. Of course I'd need to reinstate the fallback runtime exception in case no big integer math support is available.
If you and @fgrosse agree I could update the PR today.

Cool, that sounds perfect to me, and looks like Friedrich is happy with it as well

All right! I updated the PR per the discussion above and I fixed the errant tabs in my changed lines. Thank you in advance for your review.


Fonts are not working on UWP App..

Created a very simple app but the icons are not working on windows app.  I used your Page1.xaml and App.xaml.cs and the project compiles fine without errors but only shows [] where the icons should be.  It runs fine on Android.  I must be missing something...
Thanks in advance.

Hi,
Copy iconize-fontawesome.ttf to resource folder, in iOS app work perfectly ;)

Still doesn't work for me on UWP

I have this problem when run on Relase mode. On Debub mode works, but Release not work

linck I have the same issue, works with Debug, but not release.  did you ever resolve?  i'm about to dive into the code and try to figure it out.

The UWP specific overload to load the plugin assemblies worked for me.


Missing link to licensing doc

In the Webinar section https://learn.microsoft.com/en-us/microsoftteams/quick-start-meetings-live-events#webinars, there is a placeholder where the link to the licensing doc should be.

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

ID: 92fcb1f6-5cc2-f6bd-cfa9-11d811f1dbf1
Version Independent ID: 9a571e6d-8cb3-9ff8-a5c2-62deab146264
Content: Meetings, webinars, and live events - Microsoft Teams
Content Source: Teams/quick-start-meetings-live-events.md
Service: msteams
Sub-service: meetings
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

This was fixed.


fix: set known good image tag to fix yq

It has been discovered that the RC008 review job does not work with versions of YQ v4.29.1 or higher. Pinning the review job tag to the last known good tag for cimg/base which contains YQ version 4.27.5.

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/circleci/orb-tools?version=dev:c14c72a5d7ec98ac0d868b551ebbb64966cb2c03

Your orb has been published to the CircleCI Orb Registry.
You can view your published orb on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/circleci/orb-tools?version=11.5.1


Add OpenEBS provisioner as a intree provisioner in k8s

Is this a BUG REPORT or FEATURE REQUEST?
FEATURE REQUEST

What happened:

OpenEBS Provisioner is not intree provisioner in kubernetes.

What you expected to happen:

OpenEBS Provisioner shall be intree provisioner in kubernetes.

The general direction of K8s is to move towards CSI and away from in-tree.


Add ability to set NODE_ENV for npm_package

Description

Adds the ability to provide NODE_ENV to the npm install process

Important to note that setting --production overrides NODE_ENV

Issues Resolved

n/a

Check List

[x] All tests pass.
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable

Frustratingly, the test failures appear to be thanks to a Chef deprecation, not anything in this body of work.

In an effort to demonstrate that tests do, in fact, pass, I temporarily pushed up a commit that disables the deprecations_as_errors Kitchen config as well as the two Foodcritic rules that check for build_essential, as we still support Chef 13.
You can see those tests all pass in that particular build, though I‘ve since dropped the commit to limit the scope of this PR.

@tas50 this one’s pretty vital for us (so much so, we shipped our own v5.1.0, as you can see in darkskyapp/nodejs@624b094). Any chance of merging it in & shipping v6.1.0?
It’s handy for anyone doing stuff with node-config or similar environment-driven actions.

Try to get this merged.

@MarkGibbons oh wow, this has been hanging around for a while! So yeah, if someone will approve it, we should be all set, it’ll still merge cleanly.


what's the fking problem?

i am building for arm64,using command this
GOOS=linux GOARCH=arm64 go build -o build/goflyway cmd/goflyway/main.go

请阅读readme，并了解一下:https://stackoverflow.com/questions/32758235/how-to-get-all-dependency-files-for-a-program-using-golang


Could not kill browser: Cannot read private member #process from an object whose class did not declare it

Which package is this bug report for? If unsure which one to select, leave blank
@crawlee/browser-pool
Issue description
When using Puppeteer, I found that I cannot kill browsers. So if I execute browserController.close();, I get the following error:
DEBUG BrowserPool: Could not kill browser.
Cause: Cannot read private member #process from an object whose class did not declare it {"id":"Pg7SSIOKwAaACRRyxSMQZ"}

I have investigated a bit and found the problem: in PuppeteerController._kill(), the following code throws an error:
const browserProcess = this.browser.process();

The process() method is not working properly because this.browser is actually a proxy, and the process() method is not bound to the real object.
A quick fix is to add 'process' in the boundMethods just before initializing the proxy:
class PuppeteerPlugin {
    _launch() {
        // ...

        // Added 'process' at the end of the array
        const boundMethods = (['newPage', 'close', 'userAgent', 'createIncognitoBrowserContext', 'version', 'on', 'process'] as const)
            .reduce((map, method) => {
                map[method] = browser[method]?.bind(browser);
                return map;
            }, {} as Dictionary);

        // ...
    }
}

Code sample
browserController.close();

Package version
3.7.1
Node.js version
v21.5.0
Operating system
Mac OS (Darwin xxx.local 22.5.0 Darwin Kernel Version 22.5.0: Mon Apr 24 20:52:24 PDT 2023; root:xnu-8796.121.2~5/RELEASE_ARM64_T6000 arm64)
Apify platform

[ ] Tick me if you encountered this issue on the Apify platform

I have tested this on the next release
No response
Other context
I can submit a pull request if you like my fix. I'm also available on Discord (username "marcplouhinec").

A quick fix is to add 'process' in the boundMethods just before initializing the proxy:

Makes sense, PR welcome for that.

@B4nan Please find the related pull request: https://github.com/apify/crawlee/pull/2329


Installed Rancher in an RKE cluster, getting connection refused and cert-manager cannot complete challenge

What kind of request is this (question/bug/enhancement/feature request):
Not sure if it's a bug.
Steps to reproduce (least amount of steps as possible):
I deployed an RKE cluster I want to use with Rancher in HA mode. Then I installed cert-manager and Rancher with helm just fine. No errors or anything like that. However cert-manager cannot complete the challenge because requests to the challenge URL return a connection refused. I get the same with curl. One thing I am wondering is that in the ingress-nginx namespace I see only one service for the default backend. Shouldn't there be another service there? I tried removing and readding the ingress controller with RKE but it's still the same. Requests to the IP of the node on port 80 work as expected with the default backend.
Result:
Cannot connect to ingress controller on port 80 and cert manager cannot complete the challenge to issue the certificate, so I cannot open Rancher.
Other details that may be helpful:
Not sure if it may be relevant but I am using Weave for the encryption.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
2.4.5
Installation option (single install/HA):
HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Custom
Machine type (cloud/VM/metal) and specifications (CPU/memory):
VPS, 2 cores, 8GB of ram
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.0", GitCommit:"9e991415386e4cf155a24b1da15becaa390438d8", GitTreeState:"clean", BuildDate:"2020-03-26T06:16:15Z", GoVersion:"go1.14", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.6", GitCommit:"dff82dc0de47299ab66c83c626e08b245ab19037", GitTreeState:"clean", BuildDate:"2020-07-15T16:51:04Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

19.03.12

Can you supply additional info?

What RKE version did you use to create the cluster?
Are the nginx-ingress-controller pods running? (https://rancher.com/docs/rancher/v2.x/en/troubleshooting/kubernetes-resources/#ingress-controller)
What is the response when you curl the IP on port 80? (curl -v)
What is the response when you curl the FQDN? (curl -v)
What does the FQDN resolve to?

@superseb I forgot to close this. Ehm.... this is embarrassing but I had set the wrong IP in the DNS record :p


Make Average and Safe Low gas available in rpc/contractRPC.go

The DB currently only saves one value for gas, the Fast price. So in rpc/contractRPC.go, there is only the fast gas price available to use for submissions:
gasPrice := getInt(m[db.GasKey])

However, it would be better to have average and safe low prices available so miners could select which one to use. I looked into it but seems like this issue involves a good amount of work on the DB

Add better gas estimation in general.  There should be ways to look at predicted costs (some API's have it)

With the recent refactoring can add a configuration option for the GasTracker module to specify which gas to return when queried


In the installation process, when adding a sample product, an issue appears in the configurable product where the size attribute is undefined.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
Master
Issue Description
After a fresh installation and adding sample products, an issue appears on the shop front for configurable products where the size attribute is undefined. Upon checking the admin end, it was found that the size attribute is not appearing.
Preconditions

Installation must be done properly.
Sample Product must be added while installation.

Steps to reproduce

Go to Admin -> Catalog -> Product -> Check the Configurable product -> Scroll Down for varients
Check the Size -> Undefined.

Expected Result
Size must appear and it must be visble in shop front and admin end.
Actual Result
Currently it was found that the issue is appearing. I am attaching Screenshot for reference.
Admin End -> https://prnt.sc/LlroAhGNH2kE
Shop Front -> https://prnt.sc/7XOQePb7K4CZ
Screenshots
No response
Additional context
No response

Checked and found that the configurable product issue related to size attribute has been fixed and working fine. Please check the video for reference.
Video
https://github.com/user-attachments/assets/7fc0b88a-4e82-4a78-8b6b-1a7893b111bb


fw.routeRedirect

This actually doesn't work:
fw.routeRedirect(FW.ACTION_SHOW, new object[] { id });
arguments is not passed to the controller call

Fixed


iOS / Android support?

Does this library support mobile (ios via native aot and android) ?

I haven't tried it, so if you get to try it please let me know of the results. From the UWP/Unity development I remember that native AOT has issues with reflection/activator and unfortunately, these APIs are being used in several places because the original Java code base was using the same concepts. This was one of the main reasons why I eventually started working on https://github.com/jimm98y/SharpMP4 which is supposed to be AOT-compatible and reflection-free. However, it's still work-in-progress at this moment.


feat(Dropdown): highlight by character key in non-search

Behaves as in aria spec
When open, sending a character key will focus on the item that begins with that key.
If there is already an option highlighted, the search will begin from the next item.
If it reaches the end of the array and does not find any match, it will check from the start (circular movement).
If there is no option starting with that character, it will not do anything.

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS

Fails

:no_entry_sign:
All of your entries in CHANGELOG.md should be in the **Unreleased** section!

  Generated by :no_entry_sign: dangerJS


Tokenizer saved with save_pretrained_merged(save_method = 'lora') does not match its tokenizer_config.json set

Hi~
I trained a LoRA model based on the unsloth/mistral-7b-bnb-4bit model. During the training, I modified the tokenizer's chat_template and eos_token using get_chat_template(). After the training was completed, I saved the model using model.save_pretrained_merged("lora_model", tokenizer, save_method = "lora"), which also saved the tokenizer_config.json. This configuration file contains the modifications to the chat_template and eos_token. However, when I load the model using
model, tokenizer = FastLanguageModel.from_pretrained(
    model_name = "lora_model",
    max_seq_length = max_seq_length,
    dtype = dtype,
    load_in_4bit = load_in_4bit,
    attn_implementation="flash_attention_2",
)

tokenizer.chat_template # => None
tokenizer.eos_token # => </s>

I discovered that the tokenizer still retains the original settings from the unsloth/mistral-7b-bnb-4bit model, without the modified chat_template and eos_token.
By the way, I have already executed the command pip install --upgrade --force-reinstall --no-cache-dir git+https://github.com/unslothai/unsloth.git to install the latest version of unsloth.

@leechehao Oh my! I'll take a look sorry on the issue!

WHOOPS this totally crossed my head - so sorry :( I will check this in the following days - so sorry on the issue!

Extreme apologies - fixed it!! If you're on a local PC, upgrade only Unsloth (no dependency upgrades) with pip install --upgrade --force-reinstall --no-cache-dir git+https://github.com/unslothai/unsloth.git. On Colab / Kaggle no need


Limit revisions on model

Description
This PR gives the ability to remove a specific slice of revision records if they are not included in the count of max_revisions.
Basically, we keep an x number of new records on a specific module, older records get deleted.
Configuration
'max_revisions' => {type},

max_revisions have 3 expected values: false, integer and true.
If something else is given it throws Exception.
false
False is the default value in the config, and if present no actions are taken.
number
If number values are present, this int value is used for every module which has revisions on it (globally).
true
If this bool value is present, the job will look into other config value revisions.
Revisions are an array of key => value items where is key is table name and value is type integer.
If the value is a non-integer, an exception is thrown.
They are used to scope specific modules and make custom configurations for them.
Example:
  'max_revisions' => true,
  'revisions' => [
      'categories' => 5, //keep last 5 records on 'categories'
      'articles' => 15, //keep last 15 records on 'articles'
      'dummymodule' => 1 //keep last record only on 'dummymodule'
  ]

Related Issues
Fixes #1478

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

@ifox Like I have written to you on Discord, the problem here happens when Twill waits for the job to finish before it saves revisions. It's synchronous, but I wanted to make it asynchronous since it takes a visible time period before it returns a response to the user on the front end.
But, if we use database, the core thing won't work out of the box, additional setup will be required from the developer (setting ENV var and migration of jobs table).
Is that desirable for the project? (will need to document it if yes)

Hi @kalle,
Can you double-check the source & destination branch for this PR? I'm having trouble zooming in on the changes here.
I think this idea is a welcome improvement and encapsulating this in a Job seems like the right approach to me. Although, I believe the choice of sync/async should be left to the developper.
You can find an example of this in HandleNesting. It's set to use Laravel's default queue out of the box, which is synchronous and does not need a database table. This can be customized as needed on the repository.
Not saying that this is the best implementation but I do think that worrying about sync/async may be too much too soon for a newcomer, especially someone learning Laravel with Twill.
Cheers,
P.

Hi @pboivin, I will close this PR and reopen it with the source branch 3.x since like ifox recommended me to is to change the target branch, but this additional content is from 2.x on which tests fail since its using some functionality from P8. My mistake.
Regarding this async/sync approach, yes this is something that I should not be worried about, but this could be a new thing in the Twill approach since in the future developers could probably have use of it.
Where I see possible implementation with Twill (workers): scheduled content actions, chores on Twill app, etc
All of the mentioned usages are doable on the Laravel level without the need for Twill, but I'm basing this on the future of Twill since probably like all other CMS Twill will need content scheduling in the core (again my biased opinion).
Regarding HandleNesting job implementation, it's understandable that the task which was made by you is synchronous since that is expected to be resolved before returning a response to the frontend. In my usage it works that way also, but it creates not needed time frame since nothing will change if it's executed in default synchronous mode.
In fact, I followed your example on this to create this PR and your code from discussion #917 and other participants' ideas on the implementation of this job. Thank you for that.
Sorry if I sound off subject sometimes, but my background is mostly JS (nodejs) whose main concern is the execution of the application (async/sync), so I take some fears here. And don't take my intentions wrongfully, I just want to help. 😃
PS. @ifox will squash commits and changes will be clearly visible

Hey @kallefrombosnia, I'm sorry, I think this came out wrong. I did not mean to direct the last paragraph at you! What I meant to say is — From the perspective of someone starting with Twill and Laravel, it should not be necessary to setup an async queue with workers. I believe "sync" is a good default.
Anyhow, this is good work! It makes a lot of sense to introduce Jobs into Twill 👍

Hey @kallefrombosnia, want to move this forward, so I left a few comments.
Would also like to add tests for this, if you feel comfortable doing that have a go at it, otherwise let me know then I continue this ;)

Hey @haringsrob, thanks for the detailed reviews. I find your reviews very informative and logical, but unfortunately, I'm abroad so I'm unable to resolve these issues.
If you really want you can finish it, but when I'm back (next week) I'll surely look into this PR.

Hey @kallefrombosnia, if you feel up for it can you have another try/look at the code to see if I did not miss anything?
After that we can merge it into 3.x

Hey @haringsrob, I have looked into changes and they just represent my initial idea which was in config to model settings and own facade settings.
Great work, I think that's it. 😄


Graph Reads API Prototype

Right now the graph branch of the server doesn't implement the reads API. A prototype implementation of the reads API on the graph branch might be a good hackathon project.
Unfortunately, not knowing all that much about the server internals, I don't really know where one might start.

I think the best approach would probably be to try and rebase the graph branch and bring it up to date. This would require a parallel rebase of the schemas I think, but shouldn't be too hard once changes have been reasonably localised.


Team #2 Review (VnV Report): Initial State Missing in Tests

@samm82
Summary
The VnV Report does not include the initial states for any of the tests. Including the initial states will help readers further understand how the test was performed.

Deadline
April 4
Estimated Work Hours
2 hours

Initial states are listed in the VnvPlan, which you can cross reference with the Id. We didn't add it to the report to make the charts easier to read


Speed up Tokenization by optimizing cast_to_python_objects

I changed how cast_to_python_objects works to make it faster.
It is used to cast numpy/pytorch/tensorflow/pandas objects to python lists, and it works recursively.
To avoid iterating over possibly long lists, it first checks if the first element that is not None has to be casted.
If the first element needs to be casted, then all the elements of the list will be casted, otherwise they'll stay the same.
This trick allows to cast objects that contain tokenizers outputs without iterating over every single token for example.
Speed improvement:
import transformers
import nlp

tok = transformers.BertTokenizerFast.from_pretrained("bert-base-uncased")
txt = ["a " * 512] * 1000
dataset = nlp.Dataset.from_dict({"txt": txt})

# Tokenization using .map is now faster. Previously it was taking 3.5s
%time _ = dataset.map(lambda x: tok(x["txt"]), batched=True, load_from_cache_file=False)
# 450ms

# for comparison
%time _ = tok(txt)
# 280ms

I took your comments into account and added tests for cast_to_python_objects


html.template: Panic when calling an uninitialized template function

What version of Go are you using (go version)?
go1.7 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOOS="darwin"
What did you do?
Parse this template with function "Block"
Research Grants {{range Block "Research Funding History"}} 	{{.Emit "Funding Start Date, Funding End Date, Funding Title" }} 	{{range .SubSection "Funding Sources" }} 		{{.Emit "Program Name, Funding Organization" }} 	{{end}}	 {{end}}
then executing as follows
tm, err := template.New("test").Parse(templ3)
if err != nil {
t.Errorf("Parsing failed: %s", err)
}
err = tm.Execute(os.Stdout, ccv)
if err != nil {
t.Errorf("execution failed: %s", err)
}
What did you expect to see?
An error message that function does not exist
What did you see instead?
anic: runtime error: invalid memory address or nil pointer dereference [recovered]
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0xed5d4]
goroutine 5 [running]:
panic(0x1a21c0, 0xc42000a090)
/Users/salah/go/src/runtime/panic.go:500 +0x1a1
testing.tRunner.func1(0xc420090180)
/Users/salah/go/src/testing/testing.go:579 +0x25d
panic(0x1a21c0, 0xc42000a090)
/Users/salah/go/src/runtime/panic.go:458 +0x243
html/template.(*Template).escape(0x0, 0x0, 0x0)
/Users/salah/go/src/html/template/template.go:79 +0x44
html/template.(*Template).Execute(0x0, 0x2a6840, 0xc420032010, 0x1a5d00, 0xc420018300, 0xc42000b240, 0x0)
/Users/salah/go/src/html/template/template.go:101 +0x2f
salah/CCV/template.TestGoTemplate(0xc420090180)
/Users/salah/Dropbox/code/go/src/salah/CCV/template/template_test.go:119 +0x279
testing.tRunner(0xc420090180, 0x1ec5a0)
/Users/salah/go/src/testing/testing.go:610 +0x81
created by testing.(*T).Run
/Users/salah/go/src/testing/testing.go:646 +0x2ec
exit status 2

@DrGo your snippet at https://play.golang.org/p/bNEDlBZG_b doesn't compile

@odeke-em Sorry, I edited the issue to report the original error on my machine. Sorry I cannot be of more help. I noticed that if I drop the function name as in
{{range "Research Funding History"}}
I get an error instead of panic
output:
template_test.go:124: execution failed: template: test:4:8: executing "test" at <"Research Funding Hi...>: range can't iterate over Research Funding History
Also the code works flawlessly if I added the Funcs calls as in
tm, err := template.New("test").Funcs(template.FuncMap{"Block": ccv.Block}).Parse(templ3)
output:
Research
Grants
    2014/9, 2019/8, Baseline funding to the Vaccine and Drug Evaluation Centre

            Public Health Division, Manitoba Health

PASS
ok      salah/CCV/template      0.006s

so closed?

No, html. Execute should not panic (see original post for details)

Please provide a complete self-contained example that reproduces the problem. You have shown us only incomplete or incorrect programs.

I cannot share my program, but I managed to replicate the error here
https://play.golang.org/p/hmwSb9Ipds

Okay the problem here is that you are ignoring the error returned that actually says the function is undefined, you just print it but you aren't actually handling it.
--- orig.go	2016-11-19 11:41:22.000000000 -0800
+++ handled.go	2016-11-19 11:41:34.000000000 -0800
@@ -3,6 +3,7 @@
 import (
 	"fmt"
 	"html/template"
+	"log"
 	"os"
 )

@@ -34,7 +35,7 @@
 	ccv := &CCV{}
 	tm, err := template.New("test").Funcs(template.FuncMap{"List": ccv.List}).Parse(templ3)
 	if err != nil {
-		fmt.Printf("Parsing failed: %s", err)
+		log.Fatalf("Parsing failed: %s", err)
 	}
 	err = tm.Execute(os.Stdout, ccv)
 	if err != nil {

and particularly when handled the error returned says the function is not defined.
2016/11/19 11:40:18 Parsing failed: template: test:4: function "Block" not defined
exit status 1

I am aware of that.
html.Execute also warns correctly about function not defined, but it still seg faults.

tl;dr:  you are dereferencing a nil template that already failed to parse and complained in the error returned. The panic isn't in text/template.Execute but in trying to use .Execute on a nil object
Long wind
The return signature of template.Parse is (*text.Template, error)
A common pattern for Go code return signatures, is two states of the form:

(zero/nil objects..., non-nil error)
(non-nil..., nil error)

which is a pattern that almost all the functions in the stdlib(except for very few if am not mistaken).
Translating that pattern thereby to template.Parse means users only have two states to check for:

(a non-nil template, nil error)
(nil template, non-nil error)
so implicitly use results without worries of nil pointer deferences, after no error has been returned.

If we had the state you seem to be expecting ie

(non-nil template, non-nil error)
that also calls for an extra state:
(nil template, nil error)
then callers would also have to first check that the template itself isn't nil even when err == nil.

tmpl, err := template.New("gopher").Parse(tmplStr)
if err != nil {
   // Even if nil, you still have to check below
}

if tmpl != nil {
    // Explicitly check if tmpl is non-nil
}

so in total we'd then have 4 states to check for ie

(zero/nil objects..., non-nil error)
(non-nil objects..., nil error)
(non-nil objects..., non-nil error)
(nil objects..., nil error)

Handling the error enables us to implicitly use data in which no errors were returned but also to avoid unnecessary allocations and extra states of errors so we could do
tmpl, err := template.New("gopher").Parse(tmplStr)
if err != nil {
    // Handle it in here, this route is done
    return nil, err
}

// Home free to now use the template

Hope this makes sense.

Got it Emmanuel
Thanks,

Cool, I'll close this issue then.


chore: bump celestia-app to v1.0.0-rc13

Overview
bumps app to v1.0.0-rc13, core to v1.26.2-tm-v0.34.28, and the sdk to v1.18.0-sdk-v0.46.14
per release notes: this is a consensus breaking change that is targetting mocha-4 and arabica-10
Checklist

[x] New and updated code has appropriate documentation
[x] New and updated code has new and/or updated testing
[x] Required CI checks are passing
[x] Visual proof for any user facing features like CLI or documentation updates
[x] Linked issues closed with keywords

Codecov Report

Merging #2646 (e702308) into main (47047f3) will decrease coverage by 0.29%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2646      +/-   ##
==========================================
- Coverage   51.34%   51.05%   -0.29%
==========================================
  Files         158      158
  Lines       10559    10559
==========================================
- Hits         5421     5391      -30
- Misses       4669     4693      +24
- Partials      469      475       +6

see 7 files with indirect coverage changes


Propose codes for signature algorithms

As per #289 this pr proposes to register common signature algorithms used in UCANs.

lgtm; some of the numbering choices seem a little arbitrary, do you want to think a bit more about their placement and spacing before proceeding?

Thanks @rvagg! I think @expede had a reasoning to choose specific codes, but sadly I can't seem to find the thread with an elaboration. Personally I have ok with arbitrary codes as long as we have them & personally would prefer smaller ones.
@expede mind considering @rvagg's feedback here before we proceed ?

As per my comments above in the varsig entry, I'm not really clear on what the purpose of that entry is anymore, nor am I clear on why the byte forms are important for these. It doesn't seem to align with the proposal in #289 where we're doing varints, because the new comments here are about the importance of the bytes rather than the int codes. What am I missing?

nor am I clear on why the byte forms are important for these [...] What am I missing?

@rvagg It makes them easier to remember for the humans using these codes. There's quite a lot of number punning in this list — for example, 0xec for ECDSA and 0xed for EdDSA.

where we're doing varints

My understanding (as is the one in did:key AFAICT) is that the codes in this table can be encoded many ways, including but not limited to varints. The signature use case for a "varsig" would be encoded as a varint, but when looking up in this table we don't use varint, and a memotic is potentially helpful

I'm surprised you're not going to make a claim for a single-byte range for this

Actually backing up — do you mean "why not register a single code in this table, and then have a separate standard for varint subcodes elsewhere?" If so, we had discussed that, but there appear to be ranges in this table for public keys, hashes, etc. But we could be wrong!

You're not really using it as a signal value to say "here follows a varsig", but a catch-all of some kind.

That's a good point. @Gozala I think this was in your original proposal in ucan-ipld — thoughts?

I'm not completely against just counting up from 0xd000,

@rvagg this really isn't an "over my dead body" -- if we're "holding it wrong", we can just update this PR to use 0xd000, 0xd001, 0xd002, etc (or something). You tell me!

@expede @rvagg I have responded in https://github.com/multiformats/multicodec/pull/290#discussion_r981891960 but unless I"m misunderstanding / missing something we just need to decide between

Short codes
Codes that would map more closely with corresponding key codes

I personally think optimizing for size is probably better choice that optimizing for resemblance, that said I'm happy either way. But please let's move this forward as we're about to deploy code using this into production and it would be a lot more painful to change codes afterwards.

@rvagg @gozala to grease the wheels if we decide to switch to short codes, here's #293

Thanks for clarifying, I'm really just wanting to know that you have clarity about how these are going to be used and whether it maps to these entries because it sounded a little like there might have been a disconnect. I'm fine with this PR as long as you both agree and can see the path forward from merging this PR. @Gozala has access to merge and this has been approved. If you want to go with #293 then that's fine too. No objections from me on either.

I've decided to merge this as 1 byte overhead seem negligible. If it becomes a problem we could always changes those codes as those are in draft status anyway


should be iceboxed or updated

[Enter feedback here]
Would love for @?-MS to look this over and see if it is even worth keeping around; after a good hour of setup, I came to the conclusion it is incorrectly dated (at least 5 year old technology), and very hard to set up, let alone follow the "simple" directions. It just doesn't seem to work with VS17, and is anything but "simple" to implement.

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

ID: 42c0fa43-0ca5-9c86-f8c4-4b3eb7176250
Version Independent ID: bb93e3ff-0d52-9e93-c3a8-545b16f03a30
Content: Simple data app with WPF and Entity Framework 6 - Visual Studio (Windows)
Content Source: docs/data-tools/create-a-simple-data-application-with-wpf-and-entity-framework-6.md
Product: visual-studio-windows
Technology: vs-data-tools
GitHub Login: @ghogen
Microsoft Alias: ghogen

@Amondale -- thank you for your feedback.  Please consider these resources:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@ghogen -- please look into this issue.

I considered and tried most online "resources." But there must be some committee or oversight 'entity' @MSFT who decides when to sunset some tutorials or walks-through, right? This article is a time waster, very poor learning experience, and should be retired.

@Amondale -- thank you for your followup.

There's a product issue with this article, but it is scheduled to be fixed in a future VS update.


Task failed with unicode characters in /etc/environment on target host

Which version of Ansible are you running? 2.7.7

Are you running with any custom modules, or module_utils loaded? mitogen 0.2.7

Have you tried the latest master version from Git? We'll try 0.2.6 and 0.2.7

Ansible server : debian 9.6
Target host : debian 9.8
python 2.7 is used on ansible server and target host.
CallError: exceptions.UnicodeEncodeError: 'ascii' codec can't encode character u'\xe4' in position 25: ordinal not in range(128)
File "", line 3099, in _dispatch_one
File "", line 3086, in _parse_request
File "", line 596, in import_module
File "", line 1357, in load_module
File "", line 1, in
File "master:/data/mitogen-0.2.7/ansible_mitogen/target.py", line 85, in
import ansible_mitogen.runner
File "", line 1357, in load_module
File "", line 1, in
File "master:/data/mitogen-0.2.7/ansible_mitogen/runner.py", line 219, in
_etc_env_watcher = EnvironmentFileWatcher('/etc/environment')
File "master:/data/mitogen-0.2.7/ansible_mitogen/runner.py", line 134, in init
self._keys = [key for key, value in self._load()
File "master:/data/mitogen-0.2.7/ansible_mitogen/runner.py", line 151, in _load
return list(self._parse(fp))
File "master:/data/mitogen-0.2.7/ansible_mitogen/runner.py", line 163, in _parse
bits = shlex_split(line, comments=True)
File "master:/data/mitogen-0.2.7/ansible_mitogen/runner.py", line 112, in shlex_split
for token in shlex.split(str(s), comments=comments)]
With non unicode characters in /etc/environment its working well

This is now on the master branch and will make it into the next release. To be updated when a new release is made, subscribe to https://www.freelists.org/list/mitogen-announce
Thanks for reporting this!


[Gally]: master <- dev

Automatically created by Git-Ally

:tada: This PR is included in version 1.10.140 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Unable to create the guest account [3.4.3]

Checklist

[X] I am able to reproduce the bug with the latest version.
[X] I made sure that there are no existing issues - open or closed - which I could contribute my information to.
[X] I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
[X] This issue contains only one bug, that only occurs in Squawker

Affected version
3.4.3
Steps to reproduce the bug

Go to Squawker
Try to load any feed
Gives 'Unable to create guest account' error

Expected behavior
It should load the feed properly
Actual behavior
Error happens with unable to create guest account
Screenshots/Screen recordings

Logs
No response
Affected Android/Custom ROM version
Android 13 / One UI 5.1
Affected device model
Samsung Galaxy S22+
Additional information
No response

Forgot to upload this

Are you with a VPN?
If so then:

close Squawker
disable the VPN
open Squawker and get your first tweets
only after you can enable again the VPN

As you know I don't have VPN and I'm getting this

Are you with a VPN? If so then:
* close Squawker

* disable the VPN

* open Squawker and get your first tweets

* only after you can enable again the VPN

Disabled VPN, and it works flawlessly!

Not using VPN and still getting the same error on version 3.4.3

Same for me

No VPN using


new design: data provider factory

trainers may instead take data provider factories as constructor arguments instead of data providers directly.
there may be different data provider factories that may or may not reuse datasets.

done with: https://github.com/dyigitpolat/mimarsinan/commit/5aee83114a031c6f5e545811baf0e191786434ac


Golang hello-world plugin

Hi, I'm trying to follow the documentation and setup the example go plugin. KrakenD gets up, I can access the flask server behind it, however as far as I can tell, the plugin is working (I don't get the Println output from init function). I compiled the plugin as documented (had to make this change to the code though: https://github.com/getsentry/raven-go/issues/182). I'm placing the .so file next to krakend.json.
krakend.json
{
  "version": 2,
  "extra_config": {
    "github_com/devopsfaith/krakend/transport/http/server/handler": {
       "name": "krakend-example"
    }
  },
  "timeout": "3000ms",
  "cache_ttl": "300s",
  "output_encoding": "json",
  "name": "krakend-poc",
  "port": 8080,
  "endpoints": [
    {
      "endpoint": "/hello-1",
      "method": "GET",
      "extra_config": {},
      "output_encoding": "no-op",
      "concurrent_calls": 1,
      "headers_to_pass": [
        "Authorization"
      ],
      "querystring_params": [
        "name"
      ],
      "backend": [
        {
          "extra_config": {},
          "url_pattern": "/",
          "is_collection": true,
          "encoding": "no-op",
          "sd": "static",
          "method": "GET",
          "host": [
            "http://flask-basic-auth-1:5000"
          ],
          "disable_host_sanitize": false
        }
      ]
    }
  ]
}

I run krakend in using image from dockerhub, here's the docker-compose.yml
version: '3.5'

services:

  krakend:
    image: devopsfaith/krakend:latest
    ports:
      - "8080:8080"
    volumes:
      - ./krakend/:/etc/krakend/

  flask-basic-auth-1:
    build:
      context: flask-basic-auth
      args:
        - "http_proxy=${http_proxy}"
        - "https_proxy=${http_proxy}"
        - "no_proxy=${no_proxy}"
    environment:
      - APPNUM=1
      - USER=user1
      - PASS=pass1
    ports:
      - "5000:5000"

Here are logs:
...
Successfully tagged krakend-poc_flask-basic-auth-1:latest
Starting krakend-poc_krakend_1            ... done
Starting krakend-poc_flask-basic-auth-1_1 ... done
Attaching to krakend-poc_flask-basic-auth-1_1, krakend-poc_krakend_1
krakend_1             | Parsing configuration file: /etc/krakend/krakend.json
krakend_1             | 2019/11/04 09:00:28  ERROR: unable to create the gologging logger: getting the extra config for the krakend-gologging module
krakend_1             | 2019/11/04 09:00:28  ERROR: unable to create the GELF writer: getting the extra config for the krakend-gelf module
krakend_1             | 2019/11/04 09:00:28  INFO: Listening on port: 8080
krakend_1             | 2019/11/04 09:00:28  WARNING: building the etcd client: unable to create the etcd client: no config
krakend_1             | 2019/11/04 09:00:28  DEBUG: creating a new influxdb client
krakend_1             | 2019/11/04 09:00:28  DEBUG: no config for the influxdb client. Aborting
krakend_1             | 2019/11/04 09:00:28  WARNING: influxdb: unable to load custom config
krakend_1             | 2019/11/04 09:00:28  WARNING: opencensus: no extra config defined for the opencensus module
krakend_1             | 2019/11/04 09:00:28  DEBUG: no config for the bloomfilter
krakend_1             | 2019/11/04 09:00:28  WARNING: bloomFilter: no config for the bloomfilter
krakend_1             | 2019/11/04 09:00:28  INFO: registering usage stats for cluster ID 'KATLdbT86LVaKm5q0vA3UjdmjySn7z3Ep242Jpsf3qA='
krakend_1             | 2019/11/04 09:00:28  WARNING: no config present for the httpsecure module
krakend_1             | 2019/11/04 09:00:28  DEBUG: lua: no extra config
krakend_1             | 2019/11/04 09:00:28  DEBUG: botdetector middleware:  no config defined for the module
krakend_1             | 2019/11/04 09:00:28  DEBUG: AMQP: http://flask-basic-auth-1:5000: no amqp consumer defined
krakend_1             | 2019/11/04 09:00:28  DEBUG: AMQP: http://flask-basic-auth-1:5000: no amqp producer defined
krakend_1             | 2019/11/04 09:00:28  DEBUG: pubsub: subscriber (http://flask-basic-auth-1:5000): github.com/devopsfaith/krakend-pubsub/subscriber not found in the extra config
krakend_1             | 2019/11/04 09:00:28  DEBUG: pubsub: publisher (http://flask-basic-auth-1:5000): github.com/devopsfaith/krakend-pubsub/publisher not found in the extra config
krakend_1             | 2019/11/04 09:00:28  DEBUG: http-request-executor: no extra config for backend /
krakend_1             | 2019/11/04 09:00:28  DEBUG: CEL: no extra config detected for backend /
krakend_1             | [GIN-debug] [WARNING] Running in "debug" mode. Switch to "release" mode in production.
krakend_1             |  - using env:	export GIN_MODE=release
krakend_1             |  - using code:	gin.SetMode(gin.ReleaseMode)
krakend_1             |
krakend_1             | 2019/11/04 09:00:28  DEBUG: lua: no extra config
krakend_1             | 2019/11/04 09:00:28  DEBUG: CEL: no extra config detected for pipe /hello-1
krakend_1             | 2019/11/04 09:00:28  DEBUG: lua: no extra config
krakend_1             | 2019/11/04 09:00:28  INFO: JOSE: singer disabled for the endpoint /hello-1
krakend_1             | 2019/11/04 09:00:28  DEBUG: lua: no extra config
krakend_1             | 2019/11/04 09:00:28  INFO: JOSE: validator disabled for the endpoint /hello-1
krakend_1             | 2019/11/04 09:00:28  DEBUG: botdetector:  no config defined for the module
krakend_1             | 2019/11/04 09:00:28  DEBUG: http-server-handler: no plugins registered for the module
flask-basic-auth-1_1  |  * Serving Flask app "hello.py"
flask-basic-auth-1_1  |  * Environment: production
flask-basic-auth-1_1  |    WARNING: This is a development server. Do not use it in a production deployment.
flask-basic-auth-1_1  |    Use a production WSGI server instead.
flask-basic-auth-1_1  |  * Debug mode: off
flask-basic-auth-1_1  |  * Running on http://<IP_ADDRESS>:5000/ (Press CTRL+C to quit)
krakend_1             | 2019/11/04 09:00:33  WARNING: unable to create the usage report client: Post https://usage.krakend.io/session: context deadline exceeded

it looks like you are not enabling the plugin loader in your config. Try adding this section:
"plugin": {
  "folder": ".",
  "pattern": ".so"
}

Hm, I put this just under the version key, doesn't seem to help...

are you sure the plugin is mounted as expected on the docker container?

From inside the container:
# ls /etc/krakend/
krakend-example.so  krakend.json

I'm really confused. Maybe I'll try again from scratch.
BTW, I noticed this line in log:
krakend_1             | 2019/11/04 09:00:28  ERROR: unable to create the gologging logger: getting the extra config for the krakend-gologging module

Could this be a problem? (Printlns not getting printed or something)


Audiracmichelle/issue69

Hey @ekarais, can you start reviewing the changes in this pull request? Here are some additional comments on the changes https://github.com/NSAPH-Projects/topological-equivariant-networks/issues/69


Update hcsSummary

Take first object returned for hcsSummary
Clear filters upon new group_by
Update eslint

Codecov Report

Merging #215 (5aa6657) into main (ff753af) will decrease coverage by 0.05%.
The diff coverage is 5.55%.

:exclamation: Current head 5aa6657 differs from pull request most recent head 20ebd9f. Consider uploading reports for the commit 20ebd9f to get more accurate results

Additional details and impacted files

@@           Coverage Diff            @@
##            main    #215      +/-   ##
========================================
- Coverage   5.78%   5.72%   -0.06%
========================================
  Files         43      43
  Lines       1729    1728       -1
  Branches     316     317       +1
========================================
- Hits         100      99       -1
  Misses      1629    1629

Impacted Files
Coverage Δ

src/components/user-access/PermissionsWrapper.tsx
0.00% <0.00%> (ø)

...tes/components/charts/breakdown/BreakdownChart.tsx
0.00% <0.00%> (ø)

src/routes/components/data-toolbar/DataToolbar.tsx
0.00% <0.00%> (ø)

src/routes/components/export/ExportSubmit.tsx
0.00% <0.00%> (ø)

src/routes/components/page-heading/PageHeading.tsx
0.00% <0.00%> (ø)

src/routes/details/Details.tsx
0.00% <0.00%> (ø)

src/routes/details/DetailsTable.tsx
0.00% <ø> (ø)

src/routes/details/DetailsTableExpand.tsx
0.00% <0.00%> (ø)

src/routes/overview/Overview.tsx
0.00% <0.00%> (ø)

...ts/actual-spend-breakdown/ActualSpendBreakdown.tsx
0.00% <0.00%> (ø)

... and 5 more

Continue to review full report at Codecov.

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


Security 3.5: Manage Samples - Filter records based on login context

Sample selector should have records filtered based on login context. Steps summarized from #66.
HTML Tasks:

Edit manage_samples.html to convert selector class to selectpicker form-control. Copy from select line in manage_plate.html and change:

id and name attributes to "sample"

In manage_samples.html: Change javascript code to populate selector with available records. Copy code from "for index in plates" loop in manage_plate.html into "for index in samples" loop and adjust variable names.
Find all (2) instances where sample_selector.disabled is being edited, and replace with (using true or false as appropriate):

$(function () {
	$("#sample").prop('disabled', true);
	$("#sample").selectpicker('refresh');
});

Find all (3) instances where sample_selector.value is being edited and replace with (using appropriate VALUE):
$("#sample").selectpicker('val', VALUE);
Replace function add_temp_rec() with updated function from manage_plate.html. Replace correct selector variable name and reference to selection for value setting line.

Server Tasks

Pass session into ManageSample constructor
ManageSample constructor caches the session and passes it into the SampleDao constructor
SampleDao constructor caches session
Ensure all instances of ManageSample() in app_server.py and SampleDao in model.py are updated to pass session.
Follow route that loads the activity to database query that retrieves records. After ensuring that it is not used elsewhere, change Q_GET_SAMPLES to Q_GET_SAMPLES_IN_TEAM and change SQL query to support filtering by team_ID, which is passed as a single parameter to the query.

Test Plan:

Create test samples and associate them with team_IDs via manual SQL scripts:

INSERT INTO Sample (name, team_ID) VALUES ('Sample 1a', 1);
INSERT INTO Sample (name, team_ID) VALUES ('Sample 1b', 1);
INSERT INTO Sample (name, team_ID) VALUES ('Sample 2a', 2);
INSERT INTO Sample (name, team_ID) VALUES ('Sample 2b', 2);
INSERT INTO Sample (name, team_ID) VALUES ('Sample 3', 3);

Ensure that only samples associated with the login context team are available in the sample selector
Works for multiple users, each logged in with different teams
Works when there are no plates associated with a given team
Test workflows for creating, editing, and deleting records

Test Plan - SUCCESS!


querying VIAF for name authorities

The following VIAF search endpoint ought to return the correct name entity, based on the author name and work title:
http://www.viaf.org/viaf/search?query=local.names+all+"Cuthbert, Margaret"+and+local.title+all+"Adventure in radio"&maximumRecords=5&httpAccept=application/json
Adjust parameters to taste.
I tried this against some of the entries in basic.xml, and the query returned either 1 LC entity or 0 (if no name was present).  Bunt, James isn't in the LCNAF, so he has no VIAF entry.

Very cool, @alexduryee. Thanks!


Add Cypress tests for visual output as GitHub action

I have an idea for a new feature for jQuery Terminal
After creating a Cypress PoC it would be nice to finally test visuals. The code should check the CSS, maybe height, padding and margins of the output.

It seems that you can't test box model in Cypress. But you can Add Screenshot Testing with Cypress to Your Project

I think it would be better to just stick to jest and use Puppeteer and jest-image-snapshot


chore: drop black and isort in favor of ruff

What is the current behavior?

Issue URL:
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (86fce9a) 100.00% compared to head (981bf6e) 100.00%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #74   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           55        55
=========================================
  Hits            55        55

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


[stable/prestashop] Update prerequisites

Is this a new chart
No
What this PR does / why we need it:
Update prerequisites to match the ones that we test our charts with
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/retest


[SPARK-13363][SQL] support Aggregator in RelationalGroupedDataset

What changes were proposed in this pull request?
set the input encoder for TypedColumn in RelationalGroupedDataset.agg.
How was this patch tested?
new tests in DatasetAggregatorSuite
close https://github.com/apache/spark/pull/11269
This PR brings https://github.com/apache/spark/pull/12359 up to date and fix the compile caused by race condition.

cc @rxin

Can you remove "race condition" in the description? It sounds like we actually have a race condition in the code base :)

LGTM pending Jenkins.

Test build #56045 has started for PR 12451 at commit c06fbeb.

Test build #56045 has finished for PR 12451 at commit c06fbeb.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Merging in master.

Merging this failed on my laptop. Can you try merge this yourself? I think we are good to go since the failed tests are unrelated.

merging to master!


neomutt: 2017-10-27 -> 2017-12-08

TODO

test if the sendmail binary is found at runtime, as previously guaranteed by "ac_cv_path_SENDMAIL=sendmail" . If that doesn't work, we may have to overwrite https://github.com/neomutt/neomutt/blob/381b6862fc79bd32005bc0ca49ab2d5c20bc02ef/auto.def#L191 - however, while building i noticed autosetup looks for sendmail in /nix/store/.

Motivation for this change
The new version contains lots of bugfixes and some new features, compared to the old.
The Neomutt Team has changed their build system to autosetup, which means
# Look in $PATH at runtime, instead of hardcoding /usr/bin/sendmail
"ac_cv_path_SENDMAIL=sendmail"

(from https://github.com/NixOS/nixpkgs/commit/8d46191ddc99981f311cbcafbdda0ea047fb021b)
won't apply anymore. However, neomutt's autosetup behaves exactly the same way, so i need to figure out how to fix that.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@toogley The build still succeeds without the -lidn line but I get this (again):
/nix/store/519cp7axvh8s6d8bxwbif7ki09a422kb-neomutt-20171208/bin/neomutt: error while loading shared libraries: libidn.so.11: cannot open shared object file: No such file or directory

The patch that I provided to neomutt to fix that specific issue was merged but only affects the old autoconfigure script.

@andir which one? i haven't found any neomutt related commits in https://github.com/NixOS/nixpkgs/commits?author=andir

The PR was with neomutt: https://github.com/neomutt/neomutt/pull/771

@andir I cannot reproduce your error. I did nix-build -A neomutt --check ~/src/nixpkgs > .neomutt-idn-testbuild.log 2>&1 and the only thing containing error is:
error: derivation ‘/nix/store/2slc85kf64dc3qds3lfa61ch7msfnr2n-neomutt-20171208.drv’ may not be deterministic: output ‘/nix/store/519cp7axvh8s6d8bxwbif7ki09a422kb-neomutt-20171208’ differs

Did you execute neomutt?

@andir ah, i see. I can now reproduce your problem.

I have added now nullmailer as optional depency and tested it. autosetup finds the sendmail binary, when i set  withSentmail to true by default. That means, "ac_cv_path_SENDMAIL=sendmail" (from https://github.com/NixOS/nixpkgs/commit/8d46191ddc99981f311cbcafbdda0ea047fb021b) is not necessary anymore.
Maybe its also worth to add neomutt's lua as an optional dependency.


Netlify Debian Issue

Any Suggestions?
prisma-cli: 3.14.0
error message from netlify
7:32:06 PM: > Build error occurred
7:32:06 PM: Error: Query engine library for current platform "debian-openssl-1.1.x" could not be found.
7:32:06 PM: You incorrectly pinned it to debian-openssl-1.1.x
7:32:06 PM: This probably happens, because you built Prisma Client on a different platform.
7:32:06 PM: (Prisma Client looked in<EMAIL_ADDRESS>7:32:06 PM: Searched Locations:
7:32:06 PM:   /opt/build/repo/node_modules/.prisma/client
7:32:06 PM:   /opt/build/repo/node_modules/@prisma/client
7:32:06 PM:   /opt/build/repo/node_modules/@prisma/client
7:32:06 PM:   /opt/build/repo/node_modules/.prisma/client
7:32:06 PM:   /opt/build/repo/prisma
7:32:06 PM:   /tmp/prisma-engines
7:32:06 PM:   /opt/build/repo/node_modules/.prisma/client
7:32:06 PM: You already added the platforms "debian-openssl-1.1.x", "debian-openssl-1.1.x" to the "generator" block
7:32:06 PM: in the "schema.prisma" file as described in https://pris.ly/d/client-generator,
7:32:06 PM: but something went wrong. That's suboptimal.
7:32:06 PM: Please create an issue at https://github.com/prisma/prisma/issues/new
7:32:06 PM:     at Object.request (/opt/build/repo/node_modules/@prisma/client/runtime/index.js:45411:15)
7:32:06 PM:     at async PrismaClient._request (/opt/build/repo/node_modules/@prisma/client/runtime/index.js:46301:18)
7:32:06 PM:     at async getStaticPaths (/opt/build/repo/.next/server/pages/poll/[id].js:176:20)
7:32:06 PM:     at async buildStaticPaths (/opt/build/repo/node_modules/next/dist/build/utils.js:491:31)
7:32:06 PM:     at async /opt/build/repo/node_modules/next/dist/build/utils.js:632:119
7:32:06 PM:     at async Span.traceAsyncFn (/opt/build/repo/node_modules/next/dist/trace/trace.js:79:20) {
7:32:06 PM:   type: 'PrismaClientInitializationError',
7:32:06 PM:   clientVersion: '3.14.0'
7:32:06 PM: }
7:32:07 PM: error Command failed with exit code 1. (https://ntl.fyi/exit-code-1)
7:32:07 PM: info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

curent schema
generator client {
  provider      = "prisma-client-js"
  binaryTargets = ["native", "debian-openssl-1.1.x"]
}

datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

Can you please share a small reproduction? You can probably follow this guide: https://www.prisma.io/docs/guides/deployment/deployment-guides/deploying-to-netlify as starting point.

I'm going to close this as the duplicate of #13548


derive macro warning: unexpected cfg condition name: wasm_bindgen_unstable_test_coverage

version: wasm-bindgen v0.2.95
Recent rust nightly version enabled by default the unexpected_cfgs, which lead to this warning:
warning: unexpected `cfg` condition name: `wasm_bindgen_unstable_test_coverage`
  --> game/src/lib.rs:29:1
   |
29 | #[wasm_bindgen]
   | ^^^^^^^^^^^^^^^
   |
   = help: consider using a Cargo feature instead
   = help: or consider adding in `Cargo.toml` the `check-cfg` lint config for the lint:
            [lints.rust]
            unexpected_cfgs = { level = "warn", check-cfg = ['cfg(wasm_bindgen_unstable_test_coverage)'] }
   = help: or consider adding `println!("cargo::rustc-check-cfg=cfg(wasm_bindgen_unstable_test_coverage)");` to the top of the `build.rs`
   = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> for more information about checking conditional configuration
   = note: this warning originates in the attribute macro `wasm_bindgen` (in Nightly builds, run with -Z macro-backtrace for more info)

For anyone else coming across this, you can statically declare the cfg as expected by adding the following to your Cargo.toml:
[lints.rust]
unexpected_cfgs = { level = "warn", check-cfg = ['cfg(wasm_bindgen_unstable_test_coverage)'] }

Fixed by #4284.

nice, that's a better fix than just ignoring the lint

since this can break CI by emitting a warning to potentially a lot of people, it would be nice to have a patch release
thanks and have a good day :)

since this can break CI by emitting a warning to potentially a lot of people, it would be nice to have a patch release thanks and have a good day :)

I plan to make one today or at the latest tomorrow.


EPIC: video walkthrough playlist

Short videos (<5min) showing how to use Spawn. Then basics of Cosmos SDK module developnent (maybe a quick OTC module using multiple local denoms). Similar to https://cosmology.zone/learn
Maybe not in this order, but:

New Chain, Config options, and github upload
Local-Interchain testnet
Interchaintest walkthrough
Gitbub Action CI
Basics of a CosmosSDK chain
[x] Creating a base Module
[x] Quick module logic impl
Deploying to the cloud
Add / import external modules by name -> app.go
Import an SDK module from the Cosmos-SDK (maybe have this be automatic? idk)
Software Upgrade + docs w/ how to perform (maybe a Spawn cmd for handler generation?)

Request re: SDK Concept Docs
"Honestly it would be great to have actual exercises after each of the reading materials. I do know that there is code to look at, but I’d personally benefit from each reading module having either some problems to solve or even some code that we can continue building on top of throughout all of the readings"
My thoughts

For concepts; give examples in multiple languages (Go, Python, TS/JS) to showcase it in standard digestible terms. Let's people play with things (i.e. Merkle Root, basics of a blockchain / messages / tx, etc)
Have bad modules / setup apps in git to help people walk through and debug concepts. (with solutions too)
1 program / concept should walk you through from concepts to our future interchain app.

docs have begun https://rollchains.github.io/spawn/v0.50/ as a base

ICA example


dbt test_behavior="after_all" fails since it couldnt find profile_args

When I run the dbt with test_behavior="after_all":
import os
from datetime import datetime
from pathlib import Path

from cosmos.dag import DbtDag

DEFAULT_DBT_ROOT_PATH = Path(__file__).parent / "dbt"
DBT_ROOT_PATH = Path(os.getenv("DBT_ROOT_PATH", DEFAULT_DBT_ROOT_PATH))
PROJECT_NAME = "jaffle_shop"
CONNECTION_ID = "airflow_db"

example_virtualenv = DbtDag(
    # dbt/cosmos-specific parameters
    dbt_root_path=DBT_ROOT_PATH,
    dbt_project_name=PROJECT_NAME,
    conn_id=CONNECTION_ID,
    dbt_args={"schema": "public"},
    execution_mode="virtualenv",
    operator_args={
        "project_dir": DBT_ROOT_PATH / PROJECT_NAME,
        "py_system_site_packages": False,
        "py_requirements": ["dbt-postgres==1.6.0b1"],
        "schema": "public",
    },
    schedule_interval="@daily",
    start_date=datetime(2023, 1, 1),
    catchup=False,
    dag_id="example_virtualenv_test_after_all",
    test_behavior="after_all",
)

It gives an error:
[2023-07-01, 16:51:40 UTC] {local.py:183} INFO - Cloning project to writable temp directory /tmp/tmpyd3ceg6d from /usr/local/airflow/dags/dbt/jaffle_shop
[2023-07-01, 16:51:40 UTC] {crypto.py:83} WARNING - empty cryptography key - values will not be stored encrypted.
[2023-07-01, 16:51:40 UTC] {base.py:73} INFO - Using connection ID 'airflow_db' for task execution.
[2023-07-01, 16:51:40 UTC] {base.py:55} INFO - Not using mapping PostgresUserPasswordProfileMapping because schema is not set
[2023-07-01, 16:51:40 UTC] {taskinstance.py:1768} ERROR - Task failed with exception
Traceback (most recent call last):
  File "/usr/local/airflow/astronomer_cosmos/cosmos/operators/virtualenv.py", line 100, in execute
    output = super().execute(context)
  File "/usr/local/airflow/astronomer_cosmos/cosmos/operators/local.py", line 391, in execute
    result = self.build_and_run_cmd(context=context)
  File "/usr/local/airflow/astronomer_cosmos/cosmos/operators/local.py", line 246, in build_and_run_cmd
    return self.run_command(cmd=dbt_cmd, env=env, context=context)
  File "/usr/local/airflow/astronomer_cosmos/cosmos/operators/local.py", line 204, in run_command
    profile_mapping = get_profile_mapping(
  File "/usr/local/airflow/astronomer_cosmos/cosmos/profiles/__init__.py", line 61, in get_profile_mapping
    raise ValueError(f"Could not find a profile mapping for connection {conn_id}.")
ValueError: Could not find a profile mapping for connection airflow_db.

When I run the dbt with test_behavior="after_each":
example_virtualenv = DbtDag(
    # dbt/cosmos-specific parameters
    dbt_root_path=DBT_ROOT_PATH,
    dbt_project_name=PROJECT_NAME,
    conn_id=CONNECTION_ID,
    dbt_args={"schema": "public"},
    execution_mode="virtualenv",
    operator_args={
        "project_dir": DBT_ROOT_PATH / PROJECT_NAME,
        "py_system_site_packages": False,
        "py_requirements": ["dbt-postgres==1.6.0b1"],
    },
    schedule_interval="@daily",
    start_date=datetime(2023, 1, 1),
    catchup=False,
    dag_id="example_virtualenv_test_after_each",
    test_behavior="after_each",
)

I get the success logs:
[2023-07-01, 16:44:57 UTC] {local.py:183} INFO - Cloning project to writable temp directory /tmp/tmp2nt61p9v from /usr/local/airflow/dags/dbt/jaffle_shop
[2023-07-01, 16:44:57 UTC] {crypto.py:83} WARNING - empty cryptography key - values will not be stored encrypted.
[2023-07-01, 16:44:57 UTC] {base.py:73} INFO - Using connection ID 'airflow_db' for task execution.
[2023-07-01, 16:44:57 UTC] {local.py:228} INFO - Trying to run the command:
 ['/usr/local/bin/dbt', 'test', '--models', 'customers']
From /tmp/tmp2nt61p9v/dbt_project
[2023-07-01, 16:44:57 UTC] {process_utils.py:179} INFO - Executing cmd: /usr/local/bin/python -m virtualenv /tmp/cosmos-venvvjfh2hxx --python=python3

Are you willing to submit PR?

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

Code of Conduct

[x] I agree to follow this project's Code of Conduct

Reason:
The argument passed to render test_task when the test_behaviour="after_all"
arguments={**task_args, **operator_args}
https://github.com/astronomer/astronomer-cosmos/blob/main/cosmos/render.py#L250
Which results in:
operator_class='cosmos.operators.virtualenv.DbtTestVirtualenvOperator', arguments={'schema': 'public', 'conn_id': 'airflow_db', 'project_dir': PosixPath('/usr/local/airflow/dags/dbt/jaffle_shop'), 'py_system_site_packages': False, 'py_requirements': ['dbt-postgres==1.6.0b1']})])'
The argument passed to render test_task when the test_behaviour="after_each"
arguments=test_args
https://github.com/astronomer/astronomer-cosmos/blob/main/cosmos/render.py#L217
operator_class='cosmos.operators.virtualenv.DbtTestVirtualenvOperator', arguments={'schema': 'public', 'conn_id': 'airflow_db', 'project_dir': PosixPath('/usr/local/airflow/dags/dbt/jaffle_shop'), 'py_system_site_packages': False, 'py_requirements': ['dbt-postgres==1.6.0b1'], 'models': 'stg_payments', 'profile_args': {'schema': 'public'}, 'profile_name': None, 'target_name': None, 'on_warning_callback': None, 'outlets': [Dataset(uri='DBT://AIRFLOW_DB/JAFFLE_SHOP/STG_PAYMENTS', extra=None)]})]), Task(id='raw_customers_seed', upstream_entity_ids=[]
The profile validation will fail
https://github.com/astronomer/astronomer-cosmos/blob/main/cosmos/profiles/base.py#L34
Where it is searching for profile_args: dict[str, Any]
Resolution:
https://github.com/astronomer/astronomer-cosmos/blob/main/cosmos/render.py#L250
Update to: arguments={**test_args, **operator_args}
If this makes sense I am willing to make the contribution.

 import os
from datetime import datetime
from pathlib import Path

from cosmos.dag import DbtDag

DEFAULT_DBT_ROOT_PATH = Path(__file__).parent / "dbt"
DBT_ROOT_PATH = Path(os.getenv("DBT_ROOT_PATH", DEFAULT_DBT_ROOT_PATH))
PROJECT_NAME = "jaffle_shop"
CONNECTION_ID = "airflow_db"

example_virtualenv = DbtDag(
    # dbt/cosmos-specific parameters
    dbt_root_path=DBT_ROOT_PATH,
    dbt_project_name=PROJECT_NAME,
    conn_id=CONNECTION_ID,
    **dbt_args**={"schema": "public"},
    execution_mode="virtualenv",
    operator_args={
        "project_dir": DBT_ROOT_PATH / PROJECT_NAME,
        "py_system_site_packages": False,
        "py_requirements": ["dbt-postgres==1.6.0b1"],
        "schema": "public",
    },
    schedule_interval="@daily",
    start_date=datetime(2023, 1, 1),
    catchup=False,
    dag_id="example_virtualenv_test_after_all",
    test_behavior="after_all",
)

instead of dbt_args, did you try using profile_args ? If yes, does the error remain?

Thank you for your response @CorsettiS
Yes I tried using profile_args={"schema": "public"} and its gives the same error.

@CorsettiS did you get a time to look into this?
If its a real issue and not a configuration mismatch, the solution what I provided above wouldnt work. As it would only pick the last test in the loop and execute.

@ajithshetty I am just a contributor of the project, and not a member of the main team. I lack the capacity to help you on this issue. I can tell you tho that now that cosmos has reached v1.0.0, most likely this error will be gone or at least you will obtain a different error in the worst case scenario. The project underwent a huge architecture change

Thank you @CorsettiS for your response.
I have checked in the latest cosmos version and this issue has been resolved.

When the test_behaviour=after_all, the task will combine all the tests into a consolidated single task with multiple test cases.
Its good to be closed.


Time suggestions on the dashboard

The start time column should be consistent with the details page, including the date section

The timestamp column should be consistent with the time in the console, including the date section and accuracy

The timestamp column should be consistent with the time in the console, including the date section and accuracy

Inconsistent time format

Changes based on your feedback: https://github.com/dotnet/aspire/pull/2306
Some inconsistencies are intentional. We don't show dates in the grid if not needed (which will be almost always). We don't show milliseconds on a time if it's not available or needed.

Which one? Using what date format?

The date is now always shown if it is different than today's date. Also, the tooltip shows the date and time.

Yes. Done.

The result is cached based on the culture name key. The culture name key will come from the current culture.


Error in transform()

I was trying to apply UMAP.transform() to embed new data on a fitted UMAP instance. Unfortunately I get an error:
TypingError                               Traceback (most recent call last)
<ipython-input-75-c35306a6db40> in <module>()
----> 1 XY = U.transform(nx.adj_matrix(nx.subgraph(huG, E)).toarray())

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/umap/umap_.py in transform(self, X)
   1566                 self._random_init,
   1567                 self._tree_init,
-> 1568                 rng_state,
   1569             )
   1570             result = self._search(

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/umap/nndescent.py in initialise_search(forest, data, query_points, n_neighbors, init_from_random, init_from_tree, rng_state)
    159 ):
    160     results = make_heap(query_points.shape[0], n_neighbors)
--> 161     init_from_random(n_neighbors, data, query_points, results, rng_state)
    162     if forest is not None:
    163         for tree in forest:

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/dispatcher.py in _compile_for_args(self, *args, **kws)
    342                 raise e
    343             else:
--> 344                 reraise(type(e), e, None)
    345         except errors.UnsupportedError as e:
    346             # Something unsupported is present in the user code, add help info

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/six.py in reraise(tp, value, tb)
    656             value = tp()
    657         if value.__traceback__ is not tb:
--> 658             raise value.with_traceback(tb)
    659         raise value
    660

TypingError: Failed at nopython (nopython frontend)
Internal error at <numba.typeinfer.ArgConstraint object at 0x11916b470>:
--%<----------------------------------------------------------------------------
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/errors.py", line 491, in new_error_context
    yield
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/typeinfer.py", line 194, in __call__
    assert ty.is_precise()
AssertionError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/typeinfer.py", line 138, in propagate
    constraint(typeinfer)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/typeinfer.py", line 195, in __call__
    typeinfer.add_type(self.dst, ty, loc=self.loc)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/contextlib.py", line 99, in __exit__
    self.gen.throw(type, value, traceback)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/errors.py", line 499, in new_error_context
    six.reraise(type(newerr), newerr, tb)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/six.py", line 659, in reraise
    raise value
numba.errors.InternalError:
[1] During: typing of argument at /Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/umap/nndescent.py (125)
--%<----------------------------------------------------------------------------

File "../../../../../../Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/umap/nndescent.py", line 125:
    def init_from_random(n_neighbors, data, query_points, heap, rng_state):
        for i in range(query_points.shape[0]):
        ^

This error may have been caused by the following argument(s):
- argument 1: cannot determine Numba type of <class 'scipy.sparse.csr.csr_matrix'>

This is not usually a problem with Numba itself but instead often caused by
the use of unsupported features or an issue in resolving types.

To see Python/NumPy features supported by the latest release of Numba visit:
http://numba.pydata.org/numba-doc/dev/reference/pysupported.html
and
http://numba.pydata.org/numba-doc/dev/reference/numpysupported.html

For more information about typing errors and how to debug them visit:
http://numba.pydata.org/numba-doc/latest/user/troubleshoot.html#my-code-doesn-t-compile

If you think your code should work with Numba, please report the error message
and traceback, along with a minimal reproducer at:
https://github.com/numba/numba/issues/new

The condensed version of that long and complicated traceback is that UMAP's
transform doesn't current accept sparse matrices as input. Obviously I
should have had better input validation on that. There s nothing in
principle that would stop sparse matrix input from working, but I will need
to go through the code and find the places that need to be fixed or
rewritten to make it go through. Hopefully a future release will close the
gap for you. In the meantime, presuming you have a sufficiently small case
you can try casting to dense.
On Fri, Jul 13, 2018 at 9:24 AM Davide Cittaro<EMAIL_ADDRESS>wrote:

I was trying to apply UMAP.transform() to embed new data on a fitted UMAP
instance. Unfortunately I get an error:
TypingError                               Traceback (most recent call last)
 in ()
----> 1 XY = U.transform(nx.adj_matrix(nx.subgraph(huG, E)).toarray())
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/umap/umap_.py in transform(self, X)
1566                 self._random_init,
1567                 self._tree_init,
-> 1568                 rng_state,
1569             )
1570             result = self._search(
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/umap/nndescent.py in initialise_search(forest, data, query_points, n_neighbors, init_from_random, init_from_tree, rng_state)
159 ):
160     results = make_heap(query_points.shape[0], n_neighbors)
--> 161     init_from_random(n_neighbors, data, query_points, results, rng_state)
162     if forest is not None:
163         for tree in forest:
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/dispatcher.py in _compile_for_args(self, *args, **kws)
342                 raise e
343             else:
--> 344                 reraise(type(e), e, None)
345         except errors.UnsupportedError as e:
346             # Something unsupported is present in the user code, add help info
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/six.py in reraise(tp, value, tb)
656             value = tp()
657         if value.traceback is not tb:
--> 658             raise value.with_traceback(tb)
659         raise value
660
TypingError: Failed at nopython (nopython frontend)
Internal error at <numba.typeinfer.ArgConstraint object at 0x11916b470>:
--%<----------------------------------------------------------------------------
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/errors.py", line 491, in new_error_context
yield
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/typeinfer.py", line 194, in call
assert ty.is_precise()
AssertionError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/typeinfer.py", line 138, in propagate
constraint(typeinfer)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/typeinfer.py", line 195, in call
typeinfer.add_type(self.dst, ty, loc=self.loc)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/contextlib.py", line 99, in exit
self.gen.throw(type, value, traceback)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/errors.py", line 499, in new_error_context
six.reraise(type(newerr), newerr, tb)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/numba-0.38.1-py3.6-macosx-10.6-intel.egg/numba/six.py", line 659, in reraise
raise value
numba.errors.InternalError:
[1] During: typing of argument at /Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/umap/nndescent.py (125)
--%<----------------------------------------------------------------------------
File "../../../../../../Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/umap/nndescent.py", line 125:
def init_from_random(n_neighbors, data, query_points, heap, rng_state):
for i in range(query_points.shape[0]):
^
This error may have been caused by the following argument(s):

argument 1: cannot determine Numba type of <class 'scipy.sparse.csr.csr_matrix'>

This is not usually a problem with Numba itself but instead often caused by
the use of unsupported features or an issue in resolving types.
To see Python/NumPy features supported by the latest release of Numba visit:http://numba.pydata.org/numba-doc/dev/reference/pysupported.html
andhttp://numba.pydata.org/numba-doc/dev/reference/numpysupported.html
For more information about typing errors and how to debug them visit:http://numba.pydata.org/numba-doc/latest/user/troubleshoot.html#my-code-doesn-t-compile
If you think your code should work with Numba, please report the error message
and traceback, along with a minimal reproducer at:https://github.com/numba/numba/issues/new
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lmcinnes/umap/issues/81, or mute the thread
https://github.com/notifications/unsubscribe-auth/ALaKBeU-uyRGDP3gO9nF1z7rPf-tYvaxks5uGK2OgaJpZM4VO-j5
.

Training data were sparse but fitting went fine. Test data were dense. I will try with dense training data, then.

Ah, that's not as good. I'll definitely have to look into this a little
further and make sparse handling more robust in general. Sorry for the
issues.
On Fri, Jul 13, 2018 at 12:35 PM Davide Cittaro<EMAIL_ADDRESS>wrote:

Training data were sparse but fitting went fine. Test data were dense. I
will try with dense training data, then.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lmcinnes/umap/issues/81#issuecomment-404901826, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALaKBaSeOkcnx1NevSj5a6lGfzEW7LHRks5uGNpQgaJpZM4VO-j5
.


BUG: Add missing enumerate with multi-ndarray-output itk filters

Required for returning the (registered_image, transform_parameters)
output when ndarray's is passed for the fixed and moving image in
itk-elastix.

Does this relate to https://github.com/InsightSoftwareConsortium/ITKElastix/issues/37?

Does this relate to InsightSoftwareConsortium/ITKElastix#37?

This is a separate issue, but we are on the path to address it :-)


Deformed glyphs

Some glyphs have been parsed with misplaced point at the start/end of the bezier curve, and so the glyph appears to be deformed. This mostly occurs when the closing point is on the curve (not in the corner) typically on O, or 0 glyphs, but also some others (depending on the font).
Here is an example of such misplacement within O glyph in Futura Condensed ExtraBold font. The character on the left is outlined in Adobe Illustrator. On the right there is this character processed by opentype.js using Glyph Inspector (http://opentype.js.org/glyph-inspector.html) with same font loaded.

Here's a fairly rough patch I wrote to fix this, since I found similar problems with a lot of different fonts and it was driving me crazy.

Thanks @greegus and @cdave1,
which version of Futura Condensed ExtraBold are you using ? I can't reproduce the problem with version 001.003 in OTF.

@fpirsch I'm using 006.000 TrueType

Ah OK I see it now. Thanks @greegus !

I can't find the exact Futura version cited before, any link or maybe another font with the same problem?

@Jolg42 futura-006.000.zip

@greegus Thank you! Indeed I have the same problem, I'm gonna check that :)

@greegus The fix is live on http://opentype.js.org/glyph-inspector.html Thanks to @cdave1 for the fix 😉


add support for Swift 3

No API naming changes were needed as the API was designed with the Swift 3 API guidelines in mind.

Looks good to me.


Adds firefox compatability + more (once manifest v3 is supported)

Features

Firefox support (once Firefox officially supports manifest v3)
Allows for horizontal scroll wheels (yes I know...) to change volume properly
Performed a quick lint by hand

Testing
I've manually tested the unpacked folder on Chrome and zipped it up for Firefox.
Both browsers appear to be working properly.

Thanks for the firefox improvement, but I just don't understand what's the point of the horizontal scroll?
from what I know it can only be done with SHIFT + scroll wheel, which is already a function in this extension (x2 volume steps)

My mouse, and a few others allow you to rock the scroll wheel, left and right.
So it acts as a native horizontal scroll wheel.

LGTM thanks 👍🏼

I will soon release a new version on the chrome-store, and also consider publishing it on addons.mozilla.org

https://addons.mozilla.org/en-US/firefox/addon/youtube-volume-scroll/
Installing manifest v3 add-ons will be available on stable on 17.01.2023 (tomorrow at the time of writing)


TagginsController?

We have a Taggings model, but no controller. How will Taggings be created?

Discussed, added API::TaggingsController


[3.x] Tab supports badge color

[x] Changes have been thoroughly tested to not break existing functionality.
[x] New functionality has been documented or existing documentation has been updated to reflect changes.
[x] Visual changes are explained in the PR description using a screenshot/recording of before and after.

This PR adds support for using ->badgeColor() in Tab component

@saade, could you add some docs to https://filamentphp.com/docs/3.x/support/blade-components/tabs#setting-a-tab-badge as well?


organize imports

Group and order golang imports

Restarted the travis build - looks like it flaked on a build (timed out)

/lgtm
@coreydaley-redhat did you use any tooling to automate this? Would be nice if we could include it in our make generate/make verify targets.

From a previous hack day: https://github.com/coreydaley-redhat/osgoimports

Restarted the travis build - looks like it flaked on a build (timed out)

I tried to restart it but did not have access on Travis to restart the test.


FileNotFoundError: [Errno 2] No such file or directory: 'data/tets/256_tets.npz'

Hi, thanks for sharing the code!
I am trying to reproduce the MagicPony code. I have followed the instructions in readme, and tried to do testing on both horse and giraffe. The commands i used are
python run.py --config config/quadrupeds/test_giraffe_pretrained.yml --gpu 0 --num_workers 4
python run.py --config config/quadrupeds/test_horse.yml --gpu 0 --num_workers 4
However, i received the similar error message as shown below:
Loading configs from config/quadrupeds/test_giraffe_pretrained.yml
Environment: GPU 0 - seed 0
Loading testing data from data/giraffe_coco/test
/home/chenny/.conda/envs/magicpony/lib/python3.9/site-packages/torchvision/transforms/transforms.py:287: UserWarning: Argument interpolation should be of type InterpolationMode instead of int. Please, use InterpolationMode enum.
warnings.warn(
Traceback (most recent call last):
File "/mnt/nas/chenny/MagicPony/run.py", line 14, in
trainer = Trainer(cfgs, MagicPony)
File "/mnt/nas/chenny/MagicPony/magicpony/trainer.py", line 53, in init
self.model = model(cfgs)
File "/mnt/nas/chenny/MagicPony/magicpony/model.py", line 48, in init
self.netPrior = PriorPredictor(self.cfgs)
File "/mnt/nas/chenny/MagicPony/magicpony/predictor.py", line 77, in init
self.netShape = DMTetGeometry(
File "/mnt/nas/chenny/MagicPony/magicpony/geometry/dmtet.py", line 179, in init
self.load_tets(self.grid_res, self.grid_scale)
File "/mnt/nas/chenny/MagicPony/magicpony/geometry/dmtet.py", line 203, in load_tets
tets = np.load('data/tets/{}_tets.npz'.format(grid_res))
File "/home/chenny/.conda/envs/magicpony/lib/python3.9/site-packages/numpy/lib/npyio.py", line 390, in load
fid = stack.enter_context(open(os_fspath(file), "rb"))
FileNotFoundError: [Errno 2] No such file or directory: 'data/tets/256_tets.npz'
I wonder if you could kindly clarify where can i find  'data/tets/256_tets.npz'? I appreciate your time and effort in helping me. Thank you! Wish you a lovely day.

Hi, thanks for the heads up! Please use this script to download the tet grids:
https://github.com/elliottwu/MagicPony/blob/4ebaf90c90e25431e3fd65d05f7f8d0b1bb5da0d/data/tets/download_tets.sh


Have bikeshed help with publication announcements

When pushing out a new publication to TR, it's good practice to send announcements by email, to blog about them, to tweet about them, etc. But it's annoying, so people tend to skip.
However, if you look at the templates proposed in https://wiki.csswg.org/spec/publish, bikeshed already knows (most of the) information that should go in there. For the mail version, it's something like

The [Group] has published a new [Status] of [Title]:
[TR]

[Abstract]
[Status Text]
See also the change section for details:
[This Version]#changes
Please review the draft, and send any comments to this mailing list,
<EMAIL_ADDRESS>prefixed with [Shortname] (as I did on this
message) or (preferably) file them in the GitHub repository at
https://github.com/w3c/csswg-drafts/issues

And for the blog version, it's a bit different thanks to being able to use markup, but not all that different.
Granted, this assumes you're keeping the Status Text and Abstract up to date, but you should do that anyway.
If something like bikeshed echidna --with-announcements could generate that text for me, that'd be cool. If it could (given credentials) post it for me, that would be even cooler.

The point of the announcement is not just to say "we did a thing, go look at it", but to actually communicate something interesting about the status. So I'm not sure automatically posting is a good idea even though it sounds nice. It's good for the author to actually think what's relevant commentary to post.

I agree that the author should do that, but communicating something interesting about the status of the spec is also what the status section of the spec is supposed to do, and most of the time it's neglected. I'm thinking that by tying this together, editors will get an incentive to write their Status Text, and get a mail/blog for free when they do that.

And besides, if you think the spec's status text section should be different from the mail's "something interesting about the status", the tool can take that into account, and draw it from some other source, and still save some effort composing the mail, escaping the things, replacing the bits in the template, etc.

@frivoal Agree the Status section should say something interesting, but it's so much boilerplate atm it's not very useful even when it does. :( Wrt saying something different... I usually have slightly different text for announcements depending on the audience, and some announcements are CCed to e.g. www-international and some aren't etc. So there's often some amount of tweaking in my experience.

I think I'll still want the tool, at least for myself, because while I do some of the tailoring you mention, I still find that most of the time preparing announcement is converting syntax between mail and blog, filling in template details... It's not hard but its annoying.
Also, in the CSSWG, hardly anyone other than us (and Rachel Andrew) posts announcements at all. Making it easier to do so might improve that.


C_CPP:TYPEDEF IN CPP

Description
Documentation on typedef in C/C++.
Domain
C/CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

/assign

/assign


Fix timings of leading space and onFinished

I encountered some timings issues with:

a leading space in the input text lasts 4 dits instead of 7
the onFinished event is fired slightly before the end of the signal because of the low-pass filter

Other commits also remove some unnecessary code and add some comments

Please do not merge yet, I think there is still something abnormal going on.

No worries, take your time to investigate/fix!

Okay, I had (again) gotten confused between letter space and word space. That sounds better now!


Changed the file owner to root.www-data.

Fixes #75

Looks good, although symfony will need to write the cache, so this change will break this

Yes, probably you need g+w?

In a derived image I use with symfony I needed to completely reset the file modes and ownerships to www-data + the umask (to 002) of the fpm process to have have a stable experience.

@cedricziel alright, can you share your Dockerfile?

Restarted the travis build

Yeah it may break some apps. Do you mind if I submit this and push the latest image (it will happen automatically)?

Ok, I would go ahead and merge this.


Allowing to parse a wider range of keys

Attempting to solve issue #53.
I decided to opt for using Data.Text.stripEnd instead of relying on a lot of backtracking when parsing.

I've pushed. Let me know if there's anything else I should improve.


Default speed limits should be saved generally to the XML

Please, don't have these settings also saved to the game save, it's a real bummer to set up the values when many road types (and tracks as well) are downloaded.
When a player decides to have specific values, then the consideration is high that when starting a new game the same values are still to be considered.
So please let these values be saved to either the TMPE_GlobalConfig.xml or to a custom XML, so players also can have a backup of their settings.

Would this also apply to custom roads which are not included in the vanilla game (such as the NExt2 roads)?

i would also see it as a global configuration. once you went through all the roads, you don't want to do it for every save again. also i don't think that ppl play with different speed limits on different maps .. i could be wrong though. could be an option or so.

@Xyrhenix this is the point - the more custom roads you got, the more (dozens or hundreds) of repeated similar clicking you can avoid if this would be a global setting.

@DaEgi01 if someone wants to have different settings in another game, create a backup of the XML and edit the values in there.

With regards to whether or not people would use this per-map, I can see situations where one might intentionally want different defaults per map (leading example being if someone has a map designed to replicate an American city and then one to replicate a British city).
If the intent for this feature request is for it to work with all roads (including modded roads), I would suggest a separate XML file because its length could vary considerably depending on how many roads are installed and actively used. In that could potentially be something like this:
<road>
    <name>ROAD_NAME</name>
    <speedlimit>50</speedlimit>
</road>

As an idea for implementation: In the default limit editor, have an additional button that's something like "Set global default". This would handle per-road global default setting. In the mod settings panel would then be "Set all global defaults to this save's," which would iterate through all the currently installed roads and write their default speed limits in the currently-loaded save to the global limits XML.

Since it's actually stated in the options that all the options are save specific, to save people getting confused about this 1 setting that's global, what about an export / import feature instead? Get all your road speeds set, export to a file, then for each new save you can quickly just import your speed file. This would also allow you to have multiple different speed files for different games. You could have one file that just sets all roads to 100 speed units for total madness, one with sensible speeds, one for country specific speeds, and you just import the speed set you want based on the game you intend to play.

I support that.

"Save settings as default for new games" button would be most desirable!

Tagging https://github.com/krzychu124/Cities-Skylines-Traffic-Manager-President-Edition/issues/84

Duplicate of #84


Clinical age operators are not respecting mapping targets

The Age/AgeAt functions do not make use of the mapping target of the birthDateProperty of the modelInfo:
using QICore 4.1.1

define "Initial Population":
  AgeInYearsAt(date from end of "Measurement Period") in Interval[24, 64]

This translates, but without the .value accessor in the mapping of the QICore modelinfo, resulting in a runtime error attempting to treat a FHIR.date as a CQL Date.

Included in release 2.4.0


Fix #97 Duplicate extracted licenses when parsing from RDF

Fixes #97
The field Document.extracted_licenses contained duplicate ExtractedLicense objects when they are parsed from RDF files. Both extracted licenses in <licenseConcluded> and <licenseDeclared> were being added twice.
This PR makes extracted licenses are parsed only once, when <SpdxDocument> is being parsed.
Signed-off-by: Xavier Figueroa<EMAIL_ADDRESS>

@pombredanne I've noticed that there aren't any RDF parsers tests where to add one for this issue. I could eventually write them, but is it OK to meanwhile write only the needed one(s) to test this?

@pombredanne @krysnuvadga I've written some unit tests for the fields that may have been affected by the change.
I would have liked to follow the pattern suggested by @pombredanne in Gitter, as I actually did in tests for YAML/JSON parsers, but, although much of that code is reusable for this tests, it hasn't been merged yet. Moreover, for that code to work for RDF parsers, it would require some changes to avoid #91 consequences.

@xavierfigueroav This needs some rebasing

@pombredanne Rebased!

All merged. Thank you again :)