No UART Output in wifi_mqtt_pub example

Hello
I'm trying to get the wifi_mqtt_pub example running on the BL616. It builds fine, and I can flash it. But I don't get any output via Terminal.
Despite that, I get output via Terminal for the helloworld example. I don't know what the difference is.
make CHIP=bl616 BOARD=bl616dk
make CHIP=bl616 BOARD=bl616dk flash COMX=/dev/ttyUSB0
sudo picocom -b 2000000 /dev/ttyUSB0

did you tried different baudrate?

Haven't found anything which should make the baudrate different between wifi_mqtt_pub and helloworld. But will check with the scope.

Looks like there is no communication on the TXD pin.
wifi_mqtt_pub example:

helloworld example:

you must use bl618, mqtt must use psram

So if you use bl616, it will crash in psram init.

Thanks, wifi_tcp example works :+1:


white page on Android browser

Hi, thanks for w great library
The only thing stopping me to use it now is that it shows a blank page on an older Android browser 4.4.2
https://vuematerial.github.io
Looks like a bug.
Any ideas?
Thanks

Steps to reproduce
Which browser?

What is expected?
What is actually happening?

Reproduction Link

It wasn't tested on Android 4.x yet.
For now it only supports Lollipop or greater. I will fix that in the future.

Closing this issue as our focus is on the new 1.0.0 version.


ug: Team-based access to devices and device groups

Readiness

[ ] Merge (pending reviews)
[ ] Merge after date or event
[x] Draft

Overview
Why merge this PR? What does it solve?
Checklist
Optional. Add a 'x' to steps taken.
You can fill this out after opening the PR. "Did I..."

[ ] Run spelling and grammar check, preferably with linter.
[ ] Avoid changing any header associated with a link/reference.
[ ] Step through instructions (or ask someone to do so).
[ ] Review for wordiness
[ ] Match tone and style of page/section.
[ ] Run make linkcheck.
[ ] View HTML in a browser to check rendering.
[ ] Use semantic newlines.
[ ] follow best practices for commits.

[ ] Descriptive title written in the imperative.
[ ] Include brief overview of QA steps taken.
[ ] Mention any related issues numbers.
[ ] End message with sign off/DCO line (-s, --signoff).
[ ] Sign commit with your gpg key (-S, --gpg-sign).
[ ] Squash commits if needed.

[ ] Request PR review by a technical writer and at least one peer.

Comments
Any thing else that a maintainer/reviewer should know.
This could include potential issues, rational for approach, etc.

Docs for c97651fadd3b2ce33f40325cf69909a64936467c are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1971/docs/artifacts/html/index.html

Docs for d44c11a0fab9469e12716b18bd09d1259093a444 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1972/docs/artifacts/html/index.html

Docs for 252ddd2be4e00d09618f69b524ccdca456a1bb2d are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1973/docs/artifacts/html/index.html

Docs for 37812773e907e262eb1ffab80918de1890cdc7ba are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1974/docs/artifacts/html/index.html

Docs for e68d49855f2af86675f33b63847af9b66fd72289 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1975/docs/artifacts/html/index.html

Docs for 33c298c76ab63c23b324609b9e3564913567dedf are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1976/docs/artifacts/html/index.html

@doanac Initially, I thought about rewriting the whole "Team Based Factory Access" document in order to provide more details about the team-based access control in general and about access to device groups in particular. Then, as I started doing so, I realized that it is not worth it, because:

all those details are present in the overall document anyway, a user just needs to go through links referenced from the given page;

users will raise questions to the customer support and learn this stuff by trying/doing anyway, regardless the doc content.

So, I came up with something really minimalistic and written in the spirit of the current doc.

Docs for e3591a225542ad9c8247c4e80cbd5d7716af6a80 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1977/docs/artifacts/html/index.html

I restructured the change a bit and tried to address all your suggestions.

Docs for 3f364021c948ad85376b86b34022d0d2c27486ee are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1978/docs/artifacts/html/index.html

@kprosise Did you have a chance to review the updated version?

@mike-sul I will be doing so today.

Address the changes as recommended by @vanmaegima, then it should be good to go.

I've just done it, @vanmaegima please, verify.


[Docs]: Best practices for securing an Appsmith instance

Is there an existing issue for this?

[X] I have searched the existing issues

Documentation Link
No response
Discord/slack/intercom Link
No response
Describe the problem and improvement.
Add best practices for securing an Appsmith instance

documented here - https://docs.appsmith.com/getting-started/setup/best-practices#security


Unable to compile datetime, socket and time extensions with shared python 2.7.9

When compiling python 2.7.9 with this project the modules mentioned in the title can't be build.  CMake reports for all of them the following or a similar error message:
$HOME\python-prefix\src\python\Modules\timemodule.c(104) : warning C4273: '_PyTime_DoubleToTimet' : inconsistent dll linkage
        $HOME\python-prefix\src\python\Include\timefuncs.h(17) : see previous definition of '_PyTime_DoubleToTimet'
$HOME\python-prefix\src\python\Modules\timemodule.c(1062) : error C2491: '_PyTime_FloatTime' : definition of dllimport function not allowed

To reproduce use this CMakeLists.txt file. Invoke it with cmake -n "NMake Makefiles" .. in a separate build directory.

Hi Adrian,
Would you mind trying to checkout this topic and check if things works better: https://github.com/jcfr/python-cmake-buildsystem/tree/ci-support-v2.7.8
Thanks
Jc
Ps: I am travelling for the coming days and won't be able to review. @thewtex  should be able to help out :)

Thanks for pointing out your fork.  Unfortunately it does not fix the problem.
I tried the following combinations:

branch
python version
status

davidansome/master
2.7.3
compiles

jcfr/ci-support-v2.7.8
2.7.8
fails

jcfr/ci-support-v2.7.8
2.7.9
fails

The 'fails' show the same error as reported in the initial post.

Hi
Thanks for checking. I ll have a look in two weeks as soon i m back from
travelling.
In the mean time, you are welcome to contribute a pullvrequest.
Thanks
Jc
--
Sent from my mobile device
On May 18, 2015 2:10 PM, "Marcel Metz"<EMAIL_ADDRESS>wrote:

Thanks for pointing out your fork. Unfortunately it does not fix the
problem.
I tried the following combinations:
branch python version status   davidansome/master
https://github.com/davidsansome/python-cmake-buildsystem 2.7.3 compiles
jcfr/ci-support-v2.7.8
https://github.com/jcfr/python-cmake-buildsystem/tree/ci-support-v2.7.8
2.7.8 fails  jcfr/ci-support-v2.7.8
https://github.com/jcfr/python-cmake-buildsystem/tree/ci-support-v2.7.8
2.7.9 fails
The 'fails' show the same error as reported in the initial post.
—
Reply to this email directly or view it on GitHub
https://github.com/davidsansome/python-cmake-buildsystem/issues/50#issuecomment-103178901
.

Hello,
finally I could investigate this issue further and could successfully build python.
According to the VTK wiki the default python release links all extensions into the python.dll.  This project instead tries to build the extension as pyd libraries(?).  So I tried to make the time extension a builtin module and removing the timemodule.c from various places used in commit adrianbroher/python-cmake-buildsystem@3ad6d93 .  Now I don't know if this is the right way to fix the problem.  Given the fact that upstream provides a monolithic python.dll shouldn't all modules be builtins on Windows by default?

Now I don't know if this is the right way to fix the problem. Given the fact that upstream provides a monolithic python.dll shouldn't all extension be builtins on Windows by default?

Any feedback on this and the mentioned commit?  Should I make this a PR?

@adrianbroher @jrobcary  :100:  Thanks to you both the issue is now fixed in master. See 0a088b28eb4


Tests fail on v0.7.1

Many thanks for developing chwall!
I use chwall on Arch Linux and ran into an issue when using yay to update it from v0.6.0-3.
The issue still occurs when I clone this repo directly and run make test, so it doesn't appear to be makepkg-specific.
Four of the tests fail. I've attached the relevant output below, but it appears that it's expecting different values in certain parts of a generated service file. Maybe I need to uninstall chwall for the tests to pass?
For convenience, here is the output for one of the failing tests:
FAIL: test_04_create_local_systemd_service_file_from_client (tests.test_local_desktop_files.TestLocalDesktopFiles.test_04_create_local_systemd_service_file_from_client)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python3.11/unittest/mock.py", line 1375, in patched
    return func(*newargs, **newkeywargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/drew/tmp/chwall-upstream/tests/test_local_desktop_files.py", line 57, in test_04_create_local_systemd_service_file_from_client
    self.assertEqual(mock_stdout.getvalue(), result)
AssertionError: 'Serv[320 chars]very 3600 seconds\nAfter=network.target\n\n[Ti[79 chars]et\n' != 'Serv[320 chars]very 600 seconds\nAfter=network.target\n\n[Tim[76 chars]et\n'
  Service file:
  [Unit]
  Description=Simple wallpaper changer
  After=network.target

  [Service]
  Type=simple
  Environment=DISPLAY=:0
  ExecStart=python -m chwall.client next no_restart
  WorkingDirectory=/home/drew/tmp/chwall-upstream

  [Install]
  WantedBy=default.target

  Timer file:
  [Unit]
- Description=Change wallpaper every 3600 seconds
?                                    -
+ Description=Change wallpaper every 600 seconds
  After=network.target

  [Timer]
- OnActiveSec=3600
?             -
+ OnActiveSec=600
- OnUnitActiveSec=3600
?                 -
+ OnUnitActiveSec=600

  [Install]
  WantedBy=timers.target

test-output.log
Thanks again for developing chwall! Let me know if I can provide any more info.

Hi, and thank you very much for your feedback.
You are right, there is something wrong with the tests. The tests are wrongly taking some values from your own config file. I’ll try to fix that and release a new version to fix that.
In the mean time, you can still install the new version if you remove (or comment) the check function in the PKGBUILD before actually building the package (I don’t know yay, thus I let you check in their documentation how to do so).

Hi again, it should be fixed by my last release v0.7.2, already published on aur. Can you confirm if you have time? Thank you very much.

I've confirmed that 0.7.2 builds and installs successfully on my machine! Thanks so much!


Add an optional param to cli to only run requests that have tests

Description

In a CI/CD environment there exists a use case of only wanting to run requests that have tests.
Added an optional param --tests-only to the CLI arg list. If this is set it will only add request that have tests.

Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

+1

Possibly related: https://github.com/usebruno/bruno/issues/1039
This PR creates a difference between the way the collections are run in then UI vs the CLI. If this PR is merged, I propose to file a follow-up issue to add a similar option to the UI.
However, there are a few questions that this PR raises:

should a file where the tests are all commented out be included? (To me, the answer would be "no",  but this PR would include it)
should a file that has assertions but no tests be included? (To me, the answer is "yes", but this PR would skip such a file)
should a file that has tests in the post-response script be included?

this is messy. The post-response-script and the test-script are run separately, but they do pretty much the same thing. The UI gets confused when I move tests from one to the other. I think this should be cleaned up first before merging this PR -- there should not be two places where post-response scripts are being defined. @helloanoop : What do you think?

Lastly: This PR promotes a collection-writing-style that I would call "bag-of-requests", where some are used for exploratory-testing, and others are used for automatic testing, and the difference between the two is given implicitly.
I would propose a different solution to this problem: In your collection, sort your requests into appropriate subfolders. Then, we should change the runner so that you can do:
bru run my-collection --folder "subfolder with all the tests"

I think that would lead to cleaner collections, and give you more flexibility overall.

I would propose a different solution to this problem: In your collection, sort your requests into appropriate subfolders. Then, we should change the runner so that you can do:

@mj-h Yes I do agree that it would be easier to just throw all of the tests into a folder but I don't think that is very realistic for complex collections, such as having a very large API collection which many developers are using & contributing to. It is organized by api routes. The structure would be:
Super Cool Sports API
└───Teams
 │   └─── League 1
 |          └─── test1.bru
 │   └─── League 2
└───Games
 │   └─── Scores

Which matches the url path /teams/league-1/test1 for the resource.
Now if we wanted to have the CI/CD run the requests only with tests but only for a given folder as you are suggesting (correct me if I am wrong) we run into the issue of the API collection not remaining organized.
The two work arounds for this could be

Have duplicated request between original location and the tests folder. This is not ideal for a variety of reasons
Just move the request into the tests folder from it's original location. Again not ideal.

Super Cool Sports API
└───Teams
 │   └─── League 1
 │   └─── League 2
└───Games
 │   └─── Scores
└───Tests
 |   └─── test1.bru

Open to suggestions

(1) I have many tests with assertion only. I have a test with post-script and assertion.
The proposed implementation skipped them.
(2) The suggested implementation raises the cyclomatic complexity of the code which is not ideal. The implementation involves adding another if/then case, but it is not a recommended approach. Instead, using strategies would be a better option. For instance, we can use a simple strategy (all scripts) and apply different strategies for other cases as needed.

@jeff-edmondson-pci you always have the opportunity to build your fork and use your version of Bruno with the features you need.
From my point of view, there is at least a lack of documentation correction that would reflect the appearance of new features.
I also think it’s right to first discuss the need for any additions before creating a pull request.

Thanks @jeff-edmondson-pci for working on the PR and @mj-h and @rabestro  everyone for chiming in with comments

should a file where the tests are all commented out be included? (To me, the answer would be "no", but this PR would include it)

@mj-h I agree. This is edge case though. We can iteratively improve this logic later by using the decomment library to address this

should a file that has assertions but no tests be included? (To me, the answer is "yes", but this PR would skip such a file)

@mj-h  I agree. An assertion is just a "declarative" way of writing a test. I see that this has been addressed in the PR.

The post-response-script and the test-script are run separately, but they do pretty much the same thing. The UI gets confused when I move tests from one to the other. I think this should be cleaned up first before merging this PR

The separation is intentional. I agree that both these blocks (post response scripts and test scripts) do the same thing. The intent is to visually separate code related to "scripting" and "tests".

scripting - things like parsing a response and updating some collection variables
tests - assertions and tests

I have many tests with assertion only. I have a test with post-script and assertion. The proposed implementation skipped them.

@jeff-edmondson-pci This has been addressed in the PR

The suggested implementation raises the cyclomatic complexity of the code which is not ideal. The implementation involves adding another if/then case, but it is not a recommended approach. Instead, using strategies would be a better option. For instance, we can use a simple strategy (all scripts) and apply different strategies for other cases as needed.

@jeff-edmondson-pci There is a plan to include tags in the roadmap. I think the business use case of run tests that have tests/assertions is a valid one. Implementing the tags feature would take some time.
Also the --test-only arg being introduced in this PR is optional. My personal take would to be use the concept of tags (which is a pretty common patten across testing frameworks) when that feature lands in Bruno.
@dcoomber @Its-treason @martinsefcik @jeff-edmondson-pci @mj-h
The PR is GTG from my side. Let me know if you have further comments. I am planning to have this merged over the weekend.

@helloanoop Personally I would rather see in Bruno more "standardized" support for tags instead of this feature. If there would be such support then I guess this --tests-only option is not needed.
Also If I put only something like console.log('') inside tests script then it automatically executes such request, but does it mean that such request has tests ? So for me it is a bit confusing, option description is short and not enough and I mostly agree with @rabestro

Thanks everyone for the feedback and comments.
I have decided to merge this.
I agree that --tests-only (although optional) has some edge cases

Like a request can have console.log('') inside tests script then it automatically executes such request, but does it mean that such request has tests ?
What if a test block is commented out. Does it mean that such request still has tests ?

Its upto the user to ensure that

they have tests in the request and
they are not commented out and
they don't have code in the tests script that are not tests


Add distance value overlays between elements when users press a modifier key

Problem
When I'm trying to align items for a drawing, I find myself unclear about the spacing between elements. I often fallback to holding option, because Figma and Sketch both have this setting. Figma's version can be seen in this gif, with me holding down alt/option:

Potential solution

Display the rounded calculated values between polygons
Bind this overlay to the alt/option key
Add guides and snapping

There are two problems with this, one is philosophical, other is technical.

Excalidraw philosophy is to enable fast, hand-drawn, rough sketching. As such, we want to limit the amount of information (especially technical) that we display. We do allow for more precision handling when user desires (e.g. grid mode, alignment...), but it's not the default and we try to keep the UI minimal.
There were similar requests in the past, which we've turned down so far (e.g. https://github.com/excalidraw/excalidraw/issues/2391).

On Alt, same as Figma, we duplicate the element, but inlike Figma, we duplicate immediately and do not revert even if you release the Alt before you commit. So using Alt for displaying dimensions wouldn't work. Also, even if we reverted to Figma behavior, you'd still need to release Alt before committing at which point the dimensions would likely diverge on commit.

I think your use case is covered by grid mode (right-click the canvas → grid mode), especially after we support custom grid size.

You bring up two good points that I had not remotely considered 🙂.
1. Philosophical
Makes sense to avoid a full blown analogous feature to Figma et al. I don't even think I want that as much as I want some alignment features that go slightly beyond the existing. I think the grid mode may suffice completely.
2. Technical
Makes sense. I would say it's more important to have a binding to alt/option than to have it on another key, as the latter would break heuristics established by design tools.
Again, a grid mode would also make sense. I'm less interested in exact values (thus my suggestion to do some rounding, which probably amounts to some light Math.floor, and more in having values that I can roughly ballpark.
Lastly, @vjeux asked for this issue on Twitter so going to hope that this tag suffices to tie together both convos.

Note that grid mode is already implemented in case it wasn't clear :smile:

Lastly, @vjeux asked for this issue on Twitter so going to hope that this tag suffices to tie together both convos.

Yes, that's our modus operandi — we want to discuss and gain consensus on everything before either accepting or rejecting a given feature.

Okay I only just realized that grid mode existed. Let me play with it a bit and see if I can add more colour to this request/issue. Thanks @dwelle.


Improve the dump analyser script to load the missing frames.

Can we add code to dump_analyzer.sh to pass along the address of the shared libraries?
When you invoke lldb, it attempts to load in shared libraries relative to the current directory.  This works when you're on the same machine as where the core was dumped because the shared libraries are in the same place because the shared libraries are specified via absolute paths.
This does not work, however, on Lincoln because the shared libraries corresponding to the core are in different places.
To solve this, you just have to set the search path for the libraries appropriately.
For example, consider trying to debug the following core dump:
lldb /home/yugabyte/yb-software/yugabyte-<IP_ADDRESS>-b41-centos-x86_64/postgres/bin/postgres --core  core_pgapi_interrupt.22392
That line is (essentially) what dump_analyzer.sh runs after it restores the relevant release to (in this case) /home/yugabyte/yb-software/yugabyte-<IP_ADDRESS>-b41-centos-x86_64/.
To make sure we find our shared libraries from there, we just change our command to:
lldb --one-line-before-file "settings append target.exec-search-paths $(find /home/yugabyte/yb-software/yugabyte-<IP_ADDRESS>-b41-centos-x86_64/ -type d|xargs echo)" /home/yugabyte/yb-software/yugabyte-<IP_ADDRESS>-b41-centos-x86_64/postgres/bin/postgres --core  core_pgapi_interrupt.22392
Slack Message

Done

Moving project to https://github.com/ag26jan/dump_analyser_project-scripts


Add link to the Readmore button in Guidelines Button

Add (https://github.com/Clueless-Community/clueless-official-website/blob/main/CODE_OF_CONDUCT.md) href to the link tag in the Read More button also make sure that it opens in new Tab.

I would like to pick this one up if it's okay

Hi, I can work on this.
Kindly assign me

@stevenjburrows assigning it to you.


Added Update node tag UI modal and cypress testcases

Signed-off-by: Himanshi Chhabra<EMAIL_ADDRESS>:nut_and_bolt: Description: What code changed, and why?

Add Update node tag UI on node list dropdown page.

:chains: Related Resources
#4916
:+1: Definition of Done

Added dropdown for update node tag on node list page.

:athletic_shoe: How to Build and Test the Change
PFA Video
STEP 1
inside the hab studio

[default:/src:0]# build components/automate-ui-devproxy/
[default:/src:0]# start_automate_ui_background
[default:/src:0]# start_all_services

STEP 2
open new window
go to automate UI path

$ cd components/automate-ui
and run the command

npm run serve:hab

To add data https://github.com/chef/automate/blob/master/dev-docs/adding-data/adding_test_data.md#adding-data-to-infra-views
Steps node dropdown for update tag:

Go To  Infrastructure -> Chef Servers -> click server name-> click on org name -> Nodes.
Click on  ... on particular node -> click Manage node tag -> add tags in text field and click  update.

For cypress Build
https://github.com/chef/automate/tree/master/e2e
:white_check_mark: Checklist
All PRs must tick these:

[x] I have read the CONTRIBUTING document.
[x] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[x] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[x] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[x] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[x] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[x] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[x] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable
https://user-images.githubusercontent.com/61003053/119679730-375d5980-be5e-11eb-9592-c1c08106c782.mp4

@chaitali-mane I have updated the changes.

Hi @himanshi-chhabra ,

When I add test, it comes as Test and
When I tried with Test, It also comes as Test
Is that valid to make Capitalise Tag name? Please check .

@himanshi-chhabra,
We are capitalising node name here also, Node name is case sensitive or not ?
I found same thing on #5086
Can you please verify this

Hi @himanshi-chhabra ,

When I add test, it comes as Test and
When I tried with Test, It also comes as Test
Is that valid to make Capitalise Tag name? Please check .

Yes, I have checked it's in the UX design to  capitalize tag name

@himanshi-chhabra,
We are capitalising node name here also, Node name is case sensitive or not ? Can you please verify this
I found same thing on #5086

I have updated the changes


[wip] [test-only] Test 682

What this PR does / why we need it:
Which issue(s) this PR fixes:
JIRA:
Does this PR needs for other branches:

Does this PR (patch) needs to update/drop in the future?:


Uncaught RangeError: Maximum call stack size exceeded

Uncaught RangeError: Maximum call stack size exceeded
Hello, I got this error in prod build only.
I'm find this report in angular repository https://github.com/angular/angular/issues/7930
So, if this mangle param, how i can turn off him. Thanks!

Mangling is disabled by default.
Do you reproduce this from clean clone or you did changes?

@mgechev did changes, many changes... :)

Most likely the issue is not related to the seed itself but to the changes that you have done :-).
For support requests you can move the discussion to gitter.


Add Volunteer Search Location Gathered Events To Call Records

missing release notes update.

closing this as its not necessary and already done
https://github.com/bmlt-enabled/yap/blob/9b5336733bd0ab78cf4ecb3cbe388343b9feb1b8/endpoints/helpline-search.php#L62-L67


Message-passing for CUDA is broken by not allowing scalar indexing

Mentioned in #187.
Message-passing network has an operation of scalar indexing over a cuarray. Currently, new version of CUDA forbidden the use of scalar indexing, which breaks the tests. Message-passing need a new implementation for indexing vertices or edges, maybe a cuda kernel.

Now it is fixed by #253


Use universal dependencies for cross-lingual consistency

Universal dependencies (UD) is a new dependency framework with extensive work across 30+ languages. One of the main benefits is that using UD for parsing, the annotation scheme becomes much more consistent across the supported languages. I would suggest that you look into supporting this work.

We're working towards this.
The priority at the moment is to support the morphological analysis scheme. You can see discussion of it here: https://github.com/UniversalDependencies/docs/issues/212
Our current roadmap is to get other languages working for tokenization, tagging, morphological analysis, lemmatization and word vectors. We'll then work on named entity recognition and parsing for for these languages. Finally, we'll offer UD syntactic models for English.
Currently UD seems like it will result in lower accuracy for English than the OntoNotes-based, ClearNLP converted model we're offering. The ClearNLP converter produces very good output, because the conversion makes use of the trace and function annotations in the treebank, which the Stanford converter ignores.
I suspect that good parser accuracy with UD may depend on learning an intermediate representation, and applying post-process transformations. So I'm reluctant to adopt the scheme until I have time to experiment with this.

Honnibal,
since it has been more than a year from the previous request/disscussion for UD integration, how are things looking now..??
Thanks,


The function works on all databases, but tempdb

Type of Change

[ ] Bug fix (non-breaking change, fixes # )
[ ] New feature (non-breaking change, adds functionality, fixes # )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[x] Documentation
[ ] Build system

Booom! Thanks Mikey! 🔥


Enable local builds (much faster than using docker for development)

Updated the docs, happy to amend if there are any suggestions, this shouldn't differ the final output from the make build command but it is much faster for local development of tests.
Also solves the annoying issue of the tests/k6 folder being owned by root when using make build (via docker).

i updated a few things, can you please have a look. Thanks for your PR

Thanks @fschade - I have made all the changes you requested (and rebased to main).


Update checkgroup config

What does this PR do?
Part of #14294
Need to merge this into master to continue the above, as it can only read the config in the default branch (master)
This has no impact on CI atm
Does your PR introduce any breaking changes? If yes, please list them.
None
cc @carmocca @akihironitta @borda

It's under development, but the config file can only be read off the default branch (master), so I need this merged to continue.


feat: Concurrency limiter

Adds concurrency limiting to the base client class in the style of https://github.com/momentohq/client-sdk-java/pull/385.
The concurrency limiting function is almost the same as the prototype and wraps the existing generic executeGrpcFunction and executeGrpcBatchFunction methods.
The PR is broken up into 3 commits:
The first adds the concurrency limiting fixed thread pool to the base client class, along with the constructor to configure it and the close() method to shut it down. It replaces the close() in the child classes with a new abstract doClose() so that there is still a close that the children are forced to implement.
The second adds the max concurrent requests integer to the transport strategy layer of the config and plumbs it through the cache data and control clients to the base client. The storage client config doesn't have it yet. It could have request limiting if we add the configuration option.
The third commit adds the request limiting function copied from the POC draft pr.

Thanks @nand4011  - we are still waiting on reproducer/perf testing data to validate this before I should dive in too deep to review, yes?

After fixing this bug https://github.com/momentohq/client-sdk-java/pull/396, allowing me to set timeouts lower than one second, the limiter provides benefits on startup when running a large number of concurrent calls.
When running at 1000 concurrent calls and 1000tps with a timeout of 100ms:
With no limiter, there are ~2000 timeouts on startup.
With a limit of 50 concurrent calls, there are ~60 timeouts.
With a limit of 10 concurrent calls, there are ~5 timeouts.
Throughput and latency are not affected.


Finding an album object's record label

Hi everyone, I've been trying to access the record label of an album object, but I can only access these bits of info:
album_type
artists
available_markets
external_urls
href
id
images
name
release_date
release_date_precision
total_tracks
type
uri
I figured there would be a 'label' key for that, but I can't seem to access it through spotipy.

what album ID are you using? I think not all albums have a label. Can you try with 0sNOF9WDwhWunNAHPD3Baj?

Hmmm, I'm currently trying it for 7e7t0MCrNDcJZsPwUKjmOc and I see the label as Victor Victor Worldwide in the developer console on Spotify's website, but don't see label in the spotipy object keys.

what method are you using? (code)

(I understand it's a bit shoddy, just messing around with the API for now) but I'm accessing a playlist through its uri and I'm just testing it with the first track's album. Am I accessing it correctly?
client_credentials_manager = SpotifyClientCredentials(client_id=cid, client_secret=secret)
sp = spotipy.Spotify(client_credentials_manager = client_credentials_manager)
playlist_id = 'spotify:playlist:37i9dQZF1DX0XUsuxWHRQd'

results = sp.playlist(playlist_id)
tracks = results['tracks']['items']
for track in tracks:
	track = track['track']
	track_name = track['name']
	track_artist = track['artists']
	track_album = track['album'] # using this as the album object I need to get label from
	track_id = track['id']
	break

I see the label as Victor Victor Worldwide in the developer console on Spotify's website

that's because you are looking at the get-album endpoint https://developer.spotify.com/documentation/web-api/reference/albums/get-album/
whereas you are using a python method for the get-playlist endpoint:
https://developer.spotify.com/documentation/web-api/reference/playlists/get-playlist/
To get the label, you will need to do a another query just for the album: sp.album(album_id)

Ah, I see where I got it mixed up. I just made a query for the album itself and it worked as intended. Thank you so much for the help, I really appreciate it! 👍


feat(spark): support insecure mode for rest

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

merged


Indexer data upgrades for updated Explorer UI

We are aiming to support the following data ideally through the Hasura endpoint (data from the Indexer):

Transactions (internal and external)

Source/Destination Transaction Data Parsing and Display
Live tracking of source/zetachain/destination status (Hasura subscriptions) in the UI
Transaction types (Send/Cross-Chain, internal transaction labels/types)

Blocks

Transaction list per block

Validator data

Recent blocks per validator

Network Data

Transactions per network (i. e. filter transactions by networks)

Validator Details
Address Details (Source/Destination Address, Validator/ZetaChain Address)

With related transactions

Connector Address + Gas Amounts

Could we try to support all of the above? As well, if anything is not feasible or needs clarification (or should come from somewhere else), it would be good to hash out those details here!

Looks no longer relevant, please feel free to reopen otherwise


chore: release 6.0.5

Release-As: 6.0.5
For some reason, release-please got a little messed up with after the 6.0.4 release,  this should fix that
Taken from https://github.com/googleapis/release-please#how-do-i-change-the-version-number
How do I change the version number?
When a commit to the main branch has Release-As: x.x.x (case insensitive) in the commit body, Release Please will open a new pull request for the specified version.


switch to blendFunc src_alpha, one_minus_src_alpha

For issue #19.
@heshan0131 Can you experiment with this blendFunc and see if it works for your case and generally? If not we can try some other combinations or explore more advanced blending options.
@ibgreen @philogb

@ibgreen @heshan0131 Did you guys get to try this out for https://github.com/uber-common/deck.gl/issues/19?


Use Cachable in Tap.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Also added a separate version of Cachable#cache which takes a block and automatically freezes the cached value.

I'm not sure that it's intuitive to include and extend the same module in different places in the codebase especially if we're planning on doing both in the same class. It might be worth thinking up some way of differentiating the instance cache from the class/module cache.
Interestingly enough we do have this pattern in a few places in the codebase but it's never truly an instance cache.
$ cd $(brew --repo)
$ git grep --no-index -C 4 'include Cachable'
Library/Homebrew/api/cask.rb-    #
Library/Homebrew/api/cask.rb-    # @api private
Library/Homebrew/api/cask.rb-    module Cask
Library/Homebrew/api/cask.rb-      class << self
Library/Homebrew/api/cask.rb:        include Cachable
Library/Homebrew/api/cask.rb-
Library/Homebrew/api/cask.rb-        private :cache
Library/Homebrew/api/cask.rb-
Library/Homebrew/api/cask.rb-        sig { params(token: String).returns(Hash) }
--
Library/Homebrew/api/formula.rb-    #
Library/Homebrew/api/formula.rb-    # @api private
Library/Homebrew/api/formula.rb-    module Formula
Library/Homebrew/api/formula.rb-      class << self
Library/Homebrew/api/formula.rb:        include Cachable
Library/Homebrew/api/formula.rb-
Library/Homebrew/api/formula.rb-        private :cache
Library/Homebrew/api/formula.rb-
Library/Homebrew/api/formula.rb-        sig { params(name: String).returns(Hash) }
--
Library/Homebrew/readall.rb-#
Library/Homebrew/readall.rb-# @api private
Library/Homebrew/readall.rb-module Readall
Library/Homebrew/readall.rb-  class << self
Library/Homebrew/readall.rb:    include Cachable
Library/Homebrew/readall.rb-    include SystemCommand::Mixin
Library/Homebrew/readall.rb-
Library/Homebrew/readall.rb-    # TODO: remove this once the `MacOS` module is undefined on Linux
Library/Homebrew/readall.rb-    MACOS_MODULE_REGEX = /\b(MacOS|OS::Mac)(\.|::)\b/

not sure that it's intuitive to include and extend the same module

I disagree, that's just how Ruby works, but it is actually quite surprising that all existing usages so far are class-level caches.

There are a few places in the code that use cache.fetch(:key) { logic } and those can probably be cleaned up after this change too.

restrict this to methods without arguments with method.arity.zero?

I tried this in the beginning, but that actually only works when not using Sorbet at runtime because Sorbet will always wrap all methods with *args.

I'll throw another tangentially related idea out there. Maybe it'd be good to somehow programmatically collect all classes and modules that mix in Cachable and then clear those caches in-between tests. That would make it harder to forget to clear the cache of some module. This probably doesn't need to happen at the instance level though. To be clear, I'm not suggesting that this should be included in this PR. I'm just thinking out loud.

Passing on this, sorry.


preceding and following axes do not include siblings children

With this XML:
<a0>
  <b0>
    <c0 />
    <c1 />
  </b0>
  <b1>
    <c2 />
    <c3 />
    <c4 />
  </b1>
  <b2>
    <c5 />
    <c6 />
  </b2>
</a0>

If we start at c3 and use the preceding axis, we should visit c2, c1, c0, and b0, but we currently only visit c2 and b0.

Closed via #102


Initial functionality

This PR adds the basic functionality to support the SAML group attributes using Flask Multipass within Indico.
The current SAML Identity Provider in Flask Multipass does not support the SAML group attributes, see this issue for further explanation.
Implementation notes:

Groups are stored/cached in the Indico provided database, as Indico does not do this for external auth providers (see this issue), and group membership information would otherwise be lost on a server restart/crash.
Group membership information is updated on every user login.
Currently the package is designed as an Indico plugin and the code directly uses the SQLAlchemy instance from the Indico package, but this could be extended to be independent of Indico to be used for other services.
The code has only been tested locally and not yet with a real SAML IdP service, so it should be considered experimental. I have added a warning to the README.

We're missing docstrings for a few methods - are there plans to add those later or is that something that could be addressed now?

Any missing docstrings should be docstrings equivalent to the base class docstring. I avoided copy & paste there.
I always mention it on the first disable rule # noqa: D102,DCO010 docstring in base class, but did not repeat the reason "docstring in base class" everywhere.


ui: deposit: fields: license: Custom SearchBar

:heart: Thank you for your contribution!
Description
Closes https://github.com/zenodo/rdm-project/issues/213
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

[ ] I'm aware of the code of conduct.
[ ] I've created logical separate commits and followed the commit message format.
[ ] I've added relevant test cases.
[ ] I've added relevant documentation.
[ ] I've marked translation strings (for relevant code).
[ ] I've followed the CSS/JS and React guidelines (for relevant code).
[ ] I've followed the web accessibility guidelines (for relevant code).
[ ] I've followed the user interface guidelines (for relevant code).
[ ] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[ ] I've NOT included third-party code (copy/pasted source code or new dependencies).

Third-party code
If you've added third-party code (copy/pasted or new dependencies), please reach out to an architect.
Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

You license your contribution under the same terms as the current repository’s license.
You agree that you have the right to license your contribution under the current repository’s license.

LGTM! Nice that you've found another way for tapping on the search state
could you solve the eslint warnings please?


Consider a OnRenderComplete override on BlazorComponent

Scenario:
ExampleComponent:

@functions(){
    public string DivId {get;set;} = "componentDiv";
    protected override void OnParametersSet()
    {
        RegisteredFunction.Invoke<bool>("initializeSomeJSFunction", new[] { DivId });
    }
}

If the initializeSomeJSFunction in JS wants to access the div componentDiv at this point, it can't as the component has not yet rendered and is therefore not accessible by document.getElementById('componentDiv'). This is blocking porting existing JS libraries to Blazor, if the library requires that context.
If there was an override when the render is completed, then, I assume the elements will be visible to the DOM and can be used within the registeredFunction. If this is by design, what is the recommended approach to tackle a scenario like this?

I agree. It's been in the back of my mind for a while that we'll need this. I agree it's important for integrating with external JS libraries.
We also need some way of giving the Blazor component a way of referencing its own DOM element in this callback.

One important caveat is that this "After render" callback is going to be called asynchronously.  This is inevitable if we want to run the .NET code in a WebWorker, since all communication between the host frame and the web worker is async.
@muqeet-khan Do you think that will be OK for your scenarios?

@SteveSandersonMS  Yes, as long as the elem is visible to the DOM at that moment, that's all I need.
As a side note, here's another crazy land option I landed in yesterday:
JS:
Blazor.RegisterFunction('InitializeJS',function(data){
let elem = document.createElement('p')
elem.setAttribute('id','myNewParagraph');
//also do something useful with data and append it to elem perhaps
return elem;
});

MyComponent.cshtml:

@Html.Raw(_modifiedBlob)

@functions{
 public string SomeBlobOfData {get;set;} = "";
  private string _modifiedBlob {get; private set;} = "";
 protected override OnInit()
 {
   var result = RegisteredFunction.Invoke<RenderFragment\HTMLContent\string>('initializeJS');
   // this is trying to refer to the current component. Or may be the render tree?
   //this.appendDomElem(result);
   // or store in some string and display the string as Raw in the component somewhere.
   _modifiedBlob = result;
 }
}

As an example of this consider something like HighlightJS, where the component may give the registered function a blob of code and hljs will "process" that blob and return the highlighted elems.
May be above is not even needed when #167 is implemented and we could just return string from function and hold the result and use it as @HTML.Raw(SomeBlobOfData) is available.

I think this is very important, because rendering new elements into the DOM and wiring up events afterwards is pretty common. Think of a modal popup with content from blazor component(s) which calls back via an JS-event when it's been closed.

Closing in favour of #691. Note that in #691 I implemented the callback synchronously for consistency with everything else, but there is a reasonable chance we're going to make this and all JS interop async at some point if we implement web worker support.

@SteveSandersonMS  just FYI... OnAfterRender is working beautifully just as we wanted... Here's a sneak peek ...

Looks fantastic! Have you published the charts component in a NuGet package?


PSM-393 Upgrade org.jacoco:jacoco-maven-plugin 0.8.4 -> 0.8.5

This PR contains the following updates:

Package
Update
Change

org.jacoco:jacoco-maven-plugin
patch
0.8.4 -> 0.8.5

Renovate configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

Suggested commit message:
PSM-393 Upgrade jacoco-maven-plugin 0.8.4 -> 0.8.5

See:
- https://github.com/jacoco/jacoco/releases/tag/v0.8.5
- https://github.com/jacoco/jacoco/compare/v0.8.4...v0.8.5


[Feature Request] LDAP password set in cleartext in config.yml file.

Is your feature request related to a problem? Please describe
When integrating LDAP with OpenSearch for authentication, the password to authenticate to your LDAP server is set in cleartext in /etc/opensearch/opensearch-security/config.yml file. It is possible to protect the file in the system to prevent unauthorized access but the information can be viewed in opensearch-dashboard.
Describe the solution you'd like
A solution to this issue would be to use opensearch-keystore to hide the credentials.
The following options need to be secured: bind_dn and password.
Related component
Other
Describe alternatives you've considered
We tried using environment variables to hide the password but it did not work.
Additional context
Steps to reproduce:
From the opensearch-dahsboard, go to security tab then Authentication, and click on view expression, it shows the service account password that was used in clear text.

Is your feature request related to a problem? Please describe
When integrating LDAP with OpenSearch for authentication, the password to authenticate to your LDAP server is set in cleartext in /etc/opensearch/opensearch-security/config.yml file. It is possible to protect the file in the system to prevent unauthorized access but the information can be viewed in opensearch-dashboard.
Describe the solution you'd like
A solution to this issue would be to use opensearch-keystore to hide the credentials.
The following options need to be secured: bind_dn and password.
Related component
Other
Describe alternatives you've considered
We tried using environment variables to hide the password but it did not work.
Additional context
Steps to reproduce:
From the opensearch-dahsboard, go to security tab then Authentication, and click on view expression, it shows the service account password that was used in clear text.

@peternied @dblock Can you transfer this to the security repo?

@opensearch-project/triage Please transfer this to the security repo


Update garden-api.md

The destination is always the path to a directory. This change makes that more clear.

@davidm-pivotalio These docs are now gone, since the REST API is no longer supported, so the team would like to keep this document as simple as possible.

@davidm-pivotalio  - feel free to send PRs against the godoc of the API as that's the documentation we're maintaining these days.


session password

My password is being rejected when I run the MBPmid2010 fix but I can log in with it on the website without problems. What can I do? Please help, urgent.

As said on the FAQ the password you should use is the one you use to log in on your Macbook, not the one you are using on GitHub 🤣

Thank you.  It has
worked, thank you very much!
On Tue, Aug 8, 2017 at 6:27 PM, Julian Poidevin<EMAIL_ADDRESS>wrote:

As said on the FAQ
https://github.com/julian-poidevin/MBPMid2010_GPUFix/blob/master/FAQ.md#troubleshooting
the password you should use is the one you use to log in on your Macbook,
not the one you are using on GitHub 🤣
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/julian-poidevin/MBPMid2010_GPUFix/issues/27#issuecomment-321009115,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdaODFhkUvYPaQt4b_ZkHLJ3_GAfY2Waks5sWIx7gaJpZM4Ow92M
.

You're welcome.

Hi Julian
Would you by any chance have a fix for the MacBook Pro 15" early 2011 with
graphics card issues?
Regards
Tonderayi
On 8 Aug 2017 18:50, "Julian Poidevin"<EMAIL_ADDRESS>wrote:
You're welcome.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/julian-poidevin/MBPMid2010_GPUFix/issues/27#issuecomment-321015195,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdaODFZcd_COzuiBZiS_7WB6ipwoLDb2ks5sWJHEgaJpZM4Ow92M
.

I'm sorry, but as said in https://github.com/julian-poidevin/MBPMid2010_GPUFix/issues/23 I have currently no solution for this problem.

Oh OK, I had not seen it. Thank you
On 10 Aug 2017 19:10, "Julian Poidevin"<EMAIL_ADDRESS>wrote:

I'm sorry, but as said in #23
https://github.com/julian-poidevin/MBPMid2010_GPUFix/issues/23 I have
currently no solution for this problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/julian-poidevin/MBPMid2010_GPUFix/issues/27#issuecomment-321614801,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdaODIlcVHGMbcQ2wlmGZebZ8f5T0ipdks5sWzmFgaJpZM4Ow92M
.


PROPOSAL

May I suggest adding the possibility to group related collection under folders in a tree structure. This will help organize the collections with relations.

This was discussed several times before and for now remains out of the scope as it could clutter the UI.


Load test dss rest server performance

Make a simple(echo) rest server, deploy to a cloud server and run it.
Use gatling(https://gatling.io/) or other tool to test server performance.
And push the echo server code, jar(jave archive) and test results under benchmark repository.

https://github.com/ztkmkoo/dss/blob/develop/benchmark/dss/GUIDE.md

@ParkSale  I want to try this

result.zip
I get that result.
but premature close error is appeared.
please advise me

package org.example

import scala.concurrent.duration._

import io.gatling.core.Predef._
import io.gatling.http.Predef._
import io.gatling.jdbc.Predef._

class RecordedSimulation extends Simulation {

	val httpProtocol = http
		.baseUrl("http://<IP_ADDRESS>:8181")
		.acceptHeader("text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8")
		.doNotTrackHeader("1")
		.acceptEncodingHeader("gzip, deflate")
		.acceptLanguageHeader("ko-KR,ko;q=0.9,en-US;q=0.8,en;q=0.7")
		.userAgentHeader("Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/84.0.4147.105 Safari/537.36")

	val scn = scenario("RecordedSimulation")
		.exec(http("request_1")
			.get("/hello")
			.header(HttpHeaderNames.ContentType, HttpHeaderValues.ApplicationJson)
			.body(StringBody("""Test""")))

	setUp(
  		scn.inject(
		nothingFor(4 seconds),
		atOnceUsers(10),
		rampUsers(10) during (5 seconds),
		constantUsersPerSec(20) during (15 seconds),
		constantUsersPerSec(20) during (15 seconds) randomized,
		rampUsersPerSec(10) to 20 during (10 minutes),
		rampUsersPerSec(10) to 20 during (10 minutes) randomized,
		heavisideUsers(1000) during (20 seconds)
	).protocols(httpProtocol)
)
}

and I simulate with this code

Could you show me the server error log?

ServerLog.txt
Here is a LogFile.

@ParkSale
Thank you for reporting the log file.
It looks some error occurred in at io.github.ztkmkoo.dss.core.network.rest.handler.DssRestHandler.handlingDssRestRequest(DssRestHandler.java:142)
Objects.requireNonNull(context);

It seems, the io request was pushed before the handler was ready..
I will fix it in other issue.

@ParkSale
Could you deploy your test server to public server?
(and which port should i open?)

please open 8181 port. I will use that

@ParkSale
I open the port, please tell me if not working

I have a question that adding server and client spec
what type of file that I add?
And In server, is it sure that using 'cat /proc/cpuinfo' for server spec?

I have a question that adding server and client spec
what type of file that I add?
And In server, is it sure that using 'cat /proc/cpuinfo' for server spec?

Hi, i reply you on the pr. Please check it.


[Doc] What is config option tables_searchable_default

For what it's used tables_searchable_default?
Is not on documentation here https://labs.infyom.com/laravelgenerator/docs/7.0/configuration and seem only used in GeneratorForeignKey class added to defaultSearchable.
Thanks

@thewebartisan7 that flag makes all fields default searchable except primary key and timestamps. I will update the docs.
Thanks for capturing it.

@thewebartisan7 the docs are updated and you can check it here: https://labs.infyom.com/laravelgenerator/docs/7.0/configuration#options


add homework 5

added hw 5
changed 3 files: config.js, index.jsx, step.jsx

Don't commit your personal changes to config.js.
Try to be more descriptive in your commit messages and pull request titles.


Take advantage of WQ categories

@tjdasso has done some great work tackling #1324. As soon as core/mem/disk specification is merged (the core specification is already implemented in his branch), we will have the ability to tell WQ specific resource requirements for a task. A next step that I think would be extremely useful would be to implement the missing pieces in order to make WQ aware of what category the task is in (as a first pass, we can try just using the app name as a category). This will allow us to take advantage of WQ's ability to optimally size tasks, using the specified resource requirements as a first guess.
For another project I'm implementing some bioinformatics pipelines, so I would be an enthusiastic tester for this use case.

cc @dthain, @btovar, @tjdasso

@trshaffer please comment here on the task category capability in the latest WQ executor.

I have a PR in (#1677) that does exactly what @annawoodard proposes. I added two options: one to automatically assign a fresh category for each function, and one to turn on WQ's automatic resource labeling. With both of these turned on, WQ should be able to do a pretty OK job of figuring out resource requirements automatically and subdividing large workers.

Now that #1677 is merged, this should be resolved


⚠ Bofrost.at has degraded performance

In 105b45c, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1447 ms

Resolved: Bofrost.at performance has improved in 29061a9.


Roll engine 3606f7bc2d06..ccf5d619b58a (2 commits)

git@github.com:flutter/engine.git/compare/3606f7bc2d06...ccf5d619b58a
git log 3606f7bc2d06..ccf5d619b58a --no-merges --oneline
2019-08-21<EMAIL_ADDRESS>Dry up fixture comparison in embedder unit-tests. (flutter/engine#11359)
2019-08-21<EMAIL_ADDRESS>Remove engine hash from pom filename (flutter/engine#11356)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (liyuqian@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


analysis feedback changes if you include sky_engine in your project

Repro case and instructions here: https://gist.github.com/sethladd/79b6d1cdcc9d3b29b5e3acc27581944a
We'd expect there to be no warnings for the code in lib/main.dart, however the inclusion of sky_engine package into .packages is enough to trigger the warning.
The warning we get:
[warning] Unsound implicit cast from 'List<dynamic>' to 'List<Bar>'. (/Users/sethladd/tmp/repocase/lib/main.dart, line 8, col 23)

Which, we're told, is something we shouldn't be getting. The code should be valid (we're told).
Tested with:
~/tmp/repocase $ dartanalyzer --version
dartanalyzer version 1.21.0-dev.4.0

cc @danrubel

cc @yyoon

cc @leafp to confirm our code is correct?

That code should be strong mode clean, and I see no errors if I run the regular strong mode analyzer on it (I didn't try to reproduce the sky_engine issue).
I believe that flutter uses their own version of the SDK, so perhaps that has something to do with it?  That error is what I would expect to see if List.map was not properly annotated as a generic method.

I believe that flutter uses their own version of the SDK

I'm analyzing with "dartanalyzer", from homebrew, version 1.21.0-dev.4.0.
The variable here isn't "dartanalyzer" or "flutter analyze", it's "is sky_engine in my .packages".

The variable here isn't "dartanalyzer" or "flutter analyze", it's "is sky_engine in my .packages".

I don't know where the external version of sky_engine lives (the pub.dartlang version looks way out of date, and 404s on the link to the source code), but looking at the internal version I see a copy of the dart sdk.  This makes me.... suspicious... :)  I don't know why it's there if it's not being used, and if it's being used, then it seems relevant to this problem.
It's possible that this is a general analyzer bug - there was a caching issue that @stereotype441 just fixed that caused some strange inference problems, but this looks pretty unlikely to be related to me.
I know very little about how flutter works, but I know there's some voodoo going on with the "dart:flutter" etc URIs.  That seems like the most likely problem to me: that importing sky_engine is causing a different SDK to be used because of some of this voodoo.

I don't know where the external version of sky_engine lives

Oh, sorry. It comes from the Flutter SDK, which you clone from github.
We can try to track this down with Dan R. Meanwhile, thanks for confirming that the code should be strong mode clean. That helps eliminate a variable.

Here's something you can try: make the following change to the code:
  List<Bar> barList = fooList.map/*<Bar>*/((Foo f) => new Bar(f)).toList();

and see what happens.
If you get an error about number of type parameters, then it is an issue with the definition of Iterable.map that you are picking up.
If that makes the error go away, then you have a correct definition of Iterable.map, but something is causing the inference of the generic type parameter to fail.

sky_engine's version of iterable.dart has   Iterable/*<T>*/ map/*<T>*/(/*=T*/ f(E e)) =>       new MappedIterable<E, dynamic/*=T*/>(this, f);
which I guess is correct/up-to-date?

Looks right to me.

But there are failure modes that the test that I suggested above might suss out, so I would try it.
For example, something may be causing the sky_engine sdk to be analyzed in a non-strong mode, resulting in an analyzer context which is non-strong, which is then used to analyze lib/main.dart in strong mode.  Hijinks ensue.

I tried adding the /*<Bar>*/ comment as suggested, and here's the result:
youngseokyoon temp$ dartanalyzer .
Analyzing [.]...
[error] The type '((Foo) → dynamic) → Iterable<dynamic>' is declared with 0 type parameters, but 1 type arguments were given. (/usr/local/google/home/youngseokyoon/Programming/temp/lib/foo.dart, line 12, col 31)
[warning] Unsound implicit cast from 'List<dynamic>' to 'List<Bar>'. (/usr/local/google/home/youngseokyoon/Programming/temp/lib/foo.dart, line 12, col 23)
[lint] Document all public members (/usr/local/google/home/youngseokyoon/Programming/temp/lib/foo.dart, line 1, col 7)
1 error, 1 warning and 1 lint found.

sky_engine is a package built from the flutter/engine repository by our
buildbots, hosted on Google Cloud Storage and then pulled down by the
flutter tool into bin/cache/pkgs/sky_engine corresponding to the
flutter/engine version set in bin/internal/engine.version.
Here are the build rules which produce sky_engine:
https://github.com/flutter/engine/tree/master/sky/packages/sky_engine
As seth says, it's easiest to look at the package by using your local
flutter/flutter checkout and running the flutter tool once to have it
populate bin/cache.
On Tue, Nov 8, 2016 at 5:39 PM, YoungSeok Yoon<EMAIL_ADDRESS>wrote:

I tried adding the // comment as suggested, and here's the result:
youngseokyoon temp$ dartanalyzer .
Analyzing [.]...
[error] The type '((Foo) → dynamic) → Iterable' is declared with 0 type parameters, but 1 type arguments were given. (/usr/local/google/home/youngseokyoon/Programming/temp/lib/foo.dart, line 12, col 31)
[warning] Unsound implicit cast from 'List' to 'List'. (/usr/local/google/home/youngseokyoon/Programming/temp/lib/foo.dart, line 12, col 23)
[lint] Document all public members (/usr/local/google/home/youngseokyoon/Programming/temp/lib/foo.dart, line 1, col 7)
1 error, 1 warning and 1 lint found.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/6769#issuecomment-259313512,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALTvi8NExOSaAYC59TNh7ERBW8LBEp5wks5q8SRVgaJpZM4Ks-iY
.

I tried adding the // comment as suggested, and here's the result:

Ok, so that error means that the analyzer context that is being used to analyze your code was run over the SDK in non-strong mode (and so it picked up a non-generic version of Iterable.map).  I don't know anything about how the flutter modified SDKs work. @pq or @danrubel are probably the best people to talk to here.

Thanks for the analysis! Feels like we're getting closer.

The sky_engine contains an _embedder.yaml file which  affects analysis. By including the sky_engine package, you are causing dartanalyzer to analyze against a different set of dart:* libraries when analyzing lib/main.dart.

Looking at the _embedder.yaml, it points to dart libraries that have the generic methods syntax. It also turns on super-mixins.
What else can toggle "turn off strong mode" ?

Any updates on this?

I'm working towards making the analysis options more explicit via an analysis options "include" directive. Once that is in place, we can remove the modifications that the _embedder.yaml makes to how files are analyzed.

Sounds good. Thanks!

I believe that I have found the problem and have sent out a CL for review with a fix (https://codereview.chromium.org/2523313002/). We were not correctly preserving the options in the SDK when using an SDK described by an _embedder.yaml file.

Thanks for taking a look!
On Wed, Nov 23, 2016, 1:32 PM Brian Wilkerson<EMAIL_ADDRESS>wrote:

I believe that I have found the problem and have sent out a CL for review
with a fix (https://codereview.chromium.org/2523313002/). We were not
correctly preserving the options in the SDK when using an SDK described by
an _embedder.yaml file.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/6769#issuecomment-262635082,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAVZ5Q4tk9JRaAghiCdeSidHHtmeDYTks5rBLEBgaJpZM4Ks-iY
.

@pq Do you know how to tell when this is ready for us to test? How can we connect the next Dart SDK roll back to this issue?

This pull should have the fix:  https://github.com/flutter/flutter/pull/7065.  Just waiting on a review.

How can we connect the next Dart SDK roll back to this issue?

A quick thought would be to create (weekly?) issues to track dev SDK rolls so we can do just this kind of thing.  Don't want to add extra issues if it wouldn't be generally valued though.

If the rule is "you'll get the PR on the next roll of the SDK", that's something we can mentally track. I didn't know if Dart SDK rolls come from a branch or master.

If the rule is "you'll get the PR on the next roll of the SDK", that's something we can mentally track.

👍
We generally track the latest dev push.  If there's a blocker, I try and file a ticket.

BTW, I can confirm that this has been fixed. Thanks!
We've enabled strong-mode back and not seeing any issues for a couple of weeks.
@sethladd Feel free to close the issue.

Great to hear.
On Tue, Dec 20, 2016, 3:08 PM YoungSeok Yoon<EMAIL_ADDRESS>wrote:

BTW, I can confirm that this has been fixed. Thanks!
We've enabled strong-mode back and not seeing any issues for a couple of
weeks.
@sethladd https://github.com/sethladd Feel free to close the issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/6769#issuecomment-268385994,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAVZ28Z3QQXUMeaabArpDo8ofDkipL_ks5rKF_6gaJpZM4Ks-iY
.


keycloak: 19.0.2 -> 20.0.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/keycloak/keycloak/releases.
meta.description for keycloak is: "Identity and access management for modern applications and services"
meta.homepage for keycloak is: "https://www.keycloak.org/"
meta.changelog for keycloak is: ""
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 20.0.1 with grep in /nix/store/5if9dc6v2vc8y1cqqyc87pfyfx04i8xi-keycloak-20.0.1
found 20.0.1 in filename of file in /nix/store/5if9dc6v2vc8y1cqqyc87pfyfx04i8xi-keycloak-20.0.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
keycloak

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5if9dc6v2vc8y1cqqyc87pfyfx04i8xi-keycloak-20.0.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A keycloak https://github.com/r-ryantm/nixpkgs/archive/0ed8b82f1ed18958318070656f049977b594d00f.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5if9dc6v2vc8y1cqqyc87pfyfx04i8xi-keycloak-20.0.1
ls -la /nix/store/5if9dc6v2vc8y1cqqyc87pfyfx04i8xi-keycloak-20.0.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

keycloak

Maintainer pings
cc @ngerstle @talyz for testing.

Tests fail and it conflicts with #199173.


Support for 1.20.4

Are there plans to support 1.20.4?

I'm porting my mods one at a time, when I have the time. When I get around to More Red, it'll probably be 1.20.5 by then, maybe even 1.21 if mojang is faster than me


Sample: Get recycle bin items and export it to CSV

This sample reads the site URL and recycles bin stage value from the user and then exports it to CSV format.

Thank you for this contribution @chandaniprajapati I'm not at my computer at the moment and will process this tomorrow 😁

Tested and small change made to long description in sample.json file, tbh we don't yet use this much in the site - that field would be for the m365 master sample gallery. But other than that, spot on with the submission - Thank you!


Typo in chapter 2

There should be let reg = adderss as *mut u8 instead of let reg = usize as *mut u8 in mmio_write() and mmio_read() (https://osblog.stephenmarz.com/ch2.html).

Fixed. Thanks for your help!


How to access each Num in a fraction?

So from my implementation, Fraction extends AbstractNum, which means the numbers are contained in an Optional Frac, which is basically a kind of Pair. For add/sub/mul, I assume using OOP principles, we have to make use of the add/sub/mul defined in the Num class.
e.g. this.numerator.add(otherFraction.numerator)
Through the optional, I can only map the nums contained within the fraction's pair, but I can't access the nums in the other fraction pair with the map function, I think? Since it's wrapped in an Optional. Get is banned. I'm not sure how to go about doing this. Is there a lecture slide I should look at that will hopefully give me some hints?

Figured it out
do a.flatMap(x -> b.map(y -> y.add(x))) to return a new optional num

thanks! your approach helps me a lot


Cleaner

Forgot description --
I now have an index.js file in schema which just acts as an entry point for the schema directory. Any demo schema we use will be referenced via this file so we can switch between demo schemas with ease.


deprected support ?

Hi, i wonder if this lib supports deprecated field declaration.
Thank you for your work.

Not currently no, stemming from the fact that Zod doesn't so there's nothing to map.


Revert "Siddhi Doc Gen Plugin - change the mkdocs gh-deploy to a manual git push"

Reverts wso2/siddhi#519

Fixing build


DOCS Generate secret key

bin2hex(random_bytes(32)); might be a better suggestion?

I considered suggesting something PHP based, but as it's generated for your .env file, I thought a system based solution was fine. Happy to list both though 🙂

good point


HandleAddBallot is not defined

On edit distribution after adding to the ballot, while clicking on save => the error pops up

Reference: #52

fixed

Fixed in #59


#187 implement signal generator controller

Checks

[ ] Debugging completed
[x] Please debug me

Related Issue
This PR closes #187

Codecov Report
Base: 92.21% // Head: 90.25% // Decreases project coverage by -1.95% :warning:

Coverage data is based on head (1101b74) compared to base (be9787c).
Patch coverage: 33.55% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #200      +/-   ##
==========================================
- Coverage   92.21%   90.25%   -1.96%
==========================================
  Files          63       77      +14
  Lines        2837     3426     +589
==========================================
+ Hits         2616     3092     +476
- Misses        221      334     +113

Impacted Files
Coverage Δ

neclib/devices/signal_generator/fsw0010.py
24.59% <24.59%> (ø)

neclib/devices/signal_generator/fsw0020.py
24.59% <24.59%> (ø)

.../devices/signal_generator/signal_generator_base.py
72.00% <72.00%> (ø)

neclib/devices/signal_generator/__init__.py
100.00% <100.00%> (ø)

neclib/typing.py
71.42% <0.00%> (-9.53%)
:arrow_down:

neclib/devices/selector.py
90.62% <0.00%> (-2.48%)
:arrow_down:

neclib/configuration.py
88.76% <0.00%> (-1.01%)
:arrow_down:

neclib/devices/__init__.py
100.00% <0.00%> (ø)

neclib/coordinates/__init__.py
100.00% <0.00%> (ø)

tests/utils/test_data_utils.py
100.00% <0.00%> (ø)

... and 23 more

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

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


PostPreview should use mb_substr

Expected behavior

post.get_preview(40) should get ~40 words of a post
Actual behavior

If the post is in Chinese, the excerpt is as long as it takes to find 40 english or numerical words. Please use mb_substr() instead of substr()
Steps to reproduce behavior

What version of WordPress, PHP and Timber are you using?

How did you install Timber? (for example, from GitHub, Composer/Packagist, WP.org?)

Thanks for the heads-up on this @maxfenton — would you be able to submit a PR to address this? Thanks!

Assuming you want test cases and not just a quick replace of substr( with mb_substr( -- probably not for a couple of week.
(As a note, Craft CMS uses mb_substr and I've seen some issues where it trims a few extra characters off the last word, but haven't gotten to the bottom of that.)

If this is still an issue, we’re happy to accept a pull request that solves this. In the meantime, I’m closing this.


[BOT PR] to update dependencies of istio

This PR will NOT be merged automatically once checks are successful. NEEDS approval from OWNERS.
none

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: istio-testing
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
If they are not already assigned, you can assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM
The auto-update can merge instead of #13556 (I didn't realize this was posted).

/assign @costinm @geeknoid @linsun @rshriram
PTAL. This is not auto merged. CNI needs to be updated https://github.com/istio/istio/issues/13171#issuecomment-485947053

@howardjohn: GitHub didn't allow me to assign the following users: linsun.
Note that only istio members and repo collaborators can be assigned.
For more information please see the contributor guide

In response to this:

/assign @costinm @geeknoid @linsun @rshriram
PTAL. This is not auto merged. CNI needs to be updated https://github.com/istio/istio/issues/13171#issuecomment-485947053

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


overrightarrow、overleftarrow  nonsupport

These commands are not currently supported. I'll change this "bug" to a feature request.

Note: The \overrightarrow is available to a degree via the \vec command.
It doesn't extend the arrow over multiple characters but is pretty close.

Looking forward to your update。

Looking for a volunteer to implement this.  This is a non-trivial change since the over arrows are not in the math fonts.


Allow user to specify a port

Based on https://laracasts.com/forum/?p=939-problem-with-laravel-envoy/p1#p4358, you can add -p port# as part of the server string. A hack more than a real solution, I'll agree with that. I guess we could hide that fact by providing a port key which would do the magic of doing $server.' -p '.$port in the back.

Well, it might be the solution, but we quote the server informations:
@servers(['web' => '-A "'. $server .'"'])

It could be a good idea to add the port in the configuration and generate the proper connection string.

@xel1045 So we're already using it in a hacky way too :)

Sure! :yum:

You can now specify it using the ssh_options configuration.


feat: add spec.template.spec.securityContext

📑 Description

Adding spec.template.spec.securityContext in order to solve issues when working with Policy Management Tools such as Open Policy Agent and Gatekeeper (as in my case psp-pods-allowed-user-ranges). User and group were taken from https://github.com/k8sgpt-ai/k8sgpt/blob/main/container/Dockerfile#L37. One could also think about adding runAsNonRoot: true and fsGroup: 65532.
✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

ℹ Additional Information

Even though being more dynamic, other prominent project such as Grafana handle the securityContext in a similar way, see https://github.com/grafana/helm-charts/blob/main/charts/grafana/values.yaml#L142-L146 & https://github.com/grafana/grafana/blob/main/Dockerfile#L106 & https://github.com/grafana/helm-charts/blob/main/charts/grafana/templates/_pod.tpl#L9-L12

How about making this configurable in the values.yaml?

Done, see above.

@k8sgpt-ai/k8sgpt-maintainers

Should'nt we have the same logic in the operator too?

Should'nt we have the same logic in the operator too?

Would most probably make sense ....

Thanks for your contribution, if you have time please add to the operator @ChristianBieri1995 !


Fix frost-select not clearing

#PATCH#
CHANGELOG

Fixed issue with frost-select and consumer clearing selectedValue but DOM not clearing input.

Approved - I'm looking into the DOM cleanup issues with remodal to see if we can address that as well


I don't get it, where is the file mediaelement-and-player.min.js?

How can I start using this library if the required file "mediaelement-and-player.min.js" does not exist in the downloaded zip? WTF?
 Thanks.

Everything you need is in the build folder. Examples are in the demo folder.

I'm sorry. I'm drunk today.


SQL log failed with mssql.UniqueIdentifier type

My struct has a memeber of mssql.UniqueIdentifier type.
type Demo struct{
ID mssql.UniqueIdentifier
}
The output of sql logs, you will see something like this:
'<binary>'  or  '������UC��A��Y�C'
I check the code:

mssql.UniqueIdentifier type has methods:
func (u UniqueIdentifier) Value() (driver.Value, error)        for driver.Valuer interface of sql package.
func (u UniqueIdentifier) String() string                             for fmt.Stringer interface.

In gorm logger/sql.go  line: 34
Method ExplainSQL,  it will match type  driver.Valuer (line: 60) first, then got the internal type []byte, and match []byte again,
so it will be dispaly as '<binary>'  or gabled codes.

I think if we're match fmt.Stringer before driver.Valuer, it can correctly dispaly UUID.
But i'am not sure whether this change will had bad impact on other types.

A simple mitigation if logging is necessary is to wrap the mssql.UniqueIdentifier in a custom struct with Scan/Value methods implemented (e.g. type SomeName string) which calls the underlying mssql.UniqueIdentifier Scan method for scanning and cast to string (in my example) for the Value method.


Process of incorrect scan in launchers stopped by time-out only

Description
Process of incorrect scan in launchers stopped by time-out only. Need to stop this process with correct message
Steps to reproduce

Go to zafira Launchers
Tap rescan for incorrect repo e.g. "capabilities_file"
Tap Re-Scan

Actual behavior
User wait for 2+ mins. It looks like endless process.
Expected behavior (if known)
Need to stop this process with correct message
Environment
http://<IP_ADDRESS>/app

@okamara why do you mean by "incorrect repo"?

zafira version: ui v.1.27 | server v.4.1.81 | client v.4.1.64
In my as example QA made a mistake in repo name and write "test_file" instead of "test_slash". "test_file" repo is not exist.

@Boniara let's see what can be done on our end from back-end perspective - we should either timeout after some reasonable wait time or see what kind of response we get from GitHub

Cause of crash was incorrect branch name, not repo, fixed and available on stage env.

Verified in ui v.1.19-SNAPSHOT | server v.1.475-SNAPSHOT | client v.1.0-SNAPSHOT.


numpy parity: financial functions #128

complement irr/mirr/nper/rate

thanks for doing this! just a few minor-ish suggestions

Thank for your instructment.  I'm a newer to Scala/breeze and not familiar with some iterator operation.
:)


Run batch insert in a transaction

Only insert if fields actually change
Run batch insert in transaciton
Fix potential NPE on updating a field which was, and is, null

closes https://github.com/apollographql/apollo-android/issues/303

Looks like we still have spurious failures :(


Bashism breaks build on OBSD

The /script/post-build.sh file is invoked during the build phase. It's hashbang asks to be interpreted by bash, but bash is not necessarily available on BSD systems.
Modifying the hashbang to #!/bin/sh seems to be working on OpenBSD. Please consider to not use bashism in the script.

Thanks for the heads up -- I converted the script (and a couple others that are essential to the regular build process) from bash to sh. https://github.com/clangen/musikcube/commit/1f60f7c1d82481b45894f2763aeb7f090171d554

Thanks a lot!


It is too difficult to style rating icons (fill & stroke color) when using

Summary
There is no way to convey fill and stroke color information through the Rating component.
Basically, the rating icons (Star, Heart, etc.) take fillColor and strokeColor props. Those should be exposed via the Rating component.
Basic example
I propose a very simple API on the Rating component:
<Rating
  id='my-rating-wrapper'
  iconFillColor='#f0f0f0'
  iconStrokeColor='#0f0f0f'
  size={10}
  total={5}
  {rating}
/>

This would unroll to an array of icons (default Star) that would have each fillColor and strokeColor prop set to the value of the iconFillColor and iconStrokeColor props of Rating.
This proposal also implies the following:

One color for all icons
No support for gradients

Motivation
Rating should expose an API for customizing its child components.

Thank you for the suggestion. This will be in the next version.


_tkinter.TclError: no display name and no $DISPLAY environment variable

Dear Catalyst Maintainers,
Before I tell you about my issue, let me describe my environment:
Environment

Operating System: (Windows Version or $ uname --all)
Python Version: $ python --version
Python Bitness: $ python -c 'import math, sys;print(int(math.log(sys.maxsize + 1, 2) + 1))'
How did you install Catalyst: (pip, conda, or other (please explain))
Python packages: $ pip freeze or $ conda list

Now that you know a little about me, let me tell you about the issue I am
having:
Description of Issue
i'm running mininet/examples/miniedit.py in MIninet
i received this issue.
MiniEdit running against Mininet 2.2.2
Traceback (most recent call last):
File "mininet/examples/miniedit.py", line 3579, in
app = MiniEdit()
File "mininet/examples/miniedit.py", line 1110, in init
Frame.init( self, parent )
File "/usr/lib/python2.7/lib-tk/Tkinter.py", line 2537, in init
Widget.init(self, master, 'frame', cnf, {}, extra)
File "/usr/lib/python2.7/lib-tk/Tkinter.py", line 2049, in init
BaseWidget._setup(self, master, cnf)
File "/usr/lib/python2.7/lib-tk/Tkinter.py", line 2024, in _setup
_default_root = Tk()
File "/usr/lib/python2.7/lib-tk/Tkinter.py", line 1767, in init
self.tk = _tkinter.create(screenName, baseName, className, interactive, wantobjects, useTk, sync, use)
_tkinter.TclError: no display name and no $DISPLAY environment variable
Sincerely,
`sekar

Duplicate of #39, where this is resolved. It is also covered in our documentation here:
https://enigmampc.github.io/catalyst/install.html#macos-virtualenv-conda-matplotlib


Breadcrumb navigation

Overview
The secondary navigation for the Next Gen app is the breadcrumb navigation that appears at the top of the Tier 2, Tier 3, and Tier 4 pages:

Design

An example of the full breadcrumbs can be found here: https://www.figma.com/file/bVtCogdwiXj3ndpk4AWXCx/[working]-Milestone-1%3A-Read-and-Execute?node-id=1297%3A101653
The breadcrumb component can be found here: https://www.figma.com/file/bVtCogdwiXj3ndpk4AWXCx/[working]-Milestone-1%3A-Read-and-Execute?node-id=520%3A48939

Acceptance Criteria

[ ] Users should be able to navigate using breadcrumbs from any of the tier 2-4 pages diagrammed above

[ ] Breadcrumbs should not appear on tier 1 pages

Reopening to implement the correct rendering for run timestamp, protocol name, and 'Protocol Runs' link.


Create Test Suite to validate Round Trip

Create a test suite to validate round trip for components.
This already exists for modules, an example is dfinity's implementation here. A simple way to generate a test suite is convert all of these to components but that may not cover any edge cases or nuances that are specific to components.
Another idea is to use spin microservices but those are too big and tough to debug.
Also look into wasm spec tests and the wasm benchmarking platform wish you were fast for ideas on how to generate test cases.

There seems to be a test suite for the component model coming at some point: https://github.com/WebAssembly/component-model

We currently cant create a roundtrip for the spin test case https://www.diffchecker.com/pXS27KAs/
#25


(#319) Add _Statement suffix to all of the Statement kind types

Close #319
On stream you mentioned Bind_ suffix in some of those types. So I also added this:
// TODO: Remove redundant prefix `bind` from *_Statement types

@kolumb looks goode to me! :+1: Thank you for your contribution!


Error on Update - paperless-ngx

Please verify that you have read and understood the guidelines.
yes
A clear and concise description of the issue.
running the update script I get the following error:
bash -c "$(wget -qLO - https://github.com/tteck/Proxmox/raw/main/ct/paperless-ngx.sh)"

Updating to v2.13.2
[ERROR] in line 93: exit code 0: while executing command pip install -r requirements.txt &> /dev/null

What settings are you currently utilizing?
Default Settings
Which Linux distribution are you employing?
Debian 12
If relevant, including screenshots or a code block can be helpful in clarifying the issue.
No response
Please provide detailed steps to reproduce the issue.
No response

Execute rm -rf /usr/lib/python3.*/EXTERNALLY-MANAGED in the LXC console and try again.

@MickLesk I see this one regularly, we should add an announcement within the discussions so we can revere to it, like tteck did.
(https://github.com/tteck/Proxmox/discussions/3300)

Didn't solve it - same error again
I didn't have a dirctory called 'EXTERNALLY-MANGED" in any of the python3.* - I have python3.9 and python3.11
how to sove it?


Automate '.env' creation

Idea
If a user tries to run the script without making .env:

Make an input in console asking for everything needed (i.e API key).
Create a .env with everything in it

ATTENTION
Please refer to docs/CONTRIBUTING.md before doing anything.

Hey @AdiKsOnDev, I'm working on this issue right now.

Solved by @WaterNewt
Thanks!


wikipedia data

I saw instructions of retrieving CommonCrawl data, how to retrieve Wikipedia data mentioned in the paper? Thanks

Hi @seahou3e, apologies about the late response.
Looking at the paper, I'm not sure which sentence you are referring to for the Wikipedia data.
If you mean you want to select the Wikipedia data that's a subset of the Common Crawl collection, you can first generate the entire Common Crawl data following instructions at https://github.com/apple/ml-qrecc/tree/main/collection. Then, in the .jsonl files created, select Wikipedia pages by filtering on the id field of each JSON object, which has the url of the webpage as the prefix.

I see, thanks. Closing the issue.


Verification dataset

could u please tell me how to aquire the verification datasets such as "lfw.bin"
THX

done

done

do you know  how to aquire the verification datasets such as "lfw.bin"?
I have the same problem now.

You can try this pipeline(https://github.com/umairanis03/facial-recognition/blob/8344e040ea61a14a70a4e12a2c097540c3b5b5de/validation_prep.md) and make it by yourself.
You can also search on the Internet for released bin files

You can try this pipeline(https://github.com/umairanis03/facial-recognition/blob/8344e040ea61a14a70a4e12a2c097540c3b5b5de/validation_prep.md) and make it by yourself.
You can also search on the Internet for released bin files
Thank you very much for your quick reply.

I found a very good answer(https://github.com/Lingcan-M/verification-datasets-such-as-lfw.bin), hope it helps others

Nice, wish your work going smoothly

You can try this pipeline(https://github.com/umairanis03/facial-recognition/blob/8344e040ea61a14a70a4e12a2c097540c3b5b5de/validation_prep.md) and make it by yourself.
You can also search on the Internet for released bin files

Thank you very much for your quick reply. I found a very good answer(https://github.com/Lingcan-M/verification-datasets-such-as-lfw.bin), hope it helps ot

You can try this pipeline(https://github.com/umairanis03/facial-recognition/blob/8344e040ea61a14a70a4e12a2c097540c3b5b5de/validation_prep.md) and make it by yourself.
You can also search on the Internet for released bin files

Thank you very much for your quick reply. I found a very good answer(https://github.com/Lingcan-M/verification-datasets-such-as-lfw.bin), hope it helps others

Hello, I want to evaluation on RFW dataset. But I don't know how to get 'African_test.bin', 'Asian_test.bin', 'Caucasian_test.bin', 'Indian_test.bin'. Do you have them?
Thank you very much and have a nice day~


As a user I can view the list of archived tasks

... so that I can see what has been done

Subtask of #36


Question about CI

@ocefpaf I have a question about the CI for feedstock, Do the recipe maintainers have access to run the CI tests for their feedstock and can they restart the CI anytime, in case it fails?

Yep. They do :smile:

Unfortunately, I have no permissions for this actions @ocefpaf
No buttons in Appveyor

Unactive button in CircleCI

ping @ocefpaf


Intellisense not working

Environment data
dotnet --info output:
$ dotnet --info
.NET Command Line Tools (1.0.0-rc4-004711)

Product Information:
 Version:            1.0.0-rc4-004711
 Commit SHA-1 hash:  3eee54e4f9

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.12
 OS Platform: Darwin
 RID:         osx.10.12-x64
 Base Path:   /usr/local/share/dotnet/sdk/1.0.0-rc4-004711

VS Code version: 1.8.1
C# Extension version: 1.7.0-beta1 (installed to try and see if it helped, which it didn't)
Steps to reproduce
Opening a .NET Core project with a .csproj and .sln file.
Expected  behavior
Intellisense.
Actual behavior
Intellisense broken. All I get is the flame icon saying "Switch projects". When I do, it runs for a while and goes back to the same mode.
Here is the output:
Updating C# dependencies...
Platform: darwin, x86_64 (osx.10.11-x64)

Downloading package 'Mono Runtime (macOS)' (1544 KB) .................... Done!
Downloading package 'Mono Framework Assemblies' (5828 KB) .................... Done!
Downloading package 'OmniSharp (Mono 4.6)' (16807 KB) .................... Done!
Downloading package '.NET Core Debugger (macOS / x64)' (38602 KB) .................... Done!

Installing package 'Mono Runtime (macOS)'
Installing package 'Mono Framework Assemblies'
Installing package 'OmniSharp (Mono 4.6)'
Installing package '.NET Core Debugger (macOS / x64)'

Finished

Here is the .csproj file:
<Project Sdk="Microsoft.NET.Sdk" ToolsVersion="15.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">

  <PropertyGroup>
    <TargetFramework>netcoreapp1.0</TargetFramework>
    <DebugType>portable</DebugType>
    <AssemblyName>zup</AssemblyName>
    <OutputType>Exe</OutputType>
    <PackageTargetFallback Condition=" '$(TargetFramework)' == 'netcoreapp1.0' ">$(PackageTargetFallback);dnxcore50</PackageTargetFallback>
  </PropertyGroup>

  <ItemGroup>
    <EmbeddedResource Include="**\*.resx" />
    <EmbeddedResource Include="compiler\resources\**\*" />
  </ItemGroup>

  <ItemGroup>
    <PackageReference Include="NLog">
      <Version>5.0.0-beta04</Version>
    </PackageReference>
  </ItemGroup>

  <PropertyGroup Condition=" '$(Configuration)' == 'Release' ">
    <DefineConstants>$(DefineConstants);RELEASE</DefineConstants>
  </PropertyGroup>

  <Import Project="$(IlcPath)\Microsoft.NETCore.Native.targets" Condition=" '$(IlcPath)' != '' " />
</Project>

(Note: it used to work for me, but stopped when I upgraded the dotnet cli tools to rc4)

@perlun: Have you run dotnet restore? Currently, auto-restore for package references (https://github.com/OmniSharp/omnisharp-vscode/issues/770) is not implemented for .csproj yet.

@DustinCampbell yep, but doesn't help. Here is the OmniSharp output right now:
Starting OmniSharp server at 2/1/2017, 3:30:04 PM
    Target: /Volumes/extra/git/perlun/zup/zup.sln

OmniSharp server started wth Mono
    Path: /Users/plundberg/.vscode/extensions/ms-vscode.csharp-1.7.0-beta1/bin/run
    PID: 19888

OmniSharp: -s /Volumes/extra/git/perlun/zup/zup.sln --hostPID 19820 --stdio DotNet:enablePackageRestore=false --encoding utf-8 formattingOptions:useTabs=false formattingOptions:tabSize=4 formattingOptions:indentationSize=4
{"Event":"log","Body":{"LogLevel":"INFORMATION","Name":"OmniSharp.Startup","Message":"Omnisharp server running using Stdio at location '/Volumes/extra/git/perlun/zup' on host 19820."},"Seq":1,"Type":"event"}
[INFORMATION:OmniSharp.DotNet.DotNetProjectSystem] Initializing in /Volumes/extra/git/perlun/zup
[INFORMATION:OmniSharp.DotNet.DotNetProjectSystem] Auto package restore: False
[INFORMATION:OmniSharp.DotNet.DotNetProjectSystem] Update workspace context
[INFORMATION:OmniSharp.DotNet.DotNetProjectSystem] Resolving projects references
[INFORMATION:OmniSharp#MSBuild] MSBUILD_EXE_PATH environment variable set to /Users/plundberg/.vscode/extensions/ms-vscode.csharp-1.7.0-beta1/bin/omnisharp/msbuild/MSBuild.dll
[INFORMATION:OmniSharp#MSBuild] MSBuildExtensionsPath environment variable set to /Users/plundberg/.vscode/extensions/ms-vscode.csharp-1.7.0-beta1/bin/omnisharp/msbuild
[INFORMATION:OmniSharp#MSBuild] MSBuildSDKsPath environment variable set to /Users/plundberg/.vscode/extensions/ms-vscode.csharp-1.7.0-beta1/bin/omnisharp/msbuild/Sdks
[INFORMATION:OmniSharp#MSBuild] Detecting projects in '/Volumes/extra/git/perlun/zup/zup.sln'.
[INFORMATION:OmniSharp#MSBuild] Loading project from '/Volumes/extra/git/perlun/zup/zup.csproj'.
Discovered Mono file path: /Library/Frameworks/Mono.framework/Versions/Current/Commands/mono
Resolved symbolic link for Mono file path: /Library/Frameworks/Mono.framework/Versions/4.8.0/bin/mono-sgen32
[INFORMATION:OmniSharp#ProjectFileInfo] Using TargetFrameworkRootPath: /Library/Frameworks/Mono.framework/Versions/4.8.0/lib/mono/xbuild-frameworks
[INFORMATION:OmniSharp#MSBuild] Add project: /Volumes/extra/git/perlun/zup/zup.csproj
[INFORMATION:OmniSharp#MSBuild] Update project: zup
[INFORMATION:OmniSharp.Script.ScriptProjectSystem] Detecting CSX files in '/Volumes/extra/git/perlun/zup'.
[INFORMATION:OmniSharp.Script.ScriptProjectSystem] Could not find any CSX files
[INFORMATION:OmniSharp.Startup] Configuration finished.

I uninstalled rc4 and went back to preview4 instead, using this link: https://github.com/dotnet/core/blob/master/release-notes/preview4-download.md
Works like a charm now... Closing, someone else can reopen if they feel like digging more into this.


Cool stuff

I am so happy for found this.the code is more clear then cloxen's.
However, you need to know that pubg game do not give any bps by use this script.
you need make player run to give travel distance date and quit game after 2m20s.
this script can derank for now
I am look forward to the progress of this project. thx a lot.

好哒, 抽空做一做刷金币的


TODO: create a new function addClassName

Not sure how I feel about having to explicitly add a space this way and above when you put classnames as function params. It seems like you're always prepending the space. It would be cleaner and less error prone if you created a new function addClassName which would append a space (if needed) and the classname and then trim excess whitespace. Keep in mind that since this is JS, you have a pretty rich set of string methods available.
Scroll down to see all your options:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String

classList.add() is actually much cleaner way I found out.