Ensure MIPS Assembly track is ready for v2 launch

There are a number of things we're going to want to check before the v2 site goes live. There are notes below that flesh out all the checklist items.

[ ] The track has a page on the v2 site: https://v2.exercism.io/tracks/mips
[ ] The track page has a short description under the name (not starting with TODO)
[ ] The "About" section is a friendly, colloquial, compelling introduction
[ ] The "About" section follows the formatting guidelines
[ ] The code example gives a good taste of the language and fits within the boundaries of the background image
[ ] There are exercises marked as core
[ ] Exercises have rough estimates of difficulty
[ ] Exercises have topics associated with them
[ ] The first exercise is auto_approve: true

Track landing page
The v2 site has a landing page for each track, which should make people want to join it. If the track page is missing, ping @kytrinyx to get it added.
Blurb
If the header of the page starts with TODO, then submit a pull request to https://github.com/exercism/mips/blob/master/config.json with a blurb key.  Remember to get configlet and run configlet fmt . from the root of the track before submitting.
About section
If the "About" section feels a bit dry, then submit a pull request to https://github.com/exercism/mips/blob/master/docs/ABOUT.md with suggested tweaks.
Formatting guidelines
In order to work well with the design of the new site, we're restricting the formatting of the ABOUT.md. It can use:

Bold
Italics
Links
Bullet lists
Number lists

Additionally:

Each sentence should be on its own line
Paragraphs should be separated by an empty line
Explicit <br/> can be used to split a paragraph into lines without spacing between them, however this is discouraged.

Code example
If the code example is too short or too wide or too long or too uninteresting, submit a pull request to https://github.com/exercism/ocaml/blob/master/docs/SNIPPET.txt with a suggested replacement.
Exercise metadata
Where the v1 site has a long, linear list of exercises, the v2 site has organized exercises into a small set of required exercises ("core").
If you update the track config, remember to get configlet and run configlet fmt . from the root of the track before submitting.
Topic and difficulty
Core exercises unlock optional additional exercises, which can be filtered by topic an difficulty, however that will only work if we add topics and difficulties to the exercises in the track config, which is in https://github.com/exercism/mips/blob/master/config.json
Auto-approval
We've currently made any hello-world exercises auto-approved in the backend of v2. This means that you don't need mentor approval in order to move forward when you've completed that exercise.
Not all tracks have a hello-world, and some tracks might want to auto approve other (or additional) exercises.
Track mentors
There are no bullet points for this one :)
As we move towards the launch of the new version of Exercism we are going to be ramping up on actively recruiting people to help provide feedback. Our goal is to get to 100%: everyone who submits a solution and wants feedback should get feedback. Good feedback.
If you're interested in helping mentor the track, check out http://mentoring.exercism.io/

When all of the boxes are ticked off, please close the issue.
Tracking progress in https://github.com/exercism/meta/issues/104

Things that can be checked off already:

The track has a page on the v2 site: https://v2.exercism.io/tracks/mips
There are exercises marked as core
Exercises have rough estimates of difficulty

Things that need completing:

The track page has a short description under the name (not starting with TODO) - this is the blurb key in config.json
The about section needs updating to be friendly, colloquial, and compelling (while taking into account the new formatting guidelines)
The code snippet needs to be changed so it fits into the box (within the bounds of 10 lines with 42 characters each)
Exercises are missing topics
The first exercise needs auto_approve: true to be set (usually this is hello-world, but this track does not have such an exercise)

Should this track be named MIPS_Assembly?  I am thinking that the architecture is important, but so is the language being taught.

@kotp that's a good point. I'd be up for renaming it, but since the track already exists, it's going to break updates.
@iHiD did you by any chance write down a checklist for what has to be done to rename a track, when you did the smalltalk one?

Rename may be assembly-mips to allow any/all flavors to be sorted and available side by side.  May miss the other flavor(s) if they are scattered in a list.

@kotp good call. So you would suggest assembly-mips and assembly-x86?

Per discussion in https://github.com/exercism/x86-assembly/issues/1 we'd say mips-assembly

If there is no list related problems in discovery then I prefer the "as normally called" approach.


C++: Ensure that there is just one overlap for an operand in value numbering

It looks like an Operand can have multiple Overlaps. This makes value numbering assign multiple value numbers to the same instruction, giving rise to a tuple count explosion. This PR filters out those instructions with operands that have multiple Overlaps, and removes the tuple count explosion observed in bloomberg/bde.

I'd rather have us pick the most conservative overlap when there's doubt, preferring MustTotallyOverlap over MustExactlyOverlap, instead of giving up. We should be able to do that in the SSA construction rather than the value numbering.

I've done this now. Not in a way that I'm very proud of, but it seems to have the same performance characteristics as the original fix.

Unlike many of the other "Thingy has multiple Widgets for some reason" problems we've hit recently, this one seems more likely to be a straightforward bug in either SSA construction or alias analysis, rather than the usual extractor issue caused by the difficulty of pretending to link real-world C++ code. I'd prefer to chase this one down and fix the root cause, but if we don't have time for that, this workaround is as good as any.

Small observation: all of the cases I've seen (prior to this fix) seem to involve 128-bit values.  Does anything funny go on with those?

@geoffw0 Like __int128? Or some vector type? It's possible that code for the IR type system doesn't correctly handle certain esoteric types. Can you point me to a repro?

I did some more digging and found the root cause. There are two definition of the type BIT_UINT128:
From bid_functions.h:
typedef struct BID_ALIGN (16)
     {
       BID_UINT64 w[2];
     } BID_UINT128;

From bid_gcc_intrinsics.h:
typedef __attribute__ ((aligned(16))) struct
{
  BID_UINT64 w[2];
} BID_UINT128;

The BID_ALIGN (16) is just a macro that expands to the same aligned attribute as the other definition, but note the placement. The first definition puts the attribute after the struct keyword, and the second definition puts it before. This seems to be enough to make us treat the two structs slightly differently: One gets treated as an unnamed struct, and the other gets the name BID_UINT128. Any function or variable declared with type BID_UINT128 gets two types, and since the underlying structs are not the same type, they get two separate IROpaqueTypes as well. That causes the multiple overlaps, because we compute the overlap relationship(s) between all four pairs of IROpaqueTypes, two of which are MustExactlyOverlap and two of which are MustTotallyOverlap.
I'm merging this PR because we have no hope for fixing these sorts of database inconsistencies anytime soon.


Chromatic, Storybook, and UI Overhaul

partially addresses #78
closes #66 #51

wrong branch

wrong branch


Add new '%s' format to d3.time.format for the ordinal date suffix.

This change adds a new %s format to d3.time.format which prints the English ordinal suffix of the formatted date.
For example, d3.time.format("%b %e%s") used to format 01 Jan 1990 would output Jan  1st.

Thanks for the contribution! I'm a bit worried about deviating from the "standard" strptime and strftime behavior, but I see there's precedent for this in PHP, django, and datejs… so it may be worthwhile to include.
Also, ideally, there's feature parity between formatting and parsing. So, if you add %s to the format, the parser should probably be able to ignore the ordinal suffix when you parse "Jan 1st", e.g.

I will be working on this for 4.0, over in the d3-time-format repo.

Superseded by d3/d3-time-format#10.


Converting to .NET Core 2.0 broke the client

Identity server updating to IdentityServer4 v2.1.2. I only replaced AddTemporarySigningCredential with
AddDeveloperSigningCredential.
public void ConfigureServices(IServiceCollection services)
        {
            var idsrvBuilder = services.AddIdentityServer()
                // .AddTemporarySigningCredential() <== This is the only change after updating the nuget
             .AddDeveloperSigningCredential()
.AddInMemoryApiResources(Config.GetApiResources(Configuration["AppSettings:ApiName"]))
                .AddInMemoryClients(Config.GetClients(Configuration["AppSettings:ClientId"], Configuration["AppSettings:ApiName"]));

            idsrvBuilder.AddResourceOwnerValidator<ResourceOwnerPasswordValidator>();
            idsrvBuilder.AddProfileService<ProfileService>();
        }

        public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory)
        {
            app.UseIdentityServer();
        }

Client after updating to IdentityServer4.AccessTokenValidation v2.3.0.
Removed from Configure method.
app.UseIdentityServerAuthentication(new IdentityServerAuthenticationOptions
            {
                Authority = Configuration["AppSettings:IdentityServerUrl"],
                RequireHttpsMetadata = false,

                ApiName = Configuration["AppSettings:ApiName"]
            });

And added in ConfigureServices below the AddAuthorization.
services.AddAuthorization(options =>
            {
                options.AddPolicy("AccessPolicy", policy => policy.RequireClaim("scope", Configuration["AppSettings:ApiName"]));

                options.AddPolicy("RoleAccessPolicy", policy => policy.RequireClaim("role", "User"));
            });

            services.AddAuthentication(IdentityServerAuthenticationDefaults.AuthenticationScheme)
                    .AddIdentityServerAuthentication(x =>
            {
                x.Authority = Configuration["AppSettings:IdentityServerUrl"];
                x.ApiName = Configuration["AppSettings:ApiName"];
                x.RequireHttpsMetadata = false;
            });

Requesting a token from the auth service returns a bearer access token, using the access token from a client will respond with 401 Unauthorized.
Any ideas what I'm missing here?

Looks like app.UseAuthentication(); is needed in Configure.


feat(linter) expand  jsx-props-no-spread-multi to catch member expressions,closes #4894

#4981  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @camc314 and the rest of your teammates on  Graphite


#! Remove obsolete classes and forbid custom Pairs/Triples

Seem to be some left over from earlier times
Actually replaced by StateAwareWindowWiseSingleInputOperator

Thanks for the cleanup :+1:


[Question] Isn't setting align_corners=True a more popular choice in segmentation models?

In DeepLabV3 codes, when an upsampling layer is used, the align_corners argument is set as False :
https://github.com/pytorch/vision/blob/d2c763e14efe57e4bf3ebf916ec243ce8ce3315c/torchvision/models/segmentation/_utils.py#L25
https://github.com/pytorch/vision/blob/d2c763e14efe57e4bf3ebf916ec243ce8ce3315c/torchvision/models/segmentation/_utils.py#L31
https://github.com/pytorch/vision/blob/d2c763e14efe57e4bf3ebf916ec243ce8ce3315c/torchvision/models/segmentation/deeplabv3.py#L62
But researchers in the semantic segmentation field usually set this as True based on a common understanding that this will bring performance improvement(however small).

Hi,
Using align_corners=False actually make the results of the interpolation match the results of OpenCV and TensorFlow, so I would expect that using align_corners=False would be expected.
Do you have any references of researchers setting it to True elsewhere?

Well, it's more like an insider trick that is used by lots of people to get better performance at corners, and it is becoming a norm.
e.g.
https://github.com/jfzhang95/pytorch-deeplab-xception/blob/9135e104a7a51ea9effa9c6676a2fcffe6a6a2e6/modeling/deeplab.py#L31
https://github.com/speedinghzl/pytorch-segmentation-toolbox/blob/765941f8ce61390f78ecd94f155a121539be5108/networks/deeplabv3.py#L92
https://github.com/XiaLiPKU/EMANet/blob/b7d7ce0a46d4f17964a5551150742efd6ab6585b/dataset.py#L35

I wonder if this is done on purpose or is just an artifact that the error message, which reads
Default upsampling behavior when mode=bilinear is changed to align_corners=False since 0.4.0. Please specify align_corners=True if the old behavior is desired. See the documentation of nn.Upsample for details.

We would need to run some trainings with both align_corners=False and align_corners=True in order to see if there is any significant performance difference, before we consider making any changes.

I agree more trainings are necessary.
As I really don't have much resources myself, I only conducted training 3 times on each setup with mixed precision training and without COCO pretraining on this, and the average mIoU for align_corners=False & align_corners=True is 76.45% & 76.69% respectively, but it still could be just a coincidence since the training process is stochastic.
Looking forward for your training results.

Thanks for the numbers you reported! A 0.2% mIoU difference is very probably just within noise, so I wouldn't expect it to really make that much of a difference. But as you said, I should still run a few trainings just to make sure

Hi @voldemortX
@fmassa and I conducted the experiments regarding this.
We trained 3 models each for align_corners=True and align_corners=False, and we did not see difference.
Result

align_corners
True
True
True

False
False
False

Global Correct
92.5
92.5
92.8

92.5
92.6
92.4

Mean IoU
67.6
67.3
67.8

67.6
67.8
67.2

Environment

Thanks for the investigation @mthrok !


🐛 Fix gap in APIExport virtual workspace queues

Summary
Fix a gap in the APIExport virtual workspace queueing where it was possible to incorrectly return a "not found" error when trying to list a claimed resource. The gap was that we weren't queueing the dependent APIExports; i.e., if APIExport A gets updated, and APIExport B has a permission claim on A's identity, we need to make sure to queue B in this situation.
Related issue(s)
Fixes #2713

/approve


Fix Phar::running false positive error

Codecov Report
Patch coverage: 66.66% and no project coverage change.

Comparison is base (2875939) 99.23% compared to head (997a31c) 99.23%.

:exclamation: Current head 997a31c differs from pull request most recent head e661471. Consider uploading reports for the commit e661471 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##           fix/phpstan      #13   +/-   ##
============================================
  Coverage        99.23%   99.23%
============================================
  Files               56       56
  Lines              910      911    +1
============================================
+ Hits               903      904    +1
  Misses               7        7

Impacted Files
Coverage Δ

src/Process/ConcreteProcessFactory.php
96.55% <66.66%> (+0.12%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


License

Great work! I can't wait to try it out. Please add a license when you have time

Thanks for your interesting in our work. MIT license has been granted, feel free to try it out.


Notifications for editing money requests

HELD on https://github.com/Expensify/Web-Expensify/pull/38830

Details
Adds support for displaying notifications for modified expense actions
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/312244
Tests
Notifications not supported on mobile web

Create a workspace with an admin and member
Log in as the admin
Accept any prompt for notification permissions
On another device/browser login as the member and create a money request
As the admin, open the expense and reply to the thread to subscribe to it for notifications
Navigate away from the chat (eg: back to concierge)
As the member, edit the amount, description, date and merchant
On the admin device, verify for each edit a notification displays describing what changed

[ ] Verify that no errors appear in the JS console

Offline tests
None
QA Steps
Same as Tests

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

N/A

Mobile Web - Safari

N/A

Desktop

iOS

Android

Alright this tests well with the given Web-E PR. However, push notifications are not working on iOS in dev atm so I cannot test there.

Alright push notifications not working in dev is a known issue. Since this is controlled by the backend, verifying just Android should be safe since the logic is the same for both platforms.

Ready for another review! @grgia @ntdiary

@arosiclair will let you decide if you want to address @marcaaron's comments or merge

@arosiclair unit test is failing?

@arosiclair unit test is failing?

Yeah, I pointed it out here. REPORT_ACTION needs a valid actionName value.

Woops just fixed that. Ready for another review! @marcaaron

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/arosiclair in version: 1.3.75-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.75-12 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/arosiclair in version: 1.3.76-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.76-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Don't preprocess hot/win vars for sms campaigns.

moved the call to test if campaigns are in a hot/win state after the check for sms campaigns.
Fixes #4982

:ok_hand:!


[DRAFT] Dataset from polars dataframe

Pull Request Template
Checklist

[x] Confirmed that run-checks all script has been executed.
[ ] Made sure the book is up to date with changes in this PR.

Related Issues/PRs
#2015
Changes
The ability to leverage a polars dataframe for training by providing the utilities to create a burn dataset directly from a polars dataframe.
added the dataframe.rs module in the burn-dataset crate
Testing
Unit tests are included in the code

I have a few inlined comments and a question.
Polars has serde (feature) support for dataframes (see code). Probably you can bypass using JSON to deserialize row altogether. Have you looked into this?

Hey @antimora, I did, but I found that the serialize method is not fully implemented in polars yet. I've opened an issue on the Polars repo.

I have a few inlined comments and a question.
Polars has serde (feature) support for dataframes (see code). Probably you can bypass using JSON to deserialize row altogether. Have you looked into this?

Hey @antimora, I did, but I found that the serialize method is not fully implemented in polars yet. I've opened an issue on the Polars repo.

If I understood correctly not all column types supported? Can we support a subset of possible types? This still would be highly useful. Otherwise it seems one needs to encode row into JSON, which won't much more useful than having json row encoded lines in a plain text file. Or maybe I am misunderstanding the use of JSON here.

I have a few inlined comments and a question.
Polars has serde (feature) support for dataframes (see code). Probably you can bypass using JSON to deserialize row altogether. Have you looked into this?

Hey @antimora, I did, but I found that the serialize method is not fully implemented in polars yet. I've opened an issue on the Polars repo.

If I understood correctly not all column types supported? Can we support a subset of possible types? This still would be highly useful. Otherwise it seems one needs to encode row into JSON, which won't much more useful than having json row encoded lines in a plain text file. Or maybe I am misunderstanding the use of JSON here.

it seems that the pull request to fix the todo! issue in polars might get through. if that's the case then we can just utilise that. will be cleaner

I have a few inlined comments and a question.
Polars has serde (feature) support for dataframes (see code). Probably you can bypass using JSON to deserialize row altogether. Have you looked into this?

Hey @antimora, I did, but I found that the serialize method is not fully implemented in polars yet. I've opened an issue on the Polars repo.

If I understood correctly not all column types supported? Can we support a subset of possible types? This still would be highly useful. Otherwise it seems one needs to encode row into JSON, which won't much more useful than having json row encoded lines in a plain text file. Or maybe I am misunderstanding the use of JSON here.

it seems that the pull request to fix the todo! issue in polars might get through. if that's the case then we can just utilise that. will be cleaner

Yeah I think we should definitely use Dataframe's native serialization/deserialization methods. Otherwise this Dataset will be little of use.
A cool use case I envision as follows:

One can open any Polars datasource (CVS, JSON, binaries, SQL Dbs, etc) into Dataframe.
Apply dataframe filters  to filter (
Define struct for a row
Wrap with Dataframe Dataset (can also be passed to SqliteDataset or MemoryDataset to consume and cache locall)
Pass it to training/testing

BTW, your PR is waiting to fix this. Not sure if you noticed:

We spoke offline about using Polar's serde feature to deserialize a row. I am not sure if new code wasn't pushed (that you mentioned working), or there is still misunderstanding.

The code that I showed you working is the code in this PR as it stands

OK. I removed serde_json and implemented the native deserializer. Please finish up and clean up the code. Also please add more tests and data types.
One test is passing:
[burn-dataset]% cargo test dataframe --features dataframe
    Finished `test` profile [unoptimized] target(s) in 0.12s
     Running unittests src/lib.rs (/Users/dilshod/Projects/burn/target/debug/deps/burn_dataset-c350f307979d05bd)

running 1 test
test dataset::dataframe::tests::test_dataframe_dataset ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 25 filtered out; finished in 0.00s

OK. I removed serde_json and implemented the native deserializer. Please finish up and clean up the code. Also please add more tests and data types.
One test is passing:
[burn-dataset]% cargo test dataframe --features dataframe
    Finished `test` profile [unoptimized] target(s) in 0.12s
     Running unittests src/lib.rs (/Users/dilshod/Projects/burn/target/debug/deps/burn_dataset-c350f307979d05bd)

running 1 test
test dataset::dataframe::tests::test_dataframe_dataset ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 25 filtered out; finished in 0.00s

@antimora I've added support for LazyFrame, added support to deserialize a few more datatypes and added some tests. So far, there's no serialization for complex types, but this is a good starting point. Let me know what you think and if there's anything urgent I need to fix/add
Two tests are passing

@laggui @nathanielsimard I took over this PR to complete it. I made final changes and ready for your review.

Good!


Support Python 3.12

If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

It looks like protobuf 5.26.0 added python 3.12 support, which I assume this issue was waiting on?


Fix the link so that it navigates to anchor rather

I think we can already accept this PR, thanks @another-guy! :wink:


price-feeder: Oracle not robust against faulty providers

From ToB Audit
Description

The price-feeder uses Volume Weighted Average Price (VWAP) formula to compute
average price from various 3rd party providers. The established price is then sent to the
x/oracle module that commits it on-chain. The price can easily be manipulated by only
one compromised or malfunctioning oracle's 3rd party provider.

Exploit Scenario

The Binance API is used by most of the validators as one of many providers. The API is
compromised by an attacker and suddenly starts to report values that are much larger
than reported by other providers. Price discrepancies are not detected by price-feeders
modules that are used by validators using Binance API. The computed VWAP price is much
larger than it should be. It is committed on-chain. Because most validators have committed
the wrong price, the on-chain computed average is also wrong.

Recommendations

Short term, add detection if one or more providers report prices that are way too off in the
price-feeder. Temporary disable malfunctioning providers and issue an alert to
investigate the problem. Stop committing prices if it is not possible to automatically detect
which providers are broken - please note that this may result in loss of interest for
validators. In the x/oracle module consider implementing a similar mechanism that will
detect if exchange rates committed by validators are not too diverse between validators or
too different from old values.

References:

https://blog.synthetix.io/response-to-oracle-incident/

I would propose we just have safe guard checks in the price-feeder to not use a provider's price if it's ∂ from the average of the others.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Also going to add a requirement for at least three providers in config per token, since with Synthetix :

Our price oracle has a mechanism for discarding outliers and should have absorbed this discrepancy gracefully, unfortunately the price feed for KRW was only being served by two API’s at that time due to an earlier unrelated outage which had not been caught by our exception reporting. This meant the Oracle took an average of the only two remaining prices which caused it to misreport the price of KRW to our exchange rates contract.


Add gatsby-plugin-sharp support?

Hi @aripalo, this looks great!
I tried installing gatsby-plugin-sharp, but it fails because it doesn't have all the tools available to build sharp and / or its dependencies. Luckily there is an Alpine image that contains those dependencies already - see https://github.com/lovell/sharp/blob/master/docs/install.md#docker
For some background - I've recently been experimenting with Docker images for building a documentation tool for Gatsby, so you can see an example of building a Gatsby friendly container there. Sadly my image is the standard full-fat Node.js / Ubuntu image.
For reference, I ran docker run -it --rm -v $(pwd)/site:/site aripalo/gatsby-docker yarn add gatsby-plugin-sharp
And the error output was:
error /site/node_modules/sharp: Command failed.
Exit code: 1
Command: node-gyp rebuild
Arguments:
Directory: /site/node_modules/sharp
Output:
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| linux | x64
gyp ERR! configure error
gyp ERR! stack Error: Can't find Python executable "python", you can set the PYTHON env variable.
gyp ERR! stack     at PythonFinder.failNoPython (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:483:19)
gyp ERR! stack     at PythonFinder.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:397:16)
gyp ERR! stack     at F (/usr/local/lib/node_modules/npm/node_modules/which/which.js:68:16)
gyp ERR! stack     at E (/usr/local/lib/node_modules/npm/node_modules/which/which.js:80:29)
gyp ERR! stack     at /usr/local/lib/node_modules/npm/node_modules/which/which.js:89:16
gyp ERR! stack     at /usr/local/lib/node_modules/npm/node_modules/which/node_modules/isexe/index.js:42:5
gyp ERR! stack     at /usr/local/lib/node_modules/npm/node_modules/which/node_modules/isexe/mode.js:8:5
gyp ERR! stack     at FSReqWrap.oncomplete (fs.js:166:21)
gyp ERR! System Linux 4.9.60-linuxkit-aufs
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /site/node_modules/sharp

Good catch! I haven't yet gotten myself so far as to use gatsby-plugin-sharp with this Docker setup, but that is something this should support of course!
Thanks for the links! Gonna look into those!

So I finally got the gatsby-plugin-sharp to succesfully install. Had to add quite a lot of new dependencies.
I created a PR https://github.com/aripalo/gatsby-docker/pull/2, which I will close after I've had time to do more testing and figure out if I can get rid of some dependencies (probably won't before sunday).
If you wish to test this feature, see the instructions within the PR.

Nice! I'll give it a spin in the next day or two.
I saw this article the other day. Section 5 talks about installing dependencies and building files in one layer, then creating a fresh layer and copying in only the built files. Which should result in a smaller final image.. maybe it's possible to do the same with sharp?


Needs to update the readme

Needs to update the readme.md file properly

@thebishaldeb I would like to work on this issues

Sure. Upload apt. screenshots and do the needful changes.

Any update?

Can I work on this issue?

I am currently working on it
On Tue, Dec 10, 2019, 4:31 PM anushkamittal<EMAIL_ADDRESS>wrote:

Can I work on this issue?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/NJACKWinterOfCode/Instances/issues/23?email_source=notifications&email_token=AJ23PLDWP62R2XUTHNBHFI3QX5ZKJA5CNFSM4JW66RA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGO247Y#issuecomment-563981951,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ23PLAJEHNUUJZPYFSBR4LQX5ZKJANCNFSM4JW66RAQ
.

ok

@sgpritam since you're having exam, assigning it to @anushkamittal

No sir
On Tue, Dec 10, 2019, 5:22 PM Bishal Deb<EMAIL_ADDRESS>wrote:

@sgpritam https://github.com/sgpritam since you're having exam,
assigning it to @anushkamittal https://github.com/anushkamittal
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NJACKWinterOfCode/Instances/issues/23?email_source=notifications&email_token=AJ23PLB6MK7SB3B4QYMNQYDQX57GNA5CNFSM4JW66RA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGO7BIA#issuecomment-563998880,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ23PLBLX2MS4QSJTSICEN3QX57GNANCNFSM4JW66RAQ
.

It is been three days and there is no update. Please look into other issues.

Ok please update marks
On Tue, Dec 10, 2019, 5:25 PM Bishal Deb<EMAIL_ADDRESS>wrote:

It is been three days and there is no update. Please look into other
issues.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NJACKWinterOfCode/Instances/issues/23?email_source=notifications&email_token=AJ23PLB36EKVVTVS6J2IZTTQX57RXA5CNFSM4JW66RA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGO7ICY#issuecomment-563999755,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ23PLFK377FN4AWKODYHFDQX57RXANCNFSM4JW66RAQ
.

My score has not yet been updated which I have created a legit issue
On Tue, Dec 10, 2019, 5:25 PM pritam kumar<EMAIL_ADDRESS>wrote:

Ok please update marks
On Tue, Dec 10, 2019, 5:25 PM Bishal Deb<EMAIL_ADDRESS>wrote:

It is been three days and there is no update. Please look into other
issues.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NJACKWinterOfCode/Instances/issues/23?email_source=notifications&email_token=AJ23PLB36EKVVTVS6J2IZTTQX57RXA5CNFSM4JW66RA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGO7ICY#issuecomment-563999755,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ23PLFK377FN4AWKODYHFDQX57RXANCNFSM4JW66RAQ
.

Done.

Can you please tell me what exact update do you want?

The screenshot of the pages need to be updated, login and register page need to be added.
make it such that two pages can be shown in own row, with smaller size images. They shoulsd not take up much space.

Okay

@thebishaldeb I have created a PR. Please review it.
https://github.com/NJACKWinterOfCode/Instances/pull/37


added google-auth and added strong password hashers

Related Issue or bug

Added google authentication to the project

Fixes: #210
Describe the changes you've made

modified db with client id and secret key
added login with google button in login.html and register.html
setup auth app in settings.py
added routes to social login in `urls.py
added 3 new modules in requirements.txt

Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Ran tests locally: passed
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] New and existing unit tests pass locally with my changes.
[x] Any dependent changes have been merged and published in downstream modules.

Screenshots

Original
Updated

|

 |

What is the use of this and why did you use this ?

Ohhh that is my own chrome's personal extension......it has not been added to the project


extractor for data from filled form "Anzeige aufgeben"

In some cases it's more easy to fill the web page, to create a new offer. especially when creating offers with special fields like for cars or bikes. so i created a javascript script, which can be uses to extract the filled data as "ad" object. the extracted data can be included into your config.json using copy and paste.
It is used like this: Fill in all data in a Chrome browser, after clicking "Anzeige aufgeben", or "Bearbeiten" and open developer tools (right click on "Untersuchen" or press CMD+ALT+I (on Mac)). Select the Tab "console" and copy the full text of this script into it and press RETURN
It should show you the data for copying it.
https://github.com/luxlux/ebayKleinanzeigen/blob/master/extractor.js
While doing this, i noticed, some things, which a not implemented.
Unfortunately i never did something in python. May be somebody can add this:
Up to now kleinanzeigen.py script have no handling for filling checkboxes, like used i.e. for offers of cars.
This is an example category with such checkboxes:
 "caturl": "https://www.ebay-kleinanzeigen.de/p-kategorie-aendern.html#?path=210/216/alfa_romeo&isParent=false",
  "additional_category_options": { "autos.radio_tuner_b": true, ...}

also a handling for "posterType": "PRIVATE" or "posterType": "COMMERCIAL" is not implemented up to now.
Former a filling of the "imprint" field for "COMMERCIAL" should be added.

Cool thing, I'll play around with this a bit more the next upcoming days!

Possibly it could also be used to inject it from python via selenium to automatically read out offers (to get manual made changes), before an offer will be deleted and reposted. or to import all offers. or to get an offer by id.
I never used python or selenium, but i guess something like injection of javascript should be part of such a solution. injection will also be used by chrome extensions...


Behavior of lists of embedded objects seems to be broken

Lists of objects inside a structure seem to be a special case that is not behaving consistently with the rest of the definition. I'm still new to monogkit so I may be doing things incorrectly though so feel free to point me in the right direction.
I've checked in a few test cases to illustrate my points here: https://github.com/sean-lynch/mongokit/blob/master/listembed-test.py
Example:
  class ListEmbed(Document):
        use_dot_notation = True
        structure = {
            'list': [
                {
                    'name': basestring,
                    'age': int
                }
            ]
        }

You can't specify default values for the embedded object of the list. For example, I'd like to specify that the default value for 'name' is 'New User'. My expectation is that, every embedded object inside the list has this default applied to it if no name value is set. #49 was an attempt to fix this, but it simply sets the default value of the entire list, not of the objects inside the list.

fields in the embedded object are treated as required even when not specified in the required_fields attribute. With the above definition of a class, I should be able to save a new document with the structure {'list': [{'name': 'Bob'}]} but instead, it complains about age missing.

I've also looked at using DBRefs to accomplish the same thing. I've avoided it so far because I don't want to store the embedded objects separately and pay the performance penalty of rebuilding the list when I query them.

So, let me get this right, I can't just add only the fields that I need to set the default values but I have to had the whole sub document? I did this but it seems like it creates the new document with default value and than  insert the actual document I am trying to save.


Rewrote jw_hartree_fock_state

I noticed that jw_hartree_fock_state could be made more efficient. I actually wrote a more general function jw_slater_determinant which lets you instantiate any computational basis vector. I think it's an improvement.
The original function returns a CSR matrix but I noticed that we mostly use CSC so I returned a CSC. Not sure what the policy on that is (if any). Along those lines, I fixed what seems to be a typo in the initialization of identity_csc.

These are good changes! Good catch about CSC vs CSR - pre-release the code was mostly CSR but we changed to improve sparse efficiency. Unfortunately CLA bot has gone rogue again - @babbush could you check it out / override?

Actually, CLA only got annoyed when I merged master in - can you check if my license is registered properly?


Add "IS [NOT] DISTINCT FROM" comparison operators to Cond

It would be nice to have Cond.IsDistinctFrom and Cond.IsNotDistinctFrom for the comparison operators IS DISTINCT FROM and IS NOT DISTINCT FROM.

It's a bit tricky to implement IS DISTINCT FROM and IS NOT DISTINCT FROM operators. It's a part of SQL-2003 standard, but MySQL and many MySQL-like systems don't support it. I need to think of the design for a while.

I found a smart way to support these operators on different database systems. I just submit the fix and will release it soon.

Does this mean you could also support flavor-specific operators, like PostgreSQL's ARRAY type operators (@>, <@, &&, ||)? If not, what's the best way to include these in a query built with go-sqlbuilder?

It depends. Regarding to the ARRAY type operators, as I can't find proper alternatives on other DB systems, I don't think it's feasible to support it in this package.
If you want to build flavor-specific operators, expressions or statements, you can use the Var() method to create placeholders for any value and use your favorite string builder (e.g. fmt.Sprintf) to build a desired string.
Here is a sample to build SQL using @>.
type Varer interface {
	Var(value any) string
}

// ArrayContain is used to construct the expression "field @> ARRAY[values]".
func ArrayContain[V Varer, T any](v V, field string, values []T) string {
	return fmt.Sprintf(
		"%s @> ARRAY[%s]",
		sqlbuilder.Escape(field),
		v.Var(sqlbuilder.List(values)),
	)
}

labels := []string{"label1", "label2"}
sb := sqlbuilder.Select("labels").From("users")
sb.Where(
    ArrayContain(sb, "labels", labels),
)

sql, args := sb.BuildWithFlavor(sqlbuilder.PostgreSQL)
fmt.Println(sql)
fmt.Println(args)

// Output:
// SELECT labels FROM users WHERE labels @> ARRAY[$1, $2]
// [label1 label2]

Thanks for the sample!
I'm generating very dynamic queries, and I like the API of this package so far. I don't need multi-flavor support for this project as I'm only using PostgreSQL, and I do use PostgreSQL-specific features.
I hope you'll reconsider providing flavor-specific features that aren't supported by all databases. I understand the desire to support all databases with every feature, but it would be nice for those of us using only one database. If a feature is documented as flavor-specific, or if the developer knows the database they're using doesn't support the feature, they can just avoid using it.

@NCRonB I have carefully considered your suggestion, but I still feel that the responsibility for implementing flavor-specific operators should be left to the caller to implement themselves, while this library only implements the various features mentioned in the SQL standard and de facto standards. The main reason for this consideration is that there are too many variations in SQL syntax. Once a flavor is accommodated, it may lead to an increasing number of APIs being added to support various syntaxes, which could potentially clutter the API with flavor-specific APIs, inevitably increasing the cost of understanding and using this library. Since one of the goals of this library is to serve as an infrastructure for other ORM libraries, I prefer to expose interfaces that are easy to use for everyone to expand their own flavor-specific APIs, which is a direction I am very willing to pursue.
Therefore, you might consider what I can help you with to more easily implement the operators you need. Feel free to raise such requirements at any time, and I will prioritize supporting them.
By the way, perhaps we should open a new issue to discuss this topic. Could you summarize your requirements and start a new issue?

I appreciate your consideration. Having to implement flavor-specific operators and features myself kind of defeats the purpose of using the builder. I don't consider supporting flavor-specific features clutter, but I understand if you want to limit the scope of this module. My requirements are beyond this scope, so it may not be the best fit for my current project.

For PostgreSQL specific features, package pg may be the best for you. You are free to choose the appropriate library at your discretion. It's perfectly normal if this library is not suitable; finding one that meets your needs is what matters most.


Joejiong

进入Docker后进入paddle目录下：
`cd paddle`	-->	`cd ../paddle`

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.wujionghao seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Create separate fields for image extent and data footprint

The sentinel 2 importer includes the data bounds for a tile, but not the image bounds. We need the image bounds in order to place the thumbnail on a map for preview.

We need the "Image" extent for thumbnails to be able to place them on the map. The "Data" footprint represents where data exists.
The task here is twofold:

add fields and change field names
update importers to supply them

Duplicate of new issue #682


Introduce plug-ins

The optimization process of Rheem is currently affected by means of Platforms. Those can be registered with a RheemContext and all their ExecutionOperators etc. will then be considered when executing a RheemPlan. However, this abstraction is not really appropriate as there is a different between platforms and "plug-ins". A platform provides an Executor to have specific ExecutionOperators executed. However, neither the ExecutionOperators nor the mappings from Operators to ExecutionOperators nor the ChannelConversions are necessarily described (exhaustively) by a platform.
To this end, we should introduce plug-ins decouple execution platforms from the above mentioned components in Rheem. A plug-in provides Mappings, ChannelConversions, ExecutionOperators, as well as Configuration properties and requires Platforms. Also, each platform should ship with a "basic" plugin.

This issue was moved to rheem-ecosystem/rheem#16


cannot create certificate for subdomain

I successfully created a certificate for my domain "www.domain.tld" and it works fine!
Thanks for the great plugin, guys! :+1:
However, when trying to generate a certificate for a subdomain "sub.domain.tld" the following error is reported:
Error: Let's Encrypt SSL certificate installation failed: Failed letsencrypt execution: 2015-12-21 00:43:29,192:
WARNING:letsencrypt.cli:Root (sudo) is required to run most of letsencrypt functionality.
Failed authorization procedure.
sub.domain.tld (http-01): urn:acme:error:unauthorized :: The client lacks sufficient authorization ::
Invalid response from http://sub.domain.tld/.well-known/acme-challenge/A83ZrA2sR--5FpXibefwSrSoUrlZBe78ADda1MgxiFI [xxx.xxx.xxx.xxx]: 401
IMPORTANT NOTES:
- The following 'urn:acme:error:unauthorized' errors were reported by
the server:

Domains: sub.domain.tld
Error: The client lacks sufficient authorization

Why is this?
The subdomain root is htaccess password protected - can this cause problems?

sorry, seems to be a duplicate of #16 and #13


Update to kirkstone

This means updating the underlying BSP layers to a newer BSP version which will also bump up the kernel versions for the supported boards.

Done via https://github.com/balena-os/balena-fsl-arm/pull/638


feat(proofs): delete associated didcomm messages

Resolves #860

Codecov Report

Merging #1021 (95bdaa8) into 0.3.0-pre (f38ac05) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##           0.3.0-pre    #1021      +/-   ##
=============================================
- Coverage      87.87%   87.82%   -0.05%
=============================================
  Files            615      615
  Lines          14764    14772       +8
  Branches        2498     2500       +2
=============================================
  Hits           12974    12974
- Misses          1693     1700       +7
- Partials          97       98       +1

Impacted Files
Coverage Δ

packages/core/src/modules/proofs/ProofService.ts
70.96% <0.00%> (-20.70%)
:arrow_down:

packages/core/src/modules/proofs/ProofsApi.ts
85.05% <0.00%> (-0.50%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


As a User I should be able to Transition the state of a WorkItem  from e.g. Open to Close

State Transition workflow for WorkItem defined WorkItem Type
Described in Work Item Type

That can be done today!


Removed some unused variables

This PR is to address issue #489 - removed some unused variable warnings in VIC code, including:

soil_con in initialize_soil()
dmy in vic_finalize()
dmy_current in vic_store

This looks good.  Thanks @yixinmao.


KAFKA-16939: Revisit ConfigCommandIntegrationTest

https://issues.apache.org/jira/browse/KAFKA-16939
some test secnario is wrong, so fix it

@m1a2st please rebase code to see QA again

@chia7712, Thanks for your comments, I already pushed the code to trigger QA


Invalid response id after closing file in blocking context

With https://github.com/openssh-rust/openssh-sftp-client/pull/98, the problem fixed by https://github.com/openssh-rust/openssh-sftp-client/pull/77 will exist. It occurs in action https://github.com/apache/incubator-opendal/actions/runs/5798363627/job/15715936917?pr=2815.
Maybe we need to find a better solution.

There are several ways to fix this:

Simply issue a warning if we receive an unknown id, this is the most simple solution.
Store a to the runtime tokio::runtime::Handle in Auxiliary

I prefer the second way. Because a warning about unknown IDs may confuse the user and make debugging more difficult.
If you agree with that, I'll become work on it.

If you agree with that, I'll become work on it.

I do think it is more elegant way, with the Handle in Auxiliary we can also spawn new task unconditionally in {TokioCompatFile, ReadDir}::Drop #96


BPS in unknown state after programming

After programming the BPS leader board, the peripherals on the STM32F413 are not reset upon the software reset. This means, for instance, that if the BPS had the contactor enabled and is then flashed with a program that does not interface with the contactor (a test program perhaps), then the contactor will remain enabled. This also has potential effects for when the WDOG timer resets the chip (although this has not been tested yet).
The question is this: Is there a way to guarantee that after flashing or soft-resetting the board, all the peripherals are set to known and safe states?

I should note that pressing the reset button on the board completely resets all peripherals.

Can't we just solve this by:

On startup, (in fault state or critical state threads) call BSP_Contactor_Off()
In our standard operating procedures, require that the BPS is always reset after flashing new code


Cross-compile patch & build instructions

Description
Changes to enable cross-compilation with msvc-wine.
How Has This Been Tested?
Follow the build instructions, which can be found in README.md included in this pull request.
Checklist

[x] I have made corresponding changes to the documentation.

There were some last minute fixes.
This should be ready to be tested now.

Since you got a lot of if there, if it runs in wine, I guess windows shouldn't be affected at all right?
However, I've successfully compiled the branch in Windows 11.

Since you got a lot of if there, if it runs in wine, I guess windows shouldn't be affected at all right? However, I've successfully compiled the branch in Windows 11 Pro 23H2.

Yeah, for some reason checking against the default value of the option doesn't work so now we're checking if the option isn't msvc-wine to detect non-cross builds.
CI is passing, and msvc-wine works for me on Arch Linux, so this should be good to go, though it would be good if someone else could test this on Linux just to make sure it's working on more machines than mine.


defineComponent is not defined when a component is used from a module via @nuxt/kit

Environment

Operating System: Darwin
Node Version:     v16.14.2
Nuxt Version:     3.0.0-rc.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      vite, typescript, runtimeConfig, meta, css, tailwindcss, components, target, ssr, loadingIndicator, modules, postcss, build, router, server
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
This is not easily reproducible on a StackBlitz but the module in question below is publicly available to test at @michaelpumo/components-nuxt.
Describe the bug
I have a Nuxt 3 app running and any component is able to use defineComponent without it having to be imported (it's global).
Some components, we wished to live in NPM, so I created a Nuxt module in a new repository like so (nuxt.js file in root) with the intent to import it into the main site:
import { join } from 'pathe'
import { defineNuxtModule } from '@nuxt/kit'

export default defineNuxtModule({
  hooks: {
    'components:dirs'(dirs) {
      dirs.push({
        path: join(__dirname, 'src/components'),
        prefix: 'greggs'
      })
    }
  }
})

In the consuming site, I am using this module via nuxt.config.ts like so:
modules: [
  '@michaelpumo/components-nuxt/nuxt'
]

In my pages, I am using the component like <GreggsContentMedia /> etc. This component is picked up correctly in the site but throws an error regarding globals like defineComponent. "defineComponent is not defined".

It seems odd that globals like defineComponent that are able to be used in the main consuming site are not available in components imported from a module? These exact components unchanged will run just fine if used in the main site and not via the module.
This includes other globals for Vue like reactive, ref etc.
Additional context
If I import like so in the library component:
import { defineComponent, reactive, computed } from '#imports'
It works. But ideally, I don't want to have to import globals like this. Is this simply a config concern and not a bug? I have tsconfig.json etc set up that imports from Nuxt.
Funny enough, the other components that this component imports do not appear to require an explicit import for themselves such as defineComponent, ref, computed etc. I wonder why that is? They are fine as-is.
Logs
No response

@michaelpumo i tried import { defineComponent } from '#imports'; but that module was not found -- any requirements to get that working?
https://github.com/nuxt/nuxt/issues/13996 had no solution, that I could understand.

@BananaAcid Into what file are you importing it?

@danielroe VSCode just showed an error on #imports (not being found - not typed?), but it actually worked.

https://nuxt.com/docs/guide/going-further/modules#quick-start
in module.ts, added addComponent(...)
/src/runtime/components/component.vue :

<script lang="ts">
// @ts-ignore because it works
import { defineComponent } from '#imports';

export default defineComponent({
  // ..
});
</script>

npm run dev

got it working:

module: https://github.com/BananaAcid/nuxt-dropzone-filelist


Tidy some dead issue links

I've done two things:

validate issue urls, but only if we don't have an issue count to avoid false-negatives caused by 429s
remove double slashes in urls

This should sort out #986 and #972, but the fix may not show until the cache is refreshed. I'll let the dead link CI close them.

There's a limited amount to be learned from the preview because of caching, so I will merge and monitor.


ENH pass allow_maxshield to Raw when loading digpoints

Allow the use of Neuromag raw data files with IAS turned on,
i.e., don't require first Maxfiltering the data.
I haven't come up with a scenario in which this would cause a problem, but perhaps others can?

I don't think we want to enable this by default. All other raw readers throw an error, and require the user to explicitly pass allow_maxshield=True at some point to read the data. This is a safety check to ensure that users really know what they are getting themselves into. Could we instead add an allow_maxshield property or kwarg somewhere that users could set (defaulting to False)?

Fair enough, you want to keep consistency despite the fact that in this particular case only head points are loaded from the raw file. Given that this is a GUI-feature, I guess more than a kwarg is needed, specifically a tick box to enable IAS-reads. Or would you consider a popup-warning sufficient? As it is now, the traceback when trying to load a raw with IAS is cryptic for anyone not familiar with FIFF internals. I'd be happy to revise according to your preferences on this?

Or perhaps this is superfluous, if the use of non-MF'd IAS-on data should just be forbidden! Then the ENH could just be to clean up the Exception...?

Ohhh, it's only for head / dig points. Never mind then, that's fine -- sorry for the noise. LGTM.

@agramfort or @christianbrodbeck feel free to have a look and merge if you're also happy

Don't know what max shield is but if that is something you'd always want included then that modification looks good :)

FYI it's an active shielding component of Neuromag systems. Basically there are some sensors outside the helmet that detect external interference, and it induces a magnetic field to counteract it. Elekta thus cautions against using maxshield'ed data without processing with MaxFilter / SSS, because SSS should suppress all sources of external interference, whether from the MaxShield or otherwise. The risk in processing IAS data without SSS is thus that MaxShield could be introducing some fields, or not cancelling out enough external interference, in a way that you might think was brain activation when it isn't.
In any case, IAS and MaxFilter have no impact on head digtization coords, so we should be good here.

That makes sense, thanks @Eric89GXL for the explanation


bug: strict_env_var_mode does not raises exceptions within config blocks

As of now, this works in env blocks but not within config.
This likely will just involve wrapping an additional utils.expand_env_vars call in the feature flag context.
This is a follow-up from https://github.com/meltano/squared/pull/312#issuecomment-1191828551
FYI @pnadolny13

Thanks, @cjohnhanson for logging this and for digging in. Do you mind picking this up next week? Feel free to update the weight as needed. Thanks!

Related but orthogonal:

#3171


Copyright of assembly

Thank you creating this fantastic Workflow open source project!
I can't find the released package, so I build it by myself. But there aren't assembly info in file properties, will you add assembly info in future?

In addition, can I add it by myself and what content is allowed?

Released versions of the packages can be found on nuget:

https://www.nuget.org/packages/UiPath.Workflow
https://www.nuget.org/packages/UiPath.Workflow.Runtime

Hi @marius-bughium, thank you for your link, I have download the nuget packages, but these dlls have no digital signatures and no copyrights. Can you distribute packages with digital signatures and copyrights?


Basic concurrent failure tests for new recovery protocol

Create a suite of basic concurrent failure tests for testing the new recovery protocol.
 Start Y recovery right away, so there are two partially overlapping recovery groups.
[ ] ?

cc @jtfmumm @rachelblasucci

Note that more complex tests, such as ones including autoscale events, will be addressed in a separate issue once this one is complete.

Closing this as it's been delivered and will be merged with @jtfmumm 's work in snapshot-resilience


Add Ænix to the cilium users

Signed-off-by: Andrei Kvapil<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[x] Are you a user of Cilium? Please add yourself to the Users doc
[x] Thanks for contributing!

Fixes: #issue-number
Add Ænix to the cilium users

/test


Fix syntax errors

This fixes some unclosed braces to make the example code valid.

This should probably be fixed on the 6.5 branch.

Sent this to 6.5 branch, thanks!


add check_domain_ping function and elif

In some cases, in particular when using PIA as a VPN, the connection will persist and appear to be active but routing stops at the gateway.
This appears to be a "feature" of PIA to help protect anonymity.
the gateway_ping function is not sufficient in determining if traffic can be routed past the gateway since the gateway does in fact return a response.
By reaching out and pinging a domain, we can see if traffic is able to be routed through the gateway.

I've also been running into established-but-not-routed connections with our OpenVPN setup, so this looks like a good improvement.

Just wanted to follow-up and say that I've been running this for a couple weeks on a DS620slim and a half dozen DS118's and it's working perfectly. We were having occasional VPN stall-outs previously and haven't had a single stall since implementing the gateway_ping method.

Just wanted to follow-up and say that I've been running this for a couple weeks on a DS620slim and a half dozen DS118's and it's working perfectly. We were having occasional VPN stall-outs previously and haven't had a single stall since implementing the gateway_ping method.

@beornlake do you mean you are having good results with the domain_ping method that is introduced with this Merge Request? or the existing gateway_ping method?

I do in fact mean domain_ping, good catch! (I updated the original comment).

Can we get this merged into the master branch? I believe this can make a difference within the project :)

@peledies, thank you for contributing! I just made some modifications (see change notes in https://github.com/ianharrier/synology-scripts/commit/b5029af3744acb9f288b488d6f963fc9dec8f9de) to extend the functionality and condense the code a bit.
One question: was there a specific reason you did an nslookup on the hostname, rather than just pinging the hostname itself?
If you're happy with the modifications I've made, I'll go ahead and merge this into the master branch. Please also make sure your name is correct in the header area of the script. Thanks again!


Training without GPU

Good morning,
unfortunately my pc do not have an Nvidia GPU, so I have this error:
Found no NVIDIA driver on your system. Please check that you have an NVIDIA GPU and installed a driver from http://www.nvidia.com/Download/index.aspx
can I run the training on the CPU?

It seems impossible since you'll see something in the training.md
"Finetune with a single GPU:"

Training deep models on CPU is very very slow. I think it is almost impossible.
You can try some cloud GPU platforms, such as Colab.

@xinntao how to reduce the finetuning time on colab it shows 12days eta ?


[fix] Cms::GroupPermission #1223

Coverage remained the same at 85.591% when pulling 01b7fd11794cfd22403b0778b0837f879671f6f6 on itowtips:fix into 6d54850fb90debd07cfe1a2f467541f58a4f5be3 on shirasagi:master.


Inconsistent title/logo

Love the improvements to the site, but I found an inconsistency:
In /single.html and /baseof.html pages, the logo section displays the site name. But in /li.html pages, .Title is displayed, and it's a link, and a relatively large font size < h2 >, vs < p >.

Seeing this too - have you worked out why, so we can submit a PR.

Yeah, no, not a clue. I'm a tech writer, so I know enough css to be dangerous.

I'm seeing this as well. I'll try to figure out what's causing it. Great template other than this little bug.

I was able to reproduce this by having a blank _index.md page in the directory structure. Adding anything below the +++ makes the problem go away,
+++
title = "Codenesium Documentation"
description = ""
+++

Is the only fix presently just swapping out to  ?


Stripe connect does not work

Describe the bug
"Error connecting Stripe:"{message: "Failed to execute 'json' on 'Response': Unexpected end of JSON input"}
This error appears when I click in the "Connect with Stripe" button created by bolt.new.
I want to let users to connect their stripe account to the webb app.
It has been impossible to fix this issue using the chat.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-ozvykh
Steps to reproduce
Sign up with email and password, and then you will see the Stripe Button.
Expected behavior
Once you click in the Stripe button, you should be redirected to the Stripe connected account onboarding process.
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 123948186,
  "usedJSHeapSize": 119833994,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = kisull
Chat ID = 70f772838abf

Additional context
No response

i got same issue with Stripe Checkout probably comes from API calls cannot be used from webcontainer


[PE-D][Tester D] Help command not functional

typing help and pressing enter does not submit the command or display any feedback
typing help add or any other instruction in the help [command] format returns invalid command format, without instructions of how to use the command
parameter description also seems to be incorrect, returning  [command] (must be a valid command) help [command]

Labels: severity.Low type.FunctionalityBug
original: juzzztinsoong/ped#2

Help seems to work in the Java Code but not in the build. Thank you for pointing it out!


not ok 1 - Device should be in production mode

jenkins-leviathan-v2-template-2960-client-1  | [2022-07-16T02:02:47.507Z][worker-cloud]     # Subtest: SSH authentication test
jenkins-leviathan-v2-template-2960-client-1  | [2022-07-16T02:02:47.507Z][worker-cloud]         # Subtest: SSH authentication in production mode
jenkins-leviathan-v2-template-2960-client-1  |             # Setting production mode in config.json...
jenkins-leviathan-v2-template-2960-client-1  | [2022-07-16T02:03:18.692Z][worker-cloud]             not ok 1 - Device should be in production mode
jenkins-leviathan-v2-template-2960-client-1  | [2022-07-16T02:03:18.692Z][worker-cloud]               ---
jenkins-leviathan-v2-template-2960-client-1  | [2022-07-16T02:03:18.692Z][worker-cloud]               found: "error: host error:"
jenkins-leviathan-v2-template-2960-client-1  | [2022-07-16T02:03:18.692Z][worker-cloud]               wanted: "false"

https://jenkins.product-os.io/job/leviathan-v2-template/2960/consoleFull
consoleText.txt

I've seen failures around this test a few times now - where a change to the config.json seemingly isn't being respected.
We had this attempt at fixing it I think last week: https://github.com/balena-os/meta-balena/commit/916d400456ef2cb58f13c7b8b6e542c23b57e8c2 but it might not have been enough

I don't think we are waiting long enough between these tests, since config changes on slower devices could take a while to populate. Or, as we've seen before, we aren't waiting for config-json.service to be inactive before making config changes.

In this case, we are expecting the change to be instant
				}).then(() => {
					return this.cloud.executeCommandInHostOS(
						`jq -r '.developmentMode' /mnt/boot/config.json`,
						this.balena.uuid
					)
				}).then((actual) => {
					const expected = 'false';
					test.equal(actual, expected, 'Device should be in production mode ');

I'm going to rework this test a bit


Remove type resolution by singular version of the name

Pull Request Test Coverage Report for Build 175216618

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.2%) to 24.314%

Totals

Change from base Build 175112339:
-3.2%

Covered Lines:
638

Relevant Lines:
2624

💛  - Coveralls


Cannot setup EI using Vagrant configuration provided by WSO2.

Description:
Cannot setup EI using Vagrant configuration provided by WSO2.
I am following the below link:
https://wso2.com/integration/install/vagrant/get-started/
I followed all the steps exactly, as given in the tutorial, but I am getting the following error when running 'vagrant up':
`Error message:
There was an error while executing VBoxManage, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.
Command: ["hostonlyif", "create"]
Stderr: 0%...
Progress state: E_FAIL
VBoxManage.exe: error: Failed to create the host-only adapter
VBoxManage.exe: error: Querying NetCfgInstanceId failed (0x00000002)
VBoxManage.exe: error: Details: code E_FAIL (0x80004005), component HostNetworkInterfaceWrap, interface IHostNetworkInterface
VBoxManage.exe: error: Context: "enum RTEXITCODE __cdecl handleCreate(struct HandlerArg *)" at line 94 of file VBoxManageHostonly.cpp`
Suggested Labels:

WSO2 EI Vagrant, VirtualBox WSO2, Vagrant WSO2
Suggested Assignees:

Affected Product Version:
Virtual Box Version 5.2.18
Vagrant Version 2.12
WSO2 EI 6.3 from the below link is attempted to be installed:
https://wso2.com/integration/install/vagrant/get-started/
OS, DB, other environment details and versions:
Windows 10
Steps to reproduce:
Follow the steps in the link https://wso2.com/integration/install/vagrant/get-started/ verbatim, with the versions of VB and Vagrant on Windows 10.
Related Issues:

@tabishshamim10 This is a VirtualBox issue they introduced with VB5. Please go through the following issue in hashicorp/vagrant.

https://github.com/hashicorp/vagrant/issues/6068

Answered by @VimukthiPerera, hence closing.


Set up automatic deploy for dev environment

Set up automatic deploy for dev environment
Document process in internal documentation

Changed the estimate


Failed to compile "fatal error: dxgi1_4.h: No such file or directory  #include <dxgi1_4.h>"

$ ./package-release.sh master ./build --no-package
The Meson build system
Version: 0.45.1
Source dir: /home/rachael/d9vk
Build dir: /home/rachael/d9vk/build/dxvk-master/build.64
Build type: cross build
Project name: dxvk
Native C compiler: cc (gcc 7.3.0 "cc (Ubuntu 7.3.0-27ubuntu1~18.04) 7.3.0")
Cross C compiler: x86_64-w64-mingw32-gcc (gcc 7.3.0)
Native C++ compiler: c++ (gcc 7.3.0 "c++ (Ubuntu 7.3.0-27ubuntu1~18.04) 7.3.0")
Cross C++ compiler: x86_64-w64-mingw32-g++ (gcc 7.3.0)
Host machine cpu family: x86_64
Host machine cpu: x86_64
Target machine cpu family: x86_64
Target machine cpu: x86_64
Build machine cpu family: x86_64
Build machine cpu: x86_64
Checking if "winelib check" compiles: NO
Program x86_64-w64-mingw32-windres found: YES (/usr/bin/x86_64-w64-mingw32-windres)
Library vulkan-1 found: YES
Library d3d9 found: YES
Library d3d11 found: YES
Library dxgi found: YES
Library d3dcompiler_43 found: YES
Library d3dcompiler_47 found: YES
Program glslangValidator found: YES (/usr/bin/glslangValidator)
Dependency threads found: YES
Dependency threads found: YES (cached)
Build targets in project: 13
Found ninja-1.8.2 at /usr/bin/ninja
[3/242] Generating version.h with a custom command.
fatal: No names found, cannot describe anything.
[4/242] Compiling C++ object<EMAIL_ADDRESS>FAILED<EMAIL_ADDRESS>x86_64-w64-mingw32-g++  -Isrc/util/util@sta -Isrc/util -I../../../src/util -I../../.././include -fdiagnostics-color=always -pipe -Wall -Winvalid-pch -Wnon-virtual-dtor -std=c++1z -O3 -DNOMINMAX -MD -MQ<EMAIL_ADDRESS>-MF<EMAIL_ADDRESS>-o<EMAIL_ADDRESS>-c ../../../src/util/com/com_guid.cpp
In file included from ../../../src/util/com/../../d3d11/d3d11_include.h:3:0,
                 from ../../../src/util/com/../../d3d11/d3d11_interfaces.h:3,
                 from ../../../src/util/com/com_guid.cpp:3:
../../../src/util/com/../../d3d11/../dxgi/dxgi_include.h:28:10: fatal error: dxgi1_4.h: No such file or directory
 #include <dxgi1_4.h>
          ^~~~~~~~~~~
compilation terminated.
[9/242] Compiling C++ object<EMAIL_ADDRESS>ninja: build stopped: subcommand failed.

You having outdated MinGW headers is not my issue.

Your meson is also massively out of date.

Thanks for the heads up, I found MinGW 6 backport for Ubuntu 18.04 here in case this affects anyone else:
mingw-w64 6.0 backport for Ubuntu
Old meson did not block building.

Alternatively, you can use a distro where package maintainers do their job and maintain packages instead of falsely labelling anything 2+ years old stable even if it has major issues fixed in later releases.


MultiMeshInstance3D: Changes in the transform of instances stay after game is closed

Godot version
v4.0.beta4.official [e6751549c]
System information
OpenSUSE Tumbleweed 20221102, Vulkan
Issue description
Changes made in a MultiMesh instances by code stay after the game is exited. When the game is started again, it maintains the changes made in the previous run (when referencing the instances last position).
Steps to reproduce

Create a MultiMeshInstance3D node and populate a surface.

Add a script to the MultiMeshInstance3D node and alter its instances transform.

Example:
func _ready():
    for index in range(multimesh.get_instance_count()):
        # Get the position
        var pos: Vector3 = multimesh.get_instance_transform(index).origin
        # Elevate it by 1 based on instance's own position
        pos.y += 1.0
        # Set the position
        multimesh.set_instance_transform(index, Transform3D(Basis(), pos))

After each run, the instances position get higher and higher.

Minimal reproduction project
multimesh_bug.zip

Why do you expect the transform to reset? The equivalent code for any other Node, be it 2D or 3D, would have the same result.

It looks like the runtime code is changing the MultiMesh data. I just tried the equivalent code with Node3D and Node2D and it worked as expected.
In the case of MultiMesh, the alterations are kept even after restarting the engine.

I can't reproduce this issue on Beta13 the instances are in the same place every time I run the scene.


ENH: special: add ufuncs for amos

This PR adds ufuncs for the amos functions, using the new-style machinery.
cc @steppi

Thanks @izaid! If it's not too tough, can you incorporate the changes I made to amos.h  and amos/amos.h in the cephes PR:  https://github.com/scipy/scipy/pull/20390/files#diff-856882a37611b3aee9222818d9827a1c3f7b8a34cd0ec8aac40b61f3a7acd3f7).
Key points are

Using set_error instead of sf_error in amos.h
Adding #pragma once to amos/amos.h
Making all of the functions in these two headers inline
Using an amos namespace in amos/amos.h instead of prefixing all functions with amos_.
(I also made all of the static const arrays constexpr but I'm not sure how useful that actually is, so it's a judgement call if that's actually a good idea).

This way I won't have to worry about having tricky merge conflicts to resolve, and can just base things on your version of these files directly in the cephes PR.

I have also dealt with https://github.com/scipy/scipy/issues/20385 in this PR.

Thanks @izaid! If it's not too tough, can you incorporate the changes I made to amos.h and amos/amos.h in the cephes PR: https://github.com/scipy/scipy/pull/20390/files#diff-856882a37611b3aee9222818d9827a1c3f7b8a34cd0ec8aac40b61f3a7acd3f7).
Key points are
1. Using `set_error` instead of `sf_error` in `amos.h`

2. Adding #pragma once to `amos/amos.h`

3. Making all of the functions in these two headers inline

4. Using an `amos` namespace in `amos/amos.h` instead of prefixing all functions with `amos_`.

5. (I also made all of the `static const` arrays `constexpr` but I'm not sure how useful that actually is, so it's a judgement call if that's actually a good idea).

This way I won't have to worry about having tricky merge conflicts to resolve, and can just base things on your version of these files directly in the cephes PR.

@steppi Done! Let me know if there's something else that would help, otherwise ready for review.


Bug: Hard coding of backend server URL causing failure of many requests

Issue Template
Description
The backend server URL is hard-coded in the application, leading to failures in various requests when the server address changes
Steps to Reproduce
Please provide step-by-step instructions on how to reproduce the issue:

Open the application in a development or staging environment.
Attempt to perform any action that requires a backend request (e.g., fetching notes, saving a note).
Observe the error message indicating that the server cannot be reached.

Expected Behavior
The application should dynamically use the appropriate backend server URL based on the environment, allowing successful requests without hard-coded values.
Actual Behavior
Requests fail due to the hard-coded backend server URL, resulting in error messages and an inability to interact with the backend.
Screenshots

Additional Context
Add any other context or information about the issue here.

Assigned @mehul-m-prajapati


Left Join Bug ?

Hi!
Running the following code will generate a SQL with an INNER JOIN between A and Group, resulting in no A records being returned (if A has no group).
public class Group
{
   public int Id { get; set; }
   public List<A> A { get; set; }
 }

 public class A
{
   public int Id { get; set; }
   public string Name { get; set; }
   public int? GroupId { get; set; }
   public Group Group { get; set; }
}

 var items = _dbContext.A.Where(t => t.Id == 1).AsQueryable();

var query = items.Select(i => new Models.MyModel
{
  Id = i.Id,
  Name = i.Name,
  Groupname = i.Group != null ? i.Group.Name : string.Empty
});

 var result = query.ToList();

Is this by design or a bug?

Using from and DefaultIfEmpty() still produces a INNER JOIN

Duplicate of https://github.com/aspnet/EntityFramework/issues/3186


implement Error trait for error type

minilzo does not work well with error handling libraries like error-chain because its error type does not implement std::error::Error trait. Add Error impl to fix the problem.

Thanks! Are you actually using minilzo-rs somewhere?

Oh cool! Keep up the work then.


php81Extensions.mongodb: 1.16.2 -> 1.17.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for php81Extensions.mongodb is: The Official MongoDB PHP driver
meta.homepage for php81Extensions.mongodb is: https://github.com/mongodb/mongo-php-driver
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.17.1 with grep in /nix/store/qx9aksw87sh752s35lpwq15kl01krha8-php-mongodb-1.17.1
found 1.17.1 in filename of file in /nix/store/qx9aksw87sh752s35lpwq15kl01krha8-php-mongodb-1.17.1

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
php81Extensions.mongodb
php82Extensions.mongodb
php83Extensions.mongodb

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qx9aksw87sh752s35lpwq15kl01krha8-php-mongodb-1.17.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A php81Extensions.mongodb https://github.com/r-ryantm/nixpkgs/archive/ca56a0908f5b19ff925e387506ec301edd343264.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ca56a0908f5b19ff925e387506ec301edd343264#php81Extensions.mongodb

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qx9aksw87sh752s35lpwq15kl01krha8-php-mongodb-1.17.1
ls -la /nix/store/qx9aksw87sh752s35lpwq15kl01krha8-php-mongodb-1.17.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

php81Extensions.mongodb
php82Extensions.mongodb
php83Extensions.mongodb

Maintainer pings
cc @aanderse @drupol @etu @Ma27 @talyz for testing.

@GrahamcOfBorg build php81Extensions.mongodb


add the 08-pjit.ipynb

the ipynb of 08-pjit is missing, while other ipynb files.

hi @jakevdp  Thanks a lot for your reply.
do you mean we should remove this TODO after adding back the 08-pjit.ipynb ?
I do not understand why other files have notebook and 08-pjit does not have it.  Any special reason we break this design ?
Thanks

I'm not sure why the notebook was not added originally, I suspect there was some issue with the doc build.
Regarding the TODO: the issue in the TODO explains the cause of the readthedocs failure, which comes from the fact that 08-pjit.md is not included in that list.


How to play private video, need help?

Hello @Deepsonstha, did you mean you want to play private vimeo video using id?

similar to #15 and #27

Closing this issue for now, Thanks.

@Deepsonstha I recommend checking out this link for more details on playing private videos.


Cultivating Ench Lore Update

since cultivating can be added to axes i added it to the .json

now perfect title


feat: ℕ and ℚ≥0 are star-ordered rings

Also golf the existing instance for ℚ and rename Rat.num_nonneg_iff_zero_le to Rat.num_nonneg, Rat.num_pos_iff_pos to Rat.num_pos.
From LeanAPAP

Can you update the PR title and description, in light of the existing instances?

Done

ℤ is now included and happy

Please add it to the PR title

Thanks!
bors merge


Feature request: Add button to bookmark am article to the detail page

When reading an article, there's no way to book mark it. Could a button be added to the top, next to the open in browser/share/font size buttons?
Currently to bookmark after reading an article I have to navigate back to the list.

You're right, that would be handy to have a bookmark button in the reader mode.
Thank you for the suggestion, I will put it in the roadmap.


Unofficial Facebook page under Creative common

Thanks for reporting an issue, to help us file this efficiently we made this template. Feel free to ignore the template if you wish :)
If this is a bug or a problem with one of our websites or page on one of our sites, please put the URL or page address here:
Please tell us the steps to reproduce the problem, if needed:

I am from the marketing team of Cafe Bahar & Restaurant - since 1973, popular in Hyderabad and now we are adapting to new digital platforms like social media.

So now the issue is that the Facebook page is this -   https://www.facebook.com/pages/Cafe-Bahar/194356810633300 - Unofficial page - This unofficial Page was created because Wikipedia Content from the Wikipedia article Cafe Bahar (contributors) licensed under CC-BY-SA. Freebase Content from Freebase licensed under CC-BY.

So this licenced links redirects to creative common. We have even contacted facebook about this issue but could not get help. Anyone who searched for our business is redirected to this unofficial page for likes, page timeline and check-in's rather than our official page i.e,

Request you to kindly help in problem as you can update the content adn verify the above mentioned official page. Kindly let us know asap

Please tell us a little bit about how you're accessing the site or page, such as which web browser and which version of the browser:
(You can also vist https://aboutmybrowser.com/ which will give you a link you can share with us that'll tell us everything we need to know)
You can also attach a screenshot to this issue if you have a visual bug to report. That'll help us figure it out faster. http://www.take-a-screenshot.org/ has instructions on taking screenshots.
Finally, we're going to put someone's name here, so they get alerted to the new issue and can assign it to the right person. @robmyers @little-wow

The Official business page - www.facebook.com/cafebaharhyd/

Did you contact them as seen at this page: https://www.facebook.com/pg/cafebaharhyd/about/?ref=page_internal
Moreover, this isn't something that Creative Commons can handle, can it?

Hi,
Am the admin for the current Official page of the business (
www.facebook.com/pg/cafebaharhyd
https://www.facebook.com/pg/cafebaharhyd/about/?ref=page_internal) and
when i had a long talk with facebook support team they said the unofficial
page is created by creative common/wikipedia so hence they also do not have
the access to this unofficial page nor we can claim it.
Firstly, we request you to update the correct details and facebook
link/website in creative commons. Please find the attached documents
proving that the unofficial page has been created and licensed by cc.
Thanks
On Sun, Feb 25, 2018 at 4:28 PM, gho87<EMAIL_ADDRESS>wrote:

Did you contact them as seen at this page: https://www.facebook.com/pg/
cafebaharhyd/about/?ref=page_internal
Moreover, this isn't something that Creative Commons can handle, can it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/creativecommons/creativecommons.org/issues/757#issuecomment-368300079,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjG4tQrOdgffsy4jqhGdnMwlldc75s1jks5tYTzLgaJpZM4SSPxI
.

--
Sweekar Bhushan

This Github is for bug corrections and technical issues, not content.  They can help you more.

Hi,
I hope you are able to see this information to contact GitHub for such
issue in the contact of creative commons.
However i have raised this issue everywhere possible but did not get any
response except yours. It would be great if you can help us or redirect us
to get this resolved.
Please find the screenshot.
Thanks
On Wed, Feb 28, 2018 at 2:49 AM, gho87<EMAIL_ADDRESS>wrote:

This Github is for bug corrections and technical issues, not content.  They can help
you more.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/creativecommons/creativecommons.org/issues/757#issuecomment-369029534,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjG4taKCB2YdGKgGSrHCmT7zUiQwCmAPks5tZHFCgaJpZM4SSPxI
.

--
Sweekar Bhushan

I am starting to believe that this is a copyright or trademark issue. As said before, there is nothing that Creative Commons can do about this.
If you have contacted that user responsible for the unofficial page, but that turns out unsuccessful, then you may want to report the unofficial page to Facebook for copyright or trademark infringements. Please see the below pages:

https://www.facebook.com/help/1250304721728220?helpref=search&sr=3&query=trademark
https://www.facebook.com/help/399224883474207/?helpref=hc_fnav

@gho87 is correct (thank you for your help @gho87 !), Creative Commons is not the author of these pages, just of the licenses, so @sweekarbhushan you will have to contact Facebook directly.

Yes! I did that initially before getting here! but facebook customer support has clearly mentioned that as visible in the unofficial page that it has been created & licensed by wiki/cc/freebase. From there i started my journey till here.


ci: bump sdk version

We should not bump SDK version for this driver. It is expected that the maximum supported OS version there is 5.1. For higher versions UIA2/Espresso must be used

oh, I see.
Emulators by the sdk version seemed unstable on Azure.
Then, probably we can disable some tests or relax the assertion like partial matching. Let me tweak as another pr

yes, this was the initial ida.
We could set some predefined env variable for e2e tests, for example CI=1 and then add if (process.env.CI) return this.skip() to flaky tests


Update theming.md with minor grammar change

Thanks for all your hard work and very useful information.