Minor changes to ImageTrigger generation

We currently have a few small requirements which has led us to do our own hacks for the F-M-P and this causes issues whenever there is an upgrade to F-M-P.

We want to use the ImageTrigger to be able to resolve the ImageStreamTag but do not want automated deployments whenever there is a new Image generated. The only way to do so is to have a "disabled" ImageTrigger, meaning automatic is set to false.

We also need to be able to specify the namespace for the ImageTrigger, so that we can refer to images in other namespaces. In our case, we have some images that are of common concern and we want to use this across all of our applications.

This is how it would look like:
    - imageChangeParams:
        automatic: false
        containerNames:
          - demoapp
        from:
          kind: ImageStreamTag
          name: 'common-concern-image:1.0.0'
          namespace: public-images

I have changed the following code in DeploymentOpenShiftConverter.java:
                if (containerToImageMap.size() != 0) {
                    if (mode.equals(PlatformMode.openshift)) {
                        for (Map.Entry<String, String> entry : containerToImageMap.entrySet()) {
                            String containerName = entry.getKey();
                            ImageName image = new ImageName(entry.getValue());
                            String tag = image.getTag() != null ? image.getTag() : "latest";
                            specBuilder.addNewTrigger()
                                    .withType("ImageChange")
                                    .withNewImageChangeParams()
                                    .withAutomatic(false) // This will set automatic to false
                                    .withNewFrom()
                                    .withKind("ImageStreamTag")
                                    .withName(image.getSimpleName() + ":" + tag)
                                    .withNamespace(image.getUser()) // This will get the namespace
                                    .endFrom()
                                    .withContainerNames(containerName)
                                    .endImageChangeParams()
                                    .endTrigger();
                        }
                    }
                }

Can these be set as options?
@rohanKanojia

We can add a flag which you can use in your build like this:
   mvn clean install -Dfabric8.openshift.isImageTriggerAutomatic=false fabric8:deploy

It's default value would be true(which is the behavior of fabric8 maven plugin right now). How does that sound to you?

Sounds great. Though, could we rename it to setImageTriggerAutomatic (or something like that) instead of isImageTriggerAutomatic? Because it's a bit confusing when using isImageTriggerAutomatic.
Will the namespace be added by default without a flag? I'm not sure if it causes any harm to the current usecases.

It depends on whether it's specific to your use case only. But I feel that they should be given as two separate flags.  I need to check if it causes any harm to default flow of fmp.

@rohanKanojia @mojsha No issues for me right now. Yes keep default value to true

@rohanKanojia Any option is fine. I just need this ASAP to avoid doing my home-made "patches" on top of the releases from F-M-P.  Much appreciated.

@rohanKanojia Looks perfect!

@rohanKanojia Thanks alot for your efforts! This will help us with the 3.7 upgrade/migration. Can I ask how long it will take for this to be released?

Hey @mojsha
We just released FMP 3.5.38. Please give it a try. Looking forward to your feedback.
Thanks

@piyush1594 @rohanKanojia I have tested the plugin and tentatively it looks like it's working as it should. I would like to thank you and in particular @rohanKanojia  for all of his efforts! This has enabled us to start our journey towards OpenShift 3.7+.


Route not found issue

Reading the documentation, I had to proceed by clicking on the Auth from Azure-hosted apps as you can see on the image below. I get a 404 error.

404 page url =
https://learn.microsoft.com/en-us/azure/developer/javascript/sdk/authentication-azure-hosted-apps

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

ID: 7d4935f4-d5c1-f7c3-eeb0-f8b5e5fd406f
Version Independent ID: 766ee1b8-4c5d-054b-d7ec-1825d8f5d50b
Content: Quickstart: Azure Blob storage library - JavaScript
Content Source: articles/storage/blobs/storage-quickstart-blobs-nodejs.md
Service: storage
Sub-service: blobs
GitHub Login: @pauljewellmsft
Microsoft Alias: pauljewell

@Scottlexium
Thanks for your feedback! We have assigned the issue to author who will provide further updates.

Thanks so much for your feedback! The link is fixed, and the changes should go live later today.
#please-close


Different behaviors from native keyboard

I have found these behaviors are different from that of iOS native keyboard (tested on iOS 18, but I'm pretty sure these applies to lower versions too).

When inputting ' (single quote), KeyboardKit does not return to .alphabetic layout, whereas on native keyboard it returns to .alphabetic layout right after inputting single quote
On Web Search field, KeyboardKit still capitalizes, whereas on native keyboard capitalization is disabled for this field.

Hi @vegadodo! Thank you for reporting these things.

This has been fixed in the v9 branch. 👍


Refactor ProgramError

The existing ProgramError<T> was created to handle parsing errors but as the Program API expanded, it grew beyond that purpose.
The existing error type has been renamed to ProgramParsingError<T> and any variants that weren't related to parsing have been removed. There is a new ProgramError enum that contains a variant for each possible error when working with a program. It also implements From<ProgramParsingError<T>> so the public API doesn't have to deal with the generics. This makes it possible to re-use ProgramError fairly easily in the python bindings.

🚀


Revert "[test] Disable a couple of DebugInfo tests"

This reverts commit a182a7085de104bc19633d8479b9ce60d14abdb9.
I also fixed the tests as well.
NOTE: In the generic case of debug_value_addr, the self parameter with asserts
seems to be a var and without asserts a let. This is apparent at the AST level
before we even get to anything that I have changed. This seems to suggest that
there is some sort of uninitialized memory or something like that. Regardless
that is more of an AST level thing, so I just put in a regex that match both
patterns and send rdar://89237318 over to Robert Widmann!

@swift-ci test

@swift-ci test apple silicon

preset=buildbot,tools=R,stdlib=RD
@swift-ci Please test with preset macOS

preset=buildbot,tools=R,stdlib=RD
@swift-ci Please test with preset macOS

Looks like build-preset isn't working...


Problem in enabling core dumps

Question

I am a beginner in coding. When I try to run a code in java which throws any exception , instead of the name of the exception such as ArithmeticException or StringIndexOutOfBounds exception, the IDE shows a long message starting in "A fatal error has been detected by the Java Runtime Environment". It says that the core dumps have been disabled and I am not sure but I think that could be the problem.
I do not know how to fix this as I am only a beginner so any help would be immensely appreciated. Thanks a lot in advance.
Context
For context , this is the entire thing that shows up when an exception occurs:
A fatal error has been detected by the Java Runtime Environment:
SIGFPE (0x8) at pc=0x0000000117134188, pid=1476, tid=7939
JRE version: OpenJDK Runtime Environment AdoptOpenJDK (<IP_ADDRESS>+1) (build <IP_ADDRESS>+1)
Java VM: OpenJDK 64-Bit Server VM AdoptOpenJDK (<IP_ADDRESS>+1, mixed mode, tiered, compressed oops, g1 gc, bsd-amd64)
Problematic frame:
j  ExceptionHandling.MainClass.main([Ljava/lang/String;)V+0 test1
No core dump will be written. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
An error report file with more information is saved as:
/Users/shaivmehrotra/eclipse-workspace/test1/hs_err_pid1476.log
Could not load hsdis-amd64.dylib; library not loadable; PrintAssembly is disabled
Java version:
11.0.11

Your operating system and platform:
macOS Big Sur version 11.2.3 , ARM 64

Hi @shaivmehrotra15 .  I don't think core dumps being disabled is related to this crash.
A few questions and comments:

Your sample output references <IP_ADDRESS>+1, but you mention Java version 11.0.11.  Does the crash occur when you install and run under 11.0.11?
You also refer to ARM 64.  Is it correct to assume that that's a typo?
What happens when you run your code outside of the IDE?  Have you tried running ulimit -c unlimited as suggested in the output to enable core dumps?
Can you share your code, or event better, a minimal reproducible test case?  I have access to macOS Big Sur and can try to reproduce the behavior, but I think I need more information about your setup and the code to reproduce.  What version of Eclipse are you using?

Hi @shaivmehrotra15 . I don't think core dumps being disabled is related to this crash.
A few questions and comments:

Your sample output references <IP_ADDRESS>+1, but you mention Java version 11.0.11.  Does the crash occur when you install and run under 11.0.11?
You also refer to ARM 64.  Is it correct to assume that that's a typo?
What happens when you run your code outside of the IDE?  Have you tried running ulimit -c unlimited as suggested in the output to enable core dumps?
Can you share your code, or event better, a minimal reproducible test case?  I have access to macOS Big Sur and can try to reproduce the behavior, but I think I need more information about your setup and the code to reproduce.  What version of Eclipse are you using?

Hi @tmancill thanks a lot for reaching out to help. I'll try to answer your questions to the best of my knowledge.

I mentioned 11.0.11 because when I type javac -version on terminal it shows 11.0.11 . I tried reinstalling JDK 11 but there was no error at the time but when I opened eclipse to run the code again it showed the same error.
I use an m1 MacBook Air which has an ARM based chip , that's all I know regarding this thus I wrote ARM 64 as it was mentioned as an example. I do not know if that is exactly correct.
I tried to run the code on eclipse , VS code and blueJ but each time it shows the exact same result. And yes I did try to enable core dumps. Whenever I type ulimit -c unlimited on terminal nothing really happens so I don't know if it's working or not. I tried to look for other alternatives but could not find any.
4)My code was very simple it was just dividing a number by 0 as I wanted to see what exception it would throw as I was practicing exception handling and try catch block, and weirdly enough , this happens even if I do put the code in the try catch block. The version of eclipse I am using is 2020-12. Please reach out to me regarding any other information that you'd require about my setup and thanks a lot for the help.

@shaivmehrotra15 Ah, thank you for clearing that up - my mistake for not thinking about the M1-based hardware.
Newer versions of Eclipse bundle the JRE, which is possibly why you're seeing 11.0.11 on the command-line but <IP_ADDRESS> in your crashes.  That is, Eclipse is not using the same JRE/JDK you are installing.  Also, it's suspicious that the crash refers to hsdis-amd64.dylib when you are running on an ARM platform.
I'm having a hard time locating the download for Eclipse 2020-12 for Apple Silicon (and I don't have access to that hardware anyway), and I will need to follow-up with instructions on how to enable the ulimit when running under the Eclipse IDE.
In the meantime, you might try installing an updated Eclipse package for your platform to see if the issue still occurs.

@tmancill Okay I will install an updated Eclipse package but as I said the same error comes when I use VS code or BlueJ for that matter as well, which means the error should be independent of the IDE I use I think. But I'll still try reinstalling and get back to you whether I am getting the same error.
I'll wait for your follow-up instructions on the issue.

@tmancill I installed the latest Eclipse package the 2021-03 but it still shows the exact same error.

@shaivmehrotra15  The problem you are running into with the missing Hotspot disassembler for your OS and architecture appears to be the same issue reported in https://github.com/adoptium/adoptium-support/issues/211#issuecomment-738104579.  There are also some links you might find useful in https://github.com/AdoptOpenJDK/openjdk-build/issues/1922.  You might try one of the Azul builds and/or watch the progress of https://github.com/AdoptOpenJDK/openjdk-tests/issues/2171.

@tmancill I went to #221 which directed me to http://openjdk.java.net/jeps/391, but I do now understand what exactly am I to do on there. Also, I went to https://github.com/microsoft/openjdk-aarch64/releases/tag/16-ea%2B10-macos and downloaded the files and pasted them inside my jdk file( I'm not sure if I had to do that) but nothing happened. I also downloaded Azul's ARM 64 build for JDK-11 inside my JVM file but it did not make any difference either( I'm not sure if I need to do something else after downloading , because I did not) . I also went to #231 who seemed to be getting the exact same problem , which suggested the Azul fix and the https://github.com/microsoft/openjdk-aarch64/releases/tag/16-ea%2B10-macos one so I suppose they should work but they ain't for me so could you please give a small step by step instruction which I could follow as I do not understand what exactly needs to be done.

Hi @shaivmehrotra15.
After downloading and extracting the new JRE or JDK you will need to configure Eclipse (or VS Code, or whatever IDE you are using) to use it instead of the default, which is often bundled with the IDE.  This is done in Eclipse using Window -> Preferences -> Java -> Installed JREs.  When you add a new JRE, you will have the option to configure it to be the default for projects.
Within your current project, you may have to change the runtime JRE.  You access via Project -> Properties -> Run/Debug Settings -> (select your main class) -> Edit -> JRE [tab] -> Installed JREs  and select the desired JRE.  When you run your program, you will see the path to the JRE in the top of the Console window.  If you're still seeing <IP_ADDRESS>, then the preference hasn't been applied to your project.
In the example below, I've configure Eclipse to use Azul's early access release of JDK 17 for my project; the output on your platform will look different:

I hope this helps.  (Because Adoptium doesn't yet support the JVM on this platform and there is already an open issue for that, I feel like we should move any further conversation about configuring the IDE elsewhere.)

Also, it's suspicious that the crash refers to hsdis-amd64.dylib when you are running on an ARM platform.

That is entirely normal and indicates that is running the x86-64 (AMD64) under the Rosetta emulation later instead of running a native ARM64 JVM.
Adopt does have nightly nightly builds for macos/ARM64 being build but they are not yet fully tested or ready to be released.


ROE-2343 Initial changes with new test and example

JIRA link
https://companieshouse.atlassian.net/browse/ROE-2343
Change description
The current problem:
If a DAO/DTO model field datatype changes (e.g. from String to EntityNameDTO) that causes a breaking change when reading older JSON model data from Mongo (requiring a new data transformer to be written), this is not picked up by the current DtoModelChangeTest unit-test unless the data field is present in the test JSON file. If not present and not detected by developers, it can lead to a later run-time failure, possibly in the LIVE environment.
This PR proposes a solution to that by ensuring that (nearly) all fields present in the DTO/DAO model are present in the current (most recent) JSON test file, thereby ensuring that the model conversions will be exercised and tested.
The code is by no means complete and there may well be an alternative/better approach to the problem. Hence its draft status.
Work checklist

[ ] Tests added where applicable

Merge instructions
We are committed to keeping commit history clean, consistent and linear.  A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

Looks good to me as a POC


One vs All Classification

Hi,
I'm using a CNN in R to perform image classification with K different classes. However, I want to try another approach and do a "one vs all" classification with each label, so one label would be 1 and the rest would be 0.
My question is, what output layer should I use? If I use mx.symbol.SoftmaxOutput after a FullyConnected layer with one neuron is useless because it always predict 1 or even NaN for every example. LinearRegressionOutput gives a similar result.
My basic idea is that I need an output layer that performs 0.5 threshold on the single output neuron from the last FullyConnected layer.
Sorry if this is a dumb question, I'm still new to this field.
Thanks in advanced.

LogisticRegressionOutput

@piiswrong Exactly what I was looking for, it worked like a charm. Thank you very much!

hi @piiswrong
Thanks for the answer, exactly what i am looking for.
Just want to add a nab question following this issue:

which metric would be best suited for this kind of 1 vs all classification problem.

I suppose standard accuracy is not good because of the big difference in the number of samples from the two classes.

Hi @yuan1202 ,
I think I may have an answer for you. During the training phase, I just use the rmse metric, but for further analysis I do the following:
First, I make a prediction using the test set, and afterwards I analyse all the different metrics you can obtain from the confusion matrix (as you can see here). Also, you may be interested in the F1 Score metric.
I hope this is what you were looking for.

Hey @CarlosUziel
Thank you for the reply.
I will try out your approach.
I have a rather biased binary class situation where 0 class to 1 class ratio is about 19:1. But I suppose it's similar for all one vs all situation.
When I just use default accuracy metric the result is around 95% which seems not right to me because it's too close to the 19:1 ratio.
I tried to use F1 score but it's constantly stuck at 0.000000 no matter the number of epoch or output layer being SoftmaxOutput or LogisticRegressionOutput. So I am no sure where I got it wrong.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Update tooling

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (192ed4d) 98.85% compared to head (61a7956) 98.85%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #140   +/-   ##
=======================================
  Coverage   98.85%   98.85%
=======================================
  Files           1        1
  Lines          87       87
  Branches       12       12
=======================================
  Hits           86       86
  Misses          1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Create DebianDebPackaging

Description
Documentation of work done for this enhancement:
https://trello.com/c/wXo9Dny0/432-create-debian-package-for-splashkit Have been unable to implement requested functionality, this documents the work done so far.
This is not a change per-se, rather a log of work & research done for parent ticket
Type of change

[X ] Documentation (new)

@Wills2022 - Guessing this PR can be closed since it was added to the "documentation" repo?


cannot get token by username and password

try to get the token from password mode by entering username and password.
the system give you the error :

Error: param does not match pattern ^[a-zA-Z0-9_:-]+$.

in OAuth2RequestFactory.createTokenRequest in the code:
        if (authorizationGrantType == PASSWORD) {
            String username = readParameter(requestParameters, "username", STRING_PATTERN);

Can you paste the username? Does it match the pattern?

Can you paste the username?

the username is<EMAIL_ADDRESS>
Does it match the pattern?

no, cause it contains "." and "@"


How to make the background color cover the whole editing area?

Please search right side padding strategies on Blockman page. There you can see the GIF instruction.
In short, I guess you want something like this:
Go to VS Code settings,
Search blockman n04 sub04
Choose Rightmost Edge Of Viewport


How to check for overpaid?

The API docs leave me puzzled how I should check for overpaid (or underpaid) payments.
Currently my webhook is checking for:
if ($event->type == 'charge:confirmed' || $event->type == 'charge:resolved') {

Hi @Fossil01, unfortunately the current api does not support under/overpaid for webhooks. Do you have a specific reason or use case, maybe it would help me understand the need more clearly.

Hi @Fossil01, unfortunately the current api does not support under/overpaid for webhooks. Do you have a specific reason or use case, maybe it would help me understand the need more clearly.

@Fossil01 oops, correction - under/overpaid will appear on the charge:failed event, within the timeline context. Sorry for the confusion, if you have a follow-up question feel free to open another ticket, will close this one out now.


MkDocs upgrade breaks existing valid projects

Details

Read the Docs project URL: https://nodemcu.readthedocs.io/
Build URL (if applicable): https://readthedocs.org/projects/nodemcu/builds/7420945/

Expected Result
As the project builds locally just fine with MkDocs 0.17.3 it should build fine on RTD as well.
Actual Result
The build fails with
WARNING -  Config value: 'theme_dir'. Warning: The configuration option {0} has been deprecated and will be removed in a future release of MkDocs.

Aborted with 1 Configuration Warnings in 'strict' mode!

I double checked my mkdocs.yml but there's no theme_dir anywhere in sight: https://github.com/nodemcu/nodemcu-firmware/blob/dev/mkdocs.yml. However, thanks to the cat mkdocs.yml output you provide for every build (great idea!) I see that the RTD build adds it!
So, if you manipulate our build config on-the-fly then please do it in a way that doesn't break builds.
I'm happy about the MkDocs upgrade, that I found out about by coincidence, thank you 👏. But couldn't you inform affected users/projects by email when you do major upgrades? Any "release" or upgrade channel we can subscribe to?

Read the Docs modifies mkdocs.yml in order to inject the version menu where additional versions of the docs are linked and a few other things. See here. We (I) tried to do this in a compatible way in case people had pinned to a previous mkdocs version in their requirements file. However it does create a warning in the latest mkdocs. Because you've specified strict mode in your config, that became an error. As a short term fix to get building again, turn off strict mode.
As a better fix I suppose we will have to detect the version of mkdocs and do the right thing based on the version. The other opetion is to change how we inject that version data.

I'm happy about the MkDocs upgrade, that I found out about by coincidence, thank you 👏. But couldn't you inform affected users/projects by email when you do major upgrades? Any "release" or upgrade channel we can subscribe to?

We don't have an announcements mailing list but I do think that's a good idea. Currently the deployed version number is shown in the footer and links to the changelog.

I get the thing about the version menu. However, the warning appears to be about the deprecated theme_dir option. Why does RTD need that at all? Isn't the theme available in the environment under the readthedocs name (making theme_dir superfluous)?

We don't have an announcements mailing list but I do think that's a good idea.

How about a new @readthedocs-release on Twitter to which your release script posts a link to the changelog?

Why does RTD need that at all? Isn't the theme available in the environment under the readthedocs name (making theme_dir superfluous)?

We override the default readthedocs theme with a few extra templates. https://github.com/rtfd/readthedocs.org/tree/master/readthedocs/templates/mkdocs/readthedocs

Oh, it finally sunk in! You cannot install your theme because it's just a theme customization rather than a full theme.

I don't see a way around version detection. Do you?

Nope, sorry. I don't know how MkDocs is gonna behave once theme_dir (or any other option) isn't just deprecated but not supported anymore. Even the best strict: false won't help anymore. Unless their parser simply skips unknown options it's inevitable that you add version detection.
If it were just about theme options there may be a way around it but my idea is so half-baked that I don't dare to mention it.

I don't know how MkDocs is gonna behave once theme_dir (or any other option) isn't just deprecated but not supported anymore. Even the best strict: false won't help anymore.

Agreed. We will need a solution by then. I think RTD needs to get a bit smarter in how or even whether versions of sphinx/mkdocs are able to be overridden. This has been the source of a number of problems. Maybe the version used is specified in .readthedocs.yml or something and it can't be overridden by the requirements file.

Hi. I work with Marcel, so have been tracking this. The strict option does have functional advantages for a project in terms of extra checking that most projects should want to do, so this workaround does represent a functional loss at least in terms of QA.
On a separate note, Readthedocs is a great service. Thanks :smile:

As I mentioned, I view turning off strict as a short term fix to get building again. I think version detection -- that is, detecting the version of mkdocs and doing the right thing -- is a better fix but that will require changes. Do you agree? Do you see another alternative?

Do you agree? Do you see another alternative?

Yes. No.   :+1:

Because I landed here after having my RTD build break w/ strict: true...
Changing the mkdocs.yml:
theme: "readthedocs"

to
theme:
  name: "readthedocs"

fixes the issue (and lets you keep strict)
Ref: https://github.com/gluster/anthill/pull/37

@JohnStrunk not sure if this is a catch-all solution. I noticed that on https://nodemcu.readthedocs.io/en/latest/ the fly-out menu bottom left is gone/broken (used the above snippet).

Nevermind, it's actually the same issue I noticed already back in January: https://groups.google.com/d/topic/mkdocs/v7AVbeB105w/discussion. It broke when RTD moved the default MkDocs version to >= 0.16 (currently 0.17.3).

I'll report it as a separate issue here
@davidfischer I'd like to register our NodeMCU project as one of the MkDocs projects you verify the correctness of your changes against (silently assuming something like this exists). How shall I proceed?

@marcelstoer I'll definitely check your project going forward.


Update method now may receive strings with GTID sets

Hi, I think your PR will enable supporting multiple GTID update, but will it happen in practice? Every binlog event should contains only one GTID

Every binlog event should contains only one GTID

Hi!
Sure, but GTIDSet types may be used for  general GTID calculations.
For instance, we may calculate full GTID stored on the replica as  Retrieved_Gtid_Set + Executed_Gtid_Set
Both may contain multiple GTIDs from different sources.
P.S. Actually, I'm going to open source for mysql HA tool we use in cloud.yandex.ru, it uses patched go-mysql for GTID calculation. So I thought it would be right to publish this patch and use upstream version of go-mysql.


After ran "python manage.py db upgrade", I had to manually fix on line 72 this file /opt/doorman-master/migrations/versions/30e928796227_.py

Hi Marcin,
I tried to link with you on Linkedin. I'd love to connect with you.
Anyways, I am "playing" with your tool.. pretty good job and I liked. Since I am new to Flask I ahd some rocks on my way. See bellow:
bash-4.2$ python manage.py db migrate
INFO  [alembic.runtime.migration] Context impl PostgresqlImpl.
INFO  [alembic.runtime.migration] Will assume transactional DDL.
INFO  [alembic.autogenerate.compare] Detected added table 'distributed_query'
INFO  [alembic.autogenerate.compare] Detected added table 'file_path'
INFO  [alembic.autogenerate.compare] Detected added table 'node'
INFO  [alembic.autogenerate.compare] Detected added table 'pack'
INFO  [alembic.autogenerate.compare] Detected added table 'query'
INFO  [alembic.autogenerate.compare] Detected added table 'rule'
INFO  [alembic.autogenerate.compare] Detected added table 'tag'
INFO  [alembic.autogenerate.compare] Detected added table 'user'
INFO  [alembic.autogenerate.compare] Detected added table 'distributed_query_task'
INFO  [alembic.autogenerate.compare] Detected added table 'file_path_tags'
INFO  [alembic.autogenerate.compare] Detected added table 'node_tags'
INFO  [alembic.autogenerate.compare] Detected added table 'pack_tags'
INFO  [alembic.autogenerate.compare] Detected added table 'query_packs'
INFO  [alembic.autogenerate.compare] Detected added table 'query_tags'
INFO  [alembic.autogenerate.compare] Detected added table 'result_log'
INFO  [alembic.autogenerate.compare] Detected added table 'status_log'
INFO  [alembic.autogenerate.compare] Detected added table 'distributed_query_result'
Generating /opt/doorman-master/migrations/versions/30e928796227_.py ... done
bash-4.2$ cp /opt/doorman-master/migrations/versions/30e928796227_.py /opt/doorman-master/migrations/versions/30e928796227_.py.bak
bash-4.2$ vi /opt/doorman-master/migrations/versions/30e928796227_.py
bash-4.2$ vi +72 /opt/doorman-master/migrations/versions/30e928796227_.py
bash-4.2$ python manage.py db upgrade
INFO  [alembic.runtime.migration] Context impl PostgresqlImpl.
INFO  [alembic.runtime.migration] Will assume transactional DDL.
INFO  [alembic.runtime.migration] Running upgrade  -> 30e928796227, empty message
Traceback (most recent call last):
File "manage.py", line 101, in
manager.run()
File "/usr/lib/python2.7/site-packages/flask_script/init.py", line 412, in run
result = self.handle(sys.argv[0], sys.argv[1:])
File "/usr/lib/python2.7/site-packages/flask_script/init.py", line 383, in handle
res = handle(*args, **config)
File "/usr/lib/python2.7/site-packages/flask_script/commands.py", line 216, in call
return self.run(*args, **kwargs)
File "/usr/lib/python2.7/site-packages/flask_migrate/init.py", line 235, in upgrade
command.upgrade(config, revision, sql=sql, tag=tag)
File "/usr/lib/python2.7/site-packages/alembic/command.py", line 174, in upgrade
script.run_env()
File "/usr/lib/python2.7/site-packages/alembic/script/base.py", line 397, in run_env
util.load_python_file(self.dir, 'env.py')
File "/usr/lib/python2.7/site-packages/alembic/util/pyfiles.py", line 93, in load_python_file
module = load_module_py(module_id, path)
File "/usr/lib/python2.7/site-packages/alembic/util/compat.py", line 79, in load_module_py
mod = imp.load_source(module_id, path, fp)
File "migrations/env.py", line 87, in
run_migrations_online()
File "migrations/env.py", line 80, in run_migrations_online
context.run_migrations()
File "", line 8, in run_migrations
File "/usr/lib/python2.7/site-packages/alembic/runtime/environment.py", line 797, in run_migrations
self.get_context().run_migrations(**kw)
File "/usr/lib/python2.7/site-packages/alembic/runtime/migration.py", line 312, in run_migrations
step.migration_fn(**kw)
File "/opt/doorman-master/migrations/versions/30e928796227_.py", line 72, in upgrade
sa.Column('alerters', postgresql.ARRAY(String), nullable=False),
NameError: global name 'String' is not defined
So, to "fix it" (I am not a python nor Flask developer), I just edited that line 72 from
sa.Column('alerters', postgresql.ARRAY(String()), nullable=False),
to
sa.Column('alerters', postgresql.ARRAY(sa.String()), nullable=False),
So, in this way I was able to start Doorman. Great!
Now, I am trying to figure out, if during the enroolment, I would be able to have the workload (the client) to send a tag. I know, that doorman you can have a default tab, but it ould be great if the client could send this information during enrollmanet. What do you think?
Best Regards,
Flavio.

Hi @flaviod, you should not have to run python manage.py db migrate at all. That's done by developers when generating new db migration files.
You only need to run python manage.py db upgrade.
Regarding your second request, I don't think osquery has this capability... unless you do what the Heroku Windmill project does and overloads the enroll secret with some colon separated values.


fix: set fixed evaluators order and title as classmethod

Added a progress bar instead of the faiss output we were getting before. With it we can keep tracking the progress, but now it's an actual tracker, not an "I know it is working because I can see some unintended output getting printed".


User should not be able to delete all tables

It shouldn't happen, but if a user selects n tables and then chooses to remove all of them, then what will happen is that they user will be taken to the Edit headings screen, which doesn't make any sense.
See video attached. I selected 2 tables and then removed both and continued onto Edit headings.
I suggest we add an error message and tell the user that they cannot delete the final remaining table.
https://user-images.githubusercontent.com/11212467/116540897-accb1e00-a8e2-11eb-83bc-d9e347664759.mp4

@sabahfromlondon let us know your solution for this.

Most likely deleting all tables is a user error and not want they want to do. I think the following should happen

The user clicks "Remove table" for the last remaining table or only table
An error message should appear
Content for error message "You cannot remove all of the tables. If you want to remove [table name] then please restore one of the other tables or re-select available tables."

"Cancel" and "Re-select available tables" buttons should be provided

"Cancel" keep the user on the current screen with the "Remove table" button unselected
"Re-select available tables" takes the user to the Select data screen

Please see the update


stuck at creating ramdisk

hello, when i run the script, its stuck at creating ramdisk
my output:
****@Lenovo-Thinkpad-T440p:~/palera1n$ sudo ./palera1n.sh --dfu 15.2
palera1n | Version 1.0.0
Written by Nebula | Some code and ramdisk from Nathan | Patching commands and help from Mineek | Loader app by Amy
[] Getting device info...
[] Creating ramdisk
****@Lenovo-Thinkpad-T440p:~/palera1n$ sudo ./palera1n.sh --dfu 15.2
palera1n | Version 1.0.0
Written by Nebula | Some code and ramdisk from Nathan | Patching commands and help from Mineek | Loader app by Amy
[] Getting device info...
[] Creating ramdisk

can you add --debug to the end of the command and show me the output

/palera1n$ sudo ./palera1n.sh --dfu 15ug --deb
palera1n | Version 1.0.0
Written by Nebula | Some code and ramdisk from Nathan | Patching commands and help from Mineek | Loader app by Amy
[] Getting device info...
[] Creating ramdisk
[*] Getting device info... this may take a second
img4tool version: 0.197-aca6cf005c94caf135023263cbb5c61a0081804f-RELEASE
Compiled with plist: YES
Saved IM4M to work/IM4M
Version: 3fc8c093f4660f6c6e07c0c9214618733da01ffc - 36
libfragmentzip version: 0.60-120447d0f410dffb49948fa155467fc5d91ca3c8
init pzb: https://updates.cdn-apple.com/2021FallFCS/fullrestores/071-97865/96DA3A71-1A8E-43C8-AC78-64EEFAB356B3/iPad_Pro_HFR_14.8_18H17_Restore.ipsw
init done
getting: BuildManifest.plist
0% [                                                                      0% [                                                                                ]
download failed
[-] An error occurred

and i do have internet

Are you able to access that link

yep

and now its downloading the file lol

Maybe it was just a temporary internet/Apple outage, it happens

when i click the link

Oh, uhh

What happened here?

--dfu 15ug --deb

i dunno but it sems it has not copied the "ug"

Alrighty


fix: edges disappear when collapsing combo, closes: #2816; fix: drag-…

fix: edges disappear when collapsing combo, closes: #2816;
fix: drag-node with edge key, closes: #2819;

This also fixes #2825


Hhh 16759 merge fails when entity has an embedded java record transient

HHH-16759 When ComponentType is immutable, use target value instead of setting property values
This PR is replacing dead PR 6820

I stumbled into some other issues while looking into your previous PR which I wanted to analyze further. Sorry it took so long for me to get back to you. Thanks for your work on this. Superseded by https://github.com/hibernate/hibernate-orm/pull/7081


Adding draft work on hotwire concept for implementing dependent dropdown

Resolves #4398
Description

Type of change

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
Documentation update

How Has This Been Tested?

Screenshots

Hi @patelkrunal31 -- can you remind me what the status of this is and if you are actively working on it? IIRC I think we decided to close this and try another approach, but I can't remember.

Thanks for this reference work. The version we'll merge is over at #4510, so I'm closing this attempt.


(enhance)controller: default to the chaosengine resource namespace if app namespace is not specified

Signed-off-by: ksatchit<EMAIL_ADDRESS>
What this PR does / why we need it:

Adds default Kubernetes pattern of looking for target application in the same namespace where the chaosengine resource is created, if the .spec.appinfo.appns is not explicitly specified.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #255
Special notes for your reviewer:

We haven't made the attribute omitempty, just so we get a bit more visibility in terms of spec/debug. Open to thoughts/changes.

Checklist:

[ ] Fixes #
[ ] Labelled this PR & related issue with documentation tag
[ ] PR messages has document related information
[ ] Labelled this PR & related issue with breaking-changes tag
[ ] PR messages has breaking changes related information
[ ] Labelled this PR & related issue with requires-upgrade tag
[ ] PR messages has upgrade related information
[ ] Commit has unit tests
[ ] Commit has integration tests

##Testcases

Positive: Provide .spec.appinfo.appns as usual
Positive: Skip .spec.appinfo.appns -- operator looks for app in engine ns
Negative: Skip .spec.appinfo.appns, have no deploy with the specified label & verify if the right error is seen
Negative: Skip .spec.appinfo.appns, have .spec.annotationCheck: true & verify if the right error is seen


refactor: Add Recognizers-Text packages to botbuilder-stdlib

#minor
Related PR: https://github.com/microsoft/botbuilder-js/pull/ 4524
Description
This PR adds https://github.com/microsoft/Recognizers-Text into botbuilder-stdlib library, so it's easier to update vulnerable dependencies without introducing breaking changes.
botbuilder-stdlib is an internal library that contains helper functions which the rest of botbuilder libraries use. Therefore, we included the Recognizers-Text packages under the vendors folder, so botbuilder libraries can consume them.
Specific Changes

Updated libraries/botbuilder-repo-utils/src/workspace.ts to search workspaces, taking into account .gitignore and the location where the command is run. Otherwise, it will cause unwanted behaviors where nested dependencies inside node_modules were considered as a workspace.
Added all Recognizers-Text packages into the vendors folder.

All dependencies have been moved to botbuilder-stdlib package.json, so they are installed properly.
Updated all types references to match each Recognizers-Text package inside botbuilder-stdlib.

Testing
The following image shows the bot working with the new Recognizers-Text packages inside BotBuilder.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 84.076%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
20344

Relevant Lines:
22901

💛  - Coveralls

/promoted 4660


Add a config file option that turns off the extra messages for newbies

As long as there are no boring newcomer-only messages in the game (so far that's the case), this is not urgent.

Implemented. As a "convenience option" in main menu. First tutorial hint messages are coming.


Advice needed

@lisachenko I believe I might be able to use this framework for eliminating redundant test assertions but am not sure how to implement it so want your advice. Here is how the current code works:
\RedTest_TestCase_Framework extends \PHPUnit_TestCase_Framework with some default methods. For testing the application, users will need to extends \RedTest_TestCase_Framework. Currently here is how a very simple test is implemented:
$articleForm = new ArticleForm();
$this->assertTrue($articleForm->isSuccess(), $articleForm->getErrors());

list($success, $fields, $msg) = $articleForm->fillRandomValues();
$this->assertTrue($success, $msg);

list($success, $articleObject, $msg) = $articleForm->submit();
$this->assertTrue($success, $msg);

list($success, $msg) = $articleObject->checkValues($fields);
$this->assertTrue($success, $msg);`

I would like to convert the same to the following with assertions happening in the background automatically:
$articleForm = new ArticleForm();
$fields = $articleForm->fillRandomValues();
$articleObject = $articleForm->submit();
$articleObject->checkValues($fields);

Here is what I am thinking. Using your framework, I should be able to use Around invocation to first call the appropriate function and then check whether $success is TRUE or not. If it is, then just return $var. If it is not, then make the test fail. In other words, here is how it will look:
function aroundMethodExecution(MethodInvocation $invocation) {
  list($success, $var, $msg) = $invocation->proceed();
  // This is where it becomes problematic.
  // I want to implement the following here:
  if (method_exists($myTest, 'assertTrue') {
    $myTest->assertTrue($success, $msg);
    return $var;
  }
  return array($success, $var, $msg);
}

I have two questions:

Is this an appropriate use of your framework? If not, what would you recommend?
If yes, then how will I get $myTest class in aroundMethodExecution() function?

Thank you!

Ok, looks like the AspectMock framework can give your a working solution (it's based on top of goaop).
However, I don't recommend you to use AOP techniques for optimizing such small parts of code, especially for testing, as this can lead to worse readability and bad maintenance of tests.
You can also try to simplify your tests code by using just Test Spy (http://css.dzone.com/books/practical-php-testing/practical-php-testing-patterns-38) pattern an put all logic for checking into your methods, instead of repeating them in every test.

Ok, looks like this issue can be closed as well.
I'm happy that you can chose now the right way for making your code cleaner and transparent. Feel free to ask me anything if needed.


Examples on how to use would be really appreciated.

I'm having a little bit of a problem to figure out how to use this. Would appreciate any demo.

@digiiitalmb I apologize for replying you late. You can refer to the usage here because it is the same in terms of usage. I'll update the demo in time. Thank you.


predis adapter

PHP redis in an extension, while Predis in a plain php library, some people cannot include extensions in their servers do to shared hosts, cheap hostings or may not even know how to (compilation, etc)


Rename Customers to Tenants

I've used this great plugin in 5 SaaS applications and there's one main confusion in this. Usually most of multi-tenant applications have a vision to provide software for multiple businesses which have their own clients/customers. Now the confusion that is particularly with new developers on an existing system I'm facing is the term "Customers". I've renaming my tenant's customers table to clients and on some I used the name Customers too.
But I wish the customers term in this plugin could be renamed to Tenants. So we also have the flexibility to create separate customers table for customer profiling etc.

hyn/multi-tenant version: 5.1
laravel version: 5.6

The "Customer" model (and related functionality) has been removed in the 5.x branch in preparation for 5.2.  Additionally, There will be a Tenant interface, so you can use any method of tenant identification you wish.  I'm sure @luceos could go into more detail if you wish.


Replaces native scrollbar with custom for group indicator

Description
Fixes #1789
Notes (out of scope, known isues, hints for reviewing code, ...)  (optional)
Windows emulator:

Test cases

[ ] Case 1:

Given I am the usual user
When I go to this page
And I hover on group indicator
Then I see custom scroll

[auto-generated message] E2E test report on: https://algorea-static.s3.eu-central-1.amazonaws.com/branch/bugfix/group-indicator-scroll/playwright-report/index.html

Do I create another issue for #1789 (comment) ? Isn't a way to fix them all at once?

I would like to create new issue as this is common modal component, need to determinate where we want to turn off scroll, in permissions modal for sure we need it


[osf] project search seems to only search through expanded collections

Steps to reproduce:

Go to your project dashboard
With 'all my projects' collapsed, search for a project title that you have.

Expected behavior:

The search should find the project you are looking for

Actual behavior:

Nothing comes up. The search will only return a project if the 'all my projects' tab is expanded. So, it seems to just be searching through whatever is currently visible on the project dashboard, not anything underneath the visible level.

This occurs on Safari 10.9.5 and Firefox 38.0.5, but does NOT occur in Chrome. I don't have IE so I couldn't check that.

Duplicate of https://github.com/CenterForOpenScience/osf.io/issues/2779 (though you wouldn't know it without reading through the discussion)

Alllthough, it looks like they may be focusing on files tab for the fix, so maybe we'll keep this open.

Yeah, I saw that and wasn't sure if this was different enough to get it's
own issue or not? But it seemed to be talking about different parts of th
system, so I just made one and hoped for the best.
On Thu, Jun 11, 2015 at 4:33 PM, Brian J. Geiger<EMAIL_ADDRESS>wrote:

Alllthough, it looks like they may be focusing on files tab for the fix,
so maybe we'll keep this open.
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/3086#issuecomment-111270970
.

--
Courtney Soderberg, Ph.D
Statistical and Methodological Consultant
Center for Open Science

I just noticed that there is also an odd connection with #3235; if you add something to a collection, it will always appear in the search until the collection is deleted, even if it is removed from the collection.


C++ style for enum classes is missing

The style guide doesn't mention how to label enum classes:
http://www.cprogramming.com/c++11/c++11-nullptr-strongly-typed-enum-class.html

What do you mean by how to label them?

Sorry for being unclear, with "label" I mean the naming rules:
http://google.github.io/styleguide/cppguide.html#Enumerator_Names
I think for class enums that are always used with a namespace, it might make sense to not name them anymore like constants or macros.
AlternateUrlTableErrors::MALFORMED_INPUT looks quite strange to me...

We've discussed this a few times. For the time being, we've decided to leave the rule as it is, with the same rules for scoped and legacy enums.

Ok, thats fine. But maybe this should be mentioned in the guide, though?

I don't see why. The style rule is already pretty clear that it applies to all enumerators.

It wasn't clear to me, otherwise I wouldn't have asked ;)
Just thought it wouldn't hurt to make it explicit by adding "enum (scoped and unscoped)" to the text, or by adding an example with an "enum class" enumerator...

OK, I've added a clarification to our internal copy of the style guide. It will become visible on GitHub the next time we do a push.

Great, thanks :)

For my two cents, enum class forces qualification so that you shouldn't need the "hungarian notation" of 'k'.
I am guessing the history of this guideline startd with the observation that constant declarations such as const int x = 1; demanded clarity by changing to const int kX=1; and unscoped enums are semantically equivalent to const int (or whatever the chosen base integer type is).  I wonder what the argument was to apply the same style to scoped enums?

@bkramer-moz I totally agree with that, the k naming for scoped enum classes is pointless.

@bkramer-moz The purpose of a naming convention like this is to make information clear at the point where the name is used, not necessarily at the point where it's defined. In a usage like Foo::bar, it's not obvious that bar is immutable, because it's not obvious that Foo is an enum.


0.14 td-agent, buffers combined randomly in TCP source with heavy traffic case.

Check CONTRIBUTING guideline first and here is the list to help us investigate the problem.

Version
: td-agent 0.14.16
OS
: Ubuntu 16.04.2 LTS AND Windows 2012 R2 have same problem.
Your configuration
:

<system>
  workers 7
</system>
<source>
  @type  udp
  format json
  tag    TAG
  bind   <IP_ADDRESS>
  port   UDP_PORT
  body_size_limit 5MB
</source>

<source>
  @type  tcp
  format json
  tag    TAG
  bind   <IP_ADDRESS>
  port   TCP_PORT
</source>

<source>
  @type http
  port 8888
  bind <IP_ADDRESS>
  body_size_limit 32m
  keepalive_timeout 10s
</source>

<match **>
  @type forward
  heartbeat_type none
  time_as_integer

  send_timeout 30s
  buffer_queue_limit 256
  expire_dns_cache 30

  <server>
    host DESTINATION_HOST
    port DESTINATION_PORT
  </server>

  <secondary>
    @type file
    path /var/log/td-agent/forward-failed
  </secondary>

</match>

Your problem explanation. If you have an error logs, write it together.
: When fluentd receives using TCP source with so much traffic, sometimes it seems to mix buffer contents. For example, when A and B processes send packets {"key": "AAAA", "value": "111"} and {"key": "BBBB", "value": "22222222"} for each processes, fluentd sometime mixes the buffer contents like {"key" : "AAAB", "value":"22222222"}. It seems that two different packets are merged to one packet randomly.
This usually doesn't appear, but sometimes does.

My test case is below.
: make C++ program to send JSON packets with infinite loops. Let say TestSender.exe
: run 4 more TestSender.exe processes to give stress to Fluentd.
: If Fluentd configuration matches all packet to stdout or file, then you can find this error.
In addition, UDP and HTTP sources haven't appear like this problem. Only TCP source does.

Sorry for the late.
This problem is similar problem to https://github.com/fluent/fluentd/issues/1516.
Your configuration set workers 7 but it happens with workers 1, right?
Could you share your test problem for reproducing the issue?

In addition, UDP and HTTP sources haven't appear like this problem. Only TCP source does.

Hmm... in_http also uses TCP connection so frequent request may be the factor.

Yes, it happens with workers 1 too.
I'm not sure that the test program file could be shared in public. I'll check whether it's possible or not.
Hm.. A difference between http and tcp in my case, I'm suspicious that in_http closes connection after finishing sending one packet(probably one JSON content), but in_tcp keeps its connection until exiting the test program. I hope it could be helpful.

but in_tcp keeps its connection until exiting the test program. I hope it could be helpful.

I see. We need to check connection handling of libev when connection is not closed.

Could you test https://github.com/fluent/fluentd/pull/1729 patch with your test workload?

Sorry for the late.
My teammate and I have tested #1729 patch and the problem is fixed. Maybe a cause of this issue seems to be sharing buffers. Thank you for support.

FWIW I have this issue as well, and I've tested PR #1729 and can confirm that it fixes the problem for me.

Release v0.14.23.rc1 for the fix.

I am a colleague of the author of https://github.com/fluent/fluentd/pull/1729. We've tested this on our infrastructure successfully. 0.14.23 would be a great help.

You can use v0.14.23.rc1. It includes this fix.


Foundation: expose _handle from FileHandle on Windows

Add a SPI for Windows to get the raw HANDLE to enable use of Pipe
where a raw file descriptor may be needed.

@swift-ci please test

@swift-ci please test


add sha3 to the hash benchmark

Thank you! Very interesting to compare. One quick question: is there a reason you call sha256.input() twice (on lines 180 and 181)?

Oops, it's an error, we're doubling the size of the inputs. Thanks for catching it! I checked the number I gave on ethresearch are still ok, we doesn't break the block size but it's pure luck. I'm going to update the pull request...

No worries. Thanks for fixing it!


Issue with sourcemaps (later transformations)

Please refer to https://github.com/floridoo/gulp-sourcemaps/issues/124.
I am experiencing exactly the same problem using the following Gulp task (ES2015):
gulp.task("build-css", () => {
    return gulp.src(cssSrc)
        .pipe(newer({
            dest: cssDest,
            ext: ".css"
        }))
        .pipe(sourcemaps.init({debug: true}))
        .pipe(sass({
            // See: https://github.com/sass/node-sass/issues/957
            //outputStyle: "compressed"
            outputStyle: "expanded"
        }).on("error", sass.logError))
        .pipe(autoprefixer({
            // Default for Browserslist (https://github.com/ai/browserslist) @ 2015/09/18
            browsers: ["> 1%", "last 2 versions", "Firefox ESR"]
        }))
        .pipe(sourcemaps.write(".", {debug: true}))
        .pipe(gulp.dest(cssDest));
});

Could it be the same thing described in this comment?
Versions:

OS: Windows 7 (x64)
Node: 4.1.1
NPM: 3.3.4
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

@glen-84 please create a small github repo with a working gulp task that shows this issue in action.
There is simply too much noise in these issue to get a clear idea of what is happening. We need to see this issue in action to begin debugging it.

https://github.com/glen-84/gulp-sass-sourcemaps-issue
$ npm install
$ gulp
Notes:

The issue does not occur when the file is in the root of the src folder (test.scss).
To view the working CSS, uncomment the first link element in dist/index.html.
The issue occurs with the src/app/styles/test2.scss file.
Look at the button element btn class for an example, it should point to _buttons.scss, not test2.css.

Let me know if you need anything else.
Thanks.

Sorry for the delay, I will take a look a look at this shortly.

@glen-84 in the mean time, are you able to create a failing test case?

Note to self, this could be related to https://github.com/sass/node-sass/issues/1194

@glen-84 in the mean time, are you able to create a failing test case?

That was my failing test case. :grin:
But seriously, I don't know much about unit testing JavaScript. :disappointed:

@xzyfer Did you find anything?

@glen-84 I have not had time to look into this. I'm busy until LibSass 3.3.0 ships. Sorry.

This could be related to https://github.com/mozilla/source-map/issues/216

@mariusGundersen I don't think so, it doesn't work in Chrome either.

Hi guys, here is a similar problem.
Reproduction scenario:
git clone https://github.com/clee704/gulp-autoprefixer-test.git
cd gulp-autoprefixer-test
npm install
git checkout error
gulp css  # Unhandled 'error' event
git checkout lost
gulp css
head -c 200 build/css/app.css.map  # "sources" array is incomplete
git checkout works
gulp css
head -c 200 build/css/app.css.map  # correct "sources"

It works if gulp-autoprefixer is not used. It seems the source map object returned by gulp-sass is not standard complient.

I also experience a problem with gulp-sass 2.1.0 and gulp-sourcemaps 1.6.0.
It always shows the highest level in the sass code and not the deepest one.

The last version where the tests we specified in https://github.com/unic/estatico/tree/develop/test/css still work (clone the repo and run npm test) is 2.0.3 in combination with node-sass <= 3.3.3. If I update either gulp-sass to >2.0.3 or node-sass to >3.3.3, the tests break.

@xzyfer Now that LibSass 3.3.0 has shipped, when do you think that you might have time to look into this?

Unfortunately no one has submitted a test case showing exactly what sass is
required to break source maps. We know there are issue but source maps
complicated and with very specific test cases there isn't much we can do.
On 11 Dec 2015 9:10 pm, "Glen"<EMAIL_ADDRESS>wrote:

@xzyfer https://github.com/xzyfer Now that LibSass 3.3.0 has shipped,
when do you think that you might have time to look into this?
—
Reply to this email directly or view it on GitHub
https://github.com/dlmanning/gulp-sass/issues/354#issuecomment-163898753
.

@xzyfer I don't understand. I took the time to create a very simple repo that clearly shows the issue (see this comment).
@clee704 and @backflip have also supplied test cases, and @backflip has even specified what breaks and in which versions it breaks.
What else could you need? :confused:

Switching from gulp-autoprefixer to gulp-postcss seems to "fix" it for me (suggested in https://github.com/sindresorhus/gulp-autoprefixer/pull/66#issuecomment-238671153).

@xzyfer, would you mind having another look at this?

Let's wait for https://github.com/sass/libsass/pull/2216 to land in node-sass, it certainly sounds promising.

I just updated my test repo, and the problem is still there.

My issue was that the sourcemap was pointing to the wrong files in developer tools.
To get everything working the way it should you have to pipe like this:

gulp-sass (use anything but compressed, compressing before all other pipes will mess with sourcemaps)
gulp-autoprefixer
cssnano (or the uglifier of your choice)

gulp.task( 'styles', function() {
	gulp.src( [ styleSRC ] )
		.pipe( sourcemaps.init() )
		.pipe( sass({
			includePaths: [ foundation, motionUI, datepicker ],
			errLogToConsole: true,
			outputStyle: 'compact'
		}) )
		.on( 'error', notify.onError( function (error) {
			return 'An error occurred while compiling sass.\nLook in the console for details.\n' + error;
		} ) )
		.pipe( autoprefixer({ browsers: [ 'last 3 versions', '> 5%', 'Firefox ESR' ] }) )
		.pipe( postcss( [ cssnano() ] ))
		.pipe( rename( { suffix: '.min' } ) )
		.pipe( sourcemaps.write( mapURL ) )
		.pipe( gulp.dest( styleURL ) )
		.pipe( browserSync.stream() );
});


My second revup foo

Topic: foo2
Relative: foo

Reviews in this chain:
└https://github.com/andrewspinks/revup/pull/1 My first revup foo
　└https://github.com/andrewspinks/revup/pull/3 My second revup foo

#
head
base
diff
date
summary

0
1edc9d57
97cf6512
diff
Feb 14 21:28 PM
1 file changed, 1 insertion(+)


Optionally populate volunteer data in getSections query

Adds the option to populate volunteer data in the getSections query

/cib populate volunteer get sections


Update components, partials and scss to use Frontend 0.19

This PR updates the Design System to 0.19 with the following:

Build:

Change package.json to use Frontend version 0.19-alpha

Styles:

Remove references to font-smoothing and font-stack mixins and govuk-u-font-smoothing class as these are now included by govuk-typography-common.
Add responsive margins to some components as 0.19 removes top margins from typography so this needs now to be accounted for in component styles.
Halve app-c-content padding as 0.19 uses a 15px, not 30px gutter.
Now use the spacing scale instead of the gutter spacing wherever possible - coupling spacing with gutters only makes sense in very layout-y classes.

Components/partials/layout:

Remove govuk-u-font-smoothing classes as this is now handled by govuk-typography-common.
Remove override classes where a specific component/partial class is in use and move override styles into that component/partial.
Change references to breadcrumb components to breadcrumbs. Also remove reference to breadcrumb from mobile-navigation as this is no longer used in the partial.

This PR also:

Adds app-c-example--tabs class to example if the component has both HTML and Nunjucks code.

Notes to reviewer:

It's probably easiest to review this PR by reviewing the commits which are organised into the above groupings.
The Travis build succeeding is dependent on #95 - I'll rebase it into this branch once it's merged into master.

Just to check, using the prose scope is going to come later?

Thanks @dashouse for your help, I've rebased your work into this branch.
@36degrees Yes I'll raise a separate PR for the prose scope.

all good pending minor change

👏 :shipit:


I haven't, but I don't anticipate any interactions. This only touches nets and override_names_with_designators only does Modules/components

There are further improvements we can and probably should make.
The "full_name" should be improved a bit: https://github.com/atopile/atopile/issues/547
Additionally, we might wanna force in some interface naming a little more?


Add ability to create transparent slippy images for overlays.

Allows users to create images without islands, background, etc for slippy overlays.

Added cleaner grid information option for slippy maps.
https://puu.sh/DaS7A/04f5ada410.png
See: https://antihax.github.io/ for example use.

I tried yours @antihax but some of the tiles come out entirely black.

I tried yours @antihax but some of the tiles come out entirely black.

It should generate images based on what is currently seen on the screen with the exception that it will be transparent if selected.
Which ones came out black and what were the options you had selected on the application? I will see if i can replicate tonight.

I had islands enabled, water background disabled and alpha enabled. The first few zoom levels were fine but by 4-6 a quarter of the tiles were black instead of normal

The Alpha layer is what did it Im betting
On Wed, May 1, 2019 at 9:54 AM Trevor Allen<EMAIL_ADDRESS>wrote:

I had islands enabled, water background disabled and alpha enabled. The
first few zoom levels were fine but by 4-6 a quarter of the tiles were
black instead of normal
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GrapeshotGames/ServerGridEditor/pull/72#issuecomment-488304822,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIPE4QZMHNDPJMNRI5LP2CTPTGVLLANCNFSM4HD7WGJA
.

Am I not supposed to select that or you saying there could be a bug there?

It's just a guess.  I'm not sure how any of that is implemented. I THINK
the alpha layer is the transparency. And I'm guessing that it's not being
applied properly. Unchecking it will give a solid image, but you can pretty
easily remove that all white background in photoshop or gimp and make it
transparent again. Ive been doing all of mine manually
On Wed, May 1, 2019 at 11:05 AM Trevor Allen<EMAIL_ADDRESS>wrote:

Am I not supposed to select that or you saying there could be a bug there?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GrapeshotGames/ServerGridEditor/pull/72#issuecomment-488326220,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIPE4Q6U73KH3HKGGNEZMR3PTG5SXANCNFSM4HD7WGJA
.

It should allow transparency through an alpha channel because that was the intent. I.e.
Export a base set of images which includes the background (usually the background with islands) then export additional sets with no background and just specific elements, such as grid, server information, island names, discoveries. These additional sets can then be applied as layers in leaflet.

The damn slippy export is not working properly though. Not your version
specifically. Even in the official version its broke. So Ive just about
given up on a live map
On Wed, May 1, 2019 at 12:55 PM antihax<EMAIL_ADDRESS>wrote:

It should allow transparency through an alpha channel because that was the
intent. I.e.
Export a base set of images which includes the background (usually the
background with islands) then export additional sets with no background and
just specific elements, such as grid, server information, island names,
discoveries. These additional sets can then be applied as layers in leaflet.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GrapeshotGames/ServerGridEditor/pull/72#issuecomment-488358811,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIPE4QYEFMIJA67DDEK2EX3PTHKR5ANCNFSM4HD7WGJA
.

Official works unless you want a transparent background and just the islands, then you get a blue background color.
Sent from my iPhone

On May 1, 2019, at 11:58 AM, smooreace<EMAIL_ADDRESS>wrote:
The damn slippy export is not working properly though. Not your version
specifically. Even in the official version its broke. So Ive just about
given up on a live map
On Wed, May 1, 2019 at 12:55 PM antihax<EMAIL_ADDRESS>wrote:

It should allow transparency through an alpha channel because that was the
intent. I.e.
Export a base set of images which includes the background (usually the
background with islands) then export additional sets with no background and
just specific elements, such as grid, server information, island names,
discoveries. These additional sets can then be applied as layers in leaflet.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GrapeshotGames/ServerGridEditor/pull/72#issuecomment-488358811,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIPE4QYEFMIJA67DDEK2EX3PTHKR5ANCNFSM4HD7WGJA
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

It doesnt with a custom water tile because there is no autoscale file for
it. And god knows WTF that is
On Wed, May 1, 2019 at 1:03 PM Trevor Allen<EMAIL_ADDRESS>wrote:

Official works unless you want a transparent background and just the
islands, then you get a blue background color.
Sent from my iPhone

On May 1, 2019, at 11:58 AM, smooreace<EMAIL_ADDRESS>wrote:
The damn slippy export is not working properly though. Not your version
specifically. Even in the official version its broke. So Ive just about
given up on a live map
On Wed, May 1, 2019 at 12:55 PM antihax<EMAIL_ADDRESS>wrote:

It should allow transparency through an alpha channel because that was
the
intent. I.e.
Export a base set of images which includes the background (usually the
background with islands) then export additional sets with no
background and
just specific elements, such as grid, server information, island names,
discoveries. These additional sets can then be applied as layers in
leaflet.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/GrapeshotGames/ServerGridEditor/pull/72#issuecomment-488358811
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AIPE4QYEFMIJA67DDEK2EX3PTHKR5ANCNFSM4HD7WGJA

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GrapeshotGames/ServerGridEditor/pull/72#issuecomment-488361183,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIPE4Q2PX2MVXEKIJ5MKSVDPTHLORANCNFSM4HD7WGJA
.

@antihax were you able to look into it?

Yes, i replicated and this appears to be not an issue. The black thumbnails are fully transparent images.
https://puu.sh/DnvzC/04eb6f2fae.png

@GrapeshotGames this PR should be good to go if you have time to review.

Drifted too far from master.

Drifted too far from master.


Aspire Cosmos DB Integration Doesn't Work on Apple Silicone

Cosmos emulation doesn't work on Apple Silicone. Perhaps a workaround or note should be added to the doc (or better yet a fix):
https://learn.microsoft.com/en-us/dotnet/aspire/database/azure-cosmos-db-integration?tabs=dotnet-cli
Due to:
https://github.com/Azure/azure-cosmos-db-emulator-docker/issues/54#issuecomment-2447148378

Not limited to just Apple devices.  Microsoft flagship devices that are using ARM chips also have this problem.


Rounded corners on main window

Issue
Is there a way in the css to disable rounded corners on the main window -- or is that due to macs rounding of the application window by default. I tried overriding with the css and termCss in .hyper.js, but nothing seemed to work.
Reason for this is I try to doc the terminal to the top of the screen, and the rounded corners are making me insane.
Any help with this would be great!

Please follow the issue template, knowing some information from there would have helped.
What OS are you on?

Sorry about the missing template, here are the details.
Version: Latest (1.0)
Operating System: Mac 10.11.2
Gist to .hyper.js: https://gist.github.com/ryanpager/4ab0622e3ee6f0c6c6fd57f377ea1c54

@ryanpager It is not possible to get rid of rounded corners on Mac OS unfortunately. This is an upstream limitation and we can't fix it 😞

That was my thought too -- bummer. Thanks for the help!


protofile generation from runtimeschema

hi, is it possible to generate a .proto file from a Schema generated by RuntimeSchema.getSchema(Some.class)?
thanks

Yep, I have tested, it can generate .proto from object class except interface or abstract class.
public static void main(String[] args) {
      Schema<ObjectClass> schema = RuntimeSchema.getSchema(ObjectClass.class);
      String content = Generators.newProtoGenerator(schema).generate();
      System.out.println(content);
}

here's an example


Suppress "organisms" attribute view if "Organism" is disabled in settings

Can be seeen in Project lists "Organisms: Not specified"

app/views/projects/_resource_list_item.html.erb
<% if Seek::Config.organisms_enabled %>
<%= list_item_simple_list resource.organisms, "Organisms" %>
<% end %>


Use correct integer types in Cython

Currently incorrect integer types are used in Cython implementation.
It's less portable and perhaps more importantly, confusing for developers.
For example:

Pointers should be intptr_t.
Memory sizes should be size_t.
Memory offsets should be ptrdiff_t.

Hi @ninoshi, when I was working on PRs that would touch the driver wrappers, I noticed that in driver.pxd/driver.pyx there are many types that should have been changed from size_t to intptr_t but were left untouched in #1952, which made me thought that using size_t was the deliberately chosen convention for drivers, but looking at this issue I doubt it's the case.
Specifically, I am referring to the driver functions involving the following types:
https://github.com/cupy/cupy/blob/c670b6f6f1282665c9985180c10f498d8591f909/cupy/cuda/driver.pxd#L12-L18
all of which are just pointers but were represented using size_t. Is this an oversight? If so I can make a PR to fix them.

I didn't deliberately leave them. I think I simply overlooked, so please feel free to fix them!


Struggle to proper install gpflowopt

Hi,
your package looks quite interesting. Unfortunately, I cannot install it properly. It requires an old version of gpflow==0.5.0. I was able to install gpflow==0.5.0 from github, however, it seems to be incompatible with tensorflow>=2.0.
I read the issue:
https://github.com/GPflow/GPflowOpt/issues/112
And following the steps of the last suggestion. However, I cannot pass point 5. I started to adjust some code in the gpflow source files. Eventually, I got the error 'TypeError: Variable is unhashable. Instead, use tensor.ref() as the key.'. This looks like a tensorflow issue. To resolve it, a version below 2.0 is required.
I tried python 3.6.6, python 3.7 on Windows, and python 3.8 on Ubuntu and got similar errors.
Is there any way, to set up a proper environment? Is the package still maintained?
Kinds regards

GPFlowOpt is not supported anymore, we have moved to develop Trieste based on the latest GPFlow and Tensorflow versions.
https://github.com/secondmind-labs/trieste


change default function to arrow function

Changed all base function to arraw function
Fixed element which use filter function
Changed response type on route api/manga/popular and api/chapter to json
Changed some axios instance to async and using try catch to handle error
Remove unused next function on router

Thanks for opening pr, i appreciate it. This really helps me a lot


Removes source lookup from extras.jobs.Job

Since source code is no longer displayed in Nautobot Jobs (see nautobot/nautobot/issues#1462), inspect.getsource is no longer needed. Additionally, inspect.getsource will throw an exception in some cases, such as dynamically created classes.
What's Changed

This change removes source code lookups for extras.jobs.Job. Since the job source is no longer displayed in the job view, this lookup is not needed.
TODO

[X] Explanation of Change(s)
[ ] Attached Screenshots, Payload Example
[X] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

Should we do any defensive coding pre 2.0 to just set this to None temporarily?

Should we do any defensive coding pre 2.0 to just set this to None temporarily?

Based on my discussion with @lampwins it seemed like there wasn't a plan to have the source code be visible. If that isn't the case and the source may be displayed in the future, then I think it would be better to proactively wrap it up in a try/catch block to prevent the issue that I stumbled upon today.

My vote is just to rip it out, but if @bryanculver wants to do self.source = None that is fine too.

My vote is just to rip it out, but if @bryanculver wants to do self.source = None that is fine too.
To my knowledge, self.source as never a part of the publicly documented API.

Let's rip it out then.


Provide a way to stop daemon via tooling api.

When shutting down Android Studio, we also want to shutdown daemon and right new rely on some private (semi hacky way to do so).
https://android.googlesource.com/platform/tools/idea/+/mirror-goog-studio-master-dev/plugins/gradle/src/org/jetbrains/plugins/gradle/internal/daemon/DaemonStopAction.java
Also when running tests on Windows, in order to avoid file locks, it would be much easier if we could have an orderly shutdown of the daemon.

Dupe of #8219


chore: Refactor BlockingCounter

Splits BlockingCounter/Impl into BlockingCounter and EmbeddedBlockingCounter, so the latter can be re-used
Adds DCHECKs
Adds clear comments to headers

Codecov Report
Attention: Patch coverage is 68.18182% with 14 lines in your changes are missing coverage. Please review.

Project coverage is 77.16%. Comparing base (d30de86) to head (e356ec6).
Report is 13 commits behind head on master.

Files
Patch %
Lines

util/fibers/synchronization.cc
55.55%
12 Missing :warning:

util/proactor_pool.h
66.66%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #225      +/-   ##
==========================================
- Coverage   77.60%   77.16%   -0.45%
==========================================
  Files         103      104       +1
  Lines        7824     7751      -73
==========================================
- Hits         6072     5981      -91
- Misses       1752     1770      +18

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Please rebase @dranikpg


Nomad deployment

While nomad deployment is in place, it is not production ready.
Cases where a deployment fails are not handled specially. This should be thought over and then decided on.
These are some potential issues I see:

[x] #49
[x] #50
[ ] [bug] If an app fails after deployment (runtime error due to some reason), nomad will keep restarting it, but the learner won't have access to the logs to troubleshoot it.

There are two scenarios when an app deployment could fail: deploy-time, and run-time. Both must be handled gracefully.

[ ] [bug] Nomad deployment doesn't write commit hash to the changelog. This is because changelog is only written at the end of deployment, and it doesn't have access to the Git commit hash. The run_command method only logs stdout/stderr, and doesn't capture it. It needs some changing to be able to get the commit hash with git rev-parse HEAD.
[ ] [must have] Learner must be able to see the deployment status, and the checks status of the app. For example, learner should know when checks or deployment is running. Even though UI is not a concern at the moment, we need to plan how to obtain this state.
[ ] [nice to have] Runtime logs for the app must be accessible to the learner.
[ ] [bug] Manually trigger deploys from CLI. This was previously implemented for SimpleDeployment as part of a previous task. This must now use NomadDeployment.
[ ] [nice to have] An interface to manually trigger checks for a learner. This must be accessible to server admins, through the CLI.

There are some more enhancements which could be important:

[ ] An interface to manually trigger deploys, which is accessible to the learner?
[ ] An interface to manually trigger checks, which is accessible to the learner?

What we have now is not perfect, but this should be considered done for now. Future improvements should be made with new issues.
This issue can be referred back for what to improve.


Pre-release workflow

I think this is what we need to do to be able to pre-release branches pre-fixed with /next
This would work from the changeset files alone!
How it works

if the branch has next/ prefix
If the packages changed in the branch are already in pre-release state
new pre-releases will be re-released when new changesets are added!

The branch just needs to be prefixed with next/
CC @thomasheyenbrock @dotansimha
(I change the canary scripts slightly here, mostly to clean up, i think everything should still work).
TODO:

[x] pre-release script and action
[ ] does git push back to the next/ branch work?
[ ] pre-release comment that gets updated ala canary using pr comment action
[x] logic that filters out non-prerelease releases! 👀  and prevents forks from publishing from next/ branches

@dotansimha what do you think of this approach? Do you foresee any issues?

labelling do not merge until we are able to test this PR. I guess it should only impact PRs prefixed with next/


 No tests failed after correcting this behavior).

LGTM


ASP.NET Core 3.x Minimum Version Confusion

I was briefly confused about the need for > 2.8.0 version of App Insights nuget package. Checking nuget at the time, I saw 2.14.0 was the latest version. I'm guessing your versioning changed because 2.8.0 was release some 8 months prior to 2.14.0. Might be worth tweaking the message to make that more obvious.

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

ID: 4b88d8c4-0013-70aa-7997-a2203afd9743
Version Independent ID: 0ab99d5e-4ebd-8cb9-3d27-efe58def249f
Content: Azure Application Insights for ASP.NET Core applications - Azure Monitor
Content Source: articles/azure-monitor/app/asp-net-core.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@killnine Thank you for the feedback!  We are investigating this and will get back to you shortly.

@killnine You are correct that 2.14.0 is the most recent version.  Version 2.8.0 was released in late September of 2019, followed by a jump to 2.12 in late December of 2019.  Version numbers were skipped to keep in sync with the Base SDK.
In this case, the documentation is correct in that as long as you have 2.8.0 or later (updated at least in late September of 2019 or more recently), ASP.NET Core 3.X is supported.
Useful Links:
Microsoft.ApplicationInsights.AspNetCore Version History:
https://www.nuget.org/packages/Microsoft.ApplicationInsights.AspNetCore
ASP.NET Core Releases:
https://github.com/microsoft/ApplicationInsights-aspnetcore/releases

@killnine We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Reading out results fails in a JuMP example

This JuMP example min_ellispe.jl uses SCS by default. If you switch it to Mosek, reading out the data after the end of the optimization is broken.

The JuMP.primal_status() is UNKNOWN_RESULT_STATUS
Mosek output on the terminal shows the primal,dual obj value ~6.46, but JuMP.objective_value() shows 5.0
Solution matrix is messed up, too.

Mosek 9.0 has been released so closing


Empty response content fix v3

Direct port of PR: https://github.com/Azure/azure-sdk-for-java/pull/7345
Return empty string when receiving empty response from backend
Addresses this: https://github.com/Azure/azure-sdk-for-java/issues/6955

/azp run java - cosmos - tests


Script to submit release to antivirus whitelist

What it says on the tin. At the moment we'll have to run the script manually after every release. In future we may want to consider a cron job or a webhook to automate this.

As a more general note, it looks like Rollbar has the ability to track deploys, but requires that one hit an API endpoint on release. Since our release mechanism is a GitHub publish, there's no existing script that gets run to piggyback off of easily. If we end up needing two scripts (this one and the Rollbar deploy tracking), I wonder if it'd be worth investigating writing a small script that we host somewhere to accept the GitHub webhook for releases which will automatically execute this and potentially other scripts.

@krystophv I've addressed your comments. Are you happy for me to just merge this, as it contains no user-facing changes and I'm already using it to submit to antivirus whitelist?

@gniezen Yeah, feel free to merge.


Adapt ECV export for ASL

The ECV export button for ASL generates an ECV file that has some hard-coded NGT properties, like the name. Moreover, the ECV is bilingual, including the Dutch gloss field that ASL doesn't use. So:

Change the CV_ID value to "ASL Signbank" (rather than CNGT_RU-lexicon)
Make it monolingual, containing only the English gloss
Change the description field: remove the phonology and frequency information, leaving only the comma-separated translation equivalents

@ocrasborn @Woseseltops This is done except for the last item. Going to look into that now.

@ocrasborn Done.
@Woseseltops In the commit messages I gave instructions for the server specific settings.

@Woseseltops To be sure: see also commit message b680a7d

Updated the live repo and followed the instruction in the commit messages here: https://github.com/Signbank/NGT-signbank/commit/7e0471322218b6434388c3b945220235e1987a74


python3 working

This make run with python3 but not 'work' (load ui but show it wrong) :(
Tested demo.py with python2 and it's ok.
Anyway I think running is good start point for anyone (even me) to continue your work.
Basically:

Add () to print
Use . from import
Use items for iteritems
force values to int at get_vertice

For this last, I don't understood the problem but, looks like some operation return int with python2 but now (python3) return float changing your expected display result.
Thank You

With more tests I found this
https://docs.python.org/3/howto/pyporting.html#division
Then adding  "from __ future __ import division" we have same problem for python2 also.
Corrected adding // to code
Other problem is the way python3 interpret byte
http://stackoverflow.com/questions/6269765/what-does-the-b-character-do-in-front-of-a-string-literal

Edit:
Corrected problems with python3
demo.py was shows same with both (py2 and py3)

Awesome work! I can confirm it works in both python 2 and 3. Thanks for the contribution


WIP: Move to DISCONNECTED on unexpected WebSocket close.

Should fix #655.

@ricardopereira I can't run the test suite anymore.

I guess this is due to the latest Carthage+Cocoapods fixes.
Also, I've noticed that pod install doesn't install all dependencies now:
Analyzing dependencies
Downloading dependencies
Using Aspects (1.4.1)
Using Nimble (7.0.0)
Using Quick (1.1.0)
Using SwiftyJSON (3.1.4)
Generating Pods project
Integrating client project
Sending stats
Pod installation complete! There are 4 dependencies from the Podfile and 4 total pods installed.

Those are the test dependencies only. What about KSCrash, msgpack, etc.?

I improved the README #658

Thanks @ricardopereira, that did the trick.


Issues with capturing PS1 games

When capturing a PS1 screenshot, it's really messed up. I have no idea if that's a pngshot issue or expected with PS1 games (or just something weird with Symphony of the Night).

Same, i searched and found the  solution and it worked for me
https://wololo.net/talk/viewtopic.php?t=49536
Even so you will play your games on stretched mode but I find it fine

Here's a workaround:

Open the Adrenaline menu, press R to go to the Settings section, and set “graphics filtering” to “Sharp Bilinear Simple ( No Scanlines )”. In my experience, this also makes it necessary to open the official settings to turn off bilinear filtering if that is desired.
If using Adrenaline Bubble Manager, set the PS button for your PS1 bubbles to open the Adrenaline menu rather than the LiveArea. I can't rightly recall whether this is strictly necessary, but I seem to remember that it helped.

Result:

Here's a workaround:

Open the Adrenaline menu, press R to go to the Settings section, and set “graphics filtering” to “Sharp Bilinear Simple ( No Scanlines )”. In my experience, this also makes it necessary to open the official settings to turn off bilinear filtering if that is desired.
If using Adrenaline Bubble Manager, set the PS button for your PS1 bubbles to open the Adrenaline menu rather than the LiveArea. I can't rightly recall whether this is strictly necessary, but I seem to remember that it helped.

Result:

There's also PsOneScrot


Add event to google calendar feature

Addresses issues #405
This PR adds google calendar links to event pages.
Some notes:

I did not add the "Z" to the end of the ISO string for startdate and enddate in the google calendar URL link (see https://gist.github.com/cdncat/95801f3237424909f9372ceabe6d4679) because it seems to adjust the event times based on the user's local time settings
if no endtime is specified, the event will run from startime and end at starttime
if no end time nor start time are specified then it will be an event starting at 12am of the specified date

I tried handling three cases for event pages (assuming date is always available):

end time and start time do not exist
start time exists but end time does not
start time and end time both exists

I did some basic testing that it works for various events but can someone please test that the times is correct when you click on the google link?

Looks good! Could you move formatDate.js to formatDate.js and write a unit test?

I added getCalendarURL to the util section and added a test for it :)

Awesome! Thank you! 🎉

no problem! :)


Documentation missing for setting credentials

It is possible to set credentials if the server requires HTTP Basic Authentication but neither the readme nor the wiki includes documentation for these attributes.

Yeah it was added by a PR and I missed that, I'll update it.

Done


Update to newer stable version of LAMMPS

We are currently using "r12824" from lammps (list of stable versions at http://lammps.sandia.gov/bug.html ).  However, the git tag is missing (removed?) in the last week.  In addition the install process for building the shared library has been changed.

@TobiasRasp

I was able to check out the 'r12824' version without a problem:
Cloning into 'mylammps'...
remote: Counting objects: 7141, done.
remote: Compressing objects: 100% (5337/5337), done.
remote: Total 7141 (delta 2277), reused 4532 (delta 1740)
Receiving objects: 100% (7141/7141), 57.49 MiB | 881.00 KiB/s, done.
Resolving deltas: 100% (2277/2277), done.
Checking connectivity... done
Note: checking out 'bfb56a8918cb5690c5651aed62d5176544bd930a'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b new_branch_name

Checking out files: 100% (6821/6821), done.

I will still upgrade to the newest stable version.

I will update to the following (from http://lammps.sandia.gov/bug.html):
'10 Aug 2015 = stable version, SVN rev = 13864, Git tag = r13864 '

Maybe we should use the mirror (https://github.com/lammps/lammps) instead as I also just noticed how slow git.lammps.org is.

closed by #109


OpenMX: k_mesh_grid

https://nomad-lab.eu/prod/v1/staging/gui/analyze/metainfo/nomad.datamodel.datamodel.EntryArchive/run/method/k_mesh/grid
https://github.com/ondracka/electronic-parsers/blob/1b143b3624c295ab7c8c4fdf51fb8ef28fb85830/tests/data/openmx/AlN_ionic_optimization/AlN.out#L49

This actually seems to work now.


startDate not working correctly ?

Iv'e set up a small fiddle to show the bug.
set the startDate to 20.08.2012, but the limit starts from 20.09.2012
Am i doing something wrong?
http://jsfiddle.net/sourcecode_il/yEGZB/

Doesn't work for me. startDate doesn't do anything when passing it in through the initialization dictionary.

Still not working

If it is not working please create a new issue referencing this issue and provide a jsfiddle with the latest release so we have a testcase. Thanks!

Sorry, looks like the site was using an out of date version of bootstrap.datepicker.js, it is working great!

No problem! Thanks for the feedback!


Internal Links

Not sure if this an issue. But how would internal linking between slides function?
An initial try via [Go to](#3) or [Go to](/#3) would open the slide in a new tab, not navigate to the selected slide. Is there a different way to navigate between slides?

Current there's a default prop for target: '_blank' on the link component, which should probably be removed. I briefly looked into this but didn't get it in https://github.com/remarkjs/remark-external-links

Where would the change have to take place? Where does remark come into play?

Currently the webpack config is set in the CLI here and remark plugins are passed to the custom loader as options: https://github.com/jxnblk/mdx-deck/blob/master/cli.js#L48-L50

It would be great to enable user land to define if the link should be external (or maybe external is default) can be set to internal f.e. If we know we're dealing with an internal link than we could also navigate to the selected slide.

Yeah, we could detect via the href prop whether it starts with a hash or not – the link can be completely customized with the components object, but I think it's good to have smart defaults built in. But, the simplest solution is probably to remove the default prop for now


Use mrustc to bootstrap Rust compiler

mrustc is a minimal Rust compiler that can compile rustc and cargo 1.19 and 1.29, which can replace our binary bootstrap compiler and give a trusted root to the bootstrap chain.

https://github.com/guix-mirror/guix/blob/7dadce383fc5f00b795c8fa1209033469731c14b/gnu/packages/rust.scm has a many-phase approach to building a recent rustc starting with mrustc

still active (see #85542)


LOAD dialog, change text to “Please select a content type XML schema”

Added this change to issue #79.


Cytoxicity or Cytotoxicity?

This labeled is a typo, correct?
<owl:Class rdf:about="#NPO_1707">
    <rdfs:label rdf:datatype="&xsd;string"
        >cytoxicity assay</rdfs:label>

It's one I often make, and this is just to confirm "cytoxicity" is not really a valid wording.

Cytoxicity is used too, but less common [0].
0.http://www.thefreedictionary.com/Cytoxicity