remote debug on android does not work straight away - it glitches and requires action to make it work.

Operating system or device - Godot version:
Windows 7, Godot 2.1 alpha, android 4.4.2 arm
Issue description (what happened, and what was expected):
Even tho you may deploy and run project over wifi adb on remote android device, when enabling option "deploy with remote debug" - The debug does not work. No Output from runing application is received.
This is glitch and can be easly fixed with work around trick, but the trick is not obvious, especially to someone who 100 times browsed all options to make sure everything is properly set.
(see "steps to reproduce" to find out where is glitch and what trick to do to make it work - or rather what should be fixed.)
Steps to reproduce:
1 - close godot editor (this step is REQUIRED)
2 - open godot editor (this step is REQUIRED)
3 - open or create any project which use print("test"), so you can see... that you will not see it.
4 - start your wifi adb - and then you will see in editor green android icon apear.
5 - check option "deploy with remote debug" (located in menu of last icon in run options tool tab)
6 - click green android icon and your project will run on connected android device.
7 - expect to see your print("test") in editors output window - but no... you will not see it because it not works.
8 - go to settings -> editor settings -> network, and check if the IP is your WIFI where android device is connected. Even if its right IP (!) change it to seomething else and then change back to your WIFI,
9 - now it works, no matter its the same IP that didnt worked while a go - it needed to be shaked up.
10 - It will work as long as you not close godot - next time you will have again "shake" network setting.

I have same problem.
it happens if PC has several IP, like vmware installed.

it happens if PC has several IP, like vmware installed.

yes it reveals itself if computer have more than one IP's (network interfaces)
after more researchings, it seems that problem is basicaly:
settings -> editor settings -> network -> debug_host is NOT SAVED after change.
so if you have 3 IPs for example <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS> and active is <IP_ADDRESS>.

you go to debug_host setting, change it to <IP_ADDRESS>, (now debug works as long as editor is opened)
then close editor and open it again (debug does not work)
When you go to debug_host setting you see its again <IP_ADDRESS> (so the change was not saved)


[M3] Add tokens

Need to add Test PyPi and PyPi tokens for ci-cd to fully work. Specifically: "The GitHub Secrets TEST_PYPI_API_TOKEN and PYPI_API_TOKEN were added to your project’s repository."

Created a new account, copied over the tokens and the ci-cd passes cleanly. closing.


Dynamically generate adaptive script when adding steps

Describe the Issue:
In the management console. For a service provider if we added authentication steps the adaptive auth script also got updated according to the number of steps. But in the development-portal this doesn't happen.
This will be very confusing for a user transitioning from the management console to the developer-portal.
Also if we write a script with two steps and we have added only 1 step shouldn't this be validated as well?
var onLoginRequest = function(context) {
    executeStep(0);
    executeStep(1);
};

Until the user edits the script, we can keep adding steps to the script.


Validator should check if the gecko object is specified in the manifest.json

steps to reproduce:

Get a chrome extension from the Google store. Make sure to pick an extension with the same API support between Chrome and Firefox.
Change the extension to .xpi without making any other changes
Submit the add-on to https://addons-dev.allizom.org/en-US/developers/addon/submit/2
After validation is successful, click Continue

expected behavior:
Validation fails and informs that gecko object is missing from the manifest.json
actual behavior:
Validation passes. After clicking Continue, the user is stuck on Step 2 and is then notified that at least one supported application needs to be specified.

Closed in favour of the issue in addons-linter.


Failed to exclude ports when adding and routing grating couplers

Hi!
@joamatab @sebastian-goeldi
It found that I couldn't exclude any ports using 'excluded_ports' parameter in 'gf.routing.add_fiber_array()' method.
For example, when I ran code below to exclude 'o1' ports in the routing, it raised an error "ValueError: ports1=4 and ports2=2 must be equal"
import gdsfactory as gf
c = gf.components.crossing()
cc = gf.routing.add_fiber_array(
    component=c,
    grating_coupler=gf.components.grating_coupler_elliptical_te,
    with_loopback=False,
    excluded_ports = [('o1')]
)
cc.plot()

Then it raises an error below

You help would be greatly apprecaited!

I just wondered if anyone had an idea how that could be fixed?

The error you are getting is because something is trying to route or place 4 ports on one side and 2 ports on the other. But I only know that it comes from the routing function or add fiber array function from gdsfactory and I am not familiar with those.

Hi Cheng,
thank you for reporting this
See fix
https://github.com/gdsfactory/gdsfactory/pull/3355
after the fix it works well


double metric

prometheus_tsdb_lowest_timestamp_seconds
prometheus_tsdb_lowest_timestamp (unit: ms??)
I think that we could delete the second one.

my understanding is that the second one is coming from TSDB whic is unit agnostic and could be "unregistered" by prometheus when the first one is registered.

Yes, that's what's happening. They're both meant to be present.

Should we unregister the first one?
Or maybe pass a parameter "unit" to TSDB so that it can expose directly the correct metric?
Otherwise I will close this.

No, what the tsdb exports is a black box to Prometheus.


Support any number of trailing zeroes when decoding dates.

This was causing sporadic issues when the server returns dates that are valid according to the ISO standard, but not according to DateTime.ParseExact.
Fixes #64.

LGTM

@hallucinogen It recently showed up again. We're not 100% sure why. Still better to have it than not.


[ENH] Refactor of BaseDistribution and descendants - generalised distribution param broadcasting in base class

Reference Issues/PRs

Copied from this PR from sktime https://github.com/sktime/sktime/pull/5176
What does this implement/fix? Explain your changes.

Refactors the _get_bc_params function in the BaseDistribution class.
Does your contribution introduce a new dependency? If yes, which one?

No.
What should a reviewer concentrate their feedback on?

Any other comments?

PR checklist

For all contributions

[ ] I've reviewed the project documentation on contributing
[ ] I've added myself to the list of contributors.
[ ] The PR title starts with either [ENH], [CI/CD], [MNT], [DOC], or [BUG] indicating whether
the PR topic is related to enhancement, CI/CD, maintenance, documentation, or a bug.

For code contributions

[ ] Unit tests have been added covering code functionality
[ ] Appropriate docstrings have been added (see documentation standards)
[ ] New public functionality has been added to the API Reference

Codecov Report

Merging #54 (ac061b9) into main (d1b693f) will decrease coverage by 0.21%.
The diff coverage is 55.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
- Coverage   71.19%   70.98%   -0.21%
==========================================
  Files         107      107
  Lines        4958     4950       -8
  Branches      887      886       -1
==========================================
- Hits         3530     3514      -16
- Misses       1195     1201       +6
- Partials      233      235       +2

Files Changed
Coverage Δ

...o/distributions/tests/test_base_default_methods.py
37.93% <0.00%> (ø)

skpro/distributions/base.py
63.27% <50.00%> (-0.82%)
:arrow_down:

skpro/distributions/laplace.py
100.00% <100.00%> (ø)

skpro/distributions/normal.py
100.00% <100.00%> (ø)

skpro/distributions/t.py
98.61% <100.00%> (-0.14%)
:arrow_down:


Wrong result when integrating (weighed) Chebyshev polynomials

I'm logging this here as this seems to be a mpmath problem. The polynomial
'f' defined below is of order 2, and so it does not contain the 3rd
Chebyshev polynomial of the first kind: taking the scalar product
(integrating with the weight function 1/sqrt(1-x**2)) should return 0.
Sympy gets this right symbolically, but when integrating using quadts
the result depends on the given precision. For high levels of precision
quadts returns +inf, which is completely off.

Some test code to illustrate this behaviour:

>>> from sympy.interactive import *
>>> from sympy import mpmath

>>> f = x**2 + x - 2

>>> weight = diff(asin(x), x)
>>> def integrand(n, func):
...     return weight * func * chebyshevt(n, x)

>>> mpmath.mp.dps = 100
>>> mpmath.quadts(lambda t : integrand(3, f).subs(x, t).evalf(), [-1, 1])
mpf('+inf')
>>> mpmath.mp.dps = 10
>>> mpmath.quadts(lambda t : integrand(3, f).subs(x, t).evalf(), [-1, 1])
mpf('-1.781695653964e-7')
>>> integrate(integrand(3, f), (x, -1, 1))
0

Original issue for #201: http://code.google.com/p/mpmath/issues/detail?id=161
Original author: https://code.google.com/u/104490253617518974981/

Just encountered this a couple of days ago, and want to add some things if somebody wants to tackle this in the future.
This problem of lost accuracy can occur for all functions with large features near the endpoints, usually integrable singularities like 1/sqrt(x), but strictly speaking it doesn't have to be a singularity. Integrating something like 1/x^0.99 from 0 to 1 would not produce any reasonable result in the current implementation.
I think the best approach is to directly use the interval abscissas/nodes calculated for the interval [0,1] instead of [-1,1], so at least at the left endpoint there won't be any loss of precision. For singularities at both ends the user could just handle it by splitting the interval and mirroring the right one.
Or similarly the integrand could be defined with two arguments (distance to nearest endpoint) like done in boost (https://www.boost.org/doc/libs/1_76_0/libs/math/doc/html/math_toolkit/double_exponential/de_tanh_sinh_2_arg.html).

An attempt: #603.


windows 8.1 services 10.0.17763.292

hi guys,
is there a solution to run rdpwrap on windows 8.1 pro, everything is green but not working
thanks

nothing :(

windows 8.1 version is 6.3.9600.19395 july 2019 updates, dll keep at 19318 version and it's supported by updates of @fre4kyC0de


Can I have a global cache option? Bundle cache not restored on different PRs, but working within the same PR

Issue:
If I create a new PR, the bundle cache is not restored from another branch, but if I push a new commit to the PR, the bundle gets restored in the new test. If then I create a new PR with that same new commit it looks for the same cache key but does not get restored
Diagnosis:
This is a general issue of GitHub Actions' cache, see the docs:
https://docs.github.com/en/actions/advanced-guides/caching-dependencies-to-speed-up-workflows#restrictions-for-accessing-a-cache
It seems to work based on whether the base branch (for a PR) or the default branch has a cache, it's not global per repository.
Solution:
Can I have a global cache option, I often have PRs that do not affect the cached ruby bundle, so it would be beneficial to have a global cache option

I think this is because cache access is scoped only to that branch or parent branches: https://docs.github.com/en/actions/advanced-guides/caching-dependencies-to-speed-up-workflows#matching-a-cache-key
I agree that having the option to restore a cache created on any branch would be great.

Would love to see this - it would pretty much half our run time for actions on new branches

@georgepalmer my solution so far has been to add another workflow that only caches ruby bundle on push commit to master/main and any staging branch (takes about 30s if nothing to be done, and saves 5+ minutes on new PRs)

@dinushkaherath @georgepalmer The idea behind branch scoping is to avoid cross-contamination, where a branch that hasn't undergone PR review is able to inject a cache that is subsequently consumed by say a main/release branch.
Unfortunately, yes it does restrict the power of caching. Let me know what your thoughts on this.
cc @N-Usha

@bishal-pdMSFT I think it'd be great if there was the ability to opt-in to cross-branch caching. For private repositories, cache poisoning is not a big concern.
If possible, it'd still be nice to be able to exclude Dependabot PRs from saving to a global cache key, as their source is untrusted.

@bishal-pdMSFT echoing the above - I think the current behaviour is a sensible default but having the option to use a global cache would make sense for those that know it's safe for their repos.
As a quick aside I imagine this would save GitHub a lot on bandwidth each year if it was widely used

workaround: use github pages to cache binary artifacts https://github.com/milahu/sccache-bin/issues/1

Yes, something like this can be done - may be even a repo/org level setting so that each workflow need not specify these options.
Having said that, this is not a trivial work and not urgent, hence will need to be added to our backlog to be triaged and planned. I will close this issue and move it to our backlog. I will come back once we have planned this item.

@bishal-pdMSFT any status update you could share publicly?


Glyphnames are not swapped

When freezing a stylistic set into the font, glyph a.alt is assigned the codepoint of LATIN SMALL LETTER A but isn't renamed to a.
This causes problems in certain apps, e.g. Adobe Premiere on Windows I'm told.

That's by design. At this point, I have no plans to swap the glyphnames. When multiple features are being frozen, it's not trivial to "swap" the glyphnames. But you're most welcome to submit a patch that does this. For now, I've just added a --zapnames option to remove glyph names from .ttf fonts. But for .otf, glyphnames are required, and implementing renaming is a bit tedious.

When multiple features are being frozen, it's not trivial to "swap" the glyphnames.

Then maybe a switch that swaps only on single feature freezing can be checked-in? It would be useful for the usual purposes of such a script.
Working on this script is not part of my appointments just yet – though someone else might be interested in this I suppose.


made the tag request go to the right endpoint

Tag calls will go to the right place no automatically

Looks good!


[NATS] Fix issue in streaming publish

Fixes https://github.com/ballerina-platform/ballerina-lang/issues/20278
Purpose

$title

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@               Coverage Diff                @@
##             release-stage   #20286   +/-   ##
================================================
  Coverage                 ?   15.21%
================================================
  Files                    ?       50
  Lines                    ?     1341
  Branches                 ?      208
================================================
  Hits                     ?      204
  Misses                   ?     1121
  Partials                 ?       16

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 a62b992...44f7198. Read the comment docs.

Blocked due to https://github.com/ballerina-platform/ballerina-lang/issues/20306


Add a task to validate files from the build

This is open for discussion an might help when validating the full build.
What do people think?
Also any input on the actual checking script is greatly appreciated

@marcjansen Did you see #14405?

I don't think this adds much benefit. The first fix I made for v7.2.1 would have passed your check here.

Yeah, fine for me.


Initialize KServe folder in serving-catalog

This PR adds the initial folder for KServe serving catalog. The exact folder structure within the top level folders within kserve folder are still to be determined.
cc @jjk-g @ahg-g @ArangoGutierrez

/lgtm


export and test BlockIndex

Codecov Report

Merging #16 into master will decrease coverage by 0.32%.
The diff coverage is 40%.

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
- Coverage    89.3%   88.97%   -0.33%
==========================================
  Files           6        6
  Lines         243      263      +20
==========================================
+ Hits          217      234      +17
- Misses         26       29       +3

Impacted Files
Coverage Δ

src/blockarray.jl
96.77% <ø> (+12.9%)
:arrow_up:

src/blocksizes.jl
91.11% <100%> (+0.2%)
:arrow_up:

src/blockindices.jl
66.66% <27.27%> (-33.34%)
:arrow_down:

src/pseudo_blockarray.jl
92.3% <50%> (-5.42%)
:arrow_down:

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 d3aa589...c094bc4. Read the comment docs.


I can't use vw in storybook

If you are reporting a bug or requesting support, start here:
Bug or support request summary
I use VW in my project, using the webpack postcssPxToViewport plugin. The plugin automatically calculates the PX in css as a VW unit at compile time, but I tried to use the same configuration in styorybook/webpack, it doesn't work and throws an exception. I don't know how to deal with it, help me thank you.
"react": "^16.8.6",
"@storybook/addon-actions": "^5.0.10",
"@storybook/addon-console": "^1.1.0",
"@storybook/addon-info": "^5.0.10",
"@storybook/addon-links": "^5.0.10",
"@storybook/addon-storysource": "^5.0.10",
"@storybook/addon-viewport": "^5.0.10",
"@storybook/addons": "^5.0.10",
"@storybook/react": "^5.0.10",
"@types/storybook__react": "^4.0.1",
.storybook/webpack.config.js

error

Sorry, I found my error. I didn't write test in the rule, which caused the compilation error.
You can close this issue. 😂🤣


nuget & generell updates

I've seen many pull requests and forks of this, will be there be any updates on nuget or generally?

When the repo owners decide to stamp a new version, the related nuget packages available will be updated.  Last one was 7 months ago.  We have a lot of large PR's in the pipeline here and getting them all to play nicely and for them to test them all with what they do seems to be taking a little bit of time.

In order to maintain the quality, a lot of work has to be put into the PRs. In addition, there are also announcements for the next Electron and .NET releases, these are also still awaited for a big leap.
Then there is family and work, which are now prioritized. A big thank you also to @danatcofo, who is so active here!


How to make Interactible.Views avoid each other?

I have this code based off the chat head example. I can drag the chat heads around freely, but I'd like them to avoid sitting on top of each other. is there a way to use the anti gravity points or something else to make them move out of each other's way? or a way to set x and y when dragging ends? thanks.
<View style={styles.container}>
        {[1,2,3,4,5].map(num => (
          <Interactable.View
          key={num}
          style={{position:'absolute'}}
          boundaries={{ left, right, top, bottom  }}
          frictionAreas={[{damping:0.5}]}
          initialPosition={{ x: 0, y: 0 }}>
          <View style={styles.head} />
        </Interactable.View>
        ))}

      </View>

Hi! Can you please share the code for creating the chat heads?
The current example repos are not accessible anymore.


Fix panel implementation on thank you page

What is the context of this PR?
The success panel implementation on the thank you page is currently incorrect and because of this is using the blue information panel for the success message. Like this below:

This PR updates the configuration for the panel to use the variant param instead of the previously used type param. After this change the panel should use the green success panel with a green tick icon. Like this below:

How to review
Test by opening the test_thank_you schema and navigating to the thank you page. Compare this branch against main and check that the correct panel is now being used.
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

Looks good so far, something really minor I noticed is when it loads it highlighted in yellow but if you click on anywhere on the screen, the highlighting disappears. However, I'm not sure if this highlighting is supposed to happen on this component?

Looks good so far, something really minor I noticed is when it loads the thank you page, the box it highlighted in yellow but if you click anywhere else on the screen, the highlighting disappears. However, I'm not sure if this highlighting is supposed to happen on this component?

This is a feature of the success and error panels. They automatically get focus on page load for accessibility reasons because they are treated as an alert as they have important messages in them. The yellow box is to show that the panel has focus at that moment in time.

Looks good so far, something really minor I noticed is when it loads the thank you page, the box it highlighted in yellow but if you click anywhere else on the screen, the highlighting disappears. However, I'm not sure if this highlighting is supposed to happen on this component?

This is a feature of the success and error panels. They automatically get focus on page load for accessibility reasons because they are treated as an alert as they have important messages in them. The yellow box is to show that the panel has focus at that moment in time. It didn't have this before because we were using an information panel which doesn't have this feature.


Filepath copy

copying the file path to clipboard only gets the path to the directory and not the file selected
if no file or multiple files are selected it should copy the directory file path
if one file is selected it should copy the entire files path

I will add an additional button to copy path to clipboard.
/afm01/Q0/Q0703/Nikon_Example_Files/Decon_output_3/60x_1_decon.40_1x_RGB_Z008.tif

added at commit 64ebe6bfed9e181ebb104baa081a4c7daa0ef91f


WPA3 Security mode support in Documentation and Device tester IDT userdata_schema.json file

Is your feature request related to a problem? Please describe.
Yes. WPA3 security mode in WiFi is added as part of PR : https://github.com/aws/amazon-freertos/pull/2380.
This support needs update in userdata_schema.json file in devicetester package for IDT test runs .
Also documentation needs to updated : https://docs.aws.amazon.com/freertos/latest/userguide/freertos-wifi.html
Error with WPA3 security mode using IDT
The document is not compatible with schema.
see errors :

clientWifiConfig: Does not match a valid configuration format
clientWifiConfig.wifiSecurityType: clientWifiConfig.wifiSecurityType must be one of the following: "eWiFiSecurityWPA", "eWiFiSecurityWEP", "eWiFiSecurityWPA2"
testWifiConfig: Does not match a valid configuration format
testWifiConfig.wifiSecurityType: testWifiConfig.wifiSecurityType must be one of the following: "eWiFiSecurityWPA", "eWiFiSecurityWEP", "eWiFiSecurityWPA2"

Describe the solution you would like.
Below change in userdata_schema.json
eWiFiSecurityWPA3 in clientWifiConfig ==> wifiSecurityType
eWiFiSecurityWPA3 in testWifiConfig ==> wifiSecurityType
Additional context
Add any other information (including rationale, requirements, or references) about the feature request here.
Thank you!

Thanks for reporting the issue, IDT team is planning to release this feature together with next FreeRTOS release, which is planned in October.


Typo on the website

Is your feature request related to a problem? Please describe.
Just a simple typo on the website. Currently on the main site it states "Blink is unavailable some countries' app stores, including the US"
Describe the solution you'd like
Add a word to the website to correct the grammar. It should read "Blink is unavailable in some countries' app stores, including the US"
Additional context
Love Blink! Sending you some good vibes. Give Noor a hug. Cheers,

Thanks @themuzzman!! Fixed (may need to clear cache for it to show up) https://www.blink.sv/


DRILL-6118: Fixed tests due to changes that replaced WILDCARD with DY…

…NAMIC_STAR in some cases
Need the following PR to be merged into the Drill master before this:
https://github.com/apache/drill/pull/1104

@agozhiy, can you please update once the PR is merged. We can run through tests and merge it then.

DRILL-6118 has been committed to apache/drill master. Will have this merged soon (after the test run is done).

Runs are having some issue with env. Merging this in, as there are ~16 failures observed and this will fix the baseline.


pulled platform connector script out of workflow

Description
Removed the platform connector task from all DAGs, since the remote destination(thanos) has been decommissioned already.
Fixes

Codecov Report

Merging #307 (520a92f) into master (8520ee6) will increase coverage by 0.07%.
The diff coverage is 68.75%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master     #307      +/-   ##
==========================================
+ Coverage   94.52%   94.60%   +0.07%
==========================================
  Files          22       21       -1
  Lines        1023      982      -41
==========================================
- Hits          967      929      -38
+ Misses         56       53       -3

Flag
Coverage Δ

gha
94.60% <68.75%> (+0.07%)
:arrow_up:

python-3.9
94.60% <68.75%> (+0.07%)
:arrow_up:

unit
94.60% <68.75%> (+0.07%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

dags/openshift_nightlies/dag.py
93.22% <68.75%> (+0.64%)
:arrow_up:

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


Fixing query.py

There was a bug in query.py https://github.com/taspinar/twitterscraper/issues/296 that cause query_tweets() to fetch 0 tweets. The solution was in one of the answers in the threads (change of HEADERS value in query.py). I though someone should apply this to the actual repo as this package is actually amazing and such a minor problem takes away its usefulness

Thanks, this appears to work!

cmd>twitterscraper jack -l 50 --user -o jack.json
still return me this errors, why... need help please
INFO: Retrying... (Attempts left: 50)
INFO: Scraping tweets from https://twitter.com/Jack
INFO: Using proxy <IP_ADDRESS>:56874
INFO: Retrying... (Attempts left: 49)

I've already replace the Header in query.py.

@Frickson please post in this thread
https://github.com/taspinar/twitterscraper/issues/296
Would be good to get your twitterscraper version, python version, full traceback.


Add Documentation for Wi-Fi Direct

Created a support page for Wi-Fi Direct broadcasting.

WHat is Wifi direct? Can you please explain what is it and how it is related to physical web?


'visual sequence inspection evidence' review

'visual sequence inspection evidence' (ECO:0001828) is a child of 'inference from background scientific knowledge used in manual assertion'.
My proposal is to created  move this term to be a direct child of "inference from background scientific knowledge" and rename it to "inference from sequence inspection".
Additionally, all the "curator inference" terms have "used in automatic assertion" children. I'm not sure that this makes sense for curator inference terms. I think that they should all actually be manual assertion terms, since it's the curator making the inference.
To achieve this, we would obsolete all the "automatic" terms and merge the manual terms into the parent terms. 'curator inference' could get the axiom "used_in only 'manual assertion'" to say that it cannot be used in an automatic assertion.
By contrast, all "evidence based on computational logical inference" (brought up in #240) could be automatic assertion terms.
I would be happy to create a prototype branch to implement this so that we can review, but I'd like to hear others' thoughts first.

The definition for this term is wonky.  It discusses "used in manual assertion" yet it is not a typical "used in manual assertion" term.
I think 'inference from sequence inspection' is not good given the very narrow definition which is to inspection of only the promoter region.
'inference from sequence inspection' is a good term but could potentially cover a lot of different things: putential putative binding sites, alignment quality/issues, motif conservation, etc.
I agree with your point regarding automatically generated "used in automatic assertion" children - they are senseless in the context of 'curator inference'
James

From: Becky Jackson<EMAIL_ADDRESS>Sent: Sunday, November 3, 2019 10:59 AM
To: evidenceontology/evidenceontology<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [evidenceontology/evidenceontology] 'visual sequence inspection evidence' review (#242)
'visual sequence inspection evidence' (ECO:0001828) is a child of 'inference from background scientific knowledge used in manual assertion'.
My proposal is to created move this term to be a direct child of "inference from background scientific knowledge" and rename it to "inference from sequence inspection".
Additionally, all the "curator inference" terms have "used in automatic assertion" children. I'm not sure that this makes sense for curator inference terms. I think that they should all actually be manual assertion terms, since it's the curator making the inference.
To achieve this, we would obsolete all the "automatic" terms and merge the manual terms into the parent terms. 'curator inference' could get the axiom "used_in only 'manual assertion'" to say that it cannot be used in an automatic assertion.
By contrast, all "evidence based on computational logical inference" (brought up in #240https://github.com/evidenceontology/evidenceontology/issues/240) could be automatic assertion terms.
I would be happy to create a prototype branch to implement this so that we can review, but I'd like to hear others' thoughts first.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/evidenceontology/evidenceontology/issues/242?email_source=notifications&email_token=AFCUQPTMJXDNYYDGQV452SDQR3RM7A5CNFSM4JIL4RRKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HWOP2QA, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFCUQPUI3IVOY7AVOUO5LQDQR3RM7ANCNFSM4JIL4RRA.

Would it make sense to make a new "inference from sequence inspection" class under curator inference, and then move this term to be a child of that?
We could rename it "putative binding site identification by sequence inspection", or something along those lines.
I don't like having a single child under a parent class, but it leaves room for more sequence inspection terms in the future.

Based on discussion in #111, it looks like we should keep the "raw" evidence codes and make "used in manual assertion" children for these terms. IMO, we should just not make automatic evidence children for the "curator inference" terms.
I might be wrong though, is there a case where a curator inference can be automatically asserted? I would think that would be considered a computational inference.

This term was not intended to be curator inference, but author inference. The idea was that sometimes authors will identify a promoter (or some other type of sequence feature) based on the background knowledge. It was intended to be the "human" equivalent of ECO:0000028 and its children, but it could conceivably be thought of closer to ECO:0000034.

Thanks for the clarification @ivanerill
In that case, does it make sense to do:
- author inference
    - author inference from sequence inspection (NEW)
        - putative binding site identification by sequence inspection (ECO:0001828)

Hi @beckyjackson ,
Yes, although I don't know if there is a need to make it specific to "binding site". It could be "sequence feature" identification, or just leave it with the new proposed parent term.
Ivan

I haven't thought much about this yet, so just commenting after reading this thread - I'm worried about having this term specific for authors since I think there are people making primary annotations based on their own inspection of the sequences - they are not "authors" per se, I guess they are "curators". We did this all the time in our manual genome curation days. Do we need separate terms for curators and authors? Should we just have one term for a human doing this type of thing?

Hi @mgiglio99 ,
I don't know. In one case, an author is making a statement based on their prior knowledge and likely testing or otherwise leveraging that inference within a paper. In the other case, a curator is making an interpretation on an author's paper that the author themselves have not made. I agree both are human inferences, but I don't know if differential weights would be applied to them if somebody were to mine GOAs using these terms.
GO evidence codes have historically differentiated between "Author statement" and "Curator statement", but it might be time to reassess this.
Ivan

The case I'm thinking of is more along the lines of the person doing the analysis and annotations all themselves. So if I sequence a genome and then do an analysis where I find a binding site upstream of a set of genes and I make that annotation - I think I might use 'motif discovery evidence' ECO:0005531 or even just 'multiple sequence alignment'  ECO:0005555.
So, now we want a term to capture when an author has done this. I wonder about getting too granular under the author inference node, we might end up duplicating things and creating confusion.
Same is true of things under curator inference. If the curator is making this call themselves, then it is their annotation and I'm not sure it should be under a curator node.
Not sure if I'm explaining myself clearly here or not...

Correct. This is something we discussed with @mchibucos a while ago. When you are doing the inference, you may very well know that you are using a specific tool ('multiple sequence alignment' ECO:0005555), but here we were talking about an author eyeballing it (presumably based on their own knowledge of the system). I don't know how frequent this is these days, but it has certainly been common practice among people working on transcriptional regulation to just state that there is a XXX-binding site upstream of a gene, referring you to a figure containing the sequence with the relevant elements boldfaced or underlined.

Another question to bring up is the difference between 'author inference' and 'author statement'. I think we made 'author inference' not too long ago to distinguish between curators and authors, but it seems more accurate to say 'author statement'.

Suggestion: merge author inference into author statement.
Move author statement from published clinical study under author statement supported by traceable reference (unrelated).
For the 'visual sequence inspection evidence' class:
- inferential evidence (ECO:0000361)
    - inference from visual sequence inspection (ECO:0001828)
    - curator inference (ECO:0000205)
    - computational inferential evidence (ECO:0000362)

The inference from visual sequence inspection class could be used for both author inferences and curator inferences. This will be a general class, not specific to promoter regions, so we will broaden the definition. If somebody needs the more specific term in the future, we can add it.
This should always be a manual assertion. We are discussing the location of 'computational inferential evidence' in #240.
@ivanerill - if this looks OK to you, we will go ahead with this.

I personally would not merge author and curator inferences. I believe they are qualitatively different for a reasoner. One is a statement made by the author, the other an inference made by the curator based on their interpretation of the author's results/statements.
Other than that, having inference from visual sequence inspection (ECO:0001828) within inferential evidence (ECO:0000361) sounds good.

This will be a general class, not specific to promoter regions, so we will broaden the definition.

Depending on how it is defined, it risks being too generic to be useful. I would at least try to capture its original intended sense (inference of sequence features from visual inspection) in the parent or a derived term.

I personally would not merge author and curator inferences.

We didn't want to merge the author and curator inference classes. Our plan was to merge 'author inference' into 'author statement', but keep 'curator inference' distinct under 'inferential evidence'. Then, the visual sequence class would be directly under 'inferential evidence' so that it could be used for both curators and authors. Do you think it would be more correct to have two separate visual sequence inspection classes? One for authors, and one for curators?

Depending on how it is defined, it risks being too generic to be useful. I would at least try to capture its original intended sense (inference of sequence features from visual inspection) in the parent or a derived term.

Label: inference of sequence features from visual inspection
Broad synonyms: inference from visual sequence inspection, visual sequence inspection evidence
Definition: A type of inferential evidence in which sequence features are inferred based on visual inspection of the sequence.

We didn't want to merge the author and curator inference classes. Our plan was to merge 'author inference' into 'author statement', but keep 'curator inference' distinct under 'inferential evidence'. Then, the visual sequence class would be directly under 'inferential evidence' so that it could be used for both curators and authors. Do you think it would be more correct to have two separate visual sequence inspection classes? One for authors, and one for curators?

OK. Misunderstood. Certainly, no problem with this being used from both ends.

Label: inference of sequence features from visual inspection
Broad synonyms: inference from visual sequence inspection, visual sequence inspection evidence
Definition: A type of inferential evidence in which sequence features are inferred based on visual inspection of the sequence.

Sounds good.

There hasn't been any more discussion on this issue, so I'm going to go ahead and implement this.

Resolved by abad913


Added Flask-Minify

This PR adds Flask-Minify. I had to refactor something in map.js as well, because minifyier had problems getting it right,

This fixes one proposal from #93.

Reverted since it breaks "Generate style JSON" deploy step

that step will be removed once we change the file to json which it should have been from the beginning


Unable to return a TimeSpan property value

Unable to return a TimeSpan property value for a Duration value of 'Automatic'. Check the HasTimeSpan property before requesting the TimeSpan property value from a Duration
I can't seem to replicate it reliably.  I have a window where I am loading and unloading media players fairly rapidly and it seems to randomly occur.
Stack Trace:
"   at System.Windows.Duration.get_TimeSpan()\r\n   at Gu.Wpf.Media.MediaElementWrapper.<.ctor>b__3_4(Object o, RoutedEventArgs e)\r\n   at System.Windows.EventRoute.InvokeHandlersImpl(Object source, RoutedEventArgs args, Boolean reRaised)\r\n   at System.Windows.UIElement.RaiseEventImpl(DependencyObject sender, RoutedEventArgs args)\r\n   at System.Windows.Controls.AVElementHelper.OnMediaOpened(Object sender, EventArgs args)\r\n   at System.Windows.Media.UniqueEventHelper.InvokeEvents(Object sender, EventArgs args)\r\n   at System.Windows.Media.UniqueEventHelper.InvokeEvents(Object sender, EventArgs args)\r\n   at System.Windows.Media.MediaEventsHelper.OnMediaPrerolled(Object o)\r\n   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)\r\n   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)"

Thanks for reporting, looks like I dumbed something.

happened again
StackTrace = "   at System.Windows.Duration.get_TimeSpan()\r\n   at Gu.Wpf.Media.MediaElementWrapper.<.ctor>b__3_4(Object o, RoutedEventArgs e)\r\n   at System.Windows.EventRoute.InvokeHandlersImpl(Object source, RoutedEventArgs args, Boolean reRaised)\r\n   at System.W...

@julesx if you have time you can try cloning and reference a debug build to find out where the dumb is, I'm a bit lost about writing a testcase/fix for this issue.

This error occurs whenever MediaElement hasn't buffered/loaded the media, MediaElement doesn't exactly work well, the issue is probably MediaElement's fault, especially as you said that you were unloading media players rapidly - the media probably hadn't loaded depending on the operation.


[NumPy] Updates for §Indexing

Updates to the section on Indexing, including ellipsis (a pre-requisite for the Iris course) and the concept of slicing. I've also slimmed one of the code examples and updated the section summary.

@pp-mo you made some similar suggestions for this section, would you mind taking a look?

Rebase to fix merge conflicts.

Thanks @pp-mo 👍


fix the problem that files and wiki link doesn't show up on search

Purpose
Fix the problem that files and wiki link doesn't show up on search. Closes https://github.com/CenterForOpenScience/osf.io/issues/3827.

I'm trying to understand the issue being addressed and reproduce locally; which projects in search did not show the links?

@jlcohoon has some noted in #3827

Yes, but the issue didn't specify how to reproduce the bug or why those projects didn't show the links.
I clarified this in person with Nan: projects with no tags were not showing the Wiki and Files links. This PR fixes a logical error in displaying those.

PR looks good.


Add support for upserting different sources with the same name

Problem
For the un_sdg dataset we need sources with different descriptions, but same name. For instance
Source(name='World Bank', description='Further information available at: https://unstats.un.org/sdgs/metadata/files/Metadata-17-03-02.pdf', ...)
Source(name='World Bank', description='Further information available at: https://unstats.un.org/sdgs/metadata/files/Metadata-16-01-01.pdf', ...)

The problem is that we are upserting sources by their name, so if there are two sources with the same name, they'll be overwritten instead of creating two different sources.
Solution

We could have short_name for source working as a unique identifier to do upserts with
They've had the same issue in importers and fixed it by upserting only if all attributes match

The second option looks good enough. Sources are not linked to charts like variables, so creating new ones and deleting old ones (like we already do) is no big deal.

@larsyencken would you mind triaging this? Fiona said it's not a blocker, but she has a PR that depends on this.

@larsyencken so the situation has changed again. Fiona had a great idea to move that info to variable description and use single source, which means this has much lower priority now.

Proposal from chatting to @danyx23 is that:

sources.datasetId should not be NULLABLE, i.e. it should always be required, and each source is "owned" by a dataset
(sources.datasetId, sources.name) should be unique together, so a source name is unique per dataset

That means we are actually doubling down on this constraint that Fiona ran into. In her situation, we would want to either pick different names (e.g. World Bank (XYZ devision)) or to put the variation per variable into the variable metadata, like she did.

We'll make a new issue for it.


vcf2db error messages

Hi Brad,
When creating a db vcfanno reports the following:
[2017-05-22T07:21Z] skipping 'af_adj_exac_afr' because it has Number=A
[2017-05-22T07:21Z] skipping 'af_adj_exac_amr' because it has Number=A
[2017-05-22T07:21Z] skipping 'af_adj_exac_eas' because it has Number=A
[2017-05-22T07:21Z] skipping 'af_adj_exac_fin' because it has Number=A
[2017-05-22T07:21Z] skipping 'af_adj_exac_nfe' because it has Number=A
[2017-05-22T07:21Z] skipping 'af_adj_exac_oth' because it has Number=A
[2017-05-22T07:21Z] skipping 'af_adj_exac_sas' because it has Number=A
[2017-05-22T07:21Z] skipping 'af_exac_all' because it has Number=A
[2017-05-22T07:21Z] skipping 'max_aaf_all' because it has Number=A

I thought this was already fixed in the past, but it seems to be back. Could those values be parsed post vcfanno, so the columns get added to the db?
Thanks
M

so actually this should be executed after the vcfanno annotation, or on both occasions, but I'll leave that up to you, since I'm not entirely sure what the effect will be.
Cheers
M

fixed in #1941


how druid integrate with hadoop

Does druid integrate with Hortonworks Hadoop platform - HDP?
How install druid in HDP 2.4?

@gaoxuesong, druid can be configured to work with HDP 2.4 (which includes Hadoop 2.7.1 version)
You will need to follow the steps documented here -
http://druid.io/docs/latest/operations/other-hadoop.html

@nishantmonu51 Thanks


Chapter 9 - typo and optimization in code, pages 326-327

Hey! Big fan of the book here.
Typo
In Chapter 9, pages 326-327 in the code snippets there is a typo on the first_lvl_preds variable, it is written as fist_lvl_preds.
The code says:
fist_lvl_preds = np.zeros((len(X_test), 3))

But it should say:
first_lvl_preds = np.zeros((len(X_test), 3))

This would imply that the subsequent uses of the variable should also be renamed.
Optimization
In Chapter 9, page 326 third code snippet, the code looks like this:
second_lvl_oof = np.zeros((len(X_train), 3))
second_lvl_preds = np.zeros((len(X_test), 3))

for k, (train_index, val_index) in enumerate(kf.split(X_train)):
    skip_X_train = np.hstack([X_train, first_lvl_oof])
    model_1.fit(skip_X_train[train_index, :], y_train[train_index])
    second_lvl_oof[val_index, 0] = model_1.predict_proba(skip_X_train[val_index, :])[:, 1]

    model_2.fit(skip_X_train[train_index, :], y_train[train_index])
    second_lvl_oof[val_index, 1] = model_2.predict_proba(skip_X_train[val_index, :])[:, 1]

    model_3.fit(skip_X_train[train_index, :], y_train[train_index])
    second_lvl_oof[val_index, 2] = model_3.predict_proba(skip_X_train[val_index, :])[:, 1]

But it could be optimized if the skip_X_train variable is defined outside the for loop. I've rerun the code and arrived at the same results:

Hi @HectorLira, I am delighted that you are enjoying the Kaggle Book and that you are a big fan of it, and thank you a lot for spotting the typos and suggesting this enhancement. After your suggestion, both have already been implemented in this repository and in the code for the second edition of the book!


pyHexDump v1.0.0 adaptions

pyHexDump v1.0.0 changed its datatype names and introduced utf8 for UTF-8 encoded strings.

Thanks for keeping flashcontainer's pyHexDump support up2date.


Explicit expiration date for content availability

Public broadcasters sometimes are forced to remove content after some time, for legal  reasons.
7days and 30 are common.
A podcatcher will grab data and cache it so just not returning the episode in the feed isn't enough to prevent failing downloads.
Maybe also the feed isn't up to date with the file availability status.
How about being able to declare a date&time inside the feed and it's episode elements when the file will expire and no longer be available on the server?
This would allow us to not show ugly error messages and also give the user a last chance to store the episode locally before it expires.
Some stations even have a category for their video content with content they will soon no longer be available.

The correct way to deal with this is to remove it from the feed, surely?

If the feed has the last 10 episodes and the app caches the whole feed for 600 entries, it won't help.
Also you wouldn't know when an episode will be removed before it happens if the episodes just get deleted in the feed.

So is the intent to alert users to secure copies of a file that the publisher is legally required to remove?
Doesn't that open a legal problem in itself by prompting the user to circumvent the content from being removed?

I have had weird errors because of missing that info in a machine readable way.
TV stations like the German Public Station ARD already mark it on their website. So it isn't really about cheating the law here. You could remember the time limit yourself and download soon enough yourself or listen/watch the episode.

If the feed has the last 10 episodes and the app caches the whole feed for 600 entries, it won't help.

Then the app is doing it wrong. The canonical list of shows that are playable - at least, shows that the publisher wants you to be able to play - are the ones in the RSS feed.
Apps or directories shouldn't be storing shows that are no longer in the feed: that's not the publisher's intent.

Apps or directories shouldn't be storing shows that are no longer in the feed: that's not the publisher's intent.

Yes they should. If I have downloaded an episode, which then is removed from the server because the RSS feed only contains the last 10, I wouldn't want the app to wipe it off of my phone. We don't know the publisher's intent - they might do it because they're legally compelled to, they might do it to save server costs.

I think there’s a mismatch here between what an app should do and what a directory/aggregator should do. I would expect a directory to remove older episodes as they roll off the feed, but an app shouldn’t do that unless the listener has chosen to remove it.

I think it also is a nice feature to be able to curate and display a content expiring soon list so that people don't miss something.

If I have downloaded an episode, which then is removed from the server because the RSS feed only contains the last 10, I wouldn't want the app to wipe it off of my phone.

Agreed. I think that's fine. Let me slightly clarify my wording:
The canonical list of shows that are downloadable - at least, shows that the publisher wants you to be able to download - are the ones in the RSS feed.
I think most publishers don't expect older shows to be presented to listeners if they're not in the current RSS feed.
I'd also suggest that shows should NOT appear within an app if the audio is no longer available (whether that's a local cache on the device or on the fileserver).
(Incidentally, Spotify will remove a show from your local device's downloads if the publisher removes it from the RSS feed. I don't think that's the right behaviour for listeners, but I can see it could offer some legal protection for publishers.)
This sounds like a bit of best practice work for podinfra.net to be honest.

Well, you may be forced to reduce the number of episodes in your feed because of technical issues : I have 179 episodes, but can only publish the last 100s because my feed is 1Mb long (maximum size for some apps and platforms).
Yes, I'm pissed of that public broadcasters have to remove their content, but yes, It should appears somewhere, and not only in  <content> and/or <description> tag

For long feeds, you could use paged feed as advised here :
https://github.com/Podcastindex-org/podcast-namespace/issues/117#issuecomment-754031312
I think it can be a great addition to add an expiration date to encourage
listeners to listen or download a special temporary episode, but IMHO that
should be a simple information of when the item will be removed from the
feed.
It will never work to enforce apps to remove the episode, downloaded or
not, from the library of a user at a specific date. Most apps will not
enforce it, and even if they do it's a false sense of security as people
can download and save episodes manually anyway before the expiration date.
Pof Magicfingers (Giovanni Olivera)

Well, you may be forced to reduce the number of episodes in your feed
because of technical issues : I have 179 episodes, but can only publish the
last 100s because my feed is 1Mb long (maximum size for some apps and
platforms).
Yes, I'm pissed of that public broadcasters have to remove their content,
but yes, It should appears somewhere, and not only in  and/or
 tag
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Podcastindex-org/podcast-namespace/issues/318#issuecomment-990806914,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADST7IZ5CHL7GWLX3AJSG3UQHGWZANCNFSM5JGSCFHQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

There is also case where you have to remove an episode : because of legal action, because of bad quality, or retractation.
This tag would be explicit that the show is no more accessible, retired due to distribution rights

That's an HTTP 404 error on the audio, then.
I still don't understand why we need to clutter up RSS feeds with availability details. If it's in the RSS feed, it's available. If it is not, it is not.

To recreate a machine readable version of this ZDF feature.
"Last Chance- only available for a short time" (11h remaining)

@jamescridland  instead of 404, you should provide HTTP status 410

clutter up RSS feeds with availability details

It's not clutter if it's useful information. If that information needs to be displayed to the user, it has value to add it. For most cases, this will not be used, but when a content is only available for a limited time, let's say a special annoucement, for an upcoming event, for a discount, etc, it might be a nice addition to allow an "expiration date" shown to the user (Listen before wednesday)
In the same way youtubers sometimes title video : Watch this before Friday.
We could use the title in the same way, but why not include this as an attribute ? If the hosting provider allow to unpublish at a given time, this end of availability could be shown in apps as well.
I cannot see this useful for content control, because that's just a date, and the best way to remove an episode still is removing the episode from the feed. But having the option to show, in the UI to listeners, an expiration date for an episode is still something worst discussing.


CSV file of losses during traning

🚀 Feature

I would like to suggest the ability to save the losses to a csv file while training the model.
I wrote most of the required code for this features (it works, but it does not communicate with the args parameters yet such as the output dir. )
Motivation

The Motivation of this feature is an a simple way to check the graph of losses while training the model as the training time is long in order to avoid over fitting.
Pitch
The code is already written.
and it doesn't harm, and is a nice to have feature.
the only extenuation might be required is saving the csv to  the output dir by default instead of the working directory .

Alternatives

Additional context
SaveCSVFiles.zip

Hi,
I could accept such a feature in the repo. For that, I'd write a new MetricLogger similar to the one implemented in #163


Support upload of large GeoJson datasets

The current implementation relies on https://github.com/makinacorpus/Leaflet.FileLayer and is only performed at the client-side level.
Now this can cause multiple problems when managing large datasets:

the browser hangs while loading and can crash
if the connection fails when saving to the DB the data might be incomplete
if the data is not valid (eg bad topology) it cannot be saved in the DB

The process should probably be better handled on the backend side.

As a workaround we now perform saving to the DB from the client side in chunks. However, it appears that it often stops working after the first chunck has been created (eg in deployed dev version). Reducing the chunk size does not seem to avoid the problem. No specific logs appear except that the connection is lost between the client and the server just like if the client or the server is blocked.

We actually faced this issue https://github.com/socketio/socket.io/issues/2666, increasing the ping interval seems to resolve the problem.

The workaround seems to work fine for point geometries but not for other types. Indeed a bucket of 5000 polygons can lead to a very large number of points. We should probably create features buckets according to the total number of points in the geometry instead of the total number of features.


Split API into different files

Split api.go file into different files but keep the same package frida in each of them.

Implemented in 8c98c64


Scrolling feature added to file manager

When there are more files than certain limit (set to 7),
the file manager will not display more than 7 files and displays
two buttons that scrolls down/up (scrolls by 2 when possible,
scrolls by 1 instead if possible, otherwise doesn't scroll at end).

Wait hold on, there's a bug, executing/opening items after scrolling acts as if it hadn't scrolled at all. Will make a fix soon.

Thanks!


Add SSH port for AddKvmHost and AddSftpBackupStorage in zstack UI

Add SSH port for AddKvmHost and AddSftpBackupStorage in zstack UI
host  use sshPort  , SftpBackupStorage use sshport

fix: #108


[ko] Update traslation

#676, #679, #680
@Dalgona, @malkoG, @devleoper, @riseshia review please.

I'm done XD

Good to go! I think @Dalgona also loves this change.

Absolutely.


Update Post “termite-baiting-program-gold-coast”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
e123f38957c5e945a68dd93f304f2538b357ad77

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/653f467f1282c8000879db4a


chore(release): 1.156.1

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: ee926044fd0a259a50243d1a242880969c92c46a
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: ee926044fd0a259a50243d1a242880969c92c46a
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 8c559d4a451da96d7dd129cbe1e7e102a35f74ba
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 8c559d4a451da96d7dd129cbe1e7e102a35f74ba
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Location Button Listener

Hi, great plugin by the way! 👍
Just a question on handling native Google Maps APIs on Typescript (I'm new to both):
My goal is to set a listener to the Location Button click event (as described here), but so far I still haven't managed to do it via gMap
Any working example would be a great help!

Sorry, problem solved;
Just needed to look at how to implement Java interfaces in Nativescript
Relevant docs here


Is this scales good?

I notices that

the rooms / client data stores in memory.
the redis instance is used for pub/sub. for emit events between different processes

so with this adapter. I'm still storage data in memory. but the socket.io-adapter says

This module is not intended for end-user usage, but can be used as an interface to inheirt from from other adapters you might want to build.

So I'm asking is this module scales good?

or just I'm missing some info. some module does use redis for storage ?

You are right, socket objects are still kept in memory, redis is just used to broadcast messages accross servers.
Yet, as more server means you can handle more clients (along with handling node failures and such), that should scale pretty well.


fix(AbstractModel): all iteration over all public properties

Non-initialized properties were not included in iteration.  This commit also fixes setting of DateTimeInterface properties.
Fixes #123

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 13 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 99.497%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
396

Relevant Lines:
398

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

13 of 13 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 99.497%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
396

Relevant Lines:
398

💛  - Coveralls


Partial oauth stuff

We can close this now that https://github.com/beyond-all-reason/teiserver/pull/335 is merged


Update .dockerignore

@ghirschberg Should this PR be a draft? What is the reason for empty lines in dockerignore file?

@ghirschberg Should this PR be a draft? What is the reason for empty lines in dockerignore file?

sorry for this misunderstanding
I opened this PR to mimic a change from an open-source contributor (We implementing a new tests trigger from github to our internal resources)
its not on draft because github api treats draft PR's differently
I will change the title to do not merge


Create Dell EMC VMAX storage driver

Create Dell EMC VMAX storage driver as part of the Infrastructure Manager. Implement driver APIs.
This involves APIs
Register storage
List storage
List Pools
List Volumes
Add/Remove Trap config
Parse/Clear Alert
etc.

[12-May-20] Register driver API PR is in review. APIs list_pools and list_volumes in progress

/move sim


Use of custom display function with tags

Hi,
I'm trying to display my items according to this template :

 item.type  item.name

In that, I have created a display function:
display: item => {
          return createElement('span', {
              'class': this.itemClass(item.actif)
            },
            [
              createElement('span', this.itemStyle(item.type), item.type),
              item.name,
              createElement('i', {
                'class':{
                  state: true,
                  fa: true,
                  'fa-exclamation-triangle': item.late,
                  'fa-power-off': item.actif
                }
              }),
              createElement('i', {
                'class':{
                  state: true,
                  fa: true,
                  'fa-pause': item.standby
                }
              })
            ]
          )
        }

However, I get the error 'createElement is not defined', how do I implement my own model ?

Hi !
Where does that createElement function comes from ?
Are you trying to use React without JSX ?

Hi @elbywan,
Actually, I'm using Vue 2. (I am really sorry for the lack of further explanation of this post ...)
I am trying to use the render logic I guess, but since I am a beginner in vue, I am really struggling with it. As this syntax gives me compilation error I tried the more developed syntax with createElement()
display: item => item.name

(I am really sorry for the lack of further explanation of this post ...)

@The-BlackWidow No probs. 😉
A Vue.js component exposes the createElement function under a variable named $createElement.
So in your case you have to use this.$createElement instead of just createElement (just make sure that this refers to your component).
If you want to use the JSX syntax (which is of course much better) you can have a look at this issue, the babel plugin doc.  or the Vue doc. if you'd like to have some info on how to setup !
Hope it helps !

@elbywan works perfect! thank you very much

@The-BlackWidow Glad I could help 👍


Potential leak on Metal backend related to acquiring command buffer.

GFX version:
0.3.0 with PR
OS:
macOS Mojave 10.14.6 (18G87)
GPU:
Happens on both:
NVIDIA GeForce GT 750M 2 GB
Intel Iris Pro 1536 MB

I noticed a slow growing leak but couldn't reproduce it until I let the program run for 40 minutes straight with a couple of quads in the foreground. It grew for ~46mb to ~117mb but the one allocation I noticed the most was ~63mb. That being said Instruments does not indicate it as a leak so feel free to close it if it's not considered a leak.
The biggest one (~63mb) was under impl<B: Backend, C> CommandPool<B, C>
    /// Allocates a new primary command buffer from the pool.
    pub fn acquire_command_buffer<S: Shot>(&mut self) -> CommandBuffer<B, C, S> {
        let buffer = self.raw.allocate_one(RawLevel::Primary); // (`~63mb`)
        unsafe { CommandBuffer::new(buffer) }
    }

One level deeper.
fn allocate_one(&mut self, level: com::RawLevel) -> CommandBuffer {
        //TODO: fail with OOM if we allocate more actual command buffers
        // than our mega-queue supports.
        let inner = Arc::new(RefCell::new(CommandBufferInner {  //(`~63mb`)
            sink: None,
            level,
            backup_journal: None,
            #[cfg(feature = "dispatch")]
            backup_capacity: None,
            retained_buffers: Vec::new(),
            retained_textures: Vec::new(),
            active_visibility_queries: Vec::new(),
            events: Vec::new(),
            host_events: Vec::new(),
        }));
        self.allocated.push(Arc::clone(&inner)); //(~2.4mb)

I got a large trace to share, but unable to upload at this time.

Did you try running the same program on other backends?

@nicbn No, unfortunately I can't for now.

FWIW, I left my app (debug build) running all night, about 9 hours, and memory usage stayed around 108MB the entire time.
Using https://github.com/gfx-rs/wgpu-rs/commit/697393df4793e1a58578209885036114adfb9213
OS: macOS Mojave 10.14.6 (18G95)
GPU: Intel HD Graphics 630
(NOT TESTED this laptop also has a Radeon Pro 560)

Yeah I can't replicate this either anymore.


[Bug] disappeared cursor

Describe the bug
In the preview window my cursor is disappeared, it happend with every popup window as well. (It wasn't a shadow window)
To Reproduce
I opened a dm file, selected preview and the bug appeared.
Expected behavior
My cursor doesn't disappear.
Screenshots
I'll send you two videos on discord.
Desktop (please complete the following information):

OS: Win10 1903 (x64)  build: 18362.836
Version: <IP_ADDRESS>

Can you do a screenshot or recording?

https://discordapp.com/channels/588473626651787274/588474065136779274/711710305201160295


A few additions

A section in Project.xml that disables videos (totally didn't take this from the latest Psych....)
Added the missing credits icons

idk if actions are building so yea

It happens to me all the time idk what im doing w action builds LMAO


Question on list and details route

Hi all, I'm trying to find a way to map routes that looks like this:
-/directors/  map to director list
-/directors/xxxxx map to the director details
I came up with something like this:
#[derive(Target, Debug, Clone, PartialEq, Eq, Display)]
pub enum DirectorsRoute {
    #[target(index)]
    Index,
    Details(#[target(value)] String),
}

But no matter what I do, I always have "details" in the path
-/directors/  map to director list
-/directors/details/xxxxx map to the director details
What would be the best way to acheive this?
I found a way using something like this:
    Directors {
        director_id: String,
        #[target(nested)]
        target: DirectorRoute,
    },

pub enum DirectorRoute {
    #[target(index)]
    Index,
}

But then I need to pattern math with a gard where I check if the director_id is empty then display the list and if not display the details....
Is this the way? :p

Sorry for the late response. I just checked an I came up with the same solution.

pub enum App {
  #[target(index)]
  Index,
  Directors(Directors)
}

pub enum  Directors {
        director_id: String,
        #[target(nested)]
        target: DirectorRoute,
};

pub enum DirectorRoute {
    #[target(index)]
    Index,
}

The idea was that the value takes the first (local) segment. And thus would result in an empty value.
An alternative could be:
pub enum App {
  #[target(index)]
  Index,
  Directors
  #[target(rename="directors"]
  DirectorsWith{ id: String }
}

@ctron thanks for the help. I do not know why I did not get any notification on your response. I've probably missed it :)
I went for the second approach that works like a charm:
pub enum App {
  #[target(index)]
  Index,
  Directors
  #[target(rename="directors"]
  DirectorsWith{ id: String }
}


Seems like the author's cloud share link is down, cannot download EXE

https://beuaaa.synology.me/Pywinauto_recorder_installer.exe    is 404

The actual link is
https://github.com/beuaaa/pywinauto_recorder/releases/download/0.6.7/Pywinauto_recorder_installer.exe
The docs within the repo has the correct link but I believe the updated docs isn't the one currently live on read the docs.

The actual link is
https://github.com/beuaaa/pywinauto_recorder/releases/download/0.6.7/Pywinauto_recorder_installer.exe
The docs within the repo has the correct link but I believe the updated docs isn't the one currently live on read the docs.

thanks

The automatic generation of the documentation no longer worked and the link to download the installer was broken. Now it's been fixed and everything works normally.


Windows 10: SublimeLinter: WARNING: phpcs deactivated, cannot locate ''

Hey, I get this info with an empty string when I try to use phpcs linter.
The log from ST console:
reloading plugin SublimeLinter.sublimelinter
reloading plugin SublimeLinter-phpcs.linter
SublimeLinter: phpcs linter loaded
plugins loaded
SublimeLinter: WARNING: phpcs deactivated, cannot locate ''
SublimeLinter: php activated: C:\php\php.exe

PHP linter works fine, but phpcs doesn't want to. Please, could you guide me?
I tried every single fix I could find - none worked. Note that phpcs is in my path and it's working when I call it from console.

Installing it via PEAR does the job.


[AMP] The pages aren't authorable upon AMP Only mode selection

The pages in the AMP included project are not authorable in the local instance as the root container drag and drop parsys is not displayed.
The archetype command for the project creation:
mvn archetype:generate -DarchetypeGroupId=com.adobe.aem -DarchetypeArtifactId=aem-project-archetype -DarchetypeVersion=24 -DappTitle="Archetype 24 Test Site AMP" -DappId="archetype24TestSiteAMP" -DgroupId="com.archetype24TestSiteAMP" -DartifactId="archetype-24-test-site-amp" -Damp="y"

Here is snapshot of the page rendition:

@VarsTheMars the AMP version of a page is only for view/preview, you can't edit the page in that view. To edit you have to switch back to the normal view.

@msagolj I got it, the AMP mode is designed in a way which limits all the necessary JavaScript libs needed for page authoring.


Broken with yarn?

Not that I care strongly but just noticed I couldn't get it running with yarn. Got some error from React Router.
Maybe you can reproduce this by deleting the package lock and doing another install. Interesting what dependency broke?

I guess maybe these should be pinned instead of ^.
https://github.com/dalemartyn/podcast-player/blob/b583d3d510167a15450e61d6f42d8e0e50d2996a/package.json#L23-L24

Hi Dan, I've pinned them to alpha 5. Hopefully this resolves your issue with yarn. 👍


Processes name stop displaying after some time

I've got pwru log file that includes information on the processes that created the SKBs. At the beginning of the logs, the name and PID of the processes are displayed as expected. Starting with line 74415 (record 74414), the name of the processes that created the SKBs is no longer displayed, and shows as <empty>. The PID still shows up as expected.
$ pwru --version
pwru v1.0.4-1-g1e10c197d1c0

$ sudo pwru \
    --output-tuple \
    --output-meta \
    --output-file=/tmp/pwru.log \
    --ready-file=/tmp/pwru.ready \
    'host <IP_ADDRESS> or port 60154'

pwru.log

Indeed we can fetch pname in bpf kernel space via task->comm or task->mm->arg_start, but that approach might impact performance.

Probably not worth the trade-off. If this happens only when the process is terminated early, then I'd say it's fine to miss info like the process name, rather than slow down the whole execution.

Another easy fix is to cache pname in pwru user space during an skb's lifetime.

Sounds like a better option to me, for what it's worth.

Another easy fix is to cache pname in pwru user space during an skb's lifetime.

Any TL;DR how it would work?

I was thinking using a user-space map[skb]pname to cache pname conversion, but that actually doesn't make sense because an skb could be transferred between processes (e.g. from a client process to a server process).
Then I thought we could cache map[pid]pname, but the entry eviction is tricky, we should find an appropriate event to delete an pid->pname entry in map, instead of caching the entry forever. LRU? Timer? Tracing the lifetime of a process?

Not sure about others, but TBH for me a process name is very little of use when debugging.

Then I thought we could cache map[pid]pname, but the entry eviction is tricky, we should find an appropriate event to delete an pid->pname entry in map, instead of caching the entry forever. LRU? Timer? Tracing the lifetime of a process?

LRU, don't evict, just read from the cache when you cannot get the pname from the regular conversion method?

Not sure about others, but TBH for me a process name is very little of use when debugging.

Agreed it's nice to have, but not a big issue if it's occasionally broken


How to add Menus at runtime in Main Window

i want  after user login main menu is added . is it possible????
Please help me

The approach I would recommend is using a view-model where you define your LinkGroups and Links in an ObservableCollection. Your ModernWindow view should bind to this view-model. When your app is in a non-logged-in state, there are no groups and links, when logged-in you add your LinkGroups and Links. Makes sense?

Do you have any example for link-groups that can help me better?
Thanks

Consider the following viewmodel
    public class ViewModel
        : NotifyPropertyChanged
    {
        private LinkGroupCollection groups = new LinkGroupCollection();

        public LinkGroupCollection Groups
        {
            get { return this.groups; }
        }

        public void Login(string user, string password)
        {
            // TODO: validate credentials
            var success = true;
            if (success) {
                // logged in, fill groups
                var group = new LinkGroup { DisplayName = "group" };
                group.Links.Add(new Link { DisplayName = "link 1" });
                group.Links.Add(new Link { DisplayName = "link 2" });
                this.groups.Add(group);
            }
        }

        public void Logout()
        {
            // remove all groups and links
            this.groups.Clear();
        }
    }

In your MainWindows.xaml.cs, assign the view model to the window DataContext like so;
    public MainWindow()
    {
        InitializeComponent();

        var vm = new ViewModel();
        this.DataContext = vm;

        // login
        vm.Login(null, null);
    }

And in you MainWindow.xaml, make sure the MenuLinkGroups are bound to the ViewModel.Groups;
<mui:ModernWindow x:Class="muitest.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        ...
        MenuLinkGroups="{Binding Groups}">

its working from mainwindow.xaml But i want to do it from login.xaml

is it possible or not just tell? can i add menus from different xaml page. I need your reply sir if its not possible i will think on any other solution?
Your reply can save my time.
Thanks

I think you can add an event or some kind of messaging pattern in this case, I've a small app that hides part of the menu when reaches some state, I used messaging to do this. see mvvm light (https://mvvmlight.codeplex.com/) for a sample

@heraclesaraujo can you say the files for this in your project? cause it's pretty big, and I don't want to take my day to search in it.

Place this is the login. It will access the menu items in the main window
var window = Application.Current.MainWindow as ModernWindow;
window.MenuLinkGroups.Add(group);


Question on Amplify config

Hi, this is a question, not an issue:
In the step "Configure AWS Amplify", in the Serverless Stack guide, all the config is placed in an config.js in the /src folder.
My question is: is it not a security flaw, to expose such infrastructure details like S3 Bucket Name and Cognito IDs in the js file, and send them to the browser? I understand exposing the API urls and region, but is it ok to also expose the rest of that config.js file?
Thanks in advance,

Those are completely fine. The access to those resources are restricted by authentication. And we don't allow unauthenticated access to them.

Thank you!


chore: Update from _rust/main template

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.6%) to 80.0%

Totals

Change from base Build<PHONE_NUMBER>:
2.6%

Covered Lines:
24

Relevant Lines:
30

💛  - Coveralls


Attempts at getting MVP agent running.

Here are the changes we made to attempt to get the agents running.

These updates enable the demo agent to run as a backend for independent frontend development.


fix soul-example maven pom

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #1120 (b82eff8) into master (3437884) will decrease coverage by 0.70%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1120      +/-   ##
============================================
- Coverage     72.87%   72.16%   -0.71%
+ Complexity     2054     2045       -9
============================================
  Files           350      350
  Lines          7914     7914
  Branches        870      870
============================================
- Hits           5767     5711      -56
- Misses         1722     1778      +56
  Partials        425      425

Impacted Files
Coverage Δ
Complexity Δ

...soul/client/tars/TarsServiceBeanPostProcessor.java
24.00% <0.00%> (-61.34%)
5.00% <0.00%> (-9.00%)

...omara/soul/sync/data/http/HttpSyncDataService.java
63.72% <0.00%> (-8.83%)
12.00% <0.00%> (ø%)

.../soul/plugin/httpclient/NettyHttpClientPlugin.java
57.14% <0.00%> (-2.05%)
10.00% <0.00%> (ø%)

...elimiter/handler/RateLimiterPluginDataHandler.java
74.13% <0.00%> (ø)
12.00% <0.00%> (ø%)

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 3437884...b82eff8. Read the comment docs.


Basic predicates pushdown support for  SQL Server Data Connector

Implemented via https://github.com/spiceai/spiceai/pull/2768


Fix wrong implementation of "Double-Checked Locking".

Fix wrong implementation of "Double-Checked Locking".

[ ] Add a unit test to verify that the fix works.
[x] Explain briefly why the bug exists and how to fix it.
Add volatile to avoid only parts fields of DEFAULT_LOADER are visible, reference http://www.cs.umd.edu/~pugh/java/memoryModel/DoubleCheckedLocking.html

I think it is fine to keep it as it is. Any concern?


v5 breaks backwards compat

The package.json#module path should point to the esm bundle and a commonjs bundle should be built and package.json#main path should point to the commonjs build.

Please read the release notes. I don't plan to provide a CommonJS build.


How set Jaeger endpoint

Hi, i try to use the python version of the jaeger client.
In go i do that to create the tracer :
cfg := jaegercfg.Configuration{
		Sampler: &jaegercfg.SamplerConfig{
			Type:  jaeger.SamplerTypeConst,
			Param: 1,
		},
		Reporter: &jaegercfg.ReporterConfig{
		LogSpans:           true,
		LocalAgentHostPort: "<IP_ADDRESS>:5775",
	},
}

jLogger := jaegerlog.StdLogger
jMetricsFactory := metrics.NullFactory
tracer, _, err := cfg.New(
		"myservice",
		jaegercfg.Logger(jLogger),
		jaegercfg.Metrics(jMetricsFactory),
)

How do that in Python ?
Thanks

The address of the agent is configured automatically (in Go as well), you do not need to provide it in the configuration. If you still really want to, you can override the port via:
config = {
    'local_agent': {
        'reporting_port': number,
        'sampling_port': number,
    },
}

OK.
But i would like to use the jaeger all-in-one image.  :
docker run -d -p5775:5775/udp -p16686:16686 jaegertracing/all-in-one:latest

It's not on the same host.

I think Python client doesn't support talking to an agent on another host (it's also not recommended because clients use UDP to send the packets, which won't be lossless, unlike when talking over the loopback interface).
A quick fix would be to extend Python client to support reporting_host_port config param as an alternative to just the port.
Longer term, we are having ongoing discussions about supporting reporting over HTTP.

When an application is  running in docker container, i can't communicate with the jaeger client by using localhost.So providing an reporting_host will be a good choice.

I believe this, and https://github.com/uber/jaeger-client-python/pull/49 are addressed in https://github.com/uber/jaeger-client-python/pull/49

I launch the all-in-one image :
$ docker run -d -p5775:5775/udp -p6831:6831/udp -p6832:6832/udp -p5778:5778 -p16686:16686 -p14268:14268 jaegertracing/all-in-one:latest

Try this code :
import opentracing
import logging
import time
from jaeger_client import Config

if __name__ == "__main__":
    log_level = logging.DEBUG
    logging.getLogger('').handlers = []
    logging.basicConfig(format='%(asctime)s %(message)s', level=log_level)

    config = Config(
        config={ # usually read from some yaml config
            'sampler': {
                'type': 'const',
                'param': 1,
            },
            'local_agent': {
                'reporting_host': "<IP_ADDRESS>",
                'reporting_port': 5775,
            },
            'logging': True,
        },
        service_name='my-app',
    )
    tracer = config.initialize_tracer()

    with opentracing.tracer.start_span('TestSpan') as span:
        span.log_event('test message', payload={'life': 42})

        with opentracing.tracer.start_span('ChildSpan', child_of=span) as child_span:
            span.log_event('down below')

    time.sleep(2)   # yield to IOLoop to flush the spans - https://github.com/uber/jaeger-client-python/issues/50
    tracer.close()  # flush any buffered spans

And the output logs :
$ python foo.py
2017-09-11 16:34:08,509 Initializing Jaeger Tracer with UDP reporter
2017-09-11 16:34:08,512 Using sampler ConstSampler(True)
2017-09-11 16:34:08,513 opentracing.tracer initialized to <jaeger_client.tracer.Tracer object at 0x7fe7e3d68c50>[app_name=my-app]
2017-09-11 16:34:08,514 Reporting span d8f5c5f29cb6c02d:de0863b7a8ed4610:d8f5c5f29cb6c02d:1 my-app.ChildSpan
2017-09-11 16:34:08,514 Reporting span d8f5c5f29cb6c02d:d8f5c5f29cb6c02d:0:1 my-app.TestSpan
Exception in thread Thread-1 (most likely raised during interpreter shutdown):
Traceback (most recent call last):
  File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner
  File "/usr/lib/python2.7/threading.py", line 754, in run
  File "/home/vagrant/Projets/sirocco/venv/local/lib/python2.7/site-packages/threadloop/threadloop.py", line 78, in _start_io_loop
  File "/home/vagrant/Projets/sirocco/venv/local/lib/python2.7/site-packages/tornado/ioloop.py", line 904, in start
<type 'exceptions.AttributeError'>: 'NoneType' object has no attribute '_current'

reporting_host and reporting_port seem to work fine now. I think this issue can be closed? @yurishkuro

Yes, closing.
@nlamirault the exception you showed is something inside tornado, could you please open a new issue with it and include the versions of the dependencies you're using?  I have run https://github.com/yurishkuro/opentracing-tutorial/tree/master/python with the latest jaeger client and it had no issues (using Python 2.7)

@yurishkuro What is the meaning of local_agent config? If I'm running jager-all-in-one using jaeger-operator on k8s, what should be the config array here?
`# default tracer is opentracing.Tracer(), which does nothing
OPENTRACING_TRACER_CALLABLE = name + '.tracer'
def tracer():
install_all_patches()
config = Config(
config={
'sampler': {
'type': 'const',
'param': 1,
},
'local_agent': {
'reporting_host': 'jaeger',
'reporting_port': '6831',
},
'logging': True,
},
service_name='devhubserver',
validate=True,
metrics_factory=PrometheusMetricsFactory(namespace='devhubserver')
)
return config.initialize_tracer()
default is False
OPENTRACING_TRACE_ALL = True
default is []
OPENTRACING_TRACED_ATTRIBUTES = ['path', 'method', 'META', 'body', 'path_info', 'content_type', 'content_params', 'GET', 'POST', 'COOKIES', 'FILES', 'headers']`


Fix #112 : google and yandex TileLayers compatiblility for Leaflet 1.0

To merge on Leaflet 1.0 release...

What exactly is blocking this?

@miguelcobain just waiting the stable release of Leaflet core...

@brunob ok. thanks.
FWIW, current leaflet rc is very stable, and it has been for a while.
It would be great to have leaflet 1 support in a bower pre-release version.
Also, it would let some devs to test it earlier rather than later.

@miguelcobain leaflet_one branch is now resynced with master, you can test it :)

Hey, just wondering if there's a release plan for this? If there's anything you need an extra set of eyes on, etc?

@mattgrande i'm on it...

Thanks!

@mattgrande anyway see #250

@nmccready since we'll remove google script here, could you please point me what can i safely remove in this branch for karma, google specs & dev dependencies etc ?

@brunob sure why is the script being removed again. Just wondering if I need to use a other library for the next version. As for getting rid of karma,  sure go for it. However,  I recommend leaving it and doing a lot more specs for it.

why is the script being removed again

@nmccready see #250

Just wondering if I need to use a other library for the next version

yes

Awesome thank you


Add relations for inter-term navigation

When viewing a term that has multiple entries, other entries with the same name (e.g., "Glacier") are not accessible from the term currently being viewed. This can be accomplished in a variety of ways and is a missed opportunity to leverage Linked Data, relations, etc. This might also assist the user in locating the term that more aligns with the variant of their understanding.
I can imagine this being implement from either/both of UI elements but also out-of-band semantic web info like the Link HTTP response header (a la Memento).

Right now they are displayed when you click on the number in Alternative Definitions under the term itself. Is this what you mean? It's definitely not obvious, though.

@cr625 Yes, in terms of UI elements that suffices. Would we be able to surface that data in a machine-readable way? On the flip side, this would increase the size of the payload for each term, albeit perhaps trivially.

@machawk1 we could return arbitrary data at a URL endpoint. Would this be a one-off or part of a more structured API?

@cr625 An API would be useful internally as a means to associate the terms with identical spellings. For example, considering the Glacier instance at ark:/99152/h5949 (which resolves to https://yamz.net/term/ark/h5949), upon resolution we might also use this "key" as a basis for an API to get ARKs of terms with the same spelling. The resulting association (e.g., the other Glacier ARKs and perhaps their term strings) could be represented in the HTTP response headers. This could be use to populate the UI element you mentioned as well.