make_batch_reader fails on nested struct

I have an external parquet file that I'm trying to load using the hello world example, but it fails as such:
  File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/petastorm/etl/dataset_metadata.py", line 389, in infer_or_load_unischema
    return get_schema(dataset)
  File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/petastorm/etl/dataset_metadata.py", line 343, in get_schema
    'Could not find _common_metadata file. Use materialize_dataset(..) in'
petastorm.etl.dataset_metadata.PetastormMetadataError: Could not find _common_metadata file. Use materialize_dataset(..) in petastorm.etl.dataset_metadata.py to generate this file in your ETL code. You can generate it on an existing dataset using petastorm-generate-metadata.py

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "petastorm_genome.py", line 30, in <module>
    python_hello_world()
  File "petastorm_genome.py", line 22, in python_hello_world
    with make_batch_reader(dataset_url, schema_fields=FIELDS) as reader:
  File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/petastorm/reader.py", line 349, in make_batch_reader
    transform_spec=transform_spec)
  File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/petastorm/reader.py", line 432, in __init__
    stored_schema = infer_or_load_unischema(self.dataset)
  File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/petastorm/etl/dataset_metadata.py", line 393, in infer_or_load_unischema
    return Unischema.from_arrow_schema(dataset)
  File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/petastorm/unischema.py", line 266, in from_arrow_schema
    codec, np_type = _numpy_and_codec_from_arrow_type(field_type)
  File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/petastorm/unischema.py", line 392, in _numpy_and_codec_from_arrow_type
    raise ValueError('Cannot auto-create unischema due to unsupported column type {}'.format(field_type))
ValueError: Cannot auto-create unischema due to unsupported column type struct<featureId: int32, featureValue: int32, featureValueVersions: struct<key_value: list<key_value: struct<key: int32, value: int32> not null>>, _featureNameSpace: int16>```

If there are unsupported types, could you please make this clear in the documentation? Also, is it possible to have make_batch_reader support the warn_only parameter so that we can skip features that aren't supported? The schema passed into make_batch_reader allows us to skip top level column names, but if we have nested fields, can we make it ignore the unsupported subfield and deserialize the rest of the struct? Is there a time frame for supporting list structures?

The field in your error message (struct<featureId: int32, featureValue: int32, featureValueVersions: struct<key_value: list<key_value: struct<key: int32, value: int32> not null>>, _featureNameSpace: int16> is not supported by pyarrow (ARROW-1644) as an aggregate field that is not a list. We are watching that arrow ticket and once it is resolved, will reenable it in petastorm as well.
PR #357 makes sure the field is ignored with a warning and does not cause the crash.
Would appreciate if you could let me know if that PR solves your issue.


Further burn-in prevention

It'd be pretty convenient to have a setting for automatically switching sources, and/or a setting to dim or even turn off the display if there is no keypress activity for a set amount of time.

Agree, I'll put it on my TODO list. From what I can tell there's no option to dim the screen though. There's only on or off.

I had an idea to prevent at least some burn-in, it would be to "move the display" maybe 1 pixel every x seconds (maybe only in 1 axis or both?), I don't know if that is possible, but I think that it could reduce the possibility of having some burn-in

It could be combined with #13 as a sort of screensaver :)

I thought about it, but I thought it would be more complicated to implement gifs than to "shift" the screen a few pixels.


Ensure TLS Ports are picked as default, if provided in the creds

@jplebre  I'm sorry, but it looks like we were missing the .NET Foundation CLA bot when this PR was entered, can you confirm that you've signed the CLA? - https://cla.dotnetfoundation.org/SteeltoeOSS/steeltoe?pullRequest=63


[Issue #1096] - Fix lint issue from previous

Purpose
Resolves #1096
Fixed the lint issue and updating the copyright
PR checklist

[x] Add link to the issue
[x] Choose Project
[x] Choose PR label
[ ] New unit tests added
[x] Modified code has meaningful comments
[ ] All TODOs are linked with the issues
[ ] All comments are resolved

Testing
running builds and CI test

closing... wrong branch...


Retrieve key from backup

When retrieving a key from a backup, I get a mixed display:
"Installed key" seems to be the key from the backup that I was looking for
"OEM key" and "OEM edition" still show the OEM key from my current installation (not the backup)

PS: Is there an easy way to copy the retrieved keys to clipboard?

That makes sense -- yes, I saw the "OEM key not available from backup" message on a screenshot and concluded that something was displayed wrongly in my case. Thank you for the response!


fix: generate aliyun theme correctly

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
The theme content which in style/aliyun.less can not be compiled to the final ng-zorro-antd.aliyun.css correctly.
What is the new behavior?
aliyun theme is working correctly now.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

@wenqi73 Hi do you have time to review this PR?

@simplejason  No problem with ng-zorro-antd.aliyun.css, the PR can be closed.

I tried complied from source code, and the generated aliyun.css was not working porperly. Some styles in aliyun.less would be overrided by same named styles which in entry.less.
Because of this, I cannot use theme aliyun

In fact, it works well in our own project(following the official doc: https://ng.ant.design/docs/customize-theme/zh#切换主题), maybe you can provide a minimal reproduction to describe how you are using :)


feat: add a workspace start prompt in daytona code for stopped projects

add a workspace start prompt in daytona code for stopped projects
Description
ask user to start workspace if daytona code or ssh cmd is run for a stopped project

[ ] This change requires a documentation update
[ ] I have made corresponding changes to the documentation

Related Issue(s)
closes #914
/claim #914
Screenshots

Putting this in draft until we merge #1004 since they share some changes.

@Tpuljak could you pls help me with the lint check ?

@Tpuljak could you pls help me with the lint check ?

Seems like it was just a hiccup in the pipeline. Reran it and it works now.


I start a dev-server on my pc, but i cant open the web in my iphone with the same wifi

I start a dev-server on my pc, such as localhost:8080, I can open it in my pc chrome, but i cant open the web in my iphone with the same wifi, I  use the pc's ip:8080 .....help me plz

Do you want to request a feature or report a bug?

What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.

It looks like you just deleted/ignored our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please open a new issue with filled issue template. Also make sure your issue is not a question. Questions should be posted on Stack Overflow.

devtool: '#eval-source-map' =====> devtool:'inline-source-map'，


Added missing descriptions for GeometryInstance3D

GeometryInstance3D had a few missing descriptions, this PR fills them in based on my reading of the code.

Thanks!


DB_HOST vs DEFAULT_HOST

Summary
Using DEFAULT_HOST parameter for database server hostname is not working. Use DB_HOST parameter instead works.
This is not clear in documentation or this is not how it's supposed to work.
Environment

Image version / tag:  4.0.18

DEFAULT_HOST along with DEFAULT_NAME, DEFAULT_USER, DEFAULT_PASS, and DEFAULT_PORT don't exist.
DEFAULT_HOST is a special one as that is what I use to count the multiple instances and generate the schedulers..


Experiment with Adsense Auto Ads?

Google auto-optimisation option for better results: https://support.google.com/adsense/answer/7478040?hl=en

We won't use this as may place adverts in 'bad' places for users.


[v1.2][ISSUE-351]remove resnet18 downloading for ma unit test

remove resnet18 downloading for ma unit test to avoid network issue

@XinyaoWa , what is the background for this PR?

Before in UT, we need to download resnet18, but always fail due to network issue. Now I put the downloaded weights on the testing server to avoid this issue.

@XinyaoWa , what is the background for this PR?

Before in UT, we need to download resnet18, but always fail due to network issue. Now I put the downloaded weights on the testing server to avoid this issue.

Thanks for clarification, I merge UT pr firstly, which leads to a conflict here. Please help to fix it, I'll merge this PR later.

@XinyaoWa , what is the background for this PR?

Before in UT, we need to download resnet18, but always fail due to network issue. Now I put the downloaded weights on the testing server to avoid this issue.

Thanks for clarification, I merge UT pr firstly, which leads to a conflict here. Please help to fix it, I'll merge this PR later.

fixed, please have a check.


Is the acceleration effect really achieved?

Can code that enables acceleration be provided?

Sorry, we tested the acceleration by transforming the model format to OpenVino. Additionally, we currently have no plan to release the related code. You can refer to https://github.com/ModelTC/TFMQ-DM/issues/2.


"file not written to" is overly careful

Commands like cargo crev crate review and cargo crev trust open an editor and refuse to act if the file has not been saved.
The default values in the templates are often rather good (especially a crev trust often needs no adjustment). Some editors do not even offer saving if the file is unmodified. (gedit does, vim's :wq does, vim's ZZ does not).
If the check is still considered adequate, maybe a "The file was not saved. Do you confirm the (review, trust setting) with the default values? (y/n)" would be a better message to ask/show.
Cc @dns2utf8 who mentioned he ran into this as well.

Someone reported on riot that not writing should cancel, and I think that's how git commit works as well. I changed the behavior in https://github.com/crev-dev/cargo-crev/commit/e2c58485c82054d2fec9ea8b3145e01dd9148a3b .
More discussion needed to figure out behavior that is fine with everyone.

git commit aborts on an empty message; not writing is fine, and is a typical thing when running git merge or git commit --amend. Only typical git commit, the default message just happens to be empty.

We still want some reasonable way out for people that just want to cancel. So I guess asking if the user wants to cancel or commit is a best way to go. I personally got used to and enjoyed that easy way to cancel, even though I myself initially implemented it as "no write == defaults".
We could also make it a setting or something.
PRs welcome. :)


Comment for godoc

Could you comment your code for using in godoc? The documentation would help to work

Any progress?


Write a Rust program to find surface area of a cube

Description
Write a Rust program to find surface area of a cube
Surface area of cube = 6 × Side^2
Side : 2

Surface area : 24

How to contribute

Save the solution in find_surface_area_of_a_cube.rs file
Add find_surface_area_of_a_cube.rs file in find-surface-area-of-a-cube folder

!assign

Hey @victoriacheng15,
This issue has been assigned to you for more than 15 days.
We are unassigning you from this issue.
If you are still interested in contributing to this issue, please get auto-assigned on the issue by commenting !assign again.
Thanks for your contributions :)

!assign


[Bug] - Unable to add files from cloud services

What type of issue report is this?

Please check one or more that apply

[x] Bug
[ ] Regression
[ ] Question/support
[ ] Other (include details)

Which packages are you using?

Version 1.5.19036.3

What is the current behaviour?

When I go to open a file from Google Drive it won't return the item correctly.

What is the expected behaviour?

When I open a file from Google Drive the item should return correctly.

What platforms does this affect?

Please check one or more that apply

[ ] Windows (UWP)
[x] Xamarin.Android
[ ] Xamarin.iOS
[ ] Xamarin.Forms

Work on this bug was completed on PR #67, closing this off.
Fix will be made available in 1.6 release


Allow custom definition of auto-import completions?

See this StackOverflow question: https://stackoverflow.com/questions/78845297/how-do-define-custom-auto-imports-for-pylance-visual-studio-code
Where the answer determined that currently Pylance has the following auto-import completions hardcoded:

np for numpy
pd for pandas
tf for tensorflow
plt for pyplot/matplotlib
mpl for matplotlib
m for math
spio for io/scipy
sp for scipy
pn for panel
hv for holoviews

Can we allow definition of custom auto-import triggers? For example, pl for polars?

I know we've discussed this before, but I don't recall the decision and can't find an existing issue on this.
Reassigning to HeeJae who may know.

@luabud should expose it as option?

I believe this is a dupe of https://github.com/microsoft/pylance-release/discussions/2589? We currently only have 4 upvotes there

Resolving as a duplicate. @bzm3r, please upvote https://github.com/microsoft/pylance-release/discussions/2589 if you haven't already.


zephyr: add mimxrt1060_evkb to sample tests

Test alongside mimxrt1024_evk to see if the serial output is more stable during CI testing.

Early testing shows this is not performing any better than the 1024 boards so we'll stick with those.


Deal with offline crashes

We should cache them somewhere and send them later.
This is required for windows forms apps, windows phone and xamarin etc.

Do you guys have a timeline for this yet? Just curious, thanks!

It's not officially scheduled, but this is probably one of the more important issues open on this notifier. Hopefully soon, and sorry I can't be more specific.

Fixed in #55


Push technical post to Hacker Noon, YC, Ionic 11/7

From tenfour-archive created by caharding: ushahidi/tenfour-archive#1475

OK weve got this one live now! Can you connect with Hacker Noon?
https://medium.com/@dalezak/how-ushahidi-turned-tenfour-mobile-app-into-progressive-web-app-pwa-56512cbd2816

@dalezak - it looks like Hacker Noon changed their policies a little bit and they want people to submit unlisted stories. That's OK. Going forward, we can try that route.
http://www.amipublications.com/hacker-noon
Another interesting place to submit is FreeCodeCamp - their Medium gets a ton of traffic.
<EMAIL_ADDRESS>Here's a great primer on how to get published there (and how to think about Medium posts).
https://medium.freecodecamp.org/how-to-get-published-in-the-freecodecamp-medium-publication-9b342a22400e
I'm happy to work with you on the next one in the editing / ideation stages.
There are other places we can also publish technical blog posts. I'll build that out./
I'd also recommend, that, when Freecodecamp gets back to you, if they say no, you post it on HackerNews - it doesn't hurt at all and you can get your distributed team to upvote it to give you a little bit of a boost (make sure they are not the same IP address and are not obviously connected - unfortunately, this is how every team tries to work with Hacker News).
If you want to chat - I can give you more ideas on this!

Yup. Asking my buddy over there right now. Also, lets get Will or Dale to be come a contributor.
https://hackernoon.com/about

@caharding is this post live? If so, can you put the link here? I believe this is one on Ionic.

Great. Here is the updated link, it hadn't been apart of the Medium
publication:
https://medium.com/tenfour/how-ushahidi-turned-tenfour-mobile-app-into-progressive-web-app-pwa-56512cbd2816
I think we will need your friend at Hackernoon to add Will / Dale to become
a contributor.
On Mon, Dec 3, 2018 at 11:26 AM Alex Salkever<EMAIL_ADDRESS>wrote:

Yup. Asking my buddy over there right now. Also, lets get Will or Dale to
be come a contributor.
https://hackernoon.com/about
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ushahidi/tenfour/issues/1475#issuecomment-443837474,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABizaRTTKTngmaCYFZUN_ETTuJs-BtAkks5u1XrHgaJpZM4YMV3G
.

--
Charlie Harding
Product Director


Hide/Show the BottomBar

I think you should provide a way to hide and show the ButtonBar programatically (with the animation).

how can we show or hide bottom bar programmatically, i need to show bottombar fixed for the screens which does not have scroll, how can i do that

If your bottombar is in a CoordinatorLayout, you can use behavior to implement hide/show feature, the same way it can be done with a Toolbar :
public void showBottomBar(boolean show){
    CoordinatorLayout.LayoutParams params = (CoordinatorLayout.LayoutParams) mBottomBar.getLayoutParams();
    CoordinatorLayout.Behavior behavior2 = params.getBehavior();
    if (behavior2 != null) {
        behavior2.onNestedFling(mCoordinatorLayout, mBottomBar, null, 0, show ? -10000: 10000, true);
    }
}

Yeah I have implemented it, but a way to do it within the lib would be better :smile:

Back in version 1.4 there used to be 2 methods, hide() and show(), which to have been removed from version 2.0.

It looks like it's slightly implemented in the bottom Bar already but the method is private and does the reverse of what it says. Here's what's in the class.
/**      * Toggle translation of BottomBar to hidden and visible in a CoordinatorLayout.      *      * @param visible true resets translation to 0, false translates view to hidden      */     private void toggleShyVisibility(boolean visible) {         BottomNavigationBehavior<BottomBar> from = BottomNavigationBehavior.from(this);         if (from != null) {             from.setHidden(this, visible);         }     }
The BottomNavigationBehavior setHidden method is package level. So I extended the BottomBar class using the same package to be able to call this method.
`
package com.roughike.bottombar;
import android.content.Context;
import android.util.AttributeSet;
public class CSBottomBar extends BottomBar {
public CSBottomBar(Context context) {
    super(context);
}

public CSBottomBar(Context context, AttributeSet attrs) {
    super(context, attrs);
}

public void toggleShyVisibility( boolean visible ) {
    BottomNavigationBehavior<CSBottomBar> from = BottomNavigationBehavior.from(this);
    if (from != null) {
        from.setHidden(this, visible);
    }
}

}`
I kept the method "as is" even though it does the reverse of what the documentation states.  Passing false makes it visible.  True makes it disappear.  And it is animated.

Great hack! However it doesn't work for me:
Fatal Exception: java.lang.IllegalArgumentException: The view is not associated with BottomNavigationBehavior
       at com.roughike.bottombar.BottomNavigationBehavior.from(BottomNavigationBehavior.java:122)
       at com.roughike.bottombar.CSBottomBar.toggleShyVisibility(CSBottomBar.java:20)

My bottom bar is declared non-shy, perhaps that's why. I'd like a non-shy bottom bar and only hide the bottom bar in certain situations, like when the keyboard is displayed, which could greatly confuse the user.

Yeah sorry, guess I should of said this is for the Shy Bottom bar.  I had some situations where the user would filter results when the bar was still hidden and there was no way to get it to show back up.  This solved my problem by letting me tell the bottom bar to show itself if the results of the filter were empty.

I have the same problem - I do programmatical transition between tabs and I would like the bar to reappear on trasition if it was hidden

mBottomBar.animate().translationY(0)

@mStepkowski Great! thanks a lot.

+1

With the versions 2.3.0 and up, in shy mode, you can show / hide the bar as follows:
// Show:
bottomBar.getShySettings().showBar();

// Hide:
bottomBar.getShySettings().hideBar();

On non-shy mode, it's up to you how to hide / show it, since the BottomBar is just a regular View.


blog: Release v0.2

To start tracking what to share with readers, this change dumped the log
for the repositories being updated. This should help to shape the story
we're trying to tell.
While not a great start, I think it's worth commiting this precisely as
it is a start.

I have a script that maps Merge commit messages to the PR title which is far more useful in the changelog - I use it for generating the Node-RED Changelogs - https://github.com/node-red/node-red/releases/tag/2.2.1
By default it generates a list of all commits since the last tag.
This is from my .gitconfig
[alias]
  lasttag = !git describe --tags $(git rev-list --tags --max-count=1)
  changelog = "!f() { \
      tag=$1;\
      if [ -z $tag ]; then \
        tag=$(git lasttag);\
      fi;\
      echo $tag..HEAD;\
      git log --pretty=oneline --abbrev-commit $tag.. --first-parent --no-merges | sed -E 's/^.{8}/ -/g';\
      for x in `git log --pretty=oneline --abbrev-commit $tag.. --first-parent --merges | sed -E 's/^.*#//g;s/ from.*//g;/Merge branch/d'`; do\
         echo \" - \"`gh pr view $x --json title,number,author -t \"{{.title}} (#{{.number}}) @{{.author.login}}\"`;\
      done;\
    }; f"

It requires the gh command-line tool to be installed and authenticated with the API
For the flowforge/flowforge repo, it generates:
 -Email Docs (#205)
 - Update telemetry option wording & add std forge-link class (#304) @joepavitt
 - Update docs for telemetry feature (#303) @knolleary
 - Restore import following merge (#302) @knolleary
 - Fix path in installer zip (#293) @hardillb
 - Allow Admin to view all projects even if not a member (#297) @hardillb
 - Auto-accept pending invites when registering to the platform (#301) @knolleary
 - fix slug issue (#284) @sammachin
 - Fix 403 for node admin users creating projects (#292) @hardillb
 - Add link to project to download (#290) @hardillb
 - Docker docs (#265) @hardillb
 - Remove "itchy" from adjectives list (#276) @hardillb
 - Tidy up test close calls (#283) @knolleary
 - Add container driver shutdown hook (#281) @hardillb
 - Ensure tests close the app after each test to prevent hangs (#282) @knolleary
 - Add telemetry metrics component (#274) @knolleary
 - Fix startup error with uninitialised config (#267) @knolleary
 - Add migration framework for database (#263) @knolleary
 - Remove console.log from the storage library route (#262) @hardillb
 - Force teamId/ProjectId to a string (#261) @hardillb
 - Fix missing () on end of toString (#259) @hardillb
 - Update various deps to latest (#258) @knolleary
 - Fix build.yml to run a code build before test (#257) @knolleary
 - Refactor forge entry point to make it testable (#255) @knolleary
 - Allow changing the from address (#256) @hardillb
 - First pass at SES support (#254) @hardillb
 - Move initialisation of instanceId and add cookieSecret (#249) @knolleary
 - Move stub driver out of node_modules (#248) @knolleary
 - Introduce eslint standard and tidy up forge code base (#244) @knolleary
 - Check NodeJS version and log FF version (#243) @hardillb
 - Add engines section to package.json (#228) @hardillb
 - Add project automation workflow (#230) @knolleary
 - Update issues templates (#206) @knolleary
 - Fix install instructions (#201) @hardillb

Which makes me realise, we should have a CHANGELOG.md in each repo.


Synthesizer model in the case of a custom dataset

N.b. When performing Mel-Spectrogram to Audio synthesis, make sure Tacotron 2 and the Mel decoder were trained on the same mel-spectrogram representation.

Does this mean that I need to train separate models if I'm using a custom dataset? Or can I simply train Tacotron using this repo, and use the pretrained synthesizer to convert the mel's to audio? I'm currently experiencing a grinding sound in many cases. While some sentences synthesize decently, some are terribly poor. My dataset has approximately 18000 samples and totals around 30 hours of audio, and I'm 100000 steps into training, but training and validation loss has simply fluctuated around 0.36 for a long time, but quality does still seem to be gradually improving despite no change in validation loss.
I also keep getting this warning when synthesizing:
Warning! Reached max decoder steps

And each time this appears, the audio is terrible and the alignment is really bad. Every time this warning does not appear, the audio is as expected and the alignment is satisfactory.
Here's some examples of synthesized mels:
GOOD ALIGNMENT

BAD ALIGNMENT

As I mentioned previously, is this a product of using a synthesizer that was fine-tuned on a different dataset. If so, how can I fine-tune my own synthesizer?
It also seems to be certain sentences that always come out great, and others always come out poorly. It's most often sentences longer than 5-6 words where this happens most. But I can't see how this could be an overfitting problem considering I have a sufficient amount of data.

Nevermind, from a larger window of perspective, this issue changes significantly over the course of longer training. By 150,000 I'd say the problem is half as prominent as it was at 100,000.


Fix broken README code.

Fixes #412

I think we could bring back the Environment (in its new form) here, what do you think?

I think we should wait until #417 is merged before continuing.


Add app package to GitHub

I added a published EXE to GitHub, but I don't think I'll be including the AppPackage because of certificate issues.


Hybrid oil heatpump fixes

This PR

updates the 'related node' attribute for households_space_heater_crude_oil_hydrogen_heatpump_air_water_electricity in the corresponding input slider;
shortens the slider text for buildings_space_heater_crude_oil_hydrogen_heatpump_air_water_electricity
adds the correct NSA ETDataset public link for both buildings and households hybrid oil heat pumps

Updated by #4249 because of git issues.


Cannot login anymore

Hey,
I've been using the script for a while but for some reason I cannot login using the script anymore

@saisiddharth96 you should check #184

I believe that is a different issue, I'm getting "Failed to Login" error when i try to login even after a few hours

then you should follow issue reporting guideline because for me it works fine i cannot reproduce. it may be caused by some typo so do check that as well.

@saisiddharth96 here is POC where it works fine for me so i 'm closing this issue as invalid.


Feature/internal 3306

Implementation of most things mentioned in https://github.com/arangodb/planning/issues/3306

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/2428/

I don't think we should have "internal" options. IMHO the distinction between hidden-by-default and visible-by-default options is already enough.
Apart from that, we also have "obsolete" options, which are kept around only to keep older startup commands working. "obsolete" options will not affect program behavior otherwise, and we do not want to advertise them, so I think these should always be hidden and never exported, not even by --dump-options.
I am not sure if anything else needs to be done for this PR, or if you are happy with merging it. Please let me know.
Please also note that I marked all options that are added by enterprise features as enterprise options now. This was missing in previous versions. Now all options related to LDAP, encryption and one option regarding SSL are additionally marked as enterprise-only.

No internal options: fine with me
Obsolete options: yes, they should always be hidden indeed. Are they currently exported by --dump-options? If yes, can you change it to either not dump them (preferred), or add another attribute that indicates this (obsolete: true)?
LDAP etc.: hm, didn't notice that they were enterpriseOnly: false. Great that it is fixed! We should backport this to 3.3 to fix the docs there as well.
I haven't tested it yet, but feel free to merge the PR. I will do the doc generation scripts in a separate PR.

We are not showing obsolete options in --dump-options at the moment. Should this ever change, each option has an attribute "obsolete" already, which can then be used to filter out unwanted options from the results. Currently it will always contain "false".
Re hidden and non-hidden options: The decision which options are hidden and which aren't is partly arbitrary. The sheer amount of available options just requires that we only show the most important options by default. I think this is the most valid explanation, but I am not sure if that explanation needs to go into the documentation or not.
If you want to do something different depending on whether an option is hidden or not, --dump-options already provides a "hidden" attribute for each option.
Re the enterprise options: I can add the missing enterprise flags in 3.4 now. 3.3 doesn't have the enterprise/non-enterprise distinction at all.

We are not showing obsolete options in --dump-options at the moment. Should this ever change, each option has an attribute "obsolete" already, which can then be used to filter out unwanted options from the results. Currently it will always contain "false".
Re hidden and non-hidden options: The decision which options are hidden and which aren't is partly arbitrary. The sheer amount of available options just requires that we only show the most important options by default. I think this is the most valid explanation, but I am not sure if that explanation needs to go into the documentation or not.
If you want to do something different depending on whether an option is hidden or not, --dump-options already provides a "hidden" attribute for each option.
Re the enterprise options: I can add the missing enterprise flags in 3.4 now. 3.3 doesn't have the enterprise/non-enterprise distinction at all.

We are not showing obsolete options in --dump-options at the moment. Should this ever change, each option has an attribute "obsolete" already, which can then be used to filter out unwanted options from the results. Currently it will always contain "false".
Re hidden and non-hidden options: The decision which options are hidden and which aren't is partly arbitrary. The sheer amount of available options just requires that we only show the most important options by default. I think this is the most valid explanation, but I am not sure if that explanation needs to go into the documentation or not.
If you want to do something different depending on whether an option is hidden or not, --dump-options already provides a "hidden" attribute for each option.
Re the enterprise options: I can add the missing enterprise flags in 3.4 now. 3.3 doesn't have the enterprise/non-enterprise distinction at all.

@sleto-it I think we should not display whether an option is flagged as hidden in the docs, because its purpose is to not flood the shell if you run --help. WDYT?


TransformMany of ReactiveList use add / remove rather than replace

Although the PR appears to fix the issue, there was already an underlying issue with the way the operator handles Replaced items. It erroneously assumed that the original parent would contain the same items as the previous parent.

The original issue is now fixed. However I am unsure whether it will be possible deterministically apply a replace being as a parent replace does not necessarily mean that a child has been replace. @modplug I will leave this one for you to check out

This is an old one, transform many works. However dues to the nature of combining lists, replace is not possible


Fixed typos

Fixed typos within the CheckInView.

The storyboard changed significantly with the upgrade to Swift 2.0, so we'll fix typos on storyboard on master branch


DEV-86084 added command for apt public key and added --allow-releasei…

added command for apt public key and added --allow-releaseinfo-change flag for apt-get update command.

@ahsan13jan
The additional flag --allow-releaseinfo-change "Allows the update command to continue downloading data from a repository which changed its information of the release contained in the repository indicating e.g a new major release. APT will fail at the update command for such repositories until the change is confirmed to ensure the user is prepared for the change."
What happens here, is that the local apt database remembered your package-sources "release-information" to be "busterAsTesting", and when updating now "busterAsStable" is returned.
This results in the error, and the indication, that you need to "allow the change of the release version information".


Enable SA1008: Opening parenthesis should be spaced correctly

https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1008.md

Depends on #14468.


Cannot (re)Deploy Page: request failed with status code 400

Acknowledge the following

[x] I carefully read and followed the Getting Started guide.
[x] I read through FAQ and searched through the past issues, none of which addressed my issue.
[x] The issue I am raising is a potential bug in al-folio and not just a usage question.  [For usage questions, please post in the Discussions instead of raising an issue.]

Describe the bug
After the first successful commit and deployment, I would like to modify the website. However, on the newer deployments this happens:

{"message":"Deployment request failed for fdfafa4756e1fda18fb8c6dc80218065fc91b490 due to in progress deployment. Please cancel 33f692b0aac0e6da4aa66120ad5e61be425dd958 first or wait for it to complete.","documentation_url":"https://docs.github.com/rest/reference/repos#create-a-github-pages-deployment"}
Error: Error: Request failed with status code 400

33f692b0aac0e6da4aa66120ad5e61be425dd958 refers to the previous run.
You may find more details on the run here. However, the previous run(s) have been completed already. Even deleting them does not change the issue: it looks like Github considers that run still running.
To Reproduce
Steps to reproduce the behavior:

Deploy the page the first time
Edit your page and push modifications (note: I am manually re-deploying the website with $ ./bin/deploy
The error should appear

Additional context
This problem may be related to Github itself as this very recent Stackoverflow post suggests
Thanks in advance for your help 😄

thanks for reporting this! looks like a github actions issue. glad you were able to resolve it. closing the issue.


fix | Fix Failed Stacked Barchart

This ticket was used to fix the failed stacked barcharts issue experienced during deployment upon completion of the analytics page #703. Please see snapshot below:
Screenshots

Note to reviewers

Code of Conduct

[ ] By submitting this pull request, I confirm I've read and complied with the CoC 🖖

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

You can also join our Discord community.
Feel free to check out other cool repositories of the Thecyberworld.
Join the Thecyberworld GitHub Organisation by raising an issue (you will be sent an invitation).

It looks different on my screen. The problem is that the AnalyticsDoughnutChart is not in the center.

What browser do you use please?

I changed it because you don't see the problem in your screen so it is hard to adapt the code.

@duokobia Great work, thank you 👍


fix(Symbol.observable): properly defined as a unique symbol.

Resolves #5861
Resolves #4415
This is also related to issues here: https://github.com/benlesh/symbol-observable/issues/41
I don't think we can publish this in 6.x, as it will break people using older versions of @types/node, however, we do need to set an upper bounds for @types/node and version 6.x.

Added breaking change note to show in up in the CHANGELOG.


TAJO-1868: Allow TablespaceManager::get to return unregistered tables…

…pace.

@hyunsik, it looks that there is still a problem.
default> create external table customer (id int4) using text location<EMAIL_ADDRESS>ERROR: internal error: Not found Tablespace handler for<EMAIL_ADDRESS>

It may be because there is no entry for s3a in storage-default.json. Probably, s3n may has the same problem. I'll add more entries for them.

@hyunsik
I recommend default handler to FileTablespace, if a handler not found. What do you think?

@jinossy In my opinion, that would be not good because all arbitrary schemes will be handled by FileTablespace. For example, assume that users can mistype the uri like hdfx://host:port. If the default handler exists regardless of table uri, hdfx will be handled FileTablespace.
I think that the best way is to add all possible uri schemes to storage-default.json.

Hi @jinossy and @hyunsik, both points look reasonable to me.
Even though we use FileTablespace as the default handler when the handler is not found, Hadoop's FileSystem implementation will emit an error of No FileSystem for scheme: [uri_scheme].
In some sense, this error will be more useful for users than tablespace '/path/to/table' does not exist.
On the other hand, I'm also concerned with unexpected behaviours due to some other bugs if we don't handle exceptions about Tablespace by ourselves.
What do you think?

@jihoonson The error message was changed as follows: Not found Tablespace handler for ${uri}. It would look more reasonable error than that of before. If the problem is about error message, we can improve error messages.
In contrast, similar to your mention, if we use FileTablespace as  a default handler, it will make more unexpected cases because there are many code paths to handle the URI. It may make "out of control" against some error case.

@hyunsik,
I understood. if someone add custom filesystem with hdfs, they must be add tablespace handler
If we add a guide(link?) of adding tablespace in exception message, it would be better. I think

The case where someone develops custom file system rarely occurs in real field. That's too much.

Ok. Let's consider it again when it comes to a problem.
Anyway, how about changing CreateTableExecutor::getTablespaceHandler() to throw the UndefinedTablespaceHandlerException instead of UndefinedTablespaceException? This is not related to this patch, but related to the previous one.

Thank you for your comments. I've reflected your latest comment.

CreateTableExecutor::getTablespaceHandler() still throws UndefinedTablespaceException, but it is not necessary because methods called getTablespaceHandler() throws exceptions wrapped by TajoRuntimeException. For the sake of consistency, please remove it. (But, I'm not sure that throwing exceptions wrapped by TajoRuntimeException is a good convention.)
In addition, it would be great if you add any tests for UndefinedTablespaceHandlerException.

Fixed.

+1 the latest patch looks good to me. I left a trivial comment. If you agree, please reflect before commit.


Throw syntax exceptions on duplicate bindings

This commit updates the Smithy IDL parsing to throw when duplicate
bindings or members are found in the following declarations: service,
resource, and operation shapes, applied structure traits, and applied
traits with members that are structures. This behavior is unsafe for
model writers and shouldn't be valid.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Looks like some windows-based newline issues, will take a look.


[Bug]: proxy does not work

Error Message and Logs
I am getting the next error on the pages:
404
and this error in traefik:
"error: port is missing"
I did have a normal docker compose file yet
Steps to Reproduce

use docker compose file without any extra data
run the server with traefik

Example Repository URL
No response
Coolify Version
v4.0.0-beta.347
Are you using Coolify Cloud?
No (self-hosted)
Operating System and Version (self-hosted)
No response
Additional Information
No response

Could you please provide more detailed reproduction steps for this issue and a sample compose file? This will help us investigate and resolve the issue.


Error Xcode9:Swift4 using pod:@IBInspectable because its type cannot be

BoomMenuButton.swift:58:31: Property cannot be marked @IBInspectable because its type cannot be represented in Objective-C

Same issue here, any news? @Nightonke

@moonlsd i removed library and using below line of code to tableview.
let cells = tableview.visibleCells
let tableHeight: CGFloat = tableview.bounds.size.height
    for i in cells {
        let cell: UITableViewCell = i as UITableViewCell
        cell.transform = CGAffineTransform(translationX: 0, y: -tableHeight)
    }

    var index = 0

    for a in cells {
        let cell: UITableViewCell = a as UITableViewCell
        UIView.animate(withDuration: 1.0, delay: 0.05 * Double(index), usingSpringWithDamping: 0.5, initialSpringVelocity: 0, animations: {
            cell.transform = CGAffineTransform(translationX: 0, y: 0);
        }, completion: nil)

        index += 1
    }

I just removed @IBInspectable keyword from where that error appeared (it was twice for me). This will require you to unlock the VHBoomMenuButton library so you can make changes.


tar compress upgrade files

This is an attempt to make it so developers do not get confused by these
files showing up in greps, or even modifying them accidentally as part
of their PRs
Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

/test all

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
c2ab63ff096f11554ee7de65dfc91c8fe1d0a569
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
c2ab63ff096f11554ee7de65dfc91c8fe1d0a569
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Looks good to me. Few thoughts for the future:

We should probably also do something similar with the checked-in charts for the helm tests (think helm install can point to tgz?)
We should move these utils to a better place than multi_cluster_revision_test as more tests rely on them
We should automate the generation of these manifests

Looks good to me. Few thoughts for the future:

We should probably also do something similar with the checked-in charts for the helm tests (think helm install can point to tgz?)
We should move these utils to a better place than multi_cluster_revision_test as more tests rely on them
We should automate the generation of these manifests

Agree on all 3

Agree on all 3

I added the helm upgrade ones as well

I added the helm upgrade ones as well

@howardjohn: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@howardjohn: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Analytics is disabled in android

Hi
I have added the plugin to my IOS app and the analytics works fine, but when I run the android version all I can see in the logs are "Event not sent since app measurement is disabled".
If I add mFirebaseAnalytics.setAnalyticsCollectionEnabled(true); close to the mFirebaseAnalytics instantiation the android app will start sending analytics data, but according to documentation the analytics should be enabled by default.
It's also possible to make the plugin work by adding
<service android:enabled="true" android:exported="false" android:name="com.google.android.gms.measurement.AppMeasurementService" /> to the AndroidManifest.xml (link)
But it feels like I'm missing something here since I don't want to add any of these options in my app. Is something missing in the plugin?
Version used:
Cordova: 6.5.0
Android platform: 6.1.2
cordova-plugin-firebase: 0.1.21

its now fixed in master


Add simple instruction and tutorials to README.md

How to install
How to use
Some badges such as the latest release

Added at https://github.com/http4s/http4s-armeria/pull/10


Update dlp-chrome-get-started.md

tested with no admin rights and When set to NO it works

@chrfox Please merge this Pr. Thanks

checking with engineering


I have written some code for interfaces, slices, maps and higher orde…

Day 5 of #100DayOfCode challenge. This is time sitting down and going through the basic concepts of a language is feeling refreshing, often I'd just rush through to building things, and that would lead to a lot of confusion.
This again like the earlier PRs is my practice code, so if you're through it take it with a grain of salt.

The complete code wasn't added here xD
Adding it in PR #6


Custom config structs, auto provision watcher, debounced discovery

Signed-off-by: Anthony Casagrande<EMAIL_ADDRESS>PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-examples/blob/master/.github/CONTRIBUTING.md.
What is the current behavior?

Issue Number:
Closes #23
What is the new behavior?

add config watching functionality
register provision watchers on discovery (if needed)
add debounced discovery code from llrp service
convert toml to object type
add provision watcher file

Does this PR introduce a breaking change?

[ ] Yes
[X] No

New Imports
No
Specific Instructions
Are there any specific instructions or things that should be known prior to reviewing?
Other information

Majority of the code was migrated from device-rfid-llrp-go.
In regard to the default provision watchers, I came accross this inconsistency: #74

Codecov Report

Merging #75 (9e8214b) into main (ca3b9af) will increase coverage by 0.31%.
The diff coverage is 4.05%.

@@           Coverage Diff            @@
##            main     #75      +/-   ##
========================================
+ Coverage   1.80%   2.12%   +0.31%
========================================
  Files         12      12
  Lines       1162    1271     +109
========================================
+ Hits          21      27       +6
- Misses      1132    1235     +103
  Partials       9       9

Impacted Files
Coverage Δ

internal/driver/basenotificationconsumer.go
0.00% <0.00%> (ø)

internal/driver/driver.go
1.41% <0.00%> (-0.56%)
:arrow_down:

internal/driver/onvifclient.go
0.00% <0.00%> (ø)

internal/driver/onvifdiscovery.go
0.00% <0.00%> (ø)

internal/driver/pullpointmanager.go
0.00% <0.00%> (ø)

internal/driver/pullpointsubscriber.go
0.00% <0.00%> (ø)

internal/driver/config.go
17.64% <100.00%> (+17.64%)
: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 ca3b9af...9e8214b. Read the comment docs.


playground: "Format" button does not add missing import

visit https://play.golang.org/p/pKnQmVkLsw
make sure "Imports" checkbox is checked
click "Format"
observe no error message

I expected import log to be added; it is not.

Oh, nevermind. That's not the stdlib log package, it's trying to some some log.Warnf from a 3rd party library.


[AFK Messages] Incorrectly timed "You're no longer AFK!" message readout in chat

Describe the bug
The AFK overlay clears when you move your mouse around to change your character's view. However, the "You're no longer AFK!" message doesn't read out in chat until your character actually moves around.
EDIT: The AFK chat message also appears prior to the AFK overlay appearing.
To Reproduce

Go AFK (notice that the AFK chat message appears earlier than the overlay)
Move your mouse and see overlay clear
Walk around and see chat message appear

Expected behavior
I expected both things to occur from the same action
Screenshots
As a side note, and I realized these were placeholder messages, there are a couple grammatical and punctuation issues in the chat AFK messagess.

You're now marked as AFK.  (I know this one will get changed but it makes me chuckle)

You're no longer marked as AFK. You can now get legendary spawns!

They changed the message when you get AFK, but that has a typo ("no Legendary Pokemon will spawn on you. on you.") one "on you." to much

This is fixed as of next restart. Thanks for the report!


BUG: Instance scheduler fails when action is not supported

Expected behaviour
Large numbers of skipped instances belonging to an Autoscaling group should not trigger a message indicating that the number of skipped instances is too low.
Actual behaviour
main_int_test.go:46:
  Error Trace:	/home/runner/work/modernisation-platform-instance-scheduler/modernisation-platform-instance-scheduler/instance-scheduler/main_int_test.go:46
  Error:      	"20" is not greater than "20"
  Test:       	TestHandler/Test_request
  Messages:   	Number of skipped instances that belong to an Auto Scaling group seems too low. Please manually check the Instance Scheduler logs and verify this is reasonable. If it is reasonable, modify and adjust this test accordingly.

Steps to reproduce
See this github action for an example of the error ocurring: https://github.com/ministryofjustice/modernisation-platform-instance-scheduler/actions/runs/3984953166/jobs/6831747201

🆙 Update

🔧 Thresholds amended in https://github.com/ministryofjustice/modernisation-platform-instance-scheduler/pull/908 to reduce the chances of this happening again
🙈 This is more of a current "feature" than a bug 🐛 as we're currently depending on checks against the live environment using some static values to validate that code changes don't break anything. Discussed in a bit more detail in this thread 🧵
💅 In the long term, we want to move away from depending on the live state to validate code changes and shift our checks left by refactoring the code to make it more modular and easier to read, adding additional unit testing (currently only 68% of the functionality is covered by unit tests), integration tests with mocked environments instead of live ones etc.

👀 Remaining Work

Going to create a ticket to start addressing some of the technical debt issues, starting with getting unit test coverage up (which will hopefully encourage some refactors of the current code 🤞) - I won't create any more tickets than this for now, and we can decide to invest more time into this code if we feel it's worth the effort ✅


fix(bch-js): Updating to v6.6.0

:tada: This PR is included in version 2.5.2 :tada:
Your semantic-release bot :package::rocket:


cygwin: drop explicit O_TEXT

Cygwin binaries should not enforce text mode these days, just
use text mode if the underlying mount point requests it
Signed-off-by: Corinna Vinschen<EMAIL_ADDRESS>
Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Does that mean that the underlying system (Windows) doesn't use CRLF for line endings any more?
Also, what does "these days" mean?  Would this work on old Cygwin installations as well?

On Feb 15 11:41, Richard Levitte wrote:

Does that mean that the underlying system (Windows) doesn't use CRLF for line endings any more?

This is the Cygwin build, it should work as desired inside a Cygwin
installation in the first place.

Also, what does "these days" mean?  Would this work on old Cygwin installations as well?

It always worked and it will continue to work with old releases.
"These days" means, enforcing O_TEXt inside a tool has been
discouraged for Cygwin tools for a couple of years, and other
tools like sed, gawk, bash, don't do this anymore for quite some
time.
I never noticed that OpenSSL does this, too, only a recent mail on the
Cygwin mailing list notified me of this problem.  Otherwise I would have
sent this patch years ago :}
Thanks,
Corinna

This is the Cygwin build, it should work as desired inside a Cygwin installation in the first place.

So just oblige me for a moment here...  does this mean that Cygwin positions itself as a system that's entirely disconnected  from the underlying "host" (i.e. Windows), and therefore doesn't have to follow that system's text file conventions?
Basically, I'm worried about what happens if OpenSSL writes, say, a log without specifying O_TEXT, and then someone wants to read that log using a tool that expects CRLF as line ending (something as silly as the command type in a Windows command prompt).
I'll agree, though, that what constitutes a proper line ending on Windows seems to be a bit fuzzy these days, with most modern applications just accepting any convention equally...  is that what you're referring to?

Please note that I'm not saying no to this change, but I'd like to know the possible consequences

On Feb 15 05:28, Richard Levitte wrote:

This is the Cygwin build, it should work as desired inside a Cygwin installation in the first place.

So just oblige me for a moment here...  does this mean that Cygwin
positions itself as a system that's entirely disconnected  from the
underlying "host" (i.e. Windows), and therefore doesn't have to follow
that system's text file conventions?

Well, yes.  Cygwin's not following windows text file conventions
for pretty much since its beginnings.  The text mode was always
constituted by 2 different methods:

by specifing an explicit O_TEXT when opening files:
If you don't do that, the file is by default binary, using only
\n line breaks.

by mounting certain parts of your disks (entire disks or subdirs)
as text mode mounts:
If you do that, and the output is written to a file under this
mount point, the file is written O_TEXT by default, unless
O_BINARY is specified.

The Cygwin distro is O_BINARY by default.  Tools are discouraged
to use O_TEXT mode explicitely.  Rather, text mode should only be
used if the mount mode specifies it.
Note that using the default paths to files outside the Cygwin
installation directory (/cygdrive//...) are text mode
mounts by default.

Basically, I'm worried about what happens if OpenSSL writes, say, a
log without specifying O_TEXT, and then someone wants to read that
log using a tool that expects CRLF as line ending (something as silly
as the command type in a Windows command prompt).

You don't do that from within Cygwin.  There's cat, more or less ;)
Please don't overrate the usage of Cygwin OpenSSL in conjunction with
native Windows tools.  People using native Windows tools usually also
use native Windows tools linked against OpenSSL.  The major usage of
Cygwin OpenSSL is from within a Cygwin environment, and that's where it
should work most seamlessly.

I'll agree, though, that what constitutes a proper line ending on
Windows seems to be a bit fuzzy these days, with most modern
applications just accepting any convention equally...  is that what
you're referring to?

Not really, but most Windows tools already worked fine with \n
line endings anyway.  A major exception was perhaps notepad, but
even notepad is \n affine lately.
Corina

Note that using the default paths to files outside the Cygwin installation directory (/cygdrive//...) are text mode mounts by default.

So what happens with binary output if you write outside the Cygwin installation directory?
Other than that, message received, Cygwin is largely its own bubble, straying outside of it is essentially discouraged.  I can deal with that.

On Feb 15 06:19, Richard Levitte wrote:

Note that using the default paths to files outside the Cygwin
installation directory (/cygdrive//...) are text mode
mounts by default.

So what happens with binary output if you write outside the Cygwin
installation directory?

You mean explicit O_BINARY?  It's written binary :)
Corinna

Ah, gotcha.  So instead of binary vs text modes, we get binary vs "depends" modes.  Ok.

Oooh, I didn't notice the "need-cla" label before now.  This means that you will have to do one of the following:

If you consider this PR "trivial" from a copyright perspective, mark it as such by adding this in the commit message on a line of its own, separated from the rest with a blank line:
CLA: trivial

Sign a CLA (see https://www.openssl.org/policies/cla.html) and send it to us.  If you do this within the frame of your employer, we need a CCLA as well.

For formality: if you consider this PR trivial, I agree.

On Feb 15 07:55, Richard Levitte wrote:

For formality: if you consider this PR trivial, I agree.

Thanks, that was my impression as well :)
Corinna

Ping?  The 1.0.2 variant got approved and merged, perhaps this one should too?

Ping!  Low hanging fruit!

Hmmm....I don't think this is CLA trivial.

Oh my...  we disagree.  A point that may sway you is that #8249, which is the same mod for 1.0.2, got accepted as such.

IMO #8249 should not have been accepted and should be reverted (unless a CLA is submitted). Trivial submissions are for typos, grammar fixes etc. That can be extended to the equivalent in code. But this goes beyond that.

Don't let the copy-n-paste to get the platform variations separate distract from the fact this code change is indeed trivial in nature. It is simply avoiding O_TEXT. The actual changes that are made to the code are trivial even if the context diff looks a little larger than your usual comfort zone.
I'm comfortable that this was trivial and remains so on reconsideration.
However if you feel otherwise then we can vote to revert.

Don't let the copy-n-paste to get the platform variations separate distract from the fact this code change is indeed trivial in nature. It is simply avoiding O_TEXT. The actual changes that are made to the code are trivial even if the context diff looks a little larger than your usual comfort zone.

I'm comfortable that this was trivial and remains so on reconsideration.
However if you feel otherwise then we can vote to revert.

I do feel otherwise. I strongly feel we should be risk averse in what we determine to be trivial or not. If there is any doubt then we should not accept something under the "trivial" exception. There is sufficient doubt in my mind on this change that I don't think it should be accepted.
However, I am hoping that @github-cygwin will submit a CLA anyway which would make the disagreement moot and remove the need for a vote on a revert.

CLA received thank @github-cygwin!

Merged.
master:
9b57e4a1ef356420367d843f1ba96037f88316b8 cygwin: drop explicit O_TEXT
1.1.1:
a81cc6e8a240b27d0e0e6404dd29970375a4f5fc cygwin: drop explicit O_TEXT


set env variables

Hi, thanks for the great work!!
My Fritz!Box is however not reachable on its default address so it would be very practical if the next values could be set through docker "ENV" variables:
	flag.StringVar(&settings.FritzBox.IP, "gateway-address", "fritz.box", "The hostname or IP of the FRITZ!Box")
	flag.IntVar(&settings.FritzBox.Port, "gateway-port", 49000, "The port of the FRITZ!Box UPnP service")
	flag.StringVar(&settings.FritzBox.UserName, "username", "", "The user for the FRITZ!Box UPnP service")
	flag.StringVar(&settings.FritzBox.Password, "password", "", "The password for the FRITZ!Box UPnP service")

I understand it's already possible to pass config via ENV ?
https://github.com/mxschmitt/fritzbox_exporter/blob/master/cmd/exporter/exporter.go#L341

Good point, yep indeed. It should.


Feature Request: Configure Table Aliases for generic tables all at once

I've got a structure like
type PersistedState<'T> =
  { Id: Guid
    State: 'T }

that I'm persisting to marten to represent some temporal state for a variety of folds over different types of incoming events, but I'd like all of the states to be persisted to the same underlying table. Marten specializes each usage of the generic type to generate the tablename, but I'd like to be able to say something like _.Schema.For<PersistedState<'t>>().DocumentAlias("some_stable_name") so that all versions of the table (unless manually overridden) would use the same set of storage configuration.  Right now I have a helper method to specialize the type and generate the alias, but I have to remember to do that for each new kind of state I introduce into my application, which can be error-prone.

which, now that I'm actually running my code, doesn't work at all because Marten expects uniqueness across all document aliases :(

No, just no:)


Perform first-time setup

The goal here is to make it usable on a fresh CI server without manual intervention. Would appreciate if anyone already had something to quickly setup a completely fresh VM, maybe @danieltomlinson or @pietbrauer?

[ ] Enable developer-mode
[ ] Accept license

Developer mode enabling should be:
sudo -S /usr/sbin/DevToolsSecurity -enable
/usr/sbin/dseditgroup -o edit -t group -a staff _developer

This should handle accepting the license: https://github.com/magneticbear/Jenkins_Bootstrap/blob/master/accept_xcode_license.exp

Decided against auto-accepting the license in lieu of KISS. Instead, the license prompt thing will be shown if needed.


shielding/unshielding private tx example

story:

Alice sends native DOT tokens to SubstraTEE-worker’s account
Alice sends a shielding request to the worker, including a proof-of-inclusion and her incognito account
The worker enclave issues incognito-DOTs on Alice’s incognito account
Alice verifies her incognito balance (direct authenticated websocket request to worker)
Alice sends an unshielding request to the worker, stating the beneficiaries’ public DOT account
if Alice has sufficient funds, the worker decreases Alice’s incognito balance and issues an extrinsic sending these funds from the worker’s collateral account to the beneficiary.

dockerize this example to dockerhub it will become the flagship demo!
dependencies:

we need shard-specific enclave accounts for custody of bonded native DOT tokens. private key to these must be provisioned to new worker enclaves

function that creates an encrypted msg containing TrustedCall::balance_unshield() : https://github.com/scs/substraTEE-worker/blob/54d7afa225fe7bca57eb7c647328bc5849351c75/worker/src/tests/commons.rs#L59
and it is called as follows: https://github.com/scs/substraTEE-worker/blob/54d7afa225fe7bca57eb7c647328bc5849351c75/worker/src/tests/integration_tests.rs#L77

however, in the example dummy mrenclave and ShardIdentifier need to be replaced by the real ones

shielding/unshielding example implemented and tested (client/demo_shielding_unshielding.sh).
Closing issue


Some minor GLFW window init changes to be nicer and to catch error cases

Thank you for this! I totally forgot to handle window creation errors because I was rushing to get a workable prototype completed 😟


SpdyHttpEncoder fails to convert HttpResponse to SpdyFrame

Netty Version: Latest master (f1eddd6117f5e3109257f911d51b80250e17e11d)
When SpdyHttpEncoder attempts to create an SpdyHeadersFrame from a HttpResponse an IllegalArgumentException is thrown if the original HttpResponse contains a header that includes uppercase characters.  The IllegalArgumentException is thrown due to the additional validation check introduced by #4047.
Previous versions of the SPDY codec would handle this by converting the HTTP header name to lowercase before adding the header to the SpdyHeadersFrame (see here).
Not the greatest solution, but this could be 'fixed' by lowercasing the header name here and here
If you guys are comfortable with this solution (or have a better one), I would be more than happy to create a PR.
/cc @Scottmitch @jpinner - for obvious reasons :smiley:

@blucas - Sorry for breaking you :(. I guess we have at least 2 options:

Add a constructor to DefaultSpdyHeaders which takes a boolean validateHeaders argument which can make header name validation optional.  We would then need to carry this through the SPDY codec to pass along the boolean where ever the DefaultSpdyHeaders are created.
Your suggestion, plus a boolean convertHeaderNamesToLowerCase (or something similar) to control whether conversion should be done or not.

Feel free to submit a PR for one/both of the above :)

@Scottmitch - no problem, it happens to all of us :).
I have made the necessary changes for 1. above (although I've followed the way it was done for HTTP such that if the validateHeaders is set to true (default), header names and values will be validated)
Do we really need to add a boolean convertHeaderNamesToLowerCase to SpdyHttpEncoder? We don't do this for HTTP to HTTP/2 conversion (see here).  I think we should just do the same.

Do we really need to add a boolean convertHeaderNamesToLowerCase to SpdyHttpEncoder?

It is not strictly necessary. Just a potential fast path or optimization if your environment is controlled.  Feel free to leave it out...we can always add it later.

Fixed by https://github.com/netty/netty/pull/4190

Just a heads up: SpdyHttpHeaders.Names are still uppercase, is that correct?

@CodingFabian - Not that is a bug. Fixed with https://github.com/netty/netty/pull/4226


drivers: ieee802154_nrf5: fix condition for mac keys and frame counter

For multicore devices like nRF53, shim layer part of setting mac keys and frame counters is called on application core where
CONFIG_NRF_802154_ENCRYPTION is disabled (the define concerns radio driver for net core).
This commit replaces CONFIG_NRF_802154_ENCRYPTION with CONFIG_IEEE802154_2015.

Is there any dependency between IEEE802154_2015 and NRF_802154_ENCRYPTION configuration flags? I understand this patch fixes the issue for dual-core devices running ieee802154_nrf5.c on other core than the radio driver itself. But I would like to make sure it does not create a problem for single core devices.

Currently we do no have such dependency.

Currently we do no have such dependency

Does it mean it's possible to misconfigure single-core device so that ieee802154_nrf5.c expects the radio driver to encrypt the frame (IEEE802154_2015=y) while the driver itself does not support encryption (NRF_802154_ENCRYPTION=n)?

Currently we do no have such dependency

Does it mean it's possible to misconfigure single-core device so that ieee802154_nrf5.c expects the radio driver to encrypt the frame (IEEE802154_2015=y) while the driver itself does not support encryption (NRF_802154_ENCRYPTION=n)?

It seems so.

Does it mean it's possible to misconfigure single-core device so that ieee802154_nrf5.c expects the radio driver to encrypt the frame (IEEE802154_2015=y) while the driver itself does not support encryption (NRF_802154_ENCRYPTION=n)?

It seems so.

By fixing one issue you should not introduce bugs in other configurations. This patch should be modified to work correctly on both single- and dual-core configurations.

This is what it does it. With this patch in current form both configuration are correct.
Little context: currently all required options are enabled by one overlay. The plan for near future is to have one primary option which selects other required. We do not want to have here options with strict dependencies which are difficult for handling where the radio is on the different core.

currently all required options are enabled by one overlay.

I understand this patch works correctly for selected applications or samples providing configuration overlays like the one you described.  But someone might be using this driver in an out-of-tree application without such overlay. This patch could break such application.

We do not want to have here options with strict dependencies

I don't  think we need to create a dependency. But we need to make sure that configuration options are aligned between 802.15.4 driver build and ieee802154_nrf.c build. Dependency would be one way to solve it, but there are other available.

I understand this patch works correctly for selected applications or samples providing configuration overlays like the one you described. But someone might be using this driver in an out-of-tree application without such overlay. This patch could break such application.

This is very doubtful assumption that someone is using this quite new feature which were provided for Thread and it's not using Thread but some other protocol. But let say you are right in this, what do you suggest? Frankly speaking any other solution which comes to my mind seems much worse than we have right now.

But let say you are right in this, what do you suggest?

My point is that with this patch it's very easy to misconfigure an application (existing or a new one). I understand we should not babysit developers, but we should make project configuration user friendly.
My understanding of the configuration options we are discussing is like this:

IEEE802154_2015 is a system wide configuration option stating that 2015 version frames are supported
NRF_802154_ENCRYPTION_ENABLED is an option related to nRF 802.15.4 radio driver enabling frame encryption in this driver.

In single-core applications the code responsible for managing radio diriver cryptography shall be enabled only when radio driver has encryption enabled (NRF_802154_ENCRYPTION_ENABLED).
The question is how to enable this code in dual- core projects where the radio driver is replaced with a serialization library.
The simplest solution that comes to me is to use NRF_802154_ENCRYPTION_ENABLED with serialization library to enable serializing crypto-related operations. Another option would be stating that NRF_802154_ENCRYPTION_ENABLED should be enabled by default when version 2015 is supported and guard crypto-related code with IEEE802154_2015. Another option is to forcefully select NRF_802154_ENCRYPTION_ENABLED when nRK 802.15.4 driver is present and version 2015 is supported. Probably other solutions are available.
But guarding implementation of a function with one configuration option while usage of this function is guarded by an unrelated option is not a user-friendly configuration design.

My point is that with this patch it's very easy to misconfigure an application (existing or a new one). I understand we should not babysit developers, but we should make project configuration user friendly.
My understanding of the configuration options we are discussing is like this:
* IEEE802154_2015 is a system wide configuration option stating that 2015 version frames are supported

* NRF_802154_ENCRYPTION_ENABLED is an option related to nRF 802.15.4 radio driver enabling frame encryption in this driver.

In single-core applications the code responsible for managing radio diriver cryptography shall be enabled only when radio driver has encryption enabled (NRF_802154_ENCRYPTION_ENABLED). The question is how to enable this code in dual- core projects where the radio driver is replaced with a serialization library. The simplest solution that comes to me is to use NRF_802154_ENCRYPTION_ENABLED with serialization library to enable serializing crypto-related operations. Another option would be stating that NRF_802154_ENCRYPTION_ENABLED should be enabled by default when version 2015 is supported and guard crypto-related code with IEEE802154_2015. Another option is to forcefully select NRF_802154_ENCRYPTION_ENABLED when nRK 802.15.4 driver is present and version 2015 is supported. Probably other solutions are available.
But guarding implementation of a function with one configuration option while usage of this function is guarded by an unrelated option is not a user-friendly configuration design.

I think you too oversimplify the problem. For dual-core application the image on radio core is build separately which imply that we don't know if the configuration is right until run time. The only option to verify on the early stage if will work is to call get_capabilities from radio and check expected result. But this PR is not for fixing existing problems (we already can misconfigure application even single-core application) or future one (how to make sure that dual-core application is configured properly). This PR is to fix the wrong conditions for dual-core application without breaking sinle-core application. And the PR does it. Take notice that to work properly an single-core application already needs to enabled both flags: CONFIG_NRF_802154_ENCRYPTION and CONFIG_IEEE802154_2015 so nothing will change.

I disagree. Currently it is fine to enable CONFIG_NRF_802154_ENCRYPTION with CONFIG_IEEE802154_2015 disabled and use the encryption feature for 802.15.4-2006 frames. With your patch is not possible anymore.

This PR is to fix the wrong conditions for dual-core application without breaking sinle-core application.

This PR is working around the problem in dual-core application by providing incorrect configuration guards in ieee802154_nrf5.c. I'm not OK with working around problems taking shortcuts leading to other problems. If we cannot create a workaround for dual-core which does not cause problems for single-core, maybe proper fix should be implemented instead?


chore: add a11y plugin to storybook

SUMMARY:
chore: add a11y plugin to storybook
GITHUB ISSUE (Open Source Contributors)
JIRA TASK (Eightfold Employees Only):
CHANGE TYPE:

[ ] Bugfix Pull Request
[ ] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@aa1d6ca). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #319   +/-   ##
=======================================
  Coverage        ?   88.50%
=======================================
  Files           ?      400
  Lines           ?    10757
  Branches        ?     3637
=======================================
  Hits            ?     9521
  Misses          ?     1213
  Partials        ?       23

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.


4785 mvp order integration

https://app.clubhouse.io/augur/story/4785/integration-validation-market-order-form

Market Order stuff, Going to close the other PR for now.

@stephensprinkle @justinbarry I'm going ahead and merging this b/c it's blocking.  It's a big enough PR that it would still benefit from your review though (especially the view layer stuff).

@JohnDanz @JohnDanz has this been vetted for 'id' case change in seadragon?


Travis tests sometimes don't pass due to travis slowness

It seems sometimes the travis tests fail because not enough time is given for the test to converge.
Should we increase the threshold? @andrewmzhang @yangryan0

I think that's a good quick fix.
To fix it more completely maybe we should sum up the total number of updates and compare that to the loss. Although this would require investigating some sort of updates to loss relationships.

That's too complex. I suggest we relax the threshold.

Yes I think increasing the threshold should work. I added that small change
to the softmax model PR I added this weekend, since I didn't think I needed
to create another PR for such a small change.
On Tue, Jul 31, 2018, 9:51 AM João Carreira<EMAIL_ADDRESS>wrote:

That's too complex. I suggest we relax the threshold.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/jcarreira/cirrus/issues/71#issuecomment-409290745,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Abwst2XSknNeVpVZ3DnaFVD38zjYGBhVks5uMIsGgaJpZM4VlOby
.

Ok, that sounds good to me.

Can we close this?


[FTX] No Implementation of AccountService .getFundingHistory

Hi All,
There is no implementation of getFundingHistory for the AccountService of FTX.
The idea is to implementent it using :

FtxAccountServiceRaw.transferBetweenFtxSubAccount  (implemented)
/wallet/deposits (not implemented) see : https://docs.ftx.com/#get-deposit-history
/wallet/withdrawals (not implemented) see : https://docs.ftx.com/#get-withdrawal-history

And then to differentiate between the two types using
FundingRecord.Type.INTERNAL_DEPOSIT or FundingRecord.Type.DEPOSIT (and the same for WITHDRAWALS)
What do you think ?
I'm working on it and I will submit a PR when done ...

Good practices in this project is to use only one API call per operations. If not possible, please take care of rate limits by using resilience.
Originally posted by @cymp in https://github.com/knowm/XChange/issues/4195#issuecomment-891256829

Submit a PR and we can discuss.

Well anyway, I'm gonna split this into so this will be solved ...
The call to getFundingHistory on the main account should use /wallet/deposits and /wallet/withdrawals
And the call to getFundingHistory for a subaccount should use the transferts between subaccounts ..
It's the correct way I think


#17 Feature: configure initial frame size

https://github.com/MoyuruAizawa/Cropify/issues/17
Implement frameSize Cropify option: to allow configuring frame initial size or keep the same frame ratio.
This is a breaking change because the old aspect ratio variable is gone and developers should pass the new FixedAspectRatio to frameSize instead
The way I did it allows either setting the frame size or the frame aspect ratio, but can't mix both, as mixing both can be a bit hard and has many edge cases
I renamed AspectRatio to FixedAspectRatio to indicate that it's not just an initial value but a fixed value that won't allow the user to change it
Feel free to comment on any of the classes/attributes naming I can then change them
also if you think PercentageSize and FixedAspectRatio should be accessed in an easier way than doing CropifySize.FixedAspectRatio let me know, but I did it this way so devs can easily know what options they have of CropifySize, and no need to dig into the code to find out.
*Note: I didn't update the CropifyOptionSelector as this is more of an "initial value" but let me know if I should add it

LGTM. Thanks!


Knitr improvements

Improve knitr engine so that when working interactively it no longer prints two outputs under table/image output chunk.
It should know should singular outputs for each chunk.

Codecov Report

Merging #23 (159da56) into main (2c427b8) will decrease coverage by 0.10%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
- Coverage   11.06%   10.97%   -0.10%
==========================================
  Files          22       22
  Lines        2675     2698      +23
==========================================
  Hits          296      296
- Misses       2379     2402      +23

Impacted Files
Coverage Δ

R/knitr-engines.R
0.00% <0.00%> (ø)

R/package-auth.R
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Feat/build speed

Simplified base Dockerfile used during Kaniko build and added kaniko build args to Chassis service resulting in significantly faster model image builds

Huge shout out to Saumil for all of the code, docs, tutorials, and ideas that @saumil-d has added to chassis! Let's be sure to add him for those contributions cc @all-contributors

@all-contributors please add @saumil-d for code, docs, tutorials, ideas


Tensorboard generating javascript errors: Failed to load resource: net::ERR_CONNECTION_RESET

I'm running tensorboard on the Windows 10 Bash on Ubuntu shell. After bringing up the tensorboard webserver (localhost:6006) with tensorboard --logdir=/mnt/d/Dropbox/Andor/Summaries --debug and connecting to it with my Chrome browser, I don't see anything on the screen. The JS console gives me a couple of the following ERRORS:
Failed to load resource: net::ERR_CONNECTION_RESET http://localhost:6006/external/plottable/plottable.min.js
Failed to load resource: net::ERR_CONNECTION_RESET http://localhost:6006/external/polymer/polymer.html
Failed to load resource: net::ERR_CONNECTION_RESET http://localhost:6006/dist/tf-tensorboard.html
Failed to load resource: net::ERR_CONNECTION_RESET http://localhost:6006/external/iron-icons/iron-icons.html

Except for http://localhost:6006/dist/tf-tensorboard.html, the other three missing resources are all there in the correct directories (/usr/local/lib/python2.7/dist-packages/external/...). The /usr/local/lib/python2.7/dist-packages/dist/ directory does not exist in my installation.
After these errors in the JS console come hundreds of Uncaught ReferenceError: Polymer is not defined
I have searched on the web for related issues but only found this one issue which I believe is unrelated as I don't get any of the described errors in my case. In fact, there are no errors on the server's console output, no warnings, all ok (I'm running the server in debug mode).
Other parameters of my system:

I did a pip install of tensorflow

Also:
python -c "import tensorflow; print(tensorflow.__version__)
0.8.0

A small event file is attached here:
events.out.tfevents.1474905790.zip
Also, the server's console output is here (after starting the server AND making a request in the browser http://localhost:6006):
User@PC_001:/mnt/d/Dropbox/Andor$ tensorboard --logdir=/mnt/d/Dropbox/Andor/Summaries --debug
INFO:tensorflow:TensorBoard is in debug mode.
INFO:tensorflow:Starting TensorBoard in directory /mnt/d/Dropbox/Andor
INFO:tensorflow:TensorBoard path_to_run is: {'/mnt/d/Dropbox/Andor/Summaries': None}
INFO:tensorflow:Adding events from directory /mnt/d/Dropbox/Andor/Summaries/train
INFO:tensorflow:Constructing EventAccumulator for /mnt/d/Dropbox/Andor/Summaries/train
DEBUG:tensorflow:Opening a record reader pointing at /mnt/d/Dropbox/Andor/Summaries/train/events.out.tfevents.1474905790.DUCANDUGMBH_001
DEBUG:tensorflow:No more events in /mnt/d/Dropbox/Andor/Summaries/train/events.out.tfevents.1474905790.DUCANDUGMBH_001
INFO:tensorflow:No path found after /mnt/d/Dropbox/Andor/Summaries/train/events.out.tfevents.1474905790.DUCANDUGMBH_001
DEBUG:tensorflow:No more events in /mnt/d/Dropbox/Andor/Summaries/train/events.out.tfevents.1474905790.DUCANDUGMBH_001
INFO:tensorflow:No path found after /mnt/d/Dropbox/Andor/Summaries/train/events.out.tfevents.1474905790.DUCANDUGMBH_001
INFO:tensorflow:Multiplexer done loading.  It's apparently the browser (Chrome), which has an expired certificate.
Tensorboard on Windows 10 (bash on ubuntu) works well for me now with the Firefox browser.
One more unrelated thing, though:
The resource: web-animations-next-lite.min.js.map in the /usr/local/lib/python3.4/dist-packages/external/web_animations_js directory seems to be missing in the Windows python 3.4 pip install of tensorflow (I used the https://storage.googleapis.com/tensorflow/linux/cpu/tensorflow-0.8.0-cp34-cp34m-linux_x86_64.whl). I had to manually generate this one file from GitHub. Only after that, all 404 errors disappeared.

@sven1977 Glad you worked things out!  I'll suggest that you upgrade to the latest version of TensorFlow, to pick up all the bugfixes we've added since 0.8:
https://www.tensorflow.org/versions/r0.11/get_started/os_setup.html#download-and-setup
I'm closing this out, but feel free to comment or file a new bug if you have other issues.


Good Work :D

We need stuff like this in the JS build ecosystem so keep up the good work :D
I'm building systemjs-tools which is in a similar space but for the SystemJS ecosystem.

Thanks for the props! Let me know if you have any suggestions about how to improve Backpack

@jaredpalmer Didn't want to clutter you with a new issue, but seriously, good work. This looks great.
Especially thanks so much for making a default configuration. One of the most annoying things about Webpack and Babel is the fact that you must configure it every time.


Cannot read property 'map' of undefined when running initialize()

Hi,
I was able to make react-redux-localize work and it does so for as long as I navigate my React single page app from the navigation links(using React-Router 4). I do my initialization from the Homepage component and everything works fine when navigating from that component to Login but if I hit re-load on the browser from the SignIn component, the whole thing breaks because I basically loose the whole state on Redux, it goes back to empty.  In order to solve this issue I thought of passing the languages array and options as props to my SignIn  component and using mapDispatchToProps to have an init function that dispatches a call to 'initialize' with the languages array and some options as parameters in the case the 'activeLanguage' is undefined.  That I thought would restore the state in Redux and would get my page to load fine with translations and everything if the user hit the reload button on the browser.  The issue I'm having is that I'm getting the following error:

I'm doing the initialize() from the Home component.

It's hard to say for sure without seeing a working example, but you need to ensure you are calling initialize before attempting to access any translation data. In your case I would recommend calling initialize from App.js to ensure that your translations are initialized before any specific page is accessed.
Also I recommend actually initializing by passing your data to LocaizeProvider's initialize prop, which is the required format when dealing with server side rendering. This will most likely be the preferred method for initialization in future versions.

I moved the initialize call out of home.js to App.js, which is also where I have my router but it doesn't work correctly, here's my App.js:
import React from 'react';
import { BrowserRouter as Router, Switch, Route } from 'react-router-dom';
import store from './store/store';
import { Provider } from "react-redux";
import { LocalizeProvider } from 'react-localize-redux';
import { renderToStaticMarkup } from 'react-dom/server';
import { initialize } from 'react-localize-redux';
import { withLocalize } from 'react-localize-redux';
import Favicon from 'react-favicon';
import NavBar from './components/common/NavBar/SomeNavBar';
import Footer from './components/common/Footer/SomeFooter';
import HomePage from './Pages/Home/home';
import Login from './Pages/Login/index';
import CreateNewUser from './Pages/Login/index';

class App extends React.Component{
  constructor(props){
    super(props);
  }

  componentWillMount(){
    let defaultLanguage = 'en';
    let languages = [
      { name: 'English', code: 'en' },
      { name: 'Spanish', code: 'es' }
    ];
    let options = { renderToStaticMarkup };

    this.setState({
      defaultLanguage: defaultLanguage,
      languages: languages,
      options: options
    }, () => {
      initialize({
        languages: this.state.languages,
        options: this.state.options
      });
    })
  }

  render(){
    return(
      <Provider store={store}>
        <LocalizeProvider store={store}>
          <Favicon url="/img/favicon.ico" />
          <Router>
            <React.Fragment>
              <NavBar />
              <Switch>
                <Route exact path="/" component={ () => <HomePage lang={this.state.defaultLanguage}  /> } />
                <Route exact path="/signin/:lang" render={ (props) => <Login  {...props} /> } />
                <Route exact path="/createnew/:lang" render={ (props) => <CreateNewUser  {...props} /> } />
              </Switch>
            </React.Fragment>
          </Router>
          <Footer />
        </LocalizeProvider>
      </Provider>
      );
  }
 }

 export default withLocalize(App);

And here's a screenshot of what I get for the homepage on the browser:

Note: If I do initialize() from the home.js component, it works just fine, even when going to other routes(components) for as long as it is using the links in the homepage itself. If I try typing a route to a different component in the browser url it breaks. When I do initialize() from App,js, the languages array and options never get loaded to the store so none of the pages get rendered with translations as you can see in the screenshot above.

I made it work adding the initialize to the LocalizeProvier component:
<LocalizeProvider store={store} initialize={{ languages: languages, options: options}}>

After adding:
<LocalizeProvider store={store} initialize={{ languages: languages, options: options}}>
The route loads and the page gets all of its structure, labels, textboxes and the translations works fine but it craps out at some point and the rest of the styling doesn't get loaded so the page renders with none of the styling and images.  I get this error on the console:

This issue does not appear to be related to the original issue. To avoid muddying up this thread for future users I'm going to close this. If you feel there is a new issue please open a new issue, and please follow the issue template.
In addition looking at the console errors it doesn't appear to be related to anything specific to this library. The errors are stemming from jquery.min.js and bootstrap.min.js. The remaining issues appear to be lint issues.


Wrong Span Information with first-line-indent.

Description
The offset in the source span is wrong for the glyphs of a paragraph with first-line-indent.
The 3 Glyphs for the first "ABC" are correct, for the second "ABC" the offsets are off by one.
#set par(first-line-indent: 1em)

ABC

ABC

"ABC"
            0: Glyph { id: 36, x_advance: 0.69482421875em, x_offset: 0.0em, range: 0..1, span: (Span(114812161553020), 0) }
            1: Glyph { id: 37, x_advance: 0.587890625em, x_offset: 0.0em, range: 1..2, span: (Span(114812161553020), 1) }
            2: Glyph { id: 38, x_advance: 0.64599609375em, x_offset: 0.0em, range: 2..3, span: (Span(114812161553020), 2) }
"ABC"
            0: Glyph { id: 36, x_advance: 0.69482421875em, x_offset: 0.0em, range: 0..1, span: (Span(129626634011474), 1) }
            1: Glyph { id: 37, x_advance: 0.587890625em, x_offset: 0.0em, range: 1..2, span: (Span(129626634011474), 2) }
            2: Glyph { id: 38, x_advance: 0.64599609375em, x_offset: 0.0em, range: 2..3, span: (Span(1), 0) }

Reproduction URL
No response
Operating system
Windows
Typst version

[X] I am using the latest version of Typst

There's a good chance this is fixed on main. I remember fixing something like this a while back but probably after the last release.

Found it: https://github.com/typst/typst/pull/4301
Based on this, I'll close this but let me know if you can still reproduce it on main.


connection.release dosen't work

I use the connectionPool,and set the connectionLimit to 10,and I write connection.release()  in the callback of connection.query. While the count of the sleep thread of mysql  is more than 10 upto 50 .And destroy   can solve the problem but it will spend more time to create new connection.So how do I solve the problem.

Hi @shuaishenk weird indeed. We have a lot of tests for the release and it seems to work, so in general not sure what the issue would be. Can you share the following?

Version of Node.js you're using
Version of this module you're using.
Complete code that will reproduce the issue we can run
Instructions for how to run the code and how to validate the issue is occurring

Thanks!

@dougwilson I have found the problem of my code.And It works well now.
Thank you~


Migrate from Jenkins to Prow

2 Jobs needs to be created
mixerclient-presubmit
mixerclient-proxy-regression

Bumping to P0 as Jenkins is in unfixable state.

Can this be closed?

I only see presubmit


Having trouble with ruby-kickstart challenge 1:4

I know there is probably a prettier way to do this but I'm still not sure why this doesn't work.
def grade(num_books, has_read_books)
if num_books < 10 && has_read_books != true
"D"
elsif num_books <= 10 && has_read_books == true
"C"
elsif num_books (10..20) && has_read_books != true
"C"
elsif num_books (10..20) && has_read_books == true
"B"
elsif num_books > 20 && has_read_books != true
"B"
else
num_books > 20 && has_read_books != true
"A"
end
end

The problem might be the second statement, where the condition should be < 10 not <= 10.

helped by @iryna replaced  "num_books (10..20)" with "num_books>=10 && num_books<=20" and it worked


Make SafeMath a library

Implements idea from @izqui 's post https://blog.aragon.one/library-driven-development-in-solidity-2bebcaf88736
What do you guys think?
This changes modify the API, so next release should be minor, not patch

I think this is a great idea!
Just my two wei here, I personally like this API better for the calls, instead of '.safeAdd', etc. I think seeing that a function will be called it could be easily implied that the code is going to execute the library. And I think the '.plus', etc are prettier and more expressive.

Also a quick question, what is the gas overhead for performing a delegatecall for every safe operation vs doing it internally?

We should put the discussion into context. Currently SafeMath is implemented as a contract which must be inherited from in order to use the safe operations. The proposed change is to make it a library.
Like @izqui says in his blog post, libraries aren't allowed to inherit, but they can use other libraries. This implies that SafeMath has to be a library if it is going to be used as part of another library. Since we want to encourage the use of safe math in general, including in the development of other libraries, making SafeMath itself a library seems necessary.
This has consequences for the API because library functions are used different from contract functions. They can be used in two different ways. The first one is as functions by prefixing them with the library name. (Assume x, y are of type uint.)
using SafeMath;
uint z = SafeMath.add(x, y);

The second one is attaching the functions as methods to a type.
using SafeMath for uint;
uint z = x.add(y);

Seeing the two examples above I'd be inclined to remove the safe prefix from the functions, since in the first case it's redundant. However it may not be explicit enough in the second case that add is a safe operation.
Regarding add vs plus, etc., I think I prefer the former.
I don't know about the gas gost of delegatecall. Wouldn't the solidity compiler be able to compile it away to a simple call anyway? Since they are pure functions.

I don't know why I though the gas cost was different, the last version of the yellowpaper seems to express that C (σ, μ) if w=CALL ∨ CALLCODE ∨ DELEGATECALL which I understand that the three have the same gas cost and which is 700 gas.
If confirmed, 700 gas for a delegatecall seems cheap to me a definitely not a deal breaker. Can anyone confirm?
Source: http://yellowpaper.io

@izqui your analysis seems correct.
@frangio I implemented your proposed names.
If everyone agrees, I'm for merging this PR as is and turning SafeMath into a proper lib