To solve the misisng column name uniqueness case we concatenate the c…

Addresses https://github.com/scikit-learn-contrib/categorical-encoding/issues/132
The fix implemented was to use a concatenation character, #, for the concatenation of the column name and -1 for only the missing column. That way -1 can show up in the values and duplicate columns will not appear because _ is used at the concatenation character for the column name and value.
For my test, I had to use set equality because python 2 and python 3 were giving different column orderings. I have not dug too deep into why yet, but I think it may be due to differences in how sets are handled. If you want me to dig into it as part of the PR, then I can do that.

Beware that we can still get a clash of names:
    def test_one_hot_duplicate_names(self):
        X = pd.DataFrame({'sea_rch': ['dummy', 'found'],
                          'sea': ['rch#-1', 'found']})
        encoder = encoders.OneHotEncoder(use_cat_names=True)

        result = encoder.fit_transform(X)
        columns = result.columns.tolist()

        self.assertEqual(len(set(columns)), len(columns))

A citation that came to my mind:

There are only two hard things in Computer Science: cache invalidation and naming things. -- Phil Karlton

What we are trying to deal with is an old problem. In Lisp they have gensym. Rust has raw strings with a variable count of #...
I do not think we can find a sentinel value that can't appear in the input data but can be used in the column names simply because both, input data and the output can be strings. Hence, the domain of the symbols in the input can be the same as the domain of the symbols in the output -> no sentinel value may exist (assuming independence of the symbols).
Requirements for the naming function:

It must be reasonably fast.
The behavior of the function must be described in the documentation. E.g.:

    use_cat_names: bool
        if True, category values will be included in the encoded column names. Since this can result into duplicate column names, duplicates are suffixed with '#' symbol until a unique name is generated. If False, category indices will be used instead of the category values.

But I do not care about which symbol(s) we use and where do we place them.
Preservation of the attribute order should be another issue. #132 is already nontrivial.

Dang it you're right. Ok, I'm going to focus on the suffix # approach

@janmotl
Went ahead and went with the suffix approach. Tell me what you think. Thanks buddy.

Ok, I have fixed the inverse transforming by creating a mapping object that gets used in get_dummies and reverse_dummies. That way we don't generate the column names at transform time, we generate them at fit time which I think makes sense.
I tried to find opportunities to use the vectorization transforms we've started using in target encoder and the woe encoder, but I had trouble seeing the opportunities. Can you spot where I can use them?
Tell me what you think.

I am not sure this is going to be faster, but for each nominal column:

Let's have a dictionary from category -> OHE column index. Pandas Series can be used for that as well.
Let's prealocate output matrix with zeros.
Then we can just loop over the nominal column once, read the category value, look up the index, write at the appropriate row and column index 1. This step can be vectorized.
Append the generated matrix (which is just for this column) to the result.

Alternative: Instead of having the dictionary, we can have an encoding matrix. And instead of writing ones into the prealocated table, we could write whole rows from the encoding matrix.
Either way, the advantage is that we go thru the nominal column only once and not as many times as there is unique values in the column.
But before doing something, it could be meaningful to benchmark other implementations like pandas.get_dummies and see how they do it.


Relax numpy and pint versions

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 remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
48

Relevant Lines:
48

💛  - Coveralls


biometric age key gen needs to be encrypted at rest and is not

age-plugin-se keygen -o abop.biometric.age --access-control any-biometry-or-passcode
public key: es1se.............,
AGE-PLUGIN-SE-1.....................................................
maybe there is an option I am missing. if you could advise. Looks unencrypted to me.
I get it to work besides that, although with a few work-arounds, but that is a no-no for me.
I'll wait till that is fixed to follow this then...

Hi @dmore
I'm assuming you're talking about the AGE-PLUGIN-SE-1 bit. This is actually an encrypted representation of the key, only usable with the particular Secure Enclave it was created with. From Apple's Documentation:

Keys that you store in the Secure Enclave expose a raw representation as well, but in this case the data isn’t the raw key. Instead, the Secure Enclave exports an encrypted block that only the same Secure Enclave can later use to restore the key.

That said, you should of course keep your private keys private (as you do with all other private keys)

Looks to me very much like a passphrase-less equivalent of an age key. When armored in age you get
cat key.age
-----BEGIN AGE ENCRYPTED FILE-----
Y....
-----END AGE ENCRYPTED FILE-----
to use it then I need to inject the passphrase at runtime. That is what I expect on here too and that is the only way I would use age.
Is it possible to get that armor description as well so there is no doubt on the implementation because I am doubting the implementation at the moment.

@dmore Yes, this works as usual. Just follow the age instructions, but replace age-keygen with age-plugin-se keygen:
$ age-plugin-se keygen | age -p > key.age
...

yeah that is better. is encrypted and can use armor like you indicate. will give it another go. public key is hidden this way but that is not a problem. can get to it when decrypting it.

@dmore I looked into printing the public key when using redirection, like age-keygen does, but it’s too much trouble. So you’ll indeed have to decrypt the key to get the public key, or do it in 2 steps without redirection.


NPM start command giving no output

Hello @matt1as ,
For Testing purpose, there is no output I am getting after running the NPM start command
as you can see in the image below. It is running the build, but no output or error is there

After NPM start, I thought to put another command for test--> npm test -- --host='' --protocol='' --username='' --password='' --package=''
but I don't know here also it is taking, wrongly. Couldn't understand how to solve this.


Return empty source object for all entity audit events

This was meant to be part of backend PR https://github.com/getodk/central-backend/pull/1072 but i forgot to push it!!
Frontend PR https://github.com/getodk/central-frontend/pull/932 expects a source for every entity audit event, even API events, where the source is currently just {}.

What has been done to verify that this works as intended?
Why is this the best possible solution? Were any other approaches considered?
How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?
Does this change require updates to the API documentation? If so, please update docs/api.md as part of this PR.
Before submitting this PR, please make sure you have:

[ ] run make test-full and confirmed all checks still pass OR confirm CircleCI build passes
[ ] verified that any code from external sources are properly credited in comments or that everything is internally sourced

It seems a little funny to have a source object that's completely empty, but I guess it'd also be funny not to have a source property at all. I bet it simplifies Frontend to be able to assume that there is always a source property.

We will have to put something in there soon to distinguish between "API" bulk upload and API single operation when the source is not a submission.


Enable py312 builds

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Is libhkl package limiting here?

I'll look into it.

https://github.com/conda-forge/hkl-feedstock/pull/16

If addition of Py3.12 means a new release of libhkl on PyPI, that may be delayed a bit.  I'm working on a branch right now and encountering some changes in the gobject-introspection tool chain that are challenging.  The problem is that a draft version of the libhkl.so library works successfully in Python on debian buster but not on RHEL9.  It comes down to the version of the glib subproject that is selected by the gobject-introspection tag.

2024-04-29T19:23:05.6186725Z Encountered problems while solving:
2024-04-29T19:23:05.6237740Z   - package hkl-<IP_ADDRESS>3-py310h7e962ad_6 requires python_abi 3.10.* *_cp310, but none of the providers can be installed
2024-04-29T19:23:05.6238290Z
2024-04-29T19:23:05.6238657Z Could not solve for environment specs
2024-04-29T19:23:05.6239042Z The following packages are incompatible
2024-04-29T19:23:05.6242554Z ├─ hkl is installable with the potential options
2024-04-29T19:23:05.6243263Z │  ├─ hkl [<IP_ADDRESS>3|<IP_ADDRESS>01|<IP_ADDRESS>57] would require
2024-04-29T19:23:05.6243659Z │  │  └─ python_abi 3.10.* *_cp310, which can be installed;
2024-04-29T19:23:05.6244043Z │  ├─ hkl [<IP_ADDRESS>3|<IP_ADDRESS>01|<IP_ADDRESS>57] would require
2024-04-29T19:23:05.6244409Z │  │  └─ python_abi 3.11.* *_cp311, which can be installed;
2024-04-29T19:23:05.6244711Z │  ├─ hkl <IP_ADDRESS>3 would require
2024-04-29T19:23:05.6245054Z │  │  └─ python_abi 3.6.* *_cp36m, which can be installed;
2024-04-29T19:23:05.6245361Z │  ├─ hkl <IP_ADDRESS>3 would require
2024-04-29T19:23:05.6245702Z │  │  └─ python_abi 3.7.* *_cp37m, which can be installed;
2024-04-29T19:23:05.6245996Z │  ├─ hkl <IP_ADDRESS>3 would require
2024-04-29T19:23:05.6246344Z │  │  └─ python_abi 3.8 *_pypy38_pp73, which can be installed;
2024-04-29T19:23:05.6247578Z │  ├─ hkl [<IP_ADDRESS>3|<IP_ADDRESS>01|<IP_ADDRESS>57] would require
2024-04-29T19:23:05.6248134Z │  │  └─ python_abi 3.8.* *_cp38, which can be installed;
2024-04-29T19:23:05.6248511Z │  ├─ hkl [<IP_ADDRESS>3|<IP_ADDRESS>01|<IP_ADDRESS>57] would require
2024-04-29T19:23:05.6248868Z │  │  └─ python_abi 3.9.* *_cp39, which can be installed;
2024-04-29T19:23:05.6249193Z │  └─ hkl <IP_ADDRESS>3 would require
2024-04-29T19:23:05.6249524Z │     └─ python_abi 3.9 *_pypy39_pp73, which can be installed;
2024-04-29T19:23:05.6250050Z └─ python_abi 3.12.* *_cp312 is not installable because it conflicts with any installable versions previously reported.

https://github.com/conda-forge/hkl-feedstock/pull/16 did not add Py3.12
https://github.com/conda-forge/hkl-feedstock/pull/17 needs to add Py3.12

This builds fine now for Python 3.12. I'll go ahead and merge this PR.


Upgrade to Angular/Cli 1.0.0 ng build --env=qa only uses environmentSource

Bug Report or Feature Request (mark with an x)
- [X] bug report -> please search issues before submitting
- [ ] feature request

Versions.
@angular/cli: 1.0.0
node: 6.9.1
os: win32 x64
@angular/common: 2.4.10
@angular/compiler: 2.4.10
@angular/core: 2.4.10
@angular/forms: 2.4.10
@angular/http: 2.4.10
@angular/platform-browser: 2.4.10
@angular/platform-browser-dynamic: 2.4.10
@angular/router: 3.4.10
@angular/cli: 1.0.0
@angular/compiler-cli: 2.4.10
npm version = 3.10.8
Repro steps.
ng build --output-path D:\Solution\MyWebSite\a --base-href /a/ ----target=development --env=qa
The log given by the failure.
environment variables from my environment.qa.ts are not the correct values.  It appear to ONLY take the environmentSource ( environment.ts ).
baseUrl = http://xyz.com
Desired functionality.
ng build --output-path D:\Solution\MyWebSite\a --base-href /a/ ----target=development --env=qa
This environment variable should be...
baseUrl = http://www.qa.com
Mention any other details that might be useful.
Contents of my environment files.
environment.ts
export const environment = {
production: false,
baseUrl: 'http://xyz.com'
};
environment.dev.ts
export const environment = {
production: false,
baseUrl: 'http://www.dev.com'
};
environment.qa.ts
export const environment = {
production: false,
baseUrl: 'http://www.qa.com'
};
environment.prod.ts
export const environment = {
production: false,
baseUrl: 'http://www.prod.com'
};
Contents of my angular-cli.json
{
"project": {
"version": "1.0.0",
"name": "gb-ng2"
},
"apps": [
{
"root": "src",
"outDir": "dist",
"assets": [
"assets",
"favicon.ico"
],
"index": "index.html",
"main": "main.ts",
"test": "test.ts",
"tsconfig": "tsconfig.json",
"prefix": "gb",
"mobile": false,
"styles": [
"styles.css"
],
"scripts": [],
"environmentSource": "environments/environments.ts",
"environments": {
"dev": "environments/environment.dev.ts",
"qa": "environments/environment.qa.ts",
"prod": "environments/environment.prod.ts",
}
}
],
"addons": [],
"packages": [],
"e2e": {
"protractor": {
"config": "./protractor.conf.js"
}
},
"test": {
"karma": {
"config": "./karma.conf.js"
}
},
"defaults": {
"styleExt": "css",
"prefixInterfaces": false,
"inline": {
"style": false,
"template": false
},
"spec": {
"class": false,
"component": true,
"directive": true,
"module": false,
"pipe": true,
"service": true
}
}
}

I have also same issue.  Actually we should not specify any value properties on source file., we specified in corresponding env files . So while n build or ng serve  compilation it throws an error as Property 'something' does not exist on type '{ dev: boolean; local: boolean; qa: boolean; }'

I am also having this issue. After some testing, it seems that it is not the evironmentSource that is being used. I changed is to another environment file and nothing changed. I think that the problem lies with the environment.ts. The environment declared here is always used.

This is an issue for e2e also.
1st: for app to be served, I need to explicitly add --serve=true
2nd: for CI purposes we need different environments and the build fails because it cannot run with the correct app variables (@ env file)
this makes it impossible for us to run e2e

Using the environment files you provided and adding console.log(environment) to main.ts, I got these results in the browser console:

ng build

ng build --env=dev

ng build --env=qa

ng build --env=prod

So as far as I can see here, the values within each environment file are in fact being used at runtime.
I cannot reproduce the issue you are describing. If you can provide a simple repository where you see this bug, I can reopen and investigate.


create example showing calculation of error bars using output from prototypical sql on a dataframe

https://datura.readthedocs.io/en/latest/examples.html#displaying-error-patches


Keeping costs low

We've gotten reports of unexpected costs from users that left the hello world running and had two standard instances running for a month. This keeps the costs to a minimum to kick the tires.

I believe @JustinBeckwith has an opinion on this

This is great! I've updated it in favor of #171


ModuleNotFoundError: No module named 'fairseq.data.token_block_utils_fast' while using torch.hub

🐛 Bug
Hi, I'm trying to follow the tutorial of the backtranslation (fairseq/examples/backtranslation) to reproduce their results. Unfortunately, when I run the code torch.hub.list(..), it failed with errors.
In my guess, this error seems to be almost the same as #1376.
Here is a part of the errors:
/usr/include/c++/7/bits/basic_string.tcc:1067:16: error: cannot call member function ‘void std::basic_string<_CharT, _Traits, _Alloc>::_Rep::_M_set_sharable() [with _CharT = char16_t; _Traits = std::char_traits<char16_t>; _Alloc = std::allocator<char16_t>]’ without object
       __p->_M_set_sharable();
       ~~~~~~~~~^~
/usr/include/c++/7/bits/basic_string.tcc: In instantiation of ‘static std::basic_string<_CharT, _Traits, _Alloc>::_Rep* std::basic_string<_CharT, _Traits, _Alloc>::_Rep::_S_create(std::basic_string<_CharT, _Traits, _Alloc>::size_type, std::basic_string<_CharT, _Traits, _Alloc>::size_type, const _Alloc&) [with _CharT = char32_t; _Traits = std::char_traits<char32_t>; _Alloc = std::allocator<char32_t>; std::basic_string<_CharT, _Traits, _Alloc>::size_type = long unsigned int]’:
/usr/include/c++/7/bits/basic_string.tcc:578:28:   required from ‘static _CharT* std::basic_string<_CharT, _Traits, _Alloc>::_S_construct(_InIterator, _InIterator, const _Alloc&, std::forward_iterator_tag) [with _FwdIterator = const char32_t*; _CharT = char32_t; _Traits = std::char_traits<char32_t>; _Alloc = std::allocator<char32_t>]’
/usr/include/c++/7/bits/basic_string.h:5042:20:   required from ‘static _CharT* std::basic_string<_CharT, _Traits, _Alloc>::_S_construct_aux(_InIterator, _InIterator, const _Alloc&, std::__false_type) [with _InIterator = const char32_t*; _CharT = char32_t; _Traits = std::char_traits<char32_t>; _Alloc = std::allocator<char32_t>]’
/usr/include/c++/7/bits/basic_string.h:5063:24:   required from ‘static _CharT* std::basic_string<_CharT, _Traits, _Alloc>::_S_construct(_InIterator, _InIterator, const _Alloc&) [with _InIterator = const char32_t*; _CharT = char32_t; _Traits = std::char_traits<char32_t>; _Alloc = std::allocator<char32_t>]’
/usr/include/c++/7/bits/basic_string.tcc:656:134:   required from ‘std::basic_string<_CharT, _Traits, _Alloc>::basic_string(const _CharT*, std::basic_string<_CharT, _Traits, _Alloc>::size_type, const _Alloc&) [with _CharT = char32_t; _Traits = std::char_traits<char32_t>; _Alloc = std::allocator<char32_t>; std::basic_string<_CharT, _Traits, _Alloc>::size_type = long unsigned int]’
/usr/include/c++/7/bits/basic_string.h:6693:95:   required from here
/usr/include/c++/7/bits/basic_string.tcc:1067:16: error: cannot call member function ‘void std::basic_string<_CharT, _Traits, _Alloc>::_Rep::_M_set_sharable() [with _CharT = char32_t; _Traits = std::char_traits<char32_t>; _Alloc = std::allocator<char32_t>]’ without object
ERROR:root:Internal Python error in the inspect module.
Below is the traceback from this internal error.

Traceback (most recent call last):
  File "/home3/wklee/.cache/torch/hub/pytorch_fairseq_master/hubconf.py", line 45, in <module>
    import fairseq.data.token_block_utils_fast  # noqa
ModuleNotFoundError: No module named 'fairseq.data.token_block_utils_fast'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home3/wklee/.pyenv/versions/3.7.9/lib/python3.7/distutils/unixccompiler.py", line 118, in _compile
    extra_postargs)
  File "/home3/wklee/.pyenv/versions/3.7.9/lib/python3.7/distutils/ccompiler.py", line 910, in spawn
    spawn(cmd, dry_run=self.dry_run)
  File "/home3/wklee/.pyenv/versions/3.7.9/lib/python3.7/distutils/spawn.py", line 36, in spawn
    _spawn_posix(cmd, search_path, dry_run=dry_run)
  File "/home3/wklee/.pyenv/versions/3.7.9/lib/python3.7/distutils/spawn.py", line 159, in _spawn_posix
    % (cmd, exit_status))
distutils.errors.DistutilsExecError: command '/usr/local/cuda-10.1/bin/nvcc' failed with exit status 1

Environment

fairseq Version (e.g., 1.0 or master): master
PyTorch Version (e.g., 1.0): 1.7 and also 1.6 (I tried it on two environments)
OS (e.g., Linux): linux 18.04
How you installed fairseq (pip, source): pip install --editable ./
Build command you used (if compiling from source):
Python version: 3.7.9
CUDA/cuDNN version: CUDA 11.0 (for torch==1.7) and CUDA 10.1 (for torch==1.6) (I tried it on two environments)
GPU models and configuration: RTX3090 for torch==1.7 and RTX2080 super for torch==1.6 (I tried it on two environments)

Thank you, in advance!!.

It was solved by setting the CUDA_PATH from /usr/local/cuda-11.0 to /usr/local/cuda-11.1. However, I'm not sure why.

i got rid of the error by making sure these were installed:
install_requires=[
"cffi",
"cython",
'dataclasses; python_version<"3.7"',
"hydra-core<1.1",
"omegaconf<2.1",
'numpy<1.20.0; python_version<"3.7"',
'numpy; python_version>="3.7"',
"regex",
"sacrebleu>=1.4.12",
"torch",
"tqdm",
]

i got rid of the error by making sure these were installed:
install_requires=[
"cffi",
"cython",
'dataclasses; python_version<"3.7"',
"hydra-core<1.1",
"omegaconf<2.1",
'numpy<1.20.0; python_version<"3.7"',
'numpy; python_version>="3.7"',
"regex",
"sacrebleu>=1.4.12",
"torch",
"tqdm",
]


babel configuration

Hi there,
I'm sorry I'm quite new to nodejs and other tools,
tried to run "npm run  build" then realised I had not installed babel
After installing babel globally, I realised I needed to install:
env preset
and other babel plugins refered in .babelrc.
         "add-header-comment",
        "transform-es2015-modules-commonjs",
        "transform-es2017-object-entries",
        "transform-object-rest-spread",
        "transform-es2015-destructuring",
        "array-includes"

what would be the right way to install all this component ? (--save-dev ? something else ?)
Thanks you for your help in advance

Hi, since there is a package.json file you can just do npm install in the root directory after cloning the repo. It should install all the relevant packages for you.
The plugins defined in .babelrc actually requires the packages to be installed via NPM (or Yarn) first, which is why Babel is complaining that it can't find the plugins.
Hope this helps!

If your NODE_ENV environment variable is set to production then it might not install the devDependecies. In that case, you should use --save-dev.

Of course, sorry about that :)
Everything is fine now, thanks a lot !

Glad you fixed it! You can give this repository a star if it has been useful 😁

first thing I did :)


Metamodel does not match spec: DocumentSelector

Spec:
export type DocumentSelector = DocumentFilter[];

Metamodel:
{
	"name": "DocumentSelector",
	"type": {
		"kind": "array",
		"element": {
			"kind": "or",
			"items": [
				{
					"kind": "base",
					"name": "string"
				},
				{
					"kind": "reference",
					"name": "DocumentFilter"
				}
			]
		}
	},
	"documentation": "A document selector is the combination of one or many document filters.\n\n@sample `let sel:DocumentSelector = [{ language: 'typescript' }, { language: 'json', pattern: '**∕tsconfig.json' }]`;\n\nThe use of a string as a document filter is deprecated @since 3.16.0.",
	"since": "3.16.0."
}

Not sure which is right.

We should remove string since it is not part of the protocol. In the lib it still exists due to backwards compatibility.

See dbaeumer/cruel-tiglon-jade


fix js tests for schema destroyer two

The problem was in antibody.js (test: tests/antibody-test). Lot component was using regiostry lookup to lookup panel view, which was return IPanel instead of AntibodyStatus, since 'AntibodyLot' is rerouted to 'Item' in registry.js. Hardcoded the use of the AntibodyStatus panel to fix this problem; it will be naturally fixed once object-specific js views are restored.


TST: Add new test for write read NotCompleted in DataStoreDirectory

@GavinHuttley
This commit is not final. I sent it to look at writing NotCompleted in DataStoreDirectory.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.03%) to 91.724%

Files with Coverage Reduction
New Missed Lines
%

.nox/test-3-8/lib/python3.8/site-packages/cogent3/app/data_store_new.py
2
95.37%

.nox/test-3-8/lib/python3.8/site-packages/cogent3/maths/scipy_optimize.py
3
84.09%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
180327

Relevant Lines:
196598

💛  - Coveralls

As I'm about to merge this with the develop branch I'm not going to accept any new features. I will create a features branch where you can submit the new feature there.
In my view, indexing by string should be implemented on the abstract base class.


MRG, FIX: Corrmap check

Closes #6836.
@nugenta can you see if this gives you a more informative error now?

In it goes, thanks for the look @jona-sassenhagen

Backported


Loading ... failed~

Got this..

Hi, would be able to share the beancount file you used for this? Along with the version of Python, beancount and fava?

Thanks I can see what has happened now. There was a change in Fava that has made a method my plugin uses incompatible. https://github.com/beancount/fava/commit/6f4bcd5db8603a8baa4cdd190f22d62b0520db46
If you drop your fava version down to 1.21 that should fix the issue for you.
It is not likely I will have time at the moment to fix the issue but if you are able to submit a pull request with a fix, I would happily look at it.

@kklod can you please test against the latest version to see if that’s fixed your issue. This has not been published to PyPi yet so it would need to be a local install of the package.

I've published the changes to version 0.9.1 on PyPi

Thanks!  I'll try it later, kinda busy...

It works pretty well ! Thanks!


epic: badger registry v2 release

Badger Registry v2
A new version of the Badger Registry is released and available for deployment and integration.
The registry code can be viewed here.
The registry has been updated to service infrastructure such as the API, UI, and keepers.
As such, integration points will be required for cutting over existing infrastructure to the new registry.
Requirements

[x] #1217
[x] #1218
[x] #1219
[ ] #1220
[ ] #1221
[ ] #1222

@NamelsKing reassinged this one just to you for tracking as you're taking them on


Needs clarification: is it possible to restore a previously built index from disk?

Hi there,
Is it possible to restore the state of previously built arroy tree from disk? Or does the it need to be built from scratch with a Writer whenever a new process using arroy starts up? This isn't necessarily an issue, but may be worth clarifying in the README.
Thanks for open sourcing this project. I've been using it to run similarity search for audio files, and it's been great to work with!

Disregard this!


Option to delete user data

Users need the option to delete their account, and we should ensure that all the information associated with that user is deleted. However, public information such as what is still on GitHub, may need to be preserved. If the user also deletes their public GitHub information (or makes it private) then we should strive to remove the information on our side as well.

I think, besides the account deletion, there should be a way to delete particular pieces of content posted on the website. Such as forum posts, comments, etc.

Such as forum posts, comments, etc.

Well that's easy, as there is no forum and no way to leave comments :) Nor do we plan such a feature.

Let me propose a tentative spec for the user deletion feature for everyone to discuss:

It's self-service (as opposed to having to request deletion to some website admin).
What's deleted: all the info under "Profile"
Contributions from the user (a this point, entries in News) are not deleted, the rationale being that those contributions have entered the public domain. Currently this is not covered by the terms of use.

We need to decide what to do with the icon shown alongside news posted by the (now deleted) user.

There's an additional section in "Profile" named "Danger". Inside it, there's a differently colored button in the profile section with the tile "Delete my account".
Pressing a button shows a windows similar to the one used at Github. The texts will be different but you get the idea.

We request the user's email address and that s/he retype a message such as "delete my account".
Pressing the button does the deletion and shows a popup with the message "<email_address>. Your account has been deleted", plus one button "Return to homepage". Pressing the button will bring the user to well, the homepage, in anonymous mode.

I'm not so sure this 10-day grace period is so necessary, given that, if we're not deleting the user's contributions, the only information deleted is that which appears on the profile section: name, picture, update preferences; nothing terribly valuable anyway.

Sometimes people delete their account for emotional reasons which they soon regret.

As per the 06/28 meeting, let's implement immediate deletion and then we can continue with #1121

GDPR compliance
I've talked to an expert on GDPR here in Spain and the recommendations for us regarding GDPR compliance are:
Physical vs. logical deletion
It is recommended that we don't physically delete the user's data (email address, name, etc.) for a safeguard period of at least 6 months (some companies go beyond that limit and wait for a number of years), in case an authority requests this info (for a criminal investigation, for instance). After this safeguard period it is OK to physically remove the information (by purging the DB for instance).
While the user is logically deleted, it is required that the service does not use the affected information in any way --it's simply sitting there, but otherwise opaque to the entire service.
Contributions from deleted users
According to the TOU, contributions are licensed to us and we can keep news, articles, etc. uploaded by the deleted user. It is recommended that we remove personal information, like the name "Louis Tatta" and the avatar, as shown here:

Hosting country for the user DB
This is a very critical point. According to GDPR, storing personal information from EU citizens outside the EU is subjected to serious scrutiny. This has nothing to do with the company's registration country, we're talking about the physical location of the DB. In our case (assuming the current DB is located in the US), we have these options:

We move the user DB to the EU (for instance, to the Ireland region of AWS).
We enter the EU-U.S. Data Privacy Framework, which basically requires that the C++ Alliance registers with the US branch of this framework.
We justify that the service needs be hosted in the US, inform the users about this and contact each and every national data security agency in EU to let them know.

3 is basically unfeasible, and 1 is by far the simplest solution, and this the recommended option.

website avatars should have "Display Name" just like Slack

I wouldn't mind doing 1 right away and also pursuing number 2 in parallel just in case

@rbbeeston
Merged, but will need some testing by someone more familiar with the application. The test case I did was to delete the user and then confirm that they could not log in again.


"Last change by" is not always correct for FUGR includes

All FUGR includes return the same last change by (last user who changed something in the FUGR)

What kind of object?

The logic for finding the "last changed by" is located in class ZCL_ABAPGIT_OBJECT_FUGR method ZIF_ABAPGIT_OBJECT~CHANGED_BY, can you help to look up what he/she changed and checking in the debugger?
It is difficult to trace remotely

I believe the problem is that FUGR's changed_by method returns the last user who touched any of the FUGR's includes.

And which user does SAP standard report as last user?

currently not able to reproduce this issue

I will try to make an easy to reproduce example to see if this persists at some point.

Repro steps:

Create a package with a FUGR and some includes as user XXX, version it in abapgit
Get user YYY to change and activate any of the includes and not touch anything else
The user who last changed the whole FUGR as well as every include is now user YYY

@jfilak is right, the correct information is retrieved here for each include:
zcl_abapgit_object_fugr->zif_abapgit_object~changed_by
LOOP AT lt_includes ASSIGNING <lv_include>.
      SELECT unam AS user udat AS date utime AS time FROM reposrc
        APPENDING CORRESPONDING FIELDS OF TABLE lt_stamps
        WHERE progname = <lv_include>
        AND   r3state = 'A'.                              "#EC CI_SUBRC
    ENDLOOP.

but then the info is lost when the dates are sorted and the latest is used for the whole group.
SORT lt_stamps BY date DESCENDING time DESCENDING.

    READ TABLE lt_stamps INDEX 1 ASSIGNING <ls_stamp>.
    IF sy-subrc = 0.
      rv_user = <ls_stamp>-user.
    ELSE.
      rv_user = c_user_unknown.
    ENDIF.

I could reproduce the issue...
A possible solution is to change changed_by to receive the filename as an input parameter. In this way, we could know which function group includes is being processed. See below:

This would enable us to identify who lastly modified an include.
Also, <ls_local> could be passed as a parameter, instead of only <ls_local>-file. But this would trigger changes in all "serializable" objects.
Any thoughts on this?

last changed by has been removed since this issue was created

"Last Changed" is still used on Diff (ZCL_ABAPGIT_GUI_PAGE_DIFF->RENDER_DIFF_HEAD):

For objects with only one file (PROG), the current logic makes sense. For objects with multiple files (FUGR, but also CLAS which can have another file for test classes), ZIF_ABAPGIT_OBJECT~CHANGED_BY is simply not sufficient.
I like the suggestion of @EduardoCopat. Let's pass the filename to CHANGED_BY as an additional, optional parameter. Then classes for objects with multiple files can be enhanced to return the correct user (without impactive the rest).
It won't work for methods or other parts of a class which is probably the most common scenario, however.

Although SAP can version methods separately, it doesn't fit in the whole git concept which is file based. File-by-file last changed is important as you can have many includes under a FUGR, especially in older code. It is very messy to figure out who did what when you have more than one developer on the same system.

Closed by #6274


Provide a general reification function

For turning a generator into a list, recursively, so that the result can e.g. be stored in a variable and reused.

There are two other related issues here:

shift mutates the argument, regardless of whether it's a generator or a list, which means that even if you reify something and store it in a variable, the first iteration over that list will typically cause it to become empty.  Changing how shift works for non-generators only does not sound like a good solution to this problem.  Requiring clients to remember to clone variables after load is not very pleasant either, though.
Reifying the result of the last call (assuming it isn't larger than some threshold value) and storing it in a known variable name like last would be useful.


Keybindings editor's info tooltips are misaligned

VSCode Version: 1.10.1
OS Version: macOS 10.12.3 & Windows 10 (both in french)

The info tooltip displaying the keybinding in the current keyboard layout is not aligned with the key combinations and some info tooltips are displayed where there is no key combination. The key displayed on the tooltip text for these bogus tooltips is ";"
The cursor position is also misaligned on lines with info tooltips.
In the screenshot, the cursor is at the end of line 3, typing text will insert it after the comma, but the cursor is displayed 2 characters before. On line 7 the cursor will be displayed 4 characters before. The rest of a line after a word-wrap is not affected.

This affects the user's keybinding.json editor and the Default key combinations editor. The issue was already present in VSCode 1.9.

workaround: use the setting "editor.disableMonospaceOptimizations": true


Call willMoveToStickyPoint and didMoveToStickyPoint in moveToVisiblePoint

That's just when you call moveToVisiblePoint it's not quite StickyPoint and that's what may be confusing. And if this VisiblePoint is not StickyPoint does PullUp go to the closest StickyPoint or stay there?
Probably not quite an issue, as I've said, just my thoughts on this topic.

Ah I get your point now.
Let's image to have three sticky point: 10, 20 and 30, calling pullUpControllerMoveToVisiblePoint(15, animated: true, completion: nil) will cause willMoveToStickyPoint and didMoveToStickyPoint to be called and 15 to be passed as parameter but 15 isn't a sticky point.
I didn't thought about that because to me it doesn't really make sense to pass a "non sticky point" to that method but now there're two solution:

rename willMoveToStickyPoint and didMoveToStickyPoint to willMoveToPoint and didMoveToPoint
move the pull up controller's view to the nearest sticky point in pullUpControllerMoveToVisiblePoint

I think the first solution is better, if someone wants the view to be positioned to some sticky point it could just provide it to the method.
What do you think?

I also don't move PullUp to any other points, though that's just what I wondered about. Yeah, the first option will provide more flexibility and will solve one side of the issue, though then you will be able to fix PullUp on the place where it shouldn't stay because the logic is that PullUp can only be fixed on sticky points.
I also don't know what solution is better, though if we look only on the library's logic pullUpControllerMoveToVisiblePoint should move to the nearest point, after all.
That's nothing of critic issue in the end, so any of those solutions will pass, I think.

If someone will need it sometime, temporary for content not to scroll when not needed created a CollectionView subclass and setting its scrolling property in scrollViewWillBeginDragging.
//MARK: The function that is called each time user is about to scroll
    func scrollViewWillBeginDragging(_ scrollView: UIScrollView) {
        //Setting scrolling value according to opened state
        collectionView.set(scrolling: state == .open)
    }

//MARK: Custom collectin view to work with PullUp
class PullUpCollectionView: UICollectionView {
    //MARK: Variable saying if should be scrollable
    private var scrolling: Bool = false
    //MARK: Variable that saves initial content offset value
    private var initialContentOffset: CGPoint = .zero

    //MAKR: The function for setting if scrolling is enabled
    func set(scrolling: Bool) {
        //Setting scrolling
        self.scrolling = scrolling
        //And initial content offset
        self.initialContentOffset = contentOffset
    }

    //MARK: Overriding content offset
    override var contentOffset: CGPoint {
        //On set
        didSet {
            //Checking if value is not the initial and shouldn't scrolling
            if contentOffset != initialContentOffset && !scrolling {
                //Then setting value back to starter one
                contentOffset = initialContentOffset
            }
        }
    }
}

I'll go with the first solution here, since in https://github.com/MarioIannotta/PullUpController/issues/40 we talked about using function instead of callbacks the new functions will be
pullUpControllerWillMove(to point: CGFloat)
pullUpControllerDidMove(to point: CGFloat)
pullUpControllerDidDrag(to point: CGFloat)


Discrepancy in Supported Types

There appears to be a discrepancy between the types listed as supported in the GetStorage documentation and what is actually supported. This inconsistency makes it challenging for developers to understand the full capabilities of GetStorage and how to best utilize it in their projects.
I am referring to:

https://pub.dev/packages/get_storage
https://pub.dev/packages/get_storage/example
my own experiments with the package

The README states that GetStorage supports the following types: String, int,  double, Map, List (bool is not listed here).
Yet, one of the examples includes:
dartCopybool get isDark => box.read('darkmode') ?? false;
void changeTheme(bool val) => box.write('darkmode', val);

Is this being automatically casted to an int or something?
What is the full list of types that GetStorage actually supports?
Is boolean support an intentional feature that was omitted from the documentation?
Are there any other types (e.g. enums) that are supported but not mentioned in the docs?
What is the recommended approach for storing custom types or complex objects?

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx


[WIP] License feature integration

integrate license service with angular backend. For getting a dynamic license list, text.

[ ] Ensure that you followed our toolchain guide. Especially, we require a single commit
[ ] Ensure that the commit message is a good commit message
[ ] Ensure to use auto format in all files
[ ] Ensure that you appear in NOTICE at Copyright Holders
[ ] Tests created for changes
[ ] Documentation updated (if needed)
[ ] Screenshots added (for UI changes)

Please follow the contributing guideline.


Bash completion

Fix #215

I simplified the locking of the registry file in hope to solve #102. Could you please review those changes, @gnodet ?


[BUG] Possible issue with RTags not finding rc & rdm

Describe the issue
Executables rc and rdm can't be found.
I'm suspecting it's related to the difference between rtags-path and rtags-install-path. From what I can see, rtags-path best suites our needs as rtags is set to be installed in (concat doom-etc-dir "rtags/").
Also, shouldn't (when (/= (length bins) 0) be (when (/= (length bins) 2)?

In `modules/lang/cc/doctor.el`
(when (require 'rtags nil t)
  ;; rtags
  (let ((bins (cl-remove-if #'executable-find `(,rtags-rdm-binary-name ,rtags-rc-binary-name))))
    (when (/= (length bins) 0)
      (warn! "Couldn't find the rtag client and/or server programs %s. Disabling rtags support" bins))))

In src/rtags.el
(defun rtags-executable-find (exe)
  (cond ((and rtags-tramp-enabled (tramp-tramp-file-p default-directory)) exe)
        ;; for tramp let's rely on `tramp-remote-path`, so if You have some *debug*
        ;; directory to store RTags binaries, just put it to `tramp-remote-path`
        (rtags-path
         (or
          (let ((file (expand-file-name exe rtags-path)))
            (and (file-executable-p file) file))
          (let ((file (expand-file-name exe (concat rtags-path "/bin/"))))
            (and (file-executable-p file) file))))
        ((let* ((install-path (rtags-package-install-path))
                (file (and install-path (concat install-path "/rtags-" rtags-package-version "/bin/" exe))))
           (and file (file-executable-p file) file)))
        (t (executable-find exe))))

Steps to reproduce
Running bin/doom doctor should result in a warning regarding rc and rdm not being found.
System information
I'm not sure why doom/info is reporting fatal: not a git repository (or any of the parent directories): .git. It could be that I have a symbolic link from doom-emacs to ~/.emacs.d.

emacs   version    26.1
        features   XPM JPEG TIFF GIF PNG RSVG IMAGEMAGICK SOUND GPM DBUS GSETTINGS NOTIFY ACL LIBSELINUX GNUTLS LIBXML2 FREETYPE M17N_FLT LIBOTF XFT ZLIB TOOLKIT_SCROLL_BARS GTK3 X11 THREADS LIBSYSTEMD LCMS2
        build      Apr 11, 2019
        buildopts  (--build x86_64-linux-gnu --prefix=/usr --sharedstatedir=/var/lib --libexecdir=/usr/lib --localstatedir=/var/lib --infodir=/usr/share/info --mandir=/usr/share/man --enable-libsystemd --with-pop=yes --enable-locallisppath=/etc/emacs:/usr/local/share/emacs/26.1/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/26.1/site-lisp:/usr/share/emacs/site-lisp --with-sound=alsa --without-gconf --with-mailutils --build x86_64-linux-gnu --prefix=/usr --sharedstatedir=/var/lib --libexecdir=/usr/lib --localstatedir=/var/lib --infodir=/usr/share/info --mandir=/usr/share/man --enable-libsystemd --with-pop=yes --enable-locallisppath=/etc/emacs:/usr/local/share/emacs/26.1/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/26.1/site-lisp:/usr/share/emacs/site-lisp --with-sound=alsa --without-gconf --with-mailutils --with-x=yes --with-x-toolkit=gtk3 --with-toolkit-scroll-bars 'CFLAGS=-g -O2 -fdebug-prefix-map=/build/emacs-cQxEw2/emacs-26.1+1=. -fstack-protector-strong -Wformat -Werror=format-security -Wall' 'CPPFLAGS=-Wdate-time -D_FORTIFY_SOURCE=2' 'LDFLAGS=-Wl,-Bsymbolic-functions -Wl,-z,relro')
        windowsys  x
        daemonp    server-running
doom    version    2.0.9
        build      fatal: not a git repository (or any of the parent directories): .git
system  type       gnu/linux
        config     x86_64-pc-linux-gnu
        shell      /usr/bin/fish
        uname      Linux 5.0.0-27-generic #28-Ubuntu SMP Tue Aug 20 19:53:07 UTC 2019 x86_64
        path       (~/.local/bin /snap/bin ~/x-tools/i686-w64-mingw32/bin ~/x-tools/i686-pc-linux-gnu/bin ~/x-tools/x86_64-w64-mingw32/bin ~/x-tools/sh2eb-elf/bin ~/.local/bin /snap/bin ~/x-tools/i686-w64-mingw32/bin ~/x-tools/i686-pc-linux-gnu/bin ~/x-tools/x86_64-w64-mingw32/bin ~/x-tools/sh2eb-elf/bin ~/.local/bin /usr/local/sbin /usr/local/bin /usr/sbin /usr/bin /sbin /bin /usr/games /usr/local/games /snap/bin ~/.local/tool-chains/bin ~/.local/tool-chains/sh-elf/bin /usr/lib/emacs/26.1/x86_64-linux-gnu)
config  envfile    nil
        elc-files  0
        modules    (:completion (company +auto) ivy :ui workspaces doom doom-quit hl-todo modeline nav-flash ophints (popup +all +defaults) unicode vc-gutter vi-tilde-fringe window-select :editor snippets fold multiple-cursors rotate-text :emacs dired electric vc :tools (lookup +docsets) eval editorconfig flycheck flyspell magit rgb :lang assembly cc common-lisp csharp data emacs-lisp markdown (org +dragndrop +attach +babel +capture +export +present) python (sh +fish) :config (default +bindings +smartparens))
        packages   (n/a)
        elpa       (n/a)

P.S.: I'll be getting back to PR #1437. I got Irony up and running, so I see how RTags+Irony would be an ideal combo.

Would this change be sufficient?
   ;; rtags
-  (let ((bins (cl-remove-if #'executable-find `(,rtags-rdm-binary-name ,rtags-rc-binary-name))))
+  (let ((bins (cl-remove-if #'rtags-executable-find `(,rtags-rdm-binary-name ,rtags-rc-binary-name))))
     (when (/= (length bins) 0)

Also, shouldn't (when (/= (length bins) 0) be (when (/= (length bins) 2)?

No, because the cl-remove-if predicate is inverted. It will only remove binaries if they are found. This way, the following warning can complain about the list of executables that weren't found.

This should be fixed as of 009cfbc. Let me know if that isn't the case and I'll reopen the issue. Thanks for bringing it to my attention!


Update MSRV on pipelines

I'm not sure when we switch to GHA though.
Also tweaks editorconfig for YAML.

libsqlite3-sys-0.17.3 requires 1.40.0 or later:
error[E0658]: non exhaustive is an experimental feature
  --> /home/vsts/.cargo/registry/src/github.com-1ecc6299db9ec823/libsqlite3-sys-0.17.3/src/lib.rs:20:1
   |
20 | #[non_exhaustive]
   | ^^^^^^^^^^^^^^^^^
   |
   = note: for more information, see https://github.com/rust-lang/rust/issues/44109

Allowed needless_doctest_main lint since it loses readability.

The remaining failure should be fixed by #2339.


Module 2 exercises [Gideon Anim K. Bempong]

WS2_M2E2_Submission.docx

Well done, Gideon. You clearly grasped the material outlined in Module 2 and successfully applied it to these exercises. Thank you for your participation! Please be sure to fill out the post-Module survey before you begin the next Module.


[SPARK-18694][SS]Add StreamingQuery.explain and exception to Python and fix StreamingQueryException

What changes were proposed in this pull request?

Add StreamingQuery.explain and exception to Python.
Fix StreamingQueryException to not expose OffsetSeq.

How was this patch tested?
Jenkins

Test build #69596 has started for PR 16125 at commit 757cf2f.

Test build #69596 has finished for PR 16125 at commit 757cf2f.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69596/
Test FAILed.

Test build #69597 has started for PR 16125 at commit 4334557.

Test build #69597 has finished for PR 16125 at commit 4334557.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69597/
Test FAILed.

Merged build finished. Test FAILed.

Test build #69607 has started for PR 16125 at commit 169cd20.

Test build #69607 has finished for PR 16125 at commit 169cd20.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69607/
Test PASSed.

Merged build finished. Test PASSed.

Merging this to master and 2.1

Merged to master, but 2.1 failed due to mima exclude conflict. can you make another PR for 2.1 @zsxwing thanks!


The automatic update function of the added view failed to save

Exception:
SQL: insert into mem_dashboard_widget         (alias, dashboard_id,widget_Id,x,y,width,height,frequency,polling,create_by,create_time)         VALUES                        (             ?,             ?,             ?,             ?,             ?,             ?,             ?,             ?,             ?,             ?,             ?             )
Cause: java.sql.SQLSyntaxErrorException: Unknown column 'alias' in 'field list'
solution:
Add the 'alias' attribute to the 'mem_dashboard_widget ' table?

Yes.


Fix alot of stuff

"program.ts" was rename to "index.ts"
rename client "program" to "client"
fix some types
commit tsconfig.json
commit package-lock.json

I'll merge when you call it program.ts 😤


added ability to score test images to disk for dutch F3

closes https://github.com/microsoft/seismic-deeplearning/issues/229

/azp run

/azp run

/azp run


update-user-provided-service should not unset values

Issue https://github.com/cloudfoundry/cli/issues/1694 describes an
incorrect behaviour where update-user-provided-service can unset existing
credentials associated with a service even when the -p flags is not
specified. We should fix that so that values are not changed unless the
associated flag is specified. Unsetting a value should be done by
explicitly setting in to an empty string.
In the process we refactored the update-user-provided-service command
to be consistent with the v6 commands

Co-authored-by: Niki Maslarski<EMAIL_ADDRESS>

Hey @blgm thanks for this. Did you folks fix the bug on v6 after you refactored the command? or is the bug fix only on v7? Sorry i'm sure this information is listed somewhere. thanks.

Hi @abbyachau, we fixed the bug and refactored the command to v6 at the same time.


Added config files as static asset

Used go generate to bundle static assets.
/assign @JimBugwadia

/ok-to-test

/lgtm
/approve

/lgtm


Crash of U8G2 rendering on ESP8266 trying to render bitmap

Steps to reproduce:
Build any U8G2 example with the latest ESP8266 library. The display will not even initialise and the board will crash when it hits the first bitmap drawing function.
Notes:
During the testing cycle for 1.4.0 we determined that there's been a change in the ESP8266 library in the way it handles progmem, this appears to crash the board when using the U8G2 drawBitmap function.

Moved both AdaGfx and U8G2 to use XBM. A significant improvement as these can be edited in GIMP and turned into data bytes.


Drop python 3.7 support

As PyTorch did: https://github.com/pytorch/pytorch/releases/tag/v2.0.0

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

TensorFlow, as well as Transformers and Diffusers, support python 3.7 so I don't think we should drop it

@echarlaix yes right. Still think we should disable the CI on 3.7, as done in transformers. WDYT?

@echarlaix yes right. Still think we should disable the CI on 3.7, as done in transformers. WDYT?

Souns good to me


Access policy of Secrets is required

According to document, [Certificate permissions] is needed for activating SSL with BYOC, but We found that [Secret permissions] is also needed for activation. There will be below error message if [Secret permissions] is not configured.
Please help to modify document since our customer got confused.

Failed to update custom domain properties
We don't have permission to access this secret. Go to "Access policies" in your Key Vault account to give Azure CDN permission to get secrets.

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

ID: beac0740-af6e-61ce-bcfa-e783ca181b7d
Version Independent ID: d97885ea-b051-5fde-bd14-ec977e34760e
Content: Tutorial - Configure HTTPS on an Azure CDN custom domain
Content Source: articles/cdn/cdn-custom-ssl.md
Service: azure-cdn
GitHub Login: @mdgattuso
Microsoft Alias: magattus

@taminta, Thank you your query. We are looking into this and will get back to you on this thread.

PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/91078


Implemented dynamic edit input fields based on the number of trip days.
Implemented dynamic email invite list

Page not responsive
Font family not defined


Add arbitrary HTML elements?

Hi, is there a way for me to add arbitrary custom elements? I like to use semantic tagging and I'd like to be able to do something like:
myCustomElement("content") --> <my-custom-element>content</my-custom-element>

Yes, you can do it via:
var myCustomElement = domjs.ns.myCustomElement = domjs.ns.element.bind(domjs, 'my-custom-element');

Optionally you may also provide some custom constructor or methods for that element:
// This has to be defined before any `my-custom-element` is created by domjs
require('domjs/ext')['my-custom-element'] = {
  _construct: function (/* constuctor args */) {},
  methodA: function () { ... }
};

I'll keep it open, until I include this information in documentation

Very cool! Thanks for this project, there are a lot of cool patterns available with it.


[language] module for block metadata

Motivation
First step toward representing block metadata (height, time, signers, etc.) in the global state. This commit adds a new Block module with a Block.T resource that (for now) contains only the height. The module and a Block.T resource at the Association address are included in the genesis blob.
The Block module contains a prologue procedure for setting the block metadata. In the future, a special transaction invoking this procedure will be the first tx executed in every block.
Test plan
New E2E tests for reading the block height and invoking the block prologue.

It will be more like that WriteSet transaction--a distinct code path with special semantics. But the special semantics will be quite simple: just call Block.prologue.


Develop camera docker container that can run on arm

Description
Camera service must be able to:

Coming later

Additional Information
No response

Possible issue with USB through

example of usb pass through


Crystal Farming

Hello,
I created a list in Teamcraft with all 6 crystals, gave each of them a max of 9999, and imported it into GBR. The issue is, once you hit 9999 of whatever crystal you are gathering, and you're in the middle of a node (say 2/4) it gets hung up, and won't leave the node because your inventory is full. It will just sit there trying to spam gather forever. Sorry if this is already a known issue.

This is known and probably won't be fixed any time soon. Just keep a closer eye on your inventory while gathering items that have a cap.


Use node js git security advisories instead of using npm registry

This PR switches the NPM importer to use https://github.com/nodejs/security-wg instead of  https://registry.npmjs.org/-/npm/v1/security/advisories
Reasons :
https://registry.npmjs.org/-/npm/v1/security/advisories has given trouble to us in past, recently it was down for some time. The git data source is way more robust.
We also get to use GitDataSource which has alot of goodies.
https://github.com/nodejs/security-wg is the upstream
Signed-off-by: Shivam Sandbhor<EMAIL_ADDRESS>

Please rebase and I will mereg

I may wonder if using a direct download instead of a git checkout may have been simpler... but that's a small thing

Good question. Thing is GitDataSource uses checkout to detect changes inside a git repository. So if you run an  Importer again after some duration it will only fetch the data which is new/changed.


Map is not showing(Grey color) properly in ionic

I am using ionic and angularjs to show map with marker. It is working properly in browser or emulator but not properly showing(Grey color) in android device.
what's wrong I don't know?

I solve this problem. So I am closing this issue

How did u fix this issue?

Sorry I can't explain here.

Thats fine. I managed to fix. In my case I forgot to specify tiles property for map. I have added my code below which might help for someone.
            <leaflet id="drawingMap" defaults="map.defaults" watch-paths="false" data-tap-disabled="true" marker-options="markerOptions" event-broadcast="map.events" center="map.center" markers="map.markers" bounds="map.bounds" tiles="map.tiles" ng-if="map"></leaflet>

$scope.map = {
defaults: {
zoomControl: false
},
tiles: {
url: 'http://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png'
},
bounds: {},
markers: {},
center: {
lat: defaultCenter.latitude, // This is current location of user. required for my case.
lng: defaultCenter.longitude,
zoom: 10
},
events: {
map: {
enable: ["zoomend", "dragend", "popupclose", "mouseover", "mouseout", "click", "popupopen"],
logic: 'emit'
}
}
};

Ok

kl


Evaluate the current implementation of Base64, design for errors, and consider removing dependency on openssl and OS-specific APIs

We have APIs to convert to and from binary data and base64 encoded text:
Encoding accepts any amount of bytes and constructs a string, and therefore most inputs are valid
Decoding, however, can only succeed if a valid base64 encoded string is provided (with the correct character set, number of padding, and length).
The purpose of this issue is to:
Re-implement the APIs in standard C++ and avoid any native dependencies that have more complex error handling and lifetime management.
Ensure to validate and throw for invalid input while decoding.
Out-of-scope:
Some folks may provide partial text data to the Base64 decode APIs, to support streaming. These APIs wouldn't support that today, and only work for single calls where all the data is provided up front.
Another scenario is providing a buffer with a lot of data, including some after the bytes/strings needed to be encoded/decoded, and let the API consume as much as possible and return the partial result. This scenario isn't supported either.
See some of the feedback from https://github.com/Azure/azure-sdk-for-cpp/pull/1297#pullrequestreview-564685471
cc @vhvb1989

This is about underlying implementation

We have APIs to convert to and from binary data and base64 encoded text:

Encoding accepts any amount of bytes and constructs a string, and therefore most inputs are valid
Decoding, however, can only succeed if a valid base64 encoded string is provided (with the correct character set, number of padding, and length).

The purpose of this issue is to:

Re-implement the APIs in standard C++ and avoid any native dependencies that have more complex error handling and lifetime management.
Ensure to validate and throw for invalid input while decoding.

Out-of-scope:

Some folks may provide partial text data to the Base64 decode APIs, to support streaming. These APIs wouldn't support that today, and only work for single calls where all the data is provided up front.
Another scenario is providing a buffer with a lot of data, including some after the bytes/strings needed to be encoded/decoded, and let the API consume as much as possible and return the partial result. This scenario isn't supported either.

This type needs more extensive testing of inputs.  Also we should make this into a header only implementation.  I don't see a need on this one for a .cpp file.
With an extensive test suite we should be able to modify the underlying implementation post GA.

@RickWinter @ahsonkhan  I don't think this issue should be blocking-release, neither for April.
Also, I don't think we should re-implement the wheel instead of using the libraries. We are already using openSSL with libcurl and Windows.h for winhttp.  We will not remove the dependencies anyway.

I wrote an implementaition for C SDK at one time. I can port it over, with tests, in 2 days from starting work to checked in.

@ahsonkhan , I am closing this as we already ported Anton's C99 implementation and even improved with Larry :)
Feel free to re-open if you consider it


Globalize.parseNumber strips out letters I'm not sure it should

Hi. I'm trying to migrate something up from Globalize 0.x to 1.0, and I'm getting some weird behavior with numbers. Specifically, parseFloat in 0.x would consider a value like "asdf12" to be NaN, which is what I'd expect.
1.0 is returning 12. It does the same for "asdf12asdf", but not "asdf12asdf12", which is considered NaN.
I think the offending code is around here, in number.js:
	// Is it a valid number?
	value = value.match( numberNumberRe );
	if ( !value ) {

		// Invalid number.
		return NaN;
	}

It strips off the prefix and suffix text, so what's left is 12, but never checks if the prefix and suffix make any kind of sense at all. At least that's what it looks like it's doing to me, I'm not an expert on Javascript at all. :)
Is this the expected behavior now? Thanks for looking into it. :)

Thanks for bringing this issue into our attention. This should be fixed when https://github.com/jquery/globalize/issues/353 is fixed. We'll parse numbers similarly to the way we parse Dates (lexical analysis, then parsing).
Closing in favor of #353.

Awesome, thank you. :)


 Ive added the following line to my printer.py file to see if it fixes it.
'WarDeclared': self.corporation_war_declared,
and I adjusted corporation_war_declared as follows.
return '{0:get_corporation_or_alliance(declaredByID)} has been declared war on {0:get_corporation_or_alliance(againstID)}'
I looked at the ESI response and it is now passing WarHQ information, but I dont think you will be able to pull that info outside of EVE, which is why I had WarDeclared reference corporation_war_declared for printing.
Below is the GET response for you viewing pleasure.
"text":
"againstID: Defender Corp/Alliance ID\n
cost: 100000000\n
declaredByID: Agressor Corp/Alliance ID\n
delayHours: 24\n
hostileState: false\n
timeStarted:<PHONE_NUMBER>00000000\n
warHQ: <b>In game link to structure</b>\n
warHQ_IdType:\n-<PHONE_NUMBER>693\n- 35825\n",  ##I dont know what these ID's relate to for the WarHQ


Fixed vuex-shared-mutations peer dependency

<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>

👍

Could you release a v0.0.4 ?

Bump @xanf

Sorry, will do today @qkdreyer

Bump @xanf

@qkdreyer Sorry once again, ETA ~12h,  travelling now :)

Bump @xanf

Bump @xanf

Bump @xanf

Bump @xanf 😢

Sorry, @qkdreyer
Just done it

Thanks 🙏


ssl change

I just added openssl30.

Duplicate openssl-devel as openssl11
Duplicate openssl as openssl10
update USES=ssl for the following values: openssl10, openssl11, openssl30, libressl, libressl-devel
remove openssl, openssl-devel ports after ravenadm release

it might be a good idea to make "openssl" be an alias fo "openssl10" and "openssl-devel" be an alias for "openssl11" to avoid major breakage during the transition.

requires update to autoselect-ssl:single:standard too

done 2fc2cb721740173385ce93e2795faefc415e4d5d


feat: add new coachmark component

Overview

Added a new Coachmark component which uses the Popover component
Added a prop to the Popover component that allows clients to add a pattern background

PR Checklist

[ ] Related to designs: figma
[ ] Related to JIRA ticket: EGG-599
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

New Coachmark Story

📬Published Alpha Packages:@codecademy/gamut-labs@9.3.1-alpha.a32d07.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60246d3103fd9e5f6bd28bc2--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@9.3.1-alpha.d27bfb.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60269897230aa103a2997ae9--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@10.0.12-alpha.ee19bb.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://602e754316ddfe1c0864089b--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@10.0.12-alpha.48e6ba.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://602fc6743856f30e7d10e671--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@10.0.14-alpha.7777b8.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://602fcb98c0150b335b29bdce--gamut-preview.netlify.appDeploy Logs


Negative data causes trouble with stacked bar chart tooltip

What is the intent of a stacked bar chart having a negative and positive component?
From calculateStackData.js:
              const totalKey = datum.yValue >= 0 ? 'positive' : 'negative'
              // Assign the base
              datum.baseValue = start[totalKey]
              // Add the value to the base
              datum.totalValue =
                datum.baseValue + (validPoint ? datum.yValue : 0)
              // Update the totals
              scaleTotals[secondaryAxisIndex][d.primary][totalKey] =
                datum.totalValue

In the tooltip there is only one Total and it's not calculated correctly.  Anyone know the reason for the more complex collection of positive and negative totals?

Due to a massive overhaul and rewrite of React Charts to TypeScript and a massively upgraded public API, this issue is being bulk closed for triage. The latest version can be tested via the react-charts@beta tag. Documentation is in the process of being upgraded, but thankfully, everything is fully typed now, so you can explore the API that way for now.
If, after testing or playing with the v3 beta, you believe your issue is still not addressed or relevant, feel free to open a new issue.


Update mark assignment and related typical person class

Codecov Report

Merging #97 (9b0dbf7) into master (f9ee8a8) will increase coverage by 2.77%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #97      +/-   ##
============================================
+ Coverage     69.53%   72.31%   +2.77%
- Complexity      525      543      +18
============================================
  Files            87       87
  Lines          1694     1694
  Branches        185      185
============================================
+ Hits           1178     1225      +47
+ Misses          464      411      -53
- Partials         52       58       +6

Impacted Files
Coverage Δ

src/main/java/seedu/address/model/AddressBook.java
78.26% <ø> (+4.34%)
:arrow_up:

...rc/main/java/seedu/address/model/ModelManager.java
92.53% <0.00%> (+4.47%)
:arrow_up:

...address/model/assignment/UniqueAssignmentList.java
56.86% <0.00%> (+19.60%)
:arrow_up:

...ress/logic/parser/MarkAssignmentCommandParser.java
80.00% <0.00%> (+80.00%)
:arrow_up:

.../address/logic/commands/MarkAssignmentCommand.java
90.90% <0.00%> (+90.90%)
:arrow_up:

Continue to review full report at Codecov.

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


deployment within a VPC

Hi I'm attempting to get this plugin working within a VPC. Is there any way to deploy this plugin within a VPC? IS there any example of setup via AWS CDK?

There is an example cloudformation stack for the permissions at https://github.com/gocd/gocd-ecs-elastic-agent/blob/master/docs/installation.md#prerequisites however setting up an ECS cluster and networking etc is outside the scope of this plugin's documentation.


pAIs Play Music

About the PR
PAIs can play MIDIs on their own nokia-esque synths accessible through the action menu.
The spacestation14.sf2 soundfont has a new program on bank 1 called "paiSynth" that uses a cell phone beep sample, as there wasn't a consistently chiptuney midi instrument, at least in my default soundfont.
The pAI prototype now has an instrument that isn't accessible through interaction, and is instead opened up by the PAIMidi action that PAIs start out with.
Instruments now can also accept players that are the instrument owner itself.
In the future I'd like to reduce the range on these instruments, but I've been advised by Vera that to do this right now with the midi system would be somewhat complicated.
This is my first PR. Prepare for some facepalming code wise.
Screenshots
The only visual embellishment is the icon for the action.

Changelog
:cl:

add: pAIs have been outfitted with cheap ringtone synthesizers to play their own music.

would like to mention that you left the paiSynth sample (unused) in the SS14 soundfont file

Okay somehow this got mangled with an old, entirely separate branch i haven't touched for months. Let me remove all of this garbage.


12factor app doesn't work (gke, digitalocean)

First we had #521 . I tried to disable load balancing on gke (check the network settings when creating a cluster in the web ui) to see if that allows Epinio to work but after successfully pushing an application, the application route still gives a 502 Bad Gateway. There are some errors that might be relevant in the traefik pod:
...
│ traefik time="2021-06-22T12:28:58Z" level=error msg="Cannot create service: subset not found" ingress=factor12 namespace=workspace providerName=kubernetes serviceName=factor12 servicePort=8080                                           │
...
│ linkerd-proxy [  1002.635758s]  WARN ThreadId(01) server{orig_dst=<IP_ADDRESS>:8080}: linkerd_app_core::errors: Failed to proxy request: error trying to connect: Connection refused (os error 111) client.addr=<IP_ADDRESS>:55472                │

After fixing #521 maybe no longer need to disable load balancing and things just work. We need to test this.

It also needed #526 but even with both #526 and #521 in, the apps are still not accessible.

It turns out it has something to do with the app. 12factor doesn't work. The assets/sample-app in the Epinio repo and the example-wordpress both work. We need to debug the 12factor app.

The heroku sample app might not listen on IPv4 by default: https://github.com/epinio/epinio/issues/429#issuecomment-850487303

I think this was solved in slack but https://github.com/scf-samples/12factor does work.

Tested on GKE using the two install command approach and it worked with scf-samples 12 factor app.
Tried installing with the single command install on GKE but failed. So, created an issue in traefik repo https://github.com/traefik/traefik/issues/8227. Waiting for their reply

Created a PR to replace sample app with 12factor as requested. #541

@rohitsakala why replace the existing phpinfo app with such a big test case? Wasn't the story about adding it to the repo?

@manno Yes, I was confused about the story's acceptance criteria. Closed the PR

The traefik issue https://github.com/traefik/traefik/issues/8227 was moved to https://community.traefik.io/t/cant-get-external-ip-when-i-use-loadbalanceripin-gke/11000.
There is detailed thread there.
TLDR- I have to use regional static IP address to install epinio in one single command in GKE using loadBalancerIPfield in traefik service.
Also, created a new repo with 12factor app - https://github.com/epinio/example-12factor
Closing this issue now.


[KOGITO-5104] - RHPAM Kogito operator creates wrong default image tags

Many thanks for submitting your Pull Request :heart:!
https://issues.redhat.com/browse/KOGITO-5104
Please make sure your PR meets the following requirements:

[x] You have read the contributors' guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains a link to the JIRA issue
[x] Pull Request contains a description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[x] Your feature/bug fix has a unit test that verifies it
[x] You've tested the new feature/bug fix in an actual OpenShift cluster
[x] You've added a RELEASE_NOTES.md entry regarding this change

Please check OLM failures.


Fix focus scope restore losing focus

Closes
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

Go to DialogTrigger -> with Menu Trigger, use the keyboard to open the Menu, press tab, focus should be restore to the Menu Trigger button
🧢 Your Project:

Thanks @LFDanLu you can approve for me since I can't approve it
I tested in Chrome/Safari/FF


Ideally: Videos distributed directly to app

I can't quite work how we fix this..
The problem:

Some videos, and some content we don't want to be on a network such as youtube or s3. They are too private.
We need a way of sending the video directly to the client - but it not beings stored anywhere online.

No idea how to do this!

Well known solution for this issue is to use E2E (end to end) authentication & encryption. Using public-key cryptography.
This is what i think.

Using an existing E2E encrypted communication program.
We can integrate E2E within the app. And store the content encrypted at s3 or anywhere else. Since there's no way to send content over internet privately other than encrypting it. This would be fine.

For second option users should adopt a storage method for not locking themselves. (for example saving their private-key somewhere safe)
Though there's no absolute solution for implementing this fully secure as web app.
For first option you can simply use WhatsApp which has E2E integrated to it. Still their program is not open-source. You need to trust them to not run malicious code in their app. If you are following news, Five eyes demanded WhatsApp to do exactly that.

We can adopt pgp with email communication too.
It needs a bit of setup for users to get it running.


No meaningful status if gang-scheduled job has insufficient resources

Is this a BUG REPORT or FEATURE REQUEST?:
/kind feature
What happened:
When submitting a gang-scheduled job, if there are no resources available to schedule it there is no meaningful feedback to a user.
The podGroup has no events that they can't start.
The job says that it's running, with no errors or messages.
The pods themselves are created but not started.
The only place to find out that there's a scheduling error is by looking at the kube-batchd logs.
What you expected to happen:
A message/event either within the job or the podGroup stating that there are insufficient resources to schedule this job.
How to reproduce it (as minimally and precisely as possible):
Run the example job with a standard minikube start
Anything else we need to know?:
I'm probably going to be diving into this a lot more so Hi! :)
Environment:
master branch
minikube v0.30.0 on OSX
Jeffreys-MacBook-Pro:jeefy jeefy$ kubectl version
Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.2", GitCommit:"bb9ffb1654d4a729bb4cec18ff088eacc153c239", GitTreeState:"clean", BuildDate:"2018-08-07T23:17:28Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.1", GitCommit:"4ed3216f3ec431b140b1d899130a69fc671678f4", GitTreeState:"clean", BuildDate:"2018-10-05T16:36:14Z", GoVersion:"go1.10.4", Compiler:"gc", Platform:"linux/amd64"}

xref https://github.com/kubernetes-sigs/kube-batch/issues/401

When submitting a gang-scheduled job, if there are no resources available to schedule it there is no meaningful feedback to a user.

Which version are you testing? In master, we record an "Unschedulable" event if not enough resources for the job.

This is off of master. (Sorry this is long)
I'm following the directions in the Tutorial (more or less)
Dump of commands/output below
Set up environment
minikube start --kubernetes-version v1.12.1 --cpus 4 --memory 8192  && \
kubectl create serviceaccount tiller --namespace kube-system && \
kubectl create -f tiller-crb.yaml && \
helm init --service-account tiller --upgrade && \
sleep 30 && \
helm install $GOPATH/src/github.com/kubernetes-sigs/kube-batch/deployment/kube-batch --namespace kube-system

(Helm) output
Jeffreys-MacBook-Pro:jeefy jeefy$ helm status filled-umbrellabird
LAST DEPLOYED: Thu Oct 18 14:29:07 2018
NAMESPACE: kube-system
STATUS: DEPLOYED

RESOURCES:
==> v1beta1/CustomResourceDefinition
NAME                                   AGE
podgroups.scheduling.incubator.k8s.io  39s
queues.scheduling.incubator.k8s.io     35s

==> v1/Deployment
NAME        DESIRED  CURRENT  UP-TO-DATE  AVAILABLE  AGE
kube-batch  1        1        1           1          35s

==> v1/Pod(related)
NAME                         READY  STATUS   RESTARTS  AGE
kube-batch-659d977bb7-zxm55  1/1    Running  0         35s

NOTES:
The batch scheduler of Kubernetes.

Creating a job using the example job-01.yaml, however requiring "2000m" CPU
Jeffreys-MacBook-Pro:jeefy jeefy$ kubectl create -f job-01.yaml
job.batch/qj-1 created
podgroup.scheduling.incubator.k8s.io/qj-1 created

Describing the job and the podGroup
Jeffreys-MacBook-Pro:jeefy jeefy$ kubectl describe job qj-1
Name:           qj-1
Namespace:      default
Selector:       controller-uid=e507213b-d303-11e8-bc12-0800277a2afc
Labels:         controller-uid=e507213b-d303-11e8-bc12-0800277a2afc
                job-name=qj-1
Annotations:    <none>
Parallelism:    6
Completions:    6
Start Time:     Thu, 18 Oct 2018 14:30:31 -0400
Pods Statuses:  6 Running / 0 Succeeded / 0 Failed
Pod Template:
  Labels:       controller-uid=e507213b-d303-11e8-bc12-0800277a2afc
                job-name=qj-1
  Annotations:  scheduling.k8s.io/group-name=qj-1
  Containers:
   busybox:
    Image:      busybox
    Port:       <none>
    Host Port:  <none>
    Requests:
      cpu:        2
    Environment:  <none>
    Mounts:       <none>
  Volumes:        <none>
Events:
  Type    Reason            Age   From            Message
  ----    ------            ----  ----            -------
  Normal  SuccessfulCreate  15s   job-controller  Created pod: qj-1-sxnf4
  Normal  SuccessfulCreate  15s   job-controller  Created pod: qj-1-p6qrv
  Normal  SuccessfulCreate  15s   job-controller  Created pod: qj-1-gwzvz
  Normal  SuccessfulCreate  15s   job-controller  Created pod: qj-1-slwbq
  Normal  SuccessfulCreate  15s   job-controller  Created pod: qj-1-jqxln
  Normal  SuccessfulCreate  15s   job-controller  Created pod: qj-1-vzltr
Jeffreys-MacBook-Pro:jeefy jeefy$ kubectl describe podgroups.scheduling.incubator.k8s.io qj-1
Name:         qj-1
Namespace:    default
Labels:       <none>
Annotations:  <none>
API Version:  scheduling.incubator.k8s.io/v1alpha1
Kind:         PodGroup
Metadata:
  Creation Timestamp:  2018-10-18T18:30:31Z
  Generation:          1
  Resource Version:    613
  Self Link:           /apis/scheduling.incubator.k8s.io/v1alpha1/namespaces/default/podgroups/qj-1
  UID:                 e508c7b2-d303-11e8-bc12-0800277a2afc
Spec:
  Min Member:  6
Events:        <none>

If this is actually a bug let me know and I can dive in and try to figure it out. :) Also if I'm missing something, let me know!
Thanks!

The tutorial is using release-0.2 branch :) I used to handled this at https://github.com/kubernetes-sigs/kube-batch/pull/443 , please free feel to check if there's any bugs :)

My bad! You're right.
2379591517:kube-batch jeefy$ kubectl describe podgroups.scheduling.incubator.k8s.io qj-1
Name:         qj-1
Namespace:    default
Labels:       <none>
Annotations:  <none>
API Version:  scheduling.incubator.k8s.io/v1alpha1
Kind:         PodGroup
Metadata:
  Creation Timestamp:  2018-10-19T17:24:44Z
  Generation:          1
  Resource Version:    817
  Self Link:           /apis/scheduling.incubator.k8s.io/v1alpha1/namespaces/default/podgroups/qj-1
  UID:                 dec90dd3-d3c3-11e8-912c-080027dc9f66
Spec:
  Min Member:  6
Events:
  Type     Reason         Age   From        Message
  ----     ------         ----  ----        -------
  Warning  Unschedulable  6m    kube-batch  not enough resource for job

I wasn't going against master but v0.2.


Publish C hooks

Awesome project! Could C hooks be published, so that nonRust languages can take advantage of this accelerated library?
https://doc.rust-lang.org/1.5.0/book/rust-inside-other-languages.html

@mcandre Thank you for your comment. I do not know how much overhead occurs when a Rust function is called from other programming languages but it looks very nice if other languages could use Pikkr.

Agreed to this thread as well. I would be nice if pikkr can be used in C or C++.


⚠ Bofrost.lu has degraded performance

In 3768c1c, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 927 ms

Resolved: Bofrost.lu performance has improved in 44794d6 after 6 minutes.


[SPARK-24557][ML] ClusteringEvaluator support array input

What changes were proposed in this pull request?
ClusteringEvaluator support array input
How was this patch tested?
added tests

Test build #91828 has started for PR 21563 at commit b126bd4.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/4020/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/131/
Test PASSed.

Merged build finished. Test PASSed.

Test build #91828 has finished for PR 21563 at commit b126bd4.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91828/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/132/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/4021/
Test PASSed.

Test build #91830 has started for PR 21563 at commit d6e76e3.

Test build #91830 has finished for PR 21563 at commit d6e76e3.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91830/
Test FAILed.

Test build #91836 has started for PR 21563 at commit eb6022c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/4025/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/136/
Test PASSed.

Test build #91836 has finished for PR 21563 at commit eb6022c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91836/
Test PASSed.

Merged build finished. Test PASSed.

Test build #93586 has started for PR 21563 at commit 9064e7b.

@mgaido91 I am sorry to make a force push to update my git username in this PR.
Since I found that my current PRs are not linked to my account and it is troublesome to track them.
and I found that .expr.sql may not be a good way to get the output column name:
val df = Seq((1,2),(3,4),(1,1),(2,-1),(3,1)).toDF("a","b")
val z = (col("a") + col("b")).as("x").as("z")
scala> z.expr.sql
res15: String = (`a` + `b`) AS `x` AS `z`

So I still keep the orignal method.
@MLnick @mengxr @jkbradley Would you please help reveiwing this? Thanks!

Test build #93586 has finished for PR 21563 at commit 9064e7b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93586/
Test FAILed.

retest this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1546/
Test PASSed.

Merged build finished. Test PASSed.

Test build #93863 has started for PR 21563 at commit 9064e7b.

Test build #93863 has finished for PR 21563 at commit 9064e7b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93863/
Test PASSed.

@mengxr I notice that you open a ticket for supporting integer type labels in ClusteringEvalutator, would you like to shepherd this pr too?

LGTM. Merged into master. Thanks!


Window starts in background & CursorPosCallback.release crash in b41+ nightlies

Ever since b41 nightly, two problems have existed preventing me from staying up-to-date:

My game window starts in the background. If I use a build older than b41 (I think I might be on b37 or b38?), my game properly starts in the foreground. I can ALT+TAB to the background Window in b41-b44, but I shouldn't have to. What changed here?

This crash is new when closing my game -- doesn't happen with the previous builds:

java.lang.NoSuchMethodError: org.lwjgl.glfw.GLFWCursorPosCallback.release()V
at com.jme3.input.lwjgl.GlfwMouseInput.destroy(GlfwMouseInput.java:196)
at com.jme3.app.Application.destroyInput(Application.java:725)
at com.jme3.app.Application.destroy(Application.java:746)
at jmevr.app.VRApplication.destroy(VRApplication.java:633)
at Game.Main.destroy(Main.java:229)
at Game.Menu.MenuSystem.ActivateSelectedOption(MenuSystem.java:1069)
at Game.Input.Manager$2.onAction(Manager.java:423)
at com.jme3.input.InputManager.invokeActions(InputManager.java:169)
at com.jme3.input.InputManager.onMouseButtonEventQueued(InputManager.java:439)
at com.jme3.input.InputManager.processQueue(InputManager.java:857)
at com.jme3.input.InputManager.update(InputManager.java:907)
at com.jme3.app.Application.update(Application.java:712)
at jmevr.app.VRApplication.update(VRApplication.java:543)
at com.jme3.system.lwjgl.LwjglWindow.runLoop(LwjglWindow.java:380)
at com.jme3.system.lwjgl.LwjglWindow.run(LwjglWindow.java:464)
at com.jme3.system.lwjgl.LwjglWindow.create(LwjglWindow.java:307)
at com.jme3.app.Application.start(Application.java:446)
at jmevr.app.VRApplication.start(VRApplication.java:310)
at Game.Main.StartGame(Main.java:212)
at Game.Main.main(Main.java:134)

GLFWCursorPosCallback.release() is now GLFWCursorPosCallback.free() in the nightly build. Not sure about the window opening in the background though.

My game window starts in the background. If I use a build older than b41 (I think I might be on b37 or b38?), my game properly starts in the foreground. I can ALT+TAB to the background Window in b41-b44, but I shouldn't have to. What changed here?

This is likely related to the new glfwFocusWindow functionality in GLFW master. What OS are you on?

Windows 10 x64. This something that has to be newly called somewhere?

Calling glfwFocusWindow after glfwShowWindow does the trick. Changing .release to .free also fixes the other problem. Should be all set, thank you!

Lets leave this issue open, because I'm not sure what the correct behavior is. This statement (in the glfwFocusWindow doc):

By default, both windowed and full screen mode windows are focused when initially created. Set the GLFW_FOCUSED hint to disable this behavior.

and this (from the commit above):

This removes the (undocumented) behavior where glfwShowWindow would bring the window to front and set input focus.

seem to contradict each other.

@phr00t Is your window full screen or windowed?

Windowed.

@Spasi Yup, this is totally my fault. It's a regression I was hoping to get away with to introduce separation of behavior. If you prefer I'll revert to the old behavior (though still using the newer code).

Ideally it would work like the documentation on glfwFocusWindow says: new windows are focused by default (same behavior as before), unless glfwWindowHint(GLFW_FOCUSED, GLFW_FALSE) is set.

(Due to headache I'm probably saying at least some obvious things below. Sorry.)

By default, both windowed and full screen mode windows are focused when initially created.

This was poorly written. What I was trying to say with "initially created" was that this is done by glfwCreateWindow or not at all. If one turns off GLFW_VISIBLE (which appears to be what @phr00t did, hence the call to glfwShowWindow) then the default behavior of show plus focus is overridden.
Should I track whether the window has ever been shown and then additionally focus it the first time it is shown?

If I'm not mistaken, I set GLFW_VISIBLE to false, because the documentation says you should if you want to reposition it (which I was doing). I later didn't show it, like I should have. Previously, it showed anyway, so I didn't think anything of it. I think the current behavior is more correct, if it is following @Spasi's description of the documentation.

What I was trying to say with "initially created" was that this is done by glfwCreateWindow or not at all.

Thanks, that explains it.

Should I track whether the window has ever been shown and then additionally focus it the first time it is shown?

I think I would prefer it. It also doesn't change the behavior of existing programs.

The previous behavior has been restored with glfw/glfw@89f438de5a862802717a5d6e32ebfe462775cccf.


Add a way to delete a wiki from the in-wiki interface.

Just be very careful that it won't happen accidentally.

Just a comment, a thought, not support for the idea, or against it.
The one thing that stands out for me with Bob is you can get Bob to talk directly to the OS. I mean that is its brilliance. So, in my case, some of the "features" requested I don't need as I can myself develop them because Bob can do that. Hope this is clear and maybe relevant.

This is included


Add set_permissions argument

This adds an option to extract a tarball without using any of the file permissions in the tarball.
I primarily need this on Windows so that I can extract a tarball and have all the extracted files just inherit permissions from the folder where I extract them to.
If this PR is acceptable I'll add a test and docs.

The name same_permissions doesn't really suggest much of a meaning to me. It seems like this doesn't even try to set any kind of executable permissions on files that are supposed to be executable?

The name same_permissions doesn't really suggest much of a meaning to me.

I just copied that name from the original tar command, where that is a command line flag that has the same effect as the flag in this PR, AFAIK. Happy to use a better suggestion if someone has one :)

It seems like this doesn't even try to set any kind of executable permissions on files that are supposed to be executable?

Yes, exactly, this just does not touch file ACLs at all, which is probably what one wants in the vast majority of cases on Windows. Permissions on Windows are in almost all cases handled at the container level and then inherited, and not set on a per-file basis. For example I'm not aware of a single installer technique on Windows - by MS or anyone else-  that is setting individual file level permissions, permissions are always handled on some parent folder that constitutes some sort of security boundary and then inherited to all files in that folder. The equivalent of the executable flag that signals that something can be run is really the file extension on Windows.
I need a way to extract files from a tarball for https://github.com/julialang/juliaup that just does not set any file permissions on the extracted files, but has all the extracted files inherit file permissions from their parent folder.

How about calling this set_permissions then?

Yes, set_permissions is much nicer. I changed that, and also added a (pretty simplistic) test that should exercise the new code path and added some doc language.

@StefanKarpinski if we manage to merge this before Tue it might still make it into Julia 1.7, right?

Yeah, we can sneak this in. Looks good to me if you add tests and docs.

I added a test. It is not super sophisticated, but maybe good enough? Kind of tricky to write a test that really tests permissions properly, because the new behavior is that it depends on the permissions of the parent and that depends on the system on which this is running... But the test I added does exercise the code path.
Docs are also there.

Btw, once this gets merged I'll include it in Tar 1.10.0 and bump it on Julia master. I assume you don't need this on Julia 1.6.x — I don't think we can do that since this is clearly a feature that expands the API of the package.

I've made the set_permissions tests run on all platforms since this is not a Windows-specific feature — it's available on all platforms and needs to be tested along with the rest of the Tar.extract API.

Update will be included in Julia 1.7 once this PR is merged: https://github.com/JuliaLang/julia/pull/41129.


Use directly in html?

Is there a way to try this directly in html?
I've tried by adding   <script src="//cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>but I get the error: "Uncaught SyntaxError: Cannot use import statement outside a module" if I add type="module" to the script tag, then I get a different error: "Uncaught TypeError: Failed to resolve module specifier "vue". Relative references must start with either "/", "./", or "../"."
I'm guessing this isn't bundled to be used directly in html?

To use it in HTML without type="module", you need to use the global build (like with Vue) and I'm realising both, vue and vue-router are not correctly connected to the CDN. For vue-router you need to use the vue-router.global.js (same as vue)
<EMAIL_ADDRESS>
(adapt the version)
What I'm unsure is what should be the version pointed by jsdelivr by default, we can change it with jsdelivr in the package.json file and should be consistent. Previously, in vue, we would expose the minified version, that would be global.prod instead global. Let's keep this open for discussion

Hey this is great - thanks - will play around with it now 😄


Refactor and centralize configuration handling

This refactors how we handle configuration options to allow all options to be specified by CLI args, environment variables, or manifest file options. It replaces Opt with a new class Config in the utils package with the idea that we can remove calls to setProperty from the codebase with references to variables under Config, similar to Opt but also handling the setting of the values in addition to the reading of them.
There are a few things which I didn't have time to do:

[ ] remove all calls to setProperty. The remaining ones are mostly in the junit integration.
[ ] switch ConfigItem to use Optional instead of needing to check isSet
[ ] we discussed further centralizing the places that modify properties to allow Config to set properties once and then never allow them to change
[ ] the changes in getLibFuzzerSeed need to be removed because the api package should have have any dependency on the rest of jazzer
[ ] Opt, OptParser, and OptTest all still exist but currently only reference each other
[ ] ConfigItem.StrList does not handle escaping on the strings (see OptTest for the relevant code bits)
[ ] ConfigItemTest does not test each type to ensure they're serialized and deserialized as expected
[ ] I created but did not implement a test in ConfigTest to make sure that loading from a manifest file works. Though this might be unnecessary as other tests will use manifest files to set configuration options
[ ] I left some useless code commented out that should be removed

Other notes:

FuzzTargetRunner must only be loaded after a call to Config.loadConfig. This appears to be case currently but it would be nice to make this more explicit if possible.

Closing in favor of #767


windows error 500 version 2020.02.01

I want to know how to fix the error 500.
I try compile code but it still gets error 500.

i can fix it thank us