Add legacy stream protocol

Merge #2 first then change target branch to master
Remove friction when working with libraries/frameworks which use these streams.
You need this to implement a nextcloud backend https://github.com/nextcloud/server/blob/v19.0.1/lib/public/Files/Storage/IStorage.php#L272
This is not a full protocol which can handle all the fopen() flags, just 2 static methods to make a write or read stream.
It might still be necessary to implement a full read/write protocol. the S3 nextcloud backend does this by maintaining a local copy of the entire remote object and flushing it at the end. We'll see.

All changes in this PR are moved to #8


Requested scopes not being carried forward to permissions using Oauth2

I am requesting authorization through this call:
https://secure.meetup.com/oauth2/authorize
?scope=ageless+basic+event_management+group_content_edit+rsvp
&client_id=MY_CONSUMER_KEY
&response_type=code
&redirect_uri=MY_REDIRECT_URI
I see the "Connect with application" message as shown here. I note that four of the five scopes I requested are listed; all except the "ageless" scope.

I click Allow, which causes successful redirect to my REDIRECT_URI and I receive an auth code.
I then make an form encoded http post for an access token, following the format as instructed:
https://secure.meetup.com/oauth2/access
client_id=MY_CONSUMER_KEY
&client_secret=MY_CONSUMER_SECRET
&grant_type=authorization_code
&redirect_uri=MY_REDIRECT_URI
&code=CODE_I_RECEIVED_FROM_THE_AUTHORIZATION_RESPONSE
Access token is received successfully.
I then make an authenticated request to, for example, the /self/groups function. This runs OK and returns the expected result.
Now for the problem. When I go to Settings->Apps, I see my application listed under Developer Applications, but with only two scopes - basic and ageless. The other three have not been applied, as seen here:

Can anyone advise why the scopes requested, and agreed to by the user, are not being applied?

On closer examination this appears to be a site bug rather than an API bug. The scopes are assigned to the application, but the Meetup site simply does not show this under its "This app can" text. Surely this is a fairly serious bug given data protection requirements? If an application has permission to sign a user up to public events on that user's behalf, surely the site should correctly display this to be the case?


Support to read task logs from some S3 compatible cloud storage

Fixes #13166.

Description
Added quotation marks to the beginning and end of withMatchingETagConstraint() Arguments to comply with RFC7232.

This PR has:

[x] been self-reviewed.
[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

Thanks for reporting and fixing this.
I checked the original issue description, seems like that this problem occurs if the server is minio?
I'm not sure if the quotation marks returned from S3 server are removed or not, I think more robust way is to add the quotation marks only when it has no such marks.

By the way, have you tested the patch on your cluster?

Thanks for the review!
I fixed it so that it only adds quotes when the Etag value has no quotes.

By the way, have you tested the patch on your cluster?

I tested it on my cluster with a mini-application with only the code extracted from the relevant part.

@FrankChen021 Thank you for your kindness and support. I was nervous, but your support saved me. I'm happy.


[IOTDB-1609] Print the actual size of plan when it exceeds the wal_buffer_size

Contributions:

Print the actual size of plan when it exceeds the wal_buffer_size

Use PhysicalPlan.serialize(DataOutputStream stream)  to get actual size of plan after serialization.
Print infomation in IOException message

Add some test in testOverSizedWAL to verify the actual size of plan

If wal_buffer_size is less than actual size, it would throw IOException.
If wal_buffer_size is more than or equal to actual size, it would not throw IOException.

Coverage increased (+0.09%) to 66.983% when pulling 6093d100155a08e0ef85a843b98ce7382be7a55e on Cpaulyz:fix/wal_buffer_size into b5ef4d8bb5e1082ff55345cdb5e728fd1a7c2fc9 on apache:master.


Unable to launch ES service

The module works great until it tries to launch the elasticsearch instance that has been defined in the puppet file.
My puppet config:
class { 'elasticsearch':
	manage_repo  => true,
	repo_version => '1.5',
	autoupgrade => true,
	status => 'running'
}

elasticsearch::instance { 'es-01': }

When trying to run this on a fresh Debian (8 Jessie) machine - but also happened on Debian 7 machines - I get the following error:
Error: Could not start Service[elasticsearch-instance-es-01]: Execution of '/usr/sbin/service elasticsearch-es-01 start' returned 6: Failed to start elasticsearch-es-01.service: Unit elasticsearch-es-01.service failed to load: No such file or directory.

I'm using puppet in a Vagrant box so the message might be slightly different for someone who is using puppet directly.

Hi,
Support for debian 8 is coming. Debian 7 should work fine as my own tests are running nicely with it so that's weird.
In Debian 7 we still use the old init script, in debian 8 we are moving to systemd.

Cool got it.
I must have missed something. Service just launched fine on Debian 7. Apologies.

No worries, can happen :-)


try to stabilize a flaky foxx test

Scope & Purpose
Try to stabilize a flaky foxx test.
Ideally, this could fix https://arangodb.atlassian.net/browse/BTS-268
Intentionally no CHANGELOG entry has been made, as this is just a (potential) test fix.
devel port will be done too.

[x] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[ ] :book: CHANGELOG entry made

Backports:

[x] No backports required

Related Information

[x] GitHub issue / Jira ticket number: https://arangodb.atlassian.net/browse/BTS-268

Testing & Verification

[x] This change is already covered by existing tests, such as shell_server.

Link to Jenkins PR run:
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13854/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13861/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/14017/

Tests blue


Cannot find symbol cannot find symbol:  method toArrayList()

React-native 0.50.4,   react-native-admob 2.0.0-beta.3,
When trying to build with react-native run-android:
I get 5 errors likes this (for each file in /rnadmob):
/node_modules/react-native-admob/android/src/main/java/com/sbugert/rnadmob/RNAdMobInterstitialAdModule.java:117: error: cannot find symbol         ArrayList<Object> list = nativeArray.toArrayList();                                             ^   symbol:   method toArrayList()   location: variable nativeArray of type ReadableNativeArray
This happened after an update from 1.3.2 to  react-native-admob 2.0.0-beta.3 and an update of react-native from 0.45 to 0.50.
Am I doing something wrong?
I ran react-native link, cleaned cache etc.
Please advise.
Any advice or suggestion will be greatly appreciated.
I added a question on stackoverflow too:
https://stackoverflow.com/questions/47887443/android-react-native-admob-error-cannot-find-symbol-toarraylist
IOS building works ok.

@fdobre - I encountered this same issue and just rebuilt the node_modules and android folders.
$ rm -rf node_modules
$ rm -rf android
$ yarn
$ react-native link
$ yarn android

Rebuilding all of that seemed to fix everything.

I tried removing node_modules and re-linking but it didn't work. When downgrading to 1.3.2 the error disappears.
I also tried to upgrade RN to 0.51.0 and the issues persists in this version too.

@fdobre I think your android project still have the old linking.
You should open your project in android studio and check your main activity class
My answer to you on stackoverflow

I'm having this error too<EMAIL_ADDRESS>

any update


Code quality badge

Hi, your code quality is not bad. I wonder whether you'd like to track you code quality with codebeat? :)

Duplicate of #235


Microsoft CodeXGlue

Datasets from https://github.com/microsoft/CodeXGLUE
This contains 13 datasets:
code_x_glue_cc_clone_detection_big_clone_bench
code_x_glue_cc_clone_detection_poj_104
code_x_glue_cc_cloze_testing_all
code_x_glue_cc_cloze_testing_maxmin
code_x_glue_cc_code_completion_line
code_x_glue_cc_code_completion_token
code_x_glue_cc_code_refinement
code_x_glue_cc_code_to_code_trans
code_x_glue_cc_defect_detection
code_x_glue_ct_code_to_text
code_x_glue_tc_nl_code_search_adv
code_x_glue_tc_text_to_code
code_x_glue_tt_text_to_text

#978 is working on adding code refinement
maybe we should keep the CodeXGlue benchmark (as glue) and don't merge the code_refinement dataset ?

Hi @ncoop57 ! Thanks for your interest and sorry for the inactivity on this PR.
Sure feel free to create another PR to continue this one ! This one was really close to being merged so I think it won't require that much changes. In addition to my previous comments, there should also be a "Contributions" subsection (see the template of the README here)

Superseded by https://github.com/huggingface/datasets/pull/2357 .


Compiling a custom Op - Eigen including-loop

System Info.

Have I written custom code (as opposed to using a stock example script provided in TensorFlow)?: Yes
TensorFlow installed from (source or binary)?: binary
TensorFlow version: latest
Bazel version (if compiling from source):
CUDA/cuDNN version: 8.0/5.1
GPU Model and Memory: Quadro k1100m
Exact command to reproduce: trying to build

The problem
I'm tyring to build the code in
https://github.com/davidstutz/tensorflow-cpp-op-example
for Windows. The Windows version of Tf does not install with the source code so I cloned the repo and added the path to CMake. I then executed CMake as described in
https://github.com/tensorflow/tensorflow/tree/master/tensorflow/contrib/cmake
Yet the problem seems to be that the Eigen files don't do anything but re-including themselves.
Were they not meant to be included? Am I using some wrong flags that result in those imports? Other ideas? suggestions?
Cheers,

hi im new to this stuff not sure what im doing
Sent from Outlookhttp://aka.ms/weboutlook

From: Ido Freeman<EMAIL_ADDRESS>Sent: Friday, April 7, 2017 4:37:37 AM
To: tensorflow/tensorflow
Cc: Subscribed
Subject: [tensorflow/tensorflow] Compiling a custom Op - Eigen including-loop (#9042)
NOTE: Issues that are not bugs or feature requests will be closed. Please ask usage questions on StackOverflow.
 Yes
TensorFlow installed from (source or binary)?: binary
TensorFlow version: latest
Bazel version (if compiling from source):
CUDA/cuDNN version: 8.0/5.1
GPU Model and Memory: Quadro k1100m
Exact command to reproduce: trying to build

Describe the problem clearly
I'm tyring to build the code in
https://github.com/davidstutz/tensorflow-cpp-op-example
for Windows. The Windows version of Tf does not install with the source code so I cloned the repo and added the path to CMake. I then executed CMake as described in
https://github.com/tensorflow/tensorflow/tree/master/tensorflow/contrib/cmake
Yet the problem seems to be that the Eigen files don't do anything but re-including themselves.
Were they not meant to be included? Am I using some wrong flags that result in those imports? Other ideas? suggestions?
Cheers,
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/tensorflow/tensorflow/issues/9042, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZsqt-jk9vv5ama1WagUVE0H5oYanX_uks5rtiABgaJpZM4M2yhY.

@idofr : I'm not sure I udnerstand what you mean by "Yet the problem seems to be that the Eigen files don't do anything but re-including themselves." Could you please elaborate on what the problem is and the exact sequence of steps to reproduce it?
Separately, @gunan @av8ramit @mrry : Seems like the Windows pip package doesn't include the header files so the technique in https://www.tensorflow.org/extend/adding_an_op doesn't apply. Should we be including the header files now (or in release 1.2 onwards) since it seems that #8217 allows custom op libraries to be loaded in Windows?

@asimshankar That would make sense. Is there a canonical list of what header files are included? Given a list of filenames or globs, it should be fairly easy to add them to the PIP directory.
/cc @guschmue FYI.

sure, totally for this. I think the list of header files should be the same as the one in the linux pip.
There is maybe one more thing needed: building the dll makes a few compiler flags/defines and I wonder if we should generate a build.bat or a CMakeLists.txt template that we include in the pip so people don't need to deal with it ... should not be very hard to create this.

@avr8amit lets block 1.1 on the addition of missing headers into windows
pip package.
On Apr 7, 2017 11:17 AM, "Guenther Schmuelling"<EMAIL_ADDRESS>wrote:
sure, totally for this. I think the list of header files should be the same
as the one in the linux pip.
There is maybe one more thing needed: building the dll makes a few compiler
flags/defines and I wonder if we should generate a build.bat or a
CMakeLists.txt template that we include in the pip so people don't need to
deal with it ... should not be very hard to create this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/9042#issuecomment-292612639,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHlCOboRHXl3Mgj_vvf35LJs8yZqSDXGks5rtn3OgaJpZM4M2yhY
.

@mrry, @av8ramit : The list of header files to include would be defined by the bazel target //tensorflow/tools/pip_package:include_headers.
Perhaps we'll have to generate the list for CMakeLists.txt once and have some bazel test that checks that the two are always in sync.

Hi, sorry for the late respnose.
Thank you all so much for the replies, and yes, including the headers in the pip package would eventually save a lot of time, since the compilation takes a while. It would also probably make life a bit easier for everyone, as it would make sure that the same flags and configs are used in build builds.
@asimshankar
I mean something similar to
third_party/eigen3/unsupported/Eigen/CXX11/Tensor
( https://raw.githubusercontent.com/tensorflow/tensorflow/master/third_party/eigen3/unsupported/Eigen/CXX11/Tensor )
It does some minimal definitions and mostly just re-includes itself.
@vit-stepanovs was kind enough to reply to my email and shared the following link with me
https://gist.github.com/guschmue/2908e4411edc2faef6ddfe87a2ce4a1d
This seems to be exactly what I need (thanks @guschmue). Cmakes manages to create the VS project now and I'll compile either later on today or tomorrow at some point. I'll let you know how it went.
In case it's working though, it'd probably be a good idea to include this additional file into the main repo (assuming @guschmue won't mind)

Ah, the equivalent of
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/examples/adding_an_op/BUILD
for cmake ... sure. We eed to figure if this should be a version that depends on the source tree or if this should be standalone version that only depends on the python package once the header files are include in the python wheel.
For your https://raw.githubusercontent.com/tensorflow/tensorflow/master/third_party/eigen3/unsupported/Eigen/CXX11/Tensor issue: there should be a ifdef to prevent recursion but I think the main issue is that it is trying to include another Tensor from external/eigen_archive/unsupported/Eigen/CXX11/Tensor which is extracted during build. Maybe that 2nd Tensor file is not there, for example the tree wasn't build.

That Tensor issue was actually the reason I guessed to begin with that my flags were wrong. It basically seems like the file should not be included anyways. But yeah, I also added a few ifdef's there, just didn't submit a pull request for such a minor issue which probably doesn't affect the community anyways.

Is there anything special to do after the compilation to include the new Op?
I'm using the Ackermann_ops example, compiled the headers and then the Op and then tried to run the ackermann_test.py script.
It returns the following error though
`
ERROR: testBasic (main.AckermannTest)
Traceback (most recent call last):
File "C:/tf_compile/tensorflow/tensorflow/user_ops/ackermann_test.py", line 30, in testBasic
ackermann = tf.load_op_library(library_filename)
File "C:\Users\Me\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow\python\framework\load_library.py", line 64, in load_op_library
None, None, error_msg, error_code)
tensorflow.python.framework.errors_impl.NotFoundError: C:\tf_compile\tensorflow\tensorflow\user_ops\ackermann_ops.so not found
`
The actual Tf version I'm using was installed directly from the pip package

just tried this on my box and had the same issue. Silly mistake:
my example cmake copied to ackermann_ops.so where the test app was looking for ackermann_op.so. After fixing that it works on my box.

That was the first thing I've noticed but it didn't make a difference.
Do you link the entire path or just the file name?

just tried this on a fresh master. After the the normal build is done, I build with user op with
MSBuild /p:Configuration=RelWithDebInfo /p:BuildProjectReferences=false ackermann_ops.vcxproj
python \src\tensorflow\tensorflow\user_ops\ackermann_test.py
and
cd python \src\tensorflow\tensorflow\user_ops
python ackermann_test.py
both work ok.

@idofr : Are you able to build with the instructions from @guschmue ?

Building was not the problem, including the .so file was the issue

@idofr Did you use the resource loader? Did you try to load it by hand?

Automatically closing due to lack of recent activity. Since this issue is old at this point, please reopen the issue if it still occurs when tried with the latest version of Tensorflow. Thank you.

I have added the functionality to include headers in the cmake build.


[hotfix] default parcellation for allen mouse and waxholm mouse

use latest version (v3/4 for waxholm, 2017 for allen mouse) on selection

Fixed by https://jugit.fz-juelich.de/t.dickscheid/brainscapes-configurations/-/merge_requests/25
Waiting release


Suggestion for improvements and bugfixes

Description
I really like the careamics-napari but there are still some bug to make it usable. I had a look the code and created a fork to try and fix some bugs for multidimensional images.
Parts to improve:

Add batch size to prediction
Reshape the prediction to match the input axes
Enable patch_Z_spin when enable_3D is checked
Better handle the situation if the number of channels is not correct set in advanced settings

I would be happy to make a Pull Request. https://github.com/CURTLab/careamics-napari
Which part of the code?

Add batch size to prediction:

src/careamics_napari/signals/prediction_signal.py

src/careamics_napari/widgets/prediction_widget.py
src/careamics_napari/workers/prediction_worker.py

Reshape the prediction to match the input axes:

src/careamics_napari/training_plugin.py

Enable patch_Z_spin when enable_3D is checked:

src/careamics_napari/widgets/training_configuration_widget.py

Better handle the situation if the number of channels is not correct set in advanced settings:

src/careamics_napari/workers/training_worker.py

Examples
import napari
from careamics_napari import TrainPluginWrapper

viewer = napari.Viewer()
viewer.window.add_dock_widget(TrainPluginWrapper(viewer))
napari.run()

Potential solutions

Add batch size to prediction:

Add a spinbox to the tiled prediction (https://github.com/CAREamics/careamics-napari/commit/1eea399a892a69bd23fa941dc60e75e92cf9966b)

Reshape the prediction to match the input axes:

Add a _reshape_prediction function to training_plugin.py which handles the transformation (https://github.com/CAREamics/careamics-napari/commit/d4b2e908a7ac7d57c2e98dce3662e62dbf271e62)

Enable patch_Z_spin when enable_3D is checked:

Enable the spin box for z in the function _enable_3d_changed (https://github.com/CAREamics/careamics-napari/commit/cc6f8daa0201bdccaf145e8cd86077dbeac8d0b1)

Better handle the situation if the number of channels is not correct set in advanced settings:

Add a try block around the function create_configuration. A better way would be to notify the user using napari.utils.notifications (https://github.com/CAREamics/careamics-napari/commit/dd6f590c07dd69d4343d1cf3d3049e2456fb75a5)

Hi there,
Lovely!
Yes, there was big push to release the plugin for I2K, and there were plans to fix the obvious issues right after, but life caught up. I was planning to spend time fixing stuff over Christmas once the end of year craziness would calm down. Really appreciate you spending time on this.
Some thoughts following.
Batch size
Yes, totally useful! Currently, we didn't really see any improvement in speed with higher batch sizes. We think that there is an issue in Lightning creating overheads with batch_size>1. This was at the time of the tests and we have had no time to investigate further.
Absolutely mergeable directly!
Match input axes
Love this! S and T are a bit of an annoyance, but we could remember the original axe dimensions in the prediction signal object and apply the inverse mixing.
Mergeable right away!
Enable the 3D spin
Good catch! Mergeable right away as well.
The check for the configuration not being None is probably for mypy. We should investigate whether there is a point in having the configuration as Optional at all.
Originally, it was to be able to run the widget without string attached. But now it seems even the __main__ is creating the configuration.
Try catch clause
Agreed!
Regarding the napari notifications, there is a plan to make the code-base napari independent. Hence, why there are not many of these yet. Currently it is totally napari dependent, but you'll notive that it is done to a minimum level.

PRs are more than welcome!! Usually, we would ask for single PRs per topic. But everything I see is mergeable right away. So up to you, if you have multiple branches then open multiple PRs, otherwise a single one will do!
Thanks a lot!!

Hi,
I created a PR here: https://github.com/CAREamics/careamics-napari/pull/8
Happy to contribute since it is a very usefull tool!

Hi,
I found a critical bug in the configuration of tiles for 3D images: https://github.com/CAREamics/careamics-napari/commit/de6b3348d764cc7fc57af62591225a8e22e88a6c
Also I further tested the reshape_prediction method which needed some fixes and moved the function to src/careamics_napari/utils/axes_utils.py.
I added tests for mutiple combinations of axes: tests/test_predictions.py
https://github.com/CAREamics/careamics-napari/commit/2c7e2e7c1f52b00c4fb00ac251b2ec9f822a1044
How should we proceed?

Good catch!
Make a PR so I can have a look. Things are going to be slow over Christmas, but I'll try.
We usually use pytest for testing, although we did not spend the necessary time yet on writing tests for the plugin. So I'd rewrite your tests ultimately. But in a first step, I'm happy to merge it as in (I will need to have a closer look, but I'd do that in the context of a PR).
But the answer is PR, PR, PR! 😄
Thanks for spending the time!!


Adding SpotBugs as plugin causes JavaCC compile failure

Add the following line to the plugins {} section in Gradle
id "com.github.spotbugs" version '1.6.1'
No other changes in the gradle file, now a new task is generated: compileJavaccJava
This new task attempts to compile items not related to compileJavacc, and which are from directories that have external library dependencies that are not comprehended by the javacc source set:
sourceSets {
...
javacc {
java {
srcDirs 'code/src/javacc'
}
}
...
}
nor by:
compileJavacc {
inputDirectory = file('code/src/javacc/')
outputDirectory = file('code/src/java/')
}
The result is a compile failure.
The full repository with the change in place (as well as some additional spotbugs configuration that is valuable to us but not necessary to reproduce the error) can be found here: https://github.com/thpr/pcgen-formula/tree/SpotBugs

Ha, sorry, wrong window - meant to send this to the spotbugs folks


How to use lidar_svm_detect package ?

Hi ,
Thanks for amazing work and support of this free software.
I would like to use your lidar_svm_detect package under perception module. However in code I see that you have given a path to svm.model but I cant see the path and model neither.
How can I make use of that package? , Do I need that svm.model to be able to use this package ?
Thank you for your help.

Please assign someone from lidar_detect package to help me with this issue

Currently, lidar_svm_detect package does not have any function.
Your contribution to lidar_svm_detect or another method for lidar classification would be greatly appreciated.


[Bug]: Pictures in the best trip packages section

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Pictures in the best trip packages section are not visible.
Expected behavior
i would like to add clear pictures
Add ScreenShots

On which device are you experiencing this bug?
Windows
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSOC'24 contributor
[X] I have starred the repository

Please allow me to work on this and assign me with the hecktoberfest , gssoc24-ext and level labels.


V0.3.0 release

Foolishly I blended these 2 PRs together. But once [#22] has merged, (so from e68526900b29ad98da4f17625865061c8b51c725) this will just be a couple of docs changes and preparation for a 0.3.0 release.
PR for:

#23

Get read for a v0.3.0 release!


Implement C++ exception hierarchies

Create the following exceptions (modeled after fpga_result):
[ ] invalid_parameter
[ ] resource_busy
[ ] not_found
[ ] no_memory
[ ] not_supported
[ ] no_driver
[ ] no_daemon
[ ] insufficient_priveleges
[ ] reconfiguration_error

Implemented and merged in #277


The future of this project

As I'm sure you're aware, this repository and it's packages are currently for the Guilded *user API. I have been given Early Access to the bot API in order to adapt my library to the new bot API. Seeing that selfbots aren't necessarily outlawed on Guilded, support for the user API verison of this library will continue.
With this new API access, I will now be starting work on the next branch of this repo, which will include the following breaking changes:

current @guildedjs/guilded.js package will be renamed to @guildedjs/guilded.js-selfbot.
new package will be published under @guildedjs/guilded.js that covers the bot API. Will include the same caching, ws, events, and other mechanics that the old package had.
@guildedjs/rest will be adapted to work for both the bot API and the user API (atm it's literally just a fancy node-fetch wrapper with retry logic, which I'll have to rework for the bot API to take into account ratelimit headers).
user API typings in @guildedjs/guilded-api-typings will be moved to a user directory in that package, and the bot API typings will be in a bot dir with the API versions being separated by dirs with the appropriate version number (e.x. v1/, v2/).

This process will take weeks, maybe even months. There is no planned package release date. I have full permission from Guilded employees to publish my work on the bot API on GitHub.

After months of work on support for the bot API in a separate repo, I've come to the conclusion that we will no longer support the user API once the first release of the bot API comes out. The above plan is now defunct, with all the already published packages planned to be replaced with their bot API counterpart upon release. I will not be archiving this repo, but only accepting security or bug fixes. I will not be working on more API coverage for this repo.


If i git clone this project, how can i install homebrew???

as title
cause i can‘t install homebrew by
/usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"
so i want to ask if where are other ways to install homebrew

https://docs.brew.sh/Installation#untar-anywhere

https://docs.brew.sh/Installation#untar-anywhere

Thanks!!!


Custom part types don't report when they use variables

I don't think this is true: I just made an instance of a custom part type and it reported a reference to a variable.


Automatically Merged PR

I have a 2 branch in my repo.

master
master_dev (Branch Created base on master)

I added code and created a new PR base on master_dev,  PR No. 1
merge message like: wants to merge 11 commits into master from master_dev
my friend changed code in the main master branch.
whenever I created new PR to saving master branch commit in my master_dev branch  PR No. 2
merge message like: wants to merge 3 commits into master_dev from master
But That time conflict code in 1 file, So I resolve conflict and click on Make resolve button.
after merged my PR No. 2 then automatically merged my main  PR No. 1

Hi, I believe this is a question about GitHub.com, the website. This issue tracker is for a command-line utility called hub, not the website itself. For reporting bugs and requesting features for GitHub the website, please visit https://github.com/contact. Thank you!


Updating plotting.py

Updating plotting.py with changes.

Looks like you need cmcrameri in the requirements


how to integrate game plugins from moodle

Hi there
I want to know the process for integration moodle plugins for gamification courses.
thanks

Hi @hassanshuja, check the examples in the modules folder for more information.


Generate Results & other UI

This change contains the addition of GenerateResults, some improvements to TrainModel, and some other proposed UI changes.
GenerateResults shows the A.I. bot making its predictions which are then used to calculate accuracy.  The data shown is actually from the source data table, making it more illustrative than strictly accurate, but this can be improved if desired.
TrainModel and GenerateResults now show values being "plucked" out of the table, rather than scrolling the table, in an attempt to make it clearer that the box that moves across the screen is a single row from the source table.  Fades are used in an attempt to make it clearer that the data processing shown is just a subset of the overall process, which might involve hundreds or thousands of rows.
Some other UI work is presented here for consideration:

The DataCard no longer shows all the column names and descriptions.  We received feedback that this information was overwhelming on the first screen, and the information can be discovered on the subsequent screens anyway.
The DataTable now only shows the first 100 rows of data, with a small message explaining if this is the case.  This significantly improves UI performance, since the UI code was never optimised to cope with hundreds or thousands of rows of data.  All data processing continues to use the full data sets.
TrainModel can now be skipped immediately.  The three second delay was arbitrary; we now have the overlay to slow the viewer down, and the animation has a fixed duration for viewers who want to keep watching.
Instead of the Statement, TrainModel and GenerateResults now show specific headings to draw attention to what these screens are doing.
TrainModel no longer has text in its background image, which would have been difficult to localise.

https://user-images.githubusercontent.com/2205926/118765520-0a8ebe00-b830-11eb-9544-762b1fe66b5f.mov

When I run through the flow once, my results show twice - once as the top result and then in the previous results section. I think of previous as exclusive of current. Was that intentional?

Doesn't need to be part of this PR, but noting for the next style pass...

I'm a big fan of the bot head closing on the training screen, gives a nice sense of completion to the process!
Wondering about the "Testing Accuracy" heading, maybe "Calculating Accuracy" or "Testing the model" ?
On the label selection screen, what do we think about truncating the statement to just "Predict..." then adding "based on..." when we get to feature selection? Having lingering ... makes it feel like I haven't done everything I need to on that screen, but I have.
The bot jumps up as we transition from training to testing. Maybe it's fine, but wondered if the bot could be positioned similarly on both screens?  🤷♀  Or maybe the testing bot doesn't appear until the instructions are dismissed?
Maybe another Bot opportunity in the "Try it out!" section. Instead of the little grey box, we could re-use the testing beam?

I don't think we need pixel-perfect consistency between AI Lab and Oceans, but noticed that the beam strength and width differs. My only real gripe here is that I think we could make the beam color stand out a nudge more on the gray background.

I tried the flow with many features selected, and it worked pretty well:

Except things get a little squishy on the results table, which we could tidy up a bit.

I still had a pretty wide screen though; long rows could be cut off on smaller screens. I know @moneppo was looking at mobile, so wanted to note that we should check models with many features on smaller screens as part of that.

Now that we removed all of the column options from the DataCard (which feels like a good call!) I think we could bump the font size - feels tiny right now.

Wondering about the "Testing Accuracy" heading, maybe "Calculating Accuracy" or "Testing the model" ?

On the label selection screen, what do we think about truncating the statement to just "Predict..." then adding "based on..." when we get to feature selection? Having lingering ... makes it feel like I haven't done everything I need to on that screen, but I have.
Maybe another Bot opportunity in the "Try it out!" section. Instead of the little grey box, we could re-use the testing beam?

Just wanted to +1 each of these. I like "Testing the Model" vs "Testing Accuracy" - that feels easier for a teacher or student to explain over-the-shoulder if a student is confused on what's happening.
Also, adding AI Bot to the "Try it out!" feels like a good move. We got lots of feedback that having AI Bot be more visible in the process felt good to students, so this seems to meet that feedback.

Thanks @Erin007 and @dancodedotorg.  I've fixed the double generation of results, and will review all the UI feedback next since that's my focus for the next two weeks.

Looks great! I like how the animation on the TrainModel screen has a fixed duration. I also appreciate the improved UI performance!


UI - Mapping large amounts of points

As you well know, the initial view loaded in the interface is point data. However, when you plot thousands of points on a map, the user is faced with a mass of data points, which can be a little overwhelming.
For example:

How can this be dealt with? Options - load high point volume datasets/layers as either heat map or clustered view? This may confuse the user further, though as it has to be made explicit when the change is made.
So you may want to ignore this question :)
Dan

Hi Dan,
fully agreed on a good solution to the display of large datasets. One "hack", of course, is that you can force the UI into a specific initial state through the URL. E.g. while...
https://britishlibrary.github.io/peripleo-lanc/visitplus
... will open the map in points mode, this URL will open in heatmap mode:
https://britishlibrary.github.io/peripleo-lanc/visitplus/#/?/?/?/mode=heatmap
So, at least, there's a way for the host app to control initial state. But even despite of that, I think there's room for new ideas around better presentation in general.

The URL param is a good fudge fix for most people 😀


MQTT Daemon Not Connecting to MQ Broker

Yi-Hack-Allwinner Version: 0.1.2
Description:
Using the web interface to configure the MQTT then rebooting the camera does not show that the camera is connecting to the MQ Broker.
In this case, I'm using Mosquitto MQ running on a Raspberry Pi under Docker. Connecting to port 1883.
Checking the Mosquitto logs, I don't see a connection attempt to the broker.

Log in with ssh and run the program manually:
root@yi-hack:~# killall mqttv4
root@yi-hack:~# LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/home/yi-hack/lib
root@yi-2614:~# /home/yi-hack/bin/mqttv4

You should see a log like this:
Starting mqttv4 v0.0.3
Trying to connect... Client yi-cam-a sending CONNECT
Client yi-cam-a received CONNACK (0)
. connected!

Check if the client id is correct.
If you have more than 1 cam you must use different names otherwise the server will drop the connections.

root@xiaoyi:~# killall mqttv4
killall: mqttv4: no process killed
root@xiaoyi:~# LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/home/yi-hack/lib
root@xiaoyi:~# /home/yi-hack/bin/mqttv4
Starting mqttv4 v0.0.3
Can't open file "/home/yi-hack-v4/etc/mqttv4.conf": No such file or directory
Cannot open config file. Skipping.
Trying to connect... Unable to connect (Connection refused).
Unable to connect (Connection refused).
Unable to connect (Connection refused).
Unable to connect (Connection refused).
Unable to connect (Connection refused).
Unable to connect (Connection refused).

As discussed in Discord, it looks like a missing space in the setup script. There's no symlink between yi-hack-v4 and yi-hack.

Fixed here:
https://github.com/roleoroleo/yi-hack-Allwinner/commit/0f22892df1b9ad2e0ed80f1115df70d148b68b79

Issue fixed in 0f22892, closing issue.


npm package

Added changes for exporting Squire as npm package.
TODO:

Choose a name for this package on npm as squire is taken
Change the version in package.json if you want to bump the version

Steps to publish:
Change directory into your project root
npm adduser # follow the prompt after this command
npm publish

ref #75

:+1: Was just about to write something like this.

I would suggest the name squire-editor.

i would suggest squire-rte to match bower

Agreed, consistency is good.

:+1: for squire-rte for consistency.
If @neilj want to get the same name as what he stated in package.json, we could resort to contact the author see if he still uses the module that named squire on npm. Based on the stats on npm and GitHub, the project doesn't seem to be active anymore, maybe we could ask the existing owner to give way.  Refer to Package Name Disputes guide on npm on how to do it.
Also, I could squash the whole PR into 1 commit if you guys think it is better.

Published as squire-rte: https://www.npmjs.com/package/squire-rte


reintroduce godef

godef is much faster than guru in my project
GoTools: DEBUG: godef time: 0.03 seconds
GoTools: DEBUG: guru time: 2.13 seconds
code copied from GoTools so no credit to me

Hah! When this patch https://go-review.googlesource.com/#/c/22526/ lands, guru will be as fast as godef. So feel free to revert this if you want.
godef time: 0.0670468807220459 seconds
guru time: 0.09794092178344727 seconds

@ottob Great, thanks for the notice :) I've not publish the release, so we can wait a while util this patch is merged.

Already merged into Guru.


Misalignment between pagination documentation

This page
https://learn.microsoft.com/en-us/graph/paging?tabs=csharp
and this page
https://learn.microsoft.com/en-us/graph/sdks/paging?tabs=csharp
provide conflicting information regarding how to implement paging of Graph data in C# code.
https://learn.microsoft.com/en-us/graph/paging?tabs=csharp mentions using the "SkipToken" property which doesn't exist in the SDK, whereas https://learn.microsoft.com/en-us/graph/sdks/paging?tabs=csharp mentions using the PagingIterator.

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

ID: c7e1ac6d-b2ee-c495-3194-f2895e26945e
Version Independent ID: 73f817df-a829-e265-c932-7ae8b3f3e918
Content: Paging Microsoft Graph data in your app  - Microsoft Graph
Content Source: concepts/paging.md
Product: non-product-specific
Technology: microsoft-graph
GitHub Login: @MSGraphDocsvTeam
Microsoft Alias: MSGraphDocsVteam

Hi @jasonjoh - Can you assist here? Thanks.

Yes, I'll clean this up and make them consistent.

This update should be published shortly.


Use Variance library to calculate tolerances in tests

This change is

Current dependencies on/for this PR:

main

PR #1311   👈

This comment was auto-generated by Graphite.


Tokens are slightly offset on hexgrids when using TokenPlacement#place

Either a bit too low or a bit too far to the right, depending on the hex grid type.

(Probably a core issue fixed in v12 already idk, but logging it anyway to make sure.)

Issue persists in V12, but maybe there is some new API that we can use.

Just this in TokenPlacement##onMovePlacement seems to work for me.
    const center = canvas.grid.getTopLeftPoint(point);
    if (canvas.grid.isGridless) {
      center.x -= Math.round((this.config.tokens[idx].width * canvas.dimensions.size) / 2);
      center.y -= Math.round((this.config.tokens[idx].height * canvas.dimensions.size) / 2);
    }
    preview.updateSource(center);


Dev Lite Test: Center and Variability 1.1.0-dev.12

Dev-Lite Test
Mentions: @samreid, @marlitas, @catherinecarter
Simulation links

github repository for issues
phet top-level directory
sim: all_phet.html

Test Matrix

[x] Tester 1 (Tester = NS , Platform = mac + safari, Time = 2hrs {max 2 hours}
[x] Tester 2 (Tester = JD, Platform = win 11 chrome , Time = 2 hrs
[x] Tester 3 (Tester = KW, Platform = Win 11 Chrome, Time = 2 hrs

Features included

 PhET-iO
[x] Alternative Input
[x] UI Sound
 Sonification
Description
Voicing

Focus and Special Instructions
The purpose of the test is to do preliminary testing prior to interviews. We only need testing on PhET-brand, including UI sound. All three screens of the sim include alternative input, but alternative input testing should only be done on the Mean & Median Screen, and the Variability Screen. This sim will go through a more rigorous dev test later on when all issues in the repo are addressed and we are ready to take the sim towards publication.
Issues to Verify
No issues to verify

For QA...

General features

What to Test

Click every single button.
If there is sound, make sure it works.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.
Try to include browser version numbers
If there is a console available, check for errors and include them in the Problem Description.
Run through the string tests on at least one platform, especially if it is about to go to rc.
Check the Game Up harness on one platform.

PhET-iO features

What to Test

Make sure that public files do not have password protection. Use a private browser for this.
Make sure that private files do have password protection. Use a private browser for this.
Make sure standalone sim is working properly.
Make sure the wrapper index is working properly.
Make sure each wrapper is working properly.
Launch the simulation in Studio with ?stringTest=xss and make sure the sim doesn't navigate to youtube
For newer PhET-iO wrapper indices, save the "basic example of a functional wrapper" as a .html file and open it. Make
sure the simulation loads without crashing or throwing errors.

Accessibility features

What to Test

Specific instructions can be found above.

Make sure the accessibility (a11y) feature that is being tested doesn't negatively affect the sim in any way. Here is
a list of features that may be supported in this test:

Alternative Input
Interactive Description
Sound and Sonification
Pan and Zoom
Mobile Description
Voicing

Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.
Test all keyboard navigation inputs (if applicable).
Test all forms of input with a screen reader (if applicable).

Screen Readers
This sim may support screen readers. If you are unfamiliar with screen readers, please ask Katie to introduce you to
screen readers. If you simply need a refresher on screen readers, please consult the
QA Book, which should have all of the information
you need as well as a link to a screen reader tutorial made by Jesse. Otherwise, look over the a11y view before opening
the simulation. Once you've done that, open the simulation and make sure alerts and descriptions work as intended.

Platforms and Screen Readers to Be Tested

Windows 10 + Latest Chrome + Latest JAWS
Windows 10 + Latest Firefox + Latest NVDA
macOS + Safari + VoiceOver
iOS + Safari + VoiceOver (only if specified in testing issue)

Critical Screen Reader Information
We are tracking known screen reader bugs in
here. If you find a
screen reader bug, please check it against this list.

Keyboard Navigation
This sim supports keyboard navigation. Please make sure it works as intended on all platforms by itself and with a
screen reader.
Magnification
This sim supports magnification with pinch and drag gestures on touch screens, keyboard shortcuts, and mouse/wheel
controls. Please test magnfication and make sure it is working as intended and well with the use cases of the
simulation. Due to the way screen readers handle user input, magnification is NOT expected to work while using a screen
reader so there is no need to test this case.

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

Calling QA here

There is a new dev test for this, so closing


Dtype inference failing

What happened:
Running my code after upgrading to V2021.07.0 from V2021.06.2 gives me an error stating "Usually this is due to dask's dtype inference failing".
What you expected to happen:
The code should be able to run as before without issues.
Minimal Complete Verifiable Example:

#Line within the method resulting in traceback
Tickers = Dataset["ticker"].unique().compute().to_numpy()

  File "C:\Users\Ron_c\Desktop\Projects\Main.py", line 83, in ExtractTickers
    Tickers = Dataset["ticker"].unique().compute().to_numpy()

  File "C:\Users\Ron_c\anaconda3\envs\DeepLearning\lib\site-packages\dask\base.py", line 286, in compute
    (result,) = compute(self, traverse=False, **kwargs)

  File "C:\Users\Ron_c\anaconda3\envs\DeepLearning\lib\site-packages\dask\base.py", line 568, in compute
    results = schedule(dsk, keys, **kwargs)

  File "C:\Users\Ron_c\anaconda3\envs\DeepLearning\lib\site-packages\dask\threaded.py", line 79, in get
    results = get_async(

  File "C:\Users\Ron_c\anaconda3\envs\DeepLearning\lib\site-packages\dask\local.py", line 514, in get_async
    raise_exception(exc, tb)

  File "C:\Users\Ron_c\anaconda3\envs\DeepLearning\lib\site-packages\dask\local.py", line 325, in reraise
    raise exc

  File "C:\Users\Ron_c\anaconda3\envs\DeepLearning\lib\site-packages\dask\local.py", line 223, in execute_task
    result = _execute_task(task, data)

  File "C:\Users\Ron_c\anaconda3\envs\DeepLearning\lib\site-packages\dask\core.py", line 121, in _execute_task
    return func(*(_execute_task(a, cache) for a in args))

  File "C:\Users\Ron_c\anaconda3\envs\DeepLearning\lib\site-packages\dask\optimization.py", line 969, in __call__
    return core.get(self.dsk, self.outkey, dict(zip(self.inkeys, args)))

  File "C:\Users\Ron_c\anaconda3\envs\DeepLearning\lib\site-packages\dask\core.py", line 151, in get
    result = _execute_task(task, cache)

  File "C:\Users\Ron_c\anaconda3\envs\DeepLearning\lib\site-packages\dask\core.py", line 121, in _execute_task
    return func(*(_execute_task(a, cache) for a in args))

  File "C:\Users\Ron_c\anaconda3\envs\DeepLearning\lib\site-packages\dask\dataframe\io\csv.py", line 123, in __call__
    df = pandas_read_text(

  File "C:\Users\Ron_c\anaconda3\envs\DeepLearning\lib\site-packages\dask\dataframe\io\csv.py", line 178, in pandas_read_text
    coerce_dtypes(df, dtypes)

  File "C:\Users\Ron_c\anaconda3\envs\DeepLearning\lib\site-packages\dask\dataframe\io\csv.py", line 282, in coerce_dtypes
    raise ValueError(msg)

ValueError: Mismatched dtypes found in `pd.read_csv`/`pd.read_table`.

+---------+---------+----------+
| Column  | Found   | Expected |
+---------+---------+----------+
| siccode | float64 | int64    |
+---------+---------+----------+

Usually this is due to dask's dtype inference failing, and
*may* be fixed by specifying dtypes manually by adding:

dtype={'siccode': 'float64'}

to the call to `read_csv`/`read_table`.

Alternatively, provide `assume_missing=True` to interpret
all unspecified integer columns as floats.

Anything else we need to know?:
Prior to updating to V2021.07.0, the code was able to run without any issues and the values within the "siccode" column of my dataset are int64 instead of float64.
Environment:

Dask version: 2021.07.0
Python version: 3.9.5
Operating System: Windows 10 Home 64bit
Install method (conda, pip, source): Conda

Thanks for raising an issue @LazyTrader. Are you able to provide a minimal reproducible example (see https://blog.dask.org/2018/02/28/minimal-bug-reports)? That would make it possible for others to help debug.
Alternatively, you could try running git bisect on the commits between the 2021.06.2 and 2021.07.0 releases to see where things changed.
Just to confirm, reading the same CSV file and version of pandas but with different versions of dask (specifically dask=2021.06.2 and dask=2021.07.0) yields different dtype inference behavior?

Here's the minimal reproducible example:
import pandas as pd

DF = pd.DataFrame({"table": ["SF1", "SF1", "SF1"],
                   "permaticker": [196290, 124392, 122827],
                   "ticker": ["A", "AA", "AAAB"],
                   "name": ["Agilent Technologies Inc", "Alcoa Corp", "Admiralty Bancorp Inc"],
                   "exchange": ["NYSE", "NYSE", "NASDAQ"],
                   "isdelisted": ["N", "N", "Y"],
                   "category": ["Domestic Common Stock", "Domestic Common Stock", "Domestic Common Stock"],
                   "cusips": ["00846U101", "013872106", "007231103"],
                   "siccode": [3826, 3334, 6022]})

DF.to_csv(<Your Filepath Here>)

import dask.dataframe as dd

DF = dd.read_csv(<Your Filepath Here>)

Tickers = DF["ticker"].unique().compute().to_numpy()

Yes, can confirm that reading the same CSV file and using the same version of pandas but with different versions of dask (specifically dask=2021.06.2 and dask=2021.07.0) yields different dtype inference behavior. Codes are running fine after downgrading to 2021.06.2.

Thanks for the nice example snippet @LazyTrader, that's helpful. Unfortunately, I wasn't able to reproduce the issue using dask=2021.07.0 and pandas=1.3.0 (the latest pandas release). Here is the exact snippet I'm using locally (it's the same as your example, but also prints the pandas and dask versions being used):
import pandas as pd
print(f"{pd.__version__ = }")

DF = pd.DataFrame({"table": ["SF1", "SF1", "SF1"],
                   "permaticker": [196290, 124392, 122827],
                   "ticker": ["A", "AA", "AAAB"],
                   "name": ["Agilent Technologies Inc", "Alcoa Corp", "Admiralty Bancorp Inc"],
                   "exchange": ["NYSE", "NYSE", "NASDAQ"],
                   "isdelisted": ["N", "N", "Y"],
                   "category": ["Domestic Common Stock", "Domestic Common Stock", "Domestic Common Stock"],
                   "cusips": ["00846U101", "013872106", "007231103"],
                   "siccode": [3826, 3334, 6022]})

DF.to_csv("test.csv")

import dask.dataframe as dd
import dask
print(f"{dask.__version__ = }")

DF = dd.read_csv("test.csv")

Tickers = DF["ticker"].unique().compute().to_numpy()

which outputs
pd.__version__ = '1.3.0'
dask.__version__ = '2021.07.0'

without any error or warning.

Might be an issue between pandas = 1.2.5 and dask = 2021.07.0 as I am not using pandas = 1.3.0. The error might be reproduceable if you're using pandas = 1.2.5.

Unfortunately I don't see the dtype inference issue with dask=2021.07.0 + pandas=1.2.5. The only difference if the printed version numbers
pd.__version__ = '1.2.5'
dask.__version__ = '2021.07.0'

Have you confirmed that you get the error when using the exact script that James provided? I seems to me that you might have a null value in the siccode column in which case you need to include assume_missing=True in read_csv

Closing for now due to lack of response. @LazyTrader feel free to re-open if you'd like to continue


Student t distribution: beta parameter in gamma

I think the beta parameter in gamma sampling for student t distribution should be 2 instead of 0.5 :
https://github.com/tensorflow/tensorflow/blob/r1.11/tensorflow/python/ops/distributions/student_t.py
gamma_sample = random_ops.random_gamma(         [n],         0.5 * df,         beta=0.5,         dtype=self.dtype,         seed=distribution_util.gen_new_seed(seed, salt="student_t"))

Hi,
Please check pull request #23301

@missmagnum  - Hi, a PR has been raised for this and please wait until it's merged.

Nagging Assignee @harshini-gadige: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


Make Buttercup UI isomorphic

It should be possible to use this module in React Native and usual React. Something like this should be used to make this possible:
https://github.com/lelandrichardson/react-primitives

So I came from https://github.com/buttercup/buttercup-mobile/issues/23, I really want that feature implemented if possible, and I'd like to work on it if possible.
Could someone give me a guide as to what would need to be worked on?


Running p0d against google.com (ipv4) results in 0 connections counted on OSX, despite requests counting up.

this only happens on M1 mac so far.

only happens on tethered phone, why? are our HTTP connections killed by 5G?

closing this, only tethered conns on 4G/5G.


Apparent DNS failure in Docker image alpine:3.8, nslookup: can't resolve '(null)'

Seeing odd DNS behavior in Docker image alpine:3.8. I'm baffled that nslookup complains yet finds the IP address.  In comparison, ping works perfectly. See below.  This little test runs a docker container to resolve the name of the host VM. If I can get that working the next test will be to have the docker container resolve the name of other running containers.
Traced this back from behavior in an OpenJDK image, in which Java cannot resolve host names.  I'd really prefer to use an Alpine version of a Java/JRE image, it's half the size of a non-Alpine (debian) Java/JRE image, but this network glitch is kind of a killer.
So far I've run this test under a plain Ubuntu VM running docker 17.05.0-ce and under Kubernetes running docker version 18.09.1. Same behavior in both. I know there are many external variables that might affect this so it might not be an Alpine issue at all, altho issue #255 sure seems to be related.
Would someone possibly take a minute to explain please? Thanks in advance.
me@host-dev1-vm01-core:~$ docker run alpine:3.8 nslookup host-dev1-vm01-core

nslookup: can't resolve '(null)': Name does not resolve
Name:      host-dev1-vm01-core
Address 1: <IP_ADDRESS>
me@host-dev1-vm01-core:~$ docker run alpine:3.8 ping host-dev1-vm01-core
PING host-dev1-vm01-core (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: seq=0 ttl=64 time=0.061 ms
64 bytes from <IP_ADDRESS>: seq=1 ttl=64 time=0.133 ms
^C
--- host-dev1-vm01-core ping statistics ---
2 packets transmitted, 2 packets received, 0% packet loss
round-trip min/avg/max = 0.061/0.097/0.133 ms

Please, include this fix in all alpine versions https://github.com/kubernetes/kubernetes/issues/56903#issuecomment-409603030

@jcperezamin - I think that's a different issue. Also, I think that'd just be hacking in a workaround for the underlying issue, which is a linux kernel bug to be fixed in v5.0 - https://github.com/torvalds/linux/commit/4e35c1cb9460240e983a01745b5f29fe3a4d8e39

yes, you can ignore the alpine (busybox) nslookup outputting message "... can't resolve '(null)'", it's unnecessary, if you set the second param as a NS, the msg would be gone away:

# nslookup wx.qlogo.cn <IP_ADDRESS>
Server:    <IP_ADDRESS>
Address 1: <IP_ADDRESS>

and the relative busybox codes (e.g. alpine3.8 is using http://busybox.net/downloads/busybox-1.28.4.tar.bz2 , networking/nslookup.c, function nslookup_main) and the debugging logs with strace showed that the program tried the PTR qurey (reverse DNS) for the NS server first of all, you got that msg because of the empty global NS server at the initialized time.

my codes implements (workaround https://github.com/kubernetes/kubernetes/issues/56903#issuecomment-409603030) removed the default behavior to decrease that dns timed out issue, at the docker image tier, please consider the pros and cons (it worked well for me, the nodejs/java/go/c++/python application that no needs ipv6), and the perfect solution to solve the dns timed out would be others (kernel conntrack racy fixes, local node dns cache ).

I have been running most of applications (java/go/nodejs/c/c++/python) based on my customized alpine docker for 1+ years.
and highly recommend running some debugging with strace, tcpdump on the docker, you would run docker on the privileged mode to get more system capability,
can you please post some strace / tcpdump logs here?
thanks,
harper

I have similar issue in our Kubernetes cluster when I try to get ip of the DNS server
/ # nslookup kube-dns.kube-system.svc.cluster.local
nslookup: can't resolve '(null)': Name does not resolve

Name:      kube-dns.kube-system.svc.cluster.local
Address 1: <IP_ADDRESS> kube-dns.kube-system.svc.cluster.local

fixed with
spec:
  template:
    metadata:
      labels:
        app: activityreservation
    spec:
      dnsConfig:
        options:
          - name: ndots
            value: "1"

refer to https://github.com/WeihanLi/ActivityReservation/blob/dev/reservation-deployment.yaml for details

@xbmono if you are complaining about "can't resolve '(null)'" please read my explanation at https://github.com/gliderlabs/docker-alpine/issues/476#issuecomment-457124840.  It is nothing to worry about. If something else, I suggest you open a new issue.

Having the same issue with Alpine 3.9. Is there any workaround for this?

I have the same problem with alpine:3.10

I have the same problem with alpine:3.10

Same for me..

same here node:alpine brings this problem

A temporary workaround which worked for me: try to create the docker container in host mode:
--network host


HTTP::Server OpenSSL/BindTLS Slow Read hangs

Summary
This problem will only appear in HTTP::Server[bind_tls].
The server could not close the client.
This can cause Slow HTTP attacks.
You can see the request hangs by lsof -p (ESTABLISHED).
If the client continues to sleep, the server will hangs.
Server context.response.flush and context.response.close do not work.
Is there any timeout feature? Or something else?
Lsof Check
ps ax | grep {{name}}
lsof -p {{pid}} | wc -l

Example

Server

require "http/server"

server = HTTP::Server.new do |context|
  context.response.content_type = "text/plain"
  context.response.puts "Hello World!\n"
  context.response.flush
  context.response.close
end

context = OpenSSL::SSL::Context::Server.new
context.certificate_chain = "server.pem"
context.private_key = "localhost.key"
context.verify_mode = OpenSSL::SSL::VerifyMode::NONE

tls = server.bind_tls "<IP_ADDRESS>", 8888, context

puts "Listening on #{tls}"
server.listen

Client

require "http/client"

tls = OpenSSL::SSL::Context::Client.new
tls.verify_mode = OpenSSL::SSL::VerifyMode::NONE

HTTP::Client.get("https://<IP_ADDRESS>:8888", tls: tls) do |response|
  body = response.body_io.as IO
  puts [body.gets_to_end]
  sleep 50000_i32
  puts "done"
end

Lsof

COMMAND  PID USER   FD     TYPE             DEVICE  SIZE/OFF    NODE NAME
slow    5564 User  cwd      DIR                1,2      1666  443611 /Users/User/Downloads
slow    5564 User  txt      REG                1,2<PHONE_NUMBER>452 /Users/User/Downloads/slow
slow    5564 User  txt      REG                1,2<PHONE_NUMBER>131 /usr/local/Cellar/openssl/1.0.2r/lib/libssl.1.0.0.dylib
slow    5564 User  txt      REG                1,2   1999820 5486128 /usr/local/Cellar/openssl/1.0.2r/lib/libcrypto.1.0.0.dylib
slow    5564 User  txt      REG                1,2    416976  422220 /usr/lib/libpcre.0.dylib
slow    5564 User  txt      REG                1,2    243708  640350 /usr/local/Cellar/libevent/2.1.8/lib/libevent-2.1.6.dylib
slow    5564 User  txt      REG                1,2    698896  422030 /usr/lib/dyld
slow    5564 User  txt      REG                1,2 662274048 3565842 /private/var/db/dyld/dyld_shared_cache_x86_64h
slow    5564 User    0u     CHR               16,4    0t6616    1125 /dev/ttys004
slow    5564 User    1u     CHR               16,4    0t6616    1125 /dev/ttys004
slow    5564 User    2u     CHR               16,4    0t6616    1125 /dev/ttys004
slow    5564 User    3u  KQUEUE                                      count=0, state=0xa
slow    5564 User    4r     CHR               14,1      0t32     574 /dev/urandom
slow    5564 User    5u     CHR               16,4       0t0    1125 /dev/ttys004
slow    5564 User    6u     CHR               16,4       0t0    1125 /dev/ttys004
slow    5564 User    7     PIPE 0xba70a02ebaf6582f     16384         ->0xba70a02ebaf6336f
slow    5564 User    8     PIPE 0xba70a02ebaf6336f     16384         ->0xba70a02ebaf6582f
-> slow    5564 User    9u    IPv4 0xba70a02eb2050937       0t0     TCP localhost:54283->localhost:ddi-tcp-1 (ESTABLISHED)

Why close? Seems legit (and interesting!)

@asterite It seems to have been solved suddenly, and then it again failed. :(
So I open it again.😓

According to the OP this happens only on an encrypted connection, plain sockets are not affected.
Usually, TCP timeouts should ensure slow clients are disconnected. But it seems that when piping the socket stream through OpenSSL this is not working correctly.

Here's how I set some timeouts in OpenSSL land: https://github.com/crystal-lang/crystal/issues/8108#issuecomment-523778920
Does setting a write timeout do it?


Structure panel should omit "let () = ..." items and should include "let (a,b) = ..." items

plugin version: 0.73

Top level lets in the form let () = ... should not appear in the structure panel since they don't have name.  For example:

Top level lets in the form let (a, b) = ... should appear in the structure panel with names a and b.  For example:

Code for the examples is here: https://github.com/coq/coq/blob/master/vernac/vernacentries.ml

Fixed in 0.74

Looks better on # 1, thanks.
I would have split let (f_interp_redexp, interp_redexp_hook) into 2 separate entries in the structure panel.  Both are top-level variables.  They just happen to both be defined in the same let.  If I'm looking for f_interp_redexp when the structure panel is alphabetized I'd expect to find it between focus_cmd_cond and get_current_context_of_args

Also, a different issue: I'd expect the opens to be alphabetized when the panel has alphabetic order selected.  And probably I'd intermix them with other symbols.

Fixed in 0.75

Looks good!


How do i load a custom MIB

I am trying to use a custom MIB, how i load a custom one using the library?

SNMP MIB documents are processed by #SNMP Pro only. More information can be found at http://pro.sharpsnmp.com


Proposal: Go2: try: Change try() proposal to use a special character in the position where an error is returned

It looks like #32437 is the current most popular error handling proposal.  This proposal wishes to introduce a try() statement that returns the default values of all return variables if a value other than nil is returned as the last value from a function.  In normal function, the try() statement returns all values that are not the final error value.
How about instead of wrapping things in parentheses all over the place, we directly put a special key (like !! or anything else) into the place where the error is returned?  That's shorter and all other behaviour is the same.  Also, you don't have invisible returns here, or require the error to be the last thing returned.
This also discourages changing try statements together which keeps code more readable.
func Foo() (err error) {
        file, !! = os.Open("file1.txt")
        defer file.Close()
	parsed, !! = parseFile(file)
	fmt.Printf("%s", parsed)
	return nil
}

This has been suggested many times. See #32500 and
https://github.com/golang/go/wiki/Go2ErrorHandlingFeedback#recurring-themes
It even appears a few times within the comments on try().

This seems quite cryptic for people unfamiliar with Go.  That's not a fatal problem, but it is a cost.

It was also suggested at #32601 which used ? as the special character though this was soon closed by the OP, presumably due to lack of support.
Although I personally prefer #32437 for the reasons stated there, I do think this is one of the better alternative proposals. Points in its favor include:

It does save significant boiler-plate and the syntax is quite pleasant.

It's backwards compatible since, apart from a named identifier (usually err), the only other symbol currently allowed in this position is _.

It's easy to change from this syntax to what we do currently do now which is useful for debugging purposes.

However, it is cryptic and those who object to try for causing an implicit function return would probably object even more to some symbol doing the same thing. On the other hand one thing those same people might like is that it automatically rules out nested expressions and the like.

I have 2 main concerns. This eliminates the ability for error decoration and it always implicitly returns. I often return an error, but sometimes I just log it or continue in a loop.

@mvndaai Having this feature does not preclude the standard if err != nil { pattern.  You could still use a control statement to decorate, retry, or whatever you like.

@integrii I agree that is doesn't stop the current pattern from working, but it does mean that there are two patterns and switching between them is a hassle.
My biggest issue with go errors currently is when I have if _, err := foo(); err != nil { and later decide that I need the variable what was _.  I have to then refactor to
a, err := foo()
if err != nil {

Which is annoying to do and my function scope gets polluted with a long-living err variable.
I am worried adding a third way to handle errors would mean I would end up doing even more refactors.

The err is better than symbols. I can see it is an error at first sight.

This is similar to other issues such as #32500 and #32601 which have been closed.  This proposal does not have strong support, and will be cryptic for people new to Go.  Therefore, this is a likely decline.  Leaving open for one month for final comments.

There were no further comments.


(maint) add name and roles tags to vmpooler vms

Before this commit it was not possible to tell via vmfloaty or vmpooler-bitbar what a particular vmpooler vm was being used for when created by beaker.
This commit adds tags with the name and roles for vmpooler vms, for example via vmpooler-bitbar:

fyi @briancain

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/3030/

please retest this

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/3032/

Oh nice 👍 , tagging with name and role is a great idea.

@briancain will be updating vmpooler-bitbar to detect nodes with the dashboard role and offer a link directly to it from the menu, also gonna replace the template in brackets with the name on the main dropdown.

@johnmccabe thank for this PR! I've tested this and found an edge case where if you define a host without any roles, there will be no roles key defined for that vm in the vmpooler. This seems more like an issue with the vmpooler and not beaker, so I don't think there are any changes needed for your PR in that respect. Just wanted to give you a headsup that the roles key is not guaranteed to exist for the vm after tagging.
Given that we are now on beaker 3.x on master, would you mind rebasing this PR with master? That will allow the testing to go through with beaker 3 and give us better confidence that the changes will work. Thanks!

hi @tvpartytonight yeah the absence of the roles key is fine (the johnmccabe/vmpooler-bitbar#1 pr expects that behaviour for example), I'll retarget this later tonight

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/3072/

@tvpartytonight thats rebased, I don't think the CI failure is related.

@puppetlabs-jenkins retest this please jabroni

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/3113/


Cannot use custom naming on an Input type that's also a regular Type

Hey.
I have a class with a name that I don't want to expose in GraphQL, that's an input and a regular type at the same type:
#[Type(name: 'Address')]
#[Input(name: 'Address', default: true)]
class AddressDTO {}

I'd expect this to work; or, at, least, this to work:
#[Type(name: 'Address')]
#[Input(name: 'AddressInput', default: true)]
class AddressDTO {}

Instead, I'm getting this error when introspecting: "Expected GraphQL type \"Address\" to be either a MutableObjectType or a MutableInterfaceType. Got a TheCodingMachine\\GraphQLite\\Types\\InputType" or this error when actually trying to run a query with that type: RuntimeException: Cached type in registry is not the type returned by type mapper.
I'll PR a fix tomorrow.

I wonder if default: true is the issue here.

Nevermind, the second option does work. I swear I've tested it 20 times before, clearing the cache after each try and it was broken.
On that note, is the first option even supposed to work? Graphqlite automatically adds the "Input" suffix when generating a name from the class name, but it probably doesn't make sense to do so when name is manually specified?

IIRC, you do need to specify the Input suffix, else you'll end up with a type name collision.  It's redundant though and the Input suffix is implied.


createUAC: Check for colon too in URI

Otherwise the mechanism for automatic URI completion would be useless for SIP server names starting with sip, like sip.linphone.org or sipfun.tld.

Can I get a quick estimate for the next release?

I just pushed it as<EMAIL_ADDRESS>


Add discussion of destructuring.

Fixes #31.

I added a commit to address the PR comments.

:+1:

@DanielRosenwasser can you take another look?

Can you sync with the master branch?

Sure, merged.

Apart from my current feedback, :+1:.


[5.15.1] Backport to 5.15.1

Commits added:
fc6060bad9d7f7f461302b39f3320c9c53f8deeb NSFS | NC | from_file in bucket add
ae6d42485c1cf46a736c15cfb480df42fa08f52a NSFS | NC | health giving trimmed output in pipe
8a29bc963a8ec33c8c80f58b4b19a10ecaf0c0bc NSFS | NC | CLI | Change the fs_context
f8d37bf64ef6f36a3a6bf4a5da3dffac5890663c CI | NSFS | NC | Jest test - rename tmp_fs_path for concurrent run
e315e741b3cc896268a71a7c9068d6b943bfecc3 NSFS | NC | fix get-bucket-website function for bucketspace fs
3f48984e75a0d5afe47b2b236e438be90bdc547c simplify GLACIER xattr + minor fixes
eedd390ac5bb86b8d78bf40243d572dad2b43b0e NSFS | CLI | Delete the bucket only if the bucket is empty or with --force
5ec2eaefc01e708d3e21828f7c5c3226f7135170 NSFS | NC | Symlinks should contain a relative path
8f0d2cc7b4e4147dd75c9a1c965bce96058483ba Restructure RPM Build process

[ ] Doc added/updated
[ ] Tests added

Converting to draft to avoid backport it until we bump the version

Need to get https://github.com/noobaa/noobaa-core/pull/7956 in first I guess


feat: split command

Looks better to me! Thanks. 🎉

I tried the split command and it didn't function as I had expected.
1. Error with missing required argument.
I wonder that the part under the  "Missing required argument: outDir"  should happen on an expected error case:
 npm run cli -- split users_api_bundled.yaml
><EMAIL_ADDRESS>cli /Users/adam/Projects/Redocly/openapi-cli
> ts-node src/cli.ts "split" "users_api_bundled.yaml"
cli.ts split [entrypoint]
Split definition into a multi-file structure
Positionals:
  entrypoint                                                            [string]
Options:
  --version  Show version number.                                      [boolean]
  --help     Show help.                                                [boolean]
  --outDir   Output directory where files will be saved      [string] [required]
Missing required argument: outDir
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>cli: `ts-node src/cli.ts "split" "users_api_bundled.yaml"`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>cli script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      /Users/adam/.npm/_logs/2020-10-10T01_41_47_696Z-debug.log

2. Splitting lost some content.
I used this file: https://api.redoc.ly/registry/rebilly/users/users/bundle/master/openapi.yaml
I split it. It lost the parameter organizationId (and a few other things) as designated in warnings.
See the parameter defined in the original bundled file:
  parameters:
    organizationId:
      name: Organization-Id
      in: header
      description: >-
        Organization identifier in scope of which need to perform request (if
        not specified, the default organization will be used)
      schema:
        $ref: '#/components/schemas/ResourceId'
      required: false

3. There were warnings (they highlight issue 2 above), and the English/grammar needs revisions:
warning: conflict for Unauthorized - Reference files for the file: users_api/components/responses/Unauthorized.yaml are not exist ... Skip.
warning: conflict for Forbidden - Reference files for the file: users_api/components/responses/Forbidden.yaml are not exist ... Skip.
warning: conflict for NotFound - Reference files for the file: users_api/components/responses/NotFound.yaml are not exist ... Skip.
warning: conflict for ValidationError - Reference files for the file: users_api/components/responses/ValidationError.yaml are not exist ... Skip.
warning: conflict for Conflict - Reference files for the file: users_api/components/responses/Conflict.yaml are not exist ... Skip.
warning: conflict for organizationId - Reference files for the file: users_api/components/parameters/organizationId.yaml are not exist ... Skip.
warning: conflict for integrationLabel - Reference files for the file: users_api/components/parameters/integrationLabel.yaml are not exist ... Skip.
🪓 Document: users_api_bundled.yaml is successfully split
    and all related files are saved to the directory: users_api
users_api_bundled.yaml: stats processed in 1127ms

I think stats processed in 1127ms meant to be split processed in 1127ms?
Change are not exist to does not exist.
I don't understand the phrase: Reference files for the file. Do you mean Reference to the file?

🎉


Problem with overwriteRoutes and migration from v5 to v6

In v5 it was OK to add same route more than one time with {times: 1} setting.
In v6 when overwriteRoutes:true is set route is always overridden, which means I can no longer add a number of same routes which will be picked up one by one.
Please show an example how same matcher can be added more than once. Doc mentions this but there's no example.

Try overwriteRoutes: false - it's supposed to support exactly the case you're describing.
Could the docs be improved to make it clearer?

It works. But the confusing factor is that by default it errors...
https://github.com/wheresrhys/fetch-mock/blob/master/docs/v5-v6-upgrade.md#routing
Here there is clearly lack of explanation. It should be like:

set to true to override
set to false to use old "buffer-like" behavior

Also in https://github.com/wheresrhys/fetch-mock/blob/master/docs/api.md there is no mentions of this option...


flag to run or skip all the optional migrations

Command
update
Description
when running ng update @angular/core@18 @angular/cli@18 a prompt appears  asking which migrations  would you like to run.

In my company the migrations are executed through a CI system, so there are no human interaction. We just run a .sh that runs the needed commands for the update.
However, if now is required to manually interact with the prompting, our system will  no longer work.
I think that it will be very desirable to have some flag that allows to run the update completely automatically, by running or skipping all the optional migrations
Describe the solution you'd like
Would be nice to have something like this:
ng update @angular/core @angular/cli --runOptionalMigrations=true
Describe alternatives you've considered
No response

Prompts are not displayed when the terminal is TTY. Typically on CI terminals are not. If for some reason your terminal is TTY, you can force disable TTY by using  the NG_FORCE_TTY=false environment variable.

the issue isn't that the prompt is not displaying, the issues is that isn't possible nor desirable for us, to interact with the prompting. we can't depend on human interaction to run the update, ence, the need for the flag to avoid that prompting

@alvaro-b-enguidanos, you can disable the prompts by using the NG_FORCE_TTY=false environment variable.
Alternatively, you can also configure your CI environment to disable TTY, which in general it’s recommended, if it’s not done by default.


build: extract project metrics/analytics to terasology-metrics gradle plugin

It didn't take long to run in to other parts of #4985; #5013 didn't include configuration for module builds.
That was happening in common.gradle, so it seemed like a good time to extract a chunk of that stuff to build-logic, in line with the other gradle plans of #4015.
How to test

run engine-tests locally: unitTest, integrationTest, test
run module tests locally: unitTest, integrationTest, test
run check locally for checkstyle and whatnot
make sure CI has all expected reports for checkstyle, spotbugs, pmd, jacoco

Side note: in the process of moving these configurations over, I discovered that the current line of gradle-spotbugs integration has a minimum requirement of gradle 7.0. So I've set us to use their latest release of the previous major version until we do that gradle upgrade.

I pushed this to Nanoware:experimental and ran some module builds there: https://jenkins.terasology.io/teraorg/job/Nanoware/job/TerasologyModules/job/X/job/Health/job/develop/
It looks good. We'll just need one ModuleJteConfig change; I've added it to this PR's description.

Yeah, once I figured out which parts to push to Nanoware, I agree that did a good job of exercising it.

I'm glad I did manage to set up the parallel testing infrastructure to help make that happen :-) Happy you find it useful!


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/OPS-E2E-PPE/docs-archive-test-source

Docs Build status updates of commit 35a4447:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-05-12-06-44-12 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-05-12-06-44-12

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add period(.) to improve readability

Added a period (.) to line 14 to improve grammar

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Period added to line 14

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


Laravel 5.2

In Laravel 5.2 I have this error:
Call to undefined method Illuminate\Database\Query\Builder::users()
Have anyone a idea?

#push

Same problem

Same here

Help!

5.2 doesn't seem to have the config items the package is looking for, try adding
'model' => App\User::class,
to the top of /pp/config/auth ( after return [ )
there is a pull request by another user that changes where it looks for the model, but that was submitted 3 months ago and still hasn't beed added

Hey guys, sorry for the late response. I have been busy, but I will push an update soon. L5.2 introduced a lot of changes, and unfortunately, it's a bit more than a simple config change. On the bright side, we no longer need to extend the database session handler, and it provides us with extra details like the ip address and user agent.
Also, user() simply defines the belongsTo relationship. Because of how eager loading works, that will still get all activities, including guests and users. users(), usersByMinutes(), usersBySeconds(), etc. are the proper methods as stated in the documentation to query only the necessary activities, based on the specified time constraints.
Anyway, I will try to push the new changes soon (when I get home tonight).

I've pushed support for 5.2. Please try updating to version 1.1 and then running composer update. :)


Various UX and content updates

For this PR, I've largely focused on UX updates and changes, with some additional content here and there. This PR introduces the following changes:

Created /lib/util/constants.js file, moving a ton of data from the /views/about/CoreTeamAbout.vue file to it, and serving the data programmatically on the page.
Fixed a bug where images weren't being properly loaded on a page refresh.
Added an icon to external links, providing more clarity to the user by informing them that the link is going to take them to an external page.
Updated the /about/sponsors page to include more info about the various services we use for the bot.

Should be good to merge pretty quickly, haven't seen any issues when testing on my end. Please refer to the Firebase preview for testing.

On the Sponsors & usage page, add links on the

icons as some users (me) may instinctively go to click it


Add S32K344 target support

This PR introduces initial support for the NXP S32K344 along with a new method on ArmDebugSequence to allow an implementation to provide an explicit list of valid access ports since this target hard faults when accessing invalid APs.
I'm drafting this to solicit feedback on the latter part, as this might be something already being considered by probe-rs that I wasn't able to find. This seemed like a natural place for it given that it's target specific and related to DP/AP feature discovery, but I was also trying to be surgical with the change. Other libraries also do a similar thing (i.e. pyOCD).
There are some drawbacks to having it be provided by ArmDebugSequence:

StlinkArmDebug does an AP scan early on, and doesn't have sequence at hand yet
Anything using DefaultArmSequence (like probe-rs info) won't have this information

Some notes about the S32K344 support:

The reference Keil pack was deprecated, and a few register fields don't line up with what's in the refman (Rev. 8, 01/2024).
I initially implemented the debug sequences based directly on the keil pack, but they didn't work.
I tried contacting NXP support about this, in order to figure out the discrepancies, but since I'm not an NXP partner they weren't able to provide any support :roll_eyes: .
I ended up wiring an SWD protocol analyzer up to observe what SEGGER's tools (JFlash/JLink) were doing. Most of the debug sequence implementation is based on that.
I removed SRAM_1 from the yaml file. It's likely that some part of the RAM init and/or clock setup should be implemented to enable access (I was able to access most of SRAM_1 but there were some holes...), but it was unclear to me after a couple of attempts, and it wasn't strictly necessary for my use case. I plan on looking into this more at some point.

That said, it's working as expected. I've testing all of the obvious things: flashing, memory access / RTT, resets, and breakpoints

Thanks, please ignore the smoke tester error.

I'm not sure if we should add a new function to limit access port access, maybe we could handle this with data in the target file? If we already know the access ports, not sure why we need that scan at all. We've had problems with the access port scan before, so it would be great to get this improved.

I'm not sure if we should add a new function to limit access port access, maybe we could handle this with data in the target file? If we already know the access ports, not sure why we need that scan at all. We've had problems with the access port scan before, so it would be great to get this improved.

I agree with you, @Tiwalun. I think the AP discovery could be omitted altogether if the targets could provide that data.


Can't use method = ipw in Windows

Hi guys,
I'm so sorry to raise another issue, but now I have balance successfully loaded in a Python 3.10 environment, I'm trying to run through the quick start example. I can load the datasets and set targets fine, but this line:
adjusted = sample_with_target.adjust(max_de=None)
fails with the error:
ValueError: loadGlmlib does not currently work for windows
Judging by your docs on the method it's probably something to do with glmnet-python which I installed from its GitHub repo.

I'm using Mac so wasn't aware of this.
I also see it in the glmnet code: https://github.com/hanfang/glmnet_py/blob/master/glmnet_py/loadGlmLib.py
I think this will get resolved once we move to sklearn, but the ETA for this is currently end of June.
cc: @stevemandala , @SarigT


Generic overload resolution fails in for-in statement

Describe the bug
When using a Collection method with multiple overloads in for-in loop inside a generic context, the compiler reports ambiguity where there is none.
To Reproduce
Attempt to compile the following code (also attached):
func f<C: Collection>(_ c: C) {
  for _ in c.prefix(1) {}
}

Compilation fails with this error message:
test.swift:3:12: error: ambiguous use of 'prefix'
  for _ in c.prefix(1) {}
           ^
Swift.Collection:6:40: note: found this candidate
    @inlinable public __consuming func prefix(_ maxLength: Int) -> Self.SubSequence
                                       ^
Swift.Sequence:7:40: note: found this candidate
    @inlinable public __consuming func prefix(_ maxLength: Int) -> PrefixSequence<Self>
                                       ^

Expected behavior
Successful compilation.
Environment
The earliest snapshot that fails to compile is from 2022-06-07:
Apple Swift version 5.8-dev (LLVM fdfb8ac496e4538, Swift c9f10040f7f3838)

The code above compiles correctly using the 2022-06-02 snapshot:
Apple Swift version 5.8-dev (LLVM 278d67f38c6a910, Swift ee312bc1e20eb01)

Additional context
Assigning the result of c.prefix(1) before the loop or using it in a for-in loop in a non-generic context both succeed.
// OK
func g<C: Collection>(_ c: C) {
  let pre = c.prefix(1)
  for _ in pre {}
}

// OK
func h(_ c: Array<Int>) {
  for _ in c.prefix(1) {}
}

Hm, I think this is due to the fact that makeIterator and next() are type-checked together now, will investigate further tomorrow.


Proxies Checker

You forget to give value to the Total of the Proxies checker. The value(Total) of Proxies is always 0.

Fixed, please update


Does this make sense anymore

It is unmaintained, and some macros are straight up broken. Wouldn't it make more sense to work together with Fedora on their version since it's actively maintained?

@hellcp see openSUSE-zh/node2rpm and related stuff. I'm slowly porting it to golang.

why duplicate the effort, there are perfectly fine existing tools


Update rest-api-head-tags.md

typo 142: 'wich' -> 'which'

Thanks again @alexwoollam. Your change has been merged.


HTMLElement.shadowRoot is deprecated.

The contents of atom-text-editor elements are no longer encapsulated
within a shadow DOM boundary. Please, stop using shadowRoot and access
the editor contents directly instead.
HTMLElement.shadowRoot (/private/var/folders/rm/57_g6f0s1mgf8g1fvynyk0pc0000gn/T/AppTranslocation/FAA81D0A-6BBA-43B3-9288-9DE1D6EB541F/d/Atom.app/Contents/Resources/app/src/text-editor-element.js:13:10)
Object.setupCanvas (/Users/larben/.atom/packages/activate-power-mode/lib/power-canvas.coffee:36:28)
Object.prepareEditor (/Users/larben/.atom/packages/activate-power-mode/lib/power-editor.coffee:85:24)
Object.subscribeToActiveTextEditor (/Users/larben/.atom/packages/activate-power-mode/lib/power-editor.coffee:67:19)
<unknown> (/Users/larben/.atom/packages/activate-power-mode/lib/power-editor.coffee:29:24)
Function.module.exports.Emitter.simpleDispatch (<embedded>:63943:20)

aa

Hi @meiminjun
Activate power mode is not using the shadow DOM since v1.2.0
Try upgrading to the latest version.


[test] Enable dgemm regression test on eiger

PrgEnv-cray is not considered in this check. It looks like a performance bug in the Cray compiler which has not been fixed.

Hello @ajocksch, Thank you for updating!

In the file cscs-checks/microbenchmarks/cpu/dgemm/dgemm.py:

Line 42:1: E122 continuation line missing indentation or outdented
Line 42:1: E265 block comment should start with '# '

Do see the ReFrame Coding Style Guide

Codecov Report

Merging #1780 (6c0417a) into master (4435332) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1780   +/-   ##
=======================================
  Coverage   87.60%   87.60%
=======================================
  Files          49       49
  Lines        7937     7937
=======================================
  Hits         6953     6953
  Misses        984      984

Continue to review full report at Codecov.

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


INT B-18316 remove rank & duty_location from service_member table

Agility ticket
Summary
With the merge of E-04823 which removed the use of rank and duty_location off of the service member and now uses data from the orders table, we can make database changes to reduce technical debt.
I made the appropriate database changes and navigated through several flows as both customer and TOO and did not encounter any errors, but would love a thorough check from the testers of this PR.
This PR does the following:

removes rank column from service_members table
removes duty_location_id column from service_members table

This was done with a migration file that contains the following:
ALTER TABLE service_members
DROP COLUMN rank,
DROP COLUMN duty_location_id;

How to test

Access MM as customer & office
Navigate the application in all flows to ensure no errors occur (particularly in profile editing/viewing)

Database

removed rank from table service_members
removed duty_location_id from service_members

conflcts

conflcts

WHAT ABOUT NOW SUCKA


Fixes

This PR fixes

A bug that ocurs due to indexing a non Array in getter method
UnivariateFiniteArray method definition for cross_entropy and BrierScore()

@ablaom Tests are failing due to an unrelated OpenML issue

Codecov Report

Merging #375 into master will increase coverage by 0.14%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master     #375      +/-   ##
==========================================
+ Coverage   82.25%   82.40%   +0.14%
==========================================
  Files          39       39
  Lines        2626     2591      -35
==========================================
- Hits         2160     2135      -25
+ Misses        466      456      -10

Impacted Files
Coverage Δ

src/measures/finite.jl
95.32% <ø> (+5.09%)
:arrow_up:

src/univariate_finite/arrays.jl
98.18% <66.66%> (-1.82%)
:arrow_down:

src/resampling.jl
87.08% <100.00%> (+1.25%)
:arrow_up:

src/sources.jl
59.25% <0.00%> (-7.41%)
:arrow_down:

src/composition/models/inspection.jl
95.23% <0.00%> (-4.77%)
:arrow_down:

src/data/datasets.jl
86.11% <0.00%> (-2.78%)
:arrow_down:

src/composition/learning_networks/inspection.jl
87.80% <0.00%> (-0.84%)
:arrow_down:

src/hyperparam/one_dimensional_range_methods.jl
91.86% <0.00%> (-0.37%)
:arrow_down:

src/composition/learning_networks/nodes.jl
66.66% <0.00%> (-0.27%)
:arrow_down:

src/univariate_finite/types.jl
90.72% <0.00%> (-0.19%)
:arrow_down:

... and 7 more

Continue to review full report at Codecov.

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


Add route for a msg ID

This is implemented in the pub, but not in the frontend viewer

[x] viewer should have a route to get messages
[x] try encoding the dot as %2E

We are using our own encoding here where we convert the period character to -dot-.
This may be just an error/edge case with the static file server I'm using for developing


Input example is leaving the console with colored text

Suggestions
When I run go run src/github.com/c-bata/go-prompt/_example/echo/main.go the console ends up with bold text in my iterm and my xterm terminals.

Bug reports
It seems like Render.Teardown is not resetting the default colors on exit. Please let me know if I am mistaken:
 // TearDown to clear title and erasing.                                                         |   -livePrefixCallback : func() string,
 func (r *Render) TearDown() {                                                                   |   -out : ConsoleWriter
     r.out.ClearTitle()                                                                          |   -prefix : string
     r.out.EraseDown()                                                                           |   -prefixBGColor : Color
     r.out.Flush()                                                                               |   -prefixTextColor : Color
 }                                                                                |   -previewSuggestionBGColor : Color

Should this not also include r.out.SetColor(DefaultColor, DefaultColor, false)?
Expected Behavior
The console to be back to its previous text formatting after prompt.Input() finishes.
 Please provide detailed steps for reproducing the issue.
If you enter your own string, the program exits as expected - with previous console colors. However, if you accept the suggestions, the program exits with bold text.

Context
Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

Operating System: Mac OSX High Sierra
Terminal Emulator: iTerm2 and xquartx terminal
tag of go-prompt or commit revision: HEAD

It seems #85 is fixing this for me.

Awesome. This report is easy to understand. I'll fix this.


The Chrome team is planning on axing the hidden NTP features brought back by your extension

Hi Jingyi,
I thought I should bring this to your attention as this will break your much loved extension:
https://code.google.com/p/chromium/issues/detail?id=329637#c5

#5<EMAIL_ADDRESS>I think these things are always hidden or don't exist anymore and can be axed:

recently closed menu
other sessions menu
most visited page
the upper right "login container"

I've already responded in their thread as protest, and am trying to inform as many others as possible.
Kind Regards
Christopher

Thank you very much for notifying me. After looking at the changes, I find that this extension will not be affected because we have a local copy of the old NTP code. This extension depends on the new NTP features, which is the one with the Google logo and search bar. Please tell me when Google starts to delete or modify the new NTP code.

i will reopen this issue when there might be a breaking change in chrome.


Index patterns are missing

Operating System: Centos7
go version go1.12.7 linux/amd64
I am trying to run prometheus-es-adapter but got the following error :
{"level":"fatal","ts":1563764984.771872,"caller":"adapter/main.go:74","msg":"Failed to create index template","error":"Failed to create index template: elastic: Error 400 (Bad Request): Validation Failed: 1: index patterns are missing; [type=action_request_validation_exception]","stacktrace":"main.main\n\t/root/prometheus-es-adapter-master/cmd/adapter/main.go:74\nruntime.main\n\t/root/go/src/runtime/proc.go:200"}

@pokixing Which ES version are you using? I have the same problem

same with elasticsearch:5.6.8-alpine.
any help?


SharedCircularPropertiesTests.cs:55: 60m/DEV...

The puzzle 180-b1140a42 from #180 has to be resolved:
https://github.com/DenisZhukovski/ObjectToTest/blob/0588a92b7b144449622c0ee73907f0795c7620d3/tests/ObjectToTest.UnitTests/SharedCircularPropertiesTests.cs#L55-L55
The puzzle was created by Denis Zhukovski on 23-Feb-23.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 180-b1140a42 has disappeared from the source code, that's why I closed this issue.


Multi provider strategy

Make generic the strategy attached to a channel.
The idea is to be able to define a function implementing the strategy. A strategy takes all provider in input and return a "new" provider.
channels {
  sms: {
    providers: [...]
    multiProviderStrategy: string /* actual*/
            | (SmsProviderInstance[]) => ((req) => Promise<string>)
  }
}

Refactor existing strategies would be easy
fallback : (providers) => // try with the first and recursively try on error
round-robin (no-fallback) : (providers) => provider[n]

https://github.com/notifme/notifme-sdk/pull/14