[Issue #171] fix: server accepts new addresses and syncs them

Before we were answering an error when a new wallet (that wasn't in our database) got sent through this endpoint /address/transactions, so, frontend paybutton just got lots of silent errors in the network tab and no transaction would ever be recognized there.
Now if an address that we don't have hits this endpoint, we add it to our database, fetch the transactions from the node and then return the list of transactions. If this is an address with a boatload of transactions, this endpoint could take 1 minute to answer the first time and some seconds in each request, but it should not fail anymore.
Unit test was also changed to the new situation.

@vorg-code looks like one of the integration tests is failing.

@vorg-code try to stick with the:
Description:
...
Test plan:
....

structure when creating a PR.
About the PR, indeed this endpoint address/transactions/[addr] returns an error for non existent addresses; but there is already another endpoint, which is address/transactions/sync/[addr] which does exactly that of syncing up and then returning the results. So if there are calls that are failing for non already added addresses, you could replace URL of these calls by adding the /sync/.

@chedieck what motivated this PR was that the Paybutton client calls this endpoint to get info on the transactions when we click on the donate button:

when the qrcode appears, we start firing multiple times against this address/transactions/[addr] endpoint:

Since @Klakurka said that there is no need anymore for the address to be pre-registered in the server, we should accept non registered addresses coming from paybutton client. Do you think that it is better to change the logic in the client to be aware of this error of non registered address in the server so that the client calls the address/transactions/sync/[addr] endpoint, waits and then goes back to calling the other endpoint?
PS: I will use the structure from now on.

@Klakurka, what do you think about the endpoint distinction?
As far as I remember, the last time we discussed, we decided to keep both as a way to possibly avoid unnecessary overhead by not syncing every time the transactions is called. The way @vorg-code  implemented it now will only sync for non existent addresses; so this problem would not exist.
I am fine with deleting the sync/ endpoint and keeping only this one, and maybe later have it accept some optional boolean sync parameter, which would sync even addresses that already exists (in order to get new transactions), but since we are already syncing all addresses through jobs, this is not priority.

What ended up being done here is that there is now a flag ?server_only=1 that, when set like this, tells us to only retrieve what we have in the database and we will throw an error if address is not registered (old way things worked). If no flag is passed or is set to '0', and the address is not registered, we will search it in the blockchain, register address and transactions in the database and then respond (new way).
Since the frontend paybutton uses this endpoint without a flag, it will not give any error for any correct address inputted there, so that is also fixed. If we need a fast call, we can set the flag now.
Also the test was back to the way that it was, covering the possibility of address not registered in our database (with the flag being passed as well).

I'm going to think about this a bit but my initial reaction is that using a simple query string isn't ideal.

@Klakurka what in functionality terms do you feel that is lacking in this solution?

Functionally I think it does everything we want, might just want to adjust it a bit but I have to think.

What are all of the API URLs somebody could use here just so we're on the same page?

address/balance/[addr] - gets balance of the address
address/transactions/[addr]  - gets transactions of the address, if addr not in our db: add, get trx from node, saves in our db, answers
address/transactions/[addr]?server_only=1 - gets transactions from our db, if addr not in our db answers with an error
address/transactions/sync/[addr] - add addr to our db (if not there already) and syncs transactions from node

I think all of them have different use cases

When we run address/transactions/sync/[addr], how does it update over time? I forgot what we did here. I guess a more specific question would be "would calling any of the other transactions endpoints give us old data or will it always be ready/able to send back the latest data we have on that address?

When we call address/transactions/sync/[addr] or address/transactions/[addr] (and addr is not on our db) we get the latest transactions from the node and update our database.
However, after the response is sent from this endpoint back to the frontend that called it, that is it, we don't update anymore if not called again, as far as these endpoints are concerned.
We have another point in the code (/jobs) that spawns workers that are responsible by keeping our database in sync with the nodes prices and transactions behind the scenes, so that we can trust that our database is up to date. This seems to not be working so well, I tested adding an address and transferring XEC to that address and it was not automatically picked up and it never appeared in our database. But I think that is a /jobs issue and should be treated in a different PR.

Can you describe a scenario where we might use the address/transactions/[addr]?server_only=1 endpoint?
With the new functionality of our current endpoints, we may not require it.

@ScottMcDermid if we want to have just a quick check (getting all transactions for a single address from the node can take up to 2 minutes if it is a very active wallet).
If we implement, for example, a mouseover popup with transaction details, either we get the info in a few seconds or we don't need it at all, since customer won't wait with the mouse on the same spot for more than a few seconds.

@vorg-code

if we want to have just a quick check (getting all transactions for a single address from the node can take up to 2 minutes if it is a very active wallet).

Ya that makes sense.

If we implement, for example, a mouseover popup with transaction details, either we get the info in a few seconds or we don't need it at all, since customer won't wait with the mouse on the same spot for more than a few seconds.

This example makes sense, but can you describe a user scenario for this? I don't think we ever list untracked addresses in the client.
To me, the closest thing we have is an enhanced Add New Address scenario:

Prompt user with a form that accepts addresses for a button
User enters a few ecash addresses
Dynamically check server to see if we already track those addresses?
Show user information about already scanned addresses?
User can review address data of existing addresses before submitting the form

I don't think this feature is planned, though, and a different endpoint could be created for this situation.

@ScottMcDermid here in this client page, bottom right:

you can copy and paste any address, if you do paste an untracked one, it seamlessly works (with this PR, before it it gives an error and the person would need to open the server, add the address there, then come back here for it to work). I asked @Klakurka about this in the group and he said it should work even if the address was not previously added, so I thought this was a good solution (maybe I misinterpreted him, if this was not his intention)...

@vorg-code

he said it should work even if the address was not previously added, so I thought this was a good solution (maybe I misinterpreted him, if this was not his intention)...

Wouldn't specifying the server_only=1 flag for an address that was not previously added throw an error?

@vorg-code
Sorry, misread. That scenario sounds good to me! Plus the functionality is already in place, so not a huge deal if we just leave it unused in the end.
We can always change the name of the endpoint in the future if we end up using it in some other way.

@ScottMcDermid

Wouldn't specifying the server_only=1 flag for an address that was not previously added throw an error?

Yes it will. To call this endpoint with this flag, the caller must be ready to handle this possible error.

We can always change the name of the endpoint in the future if we end up using it in some other way.

Yah 👍

@vorg-code conflicts

@Klakurka conflict solved


fix: correctly keep escape sequences for interpolations in template expressions

Hi :wave:
We noticed this happening on some Terraform source files with our CDKTF convert command which uses this library.
Will resolve this issue: https://github.com/hashicorp/terraform-cdk/issues/2911
Related (currently failing) test PR: https://github.com/hashicorp/terraform-cdk/pull/3017

Hi @tmccombs 👋
Thank you for the approval! Is there anything I can do to get this merged and released? 😇

doh. I finally came back to this, and forgot there was already a PR, so I made another one.
Although, I did realize that "%%{" also needs to be escaped.
I'm going to close this. But if you want to change this to just add some of your test cases that my PR doesn't have, I'd be willing to re-open it.


Jest Debug Profile does not hit breakpoints on Windows since 1.21.0

Hola,
since 1.21.0 my Jest Debug Profile does not hit breakpoints on Windows 10.
Linux, OSX breakpoints are working fine.
My Jest Debug Profile:
        {
            "name": "Debug Jest Tests",
            "type": "node",
            "request": "launch",
            "program": "${workspaceFolder}/node_modules/jest/bin/jest.js",
            "args": [
                "-i",
                "-t", "removeById",
            ],
            "console": "integratedTerminal",
            "internalConsoleOptions": "neverOpen"
        }

regards
Tristan

Dup of https://github.com/Microsoft/vscode/issues/45657
fyi @roblourens


When required, use a static MAC address for the VM management NIC

The Cisco CSR 1000v build process includes updating the QCOW VM image (good) which results in the management MAC address being stored "somewhere". The build process tries to cope with that, but the current workaround no longer works with release 17.03.04a. Once the container is started, the VM consistently boots with the first interface being GigabitEthernet2 (not GigabitEthernet1) and then everything goes down the drain.
The fix is simple: make the MAC address of the management interface constant (not random). It shouldn't matter as the management MAC address is not visible outside of the container.
However, to be on the safe side, the static MAC address is used only when the child class requests it via the _static_mgmt_mac object property.

Thanks @ipspace


Worlds migration

Check if re-generation is needed (main world) and make

R/O of old world
new generation with admin claims
a list of claims to be migrated and the migration plan with ETA

It has been decided to migrate via chunks removing and upgrade. (due to mcedit2 is not supporting 1.13.x)


cant find variable :self

i am just installed 0.34 version .erverything is working till i use navigationExperimental following error is coming 👍

if i use choose debug remote my code is working without any problem .Kindly help is their any dependency required.

I think we would need a more detailed code example to reproduce this. I am going to close this issue but if you discover something that looks like a bug in React Native while investigating this please feel free to open a new one!


Allow custom definition for _sympy_pass_through

Fixes #3916

This seems reasonable to me. Once we settle on the design, we should update the value_of docs (and probably the resolve_parameters protocol docs) to describe this mechanism {...}

Our requirement for "magic methods" (i.e. _functions_like_this_) is that they have an associated protocol. For _resolve_parameters_, this protocol is SupportsParameterization:
https://github.com/quantumlib/Cirq/blob/8cf782554adbafed724987de3067de7ca565fa0c/cirq/protocols/resolve_parameters.py#L28
To follow the same rule for _resolver_value_, we should add a protocol (in resolve_parameters.py is fine) for that method.

@95-martin-orion, the problem with adding this in SupportsParametrization is that this new method _resolver_value_ is for values that we want the resolver to deal with, not parametrized objects themselves. We could add a new ResolvableValue protocol for this, but while we have top-level methods to invoke most protocols (like cirq.resolve_parameters for SupportsParametrization) I don't think it makes sense to add a top-level method in this case. It's really an internal detail of how the existing parameter resolution protocol works.

@95-martin-orion, the problem with adding this in SupportsParametrization is that this new method _resolver_value_ is for values that we want the resolver to deal with, not parametrized objects themselves. We could add a new ResolvableValue protocol for this, but while we have top-level methods to invoke most protocols (like cirq.resolve_parameters for SupportsParametrization) I don't think it makes sense to add a top-level method in this case. It's really an internal detail of how the existing parameter resolution protocol works.

Protocols do not require top-level methods - see SerializableByKey (although, due to imnplementation details, this protocol no longer has a "magic method" associated with it):
https://github.com/quantumlib/Cirq/blob/06e48928ed9a504dba0985ed9a4fba33709a783a/cirq/protocols/json_serialization.py#L330


Guided Remediation: Add manifest resolution

Starting to make guided remediation public #352 :tada:
This PR has the code used to resolve the dependency graph of a manifest (i.e. a package.json) using the deps.dev resolvers and find vulnerabilities within it.
Doesn't include the code to actually parse/write package.json files - will probably add that in the next PR.
Much of this has been reviewed internally already, but I've made some significant changes/refactoring to dependency_chain.go and the computeVulns() function in resolve.go, so please take a more careful look at those.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (56584f8) 79.09% compared to head (a6ac323) 78.58%.

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

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #757      +/-   ##
==========================================
- Coverage   79.09%   78.58%   -0.51%
==========================================
  Files          86       86
  Lines        6123     6123
==========================================
- Hits         4843     4812      -31
- Misses       1075     1105      +30
- Partials      205      206       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

What's a good way to test this manually before we get automated tests?

Re tests, it would be easiest to run the internal version for now. Check with @michaelkedar on this.

Yeah, manually testing this as-is would be clumsy - you have to manually fill in the Manifest struct.


Fixed #572

Fixed #572

Hi!
Thank you for your contribution!
Would you mind signing the ICLA, as described in the Contributing page?
Also, please, feel free to join our chat channel on Gitter, if you'd like to learn more about the project and/or like to find out what else you could help with.
Kind regards,
Martin

@carlspring yeah, sure. I will send ICLA later

Hi @bbb1991,
Have you had a chance to sign the ICLA?

Hi @carlspring , I've sent signed ICLA to your mail

Thank you for your contribution!
Please join our Gitter chat, if you'd like to find out how you can help with other tasks as well.


Update group step documentation with commands use examples

Following https://forum.buildkite.community/t/group-steps-missing-commands/1790, updating the example to reflect the fact that commands does work in group step.

This closes DOC-181


[CLI]: PermissionError when trying to read k8s token inside container.

Describe the bug

When using the CLI or library, a PermissionError is raised if the k8s token secret is mounted with root permissions and the user for the container is not root.

wandb login

Traceback (most recent call last):
  File "/home/skunk/.pyenv/versions/3.10.9/bin/wandb", line 8, in <module>
    sys.exit(cli())
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/wandb/cli/cli.py", line 104, in wrapper
    return func(*args, **kwargs)
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/wandb/cli/cli.py", line 243, in login
    wandb.setup(settings=login_settings)
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/wandb/sdk/wandb_setup.py", line 327, in setup
    ret = _setup(settings=settings)
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/wandb/sdk/wandb_setup.py", line 320, in _setup
    wl = _WandbSetup(settings=settings)
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/wandb/sdk/wandb_setup.py", line 303, in __init__
    _WandbSetup._instance = _WandbSetup__WandbSetup(settings=settings, pid=pid)
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/wandb/sdk/wandb_setup.py", line 108, in __init__
    self._settings = self._settings_setup(settings, self._early_logger)
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/wandb/sdk/wandb_setup.py", line 136, in _settings_setup
    s._infer_settings_from_environment()
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/wandb/sdk/wandb_settings.py", line 1525, in _infer_settings_from_environment
    settings["docker"] = wandb.env.get_docker(wandb.util.image_id_from_k8s())
  File "/home/skunk/.pyenv/versions/3.10.9/lib/python3.10/site-packages/wandb/util.py", line 1250, in image_id_from_k8s
    headers={"Authorization": f"Bearer {open(token_path).read()}"},
PermissionError: [Errno 13] Permission denied: '/var/run/secrets/kubernetes.io/serviceaccount/token'

Additional Files
No response
Environment
WandB version:
0.14.0
OS:
Ubuntu 20.04
Python version:
3.10.9
Versions of relevant libraries:
Additional Context
I think this just needs another exception handler handling the PermissionError as well as the RequestException
https://github.com/wandb/wandb/blob/e5e6fac5a449911ab4795f9e4367a6871ddcdb9c/wandb/util.py#L1244-L1253
This is related to #5134

Hi @jlhbaseball15 thanks so much for reporting this issue, and for the suggestion to handle this error more gently. I can proceed with a feature request to include PermissionError as well in the except block.
Would it be please possible to provide some more details of the use case, eg is this something that happens often in your workflow to mount the k8s secrets token with root permissions, and any code snippet how you generated that token would be really helpful details to add to the internal ticket.

Hi @jlhbaseball15 just an update on this, I have reported it to our SDK team to handle this case better. I have linked the internal ticket with this thread so as to keep you posted on any updates. Thanks once again for reporting this issue.

Hi @thanos-wandb, Thanks a lot! As for when this happens, we have on demand dev containers running where the end user is not root, but the token is mounted with root only permissions by default. We got around it by setting the token permissions to 0644 instead of 0600 but there may be use cases where that's not ideal.

Hi @jlhbaseball15 thanks for the update, glad to hear you worked around the issue for now by changing token's permissions. I will keep you posted here on any further updates!


inserted a missing 'd'

Fixed a small spelling mistake

Hi @aranm, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


Add background execution tests for android_alarm_manager plugin

https://github.com/flutter/plugins/pull/2453 fixes a regression in the android_alarm_manager where a bad ordering of calls prevented callbacks from being executed in the background. We should add tests that ensure this functionality works properly in the future so we don't regress again.
This should be possible when https://github.com/flutter/flutter/issues/38983 is resolved.
cc/ @mklim @Hixie

Since https://github.com/flutter/flutter/issues/47406 is not actually resolved until it is tested, I'm propagating the customer critical labels from that issue to this one.

Just talked to @bkonyi and @collinjackson and Collin said he's going to finish this one. Thanks!

The issue here is that this app has both Espresso and E2E tests in the same app and they're using different incompatible bindings. Filed https://github.com/flutter/flutter/issues/51781 with potential solutions. I'm going to be working on this next.
To get this test landed, I've put it into its own example app for now.

It seems like the possibilities are

second example app
land it as a manual test
wait for #51781 first


Add JAX kmeans implementation

Summary
This PR add an implementation of the k-means algorithm written in JAX.
By replacing the current calls to the sklearn implementation with this new implementation, we can now vmap and jit code which uses kmeans for initialization. We can also remove scikit-learn as a core dependency (it is still used in some demos).
Details
The changes are currently structured into different commits as follows:

Add utility function for sklearn kmeans

This just adds as wrapper around the sklearn kmeans implementation to dynamax.utils.cluster.

Update SSMs to use kmeans utility function

Refactor the calls to sklearn kmeans to use the new utility function.

Add jax implementation of kmeans

Add a new implementation of kmeans to dynamax.utils.cluster which is compatible with JAX transformations.
Add some basic tests for this implementation.

Further Testing
It would be nice to be able to test if the new implementation does roughly as good a job as the sklearn implementation (it is considerably less complex). From playing about with it thus far once I added k-means++ initialisation it seemed to work pretty well.
I'm currently working on some test code where I patch in the new implementation and check that we can get about the same goodness of fit.
Final changes before merging
Before merging I will replace the current calls to kmeans_sklearn with kmeans_jax (and perhaps rename it) and remove the kmeans_sklearn function from dynamax.utils.cluster.
Questions:

Should the kmeans implementation be marked as reserved for internal use only? - i.e. not part of the public API?

This way we have a bit more leeway to change the interface if necessary at a later date without
Is this in fact already (implicitly?) the case with the contents of dynamax.utils?

Any bright ideas on testing this?

Related issues:
Closes #315.

"Should the kmeans implementation be marked as reserved for internal use only?" No!
K-means in jax could be of independent interest, so it is worth making it a first class public citizen.
But see also
https://ott-jax.readthedocs.io/en/latest/_autosummary/ott.tools.k_means.k_means.html
Possibly also relevant
https://jax.readthedocs.io/en/latest/_autosummary/jax.lax.approx_max_k.html


Search SDK: Fixing incorrect URL in Count docs

Since this is such a minor change, we're not updating the Nuspec
version. We'll ship this change with the next feature we add.
This change was cherry-picked from the master branch.

Hi @brjohnstmsft, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (brjohnst). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


VS: Implement a Razor language package

We need to implement a Razor Language Service to ship in VS. Ideally this will live in this repo, but be consumed by the editor. Dependencies on VS are OK here as long as it can be built standalone

We have the basic skeleton of this and we shipped version 0.1 recently supporting Tag Helper discovery. The rest of the work to do is tracked in separate issues.


Add support for writing records asyncronously

Currently the fastavro.writer method expects a list of records and writes them at once.  I'd like to be able to write the file header, preserve some context, and then write each record as my application receives data, then make a final call to flush the output when finished.

Thanks for reporting, will look into it soon-ish.

I second this request. It would be really useful to not have to generate a huge list in memory and then write them all at once. It would make for more pythonic loops.

With the current design the records parameter is any iterable including a generator. This way you can write a generator that produces records when they become available the exit the generator when your done.
For example:
def gen_records():
    while True:
        record = next_record()
        if not record:  #end of records
            return
         yield record

with open('records.avro', 'w') as out:
    writer(out, schema, gen_records())


How to write to a DOMAIN SDO

I'm working on a large transfer to a DOMAIN SDO to perform a firmware update. I have left CO_CONFIG_SDO_SRV_BUFFER_SIZE at the default value of 32.
I started by examining the code domainDemo.c code from here. But, using the "master" code from here I keep triggering
                /* if data size exceeds variable size, abort */
                if (SDO->OD_IO.stream.dataLength > 0
                    && SDO->sizeTran > SDO->OD_IO.stream.dataLength
                ) {
                    abortCode = CO_SDO_AB_DATA_LONG;
                    SDO->state = CO_SDO_ST_ABORT;
                    break;
                }

from here.
At the start of the transfer SDO->sizeInd is set with the transfer size, but then does not seem to be used at all. See here.
The SDO->OD_IO.stream.dataLength appears to be limited to the size of the stream buffer and not set up for filling and then emptying the stream buffer many times. What am I dong incorrectly?

Downloading large files is possible using stream mode. Usage of Domain has been asked a few times, search issues for "domain".

SDO transfer is quite complex and I don't remember details now. Indication of data size is not required by the standard, so it may only be informative and for verification.
Example in CANopenDemo should work without problems. Is it so? If you have own implementation, you may try to do some comparisons.

I got the CANOpen domain demo to work. I see a sequence that looks like:
 (028.225714)  can0  604   [8]  C4 22 21 00 00 00 00 00
 (000.000142)  can0  584   [8]  A4 22 21 00 7F 00 00 00
 (000.000036)  can0  604   [8]  01 00 01 02 03 04 05 06
 (000.000005)  can0  604   [8]  02 07 08 09 0A 0B 0C 0D
 (000.000003)  can0  604   [8]  03 0E 0F 10 11 12 13 14
 (000.000012)  can0  604   [8]  04 15 16 17 18 19 1A 1B
 (000.000003)  can0  604   [8]  05 1C 1D 1E 1F 20 21 22

my code has a sequence that looks like
    99)      9300.4  Rx         061C  8  21 50 1F 01 E0 12 08 00
   100)      9303.1  Rx         059C  8  60 50 1F 01 00 00 00 00
   101)      9305.9  Rx         061C  8  00 E9 07 02 30 A4 15 08
   102)      9319.7  Rx         059C  8  20 00 00 00 00 00 00 00
   103)      9322.0  Rx         061C  8  10 40 EE 00 00 00 00 00

so it would seem the cocomm example (that I used for the CANOpen domain demo test) uses SDO Block transfers while my test uses SDO segmented transfers. I'll see if I can switch my implementation to use block transfers.

Segmented transfer should work too. There may be a bug. Maybe you can reproduce error with segmented transfer and cocomm?
cocomm "set sdo_block 0"

cocomm just sends the string to the canopend. The string is specified by standard CiA309. String does not include information about the variable size, except with known variable types.
If there is a large block of data being transferred, there are multiple reads/writes through socket. In Linux data goes like flow through cocomm (pipes) and data length can't be passed.

Maybe you can try without data size indication first.

You definitely can send the size indication.  I've got 2 boards both with
CanOpenNode and we were shifting 8000 bytes of data via segmented SDO.
Admittedly this was a read not a write, but when reading the domain the
first packet back had size indication and this worked between the boards
and my canmonitor PC application picked it up and correctly decoded it.
I'm checking my flash loader and it looks like (writing from a C# client to
CanOpenNode device) that i send the filesize, my method is below .,.. you
can see my implementation in all its gory details at
https://github.com/robincornelius/libCanOpenSimple/blob/master/SDO.cs
//Bigger than 4 bytes we use segmented transfer
cmd = 0x21;
expitided = false;
byte[] payload = new byte[4];
payload[0] = (byte)databuffer.Length;
payload[1] = (byte)(databuffer.Length >> 8);
payload[2] = (byte)(databuffer.Length >> 16);
payload[3] = (byte)(databuffer.Length >> 24);
        p.cob = (UInt16)(0x600 + node);
        p.len = 8;
        p.data = new byte[8];
        p.data[0] = cmd;
        p.data[1] = (byte)index;
        p.data[2] = (byte)(index >> 8);
        p.data[3] = subindex;

        int sendlength = 4;

        if (payload.Length < 4)
            sendlength = payload.Length;

        for (int x = 0; x < sendlength; x++)
        {
            p.data[4 + x] = payload[x];
        }

        can.SendPacket(p);

then after this is the flip/flop segmented transfer, with the toggle flag
and finally an end flag is set for the last packet
if you have certain canusb devices my canmonitor app will log and decode
your packets and may help with debugging..
https://github.com/robincornelius/CanOpenMonitor
On Fri, 3 Feb 2023 at 15:40, andrew-elder @.***> wrote:

It is looking more and more like this is my issue. I created this small
python app that I run against a build of the demo application and it seems
to work correctly.

domain_test.py

Test block transfers

import argparse
import os
import time
import canopen
FIRMWARE_PATH = './demoDomainTest.bin'
TRANSFER_SIZE_KB = 16
Create a test file
some_bytes = list(range(0, 256))
some_bytes = some_bytes * (TRANSFER_SIZE_KB * 4)
immutable_bytes = bytes(some_bytes)
with open(FIRMWARE_PATH, "wb") as binary_file:
binary_file.write(immutable_bytes)
create object dictionary and domainDemo SDO
objdict = canopen.objectdictionary.ObjectDictionary()
member = canopen.objectdictionary.Variable('demoDomain', 0x2122)
member.data_type = canopen.objectdictionary.DOMAIN
objdict.add_object(member)
Connect to the CAN bus
network = canopen.Network()
network.connect(bustype='socketcan', channel='can0')
node = network.add_node(4, objdict)
open the file
FILESIZE = os.path.getsize(FIRMWARE_PATH)
CHUNK_SIZE = 1024
infile = open(FIRMWARE_PATH, 'rb')
outfile = node.sdo['demoDomain'].open('wb', size=FILESIZE, block_transfer=False)
send the file in chunks
while True:
chunk = infile.read(CHUNK_SIZE)
if not chunk:
break
outfile.write(chunk)
finish
infile.close()
network.disconnect()
Thanks for your debug ideas. I'll post back here when I figure it out.
—
Reply to this email directly, view it on GitHub
https://github.com/CANopenNode/CANopenNode/issues/423#issuecomment-1416042806,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG2GCOE36HHBT3TO73L23WVURHJANCNFSM6AAAAAAUM4Y2KU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@robincornelius - thank you for your additional comments. I found a line in my version of the OD_write_domainDemo() implementation that did not match the original. I have fixed it, and now it works correctly.


Firebase/Firestore

Any chance we will get full-fledged realtime and firestore capabilities through these native libraries? The Javascript version of firestore are just not up to task due to their dependence on IndexedDB.

Firestore support was added in v8.1.0

Sweet! Any way of adding hooks to update events?

Do you mean Realtime listeners?
I had considered implementing them initially but it is not simple because I'd need to manage a map of the JS callback to the native listener. This could be added as an additional feature but would take some time.


Could you please correct the language support for Italian?

Describe the bug
A clear and concise description of what the bug is.
In current version of Italian language, we have these to calculate lastWeek:
calendar : {             sameDay: '[Oggi alle] LT',             nextDay: '[Domani alle] LT',             nextWeek: 'dddd [alle] LT',             lastDay: '[Ieri alle] LT',             lastWeek: function () {                 switch (this.day()) {                     case 0:                         return '[la scorsa] dddd [alle] LT';                     default:                         return '[lo scorso] dddd [alle] LT';                 }             },             sameElse: 'L'         },
However, I think a better way to say it is: "la domenica scorsa alle..." or "il mercoledì scorso alle...".
Also, 'alle' works for all time except for 1 o'clock for which you would say "all'una" or "all' 1:00".
Could you please fix that?

I can work on this!!


crash after OLLAMA_MULTIUSER_CACHE=1

What is the issue?
When I start loading a model, such as granite3-dense, I get this error:
error: unknown argument: --multiuser-cache
usage: /usr/lib/ollama/runners/cuda_v12/ollama_llama_server [options]
OS
Linux
GPU
Nvidia
CPU
Intel
Ollama version
0.3.14

This is intended only for development use cases (which is why it was left undocumented). It won't work in the mode you are running it in.


Several pent up frustrations ;)

These are a bulk of patches I've been maintaining in my working branch for a while. I've been using them regularly, so at least they WFM and I think I've done an all right job of keeping them out of the way for anyone who doesn't want them.

/test odf-operator-bundle-e2e-aws

/test odf-operator-bundle-e2e-aws

/test odf-operator-bundle-e2e-aws

looks like openshift-ci is not picking up the ${GINKGO} value
Code:
cd e2e/odf && ${GINKGO} build && ./odf.test \

Error:
./hack/e2e-test.sh: line 7: build: command not found
make: *** [Makefile:63: e2e-test] Error 127


Requiring as a sub package (non-root)

I'm trying to make this a dependency of another package, which is listed in my root composer.json.
I'm running into problems because this package does not have a stable release on Packagist. Would it be possible to add one, such as "v0.12.0" or "v0.12.1"?
Thank you.

I've changed the version inside composer.json
Is it right? I have to do some other operation?

Thanks for the response!
I believe you actually need to "tag" the release in git/github

Thank you :)
Hope I've done the right steps...

I believe this was just what I needed. Thank you so much!


Activity: Payment basket

Add a payment basket.
Acceptance Criteria:

Payment basket contains details of all items to pay for, with price, currency, description, count and sku.
Payment basket available on all required pages
Payment basket contents may be updated

Tasks:

[x] Create serialisable class which can be stored in the request session.
[x] Convert all basket items to basket base currency
[x] Add basket contents update functionality

Estimate (XS, S, M, L, XL): L

Completed 6194554


[Bug]: cannot ensure that projection is up to date

Preflight Checklist

[X] I could not find a solution in the documentation, the existing issues or discussions
[X] I have joined the ZITADEL chat

Environment
Self-hosted
Describe the bug
When using version v2.22.0 of ZITADEL, I receive this error when logging in:
status: 400
domain: https://domain.de/oauth/v2/token

{
  "error": "server_error",
  "error_description": "ID=OIDC-ASfh3 Message=cannot ensure that projection is up to date"
}

To reproduce

Launch ZITADEL v2.22.0
Go through an OIDC login attempt

Screenshots
No response
Expected behavior
No response
Version
2.22.0
Operating System
No response
Relevant Configuration
No response
Additional Context
Note: This is a reproduction of an issue solved in the ZITADEL chat.
It is here for search engines and future reference.
You can find more information and discussions in the chat above.

Solution: Please update your version of Cockroach DB to at least v22.2.0.


Fixing up GPU to ensure working in full BOA

Several things needed to be fixed...

Codecov Report
Patch coverage has no change and project coverage change: -0.03 :warning:

Comparison is base (04727b1) 37.18% compared to head (bf2b51a) 37.16%.

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

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #215      +/-   ##
==========================================
- Coverage   37.18%   37.16%   -0.03%
==========================================
  Files          54       54
  Lines       11167    11175       +8
==========================================
  Hits         4153     4153
- Misses       6776     6784       +8
  Partials      238      238

Impacted Files
Coverage Δ

axon/gpu.go
3.30% <0.00%> (-0.04%)
:arrow_down:

axon/looper.go
0.00% <0.00%> (ø)

axon/pool.go
89.61% <ø> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Bug: Andrha Pradesh Total tested is a date

In Item 170 here

"totaltested"

is a date.

Fixed. Thanks for reporting.


tellurium crashes when bioservices is not available

We haven't tested bioservices across all platforms yet so we don't know if it will always work or always be available. Currently, tellurium fails to import when bioservices is not present. It would be good if it was still possible to import tellurium even if bioservices is not available. @matthiaskoenig can you comment/help with this?

Yes. Will implement workaround to get models directly via httplib. Is only
needed to get the sbml from biomodels.
On May 14, 2016 12:30 AM, "Kyle Medley"<EMAIL_ADDRESS>wrote:
We haven't tested bioservices across all platforms yet so we don't know if
it will always work or always be available. Currently, tellurium fails to
import when bioservices is not present. It would be good if it was still
possible to import tellurium even if bioservices is not available.
@matthiaskoenig https://github.com/matthiaskoenig can you comment/help
with this?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/sys-bio/tellurium/issues/125

This should work now with latest commit.
I did a fallback via httplib if bioservices cannot be imported in temiriam.py.
bioservices is also used to resolve annotations within some notebook examples, but this should not affect the general import of tellurium.
@0u812 let me know if this solves the issue

Yes this solves it. Thanks so much!


Graduate dwell

Overview
This PR graduates dwell from experimental. Changes includes moving files to non-experimental locations, namespace changes, adding comments, exposing the dwell events & timing thresholds and adding a test.


Kirkstone fixes

This series mostly fixes deprecated distutils usage, but also changes the release compatibility from 'honister' to 'kirkstone'
a0e7adc03 ros_catkin.bbclass: inherit setuptools3-base
876a5e210 python3-pydot: inherit setuptools3
d0a89e1cc python3-empy: patch to use setuptools3
f2d275d1f ros-distro.conf: ROS_OE_RELEASE_SERIES -> kirkstone

Not asking for this to be merged, but just FYI: https://github.com/moto-timo/meta-ros/tree/superflore-kirkstone-2022-04
This was only running superflore and no attempt was made to build or test.

This pull request has been mentioned on ROS Discourse. There might be relevant details there:
https://discourse.ros.org/t/announcing-the-completion-of-meta-ros-milestone-17/25933/3

This pull request has been mentioned on ROS Discourse. There might be relevant details there:
https://discourse.ros.org/t/ros-2-tsc-meeting-minutes-2023-01-19/29423/7

Thanks, @moto-timo
I have pulled in your fixes into the kirkstone commit series I am getting ready to push before Monday.


AADSTS50199: token renewal doesn't work after a few hours on Edge canary

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Other... Please describe:

Browser:

[ ] Chrome version XX
[ ] Firefox version XX
[ ] IE version XX
[x] Edge version <IP_ADDRESS>
[ ] Safari version XX

Library version

Library version: 1.1.3

Current behavior

acquireTokenSilent fails after a number of hours of unuse (i.e. if I haven't used the application overnight).
Expected behavior

Once I've logged in, I should continue to be able to use acquireTokenSilent to use a refresh token and get a new access token. I've verified this behavior is seen on Chrome version 78.0.3895.0
Minimal reproduction of the problem with instructions
Including code snippets with error messages for completeness.
I set up my UserAgentApplication with a fairly standard configuration:
const configuration: Msal.Configuration = {
    auth: {
        redirectUri: chrome.extension.getURL('login_return.html'),
        clientId: applicationConfig.clientId,
        authority: applicationConfig.authority,
        validateAuthority: true,
        navigateToLoginRequestUrl: true,
    },
    cache: {
        cacheLocation: 'localStorage',
    }
};

const userAgentApplication = new Msal.UserAgentApplication(configuration);

Then I first get a token with:
const tokenRequest: Msal.AuthenticationParameters = {
        scopes: [ 'https://microsoft.onmicrosoft.com/RichCodeNav/read_public' ]
};
try {
        await userAgentApplication.loginPopup(tokenRequest);
        await userAgentApplication.acquireTokenSilent(tokenRequest);
    } catch (err) {
        try {
            await userAgentApplication.acquireTokenPopup(tokenRequest);
        } catch (err) {
            return;
        }
    }

which works, as I can see id tokens and access tokens in local storage. Renewal seems to be working (for a while) - I've set tokenRenewalOffsetSeconds to be 1000 to test, and haven't hit any problems. Then, after not using the application for a few hours, attempting to fetch a token using
await userAgentApplication.acquireTokenSilent(tokenRequest);

results in the following log messages and error:

Wed, 28 Aug 2019 15:43:42 GMT:1.1.3-Info Add msal frame to document:msalRenewFramehttps://microsoft.onmicrosoft.com/richcodenav/read_public
Wed, 28 Aug 2019 15:43:42 GMT:1.1.3-Info LoadFrame: msalRenewFramehttps://microsoft.onmicrosoft.com/richcodenav/read_public
Wed, 28 Aug 2019 15:43:43 GMT:1.1.3-Info Add msal frame to document:msalRenewFramehttps://microsoft.onmicrosoft.com/richcodenav/read_public
Wed, 28 Aug 2019 15:43:47 GMT:1.1.3-Info Returned from redirect url
Wed, 28 Aug 2019 15:43:47 GMT:1.1.3-Info Processing the callback from redirect response
Wed, 28 Aug 2019 15:43:47 GMT:1.1.3-Info State status:true; Request type:RENEW_TOKEN
popup.ts:28 Unable to get logged in user:  InteractionRequiredAuthError: AADSTS50199: Conditions to show CMSI interrupt are met.
Trace ID: 797509dd-e2a8-4b59-a034-eeb3906c0800
Correlation ID: 06451745-8778-4191-b975-1feaaa062d68
Timestamp: 2019-08-28 15:43:45Z
    at InteractionRequiredAuthError.AuthError [as constructor] (chrome-extension://dckkmoeekhppkjieeeijihmepmfgeoll/js/vendor.js:180485:28)
    at InteractionRequiredAuthError.ServerError [as constructor] (chrome-extension://dckkmoeekhppkjieeeijihmepmfgeoll/js/vendor.js:180950:28)
    at new InteractionRequiredAuthError (chrome-extension://dckkmoeekhppkjieeeijihmepmfgeoll/js/vendor.js:180889:28)
    at UserAgentApplication.push../node_modules/msal/lib-es6/UserAgentApplication.js.UserAgentApplication.saveTokenFromHash (chrome-extension://dckkmoeekhppkjieeeijihmepmfgeoll/js/vendor.js:179753:25)
    at UserAgentApplication.push../node_modules/msal/lib-es6/UserAgentApplication.js.UserAgentApplication.processCallBack (chrome-extension://dckkmoeekhppkjieeeijihmepmfgeoll/js/vendor.js:179287:29)
    at UserAgentApplication.push../node_modules/msal/lib-es6/UserAgentApplication.js.UserAgentApplication.handleAuthenticationResponse (chrome-extension://dckkmoeekhppkjieeeijihmepmfgeoll/js/vendor.js:179389:14)
    at new UserAgentApplication (chrome-extension://dckkmoeekhppkjieeeijihmepmfgeoll/js/vendor.js:178530:22)
    at Object../src/auth.ts (chrome-extension://dckkmoeekhppkjieeeijihmepmfgeoll/js/vendor.js:234848:30)
    at __webpack_require__ (chrome-extension://dckkmoeekhppkjieeeijihmepmfgeoll/js/login_return.js:79:30)
    at Object../src/login_return.ts (chrome-extension://dckkmoeekhppkjieeeijihmepmfgeoll/js/login_return.js:174:14)

That error code is nowhere in documentation (which is another place for thing that would be helpful). Any ideas? Thanks!

I would expect so! interesting.  @sameerag  @pkanher617  have you seen this before? Any thoughts?
If not lets get a call set up @jepetty  to debug together!

Thank you, I appreciate the help! Feel free to ping me (alias jepetty) as well to set up a meeting or if you need any more information. :)

Ping... any updates on this? Shall we set up some time to chat?

@jepetty Do you have a functional sample repro you can share? I've seen issues like this recently (e.g. #941), seems like this might be similar. Feel free to share on email/teams. Thanks!

I agree, they do look pretty similar! I sent you an email, happy to help out as much as I can in diagnosing this :) Thanks!

Working with @jepetty to improve MSAL usage, but for anyone else who gets the AADSTS50199: Conditions to show CMSI interrupt are met error, it is thrown when your app uses a non-https redirect uri (e.g. such as chrome-extension://). We'll make sure this gets added to the list of error codes.

Another note: this error will not show for Chrome extensions running in Chrome, but will show for Chrome extensions running in non-Chrome browsers such as the new chromium-based Edge. We have opened a ticket internally to get the new Edge whitelisted (MSFTers can ping me on Teams if they want more details on that).
Closing, as no further actionable items at this time.

Thanks for all your help @jasonnutter! You're the bomb!


Documentation lacks clarity on adding control-plane nodes with external etcd using kubeadm

What happened:
When attempting to add a new control-plane node to a Kubernetes cluster with an external etcd, using the default kubeadm command resulted in the external etcd cluster's certificate being overwritten or corrupted.
What you expected to happen:
The documentation on setting up a highly available Kubernetes cluster with kubeadm and an external etcd should provide clear warnings or guidance on this potential issue. I followed the instructions from the following links:
Setting up a HA etcd cluster with kubeadm
Setting up HA Kubernetes with kubeadm
Steps to reproduce:
Set up a Kubernetes cluster with an external etcd following the official documentation linked above.
Attempt to add a new control-plane node using the default kubeadm command as suggested in the documentation.
Solution that worked:
I had to create a custom kubeadm configuration file:
apiVersion: kubeadm.k8s.io/v1beta3
kind: JoinConfiguration
controlPlane:
  localAPIEndpoint:
    advertiseAddress: "<IP_ADDRESS>"
    bindPort: 6443
  certificateKey: "f7b1fd2db6f82731617d32f4991c11d56e49c140aea3f9ea50efc36825725028"
discovery:
  bootstrapToken:
    apiServerEndpoint: "<IP_ADDRESS>:6443"
    token: x63wk9.sfxq39oynypxdemf
    caCertHashes:
    - "sha256:d8894fe8ca0eff1928c4ee8bcb7c6d1af323b0ad0f05c7bf583bee6f02243e5f"
  tlsBootstrapToken: x63wk9.sfxq39oynypxdemf
etcd:
  external:
    endpoints:
      - https://<IP_ADDRESS>:2379
      - https://<IP_ADDRESS>:2379
    caFile: /etc/kubernetes/pki/etcd/ca.crt
    certFile: /etc/kubernetes/pki/etcd/server.crt
    keyFile: /etc/kubernetes/pki/etcd/server.key

And then used the following command to join the node:
kubeadm join --config=kubeadm-config.yaml --ignore-preflight-errors=FileAvailable--etc-kubernetes-manifests-etcd.yaml

/language en
/sig cluster-lifecycle

Thank you for the suggestion! I'm delighted to contribute and help improve the documentation. However, it's my first time contributing to the Kubernetes documentation, so I might need some guidance or references on how to proceed. I'm eager to learn and make a meaningful contribution!

the website has a guide here:
https://kubernetes.io/docs/contribute/


Database adapters for sessions

Since we've moved to session tokens, it'll be great if we can support in-memory data store (ie. Redis) for storing sessions. I think we can provide an additional adapter slot specifically for sessions, which would allows users to use a different database for storing sessions.

Added support for session adapters and published a session adapter for Redis with v0.14.1


Update README.md and add screenshot of visualizations

Also set scrollbar to auto on the visualizations in case of using a smaller screen


Support field properties with values that have :  in them

This is a pretty simple change for cases where field properties have the string ": " in them—it just ensures the line is only split one time on ": ", since everything after the first ": " should be the value.
Fixes #48.

thanks!

No problem!


.toString() Error Piping to CSV for Large Queries

I've read older threads about this issue and thought that it had been resolved.  I am receiving this error when piping data to a CSV.  It only happens to me when writing more than a million records but I'm not sure if it has to do with the number of columns selected as well.  For smaller query results everything seems fine.
Please inform.
Thanks,
Rob

I forgot to mention that I am using the SalesForce conn.bulk.query operation.
Does anyone have any information on this?  My project is falling behind with no option at this point.  Yikes!

Faced something similar.Please see https://github.com/jsforce/jsforce/issues/615#issuecomment-304396624

closing this in favor of #615


hub75: don't enable an unrelated IRQ

DREQs are not IRQs, DREQ_PIO0_TXn happens to align with TIMER_IRQ_n so this enabled a timer IRQ.
(This is something I happened to notice while poking around in there)

More importantly, just noticed that stop would disable that IRQ which seems a lot more likely to break things...

😬 I feel this pattern might be copied into a few places, will have a hunt. Thank you!


npm run typegen - generate TypeScript declaration files

Fixes https://github.com/xenova/transformers.js/issues/25
This is a basic type setup, which enables type-checking/Intellisense aswell:

I just added two example JSDoc comments, to display how it works.
Since @chelouche9 asked for it, I would like to assign you to work out more types. Are you capable of doing so?
Whenever you have added more types, you can run:
npm run typegen
And commit both the JSDoc comments and the dist/types changes, so we have synchronized types.

Wow this is great! Thanks so much! I'll wait for @chelouche9 to respond before proceeding to merge into main.
Excuse the potentially silly question (I am not too familiar with typescript haha), but does it have to be a dependency on the library itself, or can it be made into a dev dependency?
I could also try to add types while I write the code. I assume I just follow what you did and add annotations to the methods?

@kungfooman sure, I would be happy to add more types to the project. Just to make sure I understand, I need to add new JSDoc comments, then run the typegen command and the declaration files will be created?
@xenova anything I need to know about running the project, tests, etc..?

@xenova anything I need to know about running the project, tests, etc..?

My "test-suite" (if you can even call it that) is very simple. I made it before actually creating the npm package, so it is still just an ugly script that I run with npm run test (https://github.com/xenova/transformers.js/blob/main/tests/index.js)
To run the tests with local models (ofc much faster for development), you need to have the models stored in the models folder (https://github.com/xenova/transformers.js/tree/main/models) with the same structure as the HF repo I use to store the remote models: https://huggingface.co/Xenova/transformers.js. For example, you will have folders like this:
./models/onnx/quantized/openai/whisper-tiny.en/speech2seq-lm-with-past
Alternatively, you can disable using local models:
https://github.com/xenova/transformers.js/blob/4ad2581604f2b8da37e8842f9453d02ae1c4f5e4/tests/index.js#L14
but since I haven't implemented local caching, you will end up downloading the model each time you run the tests (not ideal)

@xenova great, I will start working on that once you merge :)

@xenova great, I will start working on that once you merge :)

Actually, do you mind working on this branch (or a fork of it)? Then I can merge it when it's all ready?

Sure. Although it might take some time to complete all, I though we can split the work to not lose track of the master.

Sure. Although it might take some time to complete all, I though we can split the work to not lose track of the master.

Great! The library is expanding extremely quickly, so there might be quite a lot of breaking changes, but many functions should remain the same (and I will take responsibility to manage any conflicts before merging into main)

Excuse the potentially silly question (I am not too familiar with typescript haha), but does it have to be a dependency on the library itself, or can it be made into a dev dependency?

Sorry, my bad, TypeScript is of course a dev dependency only. I fixed the package.json and package-lock.json via npm install --save-dev typescript.

I could also try to add types while I write the code. I assume I just follow what you did and add annotations to the methods?

Right, it's rather simple but some types can become complicated aswell. JSDoc is pretty powerful, but if I don't know the syntax, I've had good experiences with just asking OpenAI :sweat_smile:

@kungfooman sure, I would be happy to add more types to the project. Just to make sure I understand, I need to add new JSDoc comments, then run the typegen command and the declaration files will be created?

That's right, everything is quite straight-forward with types. :+1:

Sure. Although it might take some time to complete all, I though we can split the work to not lose track of the master.

I have the same sentiment, typing is something that developes over time. Usually when I'm stuck with some code that I cannot understand so far, I add types as a mental prop e.g.
It doesn't take too much time, in VSCode you just move the cursor over e.g. a method, type /**[enter] and it will auto-generate the entire @param template.

Right, it's rather simple but some types can become complicated aswell. JSDoc is pretty powerful, but if I don't know the syntax, I've had good experiences with just asking OpenAI 😅

I am using it as well to do all the heavy lifting 😝

Sorry, my bad, TypeScript is of course a dev dependency only. I fixed the package.json and package-lock.json via npm install --save-dev typescript.

I added it to my PR. I cannot push to your branch @kungfooman. If you guys have a better idea to manage those branhes, let me know.

#28

I added it to my PR. I cannot push to your branch @kungfooman. If you guys have a better idea to manage those branhes, let me know.

Thank you for being pro-active! I added you both as collaborators on my fork, so that should work, once you accepted the invite.
I hope you know how to add remotes and pull/push to them

@kungfooman I applied all the changes on my fork already. I will keep doing it from there :)

@kungfooman I applied all the changes on my fork already. I will keep doing it from there :)

Nice, so either:

This will be merged first and you can merge main branch
Your PR will be merged directly

I have no hard feelings for either, I'm glad about your support :muscle: :+1:
Once a typing base exists, it will be easier to make more fine-tuned type fixes here and there in smaller/more-manageable PR's

@kungfooman are you actively work on this branch?
I though for some reason you have made the setup and stopped

@kungfooman are you actively work on this branch?

I just keep it up to date basically, so either way (1) or (2) should work out nicely


vaultId on App

Description

Add vaultId to App.
Introduce AppResource (using it at creation only)
Introduce DatasetResource (not really used yet)
Backfill to set vaultId on Apps

Risk
Low
Deploy Plan

run db migration
deploy front
run ts migration

db migration was run, deploying


Missing remove buttons for selected options in cascade

The remove button only appeared for the first selection option, others weren't appearing like they are supposed to.
Expected result:
Remove buttons to appear for all selected options
Reproduction:
https://stackblitz.com/edit/github-52q5xr-yeyoni?file=scripts%2FgenAntdStyle.ts,pages%2Findex.vue

https://github.com/vueComponent/ant-design-vue/issues/6939


[New Feature]: Warn if number of installer nodes is inconsistent between versions

Description of the new feature/enhancement
This suggestion comes from the discussion in https://github.com/microsoft/winget-pkgs/issues/92590.
When new manifests are submitted, the validation pipelines should be able to identify if the number of installer nodes differs from the previous version's manifest. This can be the basis for a set of validation parameters which could trigger warnings.
Proposed technical implementation details (optional)
First iteration - Apply a label such as Possible-Missing-Metadata if number of installer nodes is less than previous version.
Second iteration - Compare not only the number of installers, but also the node type. For example - v1.0 contains x86-user-exe and x86-machine-exe, v2.0 (in PR) contains x86-unspecified-exe and x64-unspecified-exe; even though number of nodes are the same, not all the node types from the first manifest are present in the second manifest. Apply the label.
Third Iteration - Compare not only number and type, but also metadata fields. Example - v1.0 contains AppsAndFeaturesEntries (or ReleaseDate, Commands, FileExtensions, etc.) but v2.0 (in PR) does not. Apply the label

@stephengillie - Area-Validation-Pipeline please

One reason why newer manifests don't retain metadata from the previous ones is simply because their fork is out of sync with the upstream branch. Many users use YamlCreate to update manifests and it uses the local latest manifest as the base manifest for an update. If the users don't update their fork and git pull into their local repo, they won't have the latest changes that were applied to the older manifest. (I recently encountered this where I added the scope-related switches for a manifest to resolve an issue and a newer one was added by a user using YamlCreate that did not contain the switches, causing the original issue again where users were not able to upgrade the package due to scope mismatch. I noticed the user's fork was some 3000 commits behind the upstream)
We can mitigate the issue if we update YamlCreate to fetch the latest manifest from the upstream repo, or have an option for users to sync their branch first before updating a manifest. The issue doesn't appear with winget-create as it always fetches the latest manifest from the repo.


sign.sh is very out of date

The sign.sh in the root directory still refers to the now removed zep2newt.py script. Either just remove this script entirely, or update it to be and example of how to sign. Given that samples/zephyr/Makefile shows how to sign on Zephyr, and newt takes care of it on MyNewt, it is probably best to just remove the file.

docs/readme-zephyr.md contains the sentence: There is a sign.sh script that gives some examples of how to make these signatures. This should probably be updated with an example showing the command line flags similar to what is currently used in sign.sh and then the sign.sh file can be removed.


Updated validations.py python script

Updated validations.py python script
Fixed the behavior of validate_user funtion in validations.py
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!

@googlebot I fixed it.


Use the appropriate method when performing a request

This fixes droplet creation

@aidanhs since I haven't had a ton of time to work on this crate, if you'd like I can make you a collaborator on this project which would give you push/merge access and @homu access. Let me know if you're interested.

@homu r+

:pushpin: Commit 5e488e4 has been approved by kbknapp

:hourglass: Testing commit 5e488e4 with merge 68de6aa...

:sunny: Test successful - status

@kbknapp I'm up for that.

Excellent! I've added you as a collaborator, and I'll add you to @homu once I get home. Feel free to push your branches to this repo instead of a your fork if you'd like. Once I add you to homu you'll be able to use that to merge as well.

Great, thanks!

Oh, and could you push a new version of doapi-rs to crates.io when you get home as well? Just while you've got doapi on your mind :)

Will do!

@aidanhs you've been added.


doc: add Railway.app deployment

View deployment from Railway:
https://pingvin-demo-production-8dd1394.up.railway.app
This Git commit includes an update to the README.md file, which adds instructions for deploying with Railway. The instructions include detailed steps for both one-click and manual deployment, as well as links and images. The commit contains two patches, one to fix a spelling error in the README.md file, and another to add detailed deployment instructions for Railway to the README.zh-cn.md file, as well as correcting a spelling error. An appropriate commit message might be "Add Railway deployment guide and fix spelling errors". The content of this commit is aimed at explaining how to use the Railway deployment platform, making it easier for users to deploy their projects quickly, and improving the quality of the documentation by fixing spelling errors.
该 Git 提交包括 README.md 文件的更新，添加了关于使用 Railway 部署的说明。其中提供了一键部署和手动部署的详细操作步骤，并附带了链接和图片。该提交中包含了两个补丁，一个是修复了 README.md 文件中的错误拼写，另一个是在 README.zh-cn.md 文件中添加了使用 Railway 部署的详细步骤，并修正了一个错误的拼写。适当的提交信息可能是 "Add Railway deployment guide and fix spelling errors" (添加了 Railway 部署指南和修正拼写错误)。该次提交的内容是介绍如何使用 Railway 部署平台，方便用户快速部署项目，并且修复了一些拼写错误，提高了文档质量。

Hey, thanks for your contribution! I've just tested your template and I noticed that the database and files are not persistent. E.g if I redeploy Pingvin Share all users and shares get cleared. I'm not experienced in Railway but do you know if Railway supports persistent volumes?

Here are some ways to persist data:
Database

Railway provides support for various databases such as PostgreSQL, Redis, MongoDB, and MySQL.
Alternatively, you can use an outside database provider like Mongo Atlas. However, migrating the current database to a different structure may be necessary.
Deploy Prisma as another production environment in the same Railway project manually, expose the DB connection parameters into the pingvin environments.

File
Amazon S3 API support, and that's how Send stores files.
If we deploy on an edge compute platform like Railway, adding S3 support will be the most important step since Railway provides limited disk space and memory.

Hmm okay so yeah S3 storage would be nice in this case. I'm not sure how difficult this is to implement as I don't use AWS.
Pingvin Share uses SQLite, so this could get really difficult. We could store the SQLite file on S3 but this could hurt the performance and it could get expensive too as this would require many reads. Do you have a better idea?

Get it. I think we can deal the database and the files in separate ways:
Database
Use Railway provided databases (MySQL, PostgreSQL, Redis, Mongo) or Mongo Atalas (Cluster) as alternative for SQLite.

This requires more work and will change the current app structure.

I will work on this first to ensure database persistence.
File
With limited life cycle of the (uploaded) files, we can use S3 bucket as cache to store (uploaded) files temporally, here's the docs of AWS s3 JavaScript SDK.

S3 is currently supported by many cloud service providers. Take an example, Cloudflare provides 10GB/Month free bucket with fair share bandwidth.

I will work on adding S3 feature if time is available.
**Mounting cloud drive (But I not suggested do so)
Mounting personal cloud drive, e.g. OneDrive, as a disk to store (uploaded) files is another choice, which will not change current project structure and easy to get started.

But it required third-party tools, more work for deployers when deploying the forks, especially on cloud platforms. Moreover, it might violate the cloud drive agreements.

We still have to support SQLite to enable an easy installation with Docker. Maybe there is a way to configure Prisma that the user can choose the database but this could get really difficult as Prisma doesn't support generic database migrations.

With limited life cycle of the (uploaded) files, we can use S3 bucket as cache to store (uploaded) files temporally, here's the docs of AWS s3 JavaScript SDK.

What exactly do you mean with "we can use S3 bucket as cache to store (uploaded) files temporally"? Where would you store the files permanently then?

OK, I will try to use docker-compose to separate SQLite from the all-in-one docker image next step.
I meant that we could use

local drive
S3 bucket (only if set S3*** ENVs)

to store uploaded files. All files with life cycle limit or just permanent.
If we deploy on cloud, using S3 would benefit a lot since cloud platform offers quite limited disk space.
Since this PR is not appropriate to be merged currently, I will close it and make an issue instead. How about that?

Yep, let's create an issue👍


How to secure all the pages of the app?

I am trying to develop an app that has multiple screens. Whenever the user goes to the background and then comes to foreground, then I should be prompted to authenticate before going to other screen. I have tried many things but unable to understand how to implement it.
Can you help me on how to do that  @neckaros ?

Hello
Did you try to have it as described in the read me placement section?
https://github.com/neckaros/secure_application#placement

Yes, I read the complete Readme and used it as specified in the Readme. Sorry, If i wasn't complete give details of my problem.
What I am trying to accomplish is that

Anytime the app goes in the background, then it comes to foreground, I should be prompted to unlock the screen before entering in the app and this should be applied to any screen that the user is on.

What my issue is

Whenever I once unlocked the application using the lockedBuilder and navigated to the other screen, then the app is opened for every other screen, even though I have applied the secure gate just below material app.
The app doesn't get secured in the recent app option, even though there is no error in the logs

What I have tried

I have tried the AppLifeCycleState to manage when the app goes to the background, then comes to foreground. But since there is no context in the didAppLifeCycleStateChanged, so it gives me context not found error.

I have to dynamically build the screen using SecureApplicationProvider, But It doesn't work too.

I hope i am able to give the question detail clearly @neckaros.

To sum up even after unlock and moving to another screen you still want the app to be protected . That should be how it works (and is how it works in my own app)
How do you unlock when the user get back to the app? You should use unlock() and not open()
Are you able to reproduce in a minimal app?

Yeah that sums up. I tried the unlock not open. I will implement a minimal app. Then update here if the issue is resolved or not.

Hello do you still have the issue or can i close it?

Sorry to for not replying earlier. I just implemented the security natively in the android and ios. So I haven't used the library there. You can close the issue. Thank for your reply.


Solr server could not be contacted properly

I followed the below link to set up the apapche solr setup its throwing the error not able to see the databases and dashboard part
https://docs.gethue.com/administrator/configuration/connectors/#dashboards
please find the below steps
step1: hue folder
./build/env/bin/pip install sqlalchemy-solr
step2:
added the below code in the database section in pseudo-distributed.ini file but didn't workout
[[[solr]]]
name = Solr SQL
interface=sqlalchemy
options='{"url": "solr://:@:/solr/[?use_ssl=true|false]"}'
Step3: in interpreter already the below code was defined and it was commented as well,so i uncommented and tried still facig the [issues.](url

)
Interpreter:
[[[solr]]]
name = Solr SQL
interface=solr
Name of the collection handler
options='{"collection": "default"}'
getting  the below error while login to the hue server.
Solr server could not be contacted properly: HTTPConnectionPool(host='localhost', port=8983): Max retries exceeded with url: /solr/admin/info/system?user.name=hue&doAs=admin&wt=json (Caused by NewConnectionError(': Failed to establish a new connection: [Errno 111] Connection refused'))

any update on this issue?

can you try this way ?
https://gethue.com/solr-search-ui-only/
ie by adding :
[search]
solr_url=https://mysolr.example.com:5001/solr


bug(mat-form-field): label is not aligned when using text-align center

Reproduction
https://stackblitz.com/edit/components-issue-d6xpva?file=src%2Fapp%2Fexample-component.html

Steps to reproduce:

click inside the input

Expected Behavior
When the input is focused, the label and the gap in the outline should be properly aligned.
Actual Behavior
The gap is placed based on the initial value of input so if the initial value of input is not set , the label will appear misaligned.
Environment

Angular:
CDK/Material: 12.2.9
Browser(s):
Operating System (e.g. Windows, macOS, Ubuntu): Windows

We don't support this, because it isn't in the Material design spec.


Create container for PEPPAN

There is no container available for PEPPAN

added preliminary main.nf to PEPPAN module calling customized container from docker://cidgoh/peppan:latest

Tested it and its  working as expected


Move project to scrapy github organization

What do you think about moving the project to @scrapy organization, as it is used in Scrapy tutorial? We may also rename the project to quotes.toscrape.com, to make it more discoverable.

I've no pushbacks on moving it to @scrapy org, but I'm not sure whose decision this should be.

We may also rename the project to quotes.toscrape.com, to make it more discoverable.
Sounds a bit odd, specially because toscrape.com is a Scrapinghub's domain and the website is hosted on our infrastructure. Maybe quotes-toscrape?


Add automatic saving

Add automatic saving of plugin configuration
Before:
Plugin.Instance.Configuration.Instance.Test.Add(test);
Plugin.Instance.Configuration.Save();

After:
Plugin.Instance.Configuration.Instance.Test.Add(test);
I have demonstrated how the new save should look like, that is, without the extra code.

I think its a bad idea because it can be a breaking change, also because even adding a default value bool autoSave = false, won't work, because the method signatures will be the same and it won't be compiled


Add yank highlights duration and colour customisation

Adding on to my comments from the pull request that added yank highlights #2377. Currently the yank highlights are too quick to be usable for me.
Comparing to vscode the default in vscode is a lot more obvious.
It would be good, like vscode vim, to add the options vim.highlighted.yank.color and vim.highlightedyank.duration so that the length and the color of the highlight can be customised.
(these are outlined in vscode vim readme https://github.com/VSCodeVim/Vim)

Thanks for logging this, @yunti ! Was actually just thinking about this - I was trying to record a video of these for presentation on Sunday and they were really faint. I like the idea of the config settings from VsCodeVim, I'll stick with that precedent 👍

The fix for this in #2402 works perfectly. Thanks.


Allow logging the PC/SC-Lite API call results

Add the implementation of logging the results of the PC/SC-Lite API
calls that are made from a NaCl module. This should aid in the
future in the investigation of bugs in third-party middleware
extensions.
The implementation is disabled by default, since even under normal
circumstances a typical smart card middleware makes dozens of
PC/SC-Lite requests per single certificates listing / signature
operation.
This additional logging can be enabled by running these snippets in
the JavaScript console of the extension's background page:

enabling the logging of return codes:
$jscomp.scope.pcscLiteNaclClientBackend.logger.setLevel(goog.log.Level.FINE)

additionally, enabling the dumping of all values and buffers (use
with care - this may include privacy/security-sensitive data) :
goog.DEBUG=true

follow-up: the change has no reference to anything related to "goog.DEBUG=true", I guess that was just mentioned for documentation purpose?

follow-up: the change has no reference to anything related to "goog.DEBUG=true", I guess that was just mentioned for documentation purpose?

Yes, this is simply a documentation, and it's correct: the goog.DEBUG is a standard Closure variable that is used in a number of places, in particular in the debugDump() function that I'm calling in this CL.

follow-up: the change has no reference to anything related to "goog.DEBUG=true", I guess that was just mentioned for documentation purpose?

Yes, this is simply a documentation, and it's correct: the goog.DEBUG is a standard Closure variable that is used in a number of places, in particular in the debugDump() function that I'm calling in this CL.

I've added a mention of this variable into the inline comments.


Make images and section links relative to the root directory

On sub-pages, say for a blog, the navigation and images won't work because their paths are relative to the current directory.  This works well for single-page sites, where everything is always at the root, but not in this case.
We need to prepend a / to all of the locations so that we can explicitly state that all paths start at the root.

So if going down this path, it'd be good to have an example, which is where it gets a little tricky. Would the example have the same head as the main page? And if so, it should probably be templated appropriately.
At present, if we added an empty page to content/blog (in the example site) we'd just get a blank page of course. I think I'd prefer to have no example link, or a fully working example.

En las subpáginas, digamos para un blog, la navegación y las imágenes no funcionarán porque sus rutas son relativas al directorio actual. Esto funciona bien para sitios de una sola página, donde todo está siempre en la raíz, pero no en este caso.
Necesitamos agregar un prefijo /a todas las ubicaciones para poder declarar explícitamente que todas las rutas comienzan en la raíz.

I believe that in addition to this proposal, which is very timely, we must also define a base template, the theme currently does not have it, I have had to make all these changes to me, and the initial template has been partially rendered.

What do you mean by a base template? Different to the basic config.yml in the exampleSite directory?

When I refer to base template, I don't only refer to not limiting the use of the content from the config.toml (It's well done), but instead, expand and promote the use of section page templates, the theme that you publish is amazing, but with great respect, I made changes to improve the ease of making publications in a production site:

The menu you implement it from the config.toml

# Navigation section
[params.nav]
  mission = "Our Mission"
  services = "Services"
  team = "The Team"
  contact = "Get In Touch"
  # Include logo instead if HOME:
  # logo = "path/to/logo"

I preferred to use the front matter method of defining menu entries:

  {{ range .Site.Menus.main }}

      {{- .Name -}}
      {{- with .Page -}}

        {{- dateFormat " (2006-01-02)" .Date -}}

      {{- end -}}

  {{ end }}

The content of the site is limited to changing the config.toml, and I think it is more efficient to link it to Front Matter, so I deleted a certain logic from its configuration file and adapted it to the folder content.

On sub-pages, say for a blog, the navigation and images won't work because their paths are relative to the current directory. This works well for single-page sites, where everything is always at the root, but not in this case.
We need to prepend a / to all of the locations so that we can explicitly state that all paths start at the root.

I miss putting absolute paths in the partial header.html and js.html, maybe here they are even more important:

<link rel="stylesheet" href="/css/animate.css">

<link rel="stylesheet" href="/css/icomoon.css">

<link rel="stylesheet" href="/css/simple-line-icons.css">

<link rel="stylesheet" href="/css/magnific-popup.css">

<link rel="stylesheet" href="/css/bootstrap.css">

<link rel="stylesheet" href="/css/style.css">

since by extending a new section (for example: a blog), assets would not work.

{{ define "title" }}
  {{ .Title }} – {{ .Site.Title }}?
{{ end }}
{{ define "main" }}
    {{ partial "nav.html" . }}

{{ .Title }}

        {{ .Content }}
        {{ .Params.author }}

{{ end }}

So @braian125 I think you're talking about a different thing here, so I've opened another issue.


Added Sign-in/Sign-up page for Buyers

Fixes #547
Description
Added an option for signin/signup for the buyers which leads to the signin/signup form for the buyers.
Screenshots
https://github.com/Anushkabh/krishiconnect/assets/142666229/3b4ec4d5-c3a0-48b2-91fa-536805cd03bb

Please review my pull request @Anushkabh

Please review my pull request @himanshu-03

Please review my pull request @himanshu-03 @Anushkabh

Hey @samarsajad !
Please resolve the merge conflicts. Our repo underwent with a series of major changes since last 2 weeks. I request you to please sync the latest changes and resolve the conficts raised.

Make sure the images, style files and the scripts are in the desired directory.

Make sure the images, style files and the scripts are in the desired directory.

can u please mention where do you want all these files ? @himanshu-03

Author

You can check the repo and see that we have all this files in the assets directory.
We have css, js, images directory respectively over there.

Made the changes. Kindly take a look  @himanshu-03 @Anushkabh

LGTM!

I made the changes and it is working as expected. Please review it and let me know if you want any more changes @RitiChandak @himanshu-03

Can I please open a new pull request for this issue as I am Facing issues in pushing the changes in the same branch? @himanshu-03 @RitiChandak

I made the changes please take a look @himanshu-03 @RitiChandak @Anushkabh

I created a new pull request because this one had some issues that couldn'tr be solved . PTAL #854 @himanshu-03 @RitiChandak @Anushkabh


Where is it used IOU LOSS  in project PVRCNN ?

The paper mentionde it:In RCNN stage, use iou loss to train, but no representation of it was found.
can somebody give me a hand?
best wishes!

https://github.com/open-mmlab/OpenPCDet/blob/26a16123cf820e29ed565f4ad53964c66f3ea030/pcdet/models/roi_heads/target_assigner/proposal_target_layer.py#L44

https://github.com/open-mmlab/OpenPCDet/blob/26a16123cf820e29ed565f4ad53964c66f3ea030/pcdet/models/roi_heads/target_assigner/proposal_target_layer.py#L44

Thaks a lot!


detekt-gradle-plugin 1.5.1 is not published to Maven Central

Expected Behavior
All of detekt 1.5.1 is available from Maven Central
Observed Behavior
detekt-gradle-plugin 1.5.1 has not been published
Steps to Reproduce
https://search.maven.org/search?q=g:io.gitlab.arturbosch.detekt AND a:detekt-gradle-plugin
Context
I was able to use the rest of detekt 1.5.1 without adding other repositories, but this one failed.
Your Environment

Version of detekt used: 1.5.1
Version of Gradle used (if applicable): N/A
Operating System and version: N/A
Link to your project (if it's a public repository): N/A

Hi @Nycto, the Gradle plugin is only published to the Gradle plugins repository. You can find 1.5.1 there: https://plugins.gradle.org/plugin/io.gitlab.arturbosch.detekt


Feature/slim 2835 event types

SLIM-2835: Make the EventTypes more specific, and add the EventCode to the 'smartmetering' Event

retest this please

retest this please


🐛 Bug Report: ExpandableNavigation causes copy code button to be in the wrong place

📜 Description
ExpandableNavigation TechDocs Addon is enabled.
👍 Expected behavior
Not change how other components are rendered and the copy code button should be well aligned on the right.
👎 Actual Behavior with Screenshots

Note: enabling the TextSize addon seems to fix this behavior.
Instead its rendered on the left and kinda squished.
👟 Reproduction steps

Enable the ExpandableNavigation TechDocs Addon
View docs with a fenced code block

📃 Provide the context for the Bug.
No response
🖥 Your Environment
Backstage 1.18.5
Chrome (latest)
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

Hi @kunickiaj, thanks for willing to submit a pull request, I assigned the issue to you 🙌🏻
Do you have any recommendations @backstage/techdocs-maintainers ?

@kunickiaj Can you provide your techdocs config? I am not able to reproduce on the current master (6bad140c57257802d0ce6b1271622d5d88b269c5) with the default image spotify/techdocs:v1.2.3 building in docker.
I've tried with and without the <TextSize/> addon.


Highlight modified lines not working

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.76.1 (user setup)
OS Version: Windows_NT x64 10.0.22621 (Windows 11 Home)

Steps to Reproduce:
The issue is that the highlighting modified lines feature is not working when I open VS Code in any git project in Windows. I have WSL (Ubuntu 22) installed and when I open VS Code there, the highlighting modified lines feature works correctly. I has able to replicate the error with:

Install VSCode 1.75.1 (official download)
Open VSCode in any git project in Windows OS with code --disable-extensions
The highlight modified lines feature works correctly
Update VSCode to 1.76.1 using Help/Check Updates or official download
Open VSCode in any git project in Windows OS with code --disable-extensions
The highlight modified lines features does not work
You can open any git project in WSL (Ubuntu 22) with code --disable-extensions and the modified lines would work correctly. This is true for 1.76.1 and 1.75.1

Thank you very much for your time. Let me know if I can help you out in anything else.
Best,
Nico

/duplicate of #176749

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Add freezed JSON serialization

Changes
Fixes #2468
Checklist

[x] An issue to be fixed by this PR is listed above.
[x] New tests are added to ensure new features are working. Please refer to this page to see how to add a test.
[x] ./frb_internal precommit --mode slow (or fast) is run (it internal runs code generator, does auto formatting, etc).
[x] If this PR adds/changes features, documentations (in the ./website folder) are updated.
[x] CI is passing. Please refer to this page to see how to solve a failed CI.

Remark for PR creator

./frb_internal --help shows utilities for development.
If fzyzcjy does not reply for a few days, maybe he just did not see it, so please ping him.

I am not sure how I can update all the tests for @freezed at once as there are many tests that use it.

Try
./frb_internal precommit-generate

which updates code generation outputs


Fix: OpenAI gets instantiated before running constructor

Fixes #193
The problem was that the default for the openai instance var gets instantiated before the constructor is run, thus resulting in an error when the env variable is missing.

So strange.
Does OpenAI maintain global state as part of the OpenAI instance initialization?

No, it's just that private openai: OpenAI = new OpenAI({}); is executed before the constructor.

(and it throws if it does not have an apiKey as constructor or the env variable)


Add function to highlight hashtags

Progress / Closed on issue #82 Highlight Hashtags
What it Does

Words in tweets preceded with the # symbol will now appear highlighted in Twitter blue.
Add function to highlight hashtags
Add example tweets to AWS

How I Tested

Run the application
Look through your collections for saved tweets containing the # symbol
Words beginning with # will appear blue. Words with the # symbol in the middle will not be

Screenshot

Thanks @mikaelacaron. I kind of got it right after messaging. I reset everything and redid it. If I'm still having trouble in the future I'll hit you up!

Can you install SwiftLint, it seems you haven't and there are 4 errors related to it when building the project

Hi @mikaelacaron. Any attempt to shorthand these results in an error - "Binary operator '+=' cannot be applied to two 'Text' operands." Do you want to try a different approach or do you prefer I tell lint to ignore that for this function.

Hi @mikaelacaron. Any attempt to shorthand these results in an error - "Binary operator '+=' cannot be applied to two 'Text' operands." Do you want to try a different approach or do you prefer I tell lint to ignore that for this function.

@MarshallARoss  Generally I don't want to comment out SwiftLint rules, but for this I'll allow it. Add // swiftlint:disable shorthand_operator right before the extension.
Also all your comments, fix the SwiftLint warnings for those, and all the if comments, put them inside the if { } or else { }, not before them
// Check if # is at the beginning of a string
if prefixString == "" {
   output = output + Text(" ") + hashtagText

 } else if prefixString.contains("\n") {
   // Check if # is at the beginning of a word preceded by blank lines
   output = output + Text(" ") + Text(prefixString) + hashtagText

} else {
   // if # is not at the beginning of a word do not highlight
   output = output + Text(" ") + Text(String(word))
}

Also for the comment before the function use Cmd + Shift + / which will give you this, and fill it in appropriately. This will allow your comment to show up when clicking on "quick help"
/// Description
/// - Parameter tweet:
/// - Returns:

On your Tweet+Extension you don't need to put Optional in front of everything, and and use line breaks to fix most of the line length violation warnings

@MarshallARoss This tweet: the #help... is highlighted when only the #help should be highlighted
https://twitter.com/_heymikec/status/1582111738798829569?s=21

Updated and resubmitted. #164


Background download/upload

I found one issue each with background downloader and uploader.
The downloaded files are always corrupted. I made few of my onedrive files downloadable by public with Urls. However, all of them, be it image, docx or even text file, were corrupt on download. I guess the headers are not properly downloaded. I am not sure.
The upload never happens. I made one of m onedriver folders uploadable to all. I tried uploading files to that folder, nothing happens.

This is an old issue. Are you still experiencing the problem? If so, I will reopen.


Safari Extension Icon via SF Symbols

Hi!
No intentions of removing someone else's work, however SF Symbols includes a radio waves icon that can be freely used and is more inline with macOS and Apple's design language. The new one as of version 6.1.1 is too thin.
SF Symbol reference is dot.radiowaves.up.forward
The icon is here (may not display depending on how your system & Github renders the font):
􀼗

Symbol not displayed here. How about adding a screenshot?


Type mistake?

https://github.com/mlocati/spf-lib/blob/ec49c332fcf684716e00f234013af5e01f001993/src/Check/State.php#L104
--> should be getEnvironment?

Yep, it should be

More: https://github.com/mlocati/spf-lib/search?q=getEnvoronment


Harry/2 move distro specific recipes into this repo

you have removed openssl altogether instead of the dunfell one only ?

Yes I've been having issues. It's pain. But I'm hoping is sorted now.


Fix stack aligment.

Stack should be 8 byte aligned on ARM.
Fix the automatic correction of the alignment in rt_init_stack,
and make sure that all stacks are aligned by the compiler.

@perkristian ,what do you think above Adam's comments above?

I agree with Adam's comments.
I have pushed a new commit which changes the relevant interfaces to be uint32 pointers. There is still the issue of the Thread interface, but I assume that cannot be changed since it is external.

What issue in the interface? That Thread ctor defines stack parameter as pointer to unsigned char?

Yes, any stack parameters pointing to unsigned char may be unaligned and we the crashes will be random depending on the build.

The similar check you proposed in the beginning, can be applied in the ctor there, to align the stack pointer if it's not NULL.

I like @perkristian current proposal.
@0xc0170 I wonder how many people would actually be impacted if the constructor was changed to take a uint32_t* instead.  I suspect most people just pass in NULL.  I would think that the compiler error would be pretty clear as to what they need to change if it does impact them.

Thanks guys. I'll merge this.


Updated ML Model and MongoDB Class

Description
Updates to the model were made to improve results of matching algorithm. Sorting now considers pair programming desire and industry knowledge, in addition to experience level and subject.
Several variable names were changed to have consistency with the Underdog Devs backend (mentee_id -> profile_id, user_id -> profile_id).
_connect for MongoDB class is more internal functionality. A get_collection method was added to replace the flexibility _connect had.
Data Generator scripts now create more complete Mentors and Mentee data. UUID format is used to generate profile_id.
Fixes # Trello issue
Explanation video
Type of change

[x] New feature

Checklist:

[x] My code follows PEP8 style guide
[x] I have removed unnecessary print statements from my code
[x] I have made corresponding changes to the documentation if necessary
[x] My changes generate no errors
[x] No commented-out code
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

Contributors: @BrokenShell @ulloa @ev-horrosh @LynnerKevin @RylandGomez @mktran0417 @Annapurnaj91 @archana062031 @rodricobsanchez @ZacharyRock @sirivennelavempati @MichaelJP-DS @ChristineWangcy @OlatomiAdigun

matching is ready for mvp. I was wondering if it'd make sense to output matches in a full document as opposed to just ids for the mentors?

It might be a good idea if that's what the backend team needs. However, not sure if they are just going to use the ids to query their own db instead.


Terrain is not restored when using restoremap.

if someone uses nuke i cannot restore the terrain and have to restart the server

Restore map isn't intended to be used anymore as it can break several server sided scripts better to wait for restore map to just place everyone to a temporary map and back into the game.

Anyone know if the terrain api stuff lets you get the current state of the terrain and re-apply it later if needed?

You can use range copy and range paste, have no idea.
On Thu, Oct 17, 2019, 3:48 PM Sceleratis<EMAIL_ADDRESS>wrote:

Anyone know if the terrain api stuff lets you get the current state of the
terrain and re-apply it later if needed?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Sceleratis/Adonis/issues/77?email_source=notifications&email_token=AA2OIGKB6WNO7WJYCOZQ26LQPDMSZA5CNFSM4HZKE7W2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBRUDKI#issuecomment-543375785,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2OIGLGWATZLFM6JLOO43TQPDMSZANCNFSM4HZKE7WQ
.

@Sceleratis Found this on the developer hub (when I was looking at the terrian API) that might help with this,
local terrainRegion = workspace.Terrain:CopyRegion(workspace.Terrain.MaxExtents)
workspace.Terrain:Clear()
wait(5)
workspace.Terrain:PasteRegion(terrainRegion, workspace.Terrain.MaxExtents.Min, true)

Not too sure if the wait is important or not.


How to build with RDMA in new version

Click to expand!
Issue Type
Build/Install
Source
source
Tensorflow Version
master
Custom Code
No
OS Platform and Distribution
ubuntu 18.04
Mobile device
No response
Python version
3.7
Bazel version
0.26.1
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Tensorflow supports gRPC+verbs and gRPC+MPI for distributed settings. I want to know how RDMA works in Tensorflow. So I build TensorFlow from the source.
But I can't find any documentation which shows how to do it. And I find a [repo](https://github.com/tensorflow/networking) which is relative to RDMA(verbs). I try to figure out how I can combine it with TensorFlow and the README confused me.
Then I review the git log in Tensorflow's repo and find the commit `7520b33` which supports verbs. In that commit, I can find how to configure the verbs support in `.bzaelc`. I try to find the same config in `master`'s `.bazelc` but I failed.
How can I build Tensorflow with verbs? Or may Tensorflow not support the verbs?

Standalone code to reproduce the issue
./configure

Relevant log output
No response

Sorry

Hi @hellofinch ,
Yes Tensorflow v2.0 supports --config==verbs but in latest versions it is not configured with --config==verbs.
Please refer similar issues 55528 & 55524 . Please go through the attached issues and also issue #55528 is still open we can expect a response from there. If you feel this issue is duplicate of #55528 please do close this or let us confirm so that we can close on behalf of you.

@SuryanarayanaY Yeah, I have seen those issues. But none of them can satisfy me, and I don't see anyone who can handle it. It's OK for me to close the issue.
Does that mean Tensorflow does not long support the verbs? Or are there some way else can do?
One last question, I see a repo containing verbs, MPI, and some thing else. Is that solution?

@hellofinch ,
You can try building tensorflow from source using Bazel with --config==verbs by referring the steps mentioned here.If you succeed in above step next you might be needed this API with protocol ="grpc+verbs" to turn on RDMA.
Please try your luck with this and let us know if it helps.
Thankyou!

@SuryanarayanaY the option --config=verbs maybe work with TensorFlow 2.0. As the issue 55524 says, this option is gone in the new version. In 2.0, this option is here. But as you can see, there is nothing about networking. Not only verbs but also mpi are missing too.
I want to know how Tensorflow handles different tensor transmission protocols. verbs or MPI works will sound good to me.
by the way, On the web page you mentioned, I see Tensorflow still supports verbs. Does that mean Tensorflow has contained the old package?

@hellofinch , You can find the bazel files in the directory here https://github.com/tensorflow/tensorflow/tree/master/tensorflow , it should be part of one of these files , workspace0.bzl workspace2.bzl, workspace3.bzl

@hellofinch ,
To add more I have tried to bazel build using option --config=verbs it throws out immediate error stating "Config value 'verbs' is not defined in any .rc file" and please refer to the gist1 for same.
Then i have added the line build:verbs --define=with_verbs_support=true which is there in 1.x .bazelrc file to the 2.x .bazelrc file and it starts compiling.Please refer attached gist2. As compilation in colab taking more time there i stopped it forcefully. Could you please try to add the line mentioned and lets know if it compiles and works for your case.
Thankyou!


y u no inheritance

Is there a particular reason this isn't inheriting from abstract-random-access, to gain arg checking / defaults etc.?

https://github.com/mafintosh/multi-random-access/commit/65b400a69977abf0a707c7353681147fd2623c5c