asv preview not loading graphs. Lots of 404s in the console

I am trying to use asv to visualize the performance difference between two commits. I ran the benchmarks fine (it is these benchmarks, except I changed the repo url to my local repo).
But when I asv publish and asv preview, none of the graphs load. In the console, I have a lot of errors like
:8080/graphs/summary/cse.TimeCSE.time_cse.json?_=1501786104493 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/dsolve.TimeDsolve01.time_dsolve.json?_=1501786104494 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/integrate.TimeIntegration01.time_doit.json?_=1501786104495 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/lambdify.TimeLambdifyCreation.time_lambdify_create.json?_=1501786104497 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/large_exprs.TimeLargeExpressionOperations.peakmem_jacobian_wrt_functions.json?_=1501786104499 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/integrate.TimeIntegration01.time_doit_meijerg.json?_=1501786104496 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/lambdify.TimeLambdifyEvaluate.time_lambdify_evaluate.json?_=1501786104498 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/large_exprs.TimeLargeExpressionOperations.peakmem_jacobian_wrt_symbols.json?_=1501786104500 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/large_exprs.TimeLargeExpressionOperations.time_cse.json?_=1501786104503 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/large_exprs.TimeLargeExpressionOperations.time_count_ops.json?_=1501786104502 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/large_exprs.TimeLargeExpressionOperations.peakmem_subs.json?_=1501786104501 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/large_exprs.TimeLargeExpressionOperations.time_free_symbols.json?_=1501786104504 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/large_exprs.TimeLargeExpressionOperations.time_jacobian_wrt_functions.json?_=1501786104505 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/large_exprs.TimeLargeExpressionOperations.time_jacobian_wrt_symbols.json?_=1501786104506 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/logic.LogicSuite.time_dpll.json?_=1501786104509 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/logic.LogicSuite.time_dpll2.json?_=1501786104510 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/large_exprs.TimeLargeExpressionOperations.time_manual_jacobian_wrt_functions.json?_=1501786104507 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/large_exprs.TimeLargeExpressionOperations.time_subs.json?_=1501786104508 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/logic.LogicSuite.time_load_file.json?_=1501786104511 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/solve.TimeMatrixArithmetic.time_dense_multiply.json?_=1501786104515 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/solve.TimeMatrixOperations.time_det.json?_=1501786104516 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/physics.mechanics.kane.KanesMethodMassSpringDamper.time_kanesmethod_mass_spring_damper.json?_=1501786104512 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/solve.TimeMatrixOperations.time_det_berkowitz.json?_=1501786104518 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/solve.TimeMatrixArithmetic.time_dense_add.json?_=1501786104514 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/physics.mechanics.lagrange.LagrangesMethodMassSpringDamper.time_lagrangesmethod_mass_spring_damper.json?_=1501786104513 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/solve.TimeMatrixOperations.time_det_bareiss.json?_=1501786104517 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/solve.TimeMatrixOperations.time_rank.json?_=1501786104519 Failed to load resource: the server responded with a status of 404 (File not found)
:8080/graphs/summary/solve.TimeMatrixOperations.time_rref.json?_=1501786104520 Failed to load resource: the server responded with a status of 404 (File not found)

I've tried clearing the html directory and republishing and it doesn't help.
This is with asv 0.2. I also tried using the git version, but that failed:
$PYTHONPATH=~/Documents/asv python -m asv publish
Traceback (most recent call last):
  File "/Users/aaronmeurer/anaconda3/lib/python3.5/runpy.py", line 183, in _run_module_as_main
    mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
  File "/Users/aaronmeurer/anaconda3/lib/python3.5/runpy.py", line 142, in _get_module_details
    return _get_module_details(pkg_main_name, error)
  File "/Users/aaronmeurer/anaconda3/lib/python3.5/runpy.py", line 109, in _get_module_details
    __import__(pkg_name)
  File "/Users/aaronmeurer/Documents/asv/asv/__init__.py", line 17, in <module>
    from ._version import __version__, __githash__, __release__
ImportError: No module named 'asv._version'

The git version probably isn't properly installed if asv._version is
missing --- setup.py should create it.

Hmm, why do I have to run setup.py to "compile" asv? Isn't it pure Python?

Oh I see now that it isn't. I'm sure yall have a good reason for it not being so.
At the very least, it would be nice to get a better ImportError if build_ext hasn't been run.

Well, that's how it works now. Also, it's not pure-python.

Of course, there's no reason to require _version.py to be present, and the non-Python parts are optional. Pull requests making importing _version.py optional are accepted.

Usually one stores the version in the package and reads it from setup.py, not the other way around.
If you want to include the git revision, I recommend using versioneer, which handles all this logic for you, and does it the "right" way.

Anyway, back to the original issue, I got the dev version working, and the bug still persists there.

If you can upload the stuff (results etc) needed to run "asv preview"
etc. somewhere that can help debugging, that can be useful (don't have
time to look into it immediately though)

Let me know if this is enough (I just did git add -f results). There is also env and sympy directories, but they are both huge.

I think the reason why it doesn't generate any data points is because the benchmark results are not on the branch in asv.conf.json (at least for me, those commits don't exist in the repository --- maybe your situation is different?). The generated website only contains results for commits on the branches configured.

Thanks, that was the ticket. I am trying to use asv to just get an idea if a change that I want to make is slower or faster, so it isn't in master.

The web ui has not really been done with that use case in mind, so it may be clumsy to use for that. With two commits, the step detection algorithms in the web ui will also not give you results since there's no statistics.
On command line asv continuous and asv compare are more designed for this.

Thanks I'll take a look at those two.

Although I think I maybe do want more than two commits, in case I have multiple changes in my branch.
Maybe I should run asv against master^30..mybranch or something like that, so it can get reasonable statistics.


Use memory start address argument in DAQ

Partial fix for #17

This is not yet fully fixing the issue, is it?

This is not yet fully fixing the issue, is it?

no, not yet


dbt fixes for running external ClickHouse

@Ian2012 rebased on your latest changes, need a new thumb if it looks good


Plenary error after latest upgrade (June 12th, 2021)

When launching NeoVim, I'm getting:
Error executing luv callback:
...acker/start/plenary.nvim/lua/plenary/async_lib/async.lua:14: The coroutine failed with this message: ...nvim/site/pack/packer/start/neogit/lua/neogit/status.lua:290: attempt to index field 'repo' (a nil value)

This results in a completely empty status view. I am in a valid Git repo. I'll do more digging.
*Edit - updated to latest HEAD of NeoVim 0.5 and made sure plenary was updated. Same error persists

I pushed an update that might fix this. I can't really reproduce this so this is more a guess than anything.

It worked. The message is gone and the plugin is working fine agian.

Yep, fixed it for me. My apologies for the lack of information in my original issue.


add relative path (URL) support

Currently it is not possible to start plantuml-editor with a relative URL like https://sub.exmaple.com/plantuml-editor instead of https://sub.exmaple.com.
The reason for this is that the publicPath in the vue.config.js was not explicitly set. This pulls the default value /, which always loads everything from the root directory of the servers.

Since this PR is quite old and there even exists a plantuml-editor2 project by now, I will close this PR.


Issue in installing conda_packages.txt

I am trying to install all the requirement as define in conda_packages.txt in linux64, but many requiremnet packages showing this error. PackagesNotFoundError: The following packages are not available from current channels. Can you please tell me which python version and conda version, I have to use to install all these packages.

I am trying to install all the requirement as define in conda_packages.txt in linux64, but many requiremnet packages showing this error. PackagesNotFoundError: The following packages are not available from current channels. Can you please tell me which python version and conda version, I have to use to install all these packages.

Bro I am also working on the same project. Would you mind sharing your contact details(email or whatsapp number). I unable to do my project, Please help me.


Error closing error window

When OmniDB shows error window, I can't close it, I click on the red X and it doesn't close

Hi @erdusa ,
Thanks for reporting this, it will be fixed for 2.12.1.
Regards,
William

Hi @erdusa ,
We are releasing OmniDB 2.13.0 today, with a fix for this issue.
I'll keep this issue open in case you want to give us your valuable feedback, as always.
Best regards,
William

Hi @wind39
This fixed the problem
Thank you


Grammatical correction in doc

Docs Build status updates of commit a5e19f0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/tabs/how-to/authentication/auth-aad-sso.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Make the large type window mouse-selectable

Means it can be selected (and closed) if it was never made key (e.g. when opened during screensaver)
Fixes #1903
Also - various tidyups

No! I'll leave this open to remind me to fix it in a few days... :-)


Realm crash: reason: 'std::exception'

Goals
App not crashing when using Realm Studio
Expected Results
We able to use Realm Studio to view DB state in real-time (when the application works)
Actual Results
Application crashes with the following stack trace:
2020-09-11 09:55:35.176272+0300 Perenio Smart[12494:82063] *** Terminating app due to uncaught exception 'RLMException', reason: 'std::exception'

*** First throw call stack:
(
0 CoreFoundation 0x00007fff23e3de6e _exceptionPreprocess + 350
1 libobjc.A.dylib 0x00007fff512a19b2 objc_exception_throw + 48
2 Realm 0x000000011441e5f6 Z20RLMThrowResultsErrorP8NSString + 648
3 Realm 0x000000011441f157 -[RLMResults firstObject] + 183
4 RealmSwift 0x00000001153c2a7d $s10RealmSwift7ResultsV5firstxSgvg + 45
5 Perenio Smart 0x0000000107944d6d $s13Perenio_Smart19DevicesRealmServiceC06updatec21AllValuesFromWSDeviceH07payload17completionHandleryAA0jH0V_ySSSgcSgtFy0D5Swift0D0VSgcfU + 2397
6 Perenio Smart 0x0000000108a2bbc0 $s13Perenio_Smart12RealmManagerC010backgroundC9Operation9operationyy0C5Swift0C0VSgc_tFyycfU + 272
7 Perenio Smart 0x00000001073c5e60 $sIeg_IeyB_TR + 48
8 Perenio Smart 0x00000001089c73eb $s13Perenio_Smart12ThreadWorkerC12threadAction33_8CAFB1C0A9E61CB634B7C7A3E87A6E29LL9operationyyyXB_tF + 91
9 Perenio Smart 0x00000001089c744c $s13Perenio_Smart12ThreadWorkerC12threadAction33_8CAFB1C0A9E61CB634B7C7A3E87A6E29LL9operationyyyXB_tFTo + 60
10 Foundation 0x00007fff2597de42 __NSThreadPerformPerform + 209
11 CoreFoundation 0x00007fff23da1c91 CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION + 17
12 CoreFoundation 0x00007fff23da1bbc __CFRunLoopDoSource0 + 76
13 CoreFoundation 0x00007fff23da1394 __CFRunLoopDoSources0 + 180
14 CoreFoundation 0x00007fff23d9bf8e __CFRunLoopRun + 974
15 CoreFoundation 0x00007fff23d9b8a4 CFRunLoopRunSpecific + 404
16 CoreFoundation 0x00007fff23d9c7f8 CFRunLoopRun + 104
17 Perenio Smart 0x00000001089c6de1 $s13Perenio_Smart12ThreadWorkerC15populateRunLoop33_8CAFB1C0A9E61CB634B7C7A3E87A6E29LLyyF + 497
18 Perenio Smart 0x00000001089c6feb $s13Perenio_Smart12ThreadWorkerC15populateRunLoop33_8CAFB1C0A9E61CB634B7C7A3E87A6E29LLyyFTo + 43
19 Foundation 0x00007fff2597d9eb NSThread__start + 1047
20 libsystem_pthread.dylib 0x00007fff52305109 _pthread_start + 148
21 libsystem_pthread.dylib 0x00007fff52300b8b thread_start + 15
)

libc++abi.dylib: terminating with uncaught exception of type NSException

Steps for others to Reproduce

Run the application and create Realm notifications (observe some objects or a collection)
Run Realm Studio and open your realm-file
Perform some write operation in the application
Crash

Version of Realm and Tooling
Realm Studio version: 3.11.0
Realm framework version: v5.3.6, v5.4 (v5.3.5 works fine)
Realm Object Server version: not used
Xcode version: 11.7
iOS/OSX version: iOS Simulator (iOS 13.7)
Dependency manager + version: Cocoapods/1.9.3

Hi @codemeister64
Since 5.3.6 we introduced a mechanism to combat an issue in iOS 14 beta where file locking when background an app would cause it to crash. I believe what you are seeing is an byproduct of that fix we introduced.
@tgoyne would it make sense if the simulator took a different method to file locking?

https://github.com/realm/realm-core/pull/3907 switches the simulator back to the old locking scheme for Studio compatibility.

Closing this issue due to inactivity. If you have any other questions please open a new issue. Thanks.

Closing this issue due to inactivity. If you have any other questions please open a new issue. Thanks.

Closing this issue due to inactivity. If you have any other questions please open a new issue. Thanks.

Closing this issue due to inactivity. If you have any other questions please open a new issue. Thanks.


secure_demo: generate statically-keyed demo image

This format builds a disk with a single ESP partition containing
lockdown.efi, a signed shim, and a CoreOS image configured to boot
using UEFI secure boot.

this is now both obsolete and irrelevant


v1.14.0-beta3 file without "dist" directory

Just a question : is it possible to have a full archive with file that we can use directly in our web pages ?
v1.14.0-beta2 has this directory... Why this new version do not have one ?
Thank's for all & your work,
Thierry

You can build the dist directory using grunt build, or you can use something like CDNJS or JsDelivr. It's best practice to not include built files in the source repo.


Migrate to Stencil-Citadel

Migrate to Stencil-Citadel - We have moved Citadel to a different repo

@sherrybc @bc-ejoe

:+1:

thanks guys!


Add support for custom call policies

This PR adds a new function annotation, nb::call_policy<Policy>(), which causes static methods of the user-provided Policy to be invoked before and after the bound C++ function. Unlike call_guard, the call policy methods operate on the Python object level and have access to the function arguments (precall can even modify them) and return value. This allows various esoteric user customizations that might be desirable in certain circumstances, with a minimal impact on the footprint of nanobind itself.
Motivating use case: a number of the types of interest in my application support a lightweight callback mechanism. The callback type we use is trivially copyable, so it can't manage ownership of a Python object directly. A custom call policy allows us to automatically take out a reference to the underlying Python callable when a new callback is subscribed, and drop the reference when it's unsubscribed. Allowing precall() to modify the argument list permits normalization (letting equal-but-distinct Python callables map to identical C++ callback objects, so that the C++ unsubscribe logic can find the callback that was originally subscribed).
While working on this I noticed that a function annotated with nb::keep_alive will leak a reference to its return value if the keep_alive can't be set up. I worked around the analogous issue for call policies, but didn't change the implementation of keep_alive in case this is a deliberate performance tradeoff.

Hi @oremanj,
this looks like a neat feature and generalization of keep_alive. I am wondering if you would expect any run-time overhead on existing extensions? From a quick skim, I would say that everything should compile to basically the same code, but perhaps there was something I missed.

I don’t expect any overhead in terms of binary size or runtime cost for extensions that don’t use the new feature. Fixing the bug I mentioned in the last paragraph (keep_alive leaks a reference to the return value if it throws an exception) will imply a minor binary size cost for additional exception-handling landing pads, but I didn’t do that in this PR.

Do you think it would be possible to explain a baby version of such a feature? It would make it more clear that powerful extensions are feasible with this escape hatch.

Done. It's a bit too long to include in the documentation, so I added it as a unit test.

I'm going to need some time to figure out why the 3.12+ Windows/macOS tests are failing. I can't reproduce the failure on my own macOS dev machine, nor on Linux with various sanitize/malloc-check options, nor can I figure out what's going on by staring at it. If you have any ideas, I'm all ears.

On 3.12+, the CI uses stable ABI builds AFAIK. Can you repro the issue if you set NB_TEST_STABLE_ABI=ON?

The new example looks fantastic, thank you very much for that!

The stable ABI issue is a good thought, but unfortunately I'm not able to repro the CI failure on macOS even with -DNB_TEST_STABLE_ABI=ON -DNB_TEST_SHARED_BUILD=1. I tried both debug and release builds, both 3.12.0 and 3.13.0, and I verified that it is actually building stable-ABI extensions.

I figured it out - the nb_type_data dictoffset / weaklistoffset fields would be uninitialized (rather than zero) on stable-ABI builds if the corresponding feature weren't used, which would then confuse inst_traverse's assumption that nb_dict_ptr returns null if there is no dict. Unrelated to this PR, just happened to be tickled by it. I rolled the fix into this PR instead of making a separate one, hope that's OK: https://github.com/wjakob/nanobind/pull/767/commits/1d64365ae1d3d852adcfef32eefdb05e2e92271a

Thank you for finding this bug, I am surprised this had not come up before.

GitHub says: "This branch cannot be rebased safely. Rebasing the commits of this branch on top of the base branch cannot be performed automatically as this would create a different result than a regular merge."
Can you rebase on top of master?

Not sure what's going on there - the rebase succeeded without issue - but I've pushed the rebase version so hopefully it'll work now.

The CI failures are unrelated to the code (network issue fetching from PyPI) but I can't rerun the workflow since I don't have write access to the repo.

Thanks, I reran them.


Explore using StoneSoup as the image build/CI piece

To package more of our products together it would be interesting to see if we could couple StoneSoup and the templates. This would allow for StoneSoup to perform the building, verifying, and securing and then GH actions or Tekton to bump the release.
/cc @jduimovich
https://github.com/jduimovich/appstudio-e2e-demos/blob/main/demos/dogfood-builds/components/build-service.yaml
A key required flag is skipGitOpsResourceGeneration: true

This is interesting, but we should be investigating higher-priority issues now.  Keeping this open, but noting that priority should be lower.


Does it work only for Case entity?

This is great, but does it work only with Case entity?
I have a custom activity entity, where I would love to auto populate data from Outlook items.
Is there any way to configure this?

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

ID: 68bc2ed8-f878-94e7-1681-6904412b9486
Version Independent ID: 95d24cfc-bff1-d693-1353-51aab2597455
Content: Customize App for Outlook to auto populate data to the Quick Create form  (Dynamics 365 apps)
Content Source: ce/outlook-app/auto-populate-data-into-quickcreate.md
Service: crm-online
GitHub Login: @mduelae
Microsoft Alias: mkaur

Thanks for contacting us. Unfortunately, we're unable to handle Support requests for the product here. To contact Support, consider one of the options described here: https://docs.microsoft.com/dynamics365/get-started/support/
Closing this issue


Instrumentation for RegressPhenotypes

Adds in instrumentation for RegressPhenotypes which can be ported to other section of code when the need arises.

Can one of the admins verify this patch?

pinging @fnothaft @heuermh to approve Jenkins build test.

I was waiting on a successful build for #12 before adding other pull requests to Jenkins.

Jenkins, add to whitelist


Fix up HTML escaping in Reference Cells table

This removes some of the HTML escaping in the Reference Cells features table and replaces it with code.

Open Publishing Build Service: The pull request content has been published and here are some changed files links of this time:

Docset 1 - core-docs
https://stage.docs.microsoft.com/en-us/dotnet/core/api/system.componentmodel.composition.primitives.composablepartexception?branch=pr-en-us-1021
https://stage.docs.microsoft.com/en-us/dotnet/core/api/system.componentmodel.composition.primitives.contractbasedimportdefinition?branch=pr-en-us-1021
https://stage.docs.microsoft.com/en-us/dotnet/core/api/system.componentmodel.composition.primitives.export?branch=pr-en-us-1021
Please see 3392 more published files of this docset in report.

Status: Succeeded.
Open Publishing Report.

LGTM :shipit:


Docs for kpt fn eval --save

Related to https://github.com/GoogleContainerTools/kpt/pull/2841. Only merge before the next release.

We cut a release today, so we can probably merge this now ?

doc is covered by https://github.com/GoogleContainerTools/kpt/pull/2977/files


WIP -- Initial support for Artix 100t parts

Initial support for xc7a100t parts:
board   arty100t-full
arch    artix7_100t
device  xc7a100t-virt
parts   xc7a100tcsg324-1, xc7a100tfgg676-1
tests   "arty100t" versions of:  buttons, counter, ddr, uart_loopback, and soc/litex/mini_ddr.

Larger tests for soc/litex/base and scalable_proc are not included in this commit, but they also work (on board, and pass vivado_diff_fasm).

Requires this PR https://github.com/SymbiFlow/symbiflow-arch-defs/pull/1511 to be merged to get rid of the __vpr_delayless_switch__ error.

Mostly LGTM!  Couple comments added.

There appears to be an unexpected failure in the vendor tool tests presumbit:
FAILED: cd /tmpfs/src/github/symbiflow-arch-defs-presubmit-xc7-vendor/build/xc/xc7/tests/soc/litex/base/baselitex_arty/artix7-xc7a50t-virt-xc7a50t-test && /usr/bin/cmake -E remove -f baselitex_arty/artix7-xc7a50t-virt-xc7a50t-test/design_baselitex_arty_vivado.dcp && /usr/bin/cmake -E remove -f baselitex_arty/artix7-xc7a50t-virt-xc7a50t-test/design_baselitex_arty_vivado.xpr && /tmpfs/src/github/symbiflow-arch-defs-presubmit-xc7-vendor/third_party/prjxray/utils/vivado.sh -mode batch -source /tmpfs/src/github/symbiflow-arch-defs-presubmit-xc7-vendor/build/xc/xc7/tests/soc/litex/base/baselitex_arty_vivado_runme.tcl > /tmpfs/src/github/symbiflow-arch-defs-presubmit-xc7-vendor/build/xc/xc7/tests/soc/litex/base/baselitex_arty/artix7-xc7a50t-virt-xc7a50t-test/vivado.stdout.log

Given that it is related to the A50T arch, not the A100T I assume it's an unrelated unstable failure.  Let's see if it reoccurs after other changes are made.

I see a CI failure related to diff fasm for the baselitex test. Double check if eg. BUFGs are not constrained to illegal locations. I recall that we had locked them for A50T it may not be correct for A100T.

Thanks for pointing this out, Maciej.   I'll need to dig deeper on this one.  It puzzles me; the error seems to be for the 50T test, since I didn't add a 100T version (yet).   But I didn't think I touched anything that would affect the 50T tests.   I'm rerunning locally now to see if I can reproduce the failure.

I see a CI failure related to diff fasm for the baselitex test. Double check if eg. BUFGs are not constrained to illegal locations. I recall that we had locked them for A50T it may not be correct for A100T.

Hi Maciej -- it puzzles me, but the failing test was the original 50T version, which my changes shouldn't affect.   It may be an unrelated random failure.   We'll see if it occurs again in the tests now running.

I see a CI failure related to diff fasm for the baselitex test. Double check if eg. BUFGs are not constrained to illegal locations. I recall that we had locked them for A50T it may not be correct for A100T.

Hi Maciej, the CI test where this happened now died much earlier.   Can you take a look?   Is it just random?   How can I rerun the test -- just push an empty commit to my branch, or is there a button I can click?    Thanks!

Tests are failing with issue #1400 and also this error:
ERROR: [Common 17-1294] Unable to create directory [/tmpfs/src/github/symbiflow-arch-defs-presubmit-xc7-vendor/build/tests/9-scalable_proc/top_bram_n3/artix7-xc7a50t-basys3-roi-virt-xc7a50t-basys3-test/.Xil/Vivado-22251-kokoro-gcp-ubuntu-prod-1092307855/dcp1].
I also see: kokoro_log_reader: Exhausted retries waiting for bash to return an integer exit code, returning exit code 137 instead., but I'm not sure if that's an independent error, or a result of the earlier errors.

@acomodi , can you check my  failing run log?   I think I just have errors due to issue #1400, but can you see if there's anything else of concern?   Thx

@tcal-x Checked, the issues seem all to be related to #1400


Custom Eloquent Builder not resolved after relation builder methods

Larastan Version: 0.7.11
--level used: 5
Pull request with failing test: https://github.com/nunomaduro/larastan/pull/896

Description
On my project, I rely heavily on custom eloquent builders. To also take advantage of auto completion in phpstorm,  all model calls starting with the query() method.
Example:
User::query()
    ->whereHas('') // 'normal' builder method
    ->admin() // custom eloquent builder method
    ->first()

Bug:
After calling whereHas() (maybe on other relation methods as well) the custom eloquent builder is not resolved and an error is thrown.
Call to an undefined method Illuminate\Database\Eloquent\Builder<App\Models\User>::admin().

This bug only appears when combining query(), whereHas() and customBuilderMethod() in this particual order.

Thank you for your report.
It'll be resolved in a very short time. ⏳

Hi,
Did you setup your custom builders correctly? Like it's described here: https://github.com/nunomaduro/larastan/blob/master/UPGRADE.md#custom-eloquent-builders
I tested your PR locally, and it works as expected.

I hope I've setup them correctly. Here are my model, custom builder and method call:
class User extends Model
{
    /**
     * @return UserBuilder<User>
     */
    public static function query(): UserBuilder
    {
        return parent::query();
    }

    /**
     * @param \Illuminate\Database\Query\Builder $query
     * @return UserBuilder<User>
     */
    public function newEloquentBuilder($query): UserBuilder
    {
        return new UserBuilder($query);
    }
}

/**
 * @template TModelClass
 * @extends Builder<User>
 */
class UserBuilder extends Builder
{
    public function public(): self
    {
        return $this->where('public', 1);
    }
}

class HomeController
{
    public function index()
    {
        $user = User::query()
            ->where('id', 1)
            ->whereHas('relation')
            ->public()
            ->first();

        return $user->not_found;
    }
}

If you want, I can also create a test repo.

If you are taking advantage of custom Eloquent Builders for your models, you have to mark your custom builder class as generic

Please consider this.
Now TModelClass defined but not used, thus the builder class is not generic.

Mh.. I'm not sure what you mean by 'thus the builder class is not generic.'.
I've added @template TModelClass of User to my UserBuilder (as in these tests https://github.com/nunomaduro/larastan/blob/master/tests/Features/ReturnTypes/CustomEloquentBuilderTest.php#L146), but that does not change anything.

Does it look like this now?
/**
 * @template TModelClass of User
 * @extends Builder<TModelClass>
 */
class UserBuilder extends Builder

Yes, same error happening.

I think this might be because of a faulty stub. Can you change here to just static locally (in your vendor folder) and try again?

This fixed my issue! Should I update the PR or will you create a patch?

Yes please update the PR. And I think we just need the testModelWithCustomBuilderReturnsCustomEloquentBuilderAfterCustomBuilderMethodRelationChainedWithExplicitQueryMethod test. You can delete others.


Function isPositive returns true for 0

IMO, this is either a bug, or a very misleading function-naming.
Here is a simple way to reproduce:
let x = new BigNumber(0);
console.log(x.isPositive());

According to the documentation of the function, it is indeed a bug:

Returns true if the value of this BigNumber is positive, otherwise returns false.

That is, unless you change the word value to sign.
But even then, the function name will remain extremely misleading (I just spent a very long time debugging my code until realizing that this seemingly-simple function was returning the opposite of what I was expecting it to).
The same problem may occur with function isNegative returning true for a zero value.
I suggest to keep the documentation as is, and update the implementation of these two functions to verify that the number is not zero.
For example:
    P.isPositive = function () {
      return this.s > 0 && !this.isZero();
    };

Thanks

Changed the word "value" to "sign" in v8.0.0.

Hi there. First of all, thanks for all the effort that went into this package.
I understand that you can have a bignum 0 with a positive or negative sign.
Still, I just wanted to pitch in here to say that (new BigNumber(0)).isPositive() returning true was a very unexpected behavior for me. Although it is working as documented, anywhere I look I would expect x.isPositive() to behave like x > 0.
Not saying you should change how the function works (it's probably too late for that), do what you feel is best.


Update Buildkite agent queue

Update Buildkite queue from benchmark_large (obsolete and gone) to just benchmark

This can be merged.


append ".cmd" to command arg0 if it fails on Windows

Problem
Commonly on Windows, some commands require the .cmd suffix. This is often missed, leading to issues like https://github.com/neovim/nvim-lspconfig/issues/2032 , and the solution is copy-pasted into configs: https://github.com/neovim/nvim-lspconfig/pull/2039
Proposal
@williamboman https://github.com/neovim/nvim-lspconfig/issues/2032#issuecomment-1200413852 :

exepath() seems to solve this pretty nicely, as Neovim does respect PATHEXT. null-ls uses it and mason-lspconfig.nvim successfully patches cmds with it. I'd be happy to open a PR that patches commands with exepath() on Windows + normalizes all commands in server configs.

Or is there a reason why we don't use vim.fn.system() (which should do this automatically) instead of uv_spawn (vim.loop.spawn))? (Probably the reason is: that doesn't work in Lua "fast" callbacks)

I don't check the system c code but can we do it like system{fast=true}  a little change ?

possibly.

Probably the reason is: that doesn't work in Lua "fast" callbacks

yup it not work in cb unless we set {fast=true} .

What is the current state？
On windows, mason download LSP is completely unavailable at all.

Or is there a reason why we don't use vim.fn.system() (which should do this automatically) instead of uv_spawn (vim.loop.spawn))?

the reason is explained in the original change, see https://github.com/neovim/neovim/pull/16430

Previously, the built-in language server client checked if the first
argument of cmd was executable via vim.fn.executable. This ignores PATH
injected via cmd_env. Instead, we now start the client via uv.spawn, and
handle the failure mode, reporting the error back to the user.

I'm not sure there's an easy way to handle this, whilst respecting cmd_env, outside of writing some is_executable() wrapper using libuv, e.g. https://github.com/npm/node-which

What is the current state？
On windows, mason download LSP is completely unavailable at all.

I'd recommend using https://github.com/williamboman/mason-lspconfig.nvim if you're on Windows. It patches the commands for you automatically.

This would be such a terrific addition! I just tore my hair out with this error! An lsp was erroring out because the language server command did not exist, I make sure to add it to the path, and still no dice. Once I added the .cmd to the end of the lsp start command it magically worked. Thank you so much for noticing! It makes me incredibly happy that I was not the only one with this issue.


[REST API] pipeline visibility field?

Is there a REST equivalent to the GraphQL visibility field?

Hey @chrisdothtml, there is now! Thanks for highlighting its absence, we've added it to the REST API today. I'll leave this issue open until I get it added in to the docs too 👍🏻


AppController fix warning on application relocate to privileged port

Add CAP_NET_BIND_SERVICE capability for appscale-controller.service to address warning on relocate to privileged port:
Nov  4 22:03:58 3c3b3556cba0  W, [58.467811#13609]  WARN -- : Shell commmand /usr/sbin/nginx -t -c /etc/nginx/nginx.conf failed with error output nginx: the configuration file /etc/nginx/nginx.conf syntax is ok
Nov  4 22:03:58 3c3b3556cba0  nginx: [emerg] bind() to <IP_ADDRESS>:80 failed (13: Permission denied)
Nov  4 22:03:58 3c3b3556cba0  nginx: configuration file /etc/nginx/nginx.conf test failed
Nov  4 22:03:58 3c3b3556cba0  I, [58.468010#13609]  INFO -- : Reloading nginx service.

Daily build no. 6980

Demo is that the error does not occur with the fix in place:
root@euca-10-10-9-41:~# appscale deploy guestbook
Tarring application
Copying over application
Deploying service default for guestbook
Please wait for your app to start serving.
Your app can be reached at the following URL: http://<IP_ADDRESS>:8080
Updating indexes
root@euca-10-10-9-41:~# appscale relocate guestbook 80 443
Successfully issued request to move guestbook to ports 80 and 443
Your app serves unencrypted traffic at: http://<IP_ADDRESS>:80
Your app serves encrypted traffic at: https://<IP_ADDRESS>:443
root@euca-10-10-9-41:~# grep nginx.conf /var/log/appscale/*
grep: /var/log/appscale/celery_workers: Is a directory
root@euca-10-10-9-41:~#

without fix:
root@euca-10-10-4-84:~# appscale deploy guestbook
Tarring application
Copying over application
Deploying service default for guestbook
Please wait for your app to start serving.
Your app can be reached at the following URL: http://<IP_ADDRESS>:8080
Updating indexes
root@euca-10-10-4-84:~# appscale relocate guestbook 80 443
Successfully issued request to move guestbook to ports 80 and 443
Your app serves unencrypted traffic at: http://<IP_ADDRESS>:80
Your app serves encrypted traffic at: https://<IP_ADDRESS>:443
root@euca-10-10-4-84:~# grep nginx.conf /var/log/appscale/*
grep: /var/log/appscale/celery_workers: Is a directory
/var/log/appscale/controller.log:Jan 14 21:16:02 euca-10-10-4-84  W, [02.738023#2449]  WARN -- : Shell commmand /usr/sbin/nginx -t -c /etc/nginx/nginx.conf failed with error output nginx: the configuration file /etc/nginx/nginx.conf syntax is ok
/var/log/appscale/controller.log:Jan 14 21:16:02 euca-10-10-4-84  nginx: configuration file /etc/nginx/nginx.conf test failed
root@euca-10-10-4-84:~#


Image selection when adding module

User Story
AS AN admin
I WANT TO pick from a list of thumbnail image available
SO THAT I can see which image I am adding to a module (and not enter imageX.jpg
Acceptance Criteria

User can only accept one image at a time
User must select one image (not zero or >1)

This ticket refers to the "Image" section in the attached screenshot


New parameter --since go apply on git log command

With this new parameter will be possible to filter since which date the heatmap will be analysed.

this would be great! I am just not sure if @jez still maintains this project.


[BEAM-4461] Add Unnest transform.

run java precommit

@akedin The problem seems to be that SQL tries to add the same field twice to a schema when joining. I can remove this check, but it seems to me like a bug in SQL.

I am not sure what is the correct thing to do here. I think that we should prevent users from explicitly adding duplicate fields (as far as I remember BQ SQL disallows duplicate fields, don't know about standard). But I don't know if this can be a valid thing to do internally. For example, when implementing a join there potentially can exist an intermediary joined schema that combines left and right side of the join. Ambiguities should be dealt with by projection which is applied after that. Does this sound like something that can happen?

@akedin this was caused by a bug in SQL (it wasn't using the Calcite-generated names, which are disambiguated). this is now fixed.


ERROR TypeError: window.showOpenFilePicker is not a function appears in Firefox console

Problem description
The issue happens when a user imports policy using Firefox browser
Step to reproduce
Steps to reproduce the behavior:

Click on the 'Import policy'
Click on 'Import from file'

Expected behavior
A user should be able to upload policy using local files
Actual behavior
In the browser console I see the error message ERROR TypeError: window.showOpenFilePicker is not a function
Screenshots
If applicable, add screenshots to help explain your problem.

To solve this issue, we have finalised to implement drag and drop functionality for importing a File

This ticket is QA passed and closed on April 26th


Not able to run ios app locally

Action Performed:
Tried to run ios app locally (on main -> 318ffa0b6ae94b44f3337e6a8940acd9dc8ed013)
Expected Result:
It should run successfully
Actual Result:

Platform:
Where is this issue occurring?
Web
iOS ✅
Android
Desktop App
Mobile Web
Version Number:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
View all open jobs on Upwork

Triggered auto assignment to @zanyrenney (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Lol Did you do npm install? then pod install in the Ios folder.

Lol Did you do npm install? then pod install in the Ios folder.

🤦🏼 Of course

then clear cache on metro server ans uninstall the app. then try

I'm able to run the app fine on iOS on main, so this is likely a local issue, going to close this. If you are still having issues after trying to clear the cache (npm run start -- --reset-cache), please ask in our #expensify-open-source slack channel.


feat(s3): Front50 Storage: Support Server Side Encryption in S3 headers

Supports the S3 header x-amz-server-side-encryption-context, with a value of either aws:kms or AES256.

@ethanfrogers modified for enum ;)


[Feature Request]: Enable geo-replication for Redis cache.

Description
A Redis cache supports the setup of geo-replication via bicep through the use of the linked server bicep resource: https://learn.microsoft.com/en-us/azure/templates/microsoft.cache/redis/linkedservers?pivots=deployment-language-bicep
Adding this to the bicep module for deployment of a Redis cache would be a welcome addition.
However the usage of the linked server and a private endpoint proves to be a bit difficult as you cannot add a private link to a Redis cache that is geo-replicated. You first need to remove the geo-replication, then add the link and then re-enable the geo-replication. As documented here: https://learn.microsoft.com/en-us/azure/azure-cache-for-redis/cache-how-to-geo-replication
Specifically:

Private links can't be added to caches that are already geo-replicated. To add a private link to a geo-replicated cache: 1. Unlink the geo-replication. 2. Add a Private Link. 3. Last, relink the geo-replication.

I've tried to provide some kind of start of an implementation here: https://github.com/Azure/ResourceModules/compare/main...Meertman:ResourceModules:main
But it is far from complete.


Use iov_base to avoid problems with mdbx::byte type aliasing

Add path for included CMake files to fix build as submodule
Fixes #292

Addresses #292


Update howto-cluster-manager.md

Added a Warning box before deleting a Cluster Manager.

@jaredr80

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Update README.md

Hi Admin,
I noticed a small mistake in the README.md file under the section "Clone the Forked Repository" that could confuse new visitors or contributors. Even though it’s a small mistake, it can potentially lead to issues for beginners trying to clone the repository. The current description states:
"git clone git clone https://github.com/<'your-username'>/blockchain.git"
This includes an extra git clone command, which is incorrect. The correct command should be:
"git clone https://github.com/<'your-username'>/blockchain.git"
I have made this correction in my pull request. Please accept my pull request, and I hope this change will help others in cloning the repository without confusion.
Thank you!
Before:

After:

@Harish-2003 check it


🛠 [2.2] fix: Fixed Issue #10347

Issue Reference
#10347
Description
Update customer address add section in drawer.

conflicts


Need to hide search option from dropdown

Hello,
This plugin is very useful. But i want to hide that search option which is in that drop down?
is there any option to hide that search options input?
i tried with display none but it wont open drop down then
Thanks in advance.

Afghanistan
Albania
Algeria
Andorra
Angola


[UI] Footer Content Alignment Enhancement in KubeFirst Documentation

What is your feature idea?
The idea is to align all the content in the footer of KubeFirst documentation in a manner similar to the KubeFirst website.
Please find below an image depicting the footer that requires enhancement:

Why is it needed?
To enhance user experience and maintain visual consistency, making the documentation more visually appealing.
I am willing to work on this issue if it is considered a valid improvement.
Is this missing feature preventing you from using kubefirst?

[ ] Yes

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@ATREAY: thanks for reporting. I never thought about it, but you're right, the size of the cloud boxes should be the same, it will look better :)

Sure thing @fharper, should I go ahead and get started on it?

@ATREAY: yes, in that case, all cloud boxes should be the same size, not just the bottom 3.

Hey @fharper, I've got a quick question - do we keep the content inside the cloud box as it is, or should we adjust it to fit the box size?

We keep the content as it is, so adjust the box size to fit the content.

I just realized there's a different repo for the docs, but I raised the issue here. Is it acceptable if I connect this issue to that repo when I submit the PR?

I'll move the issue to the right repository, I didn't realize it was in the wrong one, no worries.


CI fix: TypeScript errors should fail builds

It's possible to have TypeScript errors, but tsup would still build successfully. Most TypeScript errors are caught by tsup, but if there are errors in places that don't end up in the build output — i.e. when you only have TypeScript errors in test files, then this could still pass. This makes out build commands resilient against these cases.

Actually, nevermind. This will make the builds slower (which can add up when you need to run 3 or more nested Turbo builds). Besides, Jest will catch those kinds of TypeScript issues for us after all, just in a different CI phase.


Cache labels.Labels to Identify Series in Remote Write

This is an experiment to have the remote write component use labels.Labels instead of []prompb.Label when passing samples around/identifying series. A label.Label uses less space so steady state memory usage should be less, with the tradeoff of transforming label.Labels to []prompb.Label for each remote request.
Results after removing some extra sources of allocations look very good. This is for a server with three remote-write targets, and ~8k samples per second ingestion.
Benchmarks
benchmark                     old ns/op     new ns/op     delta
BenchmarkSampleDelivery-8     998914        1007751       +0.88%

benchmark                     old allocs     new allocs     delta
BenchmarkSampleDelivery-8     7117           6117           -14.05%

benchmark                     old bytes     new bytes     delta
BenchmarkSampleDelivery-8     688391        640400        -6.97%

Heap profiles
Heap profiles show a 50% reduction in memory used by StoreSeries as expected.
v2.11.1:

rw-use-labels branch

Other metrics
CPU usage is very similar between the servers. However, there is a reduction in allocations:

That's looking okay. What do the benchmarks say now?

I removed the merge conflict, and updated the benchmarks, slightly fewer allocations than last time, CPU jumps around between runs but is about the same before/after.
I don't expect the graphs to change too much, but can repost them in a few hours if interested.

@brian-brazil would you mind taking another look when you have a moment?

:+1:


configurator: simplify MeshConfig API

Description:
Simplifies the signature of GetMeshConfig() in
the interface to return an object instead of a pointer.
MeshConfig is guaranteed to exist during controller
startup and there is already a fallback to return
a default MeshConfig object if not found in the
cache. Returning a pointer results in unnecessary
nil pointer handling on the caller side, which
makes the usage of this function indeterministic
when the implementation is deterministic.
This change guarantees to the caller of this function
that they will always receive a valid MeshConfig to
operate on.

Affected area:

Functional Area

Control Plane
[X]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Did you notify the maintainers and provide attribution?

Is this a breaking change? no

Codecov Report

Merging #4475 (6fb50a3) into main (e45d28a) will increase coverage by 0.01%.
The diff coverage is 69.23%.

@@            Coverage Diff             @@
##             main    #4475      +/-   ##
==========================================
+ Coverage   69.14%   69.15%   +0.01%
==========================================
  Files         212      212
  Lines       14595    14591       -4
==========================================
- Hits        10091    10090       -1
+ Misses       4452     4449       -3
  Partials       52       52

Flag
Coverage Δ

unittests
69.15% <69.23%> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/configurator/mock_client_generated.go
0.00% <0.00%> (ø)

pkg/configurator/methods.go
73.38% <75.00%> (ø)

pkg/configurator/client.go
83.33% <83.33%> (ø)

pkg/ingress/gateway.go
78.90% <100.00%> (+0.87%)
:arrow_up:

pkg/certificate/rotor/rotor.go
86.95% <0.00%> (+2.17%)
:arrow_up:

Continue to review full report at Codecov.

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


Write Tests for Django Models

Describe the task
This task is to write tests for the existing Django models that include:

all columns exist
create record??
suggest pytest

Acceptance Criteria

[ ] first
[ ] second
[ ] third

Additional context

Add any other context about the task here.
Or here

Might be more convenient to use Django specific plugin for this: https://pytest-django.readthedocs.io/en/latest/index.html#


[eas-cli] merge everything from actions/new to actions

Checklist

[ ] I've added an entry to CHANGELOG.md if necessary.

Why
I moved everything from ios/actions/new to ios/actions, and got rid of all the old REST api actions that we no longer need. Some of the REST actions were named the same thing as the Graphql actions, which is why github shows you a substantial diff instead of just a file deletion
Test Plan

[ ] current tests pass

yup, i fixed it in the stacked PR on top of this one, so i'll land both of them :)


DateTime comparison in linq expression produces npgsql error

We are using the following code to filter rules with datetime values:
rules =                         await                             session.Query<Rule>()                                 .Where(                                     rule =>                                         rule.ModelType == modelType &&                                         rule.RuleType == ruleType &&                                         (rule.BeginDate <= now && now <= rule.EndDate ||                                         rule.BeginDate <= now && rule.EndDate == null))                                 .ToListAsync();
However, this produces the following result:

"Error: Marten Command Failure:\nselect d.data, d.id, d.mt_version from tenant.mt_doc_rule as d where ((((d.data ->> 'ModelType' = :arg0) and (CAST(d.data ->> 'RuleType' as integer) = :arg1)) and (tenant.mt_immutable_timestamp(d.data ->> 'BeginDate') <= :arg2)) and (tenant.mt_immutable_timestamp(d.data ->> 'EndDate') <= :arg3)) or ((tenant.mt_immutable_timestamp(d.data ->> 'BeginDate') <= :arg4) and ((tenant.mt_immutable_timestamp(d.data ->> 'EndDate')) is null))\n\n42883: function tenant.mt_immutable_timestamp(text) does not exist"

@shanomacadaemia What's your AutoCreateSchemaObjects setting? Are you letting Marten do things automatically, or trying to control schema creation yourself? The missing function there is something that Marten should have built out for you.

That was exactly the issue, I discovered it a few moments ago. It was due to us switching the schema after creation again (errant code that should have been removed). Apologies.


Azure Kubernetes Service extension can't connect to AKS clusters with same name but are in different resource groups

Describe the bug
Have 2 clusters with the same name in the same subscription but in different resource groups
This works in version 1.3.18 and earlier version of Azure Kubernetes Service Extension, so this was broken in 1.4.0 version
e.g.
Subscription: ACS-MGMT-DEV01
cluster 1: mobileservice
resource group: eu1d-mobileservice-aks
cluster 2: mobileservice
resource group: eu1t-mobileservice-aks
Note that the tool does not connect successfully to the second cluster and shows "invalid"
Mention the platform you are using
vscode
Version: 1.85.1 (Universal)
Commit: 0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2
Date: 2023-12-13T09:48:06.308Z
Electron: 25.9.7
ElectronBuildId: 25551756
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.2.0
To Reproduce
Steps to reproduce the behavior:

Go to 'Kubernetes' CLOUDS/Azure section in vscode
Expand to  an Azure subscription that has multiples AKS with same name but are in different resource groups
Results in the cluster name 'invalid' for all but the first AKS cluster.

Expected behavior
Should connect to both clusters.
Screenshots

Desktop (please complete the following information):
  Model Name:	MacBook Pro
  Model Identifier:	Mac14,5
  Model Number:	MPHG3LL/A
  Chip:	Apple M2 Max
  Total Number of Cores:	12 (8 performance and 4 efficiency)
  Memory:	32 GB
  System Firmware Version:	10151.61.4
  OS Loader Version:	10151.61.4

Thank you so much @SOFSPEEL for initiating this. ☕❤  I can readily replicate the steps provided, and I am exploring potential solutions. We will carefully examine the issue and strive to devise the best possible solution to address it effortlessly. Thank you for sharing; I'll be in touch soon.
Gentle fyi & cc to some peers: @peterbom, @hsubramanianaks, @sprab & @qpetraroia for eyes and fyi. (I might have solution for this already and will open PR by tomorrow for your eyes) 🤹♀🙏

Hiya @SOFSPEEL, looping back to inform that the new released version for aks extension1.4.1 contains fix for this. Thanks heaps.


Better first person mesh culling

When a first person mesh annotation is set to Auto, the VRM spec says to exclude individual vertices from the first person layer if they are weighted to the head bone. Currently we exclude the entire primitive. I started work on the more correct version https://github.com/unavi-xyz/bevy_vrm/tree/first-person-mesh, which requires creating a duplicate primitive with the removed vertices, but ran into wgpu panics from bevy.

fixed was dumb


coraza-proxy-wasm minimum requirements

Hello,
Congrats for the hard work :)
I'm wondering what are the minimum requirements to run coraza-proxy-wasm with --log-level info and this kind of (following) configuration ?
  "@type": "type.googleapis.com/google.protobuf.StringValue"
  value: |
    {
      "rules": [
        "Include @demo-conf",
        "Include @crs-setup-demo-conf",
        "SecDebugLogLevel 9",
        "Include @owasp_crs/*.conf",
        "SecAction \"id:900000,phase:1,nolog,pass,t:none,setvar:tx.blocking_paranoia_level=4\""
      ]
    }

On my kubernetes cluster, envoy pod (with coraza-proxy-wasm) can't start with resources requests less than cpu/3000m and memory/3072Mi and it seems to be a strict minimum.
The same envoy configuration without coraza-proxy-wasm can start with resources requests equal to cpu/100m and memory/256Mi
It could be useful if you could provide some metrics.
Thx

Hi @mlesaout thanks a lot for coming by. I think there are a couple of reasons for such behaviour.

We load the entire ruleset in memory (in the rootFS) as we have no access to filesystem
You are loading the entire CRS which is what people do by default but I guess people might refine the rules.
All the rules loaded compile regexes which might or might not be used. One way to mitigate this would be to lazy compilation of the regexes (which bring a different set of problems in runtime).
Just out of curiosity: how would this change if the only rule would be SecRuleEngine ON?

Thanks a lot
cc @anuraaga

Thanks @mlesaout - we still need to do more measuring in various environments to get a confident set of minimum requirements. Performance testing so far has been mostly just running on local laptops and seeing how much is used.
For context on my Mac laptop, Envoy is around ~20MB without wasm and ~200MB with it with two worker threads - unfortunately with wasm there is a full memory overhead per worker thread with no real alternative. I think Envoy will automatically pick worker threads based on available CPU so by adding more CPU, it also itself requires more memory for wasm in each thread, so perhaps sticking to cpu/100m and raising the memory just a little bit more, maybe 350mb or so, could work? Also just to confirm, you mention Envoy can't start, does that mean that the pod fails without serving any requests?

Thanks @jcchavezs @anuraaga
I was loading the entire ruleset as a quick test to evaluate coraza-proxy-wasm. Setting SecRuleEngine On doesn't change nothing. Pod failed to start without serving any request.
But ... as @anuraaga suggested, I missed --concurrency parameter which seems to solve the problem. I just need to tune it now
According to envoy documentation (https://www.envoyproxy.io/docs/envoy/latest/operations/cli#cmdoption-concurrency)

If not specified defaults to the number of hardware threads on the machine
It could be useful to document it or add the parameter in example files : https://github.com/corazawaf/coraza-proxy-wasm/blob/e5c8ad0a6e776b66998270ff861da846e9e8bbec/example/docker-compose.yml#L23

Thanks

Do you feel like starting a section around requirements for k8s I'm the
readme?
On Thu, 9 Feb 2023, 10:28 mlesaout, @.***> wrote:

Thanks @jcchavezs https://github.com/jcchavezs @anuraaga
https://github.com/anuraaga
I was loading the entire ruleset as a quick test to evaluate
coraza-proxy-wasm. Setting SecRuleEngine On doesn't change nothing. Pod
failed to start without serving any request.
But ... as @anuraaga https://github.com/anuraaga suggested, I missed
--concurrency parameter which seems to solve the problem. I just need to
tune it now
According to envoy documentation (
https://www.envoyproxy.io/docs/envoy/latest/operations/cli#cmdoption-concurrency
)
If not specified defaults to the number of hardware threads on the machine
It could be useful to document it or add the parameter in example files :
https://github.com/corazawaf/coraza-proxy-wasm/blob/e5c8ad0a6e776b66998270ff861da846e9e8bbec/example/docker-compose.yml#L23
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/corazawaf/coraza-proxy-wasm/issues/148#issuecomment-1423883030,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYAT6WOGPEBMJIWORT7DWWS2EXANCNFSM6AAAAAAUVRFGPQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'll try :) I have to understand a bit more how it works first. But It looks promising.


Fixed #25892 -- Extracted inner loop in SeparateDatabaseAndState.database_backwards().

Now intermediate states in the database_backwards are cached, similar to
the executor's migrate (or _migrate_all_backwards). This makes the
migration run much faster (O(n) instead of O(n^2) over number of
database_operations).
I didn't put anything in release notes, because this is not a bug-fix. Is this correct?

@MarkusH, any concerns here? I guess a test isn't really feasible?

This is the exact method addressed by #25896, where I also added a test to cover it. I only noticed that bug when I set to optimize this, and thought that it would be cleaner to submit a simple bugfix before optimizing.

Sometimes it's possible and worthwhile to add tests for performance improvements to ensure that performance doesn't degrade as a result of future changes, that's all I was asking.

I'm up for writing such a test, if you think it's relevant. With my project the difference was clearly noticable, several seconds instead of a couple of minutes. Where can I find a sample of such a test?

I can't give a recipe for how to write one here. An example in a different context would be assertNumQueries.

Looks good to me. No idea right now how to test the complexity tbh. I might be able to figure something out next year.

Just don't write a test based on time.time() or equivalent, those are brittle even when huge margins are factored in.

merged in bd3c2900fc0e6863924d3b27f0713aa3022c061d, thanks! Feel free to follow up with a test if you think of one.


Why the videos i record always are horizontal

All the videos I record on the vertical screen are horizontal. why?

mVideoSize = CameraUtil.chooseVideoSize(map.getOutputSizes(MediaRecorder.class));
mMediaRecorder.setVideoSize(mVideoSize.getWidth(), mVideoSize.getHeight());
mVideoSize.getWidth() > mVideoSize.getHeight() even when my screen are vertical ;


Add support for Serial Wire Output (SWO) viewer (ITM instrumentation trace)

Currently the texane/stlink project doesn't support SWO. But it could be implemented of course.

You could use OpenOCD for that with the following settings:
itm port 0 on
tpiu config internal <logfile> uart off <cpu_freq_herz>

For some other readers: https://mcuoneclipse.com/2016/10/17/tutorial-using-single-wire-output-swo-with-arm-cortex-m-and-eclipse/

You could use OpenOCD for that with the following settings:
itm port 0 on
tpiu config internal <logfile> uart off <cpu_freq_herz>

For some other readers: https://mcuoneclipse.com/2016/10/17/tutorial-using-single-wire-output-swo-with-arm-cortex-m-and-eclipse/

thank you very much!


Horizontal BootstrapButtonGroup Width and Height problem with different Text Length

How i can set equal height and width for each button i have 3 button with different text length ?

https://developer.android.com/guide/topics/ui/layout/linear.html


remove magic values

There are still a few hardcoded values in the code. It works so it's not a big deal but better generate them properly (example : serverid)

does protodef support default values?

what do you mean by that ?

I'm not talking about the "magic" value of raknet here.

ah, I misunderstood, sorry.

remaining here:

[ ] client/server id
[ ] ping number

done


32k-cutoff of image data while reading from ASE 15.7/16

I have a strange behavior when reading an image field from DB with your client.
What I have found out:
AseDataReader.GetValue delivers a byte array with a lenght of max 32768 bytes.
AseDataReader.GetBytes call internaly GetValue and deliver the same 32768 bytes.
But the size of the image in the database is 148655 bytes.
I think that the underlying RowResult object holds the wrong amount of bytes in the CurrentRow instance.
It is possible that this could a bug?
Environment

.NET Standard 2.0.3
AdoNetCore.AseClient nuget package version 0.19.1
Sybase ASE 15.7 & 16.0

Best regards
Alex

seems like standard sybase behavior. For every query prefix your sql with SET TEXTSIZE X
where X is max size you expect from the data. I always use 2000000 which is more or less max size possible


NPM warnings upon install

> $ npm i                                                                                ⬡ 5.5.0 [±master ✓]
npm WARN package.json Dependency 'lodash' exists in both dependencies and devDependencies, using<EMAIL_ADDRESS>from dependencies
npm WARN deprecated<EMAIL_ADDRESS>Use vinyl-fs.symlink (or gulp.symlink with gulp 4) instead.
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs version 3 and before will fail on newer node releases. Please update to<EMAIL_ADDRESS>as soon as possible.
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained. Upgrade to<EMAIL_ADDRESS>npm WARN deprecated<EMAIL_ADDRESS>graceful-fs version 3 and before will fail on newer node releases. Please update to<EMAIL_ADDRESS>as soon as possible.

Having LoDash in both dependency trees and having dependencies that use LoDash 1 particularly should be fixed.

The versions of LoDash inside each dependency tree are different as well.

This can be closed. It was fixed by #17 and released in rc4


Fix: AvatarName if name === '""' or length is empty return address

after filtering out whitespaces

When name shows nothing, the length of empty string is actually 1. Checked with String.prototype.charCodeAt() method and it returns 32 for name.charCodeAt(0). 32 means space in UTF-8 Basic Latin system. (see here: https://www.w3schools.com/charsets/ref_utf_basic_latin.asp)


[Feature] Make upnpav services disabled by default

Upnp av services should be disabled by default.
Most of the time I end up disabling those manually, so I think disabling by default is a better choice.

Available in release 2022-11-13.1


jekyll-seo-tag does not seem to be supported

 $ docker run -v "$PWD":/usr/src/app -p "4000:4000" starefossen/github-pages
Configuration file: /usr/src/app/_config.yml
Configuration file: /usr/src/app/_config.yml
            Source: /usr/src/app
       Destination: /_site
 Incremental build: enabled
      Generating...
  Liquid Exception: Liquid syntax error (line 16): Unknown tag 'seo' in /_layouts/default.html
jekyll 3.4.5 | Error:  Liquid syntax error (line 16): Unknown tag 'seo'

Yes, it should be installed already by the github-pages gem. But you would need to enable it in your _config.yml like this:
gems:
- jekyll-seo-tag

It is enabled 😕

Any ideas?

@adius could you supply your _layouts/default.html file?

@siliconmeadow Yes, but nothing interesting to see there
<!doctype html>
<html>
<head>
  <meta charset=utf-8 />
  <meta name=viewport content="width=device-width, initial-scale=1">
  <link rel=stylesheet media=all href=/css/screen.css>
  <link rel="shortcut icon" href=/img/favicon.png>
  <link
    rel=stylesheet
    href=//maxcdn.bootstrapcdn.com/font-awesome/4.3.0/css/font-awesome.min.css>
  <link
    href=/atom.xml
    rel=alternate
    title="Adrian Sieber"
    type=application/atom+xml>
  {% seo %}
</head>
<body>
…

I've never used this gem before, but looking at http://jekyll.github.io/jekyll-seo-tag/installation/ , it says that you need to add this prior to the <head> tag:
<!-- Begin Jekyll SEO tag v2.3.0 -->
<title>Installing Jekyll SEO Tag | Jekyll SEO Tag</title>
<meta property="og:title" content="Installing Jekyll SEO Tag" />
<meta property="og:locale" content="en_US" />
<meta name="description" content="A Jekyll plugin to add metadata tags for search engines and social networks to better index and display your site’s content." />
<meta property="og:description" content="A Jekyll plugin to add metadata tags for search engines and social networks to better index and display your site’s content." />
<link rel="canonical" href="http://jekyll.github.io/jekyll-seo-tag/installation/" />
<meta property="og:url" content="http://jekyll.github.io/jekyll-seo-tag/installation/" />
<meta property="og:site_name" content="Jekyll SEO Tag" />
<script type="application/ld+json">
{"name":null,"description":"A Jekyll plugin to add metadata tags for search engines and social networks to better index and display your site’s content.","author":null,"@type":"WebPage","url":"http://jekyll.github.io/jekyll-seo-tag/installation/","image":null,"publisher":null,"headline":"Installing Jekyll SEO Tag","dateModified":null,"datePublished":null,"sameAs":null,"mainEntityOfPage":null,"@context":"http://schema.org"}</script>
<!-- End Jekyll SEO tag -->

I don't know if that would help?

My mistake - just prior to the closing  - ignore my previous comment.

I'm looking through the issues, and it looks like others have an _includes/head/seo.html file which shows some other ways it appears they've defined {% seo %}.
What I'm wondering is whether the problem with the jekyll-seo-tag gem in your circumstance is not that the gem isn't installed in the docker image, but that it's not correctly invoked in your templates?

No, I don't think so. I've been using the plugin before with https://github.com/github/pages-gem and it worked like a charm, but I'm sick of dealing with ruby & bundler issues and that's why I wanted to use a docker based solution...

 I had some weirdness with jekyll gems that went away when I blew the lock file away.

Thanks a lot @smeyfroi. This wasn't exactly the problem, but it led me on the right way.
The problem was that I didn't have a Gemfile at all 🙈 . I thought it wasn't necessary, as all dependencies are already preinstalled in the docker container.
The Liquid Exception: Liquid syntax error (line 16): Unknown tag 'seo' in /_layouts/default.html error doesn't suggest that the Gemfile is missing either.
How should this be fixed? A warning if the Gemfile is missing? Or automatically adding one?

@adius Nice!
It is confusing, and not really explained very well in the jekyll docs (that I could find anyway).
So perhaps document that both a Gemfile and the _config.yml gems declaration is required? New plugins require additions to both files.
Also maybe worth explaining that bundler is invoked during container startup: I found myself reading the Dockerfile to understand what was going on.
And the problem I had was with gem dependency conflicts that went away when I deleted the Gemfile.lock. So perhaps that's worth noting in the docs as well, as a troubleshooting thing?

Great writeput @smeyfroi 👍 We should definitely add this to the project README I think.

I can take a look and raise a PR over the next couple days.


Use maven-compiler-plugin Java version when compiling Java templates via jte-maven-plugin precompile goal

I've added jte-maven-plugin, specified precompile goal, got .class files, uploaded them to web server, tried opening the web page and got error. Checking web server logs revealed that jte-maven-plugin uses default Java version when compiling JTE templates.
I use Java 16. In my pom.xml I have
<java.version>15</java.version>
<maven.compiler.source>${java.version}</maven.compiler.source>
<maven.compiler.target>${java.version}</maven.compiler.target>

Java 15 is installed on the web server, so I get unsupported class file major version 60

Hey @izogfif, thanks for reporting! I think I've pushed a fix for this https://github.com/casid/jte/commit/9fa706b21ad8d81ffcecd66ebae4f42b23b7d10e.
Could you give it a try if this works on your end?  For this you'd need to run against the latest snapshot release 1.11.1-SNAPSHOT:

clone this repository
run mvn clean install
use 1.11.1-SNAPSHOT as jte version

Not sure, but maybe maven.compiler.release property support should be added as well: https://www.baeldung.com/maven-java-version#java9

@izogfif thanks for checking it out! You are right, we should definitely support the new --release option, too. I pushed a new commit with this change (https://github.com/casid/jte/commit/c4d46b6489aeef335faa0e501c8bc31ec74b8e47). In case the release option is set, I only pass that and ignore source/target (https://docs.oracle.com/en/java/javase/14/docs/specs/man/javac.html#option-release).

@casid checked, maven.compiler.release variable works.

Thanks for checking!
I'll soon publish a 1.11.1 bugfix release to central and let you know when it's available.

Just released 1.11.1 to central.

@casid Could you also release jte-jsp-converter there? It's not really convenient to fork the repository / install this module manually on local machine.

Good idea! I added the jte-jsp-converter module to the parent pom, so that it will be automatically deployed with each central release in the future.
For the current version I've manually pushed it to central, it should be available within the next hour.


CASSANDRA-19032 Provide truncate task for SAI

If a table is truncated during the initial build of an SAI index, there is a chance that the index build will get interrupted, and it won't get marked queryable. This patch provides a truncate task for SAI that marks the index queryable during truncation.

Committed https://github.com/apache/cassandra/commit/3928f2992f94aa2b4e56bee0f36a0bdb31087116


When using freelook mod while bridging, blocks are placed where the cameras are facing instead of whre the player's head is looking at.

While bridging and using freelook, i noticed that the blocks are placing at where the camera is looking at instead of where the player's head is looking at.
https://files.catbox.moe/buxxzo.mp4
The idea of freelook is that you are able to check around your surroundings without moving your head or changing whatever direction you're facing.
Expected behavior: blocks are being placed at where the player's head is looking at, not wherever the camera is pointing.

It's intentional (issue #15) but I'll add a toggle for it under the fixes tab.
I'll also see if I can automatically change it when freelook is installed.


Transform Order Backwards

To make a diamond shape, I would expect the following to work.
transform: [
  { rotate: '45deg' },
  { scaleY: 0.5 },
],

However, this does not work. It seems to be doing the transforms in the reverse order, as reversing it achieves the desired result.

Hey,
I just did a bit of research, the problem seems to be that the renderer loops over the transforms in the wrong direction. If .forEach doesn't loop in the wrong direction in Safari I would guess they are in the wrong order before this. My question would be if this problem occurs in production builds. If not I would go for the Object.freeze to change the order somehow.

I think the issue looks to be here. I believe MatrixMath.multiplyInto(out, a, b) is performing B*A rather than A*B (important here). In the case where we’re going forward in the loop (we are), we want to do A*B.
I can’t quite tell whether this is the intended behaviour in MatrixMath, or is a bug. It either wants fixing in MatrixMath, or the call-site I pasted. Or maybe it doesn’t matter, as it’s only used once: in that call site!
Happy to submit a PR if we’re clear on direction.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/transform-order-backwards
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub.
Also, if this issue is a bug, please consider sending a PR with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.

@facebook-github-bot label Icebox

@facebook-github-bot close


Some errors in the dutch language

Hi CoZ,
I've been using neoscan for a long time and always saw some errors in the dutch language.
Today, after checking out your Github I saw I was able to contribute to it.
Sebastiaan

@spasma Thanks for your contribution, it will be included in the next deploy


Finish "Tips" section in API.md

The tips in the API docs aren't finished:  https://github.com/mikolalysenko/regl/blob/gh-pages/API.md#tips

Should add some notes on how to deal with black screens, like remember to clear the depth buffer!


Use localStorage instead of cookies to save settings

Using localStorage instead of cookies to save settings avoids unnecessarily sending said cookies to the web server with every request.
Also, localStorage doesn't expire (by itself at least), while cookies do, which doesn't really make sense for persisting local/client-side settings.

I tested this change with Firefox 89.0.1 and it works. Just quickly checked a Chromium-based browser as well, seems to work just fine.

One (minor) negative side effect of this change is that a user will lose their settings once after the plugin is updated from a version that uses cookies to a version that uses localStorage.
I could add code that migrates potentially existing cookies to localStorage but I'm not sure if it's really worth it considering the user would just have to set a few settings once more. What do you think?

I agree, i don't think an automatic migration is needed :)
I'll probably even merge that right into the next release!
Thanks again 👍


Working improved version inspired by Canva editor

Hi,
I fixed the bugs and improved a lot the UX. I was inspired by the Canva editor.
I hope this PR will be accepted and published on pub.dev soon because I need this package in a FlutterFlow project.
Bug fixes:

the scale operation is now working with any angle
the rotation operation starts from the current angle
JSON saving/loading is working correctly

Improvements:

the scale operation keeps the current aspect ratio
in order to change the aspect ratio, I added handles on the edges (for image items, the fit is set to cover by default so they can be cropped)
when the item is too small, some handles are hidden and a drag handle appears underneath
when an item is updated (any operation), it is moved on top of all other items
the edit button is replaced by a double tap
the delete button is visible only when status is editing
more CaseStyle parameters for the frame
in the example, I added a generate button that generate items from the clipboard string

Thank you Flutter Candies for all your open source contributions!

Incredible to come across this branch. Very impressive improvement work, and it interests me greatly. The pull request should be accepted and pushed to pub.dev. Or have you already pushed it via an intermediary method? In any case, excellent work, really!

thank you @thomasllds!
no i didn't published the changes anywhere, i worked on other stuffs since then.
i hope we'll get a response because otherwise i will have to create another package in order to publish them.
the main feature that i'd like to add next is image cropping

Thank you, I see that your goal was to include this in FlutterFlow. I have also done that with version 0.1.2. However, I would really like to see your new features in a new release of this package. I don't know how to contact FlutterCandies to get them to update this package with these modifications on Pub.dev. Otherwise, how can I support you so that you can publish this package yourself quickly? Thanks    And for the image cropping, that would be amazing ;)

It seems that fluttercandies are still active. They updated a package 14 days ago. You could try to send an email to<EMAIL_ADDRESS>Otherwise publishing a package on pub.dev doesn't seem complicated. It's just that it would be confusing for everyone, but it can be a temporary solution.

0.2.0 Released
I got the email, and I apologize for the late reply, I've been busy at work lately!
Thank you for your contribution!

@xSILENCEx
It's not a problem regarding the delay, and thank you very much for this update ;)
I completely understand, work can be very demanding.
🤜🤛


Create multi-module Maven archetype

Create a maven archetype for creating new multi-module Vaadin + Spring Boot projects, with a custom theme, a custom widgetset, a module for client side code and an overlay for production mode deployments.

@peholmst Would it be as simple as to modify the original Vaadin maven-archtype which currently creates a 3 module setup. Same maven-archtype which is used within Netbeans/Eclipse ?

{project}-production
{project}-ui
{project}-widgetset

And only changes some basic code to implement Vaadin4Spring ?

To be honest, I've never used the original Vaadin maven archetype, but that sounds about right.

Chiming in out of nowhere, I agree the Maven archetype setup is not something we use either.  Our interest in this project is to tie this into Spring Cloud and use Vaadin as a micro-services UI foundation.  A multi module setup seems like added complexity.
Disclaimer: Very very new to Vaadin.  Could be a reason I am not aware of at the moment for the multi module setup.

Multi module setup is useful if you want to separate your server side code from your client side (GWT) code, or if you are building a modular application where you want to be able to deploy different versions containing different modules, or if you are deploying your application as a WAR and want to use different overlays for development, testing, production, etc. If you do not fall into one of these categories, a single module project is enough. :-)

The current multi module setup which is created by eclipse for example is very handy, especially for project like @ccit-spence is talking about. You still have a single project where you write and build you application but when you want to create your production setup you run the production module with profile and It will compress all javascript and more handy production stuff.
@ccit-spence check out the default maven Vaadin eclipse project setup. I believe it is the latest Vaadin eclipse plugin.

@GJRTimmer That does make sense.  Not fully used to the Vaadin build cycle. Was thinking in terms of Spring Boot's needs not factoring in Vaadin's needs.

@ccit-spence I tried it from a spring-boot point of view, but I found it quite hard. While starting with the Vaadin setup and adding spring-boot into it, was a lot faster, because then you don't have to think about a the Vaadin plugin configuration, to have your build automatically build the widgetset and such.

Vaadin is working on archetypes, no need to do them in the context of Vaadin4Spring.


Add Rate Limiting

@nathanclevenger  How should this be implemented? My first thought is to record every transaction to be counted and clear them out if the previous transaction was last month. With that, we also need to be able to determine the user's access plan to check the number of requests against the limit. Should that come from another service?

I'm thinking of keeping a counter which is year-month and so we always increment, never clear ... and we can see historical trends ... it's basically a counter by month ... separately we could use a counter by minute (maybe just a rounded ticks to be 100 sec for simplicity?) in the DO for rate limiting


changed pandas_udf apply wrapping to be in one place

@icexelloss , I wasn't able to do exactly what I was talking about earlier, but this is along the same lines.  Basically doing the wrapping in one place so it's easy to follow the conversions between Pandas DataFrames and Series.  As a result, it doesn't need to be wrapped in the worker.py.  Buuuut... something isn't quite right and causing a crash. If you think you might use this, I'll put in some more effort and debug it.  It should work, just probably something minor i missed.

@BryanCutler Thanks for the PR!
Yes I think this is a little better than the existing version. Also I also think the a future refactor of this is still needed anyway so my preference would be not spend too much time on this now.
If you have strong preference of merging this change. I am happy to incorporate that as well. What do you think?

I'll take another look at the problem today and see if I can get it working

Hey @icexelloss , the problem was with https://issues.apache.org/jira/browse/ARROW-1654 so I changed it to convert to arrow types in the wrapped function and all is working now.  That was not giving a good error message from the python worker (not your fault), so I think there is a problem with our protocol for error reporting :.  That's for another time though..

I think it's much cleaner to have all the wrapping done in one place, so I think we should put this in, but up to you.

LGTM! Merged.


Limit status checks before starting services (SOFTWARE-2846)

Plus bonus commit for TODO item that wasn't marked as such.
Test results here: http://vdt.cs.wisc.edu/tests/20170810-0913/results.html

@matyasselmeci how's it look now?


Suggestions don't take into account lines above

When I type:
// a function to add two numbers
|

And have my cursor where indicated, I get nothing. In other setups I get options.
What am I missing?

Duplicates of https://github.com/zbirenbaum/copilot-cmp/issues/5 maybe?

Duplicates of #5 maybe?
Yes it is


docs: exercise 06/11

3 screenshots :

exiftool -metagoofil
-recon-ng metacrawler

Checks

[ ] Your YAML file has required fields as listed in Specs
[ ] All checks have passed yaml-lint passed
[ ] You agree to contribute your work under MIT license

I also participated :D


Further improve coverage

I modified the positivity tests such that the limiters in 1D and 2D are actually triggered.

Did you have a look at the numerical solutions or CFL magic?

Did you have a look at the numerical solutions or CFL magic?

Sorry, no, not yet. Do you think you can do it, or should we just merge now and check it later?

We can merge now and check later :+1:

If we're to achieve 97% coverage again, we need to cover 30 more lines. I am looking at it right now...

If this doesn't push coverage beyond 97%, I'll give up on this goal before v0.3... Note that some checks now already make use of the Taal way: Instead of doing only integral tests, I am adding a bunch of unit tests for many methods such as entropy or pressure. I hope that's ok - if not, I propose to proceed anyway and open an issue for it (or, of course, any of you should feel free to add integral checks instead).

Do you think it would be helpful to add some comments like the following one to test_manual.jl?

Constructing indicators/controllers using the parameters below doesn't make sense.

Yes, good idea :+1:. I'll do so once the tests here have passed and before the merge.


Reify non-zero arity constructors

:(

What?

Currently only zero-arity constructors are reified, and this is enough for hello world.


How can I implement multi-players chatting online?

Excuse me, can the Python SDK implement multiplayer online chatting? Or implement the dual-stream mode?

Yes, you can implement according to the demo
https://github.com/AgoraIO-Community/Agora-Python-QuickStart


docs: Add syntax highlighting

First of all, it's great to see this being open-sourced 🙌  While going over the docs, I noticed some of the syntax highlighting was missing.

Thanks @Fokko !


EasyCLA Bug Report: Cannot edit/remove contributors or add approval criteria

Summary
NOTE: This issue was reported in Slack. I'm posting here just to keep track
Cannot interact with CLAs on CLA dashboard
Background
As a CLA Manager, when I click on edit on any button, the page locks up and becomes unusable.
Environment
Please complete the following information:

Environment:

[ ] ALL
[ ] DEV
[ ] STAGING
[x] PROD (unknown on others)

Browser:

[x] Chrome/Brave
[x] Firefox
[ ] Opera
[ ] Vivaldi
[ ] LibreWolf
[ ] SRware Iron
[ ] Dissenter
[ ] Slimjet
[ ] Midori
[ ] Edge
[ ] Lynx

Version: v1.0.XX

Additional Context
Logs from the developer console
main-es2015.c1ba09563839f6a28906.js:23 Cannot get unsaved property from storage => notifications
(anonymous) @ main-es2015.c1ba09563839f6a28906.js:23
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,32
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,198
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,306
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,235
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,22
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,79
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,64
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,208
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,275
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,312
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,534
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,190
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,462
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,383
main-es2015.c1ba09563839f6a28906.js:2587 [GraphQL error]: Message: 502: Bad Gateway, Path: getCompanyContributorAcknowledgements,contributorsList,442
vendor-es2015.026245ac40c5185d7705.js:1 ERROR TypeError: Cannot read properties of null (reading 'photoUrl')
    at Object.updateDirectives (/12-es2015.ea478d73500a95acfebb.js:1:720594)
    at Object.updateDirectives (/vendor-es2015.026245ac40c5185d7705.js:1:389745)
    at Zc (/vendor-es2015.026245ac40c5185d7705.js:1:372220)
    at ru (/vendor-es2015.026245ac40c5185d7705.js:1:379387)
    at iu (/vendor-es2015.026245ac40c5185d7705.js:1:379048)
    at Zc (/vendor-es2015.026245ac40c5185d7705.js:1:372242)
    at ru (/vendor-es2015.026245ac40c5185d7705.js:1:379387)
    at iu (/vendor-es2015.026245ac40c5185d7705.js:1:379048)
    at Zc (/vendor-es2015.026245ac40c5185d7705.js:1:372242)
    at ru (/vendor-es2015.026245ac40c5185d7705.js:1:379387)
Yn @ vendor-es2015.026245ac40c5185d7705.js:1
0:1          Failed to load resource: the server responded with a status of 403 ()
%EF%BF%BD.png:1          Failed to load resource: the server responded with a status of 404 (Not Found)
1516360001751:1          Failed to load resource: the server responded with a status of 403 ()
[.png:1          Failed to load resource: the server responded with a status of 404 (Not Found)
0:1          Failed to load resource: the server responded with a status of 403 ()

Acceptance Criteria
For the buttons on the EasyCLA console to work as intended

Added pagination to the API as part of PR: https://github.com/communitybridge/easycla/pull/3497 - awaiting UX team updates to add pagination client-side support.

@mrbobbytables I believe the issue with loading the console and being able to use the UI was resolved with the pagination introduced to the Approval Criteria. This greatly reduced the loading time and all buttons should now be working - can you confirm?
Note: Pagination was added to both contributor acknowledgements and approval criteria sections in the api - In the UI it has only been implemented in the approval criteria so far, We are also now implementing in event log.

closing this and tracking upgrades to corporate console in 3575


Add GPU configuration options to TensorFlow Go bindings

Introduced a new GPUOptions struct in TensorFlow's Go bindings to allow users to configure GPU-specific settings, such as enabling memory growth (AllowGrowth) and specifying a GPU allocator type (AllocatorType).
Added unit tests to ensure the GPUOptions behave as expected and integrate seamlessly with existing TensorFlow functionality.
Updated the session creation API to accept and apply GPUOptions during initialization, aligning the Go API with TensorFlow’s Python API.

Please sign the CLA and then tag me again to review.

Please sign the CLA and then tag me again to review.

I've signed the CLA

It still shows as unsigned,
❌ https://github.com/tensorflow/tensorflow/commit/fdbcb89b9911acecdc7d2704176c2a4632eafb0b Author<EMAIL_ADDRESS>
I cannot debug more for the next ~2 weeks due to not being home, sorry for the delay.

@kayladoann, Could you please sign CLA, as it is still unsigned?  Thank you !

So the issue with the CLA is that you signed with your Github account but the email associated to the commit<EMAIL_ADDRESS>is not associated to your GitHub account, @kayladoann.
If you can attach them, I think that would work. Alternatively, you can amend your current commit to change the email address (git commit --amend ... followed by git push -f)
I might need to click one button on our side if the process is not automated, so please ping me once you fix this.


Cannot read property 'TNSCanvas' of undefined

Sorry if this question is a miss implementation.
I would like to try to setup NativeScript/canvas and NativeScript/canvas-three in my Angular Native Script project.
Current dependencies are
    "@angular/core": "^14.0.0",
    ...
    "@nativescript/angular": "^14.0.0",
    "@nativescript/canvas": "^1.0.0",
    "@nativescript/canvas-three": "^1.0.0",
    "@nativescript/core": "~8.2.0",
    "three": "^0.141.0",

I am using Android API 32, and a virtual emulated device.
My code is
shared.module.ts
import { NativeScriptCommonModule, NativeScriptFormsModule, NativeScriptRouterModule } from '@nativescript/angular';
import { CanvasModule } from '@nativescript/canvas/angular';

const MODULES = [
  NativeScriptCommonModule,
  NativeScriptFormsModule,
  NativeScriptRouterModule,
  CanvasModule
];

home.component.html
<ActionBar title="My App" class="action-bar" flat="true">
</ActionBar>
<StackLayout class="p-20">
  <Canvas id="canvas" width="100%" height="100%" (ready)="ready($event)"></Canvas>
</StackLayout>

home.component.ts
export class HomeComponent {

  ngOnInit() {
    setStatusBarColor('dark', '#97d9e9');
  }

  ready(event) {
    console.log('[TEST => ready]');
  }

All I get is

JS: Angular is running in development mode. Call enableProdMode() to enable production mode.
JS: ERROR Error: Uncaught (in promise): TypeError: Cannot read property 'TNSCanvas' of undefined
JS: TypeError: Cannot read property 'TNSCanvas' of undefined
JS:     at new Canvas (file: src\webpack:\frontend\node_modules@nativescript\canvas\Canvas\index.android.js:31:0)
JS:     at ViewUtil.createView (file: src\webpack:\frontend\node_modules@nativescript\angular\fesm2015\nativescript-angular.mjs:1518:21)
JS:     at NativeScriptRenderer.createElement (file: src\webpack:\frontend\node_modules@nativescript\angular\fesm2015\nativescript-angular.mjs:2082:35)
JS:     at file: src\webpack:\frontend\node_modules@nativescript\angular\fesm2015\nativescript-angular.mjs:1946:39
JS:     at _ZoneDelegate.invoke (file: src\webpack:\frontend\node_modules\zone.js\fesm2015\zone.js:372:0)
JS:     at Zone.run (file: src\webpack:\frontend\node_modules\zone.js\fesm2015\zone.js:134:0)
JS:     at runInRootZone (file: src\webpack:\frontend\node_modules@nativescript\angular\fesm2015\nativescript-angular.mjs:1939:21)
JS:     at NativeScriptRenderer.descriptor.value (file: src\webpack:\frontend\node_modules@nativescript\angular\fesm2015\nativescript-angular.mjs:1946:19)

I tried using
  <Canvas:TNSCanvas height="100%" width="100%" id="canvas" (ready)="ready($event)" (loaded)="loaded($event)"  />

This works, but ready is never fired, only loaded and the object reference is empty so I can't find a easy way to get the context.
<canvas:Canvas height="100%" width="100%" id="canvas" (ready)="ready($event)" (loaded)="loaded($event)"  />

throw the same error as above.
also added require('@nativescript/canvas-polyfill') just in case in main.ts, but no success.
is it something I am setting up wrong ? I tried to copy the documentation part a bit of demo-angular/canvas-three but even tho I double checked all, it looks like I can't get it running

try polyfill before app run...
also, u can´t use 141 now... use 133~

+1 in vue3

Fixed in v2.


🚑 index out of range on preparetk

The Problem:
When I try to parse the result from whois 'zcore.tk' i found this error:
"runtime error: index out of range [5] with length 5"
Steps to reproduce:
package main

import (
	likesianwhois "github.com/likexian/whois"
	whoisparser "github.com/likexian/whois-parser"
)

func main(){
	data, _ := likesianwhois.Whois("zcore.tk")
	parsed, _ := whoisparser.Parse(data)
}

and the error will occurs...
I can see that this domain returns five parts of address wich the last one is the country...
So, i send this PR to solve this problem adding a new field Country into array of fields.

+1

Hello @EduardoRMello
Thanks for your PR, I will merge it.

If you think this repository is helpful, please share it with friends, thanks.


Docs: installation typo

On the Linux installation documentation page, there are <<<<<<< HEAD and >>>>>>> master markers. These are artifacts from a merge in Git. These should not be there.
https://kivy.org/doc/stable/installation/installation-linux.html

Looks that the issue has been fixed. Feel free to re-open if it persists.