fix: break the loop when connected

Summary by CodeRabbit

Bug Fixes

Improved connection logic by ensuring the process terminates upon a successful connection, reducing unnecessary attempts and enhancing efficiency.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 34.23%. Comparing base (06c4e02) to head (9732e01).

Files
Patch %
Lines

midealocal/device.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #244      +/-   ##
==========================================
- Coverage   34.24%   34.23%   -0.01%
==========================================
  Files          81       81
  Lines        7164     7165       +1
==========================================
  Hits         2453     2453
- Misses       4711     4712       +1

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


[Insets] Remove library from Accompanist

Insets has been deprecated for multiple releases now so I am removing it completely from the repository.
I will leave the guide up for now in case people click the link, probably one more release.

All fixed!


Add unlimited fps argument to Brave desktop file

Add the following argument to the Brave desktop file after installing.
--args --disable-gpu-vsync (adding more args makes it unstable)

moved to private task list


Customer can't sign up when this plugin is used.

This is a follow up on #184. I've had a prospective customer contact me about a similar issue again, saying this:

When I key last digit of my (normal) 16 digit card no the page clips it back to 15 digits.

Here are the details requested previously:

Card type: Visa.
Card length: 16 digits.
Issuer: AIB Bank Visa
Browser/system details: IE11 up to latest update xx071 on W10 and Dell Latitude e5440.

--
I tried disabling parts of the JS previously but that caused other problems so I re-enabled the full functionality.
This issue is pretty serious because we're losing revenue from customers who can't enter payment details.

Does this occur if one or more leading spaces are present?

I'm not sure. I can ask the customer if this information is needed.

It's easily testable, try it out on your form I the same browser!
I just recommend testing that because that's the first thing the reported behavior implies to me.  If a form field has a max length but does not trim leading and trailing whitespace, and a customer copies in a space before the card number, I would expect them to become limited to 15 visible credit card digits (+1 invisible space) and not be able to see why.
Worth a quick try.

@macgregordennis Can you get the first few numbers in the card? See here for how we match.

Closing for lack of activity. Please recreate with more details if this is still an issue!


Add unplugging devices to instructions

Hello,
I propose adding to instructions that it's necessary to unplug any devices from the device that is being flashed. Seems obvious but it's easy to forget, especially if there are other warnings in the instructions.
In my case, with default Tasmota firmware end, after setting the correct profile (Digoo DG-SP202) and clicking Save, the device rebooted and right after hearing the two relays, it blew the fuse in the fuse box and apparently also in itself. The device is now dead.
Thanks.

Done. Sorry to hear about your blown fuses! Better luck next time


Cannot load filter Error: ORA-00979: not a GROUP BY expression

Bug description
I faced this problem with superset oracle connection. There is a connection with oracle and there is data, but there is this error in the filter, this process happened after connecting to oracle postres, this problem is not there, because it is the superset settings or the queries are wrong?

and where does such an error 1 appear in the console?

How to reproduce the bug
I don't know
Screenshots/recordings

Superset version
3.1.2
Python version
3.9
Node version
16
Browser
Chrome
Additional context
No response
Checklist

[ ] I have searched Superset docs and Slack and didn't find a solution to my problem.
[X] I have searched the GitHub issue tracker and didn't find a similar bug report.
[ ] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

@michael-s-molina

I faced this problem with superset oracle connection. There is a connection with oracle and there is data, but there is this error in the filter, this process happened after connecting to oracle postres, this problem is not there, because it is the superset settings or the queries are wrong?

Hi @mujohiddin. Oracle does not support GROUP BY 1. I believe you are using a virtual dataset? If that's the case, just replace GROUP BY 1 with GROUP BY "CODE".

@michael-s-molina
but 5 months ago version didn't have this error, I got this error when upgrading to new version.

@michael-s-molina GROUP BY 1 with GROUP BY "CODE".
I could not find this part, can you tell me where to replace it

I could not find this part, can you tell me where to replace it

1 - Go to the filter configuration
2 - Check what's the dataset that's being used for the filter
3 - Go to the datasets list and check if that dataset is a virtual dataset
4 - If it is, you can edit its query

@michael-s-molina ,
I followed the steps what you suggested, but there problem is not in the dataset.  The above query was generated by Superset itself on filter pane. I think issue is with SQLA dialect, because the source was change to Postgre, it works, since Postgre supports Group BY 1, but Oracle does not. Therefore I am guesing it is due to SQLA dialect on newer version of Superset.
Moreover, I cannot create Table, Pivot charts , they are failling with the same error :

The above query was generated by Superset itself on filter pane. I think issue is with SQLA dialect, because the source was change to Postgre, it works, since Postgre supports Group BY 1, but Oracle does not.

@rusackas @betodealmeida Does any of you have an Oracle instance to test this? Or do you know who's the Oracle spec maintainer?

@mujohiddin Can you also share what was the previous version where this was working?

@michael-s-molina

I don't remember the exact version, it was about 6 months ago,
It works successfully with postgre, clickhouse, oracle
The last new superset I installed recently was 1 month before the project
in this case there is a problem with oracle

Sadly, I don't know of anyone with Oracle. It's still on the wish list in the rolodex, if anyone on this thread would be kind enough to sign up.

@mistercrunch
@villebro
@michael-s-molina
@betodealmeida
@john-bodley
There is no clear solution to this problem, what do you recommend to do, sirs

how can i detect the old superset version
@mistercrunch
@villebro
@michael-s-molina
@betodealmeida
@john-bodley

i have old version working with oracle but i have problems in new version, do i need to add some parts from old version or some scprits?
@mistercrunch
@villebro
@michael-s-molina
@betodealmeida
@john-bodley
@rusackas

Please suggest a solution, gentlemen
@mistercrunch
@villebro
@michael-s-molina
@betodealmeida
@john-bodley
@rusackas

I have also faced this issue with Oracle,
any updates  @rusackas , @michael-s-molina , @mistercrunch ?
Any workaround applied ? @mujohiddin

i have old version working with oracle but i have problems in new version, do i need to add some parts from old version or some scprits? exactly in which python file or class is this generation process taking place. If you tell me that, I will send it to you, and we will compare the status of the two versions
@mistercrunch
@villebro
@michael-s-molina
@betodealmeida
@john-bodley
@rusackas

I have also faced this issue with Oracle, any updates @rusackas , @michael-s-molina , @mistercrunch ? Any workaround applied ? @mujohiddin

No, the solution has not been found yet. I am waiting for a response from contributions

Let's see if our AI overlords can help at all...
@dosu-bot

Debugging this actually only requires having access to the Oracle SQLA dialect. I assume @mujohiddin you are using the default dialect included in SQLA? If so, I think we may be doing something wrong, as I'd be extremely surprised if SQLA has  bug of this magnitude..

I'm not clear on what may trigger sqlalchemy to use ordinals (GROUP BY 1) while using its group_by() clause, for anything. In my experience it will always use aliases.

Is this problem still not fixed? it was working in old version ie below versions 3.0.0, 3.0.1, 3.0.2
new version is showing this problem. I know there have been a lot of changes since then, but it's important that later we still have this problem, we need to find it.
@mistercrunch
@villebro
@michael-s-molina
@betodealmeida
@john-bodley
@rusackas

it's important that later we still have this problem, we need to find it.

100% agreed, but I don't think a single one of the people you mentioned here has an Oracle DB to test against. If you do, would you be able or willing to do some digging here? It's times like this where we can really use the help of external contributions to the project. The comment from the Dosu bot above may be accurate in that a different or custom SQLAlchemy dialect may be required, but an Oracle DB would be needed to test any of that.

Hi, I have just installed this version of superset on my server, is there any way to install an older version that does not have this problem with oracle? I get the same message at the time of making the charts.

You can git checkout any version tag and build from there.

This was working in versions 3.0.1 and 3.0.2, so you should see changes in later versions

so you should see changes in later versions

Let's of changes since then, it'd be great if someone could bissect this one. @rusackas published this recently -> https://preset.io/blog/using-git-bisect-to-find-and-fix-bugs/

@Sumeet

this bug was not present in version 3.0.1 and 3.0.2, this problem should be fixed in the new version.
this error is omitted

@sumeet-singh04

I found a word around for this: comment out this code in the superset/models/helpers.py file:
         try.
             sql = SQLStatement(sql, engine=self.db_engine_spec.engine).format()
         except SupersetParseError.
             logger.warning("Unable to parse SQL to format it, passing it as-is")

Here the sqlglot tool is used to parse the sql, the purpose is to make sure that there is only one statement, I guess it should be in order to avoid potential problems with custom SQL queries, but it shouldn't be necessary for the sql generated by superset itself

sql = SQLStatement(sql, engine=self.db_engine_spec.engine).format()

I found a word around for this: comment out this code in the superset/models/helpers.py file:
         try.
             sql = SQLStatement(sql, engine=self.db_engine_spec.engine).format()
         except SupersetParseError.
             logger.warning("Unable to parse SQL to format it, passing it as-is")

Here the sqlglot tool is used to parse the sql, the purpose is to make sure that there is only one statement, I guess it should be in order to avoid potential problems with custom SQL queries, but it shouldn't be necessary for the sql generated by superset itself

this worked for me! thank you @wangyan1856

Hello,
sorry, I also have this problem however, in the indicated file (superset/models/helpers.py) there is no ne portion of code as indicated by @wangyan1856 's comment .
By any chance with the latest version has this problem disappeared or am I doing something wrong?
Can I get more details for the resolution, even momentary?
Thank you


Add a WithValueTranslator option to Reconciller.

A Translator is a way to produces helm values based on the fetched custom
resource itself (unlike Mapper which can only see Values).
This way the code which converts the custom resource to Helm values can first
convert an Unstructured into a regular struct, and then rely on Go type
safety rather than work with a tree of maps from string to interface{}.
Thanks to having access to a Context, the code can also safely access the
network, for example in order to retrieve other resources from the k8s cluster,
when they are referenced by the custom resource.
Fixes: #113

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 19 (68.42%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 62.04%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/reconciler/internal/values/values.go
0
6
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
1685

Relevant Lines:
2716

💛  - Coveralls

There's a similar PR to this one (#93).
I mostly summarized my thoughts on this subject here.
My main area of concern is the timing of applying value overrides:

One notable semantic change would be that all of the translation and mapping would happen before overrides are applied. This would make setting the overrides less ergonomic since mapped values may otherwise be an internal implementation detail of the operator.

If a translator is in use, someone configuring overrides will need to be aware of how the CR is mapped to chart values because the overrides apply to the translator's output. This means potentially exposing internal chart implementation details to users applying overrides at runtime.

Thanks for pointing me at the overrides documentation, @joelanford - I guess I didn't really understand its purpose before.
Here's my stab at applying overrides first. PTAL
Rebased to take advantage of #112 but otherwise the first commit should be unchanged.

Applied the suggestions and squashed the commits.
I also tried to improve the docstrings a little and make sure they look sane in godoc.
Also added a nolint pragma for the use of Mapper to make our lint pass.
PTAL @joelanford


Unit file auto-fix-vscode-server.service does not exist.

This flake does not seem to produce a systemd service for me.
My flake.nix reads (shortened version)
{
  inputs = {
    nixpkgs.url = "github:NixOS/nixpkgs/nixos-23.05";
    vscode-server.url = "github:nix-community/nixos-vscode-server";
  };

  outputs = {self, nixpkgs, vscode-server}:
    {
      nixosConfigurations = {
        "myhost" = nixpkgs.lib.nixosSystem {
          system = "x86_64-linux";
          modules = [
            ./myhost.nix
            vscode-server.nixosModules.default
             ({ config, pkgs, ... }: {
               services.vscode-server.enable = true;
             })
          ];
        };
      };
    };
}

But when I try to enable the service with systemctl --user enable auto-fix-vscode-server.service (or disable, start) it says the unit file does not exist.
What am I doing wrong?

I just tried it in a VM and cannot reproduce your problem. Can you give me more context about where you are installing this? Is just a regular NixOS installation?

Had the same problem, and these instructions on the README solved it for me:

Enabling the user service creates a symlink to the Nix store, but the linked store path could be garbage collected at some point. One workaround to this particular issue is creating the following symlink:

ln -sfT /run/current-system/etc/systemd/user/auto-fix-vscode-server.service ~/.config/systemd/user/auto-fix-vscode-server.service


Add Support for Token Dictionaries

This PR adds support for dictionaries containing interesting keywords (tokens) that are useful for the mutation of inputs while fuzzing, particularly, when fuzzing syntax-aware programs (#174). This modification allows to provide the -dict flag to go-fuzz, so that the user can provide a dictionary file with useful tokens for the fuzzing campaign. E.g.:
-dict /path/dictionary.dict

The tokens parsed from the dictionary are stored in ROData.strLits, as those are the string literals that are used by the mutator engine when generating new fuzzing inputs.
The dictionary format that is accepted by the -dict flag is the same that is used by AFL/Libfuzzer (see https://github.com/google/AFL/tree/master/dictionaries).
This dictionary format defines that there is one token per line. Every line consists of a name followed by an equal sign and the token in quotes (e.g. name=”token”). It is also possible to define binary sequences by providing the values in hex (e.g. \xNN) within the token. To insert a backslash or a double quote within the token, it has to be escaped using a backslash (e.g. \\ or \”). \n and \t are recognized as well, since they might be useful for text-based protocols. Other problematic characters can be added by providing its hex value.
To make this implementation fully compatible with AFL/Libfuzzer’s dictionaries, token levels are supported. A level can be appended to every token, by appending @<num> to the keyword, e.g.
keyword@1=”token”
These tokens will be loaded only, if the dictionary level is equal to or greater than the specified number. The default dictionary level is 0, but it can be increased by appending @<num> to the dictionary path. E.g.:
-dict /path/dictionary.dict@1

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Reload/Refresh won't trigger breakpoint, but navigation would.

I am using 3.2 preview. I followed the instructions here and added breakpoint in the browser debugger. Because I want to break within OnInitializedAsync I figured the obvious way is to reload the page. But that didn't work. The breakpoint would only work if I navigate to the razor.page , for example using the navigation menu in my blazor app. I think this fact should be mentioned in the doc.

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

ID: ec2f61c7-9f51-6735-e328-3c7b495d70c3
Version Independent ID: 4d06e565-282d-6387-aa90-cb2c2986d35a
Content: Debug ASP.NET Core Blazor
Content Source: aspnetcore/blazor/debug.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Thanks @tedypranolo ... I'm going to move this over to the existing issue that pertains to updates for Blazor WebAssembly debugging. When the engineer supplies content for that issue, we should get feedback on this scenario.

@tedypranolo I'm re-opening to take a look at this separately from the 3.2 Preview 3 release.

@tedypranolo I can't repro this behavior here ... my breakpoint is hit both via navigation and page reload in the browser. I'll keep an :ear: open for further reports on this. If you think you've discovered a bug, open an issue with engineering at https://github.com/dotnet/aspnetcore/issues.


Flutter Doctor -v VersionCheckError: VersionCheckError:

Flutter crash report; please file at https://github.com/flutter/flutter/issues.
command
flutter doctor
exception
VersionCheckError: VersionCheckError: Command exited with code 128: git log -n 1 --pretty=format:%ad --date=iso
Standard error: fatal: bad config line 2 in file .git/config

@theromie
Please follow steps below:
cd your-flutter-repo (it's supposed to be a valid git repository)
 git log -n 1 --pretty=format:%ad --date=iso
cat .git/config

It seems the git repo is broken now. And stackoverflow maybe a better place to resolve this kind of support questions.


Resolve Promise

Feature Proposal
I need to be able to resolve Promises since currently I'm getting:

Possible Implementation
Just add await before evaluating the function

@avi12  I can work on this.

@avi12 @mud1tx having a look to the published picture, it seems more an issue related to the tick value generation which usually is not using nay promise, afaik.
Could you create a sample (codepen, codesandbox) to reproduce that behavior?


S3 | Join a Hunt

API Request
Acceptance Criteria

a PUT /hunts/:id route that takes a hunt id, finds it in mongo, and adds a supplied users email to the list of users who have joined the hunt (hunt.currentUsers array)

Approach

Use mongo to find by the ObjectID, add the email, and update the object in mongo

@anthonyshibitov


App Crash on x64 for a list of size 2300

I ran into a weird bug with this library. I have a list of 2300 items in the destination list and 0 items in the source list. So 2300 adds need to be generated. But the recursion never seems to end and the app crashes on a release build. Oddly, I don't hit a StackOverflowException or anything like that on debug. The call just never returns. Another bizarre thing is I don't hit this on x86. What I'm noticing is in the GenDiff function, somehow the value of j increases at some point and never comes to 0 to terminate the recursion.

Thank you for this report! Is this on Windows/Mac/Linux? Can you name the .NET version?
I’ll add this as a test, but I know my apps process changes larger than that so the bug must be more subtle...


Staffbase Connector - fixes

Staffbase Connector
Fixed description for /Media query

@microsoft-github-policy-service agree company="Staffbase"


Update URL of nginx-slim image

Since the commit of kubernetes/ingress-nginx1, the path has been
changed from images/nginx-slim to images/nginx. This commit updates
the path.
fixes #2852

This change is

/assign @lavalamp

FYI I consider contrib closed for new development; I will approve PRs adding other people into individual subdirectory OWNERS files, though, as long as they agree to eventually move the contents elsewhere.


Help with basic usage and syntax errors

All,
I'm trying to something rather basic: extract values from lm_sensors, which I can export as json-formatted data.
However, I keep running into syntax errors I don't understand.
This is a small extract of the json-data:
{
  "aquaero-hid-3-1cb1": {
    "Adapter": "HID adapter",
    "Fan 1 voltage": {
      "in0_input": 4.24
    }
  }
}

jello -s gives me:
_ = {};
_.aquaero-hid-3-1cb4 = {};
_.aquaero-hid-3-1cb4.Adapter = "HID adapter";
_.aquaero-hid-3-1cb4["Fan 1 voltage"] = {};
_.aquaero-hid-3-1cb4["Fan 1 voltage"].in0_input = 4.24;

As expected. So to get the value for the Fan1 voltage, I try:
cat x.json | jello _.aquaero-hid-3-1cb4["Fan 1 voltage"].in0_input and
cat x.json | jello '_.aquaero-hid-3-1cb4["Fan 1 voltage"].in0_input'
That, however, gives:
jello:  Query Exception:  SyntaxError
        invalid syntax (<unknown>, line 1)
        SyntaxError:  _.aquaero-hid-3-1cb4[Fan 1 voltage].in0_input
        query:  _.aquaero-hid-3-1cb4[Fan 1 voltage].in0_input
        data:  {'aquaero-hid-3-1cb4': {'Adapter': 'HID adapter', 'Fan 1 voltage': {'in0_input': 4.24}}}

And now I'm confused. Isn't the result of "jello -s" the concatenation of levels I need to get my result?
The same thing happens for names without spaces in them, btw.
Even trying to get the first level with
cat x.json | jello _.aquaero-hid-3-1cb4
results in a Syntax error.
Thanks!
Edit: After some trial and error, it seem that the "-3-1" is something the python syntax doesn't like. I can escape the spaces, but have not found a way to get around the "-3-1"....

Hi there - thanks for reporting this! I think you are running into shell escaping issues. Could you try your jello query with single quotes around it? That way the shell won't remove the double quotes in the bracket notation:
cat x.json | jello '_.aquaero-hid-3-1cb4["Fan 1 voltage"].in0_input'

hmm... that is strange, for some reason jello doesn't like the first key in dot notation:
X ~ % cat test.json | jello '_.aquaero-hid-3-1cb4'
jello:  Query Exception:  SyntaxError
        invalid decimal literal (<unknown>, line 1)
        SyntaxError:  _.aquaero-hid-3-1cb4
        query:  _.aquaero-hid-3-1cb4
        data:  {'aquaero-hid-3-1cb1': {'Adapter': 'HID adapter', 'Fan 1 voltage': {'in0_input':
            4.24}}}

X ~ % cat test.json | jello '_'
{
  "aquaero-hid-3-1cb1": {
    "Adapter": "HID adapter",
    "Fan 1 voltage": {
      "in0_input": 4.24
    }
  }
}
√ ~ % cat test.json | jello '_["aquaero-hid-3-1cb1"]'
{
  "Adapter": "HID adapter",
  "Fan 1 voltage": {
    "in0_input": 4.24
  }
}

It works in bracket notation, but I'm not sure why. I'll have to investigate.

Something about the name of the key makes the python interpreter think that could be a valid number?
When I change the dashes to underscores it works. I've never really tried using dashes in dictionary keys before so I haven't run into this. Since keys are typically strings it probably doesn't matter, but when using dot notation it's no longer a string but an object name in python so a dash in the name might not be legal.
√ ~ % cat test2.json| jello
{
  "aquaero_hid_3_1cb1": {
    "Adapter": "HID adapter",
    "Fan 1 voltage": {
      "in0_input": 4.24
    }
  }
}
√ ~ % cat test2.json| jello _.aquaero_hid_3_1cb1
{
  "Adapter": "HID adapter",
  "Fan 1 voltage": {
    "in0_input": 4.24
  }
}

I think I should fix the Schema output so it also encapsulates the key in brackets if it contains dashes. Looks like that is an easy fix.

Thanks!

To be fair, "jq" also runs into an error. Probably also because issues with shell escaping.
But yes, the dashes seem to be what's causing the issue. Sadly, as the output of lm_sensors contains them, I have no way of getting rid of them.

No worries - can't believe I haven't run into this before. I will fix this and add tests. In the mean time, the best way forward would be to use bracket syntax for keys like that.

You can use bracket notation for all keys like this:
cat x.json | jello '_.["aquaero-hid-3-1cb4"]["Fan 1 voltage"]["in0_input"]'

I have a fix for this in the dev branch. The Schema output now checks for invalid or reserved keynames and converts them to bracket syntax.

fixed in v1.5.5.


Add Sendable conformance to state structs

@rowjo I believe Color and Font in SwiftUI only got Sendable conformances recently. Can you see if it's possible to get this branch backwards compatible with Xcode 13.4.1? If not we can try to take a look soon.

@rowjo I believe Color and Font in SwiftUI only got Sendable conformances recently. Can you see if it's possible to get this branch backwards compatible with Xcode 13.4.1? If not we can try to take a look soon.

@stephencelis UUID too I believe. Sure, I've just pushed an Xcode 13.4.1 compatibility fix.

@rowjo Looks like the availability of #if check for ConfirmationDialog is not quite right on 14.1. Do you have a chance to look into it? Would love to get this merged!

@rowjo thanks for doing this, I hope you don't mind I finished this in #120.

@stephencelis It looks like @jpsim beat me to it! Thanks both, I'll close this PR.


Downloads from ./tools/registry.py don't use $HOME/.netrc

This means that if you're trying to set up an on-prem registry with entries for resources that need authentication to access the incorrect SHA is calculated for the archive, preventing the module being used at all.

Fixed in https://github.com/bazelbuild/bazel-central-registry/pull/227


[release-1.14] Update helm-tool to latest and regenerate helm readme

Depends on #6696 being backported first in #6704 ✅
Manual backport of #6701
Kind
/kind feature
Release Note
NONE

/approve
/lgtm


启动后报错

Ubuntu  1710   配置没有改过
2017-11-04 16:07:00,270 ProxyManager.py[line:47] INFO freeProxyFirst: fetch proxy <IP_ADDRESS>:8972
Process RefreshRun:
Traceback (most recent call last):
File "/usr/lib/python2.7/multiprocessing/process.py", line 267, in _bootstrap
self.run()
File "/usr/lib/python2.7/multiprocessing/process.py", line 114, in run
self._target(*self._args, **self._kwargs)
File "../Schedule/ProxyRefreshSchedule.py", line 96, in run
main()
File "../Schedule/ProxyRefreshSchedule.py", line 80, in main
p.refresh()
File "../Manager/ProxyManager.py", line 53, in refresh
if self.db.exists(proxy):
File "../DB/DbClient.py", line 93, in exists
return self.client.exists(key, **kwargs)
File "/home/yg/py/proxy_pool-master/DB/SsdbClient.py", line 95, in exists
return self.__conn.hexists(self.name, key)
File "/home/yg/.local/lib/python2.7/site-packages/redis/client.py", line 1853, in hexists
return self.execute_command('HEXISTS', name, key)
File "/home/yg/.local/lib/python2.7/site-packages/redis/client.py", line 578, in execute_command
connection.send_command(*args)
File "/home/yg/.local/lib/python2.7/site-packages/redis/connection.py", line 563, in send_command
self.send_packed_command(self.pack_command(*args))
File "/home/yg/.local/lib/python2.7/site-packages/redis/connection.py", line 538, in send_packed_command
self.connect()
File "/home/yg/.local/lib/python2.7/site-packages/redis/connection.py", line 442, in connect
raise ConnectionError(self._error_message(e))
ConnectionError: Error 111 connecting to localhost:8888. Connection refused.

yg@yg-VirtualBox:~/py/proxy_pool-master/Run$ python -V
Python 2.7.14
yg@yg-VirtualBox:~/py/proxy_pool-master/Run$ cat /proc/version
Linux version 4.13.0-16-generic (buildd@lcy01-02) (gcc version 7.2.0 (Ubuntu 7.2.0-8ubuntu2)) #19-Ubuntu SMP Wed Oct 11 18:35:14 UTC 2017

@power255 ConnectionError: Error 111 connecting to localhost:8888. Connection refused. 这是你数据库配置不对


Add TVP to check if Token Identifier can be logged

What is changing?
Introducing a new boolean TokenValidationParameter LogTokenId. By default, it set to true.
Why?
It will allow the library to log a token identifier if LogTokenId is set to true.

Should Wilson log any message when the TokenValidationParameter.LogTokenId is true, or this is only for SAL to use?

Right now, only the TokenValidationParameter (TVP) is getting introduced in Wilson. No action will be taken in Wilson based on the value. I will create a GitHub issue to log the Token Identifier in Wilson based on the TVP value.


Region based service URL feature not there in PostureManagementv2 service

Describe the bug
PostureManagementv2 service is unable to pick endpoint on the basis of environment set by the user.
To Reproduce
Set region as eu-de which is non-default, and observe the endpoint being hit by the service.
Expected behavior
It must have picked the eu-de based endpoint in the above case.
Must gather (please complete the following information):

SDK Version: 3.1.5
Go Version: 1.15.5
PostureManagementv2

fixed in v3.1.6


(KeePassXC) Unable to upgrade

Chocolatey Version
Other (note in the comments)
Chocolatey License
No response
Package Version
2.7.3 -> 2.7.4
Current Behaviour
You have keepassxc v2.7.3 installed. Version 2.7.4 is available based on your source(s).
keepassxc not upgraded. An error occurred during installation:
 Already referencing a newer version of 'chocolatey-windowsupdate.extension'.
keepassxc package files upgrade failed with exit code 1. Performing other installation steps.
The upgrade of keepassxc was NOT successful.
keepassxc not upgraded. An error occurred during installation:
 Already referencing a newer version of 'chocolatey-windowsupdate.extension'.

Expected Behaviour
To upgrade. :)
Steps To Reproduce
choco upgrade keepassxc -y
Environment
- Operating System: Windows 10
- PowerShell Version:
- Shell:

Chocolatey Log
https://gist.github.com/ireun/0191cd9b1a565803c6e20029375dc111

Anything else?
Chocolatey version 1.2.0

This upgrades fine here from 2.7.3 to 2.7.4. I suspect this is not a package issue. The log you posted doesn't have the relevant entries for keepassxc.
keepassxc v2.7.4 [Approved]
keepassxc package files upgrade completed. Performing other installation steps.
Installing 64-bit keepassxc...
keepassxc has been installed.
  keepassxc may be able to be automatically uninstalled.
 The upgrade of keepassxc was successful.
  Software installed to 'C:\Program Files\KeePassXC\'

Chocolatey upgraded 1/2 packages. 1 packages failed.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

All right, after some digging I found that in C:\ProgramData\chocolatey\lib there were two folders:

chocolatey-windowsupdate.extension
chocolatey-windowsupdate.extension.1.0

I've deleted the second one, that has fixed the issue. Thanks! :)


add potential as parent class #1093

closes #1093

@l-emele, @stap-m
I'd be happy, if one of you could review my draft PR as well as commits.
Just to make sure that I didn't forget anything before eventually pull requesting.
I also have to remaining workflow questions:
Question 1:
Is it necessary to add the term-tracker information to the moved classes?
('flow potential' and 'stock potential')
The wiki/term-tracker-item states

If there are already term tracker annotations for this term, add your term track AT THE TOP of the annotation.
But I'm unsure if this applies to restructurings as well ...

Question 2:
Protégé also updated the timestamp in 'catalog-v001.xml'.
Shall these changes be staged to?
If yes, I'll add a commit.
(I read in the documentation that it's used as config for imports, so I was wondering if this is rather a local thing.)

I am always unsure what to do with the catalog file. @jannahastings : Shall these changes be included or not?

The only time that changes to the catalog need to be committed are if we add new files, e.g. new modules, or change the way that the files are structured in directories. Otherwise, ignore it (perhaps add to your .gitignore?)

Yes, the term tracker needs to be updated.

I updated it accordingly.
Nevertheless, I'm asking myself why the changes shall be appended at the bottom.
Isn't it more reasonable to have the changes at the top in order to see the most recent ones at first?

But more importantly, also the definitions need slight updates. As these are now subclasses of potential, the definition has to be now: A flow potential is a potential that ... and A stock potential is a potential that ...

Done. (Hope so ...)
I adjusted it, so that all redundant information covered by the parent class are removed e.g.:
A flow potential is a potential that describes the input or output of a process per time unit.

... is the updated version of ...
A flow potential is a quantity value that describes
the upper limit of an input or output of a process in a spatial region of reference per unit time.

... based on ...
A potential is a quantity value that describes some upper limit in a spatial region of reference.

Is this how it is supposed to be?

Also additions/deletions/changes have to be documented in CHANGELOG.md (simply naming the class labels and PR number in the respective categories of the "unreleased" section.

Ah, this was a obvious one that I overlooked.

I am always unsure what to do with the catalog file. @jannahastings : Shall these changes be included or not?

The only time that changes to the catalog need to be committed are if we add new files, e.g. new modules, or change the way that the files are structured in directories. Otherwise, ignore it (perhaps add to your .gitignore?)

I also added a "quick tip" to the CONTRIBUTING.md stating this.
In case this is no good idea, we can simply drop the specific commit.

I'd put it even shorter...

Perfect. I was unsure how much simpler it shall be.

@l-emele Could you re-review this PR?
(I'm not sure if my re-request worked)

@l-emele
The proposed adjustments unfortunately break the oeo-physical import.
I verified the online failure with an offline test in Protégé – same result.
As I did my last adjustment in the GitHub WebEditor, I also crossed checked, if the error persists when using a offline editor – same result.
All my previous changes were made in Protégé, so the full iri was inserted by the programm.
Is there a configuration option, that I'm not aware of, to prevent this?
Can you indicate a solution how to incorporate these changes without breaking the oeo-physical-file?

Hm, not good if this brakes things. I just looked through the whole file. In most cases, there is just the ID (like OEO_0000abcd), but in few cases there are there are the full IRIs (like <http://openenergy-platform.org/ontology/oeo/oeo-physical/OEO_00010239>). @jannahastings : Any idea, why this is?
If we don't find a quick solution, I suggest that you ignore my last comment and revert to the working version with the IRIs.

I think this relates to the "specified IRI" from Protégé.
While working on #1112 I noticed that the active IRI was inserted.
I'll come back to this and test if this is the case after finishing #1112.


Add strategy option to configure globals-only and classes-only

Hi there! The https://github.com/tailwindlabs/tailwindcss-forms plugin by default applies global styles and scans for specific form-* classes to apply styles, and also accepts a strategy option to configure using classes-only or globals-only.
This PR adds the same option here, and also removes the dependency on mini-svg-data-uri and improves runtime performance by precomputing and then hard coding the resulting data URLs (this part is contained in the last commit, feel free to drop it if you prefer to keep it as-is).
I've published the contents of this branch to https://www.npmjs.com/package/@lewisl9029/unocss-preset-forms for testing, and it seems to be working as intended so far. Feel free to try it out yourself as well, and let me know if there's anything you'd like me to change. Cheers!
P.S. The diffs seem to get a lot cleaner with whitespace-hiding enabled: https://github.com/Julien-R44/unocss-preset-forms/pull/6/files?w=1

Hey, thanks for your PR and sorry for sitting on it for so long!
I'm not sure that's something we want for the following two reasons:

I tend to agree with the tailwind team when they say that preset-forms should be seen as a form reset and not as a collection of classes to be applied
This has made the code quite complex. Honestly, it has become difficult to read

However, I'm not totally opposed to adding this feature. On the other hand, I think we really have to make the code cleaner to add it
I think we can also do without the performance optimisations on mini-svg-data-uri. We don't have a performance problem in the first place, so I tend to say, why bother optimising it? I always prefer readability and simplicity to performance when it's not critical
I'm sorry if I'm being boring, especially as I've kept you waiting for so long

Hey Julien, no worries on the timing, and appreciate the feedback!
Re: making the code harder to read. Totally agree, but I'm struggling a bit to see how to make it better. Most of the changes here I see as necessary to enable the functionality. Do you have any specific suggestions on changes that would make this easier to read for you? Happy to implement anything you suggest. :)
Re: the svg optimizations. Happy to drop the commit as mentioned. Though for context, I'm using UnoCSS to implement Tailwind compatibility for my app (https://reflame.app), instead of using Tailwind directly, precisely for it's performance advantage, and nothing else. So performance is actually critical for my use case. Though I totally appreciate that the changes here make the library harder to maintain, so I'm happy to move it over to a personal fork and take the maintenance burden off your shoulders. :)

Cool! About the optimization thing : let's keep it. We'll just separate this out in another file
As for the rest, and the fact that the code isn't very readable: I'm going to play around a bit with the PR between tonight and tomorrow and make some quick changes

Should be all good now. Gonna merge and release it now
Please feel free to give any feedback if you encounter any issue!
Thanks again for the PR


Detect chunk's build version is out of a date on client

I'm submitting a feature request
Definetely (with no doubts)
Webpack version:
1.10.x
Please tell us about your environment:
Linux (doesn't matter in this case)
Current behavior:
Webpack don't check if loaded chunk is belong to current build version.
Expected/desired behavior:
Webpack checks chunk's build version when it loaded.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem along with a gist/jsbin of your webpack configuration.

Create entry point with require.ensure on any button click for example
Compile build for production (we've got main.js and 1.chunk.js)
Open main page in browser (and keeps opened for a while)
add new dependency to entry point
Compile bundle again
Return to opened page in browser and click button to load 1.chunk.js
Loaded chunk contain same modules, with new index numbers (because of new dependeny at the entry point)

What is the expected behavior?
Webpack signed each chunk with build version hash.
Each time when webpack loads new chunk it checks build version, if chunks version is out of date, webpack throw new Error('Webpack: loaded chunk is out of a date'), and trigger some event anywhere (with preventDefault and stopPropagation methods).
What is the motivation / use case for changing the behavior?
This allow to prevent errors due to outdated chunks, and provide simple interface to handle this exceptions.

Use the webpack hashing feature to avoid this problem. We won't add runtime checks.


Node template not getting put into a group

I just installed aegis for the node template functionality. I'm happy to say that I was able to capture a set of nodes and save as a template. However, it doesn't seem to add those nodes to a "group" (colored box), like it shows in the demo.

nvm. not sure why it wasn't working before, but it's working now.


Zombies with Zombie Awareness

Zombie Awareness overrides the zombies from running away, once they start tracking a player they seem to ignore the scarecrows entirely, even when right next to the Scary Scarecrow and Super variant, they just group up and attack the player.
Possibly bug, or could just need to add compatibility between them? Figured i'd bring it up

Can't seem to reproduce. When I lure a zombie into the range of a scarecrow, he'll run away, then try to attack me again once he is outside the range of the scarecrow. Could you please give me some steps so I can reproduce the issue?


Improve user settings capability.

Because I was implementing my own allocator, who had static store duration, i needed cpp file to be added to build process.

Are you using Box2D as a shared library (DLL)? If not, can you use extern?

Are you using Box2D as a shared library (DLL)? If not, can you use extern?

I'm using it as shared library. Therefore extern won't work.

I'm using it as shared library. Therefore extern won't work.

Why are you using it as a shared library?

Why are you using it as a shared library?

It is faster on build time. I need to rebuild my program each time I change something. With shared library linker is faster as it don't need check if all symbols exists and connect them.

It is faster on build time. I need to rebuild my program each time I change something. With shared library linker is faster as it don't need check if all symbols exists and connect them.

How much faster is linking that way? How about the overhead of loading the shared library when you launch? Box2D is a smallish library.

How much faster is linking that way? How about the overhead of loading the shared library when you launch? Box2D is a smallish library.

I'm not happy supporting box2d as a shared library. Too much feature creep. I would rather work on simulation and features and not build system stuff.


Remove moot version property from bower.json

Per bower/bower.json-spec@a325da3
Also their maintainer says they probably won't ever use it: http://stackoverflow.com/questions/24844901/bowers-bower-json-file-version-property

Thanks for the pull request, but I'm not going to be merging this in at the moment. It's nice documentation to have this in the bower.json imo, even if it's not used for anything atm. Also, we haven't been using tagging yet (hopefully will with the next beta), so I'd hate to remove it in case one of us fat fingers/brain farts on this when we do drop the next beta. We'll try to remember to re-open this if we do merge it in the future.
Thanks!


Rhino crash when clicking on GH Menu -> Speckle2 ->Open Speckle Manager

Clicking on the Speckle2 menu -> Open Speckle Manager crashes Rhino
Rhino for Mac Version: 7.23.22282.13002, 7.24.22301.23002
Speckle Version: 2.10.0-rc1, rc3
MBP M1, MBP 2015
edit: this does not occur on Windows with the same Speckle release

Should be fixed in the next rc release


Exit viewer on control + c in terminal

When visualizing anything and exiting the python program using control + c it doesn't close the screen and also doesn't kill the python code
It only closes after focusing & clicking in the window again with the following prompt in the terminal afterwards:
Caught a Ctrl-C within python, exiting program.
Is there a way to automatically stop the visualizer after exiting the python program via the terminal?
On a custom VTK viewer we implemented using PyQT the following line seems to do the trick.
# This assures the app quits on CTRL-C (when not in focus)
signal.signal(signal.SIGINT, signal.SIG_DFL)

Visualization code example
        show(
            vedo_meshes,
            camera=camera,
            N=len(vedo_meshes),
            bg="black",
        ).close()

Thanks

I just tested and putting the following snippet at the top of the plotter.py file ensures that the code terminates on control-c in the terminal.
Not sure if this would be a desired solution or if there are other options
import signal

# This assures the app quits on CTRL-C (when not in focus)
signal.signal(signal.SIGINT, signal.SIG_DFL)

Hi, if you do CTRL-C focusing on the terminal that should terminate the app (maybe after some delay).
CTRL-C focusing on the rendering window will do nothing (although it would be good to have the same).
The way to close the window while focusing on it is to press q.


Update role department

Describe your changes
Update role and department to allow all chars not just alphanumerical
Issue ticket number and link
resolves #183
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] If it is a core feature, I have added thorough tests.

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

Files with missing lines
Coverage Δ
Complexity Δ

...in/java/seedu/address/model/person/Department.java
100.00% <ø> (ø)
7.00 <0.00> (ø)

src/main/java/seedu/address/model/person/Role.java
100.00% <ø> (ø)
7.00 <0.00> (ø)


Implement inlining of height attributes.

Fixes #167

Looks good to me, but can you update the readme as well?


Many python binding functions are missing default arguments

See for example these bindings from the main imgui API:
def input_text(label, p_str, flags=0, callback, *args, **kwargs): # real signature unknown; NOTE: unreliably restored from __doc__
    """ input_text(label: str, str: str, flags: int = 0, callback: Callable[[imgui_bundle._imgui_bundle.imgui.InputTextCallbackData], int] = None, user_data: capsule = None) -> tuple[bool, str] """
    pass

def input_text_with_hint(label, hint, p_str, flags=0, callback, *args, **kwargs): # real signature unknown; NOTE: unreliably restored from __doc__
    """ input_text_with_hint(label: str, hint: str, str: str, flags: int = 0, callback: Callable[[imgui_bundle._imgui_bundle.imgui.InputTextCallbackData], int] = None, user_data: capsule = None) -> tuple[bool, str] """
    pass

In this case, it seems like callback should have a default argument such as None

Even button appears to be missing a default value for size, so all buttons are flagged with errors in my IDE
def button(label, size, *args, **kwargs): # real signature unknown; NOTE: unreliably restored from __doc__
    """
    button(label: str, size: imgui_bundle._imgui_bundle.imgui.ImVec2 = <imgui_bundle._imgui_bundle.imgui.ImVec2 object at 0x7fe9872b76b0>) -> bool

    button
    """
    pass

Hello,
What you are pointing here are likely definitions inferred from you IDE, probably by a tool such as stubgen
The text "NOTE: unreliably restored from doc") is an indication of it.
ImGui Bundle comes with fully typed and documented stubs: look at this for example
https://github.com/pthom/imgui_bundle/blob/564cf51ebeba07e9d3173c62499522c36afbdae3/bindings/imgui_bundle/imgui/__init__.pyi#L11103-L11109
You should look inside your IDE on how to tell it not to use auto-generated stubs but rather the official ones.

Hmmm I followed the instructions provided by JetBrains and it's still generating its own stubs no matter what I do.

 After doing what I showed in the image above, you also have to correct the imports, and pycharm will still never propose the correct import so from now on we have to add the imports manually rather than using the keyboard shortcut.
from imgui_bundle import imgui

But that still wasn't enough, something was utterly broken with my venv and it still wouldn't locate this import. After recreating the venv from scratch and starting with this package, it now works.

OK this issue arose again after some other venv management shanenigans and I found out that the issue is most likely that the venv absolutely has to be called .venv or it can lead to issues with PyCharm. Just noting down in case somebody else ever stumbles upon this.


Size Data Portal instance types for WMG usage

Acceptance Criteria:

Dev, staging, prod envs clusters updated via TF changes, as needed to meet performance requirements, as measured by performance tests

Tasks:

Run performance tests from previous work, above. Determine new CPU/memory requirements for Data Portal backend instance types.
Make infra updates and deploy changes, as needed.

Current state is that instances are very large (and very expensive), due to inefficiency of the /primary_dimensions_filter endpoint.  This story is responsible for simply downsizing the instance types such that annual spend on instances <$250K/yr.  Doing this will depend upon planned work (story pending) to pre-compute, and store the primary dimension values in a new static snapshot file, allowing the BE API to efficiently load this data without issuing a massive query to the TileDB expr summary cube.  (cc: @maniarathi)


Implement numpy.flip

@diorcety there's only one validation issue: you should apply clang-format-3.6 to you cpp files.

Thxs! Did you test the perf compared to numpy.flip?

ok, it turns out flip is implemented as an array view so O(1) in numpy. We are much slower....

Oh, I didn't think to that. I started from the roll source, i didn't know that was done by view and how to do this in pythran


NET 8 there is some issue with package Microsoft.Azure.Functions.Worker.Sdk version 1.17.0 and 1.17.1 when run with some other function dependencies. same code working fine with same code with version 1.16.4 .

Description
NET 8 there is some issue with package Microsoft.Azure.Functions.Worker.Sdk version 1.17.0 and 1.17.1 when run with some other function dependencies. Http trigger and timer trigger working fine. But Queue trigger is throwing error. same code for Queue trigger working fine with version 1.16.4 .
Other packages are latest version.
Steps to reproduce
please check this SQ thread. https://stackoverflow.com/questions/78067439/net-8-function-app-doesnt-load-functions-from-dependencies/78074480?noredirect=1#comment137650419_78074480
v1.17.1 error: https://i.imgur.com/jKf9RJL.png
v1.17.0 sometimes works fine (https://i.imgur.com/R3IWL95.png) sometime throws error (https://i.imgur.com/DoNNG6n.png)
lower version are working fine. v1.16.4

Thanks for reporting. We will take a look and get back to you as soon as we can.

Hi @Shandilyavivek can you upgrade to SDK version 1.17.2 and see if that resolves your issue?

@satvu Thanks, I used v1.17.2 and it fixed the issue.

Hi @satvu if possible, Could you please explain the reason why v1.17.0 and v1.17.1 was causing the issue?

Hi @satvu if possible, Could you please explain the reason why v1.17.0 and v1.17.1 was causing the issue?

https://github.com/Azure/azure-functions-dotnet-worker/pull/1946 was the root cause of all of these recently reported issues so they rolled that commit back in 1.17.2. see https://github.com/Azure/azure-functions-dotnet-worker/releases/tag/sdk-1.17.2 for details.

Closing as resolved.


[BUG]  journals with negative journal-interval-index are created  before  time  visible in caldendar before start date

Great plugin, I am switching to it from periodic notes :)
I want to create a custom weekly  journal  with the data starting from today (or any other day) and I have active calendar journals.
The coustom journals are created for preceding time for all the times they are visible in the calendar, so I can have custom-journal -1, custom-journal -2, et cetera

I only want to have  journals starting from 1
I am on Obsidian 1.5.11 and MacOS

Expected behaviour - the  journal notes should  not be created for period before start;
Also but this might be a FR - there should be a stop date that could be defined in settings for such calendar.

good point, I missed this somehow, will fix so that for interval journals with increasing index there won't be 0 and negative numbers. (I don't want to just limit by start date for cases then you start journaling not with index 1 you might still want go back till 1)

hi @srg-kostyrko thank for looking into it!
I (someone) might want to have a journal with dates (no numbers) and still  retain this behaviour;
For example the number is not important but I want this journal to be weekly meetings about a project that spans only specific time (like 3 or 6 months) and I want to quickly find a note from specific data, the order of meetings/notes is not important for me
But returning to the initial issue, could it be solved via  additional toggle in the settings  (like "strict enforce starting (and closing if you implent it) date(s)")? Or "Limit note creation by starting date");
I will create another feature request to ask about ending date :)
This way you will also get consistent behaviour independently from naming convention, which might be more intuitive to user and cover more cases

yeh that's a good point
than I'll add a toggle to limit journal entries by start and dates

released in 1.3.0


Understanding the terminal output during training

Can anyone help me understand the terminal output during training? I followed these steps: https://github.com/WillBrennan/SemanticSegmentation#training-new-projects I only adjusted minor things.
My output looks like this (with some truncation, line breaks and indentation on my part for clarity):
python train.py --train datasets/my_cat_images_train --val datasets/my_cat_images_val --model-tag segmentation_cat --model-type FCNResNet101
INFO:root:running training on cuda
INFO:root:creating dataset and data loaders
INFO:root:loaded 4114 annotations from datasets/my_cat_images_train
INFO:root:use augmentation: True
INFO:root:categories: ['cat']
INFO:root:loaded 184 annotations from datasets/my_cat_images_val
INFO:root:use augmentation: False
INFO:root:categories: ['cat']
INFO:root:creating dataloaders with 19 workers and a batch-size of 6
INFO:root:creating FCNResNet101 and optimizer with initial lr of 0.0001
INFO:root:creating model with categories: ['cat']
INFO:root:creating trainer and evaluator engines
INFO:root:creating summary writer with tag segmentation_cat
INFO:root:attaching lr scheduler
INFO:root:attaching event driven calls
INFO:root:training...
	INFO:ignite.engine.engine.Engine:Engine run starting with max_epochs=2.

INFO:root:epoch[1] - iteration[500/685] loss: 0.145
INFO:root:epoch: 1 - current lr: 0.0001
	INFO:ignite.engine.engine.Engine:Engine run starting with max_epochs=1.
	INFO:ignite.engine.engine.Engine:Epoch[1] Complete. Time taken: 00:00:09
	INFO:ignite.engine.engine.Engine:Engine run complete. Time taken: 00:00:09
INFO:root:loss: 0.147 precision: 0.833 recall: 0.880<EMAIL_ADDRESS>0.724<EMAIL_ADDRESS>0.748 Epoch: 1
	INFO:ignite.engine.engine.Engine:Engine run starting with max_epochs=1.
	INFO:ignite.engine.engine.Engine:Epoch[1] Complete. Time taken: 00:00:09
	INFO:ignite.engine.engine.Engine:Engine run complete. Time taken: 00:00:09
INFO:root:loss: 0.123 precision: 0.873 recall: 0.911<EMAIL_ADDRESS>0.781<EMAIL_ADDRESS>0.804 Epoch: 1
	INFO:ignite.engine.engine.Engine:Epoch[1] Complete. Time taken: 00:10:52

INFO:root:epoch[2] - iteration[500/685] loss: 0.416
INFO:root:epoch: 2 - current lr: 9e-05
	INFO:ignite.engine.engine.Engine:Engine run starting with max_epochs=1.
	INFO:ignite.engine.engine.Engine:Epoch[1] Complete. Time taken: 00:00:09
	INFO:ignite.engine.engine.Engine:Engine run complete. Time taken: 00:00:09
INFO:root:loss: 0.161 precision: 0.854 recall: 0.880<EMAIL_ADDRESS>0.745<EMAIL_ADDRESS>0.765 Epoch: 2
	INFO:ignite.engine.engine.Engine:Engine run starting with max_epochs=1.
	INFO:ignite.engine.engine.Engine:Epoch[1] Complete. Time taken: 00:00:09
	INFO:ignite.engine.engine.Engine:Engine run complete. Time taken: 00:00:09
INFO:root:loss: 0.116 precision: 0.872 recall: 0.922<EMAIL_ADDRESS>0.787<EMAIL_ADDRESS>0.812 Epoch: 2
	INFO:ignite.engine.engine.Engine:Epoch[2] Complete. Time taken: 00:10:52
	INFO:ignite.engine.engine.Engine:Engine run complete. Time taken: 00:21:45

Here's what I'd like to know about it:

The outputs in the training starting with "INFO:root:" are apparently defined in semantic_segmentation/engines.py, but where do the others ("INFO:ignite.engine.engine:") come from?
Why are two epochs run through as desired, but after the initially correct specification (namely "max_epochs=2"), "max_epochs=1" is then output several times?
Why is "Epoch[1]" output several times during the second epoch?
Why is it stated several times in the middle of an epoch that the epoch or the run is "complete" and a new run would be started?
What does the time information "Time taken: 00:00:09" refer to, because the real runtime is significantly longer?
Why are there intermediate results for the metrics during the epochs, although the function should only be executed when the epoch is finished? And why are there only the results of the second output per epoch in the tensorboard? https://github.com/WillBrennan/SemanticSegmentation/blob/014ea13fa9f35bef08e29b50d01c64893943e899/semantic_segmentation/engines.py#L55

Wall time,Step,Value
1667398091.2036688,1,0.8042078614234924
1667398743.5209541,2,0.8115814328193665

Possibly part of my questions come from the fact that I had only used Tensorflow and not Torch before. I thank you in advance if someone can make me understand at least part of these oddities.

Hey;

Why are two epochs run through as desired, but after the initially correct specification (namely "max_epochs=2"), "max_epochs=1" is then output several times?

So this happens as we do "one epoch" for each validation dataset at the end of each training epoch. This code also computes the validation metrics on the training set as well to make it easier to see if its overfitting.

What does the time information "Time taken: 00:00:09" refer to, because the real runtime is significantly longer?
This is why you're seeing different times for completion as well, when its referring to the shorter times its how long it takes for an "epoch" on the validation sets.

The outputs in the training starting with "INFO:root:" are apparently defined in semantic_segmentation/engines.py, but where do the others ("INFO:ignite.engine.engine:") come from?

These other logging statements come from within ignite

It definitely should be; it'll be a fairly simple change here if you want to make a PR? https://github.com/WillBrennan/SemanticSegmentation/blob/master/semantic_segmentation/engines.py#L60

Here it is: https://github.com/saskra/SemanticSegmentation/pull/1

Ah to clarify; when the metrics are calculated on the training dataset it is still performing augmentation. This is why the validation dataset performs better.

Ah, now it makes sense! :-)


As an employee that is working for an organisation that requires reporting on population information I want to create a report for all the top N populated countries in a region where N is provided by the user.

SELECT country.name, country.region
FROM country
WHERE country.region = ‘name of the region’;
ORDER BY city.population DESC
LIMIT :N;
public Country getUseCase6(String region, int n)
{
try
{
// Create an SQL statement
Statement stmt = con.createStatement();
// Create string for SQL statement
String strSelect =
“SELECT country.name, country.region”

“FROM country”
“WHERE country.region =” + region
“ORDER BY city.population DESC”
“LIMIT: ” + n;
// Execute SQL statement
ResultSet rset = stmt.executeQuery(strSelect);
// Return new country if valid.
// Check one is returned
if (rset.next())
{
Country country2 = new Country();
country2.name = rset.getString("name");
country2.region = rset.getString("region");
return country2;
}
else
return null;
}
catch (Exception e)
{
System.out.println(e.getMessage());
System.out.println("Failed to get country details of a region");
return null;
}
}


Check product store is ready on startup

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
fixes #336

License
Apache 2.0

What's in this PR?
Introduces a Ready function to the CloudInfoStore interface and checks on start that this returns true for the configured product store, failing with an appropriate message.
Implements Ready for cassandra and redis product stores by writing and reading a key from the underlying store to check the store is ready, and for in-memory just returns true.
Additional context
Tested by killing redis then starting cloudinfo - can see it fail as expected:
INFO[0000] starting application                          application=cloudinfo arch=amd64 build_date="2020-06-23T12:58:36+0100" commit_hash=83911af compiler=gc environment=development go_version=go1.14 os=darwin version=fail-on-store-error
INFO[0000] using Redis as product store                  application=cloudinfo environment=development
ERRO[0000] failed to set key to value                    application=cloudinfo cistore=redis environment=development key=/banzaicloud.com/cloudinfo/ready value="map[ready:true]"
ERRO[0000] configured product store not available        application=cloudinfo environment=development

Works normally if redis is up and running.
Checklist

[x] Code meets the Developer Guide

To Do

[ ] I'd appreciate it if someone can verify that this approach (i.e. stopping the cloudinfo process from starting rather than a pause/wait loop for the store to become available) is the best solution.
[ ] Sanity check that cassandra works correctly - I've not been able to get the cassandra container to start correctly here so haven't been able to test that (make up creates the container but it then stops with no output).

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Not sure I can fix the CI issue - I think that it's not picking up the branch name correctly for the pull request when building the docker image.

@sagikazarmark Any chance you could take a look at this? I'm happy to change it up to wait for the product store to be ready instead of terminating if that's more appropriate; let me know what you think.

@mrsheepuk sorry, I forgot about this. I'll take a look at it later today.

Okay, I went ahead and actually looked at it.
One question popped into my mind: what do you think about a ping-like check to test the connection?
Redis actually has a PING command, Cassandra probably has something similar as well (or at least a dummy query).
I guess I can actually guess the answer for that question: testing that you can actually write into the store is what ultimately ensures that the following writes can complete, but I think it's a very edge case where a store responds to a ping, but it's not ready for writing after that.
What do you think?

What do you think?

Certainly sounds like a reasonable approach - I'll have a quick look and see if it's trivial to implement that. Do you think the best approach is to terminate if the store isn't ready (as I've got here) or to ping/back-off/wait/retry indefinitely 'in process'?

Good question. I think it makes sense to have some sort of backoff, since most of these issues are caused by ordering.
I'd probably add a --wait-for-store flag/env/config and have some other config for the backoff. This could probably part of a different PR though. I think the current approach is a great start and should solve the immediate issue quickly. We can iterate later with a backoff config.

@mrsheepuk #340 fixes the tests

@mrsheepuk #340 fixes the tests

Thanks! I've rebased onto the latest master and pushed that so hopefully that will sort the CI issue.  I'll take a look at the ping operation at some point today.

@sagikazarmark I've updated to use PING for redis and to simply use the existing initSession for cassandra - works perfectly, tested with both redis and cassandra; cloudinfo exits as expected if the configured store is not available or not ready, and continues if the product store is working. Take a look and let me know if there's anything else needed for this one.

Thanks @mrsheepuk. Looks good to me!

@mrsheepuk I'll deploy this change to our test environment and tag a new release if it passes all the tests.


Transaction fails on entering otp

Flutterwave mobilemoneyug fails after OTP it says An Error occured And after inspecting the response code is 400(Bad Request)

This is an API error. Please send us the details of the transaction attempt via mail

Actually, it is solved. Thank You!


Keys.plist

Adds a new untracked plist file to store data like API keys.

Onus is on each dev to supply their own keys.plist in order to build the project. I kinda like the idea of submitting a keys.plist file with dummy values so at least the dev knows what keys need to be filled out.
Either way, whatever direction you go in :+1:

Hmm, I suppose we could commit an empty keys.plist into the repo.. and then add it into .gitignore once it exists in the repo?

I think you can still include it in this PR if you want by just force committing the dummy keys.plist

@jonuy Updated, now including a dummy keys.plist

:+1:

Making changes to keys.plist still shows up as tracked :(
Per http://stackoverflow.com/questions/7072213/how-to-stop-tracking-a-file-without-removing-it-from-repo, there is a suggestion to create a dummy file .. there's also an answer to use git update-index --assume-unchanged <file> locally, but the index will be cleared upon a git reset --hard ...


Page URL: Fix handling of base url and reverse proxy forward prefix

The url property of of the inertia HTTP response should contain the user-facing request URI, i.e. the absolute URL path and if applicable the query string.
Inertia currently has a bug where, if the laravel application has base url different from root, the url property contains the baseUrl twice (https://github.com/inertiajs/inertia-laravel/pull/372, https://github.com/inertiajs/inertia-laravel/issues/359, https://github.com/inertiajs/inertia-laravel/issues/421). This was introduced as a side-effect of a fix for a separate issue when the laravel application is deployed behind a reverse proxy, and where the reverse proxy has a forward prefix (https://github.com/inertiajs/inertia-laravel/pull/333).
This PR fixes the handling of the baseUrl case and preserves the behavior in case of reverse proxy with forward prefix. It also adds test cases for both cases.
Example: Base URL duplicated by Inertia response
There is a thread on laracasts, URL duplicates in shared hosting deployed Inertia App, which illustrates the base url problem:

However, the URL in the browser is rewritten whenever I access a page.
https://www.mydomainname.com/project turns into https://www.mydomainname.com/project/project
The same thing happens with the other route. https://www.mydomainname.com/project/github turns into https://www.mydomainname.com/project/project/github
Checking with vue devtools, Inertia.initialPage.url seems to be set to the incorrect value (the one with a duplicate)

This problem happens because the $request->getRequestUri() contains the full request URI as sent to the application server, which also includes the base url:
https://github.com/inertiajs/inertia-laravel/blob/12e7937e99f6bdb114541c7f907bef1339b1c843/src/Response.php#L102
There are more users who experienced this problem, e.g. root directory is displayed twice in URL on load and reload
Example: Reverse Proxy Deployment with Forward Prefix
Suppose the application has a public URL https://intranet.example.org/to/project. This is served through a reverse proxy, which requests the content in the background from http://localhost:8080.
Using the X-Forwarded-Prefix header, the reverse proxy can instruct symfony to prepend the forward prefix to the baseUrl. This was the setting reported in https://github.com/inertiajs/inertia-laravel/pull/333.
Since the forward prefix is not included in the getRequestUri() the problem of a duplicated base url does not arise in this case.

Related PRs
I saw there are a couple of open pull requests here addressing the same issue:

https://github.com/inertiajs/inertia-laravel/pull/446 and https://github.com/inertiajs/inertia-laravel/pull/422 both propose reverting https://github.com/inertiajs/inertia-laravel/pull/333 and setting 'url' => $request->getRequestUri(). This would cause the issue with the forward prefix in the reverse proxy setting to re-appear.

https://github.com/inertiajs/inertia-laravel/pull/437 suggests 'url' => $request->fullUrl() which solves both the base url and forward prefix issues, but as a side effect changes the url property to include the full URL, including protocol and host part. This causes other tests to fail.

Finally, https://github.com/inertiajs/inertia-laravel/pull/360 suggests basically the same change that I am proposing here ('url' => $request->getBaseUrl().$request->getPathInfo().$queryString), which works both the base url and forward prefix cases. It is only missing a test case for the forward prefix case.

User impact
Additionally to the examples above, the "inertia duplicates the base url" problem also came up in different forms on stackoverflow, e.g. Ziggy router modifying browser urls or root directory is displayed twice in URL on load and reload
Many thanks to the entire Inertia team for this amazing project! I hope this PR can contribute to solving one of the very few longstanding issues.

At this point i've just given up for any PR that fixes this problem to be merged. Sadly, i've seen a lot of them being closed or just ignored.

Following the discussion with @RobertBoes, I changed the code to make explicit that - in line with the inertia protocol example - the url property should not contain the HTTP scheme and host name.
The code now reads:
 'url' => str_replace($request->getSchemeAndHttpHost(), '', $request->getUri()),

I would have liked use the fullUrl method, but this runs into the problem that when accessing the root URL, the fullUrl method does not return a trailing slash (so one would have to write str_replace($request->getSchemeAndHttpHost(), '', $request->fullUrl()) ?: '/', which is a bit ugly).

Thanks for the PR, and I'm sorry for the delay! I ended up going with a different but very similar approach (#592), but this PR was very helpful!

laravel docker nginx subpath
i fix  Response.php     'url' => Str::after($request->fullUrl(), $request->getSchemeAndHttpHost()),
on index page is work buy  click link login url is lost


Training Issue: Cannot locate image

Describe the bug
Says the it cannot locate image_1920.png. The images in the folder go up to image_1919.png.
To Reproduce
Steps to reproduce the behavior:
Running the training guidline notebook. We used a modified version.
Desktop (please complete the following information):

OS: Linux Solus Budgie
Version 4.1

error.txt
training_guideline.txt

Hi @Paasu! It sounds like to me that it's looking for an image patch beyond what you generated. Not sure why this would come up in the training guidelines notebook. To help debug, could you please share the following things:
1 - The branch that you're using the code on/
2- The git commit # if you know it
3- Are you using some of the data that we provided, or your own data? If you're using your own, would you mind sharing a raw image and raw mask?
4- The training_guideline exported as a iPython Notebook file (*.ipynb).
Thanks!

Hi, thank you for responding.

The branch we used was axondeepseg-master.
We didn't use git, we just cloned from github
We are using our own data.

^ Above is one of the images we are using in training. I was unsure what the mask image was meant to be, I assumed it to be a copy of the raw image renamed mask.png.
We used training_guidline as a python file because for some reason jupyter wasn't working.
training_guideline.zip

Thank, hope everything goes well!

Hi @mathieuboudreau ,
Here are the tests I have done so far:

I have run the guide_dataset_building notebook to generate patches from the image.png and mask.png from data_test in AxonDeepSeg.
Then I ran the notebook training_guideline sent by Paasu on the master branch.
I was unable to reproduce the problem while training, all the images are located correctly.

Hello everyone,
I have fixed the issue. I attempted everything a second time but on a different OS (OpenSUSE Leap 15.1) and so far the training works, the final test display of an image will not work however, I will post a new issue for that.
Thank you everyone for the help!


allow all secrets to be overridden by more qualified paths

The visibility should be overridable with a more specific path, e.g. if visibility is channel, it can be overridden by host. Secrets will be generated still at channel, but the host can be manually generated.

Let it be on demand.


After updated when upload image always crop file why?

We only create a cropped thumbnail. The original image will not be cropped. Setting should_create_thumbnails to false can disable this feature.


Updating readme from branch example

Update
I updated the readme file. Please review and accept.

Thoughts
I probably shouldn't , but okay. :+1:


WebGL Renderer is showing black tiles

We are experimenting with the new WebGL renderer on the master branch.
We have OSD rendering our tile sources correctly when drawer: 'canvas'.  However, as soon as we switch to webgl the tiles all appear as completely black.
This is using a custom tile source (we have a custom implementation for getTileUrl), and the tiles are correctly fetching in the browser.  It looks like the image data is correctly making it to _onTileLoad, so it doesn't seem likely this has something to do with our implementation of getTileUrl.  With debug mode on we can further confirm that the correct metadata about tiles / levels are there, they just aren't rendering correctly.
This is a basic demonstration of what we're doing:
const viewer = OpenSeadragon({
  debugMode: true,
  loadTilesWithAjax: false,
  crossOriginPolicy: false,
  drawer: 'webgl',
  id: 'contentDiv',
  tileSources: [
    {
      tileSource: {
        getTileUrl,
        height: 65536,
        width: 131072,
        tileSize: 256,
        tileHeight: 256,
        tileWidth: 256,
        minLevel: 0,
        maxLevel: 8,
      },
    },
  ],
});

Is there some additional configuration we need to add to get the webgl renderer working with a custom tile source?

Can you provide logs from the console or a reproducible example (e.g. your tilesource + possibly accees to the data if required)? Hard to tell just from this. WebGL1 has some issues with constrains on the texture data, I remember being once forced to convert my textures to rectangles to correctly upload them, it might be a problem if your data is not perfectly rectangular. So hard to say from such little information. WebGL2 support + modularization are planned.

I've seen this with cross-origin data. WebGL doesn't let you upload data to the graphics card from a tainted canvas/cross-origin image.
Try crossOrigin: 'Anonymous' in your config and see if that helps. If the server doesn't allow that, I think you will need to stick to drawer: 'canvas'.

I realized this is likely to be a fairly common occurrence, and that there is a fairly straightforward to handle it with our current architecture.
I created a PR that detects when the WebGLDrawer cannot draw a TiledImage because of tainted data (cross-origin without CORS enabled), and falls back to using the CanvasDrawer specifically for those images.
Of course, you'll lose all the benefits of the WebGLDrawer like absence of tiling artifacts, filters (in the future), etc. But at least the viewer won't just fail to render.

BTW, @thec0keman thank you for testing the new code!

Fix for CORS worked great.  Thanks!

What about this? https://blog.chromium.org/2011/07/using-cross-domain-images-in-webgl-and.html?m=1

@Aiosa I believe what the blog post you linked is saying is that the image needs to be fetched with CORS enabled in order to be not tainted, and therefore be usable by WebGL. This is done on an <img> tag by setting crossOrigin = 'Anonymous', which is what we do in OSD with crossOriginPolicy: 'Anonymous'. The server also has to be configured to allow this.
This is what I suggested above, and it sounds like it worked for @thec0keman. Cross-origin data is just fine as long as CORS is used, and is drawn by WebGLDrawer without a problem. The internal canvas drawer accounts for the other scenario - when CORS is disallowed by the server (or by a dev who doesn't use the crossOriginPolicy setting for whatever reason). In this case, the CanvasDrawer can still display the data (but the canvas itself will be tainted and won't allow access to the pixel data).

we can either set the attribute (having IMG) or convert to data URL and back (having canvas)

@Aiosa if the data is fetched from a cross-origin site without CORS, you can't convert to data URL because the canvas will have already been tainted. You also can't change the crossOrigin attribute on the <img> once it is loaded to get around this (or at least it won't have any effect).

I believe this is now fixed by #2472.


[Bug]:  'Sign Up' Button Overlaps with Border on Login Page

What is the expected behavior?

[X] I have searched the existing issues

What happened?
On the login page, the "Don't have an account? Sign Up" text is overlapping with the border, making it difficult to read and affecting the overall layout
Add Screenshots

What browsers are you seeing the problem on?
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

I can fix this issue. Please assign it to me.


Update sphinx_rtd_theme to match what docs.ms uses

Documentation artifacts successfully built -- https://dev.azure.com/azure-sdk/public/_build/results?buildId=1253189&view=artifacts&pathAsName=false&type=publishedArtifacts


Incorrect placement of action file error message

Description
If an action file is in incorrect format it shows an error while hovered over caution emoji. This is somewhat inconvenient and the emojis hitbox is actually too small. I suggest this should be moved on top/below in action tab of that action file.
Screenshot contains both error and suggestion where it should be.
Screenshots

Gitea Version
1.21.3
Can you reproduce the bug on the Gitea demo site?
Yes
Operating System
No response
Browser Version
N/A

Would look good if something like bootstrap alert.

Would look good if something like bootstrap alert.

I didn't understand .ui.error.message do you want create a new translate block or something else?

I didn't understand by .ui.error.message do you want create a new translate block or something else?

CSS class name from base theme for it to look like an error message.


[LiveComponent] consider data-value attribute

Q
A

Bug fix?
no

New feature?
yes

Tickets
#195

License
MIT

Allow to set the value using data-value. This can be handy if the model is updated from an <a> or <button> to set it to a fixed value for example.

Hey @WhiteRabbitDE!
My apologies for the slow response on this! I like this. Could you (A) add a test and (B) update the documentation? The test would be inside model.test.ts.
Thanks!

Hallo @weaverryan ,
i just tried to add a test but i have little experience with Jest and how it is used here. Im not sure how to add my case in a proper way. Can you give me a small hint please?

Hey @weaverryan ,
thanks for that super helpful and in detail explaination 🥇  The fetchMock part was what i was missing but now things are much clearer !
I hope i can use that new knowledge in future PRs :-)

Thank you Jan!


🐞 Internal links in the Admonition are no longer clickable.

Check for existing bug reports before submitting.

[X] I searched for existing Bug Reports and found no similar reports.

Expected Behavior
Clicking on an internal link within Admonition will take me to the linked page like Callout.
Current behaviour
When clicking on an internal link in Admonition, it switches to edit mode and the link does not work.
Reproduction

Launch Obsidian in Live preview mode.
Create an Admonition and include a sentence containing an internal link.
Attempt to click on the rendered link.

Which Operating Systems are you using?

[ ] Android
[X] iPhone/iPad
[ ] Linux
[X] macOS
[ ] Windows

Obsidian Version Check
1.4.13 and 1.4.13
Plugin Version
10.1.0
Confirmation

[X] I have disabled all other plugins and the issue still persists.

Possible solution
No response

Whoops, this was a side effect of fixing #278

i have the same behavior.
Obsidian Version Check
1.4.13
[x] Windows
Plugin Version
10.1.0
I installed the previous verion (10.0.2)
and the issue wont occur.

This should be fixed in #326