Time ranges between days not handeled

I noticed that the parser gives a ParseException when processing times range between days, for example while processing the following range: "Mo-Fr 18:00-02:00".
Shouldn't this case be covered? It's common for bars/pubs/disco to open before midnight and close after.
This is the exception returned while processing previous range:
ch.poole.openinghoursparser.ParseException: End time earlier than start time at line 1, column 17
	at ch.poole.openinghoursparser.OpeningHoursParser.timespan(OpeningHoursParser.java:294)
	at ch.poole.openinghoursparser.OpeningHoursParser.time_selector(OpeningHoursParser.java:337)
	at ch.poole.openinghoursparser.OpeningHoursParser.rule(OpeningHoursParser.java:1069)
	at ch.poole.openinghoursparser.OpeningHoursParser.rules(OpeningHoursParser.java:1094)
	...

You need to turn strict mode off.


Add convenience Functions to save/load quantized Model to/from Disk

Since loading Quantized models from HF is not possible, jet.
I was searching for an easy way to safe models after quantization, as easy as loading them from HF.
And then a function to load the file again.

This probably belongs more to Axon than Bumblebee, since we need a way to store %Axon.ModelState{}. For the model itself, maybe there should be a way to quantize the model only, so it can be replicated without altering the params (so we can still build the model, instead of storing it). cc @seanmor5


[FEAT]: sticky nav and blur effect

Navbar is now sticky and added blur bg effect
This PR close issue  #653
(Like it, merge it 😵)

I love it @devvsakib!!


Axis disappearing on zoom

Summary
I trying to create a chart in my app, and see strange behaviour: Bottom axis disappearing on zoom (see video). It's also not reproducing on original mpandroidchart library.
https://user-images.githubusercontent.com/3313259/122921620-5bcc2900-d36b-11eb-88e4-e81c74821dd4.mp4
Here is a sample of code: Gist
Device (please complete the following information):

Device: Samsung Galaxy s8, google pixel emulator
Android Version 9.0+
Library Version 1.1.33

@xzwiex could you please try with the latest version?

OK thanks will try to reproduce


Produced PHA files are not compatible with XSPEC

A .pha file produced with .write_pha of the OGIPLike plugin is not readable by XSPEC, because it is missing a column.
Furthermore, its GROUPING is not recognized.
Here is the XSPEC error:
XSPEC12>data test_out.pha{1}
***Warning: Unrecognized grouping for channel(s). It/they will be reset to 1.

***XSPEC Error:  data file missing required entries: ANCRFILE

Here is the code to reproduce the problem, using the data in attachment:
from threeML import *

dd = '100724029/'

lle = OGIPLike("LLE",dd+"bn100724029_LAT-LLE_srcspectra.pha{1}",
                        dd+"bn100724029_LAT-LLE_bkgspectra.bak{1}",
                        dd+"bn100724029_LAT-LLE_weightedrsp.rsp{1}")
lle.set_active_measurements('3e4-1e7')

lle.rebin_on_background(5.0)

lle.write_pha("test_out.pha", overwrite=True)

A probably unrelated problem is produced by this similar code, using one of the .pha files in the examples:
from threeML import *

# Update this with your path
dd = '/home/giacomov/develop/3ML/examples/'

lle = OGIPLike("LLE",dd+"bn090217206_n9_srcspectra.pha{1}",
                        dd+"bn090217206_n9_bkgspectra.bak{1}",
                        dd+"bn090217206_n9_weightedrsp.rsp{1}")
lle.set_active_measurements('10.0-1000.0')

lle.rebin_on_background(5.0)

lle.write_pha("test_out.pha", overwrite=True)

and this is the XSPEC error:
XSPEC12>data test_out.pha{1}

***XSPEC Error:  data file missing required entries:
Detector channel numbers missing in extension: SPECTRUM

***XSPEC Error:  data file missing required entries: ../test_out.pha

***XSPEC Error: Error reading data: File ../test_out.pha

data_files.tar.gz

@giacomov I'm on this. Good catch! What time scale do you need this on? I'm busy until this afternoon EU time.

no worries, no rush on my side. I'm going to be offline (traveling) the whole weekend.

fixed with #95


[DNS resolver] enable resolver component tests on windows

I can't see a reason why windows is skipped with these now - there should be no test dependency issues on bazel RBE.

superseded by https://github.com/grpc/grpc/pull/34122


MNT: Re-render the feedstock [ci skip]

Hi! This is the friendly conda-forge-admin automated user.
I've re-rendered this feedstock with the latest version of conda-smithy (1.7.0) and noticed some changes.
If the changes look good, then please go ahead and merge this PR.
If you have any questions about the changes though, please feel free to ping the 'conda-forge/core' team (using the @ notation in a comment).
Remember, for any changes to the recipe you would normally need to increment the version or the build number of the package.
Since this is an infrastructural change, we don't actually need/want a new version to be uploaded to anaconda.org/conda-forge, so the version and build/number are left unchanged and the CI has been skipped.
Thanks!

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


VPC limit exceeded

Hi All,
I tried creating vpc using VPC module in terraform. I used a region only one VPC is available. But I observed below error. Any ideas on this..??
VPC module
module "vpc" {
version = "2.23.0"
source = "terraform-aws-modules/vpc/aws"
name = "${var.cluster_name}-vpc"
cidr = var.vpc_cidr
azs = ["${var.aws_region}a", "${var.aws_region}b"]
private_subnets = var.private_subnets
public_subnets = var.public_subnets
enable_dns_hostnames = true
enable_dns_support = true
enable_nat_gateway = true
one_nat_gateway_per_az = true
#single_nat_gateway = true
    tags = "${merge(local.common_tags, map("Name", "${var.cluster_name}"))}"

}

Error observed:
Error: Error creating VPC: VpcLimitExceeded: The maximum number of VPCs has been reached.
status code: 400, request id: 9559ac7e-6c55-46ff-89ca-becbe10b30dd
on .terraform/modules/vpc/terraform-aws-vpc-2.23.0/main.tf line 29, in resource "aws_vpc" "this":
29: resource "aws_vpc" "this" {
Error: Error creating EIP: AddressLimitExceeded: The maximum number of addresses has been reached.
status code: 400, request id: 678a0803-be58-4e9c-8bb4-03c3345390d8
on .terraform/modules/vpc/terraform-aws-vpc-2.23.0/main.tf line 857, in resource "aws_eip" "nat":
857: resource "aws_eip" "nat" {
Error: Error creating EIP: AddressLimitExceeded: The maximum number of addresses has been reached.
status code: 400, request id: c232c962-74e3-4848-bf40-9713e3cc8205
on .terraform/modules/vpc/terraform-aws-vpc-2.23.0/main.tf line 857, in resource "aws_eip" "nat":
857: resource "aws_eip" "nat" {

NOTE:

Currently only one vpc is allocated in that region.

Everything in AWS has a limit. By default you are limited to 5 VPCs per region. See documentation about VPC limits: here
Create support tickets through the AWS console to request increases in your limits.

I exceeded my quota and got this same error. I requested a quota increase, it went from pending -> completed and when I send a CLI request (via the aws cli) for my quota details it confirms I have a new quota of 10 (was 5).
Before I requested the quota increase I only had 3 active VPCs, but was still getting this error. I hit 5 once ever (the limit before I requested an increase). So there's that.


zsh: no matches found on curl URL

here is the screenshot.

I also run this command in my macOS. It also has this problem.

You need to escape the question mark, otherwise zsh thinks it is a globbing or wildcard character and tries to find files that match it (that's why it says no matches found). Using double or single quotes is enough: curl "wttr.in/Changsha?lang=zh".

I just came across this issue in my ZSH as well.
This seems like a bug that it should require escaping a valid URL character. My worker is able to run the same cURL command on his Mac without having to escape.

if you are using basic auth as in " curl -u user:pass " just put the user in if it gives you errors and it will prompt you for the pass, which is good anyway since you shouldn't put pass on the cl anyway.


Block.isSolidMediumSquare -> hasTopRim or similar

isSolidMediumSquare is similarly named to isSolidSmallSquare, but this is a bit misleading.
isSolidSmallSquare is true for the top of a fence, for example. isSolidMediumSquare makes me think of a comparable larger centered square, but that's not the case, considering it's false for anvils but true for hoppers, cauldrons, and composters.
Also, isSolidMediumSquare doesn't take a direction, it only checks the top face of the block. Rails and redstone gates use it to determine valid placement.
Considering these facts, it seems as though the intent of this function is to check for an outer rim on the top face of a block. Indeed, SOLID_MEDIUM_SQUARE_SHAPE is created by taking a block shape and subtracting the inner portion.

Fixed in #848 by naming it topCoversMediumSquare

That's still a pretty poor name

Okay, I'll rename it hasTopRim then

Oh, thank you!


error: rm: file .bazelversion not found

I see that this is not the right repository for the arch linux deepspeech package.


docs: specify ws and wss urls

What kind of change does this PR introduce?
docs update
Additional context
Related issue: https://github.com/supabase/realtime/issues/165

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


qhub / ovh support

Hi,
Hope you are all well !
Is it possible to support OVH also for european countries ?
Thanks for any insights or inputs on that question.
Cheers,
X

@x0rzkov it looks to me that OVH has an REST api for managing dns entries. We welcome PRs on this issue. In order to support OVH we would simply need a function implemented in qhub.providers.dns.ovh:update_record similar to https://github.com/Quansight/qhub/blob/master/qhub/provider/dns/cloudflare.py.


Resolution setting for fullscreen and borderless window

A toggle for letterboxing vs. stretching to screen should also exist, like it currently does in stable.

WidthFullscreen and HeightFullscreen already exist, for what it's worth. Also I'm not sure if we want to implement letterboxing at a framework level.

Note that the FrameworkSetting enum already contains values for letterboxing.

So.. a problem arises when we want to figure out what display the game window is currently on.
At the moment, all resolution changes only apply to the primary display (DisplayDevice.Default), which is not enough in multi-monitor setups.
The issue at hand is that OpenTK does not have a method to figure out what display a window would be on. It's not impossible to implement this manually, but should prob. be done at an OpenTK level since it might need direct access to the Win API (OpenTK currently does not even allow public access to the list of available devices, we can only get singular devices by index)

I remember reading a discussion on multi-displays in the OpenTK discord recently. Might be worth reading back.

#1588, #1688, and ppy/osu#2715 on the osu-side implemented this.
The letterboxing stuff should be tracked in another issue.


Even faster NaN checks

Can have strange behavior due to the @fastmath: https://discourse.julialang.org/t/fastest-way-to-check-for-inf-or-nan-in-an-array/76954/30?u=milescranmer
So perhaps it should do a normal sum on each segment instead (less likely to overflow if its just 64 numbers).


zzf/add masked_fill

Motivation and Context
add masked_fill for ascend.
Description

Use cases (Optional)

BC-breaking (Optional)

Checklist
Before PR:

[x] I have read and followed the workflow indicated in the Contributors.md to create this PR.
[x] Pre-commit or linting tools indicated in Contributors.md are used to fix the potential lint issues.
[x] Bug fixes are covered by unit tests, the case that causes the bug should be added in the unit tests.
[x] New functionalities are covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[x] The documentation has been modified accordingly, including docstring or example tutorials.

After PR:

[x] CLA has been signed and all committers have signed the CLA in this PR.

这个类型转化，看看后续能不能写个公共的。


The last tag is sometimes not detected

Describe the bug
Only the first tag is detected when typing tags in a new memo.
However, using the tag button of the editor allows adding multiple tags.
Steps to reproduce

login or refresh (F5) the page
create a new memo with text and several tags
the last tag is not detected and not added to the tag list

Screenshots or additional context

Please follow the best practices to write a tag.
https://github.com/usememos/memos/wiki/Best-practices-to-write-a-TAG

Please follow the best practices to write a tag. https://github.com/usememos/memos/wiki/Best-practices-to-write-a-TAG

ok, look at the top 3 memos of the second screenshot, I think it meets your best practices.

A space is required after the tag

You can treat it as a trade-off for the current version but absolutely not a bug.


MSID: 89812 Version: 2 DOI: 2<IP_ADDRESS>4892

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=89812)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/89812/v2)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>4892)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[x] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/89812/v2/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/89812)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/89812

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/89812

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/89812/v2/89812-v2.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/89812)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Figure S5 isn't displaying despite being present in biorxiv

Nicely spotted Naushin!
As far as I can tell, this looks like an error in the MECA package from bioRxiv. I'll reach out to them now.

Urgh we have version 3 loaded, not version 4. I can see that the file is present for v4 so hopefully all is otherwise good with that version. The authors state in the revision summary that v4 has:

No significant changes

🤷


SQLite3::SQLException: no such column: latitude: SELECT "cities".* FROM "cities"  WHERE (latitude IS NULL OR longitude IS NULL)

Hi,
I'm porting code from an older rails application which used the geocoder gem. There is this City model which uses non default lat/lng columns instead of latitude/longitude. This code used to work fine but now when I run "rake geocode:all CLASS=City" I get the following error:
SQLite3::SQLException: no such column: latitude: SELECT "cities".* FROM "cities"  WHERE (latitude IS NULL OR longitude IS NULL)
Below is what teh model code is and I have also copied the migration code:
class City < ActiveRecord::Base
    geocoded_by :latitude => :lat, :longitude => :lng

end
class CreateCities < ActiveRecord::Migration
def change
create_table :cities do |t|
t.string :name
t.decimal :lat, :precision => 9, :scale => 6
t.decimal :lng, :precision => 9, :scale => 6
  t.timestamps
end

end
end
I'm using ruby 1.9.2 and rails 3.2.1.
Thanks,
Maryam

What version of Geocoder are you using?

I'm using version 1.1.1

I believe the problem is in your model: the first argument to geocoded_by must be a method that returns the address and the second argument is an options hash. You've passed the options hash as the first argument.


NoSuchMethodError when running desktop with 0.0.0-web-dev-13

When running the run target in Compose, it crashes with the error
Exception in thread "main" java.lang.NoSuchMethodError: 'androidx.compose.runtime.internal.ComposableLambda androidx.compose.runtime.internal.ComposableLambdaKt.composableLambdaInstance(int, boolean, java.lang.Object)' 	at ComposableSingletons$MainKt.<clinit>(main.kt:4) 	at MainKt.main(main.kt:4) 	at MainKt.main(main.kt)
I can’t provide much else for a few hours as I am currently not able to use a computer

+1 to @shabunc We weren't able to reproduce this issue using our examples. More details/code/build.gradle will probably help to dig into this.

Ok, I found the issue, It doesn't appear to be an issue with compose, but the error occurred in compose
When I removed my dependency on Decompose, it immediately started working
I believe this is an issue with Decompose.

@CyberDie22 there is a know issue in Decompose: https://github.com/arkivanov/Decompose/issues/150
This will be fixed in when Compose beta 8 comes out, as there is a blocking bug in beta 7.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Unit test cases for bindSecurityGroupServices

More unit test cases and bug fixes in binding services found during unit test runs

@karimull please add description on what bug fixes

Following bugs are fixed that were found during unit test

During a check for ,  size of VirtualSystem List given compare to size of SFC  VirtualSystem  list we throw an exception and in that message the parameters were give wrong.

When checking any existing SFC other than the given SFC to bind connected to a SG and particular Tenant(Redundancy chain check), was getting SFC list from Virtual Systems. But in Add or update of SFC entity services we were not updating SFC list in VirtualSystem Entity.  So added a query instead of getting from virtualsystem sfc entity list, which is more preferred.

When checking for redundant SFC chains in the system, we are querying for virtual system of a given SFC is part of any other SFC. And If they exist another SFC with same VS and if a security group exist with this other SFC and same tenant , we call it a redundant chain. In this check we should not include the current SG that is configured as part of getting all SGs. Fix it.


lint: add fmt task to Makefile

(This is silly but I was fighting my editor while working on the codebase today)

:+1:


feat: now interface for containerd could be inspected

[ ] Make sure you have followed Kube-OVN Code Style.

What type of this PR
Examples of user facing changes:

Features

Which issue(s) this PR fixes:
Fixes #1950
scenario restore:

exec cmds in a chosen ovs-ovn pod as following:
yes | rm /etc/openvswtich/*
yes | rm /var/run/openvswtich/*
rm  /etc/openvswitch/*
reboot this ovs-ovn pod, and wait till the system is stable.
run the bash.

Now this shell is just for the contained cause it's not easy to deploy a docker-based environment.

Github actions failed and need description about how to use this script

(1). MountPath incorrect format due to IDE leads to action failed. Fixed now.
(2). script works as follows:
1. init crictl for runtime containerd/docker in deamonset ovs-ovn
2. list all pods which are not hostnetwork, and filter out pods which are not listed in ovs interface
3. restore interface information for pods filtered in step2(some information is not exactly the same as its initialed)
Just run the script and these steps will be completed automatically.


Recursively check subfolders of the test input pattern

What I want
I want a test features folder that contains both files adjacent to folders that also contain files
E.g.
features/
  |- a.test.js
  |- a-folder/
    |-a.nother.test.js

What I get
To the best of my knowledge, currently I can only have adjacent files or a single linear folder hierarchy
E.g.
features/
  |- a-folder/
    |-a.test.js

or
features/
  |- a.test.js
  |- b.test.js
  |- c.test.js

What happens
The tests are currently loaded form the pattern options, called 'tests'. This is the section of code that globs the pattern and adds each file path it finds to a list of file paths to be loaded
https://github.com/Codeception/CodeceptJS/blob/master/lib/codecept.js#L94-L99
tl;dr? Don;t worry, this is what it looks like
 loadTests(pattern) {
    pattern = pattern || this.config.tests;
    glob.sync(fsPath.resolve(global.codecept_dir, pattern)).forEach((file) => {
      this.testFiles.push(fsPath.resolve(file));
    });
  }

And...
....this is what I propose
loadTests(pattern) {
  pattern = pattern || this.config.tests;
  glob.sync(fsPath.resolve(global.codecept_dir, pattern)).forEach((file) => {
    if (fs.lstatSync(file).isDirectory()) {
      loadTests(fsPath.resolve(global.codecept_dir, `${pattern}/*`));
    } else {
      this.testFiles.push(fsPath.resolve(file));
    }
  });
}

^ Check if the value is a folder, if so resolves the path and recurse through the loadTests method until it reaches a file, then add that file path to the list of file paths to be loaded

@JoeChapman ./tests/**/*_test.js in codecept.json will search tests recurcive in test folder

@APshenkin you are correct, thank you.


[VL][CLEANUP] Cleanup Duplicate CMake code

What changes were proposed in this pull request?
This PR cleans up duplicate code in cpp/velox and cpp/core CMakeLists.txt and moves them to the top level common CMakeLists.txt
How was this patch tested?
Local Build

@PHILO-HE please take a look, I saw that my old PR was already merged before I could apply your suggestion of moving the common code to the top level CMakeLists.txt, I have done the cleanup in this PR. Thanks!

#8041


Revise readme

If this looks good I can merge these new instructions on creating a Maptime chapter via Starter.
cc @lyzidiamond @geografa

:+1: :+1:

Boom. You nailed it. Merge away! :game_die: :jack_o_lantern: :school:


New package: McDM v0.1.0

Registering package: McDM
Repository: https://github.com/jbytecode/McDM
Created by: @jbytecode
Version: v0.1.0
Commit: 58b1cf6fa50cb7c9bb7522f0ea81ac840acb170e
Git reference: main

Superseded by #29535 I guess

Yes, thank you @giordano


Godot editor occasionally doesn't quit cleanly

Godot version:
7a10d3ab78975a95f55b1bf97705ff8cc300da08
OS/device including version:
Linux x86
Wine x86
Issue description:
Sometimes when quitting the editor it doesn't exit cleanly and instead the following message is scrolled infinitely:
ERROR: instance_set_use_lightmap: Condition ' !instance ' is true.
   At: servers/visual/visual_server_scene.cpp:660
ERROR: get: Condition ' !id_map.has(p_rid.get_data()) ' is true. returned: __null
   At: core/rid.h:154
ERROR: instance_set_use_lightmap: Condition ' !instance ' is true.
   At: servers/visual/visual_server_scene.cpp:660
ERROR: get: Condition ' !id_map.has(p_rid.get_data()) ' is true. returned: __null
   At: core/rid.h:154

Steps to reproduce:
It varies but it seems to happen a lot when I:

Select a light node in the sponza scene
Delete it
Undo
Quit the editor

Minimal reproduction project:
Not exactly 'minimal' but it seems to trigger this behavior relatively often:
https://tmm.cx/nextcloud/s/Log1eAXen1dJzBz/download?path=%2F&files=sponza.zip

wine godot on linux?

Yes, I tested both on Linux natively and wine.

Having the same on Windows very rarely (at least if recall the exception correctly).

I haven't seen this particular error message about lightmaps when closing the editor, so I'll close this. Please comment if you can still see this error message (not any others).


Update ODE Webpage

Fixes #541

Please make sure that all the checks pass. Please add here any additional information regarding this pull request. It's highly recommended that you link this PR to an issue (please create one if it doesn't exist for this PR)

@pdelboca here are two screenshot options:

@roll Whenever you can, can you please take a look at this PR?
Contextual info:

We had a call with Pato and we agreed we will use one of the screenshots that I provided in my last comment to keep the same style that we already have on the website.
In addition to that, please remove this image from the website:

Apart from that, I think there rest of the changes were already implemented.


Unable to open Disassembly View when webview focused

Does this issue occur when all extensions are disabled?: Not really, since debugging with the disassembly view requires at least some extensions. But I'm quite confident that plugins are not the problem.

VS Code Version: 1.83.0
OS Version: Windows 10

Steps to Reproduce:

Start a debug session that supports disassembly and hit a breakpoint.
Focus some webview (even the Welcome View will do).
Run the Open Disassembly View command from the Call Stack view.
Nothing happens.

The problem appears to be in the fact that the Open Disassembly View command extends EditorAction2:
https://github.com/microsoft/vscode/blob/33876c27f082973adbddc027e0e532c3648099dd/src/vs/workbench/contrib/debug/browser/debugEditorActions.ts#L199
Which means that it only runs if one of these fields is defined:
https://github.com/microsoft/vscode/blob/33876c27f082973adbddc027e0e532c3648099dd/src/vs/editor/browser/editorExtensions.ts#L455-L459
When a webview is focused, line 455 returns null, so the command execution bails. I don't really see a reason for the Open Disassembly View command to care about the focused editor: none of its preconditions refer to anything to do with editors at all.

Not sure why it's an editor action since the only thing it does with the editor is check editor.hasModel()... which isn't necessary for disassembly to work. Can be made an Action2. PR welcome.


feat: call graph computer (without closures)

Summary of Changes
Implement a first version of the call graph computer that can handle everything but closures. These require the partial evaluator and will be done later.

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

npm package (@latest dist-tag)
npm package (@latest dist-tag)
v0.4.0
GitHub release

Your semantic-release bot :package::rocket:


Code Bug in the Page 397

Hi,
IN the page 397 while training a Model as follows:
batch_size = 2
num_epochs = 100
loss_hist = [0] * num_epochs
accuracy_hist = [0] * num_epochs

for epoch in range(num_epochs):
    for x_batch, y_batch in train_dl:
        pred = model(x_batch)
        loss = loss_fn(pred, y_batch)
        loss.backward()
        optimizer.step()
        optimizer.zero_grad()
        loss_hist[epoch] += loss.item() * y_batch.size(0)
        is_correct = (torch.argmax(pred, dim=1) == y_batch).float()
        accuracy_hist[epoch] += is_correct.mean()
    loss_hist[epoch] /= len(train_dl.dataset)
    accuracy_hist[epoch] /= len(train_dl.dataset)

plt.plot(accuracy_hist, lw=3)

here batch size is 2 so there will be 50 inner cycle and 100 epoch for each cycle. In side the inner cicyle is_correct.mean() does not looks correct, it should be is_correct.sum() otherwise  accuracy_hist[epoch] /= len(train_dl.dataset) needed to be cut half  accuracy_hist[epoch] /= len(train_dl.dataset)/2 as you already have averaged before 2 values into one.

Thanks for the note and sorry for the confusion! I believe this is something that I adjusted here in the code a while back: https://github.com/rasbt/machine-learning-book/blob/main/ch12/ch12_part2.py#L295
Will have the publisher update the  book version as well this week! Thanks!


⚠ Spryr has degraded performance

In 59827d9, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 5000 ms

Resolved: Spryr performance has improved in 3cdf88b after 7 minutes.


Choose the tehnologies to be used for the app

Should we use a very simple HTML page or add some JavaScript scripts?

No need to add JS just yet. Lt's just stick to HTML5 and some styles.
@25green28 will lead the developmen.


shade ignores transparency

the Shade plugin itself works very well but I've enabled transparency in my neovim and it seems Shade disables that transparency and changes it to a black background.

+1

+1


Error in handleChange with Undefined nodes on Aurelia-Gridstack v3.0.0 & v4.0.0

Description
Encountering a runtime error in the handleChange method within aurelia-gridstack. The issue is observed both in version 3.0.0 and persists after updating to version 4.0.0. The error occurs when handleChange is triggered with an undefined or empty nodes parameter.
Steps to Reproduce

Use grid-stack in an Aurelia component with bindable properties.

<template>
    <grid-stack float options.bind="gridOptions" ref="gridStackEl">
        <grid-stack-item //someoptions>
            <input type="date" date.bind="fromDate">
        </grid-stack-item>
    </grid-stack>
</template>

Select a date and observe that changes in unrelated bindable properties seem to trigger handleChange in grid-stack.

Expected Behavior
handleChange should not be triggered by changes in unrelated bindable properties, or it should be able to handle undefined nodes without throwing an error.
Actual Behavior
handleChange is triggered with an undefined nodes parameter, resulting in a runtime error.
Additional Context
The issue appears to be related to how Aurelia's binding system interacts with grid-stack. Removing certain bindings (e.g., date.two-way="fromDate") prevents the error, suggesting a possible conflict in change detection or event handling.
Environment

Aurelia Framework Version: 1.4.1
Aurelia-Gridstack Version: 4.0.0 (issue also present in 3.0.0)
Browser: Chrome
Language: TypeScript

Any insights or suggestions on this issue would be greatly appreciated.

Looks like this issue belongs to the gridstack repository
On Fri, 26 Jan 2024 at 8:58 am, andreitaranu @.***>
wrote:

Description
Encountering a runtime error in the handleChange method within
aurelia-gridstack. The issue is observed both in version 3.0.0 and
persists after updating to version 4.0.0. The error occurs when
handleChange is triggered with an undefined or empty nodes parameter.
Steps to Reproduce

Use grid-stack in an Aurelia component with bindable properties.

Select a date and observe that changes in unrelated bindable
properties seem to trigger handleChange in grid-stack.
image.png (view on web)
https://github.com/aurelia-ui-toolkits/aurelia-gridstack/assets/100191890/162c0e0a-a983-4177-8cde-7290513af8e8

image.png (view on web)
https://github.com/aurelia-ui-toolkits/aurelia-gridstack/assets/100191890/47b9a74f-30e5-439a-a25c-0f79dbb7ea57
Expected Behavior
handleChange should not be triggered by changes in unrelated bindable
properties, or it should be able to handle undefined nodes without
throwing an error.
Actual Behavior
handleChange is triggered with an undefined nodes parameter, resulting in
a runtime error.
Additional Context
The issue appears to be related to how Aurelia's binding system interacts
with grid-stack. Removing certain bindings (e.g., date.two-way="fromDate")
prevents the error, suggesting a possible conflict in change detection or
event handling.
Environment

Aurelia Framework Version: 1.4.1
Aurelia-Gridstack Version: 4.0.0 (issue also present in 3.0.0)
Browser: Chrome
Language: TypeScript

Any insights or suggestions on this issue would be greatly appreciated.
—
Reply to this email directly, view it on GitHub
https://github.com/aurelia-ui-toolkits/aurelia-gridstack/issues/12, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4KNPCO7A73DJNAZJT5U3DYQLIO3AVCNFSM6AAAAABCLHJJTCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEYDCMRTGQZTOMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you for the quick answer, I've reposted the issue on the gridstack repository, I guess this can be closed, if it's not relevant for the aurelia-gridstack repository.

@MaximBalaganskiy The answer I've got in the gristack repository points out that the issue is more related to the aurelia-gridstack interactions, so it would be great if you could provide a bit of help with this 🙂


Lists, concept of select

The same confusion and issues on select of DF, exists on List too.
For more info: https://github.com/CIT-UAI/geotypes/issues/6

fixed by https://github.com/CIT-UAI/geotypes/pull/9


Should a light session be tolerant to failures?

I'm reading and writing using a light_session -- the leader is node_1 and everything is zipping along fine -- reads and writes.  However, if node_1 is partitioned and then crashes, I see the leader transition to node_3, but writes using the light_session do not seem to complete?  When I debug the messages being sent, no matter which request I issue the read from, I always see the request go to node_1, which was that last known leader, who is no longer online.
The documentation seems to indicate that, with the light session configured to know about all peers, that a request would, if timed out, be rerouted to another node in the cluster.  Is this how it should work?

Seems like not a lot of issues get responded to in this repository (open since 2015) - maybe you’d be better off trying https://github.com/rabbitmq/ra https://github.com/rabbitmq/ra ?

On 6 May 2019, at 22:46, Christopher S. Meiklejohn<EMAIL_ADDRESS>wrote:
I'm reading and writing using a light_session -- the leader is node_1 and everything is zipping along fine -- reads and writes. However, if node_1 is partitioned and then crashes, I see the leader transition to node_3, but writes using the light_session do not seem to complete? When I debug the messages being sent, no matter which request I issue the read from, I always see the request go to node_1, which was that last known leader, who is no longer online.
The documentation seems to indicate that, with the light session configured to know about all peers, that a request would, if timed out, be rerouted to another node in the cluster. Is this how it should work?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/dreyk/zraft_lib/issues/33, or mute the thread https://github.com/notifications/unsubscribe-auth/AHUCR5XEKQLVMZLZPDSVLWDPUCRKFANCNFSM4HLDVDLA.

We're testing a variety of software as part of a research project, and this was one of the libraries we selected.

@cmeiklejohn Yes. light_session should be tolerated to a node fails. It knows about all peers and should try to use other nodes to get a new leader.


docs: correct virtual environment information

Description
isinstance(venv, virtual_environment) and
isinstance(virtualenv, virtual_environment) and
venv != virtualenv

Indeed, nice catch!
Thank you for your contribution @tdadela!


Changelog bump for 1.5.1 release

This PR is step two of the release process, but I'm making a PR because the changes are all mangled (we kept adding to 1.5.0 even after its release, when new entries should have gone in an Upcoming section). I've tried to reconcustrct the changelog based on this diff but I want a second set of eyes.

Ah, my apologies! I'm :+1: on merging if others have no objections?

No need to apologize! I don't mind waiting to hear from others.

Yep


Use local buffer on stack.

This allows I2c to be send across threads.
Previously it failed with

*mut u8 cannot be sent between threads safely

because a *mut u8 pointer is part of the boxed i2c_msg buffer within I2c. This buffer is now removed and instead i2c_transfer() creates a local buffer on the stack.

Seems reasonable, the buffer should only be like 500 bytes on the stack? I'd be inclined to use mem::uninitialized() instead though, considering that the whole buffer will be zeroed on every call.


more of a question than issue regarding C 3.11 and vnodes

ok, so i have a small cluster, however from the reading it seems priam is not ready for 3.11 using vnodes, is this true? and if so any idea when it will be supported?
Thanks!

We are currently working on 3.11 branch for Priam. I am shooting for somewhere about mid-August.

thanks!


Add openshift ansible crio pr test

@stevekuznetsov is this necessary or can we just add the prow config to call the existing crio job?
CC: @runcom @ashcrow

No, we need this or we won't be syncing an O-A PR

So LGT @stevekuznetsov  ?

Sure -- you will need to add the config to run it, too -- it should not be blocking today (the origin version is at like <10% success rate ATM)


Avoid extra String allocation in impl Display for LapceWorkspace

Codecov Report

Merging #1875 (dbf7afd) into master (41f7a2a) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master   #1875   +/-   ##
======================================
  Coverage    8.50%   8.50%
======================================
  Files         129     129
  Lines       56466   56465    -1
======================================
  Hits         4805    4805
+ Misses      51661   51660    -1

Impacted Files
Coverage Δ

lapce-data/src/data.rs
0.00% <0.00%> (ø)

lapce-ui/src/settings.rs
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Scroll bar and minimap arrive at the start / end before reaching the first or last line

VSCode Version: 1.11.0 - insiders
OS Version: Windows 10

-- Sorry for the English, I used Google Translator --
Steps to Reproduce:
Settings "editor.wordWrap": "on"
1.Take a file with many lines. This example was using angular.js 1.6.3
2.Scroll down the page using the minimap or the scroll bar to the end.
3.Note that the minimap or scrollbar has come to an end, but we still have more rows.
4.Go back to the beginning using the minimap or the scroll bar to start.
5.See that the minimap or the scroll bar has reached the beginning but we still have more lines.
Atom works properly.
Note: I know that VSCode has an empty space at the end in the files (this is cool), I believe this has no influence.

For very large files we artificially inflate the height of the slider such that it is easy to grab via the mouse.
This is implemented in a quite crude way, where the slider height is inflated in both vertical directions (up and down). This might lead to the slider appearing to be all the way to the bottom, even if this is not the case.

Duplicate of #6710

Thank you so much @alexandrudima . 👏 👏 😄


How does gnark start GPU acceleration?

I am at gnark/.../example, a main. go was created, such as the following code:
package main
import (
"github.com/consensys/gnark-crypto/ecc"
"github.com/consensys/gnark/backend"
"github.com/consensys/gnark/backend/groth16"
"github.com/consensys/gnark/frontend"
"github.com/consensys/gnark/frontend/cs/r1cs"
)
// CubicCircuit defines a simple circuit
// x**3 + x + 5 == y
type CubicCircuit struct {
// struct tags on a variable is optional
// default uses variable name and secret visibility.
X frontend.Variable gnark:"x"
Y frontend.Variable gnark:",public"
}
// Define declares the circuit constraints
// x**3 + x + 5 == y
func (circuit *CubicCircuit) Define(api frontend.API) error {
x3 := api.Mul(circuit.X, circuit.X, circuit.X)
api.AssertIsEqual(circuit.Y, api.Add(x3, circuit.X, 5))
return nil
}
func main() {
// compiles our circuit into a R1CS
var circuit CubicCircuit
ccs, _ := frontend.Compile(ecc.BN254.ScalarField(), r1cs.NewBuilder, &circuit)
// groth16 zkSNARK: Setup
pk, vk, _ := groth16.Setup(ccs)

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

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

}
errors：
/usr/local/go/pkg/tool/linux_amd64/link: running gcc failed: exit status 1
/usr/bin/ld: cannot find -lbn254: No such file or directory
collect2: error: ld returned 1 exit status
Should I download the article first and then compile it?
Finally, to gnark/.../example and to execute "go run -tags=icicle main.go"
What should I do?

Hi, indeed setting up the GPU support is a bit cumbersome and depends on the ICICLE library being set up correctly. Have a look at https://github.com/ingonyama-zk/icicle/tree/main/wrappers/golang on how to build ICICLE.


nixosTests.tinywl: fix by adding Mesa drivers

Description of changes
Required since upstream commit 49738406 ("render: don't fall back to Pixman if we have a render node").
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg test tinywl


Fail to run because importing error

The document is kind of wrong and we fail to run with python ./scripts/policy_gradient.py.
The error log looks like this.

Please make sure the import path is right.

Hi @tobegit3hub
Thanks for getting in touch. I've just tried running the command "python ./scripts/policy_gradient.py" from the root directory of the project and it works after doing a fresh git clone. Is there anything else that could have gone wrong? Does the module exists in games/tic_tac_toe.py?
I have thought about maybe moving the scripts into the root directory to avoid this kind of problem. What do you think?

Thanks @DanielSlaterSK for your response but I still fail to run. I'm not sure if your have "install" or do something to import this.
It would be great to move the script in root directory so that everyone can run it. Or modify the script to import the the game library in relative path.

I've moved the scripts to the root directory, does it work for you now?

Thanks @DanielSlater but another error occurs.

There will be a file current_network.p in the root directory, delete that and then re-run, also do a git pull because I've fixed some stuff.

Everything works like a charm now. Thanks @DanielSlater for supporting 👍

No problem :)


LINK-1742 | Registration creator can manage signups

Description
Allow admin users who has created the registration to manage signups and signup groups of the registration
Closes
LINK-1742

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

Comparison is base (3704f0b) 99.73% compared to head (5d5b6f7) 99.73%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #280   +/-   ##
=======================================
  Coverage   99.73%   99.73%
=======================================
  Files         590      591    +1
  Lines        9744     9758   +14
  Branches     2149     2161   +12
=======================================
+ Hits         9718     9732   +14
  Misses         23       23
  Partials        3        3

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


please set cache headers

Thank you for this package, it makes setting up a vanity import path pretty easy indeed.
I noticed that no cache headers are set in any responses:
% curl -v <url>
< HTTP/2 200
< content-type: text/html; charset=utf-8
< x-cloud-trace-context: e11dd858ab85c26bd488a27ac5deb139;o=1
< date: Sun, 24 Sep 2017 07:38:12 GMT
< server: Google Frontend
< content-length: 566
< alt-svc: quic=":443"; ma=2592000; v="39,38,37,35"
<
<!DOCTYPE html>
[…]

When using CloudFlare in front of AppEngine, it is beneficial to set caching headers to reduce traffic to AppEngine. Currently, every request needs to go to AppEngine, even though the response will be exactly the same.
I suggest that cache headers should be included with every response, with the default cache time being 24 hours or so (configurable). This should be a good default, given how rarely the config changes in practice.
See https://support.cloudflare.com/hc/en-us/articles/202775670-How-Do-I-Tell-Cloudflare-What-to-Cache- for which headers CloudFlare respects.
Thank you for considering.

Maybe we should let people set arbitrary headers to be included in the responses. It can be provided in the app.yaml. By default, we can provide a cache header with a reasonable caching time. @zombiezen, any thoughts?

SGTM.  Since the caching will be tied pretty directly with the configuration file, it seems like it ought to be the same value for each import path.
Happy to take up the issue if you want to assign it over to me.  Change looks straightforward.


addJSXCommentCommand doesn't seem to work

With
monaco-editor (0.48.0),
@monaco-editor/react (4.6.0),
@babel/parser (7.24.6),
@babel/traverse (7.24.6),
addJSXCommentCommand doesn't seem to work:
pressing " cmd / ctl " + "/ " on the line with the jsx-component or element, still results in "//  " being added at the start of the line.

I just played with your example on codesandbox.io and after updating @babel/parser  to 7.24.6 jsx commenting stops working.

Same with:
"@babel/parser": "7.24.7",
"@babel/traverse": "7.24.7",
"@monaco-editor/react": "4.6.0",


Immediately apply some shift by default (on load)

Hi,
It would be nice if immediately apply some shift by default (on load)
Mode1:
Set default shift (constant)
Mode2:
Apply shift which matches with system hour:
For example
s1 > 00:00 - 08:00
s2 > 08:00 - 16:00
s3 > 16:00 - 24:00
System time is 07:00 -> apply s1
System time is 15:55 -> apply s2
System time is 22:15 -> apply s3

and apply default (or system time) shift on dateSelect

Hi Bauyrzhan,
There is a new feature in the new release. You can enable "realtime tracking" that will meet your expectations I think.
When realtime is enabled, it checks the panel every determined interval time which shift the now is in and applies it.

With the release of 0.0.3 this feature will be available.

Real-time Tracking is very useful in kiosk mode!
But in my case we needs auto select shift (auto select - according to system time, not time picker) on dashboard load only (first open) then the user should be able to select other dates and shifts for analytics

This should be fixed in version v0.1.0. Feel free to reopen if it's not exactly what you need.

This should be fixed in version v0.1.0. Feel free to reopen if it's not exactly what you need.

What exactly was fixed or added here? I too have an issue where my date does not default to today. I'm fine with manually picking my shift to be fair. I'm kind of looking for the realtime shift selector but with option to check previous days+shifts


main blog doesn't download, side blog worked fine

I logged into Tumblr, added my blog and ticked all the boxes I wanted in blog settings but nothing happened. Tumblthree shows that the blog is online and the correct number of downloads. To check if I did something wrong I added my sideblog which worked almost immediately.
Things I tried:
Removing/re-adding the blog from the list + deleting the index files
different blog settings
logging off/in of Tumblr
The screenshot shows my last attempt.

At first it didn't give the option to add it to the queue. It worked worked when I right clicked on the blog and clicked "enqueue selction" (I didn't know I had to do that).
Thanks for the fast support!
Btw I was pretty sure that this particular blog had original posts because it's my own :)


Move Emissions Compliance Stream endpoint to Stream API

Definition of Ready

[x] Does this meet the INVEST criteria?
[x] Does this have a clearly defined user?
[x] Does this have an estimate?
[x] Does this have acceptance criteria?
[x] Does this have sufficient context for the specific functionality being developed (design, business rules, etc.)
[x] Does this have a dev link been provided in the design?
[x] Is this traceable to a feature, epic, or PI?

User Story
As a CDD user I want the emissions compliance streaming endpoints to be located in the Stream API So that I can access my file download faster
Acceptance Criteria
Given that a user has entered one or more valid values for all the parameters (see parameters below)
When API endpoint is executed
Then the system displays the records that match the criteria (containing data elements below)
Given that a user has entered one or more valid values for all the parameters in an upper case or lower case format
When API endpoint is executed
Then the system displays the records that match the criteria
Given that a user has entered one or more valid values for all the parameters without white space(s) before or after the delimiter
When API endpoint is executed
Then the system displays the records that match the criteria
Given that a user has entered one or more valid values for all the parameters with white space(s) before or after the delimiter
When API endpoint is executed
Then the system displays the records that match the criteria
Given that a user has made a GET request with the media type set to json or csv
When request is executed
Then the system displays the records that match the criteria in the appropriate format (json or csv)
Given that a user has entered one or more invalid values for a parameter
When the API endpoint is executed
Then the system displays the appropriate error message (see error message repo)
Given that a user has previewed emissions compliance  data on the cdd
When the download functionality is selected
Then the system produces a file download in the appropriate file type (csv or json)
Development Notes
Parameters
Year: Optional, multiple values allowed, Format: YYYY, Range: 1996 to current year , Allow indiv. Years, ranges, and comma sep. list
Facility ID (ORIS code): Optional, multiple values allowed
Owner/Operator: Optional, multiple values allowed, query will return all records that match owner OR operator specified
State: Optional, multiple values allowed, 2 letter postal abbreviation
Data Elements
Results (applicable to API and UI):
When the system identifies at least one record that matches the criteria, then for each record the system will display the data below and ordered by PRG_CODE > OP_YEAR > ORISPL_CODE > UNITID:
OP_YEAR
FACILITY_NAME
ORISPL_CODE
UNITID
OWN_DISPLAY
STATE
COMP_PLAN_TYPE_CD
AVG_PLAN_ID
EMISS_LIMIT_DISPLAY
ACT_EMISS_RATE
AVG_PLAN_ACT_EMISS_RATE
IN_COMPLIANCE
Validations:
Year:
Validate that the date is in the format YYYY
Validate that all years between 1996 to current year
Error Message 2: One or more < insert property > are not in the < insert format >. Ensure all are in the < insert format > FV-102
Error Message 3: Update one or more < insert property > to < insert valid range > FV-105
Facility ID (ORIS code):
Validate that the value is a number
If the value is a number, validate that the format is correct
positive integer (no negative numbers, not 0)
whole numeric value (no letters or special characters)
no more than 6 digits
Error Message: One or more < insert property > are not valid. Refer to the list of available < insert property > for valid values [placeholder for link to endpoint] FV-100
Owner/Operator:
No Validations
State:
Validate that the value matches a valid 2 letter postal abbreviation for at least 1 of the 50 states, PR, or DC
Error Message: One or more states are not valid. Use the two letter postal abbreviation (use TX, not Texas). FV-101
Does this Story require 508 Testing?

[ ] Implementing a new authoring tool that will produce code / content for the application
[ ] Implementing a 3rd party widget that will be part of the application
[ ] Adding new / updating custom functionality
[ ] Adding new / updating a page template
[ ] Adding new / replacing existing functionality (unless the replacement has already passed 508 testing)
[ ] Full Site Review prior to a scheduled EPA 508 review
[ ] Planning a video that will appear on the application
[ ] Producing a video that will appear on the application
[ ] Producing Training materials
[ ] Producing documents

Context
(User journeys, design mockups, architectural diagrams, business rules, data element definitions)
Testing
Test Case: TC734
Tester: @JanellC
Definition of Done

[x] Have all Acceptance Criteria been agreed to and validated by the scrum team?
[x] Has the code been unit tested and peer reviewed?
[x] Have the functional tests been executed?
[x] Have all defects been reviewed and dispositioned, resolved, or deferred?
[x] Is the user story ready to be deployed to the test/staging environments?

TC734 passed @tlageman-epa


pinentry-mac fails to build

Issue description
When trying to build pinentry-mac, besides #26774, I also encounter a different build failure:
% nix-build '<nixpkgs>' -A pinentry_mac
these derivations will be built:
  /nix/store/c2w9mx7gw9wak253gamvyy1mnvsz7dxq-pinentry-mac-0.9.4.drv
building path(s) ‘/nix/store/z0bxm8lb6l4cm43p0inkhrmrhbcx3jjw-pinentry-mac-0.9.4’

<...>

** BUILD FAILED **

The following build commands failed:
    Ld ./pinentry-mac-ddraybfelawzybdskxmwfbpamtbf/Build/Products/Release/pinentry-mac.app/Contents/MacOS/pinentry-mac normal x86_64
(1 failure)
make: *** [Makefile:9: build/Release/pinentry-mac.app/Contents/MacOS/pinentry-mac] Error 1
builder for ‘/nix/store/c2w9mx7gw9wak253gamvyy1mnvsz7dxq-pinentry-mac-0.9.4.drv’ failed with exit code 2
error: build of ‘/nix/store/c2w9mx7gw9wak253gamvyy1mnvsz7dxq-pinentry-mac-0.9.4.drv’ failed

Steps to reproduce
The command above.
Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)
macOS 10.12.6 (16G29)
Nix version: (run nix-env --version)
nix-env (Nix) 1.11.14
Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)
18.03pre114401.017561209e
Sandboxing enabled: (run grep build-use-sandbox /etc/nix/nix.conf)
true

/cc @matthewbauer

I think this is fixed in #28868

Looks like it, thanks!

https://hydra.nixos.org/build/59974289

It looks like that failure is related to #26774, not this one.

Fixed in nixpkgs-unstable.


Framework - Routing

This includes login/logout, clicking triggers redirects to available games, My Games and account page.

instead of testing with login/logout because that has its own workflow, the front end has basic front end navigation via links to display routing


DB Schema migration

This is an open issue to track our db schema change.
voteview_rollcalls

[x] synthID to date_chamber_rollnumber
[x] result to vote_counts
[x] votes: {id, v, p} to votes: {icpsr, cast_code, prob}
[x] Add prob to votes @lukesonnet
[x] resultparty to party_vote_counts -- and keys change from party labels to party numbers
[x] code to codes
[x] nay to nay_count
[x] yea to yea_count
[x] pre and classified fields missing from nominate @lukesonnet
[x] keyvote to key_flags
[x] support to percent_support
[x] shortdescription to short_description
[x] New description waterfall: vote_desc, vote_document_text, description, short_description, vote_question, question
[ ] Fix indices @adamboche

voteview_members

[x] stateAbbr to state_abbrev
[x] bioName to bioname (lowercase)
[x] party to party_code
[x] districtCode to district_code
[x] bio to biography
[x] stateName, cqLabel, startdate, lastmeans, occupancy, state, partyName, geo, enddate, dimweight, stateCode ALL REMOVED -- verify OK
[x] Add born and died @adamboche
[x] Crosswalk social fields @aaronrudkin
[x] wikiStatus to wiki_status
[x] Nominate: oneDimNominate to dim1; geoMeanProbability to geo_mean_probability; numberOfVotes to number_of_votes; twoDimNominate to dim2; logLikelihood to log_likelihood; numberOfErrors to number_of_errors.

Reimport voteview_parties
Web code check

[x] Search votes rewired
[x] Download votes rewired
[x] Vote view page rewired
[x] Member bio page rewired
[x] Congress page rewired
[x] Parties pages rewired
[x] All code pointing to test_voteview
[x] Drop voteview db and renamed test_voteview to voteview
[x] All code pointing to voteview
[ ] Deployed to production server

Search votes is rewired, although it now is showing votes for rollcalls that don't have nominate fields, which used to be hidden. Default should be hide these from the search page because we should at least have conditionally estimated nominates within a few hours of any new votes being added to the db.
The vote pages work (if the nominate data exists). However, the links to member pages are broken.
Please note that I wrote a generic function to waterfall in views/search_list.tpl called waterfall. That might need to go somewhere else if we want to use it for the vote page as well.

Okay, I'm updating the original post here with the field names we need to fix.

This issue should also not be closed until all of the user-facing documentation is updated.

I believe we have two outstanding index issues here:

The fulltext index was set up for just description. I corrected this, but the build script should probably be updated to reflect the actual intent.
The sparse index for key_flags is not set up at all.

Other than that the schema move was a success.

The sparse index thing is still weird--I don't think it can be done in pymongo, so we overrode it with a javascript file that mongo runs directly in build_edits.sh.
@adamboche Where are the main indexes being built now? Probably somewhere in ingestion? We should ensure we all know where so as we update our fulltext indexing strategy it's all set up properly.

Post-launch, we need to deal with the fulltext thing.

I think I know how to do the fulltext thing. Will report back.
On Sun, Jan 8, 2017 at 10:37 PM, Aaron Rudkin<EMAIL_ADDRESS>wrote:

Post-launch, we need to deal with the fulltext thing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JeffreyBLewis/WebVoteView/issues/22#issuecomment-271219230,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AS6Qi-wQUHrIXVf1IpEckHAUsr0WxZElks5rQdWPgaJpZM4J9hUU
.

I've set up a wildcard text index in rollcalls and members, so all string
fields should be indexed automatically as they are added.
On Sun, Jan 8, 2017 at 10:40 PM, Adam Boche<EMAIL_ADDRESS>wrote:

I think I know how to do the fulltext thing. Will report back.
On Sun, Jan 8, 2017 at 10:37 PM, Aaron Rudkin<EMAIL_ADDRESS>wrote:

Post-launch, we need to deal with the fulltext thing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JeffreyBLewis/WebVoteView/issues/22#issuecomment-271219230,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AS6Qi-wQUHrIXVf1IpEckHAUsr0WxZElks5rQdWPgaJpZM4J9hUU
.

Isn't this going to index a massive amount of garbage?


Merge of two internal changes

175695370  by A. Unique TensorFlower:
Implements _shared_embedding_columns and adds some tests.

--
175695349  by A. Unique TensorFlower:
Implements tf.metrics.true_negatives, adds missing tests and does

some cleanup in tf.contrib.metrics.
(This is 2 of the 3 commits in from staging
c674e27bfd68a6c990e694b6afd901bfeeaa006d)

Pushed all internal changes out already.


Rust bindings: expose safe blst_scalar constructors

In the Rust bindings, in order to use Signature::verify_multiple_aggregate_signatures you must initialize some blst_scalar values.
AFAIK, the only way to do this is via the unsafe blst_scalar_from_* functions.
It would be great if the bindings could wrap these in a non-unsafe function, if they are indeed as safe as all the other times we call the unsafe extern "C" functions.

It seems that verify_multiple_aggregate_signatures could accept something less "rigid" than blst_scalar. At least blst_scalar is quite wasteful, because rand_bits would be significantly smaller than blst_scalar bit width, right? It might take some extra interfaces, but it might be worthwhile to arrange. Alternatively one can argue that just a sequence of random bytes cast to limbs would do [as input to the subroutine in question]. Indeed, on little-endian platform lendian->limbs conversion is effectively a no-op anyway, and on big-endian "randomness" would be "applied" in reverse byte order, which is arguably as random.
This is not to say that it would inappropriate to wrap unsafe functions! Just that it seems to be an extra angle to the problem.

on big-endian "randomness" would be "applied" in reverse byte order, which is arguably as random.

Just in case, this is naturally assuming that if rand_bits is not divisible by limb's bit-witdth, the byte sequence is not conditioned accordingly. I mean that no byte is masked with few-bit mask.


fix: make UriTemplateStr repr transparent

The comments in the code imply that UriTemplateStr should have a transparent data layout, but the struct definition doesn't have the #[repr(transparent)] attribute.

Thank  you, this is very important fix!
Published v0.7.5 with this change.

MEMO: The type has been added since v0.7.0, so the fix is not needed for v0.6.x or below.


DNS bypass

It would be nice if there were a way we could bypass DNS resolution and connect directly to an IP address we specify during the request.  Something like...
client.skip_dns('<IP_ADDRESS>').get("https:/google.com").send()
SNI / cert validation could still work off the domain in the URI.

That seems like a fair desire, unsure about the best API to expose.

So I toyed with this a little by modifying the async_impl/request struct like:
pub struct Request {
    ip: Option<std::net::Ipv4Addr>,
    method: Method,
    url: Url,
    headers: HeaderMap,
    body: Option<Body>,
}

and then tried compiling / updating over and over again, making the necessary changes at each step to get past whatever errors were introduced.  I tried to stay true to how the rest of the library treated an instance of a Request at each step, but I got stuck at...
C:\dns_bypass (master -> origin)λ cargo build
   Compiling hyper v0.12.27 (C:\dns_bypass\.cargo\hyper)
error[E0308]: mismatched types
   --> .cargo\hyper\src\proto\h1\dispatch.rs:464:47
    |
464 |                         Ok(Async::Ready(Some((ip, head, body))))
    |                                               ^^ expected struct proto::MessageHead, found struct http::Ip
    |
    = note: expected type proto::MessageHead<proto::RequestLine>
               found type http::Ip

I didn't think it should be merged into request "headers" as its rather part of the IP header, but taking this approach meant I needed to modify the h2, hyper, and http libraries as well, and possibly more?

Before you go to far down that rabbit hole, I think it'd be best to think about the exposed API to the user. The implementation likely doesn't need to bother touching h2/hyper/http, and would probably just adjust reqwest's connect.rs stuff...

Ahh.  Well it's too late for me I already went down the rabbit hole!  Anyhow, I think you're right it's best to avoid changes on any dependent libraries.  I'll try again sometime this week hopefully and report back.
For the API, I already don't like .skip_dns() because "DNS" is just one of the ways by which an IP could be resolved.  I do like that the default behavior is to attempt resolution, since I believe the users of reqwest would prefer not to worry about it.
Perhaps skip_resolution would be better?  I still like 'skip', since it implies that the standard behavior is to attempt IP resolution, and shows that by calling that function, no subsequent resolution would occur.
I don't think support for non-IP protocols would be anything in-scope for this library, so accepting an std::net::IpAddr would be best?
Having anything more than a .skip_resolution(std::net::IpAddr) builder method to support this behavior, I think, would be unnecessary.

Does curl and other libraries provide an easy option to do this?

curl has --resolve <host:port:address> Resolve the host+port to this address
wget does similar by over-riding the host header, but I imagine that isn't going to be compatible with SNI?

Maybe I was confused, I'm looking now at the reqwest source and I don't see how to set my own resolver on either the client or the request.  Was that part of hyper?

@seanmonstar Can you reopen this issue? Specifying custom DNS resolution is quite helpful.

I was the one that closed it, re-opening now.

A quite useful feature.

That's really useful, especially for us to bypass DNS spoofing.

It seems that it involves modifying hyper’s source code for per-request DNS bypassing to work.

I think that the API presented to users for this could also be useful for #39.

I've played around with an implementation of this and it's definitely bubbling up to cause some ugliness so I think there are some choices to make.
Currently the HttpConnector is an enum with either the GaiResolver or the TrustDNS resolver. Here's my attempt to make a generic wrapper resolver that would allow a user to add specific overrides:
    async fn new_wrapping_resolver<R, U>(
        mut wrapped_resolver: R,
        overrides: std::sync::Arc<HashMap<String, SocketAddr>>,
    ) -> impl Service<Name, Response = impl Iterator<Item = SocketAddr>>
    where
        R: Service<Name, Response = U> + 'static,
        U: Iterator<Item = SocketAddr>,
    {
        type EitherOverrideOrWrapped<U> = itertools::Either<std::iter::Once<SocketAddr>, U>;
        tower::service_fn(move |name: Name| {
            if let Some(overridden) = overrides.get(name.as_str()) {
                return futures_util::future::ok::<EitherOverrideOrWrapped<U>, R::Error>(
                    EitherOverrideOrWrapped::Left(std::iter::once(*overridden)),
                )
                .left_future();
            } else {
                return wrapped_resolver
                    .call(name)
                    .map(|result| result.map(|iter| EitherOverrideOrWrapped::Right(iter)))
                    .right_future();
            }
        })
    }

This is the first time I've written this kind of futures + iterators code so this may be unidiomatic (happy to rework if there are better options).
The implication of this though is that the resolver becomes a new unnamed type, and there is an overhead for clients that are not using overrides at all.
Other options for a design here that I can see:

A new WrappedResolver Struct generic over the resolver. Then two new constructors for HttpConnector with names new_gai_with_overrides() and new_trust_dns_with_overrides(). Then a third variant of HttpConnector with a generic inner type. I don't know enough about how the different resolvers are used to know whether this has more implications and this means two new functions per resolver being added.
Make HttpConnector a generic new_type over hyper::client::HttpConnector rather than an enum. This would simplify connect.rs (I think) but would presumably impact the rest of the crate.

If anyone has any input on how this could be cleanly implemented I'm happy to take a swing and get a PR up.

As for user API my suggestion would be:
    reqwest::Client::builder()
        .resolve("example.com", "<IP_ADDRESS>")
        .resolve("example2.com", "<IP_ADDRESS>")
        .build()?


Replace dl-icon with clearly labeled button

https://github.com/usdigitalresponse/arpa-reporter/issues/300 mentions that users have reported not being able to find the download button. This PR replaces the nondescript download icon with a clear labeled action button that is much harder to miss.

old
new

I think it might actually be sufficient to swap to a more recognizable icon
Yeah I started with that idea but I figured it was better to make this button TOO visible instead of going with an icon that users might still miss. Better to fix this once instead of waiting for more user reports

After making the button btn-sm the screen now looks like:


[AKS] Conflict between cluster autoscaler and nodepool deletion

When deleting a nodepool that has cluster autoscaler enabled, the pods that are scheduled on the nodes in that nodepool are rescheduled, and they trigger an autoscale operation in the nodepool if they don't find a new node (because of taints or something else), because the existing nodes in the nodepool are disabled.
So the operation of deleting the nodepool fails.
To avoid this conflit, one should disable cluster autoscaler from the nodepool before deleting it.
Docs affected:
https://github.com/MicrosoftDocs/azure-docs/blob/master/articles/aks/cluster-autoscaler.md
https://github.com/MicrosoftDocs/azure-docs/blob/master/articles/aks/use-multiple-node-pools.md

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

ID: 077f1bb5-2ba9-490c-2c6e-0de292266539
Version Independent ID: 0fd87b4f-88a4-c2eb-3ebc-f2b113ac203c
Content: Use the cluster autoscaler in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/cluster-autoscaler.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@alijahnas Thanks for your feedback! We will investigate and update as appropriate.

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Great news, thanks

Thank you for the feedback.
We've added a task to our backlog and will prioritize accordingly.
#please-close

@alijahnas - I just tested and I could not recreate the issue. I was able to successfully delete the node pool without disabling the cluster autoscaler first. I will close this GitHub issue again now that we've tested the scenario. Thank you again for your feedback!
#please-close


Write hyperlinks

(and ideally figure out the best representation for the hyperlink)

any progress with this?