[alpha] Runtime exception with --debug when Json.Encode.null in init

Using Ubuntu 16.04, and 0.18 alpha.
The code below (initialising with null) fails to start and causes an exception, while using the commented out section (with string) works fine.
module Test exposing (main)

import Html exposing (..)
import Json.Encode as E exposing (Value)

-- MODEL

type alias Model =
    { gmap : Value
    }

init : (Model, Cmd Msg)
init =
    ( Model E.null, Cmd.none )
    -- ( Model (E.string "Dummy text"), Cmd.none )

-- UPDATE
type Msg
    = North

update : Msg -> Model -> (Model, Cmd Msg)
update message model =
    model ! []
-- VIEW

view : Model -> Html Msg
view model =
    text "hello world"

main =
  Html.program
    { init = init
    , update = update
    , view = view
    , subscriptions = always Sub.none
    }

[If the fix is not to test for null values, then note that in my code that originally ran into this bug, just fixing the init value to use Encode.string "dummy string" was not enough, as I got another null value being passed to init (which I have not tracked down yet) - see https://github.com/simonh1000/elm-google-maps-2]

Thanks for the report, should be fixed if you get rid of elm-stuff/ and rebuild!


fix: remove defaultprops

Since defaultprops is going to be deprecated, these changes eliminate its use and use the javascript default params.
Warning: AdvertisingSlot: Support for defaultProps will be removed from function components in a future major release. Use JavaScript default parameters instead.

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:

Thank you guys! 🫶


Updating status of a couple of projects

The PR updates is following:
-CVS book
-Vision framework tutorial

I think this looks pretty good-nice work!


Example with addClass and removeClass and toggleClass

Would you be so kind  provide two simple examples for saving a class and toggling a class please?

Sorry, don't have the time :/


Add Table of Contents to README

Description
Currently, the README lacks a table of contents, making it challenging for users to navigate sections quickly. Adding a table of contents will enhance readability and allow users to locate information faster.
@mdazfar2 I would like to work on it.
Please assign it to me under gssoc-ext and hacktoberfest.
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

Hey @T-Fathima, why do you keep raising the same issue again and again- again and again? I’m getting frustrated now. A few days back, you raised the same issue, and I closed it.
Our README.md file is good, and people can easily understand our project.
Here is proof of your previous issue where I closed it by saying something. Check this out 👇 https://github.com/mdazfar2/Ezyshop/issues/987


Corom Browser

Corom Browser - https://github.com/WhichBrowser/Parser-PHP/issues/252

Coverage remained the same at 100.0% when pulling b72be36fa53bd61054fe3623693c134a6e9d4f05 on ayumi-cloud:Corom into 4899110cd2f87b01e04ced62dbb9dec541031dee on WhichBrowser:master.


What exactly are the sources of WebImageText dataset?

In this paper there is only a vague description about the WIT dataset:

... we constructed a new dataset of 400 million (image,
text) pairs collected form a variety of publicly available
sources on the Internet.

Can you enumerate specific sources and methods used to acquire the images?

How are texts engineered? Paper talks about prompt engineering for tasks datasets but not for the 400M training dataset!!

Looking for more information here... probably the most important tech bottleneck..


Feature/issue 624 -Remote Control Modules (LIGHT, AUDIO, HMI_SETTINGS) and parameters (SIS Data)

Fixes #624
This PR is ready for review.
Risk
This PR makes minor API changes.
Testing Plan
Added new unit tests and expanded existing ones
Summary
Added RPC's, structs, enums as defined in the Remote Control Modules (LIGHT, AUDIO, HMI_SETTINGS) and parameters (SIS Data proposal.
CLA

[x] I have signed the CLA

Codecov Report

Merging #712 into develop will increase coverage by 1.02%.
The diff coverage is 98.95%.

@@              Coverage Diff              @@
##             develop     #712      +/-   ##
=============================================
+ Coverage      41.38%   42.41%   +1.02%
- Complexity      2656     2805     +149
=============================================
  Files            362      379      +17
  Lines          15835    16121     +286
  Branches        1685     1685
=============================================
+ Hits            6554     6837     +283
- Misses          8973     8976       +3
  Partials         308      308

Impacted Files
Coverage Δ
Complexity Δ

...vicelink/proxy/rpc/ClimateControlCapabilities.java
100% <100%> (ø)
34 <8> (+8)
:arrow_up:

...in/java/com/smartdevicelink/proxy/rpc/SisData.java
100% <100%> (ø)
12 <12> (?)

.../java/com/smartdevicelink/proxy/rpc/SRGBColor.java
100% <100%> (ø)
8 <8> (?)

...com/smartdevicelink/proxy/rpc/StationIDNumber.java
100% <100%> (ø)
6 <6> (?)

.../smartdevicelink/proxy/rpc/ClimateControlData.java
100% <100%> (ø)
30 <8> (+8)
:arrow_up:

...java/com/smartdevicelink/proxy/rpc/ModuleData.java
100% <100%> (ø)
14 <6> (+6)
:arrow_up:

...om/smartdevicelink/proxy/rpc/AudioControlData.java
100% <100%> (ø)
10 <10> (?)

...devicelink/proxy/rpc/LightControlCapabilities.java
100% <100%> (ø)
6 <6> (?)

...om/smartdevicelink/proxy/rpc/LightControlData.java
100% <100%> (ø)
4 <4> (?)

...m/smartdevicelink/proxy/rpc/enums/LightStatus.java
100% <100%> (ø)
2 <2> (?)

... and 32 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 0bf0067...d8fca6b. Read the comment docs.

@theRealBrett PR has been updated, please re-review when time permits.  Thanks!

@theRealBrett enums have been updated to reflect the updated proposal.  Please re-review when time permits.  Thanks!


Topic/richard

+are available through the buttons labelled:
labeled ?

Actually, either is an acceptable spelling.  I prefer the double l, but
for no good reason.

+SPARK. while Function Incr might be legal if it doesn't modify any
SPARK. -> SPARK,

Fixed.

+here instead of an error giving you the possibility of justifying the
suggest adding a comma after 'an error'

Yup.

Small editorial remark: there are two whitespaces instead of one here and
there.

Not "here and there": it's after a period.  Old-school, I guess, but it
doesn't matter for .rst.

+Ineffective statements are different than dead code: they're executed, and
+often even modify the value of variables, but have no effect on any of the
+subprogram's visible outputs: parameters, global variables or the function
+result. Ineffective statements, like unused variables, should be avoided,
The comma after "avoided" breaks the statement unnecessarily. Is it really
correct?

Yes, because it disambiguates between
 "avoided in this case, because"

and
"avoided, in this case because"
which have two different meanings.  We mean the latter.
However, I'm trying not to write hard-to-parse sentences here because they
distract from the material of the course, so I'll change this.  I don't really
see the need to refer to undefined variables here, so I changed this to:
Ineffective statements should be avoided because they make the code
less readable and more difficult to maintain.

Maybe "by specifying them"?

Yes.

+subprogram returns, it check that global and parameter outputs were
*checks

Got it.

+the ability to accurately detect of reads of uninitialized data.
Shouldn't we remove the "of" in "to accurately detect of reads"?

Yes.

A backtick is missing for the "A" at the end of the line.

Fixed.

+analysis cannot guaranty that Result is always initialized at the call
*guarantee

Yup.

+nothing to be uninitialized.
*initialized

Fixed.


Remove unused scripts/gen-chip-version.sh

Problem
We have gen_chip_version.py that seems identical in purpuse, except
the python version is referenced from our build scripts while the
shell script is never invoked.
Remove duplicate functionality.
Change overview
Delete script, change the only reference (a readme example script naming).
Testing
CI will check that compile will still succeed.

/rebase


Remove @

Hi!
Is it possible to remove @ from dirs? It is very uncommon.

Hi @egorpugin,
The reason I use @ in the folder names, is to ensure that directories are sorted such that source folders appear at the top. So far, I haven't experienced any issues with build systems etc. complaining about the @ sign.
If you experience any issues related to the @ sign, please let me know; then I will consider removing. Otherwise, I will prefer to leave it in.

Ok, it's up to you. I'm also fine with @.


No empty auth token possible (for local usage)

I want to use the clipper with a local (on my PC) running silverbullet server. For this I do not use any authentication.
However, if I leave the token config field empty (in the clipper config page), every time I open the clipper applet, it opens the config page first.
Could the auth token be made optional?
Thanks a lot!

SilverBullet Clipper uses API calls provided by SilverBullet. SilverBullet requires authentication for API calls. I suggest that you add the SB_AUTH_TOKEN as described below. You only have to add the one time.

The host URL points to the instance of your SilverBullet installation. If you're running SilverBullet locally the URL will be something like http://<IP_ADDRESS>:3000. If you are running SilverBullet externally, or accessing it via a proxy, the host URL will be something like https://silverbullet.mydomain.com

The token will be whatever token you specified in the SB_AUTH_TOKEN environment variable.
Command Line Example
SB_USER=admin:mypassword SB_AUTH_TOKEN=mysuperlongtoken SB_HOSTNAME=<IP_ADDRESS> /root/.deno/bin/silverbullet /root/Silverbullet/space

Docker Compose Example
services:
    silverbullet:
        image: zefhemel/silverbullet
        restart: unless-stopped
        environment:
        - SB_AUTH_TOKEN=mysuperlongtoken
        - SB_USER=admin:mypassword
        volumes:
        - ./space:/space
        ports:
        - 3000:3000

Thanks for the explanation!
I'm aware of the auth features, but I do not want to use them locally (in fact, I could not get them to work probably running it on a windows machine yet, anyway):
However, everything works without the auth token, I can create new clipped notes. The only thing is, webclipper starts with the config page, as soon as the auth field is empty. If I click on save, the capture page opens and I can fill out the title and so on. After clicking on capture, the page gets created.
So from my point of view, Silverbullet does not have a problem with no/empty token.
So my only "complain" is that clipper does not want to have a empty token field :-)
Maybe you could add an option to intentionally have an empty field? Or one could enter "none" in the field, which then is used to ignore the setting or use an empty token instead?
I'm aware, that this would work with local installations only and does not make sense with public ones.
On the other hand, I DO use an (other) instance, which is reachable via internet / Https, too. However, in that case I use an authentication feature of my reverse-proxy. So the webclipper does not work anyway, as there is no way to give the authentication credentials of the proxy, which asks for them before getting to the API  :-) (But THAT works as designed, and is not any fault of the clipper, of course ;-) )

Interesting! I couldn't get clipper working without an auth token locally. I'll make the change to allow an empty token ASAP for you.

I've stopped the config page opening by default if the token is missing. This is available in v1.2.2

Nevermind!
After clicking some of the other config checkboxes and saving the config again, now also Firefox opens just the "new note" popup.


fix(kubernetes-context): added to default prompt order

As requested here: https://github.com/caiogondim/bullet-train.zsh/pull/257#issuecomment-415733705
Added it just after the aws prompt which seemed like the proper place

Sadly even when making this change I can't get the prompt to work. Saddening  =(

Sadly even when making this change I can't get the prompt to work. Saddening =(

You still need to ensure the normal kubernetes configuration envs are set:
https://github.com/caiogondim/bullet-train.zsh#kubernetes-context
And you can verify that kubectl config current-context actually returns a value.

I very certainly did. I ran through every line and it was all kosher. Set
the env var. Tested it. Tested all the if statements. Tested the grep of
the config. For whatever reason it just doesn't render the prompt. Very
sad.
On Thu, Feb 21, 2019, 12:20 AM Mark Doeswijk<EMAIL_ADDRESS>wrote:

Sadly even when making this change I can't get the prompt to work.
Saddening =(
You still need to ensure the normal kubernetes configuration envs are set:
https://github.com/caiogondim/bullet-train.zsh#kubernetes-context
And you can verify that kubectl config current-context actually returns a
value.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/caiogondim/bullet-train.zsh/pull/290#issuecomment-465905010,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABQLuaNnUtCO6vym1CkY69zxDi1WGU1gks5vPlbUgaJpZM4WMTuN
.

Hi I just ran into this. I can confirm it works :smile: Can we merge this?

Just going to close this. People can find this through search engines well enough


feat: Improve Source Generation

Improves source generation to follow soli's pattern of constructing an intermediate ConstructedSource.
Checklist

[x] Create SessionSource (ConstructedSource equivalent)
[x] Refactor parsing + decompiling into a builder pattern
[ ] Unit Tests
[x] Integrate into ChiselEnv

Merging for visibility


Support advanced loghandler metadata use cases

Description
Different logging backends support different "levels of metadata", allowing users to specify essentially indexed metadata vs. standard "extra metadata".
For example, Sentry allows users to attach data explicitly to their events as tags, while everything else is attached as "extra context".
Tags will be indexed and queryable, while "extra" are only available to view from Sentry incident reports.
However, there's no way to forward this information down to individual logs.
Concrete ask
I think there's an easy path here to support this in a backwards compatible way, as an opt-in for LogHandler conformers to implement.
Essentially, we just provide an implementation-specific prefix on metadata keys labeled as tags that can be parsed before being sent to a backend.
This way, if a log handler doesn't support it, the only negative downside is potentially "unexpected" keys, but this requires the developer to have explicitly called this API
For example
extension Logger {
  /// Accesses and stores the metadata value to be used as a tag, if supported by log handlers.
  public subscript(tagKey tagKey: String) -> Metadata.Value? {
    get {
      return self[metadataKey: "tag_\(tagKey)]
    }
    set(newValue) {
      self[metadataKey: "tag_\(tagKey)"] = newValue
    }
  }
}

Hm, but you can write this extension yourself and other libraries don't know about this convention so how would they use the "tag key" -- they dont know about it, and most backends don't respect this so it is weird to expect library authors to know and respect tag/metadata difference if they never benefit from it -- so it's more for your application code and libs, in which case -- do the extension?
Why does this have to be done in swift-log?

That's a fair argument.
We have this code locally, but it's non-trivial to get right. (I've had to fix simple mistakes twice now, with oodles of tests)
I think my reflex was that of the stereotypical "this is useful sharable code, let's put it in the library!" response.
Perhaps we want a SwiftLogExtras type of module like we've done for other libraries, where this might be able to live.


Remove lines after switch

Question
Hi!
how can I remove line after switching to other timeframe? every time it's adding a new one.
Code example
def on_timeframe_selection(chart):
    # Called when the user changes the timeframe.
    new_data = import_data(
        chart.topbar["symbol"].value, chart.topbar["timeframe"].value
    )
    if new_data.empty:
        return
    chart.set(new_data, True)
    line = chart.create_line("SMA 50")
    sma_data = calculate_sma(new_data, period=50)
    line.set(sma_data, True)

Hey, you can do for line in chart.lines():       line.delete() right after selection.

thank you, it's working fine now

The line.delete() is not working in new version 2.1.
KeyError: 'line'


Refunded items display in order summary

If an attendee has refunded items, group them together under "refunded" rather than showing them in the same list. In the payments table, say what each refund was for.

This is related to the original bug reported in #260.

@nmorduch is this something that you can fix in the templates, or do you need different template variables?

I'm not really sure. I think I tried and the way it works now, you cycle through items and then add another row to ones that are refunded. It's probably possible to cycle through once to display non-refunded items, then again to display only refunded items, but there is probably a more efficient way to do that, where there is an actual list for each, rather than just a combined list.

This is related to #316.

They also still show up on the Attendee page, but can't actually be edited because attendee editing excludes refunded passes.

Resolved with merge of #504.


Write a Kotlin program to convert centimeters to feet

Description
Write a Kotlin program to convert centimeters to feet
1 foot = 30.48 centimeters
Input  : 152.4
Output : 5

How to contribute

Save the solution in program/ConvertCentimetersToFeet.kt file
Add ConvertCentimetersToFeet.kt file in convert-centimeters-to-feet folder

!assign


Update README.md (branch master).

Current contents of README.md (branch IKNL)  do not properly reflect how to use the package.

README and Wiki have been expanded/updated.


Replace .plot_psd() with .compute_psd().plot()

Just a note that we should replace all occurrences of .plot_psd() with .compute_psd().plot().

thanks for the reminder. I should have done this long ago.


fix(dialog): account for partial visibility in visibility calculation

Closes #1101

LGTM 👍

From slack: just wanted to add that the fix doesn’t seem to work when accounting for the window scroll position

That's odd ... is it any scrolling container, or just the window? If it's just the window you can probably add the x/y scroll position in your visibility calculation to sort that out

That's odd ... is it any scrolling container, or just the window? If it's just the window you can probably add the x/y scroll position in your visibility calculation to sort that out

Yeah I'll add that as well, maybe the component is in a container that doesn't fully wrap it

Yeah I'll add that as well, maybe the component is in a container that doesn't fully wrap it

Definitly double check, should be pretty easy to wrap the component in a overflow: scroll div to test


ISPN-5897 Fix compressed log handling in filter_failed_tests.sh

https://issues.jboss.org/browse/ISPN-5897

Pulling...

Integrated into master, thanks @danberindei !


Remove pybind11 submodule

Instead of using pybind11 as a submodule, we could just depend on it as a Python package. I am bringing this up, because I just spent too much time finding a bug with newer versions of Python that were already fixed in pybind11. Also, pybind11 is mature enough to not expect regular breakage (also, it follows semantic versioning).

As a downside, manual CMake builds (outside pip) would get more complicated to do.


will feature

send messages when the server is down using scheduled tasks #69

Codecov Report

Merging #120 (1e1e816) into main (aba2165) will decrease coverage by 3.27%.
The diff coverage is 7.00%.

@@            Coverage Diff             @@
##             main     #120      +/-   ##
==========================================
- Coverage   45.54%   42.26%   -3.28%
==========================================
  Files         106      108       +2
  Lines        4514     4838     +324
  Branches      673      721      +48
==========================================
- Hits         2056     2045      -11
- Misses       2104     2433     +329
- Partials      354      360       +6

Impacted Files
Coverage Δ

...g/apache/rocketmq/mqtt/common/model/Constants.java
0.00% <ø> (ø)

...apache/rocketmq/mqtt/common/model/WillMessage.java
0.00% <0.00%> (ø)

...ocketmq/mqtt/cs/channel/DefaultChannelManager.java
6.71% <0.00%> (-67.06%)
:arrow_down:

...a/org/apache/rocketmq/mqtt/cs/session/Session.java
52.29% <0.00%> (-0.61%)
:arrow_down:

...rg/apache/rocketmq/mqtt/cs/starter/MqttServer.java
0.00% <0.00%> (ø)

...mqtt/ds/upstream/processor/SubscribeProcessor.java
84.21% <0.00%> (ø)

...org/apache/rocketmq/mqtt/example/MqttConsumer.java
0.00% <ø> (ø)

...org/apache/rocketmq/mqtt/example/MqttProducer.java
0.00% <ø> (ø)

...apache/rocketmq/mqtt/example/RocketMQConsumer.java
0.00% <ø> (ø)

...apache/rocketmq/mqtt/example/RocketMQProducer.java
0.00% <ø> (ø)

... and 10 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 aba2165...1e1e816. Read the comment docs.

add some unit tests

The KV will be removed and the code can be submitted to the Develop branch based on the state machine

This PR can be closed, see #174


ENH: Add NF90IO support

Please check that the PR fulfils our requirements

[ ] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] Needs some work on genmake2

What changes does this PR introduce?
Add NF90 netcdf global file support
What is the current behaviour?
Non-global tiled netcdf support
Does this PR introduce a breaking change?
This is not a breaking change.
Other information:
docs are in pkg/nf90io/README.rst.  Happy to help incorporate.
See https://github.com/altMITgcm/MITgcm/pull/16 for previous discussion.

Whats needed on my end for this PR?

@jklymak we have been swamped with other things, but getting back to this. Probably be good to do a call. We have a plan that it would be good to discuss. Pieces of plan are

Initial meta data writing needs fixing. @jahn has a patch for this, it speeds up a lot and makes the code usable for larger runs. Without the fixes the code is not really applicable. Current code opens, closes, resopens, rewrites prolifically and that creates a big performance hit.  The patches are against the code before your last rewrite of everything.

Check CF convention support. That includes for different grids, different time units etc.... We want downstream tools (xarray, Ferret, Panoply, Pangeo, others...) to work fully out of the box (where they can) with whatever is generated.

Leave out changes to genmake2 for now and instead include a little test pregenmake script that probes for setup.  The whole nf-config, nc-config,  pnetcdf-config worlds is a little bit of a spaghetti mess. For now we are not sure how we want to we want to tie it into genmake2. The test programs currently proposed gave false negatives on several systems we tried, so they are not quite the right way to go on their own. An example of thinking on small pregenmake script is here ( https://github.com/christophernhill/netcdfhdfmpifortranhacks ).

Fix up the headers and comments in files and split up files per standard practice. Headers and comments for a routine should try and be accurate about what the routine actually does. Our long standing practice is that files should mostly default to being one file per routine so its easier to find stuff, unless there is a strong consensus otherwise for some  pieces (which there is not for this bit of code). Anything that is multi-routine per file should have the routines in alphabetic order - that again makes finding stuff easy.

Fix up 32-bit support. Right now everything is hard coded for 64-bit. The functions should use 32-bit when the writePrec option is set for that.

@christophernhill Cool, happy to do a call to pick out which of the "easy" stuff I can do to help move this along.  Certainly I can clean up the organization and the comments.
Some of the other heavy lifting might be a bit beyond my skill set just because I don't use different grids, ferret etc. I'm completely agnostic about that sort of thing - whatever works to make the tool the most comprehensive would be great, and I'm completely not wedded to any conventions I chose to use in the PR here.  This PR works for me, but I would not be at all offended if someone just wanted to use as a template and start over from scratch.  No doubt my Fortran software engineering is no great shakes either :wink:

I have put my changes here: https://github.com/jahn/MITgcm/tree/enh-nf90io.  Re @christophernhill's bullets above:

I had to reorganize everything so that all metadata is written before any variable data.  Otherwise it was prohibitively slow on my larger setup (>30 min for creating the file).  Like this, it actually works quite well.
no change
I've put my own checks into genmake2.  I use nc-config --fflags --flibs which should work on any netCDF library younger than a decade or so, and then just try compiling the test programs.  With mpi, one also has to try running, otherwise one may not catch that a netCDF library doesn't have proper parallel access support.  With this, one shouldn't have to put anything related to netCDF into the optfile.  But one has to make sure that the correct nc-config is found.
not a lot of change
I've added support for writing in single precision.

I've also added runtime parameters for file format and MPI-IO hints.  The file format parameter allows using parallel-netcdf (if compiled into the netCDF library) instead of HDF-5, which may perform better on some systems.  I found that I had to set some MPI-IO hints to make things faster, but haven't properly benchmarked this.  There is an example in the verification setup.
I can push these changes up into the pull request if that is desired...

I’ve not looked at these changes but I’m 💯 fine with closing this and having a new PR to keep things clean. Sounds like some of the changes are great!

@jklymak @jahn was suggesting to put the changes into  jklymak/MITgcm  enh-nf90io branch. I think that makes sense?

Sure he can force push here or open a new PR.  I just suspect a new PR would be easier to follow, but whatever is easier for you guys.  I just wanted to be clear that I won’t at all be offended if it’s easier to start fresh.

@christophernhill needs to do a little tidying this week

@christophernhill still needs to do a little tidying

I'm going to close this.  All the supercomputers I have access to allow xmitgcm to now work, so I'm happy to use mds files these days.  I am also not sure that this was not really slow, whereas mds files are quite fast.  Of course anyone is more than welcome to keep working on this!  Thanks for everyone's attention to this!

I'm going to close this.  All the supercomputers I have access to allow xmitgcm to now work, so I'm happy to use mds files these days.  I am also not sure that this was not really slow, whereas mds files are quite fast.  Of course anyone is more than welcome to keep working on this!  Thanks for everyone's attention to this!


Edit existing cronjobs

Is it possible to edit existing cronjobs in the crontab? If so can you provide an example on how to do this? The only thing i see in the readme is modifying a job after you create it.

You can get a reference to all jobs. See the README


Reduce dependencies

I specifically installed captainhook/captainhook-phar to keep the dependencies small but when I also install captainhook/plugin-composer it additionally installs captainhook/captainhook as it is a dependency.
As composer does not have a feature like require package A or B could you either make a new package like captainhook/plugin-composer-phar which requires the phar or replace the dependency  of captainhook/captainhook with captainhook/captainhook-phar?
I know that there is also the possibility to do this in composer:
{
  "scripts": {
    "post-autoload-dump": "vendor/bin/captainhook install -f -s"
  }
}

But this has again issues with CI as for production dev dependencies are not installed and therefore the captainhook executable does not exists in vendor/bin

Uh thats a good point. I never thought of that. So for production you are disabling plugins? Or how do you handle that?
composer-phar is also a Composer plugin.
So what are you missing? The automatic captainhook install?

For production I call composer install --no-dev so when the post-autoload-dump script is triggered composer will exit with a non zero exit code and a message like can not find executable vendor/bin/captainhook as I install captainhook as a dev dependency.
So the composer plugin is very convenient as it is also a dev dependency. On production both are not installed so I have neither the executable nor the composer hook which is perfect. On dev both are present and everything is handled automatically.
Only now that you have mentioned that composer-phar also has a composer plugin I had a deeper look into it. Unfortunately captainhook-phar/src/Plugin.php differs a lot from plugin-composer/src/ComposerPlugin.php. E.g. it ignores the extra.captainhook.force-install config.
So I personally would prefer it if the captainhook-phar would only include the phar and no composer plugin. And a separate package either this one or a separate composer-plubin-phar would provide me just the composer plugin

P.S. potentially you could also just remove the dependency all together and mention in the documentation that additionally to the composer plugin either captainhook or captainhook-phar must be installed. As both put a captainhook executable into vendor/bin it does not matter which one is installed.
You could also check for the presence of the executable in the hook and if it is not present print out a warning like Please install either captainhook/captainhook or captainhook/captainhook-phar to use this plugin and exit the script.

What do you think about this.
I will create a new Composer plugin captainhook/installer that has similar functionality as composer-plugin without the actual dependency. The only change I would make is that you can even define the captainhook executable path in the extra config section.
Then it would be the users responsibility to make sure the Cap'n is installed, but you could even use phive to do it and then link the executable.

Just saw that the repo is already present. Works like a charm. Thank you very much for the quick fix. Amazing work, both on the issue and on the captainhook itself

Yes, I'm still experimenting with the repo a bit. but I think you can already use it.
I will remove some functionality from composer-phar because it basically does similar things, it just did not support the force option. But since this is now obsolete and handled by hook-installer I can delete it there.
I will update the docs and release version 1.0.0 of the hook-installer maybe tomorrow.
Thanks for pointing out that the Composer script solution has its flaws and a plugin is actually the best way to solve the issue.


[Website] "firstRowAsHeaders" should be "firstRowAsHeader" in the sample on the page

The sample has an error.
firstRowAsHeaders should be firstRowAsHeader
{
    "type": "AdaptiveCard",
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.5",
    "body": [
        {
            "type": "Table",
            "gridStyle": "accent",
            "firstRowAsHeaders": true,

URL: https://adaptivecards.io/explorer/Table.html

I believe the schema should be updated to use firstRowAsHeaders instead of firstRowAsHeader


[BUG] - Cannot read properties of null (reading 'digest') while running in Node.js

Describe the Bug
When following the documentation example Node.js + AI Agents, the example fails to work starting from KaibanJS version 0.4.0. The error Cannot read properties of null (reading 'digest') is thrown.
Attached is the error screenshot for reference.

KaibanJS Team Config
Here is the link to the playground showcasing the issue: Example Playground
Steps to Reproduce

Clone the repository or create a new Node.js project.
Follow the tutorial steps as outlined in the documentation.
Use KaibanJS version > 0.4.0.
Run the example project in Node.js.
Observe the error mentioned above.

Current vs. Expected Behavior

Current behavior: The application throws the error Cannot read properties of null (reading 'digest') and fails to proceed.
Expected behavior: The example should run successfully as documented.

Environment

Framework/Library: [Node.js]
KaibanJS Version: [>0.4.0]
JavaScript Runtime: [e.g., Node.js v18.20.0]

I tested this example locally using Node.js v18.20.4, and it worked fine on my setup. I'll leave this open for further testing to ensure it isn't related to specific configurations or environments. Please share any additional findings!
Here my blogTeam.js
require('dotenv').config();
const { Agent, Task, Team } = require('kaibanjs');
const { TavilySearchResults } = require('@langchain/community/tools/tavily_search');

// Define the search tool used by the Research Agent
const searchTool = new TavilySearchResults({
  maxResults: 5,
  apiKey: process.env.TAVILY_API_KEY
});

// Define the Research Agent
const researchAgent = new Agent({
  name: 'Ava',
  role: 'News Researcher',
  goal: 'Find and summarize the latest news on a given topic',
  background: 'Experienced in data analysis and information gathering',
  tools: [searchTool]
});

// Define the Writer Agent
const writerAgent = new Agent({
  name: 'Kai',
  role: 'Content Creator',
  goal: 'Create engaging blog posts based on provided information',
  background: 'Skilled in writing and content creation',
  tools: []
});

// Define Tasks
const researchTask = new Task({
  title: 'Latest news research',
  description: 'Research the latest news on the topic: {topic}',
  expectedOutput: 'A summary of the latest news and key points on the given topic',
  agent: researchAgent
});

const writingTask = new Task({
  title: 'Blog post writing',
  description: 'Write a blog post about {topic} based on the provided research',
  expectedOutput: 'An engaging blog post summarizing the latest news on the topic in Markdown format',
  agent: writerAgent
});

// Create the Team
const blogTeam = new Team({
  name: 'AI News Blogging Team',
  agents: [researchAgent, writerAgent],
  tasks: [researchTask, writingTask],
  env: { OPENAI_API_KEY: process.env.OPENAI_API_KEY }
});

module.exports = { blogTeam };


middleware/etcd: implement ReverseRecords (PTR)

at the moment reverse lookups are not implemented, a dns Server should support PTRs

I'm not convinced and there are some tricky corner cases here.
Can you point out a scenario where this would fail for you?

There are some tools in the world and they do sanity checks, e.g reverse lookup if the ip and name are consistent (if i remeber correct etcd does this)

but coredns uses etcd as it backend (in this case), so that would be a weird cyclic dependency. Granted reverse zones in general should work, see #159 for that.

thanks


Enable most admission controllers for the control plane tests

Summary:
we used to be very permissive with admission controllers for the control plane tests by using --admission-control=AlwaysAdmit option. It is deprecated and it effectively disables all admission controllers. This PR introduces a much narrower setup by disabling only two admission controllers: --disable-admission-plugins=ServiceAccount,NamespaceLifecycle which are still needed for some tests.
    The merge of the first PR reset approvals but we will merge without forcing more attention after tests pass.


Add new currencies: KRW, UGX and TZS

Sync the list with TransferWise's supported currencies.  Add KRW,
UGX and TZS.
Some currencies are no longer on the list but they still seem to
work so I'll leave them in for now.
Fixes #4

I tried to install yarnpkg to build this but I ran into some errors (not related to your code).
So this is untested. Hopefully there's no syntax error.

hmm, maybe KRW's dob should have
          type: "string", format: "date"

I think this is then shown in the user's locale.  But is there a way to export it as YYYY-MM-DD to the PDF regardless of the user's locale?


Implement the Center pivot point for rectangle

This pull request fixes https://github.com/grzi/scion/issues/193

Hey @Antonio-Bennett you're almost there, but there is one more thing :

To compute the center position os a rectangle, we need an x offset and an y offset. That's different from the square because a rectangle can have different width/height

Can you do this please ?

Ahh yes that makes sense actually, ok not at home at the moment but will do this later 🙂

Ok I've made some changes tell me if I still need to fix anything :) thanks for being patient

hey @Antonio-Bennett, perfect, thanks ! No problem, these issues are here to 'learn' and be patient !
Please consider giving a little ⭐ for the project if you don't mind, it helps it to be visible in the community. 🙏

Yup for sure ⭐! Loving the rust community so far!


Fix jespa auth provider sample

👍

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: web-devel:x: sukhovaYou have signed the CLA already but the status is still pending? Let us recheck it.


Issues on IOS when getting XML in http GET request.

Environment

Android Runtime: 5.2.1
iOS Runtime: 5.2.0
Plugin(s): --
NativeScript-Angular: 7.2.1
Angular: 7.2.0

Describe the bug
When doing a HTTP GET request for a XML resource in IOS the response has status 200 OK but request crashes during parsing even though its not supposed to do any parsing:
"status": 200,
"statusText": "Unknown Error",
ok": false,
"name": "HttpErrorResponse",
"message": "Http failure during parsing for https://www.domain.com/file.xml",
"error": {
"line": 173,
"column": 61,
"sourceURL": "file:///app/tns_modules/tns-core-modules/http/http-request/http-request.js"
ResponseType is set to 'text' for the request. In Android everything works fine.
I have debugged and found that http-request.ios.ts is the source of the crash at line 204:
return NSString.alloc().initWithDataEncoding(data, code).toString();
For some reason NSString.alloc().initWithDataEncoding returns null which crashes .toString()
Expected behavior
To get back text string with the XML file data.

@jockehaddock can you share a REST API that returns  XML file on GET (so I could test the issue locally)?
Also, are you using the HttpClient? if possible please share a sample app or Playground demonstrating the issue.

For instance if i try to consume the api described here: https://www.apfy.me/api/data.bytbil.com/ianreil/
By doing:
this.http.get('http://data.bytbil.com/sellers/satramotorcenter/xml/satramotorcenter.xml', { responseType: 'text' }).subscribe(res => console.log(res));
I set up a simple app in the playground:
https://play.nativescript.org/?template=play-ng&id=TSEuED
(Not sure if its possible to link the playgrounds directly)
I get this error:
ERROR: ERROR {
"headers": {
"normalizedNames": {},
"lazyUpdate": null,
"headers": {}
},
"status": 200,
"statusText": "Unknown Error",
"url": "http://data.bytbil.com/sellers/satramotorcenter/xml/satramotorcenter.xml",
"ok": false,
"name": "HttpErrorResponse",
"message": "Http failure during parsing for http://data.bytbil.com/sellers/satramotorcenter/xml/satramotorcenter.xml",
"error": {
"line": 180,
"column": 61,
"sourceURL": "file:///app/tns_modules/tns-core-modules/http/http-request/http-request.js"
}
}

Perhaps this is a encoding issue?

@jockehaddock setting explicitly the responseType resolved the issue for me (btw it was happening on both Android and iOS on my side).
        this.http.get('https://www.w3schools.com/xml/note.xml', { responseType: 'text' })
            .subscribe(res => {
                console.log(res);
            })

Here is a working Playground demo

Yes for that specific resource. Im sorry, i think that something went wrong when posting the Playground link.
For the one below it does not work:
this.http.get('http://data.bytbil.com/sellers/satramotorcenter/xml/satramotorcenter.xml', { responseType: 'text' }).subscribe(res => console.log(res));

@jockehaddock even with that resource I am still able to obtain the stringified XML with no hassle.
[Mi Phone]: <?xml version="1.0" encoding="iso-8859-1"?>
[Mi Phone]: <car>
[Mi Phone]: <id>13265251</id>
[Mi Phone]: <regno>JCT287</regno>
[Mi Phone]: <cars xmlns:i="http://www.w3.org/2001/XMLSchema-instance">
[Mi Phone]: <brand>Peugeot</brand>
[Mi Phone]: <model>508</model>
[Mi Phone]: <modeldescription>RXH SW 2.0 BlueHDi EAT Euro 6 180hk</modeldescription>
[Mi Phone]: <firstreg>2016-12</firstreg>
[Mi Phone]: <yearmodel>2016</yearmodel>
[Mi Phone]: <miles>10220</miles>
[Mi Phone]: <price-sek>179.900</price-sek>
[Mi Phone]: <price-extra></price-extra>
[Mi Phone]: <bodytype>Kombi</bodytype>
[Mi Phone]: <fax></fax>
[Mi Phone]: <color>Gr�</color>
[Mi Phone]: <phone>08-556 346 50</phone>
[Mi Phone]: <homepage>http://www.peugeotsatra.se</homepage>
[Mi Phone]<EMAIL_ADDRESS>[Mi Phone]: <info>Antispinn, Bromsassistans, Dimljus fram, F�rddator, GPS, Android Auto, Apple CarPlay, Bluetooth, Touch-/Peksk�rm, Yttertemperaturm�tare, Multifunktionsratt, Elstol f�rare, Elstol passagerare, S�tesv�rme (fram), F�llbara baks�ten, Keyless Nyckelfri D�rrar, Farth�llare, Regnsensor, Airbag f�rare, Airbag passagerare fram, Sidoairbags, ISOFIX-f�sten bak, Antisladd, Parkeringssensorer, D�da vinkel-varning, Centr...

Is this also working on a IOS phone?

I tested your last version on the playground and it is not working Iphone 7 IOS.

@jockehaddock oh my... I am sorry I was testing on Android.. will re-test on iOS and post the results

@jockehaddock confirming this one as a bug on iOS (steps to reproduce use the Playground on any iOS device)

Confirming the bug to be reproducible with the core HTTP module
.
Steps to reproduce:
Use this test Playground on IOS device/emulator.

Issue moved to NativeScript/NativeScript #7160 via ZenHub


Remove IntelliJ files and add example how to upload app for Appium testing in Python

I would add .gitignore with idea files

will add ignore for idea

I cannot see commit or PR for idea ignore...

only superman can see invisible things.


Update important-upgrade-notes.rst

Customers are reporting this issue but it seems to be expected.

@jasonblais @wiersgallak @esethna - not sure which team made this change, but it seems to be "expected" from a dev perspective but not from a customer perspective. Can whoever's team made the change dig into if we need to take any action beyond updating documentation here?
(ie what is the impact to customers, who does this affect, what is the recommended action -- is there a way for them to maintain the setting they had previously, or is this a breaking change so they can't get the old behaviour back. If so, is that something that will cause problems for customers, or are they okay following whatever our recommended action is).

Following up with apps team, sounds like it was introduced with post metadata in v5.8, but only surfacing now due to backwards compatibility removal.


Batch acquisition (bototrch style)

Per discussion in https://github.com/ziatdinovmax/gpax/discussions/57:
Implement the batch acquisition methods that allow one to set a value for q such that the best combination of q points that jointly optimize the acquisition function are chosen.

Very interested to see how this is implemented! Let me know if I can help.


support for milliseconds

The documentation states
// anything under a second is rounded down to zero
format(999) // '0:00'

but I need to format a duration that includes ms, e.g. 01:23:235
Is this possible?

source code is easily modifiable if you need something different. if you want to extend the library to optionally support milliseconds a PR is also welcome. good luck ✌

Added in https://github.com/ungoldman/format-duration/pull/16, shipped in 3.0.0


Single buffering?

How do I disable double buffering, ie: I want to draw directly to the frontbuffer with minimal latency?
I tried glfwWindowHint( GLFW_DOUBLEBUFFER,GL_FALSE ) but ended up with no display output at all - ie: just a black screen.
Also added glDrawBuffer( GL_FRONT )/glReadBuffer( GL_FRONT ) just in case but with no luck.
I'm using Windows 7 + nVidia GTX460 + latest glfw on github.

That is the correct hint. You don't need to change any buffers, but replace glfwSwapBuffers with glFlush or glFinish.

Thanks - changing glfwSwapBuffers to glFlush did the trick.
Only problem now is lack of vsync - I was kind of hoping swapbuffers would
(only) do vsync. There's probably nothing I can do about this I guess.
On Thu, Jan 8, 2015 at 1:44 PM, Camilla Berglund<EMAIL_ADDRESS>wrote:

That is the correct hint. You don't need to change any buffers, but
replace glfwSwapBuffers with glFlush or glFinish.
—
Reply to this email directly or view it on GitHub
https://github.com/glfw/glfw/issues/415#issuecomment-69118964.

If you're going to wait for vsync then you might as well use double buffering.

Well, it's for a oculus rift vr app, and one of the goals with vr is to reduce the time between when the headset orientation is sampled (by the cpu) and when the results end up on the screen as much as possible. Double buffering adds a 16ms lag, which is OK but not ideal.
But I suspect I need to be exploring ovr's 'direct mode' instead, which sounds like it's basically single buffering but properly synced - more info here: http://doc-ok.org/?p=1057
This actually mentions a GL extension (GLX_SGI_video_sync) that can be used to wait for sync, so I can also have a play with that.

Yes, direct mode is the correct solution.


Problem with container accepting stdin data

Hi, I have tried to have running the tool described here:
https://www.attosol.com/using-docker-for-distributing-command-line-tools/
The install works fine:
$ bin install docker://attosol/dot2png
… but the utility can't be executed as described in the aforementioned article:
$ cat file.dot | dot2png > file.png
the input device is not a TTY
If called as described, it works fine:
$ cat file.dot | docker run --rm -i attosol/dot2png > file.png
Perhaps the problem is that there is no standard for executing dockerized command line tools (?)
Thanks in advance…

@pammacdotnet thx for reporting this. I've just opened #34 that should fix this. Mind trying it out and provide feedback so we can merge?
Thx!.

Sure @marcosnils! Yes! I'll keep an eye on this. However, could you please provide minimal compilation-from-source instructions in the main README.md file (3-4 lines)? I know Go is an easy language and development environment, but I'm no expert in this specific field. I'd like to help by compiling and testing the check_stdin_tty branch, but some minimal guidelines would be much appreciated. Thanks in advance.


Bug: Poor Color Contrast for Searchbox

Describe the bug
When typing in the search field using the default GNOME shell theme (Adwaita), entered text is white instead of black. Wrong color contrast is also present in other themes.
How To Reproduce

Ensure that you are using the default (Adwaita) GNOME shell theme.
Open up the clipboard indicator from the panel.
Search for text in the searchbar
Notice how the text is the wrong color

Note: When the mouse is hovering over the text searchbar field there is no shadow, meaning that there is almost 0 contrast.
Versions

GNOME Shell 42.3.1
GCH   Version: 16
Could not load schemas from<EMAIL_ADDRESS>Failed to open file<EMAIL_ADDRESS>open() failed: No such file or directory

Note: The extension is installed from a package manager, thus the system-wide filepath. There is no "schemas" directory present.
Additional context (if a crash, provide stack trace)
N/A

Can you remove this line and restart gnome (alt F2 then r): https://github.com/SUPERCILEX/gnome-clipboard-history/blob/9a5bdefd240b9a7ff3311bde6a2c8995c26898fa/extension.js#L115
I'm wondering if that might be it.

Yup that seems to have fixed it. Thank you!

Ok, I'll check to see if it can be removed for the rest of us (that's not a style I've defined).

Nope, it does affect the styling so I'll leave it. Presumably your gnome theme needs to be updated to use dark text in light theme.


ARTEMIS-4521 deleting divert using mngmnt API doesn't remove binding from journal

Strange coincidence but I stumbled upon this issue just a few days ago as well, but from having added and then removed the divert through xml config (where the divert just remains active instead)... while I highly agree that this change is good and is how the broker should work, the current behavior is actually by design... you are supposed to set the address-setting config-delete-diverts = FORCE to enable removal of the diverts.
This is controlled by:
org.apache.activemq.artemis.core.server.impl.ActiveMQServerImpl#recoverStoredDiverts()
Perhaps this setting should just be removed as well, since with this PR it will become non-functional?

@erwindon
the issue here was that deletion using JMX was not updated in the journal, and the PR seems to fix just that.
Yes, and I'm saying the same thing happens for diverts configured through the XML file, but with the addition that the divert is created again directly after getting removed instead of after a restart.
I do see now though that the setting I mentioned above config-delete-diverts = FORCE might not help in your use-case after all but for diverts configured through XML it will.
Regardless, I'm not arguing against the change, rather that the aforementioned config should also be removed as part of it since it will become non-functional after (as far as I can tell).

It is not independent, reloading file configuration uses the same method changed with this PR, i.e org.apache.activemq.artemis.core.server.impl.ActiveMQServerImpl#destroyDivert() which now removes the divert from storage.
After that the config reload runs: org.apache.activemq.artemis.core.server.impl.ActiveMQServerImpl#recoverStoredDiverts()
which basically does:
if (divertWithStoredConfig.isDeleted() && adressSettings.config-delete-diverts == FORCE) -> remove stored divert config
else -> recreate divert
So before this change, removing a divert from xml config still left it active (with some fraction of a second in downtime), provided that config-delete-diverts = FORCE is not set. Default is OFF i.e never remove a stored divert
After this change, removing a divert from xml config removes the divert, regardless what value "config-delete-diverts" has. It also removes the config from storage.
I think the change made in this PR makes a lot of sense, both for the JMX and the file configuration use case. I'm also saying that the address setting "config-delete-diverts" will no longer have any effect anywhere on anything.


BITMAKER-2132 Get Role in Project List request

Description
Get Role in Project List request
Issue
https://tasks.bitmaker.dev/issues/2132
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] My code follows the style guidelines of this project.
[x] I have made corresponding changes to the documentation.
[x] New and existing tests pass locally with my changes.
[x] If this change is a core feature, I have added thorough tests.
[x] If this change affects or depends on the behavior of other estela repositories, I have created pull requests with the relevant changes in the affected repositories. Please, refer to our official documentation.
[x] I understand that my pull request may be closed if it becomes obvious or I did not perform all of the steps above.

@Adriana618 This issue is solved. https://tasks.bitmaker.dev/issues/2132


KPA Key body recovery is possible!

It is possible to recover the key body in a known plain text attack, as a constraint satisfaction problem.  I wrote a program that, with 1000 known plain and cipher text pairs, can recover a key body in about a minute, here in my fork.
Since you force pushed your branch and broke the merge, I guess you wouldn't be willing accept a pull request from me.  I know you're just going to close this, too, and probably throw some insults my way, but whatever.  It's your loss really, not to engage in the discussion and analysis of your algorithm.  I guessed that the key could be solved as a csp in the discussion of my second pull request to you, if you recall since you went ahead and deleted all the comments.

I don't deleted useful work and respectful technical conversations.
You didn't tell something new and I believed and I still beleive that you're trying to mislead the people here and I am explaining why:

I wrote on the wiki : "o far, we detected a "unique" vulnarability, which can be "theoretically" defended by the "Crypto Experts": Intercepting both the plaintext AND the salt(nonce). But we consider it only "theoretical" because, key's original salt value is only used to encrypt&decrypt salt values to be transmitted with each ciphertext. Salt values are random by nature. If someone claims this "theoretical possibility", he must also explain us "how" he can intercept the nonce value! But anyway, a "theoretical possibility" is still a threat and you must consider this possibility in your real life applications."

You need 32 bits CRC value
You need 1000 plaintexts AND 64 bits salt(nonce) values
Roughly; the algorithmic complexity of your "key body reveal" is 2^(32+64+11) * Complexity Of your algorithm) = 2^107 * Complexity of your algorithm!
I am never insulting to you or someone else, unless, they try to insult me after having proved that their intention is not a technical conversation but to mislead the people with "partially true" information.
So far, you are "sane". And your work helps me and the community to think about the "theoretical weaknesses" of the algorithm.  And I appreciate it.
As far as you and your comments are sane, they remain here as a useful source of information.

I even suspect you're Schenier himself :)
It would be an honor, but I am not worthy.  It's not that people follow him blindly, rather that he has gained respect for doing useful work.  Its like if you go to a university, the professors are not God, and learning from them is not practicing religion.

At least 2^81. And you tell that you worked only with 2-Jumps keys! We don't know the complexity for greater number of jumps.

It works for greater than two jumps, and keys larger than 128.  This is a general attack against any number of jumps, any length of key.  Just add enough known plain texts and even the more difficult keys can be recovered in a reasonable amount of time.  I also assume that I can get the key length and jumps using the other attacks, but if not it's no big deal.  There are only 256 (one byte) possible number of jumps (including zero and one jump - so really there are only 254).  Key length must be a power of two less or equal to 64k (two bytes), so there are only 16 possible lengths (including zero through 64 - so really there are only 10 possible lengths).  I can just run the same attack 254 * 10 = 2540 times if I don't already know the jumps and key length.  That's a small number.
You claim that the complexity of brute forcing it is something like 2^81.  That would be true if every single possible key were tried.  But if we can backtrack the search early, say after a few bytes we can rule out any key with that combination of a few bytes, then we don't have to search any more keys with that combination.  We can reduce the search space to something more reasonable, that can be seared in minutes not the age of the universe.  That's what the algorithm does, backtracking search.
I don't need to guess the whole crc of the key either.  I guess I didn't explain this detail in the attack.  I only need to guess one byte of the crc at the start of the search, and then only one bit at a time for the remainder of the search.  When the search finds a solution, the guessed crc can be compared to the actual computed crc to validate a solution.  In your algorithm you could even make the initial value of V something secret, not even related to the crc of the key, and the attack would still work.

You must have a really complex computer to make those calculations in a "few minutes"!

No, I have just a plain old Intel processor on an inexpensive laptop.  You are of course free to try it for yourself, on your more worthy machine.

And there is not any simple explanation for us, immortals how you have obtained this information. :)

I assume you mean, how do I know 1000 plain and cipher texts?    Even if I know that information, a secure algorithm should not so easily be broken by a known plain text attack.  But anyway, let's consider a situation in which I might have 1000 known plain and cipher texts.
You have heard of ransomware, right?  Imagine your encryption algorithm is used in some ransomware, to encrypt all the files on my computer.  I'm sure you would just love to do that right about now?  Suppose my most recent backup is a month old, and I would lose a bunch of work from this month if I just revert to the backup.  I have at least 1000 files in the backup that I haven't touched in a month, and the same files are encrypted on my computer.  I can use that information, the known plain text files in the backup, and the cipher texts on my computer, to recover the key.  With the key, I can recover the rest of my files that were not backed up.

 You insist to consider the algorithm as a stream cipher but in fact, with padding, it more like a block cipher.
You are just exthausting me.
I have no time to play with you.
The "real", "mathematical" and concrete algorithm complexity for your algorithm is not 2^81
It is 2^81 * (256^PlainTextLength) ---> This is mathematic, not abstract "If I already know ..." kind of phantasms(even more since, PlainText is rounded up to PacketSize)
Go play with Schneier and his band in your fictif world.
If there is anyone who reads all these and believes that what you tell is a real threat after all I wrote, I suggest them to not to use this algorithm.
I am certain that this algorithm fits "perfectly" for my specific needs. I know how to use.
I also opened it to public with MIT&GPL.
I wrote in wiki what are "theoretical threats" long before some wannabees tried to some me some extreme examples from a fictional world.
I have a software to develop.
Again, go play with Schneier. He has a lot of time for these kind of discussions, not me.
If you try to write more and getting delicate, as you did last time, I will remove your comments again. Don't do that please. You told what you wanted to tell, I told what I wanted to tell. Comments will stay here.
It's enough.

https://www.schneierfacts.com/

That's funny.  Thanks for the link!

You're wellcome :)
And in order to better understand that Hohha Encryption Ecosystem is resistant to known plaintext attacks, produce 1000 encrypted Hohha packets: You know both the plaintexts and ciphertexts. (But not the salt(nonce) values!".
Try try to modify your algorithm to run "without assuming that you already know nonce values".
You will see that they are resistant to known plaintext attacks.

I'm going to take a break from this project for a little while, but I'll be back.  Working on the code is fine, and there is still more that I plan to do, but later.
There are two things on my todo list, in the following order:

The current version of the KPA key recovery attack has a run time in the range of a few seconds to just under an hour, depending on input.  I think I can do better, using iterative deepening and a value-ordering heuristic.  My goal is to improve the current attack run time to under a second, and also be able to recover larger keys with more jumps in a reasonable amount of time, like under an hour.

After a cursory review of the packet format, I believe that it is vulnerable to a ciphertext only attack.  I have a specific design for an attack in mind.  I believe that it will be able to recover the key body, the key crc, and the original key salt, given nothing more than encrypted packet headers.  To be clear, what I am saying is that I believe the algorithm is less secure with the packet header than it is without the packet header.  If COA is possible, then the only way to protect the message will be to make it impossible to intercept the cipher text - but if it's impossible to intercept, why encrypt?

Both of these todo items will take a few hours of dedicated programming, each.  I have specific changes planned for the code, just not the motivation to continue with this project for now.  To be honest, working on the code is fine, but your rejection of "theoretical" vulnerabilities, and most of all your harassing comments, really drain my energy to continue working on this.  On that note, there are two ways we might be able to proceed.

If we can't have a conversation about it, then let's just leave it at this for now.  When I'm ready, I'll return, I'll try to code up the attack, and if it works, I'll let you know.

If you're ready to be a grown up about it, then we can talk about the theoretical attacks.  You can engage in the discussion, take the time to learn the attacks and how they work, so you can really understand exactly why your algorithm is vulnerable.  Two things I won't allow are harassment, and changing the algorithm to do something else.  If you still don't accept that the attacks are possible, and I eventually have to write code to prove the attack, I don't want you to waste my time by changing the algorithm.  If you believe the algorithm is not vulnerable to attack, then there is no need to change it, right?  Therefore, if you change the algorithm, I would consider it to be your acknowledgement that yes, the algorithm is vulnerable.

If you would like to talk about theoretical attacks, show me.  Please describe in your own words how the current "theoretical" attacks work.  It's completely acceptable to say "but it only works if...", and the doesn't work if any of the assumptions are not true.  What I want to see is your willingness to consider a situation where those assumptions are true, even if the assumptions seem unreasonable, and then explain why the attack is possible and how it works.
Just so you know, the kind of reasoning I am talking about isn't "Schneier's law", practiced by his disciples or anything like that.  Schneier's law might be relevant, but it's something completely different and maybe too far over your head for you to understand - don't worry you'll work your way up eventually.  The kind of reasoning that I am talking about is propositional logic, practiced in all fields of math and science.  If you start with some assumptions, no matter how absurd those assumptions may be, by implication it can be proved that something else must be true if the assumptions are true.  Blame Chrysippus, not Schneier.


Implement tree model dump with code generator.

Split up builders.
Use pattern matching.
Implement generator for Graphviz.

Closes #4228, Closes #4561
There are few benefits of this PR:

After introducing proper code templates in generator, the semantic of generated text files become clearer.  Currently we don't have documentation around what exactly a dumped model should look like.
Eliminates parsing the text dump again in language bindings.  One can generate very weird output by parsing the dumped text, as in above linked issues.  This PR implements a generator for Graphviz, which directly parses the tree model instead of text output.
All bindings of Graphviz can accept dot file strings, so it should be easier for scala and jvm (and any other languages) to implement plotting.  @CodingCat. (I did a simple grep in the jvm package didn't find any implementation).

But there's one downside:

Now we need to pass extra parameters into c++ for Graphviz.

@CodingCat  @RAMitchell @hcho3 Could you please take a quick look? See if the added complication is worthy for fixing above issues?

Looks like a nice reorganisation of code. We don't need any changes to the C API right, because it already accepts a string specifying format?

@KOLANICH I remember sometimes ago you mentioned allowing saving model in other formats, could this be one of the building blocks for your purpose?

Will try to write the production rules for text file later.

@hcho3 Now plotting.to_graphviz in Python package returns a graphviz.Source instead of graphviz.Digraph , which means user can't modified the returned object like adding new node to the graph manually.  I can't think of a good reason to do such thing so I'm just gonna make the change.
Can you think of some use cases for modifying returned graph?  If so I will close this PR to avoid the change.

Thanks for mentioning me.

Will add detailed user document in another PR.  But I won't go over language bindings for now.  Let me fix some piled up issues first.

@hcho3 Thanks for the review.


Support subscriptions

GraphQL Java can support subscriptions: https://www.graphql-java.com/documentation/v11/subscriptions/
We just need to come up with a good way to represent the different type in the schema.
This may be able to be done without a breaking change. Can we just add a @GraphQLSubscription?

FYI I have a WIP branch here: https://github.com/smyrick/graphql-kotlin/tree/subscribtions
I think we should update the example first to use the webflux as a demo, this can be done without supporting subscriptions but add them after will be much easier and help split up all the example changes I need to do to show how to use a publisher

Whats the status for this? :roll_eyes:

@shahabio I had started working on the update but had trouble getting the example app working with spring websockets. If you want to pick up from my branch you can pull from here: https://github.com/smyrick/graphql-kotlin/tree/subscribtions

@camuthig When I run a subscription query through the playground and I put a breakpoint at the top of SubscriptionHandler.handle() I can see the value for session.handshakeInfo.headers["Sec-WebSocket-Protocol"] is graphql-ws so that is still being sent

@smyrick I added the same breakpoint and agree that the header is there. However, when I use the dev console on Chromium to look at the response, I don't see it coming back on the response.
The headers using graphql-spring-boot-autoconfigure look something like
Connection: upgrade
Date: Tue, 23 Apr 2019 08:55:26 GMT
Sec-WebSocket-Accept: <token>
Sec-WebSocket-Extensions: permessage-deflate;client_max_window_bits=15
Sec-WebSocket-Protocol: graphql-ws
Upgrade: websocket

But with your branch I'm seeing
connection: upgrade
sec-websocket-accept: <token>
upgrade: websocket

With my latest branch I have successfully connected using the following Websocket client:
https://chrome.google.com/webstore/detail/simple-websocket-client/pfdhoblngboilpfeibdedpjgfnlcodoo
Is there an issue with GraphQL Playground and Spring Webflux Websockets? Specifically the graphql-ws protocol? https://github.com/leangen/graphql-spqr-spring-boot-starter/issues/24

putting
override fun getSubProtocols(): List<String> {
    return listOf("graphql-ws")
}

in SubscriptionHandler(private val graphQL: GraphQL) : WebSocketHandler
should fix the the protocol issue as org.springframework.web.reactive.socket.server.support.HandshakeWebSocketService is checking for supported protocols in the handler


openssh: always enable with-kerberos on darwin

Motivation for this change
On darwin, /etc/ssh/ssh_config uses GSSAPIAuthentication and GSSAPIDelegateCredential, which is enabled by kerberos optional feature. Enabling this feature by default on darwin makes all packages depending on "openssh" (e.g. git) work on darwin without editing this system file.
Things done

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

[ ] NixOS
[x] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

On our build host this isn't enabled by default (commented out) and I'm pretty sure no one has touched that file, are you sure this is active by default?

You're right, those seems to be commented on my recovery partition. I'll try to reproduce (maybe joining a MS domain or another option enables this ?) but I'm also pretty sure I never uncommented those myself.

Those options were uncommented on Yosemite and commented on Sierra (dunno about El Capitan). Upgrading does not comment those lines though, so any mac coming with anything older than Sierra (or El Capitan for that matter) will have this problem. Seems I'll comment those on my system as a workaround :-)

Even if it isn't strictly required, I'm in favor of enabling kerberos in the default openssh because it will make life a lot easier for anybody who has GSSAPI-related options in their system ssh_config or ~/.ssh/config.  Without it, kerberized ssh users have to jump through extra hoops to build custom overrides for things like git, and any other tool that uses ssh internally.  This ends up cascading down as far as causing a rebuild of rust on my systems, which takes hours and hours...
Is there a reason that nixpkgs does not default to building ssh with kerberos support?  Is it just to reduce closure sizes?

I tried on fresh installs of OS X 10.9 (Mavericks) to 10.12 (Sierra), and those lines are always commented. I wonder what happened on my system (as I never reinstalled in 3 years, perhaps some software I installed one day...) ?
I'll leave this open for some time then close it unless most say here that they would like this feature nevertheless.

What @benley said, sounds reasonable.

Merged here: https://github.com/NixOS/nixpkgs/commit/4d0bba19f4e1037bcfefa9e5f0b39da00f73281e

github got a little confused.


Fix destructuring bug in Status

When Status is rendered in the browser I get TypeError: Cannot read property 'staticContext' of undefined. It looks like this is because Status expects router to be present as a property of context. This is the case for ServerRouter but not for BrowserRouter.
By destructuring with a default value of {} for router, staticContext is correctly set to undefined when router doesn't exist.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

one last tweak to fix the type

!merge


Add and fix VB code samples

Add missing VB code samples to new controls (1 new in 6.0 & some that hadn't been done previously--I couldn't get XamlHost working in VB previously but now have)
and fix some minor formatting inconsistencies.
(was #276 - but retargeted against the correct branch and without all the extra noise.)

Docs Build status updates of commit f3b8918:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/controls/Eyedropper.md
:white_check_mark:Succeeded
View

docs/controls/ImageCropper.md
:white_check_mark:Succeeded
View

docs/controls/wpf-winforms/MapControl.md
:white_check_mark:Succeeded
View

docs/controls/wpf-winforms/WindowsXamlHost.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Jitify/CUB compilation failure

Here is a case where a Jitify/CUB compilation error occurs. I came across it when I accidentally passed a complex-valued input to a function that calls sum with fixed dtype=cupy.float64. For example:
a = cupy.ones((8,), dtype=np.complex64)
a.sum(dtype=cupy.float64)

Indeed, it does not make sense to sum a complex array using a real dtype, so this does seem like a valid failure, I just wondered if we could detect this type of case and raise a simpler error message and/or bypass CUB/cuTENSOR in this case.

---------------------------------------------------
--- JIT compile log for /tmp/tmp04h8lhhn/495be8c6a63d5f0b2674719a4398fc8b_2.cubin.cu ---
---------------------------------------------------
cupy/complex/complex.h(94): warning: __device__ annotation is ignored on a function("complex") that is explicitly defaulted on its first declaration

cupy/complex/complex.h(101): warning: __device__ annotation is ignored on a function("complex") that is explicitly defaulted on its first declaration

specializations/block_reduce_raking_commutative_only.cuh(98): warning: declaration does not declare anything

cupy/cub/cub/block/block_load.cuh(118): error: no suitable conversion function from "const type_mid_in" to "_type_reduce" exists
          detected during:
            instantiation of "void cub::LoadDirectBlocked(int, InputIteratorT, InputT (&)[ITEMS_PER_THREAD], int) [with InputT=_type_reduce, ITEMS_PER_THREAD=4, InputIteratorT=const type_mid_in *]"
(149): here
            instantiation of "void cub::LoadDirectBlocked(int, InputIteratorT, InputT (&)[ITEMS_PER_THREAD], int, DefaultT) [with InputT=_type_reduce, DefaultT=_type_reduce, ITEMS_PER_THREAD=4, InputIteratorT=const type_mid_in *]"
(712): here
            instantiation of "void cub::BlockLoad<InputT, BLOCK_DIM_X, ITEMS_PER_THREAD, ALGORITHM, BLOCK_DIM_Y, BLOCK_DIM_Z, PTX_ARCH>::LoadInternal<cub::BLOCK_LOAD_DIRECT, DUMMY>::Load(InputIteratorT, InputT (&)[ITEMS_PER_THREAD], int, DefaultT) [with InputT=_type_reduce, BLOCK_DIM_X=32, ITEMS_PER_THREAD=4, ALGORITHM=cub::BLOCK_LOAD_DIRECT, BLOCK_DIM_Y=1, BLOCK_DIM_Z=1, PTX_ARCH=610, DUMMY=0, InputIteratorT=const type_mid_in *, DefaultT=_type_reduce]"
(1230): here
            instantiation of "void cub::BlockLoad<InputT, BLOCK_DIM_X, ITEMS_PER_THREAD, ALGORITHM, BLOCK_DIM_Y, BLOCK_DIM_Z, PTX_ARCH>::Load(InputIteratorT, InputT (&)[ITEMS_PER_THREAD], int, DefaultT) [with InputT=_type_reduce, BLOCK_DIM_X=32, ITEMS_PER_THREAD=4, ALGORITHM=cub::BLOCK_LOAD_DIRECT, BLOCK_DIM_Y=1, BLOCK_DIM_Z=1, PTX_ARCH=610, InputIteratorT=const type_mid_in *, DefaultT=_type_reduce]"
/tmp/tmp04h8lhhn/495be8c6a63d5f0b2674719a4398fc8b_2.cubin.cu(104): here

1 error detected in the compilation of "/tmp/tmp04h8lhhn/495be8c6a63d5f0b2674719a4398fc8b_2.cubin.cu".

---------------------------------------------------
Traceback (most recent call last):

  File "<ipython-input-35-cc7d38537641>", line 1, in <module>
    a.sum(dtype=np.float64)

  File "cupy/core/core.pyx", line 860, in cupy.core.core.ndarray.sum

  File "cupy/core/core.pyx", line 868, in cupy.core.core.ndarray.sum

  File "cupy/core/_routines_math.pyx", line 120, in cupy.core._routines_math._ndarray_sum

  File "cupy/core/_reduction.pyx", line 560, in cupy.core._reduction._SimpleReductionKernel.__call__

  File "cupy/core/_reduction.pyx", line 346, in cupy.core._reduction._AbstractReductionKernel._call

  File "cupy/core/_cub_reduction.pyx", line 684, in cupy.core._cub_reduction._try_to_call_cub_reduction

  File "cupy/core/_cub_reduction.pyx", line 520, in cupy.core._cub_reduction._launch_cub

  File "cupy/core/_cub_reduction.pyx", line 455, in cupy.core._cub_reduction._cub_two_pass_launch

  File "cupy/_util.pyx", line 53, in cupy._util.memoize.decorator.ret

  File "cupy/core/_cub_reduction.pyx", line 227, in cupy.core._cub_reduction._SimpleCubReductionKernel_get_cached_function

  File "cupy/core/_cub_reduction.pyx", line 212, in cupy.core._cub_reduction._create_cub_reduction_function

  File "cupy/core/core.pyx", line 1883, in cupy.core.core.compile_with_cache

  File "/home/lee8rx/my_git/pyir/cupy/cupy/cuda/compiler.py", line 396, in compile_with_cache
    cache_in_memory, jitify)

  File "/home/lee8rx/my_git/pyir/cupy/cupy/cuda/compiler.py", line 474, in _compile_with_cache_cuda
    log_stream, cache_in_memory, jitify)

  File "/home/lee8rx/my_git/pyir/cupy/cupy/cuda/compiler.py", line 230, in compile_using_nvrtc
    name_expressions, log_stream, jitify)

  File "/home/lee8rx/my_git/pyir/cupy/cupy/cuda/compiler.py", line 206, in _compile
    source, options, cu_path)

  File "/home/lee8rx/my_git/pyir/cupy/cupy/cuda/compiler.py", line 188, in _jitify_prep
    raise JitifyException(str(cex))

JitifyException: Runtime compilation failed

For reference NumPy only raises a Warning and not an error:
np.ones((8,), dtype=np.complex128).sum(dtype=np.float64)

/miniconda3/envs/sp_dev/lib/python3.7/site-packages/numpy/core/_methods.py:47: ComplexWarning: Casting complex values to real discards the imaginary part
  return umr_sum(a, axis, dtype, out, keepdims, initial, where)

Conditions (you can just paste the output of python -c 'import cupy; cupy.show_config()')

OS                           : Linux-4.15.0-124-generic-x86_64-with-debian-buster-sid
CuPy Version                 : 9.0.0a3
NumPy Version                : 1.19.2
SciPy Version                : 1.6.0.dev0+ba8a861
Cython Build Version         : 0.29.21
Cython Runtime Version       : 0.29.21
CUDA Root                    : /usr/local/cuda
CUDA Build Version           : 11010
CUDA Driver Version          : 11010
CUDA Runtime Version         : 11010
cuBLAS Version               : 11300
cuFFT Version                : 10300
cuRAND Version               : 10202
cuSOLVER Version             : (11, 0, 1)
cuSPARSE Version             : 11300
NVRTC Version                : (11, 1)
Thrust Version               : 100910
CUB Build Version            : 100910
Jitify Build Version         : <unknown>
cuDNN Build Version          : 7605
cuDNN Version                : 7605
NCCL Build Version           : 2803
NCCL Runtime Version         : 2803
cuTENSOR Version             : 10201
Device 0 Name                : GeForce GTX 1080 Ti
Device 0 Compute Capability  : 61

This also happens when CUB/cuTENSOR is not in use, so it's a generic CuPy error. I guess we need to a way to detect and convert the dtypes internally, raise ComplexWarning, etc. How does NumPy achieve this?
>>> import cupy as cp
>>> cp.ones((8,), dtype=cp.complex128).sum(dtype=cp.float64)
Traceback (most recent call last):
  File "/home/leofang/dev/cupy_cuda10.2/cupy/cuda/compiler.py", line 586, in compile
    nvrtc.compileProgram(self.ptr, options)
  File "cupy_backends/cuda/libs/nvrtc.pyx", line 108, in cupy_backends.cuda.libs.nvrtc.compileProgram
    cpdef compileProgram(intptr_t prog, options):
  File "cupy_backends/cuda/libs/nvrtc.pyx", line 120, in cupy_backends.cuda.libs.nvrtc.compileProgram
    check_status(status)
  File "cupy_backends/cuda/libs/nvrtc.pyx", line 58, in cupy_backends.cuda.libs.nvrtc.check_status
    raise NVRTCError(status)
cupy_backends.cuda.libs.nvrtc.NVRTCError: NVRTC_ERROR_COMPILATION (6)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "cupy/core/_reduction.pyx", line 601, in cupy.core._reduction._SimpleReductionKernel._get_function
    return _SimpleReductionKernel_get_cached_function(
  File "cupy/_util.pyx", line 53, in cupy._util.memoize.decorator.ret
    result = f(*args, **kwargs)
  File "cupy/core/_reduction.pyx", line 614, in cupy.core._reduction._SimpleReductionKernel_get_cached_function
    return _create_reduction_function(
  File "cupy/core/_reduction.pyx", line 122, in cupy.core._reduction._create_reduction_function
    module = compile_with_cache(module_code, options)
  File "cupy/core/core.pyx", line 1883, in cupy.core.core.compile_with_cache
    return cuda.compile_with_cache(
  File "/home/leofang/dev/cupy_cuda10.2/cupy/cuda/compiler.py", line 396, in compile_with_cache
    cache_in_memory, jitify)
  File "/home/leofang/dev/cupy_cuda10.2/cupy/cuda/compiler.py", line 474, in _compile_with_cache_cuda
    log_stream, cache_in_memory, jitify)
  File "/home/leofang/dev/cupy_cuda10.2/cupy/cuda/compiler.py", line 230, in compile_using_nvrtc
    name_expressions, log_stream, jitify)
  File "/home/leofang/dev/cupy_cuda10.2/cupy/cuda/compiler.py", line 213, in _compile
    ptx, mapping = prog.compile(options, log_stream)
  File "/home/leofang/dev/cupy_cuda10.2/cupy/cuda/compiler.py", line 598, in compile
    'nvrtc' if not runtime.is_hip else 'hiprtc')
cupy.cuda.compiler.CompileException: /tmp/tmpc91x0ufr/ddfd12c4936f2b662957e96d4589ee5e_2.cubin.cu(35): error: no suitable conversion function from "const type_in0_raw" to "_type_reduce" exists

1 error detected in the compilation of "/tmp/tmpc91x0ufr/ddfd12c4936f2b662957e96d4589ee5e_2.cubin.cu".

Exception ignored in: 'cupy.core._reduction._AbstractReductionKernel._launch'
Traceback (most recent call last):
  File "cupy/core/_reduction.pyx", line 601, in cupy.core._reduction._SimpleReductionKernel._get_function
    return _SimpleReductionKernel_get_cached_function(
  File "cupy/_util.pyx", line 53, in cupy._util.memoize.decorator.ret
    result = f(*args, **kwargs)
  File "cupy/core/_reduction.pyx", line 614, in cupy.core._reduction._SimpleReductionKernel_get_cached_function
    return _create_reduction_function(
  File "cupy/core/_reduction.pyx", line 122, in cupy.core._reduction._create_reduction_function
    module = compile_with_cache(module_code, options)
  File "cupy/core/core.pyx", line 1883, in cupy.core.core.compile_with_cache
    return cuda.compile_with_cache(
  File "/home/leofang/dev/cupy_cuda10.2/cupy/cuda/compiler.py", line 396, in compile_with_cache
    cache_in_memory, jitify)
  File "/home/leofang/dev/cupy_cuda10.2/cupy/cuda/compiler.py", line 474, in _compile_with_cache_cuda
    log_stream, cache_in_memory, jitify)
  File "/home/leofang/dev/cupy_cuda10.2/cupy/cuda/compiler.py", line 230, in compile_using_nvrtc
    name_expressions, log_stream, jitify)
  File "/home/leofang/dev/cupy_cuda10.2/cupy/cuda/compiler.py", line 213, in _compile
    ptx, mapping = prog.compile(options, log_stream)
  File "/home/leofang/dev/cupy_cuda10.2/cupy/cuda/compiler.py", line 598, in compile
    'nvrtc' if not runtime.is_hip else 'hiprtc')
cupy.cuda.compiler.CompileException: /tmp/tmpc91x0ufr/ddfd12c4936f2b662957e96d4589ee5e_2.cubin.cu(35): error: no suitable conversion function from "const type_in0_raw" to "_type_reduce" exists

1 error detected in the compilation of "/tmp/tmpc91x0ufr/ddfd12c4936f2b662957e96d4589ee5e_2.cubin.cu".

array(0.)


Reformat to black

re applying this pr to master

Coverage decreased (-0.01%) to 99.778% when pulling 6a36a8eb3014a3609c771302c80a4551c308836d on reformat_to_black into 1e4b81933ca6786d54617f08698b93487d199c16 on master.


Use github pages to create a live-version of the website

I made this small PR to be able to use github pages. It'll rename your .html file to index.html, create a separate branch for it called gh-pages. In order to enable gh pages, just go to Settings -> Pages -> Branch -> Select gh-pages. When building process is finished, you'll get a URL like https://JingShing.github.io/ATLYSS-Save-Editor/

How it looks like

Hi nuckle,
Thank you so much for taking the time to help me write this GitHub Actions workflow! I really appreciate your guidance and the effort you put into explaining everything clearly. Your contribution has been invaluable in helping me deploy my static website smoothly to GitHub Pages.
Thanks again,
JingShing


received an error while trying to get the state deltas for blocks 140 through 154 from name:"vp2"

Description
commit cdfbdcd66de75cb832114d8815999189c7f6da9b
Merge: dbf46b0 e3adcbe
Author: Sheehan Anderson<EMAIL_ADDRESS>Date:   Fri Jun 24 10:22:07 2016 -0400
Merge pull request #1980 from jyellick/fix-concurrent-map-write-v05

Eliminate possible broadcaster concurrent map read/write

4 peer network, security and privacy enabled, batch size = 1000, thread number = 100
Run stress test for 60 min and see the messages below 5 times in the log
14:41:12.166 [consensus/statetransfer] func1 -> WARN 35c^[[0m name:"vp2"  failed to get blocks from 154 to 139 from name:"vp2" : Remote ledger not found for receiver vp2
14:41:12.167 [consensus/statetransfer] tryOverPeers -> WARN 35d name:"vp2"  in tryOverPeers loop trying name:"vp2"  : Remote ledger not found for receiver vp2
14:41:12.176 [consensus/obcpbft] processNewView2 -> INFO 35e Replica 2 accepting new-view to view 7
14:41:12.365 [consensus/obcpbft] recvViewChange -> INFO 35f Replica 2 received view-change from replica 0, v:7, h:140, |C|:0, |P|:20, |Q|:20
14:41:14.066 [consensus/statetransfer] tryOverPeers -> WARN 360 name:"vp2"  in tryOverPeers loop trying name:"vp2"  : name:"vp2"  received an error while trying to get the state deltas for blocks 140 through 154 from name:"vp2"
Describe How to Reproduce

Use local_fabric.sh to setup a 4-peer network with security and privacy enabled, batchsize=1000
bring up SaopUI:
set thread = 100
send invokes a->b, invoke b->a, then repeat
run for 60 min
check log

@jyellick cute, it tries to sync from itself? :)

@corecode As you can see the state transfer code should never pick itself as a source to sync from (unless you can spot a bug in this logic), but there is nothing to prevent someone from explicitly instructing state transfer to sync from itself, maybe we should try to filter that out too.
PBFT will pass the peers to sync from based on the peers which have sent the checkpoint message.  If we've sent a checkpoint message then it would usually imply that our state is already current at that point, so I'm not sure why this would be occurring from the other end either.

yea, mysterious


Exclusion Report | Edit a draft |3|

Story:
As a BCeID user, I want to be able to edit a saved draft of an Exclusion Report so that I can continue filling out the report that I had previously saved.
Scenario:
GIVEN that I am a BCeID user
AND I have the Compliance Reporting role
AND I am on the Compliance Reporting page
AND I have saved a draft of an Exclusion Report
WHEN I click on the draft entry in the Compliance Reporting table
THEN I am taken to the Introduction page of the Exclusion Report
AND I am able to edit the contents of the report
https://trello.com/c/D6knfdFA/1564-exclusion-report-edit-a-draft-3

 Exclusion Report | Edit a draft |3|

resolved in sprint ursa major


Fix Error: Cannot find module 'uuid/v4'

Issue #:
Fix Error: Cannot find module 'uuid/v4'
OS: Windows 10
npm: 7.21.1
node: 12.13.1
Description of changes:
change syntax.

I create a new Pull request...
Il giorno sab 11 dic 2021 alle ore 01:36 Sichao Xue (sichax) <
@.***> ha scritto:

@.**** requested changes on this pull request.
Thanks for your contribution! Please change the target branch from
aws-sample:main to aws-sample:meeting-dev
In apps/meeting/server.js
https://github.com/aws-samples/amazon-chime-sdk/pull/58#discussion_r766888138
:

@@ -5,7 +5,7 @@
const compression = require('compression');
const fs = require('fs');
const url = require('url');
-const uuid = require('uuid/v4');
+const uuid = require('uuid');

Could you change it to below style to make the code consistent?
const { v4: uuidv4 } = require('uuid');
Cause in other repo we handle the uuid in this way.
In apps/meeting/server.js
https://github.com/aws-samples/amazon-chime-sdk/pull/58#discussion_r766888869
:

@@ -18,7 +18,7 @@ const chime = new AWS.Chime({ region: 'us-east-1' });
const alternateEndpoint = process.env.ENDPOINT;
if (alternateEndpoint) {
console.log('Using endpoint: ' + alternateEndpoint);

chime.createMeeting({ ClientRequestToken: uuid() }, () => {});

chime.createMeeting({ ClientRequestToken: uuid.v4() }, () => {});

So this will be uuidv4(), and please change the rest as well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aws-samples/amazon-chime-sdk/pull/58#pullrequestreview-829195216,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACDWHK5SBAVKN4JQYSQD4BLUQKMJTANCNFSM5D6AY4DQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Fix for the reloading server on windows.

use os.set_handle_inheritable on sockets as os.set_inheritable results in the following stack trace:
Traceback (most recent call last):
File "run.py", line 4, in
app.run()
File "C:\Users\Peter Ellis\AppData\Local\Programs\Python\Python35-32\lib\site-packages\flask\app.py", line 772, in run
run_simple(host, port, self, **options)
File "C:\Users\Peter Ellis\AppData\Local\Programs\Python\Python35-32\lib\site-packages\werkzeug\serving.py", line 666, in run_simple
os.set_inheritable(s.fileno(), True)
OSError: [Errno 9] Bad file descriptor

This is already fixed on dev. Will be out today.


As a User, I want to have an option to capitalize first letter of a name, where appropriate.

It is not a good idea to capitalize first letter all the time, as it would mask many cases of wrong input, but in some cases it is useful.

Related issues:
https://github.com/gnames/gnames/issues/67
https://github.com/gnames/gnverifier/issues/45


OCIO Output Transforms?

Hi,
Thanks for mrv2, I stumble on this project yesterday while looking for alternative sequence players and I really like it. I kinda share the points from #132 but not a big deal.
One thing I don't understand is how (if possible) to specify an OCIO output transform when rendering to a video file?
I'm not entirely sure of the current  behaviour
Thanks

Currently, you cannot specify an OCIO output transform (in Nuke's way of doing it).
You can set your custom config.ocio file in the Preferences->OCIO (F10).  Then that will allow you to select the Input Color Space transform as well as the Display/View transform in the main pull downs of the UI.  When saving a movie file, both transforms are applied to give you a WYIWYG.

both transforms are applied to give you a WYIWYG.

I don't think this is true unfortunately, I naively though so at first but posted the issue after comparing the outputs.
I will try to get a proper repro with a ground truth

If you are comparing an EXR to a Movie file with the color pipeline, you are likely going to not get a perfect match.  When going to a movie file, you loose precision due to:

Float (or Half) to 8-bits.
RGBA to YUV420 (or 444 at best).

However, if you stay in float or half (by reading and saving exrs after going through the pipeline), you should not see visible color differences.
Anyway, if you have an example where this does not hold true, post it here and I'll take a look at it.

@melMass Were you able to find the problem or get a sample test case?  I am going through the list of bugs before I hit v1.0.0.


handleSubmit callback does not show error on console of Chrome dev tool when error occurred.

I have a simple login form with 2 components is LoginForm and Login.
My LoginForm component:
import React from 'react';
import { Field, reduxForm } from 'redux-form';
import Tools from 'app/utils/helpers/Tools';

import RaisedButton from 'material-ui/RaisedButton';
import FlatButton from 'material-ui/FlatButton';

import FormInput from 'app/utils/components/FormInput';
import FormDropdown from 'app/utils/components/FormDropdown';

class LoginForm extends React.Component {
    constructor(props) {
        super(props);
    }

    render() {
        const { handleSubmit, checkSubmit, submitting, error, reset } = this.props;
        return (
            <form
                onSubmit={handleSubmit(checkSubmit)}
                >

                <Field name="email" type="email" component={FormInput} label={this.props.labels.email}/>
                <Field name="password" type="password" component={FormInput} label={this.props.labels.password}/>

                {error &&
{error}
}

                    {this.props.children}
                    <RaisedButton label={this.props.submitTitle} primary={true} disabled={submitting} type="submit"/>
                    <RaisedButton label="Reset" onClick={reset}/>

            </form>
        );
    }
}

const rules = {
    email: {
        type: 'email',
        required: true
    },password: {
        type: 'str',
        required: true
    }
}

const validate = values => {
    return Tools.validateInput(values, rules);
}

LoginForm.propTypes = {
};

LoginForm.defaultProps = {
};

// Decorate the form component
LoginForm = reduxForm({
    form: 'LoginForm', // a unique name for this form
    validate
})(LoginForm);

export default LoginForm;

And Form component:
...
loginHandle(eventData, dispatch){
    abc=def; // Code break and stop here but I can't see any error message on console of Chrome dev tool.
    let params = {
        ...eventData,
        password: md5(eventData.password)
    };
    return Tools.apiPost(apiUrls, 'login', params).then((result) => {
        if(result.success){
            dispatch(reset('LoginForm'));
        }else{
            throw new SubmissionError(Tools.errorMessageProcessing(result.message));
        }
    });
}

render() {
    ...
    <LoginForm
        checkSubmit={this.loginHandle}
        labels={labels.login}
        submitTitle="Login">
        <FlatButton
            label="Forgot password"
            primary={true}
            onTouchTap={() => this.toggleModal(true)}
            />
    </LoginForm>
    ...
}

As I want to cause an error by using expression abc=def; then code break and stop here but I can't see any error on Console of Chrome dev tool. It hard to debug inside that callback function if something goes wrong.

Having same problem and not sure why my submission is failing

This is a problem for me too. Here is a simple code to reproduce:
<button onClick={handleSubmit(data => {
      let bug = null;
      bug.foo();
    })}>Submit</button>

Issue is here:
https://github.com/erikras/redux-form/blob/master/src/handleSubmit.js#L23
Any Error that is not a SubmissionError is eaten.
I think rethrowing any error that is not a SubmissionError might resolve it without side effects...
As a hacky workaround just to be able to read the error, you can just surround the submit function that you pass to handleSubmit in a try-catch and write caught exception to console.

Because this issue saw almost no activity for more than a year, I'm closing it.


async and pyproj issues with Python 3.7 on Windows10

Context
Reinstalling Mapproxy on a fresh Windows10 64 bit server, Apache2 and mod_wsgi (coming from an old Windows Server 2008 32 bits install....).
Encountered 2 issues, one I cannot resolve:

issues that apparently async is a keyword in Python 3.7, so renaming mapproxy.util.asyc.py to async_.py (and all its references like as in https://gitlab.damiens.info/nicolas/mapproxy/commit/ee5c9589274610e43e34f31ac067e949531d9d8e) fixed that error

installing pyproj and using os.environ['MAPPROXY_USE_PYPROJ'] = 'True' in config.py to make MapProxy use pyproj fails, because try_pyproj_import function seems to fail

 def try_pyproj_import():
    try:
        from pyproj import Proj, transform, set_datapath
    except ImportError:
        return False
    log_system.info('using pyproj for coordinate transformation')
    return Proj, transform, set_datapath

installing pyproj and running
 from pyproj import Proj, transform, set_datapath
fails because set_datapath is not available:
ImportError: cannot import name 'set_datapath' from 'pyproj' (C:\Programs\Python37\lib\site-packages\pyproj\__init__.py)
downgrading to pyproj 1.9.6 fixed this:
pip install pyproj==1.9.6
Is this something that can be fixed easily (other then with using the older pyproj)?

We have also faced this issue. We are trying to use Mapproxy on NixOS.
https://github.com/NixOS/nixpkgs/pull/68628
But It breaks with same errors, when latest version of pyproj is used.

MapProxy now supports newer versions of Proj via PyProj. You can test it with:
pip install https://github.com/mapproxy/mapproxy/tarball/master

The async.py issue is also fixed in master.

Thanks for the fix @olt
Any chance for a 1.13 release soon to support latest PROJ versions?


Bug fix/alex skill cards

Issue #, if available:
Description of changes:
This fix addresses several issues with alexa skills. First it removes ssml tags from plainMessage which impacts presentation on alexa devices with screens such as an echo show. Second, it removes the extra "ok." text from plainMessage which might exist at the beginning of plainMessage due to use of chained elicitResponse bot questions. This also shows up on the echo show like devices. Finally it removes the presentation of "undefined" when subTitle does not exist in the question and the alexa card is being created.

@tabdunabi When I look at lambda/fulfillment/lib/middleware/alexa.js on the "develop" branch, this fix is definitely present. I usually only pr to the develop branch so I'm not sure why this became a pr for "main". "main" presently does not have the fix.

@bobpskier Great!, if your code in this PR made it to "develop" (maybe it was already in one of your other PRs that were merged into develop).


Error handling for mapper

So far there is no error return to the engine while doing the mapping.  code is here:

We should provide error handling while mapper goes wrong and make it to error flow.

The issue open by @lixingwang


chore: update ash dependencies

ash has been locked at 3.4.37 as in later versions eager_validate_with is broken.
linked issue: https://github.com/ash-project/ash/issues/1647

Pull Request Test Coverage Report for Build 590bf4916ed60ba0c448101e7a448ec164090fb6-PR-756
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.565%

Totals

Change from base Build e41244aa9f6e2526e94cb97ac80866153b0bf7ff:
0.0%

Covered Lines:
1227

Relevant Lines:
1523

💛  - Coveralls


Fix checkpoint name

Checkpoint name in generate_lora.py 'adapter_checkpoint' seems better to be changed to 'lora_checkpoint'.

I think this is a good change. Let's do it?


Mj update amp epic copy for non us regions

What does this PR do?
Updates AMP epic copy for non-US regions, as per this Trello card.

 US Election: copy update on AMP