poetry breaks if config.toml is invalid

[settings]

[repositories]
artifactory = {url = """
http://<snip>/pypi/pypi-virtual/simple
"""}

Traceback (most recent call last):
  File "c:\<snip>\python\python36\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "c:\<snip>\python\python36\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\<snip>\Python\Python36\Scripts\poetry.exe\__main__.py", line 9, in <module>
  File "c:\<snip>\python\python36\lib\site-packages\poetry\console\__init__.py", line 5, in main
    return Application().run()
  File "c:\<snip>\python\python36\lib\site-packages\poetry\console\application.py", line 45, in __init__
    super(Application, self).__init__("Poetry", __version__)
  File "c:\<snip>\python\python36\lib\site-packages\cleo\application.py", line 66, in __init__
    for command in self.get_default_commands():
  File "c:\<snip>\python\python36\lib\site-packages\poetry\console\application.py", line 111, in get_default_commands
    ConfigCommand(),
  File "c:\<snip>\python\python36\lib\site-packages\poetry\console\commands\config.py", line 43, in __init__
    self._config = Config.create("config.toml")
  File "c:\<snip>\python\python36\lib\site-packages\poetry\config.py", line 92, in create
    return cls(file)
  File "c:\<snip>\python\python36\lib\site-packages\poetry\config.py", line 16, in __init__
    self._raw_content = file.read(raw=True)
  File "c:\<snip>\python\python36\lib\site-packages\poetry\utils\toml_file.py", line 22, in read
    return toml.loads(f.read())
  File "c:\<snip>\python\python36\lib\site-packages\toml.py", line 373, in loads
    multibackslash)
  File "c:\<snip>\python\python36\lib\site-packages\toml.py", line 479, in _load_line
    value, vtype = _load_value(pair[1], _dict, strictly_valid)
  File "c:\<snip>\python\python36\lib\site-packages\toml.py", line 639, in _load_value
    _load_inline_object(v, inline_object, _dict)
  File "c:\<snip>\python\python36\lib\site-packages\toml.py", line 399, in _load_inline_object
    candidate_groups[0] = candidate_group + "," + candidate_groups[0]
IndexError: list index out of range

I ran poetry config repositories.artifactory "<the url>" and somehow it encoded the config.toml file in a way that couldn't be reparsed.
I am unable to unset the key, and no indication was given to the actual filepath of config.toml. I've tried reinstalling poetry (i guess to reset the files or something) and nothing other than actually editing config.toml manually will fix it.
I don't know what the solution would be, if it would be better to delete the toml, ignore the config, or exit with a helpful warning. In the meantime:
    def read(self, raw=False):  # type: (bool) -> dict
        with self._path.open() as f:
            if raw:
                try:
                    return toml.loads(f.read())
                except Exception e:
                    raise ParseError('Could not parse config.toml') from e

            return loads(f.read())

Edit: I should probably add my environment.

Windows 10
Python 3.6.5 (x64 64-bit)
pip 10.0.1
poetry 0.10.3

Thanks for reporting this!
While there is definitely a bug when adding a repository like you mentionned, I also agree that the config should be smarter in how it handles error and check the  validity of the TOML file before writing it back

Idk if this should go in a new issue or not but I figured out that it's writing an invalid toml file because of backslashes. (apparently I accidentally fudged a backslash while trying to press enter when typing the example)
[settings]

[repositories]
oopsies = {url = "http://google.com/\"}
oopsies_two = {url = """
https://cas.pacific.edu/cas/login?service=https%3A%2F%2Flum-prod.ec.pacific.edu%2Fc%2Fportal%2Flogin\\
"""}

$ poetry config repositories.oopsies "http://google.com/\"
$ poetry config --all
# python stack trace...

$ poetry config repositories.oopsies_two "https://cas.pacific.edu/cas/login?service=https%3A%2F%2Flum-prod.ec.pacific.edu%2Fc%2Fportal%2Flogin\"
$ poetry config --all
# python stack trace...

I haven't really bothered to check the location where it's writing the toml, but if it's writing using toml.dumps() (or whatever it uses, I haven't checked), then it's a problem with the toml package.

This should be fixed in the latest release with the introduction of a new TOML library: TOML Kit


Euler’s Totient function for all numbers smaller than or equal to n

fix #2549
Added Euler’s Totient function for all numbers smaller than or equal to n

@sairish2001 resloved merge conflicts

@sairish2001 Kindly review

@sairish2001


Hide expressions not being re-evaluated for fields in a nested formly-form

Please look at the bin link to js bin.
All the fields are sharing a model. However when the value of the model changes, in some cases the fields hide expressions are not re-evaluated based on this update.
See bin for steps to reproduce.

Well as to my knowledge, there are 2 separate watchers set up for hideExpressions - one on the global model, the one that you have firstName defined on and other for the subform and it watches only vm.model.address object. That said, only that second watcher fires hideExpressions on it's form so when you modify firstName the child form is not aware of that and does not fire any expressions.
You kind of hacked the hideExpression to use vm.model from the closure but you should probably only use arguments that hideExpression provides - then you would see that you cannot access firstName anyhow because your model is only vm.model.address.
You can however set up manual watcher on the child form and it will run the expressions when it changes:
{
  watcher: {
    expression: function() {
      return vm.model.firstName;
    },
    runFieldExpressions: true
  }
}

Thanks so much for the response. So if a field needs to know when several fields outside its form changes, then I have to set-up watchers for each of the other fields similar to your example?

Yes, other than that you can specify not to use model watcher (which is expensive if your models weight a lot) and use form option watchAllExpressions and set it to true which will evaluate all expressions on each digest cycle. If you chose that, you can also try to disable the automatic model watcher which would be most likely redundant (set form option manualModelWatcher to true). That should do the trick also :)

@kwypchlo thanks for such a nice explanation. One question though, where do I set watchAllExpressions to true and likewise manualModelWatcher

@mhawila use options parameter documented in here http://docs.angular-formly.com/docs/formly-form


[BUG] Fields wrapping doesnt work on multiple lines

Description of the bug
Fields wrapping in FormattersHuman does not work on multiple lines.
Screenshot of bug


Add error handling for create pull request

When the title or description is empty, a nice toast is displayed. Moreover, if the server returned an error, the message is parsed and displayed.
Fixes #213
Fixes #187
Requires https://github.com/preview-code/backend/pull/11

Works good, maybe we want to work on the design.
Give a red border to the thing that are missing, but for now I say this is good enough.


Make a bin available

Putting it in scripts section of package.json to update the tsconfig.json (mainly interested in this because of filesGlob)
right now I'm using node.repl to do this

Can you go into more detail on what you mean by, make a bin available?  I'm having trouble understanding exactly what it is you are looking for with this request.
Can you show what you are doing with node.repl to achieve your goals currently?

filesGlob isn't 'natively' supported by tsc currently. tsconfig, atomscript, sublime tsscript support it. When working outside a text editor that doesn't have either of those plugins installed, the files inside tsconfig.json never gets updated.
I'm using node.repl to emulate a command line derived from tsconfig:
> var tsconfig = require('tsconfig');
undefined
> var fs = require('fs');
undefined
> fs.writeFileSync('tsconfig.json', JSON.stringify(tsconfig.getProjectSync('.').project));
undefined

by having it install to the .bin folder, I could place it in package.json as:
{
   "scripts": {
      "tsconfig": "tsconfig --save"
   }
}

type npm tsconfig and it would update the tsconfig.json file for me

While I don't believe this would be hard to do, if your only use case is for tsconfig --save then I am not sure how much work it would be worth since (hopefully) file pattern matching will be added to TSC soon: https://github.com/Microsoft/TypeScript/pull/3232

This module no longer expands filesGlob (as of 4.0) and includes will be an official option in 2.0. I never condoned the use filesGlob personally, but do know people relied on it. However, staying in sync with official TypeScript releases and other clients is more important.


added dispatcher prop to global Flux object

This is a pretty simple PR, but I'll let you decide as to whether or not it's a good idea or not. I've been using ReactFlux in an application of mine, and I've needed to reference the raw Dispatcher that ReactFlux is using internally.  In order to do this, I did a hack where I made a module reference the _dispatcher property of an unused store.
I rather like the fact that the "dispatcher" of this library is intentionally left as an implementation detail, but nevertheless, there was  scenario where I needed this.  As a result, I made this PR in case anyone else had the same needs.
If you'd like to keep this out as part of the public API, though, I completely understand. Let me know if there's any more you'd like to talk about regarding this specific change.
Thanks!

Leland

@kjda wondering if you saw this or not? let me know if you have any thoughts. Thanks.


deprecate R.substring

R.slice dispatches to slice, so R.substring should be deprecated just as R.substringTo and R.substringFrom were in #1159.

This is very nice!

I'm still very wary of the API via implementation progression we seem to be on.
The main question should be whether we want to have a public substring function.  If we do, then we should support a function with that name, even if its functionality overlaps with that of another one, even it it's entirely subsumed by slice.
In other words, as I think I've written elsewhere.  the API should always remain king.

It'd be nice if our deprecations would say what the new solution is where applicable. I didn't assume that R.drop would work on strings.

It'd be nice if our deprecations would say what the new solution is where applicable.

Good idea. I'm sure we'd all be happy to see a pull request against ramda/ramda.github.io. :)

But .in the future, we really should add a @see tag when we deprecate if the equivalent function is available, or some other similar mechanism in the comments

Just putting a formal objection here. If I want to do a substring operation then calling substring just feels much more natural then slice to me. I find it a whole lot more expressive. In this case especially the names substringFrom and substingTo have just been much more clear than in any other lib or language. I value that much more than what could ever be the benefit of this.
A lot of these changes are going on and I just have a hard time understanding why. What is unclear to me is the actual intention of these changes. Is it to not have any two functions that can achieve the same goal? If that is correct then I suspect the ramda API could be reduced to about 10 functions depending on how you look at it.

The goal is to avoid having two names for the same function, but you're free to define aliases if like:
R.substring = R.slice;
R.substringTo = R.take;
R.substringFrom = R.drop;

It might be possible to implement these functions in the same way but from the outside one set works on strings and the other on arrays. slice, take, drop belongs to the List category. This to me is not "API is king".

@TheLudd It's odd. I feel the exact opposite way. To me a string is just a list of characters. Using slice, take and drop on lists of characters feel very natural to me. On the other hand having to use functions that embed the word "string" in their name feels unnatural. Why do I have to use a string specific function when I'm doing a general list operation on a list of characters?
I'm not saying that Ramda should accept strings in all the list functions. But doing it in the most obvious candidates feels like a better API to me.

@paldepind
A string can be viewed as a list of characters. I can also be viewed as an object. I am pretty sure that a list can be viewed as an object as well. Claiming that this is how we view these types when we do programming feels a bit forced to me. Here is a list of characters: [ 'f', 'o', 'o' ]. Is that interchangeable with 'foo' ?
While I don't share your opinion that a string is just a list of characters, I won't claim that you are wrong. But I doubt anyone can disagree with the claim that at string is a string.
Technically you are correct but I think the difference in concepts is important and and function names that clearly express their purpose just makes code more easy to read.

In dynamically typed languages we sometimes use function names, parameter names, and variable names to provide hints as to the types of the values in our programs.
The following two examples produce equivalent functions:
var getAccountPrefixes = function(accountNumbers) {
  return map(function(accountNumber) {
    return substringTo(4, accountNumber);
  }, accountNumbers);
};

//  prefixes :: Functor f => f a -> f a
var prefixes = map(take(4));

The difference is the number of hints these functions provide as to the types we're expecting. The problem with using names such as accountNumbers and substringTo is they suggest that our function is more specialized than it really is. We could use getAccountPrefixes on a list of lists of Date values, but it would be confusing to do so. Giving things descriptive names limits code reuse.

@TheLudd
I think @davidchambers said it very well. Functional programming is very much about identifying common properties in different type of things. In some languages a string really is just a list of characters. We also have [] notation for accessing characters in in strings which is also evidence that people like to think of them as lists of characters.
I think the only reason why a string in JavaScript is not just an array of characters is that arrays are mutable where as strings are immutable. But that is a technical difference and it is one that doesn't apply to Ramda.


What is the relationship with inspinia admin?

https://wrapbootstrap.com/theme/inspinia-responsive-admin-theme-WB0R5L90S
It's very similar. is inspinia admin based on gentelella? or the opposite?
Is it ok to use gentelella in commercial site?

Color scheme for both ar similar but thats where the similarities ends. INspinia is far more advanced with dozens of libraries included and also they have angular, ruby on rails verisons and much more. This is a plain HTML template and a very simple one. If simplicity is the thing you are looking for then go with this. If Angular and other owesome thins is for you then go with Inspinia.
Now that I have taken over development for this template you can finally expect new library introduction, bug fixes and other updates.

Thank you very much. Seems Inspinia is not free, although I can find a MIT licensed Inspinia in github, it confuses me.
And gentelella is also MIT license. As long as there's no license issue, i will go with gentelella.
Thanks for your efforts :)

Templates can be licensed under MIT and sold, that's no a problem at all. Because support is what you get when going for a paid version.


mono: convert unicode mono root path to ascii string

Fixes #19739
scons was choking when the environment variables where unicode strings, this will convert compatible paths to ascii or throw an exception.

I added the fix I forgot to push as part of #16987: https://github.com/godotengine/godot/pull/16987/files#diff-654ea5945e2faa32f3a0611a7abd1c8dL138

Both proposed fixes are hacky, but I prefer your hack :)

Do you think this change is better? I don't think it's possible for the mono root to contain non-ascii characters, since we get it from the environment variables or the registry.

I'll be honest the direct cast looks much more elegant. (Is it required for msbuild_tools_path as well?)
However it is possible for windows file systems to use unicode paths, and the registry will store and return them if required. I don't know how the python libraries handle this.
https://docs.microsoft.com/en-gb/windows/desktop/Intl/character-sets-used-in-file-names
https://docs.microsoft.com/en-gb/windows/desktop/SysInfo/registry-value-types

@KellyThomas Is this problem only caused because of the return value from find_mono_root_dir? If that's the case, it may be better to add the fix there instead (surround the return with str).
To clarify, this error is only present in Python 2.7. I could not reproduce it with Python 3.

I don't think so. msbuild_tools_path is not used anywhere in environment variables.

OK, I've modified this to cast before return from find_mono_root_dir()
Testing shows if mono is installed at a location containing unicode characters the build fails with a message like this:
RuntimeError: Could not find mono library in: C:\Program Files\Mono?\lib

Thanks!

Cherry picked into 3.0.6


Create 0752-open-the-lock.js

File(s) Modified: 0752-open-the-lock.js, README.md
Language(s) Used: javascript
Submission URL: https://leetcode.com/problems/open-the-lock/submissions/1079021005/

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

@sujal-goswami just removed the readme.md change, feel free to review this as well, sorry for the pressure

No need to tag me for every PRs.

@KwFung7 Thanks for your contribution 👍


Finetuning a model twice in a row breaks.

I'm doing experiments with spelling errors. I have a model trained but I want to finetune against augmented data that contains spelling errors. Doing this the first time around works fine.
> rasa train nlu --nlu data/nlu-train-spelling.yml --finetune --epoch-fraction 0.1 --fixed-model-name spelling-epoch-1
Training NLU model...

When I now want to take this finetuned model and finetune it some more ... stuff breaks.
> rasa train nlu --nlu data/nlu-train-spelling.yml --finetune --epoch-fraction 0.1 --fixed-model-name spelling-epoch-1
Training NLU model...
2021-06-28 13:32:29 WARNING  rasa.shared.utils.common  - The Incremental Training feature is currently experimental and might change or be removed in the future 🔬 Please share your feedback on it in the forum (https://forum.rasa.com) to help us make this feature ready for production.
Loading NLU model from models/spelling-epoch-1.tar.gz for finetuning...
2021-06-28 13:32:37 INFO     rasa.shared.nlu.training_data.training_data  - Training data stats:
2021-06-28 13:32:37 INFO     rasa.shared.nlu.training_data.training_data  - Number of intent examples: 15878 (151 distinct intents)

...

2021-06-28 13:32:37 INFO     rasa.shared.nlu.training_data.training_data  - Number of response examples: 0 (0 distinct responses)
2021-06-28 13:32:37 INFO     rasa.shared.nlu.training_data.training_data  - Number of entity examples: 0 (0 distinct entities)
2021-06-28 13:32:37 INFO     rasa.nlu.model  - Starting to train component WhitespaceTokenizer
2021-06-28 13:32:38 INFO     rasa.nlu.model  - Finished training component.
2021-06-28 13:32:38 INFO     rasa.nlu.model  - Starting to train component RegexFeaturizer
2021-06-28 13:32:38 INFO     rasa.nlu.model  - Finished training component.
2021-06-28 13:32:38 INFO     rasa.nlu.model  - Starting to train component LexicalSyntacticFeaturizer
2021-06-28 13:32:42 INFO     rasa.nlu.model  - Finished training component.
2021-06-28 13:32:42 INFO     rasa.nlu.model  - Starting to train component CountVectorsFeaturizer
/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/shared/utils/io.py:97: UserWarning: New data contains vocabulary of size 21830 for attribute text which is larger than the maximum vocabulary size (8911) of the original model. Some tokens will have to be dropped in order to continue training. It is advised to re-train the model from scratch on the complete data.
Traceback (most recent call last):
  File "/home/vincent/Development/taipo/venv/bin/rasa", line 8, in <module>
    sys.exit(main())
  File "/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/__main__.py", line 117, in main
    cmdline_arguments.func(cmdline_arguments)
  File "/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/cli/train.py", line 206, in run_nlu_training
    finetuning_epoch_fraction=args.epoch_fraction,
  File "/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/model_training.py", line 658, in train_nlu
    finetuning_epoch_fraction=finetuning_epoch_fraction,
  File "/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/utils/common.py", line 296, in run_in_loop
    result = loop.run_until_complete(f)
  File "uvloop/loop.pyx", line 1456, in uvloop.loop.Loop.run_until_complete
  File "/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/model_training.py", line 705, in train_nlu_async
    finetuning_epoch_fraction=finetuning_epoch_fraction,
  File "/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/model_training.py", line 766, in _train_nlu_with_validated_data
    **additional_arguments,
  File "/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/nlu/train.py", line 116, in train
    interpreter = trainer.train(training_data, **kwargs)
  File "/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/nlu/model.py", line 219, in train
    component.train(working_data, self.config, **context)
  File "/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/nlu/featurizers/sparse_featurizer/count_vectors_featurizer.py", line 735, in train
    self._train_with_independent_vocab(attribute_texts)
  File "/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/nlu/featurizers/sparse_featurizer/count_vectors_featurizer.py", line 552, in _train_with_independent_vocab
    self._fit_loaded_vectorizer(attribute, attribute_texts[attribute])
  File "/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/nlu/featurizers/sparse_featurizer/count_vectors_featurizer.py", line 593, in _fit_loaded_vectorizer
    self._update_vectorizer_vocabulary(attribute, new_vocabulary)
  File "/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/nlu/featurizers/sparse_featurizer/count_vectors_featurizer.py", line 387, in _update_vectorizer_vocabulary
    self._merge_new_vocabulary_tokens(existing_vocabulary, new_vocabulary)
  File "/home/vincent/Development/taipo/venv/lib/python3.7/site-packages/rasa/nlu/featurizers/sparse_featurizer/count_vectors_featurizer.py", line 397, in _merge_new_vocabulary_tokens
    del existing_vocabulary[f"{BUFFER_SLOTS_PREFIX}{available_empty_index}"]
KeyError: 'buf_8911'

These are my Rasa versions.
Rasa Version      :         2.6.0
Minimum Compatible Version: 2.6.0
Rasa SDK Version  :         2.7.0
Rasa X Version    :         None
Python Version    :         3.7.9
Operating System  :         Linux-5.11.0-7614-generic-x86_64-with-Pop-20.10-groovy
Python Path       :         /home/vincent/Development/taipo/venv/bin/python

It looks like the error is related to the extra buffers (KeyError: 'buf_8911') that we were creating in order to account for new sparse features that appeared during fine-tuning.
We have replaced extra buffers with dynamic feature allocation since 2.8.0 which means this shouldn't be relevant anymore and we can close it


Migrate to aws-sdk-go-v2

Part of https://github.com/iterative/terraform-provider-iterative/issues/185

@0x2b3bfa0
Why do you not include this work inside https://github.com/iterative/terraform-provider-iterative/pull/187?
It depends on this and the reason the create this PR is to be able to support https://github.com/iterative/terraform-provider-iterative/pull/187, right?

Yes, I separated it just to keep two commits after squash+merge and be able to blame better in the future.

https://github.com/iterative/terraform-provider-iterative/pull/187#issuecomment-916794514


Fixes Rclone Docker Tests

This PR fixes setup_paths.py, so that rclone docker tests now pass.

Looks like this is redundant/conflicting with #1070
@h-mayorquin, can you take a look?

Yes, the other one has a fixed in the setup.py that also needs to go through, it also fixes some other errors in the setup_paths.py.
I would like to merge https://github.com/catalystneuro/neuroconv/pull/1070  first.

Looks like #1070 fixed the docker rclone tests, so this PR is redundant: https://github.com/catalystneuro/neuroconv/actions/runs/10852452995/job/30118525526


Fix auto-reconnect in case of WS handshake failure

When using the component API with asyncio, the client component doesn't try to reconnect after a failing WS handshake, as pointed out in issues #1010 and #1030.
The issue is caused by some naming mismatch between the twisted and asyncio implementations of the WebSocketAdapterProtocol class. In particular the wamp component (wamp/component.py) tries to wrap the connectionLost() method of the underlying protocol, but the asyncio implementation only has a connection_lost() method.
Curiously enough, the affected code path doesn't seem to be invoked when using the twisted counterpart.
This PR aims at fixing these issues, enabling the component to proceed with the usual exponential backoff retry mechanism when failing to properly connect to the other endpoint.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9b6fb57). Click here to learn what that means.
The diff coverage is 13.33%.

@@            Coverage Diff            @@
##             master    #1034   +/-   ##
=========================================
  Coverage          ?   61.23%
=========================================
  Files             ?       74
  Lines             ?    13152
  Branches          ?     2091
=========================================
  Hits              ?     8053
  Misses            ?     4396
  Partials          ?      703

Impacted Files
Coverage Δ

autobahn/asyncio/component.py
0% <0%> (ø)

autobahn/asyncio/websocket.py
47.2% <50%> (ø)

autobahn/wamp/component.py
48.59% <50%> (ø)

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 9b6fb57...3681b5c. Read the comment docs.

No, this is mixing things up. The Twisted implemention is here https://github.com/crossbario/autobahn-python/blob/master/autobahn/twisted/component.py

@oberstet can I ask you more details about why "this is mixing things up", and the reason for closing the PR without any further comment?
This PR has never been about the Twisted component implementation, which indeed I didn't touch. It is about its asyncio counterpart, and its integration with the wamp component.
Issues #1010 and #1030 are real: even in a simple router restart scenario, a reconnection is never retried more than once from an asyncio based client, which is kind of a showstopper for any production application based on asyncio and this module.
I have no problem with the fact that this PR is not being accepted as it is, however I think the underlying issue is quite critical and still requires a solution.


Magnetometer data scale incorrect

After a quick review for an unrelated possible issue, running the demo shows readings of 10,000Gs+ after setting bias offsets, where other sensors read near 0. Review MagScale() scale factors.

fixed in abb3d6fd370a945cc82000e7b171b812dc98a73b


OK Debugger: Python Tutor Tracing

This PR brings in a debugger into OK (with the help of a slightly modified version of @pgbovine's Python Tutor)
Usage:

Doctests: python3 ok -q q1 --trace
Ok Tests:  python3 ok -q q1 --trace --suite 3

I haven't really tried this on projects yet.

Update: It works for project too but there's a little bit of work todo in hiding irrelevant information.

@epai - Ping on merging this? I don't think any of my comments are blockers.

I think I can work on merging this, it looks like a useful feature! @epai @Sumukh any objections?

No objections on my end, I'd suggest giving a quick trial run

Some tasks before merge:

[ ] this shows everything in the global frame even irrelevant things.
[ ] there were some issues with the old version of pythontutor, and this seems to use that? @rahularya50 can you elaborate?

@Sumukh can you take a look at my changes? I removed unused global variables, using the ast_scope module I've been developing for the last month or so (https://github.com/kavigupta/ast_scope)

That’s awesome! I’ll take a look soon
On Sun, Dec 22, 2019 at 4:36 AM Kavi Gupta<EMAIL_ADDRESS>wrote:

@Sumukh https://github.com/Sumukh can you take a look at my changes? I
removed unused global variables, using the ast_scope module I've been
developing for the last month or so (
https://github.com/kavigupta/ast_scope)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/okpy/ok-client/pull/301?email_source=notifications&email_token=AAGXNTKNFMJB54RNTUKWDVTQZ5NN7A5CNFSM4DKMEPN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHPO6GY#issuecomment-568258331,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGXNTIDJN4KALOE7PSMQE3QZ5NN7ANCNFSM4DKMEPNQ
.

@kavigupta Your changes overall looks good to me. I'm a bit concerned about what including networkx does to the ok client bundle size but maybe the impact is smaller than I would imagine

oh wow it goes from 2.0MB to 6.4MB.
I think I'll change ast_scope to remove the networkx dependency, we're really only using it for a DFS

@Sumukh removed the dependency on networkx

Nice! I haven't really tried the actual result yet but feel free to merge & deploy!

ok sounds good!


Add .*config file for further settings

Add .*config file
More options for the checker, e.g. similarity threshold

Record here to keep in track.

CSS-Checker now read css-checker.yaml for settings, you can also specify the config file path with -config=YOUR_CONFIG_FILE_PATH. Refer to: 70f00266a3122be3d38298503c66296b5ab26b2c


 How can I run the client? For example, I'm using debian so when I run in the command-line like: python2 client.py 'Example::echo'. The client can't find object reference. In serve-side is working. But in the client-side isnt.
I'm running the same example that you.
I know is something tha I typing wrong in the command-line.
error: omniORB.CORBA.BAD_PARAM: CORBA.BAD_PARAM(omniORB.BAD_PARAM_BadSchemeName
Thanks for now,

Sounds like a problem with the IOR you provide to the client. Can you check it here:
http://catior.org/cgi-bin/catior


Validation does not check consistency of Phrases

Moonscraper allows setting Global Events called phrase_start and phrase_end to signal the start and end of a line of song text. Clone Hero does not recognize or display song text, if open phrase_start events are not closed before opening a new one.
Moonscraper should check the consistency of these events when saving and display an error, if:

the number of phrase_start and phrase_end events aren't equal
no matching phrase_start is placed before a phrase_end event
no matching phrase_end is placed after a phrase_start event
there are multiple global phrase_start or phrase_end events at the same position

Testing is not set up in the Unity project, that's why I'm going to make a list of manual testing steps here:
Saving the chart should show errors if:

There is a global phrase_end event before any global phrase_start event
There is a global phrase_end event after another global phrase_end event (ignoring events that are not phrase_end or phrase_start)
There is a global phrase_start event after another global phrase_start event (ignoring events that are not phrase_end or phrase_start)
There are two or more global phrase_start or phrase_end events at the same position


how to save graphs NOT in interactive mode

Hello,
I would like to save graphs from command line and NOT usign the interactive buttons and slots that appear when we use .plt.spatial_scatter or whatever plt. how can I do that?
thanks.

solved:
`from matplotlib import pyplot as plt
with plt.rc_context():  # Use this to set figure params like size and dpi
ms_data.plt.cluster_scatter(res_key='annotation', reorganize_coordinate=4)
plt.savefig("./annotation.png", bbox_inches="tight")`


ServerChatEvent IllegalArgumentException: Illegal group reference

Every chat event triggers the following error in the console. Version 6.7
[21:25:09] [Server thread/ERROR] [Sponge]: Could not pass ServerChatEvent to Plugin{id=EssentialCmds, name=EssentialCmds, version=6.7-fc115bc}
java.lang.IllegalArgumentException: Illegal group reference
        at java.util.regex.Matcher.appendReplacement(Matcher.java:857) ~[?:1.8.0_65]
        at java.util.regex.Matcher.replaceFirst(Matcher.java:1004) ~[?:1.8.0_65]
        at java.lang.String.replaceFirst(String.java:2178) ~[?:1.8.0_65]
        at io.github.hsyyid.essentialcmds.listeners.MessageSinkListener.onMessage(MessageSinkListener.java:200) ~[MessageSinkListener.class:6.7]
        at org.spongepowered.common.event.listener.ChatListener_MessageSinkListener_onMessage28.handle(Unknown Source) ~[?:?]
        at org.spongepowered.common.event.RegisteredListener.handle(RegisteredListener.java:86) ~[RegisteredListener.class:1.8-1577-3.1.0-BETA-1025]
        at org.spongepowered.mod.event.SpongeModEventManager.post(SpongeModEventManager.java:233) [SpongeModEventManager.class:1.8-1577-3.1.0-BETA-1025]
        at org.spongepowered.mod.event.SpongeModEventManager.post(SpongeModEventManager.java:193) [SpongeModEventManager.class:1.8-1577-3.1.0-BETA-1025]
        at net.minecraftforge.fml.common.eventhandler.EventBus.post(EventBus.java:70) [EventBus.class:?]
        at net.minecraftforge.fml.common.eventhandler.EventBus.post(EventBus.java:58) [EventBus.class:?]
        at net.minecraft.network.NetHandlerPlayServer.injectChatEvent(NetHandlerPlayServer.java:86) [rj.class:?]
        at net.minecraft.network.NetHandlerPlayServer.func_147354_a(NetHandlerPlayServer.java:796) [rj.class:?]
        at net.minecraft.network.play.client.C01PacketChatMessage.func_180757_a(SourceFile:37) [lu.class:?]
        at net.minecraft.network.play.client.C01PacketChatMessage.func_148833_a(SourceFile:9) [lu.class:?]
        at net.minecraft.network.PacketThreadUtil$1.onProcessPacket(SourceFile:110) [ih.class:?]
        at net.minecraft.network.PacketThreadUtil$1.run(SourceFile:13) [ih.class:?]
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [?:1.8.0_65]
        at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:1.8.0_65]
        at net.minecraftforge.fml.common.FMLCommonHandler.callFuture(FMLCommonHandler.java:714) [FMLCommonHandler.class:?]
        at net.minecraft.server.MinecraftServer.func_71190_q(MinecraftServer.java:656) [MinecraftServer.class:?]
        at net.minecraft.server.dedicated.DedicatedServer.func_71190_q(DedicatedServer.java:364) [po.class:?]
        at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:598) [MinecraftServer.class:?]
        at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:478) [MinecraftServer.class:?]
        at java.lang.Thread.run(Thread.java:745) [?:1.8.0_65]

You need to update

This still occurs with 6.9c.
[14:20:04] [Server thread/ERROR] [Sponge]: Could not pass ServerChatEvent to Plugin{id=EssentialCmds, name=EssentialCmds, version=6.9-55b6757}
java.lang.IllegalArgumentException: Illegal group reference
	at java.util.regex.Matcher.appendReplacement(Matcher.java:857) ~[?:1.8.0_65]
	at java.util.regex.Matcher.replaceFirst(Matcher.java:1004) ~[?:1.8.0_65]
	at java.lang.String.replaceFirst(String.java:2178) ~[?:1.8.0_65]
	at io.github.hsyyid.essentialcmds.listeners.MessageSinkListener.onMessage(MessageSinkListener.java:201) ~[MessageSinkListener.class:6.9]
	at org.spongepowered.common.event.listener.ChatListener_MessageSinkListener_onMessage30.handle(Unknown Source) ~[?:?]
	at org.spongepowered.common.event.RegisteredListener.handle(RegisteredListener.java:86) ~[RegisteredListener.class:1.8.9-1694-3.1.0-BETA-1060]
	at org.spongepowered.mod.event.SpongeModEventManager.post(SpongeModEventManager.java:233) [SpongeModEventManager.class:1.8.9-1694-3.1.0-BETA-1060]
	at org.spongepowered.mod.event.SpongeModEventManager.post(SpongeModEventManager.java:193) [SpongeModEventManager.class:1.8.9-1694-3.1.0-BETA-1060]
	at net.minecraftforge.fml.common.eventhandler.EventBus.post(EventBus.java:70) [EventBus.class:?]
	at net.minecraftforge.fml.common.eventhandler.EventBus.post(EventBus.java:58) [EventBus.class:?]
	at net.minecraft.network.NetHandlerPlayServer.injectChatEvent(NetHandlerPlayServer.java:91) [lm.class:?]
	at net.minecraft.network.NetHandlerPlayServer.func_147354_a(NetHandlerPlayServer.java:793) [lm.class:?]
	at net.minecraft.network.play.client.C01PacketChatMessage.func_148833_a(SourceFile:37) [ie.class:?]
	at net.minecraft.network.play.client.C01PacketChatMessage.func_148833_a(SourceFile:9) [ie.class:?]
	at org.spongepowered.common.network.PacketUtil.onProcessPacket(PacketUtil.java:106) [PacketUtil.class:1.8.9-1694-3.1.0-BETA-1060]
	at net.minecraft.network.PacketThreadUtil$1.onProcessPacket(SourceFile:51) [fh$1.class:?]
	at net.minecraft.network.PacketThreadUtil$1.run(SourceFile:13) [fh$1.class:?]
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [?:1.8.0_65]
	at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:1.8.0_65]
	at net.minecraft.util.Util.func_181617_a(SourceFile:44) [g.class:?]
	at net.minecraft.server.MinecraftServer.func_71190_q(MinecraftServer.java:660) [MinecraftServer.class:?]
	at net.minecraft.server.dedicated.DedicatedServer.func_71190_q(DedicatedServer.java:344) [ko.class:?]
	at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:605) [MinecraftServer.class:?]
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:481) [MinecraftServer.class:?]
	at java.lang.Thread.run(Thread.java:745) [?:1.8.0_65]

@viveleroi What version of PermissionsEx are you running?

PermissionsEx v2.0-SNAPSHOT - downloaded roughly a week ago. Shall I update?

@viveleroi Yeah try updating it.

The last build of PEX was from Jan 2, which I have.
Using spongeforge-1.8.9-1694-3.1.0-BETA-1060. Yesterday we were on build 1035.

Could not reproduce this

Looking through the code I'm realizing that this is a regex problem. In MessageSinkListener I don't think you're always going to be getting what the code expects because of other chat plugins.
For example we have a chat plugin I've started porting which listens to the MessageChannelEvent.Chat event, intercepts the message, cancels the event, and sends the message to players in specific channels.
That plugin port was started before message sinks/channels were a thing so it really needs work still, but I don't believe that's a factor with the regex problem.

@viveleroi My listener should be among the first called.

I believe this only happens when a player with a "$" in their suffix chats. Our custom chat plugin formats their message with prefix/suffix (and colors not visible here)
[g] LeadMod ickyacky $: test

I'm 95% sure that the error only occurs when players with this suffix enter a message.

I didn't release you may have fixed this, will test v7 now.

This still occurs with 7.2e. Any chat message sent by a player with a dollar sign in their suffix causes this error. Until MessageChannels properly send the command source rather than null, our custom chat plugin is stuck doing it the old fashioned way with capture the message event, formatting the text, and re-sending it.

I'll test

I fully tested it, fixed a few issues today, and believe all should be fixed now.


How to apply config in docker

If I am running mediaflow proxy in docker, how do I apply below config.
I am running it for AD but it doesn't worm from stremio and shows playback error even thought I can reach /docs and other endpoints.
Would applying below config help? If yes, How do I apply it?
PROXY_URL=http://proxy:8080 ALL_PROXY=true

There are the logs I get in my container.
<IP_ADDRESS>:58808 - "HEAD /proxy/stream?api_password=squirrel&d=https%3A%2F%2Fo7XXXXXq9.debrid.it%2Fdl%2F3erkxef549f%2FConclave.2024.1080p.WEBRip.x265.10bit.AAC5.1-%255BYTS.MX%255D.mp4&r_Content-Disposition=attachment%3B+filename%3DConclave.2024.1080p.WEBRip.x265.10bit.AAC5.1-%255BYTS.MX%255D.mp4 HTTP/1.1" 403
<IP_ADDRESS>:58808 - "GET /proxy/stream?api_password=squirrel&d=https%3A%2F%2Fo7XXXXq9.debrid.it%2Fdl%2F3erkxef549f%2FConclave.2024.1080p.WEBRip.x265.10bit.AAC5.1-%255BYTS.MX%255D.mp4&r_Content-Disposition=attachment%3B+filename%3DConclave.2024.1080p.WEBRip.x265.10bit.AAC5.1-%255BYTS.MX%255D.mp4 HTTP/1.1" 403
<IP_ADDRESS>:58808 - "GET /proxy/ip?api_password=squirrel HTTP/1.1" 403

To run with docker, for example, if you're running socks5 proxy,
docker run --restart unless-stopped -p 8889:8888 -d -e api_password=dedsec -e PROXY_URL="socks5://<IP_ADDRESS>:1080" -e ALL_PROXY=true --name mediaflow_proxy mhdzumair/mediaflow-proxy

Here's how to set it up in Portainer:

Click on "Containers" → "Add Container"

Basic Configuration:

Name: mediaflow_proxy
Image: mhdzumair/mediaflow-proxy
Always restart unless stopped: ✓ (Check this option)

Port Mapping:

Host Port: 8889
Container Port: 8888
Protocol: TCP

Environment Variables (Add these):

api_password = password
PROXY_URL = socks5://<IP_ADDRESS>:1080
ALL_PROXY = true

Network Mode:

Leave as default (bridge)

Auto-start container: ✓ (Check this option)

Once you've entered all these settings, click "Deploy the container" to start it. This will create the same container configuration as the Docker command line version.

Thanks, Really appreciate giving your time to answer.
When I run and try to do speedtest with AD, it fails.
It asks for Password where I enter the password configured in docker environment variable. Then I click on All debrid speedtest and enter API key. but it fails and show this error.

I made sure I enter correct password, and error is this now.
     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "/mediaflow_proxy/mediaflow_proxy/configs.py", line 44, in get_mounts
mounts["all://"] = transport_cls(proxy=self.proxy_url)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/mediaflow_proxy/.venv/lib/python3.12/site-packages/httpx/_transports/default.py", line 296, in init
proxy = Proxy(url=proxy) if isinstance(proxy, (str, URL)) else proxy
^^^^^^^^^^^^^^^^
File "/mediaflow_proxy/.venv/lib/python3.12/site-packages/httpx/_config.py", line 214, in init
raise ValueError(f"Unknown scheme for proxy URL {url!r}")
ValueError: Unknown scheme for proxy URL URL('%22socks5://<IP_ADDRESS>:1080%22')

Thanks, Actually it worked with below config.
test

Nice, hope your issue is resolved.


Fix clear ViewBinding in fast Fragment transactions

Hello, what about merging? I have the same issue.


Add backup files

Is your feature request related to a problem? Please describe.
I accidently deleted the clients when was developing a UI and I could revert back
Describe the solution you'd like
Idialy backup wg config files after each update / delete operation
Describe alternatives you've considered
a workaround is to cron job periodic backups

@magixus Great idea. I'll do it soon.

Thanks @leonovk great works here


Update spring-boot version to 2.5.8

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: DanielFran:x: Daniel FRANCODaniel FRANCO seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Having Brackets in the username not supported

Describe the issue:
When a username contains brackets(ex: smith[test]john), functionalities of the server does not work. Some of them are, searching users in the management console, invoking username/password recovery APIs, etc.  This is happening since, during some flows, the DB query executed at the end will have the brackets in the LIKE query value.
This is the query executed for initiating password recovery. The underneath DB layer does not support having brackets in the value for the LIKE operator(Code line: https://github.com/wso2/carbon-kernel/blob/3740b55a5b37b4cbbc328f91f807e3a50043c8b8/core/org.wso2.carbon.user.core/src/main/java/org/wso2/carbon/user/core/jdbc/UniqueIDJDBCUserStoreManager.java#L2328).
SELECT DISTINCT UM_USER.UM_USER_ID
FROM UM_USER,
     UM_USER_ATTRIBUTE
WHERE UM_USER_ATTRIBUTE.UM_USER_ID = UM_USER.UM_ID
  AND UM_USER_ATTRIBUTE.UM_ATTR_NAME = 'uid'
  AND UM_USER_ATTRIBUTE.UM_ATTR_VALUE LIKE 'tharindu[SLASH]bandara'
  AND UM_USER_ATTRIBUTE.UM_PROFILE_ID = 'default'
  AND UM_USER_ATTRIBUTE.UM_TENANT_ID = -1234
  AND UM_USER.UM_TENANT_ID = -1234

There may be some other DB related reserved characters, which could lead to such scenarios.
How to reproduce:
Create a user with the name, tharindu[SLASH]bandara. Make sure to fill in the email claim value. Then initiate a password recovery request. It will fail with no user exist. Also, go to the management console and search by tharindu[SLASH]bandara, it will return zero users.
Expected behavior:
Having [ should be supported in the username, or should be documented if it's not supported.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: 5.10.0
OS: OSX
Database: MsSQL
Userstore: UniqueID JDBC + MSSQL

We can escape the '[' character to get this supported. However, I feel it would make things complicated and may have an impact on performance as well. So I'm +1 to document and restrict the usage of these characters. @thanujalk WDYT?

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Prefer Dependency Injection instead of Mocking

@richleland As I was going through the requirements for change implementation for #12 I realized that the tests depend significantly on mocking the behaviour of HTTPotion. Moving forward, would you be open to preferring a dependency injection model over mocking? This is the style preferred for Elixir core components (see "Mocks and explicit contracts" – José Valim on Plataformatec Blog).
My thoughts would be to do this once #12 is complete but prior to further development.

Makes sense to me @DavidAntaramian we can tackle this right after #12 as you said.


Structure generation and CustomObject duplication

I'm having an issue with village structure generation in a custom TerrainControl world. Village buildings are doubling up e.g.:

Also noticed some odd behaviour when adding CustomObjects, which I managed to fix by doing a  BlockCheck(0,-1,0,SAND,HARD_CLAY)

Have only edited the WorldConfig.ini which can be seen here: http://pastebin.com/RQWBVnAm
police_box.bo3 is here: http://pastebin.com/3kNtj4mB
CraftBukkit version git-Bukkit-d8637df (MC: 1.9.4) (Implementing API version 1.9.4-R0.1-SNAPSHOT)
TerrainControl v2.8.1-SNAPSHOT build 142

The first is because how Minecraft generates villages - it always spawns them above sea level, never below. You can change the sea level by chaning WaterLevelMax in the WorldConfig. It's weird vanilla behaviour. You'll find out that ocean monoments have the same issue.
The second issue is just how BO3s work - if you don't define what a suitable location is, every location is considered a suitable location, including on top of other BO3s.

Thanks for clarifying.
As a new user of TerrainControl, the behaviour was a little unexpected. Perhaps the default WorldConfig should have a WaterLevelMax setting that mimics the vanilla Minecraft values, so that noobs like me don't freak out and make unneeded issues :)

The default WaterLevelMax is 63 - this is already the default of vanilla Minecraft.


Home assistant 084.5 doesn't start anymore

home-assistant.log
Home Assistant release with the issue:
0.84.5

Frontend -> Developer tools -> Info
Or use this command: hass --version
-->

Last working Home Assistant release (if known):
0.84.3
Operating environment (Hass.io/Docker/Windows/etc.):
Debian Stretch (up to date) manual install
Please provide details about your environment.
Raspberry PI 3B, Homer assistant installed a year ago
Component/platform:
ZWave, AsusWRT, Logitech Harmony hubs,
Please add the link to the documentation at https://www.home-assistant.io/components/ of the component/platform in question.
-->
Description of problem:
Unable to connect to Home Assistant. Doesn't start since upgrade.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:
LOGS attached

Looks like you have a DNS resolution issue...
If you are running with Hass.io / Docker, then you may be experiencing the same issue as home-assistant/hassio#663 ...


Update and correct Spanish translation

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues

Merged. Thanks!


[FEATURE] Add New Projects After Creating Issue

There are more interesting projects like the ones in README.md table, you can add yours too! Just create an issue and make a PR !!
Make sure that no link broke & nothing wrong happen (things otherwise must be as it is)

Can I work on the feature?

Yes go ahead @ZahraShahid

Hey @keshavgbpecdelhi i want to work ? Pls can assign me

Hey, I want to work on this. Can you please assign me .

I would like to work on this task ,can you assign me.

Hey @keshavgbpecdelhi can I add my project of html css js by makin new issue.

Hey @keshavgbpecdelhi can I add my project of html css js by makin new issue.

@ishita126jain just make sure your contribution is unique.

Hello may I contribute to this for 2023 Hacktoberfest?

Hey can you assign me this issue ? I have a project with HTML CSS and bit of CSS its whole website made to get funds from firms all it requires is a backend.

I would like to work on this task ,can you assign me.

Can I work on this kindly @keshavgbpecdelhi


SciView update site wants to downgrade imglib2-* in my Fiji

Hi,
running update Fiji today I spotted that SciView update site was attempting to downgrade some imglib2 stuff... only caught my attention because I would expect imglib2/coreFiji update site to update the imglib2 stuff. See attached screen shot:

Isn't there an issue that SciView is coping more by mistake?
Vlado
PS: Thanks for SciView!

I just have noticed that similar behavior happens with many Update sites... it's probably just how it is in this world.... so take it easy, maybe :)

@xulman Thanks for the report. It's true that quite a few update sites have this problem, but it is a problem. The update sites should, in an ideal world, not shadow core libraries. I am tempted to disallow this in the future, although it is convenient in some cases.
For now, I obsoleted the shadowed core libraries on the SciView site, so everything should be back in sync for now; please reopen if not.


ELB / Enterprise Load Balancer service controller

New ACK Service Controller
Support for ELB / Enterprise Load Balancer
List of API resources
List the API resources in order of importance to you:

[resource 1]
[resource 2]

@mdykes-gw I'm fairly certain we won't be supporting classic Elastic Load Balancer (CLB/ELB) in ACK. The https://github.com/kubernetes-sigs/aws-load-balancer-controller project is where our Kubernetes integration with AWS load balancing services will live; it's unlikely that ACK will have much support for deep integration with Kubernetes Ingress and Service resources, since ACK service controllers only manage custom resources in the services.k8s.aws API Group, whereas projects like https://github.com/kubernetes-sigs/aws-load-balancer-controller provide implementations of core Kubernetes resources like Ingress and Service/Endpoint using AWS load balancing APIs.

/close


[Hub Generated] Review request for Microsoft.KeyVault to add version stable/2022-07-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

/azp run unifiedPipeline

/azp run unifiedPipeline

/azp run unifiedPipeline

@dw511214992 @akning-ms this PR only contains doc fixes, but both a breaking change is detected as well as whatever the powershell check is doing, which isn't obvious nor actionable. Can that be improved regardless of whether it's justified on this PR or not?
I can override given the current tags, but it would be best to get some answers regarding why a breaking change was detected first. Unrelated or previous change?


Verbs cleanup

Dependent on the work in #1179. Once that gets merged in, this should be much smaller.

The docs and integration-test specs need to be modified, find references of kismatic install

Oh wait, I see what you mean. Working on it.

Should be everything.

Fixed!

LGTM!


if/then/else keywords not working as expected

What version of Ajv are you using? Does the issue happen if you use the latest version?
6.12.2
Ajv options object

{ allErrors: true, removeAdditional: 'all', nullable: true }

JSON Schema

{
    "title": "sendSms",
    "type": "object",
    "properties": {
        "clientIds": {
            "type": "array"
        },
        "smsId": {
            "type": "integer"
        },
        "content": {
            "type": "string"
        },
        "subject": {
            "type": "string"
        },
        "marketing": {
            "type": "boolean"
        }
    },
    "required": ["clientIds", "marketing"],
    "anyOf": [
        { "required": ["smsId"] },
        {
            "if": { "properties": { "marketing": { "const": true } } },
            "then": { "required": ["content", "subject"] },
            "else": { "required": ["content"] }
        }
    ]
}

Sample data

{
  "clientIds": [
    12345
  ],
  "content": "test content",
  "marketing": false
}

Validation result, data AFTER validation, error messages
{
    "status": "failed",
    "errors": [
        {
            "path": "",
            "message": "should have required property 'smsId'",
            "params": {
                "missingProperty": "smsId"
            }
        },
        {
            "path": "",
            "message": "should have required property 'content'",
            "params": {
                "missingProperty": "content"
            }
        },
        {
            "path": "",
            "message": "should match \"else\" schema",
            "params": {
                "failingKeyword": "else"
            }
        },
        {
            "path": "",
            "message": "should match some schema in anyOf",
            "params": {}
        }
    ]
}

What results did you expect?
This data should be validated with success, as following this example: https://www.jsonschemavalidator.net/s/eydcldvd
I have no idea why jsonschemavalidator validates it but AJV doesn't.

Before looking deeper, the issue is probably that removeAdditional: 'all' removes all properties not used in "if" schema - try switching to use removeAdditional: true. See the docs on how this option works

Given how often this issue comes up, I think it may be best to deprecate in v7-alpha/beta release and to remove completely in v7/8. It is best to put additionalProperties: false into the schema and use removeAdditional: true instead.
The alternative change would be to differentiate to which schemas it applies, but while it MAY be a bit less confusing and as deterministic as it is now, it would be as unexpected for the end users - why additional properties are removed in some cases and not the others. So best if it is controlled explicitly with additionalProperties: false keyword on per-schema basis.

Before looking deeper, the issue is probably that removeAdditional: 'all' removes all properties not used in "if" schema - try switching to use removeAdditional: true. See the docs on how this option works

Yes, it worked now, thank you.

Given how often this issue comes up, I think it may be best to deprecate removeAdditional: all in v7-alpha/beta release and to remove completely in v7/8. It is best to put additionalProperties: false into the schema and use removeAdditional: true instead.
The alternative change would be to differentiate to which schemas it applies, but while it MAY be a bit less confusing and as deterministic as it is now, it would be as unexpected for the end users - why additional properties are removed in some cases and not the others. So best if it is controlled explicitly with additionalProperties: false keyword on per-schema basis.

I had no idea of the interaction between 'removeAdditional' AJV option and the if/then/else keywords.

I had no idea of the interaction between 'removeAdditional' AJV option and the if/then/else keywords.

It's not specific to If/then/else... Wherever there are properties, everything else becomes additional and with removeAdditional: all it is removed.

I had no idea of the interaction between 'removeAdditional' AJV option and the if/then/else keywords.

It's not specific to If/then/else... Wherever there are properties, everything else becomes additional and with removeAdditional: all it is removed.

Got it. Does the 'additionalProperties' property, when not used, has any default value? I mean, by default additionalProperties = false, right?

Default value is true, that is to allow additional properties

Default value is true, that is to allow additional properties

Ok, thank you for answering my doubts.


remove a couple of lines

Describe the bug
Not really a bug.  But in Snowpipe Streaming and Dynamic Tables for Real-Time Ingestion (CDC Use Case) > STEP 7
in the Java UDF
KEYGENERATOR is actually not used.
URL of where you see the bug
https://quickstarts.snowflake.com/guide/CDC_SnowpipeStreaming_DynamicTables/index.html?index=..%2F..index#6

Thanks for pointing this out.  We will remove this unneeded dependency on our next update.


Warnings that can't be suppressed?

What version of OR-Tools and what language are you using?
Version: 9.0.9048
Language: Python
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
Gurobi
What operating system (Linux, Windows, ...) and version?
MacOS v11.5.2
What did you do?
Steps to reproduce the behavior:

Install ortools via pip
Install Gurobi following the instructions here: https://www.gurobi.com/documentation/9.1/quickstart_mac/software_installation_guid.html
Add the following to ~/.zshenv: export GUROBI_HOME=/Library/gurobi912/mac64
Create a python script with the following lines:

from ortools.linear_solver import pywraplp
Solver = pywraplp.Solver.CreateSolver('GUROBI')

Run this script.

What did you expect to see
Preferably nothing
What did you see instead?
WARNING: Logging before InitGoogleLogging() is written to STDERR
I0723 00:00:32.041015     1 environment.cc:737] Found the Gurobi library in '/Library/gurobi912/mac64/lib/libgurobi91.dylib.
Warning: parameter changes on this environment will not affect existing models.

Is there any way to suppress this output?

Hello OR-Tools developers!  Thank you very much for the work you do on this software.  I find it very useful.
In particular, I'm currently using it to solve many similar optimization problems in parallel, using Python's multiprocessing library.  Unfortunately, this results in the above warnings being printed out many times, filling up the terminal.
The issue I've started here isn't a "bug" necessarily, because the excessive warning messages in the output don't stop the code from doing what it needs to do.  It's just an aesthetics thing.
So, I'm wondering:

Is there a way to suppress the warnings above?
If not, can I respectfully request that the good people at OR-Tools provide a way?

You should try:
from ortools.linear_solver import pywraplp
from ortools.init import pywrapinit

def main():
    solver = pywraplp.Solver.CreateSolver('GUROBI')
   ....

if __name__ == '__main__':
    pywrapinit.CppBridge.InitLogging('basic_example.py')
    cpp_flags = pywrapinit.CppFlags()
    cpp_flags.logtostderr = True
    cpp_flags.log_prefix = False
    pywrapinit.CppBridge.SetFlags(cpp_flags)

    main()

see: https://github.com/google/or-tools/blob/master/examples/python/integer_programming.py
note: pywrapinit is still young so any feedback is welcome ;)

Thanks, @Mizux!  I'll give it a try...

@Mizux, I got a chance to try your suggested fix.  Here's what I did:

Save the following code in a file called SuggestedFix.py:

from ortools.linear_solver import pywraplp
from ortools.init import pywrapinit

def main():
    solver = pywraplp.Solver.CreateSolver('GUROBI')
    exit()

if __name__ == '__main__':
    pywrapinit.CppBridge.InitLogging('SuggestedFix.py')
    cpp_flags = pywrapinit.CppFlags()
    cpp_flags.logtostderr = True
    cpp_flags.log_prefix = False
    pywrapinit.CppBridge.SetFlags(cpp_flags)

    main()

Run this code from the terminal with python3 SuggestedFix.py.

The output is:
Found the Gurobi library in '/Library/gurobi912/mac64/lib/libgurobi91.dylib.
Warning: parameter changes on this environment will not affect existing models.

This is better than the previous output:
WARNING: Logging before InitGoogleLogging() is written to STDERR
I0723 00:00:32.041015     1 environment.cc:737] Found the Gurobi library in '/Library/gurobi912/mac64/lib/libgurobi91.dylib.
Warning: parameter changes on this environment will not affect existing models.

So we're making progress!
I notice that if I set cpp_flags.logtostderr = False the output is further reduced to:
Warning: parameter changes on this environment will not affect existing models.

However, this last output is actually the one that I am most interested in eliminating, as it is the one that pops up most frequently in my original code.  Do you know how to eliminate this last warning message?

looking at the source code of ortools/linear_solver/gurobi_interface.cc I can't found any trace of this sentences so my best guess: it comes from gurobi solver itself.
Open question: If yes, how can we "mute" the gurobi solver... ?

Agreed, looking at Gurobi's list of "Recent Bug Fixes by Version" (https://www.gurobi.com/resource/fixes/), I see
Added back missing warning "Warning: parameter changes on this environment will not affect existing models."

There's some related text on page 6 of their documentation here: https://www.gurobi.com/wp-content/plugins/hd_documentations/documentation/9.0/refman.pdf

We should expose the log level param
https://github.com/google/or-tools/blob/86d4c543f717fa8716a9d66c25143660668bf825/ortools/base/logging.cc#L130-L132
so we can ignore the gurobi log:
https://github.com/google/or-tools/blob/86d4c543f717fa8716a9d66c25143660668bf825/ortools/gurobi/environment.cc#L737
level being:
https://github.com/abseil/abseil-cpp/blob/1ae9b71c474628d60eb251a3f62967fe64151bb2/absl/base/log_severity.h#L69-L74
Note in java
import com.google.ortools.init.CppBridge;
import com.google.ortools.init.CppFlags;
...
    CppBridge.initLogging("test");
    CppFlags myFlags = new CppFlags();
    myFlags.setLogtostderr(true);
    myFlags.setLog_prefix(false);
    CppBridge.setFlags(myFlags);
    CppBridge.shutdownLogging();


Make it possible to pass TimeUnit for method waitForElementToDisplay(downloadCompletePopup, 30)

waitForElementToDisplay(downloadCompletePopup, 30)

should be possible to rewrite in a more readable format as
waitForElementToDisplay(downloadCompletePopup, 30, SECONDS);

Also change README
...and wait for specific time and conditions
waitFor(3, MINUTES);                                // Wait for specific time
waitForElementToDisplay(downloadCompletePopup, 30); // Wait for WebElements to display within specific time

to
...and wait for specific time and conditions
waitFor(3, MINUTES);                                                                             // Wait for specific time
waitForElementToDisplay(downloadCompletePopup, 30, SECONDS); // Wait for WebElements to display within specific time


GEODE-33: clarify GEODE_HOME value

Clarify prose that identifies value to set for GEODE_HOME,
based on dev feedback.

+1

+1


Lily/MoreEvents 3 new blocks

Hi everyone!
Here a 3 new blocks.
These are the first two,

And the third is something I'm sure some of you have been waiting for...
3,
2,
1,
🎉  Showtime!  🎉

Now for those who don't see the difference...
Scratch's block is like this:

And ours is:

...Which allows us to do some cool things such as:

And many other things too!
Now let's get back to the first two blocks.
What do they do?
The first one returns the name of the broadcast that started this thread, and the other one pretty much explains itself.
Here is a little illustration for you:

And that's about it, everyone! Let's see how many 👍s we can get.

[!IMPORTANT]
This requires some changes to the VM, so it won't work until those are merged.

Notes
I'd like to add the following option, but don't know how to add it to the menu:

Otherwise, should I just add the following instead?

Or should we just leave people to do
?

I don't like this
@LilyMakesThings I'll just make a new extension then ☺


Using python requests through proxy2 fails with https://

Small sample script:
import requests http_proxy  = "http://localhost:3120" https_proxy = "http://localhost:3120" ftp_proxy   = "http://localhost:3120" proxyDict = {                "http"  : http_proxy,                "https" : https_proxy,                "ftp"   : ftp_proxy             } url = 'https://google.de' session = requests.Session() session.verify = False r = session.post(url, '', proxies=proxyDict, timeout=100)
Expected behaviour: Serving as a proxy ;)
Found behaviour:
requests.exceptions.ProxyError: HTTPSConnectionPool(host='google.de', port=443): Max retries exceeded with url: / (Caused by ProxyError('Cannot connect to proxy.', error("(10053, 'WSAECONNABORTED')",)))
or
Empty response to sample script.
Thx for your support. :)

Why do you use port 3120? Try 8080 instead.

That yields exactly the same result. I only tried different ports while testing.

I have to know where the proxy accessed and the transmitted packets between the two. Send me reproducible test case or pcap.

In my environment, the below command through proxy2 worked fine.
$ http_proxy=localhost:8080 curl -v -L https://google.de/
*   Trying <IP_ADDRESS>...
* Connected to google.de (<IP_ADDRESS>) port 443 (#0)
* found 173 certificates in /etc/ssl/certs/ca-certificates.crt
* found 692 certificates in /etc/ssl/certs
* ALPN, offering http/1.1
* SSL connection using TLS1.2 / ECDHE_RSA_AES_128_GCM_SHA256
*        server certificate verification OK
*        server certificate status verification SKIPPED
*        common name: *.google.de (matched)
*        server certificate expiration date OK
*        server certificate activation date OK
*        certificate public key: RSA
*        certificate version: #3
*        subject: C=US,ST=California,L=Mountain View,O=Google Inc,CN=*.google.de
*        start date: Thu, 03 Nov 2016 01:33:43 GMT
*        expire date: Thu, 26 Jan 2017 01:13:00 GMT
*        issuer: C=US,O=Google Inc,CN=Google Internet Authority G2
*        compression: NULL
* ALPN, server accepted to use http/1.1
> GET / HTTP/1.1
> Host: google.de
> User-Agent: curl/7.47.0
> Accept: */*
>
< HTTP/1.1 301 Moved Permanently
< Location: https://www.google.de/
< Content-Type: text/html; charset=UTF-8
< Date: Thu, 17 Nov 2016 06:54:44 GMT
< Expires: Sat, 17 Dec 2016 06:54:44 GMT
< Cache-Control: public, max-age=2592000
< Server: gws
< Content-Length: 219
< X-XSS-Protection: 1; mode=block
< X-Frame-Options: SAMEORIGIN
< Alt-Svc: quic=":443"; ma=2592000; v="36,35,34"
<
* Ignoring the response-body
* Connection #0 to host google.de left intact
* Issue another request to this URL: 'https://www.google.de/'
*   Trying <IP_ADDRESS>...
* Connected to www.google.de (<IP_ADDRESS>) port 443 (#1)
* found 173 certificates in /etc/ssl/certs/ca-certificates.crt
* found 692 certificates in /etc/ssl/certs
* ALPN, offering http/1.1
* SSL connection using TLS1.2 / ECDHE_RSA_AES_128_GCM_SHA256
*        server certificate verification OK
*        server certificate status verification SKIPPED
*        common name: www.google.de (matched)
*        server certificate expiration date OK
*        server certificate activation date OK
*        certificate public key: RSA
*        certificate version: #3
*        subject: C=US,ST=California,L=Mountain View,O=Google Inc,CN=www.google.de
*        start date: Thu, 03 Nov 2016 01:12:00 GMT
*        expire date: Thu, 26 Jan 2017 01:12:00 GMT
*        issuer: C=US,O=Google Inc,CN=Google Internet Authority G2
*        compression: NULL
* ALPN, server accepted to use http/1.1
> GET / HTTP/1.1
> Host: www.google.de
> User-Agent: curl/7.47.0
> Accept: */*
>
< HTTP/1.1 200 OK
< Date: Thu, 17 Nov 2016 06:54:44 GMT
< Expires: -1
< Cache-Control: private, max-age=0
< Content-Type: text/html; charset=ISO-8859-1
< P3P: CP="This is not a P3P policy! See https://www.google.com/support/accounts/answer/151657?hl=en for more info."

Same request with: curl -v -L https://google.de/ --proxy http://<IP_ADDRESS>:8080 --insecure
proxy2_curl.txt

I need the pcap between proxy2 and google.de, not <IP_ADDRESS> (proxy2) and <IP_ADDRESS> (host running requests).
And the curl command works fine in my environment like below.
$ curl -v -L https://google.de/ --proxy http://localhost:8080 --insecure
*   Trying ::1...
* Connected to localhost (::1) port 8080 (#0)
* Establish HTTP proxy tunnel to google.de:443
> CONNECT google.de:443 HTTP/1.1
> Host: google.de:443
> User-Agent: curl/7.47.0
> Proxy-Connection: Keep-Alive
>
< HTTP/1.1 200 Connection Established
< Server: BaseHTTP/0.3 Python/2.7.12
< Date: Thu, 17 Nov 2016 07:16:02 GMT
<
* Proxy replied OK to CONNECT request
* found 173 certificates in /etc/ssl/certs/ca-certificates.crt
* found 692 certificates in /etc/ssl/certs
* ALPN, offering http/1.1
* SSL connection using TLS1.2 / ECDHE_RSA_AES_128_GCM_SHA256
*        server certificate verification SKIPPED
*        server certificate status verification SKIPPED
*        common name: *.google.de (matched)
*        server certificate expiration date OK
*        server certificate activation date OK
*        certificate public key: RSA
*        certificate version: #3
*        subject: C=US,ST=California,L=Mountain View,O=Google Inc,CN=*.google.de
*        start date: Thu, 03 Nov 2016 01:33:43 GMT
*        expire date: Thu, 26 Jan 2017 01:13:00 GMT
*        issuer: C=US,O=Google Inc,CN=Google Internet Authority G2
*        compression: NULL
* ALPN, server accepted to use http/1.1
> GET / HTTP/1.1
> Host: google.de
> User-Agent: curl/7.47.0
> Accept: */*
>
< HTTP/1.1 301 Moved Permanently
< Location: https://www.google.de/
< Content-Type: text/html; charset=UTF-8
< Date: Thu, 17 Nov 2016 07:16:02 GMT
< Expires: Sat, 17 Dec 2016 07:16:02 GMT
< Cache-Control: public, max-age=2592000
< Server: gws
< Content-Length: 219
< X-XSS-Protection: 1; mode=block
< X-Frame-Options: SAMEORIGIN
< Alt-Svc: quic=":443"; ma=2592000; v="36,35,34"
<
* Ignoring the response-body
* Connection #0 to host localhost left intact
* Issue another request to this URL: 'https://www.google.de/'
* Found bundle for host www.google.de: 0x56235b30c5a0 [can pipeline]
* Hostname localhost was found in DNS cache
*   Trying ::1...
* Connected to localhost (::1) port 8080 (#1)
* Establish HTTP proxy tunnel to www.google.de:443
> CONNECT www.google.de:443 HTTP/1.1
> Host: www.google.de:443
> User-Agent: curl/7.47.0
> Proxy-Connection: Keep-Alive
>
< HTTP/1.1 200 Connection Established
< Server: BaseHTTP/0.3 Python/2.7.12
< Date: Thu, 17 Nov 2016 07:16:03 GMT
<
* Proxy replied OK to CONNECT request
* found 173 certificates in /etc/ssl/certs/ca-certificates.crt
* found 692 certificates in /etc/ssl/certs
* ALPN, offering http/1.1
* SSL connection using TLS1.2 / ECDHE_RSA_AES_128_GCM_SHA256
*        server certificate verification SKIPPED
*        server certificate status verification SKIPPED
*        common name: www.google.de (matched)
*        server certificate expiration date OK
*        server certificate activation date OK
*        certificate public key: RSA
*        certificate version: #3
*        subject: C=US,ST=California,L=Mountain View,O=Google Inc,CN=www.google.de
*        start date: Thu, 03 Nov 2016 01:12:00 GMT
*        expire date: Thu, 26 Jan 2017 01:12:00 GMT
*        issuer: C=US,O=Google Inc,CN=Google Internet Authority G2
*        compression: NULL
* ALPN, server accepted to use http/1.1
> GET / HTTP/1.1
> Host: www.google.de
> User-Agent: curl/7.47.0
> Accept: */*
>
< HTTP/1.1 200 OK
< Date: Thu, 17 Nov 2016 07:16:03 GMT
< Expires: -1
< Cache-Control: private, max-age=0
< Content-Type: text/html; charset=ISO-8859-1
< P3P: CP="This is not a P3P policy! See https://www.google.com/support/accounts/answer/151657?hl=en for more info."
< Server: gws
< X-XSS-Protection: 1; mode=block
< X-Frame-Options: SAMEORIGIN
< Set-Cookie: NID=91=RhJj5--x9iNtxlqqcSUv78MjJHIVCtTdNXhBMnnFrSuEsP5Vsfz4XjyrvefboDTFD6QYwAEB9fMVHzVaR7kON2Zm8LUlStNl5rUAIzm5pKWQ9KlLdyW6B9vbO-tItgTY; expires=Fri, 19-May-2017 07:16:03 GMT; path=/; domain=.google.de; HttpOnly
< Alt-Svc: quic=":443"; ma=2592000; v="36,35,34"
< Accept-Ranges: none
< Vary: Accept-Encoding
< Transfer-Encoding: chunked
<
(snip)
* Connection #1 to host localhost left intact

The curl command runs fine for me, too.
It's any request between pythons requests module & proxy2. I'll do the log of proxy2 <-> google in a few minutes. Thx for your time up til now. :)

proxy2.pcap.zip
That's the pcap between proxy2 -> google.
Thing is, for me it looks - from the error message posted above, that the issue is not between proxy2 -> google, but python-requests -> proxy2. See  ('Connection aborted.', BadStatusLine("''",))

I tried your test case but no errors raised.
$ python test.py
/home/user/tmp/test/ENV/local/lib/python2.7/site-packages/requests/packages/urllib3/connectionpool.py:843: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
  InsecureRequestWarning)

It looks like some network error in your environment.
If proxy2 outputs any traceback log to console, tell me.

There's not even a request logged:
http://i.imgur.com/WHSQ3OG.png
It's a fresh virtual machine with all updates, same behaviour on windows.
Which version of requests are you using?
I'm at 2.12.1.

It's requests 2.12.1 on Ubuntu 16.04.1 LTS on Windows 7 VirtualBox.
It seems weird but I couldn't reproduce, so debug proxy2.py by yourself.

Ok, thx for your help. I had the same issue on diverse systems, the only difference to your setup in my testing environment is Win10. I attached another pcap that shows the connection reset between my request and proxy2, just fyi. I'll try to find anything and if I come up with an idea, I'll let you know.
proxy2.pcap.zip
ll let you know.

The request fails at the following lines:
 conntype = self.headers.get('Proxy-Connection', '')         if conntype.lower() == 'close':             self.close_connection = 1         elif (conntype.lower() == 'keep-alive' and self.protocol_version >= "HTTP/1.1"):             self.close_connection = 0
As a dirty hack I added else: self.close_connection = 0and the request went through without problems. So there's no header for Proxy-Connection, leading to an unhandled case.

So, the thing is - for whichever reason, the keep-alive header is from my request while it is in your request.

Thanks for your cooperation :+1:
I improved the condition whether the connection closes or not at latest master, try it out if you have time.

Works fine for me! :)
Thx for your help again!


TIKA-4252: switch to using the parse context for additional http headers

add a parse context
allow additional data to be sent int the parse context to the fetch method

sure will do @tballison sorry didn't see this until now

Ha, @nddipiazza. I did earlier this morning. I chose your choices over mine in the merge, largely.
See https://issues.apache.org/jira/browse/TIKA-4243?page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel&focusedCommentId=17851727#comment-17851727
What we now need to do is figure out how to serialize+deserialize ParseContext with as little work as possible. :D


Globally installed npm @types packages are no longer respected in standalone JS/TS files

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.91.1 f1e16e1e6214d7c44d078b1f0607b2388f29d729 x64
OS Version: Windows 10 Pro 22H2 (19045)

Steps to Reproduce:

Create a new file.ts file anywhere in explorer and double-click it/open it in VSCode as a standalone file (not opening a folder)
Write import fs from "node:fs";
VSCode will frustratingly refuse to resolve the Node built-in, this used to be fixable (I do not recall the exact VSC version) by running npm i -g @types/node
Attempt to run npm i -g @types/node and reload VSCode
Issue still occurs.

How is one supposed to tell VSCode to load Node types in standalone JS/TS files now? I refuse to have to create an entire TSConfig/node_modules/package.json every time I want to write simple one-off scripts to be ran directly like node file.js (or bun file.ts for TS scripts)

I refuse to have to create an entire TSConfig/node_modules/package.json ... VSCode will frustratingly refuse to resolve the Node built-in, this used to be fixable

For TypeScript, this has never been a supported configuration as far as I can tell. TypeScript files don't get automatic type acquisition and never have.
For JS files, this works without any config needed. You shouldn't need to do anything.
Sharing your TS Server log might be helpful.


Internationalize "Stop this application" and "About this app" menu button titles

Do this after pr #517 has been merged.
See forum post:
http://ai2.appinventor.mit.edu/reference/other/responsiveDesign.html

Do we want the local language to be something set by the developer at build time, or should it be set by the phone's local setting?
Do we want the translations to be created in the same way as the localized command and component names?   If so, it might be nice to put the translations in messages.java but that that is (a) not in the right package for use by the components; (b) needs a mechanism to make it work when App Inventor switches languages.
In any case, this needs a design doc for the implementation.


[Windows 10] [AMD] [D3D9] Splinter Cell Conviction - cannot maximise the game after alt tabbing

edit https://github.com/doitsujin/dxvk/issues/1998#issuecomment-1013813199

The game works fine with D9VK 0.40.1 for me.
On current DXVK version, after successful injection, the game works, Vulkan is identified in MSI Afterburner OSD, but after clicking once of twice on black title screen (we can hear sound), the game only shows sky and cursor in the main menu, and everything else is black.
DXVK's d3d9.dll copied to the src\system folder. Copying also the rest of libraries causes the game not to start.
Leaving only some .dlls I somehow managed to generate more of the logs.
Older D9VK somehow improves the game performance when performance drops occur. I hope it may get even better with newer DXVK version.
The problem of poor optimization of the game is commonly known. E.g.
https://www.pcgamingwiki.com/wiki/Tom_Clancy's_Splinter_Cell:_Conviction
https://www.reddit.com/r/D9VK/comments/dh1ma4/splinter_cell_conviction_d9vk_long_test/
The good performance test places:

Iraq segment (https://youtu.be/kgwqKMEQtsA?t=3464)
Washington monument

Software information
Splinter Cell Conviction (Uplay)
QHD DSR. All details low, besides ambient occlusion, and aniso x16 (everything else decreases performance).
System information

GPU: RX 470
Driver: 20.10.1 and 21.5.1 (outdated, but the issue is reproducible  on other configurations)
OS: W10
DXVK version: 1.8.1 / 1.9.1 / 1.9.2

Apitrace file(s)
https://disk.yandex.com/d/EBlLJZFGOJEvVQ
Log files

d3d9.log: Conviction_game_d3d9.log
d3d11.log: Conviction_game_d3d11.log
dxgi.log: Conviction_game_dxgi.log
d3d9.log (d9vk): Conviction_game_d3d9.log

Did you follow this guide when making a trace: https://github.com/Joshua-Ashton/d9vk/wiki/Making-a-Trace
We can't do anything without an apitrace.

Here it is:
https://disk.yandex.com/d/EBlLJZFGOJEvVQ
Yes, that's I followed.
"The application failed to initialize properly (Oxc0000005)"
x86 libraries used.
Couldn't find anything useful about that Oxc0000005 error connected with
apitrace in Google.
I used some older version of apitrace.
Here it is:
https://github.com/apitrace/apitrace/releases/tag/windows-xp-latest
Trace files were in exe folder nearby library.
It would be nice to edit and add the solution to the instruction.
wt., 30 mar 2021 o 20:35 Robin Kertels @.***>
napisał(a):

Did you follow this guide when making a trace:
https://github.com/Joshua-Ashton/d9vk/wiki/Making-a-Trace
We can't do anything without an apitrace.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/doitsujin/dxvk/issues/1998#issuecomment-810486708,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIJ3EHHZPEGL5Q63GR5Z5YLTGIKYJANCNFSM42CPZVVQ
.

@K0bin  I cannot customize labels, so please delete "needs apitrace" if the provided apitrace is fine.

Tested on the newest DXVK 1.9.3 and it started to work.
It heavily stutters on the beginnings of the missions, which is fixed by:
https://github.com/Sporif/dxvk-async/releases/
The other problem, is that the game cannot be restored from being minimized,
while in D9VK it couldn't be minimized at all - while alt tabbing, none of windows could appear (only blue ALT+CTRL+DEL window, but not even a process manager).

Please test with latest master https://github.com/doitsujin/dxvk/actions/runs/2955815479

It's still the same. Sorry for late response.
If I try to maximise/alt+tab back, nothing happens, and suddenly I cannot move my cursor in the area of start menu when the game supposed to be maximised after I clicked the game button in start menu/alt+tabbed.

@deton24 could try with this PR ?https://github.com/doitsujin/dxvk/pull/3316. You can find a download of it here https://github.com/doitsujin/dxvk/actions/runs/4537356630
First try it out of the box just to verify its still an issue, it shouldn't do anything new out of the box besides what has been changed on master since you tested last time.
If it still happens then make a file called dxvk.conf where the dxvk log files appear (usually next to the exe but not always) insert ~d3d9.deviceLost = True` and try again.

Seeing the same issue with most recent dxvk release on W11 with an RTX 3080.


Goals for interactions

Hi
Loving this directive effort so far. Is there any future goals for having directives allowing better access to the interactions present in OL3? Draw etc.
Thanks

Hi, what exactly are you thinking about. Could you describe in more detail?

I guess he wants something like  managment the of controls. Have same request, but i guess exist a form to do that in other manner


Update to ops 2.10

It'd be great to move this charm to ops 2.x (and if doing so, then 2.10 would be a good target for now).
All the tests pass for me locally when doing this, so I'm not sure what the motivation was behind pinning to 1.x. It could be that there are issues that aren't caught by the tests that mean more needs to be done here. If you need help moving to 2.x, please reach out to @canonical/charm-tech and we'd be happy to help.

@tonyandrewmeyer could you rebase this branch?
I don't think there was any reason why this doesn't use ops 2.x  other than major rev pinning to prevent catastrophes of APIs changing.

@tonyandrewmeyer could you rebase this branch?

Done (well, it was a merge because I lazily just used the GitHub UI's sync tool, but I assume this would be squashed so it would all work out the same in the end).

#9 will  likely incorporate this same change.  Thanks for helping us though


Split Vip.RestrictedFunctions Sniff into logical groups

Parts of the Vip.RestrictedFunctions Sniff could be highly re-usable in other sniff sets. This is currently not possible as all these function sniff are within one sniff file.
I'd suggest splitting this file into several sorted by group.
Initial grouping proposal:

PHPRestricted (ereg, eval and the like)
PHPDeprecated (ereg and the like)
PHPAlternativePreferred (create_function, curl, split and the like)
WPAlternativePreferred (parse_url and the like)
VIPAlternatives (all those which have wpcom_...() alternative functions)
WPCacheRestricted (get_pages, get_post_terms and the like)
BatcacheRestricted (cookies and the like)

Opinions ?

Closing as duplicate of #582 - seems we had the same idea at the same time ;-)


Message bus options

DO NOT MERGE: it requires https://github.com/cloudfoundry/cf-message-bus/pull/3 to be merged and update the Gemfile

A short explanation of the proposed change: Allows overriding some default nats properties

An explanation of the use cases your change solves:  This will help us to prevent the issue documented at https://github.com/cloudfoundry/cloud_controller_ng/issues/726

[x] I have viewed signed and have submitted the Contributor License Agreement

[x] I have made this pull request to the master branch

[x] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests on bosh lite

Hey @frodenas,
Everything looks good to us. We just had a small question on the usage for dont_randomize_servers. How is this preventing message_bus from sending large amounts of nats messages as described (here)[https://github.com/cloudfoundry/cloud_controller_ng/issues/726]?
Thanks,
CAPI Community Pair - @jberkhahn & @michaelxupiv

@michaelxupiv @jberkhahn Just max_reconnect_attempts is really useful for #726.
I added dont_randomize_servers because https://github.com/cloudfoundry/cf-message-bus/pull/3 allows now to override this param as well, so the 3 nats options allowed by the gem (servers, max_reconnect_attempts, dont_randomize_servers) could be now set via a deployment manifest if desired.

Hey there,
We were able to reproduce the related bug from issue 726. However, when we tried pulling this change in, and setting max_reconnect_attempts to 10, we don't see any slowdown in the thrashing on port 4222 on the api vm.
Furthermore, when we go and trace the nats traffic on the message bus, we don't see any actual increases in nats messages being received.
Were there any other configuration options you needed locally to get this to work or are we doing something wrong?
Thanks,
@jberkhahn & @ericpromislow
CF CAPI Team

@jberkhahn & @ericpromislow
Did you manually created a new version of the cf-message-bus gem and vendored it in this repo?
AFAIK, https://github.com/cloudfoundry/cf-message-bus/pull/3 was merged, but nobody published a new version, at least according to rubygems. Without the new gem version, this PR is useless, and this is why I added a note at the top of the PR description. If somebody can push the new version to rubygems, then I can update this PR with the additional changes.

@frodenas
Since this has been resolved as part of #737 we are not going to bring this in.  That issue fixes the problem where the cloud controller was crashing and restarting when the database had shut down, increasing NATS traffic.
Thanks,
@zrob & @ericpromislow

If #737 fixes the problem, then I'm ok.


License

Hi,
Thanks for this wonderful piece of work. I do not see any LICENSE file nor license indication on the repository nor into the files. I've only found MIT note on package.json. To me, without license, we are in classical copyright and nobody really can fork your library or contribute to it (only exception : personal usage could be achieved).
You may want to add a LICENSE file to clarify the situation ? Thank you in advance for your answer.

Missed it somehow. Updated repo with MIT licence file.


Image previews not working when extension is upper case

Describe the Bug

The image preview of documents is not working when the image extension is in upper case
Steps to Reproduce

Upload image with e.g. .JPG as extension

Expected Behavior

The image should be previewed when it's a supported type (png or jpg)
Actual Behavior

The image is not previewed
Additional Information

The faulty check can be found here:
https://github.com/digitalfabrik/lunes-cms/blob/c77ce95505fa5b5bc4cf2086a084498524d7979b/lunes_cms/cms/models/document_image.py#L35

Just submitted a pull request that should fix this problem.


Is it possible to have a simple json example??

Using a simple data.json file to populate a simple test case??

Support questions should be directed to stack overflow.
var gulp = require('gulp');
var jade = require('gulp-jade');

var data = require('./data.json');

gulp.task('jade', function(){
    return gulp.src('**/*.jade')
        .pipe(jade({ locals: data })
        .pipe(gulp.dest('build');
});

If you want to submit a PR that adds this example to the readme with some description, I will review it.


Help Glitched

When I try to play the game I can open the hack menu and change all the hacks, but when I play new game it does not work then when I go to fix it and this come up
var request = new XMLHttpRequest(); request.open('GET', chrome.extension.getURL(fileName), false);
then also this
var gameInitBase = mainModule.prototype[obfuscate.init];
I don't know what this means but before a couple of days ago this had not changed and this was working just fine pls help I don't know what is happening maybe create a entire new hack just like this one but with more things like trigger bot, fire bot, auto dodge, this would be amazing but ya pls help.

there's a version someone made a couple hours ago that's a quick fix. it only has the aimbot but will do until the regular version gets fixed

cool
On Fri, Mar 22, 2019 at 4:49 PM NoBoat9<EMAIL_ADDRESS>wrote:

there's a version someone made a couple hours ago that's a quick fix. it
only has the aimbot but will do until the regular version gets fixed
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Kalaborative/survivio-plus/issues/351#issuecomment-475807392,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AsGmYqAv0EZQfXd-mUalE7Mu2vllnngGks5vZV3ygaJpZM4cEAT6
.

https://github.com/typico/survivio-plus
-> + wallhack + autoloot

https://github.com/typico/survivio-plus
-> + wallhack + autoloot

um it doesnt work


Any plan to support Scala 2.12?

Is there a date where we can have the jar supporting Scala 12?

fyi, this repo is pretty much abandoned.  However, there is a community driven fork which is actively maintained and has spark 3 support, hadoop 3.2 support, etc.
https://github.com/spark-redshift-community/spark-redshift


Target included in Master

New commit broke it again
You must restore this check
if (sti.getTargetDirectoryName().toLowerCase().startsWith(sti.getMasterDirectoryName().toLowerCase()))

It also broke Zip function: it wrongly detects target is same as master for ZIP


[bug?] .restangularizeElement() changes route url

I'm adding a save method to a collection.
Since the services is used in different parts of the code, it most be able to restangularize the elem if it's not done already.
    var Resouces = Restangular.service('myresources');
    Restangular.extendCollection('myresources', function(collection){
      collection.save = function(elem){
        if( !elem.hasOwnProperty('save') ){
          Restangular.restangularizeElement(null, elem, 'myresources');
        }
        console.log(elem);
        return elem
          .save()
          .then(function(res){
            collection.push(res);
            return(res);
          })
      }
      return collection;
    })

This works fine for post(), but if i try to delete the newly created object with remove() I see that the api receives a call to: '/api/myresources/undefined/{:id}'
How do i get rid of the 'undefined' in the route? (the elements fetched by the collection work fine).
Looking at the difference between the restangularized element and the other collection elements, I see that the first
.parentResource == Object {route: 'myresources', parentResource: null}

while the second
.parentResouce == null

I tried setting it to null after the call but without success.
 Restangular.restangularizeElement(null, elem, 'myresources');
 elem.parentResource = null

Is this a bug or am I doing something wrong?

I have same issue.

Confirmed with plunkr http://jsfiddle.net/b9n8h9xa/3/

Is there any workaround for this?

If anyone else encounters this I had success just using .plain() then calling restangularizeElement manually with the route. Also setting fromServer attribute to true.
  scope.example.save().then(function(data) {
    var e = restangular.restangularizeElement('', data.plain(), 'widgets');
    e.fromServer = true;
  });

Thank you! :+1:


add header, query, and path properties to models

Right now our DPG models only handle body properties, but we want to have the ability to expose header, query, and path properties on the models as well.
Current thinking is shown in this gist: https://gist.github.com/iscai-msft/7cf112542d9bc773159297147bcdd00b

taking off the GA label, we can decide how to expose these after GA in a non-breaking way

Closing in favor of https://github.com/Azure/autorest.python/issues/1991


FARMING - Removal of Pricing table

The whole green table section needs to be removed for now.

resolved, thank you Kristine.


Race condition

Just gonna submit a PR for this real quick adding a conditional

I think its a race condition

Or maybe sending in an empty array + content causes this? going to just stop sending empty arrays lol but something to consider

okay i should consider switching main logic to actor. thank you for reporting.

    ###Instructions###

    Provide exactly two words for a title for the above chat. YOU WILL BE PENALIZED for any additional words, print exactly two words

For some reason this prompt causes a crash - this is the cause of the Int32 crash i posted above

if you give me the model, the seed, and the template you've used, it will be easier for me to check later. can you provide me those details?

give me any minimal reproducible case if you can please.

Running mistral-7b-instruct-v0.2.Q4_K_M.gguf - not sure about seed, how could that be determined? It replicates every time running that string on .respond


Transaction does not have signal for errors

There is no way to detect if an error occurred when signing and sending transactions. Add signals for errors.
ETA: 1h

Signal is added now.