Allow using a different name for collecting

I would like a way to specify a different name for the collect function in the collecting macro. This would be useful in scenarios where you have nested collecting macros, or want to call the collect method inside an iterate:iter form.

I have been thinking about a macro for named collectors for a while, but it's not something I use myself so I'm not sure what the syntax should be.
One possibility would be something simple like with-collectors, where you name the collectors and the lists are returned as multiple values.
Another possibility is to assimilate the collect macro from CMUCL, the self proclaimed "ultimate collection macro", which is apparently useful enough to have made it into the code of most every Lisp implementation. E.g., SBCL.
One concern I have is that collecting in Serapeum has different semantics than most classical collecting macros; the collector is bound to a function instead of a macro, so it can escape the collecting form. I'm not sure that would be the right thing for a more general collecting macro.
Any thoughts?

Some prior art:
Cliki: http://www.cliki.net/COLLECTING
Discussion on comp.lang.lisp (2001): https://groups.google.com/forum/#!msg/comp.lang.lisp/I8KpggxO-P4/9O5M81Raef0J

One more consideration: I would like to avoid, as much as possible, package conflicts with Iterate. (This is why Serapeum no longer exports some trivial collection macros like minimizing, maximizing, and so forth).
Of course Iterate itself is also quite a nice general-purpose collection macro:

(defmacro iterating (&body body)
  `(iterate (repeat 1)
     ,@(butlast body)
     (finally (return ,(car (last body))))))

(iterating
  (collect 1 into xs)
  (collect 2 into ys)
  (values xs ys))
;; => '(1) '(2)

I've added simple with-collector and with-collectors macros. They're easy to understand, and since collecting can be defined in terms of with-collector, they don't require special maintenance.


Can the markdown.md and markdown.png files be hosted on github repo?

This way, when a user clicks on them they are directed to github to do the download.
This relates to the Persona method.

@susanjrobertson any idea?

I'm not sure what you are asking exactly @iamjessklein.
If it's do you want to link to the repo version (such as this: https://github.com/bocoup/opendesignkit/blob/master/resource-materials/persona-template.md) then yes, we just change the link in the method.

yes, that's what I'd like for us to do. (right now it feels pretty weird).


preventing pagination flashing

There's a way to prevent flashing with pagination?
When i create a big table for some seconds it appears complete before being divided in page; there is a way to prevent this?
Thanks and sorry for my english

There is no way to prevent this, unless you hide the table using CSS then display it when loading is done. You will have to use Javascript to get the loading event and then trigger the element visibility. I can't give you any code now, but maybe this is getting you on the right track.

Hi wolffe, i already used this kind of solution, but i was wonder if there was a native solution to that problem. i mean, when i call the function i already printed on dom the table so maybe it could divide in page without wait full dom ready.

I don't think there's any solution to this issue. I am using it for images, thousands of them, and I had to move on to an AJAX solution because of this limitation. What you can do is to simplify the table's structure, remove any inline styles, anything that might make loading faster.

hi wolffe, finally i decided to use the hide/display solution with css but there's no pagination events so i don't know how to do it, have u any ideas? i tried with mutationobserve but with no results. i used that kind of solutions with other js library but they all have some callback to do it. thanks in advance

No idea. I stopped using ListJS a while ago and wrote my own library, for my specific needs. ListJS looks abandoned.


Main added 2021-12-05-Polygon.md

Add article about Polygon

Deleted ## Additional Details section as it is not included in standard sections

/qualitycheck

/qualitycheck

/payout


Get predictions on input data

Hi,
When using NAR module, I want to see a rank list for predictions in other to have intuitive evaluation on recommendations articles after training process. But I only see the evaluation code to calculate metrics. Can you help me to get this ranking list?
Thank you very much.

Hi. Sorry for the late response. If you enable the hyperparameter --save_eval_sessions_recommendations at on of the NAR trainer script (e.g. nar_trainer_gcom.py), a file named 'eval_chameleon_recommendations_log.json' will be output with the labels (next-clicks) and the ranked items. Take a look in the save_sessions_chameleon_recommendations_log() method

Hi. Sorry for the late response. If you enable the hyperparameter --save_eval_sessions_recommendations at on of the NAR trainer script (e.g. nar_trainer_gcom.py), a file named 'eval_chameleon_recommendations_log.json' will be output with the labels (next-clicks) and the ranked items. Take a look in the save_sessions_chameleon_recommendations_log() method


Enable token encryption on macOS

Describe the bug
By default, on macOS, azure login places its OAuth tokens in .azure/msal_token_cache.json.
The only documentation for how to protect the tokens using Keychain appears to be in the code here:
https://github.com/Azure/azure-cli/blob/3ae3c61737ab5203de56d5c1328ae5462584d631/src/azure-cli-core/azure/cli/core/util.py#L1353-L1361
The comment here is misleading, as it says EXPERIMENTAL: Use core.encrypt_token_cache=False to turn off token cache encryption. as if the default value is True, but it's actually False.
At least, encrypt_token_cache does not appear in the list of "core" configuration values here:
https://learn.microsoft.com/en-us/cli/azure/azure-cli-configuration#cli-configuration-values-and-environment-variables
MSAL for Python also gives a somewhat misleading summary of its security properties, as it says:
https://github.com/AzureAD/microsoft-authentication-extensions-for-python#microsoft-authentication-extensions-for-python

MAC - The MAC KeyChain is used.

Which is not true in the configuration that the azure CLI actually uses it.
Finally, once encrypt_token_cache is set to true, the keychain service name and account name are set to "my_service_name" and "my_account_name" here: https://github.com/Azure/azure-cli/blob/3ae3c61737ab5203de56d5c1328ae5462584d631/src/azure-cli-core/azure/cli/core/auth/persistence.py#L44
I verified that this is actually how it's stored in Keychain Access:

Related command
az login
Errors
No errors.
Issue script & Debug output
No relevant debug output.
Expected behavior
In general the credential storage situation just ought to be tightened up:

encryption / storage in macOS keychain should be the default
encrypt_token_cache ought to be documented
credential labels seem inappropriately generic, and it should say something more like a service name of "Azure CLI" and an account name of the user's email address.
credentials also really ought to be stored in separate keychain items so that applications can be granted access control to, for example, test credentials without granting them access to production credentials in the same keychain.

There's also the msal_http_cache.bin file which contains cookies and stuff, which maybe should also be encrypted somewhere; i don't know the privilege level of these cookies but it seems relevant to look at them.
Environment Summary
azure-cli                         2.50.0 *

core                              2.50.0 *
telemetry                          1.0.8 *

Dependencies:
msal                              1.22.0
azure-mgmt-resource             23.1.0b2

Python location '/opt/homebrew/Cellar/azure-cli/2.50.0_1/libexec/bin/python'
Extensions directory '/Users/glyph/.azure/cliextensions'

Python (Darwin) 3.10.12 (main, Jun 20 2023, 19:43:52) [Clang 14.0.3 (clang-14<IP_ADDRESS>.1)]

Legal docs and information: aka.ms/AzureCliLegal

Additional context
I am filing this as a regular bug and not as a security issue because I don't see an exploit here per se, and the tools don't directly claim any security properties that are subverted by this misconfiguration.

Thank you for opening this issue, we will look into it.

Currently Azure CLI doesn't support token encryption on macOS and Linux (https://github.com/Azure/azure-cli/issues/23495). I am marking this issue as a Feature Candidate.
There is a draft work https://github.com/Azure/azure-cli/pull/20636 to support token encryption on macOS, but currently there is no ETA.

The only documentation for how to protect the tokens using Keychain appears to be in the code here

This is merely Azure CLI's source code. It is not public documentation.

At least, encrypt_token_cache does not appear in the list of "core" configuration values here

This is expected, as core.encrypt_token_cache is only used to disable token cache in some edge cases on Windows. Using it to enable token encryption on macOS or Linux is not supported.

@jiasli #20636 refers to a pull request that hasn't been merged for quite some time. Will it be merged and made available in an upcoming release or is that blocked?

The discussion in PR #20636 assumes an implementation using Apple Keyring / Keychain. One could instead allow configuration of an arbitrary password management command. This would allow Linux and Mac users to encrypt their tokens.


how to implement emoji

:moneybag: for example will return in the textview as it is ":moneybag:" not an emoji icon, what's really the idea about implementing this emoji autocompletion

yes thanks, i have this solved, got the implementation of emoji on my own project. I just gonna do it in my back end as well. :+1:

@dzenbot - is there a way to see the emojis in action in the demo? I'm running Messenger-Storyboard as the demo app but do not know how to run the autocomplete for emoji system...

:+1: Thanks - got it all figured out!


docs: update attribution for recently added libraries

Thank you for sending a pull request :heart:
Checklist
Please review the guidelines for contributing code to this repository. And check [x] all the boxes that apply:

[ ] This request pulls a feature/bugfix branch (right side) from my fork. Not my master.
[ ] This request pulls against ockam-network/ockam’s develop branch (left side).
[ ] Build succeeds ./build with no linter warnings or test failures.
[ ] All code matched our code formatting conventions.
[ ] Commit messages match our conventions and all commits
are signed.
[ ] This request includes tests and documentation for all new code.

Proposed Changes
Please describe the changes in your pull request.
If this pull request resolves an already recorded bug or a feature request, be sure to link to that issue.

@malnick I suspect testify added other indirect deps, could you confirm... if it did we should add those as well

I was going off these two recent changes to go.mod:

https://github.com/ockam-network/ockam/commit/2f562f21285eb3b374864eab8fd36c7914217d29#diff-37aff102a57d3d7b797f152915a6dc16
https://github.com/ockam-network/ockam/commit/dbaa503eccdb79d099c86ce4e3f6cc0d275e3d52#diff-37aff102a57d3d7b797f152915a6dc16

Do we want to include the library the library uses? Is our attribution only limited to what is in our go.mod?

since we're embedding the lib and its libs ... I think we inherit the obligation to give attribution.. so yes.

Do we want to include the library the library uses?

found the commit that caused the libs I couldn't account for
https://github.com/ockam-network/ockam/commit/d7ad283ca2f96bfa73698699dd87a243bf13800d#diff-37aff102a57d3d7b797f152915a6dc16
could you look those up and add their licenses as well, please

@mrinalwadhwa I updated the NOTICE with the specified libraries in that last commit. Let me know if this covers the bases.

@malnick reindent the newly added licenses .. the editorconfig linter currently complains ./build lint because .editorconfig file currently expects space indents and indents in multiples of 4 spaces

@mrinalwadhwa fixed


Update deprecated examples and comments to fit the style

The PR should summarize what was changed and why. Here are some questions to
help you if you're not sure:

What behavior was changed?

Delete a depreacted example: NilLiteral and update comments to fit the style
Fix outdated examples of Raw object section in README

What code was refactored / updated to support this change?
README.md

What issues are related to this PR? Or why was this change introduced?

Checklist - While not every PR needs it, new features should consider this list:

[x] Does this have tests?
No need
[x] Does this have documentation?
No need
[x] Does this break the public API (Requires major version bump)?
No
[x] Is this a new feature (Requires minor version bump)?
No

Awesome thanks for update the doc!


rename "develop" to "main" branch on Octopus git repository

The octopus development took place on the 'develop' branch in the past,
and has been renamed to 'main' in Q3 2022.
This change should reflect that.

Martin Lueders agrees with sticking to the @develop label.

LGTM


NullReferenceException causes game crash to desktop if user logs into Amazon Polly while TTS is in progress

I got a a NullReferenceException crash to desktop twice in a row when I was listening to an NPC speak. I realized that it wasn't the Amazon Polly voices that I expected, so while listening, I opened up the TTT config, went to Synthesizer tab->Voices, and I clicked "Save and Login". That crashed with this message in the screenshot on both times.

I was able to reproduce this same crash in one other way as well by switching from Amazon Polly while logged to the System "Voice backend" while the TTS was in progress.
I tried it again several times by switching the following while a TTS was in progress:

Switching from standard "Voice backend" to Amazon Polly "Voice backend"
Switching from standard voice to a different standard voice.
Switching from Amazon Polly voice to a different Amazon Polly voice

None of these last 3 test cases could reproduce the crash.

Possibly fixed in v1.13.1

It's fixed for me. I can't get the game to crash again following my repro steps specified above. I'll close this issue for now. If it crops back up, I'll reopen it later.


No login

Hi
I tried to do a login but, in NetBank.js:45 the code thow an error, because the body in the respond is an empty string, so JSON.parse() thows an error

What should I do in this case?

Same error here, any updates on this @icodethings?

@lroura @sebszocinski Looks like netbank has retired the mobile ajax site I was using. Sorry!


This error shows that "....might take a minute", however it cost one hour and still donot success.

C:\Users\Administrator\Desktop\ML-KWS-for-MCU\Deployment>mbed new kws_simple_tes
t --mbedlib
[mbed] Creating new program "kws_simple_test" (git)
[mbed] Adding library "mbed" from "https://mbed.org/users/mbed_official/code/mbe
d/builds" at latest revision in the current branch
[mbed] Downloading library build "5aab5a7997ee" (might take a minute)

C:\Users\Administrator\Desktop\ML-KWS-for-MCU\Deployment>mbed new kws_simple_tes
t --mbedlib
[mbed] Creating new program "kws_simple_test" (git)
[mbed] Adding library "mbed" from "https://mbed.org/users/mbed_official/code/mbe
d/builds" at latest revision in the current branch
[mbed] Updating reference "mbed" -> "https://mbed.org/users/mbed_official/code/m
bed/builds/tip"
warning: LF will be replaced by CRLF in mbed.bld.
The file will have its original line endings in your working directory.
[mbed] Couldn't find build tools in your program. Downloading the mbed 2.0 SDK t
ools...

C:\Users\Administrator\Desktop\ML-KWS-for-MCU\Deployment>cd kws_simple_test
C:\Users\Administrator\Desktop\ML-KWS-for-MCU\Deployment\kws_simple_test>mbed de
ploy
[mbed] Updating library "mbed" to branch/tag "tip"
[mbed] Downloading library build "5aab5a7997ee" (might take a minute)

This seems to be an issue with your mbed-cli environment. Can you try importing and building mbed-cli example here and raise an issue in mbed-cli if you have the same problem?

From the above pictures, I think that this issue may be created by package update. So Can you tell me which file is responsible for this?

The second picture is my local file storage which is compiled for latest successful compilation.

@navsuda Can you help me answer this question?

It seems to be ok now!

But, why is there some other compile error as the figure shows?

Hi @oneway3124
I do not understand the cause of your error, but it seems to be an error with mbed-cli specific to your environment. Can you try debugging it in a virtual-environment? Also, raise an issue in mbed-cli, some other users who have had this issue may be able to help you.
I have added an example for FRDM-K64F, which can be built without mbed-cli here. Using that as a starting point, you may be able to build it for any other mbed supported board.

OK, this environment has been installed on the laptop of my colleague, but the error looks like me.
Which virtual environment do you mean? virtualbox or vmware, I has tried it.
At the same time, I also raise one issue in mbed-cli, there is no useful answer for this question.
So, I am trying your new example for FRDM-K64F now.

@navsuda thanks for your help, I can compile the FRDM-K64F on the ubuntu system in the virtualbox.
Then, I will complete the remaining steps of NUCLEO_F411RE board. If there is some question, I will raise some isssue on this site. Thanks very much in advance.

@oneway3124
In the quantization process, how to get the value of the quantization ranges -- act_max in quant_test.py .
how to set the specific value of the parameter act_max ( x 0 0 0 0 0 0 0 0 0 0 0 ) ?

@oneway3124
你好，量化的时候   --act_max参数的值怎么确定的，从哪得来的  是1-128吗？


Run on self-hosted to unblock

cc @mudler merging this so it cvan build all packages in my arm worker as they are way too many and get out of space :/
Will revert once its pushed to the final repos and it doesnt need that much space


key error with more recent pyluwen

(venv) root@orko:~/git/tt-burnin# tt-burnin
 Detected Chips: 1
 Detecting ARC: |
 Detecting DRAM: |
 [] ETH: |
┏━━━━━━━━━━━━┳━━━━━━━━━━━━┳━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━┓
┃ Pci Dev ID ┃ Board Type ┃ Device Series ┃ Board Number    ┃ Coordinates ┃
┡━━━━━━━━━━━━╇━━━━━━━━━━━━╇━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━┩
│ 0          │ grayskull  │ e75           │<PHONE_NUMBER>1[redacted] │ N/A         │
└────────────┴────────────┴───────────────┴─────────────────┴─────────────┘
 Resetting devices on host...
 Re-initializing boards after reset....
 Detected Chips: 1
 Detecting ARC: |
 Detecting DRAM: |
 [] ETH: |

 Starting TT-Burnin workload on all boards. WARNING: Opening SMI might cause unexpected behavior

 Stopping TT-Burnin workload on all boards.

 Resetting devices on host...
 Re-initializing boards after reset....
 Detected Chips: 1
 Detecting ARC: |
 Detecting DRAM: |
 [] ETH: |
Traceback (most recent call last):
  File "/root/venv/bin/tt-burnin", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/root/venv/lib64/python3.12/site-packages/tt_burnin/main.py", line 224, in main
    with Live(Group(generate_table(devices), text), refresh_per_second=10) as live:
                    ^^^^^^^^^^^^^^^^^^^^^^^
  File "/root/venv/lib64/python3.12/site-packages/tt_burnin/utils.py", line 207, in generate_table
    current = int(hex(telem["smbus_tx_tdc"]), 16) & 0xFFFF
                      ~~~~~^^^^^^^^^^^^^^^^
KeyError: 'smbus_tx_tdc'

I got the same issue,
So, I created other conda envs for tt-burnin, and installed tt-burnin via pip3.
Then I removed smbux_tx_ at all key strings in /root/venv/lib64/python3.12/site-packages/tt_burnin/utils.pyfile. It works now. Please match the key intelem` dict.

Hey Guys, thanks for working around this issue. As of burning 0.2.0 (merged last week) I updated the dictionary keys to drop the smbus_tx prefix.


UserManager#savePlayerData doesn't actually register a player that has not joined the server.

Setting the primary group of a player that has not joined returns null after using this method.

Sample code? What storage type are you using?
The code for UserManager#savePlayerData is just a straightforward insert query. It definitely works for players that haven't joined the server.

Setting the primary group of a player that has not joined returns null after using this method

That may indeed be true, but it's not related to this issue. :) (perhaps more related to #1182)


Re-land Clang types in SIL

Fixed merge conflicts etc.
TODO:

[ ] Update some test cases in the last commit introduced in the serialization PR: https://github.com/apple/swift/pull/29670

@swift-ci please test

@swift-ci please test source compatibility

Follow up to PR https://github.com/apple/swift/pull/29239

@swift-ci please test Windows platform

Can one of the admins verify this patch?

Closing this PR, since it was submitted from a now-deleted clone.
I've fixed some of the conflicts locally, will create a new PR after fixing the rest soon.


Changed UI Design and Fixed some Basic Mistakes

Changes

Changed the Color of Text and Added Shadow for Better looks on the Homepage
Changed Navbar Design on the Homepage
Changed Homepage Gradient color so it will not mismatch the text color
Changed Navbar Design in Signup and Login Page
Fixed some basic errors in the login form

Thanks for the contributions, but there are merge conflicts. Kindly fix them so that I can merge this PR!


Improve Test Summary UX with SVG's

The test-summary/action project uses SVG's to improve the test summary UI/UX, It makes it much easier to read at a glance.

SVG gives you more power to render your results, compared to GitHub Markdown. But that action generates the SVG images on http://svg.test-summary.com. So it uses a HTTP endpoint to render the image.
This action is restricted to GitHub and uses no external services.
Not sure how this could be achieved without setting up something outside GitHub.


Safer manifest handling and some lexicon improvements

Pull Request Test Coverage Report for Build 231

54 of 136 (39.71%)  changed or added relevant lines in 17 files are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.4%) to 60.976%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

client_converter_callback.py
0
1
0.0%

tests/client_tests/test_obs_preprocessor.py
2
3
66.67%

tests/client_tests/test_tn_preprocessor.py
2
3
66.67%

tests/client_tests/test_tw_preprocessor.py
0
1
0.0%

tests/door43_tools_tests/test_project_deployer.py
0
1
0.0%

preprocessors/preprocessors.py
0
5
0.0%

resource_container/ResourceContainer.py
14
27
51.85%

webhook.py
1
19
5.26%

callback.py
3
44
6.82%

Files with Coverage Reduction
New Missed Lines
%

callback.py
3
12.86%

webhook.py
4
11.75%

Totals

Change from base Build 225:
-0.4%

Covered Lines:
2861

Relevant Lines:
4692

💛  - Coveralls

Pull Request Test Coverage Report for Build 231

54 of 136 (39.71%)  changed or added relevant lines in 17 files are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.4%) to 60.976%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

client_converter_callback.py
0
1
0.0%

tests/client_tests/test_obs_preprocessor.py
2
3
66.67%

tests/client_tests/test_tn_preprocessor.py
2
3
66.67%

tests/client_tests/test_tw_preprocessor.py
0
1
0.0%

tests/door43_tools_tests/test_project_deployer.py
0
1
0.0%

preprocessors/preprocessors.py
0
5
0.0%

resource_container/ResourceContainer.py
14
27
51.85%

webhook.py
1
19
5.26%

callback.py
3
44
6.82%

Files with Coverage Reduction
New Missed Lines
%

callback.py
3
12.86%

webhook.py
4
11.75%

Totals

Change from base Build 225:
-0.4%

Covered Lines:
2861

Relevant Lines:
4692

💛  - Coveralls


only run domains and nginx config if we have a port and ip

only run domains:setup and nginx:build-conifg if IP.web.1 and PORT.web.1 files exist
added test app that just contains a worker process

fixes #1355

Does this work for apps that have multiple web procs?

Yes. We test scaling and those tests pass.


Check go generators

Everything seems to be working as expected. There was initially a concern with the go modules but they seem fine.


Test the whole toolchain on simple c / c++ files

Introduce the compiler-rt and libc++ features/requires to achieve that goal.

  Unsupported Tests  : 4
libclang-common-10-dev should probably be installed
and -D set

Also, I think the cmake configure should fail by default if not found
not silently fail


Tracker: added anilist

Added fetching of titles from an Anilist.co user account. The titles are fetched from inside the 'Reading' list of the the Manga Library of the user.
For the titles to be scraped, it is required that the account in question is public and is accessible without any special permission from the user.
The titles are run through the removeExtraChars function and returned in a form of array of string

Closed branch due to significant rebase of the default branch and the changes in this branch not being relevant and up to date with other components


Update line tests to match new metadata system

Rewrites all the tests for the line chart type for the new metadata structure in #2346
Note: Test must be re-enabled in gulpfile.js

Coverage remained the same at 76.996% when pulling 4b96acc68550e362f5a896c3b13121f97d34d1bf on zachpanz88:shared-data into 73bc52f196062f1998b7b5461f76eeb9a0fb4432 on chartjs:master.

wooooo! Tests are passing :smile:


Lack of anti-aliasing when encapsulating any control in a Border with a set CornerRadius

Describe the bug
Avalonia's renderer does not seem to de-alias rounded corners of a Border correctly when an image is in the corner. Could apply to other controls as well.
To Reproduce
Steps to reproduce the behavior:

Create a new Avalonia project.
Add an image to your Assets folder, preferably one that is not transparent around the corners so the effect is visible, and one that is is a contrasting color to the background.
In the main window, put a Border as the main child of the window, make sure it is the same size as your chosen image, set the CornerRadius of the border to something like 20 or 30 depending on your image size, place an Image tag inside your border with your chosen image URI, then set "ClipToBounds" to "true" on the border. See screenshots for example AXAML.

Expected behavior
Avalonia should apply proper anti aliasing to the corners, like the second example screenshot below (taken from chrome).
Screenshots

Desktop:

OS: Windows 11, Windows 10, Ubuntu 20.04 via WSL2, and Manjaro Linux with NVIDIA proprietary drivers.
Version: 0.10.11 & 0.10.10

It's not just with images, but anything that gets clipped via border with corner radius has the same behavior
Looks like PushClip in Skia doesn't set the antialias argument on SKCanvas.ClipRoundRect()
https://github.com/AvaloniaUI/Avalonia/blob/f29ed1a12781dc3508bbdad3ead7191ebd3db40a/src/Skia/Avalonia.Skia/DrawingContextImpl.cs#L482-L486

So would this be as simple as a pull request that includes this argument in this call, then?

It does look that simple: https://docs.microsoft.com/en-us/dotnet/api/skiasharp.skcanvas.cliproundrect?view=skiasharp-2.80.2

Cool. I'll fork, test, and make a pull request.


Cluster-Autoscaler: add node drain logic

Currently a node is removed without any notice and pods  that live on the node killed without graceful termination. This PR adds drain logic to scale down.
cc: @fgrzadkowski @piosz @jszczepkowski

This change is

Why do you re-implement drain instead of reusing existing mechanisms?

FWIW I recently implemented my own take on this, although I haven't had much chance to test it on a live workload yet: https://github.com/Exient/contrib/commit/860f0ca0ad6f5d50e2e055dd2e28a501224eb1ba
As someone who's fairly new to Kubernetes it's encouraging to me that your approach is basically the same as mine (mark the node unschedulable and then terminate the pods), but I am a bit worried about what will happen if you have a replica set containing only a couple of pods and those pods are all located on the node that's being terminated. If the autoscaler terminates all the pods at once then there'll be a brief period where the replica set/service will have no pods available to respond to requests.
In my solution it's careful to only terminate one pod at a time from any given replica set, so as long as the desired replica count is >1 it shouldn't result in the replica dropping to zero active instances. In an ideal world we'd have a solution that would also work with replica sets which have a desired count of 1, but for us that wasn't a high priority (we're happy to run a minimum of two instances of any mission-critical pods)

Couple reasons:

The current drain logic sits inside kubectl and would require significant refactoring to get out of there.
Even after the refactoring the code is likely to be changed. This happened to the previous attempt to reuse drain node evaluation. It lasted a month and then someone changed the code in kubectl. CA code is in a different repo.
We need to have an automatic rollback/retry logic should the drain failed in the middle.
AFAIR there was a plan to move drain to the server side. So this code will eventually be removed as well.

@Phlamethrower for your use case we have pod disruption budget. Eventually CA will respect PDB as well.

@mwielgus I noticed you mentioned fallbacks for failing to delete pods or nodes, have we considered what would happen if cluster-autoscaler pod dies in the middle of draining a node? It would have marked the node as unschedulable and when it starts up again it would not start any scaling operations because the expected and actual node counts will be different (since CA does not count unschedulable nodes).

Should we also consider removing the skip-nodes-with-system-pods flag entirely? Seems like an awful idea to set --skip-nodes-with-system-pods=false and have CA drain nodes before terminating. If it elects itself for termination it would delete itself after marking a node unschedulable

@andrewsykim - for that reason we add an annotation to the node. On CA startup we will scan the nodes and set schedulable to true on nodes that have it set to false AND have the annotation.
As of skipping nodes with system pods - for now we will keep it. System pods should eventually get pod disruption budget objects and then we will use pdb only.
As mentioned in the PR title this is WIP (work in progress). I started working on this and noticed that i should probably first update the kubernetes version and migrate client to clientset (what requires some grungy work). So I sent this for initial feedback.

Ahh, thanks for explaining, makes much more sense :) looking forward to seeing this merged!

As of skipping nodes with system pods - for now we will keep it. System pods should eventually get pod disruption budget objects and then we will use pdb only.

Can you explain what you mean by this? You will set PodDisruptionBudget on system pods so they can never be evicted (thus their nodes can never be removed)? I'm not sure we want to allow this because then it would block node upgrade.

@davidopp - Hmm I see your point but this is also true for user application. They can declare PDB of minAvailable=1 for replicaset of size 1 and effectively prevent all node upgrades.
In the long run I would rather not have any custom logic in CA that somehow favors system pods. The situation that some components can evict system pods and some not (node upgrade vs ca) should be handled by (expanded) pdbs.

Tests, cleanups and clientset added.
@fgrzadkowski  PTAL

As discussed offline please add a comment to migrate to taints to avoid potential race with other components using unschedulable bit.

Switched to Taints. PTAL

LGTM, just a nit


Update description

The description for $neat-grid currently references Bourbon’s settings map, instead of Neat’s: https://github.com/thoughtbot/neat/blob/neat-2.0.0/core/neat/settings/_settings.scss#L19

Fixed in https://github.com/thoughtbot/neat/pull/480


feat(lock): add lock support

Requested feature : #313

fix https://github.com/piitaya/lovelace-mushroom/issues/313


Laravel 11 support

I try install yajra/laravel-datatables:^11.0 on laravel 11.3.1 not working.
Code snippet of problem
Problem 1
    - pestphp/pest[v2.33.6, ..., v2.34.0] require phpunit/phpunit ^10.5.10 -> found phpunit/phpunit[10.5.10, ..., 10.5.18] but it conflicts with your root composer.json require (^11.0.1).
    - pestphp/pest v2.34.1 requires phpunit/phpunit ^10.5.11 -> found phpunit/phpunit[10.5.11, ..., 10.5.18] but it conflicts with your root composer.json require (^11.0.1).
    - pestphp/pest v2.34.2 requires phpunit/phpunit ^10.5.12 -> found phpunit/phpunit[10.5.12, ..., 10.5.18] but it conflicts with your root composer.json require (^11.0.1).
    - pestphp/pest[v2.34.3, ..., v2.34.4] require phpunit/phpunit ^10.5.13 -> found phpunit/phpunit[10.5.13, ..., 10.5.18] but it conflicts with your root composer.json require (^11.0.1).
    - pestphp/pest v2.34.5 requires phpunit/phpunit ^10.5.15 -> found phpunit/phpunit[10.5.15, 10.5.16, 10.5.17, 10.5.18] but it conflicts with your root composer.json require (^11.0.1).
    - pestphp/pest v2.34.6 requires phpunit/phpunit ^10.5.16 -> found phpunit/phpunit[10.5.16, 10.5.17, 10.5.18] but it conflicts with your root composer.json require (^11.0.1).
    - pestphp/pest v2.34.7 requires phpunit/phpunit ^10.5.17 -> found phpunit/phpunit[10.5.17, 10.5.18] but it conflicts with your root composer.json require (^11.0.1).
    - yajra/laravel-datatables v11.0.0 requires yajra/laravel-datatables-export ^11 -> satisfiable by yajra/laravel-datatables-export[v11.0.0].
    - yajra/laravel-datatables-export v11.0.0 requires pestphp/pest ^2.34 -> satisfiable by pestphp/pest[v2.34.0, ..., v2.34.7].
    - Root composer.json requires yajra/laravel-datatables ^11.0 -> satisfiable by yajra/laravel-datatables[v11.0.0].

I think the issue in your pestphp installation and not on the package?

pestphp/pest[v2.33.6, ..., v2.34.0] require phpunit/phpunit ^10.5.10 -> found phpunit/phpunit[10.5.10, ..., 10.5.18] but it conflicts with your root composer.json require (^11.0.1).

@lucasmazevedo
below steps works for me after creating a new laravel 11 project.
Remove phpunit and pestphp
composer remove phpunit/phpunit
composer remove pestphp/pestphp
then run below command
composer require yajra/laravel-datatables -W
this will install all required dependencies.

@lucasmazevedo
below steps works for me after creating a new laravel 11 project.
Remove phpunit and pestphp
composer remove phpunit/phpunit composer remove pestphp/pestphp
then run below command composer require yajra/laravel-datatables -W
this will install all required dependencies.

Yeap that works, thx alot

Seems that the [laravel-datatables-export](https://github.com/yajra/laravel-datatables-export) this package requires pestphp/pest which conflicts  with the phpunit/phpunit which requires by the laravel project

It seems like pestphp was accidentally added in require and it should be in require-dev. Released https://github.com/yajra/laravel-datatables-export/releases/tag/v11.1.1, Thanks!


Add support for scalar conditions in device_assert

This sometimes happens in TorchInductor. See https://github.com/pytorch/pytorch/pull/100880.
More generally, it's useful to be able to write tl.device_assert(False, msg).

cc @Jokeren

Can you use static_assert instead? Based on the test, seems to me it's not a scalar assert but instead a compilation time assert because the result 0 == 0 is known at compilation time.
tl.device_assert(0 == 0, "x != 0")

The test was written that way to avoid having to modify the existing test too much. A better test would have had an if-else and have the something like a device_assert(False, msg) in the else branch.
In the case of torch.compile, this appears in two situations:

https://github.com/pytorch/pytorch/pull/100357#discussion_r1181719557
Our analysis on the bounds of the generated variables in a kernel is sometimes not good enough as to be able to prove that some things lie in certain range, so, in those cases, we have to fallback to generate the device_assert for these, even if you could know a priori that that variable has certain value or a value in certain range.


Added decoder docstrings

Codecov Report

Merging #129 (db32810) into main (81bc28c) will decrease coverage by 0.40%.
The diff coverage is 100.00%.

:exclamation: Current head db32810 differs from pull request most recent head 1883545. Consider uploading reports for the commit 1883545 to get more accurate results

@@            Coverage Diff             @@
##             main     #129      +/-   ##
==========================================
- Coverage   80.72%   80.31%   -0.41%
==========================================
  Files          65       65
  Lines        2148     2149       +1
==========================================
- Hits         1734     1726       -8
- Misses        414      423       +9

Impacted Files
Coverage Δ

ezflow/decoder/conv_decoder.py
97.77% <ø> (ø)

ezflow/decoder/iterative/recurrent_lookup.py
77.22% <ø> (ø)

ezflow/decoder/noniterative/operators.py
100.00% <ø> (ø)

ezflow/decoder/separable_conv.py
89.62% <ø> (ø)

ezflow/decoder/noniterative/soft_regression.py
49.41% <100.00%> (+0.60%)
:arrow_up:

ezflow/functional/data_augmentation/operations.py
72.72% <0.00%> (-16.37%)
:arrow_down:


Running into type issue with decimals in maps

Description
In indexOnWrite and backfill, I get the error: "error: 'Field selectedItem.price must be an int64.'" whenever selectedItem.price is not an integer value.
Steps to reproduce
To reproduce, I can add a document into Firestore or run a backfill.
Expected Behavior
I am supposed to successfully add this document to my Typesense collection.
Actual Behavior
I fail to upload the document to my Typesense collection.
Metadata
Typesense Version:
Was on 0.24.0, but switched to 27.1 (running docker in my own cloud server), and it still seems to be an issue.
OS:

whenever selectedItem.price is not an integer value

I assume you meant to say that sometime this field is null? If so, you want to set it as optional: true in the collection schema.

No, it is a decimal sometimes. I thought this may be related so I upgraded to v27, but I'm still getting the issue (https://github.com/typesense/typesense/issues/1327)

If it's a decimal sometimes, then you want to define the field type for selectedItem.price as `float in the schema.

Ok thanks. I don't remember setting the schema for it since it's embedded in a map and pulling that map from Firestore. Is there a way for me to get my current schema with an API call?

GET /collections will get you the schema.

You might have to explicitly set the datatype for each sub-field inside the object to prevent this issue, especially if you have mixed datatypes in Firestore.
Or alternatively, if you're able to set consistent datatypes for each field across documents in Firestore, that would be the best solution.

Ok that makes sense! Thanks. For now, I've switched it to round to an int for the field and that solves it for me.


Intellisense does not work

Hey Guys,
I have tried v0.6 and v0.10 both, but I dont get the intellisense working for the library. I am using VS Code v1.47.3.
Any idea, on how can I get it working.
The client property on Apify SDK also points to apify-client library, and hence even that is not working.

Works fine for me in WebStorm since the v1.0.0 release. The problem in the SDK is of a different nature and we'll fix it there.

Works fine for me in WebStorm since the v1.0.0 release. The problem in the SDK is of a different nature and we'll fix it there.


Tabs considerations

The following is an omnibus issue for the Tabs component. It is a summary of the main considerations surfaced in a review of the carbondesignsytem.com guidance. A more thorough set of comments can be viewed in the commented documents, tabs usage.pdf and tabs style.pdf.
Note that this website issue is also complemented by carbon issues to do with Tabs https://github.com/carbon-design-system/carbon/issues/13029 and https://github.com/carbon-design-system/carbon/issues/14373
The same two main considerations for Tabs are the same already flagged in an issue for Content Switcher

Lack of styling guidance on tabpanel
No mention of manual versus automatic tab selection

Additionally, there are a few style considerations:

Especially for the line tabs variant, the primary indicator is a light grey line. IMO, this should be at least 3:1 contrast with the page background.
there should be no focus state for the scroll chevron on the Style page. They never receive keyboard focus.
I suggest exploring a focus indicator around the whole tablist. See my comments in the Usage.pdf for ideas

Lack of styling guidance on tabpanel
On many of the examples in the usage page, there is a clear visual tab panel designation using the white area on a grey background. It ties in the selected tab item with its panel. This is a basic feature of tab panels (especially the 'contained' version) yet there is no documentation about it on either the style or usage pages.
No description of manual versus automatic tab selection
I've added information on this into the draft Accessibility tab info. I think it is reasonable for this to be discussed on the Usage page as well.

Usage updates:
-Added content switcher information on when not to use a tab
-Added scroll button to anatomy section
-Added clarity around scrolling tabs and overflow content (more to come with vertical tab updates)
-Updated information about navigating away tab and returning
-Added information about automatic verses manual tabs (gif is coming! trying to troubleshoot it and it was a blocker)
-Updated icon placement to include dismissible tab variant parameters
Style updates:
-Took out scroll focus states and opened
-Discussed darkening unselected line in default tabs to meet 3:1 contrast but the contrast was too similar to selected states and decided to keep as is
Continued exploration:
-Investigate line extension to end of the tab panel for default tabs—this seems to be a big change for some product teams but many other systems include this in similar types of tabs
-Investigate focus state going around whole tablist #15962 (explorations started but not complete)


Concurrent network requests

Context
If the wallet was able to make concurrent network requests to a daemon, then it could make requests for chunks of blocks in parallel and immediately start scanning whichever chunks get to the client first. This is how @UkoeHB's async scanner was designed. When implemented, I observed a download speed-up of ~40% on my machine pointing to a remote daemon, which translated to a ~40% speed-up in scanning compared to wallet2, since download speed is my bottleneck (source).
The epee http client wallet2 uses does not support concurrent network requests over a single connection. In order for a client to make concurrent network requests to a daemon with epee's http client, it must initiate multiple connections to the daemon, which has perf overhead. This is how I implemented the mock async scanner mentioned above (source).
libcurl's http client offers the ability to make concurrent network requests over a single connection, however, this is only supported when the server supports http/2 multiplexing or http/3. Both of which seem to require significant changes to the Monero server connection code to support.
Immediate path forward
I think the optimal immediate path forward is to move forward with a hardened client-side connection pool that wraps N http connections to a daemon and is particularly useful for scanning. I already implemented a mock connection pool in my scanner impl linked above, and I think that can easily be improved and hardened.
This shouldn't require any changes to the daemon and should be easily doable in days since it's pretty much already implemented.
Future considerations

Upgrade the monerod rpc server to support http/3 in order to support concurrent request/response handling (plus other perf benefits including faster connection time), and re-consider using libcurl in the wallet.

Of note, even if wallet3 were to use libcurl as an http client and not use the epee http client, then the connection pool would still be useful and easily reusable assuming no changes to the server. The server needs a larger overhaul to support multiplexing in order to negate the usefulness of the client-side connection pool when scanning via a single daemon.

Implement a streaming endpoint (or websocket) to support downloading blocks over a stream.

Both of these would be nice to have but I think networking dev resources are probably better allocated elsewhere at the moment (e.g. p2p encryption), since we can realize a major scanning speed-up without major server-side code that would require a fair amount of dev effort and review.

Agree.

client-side connection pool that wraps N http connections to a daemon

About how many connections do we speak here? I wonder whether we run a danger that public nodes will get pushed to their performance limits much faster if future wallets overwhelm them with connections.

I'll check resource impact with connections soon. I planned to cap the connection pool at 20. It can also easily be implemented to only open >1 connections if the client has many blocks to scan and disconnect any lingering connections when scanning is done.

One clarifying note: when using libcurl's multi interface, if the server does not support multiplexing and the client tries to make concurrent requests, then libcurl initiates new connections to the server and handles requests concurrently under the hood. Thus, switching to libcurl today and using the multi interface for concurrent requests would in fact negate the usefulness of a new client-side connection pool since it abstracts the "pooling" away. However, the client-side connection pool is simple enough code that I'm still of the view it makes sense to move forward with it.


Allow for multiple server connections from a single client.

The PyAdditive client needs to be able to connect to multiple servers and perform crude load balancing in order to improve simulation throughput.
The load balancing will assign one simulation to one server at a time. The number of threads per simulation will be fixed at 4.
Background:
Each simulation requires an additive suite license (am_module). An HPC pack allows multiple threads per solve. Each additive license includes an HPC pack for 4 threads. Additional HPC packs can be purchased.

Closed by #143


Doubt - Getting PEB Address

Hello,
I hope you are doing well. I would like to request your help in obtaining the Process Environment Block (PEB) address of a process in Windows using your module.
The PEB is a crucial data structure that contains critical information about a running process. I am interested in accessing this information to analyze and better understand the behavior of certain processes in my application.
I would appreciate if you could provide guidance on how I can use your library to access the PEB of a process. Any code examples, documentation, or related resources would be greatly appreciated.
Thank you very much for your time and consideration.
Thank you in advance,
Alejandro

Apologies for the late reply, I am in the process of moving houses.
According to the core docs here, what you want is ProcessInfo.address which can be retrieved like so:
from memflow import *
import logging

# Setup logging
FORMAT = "%(levelname)s %(name)s %(asctime)-15s %(filename)s:%(lineno)d %(message)s"
logging.basicConfig(format=FORMAT)
logging.getLogger().setLevel(logging.INFO)

class PEB(Structure):
    _fields_ = [
        # TODO: Finish
        ("_pad_0x0", c_byte * 0x10),
        ("ImageBaseAddress", c_uint64)
    ]

    def __str__(self):
        return f"ImageBaseAddress = {hex(self.ImageBaseAddress)}"

class EPROCESS(Structure):
    _fields_ = []
    _offsets_ = [
        (0x550, "PEB", POINTER64(PEB))
    ]

inventory = Inventory()
conn = inventory.create_connector("kvm")
os = inventory.create_os("win32", conn)
process = os.process_from_name("lsass.exe")
eprocess = os.read(process.info().address, EPROCESS)
print(eprocess)
peb = process.read_ptr(eprocess.PEB)
print(peb)

PEB=PEB @ 0x84ac9fa000
ImageBaseAddress = 0x7ff654cb0000

It should be noted that the PEB offset 0x550 in EPROCESS can change across windows versions, for compatibility I would first check the field eproc_peb within these configuration files.


Checkuser template not recognized when blocking/tagging accounts

https://en.wikipedia.org/w/index.php?title=Wikipedia:Sockpuppet_investigations/ASIANPOPCHANNEL&oldid=1098776017 had a {{checkuser}} template instead of a {{socklist}}.  Running spi-helper to block/tag didn't recognize the account.
Changing {{checkuser}} to {{socklist}} got things working again.

That's not a proper use of {{checkuser}}, so there's no fix to apply here; I do not think it is worth our time to work around people using templates incorrectly.


Prepare transcript index for mapping

In order to efficiently map reads to reference sequences, mapping programs require an index of the reference sequences (here: transcripts of genes coding for highly expressed ribosomal proteins, see #25 for more details).
Write a function that prepares an index of the subset of transcripts prepared in #25 with the STAR program. Accept the path to the transcripts file as an input and return the path to the built index. Use a subprocess to call the index generation. Write the resulting index to a directory path in the temporary directory created in #33. Make sure you have installed STAR v2.7.6a first (via Conda) and add it to the environment.yml file (if it is not already present from #27). Insert a call to your new function in the main function infer() in the htsinfer/infer_read_orientation.py module. Assign the result (i.e., the index directory path) to a variable that can be used subsequently to map the reads (#27).

Hi, I'd like to take this issue.

@S1moFara thanks, assigning you

Currently being implemented in #73. Closing for better overview.


Block login screen

I added everything it my .env.local and started the script but I end up with a black screen with the url and can't login
http://localhost:3000/api/auth/signin?callbackUrl=%2F

Make sure the env variables are not misspelled. This usually happens when the app cannot find any configurations for auth providers.
Also enable debug mode with NEXT_PUBLIC_DEBUG_MODE=true in the env file to print out all the variables the app is able to see.


fix #656

Description

fixes #656
Models used to test - Optional

Checklist

[x] You followed contributing section
x ] You kept your Pull Request small and focused on a single feature or bug fix.
[ ] You added unit testing when necessary
[ ] You added functional testing when necessary

WIP, added WITH NO DATA for SQL models, unsure exactly how to make this work for Spark/Python models.

@antonysouthworth-halter the functional tests are falling, please have a look 🙏🏻


Example code given in @opentelemetry/instrumentation-winston package not working

What version of OpenTelemetry are you using?
these are my deps
  "@opentelemetry/instrumentation": "^0.43.0",
   "@opentelemetry/instrumentation-winston": "^0.32.1",
   "@opentelemetry/sdk-trace-node": "^1.17.0",
   "winston": "^3.10.0"

What version of Node are you using?
it is node 20
What did you do?

I have tried the exact example given in readme file.
const { NodeTracerProvider } = require('@opentelemetry/sdk-trace-node');
const { WinstonInstrumentation } = require('@opentelemetry/instrumentation-winston');
const { registerInstrumentations } = require('@opentelemetry/instrumentation');

const provider = new NodeTracerProvider();
provider.register();

registerInstrumentations({
  instrumentations: [
    new WinstonInstrumentation({
      // Optional hook to insert additional context to log metadata.
      // Called after trace context is injected to metadata.
      logHook: (span, record) => {
        record['resource.service.name'] = provider.resource.attributes['service.name'];
      },
    }),
    // other instrumentations
  ],
});

const winston = require('winston');
const logger = winston.createLogger({
  transports: [new winston.transports.Console()],
})
logger.info('foobar');

What did you expect to see?

{"message":"foobar","trace_id":"e21c7a95fff34e04f77c7bd518779621","span_id":"b7589a981fde09f4","trace_flags":"01", ...}

What did you see instead?

{"level":"info","message":"foobar"}

What is the right way to use this plugin ?
Am I missing out something here ?

Are you using any other instrumentations? The trace context is only added if a span is active.

If I add some lines to your script to show logging when there is an active span:
% diff -u orig.js foo.js
--- orig.js	2023-09-20 11:17:25
+++ foo.js	2023-09-20 11:17:42
@@ -23,3 +23,9 @@
   transports: [new winston.transports.Console()],
 })
 logger.info('foobar');
+
+const tracer = provider.getTracer('default');
+tracer.startActiveSpan('my-span', span => {
+    logger.info('hi in my-span');
+    span.end();
+})

then we can see the expected added trace-context fields (and the result of running the logHook():
% node foo.js
{"level":"info","message":"foobar"}
{"level":"info","message":"hi in my-span","resource.service.name":"unknown_service:node","span_id":"ca6a35e2b1526177","trace_flags":"01","trace_id":"49edd66508214fe1e89bb3526784410e"}

So I believe the instrumentation is working as expected. However, perhaps the README could be improved to show that example.

A tracer instance (const tracer = provider.getTracer('default')) will have the same trace id all over the app or it will produce new one for each HTTP request (if I use logger inside express route callbacks) ?

@shivarajnaidu

A tracer instance (const tracer = provider.getTracer('default')) will have the same trace id all over the app or it will produce new one for each HTTP request (if I use logger inside express route callbacks) ?

Typically that creation of an active span would be done in the instrumentation of whatever HTTP server library you are using. So if you are using:
const http = require('http');

const server = http.createServer(function onRequest(req, res) {
  // ...
});

then the instrumentation of the http module will create a span (with a new trace-id, depending on an incoming traceparent HTTP header) for each incoming request, and that onRequest function will be in that span's async context. So a log.info(...) inside that onRequest function will use the span context specific to that incoming HTTP request.

Thanks for the reply @trentm
Its really useful .
Thank you so much

Hi, If I'm auto instrumenting the code using opentelemetry, wouldn't the span context be active automatically for my API calls? Because right now I'm auto instrumenting the code and logger.info still doesn't print the trace_id and span_id. It only does if I start an active span. @trentm @shivarajnaidu

@coderguy1998 Hard to know what may be happening without specific code. Perhaps you could post a small reproduction case of what you are seeing to a new issue or to the #otel-js CNCF Slack channel (https://cloud-native.slack.com/archives/C01NL1GRPQR) so your question doesn't get lost.

Can somebody please look into it. https://github.com/open-telemetry/opentelemetry-js-contrib/issues/2119


Placeholder not showing up

Version 2.0-beta.15
Hi. I have this config :
<multiselect :options="genders"
                     :searchable="false"
                     :close-on-select="true"
                     :multiple="false"
                     :allow-empty="false"
                     :value="value"
                     @input="updateValue"
                     placeholder="Select option"
                     deselect-label="Can't unselect"
        ></multiselect>

the placeholder that shows up is [] not really a placeholder as it is an empty array. From what I've seen it has something to do with this line :
currentOptionLabel () {
      return this.multiple
        ? this.searchable ? '' : this.placeholder
        : this.internalValue[0].length
          ? this.getOptionLabel(this.internalValue[0])
          : this.searchable ? '' : this.placeholder
    }

and this line
internalValue: this.value || this.value === 0
        ? deepClone(Array.isArray(this.value) ? this.value : [this.value])
        : []

Changing searchable to true.. shows up the placeholder

Sounds like a bug! Thanks for finding it. :)

glad i found it so it can be fixed. :) i.am using this component in my project so if i find anything else i.ll let you know.

Seems to be working fine.
https://jsfiddle.net/qfc0d5co/


YARN-1864

https://issues.apache.org/jira/browse/YARN-1864

YARN-1864 Patch dynamic hierarchical user queues
YARN-1864 Test user queues
YARN-1864 Add documentation

Can one of the admins verify this patch?


Cannot Start Profiler

We have written the needed data into your clipboard because it was too large to send. Please paste.
Select View=> Command Pallete => Profiler: Launch Profiler and getting the following error
"Unable to open 'Profiler: tcp:dsims.database.w': Cannot set property 'actionBarContext' of undefined."

@edippity I think this issue is supposed to be fixed in the 1.18.0 release.  If you're currently on that release it could be bad state left over from bug in the previous release.
Please confirm you're using 1.18.0.  To clear out the bad document window state, please try either opening any folder or closing the currently open folder.  This will reset the HotExit autorecovery cache and should remove the problematic documents trigger this message.  Please let me know if that doesn't resolve the issue.

It seems to be working now.  Thanks
From: Karl Burtram<EMAIL_ADDRESS>Sent: Tuesday, May 26, 2020 11:56 AM
To: microsoft/azuredatastudio
Cc: edippity; Mention
Subject: Re: [microsoft/azuredatastudio] Cannot Start Profiler (#10544)
@edippity https://github.com/edippity  I think this issue is supposed to be fixed in the 1.18.0 release. If you're currently on that release it could be bad state left over from bug in the previous release.
Please confirm you're using 1.18.0. To clear out the bad document window state, please try either opening any folder or closing the currently open folder. This will reset the HotExit autorecovery cache and should remove the problematic documents trigger this message. Please let me know if that doesn't resolve the issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/microsoft/azuredatastudio/issues/10544#issuecomment-634114950 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ANWLZOWMPUGZ57C7NICG723RTPRA5ANCNFSM4NIPCC3A .Image removed by sender.

Thanks for confirming - I'll close this out now and if you see it again let us know!


Tippy css for light mode only with Furo

Hello,
first of all, thanks for the package!
I'm having an issue where I want to only change the tippy's css when Furo is in Light mode. The tippys are fine in Dark mode but not readable in Light mode:
Link issue:

Inline code issue:

Any advice on how to procede?
Thanks!

Well, in fact, it is enough in the case of PyData-Sphinx-Theme to follow the “Important” admonition here.

The following CSS improves basic compatibility with furo and can be applied as documented:
.tippy-box {
    background-color: var(--color-background-hover);
    color: var(--color-foreground-primary);
    border: 1px solid var(--color-foreground-border);
}

More furo color variables: https://github.com/pradyunsg/furo/blob/main/src/furo/assets/styles/variables/_colors.scss


For issue #28, pass file operation to another process if file contents are invalid.

UUID#open_lock unlock flock before buffered output actually write to file.
this behavior cause a race condition. fix this bug without performace
penalty(I avoid using IO#fsync)

you can reproduce this bug by following script.
$ seq 10000 | xargs -P 100 -n 100 ruby -ruuid -e \
  "sleep 1 ; uuid = UUID.new ; 1000.times { uuid.next_sequence ; p uuid.instance_variable_get(:@sequence) }" \
  > ~/sequence_number.txt


fix: fixes #144, add ding-dong handshaking

ding-dong handshaking
I'm submitting a...
[x] Bug Fix
[ ] Feature
[ ] Other (Refactoring, Added tests, Documentation, ...)

Checklist

[x] Commit Messages follow the Conventional Commits pattern

A feature commit message is prefixed "feat:"
A bugfix commit message is prefixed "fix:"

[ ] Tests for the changes have been added

Description
See #144
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Thank you very much for adding this missing logic to our JS example, appreciate it!


Refactor webpack build

refactor: specify included files in TS compilation with webpack
chore: update to latest ns webpack plugin
refactor: include ns-ng/animations and googlemaps to vendor chunk

Before:

After:

run ci


Long MUG names overlap view-head on small screens

Looks like the padding-top in .stripe-heroContent is getting clobbered by the padding on .bounds.
I'd argue that _type.scss isn't the right place for this style anyway, but am open to being told otherwise

Won't fix


Python implementation and usage

Thanks for the implementation. But how do we use this library in Python?

This library would probably have to be packaged with https://github.com/PyO3/pyo3
It would be great if someone wants to help.

I have a few questions:

Can Elo-MMR (theoretically at least) provide probability that a 2-player will be draw (like TrueSkill provides)? Basically, I'm looking for a metric for quality / fairness of a 2-player match.
I see that there exists a Python implementation here. May I know whether you still would like one more Python implementation for this project of yours?

Ties are tricky to model consistently across different games, and I think I recall them being somewhat messy in TrueSkill as well. You could try to do similar math for Elo-MMR; I'm not sure how it would turn out. We obtained simpler formulas by taking the limit as the likelihood of ties goes to zero, but in doing so we gave up on predicting ties.
Making Python wrappers for the Rust implementation would be different from making a pure Python implementation, so your second point depends on which you prefer.

I open sourced the Python implementation. It is particularly slow, but I plan to release the CuPy, Numba, and standard Numpy implementation soon(tm).  You can take a look and make adjustments for yourself.
https://github.com/BeniaminC/Open-ELO/blob/main/src/openelo/systems/elommr.py


[Feature request] Add requirements.txt

It would be great to have a requirements file and/or a dockerfile for people who want to follow along a live tutorial session but don't want to use colab.
I don't think the HF docker images include gradio and streamlit though.

cc @lewtun, WDYT? This seems quite simple, I think transformers, gradio, torch, and sentencepiece should be enough for it

Sounds like a great idea to me! Would you like to open a PR for this @cakiki ? We could start with a simple requirements.txt file for now and just add an installation step in the README

@lewtun I'll open a PR!


How to play mpeg-dash + Clearkey?

Is it possible to use this player to play mpeg-dash + clearkey?
One of the code examples at https://matvp91.github.io/indigo-player/#/Configuration?id=mpeg-dash-drm-widevine-amp-playready use a playready and widevine licence:
{
sources: [
// Play a Dash manifest
{
type: 'dash',
src: 'https://amssamples.streaming.mediaservices.windows.net/622b189f-ec39-43f2-93a2-201ac4e31ce1/BigBuckBunny.ism/manifest(format=mpd-time-csf)',
drm: {
widevine: {
licenseUrl: 'https://amssamples.keydelivery.mediaservices.windows.net/Widevine/?KID=1ab45440-532c-4399-94dc-5c5ad9584bac',
},
playready: {
licenseUrl: 'https://amssamples.keydelivery.mediaservices.windows.net/PlayReady/',
},
},
},
],
}
I thought that i needed to buy a drm licence to use playready or widevine, the license listed above is for free or just for testing?

The configuration above is merely a wrapper around shaka. At the moment, it does not wrap clearkey configuration, but I'd be happy to accept a PR for this.


Temp fix for data loss caused by concurrent flush

Summary:
Temporary fix for #93. Fixing it by waiting until no flush is ongoing before starting a GC job. The drawback is GC can starve if flush runs so frequent that there's no gap between flush jobs.
Test Plan:
See the new test. The test will fail with "Corruption: Missing blob file" error before the fix.

Need to merge https://github.com/pingcap/rocksdb/pull/126 before merging this one.

Please merge latest master


Replace prints with logger calls.

Description
Replace prints with logger calls. Using print() is bad in frameworks and libraries because then there is no way for the application to control the logging. These prints ended up always in CloudWatch logs, no matter what logging level was used.

Coverage decreased (-0.08%) to 73.553% when pulling 66a4d7d55862b74c257d0f00654fcc8a86c6161b on jait:fix/logging-prints into fff5ed8ad2ee071896a94133909adf220a8e47d9 on zappa:master.

It looks like there are a lot of remaining prints in the codebase, can you look into and handle what you can in this PR?


erratic test behaviour

im interested in making this code compatible with python 2 and 3, python 3 works pretty well using 2to3.py, but the get_root() function isn't working for me.
Im using windows 7, testing is done on unaltered code from this repo
running python2 -m py.test tests
collected 33 items
during testing i get a few uac dialog boxes which i confirm
only ever passed once without errors,  There are sometimes 2 sometimes 3 tests that are failed.
testing with tox -e py27:
collected 30
No uac dialog prompts
The tests pass 50% of the time under tox, when it fails its because of the error pasted below.
I'd like some guidance on my next actions, i feel i need to get the tests working 100% of the time before moving on. Not sure what the best course of action is.

        If ignore_errors is set, errors are ignored; otherwise, if onerror
        is set, it is called to handle the error with arguments (func,
        path, exc_info) where func is os.listdir, os.remove, or os.rmdir;
        path is the argument to that function that caused it to fail; and
        exc_info is a tuple returned by sys.exc_info().  If ignore_errors
        is false and onerror is None, an exception is raised.

        """
        if ignore_errors:
            def onerror(*args):
                pass
        elif onerror is None:
            def onerror(*args):
                raise
        try:
            if os.path.islink(path):
                # symlinks to directories are forbidden, see bug #1669
                raise OSError("Cannot call rmtree on a symbolic link")
        except OSError:
            onerror(os.path.islink, path, sys.exc_info())
            # can't continue even if onerror hook returns
            return
        names = []
        try:
            names = os.listdir(path)
        except os.error, err:
            onerror(os.listdir, path, sys.exc_info())
        for name in names:
            fullname = os.path.join(path, name)
            try:
                mode = os.lstat(fullname).st_mode
            except os.error:
                mode = 0
            if stat.S_ISDIR(mode):
                rmtree(fullname, ignore_errors, onerror)
            else:
                try:
                    os.remove(fullname)
                except os.error, err:
                    onerror(os.remove, fullname, sys.exc_info())
        try:
>           os.rmdir(path)
E           WindowsError: [Error 145] The directory is not empty: 'D:\\STORAGE\\
Dropbox\\programming\\testesky\\unchanged\\esky\\tests\\patch-test-temp.win32\\s
ource'

C:\Python27\lib\shutil.py:254: WindowsError
==================== 3 failed, 30 passed in 143.72 seconds ====================
Press any key to continue . . .

Does any one know why tox is collecting different amount of tests?

Some tests are only activated if certain freezer programs are installed, e.g. it will only run the py2exe tests if py2exe is installed.

  return super(TestPatch_pybsdiff,self).tearDown()

Hmm...it's been a good long while since I touched this code, but I do remember the tempfile cleanup being pretty unreliable on windows.  The simplest option may be to either (a) ignore errors when cleaning up tempfiles from the tests, or (b) add a loop to sleep-and-retry the rmtree call until it succeeds.

i notice there are quite a few open pull requests, is there any way i can help close them?

i noticed there are quite a few open pull requests, is there any way i can help close them?

@timeyyy indeed, the truth is I just don't have the bandwidth to keep up with this project these days, and since I'm not using it for any active projects anymore it's hard to make the time.  If you're interested, it would be great to get your comments on any of the open PRs and we can work towards getting them merged.
As you noted here, one of the big problems with maintenance on this project is that the tests are kinda flaky.  It would be great to get travis-ci or similar hooked up on here to give a bit more confidence with moving the code forward on different platforms.  If you have the interest and the time, I'd be delighted to help you work towards such a setup.

:+1 Lets do this!


Implement a NOOP event mapper

To avoid the local repository wrapper taking over control of events, which is problematic given that not all interactions in IGC occur via the Egeria APIs, we must register some event mapper -- to take over control / responsibility for producing events.
We can then ensure that no events are ever sent other than the ones we know we can explicitly handle.

Closing -- agreed that we implement a generic NOOP event mapper in core, and that we can just configure repository proxies to point at this NOOP event mapper.


package_lambda: false is ignored

Describe the bug
package_lambda: false has no effect.
Sample taskcat config
project:
  name: sample-foobar
  owner<EMAIL_ADDRESS>  regions:
    - us-east-1
  package_lambda: false
tests:
  f5-sca-security-stack:
    parameters:
      QSS3BucketName: $[taskcat_autobucket]
      Foo: bar
      sshKey: placeholder
    template: templates/foo.template.json

Sample output
(...)
[INFO   ] : Packaging lambda source from <path/to/project>/lambda_functions/source/foo without building dependencies

To Reproduce
See above config.
Expected behavior
package_lambda: false is honored.
**Version (Please make sure you are running the latest version of taskcat)
❯ taskcat --version                                                                                10509   15:28:02
 _            _             _
| |_ __ _ ___| | _____ __ _| |_
| __/ _` / __| |/ / __/ _` | __|
| || (_| \__ \   < (_| (_| | |_
 \__\__,_|___/_|\_\___\__,_|\__|

version 0.9.13
0.9.13

This feature will be included in the 0.9.18 release of taskcat. In the interim, a development release is available.
pip install taskcat==0.9.18.dev7


No transactions

This is super simple and gets is where we want to go. I'll be super happy if we can contribute this back to the maintainer and less happy if we diverge from the main repo with this. We do have other projects where we vendor a non-standard version and ideally we can bring that number to zero.


Add account not working

Describe the bug
Add account ( + sign next to accounts ) is not working
To Reproduce

click + next to accounts

Expected behavior
should add a new account
Desktop (please complete the following information):
any

@fotescodev  we can prob remove the plus button, it's not active.

Or, make it add accounts?  If one is using the playground to run some scenarios involving typical production scenarios (ok maybe the playground isnt the best place for that, but...) it becomes difficult/impossible due to the 1 contract 1 account featurette.

The functionality for adding additional accounts was not implemented yet. We are going to remove the button for adding accounts (https://github.com/onflow/flow-playground/issues/632) while we work on resolving other bugs. We will be adding this feature back soon.


Add https support for cli tool

Hi Sam, or Max, or both,
This is just a little patch to handle https hosts in cli tool.
It also adds an option to disable certificates check.
I want to emphasize that I am not fluent in coffeescript so my patch may appear ugly for more experienced devs but maybe it is worth submitting this as it may be useful.
Feel free to merge, trash, whatever this :)
Anyway, good job for this application.

I also need this patch, my 0bin instance is an https one :)

+1

TY

Why not :) Thanks for merging!
I'm happy to see that people are interested in my patch.


Translation to pt-BR

Correction on translates to Brazilian Portuguese.

Thank you for all your hard work!


[QUESTION] Google Tag Manager - Full Implementation

Hello idehub,
First of all, thanks a lot for this great plugin, I decided to install it on my project, instead of native code we implemented months ago.
As an analytics lover, I think that the power of GTM is even greater than a traditional GA implementation. I wanted to know if you planned on going further with GTM, I would love to switch my implementation on this tool. And it seems that there is not a lot of good plugins out there.
Anyhow, thanks :)
Xavier

Hey @xavierlefevre!
Thanks for the kind words, good to hear that others find this module useful.
Most of the current implementation done by me has been based on my own needs. I do not do app development professionally, so my "needs" have been pretty much apps as hobby projects.
Limited time will probably keep me from giving this module all the attention it deserves, so I've been reliant on other contributers pitching in if they need a feature. I'll try to help any way I can with pull requests.
With regards to the GTM side of things, Datalayer support (#29) has already been requested. It seems like a simple enough feature for anyone to tackle.
Are there any other specific features you would like to see implemented for GTM? It's better to break it down into single issues which people (you, me or others) can tackle, rather than "full implementation".
Again, if someone wants to pull request even the most simple features, I'll be more than happy to help out.
PS.
One thing to note is that Google recently released Firebase Analytics, which now seems to be the preferred way of implementing GTM. In that regard it might be better to scope down feature requests to the essential, since the SDK used in this package is now considered "legacy".

@cbrevik  Pretty clear to me, I totally understand.
Depending on my needs and time, I might put my nose in all of that and contribute.
In the meantime, this works great and thanks again.

hi @cbrevik , this package still using legacy sdk??
regards!

Yesm this package is not using Firebase @mferranti


Adding CodeQL badge to the README.md

Exceedingly simple way to demonstrate our CI status.
This PR has:

[x] Appropriate testing (molecule, python unit tests)
[x] Appropriate documentation (README in the role, main README is up-to-date)

/lgtm

/approve


fix arbitrary output file number bug, shrink number of files and warn…

This fixes #1173 and #1284
It decreases the number of output files by removing ghost files (files with no data in them) from output.

provides a warning letting the user know that they do not have enough partitions to fit in the selected number of output files and that we had to shrink the number of output files to the match number of ddf partitions.

rerun tests

rerun tests

rerun tests

rerun tests

rerun tests


Surface <-> Mesh Conversions

This PR adds support for conversion between napari's Surface layer type and ImageJ's Mesh data type.

My only concern at this point is how normals are handled. Meshes have normals for each Vertex and Triangle.
@tlambert03 do you happen to know if there is some way for me to pass this data to the Surface layer? I see that there is a normals parameter that represents the normals, but can you actually pass the numerical values of the normals?
My only concern would be that somehow any given normal n would become -n during the conversion.  Do you have any intuition on this @ctrueden?

@tlambert03 do you happen to know if there is some way for me to pass this data to the Surface layer?

@brisvag or @alisterburt would know better than me

@gselzer Normals are often programmatically computed, so you may be able to just ignore the problem for now, and see how things go. And augment the conversion as needed later.

Not sure how normals work here, but in vispy normals are only computed, not provided. They are not held as a piece of data anywhere in napari; instead, they are generated by this visual. I don't remember if they use a right-handed or left-handed system, but I know they do follow one, so if you want your normal to be reversed, simply change the order of the vertices defining that face.

@ctrueden said:

@gselzer Normals are often programmatically computed, so you may be able to just ignore the problem for now, and see how things go. And augment the conversion as needed later.

Yeah, that's how I was doing it in my tests. I suppose we can just wait until we know it doesn't work?
@brisvag said:

Not sure how normals work here, but in vispy normals are only computed, not provided. They are not held as a piece of data anywhere in napari; instead, they are generated by this visual. I don't remember if they use a right-handed or left-handed system, but I know they do follow one, so if you want your normal to be reversed, simply change the order of the vertices defining that face.

Thanks for the speedy reply! I think I will just hope that the normals are correctly preserved during the transformation. All I am doing is making a copy of the vertices and triangles (side note, should we instead make some wrappers for the Mesh type for speed @ctrueden? Might make an issue about this) so I'd think that we are fine so long as ImageJ2 and vispy follow the same system.


Define the design pattern for the data fetching libraries

As a developer I need to have a clear comprehension of how the data fetching libraries are implemented and follow this pattern.
Acceptance criteria:
1 - Data fetching libraries design pattern definition
2 - Data library integration design with data fetching libs

needs test


About not mapping field

Hey,  I'd like to inquire about fields in the database that are not being mapped.
The schema：
func (User) Fields() []ent.Field { 	return []ent.Field{ 		field.String("name"), 		field.String("token"). 			Optional(). 			Nillable(). 			Comment("Transient field for storing token data in-memory"), 	} }
but the ‘token’ Field not in the table column. When i use Query() method ，its takes Error: Unknown column 'user.token' in 'field list'.
What do I need to add the Transient  Field? thanks

Hey @nieshaoyu 👋
In order to add static fields to your struct that do not exist in the database, you can use the custom templates as described here: https://entgo.io/docs/schema-fields#additional-struct-fields.
Closing, but please feel free to reopen or join our Discord community in case you need more help with this 😎


new documentation is not declared

It's offer to specify new documentation with highlight of new verb to can easily detect this

Could you explain your request more detailed, please?
For example, give a link to code that is not well documented?

no, my idea in new docs adding label with the "new doc" to can detect this doc is added new  in documentation


normalize table hover colors

Tables look NICE by default with styles/normalize setup:

Unfortunately they're totally illegible when you hover on them:

Color is dark:

And on hover the background is too:

I will add that my root stylesheet right now is:
@import "open-props/postcss/style";
@import "open-props/postcss/normalize";

body {
    --gap: var(--size-3);
    --header-width: calc(var(--size-header-3) + (2 * var(--gap)));
}

And oddly the app is still rendering in light mode. I was relying on open-props' @media (prefers-color-schema: light) to detect that I'm in dark mode on my machine, but it is not doing that.
Possible solutions:

Add color: var(--gray-1); to the dark background on hover.
Lighten the dark background in light mode to var(--gray-3) to dark consistent with light mode.
Make hover row-highlighting opt-in.

This also might secretly be a completely different bug: You're using it wrong, Chris. Here's why...I'm totally cool with that too.

ATM I fixed it by not worrying about dark mode at all and adding
tr:hover td {
    background-color: var(--gray-3);
}

To my root stylesheet...but, I imagine something is off here.

Not saying that this is definitely the issue, but you posted "@media (perfers-color-schema: light)"
schema with an 'a' instead of scheme
Also, do you think you could post a link to a minimal reproduction? And maybe your environment details as well? (OpenProps and normalize versions, browser + version, OS + version, etc)
I'm not seeing the issue on the Normalize CodePen demo on a Chromium browser on Mac.
https://github.com/argyleink/open-props/assets/29676413/b2e471c2-9727-4148-b5db-13d18834adfe

was gonna ask for a minimal reproduction too (or go make one), but it's already covered here! thanks!

I bet I know what's going on...Gimme five! This is almost certainly a PEBKAC error...

hi guys, how can i addapt notmalize to read from a toffle button in the app ?


[stable/concourse] Add "worker.podManagementPolicy"

Currently, I've set the default podManagementPolicy to "OrderedReady", which matches the previous behavior.
Its probably saner and more appropriate to default this to "Parallel".

/assign @mattfarina
(per @k8s-ci-robot )
cheers :D

/lgtm

/ok-to-test


fix repo debian snapshot container build

currently snapshot build breaks at container build stage

initial investigation suggests issue with debootstrap and debian testing in general:
# debootstrap testing chroot
I: Target architecture can be executed
I: Retrieving InRelease
I: Checking Release signature
I: Valid Release signature (key id 4CB50190207B4758A3F73A796ED0E7B82643E131)
I: Retrieving Packages
I: Validating Packages
I: Resolving dependencies of required packages...
I: Resolving dependencies of base packages...
I: Checking component main on http://deb.debian.org/debian...
I: Retrieving adduser 3.137
I: Validating adduser 3.137
I: Retrieving apt 2.9.2
I: Validating apt 2.9.2
[...]
I: Unpacking vim-tiny...
I: Unpacking whiptail...
W: Failure while installing base packages.  This will be re-attempted up to five times.
W: See /tmp/chroot/debootstrap/debootstrap.log for details (possibly the package /var/cache/apt/archives/cron_3.0pl1-189_arm64.deb is at fault)
W: Failure while installing base packages.  This will be re-attempted up to five times.
W: See /tmp/chroot/debootstrap/debootstrap.log for details (possibly the package /var/cache/apt/archives/cron_3.0pl1-189_arm64.deb is at fault)
W: Failure while installing base packages.  This will be re-attempted up to five times.
W: See /tmp/chroot/debootstrap/debootstrap.log for details (possibly the package /var/cache/apt/archives/cron_3.0pl1-189_arm64.deb is at fault)
W: Failure while installing base packages.  This will be re-attempted up to five times.
W: See /tmp/chroot/debootstrap/debootstrap.log for details (possibly the package /var/cache/apt/archives/cron_3.0pl1-189_arm64.deb is at fault)
W: Failure while installing base packages.  This will be re-attempted up to five times.
W: See /tmp/chroot/debootstrap/debootstrap.log for details (possibly the package /var/cache/apt/archives/cron_3.0pl1-189_arm64.deb is at fault)

So it may be an issue with debian / debootstrap upstream

fixed by 34b01b2 (updated builder uses mmdebstrap instead of debootstrap => resolve dependencies correctly)


Conflict with AutoPagerize

I use this extension to automatically show next pages on Google:
https://chrome.google.com/webstore/detail/kdplapeciagkkjoignnkfpbfkebcfbpb
The problem is, if I have chatGPTBox and AutoPagerize installed, then chatGPTBox will show up at the very bottom of the page. But if I scroll down to the very bottom, a new next page automatically loads for me - which means I cannot use chatGPTBox.

turn on Insert ChatGPT at the top of search results

That worked, thanks. I'm stupid, I should have checked the options.


Update router.ts with crypto import

Bypassing the contribution guidelines, as this is not a change directly on repository that needs a version change or something similiar. If I still need to open an issue for this and fill this in, let me know, then I will do the necessary steps.
This only changes one of the examples that are provided.
Just adding a missing import for the use of the crypto lib.

I don't think we need this import since the LTS Node 20.


Initial Commit for AvroPayloadSerializer

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @TheNeuralBit
Could you review this ? This is initial commit. I will continue develop based on your feedbacks.
Thanks

I'd be interested to see the performance of this vs FastReaderBuilder.createDatumReader and wrapping the resulting object in a Row.

That documentation is sparse, is there more somewhere?

I'd be interested to see the performance of this vs FastReaderBuilder.createDatumReader and wrapping the resulting object in a Row.

Let me try to write a JMH test

That documentation is sparse, is there more somewhere?

I'm not sure actually, I'd stumbled across it a year or two ago because I was doing something very similar to what's being done here.  https://issues.apache.org/jira/browse/AVRO-2247 is the most detail I can find on the work being done to improve the performance.
I realize now after writing that comment though that this wasn't in the java avro library until 1.10, and beam is still on 1.8 (or maybe 1.9?), so it might not be particularly relevant anyways.

That documentation is sparse, is there more somewhere?

I'm not sure actually, I'd stumbled across it a year or two ago because I was doing something very similar to what's being done here. https://issues.apache.org/jira/browse/AVRO-2247 is the most detail I can find on the work being done to improve the performance.
I realize now after writing that comment though that this wasn't in the java avro library until 1.10, and beam is still on 1.8 (or maybe 1.9?), so it might not be particularly relevant anyways.

Ah ok. If we run some benchmarks now and it looks good we could file an issue to consider switching to it when/if we upgrade Avro. For reference #19969 is the only avro upgrade issue I can find.

Good day, @talatuyarer. Thank you again for making this contribution! Would you like to still work on this PR or should I close it? We appreciate what you do for the Beam community!

Hi @damondouglas Let me work on it. I want to merge this MR to BEAM. It makes Beam's AVRO serialization more than 3x faster.


ExampleSetFlag leaks in the output generated

It seems that when I generate An openapi file after parsing an old one which is a valid openAPI file, this property "ExampleSetFlag" is either been set to false or true and then it'is added to the specification file that is generated which won't be a valid openapi speciication
Can someone help me figure out why is hapening, knowing that I am using the latest version of swagger parser
here is the artefact :

io.swagger.parser.v3
swagger-parser
2.0.33

by the way this a portion of the generated file, and here is what it looks like :
"paths" : {
"/rs-services/v2/authentication/hostedFieldsInitialize" : {
"post" : {
"tags" : [ "Authentication" ],
"description" : "",
"operationId" : "hostedFieldsInitialize",
"requestBody" : {
"content" : {
"application/json" : {
"schema" : {
"$ref" : "#/components/schemas/HostedFieldsInitializeInput",
"exampleSetFlag" : false
},
"example" : "",
"exampleSetFlag" : true
}
}
},
"responses" : {
"200" : {
"description" : "The request has succeded",
"content" : {
"application/json" : {
"schema" : {
"$ref" : "#/components/schemas/HostedFieldsInitializeOutput",
"exampleSetFlag" : false
},
"example" : "",
"exampleSetFlag" : true
}
}
}
}
}
},
and also in the components section, here is a portion of it
"components" : {
"schemas" : {
"AcceptChallengeInput" : {
"required" : [ "interfaceVersion", "merchantId" ],
"type" : "object",
"properties" : {
"comment" : {
"type" : "string",
"description" : "",
"exampleSetFlag" : false
},
"interfaceVersion" : {
"type" : "string",
"description" : "",
"exampleSetFlag" : false
},
"intermediateServiceProviderId" : {
"type" : "string",
"description" : "",
"exampleSetFlag" : false
},
"keyVersion" : {
"type" : "integer",
"description" : "",
"format" : "int64",
"exampleSetFlag" : false
},
"merchantId" : {
"type" : "string",
"description" : "",
"exampleSetFlag" : false
},
"operationOrigin" : {
"type" : "string",
"description" : "",
"exampleSetFlag" : false
},
"s10TransactionReference" : {
"description" : "",
"$ref" : "#/components/schemas/S10TransactionReference",
"exampleSetFlag" : false
},

see https://github.com/swagger-api/swagger-core/issues/4381 issue

please see this comment about using the correctly configured object mapper.
closing ticket, please reopen if you're still experiencing issues


Update project to Godot 3.2.2

Open and save all scenes and resources in Godot 3.2.2. Also, update the gitignore.
I noticed that Jetpaca has a lot of places that need improvement. If desired, I can work on it.

Thanks!


Broken link to AppEngine API in Groups documentation page

AppEngine API link in https://docs.astarte-platform.org/0.11/065-groups.html leads to a 404.

Duplicate of #337


[Nightly, possibly] Sent messages don't sync properly

This was a conversation I had on iOS, and when I went to check it in BlueBubbles, only the received messages were showing up. Exiting and re-entering the chat fixed the issue, I'll switch to the PlayStore build to see if this bug happens there too.

@CSab6482 is this still an issue?

Yes but I’m writing it off as an Apple bug as I get this bug on my native iMessage devices when I have the same conversation open on multiple devices. It’s a weird one.


ModuleNotFoundError: No module named 'SinGAN'

Hello, I was pretty thrilled to attempt your loss after reading your paper, however, when I tried to adopt the MDF loss by using Denoising.pth  it didn't work, it keeps giving me this issue, could you please tell me how to use your loss in another model?

Hi,
The same issue has been solved here : https://github.com/gfxdisp/mdf/issues/5
Thanks


ci: fix android build

#107  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Brooooooklyn and the rest of your teammates on  Graphite


pytest scans every py file for tests

This caused me issues with a utility script blowing up pytest.
This fixed it for me - but will this have any unwanted consequences?
In setup.cfg:
[tool:pytest]
testpaths =
    tests

It seems reasonable to limit the places pytest looks for tests, but we potentially have tests in 3 places:

Unit tests in tests
Doctests in docstrings in src
Doctests in documentation in docs
Adding those 3 directories seems to work

https://github.com/dls-controls/scanspec/commit/8190f0e224570fc203d4a6fff4f19ab1e1b7a347


Some form_builder methods could not be used in my form?

in active_admin/lib/active_admin/form_builder.rb i see
commit_button_with_cancel_link
cancel_link
commit_button
methods
If i use
f.commit_button in my form

=semantic_form_for [:admin, @resource] do |f|
  = f.input :some_field
  = f.commit_button

It works, but when i use commit_button_with_cancel_link , or cancel link i have "undefined method `cancel_link' for #" error.
What is going on ?

First of all, this is a good catch - the implementation of the formtastic extensions is using the old style which is no longer recommended (i.e. subclassing Formtastic::FormBuilder). Which means that it should probably be updated to support the newer version of formtastic. What this means for you is that if your version of Formtastic is at 2.x, it's possible the active_admin defined code for formtastic just won't work right.

+1
I can't use has_many method of ActiveAdmin FormBuilder in a partial view, which i think is very good. Any way to achieve it?


make cmake config usable as external dependency

current cmake config is not suitable to be used externally. (via add_subdirectory)

this CMake will be used for managing examples/demos/editor as well as the core of wolf.
If you want to link wolf as a stand-alone library to your project, please use this CMake.


[FALSE-POSITIVE] etsy.com www.etsy.com

Domains or links
etsy.com www.etsy.com
More Information
How did you discover your web site or domain was listed here?
Could not visit website
Have you requested removal from other sources?
No
Additional context
This is a online marketplace for handmade goods
:exclamation:
We understand being listed on a Phishing Database like this can be frustrating and embarrassing for many web site owners. The first step is to remain calm. The second step is to rest assured one of our maintainers will address your issue as soon as possible. Please make sure you have provided as much information as possible to help speed up the process.
Send a Pull Request for faster removal
Users who understand github and creating Pull Requests can assist us with faster removals by sending a PR on the whitelist.me file

https://github.com/mitchellkrogza/Phishing.Database/blob/master/whitelist.me/whitelist.me
Please include the same above information to help speed up the whitelisting process.

@mitchellkrogza This is still being blocked. I cannot re-open this issue.


Feature/bugs

Divide each section (except Hero) into two parts:
Title-container (centered title).
Content-container (section content).
Adjust the layout to a 30:70 ratio (Title on the left, content on the right).

commit style changes required. please commit statement in past tense


how to compress source code

i use the commond  webpack --progress -d --optimize-mangle --optimize-dedupe --optimize-occurence-order --output-filename clappr.min.js  to generator clappr.min.js but it looks like
(function webpackUniversalModuleDefinition(root, factory) {
	if(typeof exports === 'object' && typeof module === 'object')
		module.exports = factory();
	else if(typeof define === 'function' && define.amd)
		define([], factory);
	else if(typeof exports === 'object')
		exports["Clappr"] = factory();
	else
		root["Clappr"] = factory();
})(this, function() {
..........

	module.exports = keys;

/***/ }
/******/ ])))
});
;
//# sourceMappingURL=clappr.min.js.map

but i look the clappr.min.js  file at http://cdn.clappr.io/
it looks like
!function(e,t){"object"==typeof exports&&"object"==typeof module?module.exports=t():"function"==typeof define&&define.amd?
.........................

what's the problem(764kb vs 304kb), why my file is so large ,do i use the wrong package commond ?

Hey @569934390
Please can you run npm run release, and then if it doesn't work paste what gets logged to the console here?
I just did this and it works fine for me.

@569934390 can we close this?

it works,thanks, but what's different between commond 'npm run release' and 'webpack --progress -d --optimize-mangle --optimize-dedupe --optimize-occurence-order --output-filename clappr.min.js'?


Isinstance issue

Issue #, if available:
Description of changes:
This is to check if isinstance will fail the CI test on python3.7 . Im sending this PR so that all CI tests are triggered to run.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for adding a minimal code. I see what you mean now more clearly now, doing isinstance with a generic type is indeed not possible in python 3.7.
That being said, are there any example of using isinstance("x", Hyperparameter) in the code? I cannot find any, so it is not clear to me if that would be an issue.

When working on gridsearcher, there was a need to use it. See this function: https://github.com/awslabs/syne-tune/blob/main/syne_tune/optimizer/schedulers/searchers/searcher.py#L632
I circumvented the issue by using elif not isinstance(hp_range, Domain) in line L651. Ideally i wanted to check if isinstance(hp_range, Hyperparameter).

Ok thanks I see. The solution you currently have would work although it may be better to explicitly check the types that are allowed (e.g. to assert that you have the str or numeric when entering the branch).
Regarding being able to test isinstance(x, Hyperparameter), I think it is ok not to support it as it is for now a bit of a niche case which we can bypass.

Ok thanks I see. The solution you currently have would work although it may be better to explicitly check the types that are allowed (e.g. to assert that you have the str or numeric when entering the branch).
Regarding being able to test isinstance(x, Hyperparameter), I think it is ok not to support it as it is for now a bit of a niche case which we can bypass.
fair enough. I will close the PR then.