Disable skipInsignificantPunctuation in operator-pending mode

Hello!
This is a fix for #9 and #10 for cw, dw, etc.
The option skipInsignificantPunctuation is a great thing, but not suitable for operator-pending mode.
Just try it. Everyone can install this plugin from my fork to try.

Thx for the PR!
However, I am a bit skeptical about disabling punctuation skipping during operator pending-mode.
One rule that all vim motions follow (with the weird exception mentioned in the readme), is that the motions behaves the same when used in operator-pending mode, it affects the same area as one would move to when using the motion in normal mode. This ensures predictability, i.e. you easily know what area will be affected.
Your PR would break that rule and therefore make it harder to predict which area will be affected, as one has to consider one area that will be jumped over in normal mode, and a different area that will be affected when used in operator-pending-mode.


s3 server installer

create an installer for an s3 server (garage for example)

progress update:

the installer is done
the start method is still in progress, but the configs of garage still need some investigation


Implement Analysis controller

Implement Analysis controller:
Depends on #1768
Depends on #1769
Depends on #1758
In this ticket, the Analysis controller shall be implemented. The logic of the controller should be as follows:

Whenever an Analysis is added/updated, the controller should look for the referenced AnalysisDefinition.
When the referenced AnalysisDefinition has been found, the controller should gather all AnalysisValueTemplates required by the AnalysisDefinition.
For each of the referenced AnalysisValueTemplates, the controller should use the available implementations of the metric providers to fetch the data using the query specified in the AnalysisValueTemplates, including the timeframe and selectors
from the Analysis spec.
After all values are available, the controller should invoke the scoring function that has been implemented in #1767, and store the result in the
status of the Analysis. At this point, the Analysis has been reconciled successfully.

DoD

Analysis controller is implemented
Unit Tests
Integration Tests

References
Bigger picture of the concept: https://github.com/keptn/lifecycle-toolkit/issues/1650

resolved with https://github.com/keptn/lifecycle-toolkit/pull/1875


Use agent core package

:tada: This issue has been resolved in version 2.4.0 :tada:
The release is available on:

GitHub Release
NuGet Package
Chocolatey Package


Do you guys have any plans to support Windows?

I am making a window os application using flutter.
I need in-app web view can be mounted on windows

I also need the support of windows;


PoetryProject reads files and dirs using iterdir() when looking for src dir

Hi,
The current line in the PoetryProject class gets a list of files and directories, which causes an error for me on MacOS because of the inevitable .DS_Store file generated by Finder.
-> it raises ControlledError("multiple subdirectories found in src")
https://github.com/nahco314/poetry2rye/blob/abf8b4f246ab128ba4e80cced202506aa47a8db8/src/poetry2rye/project.py#L114
However the following line should work to filter only dirs:
sub_lst = [d for d in self.src_path.iterdir() if d.is_dir()]
Thanks for the nice package.
Regards.

Thank you for reporting the problem!
I'm too busy right now to get around to it and will address it in the next week or so.
If it is a simple change that can be fixed, you can create a PR and I can merge it.

https://github.com/nahco314/poetry2rye/commit/9e67b6834ca57e0fb34e11e91ac1d553fec25336 will fix it!
If it still have problem, please re-open the issue.


404 Error's on some documentation links

Steps to Reproduce:
1.	Navigate to the “Infrastructure-Deterministic” phase.
2.	Click on the “>1 COMPUTER” marker and select the “View more information” option.
3.	Similarly, navigate to “Breakglass Secret Access”,  "immutable infrastructure",  "Production Launch Platform", "Failure Injection", Multi Machine Debugging, hotspots etc, "Vanilla DDoS Protection", "Product Specific DDoS Protection (e.g. WAF)", "Proactive DDoS Countermeasures", "Self Driven Checklist Launches"

Expected Behaviour:
Clicking “View more information” for the above nodes should display markdown files with additional information.
Actual Behaviour:
All links return a 404 error page, indicating the files are missing.
Examples:

fixed (mostly) in #40

nope. lots of those are still broken. please hold.

ok it looks like the GenAI pass missed a few :( -- i pulled them back in from old/  and now they at least aren't 404s.


Debugger restart button not working anymore

Issue Type: Bug
After starting a debug session (debugging an extension) and try to restart the session by clicking the restart button in the UI it results in this error:

Also the tooltip on that button is "Reconnect" now, rather than "Restart" so my guess that is part of the problem. I have also seen the stop button turn into the disconnect button as well.
This all just started in one of the very recent insiders builds.
VS Code version: Code - Insiders 1.27.0-insider (1097b5e9ed81d5f9241648774b35048be168c203, 2018-08-31T10:38:54.277Z)
OS version: Windows_NT x64 10.0.17746

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz (8 x 4008)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
31.93GB (13.37GB free)

Process Argv
C:\Users\Eric\AppData\Local\Programs\Microsoft VS Code Insiders\Code - Insiders.exe

Screen Reader
no

VM
0%

Got the same error and found this to be related: https://github.com/Microsoft/vscode/issues/57018
That workaround solved it for me: https://github.com/Microsoft/vscode/issues/57018#issuecomment-415183598

@haschu I don't think this is related. This issue just started when I first reported, and now exists in the newly released 1.27.1 stable release as well.

I think this is also caused by Live Share as well.

@rodrigovaras fyi

Yes, live share debugging support was relying on the presence of this property during launch/attach. We are working on a fix to use the public API to avoid future breaking changes. The patch will be available soon


HEIC support

Hello,
I have a question - do you support HEIC format (the default format on the mobile / iOS devices)?

I've just tried, doesn't appear to be working..any ideas on what we can do to support heic ?

Has this changed ? thanks

imagemagick now supports heic/heif
https://imagemagick.org/script/formats.php
can we include this version in our layer?

imagemagick now supports heic/heif
https://imagemagick.org/script/formats.php
can we include this version in our layer?


Add explicit include directories setting

It would be nice if we could explicitly include directories. I have a Vault with mixed uses, one of them being writing, and I'd like to only include my "Writing" directory. The changes here are more of a proof-of-concept, but they work. If you'd consider supporting this feature and want me to flesh this out a bit more I can.

Apologies for some of the formatting changes. If you have other prettier settings outside of .prettierrc I can rework these changes to adhere to style guidelines.

The rationale for me for not using separate vaults is that I like not having to duplicate plugin settings across vaults, and I just prefer the consolidate organization of a single vault. I have about 10 plugins I'm using regularly and they all have pretty fine-tuned settings. I don't want to duplicate that across several vaults. I have a separate vault for my day job, but that's about the only reason I use a separate vault, and mostly so I can get my employer to pay for the commercial license for the work project stuff.
It seems very relevant to me for a plugin whose purpose is to read directories and files and display metrics about them to have the capability to configure which directories it is doing that for. Ideally I could add multiple, maybe comma-separated, and glob paths.
Your point about balancing configuration vs. complexity though, is very much understandable and if you don't feel like supporting this, it's up to you. I just thought this was relatively painless and straightforward to implement and would support my own selfish use case 😇, and I'm happy to do the leg work.
From a developer's perspective, this is how I expect pretty much everything touching my directories and files to behave. Docker, Prettier, eslint, npm, webpack, babel, git, etc.... they all have ways of meaningfully excluding or including directories being processed.
From an Obsidian user's perspective, just browsing a few of the plugins I use: obsidian-linter has a "Folders to ignore" setting; obsidian-periodic-notes has ways to configure which folders to place periodic notes in, or which to source templates from; Templater lets you configure template folders.
Some of these aren't quite parallel to the use case we're discussing, but to me, the overarching theme is "this plugin interacts with X, I should have control over how it interacts with X".

Thanks for the response. I do find that persuasive. Let's get the code in a place where we're both happy with it and we can merge later this month.
Btw, would you mind including an update to the README? I don't want to forget to document the new setting.

Sounds good! This was more of a proof-of-concept, so I'll flesh it out, incorporate your feedback, update the README, and do some more thorough testing.

Hey @adamhp, are you able to put the finishing touches on this PR? I think I'm ready for it.

Yep! I’ll look at it this afternoon.
On Tue, Jan 23, 2024 at 11:18 PM Isaac Lyman @.***>
wrote:

Hey @adamhp https://github.com/adamhp, are you able to put the
finishing touches on this PR? I think I'm ready for it.
—
Reply to this email directly, view it on GitHub
https://github.com/isaaclyman/novel-word-count-obsidian/pull/67#issuecomment-1907351093,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWGL5UFLI5W54CGXY2YG5LYQCDP5AVCNFSM6AAAAABAC6BEMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBXGM2TCMBZGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

@adamhp Let me know if you're too busy to finish this one and I'll pick it up.

Hey, apologies, that suddenly may be the case. Don’t want to hold this up.
Sorry about that.
On Wed, Jan 31, 2024 at 3:52 PM Isaac Lyman @.***>
wrote:

@adamhp https://github.com/adamhp Let me know if you're too busy to
finish this one and I'll pick it up.
—
Reply to this email directly, view it on GitHub
https://github.com/isaaclyman/novel-word-count-obsidian/pull/67#issuecomment-1919937938,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWGL5Q54Y6PZTO5CMVVLCDYRKVI7AVCNFSM6AAAAABAC6BEMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJZHEZTOOJTHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

No problem, totally understandable.


Persistent label shortcuts, no resetting after moving on to a new job

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

I use the label shortcuts extensively when annotating, but right now I have to keep reassigning the shortcuts each time I move on to a new job (or if I exit then go back into the same job). It would be ideal if those shortcuts were persistent at project level.
Expected Behaviour

After setting up the label shortcuts inside a job, they should not be reset after exiting the job. This way when you move on to a new job or exit and return to the same job, the label shortcuts would still be the same as the ones you set.
Current Behaviour

Label shortcuts reset to whatever it was before manual changes.
Context

Right now my workflow consists of uploading thousands of images to CVAT using FiftyOne, with this I end up with several tasks, each one with 5 jobs and each job with 100 images. This means that after annotating 100 images, I have to reset my shortcuts, which is quite time-consuming and adds up after doing it hundreds of times.

Hi, @sf-pear
Thank you for the feedback.
It makes sense


hunterize lss

Released: https://github.com/cpp-pm/linux-syscall-support/releases/tag/v0.0.0-e1e7b0a-p0


Mark messages read on stream unsubscribe

@hdorothea thanks for working on this!  I think possibly the correct implementation here might be to make this change on the Zulip backend, however; since I think we have the same bug if you unsubscribe from the webapp (though note the symptom is invisible, because of how the webapp accesses data differently).  See https://github.com/zulip/zulip/issues/5959 for the issue we recently opened for this problem.

Thanks for the feedback, it was very helpful.
I agree the bigger issue is: what should happen to read/unread on unsubscribe. And that should be discussed in the main repository issue and implemented on the backend.
As to the question of wether we should have a simple mobile front-end fix: It would be trivial to remove the unsubscribed stream from unread.streams in the state and thus from the unread-tab. However adding it back there on resubscribe with consistent behavior on reload seems hard as, as far as I can get it the unread message ids are (only?) sent with the initial state.

@timabbott @showell is this resolved server-side now?

Not yet.  We've made the decision to mark messages as read when you unsubscribe, but it hasn't been implemented yet.

I will close this PR then. We will auto-magically have this fixed in the mobile app when this lands in the backend.
@hdorothea thanks for noticing this and working on the issue. I would love it if you continue working on anything else you want to fix or improve upon.


Use UIImage's scale when resizing image

This is a bugfix.

          1 Warning

:warning:
The source files have been modified. Please consider adding a CHANGELOG entry if necessary.

          1 Message

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

  Generated by :no_entry_sign: Danger

Done 👍

Done 👍

Is this PR buggy? I can't get the correct scaled image when the scale is set to image's scale.
SwifterSwiftImageDemo.zip

Looks like it. That scale value doesn't make sense. Please go ahead and create a new PR with your fix :)

@guykogus Thanks, I have created a new PR for this, please review it when you are free and feel free to tell me where I need to change.


MQTT Broker

We must deploy a MQTT Broker, to be publicly available for remote communication with Cal-Tek's VR environment.

Requested deployment with Mosquitto MQTT Broker, could be deployed with helm on AWS cluster, or single docker container
Could register subdomain within 5ger.net
TLS encryption
Authentication
Data persistence

Deployed Mosquitto MQTT Broker, available at mqtt-broker.5gera.net port 1883 with authentication enabled.


wm-restack = generic doesn't put it below xfce4-panel

Is your feature request related to a problem? Please describe.
Achieving functionality that is already provided by common DEs can be challenging. I don't see another way to do it than to heavily script stuff. I've seen on /r/unixporn a guy getting drop down menus done on polybar which are other instances of polybar appearing. While it is an extraordinary achievment, it still looks clunky and is a pain to setup.
Currently, wm-restack = generic still puts polybar on top of the xfce4-panel if they occupy the same part of the screen. If I add
override-redirect = true, the polybar disappears. The process is still running so I'm assuming it goes behind the xfdesktop (behind the wallpaper).
Why does polybar need this feature?
One can always keep polybar for showing stuff on a bar while using xfce4-panel plugins for productivity. However, that leads to the same information being displayed more than once. For example, how can an user connect to a different wifi network with two clicks on polybar? This action is done on the xfce4-panel plugin by clickin on the icon and then clicking on the desired network from the list provided in the drop-down menu. That's 2 clicks. On polybar you could set the connections manager to open via clickable action but for example on xfce that manager doesn't allow connections to another network with 1 click. In fact, I haven't found a way to select a new network to connect with a single polybar module. One would need a command do show all the available networks which can't be done in the space of a bar so it would have to open a terminal window or a connection manager, and then to connect to a selected item from that list is another trouble.
Describe the solution you'd like
I figured that a cool way to have drop downs on the polybar would be to create an invisible

xfce4 panel on top of the polybar module and make it invisible. This way, a click on top of the visible area would trigger the xfce4 panel plugin but the visible area would still belong to the polybar. This would be useful to have the drop down calendar when clicking the clock plugin or even the Wi-Fi settings.
Describe alternatives you've considered
Possibly calling xfce4-panel widgets into the polybar? An internal module that serves as a compatibility layer? I don't see it being more viable than to simply having a better way to set the polybar priority to be above the desktop but below the xfce4-panel. The space is already allocated by the panel so there's no need to reserve with polybar.
Additional context
Is there a way to achieve what I want? Maybe I didn't find something that I could've used instead. I added some images with my paint skills and photos from the phone since print the screen key doesn't work when a panel plugin is active. Oh well...

I can make the xfce4 panel completely transparent but still work. I'd have it on top of the polybar just for the clickable actions, that was the workaround.
I compiled from source and installed before I opened this issue. polybar -v returns polybar 3.5.7-212-gb5fb4422
Running xdo lower -n polybar will make it disappear just like the override option.

Alright, I did some testing with this. Turns out, wm-restack doesn't work in xfce4 without override-redirect = true because in xfce, the polybar window isn't a top level window, but is wrapped in another window and so it can't restack relative to the root window.
In addition, if override-redirect is set, polybar is indeed placed below the background somehow. Looks like we need a wm-restack = xfce4 settings that has a restacking algorithm specific for xfce4.
PRs are welcome.

I just pushed a fix in #2961 for another restacking issue. The changes there may also work here (try the ewmh or generic restacking strategy).


duplicate message id?

The issue tracker is ONLY used for bug report(feature request need to follow RIP process). Keep in mind, please check whether there is an existing same report before your raise a new one.
Alternately (especially if your communication is not a bug report), you can send mail to our mailing lists. We welcome any friendly suggestions, bug fixes, collaboration and other improvements.
Please ensure that your bug report is clear and that it is complete. Otherwise, we may be unable to understand it or to reproduce it, either of which would prevent us from fixing the bug. We strongly recommend the report(bug report or feature request) could include some hints as the following:
BUG REPORT

Please describe the issue you observed:

I found duplicate message id when use rocketMq 4.4.0

What did you do (The steps to reproduce)?

What did you expect to see?

What did you see instead?

Please tell us about your environment:

Java8 macOSX

Other information (e.g. detailed explanation, logs, related issues, suggestions how to fix, etc):

Why does this happen and how to fix it


HAWQ-1438. Support resource owner beyond transaction boundary

(1) Add TopResourceOwner to trace resource used beyond the transaction boundary.
(2) When to auto alloc a new TopResourceOwner for each process?
Generate a singleton TopResourceOwner when setting CurrentResourceOwner to NULL.
(3) If crashed at some code beyond the transaction boundary, we should manually create
a new resource owner, because we do manually create a new one for each process.
(4) When to call ResourceOwnerDelete(TopResourceOwner) automatically for each process?
Register it at on_proc_exit().

+1

+1


Validation - Extend App Config with Configuration Parameters

Description
This PR adds validations to the extended app-config-parameters described here.

@sorin-stefan-iovita-sap will you take care of the conflict resolving?

@hilmarf , yup, I'll do that in the morning. 👍


CI: improvments

Optimise matrix jobs and used environments
update runners ubuntu-20.04 -> ubuntu-24.04
add Setup packer to maintain packer version up to date

closed accidentally

Parquetsharp has moved to Github hosted runners of arm64 arch, so we can remove Parquetsharp from implementation and keep only ILGPU


Fixed #32990 -- Simplified and optimized tag regex.

Refactors the lazy_re_compiled tag_re which used six re.escape calls into a quickly compiled regex without redundant escape characters. Adds tests for the Lexer to ensure correct behavior.
The simple expansion of the strings with escape sequences yields '(\\{%.*?%\\}|\\{\\{.*?\\}\\}|\\{\\#.*?\\#\\})' which is full of redundant escape characters.
It turns out there is more overhead in just instantiating the SimpleLazyObject with _lazy_re_compile than there is in compiling the regex immediately.
$ python -m timeit -s 'from django.utils.regex_helper import _lazy_re_compile' "_lazy_re_compile('(\\{%.*?%\\}|\\{\\{.*?\\}\\}|\\{\\#.*?\\#\\})')"
500000 loops, best of 5: 620 nsec per loop
$ python -m timeit -s 'import re' "re.compile(r'({%.*?%}|{{.*?}}|{#.*?#})')"
1000000 loops, best of 5: 277 nsec per loop

Rather than adding a test to ensure the same pattern is used (which would not pass as the pattern has been trimmed of redundant escape characters) I added tests of the Lexers that use this regex to ensure they are sensibly parsing tags from a template string.

@gtwohig Thanks for this patch :+1: Welcome aboard :boat:


feat(CategoryTheory): limits of eventually constant functors from cofiltered categories

An eventually constant functor from a cofiltered category admits a limit.

Thanks!
maintainer delegate


Give options to assign more text weights in tokenColors

I cannot assign the text weight to, like 500, using the current token theming mechanism.
Solution: Add fontWeight support in tokenColors.*.settings, and map that directly into CSS properties.

@be5invis I assume with text weight you mean font weight.
Token coloring is done with TextMate themes. TextMate themes only support bold, italic and underline. There are currently no plans to extend text mate theming capabilities.

@aeschli Does TextMate support JSON themes?
We can enable that in JSON one only

Just found this while trying to create a theme that uses font weights and typography instead of colors. Since the JSON format is already specific to Code a custom fontWeight should not be that hard to support.
Please let us take advantage of all these font weights: https://github.com/be5invis/Iosevka

@be5invis It seems not many people care about this but GitHub claims that you're not only Iosevka's creator but also a VS Code contributor. Is there anything I can do help this become a thing?

Any current thoughts on this?  I was hoping to have italics as a lighter weight....

@gubikmic, you're right, it appears to be a central piece, but I doubt it's ever persisted (which would have made it way harder to touch the design).
Also, it doesn't look like the colors themselves are limited to 9bits, at all.  f and  b seem to be just indexes into theme-specific arrays (palettes), which could potentially hold 512 colors each...
In short, it appears to be following the good old color palette idea... probably done this way for the usual reasons (save on resources, especially RAM).
On the other hand, the specific choice of 9bits for palette width is kind-of strange...
imho, 1024 (512 + 512) unique colors per theme is quite huge... I wouldn't expect any given theme to actually use more than a handful (or perhaps a few dozen maximum) unique colors... Remember, the color values themselves (RGB, RGBA, ...) could still be anything supported by vscode.
Also, I don't get the reason why the metadata stuff needs to be present for each token... yielding so much redundancy... and that appears to be going against the idea of saving RAM...
I would speculate that this whole design is probably a historical thing, carried over from the days where you had to save on both CPU and RAM....
Nowadays, one can think of several ways to do it differently. Here are a couple that come to my mind...
@all, please correct me if I am getting any of this wrong.
OPTION 1) extensible   -- (but a bit risky)
Why not just externalize the whole metadata stuff into a ephemerous cache/registry of some sorts, each entry catering for a specific combination of languageId + tokenType + tokenID (possibly semantic: like "keyword", "variable", ...) and whatever else.
That way, you could still keep the same memory footprint per token, or even lower it. (at present the "metadata" takes up 32 bits).
So, one wanted to add font-weight, it would be added to that central location for token metadata. That would be the "cleaner" approach, allowing for easier future extension.... BUT: it would be more development work... and also eat away a bit more CPU cycles, perhaps....
However, the hit on CPU does not have to be that much, if one tries to avoid/minimize hash key computations, or else employ an standard integer-indexed array for the ephemereous registry.
OPTION 2) quick & dirty   -- (and also minimal impact to the existing codebase, probably)
Another --quick & dirty-- approach could be just grab the 2 high bits from both color ids (b & g), reducing them each to 7 bits. That would amount to 256 unique colors per theme (128 fg, and 128 bg)... way more than enough, imho...
And then you would have 4 bits freed up for fontWeight => 16 unique values => representing each aprroximately in steps of 6%.
That limitation (to 4bits) doesn't have to be known to whoever sets the fontWeight. It shoudl be possible to set it to any valid percentage value (such as 25%).  That value would be rounded up or down, as an implementation detail.

Thoughts?

See also #110863

See also #110863


Syntax for portSpecs

Can someone provide an example of how to build a ContainerConfig that contains port bindings, e.g.
container port 8080 bound to host port 80 ?

It's 2 years later and I'm not sure if the API has since changed, but I believe this is correct:
portBindings.put("8080/tcp", Lists.newArrayList(PortBinding.of("HostPort", "80")));


Add support for properties

We should be able to get the current state of an actuator for instance.

Done.


Amazon Elastic File System (EFS) - How To Access Files

This project is an excellent starter-kit to help someone (me) with novice experience to build a production-ready AWS environment with minimal effort--thank you!  With that said, I'm having a difficult time accessing the files stored on the EFS from a NFS client on my mac.  Ideally, I am looking for an easy solution similar to FTP, where I can connect to an EFS server to CRUD files included in the WordPress distribution, but I don't know where to start.  On a similar note, I successfully implemented "W3 Total Cache" per the instructions found in this document: http://d0.awsstatic.com/whitepapers/deploying-wordpress-with-aws-elastic-beanstalk.pdf (page 23).  I attached an S3 Origin to my CloudFront distribution, now I can access the wp-includes and wp-content folders directly from my FTP client, Transmit, that supports S3.  I would like to access EFS in a similar fashion to FTP, please help.

Currently accessing EFS from on-premises systems is only supported using Direct Connect; https://aws.amazon.com/efs/faq/#on-premises.  There may be other ways to achieve this, I have seen mention of accessing over a TLS or SSH tunnel, though the FAQ specifically points out that access over a VPN is unsupported.
You might have some success modifying the architecture to allow the bastion host to mount the EFS File System and then use an SSH tunnel to access it from your Mac.  A better approach might be to build an SSH tunnel through to a WordPress instance, via the bastion.  Neither would really be considered best practice and so I'm afraid are unlikely to be integrated in the Reference Architecture.

Thanks @RGuilfoyle for the suggestions.  I would like to explore your last recommendation a bit more, to build a SSH tunnel through to a WordPress instance, via the bastion.  Can you walk me through what this would look like, step-by-step?  As I mentioned, I'm a complete novice.

I've searched high and low for a solution to access files, from trying to implement a VPN to trying to implement Direct Connect--I can't see to figure this one out.  Basically, all I'm trying to do is install Wordpress themes and plugins, but I get stopped by the error message "The uploaded file exceeds the upload_max_filesize directive in php.ini.".  How do I increase the upload_max_filesize limit in this environment that is preventing me from installing themes and plugins?

Jimmy, this looks like a change is needed in php.ini.  You should be able to start a bastion by changing the desired capacity on the Auto Scaling group to 1.  This can be done through the console.
Once a bastion host is booted you can log in via SSH.  From there you can log into your WordPress instances and edit the php.ini file.  This change will, however, be reverted as new instances are deployed.  I'll investigate implications of adding this to the templates to increase the default.
Note: The bastion will automatically recognise your ssh key to allow you to log in.  You will have to copy the private key to the bastion to allow onward access to the WordPress instances.

To SSH to any instance in private subnets (web, etc.):

change the min & desired instance count of the Bastion ASG to 1
wait for the Bastion instance to launch
follow the direction from our AWS Security blog to SSH to instances in private subnets - https://aws.amazon.com/blogs/security/securely-connect-to-linux-instances-running-in-a-private-amazon-vpc/

@darrylsosborne can you explain a little bit more? ;(

Step 1: Download your EC2 keypair (.pem file) on you local machine;
Step 2: Launch the Bastion Host
EC2 > Auto Scaling > Auto Scaling Group, set Desired, Min, and Max to “1”, the Bastion Host will launch automatically;
Step 3: Open console/terminal, key in the following commands:
Step 4: Load Keypar
$ ssh-add -K /[your-keypair-path]/[your-keypair-file].pem
Step 5: SSH into Bastion Host
$ ssh -A ec2-user@[EC2 Bastion Public DNS (IPv4) IP]
Step 6: SSH into Web Server
$ ssh -A ec2-user@[EC2 Web ASG Private DNS IP]
Step 7: Open php.ini
$ sudo vi /etc/php.ini
Step 8: Edit “Upload Max File Size” parameter (upload_max_filesize = 90M)
Step 9: Restart Apache
$ sudo service httpd restart
Step 10: Reset the Auto Scaling Group and Terminate Bastion Host

@jimmyjuarez u r GOD :angel:

@lfreneda, thank you.  All the thanks goes to @darrylsosborne and @RGuilfoyle for building and supporting such an amazing project.  Looking forward to where they take this thing next!

@jimmyjuarez sure.
did u have issues updating and installing plugins?

Hi,
I have a question and hope you can help me. Our Wordpress team needs access to the WordPress directory through sftp/scp.
What is the best way to do so?
Any suggestions would be really appreciated!
Thanks


[Bug]: Sometimes empty pdf document is generated

Bug description
Steps to reproduce the problem:

Just generate a pdf file where a stylesheet is linked in the  of the html.
Look at the pdf

Some of the pdfs (I cannot see any pattern) are not generated correctly.
The following shows how the pdf should look like:

In the same run, puppeteer also produces the following pdfs from exactly the same html source. As I said, I cannot see a pattern when this happens, just sometimes, I guess around 1/5 of the pdfs are wrong.

Puppeteer version
5.4.4
Node.js version
14.17.5
npm version
7.21.1
What operating system are you seeing the problem on?
Windows
Relevant log output
No response

await page.setContent(html, { waitUntil: "networkidle0" });
Setting waitUntil to networkidle0 instead of load seems to solve the issue.


Help needed - Want to apply colors from server

Hi
I use wix react-native-navigation V2. My requirement is to get the colors from server on start up from a JSON and then set the colors for all components in native-base? Is this doable?If so how? I did a similar set up in react-native-elements and it works well, want to do the same in native-base. How am I supposed to do this?
Here is what I did in react-native-elements "ThemeProvider". On start up user store is called to get the theme and then navigation screens registered with it.Now if I use a button from react-native-elements in a navigation screen ,the primary color is automatically picked up correctly
UserStore.js
  getRecentUserTheme() {
      const user = this.getUservalues()[0];
      const { theme } = user;
      const primaryColor = theme ? theme.primaryColor : '#329F9F';
      const secondaryColor = theme ? theme.secondaryColor : '#93329F';
      return {
        colors: {
          primary: primaryColor,
          secondary: secondaryColor
        },
        logo: theme.logo
      };
    }
  }))

App.js
const registerScreens = async () => {
  Screens.forEach((ScreenComponent, key) =>
    Navigation.registerComponent(
      key,
      () => props => (
        <ThemeProvider theme={theme}>
          <ScreenComponent {...props} />
        </ThemeProvider>
      ),
      () => ScreenComponent
    )
  );
};

With native base its confusing because I have to eject the themes but how to apply theme dynamically and wrap in navigation like above I dont know.
Can anyone please help me at the earliest?

@trialsNow You share the code snippet when tried this with NativeBase


Fix Windows codesign permissions

Currently, the executable bit isn't added back to the codesigned binary:
bin/java: Permission denied

@gdams
Found 1 clones.
ERROR: jscpd found too many duplicates (2.49%) over threshold (0%)
Error: ERROR: jscpd found too many duplicates (2.49%) over threshold (0%)
at ThresholdReporter.report (/node_modules/@jscpd/finder/dist/reporters/threshold.js:12:19)
at /node_modules/@jscpd/finder/dist/in-files-detector.js:82:26
at Array.forEach ()
at /node_modules/@jscpd/finder/dist/in-files-detector.js:81:28
at processTicksAndRejections (node:internal/process/task_queues:94:5)

@karianna I'll refactor in a follow up PR if that's okay

@sxa thanks for the tip. Updated PTAL


cannot find table function tableCellsSelection

I get the following error on including the plugin
Uncaught TypeError: $tables.tableCellsSelection is not a function(…) on line 188 in file plugin.js

me too:
Uncaught TypeError: Cannot read property 'selectableTableClass' of undefined


[BEAM-145] [BEAM-34] Make WindowingStrategy combine WindowFn with OutputTimeFn

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Previously:

Any user-specified OutputTimeFn overrode the WindowFn#getOutputTime
WindowFn#getOutputTimeFn provided a default OutputTimeFn
The default varied from "earliest" to "end of window"

Now:

The user-specified OutputTimeFn is used to combine the WindowFn's
assigned output timestamps.
The WindowFn does not provide the default.
The default is always to output at end of window.

For each of the tests that this effects, I had a choice: either update the timestamps in the test to be the end of window, or explicitly reset the windowing strategy to choose the minimum timestamp. The latter generally gets more useful coverage, since the latter is fairly trivial, so I generally favored it. It is also easier to migrate to. And most of the tests are overspecified anyhow and should not be examining the timestamps.

R: @bjchambers @mshields822

LGTM
Much simpler.
Like that users start with progress as the default (common ask) and can override with OutputTimeFns.outputAtEarliestInputTimestamp to instead have more outputWithTimestamp flexibility (uncommon ask).

LGTM
We should probably start a list of changes that need to be made to the Beam docs (eg., default output time behavior).

Yup, definitely.


can not run with qemu-arm-static

Your Windows build number:  (Type ver at a Windows Command Prompt)
OS Name:                   Microsoft Windows 10 Pro
OS Version:                10.0.16299 N/A Build 16299

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)

"~/wheezy/" is arm rootfs.
I have installed qemu-user-static.
For a simple test qemu-arm-static executable, ~/wheezy/bin/ls, expected results would be:
$ ~/wheezy/bin/ls
/lib/ld-linux.so.3: No such file or directory
While actual results are:
$ ~/wheezy/bin/ls
prepare_binprm: Bad file descriptor
Explicit invocation of qemu-arm-static, however, works:
$ qemu-arm-static ~/wheezy/bin/ls
/lib/ld-linux.so.3: No such file or directory
Show the binfmts list, it is OK:
`update-binfmts --display qemu-arm
qemu-arm (enabled):
package = qemu-user-static
type = magic
offset = 0
magic = \x7f\x45\x4c\x46\x01\x01\x01\x00\x00\x00\x00\x00\x00\x00\x00\x00\x02\x00\x28\x00
mask = \xff\xff\xff\xff\xff\xff\xff\x00\xff\xff\xff\xff\xff\xff\xff\xff\xfe\xff\xff\xff
interpreter = /usr/bin/qemu-arm-static
detector =
`

Same issue here.

And here

Same here under version 10.0.16299.192

https://gist.github.com/Froosh/142d0da5dddd15fdc65f66e52a2d644d

From strace.102:

read(3, 0x7fffce27eca0, 1024)           = -1 EBADF (Bad file descriptor)

File descriptor 3 is nowhere to be found in the two files you posted though. Need the contents of strace -f -o bash.armel.strace ./debian-armel/bin/bash (and some luck). There's some problematic stuff in there already, for example MADV_HUGEPAGE and /proc/sys/vm/mmap_min_addr. Those might be mortal wounds on their own, but not the kill shot.
There is still no repro here though; which is the place to start. Those three commands aren't going to fly on Real Linux either, naturally. [You'll get missing /lib/ld-linux.so.3: No such file or directory just like the OP. Because no qemu.]
This almost certainly isn't going to fly; or someone would have showed off a long time ago. But I can get at least give you the first blocker, in return for a repro or/and a complete strace.

Hehe, yeah, this was a bit rushed in the middle of doing other work, will try and get some better data later today/tonight. Full repro steps on a fresh wsl ubuntu instance, with some comparison strace between relying on binfmt and explicitly calling qemu-arm-static. I might also try the ETL logs to see if they show anything, and perhaps a native ubuntu comparison if I have time.
You've absolutely identified the missing element though: When calling qemu-arm-static explicitly, there is a call to open("/bin/bash", O_RDONLY) which returns FD 3 that doesn't happen through the binfmt assisted call.

Windows version:
Microsoft Windows [Version 10.0.17120.1]
WSL App 'Ubuntu' version:
1604.2017.922.0
Steps to reproduce on a fresh WSL Ubuntu instance:
sudo apt install debootstrap qemu-user-static
sudo service binfmt-support start
sudo debootstrap --verbose --arch=armel --variant=minbase --foreign stretch debian-armel/
sudo cp /usr/bin/qemu-arm-static debian-armel/usr/bin/
sudo mount --bind /dev debian-armel/dev
sudo mount --bind /dev/pts debian-armel/dev/pts
sudo mount --bind /proc debian-armel/proc
sudo mount --bind /sys debian-armel/sys
sudo strace -vff -o implicit.strace chroot debian-armel/ /bin/bash

Result (doesn't work):
prepare_binprm: Bad file descriptor
Strace log: https://gist.github.com/Froosh/dc472dc953dd534806d68a1040e4c10e
Alternate explicit qemu-arm-static invocation:
sudo strace -vff -o explicit.strace chroot debian-armel/ /usr/bin/qemu-arm-static /bin/bash
Result (works):
I have no name!@Froosh-Book:/#
Strace log: https://gist.github.com/Froosh/267e1c2db65e0359e56034610cf328f0
The main difference (apart from memory addresses/times/etc) is a missing call to open() which returns FD 3.
Implict invocation:
gettid()                                = 169
fstat(3, {st_dev=makedev(0, 2), st_ino=1407374884004467, st_mode=S_IFREG|0755, st_nlink=1, st_uid=0, st_gid=0, st_blksize=4096, st_blocks=2000, st_size=1021372, st_atime=2018/03/16-12:47:13.221254200, st_mtime=2017/05/16-05:45:32, st_ctime=2018/03/16-12:47:13.268238900}) = 0
geteuid()                               = 0
getegid()                               = 0
read(3, 0x7fffc8464780, 1024)           = -1 EBADF (Bad file descriptor)

Explicit invocation:
gettid()                                = 175
open("/bin/bash", O_RDONLY)             = 3
fstat(3, {st_dev=makedev(0, 2), st_ino=1407374884004467, st_mode=S_IFREG|0755, st_nlink=1, st_uid=0, st_gid=0, st_blksize=4096, st_blocks=2000, st_size=1021372, st_atime=2018/03/16-12:47:13.221254200, st_mtime=2017/05/16-05:45:32, st_ctime=2018/03/16-12:47:13.268238900}) = 0
geteuid()                               = 0
getegid()                               = 0
read(3, "\177ELF\1\1\1\0\0\0\0\0\0\0\0\0\2\0(\0\1\0\0\0\300\327\2\0004\0\0\0"..., 1024) = 1024

The main difference ... is a missing call to open() which returns FD 3.

Thanks for the repro. Possibly ptrace() is dropping ENOSYS which has come up before; although that's pretty weird on an open().
I haven't looked at the problem in detail yet, but at your first apt install step there's the usual invoke-rc.d: could not determine current runlevel on Setting up binfmt-support (2.1.6-1), which is a rocky start out of the gate.
The fact armel "works" is pretty damn cool though. I take back "almost certainly isn't going to fly", because, the silly pig flies. From inside the chroot you can do:
I have no name!@DESKTOP-4UTIQSF:/# /usr/bin/qemu-arm-static /usr/bin/lscpu
Architecture:          armv7l
CPU op-mode(s):        32-bit, 64-bit
[...]

Whatever the problem, I strongly suspect it is going to be generic to binfmt-support not qemu-arm. That's the place to look.

Okay -- @benhillis, here is a repro that is a little easier to eat than a full blown debootstrap:
$ sudo apt install qemu-user-static
$ # make the binfmt-support postinstall script happy just to be safe, but I doubt it matters
$ sudo bash -c "echo '[1] [00049] [~~  ] [runlevel] [~           ] [4.4.0-17115-Micoroso] [<IP_ADDRESS>        ] [Wed Feb 28 13:27:14 2018 STD]' | utmpdump -r > /var/run/utmp"
$ sudo apt install --reinstall binfmt-support
$ sudo service binfmt-support start # registers qemu-arm binfmt
$ cat /proc/sys/fs/binfmt_misc/qemu-arm
enabled
interpreter /usr/bin/qemu-arm-static
flags: OC
offset 0
magic 7f454c4601010100000000000000000002002800
mask ffffffffffffff00fffffffffffffffffeffffff
$ wget https://github.com/andrew-d/static-binaries/raw/master/binaries/linux/arm/file
$ hexdump ./file | head -n 2
0000000 457f 464c 0101 0001 0000 0000 0000 0000
0000010 0002 0028 0001 0000 340c 0001 0034 0000
$ chmod 755 ./file
$ ./file ./file
prepare_binprm: Bad file descriptor

The strace -f -o file.strace ./file ./file looks the same as reported, including the absent open().
mmap(NULL, 151552, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f2b1e090000
gettid()                                = 890
[the open surely happens here]
fstat(3, {st_mode=S_IFREG|0755, st_size=194976, ...}) = 0
geteuid()                               = 1000
getegid()                               = 1000
read(3, 0x7ffffa628160, 1024)           = -1 EBADF (Bad file descriptor)

I don't think the fact that qemu-arm-static is the registered interpreter matters here. That much appears to be working fine.

WSL's builtin binfmt support doesn't seem to like the fancy flags: OC. Or that's how it looks. So from the top with repetition from Froosh:
$ sudo apt install debootstrap qemu-user-static
$ sudo debootstrap --verbose --arch=armel --variant=minbase --foreign stretch debian-armel/
$ sudo cp /usr/bin/qemu-arm-static debian-armel/usr/bin/
$ # pop the interpreter as registered by qemu-user-static package
$ sudo update-binfmts --package qemu-user-static --remove qemu-arm /usr/bin/qemu-arm-static
$ # add it back without any special options
$ sudo update-binfmts --install qemu-arm /usr/bin/qemu-arm-static --magic '\x7f\x45\x4c\x46\x01\x01\x01\x00\x00\x00\x00\x00\x00\x00\x00\x00\x02\x00\x28\x00' --mask '\xff\xff\xff\xff\xff\xff\xff\x00\xff\xff\xff\xff\xff\xff\xff\xff\xfe\xff\xff\xff'
$ sudo mount --bind /dev debian-armel/dev
$ sudo mount --bind /dev/pts debian-armel/dev/pts
$ sudo mount --bind /proc debian-armel/proc
$ sudo mount --bind /sys debian-armel/sys
$ # don't forget WSL's binfmt_misc filesystem
$ sudo mount --bind /proc/sys/fs/binfmt_misc debian-armel/proc/sys/fs/binfmt_misc
$ sudo cp -af /etc/resolv.conf debian-armel/etc  # we'll want this for apt
$ sudo chroot debian-armel/ /bin/bash

And, after doing the second stage and installing a few packages with the armel apt binary, would you look at that:

I'm getting Unsupported syscall: 384 (notably on lsb_release, shrug) but otherwise it is working surprisingly well. Performance seems alright to the naked eye.
But this isn't about armel now. I have more nefarious plans in mind for this.

WSL's builtin binfmt support doesn't seem to like the fancy flags: OC. Or that's how it looks.

Ref #2103 (message), which I should have looked for harder before I started. If the flags: OC is supported, then it must be something else and removing/re-installing the interpreter works despite itself.

Awesome work!
Agreed, some experimentation shows it doesn't work with flag 'O' or flag 'C' separately, but does work without either (no flags).
ARMel appears to be working nicely, though MIPSel segfaults with access errors after running a command (e.g. ls) successfully. That's a matter for a separate issue though, when I've got more time to test and document the various architectures ;)
Fulfilling the law of WSL issues: now if only we could load loop devices and/or other filesystems (FUSE!) …
Love the idea of using qemu to get 32-bit compatibility running, inspired!

some experimentation shows it doesn't work with flag 'O' or flag 'C' separately

That makes #2103 really weird. 🤷

Fulfilling the law of WSL issues

Yep.

Love the idea of using qemu to get 32-bit compatibility running, inspired!

If qemu is in your wheelhouse (it isn't especially in mine) a cookbook on how to get x86 emulation working in situ would be valuable to a number of people. Clearly it is doable, even if not necessarily out of the box. With a little more work, maybe even some other software that still depends a lot on x86 legacy code.

Your workaround of removing and readding ARM binfmt suport works for me!  Thanks for your great work on this!

WSL's builtin binfmt support doesn't seem to like the fancy flags: OC. Or that's how it looks.

This was never enough of a showstopper to get grease since 2017. The binfmt flags support is complete in WSL2.


[Docs] Add summary page for extensions

Description
Adding a summary page to the documentation to explain the options for customising Vizro dashboard content
Screenshot
Notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.
I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorized to submit this contribution on behalf of the original creator(s) or their licensees.
I certify that the use of this contribution as authorized by the Apache 2.0 license does not violate the intellectual property rights of anyone else.
I have not referenced individuals, products or companies in any commits, directly or indirectly.
I have not added data or restricted code in any commits, directly or indirectly.

I've linted the PR for you @Joseph-Perkins  - a couple of spelling issues, which were immediately raised by vale 🥳
Also vale doesn't like if you use words such as easily or quickly 😄

Also vale doesn't like if you use words such as easily or quickly 😄 I guess because it's too subjective?

Exactly, and we don't want them to come across as marketing either.
I've made a start at the copy in a commit, but I really don't have the opportunity to finish this now, so will pick it up later. You can see the kind of language changes I'm making though: we need to remove the developer marketing and stick with facts :)
Also, just as a note, we don't "allow" we "enable" because it's not about permission but about facilitation 💅

@Joseph-Perkins @huong-li-nguyen I've made quite a few changes to the copy so have added myself to the PR and probably shouldn't approve it. Please could you check what's there now and make any final changes/tweaks before merging?

Looks good to me! I am just wondering whether we should simplify our intro to customisations. What I would like the user to take away is that they can embed any Dash component, Plotly chart or Dash callback into Vizro, even if it's not available in our library as a pre-defined model. So everything they can do with Dash is in theory possible with Vizro.

@huong-li-nguyen I've had a go at this (see https://vizro--622.org.readthedocs.build/en/622/pages/user-guides/extensions/ for latest) but I don't think I'm going to be the most effective at editing beyond this as I can't be sure I'm grouping the different customisation methods correctly.
Please could you, or @Joseph-Perkins, take a look at the sections you think could be simplified and have a go? I can always reword again if needed, but let's get your expert take on it before I do that!


Added describe command

Added a binding to run kubectl describe and display the output for the selected object. I find it useful for some object types such as HPA's. In the future it may be worth adding colorization to the output, but there are already separate plugins for that out there.

Hi @theonlyames,
Thanks for this PR.
I'm completely fine with it, but I would prefer to use Ctrl-d for "describe" (easier to remember it).
In order to not conflict with "delete" (which currently use ctrl-d) I was thinking to move delete command to alt-d. I prefer this because delete can be a destructive command, so I think it is better to use another key like alt for this one.
What do you think ?

Hi @theonlyames, Thanks for this PR.
I'm completely fine with it, but I would prefer to use Ctrl-d for "describe" (easier to remember it). In order to not conflict with "delete" (which currently use ctrl-d) I was thinking to move delete command to alt-d. I prefer this because delete can be a destructive command, so I think it is better to use another key like alt for this one.
What do you think ?

That sounds fine to me. I'll make those changes in a moment.


Refactor Project to be usable without too much async

Don't think this is necessary.

Don't think this is necessary.


question about 5-point detection

Hi, your work is fantastic. I found some datasets with ground truth, why not use the existing ground truth to carry out 5-point detection again?

Good question!

No particular reasons actually, we just detect all the dataset to get a unified definition for our convenience. You can use the original ground truth to process the data for your own purpose.


Update Contacts test coverage to use pre-recorded HTTP requests

Trello: Replace API calls with VCR - Contacts
Summary:
Creating an contact included a timestamp in the email address to ensure
creating a contact doesn't violate email constraints. Since the email
address is included in the HTTP request, VCR is unable to match the
request with a pre-recorded request. This results in VCR issuing a new
request each time this test is run.
This PR:

Removes including a timestamp in the contact's email address so VCR
can use a pre-recorded request.
Updates the tests for .createOrUpdate to be self-sufficient (creates
necessary records, runs the expectations, deletes created records)
introduces a contacts directory for VCR cassettes

I made a small syntax change that I forgot to push, which altered my local commit but wasn't reflected here, which is why this PR didn't automatically close when I merged from the command line. Sorry about that. Closing this PR since the changes have been merged.


support multiple gitlab servers

(so that users can track projects one more than one gitlab server)

current workaround is to use one config per gitlab server, e.g. glim --config glim-corporate.toml, glim --config glim-personal.toml.


Change project's name

Apparently the name "aulang" is already used for a programming language here. We'll need to rename this project to something else, preferably a word starting with "au" (since the C API is already prefixed with au_ and I don't want to go back and rename everything).
If anyone has any idea please comment it here.
(Source)

I have chosen the name, Aument.


fix: return all expected temporal entities + switch back to version 1.11.2 for detekt #334

@bobeal, I updated here the returned format for temporalValues representation.

@bobeal, I updated here the returned format for temporalValues representation.

seems good.


dynamic imports do not work for defining models

Using dynamic imports in order to define models does not work. It works fine when manually declaring the imports, but I'd prefer just being able to create model files in a directory and my app knowing they exist dynamically.
db.js:
import { Sequelize } from 'sequelize'
import env from './env.js'

const db = new Sequelize(
    env.DB_NAME,
    env.DB_USERNAME,
    env.DB_PASSWORD,
    {
        host: env.DB_HOST,
        port: env.DB_PORT,
        dialect: env.DB_DIALECT,
        logging: false,
    },
)

export default db

server.js:
import fg from 'fast-glob'
import path from 'path'
import app from '../providers/app.js'
import db from '../providers/db.js'
import env from '../providers/env.js'

fg.sync([
    './app/models/**/*.js',
    './app/routes/**/*.js',
]).forEach(async (file) => {
    await import(path.resolve(file))
})

await db.sync({ alter: true })

app.listen(env.APP_PORT, () => {
    console.log(`App listening on ${env.APP_HOST}:${env.APP_PORT}`)
})

The models do not get defined this way. Whats funny is my routes for express are done in the same way, and they work fine.
If I change it to non-dynamic imports, the sequelize definitions work fine:
import '../models/user.js'
import '../routes/index.js'

Why doesn't sequelize pick up the definitions when done via dynamic imports as opposed to manual imports?

forEach is sync and cannot return a promise, so you're not awaiting the async callback passed to it. Use array.map and Promise.all

forEach is sync and cannot return a promise, so you're not awaiting the async callback passed to it. Use array.map and Promise.all

Thank you, I didn't even think of that. My bad for blaming sequelize.
I changed my code to a for instead and it works:
const files = await globby([
    '../models/**/*.js',
    '../routes/**/*.js',
], {
    cwd: './app/entries',
})

for (const file of files) {
    await import(file)
}

await db.sync({ alter: true })


ReorderableListView Adds Huge Gap To Bottom

When using ReorderableListView and the list becomes long enough to become scrollable, there is an added gap to the bottom of the ReorderableListView, which seemingly cannot be remedied. Does the same without the use of Scrollbar, but using it to show how much further it goes down in the screenshot.
The gap that is produced is the same size regardless of the size of the widgets in the list (Tested using Card with a ListTile child).
Steps to Reproduce
return Scrollbar(
    child: ReorderableListView(
        onReorder: (oIndex, nIndex) {},
        children: List.generate(100, (index) => Text('test', key: Key('$index')),
        padding: Elements.getListViewPadding(),
    ),
);

Screenshot

Platform
Target Platform: iOS
Target OS version/browser: 13.3 developer beta 4
Devices: iPhone 11 Pro
Flutter Doctor
[✓] Flutter (Channel stable, v1.12.13+hotfix.5, on Mac OS X 10.15.2 19C57, locale en-CA)
    • Flutter version 1.12.13+hotfix.5 at /Users/jagandeepbrar/Development/flutter
    • Framework revision 27321ebbad (2 days ago), 2019-12-10 18:15:01 -0800
    • Engine revision 2994f7e1e6
    • Dart version 2.7.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/jagandeepbrar/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.2.1, Build version 11B500
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 40.0.2
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] VS Code (version 1.40.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.7.1

• No issues found!

I was able to workaround this issue by duplicating the class and modifying this line in the reorderable_list.dart file:
static const double _defaultDropAreaExtent = 100.0;

Setting it to 25.0 gives me a larger than wanted bottom margin (but better), but still allows for an item to be dropped to the bottom position of the list.

Just feel like leaving this here - ReorderableListView desperately needs API expansion. It's very limited as-is.


Option to include process exit code in the status segment

Done in https://github.com/apjanke/agnosterj-zsh-theme/commit/c928dc66f9e71bbf785f8a890a8b24ed06aed18c.


Any plans to support SAML?

http://en.wikipedia.org/wiki/SAML_2.0
https://github.com/rohe/pysaml2

Other python library:  https://github.com/onelogin/python-saml

As a general mission for this project, do you think it would be worthwhile to incorporate the corporate login providers such as OneLogin and Okta into python-social-auth? I think the standard for corporate space is to use SAML, so perhaps integrating SAML would then give this project more corporate value.

@omab I am likely going to be implementing a python-social-auth SAML backend in the near future as part of bringing Shibboleth support to the software I work on (edX). It will depend on python-saml (linked two comments up), which in turn has a few other requirements and is only compatible with Python 2.7. Would you be willing to include that backend in python-social-auth, given the extra requirements and lack of Python 3 support? We'd love to contribute it so others can use it, if possible.

@omab @bradenmacdonald I'm working on a python3 version of the python-saml.
https://github.com/pitbulk/python3-saml
Other alternative is to use pysaml2

@pitbulk Glad to hear that; I wasn't sure of the status of that fork and if it was going to be made "official" at some point.

Yes, I plan to publish it soon, now is in testing phase. Feedback welcome ;)

@bradenmacdonald I am looking at creating a SAML backend for python-social-auth as well - would you be interested in collaborating?

@ashchristopher I have already more or less finished an implementation: https://github.com/omab/python-social-auth/pull/616 . Just waiting for review and feedback. Check it out and I'd be curious to know if it works for your use case or not.

Apparently a few of abstract methods are still not implement in pyramid_strategy.py to make saml work with pyramid. They are request_post(), request_get(), request_path(), request_is_secure(). Would be great to see this fixed. Thanks.

@vsobolmaven Those should be quite simple to implement - I recommend you implement that yourself and open a pull request here to contribute it to the project.

I was wondering why @omab closed this issue but from reading https://github.com/omab/python-social-auth/issues/778#issuecomment-269242569 it sounds like SAML is supported these days. Sure enough, pull request #616 by @bradenmacdonald was merged a while back. Great! Thanks!


Tile loading not a concurrent request

Hi!
why is tile loading not a concurrent request?
As shown in the figure below, there is a certain time difference between each  image request, which is about 16ms. I guess it is related to requestAnimationFrame.

In the second picture, the request for the last tile is delayed by nearly 1s from the loading time of the first tile map.

In extreme scenes, it takes a long time to render the tiles in certain viewing areas.

How to make tiles concurrently request. Thank you!

When OSD was first built, the architectural decision was made to trigger a new image load every frame (if needed). This allows us to keep up with dynamic movement (we're always asking for the most important tile of the moment), and it allowed us to not overwhelm the internet connections of the day. Nowadays we have more bandwidth (in general, though not always, especially on mobile), so it might be interesting to consider a different approach. It would be a big architectural change, though, so it'll require a lot of consideration.
Given the way the system works (and the higher bandwidth of today), my recommendation is to use larger image tiles. Originally the suggestion was 254px (with a 1px overlap which pads some tiles out to 256). Nowadays I recommend 510 or even 1022. You can see each of your tiles are approximately 25k... you could easily take that up to several hundred K and still have good loading speed.

ok,Thank you for your suggestion.
Indeed, in my scene, the level scale of the digital slice is 0.25, and the difference in the number of pictures between the tiled images is 16 times.This relies on the digital slice itself.
If it is not possible for the time being, I will try the method you provided.Use a larger picture.

So you're getting that tile size direct from your imagery source? Good to know that that's a factor.
I think this is a fine place to discuss the possibility of concurrent requests, but yeah, that's not going to be a short-term thing.
For anyone who wants to look into it in the code, it's the logic that leads up to this line:
https://github.com/openseadragon/openseadragon/blob/6cb2c9e7bc4adebe28e386a093890a6c3e353c6b/src/tiledimage.js#L1179
I imagine we would want to be able to set a "concurrent load count" variable, so instead of picking just a single "best" tile every frame it would pick the "top N tiles".

The "concurrent load count" variable sounds very good, so that we can set the appropriate number of concurrency according to different scenarios. Because in some of our scenarios, slices are stored in the intranet, so the network is sometimes not a bottleneck.
How to do this quickly? Are there any suggestions for modification so that I can try the effect.

I can't think of a way to test it without kind of actually doing it.
I suppose one really crude initial test could be to just load every single tile that gets considered as a possible bestTile. That might be a bit much but you might learn something!
BTW, come to think of it, "concurrent load count" isn't quite the right terminology, since we are loading tiles concurrently (assuming it takes more than a single frame to load a tile), we are just not starting more than one load per frame. I'm not sure the best way to express that.

Thanks. I'll give it a try.


Forms not rendering after Hide/Show with GlobalUseMetal

With Skia4Delphi 6.0.0-beta3 on macOS (13.3.1) when using GlobalUseSkia with GlobalUseMetal (FMX project) every form except for the main form will not render any updates after closing (click on the cross) and reopening the form (Form.Show).
The form does still react to mouse inputs it seems, but it will not update/rerender any content.

I tested it on Sunday with the new release (v6.0.0-beta4) and I was not able to reproduce it. Could you try using the new version? If you can reproduce using the new version, could you attach a simple demo? Also, what version of RAD Studio are you using?

I still have the same issue with v6.0.0-beta4 with the simple example attached. I use Rad Studio 12 with macOS 13 SDK and test the app on a mac mini m1 (arm64) with macOS 13.3.
demoskia.zip
Steps to reproduce:

start the application
click onto the "Show Form2" Button
scroll the content of the memo and use the slider
close the form
click again onto "Show Form2"
try to play with the slider again (the form caption will change (so obvisously events are still working) but you won't see any change in the window content) or try to scroll the memo content

Tested on macOS Sonoma 14.0 on ARM m2 and latest RAD 12 Version 29.0.50291.5583.
I couldn't see the problem. It is working as expected here.

Can you upload the app? Then I can test whether the fact that it is working for you (but not for me) has to do with the macOS version (in which case there would still be a problem as I think that macOS 13 should still be supported) or with changes in RadStudio 12 (can't test it myself as I did not join the beta, so I can't compile with Rad Studio 12 yet...).

Additional info: on PAServer is showing a new message:
"Project1[899:9771] Each CAMetalLayerDrawable can only be presented once!"

Oh, I confirmed the problem on RAD 11.3.

thanks for testing (by the way: FMX Metal was never working for me with themes (lots of rendering artifacts) while Skia Metal is rendering correctly (except for this bug)). Looking forward to Rad Studio 12 then ;)

As previously mentioned, the issue was fixed in RAD 12. Unfortunately, it's an internal FMX bug where, even though Skia is using Metal, when the FMX canvas isn't used, the system believes that it's not using Metal.
This bug was previously reported by us on the quality portal: https://quality.embarcadero.com/browse/RSP-40435
If you have the RAD Studio version with source code access, you can patch the file FMX.Platform.Mac.pas as reported on quality portal (remember to add it to your project).
However, we recommend updating to RAD 12 once it's released.
Best regards.


Investigate error warping some files generated with nilearn

Only some subjects have files with problems being warped. So far it's 2 out of the first 20 subjects I tested my warping script on. However, it looks like it affects all files created with nilearn, including:

desc-optcomDenoised_errorts
desc-optcomMIRDenoised_errorts
desc-GODEC_rank-4_bold
desc-GODEC_rank-4_lowrankts
desc-lfofilterCleaned_errorts
desc-aCompCor_bold
desc-aCompCor_errorts
desc-GSR_bold
desc-GSR_errorts
desc-NuisReg_bold
desc-NuisReg_errorts

Most of these are created with my _generic_regression function, which uses nilearn's NiftiMasker.
The only ones that I don't think were created with that are the GODEC ones, which are created with nilearn's unmask here.
All of the files created by tedana, rapidtide, and fMRIPrep seem fine.
I ran the denoising step with nilearn version 0.8.0.

Here's an example of the antsApplyTransforms error:
Exception caught during reference file reading

itk::ExceptionObject (0x39fafa0)
Location: "unknown"
File: /home/software/ANTs/ANTs-2.3.5/ANTs-2.3.5/build/ITKv5/Modules/IO/NIFTI/src/itkNiftiImageIO.cxx
Line: 1972
Description: itk::ERROR: ITK only supports orthonormal direction cosines.  No orthonormal definition found!

 file /home/data/nbc/misc-projects/Salo_PowerReplication/dset-cambridge/derivatives/tedana/sub-20503/func/sub-20503_task-rest_desc-optcomMIRDenoised_errorts.nii.gz
terminate called after throwing an instance of 'itk::ExceptionObject'
  what():  /home/software/ANTs/ANTs-2.3.5/ANTs-2.3.5/build/ITKv5/Modules/Core/Common/src/itkProcessObject.cxx:1339:
itk::ERROR: itk::ERROR: ResampleImageFilter(0x39ffd80): Input Primary is required but not set.

Since this comes up with the rapidtide errorts output, which only uses math_img, I think the problem must come down to one of the following functions:

check_niimg
_safe_get_data
new_img_like

I ran a quick test on these three functions. Basically, I took the tedana output from one of the subjects that failed and created new files that had only been touched by one of the three functions. I also included one that was only touched with nibabel as a control.
Here's the Python code I used to create new files that should only be touched by one of the functions:
import nibabel as nib
from nilearn._utils.niimg import _safe_get_data
from nilearn._utils.niimg_conversions import check_niimg
from nilearn.image.image import new_img_like

in_file = "sub-20503_task-rest_desc-optcomDenoised_bold.nii.gz"
img = nib.load(in_file)

# Test nibabel-only operation
data = img.get_fdata()
data = data ** 2
nib_only_img = nib.Nifti1Image(data, img.affine, img.header)
nib_only_img.to_filename("tedana_nibabel.nii.gz")

# Test check_niimg
check_niimg_img = check_niimg(in_file)
check_niimg_img.to_filename("tedana_check_niimg.nii.gz")

# Test _safe_get_data
data = _safe_get_data(img)
_safe_get_data_img = nib.Nifti1Image(data, img.affine, img.header)
_safe_get_data_img.to_filename("tedana__safe_get_data.nii.gz")

# Test new_img_like
img = nib.load(in_file)
data = img.get_fdata()
new_img_like_img = new_img_like(img, data, img.affine)
new_img_like_img.to_filename("tedana_new_img_like.nii.gz")

And here's the ANTS script to apply the xforms to each of these files:
#!/bin/bash
module load ants-2.3.5

# Run xforms on tedana-generated file. Should be good.
antsApplyTransforms -d 3 \
    -i sub-20503_task-rest_desc-optcomDenoised_bold.nii.gz \
    -r sub-20503_task-rest_space-MNI152NLin6Asym_boldref.nii.gz \
    -o tedana_out.nii.gz \
    -n NearestNeighbor \
    -t sub-20503_task-rest_from-scanner_to-T1w_mode-image_xfm.txt sub-20503_from-T1w_to-MNI152NLin6Asym_mode-image_xfm.h5

# Run xforms on tedana + nibabel.

I will probably open an issue in the nilearn repo about changing the default for copy_header from False to True, but the next release won't be until January and there's no guarantee the other devs will want to make the change, so I will need to (1) create a new branch of my nilearn fork with the parameter changed, (2) clone it to the HPC, (3) install the fork version in the Power environment, (4) remove all denoised files, (5) re-run the denoising job, and (6) run the warping job.

I'm now up to re-running the denoising job. It shouldn't take more than a day or so.

sub-20503 successfully warped this time(!), so I'm going to try warping everything else.

Only the problematic runs from #15 are missing the warped files, so it's all good now.


Implement changes in Valora to inform users about reward to invite others

Based on designs

Target release: 1.11


Auto-add message recipients

Allow admins to designate members who should always be included on any messaging

Implemented!


Provide Unix Build Instructions + Scripts

Currently the mechanism to install KVM requires me to run some commands that download bits and pieces from the internet.
ASP.NET needs to have a Unix build setup that would:

Configure a source checkout for build and installation
Build the required bits from a pure source checkout
Install the required bits into the system and any auxiliary scripts

This is the way that most Unix packages are distributed nowadays.   And they have been turned into a set of well established idioms, that go like this:
 ./configure [options]
 make
 make install

Bonus points if you use GNU autoconf/automake, as it would give you:

Makefiles with standard targets (https://www.gnu.org/prep/standards/html_node/Makefile-Conventions.html)
Packaging of the source code suitable for bundling on Unix distributions
Build scripts suitable for clean reproducible builds, required by both Debian and RPM package managers

I think we should look into packaging dnvm (Debian and RPM). I'm not convinced me need to do the
./configure
make
make install

Pattern for our managed projects. We use a build.sh instead.

If you're thinking of doing .deb and .rpm files, it would make sense to change how DNX does it's installations today. For example in both Windows and Linux, it's stored in a .dnx folder under the current user's home path.
I don't know of a .deb or .rpm package that has ever installed on a per-user basis, they are all instead installed system wide.
I would imagine that things should get installed to a system wide directory (like the dnvm script and the runtimes), but the choice of active/default should be stored in a home path configuration directory?

How about something like this? The Docker image builds the .rpm and .deb packages using the current version of the DNVM script as of this posting.
https://github.com/wholroyd/dnvm

Both packages install DNVM at a system level into /usr/local/bin/dnx/

That seems wrong... By definition /usr/local/ is only for things that were not installed via packages. A package should install executables into /usr/bin/ and architecture-independent files into /usr/share/, or everything into a subdirectory of /opt/.

You're right, I'll change it.

Updated my repo

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


Why do we have while true in STARTUP action

// process STARTUP actions
export function * watchStartup () {
  while (true) {
    yield take(Types.STARTUP)
    const temp = yield select((state) => state.weather.temperature)
    // only fetch new temps when we don't have one yet
    if (!R.is(Number, temp)) {
      yield put(Actions.requestTemperature('San Francisco'))
    }
  }
}

Does it have to be inside while(true) block? given that the action is dispatched only once?

[x] Issue does not already exist
Identify exactly what aspect your issue is about

[x] ignite base application
[ ] ignite the cli
[ ] ignite generated code

You're right.  The while loop could come out.

@eladnava - I see a commit to something, but no clue what it is?   Was this going to be a PR from you?

@GantMan yeah, but I couldn't find the time to fill out the PR form.

k, I might just take care of it then.  Cheers!


fix: Allow overwrite of default avatar in comments.

Related:

#2468

Issue
Default avatar overrides could not be overwritten by the pre_get_avatar_data filter.
Solution
Set default  arg as well, while at it, add arguments more the WordPress way in avatar_out
Impact
None, this only affects sites that override the default avatar.
Usage Changes
none
Considerations
The new default url is fed into Gravatar and then returned. Ideally you would like to check if an avatar returned by gravatar is a default one and if so, overwriting the resulting url with the default url, thus not resulting in a url like:
http://0.gravatar.com/avatar/a5f7937f4bc10cfe01abb42fe5504da9?s=92&d=https%3A%2F%2Fupload.wikimedia.org%2Fwikipedia%2Fcommons%2Fthumb%2F1%2F15%2FCat_August_2010-4.jpg%2F1920px-Cat_August_2010-4.jpg&r=X
where the d arguments holds the overwritten default. This would lead to less dependance on Gravatar as a whole but I did not found a good way to do that.
Testing
no

coverage: 88.548% (+0.03%) from 88.522% when pulling 24d76d07fe19cd8b01b533e079d84892ba127319 on 2468-comment-avatar-functions-ignore-$argsdefault into f56f062cd56a706a24848915bb554615bc215542 on 2.x.


[Script]:

Aim
Movie ticket booking automation
Details
->The user gives the details of  theatre, date, and number of tickets needed
-> python script checks the availability for the same and books the tickets before any other in order to bag tickets for high in demand movies as soon as the booking are open.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

@rmoyano @altirel @SethWalkeroo @Ayushjain2205

@avinashkranjan please assign it me ASAP. I'm looking forward to contributing to your amazing repo!


Have two buttons in the bottom bar instead of skip and next

Hi, please can you please guide me how to put two buttons at the bottom of the screen instead of skip and next ?

Can you not close an issue and then open another one in the same topic?
Reopening an issue is perfectly fine if the issue itself was: 1) yours 2) not resolved 3) under the same topic.


Cosmetic. Tiny copy tweaks

Before/After
Get started BECOMES Get Started
How it works BECOMES How It works
If there’s a match, the app displays a COVID Alert. BECOMES If there’s a match, the app will display a COVID Alert.
You can change your mind at any time by going to App Settings BECOMES You can change your mind at any time by going to App Settings.
Yes I'd like to start my log BECOMES Yes, I’d Like to Start My Log
Your symptoms data helps them understand how COVID-19 is affecting NY. BECOMES Your symptoms data helps them understand COVID-19 and how it’s affecting NY.
All responses are optional. They will only be used anonymously for public health analysis. BECOMES All responses are optional and anonymous. The info is used only for public health analysis.
I am feeling well today BECOMES I'm feeling well
Nausea or Vomiting BECOMES Nausea or vomiting
Your settings will be visible here as soon as you do your first log. BECOMES Basic details you provide about yourself, such as your age and gender, can be changed here after you make your first log entry.
Separate yourself BECOMES Separate yourself.
Stay in a room that is not shared BECOMES Stay in a room that's not shared
If not possible, clean and disinfect all surfaces after each use. BECOMES If you must share one, clean and disinfect all surfaces after each use.
Throw away used tissues.  (No special handling is needed) BECOMES Throw away used tissues. Bag your trash and leave it outside your door. (No special handling is needed.)
Wash your hands often BECOMES Wash your hands often. //ADD BREAKLINE
with soap and water for 20 seconds each time, or if unavailable, use hand sanitizer with at least 60% alcohol. Use paper towels instead of shared towels BECOMES Wash with soap and water for 20 seconds. If that’s not possible, use hand sanitizer with at least 60% alcohol. Dry with paper towels instead of shared towels.
Talk to your Healthcare provider BECOMES Talk to your healthcare provider
call NYS Health Hotline BECOMES Call NYS Health Hotline
If You test positive for COVID-19 BECOMES If You Test Positive for COVID-19
Share your close contact codes BECOMES Share Your Close Contact Codes
@floridemai @ahurworth

For visual reference.
COVID copy fixes - 8_31.pdf

fixed, avail in next build

Checked 4 of them. Looks good to me.


Out of Date Screenshots

Hello,
Just wanted to let you know that the data type screenshots on this page are out of date. For example, the query editor data type menu image does not show an option for "Percentage"--which is a choice that is now in that menu.
Ben

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b83479d4-0fb8-7500-e6ec-5559c4e2fdae
Version Independent ID: 23b93f03-3ab9-d658-4243-89ebebbd6796
Content: Data types in Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-data-types.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @bgribaudo. We know we have screen updates for that article, since Power BI Desktop is always marching ahead with new UI elements and features, especially lately. We’ve forwarded your feedback to the writer, who should respond to this thread shortly. Thanks!

Thanks again for your comment - I've updated those screens to reflect the current UI.


Print all keyboard shortcuts to cheatset

Spyder has may good shortcuts. But they are well hidden in the preferences dialogue
Tools --> Preferences --> Keyboard shortcuts?
Solution:
add a print (to PDF) function to the keyboard settings so that users can have these open on a separate file while learning the programme.
References:
https://code.google.com/archive/p/spyderlib/issues/578
https://github.com/spyder-ide/spyder/issues/578
What is the expected output? What do you see instead?
Please provide any additional information below
Versions and main components

Spyder Version: 3.0.dev
Python Version: 2.7
Operating system: linux, ubuntu mint 17.3

Dependencies
jedi >=0.8.1     :  0.9.0 (OK)
matplotlib >=1.0 :  1.5.1 (OK)
nbconvert >=4.0  :  4.0.0 (OK)
numpy >=1.7      :  1.10.4 (OK)
pandas >=0.13.1  :  0.17.1 (OK)
pep8 >=0.6       :  1.7.0 (OK)
psutil >=0.3     :  3.3.0 (OK)
pyflakes >=0.5.0 :  1.0.0 (OK)
pygments >=2.0   :  2.0.2 (OK)
qtconsole >=4.2.0:  4.2.1 (OK)
rope >=0.9.4     :  0.9.4 (OK)
sphinx >=0.6.6   :  1.3.1 (OK)
sympy >=0.7.3    :  <IP_ADDRESS> (OK)

Would a shortcut to display a summary of shortcuts help? Like Ubuntu when leaving pressed the "windows" key

@goanpeca This would be perfect!

@rlaverde

I'm working in this issue, this is the current state, any opinions? improvements? Which shortcut will display the dialog?

@rlaverde, this is really nice!
Just a couple of comments:

The Pylint and Profiler sections look too big, but other than that, I like it very much!
Please test this in other platforms to see how it looks,
This list should be printable somehow. I mean, we could add a Print button to the dialog and a printable representation associated with it. @goanpeca, what do you think?

Hi @rlaverde, thanks for working on this :-)
@ccordoba12 on your comments:

Yes, hmmm maybe if the column rows are smaller than the max allowed, we could add a stretch (layout.addStretch), so that they line up and dont waste space

I can see how it looks on windows and OSX as soon as @rlaverde has a PR

That is a great idea :-p, but Probably we would need to reformat the content?, maybe for another PR cause it might need some extra tweaking?

@ccordoba12 as initial requester I would be happy with the quick display. Maybe saving as image file. But well formatted printing is not needed if let's say WIN+F1 brings this panel up very quickly.

What I could use, but don't see in the displayed shortcuts, is Insert Cell Below, which I constantly use to experiment with things, get info, or correct an error in the cell above

Cells are simply created by comments of the form #%%, so I don't think that needs a shortcut.


How to generate the index files like referit_train.pth in data.tar?

Hi,
Thank you for the great work!
When I run the training process for referit dataset, I notice the indexs of training data are loaded from referit_train.pth which is extracted from data.tar you provided. We will obtained a list for every mask after loading referit_train.pth. The length of this list is 5, and my question is for the meaning of the fifth item in the list. For example, the fifth item is a list [('r1', ['bag']), ('r2', ['black']), ('r3', ['none']), ('r4', ['right']), ('r5', ['none']), ('r6', ['none']), ('r7', ['none']), ('r8', ['none'])] when the query is 'right black bag'. I cannot understand how to generate a list like it. Would you please explain its generating process? Thanks a lot!

Hi, thank you for the question :)
I assume the question is how to get the parsed query like "[('r1', ['bag']), ('r2', ['black']), ('r3', ['none']), ('r4', ['right']), ('r5', ['none']), ('r6', ['none']), ('r7', ['none']), ('r8', ['none'])]."
We obtain these parsed results based on Licheng's parser in this repo: https://github.com/lichengunc/refer-parser2. They provide both the parsed results and the parser in case that you wish to try it on other datasets.

Thanks a lot for your detailed explanation. I observe that this parsed query has no contribution to the training (if my understanding is correct). I wonder if the parsed query could be directly ignored in the code of ReSC, supposing one wants to train the model with customized datasets.

Yes, you can safely ignore these parsed queries. I use them for results analyses only.

Get it. Your explanation help me a lot. Thank you very much.


Issue with ClearCache(url)

Hi,
I was playing with Davinci, but I've realized that the function ClearCache(url) doesn't delete anything.
Indeed, on line #628 there is this :
File.Delete(filePath + CreateMD5(url));
I don't get why a new CreateMD5 is necessary? I've changed it using the uniqueHash (making the associated changes) and now it is working properly. Did I miss something about the clearing cache process? Don't know if it is a bug or it has been made on purpose, while I don't get anything.
Thanks for your answer!
Have a nice day,
Chloé.

Hi Chloé! Right now I can't help because I'm having some troubles with my computer, once I sort that out I'll check this!

Hey!
thanks a lot!
No worries, I've realized taht I've mixed up things, and in fact,
everything is working properly!
I'll do an edit and I'll closed the issue :)
Have a nice day!
C.
On 3/15/2022 1:24 PM, TechiePi wrote:

Hi Chloé! Right now I can't help because I'm having some troubles with
my computer, once I sort that out I'll check this!
—
Reply to this email directly, view it on GitHub
https://github.com/shamsdev/davinci/issues/30#issuecomment-1067926228,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC5ZWESRTHXRTA557MBZ3YDVAB6O3ANCNFSM5QICEOSQ.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


[Feature] Use bloom filters to detect request of expired notes

Improved UX: user knows that the link has expired instead of containing a typo
Good perf: check if note ID has ever existed in O(1) time

Use e.g. https://www.npmjs.com/package/bloom-filters

Shipped in #18


Enable KaTeX behind an option flag

@csuwildcat Any thoughts? This is for a new spec at https://github.com/decentralized-identity/revocation

I like what you're doing here, but can we make sure the assets (JS) being used are not linking to external endpoint, and instead that they are baked into the page inline?

@andrewwhitehead @csuwildcat -- what is the status of this PR?  We could really use the ability to have Latex in our spec.  Is this required for that?
Thanks!

Yeah, this would probably do it, but I was asking that we have the CSS bundled inline so specs can render without dependency on some outside site to serve the styles.

Another request for completing this.  Sorry I'm not sure what is needed, or I would do it.

Updated now so that the katex flag is specified in the spec definition. When enabled, the fonts directory is copied from the katex package and would need to be distributed along with index.html.

Another of weekly pings for @csuwildcat about this.  Are you happy with what Andrew has done?  Enough to merge this PR?  This would be a big a help for spec. writers.
Thanks

this looks like it has the files baked in and not external now. can you take a look @csuwildcat?

@csuwildcat --please review and (if appropriate) merge this PR.

Ok, looking today to make sure the external file dependency part was corrected, will merge if so.

You were so close yesterday, @csuwildcat.  Maybe today?

@swcurran @andrewwhitehead was there any way to reuse the mechanism that the other files are being pulled from based on entries added to this file mapping object?: https://github.com/decentralized-identity/spec-up/blob/master/src/asset-map.json. I ask because I see fetching code and special asset grabbing code that I would hope doesn't need duplication. If the one I have that pulls on the other assets can't work, could it be made to grab the KaTeX assets with some modification?

Here's where that mapping object is digested and used to pull in the assets is lists: https://github.com/decentralized-identity/spec-up/blob/master/gulpfile.js#L30-L50. The map is basically just file paths grouped by type and location of injection, which the Gulp runner grabs, compresses, and minifies to insert at either the head or end of body, depending on where you need it.

@csuwildcat I believe the JS and CSS can be added to the current asset groups instead of being included with this method, but that wouldn't allow for optional inclusion (processing the document with KaTeX could still be optional). The KaTeX assets are already minified so I don't imagine the document size would be much different.

@andrewwhitehead -- can you please resolve the conflicts?
@csuwildcat -- once resolved, can this please, please be merged?  We could really use this feature.

Looks like the changes have been made and this is ready to merge @csuwildcat

Go  @csuwildcat  Go!!!

Yay!

I'll probably refactor to make these kinds of things abstract and more pluggable in a future release, but my failure to find the time these last few months shouldn't block you.


[BUG] terraform-provider-rancher2 harvester disk_size default value is not set correctly

Describe the bug

if users use the Rancher Terraform provider with the "new style" disk_info and network_info, they will see the following changes after reapply
	# (6 unchanged attributes hidden)  ~ harvester_config {
	  - disk_size    = "0" -> null
		# (6 unchanged attributes hidden)
	}

To Reproduce
Steps to reproduce the behavior:

Setup a Harvester v1.1.2 cluster, refer to https://docs.harvesterhci.io/v1.1/install/iso-install
Add a cloud image harvester-public/focal-server and a vlan network harvester-public/mgmt-vlan1 to the Harvester by using terraform-provider-harvester

terraform {
  required_version = ">= 0.13"
  required_providers {
    harvester = {
      source  = "harvester/harvester"
      version = "0.6.2"
    }
  }
}

provider "harvester" {
 kubeconfig = "<the kubeconfig file path of the harvester cluster>"
}

resource "harvester_image" "focal-server" {
  name      = "focal-server"
  namespace = "harvester-public"

  display_name = "focal-server-cloudimg-amd64.img"
  source_type  = "download"
  url          = "https://cloud-images.ubuntu.com/focal/current/focal-server-cloudimg-amd64.img"
}

data "harvester_clusternetwork" "mgmt" {
  name = "mgmt"
}

resource "harvester_network" "mgmt-vlan1" {
  name      = "mgmt-vlan1"
  namespace = "harvester-public"

  vlan_id = 1

  route_mode           = "auto"
  route_dhcp_server_ip = ""

  cluster_network_name = data.harvester_clusternetwork.mgmt.name
}

Setup a Rancher 2.7.4 server
Import Harvester cluster to the Rancher cluster in Virtualization Management, use  foo-harvester as the cluster name
Create a guest RKE2 cluster using the following test config:

terraform {
  required_providers {
    rancher2 = {
      source = "rancher/rancher2"
      version = "3.0.1"
    }
  }
}

provider "rancher2" {
  api_url    = "<change me>"
  access_key = "<change me>"
  secret_key = "<change me>"
  insecure = true
}

data "rancher2_cluster_v2" "foo-harvester" {
  name = "foo-harvester"
}

# Create a new Cloud Credential for an imported Harvester cluster
resource "rancher2_cloud_credential" "foo-harvester" {
  name = "foo-harvester"
  harvester_credential_config {
    cluster_id = data.rancher2_cluster_v2.foo-harvester.cluster_v1_id
    cluster_type = "imported"
    kubeconfig_content = data.rancher2_cluster_v2.foo-harvester.kube_config
  }
}

# Create a new rancher2 machine config v2 using harvester node_driver
resource "rancher2_machine_config_v2" "foo-harvester-v2" {
  generate_name = "foo-harvester-v2"
  harvester_config {
    vm_namespace = "default"
    cpu_count = "2"
    memory_size = "4"
    disk_info = <<EOF
    {
        "disks": [{
            "imageName": "harvester-public/focal-server",
            "size": 40,
            "bootOrder": 1
        }]
    }
    EOF
    network_info = <<EOF
    {
        "interfaces": [{
            "networkName": "harvester-public/mgmt-vlan1"
        }]
    }
    EOF
    ssh_user = "ubuntu"
    user_data = "I2Nsb3VkLWNvbmZpZwpwYWNrYWdlX3VwZGF0ZTogdHJ1ZQpwYWNrYWdlczoKICAtIHFlbXUtZ3Vlc3QtYWdlbnQKICAtIGlwdGFibGVzCnJ1bmNtZDoKICAtIC0gc3lzdGVtY3RsCiAgICAtIGVuYWJsZQogICAgLSAnLS1ub3cnCiAgICAtIHFlbXUtZ3Vlc3QtYWdlbnQuc2VydmljZQo="
  }
}

resource "rancher2_cluster_v2" "foo-harvester-v2" {
  name = "foo-harvester-v2"
  kubernetes_version = "v1.25.9+rke2r1"
  rke_config {
    machine_pools {
      name = "pool1"
      cloud_credential_secret_name = rancher2_cloud_credential.foo-harvester.id
      control_plane_role = true
      etcd_role = true
      worker_role = true
      quantity = 1
      machine_config {
        kind = rancher2_machine_config_v2.foo-harvester-v2.kind
        name = rancher2_machine_config_v2.foo-harvester-v2.name
      }
    }
    machine_selector_config {
      config = {
        cloud-provider-name = ""
      }
    }
    machine_global_config = <<EOF
cni: "calico"
disable-kube-proxy: false
etcd-expose-metrics: false
EOF
    upgrade_strategy {
      control_plane_concurrency = "10%"
      worker_concurrency = "10%"
    }
    etcd {
      snapshot_schedule_cron = "0 */5 * * *"
      snapshot_retention = 5
    }
    chart_values = ""
  }
}

terraform init
terraform apply

Check plan again

terraform apply

Run terraform plan

Expected behavior

the state of disk_size should not be changed after reapply
Support bundle

Environment

Harvester ISO version: 1.1.2
terraform-provider-harvester version: 3.0.1
Rancher version: 2.7.4
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630):

Additional context
Add any other context about the problem here.

Verified this bug has been fixed.

Test Information

Environment: qemu/KVM single node
Harvester Version: v1.2.0-rc2
ui-source Option: Auto
Rancher Version: v2.7.4
terraform-provider-rancher2 Version: https://github.com/rancher/terraform-provider-rancher2/pull/1149

Verify Steps

Install Harvester with any nodes
Create a image for VM creation
Create a VM network
Login to Rancher, import Harvester into Rancher named harv
Use terraform to pervision RKE2 cluster[^1]

Save the code snippet as rke2.tf
Update relevant fields which having <> in the code snippets
When newer version released, we can simply modify rke2.tf to update version, Or build from the PR fixing the bug[^2]

terraform {
  required_providers {
    rancher2 = {
      source = "rancher/rancher2"
      version = "> 3.0.1"
    }
  }
}

Execute terraform plan, field disk_size should be "0"

RKE2 cluster foo-harvester should be provisioned successfully
Execute terraform apply again, field disk_size should NOT be modified

[^1]: Code snippet for initial RKE2 cluster
```tf
terraform {
required_providers {
rancher2 = {
source = "terraform.local/local/rancher2"
version = "0.0.0-dev"
}
}
}
provider "rancher2" {
  api_url    = "<API_URL>"
  access_key = "<TOKEN>"
  secret_key = "<SECRET>"
  insecure = true
}

data "rancher2_cluster_v2" "foo-harvester" {
  name = "harv"
}

# Create a new Cloud Credential for an imported Harvester cluster
resource "rancher2_cloud_credential" "foo-harvester" {
  name = "foo-harvester"
  harvester_credential_config {
    cluster_id = data.rancher2_cluster_v2.foo-harvester.cluster_v1_id
    cluster_type = "imported"
    kubeconfig_content = data.rancher2_cluster_v2.foo-harvester.kube_config
  }
}

# Create a new rancher2 machine config v2 using harvester node_driver
resource "rancher2_machine_config_v2" "foo-harvester-v2" {
  generate_name = "foo-harvester-v2"
  harvester_config {
    vm_namespace = "default"
    cpu_count = "2"
    memory_size = "4"
    disk_info = <<EOF
    {
        "disks": [{
            "imageName": "<IMAGE_NAMESPACE_UID>",
            "size": 40,
            "bootOrder": 1
        }]
    }
    EOF
    network_info = <<EOF
    {
        "interfaces": [{
            "networkName": "<VMNETWORK_NAMESPACE_UID>"
        }]
    }
    EOF
    ssh_user = "<SSH_USER>"
    user_data = "I2Nsb3VkLWNvbmZpZwpwYWNrYWdlX3VwZGF0ZTogdHJ1ZQpwYWNrYWdlczoKICAtIHFlbXUtZ3Vlc3QtYWdlbnQKICAtIGlwdGFibGVzCnJ1bmNtZDoKICAtIC0gc3lzdGVtY3RsCiAgICAtIGVuYWJsZQogICAgLSAnLS1ub3cnCiAgICAtIHFlbXUtZ3Vlc3QtYWdlbnQuc2VydmljZQo="
  }
}

resource "rancher2_cluster_v2" "foo-harvester-v2" {
  name = "foo-harvester-v2"
  kubernetes_version = "v1.25.9+rke2r1"
  rke_config {
    machine_pools {
      name = "pool1"
      cloud_credential_secret_name = rancher2_cloud_credential.foo-harvester.id
      control_plane_role = true
      etcd_role = true
      worker_role = true
      quantity = 1
      machine_config {
        kind = rancher2_machine_config_v2.foo-harvester-v2.kind
        name = rancher2_machine_config_v2.foo-harvester-v2.name
      }
    }
    machine_selector_config {
      config = {
        cloud-provider-name = ""
      }
    }
    machine_global_config = <<EOF
cni: "calico"
disable-kube-proxy: false
etcd-expose-metrics: false
EOF
    upgrade_strategy {
      control_plane_concurrency = "10%"
      worker_concurrency = "10%"
    }
    etcd {
      snapshot_schedule_cron = "0 */5 * * *"
      snapshot_retention = 5
    }
    chart_values = ""
  }
}
```

[^2]: Code snippet to build version 0.0.0-dev from rancher/terraform-provider-rancher2#1132
- prerequisite: golang version > 1.13, we can simply use gvm to install new version
bash     git clone https://github.com/rancher/terraform-provider-rancher2     cd terraform-provider-rancher2     make
Then move the custom provider to terraform local search path
bash     PROVIDER="rancher2"     VERSION="0.0.0-dev"     OS_PLATFORM=$(uname -sp | tr '[:upper:] ' '[:lower:]_' | sed 's/x86_64/amd64/' | sed 's/i386/amd64/' | sed 's/arm/arm64/')     PROVIDERS_DIR=$HOME/.terraform.d/plugins/terraform.local/local/${PROVIDER}     PROVIDER_DIR=${PROVIDERS_DIR}/${VERSION}/${OS_PLATFORM}     mkdir -p ${PROVIDER_DIR}     cp bin/terraform-provider-${PROVIDER} ${PROVIDER_DIR}/terraform-provider-${PROVIDER}_v${VERSION}
To be awared, your platform might encouter the issue, we can simply update the folder name:
bash     cd $PROVIDER_DIR/..     # then use `uname -a` to find the correct platform name, and rename the `linux_unknown` folder