MF-175: Stray blue dot on choose location page

https://issues.openmrs.org/browse/MF-175
The absolute positioning being applied on the input label in the style guide conflicts with positioning being applied on the labels in esm-login.

Can you post a couple screenshots -- one of the location page, and one of any other radio button selector we have (if we have any)?

Can you post a couple screenshots -- one of the location page, and one of any other radio button selector we have (if we have any)?

Sure:

Thanks! It looks a little misplaced on the location page -- do you know why that is? Presumably that's a problem with the location page? Is that what https://github.com/openmrs/openmrs-esm-login/pull/55 fixes?

Thanks! It looks a little misplaced on the location page -- do you know why that is? Presumably that's a problem with the location page? Is that what openmrs/openmrs-esm-login#55 fixes?

Dunno and nope, openmrs/openmrs-esm-login#55 doesn't fix that - it only gets rid of the competing CSS positioning. I think that's what this styling https://github.com/openmrs/openmrs-esm-styleguide/blob/6993e5c0bda308a83e52fefd5fcec041f65b3e30/src/forms/forms.css#L176 intended it to look like.

Yeah, I think the inner circle is meant to be centered. What's causing the difference between the location page and in the other pages? Is the location page doing something weird, or are the other pages doing something to fix it that should be moved into the styleguide itself?

Not sure yet. Playing around with top and left values on https://github.com/openmrs/openmrs-esm-styleguide/blob/6993e5c0bda308a83e52fefd5fcec041f65b3e30/src/forms/forms.css#L122  gets it working though.

And it still works the same in the other places?

And it still works the same in the other places?

Yeah


no redirect

no redirect/ What is magic button?

Can you update the uplugin and try once more?
We've shipped new version fixing previous issues

Can you update the uplugin and try once more? We've shipped new version fixing previous issues

@hokim-m Just thought I would comment and update regarding this. I tried it just now and the redirect and validation worked for me.


M45 Ghillie Helmet

https://cm-ss13.com/wiki/Marine_Equipment#M45_Ghillie_Helmet

Done in https://github.com/CM-14/CM-14/pull/606


Add Action-Func delegates project Mahdi Ghanbari

This is the project for the Action and Func delegates in C#
evaluation article


Array attribute

python 9 is not supporting the tostring() attribute of  array

does this apply to newest version (v2.3.0) of IntelHex?

Closing this as duplicate of #45


Add SBOM creation tool

Add SBOM creation script that supports CycloneDx and SPDX formats.
We currently support CycloneDx v1.5 and SPDX 2.3 formats.
To create an SBOM for target image, this script uses the official libraries cyclonedx-python-lib
and spdx-tools provided by the CycloneDx and SPDX projects, respectively.
Creating SBOM

Run source setup-emlinux  to setup emlinux build
environment
Run create_sbom.py with the following options (all options are required).

--image IMAGE_NAME

Image name whose components are described by SBOM(e.g. emlinux-image-base)

--sbom-format SBOM_FORMAT

SBOM format(e.g. cyclonedx or spdx)

--supplier SUPPLIER_NAME

Organization name who provides the SBOM

--product PRODUCT_NAME

Product name for the SBOM

SBOM output will be stored in the following path./tmp/deploy/sbom/<IMAGE_NAME>/<IMAGE_NAME>-<SBOM_FORMAT>.json
Example
Following command line creates a cyclonedx format sbom.
../repos/meta-emlinux/scripts/create_sbom.py \
 --image emlinux-image-base \
 --supplier "Cybertrust Japan Co., Ltd." \
 --product EMLinux3 \
 --sbom-format cyclonedx

Following command line creates an spdx format sbom.
../repos/meta-emlinux/scripts/create_sbom.py \
 --image emlinux-image-base \
 --supplier "Cybertrust Japan Co., Ltd." \
 --product EMLinux3 \
 --sbom-format spdx

Gathering license information
This tool gathers license information from copyright file from
/usr/share/doc//copyright.
Lincense name normalization
The SPDX project uses a license identifier called Identifier to identify licenses.
License identifiers are defined at https://spdx.org/licenses/. The license name
listed in the Debian package copyright file may not match the SPDX license identifier
and may not be able to properly identify the license. For example, the copyright file
of a Debian package may include GPL-3+, which indicates GNU General Public License or a
later version, but this notation cannot be recognized by SPDX. Therefore, it is
necessary to convert the Debian license name to an SPDX license identifier so that
it can be recognized by SPDX.
Define license information
EMLinux uses conf/licenses/license-mapping.yml as a file to map Debian license names
and SPDX license identifiers. If you need to map licenses that are not in this mapping
file, you can extend the mapping information by creating your own mapping file. When
using a user-defined mapping file, use --license-mapping as a command line parameter.
e.g. --license-mapping
The format of the mapping file is as follows.
"Debian license notation": "SPDX license identifier"

e.g.
"GPL-3+": "GPL-3.0-or-later"

Define license
If this tool can't find the license name from the copyright file,
"unknown" will be set as the package's license name. To reduce
such cases, the conf/licenses/licenses.yml is provided to supplement
the package's license information. This file can also be extended by
the user-defined file passed via the "--licenses" parameter.
e.g. --licenses
The format of the license file is as follows.
binary package name:
   licenses: [
     LICENSE A,
     LICENSE B,
   ]

e.g.
libpangocairo-1.0-0:
  licenses: [
    MPL-1.1,
    LGPL-2.1-only,
  ]

Thanks!


Session Currency

We need to work out the mechanisms with which the third-party doing spends against sessions is able to know and support the underlaying currency of the session

My suggestion would be that the sessions use an agreement currency that can be specified at the time of session creation. Although that raises some problems for a session that was created automatically. Another option would be to specify a currency when you request the balance and get it converted on the fly, but then you get some weird artifacts when the exchange rate fluctuates between the requested currency and the currency that the session is stored as
as usual currency adds a whole level of complexity :)
I think the one thing is that a site should never need to be aware of every user's currency. I'm more OK with having a list of currencies that wallets should generally support expressing amounts in


tests not running

When running the tests in main, the actions fail with an error
ERROR demonstrator_scripts/test_kpi_from_fem.py - AttributeError: module 'pint.compat' has no attribute 'upcast_types'

It has been like this since Thursday. I was hoping this will resolve itself, looks like a dependency of one of our dependencies has introduced this.
I will try to fix it now.


[Enhancement] Infer default storage medium when creating table (#14394)

What type of PR is this：

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

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

If the storage paths reported by all the backends all have storage medium type HDD, we infer that user wants to create a table or partition with storage_medium=HDD when not explicitly specify the storage_medium property, otherwise it's SSD
(cherry picked from commit 336b33cbe150ec526cdae99f3b759cd1ea981f3f)
Checklist:

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

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

Bugfix cherry-pick branch check:

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

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

run starrocks_fe_unittest


Postage batch buy stamps transaction succeeded after many hours but not shown.

An attempt to buy a postage batch buy stamps api call failed after some time, then the transaction succeeded, gBZZ was spent but the batch not shown from the list stamps endpoint.
The transaction took 10 hours.
https://goerli.etherscan.io/tx/0x5d3ed64a4325bfc5be5320065b1213d06460e158ca4eaf858f8bddeaf6d4ef53

Relates to #1559

This is fixed by: https://github.com/ethersphere/bee/pull/2080 . @significance , would be good if you can validate the fix.

should be fixed since the node will now add batches that appear on chain, even if the http call failed or was cancelled


BinderHub CI timeouts

Over the last few weeks BinderHub travis runs have been error'ing because of timeouts more often than not. This means they are essentially useless because we get in the habit of hitting restart all the time or merging despite travis being red.
From my observation it has been mostly in the part where we ask to build one of the CI repos. The error message doesn't say much so I wonder if this is because building that repo now takes a long time or because the test command that triggers the build can't reach binderhub and then times out?
Does anyone have an idea where to start digging or what changed?

One idea: should we explicitly fetch the ubuntu bionic base image before launching the tests? repo2docker will fetch it later but maybe that is super slow or otherwise leads to timeouts?

totally agree with this - I get really nervous when we start "assuming failure" in the tests :-/

We're no longer on Travis!


Added 3 quotes

Added 3 film quotes - I put them all at the bottom as requested. Hope it's all ok!

@DarylGraves your branch has merge conflicts, pls resolve and pr again


Bug? Can't generate batches using dynamic prompting v0.4.3

Increasing batch size does nothing with this script on. Tried using no wildcards, reducing size of image, different samplers, etc.  Batch size increases overall speed of generating many images in cards with lots of vram so it's very useful.
I have a 3090 using voldy commit hash f49c08ea566385db339c6628f65c3a121033f67c

I just tested on my end - batches work fine. Do you get an error message?

no error, but it says "Prompt matrix will create 1 images using a total of 1 batches."
when the script isn't active it doesn't say that

What if you choose batch count 8, batch size 4?
On Sat, 22 Oct 2022, 10:13 trihardseven, @.***> wrote:

no error, but it says "Prompt matrix will create 1 images using a total of
1 batches."
when the script isn't active it doesn't say that
—
Reply to this email directly, view it on GitHub
https://github.com/adieyal/sd-dynamic-prompting/issues/10#issuecomment-1287655992,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADGXMQFZH4JOAM57HX6XBDWEOHZVANCNFSM6AAAAAARLUZ7GQ
.
 Disabling the script and not changing anything else makes it batch 8 files

Also other scripts like X/Y plot seem to work fine with batch too

I'm trying to reproduce your issue. Have a look at these screenshots, are your settings the same as mine?

Ah, I misread your comment. I thought you said to try batch size 4 and then 8. Doing it with your settings worked, thanks.

There is a bug, i.e. batch count is currently defined as num images whereas it should simply be the number of batches. I'll fix that, but I wanted to make sure there wasn't another issue.

Fixed - it should now work the same way as the rest of the tool


feat: create device without default password(supported for only C8000V Autonomous and Fortinet firewall devices)

feat: create device without default password(supported for only C8000V Autonomous and Fortinet firewall devices)

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 25.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 36.12%. Comparing base (c71308e) to head (de55d31).
Report is 7 commits behind head on main.

Files with missing lines
Patch %
Lines

equinix/resource_network_device.go
25.00%
2 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #826      +/-   ##
==========================================
- Coverage   36.13%   36.12%   -0.02%
==========================================
  Files         184      184
  Lines       24491    24502      +11
==========================================
+ Hits         8850     8851       +1
- Misses      15480    15489       +9
- Partials      161      162       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix HttpClient bug: Don't use and dispose of a new HttpClient for each log

See http://aspnetmonsters.com/2016/08/2016-08-27-httpclientwrong/
If a new HttpClient is used and disposed of for each log, then we will
end up with port starvation on the machine that is running the app using
HttpEndpointLogProvider. However, if we only ever create one HttpClient,
then the app will bypass load balancing and DNS changes. The middle
ground is to use an HttpClient for some amount of time, then use another
one for some time, then use another one, et cetera. Each time we swap
out for a new HttpClient, we dispose of the one that had been switched
out during the previous swap. That way, any requests that are in process
have a time to finish.

Reviewed with the team on the phone / zoom.


Infinite retries occurring without a retry directive

There should be no retries in this case, but @jiyoontbd saw this happening awhile ago. We should try to repro.

Could not repro


Fix when primary key is not "int" or "long". Fixes #380

Could you resolve the conflicts from the merge.

Sorry I did not have time to look at it up. Happy to see you were able to fix it manually!


Suggestion - use a Pipfile

Instead of putting Python requirements in requirements.txt, we should consider defining requirements in a Pipfile.
Pipfile is a pretty new spec, but it will eventually get built-in support in pip.
For now Pipfile support is implemented in pipenv.  pipenv is recommended in Python's official documentation.
Here's a Pipfile I generated from the requirements file (after adding a missing dependency - see #1).
[[source]]

url = "https://pypi.python.org/simple"
verify_ssl = true
name = "pypi"

[packages]

flask = "==0.12.2"
flask-cors = "==3.0.3"
"ruamel.base" = "==1.0.0"
"ruamel.yaml" = "==0.15.35"

[dev-packages]

[requires]

python_version = "3.6"

@bajwa-adobe - I asked this question in an email a few days ago, but I'm not sure if you saw it -
Do we need the Python component at all?  It appears that it is currently just being used to load and apply changes to the config files.  If that is all it's intended to do, then why not just manage the config files in the Electron app instead of calling out to a Python process?
In my opinion, it would be better not to require Python to run the application, especially since most of it is in a self-contained desktop app.  The only downside I can think of is that we'd need to use Node to interact with the filesystem, which would probably make it impossible to run the application in a browser for testing purposes (which is fine in my opinion, since the application will always be distributed in an Electron container).
Let me know what you think.

I had initially used js-yaml but it lost comments on save.  Similar issue highlighted on this link
I haven't found any famous YML parser/writer in NodeJS that preserve comments and original keys order. Python module ruamel.yaml supports round trip preservation of comments.

I see - that makes sense.
I still think we should consider alternative approaches.  There are a few key issues with the Python API approach that concern me:

Introducing an API on localhost may cause red flags in the security audit process.  We'll have to test the application thoroughly in a variety of environments to ensure that it poses no security risk.

I think it would be better to limit the application's dependencies.  We can't guarantee that the system running the application will have Python installed.  We can build it with pyinstaller, which creates an executable that includes a Python runtime, but that will add about 10MB to the application's size.

At the very least, I don't think we can have the API.  Even if we do get it past the security audit, customers will not want a pop-up asking them to allow it through their local firewall.
Instead of making AJAX calls to an API, we should try designing the Python component to be called as a child process.  Communication with the process would be handled with stdin and stdout.
I say we try a proof-of-concept.  I'll submit a spec that I think will work in a separate issue.


Document extraVolumeMounts section

Description Documents the properties available to configure extra volume mounts.
Testing N/A
Documentation This is the docs.

Story created for DSO: https://maverics.atlassian.net/browse/DSO-844


Fix Python-Apple-Support dependency checkout

Last Python-Support release dropped support for arm7, armv7s builds so previous release tag must be pined until corresponding archs will be dropped from tdlib.

Signed-off-by: Russel<EMAIL_ADDRESS>

Thanks, @ERussel! We have similar fix in our private repository, so the pull request will not be merged, but I've mentioned it in the corresponding issue https://github.com/tdlib/td/issues/372.

The issue was fixed in TDLib 1.4.0 in a similar manner.


Add a program that creates reports from previously produced JSON files.

Turn the non-functional criterion executable in a very useful, albeit it basic and rudimentary, tool to convert previously produced JSON files into those fancy-pants HTML reports.

Solves #143

Thanks, @merijn. The code generally looks fine. Some comments:

Using the name criterion for this executable is perhaps a bit vague. I'm thinking we should rename it to something like criterion-report to better reflect its intended purpose.
Can you update the tutorial with an example of how you'd use this program?

Yeah, in general the code needs to be cleaned up a little too, I'll do that sometime this weekend.

Ok, fixed the missing other-modules and added a short bit to the tutorial. I have more interesting plans for the executable, but probably won't have time in the coming weeks to implement those.
Since you said you wanted to make a new release soon, it's probably best to keep the executable as-is, at the moment (i.e. limited to just generating reports of JSON output) and I'll start bugging people to merge something more interesting once I get around to finishing work on it.

This broke the master on all GHC's for a bit different reasons: https://travis-ci.org/bos/criterion

@phadej Yeah, I was relying on Travis for testing (due to lack of old GHCs locally), combined with a quick in-between meetings commit and a bit overeager merge that broke stuff. Should be fixed now that #144 is merged :)


why not h3d_std * X + h3d_mean ?

https://github.com/SuperMHP/GUPNet/blob/f4e26601ba5551065c36ab7ecf17f6915015d311/code/lib/models/gupnet.py#L165
according to

h3d should be equal to
h3d = self.mean_size[cls_ids[mask].long()] + size3d_offset * h3d_std

thx

Please note that the code here:
h3d in the code here is the $\mu_h$ in the paper, NOT the $h_{3d}$ distribution.

according to:
code/lib/datasets/kitti.py line 219:
size_3d[i] = src_size_3d[i] - mean_size
and
code/lib/models/gupnet.py line 165:
size_3d = (self.mean_size[cls_ids[mask].long()]+size3d_offset)
it means that the network predict dim_offset and size_3d[0] is the real/final h3d.
my fist question why not h3d_std * X + h3d_mean? is wrong. my bad.
wo de guo.


New aka.ms link

Aka.ms link name
SecurityRoles
Category (Optional. Select None to auto-select)
None

Thank you for submitting aka.ms/SecurityRoles. Your link already exists at akaSearch.net. 🙏✅


Rollbar-Java Doesn't include the logback.RollbarAppender object

It looks like you guys tried to merge the code, but it got blocked by your coding standards tool and was never integrated into your released API.

Thanks for letting us know! We'll fix it.


show securewatch message at the project level

src/main/resources/public/img/securewatch-go-to-plot.png

can you put some white space horizontally around your text?

src/main/resources/public/jsx/imagery/collectionMenuControls.js

205 It looks like its working OK without this check. Since secure watch is the only one that uses updateLayerWmsParams, its not needed. But if you want to make this future proof, keep, but nest under if (prevProps.visible !== visible) so it only fires once, when the user changes back to the securewatch layer.


MongoDB support please :D

Interested to have mongodb in dockerrails. thanks.

Hi @jackbit,
I don't have much experience with MongoDB but if you do feel free to propose a PR. It should be quite easy to do if you already know how to setup a dev mongoldb docker container.
Thanks !


Hpu bf16

Enable bert,albert,debert,resnet50,swin-large on HPU BF16

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 3 committers have signed the CLA.:x: ZhaiFeiyue:x: JoursBleu:x: huijuanzhYou have signed the CLA already but the status is still pending? Let us recheck it.

@huijuanzh could you please also add  HPU in here and here

@JianzheXiao any more comments?


New Query Methods

Checking to see if you're interested in adding something like this to the library. I feel like it would be a nice shortcut and would read well in the code. I would implement the full suite of normal moment queries (before, after, etc) and any others you would find relevant. I have implemented between here for you to review.
Thanks.

@Aldaviva IS this library abandoned or was my pull request not adequate/wanted? Thanks.

Sorry, I never got notified there was a pull request. Will take a look shortly.

Looks good, thanks for your contribution Brad! I would be happy for you to continue working on this.
It sounds like you wish to implement additional methods, so let me know whenever you want me to merge this in.

No problem! Thanks for taking a look. Let me add in some additional queries and ill let you know.

@Aldaviva , are you planning to merge this PR and release a new version?

@Aldaviva @gastonelhordoy I believe i'm the hold up here. @Aldaviva if you want to merge this in i can create another pull request with the additional methods when i have bandwidth. Pretty busy @ work right now.

@brad-decker I see, I understood this was a first PR and then you would add more stuff later in a new PR.
@Aldaviva maybe we can have this merged as is and then slowly release new features whenever @brad-decker  is free to keep extending the functionality.

Works for me. I will get this merged and released shortly.

Merged, bumped version to 0.1.0, published to NPM, added @brad-decker to contributors.

Sweet, thanks man!


Make sure to reactivate subscription when swapping plans

When a user has a cancelled subscription, and within a UI, swaps his plan selection, we must make sure to reactivate the subscription, otherwise Cashier will take into account that the user has a "valid subscription", and the Stripe webhook that comes a few seconds later will immediately re-cancel the plan.
Adding $subscription->cancel_at_period_end = false; makes sure the plan is reactivated.

Hmm, I believe you're right. But this should be a bugfix targeted to 8.0, can you send it to there?
Can you also perhaps add a comment above this line so it's clear why we do this? Something in the nature of "If the plan was cancelled before we need to be sure to reactive the subscription..."


howto: detect a broken connection

problem
if my salesforce connection dies, i want to be able to handle & respond to that event.  is this possible?
discussion
i get a connection object after logging in.  is there any such event that is emitted?  connection.on('disconnect|error|whatever', ...) i can hook into?
thanks!

triaging old issues...
the only event the connection emits is refresh here:
https://github.com/jsforce/jsforce/blob/b661afdd21729812974f1056e392f2f58fc21096/src/session-refresh-delegate.ts#L57
When using OAuth, jsforce will refresh the access token if it does an API with with and it's expired so you don't need to handle those scenarios.
If not using oauth, jsforce just throws when using an expired token.


Automator: update proxy@master in istio/istio@master

Generated by Automator - 2023-02-14T21:23:03+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
44b5c2ce11c9426d56d419d88551b56339d59095
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


ISSUE WITH KAMELET azure-servicebus-source output is message length i…

…nstead of message

Fixes #980

@bjethwan
With this you'll get the bytes related to the binaryData.

So you can serialize as json or you can convert the payload to a String.

@oscerd Thank you !
I was using the same in my local test environment to make the azure kamelet work.
Thank you so much.


fix: Update axios and fix issue in botframework-connector

#minor
Description
This PR updates the axios package and replaces it  with fetch in botframework-connector due to a breaking change in version 1.x.x of axios.
Specific Changes

Updated axios version to ^1.6.0 in botbuilder and botbuilder-core.
Replaced axios with fetch in the botFrameworkClientImpl class.

Testing
The following image shows a bot working using CloudAdapter with the botframework-connector fix.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
82 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.4%) to 84.185%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/botframework-connector/src/auth/botFrameworkClientImpl.ts
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

libraries/botframework-streaming/src/utilities/createNodeServer.ts
1
78.57%

libraries/botframework-streaming/src/namedPipe/namedPipeServer.ts
12
50.68%

libraries/botframework-streaming/src/namedPipe/namedPipeClient.ts
17
34.09%

libraries/botframework-streaming/src/namedPipe/namedPipeTransport.ts
51
3.92%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
20333

Relevant Lines:
22871

💛  - Coveralls

Would there be a release for this anytime soon?

This will be released the week of Jan 8.


Allow null

Pull Request Test Coverage Report for Build 472331117

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

Totals

Change from base Build 471980320:
0.0%

Covered Lines:
89

Relevant Lines:
94

💛  - Coveralls

Pull Request Test Coverage Report for Build 472331117

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

Totals

Change from base Build 471980320:
0.0%

Covered Lines:
89

Relevant Lines:
94

💛  - Coveralls


Can't switch between SCM providers

VSCode Version: 1.20.1
OS Version: Windows 7

Steps to Reproduce:

Install TortoiseSVN with command line tools
Install SVN extension
On SCM view click on more options, only enable this option -->

This is my SCM providers extensions ->

VSCode is detecting git and svn ->

¿Someone can tell me how can i switch between git and svn?

Code doesn't have a switch anymore. They can all work simultaneously. Just open a workspace with either a Git or SVN repository within and they will appear in the SCM view.


Fix update conf error for int,double,boolean

Fix #15816

@rongrong PTAL.

@apc999 @rongrong Hi, would you please take a look at this PR, it seems more simple and tiny than before, thanks for the suggestion from you!

alluxio-bot, merge this please


Documented database DDL statements and ordering

Using this kit, I found it initially difficult to understand what all these DDL and schemas did, given the sequential nested hierarchy. This doco hopefully makes it easier for whoever comes after me.

thank you for this PR, I am on the fence with it for the following reasons (though I agree anything is better than nothing):

I would like to have the documentation in one place and not spread around in different repositories (since there are more then one component that makes up the system)

While this is good info, it's structured as "your understanding" of what's going on, and as documentation it would need a bit of cleanup, and I don't want to request of you to start cleaning up and tailor this to what I think it should be :).

My question is:
When you had trouble understanding what's going one (and what each file/dir in here is about) is it because you did not find the following documentation (https://docs.subzero.cloud/architecture-and-project-structure/#postgresql-db and https://github.com/subzerocloud/postgrest-starter-kit#directory-layout) or was it not enough?
Thank you

Good questions! Thanks for considering the PR.
I did find that documentation and it was useful, but as a DBA, I start every project by understanding the code running in my database before adding to it. In Postgrest, with the database as the "source of all truths", I think it's doubly important to understand what functions, tables, permissions etc. exist and why, especially when building a project that might have a public-facing API URL.
I know that it's "my understanding" of what's going on, but I'm happy to amend this with your "developer truths" if you'd like to provide them. :)


fix metric bugs for lower, support mindir export with model yaml config

Thank you for your contribution to the MindOCR repo.
Before submitting this PR, please make sure:

[x] You have read the Contributing Guidelines on pull requests
[x] Your code builds clean without any errors or warnings
[x] You are using approved terminology
[ ] You have added unit tests

Motivation
(Write your motivation for proposed changes here.)
Test Plan
(How should this PR be tested? Do you require special setup to run the test or repro the fixed bug?)
Related Issues and PRs
(Is this PR part of a group of changes? Link the other relevant PRs and Issues here. Use https://help.github.com/en/articles/closing-issues-using-keywords for help on GitHub syntax)

如果是yaml配置，因为没有预设checkpoint url，建议 必须提供local checkpoint path 作为保护

如果是yaml配置，因为没有预设checkpoint url，建议 必须提供local checkpoint path 作为保护
added


Feature/csv download

Refs: https://openscience.atlassian.net/browse/LEI-246
Purpose
CSV Exporting
Summary of changes
Refactored to use csvWriter in exp-addons
Testing notes
Added some fixtures and a CLI addon to read JSON, may need to run npm install.

The carriage return behavior is a known issue: see this ticket for some discussion. https://bugs.jquery.com/ticket/9007

@binoculars commented on this pull request.
In tests/integration/components/export-tool-test.js:

+/**

A legible way to define a *SV file in js

@param {Array<(Array|String)>} arr An array of rows

@param {String} columnSeparator The column separator (defaults to ',')

@returns string

*/
+function svJoin(arr, columnSeparator) {
return arr

   .map((item, i) =>

       Array.isArray(item) ?

           (i > 0 ? item.map(quote) : item).join(columnSeparator) :

           item

   )

   .join('\n');

I noticed that also and meant to bring it up. I haven't found any documentation of this, but it may just be an OS thing--really don't know why it works this way. Maybe we can mitigate this by testing the blob in the download.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks for updating the submodule commit; dependent PR is merged.
Erica has requested that column order be deterministic. Once that change is ready, I'll do one last pass. Thanks!

Updated with fields order to be deterministic (see app/components/export-tool/isp-fields.js and the component for changes)


Change tagging step to use new action

As a developer,
I want to change the action responsible for the tagging step,
so builds do not fail when the version is not incremented.

Some options:

https://github.com/mathieudutour/github-tag-action
https://github.com/anothrNick/github-tag-action


fix: remove end of string space

what

As there is a whitespace before closing the quotes, when trying to enable notifications, Terraform always try to set it to null:
Code snippet:
  environments = {
    main = {
      branch_name               = "main"
      enable_notification      = true
[...]

Output:
  # module.amplify_app.aws_amplify_branch.default["main"] will be updated in-place
  ~ resource "aws_amplify_branch" "default" {
      - enable_notification           = true -> null
        id                            = "xxxxxxxxxxxxxx/main"
        tags                          = {}
        # (21 unchanged attributes hidden)
    }

why

references

/terratest


S22 Ultra - Game Area Detection Problem

Preparation

[X] I read the Troubleshooting Guide
[X] My question isn't answered in any other issues (even the closed ones)

Your question
FGA Build:
1651
Problem:
Have tried the troubleshooting guide and adjusting settings / searching history but can't find specific information.
Ignore Notch Calculation: On
FGO is running full screen no black bars.
Game Area Mode: Default
Choosing support works perfectly but it is unable to work when the game loads in.
Doesn't use abilities / choose selections correctly.

Not sure how to configure the game area to work properly?
Android version
12
Screen size
3088 x 1440
RAM
12GB

Did you ever find a solution to this? I'm having the exact same issue.

I did figure it out, was stupidly that the fight speed wasn't doubled, was slow and it fixed the whole thing.

Darn. I'm still getting boxes drawn in the wrong place. Out of curiosity, what are your camera notch settings (both in FGA and the phone settings)?

I have "Ignore Notch Calculation" ON in FGA.
The phone setting is full screen, no notch.


Scheduler Job - Urgent care

Create a scheduler job to get num urgent care facilities into GCS bucket
Urgent care: https://hifld-geoplatform.opendata.arcgis.com/datasets/urgent-care-facilities/data
Possibly https://catalog.data.gov/dataset/urgent-care-facilities
Lorenzo- not sure how these datasets differ

This issue will now also include a function/service to get the data from GCS into bigquery and any data standardization that needs to be done within bigquery.

Maya - thanks for the update
Krista -  it looks like they are the same, the second "source" seems to just acknowledge that the first source exists

Cloud function deployed. We'll need to discuss how to structure our code base to have this checked in.


Using special characters like & or / or ' in a user's message doesn't work

Describe the bug
When I use special characters in a user's message then they are escaped.
If I try to escape them then I get an error.
To Reproduce
Steps to reproduce the behavior.
set a user's message that contains & or / or ' character
#user
Show me a market snapshot in David&Niagra
Expected behavior
I don't want it to be escaped if I use \ character e.g. \&
Screenshots and Log files
When I don't use the \ character then I have to replace the characters myself in requestHook

If I use \ character then I get the error
#me
show me a market snapshot in David&Niagra

    Multiple neighborhoods with varied utterances.Case1/Line 5: error sending to bot - composing body from SIMPLEREST_BODY_TEMPLATE failed (SyntaxError: Unexpected token & in JSON at position 112
        at JSON.parse (<anonymous>)

      at SimpleRestContainer._getMustachedVal (node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:412:19)
      at SimpleRestContainer._getMustachedCap (node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:407:17)
      at SimpleRestContainer._buildRequest (node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:346:36)
      at SimpleRestContainer._doRequest (node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:272:17)
      at SimpleRestContainer.UserSays (node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:158:17)
      at node_modules/botium-core/src/containers/PluginConnectorContainer.js:169:37
      at node_modules/promise-retry/index.js:29:24
      at processTicksAndRejections (internal/process/task_queues.js:94:5))
      ########################################
      Multiple neighborhoods with varied utterances.Case1/Line 5: error sending to bot - composing body from SIMPLEREST_BODY_TEMPLATE failed (SyntaxError: Unexpected token & in JSON at position 112
          at JSON.parse (<anonymous>)
      at SimpleRestContainer._getMustachedVal (node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:412:19)
      at SimpleRestContainer._getMustachedCap (node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:407:17)
      at SimpleRestContainer._buildRequest (node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:346:36)
      at SimpleRestContainer._doRequest (node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:272:17)
      at SimpleRestContainer.UserSays (node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:158:17)
      at node_modules/botium-core/src/containers/PluginConnectorContainer.js:169:37
      at node_modules/promise-retry/index.js:29:24
      at processTicksAndRejections (internal/process/task_queues.js:94:5))
      ----------------------------------------
      {"err":{},"message":"Multiple neighborhoods with varied utterances.Case1/Line 5: error sending to bot - composing body from SIMPLEREST_BODY_TEMPLATE failed (SyntaxError: Unexpected token & in JSON at position 112\n    at JSON.parse (<anonymous>)\n    at SimpleRestContainer._getMustachedVal (/Users/x/Projects/chatbot-poc2/node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:412:19)\n    at SimpleRestContainer._getMustachedCap (/Users/x/Projects/chatbot-poc2/node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:407:17)\n    at SimpleRestContainer._buildRequest (/Users/x/Projects/chatbot-poc2/node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:346:36)\n    at SimpleRestContainer._doRequest (/Users/x/Projects/chatbot-poc2/node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:272:17)\n    at SimpleRestContainer.UserSays (/Users/x/Projects/chatbot-poc2/node_modules/botium-core/src/containers/plugins/SimpleRestContainer.js:158:17)\n    at /Users/x/Projects/chatbot-poc2/node_modules/botium-core/src/containers/PluginConnectorContainer.js:169:37\n    at /Users/x/Projects/chatbot-poc2/node_modules/promise-retry/index.js:29:24\n    at processTicksAndRejections (internal/process/task_queues.js:94:5))","input":{"sender":"me","channel":null,"messageText":"show me a market snapshot in David\\&Niagra","stepTag":"Line 5","not":false,"asserters":[],"logicHooks":[],"userInputs":[]}}
      at BotiumBindings.wrapBotiumError (node_modules/botium-bindings/src/BotiumBindings.js:98:12)
      at node_modules/botium-bindings/src/BotiumBindings.js:139:33

Botium Flavour:
Botium Bindings
Botium Version
Botium Bindings 2.0.29
Additional context
Add any other context about the problem here.

Can you please check your botium-core version - when looking at the line numbers, they seem outdated.

In my test with the latest botium-core version I only had troubles with the single quote character, the others worked. Although the JSON specification requires the single quote character to be escaped, the Javascript JSON parser fails on this. I checked in a workaround in the latest botium-core codebase.


pull step reported a SUCCESS even if the a artifact was not found in the cache server

See original issue on GitLab
In GitLab by [Gitlab user @jjardon] on Aug 20, 2018, 14:21
Summary
This is

confusing
makes difficult to debug anything with the cache server, as is not possible to know if the artifacts are actually not there or they are they are but they are not being pulled from some reason

Steps to reproduce

Configure a bst project to use a cache server
This cache server is populated with the correct artifacts
Try to build
The build logs doesn't show any error / warning / log message about missing artifacts, even the pull step is SUCCESS
(for this project) bst still tries to build everything from scratch, when it should get things from the cache server

What is the current bug behavior?

Lack of logging to debug the problem

What is the expected correct behavior?

If the artifact is not in the artifact cache, at least a SKIPPED message should appear, better the SUCCESS

Other relevant information

BuildStream version affected: 1.1.6

In GitLab by [Gitlab user @Qinusty] on Aug 20, 2018, 14:34
To confirm:
Even if the artifact is not found -- This cache server is populated with the correct artifacts
Does the cache server contain the artifacts or is it empty?
Also if anything has changed then buildstream will rebuild regardless.

In GitLab by [Gitlab user @Qinusty] on Aug 20, 2018, 14:40
BuildSteam does not currently notify the user when the cache does not contain the desired artifact.
See this block of code

In GitLab by [Gitlab user @BenjaminSchubert] on Aug 24, 2018, 11:46
assigned to [Gitlab user @BenjaminSchubert]

In GitLab by [Gitlab user @BenjaminSchubert] on Aug 24, 2018, 15:26
mentioned in merge request !729

In GitLab by [Gitlab user @Qinusty] on Aug 29, 2018, 17:31
This is addressed by !729 by [Gitlab user @BenjaminSchubert]. I suggest we backport this ready for Buildstream 1.2.0.

In GitLab by [Gitlab user @BenjaminSchubert] on Aug 29, 2018, 18:00
mentioned in merge request !752

In GitLab by [Gitlab user @Qinusty] on Aug 30, 2018, 16:35
Backported in !752

In GitLab by [Gitlab user @Qinusty] on Aug 30, 2018, 16:35
closed


Disallow blank spaces for key name

Repro steps:
Leave leftmost column blank for a row
Expected:
The row is ignored
Actual:
Empty is mapped to the value of that row and every empty value replaced.

We’ve seen this issue too, it was really funny.


OCPBUGS-25313: On the alert details page, don't require all alert labels in URL

Before this change we required all of the alert's labels to be in the URL query params. This change allows the alert to be shown even if only some of its labels are in the URL, as long as there is only one such matching alert.
Fixes a bug where the alerts links in "Notifications" didn't work because they didn't include the alert's external labels.

/jira refresh

/jira refresh

/cherry-pick release-4.15

/lgtm


[e2e]: go tag main config-ldflags-noassets slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v14.2.37
Run: https://github.com/slsa-framework/example-package/actions/runs/2794619051
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-noassets.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-noassets.slsa3.yml
Trigger: push
Branch: v14.2.37
Date: Thu Aug  4 06:34:23 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/v14.2.38
Run: https://github.com/slsa-framework/example-package/actions/runs/2801710750
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-noassets.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-noassets.slsa3.yml
Trigger: push
Branch: v14.2.38
Date: Fri Aug  5 06:35:14 UTC 2022
Tests are passing now. Closing this issue.


finalize AAD functional test plans, fix Rego bug for 7.4 and change AAD P2 references to new license name

🗣 Description
Note: The G5 test plan includes tests that work for E5 as well. G5 was just a convenient way to indicate "a tenant which has the required P2 license".
This pull request includes the following changes:

Finalized the AAD functional test plans: generic test plan, G5 test plan and G3 test plan. The G3 plan tests ScubaGear to ensure it behaves correctly in tenants that do not have an AAD Premium P2 license. In G3 tenants ScubaGear should produce a message indicating that the license is missing.
Fixed a Rego bug in 7.4 - it was giving the wrong result in G3 tenants. To correct this I updated all the Rego policies (including 7.4) that require a P2 license and made sure that the fix is implemented consistently across the code base and I unit tested it.
Since Microsoft has changed their P2 license name in the AAD and M365 admin portals, I changed the ScubaGear display name of the AAD P2 license from Azure AD Premium P2 to Microsoft Entra ID P2. This change was made in the README, in the Rego, in the Rego unit tests and in the baseline document license sections.

Note: I mostly unit tested the functional test plans against the E5 tenant and the G3 tenant (only G3 specific tests) for now. I have to setup the G5, G3 and GCC High tenants to fully test the functional test plans - that will be handled as a separate activity from this PR.
*Thanks @crutchfield for your extensive support on this one.
Closes #553
🧪 Testing

[ ] Tested the generic functional test plan and the G5 test plan against E5.
[ ] Tested the G3 test plan against G3.
[ ] Tested the new Rego code against G5, E5 and G3 tenants to ensure it produces the license error message when against G3. Ensured that it produces the correct policy result for 7.4 and all policies that check the license against G3.
[ ] Tested the report output to ensure that the license message references the new Microsoft license name.

Generic tests:

G5/E5 specific tests:

G3/E3 specific tests:

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced
in code comments.
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Revert dependencies to default branches.
[ ] Finalize version.

✅ Post-merge checklist

[ ] Create a release.

Tagging @ahuynhMITRE because this PR includes changes to the AAD baseline document.

Testing Instructions for executing the automated functional test orchestrator
Refer to issues #592 and #591 for instructions on how to setup the conditional access policies in the tenant and execute the automated functional test orchestrator.
The user account that execute the test script with must have permissions to modify the tenant configurations associated with AAD policies.
The MITRE E5 tenant already has the conditional access policies setup.

Confirmed the expected output was obtained for MS.AAD.7.4v1 in the ScubaGear report for G3:

Will work with @tkol2022 to coordinate running the aad functional test against the different tenants.

Can we get a screenshot of 2.1 and 2.3 since those are G3 specific also?

@ahuynhMITRE I added a missing instruction sentence to AAD 5.2 that references "User Consent Settings." 5.4 instructions already have it. Make this change to Word version.

The test plans themselves look good as they are. Comments below are of things I noticed. Note I may have missed some = to := .
For the Rego for 5.1. The helper function AuthPoliciesBad_2_6 should be renamed to AuthPoliciesBad_5_1

Fixed this in my last commit

@buidav @ssatyapal123 I accepted all the suggested revisions and made the updates suggested that were not automatic accepts. Performed a merge with the latest Emerald branch and tested all the functions that were modified. Please resolve all the conversations and approve this PR. Thanks a lot for your testing and feedback!

@nanda-katikaneni  ready for merge testing.

The unit test failure is not related to this pr and is being addressed by #613 - merging.


Improve false positives when using Secret with stringData in k8s

What this PR does / why we need it:
Merge stringData into data before calculate diffs to prevent false positives.
Which issue(s) this PR fixes:
Fixes #2636
Does this PR introduce a user-facing change?:

Improve false positives when using Secret with stringData in Kubernetes App

/golinter fmt

I fixed. PTAL.

Thank you!
/lgtm

Thank you for reviewing.
I applied your suggestion. PTAL.

I fixed. PTAL

Great job.
/lgtm
Just a nit about the release note.

Fix a bug that causes wrong diff result on Secret manifest when the stringData field is used

Thank you.
It's not completely fixed yet (and it probably won't be), so I think it's better to include that nuance. What do you think?

Thank you. It's not completely fixed yet (and it probably won't be), so I think it's better to include that nuance. What do you think?

Sorry, I don't understand this. Why hasn't it been fixed yet?
I am thinking that this PR fixes our root problem. Please fix me if I am wrong.

/lgtm cancel

@Hosshii Back to the release note, how about this?
https://github.com/pipe-cd/pipecd/pull/3299#issuecomment-1053888140
And I think we can merge this PR now so we can check its behavior in our dev environment. If you want to add any further update later you can do by sending another PR.

https://github.com/pipe-cd/pipecd/pull/3299#discussion_r815610947

I mean users can specify the same key in both fields but it is meaningless so we don't have to care about that.

If we don't take that into account, then the release notes are fine with that. I was thinking that it might be better to mention somewhere that it doesn't take it into account (because it can generate wrong diffs for k8s valid (if meaningless) manifests), but I also think it might be too verbose, so I'll leave it to you.

And I think we can merge this PR now ...

I agree this.

/lgtm

Here you go 🚀
/approve


helm-create-plots errors

Hello,
When I try to run helm-create-plots --suite v1 outside the presentation directory, i.e, outside helm/src/helm/benchmark/presentation; I get following error:
FileNotFoundError: [Errno 2] No such file or directory: '/home/helm/benchmark_output/runs/v1/groups/the_pile.json'
Is this expected behavior? In the docs, I did not find any details in regards to running the command in a specific directory.

The problem here is that we try to create plots for every group, even for groups that are not missing to the current suite. We should change this to just skip creating plots for groups that don't exist in the current suite.

Hello, could I work on this?

@shalakasatheesh Sure, I assigned the issue to you for now. Let me know if you have any updates.
@gxxu-ml Sorry, I spent some time looking into this and I didn't find any quick workarounds.
cc @dtsip

Good catch @shalakasatheesh ! Yes, we should just check whether the corresponding group exists for each plot we are making.
(my guess is that the issue you are getting with the_pile is because there is a plot visualizing performance as a function of The Pile perplexity.)

@shalakasatheesh wonder if you have figured out a configuration or a place to change to address the issue? This feels like a very general problem that people run into ~~

Hello, @yifanmai thank you. While working on this I noticed a couple of things and I would like some feedback on it:

@dtsip was right, the plot visualization is a function of The Pile perplexity. How would you like me to go forward with this? Shall I keep the visualization w.r.t. The Pile perplexity as a separate function? Should the user parse an argument for this visualization?

While running the  following configuration,

entries: [    {description: "mmlu:subject=anatomy,model=huggingface/gpt2", priority: 1},    {description: "mmlu:subject=philosophy,model=huggingface/gpt2", priority: 1},  ]
the meta data for the model 'huggingface/gpt2' can't be found, resulting in the following error:

Traceback (most recent call last):
  File "/home/shalaka/venv/bin/helm-create-plots", line 8, in <module>
    sys.exit(main())
  File "/home/shalaka/repos/helm/src/helm/benchmark/presentation/create_plots.py", line 595, in main
    plotter.create_all_plots()
  File "/home/shalaka/repos/helm/src/helm/benchmark/presentation/create_plots.py", line 567, in create_all_plots
    self.create_all_accuracy_v_model_property_plots()
  File "/home/shalaka/repos/helm/src/helm/benchmark/presentation/create_plots.py", line 350, in create_all_accuracy_v_model_property_plots
    self.create_accuracy_v_model_property_plot(
  File "/home/shalaka/repos/helm/src/helm/benchmark/presentation/create_plots.py", line 287, in create_accuracy_v_model_property_plot
    key = model_name_to_property(model_name)
  File "/home/shalaka/repos/helm/src/helm/benchmark/presentation/create_plots.py", line 335, in get_model_release_date
    release_date = self.model_metadata[model_name].release_date
KeyError: 'huggingface/gpt2'

I'm not sure how to go about solving this. Your feedback would be much appreciated :)

@gxxu-ml unfortunately not yet. Maybe tweaking create_plots.py could help you find a quick fix?

Thanks for the investigation @shalakasatheesh ! This is basically #1146 (adding GPT-2 to schema.yaml). I can fix this next Monday (or you are welcome to send a PR earlier, if you like).

Any update on this?

Could someone please, provide a workaround? As simple as "here is 'the_pile.json' file, drop it in the directory to avoid the error.

My suggestion is to skip the plotting step until we get it working.
@percyliang I think we should drop plotting from the tutorial documentation because it just doesn't work for most people.

cc @dtsip

It is too late to remove plotting – people saw it and they want it.
Please, just make it possible to display benchmarks scores in a table in a human-readable format.
From: Yifan Mai @.>
Reply-To: stanford-crfm/helm @.>
Date: Tuesday, July 25, 2023 at 4:09 PM
To: stanford-crfm/helm @.>
Cc: "Ermolin, Sergey" @.>, Comment @.***>
Subject: Re: [stanford-crfm/helm] helm-create-plots errors (Issue #1441)
My suggestion is to skip the plotting step until we get it working.
@percylianghttps://github.com/percyliang I think we should drop plotting from the tutorial documentation because it just doesn't work for most people.
—
Reply to this email directly, view it on GitHubhttps://github.com/stanford-crfm/helm/issues/1441#issuecomment-1650685933, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF7BVAKHPLJPPI266HHJJVDXSBGZFANCNFSM6AAAAAAWONHXTM.
You are receiving this because you commented.Message ID: @.***>

My timeline for fixing plots is late Q3 to early Q4. If you need this earlier, feel free to submit a PR. I'm happy to help with review or pointers.

Plotting is now considered deprecated, so I will close this issue. Perhaps we can revisit this in the future.


Create Consumer interfaces up to 8 parameters

Currently Consumers stop at 5 (in practice this is probably enough), but we have functions that support 8 parameters so it may be a good idea to have consumers which support 8 also. Use cases are likely to be around Method Reference creation for legacy code that is badly in need of refactoring!
Would facilitate 'currying' and partial application for consumers with up to 8 parameters.

not needed - 5 is enough

not needed - 5 is enough


Don't swallow the exception. At least log it.

That's a good idea, I think pytest.fail (https://pytest.org/latest/builtin.html?highlight=pytest.skip#_pytest.runner.fail) would work perfectly here, it would fail the test and log an exception.

That would also work :)


Is possible we could use another high performance serialization framework?

Is possible we could use another high-performance serialization framework to replace json.net?
Just like Jil or NETJSON.
I mean maybe overwrite method or configuration. Thanks

Replacement is not possible, however, supporting an additional JSON serializer would be totally acceptable.

Any action here or can this be closed?


NGOs added

Issue solve #102
JobSewa Contribution Checklist ✅
Before submitting a pull request, please ensure that you've followed the guidelines below. This will help expedite the review process. Please remember to follow the Code of Conduct.
Important: Starring the repo shows your support and will ensure your contributions are reviewed in a timely manner.

[x] I have starred and forked the JobSewa repo.
[x] I have carefully read the instructions for contributing to the project.
[x] I have created a new page or section and placed it inside the appropriate folder.
[x] My contributions align with the current project structure and conventions.
[x] I have included any necessary images in a dedicated folder within assets, ensuring they are compressed and under 500kb.
[x] All images are in .png format and have been directly uploaded—no external image links are used.
[x] I understand that the maintainers will review my pull request and provide feedback if necessary.
[x] I have assigned a reviewer to this pull request.

Thank you for your contribution to JobSewa! We’re excited to work with you.

Good work, bro, and Happy Diwali! 🎉 I've merged your PR, but I noticed a couple of issues: it seems some of the new NGOs might not have images, or they’re not showing up properly on the organization's page. Let’s take a look at that to make sure everything displays as expected. Thanks for your contributions!
You can update your PR whenever you're free to attach images for the NGOs and make them look consistent with the others; otherwise, I'll need to revert your merges. Let me know if you need any help with this. Thanks!

Yes that's because you will have to put photos in your cloudinary account where you have put other icons.
Also a very happy diwali to you too!🪔☺
Thanks for merging the PR

I appreciate your contributions to the project.
Could you provide me with the pictures for the new NGOs? That way, I can upload them to Cloudinary and ensure they display correctly on the organization page.

Here are pictures of NGOs with their names. Some don't have a logo image.

Here are some more logos


Storage is "null" when using cobblestone generator

Context
I'm working on creating the Iron farm from this schematic:
https://createmod.com/schematics/ultimate-iron-farm-1300-ingot-hr
Starting this farm causes an instant crash. I'm playing the Create Extra modpack w/ a few mods added:
Sophisticated Backpacks
Sophisticated Storage
Tom's Simple Storage Mod
Trinkets
Distant Horizons
Crash Report
https://mclo.gs/je5jOei

Try removing valkyrienskies

Yep, that fixed it! Interesting that that mod in particular causes it, since Create: Interactive depends on it


Improve docs regarding import of 'main'

I spent so long banging my head against the wall, and I feel like a clarification in the docs could help others.
In all the tutorials, you import a variable called 'main' but never actually use the variable. I thought it was weird, so I just removed the import. After many hours, I realized that it is very necessary to import main...
I have never seen an API that requires importing a variable but not using the variable. In fact, most linters will say this is an error. I think because of this non-standard API choice, the tutorials should warn people that 'main' MUST be imported for apps to be served properly.

Thanks for reporting @spencerchubb! The main is being imported so that we can hack around Uvicorn requiring if __name__ == main as an entry point. If you are familiar with marker interface, this is very similar and can be called a "marker import".
While I agree the unused import is non-standard, it's the best we can do. Any PR that could make it better is more than welcome.
Meanwhile, do you have any suggestions on where this should be included in the docs to save time and prevent head injuries from banging against the wall? :)

Perhaps in the 'Apps' section of the docs would be a good place to put it? Since it seems to be required for apps but not for scripts
https://wave.h2o.ai/docs/apps
Something like this:
# Note: main must be imported even though it is not used
from h2o_wave import main, app, Q, ui

@app('/foo')
async def serve(q: Q):
    # Modify the page
    q.page['qux'] = ui.some_card()

    # Save the page
    await q.page.save()

Yes, that sounds reasonable. Would you mind opening a PR yourself?


⚠ Jenkins Plugins Artifactory has degraded performance

In f770aff, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1186 ms

Resolved: Jenkins Plugins Artifactory performance has improved in dd949e6 after 12 minutes.


Review guide for replacing servers list

We received some feedback that this guide has incorrect or outdated instructions. Test the guide and update as needed.
See: https://redocly.com/docs/cli/guides/replace-servers-url/

Also please update the following:

remove the "you do, we do" directive, it doesn't strike the right note of professional encouragement
remove the step numbers from the titles (they make the articles harder to maintain/update, so we don't use this format any more)


Refactor bundler test spec ci

https://github.com/zsluedem/aa-bundler/pull/3 here is the test pr I made for myself

LGTM. One question: What is the main reason for RUST_LOG=aa_bundler=TRACE?

LGTM. One question: What is the main reason for RUST_LOG=aa_bundler=TRACE?

Just for debugging when tests failed.


Type definitions for Jquery Knob

Created a definition file for the small but very useful jQuery Knob plugin (http://anthonyterrien.com/knob/)

jquery.knob/jquery.knob.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/jquery.knob
http://bower.io/search/?q=jquery.knob
others?

[X] has a test file? (jquery.knob/jquery.knob-tests.ts or others)
[ ] pass the Travic-CI test?

@iain8 thank you for your contributions.
please rename jquery.knob/jquery.knob.d.ts to jquery-knob/jquery-knob.d.ts
https://www.npmjs.com/package/jquery-knob

fixed (i think!)

@iain8 one more. please remove jquery.knob/ dir.

oops, now more fixed.

@iain8 thanks mate!


Use ProductUpgradeCode value from the .btdfproj in the Windows Installer Upgrade table vs. a hardcoded GUID

Use ProductUpgradeCode value from the .btdfproj in the Windows Installer Upgrade table vs. (currently) a hardcoded GUID.
This work item was migrated from CodePlex
CodePlex work item ID: '6008'
Assigned to: 'tfabraham'
Vote count: '0'

[UnknownUser@2/10/2010]
Resolved with changeset 36190.

[UnknownUser@2/21/2013]

[UnknownUser@5/16/2013]


Creating SweetAlertDialog app

Description
It is App to display various dialogue boxes using SweetAlertDialog library.
Fixes #122
Type of Change:

Android application added

Checklist:

[x] My PR follows the style guidelines of this project
[x] I have performed a self-review of my own code or materials
[x] My branch is up-to-date with the Upstream development branch
[x] I have added necessary images/screenshots of my changes

Hey @jmanan088 Great work!
Thanks for your contribution into Androapps!


Update the wiki

The first part of the wiki for the version 0.1-beta is done, we will reopen the issue when needed.

Reopening the issue for 0.2-beta


"Symantec"'s "Norton Security Online" App Detects a "SONAR" Behavior in the Updating Process

Prerequisites

Reproduced the problem in Safe Mode:
http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode

Followed all applicable steps in the debugging guide:
http://flight-manual.atom.io/hacking-atom/sections/debugging/

Checked the FAQs on the message board for common solutions:
https://discuss.atom.io/c/faq

Checked that your issue isn't already filed:
https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom

Checked that there is not already an Atom package that provides
the described functionality: https://atom.io/packages

Description
Hi, "Atom.io" Team.
I apologize for the fact that I do not have time right now to check
whether this issue has already been reported or whether it is happening
all the time. Still, it has been bugging me and I feel the urge to report it.

I cannot find the "Atom.io" tile in the "Start Screen". I can find it
in the "Apps & features" list, but I cannot start it. "Cortana" does not find it
either - silly girl, she just finds the "Atomic Clock" app in the "Store".
The only workaround that I have found for this issue has been to pin
its tile to the task bar on the first run. This is not acceptable, but at least
I can start it. I prefer not to go and find the executable binary file.
The same issue described at (1) has also been previously happening
with the "GitHub Desktop" app, that initially was not to be found anywhere.
The workaround for it has been to go to the https://www.github.com/ web page
in the "Microsoft Edge" web browser and then to open a repository from it
by switching to the "GitHub Desktop" app.
However, this issue has been miraculously solved in the meantime
and the "GitHub Desktop" tile is now found by "Cortana", but it does not appear
in the "Start Screen" yet.
Even if I can start the "GitHub Desktop" app right now, still it does not
find the "Atom.io" editor, although I must have reinstalled it at least
two times already by now.
The updating system has a strange behavior. In order to install
the latest update (1.19.7 x64), I had to turn off "Symantec"'s "Norton Security
Online" app. It is reporting a "SONAR" behavior.

Versions
"Microsoft Windows" Version 17603 (Operating System Build 15063.540) x64 © 2017
"GitHub Desktop" App Version 0.8.0
"Atom.io" Editor App Version 1.19.6 x64

Note: I have managed to configure the behavior of the "F4" ("Edit") Key within GHISLER_Christian's "Total Commander" App in order to perform the "C:\Users\Dobre\AppData\Local\Atom\Atom.EXE " "%1" Command, which is a Great Success.

Thanks for taking the time to contribute!  It looks like you're reporting a couple of different things, in the future it would be helpful if you can open one issue per item you're reporting.

I cannot find the "Atom.io" tile in the "Start Screen". I can find it
in the "Apps & features" list, but I cannot start it. "Cortana" does not find it
either - silly girl, she just finds the "Atomic Clock" app in the "Store".
The only workaround that I have found for this issue has been to pin
its tile to the task bar on the first run. This is not acceptable, but at least
I can start it. I prefer not to go and find the executable binary file.

This sounds like a duplicate of https://github.com/atom/atom/issues/14020 so you can subscribe there for updates if you'd like.

The updating system has a strange behavior. In order to install
the latest update (1.19.7 x64), I had to turn off "Symantec"'s "Norton Security
Online" app. It is reporting a "SONAR" behavior.

This has been reported before as well:
https://github.com/atom/atom/issues/13234
As mentioned in that issue, you can report this to Symantec if it's not resolved after your next update.

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.

Hi, rsese.

I guess then that the issue (2) should be reported to the Bug Tracker of the "_GitHub Desktop_" App?

Does the Starting of the "_Atom.io_" Editor App from within the "_GitHub Desktop_" App have nothing to do with the current state of the "_Atom.io_" Editor App?

I wonder.

You have not commented anything yet about that.

The same issue described at (1) has also been previously happening
with the "GitHub Desktop" app, that initially was not to be found anywhere.
The workaround for it has been to go to the https://www.github.com/ web page
in the "Microsoft Edge" web browser and then to open a repository from it
by switching to the "GitHub Desktop" app.
However, this issue has been miraculously solved in the meantime
and the "GitHub Desktop" tile is now found by "Cortana", but it does not appear
in the "Start Screen" yet.
Even if I can start the "GitHub Desktop" app right now, still it does not
find the "Atom.io" editor, although I must have reinstalled it at least
two times already by now.

I guess then that the issue (2) should be reported to the Bug Tracker of the "GitHub Desktop" App?

Sorry, yes - if you're using the new Electron-based GitHub Desktop, you can report bugs in the repository issue tracker using their issue template if there isn't an existing issue already.

Does the Starting of the "Atom.io" Editor App from within the "GitHub Desktop" App have nothing to do with the current state of the "Atom.io" Editor App?

I'm not sure, I've only briefly tried the new GitHub Desktop (I was able to open a repository in Atom via the Repository > Open in Atom menu item just now though) - if the Desktop folks think it's an Atom problem, feel free to let us know.


Issue: Invalid JSON when entering signature as text (Testnet)

I ran in to an issue the other day trying to spend from my test 2-of-3 multisig testnet address I made with Caravan. I'm getting an error ("Invalid JSON.") when I try to submit one of the signed transactions as text.
Below is the flow for what I'm doing, and resulting in the error. I've tried a lot of different combinations and addresses, no luck with any of them. Seems similar to another issue someone was having, but I wanted to provide much more info.
Overall questions:

Am I doing anything wrong, or is this a bug?
Is there a better way to be signing transactions? (other than a hardware wallet)

Step 1: Plugged in redeem script Caravan to spend from.
Redeem Script:
5221021b8f2ed15ce1311860387bc77589c160c41e57379a5a70cd73559b2e19d9d97d21038285e1e8d8512fc52dcc295e673f904fbf7d8084be450b11207b7b8aa26b46262103dfcfda3af5ad4bf015793f57fee87a863696e80274bcd2d177c9b3336a46db5553ae
Step 2: Click "Spend" and input transaction details.
I filled out the destination address, along with fees and amount.
Destination address: mzLqXEdcPnBu4i87D9eNem2JbdDNMxF79o
Amount: 0.00999643
Fee Rate (Sats/byte): 2
Estimated Fees: 0.00000357
Inputs Total: 0.01
Outputs Total: 0.01
Step 3: (I assume) the next step is to sign the resulting unsigned transaction given under "Signature 1" after I select the "Enter as Text" method.
Step 3a: Get the unsigned transaction.
Given unsigned transaction:
0100000001328d532a8eb39706d97a0ccf2df85b2ba6190b33e336b79bb46ce44cc31cdea20000000000ffffffff01763f0f00000000001976a914ce8093a1aef54b6daa6f6ec222bfb3df1107570688ac00000000
Step 3b: Sign the above unsigned transaction
I went over to https://inblockchain.it/ to sign/verify the transaction, script, everything. Make sure to change to "Bitcoin (testnet)" in settings tab in the upper right first.
I signed it with the second address in the 2-of-3:
Address: n4W379boy2thkBz8uiG1ZLMDDxxQE5cDD9
Public Key: 038285e1e8d8512fc52dcc295e673f904fbf7d8084be450b11207b7b8aa26b4626
Private Key: Available if needed. It's testnet, after all.
Signed transaction given is:
0100000001328d532a8eb39706d97a0ccf2df85b2ba6190b33e336b79bb46ce44cc31cdea2000000006a47304402203252ec9d42bdd92dbb68322b81809e18146c3ef4bd531b240d7a02067cefecf802207cea6216d05792dda0f067bba06374cbb0cf7fc461e69ae02134364448f4f9a60121038285e1e8d8512fc52dcc295e673f904fbf7d8084be450b11207b7b8aa26b4626ffffffff01763f0f00000000001976a914ce8093a1aef54b6daa6f6ec222bfb3df1107570688ac00000000
Step 4: Verify everything.
I went to the "Verify" tab to double check the amounts and destination address look good, and that it's marked as signed. Confirmed my signing address's public key was one of the 2-of-3 quorum by verifying the redeem script as well. Everything checked out.
Step 5: Go back to Caravan, add this signed transaction as the "Signature" for the "Enter as text" method under "Signature 1".
The issue: The box border turns red and returns an "Invalid JSON." error. Clicking "Add Signature" does nothing.
Troubleshooting steps taken so far:

I went back and repeated the steps from 3a to 5 with the other two addresses in the 2-of-3 quorum, thinking it may have something to do with the order in which the signed transaction text was submitted. I got the issue when signing with the other keys as well.
I checked for leading/trailing spaces. Looks clean.
I verified the signed transaction using the "Verify" tool on the site above and didn't notice any discrepancies, but I'm not the most technical.
I verified the redeem script was indeed referencing the addresses/keys I was signing with.

The form of the signature Caravan expects (as produced by the unchained-wallets library) is a JSON array [] of string signatures, one per UTXO being spent.
If you are only spending 1 UTXO, this can be confusing.  If the signature for that single UTXO is 345..., just pasting it in "naked" will fail; it needs to be inside a 1-element JSON array: ["345..."].  Could you try this and see if it works and respond back here?  Thanks!
(You're not the first person to be confused by this behavior.  We need to improve the error messaging.)

Thanks for the response Dhruv! Big fan of your podcast appearances over the fall/winter.
So, if I'm understanding you correctly, all I need to do is put ["..."] around the signed transaction and it'll work. You are correct that there is only one input.
I went back and updated to the signature to:
["0100000001328d532a8eb39706d97a0ccf2df85b2ba6190b33e336b79bb46ce44cc31cdea2000000006b483045022100a89bea8b9beca187d12a226582d30d17c216656017a6411d8f8acd8d57de7c8a02207ab466f9ae3c5981891bc2a9e6419350869a2d990f62c7d0c720d2d9eff82f270121038285e1e8d8512fc52dcc295e673f904fbf7d8084be450b11207b7b8aa26b4626ffffffff01db400f00000000001976a914ce8093a1aef54b6daa6f6ec222bfb3df1107570688ac00000000"]
That got rid of the JSON error.
However, now there's a new error: "Signature for input 1 is invalid." and it won't let me continue on to add Signature 2.
That one seems pretty self explanatory, but I'm confused as I signed it with private keys from the correct private key associated with an address in that 2-of-3 quorum. I've signed a standard address-to-address transaction with it and have had no issues.
Is there a different way I should be going about signing the transaction, or am I missing something else? Is there some special way I should be signing a transaction to account for different variables?

You need to paste just the signature not the signed transaction:
["3045022100a89bea8b9beca187d12a226582d30d17c216656017a6411d8f8acd8d57de7c8a02207ab466f9ae3c5981891bc2a9e6419350869a2d990f62c7d0c720d2d9eff82f2701"]

Thanks for the response. I'm still getting the same error "Signature for input 1 is invalid."
Could I be signing it wrong or something? Not sure if there's a recommended way I should be doing the signings.

What wallet are you signing with?
I decoded here, you can also do it with bitcoin-cli using the decoderawtransaction command.  The first part of the asm is the signature(strip off the [ALL]), the second part is the pubkey.

I'm signing it with https://inblockchain.it/ (a fork of coinb.in that supports testnet), under the "Sign" tab

I tried that link, I can not get a valid signature, where with the same key and tx I can sign in bitcoin core just fine.  It does not seem to ask for the needed information to be able to sign(scriptPubKey, redeemScript, amount).  Probably limited in signing capabilities.
I would not put too much faith into that site and I certainly would not be putting in private key info into it with real funds.

Ok, thank you for the response. I had thought it was something to do with not asking for all the correct inputs.
Looks like I need to dig in and get comfortable using bitcoin core for signing transactions.

Had the same issue and I am unable to get signatures for 2of2 from electrum in my case. Will test things with bitcoin core wallet tomorrow.

I'm also having issues with this.
Enthusiastic newbie, I set up a 2 of 3 on Testnet using a (borrowed) Ledger and samourai and wasabi wallets all set up on testnet. (I don't have any of the supported hardware wallets). Tried to send the coins back to the faucet (Caravan didn't like that address...sorry, I didn't document the error message), so I created a testnet Electrum wallet and used a receive address from that, which took care of that error.
I signed with the Ledger.
But I can't sign with the Samourai or Wasabi :-(
Are there any instructions for entering signature as text? Happy to do further reading on this, if you could point me in the right direction.
I feel bad about not returning the testnet coins.
Thank you.

If it's just one signature, make sure you enter it as ["signature"] e.g. ["3045..."] -- if you're signing multiple inputs, use comma separated e.g. ["3045...", "3044...", "3045..."]

Thank you @humanumbrella.  I used the signature from the redeem script -- is that incorrect? Does additional information need to be included/appended?
For two of 3 Wallet I signed Ledger first signature was pulled in from ledger device and accepted.
For 2nd signature I tried Samourai Signature (from redeem script)-- ["03be574cb6e2dcf721d89473e419f6815e0eac05c6cf76c17a8d92659ffd427e30 "]
and
Wasabi signature (from redeem script) -- ["03e7ab027704843dc5c6091a040e22f9bf82c42d6f1506905deed11d7de1243c9e"]
This is the info I saved when I set up the wallet:
Address: 2N5aqKzgGMhHAy8i2MynWFHAYxHUDdBQQT8
Script - a9148756e9b3d77411295e9aae16761a204563dc639487
Redeem Script - 522102dfc19b32ea24b3797fc8f52c42c6aed9989e40b2f442355c45684950d9d2f48d2103be574cb6e2dcf721d89473e419f6815e0eac05c6cf76c17a8d92659ffd427e302103e7ab027704843dc5c6091a040e22f9bf82c42d6f1506905deed11d7de1243c9e53ae
OP_HASH160 8756e9b3d77411295e9aae16761a204563dc6394 OP_EQUAL
OP_2
02dfc19b32ea24b3797fc8f52c42c6aed9989e40b2f442355c45684950d9d2f48d 03be574cb6e2dcf721d89473e419f6815e0eac05c6cf76c17a8d92659ffd427e30 03e7ab027704843dc5c6091a040e22f9bf82c42d6f1506905deed11d7de1243c9e
OP_3 OP_CHECKMULTISIG

When you're quoting signatures in your reply to me -- those are not signatures, they're the public keys. It looks like those public keys are indeed different members of your redeemscript (notice they're there between OP_2 and OP_3) but they're not signatures so obviously would fail signature validation.
I'm not sure what is actually being output when you sign from those other wallets, but you will need something that looks like ["304..."] (a signature) instead of ["03..."] or ["02..."] (a public key).
Your Ledger probably signed for 02df... and Samourai seems to be signing for 03be5... and Wasabi signing for 03e7... that's good - but you need to input the actual signatures.
Cheers


When using nuxt 3 & i18n outputs ошибку @intlify_shared.js

Discussed in https://github.com/nuxt-modules/i18n/discussions/2133

Originally posted by Roman-00 June  3, 2023
When using next 3 & i18n, it throws an error related to the cache and the @intlify package
nuxt version - 3.5.2
nuxt/i18n version - "@nuxtjs/i18n": "8.0.0-beta.12",
Screenshot of the error

Minimal build to view the error: https://github.com/Roman-00/iteca.events/tree/PR-3

Thank you for your reporting!
Unfortunately, We could not reproduce your repo.
We need more detail reproduction step.
Could you give us it please? 🙏

Hello!
repos - https://github.com/Roman-00/test-nuxt-i18n
Installed i18n in the project, then specified in the modules in nuxt.config
When starting the dev environment, this error falls into the developer console
package manager - using pnpm

I have same this issue. I add @intlify/nuxt3 to modules in nuxt.config.ts to resolved this issue.

@Roman-00
Thank you for your reproduction!
However, I could not reproduce this issue 😞
We need more infomation.
Could you try to run npx nuxi info please?

@Roman-00 Thank you for your reproduction! However, I could not reproduce this issue 😞 We need more infomation. Could you try to run npx nuxi info please?

what exactly didn't work out? When installing dependencies via pnpm i, then launch the pnp dev project, go to the browser, open the developer console and see the error in the console tab

@Roman-00 Thank you for your reproduction! However, I could not reproduce this issue 😞 We need more infomation. Could you try to run npx nuxi info please?

After executing npx nuxi info
Nuxi 3.5.2                                                                                4:38:35 PM
4:38:35 PM
RootDir: /Users/vassin/PhpstormProjects/test-nuxt-i18n                                                                                4:38:35 PM
Nuxt project info: (copied to clipboard)                                                                                4:38:36 PM

Operating System: Darwin
Node Version:     v16.20.0
Nuxt Version:     3.5.2
Nitro Version:    2.4.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      modules
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

👉 Report an issue: https://github.com/nuxt/nuxt/issues/new                                                                                4:38:36 PM
👉 Suggest an improvement: https://github.com/nuxt/nuxt/discussions/new
👉 Read documentation: https://nuxt.com

@intlify/nuxt3

?????

Does not work

Some error ...
Chrome:
caught SyntaxError: The requested module '/_nuxt/node_modules/.cache/vite/client/deps/@intlify_shared.js?v=8d5a169e' does not provide an export named 'assign' (at message-compiler.mjs?v=5cc3b937:6:18)
FireFox:
Uncaught SyntaxError: ambiguous indirect export: format

Could you try to add strict-peer-dependencies=false to .npmrc and re-install with pnpm ?

The error exists now, and has not been resolved

Some error ...
Server console doesnt have errors.
Chrome console: caught SyntaxError: The requested module '/_nuxt/node_modules/.cache/vite/client/deps/@intlify_shared.js?v=8d5a169e' does not provide an export named 'assign' (at message-compiler.mjs?v=5cc3b937:6:18)
FireFox console: Uncaught SyntaxError: ambiguous indirect export: format

Strangely, everything worked fine and at one moment project fail. Cache clearing, deleting node_modules doesnt help.

Please tell me if the problem was solved?

Could you try to add strict-peer-dependencies=false to .npmrc and re-install with pnpm ?

Here is a reproduction of the error on the sandbox - https://stackblitz.com/edit/nuxt-starter-cdfvem?file=README.md

Thanks.
I've noticed that there isn't .npmrc in your reproduction.
I've forked your reproduction, and add .npmrc on it.
so, it' works.
https://stackblitz.com/edit/nuxt-starter-8eqht8?file=.npmrc

Thanks. I've noticed that there isn't .npmrc in your reproduction. I've forked your reproduction, and add .npmrc on it. so, it' works. https://stackblitz.com/edit/nuxt-starter-8eqht8?file=.npmrc

I have this file, with shamefully-hoist=true strict-peer-dependencies=false but error exist =(

mrFANRA

I have this file, with shamefully-hoist=true strict-peer-dependencies=false but error exist =(

Here is a reproduction of the error on the

Here is a reproduction of the error on the repos - https://github.com/Roman-00/iteca
Error playback video
https://github.com/nuxt-modules/i18n/assets/58338625/fd5e6702-38ba-4403-ba62-271731c17e2a

Thanks. I've noticed that there isn't .npmrc in your reproduction. I've forked your reproduction, and add .npmrc on it. so, it' works. https://stackblitz.com/edit/nuxt-starter-8eqht8?file=.npmrc

The error remained, the above provided a video + repository in which this error is reproduced. In stackblitz.com there is no way to open the developer console to identify this problem

Thanks. I've noticed that there isn't .npmrc in your reproduction. I've forked your reproduction, and add .npmrc on it. so, it' works. https://stackblitz.com/edit/nuxt-starter-8eqht8?file=.npmrc

How will the problem be solved? Since now the problem has not disappeared

Ah, That issue had already opened
https://github.com/nuxt-modules/i18n/issues/2098
I thiink that issue is helpful.

Ah, That issue had already opened #2098
I thiink that issue is helpful.

Nothing from this branch helps

Ah, That issue had already opened #2098
I thiink that issue is helpful.

Nothing from this branch helps

I faced the same issue. Before installing @nuxtjs/i18n@next, I accidentally installed @nuxtjs/i18n. I believe this was the root of the problem in my case. So, I did a clean npm install, and it started working just fine.


Reporting coverage fails for out-of-source builds

I have a build setup on Travis which does an out-of-source build like this (roughly):
export BUILD=$(mktemp -d /tmp/build-XXXXX)
(cd ${BUILD} && cmake ${TRAVIS_BUILD_DIR} -GNinja -DCOVERALLS=ON -DCOVERALLS_UPLOAD=ON)
cmake --build ${BUILD}
cmake --build ${BUILD} --target test
cmake --build ${BUILD} --target coveralls

Since #13 got merged (I think), it fails to report coverage at all (previously I guess it just didn't fill in any Git information); see this build job:
$ if [[ "${COVERAGE}" == "ON" ]]; then (cmake --build ${BUILD} --target coveralls); fi

[1/2] Generating coveralls output...
FAILED: cd /tmp/build-bsp2Q && /usr/bin/cmake -DPROJECT_BINARY_DIR="/tmp/build-bsp2Q" -P /home/travis/build/urdh/libeu4/deps/JoakimSoderberg/coveralls-cmake/cmake/CoverallsClear.cmake && /usr/bin/ctest --output-on-failure && /usr/bin/cmake -DCOVERAGE_SRCS="*/home/travis/build/urdh/libeu4/libeu4/file-format/textual.cc*/home/travis/build/urdh/libeu4/libeu4/file-format/binary.cc*/home/travis/build/urdh/libeu4/libeu4/file-format/detail/utilities.hh*/home/travis/build/urdh/libeu4/libeu4/file-format/detail/iterator.hh*/home/travis/build/urdh/libeu4/libeu4/file-format/textual.hh*/home/travis/build/urdh/libeu4/libeu4/file-format/types.hh*/home/travis/build/urdh/libeu4/libeu4/file-format/binary.hh*/home/travis/build/urdh/libeu4/libeu4/detail/identifier.impl.hh*/home/travis/build/urdh/libeu4/libeu4/detail/phantom.hh*/home/travis/build/urdh/libeu4/libeu4/detail/identifier.hh*/home/travis/build/urdh/libeu4/libeu4/detail/date.impl.hh*/home/travis/build/urdh/libeu4/libeu4/detail/date.hh*/home/travis/build/urdh/libeu4/libeu4/detail/phantom.impl.hh*/home/travis/build/urdh/libeu4/libeu4/types.hh*/home/travis/build/urdh/libeu4/libeu4/grammar/detail/auxiliary.hh*/home/travis/build/urdh/libeu4/libeu4/grammar/detail/binary.impl.hh*/home/travis/build/urdh/libeu4/libeu4/grammar/detail/text.impl.hh*/home/travis/build/urdh/libeu4/libeu4/grammar/detail/binary.hh*/home/travis/build/urdh/libeu4/libeu4/grammar/detail/auxiliary.impl.hh*/home/travis/build/urdh/libeu4/libeu4/grammar/detail/text.hh" -DCOVERALLS_OUTPUT_FILE="/tmp/build-bsp2Q/coveralls.json" -DCOV_PATH="/tmp/build-bsp2Q" -DPROJECT_ROOT="/home/travis/build/urdh/libeu4" -P /home/travis/build/urdh/libeu4/deps/JoakimSoderberg/coveralls-cmake/cmake/CoverallsGenerateGcov.cmake
Test project /tmp/build-bsp2Q
    Start 1: file-format_textual
1/6 Test #1: file-format_textual ..............   Passed    0.03 sec
    Start 2: file-format_binary
2/6 Test #2: file-format_binary ...............   Passed    0.03 sec
    Start 3: detail_date
3/6 Test #3: detail_date ......................   Passed    0.03 sec
    Start 4: detail_identifier
4/6 Test #4: detail_identifier ................   Passed    0.02 sec
    Start 5: grammar_binary
5/6 Test #5: grammar_binary ...................   Passed    0.03 sec
    Start 6: grammar_text
6/6 Test #6: grammar_text .....................   Passed    0.03 sec

100% tests passed, 0 tests failed out of 6

Total Test time (real) =   0.17 sec
-- Found Git: /usr/bin/git (found version "1.9.1")
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
CMake Error at /home/travis/build/urdh/libeu4/deps/JoakimSoderberg/coveralls-cmake/cmake/CoverallsGenerateGcov.cmake:134 (string):
  string sub-command REPLACE requires at least four arguments.

ninja: build stopped: subcommand failed.

I assume the immediate issue is that #13 tries to change a variable which might not be set, but the underlying issue seems to be that the git code doesn't work for out-of-source builds (maybe?).

Hello,
Well yes, seems like that change is trying to replace an empty string. Possibly that string is empty because git seems to be run in the wrong directory. even though the CWD of the git comand is explicitly set to the source dir:
https://github.com/JoakimSoderberg/coveralls-cmake/blob/master/cmake/CoverallsGenerateGcov.cmake#L113
True, I have not really tried with building out side of the repository like you're doing. However, I do not   agree on your definition of an out-of-source build. The usual way of doing this in a  cmake project is to simply create a build directory inside the git repository. This still keeps the build files from polluting the source directory and can be considered out-of-source.
But I guess it is a taste thing. To me your way is just unnecessarily complex compared to:
# Travis starts out in the dir where it cloned the source
mkdir build           # This is still "out-of-source"
cd build
cmake ..
cmake --build .

Changing it to this should solve your problem. But if that is not to your liking, maybe it helps if you do an if (GIT_COMMIT_MESSAGE) around that replace. Could you try?

I did actually have it build inside the Git repository previously, but that also doesn't work well:
Filter out unwanted GCOV files:
===============================
CMake Error at /home/travis/build/urdh/libeu4/deps/JoakimSoderberg/coveralls-cmake/cmake/CoverallsGenerateGcov.cmake:252 (file):
  file RELATIVE_PATH must be passed a full path to the file:
  ^/deps/jimporter/mettle/include/mettle/suite/detail/all_suites.hpp

This is, as far as I can work out, because CMake decides to do an in-source build (e.g. uses relative paths) when I do this.
Just wrapping the replace works fine though (see also #16).

Ah ok, well yea CMake is quite picky with paths. And prefers absolute paths for everything. Not sure exactly what is happening there. But nice that it works now


Update README.md to specify a Caffe2 source build is required

Clarify that a Caffe2 source build is required per https://github.com/facebookresearch/R2Plus1D/issues/44

cc @dutran - thanks.

Thanks @orionr !


[feature request] speed up zfs send/recv

Currently lxc copy is terribly slow.
mbuffer can speed up send/recv a lot.
Copying from a script:
c_mbuffer_send="mbuffer -q -v 0 -s 128k -m 1G"
c_mbuffer_recv="mbuffer -s 128k -m 1G"

zfs send [...] | $c_mbuffer_send" | $c_mbuffer_recv | zfs recv [...]

It's probably a bit more difficult than this given that we then have to use the websocket framing to transport things over the remote server, but a similar trick should be possible in the Go code I'd expect.

On Wed, Apr 13, 2016 at 01:01:49AM -0700, Tamas Papp wrote:

Currently lxc copy is terribly slow.
mbuffer can speed up send/recv a lot.
Copying from a script:
c_mbuffer_send="mbuffer -q -v 0 -s 128k -m 1G"
c_mbuffer_recv="mbuffer -s 128k -m 1G"

zfs send [...] | $c_mbuffer_send" | $c_mbuffer_recv | zfs recv [...]

We could probably do this in LXD itself (i.e. buffer zfs output
instead of just copying it directly to websockets in the chunks it
comes in), I'll have a look.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/lxc/lxd/issues/1890

So I started fiddling with this today, and came up with this:
https://github.com/tych0/lxd/commit/5625a67cc956250574ba24492b6a66523361a398
Unfortunately it seems like zfs doesn't like whatever is happening, so it seems something might be broken at the websocket framing level.

An update on this: I profiled what happens on the sender and receiver CPU side, and it looks like basically all of the CPU time is spent in golang's crypto libraries encrypting and decrypting the websocket frames. I don't think there's really anything more we can do about that (golang is apparently smart enough to take advantage of hardware coprocessors when they're present) besides potentially adding some sort of "insecure transfer" mode.
In any case, we are doing smarter buffering now which was the focus of this issue, so I'm going to close it. Thanks!