very slow bundle generation

Description
I created a tool targeted at JS developers that want to use ClojureScript with existing JS tools like this one. However as soon as I use use some of the generated JS code the bundle generation gets extremely slow.
Expected Behavior
2:39:28 PM: Finished building JavaScript bundle in <acceptable time>ms

Just yarn start after shadow-cljs --once completed.
Observed Behavior
2:39:28 PM: Finished building JavaScript bundle in 108191ms

The startup eventually completed but took considerable amounts of time. No errors are generated and the code works just fine. Generating a new bundle after changing some code takes just as long.
Environment
Please run these commands in the project folder and fill in their results:

npm ls react-native-scripts<EMAIL_ADDRESS>npm ls react-native<EMAIL_ADDRESS>npm ls expo<EMAIL_ADDRESS>node -v: v7.10.0
npm -v: 4.2.0
yarn --version: 0.24.4
watchman version: 4.7.0

Reproducible Demo
https://github.com/thheller/npm-module-example/blob/master/examples/react-native-example/README.md
shadow-cljs --once generates code into the ./node_modules/shadow-cljs directory (ie. ./node_modules/shadow-cljs/cljs.core.js).
It generates ES3 JS code. The file is 2.4MB and contains an inline source-map with the original sourcesContent. I'm not sure why it would be so extremely slow. No other tools I have tested (webpack, browserify, create-react-app) showed noticeable slowdowns.
I'm not familiar with react-native at all and this is the first time I tried using it. If I'm supposed to exclude or configure something please let me know. I just used yarn create react-native-app generator and added one .cljs file.
Short overview for the shadow-cljs tool:
https://github.com/thheller/shadow-devtools/wiki/ClojureScript-for-JS-Devs
I uploaded all the generated CLJS -> JS code here:
http://thheller.com/files/create-react-native-app/
http://thheller.com/files/create-react-native-app/cljs.core.js
http://thheller.com/files/create-react-native-app/demo.foo.js

Ok, I figured it out. The babel :presets are responsible, doesn't seem to be possible to disable :presets for one directory.
Tried using a .babelrc with just { "presets": [] } in the directory but that doesn't seem to have an effect.
Nevermind.

@thheller Can you elaborate? What did you change to make it work correctly? We are having the same problem.

Since 0.45.0 the .babelrc works again.
I have this .babelrc in the folder that has all the generated .js code. In the case of ClojureScript the generated code is ES3 already so it does not need any babel transforms.
{
    "ignore": ["*.js"]
}

I do not know if it is possible to run custom babel transforms. I only tried not running any.

try with this commands:
sudo chown -R $USER ~/.npm
sudo chown -R $USER /usr/local/lib/node_modules
work for me, before time 5-8min, after this commands 1-2 min
source: https://facebook.github.io/react-native/docs/troubleshooting.html#content


Move examples to separate repo

The examples in the HELICS repo should be moved to a separate repo to make them easier to test, and better demonstrate linking with helics and the execution of the HELICS runner.  we need to leave a couple for compilation tests in the helics repo but the majority should be moved to a separate repo to better separate concerns.

@nightlark is this likely to be finished in 1.3(7/31/18)  or do we need to move to 1.3 first before it can be finished?

I think we can remove the examples other than the ones as a test case today. The code in the examples repo is up to date and a copy of HELICS is getting installed on Travis -- I think the remaining step is writing a script to go through each C++ example and run cmake/make to build them, then start a small federation to test the examples.

@nightlark Can you make a PR that does that.  And I will add some language to the ReadME that links to the examples Repo.    The examples repo will need a readme.md (can be brief but it needs something)


Travis Support

Great to see you're working on Travis integration :+1:

Thanx ;).
I still need to get it working.
It'll take me a bit longer to get around it again.


GH-333 Implement GridPattern support for Godot 4.3

Fixes #333
This PR introduces a new option in the GraphEdit toolbar to switch between Line and Dot based patterns.

In addition, a small cosmetic enhancement to add some shadow/fidelity to the toolbar:

The code is guarded so that the cosmetic enhancement can be used by Orchestrator 2.0 and the new feature by 2.1 when rolling the GDExtension base forward to Godot 4.3.

Applied to main (=2.1) and branch 2.0.


[SPARK-43587][CORE][TESTS] Run HealthTrackerIntegrationSuite in a dedicated JVM

What changes were proposed in this pull request?
This PR aims to run HealthTrackerIntegrationSuite in a dedicated JVM to mitigate a flaky tests.
Why are the changes needed?
HealthTrackerIntegrationSuite has been flaky and SPARK-25400 and SPARK-37384 increased the timeout from 1s to 10s and 10s to 20s, respectively. The usual suspect of this flakiness is some unknown side-effect like GCs. In this PR, we aims to run this in a separate JVM instead of increasing the timeout more.
https://github.com/apache/spark/blob/abc140263303c409f8d4b9632645c5c6cbc11d20/core/src/test/scala/org/apache/spark/scheduler/SchedulerIntegrationSuite.scala#L56-L58
This is the recent failure.

https://github.com/apache/spark/actions/runs/5020505360/jobs/9002039817

[info] HealthTrackerIntegrationSuite:
[info] - If preferred node is bad, without excludeOnFailure job will fail (92 milliseconds)
[info] - With default settings, job can succeed despite multiple bad executors on node (3 seconds, 163 milliseconds)
[info] - Bad node with multiple executors, job will still succeed with the right confs *** FAILED *** (20 seconds, 43 milliseconds)
[info]   java.util.concurrent.TimeoutException: Futures timed out after [20 seconds]
[info]   at scala.concurrent.impl.Promise$DefaultPromise.ready(Promise.scala:259)
[info]   at scala.concurrent.impl.Promise$DefaultPromise.ready(Promise.scala:187)
[info]   at org.apache.spark.util.ThreadUtils$.awaitReady(ThreadUtils.scala:355)
[info]   at org.apache.spark.scheduler.SchedulerIntegrationSuite.awaitJobTermination(SchedulerIntegrationSuite.scala:276)
[info]   at org.apache.spark.scheduler.HealthTrackerIntegrationSuite.$anonfun$new$9(HealthTrackerIntegrationSuite.scala:92)

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the CIs.

Could you review this please, @HyukjinKwon?

Could you review this, @LuciferYang ?

Thank you, @LuciferYang ! :)

Merged to master/3.4/3.3


#317 fix for initializing signature

Proposed fix for https://github.com/danvick/flutter_form_builder/issues/317

@vin-fandemand Thank you for the pull request.  I can see that you put a lot of work into making the Signature Pad field better.
@danvick Do you have anything else to add, or should I just merge this pull request?

@vin-fandemand I appreciate the good work and the commitment to see this through.
@awhitford I think we could merge after the above concern is addressed.
I still feel that we may need to find a way to make it look obvious that the initialValue is read-only later on.
Maybe we could we consider changing its background color to Theme.of(context).disabledColor, @awhitford?
Food for thought.

Thanks @danvick  and @awhitford , added the final review comments as well

Thanks a lot. Do you know when these changes will be available publicly?
Please let me know if you have any queries.
Regards,
Vineeth
On Mon, Jul 6, 2020 at 5:18 PM Anthony Whitford<EMAIL_ADDRESS>wrote:

Merged #344 https://github.com/danvick/flutter_form_builder/pull/344
into master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/danvick/flutter_form_builder/pull/344#event-3517687776,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APM2Z4K725IQQQT6SRUSIN3R2I5TBANCNFSM4OHQ27YA
.

@danvick valueTransfomer breaks because of static type of _savedValue
E/flutter ( 5930): [ERROR:flutter/lib/ui/ui_dart_state.cc(177)] Unhandled Exception: type 'String' is not a subtype of type 'Uint8List'

You are right that the _savedValue is a Uint8List:
https://github.com/danvick/flutter_form_builder/blob/34c6c337284fe07c4e3a364b116b0caec0b30be7/lib/src/fields/form_builder_signature_pad.dart#L62
https://github.com/danvick/flutter_form_builder/blob/34c6c337284fe07c4e3a364b116b0caec0b30be7/lib/src/fields/form_builder_signature_pad.dart#L124-L126
@sumitbhanushali Can you elaborate on why you are transforming the Uint8List to a String?  How do you translate the String back into a Uint8List?

@awhitford i am converting it using base64.encode to store it as image on server

@sumitbhanushali Thanks, that is helpful.
Now doesn't this expose a design flaw because you'll never be able to supply a String as initialValue to see the Signature?  (If you ever wanted to display the result as a read-only field.)
@danvick What do you think is the answer here?  (I see the code radically different in v4).

@awhitford correct me if i am wronge but isnt the job of valueTransformer is to convert between data types?

valueTransformer is a one-way function between T and dynamic:
https://github.com/danvick/flutter_form_builder/blob/34c6c337284fe07c4e3a364b116b0caec0b30be7/lib/src/form_builder.dart#L3
For a Signature, T is Uint8List and your base64 encoded result is the dynamic result.
Design-wise, we lack a reverse function to translate the dynamic result back into the T.


v0.3.0

Codecov Report

Merging #5 (abe25e4) into master (a87c264) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
- Coverage   91.76%   91.74%   -0.03%
==========================================
  Files           5        5
  Lines         328      327       -1
==========================================
- Hits          301      300       -1
  Misses         27       27

Impacted Files
Coverage Δ

src/fast_scram_definitions.erl
97.82% <0.00%> (-0.05%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a87c264...abe25e4. Read the comment docs.


using SysVinit instead of systemd

Is it a possible thing to replace systemd with sysvinit ? If it is possible, please provide the right procedure. I'm unable to connect to the XFCE desktop via RDP.
I've tried enabling and starting xrdp-sesman.service and xrdp.service services several times... the xrdp window opens up but fails to connect
I'm on WSL1
If i'm missing any additional steps with xrdp, please guide me

Please delete the issue

@VPraharsha03 Have you had any luck with starting xrdp?


NO-TICKET-fix-dependencies

Refactor from requirements.txt to setup.py
Discussion document:
https://docs.google.com/document/d/1xvGUt3lfZAAh88Jh7meg6p4jEt2smUEKcp7Wm7lZrj0/edit

This feels like a nicer solution than having to bubble the dependency up to the apps that use utils.  It does mean that dependencies work differently here than everywhere else though...  would be good to hear from @allait on which option he prefers.

This will need a patch version bump too.

I've dropped a ticket on the tech debt board to refactor this in all branches but I'm thinking doing it here first is a good idea. This is the package that is most imported by our apps.

I don't think the explanation in that document is correct. setup.py obviously knows about the notifications-python-client package since it's trying to install it to begin with. The error happens because 4.0.0 package version is not in PyPI, which is where setup.py would look since it only has the package name.
There's a long history behind pip's decision to disable dependency_links, but most of it boils down to the view that libraries shouldn't control where their dependencies are coming from, since this creates conflicts (eg your app depends on libraries a and b, both of which depend on c, but they pull it from different places, so it's not guaranteed to be the same).
Now, that's not really an issue for us, since we control both the library and the app and we don't depend on conflicting things. However, since pip has dropped dependency_links support in the past, we had to adopt the approach of copying our dependencies to the app level, as that's the place those conflicts can be explicitly resolved.
We don't need to move everything to setup.py in order to get dependency_links - we can still parse them out of requirements.txt like we used to https://github.com/alphagov/digitalmarketplace-utils/commit/508f55e (this was later removed https://github.com/alphagov/digitalmarketplace-utils/commit/7efa01d after pip has deprecated the support). I'd say the reason to keep requirements.txt around is that that's what apps do and it makes it easier to keep Makefiles / travis config / build pipelines similar.
Another problem I see with this though is that while we can change how travis installs dependencies, we don't do the same during the actual deployment. Elastic Beanstalk runs requirements.txt on its own, and chances are it doesn't include this flag. This doesn't apply to scripts, but it does mean that this approach probably wouldn't work anywhere else. And it probably won't work if we ever decide to move scripts to run from PaaS using CloudFoundry tasks.
So it feels like we're creating a few problems while trying to remove a duplicate requirements.txt line. While I don't necessarily agree with pip's design choice, it does feel like fighting against it is more trouble than it's worth.

And it looks like Travis is running an older version of pip, which still marks the process-dependency-links flag as deprecated.

setup.py obviously knows about the notifications-python-client package since it's trying to install it to begin with

The only thing it obviously has is a string that it passes to Pypi.
We can't get this library from Pypi because it currently doesn't support 2.x.

we can still parse them out of requirements.txt like we used to

Have a look at the demos I set up. There's one that shows how the old way fails. It fails even with --process-dependency-links all over the place.
git clone https://github.com/benvand/a_old_test.git
cd a_old_test
virtualenv venv
source venv/bin/activate
pip install --process-dependency-links -r requirements_for_dev.txt

Another problem I see ... this approach probably wouldn't work anywhere else

It's not our job to write hacks to integrate with other platforms. We can change EB and we can choose not to migrate to PaaS/CloudFoundry if it isn't suitable for our needs.

it looks like Travis is running an older version of pip, which still marks the process-dependency-links flag as deprecated

The deprecation warning is still in place for --process-dependency-links but since 2014 all the team have done is add functionality to support it https://pip.pypa.io/en/stable/news/

I don't think the explanation in that document is correct

I disagree. I've done a lot of research putting together all the tests, documentation and the PR
http://stackoverflow.com/questions/14399534/how-can-i-reference-requirements-txt-for-the-install-requires-kwarg-in-setuptool#comment34497476_16624700

Not having a dig, I just didn't put it all together with the sole purpose of misleading people. :)

Sorry, I probably should've been more clear. This is the part of the doc that I disagree with:

But anything in the requirements.txt which also had a git repo dependency would be handled with python setup.py install, parse_requirements wouldn’t work, str(r.req) would return ’None’ and finallythe No matching distribution error would be raised.
In short anything in a requirements file is downloaded and setup.py install is run on it.

For both the scripts repo and the examples you've set up running pip install would trigger setup.py for all dependencies. It never uses any of the nested requirements.txt files directly (only whatever was parsed into setup.py), and since the error explicitly mentions failing to install the package my conclusion is that it does indeed parse correctly. The problem is that the referenced packaged version is not available on PyPI.
Which is where parsing dependency_links comes in.

Have a look at the demos I set up. There's one that shows how the old way fails. It fails even with --process-dependency-links all over the place.

I couldn't find this example in the doc, but for --process-dependency-links to work you'd need to populate dependency_links in setup.py, which we don't do at the moment. It's easy enough to add however (again, we've done this before https://github.com/alphagov/digitalmarketplace-utils/commit/7efa01d) without having to move everything to setup.py.

I've done a lot of research putting together all the tests, documentation and the PR

I didn't mean to question this at all. But I'm not coming into this from the outside either. We've spent time investigating these things before, so I'm trying to bring context for why these decision have been made.

It's not our job to write hacks to integrate with other platforms. We can change EB and we can choose not to migrate to PaaS/CloudFoundry if it isn't suitable for our needs.

While it's true that all of those things can be changed, I think we're losing perspective at this point. The problem we have right now is that the current approach requires us to explicitly list non-PyPI dependencies in the top-level (application or script) requirements.txt. I don't think that's a reason to throw away our entire current and future app and test infrastructure. And as it stands right now, the proposed solution would require us to hack our way on Travis, EB and PaaS.
For what it's worth – locking all dependencies (including versions and source location) on the app level is not a rare approach. That's what ruby does with Gemfile.lock and that's essentially the reason requirements.txt exists (which also explains Donald's position on not parsing requirements.txt into setup.py). So the python approach would be to drop git URLs from dmutils entirely, except for testing purposes. But it adds a lot of overhead and doesn't give us much in this case, so we compromise.

With your solution we're attempting to support (and have, by the looks of it, already failed multiple times at supporting) 2 different install methods.
pip install -r requirements.txt and
python setup.py install
We're relying on a paradigm that we've been warned not to use by a pip maintainer.
We're leaving obsolete requirements files in place just to support multiple dependency gathering methods.
Or we're going to ship a module that cannot fulfil its own dependencies?
For the sake of changing a few install scripts.
I just cannot see any benefit over continuing to support requirements.txt by further hacking the setup.py when we can just drop the dependencies in setup.py and be done with it. It's already resulted in unexpected behaviour more than once.
Doing this:
dependency_links = filter(
     lambda x: x != 'None',
     [str(r.link if hasattr(r, 'link') else r.url) for r in requirements]
 )

Just further shows that we should be putting stuff in the setup.py. It also obsoletes one of these lines:
https://github.com/alphagov/digitalmarketplace-utils/pull/293/files#diff-354f30a63fb0907d4ad57269548329e3L6
It's just as much effort to put the dependency_links hack at the top of the setup.py and remove whichever of these lines becomes redundant:
 .travis.yml
  python:
    - "2.7"		    - "2.7"
    - "3.4"		    - "3.4"
  install:		  install:
 -  - pip install -r requirements_for_test.txt
 -  - pip install -e .

As it is to move the requirements to the setup.py except we improve readability, simplicity, code-cleanliness, ease of support and reduce tech-debt all at the same time.
If pip is going to use setup.py anyway why don't we? We don't need to support 2 ways of installing dependencies.

I don't think that's a reason to throw away our entire current and future app and test infrastructure

I don't get this. We're not throwing away anything.


Fix Coverity 1498611 & 1498608: uninitialised read

[ ] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge

Merged to all three branches, thanks for the review.


Different formatting for Arithmetic and Logical Operators

In tests/target/issue-831.rs a Logical operation in multiline expression is formatted as:
fn main() {
    let y = a.iter().any(|x| {
        println!("a");
    }) || b.iter().any(|x| {
        println!("b");
    }) || c.iter().any(|x| {
        println!("c");
    });
}

while formatting of a similar expression with Arithmetic operation in tests/target/expr.rs  is formatted as:
fn issue_2802() {
    function_to_fill_this_line(some_arg, some_arg, some_arg)
        * a_very_specific_length(specific_length_arg)
        * very_specific_length(Foo {
            a: some_much_much_longer_value,
        })
        * some_value
}

Why is the || operation added to the  }) line, while the * is added in a new line?  If this is in purpose, what is the logic behind the different formatting approaches?

This formatting is correct. Yes, both snippets have binary expressions, but note that the first example has closures which have their own formatting rules.
Particularly of note for closures is the required indentation for closing braces to adhere to block formatting rules, while the rules for binary expressions require block indentations for the operators and sub exprs on a new line.
We couldn't indent the closing closure brace from your first snippet like below as that would violate closure formatting:
fn main() {
    let y = a.iter().any(|x| {
            println!("a");
        }) || b.iter().any(|x| {
       .......

and we couldn't put the binary operator and next subexpr on the next line without indentation like this (would break binop rules):
fn main() {
    let y = a.iter().any(|x| {
        println!("a");
    })
    || b.iter().any(|x| {
        println!("b");
    })
    || c.iter().any(|x| {
        println!("c");
    });
}

and if you tried to still put the op and subexpr chains on their own line with a block indentation you'd end up with something wild like this, which would in turn break other indentation rules
fn main() {
    let y = a.iter().any(|x| {
        println!("a");
    })
        || b.iter().any(|x| {
            println!("b");
        })
            || c.iter().any(|x| {
                println!("c");
            });
}

Remember that even though two top level expressions may be the same, the entire AST subtree impacts their respective formatting due to the respective rules which is why there's formatting differences when those top level expressions contain different elements.

Thanks for the detailed repose.


Geoportal Server 2.6.1 in Disconnected environment with Argis Enterprise 10.6.1

In a VM disconnected from internet with OS WIN2012 R2 installed, I did a full install from ArcGIS Enterprise 10.6.1 (all the services Portal and ArcGIS Server are working well). After that I installed java jre and jdk version 8u-161-windows x64,  Tomcat apache 5.8.33, Elasticsearch-6.6.1 and Geoportal Server 2.6.1. At each step I did tests to see if the componenst are working. For the geoportal-server-catalog I followed https://github.com/Esri/geoportal-server-catalog/wiki/Installing-Geoportal-Server-2.6.1 and https://github.com/Esri/geoportal-server-catalog/wiki/Configure-geoportal-for-a-disconnected-environment but unsuccessful.
Result when I try to go to https://:8080/geoportal, I see only this:
 - only the progressbar defined in index.html.
I did the same configuration in a VM connected to internet and all is working fine.
Some Idea?

Any messages in the javascript console?
Chrome: right three vertical dots, more tools>developer tools, then click console.

All the messages to the links are corrected one by one. There are three open comments concerning the fonts.

Issue is not closed.

I have following messages, all related to index.html file

The Avenir_Next_W01_300.woff file exist in folder C:\inetpub\wwwroot\arcgis_js_v325_api\arcgis_js_api\library\3.25\3.25\esri\themes\calcite\fonts\avenir-next and the and the Avenir_Next_W01_300.woff2 file exist in folder C:\inetpub\wwwroot\arcgis_js_v325_api\arcgis_js_api\library\3.25\3.25\esri\themes\calcite\fonts\avenir-next ==> I suppose the fonts exists but are not loaded.
I changed all stylesheet references as:

did you add .woff2 as a known MIME type in IIS?

No. I can do it.
Le ven. 15 mars 2019 à 23:48, Marten<EMAIL_ADDRESS>a écrit :

did you add .woff2 as a known MIME type in IIS?
[image: image]
https://user-images.githubusercontent.com/394890/54465711-c9735e80-4739-11e9-87df-d14424242b2d.png
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Esri/geoportal-server-catalog/issues/257#issuecomment-473464741,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AND0Sy3dsYkTe5iWwbB0s1_DTcSH5T6bks5vXCNFgaJpZM4b0mGu
.

.woff was existing in IIS, I add .woff2 but no result change

perhaps an obvious one, but did you try opening the URL to the fonts directly in the browser?

Yes. The result is the download from the woff file. I try all the URL changes one by one in the web browser. All URL are positive except one and it is at Tomcat proxy.jsp.

wait, the link ends with .woff? not .woff2? both flavors exist. also add the .woff type to IIS. see this article on the issue and the resolution:
https://hotcakescommerce.zendesk.com/hc/en-us/articles/210926903-HTTP-404-Not-Found-Error-with-woff-or-woff2-Font-Files

Hi Marten, I did what was explain at the article but the issue is not resolved. Please find in attach the last log files from Tomcat. It is an access to folder problem, is it possible that this is a certificate issue?
Other question: is it necessary to install a IIS to tomcat connector?
localhost_access_log.2019-03-25.txt
Remarks with developer tool.txt
tomcat8-stderr.2019-03-25.log
tomcat8-stdout.2019-03-25.log
catalina.2019-03-25.log
commons-daemon.2019-03-25.log
geoportal.log
localhost.2019-03-25.log

a colleague ran into the same issue and then I noticed he (and you so it seems) don't have IIS front your Tomcat. Here is an article that shows how to get the mime types defined in Tomcat web.xml:
https://gist.github.com/alexanderbartels/831ae012245cbf633bc6
you'll need to do this for woff and woff2

Marten, sorry no change to do. All this mime-mapping rules exist at the web.xml file, except .woff2. No change from result?
With more deep investigate, I can see following error:
Failed to load resource: net::ERR_NAME_NOT_RESOLVED ===> 9431e9a5-7a6d-4b5f-82be-4d935972de5a.css:1
Failed to load resource: the server responded with a status of 404 () ===> ags.esribelux.eu/:1
Access to font at 'http://ags.esribelux.eu/arcgis_js_v325_api/arcgis_js_api/library/3.25/3.25/esri/themes/calcite/fonts/avenir-next/Avenir_Next_W01_300.woff2' from origin 'http://ags.esribelux.eu:8080' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. ==>(index):1
Failed to load resource: net::ERR_FAILED ==> Avenir_Next_W01_300.woff2:1
Access to font at 'http://ags.esribelux.eu/arcgis_js_v325_api/arcgis_js_api/library/3.25/3.25/esri/themes/calcite/fonts/avenir-next/Avenir_Next_W01_300.woff' from origin 'http://ags.esribelux.eu:8080' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. ==> (index):1
Failed to load resource: net::ERR_FAILED ==> Avenir_Next_W01_300.woff:1
Failed to load resource: the server responded with a status of 404 () ==> ags.esribelux.eu/:1
Uncaught ReferenceError: require is not defined ==> (index):48
at (index):48
Status code 404 ==> access problems to resolve between tomcat and IIS. Is there a solution to use a IIS Tomcat connector?

there could be a CORS issue accessing ags.esribelux.eu from port 8080.
I do notice that I get a 404 when accessing the link to the woff2 font and as well when accessing the server top address http://ags.esribelux.eu

You can not access http://ags.esribelux.eu, it is a full disconnected environment.

Marten, I did all what you suggested, but without results. To clarify the situtaion I have defined to VM the first with arcgis enterprise ags.esribelux.eu and the second with the geoportal server (elasticsearch, tomcat and geoprtal server) gps.esribelux.eu.
I continue to receive this errors:

All is linked to the XML file in attach
index.txt

Marten, Due to the problems communication between the websites, I installed the geoportal.war in IIS 8.5 (delivered with ArcGIS Server 2012 R2. I have no more problems with cors and fonts.
But I have two errors in the index.html file and no results actually: I can see only the banner. Can you investigate this?

Also here the index.html file saved in txt format
index.txt

I bet that if you open the link to the JS API you get something similar to what I get:
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

<title>404 Not Found</title>

Not Found

The requested URL /arcgis_js_api/library/3.25/3.25 was not found on this server.

that server is not entirely disconnected from the web. my request to https://gps.esribelux.eu/arcgis_js_api/library/3.25/3.25 makes it to the server (or at least to some proxy before it and it responds with 404).
if your geoportal is on the same machine as the JS API, can you use an internal address like localhost? or even a relative path like /arcgis_js_api/library/3.25/3.25?

Marten,
Absolutely false, I see next window:

I can access the api library with servername, localhost and IP adress from server gps.esribelux .eu and with servername and IP adress from server ags.esribelux.eu.
From my host I can ping it, but I can't access them: the ethernet connector is active, but is not allowed to internet access.

I forgot to mention tyhat effectively the geoportal server is on the same machine as the JS API 3.25., I have also installed Java jdk1.8.0_161.
I have stopped Tomcat, while I have activated the GeoPortal through IIS.

then you should be able to use a relative path even to the JS API that would ideally avoid the request going out to some sort of network proxy before going back to the server. from that machine itself or from your client machine, can you open that JS API url?

Direct relatif path to JS API as //ArcGIS_js_api/Library/3.25/3.25 or \arcgsi_js_api\library\3.25\3.25. When I give a relative path //gps.esribelux.eu/arcgis_js_api/library/3.25/3.25/ he goes immediately to http://gps.esribelux.eu/arcgis_js_api/library/3.25/3.25/ and access the directory.
Provenance : Courrier pour Windows 10
De : Marten
Envoyé le :01 April 2019 19:38
À : Esri/geoportal-server-catalog
Cc : MarcSimon007; State change
Objet :Re: [Esri/geoportal-server-catalog] Geoportal Server 2.6.1 inDisconnected environment with Argis Enterprise 10.6.1 (#257)
then you should be able to use a relative path even to the JS API that would ideally avoid the request going out to some sort of network proxy before going back to the server. from that machine itself or from your client machine, can you open that JS API url?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

@umacgillivray do you have suggestions for marc?

I did a fast test with the relative path but the errors are the same as before
Provenance : Courrier pour Windows 10
De : Marten
Envoyé le :01 April 2019 20:26
À : Esri/geoportal-server-catalog
Cc : MarcSimon007; State change
Objet :Re: [Esri/geoportal-server-catalog] Geoportal Server 2.6.1 inDisconnected environment with Argis Enterprise 10.6.1 (#257)
@umacgillivray do you have suggestions for marc?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

after discussing with Urban a bit. the URL to the JS API should return the init.js content, not a browse directory view of the JS API. similar to when you open this: https://js.arcgis.com/3.22
another option is if you have Portal for ArcGIS in the disconnected environment, since that also has the JS API.

If you will attack directly de init.js you need to specify Following URL //gps.esribelux.eu/ArcGIS_js_api/Library/3.25/3.25/init.js.
The use from the JS API is not a good path, I have separate expressly both, will there was een interaction between de API’s. By the installation from the Geoportal Server I lose systematically the active services from the Portal for ArcGIS. To activate them at new, I neede to uninstall the Geoportal server and Tomcat. Also be carefull with this option, will the default JS API to use with ArcGIS Enterprise disconnected is version 4.10.
Provenance : Courrier pour Windows 10
De : Marten
Envoyé le :01 April 2019 21:03
À : Esri/geoportal-server-catalog
Cc : MarcSimon007; State change
Objet :Re: [Esri/geoportal-server-catalog] Geoportal Server 2.6.1 inDisconnected environment with Argis Enterprise 10.6.1 (#257)
after discussing with Urban a bit. the URL to the JS API should return the init.js content, not a browse directory view of the JS API. similar to when you open this: https://js.arcgis.com/3.22
another option is if you have Portal for ArcGIS in the disconnected environment, since that also has the JS API.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

If you have some other suggestions, feel free to send them. I will test them as fast as possible;

Perhaps obvious, but you did update the value of:
https://[HOSTNAME_AND_PATH_TO_JSAPI]dojo

after you moved the JS API?

The update init.js and dojo.js is done. To avoid any conflicts I started a completely new setupfromboth servers and I follow the documentation to check all the updates to do.
Provenance : Courrier pour Windows 10
De : Marten
Envoyé le :01 April 2019 23:13
À : Esri/geoportal-server-catalog
Cc : MarcSimon007; State change
Objet :Re: [Esri/geoportal-server-catalog] Geoportal Server 2.6.1 inDisconnected environment with Argis Enterprise 10.6.1 (#257)
Perhaps obvious, but you did update the value of:
https://[HOSTNAME_AND_PATH_TO_JSAPI]dojo
after you moved the JS API?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

After adaption IIS and INit.js and referncing all the sources locally, Geoportal window opens. But still 6 errors: 5 for fonts and one for search.
Global five errors: 6 fonts and 1 search
Fonts not recognised:

Avenir_Next_W01_300.woff
Avenir_Next_W01_300.woff2
Avenir_Next_W01_400.woff
Avenir_Next_W01_400.woff2
CalciteWebCorelCons.ttf?r2lind
CalciteWebCorelCons.woff?r2lind

IIS MIME and web.wml at Topmcat is adapted, but it don't resolve the problem. Bypass error mention is possible with disable chrome security !!! ==> run chrome.exe --user-data-dir="C:/Chrome dev session" --disable-web-security, but chrome not allowed as official web browser.
Question: is it possible to download somewhere this fonts and to referncing them local?
What means this error: SearchPane.js:170 d {message: "Unable to load ./elastic/metadata/item/_search?fro… an error in handleAs: transformation of response", response: {…}, stack: "Error: Unable to load ./elastic/metadata/item/_sea…/arcgis_js_api/library/3.25/3.25/init.js:143:341

Please,
Can somebody explain me where I need to change the basemap references to see a map through the geoportal searchpanel URL http://cdcwtstgis03.idcn.mil.intra:8080/geoportal/#searchPanel and through the geoportal Mappanel URL http://cdcwtstgis03.idcn.mil.intra:8080/geoportal/#mapPanel. I don't receive any response from a map. Are there some particular preparation to do.

for the search map, you can change the basemap reference in  geoportal\app\context\app-config.js\app-config.js, e.g.     basemap: "http://myagsserver.esri.com/arcgis/rest/services/SampleWorldCities/MapServer",
for map viewer it would involve first set up local Portal for ArcGIS  instance, create a web map that is using the local map service created from AGS, then update \geoportal\viewer\config.json to use the web map created in Portal.
"portalUrl": "http://localagpportal.com",
"geometryService": "https://localagpportal/arcgis/rest/services/Geometry/GeometryServer",

"itemId": "6e03e8c26aad4b9c92a87c1063ddb0e3",

Close it for now, we can open a new item if it is still issue.


Fix for modifying objects via browser

it wasn't exactly a typo as it was something I forgot to change when porting the code from Risk of Rain Returns


Drop single quotes to make warnings for parquet options not confusing

Which issue does this PR close?

Closes #7867
Rationale for this change

Previous warnings could be confusing and it was decided to drop single quotes to fix this.
What changes are included in this PR?

Warnings were fixed
Are these changes tested?

Tested with cargo test locally.
Are there any user-facing changes?

Minor change in warnings

Thank you @qrilka


Remove windows_share and windows_certificate resources

https://github.com/chef-cookbooks/windows#deprecated-resources-note
We moved windows_share and windows_certificate into core for Chef 14.7. We had announced we would remove those from this cookbook late last year but didn't get to it. Let's remove them now and ship a v7 of this cookbook.
Chef 14 will be EOL in 6 weeks. Probably we  should bump the minimum version of Chef to 15 too.
This also will allow us to remove win32-certstore as a gem dependency on this cookbook.

Note: Remember to remove the deprecated path helper when a v7 is released.


Issues do not use the published labelling scheme yet.  We need to make it so.

for others like me who didnt see the readme change, it's the roslyn label list https://github.com/dotnet/roslyn/wiki/Labels-used-for-issues

let's use more the label  :smile:


[Question] Is there a way to insert column after a letter and moving rest of the columns accordingly.

I am trying to insert a new column in an existing excel, is there a way to achieve it.
I see addRows function but not addColumns.

See colomus, You can try the following code.
(async () => {
  var workbook = new Excel.Workbook();
  // var worksheet = workbook.addWorksheet('My Sheet');
  // worksheet.getColumn(6).values = [11,21,31,41,51];
  // await workbook.xlsx.writeFile('./ownerTest/demoTest/read/issue1274.xlsx');

  await workbook.xlsx.readFile('./ownerTest/demoTest/read/issue1274.xlsx');
  var worksheet = workbook.getWorksheet('My Sheet');
  var indoc = worksheet.getColumn(6);
  // indoc.values = indoc.values.concat([55, 566, 88]);
  worksheet.spliceColumns(6, 1, [55, 55, 99, 88]);
  await workbook.xlsx.writeFile('./ownerTest/demoTest/read/issue12741.xlsx');
})()

Thanks for opening this request.
Please re-open if you still have issues.
Best of luck.


380 traceback partition

Fixes https://github.com/kairos-io/kairos/issues/380

Though this should fix the original problem, I'm keeping this PR in draft while I'm trying to setup a test suite on this project. I may split it on another PR if it proves to be too much work.

with this we don't store anywhere the partition mapping right?

with this we don't store anywhere the partition mapping right?

Right. Looking at the code, I think all 3 headers will be available all the time, so we take the first that matches. But I need to verify this in a final booted iso. Up to now I've only tested it by running the challenger binary locally.

It won't be always the case - only the label is the 'minimum' that is set from the user - all the rest is generated on the node after install

with this we don't store anywhere the partition mapping right?

Right. Looking at the code, I think all 3 headers will be available all the time, so we take the first that matches. But I need to verify this in a final booted iso. Up to now I've only tested it by running the challenger binary locally.

this is fine for local iteration but It won't be always the case - only the label is the 'minimum' that is set from the user - all the rest is generated on the node after install

True. Still the code would still work then. If there is only the label, it will use the label.

with this we don't store anywhere the partition mapping right?

Right. Looking at the code, I think all 3 headers will be available all the time, so we take the first that matches. But I need to verify this in a final booted iso. Up to now I've only tested it by running the challenger binary locally.

this is fine for local iteration but It won't be always the case - only the label is the 'minimum' that is set from the user - all the rest is generated on the node after install

True. Still the code would still work then. If there is only the label, it will use the label.

This change is fine overall indeed, but we need to also save the state somehow otherwise if we specify the label only when applying  the spec  - install would work, but first boot after install won't


Restart individual scripts?

Scripthookvdotnet has a way to restart all the script but is there a way to restart an individual script when a button defined by the developer is pressed?

There is Script.Pause() and Script.Resume(). You cannot fully restart a script from script code, unless you use script instances (see Script.InstantiateScriptandScript.Abort`).

There is Script.Pause() and Script.Resume(). You cannot fully restart a script from script code, unless you use script instances (see Script.InstantiateScript and Script.Abort).

HI again. I've tried using Script.InstantiateScript and Script.Abort then set a function to Instantiate the script again but with no success. This was yesterday and I didn't had time to copy paste the error. I will post later a log if needed. This if the method I tried to use is the correct one.

Also why so many separated versions and not just one? I've got some problems with some other libraries like NativeUI where some methods are not available with scripthookvdotnet v3. One of them is setting custom colors because SHVDN 3 didn't had a reference to Color. @crosire

Hey you could see if Example ScriptInstance helps.
And for Color are you using System.Drawing;

Semi related to this question - is it possible to restart all scripts via code as opposed to manually hitting a key? Or pause/resume other scripts from your own script?

@BenSmith123 You can only pause / resume scripts you own.... you can not effect other mods.

Is it possible to read data between scripts? I want to use the data of another in my script.

Yes as long as you own both scripts.
Just refer to the namespace.

I think separating AppDomain per script assembly enables to reload an individual script (#1162), just like RPH does.


Grid size controls should have an upper limit based on screen/container size

Currently, the control to change the verification grid size has a fixed maximum of 36.
This is irrespective of the user's screen size and the size of the verification grid container.
This means that users can see and set grid sizes that are unreasonable.
e.g. A mobile user can set a grid size of 36

To fix this, the dynamic-grid-size.ts class should advertise the maximum number of grid cells that can fit in the verification grid's container.

Included in this issue:
If the grid size is forcefully increased past the screen maximum (by manually overriding the grid-size HTML attribute) we should display a warning that some items have been truncated


Batching with fuse_layers = True leads to different outputs

Text:
text = ['Short summary of self-attention:', 'Short summary of Ukraine war:', 'Short summary of relativity:']
Model:
model = AutoAWQForCausalLM.from_quantized(model_path, fuse_layers = True, safetensors = True)
Processing them sequentially (one at a time) leads to the outputs:

Self-attention is a mechanism that allows models to weigh the importance of different parts of input when making predictions. It does this by computing attention scores between each pair of inputs, and then using these scores to compute weighted averages of ...

The conflict in eastern Ukraine began in 2014, after the ousting of President Viktor Yanukovych. Pro-Russian separatists seized control of several cities and regions in the east, forming two self ...

Relativity is a theory in physics that explains the laws of space and time. It was developed by Albert Einstein, who proposed two theories: Special Relativity (1905) and General Relativity (1915).

But if you batch these together:

Self-attention is a mechanism that allows models to weigh the importance of different parts of input when making predictions. It does this by computing attention scores between each pair of inputs, and then using these scores to compute weighted averages of ...

The Ukrainian conflict began in 2014, when pro-Russian separatists seized control of government buildings in several cities in eastern Ukraine. This led to a military intervention by the Ukrainian army and ultimately a cease ...

The laws of physics are the same for all observers in uniform motion relative to one another (principle of relativity).
The speed of light is always constant, regardless of the motion of its source or observer

So essentially all but the first in the batch have different outputs than expected. This problem only happens when fusing = True.

#215 should resolve this. I need to test it more to make sure it’s correct.
Can you drop an example?

Yeah tried this and it didn't work

Yeah, I am not sure the new PR was a good fix. I have to assess this further before the next release. Have you been able to identify where the issue could be?

So the issue seems to be primarily with padded tokens/attention masks, which then leads to the incorrect outputs for batched inputs. Example:
text = ['Short summary of self-attention:']
No padding

Relativity is a theory in physics that explains the laws of space and time. It was developed by Albert Einstein, who proposed two theories: Special Relativity (1905) and General Relativity (1915).

Padded to 20 tokens

The theory of general relativity is a theory in physics that describes the laws of gravity and their relation to other forces of nature. It was developed by Albert Einstein in 1915, and it has been extensively tested and confirmed

Padded to 30 tokens

The theory of general relativity is a theory in physics that describes the laws of physics in terms of space and time. It was developed by Albert Einstein, who published his theory in 1915. The theory explains how gravity works

I think the problem is that the forward method
https://github.com/casper-hansen/AutoAWQ/blob/9c3dfa078f82b164d886c1378784e8d1510a2a97/awq/modules/fused/model.py#L22
is not using the attention_mask from the tokenizer, but rather creating a default diagonal one: https://github.com/casper-hansen/AutoAWQ/blob/9c3dfa078f82b164d886c1378784e8d1510a2a97/awq/utils/fused_utils.py#L40-L48

It looks like the main problem here is that you are not passing in the input_ids and instead the output of the tokenizer. Please see the example below:
https://github.com/casper-hansen/AutoAWQ/blob/main/examples/basic_generate.py#L23-L26

No this is not the issue. I am passing both the input_ids and attention_mask from the tokenizer to generate(). The reason your example above works is because:

There is no batching
As a corollary, there is no padding (this is where the issue arises)

To explain more clearly:
If I want to pad an input text (as is needed in batching):
text = ['Short summary of relativity:']
model_inputs = tokenizer(
    text, padding='max_length', max_length = 10, return_tensors="pt").to('cuda')

print(model_inputs)

we get 3 padding tokens (2) and 7 prompt tokens:
{'input_ids': tensor([[    2,     2,     2,     1, 11530, 14060,   302,  1016, 28283, 28747]],
       device='cuda:0'), 'attention_mask': tensor([[0, 0, 0, 1, 1, 1, 1, 1, 1, 1]], device='cuda:0')}

The problem is, this attention_mask is not being used/transformed during the prefill https://github.com/casper-hansen/AutoAWQ/blob/8110e028c7fe496287d9092d2255f3b7fa6bdd2d/awq/utils/fused_utils.py#L18-L26
so model.generate(**model_inputs) has the same output as model.generate(input_ids = model_inputs['input_ids']).
Essentially this is just being treated as a prompt of </s></s></s><s> Short summary of relativity:.

We need to implement something like this which takes into account the attention mask from the tokenizer when creating the 4d mask:
https://github.com/huggingface/transformers/blob/main/src/transformers/models/mistral/modeling_mistral.py#L892-L898

@younesbelkada any idea if this is the correct path forward?

Would be nice to have an integration with from transformers.modeling_attn_mask_utils import _prepare_4d_causal_attention_mask. Just not sure what the steps are to make it compatible as the shapes will probably be different.

I implemented something like _prepare_4d_causal_attention_mask to take into account the padding tokens. This partially fixed the issue. However, when comparing the actual vs expected tensors, it appears that two operations are causing a divergence:

The RoPE calculation
The single_query_attention calc during decoding

I need to look into these a bit further to see if I can narrow it down. Out of curiosity, how do you debug this if you can't run it locally?

I implemented something like _prepare_4d_causal_attention_mask to take into account the padding tokens. This partially fixed the issue. However, when comparing the actual vs expected tensors, it appears that two operations are causing a divergence (I'm not saying these are wrong btw, just that they don't seem to be working with the padded inputs):

The RoPE calculation
The single_query_attention calc during decoding

I need to look into these a bit further to see if I can narrow it down. Out of curiosity, how do you debug this if you can't run it locally?

The RoPE calculation should not be affected, but the attention could be. If you want to make it compatible, you would have to reshape the attention mask so that it fits the correct dimensions for these CUDA extensions to work.
For debugging, I usually rent from RunPod and SSH through VS Code.

Cool, will try and look into this


To test [PX4/NuttX] [BACKPORT] STM32F7 and STM32H7 SDMMC internal pull up usage fixed (#78)

This is to test the F7 and H7 boards in the PR

Code was flawed in that the Pins are defined with the
pullups in the definition. Since there are no alterntes pins
there is no way to remove them. So not enabling the CONFIG
pull up did nothing as did enabling them.

This is now a compile time feature as there was no reason for
it to be a run time. It wasted both flash and ram.

The code ignored the use of D0 for ready detection causing
3X+ the chatter.

No Internal PU and 100K External Note 2.5 us of chatter

With Internal PU and 100K External Note 0.7 us of chatter

Flash & RAM Prior PR
            itcm:          0 GB      2016 KB      0.00%
           flash:     1713705 B      2016 KB     83.01%
            dtcm:          0 GB       128 KB      0.00%
           sram1:       44616 B       368 KB     11.84%
           sram2:          0 GB        16 KB      0.00%

Flash & RAM This PR
Memory region         Used Size  Region Size  %age Used
            itcm:          0 GB      2016 KB      0.00%
           flash:     1713677 B      2016 KB     83.01%
            dtcm:          0 GB       128 KB      0.00%
           sram1:       44616 B       368 KB     11.84%
           sram2:          0 GB        16 KB      0.00%

Tested over 300 arm/disarms to create logs, had NO errors on our hardware with this mod.  Before this mod we'd get failures frequently, so looking good on my end.

looks the same, did you copy wrong data?


Cannot stand up iRODS 4.2.11 on Ubuntu 18 / MySQL 8

This results in MySQL 8 reporting an error about the row size being too large.

Closing.
See https://github.com/irods/irods_testing_environment/pull/155#issuecomment-1443796191 for an explanation.


Remove my password from lists so hackers won't be able to hack me

This is a security hole. This pull request should be accepted as soon as possible.

I'm also affected by this, please merge ASAP

@assafnativ @rooterkyberian could you provide any testing data like service addresses and logins so we could check and test to estimate the real impact of this change?

ROTFLMAO!

What the.....i don’t think this will solve the issue
On Thu, 21 Dec 2017 at 19:33, Krzysztof Staniorowski <
<EMAIL_ADDRESS>wrote:

ROTFLMAO!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/danielmiessler/SecLists/pull/155#issuecomment-353339873,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADo_cTu0FfmnSQSxs34YcAbHMKk11h4vks5tClAygaJpZM4RJt0h
.

@mitcom you mean, like the publicly available email address and blog address on his github page?

@assafnativ They see me trollin, they hatin...

I think it goes with out saying:

4 random words are really easier than the gibberish?

@KyrychukD wtf

Can you please add my password
dolphins
To this list so I can test it against insecure services..

If anybody here is affected too I can suggest temporally change the password to one from https://mostsecure.pw/

Is dolphin1 on the list.  ;) That's secure as it has a 1

Dolphin1!

Ah good idea, hackers will never try that..

Same here.

Profit

Is my password hunter2 safe

@dsuurlant I just see *******

is my password thisissparta safe????????

is my password thisissparta safe????????

Absolutely, if changed!

This is gold.
Sent from my iPhone

On Dec 21, 2017, at 10:39 AM, Kishan Kumar<EMAIL_ADDRESS>wrote:
is my password thisissparta safe????????
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Change it to dolphins, dolphins is safe now
21.12.2017 16:42 "Thaddée Tyl"<EMAIL_ADDRESS>napisał(a):

is my password thisissparta safe????????
Absolutely, if changed!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/danielmiessler/SecLists/pull/155#issuecomment-353382799,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACgiM34NXJyW-4C4DTitmf0OrwdXCd9Mks5tCnxqgaJpZM4RJt0h
.

nice, my 122112 password still alive...

At least I know Alligator1 will never be guessed.

@assafnativ, you had the same password as mine?

@assafnativ, you had the same password as mine?

@0xmohit not anymore, I've just change yours

troll has to be removed.
I like to use a self-description of my online behaviour as my password - I guess a$$hat isn't in the list so that's a ➕

Can somebody please remove dadada from the lists?

You can use dolphin as that is now officially safe

hey @assafnativ - I just checked and you password is working fine.

poor dolphins, how you hurted them now :<

Phew! Luckily nobody mentioned 'qwerty' here. I'm still safe.

Nice PR Please merge ASAP

This is amazing :D

@phra you forgor
thisissparta
Alligator1
hunter2
122112```

I am surprised gullible is missing from the list.

Hey Reddit!

LGTM 😄

Publish all the passwords

Is this where we type our password so it becomes unhackable? Asking for a friend.

Type your password in the comments box, github automatically stars it out ********.

@barkerd427 , yes unless it's dolphins as we already know that's secure. . .

@steadweb oh man I remember a decade back when I played Runscape, people actually fell for that!

@bweston92 habbo hotel as well, oh the poor souls.

Great! Thanks helpful internet people. ¥0uπ¶@$sW°r∆ works on approximately three systems in the world, so I normally just use SaRcAsM. The case change makes it pretty strong.

Cr. Ascii art

I would like to have the vote to save the dolphins. They didn't do anything wrong so that I really recommend to this PR for the world and humanity.

What's the porpoise of this PR?

I’m afraid that the removal of Dolphins falls under violation of UN Peace Sanction 82.62E, protecting all dolphins on the planet from removal from their homes. Removal of “dolphins” from this repository will subsequently be recognized as a declaration of war.

We could just remove the one dolphin...

I don't want to feed the original troll but I hope you won't accept this pull request, I'm still waiting for dolphins to say
"so long, and thanks for all the fish" :)

just use ******** to be on the safe side! It's 8 chars long, you are safe!

If you remove "dolphins", please replace it with "thanks for all the fish".

FYI, Dolphin != fish
On Fri, 22 Dec 2017 at 14:36, docweird<EMAIL_ADDRESS>wrote:

If you remove "dolphins", please replace it with "thanks for all the fish".
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/danielmiessler/SecLists/pull/155#issuecomment-353540440,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADo_ccTty1Da6KIH86taDbB2tfblUpvxks5tC1vqgaJpZM4RJt0h
.

Please create a new pull request with the fixes. Would be good to get better performance using this files.
Thank you :)

Now the world knows your password

10_million_password_list_top_1000_except_dolphins.txt lmao

The best PR in 2017

If you dont want change yourself, then change the world. 👍

I'm having the very same issue here. My password is qwe123, and I don't know why I'm always being hacked. So could you help me to remove my password from the list, too? sarcasm
btw, I  thought this post was from reddit.

FYI, Dolphin != fish

You seems to be <> 42

guys, sorry for being the boring guy, but you are spamming :-)

hmm, why my password not in this list ¯_(ツ)_/¯
please add this one: ␀q␀w␀e␀r␀t␀y␀

This is the best password in the world

🥇

This is a blocker please merge asap

Fixes #12345 : Cannot use SECURE password 'dolphins' as admin password because it is listed as insecure password.
This prevent one of my script from running and is a regression bug!
Once this PR is merged, we should also notify all down streams to update their database so that this bug can be fixed.

LGTM

I'm also affected by this

how?

Lol. I am sure the author of this pull request has been hacked now.

Was thinking same thing

+1 please replace with ******* so no one can see it.

Add mine: hunter2
I know you'll be seeing asterisks tho.

Hey guys, here is mine: ******. I hope none'll be able to hack me now. 😄

Best. Pull. Request. Ever. Must merge asap!

Just recommend my password 1233456 to all of you, it's safe!

It is a big issue. Please merge it ASAP

We need to do this, it is an important thing!

This needs merged ASAP. I'm shaking in the bed just thinking about my poor, poor password.

LTGM!
Merge, please!

You can make ir more secure using numbers, something like: d0lph1n5 😉

Shh! Don't tell anybody!

Mom, I am here

Better do this on the list of top 10,000 credit card pin codes as well.

Are those changes unicode safe?

Guys, I've just have a terrorful day...  Thanks to everyone :)

LOL

used this as my bank password and just lost $500,000, please merge this right now so they will have to use "Forgot Password" and I can recover my account.

Merge, Please!

Please merge as fast as possible.

Is 1,2,3,4,5,6 still good? I use that for all my passwords and pin #s

The best pull request in history of Github :1st_place_medal:

Cannot reproduce

Best PR of all time 🤣

@github Please merge this ASAP by using your super power, do not wait for the owner of the project, this is critical.

+1

This is becoming more like 9gag. Too much attention here!

LGTM

LGTM

LGTM 👍

Mergey mergey

This should be merged ASAP.

sits back and watches as this is accidentally merged one day by a new maintainer with no context, and someone chooses dolphins as the nuclear codes because it's not on these lists

@jpgoldberg BE THE CHANGE YOU WISH TO BE IN THE WORLD

Well, this is embarrassing.

huahhahahahhaa

Hey guys I just found out a great trick ! GitHub knows your password and will show it as asterisks to others ! See ******

dolphins. did that work?

dolphins2

This change breaks my workflow.

Yes @jmtame @kenanscott others see it as asterisks

Will you tag a new version of a file so I can use it? My code seriously depends on it and this is very big changes!!

LGTM

This is not real :o

Should I worry about trustno1 being unsafe?

Should I worry about trustno1?

File order is confusing, passwords should be alphabetically ordered. Also file names are not consistent and don't contain version number.

ICO when???

Can't wait for dolphin coin

:trollface:

Great catch! Please merge before it's too late. ✔

this change alone makes millions of people safer on the internet. good job :+1:

Seems legit 🤔

Fixed locally for now by changing all my passwords from 'dolphin' to 🐬
Hackers can't do emoji, right?

Does this affect whales

👍 shipit

👍 i also want to remove my own password, horses

is 12345 safe?
its the password on my luggage

This is a security hole

Surely you meant security blowhole.
I'll see myself out.

Greetings assafnativ,
It's better if you just change your passwords.

LGTM, Please Ship ASAP

my password:
hunter2
Does this appear as stars for everyone? Pls no hackz. /s
@danielmiessler this is a joke, please close it before it accidentally gets merged or treated seriously.

What's the CVE ID of this?

This patches multiple security issues and needs to be implemented immediately

EASILY GUESSABLE PASSWORDS ON AN OPEN FIELD NED!

:joy::joy::joy:

666

Don't forget to run the database migrations, just did it now it works!

~/bin/tools/cred$ python2 Cr3d0v3r.py<EMAIL_ADDRESS>

              _v->#H#P? "':o<>\_
          .,dP` `''  "'-o.+H6&MMMHo_
        oHMH9'         `?&bHMHMMMMMMHo.
      oMP"' '           ooMP*#&HMMMMMMM?.
    ,M*          -     `*MSdob//`^&##MMMH\
   d*'                .,MMMMMMH#o>#ooMMMMMb
  HM-                :HMMMMMMMMMMMMMMM&HM[R\
 d"Z\.               9MMMMMMMMMMMMMMMMM[HMM|:
-H    -              MMMMMMMMMMMMMMMMMMMbMP' :
:??Mb#               `9MMMMMMMMMMMMMMMMMMH#! .
: MMMMH#,              "*""""`#HMMMMMMMMMMH  -
||MMMMMM6\.                    [MMMMMMMMMH'  :
:|MMMMMMMMMMHo                 `9MMMMMMMM'   .
. HMMMMMMMMMMP'                 !MMMMMMMM    `
- `#MMMMMMMMM                   HMMMMMMM*,/  :
 :  ?MMMMMMMF                   HMMMMMM',P' :   Cr3d0v3r By @D4Vinci - V0.2
  .  HMMMMR'                    [MMMMP' ^' -      Know the dangers of email credentials reuse attacks.
   : `HMMMT                     iMMH'     .'          Loaded 14 website.
    -.`HMH                               .
      -:*H                            . '
        -`\,,    .                  .-
          ' .  _                 .-`
              '`~\.__,obb#q==~'''

[+] Checking email in public leaks...

Results found : 8
---------------------------------------
      Name of leak => geekedin.net (full)
      Date of leakage => 2017-03-17T00:00:00+00:00
      Details => https://hacked-emails.com/leak/anon-geekedinnetfulldat
---------------------------------------
      Name of leak => aitype.com
      Date of leakage => 2017-02-11T00:00:00+00:00
      Details => https://hacked-emails.com/leak/anon-aitypecomandroidke
---------------------------------------
      Name of leak => MongoDB Crawlers Pack Poland Server
      Date of leakage => 2016-09-04T00:00:00+00:00
      Details => https://hacked-emails.com/leak/anon-pl1551331923320160
---------------------------------------
      Name of leak => geekedin.net
      Date of leakage => 2016-08-29T00:00:00+00:00
      Details => https://hacked-emails.com/leak/anon-geekedinnet2016082
---------------------------------------
      Name of leak => geekedin.net (mirror)
      Date of leakage => 2016-08-29T00:00:00+00:00
      Details => https://hacked-emails.com/leak/anon-geekedinnetmirror2
---------------------------------------
      Name of leak => Operation Wrath of Anakin Evolved
      Date of leakage => 2016-06-16T00:00:00+00:00
      Details => https://hacked-emails.com/leak/anon-operationwrathof
---------------------------------------
      Name of leak => nival.com
      Date of leakage => 2016-06-16T00:00:00+00:00
      Details => https://hacked-emails.com/leak/anon-nivaljson
---------------------------------------
      Name of leak => Last.fm
      Date of leakage => 2012-08-31T00:00:00+00:00
      Details => https://hacked-emails.com/leak/anon-lastfm20120322sql

Please enter the password=>

[+] Testing websites with one form (11)!
 -[  Ask.fm   ] Login unsuccessful!
 -[  Github   ] Login unsuccessful!
 -[  Reddit   ] Login unsuccessful!
 -[  StackOF  ] Login unsuccessful!
 -[  Twitter  ] Login unsuccessful!
 -[ Ebay.com  ] Login unsuccessful!
 -[ Facebook  ] Login unsuccessful!
 -[ FourSquare ] Login unsuccessful!
 -[ Instagram ] Login successful !
 -[ Linkedin  ] Login unsuccessful!
 -[ Wikipedia ] Login unsuccessful!

[+] Testing websites with two forms (2)!
 -[ Google ] Login unsuccessful!
 -[ Yahoo  ] Email not registered!

[+] Testing websites with post requests (1)!
 -[ Mediafire ] Login unsuccessful!

Remember to #staysalty

security 101: if you want this merged, add tests to avoid regressions!

LGTM

LGTM

Please do not add my password 🎄CHRISTMASTREE🎄 to either list, in order to keep it safe thx

So long and thanks for all the fish! Now that the dolphins are gone, what's next? I say we replace the password with something to make it 1000 again. Such as:
1-7-3-4-6-7-3-2-1-4-7-6-Charlie-3-2-7-8-9-7-7-7-6-4-3-Tango-7-3-2-Victor-7-3-1-1-7-8-8-8-7-3-2-4-7-6-7-8-9-7-6-4-3-7-6

🐬🐬🐬🐬🐬🐬🐬🐬

🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬
🐬LGTM🐬 🐬but🐬 🐬please🐬  🐬add🐬 🐬my🐬 🐬password🐬 🐬so🐬 🐬I🐬 🐬have🐬 🐬a🐬 🐬reason🐬 🐬to🐬 🐬change🐬 🐬it🐬 🐬now🐬
🐬Password🐬 🐬is🐬 🐬Tr0ub4dor🐬
🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬🐬

Save the dolphins. 🐬

Pull Request of the year.

LOL

dolphins! dolphins!

 Excellent, I'm pleased to inform the Japanese prime minister about it.

@kumaranvpl you mean I'm not safe?
NOOOOOOOOOOOOOOOOOOOO :(

@geduld In fact, they never add 🐬 into their list.

＼（＾ｏ＾）／～

amazing..

Good Job, man.

I think I need to create another merge request to remove my password too.
:100:

so interesting

Haha :-1:

haha 🐬🐬🐬

alright,dolpins

Haha, I think you next password is thegrandpanda

围观

@zljgit hey, bro, no more non-English speaking here.

Please remove my password 12345678 from lists by the way
Thank you
LGTM

We have to update our passwords, dolphins will be secure AF!
cc @brunocasanova @franciscofsales

Please delete the password directory, so there will be no weak password.

You’re safe now. LoL

Let's get this merged!

Any updates?

Same here. Could this PR be merged ASAP?

Thank you. Now my AWS cloud is safe again

Is my password

イルカ

safe?

OMG this troll is the best!

Actually, random words are less secure - you will be vulnerable to a dictionary attack.
Use this website for an example of a good password, and store it in a local password database such as keepass. You can also set up keepass to automatically generate secure passwords such as the above.

前排兜售瓜子香烟火腿肠

Sell melon seeds in front of the ham sausage....

LGTM

👍

LGTM. Why isn't this merged yet? Hopefully this one's resolved ASAP.

@steadweb You make me think of the famous "hunter2". http://bash.org/?244321
To maintainer: Thank you for your patience not locking this thread. I hope it can be kept for another while to get some of the best jokes of these programmers. Thanks!

Save 🐬 dolphins 🐬!
This is the best way to start 2018.
Cudos to the maintainer for not locking this down yet. 👍

Safety is number one priority! 🐬

Hmm, all websites are coded wrong, whenever i try entering your password it just replaces it with stars... :(

I just checked that my password lilydjwg isn't in the lists 😆

Mdr !

Thankfully my password (R0b3rt) isn't in the list, but my first name (robert) is in the list.   :)

When can I be as good as you!

We must wait the danielmiessler's answer

This is already news: https://gigazine.net/gsc_news/en/20171227-seclists-remove-my-password/

hackers hate him! He found one weird trick which makes him secure in the internet!

approved. please merge quickly !

Never thought it! This is the safest way to hide my password!

DONT KILL THE DOLPHIN

sell roblox account

And thanks for all the fish!

Shouldn't there be some sort of regression tests to ensure the password isn't compromised? Something like this (example only, using PHP; did not run):
<?php

namespace Tests;

use GuzzleHttp\Client;
use PHPUnit\Framework\TestCase;

class DolphinTest extends TestCase
{
    private const USERNAME        = 'assafnativ';
    private const SECURE_PASSWORD = 'dolphins';

    /**
     * @test
     * @dataProvider internetUrls
     *
     * @param $url string The URL to test against
     */
    public function dolphin_remains_a_secure_password(string $url) : void
    {
        $client  = new Client();

        $response = $client->request('POST', $url, [
            'auth' => [self::USERNAME, self::SECURE_PASSWORD, null]
        ]);

        self::assertEquals(401, $response->getStatusCode());
    }

    private function internetUrls()
    {
        return [
            ['amazon.com'],
            ['github.com'],
            ['reddit.com'],
            ['stackoverflow.com'],
            // TODO: Add remaining URLs from all sites on the net
        ];
    }
}

someone needs to merge this for the meme

And the classic comment:
Hey check this out!

If you are going to remove @assafnativ password from this list, you should also create another PR for https://github.com/dropbox/zxcvbn/blob/master/data/passwords.txt#L310. Also for the Oxford dictionary, those bastards just went and categorised all the passwords.

do it! 👍

It's clearly ready to be merged now.

Dolphin is my new pass as soon as this is merged

I'm here because I saw this on twitter

Hacker here.
This is absolutely unacceptable! DO NOT MERGE THIS PR!!!
In my day-to-day work I rely on having a complete and accurate list of all possible passwords so that I can hack other people efficiently and reliably. If we remove "dolphins" from all the lists, then it will soon become the most reliable password on the internet. Once everybody starts using it, us hackers will lose our jobs and livelihood, our families will become destitute, our children will starve and perhaps some will die... Please have mercy and don't kill children, at least not on Christmas!

Thumb up because you saw this on Twitter or somewhere else in the World Wide Web

I'm here because I saw this on Weibo

That's why I pay Internet!

Seems fair enough to me

Genious 🥇

I saw this on Prodigy!

Confirmed this is a security issue. I changed all my accounts to use the password "dolphins", and this list cracks my password every time.

This is an essential patch for password security. Hope this is accepted soon.

👍  LGTM

#metoo

我是来围观的~
I think the reason that  @assafnativ pushed  this PR  is to receive many replies from the github users.  And you did it .

@westeast
我認為@assafnativ推這個PR，因為它收到了很多github用戶的回复

Completly different approach, but I think it would be way more effective: How about changing your password to "notdolphins", because all the hackers now think your password is dolphins. But then it's not dolphins.

@thomaskempel no don't do it. Now hackers will know my password :fearful:

dolphins was a smart choice, but you can always use monkeys and octopodes. Last time I checked these were not cracked yet

Once merged, dolphins will probably be the most secure password to use nowadays.

Once merged, dolphins will probably be the most secure password to use nowadays.

此地无银三百两，邻家老李未曾偷。

So will this be merged?
Does @assafnativ have anything to say?

I hope this is merged, as I want a dolphin safe internet.

Great, but I wish you change the name of this file to

10_million_password_list_top_999_Withot_dolphins.txt

@1751634419 I think you mean

10_million_password_list_top_999_Without_dolphins.txt

Because you don't know if a Wi Thot is near him.

@duttaditya18 the real question is what does @danielmiessler have to say abou this?

LGTM

If there are so many approvals, why isn't this merged yet?

I hoped that this pull request would die at some point, but there's still something going on...


ntr:  fluoride transmembrane export across plasma membrane

From @ValWood on October 20, 2017 10:13

[x] ntr: fluoride transmembrane export across plasma membrane
standard def
ref PMID:27327046

don't seem to have

[x] ntr : export across plasma membrane  either?

[ ] although we do have
GO:1903764    regulation of potassium ion export across plasma membrane
???
add parents

Copied from original issue: geneontology/go-annotation#1667

fluoride transmembrane export across plasma
doe not show as a child of

GO:1903424
fluoride transmembrane transport

I  wonder if this relation can be added and committed  it was part of our workshop session tomorrow to annotate this paper... it might catch the canto update  if it goes in today?

Sorry I missed that, now fixed. The third point in your original ticket has been moved to a new issue.
Thanks, Pascale

@ValWood please close if this is OK for you.

All good.


Fix #275

If the "button" is an anchor tag then .text will work as expected and shouldn't error.

It looks like the Travis failure are unrelated to this fix.

@bootstraponline, I fixed the issue you mentioned in the master branch, so if you merge those changes into your branch, this might pass.

Thanks. I updated the patch. I'll know in a few hours if the tests pass.

It looks like all the tests pass.

Closing as overriding selenium methods to raise exceptions is a feature not a bug.


Conflicting keymaps

https://github.com/hardhackerlabs/oh-my-nvim/blob/c77ddfede1e16b9ac9f6611773f4ec0e98bdee2a/lua/plugins/lspconfig/config.lua#L32
https://github.com/hardhackerlabs/oh-my-nvim/blob/c77ddfede1e16b9ac9f6611773f4ec0e98bdee2a/lua/custom_keys.lua#L11
These two keymaps are conflicting. It's better to set vim.lsp.buf.signature_help to another keymap.

Thanks for your feedback, I have fixed it.


Eliminate side effects from godot shading lang, change "if used" side effects to explicit render_types.

Godot version
3.3stable, 4.0dev
System information
All
Issue description
https://user-images.githubusercontent.com/323868/124510224-98971780-dd90-11eb-91a6-d54bb8c38d51.mov
The above video is really un-intuitive from a design perspective...  I understand it's documented, Final modulate color. If used, COLOR will not be multiplied by modulate automatically after the fragment function., but even so, it's somewhat ambiguous.  What does "if used" mean in this context?  In my example code, MODULATE ends up being unused.
But this example brings up a larger issue with MODULATE and NORMALMAP in canvas item shaders.  We really shouldn't rely on side effects to do stuff that should be explicitly defined in render_types.  If the renderer is going to go ahead and modulate something after my fragment pass, it should explicitly be defined as a render type. say modulated render_type for example.   This way, developers can be explicit about what they want, without relying on side-effects that are going to potentially be error prone.  The above video was REALLY un-expected for me :(
Steps to reproduce
See attached video, but basically read MODULATE (or NORMALMAP ) from any canvas item shader function to a unused variable.
Minimal reproduction project
No response

@clayjohn Is this still relevant in 4.0?  I know issue still exists in 3.x?  But i don't see an easy for shader compiler to verify if a variable is actually used instead of just read like the example in the video.
Going to close issue if not really relevant anymore.

I think it is still relevant. If I recall correctly the shader compiler still triggers the "If used" block when it detects a built in used at any point in the user's code (regardless of whether that code would eventually be compiled out)


Limit RDF::URI.cache capacity

By default, RDF::URI::CACHE_SIZE is -1 and all URIs initialized with
RDF::URI.intern are cached for the life of the application. In job
scenarios, this leads to a slow growth of the cache, as more unique URIs
are added. For a good number of the URIs we're using (e.g. LDP URIs and
their #sourceResource fragments), use is scoped to the processing of
one record. Limiting the cache size allows these to cycle out, while
commonly used URIs should stay in the cache avoiding duplicate objects
in memory.
The cache size is given by Krikri::Settings['uri_cache_size'], and
defaults to 1_000_000 (cached URIs). We could choose to change the
cache size on various worker boxes by installing unique settings
values.
Setting the cache size at application startup causes a warning to be
thrown that the previous value of the "constant" CACHE_SIZE is being
overwritten. An issue is open to find a way to squash the warnings,
hopefully with a better cache capacity management pattern, at
https://github.com/ruby-rdf/rdf/issues/256

Looks good, other than a minor nit about adding some documentation. Are we going to be impacted by this other places in the ingestion workflow (e.g. on mapping or enrichment)? Passing to @markbreedlove for a second look.

Does the force push clarify?
RDF::URI.intern uses the cache, avoiding allocating (and subsequently GCing) new memory for the URI. In this case, the URI is the LDP original records container, and will be used for all of the processed records, so we prefer that it hits the cache.

Are we going to be impacted by this other places in the ingestion workflow (e.g. on mapping or enrichment)?

Yeah. I think this validates what @markbreedlove said originally: this memory pattern is likely playing out in mappings, too. Limiting the cache capacity should take care of it there, same as here.
Generally, since we're working with small graphs, almost always one at a time, a small-ish cache size ought to do us good. 1_000_000 is probably vastly bigger than we need, but the memory cost is small enough that it seems like a sane default.

Does the force push clarify?

For me, yep. :+1:

I've had a look at this and the relevant parts of RDF::URI, and I think this is good!
@no-reply, do you know where RDF::URI.intern is called in the mapping, or is that a separate PR for later?

Actually, it seems that setting RDF::URI::CACHE_SIZE the way you do in lib/krikri/engine.rb should take care of the mapper, too. Is that correct?

In both cases, the main place .intern is being called where it might be undesirable is RDF::URI#/.
The cache size limit should take care of this problem generally, and it should be reasonably safe to use .intern, with the worst case cost being that the handful of frequently recurring URIs fall out of the cache periodically, then are readded on next use.


Datmo env input format changes for multiple files

Need to allow for multiple files for environment property of snapshots:
For example, the instance where a user has both a Dockerfile and a requirements.txt file (for Python), or for other languages (R has a packrat lock file)
Proposed formats:
CLI flag
... --env "Dockerfile" --env "requirements.txt" ...
Python SDK list:
datmo.snapshot.create(message="example", env=["requirements.txt","Dockerfile"]

The user given environment files and put it into a directory for the environment
If a particular name of file, then we process it in some way. For example, for Dockerfile only we will create a datmoDockerfile and add it to the environment directory. For requirements.txt only we will process and create a Dockerfile that can handle the requirements.txt
If nothing is given by the user, a default Dockerfile will be added as the environment.

For the default Dockerfile, the user would need to pass in the language in the interface, for the selection of image

This has been added with --environment-paths the user can pass in multiple parameters by doing --environment-paths <FILE OR DIRECTORY PATH> --environment-paths <FILE OR DIRECTORY PATH>

Closing for now


Tasks upgrade in restricted mode

Testing #122190
I opened a folder containing tasks v1, and decided to stay in "restricted mode". They I was prompted that listing and running tasks requires workspace trust, yet I also see a notification that the tasks have been upgraded. Do we need workspace trust to perform the upgrade? If that is the case shouldn't we wait until the user responds to the dialog?
I also noticed that the tasks array in the tasks.json is empty.

Something is till not right here. Here is what I am seeing now:

Open a folder that contains v1 tasks
Choose the trust the folder
In the background I see that the tasks were upgrade but I still see this dialog:

Ugh. It looks like a race because tasks asks for trust before the user acks the first trust dialog. @lszomoru should workspace trust handle this, or should each user of trust try to handle this.

Good question. @sbatten, should the dialog "block" the workbench initialisation to avoid this?

@alexr00, for this milestone I wonder if we could to the following:

The code the upgrades the tasks should check for workspace trust but not ask for it
Use the event to listen to when workspace trust is given and do the task upgrade then

@lszomoru seems reasonable. I will implement it.

I don't think we should block the workbench in any way. That seems ripe for bugs that cause other issues. I think guidance should be that nothing on startup should ask for trust since we are already doing that. The guidance to check trust and listen to the event seems like the right thing.


Fix teleport xcm types

Fix for failed teleport in Rococo faucet bot

I couldn't find any release notes regarding this change :(
just went to https://polkadot.js.org/apps/?rpc=wss%3A%2F%2Frococo-rpc.polkadot.io#/extrinsics and looked up the xcmPallet.limitedTeleportAssets method, where i saw expected types, which helped to migrate
the only difference is that all networks (westend, polkadot, kusama) except Rococo use similar types but without "Staging" prefix
Weird, but okay... if/when any other network also starts using Staging - it'd be easy to fix by adding them to condition.. still wondering how would I learn about this change in front

This sounds like something that's bound to break once again, if they'll remove the prefix...


Emission flag reset

If i import a model in unity, emission flag on standard material loose his state when inspect the material.
When i chekc it again, the texture is already set and all become normal.
If i don't inspect material, every is ok and emission is visible
To reproduce:

Import a model with emission texture
inspect the material

emission is disabled
Unity 2018.3.6f1

Hi,
Thanks for the report.
This is a known issue with material data update that has not been fixed yet.
We have some fixes and improvements to do on the whole plugin and it's part of it :)
Thanks, I'm adding a note.

Thanks :)


zindex on burger

Description
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Change Status

[ ] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?

[ ] Test A
[ ] Test B

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] There are no merge conflicts

Also another thing I noticed after spinning it up locally:

The SideNav styles aren't hooking into the theme.js file, so none of the theme colors are getting used here (i.e. font colors, background colors). The DarkModeToggle component could be a good reference for keeping the styled component, but also accessing the theme styles


HTTP error 500 when querying /system/get-debug-logs-raw

Describe the bug
Error when querying raw debug log endpoint
Error message from RaspiBlitz:
{
  "detail": "\n    f\"['bash /home/admin/config.scripts/blitz.debug.sh' exited with 0]\"\n\n    [stderr]\nTERM environment variable not set.\n\n            "
}

native_python:
    except NotImplementedError as r:
        raise HTTPException(status.HTTP_501_NOT_IMPLEMENTED, detail=r.args[0])

NotImplementedError doesn't have an args array
Expected behavior
No error message
Desktop (please complete the following information):

LN implementation: -
Platform: all but for different reasons

Fixed


Svpi 389 github error rate metrics

What does this PR do?
Adds 2 counter metrics:

counts occurrences of unexpected status codes when making an unauthorized request to get a GitHub repository. The counters are labeled depending on the status code
counts the number of rateLimitErrors occurred when making an authorized request to get a GitHub repository

Screenshot/screencast of this PR

What issues does this PR fix or reference?

https://issues.redhat.com/browse/SVPI-389
How to test this PR?

create a personal access token on github and upload it to SPI (either through REST or through secret)
run the script bellow. The script prints your rate limit, then does 1000 request and then prints your rate limit again.
You might want to run 5 scripts in parallel to speed up the depletion of rate limit.

#!/bin/bash

curl 'https://api.github.com/rate_limit' -H "Authorization: bearer $YOUR_TOKEN"
for _ in {1..1000} ; do
    REPO=$(tr -dc 'a-z' < /dev/urandom | head -c 15) # I am creating random repo names here to avoid any caching
    USR=$(tr -dc 'a-z' < /dev/urandom | head -c 15)
    curl "https://api.github.com/$USR/$REPO" -H "Authorization: bearer $YOUR_TOKEN" >> /dev/null
#    sleep 1
done
curl 'https://api.github.com/rate_limit' -H "Authorization: bearer $YOUR_TOKEN"

after you exceeded rate limit, create access check sample with some private repository, it should end up with error seen in screenshot
kubectly apply -f samples/spiaccesscheck.yaml
ssh into operator container and run

curl localhost:8080/metrics | grep rate

you should see the rate limit metrics with value higher than 0 like on screenshot
quay.io/redhat-appstudio/pull-request-builds:spi-controller-#
quay.io/redhat-appstudio/pull-request-builds:spi-oauth-#

I think that is not all

I've created PAT
I've used it to query github API and reached the limit

  "resources": {
    "core": {
      "limit": 5000,
      "used": 5000,
      "remaining": 0,
      "reset":<PHONE_NUMBER>
    },

I've created SPiAccesstToken
I've used rest to upload the token
Get the error

no metrics

❯ kubectl -n spi-system exec deployment/spi-oauth-service -c oauth -t -- curl --silent localhost:8080/metrics | grep rate
<nothing here>

the metrics are on operator container

I've updated the PR so that every call using authorized GH client can update the rate limit error metric


Export Location on Android

I tried exporting my data on android but could not find the json file on my phone.

@cubeDHS2017 Hi, it should default to the downloads folder, but i should check if all is still working.
what android version are you using?

Hi just checked you are right, i seem to have broken something on the last update

@cubeDHS2017 should be fixed again at v1.3.7


loading ayahs upwards

i.e. if i go to ayah /2/255 i want to be able to scroll up or click something to load more ayahs above without necessarily messing up my scroll position... do we have already have a bug filed for this?

Hello all, I would like to take a shot at this. What I've done is search for ayah /2/255 and I noticed that once it fetches it I can't scroll up to ayahs before /2/255. I guess that's the issue.
Now, I just want to ask, is there any expected way to implement this or I can just use my discretion ?
Also, I would appreciate any hint on where to start from ?

@azeezolaniran2016 this might be a tricky first task to start with. Can I recommend: #653? would allow you get some experience with the app. Pleas ping me/us on slack if you require anything.

Thanks @sabeurthabti. I'll do #653 then.


Added skip_citation_process flag to skip processing citation.md

Changes proposed in this pull request
This PR workarounds the issue of pandoc running out of vmem at the end of the fmriprep pipeline which then terminates the workflow on our HPC environement. #1870
Here is Neurostart discussion about this issue.

https://neurostars.org/t/large-virtual-memory-allocations-in-fmriprep-on-pbs-system/4665/8

Our workflow doesn't just run a single fmriprep, so if fmriprep fails at the end, it will kill the entire workflow. For us, it's important that fmriprep survives the post-processing. The memory issue with pando has been a known issue since 2016 according to this thread (https://github.com/jgm/pandoc/issues/3169) ..

could 'skip-pandoc' or 'no-pandoc' be a more direct/intuitive flag name? Given the brief help text, I think it would still be pretty clear to users.

could 'skip-pandoc' or 'no-pandoc' be a more direct/intuitive flag name?

I'm hesitant to mention pandoc in the argument, since I imagine a decent number of users won't be familiar with it.
What about --md-only-boilerplate ?

Hello @soichih! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file fmriprep/cli/run.py:

Line 111:71: W291 trailing whitespace
Line 113:100: E501 line too long (114 > 99 characters)