Gitstream not reporting that a workflow has failed on a diff error

Describe the bug
seeing exactly the same issue as https://github.com/linear-b/gitstream/issues/30
To Reproduce
Steps to reproduce the behavior:

The .cm automation file
The PR URL (as it contains the repo and PR identifiers)
Describe your PR relevant content
Add relevant commit SHA

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
some logs
docker.io/gitstream/rules-engine:latest

><EMAIL_ADDRESS>start
> node src/app.js

Running tag v1.151
PR: REDACTED/pull/5923
commit: 9791e680a4076d5f3cfc4249c71a10ae58d00d4f
fatal: bad revision 'development-3.0...feature/BLC-14644'
node:child_process:826
    err = new Error(msg);
          ^

Error: Command failed: cd repo && git diff 'development-3.0'...'feature/BLC-14644' -- .cm/*.cm
fatal: bad revision 'development-3.0...feature/BLC-14644'

    at checkExecSyncError (node:child_process:826:11)
    at execSync (node:child_process:900:15)
    at executeGitCommand (file:///src/utils/git.service.js:15:23)
    at isCmChanged (file:///src/utils/git.service.js:155:18)
    at getCMChanged (file:///src/utils/ruleParser.js:113:21)
    at file:///src/app.js:34:33
    at ModuleJob.run (node:internal/modules/esm/module_job:185:25)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:281:24)
    at async loadESM (node:internal/process/esm_loader:88:5) {
  status: 128,
  signal: null,
  output: [
    null,
    Buffer(0) [Uint8Array] [],
    Buffer(58) [Uint8Array] [
      102,  97, 116,  97, 108,  58,  32,  98,  97, 100,  32,
      114, 101, 118, 105, 115, 105, 111, 110,  32,  39, 100,
      101, 118, 101, 108, 111, 112, 109, 101, 110, 116,  45,
       51,  46,  48,  46,  46,  46, 102, 101,  97, 116, 117,
      114, 101,  47,  66,  76,  67,  45,  49,  52,  54,  52,
       52,  39,  10
    ]
  ],
  pid: 30,
  stdout: Buffer(0) [Uint8Array] [],
  stderr: Buffer(58) [Uint8Array] [
    102,  97, 116,  97, 108,  58,  32,  98,  97, 100,  32,
    114, 101, 118, 105, 115, 105, 111, 110,  32,  39, 100,
    101, 118, 101, 108, 111, 112, 109, 101, 110, 116,  45,
     51,  46,  48,  46,  46,  46, 102, 101,  97, 116, 117,
    114, 101,  47,  66,  76,  67,  45,  49,  52,  54,  52,
     52,  39,  10
  ]
}

@Fitz7 should be fixed now

@vim-zz I've seen this issue come back maybe.
Can I where it was fixed? an update to the docker image or the action?
Is there a docker tag I can fix our actions to that is stable?


VPU Plugin Go unit tests coverage

@alekdu we're improving Go unit test coverage for our plugins.
VPU plugin coverage is <50%. Please check if that could be improved still.
Steps:

make test WHAT=cmd/vpu_plugin
review cmd/vpu_plugin/coverage.html

There's not much code in the plugin so while many things are already covered the coverage is poor.
Some ideas:

move sharedDevNum checking to newDevicePlugin and add a test case for 0 value
test without /var/tmp/hddl_service.sock
test Scan() API as it was just added in FPGA plugin
os.Exit(1) is now redundant

Other:
/var/tmp/hddl_service.sock is not cleaned up after testing

hmm... I will look into this.

https://github.com/intel/intel-device-plugins-for-kubernetes/pull/352 please review

these changes were made...


Support passing options to useCookies?

As far as I can see, there isn't currently a way to pass options (specifically, doNotParse) to the getAll method that's called inside useCookies. Would it be possible to add a second argument to useCookies that will be passed on as the second argument to getAll?

Hi @Vinnl ,
Since those options are not very often used, they are not exposed but they very well could be. Feel free to open a PR with a proposition.


Fix some potential NPEs in SystemCapabilityManager

Fixes #1221
This PR is ready for review.
Risk
This PR makes no API changes.
Summary
This PR fixes some potential NPEs in SystemCapabilityManager
CLA

[x] I have signed the CLA

Codecov Report

Merging #1222 into hotfix/4.10.1 will decrease coverage by 0.03%.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##             hotfix/4.10.1    #1222      +/-   ##
===================================================
- Coverage             47.1%   47.06%   -0.04%
  Complexity            4327     4327
===================================================
  Files                  481      481
  Lines                23475    23475
  Branches              2683     2683
===================================================
- Hits                 11057    11049       -8
- Misses               11750    11760      +10
+ Partials               668      666       -2

Impacted Files
Coverage Δ
Complexity Δ

...smartdevicelink/encoder/VirtualDisplayEncoder.java
24.43% <0%> (-4.55%)
8% <0%> (ø)

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 9377865...665b984. Read the comment docs.


Training time for new data

Hi,
I am wondering how long it took you to train the model . How many epochs did you have to run it for. I'm trying to run it on the coco dataset. After 150 epochs, I see the loss is still reducing but the eval.py script shows the model getting worse. I see the model generating too ,many bounding boxes. Any help would be appreciated.
Thanks.

Can you share your git ? I'm trying to train on coco too, but I'm having problem with reshape of the output, it's returning error.

Hi,
Can you share your project, or give me some help ? I'm trying to train on coco too, but I'm having problems with the reshape the output. It's returning an error.


metrics-server-exporter look up all or many more namespaces

I think that metrics-server-exporter need look up all or many more namespaces.

done


Add support for notebook testing.

This patch moves the 'numba.testing' module into a proper sub-package to make room for additional functionality. It then adds some basic support for validating notebooks.

This PR fails on Python 2.6 as python -m numba.testing no longer works (being a package).
I added a new numba.runtests module that can be used instead, so if we change the CI logic to invoke that instead, this should work.

You can simply change .travis.yml to run the right module.

OK, done. (The tests pass again.)

(Neither examples nor notebooks are tested by default yet. We need to document somewhere how to run those tests.)


Prevent FASET form navigation if uploads are pending

Closes #343

Only if there are submissions that haven't uploaded (because you're offline or similar.)


The fire demo table will eventually overload and kill the frames per second

Cogito and Godot Engine Version:
Godot v4.2.2.stable.official [15073afe3]
Cogito beta 202406.02
Description:
Idling in the Laboratory scene will eventually slow way down due to the fire. Usually took 10-15 minutes.
Reproduction steps:

Load the demo
Go to the Laboratory.
Stand anywhere (though standing by the fire table will show you what happens).
Wait for 10-15 minutes
Try to look around
Notice the FPS is horrendously slow

Expected behavior:
The game to continue running at a reasonable frames per second.
Screenshots:
See attached. It's a screenshot of what the fire table looks like after leaving it for 10-15 minutes.

Oh wow!
So disabling the particles on the darts should be a quick fix for this.
I want to do create a separate area for the systemic properties with their own items, so this doesn't happen anymore and the darts will be normal darts again. 😁
Will follow up when that's done.

Heyo, so I've just pushed an update in https://github.com/Phazorknight/Cogito/commit/7d3b225dc9a16492ed3423f63f7b36238bcc989f
This now limits the amount of vfx particles that get spawned, hopefully resolving this.
Please give it a go and see if you still run into this issue.

Closing due to inactivity / assumed resolved.


Adding full screen mode on Custom Player Controls

Hi, first of all excuse me of the request but i'm not a programmer and this is the first time for me in GitHub. I'm a new amazon-ivs user and my question is how it's possibile to add the classic full screen mode button on the Custom Players Controls exemple. It will be great for my job.

Hi @paolodeleonardis !
I have updated the demo and added a fullscreen button (https://github.com/aws-samples/amazon-ivs-basic-web-sample/commit/0a252490aeff52d1e81d21aaf3719627b15c906d) so that you can see how it can be done.

Thank you so much!
Il giorno ven 27 mag 2022 alle 23:53 Max Boulin @.***>
ha scritto:

Closed #3
https://github.com/aws-samples/amazon-ivs-basic-web-sample/issues/3 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/aws-samples/amazon-ivs-basic-web-sample/issues/3#event-6695610982,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYWEZ3WJZUK5KP2YJ4OVSEDVME765ANCNFSM5ULEBAPQ
.
You are receiving this because you were mentioned.Message ID:
<aws-samples/amazon-ivs-basic-web-sample/issue/3/issue_event/6695610982@
github.com>

--
--
Paolo De Leonardis


Fix typo of risc-v readme

btw Hanhan, for doc-only changes, I just skip checks and merge without waiting :P
I think that requires admin though

btw Hanhan, for doc-only changes, I just skip checks and merge without waiting :P
I think that requires admin though

yeah, I don't have admin permission. I'm fine with it because we can turn on auto-merge. 📦
I disable the auto-merge, @rednoah91 could you also fix the typos that Scott pointed out?

btw Hanhan, for doc-only changes, I just skip checks and merge without waiting :P
I think that requires admin though

yeah, I don't have admin permission. I'm fine with it because we can turn on auto-merge. 📦
I disable the auto-merge, @rednoah91 could you also fix the typos that Scott pointed out?

Done. Please check it. Thanks.

Thanks!


Unable to download certain videos

Hi.
The following error occurs with certain videos.
Relatively recent uploads are not possible.

https://youtu.be/VGKrQu3ERgY?si=T7uQzeIbKec0s0eP
https://www.youtube.com/watch?v=09VuoiCzCug

finished with error [57] Error Domain=NSPOSIXErrorDomain Code=57 "Socket is not connected" UserInfo={NSErrorFailingURLStringKey=https://youtubekit-remote.losjet.com/v1?videoID=VGKrQu3ERgY, NSErrorFailingURLKey=https://youtubekit-remote.losjet.com/v1?videoID=VGKrQu3ERgY, _NSURLErrorRelatedURLSessionTaskErrorKey=(
"LocalWebSocketTask <5A6E716A-8ADA-40C6-A620-4C8AE9294E65>.<13>"
), _NSURLErrorFailingURLSessionTaskErrorKey=LocalWebSocketTask <5A6E716A-8ADA-40C6-A620-4C8AE9294E65>.<13>}

Both videos work for me. I'm calling it like this: YouTube(videoID: "09VuoiCzCug", methods: [.remote]).
Does it work for you when you do methods: [.local] (i.e. just without remote)?
Also when did you notice it started to happen?

I also worked on the above two.
However, it seems that if the video was uploaded immediately, it cannot be downloaded.
[URL of the site where the DL was not available]
https://youtu.be/XvyTXHEcL4g?si=qw9bYHPDEnnfEqrE
https://youtu.be/BC5vkW0mT7U?si=haCv83og4l8FMcbT
etc.
[URL of the site where the DL is now available]
https://youtu.be/GMivynf6NZ8?si=jCCx7wXgeAoHnJrs
https://youtu.be/ejGQN9jPs_k?si=OI0RRdjnCnvazOLB
etc.
"URL of the site where the DL is available.
I have described it as
I tried to DL 1.2 days ago and was not able to do so, but after some time passed or a new video in the channel was uploaded, I was able to DL the video.
This is because of the following reasons.
So, the URLs that could not be DLed may be able to be DLed in the future.

Thank you. I checked that fix. It looks fine so far.


How do we diagnose STATUS_INVALID_PARAMETER?

This applies to all IOCTLs.
If an IOCTL returns STATUS_INVALID_PARAMETER, how do we troubleshoot this? How about telling us WHICH parameter is bad (after all, there are status values specifically for this purpose, such as STATUS_INVALID_PARAMETER_1, _2, _3, ..._12)?

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

ID: 86ded8b2-3552-5f5a-bd1b-989f6d8b5616
Version Independent ID: 25704ae5-3d6f-8d75-86f4-5302c3600377
Content: IOCTL_USB_GET_NODE_CONNECTION_NAME
Content Source: wdk-ddi-src/content/usbioctl/ni-usbioctl-ioctl_usb_get_node_connection_name.md
Product: windows-hardware
GitHub Login: @windows-driver-content
Microsoft Alias: windowsdriverdev

Thanks, @BrianCatlin
@PageWriter-MSFT can you please forward this to the correct PM or Dev for consideration? Thank you.

Hi @BrianCatlin,
I’ve created an internal work item for myself to investigate this with the USB team, and I’ll make a note here when I complete the changes. In the meantime, I’m closing the GitHub issue while it’s tracked in the bug. Thanks again for helping to improve the docs.


Issues with messages sent to console.

So there's 2 issues currently with messages sent by players when they show in console.
First of all in spigot even if you clear the recipients list, the message will still be sent to the console. You have to completely cancel the event for it to not be sent.
This means since we send the message to the console ourselves there will be a duplicate. And yes at least for the normal ChatChannels the console will always be a target as shown here: https://github.com/HelpChat/ChatChat/blob/42e13413aeaca707e9c05160f49ba0d804f866e7/plugin/src/main/java/at/helpch/chatchat/channel/ChatChannel.java#L49
https://github.com/HelpChat/ChatChat/blob/42e13413aeaca707e9c05160f49ba0d804f866e7/plugin/src/main/java/at/helpch/chatchat/user/UsersHolder.java#L27
The second issue is that our format just send the message itself but not the formatting part. so no player names, etc.

Fixed the second issue in #55


Container Linux doc fixes

portions of the URLs were missing their bash substitutions and the variable exports don't apply to the following commands without using 'export'

thank you! learned some new things about bash today :)

/assign @steveperry-53

/lgtm

/approve


Missing export feature in Data Migration Assistant tool

I have found that latest version of DMA v4.2 is lacking export feature of report at end. I have installed and run this tool and found missing button of Export report at the end. Can anyone help me to resolve that ?

Hello! This forum is for feedback on the docs.microsoft.com site. Given that this item is not directly related to our site, we cannot address it here. We recommend that you use the tool feedback channel to communicate your concerns or questions.


grafana and zabbix diasplay alias with variables

Maybe it is solved, if that can anyone help with this issue. I want to display all critical (thresholded) metrics with unique (with variable) alias. There are my dashboard and my problem. Thx!

Pls help with this! The result that I expect: instead of "Link down" -> display "Interface ge-0/0/0(Description): Operational status"

Use replaceAlias() instead.


Issue 618

@homu r+

:pushpin: Commit 7bc1b4b has been approved by kbknapp

:hourglass: Testing commit 7bc1b4b with merge 37cc3c3...

:sunny: Test successful - status


Ioncube doesn't install on php 7.2

Describe the bug
Since the latest bumpup of the v2 tag, ioncube doesn't install anymore on v2
When using 2.29.0 release everything works properly
Version

[x] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2

Runners

[x] GitHub Hosted

Operating systems
ubuntu-latest
PHP versions
7.2
To Reproduce
- name: Setup PHP with PECL extension
uses: shivammathur/setup-php@v2
with:
php-version: 7.2
extensions: intl, mbstring, gd, apcu, memcached, ioncube
coverage: xdebug
Expected behavior
Properly install ioncube
Screenshots/Logs

2.29.0:

@jocel1
Please try running the workflow again, I'm unable to reproduce the issue
Test workflow: https://github.com/shivammathur/test-setup-php/actions/runs/9759633744/job/26936713788#step:2:19

Same issue after several retry. I'm also using cache-extensions module, I'll try without it:
code-quality:
timeout-minutes: 20
runs-on: ubuntu-latest
strategy:
matrix:
php-versions: ['7.2']
env:
extensions: intl, mbstring, gd, apcu, memcached, ioncube
key: cache-v1
name: Code Quality on PHP ${{ matrix.php-versions }}
permissions:
contents: 'read'
id-token: 'write'
  - name: Setup cache environment
    id: extcache
    uses: shivammathur/cache-extensions@v1
    with:
      php-version: ${{ matrix.php-versions }}
      extensions: ${{ env.extensions }}
      key: ${{ env.key }}

  - name: Cache extensions
    uses: actions/cache@v4
    with:
      path: ${{ steps.extcache.outputs.dir }}
      key: ${{ steps.extcache.outputs.key }}
      restore-keys: ${{ steps.extcache.outputs.key }}

  - name: Setup PHP with PECL extension
    uses: shivammathur/setup-php@v2
    with:
      php-version: ${{ matrix.php-versions }}
      extensions: ${{ env.extensions }}
      coverage: xdebug

I confirm the issue comes from the caching

Note caching and 2.29.0 works properly

2.30.4:

2.30.5:

Ah!, Fixed in https://github.com/shivammathur/setup-php/commit/afff805ea318c148735d2a1e88142d355311db2f. It will be in the next release.

Released 2.31.1 with the fix.


Removing row group Item Count from Powergrid

There is an option to remove the Count from grouped rows in Datagrid, could we have that in power grid too please, in the meantime how do I modify or not show, as you can see from below its not very useful in this use case.

you can remove that using CSS, add this to your stylesheet:
.ag-group-child-count {
display: none;
}

Perfect, thank you..


[BUG] - Select popover closes on label pointer up event

NextUI Version
2.1.0
Describe the bug
Clicking the <label> element opens the Select popover as expected, but it immediately closes when releasing the mouse button.
Your Example Website or App
https://codesandbox.io/p/sandbox/snowy-sun-8y839x?file=%2FApp.jsx%3A8%2C14&utm_medium=sandpack
Steps to Reproduce the Bug or Issue

Go to the example.
Click and hold down the left mouse button on "Favorite Animal".
Release the mouse button and observe the Select popover closes.

Expected behavior
As a user, I expect the Select popover to stay open, but I am seeing that it closes.
Screenshots or Videos
https://github.com/nextui-org/nextui/assets/20935774/5b10df91-6428-42a2-b58b-e8cb6189b41a
Operating System Version
Windows
Browser
Chrome

Fixed: https://github.com/nextui-org/nextui/pull/1455

Nice


SQL Error on JSON Mode Array Serialization

Base on Array Serialization doc http://medoo.in/api/insert if we use json:
$last_user_id = $database->insert("account", [
	"user_name" => "foo",
	"email" =><EMAIL_ADDRESS>	"age" => 25,
	"(JSON) lang" => ["en", "fr", "jp", "cn"]
]);

We expected the result in SQL is:
INSERT INTO "account" ("user_name", "email", "age", "lang") VALUES ('foo'<EMAIL_ADDRESS>'25', '["en","fr","jp","cn"]')

But actual result in SQL is:
INSERT INTO "account" ("user_name", "email", "age", " lang") VALUES ('foo'<EMAIL_ADDRESS>'25', '["en","fr","jp","cn"]')

There is a space in "lang" field.

Thank for the report :+1:
I fixed it with simpler way.
And follow the contribution guide that submit your code to the develop branch, and start a pull request into it next time :)

Yes, your solution more simpler and sorry for not following the guideline.


Support KSQL's WINDOW

KSQL supports WINDOW keyword for windowed queries in aggregation. The syntax is as follows for different queries:

Hopping window:

WINDOW HOPPING (SIZE 30 SECONDS, ADVANCE BY 10 SECONDS)

Session window:

WINDOW SESSION (60 SECONDS)

Tumbling window:

WINDOW TUMBLING (SIZE 5 MINUTES)

See the syntax reference here.
This PR adds support for KSQL WINDOW.

Coverage increased (+0.03%) to 83.956% when pulling 7aaf32856e93b3fb4aff40766441eb15e22d0292 on ferbncode:ksql-windows into b59151eae9d2ac536f15a0d175456c2592fed778 on JSQLParser:master.

@wumpz The build fails for openjdk-7, also for master, with the following error. Could you please point out if I should add some fix for it in my changes?
[ERROR] -----------------------------------------------------
[ERROR] realm =    plugin>org.codehaus.mojo:javacc-maven-plugin:2.6
[ERROR] strategy = org.codehaus.plexus.classworlds.strategy.SelfFirstStrategy
[ERROR] urls[0] = file:/root/.m2/repository/org/codehaus/mojo/javacc-maven-plugin/2.6/javacc-maven-plugin-2.6.jar
[ERROR] urls[1] = file:/root/.m2/repository/net/java/dev/javacc/javacc/7.0.4/javacc-7.0.4.jar
[ERROR] urls[2] = file:/root/.m2/repository/org/codehaus/plexus/plexus-utils/1.5.6/plexus-utils-1.5.6.jar
[ERROR] urls[3] = file:/root/.m2/repository/edu/ucla/cs/compilers/jtb/1.3.2/jtb-1.3.2.jar
[ERROR] urls[4] = file:/root/.m2/repository/org/apache/maven/reporting/maven-reporting-api/2.0.4/maven-reporting-api-2.0.4.jar
[ERROR] urls[5] = file:/root/.m2/repository/org/apache/maven/reporting/maven-reporting-impl/2.0.4/maven-reporting-impl-2.0.4.jar
[ERROR] urls[6] = file:/root/.m2/repository/commons-validator/commons-validator/1.2.0/commons-validator-1.2.0.jar
[ERROR] urls[7] = file:/root/.m2/repository/commons-beanutils/commons-beanutils/1.7.0/commons-beanutils-1.7.0.jar
[ERROR] urls[8] = file:/root/.m2/repository/commons-digester/commons-digester/1.6/commons-digester-1.6.jar
[ERROR] urls[9] = file:/root/.m2/repository/commons-logging/commons-logging/1.0.4/commons-logging-1.0.4.jar
[ERROR] urls[10] = file:/root/.m2/repository/xml-apis/xml-apis/1.0.b2/xml-apis-1.0.b2.jar
[ERROR] urls[11] = file:/root/.m2/repository/org/apache/maven/doxia/doxia-core/1.0-alpha-7/doxia-core-1.0-alpha-7.jar
[ERROR] urls[12] = file:/root/.m2/repository/oro/oro/2.0.7/oro-2.0.7.jar
[ERROR] urls[13] = file:/root/.m2/repository/org/apache/maven/doxia/doxia-sink-api/1.0-alpha-9/doxia-sink-api-1.0-alpha-9.jar
[ERROR] urls[14] = file:/root/.m2/repository/org/apache/maven/doxia/doxia-site-renderer/1.0-alpha-7/doxia-site-renderer-1.0-alpha-7.jar
[ERROR] urls[15] = file:/root/.m2/repository/org/codehaus/plexus/plexus-i18n/1.0-beta-6/plexus-i18n-1.0-beta-6.jar
[ERROR] urls[16] = file:/root/.m2/repository/org/codehaus/plexus/plexus-velocity/1.1.2/plexus-velocity-1.1.2.jar
[ERROR] urls[17] = file:/root/.m2/repository/commons-collections/commons-collections/2.0/commons-collections-2.0.jar
[ERROR] urls[18] = file:/root/.m2/repository/commons-logging/commons-logging-api/1.0.4/commons-logging-api-1.0.4.jar
[ERROR] urls[19] = file:/root/.m2/repository/velocity/velocity/1.4/velocity-1.4.jar
[ERROR] urls[20] = file:/root/.m2/repository/velocity/velocity-dep/1.4/velocity-dep-1.4.jar
[ERROR] urls[21] = file:/root/.m2/repository/org/apache/maven/doxia/doxia-decoration-model/1.0-alpha-7/doxia-decoration-model-1.0-alpha-7.jar
[ERROR] Number of foreign imports: 1
[ERROR] import: Entry[import  from realm ClassRealm[project>com.github.jsqlparser:jsqlparser:2.1-SNAPSHOT, parent: ClassRealm[maven.api, parent: null]]]
[ERROR]
[ERROR] -----------------------------------------------------
[ERROR] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginContainerException
The command '/bin/sh -c mvn clean package' returned a non-zero code: 1

This Number of foreign imports: 1 seems to be a maven problem. There are tons of issues regarding this.
However, I am not sure if some of my enforecer rules are failing with this.
Last thing on the dependency front was the upgrade to JavaCC 7.0.4 from 7.0.3. Could you rewind it and check if this works?

@wumpz That does help, but that also requires downgrading version for checkstyle due to similar error. (which was part of this change)

Reverted checkstyle version. The upgrade was done due to some kind of security alert from github.

@wumpz Do I revert the change? There is a similar maven problem with newer checkstyle version: https://gist.github.com/ferbncode/d8e61aa083d198694074c4382a1bacca.

I did not understand your question. I reverted the version back and needed to revert the version of JavaCC to 7.0.3 as well to get a successfull java v7 build.

@wumpz Sorry, I missed that you already fixed the versions in pom.xml in commit:  https://github.com/JSQLParser/JSqlParser/commit/6e7b976d65d7eb6433015c8ecb7d7c49134a0550. I have rebased my pull request against master. Thanks!! :)

@wumpz Do you know when you can do a review/merge of the PR? It would be really awesome if you could review it. Thanks! :)

Sorry. I totally forgot that this is a pull request about our discussion :(.
It looks ok,
I will squash and merge it, but if you want more control about the merged commit, please squash it yourself.

@wumpz Thank you :), As you suggested, I squashed the commits.

@wumpz Would you be so kind to release a snapshot for these changes? Thanks a lot! :)

Sorry I thought I did it right away. At the moment I am deploying the actual Snapshot.


Cookie parsing mis-parses when missing equals sign

Chrome and Firefox let you set cookies that don't have an equals sign:
document.cookie = "foo";
document.cookie = "session=my-flask-session";
document.cookie; // "foo; bar=my-flask-session;"

This breaks _cookie_re in https://github.com/mitsuhiko/werkzeug/blob/master/werkzeug/_internal.py#L46 - it treats foo; bar as the name of the cookie, since there's no =, causing both foo and bar to be missing from the request.
This might be the browser being non-compliant (I think https://tools.ietf.org/html/rfc6265#section-4.1.1 requires = and I've filed an issue at https://code.google.com/p/chromium/issues/detail?id=572541), but given the behavior it'd be nice for Flask to handle it.

Year, i have met the same question. Some cookie begins with ;, the followed by zero or more name-value-pair. The first ; as the name of the cookie, the first name is missing.

Fixed in d4087a3072c8387d2d4dd57f988d051a2e202b25


#180 - Modify -v/--version flag to display the version + Use -jv/--jenkinsVersion for the former behavior

This PR changes the functionality of the previous -v/--version flag (which is now called -jv/--jenkins-version) to display the version of the jenkinsfile-runner.
A few notes:

I hope I got all correct occurences of the "old" -v flag in the readme
I did not change version yet as this is a breaking change and did not want to change the wrong things
I added a new variable for the version which would be hardcoded and would have to be changed. I couldn't find any config file and parsing the pom.xml surely is not the way to go :P If there is a proper place where I would get the version number from otherwise, I will gladly change the code to do so.

I hope that the change I made is what you meant; that was the simplest I could come up with without starting to mess with maven plugins etc.

https://github.com/jenkinsci/jenkinsfile-runner/issues/185 ftr

No worries! Thanks for the opportunity to work on this and a PR that feels good for hacktoberfest :)


Update HDX Python Scraper

This change is in HDX Python Scraper and so this only requires updating that dependency

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1352

Relevant Lines:
1455

💛  - Coveralls

The database export is about 15% faster: https://github.com/OCHA-DAP/hapi-pipelines/actions/workflows/db_export.yaml


UITapGestureRecognizer is not working

Hi I'm trying to set UITapGestureRecognizer in the stickyHeader but It is not working.
Anyone know about this?
override func collectionView(collectionView: UICollectionView, viewForSupplementaryElementOfKind kind: String, atIndexPath indexPath: NSIndexPath) -> UICollectionReusableView {
    if kind == "UICollectionElementKindSectionHeader" {

       let stickySectionCell:PhotoSectionCell  = collectionView.dequeueReusableSupplementaryViewOfKind(kind, withReuseIdentifier: "PhotoSectionCell", forIndexPath: indexPath) as! PhotoSectionCell

        //Tap is not working
        let followingTap = UITapGestureRecognizer(target: self, action: #selector(PhotoUploadVC.galleryTab))
        followingTap.numberOfTapsRequired = 1
        stickySectionCell.galleryBtn.userInteractionEnabled = true
        stickySectionCell.galleryBtn.addGestureRecognizer(followingTap)

        return stickySectionCell

    }else {
        //CSStickyHeaderParallexHeader
        let headerCell:PhotoPrevHeaderCell  = collectionView.dequeueReusableSupplementaryViewOfKind(kind, withReuseIdentifier: "PhotoPrevHeaderCell", forIndexPath: indexPath) as! PhotoPrevHeaderCell

        //Tap is working fine
        let followingTap = UITapGestureRecognizer(target: self, action: #selector(PhotoUploadVC.imgTab))
        followingTap.numberOfTapsRequired = 1
        headerCell.prevImg.userInteractionEnabled = true
        headerCell.prevImg.addGestureRecognizer(followingTap)

        return headerCell;
    }

}

I solve this problem to change UICollectionViewCell -> UICollectionReusuableCell


Core || Indexing || Develop the routines to create, store and query a BM25 index on the metadata

Description
Acceptance criteria
Context

We have decided not to implement BM25 index for keyword search. We will instead implement sparse vector indices.


add tommi.space

Pingdom test result: https://tools.pingdom.com/#5e085bc560c00000

I'm getting 3.6 MB once I navigate to the main "English" homepage.
I can't in good conscience add just the simple splash screen asking what language the user prefers :stuck_out_tongue:

You’re right! Will work on it a bit more. Thank you very much.


Could not resolve... error after update to latest release

I update to latest release from 3. March and get the error below. Same happens also happens with a fresh yarn install after deleting node_modules
`
✘ [ERROR] Could not resolve<EMAIL_ADDRESS>node_modules/casdoor-vue-sdk/src/CasdoorSDK.js:1:30:
  1 │ ...asyncToGenerator from<EMAIL_ADDRESS>    ╵                          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

✘ [ERROR] Could not resolve "regenerator-runtime/runtime.js"
node_modules/casdoor-vue-sdk/src/CasdoorSDK.js:2:7:
  2 │ import "regenerator-runtime/runtime.js";
    ╵        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

You can mark the path "regenerator-runtime/runtime.js" as external to exclude it from the bundle,
which will remove this error.
`

npm: 9.1.3
node: v16.16.0v
yarn: v1.22.19

@stefanwuthrich hi, can you check starting from which release this issue happens? https://github.com/casdoor/casdoor-vue-sdk/releases

@hsluoyz from 1.4.0

@jakiuncle @imp2002

Confirm its fixed. Thank you, guys


Startup an OpenSearch Benchmark container, but put don't run tests up on startup

Description
Startup an OpenSearch Benchmark container, but put don't run tests up on startup.  Instead, put the tests into a shell script that can be invoked when the user wants to run it. That also lets the user run other tests too.

Category: Enhancement, New feature, Refactoring
Why these changes are required? Easier to control when a demo data load happens.  Also allows it to be run repeatedly, etc, etc.
What is the old behavior before changes and new behavior after changes? The benchmark loaded with fixed tests, then the container completed.  Now, the container stays ready for a dev to run the tests when they're ready, however often they like, and with easier overrides.

Issues Resolved
No jira
Testing
Manual test.  To run - exec into the opensearch-benchmarks container and run ./runTestBenchmarks.sh
Check List

[ ] New functionality includes testing

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented
[x] Commits are signed per the DCO using --signoff

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

I'm fine to approve once we add the basic auth user and password to the benchmark commands


Add default to unit item type and error type if no Message attr is present

As discussed in #4

thanks!


Clarify projectriff/buildpack OSS

From function.go:L49 the build image used for riff function create java is called projectriff/buildpack.
There does not appear to be any OSS repo to help riff users figure out where this image comes from, what it does, or what the constituent v3 buildpacks inside it are. https://github.com/projectriff/riff-buildpack appears to play a role but it is not the whole story.
cc @nebhale

Yep, this needs the help of a riff-buildpack-group repository (which can be started without pack CLI HTTP support).

Created https://github.com/projectriff/riff-buildpack-group with initial content


Cut release with Rails 7.2 Support

Seems that Rails 7.2 support was merged recently.
Is it possible to get a new version cut with that change?

@nicolas-brousse @Spone would either of you be able to cut a new version? 😄

@henrikbjorn @joelzwarrington I'll try to publish a new version tomorrow

v0.2.8 has been released 🙂


Update TypeScript Tooling in 5 minutes.md

Replaced normal string concat with JS template literal in example snippet of TypeScript Tooling in 5 minutes.md file.

Thanks, but I think we'll keep it as it is, less concepts to learn


Add units support for polybar #1651 #951

Implement feature requested in #1651 and #951.
This pr add supports for px (pixel) and pt (point) for polybar.
I followed the advices given by @patrick96 in #1651, so I add a size_with_unit structure to store the value and the unit. Unlike the size definition, the value is signed to handle negative offset.
To handle the width and height problem, I added a geometry_format_values structure which contains a percentage and the offset (a size_with_unit structure).
builder::space now uses the %{O} formatting tag but this is handled by size_with_unit_to_string helper in the new header "utils/unit.hpp" to avoid duplicate code (this function is also used in controller.cpp since I can't use builder::space here).
DPI initialization is now in the constructor of the bar. However this initialization seems strange, the DPI are only computed if dpi < 0 is explicitly specified in the configuration (see below). If the DPI isn't specified in the configuration, the default value is 96. Is it the correct behavior?
https://github.com/Lomadriel/polybar/blob/cc4e09edc41cfbd5f2f4bf7235470aa1c21bc59a/src/components/bar.cpp#L155-L178
The rest of the PR is just adapting the code to use these structures. I tried to find all keys that could uses units but I am not sure if I added units support for all the key that could benefit of this feature.
EDIT: Fixes #1700
TEAMEDIT:
Closes #1651
Closes #951
Fixes #1265

Codecov Report

Merging #1671 into master will decrease coverage by 0.08%.
The diff coverage is 22.05%.

@@            Coverage Diff             @@
##           master    #1671      +/-   ##
==========================================
- Coverage   23.34%   23.26%   -0.09%
==========================================
  Files          50       51       +1
  Lines        2090     2128      +38
==========================================
+ Hits          488      495       +7
- Misses       1602     1633      +31

Flag
Coverage Δ

#unittests
23.26% <22.05%> (-0.09%)
:arrow_down:

Impacted Files
Coverage Δ

include/components/builder.hpp
100% <ø> (ø)
:arrow_up:

include/components/types.hpp
75% <ø> (ø)
:arrow_up:

src/components/config.cpp
0.62% <0%> (-0.11%)
:arrow_down:

src/components/builder.cpp
5.95% <0%> (-0.02%)
:arrow_down:

src/drawtypes/label.cpp
2.23% <0%> (ø)
:arrow_up:

include/drawtypes/label.hpp
90% <100%> (+10%)
:arrow_up:

include/components/bar.hpp
100% <100%> (ø)
:arrow_up:

include/utils/unit.hpp
42.85% <42.85%> (ø)

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 db9a83a...d5f88fe. Read the comment docs.

I fixed the unit tests. Should I add a test for the function template <> geometry_format_values config::convert(string&& value) const? (which does the string -> geometry_format_values conversion that isn't checked anymore)

I just remove some bugs (negative margin, padding...).
Normally this should also fix #1265.

DPI initialization is now in the constructor of the bar. However this initialization seems strange, the DPI are only computed if dpi < 0 is explicitly specified in the configuration (see below). If the DPI isn't specified in the configuration, the default value is 96. Is it the correct behavior?

Moving it into bar.cpp is fine. I don't know what the original intention was. But since that part of the code is over 2 years old, I'd say it's the expected behavior now ;)

If that's the expected behavior, maybe the documentation/wiki should be modified to state that any value <= 0 cause the DPI to be auto-detected. Moreover, this detection is bugged, since if I change the resolution of my screen (like when I duplicate my laptop screen and the projector is in 4/3), I have to relaunch polybar to update this value. (Maybe an x event should cause the value to be updated)

I think we need to separate how we specify sizes and spacing.
Spacing can be in spaces, pixel, or points while size can be percentages, pixel, points or those combined in the offset format.
I first thought we can keep this together because it has lots of things in common (POINT and PIXEL), but I don't think this is worth it.

I agree, I tried to do my best but this is really confusing.

The more I think about it, the less I am a fan of converting points to pixels only when the value in pixels is needed and passing around point values. I know this is actually what I suggested we do originally because the builder would be the only place where this conversion happens. But this would make things quite inflexible because if we ever extend our code we would need to do the conversion in all places that use those pixel values.

The following paragraph supposes that the DPI might become dynamic otherwise its content should be ignored.
For that point, I think this might be a problem if the DPI changes during the execution.
Since the "point" is here to have a DPI sensitive unit, it is quite strange to convert point sizes into pixel sizes when charging the configuration. If the DPI changes, we would need to update every sizes and it is quite expensive.
Maybe introducing a sort of settings singleton that will hold some important values (like the DPI) could solve the problem even if I don't like introducing singleton.
It will allow to have a space_size struct like that:
struct space_size {
  space_type type;
  // If type == space_type::pixel converts the value in pixel if necessary and returns it.
  // Otherwise returns the size in number of spaces.
  // No need to use float in the return type since pixel or space doesn't need them.
  ssize_t get_value() const {
    if (type == space_type::point)
      return convert_to_pixel(settings::get_instance().get_dpi_x(), value);
    else
      return value;
  }
private:
  // to be sure that nobody uses it directly
  float value; // could be an union if you want since the get_value is the only place
                    // were the union will be used.
};

The geometry_size struct could be defined in a similar way with an enum type like that:
enum class geometry_size_type { PERCENTAGE, POINT, PIXEL }

enum size_type { SPACE, PIXEL };

Do you want me to use a plain C enum or may I use an enum class? (I prefer using enum class to avoid global namespace pollution)
What do you think of this?

Everything within the bar except for the modules themselves is quite static. Polybar cannot dynamically change its size or position when changes to the monitor occur. All of this is handled through screenchange-reload which just reloads polybar whenever the screen changes.
It is conceivable that we want to make things more dynamic in the future. But even then, I would not convert to pixels in the builder but rather extend the %{O} tag to also work with points and then let the renderer handle it. Either way seems fine to me, thinking about it, doing this in the renderer may even be a little cleaner. This is up to you how you want to do it, I just don't want it to happen somewhere in the middle where we potentially have to do the conversion in multiple places.
Adding get_value might look very clean in the code that uses it, but it introduces a singleton and couples the concept of units for spacing purposes to the bar class. I want to avoid that.

The geometry_size struct could be defined in a similar way with an enum type like that:
enum class geometry_size_type { PERCENTAGE, POINT, PIXEL }

Wouldn't geometry_size_type only have point and pixel? As far as I know, percentages cannot be used as offsets.

Do you want me to use a plain C enum or may I use an enum class? (I prefer using enum class to avoid global namespace pollution)

You can use enum class. I just used enum because I'm more familiar with C ;)

Everything within the bar except for the modules themselves is quite static. Polybar cannot dynamically change its size or position when changes to the monitor occur. All of this is handled through screenchange-reload which just reloads polybar whenever the screen changes.

I didn't know it worked that way, but that's seems logical with the fact that some of my modules are reset when I change my resolution.

It is conceivable that we want to make things more dynamic in the future. But even then, I would not convert to pixels in the builder but rather extend the %{O} tag to also work with points and then let the renderer handle it. Either way seems fine to me, thinking about it, doing this in the renderer may even be a little cleaner. This is up to you how you want to do it, I just don't want it to happen somewhere in the middle where we potentially have to do the conversion in multiple places.

Maybe I can tried to do that to see if it is more clean.

The geometry_size struct could be defined in a similar way with an enum type like that:
enum class geometry_size_type { PERCENTAGE, POINT, PIXEL }

Wouldn't geometry_size_type only have point and pixel? As far as I know, percentages cannot be used as offsets.

Yeah my bad, it is just point and pixel.

By the way, do you have an idea for naming the enums and structs associated with that:

spacing:

POINT: float >= 0
PIXEL: ssize_t
SPACE: ssize_t

size:

POINT: float
PIXEL: size_t

Each time I found a name it is either bad or already taken (like size)

Since we probably put these inside a namespace, naming is not such a big issue, but I think the following names are reasonable. Feel free to improve them.
enum class size_type { POINT, PIXEL }
enum class space_type { POINT, PIXEL, SPACE }

struct spacing {
  space_type type;
  ...
}

struct geometry {
  float percentage;
  ...
}

I just finished some modifications to take your review in account. Just as expected, there are two structures, one for spacing and the other for size.
The %{O} tag now supports units (%{O100px} and %{O100pt}).
There is also a commit that fixes the problem mentioned in #1527.

It would probably be useful if we also allow the use of these units in format-padding, format-offset etc. Those are set in modules/meta/base.cpp

The issue in #1700 is still not totally fixed. Offsets do produce spaces that have the proper background color, but the string I used in #1700 still does not look the same as with lemonbar:
%{O100}Filler%{B#ff0000 O100}First%{O200}Second%{O-100}

With this PR:

With lemonbar:

You see, Second is not displayed with this PR:

Fixed by https://github.com/jaagr/polybar/pull/1671/commits/fa1719ad08a770ef3b25d65bd497b3aecd0663b4

It would probably be useful if we also allow the use of these units in format-padding, format-offset etc. Those are set in modules/meta/base.cpp

If I am not wrong format-padding already support units.
https://github.com/jaagr/polybar/pull/1671/commits/ab19b3be526945857ed25b9c74935e3f71373d74 adds unit support for format-offset

I just added a commit that fixes #1814.
I can't make a separate PR since I modified the renderer :/
Sorry

I think you should make a separate PR and resolve the merge conflict once one of the two is merged.

I think you should make a separate PR and resolve the merge conflict once one of the two is merged.

Commit removed. See #1817.

Can this be rebased against master please

I squashed and rebased this against master (I tried to rebase without squashing but kept screwing up the history, so figured I didn't need to keep it anyway just to build and run it)
https://github.com/frebib/polybar/commit/7385b894a795058cd4e0db14c9320b3e8fda5c66
I rely on this fork to have polybar even remotely usable when changing DPI. Please consider reviving it @Lomadriel

Any way to help get this PR merged? E.g. implement requested code changes? I'd really love to see these units to improve HiDPI support :+1:

Until now I've been rebasing this PR onto master, but https://github.com/polybar/polybar/commit/fd556525a892d62d8a35595e0fb5cb648253d31c was a big change that will not be a trivial change to rebase onto. I may or may not attempt to fix this up again, depending on how hard it seems to be

Until now I've been rebasing this PR onto master, but https://github.com/polybar/polybar/commit/fd556525a892d62d8a35595e0fb5cb648253d31c was a big change that will not be a trivial change to rebase onto. I may or may not attempt to fix this up again, depending on how hard it seems to be

@frebib If you do attempt to resolve the conflicts and want to help get this merged, a PR would be very much appreciated :)
And of course, ask if you need help.
I think most conflicts should be quite straighforward (just tedious to get the changes from both sides in), but in the renderer and parser, it might be more difficult because there we actually changed the places where the offset handling happens.

@frebib If you do attempt to resolve the conflicts and want to help get this merged, a PR would be very much appreciated :)
And of course, ask if you need help.
I think most conflicts should be quite straighforward (just tedious to get the changes from both sides in), but in the renderer and parser, it might be more difficult because there we actually changed the places where the offset handling happens.

@patrick96 I can certainly give it a go, but I'm not at all familiar with C++ or much of the syntax/language features.
If I were to rebase this, it'd be squashed into a single commit- hopefully that's ok.
There were a few unaddressed comments from what I remember about the implementation which I should probably address at the same time.
In use I also noticed that there are still some glaring issues with unit support particularly in the tray area. I managed to hack in support for most of it, but the padding between the icons is still not correctly scaled. Hopefully that shouldn't be too tricky to solve.

If I were to rebase this, it'd be squashed into a single commit- hopefully that's ok.

Yeah, that's fine. This would have happened with this PR anyway, I think.

I can certainly give it a go, but I'm not at all familiar with C++ or much of the syntax/language features.

In that case, this might get tricky. Let me know, if you need any help.

In use I also noticed that there are still some glaring issues with unit support particularly in the tray area. I managed to hack in support for most of it, but the padding between the icons is still not correctly scaled. Hopefully that shouldn't be too tricky to solve.

Yeah, I don't think this PR touches the tray at all. Feel free to leave out units in the tray if that's easier. First priority should be to get the overall framework working.

I gave it a go. Seems to compile & run the same as the old builds did for me.
See #2394


HTS221 missed header hts221_reg.h

Describe the bug
There is no header hts221_reg.h both in src folder and in include
To Reproduce
./scripts/twister -T tests/drivers/build_all/sensor
Expected behavior
Clean build expected
Impact
Can't run tests during sensor development
Environment (please complete the following information):

OS: Linux
Toolchain GNU ARM v10.2
Commit v.2.7.1

my bad, corrupted initialization


refer to spec. memory.grow will push -1 to the stack if failed

Codecov Report

Merging #15 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
- Coverage   46.14%   46.13%   -0.02%
==========================================
  Files          39       39
  Lines        3788     3789       +1
==========================================
  Hits         1748     1748
- Misses       1757     1758       +1
  Partials      283      283

Impacted Files
Coverage Δ

exec/memory.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 9aef4b6...19819bd. Read the comment docs.


intel_adsp_cavs25: support more than 2 DSP cores

Is your enhancement proposal related to a problem? Please describe.
Current cavs25 implementation is limited to max 2 DSP cores while we need to support SKUs upto 4 cores:
zephyr/soc/xtensa/intel_adsp/common/soc_mp.c:arch_start_cpu()
»       __ASSERT(cpu_num == 1, "Only supports only two CPUs!");

Describe the solution you'd like
Support full set of 4 DSP cores.
Describe alternatives you've considered
n/a
Additional context
n/a

FYI @nashif @andyross @lgirdwood @lyakh

Closing this.  Support for all four cores is present and works on TGL through all the core tests.  The default CONFIG_MP_NUM_CPUS setting is still 2 though, out of conservatism (not wanting to expose bugs in the test suite late in a release cycle), and the fact that existing SOF topologies only want to exercise two cores anyway.


Normalize score if submission converges faster

Implements rule from PR #503 in training_policies repo.

Prints normalization factor in RCP checker if not equal to 1.0
Sets scaling.json file in results folder if score should be scaled

Are there any other components that write the scaling.json file? I am wondering if we need to avoid overwriting the file.


Command line arguments for custom filepaths

Add:

Custom paths as cli arguments for user.cfg, database, apprise.yml and supported_coin_list's files.

Usage example: python -m btb_manager_telegram --path "../bot/" --config "custom-file.cfg" --database "custom-relative-path/database.db" --coinlist "custom-relative-path/coin_list" --apprise "custom-relative-path/apprise.yml"

Dev environment files (poetry) and nohup log to .gitignore.

Fix/Refactor:

Removed extra spaces in requirements.txt between package names and versions.
Unused imports removed.

Hey @renkasiyas!
I'm having a hard time understanding the purpose of being able to manually set the path for each file of the bot.
Would you explain the use cases of such options?

Sure!
When using more than one trade bot, its docker-compose file lets you make a container for each bot using the same installation directory, only aliasing / making volumes for specific files.
Example:

binance_trade_bot/ <- installation dir

data/

bot_1.db

bot_2.db

config/

apprise_1.yml

apprise_2.yml

coinlists/

coinlist_bot_1

coinlist_bot_2

bot_1.cfg

bot_2.cfg

You could fire each telegram manager with nohup with one command pointing to the same install:
python -m btb_manager_telegram --path "../bot/" --config "bot_2.cfg" --database "data/bot_2.db" --coinlist "coinlists/coinlist_bot_2" --apprise "config/apprise_2.yml"
But, for the telegram manager you need each bot instance to have its own directory, which leads to a lot of folders with the same files, except for a few ones (apprise config, database, bot config, coin list).
I'm using 6 bots so, I think it's so lean having only one binance_trade_bot dir, and one telegram_manager dir for all of them.


Added dummy user initialization for UWP.

Added option to skip authentication in UWP version to ease testing.
Also changed the reference to SQLite with a Nuget Package as Build fails for those who don't have SQLite for UWP installed.
--Fixed navigation happening without SplitShell on Skipping authentication.

Is the repository still maintained?


[GLUTEN-1205][VL] Refactor shuffle partitioner

What changes were proposed in this pull request?
（1）Decoupling Partitioner(compute partition) and ShuffleWriter
（2）Add RoundRobinPartitioner 、SinglePartPartitioner、HashPartitioner、 FallbackRangePartitioner to support different partition methods
(Fixes: #1205)

The issues mentioned above have been resolved,  please take a look again, thanks @jinchengchenghh


i18n failed

when i visit index html, error occur below.
it might be because data-i18n="traces.lookback"> (templates/index.mustache)have not been defined in i18n files.
jquery.i18n.properties.js:271 Uncaught TypeError: Cannot read property 'length' of undefined
    at Object.$.i18n.prop (jquery.i18n.properties.js:271)
    at HTMLSpanElement.<anonymous> (i18n.js:22)
    at Function.each (jquery.js:354)
    at De.fn.init.each (jquery.js:189)
    at Object.callback (i18n.js:18)
    at callbackIfComplete (jquery.i18n.properties.js:306)
    at loadAndParseFiles (jquery.i18n.properties.js:322)
    at jquery.i18n.properties.js:319
    at Object.success (jquery.i18n.properties.js:349)
    at l (jquery.js:3268)

happened again today:
jquery.js:9600 GET http://***:31399/zipkin/trace_zh.properties 404 (Not Found)
Uncaught TypeError: Cannot read property 'length' of undefined jquery.i18n.properties.js:271
from @GuangmingLuo

https://github.com/jquery-i18n-properties/jquery-i18n-properties/issues/103

same problem, anyone help?

@JoveYu is same for you? on 'trace_zh.properties'?

I will try to workaround the problem in the build but making a copy of XXX_zh_CN.properties as XXX_zh.properties

attempted fix waiting on feedback https://github.com/openzipkin/zipkin/pull/2238

zipkin-classic (https://github.com/openzipkin-attic/zipkin-classic) is no longer maintained, Lens (the new UI) has its own I18N system added in #2487.


fix approve and reject ride

now we can really drop the RLS policy for the rides. Please aslo review the functions in supabase (approve_ride, reject_ride, is_update_allowed)

@istzen wouldn't it be easier (and less functions!) if we just had a set_ride_status method with a status parameter? That way we could also use RideStatus.approved.index in flutter instead of placing that random "2" in the supabase function.

@istzen wouldn't it be easier (and less functions!) if we just had a set_ride_status method with a status parameter? That way we could also use RideStatus.approved.index in flutter instead of placing that random "2" in the supabase function.

Yes that would be easier but would also give the users more rights than nessacary.


Add custom PersistentVolumeClaim support for Prometheus

Describe the feature request
When Prometheus restarted, it will lose all the data.
Describe alternatives you've considered
Can istio support custom PersistentVolumeClaim for Prometheus and jaeger?

Hey guys, we have the same issue that all the data are lost after restart... We are using istio's chart, but maybe would be good to migrate here smth from the "default chart"?
https://github.com/helm/charts/tree/master/stable/prometheus

I believe there is PVC support via the prom operator with the new installer. I'm going to mark this as resolved given that forward path. Please re-open if you feel more is needed here.


Unable to upload new Android apk

Does anyone have experience with signing app for android, using cordova? I used to be able to generate an apk file which could be uploaded to Google Play, but the signing method has changed recently and now I'm unable to sign. Google Play throws an error You uploaded an APK with an invalid signature (learn more about signing). Error from apksigner: ERROR: MIN_SIG_SCHEME_FOR_TARGET_SDK_NOT_MET: Target SDK version 30 requires a minimum of signature scheme v2; the APK is not signed with this or a later signature scheme . I used to use jarsigner, but now have to use apksigner, which I'm trying to use in the developing terminal, but also throws an error saying apksigner : The term 'apksigner' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

The android SDK target needs to be 30. You need at least the cordova android 10 library

the cordova stuff isnt checked into the project but you should be able to follow this: https://quasar.dev/quasar-cli/developing-cordova-apps/preparation
but when you add android specify 10
cordova platform add<EMAIL_ADDRESS>

I set up the Android development environment on my machine now and the app runs in the emulator. With the problematic Android SDK 30 even it seems. So once I get access to the Google Play Developer Account releasing the app shouldn't be a big deal, normally speaking.

Signing has now been succesfull. It appeared we had to update android as mentioned by @jphillips05, and we also had to remove a few deprecated cordova plugins.


db options does not fire select db on createClient when url (with specified db number) is provided

Version: 2.8.0
Platform: Node JS 12.8.0 on MacOS Mojave 10.14.5
Description:
as the title said, I expect when the db option is provided, it will fire select command on connect using the db option provided regardless whether url provides db number or not.

How to reproduce:
const redis = require('redis')
const url = 'redis://<IP_ADDRESS>:6379/0'
const client = redis.createClient({db: 3, url: url})
client.selected_db // this will return 0, instead of 3 which I expect
client.set('foo', 'bar') // upon checking on redis-cli this is written to db 0, not on db 3

Good catch. In reviewing createClient, the url takes precedence over any other options that can  be specified inside the URL. The options object will be evaluated first, then the url inside that object - so specifying db of 3 would be accepted, then immediately overwritten by what is in the URL. It makes sense and would be breaking change to do otherwise.
However, I see your point in how this is confusing. We should document this behaviour and possibly throw a warning (maybe an actual error) for any option that is being overwritten by the URL.

It is indeed a good idea to throw an error in such case / to at least log something for now.
The main issue is that it applies to multiple options and not only the db option.

Have there been any updates to this issue?


[BUG] Data Loss when Recreating Pods on ARM64

Describe the bug
When the pods for my postfix container are terminated and recreated on my arm64 nodes, the volume loses all data.    Works fine on x86-64.  This happens consistently on my cluster.  The disk faults and gets recreated.
To Reproduce

Create a statefulset using a longhorn volume and have it run on an arm64 node
Write data to the volume
Delete a pod from the statefulset
Wait for volume to fault
Wait for pod to spin back up
Data is gone

Expected behavior
Haz dataz
Support bundle for troubleshooting
Will be emailed, if it ever completes
Environment
K8s controller running on an athanaric node, other nodes are raspberry pi's.
Longhorn version 1.6.0 (also an issue on 1.5.3), installed via kubectl with minimal customization
13 Longhorn volumes
Each node has a longhorn disk
(venv) c4a-admin@c4a-1:~/projects/src$ kubectl get nodes
NAME        STATUS   ROLES           AGE    VERSION
athanaric   Ready    control-plane   380d   v1.25.5 # AMD Ryzen 5 5600G with 16 GB of memory, zfs pool on NVME SSD's at /safe-2
c4a-1       Ready    <none>          380d   v1.25.5 # RPI 4 with 4 GB of memory and USB->SATA HDD mounted at /media/hdd
c4a-2       Ready    <none>          380d   v1.25.5 # RPI 4 with 4 GB of memory and USB->SATA HDD mounted at /media/hdd
c4a-3       Ready    <none>          380d   v1.25.5 # RPI 4 with 4 GB of memory and USB->SATA HDD mounted at /media/hdd
c4a-4       Ready    <none>          380d   v1.25.5 # RPI 4 with 4 GB of memory and USB->SATA HDD mounted at /media/hdd
c4a-5       Ready    <none>          380d   v1.25.5 # RPI 4 with 8 GB of memory and USB->SATA SSD mounted at /media/hdd
c4a-6       Ready    <none>          380d   v1.25.5 # RPI 4 with 8 GB of memory and USB->SATA SSD mounted at /media/hdd
c4a-7       Ready    <none>          380d   v1.25.5 # RPI 4 with 8 GB of memory and USB->SATA SSD mounted at /media/hdd
c4a-8       Ready    <none>          380d   v1.25.5 # RPI 4 with 8 GB of memory and USB->SATA SSD mounted at /media/hdd
scribe      Ready    <none>          336d   v1.25.5 # AMD Ryzen 7 3700X with 32 GB of memory, NVME SSD at /

c4a-*: Ubuntu 22.04.1, Linux c4a-1 5.15.0-1044-raspi,  Note: /var is a symlink to /media/hdd/var
athanaric: Ubuntu 20.04.6, Linux athanaric 5.4.0-170-generic
scribe: Ubuntu 20.04.6, Linux scribe 6.5.0-14-generic
Network bandwidth: c4a-* nodes have 1gbps links, athanaric and scribe and 10gbps links
Impacted volumes: pvc-c778c60f-9495-4a5c-b30b-53d69ee68b74, pvc-fc8162b5-fe8e-46db-84ab-1e7a7f0098a0
Additional context
(venv) c4a-admin@c4a-1:~/projects/src/kubernetes-configs/longhorn$ diff 1.6.0/longhorn.yaml.1 longhorn.yaml
2,7d1
< # Builtin: "helm template" does not respect --create-namespace
< apiVersion: v1
< kind: Namespace
< metadata:
<   name: longhorn-system
< ---
4227c4221
<   name: longhorn-frontend
---
>   name: public
4230c4224
<   type: ClusterIP
---
>   type: LoadBalancer
4237c4231,4233
<     nodePort: null
---
>   - name: https
>     port: 443
>     targetPort: https
4560a4557,4568
>       - name: nginx-https
>         image: nginx
>         ports:
>         - containerPort: 80
>           name: http
>         - containerPort: 443
>           name: https
>         volumeMounts:
>         - name: nginx-config-https
>           mountPath: /etc/nginx
>         - name: nginx-certs
>           mountPath: /etc/nginx-tls
4567a4576,4581
>       - name: nginx-config-https
>         configMap:
>           name: longhorn-https-nginx-config
>       - name: nginx-certs
>         secret:
>           secretName: longhorn-https

@PhanLe1010 please follow up with this.

Partial event log posted above.  Faulted volume replicas were stored on the c4a Raspberry Pi nodes before and after build.

Actually, deleting pods on arm64 and recreating on arm64 also tends to hang for a long time, and eventually I give up and force delete the pods.  That's probably a separate issue though.

Support bundle completed, but it's ~400MB.  How should I get it to you?

I've uploaded the support bundle to my s3 bucket and emailed you a link to it.

Thanks @jroose
Sorry for the delay. I am checking this ticket tomorrow PST time

Any luck?

Sorry @jroose I have not check this yet as busy on other task. I will be on this by end of today or tomorrow

Is it correct that c4a is ARM-64 while athanaric and scribe nodes are x86-64 ?

No worries!  I just wanted to make sure it didn't fall through the cracks.  Yes, you are correct that all 8 c4a nodes are arm64, while athanaric and scribe are x86-64.

@PhanLe1010 -- is there more information I can provide?


Error building source for some numerical error (Linux)

Hello!
I'm trying to get this game running on Linux ( distro Mint-Cinnamon, Ubuntu based), and am running into some issues.
I got all the dependencies sorted out, and successfully generated both ninja and make files (in separate directories), but trying to build gives me the exact same error for both:
ninja:
ninja: Entering directory `/home/iijj22gg/Downloads/etterna-0.70.3/'
[729/1168] Building C object extern/newogg/CMakeFiles/ogg.dir/src/bitwise.c.o
FAILED: extern/newogg/CMakeFiles/ogg.dir/src/bitwise.c.o
/usr/bin/cc  -Iextern/newogg/include -w -MD -MT extern/newogg/CMakeFiles/ogg.dir/src/bitwise.c.o -MF extern/newogg/CMakeFiles/ogg.dir/src/bitwise.c.o.d -o extern/newogg/CMakeFiles/ogg.dir/src/bitwise.c.o   -c extern/newogg/src/bitwise.c
In file included from extern/newogg/include/ogg/os_types.h:143,
                 from extern/newogg/include/ogg/ogg.h:25,
                 from extern/newogg/src/bitwise.c:24:
extern/newogg/include/ogg/config_types.h:57:2: error: #error "No valid unsigned 16-bit types found."
   57 | #error "No valid unsigned 16-bit types found."
      |  ^~~~~
extern/newogg/include/ogg/config_types.h:83:2: error: #error "No valid unsigned 32-bit types found."
   83 | #error "No valid unsigned 32-bit types found."
      |  ^~~~~
[730/1168] Building C object extern/ne...CMakeFiles/vorbis.dir/lib/analysis.c.o
FAILED: extern/newvorbis/CMakeFiles/vorbis.dir/lib/analysis.c.o
/usr/bin/cc  -Iextern/newvorbis/lib -Iextern/newvorbis/include -Iextern/newogg/include -w -MD -MT extern/newvorbis/CMakeFiles/vorbis.dir/lib/analysis.c.o -MF extern/newvorbis/CMakeFiles/vorbis.dir/lib/analysis.c.o.d -o extern/newvorbis/CMakeFiles/vorbis.dir/lib/analysis.c.o   -c extern/newvorbis/lib/analysis.c
In file included from extern/newogg/include/ogg/os_types.h:143,
                 from extern/newogg/include/ogg/ogg.h:25,
                 from extern/newvorbis/lib/analysis.c:21:
extern/newogg/include/ogg/config_types.h:57:2: error: #error "No valid unsigned 16-bit types found."
   57 | #error "No valid unsigned 16-bit types found."
      |  ^~~~~
extern/newogg/include/ogg/config_types.h:83:2: error: #error "No valid unsigned 32-bit types found."
   83 | #error "No valid unsigned 32-bit types found."
      |  ^~~~~
In file included from extern/newvorbis/lib/codec_internal.h:22,
                 from extern/newvorbis/lib/analysis.c:23:
extern/newvorbis/lib/codebook.h:68:3: error: unknown type name ‘ogg_uint32_t’
   68 |   ogg_uint32_t *codelist;   /* list of bitstream codewords for each entry */
      |   ^~~~~~~~~~~~
extern/newvorbis/lib/codebook.h:72:3: error: unknown type name ‘ogg_uint32_t’
   72 |   ogg_uint32_t *dec_firsttable;
      |   ^~~~~~~~~~~~
In file included from extern/newvorbis/lib/os.h:27,
                 from extern/newvorbis/lib/bitrate.h:23,
                 from extern/newvorbis/lib/codec_internal.h:59,
                 from extern/newvorbis/lib/analysis.c:23:
extern/newvorbis/lib/misc.h:24:20: error: unknown type name ‘ogg_uint32_t’; did you mean ‘ogg_int32_t’?
   24 | extern int ov_ilog(ogg_uint32_t v);
      |                    ^~~~~~~~~~~~
      |                    ogg_int32_t
In file included from extern/newvorbis/lib/analysis.c:25:
extern/newvorbis/lib/scales.h: In function ‘unitnorm’:
extern/newvorbis/lib/scales.h:35:5: error: unknown type name ‘ogg_uint32_t’
   35 |     ogg_uint32_t i;
      |     ^~~~~~~~~~~~
extern/newvorbis/lib/scales.h: In function ‘todB’:
extern/newvorbis/lib/scales.h:46:5: error: unknown type name ‘ogg_uint32_t’
   46 |     ogg_uint32_t i;
      |     ^~~~~~~~~~~~
[732/1168] Building C object extern/newogg/CMakeFiles/ogg.dir/src/framing.c.o
FAILED: extern/newogg/CMakeFiles/ogg.dir/src/framing.c.o
/usr/bin/cc  -Iextern/newogg/include -w -MD -MT extern/newogg/CMakeFiles/ogg.dir/src/framing.c.o -MF extern/newogg/CMakeFiles/ogg.dir/src/framing.c.o.d -o extern/newogg/CMakeFiles/ogg.dir/src/framing.c.o   -c extern/newogg/src/framing.c
In file included from extern/newogg/include/ogg/os_types.h:143,
                 from extern/newogg/include/ogg/ogg.h:25,
                 from extern/newogg/src/framing.c:26:
extern/newogg/include/ogg/config_types.h:57:2: error: #error "No valid unsigned 16-bit types found."
   57 | #error "No valid unsigned 16-bit types found."
      |  ^~~~~
extern/newogg/include/ogg/config_types.h:83:2: error: #error "No valid unsigned 32-bit types found."
   83 | #error "No valid unsigned 32-bit types found."
      |  ^~~~~
extern/newogg/src/framing.c:121:14: error: unknown type name ‘ogg_uint32_t’
  121 | static const ogg_uint32_t crc_lookup[256]={
      |              ^~~~~~~~~~~~
extern/newogg/src/framing.c: In function ‘ogg_page_checksum_set’:
extern/newogg/src/framing.c:295:5: error: unknown type name ‘ogg_uint32_t’; did you mean ‘ogg_int32_t’?
  295 |     ogg_uint32_t crc_reg=0;
      |     ^~~~~~~~~~~~
      |     ogg_int32_t
[734/1168] Building C object extern/glew-1.5.8/CMakeFiles/glfw.dir/src/glew.c.o
ninja: build stopped: subcommand failed.

make:
make: Entering directory '/home/iijj22gg/Downloads/etterna-0.70.3'
make[1]: Entering directory '/home/iijj22gg/Downloads/etterna-0.70.3'
make[2]: Entering directory '/home/iijj22gg/Downloads/etterna-0.70.3'
Scanning dependencies of target ogg
make[2]: Leaving directory '/home/iijj22gg/Downloads/etterna-0.70.3'
make[2]: Entering directory '/home/iijj22gg/Downloads/etterna-0.70.3'
[  0%] Building C object extern/newogg/CMakeFiles/ogg.dir/src/bitwise.c.o
In file included from /home/iijj22gg/Downloads/etterna-0.70.3/extern/newogg/include/ogg/os_types.h:143,
                 from /home/iijj22gg/Downloads/etterna-0.70.3/extern/newogg/include/ogg/ogg.h:25,
                 from /home/iijj22gg/Downloads/etterna-0.70.3/extern/newogg/src/bitwise.c:24:
/home/iijj22gg/Downloads/etterna-0.70.3/extern/newogg/include/ogg/config_types.h:57:2: error: #error "No valid unsigned 16-bit types found."
   57 | #error "No valid unsigned 16-bit types found."
      |  ^~~~~
/home/iijj22gg/Downloads/etterna-0.70.3/extern/newogg/include/ogg/config_types.h:83:2: error: #error "No valid unsigned 32-bit types found."
   83 | #error "No valid unsigned 32-bit types found."
      |  ^~~~~
make[2]: *** [extern/newogg/CMakeFiles/ogg.dir/build.make:63: extern/newogg/CMakeFiles/ogg.dir/src/bitwise.c.o] Error 1
make[2]: Leaving directory '/home/iijj22gg/Downloads/etterna-0.70.3'
make[1]: *** [CMakeFiles/Makefile2:1658: extern/newogg/CMakeFiles/ogg.dir/all] Error 2
make[1]: Leaving directory '/home/iijj22gg/Downloads/etterna-0.70.3'
make: *** [Makefile:152: all] Error 2
make: Leaving directory '/home/iijj22gg/Downloads/etterna-0.70.3'

I'm not sure if I've done something wrong or if this is a bug, but I find this strange.

duplicate #620 ?

Not again... it's the one search term I didn't think of when trying to search for similar issues.
But yes, it's the same error. I must try to get cmake to cooperate within the build folder.

Thanks!


Add package json

Should I take out the line that says jquery trend is required in the readme?

Nice, looks good.
I think we should leave the note about Trend in the readme for the cases where it's not being installed with npm.


allow to make OpenAPI disable or enable for each user

Is your feature request related to a problem?
Hello is it possible to make OpenAPI disable or enable in setting
Describe the solution you'd like
would be great if users can make OpenAPI of their own enable or disable
Additional context
No response

OpenID will be removed in the next version. The access token will be displayed in the UI.

A new access token will be created for each login operation.
User can manually create access tokens that will not expire to call the api.


Demos on actual documents

Hello! Thank you for making this available - I was wondering if there are any demos that you could share where text/information is extracted from a PDF? I have seen that the current scripts uploaded are for fine-tuning, but it would be nice to have a script for just inference on new docs! Thank you

refer to this PR https://github.com/microsoft/i-Code/pull/36 for more details/demo before it is merged.


Update Structor

Updated plugin metadata to the new scheme.
https://github.com/toolCHAINZ/structor/releases/tag/v1.0

I think you created a tag but no associated release.

Ok, fixed!

Awesome thanks, its integrated.


Make normal tiles available to the autotile editor- so we can use it for setting collisions and other things on all tiles

if a tile that is available in the tileset is not set to 'Autotile', still list it in the tiles list on the left- but make the bitmask options and tab unavailable for it.
That way we can use the editor on both normal and auto tiles and be able to edit collisions,occlusion,navigation and priority of normal tiles as well

A workaround I found is to set all tiles to autotiles- so I can use the autotile editor for setting collisions and other things
@MarianoGnu might be interested in this one

https://github.com/godotengine/godot/pull/16772
its in


Feature: support for multiple files for the logpath directive into jail definition

When specifying multiple files into the logpath jail directive in a jail.conf file, correct syntax is to split them with \n (see man 5 jail.conf).
I would really prefer using an Array instead of a very long String split with multiple \n.
Moreover, since v0.9.0 there is an optional tail or head directive that can follow a given path.
So I think the fail2ban::jail defined type should take an Array as $log_path parameter:
define fail2ban::jail(
...
-  Optional[Stdlib::Absolutepath] $log_path = undef,
+  Optional[Array[Stdlib::Absolutepath]] $log_path = [],
...)

I'm not 100% sure that Stdlib::Absolutepath would accept a wildcard in pathname, or anything like /var/log/apache2/*.log tail.
What do you think about that ? I can submit a patch if you agree with the need to pass an Array as parameter. The patch would support both String and Array as $log_path parameter.
Regards,

I think this is a good idea. How would you feel about making it Variant[Array[...], String] and then handling the string as a one-element array?
Absolutepath does not accept wildcards. It would be nice to accept them, but also check that the path with wildcard is valid.
And also to accept 'tail' as per the jail.conf manpage.
I very much appreciate your contributions to this module.


Struggling to get it to work behind jupyter-server-proxy

Hello, I'm trying to get this up and running behind a jupyter-server-proxy and I'm getting some strange errors.
For context I have a JupyterHub server running in AWS that I use for development. Usually I can proxy ports for things like streamlit and code-server using jupyter-server-proxy by launching the process and visiting {base_url}/proxy/{port}/ but it doesn't seem to be working with mage-ai
Steps I have followed:

pip installed mage-ai
run mage start demo_project and it creates the scaffolding and such then starts running the server like so:

Tried to visit the {base_url}/proxy/6789/url and I get a blank white page and some 404 errors in the JS Console along the lines of GET https://{jupyter_hub_url}/hub/_next/static/css/9563de380da227ba.css net::ERR_ABORTED 404

You can see on that screenshot that requests are getting to the web server but it looks like it's having trouble retrieving the assets for the page looks like it should be trying to retrieve them from {base_url}/proxy/6789/_next/static/css/9563de380da227ba.css (I can get the CSS file if I make a GET request to this address)
Any ideas why the routing isn't working? and why it seems to work fine with other projects e.g. streamlit ?
Let me know if you need any more details 😊

Thanks for raising the issue! This is a routing issue of our static frontend files. We'll fix it ASAP.

Thanks @TPreece101 for reporting!

No problem, thanks for the speedy response and I look forward to taking the library for a spin!

Hi @TPreece101, thanks for reporting the issue! Can you share how you get streamlit working with jupyter-server-proxy? Is there anything special you need to do to get it running?

Hey @dy46 no problem. Nope, nothing special for Streamlit or code-server. Just start the process running with streamlit run app.py and then visit {base_url}/proxy/{port}/ <- Note the slash at the end but don't ask me why it's needed 😂 - it seems to be required for all port proxying using jupyter-server-proxy but it's not well documented.

Hi @TPreece101 , sorry for the delay! We looked into this issue and it's not an easy fix on our end. I wonder how urgent the issue is to you. As an alternative, you can  try running the tool locally or use our terraform script to deploy the tool on cloud.

No worries @wangxiaoyou1993, thanks for looking into it. It's not super urgent, it just means that I won't be able to try it out in my actual data pipelines. I have tried it out locally with demo data (I can't use prod data on my local machine) and unfortunately I don't have the right permissions to deploy the tool on our AWS account 😬.
I look forward to trying it out when you get around to it 😁

You can use Mage locally and connect to AWS resources. Do you have an AWS Access Key ID and an AWS Secret Access Key?
If you add that into your io_config.yaml, Mage locally can read from S3, Redshift, etc.
You can even run PySpark from Mage locally to your remote EMR cluster.

Thanks for the suggestion @tommydangerous, unfortunately it's more of a security issue having prod data running through my local machine rather than a technical problem.
My workspace is exposed to me through a platform and so the AWS infrastructure is abstracted away from me unfortunately.

Hi Folks, glad to see how far this project has come since I last saw it, you guys are killing it!
Have you had any thoughts about this? I've recently encountered the same problem with Dash apps in my environment. The way that they accommodate this kind of thing is that they have an environment variable DASH_REQUESTS_PATHNAME_PREFIX which allows you to insert a custom prefix, so in my example I set it to be f"/user/{os.getenv('USER')}/proxy/8050/" to work well behind jupyter-server-proxy.
I'm wondering whether it might be worth considering something like this for mage? It might have some good benefits when it comes to deploying in different environments 😊

Hello @TPreece101! Thank you for kind words! We'll look into this and get back to you soon!

Cool, let me know if you need any extra info @thomaschung408

Bump for this! However if whatever implementation could consider reverse proxies like Traefik or nginx in gerenal that would be awesome.

Hey @TPreece101 and others on this thread, we finally added something to resolve this issue. You can set the MAGE_REQUESTS_BASE_PATH environment variable (similar to the Dash environment variable in the earlier comment) to be proxy/6789 if your url is <base url>/proxy/6789, and you should be able to access Mage that way.


How to get Access Token

On Pressing F12, I am not getting the same screen as shown in the image.
I tried on Microsoft Edge and Chrome, but the view is similar. Please guide...

.

Found It


Images coming in bursts using reliable QoS

Bug report
Required Info:

Operating System:

Ubuntu Bionic

Installation type:

Eloquent Debs

Version or commit hash:
ros-eloquent-connext-cmake-module/bionic,bionic,now 0.8.2-1bionic.20191109.000408 amd64 [installed,automatic]
ros-eloquent-rmw-connext-cpp/bionic,bionic,now 0.8.1-1bionic.20191113.152945 amd64 [installed]
ros-eloquent-rmw-connext-shared-cpp/bionic,bionic,now 0.8.1-1bionic.20191113.152740 amd64 [installed,automatic]
ros-eloquent-rosidl-typesupport-connext-c/bionic,bionic,now 0.8.2-1bionic.20191113.152811 amd64 [installed,automatic]
ros-eloquent-rosidl-typesupport-connext-cpp/bionic,bionic,now 0.8.2-1bionic.20191113.152616 amd64 [installed,automatic]
rti-connext-dds-5.3.1/bionic,bionic,bionic,bionic,now 5.3.1-nc.x64Linux3gcc5.4.0+2 amd64 [installed,automatic]

DDS implementation:

RTI Connext

Client library (if applicable):

rclcpp

Steps to reproduce issue
Publish images using Connext over a topic using reliable for the reliability QoS setting
First terminal
export RMW_IMPLEMENTATION=rmw_connext_cpp
. /opt/ros/eloquent/setup.bash
ros2 run image_tools showimage

Second terminal
export RMW_IMPLEMENTATION=rmw_connext_cpp
. /opt/ros/eloquent/setup.bash
ros2 run image_tools cam2image

Repeat the same test using best effort reliability
first terminal
ros2 run image_tools showimage --ros-args -p reliability:=best_effort

second terminal
ros2 run image_tools cam2image --ros-args -p reliability:=best_effort

Expected behavior
Since this is localhost communication, all messages should be received quickly by the subscriber using either QoS setting
Actual behavior
Using reliable reliability the messages received at the subscriber are delayed by several seconds, and they seem to arrive in batches.

When using best effort the messages are received by the subscriber with very little delay, as expected.

Additional information
Possibly related to #302

Closing, since with Foxy now being End-of-Life, this repository is no longer used.


Can't find out Related Posts module

[x] This is a question about using the theme.
[ ] I believe this to be a bug with the theme --- not Jekyll, GitHub Pages or one of the bundled plugins.
[ ] This is a feature request.
[x] I have updated all gems with bundle update.
[x] I have tested locally with bundle exec jekyll build.

Environment informations

Minimal Mistakes version:3.4.4'
github-pages
Operating system:windows-7

Expected behavior
expecting related post module in
 https://mmistakes.github.io/minimal-mistakes/layout/uncategorized/layout-related-posts/

Steps to reproduce the behavior
just open this link
https://mmistakes.github.io/minimal-mistakes/layout/uncategorized/layout-related-posts/

in local i can see the related post but in
https://mmistakes.github.io/minimal-mistakes/layout/uncategorized/layout-related-posts/
i cant see any related post that's why i open that issue but now it seems like unnecessary issue.
sorry to bother you guys .


v2 rewrite of config-schema, todo: readme

made a full rewrite of ExtpipesConfig with dataclasses for all configuration

version updated to 2.0.0 as it is a breaking change to before
todo: README and example config

stopped trying to make the init.py > version automatically set by pyproject.toml as this failed in docker-build (which is not running a poetry install but a pip install requirements.txt-
configured the version now manually in these two places
upgrade py from 3.7 to 3.9
(aligned with cognitedata/inso-bootstrap-cli)


Vulnerability roundup 91: qemu-4.2.0: 1 advisory [7.9]

search, files

[ ] CVE-2020-15863 CVSSv3=7.9 (nixos-20.03)

Scanned versions: nixos-20.03: 977000f149b. May contain false positives.
Cc @edolstra

See also: #93985, #88387, #92872, #91309, #90958

20.03 is Frozen and EOL.
As such, issue is closed.


⚠ Bofrost.es has degraded performance

In da18f89, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1469 ms

Resolved: Bofrost.es performance has improved in bec0588 after 7 minutes.


Language names in the Choose resource languages dialog box

would it be possible to include the English or Roman script version of the language names in the Choose resource languages dialog box?


Explicit ArrowFormat parameter

Because of Arrow has two encoding types (Random Access aka Feather and IPC Stream) both of which can be accessed via File, URL and other ways, including raw existing Channel (functions were converted to public), and we should have methods for reading any of them.
Also this patch includes Boolean (BitVector) support and Arrow version upgrade

Hi! Thank you for the contributions :) I'm currently moving arrow support to separate module and re-working generic read a little. I expect no conflicts with your changes, so let's merge this a bit later.
I want to understand to understand choice of defaults for readArrow(url: URL, format: ArrowFormat = ArrowFormat.IPC), readArrow(path: String, format: ArrowFormat = ArrowFormat.IPC). Is it common to use IPC with URLs? What will happen if someone tries to readArrow("/path/to/local/file") with default format?

Hello @koperagen,
For URL I had used IPC as default because the original code used IPC for URL (url.openStream().use -> readArrow(stream: InputStream) ->  readArrow(channel: ReadableByteChannel -> ArrowStreamReader).
If someone tries to readArrow("/path/to/local/file") with default format, the result depends on type of that file :) Was it created with ArrowStreamWriter or ArrowFileWriter (both of them use Channel as sink).
Actually, IPC does not have metadata (that Feather has) and allow to read and write different dataframes one by one. IPC is better for real-time communication (if URL provides not static result but server-provided data stream) and  it's reading in future should support lambda subscriber pattern. Nevertheless, local file can also be encoded in IPC.
For example, in contribution to RCaller project I used IPC for supporting multiple values returning but the data was actually stored in the file (accessed by URI for flexibility).
In R arrow::read_arrow manual we can find that it is

a wrapper around ‘read_ipc_stream()’ and ‘read_feather()’, is deprecated. You should explicitly choose the function that will read the desired IPC format (stream or file) since a file or ‘InputStream’ may contain either

According to this recomendation, we can remove default parameters at all. I left them to make contribution more backward-compatible.

I didn't know that original URL version is for IPC streams :) I tried to read feather with it and got an OutOfMemoryError, but thanks to your comment i quickly understood why. To fix this problem i've included this commit in https://github.com/Kotlin/dataframe/pull/115. Hope it's okay.
About readArrow: it would be indeed better to follow this recomendation and have explicit readArrowIPC and readArrowFeather, so i removed readArrow

@koperagen, nice!


Multi image in Slide

I want show multi image like this( List img have 10 image and previous 3 image).
Image not selected have Opacity.
How I can do with this lib. thanks

⚠ You did not follow the contributing guidelines!
As stated in these:

You need to fill out the issue template. This step is mandatory! Not doing so will result in your issue getting closed. Don't take this personally if this happens, and feel free to open a new issue once you've gathered all the information required by the template.

Regarding your question, see #259 and particularly this comment.


chore(main): release 1.2.2

:robot: I have created a release beep boop
1.2.2 (2024-10-28)
Bug Fixes

deps: bump codecov/codecov-action from 3 to 4 in /.github/workflows (#6) (b5622c9)
deps: bump google-github-actions/release-please-action from 3 to 4 in /.github/workflows (#7) (bc687aa)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/girasolenergy/pyopenems/releases/tag/v1.2.2 :sunflower:


Python MATCH data preparation from Lens

Summary
This script reformats the lens data pulled in using their API to be fit MATCH's data standards. This script also pulls labels from our labeled csv data to label this Lens API dataset.
Related Issues

Resolves Issue #43

Backwards incompatibilities
None.
New Dependencies
Imports unique when compared to other files are:
re - (RegEx for removing/replacing punctuation)
nltk - (NLTK for intelligently tokenizing words and providing the corpus of stopwords)
ast - (AST for safely parsing stringified lists)
string - (For providing the full set of special characters)

Can you include a README that describes how to run your script? Also can you commit your output file to the repo so I can review it?

I see you sent the output in discord, checking it out now.


add createManyAndReturn Function

The createManyAndReturn function does exist in Prisma and does not in Prismock.
This is a link to the Prisma docs for this Function:
https://www.prisma.io/docs/orm/prisma-client/queries/crud#create-and-return-multiple-records

:tada: This issue has been resolved in version 1.32.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Relayer Does not bring up Wallet when Proving a Transaction

Describe the bug
When pressing "execute" to prove a transaction, nothing happens with my connected wallet (metamask).
To Reproduce
Steps to reproduce the behavior:

Search for the following transaction: 0x9fadd500066868c8290719083b0c2aad91ab4c41110bf822ac36d374bda256c9
https://optimistic.etherscan.io/tx/0x9fadd500066868c8290719083b0c2aad91ab4c41110bf822ac36d374bda256c9
Click on "Execute" on the UI

Expected behavior
A transaction will pop up, with a proof to start the 7 day withdrawal period.
Screenshots

I had the same issue when trying to "Execute" on a "Message is ready to be proved" state.
The error message in the console log was:
rkle-utils.ts:67 Uncaught (in promise) TypeError: provider.send is not a function
at makeStateTrieProof (merkle-utils.ts:67:32)
at CrossChainMessenger.getBedrockMessageProof (cross-chain-messenger.ts:1376:34)
at async Object.proveMessage (cross-chain-messenger.ts:1778:21)
at async CrossChainMessenger.proveMessage (cross-chain-messenger.ts:1474:16)
at async executeMessage (Relayer.tsx:91:9)

I had the same issue when trying to "Execute" on a "Message is ready to be proved" state.
The error message in the console log was: rkle-utils.ts:67 Uncaught (in promise) TypeError: provider.send is not a function at makeStateTrieProof (merkle-utils.ts:67:32) at CrossChainMessenger.getBedrockMessageProof (cross-chain-messenger.ts:1376:34) at async Object.proveMessage (cross-chain-messenger.ts:1778:21) at async CrossChainMessenger.proveMessage (cross-chain-messenger.ts:1474:16) at async executeMessage (Relayer.tsx:91:9)

Hey this was addressed in #17!
@cryptoharry are you still seeing this issue?

@zainbacchus I can confirm this is now resolved, and I have submitted the proof on-chain.
Thanks a lot for the help


Toggle enable/disable breakpoints

Currently the only commands that I know of to enable / disable breakpoints are:
{
        "key": "ctrl+x ctrl+i",
        "command": "workbench.debug.viewlet.action.disableAllBreakpoints"
    },
    {
        "key": "ctrl+x ctrl+l",
        "command": "workbench.debug.viewlet.action.enableAllBreakpoints"
    },

the above:

Requires remembering two keybindings.
Consumes two keybindings.

Note that Users (my case) may want to only consume one so that they can use the other keybinding (chord in my case) for something else.

Is there a commandID for toggling enable/disable breakpoints? If not, I'd like this to be a feature request for it.

No there is no command to toggle disablement.
However an extension can easily contribute such a command since we expose the Breakpoint through the VS Code api.


Dragging to connect to the end of a stack doesn't bring that stack to the front

Expected Behavior
The just-edited stack should be at the front.
Actual Behavior
The just-edited stack ends up behind other blocks.
Dragging to the top of a stack does bring the stack to the front.
Steps to Reproduce

Operating System and Browser
All

This may be an upstream issue.

@rachel-fenichel Any further thoughts on if this is upstream?

The root cause is:
 * <g> tags do not respect z-index so svg renders them in the
 * order that they are in the dom.  By placing this block first within the
 * block group's <g>, it will render on top of any other blocks.

(from Blockly.BlockSvg.prototype.bringToFront)
We're never explicitly calling bringToFront during a drag, but adding at the top level of the workspace effectively brings it to the front, while adding a block connected to another one doesn't bring it to the front.
The solution is to find the top block in the stack and bring it to the front.
Calling bringToFront on the dragging block would also work but would get expensive if the dragging block is being connected to the end of a long stack--we would be calling appendChild() for every single block above it i nthe stack.
The fix belongs in Blockly.draggedConnectionManager.prototype.applyConnections in main blockly.  In scratch-blocks it belongs in Blockly.InsertionMarkerManager.prototype.applyConnections.

Just so you know, bringing the just-edited stack to the front means bumped blocks can be occluded:

I was able to fix this by adding a call to rootBlock.bringToFront() in the function Blockly.RenderedConnection.prototype.bumpAwayFrom_, in core/rendered_connection.js.
However, I'm not familiar enough with the code know how smart or safe my solution is! So I'm not sure if I should submit a PR.

Behavior after my code change:

My fix also fixes rachel-fenichel's original case:

while you're working on this, pull in https://github.com/LLK/scratch-blocks/pull/1209 to fix a problem where bumping was sometimes disabled.
I tried your fix but got this failure:

The correct solution might be to send bumped stacks to the back but bring bumped reporters to the front.  You could do that by checking whether rootBlock has an outputConnection and only bumping if it does.
@jwzimmer would that be reasonable behaviour?


Parameter for commit author in git

Currently the author is hardcoded as "jenkins". Default value should be "entigo-k8s-gitops" and should be possible to set the e-mail and name with flags.
--git-authror-name=Martin<EMAIL_ADDRESS>

Implemented in 1fa4e8e @martivo

Implemented in 1fa4e8e @martivo

Works!

Works!


Heroku build fails

remote: Found 1 csproj files
remote: Building
remote: dotnet restore
remote: MSBUILD : error MSB1003: Specify a project or solution file. The current working directory does not contain a project or solution file.
what am i doing wrong?

Do you have a project.json or a .csproj in your project directory? Because this buildpack works only with .csproj. Yo can try another dotnetcore buildpack like "https://github.com/jaguado/aspnetcore-heroku-base.git" or try "dotnet migrate".

i found the problem. first, you have to specify the location of the .csproj file with PROJECT config option in heroku, secondly, there seems to be some fault with this particular buildscript, because the herokuinstance crashes immediately when trying to access the webpage. error 145 i believe it was.

@flabbert What's mean "PROJECT config option in heroku"?

$AlexByte It's a Variable you create on heroku config vars on settings


docs: add clarity to description

Changes to documentation from:
"
title: Bash ssh
Bash command: ssh
Used to connect to a remote computer, for example ssh 123.456.789.012 will try to establish a connection with that host. The address of the remote computer can be provided using an IP address or, if provided, an identifier.
If the remote computer requires user login, the form ssh username@remote_address can be used, which will then prompt for the user password on successful connection.
More Information:

Wikipedia

"
to instead read as above.

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Sorry, but this is unfinished work at best. The grammar is off, ssh is not a bash command and the username addition is spurious.
Also, in the future write concise commit messages that describe the change. When someone wants to look at the change, they can do it directly. The message is a summary.

Did not make sense, so abandoning it.


Fix broken links in README.md

Fixed some broken links in Additional props section

Thank you @salahzsh 👍


fix: update debug puppeteer example to 3.9.0

coverage: 99.245%. remained the same
when pulling 386d41ed4248c853eb46336d25b2f7a5736789a0 on rc-example-debug-puppeteer-3.9.0
into 9e78e15d1991068836c7c75d5448fce262cd03b6 on master.


Windows auto DNS resolution using NRPT rules

It suits better because you don't need to install additional software.

Wonderful PR, I love it


Add noisystudent train method

How about adding noisystudent training method?

Like #68, this would be nice to have ... more useful than advprop. Needs some interesting larger datasets to work with, had thoughts on OpenImages, have yet to find time to add classification support for OpenImages


Update DG

Add Sequence Diagram for add module
Update ModelClassDiagram and BetterModelClassDiagram
Fixed typo in StatusCommand.java
Fixed typo in AddModule in DG
Update Add/Delete Module header in UG to better reflect multiple modules processing capability

I'll leave the Add Modules Sequence Diagram in as part of the changes first. Will consult teaching team if similar diagrams should be left in/taken out.

https://github.com/nus-cs2103-AY2122S2/forum/issues/237

nus-cs2103-AY2122S2/forum#237 this might be the reason?

yup, in this instance its showing errors trying to resolve shadowJar

Codecov Report

Merging #107 (a7810e9) into master (2aec74e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #107   +/-   ##
=========================================
  Coverage     59.79%   59.79%
  Complexity      530      530
=========================================
  Files            92       92
  Lines          2057     2057
  Branches        245      245
=========================================
  Hits           1230     1230
  Misses          752      752
  Partials         75       75

Impacted Files
Coverage Δ

...va/seedu/address/logic/commands/StatusCommand.java
100.00% <ø> (ø)

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 2aec74e...a7810e9. Read the comment docs.


Can't find "corre_hico_1.npy"

I got this error.
No such file or directory: '/workspace/GGNet/Dataset/hico_det/annotations/corre_hico_1.npy'

upload just now.
Thanks