Model-level query sortings

It would be very handy if sortings could be defined model-based.
Fluent API:
modelBuilder.Entity<Post>().HasQuerySortOrderAsc(p => p.Title);
modelBuilder.Entity<Post>().HasQuerySortOrderAsc(p => p.Title).HasQuerySortOrderDesc(p => p.Content);
Data Annotations:
[QuerySortOrderAsc] public string Title { get; set; }
or
[QuerySortOrderAsc(1)] public string Title { get; set; }
[QuerySortOrderDesc(2)] public string Content { get; set; }
Using:
var posts = context.Posts.Where(p => p.Title.StartsWith("a").ToList();  //Without the OrderBy method, "model-level query sorting" should be used
or
var blogs = context.Blogs.Include(blog => blog.Posts).ToList();  //"model-level query sorting" should be used
or
context.Entry(blog).Collection(b => b.Posts).Load();  //"model-level query sorting" should be used
Thanks Marco

Note for triage:  I was not able to find this on the backlog, so putting it there.

duplicate of https://github.com/aspnet/EntityFrameworkCore/issues/7450 ?

Nice find, @smitpatel.  ;-)


Add Color::as_lcha

Problem
Other color spaces have a as_xxx with returns a Color and a as_xxx_f32 that returns a [f32; 4], the lch color space only has a as_lch_f32 that returns a [f32; 4].
Solution
Add a new method, as_lcha, for bevy_render::Color for converting any Color type into a Color::Lcha variant.
Note
Alongside this it would also make sense to rename as_lch_f32 to as_lcha_f32 as it also returns the alpha, not really big enough for its own issue.

Still free


'ArtifactoryTask' property 'artifactSpecs' doesn't have a configured value

How can we help?
I recently upgraded my gradle version from 6.5 to 8.1.1 , while running artifactoryPublish command getting the issue as,
A problem was found with the configuration of task ':artifactoryPublish' (type 'ArtifactoryTask').

Type 'org.jfrog.gradle.plugin.artifactory.task.ArtifactoryTask' property 'artifactSpecs' doesn't have a configured value.
Reason: This property isn't marked as optional and no value has been configured.
Possible solutions:

Assign a value to 'artifactSpecs'.
Mark property 'artifactSpecs' as optional.

It was working fine in gradle 6.5 with jfrog version 4.32.0.
I tried updating the jfrog version to 5.1.1, even after that issue persists.
Q1. Where do I need to specify artifactSpecs value? any example would help me to understand better.
Q2. How can I make it optional ?
JDK version 17
Compile SDK version 33

@balachandarg-tw
Thanks for using the Gradle Artifactory plugin.
Do you have somewhere 'artifactSpecs' configured? Would you be able to provide steps to reproduce?

@balachandarg-tw Did you solve this problem?
@yahavi I have the same problem in gradle 8.5 and jfrog plugin 5.2.0.
I add artifactory-block in top-level build.gradle.kts
buildscript {
    dependencies {
        classpath("org.jfrog.buildinfo:build-info-extractor-gradle:5.2.0")
    }
}

plugins {
    id("com.jfrog.artifactory") version "5.2.0" apply true
}

artifactory {
    setContextUrl("https://artifactory.url")
    publish {
        contextUrl = "https://artifactory. url"
        repository {
            username = "username"
            password = "password"
        }
        defaults {
            setPublishArtifacts(true)
            setPublishIvy(false)
        }
    }
}

And configure publish build.gradle.kts in feature module
plugins {
    `maven-publish`
    id("com.jfrog.artifactory")
}

configure<PublishingExtension> {
    val publication = maybeCreatePublication()
    publication.groupId = group.toString()
    publication.artifactId = name
    publication.version = version.toString()
    tasks.withType(Jar::class) {
        manifest.attributes["Implementation-Version"] = publication.version
    }

    tasks.withType<ArtifactoryTask>().configureEach {
        publications(publication.artifacts)
    }
}


Pr

 Quality gate: Failed
7 Issues

 7 Code issues

Details

 Quality gate: Failed
8 Issues

 7 Code issues
 1 Anti-patterns

Details


Implement C dynamic plug-in interface [WIP]

Based on the discussion and proposed (by @reduz) solution at #3936
I'm still trying to complete implementing all methods declared at original godot_c.h, but, as I'm not that familiar with the internal C++ API, I would like to ask for some guidance while doing it.

Oh wow

First, why are there less GODOT_IMAGE_FORMAT_* #defines's than in Image::Format enum ? Should I modify the first to match the second and allow easier conversion (cast) between int and Image::Format ?

Second, what is the correct way to create and set up a unique RID (for godot_rid_create() call) ? The default constructor initializes its internal data to NULL.

Okay, this might be a bit surprising...
I'm working on a module, DLScript.
The original proposal has some problems, I'll address that in a second.

DLScript
Having these functions isn't enough. You need a way to use the functions from external.
So basically what DLScript does is register a new scripting language that uses shared libraries as the source. This shared library makes use of the C API.
I already wrote a C++ wrapper for this C API.
In other words: what you did already is implemented, more mature and more complete.
I admit, I should've mentioned on the proposal that work is being done.
On the other hand your PR is incomplete and you should've come to the IRC to ask about this stuff instead of opening a merge request. You could've saved time because you would've known that someone's already working on this.
So I'm sorry for your wasted effort. But DLScript is already more mature and even feature complete by now. I'll open a PR in the next week when everything goes well.

The proposal uses void * to represent core types. You're using dynamic allocation for that. It's a bad choice because you want to be able to use many core types quickly  without having to worry about many allocations which take some time as well.
Feel free to check out DLScript or the resulting API I came up with. Sorry for not communicating my efforts on the original proposal.

After I finish implementing the API, how a C plug-in should register functions to be called from GDScript ? The only thing that comes to my mind is that plug-ins must implement a "const char *get_method_names()"-like function that return identifiers for all invokable methods, and all of them receives a list/array of godot_variant (Variant* cast to void*) values, to account for different number/type of arguments, and return a godot_variant as well.

Wow ! That's great to know, actually, as the main reason I was working on it was because I needed it.
Sorry for not following the correct procedure, I've worked with other projects where is fine to open a WIP PR.
I'll close this one, then. Thanks for your quick reply.

@Bitiquinho Yeah, the plan is to have it in 3.0. It's only a few QoL changes and then it'll be ready for a merge. reduz and many others are excited for this, so it's veeery likely to be in 3.0 😄
I'll link my work on the original proposal so it doesn't happen again.
WIP are not common here, I guess it would've been okay, but it's not the norm here 😄
Again, sorry for wasting your time.

Do not worry. I didn't take long and I learned something int the process.
It's really great to have it for next release. Thanks a lot for your work.


Possible to delete pending RecurringApplicationCharges?

@kevinhughes27 @maartenvg
how do you get rid of pending charges? I can't destroy or cancel them, it seems.
I, [2016-07-28T09:14:00.511980 #7785]  INFO -- : DELETE https://whydontyoujust.myshopify.com:443/admin/recurring_application_charges/2623688.json
I, [2016-07-28T09:14:00.512234 #7785]  INFO -- : --> 422 Unprocessable Entity 79 (379.2ms)
I, [2016-07-28T09:14:00.513206 #7785]  INFO -- : Headers: {"Accept"=>"application/json", "User-Agent"=>"ShopifyAPI/4.2.2 ActiveResource/4.1.0 Ruby/2.3.1", "X-Shopify-Access-Token"=>"0xDEADBEEF"}
I, [2016-07-28T09:14:00.513334 #7785]  INFO -- : Response:
{"errors":"Charges must have an 'active' status before they can be cancelled."}
ActiveResource::ResourceInvalid: Failed.  Response code = 422.  Response message = Unprocessable Entity.

Thanks for the issue. Pending charges can't be destroyed or cancelled. You can always create a new charge if the merchant decides to pick a different plan.


Update markdown-it and plugins to 4.0

Our Markdown renderer has updated to 4.0 and changed its internals. We should upgrade at some point, which will involve updating our plugins to match the new internals.

Submitted 2 PRs:
https://github.com/samchrisinger/markdown-it-toc/pull/3 @samchrisinger
https://github.com/brianjgeiger/markdown-it-video/pull/4 @brianjgeiger
Waiting for review/merging before continuing with integration.

@brianjgeiger @Ghalko  Are there breaking changes in the syntax that would affect OSF users?

@sloria There aren't any syntax changes to the markdown itself. I am sorry to have confused that. The syntax is within the module and the plug-ins, no API was changed/broken.

:+1:


Build doesn't work when spaces in atom path

If I had the atom folder in a path that included spaces such as
/Users/foouser/Open Source/atom

then script/build just outputs Installing build modules... and immediately quits. Once I put it in a path without spaces it worked fine. Not sure if this is something that can even be handled.

Build system was rewritten. Please reopen if you still get this problem.


on_block(b) and (store) progress

It seems that on_block(b) can be called many times with the same block, or with any block that is already in the store.
This results in the store not being modified and is a potential source of DoS attack.
This issue has been reported as issue 1891 in the Eth2.0 specs.

The discussion on this problem  issue 1891 has reached a consensus, so closing.


[scripts] Add scripts for architecture latency & fp16 conversion

This PR:

Add scripts for architecture benchmark & fp16 conversion

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.07%. Comparing base (aa9e96d) to head (579d9d5).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #38   +/-   ##
=======================================
  Coverage   98.07%   98.07%
=======================================
  Files          65       65
  Lines        1974     1974
=======================================
  Hits         1936     1936
  Misses         38       38

Flag
Coverage Δ

unittests
98.07% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Invalid partner key

hi, I need help.
I run application demo  digits-android and return in custom phone login that  java.lang.IllegalArgumentException: Invalid partner key
on BuildConfig.java
PARTNER_KEY = "null"

Hey! Our custom UI program is currently in closed beta. Please message me @chrisoryschak if you'd like to discuss being part of the beta. Thanks!

Hello,
I am interested about being part of your beta and get a valid Partner Key and I obviously can't contact you directly on GitHub as private messaging seems to be unavailable.
How can I contact you other way?
Thank you !

I'm also interested, is there a way to test this in an application we are currently developing?
Thanks in advance.


Additional log to indicate that database is compacting

When Besu is restarted after running for a few days the startup can appear to be stuck:
2023-04-18 08:42:06.806+00:00 | main | INFO  | RocksDBKeyValueStorageFactory | Existing database detected at /var/lib/besu. Version 2
2023-04-18 08:42:18.872+00:00 | main | INFO  | KeyPairUtil | Loaded public key 0x52e86b0e8c9cdd6579493a5371206946ef8442ee3b4b3144e1d1de1a8ed5ec6d41cfc977717b9358d1c403b915bbc9f8a1befabd71974a4f5ee7eda64a003a11 from /var/lib/besu/key

The example above shows a 12 seconds difference between the two log messages. In other occasions I've seen it taking longer than 30 seconds. Looking at Grafana it seems that the database is compacting during this time:

This PR adds this information to the log message to tell the user what is going on, so they don´t think that there is a problem.

@daniellehrner can you handle a merge by friday?


Document missing fields.

Clarify installerClass behaviour
As per discussion:

https://groups.google.com/forum/#!topic/jmeter-plugins/MyDFi8DcXw4

Find a PR that documents undocumented fields and clarifies others.
Regards
Philippe M.

Good, thanks


speed up bunch generation

Generating the bunch distribution natively within RadTrack can probably be much faster.

Fixed in commits 9cb7eb3ccf87d962092c84f7c9590976f6b754d1 and https://github.com/radiasoft/radtrack/commit/9d8d36f28a9f5d45f2a9093ad281b52f295351db.  It was plotting that was slow for too many particles.


fix: nginx config from 'sites-enabled' instead of 'sites-available'

Hi, can you provide more details please?

I'm not the creator of the PR but I think I can explain it to you just as well:
As you can see in your nginx-Dockerfile (https://github.com/maxpou/docker-symfony/blob/master/nginx/Dockerfile) your are doing it really well by symlinking the symony-config to sites-enabled and removing the default config from there.
The Problem from know is that you are including all files under "/etc/nginx/sites-available/*" instead of "sites-enabled".
Another suggestion would be to extend the include-statement by only including "*.conf" files from "sites-enabled" as far as I know this whould be the default behaviour.

In a more practical way it means thats if you can't match the server_name by adding the ip your host file, it will end up on the default config you did remove from /sites-enabled. But it's still present in a  /sites-available.
Run into a nginx config error and saw that.

Sorry maxpou, i havn't seen your comment. It's exatly what SebTM answered. Thanks

Great!
Thanks @SebTM @davidbonachera @stefankoerner


Add highlighting to logged dbt Cloud URIs

Add color to dbt Cloud URIs that are logged in any of the tasks or flows in prefect-dbt. Prefect 2.0 doesn't currently have the ability to add color to terminal logs or the logs displayed in the UI. Additional core development is needed before this item can be implemented.

Related to https://github.com/PrefectHQ/prefect/pull/6101, but haven't had the time to write tests.


Cross Validation

This would be HPCC scripts to run every week different Mar/IO tests

Would also need the following:
GUI to setup Tests
HPCC Script
Data Output
Data Visualization

TODO::
Should be links to images of code


pythonPackages.testfixtures: 6.2.0 -> 6.3.0

Motivation for this change
fixed build
Things done

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

[X] NixOS
[X] macOS
[ ] other Linux distributions

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

Iincluded in https://github.com/NixOS/nixpkgs/pull/49741. Thanks a lot!


PREST docker dial tcp <IP_ADDRESS>:54320: connection refused

I am following the document to install the prest, after running the command:
docker run -e PREST_HTTP_PORT=3000 -e PREST_PG_HOST=<IP_ADDRESS> -e PREST_PG_PORT=5432 -e PREST_PG_USER=postgres -e PREST_PG_PASS=123456 prest/prest
there is the error
9/04/20 14:14:18 [error] dial tcp <IP_ADDRESS>:54320: connect: connection refused
but i am using go get to install prest and run prest local can access the prest
How to resolve the problem?

any solutions to this problem?

any solutions to this problem?

prest_1     | Waiting for port postgres:5432 to become available...
prest_1     | Ready hosting postgres to port 5432 !
prest_1     | 2021/02/22 15:06:51 [warning] adapter is not set. Using the default (postgres)
prest_1     | 2021/02/22 15:06:51 [warning] You are running pREST in public mode.
prest_1     | [prest] listening on <IP_ADDRESS>:3000 and serving on /

yep, this issue got lost from my vision, sorry for not closing it earlier


Create GeoTIFF Datastore/Coveragestore without uploading a file?

Greetings. Is it possible to create a GeoTIFF Datastore/Coveragestore without having to upload a file?
Using a file that is already in the Geoserver?

Right now, the functionality is not available. I am open for any PR.

I think should apply for any fuction that involves uploading, allow the option to merely use a file that is already in geoserver  instead of uploading

@iamtekson  Is it possible to create a datastore/coveragestore based on a COG ?

Do you know if the Geoserver REST API allows for the creation of datastores/coveragestores that make use of the COG (Cloud optimized Geotiff) plugin?

I have no idea about this. But found this discussion in the StackOverflow. Seems like, we can integrate that too.

Hello, I am also open to work on the development of this feature.

Regarding this issue, I added the modifications mentioned on the discussion that @iamtekson mentioned to add the possibility to create coverage stores from remote cog files "But found this discussion in the StackOverflow.".
Unfortunately, I only have access to Google cloud files to test that it is working fine, is it possible that someone test the HTTP, Azure and S3 connexions for me?
This is the repo: https://github.com/Ponsoda/geoserver-rest-cog-coverage-store
Some considerations based on the plugin explanations:

Be sure that the file is in cog format (not all geotiffs are valid)
Every provider needs to have a different plugin installed (cog-http, cog-google, cog-azure and cog-aws)
To use Google, it is necessary:

Set the env variable GOOGLE_APPLICATION_CREDENTIALS with the path of the json file with the credentials
On geo.create_coveragestore add two extra parameters <remote_url> in substitution of path, with the url of the file (both https or gs schemes are valids) and, <cog_settings> with  the following value {'rangeReaderSettings':'GS'}

⚠⚠ Now starts the untested part. The rangeReaderSettings name and the credentials had been assumed by miself based on the documentation, as I couldn't find any information about it.

To use http, it is necessary:

On geo.create_coveragestore add two extra parameters <remote_url> in substitution of path, with the url of the file (both https or gs schemes are valids) and, <cog_settings> with  the following value {'rangeReaderSettings':HTTP', 'userName':, 'password':}

To use AWS, it is necessary:

On geo.create_coveragestore add two extra parameters <remote_url> in substitution of path, with the url of the file (both https or gs schemes are valids) and, <cog_settings> with  the following value {'rangeReaderSettings':'S3', 'accessKeyID':, 'secretAccessKey':}

To use Azure, it is necessary:

On geo.create_coveragestore add two extra parameters <remote_url> in substitution of path, with the url of the file (both https or gs schemes are valids) and, <cog_settings> with  the following value {'rangeReaderSettings':'Azure', 'accountKey':, 'accountKey':}


Column shape based on the y-value

Hello!
I have a small question. Is is possible to render a column depending on the y-value?
Basically, I would like that if the value is positive, only the top corners are round, but if it is negative the bottom corners are the ones that should be negative.
Any idea if that would be possible with this awesome library? :)
If yes, could you point me to the right direction please? I don't see a way to do it...

Hi @Gowsky! Ok, I think I got what you mean with your suggestion. Will try to implement that.
Thanks a lot!
I will close this issue since it seems it is not in the plan to be implemented.


issues for 8/15

@ewalke31

[ ] code univariate kalman filter and smoother
[ ] test against previous value predictor
[ ] create r markdown for presentation
[ ] use existing code where possible
[ ] read style guide https://github.com/Upward-Spiral-Science/fngs/blob/master/status/organization.Rmd

@ebridge2

[ ] run/analyze a bunch of datasets with new pipeline
[ ] explore EM algorithm and use Gui's notes
[ ] use M step with our pi vector for finding the covariance matrices of our sources
[ ]  compare covariance matrices of our sources with old method for tfMRI
[ ] fMRI dataset wise QC repo
[ ] show Eric Perlman qsub_braincloud repo

@alee156

[ ] Demonstrable SIMPLE models and proofs for each:
[ ] 2D HISTEQ AND ADAPTEQ
[ ] 3D HISTEQ AND ADAPTEQ

don't forget to simulate data too :) that is the simplest, and therefore, most important thing to do first!

see google's R style code and probably include it or at least link to it.
@ewalke31 @ebridge2 @alee156


Feature/#130

https://github.com/OAGr/foretold/issues/130

Looks pretty good. Some comments:
Can we change the errors to:
const ERR_1 = 'You have entered both floatCdf and floatPoint values. You can only submit one type of value. ';
const ERR_2 = 'Xs and Ys should be the same size.';
const ERR_3 = `Xs of length {${X_ENTERED}} exceeds maximum of length ${MAX_XS}.`;

Also, add an error,
const ERR_4 = "You must submit one kind of value." (If no values are added)

As to the ``measurementValueValidation`` function, I think it can be cleaned up & dried a bit. For one,
I like the ``!!floatCdf`` convention instead of ``floatCdf !== undefined``

Fixed.


[issue] Bad request when attempting login

when you attempt to log in the user is taken to a page saying that there is an invalid client id
tried on arch with both the aur package and with the pip package

I just checked and it's still working fine for me. If you have a custom configuration file (~/.config/rtv/rtv.cfg), I would double check that the client ID didn't get changed. It should match the default value in the repository:
https://github.com/michael-lazar/rtv/blob/36b9353236acc11be208c4cee2191b9854c9625c/rtv/templates/rtv.cfg#L63-L65

the issue was with the browser I was opening it with, vimb has issues with reddit apparently but firefox does'nt


Ability to setup Screen Markers into "Groups"

The ability to setup screen markers as a group, so that a simple click would toggle all of them would be a nice feature to have I think.
For example let's say i have 10 screen markers set up
1-5 are for activity A
6-10 are for activity B
Simply click the group and all of the markers in that group are enabled at the same time, perhaps add the ability to also expand the group so othat you can toggle on and off individual markers in the group if needed.

Duplicate of #3205


[toy-model] Don't deduplicate grouping sets

... this doesn't actually fix the issue, it just breaks it differently


Update co-digital/api-sdk to latest version 1.0.7

JIRA link
Placeholder NTRNL-554
Change description

Update co-digital/api-sdk to latest version 1.0.7

Thanks for bumping the version @Mouhajer-CO


feat: [es] Create docs/languages/context-propagation

Create feat: [es] Create docs/languages/context-propagation
related to https://github.com/open-telemetry/opentelemetry.io/issues/5206

@mlunadia Thanks! Feel free to add the cSpell exceptions to the ES dictionary under https://github.com/open-telemetry/opentelemetry.io/blob/main/.cspell/es-palabras.txt

@theletterf I think the spelling test should be successful now that the palabras PR is merged but only a maintainer should be able to trigger a re-run the workflow.


Fixed nvm for travis CI

NVM started failing on Travis
Test Plan:

see Travis results

By analyzing the blame information on this pull request, we identified @ide and @spicyj to be potential reviewers.

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.

Thanks for importing. If you are an FB employee go to Phabricator to review.


Filter custom members in TypeProvider

Moves the custom member filtering into TypeProvider so that we can maintain a clear contract that the Properties, Methods, Constructors properties only contain providers that we are actually going to generate.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Microsoft.Generator.CSharp


[gstreamer] build with 1.19.3

Redirect https://github.com/microsoft/vcpkg/pull/22910 to working branch

I will resolve the conflicts soon

Qt Multimedia's EGL::EGL is not what I understand. :(
Feel free to mention me in related issues, @JackBoosY

@luncliff Do you have any other changes left?

@luncliff Do you have any other changes left?

No. I think the changes are enough.


Unified jshint rules for the team.

https://github.com/keycloak/keycloak-nodejs-connect/pull/19#issuecomment-207418699

@lance @lholmquist since we are using semistandard-format
and this :
{
  "esversion": 6,
  "node": true
}

then this issue can be closed.


delete unnecessary string fields

Build triggered. sha1 is merged.

Build started sha1 is merged.

Refer to this link for build results (access rights to CI server needed):
/job/nd4j_PR/312/Build result: FAILURE[...truncated 4.70 MB...]	at hudson.maven.MavenModuleSetBuild.run(MavenModuleSetBuild.java:544)	at hudson.model.ResourceController.execute(ResourceController.java:98)	at hudson.model.Executor.run(Executor.java:405)Caused by: java.io.FileNotFoundException: https://api.github.com/repos/deeplearning4j/nd4j/statuses/b38c9df2067147949fcd60a5dcdb71030544a02c	at sun.reflect.GeneratedConstructorAccessor50.newInstance(Unknown Source)	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)	at java.lang.reflect.Constructor.newInstance(Constructor.java:423)	at sun.net.www.protocol.http.HttpURLConnection$10.run(HttpURLConnection.java:1890)	at sun.net.www.protocol.http.HttpURLConnection$10.run(HttpURLConnection.java:1885)	at java.security.AccessController.doPrivileged(Native Method)	at sun.net.www.protocol.http.HttpURLConnection.getChainedException(HttpURLConnection.java:1884)	at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1457)	at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1441)	at sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:254)	at org.kohsuke.github.Requester.parse(Requester.java:535)	at org.kohsuke.github.Requester._to(Requester.java:262)	... 12 moreCaused by: java.io.FileNotFoundException: https://api.github.com/repos/deeplearning4j/nd4j/statuses/b38c9df2067147949fcd60a5dcdb71030544a02c	at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1836)	at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1441)	at java.net.HttpURLConnection.getResponseCode(HttpURLConnection.java:480)	at sun.net.www.protocol.https.HttpsURLConnectionImpl.getResponseCode(HttpsURLConnectionImpl.java:338)	at org.kohsuke.github.Requester.parse(Requester.java:525)	... 13 more

Build finished. 1981 tests run, 49 skipped, 15 failed.


Made some minor grammatical changes

NOTE: Please read the CONTRIBUTING.md guidelines before submitting your patch,
and ensure you followed them all:
https://github.com/Kong/kong/blob/master/CONTRIBUTING.md#contributing
Summary
Fixed minor grammatical issues regarding comma placement
Full changelog

[Implement ...]
[Add related tests]
...

Issues resolved
Fix #XXX

Thank you but the current spelling is fine.


RCOCOA-2343 Enable building RealmSwift as a dynamic framework with SPM

While App Store Connect supports static libraries, Xcode doesn't actually build static libraries for static targets; it just links the object files directly into the app. It also doesn't implement the xcprivacy manifest merging that the documentation claims exists, so the only way to actually use our privacy manifest is to build a dynamic framework.
Because Xcode is very weird, setting the library type to .dynamic doesn't actually result in it being built as a dynamic framework; it just gives the consuming app the option of building it as dynamic. It also makes it so that having multiple targets depend on the product is broken. Hopefully that will not be a problem.
Building Realm.framework as dynamic and then RealmSwift.framework as static doesn't work, so this adds the privacy manifest to RealmSwift.framework as well.

This appears to work but the process for validating app privacy manifests is so awful that it's hard to be sure.


Fix misuse of MAKEFILE_CONFIG

As the values of RbConfig::MAKEFILE_CONFIG are intended to use in Makefile and may contain make macros, so a path in it may not match /gcc/, nor cannot execute directly.
Use RbConfig::CONFIG, whose values are expanded, for such porpose instead.

Thank you! All tests are green, so I'm merging.


Question: How to update source image of baseTexture for v5

I have been refactoring my code base from 4.8.7 to 5.0.0-rc.3. The BaseTexture class has been changed. There was updateSourceImage method that I was using for  regularly updating the source of texture. How can I update baseTexture source without updating /swaping Texture?
Expected Behavior
--
Current Behavior
--
Possible Solution
--
Steps to Reproduce
--
Environment

pixi.js version: 5.0.0-rc.3
Browser & Version: 73.0.3683.103 (Official Build) (64-bit)
OS & Version: macOS Majove 10.14.3
Running Example: -

In v5 all of dynamic behaviour of BaseTexture was moved to TextureResource, that way user can specify much more things related to the texture than in v4, for example create gradient resource:
https://pixijs.io/examples/?v=v5.0.0-rc.2#/textures/gradient-resource.js
Even more syntax suggar: https://www.pixiplayground.com/#/edit/CRxSABTpybY_HbFtWu6FO
Source canvas was updated somehow. Its possible: https://github.com/pixijs/pixi.js/blob/dev/packages/core/src/textures/BaseTexture.js#L442
You want to upload new texture resource on the same place as it the old one was. Its not possible directly:
https://github.com/pixijs/pixi.js/blob/dev/packages/core/src/textures/BaseTexture.js#L423
However you can extend BaseImageResource that way it supports swapping canvas or image.
https://github.com/pixijs/pixi.js/blob/dev/packages/core/src/textures/resources/BaseImageResource.js
https://github.com/pixijs/pixi.js/blob/dev/packages/core/src/textures/resources/CanvasResource.js

Thanks for your answer.
I have checked code base.
BaseTexture loads image via autoDetectResource then it returns ImageResource that loads the image.
I have refactor updateSourceImage from
PIXI.utils.clearTextureCache();
const url = await this.fetchImage();
this.texture.baseTexture.updateSourceImage(url);
next();

to
PIXI.utils.clearTextureCache();
const url = await this.fetchImage();
this.sprite.texture.baseTexture.resource.source.src = url; // Source is HTMLImageElement so I can reassign src property.
await this.sprite.texture.baseTexture.resource.load(); // Force to re-load.

Swapping texture causes flashing on the scene. My new question is that does it causes any problem?

There's no guarantee, when I made new ImageResource, that was not my focus, Unfortunately, I didnt think what happens if someone swaps the source. I thought about createImageBitmap and a few other things. To get you an answer I need extra research and that takes time, and I believe that's your work.
SUMMARY: I have no idea :)

Pixi is a great and powerful library so thanks for your effort and answers :)

@abdullah you were right, I'm actually adding support for source change.


Star should have occasional flares and activity

Adapting the sun particle system seems like a good place to start.
Screenshot of PG adaptation of particle system:

Added by #69 -- customized the sun particle system to scale it up to the necessary sizes. Known issue - alpha blending isn't quite right for the particle system

Created #73 to track alpha blending issue


Refactor architecture

Extract tree module
Extract middleware module
Simplify router
Rewrite tests

Coverage decreased (-0.1%) to 93.015% when pulling 461b5671258b35198423d002cd30ff4a47c150cf on develop into e0e38b0e471beb0509883fb45ab1823b5a13915b on master.

Coverage decreased (-0.1%) to 93.015% when pulling d12b08f3e87aab3d303431d480aff63a89f2993e on develop into e0e38b0e471beb0509883fb45ab1823b5a13915b on master.


Locally Connected Layer

As #2589 proposed, locally connected layer has been used in DeepFace and Alex's cifar-10 implementation. I think it would be a plus to support this layer in Keras.
Also, I added an example which try to reproduce Alex's cifar-10 network using this layer.

@fchollet Thanks for your valuable comments!

These two operations are significantly different though...

For the BatchNormalization, I would remove it from this example, although it gives a marginal improvement.

What is the final accuracy of the model; are you able to reproduce the results of the original model?

The test logloss is a little better than the cifar10_cnn.py. However, I can't reproduce the original model, which claims that it would have 87% test accuracy in 100 epochs. It would need to change input by cropping 24x24 patches from 32x32 image, but I prefer to keep it simple in example. BTW, do we have cropping in ImageDataGenerator?

Blue: cifar10_cnn.py, Purple: cifar10_alex.py

Why not have self.W already have the proper shape?

Good advise, I will modify it.

However, I can't reproduce the original model, which claims that it would have 87% test accuracy in 100 epochs.

What accuracy do you get after 100 epochs?

BTW, do we have cropping in ImageDataGenerator?

Yes, I'm pretty sure we do.
It would be great to be able to reproduce the original results.

What accuracy do you get after 100 epochs?

Test accuracy (center crop) will be around 81%. The original network uses 10 views average and manual learning rate control to achieve 87% accuracy (about 150 epochs).
They also define different learning rate for weight/bias and different std for weight initialization for different layers. I may need not have time to carefully reproduce it. Anyone wants to take the challenge?
Actually, we don't have cropping mechanism in the ImageDataGenerator. Therefore I use random shift in ImageDataGenerator and center cropping by Lambda Layer to achieve it.

I think if we can't reproduce the 87% baseline we shouldn't include the example script as it may be misleading.

I think if we can't reproduce the 87% baseline we shouldn't include the example script as it may be misleading.

Sure, I will remove it.

@fchollet Thanks for your careful review! I will push again tomorrow.

I planed to use K.batch_dot to replace multiple K.dot on different patch of input. But I think it would potentially result huge memory usage when batch_size or output_shape is large.

Not sure if it is necessary to support same mode. I prefer not haha. If needed, zero padding will be used.

Should I modify mkdoc for this new layer?

UPDATE: Parallel dot in different position (patch) by batch_dot with reduction of 40% computation time on Alex's implement (heavily depends on configuration of layer).
P.S. experiment on TensorFlow.

It looks like test_locallyconnected_2d is currently failing with both backends.

I am dealing with a theano issue, might take some time to think up a workaround.

Currently, I can't solve the problem mention in #2742, so in the case K._backend==theano and dim_ordering=='th' I will use multiple K.dot instead.

Also, update test_local.py to cover different dim_ordering. Unit test for convolutional layer might need same modification as well.

I used this build and try to run a  deepnet model with keras build.
THEANO_FLAGS=optimizer=fast_compile,device=gpu,floatX=float32 nohup sudo python model_training.py > output.out &
I am getting the below error
ERROR (theano.gof.opt): Optimization failure due to: local_useless_inc_subtensor
ERROR (theano.gof.opt): node: IncSubtensor{Inc;int64, ::, ::}(Elemwise{add,no_inplace}.0, DimShuffle{0,1}.0, Constant{0})
ERROR (theano.gof.opt): TRACEBACK:
ERROR (theano.gof.opt): Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/theano/gof/opt.py", line 1772, in process_node
replacements = lopt.transform(node)
File "/usr/local/lib/python2.7/dist-packages/theano/tensor/opt.py", line 2313, in local_useless_inc_subtensor
c = get_scalar_constant_value(node.inputs[0])
File "/usr/local/lib/python2.7/dist-packages/theano/tensor/basic.py", line 662, in get_scalar_constant_value
v.owner.op.perform(v.owner, const, ret)
File "/usr/local/lib/python2.7/dist-packages/theano/tensor/elemwise.py", line 839, in perform
super(Elemwise, self).perform(node, inputs, output_storage)
File "/usr/local/lib/python2.7/dist-packages/theano/gof/op.py", line 769, in perform
"Did you used Theano flags mode=FAST_COMPILE?"
MethodNotDefined: ('perform', <class 'theano.tensor.elemwise.Elemwise'>, 'Elemwise', 'Did you used Theano flags mode=FAST_COMPILE? You can use optimizer=fast_compile instead.')

@joelthchao Hi Joel, what's the current status here? Should I review and merge?

@jfsantos  thank you for your work. It help me a lot.

I wrote a toy example using your layer
code:
x_test=np.random.random_sample((100,5)) y_test=np.random.randint(2,size=100) model=Sequential() model.add(LocallyConnected1D(nb_filter=1,filter_length=2,input_dim=5)) model.add(Flatten()) model.add(Dense(1)) model.compile(optimizer='sgd', loss='mse') model.fit(x_test,y_test)
TypeError: an integer is required
why is it wrong?

@fchollet It should be ready. You can review it.
@xyzhang16 input_length is not set. The layer require a determined input shape.

@joelthchao  sorry, still wrong.
code is following
x_test=np.random.random_sample((100,5)) y_test=np.random.randint(2,size=100) model=Sequential() model.add(local.LocallyConnected1D(nb_filter=1,filter_length=2,input_length=5,input_dim=5)) model.add(Flatten()) model.add(Dense(1)) model.compile(optimizer='sgd', loss='mse') model.fit(x_test,y_test)
TypeError:Bad input argument to theano function with name "C:\Anaconda2\lib\site-packages\keras\backend\theano_backend.py:514"  at index 0(0-based)', 'Wrong number of dimensions: expected 3, got 2 with shape (32, 5).

@joelthchao  thank you!

Hi everyone, and thank you for your work!
Running test_local.py gives me the following warning:

...\Anaconda3\envs\python34\lib\site-packages\keras-1.0.4-py3.4.egg\keras\backend\theano_backend.py:521: UserWarning: theano.function was asked to create a function computing outputs given certain inputs, but the provided input variable at index 1 is not part of the computational graph needed to compute the outputs: keras_learning_phase.
To make this warning into an error, you can pass the parameter on_unused_input='raise' to theano.function. To disable it completely, use on_unused_input='ignore'.
**kwargs)

And trying a simple model:
    model = Sequential()

    model.add(layers.ZeroPadding2D(padding=(1, 1),input_shape = (1,nb_rows,nb_columns)))
    model.add(layers.LocallyConnected2D(2,3,3,activation = 'relu',border_mode = 'valid'))

    model.add(layers.Flatten(input_shape = (1,nb_rows,nb_columns)))
    model.add(layers.Dense(100, activation = 'relu')) #,W_regularizer=l2(0.01)
    model.add(layers.BatchNormalization(mode = 0,axis = 1))
    model.add(layers.Dropout(0.2))
    model.add(layers.Dense(80, activation = 'relu'))
    model.add(layers.BatchNormalization(mode = 0,axis = 1))
    model.add(layers.Dropout(0.1))
    model.add(layers.Dense(40, activation = 'relu'))
    model.add(layers.BatchNormalization(mode = 0,axis = 1))
    model.add(layers.Dropout(0.1))
    model.add(layers.Dense(10, activation = 'relu'))
    model.add(layers.BatchNormalization(mode = 0,axis = 1))
    model.add(layers.Dense(1, activation = 'sigmoid'))

gives me the following (truncated) error:
File "...\Anaconda3\envs\python34\lib\site-packages\theano\sandbox\cuda\nvcc_compiler.py", line 403, in compile_str
    'for cmd', ' '.join(cmd))

Exception: ('The following error happened while compiling the node', GpuJoin(TensorConstant{0}, Rebroadcast{1}.0, Rebroadcast{1}.0, Rebroadcast{1}.0, Rebroadcast{1}.0, Rebroadcast{1}.0, Rebroadcast{1}.0, Rebroadcast{1}.0, Rebroadcast{1}.0, Rebroadcast{1}.0, Rebroadcast{1}.0, Rebroadcast{1}.0, Rebroadcast{1}.0, ...
... ), '\n', 'nvcc return status', 2, 'for cmd', 'nvcc -shared -O3 -arch=sm_50 -Xlinker /DEBUG -D HAVE_ROUND -m64 -Xcompiler -DCUDA_NDARRAY_CUH=m18715462c72ed6afcd7ca5d52813ce90,-DNPY_NO_DEPRECATED_API=NPY_1_7_API_VERSION,/Zi,/MD -IC:\\Users\\sicarbonnell\\AppData\\Local\\Theano\\compiledir_Windows-7-6.1.7601-SP1-Intel64_Family_6_Model_61_Stepping_4_GenuineIntel-3.4.4-64\\cuda_ndarray -IC:\\Users\\sicarbonnell\\AppData\\Local\\Continuum\\Anaconda3\\envs\\python34\\lib\\site-packages\\numpy\\core\\include -IC:\\Users\\sicarbonnell\\AppData\\Local\\Continuum\\Anaconda3\\envs\\python34\\include -IC:\\Users\\sicarbonnell\\AppData\\Local\\Continuum\\Anaconda3\\envs\\python34\\lib\\site-packages\\theano\\gof -IC:\\Users\\sicarbonnell\\AppData\\Local\\Continuum\\Anaconda3\\envs\\python34\\lib\\site-packages\\theano\\sandbox\\cuda -o C:\\Users\\sicarbonnell\\AppData\\Local\\Theano\\compiledir_Windows-7-6.1.7601-SP1-Intel64_Family_6_Model_61_Stepping_4_GenuineIntel-3.4.4-64\\tmp9f9odf_w\\mac87adaa1b92aafdf1d5bd49bc08bc35.pyd mod.cu -LC:\\Users\\sicarbonnell\\AppData\\Local\\Theano\\compiledir_Windows-7-6.1.7601-SP1-Intel64_Family_6_Model_61_Stepping_4_GenuineIntel-3.4.4-64\\cuda_ndarray -LC:\\Users\\sicarbonnell\\AppData\\Local\\Continuum\\Anaconda3\\envs\\python34\\libs -LC:\\Users\\sicarbonnell\\AppData\\Local\\Continuum\\Anaconda3\\envs\\python34 -lcudart -lcublas -lcuda_ndarray -lpython34', '[GpuJoin(TensorConstant{0}, <CudaNdarrayType(float32, (True, False, False))>, <CudaNdarrayType(float32, (True, False, False))>, <CudaNdarrayType(float32, (True, False, False))>, <CudaNdarrayType(float32, (True, False, False))>, <CudaNdarrayType(float32, (True, False, False))>, <CudaNdarrayType(float32, (True, False, False))>, ...)

Other models without locally connected layers are working fine. Can you help me on this?
More info:
THEANO_FLAGS = floatX=float32,device=gpu,lib.cnmem=0.8,optimizer=fast_run
Using gpu device 0: GeForce GTX 770 (CNMeM is enabled with initial size: 80.0% of memory, cuDNN 5005)

LGTM overall. Please address the comments I left, otherwise looking good!

I have applied the last changes and merged this PR: https://github.com/fchollet/keras/commit/835a02c037384b4ccf0e5b2425c6dd7469bf702d
Thanks for the great new layers.

@fchollet  Sorry I am too busy to deal with it, thanks for your great help!


Survey: instance due date - calendar control not allow re-selection of date

If date cleared it resumes working.

The date control doesn't allow dates in the past to be selected, which is why some dates are disabled.
Issue can be closed.


Version number hard coded

If I am not mistaken - the version number in init is hard coded - and currentl set to the wrong version. I can recommend adding versioneer which should take care of this automatically.

thank you for pointing this out!
this is very confusing.
i am fixing with poetry-dynamic versioning, which uses git tags to figure out the version


Electron-88 (Electron Clipboard)

Spectron tests for clipboard data.
@VikasShashidhar Please review. Thanks.

LGTM 👍


Browsing drive in CMD

Hi, I'm developing a filesystem application using the mirror demo as a reference and noticed that my application can't list files in the file system while browsing it in CMD.
I want back to see, and this do not work in the original examples ether.
$n:
$dir > "Insufficient system resources exist to complete the requested service."
and the c# mirror application crashed.
(Am developing on windows 7)

Hi,
Do you have any call trace of the crash ?

Will send it to you as soon as i get home.

Also, I just tried and it is working as excepted with DokanNetMirror. Did you try with latest driver version or which version are you using?

Hi,
Has the issue been resolved? Or is it actually a bug of Dokan? I'm considering to start a project using Dokan and I hope it will work as a normal file system will.

Unable to reproduce it and no news from @entropin.
C:\>m:
M:\>dir
 Volume in drive M is DOKAN
 Volume Serial Number is 1983-1116

 Directory of M:\

12/18/2009  01:54 PM    <DIR>          .
12/18/2009  01:54 PM    <DIR>          ..
07/29/2015  10:48 AM    <DIR>          Liryna
07/14/2009  09:49 AM    <DIR>          Public
               0 File(s)              0 bytes
               4 Dir(s)     536,870,912 bytes free

Nice. I'm glad to find this open source project. Before this I was going to use the Callback File System which is labeled 10000+ USD

在 2015年8月3日，23:31，Liryna<EMAIL_ADDRESS>写道：
Unable to reproduce it and no news from @entropin.
C:>m:
M:>dir
Volume in drive M is DOKAN
Volume Serial Number is 1983-1116
Directory of M:\
12/18/2009  01:54 PM              .
12/18/2009  01:54 PM              ..
07/29/2015  10:48 AM              Liryna
07/14/2009  09:49 AM              Public
0 File(s)              0 bytes
4 Dir(s)     536,870,912 bytes free
—
Reply to this email directly or view it on GitHub.


Pressing an auto-suggested emoji crashes the app

This happens if using Gboard - the Google Keyboard.
This is reported as being the fix:
https://github.com/facebook/react-native/commit/41975f75d96ef4b606b4618461bf24d5db063b77
The commit is included in React Native 0.56

Sweet -- glad this is already fixed upstream!
Looks like https://github.com/facebook/react-native/issues/18405#issuecomment-373624413 is the RN issue report.
Though... hmm. That issue is still open.
... OK, but the linked commit (which came from https://github.com/facebook/react-native/pull/18443) is supposed to be a workaround that applies if you don't pass an onKeyPress prop to your TextInput. Which we currently don't, so I guess when we upgrade to RN 0.56 that will fix the issue for us.

BTW that upstream issue thread helps clarify what the conditions are that trigger the issue: it looks like it's not specific to Gboard, but it happens on Android when you use any keyboard to type... actually any character outside the Unicode BMP, i.e. a character that doesn't fit into one Java char. In practice that mostly means an emoji that isn't one of the oldest emoji.

Unassigning because I believe there isn't any ongoing work to do on this issue; it should just get fixed when we upgrade RN. (Though part of doing that upgrade is we should confirm that.) Thanks again @borisyankov for writing up the issue and for the research!
I just filed #2788 to track the RN upgrade. If you want to take that, please do :)

Also: thanks @adnrs96 for originally reporting this issue!

... Hmm, on the one hand I should have reread the thread more closely:

BTW that upstream issue thread helps clarify what the conditions are that trigger the issue: it looks like it's not specific to Gboard, but it happens on Android when you use any keyboard to type... actually any character outside the Unicode BMP, i.e. a character that doesn't fit into one Java char. In practice that mostly means an emoji that isn't one of the oldest emoji.

So this is something any of us can easily reproduce with an Android emulator or device. :confetti_ball: (🎊 U+1F38a CONFETTI BALL) is one handy example of an emoji that should trigger the issue.
On the other hand, I just tried that on 19.2.102 (the prod release from yesterday, for Android) on my phone, and it didn't reproduce! So there's something here we don't understand.
Maybe Android 9 fixed this? My phone is on that now; it was Android 8 (maybe 8.1) when we looked at this last, in July.
This is probably still resolved by that RN upgrade (#2789), but it'd be good to poke around at least a little more to confirm we still know some way to reproduce the issue before that putative fix. We're not aware of anything in 19.2.192, relative to a few months ago when this was reported, that would have fixed it.

And indeed this is fixed in master! 👏 (U+1F44F CLAPPING HANDS SIGN) Presumably by the RN v0.57 upgrade from #2789.
Thanks again @adnrs96 for reporting this issue!


texlive-{core,extra}: remove docs and sources

Documentation of packages and sources to build stuff don't seem to be needed in typical use cases, and they took quite some space, going from:
1.6G    /nix/store/jj5inbfzzvm8sz278vk6izlqxym718i0-texlive-core-2014
2.0G    /nix/store/b2b67jphwkhnmcl7vsqpczsnh9zcbr7p-texlive-extra-2014
3.5G    total

to:
926M    /nix/store/fg3sq8py57za1l5gqcyds9zi8as3yy5l-texlive-core-2014
818M    /nix/store/cy067520l5b5bl2v1yfwfmwkbr5kzh52-texlive-extra-2014
1.8G    total

I tested building asymptote, eprover, and some LaTeX and LyX stuff of mine.
Maintainers @7c6f434c, @jwiegley, @lovek323; perhaps also @peti is interested. If you have any good test cases, it would be good to try them.
After this reduction, it might be conceivable to build the texLive attribute on Hydra. /cc @edolstra, see 7f54f9965 discussion. (For the other unpack-only texLive packages it's still useless.)

@vcunat Can we continue to build docs for core, but just not for extra?  It is convenient to have documentation available when you're not connected to the Net (which is often when I'm writing my LaTeX documents).

Let's split docs into a separate output then. I'll update the PR to do so.

@7c6f434c: are you the author of builderDefs*? Is it even possible to pass the outputs variable like in stdenv?

Hmm, I started rewriting for stdenv.mkDerivation, and then I got to thinking of a comprehensive solution to #287...

Superseded by #287.


GraphQL server for database data

Describe the feature you'd like to request
From what I can tell, Plasmic code components are client-side only.
That means that the best way to retrieve the data is via an API route, as opposed to in the code component itself.
It's also nice to continue using Prisma, which we can only use from server-side for now. Still TBD if we can connect Prisma to Supabase on the client-side.
Describe the solution you'd like
Expose API routes for GraphQL querying of the Prisma entities
Describe alternatives you've considered
Just use Supabase client from the client-side code components to retrieve data directly from the database.
This could still be a good option for event data, but we probably don't want to do this for dynamic pages, which will be important for SEO

Based on conversations with Reuven, going to deprioritize this in favor of
https://github.com/hypercerts-org/oso/issues/142
for now. We can always go back and create an API interface for OSO data in the future.

Using GraphQL now
https://github.com/opensource-observer/oso/issues/245


Improved handling of failures, restart ICE after failure

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


Improve CI tests

We currently only test the default features; we should test different combinations of features.
We should test that the pre-generated capstone bindings match the ones dynamically generated with bindgen.

I removed the different features and added AppVeyor testing for Windows.
Testing that bindings match is probably unnecessary.


Feature/34 create agreement index

Description
Add a description of your changes here.
Is this PR related with an open issue?
Related to Issue #
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] Follows the code style of this project.
[ ] Tests Cover Changes
[ ] Documentation

Hey @r-marques . Just a quick change, it cracked the metadata API, a bracket didn't get induced in the JSON. 👍


Month view end date issue

{
"title": "Jay Jani",
"start": "2022-09-19T09:30:00.000Z",
"end": "2022-09-20T05:30:00.000Z",
"color": "#FF8F79"
},
My start date is 19th sep and end date is 20th sep but i'm showing only 19th september event.
"react-native-big-calendar": "^3.2.3",

What is your timezone?

GMT+08:00

i need also change cell height. it is possible to change height ?


add support for telescope, nvim-dressing

Thank you for the PR: could you please set-up pre-commit hooks locally and run them? It seems the stylua formatting is different (I presume the pre-commit hooks didn't run?) or you may have a local .stylua in your branch?
Have you rebased against v0.0.2 (it seems the README has a section missing about "autoload in cwd")?


cmd/mount: add an option cache-no-eviction to disable cache eviction

close #3473

Codecov Report
Patch coverage: 59.25% and project coverage change: -0.11 :warning:

Comparison is base (7ed768b) 56.34% compared to head (f291465) 56.23%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3478      +/-   ##
==========================================
- Coverage   56.34%   56.23%   -0.11%
==========================================
  Files         149      149
  Lines       36277    36296      +19
==========================================
- Hits        20440    20412      -28
- Misses      13609    13665      +56
+ Partials     2228     2219       -9

Impacted Files
Coverage Δ

pkg/chunk/cached_store.go
72.23% <ø> (ø)

pkg/chunk/mem_cache.go
56.17% <16.66%> (-2.86%)
:arrow_down:

pkg/chunk/disk_cache.go
68.07% <62.50%> (-4.49%)
:arrow_down:

cmd/flags.go
93.99% <100.00%> (+0.10%)
:arrow_up:

cmd/mount.go
56.01% <100.00%> (+0.09%)
:arrow_up:

... and 6 files with indirect coverage changes
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.


Exploration

⚠ All PRs have to be done versus 'dev' branch, so be aware of that, or we'll close your issue ⚠
What kind of change does this PR introduce?

Can it be referenced to an Issue? If so what is the issue # ?
How can we test it?

Summary

Does this PR introduce a breaking change?
Please TEST your PR before proposing it. Specify here what device you have used for tests, version of OS and version of Browser
Other information

Hi @ZenMakerLabInc can you explain the pourpose of this PR?


[5.8] Use cache for maintenance mode

In Docker Swarm we need to put all application replicas in maintenance mode. Cache is a good option for storing application state as it is shared by all replicas.

I would argue that when on maintenance mode, we might be working to upgrade/repair/maintain the apps and other services including cache. The current state uses the most minimal dependency to notify user about the status

@crynobone But current implementation makes it hard to put into maintenance in docker swarm environment

If this doesn't work for you you could just build your own maintenance mode functionality?


chart: namespace should go with release

Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>What type of PR is this?
/kind bug
What this PR does / why we need it:
Which issue(s) this PR fixes:
the release namespace should go with the helm option -n, but the chart defines another variable 'namespace', make it not convenient to install
the following will go wrong
# helm install kubean kubean-io/kubean -n test

It should go with following command, it's not helm best practice
# helm install kubean kubean-io/kubean -n test --set namespace=test

after this pr, make it right with helm install kubean kubean-io/kubean -n test
Fixes #
Special notes for your reviewer:

/approve


figure out how to share code

Now that I'm using Bass in two projects (Bass + Booklit) I'm finding myself wanting to reuse a lot of code.
How should this be designed?
It's already possible to fetch libraries and load them using a thunk, but I'd like to not have to write that code multiple times. So it should probably be built in to Bass, maybe as part of stdlib.

This might be done with https://github.com/vito/bass/commit/21a42b2f87e9c00296b508afa6290ee2a2889225

Feedback: https://github.com/vito/bass/commit/21a42b2f87e9c00296b508afa6290ee2a2889225 is halfway there, but it's still pretty verbose:
(provide [release]
  (def git
    (load (.git "alpine/git")))

  (def tabs
    (let [repo "https://github.com/vito/tabs"]
      (git:checkout repo (git:ls-remote repo "main"))))

  (def release
    (load (tabs/gh/release "vito/bass" *access-token*))))

A one-liner would be nice.
Some mockups/notes:
(def git
  (load (.git "alpine/git")))

(def release
  (git:load "https://github.com/vito/tabs" "main" ./gh/release))

Better, and nice that it abstracts away checkout/ls-remote, but it'd be nice if I didn't have to type the whole URL.
(def release
  (load (github.com/vito/tabs/main/gh/release "vito/bass" *access-token*)))

How would that github.com path work though? It would have to know that the first two Extend() calls are a user and a repo, and then a ref, and the rest is a path underneath the fetched repo. Sounds complicated, but it's an interesting rabbit hole.
What if there was a primitive (make-path extend-fn) to construct an opaque path?
Then I could implement github.com as:
(def github.com
  (make-path
    (fn [user]
      (make-path
        (fn [repo]
          (make-path
            (fn [ref]
              (make-path
                (let [repo (str "https://github.com/" (name user) "/" (name repo))]
                  (git:checkout repo (git:ls-remote repo (name ref))))))))))))

It's beautiful.

One thing a little spooky about (make-path) is it means path/traversal can call arbitrary code, whereas before it was a pure action that would just return an extended path value. But I guess that's kind of the idea?
Gonna go forward with it for now; it seems pretty intuitive and fills a gap in the language by promoting a builtin construct to a user-defineable type.

Took a shortcut and spiked an implementation that just allows the ExtendPath form to call a Combiner:
diff --git a/path.go b/path.go
index 15f3831..8ad55a9 100644
--- a/path.go
+++ b/path.go
@@ -5,6 +5,8 @@ import (
 	"fmt"
 	"path"
 	"path/filepath"
+
+	"github.com/hashicorp/go-multierror"
 )

 // Path is an abstract location identifier for files, directories, or
@@ -356,12 +358,23 @@ func (value ExtendPath) Decode(dest interface{}) error {
 // Eval evaluates the Parent value into a Path and extends it with Child.
 func (value ExtendPath) Eval(ctx context.Context, scope *Scope, cont Cont) ReadyCont {
 	return value.Parent.Eval(ctx, scope, Continue(func(parent Value) Value {
+		var errs error
+
 		var path Path
-		if err := parent.Decode(&path); err != nil {
-			return cont.Call(nil, err)
+		if err := parent.Decode(&path); err == nil {
+			return cont.Call(path.Extend(value.Child))
+		} else {
+			errs = multierror.Append(errs, err)
+		}
+
+		var comb Combiner
+		if err := parent.Decode(&comb); err == nil {
+			return comb.Call(ctx, NewList(value.Child), scope, cont)
+		} else {
+			errs = multierror.Append(errs, err)
 		}

-		return cont.Call(path.Extend(value.Child))
+		return cont.Call(nil, fmt.Errorf("cannot subpath %T: %w", parent, errs))
 	}))
 }

diff --git a/std/git.bass b/std/git.bass
index 72b74e1..4d00001 100644
--- a/std/git.bass
+++ b/std/git.bass
@@ -29,3 +29,9 @@
         ($ git checkout $ref)
         ($ git submodule update --init --recursive))
       ./)))
+
+(defn github [user]
+  (fn [repo]
+    (fn [ref]
+      (let [repo (str "https://github.com/" (name user) "/" (name repo))]
+        (checkout repo (ls-remote repo (name ref)))))))

Demo:
(use (.git "alpine/git")
     (git:github/vito/tabs/main/gh))

(run
  (from gh:cli
    ($ gh --version)
    ($ ls git:github/vito/bass/main)))

Maybe paths could just be a special type of combiner? The extend syntax is for combining paths, so it seems on brand. It'd be trivial to have them support Decode-ing into a Combiner by adding a PathCombiner type that wraps the Path value and calls Extend.
Could maybe use this for image references to help resolve #24. Both of these to use some sort of .lock file mechanism for storing the resolved paths locally and letting the user decide when to let them roll forward. That way thunks would always be constructed with precise digests.
(from docker/alpine/git/latest
  ($ git --version))

(from<EMAIL_ADDRESS>; apparently this parses as expected
  ($ git --version))

Oh duh, some paths are already combiners - it's how you construct thunks. Hmm.

#28 adds support for this:
(use (.git "alpine/git")
     (git:github/vito/tabs/ref/main/gh))

(Note the added /ref/ to distinguish from /sha/ which doesn't need to ls-remote.)
I don't know what to call the pattern github implements. It's just a combiner, but it's used in a particular way. It can't be used as a path itself, but it creates paths. Implementing one feels like implementing a request router, but it's used more like a magic lazily resolved path. I called it 'path root' here and there because it's the root of paths that it creates, and that's its solepurpose. Maybe that'll stick, but there might be a better name.

#28 adds support for this:
(use (.git "alpine/git")
     (git:github/vito/tabs/ref/main/gh))

(Note the added /ref/ to distinguish from /sha/ which doesn't need to ls-remote.)
I don't know what to call the pattern github implements. It's just a combiner, but it's used in a particular way. It can't be used as a path itself, but it creates paths. Implementing one feels like implementing a request router, but it's used more like a magic lazily resolved path. I called it 'path root' here and there because it's the root of paths that it creates, and that's its solepurpose. Maybe that'll stick, but there might be a better name.


How to use comparator with fastadapter 4.1.1 ?

itemList.withComparator is not working now

Please see the SortActivity sample.
The default list implementation does no longer come with that feature, but the CompareableItemList does.
https://github.com/mikepenz/FastAdapter/blob/b716ba353855ed8f2ad9cb8d3ee43637c33236ad/app/src/main/java/com/mikepenz/fastadapter/app/SortActivity.kt


spinlock: use spin_lock_init replace spin_initialize

Summary
spinlock: use spin_lock_init replace spin_initialize
reason:
1: spin_lock_init and spin_initialize have similar functionalities.
2: spin_lock and spin_unlock should be called in matching pairs.
Impact
spinlock
Testing
ci ostest

[Experimental Bot, please feedback here]
No, this PR does not fully meet the NuttX requirements.  While it provides some information, it lacks crucial details.
Here's a breakdown of what's missing:

Summary:  While it states what was changed, it doesn't clearly explain why this change is necessary beyond superficial similarity. What problem does using spin_lock_init solve?  Does spin_initialize have a bug? Is this a performance improvement?  Is it a stylistic change for consistency? The rationale is weak.  The mention of  "spin_lock and spin_unlock should be called in matching pairs" is irrelevant to the stated change.
Impact: Simply stating "spinlock" is insufficient.  It needs to address all the impact points explicitly (user, build, hardware, documentation, security, compatibility).  Even if the answer is "NO" for most, it needs to be stated.
Testing: "ci ostest" is not sufficient testing information.  What specific tests were run? What were the actual logs before and after the change?  The provided code blocks are empty.  It also needs information on the build host and target used for testing.

In short, the PR needs to be significantly more detailed and explicit to meet the NuttX requirements.  It needs to provide a clear and compelling rationale for the change and demonstrate, with evidence, that the change works as intended and doesn't introduce regressions.


[[FIX]] Disallow leading zeros in BigInts

Coverage decreased (-0.04%) to 99.956% when pulling c80617d6a1597774c1e370c45b2c7ba9977a0b20 on jugglinmike:bigint-octal into b02a025f95f2c7cace8a4db482e2a926d074378f on jshint:master.

Coverage decreased (-0.04%) to 99.956% when pulling c80617d6a1597774c1e370c45b2c7ba9977a0b20 on jugglinmike:bigint-octal into b02a025f95f2c7cace8a4db482e2a926d074378f on jshint:master.


python311Packages.axis: 48 -> 49

Diff: https://github.com/Kane610/axis/compare/refs/tags/v48...v49
Changelog: https://github.com/Kane610/axis/releases/tag/v49
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Already on master.

Wait, the refactor isn't done.

Result of nixpkgs-review pr 289939 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.axis
python311Packages.axis
python311Packages.axis.dist
python312Packages.axis
python312Packages.axis.dist


add gdb model

Could you add a simple example to how to add a gdb model to the scene ?

sure
you can look at chess demo

what comes after  :  phy.load("object.glb")  ?
No need for material

when I try importing a simple box i get this error :
GLTFLoader.js:219 TypeError: Cannot read properties of undefined (reading 'traverse')     at Object.onComplete [as callback] (<anonymous>:47:21)     at Object.next (Pool.js:342:25)     at Object.add (Pool.js:183:14)     at Pool.js:540:18     at GLTFLoader.js:241:6     at GLTFLoader.js:2630:5

you need a callback when loading complete
phy.load("yourObject.glb", onComplete)

onComplete = () => {
   let scene = phy.getGlb('yourObject');// is your main scene
   let mat = phy.getGlbMaterial('yourObject');// list of material
   let mesh = phy.getMesh('yourObject' );// for all mesh ex: mesh.model1
   let group = phy.getGroup('yourObject' );// for all group
}

Thank you it is a big thing making all of this  alone


Cant migrate existing PFFile data

I don't want to lose all the non-text ( images and video ) data I have stored in PFFile formate. How do we migrate the existing data over?

I would be interested in this as well.

We are tracking progress on this issue in #8


LighthouseError: PROTOCOL_TIMEOUT

[Worker] - 2023-03-03T03:03:19.339Z  VERBOSE Gathering trace
[Worker] - 2023-03-03T03:03:19.418Z  VERBOSE Gathering devtoolsLog & network records
[Worker] - 2023-03-03T03:03:49.423Z  VERBOSE Disconnecting from browser...
[Worker] - 2023-03-03T03:03:49.423Z  VERBOSE Cleaning origin data
[Worker] - 2023-03-03T03:03:49.425Z    ERROR Lab Round #1 failed {
error: LighthouseError [LHError]: PROTOCOL_TIMEOUT
at /code/node_modules/lighthouse/lighthouse-core/gather/driver.js:348:50
at new Promise ()
at Driver.sendCommandToSession (/code/node_modules/lighthouse/lighthouse-core/gather/driver.js:346:28)
at Driver.sendCommand (/code/node_modules/lighthouse/lighthouse-core/gather/driver.js:369:17)
at clearNetworkThrottling (/code/node_modules/lighthouse/lighthouse-core/lib/emulation.js:135:18)
at Object.clearThrottling (/code/node_modules/lighthouse/lighthouse-core/lib/emulation.js:107:22)
at Function.runPass (/code/node_modules/lighthouse/lighthouse-core/gather/gather-runner.js:589:21)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at async Function.run (/code/node_modules/lighthouse/lighthouse-core/gather/gather-runner.js:507:29)
at async Function._gatherArtifactsFromBrowser (/code/node_modules/lighthouse/lighthouse-core/runner.js:264:23) {
code: 'PROTOCOL_TIMEOUT',
friendlyMessage: 'Waiting for DevTools protocol response has exceeded the allotted time. (Method: Network.emulateNetworkConditions)',
lhrRuntimeError: true,
protocolMethod: 'Network.emulateNetworkConditions'
}
}
[Worker] - 2023-03-03T03:03:49.426Z     INFO Early return because error found in first round.
[Worker] - 2023-03-03T03:03:49.426Z    ERROR Error occurred during job run {
error: Error: No valid audit result for https://xxxxxxxx
at LabJobWorker. (/code/packages/runner/executor/tmp/runner-scripts/job.LabAnalyze:0.0.1/lighthouse/lighthouse-worker.js:268:23)
at Generator.throw ()
at rejected (/code/node_modules/tslib/tslib.js:116:69)
at runNextTicks (node:internal/process/task_queues:61:5)
at listOnTimeout (node:internal/timers:528:9)
at processTimers (node:internal/timers:502:7)
}
[Worker] - 2023-03-03T03:03:49.427Z     INFO Event emitted to server. { type: 'job.LabAnalyze' }
[Worker] - 2023-03-03T03:03:49.428Z     INFO receive shutdown message from parent

Retry

Retry

Failed again

It's seems only happend when Profile = IPhoneX

I also encountered this issue when I was developing locally using Docker Compose, which caused some of the Lab's runtime features to fail. The Profile was set to the default option. May I know when will this issue be fixed?

[Worker] - 2023-09-15T06:04:23.847Z    ERROR Lab Round #1 failed {
perfsee-main-job-runner-1  |   error: ProtocolError: Protocol error (Performance.enable): Target closed.
perfsee-main-job-runner-1  |       at /code/node_modules/puppeteer-core/src/common/Connection.ts:304:16
perfsee-main-job-runner-1  |       at new Promise ()
perfsee-main-job-runner-1  |       at CDPSession.send (/code/node_modules/puppeteer-core/src/common/Connection.ts:300:12)
perfsee-main-job-runner-1  |       at Page._initialize (/code/node_modules/puppeteer-core/src/common/Page.ts:591:20)
perfsee-main-job-runner-1  |       at Function.create (/code/node_modules/puppeteer-core/src/common/Page.ts:434:16)
perfsee-main-job-runner-1  |       at /code/node_modules/puppeteer-core/src/common/Target.ts:124:14
perfsee-main-job-runner-1  |       at processTicksAndRejections (node:internal/process/task_queues:96:5)
perfsee-main-job-runner-1  |       at async Promise.all (index 0)
perfsee-main-job-runner-1  |       at async BrowserContext.pages (/code/node_modules/puppeteer-core/src/common/Browser.ts:764:19)
perfsee-main-job-runner-1  |       at async Promise.all (index 0) {
perfsee-main-job-runner-1  |     originalMessage: ''
perfsee-main-job-runner-1  |   }
perfsee-main-job-runner-1  | }
perfsee-main-job-runner-1  | [Worker] - 2023-09-15T06:04:23.977Z     INFO Early return because error found in first round or error occured twice.
perfsee-main-job-runner-1  | [Worker] - 2023-09-15T06:04:24.045Z    ERROR Error occurred during job run {
perfsee-main-job-runner-1  |   error: Error: No valid audit result for https://juejin.cn/
perfsee-main-job-runner-1  |       at LabJobWorker.runLighthouse (/code/packages/runner/lab/src/lighthouse/lighthouse-worker.ts:477:13)
perfsee-main-job-runner-1  |       at async LabJobWorker.audit (/code/packages/runner/lab/src/lighthouse/lighthouse-worker.ts:76:22)
perfsee-main-job-runner-1  |       at async LabJobWorker.work (/code/packages/runner/lab/src/lighthouse/index.ts:48:9)
perfsee-main-job-runner-1  |       at async LabJobWorker.run (/code/packages/runner/shared/src/worker.ts:68:7)
perfsee-main-job-runner-1  |       at async executeJob (/code/packages/runner/shared/src/index.ts:25:5)
perfsee-main-job-runner-1  | }


add support for codecrumbs.config.js file (as alternative to CLI params)

could you take a look? https://github.com/Bogdan-Lyashenko/codecrumbs/pull/85


ImGui.Image causes a memory leak

When using ImGui.Image it causes the memory to continue creeping up by a few fractions of a megabyte per second.

Hiya, sorry for getting to this late but thanks for bringing it to my attention. But I've fixed the issue with this commit. I had allocated UVs for images to the heap but forgot to free them at the end of the call!
I'll have a release up for ImGui_GM shortly that'll address this issue.
Thanks again!

Fixed with the latest v1.0.13 release!


Skeleton for custom OTEL receivers

Description
As part of https://github.com/kyma-project/telemetry-manager/issues/728 we identified that we will require two new receivers, see https://github.com/kyma-project/telemetry-manager/issues/921 and https://github.com/kyma-project/telemetry-manager/issues/1003. The sources of the receivers need to be located somewhere and build and shipment procedures need to get established.
Criterias

Find a location for the sources of the new receivers and provide a simple skeleton module for each
Define how to test the receivers before a real image gets produced (what/when/where tests are executed)
Establish a process using Github actions up to the final third-party-image

A possible source of inspiration https://github.com/newrelic/opentelemetry-collector-components


Fix MSBuild for Unity 2020

Overview
This PR modifies certain MSBuild related scripts so that it works with Unity 2020.

/azp run


Document deployment using cloud shell

Benefit/Result/Outcome
So that users will know that a simple way to deploy MLZ in commercial Azure (and possibly Azure Gov) is to use Cloud Shell.
Description
Azure cloud shell includes all MLZ dependencies, including the AZ CLI and Terraform, so demo deployments are very simple using Cloud Shell.
Acceptance Criteria

Documentation exists on deploying using cloud shell.
The documentation includes caveats/warnings that updated versions of Terraform and AZ CLI may produce unknown results in the future.

Duplicate of #230


fix propfinds with depth 0

Fixed the response for propfinds with depth 0. The response now doesn't contain the shares jail anymore.
Fixes: https://github.com/owncloud/ocis/issues/3704
@butonic, can you please double check that this is indeed the fix.

This approach omits stating the children and uses the virtual share jail as is. But the virtual share jail does not have a proper etag and mtime ...
well, /dav/spaces/{sharejailid} now need to see all mountpoints as well, which means the share jail now has to calculate etag end mtime for the root anyway. So this might actually be the fix ... but for depth 0 we still need to aggregate the mtimes, size and use the most recent etag...

So this might actually be the fix ... but for depth 0 we still need to aggregate the mtimes, size and use the most recent etag...

Isn't this happening here? https://github.com/cs3org/reva/pull/2918/files#diff-fa4130be45a2f0a202cc47c8f7ab33722e36ceccb5ba67e881b772e077a59251L448-L462

Oh wait, I still need to test this with subfolders. I only tested it with the root folder.

Oh wait, I still need to test this with subfolders. I only tested it with the root folder.

This still worked.


Grid Menu not Showing

I tried to use the Grid Menu (by including the GRID_ACTIONS object into the plugins object).
GRID_ACTIONS: {
        iconCls: 'action-icon',
        menu: [
            {
                text: 'Delete',
                EVENT_HANDLER: () => {
                    alert('hi');
                }
            }
        ]
    }

I can see the icon, but when I click on it nothing happens. No menu is displayed (as the one displayed on the demo)
Something I noticed is that I have an UNMET PEER dependency when I installed react-redux-grid. Maybe that's connected to the problem?
--> UNMET PEER DEPENDENCY<EMAIL_ADDRESS>

Could you provide the full object your passing to the grid?

Hi Ben,
Thanks for the quick reply.
The object is:
const colums = [... My column definitions]
const pageSize = 20;
const height = '100%';
const events = {
            HANDLE_ROW_CLICK: (cell, reactEvent, id, browserEvent) => {
                 console.log('row clicked');
            }
        }
const plugins = {
GRID_ACTIONS: {
        iconCls: 'action-icon',
        menu: [
            {
                text: 'Delete',
                EVENT_HANDLER: () => {
                    alert('hi');
                }
            }
        ]
    }
}
const gridData = {
            columns,
            dataSource: submissionsUrl,
            pageSize,
            events,
            plugins,
            height
        };

The config looks good but there's a couple issues here that could be causing some problems.
I don't see a store being passed to grid -- the store is a required parameter, as all component state is managed through the store.
If you are passing a store:

Is it a custom store, or is it the store exposed from the main react-redux-grid export?
1a. If its a custom store, you will need to include the grid reducers which listen for actions like SHOW_MENU.
1b. If you want to use the built in store, make sure you pass it to the grid instantiation. You won't need to pass reducers with this store, since they will be combined at startup.

If you are not passing a store:

The grid exposes a store from the main export.

import { Grid, Store } from 'react-redux-grid';

const grid = <Grid { ...{ store: Store }  } />;

Conclusion
With your config, the grid shouldn't load at all. It should throw an error immediately. Let me know if you are/are not passing a store -- but

if you are not passing a store and the grid loads, that's an issue.
if you are passing the built in store, and the menu doesn't show, that's an issue.

Are there any errors in the console?

Hi Ben,
Sorry. My bad. I was adding the Store as a separate prop and data was being shown on the Grid.
What I forgot to add is the appropriate reducer for the menu (GridReducers.Menu).
Thanks for being so quick on responding and for the amazing project you built.
Leo.-

Actually, I still have an issue. Now I see the menu (with 2 elements on the menu array).
GRID_ACTIONS: {
                iconCls: 'action-icon',
                menu: [
                    {
                        text: 'On Hold',
                        EVENT_HANDLER: (ev) => {
                            const state = 'OnHold';
                            const projectId = ev.metaData.rowData.Id;
                            store.dispatch(updateProjectState(projectId, state));
                        }
                    },
                    {
                        text: 'Check',
                        EVENT_HANDLER: (ev) => {
                            const state = 'Checked';
                            const projectId = ev.metaData.rowData.Id;
                            store.dispatch(updateProjectState(projectId, state));
                        }
                    }
                ]
            }

But only 1 (OnHold) shows up.
Do you know what can it be? I'm using the following reducers.
dataSource: gridReducers.DataSource,
  grid: gridReducers.Grid,
  filter: gridReducers.Filter,
  menu: gridReducers.Menu,

Cheeers

So.. I basically debugged the plugin.
And I found the problem to be in the following function:
export const getUniqueItems = (items) => {
    const keys = items.map((obj) => { return obj.key; });
    const unique = items.filter((ob, i) => { return keys.indexOf(ob.key) === i; });
    return unique;
};

My menu array had objects without 'key' property. I basically copied the example from the demo folder as actually there is no documentation on the Readme about the menu plugin. But the demo folder is missing the key property.
Cheers.

Yup! Spot on. Since the menuItems will be rendered as jsx elements, a unique key is required. I will add some documentation about the menu plugin, including a note that a key is required for each item. I will also update the demoData.js file to include keys on the menuItems.


KOGITO-9427: Adjustments in the SW guides for the Timeouts topic

JIRA:

Description:

Please make sure that your PR meets the following requirements:

[X] You have read the contributions doc
[X] Pull Request title is properly formatted: KOGITO-XYZ Subject
[X] Pull Request title contains the target branch if not targeting main: [0.9.x] KOGITO-XYZ Subject
[X ] The nav.adoc file has a link to this guide in the proper category
[X] The index.adoc file has a card to this guide in the proper category, with a meaningful description

How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-7.67.x to backport the original PR to the 7.67.x branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.
If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

@MarianMacik @kaldesai @ricardozanini  Your suggestions where added, thanks.

@wmedvede, Minor suggestions! Otherwise, your content looks good to me. Thank you!

@ricardozanini definitely as this is a part of 1.40.

@MarianMacik Last suggestions where applied, thanks!

@radtriste since you're back, wanna take a look before I merge it?


Handle success and failure for v1.Migrate separately

to "Graph migration to content-addressability took %.2f seconds", the time should be computed after successful handling.

I'm not sure I agree with that. Logging migration time is there to debug reasons for slow startup. Migration can take a considerable amount of time and then end with an error. If this is confusing then maybe we should tweak the messages.

Agree with @tonistiigi We should always display the time.
Thanks!


Update version in README

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Updating the version to be used.
Which issue(s) this PR fixes:

None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

/lgtm


fix(aws-lambda) change path from request_uri to upstream_uri

Summary
when aws-lambda plugin awsgateway_compatible configuration set to true and  request-transformer replace uri configuration enabled, kong use request_uri to find serializer path args, this action doesn't respect  request-transformer configuration parameters

Full changelog
change aws-lambda plugin serializer path arg from request_uri to upstream_uri
Issue reference
Fix #8608

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


Allow Entry access during queries

Implementation of #185.

The actual Entries API itself has been implemented. Queries work correctly, and the Entries restricts queries to being within a subset of the components viewed by the Entries.
The remaining task is to add this in to the query API. This still means a Disjoint trait needs to be introduced to ensure the components viewed for iteration do not conflict with the components viewed for entry access, which will likely be as complicated as the SubSet trait, if not more so due to the need for an inverse component registry.

Entry access is now added to query results. The remaining work is to account for entry views in scheduling and provide entry access to System::run().

I'm not too worried about the slightly reduced code coverage. Anything that touches schedule/mod.rs is bound to reduce code coverage, due to the need for tests to use System implementations that will never be called.


CSS3 "initial" value Specs

CSS3 "initial" value
Specs: http://www.w3.org/TR/css3-cascade/#initial

This issue was imported from Google Moderator
Moderator votes: +22
Issue added by Luckylooke on 2013-07-18
To vote this issue up or down, simply include +1 or -1 in your comment.

+1
see https://css-tricks.com/getting-acquainted-with-initial/ => https://developer.mozilla.org/en-US/docs/Web/CSS/initial

+1

+1

Now available at http://caniuse.com/#feat=css-initial-value


Fix Button's content_margin not applying

Fixes https://github.com/godotengine/godot/issues/92602 and fixes https://github.com/godotengine/godot/issues/92466
This PR simply moves Control::_update_theme_item_cache all the way at the end of Button::_update_theme_item_cache.
Amateur

What if, instead, revert back to what was originally intended, but add another one at, again, end of that function, is this good idea?

Also, for some reason, the editor just stops working when i compiled with SCU enabled, not related to this PR, hold on a little more.

What if, instead, revert back to what was originally intended, but add another one at, again, end of that function, is this good idea?

Adding anything to _update_theme_item_cache or (or changing it) is definitely not a correct fix, I'm not sure what you are trying to do with it, it's completely unrelated to the issue.

Tested on both master [705b7a0b0] and 4.3-beta1 [557f63d03] with https://github.com/godotengine/godot/pull/92009 and this PR applied.
The editor crashes, still tinker around.

Okay, by reverting https://github.com/godotengine/godot/pull/92009 only for _notification, this fixes not only the problem but also keeps the improvement intact.

This PR fixes https://github.com/godotengine/godot/issues/92466

This PR brings back the icon problem

This PR brings back the icon problem

I'd say this would have to be resolved if this is to be merged

Superseded by #92701. Thanks for the contribution!


Refactor parser

Codecov Report

Merging #224 (8c9e2fc) into master (049b525) will increase coverage by 0.16%.
The diff coverage is 62.50%.

@@             Coverage Diff              @@
##             master     #224      +/-   ##
============================================
+ Coverage     60.13%   60.29%   +0.16%
- Complexity      681      682       +1
============================================
  Files           130      131       +1
  Lines          2611     2637      +26
  Branches        329      333       +4
============================================
+ Hits           1570     1590      +20
- Misses          891      894       +3
- Partials        150      153       +3

Impacted Files
Coverage Δ

...in/java/seedu/condonery/commons/core/Messages.java
0.00% <ø> (ø)

.../seedu/condonery/logic/parser/CondoneryParser.java
53.48% <45.45%> (-2.77%)
:arrow_down:

...java/seedu/condonery/logic/parser/CommandEnum.java
100.00% <100.00%> (ø)

.../main/java/seedu/condonery/logic/LogicManager.java
65.51% <0.00%> (ø)

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


No documentation on how to make pagefind works on SvelteKit?

Pagefind runs after Hugo, Eleventy, Jekyll, Next, Astro, SvelteKit, or any other website framework.

I still don't know how to put it in SvelteKit. Why do you make these promises and then not provide any setup guide?

See: https://github.com/CloudCannon/pagefind/issues/327

Good note generally about that page on the docs, though. I should qualify that section, in that the default "quick start" is applicable to the website frameworks with static exports, e.g. SvelteKit with adapter-static, and and Next with output: 'export'. From that point, it's a static site and Pagefind runs on the directory of HTML files produced.
For those websites in dynamic use cases, I'll link off to any existing plugins to integrate Pagefind.
So thanks for the prompt! Will fix 🙂


Add Dark mode toggle button to the Events Page

Feature Request
Describe your problem

The dark mode toggle functionality is not present on the Event page, add that button.
Note- You are not required to make changes in the appearance of the website, even if icontents are not visible, as a different issue is opened for that

can i?


duplicate key in lib/config/en.yaml

There seem to be two instances of 'faggot'

Fixed. Thanks for reporting!


Add documentation on makeprg

@robbinch or @amarpotghan can one of you help with a section on Emacs?

@soenkehahn In case you are not already using it, with makeprg you don't need a separate Makefile for every project.

@sol: Thanks for the pointer.
Just to share how my workflow is atm: I edit the makefile very often, and have project-specific rules in there, so I prefer to have separate files for every project. I did set makeprg to make -f geany recently. To avoid file name clashes with projects that already have a Makefile. And put geany in my global gitignore file.

@sol actually using the same Makefile and making with emacs (M-x compile) works perfectly. I will add it to README later.


support unclip border bbox regression

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: OceanPang:x: pangjiangmiaopangjiangmiao 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.

YOLOBBoxCoder doesn't have unclip_border, may I know why?


create a new branch

ok

thnx
On Wed, 9 Aug 2023 at 06:36, Muthugiritharan @.***>
wrote:

@.**** approved this pull request.
good work
—
Reply to this email directly, view it on GitHub
https://github.com/arpanatraining/shopping-portal/pull/1#pullrequestreview-1568843808,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBXLH6RUSGZVH5UGPPMRE3DXUMVYVANCNFSM6AAAAAA3JP3EGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Align info.description Auth section with ICM

What type of PR is this?
Add one of the following kinds:

documentation

What this PR does / why we need it:
Align Authorization and Authentication info.description section with current ICM approved text.
Which issue(s) this PR fixes:

Fixes #133

@hdamker sorry to bother you again with this, but could you please merge the PR? Thanks!


Duplicate entries in lock file (from examples)

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example

Issue description
I just checked out the examples and noticed:
https://github.com/prefix-dev/pixi/blob/944094e682f4cef5dff7d54094fc937dd6d90dcc/examples/jupyterlab/pixi.lock#L11576-L11621
Expected behavior
No duplicated entries.

Oops, my bad. I see only now that they are for different platforms. Sorry for the noise.


Create CODEOWNERS

Adding a codeowners file to auto-assign us to review Pull Requests

updated rulesets to require reviews from codeowners


Happy to see fair evaluation metric being developed

Hello Clova team,
Greeting from EasyOCR, I'm looking forward to have EasyOCR evaluated by fair metric. (I'm a bit scared of course but it's always better to know own's weakness than living in illusion) May I suggest you to add support for datasets in other language? I think UnrealText has the potential. (I have not tested it myself though.)
Thanks,
Rakpong

Oh, hi @rkcosmos !
First of all, thank you for your interest on our evaluation.
And also thanks for your contribution on OCR community with maintaining EasyOCR, an amazing OCR inference open source code utilizing our works.
As we mentioned on README,
we did not test on multi-lingual dataset since there exists more things to consider, like

reading order
calculation of text length on different language
validity of pseudo character center on other languages, and so on.

You recommended the UnrealText for test set to examine, and yes,
we are planning to target on MLT19(https://rrc.cvc.uab.es/?ch=15), which has the same kinds of languages.
We hope to develop it as soon as possible, so stay tuned!
OR, any pull requests are welcome :)

Hi, yes we use a lot of your work. Before launching our project, we looked into quite a number of repositories and found that your team produces easy-to-read + really work as advertised + under free license code. We really appreciate your team's effort in open sourcing their work.
It's good to know you are targeting MLT19. From your TODO on readme, you mention trying to calculate length of text. Let's say in Hindi you can have र+ ्+ क+ ि  = र्कि. Do you want length = 4 or length = 1? If you want length = 4, then you can just use len(). If you want length = 1, then you can render them and calculate the width of image. I can even try to create a function for that. (would like to find a way to contribute back to your team :) )


Fix and decrease Godot logo size in the HTML5 editor loader

The logo can no longer overflow the viewport. The logo size (or rather, its aspect ratio) is also specified in advance to avoid reflows while the page is loading.
This closes #44692.
Preview
https://user-images.githubusercontent.com/180032/103155850-7d28cf80-47a3-11eb-861b-a9bfdd768655.mp4

Cool!

Cherry-picked for 3.2.4.


Can't specify both a version and a recipe in a nodes run_list

Version:
12.4.0
Environment: CentOS 6.5
Scenario:
I have a node that I want to run a specific recipe from a cookbook on, but I want to also constrain the version of the cookbook to run.
Steps to Reproduce:
Add a entry to a nodes run list that specifies both a recipe and a cookbook version, E.G.
-r<EMAIL_ADDRESS>
Expected Result:
It runs the recipe from the specified version of the cookbook.
Actual Result:
Generates an error:
Chef::Exceptions::RecipeNotFound
--------------------------------
could not find recipe<EMAIL_ADDRESS>for cookbook a_cookbook

Other Info:
Looking at the source code it seems like internally Chef stores the run list as I'd expect with both a recipe and a version, so I'm not sure why the client seems to barf if you specify it in the same format in the run_list.

dup of #3620 #3618


ARMADA-517 Added graceful handling of pyperclip exceptions

What
Absorbs exceptions from pyperclip gracefully.
Why
The ability to copy tokens to the clipboard is very nice, but it breaks when the system does not have a clipboard set up. This absorbs the exception without removing pyperclip.
Task: https://app.clickup.com/t/18022949/ARMADA-517

Peer Review
Please follow the upstream omnivector documentation concerning
peer-review guidelines.

Instead, why simply not try-except when there's no clipboard available and print it anyway?

Couple of reasons for this implementation. First, putting it in a function makes it testable, which is nice. Second, we want to only show the footer text "token copied to clipboard" if pyperclip was able to do that. If it fails, we still print it but don't show that footer.


Page charset problem

In version 1.5.3 the following code:
<head> <meta http-equiv="Content-Type" content="text/html; charset=Windows-1250"><script type="text/javascript" src="jspdf.js"></script><script>function start(){var doc = new jsPDF();	doc.text('Hello world!', 10, 10);doc.save('a4.pdf');}</script><body onLoad="start()">Windows-1250</body>
generates an incorrect pdf file. If we exchange for:
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
the generated file will be correct.
The error described does not appear in version 1.5.2

Will this bug be corrected? Currently, I have to use version 1.5.2, which contains bugs fixed in 1.5.3.

After many hours of analysis, I found a solution. The problem is the line:
var byteToCharMap = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789';
in the function: encodeByteArray
should be:
var byteToCharMap = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+/=';
The question arises, however, why the problem does not occur with UTF-8?
The problem is the line:
blobSupported = new Blob (['ä']). size === 2; // Check if Blob constructor supports ArrayBufferViews
If the character encoding is Windows-1250, then blobSupported is FALSE!
You should therefore change the blobSupported recognition method or always set blobSupported to TRUE.

Thank you very much for your Research. I will apply it soon.

Will this bug be corrected?

The problem is, that this is code by @eligrey
I dont know if your changes are valid or not, because I am not an expert for Blob-Objects. If I apply the change in byteToCharMap than the pdf generation works in my computer.
I made a PR in his repo:

Fixed in eligrey/Blob.js#64

@eligrey
What do you think about the new Blob (['ä']). size === 2;  issue?

And thanks for the fast answer ;)

merged with #2361

@arasabbasi What are you asking about regarding new Blob (['ä']). size === 2;?
[[Blob]].size gives you the number of bytes used to represent the blob. If you want a count of the number of discrete Unicode characters in a string, use [...string].length (e.g. [...'ä👨🏻💻'].length === 2)

@eligrey @krzbor reported that when you are working with latin1 charset then new Blob (['ä']). size will return 1 and not 2. So your probe for detecting the Support of typed arrays is not working then.


Support for Server Side Rendering?

A small question, does localstorage-slim support Server Side Rendering with NextJS? It works fine in development but I'm not sure about production.

Yes, most certainly. It is being already used in production by several websites. (For example - gospelmusic.io)


blackberry camera fixes

handle image size from the parameters, rotate the picture to the proper position.

Cordova CI Build has one or more failures.
Commit     - Link
Dashboard - Link

Builder Name
Console Output
Test Report
Device Logs

Windows 8.1 Store
Link
Link
Link

Windows 10  Store
Link
Link
Link

Windows 8.1 Phone
Link
Link
Link

iOS
Link
Link
Link

Android Mac
Link
Link
Link


Process inherits from Edge, has two type parameters: ports (from edge) and config

Actually, config is not a type parameter as it does not prevent connection, but it is a schema, so we'll have to provide for schemas in addition to type parameters.


reload: Add some logs to help debug lost fragment issue.

See https://chat.zulip.org/#narrow/stream/9-issues/topic/zulipchat.2Ecom.20why.20autorefresh.20to.20recent.20topics for details on the bug.

Closing, since we think we fixed the original bug (and can always reopen if we're wrong).