Meta: create a meta any from a void * + meta type pair

Unfortunately, we can't construct a meta any from a void * + meta type pair since it would be impossible to generate for example container wrappers.
On the other hand, the pair above is usually what is passed back and forth across scripting boundaries. Therefore, it would be great if we could create a meta any from these objects.
A possible approach is:
auto any = type.wrap(ptr);

Any other suggestion is appreciated.

This is tricky to do in code for many reasons. I won't go in details here.
On the other hand, it's pretty easy to do it from the user side. Here is an example.
So, we can safely close the issue.

I see. Yeah this is fine, I do a lot of stuff through meta functions already.

The main problem is that we can add a constructor for meta any that accepts a void * and a meta type but it cannot generate what is needed for container and pointer-like support. So imho it doesn't worth it.

Aren't those things generated for meta-type itself? So that when the user wants to create a meta_any object from meta_type and void* they already have a fully defined meta_type that has all the necessary functions generated.

Aren't those things generated for meta-type itself?

No. At least, not this one. The problem is pretty easy: circular dependency. In fact, we can also break it. However, it complicates a little the codebase and the result isn't as satisfying as it could be.
On the other hand, we can already do this in code, so there isn't an impelling reason to pre-generate the function for all users and classes that don't need it. It add a cost in terms of compilation time and memory occupancy. For example I don't want this class or function generated for int, char and all other primitive types.
So, all in all, I think this conservative approach is better overall for many reasons. Any counterargument? :)


ghcr.m.daocloud.io/hwameistor/evictor:v0.4.0 not found

ghcr.m.daocloud.io/hwameistor/evictor:v0.4.0 not found
Mirror acceleration list may need to be updated

@FloatXD The evictor image has been synced.


[Update] LND v0.14.1

The v0.14.0rc1 is out
https://github.com/lightningnetwork/lnd/releases/tag/v0.14.0-beta.rc1

What do you think about the Postgres support? When should we consider switching (if at all)
https://github.com/lightningnetwork/lnd/blob/master/docs/release-notes/release-notes-0.14.0.md
Postgres database support
This release adds support for Postgres as a database backend to lnd. Postgres has several advantages over the default bbolt backend:

Better handling of large data sets.
On-the-fly database compaction (auto vacuum).
Database replication.
Inspect data while lnd is running (bbolt opens the database exclusively).
Usage of industry-standard tools to manage the stored data, get performance metrics, etc.

Furthermore, the SQL platform opens up possibilities to improve lnd's performance in the future. Bbolt's single-writer model is a severe performance bottleneck, whereas Postgres offers a variety of locking models. Additionally, structured tables reduce the need for custom serialization/deserialization code in lnd, saving developer time and limiting the potential for bugs.
Instructions for enabling Postgres can be found in docs/postgres.md.

need to dig into it more. Certainly would not be a good idea to deploy it for beginners before it is well tested and maybe becomes the default for LND.

v0.14.0rc2 is out, but suggest to wait with testing:
https://twitter.com/c_otto83/status/1457092049661009922
https://twitter.com/c_otto83/status/1457296938903261184

rc3 is out: https://github.com/lightningnetwork/lnd/releases/tag/v0.14.0-beta.rc3

[ ] the build_sdcars.sh should call the lnd.install.sh to avoid code duplication and make updates easier

Will need to wait for LND v0.14.1 to have this sorted: LND v.0.14.0 is incompatible with eclair and c-lightning! Please st...

v0.14.1 is out.
Has any consideration been given to tor.skip-proxy-for-clearnet-targets flag as default or a menu option? Seems like it could really boost routing performance for the network.

v0.14.1 is out.
Has any consideration been given to tor.skip-proxy-for-clearnet-targets flag as default or a menu option? Seems like it could really boost routing performance for the network.

It could improve the response time to clearnet nodes, but won't help the liquidity issues which is the limiting factor in most cases.
Also it is a privacy leak as it exposes the public IPaddress to the clearnet peers so would not offer  it as default, neither in the menu.

Also it is a privacy leak as it exposes the public IPaddress to the clearnet peers so would not offer it as default, neither in the menu.

It seems like it would fit in the configuration flow when selecting between Tor or clearnet as a 3rd option. I can't think of a reason you might want only clearnet and not this hybrid mode.

Agree. The setup is Tor by default now and it should remain so, but there could be an option to activate this "hybrid-mode" with clear warnings re. privacy.
Opt in, not on-by-default.

Yes, I meant default over clearnet, as in clearnet option could be replaced by the hybrid option. Privacy first.

I like the hybrid mode as an option. I can make sense to bundle/integrate this with the Auto-NAT option/setting .. because if you want to go hybrid with clean IP you would like to have that your public as well.

@openoms do you want to add this for v1.7.2 ... otherwise this would be a new issue for v1.8.0

Agree that would make sense to bundle this. Should be some something like "LND Activate clearnet address" to make clear what the effect is.
The setting itself is just a config entry so easy to add.
Would be nice if someone actually running a clearnet node would test it, @benjamin-wilson maybe?

BTW there is an equivalent setting for C-lightning:
https://lightning.readthedocs.io/TOR.html#quick-start-on-linux

always-use-proxy=false

@benjamin-wilson this should work on RPi: https://mullvad.net/en/help/linux-openvpn-installation/
I am using a simple openvpn config and autostart with another provider (not PIA): https://www.raspberrypi-spy.co.uk/2020/06/raspberry-pi-vpn-setup-guide/

Might be easier to do this on the router (already running pfSense). More flexible and don't have to redo when updating. Thorough tutorial by Lawrence Systems: https://www.youtube.com/watch?v=ulRgecz0UsQ


[CODE HEALTH] Remove unused / deprecated interface functions in the Consensus Module

Objective
Cleanup deprecated / unused functions in the Consensus Module and sub packages (ex. Leader_Election & VRF)
Examples of unused / deprecated functions:
func (m *leaderElectionModule) GetModuleName() string {
	return LeaderElectionModuleName
}

Origin Document
As we move fast and break interfaces, skeleton code gets left behind. The core team needs to maintain a quality standard of all modules with Code Health for a lower cognitive load for external contributors.
Goals

Clean Consensus Module of deprecated / unused functions

Deliverable

Code complete implementation
a negative code diff

Non-goals / Non-deliverables

Tackling other TODOs

General issue deliverables

[ ] Update the appropriate CHANGELOG
[ ] Update any relevant READMEs (local and/or global)
[ ] Update any relevant global documentation & references
[ ] If applicable, update the source code tree explanation
[ ] If applicable, add or update a state, sequence or flowchart diagram using mermaid

Creator: @andrewnguyen22

Some unused functions, such as the one in the issue definition GetModuleName(),  are needed because ConsensusModule interface implements the InitializableModule, which is defined as:
type InitializableModule interface {
	GetModuleName() string
	Create(runtime RuntimeMgr) (Module, error)
}

Therefore, to remove such functions we need to modify consensus module interface and what it implements, or change the common interface definitions such as InitializableModule that are used by different modules. @Olshansk what are your thoughts on this?

Other such unused functions that are needed due to imported interfaces:
from Module/InitializableModule interface:
	GetModuleName() string
	Stop() error

from ConfigurableModule interface:
	ValidateConfig(Config) error

from KeyHolderModule interface:
	GetPrivateKey() (crypto.PrivateKey, error)

Therefore, to remove such functions we need to modify consensus module interface and what it implements, or change the common interface definitions such as InitializableModule that are used by different modules. @Olshansk what are your thoughts on this?

The only place in the code where we use .GetModuleName()) is here: https://github.com/pokt-network/pocket/blob/a53a10436a91ea5f3fc55911e5b66594c9311fb8/logger/module.go#L100
func (m *loggerModule) InitLogger() {
	m.logger = m.CreateLoggerForModule(m.GetModuleName())
}

In short, it is how we're going to enable observability to automatically affix the module (and eventually other info) that logs came from, which is why we need to keep it.
Side note: If you add go above the backticks in markdown, it adds syntax highlighting in place:

Re ValidateConfig - we have an empty placeholder for it, but it is used in the consensusModule's Create
func (*consensusModule) Create(runtimeMgr modules.RuntimeMgr) (modules.Module, error) {
	var m *consensusModule

	cfg := runtimeMgr.GetConfig()
	if err := m.ValidateConfig(cfg); err != nil {
		return nil, fmt.Errorf("config validation failed: %w", err)
	}

Did you mean that we don't need them at all?

In short, it is how we're going to enable observability to automatically affix the module (and eventually other info) that logs came from, which is why we need to keep it.

Okay, that makes sense, and I understand why they are useful to us.

Did you mean that we don't need them at all?

No, I don't mean that we don't need them at all. What I wanted to underline that they are not "active and functioning" in the current stage of the code.  As you explained in your previous comment, now it is clear for ValidateConfig  and GetModuleName. So at this point, I think there is no value or meaning to try to remove interface functions as defined in the issue, because they are either placeholders, or will be useful even if they are not implemented fully yet.
However, I will check other unused functions in overall consensus, without limiting the scope to the interface functions as defined in this issue. We can continue on this issue, or if you prefer and agree we can close this one and open a new issue for this.

So at this point, I think there is no value or meaning to try to remove interface functions as defined in the issue, because they are either placeholders, or will be useful even if they are not implemented fully yet.

I think we have TODOs / comments in some places. The takeaway here is that we should add these for all interim placeholders we add.

However, I will check other unused functions in overall consensus, without limiting the scope to the interface functions as defined in this issue. We can continue on this issue, or if you prefer and agree we can close this one and open a new issue for this.

Since you are already looking, reading and understanding the Consensus module, let's use this ticket to identify other functions we could (if need be) remove. If you don't find anything, we can just close this out.
My guess it that VSCode (or some go linter) can potentially do this automatically for us, but I haven't investigated this.

Closing out outdated ticket as the code has significantly changed since this.


CI: Remove insecure db trust auth

Postgres docs says:

     You may also use "POSTGRES_HOST_AUTH_METHOD=trust" to allow all
    connections without a password. This is *not* recommended.

I've never seen trust auth as an issue for local development DBs. If someone has access to my local machine, there are so many other ways to grab the data (like getting the pw from the docker-compose file). To be on the safe side I've just checked whether or not running the DB with docker compose exposes it externally and it does not.
For that reason I don't think we are in a hurry to implement this change on muffi using projects.
I don't think it's a bad change though. It has no impact on how easy the template is to use, but removes a line of configuration and possibly confusion about why that line was there.

I realize now that this is in CI. Lets get it merged right away :-)


[bitmanip] Re-align bitmanip implementation with v.1.0.0 and v.0.93 of the specs - Part 1

This PR contains the first chunk of work to re-align the bitmanip implementation inside Ibex with the ratified v.1.0.0 spec (covering Zba, Zbb, Zbc, and Zbs only) and draft version v.0.93 (covering also Zbe, Zbf, Zbp, Zbr). In particular, this PR takes care of aligning the ratified sub-extensions Zba, Zbb, Zbc and Zbs. The changes were successfully tested using RISCV-DV. However, to keep verifying this in CI and hence before merging this PR a couple of other things need to happen. For details, see #1470.

Thanks for your review @rswarbrick ! I will not yet merge as the PR as it breaks compatibility with the current toolchain/Spike etc.


repl should restart nicely on syntax error

Special case of syntax error because repl entry was cut between multiple lines? This error should be ignored until the following lines makes a valid program as a whole ...
Shall we consider waiting for a specific signal for repl to start interpretating, instaed of trying to interpret immediately each line ? Ctrl-D or Ctrl-Z ?

Special case of syntax error because repl entry was cut between multiple lines? This error should be ignored until the following lines makes a valid program as a whole ...

Shall we consider waiting for a specific signal for repl to start interpretating, instaed of trying to interpret immediately each line  ? Ctrl-D or Ctrl-Z ?

fixed.


[Question/Feature] How to set Index Lifecycle Policy

ECS integration/library project(s) (e.g. Elastic.CommonSchema.Serilog):
Elastic.Serilog.Sinks -> 8.6.0
Elastic.CommonSchema.Serilog -> 8.6.0
Is your feature request related to a problem? Please describe.
Currently the datastream will use the standard logs index lifecycle policy. How to configure this to another policy within our code?
Describe the solution you'd like
Programaticly define the index lifecycle policy and attach them to the datastream.
Describe alternatives you've considered
Changing the ilm through kibana, but this is a hassle

The bootstrap method exposes ilmPolicy as an argument
https://github.com/elastic/ecs-dotnet/blob/b8e730d9e60c52b8c413656d536b6f56d1c438c2/src/Elastic.Ingest.Elasticsearch.CommonSchema/EcsDataStreamChannel.cs#L39
But this is not wired through in any of the logging provider integrations. Will open a PR to add this!
Thanks for bringing this to my attention


Merge only the subrun files that both M1 and M2 exist

With this pull request I changed the functionality of the merge_hdu_files.py script when the --subrun-wise option is used. Originally, even if either M1 or M2 subrun file of a given subrun ID is missing, the script processes the file and changed its name as if both M1 and M2 files exist. Here is the example of what I meant:

data/dl1/dl1_M1.Run05096367.011.h5
data/dl1/dl1_M2.Run05096367.011.h5
--> Output file: data/dl1/merged/dl1_MAGIC.Run05096367.011.h5
data/dl1/dl1_M1.Run05096367.012.h5
--> Output file: data/dl1/merged/dl1_MAGIC.Run05096367.012.h5

Now, with this script the subrun ID that either M1 or M2 subrun file is missing is skipped. If one wants to merge the file, either --run-wise option or simply running without any options can be used.

I will merge it now. Please let me know if you face any problems.


Support elastic search SSL for zombodb

I will finish it before the end of October.

this is already supported.


Feature

restaurant-page
This project comes after the introduction of the ES6 class keyword, ES6 modules, and Webpack. The main goal of this project is for the student to show the understanding of the benefits of modularized code as well as the ability to set up a javascript application made of different modules using Webpack.
Built With

Only Javascript
Webpack
Used the DOM Manipulation.
CSS
Images
Linters

hello, the linters is good, but the  errors is from the deployment  ot from the linters

thanks, to All the TSE's


Checksum mismatch on backported data causes ETL failure

Running the ETL locally on my machine:
Running 124 steps:
1. backport://backport/owid/latest/dataset_70_population__gapminder__hyde__and__un...
Traceback (most recent call last):
  File "/Users/lars/Documents/owid/etl/.venv/bin/etl", line 5, in <module>
    main()
  File "/Users/lars/Documents/owid/etl/.venv/lib/python3.9/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/Users/lars/Documents/owid/etl/.venv/lib/python3.9/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/Users/lars/Documents/owid/etl/.venv/lib/python3.9/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/lars/Documents/owid/etl/.venv/lib/python3.9/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/Users/lars/Documents/owid/etl/etl/command.py", line 81, in main
    run_dag(
  File "/Users/lars/Documents/owid/etl/etl/command.py", line 143, in run_dag
    time_taken = timed_run(lambda: step.run())
  File "/Users/lars/Documents/owid/etl/etl/command.py", line 150, in timed_run
    f()
  File "/Users/lars/Documents/owid/etl/etl/command.py", line 143, in <lambda>
    time_taken = timed_run(lambda: step.run())
  File "/Users/lars/Documents/owid/etl/etl/steps/__init__.py", line 649, in run
    dataset = backport_helpers.create_dataset(
  File "/Users/lars/Documents/owid/etl/etl/backport_helpers.py", line 80, in create_dataset
    values = load_values(short_name)
  File "/Users/lars/Documents/owid/etl/etl/backport_helpers.py", line 21, in load_values
    local_path = walden_ds.ensure_downloaded()
  File "/Users/lars/Documents/owid/etl/vendor/walden/owid/walden/catalog.py", line 196, in ensure_downloaded
    files.download(url, filename, expected_md5=self.md5, quiet=quiet)
  File "/Users/lars/Documents/owid/etl/vendor/walden/owid/walden/files.py", line 88, in download
    raise ChecksumDoesNotMatch(f"for file downloaded from {url}")
owid.walden.files.ChecksumDoesNotMatch: for file downloaded from https://walden.nyc3.digitaloceanspaces.com/backport/latest/dataset_70_population__gapminder__hyde__and__un_values.feather

The file at s3://walden/backport/latest/dataset_70_population__gapminder__hyde__and__un_values.feather has md5 08a2781daf773a397bb839f208ca24b3, but the walden config indicates it should have md5 f4bf438a89deb7d0556d89e7160b6455.

(for next week) @Marigold any ideas what might have happened?

This was my fault, I was profiling in staging environment and accidentally pushed files to walden without committing changes to the repository. I'll think what could be done to prevent this (= my stupidity) in the future.


WordPress HTML - sample markup

Would it be possible to create a HTML sample of ALL of the components preview.html, copied into a simple .html page?
We would use it to inject into a WP page for checking rendering, similar to what Konstantin set up for covid19.
All preview.html/sample.html files in one HTML page would be great (for MVP) - it just needs to be an alphabetical list of component preview.html content.
We can auto-concatenate the output into the file later.
Filename suggestion: cms-tests/wordpress.html

I was going to use phpDocumentor to create a Github Page similar to the way our theme has it https://ca-code-works.github.io/CAWeb/examples/modules.html


The VALUE for WEBSITE_RUN_FROM_PACKAGE must be 1

The value for WEBSITE_RUN_FROM_PACKAGE must be 1. If the value is 'true', it will be ignored at deployment and cause the deployment to fail from the pipeline.
Task name: AzureFunctionApp@1
Description: The value for 'WEBSITE_RUN_FROM_PACKAGE' is set to 'true' in the current source code of this task but it must be 1. If 'true' is set to this app setting, the deployment may fail with "An unknown error has occurred. Check the diagnostic log for details." and you can find "Deployment lock failure" in a trace file for App Service. This can happen especially with Functions on Linux. And if the value is set to "true", the zip file is deployed to the file server for App Service and extracted to /home/site/wwwroot but not mounted. So, /home/site/wwwroot is not set to ReadOnly. This happens because 'true' is not an expected value for 'WEBSITE_RUN_FROM_PACKAGE' and it's ignored. If the value is set to 1, the zip file is mounted and  /home/site/wwwroot is set to ReadOnly.
Documentation changes required: (Y/N) N
Added unit tests: (Y/N) N
Attached related issue: (Y/N) N
Checklist:

[x] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

/azp run

/azp run

/azp run

@rikat-ms can you please bump the task version as per the PR checklist?

@mmrazik would you please check again? I bumped the task version more than two weeks ago and it seems some approvals are required,

Hey @mmrazik , master branch was updated while this PR is waiting for review and it caused conflict in version in task.json. What should I do for it?

Sorry, I completely missed this :(
However, the fix is already in via another PR https://github.com/microsoft/azure-pipelines-tasks/blob/25a06112941d4c8c8db673000fab53ef03aac1f5/Tasks/AzureFunctionAppV1/deploymentProvider/BuiltInLinuxWebAppDeploymentProvider.ts#L12
It is pending deployment to production, though.


Question about Exomes and Genomes Filters

Hello,
Sorry if it is a bad question, I wonder why only exome data is included when I am using function "get_info_from_df"? For example, for "1:55039847:G>A", gnomAD browser shows it has 286 AC for exome and 10 AC for genome (link attached here). While the code db.get_info_from_str("1:55039847:G>A", "*") yields output
chrom                     1
pos                55039847
ref                       G
alt                       A
filter                 PASS
AC                    286.0
AN                1413248.0
AF                 0.000202
MQ                     60.0
QD                   12.991
ReadPosRankSum       -0.025
VarDP               41127.0
AS_VQSLOD            9.0984
AC_grpmax             281.0
AN_grpmax           37626.0
AF_grpmax          0.007468
AF_eas             0.007468
AF_nfe             0.000001
AF_fin                  0.0
AF_afr                  0.0
AF_asj                  0.0
Name: 0, dtype: object
As shown above, output has 286 AC for exome, but no 10 AC for genome. For your information, I am using v4.1 data, with download link presented as follows:
WES gnomAD v4.1 (hg38, 183'558'769 variants) 8.3G zipped, 19G in total - https://zenodo.org/records/11076395/files/gnomad_db.sqlite3.gz?download=1
I wonder if there are any method to get both exome and genome data?
Any feedback is greatly appreciated. Thanks!

Hi @Sophieeeeeeee ,
thank you for your question. You can download WGS v4.1 in a separate folder and have both WES and WGS by initializing two separate instances.
Please let me know if you have further questions.
Best,


Navbar randomly change font size

Operating system: MacOS Mojave 10.14.6
Browser: Safari 12.1.2

I have "over-written" the font-size of the link of the navbar with
.my-class {
font-size: 0.9rem;
}
Sometimes it happens that the font size in the browser is slightly bigger or smaller and, when smaller, :hover make it bigger. It happens just with Safari, that's why I think it's a bug.
Thanks,
SF

as per issue template, we need to see an actual live reduced test case of this happening.


[SourceKit] Mark overridable declarations as dynamic

Explanation: We only set isDynamic to true if we were inside an expression. Also set isDynamic when we are performing cursor info at an overridable declaration. This allows jump-to-definition to jump to declarations that override the one that we performed cursor info on.
Scope: SourceKit Cursor Info
Risk: Low, only affects SourceKit
Testing: Added test case
Issue: rdar://128300752
Reviewer:   @hamishknight on https://github.com/apple/swift/pull/74080

@swift-ci Please test


support for feet and meters as units

Looks done to me  @param {String} unit 'miles', 'feet', 'kilometers', 'meters', or 'degrees'


Automated cherry pick of #5774: fix: azure重装系统删除不必要的参数

Cherry pick of #5774 on release/3.1.
#5774: fix: azure重装系统删除不必要的参数

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/3.1@e843eec). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             release/3.1   #5775   +/-   ##
=============================================
  Coverage               ?   7.45%
=============================================
  Files                  ?     583
  Lines                  ?   98400
  Branches               ?       0
=============================================
  Hits                   ?    7333
  Misses                 ?   90495
  Partials               ?     572

Flag
Coverage Δ

#aFlag
7.45% <0.00%> (?)

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 e843eec...b1f320b. Read the comment docs.

/lgtm

/approve


"custom" configurable Links

Make it possible to configure Links.
One of the practical uses would be a Link pointing towards the "full" dataset (URL containing a prepared download)

done with release: https://github.com/PDOK/goaf/releases/tag/2.0.1


I paid but in was on moonpay> > > > Continuehttps://www.capitalone.com/legal/disclosureshttps://core.telegram.org/file/811140227/2/ZTXUngAbELM.193805/ba6aa233d1d4206207

          > > > > Continuehttps://www.capitalone.com/legal/disclosureshttps://core.telegram.org/file/811140227/2/ZTXUngAbELM.193805/ba6aa233d1d4206207

Originally posted by @Billiej88 in https://github.com/coinbase/cbpay-js/issues/141#issuecomment-1638720072

https://github.com/coinbase/cbpay-js/issues/141#issuecomment-1638720072 #286

          > > > > Continuehttps://www.capitalone.com/legal/disclosureshttps://core.telegram.org/file/811140227/2/ZTXUngAbELM.193805/ba6aa233d1d4206207

Originally posted by @Billiej88 in #141 (comment)


Migrate to TypeScript and use @spotify/web-scripts

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Boxing Questions

Suggestions on boxing: https://qiskit.github.io/openqasm/language/delays.html#boxed-expressions.
Rather than boxas/boxto types, use a single box type. A duration would be optional and a name required (as is the case for any type.
box[optional_duration] name {...}.
For instance, a 1 ms box w/ centered CNOT would be:
box[1ms] cx_centered {
     stretch a;
     delay[a] q;
     cx q[0], q[1];
     delay[a] q;
}

Furthermore, I assume a box may only be used within a gate or a subroutine (or implicitly as {} in a lengthof)?

So a box is not really a type, in the sense that it could be reused or passed to something else. It is just a mechanism for scoping and referring to a part of a QASM program (a subcircuit). Note that a subcircuit is different from a subroutine. A subroutine can be reused, a subcircuit is really a particular piece of code in the context it appears in. In that sense you might want to scope something without giving it a name, perhaps because you want some stretches to only matter and be resolved inside that box. This actually happens a lot, so I think having anonymous boxes is important otherwise you end up creating many unused names.
In principle I'm ok with collapsing box, boxas and boxto into one keyword with extra optional tags for name and duration. Maybe this?
box {...}
box mybox {...}
box [1ms] {...}
box [1ms] mybox {...}
(Tagging @levbishop in case he has further thoughts here).

So a box is not really a type, in the sense that it could be reused or passed to something else. It is just a mechanism for scoping and referring to a part of a QASM program (a subcircuit). Note that a subcircuit is different from a subroutine. A subroutine can be reused, a subcircuit is really a particular piece of code in the context it appears in. In that sense you might want to scope something without giving it a name, perhaps because you want some stretches to only matter and be resolved inside that box. This actually happens a lot, so I think having anonymous boxes is important otherwise you end up creating many unused names.
In principle I'm ok with collapsing box, boxas and boxto into one keyword with extra optional tags for name and duration. Maybe this?
box {...}
box mybox {...}
box [1ms] {...}
box [1ms] mybox {...}
(Tagging @levbishop in case he has further thoughts here).

I think that makes sense.
A few other minor clarifications:

I assume the box can only be used in a gate or subroutine. Could it also be used globally, similar to how one might call a gate globally?
Can a box be reused via the name. For instance, suppose you define box [1ms] mybox {...}. Can you then call it as mybox; within a gate/subroutine definition?

I think that makes sense.
A few other minor clarifications:

I assume the box can only be used in a gate or subroutine. Could it also be used globally, similar to how one might call a gate globally?
Can a box be reused via the name. For instance, suppose you define box [1ms] mybox {...}. Can you then call it as mybox; within a gate/subroutine definition?

I prefer the idea of box rather than boxto or boxas, but I have a slightly different thing in mind to @ajavadia
This ties into #121 where I think lengths are a  way of specifying constraints, not parameters, and those constraints are to be specified in square brackets as [list of linear function of stretches]
So:
// just provides barriers to optimization:
op1; // can commute across box but not into box
box {
op2; // can optimize within box but not migrate out of box
op3;
...
delay [...] ; // localized within box
...
barrier; //localized within box
...
}

// constrains both boxes to same length:
stretch mylen;
box [mylen] {...}
box [mylen] {...}

// kind of arbitrary example but after solving: mylen1=2ms, mylen2=3ms, delay of 1.1ms
stretch mylen1, mylen2;
box [mylen1*0.5, 1ms] {...}
box [3*mylen2, 2*mylen2] {...}
delay [0.1mylen1 + 0.3 mylen2] {...}

This elminates the need for the "name" field of box, and requirement to wrap it in lengthof to extract the length of that name. If you want to add another way to refer to the length of a box, you can just include that name in the list:
\\ the example from the spec, with the new syntax:
stretch mybox;
box [mybox] {
    cx q[0], q[1];
    delay[200ns] q[0];
}
delay[mybox] q[2], q[3];
cx q[2], q[3];

introduces mybox  as an alias for the duration of that box.
The original motivation  for having "name" of a box as a separate field was because of the idea that maybe you could reuse a box by name, as @zachschoenfeld33 was pondering, but that seems tricky to specify at least when the box contents refer to virtual qubits rather than physical qubits. (My suggested syntax above does not preclude allowing a "name" field in the future qasm 3.1  if we can figure out a consistent semantics to go with the syntax....)

I prefer the idea of box rather than boxto or boxas, but I have a slightly different thing in mind to @ajavadia
This ties into #121 where I think lengths are a  way of specifying constraints, not parameters, and those constraints are to be specified in square brackets as [list of linear function of stretches]
So:
// just provides barriers to optimization:
op1; // can commute across box but not into box
box {
op2; // can optimize within box but not migrate out of box
op3;
...
delay [...] ; // localized within box
...
barrier; //localized within box
...
}

// constrains both boxes to same length:
stretch mylen;
box [mylen] {...}
box [mylen] {...}

// kind of arbitrary example but after solving: mylen1=2ms, mylen2=3ms, delay of 1.1ms
stretch mylen1, mylen2;
box [mylen1*0.5, 1ms] {...}
box [3*mylen2, 2*mylen2] {...}
delay [0.1mylen1 + 0.3 mylen2] {...}

This elminates the need for the "name" field of box, and requirement to wrap it in lengthof to extract the length of that name. If you want to add another way to refer to the length of a box, you can just include that name in the list:
\\ the example from the spec, with the new syntax:
stretch mybox;
box [mybox] {
    cx q[0], q[1];
    delay[200ns] q[0];
}
delay[mybox] q[2], q[3];
cx q[2], q[3];

introduces mybox  as an alias for the duration of that box.
The original motivation  for having "name" of a box as a separate field was because of the idea that maybe you could reuse a box by name, as @zachschoenfeld33 was pondering, but that seems tricky to specify at least when the box contents refer to virtual qubits rather than physical qubits. (My suggested syntax above does not preclude allowing a "name" field in the future qasm 3.1  if we can figure out a consistent semantics to go with the syntax....)

@levbishop what is implied by box [len1, len2]? We could also perhaps get around the "reuse" issue by instead defining a new gate which can be reused for each box. In this case, you'd just be using the box as a barrier and to specify a specific length, which seems to be its main intent (rather than redefining portions of circuits).

@levbishop what is implied by box [len1, len2]? We could also perhaps get around the "reuse" issue by instead defining a new gate which can be reused for each box. In this case, you'd just be using the box as a barrier and to specify a specific length, which seems to be its main intent (rather than redefining portions of circuits).

box [len1, len2] specifies the constraint that all 3 of len1, len2, and the contents of the box must resolve to the same duration.
Defining a new gate to reuse multiple times is the right way to specify a logical intent to perform the same logical operation in different locations of a circuit. The potential advantage of reusing boxes is more subtle - you can guarantee that the gates within the boxes are compiled to the exact same physical implementation each time. To force the TeX analogy reusing a gate is like putting a snippet of text in a macro with \newcommand or \def and you'll get the same text but with potentially different line-breaks, hyphenation, etc, and reusing a named box would be more like putting the text in a \makebox, so it would be identical each time. You can imagine this being useful for some kinds of decoupling (or to go crazy one can imagine even doing zero noise extrapolation for error mitigation with some analogy to \scalebox to re-use the exact compilation but perform the scaling). The hard part with all this is that whereas for latex a box is just a rectangle of the page and can be relocated arbitrarily, for qasm the box has context within the circuit (eg the mapping of logical to physical qubits) and can't be so easily shifted.

box [len1, len2] specifies the constraint that all 3 of len1, len2, and the contents of the box must resolve to the same duration.
Defining a new gate to reuse multiple times is the right way to specify a logical intent to perform the same logical operation in different locations of a circuit. The potential advantage of reusing boxes is more subtle - you can guarantee that the gates within the boxes are compiled to the exact same physical implementation each time. To force the TeX analogy reusing a gate is like putting a snippet of text in a macro with \newcommand or \def and you'll get the same text but with potentially different line-breaks, hyphenation, etc, and reusing a named box would be more like putting the text in a \makebox, so it would be identical each time. You can imagine this being useful for some kinds of decoupling (or to go crazy one can imagine even doing zero noise extrapolation for error mitigation with some analogy to \scalebox to re-use the exact compilation but perform the scaling). The hard part with all this is that whereas for latex a box is just a rectangle of the page and can be relocated arbitrarily, for qasm the box has context within the circuit (eg the mapping of logical to physical qubits) and can't be so easily shifted.

Sorry for the delayed reply. This makes sense to me. So shall we go forward by not including any named boxes and instead having the syntax
{

}
```,
where `len1=len2=...=lenN`?

We could also likely remove `lengthof` as you can instead just use the box constraint parameter but it's not a huge deal.

The other question is, are boxes global or defines within `gates/subroutines`? It seems that based on what you are saying @levbishop, they should not be defined within circuits as the mapping to physical qubits may be different each time.

Closing via https://github.com/Qiskit/openqasm/issues/206.

How did you use the box type in qiskit? https://de.quantum-computing.ibm.com/lab/docs/iql/manage/systems/dynamic-circuits/feature-table states that it is not supported in qiskit. I am not able to use one, any suggestions are appreciated!

box is forward looking for future QPU installations and is probably only supported experimentally in the lab at the present time.


[Task] Eliminate /redirect route when all forms no longer use external salesforce forms?

Right now, we're using a /redirect route to avoid weird partial page loading/unloading behaviors on redirects to salesforce forms. Not sure if we really need this, but it's nice to have for now, but I think we could remove this if we wanted to.
Current behavior

login.gov redirects to /redirect
blank page (could be styled differently) is loaded instead of a partial page
A more minor refactor would be to clean up the back behavior

Expected behavior

all forms are served by us instead of salesforce

Looks like we're not going to be using Salesforce forms, so this will not be necessary.


Create dataset loader for QED

Dataloader name: qed/qed.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?qed

Dataset
qed

Description
QED - The QCRI Educational Domain Corpus (formerly QCRI AMARA Corpus) is an open multilingual collection of subtitles for educational videos and lectures collaboratively transcribed and translated over the AMARA web-based platform. It's developed by Qatar Computing Research Institute, Arabic Language Technologies Group. Along with English, it covers multiple SEA languages, such as vi (Vietnamese), my (Burnmese), jv (Javanese), id (Indonesia), th (Thai), tl (Tagalog), ms (Malaysia).

Subsets
-

Languages
eng, vie, tha, mya, jav, ind, tgl, zlm

Tasks
Machine Translation, Language Modeling

License
Other (other)

Homepage
https://opus.nlpl.eu/QED/corpus/version/QED

HF URL
-

Paper URL
https://aclanthology.org/L14-1675/

#self-assign


Refactor JWT module for JBallerina

Purpose
Refactor native implementation and unit tests of Crypto module for JBallerina.
Since this PR depends on https://github.com/ballerina-platform/ballerina-lang/pull/16013, please DO NOT merge this, until it gets merged.
Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #16039 into jballerina will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             jballerina   #16039   +/-   ##
=============================================
  Coverage         13.78%   13.78%
  Complexity         3381     3381
=============================================
  Files              1469     1469
  Lines             74560    74560
  Branches           9834     9834
=============================================
  Hits              10281    10281
  Misses            62955    62955
  Partials           1324     1324

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 a5babc4...d6be8a6. Read the comment docs.


Missing temps

I have 6 jk bms in total, 4 new 2 old  connected via Bluetooth.  Since the batmon update everything works as it should until the last HA update, when I had to turn bleak off for it to run again. My issue is tha I can not see any temperatures for any BMS consistently. They appear for around 10secs and disappear for around 40secs. Is this a mqtt throttling issue or a batmon issue.
Any advice please

I have also a problem with my two Supervolt batteries.
The temperature values switched very often to not available.
The log dispays parsed temperature values but the entity does not display the value.
I observe this behavior with batmon HA 1.78

The same for me as well for about 3 months,  not critical but issue exists

Hi,
I have the same problem for about 4 months now :(

Hi, Has anybody found a solution for this problem? I have the same problem for over a year now. I am using 1.78.


libsForQt5.qmltermwidget: Fix some things, make Lomiri-compatible

Description of changes
Working towards #99090.
lomiri-terminal-app needs a qmltermwidget that exposes certain interfaces. This pulls in the necessary patches, and cleans up some aspects of this derivation.

Clarify license from GPL-2 to specifically GPL-2+
Don't set INSTALL_ROOT as a substitute for using the PREFIX, not what that variable is for (it works like DESTDIR)
Pull in patches that allow lomiri-terminal-app to use this version of qmltermwidget

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2498


Use GCI/VCI's user authentication system

I want to see if we can piggyback on the Cognito user pool that the GCI and VCI use.

There's a proof of concept here: https://github.com/ClinGen/stanford-affils/commit/e6ea2b5cde137e2f88cd5f0fea3627216405bde1. If that were production code, it would need better error handling and tests.


Improve FAQ Page Design

Fixes:  #952
Description
Contribution:
->Implemented Responsive FAQ Component: Developed a user-friendly FAQ section that adapts seamlessly across various screen sizes, ensuring accessibility for both desktop and mobile users.
->Added Hover Effects: Enhanced the user interface by incorporating hover effects on the FAQ items, improving interactivity and visual feedback, making the FAQ section more engaging.
Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots

[X] I have made this change from my own.
[X] I have taken help from some online resources.
[X] My code follows the style guidelines of this project.
[X] I have performed a self-review of my own code.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] I have made corresponding changes to the documentation.
[X] My changes generate no new warnings.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers and screenshots after making the changes.

@PriyaGhosal  mam pls review my code and as you said i have modified

@Subham1
Your pull request has been successfully merged! Don't stop here—continue contributing by addressing more issues and aiming to become a top contributor. Exclusive swag awaits for those who go the extra mile!
Great contribution ! 🎉


Use JsonSerializer instead of Utf8Writer

Summary of the Pull Request
Replace Utf8Writer and manual serilalization from Editor with JsonSerializer. This makes this part of code easier to maintain and less error prone if possible changes are made in this area

PR Checklist

[x] Applies to #1672

Why not JSOn.net?

@crutkas
do you mean https://www.newtonsoft.com/json?

@enricogior  yes.  @ryanbodrug-microsoft was chatting about this on a different thread.  I Just think across the product we pick one and move forward.  Ryan suggested System.Text.Json as that is what is here.  I'm fine with that decision

Since in the FZ editor, it is used just for a couple of calls I would suggest to stick with System.Text.Json, and revisit it when moving to .NET Core 3.1.
This may be an interesting reading https://michaelscodingspot.com/the-battle-of-c-to-json-serializers-in-net-core-3/


Fix typo

Proposed changes
I just fixed a typo found in the documentation.

Thanks for fixing this!


Add contributor name to Map View

There is already a contributor name on TileView and ListView for contributions (#917), we are also adding conditional logic to have it show up as admin (#933). This is the issue to add it to MapView

Fixed in #917


Hi, can u share the code how to export origina weights to onnx?

Hi, can u share the code how to export origina weights to onnx?

@luohao123 currently it is not ready ti public release. We will release onnx exporter soon.

I have made one: https://github.com/luohao123/gaintmodels/blob/master/export_df_onnx.py

@bes-dev Hi, I found my exported onnnx inference result is wrong, can u release your version..

ALSO i meet a lot of problemzs when exporting the onnx. For excample the triu op can not erxport and the model of unet is very huge. can not export.

@luohao123 currently diffusers library should support onnx export by default.


Compile this package with "compilationMode": "partial", so the onus of full Ivy is left to when we build the full application

I use NX mono repo for many large enterprise application.  NX has the best practice of encouraging non-ivy builds for publishable library.  That is great, as Ivy kicks in at application level, and with all data (code-signature) still available, it can make the best decisions in its static complication analysis.
With that said, I do have an internal i18n module that utilizes ngx-translate/code, and it worked great till the v13 release which is ivy only.
Current behavior
A publishable library that relies on ngx-translate/* and has decided to use "enableIvy": false or "compilationMode": "partial" would fail to locate the translate pipe on compilation after upgrading to v13 of this package.
Expected behavior
V13 should not be a breaking change, Ivy flag of a publishable library as per Angular's V13 instruction can be set to  "compilationMode": "partial".
How do you think that we should fix this?
Enable the  "compilationMode": "partial" on the builds, so the full Ivy compilation is postponed to when we compile the full application.
Minimal reproduction of the problem with instructions
Add<EMAIL_ADDRESS>to your project, enable it and try to compile the application.
https://www.npmjs.com/package/@fullerstack/ngx-i18n
Environment
"@ngx-translate/core": "14.0.0",
"@ngx-translate/http-loader": "7.0.0",
"@angular/core": "13.0.2",
Browser:

[ x] Chrome (desktop) version 95

For Tooling issues:

Node version: v14.17.0
Platform:  Mac

Others:
vscode

The Angular team recommends to publish ivy only libs from v13, the goal is to remove ngcc as it takes time and ressources.
If you wish you can still use v12.x, since I'm not adding new features and just maintaining the lib so that it still works, there is no other difference for now (and probably for the foreseeable future).

I think even for the version < 14, it would be best if you add "compilationMode": "partial", to satisfy the projects that progress forward alongside angular.
// Works
application <- publishableMode ("enableIvy": false") <- ngx-translate ("enableIvy": false")

// Broken
application <- publishableMode ("compilationMode": "partial") <- ngx-translate ("enableIvy": false")

// Broken
application <- publishableMode ("compilationMode": "partial") <- ngx-translate ("enableIvy": true")


[AccessRequests] Allow Init accessor to be internal

Disallow init accessors in extensions.
Allow init accessor to be internal.

Resolves #71578

@swift-ci please test

@li3zhen1 Sorry it took so long! Would you mind fixing the test-case that is failing or should I?

@swift-ci please test

@swift-ci please test


Add the permission 'Delete version' to 'Access Policies' dialog for one blob container

Storage Explorer Version: 1.15.0-dev
Build: 20200816.2
Branch: master
Platform/OS: Windows 10/ CentOS 7.6.1810 (Core)/ macOS Catalina
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Expand one non-ADLS Gen2 storage account -> Blob Containers.
Select one blob container then right click it -> Click 'Mange Access Policies...'.
Click 'Add' -> Check the permissions on the dialog.

Expect Experience:
The permission 'Delete Versions' displays.
Actual Experience:
The permission 'Delete Versions' doesn't display.

I have check the access policy API in our SDK, it still doesn't support "Delete version" yet.

If the final non-beta SDK allows for this maybe add it? Consider risk once we know if it is possible please.

Put in 1.17.0 and we'll revisit when this is supported.

If by 1.18 we still don't have an ETA for when the SDK/service will support this, then move to future.

Tested with the current API and it still doesn't support these new permissions.

Verified this issue on the build main/20220901.1. Fixed. So, we are going to close it.


New PR to test SignCLA at 2017-11-07 20:19:14.569731385 -0800 PST m=+71.121595802

Body of PR

corp-pass-staging: test succeeded


chore: trigger windows vhd ev2 build pipeline

What type of PR is this?
Automatically trigger the pipeline to build windows vhd ev2 artifacts.
/kind feature

What this PR does / why we need it:
Generate artifacts for windows VHD sig automatically.
Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Only merge after #3439
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall first build on wanqingfu/trigger-ev2 at 60.692%

Totals

Change from base Build<PHONE_NUMBER>:
60.7%

Covered Lines:
2580

Relevant Lines:
4251

💛  - Coveralls


403 Forbidden...?

Hi, I hope that I used everything correctly.
But after successfully running  python -m readme2tex --output README.md INPUT.md I get my README.md file with

in it. Looks good at first, but the URL does not work:
403 Forbidden
RawGit will soon shut down and is no longer serving new repos. Please visit https://rawgit.com/ for more details.

run with --nocdn to mitigated the error, this flag changes links from https://rawgit.com/ to svgs/ local directory
python -m readme2tex --output README.md --nocdn INPUT.md


Lesson 11: ImportError: cannot import name 'LinkToken' from 'brownie'

When I try to run brownie run scripts/advanced_collectible/deploy_and_create.py in the terminal I get returned:

ImportError: cannot import name 'LinkToken' from 'brownie'

I've uninstalled and reinstalled brownie, closed and reopened the terminal and vscode, and deleted my build folder and recompiled.
I've also copy and pasted directly over my code from Patrick's github nft-mix and the error still pops up.
Any help would be greatly appreciated!

Hello @RohanNero
Do you have the link token mock on the test folder?
you can get it from chainlink-mix.

Hello @cromewar,
I just copied and pasted the files from inside the tests folder and the error is still popping up.

@RohanNero Do you have the init.py file?

I had forgotten to copy over the Mock contracts into the test_contracts folder.


Feature/242 label each container to cleanup at next start

@ilmax I added the first part to clean orphaned containers from previous runs. The idea is to label each container and remove the remaining ones in the next run. Until now, this will only cover containers with the status exited. If a container is still running, we don’t know if it is created from another test process.
I'll try to cover that in the next commit. We can probably assign a kind of session id to each container, to identify multiple processes.

@HofmeisterAn I went through the code and I left a nit, this change looks good to me. Solving the other problem may be a bit tricky so I was thinking maybe the user can decide what to do with an overload of the WithCleanup that allows to remove images for running container

Sharing the same session between multiple processes, do not work right away with Unix systems and .NET Core. To clean up orphaned containers that are still running we need a different approach. We probably can merge e0495011ac79d3fbca077c016d2a18873d89d779. This will still help in a lot of cases, but I would like to do some more tests. I’ll close this draft in the meantime.


I can't access google calendar

Google calendar can't access from other browser than chrome. Already change the calendar into other so you can acess the calendar now


Question about 1.19.3 release

I hate to be that kind of guy, but i'm curious if a build for 1.19.3 are in the plans.

I'm working on it when I've time. I'll try to rush this weekend, but is not set in stone.

Thanks for the reply. Looking towards it!


Edits to UG/DG, mainly adding diagrams/rewording, Fix list-register event command

Looks good to me, nice job! :smile:

fixes #139


Closes #6 | Add Loader for XCOPA

Closes #6 | Create loader for XCOPA for th (thailand) and vi (Vietnamese)
Checkbox

[ ] Confirm that this PR is linked to the dataset issue.
[ ] Create the dataloader script seacrowd/sea_datasets/my_dataset/my_dataset.py (please use only lowercase and underscore for dataset naming).
[ ] Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _URLs, _SUPPORTED_TASKS, _SOURCE_VERSION, and _SEACROWD_VERSION variables.
[ ] Implement _info(), _split_generators() and _generate_examples() in dataloader script.
[ ] Make sure that the BUILDER_CONFIGS class attribute is a list with at least one SEACrowdConfig for the source schema and one for a seacrowd schema.
[ ] Confirm dataloader script works with datasets.load_dataset function.
[ ] Confirm that your dataloader script passes the test suite run with python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py.
[ ] If my dataset is local, I have provided an output of the unit-tests in the PR (please copy paste). This is OPTIONAL for public datasets, as we can test these without access to the data files.

Ind
Source

QA

Th
Source

QA

Vie
Source

QA

Hi @FawwazMayda, thanks for contributing on this dataloader!
May I know why Indonesian language is being skipped? Both the datacard and the source dataset has id as one of the supported languages, tho.

Currently it already has id in seacrowd/xcopa so because its already implemented that is why I left with implementing th and vie

In that case, would you extend the script on sea_datasets/xcopa to cover tha and vie aswell? This shd be similar to this previously-merged PR, you may refer the implementation to here:
https://github.com/SEACrowd/seacrowd-datahub/pull/125

updated @sabilmakbar I added all those languages into single xcopa.py

Hi @FawwazMayda, can you try to run the formatter check and fix it accordingly?
I found some import & readability suggestions found like traling whitespace in a line, unused imports, fix reference of internal var in config creation (l to lang)
make check_file=seacrowd/sea_datasets/xcopa/xcopa.py

applied

updated @sabilmakbar


feat: Support customizing connection name for all commands

This PR adds a connection-name arg to all commands.
This also downgrades the Django version to v1.11.29, which might break something but I'm still trying to figure out what it'll break.

Thanks for the PR. I wasn't able to pull it down to make some refactoring (the common code for the connection-name param).
My PR (based on your work here) is https://github.com/bluedenim/vmigration-helper/pull/15

Closing in favor of #15.


Example dataset

Hi,
I am new to lesion-symptom mapping and excited to try out the mulitvariate LSM.
I am running LESYMAP on a windows machine, under the Linux subsystem. I was able to run the two demo scripts available online, one with the univariate VLSM and the other one with the multivariate LSM. I can replicate the results and generate the same figures as shown online.
However, I found something strange after carrying out some testing. The demo scripts can still run smoothly even after I deleted all the .nii.gz files from the "lesions" folder (see the screenshot below). As you can see, the variable "filenames" should have been empty, but it was not. The output looked the same as before. The file path seems fine...

R (version 3.6.2) might have already loaded the files into the workspace or something. So I restarted Ubuntu/R and repeated the process. I got the same output.
I am wondering what might be causing that.
Thanks,
Michael

I suspect the files were not deleted, because the filenames would not in the system when the folder is read:
filenames = Sys.glob(file.path(lesydata, 'lesions', 'Subject*.nii.gz'))

Could it be that you installed lesymap as root using sudo and then tried to delete as normal user, which prevented you form deleting anything. Try:
sudo rm -f /home/labuser/R/x84_64-pc-linux-gnu-library/3.6/LESYMAP/extdata/lesions/*

Also note, there is a docker variant of LESYMAP which I think performs better than WSL: https://github.com/dorianps/docker

Hi Dorian,
Yes I installed lesymap as root using sudo. The sudo rm -f code works! What should I do to add the files back to path?
Thanks,
Michael

Get the zip file from GitHub, unzip, and copy them where they were. Or
reinstall lesymap.
On Wed, Dec 18, 2019, 6:30 PM mkcyeung<EMAIL_ADDRESS>wrote:

Hi Dorian,
Yes I installed lesymap as root using sudo. The sudo rm -f code works!
What should I do to add the files back to path?
Thanks,
Michael
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dorianps/LESYMAP/issues/18?email_source=notifications&email_token=ACFJU7J27L7KMXB5DS6IAZTQZKXCXA5CNFSM4J4TNN5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHH3ALI#issuecomment-567259181,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACFJU7PJ4EAWGM2VUFXKTHTQZKXCXANCNFSM4J4TNN5A
.

I copied the 131 .nii.gz files back to the folder and ran the code again. The filenames are still empty.

Hi Dorian,
I copied the 131 .nii.gz back to the folder and ran the code again. The "filenames" was still empty.

To be honest, i am quite new to Linux/Ubuntu.
Thanks,
Michael

You can run again the devtools install command and it will hopefully just
install lesymap again, without reinstalling antsr (if antsr did not have
any new git commit since you installed it). Let.me know if you still need
help. The docker solution might be more appropriate for you.
On Wed, Dec 18, 2019, 6:49 PM mkcyeung<EMAIL_ADDRESS>wrote:

Reopened #18 https://github.com/dorianps/LESYMAP/issues/18.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dorianps/LESYMAP/issues/18?email_source=notifications&email_token=ACFJU7P5WA57IBJX465ATB3QZKZJRA5CNFSM4J4TNN5KYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOVSKLC3Q#event-2895425902,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACFJU7PBFAEDIV5KH4X3RKLQZKZJRANCNFSM4J4TNN5A
.

Hi Dorian,
Thanks for the prompt reply!
I ran the devtools install command, and it seems that everything was reinstalled. Then I ran the example script again, this time I got no plotted results (i.e., no figure popped out), although the script ran ok. I got plotted results before the reinstallation.
I tried:
sudo rm -f /home/labuser/R/x84_64-pc-linux-gnu-library/3.6/LESYMAP/extdata/lesions/*
But no files were deleted. The "filenames" also did not change.
My understanding is that if I can get the example scripts work, I can then simply replace behavior.txt and the *.nii.gz files under the "extdata" folder by my data set, and run the scripts again. Is that right? My question is how to load my data set in R and run lesymap on it.
I found the Doctor Pull command docker pull dorianps/lesymap in the Docker Hub. What should I do with it?
Thanks a lot for your help!
Michael

@mkcyeung it does not work that way. You should not touch the installation location. Instead put all your nii.gz files in a folder, then put the filenames in an R variables, just like the ones in the example. The behavior filename can also go in another variable, and pass those to lesymap(filenames,behavior).
All nifti files need to be in the same template space, LESYMAP will do some checks automatically to make sure they are.
If you are new to R, I suggest you start with a brief tutorial online to understand how it works.
The docker solution will ask you which folder you want to share with the virtual environment (you need to have the folder sharing enabled in the docker settings). But since you don't have any experience with any of these concepts, maybe you can continue with wsl for now. I plan to make a video on how the docker solution works, not soon though.

I copied and pasted the two demo folders (i.e., lesions and behavior) to desktop, and used the new paths (C:/Users/labuser/Desktop/lesions/Subject*.nii.gz; 131 rows) and (C:/users/labuser/Desktop/behavior/behavior.txt; 1 row) as the filenames and behavior variables, respectively.
I then ran this in the wsl:
list.files("C:/Users/labuser/Desktop/lesions/", pattern = NULL, all.files = FALSE, full.names = FALSE)
The output is character(0), suggesting that R could not detect these two folders, as if they did not exist.
I also ran the same code using the the R outside the wsl (i.e., the window version of R). The 131 .nii.gz file names were shown in the output.
So do I have to put in some code in the wsl to grant permission or something to these folders, so they can be identified in the wsl?
Thank you.

WSL cannot see the Windows path like that. I think the C drive is in
/mnt/c/Users...etc.
Use ls command to explore folders in Linux.
On Thu, Dec 19, 2019, 9:58 PM mkcyeung<EMAIL_ADDRESS>wrote:

I copied and pasted the two demo folders (i.e., lesions and behavior) to
desktop, and used the new paths
(C:/Users/labuser/Desktop/lesions/Subject*.nii.gz; 131 rows) and
(C:/users/labuser/Desktop/behavior/behavior.txt; 1 row) as the filenames
and behavior variables, respectively.
I then ran this in the wsl:
list.files("C:/Users/labuser/Desktop/lesions/", pattern = NULL, all.files
= FALSE, full.names = FALSE)
The output is character(0), suggesting that R could not detect these two
folders, as if they did not exist.
I also ran the same code using the the R outside the wsl (i.e., the window
version of R). The 131 .nii.gz file names were shown in the output.
So do I have to put in some code in the wsl to grant permission or
something to these folders, so they can be identified in the wsl?
Thank you.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dorianps/LESYMAP/issues/18?email_source=notifications&email_token=ACFJU7PVSUARYFMIFN63QY3QZQYGBA5CNFSM4J4TNN5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHLW3XI#issuecomment-567766493,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACFJU7LVO3RWOEAK5R7T6E3QZQYGBANCNFSM4J4TNN5A
.

Thanks for the quick reply. But I still have no idea why the two folders can't be found in the current working directory. I used the ls command.

I am affraid you need help beyond the functionality of lesymap. You will
need to understand how files and folders appear in WSL. As I said, that is
a Linux system and the Windows folders are mounted at a different location
from what you are used to see in Windows. I suggest you search online how
WSL interfaces with Windows.
On Thu, Dec 19, 2019, 10:23 PM mkcyeung<EMAIL_ADDRESS>wrote:

Thanks for the quick reply. But I still have no idea why the two folders
can't be found in the current working directory. I used the ls command.
[image: Untitled]
https://user-images.githubusercontent.com/54640402/71227662-fb5bb780-22ad-11ea-8dfa-112f8f1b18ed.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dorianps/LESYMAP/issues/18?email_source=notifications&email_token=ACFJU7MWXCMLY6C6INMAAZTQZQ3BZA5CNFSM4J4TNN5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHLYDBY#issuecomment-567771527,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACFJU7OTCUG4D5TGVBGC3T3QZQ3BZANCNFSM4J4TNN5A
.

Closing, reopen if necessary.


Apple TV 12.1.1 compatibility

Apple now validates the user agent.  Use "Remote/1021".

where do I have to use that command, in the configure.json?

You update the DacpConnection.js file, changing the User-Agent... it's part of this pull request.

If anyone wants to rush this into a homebridge running on a pi, the path to the file is /usr/local/lib/node_modules/homebridge-dacp/sr/dacp/     adding the one line from Johninaustin fixes the issue after a reboot of homebridge.

where do I have to use that command, in the configure.json?

If anyone wants to rush this into a homebridge running on a pi, the path to the file is /usr/local/lib/node_modules/homebridge-dacp/sr/dacp/ adding the one line from Johninaustin fixes the issue after a reboot of homebridge.

tested and working. Thank you!

I've also made the change and it worked like a champ!

I am having the same error as CooperCGN when trying to use alternate-playpause-switch.
Switch works to control play-pause but does not report status change if play/pause activated by bluetooth AppleTV remote or iOS controller.
error on aeSX RangeError [ERR_OUT_OF_RANGE]: The value of "byteLength" is out of range. It must be >= 1 and <= 6. Received 8     at boundsError (internal/buffer.js:58:9)     at Buffer.readIntBE (internal/buffer.js:341:3)     at _decode (/usr/local/lib/node_modules/homebridge-dacp/src/daap/Decoder.js:927:26)     at _decode (/usr/local/lib/node_modules/homebridge-dacp/src/daap/Decoder.js:933:19)     at Object.decode (/usr/local/lib/node_modules/homebridge-dacp/src/daap/Decoder.js:963:10)     at Request.request [as _callback] (/usr/local/lib/node_modules/homebridge-dacp/src/dacp/DacpConnection.js:145:29)     at Request.self.callback (/usr/local/lib/node_modules/homebridge-dacp/node_modules/request/request.js:185:22)     at Request.emit (events.js:197:13)     at Request.<anonymous> (/usr/local/lib/node_modules/homebridge-dacp/node_modules/request/request.js:1161:10)     at Request.emit (events.js:197:13)     at IncomingMessage.<anonymous> (/usr/local/lib/node_modules/homebridge-dacp/node_modules/request/request.js:1083:12)     at Object.onceWrapper (events.js:285:13)     at IncomingMessage.emit (events.js:202:15)     at endReadableNT (_stream_readable.js:1129:12)     at processTicksAndRejections (internal/process/next_tick.js:76:17) [3/5/2019, 5:11:14 PM] [DACP] Connected to Lounge


Remove long press gesture on MessageThread messages

What
Remove the long touch press gesture on messages
Why
After trying it we have decided to remove this after the UX isn't great.
How Tested
Storybook - can still edi/remove messages by touch, pointer and keyboard nav

Actually, missed a feature here, will reopen again


waybar: 0.10.1 -> 0.10.2

Description of changes
Update to version 0.10.2
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 306330 run on x86_64-linux 1

1 package built:

waybar

Remove cava bump

Done.
Sorry for that. The update script added the cava bump.

Remove cava bump

Done. Sorry for that. The update script added the cava bump.

Interesting.. might have been fooled seeing the same version number...

Result of nixpkgs-review pr 306330 run on x86_64-linux 1

1 package built:

waybar

Please resolve the merge conflict.


Iox #611 design draft shared memory object creation

Pre-Review Checklist for the PR Author

[ ] Code follows the coding style of CONTRIBUTING.md
[ ] Tests follow the best practice for testing
[ ] Branch follows the naming format (iox-#123-this-is-a-branch)
[ ] Commits messages are according to this guideline

[ ] Commit messages have the issue ID (iox-#123 commit text)
[ ] Commit messages are signed (git commit -s)
[ ] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[ ] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[ ] Relevant issues are linked
[ ] Add sensible notes for the reviewer
[ ] All checks have passed (except task-list-completed)
[ ] Assign PR to reviewer

Notes for Reviewer

First version of a design draft as a follow-up of #404
Not for v1.0 but instead probably v2.0

Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code according to our coding style and naming conventions
[ ] Unit tests have been written for new behavior
[ ] Public API changes are documented via doxygen
[ ] Copyright owner are updated in the changed files
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Closes partly #611

@mossmaurice out of the three oldest open PRs this is the one which smells the most :D. Can we close it.
If you just want to have a quick link to this PR, you can open a page on Discussion and link it from there.

@elBoberido
Good idea, done  :heavy_check_mark:
Created this discussion: https://github.com/eclipse-iceoryx/iceoryx/discussions/1864


Malloc errors on Apple M1

I have been having some trouble getting libxsmm to work on the Apple M1 architecture
$ sysctl -a | grep brand
> machdep.cpu.brand_string: Apple M1

I compiled the latest master branch (67e5663) with
$ make STATIC=1 CXX=g++ CC=gcc FC=

I then compiled and downloaded the PyFR test. Running /libxsmm/samples/pyfr/test.sh I get the following error:
$ . ./test.sh
pyfr_driver_asp_reg(78775,0x1048c0580) malloc: *** error for object 0x1048fc000: pointer being freed was not allocated
pyfr_driver_asp_reg(78775,0x1048c0580) malloc: *** set a breakpoint in malloc_error_break to debug
[3]    78775 abort      ./pyfr_driver_asp_reg $m 48000 10 > /dev/null
./mats/p1/hex/m0-sp.mtx failed!
pyfr_driver_asp_reg(78776,0x1049b8580) malloc: *** error for object 0x104a38000: pointer being freed was not allocated
pyfr_driver_asp_reg(78776,0x1049b8580) malloc: *** set a breakpoint in malloc_error_break to debug
[3]    78776 abort      ./pyfr_driver_asp_reg $m 48000 10 > /dev/null
./mats/p1/hex/m132-sp.mtx failed!
...

Recompiling with $ make STATIC=0 CXX=g++ CC=gcc FC= and trying to actually run PyFR 1.13.0 I get the following:
Python(65961,0x1030e0580) malloc: *** error for object 0x1172e8000: pointer being freed was not allocated
Python(65961,0x1030e0580) malloc: *** set a breakpoint in malloc_error_break to debug
[Wills-MBP:65961] *** Process received signal ***
[Wills-MBP:65961] Signal: Abort trap: 6 (6)
[Wills-MBP:65961] Signal code:  (0)
[Wills-MBP:65961] [ 0] 0   libsystem_platform.dylib            0x000000019c6904e4 _sigtramp + 56
[Wills-MBP:65961] [ 1] 0   libsystem_pthread.dylib             0x000000019c678eb0 pthread_kill + 288
[Wills-MBP:65961] [ 2] 0   libsystem_c.dylib                   0x000000019c5b6314 abort + 164
[Wills-MBP:65961] [ 3] 0   libsystem_malloc.dylib              0x000000019c49ba1c has_default_zone0 + 0
[Wills-MBP:65961] [ 4] 0   libsystem_malloc.dylib              0x000000019c49f060 malloc_report + 64
[Wills-MBP:65961] [ 5] 0   libsystem_malloc.dylib              0x000000019c48dfa8 free + 500
[Wills-MBP:65961] [ 6] 0   libxsmm.1.dylib                     0x0000000122b0c520 libxsmm_xfree + 116
[Wills-MBP:65961] [ 7] 0   libxsmm.1.dylib                     0x0000000122b28f34 libxsmm_dfsspmdm_create + 2864
[Wills-MBP:65961] [ 8] 0   libffi.dylib                        0x00000001aa6fc050 ffi_call_SYSV + 80
[Wills-MBP:65961] [ 9] 0   libffi.dylib                        0x00000001aa7049e4 ffi_call_int + 948
[Wills-MBP:65961] [10] 0   _ctypes.cpython-39-darwin.so        0x0000000103b84eb8 _ctypes_callproc + 864
[Wills-MBP:65961] [11] 0   _ctypes.cpython-39-darwin.so        0x0000000103b7f934 PyCFuncPtr_call + 220
[Wills-MBP:65961] [12] 0   Python                              0x00000001034e6a2c _PyObject_MakeTpCall + 132
[Wills-MBP:65961] [13] 0   Python                              0x00000001035df554 call_function + 268
[Wills-MBP:65961] [14] 0   Python                              0x00000001035d8de4 _PyEval_EvalFrameDefault + 22720
[Wills-MBP:65961] [15] 0   Python                              0x00000001035d2524 _PyEval_EvalCode + 444
[Wills-MBP:65961] [16] 0   Python                              0x00000001034e762c _PyFunction_Vectorcall + 364
[Wills-MBP:65961] [17] 0   Python                              0x00000001034e9fdc method_vectorcall + 124
[Wills-MBP:65961] [18] 0   Python                              0x00000001034e71e4 PyVectorcall_Call + 184
[Wills-MBP:65961] [19] 0   Python                              0x00000001035d3dc0 _PyEval_EvalFrameDefault + 2204
[Wills-MBP:65961] [20] 0   Python                              0x00000001035d2524 _PyEval_EvalCode + 444
[Wills-MBP:65961] [21] 0   Python                              0x00000001034e762c _PyFunction_Vectorcall + 364
[Wills-MBP:65961] [22] 0   Python                              0x00000001034e9fdc method_vectorcall + 124
[Wills-MBP:65961] [23] 0   Python                              0x00000001035df4c8 call_function + 128
[Wills-MBP:65961] [24] 0   Python                              0x00000001035d7734 _PyEval_EvalFrameDefault + 16912
[Wills-MBP:65961] [25] 0   Python                              0x00000001035d2524 _PyEval_EvalCode + 444
[Wills-MBP:65961] [26] 0   Python                              0x00000001034e762c _PyFunction_Vectorcall + 364
[Wills-MBP:65961] [27] 0   Python                              0x00000001035df4c8 call_function + 128
[Wills-MBP:65961] [28] 0   Python                              0x00000001035d76b0 _PyEval_EvalFrameDefault + 16780
[Wills-MBP:65961] [29] 0   Python                              0x00000001034e7574 _PyFunction_Vectorcall + 180
[Wills-MBP:65961] *** End of error message ***
[1]    65961 abort      pyfr run -b openmp -p rht_square.pyfrm euler_vortex_2d.ini
/opt/homebrew/lib/python3.9/site-packages/pytools/prefork.py:94: UserWarning: Prefork server exiting upon apparent death of parent
  warn(f"{who} exiting upon apparent death of {partner}")

Any help on resolving this issue/bug would be excellent.

I've debugged the issue and it is related to the following code fragments:
      LIBXSMM_ASSIGN127( &fp, &k_sparse1 );
      libxsmm_free( fp );

where we are looking to free a JIT'ed sparse kernel.  These fragments occur in both the create routine (where several kernels are generated but only the best performing is retained) and the destroy routine.  It is possible that we're the only bit of libxsmm which tries to release JIT'ed kernels at runtime and that this may require some special treatment on macOS.

yes... the entire Mac aarch64 support is a major hack right now. For now I suggest you by pass the free as done here for other stuff: https://github.com/libxsmm/libxsmm/blob/master/src/libxsmm_main.c#L1377


Proxy + without not working

1. What version of Caddy are you running (caddy -version)?
0.9.3
2. What are you trying to do?
Proxy + without
3. What is your entire Caddyfile?
:2015 {
    ext .html
    gzip
    log / stdout
    proxy /api <IP_ADDRESS>:8080/foo {
        without /api
    }
}

4.  What did you expect to see?
"GET /foo/bar HTTP/1.1"
6. What did you see instead (give full error messages and/or log)?
<IP_ADDRESS> - [02/Dec/2016:21:54:14 +0100] "GET /foo/api/bar HTTP/1.1" 502 16
=> /api is not remove
No problem if I remove /foo from proxy.
Forum thread : https://forum.caddyserver.com/t/proxy-without/1081/2

I was able to reproduce this
Oddly it seems fixed by the code in PR #1256 for  #1200 .  I will investigate further

I just tested with master and it seems to work.
curl -X GET http://<IP_ADDRESS>:2015/api/bar
"GET /foo/bar HTTP/1.1" 502 16

@sguiheux That's great.  I wonder would cc638c7 fix final part of #1200 .  I'll take a look

Great, I'll close this then. :)

This is happening to me after upgrading caddy to 0.10.
abc.xyz {
    root /var/www/xyz
    rewrite /staging/admin/dist/config.json /config/staging.json
    rewrite /staging/view/dist/config.json /config/staging.json
    rewrite /dist/config.json /config/staging.json
    redir /viewer/ /viewer.html
    redir /admin/ /admin.html
    redir /staging/view /staging/view/
    proxy /staging/view http://<IP_ADDRESS>:8081 {
        without /staging/view
    }
    redir /staging/admin /staging/admin/
    proxy /staging/admin http://<IP_ADDRESS>:8082 {
        without /staging/admin
    }
    redir 301 {
        if {>X-Forwarded-Proto} is http
        /  https://{host}{uri}
    }
    tls {
        dns cloudflare
    }
}

Using 0.9.5 a request to abc.zyx/staging/admin/dist/build.js hits http://<IP_ADDRESS>:8082/dist/build.js.
Using 0.10 a request to abc.zyx/staging/admin/dist/build.js hits http://<IP_ADDRESS>:8082/staging/admin/dist/build.js.
The only plugin I use is tls.dns.cloudflare.

@SteffenDE this is a known issue, being tracked here: https://github.com/mholt/caddy/issues/1604
For now I suggest sticking with 0.9.5 if you need without unfortunately.

Ok, thanks for your quick reaction!


Error When Running labs.run_model_bpa Command

Issue Report: Error When Running labs.run_model_bpa Command
Command Executed:
labs.run_model_bpa(dataset='AAA', workspace='BBB', language='fr-FR', export='table')

Error Message:
AmoException: The 'Database' with 'Name' = 'AAA' doesn't exist in the collection.

Description:
When attempting to execute the labs.run_model_bpa command with the specified parameters, we encounter an AmoException indicating that the database named 'AAA' does not exist. This error occurs even though the account executing the command is an admin of the workspace 'BBB' that contains the semantic model.
Context:

The dataset 'AAA'  exist in the specified workspace.
The user account running the command is an workspace admin, which typically should have access to all resources in the workspace.
- Notably, if we take control of the model, the command executes successfully.

Possible Causes:

Database Ownership: The model might require ownership permissions.

Yes, you need to be an admin of the semantic model in order to properly connect to the model and run the various subprocesses. I presume you are also interested in #171.

I experienced the same issue with one semantic model in a premium capacity Workspace. The curious thing is that the code ran successfully on another semantic model in the same Workspace. Both semantic models were configured by someone else and I didn't take over the settings of either of them. The only evident difference between them is the size:
SM1 ran successfully: 85.18 MB
SM2 with error above: 1.49 GB.
F

@fernandocalero what is your permission level for the workspace? Do the semantic models have any roles/rls/ols?
I also found that I don’t need to be the model owner to run bpa.

@m-kovalsky Hi Michael,
I'm a Global Admin and I'm not assigned a specific role in the Workspace. The semantic model doesn't have Roles nor RLS or OLS.

I just started in this position and they are not sharing too much information and are not really interested in semantic link labs, yet. I don't have access to the tenant settings. I'll have to ask. I might have been included in a security group to have access to the Workspace. Sorry I can't answer right now but I'll try and follow up here. Thank you.

@m-kovalsky ,
Hi Michael,
I have good news. I was running the following code to assign the semantic model id and workspace id to variables before running the Vertipaq Analyzer and BPA functions:
ds = fabric.resolve_dataset_id(dataset_name="MY_SEMANTIC_MODEL_NAME", workspace="WORKSPACE_NAME")
ws = fabric.resolve_workspace_id("WORKSPACE_NAME")
Then I was passing the variables to the functions as follows:
labs.vertipaq_analyzer(dataset=ds, workspace=ws)
And there's when I got the error. Now I changed the variable for the actual semantic model name and it worked!
labs.vertipaq_analyzer(dataset="MY_SEMANTIC_MODEL_NAME")

The 'dataset' parameter in the function takes the semantic model name, not the Id. Please read the documentation.


fix with some test cases

I saw you fixed the issue, I will commit my changes anyway... I have some test cases maybe you want to include.
Thanks.

Thanks, but I actually don't want duplicate tests written with different parameters to test the same functionality.
Right now, the tests are setup so that you can change the input parameters for the paged array and all test should pass regardless. Optimally, I would use a tool like QuickCheck for testing a range of different input parameters. But if I would have taken the approach to test all functionality with different hard coded tests for combinations of parameters, I would need hundreds of them.
Thanks again for notifying of the bug in the first place.


Missing LICENSE file

Please add a LICENSE file to this repo to specify what usages are allowed. Thank you!

Done, thanks!


Use **kwargs in do_blacklist

Codecov Report

Merging #868 into master will increase coverage by 0.05%.
The diff coverage is 11.11%.

@@            Coverage Diff             @@
##           master     #868      +/-   ##
==========================================
+ Coverage   63.57%   63.63%   +0.05%
==========================================
  Files           6        6
  Lines        1771     1771
==========================================
+ Hits         1126     1127       +1
+ Misses        645      644       -1

Impacted Files
Coverage Δ

chatcommands.py
48.82% <11.11%> (+0.15%)
:arrow_up:

Continue to review full report at Codecov.

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


The presence of a string of i-xxx-xxx in template raises an error

delete i-xxx-xxx normal

 It would also

Can you provide me a minimal reproduction?

Can you provide me a minimal reproduction?

sorry，I updated my library version, and it was fine


Selecting and moving multiple components at the same time in grid mode

Description
When formatting in grid mode, sometimes it is necessary to add things in later as the dashboard develops. Creating space for a new component is quite cumbersome currently as you have to move each individual component down to create space for the new component above.
Suggested solution
Being able to select multiple components would be quite useful, either via clicking and dragging mouse over desired components to move, or via holding down a hotkey like ctrl and then being able to click on multiple components and move them all together.
Example of multi-select solution on Mac:
https://github.com/marimo-team/marimo/assets/52759282/be0e4fdb-3790-459e-9a6a-872a3346b55a
Alternative
No response
Additional context
No response

We are using a library under the hood - that we can enable a way for the selected grid-items to push around other grid-items (example here). I did find this fragile though and often lead to an undesired state

Ah that is pretty cool as well, the Dynamic Minimum and Maximum Width/Height seems to be addressing the problem directly!
I still believe implementing some sort of multi-select functionality would be the most intuitive and user friendly approach. The rigidness of the current layout is good in that if you accidentally drag something somewhere you dont intend, it doesnt mess up the layout at all and implementing this automatic re-positioning could pose unintended annoyances when trying to tidy up the layout.

I agree with you. We can look into supporting some sort of shift click to select multiple to move


fix: empty sql string

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes https://github.com/StarRocks/starrocks/issues/14572
Problem Summary(Required) ：

The fixes the null check of AST2SQL.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

@mergifyio backport branch-2.5

[FE PR Coverage Check]
:disappointed: fail : 1 / 2 (50.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
1
2
50.00%
[1119]

run starrocks_admit_test

@Mergifyio backport branch-2.5


Download 503 error

Hi I am getting this error for some books (e.g.<PHONE_NUMBER>868,<PHONE_NUMBER>237,<PHONE_NUMBER>453) some others works fine.
[safari-downloader] downloaded meta files... [safari-downloader] downloaded table of content files... [safari-downloader] StatusCodeError: 503 - "\n<?xml version=\"1.0\" encoding=\"utf-8\"?>\n<!DOCTYPE html PUBLIC \"-//W3C//DTD XHTML 1.0 Strict//EN\"\n \"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd\">\n\n<!DOCTYPE html>\n\n\n<title>Site unavailable</title>\n<meta charset='UTF-8'>\n<meta name='description' content='Error page for Safari'/>\n<link href='https://fonts.googleapis.com/css?family=Source+Sans Pro:200italic,200,300italic,300,400italic,400,600italic,600,700italic,700,900italic,900' rel='stylesheet' type='text/css'>\n<link href='https://storage.googleapis.com/safari-fastly-objects/css/error.css' rel='stylesheet' type='text/css'>\n<meta name='viewport' content='width=device-width, initial-scale=1.0'>\n\n\n\n\n\n<svg xmlns=\"http://www.w3.org/2000/svg\" viewBox=\"0 0 60 10\" width=\"78\" height=\"13\"><desc>O'Reilly logo</desc><polygon points=\"28.09 1.96 28.09 0.42 21.68 0.42 21.68 9.64 28.09 9.64 28.09 8.11 23.22 8.11 23.22 5.8 27.86 5.8 27.86 4.27 23.22 4.27 23.22 1.96 28.09 1.96\"></polygon><polygon points=\"32.85 9.64 32.85 0.42 34.39 0.42 34.39 8.11 38.82 8.11 38.82 9.64 32.85 9.64\"></polygon><polygon points=\"40.07 9.64 40.07 0.42 41.61 0.42 41.61 8.11 46.04 8.11 46.04 9.64 40.07 9.64\"></polygon><rect x=\"29.71\" y=\"0.42\" width=\"1.54\" height=\"9.22\"></rect><path d=\"M1.59,6.28a4.8,4.8,0,1,1,4.8,4.8,4.8,4.8,0,0,1-4.8-4.8M4.09,4A3.27,3.27,0,1,0,6.4,3,3.27,3.27,0,0,0,4.09,4\" transform=\"translate(-1.59 -1.2)\"></path><path d=\"M19.82,6.89A2.69,2.69,0,0,0,19,1.62H14.41v9.22h1.54V7h2.14l2.32,3.84H22.2ZM15.95,5.47V3.16H19a1.15,1.15,0,0,1,0,2.31h-3.1Z\" transform=\"translate(-1.59 -1.2)\"></path><path d=\"M13.32,2.61a1.13,1.13,0,1,1-1.13-1.13,1.13,1.13,0,0,1,1.13,1.13\" transform=\"translate(-1.59 -1.2)\"></path><polygon points=\"52.9 0.42 51.03 0.42 48.66 3.85 46.3 0.42 44.43 0.42 47.89 5.44 47.89 9.64 49.43 9.64 49.43 5.44 52.9 0.42\"></polygon><path d=\"M58.31,1.2a2.41,2.41,0,1,0,2.41,2.42A2.42,2.42,0,0,0,58.31,1.2m0,4.44a2,2,0,1,1,2-2,2,2,0,0,1-2,2\" transform=\"translate(-1.59 -1.2)\"></path><path d=\"M59.4,3.09a0.72,0.72,0,0,0-.72-0.72H57.32V4.83h0.41v-1h0.69l0.49,1h0.46l-0.51-1a0.71,0.71,0,0,0,.54-0.69m-1.67-.31h0.95a0.31,0.31,0,0,1,.31.31,0.31,0.31,0,0,1-.31.3H57.73V2.78Z\" transform=\"translate(-1.59 -1.2)\"></path></svg>\n\n
\n\n
Houston, we have a problem.
\n\n
Sorry for the inconvenience. Please try again. For additional help, please contact our customer service team at 1-800-889-8969, 707-827-7019, or <a href=\"mailto:support<EMAIL_ADDRESS>\n
\n\n\n"

same as me

Same here

Same for me

I think it comes mostly from the website.
Can you access the book through the website UI ?

yes, from the website it works perfectly fine. Only the downloader fails for some reason

yes, from the website it works perfectly fine. Only the downloader fails for some reason

Ok, will investigate as soon as I have time.
In the meantime, can you confirm you tried the same ID as the OP ?
Thanks !

Which version are you using ? Could you show the output of
echo `git remote get-url origin`:`git branch --format='%(refname)'`

assuming you named the remote repository origin.
Could you also confirm you get a 503 and not a 404 HTTP error ?

Thanks for your feedback. I am using the forked version and thanks to it I got the 404 error fixed some days ago.
I was trying to download yesterday<PHONE_NUMBER>438 and it gave me the exact same error as the one described by the creator of the issue (503).
I did retry today and it worked fine, no idea why.
For the command you asked me to run, it gave me the following output:

' was unexpected at this time. I tried without this ' and then got the following:

error: option '-b' missing. please consider '--help' for more information.

Thanks for your feedback. I am using the forked version and thanks to it I got the 404 error fixed some days ago.
I was trying to download yesterday<PHONE_NUMBER>438 and it gave me the exact same error as the one described by the creator of the issue (503).
I did retry today and it worked fine, no idea why.
For the command you asked me to run, it gave me the following output:

'` was unexpected at this time.

I tried without '` and then got the following:

error: option '-b' missing. please consider '--help' for more information.

Great !
I guess 503 is returned when they encounter a server side issue but it depends on their implementation which is black box and hard to do reverse engineering.
Sorry about the command, it works only on UNIX based system... Are you running on another system ? If so, you can run it from a MinGW terminal.


A model clashes with field name "event", what to do?

First of all, thanks again for the team to provide such great package.  I personally think django-pghistory
is the best history tracking package out there.
My question: there's another 3rd package "django-scheduler" in my app, and it happens to have "event" in its model:
class EventRelation(models.Model):
    event = models.ForeignKey(Event, on_delete=models.CASCADE, verbose_name=_("event"))
    content_type = models.ForeignKey(ContentType, on_delete=models.CASCADE)
    object_id = models.IntegerField(db_index=True)
    content_object = fields.GenericForeignKey("content_type", "object_id")

Here is the tracking:
class OccasionsConfig(AppConfig):
    def ready(self):
        schedule_eventrelation_model = django_apps.get_model("schedule.EventRelation", require_ready=False)
        pghistory.track(
            pghistory.Snapshot('eventrelation.snapshot'),
            app_label='occasions',
        )(schedule_eventrelation_model)

So migration complains:
occasions.EventRelationEvent.pgh_obj: (fields.E302) Reverse accessor for 'occasions.EventRelationEvent.pgh_obj' clashes with field name 'schedule.EventRelation.event'.
	HINT: Rename field 'schedule.EventRelation.event', or add/change a related_name argument to the definition for field 'occasions.EventRelationEvent.pgh_obj'.

Both packages are important and using .event extensively, what should I do? Since "event" is a fairly common word, is it possible for django-pghistory to have more unique method name such as ".pghistory_event()"?
Thanks again for your suggestion

I found a way to make the migration pass, just adding related_name='history' will do.
        pghistory.track(
            pghistory.Snapshot('eventrelation.snapshot'),
            related_name='history',
            app_label='occasions',
        )(schedule_eventrelation_model)


MP Hands: Hand Left / Right Classification is confused (i.e. wrong often) when hand near edge of screen

Please make sure that this is a bug and also refer to the troubleshooting, FAQ documentation before raising any issues.
System information (Please provide as much relevant information as possible)

Have I written custom code (as opposed to using a stock example script provided in MediaPipe): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04, Android 11, iOS 14.4): Windows 11 Version 21H2 (Build 22000.466)
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
Browser and version (e.g. Google Chrome, Safari) if the issue happens on browser: 97.0.4692.99 (Official Build) (64-bit) (cohort: Stable)
Programming Language and version ( e.g. C++, Python, Java): JS
MediaPipe version<EMAIL_ADDRESS>Bazel version (if compiling from source):
Solution ( e.g. FaceMesh, Pose, Holistic ): Hands
Android Studio, NDK, SDK versions (if issue is related to building in Android environment):
Xcode & Tulsi version (if issue is related to building for iOS):

Describe the current behavior:
The left/right handedness classification of MP Hands isn't stable or reliable, notably when the hand is partially out of frame or partially intersected with the other hand.
Describe the expected behavior:
Expected that the handedness classification is reliable and stable, it shouldn't change from one frame to the next for the same physical hand.
Standalone code to reproduce the issue:
Provide a reproducible test case that is the bare minimum necessary to replicate the problem. If possible, please share a link to Colab/repo link /any notebook:
Just log the results.multiHandedness[handIndex] and try MP Hands, put your hand near the edge of the display or near your other hand when both hands are shown.
Here is a scaffold (not the full repro solution).
function onResults(results) {
canvasCtx.save();
canvasCtx.clearRect(0, 0, canvasElement.width, canvasElement.height);
if (
  results.multiHandLandmarks &&
  results.multiHandedness &&
  results.multiHandWorldLandmarks &&
  results.multiHandLandmarks[0] &&
  results.multiHandWorldLandmarks[0]
) {
  // Outer: Iterate over detected hands (i.e. 0, 1 or 2)
  for (let handIndex = 0; handIndex < results.multiHandLandmarks.length; handIndex++) {

    const classification = results.multiHandedness[handIndex];
    console.log(classification);
  }
}

}
Other info / Complete Logs :
Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached
E.g. Physical hand barely moved, but here is the console output of the handedness label from a sequence of detections.
{
"index": 1,
"score": 0.502685546875,
"label": "Right"
}
{
"index": 1,
"score": 0.62939453125,
"label": "Right"
}
{
"index": 1,
"score": 0.5009765625,
"label": "Right"
}
{
"index": 0,
"score": 0.638671875,
"label": "Left"
}
{
"index": 0,
"score": 0.74462890625,
"label": "Left"
}

Hi @jsegeren, Can you illustrate the above with any reference video on the above query. Thanks!

Sure, here's a cropped video so that I'm not including IP.
https://user-images.githubusercontent.com/2702674/152470451-02dd7a61-9cb2-4d9a-a5f1-0383a1ae882c.mp4

any update on this? I had similar issue when running mediapipe hands in Touchdesigner

Any update?
In my practices, except partially out of frame and partially hands intersected, the handedness would be confused as long as hand is close to nothing detection. In this situation, for example, there is a left(or right) hand exactly, MP would return two left hands or two right hands or a left hand and a right hand, the issue #3902 is one of this.
All above is about hand confusion as there is a left(or right) hand exactly. Another bug is that there would be three hands detected while i set MAX_NUM_HANDS = 2 as there is only one in front of the camera, reference #2319

Just now, MP mistook my face for a hand. The code I used is at https://google.github.io/mediapipe/solutions/hands.html#python-solution-api, and i didn't modify anything except delete section of static image.

Hello @jsegeren,
We are upgrading the MediaPipe Legacy Solutions to new MediaPipe solutions However, the libraries, documentation, and source code for all the MediaPipe Legacy Solutions will continue to be available in our GitHub repository and through library distribution services, such as Maven and NPM.
You can continue to use those legacy solutions in your applications if you choose. Though, we would request you to check new MediaPipe solutions which can help you more easily build and customize ML solutions for your applications. These new solutions will provide a superset of capabilities available in the legacy solutions.