Possible feature add or am i missing something obvious with --no-initial-data

Hi,
I'm using docker atm and trying to automate as much as possible for deployment. During the migrate step of a django app using mezzanine, the command 'python manage.py migrate --no-initial-data --noinput' seems to setup some default fixtures for cartridge/mezzanine which i am hoping to prevent.
https://github.com/stephenmcd/mezzanine/blob/master/mezzanine/core/management/commands/createdb.py
My question is, can --no-initial-data be added to the createdb.py so that 'no' will be assumed instead of 'yes' on setting up data for mezzanine. Or is there some better way i'm missing.  I could use expect at the shell, but i'm hoping for something simpler.
thanks,
-hagan

Hi @haganf,
Please use the Mezzanine mailing list for questions like this going forward.
If I understand your question correctly, you are looking for the --nodata flag for createdb. Take a look at how the command is automated in the fabfile.


confusion bug?

Hello,
I discovered a problem: Sometimes, when there is a short line in between two long lines, the pDBCG mixes up the Line Texts and IDs.
For example:
https://github.com/PRHLT/pageContourGenerator/blob/master/JA.Jv3-068.jpg
https://github.com/PRHLT/pageContourGenerator/blob/master/JA.Jv3-068.xml.orig
When I run the pDBCG on this document (with no options other than input and output specification) I get the following confusions:

line 3 gets Text and ID from line 4 and vice versa
line 11 gets Text and ID from line 12 and vice versa

In EXTRACT mode these confusions are reported:
1865 [0x7fc0e8955ac0] ERROR PRHLT.Page_File null -  Out of place line >> 2 for 3
1901 [0x7fc0e8955ac0] ERROR PRHLT.Page_File null -  Out of place line >> 3 for 2
2078 [0x7fc0e8955ac0] ERROR PRHLT.Page_File null -  Out of place line >> 10 for 11
2111 [0x7fc0e8955ac0] ERROR PRHLT.Page_File null -  Out of place line >> 11 for 10
But they're not corrected. So we end up with a wrong ground truth.
The issue occured with other documents as well.
I tried to figure out where the confusion happens. It should be in
Page_File::load_baselines() or Page_File::get_sorted_baselines().

Dear tigetmeet,
I am a bit unsure about the issue, I just downloaded the master of this code pDBCG and executed the example you provided with  -v0 -v1 -v2 and the out of place text does not appear here. Those messages were provided in the other tool to warn the users not to actually do the changes.
Furthermore uppong extracting the images and text with this tool (pageDistanceBasedContourGenerator) and printing the .txt in order I get exactly the same order as exhibited in the xml. Please see attached image with side by side comparisson of xml and output of txt files and numbers in red.

Is this an issue of the pageDistanceContourGenerator tool  or of the pageContourGenerator tool (from where you got the images) ? If it is of the other tool please close this issue and reopen in the other tool, if the issue is of this tool please provide screenshot of exact command call and output.
Best,
Vicent

Dear vbosch,
thanks for your quick and kind reply.
The issue is with the pageDistanceBasedContourGenerator, not with the pageContourGenerator. Just the example file is from pageContourGenerator.
command:
./extract_lines -i test/JA.Jv3-068.jpg -p test/JA.Jv3-068.xml.orig -o test/output/JA.Jv3-068.xml
Maybe it's clearer when I describe it this way: The IDs and Unicode texts remain in correct order, but the Baselines (and according TextLine Coords) get mixed up:

And after running EXTRACT mode, the text files do not fit the image files, so I have a wrong ground truth:
JA.Jv3-068_02_04_t1_t1_l03.txt
contains the text for
JA.Jv3-068_02_03_t1_t1_l04.png
and vice versa.

Dear tigermeet,
I have now understood your issue and was able to replicate it. It is not very common in HTR to have to generate the contours when the text is already know for each textline. This was causing a bug since I have never encountered this situation.
The distance based contour program was calling a routine to initially generate fixed sized contours just in case there are issues with the more complex routine and no solution is possible so that you always have a backup solution. This was using the calculated order of baselines "to move stuff around "but the text was not being moved with the rest.
I recommend that when you use the command you set default_above, default_below, upper_dist and lower_dist as per the corpus you are using it on. To get the best results and the best back up results possible.
I have made a new commit to the code fixing this. Please download and test. When you give me the OK I will close this ( I have also tested but I would like to be double sure).
(At some point in the future I should do a full rewrite of the page_file class it is messy and the technical debt every time I need to add something or fix something is getting huge. )
Best,
Vicent

Dear vbosch,
thanks a lot for your help!
My use case is: I have page-xmls containing baselines and text and I want to extract the line snippets with ground truth to create training data for PyLaia.
I've tested your new commit. The xml output looks good now: Baseline and text are not confused anymore. Just the ID changes - which is probably ok.

But when I visualize the TextLine Coords with Transkribus, they are still confused: The text does not fit the contour:

As a result, in EXTRACT mode I still get the confusion mentioned before, but now without the error message ERROR PRHLT.Page_File null - Out of place line >>...
Before your commit: Baseline and contour matched, but text did not match.
Now: Baseline and text match, but contour is not matching.
Can you confirm this behavior?
Thanks for your work and time. I appreciate it.

Dear Tigermeet,
I believe I have now fixed this fully. See attached screenshot. Sorry for taking so many iterations to fix this. I am right now in the middle of something else and I am using my free time to review this when I can.

Do let me know if it is now ok for you. Otherwise I guess we will have to do another iteration.
Best,
Vicent

Dear Vicent,
as far as I tested, it works perfectly now. Thank you very much!
No need to apologize! I'm very glad for your free and kind support!


Namespaces based on current path

Using the following command on https://github.com/ogioncz/eudyptes,
vendor/bin/psr4-converter map -m /tmp/.psr4-converter.json --underscore-conversion --ignore-namespaced-underscore --ignore-path="bootstrap.php" -- "App" app/

it appears that the tool would move files around based on existing namespaces:
{
    "originalName": "PostRevisionMapper",
    "newName": "PostRevisionMapper",
    "componentStmtClasses": null,
    "originalFile": "…\/app\/model\/orm\/post_revision\/PostRevisionMapper.php",
    "originalNamespace": "App\\Model",
    "newNamespace": "App\\_\\Model",
    "targetFile": "App\/_\/Model\/PostRevisionMapper.php",
    "targetFileWithoutVendor": "_\/Model\/PostRevisionMapper.php",
    "stmtClass": "PhpParser\\Node\\Stmt\\Class_",
    "exclusive": true,
    "classLike": true,
    "risky": false,
    "details": "class App\\Model\\PostRevisionMapper"
}

I would like to change the namespace based on file location (converted to camel-case) – for example, app/model/orm/post_revision/PostRevisionMapper would have a namespace App\Model\Orm\PostRevision.
Do you think this makes sense to support? Would you accept pull requests adding that as an option?

@jtojnar, This tool didn't support path based conversion, but it makes sense to support it. I created a pre-release 0.4.0 with this support. Can you test it?
Feel free to submit some PR.

Thanks for the quick implementation. I tested it and while it mostly works there are some rough edges.
composer create-project lesphp/psr4-converter
psr4-converter/bin/psr4-converter map -m /tmp/.psr4-converter.json --underscore-conversion --ignore-namespaced-underscore --ignore-path="bootstrap.php" --path-based-conversion --force-names-camelcase -- "App" app/

Trying to move convert files in-place using psr4-converter/bin/psr4-converter convert /tmp/.psr4-converter.json --ignore-vendor-path --allow-risky -- app, the old files remain.
Not sure about the ideal solution, though.

I could use a different directory (e.g. app4), remove the old one and then rename the new one to app/ but that also removes non-PHP/ignored paths.
The tool could rename the files when srcPath is the same between convert and map invocations.

Probably should include a check if the directory is not dirty when using Git, otherwise it could get mixed with other changes.

Since Git does not track file renames but uses heurictics based on file contents, if the file diff is significant enough, comitting the changes will break Git history.

Perhaps we should add a --commit-renames-first that would run git mv on the renamed files and then git commit so that the renaming is happens separately from Git’s perspective.

Or if we do not want to think about writing a commit message, we could name the option --stage-renames and skip the commit part.
Again, we should probably check that the tree is not dirty.

No way to fix references in tests?

Nette Tester prefers using .phpt file extension.
Even if I rename the tests, psr4-converter/bin/psr4-converter map -m /tmp/.psr4-converter-tests.json --underscore-conversion --ignore-namespaced-underscore --ignore-path="unit/bootstrap.php" --path-based-conversion --force-names-camelcase -- "Tests" tests/ will fail to parse them because they contain some non-class code:unit/ChatFormatterTest.php   12:12   Map error, unexpected T_VARIABLE on line 12
unit/FormatterTest.php       15:15   Map error, unexpected T_VARIABLE on line 15

The converter changes formatting during. While this is to be expected to a degree during any code transformations and user should run formatter after conversion, some aspects could surely be preserved (e.g. trailing commas in arrays and function arguments – PHP-CS-Fixer will not be able to return to the previous formatting when the converter compresses a structure to single line and removes the trailing comma, see an example).

Hi @jtojnar.
About the in-place conversion, there is not yet an option in the convert command for that. The reason is that such a conversion would have to deal with conflicts and would add a lot of complexity to the conversion process. For example, a conversion of a file could result in overwriting another file that is yet to be converted, which would cause checksum errors and there would be no way to recover the previous source anymore. I'm not sure it's a good solution. One possibility I've been looking into is a command that does just that: delete affected files from src. This would give more flexibility in the conversion process.

Hi @jtojnar.
Version 0.5.0 has been released with support to remove the converted files with clear command, you can use it to clear the source directory before moving the converted files.
About the test phpt files, you can exclude them from the conversion and just apply a rename command in the tests directory so that the class names are updated.
I'm considering this issue as resolved.

Thanks I have settled on the following workflow:

Install the tool.
composer create-project lesphp/psr4-converter

Create the map.
psr4-converter/bin/psr4-converter map -m /tmp/.psr4-converter.json --underscore-conversion --ignore-namespaced-underscore --ignore-path="bootstrap.php" --path-based-conversion --force-names-camelcase -- "App" app/

Create transformed files in the new temporary directory.
psr4-converter/bin/psr4-converter convert /tmp/.psr4-converter.json --ignore-vendor-path --allow-risky -- app2/

Commit renames separately so that Git can track them.

Copy the original files as targets.jq --raw-output '
	.srcPath as $app
	| .files
	| map(.units[])
	| map(
		($app + "/" + .targetFileWithoutVendor) as $targetFile
		| "mkdir -p \"$(dirname " + ($targetFile|@sh) + ")\""
			+ "\n" + "cp " + (.originalFile|@sh) + " " + ($targetFile|@sh)
	)
	| join("\n")' /tmp/.psr4-converter.json | sh -

Clear the original ones.psr4-converter/bin/psr4-converter clear --dry-run /tmp/.psr4-converter.json

Stage the changes in app/git add app/

Commit the changes.git commit -m 'Rename files before PSR-4 conversion

So that Git can track the renames.'

Replace the renamed files with the converted ones:
cp -r app2/* app/
rm -r app2

Update references in tests:
psr4-converter/bin/psr4-converter rename /tmp/.psr4-converter.json tests/

Run PHP-CS-Fixer and manually revert formatting changes caused by codegen.

Commit the changes.


update lazybones-template build file for our gradle plugin

build file has the original plugin coordinates

Fixed with #5


Fix getLastTransactionSortedByNonce provided parameter

Closes #602

1 was being provided as a nonceGte instead of as limit.
This bug was likely introduced in 9ab0e70aa33ebd0739ca92a90738e75251ab8224 when a new optional parameter was added (nonceGte). See https://github.com/safe-global/safe-client-gateway-nest/commit/9ab0e70aa33ebd0739ca92a90738e75251ab8224#diff-43b9156f9e631726e6cc7662bca73aa1a8f308376f72f895304bd926e0b3784fR126

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 coverage remained the same at 92.496%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4366

Relevant Lines:
4620

💛  - Coveralls


Track selection in source file

Tree view should track current selection in source xml file (corresponding tree node should be highlited).
The same idea works fine user selects some node it the view - the cursor is moved to correspondint text in text editor. But unfortunately it doesn't work in opposite direction.
Really looking forward
Roman

Done as part of the Paris release. If any issue arises, let me know. Thanks

How do I get the latest version?

Hi @RomanKhv,
The Notepad++ Plugin Manager Site is experiencing some problems. If you want to update it now you can follow the instructions at https://github.com/joaoasrosa/nppxmltreeview/wiki
Any problem let me know.
Cheers


Introduce derived NumberAttribute types

I think it would be useful to introduce derived number types for attributed based mapping.
At the moment, We have NumberAttribute but there doesn't look  to be a way to set the NumberType when used as an attribute because the IProperty.Type property is not exposed directly.
The derived types would inherit from NumberAttribute and simply set pass the appropriate NumberType to the base constructor.

Removed labels as this isn't an issue.  NumberType can be passed via the attribute constructor.


Support non-NUL terminated CStr16

According to
https://github.com/tianocore/edk2-archive/blob/072289f45c7ac13e6f70ba4ad739e95086e5d2d2/ShellPkg/Include/Protocol/EfiShellParameters.h#L25-L37, not all CStr are  null terminated and is instead a flat CHAR16 pointer in Pascal style. This is in direct contradiction to uefi::CStr16 which assumes the string to be null terminated

Oh, seems like I misunderstood something. That was an array to a UCS2 string, and each element is indeed null terminated.
#[repr(C)]
#[unsafe_guid("752F3136-4E16-4FDC-A22A-E5F46812F4CA")]
#[derive(Protocol)]
pub struct ShellParameters {
    argv: *const *const Char16,
    argc: usize,
}

impl ShellParameters {
    pub fn arguments(&self) -> impl Iterator<Item = &CStr16> {
        unsafe { slice::from_raw_parts(self.argv, self.argc) }
            .into_iter()
            .map(|&s| unsafe { CStr16::from_ptr(s) })
    }
}

    info!("args: {:?}", shell.arguments().collect_vec());

[ INFO]: yakern-bootloader\src\main.rs@078: args: [CStr16(['F', 'S', '0', ':', '\\', 's', 't', 'a', 'r', 't', 'u', 'p', '.', 'n', 's', 'h', '\0'])]

It works! (sort of)


Support for 'organizeImports'

Thank you for the plugin! I'm using v1-nightly, along with Biome 1.5.1.
In my biome.json file, I have enabled organizeImports:
  "organizeImports": {
    "enabled": true
  }

However, neither the safe fix nor the formatting commands have any effect on the import order. Am I missing something, or is this not supported yet?

This is not currently supported because the plugin uses biome lint and biome format and the organize imports seem to only work with biome check and biome ci commands.


refactor ImageResult & remove ImageScope

// Now
val painter = rememberImageActionPainter(request)
// val action by rememberImageAction(request)
// val painter = rememberImageActionPainter(action)

// Before
val painter = rememberAsyncImagePainter(request)

@qdsfdhvh It would be good to include the README updates in the PR

I've been on holiday recently so haven't had time to update the docs, will follow.


Hi dear. So what about init run with new folder structure?

Updated, thanks
https://github.com/marciocamello/yii2-app-advanced-sh/commit/cef90253aceadd8f0b4bfc58d37de9a6a0e07554


"Tag" button not completely underlined

Whenever I try to create a site using this template the tag is always having a gap in the underline. Same problem whether I run it in localhost or in Netlify. I tried to view the site using different browsers and devices but the problem remained. I am a beginner, how to revolve this issue?

Tailwind config, Its a personal choice/taste. But you can upper the underline offset in src\components\Header.astro.
Modify
...
  nav a.active {
    @apply underline decoration-wavy decoration-2 underline-offset-4;
  }
...

to
...
  nav a.active {
    @apply underline decoration-wavy decoration-2 underline-offset-8;
  }
...

or
...
  nav a.active {
    @apply underline decoration-wavy decoration-2 underline-offset-[5px];
  }
...


Featured Extension Icon Alignment

In the welcome screen:

The alignment of the extension icon seems off.
I think this is because the surrounding space is slightly different in each direction:

Didn't realize this landed. Agree that this needs some polish.


Change drag preview property cause crash on build version

Godot version:
3.2.3 official(steam version)
OS/device including version:
windows 10 professional
Issue description:
Change drag preview property will cause crash on build version( no crash on debug export or in game engine)
Steps to reproduce:

Add a itemlist and give it a preview
Change preview text in gui_input(check whether the preview is exist)
Run in godot or export debug version, run well
Export non-debug version. crash when drag

Use call_deferred cannot fix the crash
Maybe some relation with#36724
Minimal reproduction project:
PreviewCrash.zip

Set preview to null when drop data can fix the crash
func drop_data(position, data):
preview = null
return
Maybe drop data break the gui_input process and free the preview node just before "if preview != null"

The engine takes ownership of whatever control you throw at set_drag_preview. Once the drag is completed, the control is deleted by the engine itself. Your var preview then becomes invalid. So it makes sense that clearing the var mitigates the crash.

Upgrade to 3.2.4 and change your check to if preview or if preview != null, you will get error in debug build. To avoid it properly you should do if is_instance_valid(preview).
Here's a similar recent issue: #46359
Tagging as documentation, #46360 will resolve this.

Upgrade to 3.2.4 and change your check to if preview or if preview != null, you will get error in debug build. To avoid it properly you should do if is_instance_valid(preview).
Here's a similar recent issue: #46359
Tagging as documentation, #46360 will resolve this.

That will fix the crash, thanks!


add hooks for api middleware.

make it easy to handle things before/after redux-api-middleware do it's job.Such as decamelize the request params in before and camelize the response body in after.

Coverage decreased (-2.9%) to 97.059% when pulling 01a87094a52f02470348b28fd01add5e1fa94a16 on Darmody:master into 1543910cfd23e01354039f3c659af5418d8728d2 on agraboso:master.

Coverage remained the same at 100.0% when pulling 330f7ac43a95d4b007ee772ffe6ede711e0eb35a on Darmody:master into 1543910cfd23e01354039f3c659af5418d8728d2 on agraboso:master.

Coverage remained the same at 100.0% when pulling e22fe1cd5b71efbb33d69bce2715619c4892a36d on Darmody:master into 1543910cfd23e01354039f3c659af5418d8728d2 on agraboso:master.

Thanks for your contribution, but I think this PR introduces unnecessary complexities. The functionality of your after hook is essentially already incorporated in the custom payload and meta functions of type descriptors. As for the before hook, I would advocate for those transformations to be made in the code dispatching the RSAA.

First, thanks for redux-api-middleware, which makes my life easier.
If I need to decamelize the request params, I have to do it in everywhere dispatching the RSAA.Also, I have to camelize the response body in every custom function of type descriptors.
Now I implement a redux middleware to hook before/after redux-api-middleware. So I create this PR to add hook feature.

@Darmody, see #58 for an alternative way of accomplishing what you need.

Coverage remained the same at 100.0% when pulling a108de56aa127afd24397637467430e382ea1275 on Darmody:master into 1543910cfd23e01354039f3c659af5418d8728d2 on agraboso:master.


About the input format

Hi Flavio,
Sorry for bothering you again for the following problem.
I’m wondering the input format.
The “Readme" says the first column of “dt user from to” is the amount of time the user spent on “from” before going to “to”.
So the unit of the amount of time is “second"?
i.e., if I have the following .tsv text document:
--------timestamp----------userID--itemID--
2009-04-08T01:53:56Z    user01  item01
2009-04-08T01:57:47Z    user01  item02
I should turn it into the format like:
231  user01   item01   item02
However, if the time interval exceeds a day, the value of the amount of time will be big.

That's it. But this model does not consider time, it is necessary as input simply because that's how I had my files set-up for other models. You should look at:
https://github.com/flaviovdf/tribeflow
or
https://github.com/flaviovdf/prme

Okay,
Thanks a lot !

I think #4 tried to ask this, but since this issue is still open I'll ask it here. I also looked at the samples in https://github.com/flaviovdf/tribeflow and it still wasn't clear to me. Let's say I want to replicate the input data from Figure 1 of the fpmc paper (see below).

Would it be like the following?
0 user01 [a, b, c] [b, c]
0 user01 [b, c] [a,b]
0 user02 [a] [a, c]
0 user03 [d] [c, e]
0 user03 [c, e] [e]
In other words, it isn't clear how to represent the baskets. Thanks!

Hi, as we detail in the paper we focused on baskets of fixed sizes.
See pg 7 of the paper.
--
Flavio
On Thu, Nov 8, 2018 at 12:47 PM Kyle Jackson<EMAIL_ADDRESS>wrote:

I think #4 https://github.com/flaviovdf/fpmc/issues/4 tried to ask
this, but since this issue is still open I'll ask it here. I also looked at
the samples in https://github.com/flaviovdf/tribeflow and it still wasn't
clear to me. Let's say I want to replicate the input data from Figure 1 of
the fpmc paper (see below).
[image: image]
https://user-images.githubusercontent.com/11047068/48205782-cfe06c80-e33a-11e8-9d1c-fb6b2eb3923b.png
Would it be like the following?
0 user01 [a, b, c] [b, c]
0 user01 [b, c] [a,b]
0 user02 [a] [a, c]
0 user03 [d] [c, e]
0 user03 [c, e] [e]
In other words, it isn't clear how to represent the baskets. Thanks!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/flaviovdf/fpmc/issues/2#issuecomment-437018536, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAf08M4tETQqfakbeFuX2YLgy3CRaSpgks5utEQVgaJpZM4H1Hey
.


Blacklisted sites to consider adding

Here's my list of blacklisted sites to consider blocking by default.
www\.smbc-comics\.com; #commentarea

Yikes, took me a long time to add this, but at least you could block it yourself. Added it to the list now. Thanks for the heads up about which element to block.


remove_validate_delta

fixes:
https://github.com/freenet/freenet-core/issues/1243

compiles in my computer if combined with the stdlib PR.
current tests already broken by older PRs.
no more tests added.

You must probably update the stdlib submodule commit to see that it works, there is a new failing test otherwise

I just need to push again

Completed in #1274

painful to see such duplicated. I hope he forked me.

Your PR was broken and as a result we had a broken incompatible version which was not in sync with stdlib. I had to finish this up sorry, next time complete the changes in one go.

nice


Named parameter defaultValue is not defined for GeneratedRealColumn

Error when putting a default value for a real column

Thanks for the report, it's fixed on development.


(PlayMusic) Identify Artist / Song / Album / Genre of Music Play Request

There is currently not a skill that will allow a user to request to play a song / artist / album / or Genre of Music.  This is a typically needed use case for many device types.
Solution should allow for a user to state:
Play "Genre" music
Play "Song" by "Artist"
Play "Song"
Play "Album"
Play "Album" by "Artist"
The solution will need to provide disambiguation of multiple and return an Event much like the POI Find skill that would allow the skill to launch with context the music service / app of choice
Additional context
This skill is not to stream the music, but to find the appropriate information to then send to the host to allow music the correct music to be passed to a backend service API.

Replaced by #929


Crashes after performing autocrop and closing crop preview window

Describe the bug or Issue
A clear and concise description of what the bug is.
Step To Reproduce the Problem

Go to '...'
Click on '....'
Using '...' Encoder
'Avisynth or VapourSynth' Being Used
Filter(s) x is being Used.
Any Major External Files tied to the Error.
Post Both your Log file & AVS/VS Script.

Screenshots
If possible, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Windows 10 Build 1803]
Version [e.g. 1.9]

Additional context
Add any other Information or additional context that can be Helpful.
Please be Clear and as Detailed as Possible
Any Issues left unattended to for Length of Time will automatically be Closed Due to inactivity

Make sure your using the latest version, The first 1.9 had a small issue which has already been fixed.


build docs only on linux

copy to other platforms
patch dlang.org to build docs for a release
from prerelease project layout and disable
the clone/build targets so that create_dmd_release
remains the driver

The build script collapsed the other day and ceased collaboration with dlang.org's makefile.
In the last minute I noticed that linux 64-bit dmd was actually a 32-bit binary.
Unfortunately this isn't easy to fix as dlang.org insists on building dmd, druntime, and phobos itself.
Another problem with dlang.org is that the docs for a release need to be build like a prerelease.
I ended up patching dlang.org.
The other changes are throwing out plenty of unused code in the create_dmd_release.d script and there are a few smaller fixes.
I'm going to merge this right away in order to build 2.067.0-b3, but I'd be happy for any review.
Also build script experts are welcome to chime in.


Build failed on MacOS with asdf

If I install, build and run on MacOS with asdf-vm (a version manager), I face an error when start rails
Uncaught exception: dlopen(/Users/.../.asdf/installs/ruby/2.6.5/lib/ruby/gems/2.6.0/gems/pg_query-1.2.0/lib/pg_query/pg_query.bundle, 9): no suitable image found.  Did find:
	/Users/.../.asdf/installs/ruby/2.6.5/lib/ruby/gems/2.6.0/gems/pg_query-1.2.0/lib/pg_query/pg_query.bundle: mach-o, but built for simulator (not macOS)

@imcvampire Hmm, I can't reproduce this here (rbenv on MacOS)
Do you see the same problem in the new 2.0 release?

Closing this since no further updates - please let me know if this is still an issue on the most recent release.


Deposit assets from trade page

Figma discussion
We should remove the roadblock for users loading the app for the first time having to go to the assets page to deposit before the go to the trade page to swap.

6/28 sprint planning -
Waiting on design work @victortokaito

Is there a design you want implemented? I cant access the figma discussion. Otherwise I have some ideas

@milesz8 don't think we have a design for this yet. Or decided we want to do this.
We're currently undergoing a frontend redesign. May want to look into another issue for now.

One idea- what if when a user has a "from" asset selected with a positive balance, we can have the button change to deposit? They can click that, then the UI is locked until the tx completes, then the button says swap. Seems like a simple workflow that doesn't require another control @miles

9/29 meeting update -
We decided the best design would be to link to the assets page from the swap page if they have a 0 balance in Osmosis for the "from" asset. This would help inform users of the assets page, and would serve as a tutorial for how assets are deposited, withdrawn, and displayed.
It would work as follows from the swap page:

If a user has a 0 Osmosis balance for the asset in the "from" page, the swap button (or a link near it, design TBD) would show: "Deposit x ASSET from network" if they've approved the counterparty in Keplr, or "Deposit from network" otherwise
If they click the deposit button from the swap widget, we'd link to the assets page with query parameters in the URL to determine which transfer modal to have automatically opened.
User can carry out standard transfer from the transfer modal.
(Optional/stretch) when the transfer completes we'd keep the modal open and the "Transfer" button would be replaced with a "Back to Swap" button to let the user go back to the swap page with the query parameters populated to the relevant assets. (We've also considered having a link to Swap page in the success toast message)

'Begin a deposit from the swap page' Prototype

link to figma frame


Generate production table tooltips on_hover

Potentially annoying to do, so likely best done in select places. Rai says the latency in multiplayer is fine, too, so I should probably do it.
I should test the impact of this by just disabling all the tooltips and seeing what difference that makes.

This worked out great, cut refresh time of main interface in half by doing the core tooltips dynamically.


Fix nuspec namespace

Apparently there are multiple variants of the namespace for the nuspec file
https://docs.microsoft.com/en-us/nuget/schema/nuspec uses http://schemas.microsoft.com/packaging/2010/07/nuspec.xsd as example.
This might not be a complete list, but solves it for my projects at least.

Would you be able to add test cases for these?

Hmm, well there is already a test (still passing) - obviously it only tests for one variant of the namespace.
If I should follow the same pattern of test, I would have to create a new TestFiles folder complete with csproj and nuspec file (only changing the namespace). I think it might be a bit too much... or what do you think?

Agreed that's a little overkill. Thanks for the great additions!


Should we separate ERC4337 contracts deployment private key and paymaster server private key?

Currently, we use the same private key specified by users to deploy contracts (if needed) and also as the paymaster signing. Should we separate them?

Context: https://github.com/0xFableOrg/simple-op-stack-rollup/pull/15#discussion_r1328475861


Please add horizontal scroll bar to test tab

I was unable to find a previously logged issue or feature request.

The Function names of test cases are, by there very nature, often long-winded. The tree structure also tends to eat into the horizontal real-estate.

It would be really great if we could enable a horizontal scrollbar on the Testing Extension tab so we could see the function names.
This would also make the ux on the Testing Extension tab more in line with the other tabs like the Variable explorer, Call Stack and Breakpoint list.

I'm not planning to do this at the moment. We in general don't have horizontal scrollbars in our views.

This would also make the ux on the Testing Extension tab more in line with the other tabs like the Variable explorer, Call Stack and Breakpoint list.

None of these views get horizontal scrollbars

None of these views get horizontal scrollbars

The horizontal scrollbar is available if you have set the following setting to true - it's false by default. I believe it was introduced/changed in vscode v1.32
{
        "workbench.list.horizontalScrolling": true,
}

This is the screenshot from my ide:

It just would be great if the testing tab ran off the same setting.
Thanks

TIL


C++ refactoring: ak.concatenate

Thank you very much for fixing that (and finding a few untested parts of _broadcasting.py). The commit I just made was to remove some commented-out code, that's all. It's set to auto-merge when the tests pass again.

Thank you! Great! Happy to have this merged!

Woohoo! Thanks for this 😄


WIP: Upgrade Boost to 1.68.0

This PR moves us from Boost 1.65.1 to Boost 1.68.0. See the Boost release notes for specific info on modules updated/added between releases.
This PR also fixes the typo in CMakeLists.txt (which should alleviate the need for this change).
WIP: For now boost_date_time_parsers3.patch in deps/patches is being skipped, as it does not apply cleanly. I'm looking to see whether we can pass tests without it (which might imply the bug we were patching has been fixed in upstream Boost - hard to tell from the release notes).

@ylow -- there was some movement in the PR you put up for this stuff in boost years ago -- can you dig up the info for that?  Do you know if it got merged?

Thanks @ylow - I'll clean up the patch so it applies cleanly on 1.68.0.

@hoytak @ylow @nickjong Please review (whichever parts seem relevant to review, if any). I think this PR is now ready to go in; all tests pass locally and in CI.

@ylow Thanks for the review!


Add a collector to find and compare positions of two items in the stream

Sometimes it's interesting to check whether position of element A in the stream is closer to the start or not, comparing to the position of element B. This could be checked using special short-circuiting collector which returns whether (the first occurence of) A is first or B is first, or both are the same. It's an open question which return type should be used and how the absence of the elements should be tracked.

This could be implemented with stream.filter(x -> x.equals(A) || x.equals(B)).findFirst() -> result is either Optional.of(A) (A is first), Optional.of(B) (B is first) or Optional.empty() none found. Thus, it looks like no special library support is required.


add mixin for setting mdc-theme classes with props

The mixin should push CSS classes depended on set props e.g. secondary-bg

This mixin should be an alternative to create rules with SASS and mixins when you just want your component to have secondary color as background e.g. buttons..


using node HTTP module missing features

It seems that when one tries to use the node HTTP module req.setTimeout and req.abort() are undefined. This might be expected behavior except for the fact that the same bit of code works if run through browserify first. I have created a repo with the minimal code to reproduce the issue.
https://github.com/TechplexEngineer/meteor-http-browserify-repro/tree/master
What does meteor do with node-stubs that is different enough from browserify to cause these methods to nor work?
stacktrace:
Uncaught TypeError: req.setTimeout is not a function
    at Object.click button (main.js:49)
    at blaze.js:3775
    at Function.Template._withTemplateInstanceFunc (blaze.js:3744)
    at Blaze.View.<anonymous> (blaze.js:3774)
    at blaze.js:2617
    at Object.Blaze._withCurrentView (blaze.js:2271)
    at Blaze._DOMRange.<anonymous> (blaze.js:2616)
    at HTMLButtonElement.<anonymous> (blaze.js:863)
    at HTMLBodyElement.dispatch (jquery.js:4722)
    at HTMLBodyElement.elemData.handle (jquery.js:4390)

@benjamn this can be closed when 0.3.3 is published


[DVSim] Running with global sw_build_opts/sw_images causes a Python MemoryError failure on large regressions

Description
When adding a global sw_build_opts in the hjson simulation config file, large regression simulations won't start and eventually the entire run will fail due to a Python's MemoryError exception.
The root cause of the problem is a copy by reference of the global sw_build_opts to each of the Tests entries, which later copies itself over and over, causing large regression simulations to eventually run out of memory.
Another problem that arises is duplication of sw_build_opts, as each Tests entry gets the global sw_build_opts here, and then again here, although once the copy-by-reference problem is resolved it's not much of an issue.
An identical case can be made for sw_images list.
A possible solution is to deepcopy any iterable object required test entry fields (sw_build_opts/sw_images) and set them as the attribute of the Tests entry instead of just setting the reference of the global SimCfg.sw_build_opts as the attribute of the Tests object, and to treat both sw_build_opts/sw_images as a Python set instead of a list object.

This is a bit silly about our scripts, and probably not too hard to fix. Is there an easy way to reproduce the problem from here? I think I have an idea about what will fix things, but it would be nice to see it dying and then fixed by a change!

Wow, thanks for debugging this so carefully. I guess a super-stupid initial fix would be to add something like the following in create_tests as a work-around:
                    if global_val is not None and global_val != default_val:
                        if isinstance(global_val, list):
                            global_val = list(global_val)
                        setattr(test_obj, attr, global_val)

Clearly this isn't something that we should be committing to the repository! :-) But it will presumably avoid the current "append X to itself" behaviour.

This is currently blocking Nuvoton from running nightly regressions in their environment.

I think the "short-term bodge" is a very good idea. I'll file a PR in a few minutes. Honestly, this bit of the dvsim code is a right mess and I had grand plans to tidy it up a bit more dramatically. But that's a longer-term problem, I think.

I've wrapped the proposed change into PR #21885. I didn't include the Iterable check because it's a bit awkward (the Iterable type isn't something that's visible without an unusual import) and I think that calling deepcopy on an atom will waste a trivially small amount of space. Shout on the PR if I'm missing something.

Looks fine, thanks!


Document setting 'region' when sshing into EKS node

@GeorgianaElena made a lot of our docs shell commands be written in a way that directly references env vars in #2132, which makes it easier to copy paste snippets of code overall. This is especially relevant if you run many commands all referencing some cluster name, cluster region, or similar variable.
Since this is not a lengthy instruction with repeated use of the region variable, let's go for a merge!


Merge to master

Do we feel ready to move this to master to be able to properly deploy to next?

@kkellerlbl We need to have a user created with Read/Write access to the UJS database on Next. Does that Exist?

Yes, the ujs mongo user exists, and I've added a next njsw config file to Gitlab.  We'll need to do the same for appdev and production but we'll do that when we're ready to deploy.

I think its far enough along to give it a try.

Merging now so we don't get too many more commits on master.


Does it support Chinese mandarin ?

Does it support Chinese mandarin ?

how set the language?

Please describe your issue in detail.
The information you give is too limited. We have no idea what you want.

yes asr or tts? both of them support Chinese mandarin


Test the public repository for the comments

Can you read this from Pawan

yet another comment from pawan

Test 2

Test 3


testsuite hangs after test post_federated_query_*() on ppc64

When building oxigraph 0.4.0-alpha.7 on Debian, with patches 9c0b303, ed2b4e5 and 1a9f53f applied, the testsuite fails on an automated ppc64 build daemon, like this:
     Running `CARGO=/usr/bin/cargo CARGO_MANIFEST_DIR='/<<PKGBUILDDIR>>/cli' CARGO_PKG_AUTHORS='Tpt<EMAIL_ADDRESS>CARGO_PKG_DESCRIPTION='Oxigraph CLI tool and SPARQL HTTP server
' CARGO_PKG_HOMEPAGE='https://oxigraph.org/cli/' CARGO_PKG_LICENSE='MIT OR Apache-2.0' CARGO_PKG_LICENSE_FILE='' CARGO_PKG_NAME=oxigraph-cli CARGO_PKG_README=README.md CARGO_PKG_REPOSITORY='https://github.com/oxigraph/oxigraph/tree/main/cli' CARGO_PKG_RUST_VERSION=1.70 CARGO_PKG_VERSION=0.4.0-alpha.7 CARGO_PKG_VERSION_MAJOR=0 CARGO_PKG_VERSION_MINOR=4 CARGO_PKG_VERSION_PATCH=0 CARGO_PKG_VERSION_PRE=alpha.7 LD_LIBRARY_PATH='/<<PKGBUILDDIR>>/target/powerpc64le-unknown-linux-gnu/release/build/oxrocksdb-sys-b9b19a1ce9f6ad4e/out:/<<PKGBUILDDIR>>/target/powerpc64le-unknown-linux-gnu/release/deps:/<<PKGBUILDDIR>>/target/powerpc64le-unknown-linux-gnu/release:/usr/lib/rustlib/powerpc64le-unknown-linux-gnu/lib' /<<PKGBUILDDIR>>/target/powerpc64le-unknown-linux-gnu/release/deps/oxigraph-79bbfdf7ba9fe5d8 --skip test_backup --skip test_backward_compatibility --skip test_secondary`

running 47 tests
test tests::clap_debug ... ok
test tests::cli_convert_file ... ok
test tests::cli_convert_from_default_graph_to_named_graph ... ok
test tests::cli_convert_from_named_graph ... ok
test tests::cli_help ... ok
test tests::cli_ask_update_inline ... ok
test tests::cli_construct_update_stdin ... ok
test tests::cli_construct_query_stdin ... ok
test tests::cli_ask_query_inline ... ok
test tests::cli_backup ... ok
test tests::cli_load_and_dump_named_graph ... ok
test tests::get_bad_query ... ok
test tests::get_query ... ok
test tests::get_query_accept_bad ... ok
test tests::get_query_accept_explicit_priority ... ok
test tests::get_query_accept_good ... ok
test tests::get_query_accept_implicit_and_explicit_priority ... ok
test tests::get_query_accept_implicit_priority ... ok
test tests::get_query_accept_star ... ok
test tests::get_query_accept_substar ... ok
test tests::get_query_union_graph ... ok
test tests::get_query_union_graph_and_default_graph ... ok
test tests::get_query_union_graph_in_url_and_urlencoded ... ok
test tests::get_ui ... ok
test tests::get_without_query ... ok
test tests::graph_store_base_url ... ok
test tests::graph_store_lenient_bulk ... ok
test tests::graph_store_protocol ... ok
test tests::graph_store_url_normalization ... ok
test tests::post_bad_query ... ok
test tests::post_bad_update ... ok
test tests::post_dataset_file ... ok
test tests::cli_load_and_dump_dataset ... ok
test tests::cli_load_and_dump_with_format ... ok
test tests::post_query ... ok
test tests::post_unknown_query ... ok
test tests::post_unsupported_file ... ok
test tests::post_update ... ok
test tests::post_update_read_only ... ok
test tests::post_wrong_file ... ok
test tests::cli_load_from_stdin_and_dump_to_stdout ... ok
test tests::post_federated_query_wikidata ... ok
test tests::cli_load_gzip_dataset ... ok
test tests::cli_update_file ... ok
test tests::cli_select_query_file ... ok
test tests::cli_load_optimize_and_dump_graph ... ok
test tests::post_federated_query_dbpedia has been running for over 60 seconds
E: Build killed with signal TERM after 150 minutes of inactivity

Full build log is here
Debian build daemons are configured to block network access, which might be related. First I thought that a) ppc64 blocks network in a different way than build daemons for other Debian arches, and b) the call to a federated SERVICE, together caused the hang.
But then why did tests::post_federated_query_wikidata succeed and only tests::post_federated_query_dbpedia hangs?
Perhaps it is simply that tests::post_federated_query_dbpedia is the last test, and it is not really that test that hangs but the cleanup of the environment somehow?

It wasn't ppc64 but ppc64el. And seems it was a fluke, as it now succeeds again on ppc64el with seemingly no related changes.


32 bit screenshot compared on 64 bit

By default FBSnapshotVerifyView runs with FBSnapshotTestCaseDefaultSuffixes. The resulting orderedSet contains both "_32" and " _64" suffixes, in different order depending on the architecture. Is there a specific reason why both suffixes are added? I think you only want "_32" on a 32 bit device and "_64" on 64 bit device.
We run into issues with this when a test case fails. What happens is the following:
We have 32 bits images with different screenSize as the 64 bit images. When running a test it loops over the suffixes and starts with 64 bits comparison. When this succeeds the 32 bit comparison is skipped, because if comparisonSuccess || recordMode { break}. But when the test case fails it doesn't break the for loop, but continues with the 32 bits comparison which will always fail in our case because it compares 32 bits screenshot with 64 bit screenshots of different sizes. It will overwrite the images of the 64 bit comparison and we end up with a diff image containing images of different sizes.
So the second question I have is why break the for loop of suffixes on success but not on failure?
We can workaround the above issues by explicitly call FBSnapshotVerifyView  with suffixes option depending on the architecture of the test device, but it looks like it should be fixed in the framework itself.

Is this your case:
https://github.com/facebookarchive/ios-snapshot-test-case/pull/119#discussion-diff-41718978
?
@alanzeino I seen 3.0.0 tag. Is this a good moment to change this behaviour (and finally get rid of macros :P)?

Will this resolve the issue: https://github.com/PicnicSupermarket/ios-snapshot-test-case/commit/4f5730b76b8fbca12947844c60435e4ded5f007f ?

Since we don't have 32 bits images with different screenSize as the 64 bit images. So I pass a unified suffixes for both architecture:
    lazy var suffixes: NSOrderedSet = {
        let suffixes = NSOrderedSet(array: [])
        return FBSnapshotTestCaseIs64Bit() ? suffixes.reversed : suffixes.copy() as! NSOrderedSet
    }()

I'm with you on this @chunkyguy. It is not clear to me why both of the suffixes would always be attached.


[HOLD - Threads discussion] Display the workspace name in the notification for #announce in a similar way how it is displayed in the mail

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to web chrome. Create a workspace from User A account and invite User B to the workspace
On android , login with user B
From User’s A account on web , send some messages in #announce
Notice that on the Users’B notification, no workspace name is shown. Therefore, even though the message has been sent from multiple workspaces, it will be displayed the same. We can’t figure out the workspace in the notification until we click on it.
But when we receive the mail, it shows the workspace name on the subject well

Expected Result:
Display workspace name on the notification for #announce in a similar way to how it is shown on the mail
Actual Result:
Workspace name is not displayed on the notification for #announce
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.2.88-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
Issue reported by: @priya-zha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1679460032309629
View all open jobs on GitHub

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Clarifying what "New name" means exactly here https://expensify.slack.com/archives/C049HHMV9SM/p1679610701712469?thread_ts=1679460032.309629&cid=C049HHMV9SM

@MitchExpensify It's the display name of the user

Got it! Thanks, so I am able to reproduce this but I think we'll need to agree on how it should look exactly before opening it up to proposals

Asking internally here

I think we need to hold on the threads design here, switching to weekly

@Julesssss @MitchExpensify this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

IT'S HELD.

@Julesssss, @MitchExpensify Eep! 4 days overdue now. Issues have feelings too...

Held on threads design

Threads doc should be out today or tomorrow :)

Reducing the priority here as other notification issues are far more critical. Once the hold is removed, we'll be able to get this done soon.

Thread doc was updated and re-sent for reviews today 👁

Threads detailed section being updated right now

Still held for the moment

I guess we were holding to figure out what the thread notification title should be? Seems like we didn't really agree on anything here

Ok cool, I can circle back to raising the discussion in Slack after EC3! Thanks @Julesssss

Hoping to get to this soon

Circling back here and I think this is actually a feature request more than a bug. I also don't think this is something we need to work on right now so I'm closing!


feat(devtools): Use redux-devtools-extension if available

This PR enables the use of the redux-devtool-extension  if available, instead of using redux-devtools-log-monitor/redux-devtools-dock-monitor.
PR #337 goes in that direction, but is a bit more complicated then necessary and produces a linter error. Also, it does not handle the "open devtools in new window" option correctly (I think one would expect that it to open redux-devtools-extension in a new window, not redux-devtools-log-monitor).
In redux-devtools-extension there is an option to open the extension programmatically (window.devToolsExtension.open()) which results in this:

This is exactly the "npm run dev:nw" behavior, so if redux-devtools-extension is available, we'll use it.
The README should probably be updated too to reflect these changes.

Thanks for this, :+1: for getting it merged in, but I think that if the extension exists then we should always use it. Thoughts?

It is always being used if it's available. We just open it explicitly (in a new window) only if dev:nw is active. It can always be reached here: .
Does that help?

That clears it up, thank you. Driving home this afternoon, will try to get to this in the evening.

:+1: thank you, merged.


Handle error for missing config

Changes

update require statement in example
throw error is missing config
fix a typo in a method api name
bump a minor version to v0.1.0

👍

👍

Published to npm v0.1.0


Error when running package in windows

I am trying to start with this starter kit but when running 'npm run package', I get this error. `

@ package C:\Users\msarc\coding\puppeteer\project
npm run package-prepare && cp chrome/headless_shell.tar.gz dist && cd dist && zip -rq ../package.zip .

@ package-prepare C:\Users\msarc\coding\puppeteer\project
npm run lint && npm run babel && cp -r package.json dist && cd dist && PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=1 npm instal  l --production

@ lint C:\Users\msarc\coding\puppeteer\project
eslint src

@ babel C:\Users\msarc\coding\puppeteer\project
rm -rf dist && mkdir dist && ./node_modules/.bin/babel src --out-dir dist

'.' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! @ babel: rm -rf dist && mkdir dist && ./node_modules/.bin/babel src --out-dir dist
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the @ babel script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\msarc\AppData\Roaming\npm-cache_logs\2017-11-06T20_13_55_486Z-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! @ package-prepare: npm run lint && npm run babel && cp -r package.json dist && cd dist && PUPPETEER_SKIP_CHR  OMIUM_DOWNLOAD=1 npm install --production
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the @ package-prepare script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\msarc\AppData\Roaming\npm-cache_logs\2017-11-06T20_13_55_607Z-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! @ package: npm run package-prepare && cp chrome/headless_shell.tar.gz dist && cd dist && zip -rq ../package.  zip .
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the @ package script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\msarc\AppData\Roaming\npm-cache_logs\2017-11-06T20_13_55_667Z-debug.log
`
How can I fix this error and use this starter kit?

This software only works on Linux.


How to implement residual connections in gcn?

Hi  @tkipf . I have some confusion about residual connection in GCN.
def forward(self, x, adj):
        # x size: [2708, 1433]
        # adj size: [2708, 2708]
        x = F.relu(self.gc1(x, adj))   # [2708, 16]
        ......

input size: [2708, 1433] , but first layer output's size is [2708, 16]

If I implement residual like equation above.
x = F.relu(self.gc1(x, adj))  + x

Error!  size miss match.

the output of layer 'self.gc1()' is [*，16] what ever the input is

@wutaiqiang   so，  how to  implement residual  connection， when inputs feature dimension[, 1433] not equal  to self.gc1() output size [, 16] ?


Batch: add feature to flag empty rows as missing and drop empty variables

This can be used to reduce the output for larger studies that share variable lists through batches and waves but do not have everything populated for specific setups.

implemented as hide_empty_items()


make: use Os optimization

I think using -Os  is better with this microkernel because of caching, if we have a small kernel (currently with clang we have a 300kb kernel and 517kb with gcc ) it will be easier for the cpu to cache it.  (+ Now the kernel is super fast)

with -Os (clang)
make run -> entry.c line 40 : ~0.50s
without -Os: ~0.55s
so with -Os we have about 10% improvment


Status of preconditioning

Hi,
I would like to see status if preconditioning it running or not. I have a script today with Domoticz, but I like to use HA instead. I reading EV_PRECONDITIONING_MODE form vehicleStatus. There is also EV_PRECONDITION_REMAINING_RUNTIME_MINUTES that can be useful.
That should be great to have 😃

You already have this data available as attributes of the sensor.my_car_all_info entity. You can easily create template sensors from the sensor attributes, see example here
Here are example on the attributes that you might be interested in, copy paste this in the Developer Tools > Template to see the values.
{{ state_attr('sensor.my_car_all_info', 'ev status').evPreconditionOperatingStatus }}
{{ state_attr("sensor.my_car_all_info", 'ev status').evPreconditionPrioritySetting }}
{{ state_attr("sensor.my_car_all_info", 'ev status').evPreconditionRemainingRuntimeMinutes }}
{{ state_attr("sensor.my_car_all_info", 'ev status').evPreconditioningMode }}

I actually missed this data my self, so I got inspired from your question and added two template sensors that show the precondition status and remaining time. Sharing the code for more inspiration! 😉
template:
  - sensor:
      - unique_id: my_car_precondition_remaining_runtime_minutes
        device_class: duration
        unit_of_measurement: min
        attributes:
          friendly_name: "Precondition Remaining Runtime Minutes"
        # we need to handle that remaining runtime minutes is not reset if precondition is manually switched off
        state: >-
          {% set evStatus = state_attr('sensor.my_car_all_info', 'ev status') %}

          {% if is_state('binary_sensor.template_my_car_precondition_operating_status', 'on') and evStatus != None and evStatus != 'unknown' %}
            {{ evStatus.evPreconditionRemainingRuntimeMinutes | int }}
          {% else %}
            0
          {% endif %}
  - binary_sensor:
      - unique_id: my_car_precondition_operating_status
        device_class: running
        attributes:
          friendly_name: "Precondition Operating Status"
        state: >-
          {% set evStatus = state_attr('sensor.my_car_all_info', 'ev status') %}

          {% if evStatus == None or evStatus == 'unknown' %}
            off
          {% elif evStatus.evPreconditionOperatingStatus == 'OFF' %}
            off
          {% else %}
            on
          {% endif %}
        icon: >-
          {% set evStatus = state_attr('sensor.my_car_all_info', 'ev status') %}

          {% if evStatus == None or evStatus == 'unknown' %}
            mdi:fan-alert
          {% elif evStatus.evPreconditionOperatingStatus == 'OFF' %}
            mdi:fan-off
          {% else %}
            mdi:fan
          {% endif %}

Showing the template sensors when precondition switched off

Showing the template sensors when precondition switched on

Caveat

I choose to use binary_sensor (on/off) for precondition status, but the status actually got 3 states: 1) PRECLIM (on), OFF and STARTUP (starting). I did not bother to reflect all three states, but only on/off
the remaining runtime minutes is not updated according to the scan interval (default to 5 minutes, can be changed in the integration Configuration, but not sure about the side effect of increasing this update)

@msp1974 Im very new to the jlrpy, but do you think we could poll for precondition operating status and precondition remaining minutes more frequently, without updating the health status for complete vehicle? Or maybe subscribe for changes from the JLRs server? I know you have a section about the Health Status Update in README, but wanted to get your input as well.

Hi,
In the script I have been using the last three years I have used get_status every minute.
This scripted checked my Domoticz if the precondition should be on or off. I then checked EV_PRECONDITIONING_MODE if it was INACTIVE or IMMEDIATE.
But now I try to do everything I have in Domoticz in HA instread.
It is very nice in the summer to have the car at cooled all the time when we are at the beach 😊
Med vänlig hälsning
Peter Sporrsäter | +46 707-38 03 92 |<EMAIL_ADDRESS>Your needs. We please it.
PLS-IT AB
Stora Åvägen 21 | SE-436 34 Askim | SWEDEN
Websida: www.pls-it.sehttp://www.pls-it.se/
SAVE PAPER - THINK BEFORE YOU PRINT
Från: Ismar Slomic @.>
Skickat: den 9 januari 2023 14:27
Till: msp1974/homeassistant-jlrincontrol @.>
Kopia: Peter Sporrsäter @.>; Author @.>
Ämne: Re: [msp1974/homeassistant-jlrincontrol] Status of preconditioning (Issue #86)
@msp1974https://github.com/msp1974 Im very new to the jlrpy, but do you think we could poll for precondition operating status and precondition remaining minutes more frequently, without updating the health status for complete vehicle? Or maybe subscribe for changes from the JLRs server? I know you have a section about the Health Status Updatehttps://github.com/msp1974/homeassistant-jlrincontrol#health-status-update in README, but wanted to get your input as well.
—
Reply to this email directly, view it on GitHubhttps://github.com/msp1974/homeassistant-jlrincontrol/issues/86#issuecomment-1375626528, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A5DC6K2WVLKKJIZG44ZM2RDWRQG37ANCNFSM6AAAAAATTKJR3Y.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

I have not had any problem. But I have an extra data-SIM.
But if you like to have charging speed you need to do a full refresh.
Med vänliga hälsningar
Peter Sporrsäter | +46 707-38 03 92tel:+46 707-38 03 92 |<EMAIL_ADDRESS>PLS-IT AB
Stora Åvägen 21 | SE-436 34 Askim | SWEDEN
Websida: www.pls-it.sehttp://www.pls-it.se/
SAVE PAPER - THINK BEFORE YOU PRINT
9 jan. 2023 kl. 15:14 skrev Mark Parker @.***>:

@ismarslomichttps://github.com/ismarslomic - the main issue is how frequently the car updates to the JLR servers. The jlrpy only reads from the servers. Therefore in order to get more frequent updates of actual vehicle status you will need to force the car to update with the Health Status Update. I am not sure if JLR will block this due to data use on their in car sim.
—
Reply to this email directly, view it on GitHubhttps://github.com/msp1974/homeassistant-jlrincontrol/issues/86#issuecomment-1375687837, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A5DC6K3LZ5FLGPZHBMCT5GLWRQMP3ANCNFSM6AAAAAATTKJR3Y.
You are receiving this because you authored the thread.Message ID: @.***>

Alright! I think we can close this issue, as your initial question has been answered, @MrSoluz. I will however include the template sensors I shared in this issue to the recipes so other users can easily find it later on.


[Dashboard] Remove  easy mode

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR focuses on replacing all instances of the Box component with the div element across multiple files in the dashboard application, likely for consistency or performance improvements.
Detailed summary

Replaced Box with div in PermissionsTable.tsx
Replaced Box with div in LatestEvents.tsx
Replaced Box with div in templates.tsx
Replaced Box with div in detail-page.tsx
Replaced Box with div in contract-publish-form/index.tsx
Replaced Box with div in custom-factory.tsx
Replaced Box with div in metadata.tsx
Replaced Box with div in Solutions.tsx
Replaced Box with div in MiniPlayground.tsx
Replaced Box with div in FormItem.tsx
Replaced Box with div in DevRelEvent.tsx
Replaced Box with div in shared-metadata-form.tsx
Replaced Box with div in update-metadata-form.tsx
Replaced Box with div in external-links-fieldset.tsx
Replaced Box with div in events-feed.tsx
Replaced Box with div in lazy-mint-form.tsx
Replaced Box with div in mint-form.tsx
Replaced Box with div in page.tsx
Replaced Box with div in landing-fieldset.tsx
Replaced Box with div in PlanCard.tsx
Replaced Box with div in CodeBlock/index.tsx
Replaced Box with div in SponsorshipPolicies/index.tsx

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4786  👈
#4785
#4784
#4765
#4750
#4749
#4748
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @kien-ngo and the rest of your teammates on  Graphite


read file has unclear error messages for bad inputs

read file needs better error handling for options.
If the filename is supplied without quotes, we give out this bizarre error:
juttle -e "read file -file /tmp/nosuch"
Invalid flags supplied to RegExp constructor 'nosuch'

If the filename is supplied properly but does not exist, then we give the right error, albeit verbose - any chance for shortening this up?
juttle -e "read file -file '/tmp/nosuch'"
Error: internal error Error: ENOENT: no such file or directory, open '/tmp/nosuch' { [RuntimeError: Error: internal error Error: ENOENT: no such file or directory, open '/tmp/nosuch']
  message: 'Error: internal error Error: ENOENT: no such file or directory, open \'/tmp/nosuch\'',
  code: 'RT-INTERNAL-ERROR',
  info:
   { error: 'Error: ENOENT: no such file or directory, open \'/tmp/nosuch\'',
     location: { filename: 'main', start: [Object], end: [Object] },
     procName: 'read-file' } }
└───────────┴───────────┘

If the input file is not in JSON format, we should say so, rather than complaining about its first character (/tmp/hello here contains HELLO):
juttle -e "read file -file '/tmp/hello'"
Error: internal error SyntaxError: Unexpected token H { [RuntimeError: Error: internal error SyntaxError: Unexpected token H]
  message: 'Error: internal error SyntaxError: Unexpected token H',
  code: 'RT-INTERNAL-ERROR',
  info:
   { error: 'SyntaxError: Unexpected token H',
     location: { filename: 'main', start: [Object], end: [Object] },
     procName: 'read-file' } }
└───────────┴───────────┘

Additionally, I would welcome a change from read file -file to read file -path, it is clearer and would prevent misspelling / misremembering like read -file file which my fingers keep typing out. The outcome of that is correct, but also not greatly intuitive:
juttle> read -file file '/tmp/foo'
<input>:1:6:
   1:read -file file '/tmp/foo'
          ^
Expected identifier but "-" found. (JUTTLE-SYNTAX-ERROR-WITH-EXPECTED)

The first issue you mentioned has nothing to do with the file adapter per se. If you don't properly quote a file path, the grammar will try to interpret it in confusing ways:
juttle> emit | put x = /foo/bar
Invalid flags supplied to RegExp constructor 'bar'
Unhandled rejection SyntaxError: Invalid flags supplied to RegExp constructor 'bar'
    at new RegExp (native)
    at main_1 (eval at <anonymous> (/Users/demmer/node_modules/juttle/lib/compiler/index.js:32:23), <anonymous>:24:112)
    at eval (eval at <anonymous> (/Users/demmer/node_modules/juttle/lib/compiler/index.js:32:23), <anonymous>:33:75)
    at stages.flowgraph (/Users/demmer/node_modules/juttle/lib/compiler/index.js:34:9)
    at /Users/demmer/node_modules/juttle/lib/compiler/index.js:91:20
    at tryCatcher (/Users/demmer/node_modules/juttle/node_modules/bluebird/js/main/util.js:26:23)
    at Promise._settlePromiseFromHandler (/Users/demmer/node_modules/juttle/node_modules/bluebird/js/main/promise.js:507:31)
    at Promise._settlePromiseAt (/Users/demmer/node_modules/juttle/node_modules/bluebird/js/main/promise.js:581:18)
    at Promise._settlePromises (/Users/demmer/node_modules/juttle/node_modules/bluebird/js/main/promise.js:697:14)
    at Async._drainQueue (/Users/demmer/node_modules/juttle/node_modules/bluebird/js/main/async.js:123:16)
    at Async._drainQueues (/Users/demmer/node_modules/juttle/node_modules/bluebird/js/main/async.js:133:10)
    at Immediate.Async.drainQueues [as _onImmediate] (/Users/demmer/node_modules/juttle/node_modules/bluebird/js/main/async.js:15:14)
    at processImmediate [as _immediateCallback] (timers.js:383:17)
juttle> emit | put x = /foo/bar.juttle
<input>:1:16:
   1:emit | put x = /foo/bar.juttle
                    ^^^^^^^^^^^^^^^
Error: variable 'juttle' is not exported by module  code (RT-NOT-EXPORTED)

Obviously in the case of the file adapter you're more likely to fall into this problem, but I don't know what we could do about this. @dmajda do you have any bright ideas?
The other three comments including the suggestion to rename -file to -path are totally valid and should be fixed.

This is a multitude of mostly unrelated issues:

The “bizarre error” with unquoted file name. This is caused by:

/tmp/nosuch being considered a regexp literal.
Not validating regexp flags in the parser, but passing them through to JavaScript, which blows up with its own error.

I don’t think it makes sense to make the parsing more strict (I just checked and at least JavaScript, Ruby, and Perl also consider /tmp/nosuch a regexp literal) but we can definitely do our own check on the flags with a better error message. The fact that we don’t do that is a clear bug.

 This is just missing error handling in the file backend.

The unhelpful syntax error. This is just missing error handling in the file backend. Note this bug may also exist in other places where we parse formatted input.

Unintuitive error message in case of read -file file '/tmp/foo'. I think with a simple grammar change we can have a message like “Expected adapter name but "-" found.”, which is much more instructive.

Change -path to -file. Both variants are fine with me, so not opposing to the change.

@dmehra Could you please file a separate issue for each of these?


Support wildcard on mainFile

Support main_file = "src/*.cr" as mainFile requiring all in a temp file crystal tool implementations -C src/foo/bar.cr:line:col /path/for/tmp.cr

A good recommendation by @Papierkorb
Also see: https://github.com/crystal-lang-tools/scry/issues/29

I'm working on something to fix this (at LSP side), I'll publish something very soon :smile:

The underlying crystal tools don't support using wildcards for the main file, so there's nothing to do here. As well, the extension is moving away from using a configuration to set the main file


js plugin: Interfaces that extend imported types throw errors during transform calls that use plugins

Describe the bug
I'm seeing the same issue as the person in the comments in https://github.com/swc-project/swc/issues/3827. It was requested that we make a new issue, so here's a new issue! 😄
If I implement an empty plugin and use it within a transformFile call, I get an error:
Error: failed to deserialize Program

Caused by:
    missing field `expression`

However, things work if I downgrade to version 1.2.143.
Input code
import React from 'react';
import { Button } from 'somewhere';

export interface NewButtonProps extends React.ComponentProps<typeof Button> {}

Config
No response
Playground link
No response
Expected behavior
No error.
Actual behavior
No response
Version
1.2.182
Additional context
Example code:
class Plugin extends Visitor {
  visitTsType(e) {
    // the default behavior is to throw an error here, so we override it
    return e;
  }
}

const compiled = await transformFile(filePath, {
  jsc: {
    target: 'es2019',
    externalHelpers: true,
    parser: {
      syntax: 'typescript',
      tsx: true,
      dynamicImport: true,
    },
  },
  plugin: (m) => new Plugin().visitProgram(m),
  sourceMaps: true,
});

I'd also love to know if it is intended that visitTsType is not implemented and if not what the intended implementation should be.

Closing as this is bit outdated - please use new wasm plugin instead. We are open to accept fix for the existing visitor but core team may not have time to look into.

Sad to see this - use cases other than transpiling include AST traversal. Maybe we can reopen this in the future if there’s sufficient demand?

Just want to clarify something. By “demand” - I mean it in an economic sense (like in “supply and demand”) not as a command or something. Apologies for any confusion.

I'm not confused at all. I mean the number of users want it does not matter at all.


RFE: Consider using GORM to support databases other than Postgres

Happy New Year! And thanks for getting Commento development moving again! 😄
Tying Commento to Postgres limits how many people can use it. In principle, an ORM like GORM could allow a far wider audience to use Commento by making other databases an option (MySQL and SQLite in particular).
I've no idea whether anyone might be interested in doing the work to implement this, but I thought I'd file this issue on the off-chance.

Happy new year! 😄
I completely agree we should move to an ORM, this would definitely help increase where commento can run!
Currently, the only things holding this back from the original commento are:

Reliance on pgsql features like triggers
A folder full of migrations to help upgrade older versions of commento
Hardcoded SQL statements in the /api folder

I will think about how best to address all of these!

Happy new year! 😄
I completely agree we should move to an ORM, this would definitely help increase where commento can run!
Currently, the only things holding this back from the original commento are:

Reliance on pgsql features like triggers
A folder full of migrations to help upgrade older versions of commento
Hardcoded SQL statements in the /api folder

I will think about how best to address all of these!

Awesome, thank you for outlining the work and the willingness to look into the various pieces. I understand these are all significant pieces of work however, so no pressure!

Awesome, thank you for outlining the work and the willingness to look into the various pieces. I understand these are all significant pieces of work however, so no pressure!

I too would love to help in some way to support mysql. All my other services use MySQL, they also support postgresql, all except one.  And the chances of Ghost support postgresql are .... not likely, especially since it looks like they dropped support for it a few versions back.

I would love to see this. SQLite support in particular would open up hosting options a lot; would be much simpler to host serverless, etc...

I guess step one would be to choose an ORM?

Possibly, although it may make more sense to wait until there's someone willing to work on this, since they may have a strong opinion on which ORM they want to use. :-)


implement gt config commands

Includes the following:
git town alias (true | false)
git town config [--reset | --setup]
git town hack-push-flag [(true | false)]
git town help
git town install-fish-autocompletion
git town main-branch [<branch_name>]
git town perennial-branches [(--add | --remove) <branch_name>]
git town pull-branch-strategy [(rebase | merge)]

Planning on doing each in there own PR, but each should be pretty small

Can this be closed?


[AspireDeployment] azd vs-server should support authentication with clients

The azd vs-server should support authentication (Azure Entra) with clients.
As for today, it is assumed clients starting, sending requests to the azd vs-server have already authenticated (azd auth login) to the azd process. However, apart for the implied security reasons, that is not always true, resulting in an unexpected experience for users that are not aware of such requirement.
There are various aspects to consider:
a. Should the authentication token be passed on server startup or with every request?
b. If a user changes account/subscription, how is the new auth token be passed to the server? Should it be restarted?
c. How should authentication errors be reported back for a request?
Signatures or return type/data would be changed, so VS IDE should be informed to adjust current API contracts.

We should solve this the same way we solve the problem with VS Code - where we provide a way for VS to expose a server that azd uses for authentication information.  This will be the cleanest from the azd side and also give the most flexibility to the caller.  I will write up how this works end to end in VS Code, but you can also take a peak at https://github.com/Azure/azure-dev/pull/2541 in the interim to see the PR that added the feature and how it works.

@ellismg Let's talk about this bug in our next Aspire meeting.
After discussing it with @abpiskunov, we think it'd be easier if we pass a token to the vs-server rather than spawning another process (though it's doable) but adds more complexity to the scenario than what it might be solved by just passing in a new parameter.

After discussing it with @abpiskunov, we think it'd be easier if we pass a token to the vs-server rather than spawning another process (though it's doable) but adds more complexity to the scenario than what it might be solved by just passing in a new parameter.

The problem is you cannot just pass in a token - the token is bound to the specific service today - and over time POP tokens will mean that there is a unique token per request.  We need a way for azd to request a token from VS when it needs it, presenting to VS all the information that it has as part of the token request.  Passing a fixed value will not work long term.

@ellismg
Understood. Let's better discuss the whole scenario in our next meeting.
I think we're missing some of the aspects you point out.

@anvillan - I wrote up how the external authentication stuff works today and checked in in here: https://github.com/Azure/azure-dev/blob/main/cli/azd/docs/external-authentication.md.  You can follow this protocol without any changes to azd itself, all of the code is checked in and works today.
On the call, we discussed how you might want a different strategy vs an HTTP 1.1 Web Server, and instead would like to pass the connection information to InitializeAsync and that you would rather we speak JSON-RPC 2.0 over websockets instead of doing a POST over HTTP 1.1.
I'm happy to do that, but I need to understand the changes on your end.  I think it looks like the following:

Add a new argument to InitializeAsync which contains the connection information.  Would it be possible for you all to pass a fully qualified URI like ws://<IP_ADDRESS>:12345/some/optional/path and we just connect to that and start speaking our agreed on protocol, or will it just be the path?
If we are using JSON-RPC here - What's the name of the RPC method you are exposing, and the arguments?  It would be ideal if it looked something like:

Task<AccessToken> GetTokenAsync(string[] scopes, GetTokenOptions options, CancellationToken cancellationToken)
Internally, the implementation of your method would end up calling GetToken on whatever Azure SDK credential you have and handle marshalling the result back via the RPC.
Let me know what makes it easiest from the VS side to integrate, and we can make whatever changes on our end to start speaking the new protocol.

Thanks, @anvillan!

Thanks for the details. Let's discuss it in our meeting.

I had to miss today's meeting due to some personal issues - I'll set up some time with you and @abpiskunov to discuss.

Tirage update:

Scope is big on VS side to integrate with the external authentication flow in Azd.
@anvillan to post the assumptions and follow-up questions to @ellismg @weikanglim
Not a must-fix item for VS 17.10 Preview 2. Moving to next milestone.

@anvillan @weikanglim @abpiskunov @ellismg


Research: collector (post) processor for mesmer metrics attributes

An Opentelemetry Collector can be extended with a "processor": https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/processor/resourceprocessor/README.md. Such a processor can add/remove/modify attributes right before the metric data is exported to a backend, eg. a prometheus instance.
We can sometimes have attributes that can have a high cardinality, which seems to be problematic in the backend. For example, a request path attribute of an Akka Http request can be infinite:

foo/bar/1
foo/bar/2
foo/bar/3
foo/bar/4
...

We could aggregate such paths to one path attribute value: foo/bar/{id}.
To do

investigate if aggregating path attribute in the processor is actually doable and a good idea
see if it's easier than pre-aggregating the paths in mesmer's otel extension
what are the other pros/cons of using the processors?
which approach is better? pre-aggregation (in the otel extension) or the processors (in the collector)? Does it make sense to implement both approaches?

@adamgadomski I'm removing it from the scope of 0.8.0. We don't need this right now. We should focus on ZIO instead. Please reinstate the issue in the 0.8.0 milestone if you think otherwise.


HHVM support

What is the reason of dropping HHVM support? It was small just simple hack.

3 strikes. It was a hack that would have been required yet again to fix https://travis-ci.org/leafo/scssphp/jobs/90411962

Any chance to bring HHVM support back?

Maybe when https://github.com/facebook/hhvm/issues/5457 is fixed.


I can't use it in react

node_modules/tres-dxf-loader/dist/tres-dxf-loader.js' has type "any" implicitly.
Try "npm i --save-dev @types/tres-dxf-loader" if it exists or add a new declaration file (.d.ts) that includes "declare module 'tres-dxf-loader';".ts( 7016)
using
"three": "^0.160.0",
"three-dxf-loader": "^3.2.0",

where you able to solve it?


Go back to main menu after selecting chain

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #7883
[ ] Requires deployment <snek/rubick/worker>

Did your issue had any of the "$" label on it?

[ ] My DOT address: Payout

Copilot Summary

🤖 Generated by Copilot at ca99f20
Improved mobile navigation by closing the expandable section on page change. Added a route constant and a watch function to MobileExpandableSection.vue.

🤖 Generated by Copilot at ca99f20

Route to the dark abyss
Watch for the demons' kiss
Close the gate of mobile hell
Escape the expandable spell

too "side-effecty"
I would like to see a more direct solution where NavbarChainOptions emits an event that closes MobileExpandableSection

too "side-effecty"
I would like to see a more direct solution where NavbarChainOptions emits(发出) an event that closes MobileExpandableSection

I got it, I just think it's too redundant to use event, and have switched to implementing it with emit

tbh I am not quite sure how it works @shengxinjing could you explain it to me, please specifically(专门) the v-slot part

Hello, the event propagation here is a bit complex. It requires passing an operation from the slot component NavbarExploreOptions to the parent component MobileExpandableSection in order to trigger the close method.
Firstly, a new event called "closeMobileSubMenu" needs to be added to NavbarExploreOptions. This event will trigger the "@closeMobileSubMenu" event. Using v-slot as a bridge, the onCloseMobileSubMenu function can be passed to closeMobileSubMenu.
The event within v-slot will trigger the function "@closeMobileSubMenu" on the slot component within the parent component.
It's a bit convoluted, and I apologize if my explanation isn't clear. If there are any similar issues involving Vue internals, you can assign them to me for further investigation.

my english is not very good, you can see the document
https://vuejs.org/guide/components/slots.html#scoped-slots

thanks @shengxinjing !


open in editor support

sourcegraph config sample:
{
  "extensions": {
    "sourcegraph/open-in-editor": true,
  },
  "openineditor.editor" : "idea",
  "openineditor.basePath": "/Users/USERNAME/Documents/"
}

It's hard to implementation


runtime(doc): Add tags about lazyloading of menu

Close #14672
Also add the g: prefix.

thanks, makes senes!


Create 0000-01-02-DarthVaderx99.md

created a branch, created a file and made a commit, and opened a pull request

created a branch, created a file and made a commit, and opened a pull request


Removing redundant call to get_social_relationships in xbox_social_relationship_result::get_next

@jasonsandlin,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


team feedback

hi @vvoo21,
you have a great job so far, we really appreciate your work ㊗ 🎉 🎉 ,
your project will look more professional if you put more indentation in your code and also have your body code split in a header, section, and footer.
have a happy coding day

Thank you very much. I have made the required suggestions.😃


About the code of model training on CUB，where can I find it ？

Download from here -> cub.zip
This notebook shows the code and running results of the CUB experimental in the paper. But because it relies on the old version gbn implementation, there may be some minor problems in the current code base, but it should not be difficult to fix by yourself.


Cannot fully change outputDir

Im using the generateSwaggerCode command. Instead of generating the models in the build directory, I want them generated in src/gen/java/com/cb/models
So i have changed build.gradle:
swaggerSources {     petstore {         inputFile = file('petstore.yaml')         code {             language = 'java'             components = ['models']             configFile = file('config.json')             outputDir = file('src/gen/java/')         }     } }
And config.json is:
{   "dateLibrary": "java8",   "hideGenerationTimestamp": true,   "modelPackage": "com.cb.models" }
However, the models end up at src/gen/java/src/main/java/com/cb/Pet.java
I cannot figure out how to get rid of src/main/java in the path. Can this be done?

You need to generate sources into a directory and move them to src/gen/java. This is due to limitation of swagger-codegen.


Set GitHub homepage to plugin site when accepting a hosting request

In #3503, https://github.com/jenkinsci/redhat-dependency-analytics-plugin was created without a GitHub homepage set. Following existing practice in popular plugins, I set the GitHub homepage to https://plugins.jenkins.io/redhat-dependency-analytics/ but it would be nice if this was done automatically when accepting the hosting request.

Why was https://github.com/jenkinsci/harbor-plugin created without a link to https://plugins.jenkins.io/harbor/?

because it is not in the automation.

Should it be done manually in the meantime?

Not in my opinion, anything that is expected to be done should be in the automation.
I have no expectation for hosting team members to be doing things manually separately.

I would expect it to be done for consistency with existing hosted repositories, regardless of whether it is currently covered by automation.

I would expect it to be done for consistency with existing hosted repositories, regardless of whether it is currently covered by automation.

You are welcome to your expectations, but it is not a requirement of the hosting team.
The requirements for what happens when hosting a plugin are codified, not in wish-lists.

The requirements for what happens when hosting a plugin are codified

Thanks Tim, I think this should be codified as a requirement so that newly-hosted repositories are consistent with existing ones.


Update build.ts

fix typo

Thank you for the contribution.  However, the file in question no longer exists within the current code base.


ESM modules are not scope hoisted because of global polyfilling

Bug report
I found the problem why treeshaking still does not work and why esm gives bigger bundles.
https://twitter.com/IAmTrySound/status/1147174216095211520
What is the current behavior?
This piece of code leads to wrapping whole module and all its dependencies with webpack module
var global$1 = (typeof global !== "undefined" ? global :
            typeof self !== "undefined" ? self :
            typeof window !== "undefined" ? window : {});

/* WEBPACK VAR INJECTION */(function(global) {
...
/* WEBPACK VAR INJECTION */}.call(this, __webpack_require__(14)))

If the current behavior is a bug, please provide the steps to reproduce.
bundle this in production mode
import jss from 'jss';

console.log(jss)

var global$1 = (typeof global !== "undefined" ? global :
            typeof self !== "undefined" ? self :
            typeof window !== "undefined" ? window : {});

This leads to larger bundles and global warming.
What is the expected behavior?

Other relevant information:
webpack version:  4.35.2
Node.js version:
Operating System:
Additional tools:

This is fixed in the next branch (webpack 5)

Fixed in webpack@5, if somebody will face with problem feel free to open new issue, thanks for using webpack


OSA [1491486]: Cinder API cannot show more than 1000 rows

What version (commit) do you encounter this issue on?
Master
What type of issue is this (bug, feature, documentation, regression)?
bug
What is the expected behavior?
See https://bugs.launchpad.net/openstack-ansible/+bug/1491486
What is actually happening?
See https://bugs.launchpad.net/openstack-ansible/+bug/1491486
What were you attempting to do when you encountered this error (give as much detail as possible)?
See https://bugs.launchpad.net/openstack-ansible/+bug/1491486
How can we reproduce this issue?
See https://bugs.launchpad.net/openstack-ansible/+bug/1491486
How can we verify your issue is addressed?
See https://bugs.launchpad.net/openstack-ansible/+bug/1491486
If this is a large issue, what can be done immediately to alleviate the issue while working on a longer-term solution?
See https://bugs.launchpad.net/openstack-ansible/+bug/1491486
What is your evaluation on the importance of this issue, and why?
See https://bugs.launchpad.net/openstack-ansible/+bug/1491486

This section is completed during the issue resolution process. If you are reporting an issue, please do not modify the section below.
Which branches are impacted by this issue?
Remove branches that are not impacted.
For each pull request, update the branch line to reference it.

[ ] master
[ ] liberty-12.2
[ ] kilo

Upgrade impact. Consider the impact on moving between major, minor and patch releases.

[ ] This change does not impact upgrades
[ ] This change addresses a potential upgrade impact
[ ] The upgrade impact is being tracked by another issue.

The issue tracking the upgrade impact is:
Testing impact

[ ] An issue/pull-request has been create or exists to track the testing gap, and is referenced below
[ ] This change includes additional testing
[ ] This change is tested by the existing setup

Additional testing links:
Release note

[ ] This change includes a release note.
[ ] This change does not require a release note.

Doc Impact
Does your change require documentation?

[ ] Yes, and I submitted an issue to Docs issues
[ ] Yes, and I contributed a documentation patch to docs-rpc
[ ] No documentation required for this change

Additional documentation links (please add links if you have ticked yes):

We will need more details so that we can start tracking this issue.
cc: @BjoernT @git-harry

@git-harry : The upstream bug is marked as "invalid" for Kilo and Trunk and "won't fix" for Juno with no mention of the newer branches. (It was submitted about 10 months ago) What should we do with this issue?

@nategraf my (possibly incorrect) understanding is that this should be configurable with a var. What I would suggest is that you confirm if this is still a valid issue for our supported branches. If it is not valid we need to communicate with support how to fix it. This may include a docs update. If it is valid we need to work out where the fix needs to live and if upstream make sure there is a valid issue and we fix it if possible.
In addition, we no longer provide development support to Juno so you don't need to worry about that one.

I've removed the blocked label based on my previous comment. @nategraf if you still think this should be blocked let me know and we can discuss.

I looked into this issue a little further and spoke with support.
A) This is actually not a bug in openstack-ansible. It is a feature of cinder cli [1] which was not implemented until Kilo.
B) Any environments using Kilo or later will not experience this issue.
C) There is at least one customer using Juno with over 1000 volumes, and they are seeing this issue.
D) There is a workaround, which is to pull the volume data directly from the database.
@git-harry  based on this information is this a bug we are going to pursue fixing?
[1]
https://blueprints.launchpad.net/cinder/+spec/cinder-pagination
https://bugs.launchpad.net/cinder/+bug/1342192

Closed based on this bug only effecting Juno, which is EOL.
For environments which will not be upgrading to Kilo or later, query the database directly to get a complete list of volumes.


Regarding the Current State of Fuzz

Dear esteemed Mr. Robert, I happened to come across your response from 2019 stating that you were in the process of developing a more powerful TLS-fuzz tool. May I humbly ask if the new tool you referred to is TLS-Anvil? Or perhaps your esteemed team is still in the state of development. If it wouldn't be too much trouble, I would deeply appreciate any guidance or insights you could provide regarding research approaches to TLS security. I recognize that this might be a presumptuous request and I hope you don't find it impolite. Thank you sincerely for your time and consideration.

Hey,
oh, jeah fuzzing - it is still not publicly available yet. The paper is ready since 7 years but in a constant review cycle where we submit it now and then. The plan was to release the tool when the paper get accepted. We have to internally decide if we just release it at some point/eprint.

Wow! It is with great anticipation that I look forward to this. It is my sincerest hope that your academic paper receives swift acceptance. I hold the belief that the tool developed by your esteemed team is set to cause a stir. I shall pivot my research thinking in response to this. Thank you most cordially for your enlightening response! It is with genuineness that I extend my wishes for the earliest acceptance of your paper.


Automate Blood Magic notes

A lot of these existing effects can be done better (and be compressed) with current tech, but I stuck to making notes, linking effects and making damage buttons. I'll probably revisit the effects soon unless someone does first.

Make sure you're specifying traditions and spellcasting category to avoid stepping on spellcasting from archetypes.

Make sure you're specifying traditions and spellcasting category to avoid stepping on spellcasting from archetypes.

I overlooked that for sure. I'll get to it tomorrow.

Nevermind, I did it now really quick


Not registering as app

OS:
Windows 10 Home 640bit (10.0, build 14393)
Issue Descriptions:
The app doesn't register under the start menu.
It seems like it's opening as a file but not a program. I don't exactly know how to describe it.  It works properly, but doesn't register in the start menu and I can't actually get it in there.  I want to be able to use my open music media key to open it since it's my favorite music app ever! :)
Steps to Reproduce:
Install app.

I am going to add to this as I have experienced this problem.  I run the executable, I see the "Installing Google Play Music Desktop Player" Splash.  The application launches and functions just fine.  If I close it out and try to launch it again, I can not locate it under the start Menu, No Desktop Icon, Program Files / Program Files (x86) (though I am not sure where it would install to).  I run the exe again, it runs through the installation process fine and loads/works perfectly.
Windows 10 Pro N 64bit 1607 build 14393.51

I removed my install and grabbed the latest copy to test this and can't confirm.

Same issue for me.
Running an exe installs it everytime, it doesn't add it to Start menu or something.
Windows 10 Ent N x64 1607 14393.51

Okay, what is the common denominator, besides windows 10. I'm on 14393.51 as well and can't reproduce. What anti-virus do you use? Can you attach your squirrel setup logs?

@jostrander Shortcuts are no longer handled by Squirrel.  They are handled internally by Electron API's.
Can you guys please launch launch GPMDP from Command Prompt with the following commands
cd %LOCALAPPDATA%\GPMDP_3\app-3.5.0
set ELECTRON_ENABLE_LOGGING=true
set ELECTRON_ENABLE_STACK_DUMPING=true
Google%20Play%20Music%20Desktop%20Player.exe --squirrel-install

I ran the commands under both a user level and elevated CMD - I also removed the GPMDP_3 folder and ran the install file again before trying to launch it in CMD.   It seems to just hang.
`Microsoft Windows [Version 10.0.14393]
(c) 2016 Microsoft Corporation. All rights reserved.
C:\Users\Jimbo>cd %LOCALAPPDATA%\GPMDP_3\app-3.5.0
C:\Users\Jimbo\AppData\Local\GPMDP_3\app-3.5.0>set ELECTRON_ENABLE_LOGGING=true
C:\Users\Jimbo\AppData\Local\GPMDP_3\app-3.5.0>set ELECTRON_ENABLE_STACK_DUMPING=true
C:\Users\Jimbo\AppData\Local\GPMDP_3\app-3.5.0>Google%20Play%20Music%20Desktop%20Player.exe --squirrel-install
C:\Users\Jimbo\AppData\Local\GPMDP_3\app-3.5.0>
`

@morbidxbean And after running that the shortcuts weren't generated at all?

Hey!  I think we are good.   So I did not notice any icons generate in the start menu or Desktop.  However I have my actual Desktop mapped to a separate drive.  (Z:\Desktop)   The  Commands I ran (or could have been the actual installer in the first place) placed the icon in the default "Desktop" folder... (C:\Users\name\Desktop).   I looked there and sure enough the lovely Icon was there.
Same instance / somewhat different situation with the start menu I believe... (%appdata%\Microsoft\Windows\Start Menu).  When I look under there I see an Icon.  However if I look under the start menu itself I cant see it :\

@morbidxbean Wait, how do you map your desktop / appdata to a different drive.
It sounds like you did it in a strange way 👍

The Default Library folders that come with windows (Documents, Downloads, Music, Pictures, VIdeos)  If you right Click > Properties on those, there is a Location tab where you can change where the files are stored.  I choose to have them on a second hard drive based on preference.  I'm just stumped on what initially happened that put my preference system off sync lol.

So this is likely a windows issue (remapping your folders but not updated %APPDATA%)?

I am going to say that it is a windows problem.  I installed the app just fine w/ no issues under a VM running windows w/o modifications to the folder design.
@JJSax ;  Perhaps you have a similar configuration as myself?  Take a look in your C:\Users\name\Desktop directory to see if there is an icon.

It's highly likely.  If you guys want a shortcut you can just copy paste those ones.
Don't make your own as the shortcuts use a fancy trampoline launch mechanism that makes the shortcut work across updates.  If you make your own the shortcut will break after each update.