Wrapping composable functions in a class or object as extension (Improvement)

One of downsides of compose for me is that it pollutes intellisense so easily. For that I always wrap any theme properties such as colors, typography, shapes, other composable functions under wrapper objects and avoid making everything top level.
Example
@Immutable
class ThemeColors(val theme: Theme) {
 val utility0 = when(theme) {
        Theme.Light -> Color(0xFFFFFFFF)
        Theme.Dark -> Color(0xFF000000)
    }
  ...
}

I prefer to do the same for composable screens but compose-destinations expects screen names to be top level functions.
Wrap Login related screens under this object
object LoginNavHostScreens
Called wrapped screen from the object
@LoginNavGraph(start = true)
@Destination
@Composable
fun LoginNavHostScreens.SignUpOrLogInScreen() {
...
}

Wrapping screens as an extension of a class or object gives error since it is expected to be a top level function.
public object SignUpOrLogInScreenDestination : DirectionDestination {

    public operator fun invoke(): Direction = this

    @get:RestrictTo(RestrictTo.Scope.SUBCLASSES)
    override val baseRoute: String = "sign_up_or_log_in_screen"

    override val route: String = baseRoute

    @Composable
    override fun DestinationScope<Unit>.Content(
        dependenciesContainerBuilder: @Composable DependenciesContainerBuilder<Unit>.() -> Unit
    ) {
                 // This is expected to be a top level function, here gets the error
		**SignUpOrLogInScreen**(
			navigator = destinationsNavigator
		)
    }
}

Can there be an improvement for wrapping screen composables and be able to produce required code from object or class values? Thanks in advance.

Composables functions should be top-level functions at all times, there is even a lint check for this.
Using non-top-level composables is an antipattern in my humble opinion (except for rare cases like DSLs)
Also, theme properties are expected to be wrapped in an object, not an injectable/creatable class. Compose has nothing against properties of objects for non-composables like Color or Typography, but I'd avoid creating multiple instances of a hardcoded object.

I see two separate things here. I think @ilyasdirin's example shows just a global object is used as a receiver for the extension function that is the Composable Screen, but note that this Composable is still top-level, it just requires some object as the receiver.
If this is it, I don't think it's an antipattern, but I'm also not really prioritizing this 😛
What I can see myself doing is looking at receivers  just as any other parameter to the function, and if that's the case, you could provide these objects as dependencies in the dependencyContainerBuilder parameter of the DestinationsNavHost and then the lib would be able to call the Composables 🙂
Seems fair @ilyasdirin ?

Composable functions should be top-level functions at all times, there is even a lint check for this. Using non-top-level composables is an antipattern in my humble opinion (except for rare cases like DSLs)

Suppose I have a composable function hierarchy that creates a dynamic screen.  There's a bunch of state that is shared through all of these top level functions.  So am I supposed to pass 10-20 of the same args into every single function to carry the shared state down to where its needed?


Missing api-ms-win-core-console-l2-2-0.dll

Windows build number: Microsoft Windows [Version 10.0.16299.547]

The console (Debian GNU/Linux) worked perfectly fine. Then over night I had it open and the next morning it was closed without a notice (other programs were still running). Now when trying to open it I am getting the error
"The code execution cannot proceed because api-ms-win-core-console-l2-2-0.dll was not found. Reinstalling the program may fix this problem."

. I tried that, as well as repairing the Microsoft Visual C++ Redistributions (2012, 2013, 2015 & 2017) and running DISM.exe /Online /Cleanup-image /Restorehealth and sfc /scannow but nothing changed. I don't see the dll under C:\Windows\(System32 or SysWOW64).
The Windows update-log only shows one for Adobe Acrobat Reader on that date.
I guess it is rather a Windows issue than a console one but maybe someone can point me in a useful direction...

I just saw that there was a new update for the Debian GNU/Linux console and apparently installing that solved the issue...


[DM-38636] Deploy an InfluxDB Sink connectors for topics in the lsst.dm, lsst.debug and lsst.example namespaces for Sasquatch

Oh sorry, I must not have updated some tab, not it shows approved twice, so you are double approved


S26t12merge

I can't figure this out. @vrug will have to deal with it somehow. But I'm ready to help if I can.

The problem is here: https://github.com/Raku/roast/pull/661/commits/5349faafafe37c3b456e0050c2b7fa5a33c5b467, when you merged with git pull instead of git pull --rebase to put your commits on top of those. You need to clone a fresh repo, and then copy your files on top to avoid that.


Point Solana feeds at transmissionsAccount

If you have a moment to address this comment: https://github.com/smartcontractkit/documentation/pull/1118/files#r1080642944

Done


Committee representatives

@resbazaz/coffee-and-code
@resbazaz/festival2024
@val-pf
We are looking for representatives from each of ResBaz's standing committees to share their name and GitHub on our governance docs.
Add yourself if this is you.
@julianpistorius and @rbartelme have already volunteered themselves as representatives for @resbazaz/hacky-hour
Please find more info in #27

@val-pf assigned herself as the Coffee and code representative #35
We have representatives for the three events committees: CC, HaHo, and festival.


Selection from 'table' data

Hi all!
Is there a HOWTO for creation of selections on 'table' data? I failed with everything I tried so far.
Thank you very much,
. Jörg

It is currently impossible to create selections with anything other than categorical mappings. See: Issue #77 and #48.

Hello @joergplewe,
Try to use the following workaround:
https://github.com/Microsoft/PowerBI-visuals/issues/77#issuecomment-321750225

Hello @uve, I tried that but failed. It delivers good amounts of selectionIds which are all empty.

@joergplewe,
please send your source code to Power BI Custom Visuals Support<EMAIL_ADDRESS>we'll check it.

@ABKimmel @uve Is it true that highlighting also only works for categorical mappings? It would be great if this was documented.

@uve,
please disregard my objection, I finally could bring it to work.
Sample project is attached. Thank you very much!
tableSelection.zip

@joergplewe, great!


Title crashes immediately after the first round when setting up the FPS counter to higher than 60

Hi, My game crashes every single time after landing the final hit in the round, it occurs on every single FPS setting.
Current Tekken Overlay CL - TekkenOverlay-5.11.6-.2023-06-14
Current Tekken CL - Patch 5.10
Steps to repro:

Launch Tekken
Launch the Tekken Overlay in ~mods folder
Set the FPS counter to higher than 60
Play any match
Progress till the end of the round
Observe the issue

Every single setting in the Overlay is set to off except the FPS setting, enabling nametag features on/off does not change a thing sadly. I'm attaching my DxDiag config files in the ticket
DxDiag.txt
Thank you in advance!

Fixed in the latest release: https://github.com/TekkenOverlay/TekkenOverlay/releases/latest

holy crap, that was fast! thank you sooo much! <3


:bug: indy_credx error in validation of presentation when non_revoked.to is None

Good day
We're noticing a regression in behavior from 0.11 to 0.12, and seems to still be present in the latest nightly build.
In one of our proof request tests, we assert that a non-revoked (revocable) credential is valid. We do this with the verifier setting up a proof request like so:
    request_body = {
        "protocol_version": "v2",
        "comment": "Test cred is not revoked",
        "type": "indy",
        "indy_proof_request": {
            "non_revoked": {},  # {"to": time.time()},
            "requested_attributes": {
                "speed": {
                    "name": "speed",
                    "restrictions": [
                        {"cred_def_id": credential_definition_id_revocable}
                    ],
                }
            },
            "requested_predicates": {},
        },
        "save_exchange_record": True,
        "connection_id": acme_and_alice_connection.acme_connection_id,
    }

    send_proof_response = await send_proof_request(acme_client, request_body)

The above uses our own models, but indy_proof_request represents the ACA-Py IndyProofRequest model, which gets posted to /present-proof-2.0/create-request
Notice that non_revoked is an empty dict (which is different to None -- None would mean to not validate if the credential is revoked or not).
The expected behaviour, which we observe in 0.11, is that an empty non_revoked (not specifying from or to) would default to proving that the cred is not revoked at the time of presentation.
After a prover sends a presentation of a non-revoked credential (posting to /present-proof-2.0/records/{pres_ex_id}/send-presentation), in 0.11 we observe that verified returns as True in the verifier's presentation exchange record.
When attempting to upgrade to 0.12.1, we observe the following error in the agent, after prover sends presentation:
2024-06-05 13:20:44,349 aries_cloudagent.indy.credx.verifier ERROR Validation of presentation on nonce=960779015519508537837623 failed with error
Traceback (most recent call last):
  File "/home/aries/.local/lib/python3.9/site-packages/aries_cloudagent/indy/credx/verifier.py", line 72, in verify_presentation
    verified = await asyncio.get_event_loop().run_in_executor(
  File "/usr/local/lib/python3.9/asyncio/futures.py", line 284, in __await__
    yield self  # This tells Task to wait for completion.
  File "/usr/local/lib/python3.9/asyncio/tasks.py", line 328, in __wakeup
    future.result()
  File "/usr/local/lib/python3.9/asyncio/futures.py", line 201, in result
    raise self._exception
  File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/aries/.local/lib/python3.9/site-packages/indy_credx/types.py", line 467, in verify
    return bindings.verify_presentation(
  File "/home/aries/.local/lib/python3.9/site-packages/indy_credx/bindings.py", line 797, in verify_presentation
    do_call(
  File "/home/aries/.local/lib/python3.9/site-packages/indy_credx/bindings.py", line 465, in do_call
    raise get_current_error(True)
indy_credx.error.CredxError: Missing referent

This above error vanishes when I change non_revoked.to, in the verifier's initial proof request, to have some timestamp.
e.g. "non_revoked": {"to": time.time()}
Making this above change to the verifier's proof request resolves the error and verified is shown to be True again.
The reason for this bug is unclear to me. I can't see why the indy_credx rust code would say referent is missing when non_revoked.to is None, but it passes when it is set. So I'm just posting an issue in the meantime -- maybe someone has encountered the same behaviour. If it's new, then I'll investigate more in due time. Any feedback will be appreciated.

I don't think any of this code would have effected this but I recently merged something somewhat related. https://github.com/hyperledger/aries-cloudagent-python/pull/2995

@jamshale I tested with 0.12.1, which didn't include that change yet, and also tested the latest nightly build - both cases raise that error. But indeed works with 0.11

Sounds like it's a change in behaviour of aca-py, so we should probably fix it, but we should verify that we're following best practices:  https://github.com/hyperledger/aries-rfcs/blob/main/concepts/0441-present-proof-best-practices/README.md

A bit of a different angle that this made me think about is:
Should we permit implicit to == now() in the non_revoked object if it is an empty object? This seems like it could backfire if, for instance, a controller accidentally fumbles calculating a timestamp and it's request library turns {to: undefined} into {}. Should this be a validation error?
If we're okay with this implicit behavior then cool, I agree we should figure out what changed between 0.11.x and now and fix it if possible.

@dbluhm @ff137 Do we think this should be addressed for version 1.0.0? I don't want to keep adding more and more to do before we get the release done. Not exactly sure if this is important enough to tackle asap, or not?

I think it's acceptable for this to come in a later patch

I'm kind of think that Daniel is right about this, and even though has changed to require revoked.to to be set, that this is what the behaviour should actually be.
I'll leave this open for now. If we want to change it back to the old behaviour than I can tackle this.


more entities for viomi.v18

added sub entities for viomi.vacuum.v18

added translations to English

added miot-spec modifications

still incomplete due to #962 and #960

@blakadder  Please test the last commit of this PR.

this is as good as final as it gets with the current code...
i had to redo the property:fault becuase I was getting error codes not listed in miot spec for the device but I found an extended list under viomi.vacuum.v37 and used that instead
renamed siid 4 piid 13 to "Mop State" since it used a property name that was repeating and I couldn't get only that sensor


Convert dev/update_packages.py to a flutter subcommand

Do we still want this to be a flutter subcommand now that its written in dart?

It's certainly not as high a priority. Dunno how much we care about whether we have just one entry-point or not. This is only needed for people working on the flutter framework itself, right?

Correct.

I say let's not worry about it then.

In fact, we might want to just drop this entire script and just have the flutter script automatically call pub get when necessary. "flutter test" does that, and I'm finding myself writing that code again for "flutter analyze".


[PE-D][Tester D] Unable to find person by birthday

Unable to find person by birthday as instructed on User Guide. However, I was able to execute the find command with the name and birthday fields both present. But the result ignores the birthday field and returns the person even if the birthday does not match.

Labels: type.FeatureFlaw severity.Medium
original: asaierika/ped#3

Closed with #166


Added resource name

Have added the resource name so that the data-package becomes valid when submitted here

Thanks for merging this! The inclusion of the resource name enables the data-package to be linked into a Power Station Dictionary project I'm working on. I'll probably add titles (for each field) and a description for the package later this week if you're happy to include them as well.


custom media bug when nested

consider the following setup
// not nested
@media (--lg-viewport) {
    #Selector1 {
    // works as expected
  }
}

// nested
#Selector2 {
  @media (--lg-viewport) {
    // throws error (see below)
  }
}

// define query
@custom-media --lg-viewport screen and (min-width: 60em);

The error:
postcss-custom-media: Missing @custom-media definition for '--lg-viewport'. The entire rule has been removed from the output.

now if I do the following:
// define custom query name beforehand
@custom-media --lg-viewport

// nested
#Selector2 {
  @media (--lg-viewport) {
    // works
  }
}

// overwrite query
@custom-media --lg-viewport screen and (min-width: 60em);

...everything works out and resolves to @media screen and (min-width: 60em). No error whatsoever
Could this be an interoperability issue with postcss-nesting?

It's more an order issue. Be careful with the order of your transformation.
This is perfectly working in cssnext playground
I tested this
@media (--lg-viewport) {
  #Selector1 {
  }
}

#Selector2 {
  @media (--lg-viewport) {
  }
}

@custom-media --lg-viewport screen and (min-width: 60em);

got this
@media screen and (min-width: 60em) {
  #Selector1 {
  }
}

#Selector2 {
}

@media screen and (min-width: 60em) {
  #Selector2 {
  }
  }

I am just using:

postcss-import
postcss-cssnext

in that order.
could this possibly be an issue with the static site generator then? (using spike on this one.

I investigated further.
If I actually put everything in a single file it works for me too:
/*  single file works */
#Selector2 {
  @media (--lg-viewport) {
  }
}

@custom-media --lg-viewport screen and (min-width: 60em);

But I 've split things up in multiple directories. So its more like this
/* in css_modules/components-hero/lib/hero.css */
#Selector2 {
  @media (--lg-viewport) {
  }
}

/* in css_modules/theme-default/lib/theme.css */
@custom-media --lg-viewport screen and (min-width: 60em);

/* and finally in index.css */
@import 'components-hero';
@import 'theme-default';

Here is my tree:
├── css_modules
│   ├── components-hero
│   │   ├── index.css
│   │   └── lib
│   │       └── hero.css
│   └── theme-default
│       ├── index.css
│       └── lib
│           ├── default-map.css
│           └── default.css
└── index.css

Maybe postcss-import is the culprit?

Looks like yes.

Ok. Will Open the issue over there then.
Thanks


Vertica DB fails to Revive Successfully

The operator fails to revive an eon mode database from gcs. After verifying config was correct I repeatedly received the following log errors:
2022-10-19T19:02:43.627Z	INFO	controllers.VerticaDB	ExecInPod entry	{"verticadb": "data-infra-vertica/vertica-automessaging", "pod": "data-infra-vertica/vertica-automessaging-default-0", "command": "sudo su dbadmin -- /opt/vertica/bin/admintools -t revive_db --hosts=<ips> --communal-storage-location=<bucket> --communal-storage-params=/home/dbadmin/auth_parms.conf --database VerticaDB "}
2022-10-19T19:02:50.446Z	INFO	controllers.VerticaDB	ExecInPod stream	{"verticadb": "data-infra-vertica/vertica-automessaging", "pod": "data-infra-vertica/vertica-automessaging-default-0", "err": "command terminated with exit code 1", "stdout": "Attempting to retrieve file: [.../metadata/VerticaDB/cluster_config.json]\n\nValidated 3-node database VerticaDB defined at communal storage .\nCluster lease has expired.\n\nDatabase could not be revived.\nError: Database VerticaDB already exists.\nIf you want to force a new revive from communal storage on this cluster, please clean up the existing database first. You can drop the existing database by running: [admintools -t drop_db -d VerticaDB]\n", "stderr": ""}
2022-10-19T19:02:50.447Z	INFO	controllers.VerticaDB	Event logging	{"eventtype": "Warning", "reason": "ReviveDBFailed", "message": "Failed to revive the database"}
2022-10-19T19:02:50.447Z	INFO	controllers.VerticaDB	aborting reconcile of VerticaDB	{"verticadb": "data-infra-vertica/vertica-automessaging", "result": {"Requeue":false,"RequeueAfter":0}, "err": "could not execute: command terminated with exit code 1"}
2022-10-19T19:02:50.447Z	ERROR	Reconciler error	{"controller": "verticadb", "controllerGroup": "vertica.com", "controllerKind": "VerticaDB", "verticaDB": {"name":"vertica-automessaging","namespace":"data-infra-vertica"}, "namespace": "data-infra-vertica", "name": "vertica-automessaging", "reconcileID": "31f7f9b4-2bb2-4112-866c-746a778db4f7", "error": "could not execute: command terminated with exit code 1"}
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.2/pkg/internal/controller/controller.go:326
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.2/pkg/internal/controller/controller.go:273
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.2/pkg/internal/controller/controller.go:234

However, upon execing into the container and running the command manually the revive succeeds. Seems that the operator is getting out of sync somehow. Perhaps a race condition?
The output of the manual execution
Cluster lease has expired.
Preparation succeeded all hosts
Calculated necessary addresses for all nodes.
Starting to bootstrap nodes. Please wait, databases with a large catalog may take a while to initialize.
>>Calling bootstrap on node v_verticadb_node0002
>>Calling bootstrap on node v_verticadb_node0003
STATUS: vertica.engine.api.load_remote_catalog is still running on 3 hosts:<ips> as of 2022-10-19 19:07:05. See /opt/vertica/log/adminTools.log for full details.
STATUS: vertica.engine.api.load_remote_catalog is still running on 1 host: <ip> as of 2022-10-19 19:07:15. See /opt/vertica/log/adminTools.log for full details.
Load Remote Catalog succeeded on all hosts

Database revived successfully.

Follow up:
It appears the operator doesn't recognize that the revive has completed. It tries to revive again and gets stuck in an infinite loop. Manually calling admintools -t start_db -d VerticaDB resolves the problem. The nodes came up and health checks began passing.

Seems that even after the database is started, the operator doesn't detect it and continues trying to revive it. I am able to connect with vsql. Perhaps I'm missing something here.

The operator should detect that a database exists by looking to see if the <dataPath>/v_VerticaDB_node????_data directory exists. You should see evidence of this call in the operator log. Something like this on my system:
2022-10-20T13:08:14.163Z    INFO    controllers.VerticaDB   ExecInPod entry {"verticadb": "vertica/v", "pod": "vertica/v-defaultsubcluster-0", "command": "bash -c ls -d /data/vertdb/v_vertdb_node????_data "}
2022-10-20T13:08:14.251Z    INFO    controllers.VerticaDB   ExecInPod stream    {"verticadb": "vertica/v", "pod": "vertica/v-defaultsubcluster-0", "err": null, "stdout": "/data/vertdb/v_vertdb_node0001_data\n", "stderr": ""}

What does the output look like on your system? Also, if you are able to run vsql, what is the output of this sql?
select node_name, storage_path, storage_usage from disk_storage;

There is only one operator pod as expected.
For the failure to find the node data perhaps this is a misconfiguration on my end.
2022-10-20T17:20:52.081Z	INFO	controllers.VerticaDB	ExecInPod entry	{"verticadb": "data-infra-vertica/vertica-automessaging", "pod": "data-infra-vertica/vertica-automessaging-default-1", "command": "bash -c ls -d /vertica/VerticaDB/v_verticadb_node????_data "}
2022-10-20T17:20:52.473Z	INFO	controllers.VerticaDB	ExecInPod stream	{"verticadb": "data-infra-vertica/vertica-automessaging", "pod": "data-infra-vertica/vertica-automessaging-default-1", "err": "command terminated with exit code 2", "stdout": "", "stderr": "ls: cannot access '/vertica/VerticaDB/v_verticadb_node????_data': No such file or directory\n"}

where should that directory point? I currently have the local storage configured as such
    local:
      dataPath: /vertica
      depotPath: /vertica/depot

With that configuration the revive_db command results in the following file tree (at /vertica) the first time, but fails with the "already exists" error on subsequent times
│   └── VerticaDB
│       ├── dbLog
│       ├── port.dat
│       ├── procedures
│       └── v_verticadb_node0001_catalog
│           ├── bootstrap-catalog.log
│           ├── Catalog
│           │   ├── Checkpoints
│                     LOTS OF INTERNALS
│           ├── Snapshots
│           ├── spread.conf
│           ├── spread.pid
│           ├── startup.log
│           ├── tmp
│           ├── UDxLogs
│           │   └── UDxFencedProcesses.log
│           ├── vertica.conf
│           └── vertica.log
├── depot
└── temp
    └── VerticaDB
        └── temp

Related, but separate. It was a bit concerning that after the pods came up and were in a healthy state the operator ran commands that took the database offline. It seems like it may be relying on external state rather than querying current state of the pods.

Thanks for providing the full tree of your directory structure. I don't see a directory with the name v_verticadb_node0001_data. That is what the operator is looking for. If you are able to get the db up, can you run this sql? It should tell us where vertica thinks the data directory is:
select node_name, storage_path, storage_usage from disk_storage;

VerticaDB=> select node_name, storage_path, storage_usage from disk_storage;
WARNING 2817:  Could not call statvfs on /vertica/depot/VerticaDB/v_verticadb_node0001_data, No such file or directory
WARNING 2817:  Could not call statvfs on /vertica/depot/VerticaDB/v_verticadb_node0001_depot, No such file or directory
WARNING 2817:  Could not call statvfs on /vertica/depot/VerticaDB/v_verticadb_node0002_data, No such file or directory
WARNING 2817:  Could not call statvfs on /vertica/depot/VerticaDB/v_verticadb_node0002_depot, No such file or directory
      node_name       |                          storage_path                           | storage_usage
----------------------+-----------------------------------------------------------------+---------------
 v_verticadb_node0001 | /vertica/catalog/VerticaDB/v_verticadb_node0001_catalog/Catalog | CATALOG
 v_verticadb_node0001 | /vertica/temp/VerticaDB/temp                                    | TEMP
 v_verticadb_node0002 | /vertica/catalog/VerticaDB/v_verticadb_node0002_catalog/Catalog | CATALOG
 v_verticadb_node0002 | /vertica/temp/VerticaDB/temp                                    | TEMP
 v_verticadb_node0003 | /vertica/catalog/VerticaDB/v_verticadb_node0003_catalog/Catalog | CATALOG
 v_verticadb_node0003 | /vertica/depot/VerticaDB/v_verticadb_node0003_data              | DATA,TEMP
 v_verticadb_node0003 | /vertica/depot/VerticaDB/v_verticadb_node0003_depot             | DEPOT
 v_verticadb_node0003 | /vertica/temp/VerticaDB/temp                                    | TEMP

Okay, I think I understand what is happening here.
The first revive by the operator probably worked. But since it wasn't able to detect that the database exist it repeated the revive over and over. Each time it did this it got that 'Error: Database VerticaDB already exists.' error.
The operator wasn't able to find the data directory, which is why it wasn't able to detect the existence of the database. From your disk_storage output, the dataPath in the CR should be set to /vertica/depot.  Instead the operator is looking in /vertica/VerticaDB/.
Your CR spec should have this:
spec:
  local:
    dataPath: /vertica/depot
    depotPath: /vertica/depot

However, there is another issue with your setup. The catalog path and the data are in separate paths. But, the CR doesn't allow you to split them. We assume the catalog is under the dataPath. So, even if you make the above change it will complain about permissions to the catalog path.
The fix here is to be able to specify the catalog path in the CR and have a PV mounted directory backing it. How critical is this fix for you?
I will also look at improving the diagnostic messages related to paths. That way, in the future, if someone else hits this it will be more obvious what the error is.
Thanks for your patience with this one.

Since the operator fails to detect the database is up and knocks it offline this would be completely blocking for us. We can go back to a VM deployment for now. The rest of our infra is on k8s though so we'd like to move this over as soon as possible.

Thanks for the context. I'll make this a priority.


Precedence of specific dialect migrations over generic ones

Description
I have a case where a generic migration is executed instead of a specific migration.
Steps to Reproduce the Problem
I have a folder with migrations (all sql). There are some for all databases and some with a dialect suffix. Especially I have for some versions both a specific and generic one. The docs state that the specific ones have higher precedence. For some reason this works for all the migrations except the first one. After having a look into the code and also running it with a debugger I could not figure out how the migrator ensures the specific migration get's applied. The sort interface functions only compare the version number (https://github.com/gobuffalo/pop/blob/development/migration_info.go#L39:L41).
Expected Behavior
In my opinion the migrations should be sorted with respect to the DBType as well to ensure the right order.
Actual Behavior
It seems that it is possible to have a generic migration applied instead of the specific one.
Info
You can have a look at my development branch with the failing test here https://github.com/ory/hydra/pull/1775/files#diff-5f192be0f9edf1da6e39ecf5c7344ba5
It fails because it applies<PHONE_NUMBER>0001_client.up gets applied instead of<PHONE_NUMBER>0001_client.cockroach.up

closed via #533


Unix/Epoch timestamp does not parse correctly when using allowInput

When using the allowInput flag, entering a unix timestamp into a field and pressing enter does not parse the entered date correctly. I am using unix date format and an alt format for displaying the date.
Here is my fiddle with the problem:
https://jsfiddle.net/s8akwx04/
I am using the following options:
{
    allowInput: true,
    dateFormat: 'U',
    altFormat: 'm/d/y - H:i',
    altInput: true,
}

Is this not how it is supposed to work or is something wrong?
Your Environment

flatpickr version used: 4.5.2
Browser name and version: Chrome 70.0.3538.102
OS and version: Mac OS 10.12.6

@Pearce-Ropion when you're using altInput, the manually input date is expected to follow altFormat, so that the display and input format are the same.

Oh that makes sense. Thank you

@Pearce-Ropton still, you can always use  https://flatpickr.js.org/instance-methods-properties-elements/#setdate-date-triggerchange-datestrformat to update the date via timestamp


As Staff I can add a 'due date' on a pet task

Staff need to be able to define a due date for a pet task. Not all tasks require a due date.
Acceptance criteria

Tasks db table has new column called due_date type datetime
new/edit task form has a date picker where due date can be selected
index table has Due Date column showing due date value
overdue tasks should be easy to identify as overdue if the task is not completed (I suggest using the 'expired' marker from this page of the Geeks theme and placing it next to the date value in the Index table saying 'Overdue'). Maybe a Task model scope to check if a task is overdue or not is a suitable approach.
tests

Note, I suggest a PR for this issue be made after this PR to modify how we write migrations is approved and merged: https://github.com/rubyforgood/pet-rescue/pull/400

I can tackle this one!

Hey @edwinthinks I am unassigning you so we can get someone else to finish this off. Thanks for doing most of it :)

@nsiwnf this issue is still alive! Pr just needs finishing.


Update MailSniper.ps1

Updated Get_GlobalAddressList to use more granular queries in ResolveName, if required, to avoid incomplete results due to 100 result limit.

Please re-test with the latest PRs that were just added.

@RythmStick Any updates to @leebaird question?


Telemetry for Linux

Opening this Issue to start a high-level discussion about what the telemetry for linux needs to be. Some initial questions:

Protocol/data structure for the format
Minimum data set from the telemetry beacon
Cache or buffer data for later query or transmit?
??

This is a conversation starter. Feel free to add/delete/correct.
/cc @kubostech/core @kubostech/devs @ISISGerard

closing this for now. TBC.


Inability to resolve output file path on Github Actions

Seeing this really weird error where nf-test seems to resolve the path to an output file locally but not on Github Actions.
Let's take this test and this snapshot as an example.
The outdir is set to results intentionally.
✅ Local test
$ git clone https://github.com/drpatelh/nf-core-fetchngs.git
$ cd nf-core-fetchngs
$ git checkout 18a7c27

$ nf-test test --verbose --tag sra_skip_fastq_download --profile test,docker

🚀 nf-test 0.8.3
https://code.askimed.com/nf-test
(c) 2021 - 2023 Lukas Forer and Sebastian Schoenherr

Found 1 files in test directory.

Test workflow: sra/main.nf

  Test [6eb361d0] 'Parameters: --skip_fastq_download'
    > N E X T F L O W  ~  version 23.10.0
    > Launching `/home/harshil/repos/nf-core/fetchngs/nf-test/tests/6eb361d01c904cfd84075fcdee88a/meta/mock.nf` [agitated_blackwell] DSL2 - revision: 03d5b2560c
    > [0f/7dc0f4] Submitted process > SRA:SRA_IDS_TO_RUNINFO (ERR1160846)
    > [15/d37803] Submitted process > SRA:SRA_IDS_TO_RUNINFO (GSE214215)
    > [d8/788841] Submitted process > SRA:SRA_IDS_TO_RUNINFO (SRR12848126)
    > [2a/752fcf] Submitted process > SRA:SRA_RUNINFO_TO_FTP (1)
    > [71/5fb1ad] Submitted process > SRA:SRA_RUNINFO_TO_FTP (2)
    > [6c/2378b0] Submitted process > SRA:SRA_TO_SAMPLESHEET (ERX1234253_ERR1160846)
    > [50/fe29e4] Submitted process > SRA:SRA_TO_SAMPLESHEET (SRX9315476_SRR12848126)
    > [24/78fbc8] Submitted process > SRA:SRA_RUNINFO_TO_FTP (3)
    > [b9/5edd53] Submitted process > SRA:SRA_TO_SAMPLESHEET (SRX17709228_SRR21711855)
    > [62/b0219a] Submitted process > SRA:SRA_TO_SAMPLESHEET (SRX17709227_SRR21711856)
    > [f3/b6df3a] Submitted process > SRA:MULTIQC_MAPPINGS_CONFIG (1)
    PASSED (16.736s)

SUCCESS: Executed 1 tests in 16.738s

:x: Gitpod environment
If I open a Gitpod environment on this branch and run the same command it fails because it is unable to find the output files samplesheet.csv and id_mappings.csv. The only thing I can think that is somehow causing this behaviour is that I am publishing both of these files via storeDir as you can see here and here. In the diff you can see that the path to the file's are reported as results/samplesheet/samplesheet.csv and results/samplesheet/id_mappings.csv rather than just samplesheet.csv and id_mappings.csv and there is no md5sum reported so I assume the files aren't found?
$ nf-test update
Updating application...
Downloading nf-test 0.8.3 from https://github.com/askimed/nf-test/releases/download/v0.8.3/nf-test-0.8.3.tar.gz...

Done.

$ git checkout 18a7c27
$ nf-test test --verbose sra_skip_fastq_download --profile test,docker

🚀 nf-test 0.8.3
https://code.askimed.com/nf-test
(c) 2021 - 2023 Lukas Forer and Sebastian Schoenherr

Test workflow: sra/main.nf

  Test [ab5d0866] 'Parameters: --skip_fastq_download'
    > N E X T F L O W  ~  version 23.10.0
    > Launching `/workspace/nf-core-fetchngs/nf-test/tests/ab5d086625608763d890b1465d411176/meta/mock.nf` [infallible_euclid] DSL2 - revision: 5a7effb3c0
    > [b4/e5004f] Submitted process > SRA:SRA_IDS_TO_RUNINFO (GSE214215)
    > [51/b1d4ca] Submitted process > SRA:SRA_IDS_TO_RUNINFO (SRR12848126)
    > [a0/cdc769] Submitted process > SRA:SRA_IDS_TO_RUNINFO (ERR1160846)
    > [8d/e97375] Submitted process > SRA:SRA_RUNINFO_TO_FTP (1)
    > [0a/c33a92] Submitted process > SRA:SRA_RUNINFO_TO_FTP (2)
    > [55/69ac8d] Submitted process > SRA:SRA_TO_SAMPLESHEET (ERX1234253_ERR1160846)
    > [b2/069670] Submitted process > SRA:SRA_TO_SAMPLESHEET (SRX9315476_SRR12848126)
    > [8c/22a35a] Submitted process > SRA:SRA_RUNINFO_TO_FTP (3)
    > [e8/733810] Submitted process > SRA:SRA_TO_SAMPLESHEET (SRX17709227_SRR21711856)
    > [24/a98124] Submitted process > SRA:SRA_TO_SAMPLESHEET (SRX17709228_SRR21711855)
    > [fa/30978a] Submitted process > SRA:MULTIQC_MAPPINGS_CONFIG (1)
    java.lang.RuntimeException: Different Snapshot:
[                                                                                [
    {                                                                                {
        "0": [                                                                                "0": [

        ],                                                                                ],
        "1": [                                                                                "1": [
            "samplesheet.csv:md5,81856795ac19f44aeb72ae5b3042e609"                                 |                "results/samplesheet/samplesheet.csv"
        ],                                                                                ],
        "2": [                                                                                "2": [
            "id_mappings.csv:md5,ca34293cf91a004ea52627dd8b58a43a"                                 |                "results/samplesheet/id_mappings.csv"
        ],                                                                                ],
        "3": [                                                                                "3": [
            [                                                                                [
                "multiqc_config.yml:md5,6d4c3e5137704358474330207f5f2b5c",                                              "multiqc_config.yml:md5,6d4c3e5137704358474330207f5f2b5c",
                "versions.yml:md5,1496d1cbc9041e07ab8a0c25f0b054d9"                                                     "versions.yml:md5,1496d1cbc9041e07ab8a0c25f0b054d9"
            ]                                                                                ]
        ],                                                                                ],
        "4": [                                                                                "4": [
            "versions.yml:md5,1496d1cbc9041e07ab8a0c25f0b054d9",                                                    "versions.yml:md5,1496d1cbc9041e07ab8a0c25f0b054d9",
            "versions.yml:md5,9b17045ca8bdc272cb3f9d349a81d206",                                                    "versions.yml:md5,9b17045ca8bdc272cb3f9d349a81d206",
            "versions.yml:md5,b52279f7d6b891a6523d9321f3f85b47"                                                     "versions.yml:md5,b52279f7d6b891a6523d9321f3f85b47"
        ],                                                                                ],
        "fastq": [                                                                                "fastq": [

        ],                                                                                ],
        "mappings": [                                                                                "mappings": [
            "id_mappings.csv:md5,ca34293cf91a004ea52627dd8b58a43a"                                 |                "results/samplesheet/id_mappings.csv"
        ],                                                                                ],
        "sample_mappings": [                                                                                "sample_mappings": [
            [                                                                                [
                "multiqc_config.yml:md5,6d4c3e5137704358474330207f5f2b5c",                                              "multiqc_config.yml:md5,6d4c3e5137704358474330207f5f2b5c",
                "versions.yml:md5,1496d1cbc9041e07ab8a0c25f0b054d9"                                                     "versions.yml:md5,1496d1cbc9041e07ab8a0c25f0b054d9"
            ]                                                                                ]
        ],                                                                                ],
        "samplesheet": [                                                                                "samplesheet": [
            "samplesheet.csv:md5,81856795ac19f44aeb72ae5b3042e609"                                 |                "results/samplesheet/samplesheet.csv"
        ],                                                                                ],
        "versions": [                                                                                "versions": [
            "versions.yml:md5,1496d1cbc9041e07ab8a0c25f0b054d9",                                                    "versions.yml:md5,1496d1cbc9041e07ab8a0c25f0b054d9",
            "versions.yml:md5,9b17045ca8bdc272cb3f9d349a81d206",                                                    "versions.yml:md5,9b17045ca8bdc272cb3f9d349a81d206",
            "versions.yml:md5,b52279f7d6b891a6523d9321f3f85b47"                                                     "versions.yml:md5,b52279f7d6b891a6523d9321f3f85b47"
        ]                                                                                ]
    }                                                                                }
]                                                                                ]

FAILED (26.712s)

  Assertion failed:

  1 of 2 assertions failed

  Nextflow stdout:

  N E X T F L O W  ~  version 23.10.0
  Launching `/workspace/nf-core-fetchngs/nf-test/tests/ab5d086625608763d890b1465d411176/meta/mock.nf` [infallible_euclid] DSL2 - revision: 5a7effb3c0
  [b4/e5004f] Submitted process > SRA:SRA_IDS_TO_RUNINFO (GSE214215)
  [51/b1d4ca] Submitted process > SRA:SRA_IDS_TO_RUNINFO (SRR12848126)
  [a0/cdc769] Submitted process > SRA:SRA_IDS_TO_RUNINFO (ERR1160846)
  [8d/e97375] Submitted process > SRA:SRA_RUNINFO_TO_FTP (1)
  [0a/c33a92] Submitted process > SRA:SRA_RUNINFO_TO_FTP (2)
  [55/69ac8d] Submitted process > SRA:SRA_TO_SAMPLESHEET (ERX1234253_ERR1160846)
  [b2/069670] Submitted process > SRA:SRA_TO_SAMPLESHEET (SRX9315476_SRR12848126)
  [8c/22a35a] Submitted process > SRA:SRA_RUNINFO_TO_FTP (3)
  [e8/733810] Submitted process > SRA:SRA_TO_SAMPLESHEET (SRX17709227_SRR21711856)
  [24/a98124] Submitted process > SRA:SRA_TO_SAMPLESHEET (SRX17709228_SRR21711855)
  [fa/30978a] Submitted process > SRA:MULTIQC_MAPPINGS_CONFIG (1)
  Nextflow stderr:

    Obsolete snapshots can only be checked if all tests of a file are executed successful.

FAILURE: Executed 1 tests in 26.718s (1 failed)

Thanks for this detailed report 🙏 This helps a lot!
I could reproduce the issue on my local machine as well. The current implementation of the snapshot is unable to collect files that have a relative path to the $launchDir. I will work on a fix.
However, to be honest, I could not understand why it works on your local machine and how and why the MD5 sums were created. Did you start the test in the past with a different outdir that was not relative?

Thanks for looking at this 🙏🏽

However, to be honest, I could not understand why it works on your local machine and how and why the MD5 sums were created. Did you start the test in the past with a different outdir that was not relative?

Yep, I was puzzled by this too. I used a relative path in both instances i.e. outdir = "results". Locally this worked but didn't on Github Actions. Happy to test it out and let you know. I want to update the pipeline tests in nf-core/fetchngs to use this as soon as it is released anyway.


Added text for Changelog

This is a quick list of all the changes made to the Standard since July 31, 2019.  Per @iramcdonald, I made this a bulleted list. I also combined items where it seemed relevant.

Hi,
Excellent work Lois, et al.  Just the right level of detail (useful but not
so low-level
that the context is lost).
Thanks to everyone for a good job on this important bit of update Deployment
Considerations.  Which is going to be named Deployment Best Practices, I
hope?
Cheers,

Ira

Ira McDonald (Musician / Software Architect)
Chair - SAE Trust Anchors and Authentication TF
Co-Chair - TCG Trusted Mobility Solutions WG
Co-Chair - TCG Metadata Access Protocol SG
Chair - Linux Foundation Open Printing WGSecretary - IEEE-ISTO Printer
Working GroupCo-Chair - IEEE-ISTO PWG Internet Printing Protocol WGIETF
Designated Expert - IPP & Printer MIBBlue Roof Music / High North
Inchttp://sites.google.com/site/blueroofmusic
http://sites.google.com/site/blueroofmusichttp://sites.google.com/site/highnorthinc
http://sites.google.com/site/highnorthincmailto<EMAIL_ADDRESS>blueroofmusic@gmail.com(permanent) PO Box 221  Grand Marais, MI 49839
906-494-2434
On Fri, Oct 23, 2020 at 1:42 PM Marina Moore<EMAIL_ADDRESS>wrote:

@mnm678 approved this pull request.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uptane/deployment-considerations/pull/57#pullrequestreview-515877055,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE33UO5LWUIRQHNDMH2KV23SMG6BXANCNFSM4S32MI2A
.

Thanks, Ira. Unless someone tells me different, the title is "Uptane Deployment Best Practices."

Unless someone tells me different, the title is "Uptane Deployment Best Practices."

No objection, but is it worth renaming the repo? Or just changing the title? I think the Standard might mention this doc as well.
What's the benefit of "best practices" versus "considerations"? Maybe this is another point for the FAQ, i.e.: what is this doc for?

We can talk about the name tomorrow. @iramcdonald has mentioned several times that the title is more in line with similar industry documents. For my part, 'Considerations" seems like a somewhat vague term, whereas "Best Practices" to me clearly states that these are recommended practices for implementation.

Hi,
Supporting Lois's comments on the document title:  IETF, ITU-T, and other
SDOs
have a document class of "Best Practices".  And, as Lois said, this title
clearly states
that the topic is recommendations for implementation and use of a standard.
In fact, in IETF a "Best Current Practices (BCP)" document takes precedence
over
even a standards-track document, since the BCP reflects actual experience
for
implementation and deployment.
Cheers,

Ira

Ira McDonald (Musician / Software Architect)
Chair - SAE Trust Anchors and Authentication TF
Co-Chair - TCG Trusted Mobility Solutions WG
Co-Chair - TCG Metadata Access Protocol SG
Chair - Linux Foundation Open Printing WGSecretary - IEEE-ISTO Printer
Working GroupCo-Chair - IEEE-ISTO PWG Internet Printing Protocol WGIETF
Designated Expert - IPP & Printer MIBBlue Roof Music / High North
Inchttp://sites.google.com/site/blueroofmusic
http://sites.google.com/site/blueroofmusichttp://sites.google.com/site/highnorthinc
http://sites.google.com/site/highnorthincmailto<EMAIL_ADDRESS>blueroofmusic@gmail.com(permanent) PO Box 221  Grand Marais, MI 49839
906-494-2434
On Mon, Oct 26, 2020 at 7:10 AM Lois Anne DeLong<EMAIL_ADDRESS>wrote:

We can talk about the name tomorrow. @iramcdonald
https://github.com/iramcdonald has mentioned several times that the
title is more in line with similar industry documents. For my part,
'Considerations" seems like a somewhat vague term, whereas "Best Practices"
to me clearly states that these are recommended practices for
implementation.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uptane/deployment-considerations/pull/57#issuecomment-716479107,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE33UO5JERL3JSG6GU76J7TSMVKLZANCNFSM4S32MI2A
.

Maybe this is another point for the FAQ, i.e.: what is this doc for?

After reviewing what we already have, I don't think this is a good idea anymore. The intro/index page already covers this, and that's a better place to do it.

Hi,
I agree with Patrick.
FAQs should be fresh, informal, and web-based, rather than static documents.
Cheers,

Ira

Ira McDonald (Musician / Software Architect)
Chair - SAE Trust Anchors and Authentication TF
Co-Chair - TCG Trusted Mobility Solutions WG
Co-Chair - TCG Metadata Access Protocol SG
Chair - Linux Foundation Open Printing WGSecretary - IEEE-ISTO Printer
Working GroupCo-Chair - IEEE-ISTO PWG Internet Printing Protocol WGIETF
Designated Expert - IPP & Printer MIBBlue Roof Music / High North
Inchttp://sites.google.com/site/blueroofmusic
http://sites.google.com/site/blueroofmusichttp://sites.google.com/site/highnorthinc
http://sites.google.com/site/highnorthincmailto<EMAIL_ADDRESS>blueroofmusic@gmail.com(permanent) PO Box 221  Grand Marais, MI 49839
906-494-2434
On Tue, Oct 27, 2020 at 11:33 AM Patrick Vacek<EMAIL_ADDRESS>wrote:

Maybe this is another point for the FAQ, i.e.: what is this doc for?
After reviewing what we already have, I don't think this is a good idea
anymore. The intro/index page
https://uptane.github.io/deployment-considerations/index.html already
covers this, and that's a better place to do it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uptane/deployment-considerations/pull/57#issuecomment-717326588,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE33UO5DJPAXVMWAPIKTQMLSM3R3VANCNFSM4S32MI2A
.


Adding support for one more String function: strtok()

It would be nice to have strtok() functionality to parse strings and/or compare strings from my preloaded table with the kernel. The limitation right now makes it almost impossible for certain tasks to be executed.
Handwriting the solution does not seem to work since we end up with unknown opcode errors regardless of any variation

Could you share some more concrete examples of what tasks you're trying to accomplish?

I am testing some hacked together drives that I made with an ARM processor. Without going into too much detail, I would like to parse some input from the user that comes in either a CSV format, words separated by a space, or something with a delimiter so I can do my own computation on it. That is my goal as of now.

String processing in a BPF prog is tricky. It might require new kernel helpers. It'll definitely require @Birch-san 's map-backed strings. Without map-backed strings you can process at most ~150 characters. And then we'll have to bound the loop too.
This is one of those features that sounds simple but is actually quite tricky due to how restrictive BPF prog execution is.

Yes, I have been trying to try and see if I could come up with something using restrictive C but it is literally impossible with my skillset. I have been passing the strings to userspace when parsing but I do lose some of the performance benefit that I could have seen when running within BPF. I wonder if anyone could divulge more information about some kernel helpers. Maybe I can help write? Never wrote any kernel functions before but I could give it a shot.

Here's the strtol() helper:
https://github.com/torvalds/linux/blob/007b350a58754a93ca9fe50c498cc27780171153/include/uapi/linux/bpf.h#L3587
https://github.com/torvalds/linux/blob/df04fbe8680bfe07f3d7487eccff9f768bb02533/kernel/bpf/helpers.c#L500-L513

Hi! Thank you for the link but I am not sure how this would be useful for what I am trying to accomplish?

BPF helpers are implemented in the kernel. It's unlikely we'll be able to implement strtok() without help from the kernel.

So, strsep() is implemented within the kernel. Do I


fix: Make GetVersioned() available after doing LazyLoadVersion

We can use LazyLoadVersion by https://github.com/cosmos/iavl/pull/148.
But if we start app with LazyLoadVersion we cannot query app state by specific height because MutableTree.GetVersioned returns nil for other previous version after doing LazyLoadVersion.
Our nodes are facing a slow booting up problem because of the large app data, and I want to hear your opinion about whether we should modify the code like this.

@robert-zaremba I modified the checking code as you advised. I'd appreciate it if you review it again.

Hi @egonspace  - are you planning to merge this PR?

I accidentally erased the branch and recreated the PR. I'd appreciate it if you could review it again even if it's too much trouble.


Bug(sort-imports): inline comment misplace after fix

Describe the bug
If there are comments after import, sort-import will cause the comments to be misplaced
Code example
configuration
rules: {
  'perfectionist/sort-imports': ['error', {
    order: 'asc',
    // all three types are have same bug
    type: 'natural',
  }],
},

code
import A from './a' // comment A
import C from './c' // comment C
import B from './b' // comment B

after fix
import A from './a' // comment A
import B from './b' // comment C
import C from './c' // comment B

Expected
import A from './a' // comment A
import B from './b' // comment B
import C from './c' // comment C

ESLint version
9.9.0
ESLint Plugin Perfectionist version
3.8.0
Additional comments
I read code and found that the sort-imports fix only modified the position of the import statement. This may be an expected behavior to prevent multiple import statements from being on the same line?
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

It seems that the recent PR https://github.com/azat-io/eslint-plugin-perfectionist/pull/312 has fixed this issue.

Yes, I'll create release today at evening

Fixed in 398ac13.
Releases in v3.9.0.


css: Use SCSS nesting in hotspots.scss.

Testing Plan:
Mostly by diffing the styles before and after the change and manual (visual) testing.

Looks great, merged, thanks @vrongmeal!


sharee (editor role) MOVE a file by file-id into same shared folder returns 403

Describe the bug
When sharee makes API request to move a file from a sub-folder to folder in Shares space , then we get the status code 403 but the share should be able to MOVE the resouce within the shared folder
Steps to reproduce
Steps to reproduce the behavior:

As marie, create a folder folder
As marie creates a sub-folder sub-folder inside folder folder
As marie upload the files file.txt inside sub-folder
As marie, share a folder folder to einstein with permission all
As einstein, move file.txt from folder sub-folder to folder using curl commands:

curl -kv -XMOVE -ueinstein:relativity 'https://localhost:9200/remote.php/dav/spaces/<file-id>' \
-H 'Destination: https://localhost:9200/remote.php/dav/spaces/<folder-mount-id>/<filename>' | xmllint --format

Expected behavior
The status code 201
Actual behavior
< HTTP/1.1 403 Forbidden
<
<?xml version="1.0" encoding="UTF-8"?>
<d:error
	xmlns:d="DAV"
	xmlns:s="http://sabredav.org/ns">
	<s:exception>Sabre\DAV\Exception\Forbidden</s:exception>
	<s:message>cross storage moves are not permitted, use copy and delete</s:message>
</d:error>

CC @ScharfViktor @micbar

Now, it returns 502
curl -XMOVE "https://localhost:9200/dav/spaces/623ed2a4-806e-4bf3-b9be-0fdfec40229a\$b549651d-e1f8-4c30-8351-8396fb18e7b0\!e31cb741-ed30-491e-8e04-541673fb4d62" \
-H"Destination: https://localhost:9200/dav/spaces/a0ca6a90-a365-4782-871e-d44447bbc668\$a0ca6a90-a365-4782-871e-d44447bbc668/myfolder/child/afile.txt" \
-udemo:demo -vk

< HTTP/1.1 502 Bad Gateway
...
<?xml version="1.0" encoding="UTF-8"?>
<d:error xmlns:d="DAV" xmlns:s="http://sabredav.org/ns">
	<s:exception></s:exception>
	<s:message>cross storage moves are not supported, use copy and delete</s:message>
	<s:errorcode></s:errorcode>
</d:error>


Add evaluateTransactionExecutionUnitsWithLogs

As we have switched to cardano-ledger for validation in plutus-apps, we also want the logs to get the coverage data. This PR adds evaluateTransactionExecutionUnitsWithLogs — a more general version of evaluateTransactionExecutionUnits with logs.

Oh, yeah. You also need to run scripts/ormolise.sh in order to make the CI happy

Is it possible to backport it to release/1.0.x branch?

What is your goal, to get this in an earlier release?

At the moment we use c7c63dabdb215ebdaed8b63274965966f2bf408f in our cabal.project and I would like to start using this function asap. Usually we track the dependencies of cardano-wallet.

I feel like it would be confusing to extend release/1.0.x with a commit that we don't think will ever be a part of a release. The release process is all being moving to CHaPs at the moment, so things are a bit chaotic, sorry. In the short term, is there much difference for you between back-porting to release/1.0.x and just using a different branch? I'm open to suggestions? @lehins - do you have an opinion here?


AWQ quantization does not quantize model weights during inference

I am trying to use the AWQ quantization to load a quantized model. However, during inference time, the model weight is still mapped to float16 and int32. In ./vllm/engine/arg_utils.py, we cannot set the dtype to int4. And in awq.py, get_supported_act_dtypes() only supports torch.half.
Any plan to fix this issue?

Right now there aren't INT4 cuda kernels, and not every Nvidia GPU support INT4 computing.

Hi @DAIZHENWEI, AWQ (as well as GPTQ) is a weight-only quantization method, which dequantizes the weights to FP16/BF16 on the fly during matmul. You need to use a different quantization method to actually compute the operation in INT4. Unfortunately, none of the quantization method in vLLM supports this. And to the best of my knowledge, quantizing the activation beyond 8 bits significantly hurts the output quality for now.


Transpiler fails at optimization level 1 for certain coupling maps

Information

Qiskit Terra version: 0.8.2 and master branch
Python version: 3.6.3
Operating system: Ubuntu shell in Windows 10

What is the current behavior?
For a given circuit (in this case created with quantum volume test case), at the transpiler will crash at optimization level 1 for certain coupling maps.
    circuits = parallel_map(_transpile_circuit, list(zip(circuits, transpile_configs)))
  File "/home/awack/anaconda3/lib/python3.6/site-packages/qiskit/tools/parallel.py", line 100, in parallel_map
    return [task(values[0], *task_args, **task_kwargs)]
  File "/home/awack/anaconda3/lib/python3.6/site-packages/qiskit/compiler/transpile.py", line 180, in _transpile_circuit
    return transpile_circuit(circuit, transpile_config)
  File "/home/awack/anaconda3/lib/python3.6/site-packages/qiskit/transpiler/transpile_circuit.py", line 62, in transpile_circuit
    return pass_manager.run(circuit)
  File "/home/awack/anaconda3/lib/python3.6/site-packages/qiskit/transpiler/passmanager.py", line 138, in run
    dag = self._do_pass(pass_, dag, passset.options)
  File "/home/awack/anaconda3/lib/python3.6/site-packages/qiskit/transpiler/passmanager.py", line 168, in _do_pass
    dag = self._run_this_pass(pass_, dag)
  File "/home/awack/anaconda3/lib/python3.6/site-packages/qiskit/transpiler/passmanager.py", line 179, in _run_this_pass
    new_dag = pass_.run(dag)
  File "/home/awack/anaconda3/lib/python3.6/site-packages/qiskit/transpiler/passes/mapping/legacy_swap.py", line 176, in run
    serial_layerlist),
  File "/home/awack/anaconda3/lib/python3.6/site-packages/qiskit/transpiler/passes/mapping/legacy_swap.py", line 388, in swap_mapper_layer_update
    dagcircuit_output.compose_back(layer_list[j]["graph"], layout)
  File "/home/awack/anaconda3/lib/python3.6/site-packages/qiskit/dagcircuit/dagcircuit.py", line 466, in compose_back
    self.cregs)
  File "/home/awack/anaconda3/lib/python3.6/site-packages/qiskit/dagcircuit/dagcircuit.py", line 359, in _check_edgemap_registers
    raise DAGCircuitError("unmapped duplicate reg %s" % k)
qiskit.dagcircuit.exceptions.DAGCircuitError: "unmapped duplicate reg ClassicalRegister(6, 'cr')"```

### Steps to reproduce the problem
See attached test case to reproduce the issue.
[classical_register_err.py.txt](https://github.com/Qiskit/qiskit-terra/files/3298331/classical_register_err.py.txt)

### What is the expected behavior?
Transpiler should not crash

### Suggested solutions

This may be fixed after #2584 is taken care of.


pull request for e2e test at 01/26/2017 15:56:10

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Is there support for cert auth against the target service ?

There doesn't appear to be.
There doesn't appear to be a way to config the http-proxy-middleware...

If the target service is on HTTPS, yes.
servirtium-javascript - might need some small upgrade work, to be honest.

hi Paul, it's been a long time, hope you are well :). We (I'm working with @Nick-Minutello , see the comment above) need to configure the Http Proxy Middleware so we can handle certificate authentication. We have made the change and it works. We would like to create a pull request. But we don't seem to have the permission to push a branch. Can it be relaxed for us?
Happy to contribute an example of how to set cert auth.

hi Paul, it's been a long time, hope you are well :). We (I'm working with @Nick-Minutello , see the comment above) need to configure the Http Proxy Middleware so we can handle certificate authentication. We have made the change and it works. We would like to create a pull request. But we don't seem to have the permission to push a branch. Can it be relaxed for us?
Happy to contribute an example of how to set cert auth.

Hi, I hope you are doing well. Could you please fork the repository and clone it from your user branch. So in that case you can create PR from fork repository to main repository

hi, I have create a PR by forking it. Could you please take a look? Thank you

Hi Xuemin - indeed it has!
Thanks everyone for caring about this. I do intend to get back to Servirtium generally, but I have 1000 thinks on my backlog and stupidly think I can multitask.
I'm all for more modes of operation (http-proxy, agent) but one I don't want to lose is invisible service-in-the-middle where other than http (no SSL) the client does not know it is using Servirtium as there's no additional HTTP headers (not CORS added, not nothin). Anyway, I most times say yes to a PR ... my skills have been boosted every year by seeing other people's ideas in code :)

Hi Paul, can you elaborate a bit more? Can you give some suggestion how you see this working? Happy to try implement them.

Happy to answer, but I don't fully understand the question

hi Paul, hi Duong, we have done the patching - we had to upgrade Github Actions and Typescript etc to get the build working.
We would like it to be published to NPM JS so we can pull it in as a dependency. We are not sure how we can do that. Can you help?

I've forgotten how to do that. I'm sure I left notes for myself somewhere..

Ugh, NPMJS password reset link email is delayed. After that I have to do 2nd factor, too

@paul-hammant I saw @wickedwukong published 3 days ago, that's why I didnt do that


Unable to locate mscorlib

I'm trying to add a c# class library (4.5) to my MVC 5 class library. Pre beta 6 & RTM, we had been using nuget to get the library into the project. No problems.
Since beta 6 / RTM we have been running into an issue where npm says it is unable to locate mscorlib.
I've removed the whole nuget package thing, and added the 4.5 class library directly to my solution. The class library builds fine, it's pretty basic, and has the following references:

If I leave my project.json file empty, a few libraries are automatically added, mscorlib is one of them (and resolves correctly) -

The issue comes when I try to add the 4.5 class library -

Errors in ....project.json
Unable to locate mscorlib >= <IP_ADDRESS>
Microsoft .NET Development Utility CLR-x86-1.0.0-beta6-12256
It automatically adds the dependency in the project.lock.json file. If I remove it, and lock the project.lock.json file, it works correctly. However since this is being used by a bunch of projects, locking is an issue.
Any ideas? thanks!

/cc @davidfowl

@yhax Can you see if the project builds at the command-line by running dnu build from the VS developer command prompt in the same folder as the project.json file?

@danroth27 sure. So I created a brand new class library project. I tried building without adding the 4.5 library. This is the output:

I then added the 4.5 project to the solution -

And then I tried the dnu build using the VS 2015 dev cmd prompt -

(The message cut off says - Dependency specified was Contracts.V5 >= 1.0.0-* but ended up with Contracts.V5 45.5.0-beta.

What framework does Contracts.V5 45.5.0-beta target?

@davidfowl it targets .NET Framework 4.5. I also tried 4.6, with the same results.
One thing I failed to mention. The contracts library was created with VS 2013. Not sure if that matters. I tried creating an empty library and importing it into the 2015 project above. I don't get the mscorlib problem, but it tells me it can't locate the library:

I too have this issue with the released beta7 runtime, I also have the beta 7 web tools  installed. All wrapped csproj are dn 46,and xproj project json targets dnx46.

@danroth27 closing this for now; not clear if there's any remaining action here. This is all changing anyway so it's doubtful this particular issue will be of significance.


.Net: Bug: .NET, OpenAI Assistant An item with the same key has already been added. Key: step_

After upgrading to the latest version of SK <IP_ADDRESS> and OpenAi SDK 2.1.0 - beta.2 we start getting the following error message:
System.ArgumentException: An item with the same key has already been added.
Key: step_nZg6g0H4Z6tF043ZgJNhiR5B
  at System.Collections.Generic.Dictionary`2.TryInsert(TKey key, TValue value, InsertionBehavior behavior)
  at System.Collections.Generic.Dictionary`2.Add(TKey key, TValue value)
  at Microsoft.SemanticKernel.Agents.OpenAI.Internal.AssistantThreadActions.InvokeStreamingAsync(OpenAIAssistantAgent agent,
AssistantClient client, String threadId, IList`1 messages, OpenAIAssistantInvocationOptions invocationOptions, ILogger logger, Kernel kernel, KernelArguments arguments, CancellationToken cancellationToken)+MoveNext()
  at Microsoft.SemanticKernel.Agents.OpenAI.Internal.AssistantThreadActions.InvokeStreamingAsync(OpenAIAssistantAgent agent, AssistantClient client, String threadId, IList`1 messages, OpenAIAssistantInvocationOptions invocationOptions, ILogger logger,
Kernel kernel, KernelArguments arguments, CancellationToken cancellationToken)+System.Threading.Tasks.Sources.IValueTaskSource.GetResult()
  at AiEngine.SemanticKernel.SemanticKernelService.GetChatMessageContentStreamingAssistantAsync(RequestModel requestModel, OpenAIAssistantAgent assistant, String threadId, List`1 images) in

This happens from time to time, we can't catch the exact pattern.

@JP-droidic  - Thanks for reporting this.  Can you please confirm on whether you are using any plugins or functions with this agent, or is this entirely message generation?  In the meantime, I'll setup a local test to see if I can't reproduce this error for additional insight.

@crickman  Hi Chris
thank you!. Yes, the kernel has 10ish plugins added

Thank you @crickman !


Build client UX for Review this service

Mocks are in Zeplin, also available on Figma (under FOR PRODUCTION - Service Feedback)
Here is a Figma prototype to see the flow/interaction between the pages
There is additional copy added for "Learn more" > Services listed on Arrival Advisor page. "How is PeaceGeeks and bc211 using the information collected?" section is being updated to the following:

What happens when I “suggest an update” on the information listed about a service?
When you suggest an update on service information, PeaceGeeks collects and shares with bc211 your suggestions, as well as the name of the service you’re commenting on. We record the date and time that the suggestion was received. If you choose to provide your email address, we collect that so that PeaceGeeks and bc211 may contact you regarding your suggested change. If you work at the service provider organization, you will have the option to share your full name, job title and organization name. It is useful for us to know that the suggestion comes from someone who is familiar with the organization’s work. See also PeaceGeeks’ general privacy policy and bc211’s privacy page.
How does PeaceGeeks use my feedback on services?
You can give feedback about a service by selecting the smiley faces at the bottom of a service page and providing an optional comment about your experience. When you submit this feedback, PeaceGeeks collects and reviews it, and then shares your feedback with the service provider organization. We also record the date and time the feedback was received. See also PeaceGeeks’ general privacy policy.

Acceptance Criteria

[x] How was your experience with [this service] (smileys) can be seen in the service detail screen
[x] After clicking on one of the smileys, the user is navigated to a new screen where they can optionally leave a comment
[x] When clicking on the nth smiley (n=0..3), after navigating to the new screen, the smiley is selected, i.e. the app remembers which smiley was clicked in the first screen
[x] Clicking on the 'X' returns the user back to their previous page
[x] Their rating is not sent to the database if they click on the 'X'
[x] How was your experience with [this service] no longer appears in any service' service detail screen that they have previously reviewed.

Steps for QA:

Open up any service detail page and click on one of the smileys

[x] Notice that you are navigated to a service review screen
[x] You are immediately prompted to leave an optional comment

Click submit, this is not wired up yet.
Click on the X button on the top right corner

[x] Notice a modal pop up giving you the option to continue editing or discard changes

Click discard

[x] Notice you are navigated back to the same service review screen
[x] Notice that you no longer see the smiley face ratings

Refresh the app on your phone
Go back to the same service detail page you were on previously

[x] Notice that you do not see the smiley face ratings

PASSED staging 1.59.0

PASSED staging 1.59.0


Make it possible to list storages and their repositories when browsing

At the moment, it's not possible to list storages, or repositories, just the content of repositories.
It would be great to have this implemented.
This relates to #418, #419, #420, #444.

Working on this !!


defaultHour and defaultMinute ignored after updating minDate

flatpickr version used: v4.6.1
Browser name and version: Chrome Version 75.0.3770.100 (Official Build) (64-bit)
OS and version: MacOS 10.14.5
Reproduction:
https://jsfiddle.net/Lm082ng4/1/

Select a date and time
Close the date picker
Wait for the 10 second setTimeout() to trigger and clear the input
Open the date picker
Notice that the default time is now the new minDate rather than using the defaultHour and defaultTime

I believe it's due to this line in minMaxDateSetter():
if (!self.selectedDates.length && type === "min")
    setHoursFromDate(dateObj);

Great job btw with flatpickr - it really is excellent! :)

Just thinking about this some more....
To my mind you'd only want to call setHoursFromDate if the selected date is today's date and the selected date's hour is a larger value than the defaultHour value?
I may be missing something obvious but I can't think why you'd want to setHoursFromDate if no date was selected - perhaps you would if no defaultHour/defaultMinute was set?

Hello,
any news on this ?
after updating max date, default hour and minutes are lost
thanks.

Hello,
i have a solution, you need to call setHours before updating minDate :
this.$set(this.pickerConfig, 'minDate', minDate.setHours(this.pickerConfig.defaultHour, this.pickerConfig.defaultMinute));

Hello,
i have a solution, you need to call setHours before updating minDate :
this.$set(this.pickerConfig, 'minDate', minDate.setHours(this.pickerConfig.defaultHour, this.pickerConfig.defaultMinute));

hello
Where should this attribute be added

hello,
nothing has been added.
change :
this.$set(this.pickerConfig, 'minDate', minDate);

by
this.$set(this.pickerConfig, 'minDate', minDate.setHours(this.pickerConfig.defaultHour, this.pickerConfig.defaultMinute));

thanks a lot

Hello,
the solution that i gave before does not work because the min hour/minute in now set to the default one.

I can still observe this bug in version 4.6.13 https://jsfiddle.net/dsfxm7n5/3/ after setting minDate defaultHour option is ignored.

open picker and displayed hour is taken from minDate instead of defaultHour
comment setting minDate and defaultHour works correctly

Is there any workaround for this? Solution proposed above with clear will not work for me. I would like to set minDate on opening picker to prevent setting dates in the past and I would like to leave previously selected date intact.

I am also still observing this bug in version 4.6.13. If I include minDate and/or maxDate options in the pickr configuration, the defaultHour option is overwritten/ignored.


Add Table of Contents

from @SgtPooki:

Needs a Table of contents

Didn't add this because Github has a tool that contains a ToC but the blog site where this might be posted will not.


Adding JoinTasksHolder to be able to gather parents feedback

I thought about your PR and definitely I think we should go for something more straighforward.
See below my attempt to keep it simple and flexible:
diff --git a/tukio/workflow.py b/tukio/workflow.py
index 1dae466..e9d49ea 100644
--- a/tukio/workflow.py
+++ b/tukio/workflow.py
@@ -451,6 +451,17 @@ class Workflow(asyncio.Future):
             self._tasks_by_id[task_tmpl.uid] = (task, exec_dict)
             return task

+    def _join_task(self, task, result):
+        """
+        Pass data to a downstream task that has already been started (by
+        another parent). In such a situation, it is known to be a join task.
+        """
+        try:
+            # `data_received()` must be as simple callback (not a coroutine)
+            task.holder.data_received(result)
+        except AttributeError:
+            log.debug('cannot join task %s', task)
+
     def _run_next_tasks(self, task_tmpl, future):
         """
         A callback to be added to each task in order to select and schedule
@@ -471,8 +482,12 @@ class Workflow(asyncio.Future):
         else:
             succ_tmpls = self._template.dag.successors(task_tmpl)
             for succ_tmpl in succ_tmpls:
-                inputs = ((result,), {})
-                succ_task = self._new_task(succ_tmpl, inputs)
+                succ_task = self._tasks_by_id.get(succ_tmpl.uid)
+                if succ_task:
+                    self._join_task(task, result)
+                else:
+                    inputs = ((result,), {})
+                    succ_task = self._new_task(succ_tmpl, inputs)
                 if not succ_task:
                     break
         finally:

replied to most of the comments.
@ludo on your diff could you please make a branch and a PR towards my branch please ? this is difficult to use, test and interact with as it is.
But basically we will have an error on each join since the task already exists. (> think it's just an typo in the given code)
using a data received simple method Will check that

Closed in favor of rebased, simpler solution: https://github.com/optiflows/tukio/pull/14


Add a chain agnostic role based on gaia

We need a new name for the main role used to set up a node.
Options:

cosmos
node
cosmos-node

I have been thinking on how to deal with this multi chain support part. I think that the best route would be to leave/abandon/depricate the existing Gaia role in favor of a new chain-agnostic role.
The chain agnostic role (which is to be named) will start of as a copy of the Gaia.
This would allow the code to continue running until this new role matures into a point that makes sense.
The goal would be to try to remove Gaia (or other chain) specific functions out of this role, into a separate role (or task TBD) and essential build the equivalent of a base-class for cosmos.

I think that the best route would be to leave/abandon/depricate the existing Gaia role in favor of a new chain-agnostic role.
...
The chain agnostic role (which is to be named) will start of as a copy of the Gaia.

This makes sense, I agree on both points.

Todo

[ ] Decide the name of this role (https://github.com/hyphacoop/cosmos-ansible/issues/129#issue-1309557907)
[ ] Decide the name of all gaia_* parameters
[ ] Decide on chain username (currently gaia)
[ ] Decide on how chain parameters are defined
[ ] Decide when fork will happen

I made a couple of tables to help us inform the above decisions:
https://docs.google.com/document/d/1-ce2yTHeMKP6Lc-2wYPrzidu9rmWxvUmfT2bfaYKZS0/edit#

Closed in nextbranch


Develop

Codecov Report

Merging #8 (890390d) into main (f50036e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #8   +/-   ##
=======================================
  Coverage   82.85%   82.85%
=======================================
  Files           3        3
  Lines         105      105
=======================================
  Hits           87       87
  Misses         16       16
  Partials        2        2

Continue to review full report at Codecov.

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


Integration with Rocket should be made simpler if possible

Right now, to integrate with Rocket one has to set up a few things, most notable is the FromData implementation that is needed. It can be seen here: https://github.com/ChrisBuchholz/accord/blob/master/examples/rocket/src/account.rs#L25-L38
If possible, one should be able to derive it or something since those 13 lines will be the same needed for every Type in your Rocket application that uses Accord.

Closed in favor of https://github.com/ChrisBuchholz/accord/issues/12.


cannot read property 'bind' of undefined on RN 0.30

the view initially shows but fails to bind.  not using the TimerMixin seems to work for me - see  body of componentDidMount below. Any suggestions?
componentDidMount() {
const scrollFn = () => {
if (this.scrollView && Platform.OS === 'android') {
const x = this.props.initialPage * this.state.containerWidth;
this.scrollView.scrollTo({ x, animated: false });
}
};
// this.setTimeout(() => {
//   InteractionManager.runAfterInteractions(scrollFn);
// }, 10);
},

It seems to be the interactionmanager acting up on this RN version. Please let me know if anyone else is having this issue.


Cuda out of memory

Hi there, I use the latest version on Linux and I encountered this issue while running the mettack on graph, attack structure only. The dataset is 'Cora' using public split, and I set the number of perturbations to 300. Here is the following error:
RuntimeError                              Traceback (most recent call last)
/tmp/ipykernel_50192/1901287071.py in
1 # Attack
----> 2 model.attack(features, adj, labels, idx_train, idx_unlabeled, n_perturbations=num_remove, ll_constraint=False)
3 modified_adj = model.modified_adj
/tmp/ipykernel_50192/1281517747.py in attack(self, ori_features, ori_adj, labels, idx_train, idx_unlabeled, n_perturbations, ll_constraint, ll_cutoff)
301                 modified_features = ori_features + self.feature_changes
302
--> 303             adj_norm = utils.normalize_adj_tensor(modified_adj)
304             self.inner_train(modified_features, adj_norm, idx_train, idx_unlabeled, labels)
305
~/anaconda3/lib/python3.8/site-packages/deeprobust/graph/utils.py in normalize_adj_tensor(adj, sparse)
219         r_mat_inv = torch.diag(r_inv)
220         mx = r_mat_inv @ mx
--> 221         mx = mx @ r_mat_inv
222     return mx
223
RuntimeError: CUDA out of memory. Tried to allocate 44.00 MiB (GPU 0; 23.70 GiB total capacity; 20.78 GiB already allocated; 36.00 MiB free; 21.49 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
Please help, thanks!

I encountered the same issue when I used the actor dataset from pyg. Cuda out of memory. So I tried to run it on CPU with 256G RAM, but RAM was used up after 8 hours of running, and the process got killed. I set the attack budget to 5% of the total edges. Actor has 7600 nodes and 26752 edges.

I encountered the same issue when I used the actor dataset from pyg. Cuda out of memory. So I tried to run it on CPU with 256G RAM, but RAM was used up after 8 hours of running, and the process got killed. I set the attack budget to 5% of the total edges. Actor has 7600 nodes and 26752 edges.

Running on CPU would be very slow. What is your GPU memory? 32 GB should be enough for Pubmed (20k nodes).

I encountered the same issue when I used the actor dataset from pyg. Cuda out of memory. So I tried to run it on CPU with 256G RAM, but RAM was used up after 8 hours of running, and the process got killed. I set the attack budget to 5% of the total edges. Actor has 7600 nodes and 26752 edges.

Running on CPU would be very slow. What is your GPU memory? 32 GB should be enough for Pubmed (20k nodes).

I tried on GPU with 40G memory but failed.

Are you using 3090 with cuda11?
I change my environment to 1080ti with cuda10 and solve it.
P.S. cuda10 cannot work on 3090 so you have to change your gpu

I encountered the same issue when I used the actor dataset from pyg. Cuda out of memory. So I tried to run it on CPU with 256G RAM, but RAM was used up after 8 hours of running, and the process got killed. I set the attack budget to 5% of the total edges. Actor has 7600 nodes and 26752 edges.

Running on CPU would be very slow. What is your GPU memory? 32 GB should be enough for Pubmed (20k nodes).

I tried on GPU with 40G memory but failed.

I encountered the same issue. It is too low using RAM but runs out of memory using GPU with 48GB. The used dataset has 1,222 nodes and 16,714 edges. Do you have any solutions to accelerate the attack process or reduce memory usage?


Avoid additional predict call in case of ModelAnalyzer

Description
Looks like in case the model is present in erroranalysis, we are doing an additional predict() call. This is ok for small datasets but for large datasets every predict call adds sometime to response time. Hence, removing the additional predict call.
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #1701 (071b8d4) into main (35747ad) will decrease coverage by 5.18%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1701      +/-   ##
==========================================
- Coverage   88.91%   83.73%   -5.19%
==========================================
  Files         105       22      -83
  Lines        5559      418    -5141
==========================================
- Hits         4943      350    -4593
+ Misses        616       68     -548

Flag
Coverage Δ

unittests
83.73% <ø> (-5.19%)
:arrow_down:

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

Impacted Files
Coverage Δ

...ranalysis/erroranalysis/_internal/matrix_filter.py

...ai/databalanceanalysis/feature_balance_measures.py

...onsibleai/responsibleai/serialization_utilities.py

responsibleai/responsibleai/feature_metadata.py

...ponsibleai/databalanceanalysis/balance_measures.py

...onsibleai/_tools/shared/attribute_serialization.py

raiwidgets/raiwidgets/error_handling.py

...ibleai/_tools/shared/state_directory_management.py

raiwidgets/raiwidgets/explanation_constants.py

raiwidgets/raiwidgets/fairness_dashboard.py

... and 73 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Title: Resolved issue with Beacon leader being null

Changes Made:

Modified the resolver for the group query.
Also, populated the leader field of beacons inside the group.

#374

@aadibajpai  Please review my pr


cannot unmarshal number into Go struct field

Hello,
Thank you for Grafana-sync, but I faced this issue during sync my dashboards:
2021/03/05 11:46:32 unmarshal board: json: cannot unmarshal number into Go struct field Board.panels of type string for db/kubernetes-networking-namespace-pods
2021/03/05 11:46:32 unmarshal board: json: cannot unmarshal number into Go struct field Board.panels of type string for db/kubernetes-networking-pod
2021/03/05 11:46:34 unmarshal board: json: cannot unmarshal object into Go struct field TemplateVar.templating.list.query of type string for db/loki-quick-search
2021/03/05 11:46:36 unmarshal board: json: cannot unmarshal object into Go struct field TemplateVar.templating.list.query of type string for db/services-overview

Here is a gist with dashboards https://gist.github.com/ingvarch/51ef3022738d3b9011b2ce802a605aaf
I use :
Grafana v7.4.2 (29e75ad97b)
Grafana-sync - 1.1.0

Hello, thank you for reporting an issue. I will check why some dashboards are not synced

I found that if I remove grafana variables namespace and pod in loki-quick-search dashboard - I can pull this dashboard.

I think that issue with fixed in the latest version of sdk. I Will try to update SDK and test

@ingvarch can you try this version https://github.com/mpostument/grafana-sync/releases/tag/1.1.1 ?

Yes, I tried version 1.1.1 but the same problem
2021/03/05 13:27:41 unmarshal board: json: cannot unmarshal object into Go struct field TemplateVar.templating.list.query of type string for db/kubernetes-compute-resources-workload
2021/03/05 13:27:42 unmarshal board: json: cannot unmarshal number into Go struct field Board.panels of type string for db/kubernetes-networking-namespace-pods
2021/03/05 13:27:42 unmarshal board: json: cannot unmarshal number into Go struct field Board.panels of type string for db/kubernetes-networking-pod
2021/03/05 13:27:43 unmarshal board: json: cannot unmarshal object into Go struct field TemplateVar.templating.list.query of type string for db/loki-quick-search

I opened issue for grafana sdk. Will try to look at it by myself once I have free time

Hi @mpostument ,
i get the same Problem with one of my dashboards.  How could this error be debuged? Is there a verbose oder debug function?

This this sdk issue.  I will check if I can get additional information from sdk itself

This PR was merged yesterday. Potentially it should resolve issue. Will update code late today

Try if this version works https://github.com/mpostument/grafana-sync/releases/tag/1.4.1

Sorry for my delay.
It still doesn't work.
Version 1.4.1
2021/08/11 08:36:27 unmarshal board: json: cannot unmarshal object into Go struct field TemplateVar.templating.list.query of type string for db/kubernetes-networking-workload
2021/08/11 08:36:27 unmarshal board: json: cannot unmarshal object into Go struct field TemplateVar.templating.list.query of type string for db/kubernetes-persistent-volumes
2021/08/11 08:36:27 unmarshal board: json: cannot unmarshal object into Go struct field TemplateVar.templating.list.query of type string for db/kubernetes-pods
2021/08/11 08:36:27 unmarshal board: json: cannot unmarshal object into Go struct field TemplateVar.templating.list.query of type string for db/kubernetes-statefulsets
2021/08/11 08:36:27 unmarshal board: json: cannot unmarshal object into Go struct field TemplateVar.templating.list.query of type string for db/loki-quick-search

same/similar issue:
2022/06/13 16:15:19 unmarshal board: json: cannot unmarshal number into Go struct field FieldConfigCustom.fieldConfig.defaults.custom.spanNulls of type bool (panel "$deployment" of type "timeseries") for db/deployment-network-io```

Going to deploy new version, which may solve this issue

Please try this version https://github.com/mpostument/grafana-sync/releases/tag/1.4.7
It include new version of grafana SDK which may solve this issue

I tried the most recent 1.4.8 release and I am getting these errors:
2022/07/29 11:18:16 unmarshal board: json: cannot unmarshal object into Go struct field Target.targets.group of type []struct { Type string "json:\"type,omitempty\""; Params []string "json:\"params,omitempty\"" } (panel "Metric" of type "graph") for db/aws-cloudwatch-browser
2022/07/29 11:18:16 unmarshal board: json: cannot unmarshal object into Go struct field Target.targets.group of type []struct { Type string "json:\"type,omitempty\""; Params []string "json:\"params,omitempty\"" } (panel "CPUUtilization" of type "graph") for db/aws-ec2
2022/07/29 11:18:16 unmarshal board: json: cannot unmarshal object into Go struct field Target.targets.group of type []struct { Type string "json:\"type,omitempty\""; Params []string "json:\"params,omitempty\"" } (panel "RequestCount / Latency" of type "graph") for db/aws-elb-classic-load-balancer
2022/07/29 11:18:16 unmarshal board: json: cannot unmarshal object into Go struct field Target.targets.group of type []struct { Type string "json:\"type,omitempty\""; Params []string "json:\"params,omitempty\"" } (panel "Healthy / Unhealthy Hosts Count" of type "graph") for db/aws-network-load-balancer-nlb
2022/07/29 11:18:16 unmarshal board: json: cannot unmarshal object into Go struct field Target.targets.group of type []struct { Type string "json:\"type,omitempty\""; Params []string "json:\"params,omitempty\"" } (panel "CPUUtilization/FailedSQLServerAgentJobsCount" of type "graph") for db/aws-rds

The errors looks somewhat different so maybe progress?

Aug 26 09:22:46 bastion teleport[6630]: SQLite configuration is invalid: failed to unmarshal map[data_dir:/var/lib/teleport path:/var/lib/teleport/backend sync:false] into **lite.Config, json: cannot un>
Aug 26 09:22:46 bastion teleport[6630]: "\tjson: cannot unmarshal bool into Go struct field Config.sync of type string"

config item :
  storage:
    type: sqlite
    sync: OFF

when adding quote to OFF it's working :
  storage:
    type: sqlite
    sync: "OFF"

Running 1.4.10, experiencing the same issue.

I am having a similar problem with 1.7.0:
2024/02/14 11:51:44 unmarshal board: json: cannot unmarshal string into Go struct field Target.targets.groupBy of type []string (panel "Max Memory Usage (last 15m)" of type "bargauge") for db/gitlab-runners-metrics

Here is a JSON export of the dashboard in question: gitlab-runners-metrics.json

similar problem with 1.7.0 and grafana 10.2.3
2024/02/16 11:56:31 unmarshal board: json: cannot unmarshal object into Go struct field Target.targets.groupBy of type string (panel "CPU usage/s for $instance" of type "graph") (panel "Resources" of type "row") for db/alertmanager
2024/02/16 11:56:32 unmarshal board: json: cannot unmarshal object into Go struct field Target.targets.tags of type []struct { Key string "json:\"key,omitempty\""; Operator string "json:\"operator,omitempty\""; Value string "json:\"value,omitempty\"" } (panel "Global Samples Sent by Remote" of type "timeseries") for db/prometheus-remote-storage-stats

@mpostument do you have a change to take a look into my comment

https://github.com/mpostument/grafana-sync/issues/6#issuecomment-1948184832

Sorry, there is no fix yet. Looks like grafana sdk is no longer developed. I don't see any PR accepted in sdk repo. Don't have fix right now

@mpostument do you think that the grafana sdk is replaced with https://github.com/grafana/grafana-plugin-sdk-go or it's a different thing?


[Windows] Runtimes bin directory must be in Path

When doing some testing of vscode-swift, realized if the runtimes bin directory at <user dir>\AppData\Local\Programs\Swift\Runtimes\6.0.0\usr\bin\ is not in your path then you cannot run the fully resolved path to swift.exe <user dir>\AppData\Local\Programs\Swift\Toolchains\6.0.0+Asserts\usr\bin\swift.exe which I had set in vscode-swift extension. There was no output from the process so took a long time to figure out what the issue even was. Not sure if there is anything we can do here to relatively find this path when running swift.exe? Not 100% sure how I got in this state to begin with, but can reproduce by removing the Runtimes bin directory from the Path

This sounds like something went wrong with your install or you changed your system. The installer already adjusts the path: https://github.com/swiftlang/swift-installer-scripts/blob/main/platforms/Windows/rtl/msi/rtlmsi.wxs#L19-L23


Most occurrences of i and b should be converted to em or strong, depending on the context

I already did this. Not sure how I accidentally created an issue. Closing as complete.


File uploads using multipart/form-data not working

Adding this to my js file
let multipartHeaders = [
  {
    "name": "content-type",
    "value": "multipart/form-data"
  },
  {
    "name": "example-oauth-v1",
    "value": "<oauth_access_token>"
  }
];

let snippets = {
    usecasesUploadFile: new HTTPSnippet({
      method: 'POST',
      url: "https://www.example.com/api/projects/0.1/projects/15339651/files/",
      headers: multipartHeaders,
      postData: {
        mimeType: "multipart/form-data",
        params: [
          {
            name: "filedata",
            fileName: "/project_dir/sample_file.txt",
            contentType: "text/plain"
          }
        ]
      }
    }),
}

Returns this:
TypeError: form.pipe is not a function
HTTPSnippet../node_modules/httpsnippet/src/index.js.HTTPSnippet.prepare
node_modules/httpsnippet/src/index.js:118

 115 |   })
  116 | })
  117 |
> 118 | form.pipe(es.map(function (data, cb) {
  119 |   request.postData.text += data
  120 | }))
  121 |

Is there something wrong with my usage?

@gschier can you take a look?

Ah yes, I will take a look since this will also fix https://github.com/getinsomnia/insomnia/issues/994

@gschier, what is the status on this?

A fix for this is in https://github.com/Kong/httpsnippet/pull/173!


Store the whole debug log in the top run directory

Seems much better for investigating issues rather then searching
through all the scattered files across the individual steps. Also
remove special color sequences to make it more human readable.

Yes, the indent level is preserved so that the log is easy to read. Thanks for the review.


Does not return correct terrascan-exit-code

It looks like the call to tee might be swallowing the exit code from terrascan?
See https://github.com/yu-iskw/action-terrascan/blob/main/script.sh#L48

@miped Thank you for the feedback. I will investigate the behavior later.

@miped I fixed the issue and released the new version including it. Please make sure of your actions.
https://github.com/yu-iskw/action-terrascan/releases/tag/v1.1.1


Cross-compile aarch/ppc

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.


Add 'What is Prebid.js?' and 'How it Works' pages

These pages are designed to give the reader an on-ramp into
understanding Prebid.js before they try to start using it.
They're set up to appear in the site's header nav in the order in which
they should be read.
'How it Works' also includes a diagram showing how Prebid.js fits into
the page, calls external bidders, etc. to supplement the description in
the text.

@Heray do you need anything else from me to merge this in?
Also happy to join the prebid github org if that makes things easier.


Selecting a directed edge

I am unable to find a way to specify the direction when selecting an edge. If there is traffic from A -> B, and also from B -> A, looks like it chooses the direction with more traffic.
How about adding the capability to specify the direction depending upon which half of the edge was clicked?

We don't have bidirectional connections so this isn't a use case we have thought about. Would definitely welcome a PR for something like this though!

My use case involves bi-directional data flow.  Vizceral out of the box will show particles traveling in both directions simultaneously.
It is subtle to see the particles flying in both directions if one direction has a significantly higher volume than the other with respect to the maxVolume of course.  That is if one is "bigger" (non-trivial difference) than the other but they are both small with respect to maxVolume you should be able to see the particles traveling in both directions.
If you want to prove to yourself that both directions are flowing/producing particles change the metric of one direction to something different so that the particles will have different colors, e.g.
"connections": [
    {"source": "A", "target": "B", "metrics": {"normal": 117}},
    {"source": "B", "target": "A", "metrics": {"warning": 12}}
]

I would be wary of hiding the traffic on a bi-directional connection.  If one direction is so much more important that you would hide the very existence of the reverse flow... do you even need the connection you are willing to hide?

Again, thank you for the well written response @aSqrd-eSqrd.


Better feedback on wrong params in test env

The unit tests depend on data files that live in qiita_db/support_files/test_data, but if you change the BASE_DATA_DIR in your qiita config file, then it looks in the wrong place.
Tests should write out whatever files they need (and then subsequently remove them), rather than depending on files to be located in a particular location.

Changing title after feedback from @biocore/qiita-dev: "Better feedback on wrong params in test env". Basically, the solution for this will be to explicitly say what's wrong and ask the user to change it.

I guess this was added some time ago but this issue was never closed. Note that we even have a test checking for this.

By mistake, added the message for another issue I was closing.
Anyway, this should be closed too as we have improved that feedback over time; if there are any other improvements please add them as new issues. Thanks.


option to enable start animation

I think start animation is disabled. #27 #16 .  Is there any way to enable it optionally ?

Add the startAnimation property to enable the initialization animation.
Please update to version v2.7.6.

Thanks a lot


Is caching specific instance of a module possible?

Description
I am training a student-teacher model where both the teacher and the student are the same network architecture ( module code) , but the student is smaller . The teacher network is frozen, and I want to cache its outputs, while not caching student's outputs
Currently, I can create a dummy wrapper around the teacher and cache its outputs, but I am wondering if there is a native way in torchcache to achieve this.
Use Case
Teacher Network: Frozen, identical to the student network.
Student Network: Same architecture as the teacher but smaller version ( __init__ is passed different model size parameters)
Requirement: Cache the outputs of the teacher network to avoid redundant computations.

Thank you for your assistance!

Good idea @MohitBurkule! I implemented the feature, and added the documentation on how to achieve that. I will release it after the CI passes.

I just released it, it should be up in an hour or so. For instructions on how to achieve selective enablement per instance, see: https://torchcache.readthedocs.io/en/latest/usage.html#selectively-enabling-torchcache-for-module-instances