pytest fails due to docker 502 (bad gateway)

Expected behavior
pytest should run through with no failed tests (22 tests success).
Actual behavior
pytest gives 6 failed instances.
First, more of an FYI. Two of these seems to be related to the tests requiring v3 of a game instance, but v0/v4 are the only accepted instances now, e.g.
 DeprecatedEnv: Env PongDeterministic-v3 not found (valid versions include ['PongDeterministic-v4', 'PongDeterministic-v0'])

../gym/gym/envs/registration.py:145: DeprecatedEnv

I suspect the that is "simply" solved by updating the tests to use v4 (or v0 even?).
What really puzzles me is the following error which means I am not able to use anything in the universe package.
self = <docker.client.Client object at 0x107d62510>, response = <Response [502]>, explanation = None

    def _raise_for_status(self, response, explanation=None):
        """Raises stored :class:`APIError`, if one occurred."""
        try:
            response.raise_for_status()
        except requests.exceptions.HTTPError as e:
            if e.response.status_code == 404:
                raise errors.NotFound(e, response, explanation=explanation)
>           raise errors.APIError(e, response, explanation=explanation)
E           APIError: 502 Server Error: Bad Gateway ("ContainerInspect failed")

/usr/local/lib/python2.7/site-packages/docker/client.py:174: APIError

Running on MacOS and using Docker for MacOS (see next section) and is able to verify the installation by
>docker run hello-world
>
> Hello from Docker!
> This message shows that your installation appears to be working correctly.
>
>

Clearly the above failed test relates to the fact that
docker run -p 5900:5900 -p 15900:15900 --cap-add SYS_ADMIN --ipc host --privileged quay.io/openai/universe.flashgames:0.20.28
docker: Error response from daemon: ContainerInspect failed.
ERRO[0000] error waiting for container: context canceled

There are a couple of other errors, but they may be related to these so I will create a separate issue if fixing these doesn't solve them as well.
Versions
Change the username to xxx

Darwin xxx-MacBook-Pro.local 16.6.0 Darwin Kernel Version 16.6.0: Fri Apr 14 16:21:16 PDT 2017; root:xnu-3789.60.24~6/RELEASE_X86_64 x86_64
Python 2.7.13
Name: universe
Version: 0.21.5
Summary: Universe: a software platform for measuring and training an AI's general intelligence across the world's supply of games, websites and other applications.
Home-page: https://github.com/openai/universe
Author: OpenAI
Author-email<EMAIL_ADDRESS>License: UNKNOWN
Location: /Users/xxx/Documents/Code/GameAI/universe
Requires: autobahn, docker-py, docker-pycreds, fastzbarlight, go-vncdriver, gym, Pillow, PyYAML, six, twisted, ujson
---
Name: gym
Version: 0.9.2
Summary: The OpenAI Gym: A toolkit for developing and comparing your reinforcement learning agents.
Home-page: https://github.com/openai/gym
Author: OpenAI
Author-email<EMAIL_ADDRESS>License: UNKNOWN
Location: /Users/xxx/Documents/Code/GameAI/gym
Requires: numpy, requests, six, pyglet
---
Name: tensorflow
Version: 1.2.0
Summary: TensorFlow helps the tensors flow
Home-page: http://tensorflow.org/
Author: Google Inc.
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: /usr/local/lib/python2.7/site-packages
Requires: wheel, backports.weakref, protobuf, numpy, bleach, werkzeug, html5lib, mock, markdown, six
---
Name: numpy
Version: 1.13.0
Summary: NumPy: array processing for numbers, strings, records, and objects.
 It turns out that the current stable release of Docker For Mac has some sort of issue (see this post).
I installed the Docker For Mac Edge version instead and can the containerinspect error is gone. Will try to run pytest on universe again to see if there are more issues and report back.


Code quality improvements

Hey,
Pretty nice proof of concept, here are some optional code quality improvements that I had whilst reading the code.
As a side note, I have also written a Rust version of this proof of concept https://github.com/ynuwenhof/keedump
Kind Regards!

Looks good, thx
Added you to the README


🌱  the cloudbuilds to yaml

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[x] PR title follows the guidelines defined in  https://github.com/ossf/scorecard/blob/main/CONTRIBUTING.md#pr-process

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature

What is the current behavior? (You can also link to an open issue here)
fixes #443  fixes #434

What is the new behavior (if this is a feature change)?

Moved the cloudbuild configuration into YAML

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

None

Other information:

None

i dont know the use of these images, so defer to @azeemshaikh38 @oliverchang for review here.

i dont know the use of these images, so defer to @azeemshaikh38 @oliverchang for review here.

Scorecard docker image is used by https://github.com/ossf/scorecard#docker
Scorecard-tag is used whenever there is a tag pushed into main. docker pull gcr.io/openssf/scorecard:v1.0.0
This is the scorecard cron gcr.io/openssf/cron:latest that is deployed in k8s.

ping @oliverchang

ping @oliverchang

@oliverchang is in SYD timezone, so already weekend there. he should be able to review on monday his time.


Fixed typos

I fixed a few typos while keeping sensor_id constant to avoid a breaking change.

Thanks!


fix(procedure): convert js arrays to postgres array type

Just my first humble try on solving this. Currently only handles one-dimensional arrays. Maybe to naive, etc. Feedback very welcome.

Thanks so much for your contribution! 🎉
Although I feel like this is a small band aid on a more systemic problem. Might you try running the objects through JSON.stringify? Also a test case would be nice. Look at the kitchen sink example and its corresponding integration tests.

Could you elaborate more on the systemic problem …
Will look at the kitchen sink example for tests. Also not sure what you mean by running the objects through JSON.stringify — the values object?

I have added test (still local). Of course all the procedure tests fail with this pull request. I have underestimated the complexity … ☺  I assume this is because procedures can have tables as arguments or individual fields.

Ok, second try. Excuse the naming and such. Happy to restructure to your wishes.
Will fix the linting errors.

rebased to include the newest change on master …

What I mean by “systemic problem” is the very way we support table arguments in the first place. So let’s take a computed column for example:
create function full_name(person person) returns text as $$
  select person.given_name || ' ' || person.family_name
$$ language sql;

Somehow we have to call that procedure with an argument of type person. This is difficult because we don’t want to re-query the person object from the database (we just did after all), so we send the JSON object we just selected instead. Basically our two options are:
select full_name((select * from person where id = 1));

or:
select full_name('{"givenName":"Test","familyName":"User"}'::person)

We have chosen the second approach which can lead to edge-casey errors like this one. The optimal solution would be to look at the GraphQL query so we can “predict” if we will need a computed column like so:
select *, full_name(person) from person where id = 1

But that requires more effort and may be out of scope for now.

I’ll comment on what I mean about using JSON.stringify on the code 👍

Looks good to me. What do you think?

Yes. I think we can merge it.

Merged, as I want to run this project as open-open source do you want to become a collaborator?

@calebmer of course. Thats a first for me. Thanks.

Done :+1:
I’m going to wait to release this with #64.

Released in 1.5.1.


chore: change version and CHANGELOG.md

Codecov Report

Merging #522 (748df21) into master (6cd9912) will increase coverage by 0.95%.
The diff coverage is n/a.

:exclamation: Current head 748df21 differs from pull request most recent head 6d9733e. Consider uploading reports for the commit 6d9733e to get more accurate results

@@            Coverage Diff             @@
##           master     #522      +/-   ##
==========================================
+ Coverage   59.33%   60.28%   +0.95%
==========================================
  Files          18       18
  Lines         841      841
  Branches      170      170
==========================================
+ Hits          499      507       +8
+ Misses        342      334       -8

Impacted Files
Coverage Δ

src/multimanifestmodule.ts
52.23% <0.00%> (+5.97%)
:arrow_up:

Continue to review full report at Codecov.

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


Refactoring triggers for update cassandra config

Description
The Cassandra config is not updated by changing the instance_type.
Done
Fix to use cassandra_waitfor for triggers instead vm_ids.
Confirm
  # module.cassandra.module.cassandra_provision.null_resource.cassandra[0] must be replaced                                            [573/12699]
-/+ resource "null_resource" "cassandra" {
      ~ id       = "5386767767388688332" -> (known after apply)
      ~ triggers = {
          - "triggers" = "i-0b16b0c3529f5881f"
        } -> (known after apply) # forces replacement
    }

  # module.cassandra.module.cassandra_provision.null_resource.cassandra[1] must be replaced
-/+ resource "null_resource" "cassandra" {
      ~ id       = "6378479453704558095" -> (known after apply)
      ~ triggers = {
          - "triggers" = "i-0102ee7340fe629a8"
        } -> (known after apply) # forces replacement
    }

  # module.cassandra.module.cassandra_provision.null_resource.cassandra[2] must be replaced
-/+ resource "null_resource" "cassandra" {
      ~ id       = "3671813800215072783" -> (known after apply)
      ~ triggers = {
          - "triggers" = "i-05c7d7488ce3e54e5"
        } -> (known after apply) # forces replacement
    }

  # module.cassandra.module.cassandra_provision.null_resource.cassandra_waitfor[0] must be replaced
-/+ resource "null_resource" "cassandra_waitfor" {
      ~ id       = "170923916993472432" -> (known after apply)
      ~ triggers = { # forces replacement
          ~ "triggers" = "871986139252697661,t3.large" -> "871986139252697661,r5d.large"
            "vm_id"    = "i-0b16b0c3529f5881f"
        }
    }

  # module.cassandra.module.cassandra_provision.null_resource.cassandra_waitfor[1] must be replaced
-/+ resource "null_resource" "cassandra_waitfor" {
      ~ id       = "7086052489776489918" -> (known after apply)

@ymorimo Merging it but PTAL.


add eslint rule to enforce strict equality

Are you sure you want to add +245,723 lines of code? Looks like you added whole node_modules


Cannot access 'state' before initialization

What is the proper way to use form handler in sapper?
I have got error 500 Cannot access 'state' before initialization

Hello @luhovskyi this package haven't been tested with sapper yet, hence does not have guidance. Will try to get some intel, any feedback here would be welcomed.


SharecodeDecoder not working

The sharecode decover is not working for me:
`

var Steam = require('steam');
var csgo = require('csgo')
var steamClient = new Steam.SteamClient();
var steamUser = new Steam.SteamUser(steamClient);
steamClient.connect();
steamClient.on('connected', function() {
steamUser.logOn({
account_name: 'redacted',
password: 'redacted'
});
});
steamClient.on('logOnResponse', function() {
steamGC = new Steam.SteamGameCoordinator(steamClient, 730);
CSGO = new csgo.CSGOClient(steamUser, steamGC, false);
console.log(CSGO.SharecodeDecoder("CSGO-U6MWi-hYFWJ-opPwD-JciHm-qOijD").decode());
});

`
this results in

You're missing a new before csgo.SharecodeDecoder; from the example file:
/* Decoding Share Codes */
var scDecoder = new csgo.SharecodeDecoder("CSGO-U6MWi-hYFWJ-opPwD-JciHm-qOijD");
console.log("Sharecode CSGO-U6MWi-hYFWJ-opPwD-JciHm-qOijD decodes into: ");
console.log(scDecoder.decode());


Advice on how to implement comparison operators for a type whose CompareTo is inconsistent with Equals

Moved issue to .NET
Old issue: https://github.com/MicrosoftDocs/visualstudio-docs/issues/3728, opened by @peteroupc
For full discussion, see 3728.
This page should include advice on how comparison operators should be implemented in the following circumstance.

A custom number class (not from the base class library) represents a number by its significand and exponent.
It overrides Object.Equals with a method that compares the significand and exponent for equality.
It has a CompareTo method that does a numerical comparison of the two numbers, which is a non-total ordering that is inconsistent with Equals.
It implements the "<", "<=", ">=", and ">" operators to do comparisons that are consistent with CompareTo.

Should it implement the "==" and "!=" operators to be consistent with Equals or with CompareTo, or should it refrain from implementing those operators?
What if CompareTo instead does a total ordering of the two numbers (making it consistent with Equals)?  Can it implement the "<", "<=", ">=", and ">" operators to do a numerical comparison rather than a total ordering, if a numerical comparison makes more sense?  If so, should the "==" and "!=" operators be implemented to be consistent with Equals, with a non-total comparison, or with a total ordering comparison, or not be implemented?
For the purpose of this issue, an example of the number class is found in my library's EDecimal class (EDecimal.cs, EDecimalExtra.cs), except that it doesn't implement the "<", "<=", ">=", and ">" operators at the time of this writing.

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

ID: 36fdfc4f-e55f-9706-5d5e-b60b1ec9fb37
Version Independent ID: 9584f8bb-0dbf-cb31-5499-f11541f0f9a4
Content: CA1036: Override methods on comparable types (code analysis) - .NET
Content Source: docs/fundamentals/code-analysis/quality-rules/ca1036.md
Product: dotnet-fundamentals
GitHub Login: @gewarren
Microsoft Alias: gewarren

Personally, I think that the guidance should be that Equals and CompareTo should be consistent. If you choose to make them inconsistent, then you're on your own and you need to decide yourself how to handle that. The documentation shouldn't help you with that.

@svick I like that. I'm going to close this issue now.


您好，few-shot体现在什么地方？

Hi, you can refer to https://github.com/sjtuplayer/anomalydiffusion/issues/10


Get merge/conflict status

System Details

posh-git version/path:
PowerShell version: 1.0.0 beta2 C:\Program Files\WindowsPowerShell\Modules\Posh-Git
Git version: git version 2.20.1.vfs.1.1.104.g2ab7360
Operating system name and version:
Platform ServicePack Version      VersionString

Win32NT             10.0.17134.0 Microsoft Windows NT 10.0.17134.0
Issue Description
This is a feature request. I am automating an integration process that is triggered via a pull request. The process will merge the developer's feature branch with an integration branch, and then our CI system will build/deploy that integration branch.
I want to be able to comment on the PR if the merge failed/has conflicts, and would like to be able to explain which lines are conflicting (and who the commiter is so that the 2 developers can collaborate on resolving the conflict(s)
I would love if Posh-Git could give me information such as is a merge in progress, how many conflicts there are (and whom is to blame). I think the way I would like it to work is to include that information inside of Get-GitStatus, then my automation script becomes Check Get-GitStatus and if there is a conflict report to the CI system with the information surfaced.

I would be interested in writing some code for this feature if someone can provide some guidance on how they want this feature to work/be designed so that I can ensure I am starting off on the right path.

Sorry for the delayed response, @brandonmcclure.
posh-git already includes conflicted files in the Get-GitStatus response as $GitStatus.Index.Unmerged and $GitStatus.Working.Unmerged (they should match). They drive the prompt's ! status/counts:
[conflict-test +0 ~0 -0 !1 | +0 ~0 -0 !1 !]

That said, I don't think we'll want to teach posh-git how to get the more detailed information you seek, since we can Get-GitStatus for every prompt. You'll get better performance from more specific git functions, e.g. combine git ls-files --unmerged and git blame -L '/^<<<</,/^>>>>/' -- <file>.
Glad to help you polish such a script (or post a question to Stack Overflow if you get stuck), but I'm going to close this as a posh-git issue.


ENH: Add CI through GitHub actions

Add CI through GitHub actions.
Testing data and script
--
Have you

[X] Added a description of the content of this PR above
[X] Followed proper commit message formatting
[ ] Added data and a script to test this PR
[ ] Made sure that PEP8 issues are resolved
[ ] Tested the code yourself right before pushing
[ ] Added unit tests to test the code you implemented

People this PR concerns
--

Folks, I have not tagged anybody specifically, sorry. This is ready to be merged following this and this other comment, which apply to this PR as well.
Thanks.

Thanks Jon. Once again, this is a bit over my expertise level. How should I test this?

I had a look at testing this locally, but if I'm not mistaken, I think you need a given amount of disk space left so that the Ubuntu images can be pulled, so I did not try.
The relevant thing is that you can see that there are 6 checks that should be automatically triggered every time a PR is open. They test different Python configurations/versions (3.5, 3.6, etc.). More configurations can be added at a later time if we need to test them.
These checks run a basic setup (e.g. pip install -r requirements.txt and  python setup.py) and would run tests as well if we had them. Additionally, they also check the style with flake8 (and that is where they found we do not comply with it and point to the relevant files, lines and errors). So we would no longer need to test the setup locally every time a change is made.

@EmmaRenauld asked me to have a look at this.
Honestly, beyond my skills but looks fine (I guess).
I suggest you merge and if problems arise. @jhlegarreta can fix.

OK. I will fix the style issues pointed by the CI in a separate PR. If everything goes fine when opening that PR, the checks should pass.
Will try to address them during the weekend.
Thanks for the reviews.


New features to support

Hi @Yao-14 I am adding a few new features requests that are worth to incorporate over the time:

[x] allow user to import new 3D data generated with Spateo package
[ ] enable virtual slicing with the 3D smoothing model
[ ] allow creating the animations of the vector field predictions
[ ] enable visualization of differential geometry analysis results
[ ]  ...

Hi Xiaojie，thanks for your suggestions! i will add those functions to spateo-viewer.


Issues about Multi GPU

Hello author, may I ask how to train on multi GPUs? I have tried to set device to [0,1,2,3], but it seems to be invalid

Hi @M45hiro!
Unfortunately, multi-GPU training is not supported currently.


Fix W3C driver detection for WebElements

Fun fact: in the Selenium Python bindings, but not explained at all in the documentation for it, is support for auto-detecting whether a particular web driver implements the W3C spec or the Selenium API.  If the former, it then implements a number of shims automatically so existing code often doesn't need to be changed to understand the difference.  As it turns out, the implementation of passing that knowledge from the driver to each WebElement is in create_web_driver() - which needle overrides.
I fixed needle's implementation to include that piece of knowledge transfer, and now the try/except in get_dimensions() is no longer needed (once Selenium knows it's dealing with a W3C spec-based driver, it automatically calls the "rect" command instead of "location" or "size").  The screenshot changes we committed earlier are still needed, since Selenium doesn't have that kind of shim to get a screenshot of just a particular element for Selemium API drivers.
That self.w3c attribute has been present since selenium 2.47.0; I can add a condition check to make it work with even older versions if you'd like, but I'm not sure it's worth the added code complexity (that release came out almost 2 years ago).

Very neat, thanks!


Add minimal permissions guidance

Add guidance for Microsoft Graph that helps customers verify that their apps use minimal permissions.

Update on the topic. We are meeting with the DevX API team this week to get more details on the rollout and the underlying capabilities we could leverage to identify minimal permissions guidance as part of the Proxy!

Implemented in v0.8

@garrytrinder this is not quite the same as the minimal permissions plugin that we've got now. The idea here is, that we compare the scopes/roles from the access token with minimal permissions for the given set of endpoints and verify that the app isn't overconsented.


Read photo overlay layer name and image key from URL query string

Currently it's possible to read the map coordinates (center) and zoom from query string, so for example I can be on mapillary's website and click the button to edit the same view on iD:
https://www.openstreetmap.org/edit?editor=id#map=18/39.93450741752976/-105.0910279993947
I would like on Mapillary, and I think similarly it would be useful for projects using OSC and Bing, to be able to specify the image layer and key:
https://www.openstreetmap.org/edit?editor=id#map=18/39.93450741752976/-105.0910279993947?photo_overlay=mapillary&photo=o6y6pgnHjz7vdEbjY7YCtg
This link, for example, would make sure the Mapillary photo overlay is turned on, and then initiate the MapillaryJS viewer and execute moveToKey('o6y6pgnHjz7vdEbjY7YCtg') which would open that specific image, so a user can start editing right away

So I added the photo_overlay API parameter. Here's its documentation:

photo_overlay - The street-level photo overlay layers to enable.
Example: photo_overlay=streetside,mapillary,openstreetcam
Available values: streetside (Microsoft Bing), mapillary, mapillary-signs, openstreetcam

Adding the photo parameter is more involved so that part is still up for grabs!

@quincylvania awesome! I'll see if maybe we can pick up on our end from there, thanks!

I want to move to a next level here and see if we can include:

image key
token

@quincylvania I am not too knowledgeable about this but want to help @kratico understand what might be possible. Do you think you could summarize to us what we'd have to do to get that working?

Closing this and opening separate issues:
Image key: https://github.com/openstreetmap/iD/issues/6398
Token: https://github.com/openstreetmap/iD/issues/6399


[BUG]:  should remove the requirement for torch_version.major == 1 if ColossalAI supports torch 2.0

🐛 Describe the bug
colossalai/cli/launcher/run.py line 157, in get_launch_command
assert torch_version.major == 1
If ColossalAI already supports torch2.0 then we should modify the code.
Environment
Conda installed torch 2.0 by default as the version is not specified in requirements.txt in the tutorial examples.

Have created PR https://github.com/hpcaitech/ColossalAI/pull/3878

update PR https://github.com/hpcaitech/ColossalAI/pull/3896, previous one has been closed


Add a copy of the MIT license, fix typo in package.json

It is good practice to add a copy of the license to the repository. Also fixes a typo in package.json.

Thank you, @Calinou ;)


API to get Expr's type and nullability without a DFSchema

Which issue does this PR close?
Closes https://github.com/apache/arrow-datafusion/issues/1725
Rationale for this change
I am trying to create a low cost (low copy) way for Expr simplification.  More details on https://github.com/apache/arrow-datafusion/issues/1725
What changes are included in this PR?

Add a trait that can provide the needed information for schema quries to Expr
Change the functions on Expr that take a &DFSchema such as Expr::nullable to be nullable
impl this new trait for DFSchema so that existing callsites work

Are there any user-facing changes?

It is now possible to get an Expr's type without constructing a DFSchema; Existing code should continue to work

I am happy to improve / change this API -- if anyone has thoughts or suggestions please let me know


Update timestamp-by-azure-stream-analytics.md

Updating timestamp by documentation to mention a few common issues encountered by customers.

@StreamingTodd : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@JasonWHowell : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@JasonWHowell : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Order in the legend should be the same as order in the chart

Now the legend of charts seems to be a bit random...

Is this something you can do as part of https://github.com/quintel/etmodel/issues/2187, @grdw ?

What is the "order in the chart"? I don't think that's a generic term you can use.

This order is clearly reversed if you start looking from the top to the bottom.

In here the order of the legend is scrambled accross the chart.
So what do you mean with 'the order in the chart'? And for what chart?

@ChaelKruip , could you clarify this issue / your original comment?

This order in the image above is clearly reversed if you start looking from the top to the bottom.

Yes. I think it would be more natural to have 'appliances' at the bottom of the chart and at the bottom of the legend.

In this example howver the order of the legend is scrambled accross the chart.

Not really, I think the order here is:

hydrogen
hot water
electricity
geothermal
solar thermal
coal
natural gas
oil
biogenic waste

both in the legend and in the chart. This is the 'natural' way of displaying it in my opinion.

Yes. I think it would be more natural to have 'appliances' at the bottom of the chart and at the bottom of the legend.

The only way I see that happening is when somebody reverses the order_by column by hand.

The only way I see that happening is when somebody reverses the order_by column by hand.

No. Because than the series are also plotted in the reverse order.

No. Because than the series are also plotted in the reverse order.

Eh, ok. learn something new everyday. 😅
We could solve it by adding another column called legend_order_by and specify the order of the legend in there? Or we could reverse() the order as a patch fix for that one single chart but that's just a cheap shot.

Or we could reverse() the order as a patch fix for that one single chart but that's just a cheap shot.

Why would that be a cheap patch? Personally I think that would be a big improvement.
Curious about other opinions though... @jorisberkhout @antw @DorinevanderVlies @AlexanderWirtz  @dennisschoenmakers ?

Why would that be a cheap patch? P

Because I haven't looked at the behavior of other chart types and their legends yet.

Why would that be a cheap patch? Personally I think that would be a big improvement.
Curious about other opinions though... @jorisberkhout @antw @DorinevanderVlies @AlexanderWirtz @dennisschoenmakers ?

I agree with @grdw that it would be a "cheap patch" to simply reverse() the order of the legend without first understanding why the legend order is correct in some charts but not in others.
My guess is that stacked bar charts and line charts plot the first series at the bottom of the chart and grow upwards ↥ (Geothermal → Biomass → Ambient heat → ... → Coal → Oil), whereas the mekko charts start at the top-left and grow diagonally down/right ↘.
The reason cheap fixes are worrisome is that things get missed:

Changing the stacked bar and stacked line charts would be an improvement, but the merit order and "CO2 emissions biomass" charts would still have legends in the wrong order.
The "Electricity production per hour" chart also needs to have the legend reversed (and is neither a StackedBar nor StackedLine chart) – but I think it should still show the "Demand" series first.
In some charts the order of series' may change ("Merit order capacities and marginal costs"), but I'm not convinced the legends should be re-sorted when that happens (that chart is particularly quirky since it grows form the bottom-left, up/right ↗).

I'm bike-shedding a bit, but you get the idea. :dash:
--
(Since there is insufficient time before the deploy, I think it's okay to fix the main issue – stacked line and stacked bar charts – and come back to the special-case charts later).

I agree with @grdw that it would be a "cheap patch" to simply reverse() the order of the legend without first understanding why the legend order is correct in some charts but not in others.

Point taken (although I did not aim to suggest that we should implement something we don't fully understand the effects of)!

Let's remove this from the deploy milestone as it is not critical.


Dependency install fails for example projects on Windows

Windows 11
Pytrhon 3.10.2
pip 22.0.4
some text decoding problem seems to be on this line
https://github.com/italia/spid-cie-oidc-django/blob/546ba2c7f973c980835d10100fb6a5767d55a870/setup.py#L24

my workaround was replcing that line with README=""

Daniele also uses win32 and he doesn't have this problem, would you like to ask to him what's the cause/workaround for this issue?
I use only GNU/Linux, my excuses


[Feature Request] Hotkey for Insert Code

Hotkey for Insert Code  like Stackoverflow
Ctrl K for code
Ctrl L for link

Hi @SmartManoj!
It looks like Ctrl+k is already defined as a hotkey to insert a link. So, I can provide you with a simple very specific alternative to get that to work for you.
// ==UserScript==
// @name        GitHub Markdown Editor Hotkeys
// @version     0.1.0
// @description A userscript that changes the markdown editor code & link hotkeys
// @license     MIT
// @author      Rob Garrison
// @namespace   https://github.com/Mottie
// @include     https://github.com/*
// @run-at      document-idle
// @grant       none
// @icon        https://github.githubassets.com/pinned-octocat.svg
// ==/UserScript==
(() => {
	"use strict";

	function attr(el, key, label) {
		el.setAttribute("hotkey", key);
		el.setAttribute("aria-label", label);
	}

	function init(event) {
		if (document.querySelector("markdown-toolbar")) {
			document.querySelectorAll("md-code").forEach(el => {
				attr(el, "k", "Insert code <ctrl+k>");
			});
			document.querySelectorAll("md-link").forEach(el => {
				attr(el, "l", "Add a link <ctrl+l>");
			});
		}
	}

	document.addEventListener("pjax:end", init);
	init();

})();

In case you didn't notice, the custom hotkey script allows you to add custom hotkeys that change the url, so it wouldn't have worked in this case.

Off topic:
Why would you use all openuserjs , GreasyFork , github and how you maintain them ?

Most users of userscripts would search OpenUserJs & GreasyFork for scripts. And both of those sites allow using GitHub commit hooks that auto-update the scripts on their site, so maintenance is minimal. Although, I did notice that after renaming one script, I now have to manually update it on OpenUserJs.

How would you test the script and host it in github.
I am currently doing like this.

I guess I'm old school, I copy-paste. I don't think everything needs to automated.

Hi @SmartManoj!
It looks like Ctrl+k is already defined as a hotkey to insert a link. So, I can provide you with a simple very specific alternative to get that to work for you.
// ==UserScript==
// @name        GitHub Markdown Editor Hotkeys
// @version     0.1.0
// @description A userscript that changes the markdown editor code & link hotkeys
// @license     MIT
// @author      Rob Garrison
// @namespace   https://github.com/Mottie
// @include     https://github.com/*
// @run-at      document-idle
// @grant       none
// @icon        https://github.githubassets.com/pinned-octocat.svg
// ==/UserScript==
(() => {
  "use strict";

  function attr(el, key, label) {
    el.setAttribute("hotkey", key);
    el.setAttribute("aria-label", label);
  }

  function init(event) {
    if (document.querySelector("markdown-toolbar")) {
      document.querySelectorAll("md-code").forEach(el => {
        attr(el, "k", "Insert code <ctrl+k>");
      });
      document.querySelectorAll("md-link").forEach(el => {
        attr(el, "l", "Add a link <ctrl+l>");
      });
    }
  }

  document.addEventListener("pjax:end", init);
  init();
})();

In case you didn't notice, the custom hotkey script allows you to add custom hotkeys that change the url, so it wouldn't have worked in this case.

<ctrl+l> goes to address bar

Ctrl L goes to address bar sometimes

This userscript uses GitHub's method to apply hotkeys - meaning all this userscript does is adds a hotkey="x" to a link.
I don't think their code overrides browser specific shortcuts. I have F1 set to open the hotkeys settings modal (by default), but lately I haven't gotten it to do anything but open the Google Chrome help page.

Ctrl L goes to address bar sometimes

But sometimes only

This userscript uses GitHub's method to apply hotkeys - meaning all this userscript does is adds a hotkey="x" to a link.
I don't think their code overrides browser specific shortcuts. I have F1 set to open the hotkeys settings modal (by default), but lately I haven't gotten it to do anything but open the Google Chrome help page.

Works Fine for me

// ==UserScript==
// @name         New Userscript
// @namespace    http://tampermonkey.net/
// @version      0.1
// @description  try to take over the world!
// @author       You
// @match        https://github.com/Mottie/GitHub-userscripts/issues/89
// @grant        none
// ==/UserScript==

(function() {
    'use strict';
document.onkeydown=function(e){
    if (e.key=='F1'){
        e.preventDefault();
        alert(2);

}
}
})();


Side bar updates

As a user
I want easy navigation of the system
so that I can find what I need
Acceptance Criteria
Scenario 1:
WHEN the User logs in to EPIC.track in from IDIR
THEN "My Workplans" page is open by default
THEN User can see the side bar with a fixed width and the following menu items from top to bottom:

"All Work" with the list icon (default)
"My Workplans" with grid icon (active)
"Reports" with files icon (default)
"Insights" with chart - Pie icon (default)
"List Management" with gear icon (default)

Scenario 2:
GIVEN the User logged in to EPIC.track in from IDIR
WHEN the user click on reports
THEN they see the following menu items:

"EA Referral Schedule"
"Resource Forecast"
"30-60-90"

Scenario 3:
GIVEN the "Reports" menu list is open
WHEN I click on "List Management"
THEN "Reports" submenu collapsed
THEN I can see the following menu list:

"Staff"
"Indigenous Nations"
"Proponents"
"Projects"

Design Implementation / Bars & Collapsed Phases - 07-07-2023
https://www.figma.com/file/j5onJ5RZGAxahxhYp68N8m/Workplan?type=design&node-id=945%3A64853&mode=design&t=JZsXk2rJzJFDbTBi-1

Icons for side bar - .svg, white

@dinesh-aot

changes:

moving My Workplans to the top
32px height between groups, 0px height between Menu items within the group
For List Management, we'll use a different icon, Gear icon goes to the Admin Menu item

Template Icon

List Management Icon


Remove unused IOException in getFileBlocks in MasterInfo

Remove unused IOException in getFileBlocks in MasterInfo

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2570/
Test PASSed.

I think the getFileBlocks methods internally call getClientBlockInfo which still indicated can throw IOException.

In MasterInfo getClientBlockInfos doesn't throw IOException either.

Ah no worries, it was apparently call to getClientBlockInfos instead of getClientBlockInfo

it is not right that getClientBlockInfo throws IOException, so I also make the change.

+1 LGTM
Simplify Tachyon exception throwing

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2572/Failed Tests: 1org.tachyonproject:tachyon: 1tachyon.master.MasterFaultToleranceTest.faultTest
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2573/
Test PASSed.


Go 1.9.2 and Go 1.8.5

Released on October 25th
https://groups.google.com/forum/#!topic/golang-announce/s_hLxKF9ApA
When I specify golang:1.8.5 in my circle.yml, I see this message:
Build-agent version 0.0.4275-16f7565 (2017-09-26T14:09:58+0000)
Downloading previous test results
Starting container circleci/golang:1.8.5
  image cache not found on this host, downloading circleci/golang:1.8.5
Error response from daemon: manifest for circleci/golang:1.8.5 not found

(CircleCI Enterprise)
The older version of Go is working fine for now.

There appear to be some other 1.8.5 images, but not just a straight 1.8.5 as there is with 1.8.4. Likewise for 1.9.2.
https://hub.docker.com/r/circleci/golang/tags/

Hmm, I wanted to assume it was because of upstream but they appear to have the standalone tags. https://hub.docker.com/r/library/golang/tags/
Looking into this.

circleci/golang:1.9.2 not found
Getting this right now, anyone have any clue whats going on?

@oshalygin This is a different, closed issue. Please follow this instead: https://github.com/circleci/circleci-images/issues/214


read logs from container

this pr makes kubectl-flame work in envs where sidecar container is injected (for example: service mesh)

@saritip Thanks for contributing to kubectl-flame!


GTWranks

GTWcivilians script is not working without GTWranks.

GTWranks has been merged into GTWcivilians and optimized for the current stats system


realloc:  invalid next size with PhantomJS 1.9.8

Hi all,
We have a 30 node Jenkins cluster of virtual machines off two blades and got a "realloc: invalid next size"  on 3 separate VMs yesterday that build PRs--the PRs were very small compared to the rest of our fairly large codebase.   We use the PhantomJS rubygem so it doesn't look like upgrading to 2.0 is possible for us yet.
http://pastebin.com/xZ5dZ88x
This error hasn't happened today, nor has it happened before to my knowledge.  Any advice?
Thanks for your help.
--sean

I have the same issue when evaluating an html page of 13M. I tried with both 1.9.8 and 2.1.1.


[BUG] - Tabs destroyInactiveTabPanel unmounts inactive tabs' content

NextUI Version
2.4.1
Describe the bug
Hi,
I currently have 2 tabs and a Formik component in each. I need the form data to not be cleared when switching between the tabs. When I set destroyInactiveTabPanel={false}, the inactive tab's Formik instance gets unmounted as soon as I switch to the other tab.
I investigated this issue. I set a unique id on each content (simplified example).
<Tabs destroyInactiveTabPanel={false}>
  <Tab key="tab1">

First tab content

  </Tab>
  <Tab key="tab2">
    <input id="tab2" />
  </Tab>
</Tabs>

In DevTools, I can see 2
s, one for each tab. The inactive tab has a display: none; but the content inside both tabs'
s are the same, i.e. the active tab's content. When I switch to the other tab, then the same happens. The inactive tab's content is unmounted. I tried setting unique keys on each tab and their content, but no luck.
Is there anything I'm doing wrong?
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
N/A
Expected behavior
I would expect all tabs to be mounted at all times when destroyInactiveTabPanel={false}.
Screenshots or Videos
No response
Operating System Version

Browser
Chrome

@wingkwong I'm not able to get destroyInactiveTabPanel={false} working in this example. I didn't add Formik yet. Maybe you can look into it without Formik first?


Hack fix for d3 calculations which shouldn't be split in the planner

Description

Fixes issue #688
These functionals shouldn't be split in the composition planner, as the first part isn't available in psi4 as a plain functional.
This is a temporary fix. The next major version will probably do away with the composition planner altogether.
Changelog description

Skip composition planner for certain functionals
Status

[X] Code base linted
[x] Ready to go

The composition planner is called both for computing and accessing:
https://github.com/MolSSI/QCFractal/blob/8a291bc5b9063d474045c712eb4b4569acf871a3/qcfractal/interface/collections/dataset.py#L1136
https://github.com/MolSSI/QCFractal/blob/8a291bc5b9063d474045c712eb4b4569acf871a3/qcfractal/interface/collections/dataset.py#L1223
I guess there may still be a problem with existing records? But with the way datasets are currently written that shouldn't be an issue. The previously submitted ones will disappear from the dataset (because the dataset doesn't explicitly link to individual records)

The composition planner is called both for computing and accessing:
https://github.com/MolSSI/QCFractal/blob/8a291bc5b9063d474045c712eb4b4569acf871a3/qcfractal/interface/collections/dataset.py#L1136
https://github.com/MolSSI/QCFractal/blob/8a291bc5b9063d474045c712eb4b4569acf871a3/qcfractal/interface/collections/dataset.py#L1223
I guess there may still be a problem with existing records? But with the way datasets are currently written that shouldn't be an issue. The previously submitted ones will disappear from the dataset (because the dataset doesn't explicitly link to individual records)

Ah perfect, you're right, missed the usage in Dataset._compute. With this change use of a new qcportal from a new QCFractal release should generate the right set of records when called. Can basically try to resubmit the dataset and this should just work.


Migrate features from Omni Core v0.3.1

Hey guys,
we are close to release Omni Core v0.3.1, which has quite a bit of improvements, ranging from new RPCs to bug fixes and significant performance improvements. You can check out the pull requests of that release here:
https://github.com/OmniLayer/omnicore/milestone/8?closed=1
Cheers!

ok, thank you.


Add instructions for Arch Linux

Adds install instructions for Arch/Manjaro

Also fixes some Markdown according to markdownlint
Replace mentioning of GH when the module is actually loaded from kernel.org

I couldn't really figure out how to programmatically install the fitting kernel headers for Manjaro, but dkms gives you some instructions to install the needed version, anyhow.
Thank you for providing a appropriate solution to a nasty problem.
(Did I mention I'm using Arch? :wink:)

Thanks so much for your PR!


Is it possible to add a path in MqttClient?

My MQTT broker URL is ws://broker.emqx.io:8083/mqtt.
How can I specify the "/mqtt" path in the MQTTClient?

Hello, it is not yet possible, I will add the functionality when I have time. Anyway I tried to connect like this and it seems to be working anyway:
val client = MQTTClient(
    MQTTVersion.MQTT5,
    "broker.emqx.io",
    8083,
    null,
    15,
    webSocket = true
) {
    println(it.payload?.toByteArray()?.decodeToString())
}

Added in https://github.com/davidepianca98/KMQTT/commit/87b20331bd244af4bbe345c6cfc8455bfe48e5e2. Will be available in the next release


[BLE L2CAP] Reader force closing with BluetoothSocket exception instead of closing session

Expected Behavior

Scenario - Google Reader set to "use L2CAP"
3rd party holder set to BLE central + L2CAP + secure channel
Perform QR + BLE scenario
3rd party reader has an issue and Google Reader waits for device's response indefinitely
Press the close button on Google Reader to interrupt communication
Google Reader should reset the session and return to the main activity

Actual Behavior
1..5 - same behavior
6. Google Reader force closing with BluetoothSocket exception which wasn't caught by the Google Reader app
Log:
Samsung reader_20230412.txt
Google holder_20230412.txt
6071  BluetoothGattServer    D  close()
6071  BluetoothGattServer    D  unregisterCallback() - mServerIf=10
6071  L2CAPServer            E  Error getting connection from socket server for L2CAP read failed, socket might closed or timeout, read ret: -1
                                java.io.IOException: read failed, socket might closed or timeout, read ret: -1
                                	at android.bluetooth.BluetoothSocket.readAll(BluetoothSocket.java:972)
                                	at android.bluetooth.BluetoothSocket.waitSocketSignal(BluetoothSocket.java:925)
                                	at android.bluetooth.BluetoothSocket.accept(BluetoothSocket.java:700)
                                	at android.bluetooth.BluetoothServerSocket.accept(BluetoothServerSocket.java:179)
                                	at android.bluetooth.BluetoothServerSocket.accept(BluetoothServerSocket.java:165)
                                	at com.android.identity.L2CAPServer.waitForConnectionThread(L2CAPServer.java:111)
                                	at com.android.identity.L2CAPServer.$r8$lambda$c52HqxGyIEVpaU3yKzRi6GSay20(Unknown Source:0)
                                	at com.android.identity.L2CAPServer$$ExternalSyntheticLambda1.run(Unknown Source:2)
                                	at java.lang.Thread.run(Thread.java:920)
6071  L2CAPServer            D  Start reading socket input
6071  AndroidRuntime         E  FATAL EXCEPTION: Thread-6
                                Process: com.android.mdl.appreader, PID: 6071
                                java.lang.NullPointerException: Attempt to invoke virtual method 'java.io.InputStream android.bluetooth.BluetoothSocket.getInputStream()' on a null object reference
                                	at com.android.identity.L2CAPServer.readFromSocket(L2CAPServer.java:178)
                                	at com.android.identity.L2CAPServer.waitForConnectionThread(L2CAPServer.java:130)
                                	at com.android.identity.L2CAPServer.$r8$lambda$c52HqxGyIEVpaU3yKzRi6GSay20(Unknown Source:0)
                                	at com.android.identity.L2CAPServer$$ExternalSyntheticLambda1.run(Unknown Source:2)
                                	at java.lang.Thread.run(Thread.java:920)

Google apps does not support L@CAP with secure mode so issues is not relevant now


Current state of the optimizer

Hey,
thank you for sharing that code. I'm not totally aware of the current status, but only found hints to v3 of the paper. As for now, they released v5 at ICLR 2020. Is the current implementation according to that version?
Kind regards,
Raphael

Has not been updated, PRs welcome

Author open sourced TF version
https://github.com/tensorflow/addons/blob/master/tensorflow_addons/optimizers/lamb.py

Thank you! :)

Is there a possibility to use the TF-version within pytorch or does it require an extra implementation?

Not possible without a rewrite :/
ᐧ
On Sun, May 3, 2020 at 10:50 AM Raphael Scheible<EMAIL_ADDRESS>wrote:

Is there a possibility to use the TF-version within pytorch or does it
require an extra implementation?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/cybertronai/pytorch-lamb/issues/8#issuecomment-623151774,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHZJMDXGDN5RVFFOV2OF73RPWVFJANCNFSM4MPCCIQQ
.


Add optional split output location to codegen scripts

This PR adds a new --out option to the core and iOS/Darwin code generation tools. This enables out-of-source code generation, useful for custom build systems and caching artifacts. This parameter defaults to whatever the value of root is (aka the old behavior) if not specified.

We can maybe remove the other argument since we are the only ones that use this script.

Yeah, just looked and it doesn't appear to be used anymore. I've gone ahead and removed it.


Fix CronWorkflow update command

relate https://github.com/argoproj-labs/hera-workflows/pull/134
I'm sorry :bow: I noticed some bugs and fixed it.

And, I got following error. I think argo_workflows package cause this. Do you know about it?
Reason: Bad Request
HTTP response headers: HTTPHeaderDict({'Content-Type': 'application/json', 'Date': 'Wed, 20 Apr 2022 07:53
:42 GMT', 'Via': '1.1 google', 'Transfer-Encoding': 'chunked', 'Alt-Svc': 'clear'})
HTTP response body: {"code":3,"message":"cronworkflows.argoproj.io \"argo-batch\" is invalid: metadata.r
esourceVersion: Invalid value: 0x0: must be specified for an update"}

No worries @kitagry 🙂  these things happen. Any chance the PR can include a test for the two logical branches? The test can mock the service and make it raise an error when name is not passed, an assertions that the service.update is called with expected namespace and name. I believe the addition of the tests will make the codecov check pass!

And, I got following error. I think argo_workflows package cause this. Do you know about it?

This type of error seems to have been reported in ArgoCD as well 🤔  See this issue for instance. Seems like the update might benefit from the following workflow:

get the cron workflow first
update spec, schedule, etc
keep existing metadata?

I will attempt to test this today.

@kitagry any chance you tried the update request with a metadata.resourceVersion? I believe this is the object and it can be accessed via wf.metadata 🤔

I haven't try it. I'll try this tomorrow, Thank you!

@flaviuvadan Thank you for your advice! I confirmed the operation.

@kitagry, this looks awesome! This PR delivers version 3.1.0. Any chance you are open to merging main, after we merger the 3.1.0 into main, into this branch and update the project version to 3.1.1 (it's a bug fix/patch) and add an entry to the changelog? 🙂

Actually, I am going to merge this first, then merge the other PR so we deliver a single version rather than 2. I will upgrade the CHANGELOG afterwards!


Support dated tags for refs (but on the log not the reflog)

Support dated revisions.
Git has this for its reflog, but I would rather have it just on the log.

[<refname>]@{<date>}, e.g. master@{yesterday}, HEAD@{5 minutes ago}
A ref followed by the suffix @ with a date specification enclosed in a brace pair
(e.g.  {yesterday}, {1 month 2 weeks 3 days 1 hour 1 second ago} or
{1979-02-26 18:30:00}) specifies the value of the ref at a prior point in time.
This suffix may only be used immediately following a ref name and the ref
must have an existing log ($GIT_DIR/logs/<ref>). Note that this looks up
the state of your local ref at a given time; e.g., what was in your local master branch last
week. If you want to look at commits made during certain times, see
--since and --until.

(see git help revisions).
Of course lakeFS has no concept of "local", so I would expect these to operate on The One True Commit Log.
For some tasks it would be very helpful to support these.  Examples include:

What happened since last week?
Has an object been GC'ed?
How many commits per day do I have on average?

@arielshaqed
I'm thinking of adding a param CreateTag Api called addDate. If addDate == true, we add the current time in UTC format to the tag. So, if the user provided tag is foo:
foo@{2023-08-20 21:09:09 }

Does that work?

@arielshaqed I'm thinking of adding a param CreateTag Api called addDate. If addDate == true, we add the current time in UTC format to the tag. So, if the user provided tag is foo:
foo@{2023-08-20 21:09:09 }

Does that work?

@vishalsanfran I missed this, sorry.
That actually solves a completely different set of user stories.  I don't actually want all these tags.  I want to be able to ask "what was the version 2 months ago when I ran this program", "what's changed in the past 2 months", and similar.  The idea is to help messy people like myself who discover they made a mess in the past.  Your solution will only help if I remembered to tag 2 months ago :-) .  But I'm afraid I'm too messy to remember to do that.
I am also not sure why implement this feature and not just use something like
lakectl tag create lakefs://repo/my-tag@$(date --iso-8601=seconds) lakefs://repo/ref


vte: 0.56.2 -> 0.56.3

Motivation for this change
vte minor bump.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Changes: https://gitlab.gnome.org/GNOME/vte/compare/0.56.2...0.56.3


[WIP]Add keywords

Q
A

Doc update?
yes

Relative Issues
closes #35

Should use correct syntactic rule (tabs or spaces?)

There is another Keyword that we can add: LITERAL1. I think that is useful for enum class

Yeah, i marked WIP in the header because of this
On Wed, 3 Oct 2018, 10:45 Andrea Salvatori - IU6FZL, <
<EMAIL_ADDRESS>wrote:

There is another Keyword that we can add: LITERAL1. I think that is useful
for enum class
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/F-Army/arduino-dw1000-ng/pull/105#issuecomment-426558157,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWi_NhTeCS0bAuSPYfjjsioe7UyuQMQQks5uhHk9gaJpZM4XFehM
.


Method visibility in VSCode

Dear Development Team,
our developer found this bug during writing extensions, seems very serious to me.
Can you please investigate and let us me know?
The procedure GetCutomerSellToFaxNo on table “Sales Invoice Header” is not set to external like the surrounding procedures.

However, when used from AL, it works fine without parenthesis (compiles and executes) when used in the context of a page or report as the default record:

However adding brackets (as per CodeCop above) causes the following error:

To summarise, we can call this non-external procedure as long as we don’t add parenthesis as long as it is the default record, which doesn’t seem to be intended.
Trying to reproduce this with my own procedure, I added the following to the customer table in C/AL and added it to my extension (after generating and redownloading symbols):

My own procedure was rejected properly regardless of whether it has parenthesis.
I then rewrote the page extension to be based upon Customer List instead of Posted Sales Invoices.

I then confirmed that it does indeed work:

Trying to find out how exploitable this is, I tested the following bit of code:

As you can see above, the version of the message statement enclosed by “with SalesInvoiceHeader do” compiles with a warning about parenthesis as we’ve already seen, the version outside the with block fails since the procedure isn’t external.
With the second message remarked out, the code executes just fine and reports the customer number and customer fax number of the last invoice, despite the GetSellToCustomerFaxNo not being external.
As before, adding parenthesis shows that the 3 procedures are not external:

In fact you can abuse this pretty much anywhere you want as long as you don’t explicitly state the record and use a with block and don’t add parenthesis.

Thank you, we will investigate this asap.


Added new quotes

Thank you for the PR.
As mentioned in #1

Add atleast 5+ quotes with tags, moods and feel to get your PR merged

@devcer I have added 5 quotes. Please accept the PR.

@Baibhab97 Add few more tags. Adding one tag for a caption won't be helpful for the data we are trying to collect.

@devcer Sure. I will do the necessary changes and raise PR.

@devcer I have added multiple tags. Please check and let me know if anything else needs to be changed.


Wrong calculation of MTR

In the example below, the calculated MTR value is -53.41%, whereas it should be (633-303)/633=52%:

When running the code in debug mode and calling this voxel, I get the correct value:
(data_mt0[124,134,16]-data_mt1[124,134,16])/data_mt0[124,134,16]
0.521327

This issue might be related to a reported issue related to division of numpy array of integer.
A solution would be to convert the input to float before doing the division. If confirmed, unit tests should be added to make sure this function behaves properly regardless the MRI input data type.

ah! the use of np.true_divide seems to solve that bug:
data_mtr = 100 * (data_mt0 - data_mt1) / data_mt0
data_mtr[124,134,16]
-51.39968404423381
data_mtr = 100 * np.true_divide((data_mt0 - data_mt1), data_mt0)
data_mtr[124,134,16]
52.13270142180095


Training Method for ShapeNet Data

I encounter an error while trying to train ShapeNet using the BaseComplete model, preventing the training from proceeding. Could you please explain the correct method to train ShapeNet?
creating data loader...
Loading data from ./datasets/shapenet/32/ and filtering with ./datasets/shapenet/val.txt
Missing counterparts: 0
Final dataset size: 0
Traceback (most recent call last):
File "./scripts/train.py", line 119, in
main()
File "./scripts/train.py", line 48, in main
val_data_loader = load_data_loader(
File "/mnt/sda1/shape-completion/model/datasets.py", line 118, in load_data_loader
loader = DataLoader(
File "/home/anaconda3/envs/shape-completion/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 353, in init
sampler = RandomSampler(dataset, generator=generator)  # type: ignore[arg-type]
File "/home/anaconda3/envs/shape-completion/lib/python3.8/site-packages/torch/utils/data/sampler.py", line 107, in init
raise ValueError("num_samples should be a positive integer "
ValueError: num_samples should be a positive integer value, but got num_samples=0

Hello,
you need to download the shapnet ground truth models from the link provided in README, and then generate the dataset using dataset_hole.py script. Then use should have the complete and incomplete counterparts for the training.
If you have done all of this and u still get the error, the problem is in the filtering when loading the models to dataset, adjust the file based on your needs.
If you need more help, feel free to ask.


Mock up the labeller

Write a python script to perform the following. Eventually the plan is to run this python code in this lambda function:

Download the sample CORE data dump file from here. Eventually this will need to use the full data dump, but probably easiest to start with the sample since the full dump is ~400gb and 1.8tb once fully extracted. For more information about the format of these dump files see the CORE documentation.
Extract contents
Process all the individual .json files containing information about individual CORE articles, and assign a randomly selected label to them from the Biomimicry Functions list in GDrive. You can just store the functions list in a python data structure, you don't have to write code to extract it from the google sheet. This code will go away once the labeller is working.
Save labelled data to PetalLabels dynamodb table using the fields described here

duplicate of #45


feat[Core] commit for #2483.  Adds a showGridFooter option that will cre...

...ate a grid footer with the Total Items and Shown Items displayed.  Also shows Selected Items if using Selection feature.
Breaking Changes in this commit:
showFooter option renamed to showColumnFooter
footerRowHeight option renamed to columnFooterHeight

Can't merge this yet.  I am seeing issues when columns are pinned right due to the grid footer interfering with the right render container


file access when trying to run server

Files located outside the server folder cannot be accessed.
The following command:
lib . file:~{PATH TO FILE} -d
does not work.
Only works if the file is located inside of the same folder as the service.
So the command:
lib . file:./{FILENAME} -d
would work.

This seems to be isolated to using the ~ character -- relative paths and absolute paths starting with a slash work.

@jacoblee93 Can you push a new version of this PR with a Windows-specific fix as well?


Issue with API

The business category api (https://saurav.tech/NewsAPI/top-headlines/category/business/in.json)is not found.
Could u please recheck?

business category is not defined in the CATEGORIES array
https://github.com/SauravKanchan/NewsAPI/blob/b2e7c331eb852b4f48cdf45d66afd439e84a2af5/main.py#L12
That is why this API is not dumping data from the News API business category.

yea ur right!

Added business


Update path regexp functionality

Old asterisk functionality is obsolete
Adds the repeated parameters mentioned in visionmedia/express#2134

This will break backward compatibility

Closing in favour of https://github.com/strongloop/express/pull/2530


poptorch.DataLoader only loads an integer number of batches

I've found a bug in Poptorch 3.3.0 (is this fixed in 3.4.0? I don't see it mentioned in the bugfixes https://docs.graphcore.ai/projects/release-notes/en/3.4.0/release_notes/poptorch.html).
Basically, it appears that the poptorch.DataLoader only loads an Integer number of batches, so if your batch size does not exactly divide the number of items in your dataset, it leaves out some of the data.
As an example, the training set for FashionMNIST is 60,000 images. If we load this in a poptorch.DataLoader with a batch size of 128, we lose 96 records:
import torch
import poptorch
import torchvision

train_dataset = torchvision.datasets.FashionMNIST("data/", transform=torchvision.transforms.ToTensor(), download=True, train=True)

opts = poptorch.Options()

train_dataloader = poptorch.DataLoader(
    opts, train_dataset, batch_size=128, shuffle=True, num_workers=20
)

num = 0

for data, labels in train_dataloader:
    num += labels.shape[0]

print(f"Loaded {num} records from the FashionMNIST training set of 60,000 records.")

(poptorch_env) [uccaoke@mandelbrot Graphcore]$ python3 popdlbug.py
Loaded 59904 records from the FashionMNIST training set of 60,000 records.

Ah! I have discovered drop_last which defaults to True but if you set to False resolves this behaviour.  Closing.

Er.
But then of course the compile results in code that doesn't work, because the size of the last batch is wrong:
[17:24:11.400] [poptorch::python] [critical] poptorch.poptorch_core.Error: 'poptorch_py_error': Shape mismatch for input_: expected torch.Size([256, 3, 28, 28]) but got torch.Size([140, 3, 28, 28]).
This error occurred because the inputs passed at runtime don't match the inputs used to compile the model.
To recompile the model for the new inputs create a new inferenceModel / trainingModel wrapper or call destroy() on the curent one and try again.

Traceback (most recent call last):
  File "/home/uccaoke/Source/ML_Playground/MedMNIST/Graphcore/test.py", line 128, in <module>
    _, loss = poptorch_model(inputs, targets)
  File "/home/uccaoke/Applications/PopTorch/poptorch_env/lib64/python3.9/site-packages/poptorch/_poplar_executor.py", line 1261, in __call__
    self._executable_inputs.validateInputs(in_tensors)
  File "/home/uccaoke/Applications/PopTorch/poptorch_env/lib64/python3.9/site-packages/poptorch/_args_parser.py", line 138, in validateInputs
    validate(self.arg_names[i], arg, inputs.args[i])
  File "/home/uccaoke/Applications/PopTorch/poptorch_env/lib64/python3.9/site-packages/poptorch/_args_parser.py", line 117, in validate
    raise _impl.createPoptorchError(
poptorch.poptorch_core.Error: 'poptorch_py_error': Shape mismatch for input_: expected torch.Size([256, 3, 28, 28]) but got torch.Size([140, 3, 28, 28]).
This error occurred because the inputs passed at runtime don't match the inputs used to compile the model.
To recompile the model for the new inputs create a new inferenceModel / trainingModel wrapper or call destroy() on the curent one and try again.

There is a note in the documentation about this issue: https://docs.graphcore.ai/projects/poptorch-user-guide/en/latest/batching.html#rebatching-iterable-datasets
Basically, you can't run an iteration on a partial batch because that would essentially require a recompilation of the model, so you either need to drop the last batch, or you need to re-batch in order to make the last batch a full batch.


Terminal keeps show parameter undefined

I've updated some gem, and this is happens. I using regular expressions instead of cucumber expressions, so how to hint the "parameter undefined" in my terminal? how to remove the "parameter undefined" in my terminal?
<img width="550" alt="Screen Shot 2022-02-16 at 09 59 12" src="https://user-images.githubusercontent.com/22050144/154187818-5f605d5f-e58a-4e45-8adb-daed7052b171.png"

Closing as this is a copy of the following: https://github.com/cucumber/common/issues/1907


Move to Python3 (mostly)?

Currently build_unicorn_support.sh  installs in python2 by default on linux.
I don't know what the python mutators expect or need.
However, as py2 is no longer supported starting in 40 days or so, I suggest dropping py2 support completely.

IMHO we should maybe support both?
the python plugin came in via a PR, so it was not done by us and it is not a focus by us. but yeah we have to do something about it I guess.

for unicorn @andreafioraldi or @domenukk would be best to take care of this, can one of you two own this?
I will take care of the afl-fuzz python plugin.

What has to be done? Simply port the example scripts to py3?

that and installing python3 instead of python2 I guess. but @domenukk should know better :)
plus downward compatibility if python3 is not availaible (maybe 1404/trusty doesnt have it? dunno)

I am done with the python mutator, its now python3 compatible (and python3.7 is prefered over 2.7 in the autodetectionin the Makefile)
for unicorn the test.sh script will also need to be updated

       Note that Python 3 or later is not yet supported.
       Use --python=/path/to/python to specify a supported Python.

The current version of unicorn is too old, we have to wait @domenukk 's patch

@domenukk when is the patch coming?
if this takes > 5 days to get it all done I would say we release a new version today/tomorrow

I'm on holidays until Monday.
The branch works and is here: https://github.com/domenukk/unicorn/tree/unicornafl?files=1
But the afl++ build scripts still need to be adapted to use the git submodule and not fail Travis.
Also the git history could be cleaned up if you care for these things.
Feel free to adapt the build scripts, release, or wait until Monday/Tuesday :)

Probably it's better not not use git, just download the archive from the
release page like now.
On Thu, Dec 5, 2019, 12:10 Dominik Maier<EMAIL_ADDRESS>wrote:

I'm on holidays until Monday.
The branch works and is here:
https://github.com/domenukk/unicorn/tree/unicornafl?files=1
But the afl++ build scripts still need to be adapted to use the git
submodule and not fail Travis.
Also the git history could be cleaned up if you care for these things.
Feel free to adapt the build scripts, release, or wait until
Monday/Tuesday :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vanhauser-thc/AFLplusplus/issues/118?email_source=notifications&email_token=AD3LJ6QTLJGEY2QXI6HGMO3QXDOUFA5CNFSM4JRQ6RCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGALSDY#issuecomment-562084111,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD3LJ6XJBHKLOAAJ4C3YDT3QXDOUFANCNFSM4JRQ6RCA
.

With PR #138 python3 now works fine for unicorn.
While Python Mutators haven't been touched, I will close this issue for now.

With PR #138 python3 now works fine for unicorn.
While Python Mutators haven't been touched, I will close this issue for now.

I had done that seperately 10 days ago. python3 detection plus changes to the mutator part to make it work :)


[Improve][SourceConnector] Unified schema parameter, update InfluxDB Source fields to schema

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
We need  unified schema parameter, All Source Connector which used schema need use the same option name schema.
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

please assign to me,thanks

please assign to me,thanks

please assign to me,thanks


module not registered: http.matchers.maxmind_geolocation

I've just tried to build Caddy (2.4.2) with this module, and Caddy complains that the module isn't registered.
Error during parsing: getting matcher module 'maxmind_geolocation': module not registered: http.matchers.maxmind_geolocation
xcaddy doesn't show any problems with the build as far as I can see.
Would appreciate some assistance in the matter.
Thanks!

Was an issue on my end with the build. Rebuilding fixed it.


Electron forge make requires electron-prebuilt-compile, causes npm start (electron-forge start) to fail

Preflight Checklist

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Issue Details

Electron Forge Version:

6.0.0-beta.52

Electron Version:

9.1.1

Operating System:

macOS Mojave 10.14.6

Last Known Working Electron Forge version::

none

Expected Behavior
Trying to clone an existing project on a new machine
Actual Behavior
Installed dependencies and when trying to start app, works fine. Then I tried run npm make (electron-forge make) says it needs electron-prebuilt-compile. I added it and now when I try to start app, it fails.
Shows the ff error:

To Reproduce

Additional Information
<!--
Add any other context about the problem here.

For example:
* Console output when you run your `electron-forge` command with the environment variable
  `DEBUG=electron-forge:*`. (Instructions on how to do so
  [here](https://www.npmjs.com/package/debug#usage)). Please include the stack trace if
  one exists.
* Command line arguments you are passing to `electron-forge` (e.g.,
  `electron-forge make --not-a-real-flag`)
* The `config.forge` data in `package.json` or `forge.config.js` in use
* A failing minimal testcase (with a link to the code) or detailed steps to reproduce the problem.
  Using `electron-forge init` is a good starting point, if that is not the source of your problem.

In order to debug your problem further, we need a minimal testcase to reproduce your problem. Using the npx create-electron-app exampleapp command as a base, could you please create a minimal Electron app that illustrates the issue you described, and post a link to the repository here?

Here is the repo link: https://github.com/cjcanlas01/exampleapp
I installed the initial dependencies and ran npm start and it started the electron app with no problem. However, I tried to run the ff command: electron-forge make --platform=win32 --arch=x64 which required me to add electron-prebuilt-compile.

Now I added electron-prebuilt-compile and building the windows distributable works but it caused npm start to fail.

You have electron-forge v5 installed globally. Uninstall that (npm uninstall -g electron-forge) and exclusively use npm run make.

But I am trying to build windows version of the app instead of the default that is mac.


Remove unnecessary dependency on commons-codec.

The whole library was used for just a single string constant.

Codecov Report

Merging #2945 into master will not change coverage.
The diff coverage is 0%.

@@          Coverage Diff           @@
##           master   #2945   +/-   ##
======================================
  Coverage    3.69%   3.69%
======================================
  Files         247     247
  Lines       12217   12217
  Branches     1083    1083
======================================
  Hits          451     451
  Misses      11732   11732
  Partials       34      34

Impacted Files
Coverage Δ

...n/java/fr/free/nrw/commons/logging/LogsSender.java
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Include the server side code into the repository

Need to include the server side code into the repository.
This repository is public, be careful not to include any private key or token.

Hi @Liquid-Blocks,
Please help to advise steps here.
You mean we should place this configuration somewhere?


Incompatible GPL license on pngquant-bin

pngquant-bin is licensed under GPLv3 which is incompatible with this packages MIT license.

Yeah, snyk just informed me of this as I was about to do some upgrades.


getNorthWest and getSouthEast in map.getBounds

Hi!
Is there a way for me to get the NorthWest and SouthEast object in the map.getBounds function?
When I console.log map.getBounds(); I only seem to be getting the NorthEast and SouthWest lat and lng.
I read some comments and have been able to get the correct lat and lng that I'm looking for. However, they dont update when the map is dragged as the NorthEast and SouthWest lat and lng update.
How can I use the leafletBoundsHelpers service to get this information?
Here is my current code
  leafletData.getMap().then(function(map) {
    var bounds = map.getBounds();
    var southEast = bounds.getSouthEast();
    var northWest = bounds.getNorthWest();
    console.log (southEast);
    console.log (northWest);

    $scope.southEast = southEast;
    $scope.northWest = northWest;

  var bounds = leafletBoundsHelpers.createBoundsFromArray([
    [ 51.508742458803326, -0.087890625 ],
    [ 51.508742458803326, -0.087890625 ]
  ]);

                Latitude northWest: <input type="number" step="any" ng-model="northWest.lat">
                Longitude northWest: <input type="number" step="any" ng-model="northWest.lng">

                Latitude SouthEast: <input type="number" step="any" ng-model="southEast.lat">
                Longitude SouthEast: <input type="number" step="any" ng-model="southEast.lng">

Thanks!

Hi @daniel-obadia, sorry for not responding before.
Could you provide a working example of this behaviour, if you still need it? A jsfiddle would be great. I think that the problem is that you need to apply the scopy after modifying the scope.southEast and scope.northWest objects, but it's better confirm it with an example.
Here's a clean jsfiddle to start with:
http://jsfiddle.net/maistho/9jymzymu/

This issue was moved to angular-ui/ui-leaflet#26

I'm going to rework&redesign angular-leaflet-directive to be compatible with Leaflet v1.0. It will mantain almost all its functionality, and will be compatible with the current features of the directive, but I must start from a fresh point, so I'm going to close this issue. If you think it must be worked with the new version, please reopen it.


Dashbuilder's showcase webapp fine tuning (0.4.x)

A few improvements:
.- Revisit the login/logout pages and add the i18n support
.- Nav header look&feel improved. Logout widget added

@romartin Can you review & merge please?

+1


Add marks in minimap

Is your feature request related to a problem? Please describe.
I'd like to visualize marks in the minimap which is available in satellite.nvim. I try to move to neominimap without needing to using 2 plugins providing similar functionalities.
Describe the solution you'd like
Optionally display marks in the minimap or scrollbar
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
N/A
Additional context
Add any other context or screenshots about the feature request here.
N/A
I'm currently a satellite.nvim user;  Starred it while back and glad to see how the plugin evolves (and love the looking of it). Would love to use it once supporting marks. Thank you!

Good idea. I'll implement this later.

Please checkout v3.5.0

Hey - thanks for adding the feature! However it seems I can't get it displayed within the minimap. I have tried override the vim.g.neominimap  before and after calling require("neominimap").setup() non-working (I'm not using layzvim).

May I see your configuration file?

May I see your configuration file?

Just tried again, it is working beautifully but only when I have satellite installed! When I comment out "Plug 'lewis6991/satellite.nvim', I see the following error msg. Then I found local util = require("satellite.util") in your code. So I assume it requires satellite as dependency? If so, you should list it in the README.md
Error executing vim.schedule lua callback: ...m/plugged/neominimap.nvim/lua/neominimap/events/mark.lua:6: module 'satellite.util' not found:
	no field package.preload['satellite.util']
	no file '.\satellite\util.lua'
	no file 'd:\Dropbox\neovim\nvim-win64\bin\lua\satellite\util.lua'
	no file 'd:\Dropbox\neovim\nvim-win64\bin\lua\satellite\util\init.lua'
	no file '.\satellite\util.dll'
	no file 'd:\Dropbox\neovim\nvim-win64\bin\satellite\util.dll'
	no file 'd:\Dropbox\neovim\nvim-win64\bin\loadall.dll'
	no file '.\satellite.dll'
	no file 'd:\Dropbox\neovim\nvim-win64\bin\satellite.dll'
	no file 'd:\Dropbox\neovim\nvim-win64\bin\loadall.dll'
stack traceback:
	[C]: in function 'require'
	...m/plugged/neominimap.nvim/lua/neominimap/events/mark.lua:6: in main chunk
	[C]: in function 'require'
	...ugged/neominimap.nvim/lua/neominimap/buffer/autocmds.lua:146: in function 'create_mark_autocmds'
	...ugged/neominimap.nvim/lua/neominimap/buffer/autocmds.lua:278: in function 'create_autocmds'
	...m/plugged/neominimap.nvim/lua/neominimap/buffer/init.lua:4: in function 'create_autocmds'
	...ovim\plugged\neominimap.nvim/lua/neominimap/autocmds.lua:8: in function 'create_autocmds'
	...lugged/neominimap.nvim/lua/neominimap/command/global.lua:15: in function 'open_minimap'
	...lugged/neominimap.nvim/lua/neominimap/command/global.lua:58: in function 'impl'
	...ropbox\neovim\plugged\neominimap.nvim/lua/neominimap.lua:12: in function 'on'
	...ropbox\neovim\plugged\neominimap.nvim/lua/neominimap.lua:124: in function 'fn'
	vim/_editor.lua:351: in function <vim/_editor.lua:350>

Separately, not sure if this will be of much work, can I suggest:

Show lables for staged lines which is shown by gitsign in a color diff from before staged
Connect dots for git changes (so it is consistent
Show both marks and gitsigns

The dependency of satellite is solved.

Infeasible because gitsigns doesn't have an API to differentiate between staged hunks and unstaged hunks.
What do you mean by "connect the dots"?
You can do this. One way to achieve this is

winopt = function(wo)
    wo.signcolumn = "yes:2"
end,

Or you can check out this

Thanks. Plz refer to the pic below:
Re 2: You see the signs on the left by gitsigns are in staight lines, neominimap shows dot-dot-dot. Also somehow the colors are not matching.
Re 3: yes I have "yes:2" for my signcolumn, I wanted to see if I could set the same within minimap (I guess not?).

FWIW, showing git status and marks simultaneously is supported in satellite.

In your minimap configuration, add these:
winopt = function(wo)
    wo.signcolumn = "yes:2"
end,
git = {
    enabled = true,
    mode = "icon",
    icon = {
        add = "│",
        change = "│",
        delete = "-",
    },
},

If your want to change the highlight color of gitsigns, set the following highlights manually:
NeominimapGitAddLine
NeominimapGitChangeLine
NeominimapGitDeleteLine
NeominimapGitAddSign
NeominimapGitChangeSign
NeominimapGitDeleteSign
NeominimapGitAddIcon
NeominimapGitChangeIcon
NeominimapGitDeleteIcon

Thanks much. Let me try the above out.
Re the highlight groups, it would be ppl's expectation to link your color groups to what gitsigns.nvim been using given the dependency :)

In your minimap configuration, add these:
winopt = function(wo)
    wo.signcolumn = "yes:2"
end,
git = {
    enabled = true,
    mode = "icon",
    icon = {
        add = "│",
        change = "│",
        delete = "-",
    },
},

git icons work as expected.
Setting wo.signcolumn = "yes:2" will expand the width of signcolumn within the minimap but still can't separeted marks from git status.

In this case, you may want to use statuscol. See this


dvl tests don't build

The tests for dvl won't build, looks like you just wanted to print out all the fields in the DVLData struct but I did't want to make assumptions. Not a huge deal I don't think, but something to be fixed for sure.
also need to uncomment building the tests in the new CMakeLists file that lives in package/drivers, we can also remove the Makefiles you created as long as you can still run your binaries.

looks like we don't use the test files anyway


Tooltip discrepancy

The documentation of components that use the sl-tooltip-enabled mixin say that the popover property requires the title property.  And tests/integration/components/sl-button-test.js tests it this way, but

tests/integration/components/sl-progress-bar-test.js
tests/integration/components/sl-textarea-test.js
tests/integration/components/sl-tooltip-test.js

do not set the title property and the tests pass just fine.
Also review

tests/integration/components/sl-alert-test.js
tests/integration/components/sl-button-test.js
tests/integration/components/sl-checkbox-test.js
tests/integration/components/sl-drop-button-test.js
for conformity.

the tooltip component requires title; the tooltip mixin does not. Still looking into the rest of this

@SpikedKira been working on this since last friday...as per my stand up status reports.

@notmessenger @SpikedKira was assigned to this even though we established I was going to work on it last friday per our discussion on jabber.

@SpikedKira See all the components that https://github.com/softlayer/sl-ember-components/issues/1080 affects, which is related to this work/conversation, etc.  Does the related PR for this issue affect enough files?

Thank you for pointing out that I missed some.


About install

When I use carthage install the Argo，It‘s spent a lot of time to Fetching LlamaKit，And finally, it's failure

@cconecode you're probably on a very old version of Argo. LlamaKit doesn't exist anymore and is now called Result. However, Argo has no dependencies itself. Perhaps check your Cartfile and Cartfile.resolved and make sure there isn't a weird version of Argo specified somewhere. Also make sure LlamaKit isn't specified in either of those files.

Closing this for now. @cconecode let me know if you're still having issues.


Audio-only files now producing error: right-hand side of 'instanceof' is not an object

Dragging a WAV onto the video player used to work throughout the entire chain of events. I am not sure changed, but just recently now I am getting "right-hand side of 'instanceof' is not an object " when dropping a WAV onto the file upload just after it completes uploading.

This is a regression from v2.0.10 onwards. Introduced by the upgrade of jsonwebtoken-esm to 2.x which contains a breaking change: https://github.com/auth0/node-jsonwebtoken/pull/852

@stipsan To elaborate on my comment above:
This regression was introduced by this commit from renovate: https://github.com/sanity-io/sanity-plugin-mux-input/commit/d13760fa540bb3c5e9a68283f7b5f00e9e40f4fb
A temporary fix is to downgrade to 2.0.9, but a structural fix is to ensure the key passed to sign the JWT is an isntance of crypto's KeyObject and not a string as currently (here) that like should most likely be: KeyObject.from(atob(signingKeyPrivate)) instead of just atob(signingKeyPrivate)

In case anyone sees that, there's another bug in version 2.0.9. So I took a look and version 2.0.12 seems to be the last one without this problem. at least it is working for our setup. It may be of help for you @sharpsteelsoftware1

I believe this was fixed in https://github.com/sanity-io/sanity-plugin-mux-input/pull/253.  If you're still seeing this issue in recent versions please reopen 🙏


Review and add tests to Coroutines support

To be done with @konrad-kaminski.

CoBodyBuilder probably needs to be updated, suspend fun <T> body(value: T?, elementClass: Class<T>): CoServerResponse as currently implemented makes little sense to me, maybe it should be replaced by a variant that takes a suspendable function in parameter in order to transform it internally to a Mono.


newick.py should ignore Nexus chattiness and comments

Often, the tree I want to parse using newick.py comes in a nexus file, with some taxa definitions and a #NEXUS line and some other garbage (for these intents) before it. That stuff should just be ignored, so that I can just feed such a file to the nexus parser directly.
Even more importantly, sometimes such a file (eg. the one in http://science.sciencemag.org/highwire/filestream/593618/field_highwire_adjunct_files/0/1219669IndoEuropean_2MCCtrees_annotated.zip) contains comments, i.e. text enclosed in [ and ]. Ignoring that noise would be helpful.

Moved to a comment on #7 : https://github.com/glottobank/python-newick/issues/7#issuecomment-191170359


Inconsistency with connecting to Azure hosted DBs

Issue Type: Bug
Issues with connecting to Azure hosted SQL Server instances.  I can connect with other SQL clients but not Data Studio.  One thing of note is that the first connection to the DB is usually successful, but the following queries are not.  There is also a notable issue that a user cannot switch query tabs and reuse the connection.
Azure Data Studio version: azuredatastudio 1.28.0 (13e362762762e5cb76a5c8afd2bc650f48c2d3d8, 2021-04-15T00:25:54.683Z)
OS version: Darwin x64 20.3.0

System Info

Item
Value

CPUs
Apple M1 (8 x 2400)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
5, 4, 4

Memory (System)
16.00GB (0.03GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

admin-pack
Mic
0.0.2

agent
Mic
0.48.0

dacpac
Mic
1.8.0

import
Mic
1.3.0

profiler
Mic
0.11.0

Hi @shoesCodeFor
Following up on the same, can you let us know if you're still experiencing any issues with ADS when connecting to Azure hosted SQL instances? Would recommend you to update ADS to latest version to validate the same.

Closing as inactive


time_bound variable requirement when computing monthly anomaly, annual mean, etc

@matt-long, while looking at

def compute_mon_anomaly(dsm) function
tb_name,tb_dim = time_bound_var(dsm)
grid_vars = [v for v in dsm.variables if 'time' not in dsm[v].dims]
variables = [v for v in dsm.variables if 'time' in dsm[v].dims and v not in ['time',tb_name]]

# save attrs
attrs = {v:dsm[v].attrs for v in dsm.variables}
coords = {v:dsm[v].attrs for v in dsm.variables}
encoding = {v:{key:val for key,val in dsm[v].encoding.items()
               if key in ['dtype','_FillValue','missing_value']}
            for v in dsm.variables}

#-- compute time variable
time_values_orig = dsm.time.values
date = cftime.num2date(dsm[tb_name].mean(tb_dim),
                       units = dsm.time.attrs['units'],
                       calendar = dsm.time.attrs['calendar'])
dsm.time.values = date
if len(date)%12 != 0:
    raise ValueError('Time axis not evenly divisible by 12!')

#-- compute anomaly
ds = dsm.drop(grid_vars).groupby('time.month') -
dsm.drop(grid_vars).groupby('time.month').mean('time')
ds.reset_coords('month',inplace=True)

#-- put grid_vars back
ds = xr.merge((ds,dsm.drop([v for v in dsm.variables if v not in grid_vars])))
ds.time.values = time_values_orig

attrs['month'] = {'long_name':'Month'}

# put the attributes back
for v in ds.variables:
    ds[v].attrs = attrs[v]

# put the encoding back
for v in ds.variables:
    if v in encoding:
        ds[v].encoding = encoding[v]

return ds

I noticed that you are finding the time_bound variable first. I am wondering whether this is a prerequisite condition for computing monthly anomaly? In other words, can we generalize this function? Is time_bound variable earth-system models specific? If not, can we just make it an optional argument? The goal would be to have a somewhat general function that one can use with most datasets.

I think we pull out the parts of this function that handle time and consider these operations to have already been applied prior to invoking this function.
Just as background, there are two issues with time:

Interpreting the metadata converting to a date object has historically been problematic thru xarray; many issues have been resolved and cftime provides functional time processing—though I still have trouble plotting timeseries with matplotlib and cftime objects as an axis.
CESM produces files with the time stamp at the end of an averaging period; this causes the monthly mean and other operations to fail because end-of-month or end-of-year values are pushed to the next (wrong) interval. A general solution to this is to average the time_bound field, if provided.

So in summary again, I think we should isolate the special time treatment tools elsewhere, which will clean things up here.

@matt-long, Does the following workflow make sense?

Call necessary operation function (compute_climatology(), etc). These operation functions would need to have some flags as optional arguments  that will allow us to do the necessary ops such as getting time bound var for CESM datasets for instance prior doing anything else.
Retrieve original attributes and metadata
Handle time related issues in one or two separate functions, get_time_bound_var(), compute_time_var()
Apply the necessary operation (climatology, anomaly, etc).
Update the resulting dataset/datarray with original attributes/metadata
Return xarray dataset

Here's a rough draft of pseudocode for this:

def time_bound_utils(dsm):
    tb_name,tb_dim = time_bound_var(dsm)

    grid_vars = [v for v in dsm.variables if 'time' not in dsm[v].dims]
    variables = [v for v in dsm.variables if 'time' in dsm[v].dims and v not in ['time',tb_name]]

    # save attrs
    attrs = {v:dsm[v].attrs for v in dsm.variables}
    encoding = {v:{key:val for key,val in dsm[v].encoding.items()
                   if key in ['dtype','_FillValue','missing_value']}
                for v in dsm.variables}

    #-- compute time variable
    date = cftime.num2date(dsm[tb_name].mean(tb_dim),
                           units = dsm.time.attrs['units'],
                           calendar = dsm.time.attrs['calendar'])
    dsm.time.values = date
    if len(date)%12 != 0:
        raise ValueError('Time axis not evenly divisible by 12!')

def update_attrs(ds):
    attrs['time'] = {'long_name':'Month','units':'month'}
    del encoding['time']

    # put the attributes back
    for v in ds.variables:
        ds[v].attrs = attrs[v]

    # put the encoding back
    for v in ds.variables:
        if v in encoding:
            ds[v].encoding = encoding[v]

def compute_mon_climatology(dsm, get_time_bound_var=False, **kwargs):
    '''Compute a monthly climatology'''
   if get_time_bound_var:
         time_bound_utils(dsm)
    #-- compute climatology
    ds = dsm.drop(grid_vars).groupby('time.month').mean('time').rename({'month':'time'})

    #-- put grid_vars back
    ds = xr.merge((ds,dsm.drop([v for v in dsm.variables if v not in grid_vars])))

    # Update attributes/metadata by calling `update_attrs()`
    update_attrs(ds)
    return ds

For this step:
#-- compute time variable
    date = cftime.num2date(dsm[tb_name].mean(tb_dim),
                           units = dsm.time.attrs['units'],
                           calendar = dsm.time.attrs['calendar'])
    dsm.time.values = date
    if len(date)%12 != 0:
        raise ValueError('Time axis not evenly divisible by 12!')

We would need to compute time variable when we have the time_bound_var, otherwise we just use the provided time variable in the original dataset, right?


Support unknown options

I would like to capture unknown options for a tool, that will be sent to another tool that takes arbitrary parameters used in a script. Is this possible somehow?
Example
This is what invocation could look like:
> mytool.exe --verbosity diagnostic --unknown value1 value2 --unknownflag --hello world

Where --unknown value1 value2 --unknownflag --hello world are parameters sent to the script.
And this is what I would want to do when parsing:
var result = ArgumentSyntax.Parse(args, syntax =>
{
    syntax.AllowUnknownOptions = true;
    syntax.DefineOption("v|verbosity", Verbosity.Normal, VerbosityParser.Parse);
});

IDictionary<string, IList<string>> unknown = result.GetUnknownOptions();

I understand that something like this might be out of scope for what you're building here, and I'm not even sure if the API that I've proposed makes any sense outside of my needs, but I thought I would ask.
If this is something that is of interest, I would be more than happy to provide a PR.

I want this too. Except it's probably enough just to retrieve them as IEnumerable<string>--the same way they went in.


Docker Secrets: documentation and consistency

Description of the feature
I understand that this image supports for Docker secrets, e.g. instead of putting your DB password in the DB_PASS environment variable (which could be accessible outside of the container), put it into a "docker secret" that docker mounts as file in the container file system (e.g. /run/secrets/db_password) and set the env variable DB_PASS_FILE=/run/secrets/db_password. See https://docs.docker.com/engine/swarm/secrets/ for the why's and how to's of Docker secrets.
However I found the documentation about this feature lacking: i.e. no indication of which variable names can be used to point to secret files containing which parameter.
E.g.:
can I pass the DB name as a secret? Apparently no.
Can I pass the DB password as a secret? Yes.
Which variable contains the path to the secret file for the DB password? DB_PASS_FILE
The answers to these questions don't seem to be in the readme and are not straightforward.
Only digging through the code of 10-db-backup I found out that the variable names that point to secret files use the _FILE suffix (which is common practice, but not dictated by Docker or by any official standard); and that they seem to be only DB_USER_FILE, DB_PASS_FILE, S3_KEY_ID_FILE and S3_KEY_SECRET_FILE.
Initially I was trying to use a secret also for the S3 bucket name, but having no success, I had to dig through the code to find out it wasn't me getting something wrong.
Also I am not sure why DB_USER_FILE is used for CouchDB, InfluxDB, MongoDB, but NOT for MySQL, PostgreSQL and Redis. This also came out when I was unsuccessful using a secret for the DB user for my MySQL DB and I had to dig through the code to understand why.
Suggestions:

document which variable can contain paths to secret files
if there is no technical issue unknown to me, extend the usage of DB_USER_FILE to al the other database types.

Benefits of feature
Documenting the "secrets" varialbe would make it easier for the user to get to know the support for Docker secrets and how to use it.
Making the DB username consistently "secret" across all DB types would improve security of implementation and also would make it easier for the user who don't need to know whethera specific secret can be used depending on DB type.

 can I pass the DB name as a secret? Apparently no.
looks like yes:
`You can use secrets to manage any sensitive data which a container needs at runtime but you don’t want to store in the image or in source control, such as:
Usernames and passwords
TLS certificates and keys
SSH keys
Other important data such as the name of a database or internal server
Generic strings or binary content (up to 500 kb in size)

`

Hi @samuk10,
yes the docker docs say they do support secrets. However support also needs to be built into the image. For example, in the same page, docker says:

The WordPress image has been updated so that the environment variables which contain important data for WordPress, such as WORDPRESS_DB_PASSWORD, also have variants which can read their values from a file (WORDPRESS_DB_PASSWORD_FILE). This strategy ensures that backward compatibility is preserved, while allowing your container to read the information from a Docker-managed secret instead of being passed directly.

So essentially, in order to support secrets, docker images need to read from specific env variables which secret files they need to use for what, so that the user can set those variables with the name of the files where they put the corresponding secrets. What secrets to support (e.g. one for the password, one for the username, etc.) and what variable is supposed to hold their name is up to the image developer.
docker-db-backup appears to be supporting some variables containing secret file names for some secrets.
However which secrets are supported (e.g. db name, db user, db password, etc.) and the corresponding variable names are not currently clearly documented, as far as I can see. So the user struggles identifying which data they can put in secrets and which variables to populate accordingly.
Just as an example, the mariadb image readme clearly documents which secrets are supported in which variables in a dedicated section

[...]
_FILE may be appended to the previously listed environment variables
[...]
Currently, this is only supported for MARIADB_ROOT_PASSWORD, MARIADB_ROOT_PASSWORD_HASH, MARIADB_ROOT_HOST, MARIADB_DATABASE, MARIADB_USER, MARIADB_PASSWORD and MARIADB_PASSWORD_HASH (and MYSQL_* equivalents of these)

got it!

3.9.0 adds support and README is clear on what is supported by means of _FILE environment variables..


Add table support

Cc: @astrofrog, @embray
As mentioned in astropy/astropy#3733, and related to spacetelescope/asdf-standard#83.

I've updated this to support recarray columns as discussed in spacetelescope/asdf-standard#83.  This requires astropy/astropy#3759 to be merged first, however.

Note to self: Add a test where one table is explicitly inlined by the writer and another is not (I suspect that doesn't currently work).


Updates for Swift 2.2

I updated the old __FILE__, __FUNCTION__ and __LINE__ to the new #file, #function and #line naming conventions.
Also removed empty whitespaces.

Thanks :+1:


Add flat loading

Proposed changes:

Add flat loading

Type of change:

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Thanks for the contribution @somdengre.