Add SVCCA Analysis

This pull request adds two notebooks that encompass one analysis.
I apply singular vector canonical correlation analysis (SVCCA) to the sample z matrices across PCA, ICA, NMF, DAE, and VAE solutions within each ensemble z independently.
The first notebook performs the within z analysis and outputs the results. The second notebook performs the across z analysis. Lastly, the third notebook visualizes all of these results.
Note that the SVCCA code is not pip installable, so I add download instructions here 7901bef. Also note that the original SVCCA code is note compatible with python 3. Therefore, I forked/updated (https://github.com/gwaygenomics/svcca), and this is the version pulled here.
Note - I need to add a README to this module to describe what I'm doing. It is getting out of hand quick! (see #28)

Thanks for the comments @ajlee21 ! I will try to answer below

Say we have z from model A using VAE and z from model B using VAE. If my latent space is of dimension 1 then the SVCC is calculating some correlation score between my two z vectors to determine if the two z vectors output similar values as a measure of how similar the two latent spaces are (would need to think about this more, not sure if that is the best representation -- thinking that the weight matrix for how much each gene contributes to the latent features would make more sense).

Yeah, I think it might make sense to perform on the weight matrix, but I think with an important caveat. In the original paper, the authors use the sample activation outputs (z in our case) as input to SVCCA. I think the benefit is that it is scale independent and thus does not depend on the nonlinearity (so linear and nonlinear methods can be compared more easily).  The weight matrix may be influenced in this way.
Also, I am using SVCCA to assign model stability. If model A and model B output similar (but scaled differently in some way) activations from the same input, then I can make some statement on how stable the solutions are.

Within meaning comparing z's from models that are from the same algorithm (like the example above)? And Across meaning between different algorithms? For the across algorithms are your z's being compared so that the first feature in z from algorithm A is compared against the first z from algorithm B? If that's the case I'm thinking that order matters a lot and so I'm not sure how fair of a comparison this is.

Not quite what I am doing here. I say within z to mean I am comparing across algorithms, but within the same bottleneck dimensionality. I can make statements about how stable z itself is with this approach. across z means within the same algorithm, but across different bottleneck dimensionalities. I am interested in how similar the activations are when z in increased. Do we find the same patterns? When do patterns split? What do they split into?

Are these SVCCA correlation calculations performed pairwise between two z's and then the score is averaged across your 5 models?

All scores for all seeds are plotted.


Keyboard navigation doesn't work on a input with initialValue

When the Searchfield has a entered value,

Focus the Searchfield
use arrow keys to navigate suggestions

expected behavior: Suggestion can be navigated with arrow keys
actual behavior: Nothing happens

This issue seems, bit Funky unable to reproduce it most of the times.

@maheshj01  sir when  i close the keyboard all the suggestion showing from scratch not showing hat i have entered char in the textfield
https://drive.google.com/file/d/132iZYWgLzDNMllLGyOlkA15SayzkrcZe/view check this

@sabari7320  Please file a separate issue with all the details

Regression identified https://github.com/maheshj01/searchfield/commit/0fddae486c8538307b9719e9230739733d5db378#diff-0d4d907e62a927a9256ddab70f46e08912d122e3b1634116aa27498cc43a2db8R447

Fixed in latest release v1.1.9


Security issue fixes

Stored Cross Site Scripting via Filename Fixed
Directory Traversal Fixed
Upload of arbitrary files Fixed
DB file locking while saving data Fixed

https://github.com/robiso/wondercms/pull/210 closing pull request, a new one was made with a few other fixes.


config: support some flags in daemon config file

Fixes: #1003
Signed-off-by: Ace-Tang<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

LGTM, CI failed due to known issue


[Backport - Mitaka] MaaS: Use maas_external_ip_address properly

The maas_external_ip_address variable was ignored in the rpc_maas
role. This patch sets the variable to external_lb_vip_address as a
default and allows a deployer to override the default if needed.
Manual backport of 0ffd8af from master. The master branch has some
Ansible 2 updates that aren't needed for Mitaka.
Connects rcbops/u-suk-dev#1036

recheck_ceph
recheck_upgrade

recheck_upgrade
Timeout while waiting for the ElasticSearch port to be open.

recheck_upgrade

Stuck on rcbops/u-suk-dev#1065.

recheck_upgrade

Rebased the patch after the elasticsearch blocker was cleared.


Intellisense reports type of imported modules as "import <...>" instead of actual type

VSCode Version: 1.8.0-insider
OS Version: Mac OSX 10.11

Where TStep is defined, Intellisense reports the type as React.Requirable<any>

Closing as duplicate of https://github.com/Microsoft/TypeScript/issues/11099
Thanks.


Update resources in index.html

Docs Build status updates of commit 082496c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/powerpoint/index.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


graceful exit

[ ] Use SIGSTOP signal to end tasks graceful
[ ] SIGUSR1 and SIGUSR2 to speed up and slow down the process by reducing or increasing the random sleep time. s. here


*: update go-oidc to remove dependency on capnslog

#543 relieved that we're suppressing the standard logger in dex due to some capnslog side effects. The actual code at fault was not tracked down, but removing the dependency entirely fixes the issue. Since capnslog is largely deprecated, this seems like a reasonable step to take.

This also updates #473. Apparently we were just completely suppressing outside packages (e.g. go-oidc).


500 server error when generating large dataset PDF

We had an issue in the past with trying to generate a pdf with around 3000 meeting listed. We get a 500 server error. The issue was resolved at one point but now we are experiencing the same issue again.
The pdf is for a state wide printed meeting list that we use for H&I facilities and prison release packages.
The last time it happened I believe you determined it was an issue with overloading the code4recovery servers while generating a pdf from the large amount of data. We're running WP 6.4.3. TSML plugin 3.14.32 on area03.com.

Can I ask what the use case is for printing a list so large? It seems like a single list with meetings hundreds of miles apart would t be that useful to someone trying to find a meeting.
Perhaps we could help you break the list down into smaller regions?

The primary reason we do a statewide list is that they get distributed
around the state to various institutions such as treatment centers and
prisons to which the the discharge/releasee may or may not be
re-entering the local community in which they are being
discharged/released.
They also may not have access to, or know how to fully utilize a digital
device to immediately find meetings. The list is a resource that gives
them a way to find meetings in whatever community they end up in after
discharge or release.
As an example, Valley Wide H&I which encompasses most the Phoenix metro
area distributes around 1200 of them per year and other various H&I,
Bridging the Gap and Corrections Committees around the state go through
just as many.
One of the reasons we really liked the TSML plugin was the capability of
a novice level Meeting List Coordinator to maintain the list easily
online, AND be able to generate the lists for printing without needing
to have an advanced level of knowledge, or software, to be able compile
the list in a print ready format.
It worked beautifully while it was working.
We are currently out of the printed lists and I have orders for more
from various entities and need to get another printing going ASAP.
Hopefully this helps clarify our need to be able to generate the
statewide list for Arizona meetings.
Thanks,
Greg S.
Area 03 Meeting List Coordinator
On 3/2/2024 11:07 PM, Anthony Baggett wrote:

Can I ask what the use case is for printing a list so large? It seems
like a single list with meetings hundreds of miles apart would t be
that useful to someone trying to find a meeting.
Perhaps we could help you break the list down into smaller regions?
—
Reply to this email directly, view it on GitHub
https://github.com/code4recovery/pdf/issues/68#issuecomment-1975055695,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYV37EIG6JP7FXO624J6LF3YWK4YJAVCNFSM6AAAAABECFSPXSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZVGA2TKNRZGU.
You are receiving this because you authored the thread.Message ID:
@.***>

Can you send the feed url you're using to generate with? I can probably just generate one for you to deal with your immediate need. Meantime, we can discuss options internally for handling this. Being that we are not for profit, we have to work with our limited server resources.
Thanks!

We found that the memory limit increase we had put in place a while back was reverted when we changed PHP versions. We have fixed the issue. Can you try your feed again and see if you see an error?


chore: don't use global cache

close #1

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-main.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


added support for changing indicator start position

this change allows making two way seekers.  one can create seeker with a range between -100..100 and make the reference point center.

I think a better name would be progressStartPosition instead of the indicatorStartFraction. What do you think?

sure. why not?

I think a better name would be progressStartPosition instead of the indicatorStartFraction. What do you think?

I will release a new version with this feature very soon.


Adjust clock face to 24h format

Hi!
I've added code in exec() and a watcher to picker.meridien to adjust the clock face to 24h format if picker.params.shortTime is false. Please review!
Thanks!
Matthias

Alright, seems to make a lot of sense! I wasn't aware of this approach.


Setting wallpaper silently fails

On a fresh Ubuntu 16.04.1 netinstall, after build + install + config of latest wlc and sway, the wallpaper remains unset and displays only a black screen. This occurs when using a single color wall as well.
Log: https://spit.mixtape.moe/view/27812180
Conf: https://spit.mixtape.moe/view/0066fe57
Relevant snippets:
09/11/16 20:47:35 - handling config command 'output LVDS-1 res 1366x768 pos 0,0 bg ~/pictures/wallpapers/eva-01.jpg fill'
09/11/16 20:47:35 - find_handler(output) 0
09/11/16 20:47:35 - Config stored for output LVDS-1 (enabled:-1) (1366 x 768 @ 0, 0 scale 1) (bg /home/hardware/pictures/wallpapers/eva-01.jpg fill)
09/11/16 20:47:35 - handling config command 'output HDMI-A-1 res 1920x1080 pos 1366,0 bg ~/pictures/wallpapers/eva-01.jpg fill'
09/11/16 20:47:35 - find_handler(output) 0
09/11/16 20:47:35 - Config stored for output HDMI-A-1 (enabled:-1) (1920 x 1080 @ 1366, 0 scale 1) (bg /home/hardware/pictures/wallpapers/eva-01.jpg fill)

later...
09/11/16 20:47:36 - Setting background for output 0 to /home/hardware/pictures/wallpapers/eva-01.jpg
09/11/16 20:47:36 - Setting background for output 1 to /home/hardware/pictures/wallpapers/eva-01.jpg

but background remains solid black.

Hi @qlyoung,
I'm finding it nigh-on impossible to install Sway on Ubuntu; I can't get WLC to install.  There are no packages for it, and the requirements information is skeletal at best.  I did my best dredging up the various dependencies, and am now stuck at a bunch of undefined references during linking libwlc, and Google fails to find any references to most of the missing symbols.
So: any pointers on how you got as far as getting Sway to actually install?  This is beyond bleeding edge, and I understand that. It's built on a foundation of bleeding edges, from Wayland on up.
P.S.  This question has nothing to do with this ticket, but since Github removed any ability to direct message a user, this is about the only way to get a message to somebody.  Sorry about that.

@serussel, The IRC channel for sway is probably the best place for questions like this. It is #sway on irc.freenode.net and you can join the webchat here. However, they would probably tell you to read the wiki, because there is already a page dedicated to installing on Debian based systems which I think is what you want. Good luck!

@serussell I can confirm that following the wiki page for Debian based systems linked by @4e554c4c , starting from an Ubuntu Minimal install, is how I accomplished this.

@qlyoung try converting the image to png and using that — that fixed it for me. It should be documented that png is assumed unless using gdk_pixbuf.

Just tried it on the same version of Sway that I opened this issue with and yep, png's work just fine. Thanks a bunch!
I'll close out this issue since you've got a PR open for it.


Separation (or mark) of disabled add-ons into built-in and additional ones

At the moment we have this division:

In the current situation, built-in add-ons that are turned off mix with the turned-off add-ons that i install from the market by myself

It would be good if the disabled built-in add-ons have some indication that they are built-in add-ons
Maybe some small badge similar to add-ons marked with an asterisk as recommended?
Or maybe just stop showing built-in disabled add-ons in Show disabled extensions
Or maybe any different separation way?
I request about it because if you have many add-ons that are turned off, this is a bit confusing

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Update rust-secp256k1 and rust-secp256k1-sys

Closes #57
I removed the unstable feature because it was removed from rust-secp256k1. Let me know if we should keep this.
secp256k1::secp256k1_sys re-export was removed, but it seems that ffi re-export is the same thing?

Superceded by #60. Thank you!


Add example to the documentation about setting up a simple environment using Ocelot

In this ticket, the documentation (Getting Started) should be extended by a simple example on how a simple setup can be made using e.g. InfluxDB, Jaeger/Zipkin and the inspectIT agent and how to collect traces with it.

This should also shortly note what is included in the new 1.3 default configuration (e.g. tracing)


fix Issue 10669 - CTFE: using initialized static const class member no longer works

ensure semantic is run on initializer (this does not happen if the variable is a member of an aggregate).
http://d.puremagic.com/issues/show_bug.cgi?id=10669

This pull request introduced a regression:
https://issues.dlang.org/show_bug.cgi?id=12624


Implement Routing logic from JS Connector in Java

This should be in its own module called ilp-routing.
See JS impl here: https://github.com/interledgerjs/ilp-connector/pull/416

Closing this as it has been completed in the WIP connector. See here: https://github.com/sappenin/java-ilpv4-connector/tree/feature/ilp-http


Find via Props for Mounted Components

Within a mounted component, I am trying to differentiate between like elements by evaluating the difference in props. I.e. two <Link/> elements that differ by URL path:
Component JSX:
<LeftDrawer {...props} >
  <Link to="/">Link Name</Link>
  <Link to="/another-place">Another Link Name</Link>
</LeftDrawer>

This works (shallow):
const wrapper = shallow(<LeftDrawer {...minProps} />);
expect(
        wrapper.find({ to: '/' }).length
      ).toEqual(1);

This fails (mount):
const wrapper = mount(<LeftDrawer {...minProps} />);
expect(
        wrapper.find({ to: '/' }).length // Expected 0 to equal 1
      ).toEqual(1);

This works (shallow):
const wrapper = shallow(<LeftDrawer {...minProps} />);
expect(
       wrapper.find('Link[to="/"]').text()
      ).toEqual('Link Name');

This fails (mount):
const wrapper = mount(<LeftDrawer {...minProps} />);
expect(
       wrapper.find('Link[to="/"]').text() // This method is only meant to be run on single node. 0 found instead.
      ).toEqual('Link Name');

I was able to reproduce the issue with  find({to: '/'}) for mount here.

@joncursi so the reason you're seeing this disparity is because mount checks the props against the returned DOM node. So, for example, if your to prop is being rendered as an href value on an anchor element, doing:
wrapper.find({href: '/'})
``

would work. Shallow checks against the instances actual props instead, which is why its working. The `mount` API is typically used to validate the render output against a DOM, so it makes some sense that there's a disparity here. On the other hand, it's less confusing if both `find()` worked as similarly as possible.

@Aweary hmm, running that snippet on a mounted component didn't work for me:
const wrapper = mount(<LeftDrawer {...minProps} />);
expect(
        wrapper.find({ href: '/' }).length // Expected 0 to equal 1
      ).toEqual(1);

can someone post a gist of a basic test case here? From what I see in this issue, this sounds like a bug.  You should be able to call find('[to="foo"]') on a ReactWrapper and have it find something if there is a higher order component like <Link to="foo" /> in the render tree.  If it doesn't, that's wrong.

@lelandrichardson you can see an example here: https://github.com/Aweary/enzyme-test-repo/blob/issue-%23376/test.js
There is an isDOMComponent check that is being called in instMatchesObjectProps which is returning false for the MockLink instances. I'm guessing that's probably the issue.

@lelandrichardson I added this failing test case on my branch
    it('should probably not break with MockLink (?)', () => {

      class MockLink extends React.Component {
        render() {
          const to = this.props.to;
          const children = this.props.children;
          return (
            {children}
          );
        }
      }

      const wrapper = mount(

          <MockLink to="/">First</MockLink>
          <MockLink to="/second">Second</MockLink>

      );

      expect(wrapper.find({ to: '/' })).to.have.length(1);

    });

👍  Looks good! I did notice that clicking the org logo in the side nav causes the page to reload. Seems like it's not hooked up to react-router

Hi guys, do you know if there is any workaround while this issue gets fixed? Thanks!

@DanielRamosAcosta I’ve found a workaround involving findWhere:
wrapper.findWhere(node => node.props().to === '/')

I'm going to close this; please file a new issue if it's still a problem on v3.


Check return values, log errors, and more tests

This resolves #10138

[approve ci cmake]

I'll approve this when CI passes.


Build failed on Windows+Intel oneAPI

System:
Windows 10, VS 2019, Intel oneAPI 2022, lastest master branch
Command line used:
cmake -B build -G Ninja -DCMAKE_C_COMPILER=cl -Dparallel=false
Error:
CMake Error at CMakeLists.txt:69 (add_subdirectory):
add_subdirectory given source "/libseq" which is not an existing directory.

This is fixed in v<IP_ADDRESS> thanks


[Accessibility]A11y_VSCode_ViewExplorer_Keyboard: Keyboard focus goes to list box items as a whole (div containing all list items) which is non-interactive.

Environment Details:
VSCode Version : 1.24.0
OS Version : HighSierra10.13.5
Additional Details:
MAS Violated : MAS 2.1.1
Repro Steps:

Launch VS Code.
Navigate to Activity Bar and select "Explorer" button.
Navigate to "File Explorer"/ "Open Editors" list box items using "Tab".

Actual:
Keyboard focus moves to overall list box items which is non-interactive. Then using downward arrow keys focus goes to items in list box.
Expected:
Keyboard focus should not move to the items as a whole which is non-interactive. The focus should move to first item in the list box.
Recommendations:
Remove tab-index from the outer div tag that contains all the list box items.
or,
Refer below link which is repository of bug fixes code snippets:
https://microsoft.sharepoint.com/teams/msenable/mas/pages/browse-fixes.aspx
User Impact:
The keyboard only users will move to non-interactive elements on the screen which will be time consuming to reach only interactive elements on the page.
MAS Reference:
MAS 2.1.1 - https://microsoft.sharepoint.com/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={8492c4eb-c179-40ae-8777-cd044ed725a2}
Attachment For Reference:
A11y_VSCode_ViewExplorer_Keyboard_Mac_KFOnListbox.pptx
Does this issue occur when all extensions are disabled?: Yes

Dup of https://github.com/Microsoft/vscode/issues/52000


IVS-45/TAS001 Polygonal face boundary no self intersections

@evandroAlfieri
TAS	Tessellated (i.e. meshes)
 ^       ^

is this intended?

replaced by #298

is this intended?

TAS fits well with the long A sound in the second syllable. Maybe it was not originally intended that way but I see no need to revise.


HorizontalWrapLayout unable to scroll to bottom

HorizontalWrapLayout come from the POS MAUI Demo we reused it in our project with the benefit to have the layout similar to the gridview in WPF and WinUI.
Problem: This layout won't allow the scrollviewer reach to the bottom
Usage: MAUIAllControlsPage.xaml
Layout:

HorizontalWrapLayout
HorizontalWrapLayoutManager

@duydh1995  has fixed with Pull Request https://github.com/Strypper/mauisland/pull/70
CONGRAT !!!!!


Fix UID Windows

Correction of generating uid for Windows and installing does not add kool on Windows startup.

As in Windows we don't have uid, use sid and get the last number, example: S-1-5-21-3623811015-3361044348-30300820-1013
So, in this case, the uid used is 1013, this solve the problem to use root,  but it still has problems as in other cases that we saw in linux in docker images
Fix the installer was putting the kool to boot with windows start
Add arguments to build_artifacts, sample: ./build_artifacts.sh --version 1.6.0

This is related to #42

@danielsuguimoto you can review and then merge!


Syntax error in type annotations

In the light of first inline type annotations, a static type check via mypy was requested in https://github.com/pytorch/vision/pull/2034#pullrequestreview-385008031.
Running mypy on the current torchvision results in syntax errors in several type annotations. For example:
torchvision/io/_video_opt.py:86: error: syntax error in type comment '(List[int])'

Since syntax errors are fatal they cannot be silenced even with ignore_errors set. Furthermore, mypy stops the checking after it encountered a module with syntax errors.
Thus, if we want to include mypy checks in CI and test if they work we need to either fix them or disable them temporarily and fix them later. Since they seem to be related to torch.jit, I have no idea how much work fixing them would be or how much damage temporarily disabling them will do.

I've noticed similar issues as well in the past, and I tried to fix some of those in https://github.com/pytorch/vision/pull/1696
We need those type annotations as those functions are being used with TorchScript, so we need to find a way to make them compliant with TorchScript and mypy.
cc @eellison for thoughts.

You never merged the PR although @eellison approved it. Does that mean the way you did it does not work with TorchScript? If adding the return type as you did does indeed work with TorchScript could you rebase and merge your PR. I would take up the rest.

That PR had failing tests, and I never had the chance to get back to it and fix it so that it is compatible with mypy and torchscript.
@zhangguanheng66 are you still looking at fixing type annotations in #1696?

I think the the output annotation Tuple[Tensor, Tensor] is not supported by torchscript. @eellison

This is fixed by #1696 and #2195.


GDScript highlighting

Language
GDScript is the scripting language used in the Godot game engine. It would be very useful for things like tutorials and gamedev blogs to have support for this language.
Additional resources
Godot website: https://godotengine.org

Here is a language introduction:


trigger filter

requested by fei user: start logging at a certain message, 'clear' re-arms the trigger.

It doesn't sound complicated, but the next question will probably be: ok, I would like to see N messages before the trigger. (pre-trigger)  And the next question might be: ok, we want to trigger on a certain sequence of messages. I not sure we want to build a protocol-analyzer :)


Help with TheHive and MISP intigration

Problem Description
When I try to export a case or observables from TheHive to MISP i cant get it working.
Steps to Reproduce.

export case to MISP

it gives an error as follows

error: CaseExportCtrl: an unexpected error occured while exporting case.
more information...

also look at my thehive application.conf
play.modules.enabled += org.thp.thehive.connector.misp.MispModule

misp {
   interval: 1m
   servers: [
{
     name = "MISP"
     url = "http://...... MISP URL"
     auth {
       type = key
       key = "---key----"
     }
     wsConfig {}
     wsConfig.ssl.loose.acceptAnyCertificate = true
}
  ]
 }

similar issue to other user
#https://github.com/TheHive-Project/TheHive/issues/1063
Pleaae let me know if you can help with this issue, TIA.

I had the same issue while exporting the observables from thehive to MISP, Did you able to solve it?


有demo.py样例吗

@dyhBUPT 你好，运行样例有吗

Hi, sorry and I don't prepare demos. There are some visualization results in our paper.

Hi, sorry and I don't prepare demos. There are some visualization results in our paper.

@dyhBUPT 你好，请问下论文中使用的NKF是在代码中的什么地方

I'm sorry but the codes for MOT (including NKF / NeuralSORT) are not released for now. This repo focuses on the iKUN model.

你好，只有小黑子，没有demo.py样例。

You better be

I'm sorry but the codes for MOT (including NKF / NeuralSORT) are not released for now. This repo focuses on the iKUN model.

Thanks for your contribution. Is there a plan to release it?

I'm sorry but the codes for MOT (including NKF / NeuralSORT) are not released for now. This repo focuses on the iKUN model.

@dyhBUPT Thanks for your contribution. Is there a plan to release it?

Sorry, I don't have time to do this for now. You can try to reproduce the results based on DeepSORT.


Fix typo

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@brianjking can you please sign the CLA?

This was fixed in another pr


Understanding the electronics

Hi
As far as I know both Pro Micro (left and right hand) communicate with each other using serial communication. For this I thought that RX from device 1 needed to be connected to TX from device 2 (and TX from device 1 to RX from device 2)

But in Corne Keyboard electronic design I can see that the connection between both Pro Micro goes from pin RX of one of them to the pin RX of the other one (green line). I read that pin RX can only receive data but can't send. So how do both Pro Micro communicate with each other?

Thanks in advance

Hello, stumbling across this - I would like to confirm for the actual .sch file, but cant' open project's .sch.
Tried Kicad 5.1.5. and some online tools.

Split keyboard communication is implemented by QMK. There is no problem in actually connecting RX1s to each other.

I had to search a bit through th QMK doc to actualy find the technical explication, which can be resumed to:

Serial in this context should be read as sending information one bit at a time, rather than implementing UART/USART/RS485/RS232 standards.

Basically QMK uses one GPIO pin to transmit data serially using a dedicated protocol, and have nothing to do with "classic" UARTs except the unfortunate 'serial' name.
The link given by @foostan is to the source code and quite difficult to understand. Those pages in the documentation are much more helpfull:
https://docs.qmk.fm/#/feature_split_keyboard
https://docs.qmk.fm/#/serial_driver

I'll close it because it's not this project.


Manage Account: Delete Account

From #423: From the 'manage account' page, a logged-in user should be able to delete their account.
If the user is the sole owner of any projects, this should be disabled and they should be informed that they need to transfer ownership or delete the projects before they can delete their account.
A notification email should be sent to the primary email address when the account is successfully deleted.

@di
I've started diving into the code and would like to work on this issue.
I'll get back if I get any questions and will put a WIP PR.

Hi @waseem18, I appreciate your enthusiasm! However I've already begun work on this issue. If you're looking for something relevant to work on, I'd recommend searching for issues in upcoming milestones without an assignee. The project view for this repo might be helpful as well.

Thanks for the information @di

There's a branch for this here: https://github.com/pypa/warehouse/tree/delete-account, but it's on top of branches for #2830 and #2831, which should get shipped first.


allow the grafana service annotations to be customised

In a similar way to the prometheus chart, allow the service annotations to be specified.  This lets us use an internal aws load balancer, for example, by specifying: service.beta.kubernetes.io/aws-load-balancer-internal: <IP_ADDRESS>/0

@k8s-bot ok to test

/lgtm
That's how it's done! Bumped the minor chart version (to v0.4.0) because we're adding functionality. I'd love to see all of the service-related settings (serviceType, loadBalancerIP, nodePort, etc.) move into the services: map, but not a blocker.
Thanks for the contribution!


Should this be archived?

I noticed it hasn't been updated since 2013

I'd like to see it updated. Can we leave open for a while, and if someone on the team doesn't have time to do, I will do it. Update to at least OpenAPI 2.0 if not 3.0.

I'd be down to do it! Can you tell me how? I'd love to automate it with Travis CI :-) @kinlane


Java VR (Dataflow, V2, Streaming) failing: ParDoTest$TimestampTests/OnWindowExpirationTests

Test failures in org.apache.beam.sdk.transforms.ParDoTest in:

TimestampTests: https://ci-beam.apache.org/job/beam_PostCommit_Java_VR_Dataflow_V2_Streaming/1572/
OnWindowExpirationTests: https://ci-beam.apache.org/job/beam_PostCommit_Java_VR_Dataflow_V2_Streaming/1563/

These seem to directly correlate to new tests added in:

https://github.com/apache/beam/pull/15540
https://github.com/apache/beam/pull/15994

I think the TimestampTests are also failing in https://issues.apache.org/jira/projects/BEAM/issues/BEAM-13498, so maybe we can use this to track OnWindowExpirationTests failures

My guess is these shouldn't be running for Portable Runner/FnAPI tests.
Imported from Jira BEAM-13525. Original Jira may contain additional context.
Reported by: emilymye.

These are old jenkins tests, closing


SecurityOption: Changed setting value - Require Signature

Validation strings for the item value do not match the strings that apply to registry.
Setting: Domain_controller_LDAP_server_signing_requirements
Correct values (checked on Server 2016): None, Require signing
Current validation strings: [Write, ValueMap{"None","Require Signature"}, Values{"None","Require Signature"}] String Domain_controller_LDAP_server_signing_requirements;
Current registry strings: "Domain_controller_LDAP_server_signing_requirements" = @{         Value   = "MACHINE\System\CurrentControlSet\Services\NTDS\Parameters\LDAPServerIntegrity"         Section = 'Registry Values'         Option  = @{             'None' = '4,1'             'Require Signing' = '4,2'         }     }
Commit 8ba0cfe seemed to inadvertently introduce the requirement with the incorrect string:
https://github.com/PowerShell/SecurityPolicyDsc/commit/8ba0cfe4c171e8c689b42275b0c13319ac3475f3#diff-0c166052c748e9bb455bb1181a5d65ca
Verbose logs showing the problem
The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: At least one of the values 'Require signing' is not supported or valid for property 'Domain_controller_LDAP_server_signing_requirements' on class 'SecurityOption'. Please specify only supported values:
None, Require Signature.
Suggested solution to the issue
DSCResources/MSFT_SecurityOption/MSFT_SecurityOption.schema.mof
Change Line 30 to:
[Write, ValueMap{"None","Require Signing"}, Values{"None","Require Signing"}] String Domain_controller_LDAP_server_signing_requirements;
The DSC configuration that is used to reproduce the issue (as detailed as possible)
SecurityOption 'Domain Controller Security Options' {
    Name = 'Domain Controller Security Options'
    Domain_controller_LDAP_server_signing_requirements = 'Require Signing'
}

The operating system the target node is running
OsName               :
OsOperatingSystemSKU :
OsArchitecture       :
WindowsBuildLabEx    : 14393.2906.amd64fre.rs1_release_inmarket.190401-180
OsLanguage           :
OsMuiLanguages       :
Version and build of PowerShell the target node is running
Name                           Value

PSVersion                      5.1.14393.2879
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.2879
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
Version of the DSC module that was used ('dev' if using current dev branch)
published (master)

Fixed as part of PR #125


Adding configuration options for ports

Adding values to customize the ports the server listens to. Per https://github.com/lloesche/valheim-server-docker/issues/369 only the game port and game port +1 are in use, so the previous game3 port was unneeded.
Also added relevant docs.

lgtm, merging in and will release a new version of the chart after I test out the rendering.

thanks @bdelwood

@bdelwood Tested it out and added released a new version of the chart.

Thanks! That was quick! Couldn't even write my other PR for the http status changes.


Host.OSType for Mac

The Microsoft documentation is incorrect for the terminology on the OSType property when a Mac is being used. It is noted as returning 'macOS'. My testing confirms it is actually returned as 'Mac OS'.
It is noted as returning 'macOS'. My testing confirms it is actually returned as 'Mac OS'.

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

ID: d166c439-3e39-d35b-f534-a733b28c89d5
Version Independent ID: d166c439-3e39-d35b-f534-a733b28c89d5
Content: Host object in Power Apps - Power Platform
Content Source: power-platform/power-fx/reference/object-host.md
Service: power-platform
Sub-service: power-fx
GitHub Login: @jorisdg
Microsoft Alias: jorisde

Thanks for your comment, @rehkech. This issue has been assigned to the doc author for review.

Hello,
Since that comment (which I appreciate was only very recent!), further discussion on the related thread on the Community reveals there may be cases where it is output as ‘Mac OS’ OR ‘macOS’.
This is the relevant thread: Solved: Re: Determine Current Connection Method - Canvas A... - Power Platform Community (microsoft.com)https://powerusers.microsoft.com/t5/Building-Power-Apps/Determine-Current-Connection-Method-Canvas-App/m-p/2609016/emcs_t/S2h8ZW1haWx8dG9waWNfc3Vic2NyaXB0aW9ufExTT1NUVTdERzFGTVU1fDI2MDkwMTZ8U1VCU0NSSVBUSU9OU3xoSw#M644682
Happy to elaborate further if needed.
Best Wishes,
Edd
From: Bishal Goswami @.>
Sent: Friday, February 16, 2024 4:02 PM
To: MicrosoftDocs/power-platform @.>
Cc: Charnley, Edward @.>; Mention @.>
Subject: Re: [MicrosoftDocs/power-platform] Host.OSType for Mac (Issue #2488)
⚠ Caution: External sender
Thanks for your comment, @rehkechhttps://github.com/rehkech. This issue has been assigned to the doc author for review.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/power-platform/issues/2488#issuecomment-1948733831, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BAS5GZNYXEUC4A3SWWPKL7DYT57F3AVCNFSM6AAAAABDL6QR5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBYG4ZTGOBTGE.
You are receiving this because you were mentioned.Message ID: @.***>

Thank you for sending us the feedback. I updated the topic the topic with the correct value for Mac OS.
Thanks,
Mints


Mounting files into the container in the browser?

An example is given for mounting files into the container when running on a stand-alone WASI runner, but not in the context of the browser.
If I have a File or a Blob in hand in JavaScript when I am setting up the emulator, how would I expose that to the container at a certain path?
Bonus points if I can also mount something writable, like a resizable ArrayBuffer.

I am wondering the exact same thing, would be wonderful if we had the ability to attach files to a container in the browser

@adamnovak For container on WASI on browser configuration, you can provide file or directories as pre-opened fds to the container .

Example1: w/ browser_wasi_shim : https://github.com/ktock/container2wasm/blob/31f6db882faad0a450c56ae5966796c93e2c4520/examples/wasi-browser/htdocs/worker.js#L25-L33
Example2: w/ vscode-wasm : https://github.com/ktock/vscode-container-wasm/blob/26afb1790af51987687048b9bf8f16187ae7fe46/src/web/extension.ts#L76-L85


Fixed minimum required Python version

Pull Request Test Coverage Report for Build 66

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 64:
0.0%

Covered Lines:
63

Relevant Lines:
63

💛  - Coveralls

Pull Request Test Coverage Report for Build 66

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 64:
0.0%

Covered Lines:
63

Relevant Lines:
63

💛  - Coveralls


Speed up compile time computation of sum

:notebook_with_decorative_cover: Description

Problem:

Compile time sum is defined in terms of recursive class templates
which is slow.
There are no tests for sum.

Solution:

Use fold expressions to speed up the implementation and make sum a
variable template instead.
Add tests for sum.

:memo: Checklist

[X] All user-facing changes have changelog entries.
[X] The changes are reflected on docs.tenzir.com/vast, if necessary.
[X] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions

File-by-file.

CC: @dwms-qlab


Missing ExecludeFromCodeCoverageAttribute for SpanLineEnumerator

https://github.com/SimonCropp/Polyfill/blob/fb16c099c771f8469db1ba7f9e3890e2c3595862/src/Polyfill/SpanLineEnumerator.cs#L23

deployed


Check docstrings

Due to a lot of refactoring many Python docstrings are currently outdated. Need to go through the code, make sure they are still correct, and update where necessary.

Hi @dennybritz, I would like to contribute for this task. If you have already started working on this, please tell me the untouched part, so that we can distribute the work.
Looking forward to your reply.
Cheers!


Commented out code

Ejs code don't become grey while commented out.

I am not near a computer to test it. Are you sure the actual JS code is
commented out. I know you commented the html out, just not sure the actual
ejs code is commented out.
On Thu, Aug 11, 2022, 10:56 AM Usi the Storm @.***>
wrote:

Ejs code don't become grey while commented out.
[image: image]
https://user-images.githubusercontent.com/50480382/184163084-ee684c53-5a86-4995-9d16-06422536dacf.png
—
Reply to this email directly, view it on GitHub
https://github.com/Digitalbrainstem/ejs-grammar/issues/88, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMHQ7AQRBZETBU4F6MZ7JLVYUIBHANCNFSM56IPQDQQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Oh. Got it. Need to comment out EJS code with <%# tag


What should I set for client.user when using client credential with confidential client?

Hi!
Currently, my web application requires login to do any action,
but I want to allow some action even without login.
It is currently using authorization code grant flow to get access token and I am trying to
get access token by client credential grant flow when user is not logged in.
I thought using client credential grant flow would work, because it doesn't have user authetication in the flow, so the client can get access token without any user information.
Client(web application) is confidential and is registered manually before the server starts(it is not registered by any users)
Therefore, I didn't define user column in client model
However, I found that
if grant_type == 'client_credentials':
      if not hasattr(client, 'user'):
          log.debug('Client should have a user property')
          return False
      request.user = client.user

[code from flask_oauthlib/provider/oauth2.py line 894]

causing Unauthorized Client Error.
As I said before, this client isn't registered by any user.
But if define user column in client model,

what should I set for client.user?
Or is it just wrong using client credential grant flow in this situation?

Sorry!
It was my fault not reading documents carefully.
Solved it by defining a new validator!


Shellmate window resizes when terminal is <685px from second monitor

This only is happening when the terminal window itself is resizing to take up more of the screen and only when there is a second monitor to the right of the current monitor.
Video: https://drive.google.com/file/d/1oIdYMQlmkV5q25OJJMntWyBkhsmscVdn/view?usp=drive_link

@maryldyer were you able to repro this with the latest build?


Simplify handling of links

Closing as old, and no apparent reason to switch packages. Sorry for taking so long


Would like a crash reporting plugin

Firebase crash reporting or crashlytics.  Either is possible.  Requested by @lukef

See also https://github.com/flutter/flutter/issues/614 and https://github.com/flutter/flutter/issues/4070

FWIW, another potential strategic partner has asked for Crashlytics.

Is it currently possible right now to capture the stack trace when something crashes?

It's possible to capture a dart stack trace:
https://github.com/flutter/flutter/issues/614#issuecomment-292566271

I wonder how Crashlytics/Firebase Crash reporiting will handle non ObjC/Java stack traces assuming they allow custom crash uploads.
We need this for Tunes app also, i'll experiment and report back

Possible with FirebaseCrashReports if you do the following
Wrap up the crash from flutter in an Android Exception
FirebaseCrash.report(new Exception(flutterCrashString));
And that will be reported in the console

Google is recommending to NOT use firebase for crash reporting on new projects anymore. I would suggest we close this issue.

https://firebase.google.com/docs/crash/

Actually, I think we can keep this open, it's just about providing crashlytics integration.

Updated the issue description :)

I believe @yjbanov is actively working on this.

Unassigned for now because I'm not actively working on it. Still on the roadmap though.

A crash reporting option is here: https://pub.dartlang.org/packages/sentry

Closing as dupe of https://github.com/flutter/flutter/issues/614.

Looks like #614 was closed without implementing Crashlytics support.
Moreover, it was suggested to make incremental enhancements.
@Hixie Do you think it's reasonable to reopen this now as an incremental enhancement to crash reporting?

Please don't hesitate to open a new bug. Probably best not to re-use this one since it has a confusing history and not much useful information.

Thanks for opening #14765 -- for anyone interested in Crashlytics support, please cast a vote by pressing thumbs-up on the top-most comment in that new bug.

on September 9th, 2018, Firebase Crash Reporting will be fully retired and you will no longer have access to that part of the console.

Since firebase crash reporting just announced they are retiring it in firebase in 60 days do we have an update for when crashlytics will be available for flutter?

@AppleEducate I believe it means crashlytics will be default crash analytics for firebase Analytics.


MMD by class

MMD is set to be implemented across the whole dataset. However, it may be beneficial to compute it by class.

We'll want an argument to the function itself denoting whether or not to do so (probably just setting to None by default and passing in the dataset labels as a way of asking for it to be computed by class)
We'll also want an argument as to whether it should return the summed float value, the list of class similarities, or both

This might also apply to KDE metrics?


Automatic publishing to PyPi when a new release is made

We should create a workflow that automatically uploads new releases to PyPi.
One usefull github action: https://github.com/marketplace/actions/pypi-publish

We automate it, but we need an TheoCheM email account for making a PyPI account. When I know that, I can get the private token and link this repo to PyPI with the github workflow

Implemented


Fixes tests broken by edgedb-python 0.13.0a2

This requires edgedb/edgedb-python#152 and releasing 0.13.0a2 after that.

@tailhook Merge this?

Somehow missed edgedb/edgedb-python#157


Inaccurate documentation

The documentation says that to enable checksuming this line needs to be added to the config:
ofs.ckslib ++ libXrdMultiuser.so
When we attempted that we ended up getting permission denied errors when gfal attempted to checksum files after they had been written to local storage.
gfal-sum https://:1094/ ADLER32
The error xrootd reported:
241105 14:40:00 40588 ofs_chksum: .13:48@ Unable to checksum /; permission denied
we found that using the * instead of the ++ got around this issue.
ofs.ckslib * libXrdMultiuser.so

Interesting, those have different behaviors that I'm not clear on.  I'll bring it up on a meeting tomorrow I have with xrootd developers.

I think you are right, checking with production servers here, and even in the example config, we are using the *, so I think that's correct.  Thanks for the issue!  Docs fixed in c9df3ec474015581d6480729cae1e21c38452c2f


Changes/Additions to improve e2e  synchronization and to make debugging tests easier

Also has Codecept steps.d.ts for types in IDEs that support is (e.g. VS Code) for IntelliSense/autocomplete.
Try npm rum e2e-debug

Please send this PR to the main repo


[DOCS] : Make a Code_of_Conduct.md

Hey, @PritamSarbajna thanks for this fantastic beginner-friendly project.
I'm sure this will help people starting out in Open source like me
I just had a suggestion that for an Open Source project it would be helpful to have a code of conduct file.

Kindle refers to this:
https://opensource.guide/code-of-conduct/

Also if it's okay with you, could you assign me to it?

Yeah sure @lubnafathima I was also thinking about this glad you raised this issue please go for it 🙌

please check the pull request #39!

Thanks @lubnafathima ✨


Disabling domain reload makes Yarn projects compile incorrectly

What is the current behavior?

When Domain Reload is disabled, changes to Yarn scripts aren't correctly compiled in Play Mode.

Previously-fixed compilation errors still persist in play mode. (In my case it was a bad input to a YarnCommand)
New lines in a Yarn script aren't showing up in play mode.

Please provide the steps to reproduce, and if possible a minimal demo of the problem:

Set up the Quick Start demo scene given in the documentation.
Disable domain reload. (Edit -> Project Settings -> Editor -> Enter Play Mode Settings)

Keep scene reload enabled.

Enter play mode once, then return. The dialogue should be correct here.
Add new lines to the HelloYarn script.
Enter play mode again, and observe that the new lines aren't showing up. Exit play mode.
Try reimporting both the HelloYarn script and the YarnProject, and retry 5. You should see that new lines are still not showing up.
Enable domain reload, enter play mode again, and observe that the new lines are showing up again.

What is the expected behavior?

Yarn projects should be correctly compiled in play mode, even without domain reload enabled.
Please tell us about your environment:

Yarn Spinner Version: 2.2.3
Unity Version: 2021.3.16f1

Other information

It seems that Yarn projects are compiled AOT. If it's not possible to recompile a Yarn project automatically, it'd be nice to have a manual button that lets us do a clean recompilation, perhaps in the YarnProject inspector or the DialogueRunner inspector.

I'm also seeing this issue in v2.3.0-beta1.
It is especially noticeable when using the DialogueReference type. The list of available nodes is not updated until you disable fast enter play mode, and then press play to recompile.

Would also love to have a manual button or something similar. I understand the issue myself so I keep it in mind while working but designers/writers in the team keep tripping with this.

Would also love to have a manual button or something similar. I understand the issue myself so I keep it in mind while working but designers/writers in the team keep tripping with this.

I also want a recompile button. Gimme control.

I've verified on version 2021.3.27f1 that with #232 this issue has been fixed as well, so I'll be closing it for now.


kube3d: 3.2.1 -> 3.3.0

Motivation for this change
Bump kube3d/k3d to 3.3.0, also bumped k3sVersion to match the official release
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS (x86_64)
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 104681 1

1 package built:

kube3d

Result of nixpkgs-review pr 104681 1

1 package built:

kube3d

that was fast @RaghavSood !


Conductor Controller Integration

Right now, this library only works with activities and fragments, are there any plans to support Conductor's Controllers?

@adin234 thanks for the link, I had not heard of the Conductor framework before.  EasyPermissions is just meant to be a wrapper over the Android Framework permissions APIs, we don't plan to support every app architecture directly.
It's not possible to make a permissions request without somehow getting hold of an Activity or Fragment reference, even if you're using a View based framework.
As this is not currently our goal I'm going to close this issue, hope that makes sense!

Agreed. If you really need to use EP from Conductor, you could always add this hack: (Activity) view.getContext()


Revokable option get's ignored on init

Needs more info. Closing ticket for not enough info and staleness.
If you wish this ticket reopened, please use the following form to better help us reproduce your issue:
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.


Handle missing batch link reports

This commit adds an exception handler for 404 errors from link-checker-api. This can happen if the batch link report has been removed, which happens autmoatically on a schedule to prevent build-up of old reports. The app should handle this case.

I wonder if it's worth making the app run another batch check in this case, but this seems like a fine solution for now.


potential security vulnerabilities

Hi all,
it seem there is a security issue with the jszip package that used in grunt-zip>0.14.0.
i got the the warning from GitHub after upgrading grunt-zip to lastest.

there is the info that i get by using npm audit

and the only package that use jszip can be seen here

GitHub is still processing the advisory so we haven't yet received a PR for it. The advisory doesn't unlikely to impact grunt-zip users since it's around prototype pollution: https://github.com/advisories/GHSA-jg8v-48h5-wgxg
If this is high priority to you, please submit a PR. Otherwise, we'll be waiting for the one from GitHub's dependabot

@twolfson, thanks for fast response.
i just wondering if grunt-zip would still be working with jszip >3.7.0?

I don't have the bandwidth to pick this up but you can prob check their changelog to see what happened between 2.5.0 and 3.7.0

This has been resolved in<EMAIL_ADDRESS>by upgrading to<EMAIL_ADDRESS>


Show Network Errors in Status Bar

Brief Description:
When bad profiles or listings fail, show something in the status bar so what happened is clear, instead of just kicking the client back to the last history state, so the user understands what just happened.

This is kind of done. On network errors (bad profiles, bad handles, etc...), the user won't just be kicked back to the previous page. A modal will show up with a message indicating the problem first, and then the user is kicked back.
If we want, we can put these errors in the status bar. But, is this really what we want? @morebrownies
You may want to wait until https://github.com/OpenBazaar/OpenBazaar-Client/pull/977 is merged and see how the new Save Settings message looks. As I mentioned, we could style different types of messages differently.

The save settings experience is so minimal that it becomes easy to miss. When I hit save in my client, it takes about 20 seconds to complete the request and there isn't a loading indicator on the save button (when the request is processing).
As far as displaying error messages into the status bar goes, I think that may be a bit aggressive to present that amount of information in such a tight space, especially since it could easily be missed. If I try to connect to a page and it times out, I'd rather the error be front and center. I'm trying to dig up a design I had created for this in the past. I'll follow up and post it once I find it.

ok.. the save settings behavior was updated based on a suggestion initiated by @drwasho :
https://github.com/OpenBazaar/OpenBazaar-Client/issues/959
I'm not sure what we want here... Should I leave the status bar message and add in a loading spinner on the button? Should I revert back to the modal error message (although that didn't include the loading spinner either)? Should I revert back to the modal error message and put in a loading spinner?

For now, I'd say we definitely need a spinner on the save button(s). I'm also going to play around with the styling of the status bar to see if I can make it a bit more prominent when it displays.
Error messages I still think we're going to need to do something that is much more in your face and doesn't auto dismiss. Still trying to find the screenshot I was referencing, hold tight...

Here's the design for a failed connection I put together a while back. Currently, when you try to connect to an offline store there are a few UX problems that arise.

I can't stop the request once it has started. If it's hanging for a while, I'm stuck looking at a spinner. Ideally, the user would have a way to Stop/Cancel the request and immediately return back to where they were.

When the connection does fail, we should explain to the user why the connection failed and give them controls to retry or return back to where they were.

Some of the error use cases could be:
The handle you entered is not connected to a page (Back | Retry)
The page you're trying to view is currently offline (Back | Retry)
The listing your'e trying to view is currently offline (Back | Retry)
I'd like to add tooltips to each of these and present more information on hover as well, e.g. "The reason you can't reach @themes is because the owner of the page has either taken their page offline or..."

Completed.


[$500]  Group - After selecting user in Group chat, the modal is not sliding upwards

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

Version Number: 1.4.44-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Access staging.new.expensify.com
Sign into a valid account
Click on the "+" > "Start chat"
Select user to "Add to group"

Expected Result:
User expects that when a user is selected, the modal is slid upwards to show the selection
Actual Result:
The modal does not slide upwards. Also, a few selected members are missing after scrolling back upwards.
Workaround:
Unknown
Platforms:

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

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/4b479c2d-7b18-4f47-9891-fc451acddf69
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~013e254d49afd2fba2
Upwork Job ID:<PHONE_NUMBER>837190144
Last Price Increase: 2024-02-27

Production:
https://github.com/Expensify/App/assets/78819774/85d60b03-c199-471c-be8c-e5fa5bca3ef9

This seems to be coming from here, I reverted it locally and I can't reproduce. cc @Piotrfj @allroundexperts.

Proposal
Please re-state the problem that we are trying to solve in this issue.
The modal slide upwards when the user selects "Add to group"
What is the root cause of that problem?
https://github.com/Expensify/App/blob/d2a9c6faa9f450c20037a968974844f4f4325198/src/pages/NewChatPage.tsx#L118-L128
What changes do you think we should make in order to solve the problem?
To prevent removal from the list, it is require to make sure that certain conditions are met before allowing removal.
need to add a check to ensure that if an option is already on the list, it will only be removed if the number of selected options exceeds the minimum required.
What alternative solutions did you explore? (Optional)

@Piotrfj @allroundexperts  Can you please take care of this deployblocker.

There's some scrolling happening, but mostly to the wrong position (just below the selected list).
Additionally the selection logic seems to be faulty as well: the selected options sometimes reset when a new option is selected.
Here is a recording to prove the above points (left side: prod, right side: staging):
Screencast at: 2024-02-27 11-06-34.webm

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~0161b2e93e88f95be5

This might fix this issue as well.

Will test and confirm.

Okay. @Piotrfj is on it already!

I update the proposal plz take a look
cc: @allroundexperts

Okay. @Piotrfj is on it already!

@allroundexperts can you inc. a link next time so we can quickly view?
For this, I see Working on the regressions in the PR here

@allroundexperts can you inc. a link next time so we can quickly view? For this, I see Working on the regressions in the PR here

You're right. Should have included that.

Working on it, expecting some stuff done today

@kevinksullivan FYI @ishpaul777 will need to be paid out for being the C+ on this revert PR: https://github.com/Expensify/App/pull/37454

@ishpaul777 please complete the checklist above so we can wrap up payments.

Contributor+: @ishpaul777 paid $500 via Upwork
@ishpaul777 , I created a TestRail for this because QA does monthly tests for edge cases and design-related issues and they might want to include this.
https://github.com/Expensify/Expensify/issues/378695
Thx!


refactor(codegen/go): Remove rename field from codegen proto

Prefer instead to pass the rename map to the internal golang codegen plugin via options.
This diff is mostly large because I also pull golang options into a subpackage. There are only three or four meaningful changes, all in protos, internal/cmd, and internal/codegen/golang/options/options.go.

After discussion, I'm withdrawing this PR since rename is fine where it is in the plugin proto for now.


Rename readonly decorator

Rename @readonly to @if_readonly.
Fixes #56

@marusinm Do you want to merge your PR (#81) before this one?

Do you want to merge your PR (#81) before this one?

No, I would like to provide at least one more function to my PR and maybe some tests. I understand what these changes mean for me. You can merge this first.
@lachmanfrantisek

No, I would like to provide at least one more function to my PR and maybe some tests. I understand what these changes mean for me. You can merge this first.
@lachmanfrantisek

Nice!
I did not want to add you another work..;-)
I'll merge #84 and this one so you'll have better testing experience. (The response files should be generated automatically if they are not found.)


Implement a handler for 409 (concurrent modification) error in TS SDK

Implement a handler for a 409 concurrent modification error to handler.

See this PR


How would i use these in React app? (question not an issue)

I'm a fairly new to react world and just found out about progressive webapps, this ui-element samples performance is amazing on mobile (Native like) I was wondering how would i use these in reat. Any suggestions or links to guide would be greatly appreciated.

How would i use these in a fastn.js app?
Rather than open a separate issue, I have a similar question but for a different library called fastn.js.
I've been doing some testing but it involves reasonably significant design changes which I'm not sure about.
Is there some way to make these generic components that can be plugged into different js toolsets like paper, react and fastn, because it would be really useful?


Implement Test Time Augmentation

We need to scan the image from top to bottom and output all bounding boxes within it. We can probably use test time augmentation and a similar approach as we used in the previous competition.

We could investigate the show_results_side_by_side() method as we have to take the outputs and run nms() on the predictions.

Nikhil pointed out that we should be able to generate predictions in a simpler way. Since we're not doing classification and don't have any fully connected layers, we should be able to pass arbitrary sized tensors through our network and we'll just receive a correspondingly sized output which we'll convert to bounding boxes and run through NMS.
Unfortunately I'm getting an error when I used the RetinaNet from the ObjectDetection repository.
The exception:
Exception has occurred: RuntimeError
expected a non-empty list of Tensors
  File "/home/josh/git/SorghumHeadDetection/RetinaNet/RetinaNet.py", line 52, in _apply_transpose
    [func(p).permute(0, 2, 3, 1).contiguous().view(p.size(0), -1, n_classes) for p in p_states], 1)
  File "/home/josh/git/SorghumHeadDetection/RetinaNet/RetinaNet.py", line 81, in forward
    return [self._apply_transpose(self.classifier, p_states, self.n_classes),
  File "/home/josh/git/SorghumHeadDetection/main.py", line 34, in show_test_results
    z = learn.model.eval()(x)
  File "/home/josh/git/SorghumHeadDetection/main.py", line 102, in <module>
    show_test_results(tlearn, oimg=img, classes=tlearn.data.classes, start=0, detect_thresh=0.70, nms_thresh=0.35)

We have three inputs to _apply_transform():
func
Sequential(
  (0): Sequential(
    (0): Conv2d(32, 32, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1))
    (1): ReLU()
  )
  (1): Sequential(
    (0): Conv2d(32, 32, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1))
    (1): ReLU()
  )
  (2): Conv2d(32, 18, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1))
)

p_states

Empty (Probably the problem)

n_classes

Value of 2

So after not filtering out p_states, our network no longer returns empty p_states.
On some images we can successfully generate bounding boxes, on others we get an error similar to:
Exception has occurred: RuntimeError
The size of tensor a (69) must match the size of tensor b (70) at non-singleton dimension 2
  File "/home/josh/git/SorghumHeadDetection/RetinaNet/RetinaNet.py", line 15, in forward
    return self.conv_lat(self.hook.stored) + F.interpolate(x, scale_factor=2)
  File "/home/josh/git/SorghumHeadDetection/RetinaNet/RetinaNet.py", line 76, in forward
    p_states = [merge(p_states[0])] + p_states
  File "/home/josh/git/SorghumHeadDetection/main.py", line 34, in show_test_results
    z = learn.model.eval()(x)
  File "/home/josh/git/SorghumHeadDetection/main.py", line 102, in <module>
    show_test_results(tlearn, oimg=img, classes=tlearn.data.classes, start=0, detect_thresh=0.70, nms_thresh=0.35)

The error appears to be an off-by-one error that happens when the height dimension on one end of a lateral upsample merge does not match the height dimension of the other end.
Image on which it works:
train_labelled_images/C36-R6-G276-DSC00775.jpeg
torch.Size([3, 1243, 316])

Image on which it doesn't work:
train_labelled_images/C2-R34-G62-DSC01702.jpeg
torch.Size([3, 1103, 287])


FN-RN-Fixed-date

Updated the date in the front matter.
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

No review as only front matter was updated. No content has been changed.

Preview environment: https://ci.pr.harness.io/pr-412


Import data from excel or data source

Please Add import from data source feature.

My God, every MS database product (SSMS and Data Studio) should have a super awesome Excel Import Wizard. They are both MS products!  And it's been YEARS.  Free MySQL Workbench can do so much more.  Most data starts out in spreadsheets and not having a good import mechanism is completely ridiculous. dbForge sucks too.

Please prioritize Excel (*.xlsx) over other data sources.  The file format has been openly documented since 2007.  Any design or usability justification for supporting xlsx for export but not import?

Duplicate of #1871

Wait.  You closed this?


How to close a dropdown menu

I have a dropdown menu item that does a window.open in new tab (which works). But in the old tab the menu remains visible. I tried jQuery $('.dropdown-toggle').blur() but that doesn't remove the menu.

Same problem here. Suggested fix is not working. When is applied then we can't show dropdown menu after click again.

I started using dropdowns and encountered the same issue. Here is a CSS-only fix that worked for me (tested on Chromium, Firefox and Webkit):
.dropdown .menu:active {
  /* animation needed for click events to work */
  animation: bye 0;
}
@keyframes bye {
  to { display: none; }
}

Update: my above fix is actually working for non-touch devices only. To make it work also for touch devices just add the following below it:
@media(hover: hover) {
  .dropdown {
    .menu:hover {
      display: none;
    }
    .menu:active {
      display: block;
    }
  }
}


Add a validation on MigPlan name:  Starting with a number is not allowed, example '0710a' used for plan name resulted in error

I created the below, didn't realize it'd be a problem.
$ oc get migplan 0710a -o yaml
apiVersion: migration.openshift.io/v1alpha1
kind: MigPlan
metadata:
  annotations:
    touch: ccd22062-2a42-4434-8fef-c36e93142611
  creationTimestamp: "2019-07-10T20:52:07Z"
  generation: 3
  name: 0710a
  namespace: mig
  resourceVersion: "146505"
  selfLink: /apis/migration.openshift.io/v1alpha1/namespaces/mig/migplans/0710a
  uid: 9485bd0a-a354-11e9-8773-063ca044fe88
spec:
  destMigClusterRef:
    name: host
    namespace: mig
  migStorageRef:
    name: jwms3
    namespace: mig
  namespaces:
  - mysql-persistent
  persistentVolumes:
  - action: copy
    capacity: 10Gi
    name: vol21
    pvc:
      Name: mysql
      Namespace: mysql-persistent
    supportedActions:
    - copy
    - move
  srcMigClusterRef:
    name: ocp3
    namespace: mig

$ oc logs controller-manager-79bf7cd7d-99sbr -n mig
{"level":"error","ts":1562792358.9312413,"logger":"plan|wvc7f","msg":"","error":"Service \"0710a-registry-qqvq8\" is invalid: metadata.name: Invalid value: \"0710a-registry-qqvq8\": a DNS-1035 label must consist of lower case alphanumeric characters or '-', start with an alphabetic character, and end with an alphanumeric character (e.g. 'my-name',  or 'abc-123', regex used for validation is '[a-z]([-a-z0-9]*[a-z0-9])?')","stacktrace":"github.com/fusor/mig-controller/vendor/github.com/go-logr/zapr.(*zapLogger).Error\n\t/go/src/github.com/fusor/mig-controller/vendor/github.com/go-logr/zapr/zapr.go:128\ngithub.com/fusor/mig-controller/pkg/logging.Logger.Error\n\t/go/src/github.com/fusor/mig-controller/pkg/logging/logger.go:59\ngithub.com/fusor/mig-controller/pkg/logging.Logger.Trace\n\t/go/src/github.com/fusor/mig-controller/pkg/logging/logger.go:65\ngithub.com/fusor/mig-controller/pkg/controller/migplan.ReconcileMigPlan.ensureRegistryService\n\t/go/src/github.com/fusor/mig-controller/pkg/controller/migplan/registry.go:198\ngithub.com/fusor/mig-controller/pkg/controller/migplan.ReconcileMigPlan.ensureMigRegistries\n\t/go/src/github.com/fusor/mig-controller/pkg/controller/migplan/registry.go:62\ngithub.com/fusor/mig-controller/pkg/controller/migplan.(*ReconcileMigPlan).Reconcile\n\t/go/src/github.com/fusor/mig-controller/pkg/controller/migplan/migplan_controller.go:174\ngithub.com/fusor/mig-controller/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/go/src/github.com/fusor/mig-controller/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:215\ngithub.com/fusor/mig-controller/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func1\n\t/go/src/github.com/fusor/mig-controller/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:158\ngithub.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil.func1\n\t/go/src/github.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:133\ngithub.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil\n\t/go/src/github.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:134\ngithub.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait.Until\n\t/go/src/github.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:88"}

Seems unnecessarily restrictive to the user for the controller to restrict the plan name.  Instead, let's update the controller to ensure the generated name is valid.

I was surprised to find this, since it's totally valid for a k8s name to start with a number. But when I went to test it, the migplan was confusingly without a status. Is this just a controller fix?


Sign up validation

I want some sort of a validation system for the signup process to prevent anyone from making a username that's racist or means.
An error should pop up like an alert() or a message under the form telling the user, they can't use or create an account with that username. We could possibly use a third-party application for this, which could be faster than coding it ourselves.

I can work on finding a solution to this

@loshjarson  assigned.

@loshjarson I revamped the whole user stuff so there's no longer any signup/login/logout feature all user info will now be saved to cookies and saved to the database via a settings page.
What could be done is within the settings page there's a one-time username reset, which users can do and you can put the validation there instead of the signup.

@loshjarson I revamped the whole user stuff so there's no longer any signup/login/logout feature all user info will now be saved to cookies and saved to the database via a settings page.
What could be done is within the settings page there's a one-time username reset, which users can do and you can put the validation there instead of the signup.

@Chris5613 alrighty, sounds good. I will take a look at it.

@Chris5613 Im not really seeing a need for validation here especially with no passwords or tokens being used with the update. I am all ears for your ideas though

@loshjarson One of my ideas was to use the npm package " npm bad-words"
https://www.npmjs.com/package/bad-words
and from that, you add a check to see if the username being made is in there if there is then stop it.
I already have a checkUsername in there to check for other stuff but you might be able to add to it.
If a badword is found you can just do prompt() and state that they can't use that name.
Another way was to create a list and manually write a bunch of bad words then filter it with that but let me know what you think. I had a few accounts that were created with the n word or other bad words and I would like to stop those.

@Chris5613 I have also been looking at that library and think it would be the best bet.
I will start implementing and testing it asap to stop those

@loshjarson I would say do whatever you think will work and if it works then we're good.
How you implement it is entirely up to you!


Get Data from the Internet

URL of codelab:
Specify the language of the codelab if it is not English:
In which task and step of the codelab can this issue be found?
Describe the problem
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

[ ]


Consider using fastify-gql instead of apollo-server-express

According to recent benchmarks done by Ben Awad, migrating to fastify-gql+graphql-jit could result in a 4x performance improvement.

We’d mused about looking into this generally (other servers) but no issue/research yet. Thanks for kicking this off!
Sent with GitHawk

CC @mcollina 🙌

Serverless is something we think is important, so this was interesting to see:

Fastify is not designed to run on serverless environments. The Fastify framework is designed to make implementing a traditional HTTP/S server easy. Serverless environments requests differently than a standard HTTP/S server; thus, we cannot guarantee it will work as expected with Fastify. Regardless, based on the examples given in this document, it is possible to use Fastify in a serverless environment. Again, keep in mind that this is not Fastify's intended use case and we do not test for such integration scenarios.

https://github.com/fastify/fastify/blob/master/docs/Serverless.md#attention-readers

What do we lose by not being built on top of express? In particular, the community of middleware?

Ben already rolled his own taking inspiration from fastify-gql to make express-gql. No surprise his claimed motivation was that he was using some express middleware

We should make a perf baseline so that we can measure the impact of whatever change we make.

Please check out https://github.com/fastify/aws-lambda-fastify#some-basic-performance-metrics. All of this means that Fastify has a worse cold start time than express, while it provides better runtime performance (which is significant even on lambda).
I would also add that the caching technique in fastify-gql relies heavily on in-process caching to achieve maximum throughput. In-process caching is against the lambda philosophy, and as a result the benefit of something like graphql-jit are minimized.

I think what this issue outline is that we definitely need a concept of plugins for servers. We will not fulfill everyone's requirements regardless of what we choose. We already see three different use-cases/motivations:

Performance (fastify)
Ease of use (compatibility with express middlewares & or apollo-server plugins)
Lambda

There's a couple of options for us as to what we choose by default (assuming we come up with a plugin system to support any graphql server). My suggestion would be to bundle by default a server that has the best ratio performance/bundle size. And for that, I think using something "minimal" like graphql-express is our best shot. WDYT?


Possible problem when using SWXMLHash inside a framework

Please have a look at https://github.com/tkrajacic/XMLTestWithFramework
It is a minimal crashing example.
When the application is finished processing the XML file it crashes when trying to over-release some CFString. This only happens when using a NSNumberFormatter to parse a String out of the XML.
When commenting out the String parsing, everything works.
I am not sure if I am making a mistake using SWXMLHash here or if it is a bug inside SWXMLHash.
Any help would be appreciated!

I've been digging on this for the better part of an hour tonight. I went off on a tangent because my understanding of Swift strings is that they are value types. But then I discovered that I could repro the bug by putting in a hardcoded number in the call to NSNumberFormatter meaning that it never even got a reference to valueText. That really threw me. The culprit? The call to lowercaseString.
If you change the parseFromXMLNode method in XMLFormatter.swift to only have the following, it will still blow up:
    mutating func parseFromXMLNode(xml: XMLIndexer) {

        self.name = xml[XMLKeys.name].element?.text

        let maybeUnit : String? = xml[XMLKeys.temperatureUnit].element?.text

        // this seems to cause the bug
        let bugRightHere = maybeUnit!.lowercaseString
    }

Does that make any sense to you? I'm curious if there might be a bug in the beta, because I thought I read somewhere that lowercaseString and uppercaseString were moved back into Swift proper (as opposed to being in Framework).

You sir are the man! I have tried so many variants and obviously must have overlooked this.
Now I feel really bad for letting you hunt down my programming mistakes :(
I'll take it to Apple's Swift Forums to see if one of the Swift guys has more to say about it, but I think you are right about the Swift String thingy.

The bug seems to be documented already:
https://devforums.apple.com/message/1102272#1102272

No worries, I'm happy to help!


Issues with peer dependencies

[x] I have read the troubleshooting section before opening an issue.
[x] I have tried upgrading iles and vite.

Description 📖
After bumping the dependencies, issues with peer dependencies are found by pnpm.
I think @islands/feed needs to update the peer dependency version and iles needs to bump its dependencies.
Reproduction 🐞
Run ncu, pnpm i and pnpm typecheck in https://github.com/ouuan/iles-blog.
pnpm i shows:
 WARN  Issues with peer dependencies found
.
├─┬ @islands/feed 0.8.0
│ └── ✕ unmet peer<EMAIL_ADDRESS>found 0.9.0
└─┬ iles 0.9.0
  └─┬ unplugin-vue-components 0.21.2
    └─┬ unplugin 0.7.2
      └── ✕ unmet peer<EMAIL_ADDRESS>|| ^3.0.0-0": found 4.0.3 in iles

pnpm typecheck shows: (see edit log) It turns out that the type check error was caused by https://github.com/doesdev/rollup-plugin-analyzer not specifying rollup version in package.json and using rollup 2, which is not related to iles, and the peer dependency warnings are harmless. But still love to see updated dependencies with the peer dependency warning resolved :)

Dependencies Info
Run npx iles info and pnpm list (or npm list) and provide the output:
iles v0.9.0 vite v4.0.3

dependencies:
@giscus/vue 2.2.6
@types/html-escaper 3.0.0
@unocss/reset 0.48.0
@vueuse/components 9.9.0
@vueuse/core 9.9.0
date-fns 2.29.3
filesize 10.0.6
fuse.js 6.6.2
hast-util-select 5.0.3
html-escaper 3.0.3
iles 0.9.0
isbot 3.6.5
katex 0.16.4
plausible-tracker 0.3.8
prism-themes 1.9.0
simple-git 3.15.1
ts-exhaustive-check 1.0.0
unified 10.1.2
unocss 0.48.0
vue 3.2.45

devDependencies:
@cloudflare/workers-types 4.20221111.1
@iconify-json/mdi 1.1.41
@islands/excerpt 0.9.0
@islands/feed 0.8.0
@islands/headings 0.8.1
@islands/images 0.8.1
@islands/prism 0.8.0
@types/hast 2.3.4
@types/node 18.11.18
@typescript-eslint/eslint-plugin 5.47.1
@typescript-eslint/parser 5.47.1
@unocss/transformer-directives 0.48.0
@vue/eslint-config-typescript 11.0.2
@vue/runtime-core 3.2.45
eslint 8.31.0
eslint-config-airbnb-base 15.0.0
eslint-plugin-import 2.26.0
eslint-plugin-vue 9.8.0
got 11.8.5
rehype-katex 6.0.2
remark-gfm 3.0.1
remark-math 5.1.1
rollup-plugin-analyzer 4.0.0
sass 1.57.1
tsup 6.3.0
typescript 4.9.4
vite 4.0.3
vite-plugin-compression 0.5.1
vue-tsc 1.0.19

BTW, there seem to be a lot of outdated dependencies. Any reason not to bump them, or just have no time?

Hi, I was on vacation, planning to update dependencies later this week.
There are some incompatibilities related with changes in the way Vite 4 handles ESM, but there should be a way to workaround those.

Any chance that the dependencies could get updated? I'm fine even if iles itself stops gaining new features, but I hope that at least the dependencies can be up-to-date.
There has been too long since the last update, so I can understand the difficulty to do this. I'd like to help if needed, but I'm not sure how to detect compatibility problems and ensure that everything works fine after upgrading the dependencies.

Hi @ouuan!
I'm waiting for Vite 4.3 to be released to ship a new version of îles with updated dependencies.
Doing batch upgrades every now and then is more efficient for me, as otherwise updating dependencies becomes a very time consuming chore.
Primarily, there wasn't a compelling enough reason to update deps given that îles was working properly, but Vite 4.3 has some potential performance improvements that would benefit îles users, and that's a nice motivation.
Thanks for reaching out, and stay tuned 😃

I think when a batch upgrade contains breaking changes, it's harder to locate where it breaks, so it's easier to keep the dependencies up-to-date continuously 🤔
I wrote the previous comment because I was not able to use the TypeScript 5 array extends syntax whose support was added in Vite 4.2. It's not very important though.

Agreed, but it's a balancing act.
In the case of Vite, which acts as a platform for this library, given îles is making use of most of its features—some in very specific ways—upgrading always has the risk of uncovering new bugs (both in Vite and îles).
Without enough time to test, these bugs will be caught by îles users instead.
Releasing less frequently allows me to spend more time testing manually in my sites when I do a release, resulting in îles users getting more stable experience, both when they upgrade or try it for the first time through create-iles.

Good day Max! With latest update to 0.9.5 & also create-iles package as well, any plans to update peer dependencies of @islands packages? I'm using feed and getting the peer dependencies error. Thanks.

Hi, @ElMassimo! There has been a month since the release of Vite 4.3. Any plans to update the dependencies?

Sorry, big life changes. Preparing a release targeting Vite 4.4.

Awesome, can't wait 😀

@ElMassimo What's the status of 86ea918eef71a4ac3ffe212cae1b22fde28e1554? Could you please test the latest Vite 4.4 and make a new iles release?


Return full list of events for broadcasting

I have several modules that provide alternate displays for calendar events, and currently, if I want the calendar module to return all of the future events for the current month (for my MMM-MonthlyCalendar module), then I need to configure maximumEvents to be large enough that the calendar module itself needs to be hidden.  Instead, the configured display limits should only be imposed when generating the list of events to display, not when fetching events from configured calendars.

I understand the need and it's fine to implement. But I'm not a big fan of the forDisplay attribute since it doesn't tell what will happen if it's set to true. Maybe rename it to something like limitNumberOfEntries?

Codecov Report

Merging #2787 (e3857ca) into develop (fcb0d33) will increase coverage by 0.37%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #2787      +/-   ##
===========================================
+ Coverage    68.67%   69.05%   +0.37%
===========================================
  Files            8        8
  Lines          265      265
===========================================
+ Hits           182      183       +1
+ Misses          83       82       -1

Impacted Files
Coverage Δ

js/server.js
87.03% <0.00%> (+1.85%)
:arrow_up:

Continue to review full report at Codecov.

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

@MichMich  the test fail is unrelated to the code changes, same test ran locally on my machine and in all tests except node 14 (by now I think the vm running the tests was extremly slow but if this occurs again we have to increase timeout).

I'll rerun the jobs to check if that helps.


Add revManifest option to support gulp-rev manifests

Adds a new revManifest option to support JSON revision manifests as created by gulp-rev. File destination paths (relative to dest) are read from the manifest if any, are all the files are passed through if no manifest has been created yet (i.e., clean build).

Not a big fan of this tbh. This couples two plugins together. Is there no better way of achieving this?

I thought the same initially but was then thinking of the revision manifest as a JSON hashmap to the revisioned filenames instead of a gulp-rev "proprietary" file format. The option could take a dictionary instead of the filename however, on a fresh build, when there is no manifest yet, the logic would have to be handled by the consumer. Plus the consumer would have to handle the JSON parsing as well.
So yeah using that option currently requires some gulp-rev-like JSON hashmap format but the development interface is nicer that way, and "enforcing" a format is better than a clunky interface IMHO.

I'm going to pass on this. I neither need or have time to maintain this. I would recommend you just publish a fork of this task.


Unit tests related to the new delta engine 'randomly' fail on Travis

Have not been able to reproduce on my local build system.

PR #436 resolved this


small fixes

Thanks for spotting those typos!
I'm against installing truffle globally, but that's what they recommend in their README, so I think I should do the same here, to avoid problems to people who followed it.
Thanks!


Error on Mac OS:

Hi @rwst, since you asked for feedback on Windows and MacOS platforms, the app is not functional on MacOS, see:

I was able to build the app locally and worked.
(by doing the following:
org.gradle.java.home=/Library/Java/JavaVirtualMachines/temurin-20.jdk/Contents/Home # in gradle.properties

gradle run

)
I don't know from where the error comes from, so can't help much more, probably from your own IntelliJ settings?
Best,

Thanks for reporting. The error looks like it couldn't write settings.json, which is written, on Linux, in the install directory, from which the app is run. I will have to find out how this is usually handled on Mac.

I can confirm have it created in the directory locally when running gradle run

Sorry, changed the path of the error but did not solve it :(

Thanks.
OK, somone pointed me to a library that handles system-specific directories and so I would ask you to try this;
https://github.com/rwst/LitBall/releases/tag/v2317

Slowly getting there!
The initial bug is gone (👍🏼), but now the next generated file when creating a query is the issue...
see below

What is your query directory setting? What if you set it to a new directory
made by you?
On Tue, 26 Sep 2023, 18:34 Adriano Rutz, @.***> wrote:

Slowly getting there!
The initial bug is gone (👍🏼), but now the next generated file when
creating a query is the issue...
see below
[image: Screenshot 2023-09-26 at 18 32 51]
https://user-images.githubusercontent.com/44283913/270729967-565a3feb-e87a-4862-8760-0c4bf4642481.png
—
Reply to this email directly, view it on GitHub
https://github.com/rwst/LitBall/issues/1#issuecomment-1735899513, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIX6VM72F34TUTEMLSQEWDX4L7ZDANCNFSM6AAAAAA5HIC4SA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I did let the default one (/Query-).

Changing it to nothing did not solve the issue.
I could not create new ones:

What I meant was create any, or even use an existing directory, and put
that in the query path setting, not the prefix. The prefix is just to have
a common prefix that is recognised by the app.
On Tue, 26 Sep 2023, 21:07 Adriano Rutz, @.***> wrote:

I did let the default one (/Query-).
[image: Screenshot 2023-09-26 at 21 05 23]
https://user-images.githubusercontent.com/44283913/270768647-b8deca2b-1898-4af4-ae10-a61cae46d5b7.png
Changing it to nothing did not solve the issue.
I could not create new ones:
[image: Screenshot 2023-09-26 at 21 06 12]
https://user-images.githubusercontent.com/44283913/270768927-c7068f96-07a2-4137-99df-6faf5c5c7f1c.png
—
Reply to this email directly, view it on GitHub
https://github.com/rwst/LitBall/issues/1#issuecomment-1736130289, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIX6VPPPYNU6FQQJSE75ZLX4MRU7ANCNFSM6AAAAAA5HIC4SA
.
You are receiving this because you were mentioned.Message ID:
@.***>

after the above warning blocking the app, I could re-open it from the command line and it had mydir as default dir which then allowed me to perform the queries as with the local install:

Yeah, I think the only issue is the default setting, which should be rather
the platform specific user dir.
On Tue, 26 Sep 2023, 21:16 Adriano Rutz, @.***> wrote:

after the above warning blocking the app, I could re-open it from the
command line and it had mydir as default dir which then allowed me to
perform the queries as with the local install:
[image: Screenshot 2023-09-26 at 21 14 56]
https://user-images.githubusercontent.com/44283913/270771579-d566b16d-a9a1-43b4-af2f-caf790f0e917.png
—
Reply to this email directly, view it on GitHub
https://github.com/rwst/LitBall/issues/1#issuecomment-1736146436, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIX6VLKOQJW4EHWAYE4UP3X4MSXTANCNFSM6AAAAAA5HIC4SA
.
You are receiving this because you were mentioned.Message ID:
@.***>

OK thanks again. I believe all read-only problems of new installations are fixed in ths release:
https://github.com/rwst/LitBall/releases/tag/v2318
Also, your editing of the query prefix setting uncovered a design problem, which I will address in a later release, probably by introducing Advanced Settings, and of course, more documentation.


Test result not found for parametrized test with faker

Issue Type: Bug
Behaviour
Expected vs. Actual
There is an error when use faker for pytest parametrized value in a test. The test run from console completes successfully.
Steps to reproduce:

Install Faker pip install Faker
Create a file test_it.py with following content:

import pytest
from faker import Faker

f = Faker()

@pytest.mark.parametrize("param", [f.word()])
def test_success(param):
    assert param

Run test from console python -m pytest . and expect success.
Run throught Testing on the sidebar and get following error: Test result not found for: ./test_it.py::test_success[expert]

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.10.2
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Venv
Value of the python.languageServer setting: Pylance

User Settings

defaultLS: {"defaultLSType":"Pylance"}

downloadLanguageServer: true

envFile: "<placeholder>"

venvPath: "<placeholder>"

venvFolders: "<placeholder>"

condaPath: "<placeholder>"

pipenvPath: "<placeholder>"

poetryPath: "<placeholder>"

languageServer: "Pylance"

linting
• enabled: true
• cwd: "<placeholder>"
• Flake8Args: "<placeholder>"
• flake8Enabled: false
• flake8Path: "<placeholder>"
• lintOnSave: true
• banditArgs: "<placeholder>"
• banditEnabled: false
• banditPath: "<placeholder>"
• mypyArgs: "<placeholder>"
• mypyEnabled: false
• mypyPath: "<placeholder>"
• pycodestyleArgs: "<placeholder>"
• pycodestyleEnabled: false
• pycodestylePath: "<placeholder>"
• prospectorArgs: "<placeholder>"
• prospectorEnabled: false
• prospectorPath: "<placeholder>"
• pydocstyleArgs: "<placeholder>"
• pydocstyleEnabled: false
• pydocstylePath: "<placeholder>"
• pylamaArgs: "<placeholder>"
• pylamaEnabled: false
• pylamaPath: "<placeholder>"
• pylintArgs: "<placeholder>"
• pylintPath: "<placeholder>"

sortImports
• args: "<placeholder>"
• path: "<placeholder>"

formatting
• autopep8Args: "<placeholder>"
• autopep8Path: "<placeholder>"
• provider: "black"
• blackArgs: "<placeholder>"
• blackPath: "<placeholder>"
• yapfArgs: "<placeholder>"
• yapfPath: "<placeholder>"

testing
• cwd: "<placeholder>"
• debugPort: 3000
• nosetestArgs: "<placeholder>"
• nosetestsEnabled: undefined
• nosetestPath: "<placeholder>"
• promptToConfigure: true
• pytestArgs: "<placeholder>"
• pytestEnabled: true
• pytestPath: "<placeholder>"
• unittestArgs: "<placeholder>"
• unittestEnabled: false
• autoTestDiscoverOnSaveEnabled: true

terminal
• activateEnvironment: true
• executeInFileDir: "<placeholder>"
• launchArgs: "<placeholder>"

experiments
• enabled: true
• optInto: []
• optOutFrom: []

insidersChannel: "off"

tensorBoard
• logDirectory: "<placeholder>"

Extension version: 2022.2.1924087327
VS Code version: Code 1.65.2 (c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1, 2022-03-10T14:33:49.188Z)
OS version: Darwin x64 21.4.0
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz (16 x 2300)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 3, 3

Memory (System)
16.00GB (3.28GB free)

Process Argv
--crash-reporter-id 504ccf47-f288-437d-9b17-d25ed5cd2111

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30443607
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
vsclayoutctrc:30451274
dsvsc009:30452663
pythonvsnew555:30450902
vscscmwlcmc:30438804
dockersurvey3:30453609
pynewfile477cf:30450039

Hi @oleynikandrey, thank you for reaching out, so sorry for the delay!
I could reproduce the issue you're facing, we'll have to make sure to address it as we rewrite our testing support to properly account for plugins (#17242).

Hello! We have just finished our testing rewrite and are beginning the roll out to users. I have tested this issue with the re-write and I am no longer reproducing the bug! If you would like to try it yourself, you need to add this setting to your users settings.json "python.experiments.optInto": ["pythonTestAdapter"]. Let me know if it doesn’t work for you and we can re-open this issue. Thanks!


Extract javascript html tags to separate module without custom attribute handlers

Add option to  include AttributeHandler only if wanted

Add option to  include AttributeHandler only if wanted

Abandoning this


spring-r2dbc get current transaction context in application code

Hi all!
I'm using spring-r2dbc and wanted to ask:
Is it possible to obtain current transaction context in application code?
Like this:
    @Transactional
    public Mono<MyEntity> find() {
        TransactionContext context = TransactionContextManager.currentContext().block();
        // ...
    }

I tried to do it, but keep getting NoTransactionInContextException.
Any workaround or correct way to do it?
Thanks.

Ok, I understand my mistake
To anyone facing the same question:
you can use transaction context in the mono returned by your method, not outside of it.
So the code will be:
    @Transactional
    public Mono<MyEntity> find() {
        return TransactionContextManager.currentContext().flatMap(
             ctx -> {/*...*/}
        )
    }

Reactive transactions require the subscriber context. Hence, using .block() will not work. You can only obtain a transaction in a reactive flow when returning publishers.