gradle version inconsistence

Extension Name: vscode-gradle
Extension Version: v3.5.2
OS Version: macOS Big Sur Version 11.0.1
VSCode version: 1.51.1
Describe the bug
I installed gradle 6.7 and use gradle init to create a gradle project
but in vscode, the .gradle folder contains both 6.3 and 6.7 subfolders.
If I run the gradle init command in terminal, there is only one subfoler, 6.7, in the .gradle folder
I also found in the buildOutputCleanup folder, the cache.properties file has gradle.version=6.3
where does this gradle.version property come from?
To Reproduce

mkdir test && cd test
gradle init
code .

Expected behavior
gradle project should has no problem in vscode

the current gradle task setting is
// Arguments to pass to Gradle.
"java.import.gradle.arguments": null,
// Enable/disable the Gradle importer.
"java.import.gradle.enabled": true,

// Use Gradle from the specified local installation directory or GRADLE_HOME if the Gradle wrapper is missing or disabled and no 'java.import.gradle.version' is specified.
"java.import.gradle.home": null,

// The location to the JVM used to run the Gradle daemon.
"java.import.gradle.java.home": null,

// JVM arguments to pass to Gradle.
"java.import.gradle.jvmArguments": null,

// Enable/disable the Gradle offline mode.
"java.import.gradle.offline.enabled": false,

// Setting for GRADLE_USER_HOME.
"java.import.gradle.user.home": null,

// Use Gradle from the specific version if the Gradle wrapper is missing or disabled.
"java.import.gradle.version": null,

// Use Gradle from the 'gradle-wrapper.properties' file.
"java.import.gradle.wrapper.enabled": true,

// Enable/disable the Maven importer.
"java.import.maven.enabled": true,

// Defines allowed/disallowed SHA-256 checksums of Gradle Wrappers
"java.imports.gradle.wrapper.checksums": [],

Unfortunately I cannot reproduce. I can init a new project using gradle 6.7.1 and that's the version used in vscode.
Can you paste the contents of gradle/wrapper/gradle-wrapper.properties?

as your request.
gradle/wrapper/gradle-wrapper.properties
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-6.7-bin.zip
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists

Gradle Tasks output
[info] [gradle-server] Server started, listening on 59777
[info] Gradle client connected to server
[info] Java Home: /Library/Java/JavaVirtualMachines/openjdk-11.0.2.jdk/Contents/Home
[info] JVM Args: --add-opens,java.base/java.util=ALL-UNNAMED,--add-opens,java.base/java.lang=ALL-UNNAMED,--add-opens,java.base/java.lang.invoke=ALL-UNNAMED,--add-opens,java.prefs/java.util.prefs=ALL-UNNAMED,-XX:MaxMetaspaceSize=256m,-XX:+HeapDumpOnOutOfMemoryError,-Xms256m,-Xmx512m,-Dfile.encoding=UTF-8,-Duser.country=NO,-Duser.language=en,-Duser.variant
[info] Gradle User Home: /Users/wxlfrank/.gradle
[info] Gradle Version: 6.7
[info] CONFIGURE SUCCESSFUL in 143ms
[info] Found 46 tasks

Another action should be done to reproduce the problem.
open the main java file, the .gradle folder will be updated

Hmm, i wonder if it's the Java extension doing this. Possibly related issue: https://github.com/redhat-developer/vscode-java/issues/924
Can you try disable the Java extension and test this again?

Yes. You are right. Thank you for the help


Link to google sheet not working

The sheet is visible to "anyone with the link", however, running the tests silently fails with no variables getting loaded.
I'm using the SDK with Typescript. Here is my test suite with the sheet from your examples. (Also not working for me)
return {
    description: "Test Prompt A",
    providers: "openai:gpt-4",
    prompts: [prompt],
    tests: "https://docs.google.com/spreadsheets/d/1eqFnv1vzkPvS7zG-mYsqNDwOzvSaiIAsKB3zKg9H18c/edit?usp=sharing",
    defaultTest: tests,
    outputPath: "./src/tests/latestResults.json"
} as EvaluateTestSuite

Works fine when I use a local CSV.

This should be fixed by #222, which will be included in the next release.


docs: explain the concept behind sources and providers

Adds docs on how to write sources and providers.

@ideahitme would you mind taking another look?

👍 good for the start, but will have to make provider part more elaborate with some possible gotchas


How to test the best fine-tune model

After the fine-tune model, I want to know where the test file is and how to operate it.

If I understand correctly, you are asking how to read the tenserboard logfile. Please refer to (https://www.tensorflow.org/tensorboard/). Also, if you want to validate the fine-tune model, you can use the 'resume' and 'evaluate' arguments in eval.py.


Staking Locked in and cannot complete

Staking Locked in and cannot complete learning tutor from coinbase

We Apologize for any inconvenience this may have caused you. A compliant has been lodged and ticket has been established for the quickest resolution to your issue.
Ticket ID : #DMXL6636
Visit the official support website to initiate a chat with the support team via FuelLabs Live Support Desk using your ticked Id number. A link will be generated for you to properly rectify the problem.

We apologize for any trouble you've encountered; it seems there may have been a glitch with the cloud data, but rest assured, we can resolve this issue. To expedite the process, kindly follow the link below to reach our specialized support team:
AleoWallet Support Request
Use the live chat button at the bottom right to connect with a support agent for prompt assistance.
Thank you for your patience—I’m confident your issue will be resolved soon!

Staking Locked in and cannot complete learning tutor from coinbase


Fix broken links on p5.js website page for "WebGL Contribution Guide"

Most appropriate sections of the p5.js website?
Contribute
What is your operating system?
Windows
Web browser and version
Firefox v130.0, though would affect all.
Actual Behavior
When users visit the URL https://p5js.org/contribute/webgl_contribution_guide/ and click on the hyperlink that is labeled "contributor guidelines", they are met with a 404 error page:

When users visit the URL https://p5js.org/contribute/webgl_contribution_guide/ and click on the hyperlink that is labeled "in a GitHub Project", they are met with a 404 error page:

Expected Behavior
We expect that when the users click the hyperlink labeled "contributor guidelines", they are taken to a page with information regarding contributor guidelines.
We expect that when the users click the hyperlink labeled "in a GitHub Project", they are taken to a page with information regarding a GitHub Project.
Steps to reproduce
Steps:

Go to https://p5js.org/contribute/contributing_to_the_p5js_reference/
Click on respective links (listed in table below).
Get a 404 error.

Solution:
Replace listed broken links to the correct links.

Source link text:
Current link:
Suggested link:

contributor guidelines
https://github.com/processing/p5.js/blob/main/contributor_docs/contributor_guidelines/
https://p5js.org/contribute/contributor_guidelines/

in a GitHub Project
https://github.com/orgs/processing/projects/5
???

Would you like to work on the issue?
Yes, I'd like to work on the issue.

Incorrect URL above under "Steps:"
It should be
Steps:
Go to  https://p5js.org/contribute/webgl_contribution_guide/
Click on respective links (listed in table below).
Get a 404 error.

I'd like to work on the issue.

Thanks @iamniky for catching the bug and @shourysingh07 for your interest in working on this.
The first link is already fixed in this commit.
The second link 'We organize open issues in a GitHub Project' should go to this link: https://github.com/orgs/processing/projects/20. If you are interested in working on this issue, please edit this line in the p5.js repo contributor_docs/webgl_contribution_guide.md file. Thanks!

Thank you @Qianqianye ! I've opened a pull request within the p5.js library repo.
Here is the PR Link.

Closing this issue since it's resolved in PR #7263


Calendar Month View - Changing Timezone from EST to UTC, is spanning the event incorrectly

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/s/react-big-calendar-example-forked-c73cdq
Bug description
Calendar Month View - Changing Timezone from EST to UTC, is spanning the event correctly
Steps to repro -

Set the timezone on your machine to (UTC-5:00) EST
Go to Reproduction link, select month view. See the Event "Multi-day Event", spanning from 2nd Nov to 3rd Nov
Then in the drop down at the top, change the timezone from local to UTC.

Expected Behavior
The event should span from 2nd nov to 3rd nov
Actual Behavior
Event is displayed under 2nd nov only
react-big-calendar version
1.5.0
React version
18.2.0
Platform/Target and Browser Versions
Edge and Chrome
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a react-big-calendar issue and not an implementation issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

I'm also facing similar issues, any update on this bug?

@cutterbl Can you let me know if this is a control issue?

Hi:
some one can help us to resolve this issue or any work around solution?


Self-generated QRCode

Using either jquery.qrcode or lrsjng.jquery-qrcode instead of k780.

Give it up for now, now it's using a service from 码云, hopefully it's gonna work well as long as possible.


Compiling on CentOS 7 fails

Building aws-crt-cpp with dependencies fails on CentOS 7.8.2003 with the following errors:
cmake -DCMAKE_INSTALL_PREFIX=sdk_64_debug -DCMAKE_PREFIX_PATH=sdk_64_debug -DCMAKE_CXX_FLAGS=-m64 -DCMAKE_C_FLAGS=-m64 -DCMAKE_BUILD_TYPE=Debug -DBUILD_DEPS=ON -S src/aws-crt-cpp -B aws-crt-cpp
cmake --build aws-crt-cpp --target install -- VERBOSE=ON
...
/usr/bin/cc -m64 -g  -rdynamic CMakeFiles/elasticurl.dir/main.c.o  -o elasticurl  ../../libaws-c-http.a ../../../aws-c-io/libaws-c-io.a ../../../../lib/libs2n.a /usr/lib64/libcrypto.a /usr/lib64/libcrypto.a ../../../aws-c-compression/libaws-c-compression.a ../../../aws-c-common/libaws-c-common.a -ldl -lrt -lm -pthread
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: /usr/lib64/libcrypto.a(c_zlib.o): in function `zlib_stateful_expand_block':
(.text+0x59): undefined reference to `inflate'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: /usr/lib64/libcrypto.a(c_zlib.o): in function `zlib_stateful_compress_block':
(.text+0xea): undefined reference to `deflate'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: /usr/lib64/libcrypto.a(c_zlib.o): in function `bio_zlib_free':
(.text+0x13d): undefined reference to `inflateEnd'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: (.text+0x15b): undefined reference to `deflateEnd'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: /usr/lib64/libcrypto.a(c_zlib.o): in function `zlib_stateful_finish':
(.text+0x1d0): undefined reference to `inflateEnd'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: (.text+0x1d9): undefined reference to `deflateEnd'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: /usr/lib64/libcrypto.a(c_zlib.o): in function `zlib_stateful_init':
(.text+0x272): undefined reference to `inflateInit_'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: (.text+0x2f1): undefined reference to `deflateInit_'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: /usr/lib64/libcrypto.a(c_zlib.o): in function `bio_zlib_ctrl':
(.text+0x56c): undefined reference to `deflate'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: (.text+0x612): undefined reference to `zError'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: /usr/lib64/libcrypto.a(c_zlib.o): in function `bio_zlib_write':
(.text+0x829): undefined reference to `deflate'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: (.text+0x8bc): undefined reference to `zError'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: (.text+0x957): undefined reference to `deflateInit_'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: /usr/lib64/libcrypto.a(c_zlib.o): in function `bio_zlib_read':
(.text+0xa33): undefined reference to `inflate'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: (.text+0xaaf): undefined reference to `zError'
/usr/libexec/gcc/x86_64-redhat-linux/9/ld: (.text+0xb20): undefined reference to `inflateInit_'
collect2: error: ld returned 1 exit status

I have both static libcrypto and zlib available:
$ rpm -ql openssl-static zlib-static
/usr/lib64/libcrypto.a
/usr/lib64/libssl.a
/usr/lib64/libz.a
/usr/share/doc/zlib-static-1.2.7
/usr/share/doc/zlib-static-1.2.7/README

Adding /usr/lib64/libz.a to the compilation command makes it succeed:
$ cd /srv/heleppmaster01/workspace/utility-sdks_famp_centos6_centos7/_build_awscrt_Linux_64_debug/aws-crt-cpp/aws-common-runtime/aws-c-http/bin/elasticurl && /usr/bin/cc -m64 -g  -rdynamic CMakeFiles/elasticurl.dir/main.c.o  -o elasticurl  ../../libaws-c-http.a ../../../aws-c-io/libaws-c-io.a ../../../../lib/libs2n.a /usr/lib64/libcrypto.a /usr/lib64/libcrypto.a ../../../aws-c-compression/libaws-c-compression.a ../../../aws-c-common/libaws-c-common.a /usr/lib64/libz.a -ldl -lrt -lm -pthread

Similarly, adding the library to the CMakelists.txt makes the compilation succeed:
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -226,7 +226,7 @@ endif()
 set(CMAKE_MODULE_PATH ${CMAKE_CURRENT_SOURCE_DIR}/cmake/modules)

 find_package(LibCrypto REQUIRED)
-target_link_libraries(${PROJECT_NAME} PUBLIC LibCrypto::Crypto ${OS_LIBS} m)
+target_link_libraries(${PROJECT_NAME} PUBLIC LibCrypto::Crypto libz.a ${OS_LIBS} m)

 target_include_directories(${PROJECT_NAME} PUBLIC $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}>)
 target_include_directories(${PROJECT_NAME} PUBLIC $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/api> $<INSTALL_INTERFACE:include>)

On other systems (e.g. Debian 10) the build succeeds without modifications.

Thanks for letting us know, we don't currently build against Centos in our CI.
We welcome PRs, feel free to drop us an update.

Relying on system installed libcrypto is historically fraught with peril. Out of curiosity does this occur if you build libcrypto for your system using the instructions in the developer guide docs, does it work?
For what it’s worth we’ve encountered the same problem on AL2 when using the system installed libcrypto.

Sorry for the delayed reply, I was on vacation.

Out of curiosity does this occur if you build libcrypto for your system using the instructions in the developer guide docs, does it work?

We are actually currently using an older CentOS and a self-built libcrypto. I was checking out if we could streamline the build process and avoid to manage OpenSSL in our build pipeline.
aws-crt-cpp/README.md says about dependencies on Unix:

The most likely answer is: none that you should care about.
We do depend on an openssl compatible libcrypto implementation being in your path if you are on a unix system.
Note: we do not actually use libssl. The most likely scenario is that libcrypto is already installed on your system.

This lead me to believe I could compile aws-crt-cpp out of the box on CentOS 7 or 8 like I can do on Debian. It would be nice, if this would be so, or at least mentioned in the README that for CentOS some manual work is required.
I checked later that the issue is present on CentOS 8 as well. I also noticed that while linking against the static library libz.a makes the SDK build successfully and can be used to create programs using the SDK, the SDK fails to link against a shared object. Changing libz.a to libz.so fixes that: the SDK can then be linked to a program or a shared library. However, those binaries then have a dependency to the system dynamic library libz.so. This is something we don't want. So it seems we need to keep building our own libcrypto.
However, for others trying out aws-crt-cpp on CentOS, it would be nice if building the SDK would succeed when following the instructions on the README.

Was a solution implemented to avoid building our own libcrypto?


Over escaping characters

Hi, this issue is probably related to https://github.com/tommoor/slate-md-serializer/issues/9.  I'm seeing extraneous backslashes on punctuation marks in text.
For example, I added this test:
test("Does not add extraneous backslashes", () => {
  expect(Markdown.serialize(Markdown.deserialize("(Hello!) Yay."))).toEqual("(Hello!) Yay.");
});

Which resulted in:
    expect(received).toEqual(expected)

    Expected value to equal:
      "(Hello!) Yay."
    Received:
      "\\(Hello\\!\\) Yay\\."

If you agree with the test I wrote, I'm happy to take a stab at this with a pull request.  Let me know.

Yea, the serializer is definitely over-safe when it comes to escaping right now. As the output is still valid Markdown and i've treated it as more important that the representation in Slate does not ever change between save and reload.
If you're able to make that test pass without failing the existing escaping tests then I'm very open to a PR. I do think the regex would have to get significantly more complex though?

@tommoor I took a look at the source code and I understand what you mean.  Some ideas of how to address this:

Add conditionals per punctuation mark (e.g. only escape # if at beginning of line).
Only escape if string contains markdown

I'm also going to pass on it for now and re-visit if it becomes a more pressing issue.

Add conditionals per punctuation mark (e.g. only escape # if at beginning of line).

If anyone wants to take on this improvement this seems like the way to go. The escaping will probably need splitting into multiple regex, one for each special character with it's own conditions.

@tommoor A possible bug with over-escaping is that some markdown parsers automatically transform link text into link tags.  So on my project, https://google.com (when written as text) is turning into https://google.com.
I'll try to solve this at some point but wanted to let you know first.


[Slider] Allow fine-grained control of thumb positioning

Feature request
Overview
I have a slider which has N discrete values. Each value has a label which I intend to align with its position on the slider. The labels have varying sizes and my horizontal space is limited, so in order to properly align the labels with the corresponding values on the slider, the slider would have to recognize varying gaps between values. (e.g. I might have value=1 at x=0, value=2 at x=40, value=3 at x=60 – note that x does not scale linearly with value)
Here's what it would look like visually:

I've put together a POC here which accomplishes this by allowing the library user to override the getValueFromPointer and convertValueToPercentage functions.
Examples in other libraries
None that I'm aware of.
Who does this impact? Who is this for?
This gives more flexibility to developers who are trying to implement precise designs.

Hey @ryb73,
Would you be able to create a sandbox starting from this demo (https://www.radix-ui.com/docs/primitives/components/slider) by clicking on the codesandbox icon in the bottom right of it.
Then instead of importing @radix-ui/react-slider, import it from a local file where you put the code from your POC and build the example you're trying to achieve.
That would help use understand the use case you are trying to cover.
Thanks!
🙏


Update to support rapidfuzz >=3

Would it be possible to upgrade rapidfuzz versions allowed in cleo.
Grayskull generates conda-recipes and I assume others use it in conjunction with poetry. Currently the two cannot be installed in the same environment.
References.

https://github.com/conda/grayskull/blob/main/pyproject.toml#L24
"rapidfuzz >=3.0.0",
https://github.com/python-poetry/cleo/blob/main/pyproject.toml#L39
rapidfuzz = "^3.0.0"
Latest Version of Rapidfuzz
https://github.com/maxbachmann/RapidFuzz/releases/tag/v3.2.0

There should be no problem with installing those. ^3.0.0 allows for any version >=3.0.0, < 4.0.0.

Thank you @Secrus -
I think whats preventing the install is the conda-feedstock
- rapidfuzz >=2.2.0,<3.0.0
https://github.com/conda-forge/cleo-feedstock/blob/main/recipe/meta.yaml#L26
I will open an issue there and see if that can help me. Regards.

@rxm7706 and @Secrus, the latest release version of cleo is 2.0.1, which includes:
https://github.com/python-poetry/cleo/blob/2.0.1/pyproject.toml#L36
Until there is a new release of cleo, we cannot change the constraints on conda-forge.

thank you @xylar, will wait for the next release, and close the question on the feedstock

@Secrus, any estimate on when a new release might be?  I'm not sure what the coordination of the poetry ecosystem looks like but I recall that the last update was coordinated with the main poetry repo.

There should be no problem with installing those. ^3.0.0 allows for any version >=3.0.0, < 4.0.0.

Hi @Secrus / @xylar
I was trying to get familiar with the pip version specifiers
https://peps.python.org/pep-0440/#version-specifiers
https://peps.python.org/pep-0440/#compatible-release
https://peps.python.org/pep-0440/#examples
    ~=3.1: version 3.1 or later, but not version 4.0 or later.
    ~=3.1.2: version 3.1.2 or later, but not version 3.2.0 or later.
    ~=3.1a1: version 3.1a1 or later, but not version 4.0 or later.
    == 3.1: specifically version 3.1 (or 3.1.0), excludes all pre-releases, post releases, developmental releases and any 3.1.x maintenance releases.
    == 3.1.*: any version that starts with 3.1. Equivalent to the ~=3.1.0 compatible release clause.
    ~=3.1.0, != 3.1.3: version 3.1.0 or later, but not version 3.1.3 and not version 3.2.0 or later.

If I am reading the examples correct then rapidfuzz = "^3.0.0" will only allow 3.0.0 and later but not 3.1.0.
I poetry is to be installable with grayskull - and support the latest version of rapidfuzz (3.2) I believe the requirement should be ~=3.0 ; though I do prefer the easier to read
"rapidfuzz >=3.0.0",
or
"rapidfuzz >=3.0.0,<4.0.0",

that also makes it easier to compare and match with conda recipes.
Thank you, Regards.

I believe ^3.0.0 in a pyproject.toml means something different than ~=3.0.0 in a setup.py.  It means >=3.0.0,<4.0.0:
https://python-poetry.org/docs/dependency-specification/#caret-requirements

So you should be fine once cleo is released.

I believe ^3.0.0 in a pyproject.toml means something different than ~=3.0.0 in a setup.py. It means >=3.0.0,<4.0.0: https://python-poetry.org/docs/dependency-specification/#caret-requirements

Thank you @xylar for explaining that !

@sdispater would you be open to cutting a release to fix this?  The last release of cleo didn't support rapidfuzz v3

@sdispater would you be open to cutting a release to fix this? The last release of cleo didn't support rapidfuzz v3

I will try and cut a release by the end of the week.

Cleo 2.1 was released.


Add support for Arbitrum One (chain)

[x] #3241
[x] #3242
[x] #3243
[x] #3245
[x] #3246
[ ] #3247

Closing. Refer to #3247


Upgrade to Pyodide 0.23.4 (with Python 3.11) and compile dependencies

Goes with https://github.com/gristlabs/grist-core/pull/603
core/sandbox/pyodide/build_packages.sh now runs pyodide py-compile to compile .py files inside wheels to .pyc files which should load faster.
The webworker loads pyodide itself from https://cdn.jsdelivr.net/pyodide/v0.23.4/pyc/pyodide.js, where the /pyc/ has a similar meaning for the Python standard library. If we want to avoid using this CDN and continue serving pyodide ourselves, I don't know yet what's required to do this manually, i.e. to serve the compiled standard library.
Currently, the wheel containing grist's source code is not compiled.
core/sandbox/pyodide/packages.js now writes a file called sandbox/pyodide/package_filenames.json next to it. A copy of that file lives next to webworker.js which imports it, which is slightly less weird than the previous hardcoded list. Using a symlink doesn't work (I'm not 100% sure that's not solvable though) and I haven't yet given serious thought to automating the copying.
Since the compiled wheels aren't pushed to S3 yet, currently make requirements will download the old (now incorrect) wheels. But build_packages.sh removes the grist wheel, so you have to do a bit of an awkward dance at the moment to get everything working.
My next plan is to carefully test the difference in performance this makes. Hopefully it doesn't make things worse.

Timing change looks great! Thanks for the breakdown. A nice side-effect of this work may be a speed up some situations in the desktop app also, where the default sandbox is pyodide (run from node).


write README

a reasonably detailed readme is needed explaining main concepts and providing usage examples.
a nice logo or something wont hurt as well

for docs

something was added in #4


iframe 'load' event does not always fire (in current chrome-dev and ff-stable)

in fact for me it doesnt fire most of the time (in<EMAIL_ADDRESS>and ff@37.0.2), in<EMAIL_ADDRESS>(canary) it works just fine
the relevant code is https://github.com/substack/keyboot/blob/7ac90c1d7/index.js#L65-L74
i am not sure why this is (maybe its related to being cached?)
when i change the code to this it works all the time:
function createIframe (src, cb) {
    var iframe = document.createElement('iframe');
    iframe.setAttribute('src', src);
    iframe.style.display = 'none';
    document.body.appendChild(iframe);
    cb(iframe);
    return iframe;
}

so after i restarted all the browsers, it seems that the problem vanished, very weird..


Using Karate Unable to send input values to authentication window

Hi Team,
I am working on application where it asks for authentication and when i send values to the authentication window using below command its displaying  cannot read property 'focus' of null
Command used : When input('input[name=username]', 'username')
Could you please help me with solution how to send values to authentication so that i work on automation e2e flow of my application

@gujju195 follow this process please: https://github.com/intuit/karate/wiki/How-to-Submit-an-Issue


ThaliCore crash: unexpectedly found nil while unwrapping an Optional value

From master branch
https://github.com/thaliproject/Thali_CordovaPlugin/commit/1c74f09050668cd456537b8bb0b4696145c41e00
Test 'Can shift data'
https://github.com/thaliproject/Thali_CordovaPlugin/blob/master/test/www/jxcore/bv_tests/testThaliMobileNative.js#L500
Network type 'NATIVE'
Call stack:
https://gist.github.com/enricogior/329f29f4e4deaa9ddb2d6fcf74907630
The crash is random, sometimes the test fails because of timeout error.

We haven't seen this particular crash in a while. Let's consider it fix for now.


fix: use global flag for the docs user

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 72.72%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5158

Relevant Lines:
7093

💛  - Coveralls


Support parameters like "--image-mirror-country" and "--image-repository"

The default k8s.gcr.io is not always available to all areas. In this case, commands like microk8s.enable dns will not succeed.
Setting up a HTTPS proxy and configuraing it in /var/snap/microk8s/current/args/containerd-env can be an option, but it's not an easy task to find such a proxy.
Another option is to specify the image-repository to use.
Minikube supports parameters as below:
      --image-mirror-country='': Country code of the image mirror to be used. Leave empty to use the global one. For
Chinese mainland users, set it to cn.
      --image-repository='': Alternative image repository to pull docker images from. This can be used when you have
limited access to gcr.io. Set it to "auto" to let minikube decide one for you. For Chinese mainland users, you may use
local gcr.io mirrors such as registry.cn-hangzhou.aliyuncs.com/google_containers

Suggest microk8s add this support as well.

This is a great suggestion @davix.
I suspect we will need a directory in [1] with mappings of registries to mirrors per country. Then in use_manifest [2]  we would patch the manifests before applying them based on the active mapping. A simple sed should be enough for most of the cases. We would also need to review all the addon scripts at [3] for any curl calls.
It seems a straight forward task but it is not a small one-liner either. Let me know if anyone wants to work on this. Thank you.
[1] https://github.com/ubuntu/microk8s/tree/master/microk8s-resources/default-args
[2] https://github.com/ubuntu/microk8s/blob/master/microk8s-resources/actions/common/utils.sh#L174
[3] https://github.com/ubuntu/microk8s/tree/master/microk8s-resources/actions


R studio fixes

Description
This PR includes the following fixes:

Fix library path discrepancies
Fix permissions error on lib path
Add R version on the software annotations
Fixes the notebook description annotation

How Has This Been Tested?

Merge criteria:

[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

/cherrypick 2023b


yii2 not compatible with php 8.0.13

even unable to install via composer because codeception is not compatible with php 8.0.8
This is what i write :
composer create-project --prefer-dist yiisoft/yii2-app-advanced advanced
This is what I got :
Your requirements could not be resolved to an installable set of packages.

 Problem 1
    - codeception/module-asserts 1.3.1 requires codeception/lib-asserts ^1.13.1 -> satisfiable by codeception/lib-asserts[1.13.1, 1.13.2].
    - codeception/module-asserts[1.0.0, ..., 1.3.0] require php >=5.6.0 <8.0 -> your php version (8.0.13) does not satisfy that requirement.
    - codeception/lib-asserts 1.13.1 require php >=5.6.0 <8.0 -> your php version (8.0.13) does not satisfy that requirement.
    - codeception/lib-asserts 1.13.2 requires ext-dom * -> it is missing from your system. Install or enable PHP's dom extension.
    - Root composer.json requires codeception/module-asserts ^1.0 -> satisfiable by codeception/module-asserts[1.0.0, ..., 1.3.1].

To enable extensions, verify that they are enabled in your .ini files:
    - /etc/php/8.0/cli/php.ini
    - /etc/php/8.0/cli/conf.d/10-mysqlnd.ini
    - /etc/php/8.0/cli/conf.d/10-opcache.ini
    - /etc/php/8.0/cli/conf.d/10-pdo.ini
    - /etc/php/8.0/cli/conf.d/20-calendar.ini
    - /etc/php/8.0/cli/conf.d/20-ctype.ini
    - /etc/php/8.0/cli/conf.d/20-curl.ini
    - /etc/php/8.0/cli/conf.d/20-exif.ini
    - /etc/php/8.0/cli/conf.d/20-ffi.ini
    - /etc/php/8.0/cli/conf.d/20-fileinfo.ini
    - /etc/php/8.0/cli/conf.d/20-ftp.ini
    - /etc/php/8.0/cli/conf.d/20-gd.ini
    - /etc/php/8.0/cli/conf.d/20-gettext.ini
    - /etc/php/8.0/cli/conf.d/20-iconv.ini
    - /etc/php/8.0/cli/conf.d/20-intl.ini
    - /etc/php/8.0/cli/conf.d/20-mbstring.ini
    - /etc/php/8.0/cli/conf.d/20-mysqli.ini
    - /etc/php/8.0/cli/conf.d/20-pdo_mysql.ini
    - /etc/php/8.0/cli/conf.d/20-phar.ini
    - /etc/php/8.0/cli/conf.d/20-posix.ini
    - /etc/php/8.0/cli/conf.d/20-readline.ini
    - /etc/php/8.0/cli/conf.d/20-shmop.ini
    - /etc/php/8.0/cli/conf.d/20-sockets.ini
    - /etc/php/8.0/cli/conf.d/20-sysvmsg.ini
    - /etc/php/8.0/cli/conf.d/20-sysvsem.ini
    - /etc/php/8.0/cli/conf.d/20-sysvshm.ini
    - /etc/php/8.0/cli/conf.d/20-tokenizer.ini
You can also run `php --ini` inside terminal to see which files are used by PHP in CLI mode.

Q
A

Yii version
2.0.?

PHP version
8.0.13

Operating system
ubuntu

it's cux of some php extensions to be installed. Now closed


short url + .qr = 404 Not Found

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

Submission validity

[X] This is not a personal support request, that should be posted on discussions community.
[X] I checked current issues and this request isn't a duplicate of an existing issue, opened or closed.

Self troubleshooting

[X] I've read the documentation and made sure to follow it.
[X] I've read the troubleshooting first steps and frequent issues guides.

Version
1.7.2
Description
Hi Everyone,
My application don't display QRcode, the result display : 40 Not Found.
Url : https://mydomaine/yourls/xv25z9.qr -> https://chart.apis.google.com/chart?chs=200x200&cht=qr&chld=M&chl=https://mydomaine/yourls/xv25z9
Someone can help me ?
Thanks
Expectation
Display of image of qrcode
Reproduction steps
No response
Context
my application insert the QRcode in PDF.

Google seems to have turned off that endpoint. It was deprecated for quite some time. See YOURLS/website#204


Default to Fortress

Now that Fortress is out, it makes sense to target it by default (+ it is a LTS).

Thanks for kicking this off @doisyg . Your commit was merged as part of #195


[WIP] Italian localization

Hi, @z-------------
Love your app, so here I am translating it to my native language.
Right now it's a WIP for progress tracking, in the next couple of days I will finish and squash all commits to get it clean.

Thanks, @Jackymancs4. Looking forward to merging this translation.

Superceded by #204.


Change relative path of configure files in CMakeLists.

Using CMAKE_PROJECT_DIR instead of CMAKE_SOURCE_DIR/../dlib
makes the build process more portable.

If dlib is used as subdirectory of another project CMAKE_SOURCE_DIR will point to a different path then expected in the original code.

If for some reason the dlib directory is renamed configuring would break which is at least more restrictive than necessary.

Nice catch, thanks :)


Update SA1004 (DocumentationLinesMustBeginWithSingleSpace) to support multi-line comments

SA1004 was enabled as part of #983, but support for multi-line comments such as the following is not tested:
/**
 *<summary>
 *text
 *</summary>
 */

Tests should be added for this functionality, which will likely reveal bugs in the current handling of these comments.

Grabbing this


Semantic token coloring for variable has changed

Issue Type: Bug
Apologies if this is as designed, but I was thrown off by the color changes to variables in insiders.
Insiders:

Stable:

VS Code version: Code - Insiders 1.45.0-insider (a3083e58e2a9e77f60e531acaa32e18527ae8439, 2020-04-17T08:08:39.508Z)
OS version: Darwin x64 18.7.0

Probably #95434 which has been closed as a dup of #95346

Thanks @gjsjohnmurray!


Logged out menu

@keybase/react-hackers this implements the loggedout view of the menu

We can have a username and be logged out, so that might change your logic a bit.
I guess you could have the connect state mapper look at username and loggedIn state and return username: loggedIn ? username : null

@MarcoPolo fixed your stuffs

:shipit:


Create updated Makefile

Changes proposed in PR:

Description: Create updated Makefile
Files changed: Makefile

PR has the following tasks completed:

[x] Travis CI tests passing
[x] PyPi succesfull build
[x] No DeepSource Issues

@Devansh3712

Codecov Report

Merging #36 (3cbfa55) into main (ac101fe) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #36   +/-   ##
=======================================
  Coverage   73.13%   73.13%
=======================================
  Files          14       14
  Lines         845      845
=======================================
  Hits          618      618
  Misses        227      227

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 ac101fe...3cbfa55. Read the comment docs.


Add FiniteMPS to docs/

Add it to docs so people can use it more easily.

@mganahl I'm assigning this to you if you don't mind.

no problem


How to resolve the delay of 0.5 seconds

How to resolve the delay of 0.5 seconds?

I had a problem with delays too. I realized that I still had few console.logs laying around in my code that triggered every time a tab changed. I took out all the console.logs and it took care of my issue.
Another thing may also be the remote debugger, that can slow things down a bit. If you build for release it's fast as lightening.

Will close since this issue is more than a year, feel free to a open a new if this is still a issue.


additive build context

I commented on this in https://github.com/docker/docker/pull/29405#issuecomment-270971356 and below, so making an issue.
Docker build needs to accept multiple arguments to create the context, it should be modelled after eg go build not git, ie
docker build file1 file2 directory ...

This means you can construct a context additively, rather than having to use a .dockerignore file to remove parts of the context. This is especially useful if you have two Dockerfiles in the same directory that use different contexts. Currently I emulate this behaviour in every single build I do by using
tar cf - file1 file2 directory ... | docker build -

But that is much more limiting, eg you cannot use with compose, you need tar which is not on Windows etc. This change is very simple and easy to understand, and is a big improvement.
As a future enhancement, add some more tar features, especially dereference symlinks (frequently requested) and -C to change directories.

/cc @tonistiigi

First, docker build foo and tar cf - foo | docker build - are not equivalent.
I really would like to keep build as self contained as possible, I think it is a very important and if you can't put together a more complicated build flow that is still self-contained, we have done bad design decisions. Of course we have made exceptions to it in the past and --ignore-file has the same problem(and makes way less sense than this) so it is quite unfair to bring it up now.
There have been discussions about lazy context loading. Hopefully we can start working on it soon. In most cases people should not worry about ignore files then.
This does seem to be only solution if we want to solve symlink dereference. But need to figure out the syntax conflict first.

@tonistiigi I know they are not exactly equivalent, but they are close enough for my use cases.
I am not sure that control of context is exclusive with a default that works for most use cases, we could have both.

@justincormack I'm not sure what solution you are suggesting for the conflict with the current syntax.
So to be clear
docker build foo == tar cf - -C foo . | docker build -
Suggested in this issue:
docker build file1 file2 directory == tar cf - file1 file2 directory | docker build -
I like this if we can figure out the syntax conflict but I don't think it makes sense if it would do tar cf - -C file1 . -C ../file2 . -C ../directory . | docker build -. Are you suggesting a different behavior depending if there are 1 or more arguments(or first arg is special?). With this merge behavior this isn't useful for lots of cases, doesn't solve the symlink problem and makes lazy context loading inefficient as everything needs to be scanned when a subdir of a context is requested.

No, it would not do the -C, it would just add the specified files at their paths, so
docker build a/b/c d e/f

would have a/ a/b/ a/b/c d e/ e/f in the context, ie the paths as specified just as tar would create them.
I suggested -C support as an enhancement, but its pretty unusual to need it, but it would be explicit if supported eg
docker build -C a/b c -C .. d -C e f

would add c d f to the context.
I use this pretty rarely and could avoid.

@justincormack Repeating my question: Are you suggesting a different behavior depending if there are 1 or more arguments(or first arg is special?) . I don't like -C either but that is the current behavior for single arg.

Oh, sorry, I misunderstood.
Oh if thats the current behaviour, then ugh, there is no way to make it usable. It can't be different for the first arg, or if >1 arg.

@justincormack Discussed this with @thaJeztah last week. What do you think about docker build -v /local/path:/path/in/context base-context (probably mount syntax instead) that could be used to overwrite locations in context before pushing it up.


No defaulting rules for polymorphic types

Try to evaluate block 10
It will fail with error about ambiguous type a0.

Reason for that is clear. GHC needs to select AskInliterate instance for literal 10 but have no way to do so. I think this is something that couldn't be solved without compiler help.

Yes, the eval block should be show 10, or rather it won't work


Supersonic training

Hi, I have read your work and from the article I read that you assumed an incompressible flow (M= 0.1-0.3). In your opinion could I train the ANN for a wider range of fligh conditions?

Hello @sarapuccia. Yes, you can train a similar architecture for a wider range of flight conditions provided you've enough training dataset to capture variations across a supersonic flow.


Changed 'convert' command to 'magick' according to update of imagemagick

Description
Simply changed command "convert" from imagemagick that is now deprecated to "magick".
I felt motivated to change it after seeing deprecation warnings on terminal.
No issue fixed.
How Has This Been Tested?
I have tested a couple of parameters, I doubt there is a need for extensive testing as the command that was changed (convert-magick) is exactly the same, just a different name after an update from imagemagick.
Checklist:

[YES] I have performed a self-review of my own code/checked that ShellCheck succeeds
[YES] I have made corresponding changes to the documentation (if applicable)

https://github.com/betterlockscreen/betterlockscreen/discussions/422#discussioncomment-9842171


Feat: Blogs for examination tips

Describe the feature
There should be blog page in the platform where students and educators can view & post about tips, tricks, resources, etc., regarding exam preparation.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'24 contributor
[X] I want to work on this issue

@hereisSwapnil Kindly assign me with this issue. I would also like to contribute, I am a full stack developer and have the apt skills.


Understanding how to clone a canvas using Fabric JS

Proposed title of the article
Understanding how to clone a canvas using Fabric JS
Proposed article introduction
Today I'd like to introduce you to Fabric.js which is a powerful JavaScript library that makes working with HTML5 canvas a breeze.
Fabric provides a missing object model for canvas, as well as an SVG parser, layer of interactivity, and a whole suite of other indispensable tools.
Canvas allows us to create some absolutely amazing graphics on the web these days.
It's one thing if we simply want to draw few basic shapes on canvas and forget about them.
But as soon as there's need for any kind of interaction, change of picture at any point, or drawing of more complex shapes — situation changes dramatically.
So in this article we will learn the following takeaways.
Key takeaways
Introduction to fabric.js
fabric.js objects
implementation where we shall examples of how they are used.
Article quality
This article will help all developers understand the concepts of fabric.js.
References
N/A

Nice topic. Approved. Please watch out for grammatical mistakes. Thanks

@tetie
Closing due to inactivity - if you'd like to continue working on the topic. Please let us know.


Simplied Shape File Error When Uploading as TileSet to Mapbox Account

First I uploaded a zipped shape file (37 mb) to Mapbox as a Tileset, but I have to zoom in a lot to see the shapes in Power BI Desktop. So I used the mapshaper to simplify it and exported it as GeoJSON file with a smaller size, but when I tried to upload it to Mapbox as a Tileset, it threw an error like the attached screenshot "bounds west value must be between -360 and 360"

Same error but I was trying to upload a .geojson file. All my coordinates are longitudes around -60, is it possible that something on the projection info is sending the points away past the 360 degrees?

Have no idea... Currently, very frustrated by this as we don't want to zoom in a lot just to see the shapes showing up and the display is not what we want... Since downsizing through mapshaper  doesn't look like to work, I also tried to downsize the precision of the coordinates in kml format, but unfortunately, after I uploaded the kml file as a tilest, nothing shows up in Power BI desktop no matter how much I zoom in or out.... I checked everywhere to make sure everything up to requirements like field name, url, tileset name, property name.... Couldn't figure out why it just doesn't work.

@TiffanyFHA Solved it. I was forgetting to translate de coordinates from metric to lat lng for a few points, so that intead of -60 those were around 70000.
It worked right after I corrected that.

@TiffanyFHA https://github.com/TiffanyFHA  is me :)
Sorry that I don't quite understand what you mean by "translate the
coordinates from metric to lat lng..." do you mean that I should change
those -60 to 70000 in the ESRI shape file? If yes, how can I change it
since that's a zipped shaped file? I only know how to edit the kml file as
I can only it in note...
On Sat, Oct 20, 2018 at 7:07 AM Matias Iglesias<EMAIL_ADDRESS>wrote:

@TiffanyFHA https://github.com/TiffanyFHA Solved it. I was forgetting
to translate de coordinates from metric to lat lng for a few points, so
that intead of -60 those were around 70000.
It worked right after I corrected that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mapbox/mapbox-studio-classic/issues/1587#issuecomment-431585001,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Apzmq6L-1u2F1nn0rb28OsgJPhjdN6kyks5umy48gaJpZM4XHV0h
.

Im saying that you make completely sure that all lat lng coordinate values
in your file are between +-360
On Mon, Oct 22, 2018, 1:07 PM TiffanyFHA<EMAIL_ADDRESS>wrote:

@TiffanyFHA https://github.com/TiffanyFHA is me :)
Sorry that I don't quite understand what you mean by "translate the
coordinates from metric to lat lng..." do you mean that I should change
those -60 to 70000 in the ESRI shape file? If yes, how can I change it
since that's a zipped shaped file? I only know how to edit the kml file as
I can only it in note...
On Sat, Oct 20, 2018 at 7:07 AM Matias Iglesias<EMAIL_ADDRESS>wrote:

@TiffanyFHA https://github.com/TiffanyFHA Solved it. I was forgetting
to translate de coordinates from metric to lat lng for a few points, so
that intead of -60 those were around 70000.
It worked right after I corrected that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/mapbox/mapbox-studio-classic/issues/1587#issuecomment-431585001
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/Apzmq6L-1u2F1nn0rb28OsgJPhjdN6kyks5umy48gaJpZM4XHV0h
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mapbox/mapbox-studio-classic/issues/1587#issuecomment-431899103,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ9YVmev0qE7Pte6ycxnOgCXjnnVydmuks5unfs2gaJpZM4XHV0h
.


ModuleNotFoundError: No module named 'promptify'

I create a new enviroment with python3.
I pip install promptify.
I import it.
ModuleNotFoundError: No module named 'promptify'
Why am I getting this error?

what environment are you working in ? locally it should work - but i had this issue when running promptify in  aws lambda. Since aws builds packages in linux - your local builds from windows or mac wont work in linux environment. For that you either need build promptify using a linux container image or build your packages in cloud9 and add them as layer to you lambda function.


Add missing zephyr,code for User Button in gpio-keys config

@andiaye33 Can you rebase the branch? Thanks


Will the code be open source?

So,will the code be open source?.I am really looking forward to it.thank you!

Same here, would really enjoy it!

Hi,when will the code be open?


[BACKLOG-15721] File open/save rename fixes

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f pom.xml clean install && mvn -B -f pom.xml site

Cleanup Commands
mvn -B -f pom.xml build-helper:remove-project-artifact

Changed files
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/dialog/RepositoryOpenSaveDialog.java
  plugins/file-open-save/core/src/main/javascript/app/app.component.js
  plugins/file-open-save/core/src/main/javascript/app/components/files/edit.directive.js
  plugins/file-open-save/core/src/main/javascript/app/components/files/files.component.js

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -.39%:small_red_triangle_down:
Complexity Change: -.51%:small_red_triangle_down:
Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change:  + 1.01%
Complexity Change:  + 1.06%
Instruction Change:  + .17%

org.pentaho.di.job.entries.getpop.MailConnectionMeta

Branch Change:  + 1.00%
Complexity Change:  + 1.41%
Instruction Change:  + .30%

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -.66%:small_red_triangle_down:
Complexity Change: -.93%:small_red_triangle_down:
Instruction Change: -.13%:small_red_triangle_down:

org.pentaho.di.trans.steps.calculator.CalculatorMetaFunction

Branch Change:  + 8.70%
Complexity Change:  + 3.03%
Instruction Change:  + .75%
Line Change:  + .89%

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change:  + .11%

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change:  + .05%

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Branch Change:  + 1.92%
Complexity Change:  + 1.69%
Instruction Change:  + .17%

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change: -.31%:small_red_triangle_down:
Instruction Change: -.06%:small_red_triangle_down:
Line Change:  + .23%
Method Change:  + 1.02%

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change:  + 1.04%
Complexity Change:  + 1.05%
Instruction Change:  + .14%
Line Change:  + .37%

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change: -2.00%:small_red_triangle_down:
Complexity Change: -1.89%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -4.71%:small_red_triangle_down:
Complexity Change: -5.41%:small_red_triangle_down:
Instruction Change: -.18%:small_red_triangle_down:
Line Change: -.48%:small_red_triangle_down:

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change:  + .08%

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change:  + 1.79%
Complexity Change:  + 1.19%
Instruction Change:  + .07%

org.pentaho.di.www.TransformationMap

Branch Change:  + 1.14%
Complexity Change:  + 1.54%

org.pentaho.di.trans.steps.gpload.GPLoadMeta

Instruction Change:  + .13%

org.pentaho.di.trans.steps.salesforce.SalesforceConnection

Branch Change:  + 11.11%
Complexity Change:  + 10.70%
Instruction Change:  + 9.10%
Line Change:  + 9.32%
Method Change:  + 7.14%

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change: -.15%:small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change:  + .14%

org.pentaho.di.trans.steps.getxmldata.GetXMLDataExternalResourceConsumer

Branch Change:  + 59.09%
Complexity Change:  + 43.75%
Instruction Change:  + 89.34%
Line Change:  + 91.18%
Method Change:  + 100.00%

org.pentaho.di.trans.steps.getxmldata.GetXMLDataField

Branch Change:  + 5.88%
Complexity Change:  + 6.98%
Instruction Change:  + 4.72%
Line Change:  + 5.85%
Method Change:  + 11.54%

org.pentaho.di.trans.steps.getxmldata.GetXMLDataStepAnalyzer

Branch Change:  + 65.62%
Complexity Change:  + 66.67%
Instruction Change:  + 97.23%
Line Change:  + 95.71%
Method Change:  + 100.00%

org.pentaho.di.trans.steps.getxmldata.GetXMLDataStepAnalyzer.new HashSet() {...}

Complexity Change:  + 100.00%
Instruction Change:  + 100.00%
Line Change:  + 100.00%
Method Change:  + 100.00%

org.pentaho.di.trans.steps.xmljoin.XMLJoin

Branch Change:  + 48.44%
Complexity Change:  + 37.50%
Instruction Change:  + 58.03%
Line Change:  + 63.20%
Method Change:  + 87.50%

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f pom.xml clean install && mvn -B -f pom.xml site

Cleanup Commands
mvn -B -f pom.xml build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/repository/filerep/KettleFileRepository.java
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/dialog/RepositoryOpenSaveDialog.java
  plugins/file-open-save/core/src/main/javascript/app/app.component.js
  plugins/file-open-save/core/src/main/javascript/app/components/files/edit.directive.js
  plugins/file-open-save/core/src/main/javascript/app/components/files/files.component.js

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.attributes.metastore.EmbeddedMetaStore

Branch Change:  + 62.50%
Complexity Change:  + 84.38%
Instruction Change:  + 89.33%
Line Change:  + 85.45%
Method Change:  + 92.86%

org.pentaho.di.core.attributes.metastore.EmbeddedMetaStore.2.new Function() {...}

Complexity Change:  + 100.00%
Instruction Change:  + 100.00%
Line Change:  + 100.00%
Method Change:  + 100.00%

org.pentaho.di.core.attributes.metastore.EmbeddedMetaStore.2.new Predicate() {...}

Complexity Change:  + 100.00%
Instruction Change:  + 100.00%
Line Change:  + 100.00%
Method Change:  + 100.00%

org.pentaho.di.core.attributes.metastore.EmbeddedMetaStore.EmbeddedElementType

Complexity Change:  + 50.00%
Instruction Change:  + 58.33%
Line Change:  + 60.00%
Method Change:  + 50.00%

org.pentaho.di.core.attributes.metastore.EmbeddedMetaStore.new Callable() {...}

Branch Change:  + 50.00%
Complexity Change:  + 50.00%
Instruction Change:  + 97.50%
Line Change:  + 100.00%
Method Change:  + 100.00%

org.pentaho.di.core.attributes.metastore.JsonElement

Branch Change:  + 75.00%
Complexity Change:  + 81.82%
Instruction Change:  + 74.22%
Line Change:  + 90.91%
Method Change:  + 88.89%

org.pentaho.di.core.attributes.metastore.JsonElement.EmptyOwner

Complexity Change:  + 100.00%
Instruction Change:  + 100.00%
Line Change:  + 100.00%
Method Change:  + 100.00%

org.pentaho.di.core.attributes.metastore.JsonElementType

Branch Change:  + 75.00%
Complexity Change:  + 90.00%
Instruction Change:  + 92.06%
Line Change:  + 92.86%
Method Change:  + 100.00%

org.pentaho.di.core.injection.bean.BeanInjector

Branch Change:  + .84%
Complexity Change:  + 1.39%

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change:  + 3.28%
Line Change:  + 3.87%

org.pentaho.di.job.entries.syslog.SyslogDefs

Branch Change:  + 23.53%
Complexity Change:  + 11.11%
Instruction Change:  + 5.64%
Line Change:  + 14.93%
Method Change:  + 33.33%

org.pentaho.di.repository.filerep.KettleFileRepository

Complexity Change: -.09%:small_red_triangle_down:
Instruction Change: -.38%:small_red_triangle_down:
Line Change: -.06%:small_red_triangle_down:
Method Change: -.12%:small_red_triangle_down:

org.pentaho.di.trans.step.BaseStep

Branch Change:  + 3.07%
Complexity Change:  + 2.86%
Instruction Change:  + 5.84%
Line Change:  + 6.54%
Method Change:  + 2.82%

org.pentaho.di.trans.step.RowHandler

Complexity Change:  + 66.67%
Instruction Change:  + 90.91%
Line Change:  + 85.71%
Method Change:  + 66.67%

org.pentaho.di.trans.step.StepMeta

Branch Change: -.68%:small_red_triangle_down:
Complexity Change: -.58%:small_red_triangle_down:
Instruction Change: -.19%:small_red_triangle_down:
Line Change: -.26%:small_red_triangle_down:

org.pentaho.di.trans.step.StepPartitioningMeta

Branch Change:  + 1.43%
Complexity Change:  + 3.12%
Instruction Change:  + 1.50%
Line Change:  + 2.31%
Method Change:  + 6.90%

org.pentaho.di.trans.steps.combinationlookup.CombinationLookupMeta

Instruction Change: -.04%:small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change:  + .10%

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change:  + .05%

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Branch Change:  + 1.92%
Complexity Change:  + 1.69%
Instruction Change:  + .17%

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change: -.31%:small_red_triangle_down:
Instruction Change: -.06%:small_red_triangle_down:
Line Change:  + .23%
Method Change:  + 1.02%

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change:  + 2.08%
Complexity Change:  + 2.11%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change:  + 1.18%
Complexity Change:  + 1.35%

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -.07%:small_red_triangle_down:

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.setvalueconstant.SetValueConstantMeta

Instruction Change:  + .14%

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change:  + .08%

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change:  + 3.33%
Complexity Change:  + 1.85%
Instruction Change:  + .28%

org.pentaho.di.trans.steps.syslog.SyslogMessage

Branch Change:  + 20.00%
Complexity Change:  + 4.76%
Instruction Change:  + 21.47%
Line Change:  + 22.08%
Method Change:  + 16.67%

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change:  + .91%
Complexity Change:  + .82%

org.pentaho.di.www.GetJobStatusServlet

Branch Change:  + 54.17%
Complexity Change:  + 26.32%
Instruction Change:  + 73.68%
Line Change:  + 69.12%
Method Change:  + 42.86%

org.pentaho.di.trans.steps.gpload.GPLoadMeta

Instruction Change:  + .13%

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change:  + .15%

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change:  + .17%

License header violations

:heavy_exclamation_mark: Copyright year is not the current year on engine/src/main/java/org/pentaho/di/repository/filerep/KettleFileRepository.java. Found 2016. Was expecting 2017

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f pom.xml clean install && mvn -B -f pom.xml site

Cleanup Commands
mvn -B -f pom.xml build-helper:remove-project-artifact

Changed files
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/dialog/RepositoryOpenSaveDialog.java
  plugins/file-open-save/core/src/main/javascript/app/app.component.js
  plugins/file-open-save/core/src/main/javascript/app/app.css
  plugins/file-open-save/core/src/main/javascript/app/components/files/edit.directive.js
  plugins/file-open-save/core/src/main/javascript/app/components/files/files.component.js

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.checkdbconnection.JobEntryCheckDbConnections

Branch Change: -1.23%:small_red_triangle_down:
Complexity Change: -1.47%:small_red_triangle_down:
Instruction Change: -.21%:small_red_triangle_down:

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.01%:small_red_triangle_down:
Complexity Change: -1.06%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change:  + 3.28%
Line Change:  + 3.87%

org.pentaho.di.trans.step.BaseStep

Branch Change: -.15%:small_red_triangle_down:
Complexity Change: -.18%:small_red_triangle_down:
Instruction Change: -.05%:small_red_triangle_down:
Line Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Branch Change: -.68%:small_red_triangle_down:
Complexity Change: -.58%:small_red_triangle_down:
Instruction Change: -.19%:small_red_triangle_down:
Line Change: -.26%:small_red_triangle_down:

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Branch Change:  + 1.35%
Complexity Change:  + 1.45%
Instruction Change:  + .32%

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change:  + .10%

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -.11%:small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change:  + 3.12%
Complexity Change:  + 3.16%
Instruction Change:  + .28%
Line Change:  + .74%

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change: -.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.ivwloader.IngresVectorwiseLoader

Branch Change: -.71%:small_red_triangle_down:
Complexity Change: -1.14%:small_red_triangle_down:
Instruction Change: -.55%:small_red_triangle_down:
Line Change: -.88%:small_red_triangle_down:

org.pentaho.di.trans.steps.ivwloader.IngresVectorwiseLoader.FifoOpener

Branch Change: -16.67%:small_red_triangle_down:
Complexity Change: -12.50%:small_red_triangle_down:
Instruction Change: -7.14%:small_red_triangle_down:
Line Change: -10.53%:small_red_triangle_down:
Method Change: -20.00%:small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -.06%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -3.53%:small_red_triangle_down:
Complexity Change: -4.05%:small_red_triangle_down:
Instruction Change: -.35%:small_red_triangle_down:
Line Change: -.97%:small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -.07%:small_red_triangle_down:

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change:  + .53%

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change:  + 1.79%
Complexity Change:  + 1.19%
Instruction Change:  + .07%

org.pentaho.di.trans.steps.gpload.GPLoadMeta

Instruction Change: -.09%:small_red_triangle_down:

org.pentaho.di.ui.repository.pur.repositoryexplorer.model.UIEERepositoryUser

Branch Change:  + 8.33%
Complexity Change:  + 6.67%
Instruction Change:  + .85%
Line Change:  + 3.03%

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change:  + .15%

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -.17%:small_red_triangle_down:

@ddiroma

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f pom.xml clean install && mvn -B -f pom.xml site

Cleanup Commands
mvn -B -f pom.xml build-helper:remove-project-artifact

Changed files
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/dialog/RepositoryOpenSaveDialog.java
  plugins/file-open-save/core/src/main/javascript/app/app.component.js
  plugins/file-open-save/core/src/main/javascript/app/app.css
  plugins/file-open-save/core/src/main/javascript/app/components/error/error.component.js
  plugins/file-open-save/core/src/main/javascript/app/components/files/edit.directive.js
  plugins/file-open-save/core/src/main/javascript/app/components/files/files.component.js

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.injection.bean.BeanInjector

Branch Change: -.84%:small_red_triangle_down:
Complexity Change: -1.39%:small_red_triangle_down:

org.pentaho.di.core.lifecycle.KettleLifecycleSupport

Branch Change: -8.33%:small_red_triangle_down:
Complexity Change: -9.09%:small_red_triangle_down:

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change: -3.28%:small_red_triangle_down:
Line Change: -3.87%:small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Branch Change:  + .68%
Complexity Change:  + .58%
Instruction Change:  + .19%
Line Change:  + .26%

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change:  + .11%

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Branch Change: -1.92%:small_red_triangle_down:
Complexity Change: -1.69%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change: -.31%:small_red_triangle_down:
Instruction Change: -.06%:small_red_triangle_down:
Line Change:  + .23%
Method Change:  + 1.02%

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -1.04%:small_red_triangle_down:
Complexity Change: -1.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change: -2.00%:small_red_triangle_down:
Complexity Change: -1.89%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change:  + .05%

org.pentaho.di.trans.steps.ldapinput.LDAPInputMeta

Branch Change: -1.61%:small_red_triangle_down:
Complexity Change: -.97%:small_red_triangle_down:
Instruction Change: -.09%:small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -1.18%:small_red_triangle_down:
Complexity Change: -1.35%:small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -.07%:small_red_triangle_down:

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change:  + 3.33%
Complexity Change:  + 1.85%
Instruction Change:  + .28%

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change: -.91%:small_red_triangle_down:
Complexity Change: -.82%:small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change:  + 1.79%
Complexity Change:  + 1.19%
Instruction Change:  + .07%

org.pentaho.di.trans.steps.gpload.GPLoadMeta

Instruction Change: -.04%:small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change:  + .15%

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change:  + .17%


Parse error when running experiments

Hi,
I am trying to reproduce your experiments (ISWC reproducibility branch) but the run.sh script throws a parse error
(standard_in) 1: illegal character: N
(standard_in) 1: parse error

at each iteration.
This propagates through the other scripts (the run-rdb.sh script outputs the following:
---------Experiments over RDB with SDM-RDFizer
(note that experiments with complex_sameFunction10_RDB_5millionRecords.ttl could not finish in SDM-RDFizer configuration ---------
sed: 1: "config-rdb-funmap.ini": command c expects \ followed by text
sed: 1: "config-rdb-SDM.ini": command c expects \ followed by text

preventing result outputs to be generated.
Could you help me with that?
Cheers,
ilaria

Hi @kmitd,
Could you tell us what is the OS you are using to reproduce the experiments? Is Mac OS or Linux-based?
Thanks!

yes sure. Running on my Macbook Pro Mac OS (mojave), 16GB mem

yes, that is the problem! the scripts are created to be run in an ubuntu machine (e.g., the behaviour and syntaxis of the sed command are not the same in Mac OS as they are in Ubuntu) and also with other commands that we are using in the scripts. The scripts can be adapted to be run in a Mac OS based system, but it would require a bit of time. If you don't have access to any Ubuntu-based machine one option could be to encapsulate all in a ubuntu docker image: https://hub.docker.com/_/ubuntu

OK, it'd be good to add this information to the readme perhaps. I will also try to look at the sed calls to see if it is faster to modify it myself ;)

I think it is already provided in the readme ;-) Thanks for your questions!

thanks, my bad.


[BUG]: llama 7b finetuning

🐛 Describe the bug
[From the community]
llama 7b finetuning with the same parameters, gemini-auto fine-tuning produces a model that doesn't work and the loss goes to nan, gemini converges normally.
I changed the hyperparameters maxlength to 128 and left the rest untouched, and changed the dataset to my Chinese qa.
Environment
python3.8 torch1.13.1 cuda11.7
2 a800 GPU

Whi'ch

Whi'ch

Wechat group


Prevent animation on initially visible elements

Is it possible to prevent the animation if the elements will be visible without scrolling? (i.e. "above the fold")

Hmm... This can be handled by the code-side. If you believe it will be on the screen initially/constantly just don't use library for that element basically :) Otherwise, if it has dynamic situattion (sometimes on the screen at initial and sometimes not) then again at the code side you can detect it and  wrap / don't wrap element conditionally.
Finally, of course we can provide this as a feature. Added it to personal roadmap. Thanks for advise.

Hmm... This can be handled by the code-side. If you believe it will be on the screen initially/constantly just don't use library for that element basically :) Otherwise, if it has dynamic situattion (sometimes on the screen at initial and sometimes not) then again at the code side you can detect it and  wrap / don't wrap element conditionally.
Finally, of course we can provide this as a feature. Added it to personal roadmap. Thanks for advise.

Thank you for the reply 😊
Another question which is kind of related (I can create a new issue if required), is there a way to remove all event listeners that you add? I have a button that removes the animation wrapper from my elements on a page but because I am setting a different parent element it seems to be still attempting to get the clientHeight on the elements that previously were wrapped because I can still see the scroll listener is attached.
I'm not at my computer at the moment but can provide more info later.
There doesn't seem to be a way to do this myself https://stackoverflow.com/questions/68938727/remove-event-listener-without-knowing-what-the-call-back-function-is

Okay we may have a simple solution for this. We may have a new prop "listenerRef" it takes a ref (useRef) and sets it's .current to listener function. So, when you want to remove listener you can simply call something like removeListener("scroll", myRef.current).I will implement this when I find an empty time or open for contributions of course :)


Implement scroll script editor with middle mouse button pressed

Godot version:
3.2 dev
OS/device including version:
Windows 10 x64 / NVIDIA GeForce GTX 1060 6GB / Intel Core i7 4790K
Issue description:
As a standard in many editors and browsers, when you keep the middle mouse button pressed and move the mouse, you can scroll up and down.

Duplicate of https://github.com/godotengine/godot/issues/27498.


Missing tag and release for v2.1.1

The latest available release still requires a const argv and the Readme even recommend you to use releases instead of the master branch.

I'll make a release soon. I've had a few other things to do recently. I'll find some time and finish off the last few issues.

Actually I just realised that I missed a tag for that release, which is what you were asking. I've pushed a tag.


Update sbt-scalajs, scalajs-compiler, ... to 1.5.0

Updates

org.scala-js:sbt-scalajs
org.scala-js:scalajs-compiler
org.scala-js:scalajs-library
org.scala-js:scalajs-test-bridge

from 1.0.1 to 1.5.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-js" } ]

labels: library-update, semver-minor

Superseded by #346.


equals and hash code method implemented

#58

Thanks for start working on that. I have some comments:

Please use the the java.util.Objects class for creating hashCode() instead of using the prime number pattern
Please use the the java.util.Objects class for creating equals() instead of using the if-else pattern
There are still some classes without equals() and hashCode() methods in package org.jskat.control.event.skatgame
Please use curly brackets even on one line blocks.

See class TrickCardPlayedEvent as an example.
Bonus: It would be great to have hashCode() and equals() tested via unit test with the help of the library Equalsverifier (https://jqno.nl/equalsverifier/)

i will modify, test and let you know.

@b0n541 i have added the equals and hashcode method. When i added dependency for equalsverifier, i get the build failed. hence removed it.


log exceptions on non-api fails

If not using debug mode, non-JsonApiExceptions are only returned to the requestor and not logged. This makes it difficult to detect when requests are failing and what is causing the exceptions.

Thanks. This seems to be also missing upstream, so it might be good to PR it over there also (the maintainer seems to be merging again).

Good call. I'll do that.


python 3.6.0 abnormal

CodeQuery plugin doesn't work in CodeQueryMakeDb with python 3.6.0(3.5.0 test ok);
platform: MacOs 10.12.3
vim: neovim-0.2
codequery: 0.1.8

Hi @syslot,
Sorry for late response.
I can't reproduce your problem in:
Python: 3.6.0 (through pyenv)
OSX 10.12.4
vim:neovim-0.1.7
codequery:0.18
Can you help me by giving me more info to solve the problem?
(Just a guess, could you check whether pycscope works or not?)


Not work on next.js & sveltekit

I am using other.nvim, and when the file path like this:
src/routes/[slug]/+page.js
src/routes/[slug]/+page.svelte

other.nvim does not recognize the relevant files.
Maybe regex problem?

Can you give me your regex, I will be happy to help 😄

@rgroli
Thanks!
my config for sveltekit is here
https://github.com/ryoppippi/dotfiles/blob/9c96631c58728c6b34bc523798a48fc923f68856/nvim/lua/plugin/other.lua

For testing I used your mappings and setup two files with the following structure:
/src/routes/mylitte/thing/+page.js
/src/routes/mylitte/thing/+page.svelte

and it worked flawlessly.. I suspect there is something else going on..
Also your regex seems good - I first thought I must be something with the plus sign in the filename, but you escaped it correctly. As much as I hate this sentence but: I works on my machine 😄

Also, If your mapping are usable for standard sveltekit projects I encourage you to make a pr :) Would be great if we could add this to the builtin mappings.
It's really simple: just add a file under lua/other-nvim/builtin/mappings
🙏

@rgroli Thank you for check this out and I'd like to make pr for this one!

For testing I used your mappings and setup two files with the following structure:
/src/routes/mylitte/thing/+page.js
/src/routes/mylitte/thing/+page.svelte
and it worked flawlessly.. I suspect there is something else going on..

Yeah, that type of path is ok for my regex!
My problem is my regex does not work on the path like this
src/routes/[slug]/+page.js
src/routes/[slug]/+page.svelte

idk how to fix it. Could you help me?

@ryoppippi Ahh, now I see you mean literally "[slug]" I thought this was just a placeholder.. Short look into the routing doc makes it clear. Alright, I will check how to solve this ..

@ryoppippi Problem was that the [and the ] character have special meaning when used with the lua glob function. You can update the plugin and it should be fixed.

perfect! it works!! Thanks!
I'll create a pr later!!!

@rgroli
hi!
I have tested your fix and use it for all my day today
Then I started to see options that were not there before.
Why is this?
I have builtin angular on in my settings.
Before the update, there was no option to create a new file for this.

@ryoppippi
That's a feature I introduced in the previous update. see #22
It is not related to the fix from later.
If you don't want it you can turn it off easily by setting showMissingFiles = false in the config :)

I see thanks!!!


Multi-GPU/Session Runtimes!

Is your feature request related to a problem? Please describe. A clear and concise description of what the problem is. No my feature request is not related to a problem!
Describe the solution you'd like A clear and concise description of what you want to happen.  Since aws has multiple variety of different GPUS why not add this in studiolab for shorter runtimes in exchange since its free?
For an example for high loads you can choose a Tesla V100 notebook that will run for 2 hours, the default GPU can remain 4 hours
That’s my feedback, thanks

@lorazabora Thank you for your feature idea! Would you please write your idea according to the ISSUE_TEMLATE and modify the title more explicitly? It is helpful not only for us but also other community participants to understand your proposal more concretely.

Done, let me know if it’s more proper now
Thanks!

@lorazabora Thank you for your effort for adjusting the issue. I understand your request is not only selecting the multiple GPU but also choosing the trade-off between GPU spec and running time limit. If so, adjusting the title is more preferable for us.

May I kindly ask if the team is trying to add some features and that’s why runtimes aren’t available? What is the cause exactly

Hi and thank you for trying our SageMaker Studio Lab. As I have said on other threads we are experiencing quite a bit of demand for GPU. We appreciate your patience while we work on it.

I understand thanks, can you give us an estimated time on how much it should take

@lorazabora We can not guarantee the concreate solution schedule now. To realize  more sustainable management of GPU, we are listening and gathering the voice of community.
We are still in preview, so we appreciate your patience and your responsible usage.

Whenever it gets fixed, will there be an announcement here on github? Just let me know personally if you don’t mind
Thanks again for all the teams efforts

@lorazabora We will notify on the GitHub Issue #88 when GPU instance problem solved because this issue is the first issue related to this problem. If you are waiting to solve the problem, please 👍 to this issue.
And I created the issue #98 inspired from your request. If you need this feature, please 👍 also because it becomes the metrics of needs in the community and it affects our priority.
To measure the metrics, please close this issue and add 👍 to related issue if you can.

It has been quite a long time since this issue started, why can’t we get at least an estimated time line


Uncaught ReferenceError: process is not defined in Cognitiveservice SDK HTML Samples

Package Version: @azure/ms-rest-js

[x] browser

name/version:  2.0.6

Describe the bug
While running HTML samples of the cognitiveservice sdks, the code errors out in the initialization of the client itself.
To Reproduce
Steps to reproduce the behavior:

Create a TS project with dependency on @azure/ms-rest-js - version 2.0.6 (This bug has been introduced only in this version. The previous version 2.0.5 works fine)
Create a html file with the following code:

<!DOCTYPE html>
<html lang="en">
  <head>
    <title>@azure/cognitiveservices-qnamaker-runtime sample</title>
    <script<EMAIL_ADDRESS>    <script<EMAIL_ADDRESS>    <script type="text/javascript">
      const QNAMAKER_KEY = "<YOYR_KEY>";
      const QNAMAKER_ENDPOINT = "<YOUR_ENDPOINT>";
      const kbid = "<YOUR_KBID>";

      const cognitiveServiceCredentials = new msRest.ApiKeyCredentials({
        inHeader: {
          "Ocp-Apim-Subscription-Key": QNAMAKER_KEY
        }
      });

      const client = new Azure.CognitiveservicesQnamakerRuntime.QnAMakerRuntimeClient(
        cognitiveServiceCredentials,
        QNAMAKER_ENDPOINT
      );
      const customHeaders = { Authorization: `EndpointKey ${QNAMAKER_KEY}` };

      // A question you'd like to get a response for, from the knowledge base. For example
      const question = "How are you?";

      // Maximum number of answer to retreive
      const topValue = 1;

      // Find only answers that contain these metadata
      const strictFilters = [{
        name: "editorial",
        value: "chitchat"
      }];

      client.runtime
        .generateAnswer(kbid, {question, topValue, strictFilters}, {customHeaders})
        .then(result => {
          console.log(JSON.stringify(result));
        })
        .catch(err => {
          console.log("An error occurred:");
          console.error(err);
        });
    </script>
  </head>
  <body></body>
</html>

Open the file in a browser. (I have tested in Chrome/Firefox/Safari)
You can see the following error in the console:

Uncaught ReferenceError: process is not defined
    at loadEnvironmentProxyValue (proxyPolicy.ts:12)
    at getDefaultProxySettings (proxyPolicy.ts:31)
    at createDefaultRequestPolicyFactories (serviceClient.ts:427)
    at QnAMakerRuntimeClient.ServiceClient (serviceClient.ts:154)
    at QnAMakerRuntimeClientContext [as constructor] (qnAMakerRuntimeClientContext.ts:44)
    at new QnAMakerRuntimeClient (qnAMakerRuntimeClient.ts:29)
    at QnAMakerRuntimeSample.html:18

Expected behavior
The console should display the correct result instead of error.
Screenshots
N/A
Additional context
The error happens here. The code that causes this is:
function loadEnvironmentProxyValue(): string | undefined {
  if (!process) {
    return undefined;
  }

  if (process.env[Constants.HTTPS_PROXY]) {
    return process.env[Constants.HTTPS_PROXY];
  } else if (process.env[Constants.HTTPS_PROXY.toLowerCase()]) {
    return process.env[Constants.HTTPS_PROXY.toLowerCase()];
  } else if (process.env[Constants.HTTP_PROXY]) {
    return process.env[Constants.HTTP_PROXY];
  } else if (process.env[Constants.HTTP_PROXY.toLowerCase()]) {
    return process.env[Constants.HTTP_PROXY.toLowerCase()];
  }

  return undefined;
}

The error happens when the code checks for if(!process).
In the same folder, there is another file called proxyPolicy.browser.ts which does not have this issue and should have been loaded in the browser. I am not sure why the other file got loaded.
I think if that file got loaded correctly, then the issue will be resolved. Needs further investigation.

Ok. I did the investigation. My guess seems to be correct. Once I modified the configuration to load the proxyPolicy.browser.ts file, the error no longer happens and I am able to get the correct output. Created a PR for this fix (includes upgrade to the patch version)

Issue resolved in the latest release.


Optional URL for email templates

It would be great if it was possible add a root URL to configs to be injected into the email templates.  Currently I have to customize these manually, which is a bit messy with other fields like conf_key, password_key, and user_name being replaced automatically.
My use case is for testing the entire process on my local environment with a localhost URL before pushing code to prod.

I completely agree and will most likely get to this change tonight plus a couple of the other issues open.

Awesome, thank you!  I'll test it out as soon as it's in.

base_url has been added as a config option. Note that the email confirmation is a post endpoint so you will need some ui set to handle the email redirect anyways but you can now set it to localhost or whatever you need.
Also with the latest version I went through a couple other open issues and how you do hooks/email templateing has changed so you may want to check the docs again if you were using any of that.

https://github.com/jpiepkow/accesspass/releases/tag/0.5.4


Query glossary

This is a simplified version of explanations contained in Relevance tests.  Because tools are often combined, the Guide to Collections Queries will be organised by types of things searched for and cite the relevant tools deployed in each section.

Is there anything you need me to do on this in order to complete the reviews?


feat(axum): add layer

This pr adds the support for ApiMethodRouter::layer based on the original implementation of axum

Are there any blockers to getting this merged? Not being able to use .layer in this way is preventing us from getting aide set up for one of our servers 😞
Are there any other API incompatibilities?

Are there any blockers to getting this merged? Not being able to use .layer in this way is preventing us from getting aide set up for one of our servers

No, I just have so much time in a day.
LGTM, thank you!


[FEATURE_REQUEST] Option to show more information

Is your feature request related to a problem? If so, please describe.
No response
Describe the solution you'd like
Are there plans to add some more information to the cards? For instance show how many open requests there are for Overseerr or how many movies you got in Radarr.
Priority
Medium (Would be very useful)
Is this something you would be keen to implement
No

👋 @damianeickhoff
Thanks for opening your first issue here!
To help make it easier for us to investigate your issue, please follow the contributing guidelines.
If you like this project, please ⭐ star our repo.

Hi, yes there are such plans, I use arr stack myself. As soon as I have free time I will definitely do the integration


fuse: Add more directories

This PR adds the following directory in the fuse mount:

tags: Contains a directory for each tag, which only contains only the snapshots that have this particular tag
hosts: Contains a directory for each host name, which then contains the snapshots of this hostname

Closes #636

Codecov Report

Merging #1050 into master will decrease coverage by 0.26%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #1050      +/-   ##
=========================================
- Coverage   54.06%   53.8%   -0.27%
=========================================
  Files         116     117       +1
  Lines        8845    8888      +43
=========================================
  Hits         4782    4782
- Misses       3362    3405      +43
  Partials      701     701

Impacted Files
Coverage Δ

src/restic/fuse/snapshots_dir.go
0% <0%> (ø)

src/restic/fuse/meta_dir.go
0% <0%> (ø)

src/restic/fuse/dir.go
0% <0%> (ø)
:arrow_up:

src/restic/fuse/root.go
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 e60a96a...2c02efd. Read the comment docs.


Unit tests for PreparedGet #69

Closes #69
Unit tests for PreparedGet
@artem-zinnatullin PTAL

Great job :+1:

Updated =)


Update backdrop on window resize

Backdrop doesn't get updated when window gets resized.
My solution for bootstrap-tour.js:
Update line 727:
if(this.backdrop.remove) this.backdrop.remove();
Add to line 319:
_this._hideBackdrop();

Pull request? ;)

I've tested this and it works. What's the status ?

@gmircea If anyone submits a pull request, I'll merge it.

Ok, I'll try to modify and then make a pull request but I'll need some help
regarding the workflow: I must update the two js files from the build/js
folder, and then minify them. So all four js files are up to date .
How to minify the js ? Is there some preferred  tool to use ?
Thanks
On Fri, May 22, 2015 at 5:47 PM, Ulrich Sossou<EMAIL_ADDRESS>wrote:

@gmircea https://github.com/gmircea If anyone submits a pull request,
I'll merge it.
—
Reply to this email directly or view it on GitHub
https://github.com/sorich87/bootstrap-tour/issues/413#issuecomment-104679324
.

Fixed in #454

I'm still fixing the issue where the white background size is not recalculated. I'm using v0.10.2.
Normal Screen:

Resized Screen (Small to Big):

Resized Screen (Big to Small):

var tour = new Tour({
steps: [
{
element: ".a-css-filter",
backdrop: true,
title: "Some text",
content: "Some text"
}
],
template: ""
});
// Initialize the tour
tour.init();
// Start the tour
tour.start();


Removed duplicate code in conditions.ts, added documentations, and added types

I hope I didn't overdo the docs

Oops, I just noticed that this interferes with the database-wrappers, since you named some parameters a, and some values b. My friend is already working on updating the wrappers, and therefore will fix this.

I really like the implementation that you did on the conditions file- that's going to be fast. Are the changes ready to merge?

I really like the implementation that you did on the conditions file- that's going to be fast. Are the changes ready to merge?

In both the implementations of the conditions for each database-wrapper (e.g. src/databases/sql/condition.ts) the as need to be replaces by parameter and the bs is value.

 @William-McGonagle

You can just fix it in the next commit if it's easier @BergerAPI

@William-McGonagle Ready to merge. I've also changed the ifs in the wrapper-implementations to a switch case.
Working as intended:

Looks good- merging it in now!


Refactor cmd/kubectl-tag

We need to refactor kubectl-tag binary. The way it is it only works with default namespace.

Implemented by https://github.com/ricardomaraschini/tagger/pull/11


Fibers rather than threads?

Hi - thanks very much for making Pyro available. Have you thought about adding Wren-style fibers (co-operative multi-tasking) rather than threads? Seems like it would be much less difficult (both to implement and to use), and it would be a clean way to provide non-blocking I/O.
Thanks again,
Greg

Hi Greg, to be honest I haven't really thought about the best way to implement concurrency yet -- I decided to punt that question off until the core design of the language had settled down first.
I will definitely take a look at Wren's fibers -- that's an approach I've never seen before and it does look like a really interesting option for a scripting language.
Thanks for the suggestion!

I think Nystrom borrowed it from Lua and Erlang, and I vaguely remember it from working with real-time systems a looong time ago. Gives the language generators for free, doesn't lead to callback hell, plays nicely with libuv's event loop - looking at the Wren source code, it looks like replicating the stack and stack pointer would be pretty straightforward; I'm still trying to wrap my head around error handling: if a child fiber panics and the parent doesn't handle it, the parent panics - what happens to the siblings of the fiber that originally panicked?


Apollo Docker build error on aarm64 arch

Hello,
While building apollo docker with bash docker/scripts/dev_start.sh shell command, The following error occurs on aarch64(arm64 )arch.
[INFO] Load map apollo_virutal_map from image: apolloauto/apollo:map_volume-apollo_virutal_map-aarch64-latest
[INFO] Create volume apollo_map_volume-apollo_virutal_map_root from image: apolloauto/apollo:map_volume-apollo_virutal_map-aarch64-latest
[INFO] Start pulling docker image apolloauto/apollo:map_volume-apollo_virutal_map-aarch64-latest ...
Error response from daemon: manifest for apolloauto/apollo:map_volume-apollo_virutal_map-aarch64-latest not found: manifest unknown: manifest unknown
[ERROR] Failed to pull docker image : apolloauto/apollo:map_volume-apollo_virutal_map-aarch64-latest
I think the corresponding map volumes are not uploaded for ARM64 arch. Do you know how can I use the carla_apollo_bridge on aarch64 architecture?

I guess this is a network problem, have you checked whether the terminal network proxy is configured? Try canceling the network proxy.

I'm sorry, let's find out whether there are similar problems in apollo's issues

I'm sorry, let's find out whether there are similar problems in apollo's issues

I checked https://hub.docker.com/r/apolloauto/apollo/tags , the 'mage that docker/scripts/dev_start.sh tries to pull are not available there.

The seventh line of apollo.sh, SUPPORTED_ARCHS="x86_64 aarch64", indicates that apollo supports the aarch64 architecture

Hello, have you resolved the issue? I have also encountered the same problem. And I tried to modify the network settings, but it still hasn't been resolved.


build: add api test

Issue #, if available:
Description of changes:
This is attempt 3 of building API tests.
Previous attempts:
https://github.com/aws-amplify/amplify-backend/pull/788
https://github.com/aws-amplify/amplify-backend/pull/794
The algorithm is:

Checkout latest and baseline repo (i.e. PR content and PR base branches)
Publish packages to verdaccio (local registry)
Generate test projects

Use baseline API.md to generate usage snippets
Use latest as dependency to test project

Compile generated projects.

Passing compilation indicates no breaking change detected (as good as our coverage).
Failing compilation indicates breaking change.

Implemented checks include:

Enum validation
Type assignability validation

Out of scope
This is initial version of this mechanism and is focused on establishing algorithm and check.

Further expansion: exported classes and functions/constants/vars are out of scope for this PR and will be additive expansion on the top of it.
Further improvements of debug-ability: current version was developed using development loop on local machine that was working quite easy and fast.  Debugging procedure is documented in script jsdocs. This will be improved separately if needed.
Integration with semver diffs/changeset. This check can be accompanied by semver diff check. I.e. a version bump coming from changeset can be validated against API diff detected (e.g. no API change - patch, non-breaking API change - minor, breaking API change - major, detection of no API change or intentional major version increment should skip this change for given package).
Mechanism to suppress the check for intentional breaks (for now check will be optional).

Testing

Enabled this check for all packages in repo
Added integration tests that assert this mechanism can catch expected breaking changes
Unit tests for snippet generators (so that it's easier to work with them locally)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

A follow up fix https://github.com/aws-amplify/amplify-backend/pull/812 .


Libraries QA

ALL RESOLUTIONS

[x] The buttons at the top (Language, search..) should be dark green and not pink.
[x] The black line is missing at the end of each H2 section
[x] The sub menu is missing with the H2 headers.

Closing as all elements have been addressed


ModuleNotFoundError: No module named 'keras.src.legacy'

I must use tensorflow 2.14.0 for my work. But when I run it show the folowing error :
from .models import create_model, create_model_2_model_padding, create_model_2_model, create_model_2_model_patch_size
  File "F:\_brain\_code\_code_one_by_one\modules\models.py", line 23, in <module>
    from classification_models_3D.kkeras import Classifiers
  File "C:\ProgramData\anaconda3\Lib\site-packages\classification_models_3D\kkeras.py", line 2, in <module>
    from .models_factory import ModelsFactory
  File "C:\ProgramData\anaconda3\Lib\site-packages\classification_models_3D\models_factory.py", line 3, in <module>
    from .models import resnet as rn
  File "C:\ProgramData\anaconda3\Lib\site-packages\classification_models_3D\models\resnet.py", line 5, in <module>
    from ._common_blocks import ChannelSE
  File "C:\ProgramData\anaconda3\Lib\site-packages\classification_models_3D\models\_common_blocks.py", line 2, in <module>
    from keras.src.legacy.backend import int_shape
ModuleNotFoundError: No module named 'keras.src.legacy'

How can solve the problem.
with tensorflow 2.16.1 it works well

Hello,
If you use tensorflow==2.14.0, then it comes with keras==2.14.0. You can force the installation of keras>=3 but there is no guarantee that it will work smoothly (keras>=3 requires tensorflow>=2.16.1).
Therefore, you can't use the last version 1.10 of segmentation_models_3D since it works with keras>=3 only. The last version working with prior keras version is noted in the README page: https://github.com/ZFTurbo/classification_models_3D?tab=readme-ov-file#older-versions