[Performance] Slow loading started conversations

BotMan Version: dev-master
PHP Version: 7.1.5
Messaging Service(s): *
Cache Driver: Redis, File

Description:
A performance problem was detected while loading a conversation that was initially started, more commands are recorded more it becomes slow.
https://github.com/mpociot/botman/blob/master/src/Traits/HandlesConversations.php#L160
This specific line tooks about ~ 3 seconds with 15 command registered
This was recorded in debug mode, thats why took up to 9 sec

Update:
I made a test bot that has the same number of command of my real bot, and a dummy conversation. At every request also the bot sends to you the time that the method listen() took.
In my real bot, my conversations works via interactive reply (buttons), and the user never need to write a single letter. Via text written by users, conversations work well, and via interactive buttons, they behave the way that i've reported.
Here the test bot code: https://gist.github.com/SergiX44/23805394a350e546939eaaeae6b0cfa5
The timings using inline buttons:

And with plain text:

Update 2:
This seems and issue with the Telegram Driver

as you can see, foreach iteration in the getMatchingMessages method the getConversationAnswer is called, and if is a callback_query generated from a button, it update the original message again and again, flooding the telegram api and wasting a lot of time making useless requests. This should done only one time when the message is matched.


[AutoPR reservations/resource-manager] [Hub Generated] Review request for Microsoft.Capacity to add version preview/2019-04-01

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5715

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/4608


Responder Lifecycle props cannot be set via setNativeProps

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[ ] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
Environment:
OS: macOS 10.14
Node: 10.11.0
Yarn: 1.10.1
npm: 6.4.1
Watchman: 4.9.0
Xcode: Xcode 10.0 Build version 10A255
Android Studio: 3.1 AI-173.4907809
Packages: (wanted => installed)
react: 16.3.1 => 16.3.1
react-native: ~0.55.2 => 0.55.4
Description
The Responder Lifecycle props cannot be set via setNativeProps.
Reproducible Demo
https://snack.expo.io/BJLEH-I27

same issue with 0.57 version


Config start offset in setting does not usable.

I start a consumer to consume a topic, I have set the start offset earliest, but the consumer does not receive all messages.

Hi - couple of questions:

What version of Kafka re you using?
Are you activating consumer via explorer or the action menu?

Kafka version 2.1
Via explorer

Thanks for the info - i'll get back when I have some news!

Better late than never (and Christmas holidays 🙏 ). I just published 0.8.2 which should fix this. Please let me know if it doesen't work as expected.


Geometry kickoff

So we are trying to implement different geometry types into myxql. We do not have extensive postgis knowledge, we were using Mariaex for quite a while and then suddenly needed MultiPolygon, which was not included.
After many, many hours to try and get our head around the binary pattern of the internal MySQL geo format, and using external libs (Geo mix package) we finally had a small breakthrough in parsing a single point without errors!
Next up are all other types, for us, in the following order of importance:

[x] Point (more tests?)
[ ] Polygon
[ ] MultiPolygon

Then for completeness:

[ ] LineString
[ ]  Geometry
...

The first thing for the polygon we need to figure out is where the number of rings/points inside the polygon is stored. For that I think we can look into Mariaex which has support for Polygons. Then, hopefully, the bridge to MultiPolygon will not be too difficult.
Let us know what bit you would like to work on, so we do not overlap.

Small update: I got further in parsing points and polygons, but I am now really struggling with the multipolygons. I understand they have to be decoded in a nested way, with num_polygons but fail to get a good result.

@m13m another update: we can parse MultiPolygons already. It might be a good idea to let me further look into the decoding part, maybe you can check out the encoding part? Maybe it will just work, have not tested yet. In the end, we should be able to have the tests like all the other value_tests, they seem to do a roundtrip test: encode <> persist <> decode <> check.

I am right now at a Code BEAM India can't find time to look into the code
but I will get free hopefully on Monday.
On Fri, 15 Nov 2019, 2:03 am Jeroen Bourgois<EMAIL_ADDRESS>wrote:

@m13m https://github.com/m13m another update: we can parse
MultiPolygons already. It might be a good idea to let me further look into
the decoding part, maybe you can check out the encoding part? Maybe it will
just work, have not tested yet. In the end, we should be able to have the
tests like all the other value_tests, they seem to do a roundtrip test:
encode <> persist <> decode <> check.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jackjoe/myxql/issues/1?email_source=notifications&email_token=ABZNM4CYG55XT7IZFMVQP7DQTWY2ZA5CNFSM4JMVV6XKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEDGKZA#issuecomment-554067300,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZNM4C3KM25SNO7V7RACC3QTWY2ZANCNFSM4JMVV6XA
.

--
Mohd Maqbool Alam
www.maqbool.net

@m13m no problem, have a great time.
Next update is that we seem to be able to parse Point, Polygon and MultiPolygon ok, but there is something off with fetching a whole row with multiple fields. I must have done something to screw up the nested processing of the row, since once one of the new geometry fields is decoded, all consecutive fields are nested withing that.
So, on it's own, it looks like this (output from the test per type):
.%MyXQL.Geometry.Polygon{
  coordinates: [
    [[{0.0, 0.0}, {10.0, 0.0}, {10.0, 10.0}, {0.0, 10.0}, {0.0, 0.0}]]
  ],
  srid: nil
}
%MyXQL.Geometry.MultiPolygon{
  coordinates: [
    [
      [
        [{0.0, 0.0}, {10.0, 0.0}, {10.0, 10.0}, {0.0, 10.0}, {0.0, 0.0}],
        [{0.0, 0.0}, {10.0, 0.0}, {10.0, 10.0}, {0.0, 10.0}, {0.0, 0.0}]
      ]
    ]
  ],
  srid: 0
}

But than actually used in our project, with some data hidden, this is the output out of the custom Ecto type module we have, inside the load method:
>> POLY data
%MyXQL.Geometry.MultiPolygon{
  coordinates: [
    [
      [
        [
          {4.66921649135855, 51.160921488999},
          {4.6691053278458, 51.160896370051},
          {4.66905305838084, 51.1609878206652},
          {4.66906408541997, 51.1610292525988},
          {4.66907301882503, 51.1610628178101},
          {4.66909153637943, 51.161132394939},
          {4.6691506376173, 51.1611265294804},
          {4.66914650241002, 51.161108665092},
          {4.66919635887044, 51.1611035342213},
          {4.66918299789029, 51.1610547355332},
          {4.66916397147712, 51.1610133763588},
          {4.66916917878279, 51.1610042656877},
          {4.66921649135855, 51.160921488999}
        ]
      ],
      %MyXQL.Geometry.Point{
        coordinates: {4.66912842534954, 51.1610110967104},
        srid: nil
      },
      nil,
      "POI_city",
      "POI_nbr",
      "POI_street
      "POI_test_name",
      4
    ]
  ],
  srid: 0
}

The order of the fields is not that relevant, but as you can see, inside the MultiPolygon, inside the coordinates list, after the actual coords in the nested list are all other columns of the fetched row, which is wierd :D We will look into that, because it seems we are getting real close. Though we have thought that numerous times already... :)

Will close this and make more specific tickets


Add Rule to Show Invalid Field If has te format 0,80 instead of 0.80 in amount fields

Decimals are not allowed anymore in roy and revs.


[Bug]: @oclif/core dependency is duplicated when using NPM

Please confirm that you have:

[X] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
Other
Expected behavior
The @oclif/core dependency should be unique.
Actual behavior
In latest versions, we can find both @oclif/core@2 and @oclif/core@3 versions installed when using NPM.

The @oclif/core dependency has been recently bumped to v3. However, the following dependencies in @shopify/cli are still 1 version behind and they internally rely on v2, so it also gets installed: https://github.com/Shopify/cli/blob/44ada880bde37319d0cafc6957340e4fc4df0abf/packages/cli/package.json#L98-L100
This is not a problem in this monorepo because PNPM is overwriting the @oclif/core version: https://github.com/Shopify/cli/blob/44ada880bde37319d0cafc6957340e4fc4df0abf/pnpm-lock.yaml#L10
However, when using NPM to install<EMAIL_ADDRESS>we end up with the two versions installed.
It looks like the following versions of the packages are aligned with<EMAIL_ADDRESS>    "@oclif/plugin-commands": "3.1.0",
    "@oclif/plugin-help": "6.0.9",
    "@oclif/plugin-plugins": "4.2.6",

I've added this in Hydrogen to fix inconsistencies:
https://github.com/Shopify/hydrogen/blob/5c054f0bc09d73cab9d9977c035feac01e4b43a2/package.json#L94-L99
However, I'm not very familiar with the dependencies to know if we can bump them without creating other issues.

Reproduction steps

npm i
npm why @oclif/core should show 2 versions installed

Operating System
Mac latest
Shopify CLI version (check your project's package.json if you're not sure)
3.56.2
Shell
zsh
Node version (run node -v if you're not sure)
No response
What language and version are you using in your application?
No response

Hi @frandiox !
You are right, i'm working on upgrading oclif dependencies but we are waiting for a new version of some of those packages with some changes we need. After that i'll update all of them :)

Still relevant (I think?) -- @isaacroldan I see the version of @oclif/core is still overwritten in this repo but the dependencies have been updated. Is still needed to overwrite the oclif version?

All oclif dependencies have been updated to use the same version of core


[SQL-DS-CACHE-112] [POAE7-1055] Refactor agg push down rule

What changes were proposed in this pull request?
refactor agg pushdown rule, fix a bug when read same table in different nodes, all nodes will contain agg info.
Also add a quick work around to avoid group num exceeds batch size issue.
How was this patch tested?
E2E test. 21/22 TPCH query passed. (wo/ this PR, only 15/22 passed)

I will merge this one and fix q22 case in later PR.


[WIP] Added OpenStack LoadBalancer classes support

Co-authored-by: Uwe Krueger<EMAIL_ADDRESS>What this PR does / why we need it:
In order to incorporate the support of LoadBalancer classes of the out-of-tree cloud provider for OpenStack (https://github.com/kubernetes/cloud-provider-openstack/pull/619), this PR allows you to configure the corresponding LB classes via the CloudProfile and Shoot spec.
Special notes for your reviewer:
This is currently a WIP. Since official cloud provider image has not been released yet, we will test it first with a local build version for now.
Release note:
Added LoadBalancer classes support for OpenStack

Validation tests are still missing.

We will hopefully merge #1111 soon. It doesn't make sense to merge this one now because it would postpone #1111 and produce additional effort for the colleagues that are implementing the OpenStack extension controller.
Can you open a PR against the gardener-extensions repo and do these adaptions in the OpenStack controller?
Can you adapt this PR to only include the API changes that are required in the CloudProfile and Shoot?
Thank you!

I guess you are referring to #1076. Agreed that would make things worse. Then we will shrink this PR to just the spec changes in the CloudProfile and Shoot.


iTunes no longer supports IPA installation

This doc is out of date
https://docs.microsoft.com/en-us/xamarin/ios/deploy-test/app-distribution/ipa-support?tabs=windows#installing-an-ipa-using-itunes
As iTunes no longer contains an Apps section, as of 12.7

iTunes 12.7 removed the My Apps section.
Now, instead of the procedure you describe at the end (Installing an IPA using iTunes), you drag and drop the IPA file from the Finder onto the device in the iTunes window:

If iTunes says that it didn't install the app because the app already exists, delete the app from the device. This is Apple's official way:
https://support.apple.com/en-us/HT207618

Thanks for submitting an issue and we apologize for the delayed response. The requested updates haven't been made since the creation of the issue, and the timeline for resolution may vary based on resourcing, so the issue is being moved to an internal repo to better track it.


raylib-php && raylib-phpcpp support 3.5

Both PHP extensions support RayLib 3.5

@joseph-montanez Great! Thanks for the update! :D


Support querying WHOIS for information about IP blocks/addresses

WHOIS is very useful for determining who owns/operates the network that defines a given IP address, and roughly determining its location (since that's how IP blocks are assigned).
The same issues parsing WHOIS results for DNS domains apply to WHOIS results for IP addresses/subnets, e.g. ARIN uses a different format than JPNIC....
This codebase probably needs to be slightly refactored to support this, e.g.: instead of directly mapping a substring of the input to a set of per-domain regexps, just do the WHOIS query and then try to determine the registrar (and their regexp map) by parsing either the whois server name output by "whois --verbose" (not sure how portable this is) or the WHOIS service identification in response text.
For example, queries that end up going to JPNIC result in verbose output starting with "Using server whois.nic.ad.jp.", and the responses include this header:

[ JPNIC database provides information regarding IP address and ASN. Its use   ]

Queries going to ARIN produce verbose output starting with "Using server whois.arin.net.", and the responses include this header text:

#
# ARIN WHOIS data and services are subject to the Terms of Use

I suggest to use the existing https://pypi.org/project/ipwhois/ for all ip and ASN lookup's rather than implement it in this library.
It is trivial anyway to see if something is a ip or asn rather then a domain so a switch between different libraries should not be a obstacle. As a side note: https://pypi.org/project/ipwhois/ already seems to use the new RDAP standard.


Forge Server Can't Connect With Plato's Transporters Installed

Greetings,
I set up a forge server and it works for the few mods my friend and I have been using (list below), but when I add Platos Transporters mod, we are rejected from the server with an error of an incomplete set of tags. I copied the file directly from the forge server and explored the jar file to make sure the tags weren't wiped out some how. They are in the jar file.
When I remove the mod from the client side (remains on server), I can connect with no issue but the mod does not work. When I removed the mod from the server side (remains on client) the issue appears. That strikes me as though the server is not properly loading the mod. I am able to play a single player game with the mod list below without issue and all mods work.
Please let me know if you need more information, I am new to Minecraft so I may need a little guidance.
Minecraft Version: 1.16.4
MOD List:
Better Weather
Optifine
BiomesOPlenty
JEI
Platos Transporters
Xaeros Minimap
Server side log:
[21:13:03] [main/INFO]: Environment: authHost='https://authserver.mojang.com', accountsHost='https://api.mojang.com', sessionHost='https://sessionserver.mojang.com', servicesHost='https://api.minecraftservices.com', name='PROD'
[21:13:03] [main/WARN]: Ambiguity between arguments [teleport, destination] and [teleport, targets] with inputs: [Player, 0123, u/e, dd12be42-52a9-4a91-a8a1-11c01849e498]
[21:13:03] [main/WARN]: Ambiguity between arguments [teleport, location] and [teleport, destination] with inputs: [0.1 -0.5 .9, 0 0 0]
[21:13:03] [main/WARN]: Ambiguity between arguments [teleport, location] and [teleport, targets] with inputs: [0.1 -0.5 .9, 0 0 0]
[21:13:03] [main/WARN]: Ambiguity between arguments [teleport, targets] and [teleport, destination] with inputs: [Player, 0123, dd12be42-52a9-4a91-a8a1-11c01849e498]
[21:13:03] [main/WARN]: Ambiguity between arguments [teleport, targets, location] and [teleport, targets, destination] with inputs: [0.1 -0.5 .9, 0 0 0]
[21:13:03] [main/INFO]: Reloading ResourceManager: Default
[21:13:04] [Worker-Main-13/INFO]: Loaded 7 recipes
[21:13:04] [Worker-Main-13/INFO]: Loaded 927 advancements
[21:13:05] [Server thread/INFO]: Starting minecraft server version 1.16.4
[21:13:05] [Server thread/INFO]: Loading properties
[21:13:05] [Server thread/INFO]: Default game type: SURVIVAL
[21:13:05] [Server thread/INFO]: Generating keypair
[21:13:05] [Server console handler/ERROR]: Exception handling console input
java.io.IOException: The handle is invalid
at java.io.FileInputStream.readBytes(Native Method) ~[?:1.8.0_271]
at java.io.FileInputStream.read(Unknown Source) ~[?:1.8.0_271]
at java.io.BufferedInputStream.read1(Unknown Source) ~[?:1.8.0_271]
at java.io.BufferedInputStream.read(Unknown Source) ~[?:1.8.0_271]
at sun.nio.cs.StreamDecoder.readBytes(Unknown Source) ~[?:1.8.0_271]
at sun.nio.cs.StreamDecoder.implRead(Unknown Source) ~[?:1.8.0_271]
at sun.nio.cs.StreamDecoder.read(Unknown Source) ~[?:1.8.0_271]
at java.io.InputStreamReader.read(Unknown Source) ~[?:1.8.0_271]
at java.io.BufferedReader.fill(Unknown Source) ~[?:1.8.0_271]
at java.io.BufferedReader.readLine(Unknown Source) ~[?:1.8.0_271]
at java.io.BufferedReader.readLine(Unknown Source) ~[?:1.8.0_271]
at zg$1.run(SourceFile:95) [minecraft_server.1.16.4.jar:?]
[21:13:05] [Server thread/INFO]: Starting Minecraft server on REMOVED BY OP
[21:13:05] [Server thread/INFO]: Using default channel type
[21:13:06] [Server thread/INFO]: Preparing level "world"
[21:13:06] [Server thread/INFO]: Preparing start region for dimension minecraft:overworld
[21:13:07] [Server thread/INFO]: Preparing spawn area: 0%
[21:13:07] [Server thread/INFO]: Preparing spawn area: 0%
[21:13:07] [Server thread/INFO]: Time elapsed: 886 ms
[21:13:07] [Server thread/INFO]: Done (0.945s)! For help, type "help"
[21:14:32] [User Authenticator #1/INFO]: UUID of player Joe
[21:14:32] [Server thread/INFO]: REMOVED BY OP logged in with entity id 68 at (-2283.7293015580317, 67.0,<PHONE_NUMBER>2917853)
[21:14:32] [Server thread/INFO]: Joe joined the game
[21:14:33] [Server thread/INFO]: Joe lost connection: Disconnected
Client Side Log (snippet where I think problem may be, can provide full if requested):
13Feb2021 21:14:33.407] [Render thread/INFO] [mezz.jei.util.LoggedTimer/]: Starting JEI took 317.4 ms
[13Feb2021 21:14:33.412] [Render thread/WARN] [net.minecraft.client.network.play.ClientPlayNetHandler/]: Incomplete server tags, disconnecting. Missing: {minecraft:block=[platos:boat_material_blacklist, platos:boat_material, platos:scytheable]}
[13Feb2021 21:14:33.802] [Render thread/INFO] [net.optifine.shaders.SMCLog/]: [Shaders] Program loaded: shadow
[13Feb2021 21:14:33.804] [Render thread/INFO] [net.optifine.shaders.SMCLog/]: [Shaders] Program loaded: gbuffers_basic
[13Feb2021 21:14:33.808] [Render thread/INFO] [net.optifine.shaders.SMCLog/]: [Shaders] Shader info log: /shaders/gbuffers_textured.fsh
0(543) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(544) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(545) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(546) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(547) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(548) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(550) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(551) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(552) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(553) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(555) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(557) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(559) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(561) : warning C7555: 'varying' is deprecated, use 'in/out' instead
0(598) : warning C7533: global variable gl_FragData is deprecated after version 120
[13Feb2021 21:14:33.809] [Render thread/INFO] [net.optifine.shaders.SMCLog/]: [Shaders] Info log: gbuffers_textured

This a strange error that I don't think existed in previous forge versions. you may want to check the previous version of Plato's just in case that works, I would also suggest trying without optifine altogether as it generally causes more problems then solutions in 1.16+ there is other performance mods like foamflower available. I'm.not fully able to do updates now due to various things but I will check to see if there is anything obviously wrong in my build

this may be fixed now


Missing requirement

Was building the drawers and after 4 of them it suddenly said Missing requirement, though I stil had plenty of fine wood.
The hammer still worked etc.

Also, if picking up items from the drawer the game doesn't seem to recognise that I have them in the inventory when trying to build


fix(react-query): fix useSuspenseQueries broken result type (#7118)

The type inference inside GetSuspenseResults<T> is missing SkipToken as a possible return type for queryFn.
This causes inference of the returned results of useSuspenseQueries to not work, defaulting to a TData of unknown.
This PR just quickly adds SkipToken in the remainder of the code to allow inference to work again.
Fixes issue #7118.

Thanks. Can you please:

add a type test so that we make sure this doesn't break again in the future. useQueries.test-d.tsx should be the right place
can you please add the fix also to other adapters? The types for useQueries are pretty much copy-pasted to all adapters (yes, this can be simplified ... somehow, somewhen)

@TkDodo this PR is not active for 8 days, so I've opened another PR with tests: https://github.com/TanStack/query/pull/7194
I believe this fix is not relevant for other adapters because they don't have useSuspenseQueries and createQueries don't have this issue


Feature: specificy hostname and port

This fixes #9 but #10 should be merged first.

Sorry, but i don't want to break existing apps and why host part should be different from JID host part?

Because the domain in the jid, and the hostname can be two different things. Say that we have the domain chat.teletronics.ae, and we have it balanced over four nodes. They would be called: chat1.teletronics.ae, chat2.teletronics.ae, chat3.teletronics.ae, chat4.teletronics.ae ...
Then you would login with the jid<EMAIL_ADDRESS>on for example chat1.teletronics.ae.
This PR wont actually break things for the existing implementations. I've put that parameters at the end of the connect method. So for existing implementations, the hostname and port will just follow the default(Which is to take the hostname from jid, and port 5222).


Version bump to 3.0.3

Changelog:

Includes #274 to fix the repeat notifications bug

Cool! Glad to have this fixed.
On Fri, May 29, 2015 at 1:43 PM, felt<EMAIL_ADDRESS>wrote:

Merged #275 https://github.com/GoogleChrome/experience-sampling/pull/275
.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleChrome/experience-sampling/pull/275#event-317458703
.

I actually uploaded the fixed version to the CWS a few days ago but forgot to add the new version to the repo. So phew. :)


Overhaul examples

Two motivations for this change:

It is now easy to make simulated hardware that returns realistic data for testing analysis workflows when the beam is down. See docstrings for suggestive examples.

The examples.py has built up a lot of cruft as did not serve as a nice entry point for developers trying to understand the bluesky interface.

Still to do (in a later PR): unify the three different implementations of simulated flyers.

Current coverage is 82.63% (diff: 80.76%)

Merging #553 into master will decrease coverage by 0.04%

@@             master       #553   diff @@
==========================================
  Files            21         21
  Lines          4107       4032    -75
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           3396       3332    -64
+ Misses          711        700    -11
  Partials          0          0

Powered by Codecov. Last update 1d49569...6e7213a

Tests pass; coverage is mad merely because I deleted code.

👍 👍


reads the empty contacts?

i am wanted to that cannot read the empty contacts but it reads the empty contacts ?

Hey @Talha137m , if I understood you correctly, you receive some contacts without any meaningful/visible information except for the contact ID. If that's the case, I can think of the following potential causes:

fields parameter in a getAllContacts(fields:) lacks the fields, which the problematic contacts are missing.
problematic contacts don't specify or specify fields, supported by fast_contacts, as empty.

Anyway, fast_contacts doesn't filter any contacts based on whether they contain any human-readable information. As long as the contact is returned for a given query by the OS - it will appear in the fast_contacts response.


stress matrix - support for docker args / env variables

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

Resolves #4305

A couple small suggestions but otherwise looks good. Please add docs and a new example scenario (or an update to an existing network example) that illustrates consuming the build args in the dockerfile with matrix.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Flag any project with a new issue that has not seen any activity for 48+ Hours

If a user submits a new issue to a project, and there is no activity on the issue within 48 hours, flag the project.

cf https://github.com/mnapoli/IsItMaintained


LUCENE-10670: Add a codec class to track merge time of each index part

This PR add filter class for each index format classes and their writer/producer/reader/consumer and add a tracking codec class to track merge time of each index part.
Jira Issue Link

The idea seems good - we want to track merge times separately for each format, right? I wonder if the Filter classes belong in a monitoring package though. It's also a confusing name since we already have a monitor package which is something completely different. Perhaps a better place for the Filter* classes would be org.apache.lucene.codecs.filter (in core module), and then we could put the MergeTimeTrackingCodec in the sandbox module somewhere? Maybe in o.a.l.sandbox.metrics? (to avoid confusion with monitor)

Yes, it use to track merge time for each format. Thanks for the suggestion, I will make revision accordingly.

this will actually slow down the merge heavily, by preventing things like optimized bulk merges of stored fields.
I really don't think we should be doing this with a codec-wrapper. you can get this data alrady from InfoStream!

I really don't think we should be doing this with a codec-wrapper. you can get this data alrady from InfoStream!

InfoStream is not very structured though -- it's hard to see how you could extract merge times from it. You'd have to parse formatted Strings, right? It also has this comment in its javadoc: NOTE: Enabling infostreams may cause performance degradation in some components.

dude, that's nothing compared to the cost of disabling bulk merge, which will also force all stored fields to be recompressed every merge as well.
sorry, i'm completely against this PR.

Thanks for helping out here, @rmuir - can you explain how it is that bulk operations would be prevented by this change? I don't really understand, but maybe if you explained we could figure out what to do about it.

it uses instanceof: https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/codecs/lucene90/compressing/Lucene90CompressingStoredFieldsWriter.java#L686
there's nothing to "do about it", except to say, don't wrap entire codec if you want to add a stats interface. Add a stats interface if you want a stats interface.

it uses instanceof: https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/codecs/lucene90/compressing/Lucene90CompressingStoredFieldsWriter.java#L686

Thanks for sharing that link @rmuir. Intrigued, I did a quick search for instanceof in the codecs/, and found such checks in more than one place.
It seems that when we add a functionality in a new codec version, we use such type equality checks to ensure that consuming code is compatible. Adding a generic codec wrapper that branches from the inheritance chain will break such checks. Replacing such checks is a whole other discussion and not really the problem we are solving here.
I like the idea of making SegmentMerger implement some sort of a Stats interface that can return a metrics object. We can populate the TimeMetric object in SegmentMerger.merge() as each codec's merge returns. It solves what is needed here, and is easy to extend.

I like the idea of making SegmentMerger implement some sort of a Stats interface that can return a metrics object. We can populate the TimeMetric object in SegmentMerger.merge() as each codec's merge returns. It solves what is needed here, and is easy to extend.

I would suggest IndexWriter instead. it is already the public interface, just expose some counters. SegmentMerger is an implementation detail. In fact there are already some counters there (e.g. flushCount), but only exposed for testing purposes.

@rmuir Thanks for pointing this out. I will close this PR and investigate other way.


DevOps plugin for Jenkins problem

I wasn't able to use DevOps plugin for Jenkins to integrate with my Jenkins.

Request:
Method: POST
URI: http://ado1.eastasia.cloudapp.azure.com//team-events/gitPush
HTTP Version: 1.1
Headers:
{
  X-VSS-ActivityId: 73102e8f-a9e8-0001-4f0c-fb6d8f1cd501
  X-VSS-SubscriptionId: 00000000-0000-0000-0000-000000000000
  User-Agent: VSServices/16.152.29003.6
  User-Agent: (w3wp.exe)
  Authorization: Basic ********
  Jenkins-Crumb: a84242d9da6165466e70237e8a3006ab
  Content-Type: application/json; charset=utf-8
}
Content:
{"id":"03c164c2-8912-4d5e-8009-3707d5f83734","eventType":"git.push","publisherId":"tfs","message":{"text":"Jamal Hartnett pushed updates to Fabrikam-Fiber-Git:master.","html":"Jamal Hartnett pushed updates to Fabrikam-Fiber-Git:master.","markdown":"Jamal Hartnett pushed updates to `Fabrikam-Fiber-Git`:`master`."},"detailedMessage":{"text":"Jamal Hartnett pushed a commit to Fabrikam-Fiber-Git:master.\n - Fixed bug in web.config file 33b55f7c","html":"Jamal Hartnett pushed a commit to Fabrikam-Fiber-Git:master.\n
\n
Fixed bug in web.config file 33b55f7c\n
","markdown":"Jamal Hartnett pushed a commit to [Fabrikam-Fiber-Git](https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_git/Fabrikam-Fiber-Git/):[master](https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_git/Fabrikam-Fiber-Git/#version=GBmaster).\n* Fixed bug in web.config file [33b55f7c](https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_git/Fabrikam-Fiber-Git/commit/33b55f7cb7e7e245323987634f960cf4a6e6bc74)"},"resource":{"commits":[{"commitId":"33b55f7cb7e7e245323987634f960cf4a6e6bc74","author":{"name":"Jamal Hartnett","email":"fabrikamfiber4@hotmail.com","date":"2015-02-25T19:01:00Z"},"committer":{"name":"Jamal Hartnett","email":"fabrikamfiber4@hotmail.com","date":"2015-02-25T19:01:00Z"},"comment":"Fixed bug in web.config file","url":"https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_git/Fabrikam-Fiber-Git/commit/33b55f7cb7e7e245323987634f960cf4a6e6bc74"}],"refUpdates":[{"name":"refs/heads/master","oldObjectId":"aad331d8d3b131fa9ae03cf5e53965b51942618a","newObjectId":"33b55f7cb7e7e245323987634f960cf4a6e6bc74"}],"repository":{"id":"278d5cd2-584d-4b63-824a-2ba458937249","name":"Fabrikam-Fiber-Git","url":"https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_apis/git/repositories/278d5cd2-584d-4b63-824a-2ba458937249","project":{"id":"6ce954b1-ce1f-45d1-b94d-e6bf2464ba2c","name":"Fabrikam-Fiber-Git","url":"https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_apis/projects/6ce954b1-ce1f-45d1-b94d-e6bf2464ba2c","state":"wellFormed","visibility":"unchanged","lastUpdateTime":"0001-01-01T00:00:00"},"defaultBranch":"refs/heads/master","remoteUrl":"https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_git/Fabrikam-Fiber-Git"},"pushedBy":{"displayName":"Jamal Hartnett","id":"00067FFED5C7AF52@Live.com","uniqueName":"fabrikamfiber4@hotmail.com"},"pushId":14,"date":"2014-05-02T19:17:13.3309587Z","url":"https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_apis/git/repositories/278d5cd2-584d-4b63-824a-2ba458937249/pushes/14"},"resourceVersion":"1.0","resourceContainers":{"collection":{"id":"c12d0eb8-e382-443b-9f9c-c52cba5014c2"},"account":{"id":"f844ec47-a9db-4511-8281-8b63f4eaf94e"},"project":{"id":"be9b3917-87e6-42a4-a549-2bc06a7a878f"}},"createdDate":"2019-06-11T06:26:51.4129483Z"}

Response:
Status Code: 400
Reason Phrase: Bad Request
HTTP Version: 1.1
Headers:
{
  Connection: keep-alive
  X-Content-Type-Options: nosniff
  X-Error-Message: Invalid event
  Date: Tue, 11 Jun 2019 06:26:52 GMT
  Server: nginx
  Content-Length: 6957
  Content-Type: text/plain; charset=utf-8
}

Event:
{
    "id": "03c164c2-8912-4d5e-8009-3707d5f83734",
    "eventType": "git.push",
    "publisherId": "tfs",
    "message": {
        "text": "Jamal Hartnett pushed updates to Fabrikam-Fiber-Git:master.",
        "html": "Jamal Hartnett pushed updates to Fabrikam-Fiber-Git:master.",
        "markdown": "Jamal Hartnett pushed updates to `Fabrikam-Fiber-Git`:`master`."
    },
    "detailedMessage": {
        "text": "Jamal Hartnett pushed a commit to Fabrikam-Fiber-Git:master.\n - Fixed bug in web.config file 33b55f7c",
        "html": "Jamal Hartnett pushed a commit to Fabrikam-Fiber-Git:master.\n
\n
Fixed bug in web.config file 33b55f7c\n
",
        "markdown": "Jamal Hartnett pushed a commit to [Fabrikam-Fiber-Git](https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_git/Fabrikam-Fiber-Git/):[master](https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_git/Fabrikam-Fiber-Git/#version=GBmaster).\n* Fixed bug in web.config file [33b55f7c](https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_git/Fabrikam-Fiber-Git/commit/33b55f7cb7e7e245323987634f960cf4a6e6bc74)"
    },
    "resource": {
        "commits": [
            {
                "commitId": "33b55f7cb7e7e245323987634f960cf4a6e6bc74",
                "author": {
                    "name": "Jamal Hartnett",
                    "email"<EMAIL_ADDRESS>                    "date": "2015-02-25T19:01:00Z"
                },
                "committer": {
                    "name": "Jamal Hartnett",
                    "email"<EMAIL_ADDRESS>                    "date": "2015-02-25T19:01:00Z"
                },
                "comment": "Fixed bug in web.config file",
                "url": "https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_git/Fabrikam-Fiber-Git/commit/33b55f7cb7e7e245323987634f960cf4a6e6bc74"
            }
        ],
        "refUpdates": [
            {
                "name": "refs/heads/master",
                "oldObjectId": "aad331d8d3b131fa9ae03cf5e53965b51942618a",
                "newObjectId": "33b55f7cb7e7e245323987634f960cf4a6e6bc74"
            }
        ],
        "repository": {
            "id": "278d5cd2-584d-4b63-824a-2ba458937249",
            "name": "Fabrikam-Fiber-Git",
            "url": "https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_apis/git/repositories/278d5cd2-584d-4b63-824a-2ba458937249",
            "project": {
                "id": "6ce954b1-ce1f-45d1-b94d-e6bf2464ba2c",
                "name": "Fabrikam-Fiber-Git",
                "url": "https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_apis/projects/6ce954b1-ce1f-45d1-b94d-e6bf2464ba2c",
                "state": "wellFormed",
                "visibility": "unchanged",
                "lastUpdateTime": "0001-01-01T00:00:00"
            },
            "defaultBranch": "refs/heads/master",
            "remoteUrl": "https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_git/Fabrikam-Fiber-Git"
        },
        "pushedBy": {
            "displayName": "Jamal Hartnett",
            "id"<EMAIL_ADDRESS>            "uniqueName"<EMAIL_ADDRESS>        },
        "pushId": 14,
        "date": "2014-05-02T19:17:13.3309587Z",
        "url": "https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_apis/git/repositories/278d5cd2-584d-4b63-824a-2ba458937249/pushes/14"
    },
    "resourceVersion": "1.0",
    "resourceContainers": {
        "collection": {
            "id": "c12d0eb8-e382-443b-9f9c-c52cba5014c2"
        },
        "account": {
            "id": "f844ec47-a9db-4511-8281-8b63f4eaf94e"
        },
        "project": {
            "id": "be9b3917-87e6-42a4-a549-2bc06a7a878f"
        }
    },
    "createdDate": "2019-06-11T06:26:51.412Z"
}

The response says:

"HTTP Error 403"
Problem accessing //team-events/gitPush. Reason: No valid crumb was included in the request

I also tried this on Postman.  The response HTML shows:

        <meta http-equiv="Content-Type" content="text/html;charset=utf-8"/>
        <title>Error 403 No valid crumb was included in the request</title>

HTTP ERROR 403

Problem accessing //team-events/gitPush. Reason:

    No valid crumb was included in the request

        Powered by Jetty:// 9.4.z-SNAPSHOT

What can I do?

Hi,
We couldn't reproduce the issue. Can you try selecting Trigger Generic Build  option for Perform this action filed and let us know if this option works?
Thanks

I finally found the problem.  My Jenkins base URL have a slash (/) suffix that cause Jenkins produce HTTP 400 Bad Request response.

I too facing this issues it shows 404 unauthorized. what is the exact solution for this error?


Fix the timeout logic in the waitForEvent integration test helper function

Debugging #17931, by printing all parts of the event lifecycle including timestamps, uncovered that some events for which a timeout was logged actually did get triggered correctly in the browser. Going over the code and discovering https://stackoverflow.com/questions/47107465/puppeteer-how-to-listen-to-object-events#comment117661238_65534026 showed what went wrong: if the event we wait for is triggered then Promise.race resolves, but that doesn't automatically cancel the timeout. The tests didn't fail on this because Promise.race resolved correctly, but slightly later once the timeout was reached we would see spurious log lines about timeouts for the already-triggered events.
This commit fixes the issue by canceling the timeout if the event we're waiting for has triggered.
Note that this patch gets rid of the incorrect timeout logs, but this doesn't solve the problem of the correct timeout logs yet (but this patch is preparatory work for follow-up patches that address that problem).
Fixes a part of #17931.

/botio integrationtest


python310Packages.pipdeptree: 2.7.0 -> 2.7.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.pipdeptree is: Command line utility to show dependency tree of packages
meta.homepage for python310Packages.pipdeptree is: https://github.com/tox-dev/pipdeptree
meta.changelog for python310Packages.pipdeptree is: https://github.com/tox-dev/pipdeptree/releases/tag/2.7.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.7.1 with grep in /nix/store/g1s8z2n4r4nlx0ykyaxclgz3d6nqp93z-python3.10-pipdeptree-2.7.1
found 2.7.1 in filename of file in /nix/store/g1s8z2n4r4nlx0ykyaxclgz3d6nqp93z-python3.10-pipdeptree-2.7.1

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

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.cairo-lang
python310Packages.pipdeptree
python311Packages.cairo-lang
python311Packages.pipdeptree

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/g1s8z2n4r4nlx0ykyaxclgz3d6nqp93z-python3.10-pipdeptree-2.7.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/g1s8z2n4r4nlx0ykyaxclgz3d6nqp93z-python3.10-pipdeptree-2.7.1
ls -la /nix/store/g1s8z2n4r4nlx0ykyaxclgz3d6nqp93z-python3.10-pipdeptree-2.7.1/bin

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

4 packages built:

python310Packages.cairo-lang
python310Packages.pipdeptree
python311Packages.cairo-lang
python311Packages.pipdeptree

Maintainer pings
cc @charlesbaynham for testing.

Lgtm. I haven't tested but these are very minor changes.


Mn wallet crashes if reindexed for txindex=1

Current masternode win64 wallet version v<IP_ADDRESS>-g3d5c6b6-beta crashes on start if blocks were reindexed after txindex=1 in spreadcoin.conf.
How to reproduct:

Run wallet, wait for syncronization, exit.
Add txindex=1 to \Roaming\SpreadTestCoin\spreadcoin.conf
Run wallet, confirm for "You need to rebuild ..."
Wait for rebuild.
Exit and start again.

Will show:
"MinGW Euntime Assertion
Assertion failed!
Program: ..
File: src/masternodes_elected.cpp, Line 103
Expression: masternodes.erase(outpoint) != 0"

Fixed
https://github.com/spreadcoin-project/spreadcoin/commit/59dd83a782dbadf79a8b5332945700f86eb5e1cd


Sfclient stat

tracking bytes to backing store
tracking label serialization
tracking label-tracking (declassify, endose, implies, can-flow-to, taint) latencies


Add config editor deployment bundle

Ah, now I realize this is closed. Did it already get merged, or was there a change of plans?


Add exported IsNullable method to nullable types

I was very happy to see https://github.com/graph-gophers/graphql-go/pull/430 merged and am now using it. However, in my project, I felt that having an exported IsNullable method would be useful, especially when implementing Nullable custom scalar types.
This will allow one to verify that custom Nullable implementations correctly implement the NullUnmarshaller interface.
It also allows us to verify the type of the input field for things like gRPC field masks.
E.g., previously, if one had a traditional nullable field, we would not be able to generate gRPC field masks accurately for partial updates because if a field was nil we would not add the field mask. Now, users can check if a field IsNullable, and if it is, check if the Set field is true or false, in order to set a field mask accurately.
Hopefully this method is safe to export; it would be helpful for our project and I hope it will be useful to others who want to take advantage of the new Nullable functionality.

Hi @Kalbi,
all these types already have a public Nullable() {} method. This means that you can create an interface that has this method and test for yourself if the type implements it. I am not convinced that we need to expose an additional static method IsNullable. This doesn't add any functionality that is not already available. At the cost of one additional interface, you can check for yourself.


Use the remapped code coverage summary for code coverage threshold check

Codecov Report

Merging #499 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #499   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          54     54
  Lines        1686   1707   +21
  Branches      249    253    +4
=====================================
+ Hits         1686   1707   +21

Flag
Coverage Δ

#builder
100% <100%> (ø)
:arrow_up:

#runtime
100% <ø> (ø)
:arrow_up:

#srcapp
100% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

config/karma/shared.karma.conf.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@Blackbaud-BobbyEarl This should be ready for another look.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-k8s-24-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2167

Questions ?
Please refer to the Backport tool documentation


Add writing style guidelines

Opening for visibility for @SwiftWeekly/writers 😄

Going to merge, but if anyone has feedback here, please share and we can update! 😄

This is awesome. Thank you so much, @jessesquires 💪


Replicated control plane issues with docs

Bug description
Step says: istioctl manifest apply \     -f install/kubernetes/operator/examples/multicluster/values-istio-multicluster-gateways.yaml
The install directory no longer exists. Replaced with manifests/examples/multicluster/values-istio-multicluster-gateways.yaml, and fails like #23253:
istioctl manifest apply \
    -f manifests/examples/multicluster/values-istio-multicluster-gateways.yaml
Error: failed to apply manifests: validation errors (use --force to override):
unknown field "security" in v1alpha1.Values

this should be fixed by https://github.com/istio/istio/pull/23304

please reopen if the issue still exists.


Great work!

Im currently trying to integrate opencv in python with godot to share data extracted from frames and the image frames itself.
The problem is that I need the fastest way to copy ( or share ) image data to the godot texture. Although python has a shared memory access, there is no way to retrieve the data back in godot.
I was thinking about using C# for this as there is a net lib with shared memory manager, but I didnt tested it yet.

This project uses gdtype-python library which acts as translation layer between GDScript and Python data structures. If you want to use GdScript then IMO there is no more convenient (and still fast) way to communicate with external processes. Sure, the gdtype-python could be improved on Python side to achieve faster transfers or by implementing OpenCV specialized structures.
If you want to be as fast as possible then implementing Godot plugin in C++ and using OS layer is one possibility (then the same C++ code could be used in Python in form of Python binding).


Support user facing mechanism for applying third party Settings plugins

According to @oehme during a conversation I had on slack, supposedly you can also have Settings and Gradle plugins.
Currently, Settings for settings.gradle.kts doesn't allow a user to specify a plugins block inside of it.
There needs to be a mechanism added to support allowing users to specify
Expected Behavior
Should be able to specify something like this in a settings.gradle.kts file.
plugins {
    id ("my.plugin.name") version "1.2.34"
}

Current Behavior
The above doesn't compile nor do I think it would work in Groovy either.
Context
I've found myself pasting this blob of code in all of my Gradle projects I've been creating both for work and for open source projects. I'd love to make it one Gradle plugin that I can just apply and be done with.
/**
 * This configures the gradle build so we can use non-standard build file names.
 * Additionally, this project can support sub-projects who's build file is written in Kotlin.
 *
 * @param project The project to configure.
 */
fun configureGradleBuild(project: ProjectDescriptor) {
    val projectBuildFileBaseName = project.name

    val gradleBuild = File(project.projectDir, "$projectBuildFileBaseName.gradle")
    val kotlinBuild = File(project.projectDir, "$projectBuildFileBaseName.gradle.kts")
    assert(!(gradleBuild.exists() && kotlinBuild.exists())) {
        "Project ${project.name} can not have both a ${gradleBuild.name} and a ${kotlinBuild.name} file. " +
                "Rename one so that the other can serve as the base for the project's build"
    }
    project.buildFileName = when {
        gradleBuild.exists() -> gradleBuild.name
        kotlinBuild.exists() -> kotlinBuild.name
        else ->
            throw AssertionError("Project `${project.name}` must have a either a file " +
                    "containing ${gradleBuild.name} or ${kotlinBuild.name}")
    }

    // Any nested children projects also get configured.
    project.children.forEach { configureGradleBuild(it) }
}

configureGradleBuild(rootProject)

Your Environment
Gradle version 4.10.

Okay, upon further inspection, this is valid.
buildscript {
    // plugin added to classpath here
}

That being said, the plugin API is still missing.
Also, the fact that you can create plugins for Settings is, as far as I'm aware, completely undocumented.

Use case: https://github.com/GradleWeaver/better-build-file-names

This is now resolved in Gradle 6.0! Woot!
@ldaley @big-guy We should remove this bit of documentation:
https://docs.gradle.org/current/userguide/plugins.html#sec:build_scripts_only

This feature is available in Gradle 6+


Can't seek after adding a track to the queue

Describe the bug
I'm trying to start a track from a specific position where the user left of, but it seems like there are no properties for specifying the start position either when adding a track to the queue or when playing.
I also tried adding a track and then seeking to the position, but the seek does not work immediately after adding a track.
To Reproduce

Add a track to the queue:

await TrackPlayer.add([
      {
        id: 'id',
        url: 'some url'
        title: 'Title',
        duration: 180,
        description: 'Description',
      },
    ]);

Seek to a position

 await TrackPlayer.seekTo(120);

Check the position of the player

  console.warn(await TrackPlayer.getPosition());

Prints: 0
Environment (please complete the following information):
System:
    OS: macOS 11.2.3
    CPU: (8) x64 Apple M1
    Memory: 20.24 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 15.6.0 - ~/.nvm/versions/node/v15.6.0/bin/node
    Yarn: 1.17.3 - /usr/local/bin/yarn
    npm: 7.4.0 - ~/.nvm/versions/node/v15.6.0/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.10.0 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 14.4, DriverKit 20.2, macOS 11.1, tvOS 14.3, watchOS 7.2
    Android SDK: Not Found
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>77832
    Xcode: 12.4/12D4e - /usr/bin/xcodebuild
  Languages:
    Java: javac 14 - /usr/bin/javac
    Python: 3.7.6 - /Users/okarlsson/opt/miniconda3/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: 16.13.1 => 16.13.1
    react-native: ~0.63.4 => 0.63.4
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

React native track player version:
1.2.7
Are you testing on a real device or in the simulator?
Simulator
Notes
Seems like this is caused by some timing issue, I'm able to seek if I put it in a timeout like this:
setTimeout(async () => {
 await TrackPlayer.seekTo(120);
}, 2000)

i dont really know why this seekTo problem hasn't been attended to yet. I have read from different platforms too about yet no solution have been found. what are the contributors doing about it? My problem is on android, seekTo function restarts the song.

On iOS seekTo immediatelly resolves in the RN bridge.
The problem is that seekTo is an asynchronous operation.
This code shows that seekTo is asynchronous. This means that bridge
https://github.com/react-native-kit/react-native-track-player/blob/69a22cd059415c51e473a2f7cb55b7f679672af6/ios/RNTrackPlayer/Vendor/SwiftAudio/Classes/AVPlayerWrapper/AVPlayerWrapper.swift#L165-L175
Bridge:
https://github.com/react-native-kit/react-native-track-player/blob/69a22cd059415c51e473a2f7cb55b7f679672af6/ios/RNTrackPlayer/RNTrackPlayer.swift#L423-L428
Bridge behavior is incorrect on iOS. Although it might also be the initial setup that is incorrect. The seekTo after the setup might not contact the underlying player.

Another issue is that AVPlayer.seek shouldn't be called if currentTrack?.status != .readyToPlay.

Any updates on this bug? I'm facing the same issue that once after adding a new audio to the TrackPlayer, calling seekTo() function will restart the audio.

Thanks for the input here everyone. I've added this to our Project and put it in high priority. I'll review it with the other contributors too.
@t-tornado we would love your help or a PR if you can contribute too? We are flat chat :)
Cheers
Brad


Fix ImagesServiceFailureException by introducing DisplayBlobstoreServlet to serve images

In Comment class, replace the imageUrl with imageString, the keyString of the image stored in Blobstore.
In Datastore, change imageUrl property of Comment Entities to imageString.
Add DisplayBlobServlet to serve images stored in Blobstore and avoid ImagesServiceFailureException .
A GET request is sent to the servlet when an image object is created in createImageElement(imageString).
The string BlobKey of the image to be displayed is stored in the query string of the request as "blob" parameter.

Apologies for the delay in getting around to reviewing this!


Maybe better default for charset in respondWrite

Consider if the default value of the charset parameter should be extracted from the content-type header instead of being hardcoded to UTF-8. Something like:
suspend fun ApplicationCall.respondWrite(charset: Charset = response.headers["content-type"]?.let { ContentType.parse(it).charset() } ?: Charsets.UTF_8, writer: suspend Writer.() -> Unit) {
    respond(WriterContent(writer, charset))
}

Right, I think we actually need to change it to content-type altogether, and make respondText and respondWrite be mostly identical in API and defaults.

Fixed


RtpCommand.tp(): return not just positive coords

IntelliJ's decompile shows getRandom().nextInt(origin, bound) to not work like it seems was assumed.
I have not compiled and checked this fix.

Never mind.
Went and tested on the live servers; it does not only go to the SE.
Scratch the whole thing up to Xmas even lowering IQ's all around, lol.


Fix for v18.18.2+

What’s the issue here?

It just doesn't work on YT versions newer than 18.18.2. Faking version fixes this issue.

Strange. I have 18.25.1 and it’s working fine for me. I will let you test the alternative approach of this tweak soon then.

Interesting... Сould this be due to regional differences?

@dayanch96 Try
com.ps.ytclassicvideoquality_1.0.1_iphoneos-arm64.zip

@PoomSmart im on rootful

com.ps.ytclassicvideoquality_1.1.0_iphoneos-arm.zip
Ok here. @dayanch96

Nope, still doesn't work

@dayanch96 So on your device, tapping video quality menu reveals the new one and not the classic one?

yessir

yessir

Ok.. this is another server-side control from YouTube based on app version.

Obsoleted by v1.3.0

this might be a good alternative but i cant connect it to the YTMainAppVideoPlayerOverlayViewController to perform the selector. redirects to YTAppViewControllers somehow
%hook YTElementsDefaultSheetController
- (void)presentFromViewController:(id)vc animated:(BOOL)animated completion:(id)completion {
    dispatch_async(dispatch_get_main_queue(), ^{
        if (![vc isKindOfClass:%c(YTMainAppVideoPlayerOverlayViewController)] && !isQualitySelectionNode(self.sourceView.keepalive_node)) {
            return %orig;
        }

        [vc performSelector:@selector(didPressVideoQuality:)];
    });
}
%end

ps: it presents on root view controller


fix(sql_lab): Add custom timestamp type for literal casting for presto timestamps

SUMMARY

Described in detail in: https://github.com/apache/superset/issues/13009 but described here again just for clarity.
When previewing a table in SQL Lab, a SELECT * statement is generated based on table data and metadata. This particular bug occur whens previewing tables with TIMESTAMP partition for Presto/Trino data sources. Unlike some other databases, Presto doesn't automatically convert between varchars and other types. In particular, TIMESTAMP needs to be explicitly casted. The simplest way is to use the type constructor TIMESTAMP before the value.
Therefore, where_latest_partition in presto.py needs to cast the TIMESTAMP when it generates the WHERE block in the preview table statement. Accomplish this by creating new SQLAlchemy Type which subclasses TypeDecorator adding additional functionality to existing TIMESTAMP type. For each partition column, I check if the type is TIMESTAMP, and if it is I instantiate the custom TimeStamp class in the Column object which will call process_bind_params when the query compiles.
Invalid syntax (generated by /<int:pk>/table/<table_name>/<schema_name>/ -> get_table_metadata -> select_star -> where_latest_partition)
presto:events> SELECT "event_type" AS "event_type",
            ->        "functional_domain" AS "functional_domain",
            ->        "falcon_instance" AS "falcon_instance",
            ->        "trace_id" AS "trace_id",
            ->        "tenant" AS "tenant",
            ->        "core_tenant" AS "core_tenant",
            ->        "event_timestamp" AS "event_timestamp",
            ->        "table_id" AS "table_id",
            ->        "dtr_workspace_id" AS "dtr_workspace_id",
            ->        "connectors" AS "connectors",
            ->        "table_type" AS "table_type",
            ->        "row_count" AS "row_count",
            ->        "field_count" AS "field_count",
            ->        "partition_count" AS "partition_count",
            ->        "data_size_bytes" AS "data_size_bytes",
            ->        "datetimepartition" AS "datetimepartition",
            ->        "etl_ts" AS "etl_ts",
            ->        "insert_try_number" AS "insert_try_number"
            -> FROM "events"."data_tables"
            -> WHERE "datetimepartition" = '2021-02-11 18:00:00.000'
            ->   AND "etl_ts" = '2021-02-11 18:50:00.000'
            ->   AND "insert_try_number" = 1
            -> LIMIT 100;
Query 20210211_191554_05354_4nnfj failed: line 20:27: Cannot apply operator: timestamp(3) = varchar(23)

Proper syntax
presto:events> SELECT "event_type" AS "event_type",
            ->        "functional_domain" AS "functional_domain",
            ->        "falcon_instance" AS "falcon_instance",
            ->        "trace_id" AS "trace_id",
            ->        "tenant" AS "tenant",
            ->        "core_tenant" AS "core_tenant",
            ->        "event_timestamp" AS "event_timestamp",
            ->        "table_id" AS "table_id",
            ->        "dtr_workspace_id" AS "dtr_workspace_id",
            ->        "connectors" AS "connectors",
            ->        "table_type" AS "table_type",
            ->        "row_count" AS "row_count",
            ->        "field_count" AS "field_count",
            ->        "partition_count" AS "partition_count",
            ->        "data_size_bytes" AS "data_size_bytes",
            ->        "datetimepartition" AS "datetimepartition",
            ->        "etl_ts" AS "etl_ts",
            ->        "insert_try_number" AS "insert_try_number"
            -> FROM "events"."data_tables"
            -> WHERE "datetimepartition" = TIMESTAMP '2021-02-11 18:00:00.000'
            ->   AND "etl_ts" = TIMESTAMP '2021-02-11 18:50:00.000'
            ->   AND "insert_try_number" = 1
            -> LIMIT 100

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before

After

TEST PLAN

ADDITIONAL INFORMATION

[X] Has associated issue: https://github.com/apache/superset/issues/13009
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #13082 (51091d2) into master (2ce7982) will increase coverage by 16.45%.
The diff coverage is 37.36%.

@@             Coverage Diff             @@
##           master   #13082       +/-   ##
===========================================
+ Coverage   53.06%   69.51%   +16.45%
===========================================
  Files         489     1038      +549
  Lines       17314    49139    +31825
  Branches     4482     5282      +800
===========================================
+ Hits         9187    34159    +24972
- Misses       8127    14856     +6729
- Partials        0      124      +124

Flag
Coverage Δ

cypress
53.42% <61.11%> (+0.36%)
:arrow_up:

javascript
61.89% <61.90%> (?)

python
67.07% <27.98%> (?)

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

Impacted Files
Coverage Δ

...end/src/SqlLab/components/RunQueryActionButton.tsx
64.28% <ø> (+11.50%)
:arrow_up:

...perset-frontend/src/common/components/Dropdown.tsx
54.76% <ø> (+4.76%)
:arrow_up:

...ontend/src/components/ListViewCard/ImageLoader.tsx
86.36% <0.00%> (+11.36%)
:arrow_up:

...set-frontend/src/components/URLShortLinkButton.jsx
100.00% <ø> (ø)

...src/dashboard/components/HeaderActionsDropdown.jsx
71.23% <ø> (+3.66%)
:arrow_up:

...end/src/dashboard/components/StickyVerticalBar.tsx
100.00% <ø> (ø)

...rontend/src/explore/components/EmbedCodeButton.jsx
80.76% <ø> (ø)

...ntend/src/explore/components/ExploreChartPanel.jsx
82.35% <ø> (-1.23%)
:arrow_down:

...d/src/filters/components/Range/AntdRangeFilter.tsx
0.00% <0.00%> (ø)

...set-frontend/src/filters/components/Range/index.ts
30.00% <0.00%> (-45.00%)
:arrow_down:

... and 940 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 ad380fd...8b41dba. Read the comment docs.

@john-bodley this looks like something you'd be very interested in

@kekwan you'll need to fix a couple of linting issues.

thanks @john-bodley fixed the linting issues now 😅

@john-bodley can i get another look at this when u get a chance? thanks!

What is the status of this PR? This makes it impossible to work with timeseries data on Trino. :(


Added offset prop to Tooltip and Popover components

I am sorry, but I have no clue what I did. Still figuring out git. I was trying to push the tests and docs and i tried to change the commit message. I'll do the docs and test again and do a PR.

I figured it out. and have added tests and updated the docs

Props (within JSX) should use double quotes (just as HTML attributes would be) everything else should be single quotes.

@illiteratewriter we use Travis-CI to automatically run our unit tests and for code linting.
When a pull request is opened on GitHub (or commits are added to the pull request), Travis CI receives a notification and runs a build.
During the build, Travis CI update the status icon of the pull request to one of the following statuses:

a warning that the build is still running.
a notification that the build failed – the pull request should not be merged.
a notification that the build succeeded – the pull request can be merged.

Your PR fails, because eslint found some error. You can view the log here.
You should run linter before commits.

I hope this helps you. :smiley:

Thank you for being patient and helping me with this. It was great working with you. :)


Redaction and missing paragraph lines corrected

Added the "Frameworks" paragraph that describes that topic in the original document (in this traduccion was missing), also added the external links (in spanish if its possible) and corrected some spanish redaction errors.

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

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

@Kirajav Thank you for  creating this PR.  However, since it was created, the underlying document has changed.  Can you please review and resolve the current merge conflicts?  It could be the changes to the underlying document are so significant that your changes are no longer relevant.  If the latter is the case, it might be easier to just close this PR and create a new PR with any changes you still feel are important to make.
If you need any assistance or have questions about resolving the merge conflicts, please reply back or contact someone in the Contributors Room on Gitter.
Thank you.

Hi @Kirajav
As there has been no response from your end for a while, we are closing this PR.
Feel free to request this PR to be reopened or create a new PR, if you still wish to make the changes you originally added plus any additional change requests asked of you.  Also, remember to update your fork, so you have the latest version of master in case it has changed significantly.
Happy Coding!


add toggle v7.5.282

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run


Bitburner 1.4.0 - phonix.js requires 8.5gb ram :(

In a TOTAL new instance - run phoenix.js can not run on home, because it requires 8.5gb ram.
// kill all non-phoenix files on boot
     servers.map(server => server.pids).flat()
        .filter(process => process.filename != "phoenix.js" && process.filename != "sbin.keepalive.js")
        .forEach(process => ns.kill(process.pid));

If this is removed, it will be at 8.0gb ram usage, but then there is not enough ram to run sbin.keepalive.js.
Also it tries to run something on the lowest server, but with 0 threads - and thats an exception.
disableLog: Disabled logging for scan
exec: Cannot run script 'sbin.keepalive.js' (t=1) on 'home' because there is not enough available RAM!
keepalive not found
exec: Could not find script 'bin.gr.loop.js' on 'n00dles'
sleep: Sleeping for 100 milliseconds
exec: Could not find script 'bin.wk.loop.js' on 'n00dles'
sleep: Sleeping for 100 milliseconds
exec: Could not find script 'bin.gr.loop.js' on 'foodnstuff'
sleep: Sleeping for 100 milliseconds
exec: Invalid thread count. Must be numeric and > 0, is 0
Script crashed with runtime error

Ill see what happens when I get a million to upgrade the home ram :)

Good report. I think I solved the second problem already; was just a left-over remnant from before i branched dev off from main.
But that's definitely not the only instance of ns.kill() in the entire package. I'm more likely to remove ns.killall, because I've been transitioning to a pid-based decision tree. I'll think about it.

If you're trying to get under 8GB, a way better option is to comment out the hwgw import from lib.gamestates.so.js
That uses two gigs by itself on two lazy-funcs grow-analyze and hackanalyze, and at 8G, you're not running HWGW workloads anyway.


Write a csv summary of redaction runs

Create a file that is the sibling of the redacted directory. The file is a csv and contains a mapping of input file names to output file names.
This change replaces the create_redact_dir function with create_redact_dir_and_manifest, which functions nearly identically except it also creates (via <path>.touch()) the manifest file and returns a 2-value tuple (output_dir, manifest_path) where both values are paths. The test written for create_redact_dir has been updated to reflect this change.
TODO:

[x] Write tests

@marySalvi if you could PTAL and make sure these changes make sense for the UI workflow (I don't think there would be any impact but I'm not 100% sure)
@manthey if you could PTAL and let me know if you think that this simple manifest file is sufficient for now.


ACLE/DSP: implement remaining intrinsics

Adds:
__smulbb, __smulbt, __smultb, __smultt, __smulwb, __smulwt
__qdbl, __smlabb, __smlabt, __smlatb, __smlatt, __smlawb, __smlawt
and related test-cases.
cc @Amanieu @japaric

LGTM

Does not appear to compile:
error[E0432]: unresolved import `core_arch::acle::simd32`
  --> crates/core_arch/src/acle/dsp.rs:27:22
   |
27 | use core_arch::acle::simd32::int16x2_t;
   |                      ^^^^^^ could not find `simd32` in `acle`
error: aborting due to previous error


Remove 03-make_epochs.py

Closes #91

why is this needed? does it fix some issue or pb in one of our tested
dataset?

why is this needed? does it fix some issue or pb in one of our tested dataset?

I have to set this for all NeuroMag datasts I've worked with so far (Cam-CAN, and the Localizer dataset). I also don't understand why this is necessary really, but didn't have time to investigate.

Ok I know why CI is failing. We need to discuss: for the events plot, do we want one figure per run, or should I concat all events from all runs and plot them into a single figure?

why is this needed? does it fix some issue or pb in one of our tested dataset?

I have to set this for all NeuroMag datasts I've worked with so far (Cam-CAN, and the Localizer dataset). I also don't understand why this is necessary really, but didn't have time to investigate.

@hoechenberger In the localizer data there are some 'ghost' triggers that usually last 1 sample (1ms). By setting the minimal event duration to be longer than that, we are able to get only the 'real trigger events', because the trigger function we uses sends triggers that last 4-8ms.

I would write one event file per run and store it to disk in the epochs
script. Basically when you use/need them.

let's talk in the afternoon.

But but but… it's a Sunday! Maybe I should join the union after all…
No but seriously, drop me a message on Slack and we can arrange something :)
ttyl!!

+1

@agramfort There is no good way to work with Annotations directly (e.g., you cannot pass them directly + an info structtoevents_from_annotatios(), but instead you have to attach them to a Raw` first), I decided not to save the Annotations to disk. Instead, I moved the "filtered raw concatenation" to the reports script, concatenate in-place, and extract the events there for plotting. Felt the cleanest.

we should have mne.viz.plot_annotations() and annotations.plot()
functions/methods.
ok?

@agramfort
Oh, sorry, forgot to comment on the removal of min_event_duration, stim_channel, and event_id. Good thing you're asking.
It's actually very simple: If we only work with BIDS data – which is my premise here – we don't need to handle event extraction anymore, except via events_from_annotations() (bc mne-bids writes events as annotations).

min_event_duration is a parameter of mne.find_events(). Since we don't use find_events() anymore in the pipeline, this parameter can go.
stim_channel is obvious
event_id can also go bc during the conversion to BIDS, one can (should) specify the event_id, which are then stored as the name of the respective annotations. So when you want to have your events and event_id back from an mne-bids dataset, you simply do events, event_id = mne.events_from_annotations(raw). No need to require a separate event_id in the config file.

we should have mne.viz.plot_annotations() and annotations.plot() functions/methods. ok?

I agree. I can open an issue upstream.

good points
what bothers me about dropping event_id is that the convention with / is
really an MNE thing
and we should be able to process data converted with other tools like
fieldtrip.
you have an event_id parameter in events_from_annotations. It's "auto" by
default so we can
adopt this standard and set event_id="auto" in config and pass
config.event_id in events_from_annotations.
ok?

we should have mne.viz.plot_annotations() and annotations.plot() functions/methods. ok?

I agree. I can open an issue upstream.

https://github.com/mne-tools/mne-python/issues/7720

you have an event_id parameter in events_from_annotations. It's "auto" by
default so we can
adopt this standard and set event_id="auto" in config and pass
config.event_id in events_from_annotations.
ok?

Wonder if we shouldn't offer a direct way to rename events instead? So instead of asking the user to assign new names based on event codes, we could offer to allow them to map old names to new names, e.g.,
auditory_left -> auditory/left
auditory_right -> auditory/right

Thoughts?

Also… hang on a second. Currently we're relying on the presence of Annotations in many places of the pipeline. We wouldn't be able to support FieldTrip etc easily anyway without some significant modifications, no?

see API of event_id in events_from_annotations
if you name it auditory_left than no way you can do auditory for cond in mne

I guess what I'm saying is this:
No matter where or how the data was processed originally: if it's BIDS, we will want to read it via mne-bids, right?
Now mne-bids parses events.tsv and adds Annotations to the data accordingly. In this process, we lose the original event IDs. But we don't need them anyway since we have our fine annotations now.
To rename an experimental event, you would do sth like
[a.description = 'new' for a in raw.annotations if a.description == 'old']

and if you need events, you use our friend events_from_annotations().
I therefore see  no need for config.event_id… Also I should mention that even before this very PR, config.event_id wasn't used anywhere in the pipeline anymore anyway, since everything had been switched over to using raw.annotations.

Note to self after our brief phone call:

create a dict mapping old event name in BIDS dataset -> new event name for pipeline
do the mapping early on (read raw, change annotation description)
profit!

@agramfort I think I've implemented everything we've discussed 👍

Also I've started to use f-strings in the last commit. Makes certain string formatting operations so much more readable…

thx @hoechenberger
yet looking at some reports in artifacts I see rather weird stuff eg
https://294-167957354-gh.circle-artifacts.com/0/reports/ds001810/report_average_task-attentionalblink.html
https://294-167957354-gh.circle-artifacts.com/0/reports/ds000246/report_task-AEF.html
https://294-167957354-gh.circle-artifacts.com/0/reports/ds001810/report_task-attentionalblink.html

@agramfort Now this is weird, all these reports contain -eve.fif files, which are not supposed to be there. Probably an issue related to caching??? We may need to clean the derivatives folders before / after tests.
Regarding the artifacts themselves, yes, looks like we'll have some more work to do there :(

derivative folders should not be cached. Maybe delete the folders before
stroing cache
yes nobody really checked quality of results beyond sample and maybe somato


LUN-46: Translate labels

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword LUN.

yeah wrote with Sven about it, and we decided to hard code it for now.


Renaming Hub Site URL's

I assume this is the case...but if we unregister a Hub site, then we can obviously update the URL and then re-register as a Hub, right? Has anyone had success with doing this on a Hub site?

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

ID: 5475fd83-52fa-6d49-d08f-fc75fabee09c
Version Independent ID: 19ede247-e62c-9f1a-700c-4c878fec7f78
Content: Change a site address - SharePoint Online
Content Source: SharePoint/SharePointOnline/change-site-address.md
Service: sharepoint-online
GitHub Login: @kaarins
Microsoft Alias: kaarins

@CarolinaMiC Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@CarolinaMiC thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Question: Consistent output across all platforms vs maximum fidelity on a single platform

Say we have behavior X it is possible to approximate it achieving the same consistent behavior on every platform (and approximate it to the extent that anyone using the program correctly or in a numerically sound way will not have a problem), but you can also achieve its exact behavior by linking to glibc (in this case stdio's printf for floating point behavior)
I.e. in the environment that nearly all of coreutils is used today, you have the possibility to achieve the exact same output as coreutils (better drop in replacement!), but it would mean minor differences in the output of your program on linux vs windows or os X etc.
In a scenario like this:

is it better to opt for the behavior that has maximum fidelity to the original program in the context the original program was used the most?
or is it better to try to achieve consistent output across all platforms. (Using libc on one platform for one libc and not on others seems unexpected)

With printf floating point output, I've leaned towards consistent output across all platforms as opposed to being more drop-in replacement friendly on linux. But I understand that makes drop-in a little more questionable.

I think having consistent output across all platforms is probably a good idea, although I would still try to make it as drop-in replacement friendly as possible for Linux.  Basically make it as drop-in friendly as you can while still having consistent output.

I'm definitely in support of 'consistent output across all platforms'. That said, enabling drop-in replacement of GNU coreutils would be excellent: maybe make 'maximum fidelity' behaviour optional at compile-time?

I've been thinking for stuff like this it may make more sense to do something like POSIXLY_CORRECT but for enabling even more GNU-like output/behavior.  We could call it GNULY_CORRECT or something.  Utilities can then just check whether the variable is set before checking for certain command-line options or determining how to format output.


Support for Time attributes. Temporal Data.

Mapproxy is ignoring the temporal part of the query.
I cannot find any documentation on time for a wms source...
thanks much,

Hi, currently you should be able to forward the TIME parameter to a WMS source if you specify to the forward_req_params. See the note about uncached sources.
There is also some ongoing work to support caching layers with dimensions.


feat: use excerpts from WordPress API (resolves #172)

[x] This pull request has been linted by running npm run lint without errors
[x] This pull request has been tested by running npm run dev and reviewing affected routes
[x] This pull request has been built by running npm run generate without errors
[x] This isn't a duplicate of an existing pull request

Description
This PR uses the custom WordPress excerpt for post previews in the <NewsItem /> component.
Steps to test

Identify a WordPress post with a custom excerpt, e.g. https://wecount-cms.inclusivedesign.ca/wp-json/wp/v2/posts/210.
Find it on the Views page.

Expected behavior: The excerpt shown in the API is used for the preview text in the grid.
Additional information

https://wordpress.org/support/article/excerpt/

Related issues
Resolves #172.

Is excerpt a required field for creating a Wordpress post? If it's not and is missing, shall the start of the content be used for the preview text in the grid?

@cindyli From the link in "Additional information":

If Excerpt is empty, WordPress automatically creates an excerpt using the first 55 words of the post.


Delete vtcombo config file/directory upon server shutdown.

BUG=33707109
@alainjobart

LGTM I'll remove it with my other fix :)


10.0.22621.4541 Not Supported

Help me!

Duplicate of https://github.com/stascorp/rdpwrap/issues/3354.


Progress Bar in GAT

Is there a place in the gat code to add a progress bar like the spatiotemporal test has? I'm running it and I can't tell if it is a taxing on the computer or if it just silently died. Even if it is an update on each fold would be sufficient.

Teon is back!!! :)
On Tue, Sep 20, 2016 at 12:21 AM Teon L Brooks<EMAIL_ADDRESS>wrote:

Is there a place in the gat code to add a progress bar like the
spatiotemporal test has? I'm running it and I can't tell if it is a taxing
on the computer or if it just silently died. Even if it is an update on
each fold would be sufficient.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/issues/3604, or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0fijn1NbiT49RaZlxWQD1unfrOVas7ks5qrwrygaJpZM4KBB3y
.

There may be a way but I don't know it.
I actually would like this feature too, as I get a tonne of useless outputs from joblib instead :/

Teon is back!!! :)

not yet... :/ almost!

we have a progressbar in utils.py. That could be used ...

Iirc The problem was not the problem bar itself but it's use in a
parallelized context.
Le 20 sept. 2016 04:51, "Mainak Jas"<EMAIL_ADDRESS>a écrit :

we have a progressbar in utils.py. That could be used ...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/issues/3604#issuecomment-248242541,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEp7DO5Gui8ao5BGFque93kXGbDNpRwRks5qr56GgaJpZM4KBB3y
.

The clustering code is parallel, so we can hopefully do the same thing in
both places.

Right, we just have to copy
    def get_progress_bar(seeds):
        # make sure the progress bar adds to up 100% across n jobs
        return (ProgressBar(len(seeds), spinner=True) if
                logger.level <= logging.INFO else None)

in _GeneralizationAcrossTime TimeDecoding and SearchLight. @teonbrooks you up for that?

i'm grounded from coding for about one more month.

Yes, this would be very nice to have.
Also for Time-Freq computations.


Fix tests for Python 3.11

Submitting this Fedora 37 downstream patch by @hroncok (Miro Hrončok)
This will fix running some tests against Python 3.11

Fixed with https://github.com/fonttools/fonttools/commit/14f88fb781e2618a1e7ceaf2a5caa8809ac3d181

Upstream 3.11 issue has apparently been resolved: https://github.com/python/cpython/issues/94601


Dictionary<,> Not Serializing properly

I was looking at the documentation here and in the collections section, it has this example:
var fruit = new Dictionary<string,int> {{ "Apple", 1}, { "Pear", 5 }};
Log.Information("In my bowl I have {Fruit}", fruit);

So I tried that, and I get in Document DB:
{
  "EventIdHash":<PHONE_NUMBER>,
  "Timestamp": "2017-03-15 19:46:13.712-04:00",
  "Level": "Information",
  "Message": "In my bowl I have [(\"Apple\": 1), (\"Pear\": 5)]",
  "MessageTemplate": "In my bowl I have {Fruit}",
  "Exception": null,
  "Properties": {
    "Fruit": null,
    "SourceContext": "MyProgram.MyClass"
  },
  "id": "b62d1578-2056-6335-21a4-57655c12f22e"
}

The Message is wrong, it should be { "Fruit": ["Apple", "Pear", "Orange"] }
The property is also null "Fruit": null

Now if you just run a simple console app:
namespace ConsoleApp1
{
    class Program
    {
        static void Main(string[] args)
        {
            var logger = new LoggerConfiguration()
				.WriteTo.Console().CreateLogger();
                //.WriteTo.Console(formatter: new JsonFormatter()).CreateLogger();

            var fruit = new Dictionary<string, int> { { "Apple", 1 }, { "Pear", 5 } };
            logger.Information("In my bowl I have {@Fruit}", fruit);

            Console.ReadKey();
        }
    }
}

the output is the same:
2017-03-15 20:40:20 [Information] In my bowl I have [("Apple": 1), ("Pear": 5)]

But if you comment out the first line, and add in the JsonFormatter, then the Dictionary works and everything looks great... but in the console.
{"Timestamp":"2017-03-15T20:41:21.0136426-04:00","Level":"Information","MessageTemplate":"In my bowl I have {@Fruit}","Properties":{"Fruit":{"Apple":1,"Pear":5}}}

The DocumentDb sink doesn't have the capability to pass new JsonFormatter().
Let me know if you need more information

Looks like the problem is here:
https://github.com/serilog/serilog-sinks-azuredocumentdb/blob/dev/src/Serilog.Sinks.AzureDocumentDb/Sinks/Extensions/LogEventExtensions.cs#L92
Instead of as IReadOnlyDictionary<...>, this should be as DictionaryValue.

@nblumhardt good catch. Thank you!

@maldworth, appreciated for contribution!


WPE-670: Adds IWebBrowser interface

Was part of rdkservices, but should be here.

https://jenkins.code.rdkcentral.com/job/github-triggered/2111/ started


Add java 11 httpclient plugin

Add java 11 httpclient plugin
Java HTTP Client
The HTTP Client was added in Java 11. It can be used to request HTTP resources over the network. It supports HTTP/1.1 and HTTP/2, both synchronous and asynchronous programming models, handles request and response bodies as reactive-streams, and follows the familiar builder pattern.

Applied to version 3.0.0-SNAPSHOT.
Track the Java 11 httpclient library.


(Feature) Unlimited amount of custom RPC endpoints

Relates to https://github.com/poanetwork/metamask-extension/issues/26
Unlimited numbers of custom RPCs can be added.
Networks dropdown was updated to support vertical scroll.

@patitonar I was trying to visualize scrollbar before submitting a PR. And decided not to include visualization, because of some visual artefacts at dropdown opening/closing animation. I would suggest adding visualization in a separate PR.

@vbaranov
If network is LOCALHOST then
Settings->Current RPC field is  empty

or custom rpc name displayed if there is custom rpc has been added

@dennis00010011b

If network is LOCALHOST then
Settings->Current RPC field is empty

yes, it works in such way from the beginning. Could you please create a separate issue for that?

or custom rpc name displayed if there is custom rpc has been added

I didn't catch this one. Could you clarify?

@fvictorio could you take a look to this PR? do you have any concerns?

Yes, sorry. This looks good. My only comment is that I think the new RPCs should be at the end of the list. Now the list shows the newest first (after Localhost 8545). But this is something that can be done in a separate PR.

My only comment is that I think the new RPCs should be at the end of the list. Now the list shows the newest first (after Localhost 8545). But this is something that can be done in a separate PR.

It makes sense, especially for the big list of custom RPCs and while we have no ability to hide the list of custom RPCs. I added this fix to PR. Thank you.


Building

I have followed all of the tutorials on how to build the fnf source code that I can find, to no avail. I was hoping that this could be explained? Since everything I found didn't work and made no sense what so ever.

I followed this tutorial when i first compiled Friday Night Funkin'

oh tysm


Executable isn't stored in Heroku's slug

I'm seeing this when running playwright-chromium directly:
browserType.launch: Executable doesn't exist at /app/.cache/ms-playwright/chromium-920619/chrome-linux/chrome\n" +
2023-03-06T20:32:23.177135+00:00 app[worker.1]: '╔═════════════════════════════════════════════════════════════════════════╗\n' +
2023-03-06T20:32:23.177136+00:00 app[worker.1]: '║ Looks like Playwright Test or Playwright was just installed or updated. ║\n' +
2023-03-06T20:32:23.177136+00:00 app[worker.1]: '║ Please run the following command to download new browsers:              ║\n' +
2023-03-06T20:32:23.177136+00:00 app[worker.1]: '║                                                                         ║\n' +
2023-03-06T20:32:23.177137+00:00 app[worker.1]: '║     npx playwright install                                              ║\n' +
2023-03-06T20:32:23.177137+00:00 app[worker.1]: '║                                                                         ║\n' +
2023-03-06T20:32:23.177137+00:00 app[worker.1]: '║ <3 Playwright Team                                                      ║\n' +
2023-03-06T20:32:23.177138+00:00 app[worker.1]: '╚═════════════════════════════════════════════════════════════════════════╝\n' +

Steps to replicate:

Be on Heroku stack 22
Use the https://github.com/jontewks/puppeteer-heroku-buildpack#22.0.0-no-xdg-utils buildpack as the first buildpack, and PLAYWRIGHT_BUILDPACK_BROWSERS=chromium as env config
Add npx playwright-chromium install chromium to the build script in package.json
Deploy. During Heroku's deploy, DEB files are all successfully installed. However, step 3 produces zero output. Wrapping step 3 in debug echoes confirms that that step completes with no outputs or errors. Also /app/.cache doesn't exist.
Run:

await chromium.launch({
    headless: true,
    chromiumSandbox: false,
    args: ['--no-sandbox', '--disable-setuid-sandbox'],
  })

Any ideas what I'm doing wrong?

My only guess is that the output is saying to run npx playwright install but you are running a different command. I know nothing of playwright through so I'm sure you're more up to speed on that. As far as I can see here there is no issue with the libraries being installed by the buildpack correctly, so I'll close this issue as it seems to be unrelated to the buildpack itself.