Cannot ParseAndCheck on Windows without Visual Studio installed

A user has reported Fable is not working on a Windows machine without Visual Studio installed. You can see the issue here. This is also related to #535.
Repro steps
Create a FSharpChecker with keepAssemblyContents=true on a Windows 10 machine with F# 4.0 installed stand-alone (no Visual Studio) and try to parse a script file (.fsx).
Expected behavior
The script is parsed and it's possible to access the AST for JavaScript compilation.
Actual behavior
The following error occurs when running ParseAndCheckProject: Assembly reference 'System.Numerics.dll' was not found or is invalid: L1 (startup)
Known workarounds
I recommended the user to create a project (.fsproj) instead, but then the following error appeared:
> Unable to find the file 'mscorlib.dll' in any of
 C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0
 c:\Users\Manuel\Dropbox\Dev\fable-test\src
 C:\Program Files (x86)\Microsoft SDKs\F#\4.0\Framework\v4.0\: L1 (unknown)
> Unable to find the file 'FSharp.Core.dll' in any of
 C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0
 c:\Users\Manuel\Dropbox\Dev\fable-test\src
 C:\Program Files (x86)\Microsoft SDKs\F#\4.0\Framework\v4.0\: L1 (unknown)
> Unable to find the file 'mscorlib.dll' in any of
 C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0
 c:\Users\Manuel\Dropbox\Dev\fable-test\src
 C:\Program Files (x86)\Microsoft SDKs\F#\4.0\Framework\v4.0\: L1 (unknown)

Related information

Operating system: Windows 10
.NET Runtime, CoreCLR or Mono Version: Windows SDK and Net 4.6 SDK
Editing Tools (e.g. Visual Studio Version): No Visual Studio installed

This was caused by missing Microsoft Build Tools 2015


Ast refinements

Summary
Adding support for the next most commonly used ANSI SQL statements to the ast
Test Plan
manual testing during development + new unit tests

[x] PR has an associated issue: #https://github.com/DataJunction/dj/issues/271
[x] make check passes
[x] make test shows 100% unit test coverage

build failures will be fixed after merging PR #281 and rebasing this.


#43 Patch to fix intensity/altitude issue (auto max)

This patch works by automatically setting the max as it changes on each zoom level as points get combined/separated.
You no longer need to manually set options.max
When calculating the max value it considers all points (even if they are off the screen). This way you always get the same results at a particular zoom level.
Thanks to @spacemansteve for pointing me in the right direction
ca1f43026545f7e413c653e376b1fa8bf3a2b6a6

Any update on this? It would be nice to see it included.

Bumping this, would appreciate some update here.

Any update on this? Need this working !

I strongly suggest a disclaimer in the readme explaining this bug if it is not going to be be covered. This is a major bug

This pull request is a must have! I am using it and it is much more intuitive to use the heatmaps when the intensity adjusts based on the current zoom. This really deserves to be merged, even after all this time.

It's been almost 3 and 1/2 years.  Any plans on merging this in?

At this point it might be worthwhile seeing if anyone else might want to manage this plugin whilst being active?

Can this be merged into master?

Please approve this PR, this is a major bug that needs to be fixed.

@mourner do you have a chance to have a quick check ?
I've tested it on uMap and it seems to have more friendly results. The implemented change seems logic to me, but I can't say I master the topic, so a check from you would help! And if it's good for you, I'm up for releasing :)
Thanks in advance, and take care, amigo!


Merge momentum equations into the code

I will manually add Ilya's changes line by line, so that both models are kept and can be switched at will

I think it would be much better to merge the pull request.
Having both implementations side by side will make the code very messy and difficult to maintain.
Let's make a commitment to the new implementation.
The original implementation will still be available and accessible via GitHub.

I'll try to make it clean. Making such a global change without the possibility for a user to use an older version may require from us a commitment to fix the code promptly and frequently in case of bugs.


621 add uberspace infrastructure

Add service + htaccess + public symlinks for our app and a new deploy:setup task that can be run to override / create this whole lot.

Looks very good. I would love to put the information about the port into one central place, because, you know:

EVERY PIECE OF KNOWLEDGE MUST HAVE A SINGLE, UNAMBIGU- OUS, AUTHORITATIVE REPRESENTATION WITHIN A SYSTEM.

Can we maybe also link the maintanance page from #602 to the right place?

@moonglum @bitboxer glad you fell for my port decoy and insist on EVERY PIECE OF KNOWLEDGE HAVING A SINGLE, UNAMBIGUOUS, AUTHORITATIVE REPRESENTATION WITHIN A SYSTEM.
Can we now please talk about this?

http://gifs.joelglovier.com/eddie-izzard/not-listening.gif

:dancers:

Alrighty. Paging @moonglum & @bitboxer.
Deduplicated that second occurance of the Unicorn port with an ERB Template, so now it's your turn to render all our Ruby versions from one single source of truth. :trollface:

Delightful, you may :shipit: Thanks for being such a glorious role model for DRYnes :heart:


feat(styling): Allow loading .css files as a fallback if no .scss file is found(#954)

Implements loading the corresponding .css file if .scss file was requested but not found.
Resolves: #954

uitests


Remove CI warnings

Remove warnings in CI:

Update versions of actions that are using deprecated node12.
Specify python-version in pre-commit action.

I have confirmed that this does indeed remove all the warnings in CI.


API browser page looks unresponsive when the search is slower than a second or so.

Idea Title
Some kind of progress indicator should be placed at the API browser page because it looks like the site is unresponsive during search.
Issue Details
I type for IComparer, press Enter and there are 1-2 seconds in which nothing moves - Looks like the site is unresponsive.
If the result is delayed by more than 0,5 seconds then a progress bar or some other indicator should appear.

Thanks for the suggestion, @ognyandim - adding this to the backlog.

There is now a progress indicator for searches.


FW-4697 Async Search Indexing

Description of Changes

Converts search indexing signals into requests for async tasks.
Changes the indexing disconnect for migration the the new requests.

Checklist

[x] README / documentation has been updated if needed
[x] PR title / commit messages contain Jira ticket number if applicable
[x] Tests have been updated / created if applicable
[x] Admin Panel has been updated if models have been added or modified
[x] Migrations have been updated and committed if applicable
[x] Team Postman workspace has been updated if applicable

Reviewers Should Do The Following:

[x] Review the code changes here
[x] Pull the branch and test locally

Additional Notes

Some countdowns (time in seconds before the async task is run) are required for the post_save requests to function as the model needs no be created first. (more detail in the comments)
Integration tests for this functionality will be added later. Each of the requests has been manually tested for functionality and no celery errors occur.

Pytest has still not completed in ~2 hours, I suspect that the tests are having some weird interactions with celery. I'll investigate


refactor(bigint): improve encode/decode implementation

This change is

and in u8(up to 255) bytes we pretty much have an infinitely large number - so we should be quite fine.
we can probably even not use 2 bits and keep using sign encoded within it, so we'd have up to 64 bytes.

If the len is stored as compact, it already uses two bits for the scale codec, so the max value (if stored on one byte) is already 63.

@orizi
Also if we are comfortable with those values never being larger than Felt::MAX (and it seems we are, since we are willing to panic!), it should probably be enforced at the type level itself. Through some kind of wrapper or a type wich internal representation is not a potential infinite Vec, but rather a [u8; 32]. Or even a BoundedVec.
Wdyt?


Add lua package for json module written in C

This package does to install-extra target, but deletes the test cases installed in $out/share/lua/$version/cjson/tests.

Built locally, thanks.


MINOR: Standby task commit needed when offsets updated

This is a minor fix of a regression introduced in the refactoring PR: in current trunk commitNeeded always return false, which would cause standby tasks to never be committed. To go back to the old behavior we would return true when new data has been applied and offsets being updated.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

cc @mjsax @vvcephei @abbccdda

test this please


ERROR: Failed building wheel for intel-npu-acceleration-library

Describe the bug
Please help. Encountered "ERROR: Failed building wheel for intel-npu-acceleration-library" while executing "pip install intel-npu-acceleration-library".
To Reproduce
Run "pip install intel-npu-acceleration-library"
Expected behavior
Successfully installed.
[*Screenshots]

**Desktop

OS: Ubuntu22.04LTS
Python: 3.10.12
vertualenv: 20.26.0
pip: 24.0
wheel: 0.43.0
setuptools: 69.5.1
NPU: Intel(R) AI Boost
CPU: Intel(R) Core(TM) Ultra 7 165H

Thanks

Sorry I was off but yes, on ubuntu we provide only sdist so you need to build the packet and to buld you need cmake (and a working compiler). Glad you sorted out. Closing


how to keep css throught pages?

I use same css for all pages in my site, and some css are dynamic loaded by webpack.
When I visit a map page with dynamic css, it works well for the first visit , but wrong for the second visit.
the dynamic css are not loaded again because webpack think the related modules have been loaded.
so is there any way  I could keep the css not away ?

I'm not sure about dynamic css but it seems bad usage. You may be able to keep dynamically loaded css using the ignore option but it is unrecommended usage. All the knowledge or the diff to update the page must be contained in the fetched HTML in principle.
https://falsandtru.github.io/pjax-api/api/pjax/config/

Thank you.
update: {css: false}, works for me.
And I also let web server use a lean layout to supply specialized content for pjax request, which only includes special content for special page.    the  part is always same across my whole website.


This can help a lot

What feature do you want to get added on the base engine?
for x32 Bits versions there is a solution for the cutscenes, maybe you already know or something like that but the solution is to exchange the MP4 for Webm, that way it can be compiled and opened.
For what I wrote this is because of what was mentioned, maybe you can make another source code for the Psych Engine but with Webm to compile on 32-Bit PC's, Mac, Linux, Android, etc.

Ehhh nah


stopped working after update to NR2.2.2

Yesterday I updated to NR2.2.2 and today realized that the output of report node is
20.2.2022, 10:57:53[node: payload.response:js_obj](http://<myip>:1880/#)
msg : string[40]
"Unexpected token Y in JSON at position 0"

Is this packaged Maintained actively?

Hi, I built this as part of my internship at myStrom a few years ago. I am not sure whether they are planning on updating or continuing to maintain this packet in the future.


test

https://github.com/xtli12/GSNet/assets/86363634/841ebafd-57a8-4aa2-bb6f-46afcd69c566

https://github.com/xtli12/GSNet/assets/86363634/e82f37a1-f8eb-444a-b092-e144298c87af


Enhancement Request: Zones - Device membership

I think the zones feature is a very powerful feature which is particularly useful for larger networks (IMHO).
If possible, I would like to see some updates to the methods available to discover members of  a zone and adding members (i.e. boxes) to a zone.

On this zone summary page, would it be possible to make the 'Boxes' number a clickable link that would drop through to a devices listing, which uses the zone name as the search term:

This would allow a easy method of finding all boxes assigned to a zone. (This works great from the search bar within the devices page by the way, which is great...but you're bound to have a few old farts like me that don't realize you can do that)

Would it be possible to have the option to change the zone that a device is assigned to from within the 'Edit' page of the device itself? This is currently achieved by selecting the 'Edit Zones' drop-down option, but it would also be useful to be able to do it from within the box configuration itself. This is maybe a personal preference thing...I've just found myself hunting around for it a few times as I expected in to be part of the device configuration - again, this is likely to be 'old fart' territory. Now....what day is it again...?

Fab, thanks.
We've taken the edit zone button out for the time being. Mainly because there was three places to add to a zone.
We consolidated into the list view - the aim was to allow people to bulk add etc.
We've got a revamp to the zones coming soon, including the report filtering :)

This is more doable now we've updated the way the zones work. Moving to a July milestone.

Zones now show the correct count of boxes.
Adds zone name to boxes show page.
References https://github.com/cucumber-tony/cucumber-frontend/commit/4c6b1c9ed24716f773fb95a51f2b4b8f66b662c7

You can now change the box zone from the box edit page

Old fart aside, this was a fabulous suggestion!


createRouterTransport with Jest's jsdom raises error

Describe the bug
When using createRouterTransport() to mock an API, tests running with Jest in the jsdom environment can raise the following errors:

ReferenceError: TextEncoder is not defined
iterable[Symbol.asyncIterator] is not a function

This first error is because jsdom does not support TextEncoder and TextDecoder, see https://github.com/jsdom/jsdom/issues/2524. The second error is because we use instanceof Uint8Array in the code base to narrow down types, and Jest breaks it, see https://github.com/jestjs/jest/issues/4422.
To Reproduce
Get this gist, run npm ci and npm test. The tests pass, but only because jest.config.js contains the following setting:
  globals: {
    TextEncoder: TextEncoder,
    TextDecoder: TextDecoder,
    Uint8Array: Uint8Array,
  }

Removing the globals produces the errors described above.
Environment (please complete the following information):

@bufbuild/connect 0.9.0
Jest 29.5.0
Node.js: 18.15.0

Additional context
jsdom is  - quoting the repository description - "a JavaScript implementation of various web standards, for use with Node.js".
It seems odd that using it in Jest introduces incompatibility that wouldn't otherwise exist: Uint8Array is available since Node.js from 2013 and IE 10 from 2012. TextEncoder is available since Node.js from 2017 (but can easily be implemented with Node's Buffer), and the only browser that does not implement it is IE 11 from 2013.

A new issue on Uint8Array. If test files are running in sequence, they are working well with globals setting in jest.config.js.  However it complains iterable[Symbol.asyncIterator] is not a function  again when in parallel.
I suspect it's somehow changed during running and result in remaining files can't access it correctly.  We can potentially see that from globals doc in jest
If you specify a global reference value (like an object or array) here, and some code mutates that value in the midst of running a test, that mutation will not be persisted across test runs for other test files

So I setup an custom environment to reset Uint8Array before each test file running to fix that.
class CustomTestEnvironment extends TestEnvironment {
  async setup() {
    await super.setup();
    /**
     * Set TextEncoder/TextDecoder if undefined
     */
    if (this.global.TextEncoder === undefined) {
      this.global.TextEncoder = TextEncoder;
    }

    if (this.global.TextDecoder === undefined) {
      (this.global as any).TextDecoder = TextDecoder;
    }
    /**
     * Uint8Array configured in globals could be changed when running tests in parallel
     * Reset Uint8Array before each test file is running
     */
    this.global.Uint8Array = Uint8Array;
  }
}

Good catch, @nicole0707!
Uint8Array is a constructor function that does not hold state. But if the issue comes back with parallel runs, obviously something is going wrong, it might just be more complex than expected.
The docs also say:

In addition, the globals object must be json-serializable, so it can't be used to specify global functions.

Jest is most likely making a copy of globals to prevent individual tests from modifying it. I imagine this can break in mysterious ways with the constructor functions. Your approach of making them available through the environment seems to be the way to go.

Jest is most likely making a copy of globals to prevent individual tests from modifying it.

@dimitropoulos' test seems to confirm that in https://github.com/bufbuild/connect-query/pull/50#issuecomment-1568411746.
We will look into publishing a test environment as an npm package, so that it is easy to set up tests.


Release scripts

This incorporates a new helper script, tools/release_tool.py. It features commands to perform all the steps of a release:
$ python tools/release_tool.py init -t minor
$ python tools/release_tool.py changelog
$ python tools/release_tool.py version -p 1
$ python tools/release_tool.py preflight
$ python tools/release_tool.py create
$ python tools/release_tool.py watch
$ python tools/release_tool.py postflight
$ python tools/release_tool.py version
$ python tools/release_tool.py preflight
$ python tools/release_tool.py create
$ python tools/release_tool.py watch
$ python tools/release_tool.py postflight

In addition, there are meta-commands that run the entire thing for a major/minor/patch release. Typical usage would be something like this:
$ python tools/release_tool.py minor

This will make a new release branch based on main, bump the minor version number, then create and sanity check RC and regular releases, with all the versioned files updated appropriately.
This change also streamlines the Windows build. It uses the stock Rust toolchain, rather than patching with platform mingw components. It also let's the gmp-mpfr-sys crate build its internal copies of gmp and mpfr, rather than building them manually. This seems to work fine now. The release.yml workflow has been refactored to use a separate build script, which will make it possible to create identical builds when preflighting a release.
* Added tools directory.
* Added release_tool.py helper script.
* Updated CHANGELOG.md to match new format.
* Added build_tool.py helper script.
* Broke out common build operations into separate rust_build.sh and python_build.sh scripts.
* Added publish_tool.py helper script.
* Refactored release.yml workflow using new scripts.
* Moved assert_version.py to tools directory.
* Fixed compiler warnings.
* Simplified windows build to use vendored builds of gmp and mpfr in gmp-mpfr-sys crate.
* Removed patch of Rust toolchain for msys2.
* Cleaned up entire workflow.
* Implemented preflight test.
* Restored versions to 0.0.0-development.

It's awesome that Windows builds compile without strange hacks!
One reason why things could be working now, without tinkering, is because the bundled msys is still similar enough to the system msys. Fingers crossed the compiler is indeed using the system msys libs now.


native-image failing on io.helidon.microprofile.weld.ExecutorServices

Environment Details

Helidon Version: 2.0.2-SNAPSHOT
Helidon MP

Problem Description
I'm getting an exception building the native-image for the MP quickstart:
[INFO] 2020.09.02 01:15:12 WARNING ServiceLoader Thread[ForkJoinPool-2-worker-11,5,main]: Could not load service class io.helidon.microprofile.weld.ExecutorServices
[INFO] 2020.09.02 01:15:12 WARNING ServiceLoader Thread[ForkJoinPool-2-worker-11,5,main]: Could not load service class io.helidon.microprofile.weld.ExecutorServices
[INFO] 2020.09.02 01:15:12 WARNING ServiceLoader Thread[ForkJoinPool-2-worker-11,5,main]: Could not load service class io.helidon.microprofile.weld.ExecutorServices
[INFO] 2020.09.02 01:15:12 INFO org.jboss.weld.Bootstrap Thread[ForkJoinPool-2-worker-11,5,main]: WELD-000101: Transactional services not available. Injection of @Inject UserTransaction not available. Transactional observers will be invoked synchronously.
[INFO] 2020.09.02 01:15:12 INFO org.jboss.weld.Event Thread[ForkJoinPool-2-worker-11,5,main]: WELD-000411: Observer method [BackedAnnotatedMethod] public org.glassfish.jersey.ext.cdi1x.internal.ProcessAllAnnotatedTypes.processAnnotatedType(@Observes ProcessAnnotatedType<?>, BeanManager) receives events for all annotated types. Consider restricting events using @WithAnnotations or a generic type with bounds.
[INFO] 2020.09.02 01:15:12 INFO org.jboss.weld.Event Thread[ForkJoinPool-2-worker-11,5,main]: WELD-000411: Observer method [BackedAnnotatedMethod] private io.helidon.microprofile.openapi.OpenApiCdiExtension.processAnnotatedType(@Observes ProcessAnnotatedType<X>) receives events for all annotated types. Consider restricting events using @WithAnnotations or a generic type with bounds.
[INFO] 2020.09.02 01:15:26 WARNING com.netflix.config.sources.URLConfigurationSource Thread[ForkJoinPool-2-worker-23,5,main]: No URLs will be polled as dynamic configuration sources.
[INFO] 2020.09.02 01:15:26 INFO com.netflix.config.sources.URLConfigurationSource Thread[ForkJoinPool-2-worker-23,5,main]: To enable URLs as dynamic configuration sources, define System property archaius.configurationSource.additionalUrls or make config.properties available on classpath.
[INFO] Warning: RecomputeFieldValue.ArrayIndexScale automatic substitution failed. The automatic substitution registration was attempted because a call to jdk.internal.misc.Unsafe.arrayIndexScale(Class) was detected in the static initializer of rx.internal.util.unsafe.ConcurrentCircularArrayQueue. Detailed failure reason(s): Could not determine the field where the value produced by the call to jdk.internal.misc.Unsafe.arrayIndexScale(Class) for the array index scale computation is stored. The call is not directly followed by a field store or by a sign extend node followed directly by a field store.
[INFO] Warning: RecomputeFieldValue.ArrayBaseOffset automatic substitution failed. The automatic substitution registration was attempted because a call to jdk.internal.misc.Unsafe.arrayBaseOffset(Class) was detected in the static initializer of rx.internal.util.unsafe.ConcurrentCircularArrayQueue. Detailed failure reason(s): Could not determine the field where the value produced by the call to jdk.internal.misc.Unsafe.arrayBaseOffset(Class) for the array base offset computation is stored. The call is not directly followed by a field store or by a sign extend node followed directly by a field store.
[INFO] Warning: RecomputeFieldValue.ArrayIndexScale automatic substitution failed. The automatic substitution registration was attempted because a call to jdk.internal.misc.Unsafe.arrayIndexScale(Class) was detected in the static initializer of rx.internal.util.unsafe.SpscUnboundedArrayQueue. Detailed failure reason(s): Could not determine the field where the value produced by the call to jdk.internal.misc.Unsafe.arrayIndexScale(Class) for the array index scale computation is stored. The call is not directly followed by a field store or by a sign extend node followed directly by a field store.
[INFO] Warning: RecomputeFieldValue.ArrayIndexScale automatic substitution failed. The automatic substitution registration was attempted because a call to jdk.internal.misc.Unsafe.arrayIndexScale(Class) was detected in the static initializer of rx.internal.util.unsafe.ConcurrentSequencedCircularArrayQueue. Detailed failure reason(s): Could not determine the field where the value produced by the call to jdk.internal.misc.Unsafe.arrayIndexScale(Class) for the array index scale computation is stored. The call is not directly followed by a field store or by a sign extend node followed directly by a field store.
[INFO] Warning: RecomputeFieldValue.ArrayBaseOffset automatic substitution failed. The automatic substitution registration was attempted because a call to jdk.internal.misc.Unsafe.arrayBaseOffset(Class) was detected in the static initializer of rx.internal.util.unsafe.ConcurrentSequencedCircularArrayQueue. Detailed failure reason(s): Could not determine the field where the value produced by the call to jdk.internal.misc.Unsafe.arrayBaseOffset(Class) for the array base offset computation is stored. The call is not directly followed by a field store or by a sign extend node followed directly by a field store.
[INFO] [helidon-quickstart-mp:10965]     analysis:  25,338.59 ms,  4.41 GB
[WARNING] Error: Could not find registered service implementation class `io.helidon.microprofile.weld.ExecutorServices` for service `org.jboss.weld.bootstrap.api.Service`
[WARNING] com.oracle.svm.core.util.UserError$UserException: Could not find registered service implementation class `io.helidon.microprofile.weld.ExecutorServices` for service `org.jboss.weld.bootstrap.api.Service`
[WARNING] 	at com.oracle.svm.core.util.UserError.abort(UserError.java:68)
[WARNING] 	at com.oracle.svm.hosted.ServiceLoaderFeature.handleType(ServiceLoaderFeature.java:203)
[WARNING] 	at com.oracle.svm.hosted.ServiceLoaderFeature.duringAnalysis(ServiceLoaderFeature.java:112)
[WARNING] 	at com.oracle.svm.hosted.NativeImageGenerator.lambda$runPointsToAnalysis$8(NativeImageGenerator.java:720)
[WARNING] 	at com.oracle.svm.hosted.FeatureHandler.forEachFeature(FeatureHandler.java:70)
[WARNING] 	at com.oracle.svm.hosted.NativeImageGenerator.runPointsToAnalysis(NativeImageGenerator.java:720)
[WARNING] 	at com.oracle.svm.hosted.NativeImageGenerator.doRun(NativeImageGenerator.java:538)
[WARNING] 	at com.oracle.svm.hosted.NativeImageGenerator.lambda$run$0(NativeImageGenerator.java:451)
[WARNING] 	at java.base/java.util.concurrent.ForkJoinTask$AdaptedRunnableAction.exec(ForkJoinTask.java:1407)
[WARNING] 	at java.base/java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:290)
[WARNING] 	at java.base/java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1020)
[WARNING] 	at java.base/java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1656)
[WARNING] 	at java.base/java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1594)
[WARNING] 	at java.base/java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:177)
[WARNING] Error: Image build request failed with exit status 1

Steps to reproduce
Pull the latest master and build it
cd examples/quickstarts/helidon-quickstart-mp
mvn -Dversion.plugin.helidon=2.1.1 clean package -Pnative-image

On a hunch I'm casting the evil eye at https://github.com/oracle/helidon/pull/2269

See #2316

The problem, that the service file is wrong - it references non-existing service implementation.


ubuntu18 can not be ran in hiveos

DynexSolve-main-e10e4a2-ubuntu-18.04-linux-x64-core2.zip

Obsolete by now


feature/S4-285: Support additional company types.

Description
Support additional company types.
Relates to https://github.com/companieshouse/dissolution-api/pull/88
Jira Ticket
https://companieshouse.atlassian.net/browse/S4-285
Coding Standards
Code Style Guide: https://github.com/companieshouse/dissolution-web/wiki/Code-Style-Guide
Checklist

[ ] 3 Amigos
[x] Acceptance Criteria met
[x] Branch named {feature|hotfix|task}/{S4-*}
[x] Commit messages are meaningful
[ ] Manually tested
[ ] All unit tests passing
[ ] WAVE accessibility testing tool ran on new or updated pages
[ ] All UI Tests Passing
[ ] Pulled latest master into feature branch
[ ] Sonar Analysis
[ ] New Docker environment variables are consistent with those on the Rebel1 environment
[ ] If your pull request depends on any other, please link them in the description

Screenshots of new or updated views

I wonder if the content is a blocker. Yes it would be out of date, but does it need to get released to unblock production? Probably not. Keep this in mind if content is taking a while to get finalised.


Docs - Add faq and fix includes info

Add faq about #712 and #792
Fix wrong info as reported in #898
Change dynaconf_include to accept str (like includes_for_dynaconf) as proposed here.

May I add the feature proposed here to this PR or is it better to create another one?

Codecov Report

Merging #899 (dc228c2) into master (28779ef) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master     #899   +/-   ##
=======================================
  Coverage   99.88%   99.88%
=======================================
  Files          21       21
  Lines        1806     1808    +2
=======================================
+ Hits         1804     1806    +2
  Misses          2        2

Impacted Files
Coverage Δ

dynaconf/base.py
100.00% <100.00%> (ø)

dynaconf/utils/__init__.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Upgrade README to vaadin 8

There is not BeanItemContainer anymore in Vaadin8 so the reference to it should be removed.

This was already fixed in a99a7f3


Update context.locals reference

What kind of changes does this PR include?

New or updated content

Description

What does this PR change? Corrects context.locals to specify it can be used in API endpoints.

Thanks! I've committed the suggestions


Repository name

As @mr-c mentioned in a private conversation, Galaxy will be gaining CWL support, and the development of the original gxargparse basically stopped, so Galaxy specific parts can be ripped off. I need them now as a sample, but as soon as CWL tool wrapper will be finished, I'll remove them as much as the docs. Should the repository bear the old name along with that?
/cc @brainstorm

Good points. I understand you would like to cleanup and focus on, for instance, argparse support for now. I guess that a separate repository with a more generic name such as argparse2cwl would make more sense?
What's your take on this @mr-c?

The new name is fine, but no need to create a new repo: we can have this
one renamed.
On Mon, Apr 25, 2016 at 4:12 PM, Roman Valls Guimerà <
<EMAIL_ADDRESS>wrote:

Good points. I understand you would like to cleanup and focus on, for
instance, argparse support for now. I guess that a separate repository with
a more generic name such as argparse2cwl would make more sense?
What's your take on this @mr-c https://github.com/mr-c?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/common-workflow-language/gxargparse/issues/1#issuecomment-214351118

--
Michael R. Crusoe<EMAIL_ADDRESS>Community Engineer                 Common Workflow Language project
https://impactstory.org/u/0000-0002-2961-9670
https://impactstory.org/u/0000-0002-2961-9670   +32 (0) 2 808 25 52
+1<PHONE_NUMBER>

It would seem we should work on a unified code base - I'd strongly urge you not to rip the Galaxy parts out - this seems actively hostile to a large and active community of developers that you should want to embrace the CWL.

Hey @jmchilton
No intent to be hostile to Galaxy here. Since it is likely that Galaxy will support CWL tool definitions by the time this is done we thought it might be simpler to remove the Galaxy XML code paths to simplify the code base.

gxparse development has stalled but I don't think anyone considers it a dead project - I want it to be awesome for Galaxy tools and I want to add it to planemo.
Just because CWL tools might be available in Galaxy doesn't mean Galaxy tool XML files are going away and they will likely provide a superior Galaxy experience for some time.
Keeping two implementations in the code will ensure abstractions exist to keep the code general enough to allow additional tooling formats - it doesn't advance a CWL agenda but it is probably good for this codebase. It has already proven interesting enough to attract another tool format.

If you rip the Galaxy stuff out this is a full fork of the project - you can do a full fork - the license absolutely allows it. But doing a full fork of a project where the original shareholders are interested in maintaining it isn't a good way to do open source in my experience and I would hope for better from the CWL community.

Sounds like we should keep the codebase whole; thank you @jmchilton for helping us understand that.
@anton-khodak I suggest keeping the name for now. We can reconsider the name at the end of the summer.

I would wholly endorse a more general name - argparse2tool or something. I don't see a need to maintain Galaxy branding on this - I just want us all to work together.

Echoing @jmchilton, would be quite happy to see updates upstreamed, CWL folks.
It's true, I've been busy with teaching and haven't been able to make any of the planned improvements to gxargparse, but I'd be open to whatever changes are required for y'all.

Thank you @erasche, we will ensure that the result is still usable and upstreamable. I'm going to close this issue for now.

@erasche If you don't mind, could you add @mr-c, @anton-khodak and myself as contributors to this repo? I'm thinking on enabling future Github integrations with i.e TravisCI for testing and automation, so asking for permissions/accesss in advance (we're not there yet ;)).
Thanks!

Sure thing.


ci: ensure the matching poetry version is used

Also ensure the dependencies version match the one in the repo at releasing.

LGTM. We can merge this.
Documenting why are we changing the CI:

We should not be updating dependencies in CI. Hence we should remove the poetry update command
To install poetry, following the official documentation, we should use the following command: https://python-poetry.org/docs/#installing-with-pipx    pipx install poetry


SQL Server 2012 Diagnostic queries - problem with query -  latency per database file

Hi,
on SQL Server 2012 the query that collects the latency per database file, the governor metrics are not working in 2012.
I suggest just removing the following columns:
io_stall_queued_read_ms AS [Resource Governor Total Read IO Latency (ms)], io_stall_queued_write_ms AS [Resource Governor Total Write IO Latency (ms)]
Regards,
Hugo Alhandra

Great thanks, Hugo. Fixed via https://github.com/ktaranov/sqlserver-kit/commit/16f585cd310491c7a7f32f4208184b0da03f5175
https://docs.microsoft.com/en-us/sql/relational-databases/system-dynamic-management-views/sys-dm-io-virtual-file-stats-transact-sql?view=sql-server-2017


Estimator#export_saved_model section links to irrelevant guide or guide missing relevant section

Thank you for submitting a TensorFlow documentation issue. Per our GitHub
policy, we only address code/doc bugs, performance issues, feature requests, and
build/installation issues on GitHub.
The TensorFlow docs are open source! To get involved, read the documentation
contributor guide: https://www.tensorflow.org/community/contribute/docs
URL(s) with the issue:
https://www.tensorflow.org/api_docs/python/tf/estimator/Estimator#export_saved_model
https://www.tensorflow.org/guide/saved_model#savedmodels_from_estimators
Description of issue (what needs changing):
The api_docs link has a link to the guide/saved_model section. The section referenced does not exist in the guide, nor does any mention of the estimator method export_saved_model.

@C-Compton I agree with you. The link is misleading but the link is a more detailed guide on Using the SavedModel format. May be we need to change from "For a detailed guide, see SavedModel from Estimators."  to "For a detailed guide on SavedModel, see Using the SavedModel format"
Please let us know if you want to contribute and update any docs through a PR. Thanks!

I can work on the PR. Shall I go ahead with your suggestion @jvishnuvardhan ?

This issue can be closed the PR has been merged. @jvishnuvardhan . Thank you.


Increase timeout values for expose tests further.

What this PR does / why we need it:
Increases the time out values for expose tests from 5 to 7 minutes, as one of them has again run into a timeout.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

Hi @dhiller thanks for this, I am wandering, do we have a networking issue? Have these tests been always flaky on the sriov and networking tests?
I have seen tests time out due to network load on the test cluster. I am wandering whether the issue is back.

Hi @dhiller thanks for this, I am wandering, do we have a networking issue? Have these tests been always flaky on the sriov and networking tests?

The tests in expose_test I've been addressing have been flaky due to timeout issues only IMHO, given a good enough timeout they started to be green nearly all the time. I could reproduce the flakiness by setting the test system under load via stress-ng. For this special test the timeout value seemed not to be good enough in one test. So i gradually increased again to hopefully make it always green.

I have seen tests time out due to network load on the test cluster. I am wandering whether the issue is back.

Hmm, can't tell honestly. I'm not a network expert, so I couldn't debug such an issue easily. If you have some spare cycles you might take a quick glance. Also if you can recommend a network stress testing tool, I'm eager to learn.

Unfortunately I am not an network expert either. These types of errors were usually solved by @phoracek .

/test pull-kubevirt-e2e-kind-k8s-sriov-1.14.2


Set AKMicrophone to use non-default input

Hello! I was wondering if it's possible to set an instance of AKMicrophone to use a different input on macOS (change to a usb microphone / digital audiobus / etc.). I found the method AudioKit.setInputDevice(AudioKit.AKDevice), but that changes the entire system default input, which not only changes every instance of AKMicrophone, it also changes the input for every other app using the default microphone.
EZMicrophone has the ability to use a specific input device, but I can't figure out how to pipe its output into the rest of the system.
I would appreciate any help you can give on getting this to work.
Thanks,
-tlf

Hi, this sounds like an important issue, but I don't have the solution for it yet, so I'll leave it open until we can look at it further (probably after Xcode8 conversion is done).

I'm hoping this solves your issue.  I could only test with two devices though, so maybe it still needs work. I'll close this issue, but feel free to reopen it if necessary.

I'm currently unable to test it, but I will get back to you as soon as I find out. Thanks for the update!

Well, the selection of the microphone appears to be working, however now I'm getting a new error, as documented in issue #631

Forgot to include this in the previous comment:
Is there some way to set the output device as well? I see AudioKit.setOutputDevice(_:), but that has the same problem as AudioKit.setInputDevice(_:) does. If I'm missing some way to do this, I apologize, but that would be huge boost to my workflow, plus would be very useful for many audio routing applications (in both relevant senses of the word).
If you want, I can make this its own ticket.
Thanks again,
-tlf

I can probably doing something similar for output, but yeah, make a new issue please.


[style]: Improve the Button Placement for "Add Your Profile", "Opportunity Hub", "Spotlight", and "Connect"

What's the style idea?
The current placement of the buttons "Add Your Profile", "Opportunity Hub", "Spotlight", and "Connect" is not visually appealing and impacts the user experience. The buttons feel misaligned and cluttered, making the interface look unorganized.
The buttons should be placed in a way that enhances visual appeal and aligns with the design aesthetics of the page. This would provide a better user experience by improving readability and navigation.
Add screenshots

Code of Conduct

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

@Utkarshrajmishra ,I wanna work on this issue .please assign it to me

@irshadsiddi, I am working on this.

@Utkarshrajmishra my bad carry on

Hey @irshadsiddi, can you please assign it to me?


cache

I have two datasets of the same size。when I open the cache and do the second query in the MySQL ,I find it fas。 However in Tidb，I have opened the cache and increase cache about 10G ,But In the second query, tidb is not fast to return a result .I have seen the website of https://www.pingcap.com/docs-cn/op-guide/tune-tikv/ 。Can you help me ?

I think you need some execution plan tuning of SQL , please take a look about https://www.pingcap.com/docs-cn/sql/sql-optimizer-overview/ .

I think you need some execution plan tuning of SQL , please take a look about https://www.pingcap.com/docs-cn/sql/sql-optimizer-overview/ .

I am very thank you help me


Add Standard JS linting and travis.yml, fix new lint errors

Resolves #11

Yay! https://travis-ci.org/GoogleChrome/wptdashboard/builds/252199701

Yeah 🤔 the spaces after function names is not super natural to me, but the no semicolons rule seems to have some good reasoning behind it.

Yep -- the reason why we have style rules and linters is so that we can avoid being distracted by differences in mundane things, and focus on the logic. As long as the style is consistent and readable, it's good.


create service bus object with @Autowire

Environment

Spring boot starter:

service bus spring boot starter

OS Type: Windows/Linux/MacOS

Java version:

1.9
1.8
1.7

Summary

Queue/Topic/Subscription objects must be created before you use them.
See also azure servicebus spring boot starter guide.
Reproduce steps

Expected Results

It would be very useful if Autowired directive can create service bus objects (Topics,Queues,Subscriptions) if they don't exist.
Actual Results

Thanks for your proposal, but I am a little confused about your expected results part. In current codes, it will create these objects automatically if you config the specific properties, you only need to @Autowire them if you need to use it. There is an example: https://github.com/microsoft/azure-spring-boot/tree/master/azure-spring-boot-samples/azure-servicebus-spring-boot-sample

On example you referred, point 3 of section ["Create Service Bus on Azure"] (https://github.com/microsoft/azure-spring-boot/tree/master/azure-spring-boot-samples/azure-servicebus-spring-boot-sample#create-service-bus-on-azure), says to create queue and topic from portal before and then copy object names on application.properties.

I think I get your point, but it may cause some other problems, for example, if you run some test codes, then some useless queues or topics will be created in your service-bus resource, it seems not good. But maybe we can provide a property to control the automatically creating for queues and topics like auto.create=true/false. We need to do some investigation and discuss its feasibility, then give you a response. Thanks for your proposal!

Thanks a lot for your effort.
It's great for us if behaviour and features are as similar as possible to RabbitMQ and configurable by autowired (object constructor, connection timeout, client retry,handlers, etc)

Closing this issue.
We've deprecated the Service Bus starter in this repo, and you could use our Spring Cloud Stream Binder implementation instead, check it out here https://github.com/microsoft/spring-cloud-azure/tree/master/spring-cloud-azure-stream-binder. And I remember the resources will be created there if an Azure subscription is provided.


All BoxBilling references (literals, links, components, installer) replaced in favor of FOSSBilling

Hi... I just replaced all literal string referring BoxBilling, as well as al the links for documentation, community, etc.
⚠ Notes:

The copyright notices were left untouched until a resolution in #86
The directories and parts of code referring to */boxbilling/* paths were left untouched as well so some media won't break
Replaced the Facebook page reference in the last step of the install by our Discord server widget (Good idea?)
4. Unsure on how to proceed in bin/gettext_header.php

dang it sorry I'll correct them

dang it sorry I'll correct them

Done. They bypassed the regex that checked for the copyright notice header 🥲

Can you remove the references to 'FOSSBilling PRO' or 'PRO version of FOSSBilling' - I noticed a couple in there that were changed from BoxBilling PRO references that should already not have been there.

Can you remove the references to 'FOSSBilling PRO' or 'PRO version of FOSSBilling' - I noticed a couple in there that were changed from BoxBilling PRO references that should already not have been there.

Sure

Can you remove the references to 'FOSSBilling PRO' or 'PRO version of FOSSBilling' - I noticed a couple in there that were changed from BoxBilling PRO references that should already not have been there.

Sure

Done

@HunterNyan, can you mark the requested changes resolved?


SHOW DATA noexist has been broken

Reported by steven c hankin on 9 Nov 2015 19:07 UTC
As of Ferret V5.5 the command SHOW DATA noexist returned an error message that included the name of the nonexistent file
        NOAA/PMEL TMAP
        FERRET v5.50
        Linux 2.4.3-12smp - 01/15/03
        09-Nov-15 11:03

yes? show dat noexist
 **ERROR: unknown data set: noexist

This is important if the error message is generated by a script.  The latest Ferret, however, fails to provide the name of the non-existent file
/home/users/hankin/FERRET32/fer> ferret
        NOAA/PMEL TMAP
        FERRET v6.95
        Linux 2.6.18-406.el5PAE 32-bit - 10/27/15
         9-Nov-15 11:06

yes? show dat noexist
 **ERROR: unknown data set:
yes?

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2322

Comment by steven.c.hankin on 9 Nov 2015 19:12 UTC
just noticed:
-rw-r--r-- 1 hankin tmap  8505 Oct 26 12:02 show_data.F

Might this bug have been introduced in a current source code mod?

Comment by ansley.b.manke on 9 Nov 2015 19:20 UTC
No, I see it in versions of Ferret going back a bit, to perhaps the changes for double-precision/ 6-D Ferret.

Comment by ansley.b.manke on 10 Nov 2015 01:36 UTC
This is fixed in xeq/xeq_show.F  It came in with some of the changes for attribute handling, when show_data.F was split off as a separate routine from xeq/xeq_show.F

yes? SHOW DATA noexist
 **ERROR: unknown data set: noexist
yes? SHOW DATA 2
 **ERROR: unknown data set: 2


Update readme

[ ] add link to Redux docs instead of What is Flux
[ ] describe Native stuff
[ ] explain how to run Este in production (no gulp)
[ ] fix Android and remove WIP
[ ] how to create bare Este easily
[ ] Node 4+ is required once Node 5 is tested

done


Nightly K8s Matrix CI 2023-07-05: Failed

action url: https://github.com/spidernet-io/spiderpool/actions/runs/5468538906

error, found error log !!!
----- check gorouting leak in kube-system/spiderpool-controller-59cfd6f4b4-tgpsh
maybe goroutine leak, found 325 goroutines in kube-system/spiderpool-controller-59cfd6f4b4-tgpsh , which is bigger than default 300

add to 350


chore: Add npm maintainers

This adds a maintainers field to the package.json. Ideally, someone else should be in this field, too. Please add another contributor as a maintainer for this npm package, such as @shamb0t, by typing: npm owner add @shamb0t. Then, add them to the maintainers field in this PR, or ping me to do it. Thank you.

Hold off on this. There's an issue with inconsistent docs for npm's package.json. Looking into it.

Figured out a workaround. In keeping with orbitdb/welcome#55, I add an orbitdb field here. I also suggest @shamb0t as an extra maintainer, as she is already a contributor here. When this is merged, she should be added.
@haadcode Let me know if this works. If it does, I'll go through and do similar PRs to the rest of the org.

In this, is the orbitdb key arbitrary or does that map to an npm org?

It's arbitrary. It's just for us to use. maintainer and maintainers at the top-level might be an issue.

How is the array of names in orbitdb.maintainers consumed on the NPM side?

It isn't. I'd like to build a tool to check the veracity of the maintainers list against the npm info output. For lack of a better space, I opened an issue about that here: https://github.com/orbitdb/repo-template/issues/6. For now, I'd manually check it.

aha! ok. Then go wild, my friend

LGTM :+1:

Updated my email, good to go 👍


Change web page icon

FEATURE:
Add support for setting web page icons, changing it from the default pglet icon.

Okay after looking at adding this and #133 I'm pretty lost as i've never touched TypeScript in my entire life haha, my initial idea to implement it was messy and confusing

No worries, I'll give a thought how it could be better implemented.


Fix npm install typo in contributing notes

The readme listed bower instead of babel.
Also, I added babel to the devDependencies packages so that it is easier to install for contributors.


fix/fivetran

fivetran_adwords_url_performance needs column names updated to be consistent with Fivetran output.

@ChrisLawliss looks like you worked on some fivetran updates previously - mind taking a look at this when you have a chance?


README.md: typo

"konw" to "know"
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for your contribution.


Build fails because include nghttp2/nghttp2.h is not found

Compilation failed on the include of #include <nghttp2/nghttp2.h>.
I had slightly changed cmake deps to use use protobuf 4.25.1.
Is there a misconfiguration on my part?
My build was:
$ git clone https://github.com/uatuko/grpcxx.git --recursive
$ cd grpcxx
$ vi cmake/dependencies.cmake
	...
	# protobuf
	set(Protobuf_DEBUG ON)
	set(Protobuf_INCLUDE_DIR /home/panta/.local/include)
	#find_package(Protobuf 3.15.0 CONFIG REQUIRED)
	find_package(Protobuf 4.25.1 CONFIG REQUIRED)
	...
$ rm -rf build ; mkdir -p build && cd build && cmake -DENABLE_LIB_ONLY=OFF -DGRPCXX_BUILD_EXAMPLES=on ..
$ make
...
[ 36%] Building CXX object _deps/nghttp2-build/src/CMakeFiles/nghttpx_static.dir/util.cc.o
/home/panta/grpcxx/build/_deps/nghttp2-src/src/util.cc:65:10: fatal error: nghttp2/nghttp2.h: No such file or directory
   65 | #include <nghttp2/nghttp2.h>
      |          ^~~~~~~~~~~~~~~~~~~
compilation terminated.
make[2]: *** [_deps/nghttp2-build/src/CMakeFiles/nghttpx_static.dir/build.make:76: _deps/nghttp2-build/src/CMakeFiles/nghttpx_static.dir/util.cc.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1426: _deps/nghttp2-build/src/CMakeFiles/nghttpx_static.dir/all] Error 2
make: *** [Makefile:136: all] Error 2

Thank you in advance!

Hey @panta, I don't see just changing to use Protobuf 4.25.1 causing the builds to fail on nghttp2.
But, it is possible the nghttp2 build fails while building other bundled executables. Can you try -DENABLE_LIB_ONLY=ON (or completely  removing -DENABLE_LIB_ONLY=OFF)?

You should also be able to use protobuf 4.25.1 by just using -DProtobuf_INCLUDE_DIR=/home/panta/.local/include cmake flag, without having to change cmake/dependencies.cmake.


Media& content permissions?

If I need to hire someone to add content, how would I address the following concerns:

Deleting/ Editing previous content.
Deleting existing media images.

What's the advice to overcome those concerns?

I've noticed that in the permission, there is only checkbox me media, but, how to prevent deleting media while allow adding media?

Any suggestion here?

@hopewise for now it is not possible to do it, I will add for the next versions.


Want to add link in surprise me button

Hey @adaharshsingh
I go through your website and like it ,
but I got idea to integrate your repo link in surprise button to star your repo
if you like my idea will you please assign me this issue .
Thank you!

I wish to contribute. Kindly assign me this.

@adaharshsingh ... Please assign this issue to me


Continiuns reboots

Hi..!!
I'm new and I'm started wiith this things and I have had a problem
In the line 39 in the Simple Example
float tempC = sensors.getTempCByIndex(0);
I by error I have put
float tempC = sensors.getTempC(0);
Arduino IDE 1.8.19 compile OK...and I can upload to the board
But I have continiuns reboots and I think that has broken the sensor
I have corrected the problem and a new sensor and all OK...the old always -127
Board ESP32 DevKit v1
Sensor DS18B20 (TO-92) in ONE_WIRE_BUS 26 with a simple little board
Is normal this issue?
Thanks
https://github.com/milesburton/Arduino-Temperature-Control-Library/blob/master/examples/Simple/Simple.ino

Good day,
This sounds like a power supply issue rather than a bug in the library. I'd suggest raising a post on the Arduino forum for general support, I'm sure they can advise you as needed.
To avoid confusion I'll close this ticket, if you think this is in error please let us know.
Thanks,
Miles

Hi...!!!
I also thought that but... you can see it in this video
I can't test in arduino
https://mega.nz/file/fZNCSCiD#TAyoRiiUz4_XtIDZsvTDAjr6TeLwIP2fxh_O4P0N-uA
[https://mega.nz/rich-file.png]https://mega.nz/file/fZNCSCiD#TAyoRiiUz4_XtIDZsvTDAjr6TeLwIP2fxh_O4P0N-uA
86.64 MB file on MEGAhttps://mega.nz/file/fZNCSCiD#TAyoRiiUz4_XtIDZsvTDAjr6TeLwIP2fxh_O4P0N-uA
mega.nz
Thanks..!!

De: Miles Burton @.>
Enviado: sábado, 23 de abril de 2022 7:33
Para: milesburton/Arduino-Temperature-Control-Library @.>
Cc: HugoCar77 @.>; Author @.>
Asunto: Re: [milesburton/Arduino-Temperature-Control-Library] Continiuns reboots (Issue #213)
Good day,
This sounds like a power supply issue rather than a bug in the library. I'd suggest raising a post on the Arduino forum for general support, I'm sure they can advise you as needed.
To avoid confusion I'll close this ticket, if you think this is in error please let us know.
Thanks,
Miles
—
Reply to this email directly, view it on GitHubhttps://github.com/milesburton/Arduino-Temperature-Control-Library/issues/213#issuecomment-1107369454, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AONWMNTJCK2VR4BQZ5ARLYTVGODS5ANCNFSM5UEBOU7Q.
You are receiving this because you authored the thread.Message ID: @.***>


URLs do not use OpenType features under LuaLaTeX

Description
When writing a URL that includes numbers, these will appear as lining figures regardless of which OpenType features are enabled. This is a very old bug, previously filed under Luaotfload, but the issue was determined not to rest there and it appears to have dropped off the radar.
Check/indicate

[X] Relevant for LuaTeX
[X] Issue tracker has been searched for similar issues?
[X] Links to <tex.stackexchange.com> discussion if appropriate

Minimal example demonstrating the issue
\documentclass{article}

\usepackage{unicode-math}
\setmainfont{TeX Gyre Pagella}[Numbers = {Lowercase, Proportional}]
\usepackage{url}
\urlstyle{same}

\begin{document}

<PHONE_NUMBER>

  \url{http://example.com/1234567890}

\end{document}

Further details
There is a workaround on StackExchange but it no longer works with newer versions of LuaLaTeX.

There was a workaround on StackExchange, but it no longer works with newer versions of LuaLaTeX.

Sverre's answer works fine for me (but you have to change  the font name, Libertine Serif imho no longer works.)

Sverre's answer works fine for me

Quite right, thank you – I've found the problem there, at least!

I don’t see a fontspec issue here, it seems to be bad interaction between url package (that uses math mode) and luaotfload’s node mode (that does not work in math mode).

Where then would one file a report on this? In spite of it providing core functionality, url appears to be unmaintained.


Adding basic Swedish translation.

There may be some minor errors in this translation, because in some cases it's not clear whether the English word is a verb or a noun (such as "copy") but it should be good enough.

Wonderful and thanks! Will merge in just a second.
Thanks for pointing out the verb / noun issue too. Going to add a to-do item for myself to write a translation guide to help with future efforts.
Cheers!


Pass in design temperature for rules engine

The rules engine requires the design temperature, which is a constant value that is looked up by county (or overridden by the user). The CSV file containing the design temperature data is located at https://github.com/codeforboston/home-energy-analysis-tool/tree/main/design_temp.

@plocket We should take these three APIs outputs and prepare something we can pass to python:
https://github.com/codeforboston/home-energy-analysis-tool/issues/20#issuecomment-1707485014

Current situation:
JS team is trying to get analysis data from pyodide, but gets an error that design_temperature isn't defined.
Plan:

JS team will pass in 12 for design_temperature to move forward with other development.
JS team will use the address to figure out what the shape of the geocode data is.
Python team will make a function that takes geo data and returns a design temperature.
When the Python team has figured out what they need from the data, they will create a function signature and the js team can make types and pass in data that matches the signature.


Update documentation for OFFSET and FETCH FIRST

Applied comments.

@electrum could you please review?

Merged, thanks!


Apache Dubbo Binding imports vulnerable UUID package

Hi @LaurenceLiZhixin,
Can you please consider upgrading the Apache Dubbo binding to a newer SDK version? The current one uses a vulnerable version of library github.com/satori/go.uuid
It is unclear if a newer version of that library will work with the Apache Dubbo binding at this time.

I opened this PR
https://github.com/apache/dubbo-go/pull/2181


Each row in sparse matrix is in reverse after indexing with array

Each row in sparse matrix is in reverse after indexing with array
Reproducing code example:
import scipy
import scipy.sparse
import numpy as np
boo = scipy.sparse.csr_matrix(np.random.random((2, 2)))
print np.all(boo[[0,1], :].data == boo.data)
print zip(boo[[0,1], :].data, boo.data)

Output:
False
[(0.5987041996888242, 0.36317937800775757), (0.36317937800775757, 0.5987041996888242), (0.6219931572555458, 0.7901446409249704), (0.7901446409249704, 0.6219931572555458)]

Scipy/Numpy/Python version information:
('1.0.0', '1.14.1', sys.version_info(major=2, minor=7, micro=6, releaselevel='final', serial=0))

Nothing is guaranteed about the data order after indexing.
Perhaps you meant
np.all(boo[[0,1], :].toarray() == boo.toarray())


Pleonastic Transposition?

Given 'unique PK' is tautological, should the 6th bullet point under Restrictions read:
"A sparse column cannot be part of a UNIQUE clustered index or a u̶n̶i̶q̶u̶e̶ primary key index."

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

ID: 46ab5559-d930-39c4-b27c-898d2883efe2
Version Independent ID: ac1acd3d-c154-2934-5daf-fbacab4df7ba
Content: Use Sparse Columns - SQL Server
Content Source: docs/relational-databases/tables/use-sparse-columns.md
Product: sql
Technology: table-view-index
GitHub Login: @stevestein
Microsoft Alias: sstein

@KevinPence -- thank you for your feedback.
@stevestein -- please look into this issue #4395.

@KevinPence - thanks for the feedback. Considering it's not technically wrong and has been this way for several versions we're going to leave it as is.
Thanks!
#please-close

@stevestein, From your response I'm not sure if you caught my entire question.  I should have been much more clear and direct instead of having fun with words.
I think your reply is to my observation that "unique" is redundantly applied to "primary key."  That is not the issue, per se.  It's just there to explain why I'm raising my real question.
I want clarity on whether sparse columns are not allowed in any clustered index or just unique ones.  Is the "unique" redundancy just a redundancy, or does the word belong with "clustered index" instead of "primary key"?
Thanks for considering.


ILogger implementation

As talked about in PR #157 replace the in-built logger for the Microsoft ILogger framework

Update the few places using a logger
Move LogLevel enum to native and update callback
Remove old logger class

Some nuget updates were required along the way

Should we be taking an ILogger<Something> everywhere instead of a plain ILogger?
I'm not very familiar with ILogger, but it looks like the convention is to take ILogger<ThisClassName> in the constructor for ThisClassName? I assume that would mean that you don't have to include [LLamaExecutor] in the text of the message and it would be automatically categorised by type for you.

We should, however that would force the end user to pass down those typed loggers, that was one of the concerns in my first PR.
The problem is without DI or a Factory pattern the typed loggers are clunky to use.

Ah sorry, I missed that in your first PR.
How "clunky" are we talking? Could you show an example?

Ah sorry, I missed that in your first PR.
How "clunky" are we talking? Could you show an example?

pudo code, but example, say context and executor both had typed loggers in the constructor, you would need to create to loggers to do simple tasks
var contextLogger = MyLoggerFactory.Create<LLamaContext>();
var executeLogger= MyLoggerFactory.Create<InstructExecutor>();

using var model = LLamaWeights.LoadFromFile(parameters);
using var context = model.CreateContext(parameters, contextLogger);
var executor = new InstructExecutor(context, executeLogger);```

using ILogger they can just pass down any logger they have

Oh I was imagining way worse than that! I think that's fine personally.
If people really hate then can easily work around it in the future (e.g. add another constructor that just accepts non-generic ILogger, or one which accepts an ILoggerFactory as you suggested), but I don't think that's likely to be needed.

Oh I was imagining way worse than that! I think that's fine personally.
If people really hate then can easily work around it in the future (e.g. add another constructor that just accepts non-generic ILogger, or one which accepts an ILoggerFactory as you suggested), but I don't think that's likely to be needed.

Then nesting also becomes an issue, if Executor creates a class that also has its typed logger, how do we pass it down?
I think it has to be ILogger? or we create the ILoggers and the user provides us thiet ILoggerFacory if they want to

The we could add a global Configure(ILoggerFactory, Numa, Cuda) an the user can just register in one place

Regret opening this can of worms :p

Agreed 😆
Let's just stick with ILogger as you have it in this PR - it's nice and simple and covers most of what we need for now.
I'll merge this if you agree?

yep, happy to megre and run, lol


Admonition support

It would be cool to support github's admonitions

@zefhemel I'd love to work on this. I've started digging through the code-base and existing plugs. I can see the syntax: section of a manifest where you can match custom markdown via a regex, but not sure how to hook in custom rendering. Can you give me a push in the right direction here?

Great! If my understanding is correct this is not actually a syntax extension, it's mostly about presenting existing syntax (block quotes or fenced code blocks) differently.
This is stuff that needs to happen in a CodeMirror plug-in. Here's one for block quotes: https://github.com/silverbulletmd/silverbullet/blob/main/web/cm_plugins/block_quote.ts in that same folder there's also one for code blocks. You'll likely have to special case one of those.

Obsidian seems to have a popular plug-in that uses fenced code blocks. May also be good to support: https://github.com/valentine195/obsidian-admonition

Ah no, Obsidian itself has "call out boxes" with again slightly different syntax 😃 great. https://help.obsidian.md/How+to/Use+callouts

Awesome, thanks for the links. Will get stuck into this and let you know how I go soon.
Also I think it would be good for this to eventually support different syntax'es as it seems there is not much agreement over which is best.

Hi @zefhemel, I've made some progress with this, but have hit a problem. I need to create a ParseTree instance from a fragment of raw markdown text, and this can be done via the syscall api like so:
import { parseMarkdown } from "../../plug-api/silverbullet-syscall/markdown.ts";

const t = await parseMarkdown(text);

But this is an async function, and the CodeMirror syntaxTree(state).iterate({enter: ...}) api does not support async.
I've attempted to put the contents of the enter callback in an async IIFE, but this throws an exception within the parseMarkdown function, something to do with the syscall:

Uncaught TypeError: B8 is not a function
at j8 (markdown.ts:6:10)

I can push up a WIP pull request if that helps.

A few things:
There are two "worlds" here: the regular world running JS directly as part of the server or browser with direct access to everything. This is the world of CM extensions like you're building.
There's also the world of plugs, an isolated, sandboxed environment where you can only do limited things and you access most things through syscalls like the parseMarkdown one you're using here.
The confusing thing is that both are written in TypeScript and live side by side (almost) in the code base.
What you're doing wrong here is make a syscall from non-plug code, which will not work regardless of being sync or async.
Second, what you're iterating through here is a lezer parse tree, which is similar to the psrseMarkdown style parse tree, but only contains positions of syntax nodes, not its actual text directly. You get the text value by slicing it from state.doc
An example is here: https://github.com/silverbulletmd/silverbullet/blob/850077bad0619d5d321bbcd4ca7589e1d51c5a22/web/cm_plugins/command_link.ts#L28
While it's possible to use parseMarkdown to get that sort of tree, I wouldn't recommend it. I'd just get the syntax tree's node text like in link and pull out the information you need with perhaps a regex.
That said, if you do really want to get that markdown tree from this environment you can. This is what lezerToParseTree does, an example is here: https://github.com/silverbulletmd/silverbullet/blob/850077bad0619d5d321bbcd4ca7589e1d51c5a22/web/cm_plugins/table.ts#L94
By the way, CM plug-ins always are synchronous so doing async stuff will never be an option.

Cheers for the info. One of the reasons I went down this approach is for nested markdown within the admonition, but architecturally this approach won't work as per your explanation.
I'm now approaching things differently. Since the existing QuoteMark node already supports nested markdown I'm looking to augment the existing DOM with CSS classes and extra elements for icon and title. This approach looks promising so far, will update you soon.

Yes, that sounds right to me 👍

@zefhemel have pushed a WIP PR, please take a look when you get a chance. #186


Installation facts are not always collected

There are several occasions of installations that don't have any facts in the callhome db.
Need to understand why do they fail to send them.

More than half of the cases where no facts are collected can be explained by the following scenario:

User starts lakeFS with an empty DB.
User runs setup through the API or UI (and not the CLI).
User does not restart lakeFS ever.

Since we only collect metadata when the server starts (or when the setup runs from the CLI) - metadata for this installation is never sent.


[api] Refactoring FeatureHelper

Moving API specific code from app/helpers/feature_helper.rb to the API Controller's features.rb file.

@gtanzillo @imtayadeway please review.  Thanks.

:+1:

:+1: LGTM


Acess denied issue

I tried importing my beatmap packs with the program and everytime i'm getting hit with acess denied. Tried running cmd/program as admin and messed around with folder permissions but no success. No idea how i can fix that would u mind pointing me in the right direction even though it is user error and not program issue.
E:\Downloads\xd>OsuPackImporter.exe "E:\Downloads\xd\nomod consistency pack" -v
INFO: Importing archive E:\Downloads\xd\nomod consistency pack

Unhandled exception. System.UnauthorizedAccessException: Access to the path 'E:\Downloads\xd\nomod consistency pack' is denied.
   at Microsoft.Win32.SafeHandles.SafeFileHandle.CreateFile(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options)
   at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize)
   at System.IO.Strategies.OSFileStreamStrategy..ctor(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize)
   at System.IO.Strategies.FileStreamHelpers.ChooseStrategyCore(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize)
   at System.IO.Strategies.FileStreamHelpers.ChooseStrategy(FileStream fileStream, String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, Int64 preallocationSize)
   at System.IO.File.OpenRead(String path)
   at OsuPackImporter.Collections.ExtendedCollection..ctor(String path, ProgressContext context)
   at OsuPackImporter.Program.<>c__DisplayClass6_0.<Run>b__0(ProgressContext ctx)
   at Spectre.Console.Progress.<>c__DisplayClass25_0.<Start>b__0(ProgressContext ctx) in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 65
   at Spectre.Console.Progress.<>c__DisplayClass27_0.<<StartAsync>b__0>d.MoveNext() in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 98
--- End of stack trace from previous location ---
   at Spectre.Console.Progress.<>c__DisplayClass28_0`1.<<StartAsync>b__0>d.MoveNext() in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 133
--- End of stack trace from previous location ---
   at Spectre.Console.Internal.DefaultExclusivityMode.RunAsync[T](Func`1 func) in /_/src/Spectre.Console/Internal/DefaultExclusivityMode.cs:line 40
   at Spectre.Console.Progress.StartAsync[T](Func`2 action) in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 116   at Spectre.Console.Progress.StartAsync(Func`2 action) in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 96
   at Spectre.Console.Progress.Start(Action`1 action) in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 69
   at OsuPackImporter.Program.Progress(Action`1 action)
   at OsuPackImporter.Program.Run(Options options)
   at OsuPackImporter.Program.<>c.<Main>b__4_0(Options o)
   at CommandLine.ParserResultExtensions.WithParsed[T](ParserResult`1 result, Action`1 action)
   at OsuPackImporter.Program.Main(String[] args)

From what I can deduce by the filename, you are trying to import a folder?
THe program doesn't support importing folders (yet?), I should probably add guards against trying to import folders for the time being.
Try zipping the folder and import that instead.

Hmm, ty for the reply, seems to go further now than access denied but still stuck on other error message. I would assume it means beatmap pack is too big ?
E:\Downloads\xd>OsuPackImporter.exe "E:\Downloads\xd\nomod consistency pack.7z"
INFO: Importing archive E:\Downloads\xd\nomod consistency pack.7z

.

Unhandled exception. System.IO.IOException: Stream was too long.
   at System.IO.MemoryStream.Write(Byte[] buffer, Int32 offset, Int32 count)
   at System.IO.Stream.CopyTo(Stream destination, Int32 bufferSize)
   at System.IO.Strategies.BufferedFileStreamStrategy.CopyTo(Stream destination, Int32 bufferSize)
   at System.IO.FileStream.CopyTo(Stream destination, Int32 bufferSize)
   at System.IO.Stream.CopyTo(Stream destination)
   at OsuPackImporter.Collections.ExtendedCollection.GetArchive(Stream stream)
   at OsuPackImporter.Collections.ExtendedCollection.Parse(ProgressContext context)
   at OsuPackImporter.Collections.ExtendedCollection..ctor(Stream stream, String name, ProgressContext context)
   at OsuPackImporter.Collections.ExtendedCollection..ctor(String path, ProgressContext context)
   at OsuPackImporter.Program.<>c__DisplayClass6_0.<Run>b__0(ProgressContext ctx)
   at Spectre.Console.Progress.<>c__DisplayClass25_0.<Start>b__0(ProgressContext ctx) in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 65
   at Spectre.Console.Progress.<>c__DisplayClass27_0.<<StartAsync>b__0>d.MoveNext() in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 98
--- End of stack trace from previous location ---
   at Spectre.Console.Progress.<>c__DisplayClass28_0`1.<<StartAsync>b__0>d.MoveNext() in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 133
--- End of stack trace from previous location ---
   at Spectre.Console.Internal.DefaultExclusivityMode.RunAsync[T](Func`1 func) in /_/src/Spectre.Console/Internal/DefaultExclusivityMode.cs:line 40
   at Spectre.Console.Progress.StartAsync[T](Func`2 action) in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 116   at Spectre.Console.Progress.StartAsync(Func`2 action) in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 96
   at Spectre.Console.Progress.Start(Action`1 action) in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 69
   at OsuPackImporter.Program.Progress(Action`1 action)
   at OsuPackImporter.Program.Run(Options options)
   at OsuPackImporter.Program.<>c.<Main>b__4_0(Options o)
   at CommandLine.ParserResultExtensions.WithParsed[T](ParserResult`1 result, Action`1 action)
   at OsuPackImporter.Program.Main(String[] args)

For the record, how big the file is?

2.87GB uncompressed

Yup, that might be it. MemoryStream doesn't seem, to support buffers more than 2 GB by default.
I might need to find an alternative to that.
I don't have access to a Windows box as of right now but it could be useful if you could attach your compressed pack, so that I could check against it later.

https://drive.google.com/file/d/1BHuxOgakjm9iTLro21zJpWE8Oq4mw4Ce/view?usp=sharing
This should work. As for current issue i'm just gonna split the pack in 2 and then merge collections

I think the original issue is solved I guess, I'll open a new issue for the file size.


[stable/nextcloud] nextcloud 16, health config settings

What this PR does / why we need it:

upgrades to nextcloud 16
makes use of new env vars (update, smtp) - redis will follow
small improvements

Which issue this PR fixes

fixes #14090

Special notes for your reviewer:
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/lgtm


Leaderboard: final touches

In this branch, I have added the following features:

[ ] Adding the styles of my choice to improve the look and feel of the application.
[ ] I kept the general layout of the wireframe.

Thank you!


Added support for passing the endpoint in the push stanza

I'm thinking of moving to request since the built in https library is kinda tricky.

@davidchiles for module.exports.sendMessage = function (endpoint, token, message, cb) { the endpoint parameter is currently optional and overrides some sort of defaults included with the server configuration. Should we support both and fall back, or support endpoint only? I think omitting the endpoint should probably be an error because it could result in requests to the wrong endpoint.

I'm all for just using the passed in endpoint but we need to update the XMPP clients. For now having a default fallback works


Community Problem

When i open the community build i get a black screen and nothing happens no errors nothing.

You probably just need to wait for it to load. Are you able to click on the window and get it to capture your mouse?


[question] Checkpoints not saved?

Running with --save-freq 2500 - and I can't seem to find checkpoints anywhere.. Where should I expect to find my models?
Code example
#!/bin/bash
mkdir -p /data
python train.py --algo ppo --env Robocode-v2 \
                           --gym-packages gym_robocode \
                           --tensorboard-log /data/stable-baselines/  \
                           --save-freq 2500 \
                           -f /data/logs \
                           --optimization-log-path /data/opt_logs\

Everything is in the logs directory. I read through the colab notebook and things made more sense.

Where at in logs? Cant seem to find it

Where at in logs? Cant seem to find it


Use doubles for startTime and endTime

don’t lose precision by doing integer arithmetic

It looks like the resolution of the timer is a factor of 10 less granular exclusively on node.js v12 in 64-bit.  Does anyone know why?

Hello, @jaredp , thank you for contribution.
I need some time to test this on compatibility with DevTools frontend in node-inspector.
About test - it is very speculative. Testing in cloud containers like Travis and Appveyor can't guaranty what this async operation will be finished in time below 3*delay.
I prefer to see here only least checker.

Hi @3y3, thanks for taking a look.
The test failed because profiler reported 1.5469829999999547 seconds for what should have been a delay of 0.15 seconds.  We should expect on a box that's not too busy to see an elapsed time a little over 0.15 seconds but likely less than 0.16 seconds; 0.15469... seconds would be perfectly expected.  On an AppVeyor box busy with other things, we can expect to see an elapsed time a coefficient more, but this is exactly an order of magnitude, so I think the reporting is accurate, and for some reason the resolution is different in this particular config.

@jaredp , 1.5469829999999547 is a real timeout for this case. In other words iteration on AppVeyor and on Travis can be very slow sometimes. We can't expect what elapsed time will be in interval 150ms < elapsedTime < 160ms. We also can't expect 150ms < elapsedTime < 150h.
I prefer to merge this change without new test.

Okay, removed the < 350ms test

ping

bump


zef is not installed on windows-latest

Having a script run:
run "zef", "install", "."

results in:
Failed to spawn process zef: no such file or directory (error code -4058)

whereas that works fine on ubuntu-latest and MacOS-latest.
Example: https://github.com/lizmat/sourcery/actions/runs/9709948595/job/26799645336

@patrickbkr @skaji

Maybe
run "cmd.exe", "/x/d/c", "zef", "install", ".";

works.

In windows, zef is actually zef.bat.

Ahhh... ok, so $*DISTRO.is-win ?? 'zef.bat' !! 'zef' indeed does the trick.  Thanks!  Closing.

I think this is not the real issue and won't help. AFAI recall, Windows auto completes the '.bat' (in addition to loads of other file extensions) when calling a filename without an extension. This might not be true for all API functions that perform a program call, but for the typical ones it is.
On June 28, 2024 4:26:38 PM GMT+02:00, Elizabeth Mattijsen @.***> wrote:

Ahhh... ok, so $*DISTRO.is-win ?? 'zef.bat' !! 'zef' indeed does the trick.  Thanks!  Closing.
--
Reply to this email directly or view it on GitHub:
https://github.com/Raku/setup-raku/issues/36#issuecomment-2197067746
You are receiving this because you were mentioned.
Message ID: @.***>

FWIW, it does appear like it does fix the issue: https://github.com/lizmat/sourcery/actions/runs/9714371905 .
So it looks like it doesn't for run?


greynoise community api worker

Greynoise.io community API worker
https://developer.greynoise.io/reference/community-api

As with abuseipdb worker; we should discuss vocabulary for classification

example output from API
{     "ip": "<IP_ADDRESS>",     "noise": true,     "riot": false,     "classification": "malicious",     "name": "unknown",     "link": "https://viz.greynoise.io/ip/<IP_ADDRESS>",     "last_seen": "2021-05-17",     "message": "Success" }

closing. Reopen when we find a good way to "tag" IP addresses with a "type"


dont require page title to retrieve a notion page

If I try to retrieve a page that doesn't have title set, then page.properties is undefined, causing an error. Just checking for properties seems to fix it for me.

Codecov Report

Merging #342 (74c369b) into master (240ca0d) will decrease coverage by 0.04%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #342      +/-   ##
==========================================
- Coverage   99.69%   99.64%   -0.05%
==========================================
  Files         247      230      -17
  Lines        2595     2541      -54
  Branches      361      354       -7
==========================================
- Hits         2587     2532      -55
- Partials        8        9       +1

Flag
Coverage Δ

cache
98.28% <ø> (ø)

constants
100.00% <ø> (ø)

core
99.39% <50.00%> (-0.12%)
:arrow_down:

discourse
100.00% <ø> (ø)

endpoints
100.00% <ø> (ø)

errors
100.00% <ø> (ø)

extract
100.00% <ø> (ø)

fabricator
99.75% <ø> (ø)

graphql
100.00% <ø> (ø)

idz
?

init
?

inline_blocks
100.00% <ø> (ø)

lineage
100.00% <ø> (ø)

logger
100.00% <ø> (ø)

notion_formula
100.00% <ø> (ø)

operations
100.00% <ø> (ø)

permissions
100.00% <ø> (ø)

sync
100.00% <ø> (ø)

tasks
100.00% <ø> (ø)

traverser
100.00% <ø> (ø)

utils
100.00% <ø> (ø)

validators
100.00% <ø> (ø)

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

Impacted Files
Coverage Δ

packages/core/libs/Api/Nishan.ts
98.03% <50.00%> (-1.97%)
:arrow_down:

packages/init/libs/comment.ts

packages/init/libs/View/sort.ts

packages/init/libs/View/aggregation.ts

packages/idz/libs/Transform/transformToUuid.ts

packages/init/libs/spaceView.ts

packages/init/libs/View/filter.ts

packages/init/libs/blockMetadata.ts

packages/idz/libs/Transform/transformToId.ts

packages/idz/libs/Validate/index.ts

... and 8 more

Continue to review full report at Codecov.

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


[20.03] symmetrica: 2.0 -> 3.0.1

Motivation for this change
This fixes a regression caused by the glibc update. See https://github.com/NixOS/nixpkgs/pull/82239 for more details.
The major version bump is just because the project changed ownership. This is essentially a bugfix and cleanup release. Further, sage is its only consumer. Therefore I think it is fine to include in 20.03 (since it fixes a regression).
@worldofpeace do I have your okay for this backport?
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@disassembler just noticed @worldofpeace has a travel notice up. Is this backport okay? Keep in mind that

the release changes no functionality as far as I'm aware
sage is symmetrica's only consumer and is broken without this update. I doubt anybody was using symmetrica directly.
it would be possible to cherry-pick only the one bugfix that unbreaks sage, but that would probably leave other bugs and just be additional effort.

Changelog indeed seems to be pure-cleanup from a brief skimming.

Barring any objections, I'll take the liberty to merge this this weekend. I don't see any disadvantages and would like to get this in before the official 20.03 release. The release managers probably have their hands full.


OnLogin Event is never Fired, nothing happens

LoginAsync() is triggered, testing on Android..
A box pops open to select a User to continue with, after that nothing happens ..
OnLogin is never fired, no permissions are requested although the Scope is filled.
CrossGoogleClient.Current.OnLogin += CrossGoogleClient_OnLogin;
var googleUser = CrossGoogleClient.Current.LoginAsync();
Why doesn't anything happen after the account selection ?
Did I miss something ?? I followed the Setup as described.

Hey @XceSx6,
Did you get a look at the Sample project on the repo? could you verify that you have the last version and that the iOS and Android setup are completed correctly?
You could use the sample project to test that is working by swapping out the .json file, and adding your own client ID and .json file

Hey Luis,
I did look at the Sample project, but after replacing the JSON and testing it still does nothing, same as before.. Click Login but nothing happens.
On my own project I followed the Guidelines on GitHub, and when i click Login on my button it does popup an Account selector, but then nothing.
So I don’t know what I’m doing wrong here :x
Once i get through my Login, I can help @nguyenthanhliemfc https://github.com/nguyenthanhliemfc  with using the ActiveToken to authenticate his Google Services.
Kind regards, we can Skype if you like, its<EMAIL_ADDRESS>From: Luis Pujols<EMAIL_ADDRESS>Sent: Thursday, 23 May, 2019 23:42
To: CrossGeeks/GoogleClientPlugin<EMAIL_ADDRESS>Cc: Dirk Bourgeois<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [CrossGeeks/GoogleClientPlugin] OnLogin Event is never Fired, nothing happens (#34)
Hey @XceSx6 https://github.com/XceSx6 ,
Did you get a look at the Sample project on the repo? could you verify that you have the last version and that the iOS and Android setup are completed correctly?
You could use the sample project to test that is working by swapping out the .json file, and adding your own client ID and .json file
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/CrossGeeks/GoogleClientPlugin/issues/34?email_source=notifications&email_token=AECDMY7POL3NZG5LPC2BHSTPW4FS3A5CNFSM4HPGV7U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWDSFXI#issuecomment-495395549 , or mute the thread https://github.com/notifications/unsubscribe-auth/AECDMY72ZJZ27WQEWMCFFZTPW4FS3ANCNFSM4HPGV7UQ .  https://github.com/notifications/beacon/AECDMYZGQNOV4NVPIXGRQADPW4FS3A5CNFSM4HPGV7U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWDSFXI.gif

Problem Solved, by configuring correctly the project following the guides.

And the guide is?

I agree with @breula , no idea what the Fix is..
Although following the Guide, the OnLogin event never fires.

I agree with @breula , no idea what the Fix is..
Although following the Guide, the OnLogin event never fires.

After install all Nupackages installed in the sample, etc.. etc.. At least now is firing but not "Plugin.GoogleClient.Shared.GoogleClientBaseException: '10: '" Same happens running the sample.

Yep. simple does not works.. The Facebook one works at least. Les't go try to find another way, if succed i will post solution here.

Done, I used in a Command as:
public async void LoginWithGoogle()
{
CrossGoogleClient.Current.OnLogin += (s, a) =>
{
switch (a.Status)
{
case GoogleActionStatus.Canceled:
App.Current.MainPage.DisplayAlert("Cancelado", a.Message, "OK");
break;
case GoogleActionStatus.Unauthorized:
App.Current.MainPage.DisplayAlert("Não autorizado", a.Message, "OK");
break;
case GoogleActionStatus.Completed:
break;
case GoogleActionStatus.Error:
App.Current.MainPage.DisplayAlert("Erro", a.Message, "OK");
break;
default:
break;
}
};
}
Plus add "Entitlements.plist" as "BundleResource" in Build Action and choosen "Entitlements.plist" on Custom Entitlements for IOS Project
And...
private void OnLoginCompleted(object sender, GoogleClientResultEventArgs<Plugin.GoogleClient.Shared.GoogleUser> loginEventArgs)
{
if (loginEventArgs.Data != null)
{
Plugin.GoogleClient.Shared.GoogleUser googleUser = loginEventArgs.Data;
//User.Name = googleUser.Name;
//User.Email = googleUser.Email;
//User.Picture = googleUser.Picture;
var GivenName = googleUser.GivenName;
var FamilyName = googleUser.FamilyName;
            // Log the current User email
            Debug.WriteLine(googleUser.Email);
            IsLoggedIn = true;

            var token = CrossGoogleClient.Current.AccessToken;
            Token = token;
        }
        else
        {
            App.Current.MainPage.DisplayAlert("Error", loginEventArgs.Message, "OK");
        }
        googleClientManager.OnLogin -= OnLoginCompleted;
    }

IOS works 100%
Android till got Erro "10;"


MH-13141 Correctly initialize stats service

Steps to reproduce:

Open the Admin UI in your browser
Maybe reload the page (forcing full reload to prevent caching)

Actual Results:
Sometimes, the Admin UI Counters on the Events page don't work.
Expected Results:
The counters should work correctly
Workaround (if any):
Reload page until it works
Analysis:
The problem is the result of incorrect component initialization: The statsService is configured asynchronously which causes it to be used before it has been initialised.

@staubesv Before I merge this: Can you add the Jira Ticket number to your commits and maybe squash minor commits into one?

@KatrinIhler I have intentionally done this with several commits so that it is documented what has been changed. I don't think that it is important to have the JIRA ticket numbers as the pull request already has this.
Generally I highly recommend to not change GIT commits.


What performance can be expected with Pyleus?

I've been using Pyleus on a server with two 2.6 GHz AMD Opteron processors and 4GB of RAM, and get about 1500 tuples/second throughput. I'm using Kafka for input and limited the input speed to that because if I go any higher the CPU gets saturated. I'm wondering if that's normal or sounds really slow. For regular Storm I understand that it would be pretty slow, but there's a lot of overhead from stdin/stdout communication with shell bolts, is that right?
Basically I'd like to know if Pyleus is quite a bit slower than Storm+Java or if I'm doing something wrong?

I haven't any official data to share, since performance is really dependent on the topology and the workflow you are using to measure them, but there is no doubt that Pyleus is slow compared to pure Java Storm topologies.
However, the major pain point is not Pyleus, but the Storm multilang framework. On top of the performance loss due to Python being notably slower than Java and stdin/out communication overhead, you need to add the time spent in an additional serialization/deserialization step, so that tuples can be read by any non-JVM language (which is not possible if you leave tuple in kryo format,b as they are encoded in Java Storm).
We already tried to mitigate the issue adding a msgpack serializer, which considerably boosts performance if compared to the default one based on JSON that Storm provides.
An easy tweak to improve performance is to set need_task_ids to False when emitting from bolts. This avoid you a read/write network operation for each tuple you emit (it's a tip written in the docs https://yelp.github.io/pyleus/storm/bolt.html#pyleus.storm.bolt.Bolt.emit). This only works if your topology doesn't care about tasks ids, of course.
You can also try to tweak stuff like ackers, max_spout_pending, message_timeout_secs and max_shellbot_pending in your topology definition yaml file (https://yelp.github.io/pyleus/yaml.html), to play with components parallelism or to disable guaranteed message processing mechanisms.
In addition to that, any customization like user-defined JVM options may help you as well. For any optimization specifically related to Storm, the Storm documentation and mailing list are your friends.
Having said that, I am fairly sure that, due to the presence of the msgpack parser, Pyleus is way faster than a vanilla Python topology developed using the Python API available in Storm, but it is still far from the performance you can obtain writing your topology in Java.

Thanks for that detailed answer! I understand that it's not a Pyleus issue but it's good to get a confirmation that Java topologies can be quite a bit faster.
I already use need_task_ids=False for all my emits and I think ackers etc. are configured well. Unfortunately I need to use the JSON serializer because I had issues with msgpack, so I guess that explains a bit of it. Thanks again!

You're welcome. I'm going to close the issue, if my comment answers your question. Feel free to reopen it or to update in case you find out anything useful to share.


Can't you add another language? For example, Korean.

Can't you add another language? For example, Korean.

Feel free to reopen if you come up with a question.


[CODEOWNERS] Updates for org changes

Summary
The focus of these changes is to remove an individual who no longer is responsible for the products which their GH account is associated to.
Details

Move Event Grid ownership to the engineering team

Remove Relay, as there is no Azure SDK package for it owned by the Azure SDK team and we should be redirecting support for it and not opening issues here.

/check-enforcer override