Automatically warn or convert unformatted URLs in the documentation site

Description

Context: #4173 and #4195
We issued a one-off change to update all links in the DAGs documentation page, but it's likely that we will encounter the issue of unclickable links in documentation in the future.
We should build out either a Sphinx plugin or a Vale linting rule to handle these cases, and either warn when a link is unformatted (e.g. http://{link} instead of the correct <http://{link}>) or automatically convert those links into the formatted version.
@sarayourfriend may be able to provide context on the Vale approach.
For the Sphinx approach, we could use something like our existing extensions which automatically link GitHub usernames or GitHub issues. We would have a similar extension which would rewrite unformatted links to the formatted version in the produced HTML files.
Alternatives

Leave this without a check and occasionally audit the project to ensure all links in the documentation are clickable.

Okay, I wrote a summary of how to accomplish this with Vale, but it occurred to me that it was weird MyST didn't already behave this way to begin with, and sure enough it has a syntax extension that does this for you: https://myst-parser.readthedocs.io/en/latest/syntax/optional.html#linkify
We just need to enable that extension :slightly_smiling_face:

Here's what I originally wrote, just in case it is relevant for some reason, but unless the MyST extension doesn't do what we need, this bit shouldn't matter.
The Sphinx approach is definitely doable, but will incur an cost in our docs build time every time, so I'd rather try other approaches. I won't cover how I'd approach that because (a) it's almost certainly complicated and (b) for me to describe it would require sitting down and probably just doing it a.k.a., exploratory coding. If we do it with Sphinx, I don't think this is a good issue for someone who hasn't already worked on a Sphinx plugin before. Vale is a strong preference for me, it should be much simpler to do it there, if it is possible to do.
For Vale, first check out the documentation on writing Vale styles and the different possibilities: https://vale.sh/docs/topics/styles/
It should be possible to use a regex to find instances of raw links that are not wrapped in angle brackets, something like: \shttps?://.[^>]?(\s|\.). The idea being: match the scheme with whitespace in front and terminating in either whitespace or a period, and allowing anything before the period/space other than > to avoid >. and >  matching; almost certainly doesn't work exactly, but it is where I would start.
I'm guessing that the "conditional" or "existence" style forms are the mostly likely to work for this.
One more thing to keep in mind, is that Vale has different "scopes" for rules, and we'll probably need to choose a more specific scope than the defaults to avoid false positives (specifically inside of code blocks): https://vale.sh/docs/topics/scoping/. IIRC this is why the Openverse/TermCasing style targets the text scope.

I would love to take on this. Currently trying my hands on #3847

Great, thanks @madewithkode, no rush at all on this one :+1:

Hi @sarayourfriend @AetherUnbound resolving this issue is likely to render  the changes made in https://github.com/WordPress/openverse/pull/4195 redundant. What would you advise in respect to that?

Revert those changes in this MR
Leave as is
Revert the changes in a separate dedicated issue

As long as the current syntax works in both cases (the docs site and in Airflow's markdown renderer), I think leaving as-is is fine!

@AetherUnbound I have confirmed it works for the doc site, how do I check for(where do I find) Airflow's markdown renderer?

I looked into the Airflow instance and confirmed that the links are rendered properly. You can verify the same on your local Airflow instance, which can be started by following the catalog quickstart guide.

Thanks @dhruvkb. I have been able to verify this as well.


Optimistically assume that Mac runner will pass

Problem
Currently MacOS takes quite a long time, though for most PRs, if ubuntu passes, then Mac passes too.
Happy Case
We can:

Add ability to add a label to run MacOS on a particular PR
Always run MacOS on masters

The label is due to the fact that certain changes will likely need to be tested with MacOS before merging due to the high likely hood of them failing. For example, if the barretenberg dependency is updated or if CI is updated.
The second ensures that we don't neglect MacOS and coupled with the merge queue, we can amortize the MacOS runner over multiple PRs in the queue since the merge queue can merge together multiple PRs before trying to merge to master.
Alternatives Considered
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
No
Support Needs
No response

You currently can't have required checks for MQ but not for the branch; you'd have to have a run "skip with success" but that makes debugging failures very confusing when they only happen in the MQ.

@guipublic suggested running execute on Mac and prove and verify on linux


Migrate to different hosting

On May 1st Bintray will be sunset (https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter/). Do you already have plans to move sbt-revolver somewhere else?
The brownouts mentioned on that page have already started I think.

Work is already underway to migrate existing artifacts under https://repo.scala-sbt.org to a new place sbt-wide ( https://github.com/sbt/sbt/issues/6294 ).
For new releases, I imagine we would update or release procedure to publish to that new place.

See #112 - relying solelyon repo.scala-sbt.org might cause troubles in the future.

Probably best will be to create a 0.10.0 without any substantial changes and publish that to maven central.

0.10.0 is on its way to Maven Central.


plugins/OWNERS: expand contribex TL alias

Expand the contribex TL alias to get pinged on plugin changes (ref). This way we also avoid creating an OWNERS_ALIAS file. I could not find the use of any other aliases in the existing OWNERS files.
/cc @Priyankasaggu11929 @cblecker
/sig contributor-experience

/lgtm
Thanks @MadhavJivrajani!

/approve


Inaccurate response time?

locust: 0.8a2
MacBook Pro 2016
python 3.6.2

I ran some benchmarks with locust, wrk and ab, and locust always reports an average response time around 300ms, while the other 2 report 4ms. Same server, same settings, same bandwidth.
I assume the BIO nature of requests lib has no effect to the response time, right?
I know locust isn't for benchmark, but how can we trust its result if it can't even get the (remotely) correct stats of a 100B static page? For most APIs in most .com company, this extra 300ms is unacceptable.
wrk --latency -t 8 -c 200 -d 120s --timeout 10s http://<IP_ADDRESS>/nginx_status

Running 2m test @ http://<IP_ADDRESS>/nginx_status
  8 threads and 200 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency    53.94ms  178.14ms   3.28s    94.62%
    Req/Sec     4.37k   531.52     6.98k    69.51%
  Latency Distribution
     50%    4.05ms
     75%    4.86ms
     90%  161.98ms
     99%  741.21ms
  4175822 requests in 2.00m, 1.02GB read
Requests/sec:  34792.40
Transfer/sec:      8.74MB

ab -k -r -n 4000000 -c 200 -s 10 "http://<IP_ADDRESS>/nginx_status"

This is ApacheBench, Version 2.3 <$Revision: 1796539 $>
Copyright 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Licensed to The Apache Software Foundation, http://www.apache.org/

Benchmarking <IP_ADDRESS> (be patient)
Completed 400000 requests
Completed 800000 requests
Completed 1200000 requests
Completed 1600000 requests
Completed 2000000 requests
Completed 2400000 requests
Completed 2800000 requests
Completed 3200000 requests
Completed 3600000 requests
Completed 4000000 requests
Finished 4000000 requests

Server Software:        nginx/1.10.3
Server Hostname:        <IP_ADDRESS>
Server Port:            80

Document Path:          /nginx_status
Document Length:        115 bytes

Concurrency Level:      200
Time taken for tests:   117.530 seconds
Complete requests:      4000000
Failed requests:        9568
   (Connect: 0, Receive: 0, Length: 9568, Exceptions: 0)
Keep-Alive requests:    3960103
Total transferred:<PHONE_NUMBER> bytes
HTML transferred:       460009568 bytes
Requests per second:    34033.92 [#/sec] (mean)
Time per request:       5.876 [ms] (mean)
Time per request:       0.029 [ms] (mean, across all concurrent requests)
Transfer rate:          8772.79 [Kbytes/sec] received

Connection Times (ms)
              min  mean[+/-sd] median   max
Connect:        0    0   0.9      0     216
Processing:     0    6  27.0      4    6547
Waiting:        0    6  27.0      4    6547
Total:          0    6  27.1      4    6547

Percentage of the requests served within a certain time (ms)
  50%      4
  66%      4
  75%      4
  80%      4
  90%      5
  95%      6
  98%      9
  99%     12
 100%   6547 (longest request)

PYTHONPATH=. locust -f tests/test_server.py --no-reset-stats

"Method","Name","# requests","# failures","Median response time","Average response time","Min response time","Max response time","Average Content Size","Requests/s"
"GET","/nginx_status",60595,0,290,288,2,1735,114,424.08
"None","Total",60595,0,290,288,2,1735,114,424.08

"Name","# requests","50%","66%","75%","80%","90%","95%","98%","99%","100%"
"GET /nginx_status",60595,290,320,360,390,470,540,650,750,1735
"None Total",60595,290,320,360,390,470,540,650,750,1735

locustfile:
from locust import TaskSet, HttpLocust, task

class HelloWorld(TaskSet):
    @task
    def foo(self):
        self.client.get('/nginx_status')

class Run(HttpLocust):
    host = 'http://<IP_ADDRESS>'
    min_wait = 0
    max_wait = 0
    stop_timeout = 120

    task_set = HelloWorld

That definitely sounds strange. You should definitely get response times very similar to ab and wrk.

I assume the BIO nature of requests lib has no effect to the response time, right?

Nope, gevent makes it non-blocking.
Is this regardless to how many Locust users you simulate?

@heyman
I ran the same script on a linux server today, here's the result:

NOT using master/slave mode:

num
RPS
avg. RT

1
~275
3ms

20
~620
20+ms

100
~610
100+ms

Seems totally unusable.

Using master/slave mode with 7 slaves on the same server(8-core cpu):

num | RPS | avg. RT
--- | --- | --- | ---
1 | ~275 | 3ms
2 | ~550 | 3ms
3 | ~830 | 3ms
20 | ~3900 | 4ms
50 | ~4100 | 9ms
... | ... | ...
1400 | ~3700 | ~170ms
Seems meaningless using more than 20 coroutines.

And I also tested some real APIs:
API1

jmeter(400 threads): avg ~90ms, rps ~2000
locust(400 users, 7 slaves): avg ~580ms, rps ~500

API2:

jmeter(200 threads): avg ~140ms, rps ~1300
locust(200 users, no slave): avg ~1700ms, rps ~80

CentOS 6.9
python 3.6.3
locust 0.8, using zeromq

settings(both client & target server):

max open files 262144
tcp port range 1024-65535
tcp fin timeout 15s
tw_reuse=1

with --no-web option:
PYTHONPATH=. locust -f tests/test_server.py --no-web --csv=test -c 10 -r 1 -t 120s

 Name                                                          # reqs      # fails     Avg     Min     Max  |  Median   req/s
--------------------------------------------------------------------------------------------------------------------------------------------
 GET /nginx_status                                              67273     0(0.00%)      11       4     296  |      12  621.40
--------------------------------------------------------------------------------------------------------------------------------------------
 Total                                                          67273     0(0.00%)                                     621.40

[2017-10-12 20:02:12,491] xg-build/INFO/locust.main: Time limit reached. Stopping Locust.
[2017-10-12 20:02:12,499] xg-build/INFO/locust.main: Shutting down (exit code 0), bye.
 Name                                                          # reqs      # fails     Avg     Min     Max  |  Median   req/s
--------------------------------------------------------------------------------------------------------------------------------------------
 GET /nginx_status                                              68187     0(0.00%)      11       4     296  |      12  621.30
--------------------------------------------------------------------------------------------------------------------------------------------
 Total                                                          68187     0(0.00%)                                     621.30

Percentage of the requests completed within given times
 Name                                                           # reqs    50%    66%    75%    80%    90%    95%    98%    99%   100%
--------------------------------------------------------------------------------------------------------------------------------------------
 GET /nginx_status                                               68187     12     13     13     14     14     15     16     29    300
--------------------------------------------------------------------------------------------------------------------------------------------
 Total                                                           68187     12     13     13     14     14     15     16     29    300

locust
PYTHONPATH=. locust -f tests/test_server.py --no-web -c 200 -r 10 -t 120s

[2017-10-12 21:11:49,586] xg-build/INFO/locust.main: Time limit reached. Stopping Locust.
[2017-10-12 21:11:49,746] xg-build/INFO/locust.main: Shutting down (exit code 0), bye.
 Name                                                          # reqs      # fails     Avg     Min     Max  |  Median   req/s
--------------------------------------------------------------------------------------------------------------------------------------------
 GET /random_sleep?min=100&max=100                              54186     0(0.00%)     272     125     529  |     270  576.20
--------------------------------------------------------------------------------------------------------------------------------------------
 Total                                                          54186     0(0.00%)                                     576.20

Percentage of the requests completed within given times
 Name                                                           # reqs    50%    66%    75%    80%    90%    95%    98%    99%   100%
--------------------------------------------------------------------------------------------------------------------------------------------
 GET /random_sleep?min=100&max=100                               54186    270    300    310    310    330    350    380    400    530
--------------------------------------------------------------------------------------------------------------------------------------------
 Total                                                           54186    270    300    310    310    330    350    380    400    530

jmeter with 200 threads (avg 131ms, rps 1508)
Label,# Samples,Average,Median,90% Line,95% Line,99% Line,Min,Max,Error %,Throughput,Received KB/sec,Sent KB/sec
HTTP Request,181222,131,130,152,158,189,101,282,0.000%,1508.96359,393.45,434.71
TOTAL,181222,131,130,152,158,189,101,282,0.000%,1508.96359,393.45,434.71

there is nothing actionable in this ticket.. closing.

hello
I have the same issue, I used extra vegeta as another http bench tool and I have the same results as @keithmork. All of them ( I used the same tools and @keithmork ) returns much lower time for very simple app (tornado + hello world) and go lang (same implementation).
Locust returns like 1.5 sec for Simple hello world (?!?). Can You take a look on this ?

yes I know, I have set them to different values most between 1 and 5.
vegeta:
Bucket           #     %       Histogram
[0s,     20ms]   1091  24.24%  ##################
[20ms,   30ms]   2880  64.00%  ################################################
[30ms,   60ms]   489   10.87%  ########
[60ms,   90ms]   18    0.40%
[90ms,   120ms]  14    0.31%
[120ms,  150ms]  5     0.11%
[150ms,  180ms]  0     0.00%
[180ms,  250ms]  3     0.07%
[250ms,  300ms]  0     0.00%
[300ms,  500ms]  0     0.00%
[500ms,  600ms]  0     0.00%
[600ms,  700ms]  0     0.00%

Percentage of the requests completed within given times
 Name                          # reqs    50%    66%    75%    80%    90%    95%    98%    99%   100%
-----------------------------------------------------------------------------------
 GET /ping                 7    630    880   1400   1400   1500   1500   1500   1500   1538
-----------------------------------------------------------------------------------

I tried to do bench marking of the locust response times with enterprise tools
Strangely for the same  requests the response times are huge for Locust and could achieve 180 RPS for the same test where as with Enterprise tool it is around  264 RPS
I tried FasttHTTPUser too
If any one find the same behavior , please comment and if any one resolved this type of issue let me know .  Thanks  in advance

Hi @kishoregorapalli ! There could be a number of reasons for this.
Assuming you didnt run into issues regarding CPU usage (shouldnt be an issue at such low load, but check your logs just in case) one thing that might differ between tools is how connections are handled (typically locust users will reuse connections, and sometimes load balancers/firewalls cause issues with this).
But the first thing you should do is ensure you have enough processes. The easiest thing you can do (assuming you are on latest) is just add --processes -1 to launch one process per core on your machine.
Read more here: https://github.com/locustio/locust/wiki/FAQ#increase-my-request-raterps
If you still have issues, please file a new ticket with complete logs, locust version etc.


Fix broken headings in Markdown files

GitHub changed the way Markdown headings are parsed, so this change fixes it.
See bryant1410/readmesfix for more information.
Tackles bryant1410/readmesfix#1

Hey, thank you!


Documentation for setting Azure notebook storage

What is this PR for?
This PR adds general info and documentation on setting Azure storage in the docs/storage.md folder where we have info about all the supported pluggable storage layers.
What type of PR is it?
Documentation
Todos

[x] - add docs
[x] - change description and order in zeppelin-site.xml.template

What is the Jira issue?
How should this be tested?
Documentation follows the steps in zeppelin-site.xml.template. may need to have account to test.
Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

Thank you @khalidhuseynov 👍
Looks great to me, will merge if there is no more discussion.

@khalidhuseynov Sorry for my late comment. Except that LGTM 👍

thanks for review! @AhyoungRyu then do we need to add link here as well?

@khalidhuseynov Yeah that would be better :)

@khalidhuseynov seems like we need to rebase on latest master and re-trigger CI for this one, before merging.

@bzz rebased from master


Brand new React Boilerplate with Create React App 🎉

Brand new React Boilerplate with Create React App 🎉
Repo: React Boilerplate CRA
Freshly built React Boilerplate on top of the create-react-app. CRA is leading the industry and is a powerfull bootstrapping tool. I(also a maintanier here) decided to join their strengths and make a new boilerplate that  is much easier to maintain and develop, also providing a much stronger/robust base with CRA. Bootstrapping responsibility is now on the CRA and the react-boilerplate takes care of the always promised better architecture and tools
Since its built from zero again there are many nice changes and features.
Major Changes & Updates

New UI 🎉Take a look

Example application comes with much better, tutorial-based web app
Users can check the features out by trying them one-by-one

Redux Toolkit 🎉

Redux Toolkit was on hold for quite a time here on dev.

Documention on Gitbook 🎉 Take a look

Documentation moved to gitbook with many updates and touches

i18next

Replaced react-intl with i18next
Much simpler and react friendly.

Typescript

Boost your DX, scalability, productivity. Typescript is the future for complex projects.
Completely built with ts. Therefore, 100% intellisense support EVERYWHERE🎉
type-safety and compile-time-safety will be your best friend :)
Still want to write js? Hmm, you still can. I'm sure you will quit very soon once you get the taste:)

Minor Changes

Started from scratch. Therefore all the dead, not necessary codes and sections are removed
Switched to styled-components/macro instead of a babel-plugin
Switched to github workflows.

and many more touches...
Version 0.1.0 is published and ready to start new projects. All the opinions and contributions are very welcome. Just getting started, so there might be some bugs and typos and you know.... Any input from you is valuable at this point

@julienben @gretzky @mxstbr
What are your opinions? With the CRA taking care of the half of the work it becomes much easier to maintain for us, meaning less time consuming and better productivity. Currently this repo is very inactive. Everyone is busy lately(lot of issue and pull requests are waiting) and everything is falling behind (packages and configurations). Plus JS is no more maintanable, TS superceeds it undoubtly for me. Everything is just much better with it.

Description Steps to reproduce Versions (to avoid issue-close-app)

Have you upgraded react-redux in this version?

yes its the latest version.

Are you considering a plan to add immerjs as default dependency? @Can-Sahin

Why? Lets continue at the repo issues by the way not here:) If something is missing you can open an issue there

This is great @Can-Sahin - a long time coming. Excellent work. I think a lot of people will really enjoy this.
The only hangup I have here is that this template is in TypeScript, which 'regular' react-boilerplate is not. Don't get me wrong, I prefer TS, but I don't want this to be a blocker for anyone that would find using CRA with rbp helpful, only to find its not a 1 to 1 match if they ever need to cross-reference. Of course don't spend more time than you need/want on conversions! I've been on this project for about 3 years now, and past experience with users tells me there might be a road block or confusion with the mix of TS/JS.

I see but why would anyone need to compare or cross reference to the original one? Its another version and honestly I think its much better(after 2 years of using and contributing to this boilerplate).
It will of course leave things behind and move on. One thing is leaving JS for me (users can still write in js though on top of this). Actually boilerplate's motto is scalability and TS must be the first building block of it. It was discussed here as well favoring TS. So, imo, TS must be the path eventually. Due to the nature of TS it can never be like JS again. It adds many good things and simplifies everything.
The problem with the original one is now:
1 - Its stale. Lot of issue and pull requests waiting for a long time.
2 - Its really hard to maintain. All the bootstrapping thing I realized CRA was doing is quite amazing. This lacks so many stuff and has so many outdated stuff. JS world is very fast and here we are obviously not keeping up with it. So making CRA responsible for those parts is better for us and for users too. I can now focus only on rest and comfortably rely on CRA. Less effort but better quality.
Considering these, to provide a better boilerplate CRA and TS is a definitive choice for me. That's why I did it. I developed like 3 projects using this boilerplate and seen lot of downsides of custom bootstrapping. I am migrating to CRA slowly. For instance it took me 4 hours to realize some dead code in boilerplate prevents tree-shaking. And it was like that for quite a while and everyone has that problem without being aware of it. Who knows how many other things are causing hidden problems like this simply because its very hard to test and maintain properly.
All in all I think this boilerplate provides solid ideas and tools however fails to keep the quality and be active due to the reason i listed above. It would be much better to evolve to keep up the good work.
What do you think @gretzky also @julienben it would be nicer to hear your opinions as well. My plan is to maintain only the CRA version now. I also a have a dream to merge that in here so that react-boilerplate continues to improve. Otherwise I think and currently see its slowly dying.

Like I said before, I don't disagree with these changes. I will say, however, that we all know the frontend world tends to enjoy rearchitecting things at a fast pace without looking back. There isn't anything wrong with that, since TS and CRA are widely used at this point, but IMO it would be unfair to say "CRA and TypeScript are the way now" and move to that without some kind of write up / plan for how to move in that direction so that users know the motives and plans behind this.
The other thing on my mind is that this is a large undertaking, and not that many people are super active in maintaining this these days, myself included. My fear is that 1 person making 2 extremely large changes to a boilerplate that's widely used will not be sustainable.
tl;dr I don't disagree with these changes / moving in this direction, but I don't want to be a part of the frontend stereotype of "rearchitecting for the sake of rearchitecting and leave everything else behind". I don't find that sustainable or inclusive.

Speaking just on the technical side, here are some tradeoffs I can see potentially arising.

Being beholden to changes made in CRA. This may just get annoying after awhile trying to keep up, but we'd have to.
Having to filter issues that are CRA-related, or actual rbp-related. Again, just a potential annoyance, but I'm guessing we may get issues that have to do with CRA and not rbp.

A general tradeoff is "how does this effect users who modify this boilerplate to fit their projects needs?" When things work "magically" it's great, but it isn't one-size-fits-all as a lot of people like to think.
Again these are just potential tradeoffs to consider.

Definitely understanding what you mean by undertaking. Don't want to make big jumps on a long running repo. Thats why I opened another repo instead of changing the typescript version of this repo (which is being used by others for 1+ year now). If its gonna happen it must happen slowly in time and with everyones mutual agreement.
But the problem is like you have said. This is becoming more stale (nobody maintaining actively) and its indeed hard to maintain. My point is to drive to a direction (in this case to CRA) instead of standing still and disappointing people. So many pull requests and issues and probably so many bugs are awaiting.
This boilerplate is not updatable anyway not like a npm package. Everyone cloning is starting from scratch probably. So every new version is mostly for new comers.  Rearchitecting can help everyone. Nothing is a breaking change in that sense.
All my idea was lets do something so that maintaining becomes easier for us and we can provide better tools and practices for everyone instead of drowning in bootstrapping problems which CRA does a pretty good job at. Everyone benefits.

I already have 2 CRA related issue open ahah :)
But its kinda good for me. Gets me in track with CRA.
CRA has eject option. You can edit everything if you want. After that you are all alone and you wont get updates. Thats upto users who want to customize their projects. And for those who are fine with default settings (which covers majority) they get updates and be much more sustainable.

I think we're on the same page here @Can-Sahin, great work again. I always wanted to bootstrap rbp to CRA, thank you for bringing that to life.

:) Lets see how its gonna turn out. Getting issues slowly :)
You know I just don't want to watch rpb dying slowly due to inactivity and be outdated. Still I find the toolset and practices very valuable and wanna keep it alive. All this effort and comments originates from this wish :) This is simply my solution and suggestion. Just wanted to discuss

@gretzky I tried to contact julien (2 emails) and no response. So I am guessing he isn't in the game anymore?? There hasn't been any sort of activity last 6 months and no release over a year. I am quitting to look at the pull requests and issue since I have no relation or time left to this version. If you think that carrying the CRA version over could be a solution then I can work on it. If not, then for me it's flatline :(

@Can-Sahin whilst I'm not able (anymore) to dedicate much, if any, time to helping on this project, I feel moving the project to leveraging CRA is inline with @mxstbr intent for where this project would inevitably go. I have (nor am I sure if I'll get time) not yet looked at the work you've done, but wanted to express my understanding of relayed wishes for the project's trajectory, i.e. to go toward leveraging/integrating with CRA.

Does this mean new projects should be started with React Boilerplate CRA and not this repo?

@jwinn said it best, I agree.

@joeyparis I'd say it would be up to your discretion for now, take a look around both repos and see which would fit your projects needs.

@joeyparis I am totally new to React Boilerplate, too. From my first impression I would go with React Boilerplate CRA. With the exception you can't stand any TypeScript. But you can use JavaScript if you want.

So @gretzky  @jwinn  should I work towards that and release v5? Ofc with lot of descriptions etc explaining and giving a quick walkthrough on changes etc.
Since this is JS and CRA version is TS, I am also thinking of making the javascript version of the app/components and containers folders to set an example for the people still wanting use JS.

@Can-Sahin I'm of the opinion that--especially with JS version mentioned--seems the correct path for v5 of the bp. I'll try to get some time to review your PR/submission.

Well the two repos are extremely different now and I cannot merge anything. Plus I already implemented some of the pull requests waiting here in my repo. So my idea is to merge some of the pull requests first(just to honor the people, because their suggested changes are already in my repo) then truncate the entire repo and finally transfer my repos commits entirely. I have to transfer the repo here(its built from scratch) and I guess this is the way that I can keep the contributions as well.
You can just review the other repo whenever you have time to:)

Now that I have some time...
1- I am gonna merge some pull request just to honor them as said. They are already implemented in CRA version
2- Create another branch and then truncate the entire repo. Then, will pull from my remote the whole git history.
3- Make the adjustments in readme etc and add some documentation on the update.
4- Submit a PR.

Hi guys
Just a suggestion: you should add a prominent message to the top of your readme explaining this repo is deprecated and pointing to the CRA repo. At first glance the project seems abandoned. It's only by chance that I checked the issues.
This is also the only repo that appears when Googling "React app boilerplate" (the CRA one doesn't)


Removing superfluous HTTP requests in tests of local setters/getters.

Fixes #778.

Coverage remained the same at 100.0% when pulling 0d43bf31ced83d6883af495cf13643993c89ade8 on dhermes:fix-778 into 6283ab9c9d7fa06f6800079bd882b573c4de7cf7 on GoogleCloudPlatform:master.

LGTM


Installed loadcaffe, but cannot run

I have this error while running a code using loadcaffe.
The file libloadcaffe.so is at that location, but I don't understand why it wasn't able to load that library.
/home/phong/torch/install/bin/lua: /home/phong/torch/install/share/lua/5.2/trepl/init.lua:389: /home/phong/torch/install/share/lua/5.2/loadcaffe/ffi.lua:10: could not load library /home/phong/torch/install/lib/lua/5.2/libloadcaffe.so
stack traceback:
[C]: in function 'error'
/home/phong/torch/install/share/lua/5.2/trepl/init.lua:389: in function 'require'
test.lua:1: in main chunk
[C]: in function 'dofile'
...hong/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:150: in main chunk
[C]: in ?

Did you solve this problem?


Create code coverage workflow

Can create a badge afterwards.
Links:

related stackoverflow : https://stackoverflow.com/questions/71442449/how-to-access-github-action-output-in-a-badge
shields io dynamic badge from json : https://shields.io/badges/dynamic-json-badge
How to get a stable URL for the JSON artifact: https://nightly.link/
Output dynamic badge to a gist page : https://nedbatchelder.com/blog/202209/making_a_coverage_badge.html
Upload badge as artifact : https://github.com/emibcn/badge-action/blob/master/.github/workflows/test.yml#L44


[C#][Facebook] Persistent Menu with Global Actions

Hello,
Can anyone tell me where can I find some example on how to use new v3.1 Named Dialog Actions feature to implement global actions like help, resulting in a Facebook Persistent Menu?
Thanks,

I don't think BotBuilder has support for that within the library.
I just POSTed the JSON according to Facebook's documentation. For the payload, I used something like "PAYLOAD_DELETE_DATA".
On the SDK, I used a Middleware to listen for incoming payload messages and treated the result/fired up the correct Dialogue.

 bot.use({
        botbuilder: function (session, next) {
            if (PAYLOAD_DELETE_DATA === session.message.text) {
                   session.beginDialog("/delete");
            }
            ...

I don't think BotBuilder has support for that within the library.

Oh, but it does! V3.1 release notes say: Form Flow for automatically generating a Bot from a C# class for filling in the class and that supports help, navigation, clarification and confirmation. I think the examples are just not yet available.

You should be able to use ChannelData to send the native blob to Facebook - for example, see https://github.com/Microsoft/BotBuilder/issues/1077.  @chrimc62 is there any Facebook persistent menu support in FormFlow?

There is no Facebook persistent menu support in FormFlow directly, but you can override the IFormBuilder.Prompter with your own implementation of PromptAsyncDelegate which can do any mapping you like including to Facebook persistent dialogs.


Add links on the author info

Move note links to prominent parts instead of "and more links"
Add zenn link under twitter link
Add a link to the old blog and the old old blog in "and more links"

https://github.com/falgon/roki-web/pull/370


Invalid host defined options error in http crawler when used with pkg post ESM edition

Which package is this bug report for? If unsure which one to select, leave blank
@crawlee/http (HttpCrawler)
Issue description
Error
makdi-test  | DEBUG HttpCrawler:SessionPool: No 'persistStateKeyValueStoreId' options specified, this session pool's data has been saved in the KeyValueStore with the id: ce096038-12bd-4473-ab69-d0e0fc741228
makdi-test  | INFO  HttpCrawler: Starting the crawler.
makdi-test  | DEBUG HttpCrawler:AutoscaledPool:Snapshotter: Setting max memory of this run to 1963 MB. Use the CRAWLEE_MEMORY_MBYTES or CRAWLEE_AVAILABLE_MEMORY_RATIO environment variable to override it.
makdi-test  | DEBUG HttpCrawler:SessionPool: Created new Session - session_MrRYxu57S9
makdi-test  | WARN  HttpCrawler: Reclaiming failed request back to the list or queue. TypeError: Invalid host defined options
makdi-test  |     at HttpCrawler._requestFunction (/snapshot/makdi/node_modules/@crawlee/http/internals/http-crawler.js:423:13)
makdi-test  |     at /snapshot/makdi/node_modules/@crawlee/http/internals/http-crawler.js:366:94
makdi-test  |     at wrap (/snapshot/makdi/node_modules/@apify/timeout/index.js:52:27)
makdi-test  |     at /snapshot/makdi/node_modules/@apify/timeout/index.js:66:7
makdi-test  |     at AsyncLocalStorage.run (node:async_hooks:319:14)
makdi-test  |     at /snapshot/makdi/node_modules/@apify/timeout/index.js:65:13
makdi-test  |     at new Promise (<anonymous>)
makdi-test  |     at addTimeoutToPromise (/snapshot/makdi/node_modules/@apify/timeout/index.js:59:10)
makdi-test  |     at HttpCrawler._handleNavigation (/snapshot/makdi/node_modules/@crawlee/http/internals/http-crawler.js:366:76)
makdi-test  |     at async HttpCrawler._runRequestHandler (/snapshot/makdi/node_modules/@crawlee/http/internals/http-crawler.js:309:13) {"id":"3FULW9cbbkMrV4R","url":"https://crawlee.dev","retryCount":1}
makdi-test  | DEBUG HttpCrawler:SessionPool: Created new Session - session_jF6xxRoKff
makdi-test  | WARN  HttpCrawler: Reclaiming failed request back to the list or queue. TypeError: Invalid host defined options
makdi-test  |     at HttpCrawler._requestFunction (/snapshot/makdi/node_modules/@crawlee/http/internals/http-crawler.js:423:13)
makdi-test  |     at /snapshot/makdi/node_modules/@crawlee/http/internals/http-crawler.js:366:94
makdi-test  |     at wrap (/snapshot/makdi/node_modules/@apify/timeout/index.js:52:27)
makdi-test  |     at /snapshot/makdi/node_modules/@apify/timeout/index.js:66:7
makdi-test  |     at AsyncLocalStorage.run (node:async_hooks:319:14)
makdi-test  |     at /snapshot/makdi/node_modules/@apify/timeout/index.js:65:13
makdi-test  |     at new Promise (<anonymous>)
makdi-test  |     at addTimeoutToPromise (/snapshot/makdi/node_modules/@apify/timeout/index.js:59:10)
makdi-test  |     at HttpCrawler._handleNavigation (/snapshot/makdi/node_modules/@crawlee/http/internals/http-crawler.js:366:76)
makdi-test  |     at async HttpCrawler._runRequestHandler (/snapshot/makdi/node_modules/@crawlee/http/internals/http-crawler.js:309:13) {"id":"3FULW9cbbkMrV4R","url":"https://crawlee.dev","retryCount":2}
makdi-test  | DEBUG HttpCrawler:SessionPool: Created new Session - session_0RMbGUTi5D
makdi-test  | WARN  HttpCrawler: Reclaiming failed request back to the list or queue. TypeError: Invalid host defined options
makdi-test  |     at HttpCrawler._requestFunction (/snapshot/makdi/node_modules/@crawlee/http/internals/http-crawler.js:423:13)
makdi-test  |     at /snapshot/makdi/node_modules/@crawlee/http/internals/http-crawler.js:366:94
makdi-test  |     at wrap (/snapshot/makdi/node_modules/@apify/timeout/index.js:52:27)
makdi-test  |     at /snapshot/makdi/node_modules/@apify/timeout/index.js:66:7
makdi-test  |     at AsyncLocalStorage.run (node:async_hooks:319:14)
makdi-test  |     at /snapshot/makdi/node_modules/@apify/timeout/index.js:65:13
makdi-test  |     at new Promise (<anonymous>)
makdi-test  |     at addTimeoutToPromise (/snapshot/makdi/node_modules/@apify/timeout/index.js:59:10)
makdi-test  |     at HttpCrawler._handleNavigation (/snapshot/makdi/node_modules/@crawlee/http/internals/http-crawler.js:366:76)
makdi-test  |     at async HttpCrawler._runRequestHandler (/snapshot/makdi/node_modules/@crawlee/http/internals/http-crawler.js:309:13) {"id":"3FULW9cbbkMrV4R","url":"https://crawlee.dev","retryCount":3}

Works fine with<EMAIL_ADDRESS>(before ESM migration), fails since<EMAIL_ADDRESS>Refer Dockerfile for reference https://github.com/teammakdi/makdi-test/blob/main/Dockerfile#L14
Code sample
https://github.com/teammakdi/makdi-test
Package version
3.6.2
Node.js version
18.17.0
Operating system
Ubuntu
Apify platform

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

I have tested this on the next release
No response
Other context
No response

Hey, I'm unable to reproduce the issue you're having when running the code in your example repository. Could you confirm if you can consistently reproduce this issue? Does it happen in docker?

Hey @vladfrangu
Yes, it only fails via docker. Works fine if run directly on host machine via node.
Also in docker, it fails when used with https://github.com/vercel/pkg
You can checkout the test project and run it via docker
docker-compose up --build

Is there any reasons you use pkg in docker? 👀
Could you also try to reproduce this without pkg?

Hey, it is reproducible only with pkg tool.
We have a requirement to convert our app to binary hence using the same.
It however works with<EMAIL_ADDRESS>not with<EMAIL_ADDRESS>


Implemented New Home Page

This PR is to incorporate the new home page designs with the new countdown. Upcoming events have been removed, but I will try to find a way to add these back in if the general consensus is to keep upcoming events. Note that the log out button is now located on the top left corner of the home page because users will not have their own QR code popup anymore.
iPhone 11 Max Pro:

iPhone SE:

Upcoming events have been added back to the home page UI.
iPhone 11 Max Pro:

iPhone SE:


agenda support?

I can not use agenda with type="jalaali" and it is english. Is agenda supported?

@h4m3d66 That is why I have not updated the description. There are still some kinks to figure out. You are welcome to create a Pull Request.

@h4m3d66 That is why I have not updated the description. There are still some kinks to figure out. You are welcome to create a Pull Request.

@h4m3d66 That is why I have not updated the description. There are still some kinks to figure out. You are welcome to create a Pull Request.

@h4m3d66 That is why I have not updated the description. There are still some kinks to figure out. You are welcome to create a Pull Request.


Error trying to build on Mac

Hey,
I'm on a Mac High Sierra
I've got all dependencies (including openssl installed through brew) but when I run "cmake .." on the build folder I get the following error:
-- Checking for module 'eet>=1.18.0'
--   Package 'openssl', required by 'emile', not found
CMake Error at /usr/local/Cellar/cmake/3.11.4/share/cmake/Modules/FindPkgConfig.cmake:418 (message):
A required package was not found
Call Stack (most recent call first):
/usr/local/Cellar/cmake/3.11.4/share/cmake/Modules/FindPkgConfig.cmake:585 (_pkg_check_modules_internal)
CMakeLists.txt:72 (pkg_check_modules)
-- Configuring incomplete, errors occurred!
See also "/Users/lcassa/Downloads/war2edit-master/build/CMakeFiles/CMakeOutput.log".
See also "/Users/lcassa/Downloads/war2edit-master/build/CMakeFiles/CMakeError.log".

Hey,
mh... I'm not quite sure. I've never had the chance to try High Sierra. I know that macOS does not like openssl, and unfortunately it is a strong dependency of the EFL (which is the GUI framework war2edit is built with).
Maybe pkg-config is not able to find an openssl.pc file in the PKG_CONFIG_PATH. Can you please run the following:
pkg-config --modversion openssl

If it returns something like
Package openssa was not found in the pkg-config search path.
Perhaps you should add the directory containing `openssa.pc'
to the PKG_CONFIG_PATH environment variable
No package 'openssl' found

You should try to locate where homebrew installs openssl.pc and add it to the PKG_CONFIG_FILE environment variable (assuming a POSIX-compatible shell):
export PKG_CONFIG_PATH="$PKG_CONFIG_PATH:<path/to/dir/containing/openssl.pc>"

Ok, I figured it out
I ran:
brew info openssl
found this path:
"For pkg-config to find this software you may need to set:
PKG_CONFIG_PATH: /usr/local/opt/openssl/lib/pkgconfig"
exported this way:
export PKG_CONFIG_PATH="/usr/local/lib/pkgconfig:/usr/local/opt/openssl/lib/pkgconfig"
then I hit this issue:
-- Checking for module 'liblzma'
--   No package 'liblzma' found
so I ran this command:
brew install xz
and the cmake finished successfully 😄
thanks for your help @jeanguyomarch !


Process failed when submitting URL

Traceback (most recent call last):
File "./process.py", line 124, in
main()
File "./process.py", line 115, in main
autoprocess(parallel=args.parallel)
File "./process.py", line 83, in autoprocess
sample = db.view_sample(task.sample_id)
File "/home/tmirt/tools/cuckoo/utils/../lib/cuckoo/common/utils.py", line 267, in inner
return f(self, *args, **kwargs)
File "/home/tmirt/tools/cuckoo/utils/../lib/cuckoo/core/database.py", line 1105, in view_sample
sample = session.query(Sample).get(sample_id)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/query.py", line 818, in get
if len(ident) != len(mapper.primary_key):
TypeError: object of type 'NoneType' has no len()
Test Case:

Submit case by url:
eg.:
https://www.virustotal.com/intelligence/download/?sha256=de633994b6217e2d5dfe38d48a6caf0b248d4fe0f60812953052841cef77ae61

Process result should be turn off (process result on a new thread)

Just found out:
Upon checking tasks db.
ID     | Category| Status | Samle Id
| 107 | url      | completed |      NULL |
| 108 | url      | completed |      NULL |
this will produce exception.

Confirmed bug, thank you very much for reporting. I am going to fix it.


Server codegen and "target must reside inside a location in the $GOPATH/src"

Problem statement
go-swagger complains target not in $GOPATH when, I think, it is ...
$ echo $GOPATH
/home/sandy/go

$ pwd
/home/sandy/go/src/my_org/my_project

$ alias swagger="docker run --rm -it -v $HOME:$HOME -w $(pwd) quay.io/goswagger/swagger"
$ alias
...
alias swagger='docker run --rm -it -v /home/sandy:/home/sandy -w /home/sandy/go/src/my_org/my_project quay.io/goswagger/swagger'

$ ls -la subdir/spec/
...
-rw-rw-r-- 1 sandy sandy 10116 Oct 31 15:26 project.swagger.yaml

$  swagger generate server -f subdir/spec/mosaic.swagger.yaml -t output
2016/11/24 17:16:22 building a plan for generation
2016/11/24 17:16:22 target must reside inside a location in the $GOPATH/src

But ... but ... it is?

Can you change your alias a little?
I think either of these should work:
alias swagger='docker run --rm -it -v /home/sandy:/root -v /home/sandy/go:/go -w /go/src/my_org/my_project quay.io/goswagger/swagger'

or
alias swagger='docker run --rm -it -e GOPATH=/home/sandy/go:/go -v /home/sandy:/home/sandy -w /home/sandy/go/src/my_org/my_project quay.io/goswagger/swagger'

The explanation is that while on your local machine the project definitely resides in the go path, but the go path in the docker container is at /go not at your home directory.
There are also a  few items pre-installed in the gopath of the container you're using so that also makes it so you can't fully overwrite the go path with yours.
It is possible though but then you have to make sure you have these packages in your local go path (and compiled for linux not another OS) https://github.com/go-swagger/go-swagger/blob/master/Dockerfile#L5-L10

Awesome ... works like a charm.
Thanks @casualjim!


Issue 379 origin should include scheme

Regarding #379
I've added checks against schemes so that different schemes are not allowed. If scheme is not provided, request is processed, but deprecation warning is raised.
I am not sure whether checking on request processing is best idea, an alternative would be to check it on project setup, I am not sure how this would work with CORS defined in model.
I have also skipped checking regex, since user might deliberately avoid passing scheme.
I am open for suggestions.

Great, I was facing the same issue.
I think the is no reasons to check on request processing.
One more idea is to process passed host on project setup and create all available  variants
eventually, we will receive something like this
input:
    CORS_ORIGIN_WHITELIST = (
         'google.com'
     )

array to use in each response:
    [
       'http://google.com', 'https://google.com'
    ]

Also, a warning message might be in the project set up.
Waiting for merge

Since this is a security feature I would like to avoid any behavior that is not immediately obvious and might be subject to some sort of missclicks. I'd rather make user explicitly define schemes they are OK with than risk having a situation when they wanted to allow only one of them, but forgot to change some settings and got loosened security policy.
I'll implement checks on startup though.

Yes, you are right. I mean to apply this flow for old defined configs.
I this case will be warning that this is deprecated and won't work in feature releases. but this host will be apllied for two schemas.

Hi @wgonczaronek ,
This is a great start, thank you very much. Sorry I've been slow to respond, it's been a busy season with Djagnocon Europe.
I've pushed a few cosmetic changes, but I still have some more things to think through. Parsing the list every time seems unnecessary, and the model is adding a lot of confusion to the code. @melnychukvitaliy 's suggestion to automatically update old configuration to new is a good one as it would reduce the amount of edge cases in the code but it's impossible to know all the schemes that users were previously working with, there will be edge cases such as onion:// and ftp://.
The model is also making things awkward here. I've been tempted to drop the model for a long time, since I doubt many users have used it (no one had any problems after making it abstract in 2.0.0). It was added in #27 without any discussion and has been in many users' databases since.
Also the code has some weird branches for supporting scheme-less origins. parsed_origin.netloc or parsed_origin.path is especially confusing until you understand that urlparse without a scheme puts everything in path. This makes me tempted to go ahead without a deprecation perio, and instead force a major version release to make everyone add schemes in their settings. In my experience deprecation warnings are ignored 80% of the time, and we want to help users be secure here.
I'm still forming a plan. I'll probably merge this as-is as part of it.
Thanks for pushing on this!
Adam

Hi, I have missed the notice that you commented on the issue.
I agree with the problem with schemes that could be used. There are plenty of them and I can imagine a situation where some custom ones are used.
I also agree with the issues with model. This was a tricky part for me to check it. Same applies to urlparse. Unfortunately handling both urls with and without schema was problematic. Forcing users to use schema would greatly simplify this code.
Let me know if you'd like me to implement some further changes. I am happy to help.

Oh, thanks for the reply. I merged it because I'm working on it right now, first removing the model, then moving to non-deprecation path. Don't worry about it, thanks for your contribution!


RELATED: SD-970 fix type-guard

Currently destined for release R208.
This release is zero downtime release.
Code drop: 2020-07-02 00:00.

Story SD-970 Control filters - Low level message handler
Status: Hacking
Priority: Major
Assignee: Lan Huynh

Build succeeded (check pipeline).

https://checklist.intgdc.com/job/client-libs/job/gooddata-typings-pull-request-compliance-check/114/ : SUCCESS in 8s
https://checklist.intgdc.com/job/client-libs/job/gooddata-typings-unit-tests-zuul-docker-el7/197/ : SUCCESS in 50s

LGTM

Build succeeded (gate pipeline).

https://checklist.intgdc.com/job/client-libs/job/gooddata-typings-pull-request-compliance-gate/41/ : SUCCESS in 5s
https://checklist.intgdc.com/job/client-libs/job/gooddata-typings-unit-tests-zuul-docker-el7/200/ : SUCCESS in 29s
https://checklist.intgdc.com/job/compliance/job/gooddata-typings-fossa-scanning-zuul-el7/5/ : SUCCESS in 15s (non-voting)


Store subcharts inside the kubecf repo

This will make it possible to inspect changes during version bumps using regular git tools and also allows for text search against the charts.
Fixes #987

Please review the 2 commits separately; only https://github.com/cloudfoundry-incubator/kubecf/commit/fee86a4 contains manual changes, and except for deletion of a single line of code and the removal of .gitignore is solely documentation update.
The second commit then executes the commands documented in the first commit to add the expanded charts to this repo.
The most important part of the review would be to double-check that these commands does cover all possible scenarios in the future: add/removed/updated subcharts, add/removed/changed files belonging to one of the subcharts.
Tagging @mook-as for review because he has commented on #987. "No good deed shall go unpunished!"

so this should work.

I've tested that this change works before making the PR. 😄
And for the test it deployed with Eirini, so I was worried that using expanded charts were somehow incompatible with conditions in requirements.yaml. But then I realized that it was a proper Eirini deployment, with no diego-cell pods, and eventually found the hidden config file in my tree that had features.eirini.enabled: true.


Added propcore.

Let me know if anything is wrong. I've tested it and it works.

Codestyle. First thing you should check when submitting a PR, it's all on the readme.

Also @thegrb93, The reason issue #19 was Entity/Propcore was because Propcore is a special case of an entity spawning system. This should really be done by users specifying prop_physics as the entity.
But as previously mentioned, it's a permissions nightmare and open to significant abuse. If something can be written by the users, then it should be. We're not about adding convenience everywhere when they have all the tools to implement it. ( In this case that would be through the yet-to-be-completed entity core ).
Can @Xandaros chime in on this, just to confirm or reject any statements. Want to make sure we're on the same page.

Mind putting comments where my mistakes are? I don't see any.

Also why over-complicate when all you need is prop_physics?

Removed the duplicator functions. I was just working off of Garry's MakeProp code.

Mind putting comments where my mistakes are? I don't see any.

Codestyle everywhere, almost all function declarations, table indexing etc. It might just be easy for me to spot because we've been doing it for so long. Make sure you carefully look at the codestyle guidelines and the additions in your commit.
Why over-complicate when all you need is prop_physics?

The Entity core is still needed, and props are a sub-set of entities; Specifically, prop_physics. All props are entities, but not all entities are props. The 'whitelist' you mention is the entire permissions system, and it very much does make sense. Want admins to spawn some entities, but not regular players: permissions. Want to give general players the ability to spawn custom entities you've added with starfall: permissions.
What doesn't make sense is implementing a prop core system which is pretty much copy/paste of a single-use case of the larger entity system. This is one of the reasons that this system has been on hold, as it requires careful design decisions. We really wish to avoid this 'idea' that convenience is 'key' to the users, and we've opted for the more 'proper' implementation of a system. The tools are provided for the users, therefore they can make these systems if they don't require specific interfacing with GLua. There are some exceptions to this rule, but generally it holds.
Please wait for @Xandaros to comment on this, he might be able to explain our stance a bit more clearly than I can.

The coding style is the same as the hologram library so you might want to fix the hologram library's style.

See #7. Changing an existing codebase over to new codestyle guidelines is 'difficult' work. You were told multiple times what guidelines to follow and where to find them. Those guidelines hold for any PR, including those by @Xandaros and myself. As we change sections of the codebase we oftentimes fix codestyle of surrounding sections as we go, thus eventually ensuring all code conforms to the guidelines.
Now please, this is the last time I will tell you, look at the codestyle guidelines in the readme and follow them. Your PR will not be accepted if it does not conform to the stated guidelines.

I don't care. Doesn't affect my repository any. Take it or leave it, but if you want to get uppity about it I won't mind keeping all my additions to myself and my server.

omg ^^

propent:CallOnRemove( "starfall_propgram_delete", propOnDestroy, propdata, instance.player )

Clear evidence of find/replace mistake, therefore evidence of copy/paste. Closing this PR as lack of brain cells.
@thegrb93 It's very ignorant and brazen to brandish work, claiming it to be your own, to an upstream repository without checking it, showing resistance to PR guidelines when stated, and providing code which is a blatant copy-paste of our existing work. Yes the systems are similar, but copy-pasting without understanding why breeds bad coders. I will not help reinforce a model that promotes plagarism and lack of originality and application of knowledge.

It's funny you think reusing code (in the same project no less) is considered plagiarism. I copy pasted it to save time and got the task done. Something you still haven't even started after a year. Thanks for spotting that bug, I'll fix it.

We'd rather have missing features than unmaintainable code.
Granted, our standards are quite high and the original codebase still needs to be put up to standard, but new features should follow the guidelines to make it easier to maintain the code and have a common style across the whole project.
Fixing the code style shouldn't be too much of an issue, as most of the copied code is already in the correct style, but there are still a few places that don't follow it.
You might also want to either squash the commits or change the commit messages to more meaningful names. "Added permissions"; for example, makes sense in the context of the PR, but once it is merged in, this context is lost and from the message alone, you might think that it adds the whole permission system.
As for copying from the holograms library, that's fine, but you might want to mention it in the commit message. Not doing so is indeed plagiarism. (Yes, it is open source, but if you read the license you'd see that you are required to credit the respective author. Since it is in the same project, just saying where you got it from is sufficient, since authorship is implicit in the git history)


Merge azure account provider and azurecore extensions

The main reason to merge these extensions is to reduce the overhead of having duplicate node_modules directories.  This change removes ~2000 files from the packaged build.

@kevcunnane I'll see if we can revisit after we pick up the next VS Code merge.  Ideally these should be different extensions, but since they are both azure-related I think it's acceptable.  Also, we should see who owns the ADAL node module and try to remove the dependency to lodash which is bringing over 1000 files itself..though perhaps the VS Code merge will help address that problem.


[FormRecognizer] RecognizeCustomFormsFromUri failing in nightly runs

FormRecognizer nightly test runs are failing with:

Error message
Azure.RequestFailedException : Invalid model created with ID 0931469c-88d4-4aaf-85ad-5451173b60a7
Status: 200 (OK)
ErrorCode: 3013
Additional Information:
error-0: 3013: Operation exceeded maximum processing time.
Content:
Headers:
ms-azure-ai-errorcode: REDACTED
x-envoy-upstream-service-time: 28
apim-request-id: ef815367-30fb-4144-847c-ef49685af2a0
Strict-Transport-Security: max-age=31536000; includeSubDomains; preload
X-Content-Type-Options: nosniff
x-ms-region: REDACTED
Date: Fri, 21 Apr 2023 05:13:41 GMT
Content-Length: 291
Content-Type: application/json; charset=utf-8
Data:
error-0: 3013: Operation exceeded maximum processing time.
Stack trace
at Azure.Core.OperationInternal1.GetResponseFromState(OperationState1 state) in /mnt/vss/_work/1/s/sdk/core/Azure.Core/src/Shared/OperationInternalOfT.cs:line 304
at Azure.Core.OperationInternal1.UpdateStatusAsync(Boolean async, CancellationToken cancellationToken)    at Azure.Core.OperationInternalBase.UpdateStatusAsync(CancellationToken cancellationToken) in /mnt/vss/_work/1/s/sdk/core/Azure.Core/src/Shared/OperationInternalBase.cs:line 87    at Azure.Core.OperationPoller.WaitForCompletionAsync(Boolean async, OperationInternalBase operation, Nullable1 delayHint, CancellationToken cancellationToken) in /mnt/vss/_work/1/s/sdk/core/Azure.Core/src/Shared/OperationPoller.cs:line 85
at Azure.Core.OperationInternalBase.WaitForCompletionResponseAsync(Boolean async, Nullable1 pollingInterval, String scopeName, CancellationToken cancellationToken) in /mnt/vss/_work/1/s/sdk/core/Azure.Core/src/Shared/OperationInternalBase.cs:line 202    at Azure.Core.OperationInternal1.WaitForCompletionAsync(Boolean async, Nullable1 pollingInterval, CancellationToken cancellationToken) in /mnt/vss/_work/1/s/sdk/core/Azure.Core/src/Shared/OperationInternalOfT.cs:line 257    at Azure.Core.OperationInternal1.WaitForCompletionAsync(CancellationToken cancellationToken) in /mnt/vss/_work/1/s/sdk/core/Azure.Core/src/Shared/OperationInternalOfT.cs:line 196
at Azure.AI.FormRecognizer.Training.CreateCustomFormModelOperation.WaitForCompletionAsync(CancellationToken cancellationToken) in /mnt/vss/_work/1/s/sdk/formrecognizer/Azure.AI.FormRecognizer/src/FormTrainingClient/CreateCustomFormModelOperation.cs:line 73
at Azure.AI.FormRecognizer.OperationExtensions.WaitForCompletionAsync(Task1 operation, CancellationToken cancellationToken) in /mnt/vss/_work/1/s/sdk/formrecognizer/Azure.AI.FormRecognizer/src/OperationExtensions.cs:line 92    at Azure.AI.FormRecognizer.Samples.FormRecognizerSamples.RecognizeCustomFormsFromUri() in /mnt/vss/_work/1/s/sdk/formrecognizer/Azure.AI.FormRecognizer/tests/samples/V3.1/Sample2_RecognizeCustomFormsFromUri.cs:line 28    at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter1.BlockUntilCompleted()
at NUnit.Framework.Internal.MessagePumpStrategy.NoMessagePumpStrategy.WaitForCompletion(AwaitAdapter awaiter)
at NUnit.Framework.Internal.AsyncToSyncAdapter.Await(Func`1 invoke)
at NUnit.Framework.Internal.Commands.TestMethodCommand.RunTestMethod(TestExecutionContext context)
at NUnit.Framework.Internal.Commands.TestMethodCommand.Execute(TestExecutionContext context)
at NUnit.Framework.Internal.Commands.BeforeAndAfterTestCommand.<>c__DisplayClass1_0.<Execute>b__0()
at NUnit.Framework.Internal.Commands.BeforeAndAfterTestCommand.RunTestMethodInThreadAbortSafeZone(TestExecutionContext context, Action action)

For more details check here:

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2717598&view=results

@jsquire for notification.

Already tracked by https://github.com/Azure/azure-sdk-for-net/issues/31914.


Change theme color from light to dark

Good morning to all, you can guide me how to change the code so that the context menus appear white with black letters and not a dark gray background.
I would thank you a lot!

It's totally up to the developer, you can add the button that switches between two different style schemes. In webview mode everything can be done on web-end.

can you help me? please!

see, for web end, you need to alter CSS and on apps end, we have our color scheme in colors.xml and to make your app able to switch between both themes, here's a trick; just add more colors according to your dark scheme to the same xml file and save switches as cookies that will help you on both web and app end.
I guess there's nothing more to it and it's been a while, I'm closing this issue, if you feel like asking more questions, be free to dude.


How do I use an img src in Carousel Items?

The Carousel example in the reacstrap docs uses 'data:image' for img src. How do I use 'img'?

components: Carousel
reactstrap version 0.31.5
import method umd/csj/es (if umd, are you using the "full" version (only for v5+)?)
react version 16.1.0
bootstrap version 4.0.0-beta

I am sure it is a silly question, but the need overrides the shame :-)
The example in the doc is using the following:
<CarouselItem
          onExiting={this.onExiting}
          onExited={this.onExited}
          key={item.src}
          src={item.src}
          style={imgStyle}
          altText={item.altText}
        >

Where src is:
src: 'data:image/svg+xml;charset=UTF-8,...
I would like to use img instead. How do I define src for that?
I tried:
src: 'book1.jpg'
But it doesn't work.

src: 'book1.jpg' will work as long as the path to the image is relative the URL. If you're image is somewhere specific (like an images directory/folder at the root of your domain/site) you'll need to do /images/book1.jpg,
Reactstrap isn't doing any magic here, it is the standard browser fetching and rendering of your image. See https://www.w3schools.com/hTml/html_filepaths.asp for the possible ways to resolve your path.

Thanks, you were right, path issue. I knew it was silly :)

Thank you very much!
Best Regards
Yossi
On Tue, Nov 14, 2017 at 10:02 PM, Evan Sharp<EMAIL_ADDRESS>wrote:

src: 'book1.jpg' will work as long as the path to the image is relative
the URL. If you're image is somewhere specific (like an images
directory/folder at the root of your domain/site) you'll need to do
/images/book1.jpg,
Reactstrap isn't doing any magic here, it is the standard browser fetching
and rendering of your image. See https://www.w3schools.com/
hTml/html_filepaths.asp for the possible ways to resolve your path.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/reactstrap/reactstrap/issues/684#issuecomment-344380413,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AO-VYxvEnXlsiA69QjlrckM6FVa1o1uCks5s2fHDgaJpZM4Qdwbe
.

Hello @rahamin1 ,
can you please help me out to debug this issue, I am still stuck. I am entering the path but the carousel is not working. Here is my code

I tried using import
I tried using require
I tried giving the path directly. Moreover, I may not know the actual path of the image, since these are getting built by webpack, even I have tried using /static/media in the relative path, but of no use.

import React , {Component} from 'react';
import { Carousel,
  UncontrolledCarousel,
  CarouselItem,
  CarouselControl,
  CarouselIndicators,
  CarouselCaption } from 'reactstrap';
  import himalaya from './images/himalaya.jpg';

 const items = [
  {
    src : {require('./images/beach.jpg')}, // even I tried using require, but no result
    altText: 'img can not be displayed',
    caption: '-Roxane Gay',
    header: '“Long walks on the beach are the supposed holy grail of a romantic evening. The beach becomes a kind of utopia - the place where all our dreams come true.”'
  },
  {
    src: {himalaya}, //by import, this is getting treated as a constant.
    altText: 'img can not be displayed',
    caption: '-Karen Blixen',
    header: '“You know you are truly alive when you’re living among lions.”'
  },
  {
    src:'./images/stream.jpg',
    altText: 'img can not be displayed',
    caption: '-Edmund Hillary',
    header: '“It is not the mountain we conquer but ourselves.”'
  }
];

Below is my folder structure:

Please let me know, where am I missing. It will be of great help.
Thanks

Sorry, no idea...


TagBot trigger issue

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

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

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

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

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


can we distribute libcuinj64.so and libaccinj64.so?

They are not mentioned in https://docs.nvidia.com/cuda/archive/10.2/eula/index.html#cuda-toolkit-supplement-license-agreement
cc @kkraus14, @jakirkham

If it's not included in the EULA we should assume we're not allowed to distribute it. As far as I know libcuinj and libaccinj are used by profiling tools like nvprof and typically aren't used by any libraries.
Will follow up internally to double check on this though.

Submitted PR ( https://github.com/conda-forge/cudatoolkit-feedstock/pull/34 ) to drop these libraries.


[PlanBuild] Display missing resources on plan piece hover

I'm not really clear on what you mean by this? it already shows when hovering over a plan to interact with it?

well, yes. there the thing on the lower screen border :P nvm


Wrong header size / missing hashtag

https://github.com/tubs-alg/AlgLab-Summer-24/blame/8e687510d613494459eb2668e3e4efa68854b23d/sheets/03_sat/exercises/02_hc_btsp/README.md#L53

Oh. We hoped that you were slower and we still had time to polish the third sheet a little. Have you already completed the first two sheets or are you only checking the third sheet out?


Sockets "beforeConnect" won't be called

The socket.js config method beforeConnect doesn't work:
First, the socket options are set in sails-hook-sockets/lib/initialize.js
opts.allowRequest = app.config.sockets.beforeConnect;
And then opts.allowRequest is being overwritten by checkRequest in sails-hook-sockets/lib/node_modules/socket.io/lib/index.js:
opts.allowRequest = this.checkRequest.bind(this);

sorry, duplicate of: https://github.com/balderdashy/sails/issues/2613


Discuss later if we want to change franchise and story id to slugs based on the product requirement

Will re create as needed


Add getting started section to readme

README
please add some more info to readme file

Added getting started in README file


Error listing changed packages in a nested module

Hi! 👋 I was curious if anyone has used gta in a repo that makes use of nested Go modules? I'm currently running into the following error when running gta in a repo that has a nested Go module when files in the nested module are changed (everything works fine if any files in the top-level module are changed):
17:46:02 main.go:82: can't list dirty packages: github.com/jeromefroe/gtatest/othersrc/pkg not found

I've created a minimal example of my repo layout at github.com/jeromefroe/gtatest. The repo looks like the following:
$ git clone git@github.com:jeromefroe/gtatest.git
Cloning into 'gtatest'...
remote: Enumerating objects: 20, done.
remote: Counting objects: 100% (20/20), done.
remote: Compressing objects: 100% (11/11), done.
remote: Total 20 (delta 3), reused 19 (delta 2), pack-reused 0
Receiving objects: 100% (20/20), done.
Resolving deltas: 100% (3/3), done.

$ cd gtatest/

$ tree
.
├── go.mod
├── othersrc
│   ├── go.mod
│   └── pkg
│       └── pkg.go
└── src
    └── pkg
        └── pkg.go

4 directories, 4 files

There is a top-level Go module at the root of the repo, and a nested Go module in the othersrc directory.
If I make a change to a file in the top-level module, then gta works just as expected:
$ git checkout jerome/change-parent-package
Branch 'jerome/change-parent-package' set up to track remote branch 'jerome/change-parent-package' from 'origin'.
Switched to a new branch 'jerome/change-parent-package'

$ git diff master --name-only
src/pkg/pkg.go

$ gta -include github.com/jeromefroe/gtates
github.com/jeromefroe/gtatest/src/pkg

However, if I change a file in the nested module, then I run into the aforementioned error:
$ git checkout jerome/change-child-package
Switched to branch 'jerome/change-child-package'
Your branch is up to date with 'origin/jerome/change-child-package'.

$ git diff master --name-only
othersrc/pkg/pkg.go

$ gta -include github.com/jeromefroe/gtates
17:45:56 main.go:82: can't list dirty packages: github.com/jeromefroe/gtatest/othersrc/pkg not found

I took a pass on the code base to try and better understand the problem I was running into, and I believe it's caused by the fact that the call to packages.Load to list all packages in the module doesn't return any packages in the nested module. As far as I understand packages.Load has the same functionality as go list and the latter doesn't listed nested modules:
$ go list ./...
github.com/jeromefroe/gtatest/src/pkg

Any packages in the nested module that are changed will still be returned by the git differ though, and so when they are passed to the PackageFromImport function an error is returned because the packages were never found in the set of loaded packages and so were never added to the dependency graph.
If it's of any benefit, I think it's fine if I have to run gta separately in each nested module in my repo (there's only a small number of them so it's easy to just enumerate them in our build tooling). The only tricky part I think would be detecting when changes in the top-level module are depended on by packages in a nested module.

You're right about the nature of the problem: go list, which is depended on by packages.Load will not list packages in the nested module when it's run in the top-level module.
If you want this to work, then you'd need to run gta for each of the nested modules, but I'm not sure if gta in its current form could handle the top level module when a package in a nested module is modified.  With a small code change, it should be possible to cause gta to ignore files there were changed in a nested module relative to the ambient module when gta is run (assuming that doesn't work currently).
Even if that code change is made, I'm afraid you may run into problems when a nested module is a dependency of the ambient module when gta is run; gta is built with the expectation that dependencies are vendored. I'd like to change that, but won't have time to do that for a while.
I realize that you may not have an option to break your modules out into their own repositories, but just in case you do and you haven't seen this advice before: https://github.com/golang/go/wiki/Modules#should-i-have-multiple-modules-in-a-single-repository

Hi @bhcleek! Yea, the current setup of the repo with its use of a nested module is certainly not ideal, but I don't think I'll be able to change it anytime soon. Fortunately, the nested module is much smaller than the top-level module so running the full suite of tests for it should be okay for the foreseeable future. The biggest time savings will be from not having to run the full test suite for the top-level module on pull requests.

With a small code change, it should be possible to cause gta to ignore files there were changed in a nested module relative to the ambient module when gta is run (assuming that doesn't work currently).

That would be great for our workflow! And I'd be happy to create a pull request if it would help. I was thinking about adding an additional exclude flag to the binary which would be the functional opposite of the include flag and would remove from consideration any packages which matching the given set of prefixes. What do you think of that approach?

@jeromefroe apologies for not getting back to you last week; the notification about your comment got buried in my e-mail.
I don't think we need an exclude for this; we literally need to detect when the changed files are in a different module. There's already code that tries to do something like this, but we need to extend it to detect multiple modules in a way that isn't not currently.
I'm not precisely sure where we need to make this change, but I think we probably need to update dependencyGraph to work with multiple directories: https://github.com/digitalocean/gta/blob/f497543409c4063829d25a1a8ee3ad9ecda93430/packager.go#L215-L304
It might be worth it for us to wait for the go.work proposal to be implemented, because trying to use dependencyGraph as-is is going to require changing the working directory when packages.Load is run to be within each module. 🤔


Reverse Scrolling and Tracking Speed Setting

closes: SMAR-45, SMAR-47.

We will need to save the settings to disk and load it we that settings persist across app restarts, but we can do that in a different PR. Lets just decide what to do about zoom settings then we can merge this


support for loading environments from file

Just dropping a note here in case anyone else will find it useful.
We needed support for loading environment overrides from a file (migrating off chef-server, lots of existing cookbooks, etc).
This patch, along with chef 11.4.0 and https://github.com/opscode/chef/pull/649 got the job done for us.
+THINGS_TO_COMBINE="nodes roles environments data_bags handlers"

 CHEF_ROOT="/var/chef-solo"

diff --git a/solo.rb b/solo.rb
index ab23ad1..2f09ced 100644
--- a/solo.rb
+++ b/solo.rb
@@ -4,6 +4,7 @@ file_cache_path "/var/chef-solo/cache"
 # These are created by run-chef.sh, with symlinks
 role_path "/var/chef-solo/combined/roles"
 data_bag_path "/var/chef-solo/combined/data_bags"
+environment_path "/var/chef-solo/combined/environments"

 # The checkouts each may contain a cookbooks directory.  run-chef.sh will
 # write a .cookbook-order file, and we should respect it.
@@ -21,3 +22,5 @@ cookbook_path cookbook_paths
 # Note that we also have handler under the combined/handlers area
 Dir.glob('/var/chef-solo/scripts/handlers/*.rb') { |f| require f }
 Dir.glob('/var/chef-solo/scripts/handlers/site/*.rb') { |f| require f }
+
+environment ENV['CHEF_ENVIRONMENT'] || "_default"

Also thanks to omniti. chef-solo-helper is very useful!

> 2 years old. closing.


fix: load after nvim-web-devicons (fixes #31)

fixes the conflict with nvim-web-devicons by loading barbecue.nvim after it.

Doesn't the requires = "kyazdani42/nvim-web-devicons" part do the same thing?

kinda, requires = "kyazdani42/nvim-web-devicons" tells packer that barbecue.nvim needs that plugin to work, but not the specific order in which to load the plugins. We do that with the after command, this way we avoid any conflict with nvim-web-devicons.
I saw it discussed in the NvChad repo (here)

Please rebase towards dev and force push.

Please rebase towards dev and force push.

i changed the PR target, i wrote the PR from GitHub since it was a oneliner.

Please rebase towards dev and force push.

i changed the PR target, i wrote the PR from GitHub since it was a oneliner.

I don't know how to do this on the github web, but please clone your fork (with --branch patch-1) and run the following
git rebase remotes/origin/dev
git push -f

ok soo, should i still clone and stuff?

ok soo, should i still clone and stuff?

Yep, because your pr contains duplicated commits from dev which hasn't been merged yet.

okok, i just did

Thanks!


FIX:  package-requirements always returns exit code 0, even on failure

Issue #:
Closes #388
Description of changes:
Check the exit code for pip3 download -r ... in entrypoint.sh. If non-zero, exit with the same error code.
Note: This only solves one particular case. A more thorough solution would be to use something like set -e for all these "utility" commands (package-requirements,test-startup-script, etc). But I imagine that is a larger (and possibly internal) discussion.
If this change is accepted, I also request that it get merged to all lower environments (or at least all supported ones) as well. This bug exists in v2.2.2-v2.9.2.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi - would someone kindly take a look at this?
Thank you in advanced!


adding-view.md: add missing words

Before:

The default view file has the same name as the method (Index), so in the /Views/HelloWorld/Index.cshtml is used.

After:

The default view file has the same name as the method (Index), so the view template in /Views/HelloWorld/Index.cshtml is used.

Thank you for the PR, @dharmatech! Your change will go live within the next 24 hours.


Incorrect code highlight std::swap

I'm getting an error highlight for std::swap as shown in this image - https://photos.app.goo.gl/JWpLGewodLgZDcRn6
Here's the program - https://gist.github.com/GauthamBanasandra/c69bad2dcbb96d82834b1fb26f00fcee
The error highlight is seen on this line - https://gist.github.com/GauthamBanasandra/c69bad2dcbb96d82834b1fb26f00fcee#file-test-the-rods-cpp-L129
Please note that the program compiles and runs fine with Visual C++ compiler.

VSCode Version: 1.31.1
OS Version: Windows_NT x64 10.0.17763

Steps to Reproduce:

Just open the program https://gist.github.com/GauthamBanasandra/c69bad2dcbb96d82834b1fb26f00fcee in Visual Studio Code.

Does this issue occur when all extensions are disabled?: No

It sounds like you are using the C++ extension, please file this on the C++ extension repo

Sure @roblourens . In case anybody wants to follow - https://github.com/Microsoft/vscode-cpptools/issues/3280


FR: Add every-n-month option (instead of monthly?) period

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Please add the possibility to select/set the frequency of every-n-month
Describe alternatives you've considered
For a bi-monthly occasion, I need to pick 6 annual event, then group them.
Additional context
None

Bad to hear that from you Mr. Wontfix bot. :S (upvote)


requiremets.txt might help with potential user issues

Hi! I just started using you amazing tool for a university robotics project.
We as a team have already spend numerous days on trying to figure out the error the constantly get with any kind of json we are using
    return _mask.decode(rleObjs)
  File "pycocotools/_mask.pyx", line 145, in pycocotools._mask.decode
  File "pycocotools/_mask.pyx", line 127, in pycocotools._mask._frString
TypeError: list indices must be integers or slices, not str

which happens in the following line of code:
masks = [maskUtils.decode(ann['segmentation']) for ann in dataset_dict['annotations']].
We have also tried CanaTree100 dataset that you provided.
I would really appreciate your help since we want to upgrade your work and train it on jungle data.

Hello,
I had a similar error some time ago, installing detectron2 leads to many errors that are related to version incompatibilities between several libraries.
To save time I created a notebook on google collab that allows to run everything (check the use of gpu in google collab).
If you have any questions, don't hesitate!
https://colab.research.google.com/drive/1x7WDmrDRuaLULEGXVmue7TKoieFysIAs?usp=sharing

Thank you a lot! Could you please make it public?

is is good now ?
https://colab.research.google.com/drive/1lRF7GhxwFwVgJBtBLy5TYLXD_vnRDHDv?usp=sharing

Hi! I just started using you amazing tool for a university robotics project.
We as a team have already spend numerous days on trying to figure out the error the constantly get with any kind of json we are using
    return _mask.decode(rleObjs)
  File "pycocotools/_mask.pyx", line 145, in pycocotools._mask.decode
  File "pycocotools/_mask.pyx", line 127, in pycocotools._mask._frString
TypeError: list indices must be integers or slices, not str

which happens in the following line of code:
masks = [maskUtils.decode(ann['segmentation']) for ann in dataset_dict['annotations']].
We have also tried CanaTree100 dataset that you provided.
I would really appreciate your help since we want to upgrade your work and train it on jungle data.

Hi, I remember getting this error a few times when switching between the synthetic/real dataset. IIRC this issue is caused by the segmentation mask encoding method. Depending on which encoding (RLE or bitmask) was used for the segmentation mask annotations, this error arises. In your case, it would seem that your annotations are already in RLE format since the
return _mask.decode(rleObjs) is an error.
masks = [maskUtils.decode(ann['segmentation']) for ann in dataset_dict['annotations']]
This is because the maskUtils.decode function decodes a binary mask into a RLE format (https://github.com/cocodataset/cocoapi/blob/master/PythonAPI/pycocotools/mask.py), but your input is already in RLE format. You might be able to simply replace it with
masks = [ann['segmentation'] for ann in dataset_dict['annotations']]
Hope it helps!


how to use argument of form --long-opt?

If I try to use an argument with a hypen in it, I get a compile error like:
error[E0609]: no field `args` on type `&argi::Argument<'_>`
  --> bin/main.rs:44:11
   |
44 |     match data!(bool, ctx => --include-methods) {
   |           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ unknown field

Is this supported?   Could it be?  Perhaps by specifying the arg in quotes/string...

It's a known issue in the beta unfortunately, for now the workaround is to do the quite ugly --include_methods. I'm getting back after a break so this is next on the list!


Consider adding #[payable] to Rust docs

I noticed that I couldn't find "payable" in the Rust docs site.
https://docs.rs/near-sdk/latest/near_sdk/index.html?search=payable
I thought #[payable] was removed, checked if it was in the near-sdk-macros crate. I'm not positive of how that macro works, but it seems like it might be important to index in the appropriate crate's documentation so a search could find it.
I've played with the Rust docs a bit and it can be very particular ;) Maybe we can start with anything that gets the content indexed for search, even if it doesn't fit as nicely as we'd like.

Getting macro attributes documented is really tricky, but this is indeed one of the most important aspects of near-sdk.

Resolved in #1185


LookupScript in LanguageRow.cs

Hi,
I'm a starter with serenity and MVC so I'm a little lost
I try to reuse Language data table in order to know customers speaking language.
If I add [LookupScript("Administration.Language")] to LanguageRow.cs then LanguageSelection in Main Page does not work.
I would appreciate so much any help
Thanks
Javier

try adding another class for lookup like this
    [LookupScript("Administration.Language", Permission = "?")]
    public class LanguageLookup : RowLookupScript<LanguageRow>
    {
        public LanguageLookup()
        {
        }
}

Thank you so much,
There is a class called LanguageLookup with that LookupScript, so if I add that to LanguageRow the first one fails.
What I did is add another LookupScript to LanguageRow.
I close the issue
Thanks so much


Method not found 'Android.Runtime.JNIEvn.CallStaticOjbectMethod

Hi Martijn,
I download your code, and running with Android 5.0, it throw error "Method not found 'Android.Runtime.JNIEvn.CallStaticOjbectMethod" on line "mediaPlayer = Com.Google.Android.Exoplayer.ExoPlayerFactory.NewInstance(1)", I'm not sure how to resolve the problem, any suggestions?
Thanks,
Shirley

Thanks for reporting! I'll look into this.

Just checked the source on an Android 5.0.1 AVD - everything works good...
@Shirley923 Can you provide more informations about the device you used?

@martijn00 I think we can close this then...


Wormhole Solana to Terra VAA issue / Network Consensus

Hi there,
My transaction got stuck at "waiting for wormhole network consensus" when I was transferring UST from Solana to Terra. The tokens are stuck in between networks.
When I try to redeem from the transaction signature, Wormhole says "requested VAA not found in store"
Any ideas to retrieve my tokens? Thanks.
Here's the signature:
https://explorer.solana.com/tx/5k1XLh4X2YcmCH3Q8aijhFdjKbYhPZgjqErxLjFSqY3bfPQTFryPHejtFAnny4ZLD2nf7ppQUyxrSdKHLULifh8u

See Discord linked on https://wormholenetwork.com for v2 support, this repo is for the v1 UI only


Feature/#6 祝日を考慮した日付リストを取得する関数の作成

Codecov Report

Merging #7 into develop will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop       #7      +/-   ##
===========================================
+ Coverage    94.53%   94.59%   +0.06%
===========================================
  Files           26       27       +1
  Lines          750      759       +9
===========================================
+ Hits           709      718       +9
  Misses          41       41

Impacted Files
Coverage Δ

pystock/utilities.py
100.00% <100.00%> (ø)

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 95c9320...72cc0ab. Read the comment docs.


 NO
have you edited the missionfile?: Yes
Issue:
Convoy missions seem to fail to spawn or get going correctly most of the time. Upon closer investigation of a convoy that failed to move all of the vehicles were flipped. They may be spawning too close together causing physics bugs but this is unconfirmed.

give the .rpt (log) file and we will look into it


Feature request: Make HRESULT::from_win32 public

Motivation
As far as I can see, there is currently no public way to convert a win32 error code to a HRESULT, similar to the HRESULT_FROM_WIN32 function in C. This would be helpful in a case where I'm implementing a dll function that returns HRESULT but should communicate errors that are in the win32 error code format. Converting to core::Error might be a bit too much for that.
Drawbacks
No response
Rationale and alternatives
No response
Additional context
As this is just the case of removing a (crate) from the current pub statement, I'm happy to provide a pr for this.

Not opposed to that; just throwing a note here that from_win32 hardcodes the facility to FACILITY_WIN32. We may want to expose an API that allows for overriding that (e.g. FACILITY_INTERNET).

Note that WIN32_ERROR already provides convertibility to HRESULT and Error.
https://github.com/microsoft/windows-rs/blob/42963ebf439cf32921df3c15b98e0ace917f24d3/crates/libs/bindgen/src/extensions/mod/Win32/Foundation/WIN32_ERROR.rs#L32-L41

@kennykerr thanks for pointing me at that. This way is a bit less obvious but certainly very helpful for my use case.


Use Parcel

Multiple target is buggy with generating types, but can be done within a script with using Parcel API

Parcel kinda sucks, especially for ESM & CJS exports at the same time. Maybe we can check https://github.com/vercel/ncc