Replace hard coded mqtt server

Probably to localhost or <IP_ADDRESS>

can a remote mosquitto host be configured with ip port user password?

Present code doesn't account for this, but when it does adding optional port and password would probably be a good idea.
The quick and dirty way to get this going is to hard code whatever you need to.  For a more permanent solution I'd have to research how best to keep it secure.
If you're asking about server configuration, that's beyond the scope here.  Some brief Googling reveals plenty of leads.


Failed to build gem native extension on Linux with no explanation

Hi There,
I'm using vagrant with Ubuntu 14.04 LTS (Trusty Tahr) and have installed sqlite3-dev. When I try to install mailcatcher, the installation fails and I don't get a nice debug message telling me what went wrong. So I'm posting what happened here in hopes that someone will be able to help me out. Thanks!
Here's what happened with the gem installer
vagrant@vvv:~$ sudo gem install mailcatcher
Building native extensions.  This could take a while...
ERROR:  Error installing mailcatcher:
        ERROR: Failed to build gem native extension.

        /usr/bin/ruby1.9.1 extconf.rb

Gem files will remain installed in /var/lib/gems/1.9.1/gems/eventmachine-1.0.3 for inspection.
Results logged to /var/lib/gems/1.9.1/gems/eventmachine-1.0.3/ext/gem_make.out
vagrant@vvv:~$ cat /var/lib/gems/1.9.1/gems/eventmachine-1.0.3/ext/gem_make.out
/usr/bin/ruby1.9.1 extconf.rb

Here's what happened when I tried to install from source
vagrant@vvv:~/mailcatcher$ gem build mailcatcher.gemspec
  Successfully built RubyGem
  Name: mailcatcher
  Version: 0.6.0
  File: mailcatcher-0.6.0.gem
vagrant@vvv:~/mailcatcher$ sudo gem install mailcatcher-0.6.0.gem
Fetching: json-1.8.1.gem (100%)
Building native extensions.  This could take a while...
ERROR:  Error installing mailcatcher-0.6.0.gem:
        ERROR: Failed to build gem native extension.

        /usr/bin/ruby1.9.1 extconf.rb

Gem files will remain installed in /var/lib/gems/1.9.1/gems/json-1.8.1 for inspection.
Results logged to /var/lib/gems/1.9.1/gems/json-1.8.1/ext/json/ext/generator/gem_make.out
vagrant@vvv:~/mailcatcher$ cat /var/lib/gems/1.9.1/gems/eventmachine-1.0.3/ext/gem_make.out
/usr/bin/ruby1.9.1 extconf.rb

thank you every much

You have to install ruby-dev. It's working with the 1.9 version of ruby.
Kind regards,

Thank you! This has helped me point me in the direction of using RVM to help install ruby on rails on Linux Mint

I can confirm that installing sudo apt-get install ruby-dev before sudo gem install mailcatcher worked fine.

install:
sudo apt-get install libgmp3-dev

Thanks @kenorb. This should really be part of the installation steps. I just spent about 45mins compiling ruby 2 and trying to install mailcatcher try & error style. This is the reason we have package managers.
Don't want to complain though, thanks for the great work guys.

@kenorb 's instructions worked for me:

installing sudo apt-get install ruby-dev before sudo gem install mailcatcher

installing sudo apt-get install ruby-dev before sudo gem install overcommit

Does that mean brew install ruby-dev for osx? It says No available formula with the name "ruby-dev"

Nope, that's the debian package name, not necessarily the homebrew name.
This might do it for you: https://github.com/rbenv/rbenv

sudo apt-get install ruby-dev  is working for me too.
But now I have to update Ruby to Version >= 2.0
Fetching: mime-types-data-3.2016.0221.gem (100%)
ERROR:  Error installing mailcatcher:
	mime-types-data requires Ruby version >= 2.0.

sudo apt-get install ruby-dev is working for me too.
This works for me too Thanks . :dancer:

@kkrieger85 See https://github.com/sj26/mailcatcher/issues/277#issuecomment-202617739

I saw the error Failed to build gem native extension too on CentOS 7.3 and vagrant 1.8.7.
I resolved it after installed ruby-devel and gcc (yum -y install ruby-devel gcc)

Thanks guys. I was traing to install sass using sudo gem install sass --no-user-install and I have the same error. I'm doing it on Chrome OS / Chromebook in core linux installed using Crouton. I did what many of you suggest with one more thing.
sudo apt-get install ruby-dev libsqlite3-dev gcc man

Thanks alot guys. I'm also facing the same error with Linux Mint 18.2 and ruby 2.3.1 .sudo apt-get install ruby-dev libsqlite3-dev gcc man solved the issue for me

In case someone struggles with OpenSuse:
# ruby 2.1.x
zypper in gcc make ruby2.1-devel ruby2.1-devel-extra
# or ruby 2.2.x
zypper in gcc make ruby2.2-devel ruby2.2-devel-extra

After run the following cmd , it will work fin for me
sudo apt-get install ruby-full

Ubuntu 14.04.5 LTS (and using the standard "Trusty" package repositories in /etc/apt/sources.list)
Fixed with:
sudo apt-get install ruby-dev
sudo apt-get install ruby2.0
sudo gem install mailcatcher

This is what worked for me:
apt remove ruby apt install software-properties-common apt-add-repository ppa:brightbox/ruby-ng apt update apt install ruby2.4 ruby2.4-dev libsqlite3-dev gem install mailcatcher

Same error happened to me on Ubuntu bash when I'm trying to install any of gem packages, I always get that ERROR: Failed to build gem native extension.
Then it solved using
sudo apt-get install build-essential dh-autoreconf
I also installed gcc & ruby-dev

I can confirm that installing sudo apt-get install ruby-dev libsqlite3-dev before sudo gem install mailcatcher worked fine.

Thanks  a lot, this solution was useful for me.

You have gcc?
Try 'apt-get install build-essential' before installing gem.

This was the only solution that worked for me on 2020. Thank you so much!


Shell integration cannot be enabled for executable "/bin/zsh"

In our integration tests:
      environmentVariableCollection
[25010:0420/174250.451393:INFO:CONSOLE(104)] "%c[IPC Library: Pty Host] %c%c WARN color: blue color:  color: #993 Shell integration cannot be enabled for executable "/bin/zsh" and args -l", source: vscode-file://vscode-app/Users/bpasero/Development/Microsoft/vscode/out/vs/base/common/console.js (104)
        ✔ should have collection variables apply to terminals immediately after setting (1418ms)
[25010:0420/174251.870999:INFO:CONSOLE(104)] "%c[IPC Library: Pty Host] %c%c WARN color: blue color:  color: #993 Shell integration cannot be enabled for executable "/bin/zsh" and args -l", source: vscode-file://vscode-app/Users/bpasero/Development/Microsoft/vscode/out/vs/base/common/console.js (104)
        ✔ should have collection variables apply to environment variables that don't exist (1418ms)
[25010:0420/174253.297744:INFO:CONSOLE(104)] "%c[IPC Library: Pty Host] %c%c WARN color: blue color:  color: #993 Shell integration cannot be enabled for executable "/bin/zsh" and args -l", source: vscode-file://vscode-app/Users/bpasero/Development/Microsoft/vscode/out/vs/base/common/console.js (104)
        ✔ should respect clearing entries (1419ms)
[25010:0420/174254.722725:INFO:CONSOLE(104)] "%c[IPC Library: Pty Host] %c%c WARN color: blue color:  color: #993 Shell integration cannot be enabled for executable "/bin/zsh" and args -l", source: vscode-file://vscode-app/Users/bpasero/Development/Microsoft/vscode/out/vs/base/common/console.js (104)
        ✔ should respect deleting entries (1418ms)
        ✔ get and forEach should work

I am still seeing that error on macOS when I run scripts/test-integration.sh and besides, there is now a dirty file outgoing:

How do I make it not dirty? I see what you mean, but I'm following suit here.

@meganrogge can we not check-in this change to the repo then? I do not think our test run should leave files dirty in the repo:

Sure, I have a workaround for this but it will be ugly and less ideal. Do you know why the other settings aren't sticking around too in the screenshot above? @bpasero

It looks like if I add config.update(settingName, undefined), it will remove that entry. so i'll do that in the teardown suite


Prevent module shadowing in pyspark_runner.py

Motivation
PySparkTask uses spark-submit to run the script pyspark_runner.py. Because it is run as a script the modules and packages from its directory (luigi/contrib/) shadow the global modules. In particular the package hdfs which is used by webhdfs_client.py is shadowed by the luigi.contrib.hdfs package. Thus PySparkTask does not work together with webhdfs.
An example task which highlights the problem can be found below:
import luigi
import luigi.contrib.hdfs
import luigi.contrib.spark

class MyTask(luigi.contrib.spark.PySparkTask):
    def output(self):
        return luigi.LocalTarget("not_present.txt")

    def main(self, sc):
        client = luigi.contrib.hdfs.webhdfs_client.WebHdfsClient("some_host")
        print(client.client)

Description
The problem is resolved by putting the current directory at the end of the path list sys.path in pyspark_runner.py.
Testing
I tested manually that this resolves the problem (for several weeks in production). I thought about adding a unit test for this but did not come with a good solution. The difficulty is that the problem occurs only when pyspark_runner.py is run as a script. But running the script necessarily tries to start spark by creating a SparkContext, which I guess is not desired for the unit tests. Suggestions on how to test this are welcome.

@jthi3rry Could you take a quick look at the patch?

LGTM @adaitche @dlstadther

Thanks @adaitche !


TypeError: Cannot read properties of null (reading 'includes')

Looks like not every response has a content-type
https://github.com/kwhitley/itty-fetcher/blob/v0.x/src/itty-fetcher.ts#L130
TypeError: Cannot read properties of null (reading 'includes')
    at /Users/tylervanderhoeven/Desktop/Web/XLMBQ/xlm-bible-quizzing-wrangler/dist/node_modules/.pnpm/itty-fetcher@0.8.1/node_modules/itty-fetcher/dist/itty-fetcher.mjs:1:865
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at sendCode (/Users/tylervanderhoeven/Desktop/Web/XLMBQ/xlm-bible-quizzing-wrangler/dist/src/utils/email.ts:13:3)
    at Object.handle (/Users/tylervanderhoeven/Desktop/Web/XLMBQ/xlm-bible-quizzing-wrangler/dist/node_modules/.pnpm/itty-router@4.0.0-next.46/node_modules/itty-router/index.js:1:580)
    at ServiceWorkerGlobalScope.[kDispatchFetch] (/opt/homebrew/lib/node_modules/miniflare/node_modules/@miniflare/core/src/standards/event.ts:385:13)
    at Server.<anonymous> (/opt/homebrew/lib/node_modules/miniflare/node_modules/@miniflare/http-server/src/index.ts:291:20)

https://github.com/kwhitley/itty-fetcher/pull/33
Maybe helpful?

Sorted via #33 :)


Making Button Options Easier to Use?

When  I was working on creating a situation, I realized that the getOptions() method is very bad to use when you have multiple conditionals for the options. The manipulation of the String[] may not result in proper function.
Let's say you remove something from an existing String[]. Do the buttons still function properly or would a blank/null button be formed?
If that is the case, what about having mutually exclusive options which increase your actual options count to >6? Would that experience a run-time error or a logic error, removing the 7+th buttons?
My current idea is to instead of using a String[], we use a Arraylist.

Using ArrayList syntax gets messy really fast, so I'll just implement code that skips over null entries in the String[] entirely, so they don't get displayed in the first place.


fix(theme): switch theme color between grayBase and grayDark [BREAKING CHANGES]

Issue:
#604 @liuderchi
What I did
How to test

Deploy preview for mcslite ready!
Built with commit b6284652f4cac9d82a28fa1d2e46fa248a2e8fc8
https://deploy-preview-605--mcslite.netlify.com

Deploy preview for mcs-lite-introduction ready!
Built with commit b6284652f4cac9d82a28fa1d2e46fa248a2e8fc8
https://deploy-preview-605--mcs-lite-introduction.netlify.com

Deploy preview for mcs-lite-ui ready!
Built with commit b6284652f4cac9d82a28fa1d2e46fa248a2e8fc8
https://deploy-preview-605--mcs-lite-ui.netlify.com


Core admittance

Fixed typo in core_admittance query

why is debug in the repo  especially at the root of the repo?

this fix is also in PR 13 I'm going to close this PR without merging.


Discord Party Drops

Allow people within the party to see the loot that others within the party receive. For example, if it is apart of their settings to show an items above ~5K, everyone in the party would see what was dropped.

Possible duplicate of #7111 ?

For the sake of separation, I'll update 7111 to be strictly about shared XP tracking, this can track shared loot specifically.


init doesn't exit if git-lfs doesn't exist

Changes proposed in the pull request:
-Got rid of the exit for checking git-lfs in init.sh as this is not a deal breaker and the consequences are discussed in the warning given

@OSUOSC/website-team

@malide you make a fair point


Please add a walkthrough for reading and processing CSV files

As acknowledged in this blog post: https://blogs.msdn.microsoft.com/david_burgs_blog/2017/05/19/logic-app-flat-file-encoding-and-decoding-documentation-references/ the documentation for dealing with csv files is hard to come by.
The flat file connector has virtually no documentation on it's use. What is there lacks detail around how to set up schemas for csv files (it only states that you should pick the schema, but doesn't provide anything useful if you don't have a schema).
In fact, there really should be a parseCSV action as well.

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

ID: 9b53be0a-5adf-becf-f005-ab5ac4e93f77
Version Independent ID: 644b795a-e152-c224-c366-1c1dd5ab2778
Content: Examples & common scenarios - Azure Logic Apps
Content Source: articles/logic-apps/logic-apps-examples-and-scenarios.md
Service: logic-apps
GitHub Login: @jeffhollan
Microsoft Alias: LADocs; jehollan

@hannesne Thank you for your feedback! We have assigned this issue to the author to investigate further and take the right course of action.

@hannesne Hi Hannes, thanks for your suggestion! We've created a work item for this request on our backlog to address in a later sprint.
@mike-urnun-msft #please-close


Small fixes after pylint/flake cleanup

Check if this PR fulfills these requirements:

[x] My changes respect the Kodi add-on development rules
[x] I have read the CONTRIBUTING document.
[x] I made sure there wasn't another one Pull Requests opened for the same update/change
[x] I have successfully tested my changes locally

Types of changes

[ ] New feature (non-breaking change which adds functionality)
[ ] Feature change (non-breaking change which change behaviour of an existing functionality)
[ ] Improvement (non-breaking change which improve functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
After reviewing #194 after an issue (#217) was found, I found one other issue.
Which this PR fixes.

Got it!
In the former code, we were doing an action for mediatype=MOVIE or for mediatype=SHOW.
In your code before the fix, if the type is MOVIE, the code will jump to for season_key in item.keys(): and run into an error.
So this fix is very right.

As most of the source code, annotations would be helpful 😉

I agree, and have to admit that piece of code I wrote was not a model of art...

@Smeulf There is no real issue with the code, but I learned earlier this year something called return early or guard code and it is something I unknowingly was practising already to keep indentation to a minimum (and readability to a maximum).
It is very pythonic and well-explained here: https://openfolder.sh/returning-early

And by extension, it reduced the code complexity if I'm correct.


Login Through Google Cannot handle token prior to timestamp

My php
<?php require

    "vendor/autoload.php";
//Step 1: Enter you google account credentials

$http     = new GuzzleHttp\Client(['verify' => 'cacert.pem']);
$g_client = new Google_Client();
$g_client->setHttpClient($http);

$g_client->setClientId("");
$g_client->setClientSecret("");
$g_client->setRedirectUri("http://localhost:8000");
$g_client->setScopes(Google_Service_PeopleService::USERINFO_PROFILE, Google_Service_PeopleService::USER_EMAILS_READ);

//Step 2 : Create the url
$auth_url = $g_client->createAuthUrl();
echo "Login Through Google ";

//Step 3 : Get the authorization  code
$code = isset($_GET['code']) ? $_GET['code'] : null;

//Step 4: Get access token
if (isset($code)) {

    try {

        $token = $g_client->fetchAccessTokenWithAuthCode($code);
        $g_client->setAccessToken($token);

    } catch (Exception $e) {
        echo $e->getMessage();
    }

    try {
        $pay_load = $g_client->verifyIdToken();

    } catch (Exception $e) {
        echo $e->getMessage();
    }

} else {
    $pay_load = null;
}
echo '
';
print_r($pay_load);
echo '
';

The error
Login Through Google Cannot handle token prior to 2018-11-07T05:25:08+0200

Notice:  Undefined variable: pay_load in C:\Users\user-pc\Desktop\g\google.php on line 45

I tried adding JWT::$leeway = 5 I get undefined variable. 20 minutes ago everything was working. I'm using the php built in web server, the time should be correct

Duplicate of #827

Please see the discussion their and in linked issues for help with your question.


Group.GetMember question

Reading the code of this line:
https://github.com/manicminer/hamilton/blob/aa2110f5eb5c7a17c14b947ca36c1461634920d3/clients/groups.go#L174
If I need to pass the tuple (group_id, member_id) and the return is an ID of the member, the method shouldn't be called IsMemberOf?
I would expect to have the full member object if I do a GetMember

Hi @yriveiro, thanks for the question! Members and owners have a bit of extra complexity because the member object is cast as a generic DirectoryObject by the API - since it can be a user, a service principal, or another group. This is definitely something that can be improved on, if you have any ideas or preferences I'd love to hear them :)

I'm using the GetMember to check if an object from the Directory Object is a member of a group once the client doesn't provide it and iterate the full list of members of the group is a kinda waste of CPU cycles.
I would change this to:
func (c *GroupsClient) IsMemberOf(ctx context.Context, groupId, memberId string) (bool, error) {
    _, status, err := c.GetMember(groupId, memberId string)
    if err != nil {
        false, err
    }

    return true
}

And the equivalent for IsOwnerOf also and let GetMember/GetOwner return the full response from the API once has some extra info.

I can see a GroupsClient.isMemberOf() method being useful, I'll look to add that (you're also welcome to send in a PR with the version in your last comment :).
I'm planning to look at ways to implement a generic DirectoryObjectsClient and if that works out then I think it makes sense to hook up GroupsClient members/owners methods to work with that.


Blocking disposable initialization when disposing

fixed #50

Current coverage is 91.4% (diff: 60.0%)

Merging #51 into master will decrease coverage by 0.3%

@@             master        #51   diff @@
==========================================
  Files            65         65
  Lines           770        773     +3
  Methods           0          0
  Messages          0          0
  Branches         61         64     +3
==========================================
  Hits            707        707
  Misses           38         38
- Partials         25         28     +3

Powered by Codecov. Last update 79bbac5...7ec62c1


graphman: pause subgraph before reassigning

Resolves #5253
The subgraph being reassigned is now paused if it wasn't and is then resumed once the reassigment process is complete.

Sorry this has been sitting for so long.
I think the change is totally fine, but I am curious if that really addresses the problem in practice - my experience has been that it can take a while for a subgraph to actually process the pause request. Have you seen that? Do you still get a race when that happens?
The basic issue is that the current mechanism for handling assignments doesn't provide a reliable way to say "node x is done with a subgraph so node y can start working on it" which opens the dorr to these races.


Unboxed atoms

Pull Request Description
TBD. Nothing works here, PRing not to lose it

Important Notes

Checklist
Please include the following checklist in your PR:

[ ] The documentation has been updated if necessary.
[ ] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[ ] Unit tests have been written where possible.
[ ] If GUI codebase was changed: Enso GUI was tested when built using BOTH
./run ide build and ./run ide watch.

Pull Request Description
TBD. Nothing works here, PRing not to lose it

Please update PR's description to match reality and share some performance numbers.


reif_libcg

@mreif
I pulled and ran the container and tried to reproduce the evaluation using sbt run with option 2, but was given the following error message:
[info] Running org.opalj.evaluation.EvaluationStarter
[info]
[info]
[info] PROCESSING: 0 with org.scala-lang | scala-compiler | 2.10.4
[info]
[info]
[info] CPFILE: /home/libcg/evalProjects/org.scala-lang/scala-compiler/jars/scala
-compiler-2.10.4.jar
[info] LibFiles: /usr/lib/jvm/java-8-openjdk-amd64/jre/lib
[info] /home/libcg/evalProjects/org.scala-lang/scala-library/jars/scala-library-
2.10.4.jar
[info] /home/libcg/evalProjects/org.scala-lang/scala-reflect/jars/scala-reflect-
2.10.4.jar
java.lanat scala.sys.package$.error(package.scala:27)d from runner: 137
[trace] Stack trace suppressed: run last compile:run for the full output.
[error] (compile:run) Nonzero exit code returned from runner: 137
[error] Total time: 201 s, completed Jul 22, 2016 3:55:14 AM

Any idea what might be the cause? I'm on Windows 8.1.

Brief Summary
This tool could run as described. It contains two program: Call  Edge Count Analysis and Evaluation Starter.
Insightful
This tool can construct call graph for Java libraries and give some analysis results. The effect of different assumptions on the build call graph can then be studied. The list of dead methods could also provide insights into quality issues of long living software projects.
Useful
The list of dead-methods could be useful.  Docker container is easy to use.
Usable
The source code of OPAL is available on Bitbucket. There is an official website of OPAL and libcg. But the document for this tool is not clearly displayed. A Docker container is provided, so it’s easy to start. The generate custom call graph numbers also works well.
One thing is that, we need machines with at least 6G memory to run the tool, instead of 4G as described by the authors. There is also no explanation about where the graph produced is stored and what it looks like.

@ctreude
I tried to investigate your error message but I'm not able to reproduce the error. Can you please run it again with "sbt compile:run" to print the full stack trace.
I would additionally like to know the configuration of your docker environment, since the suppressed error could be an OutOfMemoryException. You are required to give your docker process at least 6GB (as hongyujohn pointed out, this is actually a type that needs to be fixed.) of RAM.
best
Michael

@mreif
Yes, looks like it's a memory problem:
[info] Running org.opalj.evaluation.EvaluationStarter
[info]
[info]
[info] PROCESSING: 0 with org.scala-lang | scala-compiler | 2.10.4
[info]
[info]
[info] CPFILE: /home/libcg/evalProjects/org.scala-lang/scala-compiler/jars/scala
-compiler-2.10.4.jar
[info] LibFiles: /usr/lib/jvm/java-8-openjdk-amd64/jre/lib
[info] /home/libcg/evalProjects/org.scala-lang/scala-library/jars/scala-library-
2.10.4.jar
[info] /home/libcg/evalProjects/org.scala-lang/scala-reflect/jars/scala-reflect-
2.10.4.jar
[info] #
[info] # There is insufficient memory for the Java Runtime Environment to contin
ue.
[info] # Native memory allocation (mmap) failed to map 196608 bytes for committi
ng reserved memory.
[info] # An error report file with more information is saved as:
[info] # /home/libcg/.sbt/0.13/staging/847a683ed034ccc78f22/evaluation/hs_err_pi
d332.log
[error] OpenJDK 64-Bit Server VM warning: INFO: os::commit_memory(0x00000006f880
3000, 196608, 0) failed; error='Cannot allocate memory' (errno=12)
java.lang.RuntimeException: Nonzero exit code returned from runner: 1
        at scala.sys.package$.error(package.scala:27)
[trace] Stack trace suppressed: run last compile:run for the full output.
[error] (compile:run) Nonzero exit code returned from runner: 1
[error] Total time: 83 s, completed Jul 25, 2016 2:43:20 AM

I'm running docker run -ti -m 6g mreif/fse2016:evaluation but that doesn't seem to do the trick.

Note these labels are still "under discussion" and are still subject to change prior to the final notifications Friday.

@mreif @ctreude
Got the same error message as Christoph at first. Then tried to increase the memory, but was not able to run the container so far. Below is what was tried:
First, the Java Virtual Machine was not able to allocate memory. This error was detected after running the docker container using the following command line:
docker run -ti mreif/fse2016:evaluation
Therefore, proceeded to increase the memory limit for the container. This was the new command line to start the container, with 20GB as its memory limit:
docker run -t -i -m 20g mreif/fse2016:evaluation
The attempts using the previous command line threw the following error:
[info] Running org.opalj.evaluation.EvaluationStarter
[info]
[info]
[info] PROCESSING: 0 with org.scala-lang | scala-compiler | 2.10.4
[info]
[info]
[info] CPFILE: /home/libcg/evalProjects/org.scala-lang/scala-compiler/jars/scala
-compiler-2.10.4.jar
[info] LibFiles: /usr/lib/jvm/java-8-openjdk-amd64/jre/lib
[info] /home/libcg/evalProjects/org.scala-lang/scala-library/jars/scala-library-
2.10.4.jar
[info] /home/libcg/evalProjects/org.scala-lang/scala-reflect/jars/scala-reflect-
2.10.4.jar
java.lang.RuntimeException: Nonzero exit code returned from runner: 137
at scala.sys.package$.error(package.scala:27)
[trace] Stack trace suppressed: run last compile:run for the full output.
[error] (compile:run) Nonzero exit code returned from runner: 137
[error] Total time: 46 s, completed Jul 25, 2016 11:24:43 PM
libcg@148213ac3539:~/evaluation$
I am not familiar with the Scala programming language, therefore not sure how to approach this further... I am submitting my preliminary review for now, based on what I could try out and evaluate.

[Recommendation]: Maybe Gold
The problem addressed by the artifacts is hard to determine based on the paper and unfortunately a pre-print of the research paper is not available in order to better understand the ideas behind this set of artifacts. As it stands, the paper is rather confusing, as it discusses the insightfulness aspect related to call graphs, the usefulness for the list of dead methods and the usability of the list and the source code.
[Insightful]

Timely: Section 2 does not illustrate appropriately the problem that this tool/artifact is solving. The space constraint for this submission would be the main reason to not include a complete explanation, but in this case a preprint version of the research paper as part of the submission would be appropriate.
Makes researchers smarter: The tool seems to support researchers when they want to design static analysis on bytecode. Therefore, one can say that it makes researchers more productive.

[Useful]

Serves a useful purpose?: Definitely. It helps to conduct static analysis on Java bytecode.
Serves a purpose that would otherwise be tedious, prolonged, awkward, or impossible? Yes. Since bytecode is a low-level language, making it difficult to understand, tools that facilitate its abstraction are always helpful.
Cost-effective? It can require quite some time to understand the source code, so it may not be very cost-effective.

[Usable]

Easy to understand? The existing build offers a script to run custom analysis. Extensions of the tool will involve the comprehension of the source code, which might not be an easy task.
Accompanied by tutorial notes? I was unable to get the Docker container (see below, in the “Easy to download” section for the errors I got). However, the paper contains enough information to replicate the initial evaluation, and to run additional analysis on custom Java programs.
Easy to download: The code is in a public repository. However, Docker is needed to obtain the other artifact. Got errors that I could not solve trying to do that, so one of these artifacts is definitely not easy to download.
In the readme file, the provided link for the list of dead methods does not work. The “.org” in the link (instead of “.de”) seems to be the culprit.
Available in a virtual machine image? Yes. It uses Docker.
Supported by configuration management tools to permit easy updates? Yes. The source code is in a public Git repository, and the virtual machine was created using Docker.

Other comments:

There is a typo in the title of Section 2: INSIGHTFULNES -> INSIGHTFULNESS
The README file in GitHub mentions in the DISCLAIMER that “The numbers produced by this version of the algorithms will slightly differ from the ones in the paper. This is due to the usage of a different Java Class Library (in terms of operating system and version) and, in particular, also due to minor bug fixes. The camera ready copy of the paper will contain the updated numbers. Note, that none of the messages stated in the paper are affected at all.” Does this refer to this artifact paper or also the research paper? Do the new numbers impact the results obtained in the main research track paper?

@obaysal:

looks like @shaiduc agrees with our "gold" label, therefore i would say "gold". do you agree?

the access to a pre-print describing the work is an issue that plagued several reviewers. i do not think it necessarily harmed this cycle of reviewing (since other artifact tracks also work on such brief info)

that said,  for FUTURE artifact tracks, I would propose that authors deposit (a) the artifact 2 pager and (b) an article pre-print.

@timm agree. Keeping the decision as is.


How to connect to wekan with external openLDAP user.

How to connect to wekan with external openLDAP user.

Try does this work:
https://github.com/wekan/wekan/wiki/ldap#openldap


Zooming in and out is limited when using mouse scroll wheel or trackpad

Using a mac trackpad or a mouse with a scroll wheel on my pc, I am limited to a certain zoom level and cannot zoom in or out. I am forced to use the zoom out button on the map.

Scrollwheel zoom was turned off in the original Code For America version do to offset zooming. Need to investigate this more. Related: https://github.com/codeforamerica/trailsy/pull/121

Okay. I've turned it back on. Lets see how it goes..

Scroll zoom does in fact act weird due to offset zooming. Leaving it on for now but need to investigate custom scroll zoom or alternative.


Update to SPFx 1.10 and enable teams deployment

Q
A

Bug fix?
no

New feature?
yes

New sample?
no

Related issues?

What's in this Pull Request?
Updated sample to SPFx 1.10  and enabled teams deployment

thx @joelfmrodrigues, appreciated.


Ensure a single line of CLI output per query

Steps To Reproduce
Create a page with multiple queries and build it
Expected Behavior
A single line of CLI output per query
Actual Behaviour
Due to changes made when we added multithreading, the query puts print on top of each other

@archiewood here's what we're thinking. LMK if this checks out to you.
Updating the lines themselves is gnarly with queries running in parallel. Instead, we'll just report that queries are running, then output the final state of each query, and possibly that all queries are done.
running [n] queries
query_1 done from db
query_3 done from cache
query_2 error
....
ran [n] queries

Makes sense to me.
It's definitely possible to update multiple lines at once in a CLI output (I think python installs rely on this a lot), but I don't think the juice is worth the squeeze.
Aside
The other thing i would consider (if we are touching this) is giving an indicator of which location the query is running on.
running [n] queries
query_1 (index.md) done from db
query_3 (sources) done from cache
query_2 (sources/marketing) error
....
ran [n] queries

tight is good 👍🏼


ci: Check whether pod is being terminated before deleting it

Some pods might be already being terminated in GKE clusters, which
caused test suite to fail. This change adds a check to make sure we
don't try to delete terminating pods.

test-gke

test-runtime

test-net-next

test-4.19

test-4.9

Coverage increased (+0.008%) to 36.887% when pulling db2b4a769a32963206d1aae3ae133c9e465c79d3 on pr/dont-delete-terminating-pods into df16d5d011ebadc37fead0e1c296f1ab6e99bf5c on master.

test-runtime

test-4.19

test-gke


ENH: Possibility to pass scipy.optimize.Bounds object to scipy.optimize.linprog

Is your feature request related to a problem? Please describe.
The handling of the bounds in linprog, mostly if they are of type [[None, None]], is not explicit enough and can give the (false) impression that a problem's non-feasibility is due to that.
Sometimes it also feels that it would be preferable to use numpy arrays instead of lists for sake of consistence.
Internally linprog turns the bounds into a numpy array again (None becomes np.nan), and then the nan is converted into np.inf. This behavior is not reflected in the documentation even though passing np.nan or np.inf instead of None, or passing a numpy array is in principle equivalent.
Finally, scipy implements a Bounds object, that would seem coherent to use with linprog.
Describe the solution you'd like.

A documentation update reflecting the internal behavior for the bounds parameter.
The possibility to pass a scipy.optimize.Bounds object (implementation seems trivial, with e.g. np.vstack([Bounds.lb, Bounds.ub]).T ) instead of the usual sequence.

Describe alternatives you've considered.
No response
Additional context (e.g. screenshots, GIFs)
No response

Have you considered using scipy.optimize.milp? Under the hood, it uses essentially the same code, but it supports the use of Bounds and LinearConstraints.

Indeed, the two methods seem to be very similar, but then wouldn't it be logical to allow for the usage of  Bounds as an argument for linprog?
In my use-case i prefer the problem statement given by linprog as the problem has both the explicit inequality and equality constraints.
This proposition doesn't add much in the end. A bit more flexibility for the  Bounds arguments and a bit more clarity in the documentation.

I'm +1 on having a consistent interface - but don't currently have enough spare time to implement it.

If we allow bounds to accept instances of Bounds, why not also add the ability to specify constraints with instances of LinearConstraint?
If we should, how would those be accepted - with a new argument used in parallel with Aeq, Aub, etc? Do we allow both the new argument and Aeq, Aub, etc. to be used at the same time?
Whatever the answers, my concern is that this adds to an already complex interface that has been updated incrementally over the past decade or so.
Personally, I've thought of milp as a successor to the linprog interface to HiGHS. It was given a new name (rather than changing linprog) because there was also demand for having a function named milp, but at the same time, it also allowed us to clean up the interface without having to tiptoe around backward compatibility concerns.
From this perspective, no, I don't think it is required to keep the linprog interface in sync. Rather, I see it as a reason to add only to milp, and perhaps unify the two in a SciPy 2.0. I know this isn't the only perspective - others see an advantage to having a separate function specifically for linear programming without integrity constraint. That's fine. I'm just trying to responding to why I would not personally invest time into updating linprog.

I am currently struggling to come up with a consistent interface for the SLSQP rewrite. There are lots of tiny helper functions here and there from the actual function input to SLSQP solver mostly regarding the bounds objects. I agree with @mdhaber that the interface is very tedious and every one of them fix some obscure issue from the last decades. So over the years, necessarily, it became a band-aid tower apparently (it is a similar story in all modules of varying degree).
If there is a unified way to get rid of old bounds in a backwards compatible way before SciPy 2.0, I'm +1 for it. I just don't have enough experience around the optimize codebase hence can't form an opinion about the effort needed.
Regarding the milp, I think problems with integral constraints are sufficiently different than regular problems (also tend to be pseudo-separate discipline) that might warrant a separate function. Though we have not heard much about this distinction here or elsewhere online so probably a minor point.

Thanks for the clarification, I wasn't aware of a future interface change for HiGHS.
I will try rewrite my problem for milp. It is true that having both greater/ or lesser/ - then constraints, is an advantage and may be preferable on the long term, even in my specific case.
And if linprog is only kept for backwards compatibility reasons than it seems not to be the best place where to spend time.  + My preference may come from the large pool of Matlab users around me that use the function of the same name.
So I think this issue can be closed.

I started with Matlab, too, and contributing to linprog (writing the original interior point method, then revised simplex, then helping wrap HiGHS) is what got me started working on SciPy, so I get it!

It is true that having both greater/ or lesser/ - then constraints, is an advantage and may be preferable on the long term, even in my specific case.

It should be quite easy to wrap your existing constraints in LinearConstraint objects. Something like:
from scipy.optimize import LinearConstraint
ub = LinearConstraint(A_ub, -np.inf, b_ub)
eq = LinearConstraint(A_eq, b_eq, b_eq)

and it sounds like you are already familiar with Bounds.
Then it's just milp(c, bounds=bounds, constraints=(ub, eq).

Thank you very much, its very nice having one of the original developers helping out!
And I guess the rest will sort itself out in future versions of scipy, good luck with that! :)


[FEATURE] Create nodes in parallel to reduce the cluster creation time

Scope of your request
I've been quickly walking through the code.
Currently, all the nodes are created sequentially, and one by one:

Init node, if any
Cluster nodes
Aux nodes

This increase the k3d cluster create time that impacts the UX.
Describe the solution you'd like
At least for the cluster nodes, there is a chance to create them in parallel which may potentially reduce the cluster creation time, without significant code changes required.
What do you think?

Hi @brightzheng100 , thanks for opening this issue!
You're absolutely right there. In fact, that's how we did it back in v3.x... but then it broke cluster restarts. While this mainly happened in HA setups, where the server nodes have to come up one-by-one, sequentially, so that etcd properly learns its cluster members, it also broke from time to time, when too many agents tried to register to the cluster.
Aux nodes currently only consist of the loadbalancer, which has to come up after all target nodes (which in turn depends on the flags a user passed in, so it's the "way of least magic required" to simply start it after all other nodes).
So the only thing we could bring back in is the concurrent creation of agent nodes to speed things up a bit. I think there's even a TODO item somewhere for this (among many others :grimacing:)

Thanks for sharing the background @iwilltry42
That's exactly what I'm thinking too: to some kind of nodes, the sequence may matter.
But in most of the cases, we're creating a cluster with 1 server, x agent nodes so the agent nodes may still slow down the overall creation process -- that's why I asked too whether we can build the agent nodes in parallel.
Maybe I can dive deeper into the code and potentially make it another small PR too:)
(yeah, I can imagine you're busy as the versioning in K3d is quite aggressive)

Enhancement implementing this will land in the next major release (v5.0.0) :+1:

versioning in K3d is quite aggressive

Is it too aggressive? :thinking:


Tiling angle support

following back from issue: https://github.com/lovell/sharp/issues/1115
I exposed the angle argument for the .tile() function. It didn't really solve my problem, but it rotates the image. It looks equivalent to do .rotate() before .tile()

Coverage increased (+0.005%) to 99.244% when pulling 42a49fef7ca8dda28dbdddfe296b7fab61b6df2a on BiancoA:tiling-angle-support into 1a4e68096f8ab249375691e0d85a216cbd9914a7 on lovell:master.

Brilliant, thanks for the updates Andrea. This feature will be included in the forthcoming v0.19.1.


Unable to run shiny app

Hi,
I am using the buildpack mentioned for heroku-16 stack; and though CF PUSH works fine, the application does not start due to some missing libraries for GOMP and GLIBC.
CF PUSH extract:
Staging...
-----> Vendoring R 3.4.0 for heroku-16 stack (latest)
Downloading and unpacking R binaries (http://heroku-buildpack-r.s3.amazonaws.com/heroku-16/R-3.4.0-binaries-latest.tar.gz)
-----> Executing init.r script
R 3.4.0 successfully installed
Exit status 0
...
0 of 1 instances running, 1 crashed
FAILED
Error restarting application: Start unsuccessful
CF LOGS extract:
2017-07-26T12:29:10.97-0700 [APP/PROC/WEB/0] ERR /usr/lib/R/bin/exec/R: /usr/lib/x86_64-linux-gnu/libgomp.so.1: version GOMP_4.0' not found (required by /home/vcap/app/.root/usr/lib/libR.so)    2017-07-26T12:29:10.97-0700 [APP/PROC/WEB/0] ERR /usr/lib/R/bin/exec/R: /lib/x86_64-linux-gnu/libm.so.6: version GLIBC_2.23' not found (required by /home/vcap/app/.root/usr/lib/libR.so)
Can you please let me know how this issue can be resolved?
Thanks and Regards,
Sricharan

Same issue here. Tested pushing a basic hello-world shiny app to CF using both heroku-16 and older version heroku, they threw the same error -- caused by incompatible libgomp.so version required by libR.so (see message below). Would really like to see this resolved.

2018-03-12T11:41:29.07-0700 [DEA/182] OUT Starting app instance (index 0) with guid xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
2018-03-12T11:42:07.97-0700 [App/0] ERR /usr/lib/R/bin/exec/R: /usr/lib/x86_64-linux-gnu/libgomp.so.1: version GOMP_4.0' not found (required by /home/vcap/app/.root/usr/lib/libR.so)    2018-03-12T11:42:07.97-0700 [App/0] ERR /usr/lib/R/bin/exec/R: /lib/x86_64-linux-gnu/libm.so.6: version GLIBC_2.23' not found (required by /home/vcap/app/.root/usr/lib/libR.so)
2018-03-12T11:42:07.97-0700 [App/0] OUT
2018-03-12T11:42:07.99-0700 [DEA/182] ERR Instance (index 0) failed to start accepting connections


Include tar.gz files in Build.RS nupkg

After https://github.com/aspnet/Coherence-Signed/pull/478 we need to include .tar.gz files in the Build.RS nupkg.
Note that this package is used mainly for testing with JitBench on x-plat. cc @rynowak This is not blocking installer work since they obtain the stores from the drop share.
cc @muratg to decide whether this should go into rel/2.0.0-preview1 or only dev.

cc @mikeharder


The delete and edit buttons for deleted user records should be unavailable

In GitLab by @cuiaq123 on Nov 6, 2017, 14:15
If you are reporting a problem, please make sure the following information are provided:

Version of board (last git commit number) , docker engine and docker-compose.
Config files of board, you can get them by packaging "board.cfg" and files in the same directory, including subdirectory.
Log files, you can get them by package the /var/log/board/.
Other necessery information.

Please see attached picture

In GitLab by @liyanq528 on Nov 8, 2019, 07:40
fixed by pull request #595

In GitLab by @sokril on Nov 8, 2019, 08:13
closed

In GitLab by @sokril on Nov 8, 2019, 08:13
assigned to @liyanq528

In GitLab by @liyanq528 on Nov 11, 2019, 08:14
mentioned in merge request !268

In GitLab by @liyanq528 on Nov 13, 2019, 04:04
mentioned in merge request !260

In GitLab by @liyanq528 on Nov 13, 2019, 10:09
mentioned in merge request !364


[FLINK-18289][Checkpoint] Ensure notifyCheckpointAborted interface work in UDF operator

What is the purpose of the change
If udf operator with CheckpointListener overrides the new notifyCheckpointAborted interface, this PR ensures their logic could work well.
Brief change log

Override notifyCheckpointAborted in AbstractUdfStreamOperator.java

Verifying this change
This change added tests and can be verified as follows:

Added udf operator to override notifyCheckpointAborted in NotifyCheckpointAbortedITCase.java.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@flinkbot run travis

@flinkbot run azure

I noticed that the CI result has been success.

Thanks for pinging. I forgot to merge this. Shame that we didn't make it to RC2 😳 But I'm pretty sure there will be RC3.


Upgrade jekyll-open-sdg-plugins to 1.2.0.pre.beta4

This upgrades to a more recent version of the jekyll-open-sdg-plugins Gem. In doing so, the "goal_image_extension" setting will start working, so it is switched back to the default "png".
Also an unrelated change - this turns on two recommended (but optional) features: accessible_charts and accessible_tabs. These features are the result of a recent accessibility audit, and improve the screenreader and keyboard support for tabs and charts.

I think tests will fail here until #11 is merged.


Invalidate timing info buffers when destorying AAudio stream.

aaudio_stream_get_position() returns incorrect result because aaudio_stream_init() recycled destroyed stream where the timing_info buffers contain stale data.

This is to fix Fenix bug 1872354


[Request] Allow the withoutMigrations() method on Illuminate\Foundation\Testing\CrawlerTrait to accept arguments of which middleware to disable.

The WithoutMiddleware option is a fantastic feature for testing APIs. However, there are some packages that use middleware as a way to slip in new functionality. For example, Dingo API adds its own routing list by using a middleware. When testing, this gets to be a problem because if you want to disable middleware for security, you also end up disabling the routing middleware causing all HTTP calls
$this->get(),
$this->post(),
// etc.

to end up being 404 errors.
It would be great to be able to say
$this->withoutMiddleware(['auth', ...]);

Thanks

@EnolaLabs Yes it would be great especially for testing.

This isn't currently easily possible, sorry


comanage testing scenarios for Grouper replacement

We need to test enrollment flows and account linking.
Test users can be found on tab 2 of this worksheet.
All test users have been expunged from dev. See Bonnie for assignment of a test user.
The following scenarios should be tested:
User Enrollments:

enroll with an HCID
enroll with Google
enroll with Twitter
add a login method (Google and/or Twitter)

Account linking:

Test registration with a user that should pick up a society membership
Link user to society if membership not picked up

Testing with Barty Test (hctester10). Filled in initial info, waited through one or two quick background checks, and was taken to T&C. Love the Enrollment Flow checklist, though T&C is now higher.
Agreed to T&C and was taken to email verification. Alas, no email has been received in<EMAIL_ADDRESS>either in inbox or spam.

Removed comment. Error is mine.

Okay, did it without the error this time. Enrolled hctester10 (Barty Test). On filling out initial form and clicking submit was taken to T&C, which needs to appear higher in the enrollment flow checklist:

Then was taken to email verification (successful), and then various background processes appeared on the screen, including one saying that it was checking for T&C acceptance. Is that right?
Barty was successfully enrolled, and in registry I can see that he picked up HC, MLA, and ARLISNA, but he does not have the MLA badge. I took him into MLA to see whether he appears to have access, and in the groups list he gets this:

<EMAIL_ADDRESS>is a member of MLA, SAH, UP and HC. MLA is working fine, but SAH and UP join groups buttons are grayed out and not clickable. Comanage shows that the user is active for SAH and UP.
Enrollment worked fine, linking a new account method worked fine, but the user is not able to join groups in two of their societies.

hctester70 is a member of SAH (only one society). They are able to join groups on SAH, but not on HC. HC join group buttons are grayed out, but the ones on SAH work.

hctester70 has the correct labels for their society. hctester40 has no labels for their societies.
hctester40
https://registry.hcommons-dev.org/registry/co_people/canvas/127499
Ferdinand Test – Humanities Commons (hcommons-dev.org)
hctester70
hctester70 (other user) https://registry.hcommons-dev.org/registry/co_people/canvas/127521
https://sah.hcommons-dev.org/members/hctester70/

hctester100 seems to be working.  Created registration with Gmail successfully, logged in successfully, confirmed name and the one assigned society displayed properly for the profile.  Confirmed user could request to join groups in the society.

hctester20 received no roles in WordPress.
https://registry.hcommons-dev.org/registry/co_people/canvas/127527

Katie and I had successful tests this morning with hctester20 and hctester40. We're ready for the next round.


MNT: Various and sundry changes

In summary, this fixes some bugs of mine that we really need better testing to catch. Will add tests tomorrow.


[UI] UI should prevent users to changed the per-host-subnet label

Rancher versions:
rancher/server: v1.6-development
rancher/net:v0.13.0
Docker version: (docker version,docker info preferred)
17.06.2-ce
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
AWS
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
When change/remove the per-hot-subnet label, Will produce a lot of routing rules on the host, will not be cleared.
link https://github.com/rancher/rancher/issues/10075

Rancher/server: 1.6.11-rc5     UI: 1.2.26
Rancher/server: 2.0.0-alpha10     UI: latest
Rancher 1.6 and Rancher 2.0 are test passed.


Fix cut & copy for iPad

This PR fixes #101564
Why do not place anything in the textarea for the iPad.
Can you tell why? I can help.

@alexdima
When will it be processed?

guys, please merge it! It works, I tested :crossed_fingers: I'm really missing that thing!

I found it's a negative side effect of https://github.com/microsoft/vscode/commit/5d1c4a9b10d4485217f12fcfadd89e121069e040 which fixes the iPad detection. Previously on iPadOS, we thought it was not iPad as the user agent is now macOS so we always put the screen reader content into the textarea (so the textarea is never empty). With the fix, we never put anything into the clipboard and since copy falls back to document.execCommand('copy'), we never copy anything out of the textarea.
Since it works fine on iPadOS with isIPad === false previously, I think this fix is fine for iPad.


⚠ PrivateHD has degraded performance

In fe121f7, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 4898 ms

Resolved: PrivateHD performance has improved in 08be78c.


Update Support for Academy

@MarkvanMents , I reviewed the proposed changes and these are looking good to me.

Approval received. Merging.


scikit-learn: 0.24.1 -> 1.0.1

Motivation for this change
Updates to the stable branch of scikit-learn.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

fixed in #147426


Disallow arrow functions and enforce space consistency.

Farewell my react's best friends 😂

The majority approved, this is a democracy.


samples-work-in-progress CSharp bot-authentication-msgraph: LoginStepAsync endDialog issue

Sample information
Sample type: samples-work-in-progress CSharp bot-authentication-msgraph
https://github.com/Microsoft/BotBuilder-Samples/tree/samples-work-in-progress/samples/csharp_dotnetcore/24.bot-authentication-msgraph
NOTE: I'm using the branch recommended by @Kaiqb which targets v4.3
The MainDialog.cs LoginStepAsync logic actually breaks out of the WaterfallDialog with EndDialogAsync. This means that CommandStepAsync never gets executed.
https://github.com/Microsoft/BotBuilder-Samples/blob/1fc253e200effe553328486a2af23a655be55ca7/samples/csharp_dotnetcore/24.bot-authentication-msgraph/Dialogs/MainDialog.cs#L61
        private async Task<DialogTurnResult> LoginStepAsync(WaterfallStepContext stepContext, CancellationToken cancellationToken)
        {
            // Get the token from the previous step. Note that we could also have gotten the
            // token directly from the prompt itself. There is an example of this in the next method.
            var tokenResponse = (TokenResponse)stepContext.Result;
            if (tokenResponse != null)
            {
                await stepContext.Context.SendActivityAsync(MessageFactory.Text("You are now logged in."), cancellationToken);
                return await stepContext.PromptAsync(nameof(TextPrompt), new PromptOptions { Prompt = MessageFactory.Text("Would you like to do? (type 'me', 'send <EMAIL>' or 'recent')") }, cancellationToken);
            }

            await stepContext.Context.SendActivityAsync(MessageFactory.Text("Login was not successful please try again."), cancellationToken);
            **return await stepContext.EndDialogAsync();**
        }

I changed it to prompt the user for commands first and last thing it does is prompt for login.
   public class MainDialog : LogoutDialog
    {
        protected readonly ILogger _logger;

        public MainDialog(IConfiguration configuration, ILogger<MainDialog> logger)
            : base(nameof(MainDialog), configuration["ConnectionName"])
        {
            _logger = logger;

            AddDialog(new OAuthPrompt(
                nameof(OAuthPrompt),
                new OAuthPromptSettings
                {
                    ConnectionName = ConnectionName,
                    Text = "Please login",
                    Title = "Login",
                    Timeout = 300000, // User has 5 minutes to login
                }));

            AddDialog(new TextPrompt(nameof(TextPrompt)));
            AddDialog(new ChoicePrompt(nameof(ChoicePrompt)));

            AddDialog(new WaterfallDialog(nameof(WaterfallDialog), new WaterfallStep[]
            {
                PromptForCommandStepAsync,
                ProcessCommandStepAsync,
                ExecuteCommandStepAsync
            }));

            // The initial child Dialog to run.
            InitialDialogId = nameof(WaterfallDialog);
        }

        private async Task<DialogTurnResult> PromptForCommandStepAsync(WaterfallStepContext stepContext, CancellationToken cancellationToken)
        {
            var command = stepContext.Context.Activity.RemoveRecipientMention();
            if ((command != "me") && (!command.StartsWith("send")) && (command != "recent"))
            {
                return await stepContext.PromptAsync(nameof(TextPrompt), new PromptOptions { Prompt = MessageFactory.Text("Would you like to do? (type 'me', 'send <EMAIL>' or 'recent')") }, cancellationToken);
            }
            else
            {
                //persisted just for this turn
                stepContext.Values["command"] = stepContext.Context.Activity.Text;
                return await stepContext.NextAsync(stepContext.Context.Activity.Text);
            }
        }

        private async Task<DialogTurnResult> ProcessCommandStepAsync(WaterfallStepContext stepContext, CancellationToken cancellationToken)
        {
            //If we didn't get a recognize command start over
            var command = stepContext.Context.Activity.RemoveRecipientMention();
            if ((command != "me") && (!command.StartsWith("send")) && (command != "recent"))
            {
                return await stepContext.ReplaceDialogAsync(nameof(WaterfallDialog));
            }

            //persisted just for this turn
            stepContext.Values["command"] = stepContext.Result;

            // There is no reason to store the token locally in the bot because we can always just call
            // the OAuth prompt to get the token or get a new token if needed.
            return await stepContext.BeginDialogAsync(nameof(OAuthPrompt), null, cancellationToken);
        }

        private async Task<DialogTurnResult> ExecuteCommandStepAsync(WaterfallStepContext stepContext, CancellationToken cancellationToken)
        {
            if (stepContext.Result != null)
            {
                // We do not need to store the token in the bot. When we need the token we can
                // send another prompt. If the token is valid the user will not need to log back in.
                // The token will be available in the Result property of the task.
                var tokenResponse = stepContext.Result as TokenResponse;

                // If we have the token use the user is authenticated so we may use it to make API calls.
                if (tokenResponse?.Token != null)
                {
                    var parts = ((string)stepContext.Values["command"] ?? string.Empty).ToLowerInvariant().Split(' ');

                    var command = parts[0];

                    if (command == "me")
                    {
                        await OAuthHelpers.ListMeAsync(stepContext.Context, tokenResponse);
                    }
                    else if (command.StartsWith("send"))
                    {
                        await OAuthHelpers.SendMailAsync(stepContext.Context, tokenResponse, parts[1]);
                    }
                    else if (command.StartsWith("recent"))
                    {
                        await OAuthHelpers.ListRecentMailAsync(stepContext.Context, tokenResponse);
                    }
                    else
                    {
                        await stepContext.Context.SendActivityAsync(MessageFactory.Text($"Your token is: {tokenResponse.Token}"), cancellationToken);
                    }
                }
            }
            else
            {
                await stepContext.Context.SendActivityAsync(MessageFactory.Text("We couldn't log you in. Please try again later."), cancellationToken);
            }

            return await stepContext.EndDialogAsync(cancellationToken: cancellationToken);
        }
    }

#Repro steps
You can repro this by signing in. then typing "me" and seeing result. then typing "me" again. One time it'll respond with the me command, the next time you type me it returns with "you are now logged in".

@jthake-msft the objective of the sample is to show the OAuthPrompt against the graph.  If you log in unsuccessfully, then we break out.  so what you have provided is a much more bullet proof approach, but this is not my intention with the sample.

OK At Build 2019 I'll fork this repo and hostin Microsoft Graph and send people to our samples. This sample could never be used to spring board to anything like a real world scenario.

I totally agree with @jthake-msft   We do not need flawed logic to demonstrate how to use the OAuthPrompt.  This is confusing, and we are going to be explaining this flaw in the sample to other customers in the future.

I hear you.  how do I balance the feedback of making samples simple, and focused on understanding 1 concept with the desire to provide production possible code?


Help

How to set multiple links between two nodes in your project? Could you give me any suggestion or example? Thanks so much.

could you show me your input data, so we know what you're trying to achieve? you can have multiple relationships between two nodes, but they will be overlaying each other with this library

How to set multiple links between two nodes in your project? Could you give me any suggestion or example? Thanks so much.
hi, bro
maybe this project can help you
https://github.com/Alex-Li2018/neo4j-d3-graph


Trying to design simpler inner join

Hello, I'm trying to do a similar concept to find_also_related, but for inner joins instead of outer (which can then return a tuple of (element, element) instead of (element, Option). However, I don't understand where the Option part of that query is actually coming from (from select_also). I'm still not great at following traits like this so was just hoping to have some guidance on that, or if this would be a good idea to have through a PR.
Current implementation is just
pub fn find_also_inner_related<R>(self, r: R) -> SelectTwo<E, R>
where
    R: EntityTrait,
    E: Related<R>,
{
    self.join_join(JoinType::InnerJoin, E::to(), E::via())
       .select_also(r)
}

Hey @Dispersia, welcome!
The code snippet that control returning (element, Option) was

https://github.com/SeaQL/sea-orm/blob/a517e9db1aa8447300c25c3cf7fcf06724d0b698/src/executor/select.rs#L97-L110

I think it's good that we can a public API specifically for mandatory relation. A PR would be welcomed!
Feel free to let us know if you need any help :P


docstrings/sphinx: Support "Warns" sections

Is your feature request related to a problem? Please describe.
The goal is to have a common set of features between all parsers.
Describe the solution you'd like
The Sphinx parser should be able to parse "Warns" sections, if it makes any sense.

Closing for cleanup, but if you'd like to work on this, just ask and I'll reopen!


Docker 1.9.1 unable to run container

I have a complex app, consisted of ~10 containers.
This containers are configured via env variables, about 15-30 variables per container.
Sometimes container can't start with following message:
failed to cleanup ipc mounts:\nfailed to umount /var/lib/docker/containers/50fbd48156f40ea4b72419c869b3f9c390e69a2f0818150542b2c3664a10be8f/shm: invalid argument\nfailed to umount /var/lib/docker/containers/50fbd48156f40ea4b72419c869b3f9c390e69a2f0818150542b2c3664a10be8f/mqueue: invalid argument
after hours of different experiments I found, that broken container can be started if add or remove just one symbol from env variables value. no matter what variable changed, just remove one byte - and it works. if i return deleted symbol - container can't start again.
here is example of config.json from broken container: http://pastebin.com/ZPNa36fn
here is a 'docker export'-et brocker container: http://wasilyev.ru/mq.tar.gz
Downgrading to the docker 1.8.3 fix this situation.
[root@ba0 ~]# docker version
Client:
Version:      1.9.1
API version:  1.21
Go version:   go1.4.2
Git commit:   a34a1d5
Built:        Fri Nov 20 13:25:01 UTC 2015
OS/Arch:      linux/amd64
Server:
Version:      1.9.1
API version:  1.21
Go version:   go1.4.2
Git commit:   a34a1d5
Built:        Fri Nov 20 13:25:01 UTC 2015
OS/Arch:      linux/amd64
[root@ba0 ~]# docker info
Containers: 1
Images: 15
Server Version: 1.9.1
Storage Driver: devicemapper
Pool Name: docker-202:1-16830920-pool
Pool Blocksize: 65.54 kB
Base Device Size: 107.4 GB
Backing Filesystem:
Data file: /dev/loop0
Metadata file: /dev/loop1
Data Space Used: 2.168 GB
Data Space Total: 107.4 GB
Data Space Available: 30.57 GB
Metadata Space Used: 2.171 MB
Metadata Space Total: 2.147 GB
Metadata Space Available: 2.145 GB
Udev Sync Supported: true
Deferred Removal Enabled: false
Deferred Deletion Enabled: false
Deferred Deleted Device Count: 0
Data loop file: /var/lib/docker/devicemapper/devicemapper/data
Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
Library Version: 1.02.107-RHEL7 (2015-10-14)
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.10.0-229.14.1.el7.x86_64
Operating System: CentOS Linux 7 (Core)
CPUs: 2
Total Memory: 7.389 GiB
Name: ba0.s22091621.dev.qubell.com
ID: UJNO:KNEN:5BKJ:BBFC:XLVU:UCZR:RJTZ:MJOS:4AYS:F5FN:UQMW:CVCZ
[root@ba0 ~]# uname -a
Linux ba0.s22091621.dev.xxxxxx.com 3.10.0-229.14.1.el7.x86_64 #1 SMP Tue Sep 15 15:05:51 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
It is centos 7 on aws (ami-61bbf104)
Reproduced several times on different vms.
Steps to reproduce: import container, run it with parameters from config.json
Actual result: containter can't start
Expected result: container up and running

Hit this same issue on an Amazon AMI just now, for some reason running with devicemapper solved my issue.
Tried from the below issue.
https://github.com/docker/docker/issues/18967

The log messages you mentioned are not errors, just messages in the daemon logs that it couldn't cleanup, and that's ok.
Do you have the error message provided from the CLI?
This is eerily similar to #14203

[root@ba0 ~]# docker start mq
Error response from daemon: Cannot start container mq: [8] System error: read parent: connection reset by peer
Error: failed to start containers: [mq]
[root@ba0 ~]#
Jan 23 06:21:10 localhost docker: time="2016-01-23T06:21:10.344047433Z" level=warning msg="failed to cleanup ipc mounts:\nfailed to umount /var/lib/docker/containers/d39516dfc91ede7ab79750a91b1df7092dc6498a0a171b36d50f7f8e598ac010/shm: invalid argument\nfailed to umount /var/lib/docker/containers/d39516dfc91ede7ab79750a91b1df7092dc6498a0a171b36d50f7f8e598ac010/mqueue: invalid argument"
Jan 23 06:21:10 localhost docker: time="2016-01-23T06:21:10.344088277Z" level=error msg="Error unmounting device d39516dfc91ede7ab79750a91b1df7092dc6498a0a171b36d50f7f8e598ac010: UnmountDevice: device not-mounted id d39516dfc91ede7ab79750a91b1df7092dc6498a0a171b36d50f7f8e598ac010"
Jan 23 06:21:10 localhost docker: time="2016-01-23T06:21:10.344200992Z" level=error msg="Handler for POST /v1.17/containers/d39516dfc91ede7ab79750a91b1df7092dc6498a0a171b36d50f7f8e598ac010/start returned error: Cannot start container d39516dfc91ede7ab79750a91b1df7092dc6498a0a171b36d50f7f8e598ac010: [8] System error: read parent: connection reset by peer"
Jan 23 06:21:10 localhost docker: time="2016-01-23T06:21:10.344223913Z" level=error msg="HTTP Error" err="Cannot start container d39516dfc91ede7ab79750a91b1df7092dc6498a0a171b36d50f7f8e598ac010: [8] System error: read parent: connection reset by peer" statusCode=500

Yup, I think this is a duplicate of https://github.com/docker/docker/issues/14203
Error response from daemon: Cannot start container mq: [8] System error: read parent: connection reset by peer

But, thanks to awesome work by Shopify investigating that issue https://github.com/docker/docker/issues/14203#issuecomment-174177790, it looks like we may be able to resolve that issue :+1:
I'm gonna close this issue, because I think this is a duplicate. Please subscribe to the linked issue (using the "subscribe" button on that page) to stay informed on updates.
Feel free to comment here after I've closed, and thanks so much for reporting!


Select shipping address as returning customer: consistent button names

Reproduce:

login with account
make sure the account already stored a shipping address
put a random product in cart and open cart
click "Proceed to checkout"

Issue: Button name is inconsistent

button name on checkout pages for guest order: "Continue"

button name on shipping address page for returning customer: "Continue"

button name on billing address page for returning customer: "Use this billing address"

button name on payment page for returning customer: "Use this credit card"

Issue not fixed for poster v2.1.


[Bug]: useTransition deprecated in favor of useNavigation

Which packages are impacted?

[X] remix-validated-form
[ ] @remix-validated-form/with-zod
[ ] @remix-validated-form/with-yup
[ ] zod-form-data

What version of these packages are you using?

remix-validated-form: 4.6.10
@remix-run/react: 1.14.3

Please provide a link to a minimal reproduction of the issue.
N/A
Steps to Reproduce the Bug or Issue
Navigate to a page that uses ValidatedForm
Expected behavior
No warnings should be in the console. Instead, the following is shown:
react_devtools_backend.js:2655 ⚠ DEPRECATED: The `useTransition` hook has been deprecated in favor of `useNavigation` and will be removed in Remix v2.  Please update your code to leverage `useNavigation`.

See https://remix.run/docs/hooks/use-transition and https://remix.run/docs/hooks/use-navigation for more information.
    at ValidatedForm (http://localhost:3000/cod-v2/build/_shared/chunk-FVKP76MZ.js:2478:3)

Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome
Version: 108

Additional context
No response

Thanks for the callout! Addressing this will require us to increase the minimum supported remix version, so we'll need to do a major version bump.
Have the team given any estimate of when v2 might be released? If there are any other API changes we need to update for v2 down the road, I'd likely want to include all of them in the same "v2 support" release.
Definitely want to fix this, but also want to err on the side of taking my time with it since it's currently non-urgent.

@airjp73 This might be of interest to you https://remix.run/blog/future-flags.  Cheers!

@airjp73 Remix 1.15.0 officially deprecates the rest of the v1 APIs in favor of v2, so it looks like we're getting closer to v2.

Fixed in #284
Hoping to cut the v5 release sometime later today.


How to restore symbols-outline state on session restore?

I use auto-session plugin to manage my sessions. How can I restore the state of symbols-outline with it? Currently it restores empty buffer. Maybe there are some settings or maybe another session plugin which would work with symbols-outline in this case?

If you use nvim-possession you can pass a callback to the post_hook argument to restore states of additional buffers, for instance in this case the symbols outline.
However, this works with lua callbacks only, and as far as I know this plugin doesn't expose the corresponding require("symbols-outline").toggle/open() method or such (if it did, you could just pass it to the post_hook callback).


Add a method for in-place generating a default Procfile

Description of feature
While this isn't useful outside of the Dokku context, it would be great if we could generate a default Procfile based on whatever language was in use. If no Procfile exists, we would generate a default one based on the following template:
web: function.handler

The function.handler would be different depending on the language, but would be defined as follows:

dotnet: test::test.Function::FunctionHandler
go: bootstrap
nodejs: function.handler
pip: function.handler
ruby: function.handler

We could get smart and detect the function filename from lambda_function.$ext as well.

I wonder if we could instead set this as the default cmd? 🤔

If a file isn't detected, then we would default to... Nothing? I think that makes most sense.
It might also be cool to specify a --handler argument that would override anything autodetected. This --handler would also be set as the default CMD in the generated docker image from #7.


Add Juicy/tattoo to trigger Juicy/imported-template tests after each build

https://github.com/Starcounter/RebelsLounge/issues/64

ping @warpech  @PatrykSzwer  I'd disregard failing travis, it's a separate issue, as you can see here
https://github.com/Juicy/juicy-html/tree/debug-travis and here https://travis-ci.org/Juicy/juicy-html/builds

Can you please explain (perhaps in the README.md of Juicy/tattoo) how will the build of imported-template know which version of juicy-html to choose?
In https://github.com/Juicy/tattoo/blob/455d9f5af88df3c20d9b84bb63133a801d037658/juicy-tattoo.js#L52 it looks as if it always takes master.

Yes, that's easiest possible step, to get viable effect. As the title states, it calls imported-template tests after each build. It's using master branch only, but that's already something that would let avoid many problems we had in the past.

So that's not usable for verifying PRs before the merge.
Then I suggest to add this to .travis.yml in juicy-html at minimum:
branches:
  only:
  - master

I have made an issue about a possible long term solution: https://github.com/Juicy/tattoo/issues/1

So that's not usable for verifying PRs before the merge.

You're right, but it's useful after PR merge.
AFAIK
branches:
  only:
  - master

Would stop firing any builds for other branches, and we would like to run juicy-html tests for those.


feat: add warnings for errors 401 and 409

Closes #561.

@all-contributors add janek for this idea


refactor: Finish refactoring of RealTimeITL3PlanningClient

Explicitly pass parameters instead of using kwags and add type annotations.

Thanks!
Please add the context to MRs like this: We're removing kwargs since these arguments are the only ones that the underlying planningclient accepts, and the function signature is uninformative otherwise.

And could you run autotester for these as a sanity check, too?


[FEAT] Use model description as "note" in DBML.

Is your feature request related to a problem? Please describe.
All of my models have a description that describes what it is for and what it is not for. Also it tells limitations.
Describe the solution you'd like
To automatically use the "description" as the "note" in the generated DBML for dbdocs.io
Describe alternatives you've considered
Copying them manually but this is too much work.
Additional context
Example DBML from dbdocs:

Hi @SvensonDW Thanks for the feedback!
But I am surprised a bit with this one because it is already implemented.
Can’t you see the column description populated as note in dbml? If yes, please help to give more info to reproduce it.
Thanks a lot

Ah please ignore if this is model level, and it makes sense to add it. It will be done shortly, stay tuned 🏃

@datnguye  Yes for the model level, thanks so much! 😄

@SvensonDW it should be available soon in 1.8.0 🎉


Make clickable elements look clickable -- warning symbol buttons on Overview screen

On the Overview panel there are 2 warning symbols: ⚠ which would seem to indicate something of great importance to a user. But they don't currently provide any visual clue that they are actionable elements. If they had a more clickable-appearance (borders or beveled button edges) it could help users more quickly understand what they are being alerted to.

I didn't even know they were clickable, so they can't be that important :-)

@statusquont Would it be ok if those elements become buttons?

I concur with @statusquont and think these warnings should be clickable. Also ok with the proposal of @hebasto to have them as buttons. Thanks.

@polylunar

I concur with @statusquont and think these warnings should be clickable.

Currently, those GUI elements are clickable already.
They not look clickable though :)

Yes, sorry, should've been clearer, it would be good for them to be better identifiable as clickable, thanks.


CI Test randomly failed.

Gitea Version
#18701
Git Version
No response
Operating System
No response
How are you running Gitea?
on the CI
Database
No response
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Description
2022/02/10 12:06:25 .../queue/workerpool.go:537:doWork() [E] Unhandled Data in queue 3
--- FAIL: TestChannelQueue_Pause (0.60s)
    queue_channel_test.go:277:
        	Error Trace:	queue_channel_test.go:277
        	Error:      	handler chan should contain test1
        	Test:       	TestChannelQueue_Pause

Screenshots
No response

outdated


refactor: return initial exception and check if it's user error

SUMMARY

Refactor sqllab to return initial exception and use the status code to indicate whether this error should be logged as an exception. We are going to log from the handle_api_exception decorator instead of the lower level.
https://github.com/apache/superset/blob/9edb581f103d3e29b40d1c814cbe935a3d36f486/superset/views/base.py#L202
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@betodealmeida @john-bodley Here are more examples of us refactoring to bubble up errors to the flask or celery level.


Can't download the blockchain / HDD is read-only

I've been trying to download the blockchain for hours now.
The torrent is stuck at "Inactive:", and FTP fails do download right after I turn it on.
At first, I thought it's because of my internet connection, but pinging google works just fine.
Then, I found out that when I try to nano a txt file in the HDD directory (/mnt/hdd/), I can't save the file because I get "Read-only file system" error. After trying to make it writable I get "write-protected" error.
Anyone got any ideas what's wrong?

what is your out put of... ls -al /mnt/hdd
try downloading with this:
sudo wget -c -r -P /mnt/hdd/download -q --show-progress<EMAIL_ADDRESS>

what is your out put of... ls -al /mnt/hdd

total 24
drwxr-xr-x 3 root root  4096 Jan  1 10:10 .
drwxr-xr-x 3 root root  4096 Jan  1 10:11 ..
drwx------ 2 root root 16384 Jan  1 10:10 lost+found

try downloading with this:
sudo wget -c -r -P /mnt/hdd/download -q --show-progress<EMAIL_ADDRESS>
When I try this, nothing happens

try:
mkdir /mnt/hdd/download
then try that wget command again.
if it "does nothing" what do you see on the screen/terminal?

What happens when you run this?...
./00mainMenu.sh

try:
mkdir /mnt/hdd/download

mkdir: cannot create directory ‘/mnt/hdd/download’: Read-only file system

What happens when you run this?...
./00mainMenu.sh

I get the setup window (Choose either Bitcoin or Litecoin setup)
Here's the response during HDD init if you're interested

That looks OK so far. What happens later?

I have to choose how to download the copy of blockchain
When I choose Torrent here's the process:

And then it's stuck at "Inactive"

And when I choose FTP I get this screen almost instantly (there's a short print showing the link to the ftp server)

Can't get past this point

are you able to at least ping that machine that is the FTP server?...
ping w0189aba.kasserver.com

are you able to at least ping that machine that is the FTP server?...
ping w0189aba.kasserver.com

pinging works perfectly fine
At this point I'm pretty certain the issue is that the HDD is in read-only mode

I was looking at the script. Strange how it does this with a background process. What's your output of:
sudo screen -S Download -ls

do either of these succeed or same read-only error?
sudo -u root mkdir /mnt/hdd/download
sudo -u pi mkdir /mnt/hdd/download

sudo screen -S Download -ls

No Sockets found in /run/screen/S-root.

sudo -u root mkdir /mnt/hdd/download
sudo -u pi mkdir /mnt/hdd/download

Both of these give me the same error
mkdir: cannot create directory ‘/mnt/hdd/download’: Read-only file system

may be a clue?
https://www.raspberrypi.org/forums/viewtopic.php?t=210349#p1299075

Do you have another HDD that you can try and see if the same problem happens? I wonder if even a thumb/pen drive could be used just as a test - or if the setup scripts will detect it's not big enough for the ~230GB blockchain and so disallow it.

There seems to be some errors there.
Unfortunately don't have any other drives to test it.
If the power usage is the problem, how do I solve it then?

You can also try these manual steps to see if it gives insights to the problem:
verify your drive is /dev/sda with fdisk -l
try remounting it as read-write:
sudo mount -o remount,rw /dev/sda
if it complains about write-protected, you may also have to run sudo hdparm -r0 /dev/sda
Then try remounting read-write again.
if success, then try creating that dir again:
sudo mkdir /mnt/hdd/download

fdisk returns this:
Disk /dev/sda: 931.5 GiB,<PHONE_NUMBER>016 bytes,<PHONE_NUMBER> sectors
Units: sectors of 1 * 512 = 512 bytes
Sector size (logical/physical): 512 bytes / 512 bytes
I/O size (minimum/optimal): 512 bytes / 512 bytes
Disklabel type: dos
Disk identifier: 0xfb09f60f

But when I try to sudo mount -o remount,rw /dev/sda I get mount: /dev/sda not mounted or bad option

I wiped and reflashed the sd card so I can't get the full log, but here's the current log when I try to download the blockchain
Jan  1 14:33:29 RaspiBlitz kernel: [  255.207488] sd 0:0:0:0: [sda] tag#0 UNKNOWN(0x2003) Result: hostbyte=0x00 driverbyte=0x08
Jan  1 14:33:29 RaspiBlitz kernel: [  255.207500] sd 0:0:0:0: [sda] tag#0 Sense Key : 0x5 [current]
Jan  1 14:33:29 RaspiBlitz kernel: [  255.207506] sd 0:0:0:0: [sda] tag#0 ASC=0x24 ASCQ=0x0
Jan  1 14:33:29 RaspiBlitz kernel: [  255.207515] sd 0:0:0:0: [sda] tag#0 CDB: opcode=0x2a 2a 08 00 00 08 00 00 00 08 00
Jan  1 14:33:29 RaspiBlitz kernel: [  255.207521] print_req_error: critical target error, dev sda, sector 2048
Jan  1 14:33:29 RaspiBlitz kernel: [  255.207527] Buffer I/O error on dev sda1, logical block 0, lost sync page write
Jan  1 14:33:29 RaspiBlitz kernel: [  255.349616] EXT4-fs (sda1): mounted filesystem with ordered data mode. Opts: (null)
Jan  1 14:33:35 RaspiBlitz _background.sh[593]: auto-unlock is OFF
Jan  1 14:33:37 RaspiBlitz kernel: [  263.634476] sd 0:0:0:0: [sda] tag#0 UNKNOWN(0x2003) Result: hostbyte=0x00 driverbyte=0x08
Jan  1 14:33:37 RaspiBlitz kernel: [  263.634492] sd 0:0:0:0: [sda] tag#0 Sense Key : 0x5 [current]
Jan  1 14:33:37 RaspiBlitz kernel: [  263.634504] sd 0:0:0:0: [sda] tag#0 ASC=0x24 ASCQ=0x0
Jan  1 14:33:37 RaspiBlitz kernel: [  263.634519] sd 0:0:0:0: [sda] tag#0 CDB: opcode=0x2a 2a 08 3a 04 08 18 00 00 08 00
Jan  1 14:33:37 RaspiBlitz kernel: [  263.634530] print_req_error: critical target error, dev sda, sector 973342744
Jan  1 14:33:37 RaspiBlitz kernel: [  263.634544] print_req_error: critical target error, dev sda, sector 973342744
Jan  1 14:33:37 RaspiBlitz kernel: [  263.634575] Aborting journal on device sda1-8.
Jan  1 14:33:37 RaspiBlitz kernel: [  263.635185] sd 0:0:0:0: [sda] tag#0 UNKNOWN(0x2003) Result: hostbyte=0x00 driverbyte=0x08
Jan  1 14:33:37 RaspiBlitz kernel: [  263.635197] sd 0:0:0:0: [sda] tag#0 Sense Key : 0x5 [current]
Jan  1 14:33:37 RaspiBlitz kernel: [  263.635209] sd 0:0:0:0: [sda] tag#0 ASC=0x24 ASCQ=0x0
Jan  1 14:33:37 RaspiBlitz kernel: [  263.635221] sd 0:0:0:0: [sda] tag#0 CDB: opcode=0x2a 2a 08 3a 04 08 00 00 00 08 00
Jan  1 14:33:37 RaspiBlitz kernel: [  263.635229] print_req_error: critical target error, dev sda, sector 973342720
Jan  1 14:33:37 RaspiBlitz kernel: [  263.635241] Buffer I/O error on dev sda1, logical block 121667584, lost sync page write
Jan  1 14:33:37 RaspiBlitz kernel: [  263.635272] JBD2: Error -5 detected when updating journal superblock for sda1-8.
Jan  1 14:33:38 RaspiBlitz kernel: [  263.914030] EXT4-fs (sda1): previous I/O error to superblock detected
Jan  1 14:33:38 RaspiBlitz kernel: [  263.914764] sd 0:0:0:0: [sda] tag#0 UNKNOWN(0x2003) Result: hostbyte=0x00 driverbyte=0x08
Jan  1 14:33:38 RaspiBlitz kernel: [  263.914782] sd 0:0:0:0: [sda] tag#0 Sense Key : 0x5 [current]
Jan  1 14:33:38 RaspiBlitz kernel: [  263.914794] sd 0:0:0:0: [sda] tag#0 ASC=0x24 ASCQ=0x0
Jan  1 14:33:38 RaspiBlitz kernel: [  263.914809] sd 0:0:0:0: [sda] tag#0 CDB: opcode=0x2a 2a 08 00 00 08 00 00 00 08 00
Jan  1 14:33:38 RaspiBlitz kernel: [  263.914818] print_req_error: critical target error, dev sda, sector 2048
Jan  1 14:33:38 RaspiBlitz kernel: [  263.914837] Buffer I/O error on dev sda1, logical block 0, lost sync page write
Jan  1 14:33:38 RaspiBlitz kernel: [  263.914889] EXT4-fs error (device sda1): ext4_journal_check_start:61: Detected aborted journal
Jan  1 14:33:38 RaspiBlitz kernel: [  263.925198] EXT4-fs (sda1): Remounting filesystem read-only
Jan  1 14:33:38 RaspiBlitz kernel: [  263.932401] EXT4-fs (sda1): previous I/O error to superblock detected
Jan  1 14:33:38 RaspiBlitz kernel: [  263.933125] sd 0:0:0:0: [sda] tag#0 UNKNOWN(0x2003) Result: hostbyte=0x00 driverbyte=0x08
Jan  1 14:33:38 RaspiBlitz kernel: [  263.933147] sd 0:0:0:0: [sda] tag#0 Sense Key : 0x5 [current]
Jan  1 14:33:38 RaspiBlitz kernel: [  263.933158] sd 0:0:0:0: [sda] tag#0 ASC=0x24 ASCQ=0x0
Jan  1 14:33:38 RaspiBlitz kernel: [  263.933174] sd 0:0:0:0: [sda] tag#0 CDB: opcode=0x2a 2a 08 00 00 08 00 00 00 08 00
Jan  1 14:33:38 RaspiBlitz kernel: [  263.933183] print_req_error: critical target error, dev sda, sector 2048
Jan  1 14:33:38 RaspiBlitz kernel: [  263.933197] Buffer I/O error on dev sda1, logical block 0, lost sync page write

If you're sure the HDD is not defective, or flaky, perhaps using an AC powered USB hub between the HDD and Pi will help. Or getting a better Pi power supply.  You tried already swapping USB ports and USB cables, right?

After checking my drawer I found another HDD that after connecting to Raspberry started working.
At first, I thought it was the drive's fault, so I swapped old drive to a new case. Surprisingly it stopped working after that.  Finally, I swapped the cases and now it's working flawlessly.
Sorry for wasting so much of your time @fluidvoice and I really appreciate your help.
TL;DR: The problem was with faulty HDD case

Nice! glad you got it working.  It's fine. In helping people I'm learning a lot about how all these shell scripts work. Sadly, I'm not impressed with them. They are very fragile things and hard to troubleshoot when something goes wrong.


Program does not work

Win 8.1 x64. Program does not work. I've tried everything
Log file:
The following properties have been set:
Property: [AdminUser] = true {boolean}
Property: [InstallMode] = HomeSite {string}
Property: [NTProductType] = 1 {int}
Property: [ProcessorArchitecture] = AMD64 {string}
Property: [VersionNT] = 6.3.0 {version}
Running checks for package 'Microsoft .NET Framework 4.5 (x86 and x64)', phase BuildList
Reading value 'Version' of registry key 'HKLM\Software\Microsoft\NET Framework Setup\NDP\v4\Full'
Read string value '4.5.51650'
Setting value '4.5.51650 {string}' for property 'DotNet45Full_TargetVersion'
The following properties have been set for package 'Microsoft .NET Framework 4.5 (x86 and x64)':
Property: [DotNet45Full_TargetVersion] = 4.5.51650 {string}
Running checks for command 'DotNetFX45\dotNetFx45_Full_x86_x64.exe'
Result of running operator 'ValueEqualTo' on property 'InstallMode' and value 'HomeSite': true
Result of checks for command 'DotNetFX45\dotNetFx45_Full_x86_x64.exe' is 'Bypass'
Running checks for command 'DotNetFX45\dotNetFx45_Full_setup.exe'
Result of running operator 'ValueNotEqualTo' on property 'InstallMode' and value 'HomeSite': false
Result of running operator 'VersionGreaterThanOrEqualTo' on property 'DotNet45Full_TargetVersion' and value '4.5.50709': true
Result of checks for command 'DotNetFX45\dotNetFx45_Full_setup.exe' is 'Bypass'
'Microsoft .NET Framework 4.5 (x86 and x64)' RunCheck result: No Install Needed
Launching Application.
URLDownloadToCacheFile failed with HRESULT '-2146697208'
Error: An error occurred trying to download 'https://dl.dropboxusercontent.com/u/260255445/Democracy 3/ClickOnce/Democracy3LanguageSelector.application'.

I don't know where you found this log but it tell you that it can't download :
https://dl.dropboxusercontent.com/u/260255445/Democracy 3/ClickOnce/Democracy3LanguageSelector.application
can you try to access this url from the computer you try to install the application and tell me if you see something ?


Mixing require and import cause problem when using rollupjs for bundling

https://github.com/open-telemetry/opentelemetry-js/blob/a3b77387012a5f53c193efdefcbf5f6272876e4d/packages/opentelemetry-tracing/src/BasicTracerProvider.ts#L38
If mixing require and import, require keyword will still exist in the output that rollup generated for browser, causing error "require is undefined".
https://github.com/rollup/rollup-plugin-commonjs/issues/239#issuecomment-337892945
why not just import * as merge from 'lodash.merge'; instead?

We could probably safely enable the esModuleInterop for the build, but...
You can use the transformMixedEsModules in the latest version of the rollup commonjs plugin @rollup/plugin-commonjs
https://www.npmjs.com/package/@rollup/plugin-commonjs#transformmixedesmodules

@elsonz can you confirm if this works to fix your issue?

@elsonz can you confirm if this works to fix your issue?

thanks! transformmixedesmodules works!
At first, I modified the esm code in node_modules, and use patch-package for a patch. It's also another way to work around.
import * as merge from 'lodash.merge';
var mergedConfig = merge.default({}, DEFAULT_CONFIG, config);


Custom column headers

Similar to issues #32 and #16 I would like to be able to modify the column header. Specifically, I would like to be able to add React-Bootstrap tooltips to my column headers.  So when you hover over a column header it explains in more detail what that header means.
Related to this is being able to have a column header reference name that is different than the text that is displayed there.  For example, is it possible to have a column named something like Order Number but internally it is referred to as OrderNum?  Or maybe you make your site international and use the text Número de orden for Spanish speakers, but I want to be able to write code that just references a single variable name.
The place where I'm currently experiencing some amount of nuisance with this, is when external results gets sorted.  I have to convert Count to NumSystems on my server for the code shown here: https://github.com/DynamicTyped/Griddle/issues/14#issuecomment-64094434
If I then decide to rename it on the front-end to Number of Systems, I'll have to make another change on my server.  I would prefer not to have to make changes to my server when I only change some text on my front-end.
This is sort of two issues, but very related as it all comes down to wanting to modify what is shown for my column header, and wanting to be able to reference that column using a different name.

Hi @ryanlanciaux. Do you have any updates on this issue? I'm gonna render a react component in the header row instead of a string. Do you have any suggestion or advice?

Also interested in this!

+1 here. would go with custom rows quite nicely. any progress?

Hey there -- we are making progress on the next version and this is something that will be available. Beta should be out in the next week or two.

So we totally missed the week or two (rewrote another part of what we were changing). We are moving this issue to the griddle-render project https://github.com/GriddleGriddle/griddle-render/issues/13 and using it as acceptance criteria for when we release the new stuff. Closing this one for now but feel free to comment here or on the other with any questions.

@ryanlanciaux , hi, i got stuck in a problem where i am passing a function declared in react component , to another custom component.
following is my column meta for 'Batch' component:
var columnMeta = [
{
"columnName":"id",
"displayName":"Select",
"customComponent":BatchID,
"customComponentProps": {myFunc : this.getExternalData}
}
]
here for column "id" i am using a customComponent "BatchID" and want to pass a property to this "BatchID" component, which is a function for parent component "Batch"
var Batch = React.createClass({
getInitialState:function(){
var initial = {
"results":[],
"currentPage":0,
"maxPages":0,
"PageSize":PublicMethod.PageSize(),
"SortColumn":null,
"SortAscending":true,
"filters": {
"airlineName":""
}
};
return initial;
},
// general lifecycle methods
componentWillMount: function() {
},
componentDidMount: function() {
	this.getExternalData();
},
// Method will be for external data method
getExternalData: function(page) {
	page = page||1;

	// Preparing the querystring.
	// Sample querystring := 'airline='+airlineName+'&pageNum=0&pageSize=10'
	var url = '/App/api?';
	url += 'view=browseBatch&pageNum=' + (page - 1) + '&pageSize=' + this.state.PageSize;
	url += this.state.filters.airlineName!=''?'&airline='+this.state.filters.airlineName:'';
	// Call the API as required.
	$.ajax({
		type:"GET",
		dataType:"json",
		url:url,
		cache:false,
		success:function(_data){
			if(typeof(_data) == 'string') {
				_data =JSON.parse(_data);
			}

			this.setState({
				results: _data["data"],
				currentPage: page - 1,
				maxPages: Math.ceil(parseInt(_data["totalCount"]) / PublicMethod.PageSize())
			});
		}.bind(this),
		error: function(xhr, status, err) {
			console.error(this.props.url, status, err.toString());
		}.bind(this)
	})

	},
// render the component
render:function(){
	return(

Browse Batch

						<select id="airlineName" className="form-control" onChange={this.setFilter}>
							<option value="">Select Airline</option>
							<option value="AAL">AAL</option>
							<option value="JAL">JAL</option>
							<option value="ASA">ASA</option>
							<option value="DAL">DAL</option>
							<option value="AMX">AMX</option>
							<option value="GG">GG</option>
						</select>

			<Griddle
				useExternal={true} externalSetPage={this.setPage}
				externalChangeSort={this.changeSort}
				externalSetFilter={this.setFilter}
				externalSetPageSize={this.setPageSize}
				externalMaxPage={this.state.maxPages}
				externalCurrentPage={this.state.currentPage}
				results={this.state.results}
				resultsPerPage={this.state.resultsPerPage}
				externalSortColumn={this.state.SortColumn}
				externalSortAscending={this.state.SortAscending}
				showFilter={false} showSettings={false}
				columns={columns} columnMetadata={columnMeta}
				sortDefaultComponent={ }
				useGriddleStyles={false} useCustomPagerComponent={true}
			/>

	)
}

});
while i am looking into the props at "BatchID" i am getting this prop key "myFunc" with value "undefined".
Please give your suggestion ASAP.


Added "unicodecsv" to overrides as Python 3's CSV module does support Unicode

This commit adds "unicodecsv" to overrides.json as this is a drop-in replacement for Python 2's CSV module to support Unicode. Python 3's CSV module does support Unicode so those upgrading to Python 3 should transition from unicodecsv to csv from the standard library.

Thanks!


[TRIVIAL] Disable formatting of operator&

Clang 10.0.0 and clang 10.0.1 disagree on formatting operator&. Disable for now.

Only the last commit needs to be reviewed.

Minor point: it's not a difference between 10.0.0 and 10.0.1, it's a difference between nightly versions of 10.0.1. They update their PPA every day it seems.


pyauditor tests in CI are flaky

The pyauditor tests in the CI randomly fail. Rerunning them usually works. I wasn't able to reproduce this locally yet.

This could not be fixed with #255, see e.g. https://github.com/ALU-Schumacher/AUDITOR/actions/runs/4597844915/jobs/8121084553?pr=255
The following log lines might be related to the crash:
{"v":0,"name":"AUDITOR","msg":"[HTTP REQUEST - END]","level":30,"hostname":"fv-az574-655","pid":3134,"time":"2023-04-03T14:30:55.912140342Z","target":"tracing_actix_web::root_span_builder","line":41,"file":"/home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tracing-actix-web-0.7.3/src/root_span_builder.rs","http.client_ip":"<IP_ADDRESS>","otel.status_code":"OK","http.route":"/get","http.target":"/get","elapsed_milliseconds":1,"http.status_code":200,"http.user_agent":"auditor/0.0.7","http.method":"GET","otel.name":"HTTP GET /get","request_id":"be52ed14-11c3-4ac4-9ddc-e2a0b92e6839","http.flavor":"1.1","http.scheme":"http","otel.kind":"server","http.host":"<IP_ADDRESS>:8000"}
LOCAL TIMEZONE: Etc/UTC
Testing /health_check endpoint
get should not return anything because there are not records in Auditor yet
Adding a record to Auditor
/home/runner/work/AUDITOR/AUDITOR/./pyauditor/scripts/test_components.py executed in 0.03 seconds.
Fatal Python error: PyGILState_Release: thread state 0x7fc168005b70 must be current when releasing
Python runtime state: finalizing (tstate=0x0000559b87c61280)

Thread 0x00007fc1771c8000 (most recent call first):
  <no Python frame>
./scripts/test_pyauditor.sh: line 55:  3146 Aborted                 (core dumped) python3 $script
+ cleanup_exit
+ setsid nohup bash -c '
    kill 3134
    rm -rf .env_test
  '
{"v":0,"name":"AUDITOR","msg":"SIGTERM received; starting graceful shutdown","level":30,"hostname":"fv-az574-655","pid":3134,"time":"2023-04-03T14:30:56.038562809Z","target":"actix_server::server","line":328,"file":"/home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/actix-server-2.1.1/src/server.rs"}

This could also not be fixed by updating the pyo3 bindings to the latest version (#356)


[BUG] Arrow keys wont work

Describe the bug
Arrow keys wont work
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Install 'npm install -g @microsoft/inshellisense'
execute 'inshellisense bind'
select bash
Enter some commands
use arrow up or arrow down
Nothing happens

Expected behavior
 if (key.upArrow) {
    setSelectionIdx(Math.max(0, selectionIdx - 1));
}
else if (key.downArrow) {
    setSelectionIdx(Math.min(availableShells.length - 1, selectionIdx + 1));
}

Self-explanatory.I think setSelectionIdx will not executed, i dont know..
Environment

OS: Debian 12, Linux my.fruit <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Fri Jan 27 02:56:13 UTC 2023 x86_64 GNU/Linux
Output of inshellisense --version: 0.0.1-rc.4
Nodejs Version: node -v (v18.13.0), npm -v (9.2.0)

Duplicate of #9

Closing as this was fixed in 0.0.1-rc.5


add cached country names and currency detection

some improvements made by @jburg88

@jburg88 I would love to merge your improvements into my branch if you agree.

@jburg88 Thanks for contributing (added a contributors link to the readme) to Chames! 🙂
Feel free to submit pull requests at any time.