Missing User Attributes

[x] Users should be soft-deleted. Jetstream performs hard deletion out of the box. This'll mess up auditing.
[ ] Role should be nullable
[ ] Patron flag (future feature-flagging possibilities)
[x] Lock/Ban flag

Soft deletion was added in 156c5989fc24b8644f708db4a6d0aa9bf2b5833e

While there exist packages for bans, soft deletion has the side effect of functioning as a ban since authentication will for soft deleted users. If our needs change going forward, an issue can be filed. Leveraging this side effect is good enough for now, IMO.
Jetstream 2.X shipped with enhanced team functionality. It might make more sense to ditch the role attribute in favor of admin/contributor/read-only/patron teams.

Role has been removed in favor of Jetstream teams.
Jetstream teams provides the possibility to create teams like 'Patrons', 'Encoders', 'Contributors', etc.

Closing.


fix: update scripts

An error caused the deployment script to crash when missing an evm private key in the .env file.

Time spent on this PR: N/A
Pull request type

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Error in script prevents CI deployment of contracts.
Resolves N/A
What is the new behavior?

Scripts are updated in order to allow deployment with missing evm private key.

This item belongs to payment request #E89437 on OnlyDust:

from Eikix to greged93
6 items included
$1,000 for 2 days of work

This item belongs to payment request #256486 on OnlyDust:

from Eikix to greged93
6 items included
$1,000 for 2 days of work

This item belongs to payment request #256486 on OnlyDust:

from Eikix to greged93
6 items included
$1,000 for 2 days of work

Payment request #E89437 has been processed and payment is complete. See you around on OnlyDust.

This item belongs to payment request #256486 on OnlyDust:

from Eikix to greged93
6 items included
$1,000 for 2 days of work

Payment request #256486 has been processed and payment is complete. See you around on OnlyDust.


Feature/details participant list tables

This adds a "Common Details Participant List Table", which is used by both exclusion and inclusion tables.
2 notes:

Currently the data for these tables is 3 types of segments (4 if you count "All"), which all have different fields that don't always map sensibly just to "Name" and "Type". See the experiment json and the picture. So this will probably be changing, as well as the Figma design, so please ignore these finer points.

We have been talking about redesign for flag segments as their own entity. Part of that discussion should be how to uniformly describe a table data structure row type that uses fields that match the column names so the UI doesn't have to suss things out to know what to display. See the data type that I have mapped all the segments to to accomplish this, the UI probably shouldn't need sweat so much: https://github.com/CarnegieLearningWeb/UpGrade/pull/1670/files#diff-49cbfb375bc721764b878b8d116557be324c4564b2d123b416a25fdf9f5b5a7cR140

We should try to create a common-table. I originally thought it would be hard, but after I created these tables separately, I found that it was very easy to turn into a common component. I would like to see if I can take this a step further and create a common component that can be used by the root table and any other tables, but that may be easier said than done, so I decided to not do that in this PR.

This data object can be supplied in postman to get what you see (you will need to create an actual Segment on your local and pass the id in here):
PUT /featureflags
{
    "createdAt": "2024-06-12T17:15:10.398Z",
    "updatedAt": "2024-06-12T17:15:16.066Z",
    "versionNumber": 2,
    "id": "580cdd42-df6f-4038-917a-edd276b4b67a",
    "name": "abc",
    "key": "abc",
    "description": "",
    "context": [
        "mathstream"
    ],
    "tags": [],
    "status": "enabled",
    "filterMode": "includeAll",
    "featureFlagSegmentInclusion": {
        "createdAt": "2024-06-12T17:15:10.398Z",
        "updatedAt": "2024-06-12T17:15:10.398Z",
        "versionNumber": 1,
        "segment": {
            "createdAt": "2024-06-12T17:15:10.402Z",
            "updatedAt": "2024-06-12T17:15:10.402Z",
            "versionNumber": 1,
            "id": "479dbaee-02fe-4336-b329-bb3a92bbae4f",
            "name": "580cdd42-df6f-4038-917a-edd276b4b67a Inclusion Segment",
            "description": "580cdd42-df6f-4038-917a-edd276b4b67a Inclusion Segment",
            "context": "mathstream",
            "type": "private",
            "individualForSegment": [
                {
                    "userId": "testGuy1"
                },
                {
                    "userId": "testGuy2"
                }
            ],
            "groupForSegment": [
                {
                    "groupId": "school-one",
                    "type": "schoolId"
                },
                {
                    "groupId": "school-two",
                    "type": "schoolId"
                },
                {
                    "groupId": "school-three",
                    "type": "schoolId"
                }
            ],
            "subSegments": [
                {
                    "id": "6c02dea8-fa17-43b4-90ee-e1c3e6fc8173"
                }
            ]
        }
    },
    "featureFlagSegmentExclusion": {
        "createdAt": "2024-06-12T17:15:10.398Z",
        "updatedAt": "2024-06-12T17:15:10.398Z",
        "versionNumber": 1,
        "segment": {
            "createdAt": "2024-06-12T17:15:10.464Z",
            "updatedAt": "2024-06-12T17:15:10.464Z",
            "versionNumber": 1,
            "id": "f3826196-8bc0-4bb0-b56d-b6432cfc93f6",
            "name": "580cdd42-df6f-4038-917a-edd276b4b67a Exclusion Segment",
            "description": "580cdd42-df6f-4038-917a-edd276b4b67a Exclusion Segment",
            "context": "mathstream",
            "type": "private",
            "individualForSegment": [],
            "groupForSegment": [
                {
                    "createdAt": "2024-06-12T17:16:42.125Z",
                    "updatedAt": "2024-06-12T17:16:42.125Z",
                    "versionNumber": 1,
                    "groupId": "school-four",
                    "type": "schoolId"
                }
            ],
            "subSegments": []
        }
    }
}

I updated for most comments. For the extra functions in the selectors, I have a plan for that which I will share in an issue. Can we merge this in with those functions still in there? There should be a minor refactor that will include some unrelated stuff from this particular PR.
Also, for the common tables, that's why I created only a common table template component for these tables that have the exact same columns and styling and not for all. But I still kinda want to play around with the idea of a common table, if it's feasible. We'll see, maybe it's not.


[make error] 3rdparty/libbacktrace/configure  Syntax error

root@00048971l16:/tvm/tvm/build# make -j4
[  0%] Creating directories for 'project_libbacktrace'
[  1%] Building CXX object CMakeFiles/tvm_libinfo_objs.dir/src/support/libinfo.cc.o
[  2%] No download step for 'project_libbacktrace'
[  2%] No checkout step for 'project_libbacktrace'
[  2%] No update step for 'project_libbacktrace'
[  2%] No patch step for 'project_libbacktrace'
[  2%] Performing configure step for 'project_libbacktrace'
: not foundlopSoft/tvm/tvm/cmake/libs/../../3rdparty/libbacktrace/configure: 14: /mnt/d/DevelopSoft/tvm/tvm/cmake/libs/../../3rdparty/libbacktrace/configure:
/mnt/d/DevelopSoft/tvm/tvm/cmake/libs/../../3rdparty/libbacktrace/configure: 30: /mnt/d/DevelopSoft/tvm/tvm/cmake/libs/../../3rdparty/libbacktrace/configure: Syntax error: newline unexpected (expecting ")")
CMakeFiles/project_libbacktrace.dir/build.make:130: recipe for target 'libbacktrace/src/project_libbacktrace-stamp/project_libbacktrace-configure' failed
make[2]: *** [libbacktrace/src/project_libbacktrace-stamp/project_libbacktrace-configure] Error 2
CMakeFiles/Makefile2:985: recipe for target 'CMakeFiles/project_libbacktrace.dir/all' failed
make[1]: *** [CMakeFiles/project_libbacktrace.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[  2%] Building CXX object CMakeFiles/tvm_objs.dir/src/arith/analyzer.cc.o
[  2%] Building CXX object CMakeFiles/tvm_objs.dir/src/arith/bound_deducer.cc.o
[  2%] Building CXX object CMakeFiles/tvm_objs.dir/src/arith/canonical_simplify.cc.o
[  2%] Built target tvm_libinfo_objs
[  2%] Building CXX object CMakeFiles/tvm_objs.dir/src/arith/const_int_bound.cc.o
[  2%] Building CXX object CMakeFiles/tvm_objs.dir/src/arith/detect_linear_equation.cc.o
[  3%] Building CXX object CMakeFiles/tvm_objs.dir/src/arith/domain_touched.cc.o
[  3%] Building CXX object CMakeFiles/tvm_objs.dir/src/arith/int_constraints.cc.o
...
...
[ 91%] Building CXX object CMakeFiles/tvm_objs.dir/src/relay/backend/contrib/example_target_hooks/tir_to_runtime.cc.o
[ 91%] Building CXX object CMakeFiles/tvm_objs.dir/src/contrib/hybrid/codegen_hybrid.cc.o
[ 91%] Built target tvm_objs
Makefile:145: recipe for target 'all' failed
make: *** [all] Error 2
Environment
ubuntu 16.0.4
cmake version 3.22.3
llvm 6.0
gcc 5.4.0
I set these configs at build/config.cmake:
set(USE_GRAPH_EXECUTOR ON)
set(USE_PROFILER ON)
set(USE_RELAY_DEBUG ON)
set(USE_LLVM /usr/lib/llvm-6.0/bin/llvm-config)
I had test though cmd: "bash ./configure --help" and "dash ./configure --help" all  throw Syntax error;
Anyone had idear?

I came cross the same crash. is there any workaround to solve the bug?

We could turn off libbacktrace via the following setting in config.cmake:
set(USE_LIBBACKTRACE OFF)

And of course, mixing different toolchains in a C++ project could cause potential issues. Would be great if we could finally unify the autoconf toolchain used in libbacktrace into cmake

@jackyflame I'm having trouble reproducing this issue. I tried on a Ubuntu 16.04 VM and could not reproduce. Can you provide more information about your environment? What version of bash and sh are you using? What kind of machine are you running on?  If so, could you provide some more information to help us debug? For example: what version of bash and sh are you using, what kind of machine are you running on (Is it a VM on a popular cloud provider so I could try to reproduce on my own instance), and what your architecture is.

@zhyncs It appears that there was an error in your build but the amount of the build log that you have provided does not show the original error. Could you provide the full build log.

@jackyflame Are you still having this issue? Happy to help if it is still an issue.
If so, could you provide some more information to help us debug? For example: what version of bash and sh are you using, what kind of machine are you running on (Is it a VM on a popular cloud provider so I could try to reproduce on my own instance), and what your architecture is.

I got the error,how do i solve it?
[ 88%] Built target tvm_objs
[ 88%] Building CXX object CMakeFiles/tvm_libinfo_objs.dir/src/support/libinfo.cc.o
[ 88%] Built target tvm_libinfo_objs
[ 89%] Creating directories for 'project_libbacktrace'
[ 89%] No download step for 'project_libbacktrace'
[ 89%] No checkout step for 'project_libbacktrace'
[ 89%] No update step for 'project_libbacktrace'
[ 89%] No patch step for 'project_libbacktrace'
[ 89%] Performing configure step for 'project_libbacktrace'
/bin/sh: 1: /mnt/home/projects/litepose/tvm/3rdparty/libbacktrace/configure: not found
CMakeFiles/project_libbacktrace.dir/build.make:130: recipe for target 'libbacktrace/src/project_libbacktrace-stamp/project_libbacktrace-configure' failed
make[2]: *** [libbacktrace/src/project_libbacktrace-stamp/project_libbacktrace-configure] Error 127
CMakeFiles/Makefile2:174: recipe for target 'CMakeFiles/project_libbacktrace.dir/all' failed
make[1]: *** [CMakeFiles/project_libbacktrace.dir/all] Error 2
Makefile:135: recipe for target 'all' failed
make: *** [all] Error 2

@wolfworld6 Can you check that all submodules are up to date by running git submodule init; git submodule update. If that does not work, can you post the full configure + build log?

@wolfworld6 Can you check that all submodules are up to date by running git submodule init; git submodule update. If that does not work, can you post the full configure + build log?

It works. Previously I failed for the same reason. Now I know the submodule is hided by git.  git submodule init; git submodule update works


Incorrect Syntax on Dynamic data masking doc

The syntax for step 9 in Granular permission example is not valid

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

ID: 46d10c12-b07b-1541-0aa4-5f9336f81fab
Version Independent ID: e5d0e48b-dc13-0b42-3c72-f5c82ecd9592
Content: Dynamic data masking - Azure SQL Database
Content Source: azure-sql/database/dynamic-data-masking-overview.md
Service: sql-db-mi
Sub-service: security
GitHub Login: @Madhumitatripathy
Microsoft Alias: matripathy

I was using incompatible version.

Thanks for sharing the update. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


Default Values missing due to PR #106

Describe the bug
After PR #106 was merged default values for props stopped working.
To Reproduce
Paste
<template>

    <slot name="header">

      title
    </slot>

</template>

into vuese explorer and you will see that all default values are missing.

Thank you so much! <3


Documentation: adding javadocs to json classes and test classes

What feature are you asking for
adding javadocs to json classes and test classes
Describe the solution you'd like
A javadoc above most of the methods and any classes that need better explaining
Reason
for documentation purposes, and so its easy to understand what the functions and classes are responsible for in the future

Approved


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/48537

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/49525

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/49550


Add typing hints

Recently it seems that more and more Python libraries include typing hints. Personally, I'm a fan of type hints while developing. Are there any plans to add typing hints to Fiona?
I'm not 100% sure what the best way to deal with the dynamic nature of the dataset schemas is. Maybe the implementation of static typing of psycopg3 could be used as inspiration: https://www.psycopg.org/psycopg3/docs/advanced/typing.html

There wasn't much point in doing this before the Feature, Geometry, Properties classes. They will help quite a bit. Maybe a Schema class would, too?

In a naive attempt to model the parameters to fiona.open (v1.8) we created this:
from typing import TypedDict

class FionaBaseDict(TypedDict):
    properties: dict[str, Any]

class FionaSchemaWithoutGeometry(FionaBaseDict):
    geometry: None

class FionaSchemaWithGeometry(FionaBaseDict):
    geometry: str

FionaSchema = Union[FionaSchemaWithoutGeometry, FionaSchemaWithGeometry]

class RecordGeometry(TypedDict):
    type: str
    coordinates: tuple

class RecordWithoutGeometry(FionaBaseDict):
    geometry: None

class RecordWithGeometry(FionaBaseDict):
    geometry: RecordGeometry

Record = Union[RecordWithoutGeometry, RecordWithGeometry]

I'm not yet familiar with the new Feature, Geometry, Properties classes. That would probably simplify things. In any case I would be interested to help get type hints into fiona, either by sharing opinion or contributing code.

@StefanBrand @rbuffat what would you think about collaborating on stubs for 1.9 and then possibly moving the stubs into the code at 2.0?

@sgillies I can have a go at it but probably during the next few weeks I don't have too much time available, thus it can take a while.  I will also need to dive a bit deeper into the Python typing system and maybe look at how other projects are implementing dynamic types (Sqlalchemy or pydantic come to my mind here, perhaps it would also be worth asking somebody familiar with mypy/pyright for some inputs). Being able to static type check writerecords() or the collection iterator would probably be the most useful thing, but I guess also the hardest to implement.

but probably during the next few weeks I don't have too much time available

Maybe that's not too bad in this case. I myself should have more availability starting from (mid?) April. I haven't much worked with type stubs instead putting type hints directly into .py scripts. It will be an interesting exercise. If time permits I will look into type stub implementation.

Almost two months have passed. In the meanwhile I have setup a wiki page in our fork and added commands to generate type stubs using several tools. The resulting stubs can be found in branches in our fork.
Here are all instructions and links: https://github.com/EOX-A/Fiona/wiki/Generating-Python-Stubs
Since using tools to generate type stubs resulted in a big amount of type annotations, I welcome any reviews and comments. The initial focus should be on writerecords() and the collection iterator, as mentioned by @rbuffat above:

Being able to static type check writerecords() or the collection iterator would probably be the most useful thing, [...]

PS.: The lightning talk proposal for FOSS4G 2023 was accepted. So I really need to put some flesh on this. :grin:

We are looking for initial feedback on the  #1259 draft. Please give your comments. :ok_hand:

Cross reference: https://github.com/jazzband/geojson/issues/167


Release of 1.5 to Rubygems

Hi @billdueber
Just checking in to see if we can get the latest version of library_stdnums released to rubygems.org?
Many thanks in advance.

Just released 1.6.


Add --max-workers and heuristics for parallel operations

add a global flag for --max-workers so users can limit the number
of parallel operations for a given function.  also, when not limited
by max-workers, we implement a heuristic function that returns the
number of preferred parallel workers based on the number of CPUs and
the given operation.
Signed-off-by: baude<EMAIL_ADDRESS>

bot, retest this please

@baude looks like some of the rootless tests aren't hip.

Who do you envision every using this option.

I'm the troublemaker on this one.  My thinking was if the default setting ever caused performance issues, or we found that the machine had room to let more crank through then the default and it helped speed up processing, it would be good to have the toggle available.
Given that, I don't anticipate many folks will use it, but I'd bet my bottom dollar we'll land in a situation where it will be useful some day.

@rhatdan @mheon and @TomSweeneyRedHat made me promise to implement this option so I cannot speak to that.  That said, I could see where someone might want to eliminate what they might consider to be too much parallelism

Someone is going to need this (esp. in memory-constrained situations) to not blow up their system with a thousand parallel runc processes

Is this a podman thing or a libpod thing?  If a libpod thing, I would prefer it to go into libpod.conf.

Probably a Podman/Varlink thing more than a Libpod thing - I don't think we want this in libpod, it doesn't strike me as desirable in CRI-O

Well I can hold my nose on this.  I am not a big fan of adding options, that just clutter up the CLI and no one ever uses.  Plus we need to tests what happens when I set the number to 0?  What happens when I set it to 100,000?  Does this options cause breakage on different kinds of systems?

We should detect 0 and throw angry errors. 10k... I don't actually know if we ALWAYS spin up the maximum number of workers, or if we only spin then up proportionate to the number of containers. It's probably safe if it's the number of containers, we'll just end up with gigabytes of memory spent on runc.

@rhatdan when you are done with your spiel, lets talk it out ... i'm flexible here.  would you prefer I leave the --max-workers part out for now ?

made compromise with @rhatdan ... please /lgtm when green

bot, retest this please

/approve

/lgtm


Add draw_chimera_embedding

Adds a function that draws embeddings in chimera with matplotlib (similar to draw_chimera, but colors represent distinct chains instead of bqm model weights).

Closed by #56


Ionic 2 - IOS 10 Keyboard bug

Short description of the problem:
This happens on every ios 10 device.
If you open a dialog (alert) und enter something into an input field and you close the dialog (by clicking the close button), then the keyboard will stay open and not close.
I've tried to use the ionic keyboard plugin to close the keyboard manually upon clicking the button (Keyboard.close()), but it has zero affect.
I've created a new blank project (ionic start testProj --v2 --ts) and added a dialog with an input field and a button (actually took the code from the docs to make sure), it has the same effect.
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
There are no error messages or whatsoever.
Which Ionic Version? 1.x or 2.x
2
Run ionic info from terminal/cmd prompt: (paste output below)
Your system information:
Cordova CLI: 6.3.1
Gulp version:  CLI version 3.9.1
Gulp local:   Local version 3.9.1
Ionic Framework Version: 2.0.0-beta.11
Ionic CLI Version: 2.0.0-beta.37
Ionic App Lib Version: 2.0.0-beta.20
OS:
Node Version: v6.3.0

It feels like there is something blocking, since the log output and a lot of events are having a weird behavior on ios.

Any news about this?
Got a release on hold because of this bug.

@John0x would you mind trying this with our wkwebview plugin. Thanks!

Sadly it does not fix the bug.

I installed the plugin properly and it was running (confirmed), but had still the same behaviour.

Same problem here, the keyboard does not close and it's a big problem for iOS apps...

This happens in Ionic 2 RC0 as well; fixing this soon would greatly improve the confidence of those of us who are considering Ionic for a new app but want assurance that it will stay on top of device platform updates.

For me it's a pain to update my ionic2 beta 11 app version to RC0..

Yes, publishing a workaround that works prior to RC0 would also be super nice since the update isn't trivial.

I start to update today but as I'm not sure if there will be some big change log soon, I think it's better to wait and update my app when Ionic team will ensure there will not be any other BIGGGGG Change....

It's the RC, so there shouldn't be alot of big changes anymore.
The Keyboard doesn't close automatically but you can close it yourself by subscribing to the onWillDismiss event and then close it manually (Keyboard.close() with the Keyboard plugin).

Thanks @John0x this is a good fix.
Someone here notice there is a strange behavior when you launch an alert ?
The view (only the content not the header) is turning for some millisecond and then the alert is shown ?

I'm seeing this problem too - the keyboard is not disappearing after an alert.  IOS 10 (iPhone SE) running from within Ionic View.

Definitely a bug. I was also able to get rid of the keyboard by using onDidDismiss() and Keyboard.close() on the alert controller, but still looks unprofessional as the screen scrolls up including the header. Hope it gets fixed soon.

Is there an update to this bug? With RC.1 and iOS10 I still see the bug.

Yep I also see this one with rc1.

It also happens to me on RC1, and not in an alert. Just on a regular input on a regular form, if you send the form by tapping on return key, after the redirect with this.nav.setRoot the keyboard keeps open.

Working in this issue! Sorry for taking so long

I made a workaround importing Keyboard  from 'ionic-native' anda executing 'Keyboard.close()' after the action I needed

Hi,
Importing the Keyboard and force it to close partially worked for me, as the input dialog sometimes stays open and freezes the app.
will this be fixed in the next version? Any hind on a release date for it?
Thanks for your effort.
Cheers

I am also experiencing this issue with an Ionic 1 project.  The keyboard was working fine until I updated my iPhone to iOS 10.1.1.
Will the fix be applied to both Ionic 1 and 2?
Thanks

Same here on Ionic2rc3 I'm using Ionic-native and Keyboard.close() meanwhile.

Thanks for the report everyone, we're working on this right now and I'll keep everyone updated, thanks

I've made some updates to how alerts work when there's a text input, and it's on a mobile device. Would this group be able to test out our nightly version and let me know how it goes? Thanks

It works great in nightly! The input "jumps" into position though, instead of a smooth scroll animation like when you focus an input in Safari for example, is that intended?

The keyboard is working great on iOS 10.1.1. Although the alert isn't vertically centered anymore and it doesn't get "pushed up" by the appearing keyboard. Is that intended?

@ohh2ahh It seems that it is intended to solve the keyboard overlap issue, per the commit:
// this alert has a text input and it's on a mobile device so we should align
// the alert up high because we need to leave space for the virtual keboard
// this also helps prevent the layout getting all messed up from
// the browser trying to scroll the input into a safe area

How can i implemented the changes from @adamdbradley ? I still have the alert input keyboard issue and no idea how to solve this or make a workaround.

@Daphko you can install ionic native keyboard plugin and use it like this
createTodo(){
    var id = new Date();
    let prompt = this.alertCtrl.create({
      title: this.translate.instant("todo.addTodo"),
      message: this.translate.instant("todo.addTodoMessage"),
      inputs: [
        {
          name: 'title'
        }
      ],
      buttons: [
        {
          text: this.translate.instant("ui.cancel"),
          handler: () => Keyboard.close()
        },
        {
          text: this.translate.instant("ui.save"),
          handler: data => {
            this.todoService.createTodo({
              _id:'todo:'+id.getTime(),
              title: data.title,
              type:'todo',
              created_at : new Date().toISOString()
            });
          }
        }
      ]
    });

    prompt.present();

  }

Thx @numerized  but the solution has one problem. If you click outside the alert box, the alert box closes and the keyboard stays open. Thats because there is no handler for this click action. Do you have any solution for this?

I think onDidDismiss and onWillDismiss are called.
prompt.onDidDismiss(()=> {
   Keyboard.close()
})

Hi!
I'm having this issue with the ion-searchbar.
The keyboard just don't go away.
No answer for this.keyboard.close(); and console.log("keyboard open:", this.keyboard.isOpen()); always prints true.
Any info that could fix this?
Your system information:
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.6
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.0.1
ios-deploy version: 1.9.0
ios-sim version: Not installed
OS: macOS Sierra
Node Version: v6.9.1
Xcode version: Xcode 8.2.1 Build version 8C1002

@arturalkaim try Keyboard.close(); instead of this.keyboard.close();
import {Keyboard} from 'ionic-native';

Thanks @peterpeterparker!
It should close on blur but this worked as a workaround...

I have issues with ionic 1.3.2 and iOS 10.1.1. When keyboard  pop up the page scroll up with lot of space in between the keyboard and input field. This was working fine in iOS 9.1.1.

When I use Keyboard.close(); the keyboard closes but whenever I touch the screen it reopens. Any ideas anyone?

@robvr I am also having this issue. Did you ever find a fix? I am using iOS 11.


ARROW-10808: [Rust][DataFusion] Support nested expressions in aggregations.

This change is a proposal to broaden the SQL planner's support for queries such as:

SELECT that does not include the columns from the GROUP BY:SELECT MAX(b) FROM t GROUP BY a;

SELECT with an aggregate function that is nested within another expression (in this case a binary op):SELECT MAX(b) + 1 FROM t GROUP BY a;

SELECT with an aggregate's arguments that are not simple columns:SELECT MAX(b + 1) FROM t GROUP BY a;

GROUP BY with an expression that is not a simple column:SELECT a + 1, MAX(b) FROM t GROUP BY a + 1;

SELECT with an aggregate on a column also in the GROUP BY (though not an entirely useful query, it is valid):SELECT MAX(a) FROM t GROUP BY a;

SELECT * with a GROUP BY that enumerates all of the columns:SELECT * FROM t GROUP BY a, b, c, ...

There are two existing tests where behaviour has changed:

select_7480_1(): The scenario is not all of the GROUP BY expressions are in the SELECT expressions - this is an intentional change as outlined above.
select_nested_with_filters(): The outer projection is dropped, which is a side-effect of project() projecting only if necessary. This conditional projection serves to keep the other existing tests passing, but failed to do so in this case. Please let me know if this should be revised.

Also note that, because there were some changes to how the exprs/columns are processed and validated, I included a number of tests of existing behaviour to help validate there are no regressions (for example, SELECT'ing duplicate columns).

I will try and review this later today but more likely it will be tomorrow sometime before I have a chance

Thank you @alamb and @jorgecarleitao!
I will get the comments/issues/rebase addressed sometime in the next few days.

Codecov Report

Merging #8836 (dfdafdf) into master (1a7b692) will decrease coverage by 0.05%.
The diff coverage is 76.08%.

@@            Coverage Diff             @@
##           master    #8836      +/-   ##
==========================================
- Coverage   83.26%   83.21%   -0.06%
==========================================
  Files         196      197       +1
  Lines       48131    48428     +297
==========================================
+ Hits        40077    40299     +222
- Misses       8054     8129      +75

Impacted Files
Coverage Δ

rust/datafusion/src/logical_plan/expr.rs
76.76% <ø> (ø)

rust/datafusion/src/physical_plan/functions.rs
82.02% <ø> (ø)

rust/datafusion/src/sql/utils.rs
55.11% <55.11%> (ø)

rust/datafusion/src/sql/planner.rs
84.81% <95.21%> (+3.08%)
:arrow_up:

rust/datafusion/src/physical_plan/udaf.rs
78.94% <100.00%> (+1.16%)
:arrow_up:

rust/datafusion/src/physical_plan/udf.rs
78.26% <100.00%> (+2.07%)
:arrow_up:

rust/datafusion/src/logical_plan/builder.rs
90.05% <0.00%> (-1.05%)
:arrow_down:

rust/parquet/src/encodings/encoding.rs
95.43% <0.00%> (+0.19%)
:arrow_up:

... and 1 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 1a7b692...dfdafdf. Read the comment docs.

This is breaking a test in mac. I am not sure it is related. Could you rebase it so that I can safely merge it?

@jorgecarleitao: Updated, tests are now passing.

Everything looks good from here -- I also skimmed the PR again but I didn't go through it in detail. Thanks again @drusso


ENH: Add explicit format registration to Plugin objects

A "bug" was discovered while removing the duplicate BIOMv1 directory type registration from q2-types. The framework loses the ability to know this format still exists, as it was previously scraping transformer annotations, and semantic type to format registrations.
This PR adds explicit format registration to plugin objects, and makes them accessible through the PluginManager, a nice side effect is that this greatly reduces the runtime of qiime.sdk.util.parse_format.

Thanks @jakereps! @gregcaporaso, @ebolyen, and I chatted and think it would be great to have the plural forms (register_semantic_types, register_formats) to reduce boilerplate. The singular forms would be removed. It seems like a worthwhile change to make now since you're already going to update the plugins to use the new format registration API, so the plural semantic type registration updates could happen in those same PRs. Are you good with this plan?

I forgot to mention in my review above that we want the plural registration forms instead of singular. @jakereps let me know when you're ready for another passthrough.

Thanks @jairideout! Everything should be addressed, and pluralization completed. Updated the q2-types PR to match the plural registration, and have a demux and classifier PR ready to go, will open them after this class!

Awesome, I really like the plural registrations 👍  Thanks @jakereps!

:+1:


Segwit leftovers require removal

Describe the issue
What files need to be fixed? (Please list them all)
miner.h, mining.cpp, validation.cpp, validation.h
What needs to be done?
Remove some segwit left-overs
Why?
To meet end-goal

First commit f9d5c2cca92413df1d9f26b5e6317f0da970f58d is only the first part of necessary changes to remove all of the remaining SegWit code.
I will await feedback on removing WITNESS_SCALE_FACTOR and GetTransactionWeight() from many more files.  Additionally SERIALIZE_TRANSACTION_NO_WITNESS can be removed.

I think its pretty much been nixed from the codebase at this point.

There are still some parts left.

I reopened this because there are still a few parts of Segwit remaining.  Several things were found this week.

Everything is removed except for a few unused parameter warnings.  This will be addressed in another issue: #27.


Added orders to assets

For easy usage of item orders and impossibility to mistake string orders (which also looks worse for me).

Is there something else what is to be done with this PR?

A short explanation of itemdrag and itemuse plus example on how to use would be nice. But it's already looking great now and I can merge if you want 👍 :)

Yay :D
Added last part of comments. The rest of them are on you now ;)

Don't close PRs, wait for them to be merged ;)


Custom colors for minimap

Added color for winnowing spirit alongside eoe & qz.
Changed soul tormentor counter to only show when in DoA.
Added custom agent colors for the minimap that are saved in the INI.

I shall add a bit of complexity in the UI because I think it's important to be able to at least delete custom colors, so hold off on merging this for now at least.

Thanks. Do you think it's worth it having a separate ini file? Similar to Markers.ini

It would be a bit more organised. Probably makes more sense to do that for hotkeys, for me at least I have about 20 hotkeys but only 3 colored enemies.

Yea I guess we can do that too. Initially all settings were in one GWToolbox.ini but I've been splitting them. I moved teambuilds out of the main ini in 2.6, we can do the same for hotkeys.

I merged, but looking at the code I still have a doubt: why collect custom colored agents, sort them, and render them?
I understand we want to render the colored agents after all the default ones, but what is the sort for? And as an alternative, could we loop and render non-colored first, and then loop again and render colored after? (without building the array of custom_agents).

Also about the interface, I think it would make more sense to match the UI style of the custom markers, rather than that of the hotkeys.

The sorting is because I want them to be sorted :P if I have a large group of soul tormentors, I need to be able to see if one fiend has spawned in the middle of them, so I need fiends to be rendered on top of souls for example.
I mean you can change the custom_agents vector to a list, or build it as a map and loop through the colors_custom vector instead of sorting if you are so worried about performance - but I really think it is effort spent on a non-issue.


⚠ Samsung has degraded performance

In 14924e7, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2657 ms

Resolved: Samsung performance has improved in 0ce0f69.


Making the VPN tunnel last_status_change field an ephemeral.

Coverage decreased (-0.005%) to 63.64% when pulling 2d69cc41b799721764462b12ae174aef2c94cf45 on add_ephemeral_to_vpn into 9c5ce3b9dff37bb1a51abcb8a32d992bd0771579 on develop.


Can change any type's useInTable value to be server

I'm submitting a ...

[X] bug report
[ ] new feature / enhancement request
[ ] improvement request (usability, performance, tech debt, etc.)
[ ] other

Traffic Control components affected ...

[ ] CDN in a Box
[ ] Documentation
[ ] Grove
[ ] Traffic Control Client
[ ] Traffic Monitor
[X] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] unknown

Current behavior:

When making a PUT request to /types/{id}, where id identifies a type that does not belong in the servers table. The API allows that type to be changed from the table it was using. For example one can update a type using  the deliveryservice table to use the server table.
HTTP/1.1 200 OK
Content-Type: application/json
{
    "alerts": [
        {
            "text": "type was updated.",
            "level": "success"
        }
    ],
    "response": {
        "id": 1,
        "lastUpdated": "2020-03-26 20:49:44+00",
        "name": "HTTP",
        "description": "HTTP Content Routing",
        "useInTable": "server"
    }
}

Expected / new behavior:

Should not be allowed and return:
HTTP/1.1 400 Bad Request
Content-Type: application/json
{
    "alerts": [
        {
            "text": "can not update type",
            "level": "error"
        }
    ]
}

Minimal reproduction of the problem with instructions:

Anything else:

Yeah, I think you should only be able to:
a. create a type with useintable=server (any other values for useintable should be rejected)
b. update a type where useintable=server but can't change the useintable value to something else.
c. delete a type where useintable=server
imo this should all be enforced at the api level. UI level is a bonus maybe but not really necessary.
fyi, the useintable thing is very strange...

Fixed by https://github.com/apache/trafficcontrol/pull/4600


Random  StatusCode.UNAVAILABLE: Connection reset by peer

What version of gRPC and what language are you using?
python 3.7
grpcio latest
What operating system (Linux, Windows,...) and version?
Debian  v9 (Docker container)
What runtime / compiler are you using (e.g. python version or version of gcc)
Kubernetes (EKS)
What did you do?
Invoke a function via gRPC
What did you expect to see?
Normal invocation and computed result
What did you see instead?
<_InactiveRpcError of RPC that terminated with:
    status = StatusCode.UNAVAILABLE
    details = "Connection reset by peer"
    debug_error_string = "{"created":"@1626692243.576262296","description":"Error received from peer ipv4:<IP_ADDRESS>:9010","file":"src/core/lib/surface/call.cc","file_line":1066,"grpc_message":"Connection reset by peer","grpc_status":14}"

Anything else we should know about your project / environment?
There are no patterns to how this error occurs. We have no means of reproducing it, Which would have been helpful when troubleshooting

I have encountered same problem, It's somehow doesn't happen with using retry at below link, though I don't think retry is proper solution.

"Connection reset by peer"

This sounds like a TCP-level issue. Are you sure that you have stable connectivity to your backend?


Add multithreaded env

Description
Allows creating multithreaded environments based on EnvPool library. This is an alternative to ParallelEnv, which uses multiprocessing parallelism. EnvPool-based approach is faster, but less flexible, as one can only create environments implemented in EnvPool (whereas ParallelEnv parallelises arbitrary TorchRL environment)
Changes:

Add MultiThreadedEnv environment to torchrl/envs/vec_env.py
Add benchmarking script to benchmarks/benchmark_batched_envs.py
Add an option for CompositeSpec to return only non-nested keys - otherwise step didn't work for environments with multiple observations, which have nested observation spec (like Cheetah which returns position and velocity)
Add tests

In this PR I only added synchronous EnvPool interface, when batch size is equal to the number of workers, but in principle it's possible to add the asynchronous one too.
Motivation and Context
Closes #591
Benchmark
The following benchmark runs env.rollout(max_steps=1000) for three types of batched environments: SerialEnv, ParallelEnv, and MultiThreadedEnv:
pip install pandas
python benchmarks/benchmark_batched_envs.py

| num_workers_1_cpu | num_workers_4_cpu | num_workers_16_cpu | num_workers_1_cuda | num_workers_4_cuda | num_workers_16_cuda
-- | -- | -- | -- | -- | -- | --
Serial, s | 0.217 | 0.433 | 1.357 | 0.319 | 0.645 | 1.924
Parallel, s | 0.206 | 0.234 | 0.275 | 0.296 | 0.502 | 1.428
Multithreaded, s | 0.13 | 0.153 | 0.161 | 0.187 | 0.211 | 0.242
Gain, % | 36.9 | 34.6 | 41.5 | 36.8 | 58.0 | 83.1
One can see that on CPU MultiThreadedEnv is ~40% faster than ParallelEnv, depending on the number of workers. Neither of the models is benefiting from GPU in the current setup, or I'm not using it in the right way.
Environment creation time is excluded for all benchmarks.
Tests
Tests has been added to tests/test_libs.py::TestEnvPool and run in a separate pipeline unittest_linux_envpool_gpu located in .cicrleci/unittest/linux_libs/scripts_envpool.  Many of the tests are the same as for SerialEnv and ParallelEnv
Comparing _BatchedEnv and MultiThreadedEnv output: different RNGs
One test which is lacking is a check that results from SerialEnv/ParallelEnv and MultiThreadedEnv are the same for the same environment configuration and the seed. This is because they generate random numbers in a different way:

EnvPool uses C random number generator from std::uniform_real_distribution (here and here)
SerialEnv/ParallelEnv with Gym envs take random numbers from Gym, which uses numpy RNG (here and here)

I googled around, but didn't find a good way to use the same random number generator on both sides:

https://studywolf.wordpress.com/2012/10/02/a-common-random-number-generator-for-c-and-python/
https://stackoverflow.com/questions/65871948/same-random-numbers-in-c-as-computed-by-python3-numpy-random-rand

Types of changes
What types of changes does your code introduce? Remove all that do not apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds core functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)
[ ] Example (update in the folder of examples)

Checklist
Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!

[x] I have read the CONTRIBUTION guide (required)
[x] My change requires a change to the documentation.
[x] I have updated the tests accordingly (required for a bug fix or a new feature).
[ ] I have updated the documentation accordingly.

@vmoens @shagunsodhani Feel free to have a look!

cc @albertbou92 @SebastianAment @Benjamin-eecs

I think your benchmarks could be improved for parallel env
Starting the workers take a tremendous amount of time, sometimes > 30 secs, but once it's started it should be faster.
The first few rollouts collected should be discarded from the benchmark

I think your benchmarks could be improved for parallel env Starting the workers take a tremendous amount of time, sometimes > 30 secs, but once it's started it should be faster. The first few rollouts collected should be discarded from the benchmark

Indeed, that was the case - after I excluded environment creation from the benchmarked time, ParallelEnv became faster than SerialEnv. Updated the benchmarking script and the PR description


x/website:

Overview: on page https://golang.org/doc/tutorial/greetings-multiple-people, I'd expect the line messages, err := greetings.Hellos(names) to be bolded and in blue (i.e. rgb(0, 125, 156)), rather than in the default black font. This is because the function invocation greetings.Hello has been changed to greetings.Hellos in this step of the tutorial

What version of Go are you using (go version)?

$ go version
go version go1.16.5 darwin/amd64

Does this issue reproduce with the latest release?
It's a website formatting issue, not dependent on the go version
What operating system and processor architecture are you using (go env)?
It's a website formatting issue, not dependent on the go version
What did you do?
Visited https://golang.org/doc/tutorial/greetings-multiple-people and worked through the tutorial
What did you expect to see?
The following bolded, with color blue (i.e. rgb(0, 125, 156))

messages, err := greetings.Hellos(names)

What did you see instead?
The following in the normal formatting:

messages, err := greetings.Hellos(names)

Also, the line fmt.Println(messages) should be bolded and in blue as well

Note: this would be fixed by cl 277454, currently under review (waiting for the author).


Redesign the Blog Section

Description
I would like to redesign the Blogs section according to UI of the website.
Kindly assign this issue to me.
Thank you !!
Screenshots
No response

@Ritika122001 go for it


Experiment localisation

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

#4549  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nalcalag and the rest of your teammates on  Graphite


Allow paths other then /etc/nixos?

Hello.
My system configuration is in ~/repos/nixos-configs, and it's a flake based system. Running nix-olde I get:
Fetching 'repology'
Fetching 'installed'
Fetching 'available'
thread 'thread '<unnamed><unnamed>' panicked at '' panicked at '/etc/nixos does not exist: Os { code: 2, kind: NotFound, message: "No such file or directory" }/etc/nixos does not exist: Os { code: 2, kind: NotFound, message: "No such file or directory" }', ', src/available.rssrc/installed.rs::20:53
43note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
:61
'available' done, took 0.00 s.
'installed' done, took 0.00 s.
'repology' done, took 83.22 s.
thread 'main' panicked at 'a scoped thread panicked', src/main.rs:39:9

And it seems that there is no observable way to make it search for my nixos config elsewhere.

Yeah, right now nix-olde hardcodes /etc/nixos in a few places: https://github.com/trofi/nix-olde/blob/main/src/installed.rs#L22
I think it's worth adding  --flake argument by mimicking existing tools (maybe nixos-rebuild?)
How do you usually specify non-default flake in things like nixos-rebuild?

I think it's worth adding  --flake argument by mimicking existing tools (maybe nixos-rebuild?)
How do you usually specify non-default flake in things like nixos-rebuild?

Just like you imagined. Here are example usages I imagine for nix-olde:
# Default to the $HOSTNAME flake
nix-olde --flake ~/repos/nixos-configs
# Specify a specific configuration
nix-olde --flake gitlab:doronbehar/nixos-configs#office
# Override flake inputs
nix-olde --flake gitlab:doronbehar/nixos-configs#home --override-input nur github:doronbehar/nur/my-fork --override-input nixpkgs github:doronbehar/nixpkgs/my-pkg-update

Whereas the --override-input flags are also supported by at least nixos-rebuild. Maybe the --nixpkgs flag should be removed / aliased to --override-input nixpkgs.

Did absolute minimum amount of plumbing in https://github.com/trofi/nix-olde/commit/f5ffced3c92818666128479a868e9fdc8f7b0e92 to at least be able to redirect from /etc/nixos directory.
I wonder if it makes the tool to run for you.
I'll add .#attr a bit later.

I wonder if it makes the tool to run for you.

No:
Fetching 'repology'
Fetching 'installed'
Fetching 'available'
'installed' done, took 19.92 s.
'available' failed, took 29.58 s.
'repology' failed, took 33.62 s.

Error: MultipleErrors([Canceled("Repology fetch"), CommandFailed { cmd: ["nix-env", "-qa", "--json", "--arg", "config", "import <nixpkgs/pkgs/top-level/packages-config.nix>", "--option", "build-users-group", "\"\""], output: Output { status: ExitStatus(unix_wait_status(256)), stdout: "", stderr: "error:\n       … in the left operand of the update (//) operator\n\n         at /home/doron/.nix-defexpr/channels_root/nixos/lib/attrsets.nix:637:18:\n\n          636|   recurseIntoAttrs =\n          637|     attrs: attrs // { recurseForDerivations = true; };\n             |                  ^\n          638|\n\n       error: attribute 'zabbix60' missing\n\n       at /nix/store/jkd86nm1y8ihxh8yj9h4c366121hrgq9-source/pkgs/top-level/packages-config.nix:9:86:\n\n            8|   # purposes of the index.\n            9|   packageOverrides = super: with super; lib.mapAttrs (_: set: recurseIntoAttrs set) {\n             |                                                                                ^\n           10|     inherit (super)\n       Did you mean one of zabbix40, zabbix50 or zabbix?\n" } }])

Aha, that is a bit of progress. This way it's slightly more readable:
"""
Error: MultipleErrors([Canceled("Repology fetch"), CommandFailed { cmd: ["nix-env", "-qa", "--json", "--arg", "config", "import <nixpkgs/pkgs/top-level/packages-config.nix>", "--option", "build-users-group", """"], output: Output { status: ExitStatus(unix_wait_status(256)), stdout: "", stderr: "error:\n       … in the left operand of the update (//) operator\n\n         at /home/doron/.nix-defexpr/channels_root/nixos/lib/attrsets.nix:637:18:\n\n          636|   recurseIntoAttrs =\n          637|     attrs: attrs // { recurseForDerivations = true; };\n             |                  ^\n          638|\n\n       error: attribute 'zabbix60' missing\n\n       at /nix/store/jkd86nm1y8ihxh8yj9h4c366121hrgq9-source/pkgs/top-level/packages-config.nix:9:86:\n\n            8|   # purposes of the index.\n            9|   packageOverrides = super: with super; lib.mapAttrs (_: set: recurseIntoAttrs set) {\n             |                                                                                ^\n           10|     inherit (super)\n       Did you mean one of zabbix40, zabbix50 or zabbix?\n" } }])
"""
I wonder why your nixpkgs attribute does not evaluate as is and complains about zabbix60. Could it be that you have invalid overlays in .config/nixpkgs/config.nix or similar?
A simple test would be to run something like this:
$ nix-env -qa --json --arg config 'import <nixpkgs/pkgs/top-level/packages-config.nix>' --option build-users-group '""' -I nixpkgs=https://github.com/NixOS/nixpkgs/archive/refs/heads/master.tar.gz -f https://github.com/NixOS/nixpkgs/archive/refs/heads/master.tar.gz

It's expected to finish successfully and print package attributes available in master branch as:
{
  "AMB-plugins": {
    "name": "AMB-plugins-0.8.1",
    "outputName": "out",
    "outputs": {
      "out": null
    },
    "pname": "AMB-plugins",
    "system": "x86_64-linux",
    "version": "0.8.1"
  },
...

If it fails let's try to figure out why.

A simple test would be to run something like this:

That command doesn't fail and it prints a similar json. I hope that these nixpkgs tarballs are not injected in the code, because it doesn't make sense if someone uses their own nixpkgs fork defined in their flake.

That command doesn't fail and it prints a similar json.

Aha, that's reassuring. There is probably something more subtle that is happening in your flake.

I hope that these nixpkgs tarballs are not injected in the code, because it doesn't make sense if someone uses their own nixpkgs fork defined in their flake.

Yeah, it's not exactly what nix-olde does, but an approximation: the command is the same, but the path to nixpkgs is a local path inferred from installed system. But it's not very robust and relies on the heuristics: it looks up a flake input named nixpkgs (somewhat more reliable) and falls back to <nixpkgs> if that exists (a lot less reliable as people sometimes have <nixpkgs> vary quite a lot between users).
I'll try to add a bit more logging into verbose mode so it would be obvious which methods are attempted to extract local nixpkgs paths.

Try running https://github.com/trofi/nix-olde/commit/c638b6c5bd8bbb37314429ebde75e6dd1e0767a3 with -v. It should print all the executed commands.
For me installed packages are extracted with the following commands:
DEBUG: Running ["nix", "--extra-experimental-features", "nix-command", "--extra-experimental-features", "flakes", "flake", "archive", "/etc/nixos", "--json"]: ExitStatus(unix_wait_status(0))
...
DEBUG: Running ["nix-env", "-qa", "--json", "--arg", "config", "import <nixpkgs/pkgs/top-level/packages-config.nix>", "--option", "build-users-group", "\"\"", "-I", "nixpkgs=/nix/store/7ciackz23d1j54yaa57cynp0ksawrzy5-source", "-f", "/nix/store/7ciackz23d1j54yaa57cynp0ksawrzy5-source"]: ExitStatus(unix_wait_status(0))

-vv should print all command outputs, but it will be quite a bit to sift through. I hope the commands themselves should be enough to rerun manually and see why it fails to infer for you.

Thanks for the update, here's the output of it:
 nix --tarball-ttl 0 run github:trofi/nix-olde -- -v --flake repos/nixos-configs
Fetching 'repology'
DEBUG: Fetching from repology: ""
Fetching 'installed'
Fetching 'available'
DEBUG: Running ["nix", "--extra-experimental-features", "nix-command", "--extra-experimental-features", "flakes", "flake", "archive", "/home/doron/repos/nixos-configs", "--json"]: ExitStatus(unix_wait_status(256))
DEBUG: Running ["curl", "--compressed", "https://repology.org/api/v1/projects/?inrepo=nix_unstable&outdated=1"]: ExitStatus(unix_wait_status(0))
DEBUG: Fetching from repology: "beneath-a-steel-sky/"
DEBUG: Running ["curl", "--compressed", "https://repology.org/api/v1/projects/beneath-a-steel-sky/?inrepo=nix_unstable&outdated=1"]: ExitStatus(unix_wait_status(0))
DEBUG: Fetching from repology: "cloudflare-warp/"
DEBUG: Running ["curl", "--compressed", "https://repology.org/api/v1/projects/cloudflare-warp/?inrepo=nix_unstable&outdated=1"]: ExitStatus(unix_wait_status(0))
DEBUG: Fetching from repology: "droidcam-obs/"
DEBUG: Running ["nix", "--extra-experimental-features", "nix-command", "--extra-experimental-features", "flakes", "eval", "--impure", "--raw", "/home/doron/repos/nixos-configs#nixosConfigurations.office.config.system.build.toplevel.drvPath"]: ExitStatus(unix_wait_status(0))
DEBUG: Running ["curl", "--compressed", "https://repology.org/api/v1/projects/droidcam-obs/?inrepo=nix_unstable&outdated=1"]: ExitStatus(unix_wait_status(0))
DEBUG: Fetching from repology: "fluentd/"
DEBUG: Running ["nix", "--extra-experimental-features", "nix-command", "show-derivation", "-r", "/nix/store/yjvmdhgcyg8fvqyyxbahw1z8qlpq5yql-nixos-system-office-23.11.20230725.ef99fa5.drv"]: ExitStatus(unix_wait_status(0))
'installed' done, took 12.46 s.
DEBUG: Running ["curl", "--compressed", "https://repology.org/api/v1/projects/fluentd/?inrepo=nix_unstable&outdated=1"]: ExitStatus(unix_wait_status(0))
DEBUG: Fetching from repology: "gnome:taskwhisperer/"
DEBUG: Running ["curl", "--compressed", "https://repology.org/api/v1/projects/gnome:taskwhisperer/?inrepo=nix_unstable&outdated=1"]: ExitStatus(unix_wait_status(0))
DEBUG: Fetching from repology: "haskell:amazonka-elasticache/"
DEBUG: Running ["curl", "--compressed", "https://repology.org/api/v1/projects/haskell:amazonka-elasticache/?inrepo=nix_unstable&outdated=1"]: ExitStatus(unix_wait_status(0))
DEBUG: Fetching from repology: "haskell:diohsc/"
DEBUG: Running ["curl", "--compressed", "https://repology.org/api/v1/projects/haskell:diohsc/?inrepo=nix_unstable&outdated=1"]: ExitStatus(unix_wait_status(0))
DEBUG: Fetching from repology: "haskell:lambdabot-haskell-plugins/"
DEBUG: Running ["curl", "--compressed", "https://repology.org/api/v1/projects/haskell:lambdabot-haskell-plugins/?inrepo=nix_unstable&outdated=1"]: ExitStatus(unix_wait_status(0))
DEBUG: Fetching from repology: "haskell:stb-image-redux/"
DEBUG: Running ["nix-env", "-qa", "--json", "--arg", "config", "import <nixpkgs/pkgs/top-level/packages-config.nix>", "--option", "build-users-group", "\"\""]: ExitStatus(unix_wait_status(256))
'available' failed, took 19.55 s.
DEBUG: Running ["curl", "--compressed", "https://repology.org/api/v1/projects/haskell:stb-image-redux/?inrepo=nix_unstable&outdated=1"]: ExitStatus(unix_wait_status(0))
'repology' failed, took 21.47 s.

Error: MultipleErrors([Canceled("Repology fetch"), CommandFailed { cmd: ["nix-env", "-qa", "--json", "--arg", "config", "import <nixpkgs/pkgs/top-level/packages-config.nix>", "--option", "build-users-group", "\"\""], output: Output { status: ExitStatus(unix_wait_status(256)), stdout: "", stderr: "error:\n       … in the left operand of the update (//) operator\n\n         at /home/doron/.nix-defexpr/channels_root/nixos/lib/attrsets.nix:637:18:\n\n          636|   recurseIntoAttrs =\n          637|     attrs: attrs // { recurseForDerivations = true; };\n             |                  ^\n          638|\n\n       error: attribute 'zabbix60' missing\n\n       at /nix/store/jkd86nm1y8ihxh8yj9h4c366121hrgq9-source/pkgs/top-level/packages-config.nix:9:86:\n\n            8|   # purposes of the index.\n            9|   packageOverrides = super: with super; lib.mapAttrs (_: set: recurseIntoAttrs set) {\n             |                                                                                ^\n           10|     inherit (super)\n       Did you mean one of zabbix40, zabbix50 or zabbix?\n" } }])

It seems that the main difference is your -I nixpkgs=/nix/store/...-source and -f /nix/store/...-source arguments. Here is my flake's repo:
https://gitlab.com/doronbehar/nixos-configs

Aha, it failed to extract the path to nixpkgs from this command:

DEBUG: Running ["nix", "--extra-experimental-features", "nix-command", "--extra-experimental-features", "flakes", "flake", "archive", "/home/doron/repos/nixos-configs", "--json"]: ExitStatus(unix_wait_status(256)

Probably because some of it's inputs are not accessible to the current user:
$ flake archive .# --json
fatal: '/etc/nixos/syncthing-private' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
error:
       … while fetching the input 'git+file:///etc/nixos/syncthing-private?ref=refs/heads/master&rev=37c29e867275266c06ffd6be84ceebe321137f86'

       error: program 'git' failed with exit code 128

Aish, I see. I moved my /etc/nixos folder from there to ~/repos/nixos-configs because I had permissions issues with that folder, and I forgot to fix the path to the flake. Now using a proper url, it seems to work! Thanks.
It would have been nice if nix-olde would have been more specific in it's error - mentioning exactly which nix command returned non zero exit code.

Yeah, that sounds very reasonable. It will require a bit of restructuring as nix-olde does a bit of probing for both flake and non-flake systems and would otherwise spam a bit too much.
I added a tiny hint in https://github.com/trofi/nix-olde/commit/edc610f8a7d9a17466fa09aa577b280ab097c37a.
Still need to expand support for more complete flake syntax input and show a nicer banner what nix-olde inferred out of system probing.

Thanks @trofi for all the work! Indeed now this works well. I tried to reproduce the issue, by pointing nix-olde to an old commit of my nixos-configs flake, and so I tried:
nix --option tarball-ttl 0 run github:trofi/nix-olde -- --flake gitlab:doronbehar/nixos-configs/39a5e69fdc1a5705b64d1c9d24157582b0f68362#ZENIX

And it did not fail. Could it be that nix-olde doesn't parse the last part of the flake url?

I think it should work, but not sure. Added the following minimal error printing by default: https://github.com/trofi/nix-olde/commit/9ea7a525e06a0feb865bd0dc998a172b0f17714b
It prints the following for your command:
$ LANG=C nix run .# -- --flake gitlab:doronbehar/nixos-configs/39a5e69fdc1a5705b64d1c9d24157582b0f68362#ZENIX
INFO: Failed to canonicalize path gitlab:doronbehar/nixos-configs/39a5e69fdc1a5705b64d1c9d24157582b0f68362. Assuming flake syntax.
Fetching 'installed'
Fetching 'available'
Fetching 'repology'
INFO: Failed running ["nix", "--extra-experimental-features", "nix-command", "--extra-experimental-features", "flakes", "flake", "archive", "gitlab:doronbehar/nixos-configs/39a5e69fdc1a5705b64d1c9d24157582b0f68362", "--json"]: ExitStatus(unix_wait_status(256))
INFO: Result of ["nix", "--extra-experimental-features", "nix-command", "--extra-experimental-features", "flakes", "flake", "archive", "gitlab:doronbehar/nixos-configs/39a5e69fdc1a5705b64d1c9d24157582b0f68362", "--json"]: Output { status: ExitStatus(unix_wait_status(256)), stdout: "", stderr: "\r\u{1b}[Kfatal: '/etc/nixos/syncthing-private' does not appear to be a git repository\nfatal: Could not read from remote repository.\n\nPlease make sure you have the correct access rights\nand the repository exists.\n\r\u{1b}[Kerror:\n       … while fetching the input 'git+file:///etc/nixos/syncthing-private?ref=refs/heads/master&rev=37c29e867275266c06ffd6be84ceebe321137f86'\n\n       error: program 'git' failed with exit code 128\n" }

Still a /etc/nixos/syncthing-private repo. It is referred in https://gitlab.com/doronbehar/nixos-configs/-/blob/39a5e69fdc1a5705b64d1c9d24157582b0f68362/flake.nix#L4 in that commit. Failure looks intended (at least for me).
I wonder what failure you see now.

I wonder what failure you see now.

I am not sure, but what bothers me is that I don't see any failure :). Perhaps my /nix/store still has that flake source, and that's why it is possible for it to be evaluated. I propose this because that flake hasn't changed it's source hash in the commits that fixed the syncthing-private.url to a valid url. Hence perhaps it's not a real issue with nix-olde, and this could be closed :)
Thanks for all the work and help so far :). This tool is very valuable :rocket: .

Let's declare it done then as you got something running. I'll try to make a few related improvements separately.


python310Packages.azure-keyvault: 4.1.0 -> 4.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.azure-keyvault is: "This is the Microsoft Azure Key Vault Client Library"
meta.homepage for python310Packages.azure-keyvault is: "https://github.com/Azure/azure-sdk-for-python"
meta.changelog for python310Packages.azure-keyvault is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 4.2.0 with grep in /nix/store/ziv5rhp7x86v4md4pc7j2d0pprqpsmj3-python3.10-azure-keyvault-4.2.0

found 4.2.0 in filename of file in /nix/store/ziv5rhp7x86v4md4pc7j2d0pprqpsmj3-python3.10-azure-keyvault-4.2.0

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

3 package rebuild(s)

3 x86_64-linux rebuild(s)
3 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
azure-cli
python310Packages.azure-keyvault
python39Packages.azure-keyvault

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ziv5rhp7x86v4md4pc7j2d0pprqpsmj3-python3.10-azure-keyvault-4.2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.azure-keyvault https://github.com/r-ryantm/nixpkgs/archive/5113bd5f12995611089c6d13f41e3da776b6c5e9.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ziv5rhp7x86v4md4pc7j2d0pprqpsmj3-python3.10-azure-keyvault-4.2.0
ls -la /nix/store/ziv5rhp7x86v4md4pc7j2d0pprqpsmj3-python3.10-azure-keyvault-4.2.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

azure-cli
python310Packages.azure-keyvault
python39Packages.azure-keyvault

Maintainer pings
cc @jonringer for testing.

Result of nixpkgs-review pr 166749 run on x86_64-linux 1

3 packages built:

azure-cli
python310Packages.azure-keyvault
python39Packages.azure-keyvault


Update key64x64x64 Object Index

I think that is over-constraining on the use case, so I am updating the index only require that {primary,secondary,tertiary} is unique, but {secondary,tertiary} and {tertiary} need not be unique.
I have included the necessary patch, please verify this works and generate a pull request.
--- a/libraries/chain/include/eos/chain/key_value_object.hpp
+++ b/libraries/chain/include/eos/chain/key_value_object.hpp
@@ -191,18 +191,19 @@ namespace eos { namespace chain {
                member<key64x64x64_value_object, AccountName, &key64x64x64_value_object::code>,
                member<key64x64x64_value_object, AccountName, &key64x64x64_value_object::table>,
                member<key64x64x64_value_object, uint64_t, &key64x64x64_value_object::secondary_key>,
-               member<key64x64x64_value_object, uint64_t, &key64x64x64_value_object::tertiary_key>
-            >,
-            composite_key_compare< std::less<AccountName>,std::less<AccountName>,std::less<AccountName>,std::less<uint64_t>,std::less<uint64_t> >
+               member<key64x64x64_value_object, uint64_t, &key64x64x64_value_object::tertiary_key>,
+               member<key64x64x64_value_object, typename key64x64x64_value_object::id_type, &key64x64x64_value_object::id>
+            >
          >,
          ordered_unique<tag<by_scope_tertiary>,
             composite_key< key64x64x64_value_object,
                member<key64x64x64_value_object, AccountName, &key64x64x64_value_object::scope>,
                member<key64x64x64_value_object, AccountName, &key64x64x64_value_object::code>,
                member<key64x64x64_value_object, AccountName, &key64x64x64_value_object::table>,
-               member<key64x64x64_value_object, uint64_t, &key64x64x64_value_object::tertiary_key>
+               member<key64x64x64_value_object, uint64_t, &key64x64x64_value_object::tertiary_key>,
+               member<key64x64x64_value_object, typename key64x64x64_value_object::id_type, &key64x64x64_value_object::id>
             >,
-            composite_key_compare< std::less<AccountName>,std::less<AccountName>,std::less<AccountName>,std::less<uint64_t> >
+            composite_key_compare< std::less<AccountName>,std::less<AccountName>,std::less<AccountName>,std::less<uint64_t>,std::less<typename key64x64x64_value_object::id_type> >
          >
       >
    >;

Im still working on this issue, the patch applies OK but some record handling functions are broken. next_record/prev_record for instance, now have to first find the record (using the primary_scope) and then project the iterator found into the proper index to know which record follows/precedes. This also leads to a simplification of the record handling functions that needs to be finished.


Show progress in Listr for slow tasks

Listr can appear to hang when doing a slow operation like downloading images for the first time. We should show this progress.
Expected Behavior
Progress bar should be included in Listr output
Current Behavior
Listr appears to hang
Possible Solution
Redirect stream from docker-compose to Listr task.output?
Alternatives Considered

Additional Context

Oh yeah, that would be great. Just thought about it yesterday - mn-bootstrap works very slowly in Travis and no build passes at the moment, so that would help a lot with that

I got really stuck here trying to understand streams and promises, so I'm not sure if this approach is correct. Output currently looks like this:
strophy@X250:~/Code/docker-dashd$ mn setup
mn-bootstrap 0.18.0-dev.1

✔ Set configuration preset
  › Selected testnet as default config
✔ Set node type
  › Selected fullnode type
✔ Configure external IP address
  › <IP_ADDRESS> is set
✔ Initialize Tenderdash
✔ Update config
strophy@X250:~/Code/docker-dashd$ mn start
✔ Start testnet full node
  ✔ Check node is not started
  ✔ Start services
Creating network "dash_masternode_testnet_default" with the default driver
Creating volume "dash_masternode_testnet_core_data" with default driver
Creating volume "dash_masternode_testnet_drive_abci_data" with default driver
Creating volume "dash_masternode_testnet_drive_mongodb_data" with default driver
Pulling dapi_tx_filter_stream (dashpay/dapi:0.17)...
0.17: Pulling from dashpay/dapi
Digest: sha256:4a4de51e33d44b028adeecd34629eb07d01874c3bedc304ef0efdd761f68eac2
Status: Downloaded newer image for dashpay/dapi:0.17
Creating dash_masternode_testnet_drive_mongodb_1 ... done
Creating dash_masternode_testnet_core_1          ... done
Creating dash_masternode_testnet_sentinel_1      ... done
Creating dash_masternode_testnet_drive_abci_1    ... done
Creating dash_masternode_testnet_dapi_insight_1     ... done
Creating dash_masternode_testnet_drive_tenderdash_1      ... done
Creating dash_masternode_testnet_dapi_tx_filter_stream_1 ... done
Creating dash_masternode_testnet_dapi_api_1              ... done
Creating dash_masternode_testnet_dapi_envoy_1            ... done
Creating dash_masternode_testnet_dapi_nginx_1            ... done
strophy@X250:~/Code/docker-dashd$

I wonder if we could at least match the indentation of the task displaying the output somehow? And maybe filter to only show some of the output (e.g. lines starting with "Pulling")?

I got really stuck here trying to understand streams and promises, so I'm not sure if this approach is correct. Output currently looks like this:
strophy@X250:~/Code/docker-dashd$ mn setup
mn-bootstrap 0.18.0-dev.1

✔ Set configuration preset
  › Selected testnet as default config
✔ Set node type
  › Selected fullnode type
✔ Configure external IP address
  › <IP_ADDRESS> is set
✔ Initialize Tenderdash
✔ Update config
strophy@X250:~/Code/docker-dashd$ mn start
✔ Start testnet full node
  ✔ Check node is not started
  ✔ Start services
Creating network "dash_masternode_testnet_default" with the default driver
Creating volume "dash_masternode_testnet_core_data" with default driver
Creating volume "dash_masternode_testnet_drive_abci_data" with default driver
Creating volume "dash_masternode_testnet_drive_mongodb_data" with default driver
Pulling dapi_tx_filter_stream (dashpay/dapi:0.17)...
0.17: Pulling from dashpay/dapi
Digest: sha256:4a4de51e33d44b028adeecd34629eb07d01874c3bedc304ef0efdd761f68eac2
Status: Downloaded newer image for dashpay/dapi:0.17
Creating dash_masternode_testnet_drive_mongodb_1 ... done
Creating dash_masternode_testnet_core_1          ... done
Creating dash_masternode_testnet_sentinel_1      ... done
Creating dash_masternode_testnet_drive_abci_1    ... done
Creating dash_masternode_testnet_dapi_insight_1     ... done
Creating dash_masternode_testnet_drive_tenderdash_1      ... done
Creating dash_masternode_testnet_dapi_tx_filter_stream_1 ... done
Creating dash_masternode_testnet_dapi_api_1              ... done
Creating dash_masternode_testnet_dapi_envoy_1            ... done
Creating dash_masternode_testnet_dapi_nginx_1            ... done
strophy@X250:~/Code/docker-dashd$

I wonder if we could at least match the indentation of the task displaying the output somehow? And maybe filter to only show some of the output (e.g. lines starting with "Pulling")?

Can we also show an error if a docker container keeps restarting ? (Drive likes to do that, then you just see a stuck Task without a title but you don't know if it just tasks long or if something has gone bad in the background)


Added website docs and mdoc.

Type of change: documentation

Impact: no functional change

Development Phase:  implementation
Release Notes
Added living documentation with repository. This will enable us to enforce better documentation of new features as part of their upstream process.

You're just doing this so that you get git credit for all of these lines....

Well SOMEBODY's gotta get that git credit, eh?? :P

lol


Don't overwrite last line

So I discovered that if the last line of an output does not have a newline attached (e.g. cat a file that doesn't end with \n, echo -n, etc.), this will just overwrite it with the prompt and carry on. I don't think this is desirable behaviour, so I've written a quick fix for it. There should now be a % with background and foreground inverted at the end of such a line, and the prompt will be on the next one (similar to the way zsh handles this case).

Hi!
I agree this is not the desired behaviour, and I like it the way zsh handles.
I was working on some large refactorings to the parsing module, which introduced a merge conflict here. That's why I hope you don't mind me taking over and implementing the behaviour you submitted in 16d40189fe486fc0fc6104fe280ce9ce222e2bec.
The changes will go out with the next release.
Cheers,
Sam


Improve LED instrument

Use two Pixmaps to signal on/off state.  Might require implementing image editor first.

Implemented in 935a244af4d09d21e4b3d8107a73bbb889244b66


Ftr: support drop index ast

support drop index
#167

Codecov Report

Merging #207 (b76f955) into master (055f7bf) will decrease coverage by 0.09%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #207      +/-   ##
==========================================
- Coverage   36.81%   36.72%   -0.10%
==========================================
  Files          83       84       +1
  Lines       12914    12946      +32
==========================================
  Hits         4754     4754
- Misses       7567     7599      +32
  Partials      593      593

Impacted Files
Coverage Δ

pkg/runtime/ast/ast.go
77.80% <0.00%> (-0.89%)
:arrow_down:

pkg/runtime/ast/drop_index.go
0.00% <0.00%> (ø)

pkg/runtime/ast/proto.go
57.14% <ø> (ø)

pkg/runtime/optimize/optimizer.go
28.50% <0.00%> (-0.38%)
:arrow_down:

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 055f7bf...b76f955. Read the comment docs.


Release on NPM?

Hi,
I hope I'm not missing something obvious, but can this be added to a project via npm and if not are there plans to make this possible? I am working on a project where we probably want to use this package but not directly inside a webpage so I'm not sure if the recommended method of <script src="https://loc8.us/maplibre-gl-vector-text-protocol/dist/maplibre-gl-vector-text-protocol.js"></script> works for our use case.
Any suggestions?
Cheers,
Pete

I have been thinking about creating an NPM package for this. I've actually never done it, so there will be a learning curve for it, but I think it should be possible. Are you using Webpack, Rollup, or something else to compile your website? This project uses Rollup.

Great library! I'm sure you'll find publishing to npm quite straightforward. Definitely recommend it for visibility and easy of use.
In the meantime, it can be installed directly from GitHub like so:
npm install git+https://github.com/jimmyrocks/maplibre-gl-vector-text-protocol.git

I finally got this up in NPM: https://www.npmjs.com/package/maplibre-gl-vector-text-protocol


Fixed wrong frame parsing

LGTM

Green(ish, other than gcov for travis and a c/opt for macosx for jenkins)

Yeah, will merge when I'm not in bed :)
On Wed, Jul 15, 2015, 12:32 AM David G. Quintas<EMAIL_ADDRESS>wrote:

Green(ish, other than gcov for travis and a c/opt for macosx for jenkins)
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/pull/2450#issuecomment-121515051.


Service Dialog Import Hangs

When importing Service Dialogs the UI hangs the logs reveal the following error
FATAL -- : Error caught: [ArgumentError] The partial name (layouts/listnav/) is not a valid Ruby identifier; make sure your partial name starts with underscore, and is followed by any combination of letters, numbers and underscores.
17245 /Users/madhukanoor/.gem/ruby/2.2.2/gems/actionview-4.2.3/lib/action_view/renderer/partial_rend      erer.rb:541:in raise_invalid_identifier' 17246 /Users/madhukanoor/.gem/ruby/2.2.2/gems/actionview-4.2.3/lib/action_view/renderer/partial_rend      erer.rb:522:in retrieve_variable'
17247 /Users/madhukanoor/.gem/ruby/2.2.2/gems/actionview-4.2.3/lib/action_view/renderer/partial_rend      erer.rb:392:in setup' 17248 /Users/madhukanoor/.gem/ruby/2.2.2/gems/actionview-4.2.3/lib/action_view/renderer/partial_rend      erer.rb:293:in render'
17249 /Users/madhukanoor/.gem/ruby/2.2.2/gems/actionview-4.2.3/lib/action_view/renderer/renderer.rb:      47:in render_partial' 17250 /Users/madhukanoor/.gem/ruby/2.2.2/gems/actionview-4.2.3/lib/action_view/renderer/renderer.rb:      21:in render'
17251 /Users/madhukanoor/.gem/ruby/2.2.2/gems/actionview-4.2.3/lib/action_view/helpers/rendering_hel      per.rb:32:in render' 17252 /Users/madhukanoor/.gem/ruby/2.2.2/bundler/gems/jquery-rjs-33e0fc1c0b25/lib/jquery-rjs/renderi      ng.rb:8:in render_with_update'
17253 /Users/madhukanoor/devsrc/manageiq/app/views/layouts/_listnav.html.haml:5:in _app_views_layou      ts__listnav_html_haml__2019762996276986399_70250728227900' 17254 /Users/madhukanoor/.gem/ruby/2.2.2/gems/actionview-4.2.3/lib/action_view/template.rb:145:in b      lock in render'
17255 /Users/madhukanoor/.gem/ruby/2.2.2/gems/activesupport-4.2.3/lib/active_support/notifications.r      b:166:in instrument' 17256 /Users/madhukanoor/.gem/ruby/2.2.2/gems/actionview-4.2.3/lib/action_view/template.rb:333:in i      nstrument'
17257 /Users/madhukanoor/.gem/ruby/2.2.2/gems/actionview-4.2.3/lib/action_view/template.rb:143:in `r      ender'

Erik C found the bug to be in app/views/layouts/_listnav.html.haml
= render :partial => "layouts/listnav/#{filename}", :locals => locals if filename

He is fixing another issue in a related bug
https://bugzilla.redhat.com/show_bug.cgi?id=1252976

Fixed in https://github.com/ManageIQ/manageiq/pull/4051


[Monitor] Remove monitor nspkg

This is not used and is no longer needed.

API change check
API changes are not detected in this pull request.

Needs https://github.com/Azure/azure-sdk-for-python/pull/27101 in first.


Allow long values in the key or value fields for URIExtractionNamespace

Sometimes people like to put Long values in JSON or other stuff. This helps make sure those cases get parsed correctly without having to do an extra ETL step to convert it to string

LGTM

:+1:

can we add a test?

@xvrl added test


In Schools Summary Report tab change letter grade to ELA score

Is this what is supposed to change?  The badge in the top corner of the window?

No it is the letter grades in this column.

ahh...  Gotcha.  Thanks.


Conditioned sample

Hello, can your code  implement conditional sampling?

No, I didn't implement that.


Add CfP to homepage

I found it surprisingly hard to find the CfP, so thought maybe others would too.  This PR adds the CfP on the home page:

Just an idea. It's ok --  I'm not precious about it. Feel free to close or re-work if it's not really what's wanted :-)

:+1: ... @larsrh do you want to do this now or wait for the site refresh?

Can't hurt to have it on the homepage already :smile: Thanks, @d6y!


Define a class in begin/end block

This program compiles fine:
begin
  class Foo
  end
end

Should it? It doesn't seem to make sense to define a class in a begin/end block.
As soon as you add a rescue clause for example, it no longer compiles:
begin
  class Foo # Error: can't declare class dynamically
  end
rescue
end

No, it should not compile.

So this is a bit more complicated than it seems.
A macro block is not dynamic, so we want the following to work:
macro foo
  {{yield}}
end

foo do
  class Foo; end
end

The problem is: MacroInterpreter wraps {{ yield }} expression in begin ... end. I presume this is to restrict variable scope in the block?
This doesn't seem to be correctly implemented though, because simply wrapping the yield in parenthesis ({{ (yield) }}) avoids the begin ... end wrapper.
Removing that wrapper however doesn't seem to break any specs, so maybe we could just go with that?

A begin end is just another way to write parentheses. I don't mind being able to define classes and methods inside groupings of expressions. So I would personally close this issue as invalid.


main_v2 - common: Add Zephyr Kernel patches

Add Zephyr Kernel patches
Summary:

Add Zephyr Kernel patches

drivers: ipmb: Extend the ipmb buffer
drivers: i2c: Correct the timeout time as 35ms
peci: aspeed: Avoid race condition of accessing peci device.

Dependency: PR #231
Test Plan:

Build code: pass

close the PR since the codes have been merged


can it support whisper-turbo?

can it support whisper-turbo?

Yes. I have test it with deepdml/faster-whisper-large-v3-turbo-ct2 and it works correctly.

Yes. I have test it with deepdml/faster-whisper-large-v3-turbo-ct2 and it works correctly.

what was the performance difference? anything notable running largev3 vs standard faster whisper?

Of course, it has better performance than large-v3, and the model is about half the size. However, in terms of quality, I was not satisfied with it due to a significant decrease in performance for the Persian language. (English is different and has only a slight decrease.)

Yes. I have test it with deepdml/faster-whisper-large-v3-turbo-ct2 and it works correctly.

What I've noticed is that I can't use the translate function? For example with the large-v3 I can listen to the Dutch language and also transcribe it, but when I set translation to True it still gives me Dutch output. But other than that it does seem to work pretty well! Uses only half of the GPU resources compared to faster-whisper-large-v3, even though it loses a little bit of accuracy.

It does support already but with a few changes to server.py so as to add whisper-turbo as a valid model, but stay tuned for a PR which updates faster-whisper to a newer package and supports it in a future release.

I'm a little bit confused, I ran this client and it seemed to work (I think that it is using turbo)
from whisper_live.client import TranscriptionClient
client = TranscriptionClient(
  "<IP_ADDRESS>",
  9090,
  lang="en",
  translate=False,
  model="turbo",                                      # also support hf_model => `Systran/faster-whisper-small`
  use_vad=False,
  save_output_recording=False,                         # Only used for microphone input, False by Default
  output_recording_filename="./output_recording.wav", # Only used for microphone input
  max_clients=4,
  max_connection_time=600
)

client()


[ISSUE #3037] Fix Bug The Rewrite Plugin should support {PathVariable…

…} request
// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

2022-03-14 10:51:54 [boundedElastic-1] INFO  org.apache.shenyu.plugin.base.AbstractShenyuPlugin - rewrite selector success match , selector name :internal-api-demo
2022-03-14 10:51:54 [boundedElastic-1] INFO  org.apache.shenyu.plugin.base.AbstractShenyuPlugin - rewrite rule success match , rule name :/internal-api/demo/order
2022-03-14 10:51:55 [boundedElastic-1] INFO  org.apache.shenyu.plugin.base.AbstractShenyuPlugin - springCloud selector success match , selector name :/internal-api
2022-03-14 10:51:55 [boundedElastic-1] INFO  org.apache.shenyu.plugin.base.AbstractShenyuPlugin - springCloud rule success match , rule name :/internal-api/demo/**
2022-03-14 10:51:55 [boundedElastic-1] INFO  org.apache.shenyu.plugin.httpclient.AbstractHttpClientPlugin - The request urlPath is http://<IP_ADDRESS>:3737/demo/order/path/1/godfje@, retryTimes is 0, retryStrategy is current

Codecov Report

Merging #3038 (308f429) into master (5051b2b) will decrease coverage by 0.43%.
The diff coverage is 40.00%.

@@             Coverage Diff              @@
##             master    #3038      +/-   ##
============================================
- Coverage     58.84%   58.40%   -0.44%
+ Complexity     4369     4324      -45
============================================
  Files           690      691       +1
  Lines         19890    19888       -2
  Branches       1790     1788       -2
============================================
- Hits          11704    11616      -88
- Misses         7201     7289      +88
+ Partials        985      983       -2

Impacted Files
Coverage Δ

...rg/apache/shenyu/plugin/rewrite/RewritePlugin.java
50.00% <25.00%> (-10.00%)
:arrow_down:

...org/apache/shenyu/common/utils/PathMatchUtils.java
80.00% <100.00%> (+5.00%)
:arrow_up:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
68.75% <0.00%> (-31.25%)
:arrow_down:

...henyu/admin/service/impl/UpstreamCheckService.java
42.85% <0.00%> (-20.64%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
52.12% <0.00%> (-11.71%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
69.10% <0.00%> (-8.14%)
:arrow_down:

.../apache/shenyu/admin/model/dto/BatchCommonDTO.java
50.00% <0.00%> (-7.15%)
:arrow_down:

...apache/shenyu/admin/model/dto/AuthPathWarpDTO.java
57.14% <0.00%> (-7.15%)
:arrow_down:

...org/apache/shenyu/admin/model/dto/AuthPathDTO.java
58.82% <0.00%> (-5.89%)
:arrow_down:

... and 28 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 5051b2b...308f429. Read the comment docs.


Incorrect module spacing

Was fixed in https://github.com/cdelker/ziamath/issues/9, but happens again.

Fixed by 3a745a132ace473c41c1bdc6c68106f305ea5a95. Appeared to be an issue with \left| and \right|, but not | by itself.


Fix histogram counting algorithm for filtered items

This version properly counts filtered items in right most bin. I've got an automated test for the algorithm, just didn't move the algorithm to a separate function yet.

/sem-approve


Which mecanism does orbitdb use for read/write access?

read/write access control isn't implemented in IPFS spec, so which mecanisme is used by orbitdb ?

IIRC, we haven't implemented this yet. One of the other maintainers should know.

Hi @maroodb!
There is currently no read-access control. Anyone with the address and network access can read the contents of the db. For write-access, an immutable list of write-capable keys must be added at db creation as shown here. Though we are in the process of changing this mechanism to add support for dynamically adding write-capabilities (we are removing orbitdb.key in favor of using an identity as orbitdb.identity). Key-revocation is still an open discussion and is not currently being implemented. I hope this answers your question!

Closing and referring people to the OrbitDB Field Manual if they have more questions.


Add Table.delete_rows function allowing deleting rows in a database table.

Just a Database API. In-Memory should have an Unsupported Operation (join Left Exclusive has same result).
For Database, ony can act on a trivial table (as per update_datebase_table).
Locked behind the Output context.
Dry run doesn't affect the table but should compute the row_count.

Table.delete_rows
    self -> ## Trivial Table whose rows are will be deleted.
    key_values_to_delete : Table -> ## Extra columns are ignored, primary key columns must exist and must map by name.
    (key_columns : Vector | Nothing = default_key_columns self) -> ## By default uses the primary key of self.
    Integer (row_count of deleted)

For example, my_students.delete_rows (student_leaving_table) ["first_name", "last_name"] returning number of rows (e.g. 31).
Additionally, we should add a function is to delete all rows.
Connection.truncate_table target_table:Table -> Integer returning the row count of deleted.
Similar to drop_table.
Additionally, flip update_database_table to be consistent:
Table.update_rows
    self ->  ## Trivial Table whose rows are will be updated.
    source_table:Table -> ## Set of data
    (update_action : Update_Action = Update_Action.Update_Or_Insert) ->
    (key_columns : Vector | Nothing = default_key_columns self) ->
    (error_on_missing_columns : Boolean = False)
    (on_problems : Problem_Behavior = Problem_Behavior.Report_Warning)

Do we feel that a "filter_rows" where we produce effectively the result but in a query state is useful?

Equivalent to a Left Exclusive join effectively.


[FLINK-3708] Scala API for CEP (initial).

This module flink-cep-scala adds a Scala pendant for the  Java CEP API to Flink.
I created Scala classes for Pattern and PatternStream, helper classes, as well as corresponding tests.
PatternStream in flink-cep is extended to obtain explicit TypeInformation from Scala code.
Pattern in flink-cep is slightly modified in the type parameters for member previous.

Thank you for this great contributio! I think that CEP and a concise Scala API go very well together. :-) I'm not a Scala expert, but I'll also have a look later.

Thanks Ufuk! I just saw that the build actually failed and it seems to me that the problem is indeed related to removing the dependency to asm from the pom.xml as Robert suggested? So I could either add the dependency again or do you see any option to solve this otherwise?
There is one remaining question that I already started to discuss with Till about the select functions in PatternStream. The Java API wants to call them with java.util.Map as parameters, but I don't think we want Java classes in our Scala API. Currently I see two options how to change this without touching the Java API:

Use Scala mutable.Map and automatic conversion.
Use Scala immutable.Map and explicit conversion. From how I interpret the actual use of the Map in this function, the semantics are actually that of an immutable map.

If the impact of the conversion is not an issue, I have already implemented version 2 (it is currently commented out in favor of a version that still uses Java Map.

Does anybody know why we needed to add the asm dependency for gelly scala? @tillrohrmann @vasia ?

It looks like it was added as part of FLINK-3136.

Thanks for your good contribution @StefanRRichter. I had some inline comments.
It would be good to revisit the link tags in the ScalaDocs and replace them with the ScalaDoc link syntax. I haven't marked all occurrences in the code.
Furthermore, it should be possible to completely throw out the ClassTag context bound. I have only marked the first occurrences where I've noticed it.
The Scala code contained several lines which exceeded the maximum line length of 100 characters. The Scala style check plugin, which detects these style violations, is executed when you run mvn verify. Thus, it is always a good idea to run mvn verify once locally before pushing commits to a PR.
Ping me once you've addressed my comments. Then I'll make sure that the PR is merged.

The current version should cover all of your feedback and could be pulled @tillrohrmann .

Good work @StefanRRichter. I had only some minor comments left. Once these are addressed, the PR should be ready to be merged :-)

All mentioned issues should be addressed now and the PR is ready.

Failing test case is unrelated. Really good work @StefanRRichter. Will merge it.
As a follow up, we should update the CEP documentation to also include Scala code examples.


Update 2.5 ARM testssl test

I have tested with the ARM cluster.
Testssl pod is deploying successfully.
assert.OutputContains(
   "P-256",
   "Results included: P-256",
   "Results not include: P-256"))

unable to find this "P-256", except this remaining 2 asserts run successfully.


ContentType crash

Stack:
Error on line 1, column 16: Invalid media type: expected "/".
  ╷
1 │ applicationjson
  │                ^
  ╵
#0      wrapFormatException (package:http_parser/src/utils.dart:15:5)
#1      new MediaType.parse (package:http_parser/src/media_type.dart:46:7)
#2      HttpRequestContext.from (package:angel_framework/src/http/http_request_context.dart:92:21)
#3      AngelHttp.createRequestContext (package:angel_framework/src/http/angel_http.dart:90:31)
#4      Driver.handleRawRequest (package:angel_framework/src/core/driver.dart:94:12)
#5      Driver.startServer.<anonymous closure>.<anonymous closure>.<anonymous closure>.<anonymous closure> (package:angel_framework/src/core/driver.dart:50:20)
#6      _RootZone.runUnaryGuarded (dart:async/zone.dart:1374:10)
#7      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:339:11)
#8      _IterablePendingEvents.handleNext (dart:async/stream_impl.dart:540:18)
#9      _PendingEvents.schedule.<anonymous closure> (dart:async/stream_impl.dart:670:7)
#10     _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
#11     _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)
#12     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#13     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:169:5)

If you send a badly formatted ContentType header, angel will crash fatally.  Can it be default behavior to send a 415 error when this happens?

#270

This project is going to be deprecated soon. I will soon update the messaging to reflect on this.
I don't think that status 415 is really the right response code here; it should be a 400, as the user sent a bad request.

1. This project is going to be deprecated soon. I will soon update the messaging to reflect on this.

I'm really sorry. You did a great job with Angel and I like it very much. I'm Sad, cause I will have to migrate my back end to Aqueduct... but this is! Many thanks.

@thosakwe
Why will this project be discontinued?

@insinfo https://www.reddit.com/r/dartlang/comments/h0z413/looks_like_the_angel_webbackend_framework_wont_be/ftpaxmo/?context=3

@thosakwe
I just read your post and I understand your arguments, if you don't mind, I would like to keep maintaining this project I don't have much time where I work, but as I am using Angel in the company where I work I would like to keep working doing bug keeping and maybe minor updates.

I don't have a problem with transferring ownership. I don't know, however, how much time I will have over the course of the next few weeks to devote to even that. So you may have to wait some time (or fork the project entirely).

So you may have to wait some time

No problem, I can wait until you have time. The problems I was having with Angel I already managed to solve in a way that suits me at the moment.
I have already solved these problems.
https://github.com/angel-dart/angel/issues/267
https://github.com/angel-dart/angel/issues/269
#this about writing dependencies is to solve a bug in the postgres driver
#which prevents it from working with dart 2.8.4 and a bug with form data
#with UTF8 data (accented characters)
dependency_overrides:
  postgres: ^2.1.1
  http_server:
    git:
      url: https://github.com/dart-lang/http_server.git
      ref: df31d20


fluid examples: copy/paste error

example READMEs reads "Run npm install from the sudoku root" for all examples

This is fixed.