Memory Usage documentation

Is there any documentation on memory used by the different types?
I cannot find any
https://dart.dev/guides/language/language-tour
Is a bit talking about memory representations but not that much.
For example it says:

true and false, which are both compile-time constants.

So if I do:
final boolean = true;
final boolean2 = true;

Does it mean that I allocate 1 byte for the boolean (I suppose, I can't find anything).
And that I also allocate 128 bit for the two references?
Since dart if I'm correct is a references passed by value kind of language, if I do:
final integer = 1;

Does it means that I allocate 64 bit for the reference and 64 bit for the int value?
In brief is that true to say that it cost 64bit to store a boolean in a variable and 128 bit to store an Integer?
What about null values?
If i do:
Class Foo {
    final int? integer = null;
}
Foo();

What happens? Do i allocate the 64 bit reference to the integer?
Or is it allocated only if i use it? How is it handled memory-wise?

There is no detailed documentation on this beyond source code. Dart has a lot of different execution modes (native AOT and JIT, 2 different JS backend,  WASM backend, etc) and runs on many different platforms (ARM, ARM64, X64, IA32, RISC-V) so it is pretty hard to give a single answer to a question "how many bytes a specific object takes".
Lets assume that you are asking about the native implementation.
final boolean = true;
final boolean2 = true;

The true object itself exists as a single canonical instance, it takes 2 pointer size words on all platforms - because that's the smallest instance size as all allocation sizes are aligned to 2 pointer size words boundary. (e.g. 16 bytes on 64-bit platforms and 8-bytes on 32-bit platforms).
The reference to a boolean object, assuming that it actually needs to exist - e.g. that this is a global field, which was not eliminated by some optimisation, takes either a pointer size word or half of it - depending on whether this is a compressed pointer configuration or not. (e.g. on 32-bit platform it will be 4 bytes, on 64-bit platform - 8 bytes, but on 64-bit platform with compressed pointers it is going to be 4 bytes again).
final integer = 1;

Here things start getting complicated. Integer variables might or might not be unboxed. If they are unboxed - then they are passed around as 64-bit integer values, no references involved. If integer variable is not-unboxed then things depend on whether particular integer value is small enough to use a tagged small integer (smi) representation. This representation puts integers directly inside the reference variables using the least significant bit to distinguish between a tagged integer and a reference. If integer does not fit then it is put into a box which would take 16 bytes. To put it all together:

If unboxed - 8 bytes on all platforms.
If boxed:

if small enough (e.g. representable as a reference-size-in-bits-minus-1-bit signed integer)  directly in the reference variable (pointer size - 4 bytes on 32-bit and compressed 64-bit, 8 bytes on 64-bit).
otherwise reference size plus box size is needed (box is 16 bytes on all platforms)

(Again this assumes that this variable or field is not eliminated by some optimisation).
Whether int variables are unboxed or not depends on few factors, execution mode needs to be AOT, the variable needs to be provable non-null integer and it should not be a parameter in a position where somewhere in the hierarchy there is a signature override which requires boxing (e.g. parameter of Object type).
When we start talking about object allocation things get yet more complicated, e.g. if you write:
class Foo {
    final int? integer = null;
}

void main() {
  Foo();
}

Then in AOT you will not allocate anything at all (not Foo nor Foo.integer). Optimizations will eat the whole thing.
If you use result of allocation of Foo then whether space for integer is allocated or not depends on whether it is ever read or not.
If it is read then there will be a reference sized field in Foo for it.
Hope this sheds some background on this.

Thank you very much for those insights.


Issue #216 CacheConfiguration to JSR-107 Configuration bridging

This covers part of issue #216.
The use of Ehcache 3 templates will be in another pull request.

test this please


Sort Artist and Location facets alphabetically instead of by count

Sort these two facets alphabetically by default; when users click more, allow option to sort by count.

changed the sort

Artists facet looks great! Location is still sorting by count, though.

I prefer to keep the sort for the locations by count, as it seems to be more useful to see the quantity by location. Closing this ticket in the interest of time, feel free to open if you think it's important to sort by name.


Bot->Master trades

Implementation of Bot->Master trades. Should cover #19
Send all items from "Community" inventory of bot to master by trade offer:

on command !loot or !loot <BOT>
periodical on timer
on farming finished.
Added three new fields in config file, should be self-explanatory.
Based on https://github.com/Alex7Kom/node-steam-tradeoffers/

Apart from above, it looks OK.
If we're sending JSON in POST, we should have a new class with all required fields, and serialize our object to pure string in order to send it through WebBrowser.UrlPost(). Specifying structure in the code dramatically reduces readability, not to mention that I can't even see clearly what properties we actually have.
Catch the JSON we're sending, put it in http://json2csharp.com/, and if the JSON is good enough, it'll help you generate the class.

You are more demanding than my boss.

I just like to have quality code in my projects :+1:
I'll merge it when I find some time, along with some tests and misc style fixes, thanks!

[suggestion] If you add a variable to the function "GetInventory" and "SendTradeOffer", you can send traide from any inventory.
Example for tf2:
GetInventory(string inventory_name = "Community") {
	switch(inventory_name)
		case "tf2":
			jObject = await WebBrowser.UrlGetToJObject("https://steamcommunity.com/my/inventory/json/440/2?trading=1", Cookie).ConfigureAwait(false);
		default:
			jObject = await WebBrowser.UrlGetToJObject("https://steamcommunity.com/my/inventory/json/753/6", Cookie).ConfigureAwait(false);
	...
}
SendTradeOffer(List<SteamItem> inventory, ulong partnerID, string inventory_name = "Community", string token = null) {
	...
	string select_appid = "";
	string select_contextid = "";

	switch(inventory_name) {
		case "tf2":
			select_appid = "440";
			select_contextid = "2";
		default:
			select_appid = "753";
			select_contextid = "6";
	}

	SteamTradeOfferRequest trade = new SteamTradeOfferRequest();
	foreach (SteamItem item in inventory) {
        trade.me.assets.Add(new SteamItem() {
            appid = select_appid,
            contextid = select_contextid,
            amount = item.amount,
			assetid = item.id
		});
	}
	...
}

And add new commands for bot: !loottf2 !lootcsgo !lootdota2 and etc.
PS This code is just an example. :) need to think how to make the code more elegant...`

@Shmurdik why do we need this? It's a card trading bots, where would it get tf2 or csgo items?

Ok, I just suggested. :)

I'm just asking too.

bot has great potential
and easy to implement


Example code for DXGI_HDR_METADATA_HDR10 MaxMasteringLuminance is wrong

The sample c++ code at the bottom of the page does not match the definition of MaxMasteringLuminance.  It was using 0.001 nit units, when it is defined as using units of whole nits.

@mattrwoz : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Minor project Makefile fixes

This change introduces minor fixes to the main project Makefile:

Removed the test_cases target that is not defined, but referenced as a dependency of many other targets in the Makefile.
Ensured that the sgx-test-collateral and nitro-test-collateral are added as appropriate dependencies for various SGX and Nitro-related targets, thereby ensuring that the policies and other test-collateral are built appropriately.

+1 reached, merging.


Change name of image file and edit AboutUs

This PR changes the format of the image file and edited the AboutUs

LGTM


Packed Fields in Aggregations Are No Longer Being Split

As reported in slack....
Hitting an aggregation endpoint like: /api/v0.1/discovery/resources/aggregations?q=hamilton&search_scope=all used to return:
{
value: "orgs:1000",
count: 83788,
label: "Stephen A. Schwarzman Building"
}

and now returns:
{
value: "orgs:1000||Stephen A. Schwarzman Building",
count: 83788,
label: "orgs:1000||Stephen A. Schwarzman Building"
}

git bisect tells us that us that it is broken in 482af0341e61d68bc5fb3aefbc9a36268cb3056e.
I'm going to write a spec against this regression and try to fix it.

This was caused by the removal of passing packed_fields as a serializationOption in this line.


Feature Request: get_values similar to write_values

Hi,
We have a use-case where it would be great to be able to get multiple specific values back at a time. Is it possible to have a 'get_values' function that is like 'write_values', but takes a list of data point tuples and returns a list similar to what is passed INTO write_values?
I actually wrote a draft version of this already, though it's based on an older version of get_value().  Maybe it's still useful as a starting point.  I tried pasting it, but the code pasting is a bit weird with quotes, so I attached a file.
get_values.txt
I also updated the code here...
https://github.com/raeldor/tm1py/tree/feature/add-get_values

Please open a PR with this function. It makes sense to include it in TM1py IMO.

Done.
https://github.com/cubewise-code/tm1py/pull/954

Solved with #1020


Option conflict should be handled

Option conflict should be handled. As of right now, there are no handling at all

Has been handled, but might be subject to a bug. Not thouroughly investigated


feat: consolidation the subsystems of  coenzyme A metabolism

Description of the issue:

Coenzyme A (CoA) is an important molecule that plays a critical role in the metabolism of carbohydrates, fats, and proteins in human (and animal) body.

Current feature/value/output:

In Human-GEM, now there are 3 separate but interlinked subsystems related to CoA metabolism: Pantothenate and CoA biosynthesis, CoA synthesis, and CoA catabolism. for example:

MAR08978 from CoA catabolism is the extracellular version of MAR07731 from Pantothenate and CoA biosynthesis
MAR08204 and MAR08984 from CoA synthesis are the extracellular version of MAR04732 and MAR08792 from Pantothenate and CoA biosynthesis, respectively
MAR04718 from Pantothenate and CoA biosynthesis is the cytosolic version of MAR01634 from CoA synthesis
MAR11416 from CoA synthesis and MAR04723 from Pantothenate and CoA biosynthesis are both catalyzed by PPCS

Expected feature/value/output:

Merge the three subsystems into one Pantothenate and CoA metabolism

I hereby confirm that I have:

[ ] Tested my code on my own computer for running the model
[X] Done this analysis in the main branch of the repository
[X] Checked that a similar issue does not exist already

fixed in #492


Spaces in javascript loop leads to bad syntax highlighting

From @battmatt on June 7, 2016 21:51

VSCode Version: 1.2.0
OS Version: OS X 10.11.5

Steps to Reproduce:
See Image

Copied from original issue: Microsoft/vscode#7363

From @battmatt on June 7, 2016 21:52
After removing spaces:

Fixed by #257


Re-establish support for ONNX conversion

Do we still want to do that @sheiksadique ? With NIR this seems less relevant now

I agree. Removing the corresponding files from the source. in #198 .


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_b8ad225a-ec77-45cc-8a23-2aa606a836da/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_b8ad225a-ec77-45cc-8a23-2aa606a836da.git

Docs Build status updates of commit 68a86a6:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: RepositoryUnauthorizedWithSourceCommit] Git data access to 'https://github.com/opstest2/docs-archive-test-source' (branch: Release_Archive_main_2022-02-17-01-19-18) is not authorized.. It looks like this might be the first time user: v-alji submitted a pull request to this repo. Before user can run builds in a pull request, he/she can enable 'v-alji' GitHub account in Docs by visiting Docs Portal (https://ops.microsoft.com). After enabling his/her account (v-alji), close and then reopen this pull request to start another build.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


update temporal baseline threshold with new dataset

Need to rewrite thresholding function to incorporate new sensors

this has been implemented for glacier point objects in https://github.com/vdevauxchupin/ITS_LIVE_TOOL/pull/11

not sure about implementing for larger objects such as centerline, glacier because that would involve a decision about choosing a spatially representative min. threshold value, maybe best to leave up to users ?


Anchoring on pseudo elements

Pseudo elements should be supported as valid anchors.
From the spec, a  part-like pseudo-element is a valid anchor.
Implicit anchoring of pseudo-elements is specified as well.

I was working on this (needed it for a project of mine) and got most of the way there but got stuck at getting the position (coordinates) of the pseudo-element.
I found two ways:

Try to calculate it using element.getBoundingClientRect() and getComputedStyle(element, '::pseudo-element-name'): This is what I started out with but there are too many edge cases to handle.
Get the styles of the pseudo-element, hide the pseudo-element, create an new element in place of the pseudo-element with the same styles and get it's position (found here: https://stackoverflow.com/a/27377952): Haven't tested this yet, performance might not be good, this should work for ::before and ::after but not sure about other pseudo-elements (also do you know what a "part-like pseudo-element" means?)

Option one works for me because I can handle exactly the cases that matter for me, but I will try option two out tomorrow and if it works open a PR.

Thanks, @ayoreis! I forgot to give attribution that your work was the reminder that we needed to open this.

2. also do you know what a "part-like pseudo-element" means?

Here's the relevant issue- https://github.com/w3c/csswg-drafts/issues/8615.
This [commit](https://github.com/w3c/csswg-drafts/commit/6b50e77d1748fb74dd4bcb73616f63158680e400 a) may be helpful as well, as it shows a more verbose description before the part-like pseudo-element language is introduced. Essentially, it looks like if you're not able to detect the box, we likely don't need to support it.
I'm curious to see how the second option turns out!

Quick update: Got it semi working, don't know about performance yet, tomorrow I will try to have it fully working and test performance.

@ayoreis I tracked down a better definition of part-like pseudo-elements here- https://drafts.csswg.org/css-pseudo-4/#part-like-pseudo-elements
Essentially, we should also be able to support-

::file-selector-button (anchoring is currently supported in Chromium)
::details-content (this is only a proposed pseudo element at this point, there is no browser support yet.)

Will the fake element strategy work for these?
Outside of standard pseudo elements, it would be nice if we could support ::-webkit-slider-thumb, but that's tricky as it isn't cross-browser.

Not really, since the fake element technique needs to replace the pseudo-element with a real element for a moment, and input element can't have children.
For the ::details-content pseudo-element, when browsers support it they will probably also support anchor positioning.
I just had another idea which might work tough, if I set the pseudo-element to position: absolute; and inset: auto;, then getComputedStyle(element, '::pseudo-element-name') should give enough info on the element position. This would not have the same limitation, testing it right now.


Add new MAC to regexp for Xiaomi Mijia

Add new mac prefix to support newer Xiaomi Mijia Bluetooth Temperature devices.

@CWempe Thanks for the addition of the new MAC prefix
BUT
I am getting the following errors:
Initial connection to Mi Flora sensor "TemHum" (58:2D:34:32:F3:3D) failed.
AND
Failed to retrieve data from Mi Flora sensor "TemHum" (58:2D:34:32:F3:3D), success rate: 0%
Any ideas how this can be solved please?

@cbugeja
Please open a new issue.
I don't know if your problem is related to this pull request.
And I do not use my device anymore.
Maybe someone else can help you.


rustfmt deletes vertical bar for match branches!

The vertical bar for match brances was just stabilized, but rustfmt deletes them.
Thanks for the great work.

The stabilization PR removed beginning_vert field from ast::Arm. We need to bring it back to support it in rustfmt.

Note that rustfmt will not remove the vertical bar at the beginning of march arms, but it will not format the entire arm with the beginning vertical bar.
@rusterize Would you tell us which version of rustfmt you are using please? I think you might be using older version of rustfmt.

$ rustfmt --version
0.3.8-nightly (346238f 2018-02-04)

Before rustfmt:
        match anchor {
            | &Some(ref row_index) => Ok(row_nodes_borrow
                .get(row_index)
                .ok_or(RowIndexNotFound(format!("{}", row_index)))?),

            | &None => Ok(&self.root_node_iter),
        }

after rustfmt:
        match anchor {
            &Some(ref row_index) => Ok(row_nodes_borrow
                .get(row_index)
                .ok_or(RowIndexNotFound(format!("{}", row_index)))?),
            |
            &None => Ok(&self.root_node_iter),
        }

@rusterize Thank you for sharing the code! I could confirm the issue. And it should be fixed with #2464.

I'm not 100% sure what Rustfmt should be doing here, see https://github.com/rust-lang-nursery/fmt-rfcs/issues/119, however, it is clear the current behaviour is bad so I've merged the fixing PR.


Services using gRPC will break when hitting the max gRPC message size limit

gRPC message default size limit (which  I think was introduced in tonic 0.9) is set to 4MB.
With a lot of splits or indexes, a gRPC call to the metastore will return an error. For now, we saw a few examples happening on a quickwit cluster:

a search request on tens thousand splits will hit the limit. The search request errors after calling list_splits (the merge planner too was concerned but we fixed the request to decrease the number of requested splits, see #3585)
with an index with a lot of partitions,  list_indexes_metadatas will error. This call is particularly critical.

I have encounter this on search request. Is there a way to increase this limit?

@mingshun We can fix it, but we need to make sure there are no underlying problem. That's a lot of splits.
How many splits/documents do you have?

I have a PR that stream splits somewhere, I can revive it.


[fix] handle-callback-err (nested property access expression)

@blakeembrey Can you check the new tests to see if I may be missing something?

Nice, LGTM 👍 I see the issue now too! 😄

Awesome, i'm releasing.

@blakeembrey And it's released, hopefully your builds are ok now.

@jmlopez-rod LGTM 👍


Feature/data architecture

Added query for github API using GithubApiRequestOptions
Remaned aboutMe component to AboutUs
Connected AboutUs component to service

Please note: Since we are querying the github API unauthenticated, there's a limit to 60 requests per hour, per IP.

Closes #5. @dmifsud

https://github.com/davidstellini/code-pulit/tree/feature/data-architecture/data/people


Display unit in map tool legend

thanks for the change! maybe add 3-5 more px between the unit and the legend color block?

Done


memory leak since v0.2.6

Since v0.2.6 we experience a bigger memory leak in production - we tracked things down to commit
3dbb651747a43246d5dcc0c8f29a1447352f076e.
We tried all the commits for a while in production (all tests with same Node version 4.4.3):

With commit 3dbb651747a43246d5dcc0c8f29a1447352f076e (not in the screenshot) the leak is already there.
You may also notice, the leak affects only "rss" usage, not the v8 heap.
We assume the leak occurs when the connection is closed by the server (we have a very small wait_timeout of 60 sec) ... but not sure. @devzer01 maybe you can have a look at it?

Yeah it seems now that the leak in c bindings is exposed after fixing javascript, thanks for the issue report. We also experienced the issue after upgrading to 0.2.6

Will take a look at the c code and push a PR

@devzer01 did you already find a solution here, or the cause of problem? We reviewed your changes recently, but didn't really find the reason for the memleak.

The issues is in c bindings, because it's not releasing the handler from c
side
Sent via mobie
On Jul 12, 2016 22:12, "Mark Plomer"<EMAIL_ADDRESS>wrote:

@devzer01 https://github.com/devzer01 did you already find a solution
here, or the cause of problem? We reviewed your changes recently, but
didn't really find the reason for the memleak.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mscdex/node-mariasql/issues/144#issuecomment-232057742,
or mute the thread
https://github.com/notifications/unsubscribe/AEIMeZ_G1xg8wCQPQnpAFL_TpGvC9NnDks5qU58tgaJpZM4IXGbc
.


Wrong key parsing with non-US layout on Windows

Reported by SitiSchu on Telegram: on a German keyboard you use AltGr + ß to type a backslash; but Elvish parses this as Ctrl-Alt-[ and complains that it is unbound.

Interestingly, elvish v.0.12 (Windows) even interprets keys without modifiers in strange fashions.
Keyboard layout: German (Switzerland)
The "ASCII alphabet" (a-z) works properly, even though this is a QWERTZ layout.
However, every letter is transformed into its upper case version.
Anyway, on the current elvish-HEAD.exe, the above problems went away.
a yields a, shift-a yields A.
The only keys that work strangely now are the following keys:
I have pressed the keys near the umlauts in sequence, then the same thing again with the shift modifier applied.
Because ¨ is a dead key, I also pressed the umlaut o after it. Elvish, however, printed ` as soon as I hit the dead key. If I hit ¨ the dead key twice, I would expect to receive ¨¨, elvish, gives `¨¨. To get a single ¨, I would hit ¨<space>, elvish interprets this as `¨

I typed
ü
¨
o
ö
ä
$

Elvish printed
ü
`
ö
ö
ä
$

I typed
Elvish printed

è
Unbound: Shift-;

!
!

é
Unbound: Shift-'

à
Unbound: Shift-\

£
<nothing>

This also happens with Brazillian Portuguese (ABNT2) keyboard.


How to reproduce the results of the baseline method.

Thanks for your dataset and baseline code!
Could you share the number and the type of GPUs that you use to reproduce the results?
Thanks for your attention!

Hi, here are some info, also you can check out #3 and #10.

Thanks for your detailed help which solves my problem.


Tutorial/Suggestions inside lakeFS UI

As discussed in the risks & mitigations meeting, to increase understanding and usability, when run in local mode,  include dismissable pointers or suggestions of things you can do with lakeFS.
This idea needs to be fleshed out more, but this is the basic concept.

Currently, on a new repository, there is a getting started page.
Adding 'get started' page also in the following places:

No repositories - getting started, create repo and link to docs.
No actions - link to docs for more about actions.

@peacing any other places or links you can think we should add 'get started'?


[BugFix Issue #106] authorize to override count using pagy

One might want to override the total given in the headers.
It is already possible with will_paginate and kaminari, I modificated the function for pagy in order to have the same possibility.

Thanks!


SAP Hana container

How about adding a container for SAP HANA?
In case it is intended to integrate it I would be happy to contribute (I already have a working prototype...).

I just saw that there already is a PR: https://github.com/testcontainers/testcontainers-java/pull/3017

Duplicate of #1337, also see #3017
it looks like the technology isn't very container friendly, plus comes with a licensing issue - not something that we can easily add to our "supported" modules


How to use draggable with callback function?

In the 1.x or 2.x version, I would use an event. In the 2.x, triggering an event when you drag a marker would be:
$marker = new Marker(new Coordinate());

$map = new Map();
$map->getOverlayManager()->addMarker($marker);

$map->getEventManager()->addEvent(new Event(
    $marker->getVariable(),
    'dragend',
    'function() {alert("Marker dragged!")}'
))


Virtualservice 404 and ECONNREFUSED after 15 minutes

Describe the bug
Currently, we have an ISTIO setup with the following options below, everything works fine for 15 to 30 minutes then we start to receive every 3 or 4 requests one 404 or even worse one ECONNREFUSED in the ingress-gateway, all pods are healthy everything is up and running. During the issue, I've run the tests locally in the pods itself. After re-apply the virtualservice rule this behavior goes away.
Also, I've tried to change the HPA for the ingress-gateway to decrease the threshold and increase the replicas but still with the same issue.
Expected behavior
I should be able to do many requests as I want to and receive a 200 in all of them since my pods are healthy
Steps to reproduce the bug
Virtualservice:
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: my-virtualservice
spec:
  hosts:
  - "*"
  gateways:
  - my-endpoint-gw
  http:
  - match:
    - uri:
        exact: /package/_health
    - uri:
        prefix: /app/package/prices/
    - uri:
        prefix: /app/package/filters/
    - uri:
    route:
    - destination:
        host: my-endpoint.com
        port:
          number: 3000
    retries:
      attempts: 3
      perTryTimeout: 2s

siege -c 10 -H "Host: my-endpoint.com" http://<IP_ADDRESS>

Version
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.11", GitCommit:"637c7e288581ee40ab4ca210618a89a555b6e7e9", GitTreeState:"clean", BuildDate:"2018-11-26T14:38:32Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"11+", GitVersion:"v1.11.5-gke.5", GitCommit:"9aba9c1237d9d2347bef28652b93b1cba3aca6d8", GitTreeState:"clean", BuildDate:"2018-12-11T02:36:50Z", GoVersion:"go1.10.3b4", Compiler:"gc", Platform:"linux/amd64"}
Installation
helm template istio-${ISTIO_RELEASE}/install/kubernetes/helm/istio \
        --name istio \
        --namespace istio-system \
        --set gateways.istio-ingressgateway.loadBalancerIP=${CLUSTER_IP} \
        --set global.mtls.enabled=false \
        --set global.proxy.autoInject=disabled \
        --set tracing.enabled=true \
        --set servicegraph.enabled=true \
        --set telemetry-gateway.grafanaEnabled=true \
        --set grafana.enabled=true > istio.yaml

Environment
Google GKE
Cluster state

Update: Since I'm using the GKE I decided to use the ISTIO implementation handled by Google itself the problem goes away

There might be a real issue that we need to fix here, but at this point I don't think we will ever get time to investigate an issue that no one is currently seeing (since you switched to Istio on GKE), so I am going to close this.
If similar issues are seen a new issue can be opened and we can investigate again.


How to print a string with a bluetooth printer?

There are two ways to connect with a printer :

AIDL
Bluetooth

How does flutter work on it?

Did find a way to do it? I have the same requirement


[Task]: Add performance test for complex application

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
Current performance tests are run on applications with few entities and worker time is not measured. To understand the degradation in performance for evaluation we will need to create test applications with more entities.

[ ] Add metrics for worker time ( createFirstTree, updateDataTree )
[ ] Add tests with more number of entities

Ideal complex application

4 Tabs each having table widget data loading from JSObject variable.
5 Modal with each having 5-6 widgets or Form

In total around 100 entities that bound together.

@Rishabh-Rathod can you talk about why this should be prioritized?

@somangshu As we track the performance of evaluation in a different way now, the above task is not required.


Patrol Summary record view

When viewing the patrol summary - the back button on the top has the name of the vessel. This makes it appear that tapping back from the boarding record will take them back to the vessel profile. But tapping back takes them back to the patrol summary. Please change this to 'Back'.

This seems to have been done.  "Back" button now shows "Cancel".  I can change to "Back'" if that's still wanted.

Hi! Can you confirm which pages you see the words on? I think what I'm seeing is:
"Cancel" on the Patrol Summary page (which you get to by clicking the "At Sea" toggle after boarding at least one vessel and submitting a report.
If, on the Patrol Summary page, you click to one of the boardings, the title is "Boarding" and there is a button called "Back" on that page.
If that's what you see, then yes, this can be closed, and feel free to request work on another issue.

For clarification:
The patrol summary says “Cancel” which is correct:

And if I tap on the boarding i see this screen, which has “Back”, also correct:


Not having any luck using external mqtt server

Hi, thanks for this. Exactly what I was looking for.
I run my homelab MQTT server on a different machine than teslamate.  Teslamate is writing to the MQTT server but I can't get the ABRP container to read from it:
{'--model': None,
 '-a': False,
 '-h': False,
 '-l': False,
 '-p': False,
 'CAR_NUMBER': None,
 'MQTT_PASSWORD': None,
 'MQTT_SERVER': None,
 'MQTT_USERNAME': None,
 'USER_TOKEN': None}
3long_awd19
Connected with result code 5
2022-04-05 17:38:39: Car is  (unknown state), not sending any update to ABRP.
Connected with result code 5
2022-04-05 17:38:40: Car is  (unknown state), not sending any update to ABRP.
2022-04-05 17:38:41: Car is  (unknown state), not sending any update to ABRP.
Connected with result code 5
2022-04-05 17:38:42: Car is  (unknown state), not sending any update to ABRP.
2022-04-05 17:38:43: Car is  (unknown state), not sending any update to ABRP.
2022-04-05 17:38:44: Car is  (unknown state), not sending any update to ABRP.
Connected with result code 5

Response code 5 = 0x05 Connection Refused, not authorized
Here's my scrubbed docker-compose:
  ABRP:
    container_name: TeslaMate_ABRP
    image: fetzu/teslamate-abrp:latest
    restart: always
    environment:
      - MQTT_SERVER=<IP_ADDRESS>
      - MQTT_USERNAME=bob
      - MQTT_PASSWORD=bobspassword
      - USER_TOKEN=XXXABC
      - CAR_NUMBER=1
      - CAR_MODEL=3long_awd19 #NOTE: This is optional, see below

The Teslamate container itself successfully connects and sends updates to MQTT with the following env variables in docker compose - which values are identical to the values passed to the ABRP container:
      - MQTT_HOST=<IP_ADDRESS>
      - MQTT_USERNAME=bob
      - MQTT_PASSWORD=bobspassword

took a stab at just running in python, but getting an error there too.  Would prefer docker so only including this in case (again) I misconfigured something obvious:
john@teslamate:~/teslamate-abrp$ python teslamate_mqtt2abrp.py -a XXXABC 1 <IP_ADDRESS> bob bobspassword --model 3long_awd19
  File "teslamate_mqtt2abrp.py", line 90
    "car_model":f"{CARMODEL}",
                            ^
SyntaxError: invalid syntax

Anything stick out that I might have messed up?

Did some more troubleshooting this AM.  Tried just running an MQTT server in the same docker-compose as the ABRP container and the rest of the Teslamate stack.  This time it looks like ABRP container connects to MQTT (result code 0), but still no car updates.
In both cases, (local MQTT / remote MQTT ), the logging in the beginning of the python output show that no variables are being passed to python (e.g. CAR_NUMBER: None) and that MQTT authentication (-a) is False?
{'--model': None,
 '-a': False,
 '-h': False,
 '-l': False,
 '-p': False,
 'CAR_NUMBER': None,
 'MQTT_PASSWORD': None,
 'MQTT_SERVER': None,
 'MQTT_USERNAME': None,
 'USER_TOKEN': None}
3long_awd19
Connected with result code 0
2022-04-06 12:28:06: Car is  (unknown state), not sending any update to ABRP.
2022-04-06 12:28:07: Car is  (unknown state), not sending any update to ABRP.
2022-04-06 12:28:08: Car is  (unknown state), not sending any update to ABRP.
2022-04-06 12:28:09: Car is  (unknown state), not sending any update to ABRP.
2022-04-06 12:28:10: Car is  (unknown state), not sending any update to ABRP.
2022-04-06 12:28:11: Car is  (unknown state), not sending any update to ABRP.

tried with or without specifying the model (m3 LR AWD 2018 w/ 19inch wheels).  When I don't specify the model, it doesn't look like the car is detected:
TeslaMate_ABRP | Car model automatically determined as:

Not sure if any of this helps.  Will keep troubleshooting.

Hey @johnny2678,

First thing I would check, if you haven't done so yet, is that TeslaMate is indeed publishing the data to the MQTT server. Preferably, do this from the same machine you are trying to run teslamate-abrp from. MQTT Explorer is a great cross-platform tool for that. This will confirm that TeslaMate data is indeed being published to the MQTT server and that the machine running teslamate-abrp indeed has access to that data.
I can't see anything wrong in your config right now.

In both cases, (local MQTT / remote MQTT ), the logging in the beginning of the python output show that no variables are being passed to python (e.g. CAR_NUMBER: None) and that MQTT authentication (-a) is False?

That's normal: this debug output is only for the variables passed through the CLI. If you are using ENV vars (which is the case with docker-compose), they will not be displayed here. What you could do is add a "print(interesing var)" before line 70. You can actually see that your car model var is being passed correctly since "3long_awd19" is being printed (from line 69), which is actually a line that shouldn't be here in the first place ;).

tried with or without specifying the model (m3 LR AWD 2018 w/ 19inch wheels). When I don't specify the model, it doesn't look like the car is detected:

Your car is a "special config" for which I have no reference (hence the script being unable to set it automatically). Could you provide me with the "trim_badging" being published to ABRP? I'd gladly add it to the list of automatically detected cars !

took a stab at just running in python, but getting an error there too. Would prefer docker so only including this in case (again) I misconfigured something obvious:

That shouldn't happen. I'll take a look at this later to see if I can reproduce.

Anyhow, let's make sure everything is working properly with MQTT (with point 1.) and work out from there.

4. Could you provide me with the "trim_badging" being published to ABRP? I'd gladly add it to the list of automatically detected cars !

looks like 74D

I was able to get MQTT/ABRP containers to work when running as part of the same local docker-compose stack - I forgot to update the main TESLAMATE MQTT settings and it was configured to send data to my external MQTT server while the ABRP container was trying to read from the local one... silly mistake on my part.
2022-04-08 11:19:36: Car is asleep, updating every 30s.
TeslaMate_ABRP | Data object sent: {'utc':<PHONE_NUMBER>, 'soc': '90', 'power': 0, 'speed': 0, etc........

Still getting an Connected with result code 5 when trying with my external MQTT server.  Confirmed that the TESLAMATE container is sending car data to the external MQTT server with MQTT explorer successfully:

added my teslamate section of the docker-compose if it helps.  Teslamate is able to connect and send MQTT data, but for some reason ABRP container can't read them with the same settings
services:
  teslamate:
    image: teslamate/teslamate:latest
    restart: always
    environment:
      - ENCRYPTION_KEY=abc123
      - DATABASE_USER=teslamate
      - DATABASE_PASS=teslamate
      - DATABASE_NAME=teslamate
      - DATABASE_HOST=database
      - MQTT_HOST=<IP_ADDRESS>
      - MQTT_USERNAME=bob
      - MQTT_PASSWORD=bobspassword
#      - MQTT_HOST=mosquitto
    ports:
      - 4000:4000
    volumes:
      - ./import:/opt/app/import
    cap_drop:
      - all

  ABRP:
    container_name: TeslaMate_ABRP
    image: fetzu/teslamate-abrp:latest
    restart: always
    environment:
      - MQTT_SERVER=<IP_ADDRESS>
#      - MQTT_SERVER=mosquitto
      - MQTT_USERNAME=bob
      - MQTT_PASSWORD=bobspassword
      - USER_TOKEN=xyz987
      - CAR_NUMBER=1
#      - CAR_MODEL=3long_awd19 #NOTE: This is optional, see below

volumes:
  teslamate-db:
  teslamate-grafana-data:
  mosquitto-conf:
  mosquitto-data:

Anyways - thanks for looking at this.  At least I have it working when running MQTT locally.  Let me know if I can provide any more troubleshooting.

Thanks for providing the info regarding the trim. In this case (when everything works) your car should actually be detected as 3long_awd if no model is provided (which is only half-correct in this case). Unfortunately, since I am missing all the possible reported trim/wheels combination, I am unable to fully implement this. You are better off setting the model car through the env variables.
Connected with code 5 means connection refused. Looking at the code, I see where the error is: I fudged up and the logic for an authenticated connection with env vars is just plain wrong. I will fix this ASAP and report back here, so you can re-pull the image and try it again. Sorry about that !

I just implemented a fix, but I have no way to easily test it out (don't forget to re-pull the latest image). I'll re-open the issue and let you test it out and tell me it worked !

Nope. It's a big mess. I'm reverting my changes and will have to take a better look at it another time. Sorry.

Also interested in a fix here. Busy in the next few days, but might take a crack at resolving this in some spare time as I have some experience with the paho-mqtt library

Should be OK now (hopefully :sweat_smile:). Let me know if it is working on your end and close as needed.
I might get around to implement a better error handling at some point.

Should be OK now

looks good! thanks for looking at this. will close and reopen if i have any issues. Cheers!


API Additions

Added a few API elements to better make use of what's going on with Reflux.GlobalState
Main two additions:
Reflux.getGlobalState and Reflux.setGlobalState methods. Former allows a deep clone of the Reflux.GlobalState which won't continue to mutate (so you can easily store snapshots of the global state) and the latter allows you to set the global state at any point (including with partial global state objects that will only override the stores/properties represented). This basically allows an API for state based time travel.
Also added is the Reflux.initializeGlobalStore method. With expanded global state ability comes more usefulness in working with ES6 stores both in and outside of components. This allows instantiation of a proper singleton instance of a store and setting it up for tracking by the global state without any need to attach to a component.
Tests, documentation, and a lot of code commenting was also added.

Checking it out.  @spoike any initial thoughts?

Anyone have any input on this?
Pretty minimal API additions, but they make for some pretty huge additions to the capability of the library. Being able to get/set full or partial global states at any time opens up large possibilities.

LGTM. Haven't had time to check it out yet but judging by the diffs it
looks OK.
ons 24 aug. 2016 kl. 13:18 skrev Bryan Grezeszak<EMAIL_ADDRESS>
Anyone have any input on this?
Pretty minimal API additions, but they make for some pretty huge additions
to the capability of the library. Being able to get/set full or partial
global states at any time opens up large possibilities.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reflux/refluxjs/pull/481#issuecomment-242059965, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAkJUTBjsButN_RXnOv9QG5VqZYSlypks5qjESVgaJpZM4Jk_Pd
.

Anyone else have any thoughts? I've used this version in a small project thus far without issue. It's probably about as stable as any first release of some API additions. I think we're good to go.

I also noticed in my recent usage that the ability to attach a store and only update off of certain parts of its state would be convenient. So I added an ability to do so on a component-by-component basis via setting this.storeKeys in the component constructor.

only update off of certain parts of its state

isn't that achieved via connectFilter?

@itkach - No. This is the ES6 API which does not use mixins.

ok, but then shouldn't it be this.storeFilter to match this.store and this.storeFilters to match this.stores? this.storeKeys looks useful, but limited

@itkach - It does not work the same as the older storeFilter method, so there does not seem to be any reason to name it after it. That would only lead to confusion.

@itkach - FYI: I am open to another name if we want to look into that.
I just don't think it should be storeFilter since we shouldn't have multiple functionalities that work very very differently but have the same name. The implementation of storeFilter involves passing functions to a function and such, and is very imperative, whereas storeKeys is literally just an array of the key names you want to keep and is very declarative. Getting the two confused would definitely lead to whatever you're trying to do not working at all!
My logic behind the naming was that you're declaring the keys (in the key/value pairs) from the store's state that you want merged into the component state, hence storeKeys. But if we can think up a better name I'm all for it.

@BryanGrezeszak I understand what this.storeKeys does and  I'm not suggesting a different name for it. The name is fine, but functionality is limited. To me it seems that this effectively sets up a specific filter function, and so is a limited form of what could previously be achieved via connectFilter in mixins-based api. Why not set a filter function, say via this.storeFilter = filterFunc, or, if multiple stores are connected using this.stores={storeName1: store1, storeName2: store2}, with this.storeFilters = {storeName1: filterFunc1, storeName2: filterFunc2}.

The implementation of storeFilter involves passing functions to a function and such, and is very imperative

Did I miss something and there's already a storeFilter in the new ES6 api? I'm not seeing it

@itkach - Those are perfectly good suggestions for API features. They just aren't this API feature :)
The current ES6 API for merging store states into components is a more declarative API. Are there people out there that want fully imperative stuff to manipulate the connection between store and component far beyond a simpler declarative API? Sure...including yourself. And eventually I'll get around to giving them that too. But I only have so many hours and right now I want to make sure the more declarative ways of doing things are solid and robust.
The ES6 API simply merging the state of the store into the component it's attached to I find this gets you 95% of what you need with good enough architecture planning, but the ability to optionally limit that merging to just a couple keys within a store's state fills in that last 5%, so I'm adding it. That way your declarative API does all you need while still being pants-on-head simple to learn and use.
That's all this feature is, rounding out some bits with this declarative API.
Fully imperative stuff will come eventually.

Did I miss something and there's already a storeFilter in the new ES6 api? I'm not seeing it

I was just referring to the original storeFilter.

I was just referring to the original storeFilter.

What's original storeFilter? Did you mean Reflux.connectFilter from mixin api?

more declarative API.

I'm not sure what's imperative about declaring that there is a filter to be applied.
Filter function allows to re-shape data received by component, or convert values, or extract parts of state deeper than one key. In comparison, the proposed this.storeKeys api is not very compelling.

@itkach - see my edit to that, I was getting all backwards in what I was calling what between our comments.

@itkach - As for the second part: it is very much imperative for an API to have you supply a function that determines via its own logic how to filter things and gives the result, absolutely. That's not a bad thing...it's just not the side of things I'm working on right now.
Think array reverse and sort. I can reverse an array by just supplying the appropriate function to do so to sort. But reversing is commonly enough needed that having it's own declarative way is plenty worthwhile.
Similar here. The general nature of how the ES6 API tends to push the architecture leaves one hole that I've come across as far as declarative usage: simply being able to say "only take these properties from the store". I've mostly found it with tinier components that can piggyback off of stores used by larger ones but that don't need all their data, and I've seen it a lot. It's a tiny feature with a big return, worth its own declarative concept.
It also in no way prevents a more imperative method like that you suggest from being implemented, nor is it intended to. I'm simply not on to the more imperative stuff yet.

FYI:
There are people getting confused about versions out-in-the-wild. I'm getting confused questions about it. People are getting Reflux from other sources (npm, cdn's, etc.) and saying "this is the same version number as the repo, but none of these features are here!"
At this point there are, that I know of, at least 3 versions of the same (numbered)version of reflux in existence, with completely different API features and bugfixes present.
When I did my original ES6 additions I assumed we'd use some of the basic concept of semantic versioning and upped the middle digit to 0.5.0 since it was non-breaking API additions. It was decided that we don't want to call it 0.5.0 yet. I disagree with breaking from semantic versioning, but it's not my repo. But even if we don't want to use semantic versioning I highly urge there to be some form of version changing whenever we update the main repo here. Two different "versions" of the same version should not exist.
Since you guys want to do version changing your own way I haven't done anything with it in this PR, btw. That's why I'm letting you know here. Should you decide to merge this, I recommend you update version numbers in some manner. Do it however you want to do it, but please have different numbers!

As promised, a mapStoreToState method is now in with full control over mapping store states into component states. And of course includes documentation and unit tests.

Nice, thank you. Since this works the same as Reflux.connectFilter from mixin-based api, perhaps it makes sense to call it that? Or better yet, have a single this.connect to take store, state key and optional filter function. That would combine functionality of Reflux.connect and Reflux.connectFilter. In mixin api, Reflux.connect also could grow optional filter filter function and Reflux.connectFilter be deprecated, so we'd have the same clean, compact api both in ES6 and mixin-based use cases. What do you think?

@itkach - I guess to me it doesn't seem like it operates that similarly to connectFilter, just has similar capabilities.
It could be expanded to cover the original API as well, but it would have to work slightly different.
The reason for that is that the way this works is that when you use .setState({foo:'bar'}) in your ES6 store, it passes that object to your callback mapping function so that you can map just actual changed properties and only re-render as needed. This works because the ES6 stores have state in a common way (on a .state property) and call trigger in a common way when you use the store's setState (passing whatever you set via setState).
In the original API you really can store state however you feel like and trigger however you feel like. So because of that there would still be slight differences in its operation and it would include requiring certain patterns be followed for what you pass to trigger and possibly to how you store state in your store. But it's definitely something that can be discussed.

Is there any chance we can merge and publish this? I desperately need this mapping function and ES6 support via npm.
In addition to that, this project starts to look quite dead, except for the work done by @BryanGrezeszak.
That being said - if this doesn't get merged to npm - can anybody recommend a ES6-compatible alternative to Reflux? One that wouldn't be that different so that rewriting would be quite fast?

@dodekeract - For my own usage I've got all my newest work on Reflux published to npm and bower as reflux-edge. Feel free to use it. Then if this stuff gets merged all you'll have to do to be official is remove -edge.
I've been using it for some pretty huge-scale React projects and it's been doing great. It holds everything that's here in this PR plus a Reflux.Component.extend feature that I'm experimenting with.

@BryanGrezeszak thank you so much. That's exactly what I need.

@BryanGrezeszak It's looking great! Great addition! 😀

I don't know if anyone is ever going to ever come back and merge anything, but just in case I figured I'd bring it up to date with my reflux-edge repo just in case.

Sorry about me being away from the project for so long. I've been pinged to add @BryanGrezeszak as member to reflux projects and publish rights to npm. Is @BryanGrezeszak and everyone else OK with that?

@spoike, since I pushed you guys with emails into making @BryanGrezeszak a collaborator, I'm more than happy with it, and will be even happier if he wants to take this project to move it forward! :D

@spoike I'd be so happy to see him as the new maintainer or at least a member with merge & publish rights. Since this project is basically inactive except for him, I actually really regretted choosing it.
He seems to know his stuff and understand reflux completely. Please give him the rights he needs (assuming he's willing to maintain this).

I can't guarantee I'll be around all the time, but having some privileges to help keep things up would definitely be welcomed.


Fix payments that take longer than the escrow delay

Issue:
EXTv1 canisters use an escrow lock time to ensure an NFT is available while a purchaser is completing payment. Locked escrows originally would be locked for 10 minutes to allow the purchaser enough time to settle the purchase. At one point, we started to lower the lock times to 5 mins and 2 mins. Unfortunately this is starting to cause issues for buyers with slower systems who take longer than 2 mins to complete payment, which causes the purchase to fail BUT no refund is sent (as EXTv1 had no refund mech. built in).
Current fix:
We want to add a time limit on the frontend, set to 2 minutes. When a user purchase an NFT (this will be before we call the lock() call: https://github.com/Toniq-Labs/entrepot-app/blob/main/src/App.js#L302) we store the current time. Then, before we send (https://github.com/Toniq-Labs/entrepot-app/blob/main/src/App.js#L313) we check to ensure it has been less than 2 minutes since our stored time.
Please define the TIMELIMIT as a constant at the top of App.js (set to 2 minutes).
Outcome:
ICP payments should fail if the time between locking and sending payment is more than 2 minutes on the frontend. All the changes should be limited to the App.js file.
Future Thoughts:
We want to eventually pull the escrow timeframe from the API in future.

@electrovir can you move onto the above, it is higher priority right now


Test Plan execution status always in inprogress

this is duplicate i saw you reply me on this
https://github.com/gianlucamangiapelo/wdio-azure-devops-service/issues/8


updateClassifiers does not include scala-library source

steps
git clone git@github.com:scala-js/scala-js.git
cd scala-js
sbt library/package

problem
updateClassifiers does not include scala-library’s sources.
java.lang.RuntimeException: Could not fetch scala-library sources for version 2.11.8
        at scala.sys.package$.error(package.scala:27)
        at Build$$anonfun$scalalib$1$$anonfun$apply$78$$anonfun$55.apply(Build.scala:851)
        at Build$$anonfun$scalalib$1$$anonfun$apply$78$$anonfun$55.apply(Build.scala:851)
        at scala.Option.getOrElse(Option.scala:120)
        at Build$$anonfun$scalalib$1$$anonfun$apply$78.apply(Build.scala:850)
        at Build$$anonfun$scalalib$1$$anonfun$apply$78.apply(Build.scala:840)
        at scala.Function1$$anonfun$compose$1.apply(Function1.scala:47)
        at sbt.$tilde$greater$$anonfun$$u2219$1.apply(TypeFunctions.scala:40)
        at sbt.std.Transform$$anon$4.work(System.scala:63)
        at sbt.Execute$$anonfun$submit$1$$anonfun$apply$1.apply(Execute.scala:226)
        at sbt.Execute$$anonfun$submit$1$$anonfun$apply$1.apply(Execute.scala:226)
        at sbt.ErrorHandling$.wideConvert(ErrorHandling.scala:17)
        at sbt.Execute.work(Execute.scala:235)
        at sbt.Execute$$anonfun$submit$1.apply(Execute.scala:226)
        at sbt.Execute$$anonfun$submit$1.apply(Execute.scala:226)
        at sbt.ConcurrentRestrictions$$anon$4$$anonfun$1.apply(ConcurrentRestrictions.scala:159)
        at sbt.CompletionService$$anon$2.call(CompletionService.scala:28)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
        at java.lang.Thread.run(Thread.java:745)
[error] (scalalib/*:fetchScalaSource) Could not fetch scala-library sources for version 2.11.8

expectation
[success] Total time: 52 s, completed Sep 22, 2016 2:38:59 PM

notes
On my colleague’s machine it works. Some information on my configuration:
$ dpkg-query -l sbt
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name                                              Version                       Architecture                  Description
+++-=================================================-=============================-=============================-=======================================================================================================
ii  sbt                                               0.13.12                       all                           sbt, the interactive build tool

$ ls -al ~/.sbt/0.13
total 24
drwxrwxr-x 4 julien julien 4096 Aug 12 22:11 .
drwxr-xr-x 5 julien julien 4096 Aug 12 22:17 ..
-rw-rw-r-- 1 julien julien  133 Aug 12 22:10 sonatype_credentials.sbt
drwxrwxr-x 2 julien julien 4096 Jul 15 14:27 staging
drwxrwxr-x 3 julien julien 4096 Aug 12 22:11 target

I can't reproduce it, sadly.
For the record, the current HEAD of scala-js is e939ae5b98c88fc85730c02765f0187c11bcc800

I think the hardest part in solving this issue is to find how to reproduce it. What additional information about my environment could help?

I'm not sure..
Pinging @sjrd who authored the updateClassifier section of fetchScalaSource, in case he might have any ideas or perhaps has seen others hit this issue.

@dwijnand "My colleague" was referring to me (I think).
I've look at the issue on @julienrf's machine, and I couldn't figure out what was wrong. Lots of people have cloned and built successfully Scala.js for a long time, and it's the first time I've seen this happening :(

Close as WFEE? (works for everyone else) :-P
Hmm, this is tricky, but I might be looking at updateClassifiers at some point, so I'll keep this in mind.


Ensure that the patch is clean

Your patch is clean in that it (a) removes but does not comment out obsolete code and (b) does not produce extraneous output that is not required for your task (such as debug output) and (c) does not add unnecessary whitespace changes (such as adding or removing empty lines).

Seems clean to me


New example scene: Score

Simple example that uses Variables, Events and Event Listeners. This PR is related to #79

I accidentally added two other commits to the pull request. Is there an option to remove those from the PR?


ERROR: prediction failed for super95_official_e10 : cannot access local variable 'date_text' where it is not associated with a value

Hi,
The logs show following errors:
2024-09-07 09:54:36.557 ERROR (MainThread) [custom_components.carbu_com.sensor] ERROR: prediction failed for super95_official_e10 : cannot access local variable 'date_text' where it is not associated with a value
2024-09-07 09:54:37.514 ERROR (MainThread) [custom_components.carbu_com.sensor] ERROR: prediction failed for super98_official_e5 : cannot access local variable 'date_text' where it is not associated with a value
2024-09-07 09:55:32.209 ERROR (MainThread) [custom_components.carbu_com.sensor] ERROR: official failed for super95_official_e10 : 'NoneType' object has no attribute 'get'
2024-09-07 09:55:32.210 ERROR (MainThread) [custom_components.carbu_com.sensor] ERROR: official failed for super98_official_e5 : 'NoneType' object has no attribute 'get'

Thx for reporting. Will look into it later today.

should be fixed as of R11.2.2


esri-fixed-data-table: enhancement

[ ] Resizable column
[ ] Client-side sort
[ ] Client-side filter

Fixed Data Table Examples
https://facebook.github.io/fixed-data-table/example-resize.html
https://github.com/facebook/fixed-data-table/tree/master/site/examples


22:14:14 Injection of (process)  (PID [REDACTED[) failed. Sandboxes are not supported

im encountering this issue when I attempt to inject into processes like backgroundtaskhost, opera.exe process, the "backiee.exe" app from microsoft store (the wallpaper thing), one process of discord.exe (other discord processes are injected), fontdrvhost.exe, one of the "Messenger" app processes (there are 2), msedge.exe, etc

I'm not sure on what integrity level Discord.exe runs with, but more than likely it's a sandbox (low integrity level). Injecting sandboxes has always been troublesome. Afterall, these processes usually communicate to their "masters", so to speak, so they typically never list directories, or processes. Many times, sandboxed processes don't have those privileges anyway.
Here is the location in the code responsible for not injecting sandboxed processes:
// Sandboxes tend to crash when injecting shellcode. Only inject medium IL and above.
DWORD integrityLevel;
if (GetProcessIntegrityLevel(process, &integrityLevel) && integrityLevel >= SECURITY_MANDATORY_MEDIUM_RID)
{
	...


Fixed openInStore func support locale and added appName params

Important breaking change

Change parameter from appStoreId, playStoreId to Object { appName, appStoreId, appStoreLocale, playStoreId }

LGTM, but would you mind updating the API docs in the Readme to reflect your incoming changes? Thanks for the contributions!

@FiberJW Sorry for late reply. I already updated API docs.

Thanks for the contributions! Merging now 💯


X-CF-APP-INSTANCE not having the desired effect with route-service

Issue
Using X-CF-APP-INSTANCE in the request does not guarantee that the wanted instance is reached.
Context
When using a route-service in UPS mode and adding header X-CF-APP-INSTANCE to the requests, it's not guaranteed that the request will reach the wanted instance.
Steps to Reproduce

Create a simple app and scale it to more than one instance
Create a route-service app
Bind the route of the app (step 1) to the route of the route-service app (step 2)
Do a request to the app (step 1) including header X-CF-APP-INSTANCE pointing to one of the instances (e.g.: 0)
Check the app logs to see that sometimes other instances will respond

Expected result
Only the instance specified in header X-CF-APP-INSTANCE should respond to the request.
Current result
Any instance of the app can respond to the request.
Possible Fix
It seems that the X-CF-APP-INSTANCE header gets deleted (code here)before it reaches the route-service. Therefore, the second time the request reaches the Gorouter, it does not know that it should route the request to a particular instance.

Hi @aparolin, Can you tell us more about the impact you have from this? Is there a production usecase that you are trying to use this for.

Hi @shubhaat, there is no productive impact on our side yet. We have found that out during a performance test in which that header was used to orchestrate/synchronize the start/stop of kafka consumers in multiple instances of an application.
That was fine until we introduced the route-service. Then we got this unexpected side-effect.

@aparolin Thanks for the response.  Would you be open to making a PR for this? Given the current state of our backlog and the number of issues we have, we will likely not be able to prioritize fixing this soon. We have been trying to prioritize PRs higher.

Hi @aparolin,
We are closing this issue due to inactivity. If this is still an issue, please feel free to reopen with more information about its impact.
Thanks,
@rodolfo2488 and @barakyo, CF Networking Program Members


Fixes #1507: suppress libwebsocket logging races and update tests

Best I can tell the race is due to debug logging done during the supposedly thread safe call to lws_cancel_service().
It looks like the ubuntu build of libwebsockets may have enabled a static-scoped log buffer used by the debug logging, and lws_cancel_service() is issuing a debug log message.

 Great work !


$colorMode template variable is not typed

Hello,
Thank you for the module.
It seems $colorMode is not typed and it shows type error in TS.

Could you please provide an example of how you are using $colorMode or a minimal reproduction? $colorMode should be typed when accessed within the template or in script setup by:
const {
  $colorMode
} = useNuxtApp()

But generally, I would recommend you to use the composable useColorMode.

Closing this due to inactivity. Please open a new issue with a minimal production if the issue persist


Updates

/botio build

From: MushinBot (botio-academy.horizen.global)

Received
Command cmd_build from @tuanhvdev received. Current queue size: 0
Live output at: https://botio-academy.horizen.global/b3f359e422eb00d/output.txt

From: MushinBot (botio-academy.horizen.global)

Success
Full output at https://botio-academy.horizen.global/b3f359e422eb00d/output.txt
Total script time: 1.89 mins
You can download the generated site at: https://botio-academy.horizen.global/latest.tar.gz
Or view a preview at https://devweb-academy.horizen.global

/botio build

From: MushinBot (botio-academy.horizen.global)

Received
Command cmd_build from @tuanhvdev received. Current queue size: 0
Live output at: https://botio-academy.horizen.global/ae812d7ea94ea92/output.txt

From: MushinBot (botio-academy.horizen.global)

Success
Full output at https://botio-academy.horizen.global/ae812d7ea94ea92/output.txt
Total script time: 2.31 mins
You can download the generated site at: https://botio-academy.horizen.global/latest.tar.gz
Or view a preview at https://devweb-academy.horizen.global

/botio build

From: MushinBot (botio-academy.horizen.global)

Received
Command cmd_build from @tuanhvdev received. Current queue size: 0
Live output at: https://botio-academy.horizen.global/462b231724dceaf/output.txt

From: MushinBot (botio-academy.horizen.global)

Success
Full output at https://botio-academy.horizen.global/462b231724dceaf/output.txt
Total script time: 2.00 mins
You can download the generated site at: https://botio-academy.horizen.global/latest.tar.gz
Or view a preview at https://devweb-academy.horizen.global

/botio build

From: MushinBot (botio-academy.horizen.global)

Received
Command cmd_build from @tuanhvdev received. Current queue size: 0
Live output at: https://botio-academy.horizen.global/7f96fa818c59822/output.txt

From: MushinBot (botio-academy.horizen.global)

Success
Full output at https://botio-academy.horizen.global/7f96fa818c59822/output.txt
Total script time: 1.94 mins
You can download the generated site at: https://botio-academy.horizen.global/latest.tar.gz
Or view a preview at https://devweb-academy.horizen.global


2 features

These are my ideas :

add how to install it to readme on the code page so i know and other people how to install it.

cause its for fun purposes add a .py file that removes the virus from ur computer if u tested it "by accident" or not by accident

Its a collection of destructive scripts. So no installation guide.

2.There is no way to detect project Evoli.


Feature Requests: Add tests for all functions

Is your feature request related to a problem? Please describe.
Add the tests for all functions & add GitHub action to test them automatically.
Few useful resources:

https://www.youtube.com/watch?v=Kj1WwpPFr-I
https://www.youtube.com/watch?v=DhUpxWjOhME
https://wangonya.com/blog/pytest-parametrize/

Add coverage to check automatically test coverage.
https://coverage.readthedocs.io/en/7.2.3/
https://app.codecov.io/gh/unna97/twarc2sql/new

(- Split this feature into multiple issues?)

For SQL database tests:
https://coderpad.io/blog/development/a-guide-to-database-unit-testing-with-pytest-and-sqlalchemy/


Fix the Component Governance issue on System.Text.Json

https://msdata.visualstudio.com/SQLToolsAndLibraries/_componentGovernance/azure-functions-sql-extension/alert/11673675?typeId=23606141&pipelinesTrackingFilter=0
Pin System.Text.Json to 8.0.4 adding it as an explicit dependency since all the root dependencies are up to date.

@MaddyDev The CG alert still firing should be fixed - you needed to add the package reference to the projects themselves.
But I don't think we can even do this currently - it's a major version bump for the extension dependencies. Will need to consult with the host team on whether this one will be allowed.

Confirmed with Component Governance support<EMAIL_ADDRESS>that the version we have is not vulnerable and they removed the alert. Closing this as no longer required.


It should warn or throw when used without instance

ts-mockito requires the use of instance (see: #87) to get the actual mocked object instance instead of a helper object. This is fine.
I find its behavior strange though when used in a test-case when not used with instance. (I have asked and answered a question on StackOverflow about "How to mock a method using ts-mockito?
" because of it.)
TypeScript will transpile it, my Intellij WebStorm IDE will show no error, it will even autocomplete functions on the non-instance "mock".
From my perspective, the mock generated by mock looks like the actual mock instance. So I was very confused and spent a few hours figuring out why I was not getting the actual mocked value but this ts-mockito "proxy" object.
I'd propose that ts-mockito should have proxy virtually every method and issue a "Did you forget to use instant?" hint if possible. It will not catch all cases but for the ones were the developer is calling methods on the proxy object, they would get a more straightforward hint and not a generic runtime TypeError: actualReturnValue.{whatEverWasCalled} is not a function.

ts-mockito requires the use of instance (see: #87) to get the actual mocked object instance instead of a helper object. This is fine.

But it looks like other mocking frameworks don't need a separate call to make an instance? For example TypeMoq: https://github.com/florinn/typemoq#dynamic-mocks

Same here. Got confused between Mock and instance(Mock) and spent great time finding the problem.
Mockito from Java is like @k0pernikus described, there are only instances. Shouldn't ts-mockito follow the same pattern?


fake name can trip up teams

One of the fake names is D'Amoreburgh, which is not valid as a team name.
https://travis-ci.org/gratipay/gratipay.com/builds/138331186#L673

Dupe of #3722.


[SB] Review  and  fix  Livetest - april

Review live test for April release.

ServiceBusReceiverClientIntegrationTest.transactionMessageAndSettle:287
ServiceBusReceiverClientIntegrationTest.sendReceiveMessageWithVariousPropertyTypes:752
ServiceBusReceiverClientIntegrationTest.receiveMessageAndComplete:340
ServiceBusReceiverClientIntegrationTest.receiveAndDefer:706
ServiceBusReceiverAsyncClientIntegrationTest.receiveDeferredMessageBySequenceNumber:866
ServiceBusReceiverClientIntegrationTest.deadLetterMessage:525

completed


backend: add MiniCodeBuf

This eliminates 3 global variables and some #if's.

Auto-merge toggled on


refactor(@angular-devkit/build-angular): clean up differential loading config generation

Closes #16098

👍


Clients unable to connect

Here is the error
Recovered from runtime panic: interface conversion: interface is nil, not common.GetRequest
Panic location:  Panic occured at: /mnt/builds/slave/workspace/rend/gopkg/src/github.com/netflix/rend/memproxy.go:main.handleConnection (line 366)
This line suggests like a metrics increment.

The binary protocol does not respond properly when it encounters an unknown opcode. It replies with a nil struct, common.RequestGet, and a nil error causing this error when the main program loop tries to decode the parsed request. The nil error meant the error checking code couldn't do its job and casting the nil to common.GetRequest failed.
The solution is pretty simple: Respond with the proper error when the command is unrecognized.

this was fixed in 89ef7b51646e6054cfc7e10718d924a152dcbbbb


Sending units via minimap is possible

Further upgrade idea,
If right clicking on the minimap would set the rally points as well, that would be useful as well

Please give me few days to review

Looks good for starters, thanks. Later on we may rework it + we may need to add owner check.


Update cncf.yaml - review check_sets for Score

According to https://clomonitor.io/docs/topics/checks, review the check_sets for the different repos/projects of Score.

Hi @mathieu-benoit
One of the repositories should have the community check set (probably spec?), I overlooked this when reviewing it before. Could you please fix it? Thanks!

@tegioz, oh ok, gotcha, sorry about that and makes sense, just did: https://github.com/cncf/clomonitor/pull/1637. Thanks!

Thank you!


Image uploads into the rich text editor no longer resolve properly after an upgrade

https://github.com/vanderbilt/redcap-external-modules/blob/42983f07b14de94366b7edfafbd555a69a601ff9/manager/rich-text/get-uploaded-file-list.php#L119
Uploaded content in the RTE is saved with the full versioned path.  After an upgrade, images no longer render as expected.  I need to go into the database and globally update all references to the current REDCap version, or manually go into each project's EM config and update via the raw code view.
Looking for a non-versioned way to store this content so it handles upgrades a bit more gracefully.

It looks like we haven't encountered this issue because we use the "override" location for the external modules core code (instead of the version inside the redcap version dir).  After the rich text editor was added, we made it possible to make module requests via the "/api" path.  I think changing the rich text editor to use "/api" urls for embedded images would fix the problem in a way that would still work if the EM core directory was ever overridden.  I'll also look into using relative paths while I'm at it.  Stay tuned...

I just created this pull request:
https://github.com/vanderbilt/redcap-external-modules/pull/120
@dr01d3r, I think the solution is solid, but I'd love to hear any feedback you might have on it.

From the looks of it, this ends up saving the new versionless url in the module config.  Will anything need to happen with legacy uploads?

Any thoughts about how a new 'versionless' endpoint (other than API) could help solve this as it applies not only to EMs but also to some bookmarks, emails, etc...

@dr01d3r, legacy URLs will still work (until the REDCap version changes).  To switch to the new URLs, images/media will need to be removed, re-added, and saved for each popup.  Existing file uploads can still be when re-adding images/media.

@123andy, at the moment adding another endpoint doesn't seem like it would add any value.  I already view EM's index.php as our real endpoint, and the API is a simple pass through.   That's why I don't think we should put additional logic at the API endpoint (not even NOAUTH handling), as it will limit our control in EM's index.php.  If the code mentioned in issue #119 if really the simplest fix, then I think we have a fundamental design issue elsewhere that should and could be simplified without adding any additional moving parts.

Mark - my patch is to restore function but still leaves a fundamental issue that needs to be addressed with shibboleth sites...  Some tweaking in the bootstrap process of API/index.php might get around this if we delay the System::initGlobalPage() until AFTER the ExternalModules/index.php...  Happy to discus options.

I definitely think delaying System::initGlobalPage() until after ExternalModules/index.php would be a good move to make sure we have the freedom/control we need to evolve.

PR #120 has been merged, and should resolve this.  Closing...


Add Errno::ECONNABORTED to the list of rescued errors in the available? function

Attempts to attach over winrm using SSL when the SSL endpoint does not
yet exist will result in an ECONNABORTED exception.
Obvious fix.
Fixes #605

Thanks @lomeroe !


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-26-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 2496

Questions ?
Please refer to the Backport tool documentation


❌ can't install on CentOS

In order to reproduce:

Get a CentOS minimal
Clone the repo
Launch setup.sh

The error:
/src/utils/global.sh is missing: aborting script

The error is here: integry_check.sh

In order to reproduce:

Get a CentOS minimal
Clone the repo
Launch setup.sh

The error:
/src/utils/global.sh is missing: aborting script

The error is here: integry_check.sh

It fails only to create the 'up' alias. If you launch ./src/emerger.sh it works.

In order to reproduce:

Get a CentOS minimal
Clone the repo
Launch setup.sh

The error:
/src/utils/global.sh is missing: aborting script

The error is here: integry_check.sh

It fails only to create the 'up' alias. If you launch ./src/emerger.sh it works.

It somehow works also with the alias now, can't say what is happening with this. @archetipico

@archetipico if you have a quick vm with this distro you can confirm if this error persists by following the guide in description.

I don't have one, can you retry the code now that it's updated?

I don't have one, can you retry the code now that it's updated?

I don't have that machine anymore


[Newton] RO-1912 Remove site.yml

This patch removes the site.yml, which was unmaintained and unused.
Issue: RO-1912

recheck_cit_trusty_leapfrogupgrade_swift

recheck_cit_trusty_minorupgrade_swift

recheck_cit_trusty_leapfrogupgrade_swift

recheck_cit_trusty_leapfrogupgrade_swift

recheck_cit_trusty_leapfrogupgrade_swift

recheck_cit_trusty_leapfrogupgrade_swift


Fixed enchanted books not being extracted in the EnchantmentExtractor

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.BuildTools seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Why is this a data property rather than accessor property?

Precedent for getting the global this value are all getters.

node's global isn't a getter, it's a data property - i didn't think to check what window, self, etc were before writing this spec.
However, generally getters are only usually used in the language on a prototype (to share functionality across instances) - what would be the advantage of a getter here?

One clarification, is this property mutable? I confirmed it was in a few places at least.
If it is mutable, it might be worthwhile considering if mutating it needs to have any side effects, is there such concerns for web contexts — relating likely to Window/WindowProxy?
With a setter, you keep track of the assigned value — in such cases, would tracking this value beyond the confines of the global object be an additional complexity and would that be in scope here?

Yes, like most things in the language, it is mutable. Web contexts who use WindowProxy can decide what side effects it needs to have, but the spec mandates none.
That’s also a good point with the setter - there’d be no good way to track it. However, in that case it would only be an accessor, with no setter.

FWIW, I'm happy with closing this. I'd prefer consistency, but ECMAScript probably uses data properties for a bunch of things that can be used to argue that consistency is here, just with different things than I'd like.

Given NaN, Infinity, undefined, the builtin namespaces, console, and the data properties on Math, I think that without a compelling reason to use an accessor, the ES convention would be to make it a data property.
Closing for now. Thanks for the issue!

@ljharb

like most things in the language, it is mutable.

NaN Infinity undefined are non-writable (and window in Chrome is non-writable, in other browsers it's a getter without setter), why globalThis is writable (mutable)?

@hax in general things have to have a very strong reason to be nonwritable; part of the requirements in the explainer is that globalThis has to be virtualizable, which requires that it be writable.

@ljharb
As I pointed out, window is already non-writable in all browsers. Even non-writable globalThis is "virtualizable" (PS. I don't find the definition of it in this repo), you still need to find solution to make window "virtualizable". If there was a solution, it could also apply to non-writable globalThis.

I don’t see a compelling reason to make it nonwritable; the overwhelming convention in JS is that all builtins are mutable, and there are very few exceptions - which are usually for legacy reasons. Either way this is off topic for this issue; please file a new one if you wish to discuss it further.


space left if title not given

space is left under the icons if name not given

Check this out : https://github.com/roughike/BottomBar/issues/655

A duplicate of #373, so closing this one.
Expect to see this with the next release.


Fix base64 encoding of long strings

ppconsul::helpers::encodeBase64() did not account for the fact that
base64_encode_block() adds line breaks every 72 symbols or so. Let's
just remove this "feature" from base64_encode_block().

Please add unit test(s) for base64 encodind long strings under https://github.com/oliora/ppconsul/blob/master/tests/unittests/helpers.cpp

Done.

Thank you for the fix!


[BUG] Fix Formbricks Repo Icon not showing up on profile page

Issue Summary

The repo icon of the Formbricks repo is not showing like it is for the oss.gg repository.
It might be a bit confusing right now, because both repos have the Formbricks organisation logo as their picture:

We will introduce a file upload in the repo settings in a FEATURE ticket.
Steps to Reproduce
check out my public profile page:
https://oss.gg/jobenjada
Expected behavior
show all the repo icons like so:

Other information
No response
Screenshots
No response
Desktop (please complete the following information)
- OS:
- Node:
- npm:

/assign

/assign

/assign

closing for now as we are revamping the UI


Healthcheck for Redisson

I am using this client in a dropwizard service and I want to add a healthcheck. Jedis has a PING command, here using the RedissonClient I can't issue a ping command. I need to create a instance of RedisClient for the same. I should be able to retrieve instance of RedisClient from RedissonClient, or suggest me a way to perform a healthCheck for the same.

You can use RedissonClient.getNodesGroup.pingAll() or invoke ping method on each Node from RedissonClient.getNodesGroup.getNodes list.


Lots of OutOfRangeInput errors when trying to copy very large dataset from S3

Which version of the AzCopy was used?
10.3.4
Which platform are you using? (ex: Windows, Mac, Linux)
Windows, Mac
What command did you run?
azcopy.exe copy https://very-large-dataset.s3-us-west-2.amazonaws.com/some-folder "https://accountname.blob.core.windows.net/very-large-dataset?sastoken" --recursive=true

What problem was encountered?
Getting lots of OutOfRangeInput errors when trying to copy very large dataset from S3. Not a single file was copied over.
RESPONSE Status: 400 One of the request inputs is out of range.
   Content-Length: [226]
   Content-Type: [application/xml]
   Date: [Sun, 23 Feb 2020 03:37:03 GMT]
   Server: [Windows-Azure-Blob/1.0 Microsoft-HTTPAPI/2.0]
   X-Ms-Error-Code: [OutOfRangeInput]
   X-Ms-Request-Id: [5552e81f-c01e-0018-6efa-e94f39000000]
   X-Ms-Version: [2018-11-09]
Response Details: OutOfRangeInput<Message>One of the request inputs is out of range. </Message>

After 10-15 mins, following error is returned in the console:
0.0 %, 0 Done, 204 Failed, 9796 Pending, 0 Skipped, 10000 Total (scanning...),

failed to perform copy command due to error: cannot start job due to error: cannot list objects, read tcp [2456:1700:ab98:52b0:d8f:974b:7638:sd34]:23489->[2456:1fa0:416d:fe0:34ed:ee21::]:443: wsarecv: An existing connection was forcibly closed by the remote host..

How can we reproduce the problem in the simplest way?
Create a very large dataset in S3, with files running in the gigabytes. Total size of the dataset I am trying to copy is more than 1 TB.
Have you found a mitigation/solution?
No.

Hi, I can't tell what's going wrong based on the above info. Any chance you can zip up the AzCopy log file and email it to me? My email address is in my profile.  Log files contain filenames but do not contain any security credentials.

Emailed logs to dev.
Also, in case anyone else runs into this issue, I was able to copy over many TB's of data from S3 to Azure blobs using Azure Data Factory, Copy Data.

@manastalukdar I hope you were able to get your questions answered via email. Closing this issue for now as it is has been quite some time. Feel free to open a new issue if necessary.


69.4 contributions not showing correctly

i added my contribution to the 69.4 coding challenge and it was showing correctly but it seems this is no longer the case

More of a reason for #417! (I keep merging things that break the site.) I believe I fixed it with https://github.com/CodingTrain/Rainbow-Code/commit/0eb329fa64073d2134070f0121cfba328b8840ea. If you see it please close the issue!

now showing correctly! closing the issue


Preprocessor feature for imports

Preprocessor feature for imports
this feature allow some imports according to:

IMPORTS_MAP that has "whitelisted" imports
IMP_TOKEN preprocessor token now #:>
it replaces lines, nothing more nothing less

Preprocessor demo:
before preprocessor:
#:>imp-time
#:>dafsgdffdsdg
#:>imp-this
#:>imp-sys
print(time.time())

after preprocessor:
import time
#:>dafsgdffdsdg
import this
#:>imp-sys
print(time.time())

NOTES:
sys is not whitelisted as import nor as "imp-sys"
dafsgdffdsdg is not an avabile preprocessor directive
WARNING ⚠ the added modules they shouldn't have any access to files but please recheck anyway

I am rewriting the bot completely using telethon, aiohttp, and rextester API.
My future version will  ( wait for at least 2 days )

support all imports
support file handling as well.
can execute code longer than 6s
code execution will not eat server CPU
will be much faster
no polling or webhook, no bot API, directly connect with telegram server
There will be no ban and code execution will be non-blocking.
faster response inline
statistics of code

What is non-blocking ?
Take the example of the current version:
Suppose you give an infinite loop. It will block the bot for 6s. Suppose I give a simple calculation during this time, the bot will do it only after 6s.
In the new version:
You can execute code that takes a longer time, till the rextester limit.
 In the meantime, it can do calculations by other people.

I am rewriting the bot completely using telethon, aiohttp, and rextester API.
My future version will ( wait for at least 2 days )

support all imports
support file handling as well.
can execute code longer than 6s
code execution will not eat server CPU
will be much faster
no polling or webhook, no bot API, directly connect with telegram server
There will be no ban and code execution will be non-blocking.
faster response inline
statistics of code

What is non-blocking ?
Take the example of the current version:
Suppose you give an infinite loop. It will block the bot for 6s. Suppose I give a simple calculation during this time, the bot will do it only after 6s.
In the new version:
You can execute code that takes a longer time, till the rextester limit.
 In the meantime, it can do calculations by other people.

Concerns:

responsability concerns, if someone is hacking the NASA by using your bot you will be the guy under interrogation first, and we don't know if they can find the real hacker
design concerns, as far as i know the mtproto is used only by clients and not the bots, you are building a userbot and no one needs something like this, you should just convert this one to a webapp to add the webhook support
compatibility, because someone may want to run something like this on his PC or some weird condition you should put this as/in a legacy repo or add a config for legacy mode but let them use anyway, is a waste of really good code otherwise to me (not my code, i have done just 2 pull requests)

Concerns:

Thanks a lot for raising some valid points

responsability concerns, if someone is hacking the NASA by using your bot you will be the guy under interrogation first, and we don't know if they can find the real hacker

The code will be executed by rextester, not by my server. So how is my server responsible ? If someone does a bad thing using an online ide, is the company responsible ? Not at all !!

design concerns, as far as i know the mtproto is used only by clients and not the bots, you are building a userbot and no one needs something like this, you should just convert this one to a webapp to add the webhook support

You can make normal bots also using telethon( by signing in into a bot account ). Read this for more info HTTP Bot API vs MTProto.

MTProto clients (like Telethon) connect directly to Telegram’s servers, which means there is no HTTP connection, no “polling” or “web hooks”. This means less overhead, since the protocol used between you and the server is much more compact than HTTP requests with responses in wasteful JSON.
— Telethon Docs

This one feature really attracts me towards telethon. Earlier I used to oppose telethon thinking it is too complex.
I am also exploring aiogram which supports asynchronous, but connects via bot api.

compatibility, because someone may want to run something like this on his PC or some weird condition you should put this as/in a legacy repo or add a config for legacy mode but let them use anyway
If someone wants to run old code, they can clone the git repo, and switch to an older release.

, is a waste of really good code otherwise to me (not my code, i have done just 2 pull requests)
Nothing is wasted. It is more about the learning experience. The effort never goes to waste.

If you see the releases of any famous repos in github, you will find nearly equal no of lines of deletion as of addition.
( go to the Insights tab, and click Code Frequency )
Some examples:
Telethon

Python telegram bot

Bootstrap ( 145 k stars )

So deleting old code is pretty common :joy:

i am merging your pr. and creating a new release. But this will get overridden by future releases

@all-contributors add @gabrielesilinic for code


ARROW-6913: [R] Potential bug in compute.cc

The code is definitely covered by tests, but I guess nothing triggered the condition where it would do the unexpected thing.

Thank you @nealrichardson !


adjust tests for revised log formatting in production

The revised log formatting only affects production, right?
This entire test suite exists twice, once for prod (test/tests/error-handling-prod.test.js) and once for dev (test/tests/error-handling-prod.dev.js). The logging expects, however, currently only exist in the prod tests since they were thought to be the same for both scenarios.
Maybe it would be useful to also copy the original logging expects over to the dev test suite? I can also do that real quick if thats easier for you?

The revised log formatting only affects production, right?

right

This entire test suite exists twice, once for prod (test/tests/error-handling-prod.test.js) and once for dev (test/tests/error-handling-prod.dev.js). The logging expects, however, currently only exist in the prod tests since they were thought to be the same for both scenarios.
Maybe it would be useful to also copy the original logging expects over to the dev test suite? I can also do that real quick if thats easier for you?

👍