Change dpi of PDFs

It seems that we can't change dpi of generated PDFs, will it be a feature ?
For some case, it would be useful.

I figured out a small workaround for this. In my case I want to generate PDFs in the DIN A4 format, which corresponds to a pixel size of 794 * 1122. I'm setting this additionally to the normal pdf format (which is still set to format: A4) in page.setViewport() which allows me to set a deviceScaleFactor.
const page = await browser.newPage();
await page.setViewport({width: 794, height: 1122, deviceScaleFactor: 2});
await page.goto(`https://www.example.org`, {waitUntil: 'networkidle'});
await page.pdf({
    path: filePath,
    format: 'A4',
    printBackground: true
});

https://bugs.chromium.org/p/chromium/issues/detail?id=741049 covers the Chromium issue, comment there if you'd like support!

I figured out a small workaround for this. In my case I want to generate PDFs in the DIN A4 format, which corresponds to a pixel size of 794 * 1122. I'm setting this additionally to the normal pdf format (which is still set to format: A4) in page.setViewport() which allows me to set a deviceScaleFactor.
const page = await browser.newPage();
await page.setViewport({width: 794, height: 1122, deviceScaleFactor: 2});
await page.goto(`https://www.example.org`, {waitUntil: 'networkidle'});
await page.pdf({
    path: filePath,
    format: 'A4',
    printBackground: true
});

brilliant. seems to work for me!

I figured out a small workaround for this. In my case I want to generate PDFs in the DIN A4 format, which corresponds to a pixel size of 794 * 1122. I'm setting this additionally to the normal pdf format (which is still set to format: A4) in page.setViewport() which allows me to set a deviceScaleFactor.
const page = await browser.newPage();
await page.setViewport({width: 794, height: 1122, deviceScaleFactor: 2});
await page.goto(`https://www.example.org`, {waitUntil: 'networkidle'});
await page.pdf({
    path: filePath,
    format: 'A4',
    printBackground: true
});

Do I understand it right, that if I want to print some html in high resolution using this method it should be of flexible (not limited size) ?

"deviceScaleFactor" has no effect at all


Update related to ADFS #3151 as important note.

Added important note #3151
Copy review @nenonix @JohanFreelancer9 is required on this one guys.
Thank you in advance!

@LanaChin PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

Pinged PM.

Thanks j0rt3g4. The folder structure of this topic has changed so I opened a separate PR for this change. The topic is now updated. Closing this without merging.


No NPM install script

Following along with the documentation at: http://wejs.org/docs/we/getstarted
Thing being, the package.json from the generated app doesn't include an install script. I've been unable to hack a way to install this yet.

@mnzt Thanks for your report, i will update the wejs yeoman generator.
For now you can test the we.js conference/event project in https://github.com/wejs/we-project-conference how are in development but have the install script

@albertosouza any update on this?

@mnzt i updated the generator, CLI and the installation doc now
update your we.js cli and generator with:
obs: may need sudo
npm install we@latest -g
npm install generator-wejs@latest -g

And follow http://wejs.org/docs/we/getstarted updated instructions.

I recorded a video with how to create a project to type blog with we.js, is en but should help in this issue.
Link: https://www.youtube.com/watch?v=TtLcAT1lCMU
I will close this issue because we already made several improvements in installation and creation of a basic project


"Fake shares" attack resistant and Double nonce

Just came across the issue that most pools had with the Fake shares attack, how do this pool do against these attacks ?

python script nonce_fast.py verify all shares submitted since last processed block, in case share is fake, then will be delated and won't be used in calculations.
Read more about ethpow
ethpow.check_pow(block_number, header_hash, mixhash, nonce, difficulty)

Impressive feedback here!
Den tirsdag 9. februar 2016 skrev karek314<EMAIL_ADDRESS>følgende:

python script nonce_fast.py verify all shares submitted since last
processed block, in case share is fake, then will be delated and won't be
used in calculations.
Read more about ethpow
ethpow.check_pow(block_number, header_hash, mixhash, nonce, difficulty)
—
Reply to this email directly or view it on GitHub
https://github.com/karek314/Ethereumpool.co---Ethereum-Mining-Pool-Full-Source/issues/5#issuecomment-182103613
.

--
Jon Ramvi408 70 027


fixes #5989: respect Uri locality in UrlFormLifter

This looks like an old bug that applies to 0.22.  I'm going to look at applying your changes there and then merge it forward to 0.23 and 1.0 series, but this looks good.

Backported to series/0.22 in #5994.


How can I get current request id?

Hi,
Although it's very helpful to have request id included in logs, I would also like to return this Id to the user on error screen. I know I can get it directly from current HttpContext, but it might not be there yet and I have to know internals of HttpRequestIdEnricher.
Thanks

Hi @wziska,
Instead of HttpRequestIdEnricher, you can use HttpRequestTraceIdEnricher, which uses an id pre-allocated by ASP.NET (may even be IIS, actually):
var serviceProvider = (IServiceProvider)HttpContext.Current;
var workerRequest = (HttpWorkerRequest)serviceProvider.GetService(typeof(HttpWorkerRequest));
var requestId = workerRequest.RequestTraceIdentifier;

HTH!

Unfortunately I'm getting Guid.Empty from RequestTraceIdentifier. Tried to add  element to web.config, but it didn't help.

Thanks for the investigation. Build 1.1 of the package (CI should publish soon) now has HttpRequestIdEnricher.TryGetCurrentHttpRequestId(). Cheers!

Thanks for quick feedback. I initially though about extension method for HttpContextExtensions , but static method in HttpRequestIdEnricher works too.
Regards.


Provide predicted aligned error (PAE) from the ESM Metagenomic Atlas prediction API

Currently predictions run with the ESM Metagenomic Atlas API, for example
curl -X POST --data "GENGEIPLEIRATTGAEVDTRAVTAVEMTEGTLGIFRLPEEDYTALENFRYNRVAGENWKPASTVIYVGGTYARLCAYAPYNSVEFKNSSLKTEAGLTMQTYAAEKDMRFAVSGGDEVWKKTPTANFELKRAYARLVLSVVRDATYPNTCKITKAKIEAFTGNIITANTVDISTGTEGSGTQTPQYIHTVTTGLKDGFAIGLPQQTFSGGVVLTLTVDGMEYSVTIPANKLSTFVRGTKYIVSLAVKGGKLTLMSDKILIDKDWAEVQTGTGGSGDDYDTSFN" https://api.esmatlas.com/foldSequence/v1/pdb/
described here
https://esmatlas.com/about#api

does not give any way to get the predicted aligned error (PAE).  The PAE data is a key aspect of the prediction indicating if the protein domains are packed correctly.  It would be very useful to provide both the PDB predicted structure and JSON PAE file.  One way to achieve this would be to create a new REST API that returns a zip file containing the PDB and JSON files.

The PAE data can be viewed and interactively analyzed together with the protein structure in the ChimeraX visualization program that I develop.  I've added an ESMFold prediction capability to ChimeraX using the Meta server and would like it to be able to get the PAE data.
 https://www.rbvi.ucsf.edu/chimerax/data/esmfold-nov2022/esmfold.html

Hi Tom, thanks for your work on ChimeraX, we were impressed by the beautiful visualizations!
I'll add this feature request to our internal tracker and prioritize accordingly.

Thanks!


Add a 'auto' imagej like option

IT would be interesting to have (yet) another option for defining the colormap min/max values.
Closer to what imageJ is doing:

note: Suggested by Elodie

Can you provide a link to the relevant doc, the screen shot is not readable

Yes sorry. Here is the link: https://imagej.net/ij/docs/guide/146-28.html


FP_MinusInfty > 0 returns True

Comparisons of core.special_values such as signed infinities return erroneous results
when compared with numerical values:
from metalibm_core.core.special_values import FP_MinusInfty
from metalibm_core.core.ml_formats import ML_Binary32

print(FP_MinusInfty(ML_Binary32) > 0)

Partially fixed in 986e37f0dd117892de4cdd8958b0a2826b65817e
Now comparison raise NotImplementedError

Fixed by commit 5e7efd6 integrated in master branch
unit test added in commit 56919f6 (also integrated in master)


Internships Section: Two Links Not Working, Please Update

The following two links are not working:

Letsintern : Get a smart and challenging internship for you from the LetsIntern.
PerfectIntern: Get help finding a paid internship, resume prep, interview prep, and more!

It seems the URLs for these links may have changed. Upon checking, I found the following updated addresses:

[Letsintern] (https://letsintern.in/)
[PerfectIntern] (https://www.theperfectintern.co.ke/)

Hi, I came across this issue while exploring the repository, and I believe I can resolve it. I would love the opportunity to work on this with your guidance. Could you kindly assign this issue to me?
Thank you so much!

hey @csengineer1990  the address you provided for the perfectIntern is also not working, may be there is some updates in this address

Yes, that could be the case. The last time I checked, it was working fine, but it's possible they’ve made some updates since then.

Hi, may I make changes to the internship section, or would you prefer any specific suggestions? Please assign this issue to me with the Hacktoberfest label if possible.


rio: 0.0.19 -> 0.0.27

Refs: #260413.
Fixes: #257334.
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Would you mind adding to following to fix the darwin build?
diff --git i/pkgs/applications/terminal-emulators/rio/default.nix w/pkgs/applications/terminal-emulators/rio/default.nix
index 314e127b6a44..6ebd8f6f521d 100644
--- i/pkgs/applications/terminal-emulators/rio/default.nix
+++ w/pkgs/applications/terminal-emulators/rio/default.nix
@@ -30,7 +30,7 @@
 let
   rlinkLibs = if stdenv.isDarwin then [
     darwin.libobjc
-    darwin.apple_sdk.frameworks.AppKit
+    darwin.apple_sdk_11_0.frameworks.AppKit
   ] else [
     (lib.getLib gcc-unwrapped)
     fontconfig
@@ -62,8 +62,8 @@ rustPlatform.buildRustPackage rec {

   nativeBuildInputs = [
     ncurses
-    cmake
   ] ++ lib.optionals stdenv.isLinux [
+    cmake
     pkg-config
     autoPatchelfHook
   ];


refactor: expose isOpen to use in arrow-icon slot

Description
When using the custom arrow-icon slot from SidebarDropdownItem component, it is not possible to use the isOpen ref value used in the official component in the Flowbite-Vue library.
Suggested Fix

expose the isOpen ref value with defineExpose
add documentation for this

@cogor what is the current status of my MR review? All the pipelines are passing, so I am just awaiting your review and merge 🙂

@cogor what is the current status of my MR review? All the pipelines are passing, so I am just awaiting your review and merge 🙂


Strings are stored as BLOBs using ruby-1.9.3-preview1

@gucki reported a strange behavior of factory_rails using 1.9.3-preview1 (thoughtbot/factory_girl#166), which was (mis)identified as a ruby error.
However, the problem is more general. It happens with AR timestamp fields too, as my tests showed (see my comment there for details).
While I could prepare a failing test for this issue, I went the other route: I bisected ruby from v1_9_2_290 to v1_9_3_preview1, and the first error occured at ruby/ruby@a8f5a06a.
I believe while the change itself was in ruby, sqlite3-ruby should be fixed: my tests using 1.9.3-preview1 + mysql2 were succeeded.

This issue's afterlife is just amazing :)


maps-sdk-for-ios-samples/GooglePlaces/Pods/Target Support Files/Pods-GooglePlacesDemos/Pods-GooglePlacesDemos.default.xcconfig: unable to open file (in target "GooglePlacesDemos" in project "GooglePlacesDemos")

When opening .xcodeproj and try to build, the captioned message appears

problem resolved after doing POD install for GooglePlaces


bump github action versions

This modifies the release workflow, but that is skipped in de CI. I would really like to see a succesfull release with these changes.

how would you like me to test that before merging the PR?

This modifies the release workflow, but that is skipped in de CI. I would really like to see a succesfull release with these changes.

how would you like me to test that before merging the PR?

Create a tag with that results in a package with a name that pip doesn't automatically installs

tested and it works


[Snyk] Upgrade org.apache.tomcat.embed:tomcat-embed-jasper from 10.1.8 to 10.1.9

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade org.apache.tomcat.embed:tomcat-embed-jasper from 10.1.8 to 10.1.9.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released a month ago, on 2023-05-09.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

deleted tomcat10


feat: add issue.scope option

The plugin uses TypeScript settings regarding files that are included in the compilation. It can be a different set of files comparing to the webpack compilation. To not display issues that are related to files outside webpack compilation, an issue.scope option has been added.
BREAKING CHANGE: 🧨 Issues outside webpack compilation will not be reported by default. See
issue.scope option.

:tada: This PR is included in version 5.0.0-beta.6 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


bug: invalid JSON example

Bug Report
SNIPs version:
SNIP-12
https://github.com/starknet-io/SNIPs/blob/5d5a42c654c27b377d8b7f90b453065fd19ec2eb/SNIPS/snip-12.md
Current behavior:
The snip contains a JSON example of a TypedData but the JSON is invalid.
Ref: https://github.com/starknet-io/SNIPs/blob/main/SNIPS/snip-12.md#json-example

Expected behavior:
A valid JSON, without errors

I opened a PR to fix that:
https://github.com/starknet-io/SNIPs/pull/119


Simplifying Definition etc

DefinitionName renamed to TypeName
Definition interface renamed to TypeDefiner (golangy!) and simplified, it has the TypeName of the type it is defining and the Type that it will be bound to. AsDeclarations is on this type.

Add SimpleTypeDefiner for when we want to give names to simple types that aren't structs/enums. This is functionality we didn't have before (see next point as well).

CreateRelatedDefinitions is renamed & massaged to make CreateDefinitions which will create both the primary definition and any associated definitions.
This latter thing fixes a few issues:

previously we only named structs & enums. now a refHandler can give a name to anything like a map, etc (this is actually used a lot).
removes the special-casing of structs that we had in refHandler
removes the mutation of canonicalName in EnumType
this is also necessary to enforce that internal structs are always named, which is needed to actually run controller-gen properly

Outstanding TODOs:

Make the special-casing in struct's CreateDefinitions generic.

Pull Request Test Coverage Report for Build 112090581

-166 of 179 (7.26%)  changed or added relevant lines in 17 files are covered.
12 unchanged lines in 7 files lost coverage.
Overall coverage increased (+1.03%) to 30.019%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

hack/generator/pkg/astmodel/array_type.go
0
3
0.0%

hack/generator/pkg/astmodel/optional_type.go
0
3
0.0%

hack/generator/pkg/astmodel/primitive_type.go
0
3
0.0%

hack/generator/pkg/astmodel/type_definer.go
0
3
0.0%

hack/generator/pkg/astmodel/map_type.go
0
4
0.0%

hack/generator/pkg/codegen/code_generator.go
0
4
0.0%

hack/generator/pkg/astmodel/struct_definition.go
1
6
16.67%

hack/generator/pkg/astmodel/enum_definition.go
0
8
0.0%

hack/generator/pkg/astmodel/enum_type.go
0
10
0.0%

hack/generator/pkg/astmodel/package_definition.go
0
18
0.0%

Files with Coverage Reduction
New Missed Lines
%

hack/generator/pkg/astmodel/array_type.go
1
44.0%

hack/generator/pkg/astmodel/optional_type.go
1
0%

hack/generator/pkg/astmodel/primitive_type.go
1
44.44%

hack/generator/pkg/astmodel/struct_definition.go
1
43.9%

hack/generator/pkg/astmodel/enum_definition.go
2
0%

hack/generator/pkg/astmodel/enum_type.go
2
0%

hack/generator/pkg/jsonast/jsonast.go
4
39.95%

Totals

Change from base Build 112084139:
1.03%

Covered Lines:
468

Relevant Lines:
1559

💛  - Coveralls

Broadly makes sense to me, though I'm unsure how it works now. #shrug

Two main loci of changes:

Enabling controller-gen by naming structs nested inside other types:

Where we used to have some special code in jsonast naming the StructType and then calling CreateRelatedDefinitions we now have a function CreateDefinitions which names the type and returns any extra definitions like what CreateRelatedDefinitions did (so CreateDefinitions is basically naming with CreateRelatedDefinitions attached). This means refHandler can now grant a name to any type.
In addition to that, CreateDefinitions names everything inside that type that has to be named for controller-gen to work (i.e. enums/structs), including types that are nested inside other types like maps/arrays/…;  CreateInternalDefinitions is the function that does this work (CreateRelatedDefinitions didn't do this before).

Clarification changes:

TypeDefiner takes over what Definition was doing before but it doesn’t participate in any of the Type stuff that was mixed into the Definition interface; I think this makes it cleaner. DefinitionName is a simple renaming of TypeName without much else changed. StructReference was removed entirely and the isResource bit moved into StructDefinition.


access to previous success cron from CronJob instance

I needed to access to previous success cron from my CronJob instance, to get previous success start and end date. This issue shall provide such access:
class MyCronJob(CronJobBase):
    RUN_EVERY_MINS = 15 # every 15 minutes

    schedule = Schedule(run_every_mins=RUN_EVERY_MINS)
    code = 'my_app.my_cron_job'    # a unique code

    def do(self):
        prev_success_cron = self.get_prev_success_cron()
        if prev_success_cron is not None:
            # access to all fields of models.CronJobLog here

Hello.
Thank you for this great app!
When this feature will be available on PyPi? Current version (0.4.0) does not have get_prev_success_cron and I really need it:)

Hi 0.4.1 should be out by the end of the week.

Pardon me, but this fix seems to be broken:)
get_prev_success_cron() is always None. Why?
Lets look where into ``django_cron`:
self.msg = self.cron_job.do() # My code runs here, ``get_prev_success_cron`` is None
self.make_log(self.msg, success=True)
self.cron_job.set_prev_success_cron(self.previously_ran_successful_cron) # It is set here.

How can anyone access get_prev_success_cron in do ?)


ScRNA-seq and bulk RNA-seq data format required to perform imputation

Hello, Bis Author! This algorithm looks good. What format of scRNA-seq data and bulk sequencing data do I need to input when performing imputation? Is it counts data, or needs to input cpm data similar to "data" layer of Seurat objects for scRNA-seq as well as tpm data for bulk sequencing?
Thanks！

Thank you very much for your interest in our work! For both scRNA-seq and bulk data, count data should be used as input. Afterward, normalization can be performed using scanpy.pp.normalize_total for both datasets. If you have any further questions, feel free to reach out!


Solved Issue #187- Github icon clearly visible now

PLEASE MERGE MY PULL REQUEST

please delete the repo and fork again to do your changes


Add a Timer Section with an alarm Clock

I would like to add a section of timer in which an alarm clock can be added. This will improve the multi-fuctionality of the Project. Please assign me this issue.

I would like to add a section of timer in which an alarm clock can be added. This will improve the multi-fuctionality of the Project. Please assign me this issue.

Okay.


Transaction Support with InMemoryAdapter

Given:
var db = Database.UseMockAdapter(new InMemoryAdapter());
db.BeginTransaction()
throws NotSupportedException - "Specified method is not supported"
Is there any recommended way to use both transactions AND unit tests with the InMemoryAdapter?
I guess I could just subclass the InMemoryAdapter and implement a dummy IAdapterWithTransactions which just calls the non-transactional methods and implements empty commit/rollback methods.
Or are there better approaches?
Update:
I just did a quick and dirty subclassing as per my previous comment. Tests come out in green, so that solves the issue for me.
https://gist.github.com/1665747#file_in_memory_adapter_with_shim_transactions.cs

That's an oversight. I'll fix and release in 0.14, which will be out by end of the week.
Mark

Sounds great!
When doing that - also consider that people will want to write natural joins while unit testing with the InMemoryAdapter. I presume this might/could be possible by providing a mock schema?

There is facility for setting up relationships in the InMemoryAdapter, although the syntax is awful and I'm going to improve it.

InMemoryAdapter will now not throw an exception when a Transaction is used, although CRUD operations are not actually transactional yet.

Works for me.


部分经纬度无法被正确解析

Version
4.4.0
Reproduction link
https://codepen.io/jiujiale/pen/RwwpGvE
Steps to reproduce
直接打开链接就会出现 worldGeo.getRegionByCoord 解析不出 region，应该[-74, 40.743]经度在前在New York 地区
What is expected?
getRegionByCoord 可以正确解析出 region
What is actually happening?
部分经纬度无法被解析出正确的region
example : [-74, 40.743] 经度在前

I think it's because the precision of GeoJSON is not enough.

I drew the point [-74, 40.743] on the geo component. It's outside the region.

OK. I see.
Will this situation be improved in later versions?

I'm afraid it can't be easily fixed because world GeoJSON don't have this high precision.
You can describe your purpose and let's see if we can work around .it

You can try to search the neighbor points by adding a small offset. I think it will raise the success possibility.

uh, that's a good plan B.
Thanks for your help~


[PE-D] Unable to calcuate group expenses

From the UG, it is expected that in a scenario where, in a group of 2 contacts, and 1 of the contact has an expense, the calculated payment will be that the other contact pays half of the expense to the first contact.
However, the app shows that There are no payments to be made!.
Test group and contacts:

Expenses of the group:

Result for calculatepayments gn/Group1:

Labels: type.FunctionalityBug severity.High
original: JeffZincatz/ped#8

Hi,
Thank you for the feedback!
Could you give me an idea of the order of commands you utilised?
I am assuming you created a fresh group with 2 members.
This group had no expenses.
You subsequently added an expense using the command
addexpense n/Test1 gn/Group1 $/SOME_AMOUNT d/testExpense
Then you entered calculatepayments gn/Group1 and found the message that no payments were to be made.
Expected behaviour was to show the message: "Test4 pays $SOME_AMOUNT/2 to Test1"
Is this an accurate description of your journey?


Merging new API version from private pr repo https://github.com/Azure/azure-rest-api-specs-pr/blob/RPSaaSMaster/specification/connectedvmware/resource-manager/Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

Can you please share Private Repo PR link, which is SignedOff?

Swagger Validation Report
✔BreakingChange succeeded [Detail] [Expand]
⚠Breaking Change(Cross-Version): 28 Warnings warning [Detail]
compared swaggers (via Oad v0.10.2)]	new version	base version
connectedvmware.json	2022-07-15-preview(c9bc94f)	2022-01-10-preview(main)
The following breaking changes are detected by comparison with the latest preview version:
Rule	Message
⚠ 1008 - ModifiedOperationId	The operation id has been changed from 'VirtualMachines_Create' to 'VirtualMachines_CreateOrUpdate'. This will impact generated code.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L1828:7
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L1828:7
⚠ 1008 - ModifiedOperationId	The operation id has been changed from 'VirtualMachines_List' to 'VirtualMachines_ListAll'. This will impact generated code.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L2246:7
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L2237:7
⚠ 1008 - ModifiedOperationId	The operation id has been changed from 'VirtualMachines_ListByResourceGroup' to 'VirtualMachines_List'. This will impact generated code.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L2289:7
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L2280:7
⚠ 1008 - ModifiedOperationId	The operation id has been changed from 'HybridIdentityMetadata_ListByVm' to 'HybridIdentityMetadata_List'. This will impact generated code.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L3396:7
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L3387:7
⚠ 1008 - ModifiedOperationId	The operation id has been changed from 'GuestAgents_ListByVm' to 'GuestAgents_List'. This will impact generated code.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L4001:7
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L3933:7
⚠ 1009 - RemovedRequiredParameter	The required parameter 'name' was removed in the new version.
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L100:11
⚠ 1009 - RemovedRequiredParameter	The required parameter 'name' was removed in the new version.
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L157:11
⚠ 1009 - RemovedRequiredParameter	The required parameter 'name' was removed in the new version.
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L3456:11
⚠ 1009 - RemovedRequiredParameter	The required parameter 'name' was removed in the new version.
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L3526:11
⚠ 1009 - RemovedRequiredParameter	The required parameter 'name' was removed in the new version.
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L3593:11
⚠ 1009 - RemovedRequiredParameter	The required parameter 'name' was removed in the new version.
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L3648:11
⚠ 1009 - RemovedRequiredParameter	The required parameter 'name' was removed in the new version.
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L3701:11
⚠ 1010 - AddingRequiredParameter	The required parameter 'virtualMachineName' was added in the new version.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L100:11
⚠ 1010 - AddingRequiredParameter	The required parameter 'virtualMachineName' was added in the new version.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L157:11
⚠ 1010 - AddingRequiredParameter	The required parameter 'virtualMachineName' was added in the new version.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L3465:11
⚠ 1010 - AddingRequiredParameter	The required parameter 'virtualMachineName' was added in the new version.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L3535:11
⚠ 1010 - AddingRequiredParameter	The required parameter 'virtualMachineName' was added in the new version.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L3602:11
⚠ 1010 - AddingRequiredParameter	The required parameter 'virtualMachineName' was added in the new version.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L3657:11
⚠ 1010 - AddingRequiredParameter	The required parameter 'virtualMachineName' was added in the new version.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L3710:11
⚠ 1017 - ReferenceRedirection	The '$ref' property points to different models in the old and new versions.
New: common-types/resource-management/v3/types.json#L305:9
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L6150:9
⚠ 1017 - ReferenceRedirection	The '$ref' property points to different models in the old and new versions.
New: common-types/resource-management/v3/types.json#L280:11
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L6173:11
⚠ 1025 - RequiredStatusChange	The 'required' status changed from the old version('True') to the new version('False').
New: common-types/resource-management/v3/types.json#L261:7
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L6063:7
⚠ 1029 - ReadonlyPropertyChanged	The read only property has changed from 'false' to 'true'.
New: common-types/resource-management/v3/types.json#L262:9
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L6064:9
⚠ 1029 - ReadonlyPropertyChanged	The read only property has changed from 'false' to 'true'.
New: common-types/resource-management/v3/types.json#L267:9
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L6068:9
⚠ 1029 - ReadonlyPropertyChanged	The read only property has changed from 'false' to 'true'.
New: common-types/resource-management/v3/types.json#L272:9
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L6072:9
⚠ 1029 - ReadonlyPropertyChanged	The read only property has changed from 'false' to 'true'.
New: common-types/resource-management/v3/types.json#L277:9
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L6076:9
⚠ 1042 - ChangedParameterOrder	The order of parameter 'retain' was changed.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L2029:9
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L2029:9
⚠ 1042 - ChangedParameterOrder	The order of parameter 'force' was changed.
New: Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json#L2029:9
Old: Microsoft.ConnectedVMwarevSphere/preview/2022-01-10-preview/connectedvmware.json#L2029:9
✔CredScan succeeded [Detail] [Expand]
⚠LintDiff: 0 Warnings warning [Detail]
✔Avocado succeeded [Detail] [Expand]
✔ApiReadinessCheck succeeded [Detail] [Expand]
⚠~[Staging] ServiceAPIReadinessTest: 0 Warnings warning [Detail]
✔~[Staging] SwaggerAPIView succeeded [Detail] [Expand]
✔ModelValidation succeeded [Detail] [Expand]
✔SemanticValidation succeeded [Detail] [Expand]
✔PoliCheck succeeded [Detail] [Expand]
✔PrettierCheck succeeded [Detail] [Expand]
✔SpellCheck succeeded [Detail] [Expand]
✔Lint(RPaaS) succeeded [Detail] [Expand]
✔CadlValidation succeeded [Detail] [Expand]
✔PR Summary succeeded [Detail] [Expand]
Posted by Swagger Pipeline | How to fix these errors?

Take a look at the breaking changes warnings flagged out in the Breaking Changes section.
Is previous preview version being used by existing customers? and same customers will be using new version also?

Can you please share Private Repo PR link, which is SignedOff?

Here are the list of commits (each commit is a separate PR which was signed off) for this new API version in private repo:
https://github.com/Azure/azure-rest-api-specs-pr/commits/RPSaaSMaster/specification/connectedvmware/resource-manager/Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview

Take a look at the breaking changes warnings flagged out in the Breaking Changes section.>
Is previous preview version being used by existing customers? and same customers will be using new version also?

The previous preview version (2022-01-10-preview) is available for customers, but we are still in public preview. The warnings in the new API version seem to be due to a few changes we did in this new API version:

Updating a few operation names for consistency across multiple Arc RPs (Microsoft.ConnectedVMwarevSphere, Microsoft.Scvmm, Microsoft.AzureStackHCI)
Moving to common error type from our own definition in the swagger.
Parameter rename for readability.

We should be ok as we don't have customers with hard dependency on the generated sdks yet.

    "publicKey": {

Why is this property removed?
Also, this property was added in new version?

Refers to: specification/connectedvmware/resource-manager/Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json:5010 in 63e2787. [](commit_id = 63e27877973a484c1795964dc911ec1ed8460159, deletion_comment = True)

Rest LGTM.

    "publicKey": {

Why is this property removed? Also, this property was added in new version?
Refers to: specification/connectedvmware/resource-manager/Microsoft.ConnectedVMwarevSphere/preview/2022-07-15-preview/connectedvmware.json:5010 in 63e2787. [](commit_id = 63e2787, deletion_comment = True)

Yes, during the development of the new API version, we added top-level publicKey field in VMProperties. Later we decided to move it to guestAgentProfile.clientPublicKey as that fits better in our scenario. This is not a breaking change as this version is not yet available to customers.

/azp run


Add dgml reporter

I am trying to refactor a project that has a bunch of hidden cycles exposed by BuildXL
It was nice to visualize the dependency graph that Lage inferred to see what the best way of splitting some projects into two to avoid having those cycles.
I started with a separate tool that translated the json logger output info into a Dgml file, but then I realized that it and just bake it into Lage as it seems useful for others as well.

@kenotron : Friedly ping as this might have fallen through the cracks during the holidays.


the functionality for sharing Blink Circles Card is broken

Describe the bug
Users can't share their Blink Circles Card.
To Reproduce
Steps to reproduce the behavior:

Go to 'People'
Click on 'View my Circles'
Click on 'Share my Circles'
See error

Expected behavior
When users click on 'Share my Circles' It's supposed to open the OS share modal and have the option to share the Blink Circles Card to multiple channels.
Screenshots
User having the problem: https://t.me/blinkbtc/8739
Additional context
Users instead of sharing their Blink Circles Card, only share the default tweet without the image https://x.com/luis51contreras/status/1770645244398194699

@rudyblink screenshots are always helpful, ie: in addition to the "4. See Error"

@nicolasburtey Ok, thank you


Warning during node transformation when using gatsby-transformer-json

Every time I build my Gatsby app I get this error from the plugin:
 ERROR

Plugin gatsby-transformer-json returned an error Could not find function loadNodeContent for plugin gatsby-source-github-api

  Error: Could not find function loadNodeContent for plugin gatsby-source-github-api

  - nodes.js:58 resolve
    [my-project]/[gatsby]/dist/db/nodes.js:58:15

  - new Promise

  - nodes.js:49 module.exports.loadNodeContent.node
    [my-project]/[gatsby]/dist/db/nodes.js:49:12

  - gatsby-node.js:62 Object.onCreateNode
    [my-project]/[gatsby-transformer-json]/gatsby-node.js:62:25

  - api-runner-node.js:230 runAPI
    [my-project]/[gatsby]/dist/utils/api-runner-node.js:230:37

  - api-runner-node.js:352 resolve
    [my-project]/[gatsby]/dist/utils/api-runner-node.js:352:15

  - debuggability.js:313 Promise._execute
    [my-project]/[bluebird]/js/release/debuggability.js:313:9

  - promise.js:488 Promise._resolveFromExecutor
    [my-project]/[bluebird]/js/release/promise.js:488:18

  - promise.js:79 new Promise
    [my-project]/[bluebird]/js/release/promise.js:79:10

  - api-runner-node.js:351 Promise.mapSeries.plugin
    [my-project]/[gatsby]/dist/utils/api-runner-node.js:351:12

  - util.js:16 tryCatcher
    [my-project]/[bluebird]/js/release/util.js:16:23

  - reduce.js:155 Object.gotValue
    [my-project]/[bluebird]/js/release/reduce.js:155:18

  - reduce.js:144 Object.gotAccum
    [my-project]/[bluebird]/js/release/reduce.js:144:25

  - util.js:16 Object.tryCatcher
    [my-project]/[bluebird]/js/release/util.js:16:23

  - promise.js:517 Promise._settlePromiseFromHandler
    [my-project]/[bluebird]/js/release/promise.js:517:31

  - promise.js:574 Promise._settlePromise
    [my-project]/[bluebird]/js/release/promise.js:574:18

The error is inconsequential and has the app builds fine. As far as I could tell, commenting out this line makes the warning go away but I'm not quite sure why it pops up in the first place. :s

Thanks! I'll investigate as soon as I have some time.


Fixes a bug with long string pair serde where null and empty strings are treated equivalently

Description
This PR fixes a bug with the long string pair serde where null and empty strings are treated equivalently, and the return value is always null. When 'useDefaultValueForNull' was set to true by default, this wasn't a commonly seen issue, because nulls were equivalent to empty strings. However, since the default has changed to false, this can create incorrect results when the long string pairs are serded, where the empty strings are incorrectly converted to nulls.
Original behavior:

value
deserialize(serialize(value))
round trip is same

new SerializablePairLongString(123, null)
new SerializablePairLongString(123, null)
Yes, behaviour is correct

new SerializablePairLongString(123, "")
new SerializablePairLongString(123, null)
No, behaviour is incorrect

New behavior:

value
deserialize(serialize(value))
round trip is same

new SerializablePairLongString(123, null)
new SerializablePairLongString(123, null)
Yes, behaviour is correct

new SerializablePairLongString(123, "")
new SerializablePairLongString(123, "")
Yes, behaviour is correct

Note, this now opens up possibilities of reading and writing with the different versions of the serdes - the new serde can deserialize the serialized value from the older serde, and the older serde can deserialize the value from the newer serde (during cluster upgrade and rollback)
The behavior in those cases will be:
(note: P(123, null) is equivalent to new SerializablePairLongString(123, null))

serialize using new, deserialize using old

value: P(123, null)

original behaviour -> deserializeOld(serializeOld(P(123, null))) = P(123, null)
deserializeOld(serializeNew(P(123, null))) = P(123, null)
Behaviour matches the original behaviour

value: P(123, "")

original behaviour -> deserializeOld(serializeOld(P(123, ""))) = P(123, null)
deserializeOld(serializeNew(P(123, ""))) = P(123, null)
Behaviour matches the original behavior, however is incorrect

serialize using old, deserialize using new

value: P(123, null)

original behaviour -> deserializeOld(serializeOld(P(123, null))) = P(123, null)
deserializeNew(serializeOld(P(123, null))) = P(123, "")
Behaviour doesn't match the original behaviour

value: P(123, "")

original behaviour -> deserializeOld(serializeOld(P(123, ""))) = P(123, null)
deserializeNew(serializeOld(P(123, ""))) = P(123, "")
Behaviour doesn't match the original behaviour, however the output is what was actually entered by the user.

In the second case, using the old serde means that the info whether the string was null or empty was lost. Therefore returning null (old behavior) or "" (new behavior) should be acceptable because the behavior was incorrect in the first place. This only happens when the old writer has deserialized the information, which already means that the distinction between null & empty was already lost.
Release note

Key changed/added classes in this PR

MyFoo
OurBar
TheirBaz

This PR has:

[ ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

IMHO
value: P(123, null)

    original behaviour -> deserializeOld(serializeOld(P(123, null))) = P(123, null)
    deserializeNew(serializeOld(P(123, null))) = P(123, "")

I think the behavior change here is a rolling upgrade show stopper. User's query will start giving wrong results.


Unify ASM

With the changes being made to generalize over programs we don't need separate asm's.
The Constraint asm can be moved a group up and instead of calling the top level StateRead it should just be named something like Ops.

Closed via #229, landing in #212.


[New Issue] All NPCs can only say "Oh no, what are we going to do now?"

After update 3 on Xbox all NPCs can only say "Oh no, what are we going to do now?". All dialogue interactions have been replaced by this one response. Usually it is not audible but only in a subtitle, but it has on occasion been spoken, even by traders in space. A few times I was able to play a quest without the bug appearing up to a point but if there is a step requiring me to speak with a character then they only say this line instead of mission dialogue and this halts the gameplay. I have tried to load earlier saves but the bug appears there also. Uninstalling and reinstalling also didn't work.

Details
First happened after receiving the ancient branch in Late Bloomer mission on Akila when returning to Jameson. In main quest on Freya 3 Nishina Research Station I was able to play quest up to the second objective requiring me to speak with Raphael after the pantry door key was used. Then he started saying the line too and it caused me to get stuck in the destroyed station universe without a vortex exit as these had not yet spawned.
Game Version
85391C36 V<IP_ADDRESS>
Language
English (EN)
Have you recreated this issue on a new game?
Yes, I can recreate this issue.
Reproduction Steps
After update 3, speak with any NPC until they start saying "Oh no, what are we going to do now?" Then try to speak to other NPCs and pretty soon all will say this one lune and nothing else.

Related to #915? Maybe this was introduced with the 1.10.31 patch.

Fixed by Bethesda in 1.10.32.

#915 was not fixed in 1.10.32, at least when I loaded my save that was next to them. I'll keep playing, hopefully it reloads or something.

Should be. One of the two fixes was specifically targeted at this bug, and other players are reporting that it's fixed.

Should be. One of the two fixes was specifically targeted at this bug, and other players are reporting that it's fixed.

Where did you see that? There's no patchnotes posted with this change.

"Addressed an issue that could cause characters to become stuck saying the same line of dialogue."

"Addressed an issue that could cause characters to become stuck saying the same line of dialogue."

Source: dude trust me bro? Where did you get this?

https://twitter.com/BethesdaStudios/status/1772655959740985761

https://twitter.com/BethesdaStudios/status/1772655959740985761

Thanks! I wish they had also fixed #913 but I guess asking Bethesda to do their job is a bridge too far, especially since they broke it with the patch.

Actually I see now that you were referring to an issue other than this one we're talking on. Today's hotfix addressed the "Oh no, what will we do" bug. The guard dialogue bug is different. It's actually caused by Spriggit merges of the community patch. Download this version to fix it: https://github.com/Starfield-Community-Patch/Starfield-Community-Patch/releases/download/0.1.1/StarfieldCommunityPatch.<IP_ADDRESS>Mar24.No.Spriggit.zip

Actually I see now that you were referring to an issue other than this one we're talking on. Today's hotfix addressed the "Oh no, what will we do" bug. The guard dialogue bug is different. It's actually caused by Spriggit merges of the community patch. Download this version to fix it: https://github.com/Starfield-Community-Patch/Starfield-Community-Patch/releases/download/0.1.1/StarfieldCommunityPatch.<IP_ADDRESS>Mar24.No.Spriggit.zip

Thanks!


A service to search all existing contracts inside eth-bytecode-db for new chains

When a chain runs blockscout instance on existing data, the number of verified contracts is zero initially. Even though some of the contracts may be verified through eth-bytecode-db search.
We want to create a script that will look for all existing non-verified contracts and tries to verify them via eth-bytecode-db lookup. That may be enhanced further to make a search for all newly indexed contracts.

Implemented in https://github.com/blockscout/blockscout-rs/tree/rimrakhimov/eth-bytecode-db-extractors/contracts-lookup.
Tried on Optimism Mainnet (https://optimism.blockscout.com/stats) - resulted in ~1.5 million of verified contracts
Need to create an instruction how to use the script, and merge it into main branch

https://www.notion.so/blockscout/Existing-contracts-lookup-for-new-chains-f51c56d38fa0401f9ea251eea16284cf


Update Neutron to v5.0.4

Pre-release image: ghcr.io/akash-network/cosmos-omnibus:v1.0.4-rc3-neutron-v5.0.4


Remote source rejected with status code 404

Error: remote source rejected with status code 404
at ClientRequest. (/home/container/node_modules/@napi-rs/canvas/load-image.js:69:23)
at Object.onceWrapper (node:events:633:26)
at ClientRequest.emit (node:events:518:28)

This indicates that the image url you are using is invalid. You can try changing the image url for loadImage function.

Same problem

Could the URLbe  accessed by your browser? @Twinkle069


Modal component replacements

To ensure consistent behavior and a streamlined development experience, we want to replace the existing modal components with ones that follow our established convention. This change will also simplify the component logic, improving readability and maintainability.

private create submit modal & notification modal.
public create submit modal & notification modal.
private edit submit modal & notification modal.
public edit submit modal & notification modal.
private delete submit modal & notification modal.
public delete submit modal & notification modal.
private review modal & notification modal.
public review modal & notification modal.
private delete modal & notification modal.
public delete modal & notification modal.

All the modal windows work as expected.


Improve Info-Page and FAQ

More content should be added to the Info Page. The FAQ page contains only a few entries so far. Here a brainstorming and an elaboration of further questions and texts should be done.

done


Cache Config.compute property

The compute property in Config calls Scriptor.runner() if the source for a data set is a python script. In Ingestor.translator, there are potentially 3 total calls to this property, one via compute_parsed here and then two others via Config.compute_json and compute_yml here (each of these three properties reference the compute property).
In cases where Scriptor.runner() is making web calls (such as doitt_building_footprints), this is causing 3 calls to download the same file, each time creating a new temporary download file.
doitt_buildingfootprints still failing for unrelated reasons, but at least it's failing quicker now that it's only downloading once!

Doesn't seem like any field is set during running so I don't think that there's a risk of any unintended consequences here

Hey @fvankrieken and @damonmcc! I am also having issues with building footprints with data library when I come across this work you did. I guess one insight I had about building footprints is that it works fine for csv but not pgdump telling me that it probably some bad geometries in the dataset causing the gdal translation error we are seeing. But I remember this kind of thing from building footprints in the past but usually "fixed itself" after a new release but this time months have passed. Just wondering if you have any new thoughts about this issue and happy to collaborate on fixing this! Thanks!

I think one of our GIS folks might have figured out part of this in terms of some preprocessing for the doitt_buildingfootprint, @damonmcc am I remembering that wrong? Do you/Jack have docs around those issues you were working on?

yup GIS tackled DTM preprocessing and tracked that work in this issue in the edm-overview repo: https://github.com/NYCPlanning/edm-overview/issues/891

Ah that was dof_dtm and not doitt_buildingfootprints?

true! sounds like the same issue of geometries that data_library can't repair


Refactor start_proteus.py

As discussed at yesterdays meeting, I'm opening this issue to track the progress with refactoring start_proteus.py.
The goal is to move all the logic etc into the library so that it can be re-used elsewhere. The goal is to have 3 entry points:

start_proteus.py, this will work in the same way as before. It will just be a lot smaller. This can be used to specify config variables, paths etc, but the main code will be elsewhere.
proteus start, this will be a CLI tool that starts proteus. The advantage is that it can be started from anywhere and does not require the user to git clone the repo. This will be more relevant as we move away from the submodules.
Python api, e.g. in a jupyter notebook. This is somewhat similar to 1. This will enable more exploratory use of the code, e.g. by importing the proteus class into your code:

from proteus import Proteus

handler = Proteus(cfg='input/default.cfg')
handler.start()

In this sense, will all variables for a given run then be contained within the PROTEUS class and objects within it? I can see that this might also make multiprocessing significantly simpler.


Splitting Bigtable Row into 3 classes based on the mutation type / RPC method used in commit()

Docs are still being updated, but I wanted to get this out there to speed up the review. Alternate implementation based on #1550.
I strongly urge reviewers to examine this commit-by-commit. The commit messages are fairly helpful (for the most part) and the diff of the PR as a whole is not so easy on the eyes.
Fixes #1548.
/cc @jonparrott @jgeewax

@tseaver just saw tests were failing on Travis. Looking into it.
What do we need to do to keep this review moving? Really close to the finish line and want to get a release out

OK the failure is simple to address (just a long failure)

*lint failure. Phone autocorrect!

This mostly LGTM, pending the one open item.

LGTM

Merging this, will handle the open item in a small separate PR.


Record parameters and primary results in annotation metadata

We started this in #998.  There, we added this to the annotation attributes for Nuclei Detection.  It has been previous done for Positive Pixel Count.  We'd like to also add the appropriate parameters for the other clis that we expose.  These could each be their own PR for clarity.
Besides the input parameters, anything stored in the returnParametersFile is a good candidate to include in the output  annotation['attributes'] dictionary.   If a cli does return a returnParametersFile (which happens if there are non-file output parameters), adding these results to the output annotation(s) if they exists or adding an output annotation just to hold those values would probably be better.  This would involve refactoring the xml to not have those outputs explicitly.

[x] BackgroundIntensity
[x] ColorDeconvolution
[x] ComputeNucleiFeatures
[x] NucleiClassification
[x] SeparateStainsMacenkoPCA
[x] SeparateStainsXuSnmf
[x] SuperpixelSegmentation

@manthey I was going through the Positive pixel count. Even though it is returning an annotation file, there is a returnParametersFile present in the UI and it is providing some important values. I would like to make changes in the PPC so that all the parameters are received via an annotation file.

Yes, please.  We can stop generating the returnParametersFile.


Cardinalities missing in XMI export

There are a few cardinalities in the content, that are not included in the output as XMI. One example is the property target cardinality 2..2 of the propperts maps in AgentSimilarityPair.

some more examples on: https://ddi-alliance.atlassian.net/browse/DMT-108

as i wrote in DMT-108, is this just a matter of adding 2..2 in the list of cardinalities?

deployed https://github.com/ddialliance/lion/commit/6b7b1c7e1ec9f5bb2124430b2064edb0d919b6a1 to production. @OliverHopt is this resolved?

Solved


Add pipelines-as-code for enterprise contract

Tekton is adopted as a primary CI and CD ( images builds ) for enterprise-contract

/test appstudio-e2e-deployment

@mbharatk, I don't quite follow what is being achieved by this PR. Can you explain it to me (in simple terms :smile: ) ?

@lcarva context -> PLNSRVCE-270. We are applying Pipelines-as-code for each repostiory. But in this case it would need some discussion why hacbs-contract is using own repositories instead of redhat-appstudio ones.

/test appstudio-e2e-deployment


Support comma-separated application names properly

Fixes #2

Thanks for the excellent contribution @cars10w, I've done a new release with your bug fix included.


Search by column

How do you search by columns such as:
this.dataTable.column(1).search(id).draw();

I figured out the solution to this. To search by a column using this wrapper, use
this.$refs.table.dataTable.column(1).search(id).draw()
Please not the column(1) is the column you want to search with. You could change that to 2, 3 or whatever column you want to filter with.
The value in .search(id) is the text you want to search with.
:smile:


Update helm opensearch version

Description
Update appVersion to 1.2.0
Issues Resolved
#151
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi @derek-diaz
Thanks for making the PR. Can you bump the version and add CHANGELOG for the PR.
Follow the PR #149 to get an idea on what needs to be done.

Would this considered to be a Major, Minor or Patch release?

@TheAlgo

Would this considered to be a Major, Minor or Patch release?

Patch release

@TheAlgo Alright I think everything should be good now. Do let me know if I missed anything!

Hi @derek-diaz strange that DCO does not pass can you rebase and push as one commit again?
Thanks.


question about the spec folder

hey, I was browsing through your code and was curious about the spec folder. What does it do? It seems like some kind of testing code.
I am the author of a Factorio mod myself, but never touched lua outside of that. If you have a way of writing automated tests for your mod code I would like to know more about it.

Hello :)
yes, that is test code. I had the same problem when I started todo list years ago. As there was nothing back then I created Faketorio. However I kinda stopped playing after that so the project is very barebones at the moment.
The basic idea is that all code should be tested. For pure logic this can be achieved with unit tests. However everything that runs inside the game logic needs the actual game.
Faketorio packages itself with your mod as a factorio mod and gives you certain callbacks at runtime to interact with tests. This gives you the possibility to run automatic tests within the real game.
While really nice in theory it has the problem of not working headless and in a CI server.
Feel free to ask more questions as they come up :)

Considering answered. Archiving.


Fix nginx config issue

#179 Fix Nginx config issue.

@Maxpeng92 You have a same / here Also you didn't remove network_mode as you mention
If you don't create virtual network by commenting out network_mode
Did you test composition before sending PR ? Is it working ?

Why is Semaphore failing here and I also can not see DCO check

@nmarcetic You only need to make sure not have a "/" when you use regex in Nginx config, that's why I only change line 152. Also, I mentioned I will let docker-compose file as what it was because you guys properly want to switch to docker swarm that's why I didn't comment out network_mode.

@Maxpeng92 please include the change into #185. These are "one liners" and are closely related, there is no point having 2 PRs for that.

@mijicd Sure. I did.


Metadata handling of SamplerV2 with fake backend does not match behaviour with real backend

Describe the bug
Metadata passed through jobs run with SamplerV2 on a fake backend (or AerSimulator) is in a different format to that received from SamplerV2 on real backends.
After issue #1551 and PR #1596 , circuit metadata is retained in the PrimitiveResult, but it lives in the top-level metadata attribute, and not the circuit_metadata key within this attribute as is the case with real backends.
Steps to reproduce
Run a simple circuit with metadata attached on fake backend and real backend using SamplerV2 primitive:
from qiskit.circuit import QuantumCircuit
from qiskit.transpiler.preset_passmanagers import generate_preset_pass_manager
from qiskit_ibm_runtime import SamplerV2 as Sampler, QiskitRuntimeService
from qiskit_ibm_runtime.fake_provider import FakeBrisbane

# Bell Circuit
qc = QuantumCircuit(2)
qc.h(0)
qc.cx(0, 1)
qc.measure_all()
qc.metadata = {0: 1}

# Run the sampler job locally using FakeBrisbane
fake_brisbane = FakeBrisbane()
pm = generate_preset_pass_manager(backend=fake_brisbane, optimization_level=0)
isa_qc = pm.run(qc)
print(f'circuit metadata: {isa_qc.metadata}')

# You can use a fixed seed to get fixed results.
options = {"simulator": {"seed_simulator": 42}}
sampler = Sampler(backend=fake_manila, options=options)
result = sampler.run([isa_qc]).result()
print(f'fake result metadata: {result[0].metadata}')

# Run on real device and compare metadata passthrough
service = QiskitRuntimeService()
backend = service.get_backend("ibm_brisbane")
sampler = Sampler(backend=backend)
job = sampler.run([isa_qc])
result = job.result(timeout=300)
print(f'real result metadata: {result[0].metadata}')

circuit metadata: {0: 1}
fake result metadata: {0: 1}
real result metadata: {'circuit_metadata': {'0': 1}}

Expected behavior
Fake and real result metadata dictionaries should be the same, with the injected circuit metadata living in the circuit_metadata key.
Suggested solutions
Modify changes introduced in #1596 to place the metadata:
metadata={"circuit_metadata" : pub.circuit.metadata}

Additional Information

qiskit-ibm-runtime version: 0.23.0
Python version: 3.10.14
Operating system: macOS Sonoma 14.5

Fixed in https://github.com/Qiskit/qiskit/pull/12784, in Qiskit 1.2


Queries on first loaded page are never fetched again

Problem
Using the SSR ApolloWrapper as defined in the README, any page that has useSuspenseQuery will be filled with data (fetched on the server) right away. That data, however, is never refetched no matter what fetchPolicy you define. The pages are not cached (export const dynamic = 'force-dynamic'), but the query never seems to be run again.
Queries on other pages are fetched as expected, it's just the first page that is loaded (regardless of which page that is) will keep displaying the results of its initial SSR fetch.
How to reproduce
I have set up a CodeSandbox to demonstrate this specific issue.

Open page one (hard refresh).
Note the total votes.
In a separate tab, increment the vote count.
Visit page two, and go back to page one.
We expect the total count to first show the cached value, and then be updated with the latest value (cache-and-network).
The value is never updated. :x:

We can repeat step 3 and 4 indefinitely, but page one will always show the initial total votes from step 2.
When page one is not the initial page we load, we get the expected behavior on page one:

Open page two (hard refresh).
Visit page one.
Note the total votes.
In a separate tab, increment the vote count.
Visit page two, and go back to page one.
We expect the total count to first show the cached value, and then be updated with the latest value (cache-and-network).
The value is updated. :heavy_check_mark:

Video of issue occurring in CodeSandbox
https://github.com/apollographql/apollo-client-nextjs/assets/1277208/2cf749ef-f419-4ea8-8cc2-3e77c0bb93e4
CodeSandbox
https://codesandbox.io/p/sandbox/interesting-dew-v7ccyr

Thanks for the bug report! I believe I fixed it over in #59. Could you give that one a go and report back if it works for you?
npm i<EMAIL_ADDRESS>

Thanks for the bug report! I believe I fixed it over in #59. Could you give that one a go and report back if it works for you?
npm i<EMAIL_ADDRESS>

Thanks for the quick reply and fix. After updating the package in the CodeSandbox it works as expected.


Email owners, contributors, and readers checkbox not present

Regarding the alert option to "Check Email owners, contributors, and readers option if you want the subscription administrators and co-administrators to be emailed when the alert fires", I do not see this checkbox in the portal. Has this option been removed?

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

ID: e119056c-5238-b244-7d87-bc383c40cb13
Version Independent ID: eb427e4c-c54e-06ad-54e0-efafde1df5f8
Content: Configure metrics alerts for Azure Database for PostgreSQL in Azure portal
Content Source: articles/postgresql/howto-alert-on-metric.md
Service: postgresql
Product: unspecified
GitHub Login: @rachel-msft
Microsoft Alias: raagyema

@snpdev Thanks for the comment. We are actively investigating and will get back to you soon.

@snpdev Adding a metric alert versus an activity log alert has different implications for configuring the means for the notification process.
When you select Metric Alert, you are presented with the 'Email owners checkbox'.

When you select the Activity Log Alert, you are presented with additional options, in addition to email:

Thanks Mike. I see the checkbox today, but not yesterday. Here is a screenshot I took on May 29th:

@snpdev Thanks for the update. We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.
Regards,
Mike


[Bug] Incorrect CSS causes repaint with every caret blink

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[ ] Not reproducible in the monaco editor playground

Monaco Editor Playground Link
No response
Monaco Editor Playground Code
No response
Reproduction Steps
Enable "Paint Flashing" in Chrome dev tools (tested with Chrome 126.0.6478.127 on macOS 14.2.1 (23C71)).
In some cases, the dev tools paint reflash doesn't work for the embedded iframe in playground. However, the broken repaint on caret blink should still be observable in the editor on the left
Actual (Problematic) Behavior

Try any monaco version after the referenced CSS change, including the latest one (0.50.0) and observe the entire editor repaint with each caret blink

https://github.com/user-attachments/assets/e53de4d1-e800-474c-9e52-ed600700e10c
Expected Behavior

Try again with a version before the referenced CSS change (eg: 0.43.0) and notice only the caret re-paints.

https://github.com/user-attachments/assets/d77c1383-5881-4a81-bd4e-d6e397237360
Additional Context
The CSS block here seems to be incorrect: https://github.com/microsoft/vscode/blob/1.91.1/src/vs/editor/standalone/browser/standalone-tokens.css#L40-L55
Presumably all the .monaco-editor, .monaco-diff-editor <foo> rules should instead be .monaco-editor <foo>, .monaco-diff-editor <foo>
As a side-effect, this change appears to cause the repaint on every caret blink as described above.

this behaviour is insane, destroys frame rate

can we get a fix for this?


Upgrade CI box underlying network and add automount script

See guidance in Hetzer Cloud UI

New 1TB filesystem mounted on the jenkins jobs directory - existing data copied across, leaving the workspace directory where it was. Original directory is in jobs-ORIGINALFS.200312.sxabak alongside jobs.

And we installed the scripts and networking upgrade.


chore: remove all asserts in source code, replace with error for internal exceptions, and InvalidDirectiveExceptions for input errors

Description of changes
Today, we have a number of assert statements in source which we shouldn't do. This PR replaces those with either Errors for faults, or InvalidDirectiveExceptions where invalid input is caught, to provide better visibility into operator error vs bugs.
Issue #, if available
N/A
Description of how you validated changes
I would have like to add more unit tests, but given the surface area of the change that was slowing me down, so I'm relying on e2e tests to verify all use-cases here.
Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #765 (9a4f620) into main (6a7b9f3) will increase coverage by 0.24%.
The diff coverage is 85.46%.

@@            Coverage Diff             @@
##             main     #765      +/-   ##
==========================================
+ Coverage   63.14%   63.39%   +0.24%
==========================================
  Files         278      278
  Lines       18125    18220      +95
  Branches     4375     4431      +56
==========================================
+ Hits        11445    11550     +105
+ Misses       6093     6074      -19
- Partials      587      596       +9

Impacted Files
Coverage Δ

...plify-graphql-auth-transformer/src/utils/schema.ts
94.69% <ø> (ø)

...searchable-transformer/src/cdk/create-cfnOutput.ts
100.00% <ø> (ø)

...transformer-core/src/transformer-context/output.ts
2.43% <0.00%> (-0.35%)
:arrow_down:

...graphql-transformer-core/src/utils/schema-utils.ts
18.18% <0.00%> (-4.55%)
:arrow_down:

...graphql-elasticsearch-transformer/src/resources.ts
100.00% <ø> (ø)

...ansformer-core/src/transformer-context/resolver.ts
5.69% <9.09%> (-0.16%)
:arrow_down:

...fy-graphql-auth-transformer/src/resolvers/field.ts
71.42% <33.33%> (+2.04%)
:arrow_up:

...-graphql-auth-transformer/src/accesscontrol/acm.ts
94.00% <50.00%> (+9.00%)
:arrow_up:

...mplify-graphql-relational-transformer/src/utils.ts
79.24% <50.00%> (-2.17%)
:arrow_down:

...l-transformer-core/src/transformation/transform.ts
16.00% <50.00%> (-0.32%)
:arrow_down:

... and 44 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update CONTRIBUTING.md

Issue: #523

@Graviton5 as I stated in the issue you can't work on it as it is for first-timers-only. Please work on other issues.


Mother 3 - scrolling corruption

Hi there,
Just trying out this emulator to get back into some GBA era games, the LCD filters and color correction look great! I did notice a bug though when trying out the English translation (v1.1) of MOTHER 3. Basically, when you walk around for a bit, the background sprites seem to "stick" occasionally so the background gets increasingly garbled over time:

Plugging the exact same ROM into NanoBoyAdvance, I get no such issues (same location, for reference of what the background should look like):

Enabling/disabling filters and ghosting settings didn't have any effect, so I'm assuming it's a glitch with the emulation itself? Also, this bug happens in both SkyEmu v3 (where I originally noticed it) as well as SkyEmu v2, so it's not a recent regression. This is using the official Linux binary from the GitHub releases page on a Fedora 38 system. Let me know if there's any other details I can provide!

Are you using an official GBA BIOS?

@skylersaleh I am! Without the BIOS the results are the same. The same BIOS renders the game normally without glitches with NanoBoyAdvance, so I'm reasonably sure it's a good BIOS dump. Are there any additional details I can provide?

Can also reproduce, only on Desktop - not on Web.

Can also reproduce, only on Desktop - not on Web.

Did you happen to capture a savestate that I could use to try to reproduce on my end?

I just submitted some fixes to dev that hopefully fixes this. Can you give it a test?
https://github.com/skylersaleh/SkyEmu/pull/321


Minimize convert_integer_literal tests

Closes #6455
This PR removes some redundant tests around convert_integer_literal.
First, it removes test cases about converting integer with underscores except for just one case. This is because the process of removing of underscores in given integer literals is independent from radix.
Second, as well as underscore cases, it removes test cases about converting integer with suffix except for just one case. This is because the process of handling suffix is independent from radix.
Finally, it removes small decimal integer cases. This is because "special case where they return early" has been removed by the PR #6482, so we have no need to retain this cases.

Perfect, thanks!
bors r+


[Bug]: Continuous usage stats are incorrect when chunked prefill is enabled

Your current environment

The output of `python collect_env.py`
$ python collect_env.py
Collecting environment information...
 The first response should have completion_tokens=0, this is how it behaved in previous versions of vLLM. Works fine without chunked prefill.
Might be related to #8625 but looks slightly different? I will debug it now and take a look at that one too.
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

@tdoublep I expect the recent changes I made fixed this issue, they included skipping streaming responses for intermediate prompt chunks. I will verify that when I get a chance!


fix: anchor not found error

this made camel-website checks fail

I saw this issue before I saw this pull request, should be fixed with 0f970ba2b711774624a0566c39afb9fed538cdc0, sorry for not seeing this before and thank you for keeping a close eye on this @Nayananga  :+1:


[Bug]: stage-maven-release.sh process leaves artifacts in the upload directory

After running stage-maven-release.sh a file is created that will automatically be uploaded if the script runs a second time, the upload process should be idempotent to prevent pollution of the environment or for a retry of the process.
Filename:
{SONATYPE_PROFILE_ID}.properties

File Contents:
#Generated by org.sonatype.plugins:nexus-staging-maven-plugin:1.6.5
#Thu Oct 07 02:31:22 UTC 2021
stagingRepository.id=orgopensearch-1012
stagingRepository.url=https\://aws.oss.sonatype.org\:443/content/repositories/orgopensearch-1012
stagingRepository.profileId={SONATYPE_PROFILE_ID}
stagingRepository.managed=false

See AbstractStagingDeployStrategy.java from org.sonatype.plugins:nexus-staging-maven-plugin where this is documented.  It does not look like it can be disabled or the file location altered.

Closing this issue as we are noticing this issue anymore now.


Cannot build when referencing 'FSharp.Data'

How to reproduce:
Prepare
 dotnet new console -lang F#
 dotnet add package FSharp.Data -v 3.0.0

Command that causes the problem
dotnet build

Expected behavior
Build successfully
What happens
Microsoft (R) Build Engine version <IP_ADDRESS>826 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.

  Restoring packages for /home/andre/Documents/Projects/MyProject/MyProject.fsproj...
  Restore completed in 201.84 ms for /home/andre/Documents/Projects/MyProject/MyProject.fsproj.
error FS3031 : The type provider '/home/andre/.nuget/packages/fsharp.data/3.0.0/lib/netstandard2.0/FSharp.Data.dll' reported an error : Assembly attribute 'TypeProviderAssemblyAttribute' refers to a designer assembly 'FSharp.Data.DesignTime' which cannot be loaded or doesn't exist. Could not load file or assembly 'FSharp.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified. [/home/andre/Documents/Projects/MyProject/MyProject.fsproj]
FSC : warning FS3005: Referenced assembly '/home/andre/.nuget/packages/fsharp.data/3.0.0/lib/netstandard2.0/FSharp.Data.dll' has assembly level attribute 'Microsoft.FSharp.Core.CompilerServices.TypeProviderAssemblyAttribute' but no public type provider classes were found [/home/andre/Documents/Projects/MyProject/MyProject.fsproj]

Build FAILED.

FSC : warning FS3005: Referenced assembly '/home/andre/.nuget/packages/fsharp.data/3.0.0/lib/netstandard2.0/FSharp.Data.dll' has assembly level attribute 'Microsoft.FSharp.Core.CompilerServices.TypeProviderAssemblyAttribute' but no public type provider classes were found [/home/andre/Documents/Projects/MyProject/MyProject.fsproj]
error FS3031 : The type provider '/home/andre/.nuget/packages/fsharp.data/3.0.0/lib/netstandard2.0/FSharp.Data.dll' reported an error : Assembly attribute 'TypeProviderAssemblyAttribute' refers to a designer assembly 'FSharp.Data.DesignTime' which cannot be loaded or doesn't exist. Could not load file or assembly 'FSharp.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified. [/home/andre/Documents/Projects/MyProject/MyProject.fsproj]
    1 Warning(s)
    1 Error(s)

Time Elapsed 00:00:02.68

Environment
$ uname -a
Linux AndreBook 4.18.9-arch1-1-ARCH #1 SMP PREEMPT Wed Sep 19 21:19:17 UTC 2018 x86_64 GNU/Linux

$ dotnet --list-sdks
2.1.403 [/opt/dotnet/sdk]

$ dotnet --list-runtimes
Microsoft.NETCore.App 2.1.5 [/opt/dotnet/shared/Microsoft.NETCore.App]

Additional comments
I've found some related issues in dotnet/cli#3776 and  netcorecli-fsc/#16, both created by @svick. It seems it should be working by now.

Just to say I don't repro this on Ubuntu 16.04 running on Bash Ubuntu for Windows
Will try on another linux distribution

Hmmm I tested this on an Azure Linux VM and it worked there too
dsyme@donlinux2:~/test$ uname -a
Linux donlinux2 4.15.0-1023-azure #24-Ubuntu SMP Tue Aug 28 17:35:08 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
dsyme@donlinux2:~/test$ dotnet --list-sdks
2.1.403 [/usr/share/dotnet/sdk]
dsyme@donlinux2:~/test$ dotnet --list-runtimes
Microsoft.AspNetCore.All 2.1.5 [/usr/share/dotnet/shared/Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.5 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.5 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
dsyme@donlinux2:~/test$

I'm not sure what to do to repro this

I will try to find steps that are more reproducible

I can reproduce this on Arch Linux. And in fsharp/FSharp.Data#1220 there is another Arch user affected by it. It looks like the issue appears on Arch Linux only, therefore I opened a bug report there.

I can also reproduce this issue on Arch Linux.


Add script generation for MacOS for the strongbox-distribution

Task Description
We need to enable the generation of shell scripts for MacOS in the strongbox-distribution.
Have a look at the appassembler plugin.
In theory, the work has already been done, but has never been tested under MacOS.
Hints:

https://github.com/strongbox/strongbox/blob/master/strongbox-distribution/pom.xml#L276
https://github.com/strongbox/strongbox/blob/master/strongbox-distribution/src/main/assembly/distribution.xml#L45

Help

Our chat
Points of contact:

@carlspring
@sbespalov
@fuss86

I tested creating the distribution on MacOS, and found the following to work:

mvn install succeeds.
The resulting archives (zip and tar) work on MacOS and on Linux for installing and running strongbox.
I'll work on documenting some install procedures for MacOS.

Excellent news! Thanks for the update! :)

Thanks for your help!


Write buffering

In some scenarios we were allowing transfer to be called inside write
before transfer had fired it's callback. We now respect those callbacks
and buffer our data internally.

Closing this due to lack of activity and progress in #30 for now.


Update google-oauth-jwt.js

Updated to make users able to specify if they want the entire body respone or just the auth token

Thanks for this PR. I will merge it. I don't know why do you need, however if you made this PR I assume that you needed it :) so I can merge it.


Version 2.2 Release

Updates EnyimMemcachedCore to varsion 2.2

Prevents infinite loop when a node is down on binary mode

Replaces DnsEndPint with (IP)EndPoint

The 'netcore' branch is removed and after that the 'master' branch will be main branch on this fork.

Sorry I mistaked


provisioning with cyliners

can this module support capacity provisioning using cylinders? if not can this be functionality added into this module?

Yes, this supports capacity provisioning using cylinders. On the methods 'create_non_empty_storage_group' and 'add_new_vol_to_storagegroup', simply pass in 'CYL' in for the capUnit; e.g. u4v.add_new_vol_to_storagegroup('my_sg', '2', '2000', 'CYL).


Prevent smoothing rollback

Increase max filtering values by a little bit because it's been bothering me for a while.
Prevent smoothing rollback:
before
after

you should upload the video directly to github, files in discord now have a limited time code in their link, so you will stop being able to watch the video

I see a lot of else now, what happens if tracker filtering is None?

you get free smoothing

I see a lot of else now, what happens if tracker filtering is None?


[Suggestion] Add support for BigWigs Packager single TOC file

Further to issue https://github.com/p3lim/toc-interface-updater/issues/7, I would like to reopen this as a suggestion rather than a bug report.
While the game itself doesn't support a single TOC files with multiple versions for the different releases, the BigWigs Packager supports this functionality when releasing an addon as a convenience.
https://github.com/BigWigsMods/packager#single-toc-file
While this functionality started out as a way to generate multiple .zip files for release on CurseForge, it has since evolved into a convenience feature for managing a regular single .zip release. When building a release of a project with the packager, it takes the versions specified within the TOC and generates multiple TOC files following the Blizzard spec.
This offers advantages over manually maintaining multiple TOC files for each game version, primarily removing the need to maintain duplicate information such as name, description, saved variables, addon files, and libraries.
I personally use this format for my own projects, and as you can see from the release the packager automatically creates multiple TOC files as per the blizzard spec.
I personally think this would be a beneficial feature to support, as it would make maintaining a project just that little bit more convenient.
Thanks!

v3.1 has been pushed to support this, and v3 points to this tag now.

Fantastic, thank you!


Get X package

Please add the getx package to favourite https://pub.dev/packages/get

there has been some drama due to false claims  and providing questionable benchmarks
Let alone that there are several valid reason not to use getx
I'd be very careful before "validating" it with a flutter favorite status if these fails claims aren't clarified
(I'm surprised that somehow getx made it to the list of state management options)

See https://docs.flutter.dev/development/packages-and-plugins/favorites#flutter-ecosystem--committee for the nomination process.
Closing as this process isn't managed via the issue tracker.


SFDX Auth Url not valid after 14/01 release

Summary
Today our deployment started failing with the error "force:auth:sfdxurl is not a sfdx command" that is reported in another issue. After modifying "force:auth" to "auth" as a suggested workaround that issue was solved but we then got an error for invalid SFDX auth url.
ERROR running auth:sfdxurl:store:  Invalid sfdx auth url. Must be in the format<EMAIL_ADDRESS>The instanceUrl must not have the protocol set.
The url was the same that used to work. I even tested in local where I didn't have updated to the new version and the url still worked.
Steps To Reproduce:
sfdx auth:sfdxurl:store -f credentials/develop-org.json -a targetORG
Expected result
Auth succeeded
Actual result
ERROR running auth:sfdxurl:store:  Invalid sfdx auth url. Must be in the format<EMAIL_ADDRESS>The instanceUrl must not have the protocol set.
Additional information
Feel free to attach a screenshot.
sfdx-cli/7.83.0-eba9c6a927 linux-x64 node-v12.18.3
@oclif/plugin-autocomplete 0.1.5 (core)
@oclif/plugin-commands 1.3.0 (core)
@oclif/plugin-help 3.2.0 (core)
@oclif/plugin-not-found 1.2.4 (core)
@oclif/plugin-plugins 1.9.4 (core)
@oclif/plugin-update 1.3.10 (core)
@oclif/plugin-warn-if-update-available 1.7.0 (core)
@oclif/plugin-which 1.0.3 (core)
@salesforce/sfdx-trust 3.4.3 (core)
alias 1.1.3 (core)
analytics 1.12.1 (core)
auth 1.3.0 (core)
config 1.2.1 (core)
generator 1.1.3 (core)
salesforcedx 50.9.1 (core)
├─ schema 1.0.1 (core)
├─ limits 1.0.0 (core)
├─ salesforce-alm 50.9.0 (core)
├─ templates 50.1.0 (core)
├─ @salesforce/sfdx-plugin-lwc-test 0.1.7 (core)
├─ custom-metadata 1.0.10 (core)
└─ apex 0.1.4 (core)
sfdx-cli 7.83.0 (core)
Ubuntu
18.04.5
LTS

Please see #811 - we will make sure this is addressed in the next release.

Please see #811 - we will make sure this is addressed in the next release.


g3doc tutorial mnist.py tf.range missing argument

 indices = tf.expand_dims(tf.range(batch_size), 1)
in https://github.com/tensorflow/tensorflow/blob/master/tensorflow/g3doc/tutorials/mnist/mnist.py#L94
which should be indices = tf.expand_dims(tf.range(0, batch_size, 1), 1)
The tutorial doc shows the correct code.

Apologies for the inconsistency.  We fixed the code to allow one-argument range, but the release hasn't changed and that's probably what you're using.  Good to know about the synchronization issue; we'll keep it in mind in future.

I'm going to fix this in git to be compatible back to 0.5.0.

Fixed, will be part of the next git commit.


AT+CWLAPOPT gives ERR CODE:0x01060204 when using optional parameters

Environment

Development Kit: none
Module or chip used: ESP32-WROOM-32
Power Supply:        external 5V
AT version: version:<IP_ADDRESS>(883f7f2 - Jul 24 2020 11:50:07)
SDK version SDK version:v4.0.1-193-ge7ac221
Bin version: 2.1.0(WROOM-32)

Problem Description
When using the AT+CWLAPOPT command on the default AT firmware gives ERR CODE:0x01060204 when using optional parameters
e.g. sending the message: "AT+CWLAPOPT=1,31" works fine and results in and OK message.
Sending: "AT+CWLAPOPT=1,31,-100" result in the error message: ERR CODE:0x01060203
Sending: "AT+CWLAPOPT=1,31,,255" results in the error message: ERR CODE:0x01060204
Sending: "AT+CWLAPOPT=1,31,-100,255" result in the error message: ERR CODE:0x01060204
Expected Behavior
To have OK returned instead of the error messages.
Actual Behavior
Returns error messages when including optional parameters.
Steps to repropduce

send "AT+CWLAPOPT=1,31,-100" as an AT messagge

@jortbmd according to the err code described here, https://docs.espressif.com/projects/esp-at/en/latest/Compile_and_Develop/AT_API_Reference.html#_CPPv417esp_at_error_code
0x01060203 means: command parameter counts mismatch (ESP_AT_SUB_PARA_NUM_MISMATCH), that is, expected two parameters, but three parameters were parsed.
0x01060204 means expected two parameters, but four parameters were parsed.
Actually, the v2.1.0 AT firmware, AT+CWLAPOPT just supports two parameters, see as:
https://docs.espressif.com/projects/esp-at/en/release-v<IP_ADDRESS>_esp32/AT_Command_Set/Wi-Fi_AT_Commands.html#cmd-LAPOPT
the latest AT firmware can support to four parameters, see as:
https://docs.espressif.com/projects/esp-at/en/latest/AT_Command_Set/Wi-Fi_AT_Commands.html#cmd-lapopt
Please get the latest firmware from the GitHub Actions: https://github.com/espressif/esp-at/actions or compile AT firmware according to https://docs.espressif.com/projects/esp-at/en/latest/Compile_and_Develop/How_to_clone_project_and_compile_it.html

Thank you! This helps. I could not find any updated binaries for the release. So I assumend it was the latest.


[Documentation] Error in documentation for NavigationRail constructor

The docs for the NavigationRail constructor state that assignments must be larger than one element:

The value of [destinations] must be a list of one or more [NavigationRailDestination] values.

But, based on the code and description of the destinations field, we can conclude that there must be more than two elements:
assert(destinations != null && destinations.length >= 2)

/// Defines the appearance of the button items that are arrayed within the
/// navigation rail.
///
/// The value must be a list of two or more [NavigationRailDestination]
/// values.
final List<NavigationRailDestination> destinations;

Hi @LamaMen, this has been fixed on the master channel. See the screenshot below taken from https://master-api.flutter.dev/flutter/material/NavigationRail/NavigationRail.html

and this one below taken from the source code.

Closing this as it has already been fixed on the master channel.
Thank you


Question: How to show the SVG files in Android-Studio, using "Android" mode?

It's just that resources are visible there, but not the SVG files:

Do I have to switch the mode? Or is there anyway else?
Is it possible to put the svg files into res/raw or assets , and while building, to ignore them ?

There isn't a way to do it right now; the Android project view is highly curated.
I don't believe there is a way to include it in /res/raw and not have it be included in the build. You could exclude those files from the source set, but then the Android project view wouldn't see it. :)

That's too bad. was hoping that a nicer integration would be possible.
But it's an open IDE, no? isn't there a way to make a plugin that would make it work this way?
or use gradle to do it?

I could look into it, I'm not sure how high priority it is since you can view all those resources in project mode.

I know. But project mode also shows a lot of junk and doesn't merge resources. :(

This isn't really something that can be fixed by this plugin; it's up to the IDE devs to make it compatible.


FileDownloadNetworkPolicyException Should Trigger Error Callback

Right now, the following exception would be triggered when attempting to download in cellular after setWifiRequired(true) is called:
 com.liulishuo.filedownloader.exception.FileDownloadNetworkPolicyException: Only allows downloading this task on the wifi network type
at com.liulishuo.filedownloader.download.DownloadLaunchRunnable.checkupBeforeConnect(DownloadLaunchRunnable.java:690)
at com.liulishuo.filedownloader.download.DownloadLaunchRunnable.run(DownloadLaunchRunnable.java:217)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607)
at java.lang.Thread.run(Thread.java:761)

This is not very helpful because the exception can't be caught and would just crash the app. It is much more helpful to trigger an error callback instead of crashing.

Thanks so much~


averageRating getting from 2 endpoints  has a different data type.

Describe the bug
Product averageRating getting  from endpoints -
/api/v1/products and /api/v1/products/{productId}/reviews/statistics  has different  data type.
/api/v1/products/{productId}/reviews/statistics   has  "avgRating": "string" - <class 'str'>
/api/v1/products  has  "averageRating": 0   - <class 'float'>

I can't seem to find the ProductInfoDto class file. Maybe the avgRating field in the DTO is a String ?

I can't seem to find the ProductInfoDto class file. Maybe the avgRating field in the DTO is a String ?

I compared the avgRating from JSON responses and got an error in a test that I tried to compare a different type of data.

Is this closed. I see a PR is closed but the issue is still open.
let me know if this is closed or else I can pick it up.
PS - I'm new to open source contribution so also let me know if I'm misunderstanding something.


WIP Model card

This is a WIP for model card. Opening this PR so you could take a quick look to see if the behavior seems good.
I will add what I can add on top of it (Evaluation results, model plot and so on)

I checked out from my documentation PR, feel free to ignore the parts about adding docstrings.

It looks good as a starting point. I just want to ask to add a notebook that shows before and after running create_card() function to clearly see what the generated model card looks like. WDYT?

@deep-diver sure I'll do that 🙂

@merveenoyan thanks! that would be great 👍🏼

@deep-diver
@merveenoyan is finishing the first version of the model card feature in this PR. She wanted to test HFPusher (which has the model card utility) in isolation, preferably in a notebook.
Do you have any starter notebook which she could use?

The revision of the test repository lies here, I tested like below: https://huggingface.co/merve/tfx-blessed-model/blob/merve/README.md
import tensorflow as tf
from runner import deploy_model_for_hf_hub

model = model_init() # random dummy model
model.save("./model_save_dir")

model_card_metadata = {}
deploy_model_for_hf_hub(
    username="merve",
    access_token=access_token,
    repo_name="tfx-blessed-model",
    model_path="./model_save_dir",
    model_version="merve"
)

@deep-diver had to add create_card to __init__ so it was importable :/

Closes #43


pkg/cluster: add kind field in the kubeadm config option

Starting from kubeadm v1.12.0, both kind and apiVersion are necessary in the kubeadm config file when initializing the cluster. If kind is missing, kubeadm init simply fails like this:
master-ajk8ix invalid configuration: kind and apiVersion is mandatory information that needs to be specified in all YAML documents
Failed to start cluster ...

Last week I tested this PR with k8s v1.9, v1.10, v1.11, and v1.12.
Will merge as it's necessary for running with v1.12.


🪲 Columns not being bound correctly

SELECT table1.* FROM 'tables/table1.csv' INNER JOIN 'tables/table2.csv' USING (a)

The engine does not correctly understand 'tables/table1.csv' and 'table1' are the same - this is probably correct

I'd say this is correct behaviour.


Question: Removing keys

Is it possible to remove keys from project/group/common?

No. But feel free to create a PR for that feature if you need it!


typing gets slow as conversation gets long

To reproduce,

connect to the debugbot
observe how typing a message to the debugbot is not slow
type 'send 9' a couple of times, wait until the bot has sent its messages.
observe that typing a message to the debugbot is slower now

@quasarchimaere thinks it seems related to #1109

no longer relevant due to state change PR


nb

牛的，很好用

https://t.me/OfficialTonshi_bot?start=MldMWHhkU2JDNmdHRFJ2YktvTCtSMGlZYjg4dzhPT0lrNDJOaE5tenZrTT0
这种也可以写一个吗

I looked at the application written in Unity, it's better to look for a bot written in Python or Node.js that will just send requests


cpu-simple: Do not wait for a cache response for prefetches

Currently, a prefetch instruction will result in an infinite sleep where the CPU never wakes back up as it is expecting a dcache response even though this will not be delivered for prefetches. Work around this by overriding the _status field for prefetches in completeIfetch().
This allows my test workload from https://github.com/gem5/gem5/issues/1139 to continue running beyond strlen() when using TimingSimpleCpu (but not other CPUs such as MinorCPU).
Partially fixes: https://github.com/gem5/gem5/issues/1139
Change-Id: Ic44bdb87f4099b11a7f9c6c99768a12fbef5842e

Hi @arichardson, there are a couple of tests that are failing. Could you please fix that before this can be reviewed/merged? Thank you.

Hi @arichardson. Software prefetching has a hacky implementation in gem5 which makes it buggy in some cases.
I have already seen several PRs trying to put a patch on a specific problem without really solving the underlying issue.
While this PR fixes your run, I am afraid is not enough to fix SW prefetching properly. That would probably require to handle cpu,arch,mem-cache,mem-ruby implementation of SW prefetches at the same time.
For example in the PR you force the CPU into running but this is simply because the CPU never gets a prefetch response message. However, the gem5 CPU does require a response for the prefetch (even if in real CPUs it does not make sense) as you can see from the SoftPFReq definition:
/* SoftPFReq */
{ {IsRead, IsRequest, IsSWPrefetch, NeedsResponse},
        SoftPFResp, "SoftPFReq" },

So why does your CPU never gets a response? It does so because your data prefetch targets uncacheable data (as you can see from the debug log you posted). Therefore:

It should have never been issued in the first place
It never gets a dummy response from the cache: https://github.com/gem5/gem5/blob/stable/src/mem/cache/cache.cc#L380 (it is uncacheable)

So I believe a partial solution which would make things work for you would be related to point 1, which is: discard a SW prefetch if it targets uncacheable data as data won't be stored anyway...

I believe I have a better fix for this now, scoped to the Arm MMU code. The new change allows my new reduced test case to pass for all CPU models.

Hi @giactra @arichardson, could you please provide an update on the status of this PR? Thank you!

Closing this PR due to inactivity. Feel free to reopen it whenever needed. Thanks!

Sorry for the delay - I will aim to address the feedback shortly.