Publish the react 16 compatible version to NPM

Hi @clauderic!
I see you've updated this library's peerDeps to be compatible with react 16. Can you publish a version containing that change to NPM?

+1

For those interested, I've published it myself as the package amo-react-sortable.

@sedge  I just installed your package but it has a peer dependency of react 0.14

@cyrus-za You're mistaken. Perhaps you installed the wrong version? A fresh install of npm install amo-react-sortable has this in the package.json:
  "peerDependencies": {
    "react": "^0.14.0 || ^15.0.0 || ^16.0.0",
    "react-dom": "^0.14.0 || ^15.0.0 || ^16.0.0"
  },

Let me know if I've overlooked something.

It was due to not uninstalling react-sortable-hoc yet. All good here
On Thu, Mar 15, 2018, 02:04 Kieran Sedgwick<EMAIL_ADDRESS>wrote:

@cyrus-za https://github.com/cyrus-za You're mistaken. Perhaps you
installed the wrong version? A fresh install of npm install
amo-react-sortable has this in the package.json:
"peerDependencies": {
"react": "^0.14.0 || ^15.0.0 || ^16.0.0",
"react-dom": "^0.14.0 || ^15.0.0 || ^16.0.0"
},
Let me know if I've overlooked something.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clauderic/react-sortable-hoc/issues/351#issuecomment-373215572,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABwqZWzitphdFgcsnN975xtGWebHH4ynks5tea_9gaJpZM4Sk3aT
.


[FLINK-14758] Add Executor-related interfaces and executor discovery.

What is the purpose of the change
This PR introduces the basic mechanisms for the Executors and FLIP-73. Namely the Executor interface, the ExecutorFactory and their discovery based on java service discovery. In addition, it wires the discovery to the ExecutionEnvironment and the StreamExecutionEnvironment but this has no effect, as the existing environments override the execute() method.
Brief change log
Described above and in the referenced FLIP.
Verifying this change
This change added tests and can be verified by running the two ExecutorDiscoveryTests, one in flink-java and one in flink-streaming-java.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Thanks for the review @aljoscha ! Merging.

Merged with f074d1ff0d7115c64c966778cc08f0eae8db38e6

I will push the changes as a follow-up commit. Thanks for looking into it @dawidwys


Any breaking change between 0.7.0-Alpha and 0.7.0-Beta?

Describe what you would like to clarify about Kover
While trying to bump up org.jetbrains.kotlinx:kover-gradle-plugin:0.7.0-Alpha to use 0.7.0-Beta in a project with gradle convention plugin, the build fails with
Unresolved reference: disabledForProject
...
Unresolved reference: xml
...
Unresolved reference: onCheck
...
Unresolved reference: html

It is almost as if the plugin is not getting applied.
With 0.7.0-Alpha, there is no issue.

Sample repository: https://github.com/ashish12/kover-sample, with the latest commit on 0.7.0-Beta, and the one before on 0.7.0-Alpha.

Hi,
for Kotlin/JVM, Kotlin/Multiplatform projects you need to move xml, html or verify to defaults block, e.g.
was
koverReport {
    xml {
        // ...
    }
}

now
koverReport {
    defaults {
        xml {
            // ...
        }
    }
}

Property disabledForProject was renamed to disable() function

Thanks. After making the above changes, it is working fine.
https://github.com/ashish12/kover-sample/actions/runs/4860032861/jobs/8663382630


fix: linkinator skip paths

Separated --skip instructions to make it easier to specify which URL patterns we want to skip.

"Check Broken Links" should fail, as there is a broken link on https://www.zetachain.com/docs/developers/evm/bitcoin/

@lucas-janon @bbbeeeee @andresaiello @brewmaster012 please, review.


Allow to exclude labeled return statements - #1317

Added option to exclude labaled returns returning from lambda - active by default and the option to ignore all labaled returns - off by default. Still approved :) ?


GPUImageMovieWriter audioEncodingTarget frame drops

This isn't really an issue with GPUImage so sorry if this is the wrong forum but I was curious if anyone had any experience with reducing the main thread hit from using the audioEncodingTarget. Whenever I start recording with the audioEncodingTarget set, there's always a black frame at the very beginning. If I don't set the audioEncodingTarget there's no black frame.
I thought it might be something the audio category switching but that doesn't seem to help. Any input would be greatly appreciated!

Looks like its a proponent of addAudioInputsAndOutputs in GPUImageVideoCamera

So it looks like if I add the audio targets once in the initial configuration setup, it works fine. Adding and removing them on the fly is causing the frame drops. I think I'll just set up a different initialization category for switching between on the fly audio setup / teardown and one initial audio setup / teardown.

in my case, if i don't call addAudioInputsAndOutputs at init , when i start record, the preview will be black for a second , but if i call it, preview don't get black, then there's a black frame at the beginning of the output movie. comfused...how to resove


Authenticate issue with social logins

Trying to get Facebook working by integrating it with Foundation Apps.
Pretty new to it all, so hopefully something simple I overlooked.
I set up everything according to docs and get following back:
Error: Expecting a token named "token" but instead got: "<!doctype html>\n\n  \n    <meta charset=\"utf-8\" />\n    <meta name=\"viewport\" content=\"width=device-width, initial-scale=1.0\" />\n    <title>1024 awards</title>\n    <link href=\"/assets/css/app.css\" rel=\"stylesheet\" type=\"text/css\">\n    \n    \n    \n    \n  \n  \n
\n
\n
\n
Foundation for Apps
\n
Home
\n
Log in
\n
Register
\n
Logout
\n
\n
\n
\n
\n
\n  \n\n"
    at Object.angular.module.constant.provider.factory.shared.setToken (foundation.js:41618)
    at foundation.js:41681
    at processQueue (foundation.js:18346)
    at foundation.js:18362
    at Scope.$get.Scope.$eval (foundation.js:19564)
    at Scope.$get.Scope.$digest (foundation.js:19380)
    at Scope.$get.Scope.$apply (foundation.js:19669)
    at done (foundation.js:14796)
    at completeRequest (foundation.js:14986)
    at XMLHttpRequest.requestLoaded (foundation.js:14927)

Similar response with Twitter.
Thing is that I do get a correct URL back with the ?code=xyz parameter. But when it is checking for the token in the response, it just gives back my template...
Thanks!

Found the info I needed on https://github.com/sahat/satellizer/wiki/Login-with-OAuth-2.0
Thanks for the docs!


Implement footer and registration page requirements on https://terra.biodatacatalyst.nhlbi.nih.gov/ [SATURN-1416]

Added links to the footer.
Though they are external links, the mockup shows no pop-out symbol next to each link so those were left off. I'm currently in a conversation with UX about whether to add them.
Though part of the ticket, the registration text requirement was left off of this branch as it is a separate task and the implementation is currently a little foggy, so @panentheos and I determined we could at least push the footer changes right now.
Ticket here.

Update:

talked to UX and we will not be using pop-out symbol for external links
I added the registration page updates because it was fast and it's really easy to check
we don't where our Terra ToS or Privacy Policy will live currently


Fixed broken links with duplicate hashtags

Some of the Types links had broken links which linked to ##types instead of #types.

Docs Build status updates of commit 51e40cd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-json-autogeneration.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Day459

Resolves #927.

Pull Request Test Coverage Report for Build 1721

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

Totals

Change from base Build 1727:
0.0%

Covered Lines:
5

Relevant Lines:
5

💛  - Coveralls


Preview window over navbar in Homepage

Description
The preview window appears on top of the navbar on scroll. as seen in the image (right)

Link to Reproduction
https://chakra-ui.com/
Steps to reproduce

Go to 'https://chakra-ui.com/'
Scroll down to 'Less code. More speed section'
See error

Chakra UI Version
2.3.7
Browser
Google chrome Version 107.0.5304.88
Operating System

[ ] macOS
[X] Windows
[ ] Linux

Additional Information
I don't know if this was meant to be so or if it's a bug

Thanks for opening this issue @Barissa-Imran. This issue happens only while the sandpack component is loading, but it clears out afterwards, so I wouldn't put too much priority on it.

Hello, by inspecting the browser and looking for the component I have found out where the problem is coming from, as expected it's related to the z-index css attribute, if you remove the attribute or change the value of it in the class that appears in the screenshoot that I append to this message it fixes the issue.](url)


[FEAT]: Documentation about TypeOrmStore in express-session in NestJS using PostgreSQL

Description
As Typeorm has depricated the getConnection, getRepository methods, we are unable to store the session in the DB.
Another way is as follows:
import { TypeormStore } from 'connect-typeorm/out';
import { DataSource } from 'typeorm';
import * as session from 'express-session';
// other imports

const sessionRepository = app.get(DataSource).getRepository(SessionsEntity);

app.use(
    session({
      // other session options
      store: new TypeormStore().connect(sessionRepository),
    }),
);

Additional Information
No response

Duplicated #37


Implement the list command to support different list

Dependencies: #71
Format: list [todo/schedule/done/all]
E.g.

list todo
list schedule
list done
list all

*NOTE:

By default, example 1 will list everything that are not done unfiltered.
Example 4 will list everything that are done and not done unfiltered..

Refer to issue #70

Complete list to fulfil different list types


Compile problems on Mac on 0.4.0 source tree

Strum: I just tried re-making this from your source but running into new issues. I'll include the make dump below in case you have ides on how to tweak the source:
$ make
c++ -fPIC -I../../include -I../../dep/include -DVERSION=dev -DVERSION_dev -MMD -O3 -march=nocona -ffast-math -Wall -g -DARCH_MAC -mmacosx-version-min=10.7 -std=c++11 -stdlib=libc++ -c -o build/src/MentalABSwitches.cpp.o src/MentalABSwitches.cpp
src/MentalABSwitches.cpp:45:23: error: no matching constructor for
initialization of 'rack::Module'
...: Module(NUM_PARAMS, NUM_INPUTS, NUM_OUTPUTS, NUM_LIGHTS) {}
^      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../include/engine.hpp:43:2: note: candidate constructor not viable: requires
3 arguments, but 4 were provided
Module(int numParams, int numInputs, int numOutputs) {
^
../../include/engine.hpp:33:8: note: candidate constructor (the implicit copy
constructor) not viable: requires 1 argument, but 4 were provided
struct Module {
^
../../include/engine.hpp:41:2: note: candidate constructor not viable: requires
0 arguments, but 4 were provided
Module() {}
^
src/MentalABSwitches.cpp:99:7: error: unknown type name 'lights'; did you mean
'Light'?
lights[B_LEDS + i].value = 1.0;
^~~~~~
Light
../../include/app.hpp:162:8: note: 'Light' declared here
struct Light : TransparentWidget {
^
src/MentalABSwitches.cpp:99:25: error: cannot use dot operator on a type
lights[B_LEDS + i].value = 1.0;
^
src/MentalABSwitches.cpp:100:7: error: unknown type name 'lights'; did you mean
'Light'?
lights[A_LEDS + i].value = 0.0;
^~~~~~
Light
../../include/app.hpp:162:8: note: 'Light' declared here
struct Light : TransparentWidget {
^
src/MentalABSwitches.cpp:100:25: error: cannot use dot operator on a type
lights[A_LEDS + i].value = 0.0;
^
src/MentalABSwitches.cpp:108:7: error: unknown type name 'lights'; did you mean
'Light'?
lights[B_LEDS + i].value = 0.0;
^~~~~~
Light
../../include/app.hpp:162:8: note: 'Light' declared here
struct Light : TransparentWidget {
^
src/MentalABSwitches.cpp:108:25: error: cannot use dot operator on a type
lights[B_LEDS + i].value = 0.0;
^
src/MentalABSwitches.cpp:109:7: error: unknown type name 'lights'; did you mean
'Light'?
lights[A_LEDS + i].value = 1.0;
^~~~~~
Light
../../include/app.hpp:162:8: note: 'Light' declared here
struct Light : TransparentWidget {
^
src/MentalABSwitches.cpp:109:25: error: cannot use dot operator on a type
lights[A_LEDS + i].value = 1.0;
^
src/MentalABSwitches.cpp:138:14: error: use of undeclared identifier
'createLight'
addChild(createLight<MediumLight>(Vec(62, group_spacing ...
^
src/MentalABSwitches.cpp:138:38: error: use of undeclared identifier
'GreenLight'
addChild(createLight<MediumLight>(Vec(62, group_spacing ...
^
src/MentalABSwitches.cpp:144:2: error: expected '}'
}
^
src/MentalABSwitches.cpp:115:50: note: to match this '{'
MentalABSwitchesWidget::MentalABSwitchesWidget() {
^
12 errors generated.
make: *** [build/src/MentalABSwitches.cpp.o] Error 1

looks like you are trying to build against an older version of rack
I uploaded the code that should build under .04 in the /old_versions folder

@Strum: I've been building against this source-- tagged v0.4.0:
https://github.com/VCVRack/Rack/releases
so I am not using dev. Perhaps I should be.
I'll try the /old_versions and let you know what happens. Does this have the latest module(s) ?

yes the older version code in the zip file in the /old_version folder has the master clock, gate maker and patch notes and should build under .040
the new code will only work with the dev version and also has the new modules in it

Just tried building the old version. Still get errors. See if anything looks obvious from the dump:
src/MentalKnobs.cpp:79:21: error: call to 'abs' is ambiguous
knob_value[i] = std::abs(knob_value[i]);
^~~~~~~~
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk/usr/include/stdlib.h:137:6: note:
candidate function
int      abs(int) __pure2;
^
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/stdlib.h:115:44: note:
candidate function
inline _LIBCPP_INLINE_VISIBILITY long      abs(     long __x) _NOEXCEPT ...
^
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/stdlib.h:117:44: note:
candidate function
inline _LIBCPP_INLINE_VISIBILITY long long abs(long long __x) _NOEXCEPT ...
^
src/MentalKnobs.cpp:86:20: error: no member named 'round' in namespace 'std';
did you mean 'roundl'?
octaves[i] = std::round(output_value[i]);
^~~~~~~~~~
roundl
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk/usr/include/math.h:491:20: note:
'roundl' declared here
extern long double roundl(long double);
^
src/MentalKnobs.cpp:87:22: error: no member named 'round' in namespace 'std';
did you mean 'roundl'?
semitones[i] = std::round((output_value[i] - octaves[i])*12);
^~~~~~~~~~
roundl
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk/usr/include/math.h:491:20: note:
'roundl' declared here
extern long double roundl(long double);
^
src/MentalKnobs.cpp:94:30: error: no member named 'round' in namespace 'std';
did you mean 'roundl'?
} else output_value[i] = std::round(output_value[i]);
^~~~~~~~~~
roundl
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk/usr/include/math.h:491:20: note:
'roundl' declared here
extern long double roundl(long double);
^
src/MentalKnobs.cpp:97:22: error: no member named 'round' in namespace 'std';
did you mean 'roundl'?
display_value[i] = std::round(output_value[i]);
^~~~~~~~~~
roundl
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk/usr/include/math.h:491:20: note:
'roundl' declared here
extern long double roundl(long double);
^
5 errors generated.
make: *** [build/src/MentalKnobs.cpp.o] Error 1

sorry i don't have a mac or xcode, it's interpretation of the C++ standard is different from gcc i guess
just remove the std:: with a search and replace and use the suggested functions instead of the standard ones.
you could also install some shell tools and gcc and build it in a similar way you do for windows, there must be a mac system to do that.

Let me play around and see if I can get it to work. Yes, Gnu should be easily implementable.

Here is what I have right now. I removed std:: and that resolved several errors. I now get 2 errors, so getting closer! I ttried getting rid of std:: in front of getw or setw but now it complains of those functions or methods. Any suggested alternatives for getw or setw?
I checked with the group on the advsability of moving from Xcode to Gnu and was advised against it, so I'd like to see if I can compile this on the Mac using Xcode.
Let's see if we can work through this - it is close.
......
c++11 -stdlib=libc++ -c -o build/src/MentalKnobs.cpp.o src/MentalKnobs.cpp
src/MentalKnobs.cpp:127:19: error: use of undeclared identifier 'setw'; did you
mean 'getw'?
to_display << setw(3) << *value;
^~~~
getw
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk/usr/include/stdio.h:389:6: note:
'getw' declared here
int      getw(FILE *);
^
src/MentalKnobs.cpp:127:24: error: cannot initialize a parameter of type
'FILE *' (aka '__sFILE *') with an rvalue of type 'int'
to_display << setw(3) << *value;
^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk/usr/include/stdio.h:389:17: note:
passing argument to parameter here
int      getw(FILE *);
^
2 errors generated.

I think I got it (for version 0.4.0 tagged source). Mainly it was a matter of removing a few things
Strums_Mental_VCV_Modules_04h_source.zip

@metaphorz, good stuff, can you either make a pull request to add your build to the mac releases folder or upload it somewhere and I can add it?

Oops missed this one.  I think if you click on that zip file, it is all there. It is unfortunately labeled "source" at the end, but it isn't the source-- it is the zip that Mac folks will need to download to run your modules on 0.4.0


feat: allow reference types by using internal elem

In my use case exported functions can use &Type because it gets deserialized.  So from the caller's perspective it is Type.
If you don't want this feature then perhaps we can add a CLI arg or ENV variable as a flag.

Ok let's merge it. As reference doesn't exist in the wit specs I think it's not a big deal.


page helper in config.rb can no longer take a regex argument

In v4, the page helper cannot take a regex argument. So this no longer works to prevent directory index creation for all my 404.html pages:
page /404\.html/, :directory_index => false

Is this an intended feature removal, or a regression?

Looks like globs still work, though. I'd like to be able to use regexes wherever I can use globs.

There was a mis-type def'd bit, but test passes.

Waiting for confirm :)

Blocked on further testing by #1749.

Cool, that other one should be resolved.

Awesome, the page helper is back in action!


Added ability for user to provide resource group overrides

Added per-resource group environment files that allows the user to provide JVM options and Accumulo property overrides

This is an alternate solution for #5099 . If this is merged, then #5099 can be closed.

I think I liked the #5099 solution to add the feature to the cluster yaml, rather than have multiple config files filling the config directory. However, I don't feel strongly about it.


consent_date and usage_date updated on login

In the consent table, both consent_date and usage_date are updated on login, also when the user had previously accessed the service.
In any case consent_date should not be updated when logging in to something that one has already given consent for, so that's a bug to be fixed. I wonder though if we should keep usage_date at all. This causes one write per login to our database. Nicer approaches are possible that can avoid this number of writes.
This has been observed in 4.7 and 5.0 (but may be the case much longer already).

consent should always only store the date of (first) consent, no longer the date of last use.

not storing the last_usage could make it hard to detect consents that have not been used for X amount of time. Could for instance be relevant if a given consent expires (or could expire) after a certain period. I don't think something like that exists at the moment, but removing the field removes that possibility as well. Not sure if that is even needed/wanted, but wanted to point it out just to be sure.

Thanks. We're aware of that.

Done in #367 and OpenConext/OpenConext-profile#73


Fix insufficient logging including in the messages the secret file name and username

I have several problems because this lib does not log critical information about username or secret file in error messages.
Ex:
AUTHPRIV ERROR pam_google_authenticator Failed to compute location of secret file
AUTHPRIV ERROR pam_google_authenticator Invalid verification code
AUTHPRIV ERROR pam_google_authenticator Trying to reuse a previously used time-based code. Retry again in 30 seconds. Warning! This might mean, you are currently subject to a man-in-the-middle attack.
AUTHPRIV ERROR pam_google_authenticator Too many concurrent login attempts. Please try again.
...

Thanks!


Create new section to list all "symbols" and rendering of them in the browser using MathML

https://github.com/plurimath/plurimath/blob/main/Latex-Supported-Data.adoc
https://github.com/plurimath/plurimath/blob/main/AsciiMath-Supported-Data.adoc
https://github.com/plurimath/plurimath/blob/main/MathML-Supported-Data.adoc

What fonts are supported? In what format (PDF, HTML, Word ...)?
plurimath-constants-fonts.zip
What font works best with Plurimath symbols? Which one is the most complete?

Some table like this?

We can use the generation mechanism shown here:

https://github.com/plurimath/plurimath.github.io/issues/32


Rebase PRs used in suggested programming tasks

PRs given as examples in Developer Guide: Appendix A: Suggested Programming Tasks to Get Started need to be rebased to the latest master and ensure they still work as advertised.

This issue will be resolved once #789 is merged. However, if we merge #792 (which fixes the bug for undo/redo commands), I think #599 won't be broken, but it'll be buggy (see #792).

However, if we merge #792 (which fixes the bug for undo/redo commands), I think #599 won't be broken, but it'll be buggy (see #792).

I will take a look at #599 again when that happens then. Don't think I can do anything else to rectify that problem now.

@yamgent #599 isn't broken but it's buggy (when you undo/redo on a filtered list for RemarkCommand). The fix can be found in #792 [3/6] :)

I've re-run Travis for the PRs mentioned above and they are all passing :)


Case wise when receiving the updateNewChat parameter

{"@type":"updateBasicGroup","basic_group":{"@type":"basicGroup","id":4093393850,"member_count":3,"status":{"@type":"chatMemberStatusMember"},"is_active":true,"upgraded_to_supergroup_id":0},"@client_id":1}
{"@type":"updateSupergroup","supergroup":{"@type":"supergroup","id":1701774173,"usernames":{"@type":"usernames","active_usernames":["dfadsdsad"],"disabled_usernames":[],"editable_username":"dfadsdsad"},"date":1705682455,"status":{"@type":"chatMemberStatusMember"},"member_count":0,"boost_level":0,"has_linked_chat":false,"has_location":false,"sign_messages":true,"join_to_send_messages":true,"join_by_request":false,"is_slow_mode_enabled":false,"is_channel":false,"is_broadcast_group":false,"is_forum":false,"is_verified":false,"restriction_reason":"","is_scam":false,"is_fake":false,"has_active_stories":false,"has_unread_active_stories":false},"@client_id":1}
basicGroup
supergroup
I may have OCD，The size is too arbitrary😂

"Supergroup" is a single word like "Superman"/"supermodel". "Basic group" is an adjective and a noun like "basic man"/"basic model".


v9.0.0-rc.1.24431.7: fails to build aspire in CI=true environments

Describe the Bug
aspire fails to build from source if CI=true is present in the environment, which is the case on many CI platforms e.g. Travis CI and GitHub Actions, preventing source builds in such CI environments without kludge.
Code responsible for the behavior
Steps to Reproduce
git clone -b v9.0.0-rc.1.24431.7 --recurse-submodules https://github.com/dotnet/dotnet.git
cd dotnet
docker run --rm --platform linux/amd64 -v $(pwd):/vmr -e CI=true -ti mcr.microsoft.com/dotnet-buildtools/prereqs:centos-stream9

# inside docker
cd /vmr
./prep-source-build.sh
./build.sh -so --clean-while-building -c Release /p:PortableBuild=true

Other Information
Example run: https://github.com/loongson-community/dotnet-unofficial-build/actions/runs/10969826008/job/30463397000
Although the project is set up for linux-loongarch64 cross builds, the failure occurs at Stage 1 so still relevant to linux-x64.
Relevant log message:
/vmr/src/aspire/eng/Versions.targets(49,67): error MSB4057: The target "GitInfo" does not exist in the project. [/vmr/src/aspire/src/Microsoft.NET.Sdk.Aspire/Microsoft.NET.Sdk.Aspire.csproj]

I'm able to work around the problem by simply overriding CI for build.sh invocations like this.

This issue is not localized to source-build.
When I ran a local build of the aspire repo with export CI=true and ./build.sh -ci --configuration Release --restore --build --pack, it failed with the same error:
/root/repos/aspire/eng/Versions.targets(49,67): error MSB4057: The target "GitInfo" does not exist in the project.
Looking through the Aspire repo, it's true that GitInfo target does not exist. The issue is originating from these lines. This does not come up in other builds because using -ci does not trigger CI=true.
I'm going to transfer this issue to the aspire repo.
cc @dotnet/aspire-admin


Feature/ma 72

This PR adds a fully functional short walk task and tests.
The short walk test asks the user to walk for X number of steps, while they do this, the device will  be recording the number of steps the user takes using the pedometer sensor if available, or if not available, an accelerometer pedometer fusion algorithm I found on the internet.  I think it needs a few iterations of improvement, so please offer suggestions if you have any.  Anyways, while this is happening, it also records accelerometer and device motion values.  The step goes for either X number of steps, or a timeout in seconds, based on number of steps.  There are two steps like this, and then a resting step in which the user stays still.
This also includes the walk back and forth task.

Looks pretty good to me.  Left a few comments/questions.


Callbacks are invoked from read_loop

At the moment, the user code (the callbacks) are invoked for every message read within the reader loop.
Imagine an application with a lot of callbacks registered on a lot of addresses, and each one performing say, IOs, for every message received. => This would not scale, at all. (the read loop would be terribly slow)
Investigate on "channels" or consider using message queues like crossbeam or mpsc
Or change design completely, "sending" messages from the read loop to the user in an other way.

Fixed, now using iterators


Make ImGui.Text() as an alias of ImGui.TextUnformatted()

https://github.com/mellinoe/ImGui.NET/blob/9b74ee4f797daeb8468d1ed9b8a5c0e804f9e52e/src/ImGui.NET/Generated/ImGui.gen.cs#L13250
In the original ImGui, ImGui::Text is meant to be used in the "printf way".
Obviously ImGui.NET cannot support C or C++'s native variadic parameter list, so we can't pass arguments to ImGui.Text anyway. That means the format string in ImGui.NET is basically useless, and it might even cause crashes, for example:
ImGui.Text("printf uses %s to print a null-terminated string");

This causes the native version of the function to try to get a char pointer from the native stack, of which we do not have any control, and it usually causes crashes.
I think it'd better to make ImGui.Text() as an alias of ImGui.TextUnformatted(), when not dealing with format string, they will have the same output. And we can also avoid misuse by having this alias.

This is an interesting idea, and you are right that it could be dangerous to some users. I don't think passing C-style format strings is something a lot of normal C# users of ImGui would expect to work, but there's also a lot of example code out there using them, which might confuse people.
This affects more than ImGui.Text, though. There's a number of other functions with variadic parameters, like TextColored, TextDisabled, TreeNode, and others. A quick search of "fmt, ..." in imgui.h reveals quite a few. I don't know if there's a general way to solve this, unfortunately, without modifying the native code itself to remove the variadic parameters. There are no alternative, non-variadic versions for any of the other functions.

I learned this the hard way :)
https://github.com/mellinoe/ImGui.NET/issues/177

@mellinoe Just had an idea in mind, though I'm not sure how well would it work xD
Since those ImGui functions can accept multiple arguments (I mean the native version of ImGui), we could import them as something like
[DllImport("cimgui", CallingConvention = CallingConvention.Cdecl)]
public static extern void igText(byte* fmt, UIntPtr /* I meant size_t */ count, byte* text);

Then wrap it as
static readonly byte[] stringFmt = XXX.GetBytes("%.*s");

public static void Text(string text)
{
    byte[] bytes = XXX.GetBytes(text);
    ImGuiNative.igText(stringFmt, bytes.Length, bytes);
}

Reference: https://stackoverflow.com/a/3767300/4399840

This is an issue for more than just that function.
TextWrapped(string fmt) and BulletText(string fmt) also do this.


zero check and send/transfer fixes

closes #38

Tried fixing #40 by making changes according to the audit report. But encountered with few warnings, contract size exceeded being the major one.

@techpandora @vishaldantuluri @kishan79

Tried fixing #40 by making changes according to the audit report. But encountered with few warnings, contract size exceeded being the major one.  @techpandora @vishaldantuluri @kishan79

https://consensys.net/diligence/blog/2019/09/stop-using-soliditys-transfer-now/
reference for the changes done.


Getting Return key in another langauge

I don't know why I am getting return key in another language. If anyone can help I would be really thankful!
I have only added
SystemKeyboard()

Hi @aryan1107
This is due to me accidentally committing this into the English locale when developing support for Armenian. Upgrading to 6.8.1 should fix it.


Shutting down the outbound side of the channel should not accept futu…

…re writes
Motivation:
Implementations of DuplexChannel delegate the shutdownOutput to the underlying transport, but do not take any action on the ChannelOutboundBuffer. In the event of a write failure due to the underlying transport failing and application may attempt to shutdown the output and allow the read side the transport to finish and detect the close. However this may result in an issue where writes are failed, this generates a writability change, we continue to write more data, and this may lead to another writability change, and this loop may continue. Shutting down the output should fail all pending writes and not allow any future writes to avoid this scenario.

Build failure appears to be unrelated ... captured in https://github.com/netty/netty/issues/3138#issuecomment-319524739

Same build failure on a different host (Debian 7-1) https://garage.netty.io/teamcity/viewLog.html?buildId=22855&buildTypeId=pulls_netty&guest=1 ... not sure why this would be related yet ... needs more investigation.

no build failure this time ... kicking off another build.

no build failure ... kicking off another build (round 2)

@normanmaurer - Comments addressed ... wdyt about https://github.com/netty/netty/pull/7041#discussion_r130725503 ?

@Scottmitch like I said please open an issue and assign to me :)
https://github.com/netty/netty/pull/7041#discussion_r131238579

4.1 (237a4da1b) 4.0 (bc8ddd4df)


Make sure that once ES's 10k image limit is reached, we see a variety of datasets on homepage

Potentially, could use "diversified sampler aggregation", but I'm uncertain how to integrate that into reactivesearch


Implement Consistency Model Posterior Estimation

Implement an interface for our consistency model posterior estimation algorithm Consistency Models for Scalable and Fast Simulation-Based Inference

The PR seems to be merged. So I am closing this issue.


<EMAIL_ADDRESS>hash check failed

Installing 'w10privacy' (<IP_ADDRESS>) [64bit]
W10Privacy.zip (2.3 MB) [=====================================================================================] 100%
Checking hash of W10Privacy.zip ... ERROR Hash check failed!
App:         naderi-scoop/w10privacy
URL:         https://www.winprivacy.de/app/download/12302828636/W10Privacy.zip
First bytes: 50 4B 03 04 14 00 00 00
Expected:    c8d89e8134781b0386a93eb7cf759383e2cbf10ed37238a3ac93d8061ce0fecd
Actual:      da79e51ead606708fb97473ffd4f83133ca6c6a36a723921783cdbfacac015e6

Please try again or create a new issue by using the following link and paste your console output:
https://github.com/naderi/scoop-bucket/issues/new?title=w10privacy%4<IP_ADDRESS>%3a+hash+check+failed

@CypherNL Thank you for report.
It fixed now.

@CypherNL Thank you for report.
It fixed now.


Typo in add_sequential_constr

Small thing. The package supports only torch.nn.linear and torch.nn.ReLU but the documentation says "Decision variables used as input for logistic regression". I think you meant regression.
Also, I noticed you support logistic regressions now. Do you plan to add torch model with sigmoid activation functions later?

Thanks for reporting the typo. I'll fix it with #313
Regarding logistic regression, we support scikit-learn's logistic regression with two classes currently.
Supporting neural networks with sigmoid activation is something we are thinking about but didn't do so far (adding the model is not very difficult but we will have to see how hard it is to solve then).

It should be fixed now https://gurobi-machinelearning.readthedocs.io/en/stable/auto_generated/gurobi_ml.torch.sequential.add_sequential_constr.html#gurobi_ml.torch.sequential.add_sequential_constr
For the sigmoid, it is something we are considering. As Gurobi is adding better and better support for nonlinear functions, we will wait a bit.


Get additional metadata from lineTouchData

Discussed in https://github.com/imaNNeoFighT/fl_chart/discussions/1133

Originally posted by KapitanPL September  3, 2022
Hi Guys,
I generate my LineChartBarData based on model.
I then want to let user interact with the model via LineTouchData.
LineTouchData.lineBarSpots!.first.bar seemed to reference the LineChartBarData.
So I tried to extend LineChartBarData to contain my metadata and set this extended object as LineChartBarData. It works, data is displayed.
class LineBarDataMeta extends LineChartBarData {
  LineBarDataMeta(
      {
      ...
      required this.displayed,
      required this.dataKey});
  bool displayed;
  String dataKey;
}

However when I try:
lineTouchData: LineTouchData(
        touchCallback: (p0, p1) {
          if (p0 is FlTapDownEvent &&
              p1 is LineTouchResponse &&
              p1.lineBarSpots != null) {
            var data = p1.lineBarSpots!.first;
            if (p1.lineBarSpots!.first.bar is LineBarDataMeta) {
              var spots = (p1.lineBarSpots!.first.bar as LineBarDataMeta);
              print("key: ${spots.dataKey}");
            }
            print("touch!: ${p1.lineBarSpots}");
          }
        },
      )

the if (p1.lineBarSpots!.first.bar is LineBarDataMeta)  part is never true.
How else can I pass some metadata to the lineTouchData.touchCallback in order to get it when processing the callback?
Edit:
Okay, I get it. There is p1.lineBarSpots![n].barIndex
The problem is that if you have two or more spots with the same X-coordinate you do not know which one you tap.
They are ordered by y-value.
And moreover the FlTapDownEvent has coordinates in "screen" coordinates, however the spot is in the chart coordinates. (x/y values).
This is going to be a nightmare!
How to know which one of the spots I clicked?

LineChartBarData is supposed to be a immutable object.
That's why In the LineChartBarData class, we have two methods copyWith() and lerp(). It means both of them change the type strictly to the LineChartBarData.
For example in the copyWith() function we need to recreate the object with new values.
So you are not able to pass a subclass of LineChartBarData (you cant get it back).
As you mentioned, you have access to p1.lineBarSpots![n].barIndex which gives you the barIndex. Also there is a spotIndex property that gives you the spot Index.
So you can use this code to access the spot's y value:
final bar = res!.lineBarSpots!.first.bar;
final spotIndex = res.lineBarSpots!.first.spotIndex;
final spot = bar.spots[spotIndex];
print(spot.y);

Does it make sense?

@imaNNeo I believe this question to be quite useful. Let's suppose you have a chart where you represent money spent in multiple currencies. Now, your data looks like this:
[
  {
    "currency": "Euro",
    "total": 100,
    "date": "01/01/2023"
  },
  {
    "currency": "Dollar",
    "total": 100,
    "date": "01/01/2023"
  }
]

If the X on your chart is the date, and the Y is the total, then both dollar and euro will have the same spot data. So having a metadata field in FlSpot to assign the currency (for example) is the only way to be able to display a tooltip where you can add the currency name as a prefix.


Pant style pattern of stars

This is my first time in Hacktoberfest I'm beginner in programing and this is my first pr,
I used while loop for making pattern output is in screenshot

@aymansharma I'm closing the PR as it is not fulfilling the requirement...you can send a PR again after maintaining all the steps and eligibility criteria for a valid PR...


Call commands from other commands

Great effort with this library, I like the new features you have been implementing
I was wondering how you would go about handling or structuring the following requirement
Say I have 3 commands
Command1, Command2 and All
For my All Command I would like to call Command1 and Command2
Currently I'm able to do this by doing the following:
[DefaultMethod]
public void AllCommand() {
   var command1 = new Command1();
   command1.DoSomethng();
}

While this works, the issue is if Command1 has dependencies IFooService this wouldn't be injected because I'm newing up commmands.
I'm just wondering how you would about structuring such a requirement?
Thanks for your time and effort

I realised that when I do new Command1() I can set the properties for my injected services
[InjectProperty]
public IFooService FooService { get; set; }

var command1 = new Command1 {
     FooService  = FooService
}

The AllCommand would then need every service that would be required by whatever commands it needs to use. I'm thinking there must be a cleaner way for this?

This seems a very specific requirement. It's best if you implement it. Or suggest a way it can be done in a generic way.

@bilal-fazlani Yes I was thinking it could be a specific requirement, I didn't know if there was some support for this already.
If I was to think in generic temrs, I would have to mark my Commands as ICommand and then I would be able to call my commands through injection, just thinking out loud.
But you are correct perhaps not needed as a generic solution

Could your All command have a dependency on Command1 and Command2?  the DI framework would inject all requirements into those services

@drewburlingame I just saw this comment now, yes I would like to achieve that. Have the ability for a command to depends on another command.
How would you go about letting DI inject these requirements? Could you provide some sample code?

@dotnetshadow  since you were able to solve this with DI, do you mind if we close this PR?

@drewburlingame we can close this.
Guess i was wondering if the way i solved it was a good way or not

Yes, it's the best way to solve it.


Is it the recommended client lib to use the Pub/Sub API?

Hi all,
It's more of a general question than a real issue. Since this library is mentionned in the Salesforce documentation, is it recommended for production or is it just a side project?
Thanks!

You could say that it's a side project given that it's on my GitHub. Salesforce doesn't ship official Pub/Sub API clients for external languages and, as far as I know, there are no plan to do this.
This being said I think that this project is the best Pub/Sub API client that you can find for Node.js at the moment. Simply keep in mind that this project is not an officially supported lib (no support or SLA) but I maintain it frequently with the help of the community.


UAT | Confirm Court

Currently, on submission of e-filing, the user is asked to confirm court.
Expected behaviour

The confirm court pop up only appears if there are more than 1 courts in MDMS.
In case only 1 court is available, this step is not required

Issue is fixed, please find the reference
https://jam.dev/c/8ea56fd9-d88c-4da2-8c35-bac66a4dba69

This Issue is fixed and working as expected in UAT


Add option for open class name

For more flexibility

Thanks for the suggestion.
We considered adding that too. However, for the sake of simplicity, we dropped that idea.
Closing this issue for now. We are open to considering it and re-opening this issue if a lot of people demand for it.

I would love this option as well!
Reason being, I am creating a pattern library (i.e. mini bootstrap) and would like to drive consistency on what the open/active class name is across the various components of the library.
Again though, great work with this - it is a super useful library!
Cheers,
TK


SteamVR Graphics Error (Nvidia GTX1060) on Ubuntu 24.10, ALVR 20.11.1, SteamVR 2.8.8

Hi, I am new to ALVR and I couldn't find help for my issue. Apologies that I am not very experienced with VR on Linux.
I managed to install ALVR on my Quest 3 headset (20.11.1).
When I start the ALVR streamer there is a orange sign "The streamer is not connected! VR headsets will not be discovered". But as soon as I click on "Launch SteamVR" the headset is found and gets connected. OK, fine for me so far.
But SteamVR shows an error: SteamVR Graphics Error. You may need to update your computer's graphics driver. (405)
My Ubuntu 24.10 is a fresh install and I did all the latest updates possible. That includes graphics drivers.
I have a Gigabyte Brix VR with Nvidia GTX1060 graphics card. It has a Intel i7 7th gen processor and it is not Windows 11 compatible and I try to escape from Windows.
I don't know where to go from here. Everything else I found is for older versions and I was told that Ubuntu 24.10 has solved many issues with Wayland and should work.
Any suggestions? I hope this is just a simple thing I forgot to do ...
Peter

Seems a general issue with the graphics drivers. Not ALVR related. Sorry, please close this.

Closed


Added InputDebouncer and PinBusDebounced helper classes and others minor changes

Please review mainly the PinBusDebounced helper class to use it in PortIn class

@emanuelpinto,
After discussion and latest develop commit the PR is now deprecated!
PR still have valuable functions but due to the latest class naming, they should be implemented in other classes.


fix(contributing): typo & grammatical errors

if there are more fixes, add them to this PR, do not create new ones. if not, please let us know and PR will be merged.

Haven't found anything serious in .md files requiring correction (but at least for now).

as I understand it this is ready for merge.


creating a file for python but it shows c++

can you help me when i create a python file it create c++ file and i'll download the extension but i dont know what is the problem

@Anthony2345678 Can you provide more details, like screenshot/gif of what is going on?


[enhancement] Implement infinite scrolling on Article Details page

Hi! I'll love to collaborate on the issue. Can you assign me?


fix: downgraded to es2015 preset to work on ie11

fixes #132

Codecov Report

Merging #133 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #133   +/-   ##
=======================================
  Coverage   97.62%   97.62%
=======================================
  Files          20       20
  Lines         673      673
  Branches       53       53
=======================================
  Hits          657      657
  Misses         16       16

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e0c8e6e...2f4841e. Read the comment docs.


Can edgevpn be configured to use a different non-routable network for VPN than 10.x.x.x

In the edgevpn README it says...
An example of running edgevpn on multiple hosts:

# on Node A
$ EDGEVPNTOKEN=.. edgevpn --address <IP_ADDRESS>/24
# on Node B
$ EDGEVPNTOKEN=.. edgevpn --address <IP_ADDRESS>/24
# on Node C ...
$ EDGEVPNTOKEN=.. edgevpn --address <IP_ADDRESS>/24

For instance per IANA these are the choices for non-routable networks:

10.0. 0.0/8 ( Range: 10.0. 0.0 – 10.255. 255.255 ) – Available IPs: 16,777,214.
172.16. 0.0/12 ( Range: 172.16. 0.0 – 172.31. 255.255 ) – Available IPs: 1,048,574.
192.168. 0.0/16 ( Range: 192.168. 0.0 – 192.168. 255.255 ) – Available IPs: 65,534.

Is there a way to configure/tell edgevpn to use 172.16.x.x for its VPN network IPs instead of 10.x.x.x

non-issue


Check ol.source.UrlTile#urls property for null value

Fixes #4446

Very nice fix, thanks @fredj. Please merge.

do we need a 3.11.2 release?

If so, then we should wait until I have a fix for #4441.

I'll cherry pick these fixes for 3.11.2.


C# / .NET features

I just came across this project. Looks so awesome!
A few questions:

Does it support dependency injection (ILogger, repositories, etc) and if so how does it register those dependencies?
Does it support a static class/method (thinking about functions for F#)?

Thanks in advance =)

cc/ @allantargino @lorenzo-ange

I'm not an authority here but, as far as I read the code

No, it does not. It uses reflection to load your compiled assembly and to create your class. No DI system, but it should not be runtime's concern, since in theory your code must work all by itself.
No, it does not. It creates an instance using Activator.CreateInstance, finds your method with GetMethod, and invokes it using Invoke using previously created instance.

https://github.com/kubeless/runtimes/blob/13e5cf5d6d7a6bb8a6aa82ea2aebeed384f3d85f/stable/dotnetcore/src/Kubeless.Core/Invokers/CompiledFunctionInvoker.cs#L19-L32


OotD Loading & Shown in Task Manager But Nothing Visible

I have OotD installed on one computer for quite some time without issue, but just attempted to put it on another computer and am not able to see it on my computer.  It shows up in the notifications and in task manager and I can make changes to it, but nothing shows up on desktop.  It did give me an error a few times stating "Could not read the calendar" but I believe this is because of the view/item I have tried choosing.  I have tried the suggestion of resetting views, repairing pst file, removing and reinstalling OotD, and deleting the registry key that sets location on screen.  None of these have worked.   I am running Windows 7 (64-bit), with Outlook 2013 in Office Professional (32-bit), and connected to Exchange Server.  My other computer that is running it fine has the same configuration with the Exchange Server but is running Outlook 2007 with Windows 10.  Not sure what else I can try here.
Thank you

That could not read calendar error is something I haven't been able to track down.  Is your office installation fully up to date?

Michael:
Thank you for your prompt response.  I double check and performed updates again to be sure, but I am and appears was already running the most recent version of Outlook 2013.  It appears the error message only comes when I change to particular view or item and the rest of the time the taskbar/notifications make it appear as if all if running fine.  I am wondering if you have had any issue with display utilities specifically for multiple monitors.  I am running numerous monitors with DisplayFusion and wonder if this is causing a bump.  I tried to turn this off and see if their was an improvement, but nothing seemed to work.  Any thoughts?
Thanks

Michael:
On a side note,  I was using this computer without additional monitors the other day and was surprised to find that OotD appeared and was working properly with no apparent difference between this instance and when it was not working with multiple monitors with the obvious exception being that OotD is visible.  I have tried again to run with multiple monitors and removing DisplayFusion to ensure that OotD is not being suppressed or overrode, but to no avail.  Any thoughts?
Thanks

See this: https://social.technet.microsoft.com/Forums/ie/en-US/5bf4ad1b-e8fc-40a2-8cdd-5970cfb66b27/outlook-2010-could-not-read-the-calendar?forum=outlook for a possible solution to the "Could not read the calendar" message.


Add FieldRename.pascal

Adds a pascal value to the FieldRename enum, for PascalCase JSON field name encoding.
Resolves #474.

I signed the Google CLA.

I suppose the Travis build fails because json_serializable depends on json_annotation: '>=2.1.0 <2.3.0', which this PR also changes. I don't know how to fix this.

I suppose the Travis build fails because json_serializable depends on json_annotation: '>=2.1.0 <2.3.0', which this PR also changes. I don't know how to fix this.

You need to add a dependency override to json_serializable.
See https://github.com/dart-lang/json_serializable/commit/9fcee71528f17f8e9e80e90003264e84d048977b#diff-dd8c9829cd89b90a9d3841afafe125be

@natebosch @jakemac53 – thoughts on this?

Going to wait on Lasse to comment on the string bits...
*From: *Nate Bosch<EMAIL_ADDRESS>*Date: *Wed, May 8, 2019 at 2:58 PM
*To: *dart-lang/json_serializable
*Cc: *Kevin Moore, Comment
@natebosch approved this pull request.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/json_serializable/pull/475#pullrequestreview-235312919,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAEFCQWVZJ5IHICMREOFXDPUNEGXANCNFSM4HLLHRFQ
.


[GH-15305][MM-27677] Improve empty state for Recent Mentions.

Summary
Improve empty state for Recent Mentions as per Figma File.
Recent mentions, Pinned messages and Flagged post, all three requires the title to be center as mentioned in 15300, 15304. Because NoResults is used only in these three places, default style is directly changed.
Also I could not change the icon as backgroud grey and '@' as blue. This is the closest I could get with the given ion-icons.
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/15305
JIRA: https://mattermost.atlassian.net/browse/MM-27677
Checklist

[x] Has UI changes
[x] Includes text changes and localization file updates

Device Information
This PR was tested on: Pixel 3a, Android 10
Screenshots

@jp0707 this PR is also affecting mattermost/mattermost-server#15300 for a good reason, I think it could be good if you can comment on the issue and add the needed changes here tackling two for the price of one?? up to you really if you want to do that or leave it as is just for recent mentions ;)

@enahum : Sounds good. I have changed messages and titles for Pinned Posts and Flagged Posts. Please take a look.
One thing I haven't done is changing the logo for pinned posts as it uses png image and none of the "pin" ion icons look the same. Other than that, all the empty screens are done now.

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/9738
https://circleci.com/gh/mattermost/mattermost-mobile/9736

Artifact links:
https://9738-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk
https://9736-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa

Thank you @matthewbirtch. All those changes are done now. Please take a look.

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/9966
https://circleci.com/gh/mattermost/mattermost-mobile/9965

Artifact links:
https://9966-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk
https://9965-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa

@matthewbirtch the one piece of feedback about the text, I’m not sure if we should be focusing in that as the text can be smaller or longer in other languages... thoughts?

Done

@matthewbirtch  Done. PTAL

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/9992
https://circleci.com/gh/mattermost/mattermost-mobile/9991

Artifact links:
https://9992-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa
https://9991-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk

Building app in separate branch.

@matthewbirtch Done. PTAL

For some reason, I'm not seeing the changes in the latest build. Any ideas why?

@matthewbirtch @metanerd I believe this is an issue with the CI. There's an open ticket to make app builds unique. In the meantime I kicked off a new build. Be on the lookout for build-pr-4715-5e1d442 in QA Builds. Thanks

@matthewbirtch @metanerd I believe this is an issue with the CI. There's an open ticket to make app builds unique. In the meantime I kicked off a new build. Be on the lookout for build-pr-4715-5e1d442 in QA Builds. Thanks

That did the trick! Thanks @josephbaylon. I'll approve this PR now


The Vue Semantic Setver crashed 5 tiles in the last 3minutes

The Vue Semantic Server server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.
Volar version: 1.0.14

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
[Error - 3:10:41 PM] The Vue Semantic Server server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.

FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
The Vue Semantic Server server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.
Volar version: v1.0.24

<--- JS stacktrace --->

FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
 1: 0xb09980 node::Abort() [/home/volodymyr/.vscode-server/bin/e8a3071ea4344d9d48ef8a4df2c097372b0c5161/node]
 2: 0xa1c235 node::FatalError(char const*, char const*) [/home/volodymyr/.vscode-server/bin/e8a3071ea4344d9d48ef8a4df2c097372b0c5161/node]
 3: 0xcf784e v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/home/volodymyr/.vscode-server/bin/e8a3071ea4344d9d48ef8a4df2c097372b0c5161/node]
 4: 0xcf7bc7 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/home/volodymyr/.vscode-server/bin/e8a3071ea4344d9d48ef8a4df2c097372b0c5161/node]
 5: 0xeaf465  [/home/volodymyr/.vscode-server/bin/e8a3071ea4344d9d48ef8a4df2c097372b0c5161/node]
 6: 0xebf12d v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/home/volodymyr/.vscode-server/bin/e8a3071ea4344d9d48ef8a4df2c097372b0c5161/node]
 7: 0xec1e2e v8::internal::Heap::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/home/volodymyr/.vscode-server/bin/e8a3071ea4344d9d48ef8a4df2c097372b0c5161/node]
 8: 0xe8336a v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationType, v8::internal::AllocationOrigin) [/home/volodymyr/.vscode-server/bin/e8a3071ea4344d9d48ef8a4df2c097372b0c5161/node]
 9: 0x11fc0b6 v8::internal::Runtime_AllocateInYoungGeneration(int, unsigned long*, v8::internal::Isolate*) [/home/volodymyr/.vscode-server/bin/e8a3071ea4344d9d48ef8a4df2c097372b0c5161/node]
10: 0x15f0b19  [/home/volodymyr/.vscode-server/bin/e8a3071ea4344d9d48ef8a4df2c097372b0c5161/node]
[Error - 2:40:17 PM] The Vue Semantic Server server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.

Duplicate of #1816

这是来自QQ邮箱的假期自动回复邮件。您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。


Syntax to update a map with other elements of the map itself?

I'm looking for something like S[ r |-> S(v)]. i.e. if S maps v to x, then update S to map r to x. In other words, I'd like a way to call the map with some value within an update rule. Is this possible in K?

S [ r <- S[v] ]

Ooh, I didn't try that. Thank you very much! What happens if S[v] is empty?

you should not use the syntax S[v] in a context where you don't know whether v is in the domain of S. The meaning of such an expression in the theory of K is bottom, ie, no such term exists, and in theory it would make the entire configuration nonsatisfiable and thus removed from the list of results. But in practice this is likely to be buggy. You are better off writing rules that either explicitly match on v as a key, or else writing a side condition that says v in_keys(S).
The reason your rule does not parse is that K does not support polymorphic sorts yet. Thus, the sort of Sigma is Map and not Map{KVar, Int}, which means the sort of Sigma[X1] is KItem, not Int. You have two choices: you can explicitly cast the map lookup to an int like so (the configuration will get stuck if an int does not in fact exist there): {Sigma[X1]}:>Int.
A more idiomatic solution would be to rewrite the rule to explicitly match on the keys of Sigma, like so:
rule <k> X1:KVar + X2:KVar => !X:KVar ... </k>
     <state>... X1 |-> I1 X2 |-> I2 (.Map => !X |-> X1 +Int X2) ...</state>

Note my use of +Int instead of +. +Int is the addition over the theory of integers. + is most likely an expression you defined in the grammar of your language, which I suspect is not what you intended.

Ah, I see. Thanks for the reply!
My actual intention was for Sigma to be a map from variables to symbolic expressions. I currently am using it as a symbolic environment. So I my intention was actually to have the symbolic term X1 + X2 if X1 and X2 are variables. But I think I understand now why the rule does not parse. Thanks!


minor refactoring

Coverage increased (+0.2%) to 37.011% when pulling 539a3e4b34c53df9f8f2209ef8a9e1203fd44126 on gabrielburnworth:refactoring into ec0566f11390f4566db1508fc857dae2cec1b222 on FarmBot:master.


Alch animation transmogs.

Some ideas:
fire surge (anim 7855 with gfx 1464, then 30 client ticks later anim 713 and gfx 113, both starting on frame 8):

fire surge (anim 7855 with gfx 1464, then 30 client ticks later gfx 1466):

Something with 4069 (teletab) would be neat, but idk which gfx would go with it.

So I added a setting to overwrite the alch animation with some of the existing effects in the plugin. However, it'll take a little more work to be able to play multiple GFX/animations for a given effect which I'll do in the future.


Updated cssgram.css

Added Valencia filter

hey! thanks for the contribution -- you should be editing the .scss file though :P (see File Structure Outline in the README)


unionall() fails with single element

OrientDB Version: 2.2.18 build UNKNOWN@rf31f2e10de758cbdef4cee27451b4065b94d9ce2; 2017-03-04 00:50:53+0000
Expected behavior
We have a SQL-query builder, which -in one of the subqueries- uses "unionAll()" in a generic way to combine the results of one or more subqueries:
SELECT unionAll($var1.x , $var2.x, ... $varN.x) let
var1 = (SELECT set(rid) AS x FROM ... WHERE ...),
var2 = (SELECT set(rid) AS x FROM ... WHERE ...),
...
varN = (SELECT set(rid) AS x FROM ... WHERE ...)

In cases where there is more than 1 subquery, and hence more than 1 variable inside the "unionAll()", the results are correct:
SELECT unionAll($var1.x, $var2.x) AS rowid
      let var1 = ( SELECT set(rid) AS x FROM INDEX:idx_cdr_af_20170521_3 WHERE key LUCENE ' *80095043* '),
       var2 = ( SELECT set(rid) AS x FROM INDEX:idx_cdr_af_20170521_3 WHERE key LUCENE ' *800970* ')

produces
{
    "result": [
        {
            "@type": "d",
            "@rid": "#-2:3",
            "@version": 0,
            "rowid": [
                "#591:3964746",
                "#593:363408",
                ...
                "#595:4029786"
            ],
            "@fieldTypes": "rowid=z"
        }
    ],
    "notification": "Query executed in 3.681 sec. Returned 1 record(s)"
}

However, in cases where there is only a single subquery with a non-empty result-set, and hence only a single variable inside the "unionAll()", then the results are empty:
For example, using the single subquery
SELECT set(rid) AS x FROM INDEX:idx_cdr_af_20170521_3 WHERE key LUCENE ' *80095043* '

we get a result "x" consisting of 82 RIDs :
{
    "result": [
        {
            "@type": "d",
            "@rid": "#-2:0",
            "@version": 0,
            "x": [
                "#591:3964746",
                "#593:363408",
                ...
                "#594:335493"
            ],
            "@fieldTypes": "x=e"
        }
    ],
    "notification": "Query executed in 1.418 sec. Returned 1 record(s)"
}

But with the outer unionAll(), the result is empty:
SELECT unionAll($var1.x) AS rowid
      let var1 = ( SELECT set(rid) AS x FROM INDEX:idx_cdr_af_20170521_3 WHERE key LUCENE ' *80095043* ')

produces
{
    "result": [

    ],
    "notification": "Query executed in 1.481 sec. Returned 0 record(s)"
}

-> bug in "unionAll()" ?

Hi @rdelangh
unionAll() function, when used with a single input value, is evaluated as an aggregate function, ie. it aggregates results coming from query target (and your query has no target...)
Try this to get what I mean:
Create class AA;
insert into AA set tags = ['a', 'b'];
insert into AA set tags = ['c', 'd'];

select unionAll(tags) as foo from AA

You will obtain
foo = ['a', 'b', 'c', 'd']

Thanks
Luigi

hi Luigi,
can you please define "target" in "(and your query has no target)" ?
thx
Rob

Hi @rdelangh
For target I mean the FROM of the query, eg. a class or a subquery
Thanks
Luigi


Support upsert requests

Upsert requests (using PUT method) are useful to update existing or
create a new entity within a single API call.

Thanks @lest!

I appreciate this PR / feature, but is this supported in Mailchimp's 3.0 API? I keep getting a 405 Error (Method Not Allowed) – http://kb.mailchimp.com/api/error-docs/405-method-not-allowed – when trying to use it.

@krnjn Yes, it's supported in API 3.0. You can check the spec added in this PR for an example of the request which I also tested manually.

@lest ah I see now, there's a slight difference in the manner this works compared to the documentation in the README. In particular, you have to search for the list member using the hashed_email (i.e. members(hashed_email).upsert rather than simply doing the members.upsert as the documentation seems to state). Does that sound about right? (my specs are passing now that I've changed it to that).
@amro would it be possible to add some clarification to that section of the README?

@krnjn Yes, definitely. Sorry to lead you astray with poor documentation there. I'll update the README.

And to your question, yes, that's correct re:passing the hashed email address.


Mimic sklearn._binary_clf_curve for choosing metrics thresholds

calculate_metrics_by_thresh uses all informative thresholds by default but lets you pass in your own set of thresholds e.g. if you want to use fewer so that it runs faster. IIUC scikit-learn's precision_recall_curve by constrast uses sklearn._binary_clf_curve to select an informative subset of np.linspace(0, 1, 101). I think that's a better approach because it limits the maximum number of thresholds at 100, which should pretty much always be more than enough, without requiring the user to do anything. We would probably want to copy sklearn._binary_clf_curve to avoid depending on a private function from another library.

I agree, I can't imagine a scenario where more than a 100 thresholds give anything extra informative!


Bug: React$1.Component is not a constructor

Current Behavior
npm run dev

<EMAIL_ADDRESS>dev
next dev

▲ Next.js 13.5.5

Local:        http://localhost:3000

✓ Ready in 2.2s
○ Compiling /page ...
warn - No utility classes were detected in your source files. If this is unexpected, double-check the content option in your Tailwind CSS configuration.
warn - https://tailwindcss.com/docs/content-configuration
✓ Compiled /page in 4.7s (758 modules)
⨯<EMAIL_ADDRESS>(6618:23) @ $$$reconciler
⨯ TypeError: React$1.Component is not a constructor
at webpack_require (/Users/TW/IdeaProjects/myapp/.next/server/webpack-runtime.js:33:42)
at eval (./src/app/page.js:7:69)
at (rsc)/./src/app/page.js (/Users/TW/IdeaProjects/myapp/.next/server/app/page.js:173:1)
at Function.webpack_require (/Users/TW/IdeaProjects/myapp/.next/server/webpack-runtime.js:33:42)
at async Promise.all (index 0)
null
⨯<EMAIL_ADDRESS>(6618:23) @ $$$reconciler
⨯ TypeError: React$1.Component is not a constructor
at webpack_require (/Users/TW/IdeaProjects/myapp/.next/server/webpack-runtime.js:33:42)
at eval (./src/app/page.js:7:69)
at (rsc)/./src/app/page.js (/Users/TW/IdeaProjects/myapp/.next/server/app/page.js:173:1)
at Function.webpack_require (/Users/TW/IdeaProjects/myapp/.next/server/webpack-runtime.js:33:42)
at async Promise.all (index 0)
digest: "4204431394"
null
⨯<EMAIL_ADDRESS>(6618:23) @ $$$reconciler
⨯ TypeError: React$1.Component is not a constructor
at webpack_require (/Users/TW/IdeaProjects/myapp/.next/server/webpack-runtime.js:33:42)
at eval (./src/app/page.js:7:69)
at (rsc)/./src/app/page.js (/Users/TW/IdeaProjects/myapp/.next/server/app/page.js:173:1)
at Function.webpack_require (/Users/TW/IdeaProjects/myapp/.next/server/webpack-runtime.js:33:42)
null
TypeError: React$1.Component is not a constructor
at $$$reconciler (webpack-internal:///(rsc)/./node_modules/@pixi/react/dist/index.cjs-dev.js:5499:35)
at eval (webpack-internal:///(rsc)/./node_modules/@pixi/react/dist/index.cjs-dev.js:19304:17)
at<EMAIL_ADDRESS>(/Users/TW/IdeaProjects/myapp/.next/server/vendor-chunks/@pixi.js:2250:1)
at webpack_require (/Users/TW/IdeaProjects/myapp/.next/server/webpack-runtime.js:33:42)
at eval (webpack-internal:///(rsc)/./src/app/page.js:7:69)
at (rsc)/./src/app/page.js (/Users/TW/IdeaProjects/myapp/.next/server/app/page.js:173:1)
at Function.webpack_require (/Users/TW/IdeaProjects/myapp/.next/server/webpack-runtime.js:33:42)
at runNextTicks (node:internal/process/task_queues:60:5)
at listOnTimeout (node:internal/timers:540:9)
at process.processTimers (node:internal/timers:514:7)
at async ek (/Users/TW/IdeaProjects/myapp/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:11:455639)
at async eJ (/Users/TW/IdeaProjects/myapp/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:11:459729)
at async eY (/Users/TW/IdeaProjects/myapp/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:11:460204)
at async eY (/Users/TW/IdeaProjects/myapp/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:11:460335)
at async eK (/Users/TW/IdeaProjects/myapp/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:11:463747)
at async /Users/TW/IdeaProjects/myapp/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:11:464225 {
digest: '3118112480'
}
⨯<EMAIL_ADDRESS>(6618:23) @ $$$reconciler
⨯ TypeError: React$1.Component is not a constructor
at $$$reconciler (webpack-internal:///(rsc)/./node_modules/@pixi/react/dist/index.cjs-dev.js:5499:35)
at eval (webpack-internal:///(rsc)/./node_modules/@pixi/react/dist/index.cjs-dev.js:19304:17)
at<EMAIL_ADDRESS>(/Users/TW/IdeaProjects/myapp/.next/server/vendor-chunks/@pixi.js:2250:1)
at webpack_require (/Users/TW/IdeaProjects/myapp/.next/server/webpack-runtime.js:33:42)
at eval (webpack-internal:///(rsc)/./src/app/page.js:7:69)
at (rsc)/./src/app/page.js (/Users/TW/IdeaProjects/myapp/.next/server/app/page.js:173:1)
at Function.webpack_require (/Users/TW/IdeaProjects/myapp/.next/server/webpack-runtime.js:33:42)
at runNextTicks (node:internal/process/task_queues:60:5)
at listOnTimeout (node:internal/timers:540:9)
at process.processTimers (node:internal/timers:514:7)
at async ek (/Users/TW/IdeaProjects/myapp/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:11:455639)
at async eJ (/Users/TW/IdeaProjects/myapp/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:11:459729)
at async eY (/Users/TW/IdeaProjects/myapp/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:11:460204)
at async eY (/Users/TW/IdeaProjects/myapp/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:11:460335)
at async eK (/Users/TW/IdeaProjects/myapp/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:11:463747)
at async /Users/TW/IdeaProjects/myapp/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:11:464225 {
digest: '3118112480',
page: '/'
}
null
⚠ Fast Refresh had to perform a full reload due to a runtime error.
✓ Compiled /_error in 206ms (900 modules)
⚠ Fast Refresh had to perform a full reload due to a runtime error.
✓ Compiled /favicon.ico/route in 174ms (605 modules)
Expected Behavior
It should be work.
Steps to Reproduce

I create a new next app
npx create-next-app

Install pixi.js and @pixi/react
npm install pixi.js @pixi/react

I change the page.js as following:
import { Stage, Container, Sprite } from '@pixi/react'  export default function Home() {  return (    <Stage width={800} height={600}>      <Container>        <Sprite image="https://s3-us-west-2.amazonaws.com/s.cdpn.io/693612/IaUrttj.png" x={100} y={100} />      </Container>    </Stage>  ) }

Environment
"@pixi/react": "^7.1.1",
"next": "13.5.5",
"pixi.js": "^7.3.1",
"react": "^18",
"react-dom": "^18"
Possible Solution
No response
Additional Information
It works if I remove next.js. Anyone can help?

I add 'use client' and it fixed.


Setup war project

@bot-gradle test this

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

I've triggered the following builds for you:

PullRequestFeedback build

We can do this is multiple steps.  Step 1 is extracting just the war plugin here.

Step 2 is extracting the ide plugins here.


Update YASK interface to use newer grid APIs for left/right halos/pads.

Looks good. Cannot merge immediately because we've got broken Travis ATM.

@chuckyount rebased on most recent master. Let's see if Travis behaves now.

Oh, flake8 is complaining: https://travis-ci.org/opesci/devito/jobs/373371929#L3106

WARNING : When merging this, also update our own fork of YASK

Codecov Report

Merging #557 into master will decrease coverage by 0.03%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #557      +/-   ##
==========================================
- Coverage   75.82%   75.79%   -0.04%
==========================================
  Files          74       74
  Lines        7021     7024       +3
  Branches     1427     1427
==========================================
  Hits         5324     5324
- Misses       1524     1527       +3
  Partials      173      173

Impacted Files
Coverage Δ

devito/yask/function.py
0% <0%> (ø)
:arrow_up:

devito/yask/wrappers.py
0% <0%> (ø)
:arrow_up:

devito/yask/data.py
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9a7fc91...9458ad7. Read the comment docs.

Merged.


Create README.md

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

I'd recommend to remove this link. It performs builds of pull requests, so the public status may become red if somebody submits a wrong PR

Is there a different link pointing to master branch in main repository?  If somebody violates the process and the build fails, committers and plugin maintainers get notifications from pull request builders. So we don't expect a long "failed status" as well.
In general it would be useful to have a direct link to the master branch. I'd recommend to create a JIRA issue in the INFRA project and describe the cases you see there. After that somebody from ci.jenkins-ci.org or jenkins.ci.cloudbees.com admins may take the task and implement it.

Yeah I would suggest skipping the status icon too, it will usually be misleading.

(There are plans afoot to replace the current plugin builds with a multibranch system on ci.jenkins-ci.org, which would not suffer from this issue.)

@jglick Could you point me to the updated URL format? Thanks


Icon appearing left in Kohana

Hi, and thanks for the great component !
I noticed a display issue in the Kohana component with the 'envelope' icon from FontAwesomeIcon.
I did not try more icons, the 'key' one works well.
Here you can see the problem :

The icon still appears a little when the TouchableWithoutFeedback is not touched.
I modified the
transform: [                 {                   translateX: focusedAnim.interpolate({                     inputRange: [0, 1],                     outputRange: [-15 - iconSize, 0],                   }),                 },               ]
from Kohana.js
to :
transform: [                 {                   translateX: focusedAnim.interpolate({                     inputRange: [0, 1],                     outputRange: [-20 - iconSize, 0],                   }),                 },               ]
and it seems to work. But I don't want to have to modify the Kohana.js as I will likely export the project and my coworkers will npm install without this change made.
I'm not sure my change works on all icons though.
Thanks

Hi @mbarbier003, thanks for reporting!
You can always fork the repo and make your changes there, and you can point that repo in your package.json file.
Meanwhile, we can try finding an appropriate way to fix the issue for everyone.

Hi @halilb,
I will fork and point to my repo until you find and release a suitable long-term solution :)
Thanks

+1 for the modification


MacVim always links to system Ruby

Unlike Vim, which will link to the first available Ruby (for instance, Homebrew Ruby in /usr/local), MacVim always links the OS version of Ruby. This causes plugins that depend on Ruby to be usable under Vim or MacVim, but not both.
Refs Homebrew/homebrew-core#10824

you are right. thanks.

@splhack thanks so much! Will this require a downstream change in Homebrew?

It'll work without any changes, I think.

@splhack awesome. Any idea when the next snapshot will be taken?

maybe after the test failure fixed.


The format of the apiUrl and frontUrl arguments should be specified in the README

These should look like:
apiUrl: https://lago-api.yourdomain.com
frontUrl: https://frontend.yourdomain.com

not apiUrl: lago-api.yourdomain.com. Otherwise the front end attempts to post to frontend.yourdomain.com/lago-api.yourdomain.com.
Additionally, trying to do https://frontend.yourdomain.com fails too.

+1, It could also be great to split scheme with url like :
apiUrl: lago-api.example.com
apiUrlScheme: https
frontUrl: frontend.example.com
frontUrlScheme: https

And this could be used as default in ingress ressource for hosts in order to avoid a repeat.
I could submit a PR if others are interested.

@maximiliend feel free to open one!

These should look like:
apiUrl: https://lago-api.yourdomain.com
frontUrl: https://frontend.yourdomain.com

not apiUrl: lago-api.yourdomain.com. Otherwise the front end attempts to post to frontend.yourdomain.com/lago-api.yourdomain.com.
~Additionally, trying to do https://frontend.yourdomain.com fails too.~

It is not working


Update background task to use the multi-user endpoint

https://github.com/ornicar/lila#get-apiuser-fetch-many-users
We can search based on the lichess4545-league team and possibly the lone-wolf-3030 team as well. Not every player is guaranteed to be on those teams, so we'll need to keep track of which players weren't updated and request their ratings individually.

I've added the code to implement this, but rate-limiting seems to be a real issue in practice so it's disabled for now (just doing it one-by-one like before).


MAINT/DOC: Pre-render notebook HTML for faster loading

Crossref https://github.com/adriangb/scikeras/pull/123#discussion_r524840156

@stsievert it seems like GitHub automatically caches notebook rendering.
I was looking at this notebook, and the first time it loaded it took ~10 seconds. After that, it loaded instantly, even in another browser, logged out of my account. I'm not sure how long they cache it for though.
Can you check if it loads quickly for you?

@stsievert it seems like GitHub automatically caches notebook rendering.
I was looking at this notebook, and the first time it loaded it took ~10 seconds. After that, it loaded instantly, even in another browser, logged out of my account. I'm not sure how long they cache it for though.
Can you check if it loads quickly for you?

In the Opera browser (which relies on the Chromium):

First time: waited 13s, got message "something went wrong. Try again?"
Second time: 2–3s.
Subsequent times in different browsers: between 2–3s.

There's a variety of tools to support static HTML rendering notebooks in Sphinx docs, including nbsphinx, sphinxcontrib-jupyter and (possibly) jupyter-sphinx.

In the Opera browser (which relies on the Chromium):

First time: waited 13s, got message "something went wrong. Try again?"
Second time: 2–3s.
Subsequent times in different browsers: between 2–3s.

There's a variety of tools to support static HTML rendering notebooks in Sphinx docs, including nbsphinx, sphinxcontrib-jupyter and (possibly) jupyter-sphinx.

Interesting. I'm not sure how they do their caching...
Aside from pre-rendering, another option may be to just link only Google Colab and not GitHub.
Another option I've considered is to move to documentation like FastAPIs, where the code is in separate Python files that are embedded into the docs, and that also get tested as part of the library's tests.

Interesting. I'm not sure how they do their caching...
Aside from pre-rendering, another option may be to just link only Google Colab and not GitHub.
Another option I've considered is to move to documentation like FastAPIs, where the code is in separate Python files that are embedded into the docs, and that also get tested as part of the library's tests.

and that also get tested as part of the library's tests.

nbsphinx runs the notebooks every time you build the documentation (as long as the output cells are empty). I'm sure you can integrate this into your CI runs, and empty output cells would make the diffs a lot easier to work with on GitHub.

and that also get tested as part of the library's tests.

nbsphinx runs the notebooks every time you build the documentation (as long as the output cells are empty). I'm sure you can integrate this into your CI runs, and empty output cells would make the diffs a lot easier to work with on GitHub.

I would like to reduce the diffs for notebooks. I will look into it, thank you for the continued recommendations.

I would like to reduce the diffs for notebooks. I will look into it, thank you for the continued recommendations.

I implemented a docs overhaul in #174 . I initially tried to implement something simple like https://github.com/adriangb/scikeras/issues/130#issuecomment-765541036, but I ran into issues because RTDs workers kept crashing due to the resource usage of our notebooks. So I arrived at another solution that basically consists of:

Storing the notebooks as markdown so that the diffs are kept clean.
In GitHub actions, run all notebooks and build sphinx docs.
Deploy the fully rendered docs to the docs-deploy branch, which is served via GitHub pages.
This includes building preview docs for PRs.

I implemented a docs overhaul in #174 . I initially tried to implement something simple like https://github.com/adriangb/scikeras/issues/130#issuecomment-765541036, but I ran into issues because RTDs workers kept crashing due to the resource usage of our notebooks. So I arrived at another solution that basically consists of:

Storing the notebooks as markdown so that the diffs are kept clean.
In GitHub actions, run all notebooks and build sphinx docs.
Deploy the fully rendered docs to the docs-deploy branch, which is served via GitHub pages.
This includes building preview docs for PRs.

We are now pre-rendering notebooks into HTML. I'm going to close this issue. Thank you @stsievert  for the original ask.


NSFW module, search service, and additional fixes

some code taken from old versions of nadekobot

also don't worry it checks if the channel is an nsfw channel or not

i t i s n o t C A P S

indeed

e621 now supported btw

Hi I see comment box so I make comment wait will everyone see this?


[Feature-8485][task-plugin] add EMR task plugin

Purpose of the pull request
pr for this issue https://github.com/apache/dolphinscheduler/issues/8485

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #8487 (dfa6089) into dev (2a844dc) will decrease coverage by 0.02%.
The diff coverage is 51.81%.

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

@@             Coverage Diff              @@
##                dev    #8487      +/-   ##
============================================
- Coverage     45.41%   45.39%   -0.03%
- Complexity     4032     4046      +14
============================================
  Files           686      691       +5
  Lines         26725    26834     +109
  Branches       2870     2880      +10
============================================
+ Hits          12138    12182      +44
- Misses        13441    13499      +58
- Partials       1146     1153       +7

Impacted Files
Coverage Δ

...olphinscheduler/common/task/emr/EmrParameters.java
0.00% <0.00%> (ø)

...hinscheduler/common/utils/TaskParametersUtils.java
52.17% <0.00%> (-2.38%)
:arrow_down:

...pache/dolphinscheduler/spi/task/TaskConstants.java
0.00% <ø> (ø)

...lphinscheduler/plugin/task/emr/EmrTaskChannel.java
0.00% <0.00%> (ø)

...heduler/plugin/task/emr/EmrTaskChannelFactory.java
0.00% <0.00%> (ø)

...ache/dolphinscheduler/plugin/task/emr/EmrTask.java
64.28% <64.28%> (ø)

...olphinscheduler/plugin/task/emr/EmrParameters.java
66.66% <66.66%> (ø)

...apache/dolphinscheduler/common/enums/TaskType.java
96.00% <100.00%> (+0.16%)
:arrow_up:

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
48.21% <0.00%> (-7.15%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.70% <0.00%> (-2.82%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2a844dc...a6d8394. Read the comment docs.

some dependencies confict need  to check , close


[5.1] Update Builder.php

// an example
$wheres['columnA']= ['=','test']；
$wheres['columnB']= ['<',18];
DB::table('table')->where($wheres)->get()；

Please fix the cs, and add a test.

You've sent two PRs that seem a bit confiusing. Can you please send a single PR with correct CS.


Close button in DatetimePicker when sideBySide is true

There is a close button on sideBySide is false. Can i show close button by sidebyside mode?

You can enforce the toolbar to be showed in sideBySide Mode by setting toolbarPlacement to "top" or "bottom".
This might be a bug as it should also be shown on default value.

Hi, I’ll be closing issues that have been answered by the community.


Extend the structure of how the IRS can handle Policies

As IRS Consumer,
I want to add policies the same way as EDC handles those ,
so that there is no structural change within the overall architecture of where the Services acts.
LOP

@jzbmw  should we use json-ld as the EDC uses that. This might increase the complexity. Odrl: Prefix would not be used if we do not use json-ld . Alternative, we could use the term  odrl:action as a string attribute and ignoring the namespace :odrl

Hints / Details

policy store concept https://confluence.catena-x.net/display/IRS/[TRI-1320][TRI-1512]+IRS%3A+PolicyStore
Example:

{
  "@context": {
      "odrl": http://www.w3.org/ns/odrl/2/
  },
  "@type": "PolicyDefinitionRequestDto",
  "@id": "ap-bpn-and-up-member-trace-idtrace-up",
  "policy": {
      "@type": "Policy",
      "odrl:permission": [
          {
              "odrl:action": "USE",
              "odrl:constraint": {
                  "@type": "LogicalConstraint",
                  "odrl:and": [
                      {
                          "@type": "Constraint",
                          "odrl:leftOperand": "Membership",
                          "odrl:operator": {
                              "@id": "odrl:eq"
                          },
                          "odrl:rightOperand": "active"
                      },
                      {
                          "@type": "Constraint",
                          "odrl:leftOperand": "FrameworkAgreement.traceability",
                          "odrl:operator": {
                              "@id": "odrl:eq"
                          },
                         "odrl:rightOperand": "active"
                      },
                      {
                          "@type": "Constraint",
                          "odrl:leftOperand": "PURPOSE",
                          "odrl:operator": {
                             "@id": "odrl:eq"
                          },
                          "odrl:rightOperand": "ID 3.1 Trace"
                      }
                  ]
              }
          }
      ]
  }
}

Outcome / Acceptance Criteria

Outcome

...

Acceptance Criteria

[ ]  EDC policy schema is used for importing policies into Policy store
[ ]  it shall be additionally possible to add a policy in the schema described above

Out of Scope

...

built uppon json-ld

@ds-alexander-bulgakov ready for QA

@ds-psosnowski as already discussed could you please adjust the insomnia-collection with new JSON-body? Thank you!

Insomnia collection has been extended and previously described issue with missing createdOn has been fixed, see e.g.:

Ticket is done and was accepted in PO-review already.


[rancher/charts] Changes to the requirements.lock pushes unnecessary changes to the generated branch

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
No clear way to reproduce; the error comes up whenever the subcharts that are referred to with wildcard characters (which means, "get me the latest version of the subchart") have released new versions that meet the wildcard's constraints. As a result, the requirements.lock for a chart changes on a helm dependency update that is run on make prepare / make charts / make ci, which results in modifications to the packages/ directory that are committed by the push-to-assets-branch script.
Result:
packages/ shows up in the main repo
Other details that may be helpful:
Take a look at https://github.com/rancher/charts/tree/91a386acbd9e84d5aee47e67ebd7897815e63c29
This doesn't cause a serious impact on any of the charts, but it's something that we should fix to avoid leaving behind unnecessary artifacts in the main branch (or other generated branches in repos like rancher/partner-charts that use these scripts.

Closing this issue out in favor of https://github.com/rancher/rancher/issues/28730#issuecomment-697596235, which should be tested to make sure this issue is addressed.


RFC: Formally document prop function signatures

Issue
Currently, props that take a function do not formally document their signatures.  We have inconsistent/informal function signature documentation, like in Rating onRate:

Not only is this inconsistent, but there is no validation nor linting to aide contribution.  We should start formally documenting these.
Proposal
One approach could be to use standard JSDoc tags, which we are already parsing and using.
/**
 * Called after user selects a new rating.
 * @param {SyntheticEvent} event
 * @param {object} data
 * @param {*} data.rating
 * @param {*} data.maxRating
 * @return {void}
 */
onRate: PropTypes.func,

We could then construct a formal signature in ComponentProps.js with the docs/app/docgenInfo.json in any format.  It could be placed in the description column or type column. It might look something like:
void onRate(
  SyntheticEvent event,
  Object data
  Number data.rating
  Number data.maxRating
)

Details:
We use a custom plugin stardust/gulp/plugins/gulp-react-docgen.js to parse doc blocks on propType and components.  It handles parsing JSDoc tags with doctrine.  All this info is written to a single JSON file, docs/app/docgenInfo.json. There are React components for the docs that read this JSON info in order to generate descriptions, props info, etc.

Would be nice if this was done in a way that allowed us to easily generate TS/Flow definitions.

On that note, I've been considering completely abstracting our props definitions into a plain object.  It has several benefits:

Generate better docs. react-docgen is great, though we've had to extend it already and it still can't detect most of our props types and enums.  We'd have to write custom AST parsers to do this.  A plain object would allow us to generate docs for literally anything with ease.

Component Explorer / Sandbox, see #427.  If we had a single plain object with all the information necessary about props, then we could have a completely automated component explorer.  Currently, we can't merge that since it relies on reading propTypes, which again, are severely lacking support for all our uses.

Generate definitions for anything, propTypes/Flow/TS/etc.  With a plain object that describes the props, it would be trivial to parse it and generate definitions for what ever you wanted.

FWIW, I recently read on a React issue that Facebook has considered/wants to remove PropTypes from React.  Not sure if this means "into a separate package" or simply drop support for it altogether.  In either case, having a plain object abstraction frees us from implementation constraints.

This will be done as a result of #623 which also includes tasks lists.  We will also be able to parse this as a plain object and do with it what we may.  Closing in favor of the other issue.

Many components now document their callback signatures.  I've shipped an update to the docs to display this:


anchor provider depends on fs module

Bundling the anchor package for usage inside a browser-based react app is currently causing build failures. These can be circumvented by configuring webpack, but I think this kind of solutions is far from ideal, as it increases the friction for a first time anchor user. What would we need to change, to prevent the following issue?
I am using roughly the following code to create a new Program inside a react app
import * as anchor from '@project-serum/anchor'
import poolIdl from '../idls/ido_pool'

const provider = new anchor.Provider(connection, wallet, anchor.Provider.defaultOptions())
const programId = new anchor.web3.PublicKey('E5s3D6B3PJinuB9kb3dicxfi3qUNLUGX6hoPawhbqagt')
const program = new anchor.Program(poolIdl, programId, provider);

This will result in the following error:
./node_modules/@project-serum/anchor/dist/esm/provider.js:87:0
Module not found: Can't resolve 'fs'
null

I found a common workaround for this issue in this stack overflow post: https://stackoverflow.com/questions/57161839/module-not-found-error-cant-resolve-fs-in

Agreed this is undesireable and should be fixed.

Does https://github.com/project-serum/anchor/pull/436 close this @armaniferrante?

Does #436 close this @armaniferrante?

I suspect it does but I haven't reproduced this issue to confirm.

I did not test but I think that isBrowser in #436 does not fix the issue. Field browser in package.json for workspaces / NodeWallet should help to resolve issue :thinking:

I think it should fix the fs issue for now? but i'll leave it to you guys to resolve.
Very basic demo of anchor being loaded in React without errors here - https://codesandbox.io/s/floral-star-3sk2w?file=/src/App.tsx

I think it should fix the fs issue for now? but i'll leave it to you guys to resolve.
Very basic demo of anchor being loaded in React without errors here - https://codesandbox.io/s/floral-star-3sk2w?file=/src/App.tsx

Looks resolved to me based on this code sandbox.
Please re-open if it's still an issue.

I can't repro it on sandbox for some reason but this is still causing an exception for me. Does this line have to be gated behind isBrowser too? @armaniferrante

Hmmm, I'm hitting this problem as well @woniesong92. 🤔

Wondering what you did to work around it @woniesong92? That's the line that has issues for me too.

Because I'm using next.js which uses webpack, I simply added this in my next.config.js
module.exports = {
  webpack: (config, { isServer }) => {
    if (!isServer) {
      config.resolve.fallback.fs = false;
    }
    return config;
  },
}

It'll work around it. 😃 Hopefully this helps someone!

I am still seeing issues around this fs moule in nextj even with the above webpack config change. However the error is now pointing to readFileSync
TypeError: __webpack_require__(...).readFileSync is not a function

pointing to this line: https://github.com/project-serum/anchor/blob/cfba251fc4cb366adcd4ad48eefee2201d7448c5/ts/src/provider.ts#L236

I created a minimal repro repo for Next.js here: https://github.com/vpontis/anchor-next-fs-issue
Next.js is a pretty popular React framework and I think Anchor would benefit from working well with Next.js since it would make Solana beginner-friendly.
@armaniferrante, any chance we can re-open this issue?

@vpontis would be awesome if you took a look at this.

I receive this error with React / Gatsby.js:

Not sure if it is related?

the workaround from above works for the dev run, but fails for the prod build. Do you guys have suggestions on how to fix that?

I receive this error with React / Gatsby.js:

Not sure if it is related? If so, I can also provide a repro repo.

Yep still getting this too


Starting new plot not with time but with end of phase 1

Starting new plot (in a queue) not with time delay but with end of phase 1 would be great.

More or less "solved" in issue #36

Yes actually this is pretty much it.


Documentation: Add graceful restart section in BGP documentation

Introduce BGP Graceful Restart documentation in bgp-control-plane.rst document.

Thank you @qmonnet, I have addressed all the comments.

Doc change only, and documentation workflows have passed successfully. This is ready to merge.


4.x Selected items order

Expected behavior and actual behavior
I have a problem with selected items order. They are sorted in alphabetical order, same as dropdown list. Is there an option to put every new selected item to the end of list?
Environment
Browsers

[x] Google Chrome
[x] Mozilla Firefox
[x] Internet Explorer

Operating System

[x] Windows
[x] Mac OS X
[x] Linux
[x] Mobile

Libraries

jQuery version: 1.11.2
Select2 version: 4.0.1

This is a duplicate of https://github.com/select2/select2/issues/3106.


remove "nothing subclass of nothing" tautology

Added --exclude-tautologies to rule for building po.owl. This removed the nothing subclass of nothing tautology from po.owl.
A number of high-level BFO classes were removed after running make release.

@cooperl09 OK if I merge this?


The pre-stop hook should try to push changes to remote

Now we push uncommitted changes to an autosave branch, but perhaps even more important would be to push unpushed changes to origin.
With the current implementation, committed but unpushed changes are lost (which uncommitted changes are pushed to the autosave branch). We need to also save the unpushed commits.

https://github.com/SwissDataScienceCenter/renku-notebooks/pull/164

closed by #174