[bruteforce] Splunk login

#!/usr/bin/python
# -*- coding: utf-8 -*-

# pip2 install --user splunk-sdk

import splunklib.binding as binding
import logging

logging.basicConfig(level=logging.INFO)
log = logging.getLogger(__file__)

def splunk_login(host, port, username, password):
    """Login Splunk based on API.
    """
    login_status = False
    try:
        service = binding.connect(
            host=host, port=port,
            username=username, password=password)

        login_status = True
        log.info("Attempting %s:%d - Login successfully" % (host, port))

    except binding.AuthenticationError as e:
        log.info("Attempting %s:%d - %s" % (host, port, e.message))

    return login_status

if __name__ == '__main__':

    host = "splunklab.example.com"
    port = 8089
    username = "username"
    password = "password"

    splunk_login(host, port, username, password)

"""
$ python2 splunk_login.py
INFO:splunk_login.py:Attempting splunklab.example.com:8089 - Login failed.

$ python2 splunk_login.py
INFO:splunk_login.py:Attempting splunklab.example.com:8089 - Login successfully

"""

$ nmap -Pn -n -sV -p8089 <IP_ADDRESS>

Starting Nmap 7.00 ( https://nmap.org ) at 2018-03-08 04:00 EST
Nmap scan report for <IP_ADDRESS>
Host is up (0.0059s latency).
PORT     STATE SERVICE  VERSION
8089/tcp open  ssl/http Splunkd httpd

Service detection performed. Please report any incorrect results at https://nmap.org/submit/ .
Nmap done: 1 IP address (1 host up) scanned in 30.41 seconds


hasMap is for microdata only

Which schema.org object is this about if any?

Name - hasMap
Link - https://schema.org/hasMap

According to the spec, hasMap is for microdata only. Probably shouldn't be included in JSON+LD output but currently, it is.
{
  "@context": "http://schema.org",
  "@type": "LocalBusiness",
  "name": "Schema.NET.OneOrMany`1[System.String] Shropshire",
  "description": "",
  "image": "https://www.bluebirdcare.co.uk/getmedia/4d3f2336-0cf7-4ad0-bd4a-276b71e9d284/bluebird-logo-blue.png?width=160&height=98&ext=.png",
  "url": "https://www.bluebirdcare.co.uk/shropshire",
  "address": "3 The Parade\r\n Lansdowne Road\r\n Bayston Hill\r\nShrewsbury England SY3 0HT",
  "award": [],
  "contactPoint": [
    {
      "@type": "ContactPoint",
      "contactType": "customer support",
      "email"<EMAIL_ADDRESS>      "telephone": "+44 1743 874343"
    },
    {
      "@type": "ContactPoint",
      "url": "https://www.bluebirdcare.co.uk/shropshire/careers",
      "contactType": "customer support",
      "email"<EMAIL_ADDRESS>    }
  ],
  "hasMap": "https://www.google.com/maps/place/3%20The%20Parade%0D%0A%20Lansdowne%20Road%0D%0A%20Bayston%20Hill%0D%0AShrewsbury%20England%20SY3%200HT",
  "logo": "https://www.bluebirdcare.co.uk/getmedia/4d3f2336-0cf7-4ad0-bd4a-276b71e9d284/bluebird-logo-blue.png?width=160&height=98&ext=.png",
  "memberOf": {
    "@type": "Organization",
    "name": "UKHCA",
    "description": "United Kingdom Homecare Association's mission is to promote high quality, sustainable homecare so people can continue to live at home and in their community.",
    "url": "https://www.ukhca.co.uk/",
    "logo": "https://www.ukhca.co.uk/images/ukhca.png"
  },
  "parentOrganization": {
    "@type": "Organization",
    "name": "Bluebird Care",
    "description": "Bluebird Care provides the highest quality of home care, leading the field in customer service.",
    "sameAs": [
      "https://www.facebook.com/BluebirdCareUK/",
      "https://twitter.com/BluebirdCare",
      "https://www.linkedin.com/company/bluebird-care",
      "https://www.instagram.com/bluebirdcareuk/"
    ],
    "url": "https://www.bluebirdcare.co.uk/",
    "address": "Bluebird Care (Franchise Support Centre), Charles House, Charles Street, Petersfield, Hampshire, GU32 3EH",
    "award": [
      "Award2",
      "Award3",
      "Award1"
    ],
    "contactPoint": [
      {
        "@type": "ContactPoint",
        "contactType": "customer support",
        "email"<EMAIL_ADDRESS>        "telephone": "+44 1730 260 026"
      },
      {
        "@type": "ContactPoint",
        "contactType": "technical support",
        "telephone": "+44 1730 260 026"
      },
      {
        "@type": "ContactPoint",
        "url": "https://www.bluebirdcare.co.uk/careers",
        "contactType": "customer support"
      }
    ],
    "department": {
      "@type": "Organization",
      "name": "Head office",
      "address": "Bluebird Care (Franchise Support Centre), Charles House, Charles Street, Petersfield, Hampshire, GU32 3EH",
      "logo": "https://www.bluebirdcare.co.uk/getmedia/4d3f2336-0cf7-4ad0-bd4a-276b71e9d284/bluebird-logo-blue.png?width=160&height=98&ext=.png"
    },
    "logo": "https://www.bluebirdcare.co.uk/getmedia/4d3f2336-0cf7-4ad0-bd4a-276b71e9d284/bluebird-logo-blue.png?width=160&height=98&ext=.png",
    "memberOf": {
      "@type": "Organization",
      "name": "UKHCA",
      "description": "United Kingdom Homecare Association's mission is to promote high quality, sustainable homecare so people can continue to live at home and in their community.",
      "url": "https://www.ukhca.co.uk/",
      "logo": "https://www.ukhca.co.uk/images/ukhca.png"
    }
  },
  "telephone": "+44 1743 874343"
}

Interesting. These are the URL's we query to get info from Schema.org to generate all our code:
https://schema.org/version/latest/all-layers.jsonld
https://schema.org/docs/tree.jsonld
I don't see anything in there for us to be able to tell that we should not generate code for this property.

I would raise this issue with the schema.org GitHub repo and see what they say. If there is a way I can tell to remove this, I will.


Perfomance when wrap bottom bar with Stack.Navigator

The bottom tab is so lagging when i combine Curved Bottom Bar and Stack navigator. The layer is like this:

Stack.Navigator:

Stack.Screen with component is Curved Bottom Bar
Stack.Screen is other screen.

Thank you.

Yes i am also facing issue while using nesting navigation.when clicking tab icons so much delay to navigate

hi @ducpt-bili , @arun-saleth ,
What steps can I follow to replicate the issue you are experiencing?
Have you tried using React.memo and then wrapping the BottomBar?

@hoaphantn7604  I tried memo.Its woking perfectly.Thank you for your Suggestions

Hi @arun-saleth
Can you share with me where to handle useMemo?

@hoaphantn7604 @arun-saleth
it's slow for me, I don't know why, tried useMemo didn't change
maybe i wrote it wrong, could you explain how to use useMemo or if there is another solution to this issue please help

CHÀO@ducpt-bili,@arun-saleth, Tôi có thể làm theo những bước nào để tái hiện vấn đề bạn đang gặp phải? Bạn đã thử sử dụng React.memo và sau đó gói BottomBar chưa?

Do you mean wrap usememo to cover the entire bottom bar?


Data production tools

This change is

Coverage decreased (-0.1%) to 74.407% when pulling 4e3e9414be87c4cf5fc42db029d53da9d1dc841f on data-production-tools into 45b17c35a6ac6337d10e156da6bf41eeb5f6a96a on master.

Coverage increased (+0.06%) to 74.608% when pulling 62ee92697586d3f2190e9030aa2be2c0f512959d on data-production-tools into 45b17c35a6ac6337d10e156da6bf41eeb5f6a96a on master.

Coverage increased (+0.06%) to 74.604% when pulling f248c252e368ec67b8fd30ad9744cddade140cde on data-production-tools into 45b17c35a6ac6337d10e156da6bf41eeb5f6a96a on master.

Coverage increased (+0.06%) to 74.604% when pulling 504b0c8aa0dc7ab9da9bffb436142a980dc02c7f on data-production-tools into 45b17c35a6ac6337d10e156da6bf41eeb5f6a96a on master.


Adding section on how to call dtexec from Bash shell

Adding section on how to call dtexec from Bash shell, particularly to pass in parameters and connection string values.

@jeroenterheerdt : Thanks for your contribution! The author, @douglaslMS, has been notified to review your proposed change.

@craigg-msft or @CarlRabeler - can you review or assign to someone else? GitHub will not let me assign the PR to @douglaslMS .  thanks!

Please assign to Craigg-msft
From: Jak Koke<EMAIL_ADDRESS>Sent: Monday, March 4, 2019 12:37 PM
To: MicrosoftDocs/sql-docs<EMAIL_ADDRESS>Cc: Carl Rabeler<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/sql-docs] Adding section on how to call dtexec from Bash shell (#1708)
Assigned #1708https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fsql-docs%2Fpull%2F1708&data=02|01|Carl.Rabeler%40microsoft.com|e21fb2d0059e4697cd2308d6a0e11cf8|72f988bf86f141af91ab2d7cd011db47|1|0|636873286046301122&sdata=VTBmKqMhuxO6OwN6voBUe7nsrE%2BaP0M8d2cL3WOamUM%3D&reserved=0 to @CarlRabelerhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FCarlRabeler&data=02|01|Carl.Rabeler%40microsoft.com|e21fb2d0059e4697cd2308d6a0e11cf8|72f988bf86f141af91ab2d7cd011db47|1|0|636873286046301122&sdata=%2FO7wE4qsYt4Lh5DjlmhE%2B%2Fy4kkSNQkW7xJqVdOwS4KU%3D&reserved=0.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fsql-docs%2Fpull%2F1708%23event-2179047716&data=02|01|Carl.Rabeler%40microsoft.com|e21fb2d0059e4697cd2308d6a0e11cf8|72f988bf86f141af91ab2d7cd011db47|1|0|636873286046311131&sdata=aVie0Vd4gSAhijzDIOyR%2FOelTw5Vnz0GaAyIxcqkY60%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMCBnw0dVcTOAyENDpvdES6SOCc3ftGEks5vTYPbgaJpZM4bc79w&data=02|01|Carl.Rabeler%40microsoft.com|e21fb2d0059e4697cd2308d6a0e11cf8|72f988bf86f141af91ab2d7cd011db47|1|0|636873286046311131&sdata=V59XTA9hoOlOyXLUMEDtiDJNkwTFbkwdNQACL0%2BV1vE%3D&reserved=0.

@jeroenterheerdt : Thanks for your contribution! The author, @douglaslMS, has been notified to review your proposed change.

@JasonWHowell .

#sign-off

@JasonWHowell: I'm sorry - only the author of this article, @douglaslMS, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Add adjust_color_balance method in Image

Is your feature request related to a problem?
The Image Class prior to #506 had an adjust_color_balance method, which we removed because PyTorch has no such method.
Desired solution
Implement a method with that functionality, or implement a pre-trained NN.
Possible alternatives (optional)
No response
Screenshots (optional)
No response
Additional Context (optional)
No response

:tada: This issue has been resolved in version 0.18.0 :tada:
The release is available on:

v0.18.0
GitHub release

Your semantic-release bot :package::rocket:


added save to pdf operation

Hello,
Added save to pdf button to the render operation. It was mentioned in the issues. https://github.com/gchq/CyberChef/issues/1496
Please take a look!

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


internal error when creating popup attached to text property

Describe the bug
Creating a popup window attached to a text property might raise an internal error.
To Reproduce
Run this shell command:
vim -Nu NONE -S <(cat <<'EOF'
    call range(2)->setline(1)
    call prop_type_add('popupNumberColumn', {})
    call prop_add(1, 1, {'type': 'popupNumberColumn'})
    call popup_create('', {'textprop': 'popupNumberColumn'})
    $
    h
    call feedkeys("\r", 'xt')
EOF
)

E315 and E967 are raised:
E315: ml_get: invalid lnum: 3
E967: text property info corrupted

Expected behavior
No error is raised.
Environment

Vim version: 8.2 Included patches: 1-2188
OS: Ubuntu 16.04.7 LTS
Terminal: xterm(362)

Additional context
backtrace

I cannot reproduce this one

I cannot reproduce this one

Ah, I think it's because the issue is influenced by the terminal's height; the latter seems to need at least 24 lines.  Here is a new snippet which should work better:
set lines=24
call range(2)->setline(1)
call prop_type_add('popupNumberColumn', {})
call prop_add(1, 1, {'type': 'popupNumberColumn'})
call popup_create('', {'textprop': 'popupNumberColumn'})
$
h
call feedkeys("\r", 'xt')

Ah, I think it's because the issue is influenced by the terminal's height; the latter seems to need at least 24 lines.  Here is a new snippet which should work better:

I meant less than 24 lines, sorry.  I can reproduce with a default geometry (80 colums x 24 lines).

It should work no matter the terminal's geometry.

But it needs to be sourced like this:
vim -Nu NONE -S <(cat <<'EOF'
    call range(9999)->setline(1)
    call prop_type_add('popupNumberColumn', {})
    call prop_add(1, 1, {'type': 'popupNumberColumn'})
    call popup_create('', {'textprop': 'popupNumberColumn'})
    $
    h
    call feedkeys("\r", 'xt')
EOF
)

It should work no matter the terminal's geometry.

Ah no, it doesn't work with any geometry, but it works with 80 columns x 24 lines.

Ah no, it doesn't work with any geometry, but it works with 80 columns x 24 lines.

After doing a few more tests, it's more complex than that.
Depending on the height of the terminal:

there can be no issue
there can be an internal error and a text property info might be corrupted
there can be no error but Vim ends in a messed up state

1. happens when the buffer displayed in the bottom split contains fewer lines than what its window can display.
2. and 3. happen when the buffer displayed in the bottom split contains more lines than what its window can display.  2. happens below a certain height; 3. happens above.
Here is a gif showing the messed up state in which Vim ends up in 3.:

So, here is a command which guarantees the internal error to be raised:
vim -Nu NONE -S <(cat <<'EOF'
    set lines=24
    call range(2)->setline(1)
    call prop_type_add('popupNumberColumn', {})
    call prop_add(1, 1, {'type': 'popupNumberColumn'})
    call popup_create('', {'textprop': 'popupNumberColumn'})
    $
    h
    call feedkeys("\r", 'xt')
EOF
)

And here is another command which does not raise an internal error, but causes Vim to end in a messed up state:
vim --clean -S <(cat <<'EOF'
    set lines=42
    call range(999)->setline(1)
    call prop_type_add('popupNumberColumn', {})
    call prop_add(1, 1, {'type': 'popupNumberColumn'})
    call popup_create('', {'textprop': 'popupNumberColumn'})
    $
    h
    call feedkeys("\r", 'xt')
EOF
)


Cant acces storeweb in 0.20

Thanos, Prometheus and Golang version used:
Thanos : 0.20.0
Object Storage Provider: filesystem
What happened:
Trying to expose storeweb behind reverse proxy, working in 0.19, not in 0.20
What you expected to happen:
Working http exposition
How to reproduce it (as minimally and precisely as possible):
storeweb:
    image:  thanosio/thanos:v0.20.0
    logging:
      driver: "json-file"
      options:
        max-size: "10m"
        max-file: 3
    deploy:
      replicas : 1
      labels:
        - com.df.notify=true
        - com.df.serviceDomain=store
        - com.df.port=10902
    environment:
      - SERVICE_NAME=thanos-storeweb
    command:
        - tools
        - bucket
        - web
        - --objstore.config-file=/thanos/storage.yml
        #- --refresh=5m
    volumes:
      - /data/monitoring/prometheus:/prometheus
      - /data/monitoring/thanos:/thanos
    networks:
      default:
      monitoring:
      proxy:

Full logs to relevant components:
level=error ts=2021-05-03T07:55:24.087004883Z caller=http.go:48 protocol=http http.component=server msg="failed to parse host port for http log decision" err="address store.dc1.local: missing port in address"
Anything else we need to know:
Docker swarm, with docker flow reverse proxy

Hum, same problem than #4128 since i can access in 0.20 with "store.dc1.local/new/blocks" url, still the error thought

It seems a bit more related to https://github.com/thanos-io/thanos/pull/4041 - I'm actually not sure if there is a bug, or that there is some configuration missing.


Adding additional tags to ELBs for purposes of metrics collection

How will this be used? Couldnt the hostname and environment inferred from the elb_name or the elb_name from the hostname and environment?

You're right, it doesn't make sense to make both.
Because we separate them out into different tags for the instances and ASGs, I think it makes sense to do the same thing with the ELBs. So I think the elb_name tag should be removed and the name of the ELB should be inferred from the other tags.
I think using the same tagging structure for all three of these services should make it easier to do templating in both Grafana and Datadog, and it doesn't really make much sense to have different tagging conventions for things like this.

I want to combine this with #7


Add game event template

Desired Outcome
Add game event page.
Implemented Changes

Write function that displays GameEvent.
Add templates directory in game-event for GameEvent template:
Added in directory:

game-event.html
join-event.html
result/html

Add tests for game-event template
added routing for /game-event/ in meatballs/urls.py and in game-event/urls.py.

Connected Issue/Story
closes #76
Dependencies

[x] This PR does not depend on other PR's

Definition of Done

GameEvent will be displayed, with option to join to the event or removing yourself from an event.

Test coverage

This PR has tests related to the game-event template.

Documentation

[ ] This PR does not require updating any documentation

You can see how the site looks like for now
![צילום מסך 2023-05-14 ב-14 17 07](https://github.com/redhat-beyond/MeetBalls/assets/106595737/845d6369-cdc6

![צילום מסך 2023-05-14 ב-14 17 27](https://github.com/redhat-beyond/MeetBalls/assets/106595737/3a5c6a5b-0d75

-4a17-a96b-647cfa5f0109)
-4593-979c-dab3a238af72)

Looks good at all!
Please fix the PyTest fail for merging as soon as possible

Have to say, your PR is full with logic, so first of all Good Job !!
now i think the game-event.html can have a context of the game-event-players, and you should print out the participants as well.
this way you will know if event has some participants that brings the ball with them or not..
I think it's important, and not so complicated to implement, what do you say ?

Have to say, your PR is full with logic, so first of all Good Job !! now i think the game-event.html can have a context of the game-event-players, and you should print out the participants as well. this way you will know if event has some participants that brings the ball with them or not..
I think it's important, and not so complicated to implement, what do you say ?

You are defiantly right, I added it.
https://github.com/redhat-beyond/MeetBalls/assets/106595737/443c1c89-4272-45f3-aa13-21f7a969517e


Merge Upstream Maser into faiss

Merge Upstream Maser into faiss

Codecov Report

:exclamation: No coverage uploaded for pull request base (faiss@deeefeb). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             faiss      #10   +/-   ##
========================================
  Coverage         ?   78.53%
  Complexity       ?      396
========================================
  Files            ?       63
  Lines            ?     1565
  Branches         ?      141
========================================
  Hits             ?     1229
  Misses           ?      278
  Partials         ?       58

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 deeefeb...f549e3e. Read the comment docs.

Codecov Report

:exclamation: No coverage uploaded for pull request base (faiss@deeefeb). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             faiss      #10   +/-   ##
========================================
  Coverage         ?   78.53%
  Complexity       ?      396
========================================
  Files            ?       63
  Lines            ?     1565
  Branches         ?      141
========================================
  Hits             ?     1229
  Misses           ?      278
  Partials         ?       58

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 deeefeb...f549e3e. Read the comment docs.


panc: code cleanup and speedup

Main performance improvements through caching; further code cleanup to get saner callggraphs and traces

This cuts our full compile time in half (around 1000 profiles), the checkStringIndex is roughly 10-15% and should be generic for all usage (given enough profiles); the retrievePanSource caching is 35-40% gain (given enough profiles and usage of includedirs (5 in our case, via cluster.build.includes) and LOADPATH (2 in our case))
YMMV

Just tested this at RAL:
aquilon
With a small test sandbox this does seem to reduce the compile time slightly, but more importantly introduces no unusual changes to the profiles.
scdb
No measurable speed-up, but also no changes to profiles.

@jrha heh, i'm surprised and a bit dissapointed. if you find the time, you could try to connect viusalvm to the compiler (just start visualvm, it will see any java proc) and do cpu profile dump (it will slow things down). i can have a look and see if anything is different in aquilon.

I'm not sure the aquilon test was large enough to draw any conclusions from, I can profile the scdb build, but I don't know what I'm doing!

@jrha just connect visualvm, start cpu profiing, stop cpu profiling and send me the file 😄

Ok sure, which file? :innocent:

Quick testing shows that based on the "Total time" reported by panc itself, there's about 10% speedup when generating uncompressed JSON (nice), and about 20% slowdown when generating uncompressed XML (ouch!), both compared to 10.2. Even in the XML case, some batches are slower and some are quicker, so I'll need to dig a bit deeper into where the time is spent. In real life, compressed output speed is what matters for us, but using compression makes comparing the output between two runs more difficult - I'll try that later if I find the time.
The only other difference I see is some Unicode characters, which were previously replaced by '??', now are appearing in the output.

@gombasg thanks for testing, but can you also compare to current master https://jenkins0.ugent.be/job/panc/1822/org.quattor.pan$panc/
i do not know where the xml slowdown could come from; but i haven't tested xml output myself. i also do not know what you use to compare, but there shouldn't be a real issue with using gzipped files (we use  the following to compare gzipped json diff -u <(zcat old.json.gz) <(zcat new.json.gz)) (well wrapped in script etc https://github.com/stdweird/quattor-SCDB-ugent/blob/master/zdiff.sh)

@gombasg jar files from latest master are in jenkins, eg https://jenkins0.ugent.be/job/panc/lastSuccessfulBuild/org.quattor.pan$panc/

Closing this PR. will be replaced by smaller PRs, hoping this makes merging easier


Build failing during npm start

There are some extra commas in RequestController and DeliveryController that are causing the build to fail at runtime with npm start
Getting the error
Starting child process with 'node ./bin/www'
/home/abhi/Dev/openSource/chefsForPR/controllers/RequestController.js:26
        ));
        ^

SyntaxError: Unexpected token )
    at createScript (vm.js:56:10)
    at Object.runInThisContext (vm.js:97:10)
    at Module._compile (module.js:542:28)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/home/abhi/Dev/openSource/chefsForPR/controllers/index.js:2:28)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
Program node ./bin/www exited with code 1

I can take this on

@apravink can you updgrade your node to version 8? it fails on previous versions because it uses the arrow function syntax in a few places. If you can upgrade that's better. otherwise we can consider not using the new syntax, but there are some improvements in Node8 that are worth having, https://nodejs.org/en/blog/release/v8.0.0/


Fix hiding comments

Closes #1561.

GH changed the aria-label attribute of the summary tag.
Now we have to close the parent dropdown manually (don't know how that was closed automatically before, or if it even used to close before).
I know that I changed the comments and variable names a little bit, will be waiting for @bfred-it to suggest changes.

@sindresorhus Anything Holding this from being merged?

Seems ok


Fix checkmark for clean repo in bureau theme

The checkmark in clean repo wasn’t dispalyed. Now it fixed.

I’ve just rebase the branch from master.

Cleaner diff, for reference.
Can you go a little further and turn those grep calls into command grep? That way we bypass the grep alias.

I’ve replaced grep with command grep, thanks for issue.

:+1:

:+1:

This change seems to have broken the logic for reporting 'clean' repos.  Still testing, but at least when a repo is 'ahead', it still shows the checkmark for 'clean'.

Yes, sorry, I'm aware of it and working on a PR. I'll let you know when it's fixed.

@mtnbikenc please have a look at #4976.


method to check whether popup is showing or not

First of all,
thanks for this library, is there any way i can know,  whether popup is showing or not, as i know, a method exist there but i can compare it to check whether  popup is dismissed or showing.
like nPopupEvent: (event, selectedMarkers) {}
here event  is a sealed, and i am not able to use it whehter it is showing or hiding.
Regards,
Vikram Singh

Hi @TheReprator, yes this is possible. See: https://pub.dev/documentation/flutter_map_marker_popup/latest/flutter_map_marker_popup.extension_api/PopupState/isSelected.html
You can also get a list of the selected markers (selected means they have a popup showing), you can see how this is used in the example: https://github.com/rorystephenson/flutter_map_marker_popup/blob/1c88a84a877e71fc98e0a1845f47125f9ff369da/example/lib/popup_outside_of_map.dart#L114

@rorystephenson apology, i hadn't gone through that, but after using this, i am getting following error,
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: 'package:provider/src/provider.dart': Failed assertion: line 274 pos 7: 'context.owner!.debugBuilding || E/flutter (12245):           listen == false || E/flutter (12245):           debugIsInInheritedProviderUpdate': Tried to listen to a value exposed with provider, from outside of the widget tree. E/flutter (12245):  E/flutter (12245): This is likely caused by an event handler (like a button's onPressed) that called E/flutter (12245): Provider.of without passing listen: false. E/flutter (12245):  E/flutter (12245): To fix, write: E/flutter (12245): Provider.of<PopupState?>(context, listen: false); E/flutter (12245):  E/flutter (12245): It is unsupported because may pointlessly rebuild the widget associated to the E/flutter (12245): event handler, when the widget tree doesn't care about the value. E/flutter (12245):  E/flutter (12245): The context used was: ScreenMap(state: _ScreenMapState#1523c)
Repo:  https://github.com/TheReprator/flutter_map_tracking/blob/marker_bug/lib/screens/screen_map.dart

@TheReprator Try PopupState.maybeOf(context, listen:false)!.selectedMarkers

@rorystephenson
PopupState.maybeOf(context, listen: false)?.selectedMarkers
it is always returning null,  whether marker is clicked or not, on tap of map.

@rorystephenson PopupState.maybeOf(context, listen: false)?.selectedMarkers
it is always returning null, whether marker is clicked or not, on tap of map.

That's odd, can you share a minimal reproducible example?

@TheReprator I just had a quick look at your repo and you don't seem to be using PopupScope. PopupState is available from a child of PopupMarkerLayer. If you need to access the PopupState from a widget which is not a child of PopupMarkerLayer you need to raise the state up by wrapping a parent widget, for example Scaffold, in a PopupScope. Then you can call PopupState.of(context) using the context from a descendant widget of PopupScope.


Convert amateur_test to capybara

Coverage: 93.673%. Remained the same when pulling 0e9e6553e41aaabe17c1f4f8bab1b463a4aa16bf on nimmo-refactor-post-observation-test-capybara into d5acf1371acde197899d6cd116a6f87d8bba97db on main.


script: Support installing from a clean state

This adds a --clean flag to the install script which will do the following before proceeding with the installation:

prompt the user to confirm the clean
stop flynn-host
kill old containers
destroy ZFS volumes
remove Flynn files and directories

Closes #1208.

LGTM!


Add file .env.json file and process.env support in the whole project with webpack.EnvironmentPlugin

@venimus please check the comment at the .gitignore.
@velzevur I finnaly fixed it. @DeeJayElly also check.
Thanks to @Chima1707 that he inspire me to create this PR because of his questions about project up and run and compiling while he checking my other PR.
My idea is to use .env.json.
and next it can be used in the codebase like
process.env.SOME_VAR

it can he handy.
So at.env.json we can define:
{
    "WEBPACK_DEV_SERVER_PROXY_TARGET": "http://league.superhero.com/"
}

And webpack config itself will use it and other files too.
PS: it also may be a good contribution for the upstream because they users have some problems with environment like I saw in issues.

Also can be upstream contribution

@venimus it also can be just an .env file but I need to parse it. I can focus on that after tip button delivery, please help me make sure that it's uploaded.


GitHub field

Integrate the Github field into Person and update tests accordingly.
Note that the respective data folders now contain Github as a mandatory field.

Codecov Report

Merging #88 (896dd91) into master (0843663) will increase coverage by 0.67%.
The diff coverage is 81.48%.

@@             Coverage Diff              @@
##             master      #88      +/-   ##
============================================
+ Coverage     62.10%   62.77%   +0.67%
- Complexity      464      477      +13
============================================
  Files            83       84       +1
  Lines          1660     1698      +38
  Branches        172      179       +7
============================================
+ Hits           1031     1066      +35
- Misses          572      573       +1
- Partials         57       59       +2

Impacted Files
Coverage Δ

.../java/seedu/address/logic/commands/AddCommand.java
100.00% <ø> (ø)

.../java/seedu/address/model/util/SampleDataUtil.java
20.00% <0.00%> (ø)

...c/main/java/seedu/address/model/person/Github.java
80.00% <80.00%> (ø)

...java/seedu/address/logic/commands/EditCommand.java
94.93% <87.50%> (-0.96%)
:arrow_down:

...c/main/java/seedu/address/model/person/Person.java
98.11% <87.50%> (+0.19%)
:arrow_up:

...a/seedu/address/logic/parser/AddCommandParser.java
86.36% <100.00%> (+0.64%)
:arrow_up:

...ain/java/seedu/address/logic/parser/CliSyntax.java
87.50% <100.00%> (+1.78%)
:arrow_up:

.../seedu/address/logic/parser/EditCommandParser.java
93.33% <100.00%> (+0.47%)
:arrow_up:

...in/java/seedu/address/logic/parser/ParserUtil.java
97.82% <100.00%> (+0.26%)
:arrow_up:

.../java/seedu/address/storage/JsonAdaptedPerson.java
93.33% <100.00%> (+1.02%)
: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 0843663...896dd91. Read the comment docs.


Fixes issue with wrong test description when using JUnit 4.12 and above.

With JUnit 4.12 and above the org.junit.runners.BlockJUnit4ClassRunner#describeChild method caches the created method description. This leads to several issues when using the WebDriverRunner:

Test methods are named same even when related to different browsers
Screenshot files are only stored once since the screenshot name is based on test method description.

The JUnit change can be seen here: https://github.com/junit-team/junit/commit/08719e9b61d07c17b6a4f599ecd2cd1a1519120a
This change restores the original behavior by overriding describeChild method in WebDriverRunner!

Thanks, I'll merge this as soon as I have time to take a look at it. Would preferably upgrade the junit version in the pom as well.

Just release your PR in version 2.2.0.


Walk removed?

Hey, the readme for the project still says that that project proxies walk from fs-extra, but walk was removed in fs-extra 2.0.0.
Do you still intend to support walk? fs-extra moved it into its own package klaw

@MarcoThePoro Hey, thanks for the catch! I didn't realize the README still mentioned walk. fs-promise does not support it; we just didn't correctly update the docs. Care to submit a PR removing it from the readme?

Yeah sure
On Wed., 19 Apr. 2017, 12:09 am Ryan Zimmerman<EMAIL_ADDRESS>wrote:

@MarcoThePoro https://github.com/MarcoThePoro Hey, thanks for the
catch! I didn't realize the README still mentioned walk. fs-promise does
not support it; we just didn't correctly update the docs. Care to submit a
PR removing it from the readme?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kevinbeaty/fs-promise/issues/28#issuecomment-294856469,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHazr0HYLeYQg0K2PkosAVfZN46Dpj1Bks5rxMQhgaJpZM4NAIDp
.


fix reviewapps

Description

Fixes # (issue)
Type of change

[ ] Document update or simple typo fix
[ ] Maintenance/update components
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

[ ] I have checked backward/forward compatibility that may cause regarding this change.

dev へのみの影響のため merge してしまいます。


Inspect not showing setting definition location

steps

Define a local plugin FooPlugin

import sbt._, Keys._

object FooPlugin extends AutoPlugin {
  override def requires = plugins.JvmPlugin
  override def trigger  = allRequirements

  override def projectSettings = Seq(
    unmanagedJars in Compile := {
      val cp = (unmanagedJars in Compile).value
      println("hi mom")
      cp
    }
  )
}

problem
The setting is definitely applied:
> show unmanagedClasspath
hi mom
[info] *
[success] Total time: 0 s, completed 03-Aug-2017 17:44:58

> show compile:unmanagedClasspath
hi mom
[info] *
[success] Total time: 0 s, completed 03-Aug-2017 17:45:04

But it's not appearing in inspect:
> inspect compile:unmanagedClasspath
[info] Task: scala.collection.Seq[sbt.Attributed[java.io.File]]
[info] Description:
[info] 	Classpath entries (deep) that are manually managed.
[info] Provided by:
[info] 	{file:/s/t-unmanagedJars/}t/compile:unmanagedClasspath
[info] Defined at:
[info] 	(sbt.Classpaths) Defaults.scala:1114
[info] Dependencies:
[info] 	compile:unmanagedClasspath::streams
[info] 	compile:buildDependencies
[info] 	compile:settingsData
[info] 	compile:configuration
[info] 	compile:thisProjectRef
[info] Reverse dependencies:
[info] 	compile:externalDependencyClasspath
[info] Delegates:
[info] 	compile:unmanagedClasspath
[info] 	*:unmanagedClasspath
[info] 	{.}/compile:unmanagedClasspath
[info] 	{.}/*:unmanagedClasspath
[info] 	*/compile:unmanagedClasspath
[info] 	*/*:unmanagedClasspath
[info] Related:
[info] 	test:unmanagedClasspath
[info] 	runtime:unmanagedClasspath

expectation
I expect FooPlugin to be present under "Defined at".
notes
sbt version: 0.13.16

Also does not appear if the formulation is:
unmanagedJars in Compile ~= { cp =>
  println("hi mom")
  cp
}

One is Compile / unmanagedClasspath and the other is Compile / unmanagedJars.
Are you asking for inspect tree but for defined at?

Well, it looks like I've recovered from the blindness I'd developed in both eyes back in 2017...


Improve mix() fallback. Fixes #23043

Please check #23043 for reasonings behind this.

I am closing this pull request because it lacks sufficient explanation, tests, or both. It is difficult for us to merge pull requests without these things because the change may introduce breaking changes to the framework.
Feel free to re-submit your change with a thorough explanation of the feature and tests - integration tests are preferred over unit tests. Please include it's benefit to end users; the reasons it does not break any existing features; how it makes building web applications easier, etc.
Thanks!

All explanations must be with the original PR.


No 'version' key in the manifest file for custom integration 'gecko'

I noticed this in the logs today :
2021-03-10 17:07:07 WARNING (MainThread) [homeassistant.loader] No 'version' key in the manifest file for custom integration 'gecko'. This will not be allowed in a future version of Home Assistant. Please report this to the maintainer of 'gecko'

Thanks for the spot. I'm working on another project at the moment, but I'll try to get this fixed soon so that it doesn't break HA.

I think it would probably be as simple as adding :
"version": "1.0",
into :
https://github.com/gazoodle/gecko-home-assistant/blob/master/custom_components/gecko/manifest.json
let me add that line manually on my installation to see if if fixes the error

Yeah, just dropping that in there work for me. I created a pull request with the change. Hope I did it right, it's the first one I've done on github

@brewston fixed, thanks!


Do an editorial pass

Intentionally left the editorial doc blank for another PR

I already crossed off the items resolved by this PR in https://github.com/TheBevyFlock/bevy_quickstart/issues/14


Feature/expose error type

I tried to export EtaError types introducing new types like @nebrelbug did in THIS issue.
Some point about he PR:

I put an EtaXXXError instead EtaError in all point. So at the and I asked myself "Is now useless EtaError type? Can I remove it"
I added a test for each error interface. I tried to throw starting from top-level api. For example using .render() even if the exception is inside .readFile()
I added never as return type since for me a function that throw 100% of times an error equals to a function that never run an error (token from TS docs)

I don't know if this PR follow exactly the structure described here:
https://github.com/eta-dev/eta/issues/243#issuecomment-1646243319
But probably this PR can be a starting point and I'm happy to change some part after review

@multivoltage thanks for the PR! This looks great. Just a few changes before I can merge:

Rename EtaRuntimeErr to EtaRuntimeError
I think it would be great if all Eta errors extended a base class EtaError, which could just be declared like class EtaError extends Error {}.

I appreciate the TypeScript updates!

Also if you can make sure to follow the Commitlint format for commit messages, that's what's causing tests to fail right now. Thanks!!

npx commitlint --from ee0a1d44addd162b53b20329799b93628fcba22e~6 --to ee0a1d44addd162b53b20329799b93628fcba22e --verbose
⧗   input: feat: export EtaNameResolutionError
✔   found 0 problems, 0 warnings
⧗   input: feat: add use case for EtaFileResolutionError
✔   found 0 problems, 0 warnings
⧗   input: feat: export EtaFileResolutionError
✔   found 0 problems, 0 warnings
⧗   input: feat: use custom error for compile() api
✔   found 0 problems, 0 warnings
⧗   input: feat: export EtaRuntimeError
✔   found 0 problems, 0 warnings
⧗   input: feat: export EtaParseError
✔   found 0 problems, 0 warnings

In my local pc seems ok (node 18)

@multivoltage hmm, you're right. I'm not actually sure what the problem is, but it seems like it may be a problem with the version of commitlint that the GitHub Action is installing.

@nebrelbug I replicated issue.
In. my local pc like I said I run "npx commitlint....".
But in you yaml you install before
      - name: Install commitlint
        run: |
          npm install conventional-changelog-conventionalcommits
          npm install commitlint@latest

and then use npx.
I think you have 2 possible fix:

remove step to install the two deps and keep going to use "npx commitlint" like in my local pc.
Installing same deps in my pc and run "npx commitlint" the error is the same. I solved running also @commitlint/format

Please give me a feedback. Do you plan to fix this in seperate pr?

@multivoltage I'll fix the commitlint issue in a separate PR later. In the meantime, I'll merge anyways if you can fix the issues I mentioned above.

@nebrelbug
Sorry. I did not notice your first comment :)

I think it would be great if all Eta errors extended a base class EtaError, which could just be declared like class EtaError extends Error {}.

I thought the same thing inirtially but then I ask my self?

What EtaError give me more than a basic Erro? Anyway I will do it :)

@multivoltage thank you! Having a base EtaError class will be useful if people want to catch all errors that Eta throws.

@multivoltage released in Eta 3.4.0:)


Issue of 100% CPU usage when closing WebSocket connections.

Prerequisites

[X] I am using the latest version of Locust
[X] I am reporting a bug, not asking a question

Description
Custom WebSocket stress test script, environment: 20 cores, 32GB RAM, 1000M bandwidth. Run the script in this environment. Only log in one user and do not send messages, just need to stay online. When clicking STOP on the web UI interface, after 20-30 seconds, a warning appears: 'CPU usage above 90%!'. If using --headless, this problem obviously doesn't occur. However, consider a situation where, during automated loading, this might be unfriendly.
Command line
locust -f mylocustfile.py
Locustfile contents
#!/usr/bin/env python
# -*- coding: utf-8 -*-

class WebSocketUser(HttpUser):
    wait_time = between(1, 2)

    def on_message(self, ws, message):

    @staticmethod
    def on_close(ws, close_status_code, close_msg):

    @staticmethod
    def on_open(ws):

    @task
    def pft(self):
        ws = websocket.WebSocketApp('wss:xxxxxxxxxxxxxxxxx&transport=websocket',
                                    on_message=self.on_message,
                                    on_error=self.on_error,
                                    on_close=self.on_close)
        ws.on_open = self.on_open
        self.run_forever_with_retry(ws)
        self.active_websockets.append(ws)

Python version
3.9.12
Locust version
2.20.0
Operating system
Linux \ mac

Investigation revealed that the issue is with the websocket-client version


EncodingWarnings when warn_default_encoding set

When running tests with pytest-mypy enabled and PYTHONWARNDEFAULTENCODING=1, I see these warnings:
docs/conf.py::mypy
docs/conf.py::mypy
  /Users/jaraco/code/jaraco/pytest-enabler/.tox/python/lib/python3.11/site-packages/pytest_mypy.py:272: EncodingWarning: 'encoding' argument not specified
    stdout, stderr, status = mypy.api.run(

docs/conf.py::mypy
  /Users/jaraco/code/jaraco/pytest-enabler/.tox/python/lib/python3.11/site-packages/pytest_mypy.py:312: EncodingWarning: 'encoding' argument not specified
    with open(results_path, mode="w") as results_f:

docs/conf.py::mypy-status
pytest_enabler/__init__.py::mypy
tests/test_enabler.py::mypy
  /Users/jaraco/code/jaraco/pytest-enabler/.tox/python/lib/python3.11/site-packages/pytest_mypy.py:306: EncodingWarning: 'encoding' argument not specified
    with open(results_path, mode="r") as results_f:

================================================================================== mypy ===================================================================================
Success: no issues found in 3 source files
======================================================================== warnings summary (final) =========================================================================
.tox/python/lib/python3.11/site-packages/pytest_mypy.py:331
  /Users/jaraco/code/jaraco/pytest-enabler/.tox/python/lib/python3.11/site-packages/pytest_mypy.py:331: EncodingWarning: 'encoding' argument not specified
    with open(config._mypy_results_path, mode="r") as results_f:

See PEP 597 for background.

Thanks for the report!
I can't figure out what's causing that first one:
docs/conf.py::mypy
docs/conf.py::mypy
  /Users/jaraco/code/jaraco/pytest-enabler/.tox/python/lib/python3.11/site-packages/pytest_mypy.py:272: EncodingWarning: 'encoding' argument not specified
    stdout, stderr, status = mypy.api.run(

I filed an issue with Mypy for that: https://github.com/python/mypy/issues/14603

Thanks for the report!
I filed an issue with Mypy for this one:
docs/conf.py::mypy
docs/conf.py::mypy
  /Users/jaraco/code/jaraco/pytest-enabler/.tox/python/lib/python3.11/site-packages/pytest_mypy.py:272: EncodingWarning: 'encoding' argument not specified
    stdout, stderr, status = mypy.api.run(


Update the AdaGrad optimizer class API docs 📒

This PR updates the AdaGrad optimizer class API docs, following similar work on Adam.

Expanded the description of AdaGrad ↔ TF 2.x/Keras docs.
Defined a bunch of parameters from the existing comments.
Added a small LeNet example adapted from /swift-models/ (tested separately on MNIST).

I'm continuing going through the rest of the commonly used APIs. More PRs on their way.
Thanks team.

(hope you don't mind the commit's name - a late night copy/paste error - or else I can undo and redo with a proper name)

the 2nd commit dealing with line-wrapping was probably not necessary @saeta but it removed the kokoro:run


feat(checkpoints): add checkpoints.create to the Security Token Entity

Also upgraded the SDK to use Polymesh 2.5

:tada: This PR is included in version 4.0.0-alpha.36 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 5.0.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 5.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 5.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Type conversions generated in cases where they're not needed.

I ran into a real head scratcher today, and I think I figured out the root cause. I wrote up the whole issue at https://github.com/IntelliTect/Coalesce/issues/25, which I'd encourage a read through of to get context for this. Look specifically at the (System.Object) cast in the second query's DebugView.
Basically, it seems that type conversions are being generated in cases where they aren't strictly needed, and this is causing EF Core to get confused when interpreting query results. The fault partly lies with EF Core, but I figure a change here will be easier to accomplish than a change to EF Core.
I believe the culprit to be this spot: https://github.com/StefH/System.Linq.Dynamic.Core/blob/master/src/System.Linq.Dynamic.Core/ExpressionParser.cs#L1464
When there is a literal null as one of the two operands in an equality operator, I'm guessing that it is assumed to have type "System.Object", and so the left-hand side is being explicitly converted to System.Object. Would it be possible to add handling here to check for literal null and skip the emission of a conversion in such cases? Or am I way off base here on what is causing this cast to be created?
Thanks for taking a look, and thanks for maintaining this library!

It's indeed related to the NullLiteral, but the issue is found here:
https://github.com/StefH/System.Linq.Dynamic.Core/blob/master/src/System.Linq.Dynamic.Core/ExpressionParser.cs#L654
I need to add an extra check here for NullLiteral.

New NuGet added.

Thank you so much!


CodeQL docs: update titles, add intros, and a few content updates

The PR makes a start at addressing the tasks in https://github.com/github/product-documentation/issues/2163.
tldr for any members of the C analysis team who stumble upon this: We are trying to align the CodeQL documentation with the style and structure used in the GitHub help docs. There will be a series of PRs with updates. We'll try not to do anything that could change the technical meaning of the text and may ping you with questions as we are going along.
Summary of changes

Update titles
Update links to account for new titles
Add intros
Rework the CodeQL for C and C++ homepage to include only the autogenerated TOC.
Some small changes to content:

Attempt to make the flow from the new intro to the main article a bit smoother
Fix a few inline links

TODO
Possibly next PR:

Rework further reading sections, and save in a reusable snippet as suggested by @felicitymay in #2823. I'll wait until that PR is merged to avoid conflicts.
More thorough review for localization issues.

Later PRs:

Think long and hard about the most appropriate content models to apply.
Review content, adding referential, conceptual, and procedural sections in certain topics.

For more info, see notes on https://github.com/github/product-documentation/issues/2163.

@shati-patel, @felicitymay@: thanks for the comments and suggestions. Is there anything else you think I should address now? (I intend to revisit this when all of the other initial 'CodeQL for X'  PRs have been completed, as mentioned on #2866 and #2823.)

Thanks for the updates 👍

I removed the WIP label now. Any further thoughts before i merge?

I think this is fine to merge as-is.


FR: allow type:spellcasting in @Checks

Since per-tradition proficiency is gone, allowing spellcasting as an alias for 'first tradition found' would lead to potentially fewer redundant @Checks

I've been informed @Check[type:spell] is a thing and sometimes I should just type less.


C\task\RemoveElement.c

Complete the task and attach a screenshot of your code and output. You may take the input yourself according to the task.

Can you please assign this issue?

can you assign me this issue?


Initial support for nimble lock files

Add parallel ci-lock.yml
In order to sync the lock file do the following:

nimble lock --developFile=nimble-update-lock.develop

Note that --developFile support for lock is part of  this PR https://github.com/nim-lang/nimble/pull/1125 to be merged. If it is not merged, then you can rename nimble-update-lock.develop to nimble.develop and nimble lock will work just fine.

I have used different nim version for the lock file because current nim does not have the proper nimble structure. I can bump the version in NBS if you want. I have also changed the versions of some of the packages in vendor folder to pass nimble version checks

Looks good. There will be some conflicts with my PR to update the deps, but I can handle merging the conflicts. That should resolve the build errors you're seeing.

@yyoncho I still have this in my work queue. It's just been bumped as I needed to work on some other things first.

@dryajov mind reviewing this? It'd be nice to get Nimble running aside NBS.

I'm killing this as we've sort of embraced nimbus-build-system, so our lock files are the commit versions in the /vendor folder. Until package maintainers do proper versioning (so that the lockfile reflects one specific solution to dep constraints), I also don't see much of a win in using lockfiles as they are more complicated to deal with than submodules.

Keep in mind that the plan is still to migrate to nimble and lockfiles, once we migrate to nim 2.0


[5.4]Eloquent Mysql Foreign key Failure increments integer pair errno 150

Laravel Version: 5.4.17
PHP Version: 5.6
Database Driver & Version:
MYSQL -> 5.5.42

Description:
MYSQL will fail to pair columns to create foreign keys that are not of the same type and defined length. The failure occurs when pairing increments which has a primary key and index does not pair with the same column type without an index already on that column in that table.
ie. table X has no FK on column post_id int=(11) and posts has id with a primary key. increments = int(10).
Eloquent creates a migration with int(10) on increments on primary keys but when integer is used it is int(11). This throws a mysql error.
If the same type and length of int is used making a FK will throw an error because there is no index for on column post_id as an example. Also i dont want to have to create indexes just so i can create a FK. It should be done in one command.
Steps To Reproduce:
`Schema::create('questions', function ( Blueprint $table ) {
       $table->increments('display_order_id');
            $table->integer('provider_id');
            $table->integer('next_question');
            $table->integer('question_id');
            $table->integer('display_order');
    });`

Migrate with this code in a new migration file and you will see on any MYSQL GUI manager the lengths or bytes are wrong, they dont match.
See example from link from my database.
http://imgur.com/a/kwG8B

Default for integer is 11, default for unsigned integer is 10. You need to use unsignedInteger to create a foreign key index.
        Schema::create('test', function (Blueprint $table) {
            $table->increments('id');
            $table->unsignedInteger('user_id');

            $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');;
        });


feat: I18n (Internalization)

Japanese Example:

Spotted an issue when hovering over the GET button here. Reverted styling changes because they were a bit over my head, but we'll reintroduce and fix a few more styling in a patch tomorrow/next week


node-sass silently fails when importing files

First got this issue when my Gulp Sass tasks suddenly stopped finishing. I tried to compile my Sass with node-sass commandline function but it outputs nothing.
I later isolated the issue to an import for Susy. The file which had the @import line, is located in:
config/structure/_susy.scss

Relative to the main style.scss which imports everything in my project.
Interestingly however, the Susy NPM package wasn't installed in my system. When I changed the @import to @import 'sosy', it errors. When I move it to another file, it errors.
Then I think: maybe the name of the file _susy.scss is the problem?
Sure enough, when I renamed it from _susy.scss to _susyconf.scss and re-ran it, everything works!
However it took me quite a while to find a problem. This should've outputted as an error, or basically not silently failing while I'm working.

Can you try to reproduce the problem using plain node-sass without using gulp? Also "it errors" is not enough information to diagnose the problem. The information in https://github.com/sass/node-sass/blob/master/TROUBLESHOOTING.md might help to diagnose the problem.

Sorry if I seemed unclear.
I did use node-sass commandline after troubleshooting gulp. And the problem I had is not that it errors, but that it doesn't error.

I had already solved my problem, but for those who would inevitably name their Susy config partials _susy.scss and import Susy in that file, it would be very irritating to see no feedback.

Here's the node-sass -v output if it's necessary:
node-sass       3.4.2   (Wrapper)       [JavaScript]
libsass         3.3.2   (Sass Compiler) [C/C++]

I am sorry,
I still cannot get how to work on this problem. Can you publish a short, self-contained example how to reproduce the problem with node-sass?

I have tried this:
=== start.scss===
@import 'susy';

=== _susy.scss ===
@import 'susy';

it seems to core dump with after a while with "invalid instruction", the backtrace suggests that libsass loops endlessly including the file.

(confirmed with pure sassc).

I have filed https://github.com/sass/libsass/issues/1801 for this.

Thanks for digging into this @saper. We'll track this in LibSass.


Conditional independency tests

Can you provide an absolutely minimal example (smallest possible graph) that requires no biological explanation to illustrate this phenomena?


Add a Bitdeli Badge to README

Pull request made by @ashwin31 at https://bitdeli.com

Coverage remained the same at 33.76% when pulling d79da4f25154d97c2a372de92a95879d0651a09b on bitdeli-chef:master into 962b92e3a2382c7912ab7c8244873312b508eab6 on MicroPyramid:master.


BIGTOP-3297. Bump HBase to 1.5.0.

We need the patch of HBASE-19663 to fix javadoc compilation failure.
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.4:site (default-site) on project hbase: Error generating maven-javadoc-plugin:2.10.3:aggregate:
[ERROR] Exit code: 1 - warning: unknown enum constant When.MAYBE
[ERROR]   reason: class file for javax.annotation.meta.When not found
[ERROR] warning: unknown enum constant When.ALWAYS
[ERROR] warning: unknown enum constant When.UNKNOWN
[ERROR] javadoc: error - class file for javax.annotation.concurrent.Immutable not found
[ERROR]
[ERROR] Command line was: /usr/lib/jvm/java-1.8.0-openjdk-<IP_ADDRESS>.b09-0.el7_7.x86_64/jre/../bin/javadoc -J-Xmx2048m @options @packages

./docker-hadoop.sh -c 3 --smoke-tests passed on my local with the following configuration.
docker:
        memory_limit: "8g"
        image: "bigtop/puppet:trunk-centos-7"

#repo: "http://repos.bigtop.apache.org/releases/1.4.0/centos/7/$basearch"
distro: centos
components: [hdfs, yarn, mapreduce, hbase]
enable_local_repo: true
smoke_test_components: [hbase]

Phoenix 4.15.0-HBase-1.5 was successfully built based on the fix here. (Please refer to BIGTOP-3298.)

Looks great. +1.

+1
I'll push to master shortly. Thanks @evans-ye and @iwasakims for the patch and review.

Thanks, @evans-ye and @youngwookim. Closing this.


Unregister dynamic preferences models from django admin

I have created my own custom form page for updating my global preferences and have no need to view them within django's admin site and therefore would like to remove them from it to reduce clutter.
To unregister default django models such as Groups and Users, you can simply unregister them from within your admin.py file:
`from django.contrib.auth.models import User
admin.site.unregister(Group)`
Is there a way to unregister global preferences from the admin page? As I have not been able to find out how thus far.

Have you tried inspecting the source code?
https://github.com/agateblue/django-dynamic-preferences/blob/4dd668f1b60523c6205b1df653501b8b7ca9023b/dynamic_preferences/admin.py#L77


mime: FormatMediaType should quote control characters

What does 'go version' print?
go version devel +1afdecdd77c1 Sat Mar 29 17:10:25 2014 -0400 linux/amd64

What steps reproduce the problem?
http://play.golang.org/p/LcQhZpJkX2

1. Call mime.FormatMediaType with parameters containing an ASCII control character
2. See the return value

What happened?
It returns a formatted string containing a control character as is.

What should have happened instead?
It should quote a control character with \ or return the empty string. Here is the
definition of quoted-string (http://www.ietf.org/rfc/rfc2616.txt). A control character
can appear as quoted-pair but not as qdtext.

       quoted-string  = ( <"> *(qdtext | quoted-pair ) <"> )
       quoted-pair    = "\" CHAR
       CHAR           = <any US-ASCII character (octets 0 - 127)>
       qdtext         = <any TEXT except <">>
       TEXT           = <any OCTET except CTLs,
                        but including LWS>
       CTL            = <any US-ASCII control character
                        (octets 0 - 31) and DEL (127)>

(It's actually surprising to me that it allows even a NUL after a backslash, but it's
what the standard says. It may be a spec bug?)

Content-Type and Content-Transfer-Encoding are defined in RFC 2045:
     content := "Content-Type" ":" type "/" subtype
                *(";" parameter)
                ; Matching of media type and subtype
                ; is ALWAYS case-insensitive.
     parameter := attribute "=" value
     attribute := token
                  ; Matching of attributes
                  ; is ALWAYS case-insensitive.
     value := token / quoted-string
     token := 1*<any (US-ASCII) CHAR except SPACE, CTLs,
                 or tspecials>
     tspecials :=  "(" / ")" / "<" / ">" / "@" /
                   "," / ";" / ":" / "\" / <">
                   "/" / "[" / "]" / "?" / "="
                   ; Must be in quoted-string,
                   ; to use within parameter values

quoted-string as defined in RFC 2822:
NO-WS-CTL       =       %d1-8 /         ; US-ASCII control characters
                        %d11 /          ;  that do not include the
                        %d12 /          ;  carriage return, line feed,
                        %d14-31 /       ;  and white space characters
                        %d127
qtext           =       NO-WS-CTL /     ; Non white space controls
                        %d33 /          ; The rest of the US-ASCII
                        %d35-91 /       ;  characters not including "\"
                        %d93-126        ;  or the quote character
qcontent        =       qtext / quoted-pair
quoted-string   =       [CFWS]
                        DQUOTE *([FWS] qcontent) [FWS] DQUOTE
                        [CFWS]

RFC 5322 obsoletes RFC 2822 and defines it in different way:
   qtext           =   %d33 /             ; Printable US-ASCII
                       %d35-91 /          ;  characters not including
                       %d93-126 /         ;  "\" or the quote character
                       obs-qtext
   qcontent        =   qtext / quoted-pair
   quoted-string   =   [CFWS]
                       DQUOTE *([FWS] qcontent) [FWS] DQUOTE
                       [CFWS]
   quoted-pair     =   ("\" (VCHAR / WSP)) / obs-qp
; from RFC 5234
         VCHAR          =  %x21-7E
                                ; visible (printing) characters

so as I understand, it basically discourages any non-FWS control characters now.


Add common fields

Codecov Report
Base: 19.29% // Head: 19.23% // Decreases project coverage by -0.07% :warning:

Coverage data is based on head (07a6246) compared to base (0ff164f).
Patch coverage: 44.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #217      +/-   ##
==========================================
- Coverage   19.29%   19.23%   -0.07%
==========================================
  Files          18       18
  Lines        2990     3005      +15
==========================================
+ Hits          577      578       +1
- Misses       2413     2427      +14

Impacted Files
Coverage Δ

src/zeek/dce_rpc.rs
65.11% <33.33%> (-3.18%)
:arrow_down:

src/zeek/http.rs
69.87% <33.33%> (-1.73%)
:arrow_down:

src/zeek/kerberos.rs
68.83% <33.33%> (-1.84%)
:arrow_down:

src/zeek/ntlm.rs
67.92% <33.33%> (-2.67%)
:arrow_down:

src/zeek/rdp.rs
62.06% <33.33%> (-4.60%)
:arrow_down:

src/zeek/smtp.rs
67.34% <33.33%> (-2.87%)
:arrow_down:

src/zeek/ssh.rs
68.83% <33.33%> (-1.84%)
:arrow_down:

src/zeek/conn.rs
63.07% <100.00%> (ø)

src/zeek/dns.rs
62.72% <100.00%> (+0.34%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[KFNBC] Store controller status in .status.notebookController stanza

Feature description
Following K8s best practices, the status of the controller should be in the .status field instead of .spec.
Move the .spec.notebookControllerState stanza to .status.notebookController.
https://github.com/opendatahub-io/odh-dashboard/blob/fee96c0414d0deca2504ec3be230fc4f5c01b16c/manifests/base/odh-dashboard-crd.yaml#L84-L105

Updating .spec is bad form, so this is rated higher because we should look to be predictable with our resources so that it can be understood by all how to consume data.

Also switching it to a bug as this is not the intended use of K8s resources.


Overflow Int32 while parsing symbol code from page causes unexpected behavior.

Hi there.
I found a bug with parsing symbol codes at HtmlTokenizer.GetNumericCharacterReference method.
Firstly, I wasn't prepared for unhandled exception at HtmlDocument.Parse method so I logged exception at PortableExtensions.ConvertFromUtf32 which one told me that I was out of range of [0x000000; 0x00D800) || (0x00DFFF; 0x10FFFF] values.
I fix this by a little addings to condition at HtmlEntityService.IsInvalidNumber method. I send you pull request with this fix.
I send it to master, because I really want to fix this bug quickly and I don't want to use my fork instead of main AngleSharp repository.
Please, look it :)
But you should do something with overflow. There is a little example of unexpected behavior:

First: &#x80000000;
 This should overflow int32 to -2147483648 value. Throws exception, but raising CharacterReferenceInvalidNumber error after my fix.

First: &#x100000000;
 This should overflow int32 to 0 value. Raising CharacterReferenceInvalidCode here, but I don't know how correct is this.

First: &#x100000001;
 This should overflow int32 to 1 value. Raising CharacterReferenceInvalidRange here, but same, I don't know anything about how correct is this in current design.

First: &#x10000FE0F;
 And finally. This should overflow int32 to coffin symbol value (0xFE0F). But no raising of error here.`

Hello! @FlorianRappl can you tell me, in current design, what error should rise at method HtmlTokenizer.GetNumericCharacterReference when Int32 overflowing in PortableExtensions.ConvertFromUtf32 method?

I`ll try to make pull request for this bug then.


Add Her Name Is ???Lady by Tree Palm

Song Name
Her Name Is ???Lady
Artist Name
Tree Palm
Source
Youtube
Youtube Link
PkzVrZPgAKE
Direct File Link
No response
Song ID
818027
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: Any Level
Downloaded: Yes

nvm


List of test methods for TestCases

Is there some way how get in test results list of all test methods in TestCases?

What exactly do you have on mind?
Grouping the test methods under their test case the same way PHPUnit or JUnit plugins do? That is a tricky one as it needs the Tester's OutputHandler to provide information about the tests in a more structured manner; I'm planning to implement it as a part of #18, in one way or another

Exactly what i mean. So i guess you can close it. Maybe you can add some comment with planning feature, i will not spam with ideas which you have in head already. :)

No worries, feel free to keep spamming me with ideas, the amount of feedback I've got from you is tremendous and much appreciated!
For convenience, I've added the list directly to #18; other ideas and plans are already outlined in issues since my head can't keep them all 😉


Support L4 proxy CRDs in ExpressionRoutes mode

Is there an existing issue for this?

[X] I have searched the existing issues

Problem Statement
Kong gateway will implement L4 proxy support of expressions router: https://github.com/Kong/kong/pull/11071. When the Kong gateway part is finished, KIC should start to support the CRDs to specify L4 proxy configurations. The following CRDs are included:

TCPIngress
UDPIngress
TCPRoute
TLSRoute
UDPRoute

Proposed Solution
When ExpressionRoutes feature gate is enabled, and Kong gateway runs expressions router:

Translate the L4 proxy CRDs to expression based Kong routes if Kong gateway supports L4 proxy with expressions based router.
Otherwise, still disable translation of L4 proxy CRDs.

Additional information
No response
Acceptance Criteria

[ ] When Kong gateway supports L4 proxy in expression router, KIC translates the L4 proxy CRDs correctly

[ ] included CRDs: TCPIngres, UDPIngress, TCPRoute, UDPRoute, TLSRoute

[ ] when Kong gateway does not support them, still disable translation of the CRDs.

### Tasks
- [ ] https://github.com/Kong/kubernetes-ingress-controller/issues/4539
- [ ] https://github.com/Kong/kubernetes-ingress-controller/issues/4540
- [ ] https://github.com/Kong/kubernetes-ingress-controller/issues/4541
- [ ] https://github.com/Kong/kubernetes-ingress-controller/issues/4542

Blocked by https://github.com/Kong/kong/pull/11538 being released in Gateway 3.4.1

close this since ALL CRDs are supported now.


Unable to use RadioButton Control with ListView in Xamarin Forms

Added Experimental Flag in MainActivity.cs before Forms. Init for RadioButton but still getting runtime exception: System.InvalidOperationException: The class, property, or method you are attempting to use ('RadioButton') is part of RadioButton; to use it, you must opt-in by calling Forms.SetFlags("RadioButton_Experimental") before calling Forms.Init().

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

ID: a688fe91-33ad-efea-aa29-d56b3c11b42a
Version Independent ID: c0e81ed5-0eb5-a8cc-9eb6-7baf7e464878
Content: Xamarin.Forms RadioButton - Xamarin
Content Source: docs/xamarin-forms/user-interface/radiobutton.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @nketu1994
Are you calling Forms.SetFlags more than once (with different flags), or from more than one place (e.g. MainActivity and App.xaml.cs)?


OPEN AI API ERRORS

Never does anything, only those errors. Nothing about it on plugin page too(
I hope you can tell me what to do)

The same error happened when I change the max tokens from 1024 to 2048.


Add simple Nim implementation

A simplest implementation with minimal allocations.
Some may find my usage of parens excessive for Nim, this is a matter of preference.

Wow, there's already a Nim version. I'm so sorry, I needed to refresh the page before opening this PR.
However, my version is simpler while being effectively the same (although, I'm not sure about the speed effect of ordering of splitting and lowercasing).

Yeah, we just added a nim version. Yours does look simpler, which is what we're aiming for in simple.nim. Feel free to update this PR off the latest master to change to your simplified version.

Updated the PR

Updated the PR.

Note:
The current Nim version has some changes in comparison to this version:

Using a main() proc - the Nim compiler can sometimes optimise better when code is within a procedure than it can when in global scope
Explicit imports rather than implicit - this is purely a matter of preference
Explicit sorting when ordering - again, a matter of preference

Otherwise, the code is pretty similar which is nice to see - the Nim standard library is pretty complete and offers simple and obvious ways to accomplish tasks like this :)
It also looks like there are some unrelated changes in this PR with some Julia code too.

Yes, using the main() proc is probably the correct decision, even if it may not bring any benefits in this case. Explicit imports are also have become the recommended way since not so long ago, rightly so. Unrelated commits are the result of syncing the branch, I probably should have done something else.
I'm absolutely positive there weren't any open or closed PRs with Nim versions nor .nim files in master branch at the time of me making the fork, don't understand how I could have missed it. Feel free to close this PR.

Yeah, I think I'll close this PR in favor of the other one. If you or someone else wants to do another PR with an optimized.nim at a later date, I'll happily consider that. Thanks for your contribution!


Allow Tabs to have an auto width

Related #774
Currently, the width of Tab is hardcoded to be 10rem. I often find myself manually overriding the CSS so that tabs with longer text have an automatic width.
Because carbon-components currently does not support it, we can inline the rule using the style prop. In the flagship repo, Tabs will have an auto width in v11 https://github.com/carbon-design-system/carbon/issues/10031

Released in v0.48.0


Stuck Clover Boot Loader Window

Steps to reproduce:

Go to Boot tab
Select Clover and let Hackintool list an update
Select Update
When the Save dialogue prompt appears, select cancel

A secondary Clover update window emerges but hitting cancel does not dismiss it.

This should be fixed in the next release


Hover Card works very slowly

While using the hover card component, I noticed it's prolonged and mostly ambiguous for users. Users usually take it by mistake with button or link and click over it. It would be great if it shows the content as soon as we hover the icon.

The hover card component internally uses radixs hover card and exposes the props of that which support specifying openDelay and closeDelay.
So you should be able to do:
 <HoverCard openDelay={0}>
 ...
 </HoverCard>

Thanks


Issue when NEMO doesn't use land processors

Hi,
I had a quick look over the issues history and I didn't find anything, so I hope my question has not been already answered.
I want to use OceanParcels with NEMO fields.
NEMO offers the possibility to increase computational efficiency by removing processors where no ocean point is defined. This has the drawback of generating NaN values (hence defined only on some land points) on all variables of outputs files, including longitude and latitude.
I tried a short test and it looks like this is an issue for OceanParcels :
In [29]: pset.execute(kernels, runtime=delta(days=10), dt=delta(days=10),
    ...:                  recovery={ErrorCode.ErrorOutOfBounds: DeleteParticle},
    ...:                  output_file=pfile)
    ...:
INFO: Compiled JITParticleAdvectionRK4periodicBC ==> /tmp/parcels-1946/f6772e9ab29b2469ffdff9b6e8c35d01.so
xsi and or eta are nan values
xsi and or eta are nan values
xsi and or eta are nan values
xsi and or eta are nan values
xsi and or eta are nan values
xsi and or eta are nan values
xsi and or eta are nan values
xsi and or eta are nan values
xsi and or eta are nan values
xsi and or eta are nan values
Correct cell not found after 1000000 iterations
Correct cell not found after 1000000 iterations
Correct cell not found after 1000000 iterations
Correct cell not found after 1000000 iterations
Correct cell not found after 1000000 iterations
Correct cell not found after 1000000 iterations
Correct cell not found after 1000000 iterations
Correct cell not found after 1000000 iterations
Correct cell not found after 1000000 iterations
Correct cell not found after 1000000 iterations
WARNING: ParticleSet is empty on writing as array

Can anyone think of an easy solution here ?
Thanks in advance.
Thomas.

Thanks for reporting, Thomas. This is indeed something that will need to be addressed, as we want to support Parcels on NEMO.
We can have a look at ways to fix this issue of NaN lons and lats, but it would be easiest if we have a dataset to work with. Can you provide us with a link to an example dataset where this occurs?

This is a snippet I used to copy the coordinate fields from the coordinates.nc, that had the full fields, to the data files with NaN in the coordinates. It won't run out of the box for your setup, but you'll get the idea:
import netCDF4 as nc4

for tp, grid in zip(['grid_U', 'grid_V', 'grid_T', 'icemod'], ['u', 'v', 't', 't']):

    print(tp, grid)

    data_dir_in  = '/... /LPE/'
    data_dir_out = '/.../'
    file_name_pattern = 'ORCA05*_1m_????0101_????1231_{}.nc'.format(tp)
    file_names = sorted(glob.glob(os.path.join(data_dir_in, file_name_pattern)))

    lon_name = 'glam{}'.format(grid)
    lat_name = 'gphi{}'.format(grid)
    print(lon_name, lat_name)

    ds_coord = nc4.Dataset('/... /' +
                           'coordinates_ORCA_R05_iom.nc')
    lon = ds_coord.variables[lon_name]
    lat = ds_coord.variables[lat_name]

    for fn in file_names:

        fn_out = os.path.join(data_dir_out, os.path.basename(fn))
        print(os.path.basename(fn_out))

        #   copy to fn_out
        shutil.copy(fn, fn_out)
        # change permissions of owner --> o+rw
        os.chmod(fn_out, 0600)
        ds = nc4.Dataset(fn_out, mode='a')

        for var_name in ds.variables.keys():

            if 'nav_' in var_name:
                continue
            var = ds.variables[var_name]
            var[:] = np.ma.filled(np.ma.asarray(var[:]), 0.0)

        ds.variables['nav_lon'][:] = lon[:]
        ds.variables['nav_lat'][:] = lat[:]

        ds.close()

    ds_coord.close()

I think, the most sustainable way of solving this is to "repair" the LPE (labs processor elimination) data right after producing them as described above.
An alternative would be to let Parcels allow for coords from a different file.

Thanks for your answers.
The most efficient way to solve this issue is indeed to have NEMO outputs that directly include full coordinates. I think that this problem has been handled recently by NEMO developers, so this should not be an issue in next versions of NEMO.
However, there are plenty of existing simulations (and plenty new will be created) produced with this issue. I guess "repairing" files with this snippet is a fine solution too.
I was indeed more thinking in a solution where OceanParcels can read the coords from the mesh_mask (or coordinates) file, as it has already access to it.
@erikvansebille : do you still want a sample file to check at this more closely ?

Which function do you use to load the data?
If you use fieldset.from_nemo, as in the examples, you don't read lon/lat from the velocity files but from a coordinates file or mask file. And I thought lon/lat were not nan in this file, no?

From: thomasarsouze<EMAIL_ADDRESS>Sent: Thursday, August 09, 2018 12:30 PM
To: OceanParcels/parcels
Cc: Subscribed
Subject: Re: [OceanParcels/parcels] Issue when NEMO doesn't use land processors (#406)
Thanks for your answers.
The most efficient way to solve this issue is indeed to have NEMO outputs that directly include full coordinates. I think that this problem has been handled recently by NEMO developers, so this should not be an issue in next versions of NEMO.
However, there are plenty of existing simulations (and plenty new will be created) produced with this issue. I guess "repairing" files with this snippet is a fine solution too.
I was indeed more thinking in a solution where OceanParcels can read the coords from the mesh_mask (or coordinates) file, as it has already access to it.
@erikvansebillehttps://github.com/erikvansebille : do you still want a sample file to check at this more closely ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/OceanParcels/parcels/issues/406#issuecomment-411713239, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AdpxJ_-4uH6S8BQh-u5FjRAR5CjnPdULks5uPA9igaJpZM4V1Xy1.

I use fieldset.from_nemo but, if I put the name of coordinates in the mesh_mask (i.e. 'glamu','gphiu', etc...) it fails while if I put the name of coordinates in the U/V files (i.e. 'nav_lon','nav_lat') then it reads correctly.
So it looks like in my case it's reading coordinates from the files rather than from the meshmask.
def loadNEMOfile(filenameu, filenamev, config):
    if config=='ORCA1L75':
        filenamemesh = '.../mesh_mask_nemo.Ec3.2_O1L75.nc'
    filenames  = {'U': filenameu,
                  'V': filenamev,
                  'mesh_mask': filenamemesh }
    variables  = {'U': 'uo', 'V': 'vo'}
     dimensions = {'U': {'lon': 'glamu','lat':'gphiu', 'time': 'time_counter'},
                   'V': {'lon': 'glamv','lat':'gphiv', 'time': 'time_counter'}}

    fieldset = FieldSet.from_nemo(filenames, variables, dimensions, mesh='spherical', allow_time_extrapolation=True)

    return fieldset

 ncdump -h mesh_mask_nemo.Ec3.2_O1L75.nc
netcdf mesh_mask_nemo.Ec3.2_O1L75 {
dimensions:
	x = 362 ;
	y = 292 ;
	z = 75 ;
	t = UNLIMITED ; // (1 currently)
variables:
	float nav_lon(y, x) ;
	float nav_lat(y, x) ;
	float nav_lev(z) ;
	double time_counter(t) ;
...
	float glamt(t, y, x) ;
	float glamu(t, y, x) ;
	float glamv(t, y, x) ;
	float glamf(t, y, x) ;
	float gphit(t, y, x) ;
	float gphiu(t, y, x) ;
	float gphiv(t, y, x) ;
	float gphif(t, y, x) ;
...
// global attributes:
		:file_name = "mesh_mask.nc" ;
		:TimeStamp = "23/02/2016 15:55:35 +0100" ;
}

In [5]: fieldset = loadNEMOfile(pathu, pathv, config)
   ...:
INFO: file '_esarchive_releases_models_nemo_v3.6_ecearth_inidata_nemo_ORCA1L75_rotation_angles.nc' not generated since it is newer than '/esarchive/releases/models/nemo/v3.6_ecearth/inidata/nemo/ORCA1L75/mesh_mask_nemo.Ec3.2_O1L75.nc'.
      If you want to re-generate it, please remove existing file first.
WARNING: Casting lon data to np.float32
WARNING: Casting lat data to np.float32
WARNING: Casting time data to np.float64
WARNING: Casting depth data to np.float32
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-5-f22ae4ff6b61> in <module>()
----> 1 fieldset = loadNEMOfile(pathu, pathv, config)

<ipython-input-4-6f267246057a> in loadNEMOfile(filenameu, filenamev, config)
      8     dimensions = {'U': {'lon': 'glamu','lat':'gphiu', 'time': 'time_counter'},
      9                   'V': {'lon': 'glamv','lat':'gphiv', 'time': 'time_counter'}}
---> 10     fieldset = FieldSet.from_nemo(filenames, variables, dimensions, mesh='spherical', allow_time_extrapolation=True)
     11     return fieldset

/home/Earth/tarsouze/miniconda3/envs/parcels/lib/python2.7/site-packages/parcels/fieldset.pyc in from_nemo(cls, filenames, variables, dimensions, indices, mesh, allow_time_extrapolation, time_periodic, **kwargs)
    241
    242         return cls.from_netcdf(filenames, variables, dimensions, mesh=mesh, indices=indices, time_periodic=time_periodic,
--> 243                                allow_time_extrapolation=allow_time_extrapolation, **kwargs)
    244
    245     @classmethod

/home/Earth/tarsouze/miniconda3/envs/parcels/lib/python2.7/site-packages/parcels/fieldset.pyc in from_netcdf(cls, filenames, variables, dimensions, indices, mesh, allow_time_extrapolation, time_periodic, full_load, **kwargs)
    164             fields[var] = Field.from_netcdf(paths, var, dims, inds, mesh=mesh,
    165                                             allow_time_extrapolation=allow_time_extrapolation,
--> 166                                             time_periodic=time_periodic, full_load=full_load, **kwargs)
    167         u = fields.pop('U', None)
    168         v = fields.pop('V', None)

/home/Earth/tarsouze/miniconda3/envs/parcels/lib/python2.7/site-packages/parcels/field.pyc in from_netcdf(cls, filenames, variable, dimensions, indices, mesh, allow_time_extrapolation, time_periodic, full_load, **kwargs)
    267         if not isinstance(filenames, Iterable) or isinstance(filenames, str):
    268             filenames = [filenames]
--> 269         with NetcdfFileBuffer(filenames[0], dimensions, indices) as filebuffer:
    270             lon, lat = filebuffer.read_lonlat
    271             depth = filebuffer.read_depth

/home/Earth/tarsouze/miniconda3/envs/parcels/lib/python2.7/site-packages/parcels/field.pyc in __enter__(self)
   1039             self.dataset = xr.open_dataset(str(self.filename), decode_cf=False)
   1040             self.dataset['decoded'] = False
-> 1041         lon = getattr(self.dataset, self.dimensions['lon'])
   1042         lat = getattr(self.dataset, self.dimensions['lat'])
   1043         xdim = lon.size if len(lon.shape) == 1 else lon.shape[-1]

/home/Earth/tarsouze/miniconda3/envs/parcels/lib/python2.7/site-packages/xarray/core/common.pyc in __getattr__(self, name)
    174                     return source[name]
    175         raise AttributeError("%r object has no attribute %r" %
--> 176                              (type(self).__name__, name))
    177
    178     def __setattr__(self, name, value):

AttributeError: 'Dataset' object has no attribute 'glamu'

You don't have to use glamu and so on.
You have to use the glamf for both U and V.
Since the c-grid interpolator uses that grid to correctly compute the velocities.
The nemo example will help you.

From: thomasarsouze<EMAIL_ADDRESS>Sent: Thursday, August 09, 2018 12:56 PM
To: OceanParcels/parcels
Cc: Philippe Delandmeter; Comment
Subject: Re: [OceanParcels/parcels] Issue when NEMO doesn't use land processors (#406)
I use fieldset.from_nemo but, if I put the name of coordinates in the mesh_mask (i.e. 'glamu','gphiu', etc...) it fails while if I put the name of coordinates in the U/V files (i.e. 'nav_lon','nav_lat') then it reads correctly.
So it looks like in my case it's reading coordinates from the files rather than from the meshmask.
def loadNEMOfile(filenameu, filenamev, config):
if config=='ORCA1L75':
filenamemesh = '.../mesh_mask_nemo.Ec3.2_O1L75.nc'
filenames  = {'U': filenameu,
'V': filenamev,
'mesh_mask': filenamemesh }
variables  = {'U': 'uo', 'V': 'vo'}
dimensions = {'U': {'lon': 'glamu','lat':'gphiu', 'time': 'time_counter'},
'V': {'lon': 'glamv','lat':'gphiv', 'time': 'time_counter'}}
fieldset = FieldSet.from_nemo(filenames, variables, dimensions, mesh='spherical', allow_time_extrapolation=True)

return fieldset

ncdump -h mesh_mask_nemo.Ec3.2_O1L75.nc
netcdf mesh_mask_nemo.Ec3.2_O1L75 {
dimensions:
x = 362 ;
y = 292 ;
z = 75 ;
t = UNLIMITED ; // (1 currently)
variables:
float nav_lon(y, x) ;
float nav_lat(y, x) ;
float nav_lev(z) ;
double time_counter(t) ;
...
float glamt(t, y, x) ;
float glamu(t, y, x) ;
float glamv(t, y, x) ;
float glamf(t, y, x) ;
float gphit(t, y, x) ;
float gphiu(t, y, x) ;
float gphiv(t, y, x) ;
float gphif(t, y, x) ;
...
// global attributes:
:file_name = "mesh_mask.nc" ;
:TimeStamp = "23/02/2016 15:55:35 +0100" ;
}
In [5]: fieldset = loadNEMOfile(pathu, pathv, config)
...:
INFO: file '_esarchive_releases_models_nemo_v3.6_ecearth_inidata_nemo_ORCA1L75_rotation_angles.nc' not generated since it is newer than '/esarchive/releases/models/nemo/v3.6_ecearth/inidata/nemo/ORCA1L75/mesh_mask_nemo.Ec3.2_O1L75.nc'.
If you want to re-generate it, please remove existing file first.
WARNING: Casting lon data to np.float32
WARNING: Casting lat data to np.float32
WARNING: Casting time data to np.float64
WARNING: Casting depth data to np.float32
AttributeError                            Traceback (most recent call last)
 in ()
----> 1 fieldset = loadNEMOfile(pathu, pathv, config)
 in loadNEMOfile(filenameu, filenamev, config)
8     dimensions = {'U': {'lon': 'glamu','lat':'gphiu', 'time': 'time_counter'},
9                   'V': {'lon': 'glamv','lat':'gphiv', 'time': 'time_counter'}}
---> 10     fieldset = FieldSet.from_nemo(filenames, variables, dimensions, mesh='spherical', allow_time_extrapolation=True)
11     return fieldset
/home/Earth/tarsouze/miniconda3/envs/parcels/lib/python2.7/site-packages/parcels/fieldset.pyc in from_nemo(cls, filenames, variables, dimensions, indices, mesh, allow_time_extrapolation, time_periodic, **kwargs)
241
242         return cls.from_netcdf(filenames, variables, dimensions, mesh=mesh, indices=indices, time_periodic=time_periodic,
--> 243                                allow_time_extrapolation=allow_time_extrapolation, **kwargs)
244
245     @classmethod
/home/Earth/tarsouze/miniconda3/envs/parcels/lib/python2.7/site-packages/parcels/fieldset.pyc in from_netcdf(cls, filenames, variables, dimensions, indices, mesh, allow_time_extrapolation, time_periodic, full_load, **kwargs)
164             fields[var] = Field.from_netcdf(paths, var, dims, inds, mesh=mesh,
165                                             allow_time_extrapolation=allow_time_extrapolation,
--> 166                                             time_periodic=time_periodic, full_load=full_load, **kwargs)
167         u = fields.pop('U', None)
168         v = fields.pop('V', None)
/home/Earth/tarsouze/miniconda3/envs/parcels/lib/python2.7/site-packages/parcels/field.pyc in from_netcdf(cls, filenames, variable, dimensions, indices, mesh, allow_time_extrapolation, time_periodic, full_load, **kwargs)
267         if not isinstance(filenames, Iterable) or isinstance(filenames, str):
268             filenames = [filenames]
--> 269         with NetcdfFileBuffer(filenames[0], dimensions, indices) as filebuffer:
270             lon, lat = filebuffer.read_lonlat
271             depth = filebuffer.read_depth
/home/Earth/tarsouze/miniconda3/envs/parcels/lib/python2.7/site-packages/parcels/field.pyc in enter(self)
1039             self.dataset = xr.open_dataset(str(self.filename), decode_cf=False)
1040             self.dataset['decoded'] = False
-> 1041         lon = getattr(self.dataset, self.dimensions['lon'])
1042         lat = getattr(self.dataset, self.dimensions['lat'])
1043         xdim = lon.size if len(lon.shape) == 1 else lon.shape[-1]
/home/Earth/tarsouze/miniconda3/envs/parcels/lib/python2.7/site-packages/xarray/core/common.pyc in getattr(self, name)
174                     return source[name]
175         raise AttributeError("%r object has no attribute %r" %
--> 176                              (type(self).name, name))
177
178     def setattr(self, name, value):
AttributeError: 'Dataset' object has no attribute 'glamu'
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/OceanParcels/parcels/issues/406#issuecomment-411719050, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AdpxJ9aZ81i7XlI8xx4-iji93DplGNtCks5uPBVMgaJpZM4V1Xy1.

Great, let us know if it works for you from git, so that we can close this Issue afterwards.
And sorry that we are lagging behind with the conda releases, and that they may not be in sync with what's on oceanparcels.org!

Hi agin,
installed with pip 1.0.4.dev125+g5516597 and it seems to work fine.
I think the issue can be closed.
Thanks for your help.
Thomas.


Bump GitHub action workflow to latest version

This PR corrects broken image links in README.md. It also bumps the checkout action to its  latest version.

Thanks


Multiscreen/Multiwindow rendering

Reference: https://discourse.vvvv.org/t/link-topics-2-3d-engine/16544/3

This would be incredibly useful for me


Test-DbaEditionChange

Is this a feature OR bug:
feature post 1.0
Details
Turn this into a command: https://docs.microsoft.com/en-us/sql/database-engine/install-windows/supported-version-and-edition-upgrades
So that https://github.com/sqlcollaborative/dbatools/issues/1029 can use it.

wontfix


500 Internal Server Error

Hi @owen2345,
I am getting 500 Internal Server Error with latest changes if I try to upload images using S3.
I am getting this error with your latest commit on image cropping.
Completed 500 Internal Server Error in 256ms (ActiveRecord: 10.2ms)

NoMethodError (undefined method `call' for nil:NilClass
Did you mean?  caller):

/Users/mazhar/.rvm/gems/ruby-2.3.0@rails5.0/bundler/gems/camaleon-cms-d390617ddfea/app/uploaders/camaleon_cms_aws_uploader.rb:66:in `add_file'
/Users/mazhar/.rvm/gems/ruby-2.3.0@rails5.0/bundler/gems/camaleon-cms-d390617ddfea/app/helpers/camaleon_cms/uploader_helper.rb:61:in `upload_file'
/Users/mazhar/.rvm/gems/ruby-2.3.0@rails5.0/bundler/gems/camaleon-cms-d390617ddfea/app/controllers/camaleon_cms/admin/media_controller.rb:83:in `upload

Here is my application helper file.
def aws_s3_for_all_subdomains(args)
    args[:server] = 's3'
    args[:aws_settings] = {  region:     ENV["AWS_REGION"],
                             access_key: ENV["AWS_ACCESS_KEY_ID"],·
                             secret_key: ENV["AWS_SECRET_ACCESS_KEY"],·
                             bucket:     ENV["S3_BUCKET_NAME"],·
                             cloud_front: 'https://d1dtr73lu3mys4.cloudfront.net'}
··········
   args[:aws_settings]['inner_folder'] = "#{current_site.name}"
  end

Hi @mazharoddin
I tested in my local but I can not see the error uploading, except the edit/resize image "cors error".
Could you please try again?
Regards,

Sure @owen2345.. I will re-test. thanks.

Hi @owen2345,
I re-tested with different setup, I am getting same error. Thanks.

Please check again!

Hi @owen2345,
I am able to upload images using S3 now, but now I am unable to edit them, shall I open new issue for this.

@mazharoddin I had the same problem, but it was for my CORS configuration for S3 and Cloudfront: please try again and you will see an image and instructions to fix it.
Regards,

ohh ok ..thanks.


v1.6.0

Checklist

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

closes #40

@conda-forge-admin, please rerender

@maresb, one suggestion would be we could constrain python >=3.8 (which is different from upstream but consistent with the rest of conda-forge) and drop importlib-metadata entirely.  What do you think?  It's also fine as it is.

On second thought, I'm inclined to keep it as we have it here.  There may very well be folks still counting on python 3.7 support for some reason and there's no reason to drop it other than clean-up.


[WIP] Simplify F.dropout test

This PR aims at rewriting dropout function test.
rel: #6071

Thanks for the contribution.
#7798 and #7992 seem to be related :)

Sorry, as we have ceased to add new functionalities to chainer, we are now closing PRs that do not explicitly address bugs or severe issues.
Thanks for your contribution!