Provide function signatures

When solving Rust exercises the first thing I do is figuring out the name and the right type signature and I'm wondering about the usefulness of this. This process is busywork, I doubt anybody is learning from it and in some cases it requires reverse engineering  the thought process of the exercise creator.
I think the the signatures could be added to the readme or maybe a skeleton src/lib.rs could be provided with the exercises.
Look at the Parallel letter frequency exercise for example. The solution has the signature of frequency(&[&str], u32) -> HashMap<char, u32> but nowhere in the readme or in the test cases are mentioned what is the second parameter supposed to be. It can be assumed from the name of one test case that it is the number of workers.
I was confused by it at first. This confusing could have been removed if the readme had a line similar to this:

Your task is to implement the following function: fn frequency(text: &[&str], workers: u32) -> HashMap<char, u32>

What is your opinion on this?

I'm entirely on the fence. And I think whichever way we go there's going to be a group of students that don't like it.
For me, I like not having the signature. I'm entirely new to Rust and static typing and I've found that I like puzzling out what I have to implement by looking at the tests. But that might just be me!
As for parallel letter frequency, I think the API could be improved. I find the current implementation to be hard to parse. Again, that might just be me.

I'm also on the fence. In the Go track we've provided stub files early on, and then we phase them out after the first few. For the most part reverse engineering function signatures doesn't seem to be difficult in Go, though, and I don't have experience with Rust, so I can't speak to how much busywork that adds here.

Take a look at the output of cargo test if I run an exercise with empty source file and another one with an empty function with a wrong type signature:
https://gist.github.com/ijanos/59644c68af57e38d5fd81ba6b1baee27
Now compare it to the output once you get the type signature right:
https://gist.github.com/ijanos/53cb04393b6c4e80e5ecbcd908bae0c2
Interpreting the errors is definitely not hard but it can be intimidating at first and mildly annoying if someone doesn't get the types right at the first try.
I think extending the README is a good middle ground to this issue. People still have the option to figure out the function signature while others can just copy paste it and start solving the real exercise.

We can extend the README by adding a HINTS.md file in the exercise directory.

Let's try the HINTS.md approach in an early exercise and see how it looks.

I'm fine with this.
Would the HINTS file contain anything else?

It's going to show up as part of the README, so maybe a sentence explaining that this is the expected function signature?

Yeah, that would be nice.

I agree with @IanWhitney in that it's helpful/fun to puzzle out the function signatures (and struct/impl too).
I think it's very useful, but I think it should be made explicit. I think exercises to teach about writing the signatures may be a good approach.
Until those exercises get created, though … I think having hints in certain exercises would be a good approach. Possibly in the test file itself. There were very few where I was 😖

I spent lots of time this weekend to work on this track. I found it really annoying to figure out which functions the test cases do expect. It also conflicts with the way how it is intended to fix the tests. You cannot ignore all the tests marked with ignore, because it results in build errors. So you need to inspect all test cases anyway.
Even then, it is easy to overlook some function and then you see only the huge errors created by marco expansion.
Therefore, I am in strong favor of providing function signatures. I would even put them into a stub src/lib.rs file. (Also I found it annoying to always run mkdir src; touch src/lib.rs. Why is the file not just created?)
I found the Forth exercise to be a pleasant surprise to already contain a lib.rs file. Slightly less convenient, but still better than the rest is the Robot Name exercise which contains these information in the test file.
Whatever this discussion will result in, I think the  Forth and Robot Name exercises should follow the same pattern than the rest. It does not really make sense for those middle and late exercises to introduce function signatures.
Would you consider merging stub files with function signatures, if I create a pull request for them?

I think stub files with signatures are great. Thanks! I'm also bugged by the compiler failing due to ignored tests.

I'm in favor of a at least an empty src/lib.rs making the directory and the file is busywork.

Now that we're starting to add src/lib.rs stub files, I'm going to close this issue. Thanks for the suggestion, @ijanos !


Move local-project dir inside sandcastle subfolder

We need both upstream and dowstream repos in sandcastle shared folder.
For this reason I need to move the upstream repo from the root to a subfolder.
Fixes #1956
Merge after packit/packit#2054

RELEASE NOTES BEGIN
Packit now sets PACKIT_UPSTREAM_REPO and PACKIT_DOWNSTREAM_REPO environment variables for release syncing actions. The variables represent paths where the respective git repositories are cloned
RELEASE NOTES END

I'd change release notes to something like this:
Packit now sets PACKIT_UPSTREAM_REPO and PACKIT_DOWNSTREAM_REPO environment variables for release syncing actions. The variables represent paths where the respective git repositories are cloned.

LGTM, but I haven't tested it.

Me neither, because I can not test locally actions in sandcastle, since I have not openshift.
I can just see that the working dir for dg is under the shared sandcastle dir and I hope this will be enough.


Ll insertions

- [x] Can successfully add a node to the end of the linked list
- [x] Can successfully add multiple nodes to the end of a linked list
- [x] Can successfully insert a node before a node located i the middle of a linked list
- [x] Can successfully insert a node before the first node of a linked list
- [x] Can successfully insert after a node in the middle of the linked list
- [x] Can successfully insert a node after the last node of the linked list
-


docs: combine repeated code tab shortcodes into single shortcode

What I did

Combined repeated shortcodes on the code tab to a single shortcode renderCodeDocs
Adds ability to hide and show description under component tag header hideDescription
Updates rh-navigation-secondary subcomponent  JSDoc descriptions (still need tweaking)

Closes #1052
Closes #1073
Testing Instructions

View deploy preview for each component look for regressions in information not shown or incorrect information shown.

Notes to Reviewers

will this make it easier to migrate when we remove the templates from pfe tools, or harder?

will this make it easier to migrate when we remove the templates from pfe tools, or harder?

Thats a good question, my assumption is templates sort of just move downstream but we create an API in pfe-tools for the shared data access.
I tried to keep this PR simple by basically just combining those calls to a single call.  Not sure it will make it any more difficult to migrate later as we'd still have to make a call to the data and apply that to a template locally, maybe this way the short code can remain the same, but internally just calls locals templates and applies the data from the pfe-tools API.
So instead of calling this.ctx._.renderEvents and getting back HTML we'd call this.ctx._.eventsData and just get back a JSON object.
Did you have something different in mind?

Sounds good just wanted to try to keep our eyes in our heads


potentailly add a new field to the server config

hey, recently, while working on my custom provider for dragonfly, I realized that when players get spawned in, they are always going to get spawned in using the server worlds (overworld, nether, end).
This is great but this means that we'd have to merge our dimensions into a single world. Not everybody wants to do that.
As you can see here: https://github.com/bedrock-gophers/provider/blob/master/provider/settings.go#L16-L19
I've made it so you can give a custom world "resolver", which you can use if you are using my custom player provider
here's how we use it on moyai:
func wrld(dim world.Dimension) *world.World {
	switch dim {
	case world.Overworld:
		return Overworld()
	case world.Nether:
		return Nether()
	case world.End:
		fmt.Println("end")
		return End()
	}
	return nil
}

func NewServer(config server.Config) *server.Server {
	providerSettings := provider.DefaultSettings()
	providerSettings.UseServerWorld = false
	providerSettings.World = wrld
	providerSettings.FlushRate = time.Minute * 10
	providerSettings.SaveEffects = false

	playerProvider = provider.NewProvider(&config, providerSettings)
	srv = config.New()
	return srv
}

I was wondering if the main developers would be OK with adding such a field to the server config
(only the world function I mean)
if so, please suggest field names so I can start working on a PR

This can already be done by providing a world using this field in the player's data.

Player's data is not readily accessible to the user so there does need to be some other solution


Consistent Font Changes (Remove Gotham)

Description
JIRA: Dock-2022
Ticket states 2 bugs:

"Part 2" appears twice
Part 1 has different style compared to Part 2 (3,4,5.. etc.)

Without Changes:
Repeated Part 2:

Mismatched Styles:

Part 1 is font Gotham, Part 2 is Roboto

With Changes:

Part numbers are corrected
Style matches Part 1 (changed font from Roboto to Gotham, continued existing "font" class)

Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that your code compiles by running npm run build
[x] If this is the first time you're submitting a PR or even if you just need a refresher, consider reviewing our style guide
[x] Do not bypass Angular sanitization (bypassSecurityTrustHtml, etc.), or justify why you need to do so
[x] If displaying markdown, use the markdown-wrapper component, which does extra sanitization
[x] Do not use cookies, although this may change in the future
[x] Run npm audit and ensure you are not introducing new vulnerabilities
[x] Do due diligence on new 3rd party libraries, checking for CVEs
[x] Don't allow user-uploaded images to be served from the Dockstore domain

Update:

Removed all mentions of Gotham, I had a look around the website and couldn't find any other visual instances of it on any other pages so it doesn't look like there will be a lot affected

The Quick Start page is now all in Roboto and looks like this:


MembersUpdate Requires Strict Checking (Old 301)

In gitlab by @A-Lawrence on Aug 31, 2015, 01:07
Certain outputs don't appear because we're not strict checking.

In gitlab by @A-Lawrence on Aug 31, 2015, 01:09
mentioned in issue #229

In gitlab by @A-Lawrence on Aug 31, 2015, 01:12
Fixes fluctuating member statuses. Resolves #488453 (284) and #301
ÔåÆ <<cset 738a99ff1824>>
NFarrington - 2015-02-28T22:04:50.560844+00:00

In gitlab by @A-Lawrence on Aug 31, 2015, 01:16
mentioned in issue #253

In gitlab by @A-Lawrence on Aug 31, 2015, 01:16
mentioned in issue #255


The method 'VoidCallbackIntent' isn't defined for the class '_TreeViewItem'.🐛

Because of this error, the project does not work. Needs urgent regulation.
error: The method 'VoidCallbackIntent' isn't defined for the class '_TreeViewItem'.

Can you post the logs of flutter doctor -v here?

Flutter 3.0.5 is installed, do I need to use the latest version for this error to go away?

The situation got better after I updated the Flutter version to the latest.

@umutocak  What is your version now. I am seeing this issue in latest flutter version as well on linux platform.

@sunnykarira-ms  my flutter version 3.3.3


Merge 3 -> 6

➡ Forward port
Port ign-gazebo3 to ign-gazebo6
Branch comparison: https://github.com/ignitionrobotics/ign-gazebo/compare/ign-gazebo6...ign-gazebo3
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

We'll also need to port ignition-release/ign-gazebo3-release#16 to ign-gazebo6-release

https://github.com/ignition-release/ign-gazebo6-release/pull/18


[Request] a java version of Truecraft

I really like Truecraft and the purpose of it for being close to core Minecraft , but its been a real hassle to make work properly, or the dev team intended this game only for Windows users, if its true i m ok with it but if its created for all it should be easier, thanks for all your work.

lol absoultely not


Document the use of Parity Scale codec

This topic was raised by Alexander Strokov recently, and since I don't see it, I'm adding one.
We should have a more detailed explanation of Parity Scale codec, what it provides for Iroha and how it's used over the HTTP(s) protocol.

need to create a jira to track this


Implement support instrumentation

This pull request addresses a compilation error encountered when using diesel-tracing with diesel version 2.2.0.
The error is due to missing implementations for instrumentation and set_instrumentation methods required by the Connection trait.
The compilation error is as follows:
   Compiling diesel-tracing v0.2.5
error[E0046]: not all trait items implemented, missing: `instrumentation`, `set_instrumentation`
  --> /usr/local/cargo/registry/src/index.crates.io-6f17d22bba15001f/diesel-tracing-0.2.5/src/pg.rs:91:1
   |
91 | impl Connection for InstrumentedPgConnection {
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ missing `instrumentation`, `set_instrumentation` in implementation
   |
   = help: implement the missing item: `fn instrumentation(&mut self) -> &mut (dyn Instrumentation + 'static) { todo!() }`
   = help: implement the missing item: `fn set_instrumentation<impl Instrumentation>(&mut self, _: impl Instrumentation) where impl Instrumentation: Instrumentation { todo!() }`

Thank you for your time and consideration.

Thank you :pray:


RFC: Include sampling in the spec

Right now, the spec intentionally doesn't mention sampling. And indeed, some parts of sampling (client-side vs server-side, ...) depend on the specific implementation. There are, however, two parts to sampling that could, and most likely should, be covered in the spec:

Baggage propagation shouldn't be affected by sampling.
We do have a standard tag for the sampling priority. This leads me to believe that we should document the tag in the spec, and specify that there needs to be a way to force sampling of a request.

I'm somewhat ambivalent about point 2 and will easily be convinced that this is too implementation-specific. I do feel strongly about point 1.
/cc @bensigelman @adriancole @yurishkuro

@dominikh  I'm curious to hear your reasoning for point 1 and get a better understanding of it. To be upfront I don't have an opinion on this so bear with me for a second.
If I have some tracing implementation that only supports upfront sampling, such that only the root span gets to make that decision, one thing I might want to do is use a Noop span implementation to reduce the overhead of something that isn't being recorded anyway. Adding baggage doesn't do anything and as a result baggage is essentially disabled. Would this use case be in violation of point 1 or is this irrelevant since children spans have no say in sampling? Maybe I'm thinking too specifically?

@bg451 The way I understood @bensigelman on Gitter, baggage may be used to transmit data that is useful for things like resource usage accounting or authentication information. From that follows that baggage shouldn't be affected by the sampling of a trace.

@dominikh I appreciate (and even agree with) your core point: that baggage propagation isn't useful unless it's always-on. That said, OpenTracing is only designed to work properly when the entire distributed system uses Tracer implementations from the same vendor (whether that be Zipkin, Appdash, whatever); as such, why not leave this matter to the tracing implementations?
I.e., if someone wants to create a MinimumOverheadTracer that drops baggage some of the time, they just need to be sure to advertise that fact... developers who use OpenTracing can adopt or not-adopt MinimumOverheadTracer as they choose.
My thinking here is just that some people will never use baggage (which is fine), and they may want to adopt Tracer impls that are more efficient because they don't support it in some/all cases.
I recognize that I'm basically arguing against true interop here... I think that's because I've considered Baggage to be an experimental feature. I would be less willing to forgive/accept a Tracer implementation that didn't support Span.SetTag, for instance.

@bensigelman Come to think of it, my first point isn't really about sampling, but about the fact that baggage itself isn't defined very well. Including myself, I know of two people so far who made the mistake of applying sampling to baggage, because baggage sounded like a variation of tags that just avoids extra roundtrips to get to the data.
It isn't made clear that baggage is really about context propagation and that baggage and tracing are really two different concepts stuffed into the same spec.
However, to fully explain baggage, sampling needs to be at least mentioned.
My second point, however, truly is about sampling. We have a standard tags whose semantics aren't well defined because things like forced sampling aren't covered in the spec. IMO, tags should have a consistent meaning across implementations as much as possible.

... continued at https://github.com/opentracing/specification/issues/11.


Description and nickname on individual license pages

Individual license pages don't display the license description or (if available) nickname provided in the front matter of files in _licenses, e.g., http://choosealicense.com/licenses/eupl-1.1/ and https://github.com/github/choosealicense.com/blob/gh-pages/_licenses/eupl-1.1.txt
This info seems somewhat useful, especially if one gets to an individual license page through a search engine rather than by navigating the site.
These fields are accessible as page.description and page.nickname and might be added to _layouts/license.html or _includes/sidebar.html, depending on where they ought be displayed.
I'm terrible at making this look nice. If anyone wishes to tackle this, make a pull request. Alternately, if anyone feels this is a terrible idea, scream.

Hi @mlinksva,
I think this is a good idea. I attached something I came up with, I tried to keep it minimal and give more space for any future additions if needed.  I saw that the site is using qtip2 so I was going to use it when the user hovered over the link.  Let me know what you think.

@LeoFNaN thanks for coming up with this! Seems reasonable to me for the description. I don't think hover-over to see nickname often shorter than "nickname" is very reasonable. But maybe the nickname could be in the title area of the hover over box, rather than the duplicative "Description"?

As a user, I would expect the description to appear in a paragraph below the title and before the full text of the license. I can't see any good reasons to use a "hover to see" mechanism when we could just plainly add the text there.
By the way, the nickname could be in the sidebar, before the usage instructions.

I generally dislike hovering or clicking to see, but in this case I like that it would very crisply separates the summary from the license text.
Of course the description could also go in the sidebar. Maybe that's the easiest thing to do.

Ok you guys bring up some good points, I was in a make it interactive mode last night :)  So description block in the sidebar, then the nickname and the rest of the content after.

Would you like to make a PR implementing in the sidebar?

I'm actually trying to run the repo on my machine but having issues with script/server throwing an error...I wanted to take a look at it before I created a PR.
 Conversion error: Jekyll::Converters::Scss encountered an error while converting 'assets/css/application.scss':                     Invalid CSS after "...s/normalize.css": expected "{", was "" on line 1

@LeoFNaN I'm not getting that error. Did you follow the steps in https://github.com/github/choosealicense.com#run-it-on-your-machine ? If there's some way the documentation or something else could be improved to avoid other contributors getting that error, I'd like to do so.

@mlinksva Yes that's what I followed.  In the application.scss file there are 2 rows of dashes at the top of the file and when I removed them script/server was able to complete.  But then application.css wasn't generated so the css wasn't being applied, I added application.css to the dir just so I can have a look at the changes.  I may have missed running something.  I'm on a win7 computer.

How about displaying nickname and family side by side? check out the image.

Also, has it been considered to changing the function of the button from copy license to clipboard to Download the LICENSE.txt file? Adding a little convenience and possibly removing the content about creating the file under how to apply.

The family shouldn't be displayed. It's used for grouping on /licenses but isn't 100% accurate taken as facts about licenses.
The copy button is useful for pasting into an editor, but download with conventional filename seems useful too. Maybe another button, "Download as LICENSE.txt" or similar?
Please do open a separate issue about the error you've gotten and I'll try to reproduce.

Note the download could be created with javascript, like described in http://cwestblog.com/2014/10/21/javascript-creating-a-downloadable-file-in-the-browser/ or alternately Jekyll would have to output a bare LICENSE.txt file for each license. The latter would be slightly nicer as bare license texts would have URLs.

Ok family meta noted.  Let me know what you think but I feel moving how to apply underneath permissions, _conditions _and limitations will provide more context with the use of the license rather then having how to apply at the top (which can be further justified to move to the bottom if its decided to add a download button).  It's a small matter but just letting it out there.
Yeah I think another button will be nice and thanks for the link, should I create a separate issue for it and see where it goes?

I really like your proposal overall @LeoFNaN, with the caveats pointed out by @mlinksva. I also like the option to have urls for the bare licenses. Moving "how to apply" to the bottom also seems like a good choice to me, assuming a workflow where one would learn about the license first, and then decide to use it.

I'm not thrilled with moving how to apply down because it'd often be below the fold. Longer term I wonder if it'd make sense to reserve the sidebar for actions and application documentation (button(s) and how to apply) and description into the main column, but clearly separated from the license text. Permissions/conditions/limitations could be arranged horizontally as they are on /licenses. Then you'd have
TITLE            |  copy
                 |  download
Description      |  howto
                 |  who's using
License Text     |

In any case let's get the download action in first. Go ahead and create an issue for it, or a PR directly.

I wouldn't mind the usage instructions being presented in a popup/tooltip format, and thus fit next to the copy/download buttons, in a single line on the sidebar. As in: copy | download | how to apply?
That said, I also like the idea of "reserving the sidebar for actions and application documentation (button(s) and how to apply) and description into the main column"

Alright I applied a mix of the feedback.  I added Nickname next to Description title, and altered the how to apply content to make up for whichever use case (copy or download) and changed the button colors a little (that can be reverted if preferred).
@waldyrious I tried to apply the buttons and how to apply on one line but it seemed a little busy for one line also the sidebar felt empty and unbalanced in comparison to description and P|C|L being placed in the main column.

I love it :) great work!

Indeed, it looks great. I don't think the Description heading is needed, nor the dash(?) before Nickname.

Nor the word "nickname" itself, I suppose.

Thanks guys.  i feel like the nickname by itself may be out of place especially since the description heading may be removed, should the nickname be moved?
@mlinksva should i perform the changes off the current pr or close it and create another?
i'll get on the button after this one :)

@LeoFNaN create another PR. I just merged #382

Regarding the nickname, I think it's useful to display. Maybe in the new PR try without any label. That might work decently.

Fixed by #399. Thanks @LeoFNaN!


Add link to stats page in ui and ui_wiki

Looks good.  Tested on my localhost.  Link appears in the right spot and works correctly.

Thanks for working on this, sorry that i couldnt follow thru when i volunteered to do this.

On Oct 12, 2017, at 10:14 AM, Aaron Halfaker<EMAIL_ADDRESS>wrote:
Merged #202.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


python311Packages.autoit-ripper: remove stale postPatch section

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

2 packages failed to build:

python312Packages.karton-autoit-ripper
python312Packages.karton-autoit-ripper.dist

6 packages built:

python311Packages.autoit-ripper
python311Packages.autoit-ripper.dist
python311Packages.karton-autoit-ripper
python311Packages.karton-autoit-ripper.dist
python312Packages.autoit-ripper
python312Packages.autoit-ripper.dist


Move codenvy

They were recently acquired by redhat and are going to be merged into the openshift team so it might make sense to remove them https://blog.codenvy.com/redhat-acquires-codenvy-c84e1202a042

@Moshe-Immerman could you modify the cloud66 entry to turn it into https ?


[FR] support benchmarking max memory usage

This package is a thin wrapper around the upstream criterion package so you'll want to open an issue there: https://github.com/haskell/criterion/issues


"TypeError: r_.clear is not a function" in Docker container

I am using NextJS (v13.0.3) and implemented the basic example (https://github.com/brimdata/react-arborist#examples). When running this in a production build (e.g. npm run build npm run start) the example loads and works as expected. When I create a docker container and launch that, the same example produces the following errors:

It seems to stem from the deselectAll function call, however, I am not making calls to that function and the error occurs when attempting to load the tree UI.

It's a bug in SWC minifier, adding swcMinify: false, to next.config.js fixes the issue

@remorses Amazing. Saved me another frustrating day at the office :) Thank you!
I was also getting the same thing, rather with TypeError: r_.only is not a function, only in a built version of my next.js app, running in pm2.
swcMinify: false solved the issue.

@remorses is there anything that needs to change in this library? Do we need to upgrade a dependency?


Create "Status board" dashboard in GitHub Project

This dashboard will help to easily view our issues and sort them by status.
Tasks
"Status board" dashboard should contain next tabs:

Todo
In Progress
In Review
Done

Also added "Backlog" tab. This tab will contain all of the issues we need to solve before finishing internship.


Consider using a library to simplify URL processing

Such as http://www.gorillatoolkit.org/pkg/mux

We're now using go-restful.


Cookie should be checked with data base

Is your feature request related to a problem? Please describe.
The start screen tells me "welcome back". But when I click enter, I get an error saying "user unknown"
Describe the solution you'd like
It should not only if I have a cookie, but if this cookie is know/valid. If I have a cookie but it is not valid, there could be a message like "Your account expired. We try to avoid it, but sometimes we need to reset everything. Click here to create a new account" which would delete the cookie.
Describe alternatives you've considered
If a cookie is detected, the user is automatically logged in (skip the "enter" page). If automatic log in does not work. Show a "Log In or  Sign Up" page.
Additional context
Loging in is the very very first action a user does. It should run as smooth as possible.

done


"ValueError: list.remove(x): x not in list"

  File ".........", line 343, in _run_event

    await coro(*args, **kwargs)
  File "....\main.py", line 51, in on_message
    skin = api.playerSkins(username=message.content.split()[2])
  File "..........\namemc.py", line 286, in playerSkins
    skin_hash_list.remove('javascript:void(0)') # just have to get rid of something that stays in the code idk y
ValueError: list.remove(x): x not in list

Please help, I tried fixing the issue myself but nothing really worked.

I'll look into it.

Question, can you give me the code you were using in that segment so I can see what caused the issue.

Question, can you give me the code you were using in that segment so I can see what caused the issue.

Unfortunately I can't really show much of my code since I am using a new API now, but I did add api = namemc.namepy() to the beginning of my code and message.content.split()[2] was a valid Minecraft username.

I'm pretty sure the issue resides in where you try removing javascript:void(0) from skin_hash_list, but it's actually not always there. Also noticed it only worked with my skin and no one else's?

What is your skin hash?

Ok I found that this issue occurs when the player has many amount of skins

When a player has a large amount of skins it will not display the javascript error I will fix this

Fixed will be rolling out the update!

Fixed

Thank you! I was really confused why it only worked on my skin, lol

Sadly there is an bug where it does not show all the skins if you have a large amount of skins currently I will have to fix.

Its probably the scraping as when you do it, it does not show all the skins but I know of a fix on how I can fix this

May I ask why you have to scrape it and not get it directly from the API? After all this is an API wrapper

Sadly, namemc or mojang api does not offer these features with there api so I have to resort to scraping.

Alright, I guess that's understandable.

Yea I want them to add more api features. :/


Chris development

Added an initial diagnostic with module save_peak. All peaks used for gamma analysis are saved in a folder unique for that sample. PNGs are saved there and contain what the Gamma_Analysis code determined to be the peak and compton regions. The image and regions are generated using the Spectrum_Peak_Visualization modules.
Currently, the unique Cs134 compton regions are not fully considered and must be added to the visualization modules.
Another CSV file is also generated that contains insignificant concentrations replaced with N.D.. This should be easier for users to post to the Radwatch website.

When I was making this, I was just following the isotopes that were on the spreadsheet. I wasn't sure if we were going to use them in the future, but we could probably get rid of those since I'm guessing only the main five isotopes are reported. Although if people are curious about other isotopes (an unlikely if), new isotopes wouldn't be hard to add.
I also think some speed issues are possibly related to the structure of peak_visualization, where it was looking at all the peaks when I want to look at a specific one. I added one input that might make it a bit faster, but less isotopes will obviously speed it up.

Thanks, I'll take a look now.
Note that Mustapha and I rewrote ROI_Maker and updated Gamma_Analysis.peak_measurement() to use ROI_Maker. Spectrum_Peak_Visualization will use the new ROI_Maker too, Mustapha is working on that. So probably don't work on the visualization until that one gets merged too.

You've got a merge conflict to resolve in Spectrum_Peak_Visualization.py and it's causing an error whenever SPV is imported.
Basically both you and Tyler added some xlim code. Remove the <<<<< ====== >>>>> and pick whichever xlim code is better (I don't think it matters too much).

Which xlim code are you referring to, I only see two cases of plt.xlim(xmin=0), which I think I'll keep.

Well, the plt.xlim(xmin=0) sets the left edge of the x range, which undoes part of the first xlim.
I guess what we really want is
if energy_range:
    plt.xlim([energy_range[0], energy_range[1]])
else:
    plt.xlim(xmin=0)


MNIST dataset - gzip: train-images-idx3-ubyte.gz: not in gzip format

From<EMAIL_ADDRESS>I'm trying to use:
from tensorflow.examples.tutorials.mnist import input_data
mnist_data = input_data.read_data_sets('MNIST_data', one_hot=True)
but I am getting the error that the downloaded file is not in GZip format.
There are several bug reports on this, but none of them seem to solve the issue.
I found train-images-idx3-ubyte.gz in the local directory MNIST_data, but even trying gunzip fails:
gzip: compressed data not read from a terminal. Use -f to force decompression.
For help, type: gzip -h
(tf) [bduser@param03 MNIST_data]$ gunzip train-images-idx3-ubyte.gz

gzip: train-images-idx3-ubyte.gz: not in gzip format

I'm thinking that this is a problem with the original datafile. Maybe the GZip format has changed or is in conflict?? Or perhaps my OS' version of GZip can't understand the GZip file??
I am using TF 1.4 on CentOS Linux release 7.4.1708 (Core)
Thanks.
-Tony
>>> from tensorflow.examples.tutorials.mnist import input_data
>>> mnist_data = input_data.read_data_sets('MNIST_data', one_hot=True)
Successfully downloaded train-images-idx3-ubyte.gz 727 bytes.
Extracting MNIST_data/train-images-idx3-ubyte.gz
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/bduser/miniconda2/envs/tf/lib/python2.7/site-packages/tensorflow/contrib/learn/python/learn/datasets/mnist.py", line 242, in read_data_sets
    train_images = extract_images(f)
  File "/home/bduser/miniconda2/envs/tf/lib/python2.7/site-packages/tensorflow/contrib/learn/python/learn/datasets/mnist.py", line 56, in extract_images
    magic = _read32(bytestream)
  File "/home/bduser/miniconda2/envs/tf/lib/python2.7/site-packages/tensorflow/contrib/learn/python/learn/datasets/mnist.py", line 38, in _read32
    return numpy.frombuffer(bytestream.read(4), dtype=dt)[0]
  File "/home/bduser/miniconda2/envs/tf/lib/python2.7/gzip.py", line 268, in read
    self._read(readsize)
  File "/home/bduser/miniconda2/envs/tf/lib/python2.7/gzip.py", line 303, in _read
    self._read_gzip_header()
  File "/home/bduser/miniconda2/envs/tf/lib/python2.7/gzip.py", line 197, in _read_gzip_header
    raise IOError, 'Not a gzipped file'
IOError: Not a gzipped file
>>> exit()

It doesn't look like the downloaded file worked properly. Please cat the file and look what those 700bytes are. I'm sure they are a text error message saying why the download fails. See, my MNIST_data/train-images-idx3-ubyte.gz is 9912422 bytes and yours is 727 bytes. Catting the file will probably tell you why it wasn't downloaded properly...
>>> from tensorflow.examples.tutorials.mnist import input_data
>>> mnist_data = input_data.read_data_sets('MNIST_data', one_hot=True)
Successfully downloaded train-images-idx3-ubyte.gz 9912422 bytes.
Extracting MNIST_data/train-images-idx3-ubyte.gz
Successfully downloaded train-labels-idx1-ubyte.gz 28881 bytes.
Extracting MNIST_data/train-labels-idx1-ubyte.gz
Successfully downloaded t10k-images-idx3-ubyte.gz 1648877 bytes.
Extracting MNIST_data/t10k-images-idx3-ubyte.gz
Successfully downloaded t10k-labels-idx1-ubyte.gz 4542 bytes.
Extracting MNIST_data/t10k-labels-idx1-ubyte.gz

Yes. It looks like my corporate firewall is preventing the service from downloading the file.
I thought the proxy settings were correct on this machine, but apparently not.
Thanks for the help.

(tf) [bduser@param03 MNIST_data]$ cat train-images-idx3-ubyte.gz

Request Error

Request Error (invalid_request)

Your request could not be processed. Request could not be handled

This could be caused by a misconfiguration, or possibly a malformed request.

For assistance, contact your network support team.

Glad you figured out the problem. Closing for now.


Method not accessible: wrong error message

The error message for inaccessible methods is wrong: "Method not accessible to ". The correct message should be something like "Method  not accessible to ".

Also use the new origin facility for the error message: error source is the class that cannot access the method, origin is the inaccessible method.


Same user can't login from 2 computer at a time

Hi I am using laravel 4.2, I can't login with same user details from 2 device at a time. Only 1 can be logged in. Can you please share me some code which will give user to access same user from both device.

Hi, for L4.2, we're using Cartalyst's Sentry package for authentication. To enable multiple logins, you may need to change the source of Sentry. Take a look at http://stackoverflow.com/a/17399735

This method is probably better http://stackoverflow.com/a/23394423

Hi It's still not working . I followed this url: http://stackoverflow.com/questions/17393833/laravel-enable-sentry-user-account-be-used-in-multiple-computers/17399735#17399735 but it's not working. Please help me.

Hi, I've tried that quite long ago. Follow the one from Gravy instead.
http://stackoverflow.com/a/23394423
Create a User.php model with this content:
use Cartalyst\Sentry\Users\Eloquent\User as SentryUser;

class User extends SentryUser
{
    // Override the SentryUser getPersistCode method.

    public function getPersistCode()
    {
        if (!$this->persist_code)
        {
            $this->persist_code = $this->getRandomString();

            // Our code got hashed
            $persistCode = $this->persist_code;

            $this->save();

            return $persistCode;
        }
        return $this->persist_code;
    }
}

Then you need to publish the config file of Sentry with this command:
php artisan config:publish cartalyst/sentry

It should create a file in app/config/packages/cartalyst/sentry/config.php.
In that file, replace
'model' => 'Cartalyst\Sentry\Users\Eloquent\User',

with
'model' => 'User',


[kws_streaming] export stream_state_internal savedmodel stuck at  keras.engine.functional.reconstruct_from_config

system: ubuntu 20.04
tensorflow:  2.9.1
Based on the experiments/ , I have success in train & export saved model of non_stream .
Tf2.9.1 is the only version I tried can train the models with some little fixed , I can't find  tf_nightly-2.3.0.dev20200515 in the experiments/  。
But when i tried to export saved model of stream_state_internal.
I found it stuck at reconstruct_from_config , the node_index_map is  always empty ,so node_index_map return always None. It keeps add node into unprocessed_nodes.
I print some variables .

Thanks!

@IvyTang even if you find version tf_nightly-2.3.0.dev20200515 you will have to use the corresponding version of kws_streaming at github. For example you can pick a version which is close to tf_nightly-2.3.0.dev20200515  at link and kws_streaming version of Oct 23, 2020, e.g. commit). kws_streaming is always at sync with the latest version of TF, so it is better to use TF nightly version. I would suggest to try another model e.g. 'gru' and let me know if it works (there is no need to train it: train for 1000 iterations and check that it can be converted).

Hello @rybakov
it seems that tf_nightly-2.3.0.dev20200515 cannot be downloaded anymore. I used the last google research package release with TF 2.14 and I got the same issue as mentioned here : https://github.com/google-research/google-research/issues/1753.
Could you please either tell us where to find the tf_nightly release or which TF release has been correctly been tested successfully with the last kws_streaming ?
thanks a lot !

Looks like someone filed an issue with tensorflow about this infinite loop in clone_model here, and the approved solution is to migrate code to use keras3 instead of tensorflow.keras.


typescript import not working

Using qunit-dom in an Ember project with ember-cli-typescript
ember ts:precompile gives errors:
tests/filename...(line, col): error TS2339: Property 'dom' does not exist on type 'Assert'.
As per the readme, I've added the import
// test-helper.js
import Application from '../app';
...
import { start } from 'ember-qunit';
import 'qunit-dom';
...

Faced the same issue. You should make sure that you are on qunit-dom >= 0.7.1<EMAIL_ADDRESS>installs ^0.6.2. Also make sure that tests/test-helper.ts is a typescript file. Otherwise typescript doesn't seem to recognize the import.


[refactor] better organise namespace

TERMINATE() macro is removed; use RTE_THROW instead
STOP() macro removed; use RTE_THROW instead
WARNING() macro is removed; use RTE_WARNING
in many paces assert() macro is replaced by RTE_ASSERT()
utils:: namespace is removed

math functions are moved to math_tools.hpp
time functions are moved to time_tools.hpp
output stream functions are moved to ostream_tools.hpp
string functions are moved to string_tools.hpp

some files are moved to ./core folder in preparation to remove SDDK
splindex class is moved out of sddk:: to sirius:: namespace
old interface to Plasma eigensolver is removed
some unused gpu kernels are removed
acc:: namespace is re-organised and moved to ./core

This PR touches a lot of files, but it doesn't change the logic of the code. Only source file reshuffling and namespace organisation. Unused cuda code is removed. What is left is la:: space, but this will be done in separate PR after https://github.com/electronic-structure/SIRIUS/pull/900

cscs-ci run default

I wonder if we can just put ETH Zurich / CSCS instead of the authors because we can always trace the authors back with github and commits.

The git blame is not necessarily the same as the author, due to reformatting or other cosmetic changes.

I saw in other codes that each file is equipped with boile plate
I will be happy to switch to ETH Zurich / CSCS ownership or even sirius-org. I just need to find out if I can "just do it".

I wonder if we can just put ETH Zurich / CSCS instead of the authors because we can always trace the authors back with github and commits.

The git blame is not necessarily the same as the author, due to reformatting or other cosmetic changes.It does not scale with projects that have far too many devs. We are three so it is ok.

we do not have to do it now, I wanted to put the question on the table because we will eventually have to do something like this. I guess we can just leave it as is until SIRIUS really see external developers contribute to it.

Licencing each file is also fine.


[tck] org.osgi.service.jdbc - enhancements

switch to junit 5
use osgi-test
remove tests on custom properties ("junk" -- derby related test)
test (notNull) return value of DataSource and Driver methods

Discussion:
Spec uses should and optional this would be a way to test this:

test "should" with separate @Tag and "@EnabledIfSystemProperty"

Discussion:
Spec uses should and optional this would be a way to test this:

test "should" with separate @tag and "@EnabledIfSystemProperty"

This actually raises a long-standing issue with the service, which is that there is no way for the client to know if a method is implemented or not. Would it make more sense to define some service properties on the DataSourceFactory service so that everyone can know what's supposed to work?

It would ! It just getting to late yesterday. And some thing i would not address in this PR because this should just fix some issues I had on the implementation site.
There are two topics:

SHOULD and OPTIONAL sentences in the Spec -> @Tag ed Testmethods and "@EnabledIfSystemProperty"
Unclear which method can be used without exception.

Service-Properties should be specified in an 1.1 version.
e.g.
org.osgi.service.jdbc.capability={ "DataSource", "ConnectionPooledDataSource", "XADataSource", "Driver"}

or
org.osgi.service.jdbc.capability.dataSource=true
org.osgi.service.jdbc.capability.ConnectionPooledDataSource=true
org.osgi.service.jdbc.capability.XADataSource=true
org.osgi.service.jdbc.capability.Driver=true


Remove ClusterRole and ClusterRoleBinding from the chart

As these applications are namespace'd scope; we should remove the cluster level resources like ClusterRole and ClusterRoleBinding
This chart should only support creation of resources which are only namespace scoped

fixed by https://github.com/stakater-charts/application/pull/118

Too bad, I was looking for a chart that supports ClusterRole and CRB resources. That one was perfect apart for that part :disappointed:


Fix for deadlock. Issue #183

The double 'await' in HttpDocumentRetriever.cs caused a deadlock.
Introduced new variable 'keys' in OpenIdConnectConfigurationRetriever.cs for clarification.

This pull request is now managed with CodeReviewHub. Refresh.


V12 ETA?

Any news on how long it will approxcimately take until CPR will be V12 compatible? thanks in advance!

CP*

There's an open beta available on the CPR Discord.


csv output issue with -w option.

It is not uncommon for registrar names to have commas, which pushes "whois_created" and "ssdeep" values right.  "Quoting" the output of the comma separated values should resolve it.
Example of WHOIS with comma in registrar name:
subdomain,xxxx.xxxx.xxx,199.59.xxx.220,,,,,TurnCommerce, Inc. DBA NameBright.com,2000-02-21,

Thanks,
Andy

This has been already fixed with commit d7395646a2521d00aeb152aaefc16d3e6727e87c.

Guess I should freshen up.  Sorry about that....appreciate passing that along.


Add option to start new layouts with desktop offset off

In most of my usage I'd like to have the desktop offset turned off, with zero margins anywhere. But I would still like to be able to activate the offset in order to see files which have been downloaded to my desktop.
Please could we have an option to start all layouts with the desktop offset de-activated, so we can only activate it when needed?

An option like this would make configuration a little bit easier. However, you can achieve the same thing by toggling the offset on desktop creation.
Here is an example from my config file:
# create desktop, move window and follow focus
shift + cmd - n : chunkc tiling::desktop --create;\
                  id=$(chunkc tiling::query --desktops-for-monitor $(chunkc tiling::query --monitor-for-desktop $(chunkc tiling::query --desktop id)));\
                  chunkc tiling::window --send-to-desktop $(echo ${id##* });\
                  chunkc tiling::desktop --focus $(echo ${id##* });\
                  chunkc tiling::desktop --toggle offset

# create desktop and follow focus
cmd + alt - n : chunkc tiling::desktop --create;\
                id=$(chunkc tiling::query --desktops-for-monitor $(chunkc tiling::query --monitor-for-desktop $(chunkc tiling::query --desktop id)));\
                chunkc tiling::desktop --focus $(echo ${id##* });\
                chunkc tiling::desktop --toggle offset

Note the call to chunkc tiling::desktop --toggle offset in the last line(s).


Can't get secondary manger/owners

Sport
Football
Summary
Only one manager/owner is returned on teams that have multiple manager/owners.
for team in league.teams:
    for owner in team.owners:
        print (owner)

Logs/Data
No response

I created a PR that should check all owners for a team

should be fixed in v0.37.1!


Fix icon for RFmx DPD Configure Common

Icon for Configure Common in DPD should read "Config Common" but is incorrect.

@smooresni
Originally posted by @Hanrock007 in https://github.com/NISystemsEngineering/rfic-reference-design-libraries-labview/pull/90#issuecomment-627928189

RFmxNR.lvlib:NR Configure Common.vi has the same issue.


Consolidate historical browser metric data

Historical metric data (i.e., all metric data except "API Velocity" metric data) follow the same pattern. They capture a data point at a particular point in time, and store (1) the main browser release used for metric calculation, (2) any previous browser releases used for metric calculation, and (3) releases of other browsers used for metric calculation.
This information is now consolidated in a single class that also stores the "type" of metric associated with the data point (an enum value). Data attached to enum values capture metadata and documentation related to each individual metric.

Codecov Report

Merging #70 into master will increase coverage by 0.07%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master      #70      +/-   ##
==========================================
+ Coverage   66.18%   66.26%   +0.07%
==========================================
  Files          32       30       -2
  Lines        1325     1328       +3
  Branches       97       98       +1
==========================================
+ Hits          877      880       +3
  Misses        448      448

Impacted Files
Coverage Δ

lib/chart/aggressive_removal_chart.es6.js
22.22% <ø> (ø)
:arrow_up:

lib/chart/browser_specific_chart.es6.js
22.22% <0%> (ø)
:arrow_up:

lib/chart/failure_to_ship_chart.es6.js
22.22% <0%> (ø)
:arrow_up:

lib/confluence/aggressive_removal.es6.js
100% <100%> (ø)
:arrow_up:

lib/confluence/browser_specific.es6.js
100% <100%> (ø)
:arrow_up:

lib/confluence/failure_to_ship.es6.js
100% <100%> (ø)
:arrow_up:

lib/client/api_confluence.es6.js
12% <50%> (-6.52%)
:arrow_down:

... and 2 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 d43159f...eed33e7. Read the comment docs.


Get server ip:port from controller

I would like to show the server listening ip:port on a page, but Im not sure if its possible at the moment?
The reason is I have a revel app deployed on multiple servers behind a load balancer, and it would be useful for debugging purposes to quickly know which server that served a certain request. One way to easily handle this would be to print out the ip:port on a page.

Have you tried revel.Config.String("http.addr") and/or revel.Config.StringDefault("http.port", "default value")?

Would be handy if it was a "property" of the controller though. imho ??

I have the same app.conf everywhere, the two servers where I have deployed my revel app are identical (except that they have their own IPs). The easiest solution would definitely be if it would read the "real" ip and not a configuration/environment value, since otherwise I must configure the two servers differently.

@viblo Alternatively, it is possible to use revel.Server.Addr to get an address that is listened by your app.

That was strange, revel.Server.Addr returns :43812, same as is printed in revels log. However, in my revel config file I have http.port=18227? I havent event noticed that it actually listens on both ports. (Where does 43812 come from? I thought it would only listen on what is written in config file). Anyway, that value is no help since it doesnt include IP and its not even the port I use to connect to.

@viblo Are you starting your app as run.sh (after revel build path/to/myapp path/to/dest) or as revel run?

@anonx with revel run. When I now test with run.sh it shows the correct port. I didnt realize that it behaved differently depending on the method used to run. (I do use run.sh in production, so no big concerns about revel run). However, I still dont have the IP.

@viblo There is a difference between revel run and run.sh. revel run starts your app and reverse proxy (that one which shows you user friendly errors when there are some), run.sh starts your app only.
You don't have IP as by-default your app listens *:port (i.e. all ports). To modify that behaviour you need to set http.addr in your conf/app.conf. As an example:
http.addr = ${ADDR}

Then run your app as ADDR=x.x.x.x ./run.sh when you want specific IP. And just run.sh and revel run will still work as before listening all ports.

Yeah, thats what I would ideally avoid doing, easier if it had the actual listening ip for the request available, but Im not sure if its even possible if the server is setup to listen to *. Anyway for now I have solved my problem by using os.HostName() instead.

Due to the common use of wildcard IP addresses, the best thing to do is add some app logic to your code that records os.Hostname().
There isn't an ideal way to change Revel's server.go to  use a specific IP address from a list of available IPs.


Add 1.21.x releases to release notes. Update latest Compose in config

Proposed changes
Some release notes were missing from the Compose release file.
Compose version in _config.yml had not been updated
cc @tfoxnc - Couple omissions on my part from #6994

Deploy preview for modest-edison-79d82e ready!
Built with commit afecc5aeea3fb5404cb82dd033c6a1d56a7f3987
https://deploy-preview-7086--modest-edison-79d82e.netlify.com


Update task status during a network allocation failure

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "network_alloc_errors" git@github.com:wsong/swarmkit.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

I think we should move it forward.

This was superseded by https://github.com/docker/swarmkit/pull/2287


Request URL Your URL didn't respond with the value of the challenge parameter

**Python version 3.6
rasa-core            0.11.1
Operating system (windows, 10...):
**Issue
Error while Integrating with slack When i am giving request URL it is giving error message. Your request URL gave us a 500 error. Update your URL to receive a new request and challenge value.
URL am passing
https://horoscopebot1234.herokuapp.com/webhooks/slack/webhook
My Standalone Script for Slack Chatbot
from rasa_core.channels.slack import SlackInput
from rasa_core.agent import Agent
from rasa_core.interpreter import RasaNLUInterpreter
import os
from rasa_core.utils import EndpointConfig
interpreter = RasaNLUInterpreter("models/nlu/default/horoscopebot/")
MODEL_PATH = "models/dialogue"
action_endpoint = EndpointConfig(url="https://horoscopebot1234-actions.herokuapp.com/webhook")
agent = Agent.load(MODEL_PATH, interpreter=interpreter, action_endpoint=action_endpoint)
input_channel = SlackInput(
slack_token="xoxb-587227996295-583760948608-Ox3hDGfL7rgS18t93vwOfHlz",
# this is the bot_user_o_auth_access_token
slack_channel="chatbot"
# the name of your channel to which the bot posts (optional)
)
s = agent.handle_channels([input_channel],  int(os.environ.get('PORT', 5004)), serve_forever=True)
*Screen shot of error

hmmm what was resolved here?

Any resolution of this issue

to be precise :

setup the credentials for slack (you did it already)

run ngrok and copy/paste the URL of the tunnel inside credentials.yml

save the file credentials.yml

run rasa run actions -vv to handle the actions

run rasa run -vv to start rasa and ALL the connectors

eventually set the URLs from ngrok inside the slack admin console : https://api.slack.com/apps/YOUR_APP_ID

open slack and the channel running your bot

start conversing with your bot :slight_smile:

Now it should work!


DNN to select WNN Permutation

TBD on what level, but suggest how can DNN be used to select the best permutation & sampling size to fit the best for a particular task.
Also, answer the question if there is a Perfect training - where are we loosing on accuracy, and what/where more degrees of freedom are needed.

Not sure I understand this item!


Create 2021-00004 - Leaked Phishing Credentials.md

Added report for case 2021-00004, as drafted

Can't merge myself, thus waiting for someone with merge rights to merge it.


Spring Force Contribution - Grass

Link to demo here: https://editor.p5js.org/leolewan/full/g_Dh6JvL1

Community Contribution
Link to live project, video, or image:
https://editor.p5js.org/leolewan/full/g_Dh6JvL1
Sharing

[x ] I would be happy to have my project shared on The Coding Train's social media!

Instagram: @leolewan
Twitter: @leolewan

This look really cool!

@gruselhaus thanks!

@gruselhaus I create two commits do you need me to squash them before merge?

Nope, PR got merged btw 😅


add min and max information in the decide output for continuous outputs

should also update README.md

Same as here: https://github.com/craft-ai/craft-ai-client-python/pull/129


Adjust C++ standard to version specified in CMake

it seems like current value is in incorrect format

Thanks ☺ 👍 !


[Bug]

Describe the Bug
While trying to launch discord it gives me an error ! Discord does not launch. btw repairing, uninstalling and installing again don't work either !
To Reproduce
Expected Behavior
expected to launch
Screenshots

Discord Version
Canary
Additional Context

I've been having the same issue. Even tried completely uninstalling all Discord versions + Better Discord and doing clean install. That didn't make a difference for me :(

Duplicate of #821


Add way to generate JSON schema for all API/DB structs

Once there are real deployments we will have to write migrations and keep the API versioned/consistent. To make sure no changes happened or to see when/what happened it would be good if we could generate schemas for all our DB persisted and API transmitted structs.
Probably easiest via schemars.

Closing this for now


v1.3 scan: Auth Mgmt Med 1

CampDv1.3APIupdate.xlsx
Application Information Leakage is an application weakness
where an application reveals sensitive data, such as
technical details of the web application or environment.
Application data may be used by an attacker to exploit the
target web application, its hosting network, or its users.
Therefore, leakage of sensitive data should be limited or
prevented whenever possible. Information Leakage, in its most
common form, is the result of one or more of the following
conditions: A failure to scrub out HTML/Script comments
containing sensitive information or improper application or
server configurations.
Failure to scrub HTML/Script comments prior to a push to the
production environment can result in the leak of sensitive,
contextual, information such as server directory structure, SQL
query structure, and internal network information. Often a
developer will leave comments within the HTML and/or script
code to help facilitate the debugging or integration process
during the pre-production phase. Although there is no harm in
allowing developers to include inline comments within the
content they develop, these comments should all be removed prior
to the content's public release.
Software version numbers and verbose error messages (such as
ASP.NET version numbers) are examples of improper server
configurations. This information is useful to an attacker by
providing detailed insight as to the framework, languages, or
pre-built functions being utilized by a web application. Most
default server configurations provide software version numbers
and verbose error messages for debugging and troubleshooting
purposes. Configuration changes can be made to disable these
features, preventing the display of this information.
Error Message Exposed via 'POST api.epa.gov/easey/staging/auth-mgmt/tokens/client/*'

I think from where we are getting the error messages that a generic message needs to write a custom message so our issue will be resolved because generic and user-friendly error messages instead of exposing detailed error messages that might reveal internal details.

False Positive. Approved reclassification on 3/4/2040


Improved build process

So because of a tweet, there is some interest in improving the build process for this project.
I'm game, but we need to identify requirements for something to be viable as a replacement for what we currently have with npm run scripts.
Required:

It cannot be slower than the current build process.
It must be easy to maintain and extend. (This is subjective, sure, but I think most of us can make fairly objective decisions here)
Must have exactly the same output in terms of what lands in the dist/ directory.
Cannot change our testing suite again.
Cannot completely restructure our codebase for the sake of the build script.
Must run cleanly with solid console output in Travis.
Ability to build pieces of our distribution one at a time (CJS vs ES6 vs AMD, etc)
Should build on Windows
Needs to have a solid upgrade path for newer versions of TypeScript

Nice to have:

Faster development time builds. Things like aborting the previous build and starting a new one if you save a file. Maybe only building what needs built
Flexibility for when we split our code base into separate packages (moving WebSocketSubject to an rx-dom type package, for example)

Why aren't we (meaning, mostly me) spending more time on this?
Mostly because what's in place works, and there are more important things to deliver on with a limited number of contributors.

It should also build on Windows: #1601

gulp #1592?

@Igorbek way back when this project started, gulp-typescript wasn't an option because it didn't support the version of TypeScript that we were using. We'd need to confirm that it supports 1.8.10, with a solid upgrade path to 1.9 and higher.

... in fact, I've just updated that as a requirement.

@Igorbek - This popped up on twitter originally and the discussion about npm scripts vs. gulp vs. something new. I'm totally up for helping out here but imo the best result is going to be the mix of gulp / scripts the produces the most optimal build workflow for the rxjs devs.
@kentcdodds had also mentioned interest in some sort of middle ground solution / pattern, I think his input here would be invaluable.
@blesh - I'm using gulp-typescript on 1.8.10 without issue

Oh... So... 😅 , I spent the day to build a solution for this (mostly for fun). Thanks for mention @d3viant0ne!
This is still a bit of a work in progress, but I think that this could really help things here:
https://github.com/kentcdodds/p-s
Feedback is definitely welcome.

@blesh - My first question after that twitter thread was, do any of the rxjs contribs have any gripes with the current build system?
I personally agree with @sindresorhus in regards to the build system, i'm not the one working on the project. Point being, in reality if it isn't broken in the eyes of the contribs, does it really need fixing?

Might I recommend scripty?

Unless I'm mistaken, this wouldn't solve the problem of having several dozen lines of scripts.

no, it would not. but that was not one of the requirements...

I realize that's not one of the requirements so feel free to ignore. It's the reason I'm not using it and built p-s. Just thought I'd mention that.

The only real requirement is that whatever the new solution is, it's fundamentally better than what exists.

@fyockm - For reference, the dozens of lines of scripts is how this all came about in the first place.
@blesh - Next question, Do you want this with rxjs 5 or after it's out of beta for less churn?

I've shared my opinion in https://github.com/ReactiveX/rxjs/issues/1592, would vote to npm scripts than other task runner. I know there's lot of different opinion for this, just my 50¢. I just don't see strong benefit by introducing task runner with its own complexity of writing configurations.

gulp is not my cup of tea. better-npm-run is pretty fresh. combining with parallelshell would probably speed things up.

@trxcllnt - Thanks for the suggestion, I just started tinkering with better-npm-run and i'm liking it so far. And for reference, i'm good with gulp but I don't just want to default to it. Like all things in life, somewhere in the middle lies the right answer.

combining with parallelshell would probably speed things up.

Just FYI, p-s supports running scripts in parallel :-)

@kentcdodds - I'm playing with it now. Converting something gulp heavy to see how much code I can remove

I'm favor parallelshell approach than gulp & gulp plugins based system.
gulp can manage to task dependency and task concurrency almost nicely, however, gulp runs on a single process so it would be a bottleneck, and gulp-plugin ecosystem requires us to use indirect dependencies.

My opinion is that the build system we have right now with npm scripts is fine. It's "big" because there is essential complexity, but when you need to tweak some task or add a task, it's a no-brainer. As an RxJS maintainer myself, these npm scripts have never been a headache to me, but I'm certain gulp would be (as it has been in the past for me).
I don't see any gain in moving the build system to something else just because someone on Twitter has an opinion. Look at it from a practical perspective: me and kwonoj are the "housekeepers" here and we would prefer npm scripts.
Windows users: I'm sorry, just use a Virtual Machine running an OS with proper bash or try to get Windows' own bash working. Don't look at this as an insult, I did this myself for a couple of years. I had a Windows machine and just installed Ubuntu as a virtual machine to get things done.

@staltz, I should preface everything I say with: "I'm not a contributor so don't listen to anything I say. I'm not allowed to have an opinion here" :-)
Of I were in your shoes, I'd probably stick with npm scripts too. I built p-s to see what it'd look like and because I know some people would appreciate it. I'm not 100% convinced that I'll actually use it in any of my own projects because I find npm scripts good enough.
That said, there are definitely tools that you can use to make your scripts more Windows user friendly if you're interested in lowering the barrier of contribution (which I expect @blesh is). I'd link these for you but I'm on mobile so forgive me:

rimraf
cpy-cli

Just looked through your scripts and I think using those two packages will get you most of the way there for supporting Windows better. Along with double checking that scripts are properly escaped (I believe globs need to be double-quoted).
Good luck y'all!

I'm fine with the tasks as they are, at first I thought "god this is weird" but it's slowly grown on me, in that it's simple and straight forward.  We don't have to deal with more advanced cases, we're not making a big large application.  We're making a simple library, as long as the builds suit the needs of the consumers with cjs, amd, etc, then what is the harm?
The only annoying thing I've noted is that since tests rely on the compiled library, you have npm run build_cjs then npm test.  I'm sure this flow could be modified with a daemon of some sort.
Personally I develop on windows full time, and the only issue I've ever seen is the occasional error from tslint that slips through because the tslint cli doesn't support globs on windows.
I do ensure that I have C:\Program Files\Git\usr\bin in my PATH, as it contains utilities like rm.exe which is a windows compiled binary.  I don't run the git bash shell, cgywin or anything special.  I do use Cmder over the default windows console.

On the heels of @kentcdodds, I would also propose using commands from shelljs to make things a little more Windows friendly. It contains many of the Unix style commands for use across platforms in one library, for whatever that's worth.

I'm totally favorable of using rimraf, mkdirp etc to try to make npm scripts work better in Windows :+1:

@staltz - I don't think anyone is looking to change things for the sake of doing so, though that is pretty much how the thread on twitter started.
The requirements are pretty clear and no matter what the tooling preferences are, It's hard to argue that there isn't room for improvement in the realm of development experience / general usability.
Nobody is suggesting going all "gulp-tastic" but if improvements can be made while picking up some of the nice to haves, how is that not a net gain? In particular when the people raising their hands to do it are not people who contribute to the project already.

FYI angular team is also evaluating build systems at the moment.

@alexeagle - Is there an issue open related to the discussion? I'd be curious to see the discussion / proposed options. The build tooling is pretty much everyones universal pain point right now :/

No open issue, sorry.
Internally we have nice TypeScript and JavaScript rules for our build tool, http://bazel.io/
It has the advantage that we can easily hook up a file watcher and a background tsc worker to get immediate, incremental re-builds and execute tests and binaries. But we have not open-sourced all the bits yet and Angular needs something sooner.
Looking into Gulp 4 for better description of ordering constraints between tasks.
Will probably still use our custom Broccoli layer for incremental typescript compiles.
Also planning to incorporate docker as the environment spec for better portability and reproducibility.

closing as inactive.


Include the target_uri in target uri is not valid error messages

Noticed that the error message here could be improved while looking at something where the target uri was being set incorrectly
This is also something towards https://github.com/grpc/grpc/issues/22883. Though this doesn't affect RPC status messages, it does improve the GPR_ERROR logs that are emitted upon channel creation failure: https://github.com/grpc/grpc/blob/86aa3579dfe8dbc9bc6258b894fa699e98752c75/src/core/lib/surface/channel.cc#L77

python failure: https://github.com/grpc/grpc/issues/23028
interop failure: https://github.com/grpc/grpc/issues/23785


Add optional state to events

Codecov Report
Base: 19.82% // Head: 24.81% // Increases project coverage by +4.99% :tada:

Coverage data is based on head (afe1647) compared to base (5353ba4).
Patch coverage: 86.86% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #132      +/-   ##
==========================================
+ Coverage   19.82%   24.81%   +4.99%
==========================================
  Files          25       25
  Lines        2411     2599     +188
==========================================
+ Hits          478      645     +167
- Misses       1933     1954      +21

Impacted Files
Coverage Δ

src/controllers/add.rs
0.00% <0.00%> (ø)

src/importers/balfolknl/mod.rs
28.64% <0.00%> (-0.15%)
:arrow_down:

src/importers/folkbalbende/mod.rs
0.00% <0.00%> (ø)

src/importers/webfeet/mod.rs
20.48% <0.00%> (-0.11%)
:arrow_down:

src/model/filters.rs
23.62% <11.11%> (-0.64%)
:arrow_down:

src/model/event.rs
29.67% <16.66%> (-0.19%)
:arrow_down:

src/model/events.rs
75.78% <99.41%> (+12.20%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add discordemoji cmd

Fetches emojis listed on https://discordemoji.com/
has an optional "count" param
E.g. for 'thinking' emotes: +discordemoji thinking.
It randomly sorts them first so if theres 100+ thinking emojis, you dont get the same 4 ones each time.

It's used here: https://github.com/dirigeants/klasa-pieces/blob/e3fadcf0db13cfe2b38cdaedfc19ac25e45a3bf0/commands/Tools/discordemoji.js#L18
The API returns every single emoji together, which is fetched once on init(), and then when the command is used, it searches that cached data.
Not sure what you mean by 'to the constructor', do it in the constructor instead of init?

No, I meant to add this in the constructor, to avoid object shape transitions.
this.emojis = null;


Support different styles of synchronization for local bucket

Currently the only one style of synchronization is supported - lock-free algorithm based on CAS and immutability.
It is need to support two additional types:

SYNCHRONIZED: for case when client does not care of contention too much because its primary goal is avoiding of memory allocation.
NONE:  for case when client does not need in synchronization at all, because synchronization is provided by thirdparty library(like Akka or RxJava), or because client wants to manage synchronization by itself.

public enum ConcurrencyStrategy {

     LOCK_FREE,
     SYNCHRONIZED,
     NONE

}

By default the Builder API should use LOCK_FREE strategy
public Bucket build() {
public class LocalBucketBuilder extends AbstractBucketBuilder<LocalBucketBuilder> {

      public Bucket build() {
            return build(ConcurrencyStrategy.LOCK_FREE);
      }

      public Bucket build(ConcurrencyStrategy concurrencyStrategy) {
           ...
      }

}

Done. will be released in scope of 1.3.0


Single select have differ behavior then system selectbox

Hi,
I would like to give you idea about behavior of single (=not multiple) select picker with "tab" key leaving.

Expand select picker options
With arrows move cursor to different option
Press tab to leave select picker.

Current:
After leave select picker with tab, ORIGINAL option will be selected, not the focused one.
System selectbox behavior:
After move option focus via arrows, focused option will be as selected after TAB pressed. So the selected is FOCUSED option via arrows.
System selectbox differs option focus after arrow up/down pressed and mouse over focus.
Best regards,
Michal

You'll want to enable selectOnTab if you want that functionality. But I agree, in the next major release selectOnTab should be true by default, since that most closely resembles standard select functionality.

See #2618


Out of Memory issue not showing up during deployment.

This is not due to the shm_size. There is no logging that said there is not enough memory for the container, but upon further investigation, that seems like it was the problem.

Hmm based on the initial warnings in the terminal you may want to add the following code to your Dockerfile.
Dockerfile:
 deploy:
      resources:
        reservations:
          memory: 1GB

Can you post your Dockerfile code please? This may fix the issue you're running into! You may need to add the following code and tinker with the optimal memory size that you need

The debugger now has access to out of memory errors so should give you better hints.


Problem with time_reached / at_the_end_of_time ?

I haven't reproduced this for myself, but user on the Adafruit Discord a user reported:

Program runs without generating any junk; but after nearly an hour it suddenly starts seeing an endless string of:
0xff 0xff 0xff 0xff .. from the USB port!

and showed this loop, citing https://github.com/wa1tnr/camelforth-rp2040-aU/blob/main/17-camelforth-a.d/pico-examples/camelforth-a/forth/rp2040_pico_getkey_usb.inc:
uint8_t ch_read;
do {
    ch_read = getchar();
} while (( ch_read == '\0' ) || ( ch_read == 0xff ));
// valid character now in ch_read

I noticed that getchar() uses a wait until at_the_end_of_time.  However, at least superficially it looks like "the end of time" is just 1.19 hours after boot!
absolute_time_t is a 64-bit quantity, but at_the_end_of_time is initialized to ULONG_MAX, a 32-bit quantity (0xfffffffu):

src/common/pico_time/time.c:17: const absolute_time_t ABSOLUTE_TIME_INITIALIZED_VAR(at_the_end_of_time, ULONG_MAX);

Instead, it seems it should be initialized to another value, possibly UINT64_MAX from stdint.h.
A second problem is likely to arise when waiting for absolute times after 1.19 hours:
PICO_WEAK_FUNCTION_DEF(time_reached)
bool PICO_WEAK_FUNCTION_IMPL_NAME(time_reached)(absolute_time_t t) {
    uint64_t target = to_us_since_boot(t);
    if (target > 0xffffffffu) return false;
    return time_us_64() >= target;
}

The if line should possibly read if (target == at_the_end_of_time) return false; and similarly for other sites.

Following.  Thanks jepler.

Note the only place you'll hit this right now is with explicit use of at_the_end_of_time by code - likely the getchar_with_timeout method you mention; the second file (with the PICO_WEAK_FUNCTION_DEF) is for host compilation.

Note the only place you'll hit this right now is with explicit use of at_the_end_of_time by code - likely the getchar_with_timeout method you mention; the second file (with the PICO_WEAK_FUNCTION_DEF) is for host compilation.

I'm not quite following you - by code are you referring to end-user code?  Because the issue's exposed by the implementation of getchar in stdio.c where the end-user isn't attempting to use any timeout.

yes it is a bug; i was saying it really for practical purposes only affects getchar unless you use at_the_end_of_time yourself

multicore.c may also have some weird behavior once the pico's been running for more than a ULONG_MAX.  I haven't dug into it far enough to determine if it's expecting to be using an absolute or relative "until".

that was the "for all practical purposes" bit... it will obviously be fixed in passing, but there is no reason to sit in that method for a long time

I just want to clarify for others reading that this does break the use of getchar() (which does not take a time as an argument, instead using at_the_end_of_time internally). Using getchar_timeout_us will be OK if you use a finite timeout. However, the SDK documentation suggests using getchar() in 2.7.1 (the documentation/examples for reading characters is in general pretty sparse...see raspberrypi/pico-examples#51). I've also seen similar problems with fgetc and fgets when reading from stdin. Pretty much anything other than the pico specific getchar_timeout_us with a finite timeout will eventually cause the pico to crash.
I realise there is a PR to fix this already (which I will test shortly). I mainly wanted to document this so it's easy for other people to find via search terms because I've been bashing my head against a wall for several weekends wondering why my serial communication locks up after 1.19 hours when almost identical code on other microcontrollers does not. I'm pretty sure this is why. I'll add a comment in a few hours to the PR should I verify it fixes my issue.

https://github.com/wa1tnr/camelforth-rp2040-aU/blob/main/17-camelforth-a.d/pico-examples/camelforth-a/forth/rp2040_pico_getkey_usb.inc#L3

I've re-titled the issue to mention getchar().

merged into develop

I seem to be having an issue with the fix in #145 where serial communication using getchar() is blocked in core 0 while core 1 is busy. It's fine when core 1 is blocked on reading from the multicore fifo, but while core 1 is doing something it seems like serial communication in core 0 is buffered until core 1 stops/blocks (at which point getchar() works as normal again). This issue does not appear in SDK v1.0.1, but does appear from the first commit onwards in #145.
I haven't had time to create a minimal working example (probably won't until next weekend) so I was wondering if anyone else had seen something like this while using #145?


Twitter-bot images not loading, but only in client.

I upgraded Zulip to 1.4.0 and now twitter-bot delivered messages are not displaying images, but only in the client.

macOS Sierra
Zulip-server 1.4.0

Electron Client:

Web Browser:

@akashnimare we can setup the Zulip twitter integration on zulip.tabbott.net to make it easy to reproduce this if you like...

@timabbott yes, please.

OK, setup done! :)

@GervaisdeM could you reproduce this on zulip.tabbott.net ?

Fixed in 34a62e2.

@akashnimare I think we shouldn't be disabling the web security option -- that's important to protecting users of the app from potential attacks.  Why did you feel you needed to disable it?  Based on http://electron.atom.io/docs/api/browser-window/, it seems to me that maybe what you wanted to do was just enable allowDisplayingInsecureContent?

protecting users of the app from potential attacks

@timabbott I didn't thought about this. Enabling allowDisplayingInsecureContent option will also work.

Great!


configuring base url for elysiajs url

I want to use path based routing for my application
e.g.
https://example.com/dev/app1/login  -->  Dev server
https://example.com/prod/app1/login --> Prod server
May I know whether i can define the basename similar to this for reactjs?
<BrowserRouter  basename={'/react-student-management'}>
https://stackoverflow.com/questions/41440240/configuring-apps-basename-in-react-router
thank you!

You can use prefix in constructor to achieve this:
new Elysia({ prefix: '/basepath' })
    .get('/hi', () => 'hi')
    .listen(3000)

// http://localhost:3000/basepath/hi -> 'hi'

Feels free to reopen the issue if this doesn't answer your question.


0.33.0-hf1

Current coverage is 64.58% (diff: 100%)

Merging #1863 into release will decrease coverage by 0.03%

@@            release      #1863   diff @@
==========================================
  Files           704        704
  Lines         50829      50829
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          32848      32828    -20
- Misses        17981      18001    +20
  Partials          0          0

Powered by Codecov. Last update f05321d...98f878c


Credential Verification Failure

Occasionally credential verification can fail with a Server Error (500).
This occurs when the /credential/<Id>/verify endpoint is hit repeatedly.  For example refreshing (Ctrl-F5) https://orgbook-dev.pathfinder.gov.bc.ca/api/credential/100/verify repeatedly will eventually trigger the error.
Controller (API) Error:
ERROR 2020-03-23 14:59:03,873 log 1<PHONE_NUMBER>28256 Internal Server Error: /api/credential/100/verify
Traceback (most recent call last):
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/connection.py", line 157, in _new_conn
    (self._dns_host, self.port), self.timeout, **extra_kw
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/util/connection.py", line 84, in create_connection
    raise err
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/util/connection.py", line 74, in create_connection
    sock.connect(sa)
OSError: [Errno 113] No route to host

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/connectionpool.py", line 672, in urlopen
    chunked=chunked,
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/connectionpool.py", line 387, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/http/client.py", line 1239, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/http/client.py", line 1285, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/http/client.py", line 1234, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/http/client.py", line 1026, in _send_output
    self.send(msg)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/http/client.py", line 964, in send
    self.connect()
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/connection.py", line 184, in connect
    conn = self._new_conn()
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/connection.py", line 169, in _new_conn
    self, "Failed to establish a new connection: %s" % e
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x7fa5cc7ed7f0>: Failed to establish a new connection: [Errno 113] No route to host

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/requests/adapters.py", line 449, in send
    timeout=timeout
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/connectionpool.py", line 720, in urlopen
    method, url, error=e, _pool=self, _stacktrace=sys.exc_info()[2]
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/util/retry.py", line 436, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='agent-indy-cat', port=8024): Max retries exceeded with url: /present-proof/records/c26fd2d4-d4a1-45a5-8b50-44f75756ac8d (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fa5cc7ed7f0>: Failed to establish a new connection: [Errno 113] No route to host',))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/django/core/handlers/exception.py", line 34, in inner
    response = get_response(request)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/django/core/handlers/base.py", line 115, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/django/core/handlers/base.py", line 113, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/django/views/decorators/csrf.py", line 54, in wrapped_view
    return view_func(*args, **kwargs)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/rest_framework/viewsets.py", line 114, in view
    return self.dispatch(request, *args, **kwargs)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/rest_framework/views.py", line 505, in dispatch
    response = self.handle_exception(exc)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/rest_framework/views.py", line 465, in handle_exception
    self.raise_uncaught_exception(exc)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/rest_framework/views.py", line 476, in raise_uncaught_exception
    raise exc
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/rest_framework/views.py", line 502, in dispatch
    response = handler(request, *args, **kwargs)
  File "/home/indy/api/v2/views/rest.py", line 268, in verify
    headers=settings.ADMIN_REQUEST_HEADERS,
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/requests/api.py", line 76, in get
    return request('get', url, params=params, **kwargs)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/requests/api.py", line 61, in request
    return session.request(method=method, url=url, **kwargs)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/requests/sessions.py", line 530, in request
    resp = self.send(prep, **send_kwargs)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/requests/sessions.py", line 643, in send
    r = adapter.send(request, **kwargs)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/requests/adapters.py", line 516, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='agent-indy-cat', port=8024): Max retries exceeded with url: /present-proof/records/c26fd2d4-d4a1-45a5-8b50-44f75756ac8d (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fa5cc7ed7f0>: Failed to establish a new connection: [Errno 113] No route to host',))

Agent Error:
2020-03-23 14:58:22,431 aries_cloudagent.transport.outbound.manager ERROR >>> Posting error: http://api-indy-cat:8080/agentcb/topic/present_proof/;  Re-queue failed message ...

Discovered when investigating; https://github.com/bcgov/TheOrgBook/issues/874

This one is not easy to reproduce.  I've been trying on an off for a while now without success (failure actually).

To me it looks like an error in the environment:
OSError: [Errno 113] No route to host

Looks like this is different from TheOrgBook one.

Yes, this one is completely different.

@ianco, I have to agree, it does look that way.  I was hoping to get some more detailed logging.

Not having any luck reproducing.  Going to close this assuming this was an anomaly.