Can not import ngtpy with torch (segmentation fault)

I did not know this exactly pytorch issue or ngt issue. I also submit this issue in pytorch
https://github.com/pytorch/pytorch/issues/26405#issue-495238678
🐛 Bug

I can import pytorch properly

After import other library (ngtpy, flair, or other), it raised segfault core dump

If I swap the import (ngtpy first), it will raised free(): invalid pointer.

I did not know what happened. I use ngt (https://github.com/yahoojapan/NGT)
To Reproduce
Steps to reproduce the behavior:

Import torch, then other library -> Segfault
Import other library, then torch -> Free(): invalid pointer

Expected behavior
can import properly
Environment
PyTorch version: 1.2.0
Is debug build: No
CUDA used to build PyTorch: 10.0.130

OS: Ubuntu 18.04.3 LTS
GCC version: (Ubuntu 7.4.0-1ubuntu1~18.04.1) 7.4.0
CMake version: version 3.10.2

Python version: 3.6
Is CUDA available: Yes
CUDA runtime version: 10.0.130
GPU models and configuration:
GPU 0: Tesla V100-DGXS-32GB
GPU 1: Tesla V100-DGXS-32GB
GPU 2: Tesla V100-DGXS-32GB
GPU 3: Tesla V100-DGXS-32GB

Nvidia driver version: 410.104
cuDNN version: /usr/lib/x86_64-linux-gnu/libcudnn.so.7.5.0

Versions of relevant libraries:
[pip3] numpy==1.17.2
[pip3] pytorch-transformers==1.2.0
[pip3] torch==1.2.0
[conda] blas                      1.0                         mkl
[conda] mkl                       2019.3                      199
[conda] mkl-service               1.1.2            py37he904b0f_5
[conda] mkl_fft                   1.0.10           py37ha843d7b_0
[conda] mkl_random                1.0.2            py37hd81dba3_0

Additional context
I inspect it using gdb, and the backtrace output:
#0  0x00007fff1eaf3e3e in pybind11::detail::make_new_python_type (rec=...) at /opt/python/cp36-cp36m/include/python3.6m/pybind11/detail/class.h:565
#1  0x00007fff1eaf938e in pybind11::detail::generic_type::initialize (this=this@entry=0x7fffffffc900, rec=...) at /opt/python/cp36-cp36m/include/python3.6m/pybind11/pybind11.h:902
#2  0x00007fff1eac1a97 in pybind11::class_<Index>::class_<> (name=0x7fff1eb61378 "Index", scope=..., this=0x7fffffffc900)
    at /opt/python/cp36-cp36m/include/python3.6m/pybind11/pybind11.h:1092
#3  pybind11_init_ngtpy (m=...) at src/ngtpy.cpp:436
#4  0x00007fff1eac34d0 in PyInit_ngtpy () at src/ngtpy.cpp:421
#5  0x00000000005e4268 in _PyImport_LoadDynamicModuleWithSpec ()
#6  0x00000000005e4522 in ?? ()
#7  0x000000000056246e in PyCFunction_Call ()
#8  0x00000000004fed26 in _PyEval_EvalFrameDefault ()
#9  0x00000000004f6128 in ?? ()
#10 0x00000000004f7d60 in ?? ()
#11 0x00000000004f876d in ?? ()
#12 0x00000000004f98c7 in _PyEval_EvalFrameDefault ()
#13 0x00000000004f7a28 in ?? ()
#14 0x00000000004f876d in ?? ()
#15 0x00000000004f98c7 in _PyEval_EvalFrameDefault ()
#16 0x00000000004f7a28 in ?? ()
#17 0x00000000004f876d in ?? ()
#18 0x00000000004f98c7 in _PyEval_EvalFrameDefault ()
#19 0x00000000004f7a28 in ?? ()
#20 0x00000000004f876d in ?? ()
#21 0x00000000004f98c7 in _PyEval_EvalFrameDefault ()
#22 0x00000000004f7a28 in ?? ()
#23 0x00000000004f876d in ?? ()
#24 0x00000000004f98c7 in _PyEval_EvalFrameDefault ()
#25 0x00000000004f4065 in _PyFunction_FastCallDict ()
#26 0x000000000057c8f1 in _PyObject_FastCallDict ()
#27 0x000000000057cc5e in _PyObject_CallMethodIdObjArgs ()
#28 0x00000000004cf5dd in PyImport_ImportModuleLevelObject ()
#29 0x00000000004fb864 in _PyEval_EvalFrameDefault ()
#30 0x00000000004f6128 in ?? ()
#31 0x00000000004f9023 in PyEval_EvalCode ()
#32 0x00000000006415b2 in ?? ()
#33 0x000000000064166a in PyRun_FileExFlags ()
#34 0x0000000000643730 in PyRun_SimpleFileExFlags ()
#35 0x000000000062b26e in Py_Main ()
#36 0x00000000004b4cb0 in main ()

Fix regarding this issue: #34

Thank you for your detailed information.
As far as I investigated, this segmention fault is caused by pybind11 v2.3.0. To avoid this fault, please build ngtpy from source with pybind11 v2.4.0 or older than equal v2.2.4. Ngtpys of pypi from v1.7.6 to v1.7.9 have the same problem. Another option is using ngt python ctypes bindings.

I have released a ngt python package to resolve this issue.


⚠ Bofrost.be has degraded performance

In 566a9d9, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1321 ms

Resolved: Bofrost.be performance has improved in cf45b0d after 15 minutes.


Fix apparent ggZH bug

@gpetruc can you please check that this was indeed an unintended typo when getting the numbers from this table, rather than a feature to compensate something else? @nucleosynthesis and I think this may be a typo.

Yes, it's a typo.


Report formatting messed up when long task names

If a task name is longer than 30 characters then the report is messed up and hard to read. I am dynamically creating task names which is leading to longer than usual names.
Task                          Duration
--------------------------------------------------
Build-Order                   00:00:00.0081275
Nuget-Restore                 00:00:04.1434154
Build-ReleaseDebug-Common-Library00:00:06.2929425
Build-ReleaseDebug-Services-All00:00:20.0597646
Build-ReleaseDebug-Module-Analysis00:00:03.9095751
Build-ReleaseDebug-Module-Documents00:00:04.7935222
Build-ReleaseDebug-Module-Messaging00:00:02.2321549
Build-ReleaseDebug-Site-App   00:00:23.4270291
Build-ReleaseDebug-Site-Assets00:01:11.4500985
Build-ReleaseDebug-Site-Authentication00:00:04.0556270
Build-ReleaseDebug-Site-Instance00:00:02.4231884
Build-ReleaseDebug-Site-Manage00:00:19.0323366
Build-ReleaseDebug-Site-My    00:00:25.8938716
Build-ReleaseDebug-Site-Pay   00:00:02.9447329
Build-ReleaseDebug-Site-Widget00:00:01.7430201
Default                       00:00:00.0117797
--------------------------------------------------
Total:                        00:03:12.4211861

The #268 PR leaves most reports the same. If any of the task names are greater than 30 characters long it widens the report e.g. the above now looks like:
Task                                   Duration
-----------------------------------------------------------
Build-Order                            00:00:00.0107507
Nuget-Restore                          00:00:03.3841406
Build-ReleaseDebug-Common-Library      00:00:03.2489604
Build-ReleaseDebug-Services-All        00:00:09.3761721
Build-ReleaseDebug-Module-Analysis     00:00:01.9135796
Build-ReleaseDebug-Module-Documents    00:00:02.7120511
Build-ReleaseDebug-Module-Messaging    00:00:01.6872557
Build-ReleaseDebug-Site-App            00:00:13.5204454
Build-ReleaseDebug-Site-Assets         00:00:44.0441283
Build-ReleaseDebug-Site-Authentication 00:00:02.1955409
Build-ReleaseDebug-Site-Instance       00:00:01.5250945
Build-ReleaseDebug-Site-Manage         00:00:10.7844529
Build-ReleaseDebug-Site-My             00:00:15.0811007
Build-ReleaseDebug-Site-Pay            00:00:02.1624435
Build-ReleaseDebug-Site-Widget         00:00:01.1402457
Default                                00:00:00.0016058
-----------------------------------------------------------
Total:                                 00:01:52.7879679

#268 merged fixes  #267


[Feature Request] same random seed for every env in AsyncEval

🚀 Feature
When training with ARS in combination with AsyncEval, multiple environments are run at the same time. When seeding these environments, all environments get a different seed. There should be an option to seed all the environments with the same seed at the start of each time ARS.evaluate_candidates() is run.
https://github.com/Stable-Baselines-Team/stable-baselines3-contrib/blob/5c81398ef854dde4eeaed51e0715c5de18a9d344/sb3_contrib/ars/ars.py#L165
https://github.com/Stable-Baselines-Team/stable-baselines3-contrib/blob/5c81398ef854dde4eeaed51e0715c5de18a9d344/sb3_contrib/common/vec_env/async_eval.py#L154
Motivation
Some environments have random values generated in the reset function, for instance external factors that are random. When running evaluate_candidates, these random values can have an effect on the returned rewards, which makes that some good sets of params get bad rewards and bad params get good rewards. This makes training slower. In order to mitigate this, while still generating different random values for external values, all environments in AsyncEval should be seeded with the same random number at the start of evaluate_candidates, or this should at least be an option.
Pitch
Add the following lines to the start of ARS.evaluate_candidates:

        if async_eval is not None:
            # Multiprocess asynchronous version
            async_eval.send_jobs(candidate_weights, self.pop_size)
            results = async_eval.get_results()
            async_eval.seed(self.seed)

Add the following lines to AsyncEval.seed
  if seed is None or seed == 0:
            for idx, remote in enumerate(self.remotes):
                remote.send(("seed", np.random.randint(2**32 - 1, dtype="int64").item() )) # seed all envs with the same random seed
        else:
            for idx, remote in enumerate(self.remotes):
                remote.send(("seed", seed + idx))
        return None

And change the worker so that it doesnt return values after seed:
elif cmd == "seed":
                # Note: the seed will only be effective at the next reset
                vec_env.seed(seed=data)

Alternatives
None.
Additional context
At least in my specific environment this method leads to great improvements in training.
Checklist

[X] I have checked that there is no similar issue in the repo
[X] If I'm requesting a new feature, I have proposed alternatives

Hello,
I think you are missing an important alternative, which is also recommended: evaluating each candidate for multiple episodes to remove noise due to env stochasticity.
Also, even if you seed the each env the same the first time, they will end up in different states after each trial because of the different candidates (you also don't want to optimize for a specific seed of your env).
See issues in RL Zoo:

https://github.com/DLR-RM/rl-baselines3-zoo/issues/204 (and linked issues)
https://github.com/DLR-RM/rl-baselines3-zoo/issues/314


what is tensorflow's pretrained checkpoint output node?

i want to use pre-trained model in deploy, so, i want to transfer checkpoint to pb file, but when i use emb/biases node, there only freeze this node to pb, so, which node is the output for object tracking inference

I'm not sure I can completely follow your question. We don't to any object tracking. We also don't provide code for freezing the model so you will have to write it yourself. If that code doesn't work it will be hard to offer support here. I know from experience that you can get it to work though. You will have to feed the input and fetch the output from the endpoints['emb'] tensor.

I added a freeze.py file that freezes the model, here: https://github.com/arcelien/triplet-reid. It's probably easiest to add a name to endpoints['emb'] to reference it once you freeze it.

I have figured out it. Thanks a lot. @arcelien

FYI: I just merged @arcelien's freeze code.


pageextension add new fields does not work

Hi..
I just tired with October build and found the issue when i add new field in pageextension does not work.
Below is the my code.
pageextension 70092504 CustomerCardExtension extends "Customer Card"
{
layout
{
addafter(Address)
{
field(Priority;Priority)
{
}
}
}

}
I tried all functions addafter,addbefore,addlast and addfirst but all not working. When i tried to moveafter,movebefore,movefirst and movelast with exiting fields it's working fine.
Please let me know if i am missing anything here.

Did you set the ApplicationArea for the fields?

Hi Thomas,
I set addafter(Address). is it ApplicatoinArea ?
If not then can you please guide me.
Thanks

@priteshsomani  maybe the same thing has happened to you as to me.
twityde is saying your extension code should look like:
pageextension 70092504 CustomerCardExtension extends "Customer Card"
{
layout
{
addafter(Address)
{
field(Priority;Priority)
{

          ApplicationArea=Basic' //for example
}
}

}
}

Absent the ApplicationArea property, every field declared in AL will not be visible in the UI.
I was porting over something as a test, so I didn't want to go update all the fields to try this out.
So, if you go to Application Areas, and uncheck all the Application Areas. This page is set to be non-editable. I tried making it editable and that did work correctly.

Since September Update the dev environments mimic more closely the Dynamics 365 environment which means Application Area needs to be set for all controls.
We are looking into extension-wide setting for Application Area so you won't have to apply it to each individual control in the future.


WIP Node local DNS cache

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #389
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Node local DNS cache

this one is replaced by #704


format: code in readme

summary
The format of the code in the readme is not standardized, so I formatted the code. 😃

Hey @rfyiamcool,
Please do not open such PRs. You should first have to create an issue if you find a bug or want to offer an improvement. Olric is not your pet project. Random PRs without a description are not acceptable.

Hey @rfyiamcool,
Please do not open such PRs. You should first have to create an issue if you find a bug or want to offer an improvement. Olric is not your pet project. Random PRs without a description are not acceptable.

sorry.


Simulation runtime error feedback

Is your feature request related to a problem? Please describe.

Real robots sometimes return an error or notify the user in another way if a runtime error occurs.
The simulation currently has undefined behaviour it that case

Describe the solution you'd like

Some sort of error message, or notification that the program crashed
Preferably with the cause of the error: which block caused it, where to start looking

there is a function dbcException in util. This should be used in these cases. But the exception thrown is never caught and thus forgotten. It appears in the console log only. There should be an wrapper in the simulation engine, that catches and display an appropriare message


serve.sh script can not be located (again)

I am having the same issue as #1 . There have been commits that fix the issue, but they have subsequently been reverted. I am on OSX 10.10 and having the fixes from #1 in made it work for me. Without those fixes, I get the same error.

I'm looking into this. My own copy started breaking badly when I applied the fix, and some others reported the same. Thus a revert was made.
I am considering abandoning this fork, because the original homestead has now implemented everything I improved upon it - in response to this, they even made it possible to boot up again without having PHP installed locally, so this really has very little reason to exist any more.

I still prefer homestead improved for the simple need  to setup ssh keys. Not a big deal, but when all my other vagrant boxes I just vagrant ssh and I am in. I prefer consistency :)
Regardless, thanks for the effort.


6.0.0-canary.3 Styled JSX presets not applying

For whatever reason, the latest canary with babel 7 isn't working well with styled jsx plugins via custom babelrc.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Expected to build the page and display in dev environment, instead spits error each build.

Current Behavior

Instead of building page, the following error is displayed.
Module build failed: Error: Nesting detected at 11:9. Unfortunately nesting is not supported by styled-jsx.

Steps to Reproduce (for bugs)

Download and install with-styled-jsx-plugins example from next repo.
yarn add<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>yarn dev
goto localhost:3000

I can verify that the babel preset next/babel isn't receiving any custom options, so when the preset looks for the options, they aren't found.
https://github.com/zeit/next.js/blob/canary/server/build/babel/preset.js#L55
Context

Impossible to build pages currently.

Your Environment

Tech
Version

next
6.0.0-canary.3

node
9.3.0

OS
High Sierra

browser
Chrome

etc

Interesting, good catch! Feel free to check it out 👍 (I'll do it eventually)

Opened PR #4162 to address this issue.

I got the same "Unfortunately nesting is not supported by styled-jsx." error when I upgraded to Next 6 canary. I figured out that I had to modify my .babelrc file.
This is what worked before:
{
  "presets": [
    [
      "next/babel",
      {
        "styled-jsx": {
          "plugins": ["styled-jsx-plugin-sass"]
        }
      }
    ]
  ]
}

and this is what I had to change it to for Next 6:
{
  "presets": ["next/babel"],
  "plugins": [["styled-jsx/babel", { "plugins": ["styled-jsx-plugin-sass"] }]]
}


fix: changed url of the installation script fixes #51

@CircleCI-Public/orb-publishers could you guys review this PR. As long as the issue affects a lot of users

@Jaryt @EricRibeiro @KyleTryon looks like mentioning by the team name doesn't work

Implemented in #54


Example to connect OracleDataSrc(ADW) from OCI cloudshell with 'oracle' format

Example to connect OracleDataSrc(ADW) from OCI cloudshell with 'oracle' format

@Tejashwa Please fix typos in the commit message

Add a readme or update existing one with command to run the job

approved.


Add capability to set loglevel to trace during runtime

https://github.com/networkservicemesh/deployments-k8s/issues/12296

You mean something like adding a function that encapsulates all this signal thing to the sdk, and just calling a single function from the cmds? It sounds awesome, I feared that I would have to copy paste the current pr to every cmd, but this makes it a lot easier, I'll look around.

Put the implementation here, not sure whether the log directory, or the logrus subdirectory would be preferred, but this way, I could just use function from the log module directly which was convenient.
https://github.com/networkservicemesh/sdk/pull/1672


Bundle-Adjusting Neural Radiance Fields

Is the weighting parameter α random or does it vary according to a regular pattern?

Hi @Lijingbohere, it is (heuristically) controlled as a function of the optimization progress. Please see the experiments section in the paper for details.


Uncalled-function check misses usage of complex property access

Code
From chrome-app on Definitely Typed:
if (chrome.platformKeys.subtleCrypto &&
    chrome.platformKeys.subtleCrypto() &&
    chrome.platformKeys.subtleCrypto().sign &&
    chrome.platformKeys.subtleCrypto().exportKey) {
    chrome.platformKeys.subtleCrypto().sign
    chrome.platformKeys.subtleCrypto().exportKey
    console.log('Subtle crypto working (Y)')
}

Expected behavior:
No error
Actual behavior:
Error: "Did you mean to call 'sign'?" (and exportKey)

@a-tarasyuk you might be interested in this too.

This works as expected with TypeScript 4.2.0-dev.20201119. It provides an error with strictNullChecks enabled, and provides no error with strictNullChecks disabled.

There should be no error under any setting since chrome.platformKeys.subtleCrypto().sign is used in the body of the if.


Try assembling 3x times with and w/o native_assets.

Debugging https://github.com/flutter/flutter/issues/158560.

Going to try this in a better way in https://github.com/flutter/flutter/pull/158895.


Logic Apps doesn't accept https endpoint for Service Bus connection

When using Service Bus Message trigger in Logic Apps, the Service Bus endpoint URL displayed on Properties page cannot be used used.
The Properties page displays Service Bus endpoint as a Https endpoint. Logic Apps requires sb:// endpoint and if provided with Https:// endpoint, it throws following error "The provided Uri scheme 'https' is invalid; expected 'sb'.".
Hence the steps mentioned in https://learn.microsoft.com/en-us/azure/connectors/connectors-create-api-servicebus?tabs=consumption#get-endpoint-url will not work.
The Properties page doesn't display the sb:// endpoint at all. Instead this has to be extracted from the Shared Access Policies >> RootManageSharedAccessKey >> Primary or Secondary connection string.

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

ID: c2aeacc5-5b74-c4be-335e-c7e768854e4a
Version Independent ID: d8693b52-fdd6-cc68-bc12-26807454a80b
Content: Connect to Azure Service Bus from workflows - Azure Logic Apps
Content Source: articles/connectors/connectors-create-api-servicebus.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

@MithunChopda
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Hi @MithunChopda,
Thank you for your feedback! Actually, steps exist for both the Service Bus namespace connection string (sb://) and the Service Bus namespace endpoint URL (https://). The connection string is required to make the connection, while the endpoint URL is used for specific authentication types when you use the Service Bus managed connector.
I'll clarify these differences in the docs, but both sets of steps require documentation. Thanks!

@MithunChopda: A doc update will go live on 12/13/2023 after 11 AM Pacific Time.


chore(master): release 2.0.0

:robot: I have created a release beep boop
2.0.0 (2024-09-26)
⚠ BREAKING CHANGES

remove old and add new versions

Features

improve distribution loading and handling (05b08d2)
move to poetry (f522b0b)
remove old and add new versions (f220110)

Bug Fixes

add timeouts to requests.get (5fe7d41)
handle version at package level (8d3c492)
move checksum verification into a separate function (5eed052), closes #1

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.0.0
:sunflower:


[Github CI] Reenable MSVC compiler testing

@Yaraslaut the current way of finding Catch2 on windows produced a linker error. This is why I disabled windows CI back then. It should always favor system packages if present anyways, so I've tweaked that bit a bit.
But now I realized, that the tests where breaking on MSVC. I hope to find some time to check on why soon, too.

This is good that we test now on all platforms, i will try to fix the issue with the enum name

Oh thank you. I'm apologizing ahead of time, because I fear that I broke it :)


Explicitly set namepsace for ipamutils.ElectInterface

During the run of TestCreateMultipleNetworks() (which runs in  a netns) occasionally ipamutils.ElectInterfaceAddresses() returns a network which is already being used, causing the test to fail. While debugging the test failure it turned out  the root cause was netutils.CheckRouteOverlaps() returning host routes during the failure. This looks the same issue that was fixed by #476.
Now that the address pool election is moved away from bridge code into ipamutils via #588 , its execution is no longer tied to the netns by the fix introduced in #476.
In fact the random TestCreateMultipleNetworks() failures in circle ci started after #588.
Closes #652
Signed-off-by: Alessandro Boch<EMAIL_ADDRESS>

LGTM.
ping @mrjana

@aboch Good find. LGTM


ACM-12098: Implement CSV export for host inventory

Regarding: ACM-12098
Export table contents to CSV
Holding PR so #3681 can merge ahead of it.

Example output:
hostenvironments-1721437848388 (1).csv

/hold

/unhold


TextureProgressBar.min_range Shows Incorrect Range of Texture

Godot version
v3.4.4.stable.official [419e713a2]
System information
Gentoo Linux, GLES2
Issue description
When min_value of a TextureProgeressBar is set to a positive value, the min_value seems to be subtracted from the value parameter, instead of clamping it.
apparent_value = value - min_value
This happens in the editor, and in a running project.

Steps to reproduce

Create a TextureProgressBar.
Set Value to 50.
Set Min Value to 40-50.
The texture should be clamped around half-visible, but only a sliver is visible.

Minimal reproduction project
progress-bug.zip

I think I've misunderstood the description. Disregard.


[feat] Add a new option to skip enabling on CI

As title

Hi! I think this is going to be implemented once #96 make it to npm

#96 is a bit different for skipping on committing, while this issue aims skipping installation at the beginning.
I'd like to work on a PR soon.

@toplenboren I'd like to reuse SKIP_SIMPLE_GIT_HOOKS environment variable, how do you think?

@JounQin Many CI environments normally set the CI variable by default (https://docs.travis-ci.com/user/environment-variables/#default-environment-variables, https://docs.travis-ci.com/user/environment-variables/#default-environment-variables, https://docs.github.com/en/actions/learn-github-actions/variables), so it would be best to also support this by default.

@peschee I think that would be a BREAKING CHANGE?

Yes, probably. But are there instances where you'd want the hooks to be installed in a CI environment?

Yes, I understand. Breaking changes can happen though, so I'd be inclined to still vote for handling CI env vars, since this is what many other tools do.

I'm fine with BREAKING CHANGE or major version personally, let's wait for @toplenboren to confirm.

Why do we need any git hooks to be set in CI ?
Maybe we can just add the guard to postinstall script, so that it does not install git hooks if it is executed inside CI environment

Husky uses a custom env var for this: https://typicode.github.io/husky/how-to.html#ci-server-and-docker
Maybe it is easier to off-load the responsibility of skipping to the user all together. Using something like is-ci is pretty straightforward…

Why do we need any git hooks to be set in CI?

Checking commit message on CI for example.

By the way, what is your usecase @JounQin?

@toplenboren
In my company, we have components which checks previous commit message on CI.
While in some other components, we don't want to enable git hooks because we could commit automatically with other tools.

@toplenboren
In my company, we have components which checks previous commit message on CI.
While in some other components, we don't want to enable git hooks because we could commit automatically with other tools.

Thank you for clarification!
Why simply using SKIP_SIMPLE_GIT_HOOKS would not work? Hooks will be installed, but not executed

@toplenboren
That's the point, we don't want the hooks to be installed and logs echo "[INFO] SKIP_SIMPLE_GIT_HOOKS is set to 1, skipping hook." on every commit.

Let's add SKIP_INSTALL_SIMPLE_GIT_HOOKS env variable support, and add CI variable support

Case: SKIP_INSTALL_SIMPLE_GIT_HOOKS is not set

If not set and if in CI -> do NOT install hooks
If not set and not in CI -> install hooks

SKIP_INSTALL_SIMPLE_GIT_HOOKS=1

Install hooks

SKIP_INSTALL_SIMPLE_GIT_HOOKS=0

do NOT install hooks

This way we will:

have the env var namings consistent: SKIP_*
be able to address each case
not need to do a major release

What do you think, @JounQin. Will this cover your usecases?

Do you want to add a dependency like is-ci? I think it's out scope of this tool, the user should easily add an environment variable for their needs.

It seems we're talking about two different things…
The way I meant it is what is in the husky docs. They DO NOT include the is-ci package in husky, but rather rely on the user controlling whether they call husky install or not (using is-ci or any whatever other method).
My point was: this library is probably better of not including an additional dependency.

Oh, yes, I also think we should not check about CI on this lib's side. Do you have the consensus here?

I honestly really like this idea:

@JounQin Many CI environments normally set the CI variable by default (https://docs.travis-ci.com/user/environment-variables/#default-environment-variables, https://docs.travis-ci.com/user/environment-variables/#default-environment-variables, https://docs.github.com/en/actions/learn-github-actions/variables), so it would be best to also support this by default.

Let's just check CI variable

Case: SKIP_INSTALL_SIMPLE_GIT_HOOKS is not set

If not set and if in CI -> do NOT install hooks
If not set and not in CI -> install hooks

not need to do a major release

Isn't it a BREAKING CHANGE for not installing on CI by default?

Let's just check CI variable and add INSTALL_SIMPLE_GITH_HOOKS to explicitly configure this package for narrow usecases

OK, I'm personally fine with this strategy.

Case: SKIP_INSTALL_SIMPLE_GIT_HOOKS is not set

If not set and if in CI -> do NOT install hooks
If not set and not in CI -> install hooks

not need to do a major release

Isn't it a BREAKING CHANGE for not installing on CI by default?

Well.. technically yes.. :( I think we can do a major release, no problem
We can also split: first add the ability to configure INSTALL_SIMPLE_GIT_HOOKS and then add CI processing and update major

I'll work on it soon.


Data Loss on Rotating Screen

Describe the bug
In TicketsFragment, changes on Spinner is not retained when rotating
Screenshots
Before rotating:

After rotating:

Would you like to work on the issue?
No

I would like to work on this.

Please go ahead

Has this issue been resolved ?

@tavishjain I don't think so...

Ok then. May I work upon this ?

Of course

duplicate of #967

@pranavpandey1998official this one is on TicketsFragment, #967 is on AttendeeFragment, it is two different issue. Please take a look again.
@iamareebjamal can you open this issue again?

#1097 sorry i reference the wrong one

Hello I would like to work on this issue! It seems still not resolved and No one is working on it


buildtest-framework pypi package doesn't work

Describe the bug
pip3 install doesn't work with any releases past buildtest-framework-0.2.2.
Installed on OSX running Python 3.9.2
To Reproduce
$ pip3 install buildtest-framework
Collecting buildtest-framework
  Downloading buildtest-framework-0.6.3.tar.gz (45 kB)
     |████████████████████████████████| 45 kB 1.5 MB/s
    ERROR: Command errored out with exit status 1:
     command<EMAIL_ADDRESS>-c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/private/var/folders/q7/dxctxdc16yxc8ms51wyxwddrwxqnlb/T/pip-install-noxpqb1l/buildtest-framework_359cacfe4f044ac3ba7af070bfdb9298/setup.py'"'"'; __file__='"'"'/private/var/folders/q7/dxctxdc16yxc8ms51wyxwddrwxqnlb/T/pip-install-noxpqb1l/buildtest-framework_359cacfe4f044ac3ba7af070bfdb9298/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /private/var/folders/q7/dxctxdc16yxc8ms51wyxwddrwxqnlb/T/pip-pip-egg-info-aoqjzgsc
         cwd: /private/var/folders/q7/dxctxdc16yxc8ms51wyxwddrwxqnlb/T/pip-install-noxpqb1l/buildtest-framework_359cacfe4f044ac3ba7af070bfdb9298/
    Complete output (8 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/q7/dxctxdc16yxc8ms51wyxwddrwxqnlb/T/pip-install-noxpqb1l/buildtest-framework_359cacfe4f044ac3ba7af070bfdb9298/setup.py", line 29, in <module>
        from buildtest.tools.config import BUILDTEST_VERSION
      File "/private/var/folders/q7/dxctxdc16yxc8ms51wyxwddrwxqnlb/T/pip-install-noxpqb1l/buildtest-framework_359cacfe4f044ac3ba7af070bfdb9298/buildtest/tools/config.py", line 53, in <module>
        fd = open(BUILDTEST_CONFIG_FILE,'r')
    FileNotFoundError: [Errno 2] No such file or directory: '$HOME/.local/buildtest/config.yaml'
    [Errno 2] No such file or directory: '$HOME/.local/buildtest/config.yaml'
    ----------------------------------------

Expected behavior
There should be a guard in case the file is not present.

hi @frobnitzem that is very outdated installation guide. buildtest used to publish to pypi but now we just need to clone buildtest and source setup script. See https://buildtest.readthedocs.io/en/devel/installing_buildtest.html
This should fix the issue. Please let me know if you run into any issue for installation.

hi @frobnitzem were you able to get pass this issue. Let me know if you need any further help.

@frobnitzem I assume this is working for you now, if you need any help feel free to reopen this issue.


Remove the browserid login code which was never used

This was motivated by https://www.djangoproject.com/weblog/2015/aug/18/security-releases/ and an email from Michael Kelly:
FYI any sites using django-browserid are vulnerable, as we expose the logout view without a login_required check. Ya'll should upgrade if you're using it. :D

We considered issuing a django-browserid update to add that in, but if you're going to update to the latest django-browserid, you might as well upgrade Django as well.

If that's still a problem for your site, let me know why and we'll see what we can do.

Thanks!
- Mike Kelly

Most of the changes are following a dumb search for browserid and removing everything that seemed related.

It was briefly used, on dev, but definitely a welcome cleanup.

Very nice. I'd suggest leaving the source column in the database, and just deleting the index and model field for now, but I won't insist on it. r+


BO: Fix overlapping columns

Questions
Answers

Description?
'Change position' label in Modules/ps_mainmenu/Configure has no enough space and next columns overlay it

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes PrestaShop/PrestaShop#26054.

How to test?
check the issue description

Thanks @hibatallahAouadni & @florine2623


Fixed #31114 -- Fixed HttpRequest.build_absolute_uri() crash with reverse_lazy() locations.

https://code.djangoproject.com/ticket/31114

@jdufresne Thanks :+1: I moved tests to urlpatterns_reverse/tests.py.


Update config.py - Add EUS1/Milan pricing

added support For eu-south-1
Issue #, if available:
Script was failing for customer that has Tables on eu-south-1 Region
Description of changes:
Added pricing and description for MXP
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I fixed a mistake in the pricing and moved the data to a new line at the bottom. Can you please pull down the latest commit, mine, and verify the code runs correctly and provides a solution? Once you can confirm, I will merge the change.

I confirm that run successfully.thanks


MAP: Use highlight color for markers that correspond to the current search

(part of #260, putting here for narrower scope).
We want to highlight those markers on the map which correspond to the current search.
This will require factoring out the search process so that there's a single point of search aggregation (currently in SearchResults), which spawns two searches: one, paginated, for the search results, which returns everything required for displaying those, and one, not paginated, which returns everything the map needs.

I will do mockups of the following options:
OPTION 1:
Cases - red
Methods - yellow
Orgs - blue
Other - grey
OPTION 2:
results - black
other - grey
OPTION 3:
results - red
other - black

See #584 for colour mockups


SessionManager and Server Hot Swaps

Hi,
I was looking over the implementation of this session manager and I have a question. Here's how I understand it's working:
The sessions are flushed to DynamoDB in 2 cases: when tomcat is stopped and 30 seconds after the session turns idle (configurable).
So here's my situation: I have an instance in ELB with sticky sessions that I want to take out of traffic. I have connection draining to ensure active connections complete however due to the 30 second flush interval if I take the instance out of traffic (but not stop it) I will loose all sessions that have been active within the last 30 seconds. Is there a way around this? To me it looks like this is not possible with this session manager. Even if I set the flush to 1 seconds, I can still loose sessions.
Thank you

Not sure how I missed this and the answer probably won't help you now but Tomcat will attempt to unload sessions on a clean shutdown to the persistent store.


move DDP docs from speed guide

What does this PR do?
A simple PR that moves the docs on DDP optimizations to one place.
Fixes #11698

Closing this in favour of #12448.
@shabie Thank you for your contribution!


Implemented using local sbt in publishing

The goal is to use local sbt script everywhere. In quasar it was used very consistently. The only place was publishAndTag script that is created by sbt-slamdata. Plugin was updated so that it requires $SBT with sbt path to work.
I exported this variable from env.global section of travis file. As all scripts are run from the main directory, this should work.
Alternative way is to add export to the line in scripts/constants that I deleted. But I think that travis env looks more predictable.

@djspiewak latest quasar build failed because of lack of $SBT This PR should have been merged before as sbt-slamdata 0.7.0 includes changes from 0.6.2 which requires this variable set.
I rebased on top of master, it also retriggered build that failed for some odd reason before.

Change has already been implemented in master.


DataQuery applyRelation has_one join incorrect foreignKey

$has_one = array_flip($model->has_one());
I getting incorrect foreignKey while I have multiple has_one with same class.
e.g.
$has_one = array(
'Owner' => 'Member',
'Father' => 'Member',
'Son' => 'Member'
);
It always getting the "Son" as foreignKey.

possibly fixed by #3799 ? @kinglozzer ?

@hutlim have you defined the ´has_many´ side using the dot-notation like the example in the docs? Take a look at the last example under has_many http://doc.silverstripe.org/en/developer_guides/model/relations/

Yes. It's fixed on version 3.1.10.
I'm gonna upgrade framework to 3.1.10 version.
Thanks.

These error happen again. It's getting the last value as foreignKey.
Here is the example.
class OrderPackage extends DataObject {
private static $has_one = array(
'Member' => 'Member',
'PaidBy' => 'Member',
'SoldBy' => 'Member'
);
}
debug::dump(OrderPackage::get()->filter('Member.Username', 'Hut.L')->sql());exit;
Below is the query debug.
SELECT DISTINCT "OrderPackage"."ClassName", "OrderPackage"."Created", "OrderPackage"."LastEdited", "OrderPackage"."MemberID", "OrderPackage"."PaidByID",
"OrderPackage"."SoldByID", "OrderPackage"."ID", CASE WHEN "OrderPackage"."ClassName" IS NOT NULL
THEN "OrderPackage"."ClassName" ELSE 'OrderPackage' END AS "RecordClassName" FROM "OrderPackage"
LEFT JOIN "Member" ON "Member"."ID" = "OrderPackage"."SoldByID" WHERE ("Member"."Username" = 'Hut.L')
ORDER BY "OrderPackage"."Created" DESC

I've debug the code from the DataQuery.php
public function applyRelation($relation) {
	// NO-OP
	if(!$relation) return $this->dataClass;

	if(is_string($relation)) $relation = explode(".", $relation);

	$modelClass = $this->dataClass;

	foreach($relation as $rel) {
		$model = singleton($modelClass);
		if ($component = $model->has_one($rel)) {
			if(!$this->query->isJoinedTo($component)) {
				$has_one = array_flip($model->has_one());
				debug::dump($has_one);exit;

DataQuery.php:612 - DataQuery::applyRelation() -
Array
(
[Member] => SoldBy
)

Hmm, looks like this could be a regression from #3547 (though looking at DataObject::getReverseAssociation() which I removed, this bug may have always existed independent of that change)

OK, this may be related slightly to #3839
It seems that the queries aren't very good at dealing with multiple queries on the same table.

Whoa.. the problem is with array_flip! Should have a fix and test for review soon :)

#3956


Plot.image render SVG elements

I've found that rendering <svg> elements - specifically those made using plot - can be a powerful visualization technique (example). This is a first pass at supporting svg rendering through the Plot.image mark.
Unsure if this is a desired feature or approach, but figured I could start the conversation.

Thanks for your thoughts! I definitely imagine this as a data-driven mark, as in the notebook (sorry the test wasn't more representative of the use-case).
Great point that this may nudge users into a bit of an anti-pattern where they're rendering plots with different scales and showing them together.
In some cases, users may not be concerned with a shared scale across plots (for example, drawing country outlines):

I'll think a little more about how (if) a mark like Plot.svg could behave (or be described) in a more generalizable way. Will also look at the rendering.

I like the idea of subplots or “subfacets” that share scales but can be positioned arbitrarily in x and y.

Interesting - definitely a bit of a larger undertaking 😭 , will think about it!

A bit like https://observablehq.com/@observablehq/plot-of-plots?

Yes, but with shared scales. I think it would be a similar mechanism to how facets currently work, with the difference being that you can position the facet arbitrarily in the main plot’s x and y coordinate space, and then the subplots have their own x and y scales (shared across subplots).

There's also overlap with the desire for custom tips (#1767) - if subplots can be arbitrarily positioned in x/y space, then with a Pointer transform, they could allow for custom tip content.
Enabling the Tip mark to render svg elements (most importantly, Plots) would get most of the functionality here (except for shared scales).

See #1870


Open internal modules in preview tab

Debug a node script like this

eval('debugger;//123')

When breaking in the eval'd script, leave the tab open in the editor
Change the eval'd code in some way, e.g. change the comment
Debug it a second time
If the same number is assigned, e.g. VM65 on both runs, the editor will open the old version of the script which is already open. It doesn't send a scriptRequest to the debug adapter to get the new script text. This leads to confusing situations when debugging eval code.

Dup of an issue that we do not dispose scripts on end of debug session -> there is currently an open issue assigned to Joao but I can not find it.
Since we need a mechansim that debug can tell the content provider that a content is no longer valid.
If you can not find the original issue and you are passionate let me know and I can try harder to dig it up

Can they be opened as preview tabs?

@roblourens yes we could open all internal modules as preview tabs and I like that idea

@isidorn am I able to set breakpoints in preview tabs?

@weinand yes.


Support for apache Hudi

Hello,
Now dbt spark has support to delta lake, but would be good to support apache hudi too.
https://hudi.apache.org/
Apache Hudi is our default data format in all areas of our datalake.
Thank you so much!

Thanks for creating this support issue @rubenssoto!
I'm one of the Apache Hudi committers, I'm already working on creating a PR for this codebase or create a new dbt hudi package to support apache hudi format. I can volunteer to take up this task.
cc @vinothchandar

thank you so much @vingov we use apache hudi in all our datalake, so support apache hudi is mandatory for us
thank you so much

Very cool to see this issue, and the PR to go along with it :)
I don't have experience using Hudi, so I'd like to find someone who does. In the meantime, I'll take a look and help however I can

Maybe a file_format is missing in the documentation? Or somewhere that explains how to use it. I thought it could be as easy as specifying file_format = 'hudi', but I can't find a mention to Hudi in the documentation, so not sure.
And in case is because I didn't see it, I wanted to avoid opening an issue.


[Improvement] Refactor state computation/orchestration layer

state computation/orchestration layer (sits in the middle, between GS retrieval module and driver layer), the concurrent dispatching of state orchestration greatly impact the throughput and latency of vpc updates through ovs

Evaluating cpp high performance logic agnostic framework to leverage for goal-state to flow manipulation


Make sure uploaded layer is added to FeatureStore

SDK-403

Coverage increased (+0.06%) to 71.078% when pulling dd872129d3e4b990832ebea268044c3c433417b2 on selection-fixes into bc52d7eb7243c3c8986f23da80133b22ef21cb02 on master.

Coverage increased (+0.05%) to 71.067% when pulling c39885614d3f4b1b7f052f64caa6f2ed587e5573 on selection-fixes into bc52d7eb7243c3c8986f23da80133b22ef21cb02 on master.


Add spring-configuration-metadata.json

Hi,
I'm doing my first Spring Boot 2 app, and everything I'd expect to work with auto-completion for properties is working ... apart from the security bits.
I get prompted with all the deprecated properties, but not the actual ones.
Looking at the jars, I can see additional-spring-configuration-metadata.json, but not spring-configuration-metadata.json, as is present in other Spring Boot components

@nealeu Thanks for the report! This is now fixed in master

Thanks for the fast turnaround :+1:


Link to the schema

Some users may want a link to the schema so they can just load it into their editors. It should be explained within the README file.

Show the example of autocomplete features in VSCode

Example Image


multibind with ports tracking

Problem
#611
With SO_REUSEPORT and multi sock biding, the validator socks may bind to conflicting ports.
Summary of Changes

keep track of ports and reused_port when we bind sock, and check for already binded sock before binding.

The actual change is in the last commit of this PR.
Fixes #

ci :pensive:
lemme guess... now we're running out of ports

ci 😔
lemme guess... now we're running out of ports

I think the issue with the test is that we are creating multiple nodes on the same tpu_port for reuse.
i.e. n1 and n2 will collide on the reuse tpu_port.
Previously, we were not reusing port. when n2 comes up, it will see that the tpu_port is already used by node1. Then node2 find another port. Everything is good.
But with multibinding, n2 comes up and sees that tpu_port is used by node_1. But because reuse flag is true, n2 think he can reuse that port. Then the socket is messed up and the test hangs.
The fix is adding a flag to indicate that we are the first one who bind to a reuse port. And handle it differently than the following reuse port binding.
I pushed a commit for this change. my local run passed.
Let's see how it does on the CI.

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

Project coverage is 81.8%. Comparing base (562254e) to head (ec6999d).
Report is 12 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master     #626     +/-   ##
=========================================
- Coverage    81.8%    81.8%   -0.1%
=========================================
  Files         851      851
  Lines      230155   230354    +199
=========================================
+ Hits       188443   188605    +162
- Misses      41712    41749     +37


Extract genesis config generation into reusable library function

What was wrong?
Context
To facilitate genesis of a testnet, we define a self-contained configuration file that any software in the network can read and find the relevant information to begin their job at genesis.
There is a trinity Application defined under trinity.components.eth2.network_generator that generates this configuration and writes the data to disk. It would be nice to pull out the configuration generation code into a library function so that it can be used in other places.
The main use case is if the configuration on disk is missing for some reason:
try:
    config = _read_genesis_config()
except FileNotFoundError:
    config = _generate_config_anyway()
finally:
    ...  # do stuff with config

How was it fixed?
Extracted genesis config logic into _generate_genesis_config()
To-Do

[x] Clean up commit history

[x] Add entry to the release notes

Cute Animal Picture

@ralexstokes let me know if there's anything else that needs changing. I also don't have merge access in trinity so if you want to merge this - or I can ping whoever's an admin (piper?) to get merge access.

@njgheorghita sorry for the delay on this and thanks a bunch!!


Powerline colors with neovim and vim-colors-solarized

Hello,
I'm using urxvt and I get different colors in my neovim powerline bar (same thing with vim) when switching from solarized-dark to solarized-light compared to what I can have by using the toogle feature in vim-colors-solarized:

Left one is neovim with background=dark and dynamic-colors switch solarized-light, the right one is neovim with background=light and dynamic-colors switch solarized-dark. It seems to me that the right one is the correct colorscheme.
Xresources: https://github.com/Anthony25/dotfiles/blob/desktop/xorg/Xresources
nvim config: https://github.com/Anthony25/dotfiles/blob/desktop/nvim/init.vim
I tried to remove all colors in my Xresources (the """ Solarized """ bloc) without seeing any change in nvim.
Thanks!
Edit: FYI, no issue with solarized-dark

Hum, in fact that is maybe an issue with how vim-airline applies its theme. As background=dark, vim-airline tries to apply a solarized-dark theme. I'm closing this issue as it has nothing to do with this project.
Thanks for your work, btw!


.turbo-cookie file created in monorepo root

What version of Turborepo are you using?
1.5.5
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Mac
Describe the Bug
When upgrading to<EMAIL_ADDRESS>and running a command a .turbo-cookie file appears in the root of the monorepo. First reported in #2034 (not yet clear if it is related to the daemon)

https://github.com/vercel/turborepo/issues/2034#issuecomment-1260816431
https://github.com/vercel/turborepo/issues/2034#issuecomment-1263475701

Expected Behavior
N/A
To Reproduce
N/A

@klapec @fveauvy can you tell me what package manager you're using? I tried repro'ing in a simple case https://github.com/mehulkar/turbo-repro-cookie-file and I'm not able to reproduce the issue yet. (Feel free to jump into our Discord if you want to chat a little more synchronously! I'll hang out there for a little while)

I can't log into discord right now (it's 8:30pm in here, family stuff :P) but here are the details:
<EMAIL_ADDRESS><EMAIL_ADDRESS>

As I said in #2034 :
My package manager is<EMAIL_ADDRESS>and using<EMAIL_ADDRESS>What's weird is I don't have this issue when I'm using vscode integrated terminal to run turbo commands.

Currently looking into if this is related to turbo being invoked from a global install, rather than local to a repository

Verified that this is not related to global install, but instead to slow filewatching queues on macos. In the event where we timeout, turbo was failing to cleanup the file it was using to check for filewatching capabilities. This should be fixed in the linked PR and will be in an upcoming release.


Optimize Parameters call for performance

Currently, the code implemented in Parameters __call__ contains two for-loops. Time this and try to find a more optimal (and still human-readable) alrernative.

Useful reference for finding slow parts of the code: https://www.youtube.com/watch?v=m_a0fN48Alw


[slider/priceRange]: Allow changing the attribute filtered programmatically

Imagine I have a list of items to sell, and for each item I have their price stored in various currencies (euro, dollar, yen, etc). In my UI, I only want to display prices in the selected currency, but still allow users to swap displayed currency (through a dropdown menu) if they need to.
I also want to let them filter the results through a slider/priceRange widget.
A naive implementation would be to add three widgets, one for each currency, and add a bit of logic to hide/show them based on the currently selected currency. This will unfortunatly yield bad results: if I set a range in euros, then switch to dollars, then the range in dollar will not be correctly set. And worse, I now have no way to change or revert the range in euros.
I see various ways to fix this issue:

Exposing a .changeAttribute() method on the widget itself that would allow keeping the same widget, but changing its targeted attribute
Allowing to .removeWidget() the same way we .addWidget() on the instantsearch instance directly and letting the user destroying the previous dollar slider and adding a euro slider instead.

What do you think might be the best way? Or is there a more obvious solution I did not see?

:+1: for this feature. We'd like to use it ASAP. Thank you!

We should be able to solve this by providing:

a way to update the configuration of a widget (update the slider configuration and discard the previous configuration)
a way to force a render of the interface (so that transformData on the hits widget runs again and you are able to switch the #price attribute you are using based on what the user selected in a custom widget)

What do you think?

I would name it updateConfiguration (as to be clear that this is not the way to init a widget). What would be perfect would be to be able to update every configuration option, but I'm not sure how possible it will be (like changing the bound DOM container, the templates, etc).

To solve this, I propose to have:

search.removeWidget(widget) that will remove the widget and clean any needed DOM/events

Then you would just add a new slider with a different configuration.
This is a lot simpler and could be done for every widget.
Moved to #1065


Date conversion to format DD.MM.YYYY produces different result

Description of the Issue and Steps to Reproduce:
I'm using moment.js to handle my date time values in my web application. I'm using follow format: DD.MM.YYYY for example 29.09.2016. Now I have some input fields, where I can type in the day of birth of a person. When I type a short year between 62 and 99, it converts it like this for example: input = 29.09.82 it converts it after pressing tab to 29.09.2082 (puts 20 before the 82). When I write a short year between 00 and 62 in converts it like this example: input = 29.09.55 would be converted to 29.09.1955 (puts 19 before the 55). So the first range will be converted to a year in the 21th century and the second range would be converted to a year in the 20th century (this seems to be default, I don't know why).
moment('29/09/55', 'DD.MM.YY').format('DD.MM.YYYY') prints "29/09/2055"
moment('29/09/82', 'DD.MM.YY').format('DD.MM.YYYY') prints
"29.09.1982"
Environment:
Thu Sep 29 2016 17:31:48 GMT+0530 (India Standard Time)
VM401:2 9/29/2016, 5:31:48 PM
VM401:3 -330
VM401:4 Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2785.116 Safari/537.36

2-digit years are inherently ambiguous, so there has to be some rule for deciding which century the date goes in. Here's the one Moment uses by default:
parseTwoDigitYear = function (input) {
    return toInt(input) + (toInt(input) > 68 ? 1900 : 2000);
};

You can override that function like this:
moment.parseTwoDigitYear = function(input) {
  return toInt(input) + 1900;  //assume all dates are in the 20th century
}

...or whatever you want.


bug: global questions can be answered multiple times

Describe the bug
The same user can answer global questions multiple times.
To Reproduce
Steps to reproduce the behavior:

Get a global question
Answer the global question
Answer the global question again

Expected behavior
A user can only answer a global question once.
URLs
https://sparkler.cc/question/212ab7fdb597edac33cae4f6e67e2fd8ebe64453b8a278753c944daeebb19fda

Fixed, thank you!

@trisuaso works! I'd suggest additionally hiding the response form in the interface if you already answered.
Getting a "You are not allowed to do this!" might be confusing to users. So it's best to not show the form at all to prevent this from popping up, or make the error message a bit more descriptive.

Should be better now!