Documentation of the scraping use case

Document the use case as a build on from #13

done in #34


GH-38033: [R] Allow code() to return package name prefix.

Rationale for this change
#38033
What changes are included in this PR?

Added get_pkg_ns() helper.
Added call_name private method to DataType class to store the string name used in the code call. Refactored code() public method to use call_name.
Converted all $code() call(...) to $code(namespace = FALSE) call2(..., .ns = if(namespace) "arrow") in DataType, Schema, and DictionaryType.
Added code to Schema docstring.
Updated expect_code_roundtrip to pass namespace and ... arguments.

Are these changes tested?

Implemented new tests for code() for (almost?) all types
Added test to Schema$code() to also cover case of namespace=TRUE

Are there any user-facing changes?
Yes, user-facing changes, but no breaking changes to any public APIs.

Closes: #38033

@paleolimbot @thisisnic I believe the PR is ready for final review -- it's passing all my tests, including the new ones. Two notes/questions:

map_of does not do a roundtrip because it's represented as list_of(struct(key, value)) instead of map_of(key, value). Should I change that in this PR?
explicit_pkg_name seems like a very long argument name -- any ideas of how to shorten? Perhaps use_ns or with_ns (bool) to match the .ns argument (string) in call2?

Testing on e0c655f:
test-schema.R
[ FAIL 0 | WARN 0 | SKIP 0 | PASS 84 ]

test-type.R
[ FAIL 0 | WARN 0 | SKIP 1 | PASS 245 ]

── Skipped tests (1) ──────────────────────────────────────────
• Work around masking of data type functions (ARROW-12322)
  (1): test-type.R:116:3

Ran all tests on c0a80a9
[ FAIL 0 | WARN 8 | SKIP 32 | PASS 8149 ]

The CI job which is failing is the one which runs the linter.  If you run styler:::style_active_file() on the files that have changed, this will fix it.

@thisisnic This is my first PR for a major project like this, so I appreciate your patience with me!
I had totally missed that the tests in test-data-type.R already covered $code(), and per your recommendation I was able to cover almost all the new code by reverting all my changes to testing and adding some changes for namespace=TRUE. Let me know if you think that's stretching too far beyond what's appropriate for this PR.


New valid JSON templates throw 422 Unprocessable Entity error

Hi! I've been using this library for years now (thank you) and unfortunately, for some reason, cannot get it to work when trying to upload new Online Store 2.0 JSON template files. It always results in a 422 Unprocessable Entity error.
{
  "layout": "fall-winter-2021",
  "sections": {
    "main": {
      "type": "hero-big-headline"
    }
  },
  "order": [
    "main"
  ]
}

The layout file also exists, the section file hero-big-headline exists, and also saving the same code via the theme's code editor saves just fine. Is it because this lib is expecting these files to be liquid files? Any idea how to spit out the exact error that is causing the error to be triggered? Thanks for any help you might be able to offer.

Not sure, sorry! It's been a while since I've done any theme dev.
422 Unprocessable Entity is Shopify's API response as-is, I would imagine the issue may be:

A problem in the file's schema
A liquid template with the same name as the .json file
anything else that's triggering Shopify's API to complain

Did you try uploading that file manually via their Theme editor to see if it works or if you get any clearer errors?

Gotcha, that's unfortunate, saving the json file directly via the theme's code editor throws no errors and I am able to edit and modify it's main sections + add my custom modular sections + reorder them around with no problems.
I'm thinking after so many years using this dev workflow, the time has come to stop using it and adopt Shopify's new git CLI for dev environments / deployment.

Ha! It’s probably time. Didn’t even know they released new tools for that. I created this back then precisely because Shopify’s tools were so poor, and I hope the new tools make this package unnecessary.
If anything I would like to update this workflow to be less dependent from a specific tool chain these days.

i was pretty excited when they announced it during their Unite conference recently.
Updates to dev workflow was just one of many exciting things that are now available including sections in all pages including product pages, not just the homepage.
https://shopify.dev/themes/tools/cli


Clean up error handling

We had a call to _app.UseExceptionHandler("/Error"); in our startup code, but there was no page that would actually handle that. That would, in various configurations, lead you to either a 404 page or to the login page.
Note that this only really happens for errors that occur outside of the normal operation of the site. For most exceptions, they'd be handled via the blazor circuit and you'd get the yellow bar in the browser. So you're only really likely to see this happen for more catastrophic errors when the site isn't functional at all (though it's not strictly limited to that).
This PR cleans up the code that sets that up ( _app.UseDeveloperExceptionPage() is already called for development environments when using WebApplication.CreateBuilder()) and adds a page that actually responds to that route.
Since the page renders outside of the normal circuit and we don't know how much else is broken when the error occurs, the error page does not have the main layout and more closely resembles the login page. Otherwise it is essentially the standard error page in the blazor template. We can add more in the future, but it seemed of primary importance to just fix the misconfiguration and have a page that responded to the route at all.

Microsoft Reviewers: Open in CodeFlow

/backport to release/8.0


Porting to python3

I have been a very happy user of send.py for many years now. Thanks a lot for making this available. After offlineimap moved to python 3, I am trying to eliminate dependence on python 2 completely.
How hard would it be to port send.py to python 3?  If you think that it is only a question of changing some python 2 libraries to equivalent python 3 libraries, I could try doing so myself and submit a pull request. If you think it will take more effort, I will stick to python 2 for some more time.

That's great to hear, glad to see you have gotten a lot of use out of this little tool.
I think if you'd like to try to port to Python 3, you should give it a shot!  I believe it should be mostly the same code, with just some following tweaks:

Imports for some packages likely need to change
There is some string/byte encoding that will likely need to be addressed

Just about every port I've seen for Python 2 ends up doing these things.  If you get stuck, just let me know!  I can also aim to do this over the holiday

These were the changes that I made to make send.py work in python 3
-from ConfigParser import ConfigParser
+from configparser import ConfigParser
-import urllib
+import urllib.request
+import urllib.parse
+import urllib.error
-    response = urllib.urlopen(oauth.request_url,
-                              urllib.urlencode(params)).read()
+    response = urllib.request.urlopen(
+        oauth.request_url,
+        urllib.parse.urlencode(params).encode("utf-8")).read()
-    auth_string = base64.b64encode(auth_string)
+    auth_string = base64.b64encode(auth_string.encode("utf-8")).decode("utf-8")

Perhaps import urllib.error is not needed but 2to3 suggested it and I was too lazy to check whether it is needed.
I am not submitting a proper pull request with a patch because my copy of send.py has some other changes that would make a patch file useless.

Thanks!  I have merged this in.


Drop support for Python 3.6

Closes #86

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 4 (75.0%)  changed or added relevant lines in 2 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 90.851%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sphinx_toolbox/more_autodoc/typevars.py
1
2
50.0%

Files with Coverage Reduction
New Missed Lines
%

sphinx_toolbox/more_autodoc/typevars.py
2
91.09%

sphinx_toolbox/more_autodoc/typehints.py
4
83.48%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
3843

Relevant Lines:
4230

💛  - Coveralls


Update default value for druid.emitter in Configuration.md

I'm not 100% sure, but looking at the source code (EmitterModule.java) and my experience with a clean druid 0.7 cluster, I believe the default value for druid.emitter to be noop instead of logging, as I was getting no emitted logs at all when not specifying druid.emitter in any of the runtime.properties, despite having defined a metric monitor.

Hi @mrijke, great catch!
Thanks for the contrib! We have a slightly new CLA available at: http://druid.io/community/cla.html, do you mind signing that?

OK, signed! :)


FlushCommit should return successfully finished commits even when some commits are cancelled

Imagine the following provenance structure:
A -> B -> C -> D

Currently if C is cancelled, when you FlushCommit(A), you get an error saying that C is cancelled.  Ideally, you should also be notified that A and B have been successfully finished.
Or this structure (fan-out):
  A
/ | \
B C D

Again, if C is cancelled and you FlushCommit(A), you are only told that C is cancelled.  Ideally you should also be told that B and D have been successfully finished.

Also, if there are more than one cancelled commits in the fan-out case, we should report all the cancelled commits.

Also, it should block until all consequences have finished: in the event of a combination of some successful commits and some errors leading to cancelled commits, it should not return immediately upon encountering an error but should block until everything has flushed.

Definitely understand the use case here and think it's a good thing to address.
I have some questions about the implementation though, afaik it's not possible to return an error and a response from a grpc call. It'll only serialize one of those things. So it seems like the only way to get FlushCommit to behave like this would be to make it never error. That would definitely cause problems with some of the ways we're using FlushCommit internally and also seems like it might be a slightly weird set of semantics. I'm wondering if maybe this issue should be about introducing a different API call that does what we want rather than about extending FlushCommit. Or maybe we can just add a flag to FlushCommit that does what we want. Interested to hear what others think.

For FlushCommit to error when there's a downstream error seems to make sense. However, if it's not a fatal error with regards to executing the flush (like a network error), then it shouldn't return with the error until all downstream consequencies have terminated, whether they are successful or not. In other words, if it can flush, it should flush.
As far as how to discover the failure(s), that could be through the same API (maybe with a flag that lets you either return only successful commits, or return all commits in order look through them to discover their statuses) or could be through a different API.

@brendanacassidy thanks for clarifying, very helpful. I think what you've said totally makes sense. Flush returning should indicate that everything that's going to run due to the commit set has. That fits well with the semantics we already have and shouldn't be too hard to incorporate.

I'm going to jump in here and say that it might make sense to separate errors which occur in the pipeline from errors that occur during the flush. Leave the error value for GRPC for cases when the flush call itself fails due to network issues or the commit not existing....etc. Have the actual response contain two sets of data one of successful commits and another of failed commits and their associated errors.  This allows for differentiated errors with pipeline and other errors as well as provides a mechanism for displaying more that one error in the pipelines.

I believe this is solved w the upcoming 1.7 model. flush-commit blocks until commits are finished, and even failed jobs will finish commits.


Make repo compatible with the webui

We can add a python script under scripts/mecha_extension.py which would serve as the entry point for the webui extension. I outline here what I think we want to do, but I'm open to any suggestion really.
Must have:

select a .mecha recipe in the checkpoints dropdown to load a recipe instead of a model. The recipe is merged on the spot. This allows to save a lot of disk space while also making it very easy to explore highly complex alternative model weights composition

Should have:

create recipes from within the webui and run them. I'm not sure what interface to use for this, maybe we can just have a textbox in which we hand-craft them

Yeah I think the biggest boost for testing would be being able to go through various recipes in batches and even iterate through them with something like an XYZ and see the results of those experiments immediately. That would already be halfway to unlocking not just executing already existing ideas but go into exploration territory.
 Imagine being able to do a shitload of merges, more than you could manually put together, and then being able to visually confirm the results without having to manually load and gen and unload them throughout the process, and without your gpu or ram breaking a sweat.
And maybe in the future Mecha could even do the exploration part with something like this
Who knows. I have suggestions but no knowledge to explain the technicals and I can't help implement them, which makes suggesting things feel a bit awkward. But I like everything so far and I'm thankful for what you e made.


Added unittests for pkg/routes/common/org_id.go.

Description
Added unittests for pkg/routes/common/org_id.go.

Fixes # (issue)
Type of change
What is it?

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[ ] Refactor

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] I run make pre-commit to check fmt/vet/lint/test-no-fdo

By the way, it doesn't look like the function GetAccountOrOrgIDFromContext is used anywhere. How about deleting it?

From 0% to 86.96%

Codecov Report
Base: 44.34% // Head: 44.55% // Increases project coverage by +0.20% :tada:

Coverage data is based on head (3618626) compared to base (7e7a794).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1742      +/-   ##
==========================================
+ Coverage   44.34%   44.55%   +0.20%
==========================================
  Files          70       70
  Lines        9810     9811       +1
==========================================
+ Hits         4350     4371      +21
+ Misses       5024     5005      -19
+ Partials      436      435       -1

Impacted Files
Coverage Δ

pkg/routes/common/org_id.go
81.25% <ø> (+81.25%)
:arrow_up:

pkg/services/images_iso/main.go
58.65% <0.00%> (+0.32%)
:arrow_up:

pkg/services/devices.go
50.43% <0.00%> (+0.32%)
:arrow_up:

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

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

/lgtm

@ldjebran yoohoo :)

@mgold1234  yoohoo

/retest


Reload data every trial

Why is the data reloaded every Trial，This is a waste of time

@Jie-Yuan Could you provide more detailed description, for example, what data, which part reloaded the data?

@Jie-Yuan Could you provide more detailed description, for example, what data, which part reloaded the data?

Copyright (c) Microsoft Corporation
All rights reserved.

MIT License

Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated
documentation files (the "Software"), to deal in the Software without restriction, including without limitation
the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and
to permit persons to whom the Software is furnished to do so, subject to the following conditions:
The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.

 IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
import nni
from sklearn.model_selection import train_test_split
from sklearn.datasets import load_digits
from sklearn.preprocessing import StandardScaler
from sklearn.svm import SVC
import logging
import numpy as np
import time
LOG = logging.getLogger('sklearn_classification')
def load_data():
'''Load dataset, use 20newsgroups dataset'''
digits = load_digits()
X_train, X_test, y_train, y_test = train_test_split(
digits.data, digits.target, random_state=99, test_size=0.25)
ss = StandardScaler()
X_train = ss.fit_transform(X_train)
X_test = ss.transform(X_test)

LOG.debug("load_data!!!")

return X_train, X_test, y_train, y_test

def get_default_parameters():
'''get default parameters'''
params = {
'C': 1.0,
'kernel': 'linear',
'degree': 3,
'gamma': 0.01,
'coef0': 0.01
}
return params
def get_model(PARAMS):
'''Get model according to parameters'''
model = SVC()
model.C = PARAMS.get('C')
model.kernel = PARAMS.get('kernel')
model.degree = PARAMS.get('degree')
model.gamma = PARAMS.get('gamma')
model.coef0 = PARAMS.get('coef0')
return model

def run(X_train, X_test, y_train, y_test, model):
'''Train model and predict result'''
model.fit(X_train, y_train)
score = model.score(X_test, y_test)
LOG.debug('score: %s' % score)
nni.report_final_result(score)
if name == 'main':
X_train, X_test, y_train, y_test = load_data()
try:
    # get parameters from tuner
    RECEIVED_PARAMS = nni.get_next_parameter()
    LOG.debug(RECEIVED_PARAMS)
    PARAMS = get_default_parameters()
    PARAMS.update(RECEIVED_PARAMS)
    LOG.debug(PARAMS)
    model = get_model(PARAMS)
    run(X_train, X_test, y_train, y_test, model)
except Exception as exception:
    LOG.exception(exception)
    raise

@Jie-Yuan yes, every trial is an independent job. But NNI allows users to load data only once (especially when data loading is very time consuming), please refer to multi-phase for the usage.

@Jie-Yuan  please reopen if you still have issue on this.


Update stream-analytics-scale-jobs.md

Hi @pablissima, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (paulettm). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Docker tagged action failing

We wished to use the tagged version of the plugins but the current action that publishes them is not working.

@l-lafin this was actually a bug in the HSP Docker Registry which was only solved recently. Tagged builds are working again.


fix: README to work with sheldon

I added an example of sheldon.
ref: https://github.com/odanado/dotfiles/blob/5eed207ed221b8289afa2714d83d798065dca1ff/config/sheldon/plugins.toml#L50-L51

Thanks


hallucination in the T2S stage.

Does the autoregressive decoding of the T2S stage induce random hallucinated results such as repeated words/phrases or long silences? how does this relate to the reported WER results?

#17 related.


Fixing T1574.013 TIDs in excel sheet

I did not see the develop branch being active. Feel free to use the updated excel sheet and close this PR.
Description of what has changed

Fixing KernelCallbackTable TIDs to T1574.013 on excel sheet for nist800-r5 mappings. It was referenced with the following TIDs instead T1574.014, T1574.015, T1574.016, T1574.017, T1574.018, T1574.019

Thank you for bringing this to our attention. Updates have been merged.
Tiffany


4DGS on pedestrian within urban scenes

Hi, I'm currently applying 4D Gaussian Splatting to reconstruct pedestrians within an urban scene. However, I'm encountering issues with accurately reconstructing certain parts of the pedestrians, particularly the feet, hands, and legs of some individuals. These regions, which involve faster movement, tend to appear more blurry or distorted in the reconstruction results.
Does anyone have suggestions or advice on how to improve the reconstruction quality for these fast-moving parts? Any insights or recommendations would be greatly appreciated.
Thanks in advance!

Hi, @AutonomierLong.
Very sorry for the late reply. I think reconstructing pedestrians with subtle movements from few observations in driving scene videos is a little bit ill-posed. According to our experience, it is difficult to really build association between pedestrians across different frames, without appropriate priors as constraint. OmniRe seems to focus on this issue, their approach might be helpful for you.


🚀[Release v2.13.0] Merge into Main

Overview
This release has many bugfixes and includes remote config and tech debt retirements
Release Notes
Bugfixes

7 Crashlytics retirements

Tech debt

7 Maintenance and code improvements

Features

Remote configuration capability

LGTM 🎸 merci beaucoup!

Pas de quoi messeur!

Awesome, thank you! @kcw-grunt

You folks are awesome!

So proud of our progress @andhikayuana  and @josikie 🔥🔥 🔥 🔥 🔥 🔥


[BUG]: ERROR message recived white move file to workspace.

How are you running AnythingLLM?
AnythingLLM desktop app
What happened?

when i move file to workspace ,it pop up failture everytime,it says "Error:1 documents failed to add fetch failed"
, no matter i upload a txt file or a docx file. it happens everytime, how to sovle this? or there are specifications for file format?
(windows11 desktop app)
i tried create new workspace but no use.same error.
my LLM preference is LMStudio,Embedding Preference is AnythingLLM Embedder
Are there known steps to reproduce?
No response

https://github.com/Mintplex-Labs/anything-llm/issues/821#issuecomment-1968217337
Your IP is likely blocked by hugging face due to sanctions, VPN, or something else and you cannot download the native embedder model. The above comment shows how to fix that.

#821 (comment)
Your IP is likely blocked by hugging face due to sanctions, VPN, or something else and you cannot download the native embedder model. The above comment shows how to fix that.

thank you problem solved!

#821 (comment)
Your IP is likely blocked by hugging face due to sanctions, VPN, or something else and you cannot download the native embedder model. The above comment shows how to fix that.

thank you problem solved!

Could you please tell me how you solved the problem, as I am also using a Windows 11 system and downloaded all the files from that website link into AppData\Roaming\anythingllm-desktop\storage\models\Xenova\all-MiniLM-L6--v2 directory but failed to make it work even after restarting.

Make sure all of the files are readable by the system. Sometimes they can be downloaded but unable to be read/executed and that makes them broken as well


Adding the option of holding the Command key to add a point

Adding the option of holding the Command key on the Mac to add a point. Because Ctrl + click simulate right click.

Thank you for your help!


Please assign to me a program in Java that returns the number of digits of a given number

pls assign me this issue @dark-coder-cat


wishlist: SVG support

Currently knob is based on canvas. the knob will likely become blurry when resizing the knob by the 3rd party JQUERY plugin (e.g. freetrans).
If Knob can be based on SVG, it can be perfectly resized to any scale.

+1


Jumping to the Next Change does not work when Neovim plugin is turned on

How to reproduce:

Install/Enable vscode-neovim
Change a file in several places in a git repository
"Open Changes"
Click "Next Change"

Result: the cursor either jumps to the first change but never jumps to the next or simply does not jump at all.

I found that it works fine only in insert mode.

@alanxmay, nice workaround.
Thank you!

I have faced a similar problem.
I believe this problem started around 0.0.92. I'm now using 0.0.91 and do not have this problem.

This might be fixed by https://github.com/vscode-neovim/vscode-neovim/pull/1258

I'm still reproducing the issue in normal mode with the latest releases of VSCode & VSCode Neovim. It does work as expect in insert mode (but this isn't very convenient for code reviews).


Swift Package Manager support

Hi! We're evaluating BoxCast and I'm wondering if you would integrate Swift Package Manager as an integration option?
I can take a shot at the Package.swift manifest and open a PR if that's helpful?
Thank you!

Hi @toemat! Sorry for the delayed response.  I'd love if you could open a PR; we're certainly open to switching to Swift Package Manager!

Hey @toemat thanks for the PR and update.  Your addition was merged into master with only one small change. Reset channelId back to the default string for consistency sake. Otherwise, your addition works as expected. Thanks for contributing!


Updating to <IP_ADDRESS> causes ReferenceError: Buffer is not defined

When I updated from <IP_ADDRESS> to <IP_ADDRESS> I starting to get the error. I tried to replicate with an repo but I couldn't.
I'm using the 0.2.3 version, but I tried different ones.
As soon as my app finish to update to the latest version of Meteor and the meteor core apps, it stopped working. If I switch branches (for the ones that are not in the latest version) or downgrade the Meteor app, everything works.
Would you know what's going on or how can I solve/help with this issue?

It does make sense and I think it's worth to save 22kb. I'll take a look, but as far as I remember we don't need Buffer in the client side, only server side.

https://github.com/meteor/meteor/issues/8645#issuecomment-305218365

@benjamn Isn't meteor-node-stubs a core package? So you remove Buffer while using it in a core package which is added on every new Meteor project (https://github.com/meteor/guide/issues/314) by default. Why do you not add it automatically in the node-stubs package, so if I don't need node-stubs, Buffer will be also removed.

@dnish I definitely agree the Buffer polyfill should go away when nothing that uses it is being imported. I think the problem here is that the crypto polyfill assumes a global Buffer constructor exists, when it should be using require("buffer").Buffer instead, to be safe.

This should be fixed now, since I forked the problematic packages and dealt with the Buffer references. Let me know how it goes!


lib_2/html/js_interop_1_test fails on ddk-linux-release-firefox

Likely introduced in https://dart-review.googlesource.com/c/sdk/+/139822. Fails with a runtime error, stack trace is incomplete:
Deobfuscated error and stack:
<no error message found>
    at window.onerror called:  unparsed
    at <fn>                    ../../../../../dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 147:26
    at <fn>                    Error

Looked into it a bit, seems like there's a failure when running/at the start of the listener, but couldn't get much more info due to it being ran in a Zone.


Sprockets::DoubleLinkError: Multiple files with the same output path cannot be linked ("application.js")

I got this issue after add gem 'jsbundling-rails', '~> 0.2.2' to my application
rake aborted!
Sprockets::DoubleLinkError: Multiple files with the same output path cannot be linked ("application.js")
In "/Users/xx/xxx/app/assets/config/manifest.js" these files were linked:

/Users/xx/xxx/app/assets/javascripts/application.js
/Users/xx/xxx/app/assets/builds/application.js

Tasks: TOP => assets:precompile
(See full trace by running task with --trace)
Please advise how can I fix this

You have to rename one of the two files. They're both being generated at the same path which doesn't work. Normally if you use builds/application.js you don't also need app/assets/javascripts/application.js.


[Task]: When received funds from faucet, mana is not displayed in tx history until logout-login

Task description
When received funds from faucet, mana is not displayed in tx history until logout-login

after logout and login

looks like the issue is because the first tx details is generated by a tx derived from an output, instead of being generated by the transactions returned by the SDK


Snippets fail to run on macOS

Description
Running Snippets either via swift run <snippet> or swift package learn crashes with library not loaded error. This appears to work fine on Linux.
Expected behavior
Expected snippet to run
Actual behavior
It prints the error
dyld[48638]: Library not loaded: @rpath/libXCTestSwiftSupport.dylib
  Referenced from: <1701C4F4-A6E2-3E29-8B1C-34E728CC1B14> /Users/adamfowler/Developer/server/hummingbird/hummingbird/.build/x86_64-apple-macosx/debug/print
  Reason: tried: '/usr/lib/swift/libXCTestSwiftSupport.dylib' (no such file, not in dyld cache), '/System/Volumes/Preboot/Cryptexes/OS/usr/lib/swift/libXCTestSwiftSupport.dylib' (no such file), '/Users/adamfowler/Developer/server/hummingbird/hummingbird/.build/x86_64-apple-macosx/debug/libXCTestSwiftSupport.dylib' (no such file), '/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift-5.5/macosx/libXCTestSwiftSupport.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift-5.5/macosx/libXCTestSwiftSupport.dylib' (no such file), '/usr/lib/swift/libXCTestSwiftSupport.dylib' (no such file, not in dyld cache), '/System/Volumes/Preboot/Cryptexes/OS/usr/lib/swift/libXCTestSwiftSupport.dylib' (no such file), '/Users/adamfowler/Developer/server/hummingbird/hummingbird/.build/x86_64-apple-macosx/debug/libXCTestSwiftSupport.dylib' (no such file), '/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift-5.5/macosx/libXCTestSwiftSupport.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift-5.5/macosx/libXCTestSwiftSupport.dylib' (no such file<EMAIL_ADDRESS>(no such file), '/usr/lib/swift/libXCTestSwiftSupport.dylib' (no such file, not in dyld cache), '/System/Volumes/Preboot/Cryptexes/OS/usr/lib/swift/libXCTestSwiftSupport.dylib' (no such file), '/Users/adamfowler/Developer/server/hummingbird/hummingbird/.build/x86_64-apple-macosx/debug/libXCTestSwiftSupport.dylib' (no such file), '/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift-5.5/macosx/libXCTestSwiftSupport.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift-5.5/macosx/libXCTestSwiftSupport.dylib' (no such file), '/usr/lib/swift/libXCTestSwiftSupport.dylib' (no such file, not in dyld cache), '/System/Volumes/Preboot/Cryptexes/OS/usr/lib/swift/libXCTestSwiftSupport.dylib' (no such file), '/Users/adamfowler/Developer/server/hummingbird/hummingbird/.build/x86_64-apple-macosx/debug/libXCTestSwiftSupport.dylib' (no such file), '/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift-5.5/macosx/libXCTestSwiftSupport.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift-5.5/macosx/libXCTestSwiftSupport.dylib' (no such file), '/usr/local/lib/libXCTestSwiftSupport.dylib' (no such file), '/usr/lib/libXCTestSwiftSupport.dylib' (no such file, not in dyld cache)
zsh: abort      swift package learn

Steps to reproduce

Add snippet to package
Run swift package learn, select Snippet and run
Or swift run <snippet name>

Swift Package Manager version/commit hash
No response
Swift & OS version (output of swift --version && uname -a)
swift-driver version: 1.62.15 Apple Swift version 5.7.1 (swiftlang-<IP_ADDRESS>.3 clang-14<IP_ADDRESS>)
Target: x86_64-apple-macosx13.0
Darwin Adams-iMac-Pro-2.local 22.1.0 Darwin Kernel Version 22.1.0: Sun Oct  9 20:14:54 PDT 2022; root:xnu-8792.41.9~2/RELEASE_X86_64 x86_64

If the snippet happens to link XCTest, this looks expected to me. Usage of XCTest implies setting DYLD_LIBRARY_PATH on macOS since its location is always relative to Xcode/SDK.
cc @bitjammer

The snippet itself doesn't link XCTest but one of the targets in the Package does. Is there anyway around that?

Just to confirm, you are using the Swift compiler bundled with a releaed Xcode, correct?
I believe I just fixed this issue a few days ago in https://github.com/apple/swift-package-manager/pull/5929, so should be resolved in the next minor Xcode release.

Yes I am using swift 5.7.1 that came with Xcode.

Okay, that’s good to know. Without seeing anything more with your package, I believe a snippet is linking to a test support library unnecessarily. As long as that library isn’t a package product, as they usually aren’t, then I expect this to be fixed in the next minor release. Thanks for the report!

I verified on a separate project that snippets work fine as long as none of your targets link in XCTest


CoreHttpProvider[sendRequestInternal] - 424Error during http request

These are the scopes i am providing in authenticationHelper class in android.
private String[] mScopes = { "User.Read", "Calendars.Read", "User.ReadWrite", "User.ReadWrite.All", "OnlineMeetings.Read", "OnlineMeetings.ReadWrite"};
This what i am getting from token.
"scp": "AccessReview.Read.All AccessReview.ReadWrite.All Application.Read.All Application.ReadWrite.All Directory.AccessAsUser.All Directory.Read.All Directory.ReadWrite.All Files.Read Files.Read.All Mail.Read Mail.ReadBasic Mail.ReadWrite OnlineMeetings.Read OnlineMeetings.ReadWrite openid Organization.Read.All Organization.ReadWrite.All OrgContact.Read.All profile Tasks.ReadWrite User.Read User.Read.All User.ReadBasic.All User.ReadWrite User.ReadWrite.All email",
In Azure AD , these are the api permissions i have added.

I am able to update users from graph explorer.

But While using below code in android giving error.
User user = new User();
LinkedList businessPhonesList = new LinkedList();
businessPhonesList.add("79XXXXXX94");
user.businessPhones = businessPhonesList;
user.officeLocation = "Bangalore";
    mClient.me()
            .buildRequest()
            .patch(user);

implementation 'com.microsoft.identity.client:msal:1.4.0'
implementation 'com.microsoft.graph:microsoft-graph:1.7.1'
I am getting this error while calling above method.
2020-09-29 13:14:13.361 16702-16702/com.XXX.XXXXX E/global: CoreHttpProvider[sendRequestInternal] - 424Error during http request
2020-09-29 13:14:13.361 16702-16702/com.XXX.XXXXX E/global: Throwable detail: com.microsoft.graph.core.ClientException: Error during http request
2020-09-29 13:14:13.362 16702-16702/com.XXX.XXXXX E/AndroidRuntime: FATAL EXCEPTION: main
Process: com.XXX.XXXXX, PID: 16702
com.microsoft.graph.core.ClientException: Error during http request
at com.microsoft.graph.http.CoreHttpProvider.sendRequestInternal(CoreHttpProvider.java:422)
at com.microsoft.graph.http.CoreHttpProvider.send(CoreHttpProvider.java:204)
at com.microsoft.graph.http.CoreHttpProvider.send(CoreHttpProvider.java:184)
at com.microsoft.graph.http.BaseRequest.send(BaseRequest.java:306)
at com.microsoft.graph.requests.extensions.UserRequest.patch(UserRequest.java:91)
at com.phtl.teamsapp.GraphHelper.updateUser(GraphHelper.java:66)
at com.phtl.teamsapp.CalendarFragment$1.onSuccess(CalendarFragment.java:48)

I reproduce this in Graph Explorer as well. I don't believe users can update officeLocation. That information is considered to be "owned" by the company, not the user. At least that's how it was explained to me by the identity team :D.
See https://docs.microsoft.com/graph/permissions-reference?context=graph%2Fapi%2Fbeta&view=graph-rest-beta#remarks-29

User.ReadWrite and User.Readwrite.All delegated permissions allow the app to update the following profile properties for work or school accounts:

aboutMe
birthday
hireDate
interests
mobilePhone
mySite
pastProjects
photo
preferredName
responsibilities
schools
skills


Error when running turtlebot3 bringup

Hi, following the steps in bringup a turtlebot3, I am currently stuck with this error:
pi@raspberrypi:~ $ roslaunch turtlebot3_bringup turtlebot3_robot.launch
... logging to /home/pi/.ros/log/bb0b8944-dc40-11e8-b6d2-48e24421dfbd/roslaunch-raspberrypi-1073.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.
started roslaunch server http://<IP_ADDRESS>:42113/
SUMMARY
PARAMETERS

/rosdistro: kinetic
/rosversion: 1.12.13
/turtlebot3_core/baud: 115200
/turtlebot3_core/port: /dev/ttyACM0
/turtlebot3_core/tf_prefix:
/turtlebot3_lds/frame_id: base_scan
/turtlebot3_lds/port: /dev/ttyUSB0

NODES
/
turtlebot3_core (rosserial_python/serial_node.py)
turtlebot3_diagnostics (turtlebot3_bringup/turtlebot3_diagnostics)
turtlebot3_lds (hls_lfcd_lds_driver/hlds_laser_publisher)
ROS_MASTER_URI=http://<IP_ADDRESS>:11311
process[turtlebot3_core-1]: started with pid [1082]
process[turtlebot3_lds-2]: started with pid [1083]
process[turtlebot3_diagnostics-3]: started with pid [1084]
[ERROR] [1540906375.858127004]: An exception was thrown: open: No such file or directory
[turtlebot3_lds-2] process has died [pid 1083, exit code 255, cmd /home/pi/catkin_ws/devel/lib/hls_lfcd_lds_driver/hlds_laser_publisher __name:=turtlebot3_lds __log:=/home/pi/.ros/log/bb0b8944-dc40-11e8-b6d2-48e24421dfbd/turtlebot3_lds-2.log].
log file: /home/pi/.ros/log/bb0b8944-dc40-11e8-b6d2-48e24421dfbd/turtlebot3_lds-2*.log
[INFO] [1540906376.792863]: ROS Serial Python Node
[INFO] [1540906376.851901]: Connecting to /dev/ttyACM0 at 115200 baud
[INFO] [1540906379.088580]: Note: publish buffer size is 1024 bytes
[INFO] [1540906379.091671]: Setup publisher on sensor_state [turtlebot3_msgs/SensorState]
[INFO] [1540906379.117068]: Setup publisher on firmware_version [turtlebot3_msgs/VersionInfo]
[INFO] [1540906379.208859]: Setup publisher on imu [sensor_msgs/Imu]
[INFO] [1540906379.228594]: Setup publisher on cmd_vel_rc100 [geometry_msgs/Twist]
[INFO] [1540906379.256692]: Setup publisher on odom [nav_msgs/Odometry]
[INFO] [1540906379.283882]: Setup publisher on joint_states [sensor_msgs/JointState]
[INFO] [1540906379.300244]: Setup publisher on battery_state [sensor_msgs/BatteryState]
[INFO] [1540906379.317624]: Setup publisher on magnetic_field [sensor_msgs/MagneticField]
[INFO] [1540906389.713450]: Setup publisher on /tf [tf/tfMessage]
[INFO] [1540906389.739880]: Note: subscribe buffer size is 1024 bytes
[INFO] [1540906389.741590]: Setup subscriber on cmd_vel [geometry_msgs/Twist]
[INFO] [1540906389.767742]: Setup subscriber on sound [turtlebot3_msgs/Sound]
[INFO] [1540906389.804781]: Setup subscriber on motor_power [std_msgs/Bool]
[INFO] [1540906389.829968]: Setup subscriber on reset [std_msgs/Empty]
[INFO] [1540906389.850336]: Setup TF on Odometry [odom]
[INFO] [1540906389.854633]: Setup TF on IMU [imu_link]
[INFO] [1540906389.858519]: Setup TF on MagneticField [mag_link]
[INFO] [1540906389.862347]: Setup TF on JointState [base_link]
[INFO] [1540906389.883494]: --------------------------
[INFO] [1540906389.887376]: Connected to OpenCR board!
[INFO] [1540906389.891327]: This core(v1.2.3) is compatible with TB3 Burger
[INFO] [1540906389.894822]: --------------------------
[INFO] [1540906389.898775]: Start Calibration of Gyro
[INFO] [1540906392.414749]: Calibration End
I was able to solve this error by changing the USB cable connecting raspberrypi and lidar board yesterday.  However after turning on turtlebot3 again, this error cannot be eradicated and I cannot seem to find out why. Please help. Thank you.

Hi :)
Can you test that catkin_make again after remove devel and build folder? (Need source ~/.bashrc command)
And launch turtlebot3_lidar only not turtlebot3_robot

Hi there!
Sorry I don’t really understand as I am still kind of new to this.. do you mean to execute this command?
$ cd ~/catkin_ws/
$ rm -rf src/ build/ devel/
$ mkdir src
$ cd ~/catkin_ws/src
$ git clone https://github.com/ROBOTIS-GIT/hls_lfcd_lds_driver.git
$ git clone https://github.com/ROBOTIS-GIT/turtlebot3_msgs.git
$ git clone https://github.com/ROBOTIS-GIT/turtlebot3.git

Input command:
pi@raspberrypi:~ $ cd ~/catkin_ws/
pi@raspberrypi:~/catkin_ws $ rm -rf build/ devel/
pi@raspberrypi:~/catkin_ws $ source /opt/ros/kinetic/setup.bash
pi@raspberrypi:~/catkin_ws $ cd ~/catkin_ws && catkin_make -j1
catkin_ws build was successful with no errors.
Input code to launch lidar:
pi@raspberrypi:~ $ roslaunch turtlebot3_bringup turtlebot3_lidar.launch
Output:
`... logging to /home/pi/.ros/log/8e82c4de-dce3-11e8-9630-3065ec8d1b60/roslaunch-raspberrypi-2884.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.
started roslaunch server http://<IP_ADDRESS>:34627/
SUMMARY
PARAMETERS

/rosdistro: kinetic
/rosversion: 1.12.13
/turtlebot3_lds/frame_id: base_scan
/turtlebot3_lds/port: /dev/ttyUSB0

NODES
/
turtlebot3_lds (hls_lfcd_lds_driver/hlds_laser_publisher)
ROS_MASTER_URI=http://<IP_ADDRESS>:11311
process[turtlebot3_lds-1]: started with pid [2893]
[ERROR] [1540974585.664563059]: An exception was thrown: open: No such file or directory
[turtlebot3_lds-1] process has died [pid 2893, exit code 255, cmd /home/pi/catkin_ws/devel/lib/hls_lfcd_lds_driver/hlds_laser_publisher __name:=turtlebot3_lds __log:=/home/pi/.ros/log/8e82c4de-dce3-11e8-9630-3065ec8d1b60/turtlebot3_lds-1.log].
log file: /home/pi/.ros/log/8e82c4de-dce3-11e8-9630-3065ec8d1b60/turtlebot3_lds-1*.log
all processes on machine have died, roslaunch will exit
shutting down processing monitor...
... shutting down processing monitor complete
done
`

Sorry for inconvenience, Can you please test it without ROS?
I give you a reference how to run it
http://emanual.robotis.com/docs/en/platform/turtlebot3/appendix_lds_01/#user-guide-for-driver

Hi, I was able to solve the problem by changing the cable connecting LIDAR to Raspberry Pi.
Now I am currently stuck with another problem at step 9, SLAM. When i ran this command:
roslaunch turtlebot3_slam turtlebot3_slam.launch slam_methods:=gmapping in RVIZ, I am unable to see any image produced by LIDAR where the status says "No image received". Do you know how to solve this issue? Please help thank you.

This issue persisted even at step 11: Simulation where i ran turtlebot3 simulation in gazebo and after launching command to open RVIZ: roslaunch turtlebot3_gazebo turtlebot3_gazebo_rviz.launch the camera does not display any image as well. Is there something wrong with my currently installed RVIZ package or is there problem somewhere else. I tried googling around but couldn't find any solution for this problem.

Sorry for late reply.
TB3 burger doesn't have a camera. If you want to show image, you are supposed to install camera on your TB3.

This issue will be closed since there were no actions for a while. You can reopen this issue to show this issue to the users whenever. Thanks.


Have RestClientProxy extend AutoCloseable

If RestClientProxy extended java.lang.AutoCloseable callers could use try-with-resources and could use the client close feature without needing a compile-time dep on smallryte-rest-client.
Of course users who did that would need to deal with the fact that AutoCloseable.close() throws Exception. The subinterface doesn't have to declare that though, so existing callers or ones not wanting to deal with that can cast to the subinterface.

Ability to close the client will actually become a portable feature in MP RestClient 1.3: https://github.com/eclipse/microprofile-rest-client/releases/tag/1.3-RC1

Cool; I'll close this then as that covers the need.


Add openEuler and OpenCloudOS support

Add openEuler and OpenCloudOS support. I have tested them and it works well.

Hello! Since you seem to have the os-release files on-hand, could you add them into respective folders in src/linux/tests/ and create tests in src/linux/file_release.rs?

Did I add these tests correctly?

The tests look good to me!


Add the feature to manually control redraw

Using this feature to manually repaint when the need to update arises will reduce the CPU and GPU load on the Desktop Window Manager(Windows) or WindowServer(MacOS).
In situations where machine performance is poor, the load on these window managers can be worrisome.
This is a feature needed for a very minor use case that only I need as I am creating tools that are always running.
This is not a perfect retained mode.

I just realized that there is a https://github.com/17cupsofcoffee/tetra/issues/119

You're right that #119 would be the ideal solution, but I'd maybe be open to adding something like this as a stopgap :+1:
That said, do you have any particular data that makes you concerned about the overhead of presenting every frame? I've not run into any issues with it, but I know you're using Tetra for a slightly more niche use-case than I am :)

Default

disable_auto_redraw

Default

disable_auto_redraw

Here is my file manager in action.
How accurate this measurement is is open to question, though.
I'm a little concerned about windowserver on MacOS.
I don't think this is something you need to worry about with modern many-core CPUs, and
I'd rather buy a more expensive CPU than worry about this, but
It is my selfishness that I want to give rustc as much CPU power as possible.
It's not much use in today's world, but on a poorly performing macbook, it might have some effect on battery life.

My only concern is that I'm not sure if skipping calls to graphics::present is the right approach to avoid redraw - your game loop is still going to be running in the background and taking up CPU if you do that, so I'm wondering if something along the lines of SDL_WaitEvent (i.e. sleep the thread until the next event) is more like what you'd need. I don't really write GUI applications, so I don't know how people generally handle this 😅
I'll give this some thought/ask around about how other people have achieved stuff like this. I've also made some progress with implementing #119 (though I still need to figure out a nice API), so you might potentially be able to use that instead of this PR.

It's not common to make tools with tetra like I did, but I think there are more and more examples of making tools with game frameworks these days.
LDtk, Pixel Fx Desinger, etc.
In the worst cases, people are using Unity to create tools.
Tools should be made using UI frameworks with retained mode, but when it comes to something that works cross-platform and can handle images and pixel art beautifully, using a game framework is probably the quickest way to go.
It is not my desire to take up your time by making you deal with niche use cases.

Tools should be made using UI frameworks with retained mode, but when it comes to something that works cross-platform and can handle images and pixel art beautifully, using a game framework is probably the quickest way to go.

💯
I definitely want to support this use case better in some way, I just want to make sure I'm not adding extra complexity via feature flags if there's a simpler/easier approach.


Large File Uploading Issue with RAM (x/post to Flask)

Writing this as an open-ended discussion and will be submitting the same information to the respective packages involved in this issue. Please note that this issue has been raised several times before, but has yet to be fixed / resolved. (Flask #2086, Werkzeug #1041, Werkzeug #510, Codalab-Worksheet #301, see also: StackOverflow Question)
Currently, the default_stream_factory  that is embedded in Werkzeug (a Flask Dependency) splits files and performs one of two operations:

If under 500kb, store the uploaded file in-memory.
If over 500kb, store the file in a temp file in /tmp.

This creates one main visible problem:

Machines with a smaller /tmp may send a traceback to the user with a "No Space on Device" OSError, not allowing uploads to complete.

And one hidden problem:

Large File Uploads (>500kb as defined in default_stream_factory) still eat into the RAM of the host machine, despite being sent to /tmp via TemporaryFile or SpooledTemporaryFile.

This redundancy can lead to a severe issue.
I was testing NS on a 4GB machine and performing a 4GB upload, which crashed the machine, and led me do two things:

Up my /tmp size to 10GB in /etc/fstab
Investigate the Memory Leak

I've not found a fix for this or any surefire way of counteracting the problem. Despite ensuring that /tmp has more than enough storage, RAM is still consumed. If the upload eats the RAM to 100% a system crash will occur, throwing the server offline. Somewhere in the reading I've been doing it's been suggested that Werkzeug stores the files into memory regardless of size (rendering formparser useless?) and has sloppy management with the data.
I have confirmed that this is reproducible by uninstalling and reinstalling both Flask and Werkzeug and trying it on multiple machines.
The intended behavior (storing in /tmp and not RAM) isn't happening.
Environment:

Python Version: 3.9.5
Werkzeug Version: 2.0.1

Any possible fixes or monkey patches would be appreciated; collaboration and discussion is welcome!

The parser was updated in 2.0 to be more memory efficient. It's not clear that any further memory issues are under Werkzeug's control. For example, I don't know how
You can override Request._get_file_stream to return any stream object you like. For example, you could always write to a physical file:
import tempfile
from werkzeug import Request as Request

class FileRequest(Request):
    def _get_file_stream(
        self,
        total_content_length,
        content_type,
        filename=None,
        content_length=None,
    ) -> t.BinaryIO:
        return tempfile.TemporaryFile("rb+")

app = Flask(__name__)
app.request_class = FileRequest

If you believe it's still Werkzeug's parser, you're welcome to investigate further and open an issue with specific findings. That would be a good start to identifying a fix and producing a PR as well. For now, I'm going to close this issue.

Using the fil memory profiler with both the standard Request and the FileRequest example above, the memory used by Python appears constant.


CASM-4467 stable 1.5

Summary and Scope
Cray-cli recently introduced a -tenant option as part of cray init command. When it was introduced, this tenant flag was required. However, this caused issues with existing scripts as none of them supplied the -tenant flag.
The -tenant flag was then made optional. However, the iuf-container image still used the version of cray-cli where the -tenant flag was required. This is causing issues in deliver-product and deploy-product stages.
Is this change backwards incompatible, backwards compatible, or a backwards compatible bugfix?
Yes
Issues and Related PRs
List and characterize relationship to Jira/Github issues and other pull requests. Be sure to list dependencies.

Resolves CASM-4467
Merge with/before/after:

https://github.com/Cray-HPE/cray-nls/pull/194
https://github.com/Cray-HPE/cray-nls/pull/193
https://github.com/Cray-HPE/cray-nls-charts/pull/57

Testing
Tested on:

Wasp
Surtur

Test description:
How were the changes tested and success verified? If schema changes were part of this change, how were those handled in your upgrade/downgrade testing?
Ran IUF
Risks and Mitigations
Are there known issues with these changes? Any other special considerations?
No
Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[x] CHANGELOG.md updated
[x] Testing is appropriate and complete, if applicable
[x] HPC Product Announcement prepared, if applicable

/backport release/1.5


Tuning scale: uncrowd overlapping notes

As a user, I don't want to see overlapping notes on a tuning scale.
Note: I'm calling this a bug, because currently, important visual data is obscured. Enhancement with an improved positioning algorithm will fix it.
Proposed (simple) solution
Where rendering a note in the default location would overlap previous notes, place the note lower in the row band. For each note attempt to place at the top, and repeatedly try lower until there is space.
This requires expanding the row band, and consequently the total size of the tunings SVG.
Stretch goal
Analyse whether other patterns and vertical spacings may result in better distributions or keep row expansion to a minimum. If note importance is implemented, see whether that can be used to ensure the important notes are placed at the top of the tuning band.

Example of overcrowding

Uncrowded. The first scale's lines overrun through the bottom, as we don't know the total size of all tuning rows when we're drawing the first row. The fix would be to pass it up the hierarchy, and at the top level, render stored SVG in draw order (like we do at the tuning level).


DirectionsService not working on Android

I'm submitting a ...

[ ] question
[x] any problem or bug report

OS:  (check one with "x")

[x] Android
[ ] iOS
[ ] Browser

cordova information:  (run $> cordova plugin list)
call-number 0.0.2 "Cordova Call Number Plugin"
com-darryncampbell-cordova-plugin-intent 2.0.0 "Intent Shim"
com.googlemaps.ios 3.9.0 "Google Maps SDK for iOS"
cordova-androidx-build 1.0.4 "cordova-androidx-build"
cordova-plugin-android-permissions 1.1.0 "Permissions"
cordova-plugin-androidx-adapter 1.1.1 "cordova-plugin-androidx-adapter"
cordova-plugin-androidx 2.0.0 "cordova-plugin-androidx"
cordova-plugin-camera 5.0.0 "Camera"
cordova-plugin-device 2.0.3 "Device"
cordova-plugin-firebasex 10.2.0-cli "Google Firebase Plugin"
cordova-plugin-geolocation 4.0.2 "Geolocation"
cordova-plugin-googlemaps 2.8.0-20200709-2008 "cordova-plugin-googlemaps"
cordova-plugin-ionic-keyboard 2.2.0 "cordova-plugin-ionic-keyboard"
cordova-plugin-ionic-webview 5.0.0 "cordova-plugin-ionic-webview"
cordova-plugin-network-information 2.0.2 "Network Information"
cordova-plugin-screen-orientation 3.0.2 "Screen Orientation"
cordova-plugin-splashscreen 6.0.0 "Splashscreen"
cordova-plugin-statusbar 2.4.3 "StatusBar"
cordova-plugin-uniquedeviceid 1.3.2 "UniqueDeviceID"
cordova-plugin-vibration 3.1.1 "Vibration"
cordova-plugin-whitelist 1.3.4 "Whitelist"
cordova-sqlite-storage 5.1.0 "Cordova sqlite storage plugin - cordova-sqlite-storage plugin version"
cordova.plugins.diagnostic 5.0.2 "Diagnostic"
es6-promise-plugin 4.2.2 "Promise"
phonegap-plugin-barcodescanner 8.1.0 "BarcodeScanner"

If you use @ionic-native/google-maps, please tell the package.json (only @ionic-native/core and @ionic-native/google-maps are fine mostly)
@ionic-native/core : "^5.27.0"
@ionic-native/google-maps : "^5.27.0-beta-20200630"

Current behavior:

      'origin': this.origin,
      'destination': this.destination,
      'travelMode': TravelModeId.DRIVING
    })

return Promise
Expected behavior:

DirectionsService.route(...)

throw exception
cordova.callbackFromNative('PluginDirectionsService958478195',false,9,["NOT_FOUND"],false);

Related code, data or error log (please format your code or data):
https://github.com/timkovik/postsrvs/tree/stable

Not ready to use in Android and iOS

I know, but it used to work. Is there a demonstration repository https://github.com/mapsplugin/ionic-googlemaps-quickdemo-v5, or should it not be taken seriously?
To sum up: did I understand correctly that there can be no constructive advice in this case?

This plugin uses Google Maps JavaScript v3 on browser platform. That's why it works on the demo page.
But Google Maps SDK for Android/iOS do not have the direction feature. Directions API is separated. That's why can not.

oh, thanks! Now im understand


SPARK-7021 Add JUnit output for Python tests

WIP

Can one of the admins verify this patch?

Jenkins, this is ok to test.

Merged build triggered.

Merged build started.

Test build #41290 has started for   PR 8323 at commit d93e7cd.

Test build #41290 has finished for   PR 8323 at commit d93e7cd.

This patch fails RAT tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class _DelegateIO(object):
class _TestInfo(object):
class _XMLTestResult(_TextTestResult):
class XMLTestRunner(TextTestRunner):

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41290/
Test FAILed.

Merged build finished. Test FAILed.

https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41290/console
Recording test results
ERROR: Publisher 'Publish JUnit test result report' failed: No test report files were found. Configuration error?
Test FAILed.
Can somebody comment on how is Jenkins configured to look for JUnit results (or did this build fail on RAT)? Thanks

Merged build triggered.

Merged build started.

The build failed on RAT:
Could not find Apache license headers in the following files:
 !????? /home/jenkins/workspace/SparkPullRequestBuilder/dev/xmlrunner/__init__.py
 !????? /home/jenkins/workspace/SparkPullRequestBuilder/dev/xmlrunner/xmlrunner.py
[error] running /home/jenkins/workspace/SparkPullRequestBuilder/dev/check-license ; received return code 1

I just added the headers, xmlrunner is LGPL (and only used to run the tests).

Test build #41291 has started for   PR 8323 at commit d4f48ba.

Test build #41291 has finished for   PR 8323 at commit d4f48ba.

This patch fails RAT tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class _DelegateIO(object):
class _TestInfo(object):
class _XMLTestResult(_TextTestResult):
class XMLTestRunner(TextTestRunner):

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41291/
Test FAILed.

Is this a no-go because of the license? Thanks

@gliptak Is https://pypi.python.org/pypi/unittest-xml-reporting/1.12.0 an alternative? It is BSD license. We can install the package on Jenkins machines, instead of including the source code in the codebase.

@mengxr Would installing on Jenkins be independent from the codebase (the import xmlrunner just be available)? Thanks

If this new library works, I think we could install it on Spark Jenkins. Could you test it locally and see whether it works?

I tried this locally and the files were generated. Somebody with visibility on Jenkins might need to comment on file locations ...

Merged build triggered.

Merged build started.

Test build #42930 has started for   PR 8323 at commit a6b3a5b.

Test build #42930 has finished for   PR 8323 at commit a6b3a5b.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42930/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #42933 has started for   PR 8323 at commit 05e5a37.

Test build #42933 has finished for   PR 8323 at commit 05e5a37.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42933/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #42935 has started for   PR 8323 at commit 18f5901.

Test build #42935 has finished for   PR 8323 at commit 18f5901.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42935/
Test FAILed.

Please fix Python style issue:
./python/pyspark/streaming/tests.py:1303:13: E128 continuation line under-indented for visual indent

@mengxr Thanks for the local lint pointer

Merged build triggered.

Merged build started.

Test build #42980 has started for   PR 8323 at commit e9a711e.

Test build #42980 has finished for   PR 8323 at commit e9a711e.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42980/
Test FAILed.

@shaneknapp Could you take a look at this PR and see whether we can install https://pypi.python.org/pypi/unittest-xml-reporting/1.12.0 on Jenkins?
@gliptak We shouldn't make xmlrunner a requirement on local machines. If it is installed, we output XML. Otherwise, we should fallback to the default behavior. This could be done by catching exceptions at the import line.

could you create a spark issue for the plugin and assign it to me?
also, i won't be able to do this until monday, as i'm about to leave town
for the weekend.
On Thu, Sep 24, 2015 at 2:50 PM, Xiangrui Meng<EMAIL_ADDRESS>wrote:

@shaneknapp https://github.com/shaneknapp Could you take a look at this
PR and see whether we can install
https://pypi.python.org/pypi/unittest-xml-reporting/1.12.0 on Jenkins?
@gliptak https://github.com/gliptak We shouldn't make xmlrunner a
requirement on local machines. If it is installed, we output XML.
Otherwise, we should fallback to the default behavior. This could be done
by catching exceptions at the import line.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/8323#issuecomment-143059734.

Merged build triggered.

Merged build started.

@mengxr Revised.

Test build #42996 has started for   PR 8323 at commit 66bdb2b.

Test build #42996 has finished for   PR 8323 at commit 66bdb2b.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42996/
Test FAILed.

Created https://issues.apache.org/jira/browse/SPARK-10830.

test this please

Merged build triggered.

Merged build started.

Test build #43015 has started for   PR 8323 at commit 66bdb2b.

Test build #43015 has finished for   PR 8323 at commit 66bdb2b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43015/
Test PASSed.

test this please

unittest-xml-reporting is installed on AMPLab Jenkins. So let's call Jenkins again.

Test build #1821 has started for   PR 8323 at commit 66bdb2b.

Test build #1821 has finished for   PR 8323 at commit 66bdb2b.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1810/testReport/
doesn't seem to list the Python tests.
Can somebody with access to Jenkins configuration review?
Thanks

i'll take a look at the config tomorrow
On Mon, Sep 28, 2015 at 6:02 PM, gliptak<EMAIL_ADDRESS>wrote:

https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1810/testReport/
doesn't seem to list the Python tests.
Can somebody with access to Jenkins configuration review?
Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/8323#issuecomment-143914542.

@gliptak We use **/target/test-reports/*.xml on Jenkins to locate XML reports. So target/surefire-reports may not work. (@shaneknapp please correct me if I'm wrong.)

Merged build triggered.

Test build #43090 has started for   PR 8323 at commit 66bdb2b.

Test build #43090 has finished for   PR 8323 at commit 66bdb2b.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43090/
Test FAILed.

@mengxr I pushed a commit with the updated location.

Merged build triggered.

Merged build started.

Test build #43104 has started for   PR 8323 at commit cd654c7.

Test build #43104 has finished for   PR 8323 at commit cd654c7.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43104/
Test FAILed.

test this please

Merged build triggered.

Merged build started.

Test build #43107 has started for   PR 8323 at commit cd654c7.

Test build #43107 has finished for   PR 8323 at commit cd654c7.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43107/
Test FAILed.

Though Jenkins failed, I can see the report now: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43107/testReport/junit/(root)/SerializationTestCase/test_pickling_file_handles/.

test this please

Merged build triggered.

Merged build started.

Test build #43109 has started for   PR 8323 at commit cd654c7.

Test build #43109 has finished for   PR 8323 at commit cd654c7.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43109/
Test FAILed.

test this please

Merged build triggered.

Merged build started.

Test build #43113 has started for   PR 8323 at commit cd654c7.

Test build #43113 has finished for   PR 8323 at commit cd654c7.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43113/
Test FAILed.

@mengxr Thank you for guiding this

test this please

it seems that this PR has a higher-than-normal failure rate .. I don't know why. +@davies

Merged build triggered.

Merged build started.

Test build #43176 has started for   PR 8323 at commit cd654c7.

Test build #43176 has finished for   PR 8323 at commit cd654c7.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43176/
Test FAILed.

Merged build finished. Test FAILed.

test this please

Merged build triggered.

Merged build started.

Test build #43252 has started for   PR 8323 at commit cd654c7.

Test build #43252 has finished for   PR 8323 at commit cd654c7.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43252/
Test FAILed.

Huh, looks like the last two failures were for the same PySpark test:
======================================================================
FAIL [0.000s]: test_pickling_file_handles (__main__.SerializationTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/jenkins/workspace/SparkPullRequestBuilder/python/pyspark/tests.py", line 259, in test_pickling_file_handles
    self.assertEqual(out1, out2)
AssertionError: <StringIO.StringIO instance at 0x2d65a28> != <StringIO.StringIO instance at 0x2ffdb90>

I guess that this doesn't handle doctests, right? Is there any way to get XML for those, too? If not, could you edit the PR title / description to reflect this?

@gliptak, can you try to reproduce the failure locally? See ./pyspark/run-tests --help for hints on how to run only this test suite under Python 2.6.

@JoshRosen I do not have python2.6 installed (or available as a package) locally

@JoshRosen How are the doctest-s triggered on the commandline and in Jenkins?
$ python/run-tests --help
No handlers could be found for logger "root"
Usage: run-tests [options]

Options:
  -h, --help            show this help message and exit
  --python-executables=PYTHON_EXECUTABLES
                        A comma-separated list of Python executables to test
                        against (default: python,python3.4)
  --modules=MODULES     A comma-separated list of Python modules to test
                        (default: pyspark-core,pyspark-ml,pyspark-mllib
                        ,pyspark-sql,pyspark-streaming)
  -p PARALLELISM, --parallelism=PARALLELISM
                        The number of suites to test in parallel (default 4)
  --verbose             Enable additional debug logging

What is the next step on this pull request? Thanks

@gliptak I had an offline discussion with @davies . Let's do the following:

Change L259 in pyspark/tests.py to:

  # Regression test for SPARK-3415
  def test_pickling_file_handles(self):
      if not xmlrunner:
          ser = CloudPickleSerializer()
          out1 = sys.stderr
          out2 = ser.loads(ser.dumps(out1))
          self.assertEqual(out1, out2)

Create a JIRA for this so we remember to fix it. The issue is CloudPickeSerializer wraps stderr, which conflicts with xmlrunner. But it might take some time to fix.
2. Change the PR title to `Python unit tests" since we do not report doctests.
Then we can run Jenkins again and see whether it works.

Merged build triggered.

Merged build started.

Test build #43863 has started for   PR 8323 at commit 177fef9.

Test build #43863 has finished for   PR 8323 at commit 177fef9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43863/
Test PASSed.

Right now, all the Python unit tests are in the package of (root), is there a way to set a package for it (such as pyspark)?

@davies Not that I see. I opened https://github.com/xmlrunner/unittest-xml-reporting/issues/95 to discuss.

@JoshRosen I opened https://github.com/xmlrunner/unittest-xml-reporting/issues/94 to discuss doctests processing.

This looks good to me. I think we can address the test namespace issue in a later PR. @davies @JoshRosen ?

LGTM, merging this into master, thanks!

As per https://github.com/xmlrunner/unittest-xml-reporting/issues/95 the Python tests do not have packages (or possibly not being recognized by https://github.com/xmlrunner/unittest-xml-reporting/

Adding packages to XMLs at https://github.com/apache/spark/pull/9263


v1.5 backports 2019-08-15

#8908 -- cilium: install transient rules during agent restart (@borkmann)
#8901 -- Dockerfiles: update to golang 1.12.8 (@ianvernon)
#8925 -- Fix proxy port leak on endpoint delete (@joestringer)

Once this PR is merged, you can update the PR labels via:
$ for pr in 8908 8901 8925; do contrib/backporting/set-labels.py $pr done 1.5; done

This change is

never-tell-me-the-odds

never-tell-me-the-odds

never-tell-me-the-odds

never-tell-me-the-odds

test-missed-k8s

Health test failed: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-K8s/2383/

test-missed-k8s

Upstream K8s tests have been out of commission for some time in v1.5; all other tests passed, so am merging.


Implement expiration date for shares

Expiration date of share is not yet supported in OCIS for any of the share types:

user
group
public link

The API test scenarios for these are currently running in CI and are in the expected-failures file.
ToDo:

[x] find all the share expiration scenarios and put them together in the expected-failures file, linked to this issue, and in sections for user, group, and public link shares #2412
[ ] implement in oCIS
[ ] make it configurable in oCIS with similar options to oC10

This was the original post, preserved for posterity:
As I can see in the code this is not implemented yet.
As for tests, it seems most will stay disabled because we need to set some settings with occ...

[ ] tag the matching tests with this ticket
[ ] implement expiration date for user shares
[ ] somehow make it configurable as we have no occ in ocis yet
[ ] enable tests

@pmaier1 FYI for adding to the roadmap somewhere. feel free to move to another project than MVP as this is blocked by the config stuff
@individual-it FYI

This issue specifically mentions expiration date for user shares. I think that we also want expiration dates for group shares and public link shares. I suspect that implementation for all of these will be done together.
@pmaier1 is this issue still current (or is there another issue somewhere about this)?
Do we expand this issue to also cover group shares an public link shares? Or create separate issues?

Assigned myslef so that I see this next week. This is a good example of a feature that is not implemented yet. Currently some of the related test scenarios are tagged in a way that they do not run in oCIS CI, and some are running in oCIS CI and listed in expected-failures. We need a consistent process for tagging tests.

@pmaier1 is this issue still current (or is there another issue somewhere about this)?

This is still current but it is clearly a feature on the roadmap, not a bug. I'd be happy to exclude this feature from our hardening efforts and start working on it when the time has come.

@pmaier1 should I edit the title to be "Implement expiration date for all shares" and mention the various share types in the top ost?
(i.e. make this an issue about all expiration dates)

@pmaier1 should I edit the title to be "Implement expiration date for all shares" and mention the various share types in the top ost?

If I'm not mistaken, the expiration date for links feature is already implemented. If that's not the case, I agree with your proposal.

Note:
Scenario Outline: sharing with default expiration date enabled but not enforced for groups, user shares with expiration date
That fails (and other group expiration date test scenarios)
Scenario Outline: Create a public link with default expiration date set and max expiration date enforced and access using the public WebDAV API
That also fails: apiSharePublicLink1/createPublicLinkShare.feature:652 is in the expected failures file and I can see it failing in a recent drone build.
So share expiration is not yet working for any of the share type. I will adjust the top post.

These policies (enforce expiration date for user shares / public links) are also new features for oCIS - to be handled via the roadmap.

This issue is referenced in expected-failures. Reopening.

expiration date for shares works correct. I close issue
The tests were removed from the expected failure file and rewritten using e2e https://github.com/owncloud/web/pull/10292


coursier is participating to Google Summer of Code 2021!

Scala Center is leading this year’s Google Summer of Code, coursier will be a part of it. If you are interested in joining, please check out more here.
See also this blog post for more details, and the full project list here.
One project is directly related to coursier:

coursier - autocompletion. It should be mentored by @eloots, with possible help from myself (@alexarchambault).

If you are interested in working on this project, do get in touch with @eloots (via email?) or myself (@alexarchambault, I can be reached via Gitter).

Closing, as the student application period is now over.


Local Variables

Resolves
Towards allowing creating local variables.
Towards resolving LLK/scratch-vm#1278.
Proposed Changes
Allow for local variables by passing in extra scope information (provided by the gui). This information is included in the VarCreate event. Pass in a list of additional var names to check against when creating a variable (e.g. local var names from all sprites when checking for name conflicts to create a global var.
Reason for Changes
Allowing creation of local variables (providing the relevant information to the VM through the var_create event).
Related PRs
This PR should be merged first.
This PR is also related to the following PRs:
LLK/scratch-vm#1301
LLK/scratch-gui#2548

As we talked about offline, this looks good but I think we should include xml serialization in this so that we don't end up "lying" to scratch blocks about global/local variables, so in the future if we do want to use this information (e.g. for rendering global vars differently) it doesn't turn out to be wrong. This will also require an update to the linked VM issue. Even though it isn't strictly needed, we should make sure to keep the data model clean and correct.

@paulkaplan I made the serialization change and another change as well to fix variables in the vertical playground (they weren't working because additionalVars was not getting initialized without the gui connection).


add Datastore v1 & Bigtable v2 protos

@callmehiphop does this look right for Bigtable v2?

Well, they also updated the admin API and actually split it off into two separate APIs (instance & table).
https://github.com/googleapis/googleapis/tree/master/google/bigtable/admin/v2

Okay, I'll pull that commit out of this PR. Feel free to break our existing API if necessary. Can you send yours soon so we can introduce both Datastore v1 and Bigtable v2 support in 0.6.0?

I've been using the function version of googleProtoFiles() so if you want to ship without Bigtable, don't feel the need to wait on me :)
Otherwise, yes! I will do it soonish

Cool. gcloud-node talk: It would be better to use the API from google-proto-files rather than the function to avoid having a dependency within our Bigtable module, since we already have it in our common module.


v0.2.3 not show completer-list-item

after update from v0.2.2 to v0.2.3 - not show completer-dropdown-holder

@KANekT can you add some more details?

https://s.mail.ru/8xRB/YGtkCZ84Q
not showed
in demo use 0.2.3?

@KANekT the demo uses 0.2.3 and it shows both the title and the description.


Pin is Sealed in Nuget package

VERSIONS

Xamarin.Forms.GoogleMaps - 3.3.0

PLATFORMS

[x] Android
[x] iOS
[ ] UWP

ACTUAL BEHAVIOR
I can't inherit from the pin, even though you can from the normal Xamarin.Forms.Maps. Also I see that in the code here, sealed has been removed from pin.
EXPECTED BEHAVIOR
I want pin to not be sealed in the nuget package. - So please just release the newest version of the code. Or just a version where pin isn't sealed.

I am also waiting for this to happen as I want to display Url's on the Information window when you click on the pin which is not there in the default Pin class.

@amay077  when would you be able to push this fix to nuget release ?

@amay077  when would you be able to push this fix to nuget release ?

@amay077 When will this be released on nuget? I see there is a PR#701 merged. but latest nuget package version 4.0.0-beta1, still has sealed class Pin


ember-cli-tailwind should use the path to css defined by the host application

ember-cli-tailwind looks for the app/styles/app.css entry point. Is it not possible to use whatever path configured by the host application?
If for example I override the default configuration in my ember-cli-build:
  let app = new EmberApp(defaults, {
    // Add options here
    outputPaths: {
      app: {
        css: {
          'app': '/assets/application-name.css'
        }
      }
    }
  })

I would expect ember-cli-tailwind to respect my configuration when looking for the entry point.

I didn't know about this config option! Yes this should work.
Is it just a bug in the blueprint (i.e. when you install e-c-tailwind, it tries to add @import to app.css)? Does it work if you add that line to your application-name.css?

Hi @samselikoff! In reality I actually want to achieve something else. I want to change the input path of css. In this case I want the app.css to reside in the app root and not in styles/app.css.  I realize this now has turned into an ember-cli question.
But of course it would be good if ember-cli-tailwind respected whatever inputPath and outputPath configured into the application.
If ember-cli-tailwind gets the input path to the main css file in a non-hardcoded way, then you could just close this issue.

Ah I see. Yes going outside of treeForStyles will require some custom Ember CLI work.
I don't think Ember CLI Tailwind needs an input path to your app's main css file... the default blueprint tries its best to add an @import 'tailwind.css' line to the right file, but if it's in a non-standard spot you should be able to just add that line yourself. (Ember CLI Tailwind puts your built tailwind.css file into the root of your treeForStyles, so it should be accessible from that location regardless of how you're consuming it.)

Yes, I guess most people will not mess with the defaults and so it is not worth checking this in a less hard-coded fashion. I will really enjoy when the new file layout comes.
Please feel free to close this.


Is there a quick way to use distributed tensorflow instead of GPU?

Is it possible to run the code in a multi-core platform where there is no GPU? I can see that the setup.py requires locating CUDA.

Not explored that space yet.

closing, please check https://github.com/endernewton/tf-faster-rcnn/issues/121


support more dynamic cluster header

Title: support formatted cluster header
Description:
cluster_header is used to select dynamic upstream cluster based on the request headers.
This function is very useful. But I would like to make this feature more flexible with a core.v3.TypedExtensionConfig API. Like being able to select an upstream cluster based on a number of different headers, or formatting the header values and selecting the corresponding upstream cluster, etc.

It sounds like use header to control the route.

Yea, cluster_header is used both in weighted cluster and general cluster_specifier level.
Actually there is already a ClusterSpecifierPlugin in RouteConfiguration can be looked up by name.
The proto API is there though its implementation is not added yet. Does this meet your needs?

@tyxia Thanks. It's something like that.
But what I want is to add an core.v3.TypedExtensionConfig in the RouteAction directly. Then we can provide some route level config by this field. And by this way, we can even implement the cluster_header and weighted_cluster by this field.
  oneof cluster_specifier {
    option (validate.required) = true;

    // Indicates the upstream cluster to which the request should be routed
    // to.
    string cluster = 1 [(validate.rules).string = {min_len: 1}];

    // Envoy will determine the cluster to route to by reading the value of the
    // HTTP header named by cluster_header from the request headers. If the
    // header is not found or the referenced cluster does not exist, Envoy will
    // return a 404 response.
    //
    // .. attention::
    //
    //   Internally, Envoy always uses the HTTP/2 *:authority* header to represent the HTTP/1
    //   *Host* header. Thus, if attempting to match on *Host*, match on *:authority* instead.
    //
    // .. note::
    //
    //   If the header appears multiple times only the first value is used.
    string cluster_header = 2
        [deprecated = true, (validate.rules).string = {min_len: 1 well_known_regex: HTTP_HEADER_NAME strict: false}];

    // Multiple upstream clusters can be specified for a given route. The
    // request is routed to one of the upstream clusters based on weights
    // assigned to each cluster. See
    // :ref:`traffic splitting <config_http_conn_man_route_table_traffic_splitting_split>`
    // for additional documentation.
    WeightedCluster weighted_clusters = 3 [deprecated = true];

    // envoy.router.cluster_specifier.cluster_header
    // envoy.router.cluster_specifier.weighted_header
    // envoy.router.cluster_specifier.other_custom_extension
    core.v3.TypedExtensionConfig custom_cluster = 5;
  }

I think this approach might be more flexible, what do you think?

@wbpcode Thanks for laying out your thoughts and example.
IMO, the existing design is better than putting core.v3.TypedExtensionConfig in the RouteAction directly.

Existing design that puts ClusterSpecifierPlugin(which has TypedExtensionConfig inside) in top level RouteConfiguration avoids config bloat. Thinking about the case when many different routes use same custom cluster extension (i.e., custom plugin), we don't need to specifies/duplicates at each individual route level many times.

Existing design can achieve all the flexibility you want? If route-level flexibility is wanted, then at the individual route level, you just need to specify the key to look up the extension from repeated ClusterSpecifierPlugin in RouteConfiguration.

Does this make sense to you?

@wbpcode Thanks for laying out your thoughts and example.
IMO, the existing design is better than putting core.v3.TypedExtensionConfig in the RouteAction directly.

Existing design that puts ClusterSpecifierPlugin(which has TypedExtensionConfig inside) in top level RouteConfiguration avoids config bloat. Thinking about the case when many different routes use same custom cluster extension (i.e., custom plugin), we don't need to specify/duplicate the config at each individual route many times.
Existing design can achieve all the flexibility you want, no? If route-level flexibility is wanted, then at the individual route level, you just need to specify the key to look up the extension from repeated ClusterSpecifierPlugin in RouteConfiguration.

Does this make sense to you?

@tyxia thanks for your patient and detailed reply.
Right. We just need to set a unique name for the route that need route level specificed config. But the original API may needs some minor updates.
We may need a independent plugin/provider name which will be specified by the single route. Because the name in the TypedExtensionConfig may be used by the getAndCheckFactoryByName API. So the name in TypedExtensionConfig is best used only to denote the type of the extension. 🤔
Is there any progress on this work so far? If not, I would like to try to implement it. And would the field name cluster_provider be better than the cluster_specifier_plugin?

@wbpcode  Actually the name of in TypedExtensionConfig has been evolved to represent the name rather than the type nowadays. Please see the comment
I agree that alternatively we can use something like map<string name, google.protobuf.Any> so that name which is key of the map is specified by single route.  In current design we can just use the name in the TypedExtensionConfig as the key to achieve the same behavior.  I think the reason API is designed as it is because we are moving from any to TypedExtensionConfig.
Yea, AFAIK, no one has worked on the implementation yet. But add @markdroth to confirm as I believe that extension was originally introduced for grpc use case

@tyxia get it.

@tyxia An initial commit is pushed.
And in the process of implementing it, I thinked more about the exist API design.
Although the existing design that puts ClusterSpecifierPlugin(core.v3.TypedExtensionConfig) in top level RouteConfiguration may reduced repeated configurations. It still make the API more complex.
And in practice, we should rarely use this extension. In other words, the benefits of the exist design may not be obvious.
Also, in the current design, if we need to update an extension configuration, we have to update the entire route table. This is not friendly to users using vhds.
So in the first commit, I keeped the exist design and add a new core.v3.TypedExtensionConfig field in the route.


Problems with changed CIDR in CoreDNS

Hi guys, I have changed the https://microk8s.io/docs/change-cidr and the Service CIDR.
But can't enable the DNS Addon cause of this line:
https://github.com/canonical/microk8s-core-addons/blob/e2c013f25f7d2ee1a0ef71ad408c55f41ad2e478/addons/dns/coredns.yaml#L149
I'll change it manually and apply it, but perhaps there is a better solution for other people who come across the same issue.

Hi @kanimaru
Sorry for missing this. In MicroK8s 1.27 it is possible to do:
microk8s enable dns /etc/resolv.conf <IP_ADDRESS>

to specify the cluster IP address of the coredns service.
See https://github.com/canonical/microk8s-core-addons/pull/181


build synthetic seismogram in a specific station

Hi everyone
I have to have a script that read :
(1) source latitude, longitude, depth, and fault parameters e.g., strike, rake, dip, length of fault, M0 etc
(2) station coordinate e.g., latitude, longitude
(3) velocity model
then create synthetic seismogram at that station. Do you have such script or can you give me some hints how can I do it using KIWI.
many thanks

This is the wrong repo for that, this is an objective-c test runner for macOS and iOS projects. I did a quite googling but don't really know where to refer you. Good luck.


Change a few method signatures

Summary
Change a few method signatures to accept impl Into<String>, instead of &str
Motivation
Currently, several methods like MessageDialog::set_title, MessageDialog::set_description or FileDialog::add_filter accept &str as one of the arguments - this is extremely inconvenient, as if you have a String from format! macro, one has to convert it to &str and sometimes it may not be possible, due to the borrow checker.
And internally - titles, descriptions, extensions, etc. are stored as Strings, so it'd make more sense to convert anything to a String in the methods themselves.
Why Into<String> and not ToString?
To tackle this question, we first need to know the difference between two traits.
ToString - converts the value to String without consuming it.
Into<String> - consumes the value while converting it to a String.
All methods in this PR use the arguments to show data in message boxes - therefore it's most unlikely for that data to be used after the message box is shown.
If one really needs to use the argument after a method from rfd, one can clone or copy the value.
Compatibility with existing codebases
This PR doesn't require any overhaul for the users of the library, as &str implements both Into<String> and ToString

Thanks!


New Octicons don't have enough contrast

Related #76909
Old

New

Bold

This feedback is not around the shape of the icon but the fact that our current octicons all have a solid background applied which puts a very nice contrast on top of the status bar. With the new transparent icons we are loosing this contrast. They almost look like the reverse icon of what we have today (e.g. now transparent where we used to have white background).
Originally posted by @bpasero in https://github.com/microsoft/vscode/pull/79234#issuecomment-521899520

I think when they were implemented they were designed and tested on high-dpi screens.
For more "traditional" setups like 24" 1080p, they look blurry because they get antialiased to transparency.

@strafbier this issue is about Octicons in the status bar and quick pick, not the activity bar.
See this comment about why they appear blurry.

The solid icons would be fine if they had the same positioning/height/spacing as New and Bold, rather than appearing as off-center or not taking up enough of the space on the status bar.

The new icons are horrible designed and not helpful for perception.

@sstaub can you elaborate? Are you referring to our new icons everywhere or just the Octicons?

Both, the icons on the sidebar also on the statusbar (insider version).
The buttons and symbols on statusbar and sidebar were better before, because they had a better contrast.
Maybe we can choose between old and new design in settings.

Thanks everyone for the feedback. In trying to keep our issue count on a manageable level, I'm going to close this issue as a duplicate of https://github.com/microsoft/vscode/issues/79130 since that will solve this issue. Please upvote the issue there.


Clan Support

Hey really enjoyed the plugin!
I went and implemented the new chats menu entries and highlighting.
I also added when someone is invited to the clan and accepts it auto checks if they are in the list.

Awesome! I'll merge this in tomorrow!


Add link to burn your select tags

The best way to explain why we’d like people not to use native select
tags is to get them to watch this talk.
Also mention the HTML select element (next to drop-down lists), so it’s clear we’re talking
about the same thing.

I agree this is a great way to illustrate the evidence behind that point, just to say it's worth checking with the author before publishing this.
@alicebartlett are you ok with this forming part of elements?

I should have asked @alicebartlett in the first place, sorry Alice and let me know if this is okay with you.
I'm trying to provide as much evidence from user research as I can behind our guidance, hence wanting to add this link - and the other links to relevant hackpads too.


support updating status of resources managed by other controllers

Thanks @enisoc for your efforts. This makes writing controller much easier :)
I am trying to write a controller to create a load balancer service in external load balancer when a service is created in the cluster with type: LoadBalancer. I  also need to create a corresponding NetworkPolicyto allow traffic from LB nodes to worker nodes for the given service. Looking from the document DecoratorController seems like a close fit (with an issue) as CompositeController assumes ownership of parentresource. In my current use case parentresource is Service.v1.
DecoratorController creates the desired NetworkPolicy object but does not allow me to update the Statusfield of  the service which is target resource as defined below.
apiVersion: metacontroller.k8s.io/v1alpha1
kind: DecoratorController
metadata:
  name: lb
spec:
  resources:
  - apiVersion: v1
    resource: services
  attachments:
  - apiVersion: networking.k8s.io/v1
    resource: networkpolicies
  hooks:
    sync:
      webhook:
        url: http://service-per-pod.metacontroller/sync-lb

I need to update the Status field with info with newly created Loadbalancer IP and hostname, so users can see the IP address they can use to reach their service. I have created an example using jsonnet which in gist show I would like to use.
So wondering is there any plan to add support for this feature in DecoratorController or create any other controller which can let me do what I need in this use case.

DecoratorController doesn't let you modify status because your changes would be overwritten anyway by the real Service controller. The Kubernetes controller model allows multiple users/clients/controllers to collaboratively edit spec, but status is completely owned by only one canonical controller for a given resource.
The usual way to plug in an implementation for type: LoadBalancer is through the cloudprovider interface. Unfortunately, this plug-in mechanism is not designed around controllers. Since the Service controller code interacts with the LoadBalancer implementation through compiled-in function calls, rather than through declarative API objects, there's not a perfect way for Metacontroller to help here short of replacing the Service controller entirely.
If the only reason you need to update status is to communicate the external IP to users, you could put that info in an annotation. It's not ideal because you'd need to tell users to look in a different place than usual, but that might be acceptable depending on your use case.

Thanks for your reply. I can discuss with our team the use of annotations for info rather than status. As that will help in reducing our code size.


Removing Publishing Steps from public build

Remove publishing symbols and nupkg package from public build to internal release pipeline

.vsts-release.yml should be obsolete at this point? can we remove it now too?

sorry one more: there is usage of PushNupkg which is obsolete across the GitHub yamls?


can't use MultiSamlStrategy

Error: Cannot find module 'passport-saml/MultiSamlStrategy'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:581:15)
    at Function.Module._load (internal/modules/cjs/loader.js:507:25)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:20:18)

Hello @sibelius, it's a typo in the documentation. The file name is multiSamlStrategy so
• either we change it to be required as require('passport-saml/multiSamlStrategy')
• or rename the file and require('passport-saml/MultiSamlStrategy')

@stavros-wb this does not work because only lib in on npm version
I've made a PR to fix this https://github.com/bergie/passport-saml/pull/304
and let the code be more consistency


Tweak package annotations

Update the Slack and GitHub package annotations, including shortening the descriptions add adding package level parameters.
@vinodmut any more input you can provide on what's needs to be done?

Defer to @nickm.

Not applicable for this repository.