When I try to run the example jupyternotebook file, a ‘Keyerror" arose

Hello Florian,
Unfortunately I cannot help you with just this information. What I can say
is that I am working on more detailed examples to be published soon.
Hopefully they will be able to better help you understand how the library
works. I would like to remind you that the library is still in very early
stages of development and any help (through pull requests) is highly
appreciated. So if you run into bugs, major design problems and/or
suggestions, please open a pull request.
Best,
Gustavo Amarante
On Mon, Oct 24, 2022 at 3:49 AM Florianismeandonlyme <
@.***> wrote:

[image: image]
https://user-images.githubusercontent.com/58820847/197464448-7a59fe02-7e66-4c08-9189-fb3401ed3d04.png
—
Reply to this email directly, view it on GitHub
https://github.com/gusamarante/pydsge/issues/4, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHXQG5QKSGE3AM4TYISZUVLWEYWQBANCNFSM6AAAAAARMVNG24
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you for this job, Gustavo.
I have a problem.
When running Jupyter, I encounter an issue with the creation of the H5. You mention that if it doesn't exist, it's created. However, when I estimate the DSGE, it indicates that it doesn't exist and gives an error. Thank you."

Hello there. I am glad that you want to use DSGEPy.
With the information you sent me, it is not possible to pin down any
problems. But I will suggest a workaround for the most common cause of
issues. Are you using the correct version of pykalman? On the dsgepy
README, it points you to the correct version of the library that you need
to use. But here is the link https://github.com/gusamarante/pykalman to
it. Make sure that when you set up your python environment with this
version of pykalman.
Best,
Gustavo Amarante
On Tue, Dec 26, 2023 at 6:12 AM mhidper @.***> wrote:

Thank you for this job, Gustavo.
I have a problem.
When running Jupyter, I encounter an issue with the creation of the H5.
You mention that if it doesn't exist, it's created. However, when I
estimate the DSGE, it indicates that it doesn't exist and gives an error.
Thank you."
—
Reply to this email directly, view it on GitHub
https://github.com/gusamarante/dsgepy/issues/4#issuecomment-1869391738,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHXQG5RK7IPC7FWLJMZ7BUDYLKIJRAVCNFSM6AAAAAARMVNG26VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRZGM4TCNZTHA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Hi, Gustavo!! thanks for the response.
I believe so, but I'm not entirely sure. I'll make sure that it's the version I'm using and I'll let you know.

Hello,
The problem seems to be related with the file hd5 in pytables.
I have this error:

FileNotFoundError                         Traceback (most recent call last)
File [c:\Users\Usuario\anaconda3\envs\tf\lib\site-packages\dsgepy\lineardsge.py:171](file:///C:/Users/Usuario/anaconda3/envs/tf/lib/site-packages/dsgepy/lineardsge.py:171), in DSGE.estimate(self, file_path, nsim, ck, save_interval)
[170](file:///C:/Users/Usuario/anaconda3/envs/tf/lib/site-packages/dsgepy/lineardsge.py:170) try:
--> [171](file:///C:/Users/Usuario/anaconda3/envs/tf/lib/site-packages/dsgepy/lineardsge.py:171)     df_chains = pd.read_hdf(file_path, key='chains')
[172](file:///C:/Users/Usuario/anaconda3/envs/tf/lib/site-packages/dsgepy/lineardsge.py:172)     sigmak = pd.read_hdf(file_path, key='sigmak')
File [c:\Users\Usuario\anaconda3\envs\tf\lib\site-packages\pandas\io\pytables.py:418](file:///C:/Users/Usuario/anaconda3/envs/tf/lib/site-packages/pandas/io/pytables.py:418), in read_hdf(path_or_buf, key, mode, errors, where, start, stop, columns, iterator, chunksize, **kwargs)
[417](file:///C:/Users/Usuario/anaconda3/envs/tf/lib/site-packages/pandas/io/pytables.py:417) if not exists:
--> [418](file:///C:/Users/Usuario/anaconda3/envs/tf/lib/site-packages/pandas/io/pytables.py:418)     raise FileNotFoundError(f"File {path_or_buf} does not exist")
[420](file:///C:/Users/Usuario/anaconda3/envs/tf/lib/site-packages/pandas/io/pytables.py:420) store = HDFStore(path_or_buf, mode=mode, errors=errors, **kwargs)
FileNotFoundError: File ejemplo.h5 does not exist
Thanks


Getting the Project to Include Backend and Integrate Features

TL;DR: We need to add a backend to the project to handle server-side logic and integrating it with the existing frontend. We'll build APIs, improve the project’s functionality, and aim for smooth frontend-backend communication.

Overview
We need to extend the project by adding a backend to handle server-side logic, as well as integrate new functionality into both the frontend and backend. The backend will be implemented using Express (or your choice of framework) and will serve as the API for handling various features, like user authentication, data storage, and future functionalities.
Key Objectives

Backend Setup: Set up a Node.js/Express-based backend to handle API requests.
API Integration: Create REST APIs for core features like user management, data retrieval, etc.
Add Cache databases: Implementing redis database, cloudinary for image processing and so on.
Frontend-Backend Integration: Ensure seamless communication between frontend and backend through API calls (using proxy or direct routes).
Future Frontend Enhancements: Refactor some frontend components to work with the new backend and add any additional features required.

Detailed Plan

Create a server/ folder in the project root for backend code. The structure will follow MVC (Model-View-Controller) conventions.

Controllers will handle business logic.
Routes will manage API endpoints.
Models for database interactions.
Middleware for handling requests, authentication, etc.

Set Up Routes for core features:

User Authentication: Sign up, log in, and authentication flows.
Data Management: Routes to manage any dynamic content like user data or application-specific data.

API-Frontend Communication:

Update the Next.js next.config.js to proxy backend requests for seamless integration. (I will need some help on this, would be glad if anyone collaborated)
Modify frontend components to fetch data from the backend instead of hardcoded/static data.

Backend Environment Variables:

Use environment variables for sensitive data (e.g., database URL, API keys).
Include documentation or .env.example to guide setup.

Impact

This refactor will add server-side capabilities to the project.
Future contributions will require interaction with the backend for dynamic features.

Request for Feedback
Before proceeding, we'd love to gather feedback or suggestions on the following:

Any additional features or routes we should consider for the backend.
Best practices to follow for backend/frontend integration.
Ideas for improving user experience with the new setup.

For reference, I had actually done a few Authorization apis

Request: Please Assign Me this, with Gssoc ext, hacktoberfest And the level (i assume its a high difficulty one!)

Sure @BhairabMahanta you can work on this issue, just make sure to -

Send a photo here of postman auth before raising a PR
Follow the proper file structure
Star the repo :)

All the best!

Photos of working API's

Registering User
Entering OTP
Incorrect Password
Logging in
May I raise a PR now?

Great work @BhairabMahanta, please raise the PR


ncp: Add support for PROP_STREAM_RAW

This change allows the NCP to be a packet sniffer when used with host software that supports this feature.
See the section of the Spinel protocol document titled "Sniffing Raw Packets" for an example of how this property is intended to be used.

Added "work-in-progress" label since this code has not yet been tested (but I am working on that).

Current coverage is 65.06% (diff: 29.62%)

Merging #644 into master will decrease coverage by 0.08%

@@             master       #644   diff @@
==========================================
  Files            99         99
  Lines         14338      14365    +27
  Methods        1972       1976     +4
  Messages          0          0
  Branches       1684       1687     +3
==========================================
+ Hits           9341       9347     +6
- Misses         4466       4483    +17
- Partials        531        535     +4

Powered by Codecov. Last update 4bd6297...83202dd

a few more minor changes  are on the way.

The wpantund side of this change is being tracked by wpantund pull request #60.

Related to #34.


Incompatibility with Ghidra 11.1.1

Describe the bug
Ghidra removed PluginCategoryNames.DECOMPILER at some point, so the project won't build with the latest Ghidra version linked.
To Reproduce
Steps to reproduce the behavior:

set GHIDRA_INSTALL_DIR to the path of your Ghidra 11.1.1 installation
run gradle build
Observe
See error:

/path/to/Cartographer/src/main/java/cartographer/CartographerPlugin.java:70: error: cannot find symbol
    category = PluginCategoryNames.DECOMPILER,

Expected behavior
The project builds with latest Ghidra
Screenshots
N/A
Additional context
You should just choose an appropriate alternative category, with that the project builds.

#8, #9 and #10 both fix this issue

#8, #9 and #10 all fix this issue

Thanks, I missed those but they seem quite useful to base my further work on!


tracking pipeline

Thank you for the great work and the code release.
Is there any estimated date for the Demo release, because I am really curious how the code will run on a custom made dataset.
Thanks in advance!

You can check out the realtime inference I attempted on a coconut carton here: https://www.linkedin.com/posts/sia-zhen-hao_opencv-computervision-machinelearning-activity-6965918457116733440-IAXE
The images and labels used to generate the SfM point cloud are generated synthetically after 3D-scanning the object (the authors have yet to release the OnePose Cap app for collecting real data, so synthetic data will have to do for now).

ok, thanks


[$1000] Zooming in on an attached image can drag it totally outside of the screen

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

Action Performed:

Open mobile App and go to any chat
Send a mobile phone screenshot
Click to view in image previewer
Zoom in the image and drag/swipe it horizontally
Notice the screenshot image can be dragged totally outside of screen

Expected Result:
Image shouldn't be dragged outside of screen
Actual Result:
Image can be dragged outside of screen
Workaround:
Unknown
Platforms:

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

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/227988636-628ac493-2841-42c8-8ff1-324d5216b25d.mp4
https://user-images.githubusercontent.com/93399543/227988672-fe42d2bc-5bdb-47e2-b8a0-30d067cc1eb7.mp4
Expensify/Expensify Issue URL:
Issue reported by: @eh2077
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1679834477441149
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~016080ce57b6109b66
Upwork Job ID:<PHONE_NUMBER>781842432
Last Price Increase: 2023-04-06

Triggered auto assignment to @miljakljajic (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Chatted 1:1 with @zanyrenney and agreed that this is more of a minor UX improvement than a bug, but worth fixing!

Job added to Upwork: https://www.upwork.com/jobs/~016080ce57b6109b66

Triggered auto assignment to @techievivek (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

##Proposal
Please re-state the problem that we are trying to solve in this issue.
When image is zoomed you must not be able to drag/swipe it in full. It must stay in its borders.
What is the root cause of that problem?
The issue is caused by eact-native-image-pan-zoom library to use image preview.
What changes do you think we should make in order to solve the problem?
Either add option/parameter to disable the current behavior. OR use other library for image preview.
What alternative solutions did you explore? (Optional)
None

Proposal
Please re-state the problem that we are trying to solve in this issue.
image can be dragged totally outside of screen when Zoom in and drag/swipe it horizontally.
on also vertically (in specific case);
if width > height it outside when zoom and swipe vertically.
if height > width it outside when zoom and swipe horizontally.
What is the root cause of that problem?
we listen when image load onLoad={this.configureImageZoom} to get image width and height and calculate them based on the aspect ratio to use it in <ImageZoom ....
but we when calculate width and height based on aspectRatio we only change the large dimension of them and use the other dimension without change.
the problem here is the given width and height from onLoad={this.configureImageZoom} is the real size of image not the render size on screen.
   //we only change one dimension
    if (imageHeight > imageWidth) {
        imageHeight *= aspectRatio;
    } else {
        imageWidth *= aspectRatio;
    }

What changes do you think we should make in order to solve the problem?
we need to calculate both given real width and real height based on aspectRatio.
by replacing the code above with this.
imageHeight *= aspectRatio;
imageWidth *= aspectRatio;

result after change
https://user-images.githubusercontent.com/41129870/228984640-c5807341-1bfc-4417-aa25-cbaad928db01.mov
What alternative solutions did you explore? (Optional)

📣 @ahmedGaber93! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.

Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

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

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

Not overdue, waiting for @0xmiroslav to have a look at the submitted proposals.

@0xmiroslav what do you think?

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

@miljakljajic @techievivek @0xmiroslav this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@miljakljajic @techievivek @0xmiroslav this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@Prince-Mendiratta

When we upload an image of a mobile screenshot and open it up on native devices, it is possible to scroll way beyond the boundary of the container, leading to bad user experience.

I'm sorry I can't follow the root cause here. Like scroll way beyond the boundary of the container, what does this mean, and do we just allow users to scroll an image?

@ahmedGaber93
Though the proposal looks good to me I am having a hard time following it 100%, could we use diagrams to discuss this more?

@techievivek
the issue in calculating image dimension in function configureImageZoom
in Image component we use resizeMode={Image.resizeMode.contain} which maintain the image's aspect ratio.
How ResizeMode contain scaled image
one dimension scaled to fit screen and the othe will be calculate based on it and image aspectRatio.
but the two dimension will be change after scaled.

the correct way to calculate image dimension in image ResizeMode contain before image amount
note we can get them after image amount in Image onLayout

Sorry, I was unable to review it today. I will take a look at it first thing tomorrow.

@ahmedGaber93 Thanks for the additional notes, they made it easier to understand the proposal. Also, this medium article was helpful in understanding different resize modes.
But it looks like if we just do
imageHeight *= aspectRatio;
imageWidth *= aspectRatio;

then it may distort the image in cases when the container is wider/taller than the original image, no?
Also, regarding the alternate solution.

other solution is to get image width and height from image onLayout but it will be after images mount.

How would this be different from using onLoad?
let imageWidth = nativeEvent.width;
let imageHeight = nativeEvent.height;

I am going to cc @bernhardoj to have a look at the above proposal, as he seems to be familiar with the code.

@techievivek

then it may distort the image in cases when the container is wider/taller than the original image, no?

no, because we need to get the image width and height not set them, image dimentions already settings by react native image prop resizeMode={Image.resizeMode.contain}
we need only to get them to use it in <ImageZoom>

@techievivek
and regarding to use onLayout rather than onLoad
onLayout will give us the render dimentsion directly, but after image mount this may cause to visual glitches or shift according to this comment
https://github.com/Expensify/App/blob/ddcf4209b90c8917fca0eea4f97a19ffe538a0f2/src/components/ImageView/index.native.js#L219-L221

I agree with the proposals. The image size returned by onLoad is the real size of the image https://github.com/DylanVann/react-native-fast-image#onload-event--void, therefore we need to scale it uniformly.
And as @ahmedGaber93 mentioned above, it won't distort the image because both imageWidth and imageHeight is only being used for ImageZoom component. The image shown to the user is resized by Image (RN Fast Image) component.
So, Image and ImageZoom hold a different value of width & height, thus the issue happen. For example, an image height is greater than its width. If we are using the current logic, we only scale the height, leaving the width still has the original size.

That is why we can scroll to the excessive width.

Thanks, @bernhardoj, for taking the time to look at the proposal.

@0xmiroslav can you take a look at the submitted proposals, looks like we have a potential fix now.

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

Not overdue, the proposal by @ahmedGaber93 look good to me, waiting for @0xmiroslav to have a look and review the proposal.

@miljakljajic @techievivek @0xmiroslav this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ and will need to go internal. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks!

@techievivek @0xmiroslav PR is ready for review.

@0xmiroslav
I do not have permission to request reviewers.

Thanks @0xmiroslav.
I'm sorry, I missed that.
I edit it, but reviewers not changed, I think it works for the first time only.
What do you suggest now, can I close PR and open new one?

@0xmiroslav
I try the second option, but not work with me.

PR description

Reviewing label has been removed, please complete the "BugZero Checklist".

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.3.4-0 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/17620

If no regressions arise, payment will be issued on 2023-05-02. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] [@0xmiroslav] The PR that introduced the bug has been identified. Link to the PR:
[ ] [@0xmiroslav] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] [@0xmiroslav] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [@miljakljajic] Determine if we should create a regression test for this bug.
[ ] [@0xmiroslav] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
[ ] [@miljakljajic] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

Contracts extended to @0xmiroslav @eh2077 and @ahmedGaber93

@miljakljajic Thanks!
I think this issue eligible for bonus.
Assign 2023-04-18T12:45:40Z
Merged 2023-04-21T08:27:19Z
PR is merged within 2 days, 19 hours

Great, I'll apply the bonus on payment.

I think we're all good here! Paid, contracts ended. Closing!


Create uk-UA.json

Added Ukrainian localisation of the Plugin Manager.

 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.

Please update the file with new phrases (see other translations). And merge this edit with the change langs.json (another PR).

Hello, thank for your request. We have already added it in this commit.


docker.service fails to start

Initial check.

[X] I added a very descriptive title above.
[X] I searched through the issues in this repository to find a similar problem, but didn't find any.
[X] I searched through Google (including the 2nd page of results) and didn't find any relevant information.
[ ] I asked ChatGPT for help but did not get any meaningful response.

Please describe what triggered the error.
Installing docker and attempting to start it on Arch Linux via systemd.
What did you expect to happen?
Expected a local Docker engine to start running.
What went wrong?
× docker.service - Docker Application Container Engine
     Loaded: loaded (/usr/lib/systemd/system/docker.service; disabled; preset: disabled)
     Active: failed (Result: exit-code) since Tue 2023-05-30 18:14:53 BST; 3min 21s ago
TriggeredBy: × docker.socket
       Docs: https://docs.docker.com
    Process: 55042 ExecStart=/usr/bin/dockerd -H fd:// --containerd=/run/containerd/containerd.sock (code=exited, status=1/FAILURE)
   Main PID: 55042 (code=exited, status=1/FAILURE)
        CPU: 111ms

May 30 18:14:53 vesheljinn systemd[1]: docker.service: Scheduled restart job, restart counter is at 3.
May 30 18:14:53 vesheljinn systemd[1]: Stopped Docker Application Container Engine.
May 30 18:14:53 vesheljinn systemd[1]: docker.service: Start request repeated too quickly.
May 30 18:14:53 vesheljinn systemd[1]: docker.service: Failed with result 'exit-code'.
May 30 18:14:53 vesheljinn systemd[1]: Failed to start Docker Application Container Engine.

Please specify your 𝘤𝘰𝘯𝘥𝘢 version.
23.3.1
Operating System
Linux
Here you can specify more details regarding your operating system.
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Additional information
No response

I need some more specifics:

What was your exact command that is failing?
How did you install Docker?
What has systemd to do with this?
Are other containers built for u normally?
Have u ever seen this error before?

What was your exact command that is failing?

systemctl start docker.service is the main one, as running any docker command returns an error saying the service is not running.

How did you install Docker?

pacman -S docker

What has systemd to do with this?

It's the default system configuration software for Arch.

Are other containers built for u normally?

No, this is my first time trying docker. Clean install of docker following the official documentation.

Have u ever seen this error before?

Not outside this use case, but starting docker is the main issue so I suspect other containers would result in the same error.

service is not running.

Well, yes, first you need to start the engine to be able to run any docker commands
Hmm, I've never worked on Arch, cannot say if the installation was correct or not...

starting docker is the main issue so I suspect other containers would result in the same error.

I think this is pretty critical, could you please check another container?
Could you please try my old data-analytic container and let me know how it went? Alternatively (if sth fails there), any other from DockerHub, ex: default miniconda3 would do.
I have just tried warlock myself on another machine with Docker on Ubuntu 22.04 and worked fine.

So yeah, the problem is likely somewhere on my Arch laptop, and I haven't yet figured out what's up.
Ran it fine yesterday in a virtual machine on my girlfriend's Windows laptop, and also on my office computer running Ubuntu today.

Alright, thats what I suspected, given the container builds and runs on the CI: https://github.com/AngryMaciek/warlock/actions/runs/4942368731/jobs/8835817398
I consider it closed then!


feat: add contents of the message box to the prompt

Sometimes it's useful to supply some additional instructions to the model for generating the commit message as it can misinterpret the code change. This PR appends the text already entered in the message box to the prompt.

+1 this! Please merge @Sitoi

Seems to be included in 0.0.9, closing this.


data.MIDIVALUE has two different ways of indicating 1-based integers

@midi.instrnum says: "Use an integer for a 0-based value. An integer preceded by "in" indicates a 1-based value."
https://github.com/music-encoding/music-encoding/blob/develop/source/modules/MEI.midi.xml#L104-L105
However, data.MIDIVALUE itself says:
"Generic MIDI value. One-based values must be followed by a lower-case letter "o"."
https://github.com/music-encoding/music-encoding/blob/cc15a5423e132f574470e12c0e7b8900d00e9331/source/modules/MEI.xml#L2756-L2763
So which is it? in1 or 1o? I tried to follow the original issue (#510), which seems to suggest that 1o is what was chosen, but it should be clarified.
@pe-ro @craigsapp?

The final decision was to use the "o" suffix.  The description of @.***' is incorrect.
From: Andrew Hankinson @.>
Sent: Monday, March 14, 2022 8:13 AM
To: music-encoding/music-encoding @.>
Cc: Roland, Perry D (pdr4h) @.>; Mention @.>
Subject: [music-encoding/music-encoding] data.MIDIVALUE has two different ways of indicating 1-based integers (Issue #929)
@midi.instrnum says: "Use an integer for a 0-based value. An integer preceded by "in" indicates a 1-based value."
https://github.com/music-encoding/music-encoding/blob/develop/source/modules/MEI.midi.xml#L104-L105
However, data.MIDIVALUE itself says:
"Generic MIDI value. One-based values must be followed by a lower-case letter "o"."
https://github.com/music-encoding/music-encoding/blob/cc15a5423e132f574470e12c0e7b8900d00e9331/source/modules/MEI.xml#L2756-L2763
So which is it? in1 or 1o? I tried to follow the original issue (#510https://github.com/music-encoding/music-encoding/issues/510), which seems to suggest that 1o is what was chosen, but it should be clarified.
@pe-rohttps://github.com/pe-ro @craigsapphttps://github.com/craigsapp?
—
Reply to this email directly, view it on GitHubhttps://github.com/music-encoding/music-encoding/issues/929, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABLSX3MN45Y7A4VENEF5EGTU74UM3ANCNFSM5QVKTX2Q.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Thinking about it now, I like:

plain integers are 0-index
integers prefixed with in are instrument numbers that are 1-indexed

The idea with o postfix is that it means "ordinal" (1-indexed).   Perhaps using ord would be better to avoid confusing with 0.   This system is useful for applying across multiple attributes (particularly for MIDI).   For example MIDI channel 1 would be 1ord which internally in MIDI is the integer 0.  So I propose using ord instead of o to avoid any possible confusion between a letter and number.
Use of the in prefix makes it easier to understand, since in1 means instrument 1 (first instrument).  Likewise, MIDI channel 1 could be ch1 or 0.   While these could also both be done with 1ord, the range of MIDI channels is 0-15 while Instruments are 0-127, so that might help in range checking (for example ch17 is invalid).

@ahankinson is this any clearer for you now? Do we need to add anything that was written here to the guidelines?

The guidelines still have a discrepancy:

https://music-encoding.org/guidelines/dev/attribute-classes/att.midiInstrument.html
https://music-encoding.org/guidelines/dev/data-types/data.MIDIVALUE.html

The first says integers preceded by "in", while the second one (at the top of the page) says "One-based values must be followed by a lower-case letter "o"."
I think the fix is to change the description of data.MIDINAMES to align them.


Result Context, watchMutation, CacheRereadPolicy

Closes #793
Closes #774
Breaking changes

Broke default Mutation behavior, which can be regained by overriding DefaultPolicies.watchMutation.

Fixes / Enhancements

Fixed Mutation behavior by introducing watchMutation, which should mitigate the problems in #774
Added CacheRereadPolicy for controlling normalized re-reads and rebroadcast behavior more granularly
Added response.context to QueryResult (#793)
Added QueryResult.unexecuted singleton for unexecuted mutations

watchMutation default policies are the same as mutation:
Policies(FetchPolicy.networkOnly, ErrorPolicy.none, CacheRereadPolicy.ignoreAll)

the old behavior can be regained via
Policies(FetchPolicy.cacheAndNetwork, ErrorPolicy.none, CacheRereadPolicy.mergeOptimistic)

Docs

Added docs on CacheRereadPolicy and watchMutation

:tada: This PR is included in version 4.0.0-beta.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.0.0-beta.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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

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


Add a Feature Request issue template

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


iptables: Chain 'MASQUERADE' does not exist

I am trying to make the python script run on a raspberry pi zero W I have connected to my local network. It is running raspberry pi os.
When I try to add a new rule to iptables this is the console output:
$ iptables -t nat -A POSTROUTING -s my-local-ip/24 -o wlan0 -j MASQUERADE
iptables v1.8.2 (nf_tables): Chain 'MASQUERADE' does not exist
Try `iptables -h' or 'iptables --help' for more information.

Any idea on what to do? Am I maybe missing some additional dependencies?

Ah nevermind. I needed root permissions..


Could not get data. Check your read access key and ensure you have not reached your quota limit on self hosted version

Trying to get FSE Planner going on a Linux machine (Apache) because I just happen to have one of those running on my LAN but I can't get it to work.
Both the release file and a git clone with npm build give the same error when trying to pull data from FSE.
From what I can tell it's not actually trying to connect either and there are no errors in the browser debug console.
I did notice that while doing npm build it complains about an optional package 'fsevents' which is a MAC only package.
Not sure if FSE-Planner only runs on a mac ?

Should be working... I am running Windows (and Linux server), nothing to do with MAC...

What browser are you using to access the app? Does it work when you load data with this browser using the main website (https://piero-la-lune.fr/fseplanner/)?
Could you share the exact output you get when running npm build
Could you:
a. Open the Developer Tools in your browser, go to the Network tab
b. Try loading data (you should get the error)
c. Check the Network tab, you should see an outgoing GET request to piero-la-lune.fr => can you share a screen of the Network tab?

Should be working... I am running Windows (and Linux server), nothing to do with MAC...

What browser are you using to access the app? Does it work when you load data with this browser using the main website (https://piero-la-lune.fr/fseplanner/)?
Could you share the exact output you get when running npm build
Could you:
a. Open the Developer Tools in your browser, go to the Network tab
b. Try loading data (you should get the error)
c. Check the Network tab, you should see an outgoing GET request to piero-la-lune.fr => can you share a screen of the Network tab?

I have the whole build log ready but when checking the requests from the debug screen i noticed it is trying to query 'https://piero-la-lune.fr/fseplanner/proxy/data?userkey=etc' instead of the FSE server and getting a 404 not found.
Still need the build log after that one ?

I have the whole build log ready but when checking the requests from the debug screen i noticed it is trying to query 'https://piero-la-lune.fr/fseplanner/proxy/data?userkey=etc' instead of the FSE server and getting a 404 not found.
Still need the build log after that one ?

Ok, I get what is happening.
Because of the Cross-Origin header not being correctly set on FSE servers, the browser cannot directly interrogate FSE (you get a cross-origin error).
That is why I had to implement a proxy on my server, and route all requests through this proxy. I configured it so it would only accept requests from users using the https://piero-la-lune.fr/fseplanner/ instance. That is why you are getting a 404 error. FYI, my nginx proxy configuration:
location /fseplanner/proxy {

        if ($http_origin !~* "(^null$|^$|^https?://(piero-la-lune.fr|localhost:3000)$)") {
                return 404;
        }

        rewrite ^/fseplanner/proxy/(.*)$ /$1 break;

        # Activer le proxy
        proxy_set_header                X-Real-IP $remote_addr;
        proxy_set_header                X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_pass                      https://server.fseconomy.net;
        proxy_redirect                  off;
        proxy_buffers                   32 16k;
        proxy_busy_buffers_size         64k;

        # Ajouter les headers de contrôle d'accès CORS
        add_header    'Access-Control-Allow-Origin' "$http_origin" always;
        add_header    'Access-Control-Allow-Methods' 'GET' always;
        add_header    'Access-Control-Allow-Headers' 'Origin, X-Requested-With, Content-Type, Accept' always;
        add_header    'Access-Control-Allow-Credentials' 'true' always;
}

My bad, I should have stated in the README that you need to configure your own proxy to make it work.
You have to options:

Simpler - use my proxy: just give me the DNS/IP you are using to access your own instance, I will whitelist it
Better - use your own proxy: before building the app, you have to change the variable REACT_APP_PROXY in the .env file, and set up your own proxy in Apache (or use another existing one, such as https://cors-anywhere.herokuapp.com/)

Ok, I get what is happening.
Because of the Cross-Origin header not being correctly set on FSE servers, the browser cannot directly interrogate FSE (you get a cross-origin error).
That is why I had to implement a proxy on my server, and route all requests through this proxy. I configured it so it would only accept requests from users using the https://piero-la-lune.fr/fseplanner/ instance. That is why you are getting a 404 error. FYI, my nginx proxy configuration:
location /fseplanner/proxy {

        if ($http_origin !~* "(^null$|^$|^https?://(piero-la-lune.fr|localhost:3000)$)") {
                return 404;
        }

        rewrite ^/fseplanner/proxy/(.*)$ /$1 break;

        # Activer le proxy
        proxy_set_header                X-Real-IP $remote_addr;
        proxy_set_header                X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_pass                      https://server.fseconomy.net;
        proxy_redirect                  off;
        proxy_buffers                   32 16k;
        proxy_busy_buffers_size         64k;

        # Ajouter les headers de contrôle d'accès CORS
        add_header    'Access-Control-Allow-Origin' "$http_origin" always;
        add_header    'Access-Control-Allow-Methods' 'GET' always;
        add_header    'Access-Control-Allow-Headers' 'Origin, X-Requested-With, Content-Type, Accept' always;
        add_header    'Access-Control-Allow-Credentials' 'true' always;
}

My bad, I should have stated in the README that you need to configure your own proxy to make it work.
You have to options:

Simpler - use my proxy: just give me the DNS/IP you are using to access your own instance, I will whitelist it
Better - use your own proxy: before building the app, you have to change the variable REACT_APP_PROXY in the .env file, and set up your own proxy in Apache (or use another existing one, such as https://cors-anywhere.herokuapp.com/)

I'll see if i can set up my own proxy.
Tried giving the FSE people a poke about their header ?

I'll see if i can set up my own proxy.
Tried giving the FSE people a poke about their header ?

I asked, we will see what happens...
https://www.fseconomy.net/forum/feature-request/127259-cors-headers-for-datafeeds

Thanks to airboss, the correct HTTP headers have been added to the FSE datafeeds, and the proxy is no longer required


Update virtual-networks-nsg.md

Updated priority limits

Hi @Anitha-Adusumilli, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


Error when building with --prod switch

Issue description
I get the following error when running ng --prod
ERROR in Directive ɵb in<EMAIL_ADDRESS>has no selector, please add it!
Steps to reproduce and a minimal demo of the problem
Run ng --prod on a project that uses AGM AgmGeocoder.
Current behavior
Error at build time.
Expected/desired behavior
Build with no errors.
angular-google-maps, Angular, & any other relevant dependency versions
<EMAIL_ADDRESS><EMAIL_ADDRESS>

agm 3.0.0 works with angular 9.1+, not with angular 8. You should've received a warning when you installed it in npm/yarn.
To work with angular 8, use agm 1.1.0


Add code translation example

Type of Change
feature
Description
Add code translation example.
User can translate coding language from one to another (e.g. from Java to Python).
Two HTTP APIs are supported for both non-stream and streaming mode:

/v1/code_translation
/v1/code_translation_stream

TODO

[ ] add UT (WIP)
[ ] add ui (@WenjiaoYue)
[ ] enable CI (@chensuyue @XuehaoSun )

How has this PR been tested?
Local tested on Gaudi2. (Python, Java, C++, Golang, Rust)
Dependency Change?
None.

LGTM


Get-Date -UFormat does not match reference implementation [meta]

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Get-Date -UFormat
Format specifiers

[ ] powershell/powershell#20974
[ ] powershell/powershell#20975
[ ] powershell/powershell#20976
[ ] powershell/powershell#20977
[ ] powershell/powershell#20978
[ ] powershell/powershell#20979

See Date conversion specifiers or strftime.
Based on running the date -d $date +%c as a reference implementation (see Environment for details).
Expected behavior
# value issues
PS> Get-Date 1999-01-01 -UFormat %U
00
PS> Get-Date 2000-02-14 -UFormat %U
07
PS> Get-Date 2000-02-14 -UFormat %W
07
PS> Get-Date 2001-03-29 -UFormat %W
13
PS> Get-Date 2005-09-20 -UFormat %q
3
PS> Get-Date 2006-11-03 -UFormat %q
4

# presentational issues
PS> Get-Date 1999-01-01 -UFormat %c
Fri Jan  1 00:00:00 1999
PS> Get-Date 2000-02-14 -UFormat %c
Mon Feb 14 00:00:00 2000
PS> Get-Date 2001-03-29 -UFormat %D
03/29/01
PS> Get-Date 2002-05-12 -UFormat %D
05/12/02
PS> Get-Date 2003-06-25 -UFormat %x
06/25/03
PS> Get-Date 2004-08-07 -UFormat %x
08/07/04

Actual behavior
# value issues
PS> Get-Date 1999-01-01 -UFormat %U
0
PS> Get-Date 2000-02-14 -UFormat %U
6
PS> Get-Date 2000-02-14 -UFormat %W
6
PS> Get-Date 2001-03-29 -UFormat %W
12
PS> Get-Date 2005-09-20 -UFormat %q
q
PS> Get-Date 2006-11-03 -UFormat %q
q

# presentational issues
PS> Get-Date 1999-01-01 -UFormat %c
Fri 01 Jan 1999 00:00:00
PS> Get-Date 2000-02-14 -UFormat %c
Mon 14 Feb 2000 00:00:00
PS> Get-Date 2001-03-29 -UFormat %D
03-29-01
PS> Get-Date 2002-05-12 -UFormat %D
05-12-02
PS> Get-Date 2003-06-25 -UFormat %x
06-25-03
PS> Get-Date 2004-08-07 -UFormat %x
08-07-04

Error details
No response
Environment data
Prerequisites

    Write a descriptive title.
    Make sure you are able to repro it on the latest released version
    Search the existing issues.
    Refer to the FAQ.
    Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce

Get-Date -UFormat %q

This is not currently implemented, but could be added as (dateTime.Month-1)/3+1 or 1+int.DivRem(dateTime.Month-1, 3).Quotient. See Date conversion specifiers or strftime.
Expected behavior

Based on running the `date -d $date +%q` as a reference implementation (see Environment for details).

PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

WSL Ubuntu details:

$ uname -a
Linux localhost <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Thu Oct 5 21:02:42 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Visuals
No response

 $ & {
         'locale   : {0}' -f $env:LC_TIME
         'PSCulture: {0}' -f [Threading.Thread]::CurrentThread.CurrentCulture
         $mods =
         "a","A","b","B","c","C","d","D","e","F","h","H",
         "I","j","k","l","m","M","N","p","P","q","r","R",
         "s","S","T","u","U","V","w","W","x","X","y","Y",
         "z","Z"

         $mods | % {

             [pscustomobject] @{

                 'mode' = "%$_"
                 'UFormat'= Get-Date '2023-12-31' -uf "%$_"
                 'date' = date -d 2023-12-31 "+%$_"
             }
         }}

locale   : en_US.UTF-8
PSCulture: en-US

mode UFormat                  date
---- -------                  ----
%a   Sun                      Sun
%A   Sunday                   Sunday
%b   Dec                      Dec
%B   December                 December
%c   Sun 31 Dec 2023 00:00:00 Sun 31 Dec 2023 12:00:00 AM TMZ
%C   20                       20
%d   31                       31
%D   12/31/23                 12/31/23
%e   31                       31
%F   2023-12-31               2023-12-31
%h   Dec                      Dec
%H   00                       00
%I   12                       12
%j   365                      365
%k    0                        0
%l   12                       12
%m   12                       12
%M   00                       00
%N   N                        000000000
%p   AM                       AM
%P   P                        am
%q   q                        4
%r   12:00:00 AM              12:00:00 AM
%R   00:00                    00:00
%s<PHONE_NUMBER><PHONE_NUMBER>
%S   00                       0
%T   00:00:00                 00:00:00
%u   7                        7
%U   52                       52
%V   52                       52
%w   0                        0
%W   52                       52
%x   12/31/23                 12/31/2023
%X   00:00:00                 12:00:00 AM
%y   23                       23
%Y   2023                     2023
%z   z                        +0000
%Z   +00                      TMZ

It would be nice to add a column for strftime.

It would be nice to add a column for strftime

Found on Arch Wiki:
#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int
main(int argc, char *argv[])
{
    char outstr[200];
    time_t t;
    struct tm *tmp;
    t = time(NULL);
    tmp = localtime(&t);

    if (tmp == NULL) {
        perror("localtime");
        exit(EXIT_FAILURE);
    }

    if (strftime(outstr, sizeof(outstr), argv[1], tmp) == 0) {
        fprintf(stderr, "strftime returned 0");
        exit(EXIT_FAILURE);
    }

    printf("%s\n", outstr);
    exit(EXIT_SUCCESS);
}

 $ & {
         'locale   : {0}' -f $env:LC_TIME
         'PSCulture: {0}' -f [Threading.Thread]::CurrentThread.CurrentCulture
         $mods =
         "a","A","b","B","c","C","d","D","e","F","h","H",
         "I","j","k","l","m","M","N","p","P","q","r","R",
         "s","S","T","u","U","V","w","W","x","X","y","Y",
         "z","Z"

         $mods | % {

             [pscustomobject] @{

                 'mode' = "%$_"
                 'UFormat'= Get-Date -uf "%$_"
                 'date' = date "+%$_"
                 'strftime' = ./a.out "%$_"
             }
         }}

Output (TMZ is your timezone abbreviation):
locale   : en_US.UTF-8
PSCulture: en-US

mode UFormat                  date                            strftime
---- -------                  ----                            --------
%a   Mon                      Mon                             Mon
%A   Monday                   Monday                          Monday
%b   Jan                      Jan                             Jan
%B   January                  January                         January
%c   Mon 01 Jan 2024 10:28:58 Mon 01 Jan 2024 10:28:58 AM TMZ Mon Jan  1 10:28:58 2024
%C   20                       20                              20
%d   01                       01                              01
%D   01/01/24                 01/01/24                        01/01/24
%e    1                        1                               1
%F   2024-01-01               2024-01-01                      2024-01-01
%h   Jan                      Jan                             Jan
%H   10                       10                              10
%I   10                       10                              10
%j   001                      001                             001
%k   10                       10                              10
%l   10                       10                              10
%m   01                       01                              01
%M   28                       28                              28
%N   N                        882901279                       %N
%p   AM                       AM                              AM
%P   P                        am                              am
%q   q                        1                               %q
%r   10:28:58 AM              10:28:58 AM                     10:28:58 AM
%R   10:28                    10:28                           10:28
%s<PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER>
%S   58                       58                              58
%T   10:28:58                 10:28:58                        10:28:58
%u   1                        1                               1
%U   0                        00                              00
%V   01                       01                              01
%w   1                        1                               1
%W   0                        01                              01
%x   01/01/24                 01/01/2024                      01/01/24
%X   10:28:58                 10:28:58 AM                     10:28:58
%y   24                       24                              24
%Y   2024                     2024                            2024
%z   z                        +0000                           +0000
%Z   +00                      TMZ                             TMZ

I suggest to get WG conclusion that we want the UFormat to exactly match the Linux/C strtime.
We have previously made fixes for some formats and they were accepted as breaking changes because the behavior of UFormat should match its purpose, namely to match the Unix implementation.
Reference implementation
Alternative 1
Alternative 2
I suggest comparing our implementation with these, add what is missing, and correct deviations.


index.js:34416 Uncaught TypeError: createHistory is not a function

Version ^2.1.2
react-router version:  "react-router": "^3.0.0",
history version: "history": "^4.3.0",
Steps to reproduce
    import React from 'react';
import ReactDOM from 'react-dom';
import { Provider } from 'react-redux';
import { createStore, applyMiddleware, compose } from 'redux';
import { Router, browserHistory } from 'react-router';
import { reduxReactRouter, routerStateReducer, ReduxRouter } from 'redux-router';
import reduxThunk from 'redux-thunk';
import { createHistory } from 'history';

import rootReducer from './core/rootReducer';
import routes from './core/routes';

import * as axios from './utils/axios.js';

import { AUTH_USER } from './authentication/types';

const store = compose(applyMiddleware(reduxThunk),
    reduxReactRouter({ routes, createHistory })
)(createStore)(rootReducer);

const token = localStorage.getItem('token');

if (token) {
    store.dispatch({ type: AUTH_USER });
}

ReactDOM.render(
    <Provider store={store}>
        <ReduxRouter>
            <Router>
                {routes}
            </Router>
        </ReduxRouter>
    </Provider>
    , document.querySelector('.main')
);

Expected Behavior
Work properly.
Actual Behavior
It return an error index.js:34416 Uncaught TypeError: createHistory is not a function(…)

After changing import { createHistory } from 'history'; to import createHistory from 'history/createBrowserHistory';
The error become Uncaught TypeError: history.getCurrentLocation is not a function
Solution?

Did you solved this issue finally?
Thanks

@andfk no. I've changed my plugin to react-router-redux instead. maybe the problem lies within the history package i used is unsupported

@ssuhat great. I ended doing the same. Thanks for reply

Why was it closed? It still doesn't work. Moreover react-router-redux is not a solution. It doesn't have params in store. I propose to reopen this bug.

@Darmikon ok i reopen it.

Any news here? :(

I had the same error when using useRouterMatch, which apparently can't be used in the same component that defines the routing.
I don't know if it would be the same here, but in my case moving the  tags from App.js to ReactDom.render in index.js solved the issue.
`import {BrowserRouter as Router} from 'react-router-dom'
ReactDOM.render(

,
document.getElementById('root'))`


[feature suggestion] sparse-dense abstraction layer

This is just an idea, but I think it would be nice to provide a vector/matrix interface that abstracts away the concrete encoding, spare or dense. The abstraction layer would automatically convert between encodings, transparently to the user. For example,

multiplying two sparse matrices would yield a sparse matrix
multiplying two dense matrices would yield a dense matrix
multiplying a dense and a sparse matrix would either

if the dense matrix is spare, convert it to a sparse matrix and return a sparse matrix
otherwise, convert the sparse matrix to dense one and return a dense matrix
etc.

Since sparsity checks may be expensive, an important consideration would be when and where to perform them. There could be both an automatic mode, where sparsity checks & conversions are performed fully automatically, and a manual mode, where the user could say explicitly through the API when to perform sparsity checks.
I think such an abstraction layer could be super useful for prototyping.

Good comment, really appreciate it. Indeed, interoperation on different number types is really desirable when fast prototyping. Most of the time in my research, I simply want to try something in utop quickly.
That is actually the motivation of Ext module in Owl. With Ext module, you can do basic math operations on different number types. Please refer to this tutorial: Operators and Ext Module.
But price we pay for this convenience is: type checking (by wrapping everything in one type) and performance degradation (due to the big match). I haven't really benchmarked about performance degradation yet. However, Ext is just an option for you to do fast prototyping. You can always use the corresponding functions in Owl for better type checking and performance. I think the the ultimate solution is whenever Modular Implicits is included in OCaml official release.
Currently, this interoperation only happens between dense data structures, but Sparse structure is definitely on my todo list. Actually, there is already placeholder in this file for the feature you suggested, I just need to implement the corresponding modules in this file.
Before this happens, I still need to refactor the sparse module a bit (dense module takes me most of the time at the moement).

in my opinion abstracting sparse/dense is tricky. almost by definition, sparse matrices are big -- otherwise one would simply use dense matrices. but for big objects, you really want to be explicit about what you're doing to them, whether new memory is allocated etc. automating this is asking for unexpected out-of-memory situations.

Close for now due to removal of Sparse modules.


chore(litmus-portal): Added loader in verify and commit finish button

Signed-off-by: Amit Kumar Das<EMAIL_ADDRESS>
Proposed changes
This PR adds:

Loader in the finish button of verify and commit screen

Fixes the issue #2549
Types of changes
What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

[ ] New feature (non-breaking change which adds functionality)
[x] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices applies)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the CONTRIBUTING doc
[ ] I have signed the commit for DCO to be passed.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have added necessary documentation (if appropriate)

Dependency

Please add the links to the dependent PR need to be merged before this (if any).

Special notes for your reviewer:

/merge


[docs] Revise Joy UI "Modal" page

#33998
Hello hello @samuelsycamore ! I took forever getting back into this, but here it is!
I tried keeping it fairly similar to what our conversations have been in the past, and I'm curious to see the direction you go in regarding some parts of it, since I was kind of wondering what was important to mention vs what wasn't.
I hope you're having a good start to the week!

Also genuinely unsure why this branch has other stuff mentioned???

Hmm yeah there's some weird git stuff happening here... @LadyBluenotes have you sync'd your fork of material-ui recently? I think what we're seeing in the git history is from when I did some cleanup on a different PR of yours that involved me making changes to your fork. Soooo I believe you need you update your fork, and then merge that into this branch to clear up the conflicts, and then this should be ready for review. 🤝


python function app works locally but not when deployed, ImportError: ImportError: libpq.so.5: cannot open shared object file: No such file or directory.

Please provide us with the following information:

This issue is for a: python function app works locally but not when deployed,
- [ ] bug report -> python function app works locally but not when deployed, ImportError: ImportError: libpq.so.5: cannot open shared object file: No such file or directory.
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

deploy
execute

Any log messages given by the failure

Result: Failure Exception: ImportError: libpq.so.5: cannot open shared object file: No such file or directory. Troubleshooting Guide: https://aka.ms/functions-modulenotfound Stack: File "/azure-functions-host/workers/python/3.8/LINUX/X64/azure_functions_worker/dispatcher.py", line 271, in _handle__function_load_request func = loader.load_function( File "/azure-functions-host/workers/python/3.8/LINUX/X64/azure_functions_worker/utils/wrappers.py", line 34, in call raise extend_exception_message(e, message) File "/azure-functions-host/workers/python/3.8/LINUX/X64/azure_functions_worker/utils/wrappers.py", line 32, in call return func(*args, **kwargs) File "/azure-functions-host/workers/python/3.8/LINUX/X64/azure_functions_worker/loader.py", line 76, in load_function mod = importlib.import_module(fullmodname) File "/usr/local/lib/python3.8/importlib/init.py", line 127, in import_module return _bootstrap._gcd_import(name[level:], package, level) File "", line 1014, in _gcd_import File "", line 991, in _find_and_load File "", line 961, in _find_and_load_unlocked File "", line 219, in _call_with_frames_removed File "", line 1014, in _gcd_import File "", line 991, in _find_and_load File "", line 975, in _find_and_load_unlocked File "", line 671, in _load_unlocked File "", line 783, in exec_module File "", line 219, in _call_with_frames_removed File "/home/site/wwwroot/wsmtimer/init.py", line 16, in  import psycopg2 File "/home/site/wwwroot/.python_packages/lib/site-packages/psycopg2/init.py", line 51, in  from psycopg2._psycopg import ( # noqa

Expected/desired behavior

it should work

OS and Version?

default linux

Versions

v2

Mention any other details that might be useful

Thanks! We'll be in touch soon.

Was able to fix this by using psycopg2-binary instead of psycopg2 in your requirement.txt file
I referred here - https://learn.microsoft.com/en-us/answers/questions/174155/python-function-app-works-locally-but-not-when-dep
although binary usage is not mandatory, but I am not sure if there is any other workaround without using custom containers to run my function!


Doesn't support cc or bcc?

I don't get any e-mails that are in the cc or bcc in the header. When I removed mailhog, cc works. E-mail gets sent properly.

It's sort of an oversight...
If it's just a Cc or Bcc header in the SMTP message, this will be ignored entirely since MailHog doesn't attempt to do anything based off the SMTP message body. It only acts on RCPT TO SMTP commands.
Assuming your mail client is sending the RCPT TO commands for each cc/bcc recipient, MailHog should see those recipients (if you check the MailHog log output), but the message only gets returned once via the API, and is only stored once in the storage backend. Strictly speaking, MailHog doesn't store the 'delivery' of a message, it stores the raw incoming SMTP conversation.
The MailHog API should be returning all of the message recipients (e.g. in the .To and .Raw.To fields), but that gets ignored by MailHog UI.
I'm not really sure what expected behaviour would be - it's not something I ever considered, but storing the message only once regardless of the number of recipients naturally fell out of the way MailHog is built.
The obvious options would be storing the message multiple times (one per recipient), or making the information visible in the UI somehow.
How would you like it to work? :smile:

Hello,
I personally used to see all messages. If for example there is 1 recipient and 2x BCC recipients, then I see 3 emails in my inbox. I understand that when testing something, there could be hundrends of emails in the inbox - but I can't see any downsides of this.
Also, if there would appear 'release' button someday, then we could even deliver 1 on these messages (like sometimes customers while testing wanna see how email looks like).
Jaroslav

Well I would like to see all the e-mails otherwise, as a developer, I would be confused why my my app is not working. Which was the case :)

/\ Agreed

I'm happy to add something, just struggling a bit with how the UI (or API?) would handle that - listing the message multiple times (or, storing it multiple times) could be problematic, e.g. if the recipient list is a few hundred addresses.
The CC/BCC headers should be available in the UI, though they are hidden by default - perhaps making them visible by default instead might help? Or displaying the EHLO/RCPT/MAIL/DATA values somewhere in the UI too?
MailHog was designed as a tool to test the SMTP conversations, which is why each outgoing e-mail is only currently stored/displayed once regardless of the number of recipients. The API and UI are returning data captured from the EHLO/RCPT/MAIL/DATA commands, and just happens to make it available via a UI which looks vaguely like a mail client.
Making it display a single SMTP conversation multiple times (once per recipient) suggests a subtly different use-case to MailHog's original purpose, but not necessarily one we can't support :+1:

I don't get any e-mails that are in the cc or bcc in the header. When I removed mailhog, cc works. E-mail gets sent properly.

I'm not sure it's relevant to this, but it just occurred to me - the values of the CC/BCC headers don't actually matter for SMTP delivery, the message is only delivered to the recipients in RCPT commands.

The CC/BCC headers should be available in the UI, though they are hidden by default - perhaps making them visible by default instead might help?

How can I configure the UI to show Cc/Bcc headers? I can't find this in the docs.
I am attaching screenshot (made by modifying the page in a browser) how would I like to have Cc/Bcc displayed (in message detail).

I'm having the same issue and would also like to know if there is an option to turn on CC and BCC display in the UI?  If so, how?  I'm unable to find documentation for this.

@jankonas @eymclueles you can't currently make the UI always display CC/BCC data, but it's there if you click the Show Headers button on the right
I'll update the UI to display CC/BCC headers automatically, which should partially resolve this

@ian-kent Which release can I display the CC/BCC in the Web UI?

@amymengfan This issue is still open, so I don't think BCC/CC has been implemented yet.

@eymclueles OK, thank you. I will wait, or maybe try to fix it and rebuild the app.

So long as headers contain CC and BCC (testable via API?), that should be more than sufficient for testing that an email was sent with those headers. The idea of testing a server that is not controllable via application beyond following specification is confusing to me.

It seems to me that Mailhog also doesn't support BCC only mails (empty TO), has anybody experience with that?
Mail error: 550 Invalid syntax in RCPT command

@ian-kent
Could you please provide the steps to enable then CC functionality in Mail trap ?, i don't see  show header option ?

Conceptually I think of the Mailhog inbox as being the combination of everyone's inbox, so it would make sense to me for messages to show multiple times if they were sent to multiple people.
This limitation of Mailhog has contributed to a serious bug in making it to production in our application. Our application sends some emails with TO, and some with BCC. We've discovered an edge case that can cause it to send an email with the correct TO, but an additional, incorrect BCC. There's no way to spot that by looking at the mail in Mailhog, because it just shows the TO.

I think this BCC functionality doesn't work yet, and is required for my client and me on a dev environment. How can we make this work?

It seems that BCC and CC email headers do appear when the Show Headers section is open.
Although it is dependent on whether the email client drops the BCC header or not from the message.
For example, Sendmail doesn't drop the BCC header whereas PHPMailer's SMTP client does:
Email sent via Sendmail

Email sent via PHPMailer SMTP

So whether the BCC header is sent or not is dependent on the email client's implementation. The reasoning for this can be found in RFC5322 in section 3.6.3 and paragraph about the BCC field.

The difference between BCC and CC is that the CC email addresses are included in the email’s MIME (in the "CC" header field).
BCC addresses do not appear in the MIME source of the email, hence the recipient cannot see the BCC list. The BCC addresses are, however, passed in "RCPT TO" commands to the SMTP server when sending the email. (and so are the To and CC addresses.)

Any update on this ? I want to see BCCs and CCs from all methods of mailing either in headers or somewhere else. This is a needed feature and it's being asked for since 2016.

How much effort would it take to implement that? Are there technical complications? Would funding through something like BountySource make sense here?

See https://github.com/mailhog/MailHog/issues/261

+1

This is a big challenge for us with Mautic - we use Mailhog for our testing images so when users are checking if the correct things are being set by Mautic when emails are being sent, they don't see the BCC or CC fields. Would love to see an implementation :)

When using symfony/mailer package to send mail, the cc and bcc are not in the headers in mailgun. But using that same library if I send through Mailgun, it shows the cc and bcc in the headers in my Gmail? Any thoughts on that? There may still be a bug here somewhere on the display of the cc and bcc in the headers.


[BUG] FLATTEN does not flatten complex types

This may be a client problem, if it is parsed as JSON, it will be presented an [object Object] on the client


Unable to specify exported file name

Definitely need this.

@carthax12 I opened a pull request here.
https://github.com/Inspirare-LLC/Radzen.Blazor.GridExportOptions/pull/8
@JonasPetraska can you please review this when you have a moment? It's a fairly small change that could benefit a lot of users.

@JonasPetraska are you able to approve @StevenGarberg 's PR #8 and release a new NuGet version? This basic feature would be super helpful. Thanks!


Add Japanese locale

I added Japanese locale 😃
Almost words were translated by auto-translate service.
And then, I checked those words on development environment. Looks good.
It's not perfect for now. But, I hope those will be improved by other Japanese contributors.

@camelmasa we use Transifex for all our translations, if you want to help with translations can you ask to join there as a translator on the Japanese translation resource? It's already 40% translated.
https://www.transifex.com/ob1/openbazaar
When we do a release, we copy all the latest translations for the languages we currently support, plus any new ones that have reached 80% completion or more, from Transifex.
If we merged this PR, it would be overwritten the next time we pulled the files from Transifex. It's much better to make any changes there.

@jjeffryes Thanks for comment 😃 Got it. I'll join Transifex when I have time.


ng test: Uncaught TypeError: Cannot set property 'de' of undefined

Expected behaviour
When executing ng test command, it should not throw library involved errors, but it should proceed with the Karma test cases.
Actual behaviour
It throws the following error:

Chrome 59.0.3071 (Windows 10 0.0.0) ERROR
Uncaught TypeError: Cannot set property 'de' of undefined
at webpack:///~/bootstrap-datepicker/dist/locales/bootstrap-datepicker.de.min.js:1:0 <- src/test.ts:124812
Chrome 59.0.3071 (Windows 10 0.0.0) ERROR
Uncaught TypeError: Cannot set property 'de' of undefined
at webpack:///~/bootstrap-datepicker/dist/locales/bootstrap-datepicker.de.min.js:1:0 <- src/test.ts:124812

I would like to point out that when executing ng serve it works perfectly, I encounter issues just when testing.
Datepicker version used
^1.7.0.
Example code
This is where I'm importing the library:
Module:
import 'bootstrap-datepicker/dist/js/bootstrap-datepicker.min.js';
import 'bootstrap-datepicker/dist/css/bootstrap-datepicker3.min.css';

import { NKDatetimeModule } from 'ng2-datetime/ng2-datetime';

import 'bootstrap-datepicker/dist/locales/bootstrap-datepicker.de.min.js';
import 'bootstrap-datepicker/dist/locales/bootstrap-datepicker.es.min.js';

I added this code to karma.conf.js in order to avoid jQuery UI issues that I encountered previously:
files: [
	{pattern: './node_modules/jquery-ui-npm/jquery-ui.min.js', watched: false},
	{pattern: './src/test.ts', watched: false}
],

Finally fixed it!
I added to karma.conf.js this:
{pattern: './node_modules/bootstrap-datepicker/dist/js/bootstrap-datepicker.min.js', watched: false},
The final files array is like this:
files: [
	{pattern: './node_modules/jquery-ui-npm/jquery-ui.min.js', watched: false},
	{pattern: './node_modules/bootstrap-datepicker/dist/js/bootstrap-datepicker.min.js', watched: false},
	{pattern: './src/test.ts', watched: false}
],

@SrAxi Good to hear you've been able to fix the problem!


Not correctly served static files

Hi, i noticed that some static files fetched without correct HTTP headers and its causes compilation issues (please see the attached file)

same issue, in my case dist/assets/*.png with name with no whitespace char are download correct,but the logos with whitespace no

@issuehunt has funded $50.00 to this issue.

Submit pull request via IssueHunt to receive this reward.
Want to contribute? Chip in to this issue via IssueHunt.
Checkout the IssueHunt Issue Explorer to see more funded issues.
Need help from developers? Add your repository on IssueHunt to raise funds.

@sindresorhus has rewarded $45.00 to @hadeeb. See it on IssueHunt

:moneybag: Total deposit: $50.00
:tada: Repository reward(0%): $0.00
:wrench: Service fee(10%): $5.00


Update to Phaser 3.60 and fix related bugs

This commit updates Phaser to v3.60.0
Also:

updated phasereditor2d-launcher to version 3.61.0, for full Phaser 3.60.0 support
changed the NineSlice component to use the PhaserEditor2D NinePatchContainer, as updating Phaser broke the component.
changed any references from Phaser.Tweens.Tween.remove() to Phaser.Tweens.TweenManager.remove(Phaser.Tweens.Tween) as the former does not work with the overhauled tween manager.
changed the relative positioning of saved penguin cards from BOTTOM_RIGHT to CENTER to fix them being misaligned.
made the SnowballFactory set a snowball's Y position on the end of the tween, as the overhauled tween manager does not send the update event on the last frame of a tween anymore.

There is a conflict in the pull if you do not correct it the code will never be able to be pulled I advise you to check with the developer on Discord 😅

There is a conflict in the pull if you do not correct it the code will never be able to be pulled I advise you to check with the developer on Discord 😅
@HinataNekoNya Hello. I am in communication with the developer and talk to them frequently. It is not necessarily my responsibility to resolve these conflicts, as there were none when I originally made the pull request.
I will resolve them anyway.

Addressed in 8e363d5abda8cc2aefeb0f7da77df2eb8127601e


wasmparser: make ahash an optional dependency

The ahash dependency of wasmparser crate pulls in a considerable amount of dependencies. Here's an excerpt from cargo tree to illustrate it:
    │       ├── ahash v0.8.11
    │       │   ├── cfg-if v1.0.0
    │       │   ├── once_cell v1.19.0
    │       │   └── zerocopy v0.7.32
    │       │   [build-dependencies]
    │       │   └── version_check v0.9.4

This adds binary size overhead and introduces more unsafe code into the build.
I'd like to have the option to remove ahash and its dependencies from the dependency tree for use in high-assurance environments.

It seems that ahash is only used directly when the std feature is disabled: https://github.com/bytecodealliance/wasm-tools/blob/df542a839b1fc6c1345c68abf5da21ca9ef5f9fe/crates/wasmparser/src/map.rs#L112-L126
Something also enables the ahash feature in hashbrown in the final Cargo.toml uploaded to crates.io, although I'm not sure what does that exactly:
[dependencies.hashbrown]
version = "0.14.3"
features = ["ahash"]
default-features = false

You can inspect the Cargo.toml uploaded to crates.io like this:
cargo install cargo-dl
cargo dl -x wasmparser
cat wasmparser/Cargo.toml

Thanks for the report! Out of curiosity, have you noticed a measurable size increase with this support? Or otherwise are you mostly worried about the new deps?
Ideally this would be a no_std-only dependency but unfortunately Cargo doesn't have great support for that I think (or at least not that I know of). Otherwise the next-best-thing would probably be to work with ahash upstream to slim the set of dependencies. The ahash dependency isn't just used by wasmparser directly but also through hashbrown where the dependency is unconditional too.

Thank you for the quick response!

Out of curiosity, have you noticed a measurable size increase with this support?

My use case is very basic, and it seems ahash and all associated code just gets removed by dead code elimination. So now that I've measured it, the binary size overhead doesn't seem to be significant (when using only a small subset of wasmparser, anyway).

Or otherwise are you mostly worried about the new deps?

Yes. I want to make my code very easy to adopt for organizations, which often involves a security review of all dependencies. Until I added wasmparser, the dependency tree was very small and 100% safe Rust all the way down (except for std). I even rolled my own parser for ELF/PE/Mach-O so that I could guarantee binary parsing with no unsafe code, heap allocations or panics.
Now that wasmparser added large crates with unsafe code to the tree, adopting my project will be a lot more expensive.

The ahash dependency isn't just used by wasmparser directly but also through hashbrown where the dependency is unconditional too.

ahash is an optional dependency for Hashbrown, see https://github.com/rust-lang/hashbrown/blob/835936584e33480f6afbf1ae9be7151c1b27d595/Cargo.toml#L17

unfortunately Cargo doesn't have great support for that I think (or at least not that I know of)

Indeed: https://github.com/rust-lang/cargo/issues/1839
I think our best bet is to have std and no_std features and require the user to choose one. That would be a semver-breaking change, but given the v0.206.0 version number it doesn't look like frequent semver breaks are a concern.

Ok thanks for confirming on the binary size piece, that confirms my gut feeling here too. Also definitely understood about the dependency tree, that's a concern for us in Wasmtime as well!
Having to choose between std/no_std won't really compose well I think (e.g. Cargo's feature support doesn't support mutally exclusive features), but let me perhaps ask another question for this. Does your use case involve validating a WebAssembly module or just parsing it? If it's just parsing then I'm pretty sure we could easily put all the hash-map-bits and such behind a new "validate" feature or similar. That means you'd still be able to parse but wasmparser wouldn't be able to validate though.
If that'd work for your use case then that might be the best way forward?

Yes, that would be excellent! I do not need validation at all, I just need to pass the high-level structure. Being able to just disable the validation feature with its hashmaps and other heap allocations would be perfect.

Ok sounds good, I'll work on splitting that out in the near future. And also, yes, we don't currently have semver-compatible releases, so this'll be a 207.0.0 version.

Thanks a lot!
And yes, semver is a non-issue. I have anticipated that and didn't expose wasmparser types publicly in my libraries.

v0.207 with this change has been released, and I've upgraded my tools to it: https://github.com/rust-secure-code/cargo-auditable/pull/150
It works great for my use case. Thanks a lot!


[grpc] CodeQL Compiler missing Control Flow Guard in Third Party DLL files

Environment

OS: [Windows]

Description:
It appears vcpkg grpc is using external packages here, including ZLIB, SSL, PROTOBUF, ABSL, UPB, RE2, and CARES
It is important that the compiler for these third party dll files pass the /guard:cf on both the compiler and linker command lines. Binaries also require the /DYNAMICBASE linker option in order to enable CFG.
We have received particular warnings for libprotoc.dll (from PROTOBUF), libprotobuf.dll (from PROTOBUF), libprotobuf-lite.dll (from PROTOBUF) cares.dll (from CARES), and libcrypto-1_1-x64.dll (from SSL)

Please provide steps to reproduce these issues.

Thanks for posting this issue. Please reopen this issue if this is still a problem for you.


No connection to Anki on 2.1.17

Anki version: 2.1.17
AnkiConnect version: 2020-01-06
On main window:
"Connection Refused and no cached data"
When trying to refresh decks:
"No connection to Anki. Please run Anki and click Refresh button to reload all fields"
Anki is open in the background.

Thanks a lot!!


Active passive

using mc instead of python minio

Codecov Report

Merging #41 into master will decrease coverage by 0.23%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #41      +/-   ##
==========================================
- Coverage   52.37%   52.13%   -0.24%
==========================================
  Files         269      259      -10
  Lines       42151    32197    -9954
==========================================
- Hits        22076    16786    -5290
+ Misses      18138    13478    -4660
+ Partials     1937     1933       -4

Impacted Files
Coverage Δ

cmd/gateway-startup-msg.go
50% <0%> (-11.12%)
:arrow_down:

cmd/posix-errors.go
38.33% <0%> (-11%)
:arrow_down:

cmd/web-router.go
69.69% <0%> (-8.09%)
:arrow_down:

cmd/sts-handlers.go
14.63% <0%> (-8.01%)
:arrow_down:

cmd/xl-v1-common.go
79.48% <0%> (-7.47%)
:arrow_down:

pkg/quick/errorutil.go
78.43% <0%> (-6.38%)
:arrow_down:

cmd/fs-v1-helpers.go
63.82% <0%> (-6.21%)
:arrow_down:

cmd/fs-v1-rwpool.go
66.66% <0%> (-5.75%)
:arrow_down:

cmd/retry.go
81.81% <0%> (-5.69%)
:arrow_down:

pkg/quick/encoding.go
50.66% <0%> (-4.55%)
:arrow_down:

... and 236 more

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 7ad0fe8...2a08a9e. Read the comment docs.


Grid enhancements

windows copy paste \r\n issue
recomputeRows when width for any header change ( example. resizable cols)
modified style overflow:hidden in rowHeaderGrid as per react-virtualized docs.
modified code to support ellipsis for virtualized renderer ( also tested with flexrender-stories)
validating data before commit on edit

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat(scripts): update query-engine-wasm together with other engines

Update @prisma/query-engine-wasm together with other engines in the
pnpm bump-engines script.
This won't work yet until we start publishing the query-engine-wasm in
lockstep with other engines, and needs to be merged after
https://github.com/prisma/prisma-engines/pull/4491 and
https://github.com/prisma/engines-wrapper/pull/479 are merged and the
first engine commit is published with the new workflow.
Closes: https://github.com/prisma/team-orm/issues/586

the wasm engine is automatically published now, merging this


InsertAsync ignores explicit DbConnection

Version of Mighty
Mighty 3.1.3 (netstandard2.1)
Database
MariaDB 10.1.37
ADO.NET Driver and version
MySql.Data 8.0.23
.NET version
.NET 5.0
What happened
Example code:
private readonly DbConnection _connection;
public LicensesController(DbConnection Connection)
{
	// Initialize connection using dependency injection
	this._connection = Connection;
}
public async Task<IActionResult> AddNote(ulong Id, string Note)
{
	await using DbTransaction t = await _connection.BeginTransactionAsync();
	...
	await new MightyOrm<LicenseNote>().InsertAsync(_connection, new object[]{new LicenseNote
	{
		LicenseId = Id,
		Timestamp = DateTimeOffset.UtcNow,
		Content = Note
	}});
	...
	await t.CommitAsync();
}

_connection object is properly initialized and connection is open, but when InsertAsync is called, it throws exception:
MySql.Data.MySqlClient.MySqlException (0x80004005): Unable to connect to any of the specified MySQL hosts.
 ---> System.ArgumentException: The parameter 'host' cannot be an empty string.
   at System.Net.DnsEndPoint..ctor(String host, Int32 port, AddressFamily addressFamily)
   at System.Net.DnsEndPoint..ctor(String host, Int32 port)
   at System.Net.Sockets.Socket.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
   at System.Net.Sockets.Socket.ConnectAsync(String host, Int32 port)
   at System.Net.Sockets.TcpClient.ConnectAsync(String host, Int32 port)
   at MySql.Data.Common.StreamCreator.GetTcpStream(MySqlConnectionStringBuilder settings)
   at MySql.Data.Common.StreamCreator.GetStream(MySqlConnectionStringBuilder settings)
   at MySql.Data.MySqlClient.NativeDriver.Open()
   at MySql.Data.MySqlClient.NativeDriver.Open()
   at MySql.Data.MySqlClient.Driver.Open()
   at MySql.Data.MySqlClient.Driver.Create(MySqlConnectionStringBuilder settings)
   at MySql.Data.MySqlClient.MySqlPool.CreateNewPooledConnection()
   at MySql.Data.MySqlClient.MySqlPool.GetPooledConnection()
   at MySql.Data.MySqlClient.MySqlPool.TryToGetDriver()
   at MySql.Data.MySqlClient.MySqlPool.GetConnection()
   at MySql.Data.MySqlClient.MySqlConnection.Open()
   at System.Data.Common.DbConnection.OpenAsync(CancellationToken cancellationToken)
--- End of stack trace from previous location ---
   at Mighty.MightyOrm`1.OpenConnectionAsync(CancellationToken cancellationToken)
   at Mighty.MightyOrm`1.ScalarAsync(DbCommand command, CancellationToken cancellationToken, DbConnection connection)
   at Mighty.MightyOrm`1.ActionOnItemAsync(OrmAction originalAction, Object item, DbConnection connection, CancellationToken cancellationToken)
   at Mighty.MightyOrm`1.ActionOnItemsWithOutputAsync(OrmAction action, DbConnection connection, IEnumerable`1 items, CancellationToken cancellationToken)
   at Mighty.MightyOrm`1.ActionOnItemsAsync(OrmAction action, DbConnection connection, IEnumerable`1 items, CancellationToken cancellationToken)
   at Mighty.MightyOrm`1.InsertAsync(DbConnection connection, Object[] items)
   at Server.Controllers.LicensesController.AddNote(UInt64 Id, string Note)

The ActionOnItemAsync methods pass my DbConnection to DbCommand, but no DbConnection is passed to ScalarAsync. Then since connection parameter of ScalarAsync is null, it tries to create new connection using empty ConnectionString.
What should have happened
ScalarAsync should use DbCommand.Connection if property is set otherwise use current logic. And of course if this property is already set, connection should not be disposed.

Okay, let me have a look...

You are right that the DbConnection object is not being passed through in this case, and that this needs a fix; and associated new tests.
You additionally have the issue that you appear to be creating an instance of Mighty which does not know which database it is for (which should not be being permitted!), unless you have used the Mighty GlobalConnectionString setting (e.g. with just a ProviderName in the connection string) to set that elsewhere? (By the way, Mighty as currently designed is not able to set the database provider later than in the constructor, i.e. it does not currently detect this from any DbConnection passed in to individual methods; and it is not realistic to update it to do so, without a major redesign.)
Apologies for the delay in updating the code to get you a fix, but I wonder could you for now inject the connection string, or a class which provides the connection string? For the latter, you could look at creating an instance of Mighty's ConnectionProvider object.
I appreciate that what you are doing should work - and requires a fix! - but that said, passing in DbConnection objects is meant to be for exceptional cases in Mighty (though not least for managing transactions, if that is what you are doing?); this is because all the supported providers pool connections, so the normal use case is meant to be to pass in (or perhaps globally specify, if you wish) the connection string, and to allow the underlying provider to efficiently get you a DB connection from its pool.

Oh yes, I forgot to mention that ProviderName is set through GlobalConnectionString.
Sadly injecting ConnectionString is not a solution since I need to manage transactions manualy.
Anyway this issue is nothing critical for me. I wrote workaround using custom SQL query and DbCommand. Yes, it's not nice and clean solution, but it works now.

Thank you for your faith! I'll try to get out a fix asap.

Thank you for your faith! I'll try to get out a fix asap.

@jeffraska -
It turns out there was a lot that needed massaging and updating, to polish up both async support and explicit DbConnection support in the CRUD methods: New, Insert, Update, Save, Delete. I've pretty much chased through all the code paths, helped by the large - and now a little larger! - test suite, so now getting there: https://github.com/MightyOrm/Mighty/compare/fixes  .
I do need to run the last test of making sure that the new build is binary compatible with older builds, and also, probably, merging these changes on the v3 branch into the v4 branch before releasing them on v3 - both because I need to anyway, and because the v4 branch has additional checks on the consistency of method signatures.
So if possible please do bear with me, and thanks anyway for reporting something that it turns out definitely needed attention!

I have done the binary compatibility check, so this new alpha version is already stable and usable:
https://www.nuget.org/packages/Mighty/3.2.0-alpha1
I still need to do the 'merge with 4.0.0 and check the method signatures' step; so it is possible that the order of arguments in some of the new methods added above may change slightly before release.
But the above version is safe, stable and binary and compilation compatible with 3.1.3, in case it helps you to have the new features now rather than in a few more days.

Thanks very much. I'll try to test it in my project as soon as I can.

I have now published these updates as release 3.2.0.
The order of DbConnection and CancellationToken params will change in next major version of Mighty (was to be called v4, but will probably be called v5 now since will have explicit .NET 5 support), but that will apply to all methods, not just the few ones I newly added here - so basically what I put up as 3.2.0-alpha1 was already fine for 3.2.0.
I believe the problems you had are now fixed and tested as fixed - but do let me know if anything is not working for you.

I just test it, works great thanks.


Add rule to require space before comment

Our standards state that there is a space at
the beginning of each comment, but the corresponding
ESLint rule was never enabled.
This adds 'spaced-comment': [ 'error', 'always' ] to
our config.

Good addition!


Compile with config.json got error

Yes.  When you compile, you should specify the above dictionary (via the config.json - restler.exe compile ), which includes the custom payload you added above.  The compiler will read all the custom payloads in the dictionary and generate the corresponding grammar primitives.
Originally posted by @marina-p in https://github.com/microsoft/restler-fuzzer/issues/197#issuecomment-832898724
Hi, I've tried the config.json to automatically generate custom payloads in dict.json and grammar.py.
But I got an error while compiling by ./Restler compile config.json.
My two specifications are under /restler_bin/restler/spec dir
My config.json is under /restler_bin/restler:
1 {
2    "SwaggerSpecConfig": [
3        {
4             "SpecFilePath": "/spec/iottalkv1_ccm_api_swagger2.0_20210328.yaml",
5             "DictionaryFilePath": "ccm_dict.json"
6        },
7        {
8             "SpecFilePath": "/spec/iottalkv1_csm_api_swagger2.0_20210329.yaml",
9             "DictionaryFilePath": "csm_dict.json",
10            "Dictionary": {
11                "restler_custom_payload": {
12                    "min_max": ["[[1, 1]]", "[[1, 2]]", "[[3, 4]]", "[[1, 10]]"]
13                }
14            }
15        }
16     ]
17 }

The error is as following:
ERROR: Invalid format for compiler config file config.json. Please refer to the documentation for the compiler config file format.

ERROR: Deserialization exception: "An exception occurred during deserialization: Newtonsoft.Json.JsonReaderException: Error reading string. Unexpected token: StartObject. Path '', line 10, position 27."

How can I solve it??
Thank you !

Hello @rreexxllii0310,
Apologies for the delay.
This is due to a bug in our documentation.  The "Dictionary" field is a string rather than an object type, so it has to be specified as below (note, you can specify either the DictionaryFilePath or an inlined Dictionary but not both - the error is not very clear, we will update this as well).
Thanks,
Marina
 {
    "SwaggerSpecConfig": [
        {
            "SpecFilePath": "d:\\git\\restler-fuzzer\\demo_server\\swagger.json",
           "Dictionary": "{\"restler_custom_payload\": { \"min_max\": [\"[[1, 1]]\", \"[[1, 2]]\", \"[[3, 4]]\", \"[[1, 10]]\"]}}"
        }
     ]
}

@marina-p
Thanks for replying!
I'll try it later.

@marina-p
Hi, I've tried to use config.json to compile.
But, I got task failed error:
ERROR: Compiler failed. See logs in /home/rex/restler_bin/restler/Compile directory for more information.
I saw restler-20220117-121013.log in Compile directory:
Restler Information: 0 : Starting task Compile...
    DateTime=2022-01-16T16:10:14.0670826Z
Restler Information: 0 : Launching 'dotnet "/home/rex/restler_bin/restler/../compiler/Restler.CompilerExe.dll" "/home/rex/restler_bin/restler/Compile/config.json"'
    DateTime=2022-01-16T16:10:14.1496358Z
Restler Information: 0 : Process execution terminated in 00:00:00.8963547 with exit code 0x86: 'dotnet "/home/rex/restler_bin/restler/../compiler/Restler.CompilerExe.dll" "/home/rex/restler_bin/restler/Compile/config.json"'
    DateTime=2022-01-16T16:10:15.0499205Z
Restler Information: 0 : (207) dotnet "/home/rex/restler_bin/restler/../compiler/Restler.CompilerExe.dll" "/home/rex/restler_bin/restler/Compile/config.json" exited with code: 134
    DateTime=2022-01-16T16:10:15.0539154Z
Restler Error: 0 : Compiler failed. See logs in /home/rex/restler_bin/restler/Compile directory for more information.
    DateTime=2022-01-16T16:10:15.0591940Z
Restler Information: 0 : Task Compile failed.
    DateTime=2022-01-16T16:10:15.0614573Z
Restler Information: 0 : Collecting logs...
    DateTime=2022-01-16T16:10:15.0648925Z

Could you help me?
Thank you!

Hello @rreexxllii0310,
Could you please share the output in Compile\StdErr.txt, and the last few lines of Compile\StdOut.txt?
Thanks,
Marina

Hello @marina-p,
I saw the error in StdErr.txt, and I found the error was produced since I have two SpecFilePath with two different specs.
So, I tried to delete one of them, and It successed.
Is there any method to compile two specs at once?
Thank you!

@rreexxllii0310
Yes, they should be placed in an array as follows (see CompilerConfig.md)
"SwaggerSpecFilePath": [ "spec1.json", "spec2.json"]

@marina-p
Thank you!


Tesseract 4 and ita.traineddata

Hi,
when inizializing tesseract 4 with ita language (ita.traineddata for tesseract 4), the application crash on  configEngine method of G8Tesseract.mm.

Please help me, thanks.

We support only for C++. Please replicate problem with tesseract executable or C++ code. Otherwise please use tesseract user forum for your project.


Error while refreshing pending block on localhost: json: cannot unmarshal string into Go struct field JSONRpcResp.error of type map[string]interface {}

my config file
{
"threads": 2,
    "proxy": {
            "listen": "<IP_ADDRESS>:8546",
            "clientTimeout": "3m",
            "blockRefreshInterval": "100ms",
            "hashrateWindow": "15m",
            "submitHashrate": false,
            "luckWindow": "24h",
            "largeLuckWindow": "72h"
    },

    "frontend": {
            "listen": "<IP_ADDRESS>:8081",
            "login": "admin",
            "password": ""
    },

    "upstreamCheckInterval": "5s",
    "upstream": [
            {
                    "pool": true,
                    "name": "localhost",
                    "url": "http://<IP_ADDRESS>:8787/",
                    "timeout": "10s"
            },
            {
                    "name": "main",
                    "url": "http://<IP_ADDRESS>:8545",
                    "timeout": "10s"
            }
    ],

    "newrelicEnabled": false,
    "newrelicName": "MyEtherProxy",
    "newrelicKey": "SECRET_KEY",
    "newrelicVerbose": false

}
"url": "http://<IP_ADDRESS>:8787/",   is  eth-proxy   address
i don't know how to deal it,  so i post here.
Thanks

I think upstream 0 is not compatible in pool mode, so just remove it.

Thanks for your response.   if i remove  some code as below ,
{
"pool": true,
"name": "localhost",
"url": "http://<IP_ADDRESS>:8787/",
"timeout": "10s"
},
it  seems  work normal  ,  but  the eth-proxy  is useless ?   do i  still need eth-proxy ?
Thanks


Issue Finding More Options Button

If a user has enabled https://support.google.com/calendar/answer/10729749, it is possible that the extension's ability to locate and click the "More Options" button will be impacted. Instead, the user might find that the extension opens and different dialog, inviting them to modify their scheduling information.

There is a fundamental assumption baked into much of the Google-component that appears to be unreliable. Namely, that elements can be uniquely identified via attributes like [jscontroller] and [jsname]. This is the reason for the implementation below:
function findMoreOptionsButton(): HTMLElement | undefined {
  const potentialSelectors = [
    // orignal version
    'div[role="button"][jsname="rhPddf"]',
    // updated as of 24 Aug 2022
    'button[jsname="rhPddf"]',
  ];

  for (const selector of potentialSelectors) {
    const button = document.querySelector<HTMLElement>(selector);
    if (button) {
      return button;
    }
  }

  console.warn("brave-talk-gcalendar-extension: More Options button not found");

  return undefined;
}

What we have found instead is that attributes like [jsname] are insufficient to distinguish between elements at times. As such, a different approach will be needed to locate crucial elements, such as the "More options" button on the Quick Edit dialog.
One idea is to pass the active dialog element as a reference to the findMoreOptionsButton, and use that as a starting point to locate the local "More options" button.


Added Footer

CAN YOU PLEASE UPDATE IT ACCORDING TO THE LATEST MERGE..THIS HAS SOME CONFLICTS

this has some conflicts with the new merges can you please resolve


Should merging into a branch not indent the text?

Testing #226648
Current:

Proposal:

or

While the looping around line does look cool, it ends up indenting the commits which makes it look like something special may have happened on all the indented lines which it did not:

Good