Destroy effect doesn't work on bot messages + No error messages

Hi i need a little help.
whatever i do i can't delete any messages that bot send, tested on (DiSky 4.0 beta 4 and DiSky release 4.0.1)
My code (no errors in console):

I don't even know if im doing something wrong, because even if i type "sgasgas" for example after "discord command"/"on discord command" event, skript doesn't show any errors.

This happens sometimes with a specific Server's or Skript's state (aka features on Skript or the server has changed from their original values, therefore it makes an unexpected behavior.) I sent a message over Skript's developer channel to know how could I fix it, and if it's even because of DiSky, but no answer was successful and we still don't know why this happens.

I finally found a fix for this.
The way to load the scope's sections (like trigger:) is different between the discord command scope & the defined bot scope.
Indeed, discord commands use the Vixio's loading way (Try/Catching retaining logs) while the new scope system uses the plain Skript system (as if we load an actual script).
I replaced the trigger loading code of the discord command with the new scope code.


Add a validation for conversion hosts

When a no conversion host is configured in the destination EMS, the migration tasks will stall because the throttler won't be able to assign a conversion host. To avoid hanging requests, we should check that at least one conversion host is available in the destination EMS. This PR implements a validation method in ServiceTemplateTransformationPlanRequest, that will be exposed in Automate.
Associated RHBZ: https://bugzilla.redhat.com/show_bug.cgi?id=1640816

@miq-bot add-label transformation, enhancement, hammer/yes
@miq-bot add-reviewer agrare

@miq-bot add_label blocker

@miq-bot add-reviewer jameswnl

@fdupont-redhat just remove the byebug then I'm good with this, @jameswnl what do you think?

Just realized I didn't hit the submit after adding these comments 😢

Hammer backport details:
$ git log -1
commit 898d25b81c271b2b35ac0969d577e93f06b33d3b
Author: Adam Grare<EMAIL_ADDRESS>Date:   Wed Oct 31 13:58:06 2018 -0400

    Merge pull request #18135 from fdupont-redhat/bz1640816

    Add a validation for conversion hosts

    (cherry picked from commit 8340f7c01af08acbcf76623ed562019b8bdb52d3)

    https://bugzilla.redhat.com/show_bug.cgi?id=1640816


docs: bump mkdocs docker image to 3.5.35

Summary
Changes

Please provide a summary of what's being changed

This PR bumps squidfunk/mkdocs-material image to v3.5.35 that fixes the issue in #3073

Please add the issue number below, if no issue is present the PR might get blocked and not be reviewed

Issue number: closes #3073

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Should have double checked.


[stable/unifi] hotfix: fixing ingress name + nodePorts references

Signed-off-by: Arno<EMAIL_ADDRESS>This should be reviewed fast, this is only a hotfix with a few changes.
Is this a new chart
Nope.
What this PR does / why we need it:
Some mistakes were still in, preventing the deployment in some cases
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

@billimek
@mcronce

/ok-to-test

/ok-to-test

/lgtm


Removed @hanngos from Codeowners

Description
Changes proposed in this pull request:

Removed @hanngos from Codeowners

Related issue(s)

/unhold


Sql

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the commit message is formatted like:
[GEARPUMP-<Jira issue #>] Meaningful description of pull request
[ ] Make sure tests pass via sbt clean test.
[ ] Make sure old documentation affected by the pull request has been updated and new documentation added for new functionality.

@buddhiayesha2015 could you squash your commits and get rid of the merge commit Merge remote-tracking branch 'origin/sql' into sql ?


⚠ Bofrost.lu has degraded performance

In 701205f, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1514 ms

Resolved: Bofrost.lu performance has improved in 87f7f69 after 6 minutes.


Ivy sponza

Hello!
Where from I can load ivy sponza scene?
Did you change somehow scene from Intel repo?

You can get the scene from here: https://www.intel.com/content/www/us/en/developer/topic-technology/graphics-research/samples.html
It's split into multiple parts and loaded as multiple parts, see https://github.com/SaschaWillems/VulkanPathTracer/blob/master/src/main.cpp#L641.
Note that the scenes are very taxing, esp. the base scene with the 4k x 4k textures. So if you're running into out of memory problems, you may need to scale down the textures before loading the scenes. You can easily do that with https://gltf-transform.donmccurdy.com/cli.html.

and rename new_sponza/NewSponza_Main_Blender_glTF.gltf to new_sponza/NewSponza_Main_Blender_glTF_sm.gltf
Also I find typo error src/main.cpp line 617
models[1] rewrite as models[0]

yes, either change the gltf file name or change the code to use a different file name.
I'll take a look at the typo, but note that this is a personal playground and not guaranteed to always be in a working state.

Thanks for fixing!


[Feature Request] React hooks support.

Libreact is the best repo to start supporting react hooks.

Already started: https://github.com/streamich/react-use


CONSTRAINT TRIGGER support

@a8m as discussed in Discord
Hello! When I define a postgres constraint trigger, atlas just creates a regular one
input
CREATE CONSTRAINT TRIGGER trg
    AFTER INSERT
    ON table
    INITIALLY DEFERRED
    FOR EACH ROW

migration
CREATE TRIGGER trg
    AFTER INSERT
    ON table
    FOR EACH ROW

Closed with https://github.com/ariga/atlas/releases/tag/v0.29.0.
Thanks for reporting this 🙏


[SP-6409] Backport of PDI-19803 - Loss of functionality present in th…

…e PDI 'SFTP Put' step due to outdated jsch-0.1.54.jar (9.3 Suite)
Update JSch for ssh rfc compliance

changed jsch to actively maintained version from mwiede
dialog: compression wasn't counting for changes
was accepting port 0
updated from deprecated FileUtil for vfs

(cherry picked from commit 6844a3c21dd1c524b61613600db5e68dd287eed3)
)
Original PR: #8933
@bcostahitachivantara @renato-s @andreramos89

:x: Build failed in 14m
Build command:
mvn clean verify -B -e -Daudit -amd

:exclamation: No tests found!
Errors:
Filtered log (click to expand)

script returned exit code 1

:information_source:  This is an automatic message

:white_check_mark: Build finished in 39m 39s
Build command:
mvn clean verify -B -e -Daudit -amd

:exclamation: No tests found!
:information_source:  This is an automatic message


update dashboard for initial LGSS digital metrics

Not for merging yet - just raising for awareness

This now looks somethiing like:

Can be reviewed for merging now, pie chart needs some work on colours but could be done another time


Not able to get co-ordinates on regular interval

Hi there,
The plugin works fine on my Android device, this is exactly I was looking for, The only need I have is regardless of the user's state, every 2 minutes I need the co-ordinates. I am using these cofigurations right now as suggested in doc and mentioned in some of the issues. But no luck. Please let me know if I am missing anything.
This is the code I have.
`function onDeviceReady () {
var callbackFn = function(location,$scope) {
alert(location.latitude);
alert(location.longitude);
$scope.lat.push(location.latitude);
$scope.lang.push(location.longitude);
};
var failureFn = function(error) {
alert('BackgroundGeoLocation error');
};
backgroundGeoLocation.configure(callbackFn, failureFn, {
desiredAccuracy: 10,
notificationIconColor: '#4CAF50',
notificationTitle: 'Traking Status',
notificationText: 'ON',
notificationIcon: 'notification_icon',
debug: true,
stopOnTerminate: false,
locationService: backgroundGeoLocation.service.ANDROID_FUSED_LOCATION,
interval: 1000,
activitiesInterval: 1000
});
$scope.start = function()
{
backgroundGeoLocation.start();
}
$scope.stop = function()
{
backgroundGeoLocation.stop();
}`
One more thing, I am not able to see the text,icon,status I set for the notification area.
Any help is appreciated.

dhaval

 interval: 120000, // <!-- poll for position every two minute
 fastestInterval: 120000

notification Icon property is applicable for only android >= api level 21, just incase you are testing it on jellybean/kitkat. try without setting icon

Use ANDROID_FUSED_LOCATION and set interval as @uzumakinaruto123 is suggesting.
BUT keep in mind, when device is idle, it turn's gps off to save power (so there's no guarantee you'll get updates every 2 mins). However you can suggest that position didn't changed anyway.

Thanks guys. Gonna make changes you both suggested and make some journeys today.
@uzumakinaruto123 I am testing on Android 5.1 and API level is 23. What could be the other issue you think ?
@mauron85 I used disableElasticity:true as a config parameter and its giving me location update almost every 2 mins. This is just a first test though, can say more after more testing.

There is no such option disableElasticity in this plugin. I know there is disableElasticity in former project cordova-background-geolocation-lt, which I have forked from.

Lol. I thought that's why m getting updates. Will remove it then. Thanks

To achieve regular updated for version 2.0.0 ise ANDROID_ACTIVITY_PROVIDER with option stopOnStillActivity false


io: add error check on pipe close functions to avoid error overwriting

The current implementation allows call multiple times Close and CloseWithError in every side of the pipe, as a result, the original error can be overwritten.
This CL fixes this behavior adding an error check on close functions, and returning an error in case the pipe read/write is already closed and keeping the first error still available.
atomicError has been removed and changed for a mutex because the check needs.
Fixes #24283

@jmartin82
https://github.com/golang/go/blob/master/src/io/pipe.go#L73
https://github.com/golang/go/blob/master/src/io/pipe.go#L112

Do you really need to keep p.erMu locked during p.once.Do(func() { close(p.done) })?

Do you need p.once.Do(...) at all?  As you seem to disallow repeated p.close().

@av86743 Thanks for your fast review.

Do you really need to keep p.erMu locked during p.once.Do(func() { close(p.done) })?
No, you are right I can unlock it before.

Do you need p.once.Do(...) at all?  As you seem to disallow repeated p.close().
It's a bit tricky because you can close the pipe from both sides (read/write) but you only have to close the channel once. if I remove the once I have to find an alternative (with a flag for example) to keep the same behavior or only allow to close one time the pipe, without matter if is the read or write side.

@av86743 Solved both things, thanks!


Updated README

Added the Figment pool into the "Staking Pools" section.
" * Figment A staking pool with 10% fee "

sorry for the delay, was waiting for another PR to be merged. Figment is now merged ⚡


refactor: Make '/v1/database/status' authenticated.

Description
Originally, /v1/database/status was open to the world to allow consumers—primarily ACS UI—to to zero in on the problem with the database before authentication because rendering login screen requires fetching info on auth providers from the database. However, ACS UI does not call this endpoint from the login screen and apparently does not plan to do so in the nearest future.
Given the primary use case is off the table, this PR tightens access for the endpoint to only authenticated users.
Checklist

[ ] Investigated and inspected CI test results
[ ] ~Unit test and regression tests added~
[ ] ~Evaluated and added CHANGELOG entry if required~
[ ] ~Determined and documented upgrade steps~
[ ] ~Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

Testing Performed
Manual test TBD.

I discovered a weird edge case, where if this status calls fails once you are logged in, we are still showing the "Backup your database before the upcoming Postgres migration..." banner, too.
Would you like me to push a change to this branch ?

Oh yes, if you prefer, please.

I discovered a weird edge case, where if this status calls fails once you are logged in, we are still showing the "Backup your database before the upcoming Postgres migration..." banner, too.
Would you like me to push a change to this branch ?

Ooops, turns out it was user-dismissible, and different ports matter (8000).
We probably should pull that warning out, but in a separate PR.
Approved.


Remove shebang, exec bits on run_*.py examples

Fixes: #413

Coverage increased (+0.07%) to 78.03% when pulling 264dfd4a0d6c15839027a5f08947f5efd9063db4 on alubbock:run_examples_fix_exec into 66e70a05409c19a1cc1962d90b5dbd64f3c1bc18 on pysb:master.


[Question] How to use harvester's CSI and CNI inside the VM's

All the documentation that i can find uses rancher. I am trying to setup a cluster in harvester's VMs
So, in a normal k8s created with kubeadm init inside harvester, how can i use the network plugins and storage class provided from harvester?

Hi @migs35323, thanks for filing the question.

For both harvester-csi-driver and cloud-provider-harvester, you can find generate_addon.sh script under deploy folder. Run it with Harvester kubeconfig to generate a new service account and kubeconfig.
Move generated kubeconfig to guest cluster's folder /etc/kubernetes/cloud-config.
You can install them by helm. Their charts are in harvester/charts.
For harvester-csi-driver, remember to change cloudConfig.hostPath to /etc/kubernetes/cloud-config.


when the filter is applied, and click on select all checkbox,  'multi' and 'all' events are fired

Grid is loaded with data
Apply a filter for a column, this makes grid to show only filtered records
check 'select all' check box
4.Both vxGridRowMultiSelectionChange and vxGridRowAllSelectionChange events are fired

only 'vxGridRowAllSelectionChange' is fired now.

fixed


liveclock

Hey @shivamadlakha , since you are neither responding nor acknowledging that you are doing work on this project, I am going to close this pull request, if you still working on this app just give another PR.
Thank You!.


Ecto benchmarks

The initial benchmark suite for Ecto. Tests rely on the Benchee  library.

Here are some of the results that I've gotten from running in my laptop.
Repo.insert!/1
Repo.all/2
Repo.to_sql/2
Repo.load/2

Any way we can help move this PR along? :grin:

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Create find_first_set_bit.cpp

my 1st pull request is accepted. but, the 2nd one is not accepted. HELP me, to find out the reason.


Incompatible with the Affinity mod

Any info on given Accessories and Compat Version combined with the version of affinity and the log/crash log of the issue.

Uploading latest.log…

Any info on given Accessories and Compat Version combined with the version of affinity and the log/crash log of the issue.

2024-08-08-1.log.gz
latest.log

The fix would for @gliscowo to update Endec version within the mod to deal with the incompatibility.

Closed as fixed


Additional Host CPU properties.

Add Host properties.
cpu_sockets  (int) - number of CPU sockets of this host.
VMWare: MOR:HostSystem.summary.hardware.numCpuPkgs
cpu_total_cores (int) - total number of CPU cores on this host.
VMWare: MOR:HostSystem.summary.hardware.numCpuCores

Done.


FancyZones shows the wrong layout after disconnecting and reconnecting docking station (via Thunderbolt 3)

Microsoft PowerToys version
0.35.0
Running as admin

[X] Yes

Area(s) with issue?
FancyZones, FancyZones Editor
Steps to reproduce
Disconnect laptop from Thunderbolt 3 docking station and then reconnect. The FancyZones layout reverts to a priority grid style layout, even though in the zone editor/selector, it shows my desired layout selected.
Desired layout:

Layout when I shift + drag window after reconnecting to dock:

Bug report log:
✔ Expected Behavior
My currently selected layout to be active when I shift + drag window
❌ Actual Behavior
A different (unselected) layout shows up when I shift + drag window
Other Software
No response

Hey @tpitts15 what version of Windows are you running?

I'm on build 19041.928 of Windows 10.

From: Deondre Davis @.>
Sent: Wednesday, April 21, 2021 2:55:47 PM
To: microsoft/PowerToys @.>
Cc: tpitts15 @.>; Mention @.>
Subject: Re: [microsoft/PowerToys] FancyZones shows the wrong layout after disconnecting and reconnecting docking station (via Thunderbolt 3) (#10792)
Hey @tpitts15https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftpitts15&data=04|01||87d0b7a55627401c308f08d904ff74ed|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637546317485349301|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=yr1wGE3rB3dLdF4isAbCXveVHJyBC3WhWCPsBgZQCgk%3D&reserved=0 what version of Windows are you running?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FPowerToys%2Fissues%2F10792%23issuecomment-824318181&data=04|01||87d0b7a55627401c308f08d904ff74ed|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637546317485359290|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=bffAqRSRluOePtjk6pw%2FNEzZe4Oa5hvd8KoP%2BgrHPWs%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACCWDS7HNCC2XMFZG2MCG43TJ4UUHANCNFSM43CKF3BQ&data=04|01||87d0b7a55627401c308f08d904ff74ed|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637546317485359290|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=7k627INTPXcPDJkt3aPOCM5jPHLW8VpbClFe9YA2FZk%3D&reserved=0.

Currently, the only workaround is to restart PowerToys every time I return my laptop to the dock, which is not ideal.


Unhandled Rejection: TypeError: Cannot read properties of undefined (reading 'thumbnails')

Describe the bug

When I try to get a video via ytdl() on serverless deployments.
Unhandled Rejection: TypeError: Cannot read properties of undefined (reading 'thumbnails')
    at Object.k [as cleanVideoDetails] (/var/task/.next/server/app/api/get_video/route.js:1:28364)
    at P (/var/task/.next/server/app/api/get_video/route.js:3:377)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Node.js process exited with exit status: 128. The logs above can help with debugging the issue.

Debug File

Environment

@ybd-project/ytdl-core version:^5.0.1-test.4"
Node.js version: 20
Operating system: Linux

Thank you for your report. We are currently identifying the cause of the bug and fixing it.

Version v5.0.3, which appears to fix this bug, is now available.
Please try it.

Ok, I tried it and another bug appears.
Unhandled Rejection: InvalidArgumentError: invalid url
    at /var/task/node_modules/undici/index.js:57:13
    at Object.f [as request] (/var/task/.next/server/app/api/get_video/route.js:6:4436)
    at /var/task/.next/server/app/api/get_video/route.js:6:217
    at i.getOrSet (/var/task/.next/server/app/api/get_video/route.js:1:7818)
    at O (/var/task/.next/server/app/api/get_video/route.js:6:172)
    at Object.D [as decipherFormats] (/var/task/.next/server/app/api/get_video/route.js:6:851)
    at L (/var/task/.next/server/app/api/get_video/route.js:3:848)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  code: 'UND_ERR_INVALID_ARG'
}
Node.js process exited with exit status: 128. The logs above can help with debugging the issue.

That has been fixed in v5.0.4. Please try a fresh install.

Oh, I will

Also, v5.0.4 contains useless logs and data. v5.0.5 should be installed.

Checked it.

Checked it.

Is it working?

No, another bug appears on vercel.
Unhandled Rejection: Error: This video is unavailable
    at y (/var/task/.next/server/app/api/get_video/route.js:1:20233)
    at /var/task/.next/server/app/api/get_video/route.js:1:22211
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Node.js process exited with exit status: 128. The logs above can help with debugging the issue.

What is the video URL?

And here are warning messages.

And here are warning messages.

It is normal. Currently, iOS and Android players will output that log because of the error.

Yes, I did.
const stream = ytdl(`https://youtube.com/watch?v=${url}`, {
          agent,
          filter: 'audioonly',
          quality: 'highestaudio',
          format: 'mp3',
          poToken,
          visitorData
        }).on('error', e => {
          console.log(e, 'error at get_video')
          throw e;
        });

Try generating a new one.
Also, what is specified in the agent?

The agent for cookies.
const cookies = [{
          "domain": ".youtube.com",
          "expirationDate":<PHONE_NUMBER>.220501,
          "hostOnly": false,
          "httpOnly": true,
          "name": "__Secure-1PSID",
          "path": "/",
          "sameSite": "unspecified",
          "secure": true,
          "session": false,
          "storeId": "0",
          "value": "g.a000mAgEOqoSd3RlbdBjNWAGnCWmcFtdOYwMaAYNJhfnnAyqaONtGaKgncRerL0dDxPwwp8sGQACgYKAZcSARUSFQHGX2MiaMV9LPBojznMJKAJiJcXyBoVAUF8yKqTxm8uenq1M5aIWXtBZrX00076",
          "id": 2
        },]
        const agent = ytdl.createAgent(cookies)

and I try generate new one and will reply.

Please do not specify cookies.

Ok.

I applied my new token on serverless environment variables, but same error appears on logs.
Unhandled Rejection: Error: This video is unavailable
    at y (/var/task/.next/server/app/api/get_video/route.js:1:19869)
    at /var/task/.next/server/app/api/get_video/route.js:1:21847
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Node.js process exited with exit status: 128. The logs above can help with debugging the issue.

Glitch and Vercel Functions do not generate errors, so it is possible that the poToken and visitorData are incorrect, either due to geographical factors or because the poToken and visitorData are incorrect.
Vercel: https://ytdl-core-test.vercel.app/api/v3/info/full/?videoUrl=https://youtu.be/oMfMUfgjiLg
Glitch: http://marsh-numerous-oak.glitch.me/stream/basic/?ytid=oMfMUfgjiLg

The issue has been resolved and the issue is closed.

My Biggest mistake.

And Thanks!


UnitTest for BSD and QL packets start and end time

in:
stixcore\products\tests\test_bsd.py > test_xray
stixcore\products\tests\test_quicklook.py > test_quicklook

[x] find good test cases (known timing) for the start and end time of QL and BSD data products.
[x] reactivate the time assertations

fixed in #405


Swap parameter order to match sklearn scorer

This PR solves issue #193, making our API more consistent with scikit-learn.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.24%. Comparing base (32132b1) to head (d7efe56).
Report is 207 commits behind head on development.

Additional details and impacted files
@@               Coverage Diff               @@
##           development     #215      +/-   ##
===============================================
- Coverage        97.30%   97.24%   -0.07%
===============================================
  Files               18       19       +1
  Lines             1669     1813     +144
===============================================
+ Hits              1624     1763     +139
- Misses              45       50       +5

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


Added support for handling integer overflow

Primitive parsing APIs will now return false if overflow occurs (per #443). Also added tests for these cases, which should neatly fulfill #704 for now.

Hi @jonfortescue, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Ah, you're right. I'll have to rethink this next week.

Looks good.
Thanks!


Added Additional Place Holders for New Docs

Added Additional Place Holders for New Docs
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[x] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Looks good to me.


asking help

I am sorry.
I have return to this site after long time absence.
I forgot my username and password for using GitHub.
How can I recover them?
My previous username and password do not work.
Thank you.

You can try to reset using your email: https://github.com/password_reset and if that doesn't work the folks at<EMAIL_ADDRESS>should be able to help you out. Good luck!

I got it. Thank you so much.

:sparkles:


Support alternative metrics on accelerated TGI / TEI instances

Description
Changes:

Provide accelDevice Helm variables also in other charts using accelerated TGI / TEI
Use different custom / HPA metrics depending on whether TGI / TEI is accelerated

Queue size metric is a bit better for scaling TGI Gaudi instances than the one used for Xeon scaling, and queries for it are simpler.  HPA rule updates act also as examples on use of different HPA algorithm types (Value vs. AverageValue).
Issues
n/a.
Type of change

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

Dependencies
n/a.
Tests
Did manual testing of HPA scaling of 1-4 TGI Gaudi instances (4 = maxReplicas value), using different TGI-provided metrics, and batch sizes.
PS. none of the TGI / TEI metrics seem that good for scaling, their values mostly indicate just whether given component is stressed, not how slow the resulting (e.g. ChatQnA) queries are.   Meaning that HPA scaling factor causes scale up when they get loaded, and it drops extra instances when load goes almost completely away, but that scaling can be quite far from a really optimal one.
Something like https://github.com/opea-project/GenAIExamples/issues/391 (and fine-tuning the HPA values for given model, data type, OPEA version etc) would be needed for more optimal scaling.

@eero-t could you double check the CI failure on xeon with cpu-values.yaml? You may try to reproduce and debug locally by helm install chatqna . -f cpu_values.yaml to install chatqna on xeon platform, and  helm test chatqna to figure out if something is failing

Failure in CI CPU values file test is for the TGI service response check: https://github.com/opea-project/GenAIInfra/actions/runs/10991943008/job/30556825567?pr=451
This PR changes only HPA files and CI skips HPA testing, so the reason for the failure is some change done before this PR.
What CPU values file does
CPU values file overrides deployment probe timings and resources from defaults: https://github.com/opea-project/GenAIInfra/blob/main/helm-charts/common/tgi/values.yaml#L58
So that ChatQnA service can be scaled better without failing, as scheduler knows how much resources they need, and kubelet uses higher QoS class for them: https://kubernetes.io/docs/tasks/configure-pod-container/quality-service-pod/
Increased probe timings cannot be the cause for failure, and memory limits should be large enough, not to be an issue either.  CPU limits are quite tight though (8 CPUs) to allow running multiple TGI instances on same node, especially with NRI policies.
Reason for fail
CI test checks TGI log output, but appears to be doing it too soon.
CPU limits can can explain slowdown from default (no resources specified), when compared to situation where only single TGI instance is running / node has plenty of resources free.
But CPU values file passed earlier, so something else has changed since it was merged, either in CI setup, or the TGI setup (other than values specified in CPU file).
Potential workarounds
Things that can be done (in some other PR):

CI:

increase time for checking TGI log, or
return CI setup to earlier state where TGI startup was faster

CPU values file:

increase (TGI) CPU resources

Another possibility is the latest image updated from docker hub, which might cause different behaviors.
The returned output seems correct, however, helm test for llm-uservice failed.
Need further investigation on this.

Reviewed tests that were run for all PRs that were merged since cpu-values.yaml was added in #386.
=> this and #451 are first PRs for which CI tested cpu-values.yaml, despite there being several merged PRs which could have broken things for it already earlier, as they updated CPU / Xeon related files in ChatQnA and its dependencies (e.g. #444).
Additionally, when looking at the passing Helm E2E tests in that merged #386 PR, e.g. for "xeon, values, common": https://github.com/opea-project/GenAIInfra/actions/runs/10800184120/job/29957611425?pr=386
There are lot of "Couldn't connect to server" errors (e.g. for data-prep), but the test passed despite failures in functionality not touched by that PR.
Then when looking at same passing test in later PRs, tests do not log  any more what they are doing: https://github.com/opea-project/GenAIInfra/actions/runs/10940026240/job/30371535827?pr=444
I.e. it seems that CI tests cannot be trusted to guarantee any of the functionality working, or catching when that functionality actually broke.
=> Somebody (not me) needs to bisect repo content in the CI environment to see when things actually broke.

Additionally, E2E tests pass the tests even if log file content was wrong, if the log file had wrong name, see:
https://github.com/opea-project/GenAIInfra/blob/main/.github/workflows/_helm-e2e.yaml#L142

Additionally, when looking at the passing Helm E2E tests in that merged #386 PR, e.g. for "xeon, values, common": https://github.com/opea-project/GenAIInfra/actions/runs/10800184120/job/29957611425?pr=386
There are lot of "Couldn't connect to server" errors (e.g. for data-prep), but the test passed despite failures in functionality not touched by that PR.

This is a workaround in the testpod to avoid some opea services' probe health check status doesn't got populated in the K8S service availability, sleep for a short period of time between 'helm install --wait' and 'helm test' could be another option.

Then when looking at same passing test in later PRs, tests do not log any more what they are doing: https://github.com/opea-project/GenAIInfra/actions/runs/10940026240/job/30371535827?pr=444

What does the test do is in each helm chart's template/tests/test-pod.yaml. The log only shows the test output.

I.e. it seems that CI tests cannot be trusted to guarantee any of the functionality working, or catching when that functionality actually broke.

ok, I root caused this issue. GenAIComps PR 608 plus cpu-values significantly increases the test run time, which trigger the default 5m timeout of helm test. Please wait for PR #456 to land-in first

ok, I root caused this issue. GenAIComps PR 608 plus cpu-values significantly increases the test run time, which trigger the default 5m timeout of helm test. Please wait for PR #456 to land-in first

How much cpu-values.yaml increased the test time with PR 608?   If it's really significant, it would be better to raise (also) the CPU limits in it.
(The values I've used may be too small.  At that time I had only couple of Xeon nodes for scaling testing, and smaller values allowed stuffing more instances to a single Xeon node.  However, I used also NRI to isolate pods better, which is not the default k8s setup...)

This is a workaround in the testpod to avoid some opea services' probe health check status doesn't got populated in the K8S service availability quickly enough.

Tests should be predictable and fail on errors, not paper over them.

Sleeping for a short period of time between 'helm install --wait' and 'helm test' could be another option.

Script should wait until pod is ready to accept input before trying to use it, or fail test if wait timeouts.
For example, this fails unless there are matching TGI pods, and all of them are in Ready state, within given timeout:
if ! kubectl wait --for=condition=Ready --timeout=60s --selector=app.kubernetes.io/name=tgi pod; then
  echo "ERROR: TGI pods not ready within 60s"
  exit 1
fi

What does the test do is in each helm chart's template/tests/test-pod.yaml. The log only shows the test output.

That's not really good enough.  One would need to dig out all the relevant test template files from the given PR branch, go through all relevant values files and deduct results for all variables & conditionals in the template.   That's way too much work and error prone.
Tests should log what they're actually testing, otherwise they're not worth much.  Without logging it's nightmare to verify them testing everything that's relevant (and preferably skip completely irrelevant things), and debug failures.

This is a workaround in the testpod to avoid some opea services' probe health check status doesn't got populated in the K8S service availability quickly enough.

Tests should be predictable and fail on errors, not paper over them.

Sleeping for a short period of time between 'helm install --wait' and 'helm test' could be another option.

Script should wait until pod is ready to accept input before trying to use it, or fail test if wait timeouts.
For example, this fails unless there are matching TGI pods, and all of them are in Ready state, within given timeout:
if ! kubectl wait --for=condition=Ready --timeout=60s --selector=app.kubernetes.io/name=tgi pod; then
  echo "ERROR: TGI pods not ready within 60s"
  exit 1
fi

What does the test do is in each helm chart's template/tests/test-pod.yaml. The log only shows the test output.

That's not really good enough. One would need to dig out all the relevant test template files from the given PR branch, go through all relevant values files and deduct results for all variables & conditionals in the template. That's way too much work and error prone.
Tests should log what they're actually testing, otherwise they're not worth much. Without logging it's nightmare to verify them correctly testing everything that's relevant (and preferably skip completely irrelevant things), and debug failures.

This is a workaround in the testpod to avoid some opea services' probe health check status doesn't got populated in the K8S service availability quickly enough.

Tests should be predictable and fail on errors, not paper over them.

Sleeping for a short period of time between 'helm install --wait' and 'helm test' could be another option.

Script should wait until pod is ready to accept input before trying to use it, or fail test if wait timeouts.
For example, this fails unless there are matching TGI pods, and all of them are in Ready state, within given timeout:
if ! kubectl wait --for=condition=Ready --timeout=60s --selector=app.kubernetes.io/name=tgi pod; then
  echo "ERROR: TGI pods not ready within 60s"
  exit 1
fi

What does the test do is in each helm chart's template/tests/test-pod.yaml. The log only shows the test output.

That's not really good enough. One would need to dig out all the relevant test template files from the given PR branch, go through all relevant values files and deduct results for all variables & conditionals in the template. That's way too much work and error prone.
Tests should log what they're actually testing, otherwise they're not worth much. Without logging it's nightmare to verify them correctly testing everything that's relevant (+ preferably skipping completely irrelevant things), and debug failures.

Script should wait until pod is ready to accept input before trying to use it, or fail test if wait timeouts.
For example, this fails unless there are matching TGI pods, and all of them are in Ready state, within given timeout:
if ! kubectl wait --for=condition=Ready --timeout=60s --selector=app.kubernetes.io/name=tgi pod; then
  echo "ERROR: TGI pods not ready within 60s"
  exit 1
fi

the current helm install --wait actually have already done exactly the same as the above, but for all pods I think,

the current helm install --wait actually have already done exactly the same as the above, but for all pods I think,

Oh, that waits also for Ready state. Yes, that would be the correct thing to do:
$ helm install -h
...
--wait if set, will wait until all Pods, PVCs, Services, and minimum number of Pods of a Deployment, StatefulSet, or ReplicaSet are in a ready state before marking the release as successful. It will wait for as long as --timeout


teleporter error about execution of oteleporter.sh

OrientDB Version: 2.2.30
Java Version: jdk 1.8
OS: centos7
When I use oteleporter.sh to import data from postgresql to orientdb, it has a error. The description is as follows, and it never occurred before. This operation is the same as the previous operation, but it occurs the error.
ERROR:
com.orientechnologies.teleporter.exception.OTeleporterRuntimeException - com.orientechnologies.orient.core.exception.OSerializationException: Error on unmarshalling JSON content '   301 Moved Permanently   Moved Permanently The document has moved here' : content must be between { }
at com.orientechnologies.teleporter.util.ODriverConfigurator.readJsonFromUrl(ODriverConfigurator.java:233)
at com.orientechnologies.teleporter.util.ODriverConfigurator.checkConfiguration(ODriverConfigurator.java:102)
at com.orientechnologies.teleporter.main.OTeleporter.execute(OTeleporter.java:248)
at com.orientechnologies.teleporter.main.OTeleporter.main(OTeleporter.java:205)

I look forward to your help.
Thanks very much.

Hi,
We fixed this, closing.


Combine API error

Stackdriver Error Reporting error detail
AttributeError: 'SedShadedArea' object has no attribute 'getLogY'
https://console.cloud.google.com/errors/CJGlwMT995K17wE?project=biosimulations&src=VESPA&time=P30D

Underlying issue in biosimulators/Biosimulators_utils#100

Can close once this package is bumped to biosimulators-utils 0.1.162

:tada: This issue has been resolved in version 9.10.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feat: support From<&[u8; N]> for Bytes

Hey @cjwcommuny! Out of curiosity, are you having an issue using Bytes::copy_from_slice directly or is this about the convenience of calling from/into?

@braddunbar  I would like to write a function like:
fn foo(x: impl Into<Bytes>) {}

foo(b"hello");
foo(Bytes::new());

I believe that implement <&[u8; N]> for Bytes would improve the ergonomics.


Localization is needed

Can you start using the Transifex?
And I sent you a file with translation into Russian )

I started a Transifex project for the app: https://www.transifex.com/projects/p/trulyyours/
You can either tell me your Transifex account id or send the file by email :)

You have already added me in Russia team ))

Thanks for the translation! :)

you're welcome ; )


[BUG]: Admin | Sync Template v3 (Connectors) not running

Describe the bug
I updated the Core components from version 1.78 to version 1.89.
The Admin | Sync Template v3 (Connectors) flow not running
Component (please tell us which flow or app you are experiencing issues with):

Solution Center of Excellence - Core Components
Version 1.89
Admin | Sync Template v3 (Connectors)

To Reproduce
Steps to reproduce the behavior:

Go to 'Admin | Sync Template v3 (Connectors)' details
Click on 'Run'
See error

Edit the flow
Check connection references for all actions

Save the flow
ReRun the flow
Always the same error

Additional context
No connection is displayed on the flow detail page.

Hello,
our latest version of the Core Components is 1.98 (https://github.com/microsoft/powerapps-tools/blob/master/Administration/CoEStarterKit/Individual Components/CenterofExcellenceCoreComponents_1_98_managed.zip) - did you upgrade to this one?
Thank you
Manuela

I had downloaded the starter kit on February 24.
And I had the following versions:

I just got the latest version and I'm going to test. But for your information, there are (1) and (2) on some of the .zip files:

Hello,
we have a cadence of monthly releases, typically we release a new version with bug fixes or new features once a month and we try to do it in the first week of the month.
Each solution should only be in the CoEStarterKit.zip file once, but because I exported them a few times they will have the (1) and (2) at the end - that should not impact the experience but thanks for pointing it out, I'll rename the files with the next update.
Let me know if you still experience issues with the latest version!
Thank you
Manuela

Same problem with the 1.98 version.
For additional information, this is the only flow for which I have this problem

Finally: same problem for the "Admin | Sync Template v3 (PVA)" flow

Hello, when you did the upgrade, did you do so following the instructions at installing updates?
If you do not do this, and have unmanaged layers, you will not see the changes in the flow.

closing out issues without action items on us

we deleted all the solutions and reinstall only the core component (v1.98) and the problem is still present for the 2 flows: no connection reference on the flow detail page and no hystory


Entities Virtual metadata update

We redesigned the Virtual metadata configuration to be more configurable.
It now works with customizable beans and we started with:

The VirtualBean Interface (https://github.com/DSpace/DSpace/pull/2270/files#diff-1b366df97eb771a4b229816bb1692ecdR1) containing the common getValue method
The Concatenate bean (https://github.com/DSpace/DSpace/pull/2270/files#diff-f41cc744f46ebcac6581dcb9ef69348eR1) which will concatenate the configured metadata fields with the configured separator where https://github.com/DSpace/DSpace/pull/2270/files#diff-548d15d27982aa24f407dcea17b6aea6R149 displays a sample use for concatenating the last name and first name, separated with a comma
The Related bean (https://github.com/DSpace/DSpace/pull/2270/files#diff-c74acefcfc4aa4680416db1c3f7d0b52R1) which will retrieve metadata fields from an indirectly related item. This is e.g. used to display the Journal's ISSN number in the Publication by traversing the Journal Issue, Journal Volume, Journal

https://github.com/DSpace/DSpace/pull/2270/files#diff-548d15d27982aa24f407dcea17b6aea6R242 for the Journal Issue
https://github.com/DSpace/DSpace/pull/2270/files#diff-548d15d27982aa24f407dcea17b6aea6R284 for the Journal Volume
https://github.com/DSpace/DSpace/pull/2270/files#diff-548d15d27982aa24f407dcea17b6aea6R232 for the Journal
As a result the Publication https://dspace7-entities.atmire.com/items/5ba16bc6-3efc-441f-8235-030952d78460/full contains the ISSN number 1945-7774, and also the journal.title and relation.isJournalOfPublication from the journal https://dspace7-entities.atmire.com/items/a23eae5a-7857-4ef9-8e52-989436ad2955/full

This codebase also contains the commits from
https://github.com/DSpace/DSpace/pull/2269. Please keep comments specific to the CSV import in that PR

We will verify why the travis failed on some Integration Tests and update the branch

It there a way to get multiple virtual metadata fields, if the related item has multiple values in this field? If I understand the code, all values will be concatenated inside a single field. Maybe it would be better to change the interface of VirtualBean, that it can return multiple values?

I have the same doubt as Alexander. And also ask if
org.dspace.content.virtual.Concatenate will be required for every field? To me, the name "concatenate" is used when we want to build a bigger string based on smaller ones. Just to understand, in the context of Virtual Metadata, you choose "concatenate" because you could have a multivalued field (example: dc.subject) and you need to return a single field with all values (with a separator), is it?

The commits applied so far are to ensure the branch is up-to-date with https://github.com/DSpace/DSpace/pull/2269 since it is dependent on that PR
The actual discussed improvements will be applied next

Raf has completed the requested changes:

Modify VirtualBean to support returning a List of Strings instead of a single String.

This is now visible at https://github.com/DSpace/DSpace/pull/2270/files#diff-1b366df97eb771a4b229816bb1692ecdR32 and supported in the classes using this method

Create an additional VirtualBean implementation for returning all values of the given metadata fields as separate elements in the list (instead of a single concatenated value)

This new class is visible at https://github.com/DSpace/DSpace/pull/2270/files#diff-2656552425540556ba71b04d30368a6cR47
A sample configuration of this been is visible at https://github.com/DSpace/DSpace/pull/2270/files#diff-548d15d27982aa24f407dcea17b6aea6R264

Use the Concatenate Bean only when the values have to be concatenated (e.g. concatenating last name and first name in a single author field)

A sample configuration of this been is visible at https://github.com/DSpace/DSpace/pull/2270/files#diff-548d15d27982aa24f407dcea17b6aea6R166

The documentation in the Spring config has been included as well.
The Travis is also succeeding with the latest Travis fix from @tdonohue

Thanks for the reviews, I'll merge the PR since it has 2 approved reviews
The message
https://github.com/DSpace/DSpace/pull/2270#discussion_r243534269 is part of a comment, and renaming the actual variable completely would be too independent of the entities to adjust here.


implement garbage collection

what if I tag some ref for eviction, but then I quit the program before the eviction occurs? The files will never never be removed from the file system?

Mmm good point, so you would first need to iterate through all the hamt entries, traverse and add all the DAG links to a CIDSet, then you would open the blockstore all keys channel and iterate over them, if a key is not in the CIDSet, GC the block.


NoneType exception on importing from admin site

I'm getting a NoneType exception when the __str__ method in my model references a self.* field. The issue crops up in version 0.4.5 only(0.4.4 works fine)
The offending line is https://github.com/django-import-export/django-import-export/blob/0.4.5/import_export/resources.py#L414 which calls __str__ on the model instance.
Perhaps self.* fields aren't set until the model is saved here(atleast for new rows)?

Hi @roshanr95, is it possible to add failing test for this?

Looks like #439 duplicate

Fixed in 0.6.0 with #505


Image not rendering after saving

Issue
So i've been creating a blog-type website and I want to be able to use images of course, however after I add an image link and save the content, the image is no longer rendered.
Preview
issue_preview_video.webm
Other
I tried modifying the ImageViewBlock component aswell, using next/image and default img tag however there's no change.

@lanoow When you add an image, it does not trigger the onChange event. To solve this, you need to create a new paragraph so that the img attributes are included in the onChange event. However, this approach isn't ideal. In this case, I would suggest adding something like editor.commands.createParagraphNear() after adding the image. It would look something like this:
// image-edit-block.tsx
const handleLink = () => {
  editor.chain().focus().setImage({ src: link }).run()
  editor.commands.createParagraphNear()
  close()
}

const handleFile = (e: React.ChangeEvent<HTMLInputElement>) => {
  const files = e.target.files
  if (!files) return
  const reader = new FileReader()
  reader.onload = e => {
    const src = e.target?.result as string
    editor.chain().focus().setImage({ src }).run()
    editor.commands.createParagraphNear()
  }
  reader.readAsDataURL(files[0])
  close()
}

@lanoow I still don't have the time yet to finish all the remaining features, including images. But the above answer should be enough to fix your issue.

@lanoow Could it be related to the throttleDelay prop? Setting it to 0 might give you immediate updates.

@lanoow When you add an image, it will not trigger the onChange if you don't have any paragraph set. However, in your video, you do have a paragraph. To ensure that the onChange event is always triggered when an image is added, you can do something like this:
// image-edit-block.tsx
const handleLink = () => {
  editor.chain().focus().setImage({ src: link }).run()
  editor.commands.createParagraphNear()
  close()
}

const handleFile = (e: React.ChangeEvent<HTMLInputElement>) => {
  const files = e.target.files
  if (!files) return
  const reader = new FileReader()
  reader.onload = e => {
    const src = e.target?.result as string
    editor.chain().focus().setImage({ src }).run()
    editor.commands.createParagraphNear()
  }
  reader.readAsDataURL(files[0])
  close()
}

So I did try doing that, however it's still not rendering.

@lanoow Could it be related to the throttleDelay prop? Setting it to 0 might give you immediate updates.

It's currently set at 1000, tried setting it at 0 but no chanage. It's still not rendering it. When inspecting the image tag is there and the src is there as well but it's not rendering it.

@lanoow Could you create a minimal reproduction using StackBlitz or a similar online code editor and share the link here? This would help me understand and troubleshoot the problem more effectively.

@lanoow Could you create a minimal reproduction using StackBlitz or a similar online code editor and share the link here? This would help me understand and troubleshoot the problem more effectively.

Yeah sure, give me a few minutes!

@Aslam97 here's a reproduction of the repo. link

@lanoow You almost got the whole code there 😅. Anyway, I couldn't try it because of dependency errors, but I have tried to reproduce it myself and it's all working fine. The only issue I found was that whenever the submit button was clicked, my own form got submitted too. This was fixed here: #21 and I added an example here that you can take a look at: #22
https://github.com/user-attachments/assets/582d0045-b922-4786-828f-d3634c9838e8

You should be able to run it if you do npm install --force, i'm using bun and idk what's the problem with npm.
The problem I'm having isn't related to the form but to the image not rendering for unknown reasons. I'm gonna try something but I'm not sure it'll work. I'll include a preview after I try it

@Aslam97 so what's interesting is, this is the url that's put in

@lanoow and whats the url that being save to the database?

@Aslam97 Here's what's in the DB column:
"
nrjgnjrngjrng

"

And that's the error I get if I try to use next/image
Error: Failed to parse src "\"\\"https://images.unsplash.com/photo-1725203574073-79922f64110a\\"\"" on `next/image`, if using relative image it must start with a leading slash "/" or be an absolute URL (http:// or https://)

I got rid of the \" and stuff, however the image isn't rendered still.

@Aslam97 Still the same yeah. Here's the request tab.

I think this is the problem, for some reason there's http://localhost:3000/ infront of the image but I have no idea where it comes from.

@Aslam97 no, no I don't

@lanoow tried with nextjs and works fine for me

@Aslam97 I guess it could be because of the next-intl package or authjs.

Here's my middleware.ts file
import createMiddleware from 'next-intl/middleware';
import { NextRequest } from 'next/server';
import authConfig from "../auth.config";
import {routing} from './i18n/routing';
import NextAuth from "next-auth";

const handleI18nRouting = createMiddleware(routing);

const { auth } = NextAuth(authConfig);

export default auth(async function middleware(req: NextRequest) {
	return handleI18nRouting(req);
});

export const config = {
	matcher: [
		"/((?!api|static|.*\\..*|_next|favicon.ico|logo.png|robots.txt).*)",
		"/(en|bg)/:path*",
		"/dashboard/:path*"
	]
}

@lanoow try this
export const config = {
  matcher: [
    // Skip Next.js internals and static files
    "/((?!_next|static).*)",

    // Include specific file types in search params
    "/(.*\\?.*(\\.html?|\\.css|\\.js|\\.json|\\.jpe?g|\\.webp|\\.png|\\.gif|\\.svg|\\.ttf|\\.woff2?|\\.ico|\\.csv|\\.docx?|\\.xlsx?|\\.zip|\\.webmanifest))",

    // Always run for API and tRPC routes
    "/(api|trpc)/.*",

    // mine
    "/(en|bg)/:path*",
    "/dashboard/:path*"
  ]
};

@lanoow you could try asking on Claude(dot)ai about regex.

@lanoow try this
export const config = {
  matcher: [
    // Skip Next.js internals and all static files, unless found in search params
    "/((?!_next|[^?]*\\.(?:html?|css|js(?!on)|jpe?g|webp|png|gif|svg|ttf|woff2?|ico|csv|docx?|xlsx?|zip|webmanifest)).*)",
    // Always run for API and tRPC routes
    "/(api|trpc)/.*",

    // mine
    "/(en|bg)/:path*",
    "/dashboard/:path*"
  ]
};

Tried it, still the same, no change.

@lanoow hei do you solve this? if yes I ll close this issue

@Aslam97 um, no I haven't figured it out. I'll close it if i find a solution.

No solution found. Issue seems to be with the project.


chore(deps): update module github.com/google/go-github/v67 to v68

This PR contains the following updates:

Package
Type
Update
Change

github.com/google/go-github/v67
require
major
v67.0.0 -> v68.0.0

Release Notes

google/go-github (github.com/google/go-github/v67)
v68.0.0
Compare Source
This release contains the following breaking API changes:

feat!: Remove Project (classic) support (#3380)
BREAKING CHANGE: All support for "Project (classic)" is removed and Project responses are replaced by ProjectV2 (renamed from ProjectsV2).
fix!: Remove unused Stats field from Commit struct (#3395)
BREAKING CHANGE: The unused Stats field is removed from the Commit struct.

...and the following additional changes:

Bump go-github from v66 to v67 in /scrape (#3361)
Map more missing installation permissions (#3363)
build(deps): bump github.com/alecthomas/kong from 1.4.0 to 1.5.0 in /tools (#3365)
fix: Preserve HTTP Response in URL Errors (#3369)
Update OpenAPI (#3374)
feat: Add assignment for org role associations (#3371)
build(deps): bump codecov/codecov-action from 5.0.7 to 5.1.1 (#3375)
build(deps): bump github.com/alecthomas/kong from 1.5.0 to 1.6.0 in /tools (#3376)
build(deps): bump golang.org/x/sync from 0.9.0 to 0.10.0 in /tools (#3377)
build(deps): bump golang.org/x/net from 0.31.0 to 0.32.0 in /scrape (#3378)
chore: Deprecate legacy ID-based team methods (#3373)
feat: Support resolution_comment to update alert API (#3357)
feat: Add custom_property events (#3379)
feat: Deprecate and replace Bool,Int,Int64,String with Ptr using generics (#3355)
feat: Add GET for custom org repo role (#3372)
chore: Remove commented out test code (#3381)
build(deps): bump golang.org/x/crypto from 0.27.0 to 0.31.0 in /example in the go_modules group (#3383)
feat: Add enterprise custom properties (#3382)
Change header keys to be canonical (#3389)
chore: Sort std imports (#3390)
feat: Document review request events in Timeline struct (#3391)
fix: DownloadReleaseAsset handles renamed repository (#3392)
Bump version of go-github to v68.0.0 (#3397)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

ℹ Artifact update notice
File name: go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

1 additional dependency was updated

Details:

Package
Change

golang.org/x/term
v0.27.0 -> v0.27.0


[Error Handling] Better messaging around cross-partition tests that are certain to fail

Need to handle cross-partition launches better when creds for the additional partitions aren't present; It'll fail, so don't bother launching.

Sample config to repro:
project:
  name: my-demo
  shorten_stack_name: true
  template: demo.template
  regions:
    - us-east-1
    - us-gov-east-1
    - us-gov-west-1
    - cn-north-1
    - cn-northwest-1
  tests:
   t1:
    parameters:
      EmailAddress<EMAIL_ADDRESS>
Below is the error message when running taskcat:

·[0;30;41m[ERROR  ] : ClientError An error occurred (InvalidToken) when calling the GetBucketLocation operation: The provided token is malformed or otherwise invalid.
--
786 | Traceback (most recent call last):
787 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/site-packages/taskcat/_cli.py", line 46, in main
788 | cli.run()
789 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/site-packages/taskcat/_cli_core.py", line 308, in run
790 | return getattr(command(), subcommand)(**args)
791 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/site-packages/taskcat/_cli_modules/test.py", line 178, in run
792 | test_definition.create_stacks()
793 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/site-packages/taskcat/_cfn/threaded.py", line 63, in create_stacks
794 | fan_out(self._create_stacks_for_test, {"tags": tags}, tests, threads)
795 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/site-packages/taskcat/_cfn/threaded.py", line 22, in fan_out
796 | results = pool.map(func, payload)
797 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/multiprocessing/pool.py", line 364, in map
798 | return self._map_async(func, iterable, mapstar, chunksize).get()
799 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/multiprocessing/pool.py", line 768, in get
800 | raise self._value
801 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/multiprocessing/pool.py", line 125, in worker
802 | result = (True, func(*args, **kwds))
803 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/multiprocessing/pool.py", line 48, in mapstar
804 | return list(map(*args))
805 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/site-packages/taskcat/_cfn/threaded.py", line 76, in _create_stacks_for_test
806 | stacks = fan_out(Stack.create, partial_kwargs, test.regions, threads)
807 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/site-packages/taskcat/_cfn/threaded.py", line 22, in fan_out
808 | results = pool.map(func, payload)
809 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/multiprocessing/pool.py", line 364, in map
810 | return self._map_async(func, iterable, mapstar, chunksize).get()
811 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/multiprocessing/pool.py", line 768, in get
812 | raise self._value
813 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/multiprocessing/pool.py", line 125, in worker
814 | result = (True, func(*args, **kwds))
815 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/multiprocessing/pool.py", line 48, in mapstar
816 | return list(map(*args))
817 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/site-packages/taskcat/_cfn/stack.py", line 272, in create
818 | url=s3_url_maker(
819 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/site-packages/taskcat/_common_utils.py", line 44, in s3_url_maker
820 | response = s3_client.get_bucket_location(Bucket=bucket)
821 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/site-packages/botocore/client.py", line 357, in _api_call
822 | return self._make_api_call(operation_name, kwargs)
823 | File "/root/.pyenv/versions/3.8.1/lib/python3.8/site-packages/botocore/client.py", line 676, in _make_api_call
824 | raise error_class(parsed_response, operation_name)
825 | botocore.exceptions.ClientError: An error occurred (InvalidToken) when calling the GetBucketLocation operation: The provided token is malformed or otherwise invalid.


remove previously deprecated nodejs:10 and go:1.11 kinds

To prepare for next openwhisk core release, remove deprecated runtimes from runtimes.json, tests, and documentation.

Codecov Report

Merging #5147 (dd4a159) into master (a99b0a8) will decrease coverage by 38.99%.
The diff coverage is n/a.

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


404 in contrib.layers documentation

In the official doc for version 1.0, this link to tf.contrib.layers.linear in here leads to a 404.

That tf.contrib.layers.linear link should probably just be removed.  There are no op links for the other activation function aliases (partial objects), tf.contrib.layers.relu or tf.contrib.layers.relu6, on the Python API Guides page.  I can remove the @{tf.contrib.layers.linear} line from the docs_src file and submit a pull request, but I think that I might first check that all other @{} links work (at least on the tf.contrib.layers page.)

Thanks for filing the issue @Mec-iS , and thanks for the analysis @ClarkZinzow !
I'm closing this out, since #8029 covers this as well.


Added 0-arity TedFunction implicit casting to Term and added more Ted…

…Function options (up to 5 input arguments)

Looks great.  Thanks!


EditorSimpleWorker.loadForeignModule

same problem at https://github.com/microsoft/monaco-editor/issues/1690#issue-532479802
my code:
 this.editor = monaco.editor.create(document.getElementById("monaco"), {                 value: "",                 language: "javascript",                 theme:"vs-dark"             });
the error meets :
Uncaught Error: Unexpected usage
Error: Unexpected usage
at EditorSimpleWorker.loadForeignModule (editorSimpleWorker.js?ccf6:540)
at eval (webWorker.js?af50:54)
at eval (errors.js?fdcc:12)
when i delete the props language is ok,why?

You are not defining the worker locations. Please read https://github.com/microsoft/monaco-editor/blob/master/docs/integrate-esm.md

You are not defining the worker locations. Please read https://github.com/microsoft/monaco-editor/blob/master/docs/integrate-esm.md

this is my code:
in page.vue:
`
import * as monaco from 'monaco-editor/esm/vs/editor/editor.api';
let monaco = require('monaco-editor');
this.editor = monaco.editor.create(this.$refs.monaco, {
value: "",
language: "json",
theme:"vs-dark"
});
`
in nuxt.config.js:
`
const MonacoWebpackPlugin = require('monaco-editor-webpack-plugin');
build: {
transpile: [/^element-ui/],
extractCSS: { allChunks: true },
plugins: [
new MonacoWebpackPlugin({
languages: ['typescript', 'javascript', 'css', 'json']
})
],
/*
** You can extend webpack config here
*/
extend (config, ctx) {
}
},
`
but when run in broswer,there meet the errror:
ReferenceError
navigator is not defined

i just first use in nuxt project，how to use in nuxt??it likely due to the usage mistake in nuxt pulgin?

You are not defining the worker locations. Please read https://github.com/microsoft/monaco-editor/blob/master/docs/integrate-esm.md

You are not defining the worker locations. Please read https://github.com/microsoft/monaco-editor/blob/master/docs/integrate-esm.md

this is my code:
in page.vue:
import * as monaco from 'monaco-editor/esm/vs/editor/editor.api';

 let monaco = require('monaco-editor');
                this.editor = monaco.editor.create(this.$refs.monaco, {
                    value: "",
                    language: "json",
                    theme:"vs-dark"
                });

in nuxt.config.js:

const MonacoWebpackPlugin = require('monaco-editor-webpack-plugin');

 build: {
        transpile: [/^element-ui/],
        extractCSS: { allChunks: true },
        plugins: [
            new MonacoWebpackPlugin({
                languages: ['typescript', 'javascript', 'css', 'json']
            })
        ],
        /*
        ** You can extend webpack config here
        */
        extend (config, ctx) {
        }
    },

but when run in broswer,there meet the errror:
ReferenceError
navigator is not defined

i just first use in nuxt project，how to use in nuxt??it likely due to the usage mistake in nuxt pulgin?

now,i use the nuxt pulgin config solve the naviagtor error,but the initial problem is still exsist,at console:
languageFeatures.js?ff36:85 Error: Unexpected usage     at EditorSimpleWorker.loadForeignModule (editorSimpleWorker.js?ccf6:540)     at eval (webWorker.js?af50:54)
my code is:
in page:
import monaco from "~/plugins/monaco" this.editor = monaco.editor.create(this.$refs.monaco, {                     value: "",                     language: "json",                     theme:"vs-dark"                 });
in nuxt.config.js:
const MonacoWebpackPlugin = require('monaco-editor-webpack-plugin');  build: {         vendor: [             // 'monaco'         ],         transpile: [/^element-ui/],         extractCSS: { allChunks: true },         plugins: [             new MonacoWebpackPlugin({                 languages: ['json']             })         ],         /*         ** You can extend webpack config here         */         extend (config, ctx) {         }     },

You are not defining the worker locations. Please read https://github.com/microsoft/monaco-editor/blob/master/docs/integrate-esm.md

hello,now i solve the pro of languageFeatures.js?ff36:85 Error: Unexpected usage at EditorSimpleWorker.loadForeignModule (editorSimpleWorker.js?ccf6:540) at eval (webWorker.js?af50:54) by require('monaco-editor/esm/vs/editor/editor.api') but not
require('monaco-editor'),but then i see another error that： in node_modules,it miss the languages package of json:
i use:
monaco-editor/esm/vs/basic-languages/json/monaco.contribution,
it not found。 why it dosen't support json?

You are not defining the worker locations. Please read https://github.com/microsoft/monaco-editor/blob/master/docs/integrate-esm.md

hello,now i solve the pro of languageFeatures.js?ff36:85 Error: Unexpected usage at EditorSimpleWorker.loadForeignModule (editorSimpleWorker.js?ccf6:540) at eval (webWorker.js?af50:54) by require('monaco-editor/esm/vs/editor/editor.api') but not
require('monaco-editor'),but then i see another error that： in node_modules,it miss the languages package of json:
i use:
monaco-editor/esm/vs/basic-languages/json/monaco.contribution,
it not found。 why it dosen't support json?

I want use the "format code" function when right mouse click. As you said, it will solve the error, but the function bar  is disappear. What should I do?

@alexdima Hello, I read the document and the error above still appear. I think the Monaco Editor Loader Plugin will help me to deal with the webworker or something. Please help me.


Failed to complete the word when type a few letters of the word.

Environment data

Repro Steps

Actual behavior

When type "ctxt", failed to find "Context"

Is this a regression or a feature request?

Closing old issue. If this is still a problem, please reopen with the information requested. thanks


Convert LibreOffice formats to Microsoft formats

Hi,
I currently use a libreoffice installed on PHP container to convert my ODT and ODS files to PDF, DOCX and XSLX.
I would love to switch to Gotenberg as the exceptional job you did would solve my problem of converting multiple files at the same time.
I did several tests and it seems that it is possible to convert only to PDF.
Do you plan to add the ability to convert to Microsoft format?
This is how I use my local libreoffice to convert:
loffice --norestore --invisible --headless --nologo 'macro:///Standard.Convert.ConvertTo("tmp/export.odt","/tmp/export.docx","MS Word 2007 XML","")'
cFilter:
csv => "Text - txt - csv (StarCalc)"
xls => "MS Excel 97"
xlsx => "Calc MS Excel 2007 XML"
doc => "MS Word 97"
docx => "MS Word 2007 XML"

In my Dockerfile :
COPY Convert.xba /usr/lib/libreoffice/presets/basic/Standard/
RUN sed -i 's/<\/library:library>/ <library:element library:name="Convert"\/>\r\n<\/library:library>/g' /usr/lib/libreoffice/presets/basic/Standard/script.xlb

Convert.xba
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE script:module PUBLIC "-//OpenOffice.org//DTD OfficeDocument 1.0//EN" "module.dtd">
<script:module xmlns:script="http://openoffice.org/2000/script" script:name="Convert" script:language="StarBasic">REM  *****  BASIC  *****

Sub Main

End Sub

Sub ConvertTo( cFile, cOutFile, cFilter, cLoadFilter )
        On Error Goto ErrorConv

        cURL = ConvertToURL( cFile )
        ' oDoc = StarDesktop.loadComponentFromURL( cURL, "_blank", 0, Array( MakePropertyValue( "FilterName", cFilter ) ))
        if (cLoadFilter <> "") then
        	oDoc = StarDesktop.loadComponentFromURL( cURL, "_blank", 0, Array(  MakePropertyValue("Hidde", False), MakePropertyValue( "FilterName", cLoadFilter ) ))
        else
        	oDoc = StarDesktop.loadComponentFromURL( cURL, "_blank", 0, Array( MakePropertyValue("Hidde", False) ))
        endif
        cURL = ConvertToURL( cOutFile )
        oDoc.storeToURL( cURL, Array( MakePropertyValue( "Overwrite", True), MakePropertyValue( "FilterName", cFilter ) ))
        oDoc.close( True )

        Exit Sub

        ErrorConv:
        	StarDesktop.terminate()
End Sub

Function MakePropertyValue(Optional cName as String, Optional uValue) As com.sun.star.beans.PropertyValue
        Dim oPropertyValue as New com.sun.star.beans.PropertyValue
        If Not IsMissing( cName ) Then
                oPropertyValue.Name = cName
        EndIf
        If Not IsMissing( uValue ) Then
                oPropertyValue.Value = uValue
        EndIf
        MakePropertyValue()= oPropertyValue
End Function

</script:module>

EDIT: Just with unoconv :
unoconv -f docx export.odt

Hello @Kuppit 👋
Currently, Gotenberg focuses on the PDF format as output, nothing else.
That being said, you could adapt the code here https://github.com/gotenberg/gotenberg/blob/main/pkg/modules/libreoffice/uno/uno.go#L295 to fit your need 😄


FlexSlider - GridFieldBulkUpload

if class_exists, add bulk tools to gridfield

Current coverage is 93.75%

Merging #55 into master will decrease coverage by -3.61% as of 2d70b94

@@            master     #55   diff @@
======================================
  Files            2       2
  Stmts           76      80     +4
  Branches         0       0
  Methods         10      10
======================================
+ Hit             74      75     +1
  Partial          0       0
- Missed           2       5     +3

Review entire Coverage Diff as of 2d70b94

Powered by Codecov. Updated on successful CI builds.


Tooltip Hover in a multi-series chart when not shared is broken

Codepen
https://codesandbox.io/s/apexchart-hover-series-issue-prjcd
Explanation
I have a simple time series line graph and the hover of the series is not precise, see this video

The tooltip behavior, when not shared in a multi-series chart, is buggy.
I have tried several ways, but couldn't find a solution to this issue yet.
The code for detecting data-point in a multi-series chart when tooltip.shared: false is located at https://github.com/apexcharts/apexcharts.js/blob/master/src/modules/tooltip/Utils.js#L121
If anyone with a knowledge of detecting close items based on cursor position can help, that would be great!

I have the same problem with the unprecise hover while using a multi-series-chart.
Maybe an idea is to check, if the cursor is over the right color.... so the tooltip could depends on the color of the chart.... so you could avoid, that the tooltip from a wrong chart is shown.

The code for detecting data-point in a multi-series chart when tooltip.shared: false is located at https://github.com/apexcharts/apexcharts.js/blob/master/src/modules/tooltip/Utils.js#L121

By the way: Thanks for the link but acutally we need the code when tooltip.shared is true (like in the video), dont we....

Any update on this bug @junedchhipa ? I have a line series and a scatter series and I can show the tooltip on the points of my scatter, because it always show the tooltip on the points of the line. tooltip.shared=true is not working

any update on this issue ?

The tooltip behavior, when not shared in a Multi-series chart, is still buggy. Does it work for you, guys? thanks

This is still not working


Small typo 2.6.4 instead of 2.6.3

https://github.com/zendframework/zend-mvc/releases/tag/release-2.6.3
The title is 2.6.4 so i was searching on packagist but it was not published, since it's the 2.6.3 release :smile:

Fixed; thanks!


Restricted to Android 8

Official Google docs say this should be supported for Android 5+, why is the site restricted to 8.1?
https://support.google.com/googlepay/answer/10890261?hl=en#zippy=%2Ccant-add-your-covid-card

Also this is not working for Android 8.1 either. The welcome text would ask me to try the Gpay button, but the actual button would be locked out saying I need 8.1+.

fixed... thx for the PR!


Rollup code splitting demo

Blocked by: rollup/rollup#2391

@filipesilva we don't depend on Angular from npm - it's a bazel-to-bazel dependency so we include Angular's WORKSPACE in ours as an http_archive.
Since it's green on CI I'll merge this, we should still figure out why you're getting an error...


Preserve  element position

Right now, the layout dialect always puts the <title> element as the first element in a <head> block because (if I remember right) it makes the code a lot easier to work with separating this element that has a lot of special rules (because of all the title merging jazz that happens) from everything else.  This does mean that its reordering a dev's markup, which is not ideal.
<head> element merging shouldn't do this, instead letting the merge strategy that has been configured decide the position of the element instead.
Some things to consider:

Often there's a <title> in both the layout and view.  With the default merging strategy, if the view has a title then it'll be placed after everything else.  Should that then mean the merged title be placed there?
Although I don't know of anything that relies on the title being first, this could be a breaking change!  If so, then maybe I should introduce a new merging strategy that retains this behaviour (a "title first appending" strategy?) and make that the default on release, but note that on the next major version that behaviour will change to a plain old appending merge strategy.

I modified the element sorting and introduced new sorting strategies that would become the default in 3.x for #176.  Delete the old strategies and rename the new ones to the old names come 3.x.


Missing ARM_SUBSCRIPTION_ID variable

[Enter feedback here]
prepare_region.sh expects the ARM_SUBSCRIPTION_ID variable also. Please put the following into the documentation before calling the shell script:
export ARM_SUBSCRIPTION_ID=
thanks a lot

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

ID: c00bf88e-38bf-eaa8-923a-aaa94c635447
Version Independent ID: b27a7fbd-decb-8306-ac46-ea18a6d17e96
Content: About Control Plane deployment for the SAP Deployment automation framework - Azure Virtual Machines
Content Source: articles/virtual-machines/workloads/sap/automation-deploy-control-plane.md
Service: virtual-machines-sap
GitHub Login: @KimForss
Microsoft Alias: kimforss

Hello @yertaylan67
I have assigned the issue to the content author for further review.


Add signing to the CI

Make the deployment in CI optional so that we can install from the bundle.
/cc @ybettan @enriquebelarte

/lgtm


Speech recognition continuous

Does it support speech recognition continuous?

No, after a time it close the stream.


Change max-len severity from error to warning

Pull request opened by github-pullrequestcreator.

max-len is a style rule and should not block a merge.
We have to disable this rule very often, since our localization keys and URL strings are often longer than 120 characters.
I propose we keep the rule as a warning only.

tsk tsk ... totally slackers 😜

:tada: This PR is included in version 4.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Static Members listed as "Members"

This is a docca problem

For example array::max_size is listed as:

Moved to https://github.com/boostorg/docca/issues/47


So much request hate :(

We don't use the "forever" agent by default (any more, maybe we did at one time before I was involved with request).  Also document the fact that agent: false is really the key to not using the node core pooling behavior.

:+1:


Fault tolerant storage for Jupterhub

Jupyter pods are storing data in the pod volume. So if the pod dies you would lose any notebook/file edits.
We should be using a fault tolerant volume so that if the pod dies we don't lose our data.
/cc @foxish

As we discussed - this should just need additional config if we want PV per pod. If we want something that also allows sharing like NFS, we may need to add the necessary config to run that.
cc/ @yuvipanda for thoughts on sharing between notebooks.

I'd be happy with whatever's easiest. Do you have a pointer to the config that need's to change?
I'd like to start using Jupterlab on Kubeflow to write examples for Kubeflow. But I don't want to do that until we have fault tolerant storage.

The quickest thing may be using a PV per Jupyter pod. It needs adding a spawner option.
c.KubeSpawner.user_storage_pvc_ensure = True

It may need a storage class also to be set through c.KubeSpawner.user_storage_class if it doesn't auto-provision a PV automagically already in the environment it's being run in.

Note that for this and other JupyterHub config things, we have a helm chart in https://github.com/jupyterhub/zero-to-jupyterhub-k8s/tree/master/jupyterhub that's very actively maintained and heavily used. I'd love to re-use that here than build from scratch.

See docs about chart at z2jh.jupyter.org

To be clear, I'm happy for this to be using KubeSpawner + Vanillay JupyterHub too! But we've put in a bunch of effort into the helm chart to provide easy OOTB solutions to things like:

Updating config of hub without disrupting users
Automatic HTTPS
Multiple authenticators
Load tests (http://github.com/yuvipanda/jupyterhub-loadtest) to validate performance at scale, etc

That we want to make sure you can re-use all that work, and we can re-use any improvements you make to the hub deployment without having to re-invent it. I understand that not everyone wants to use helm, but I do want to try find a path to not having y'all duplicate all our work there...

As examples of other projects that include the JupyterHub helm chart as a dependency and build on it, see http://github.com/jupyterhub/binderhub. As examples of direct deployments that use JupyterHub and other charts, see http://github.com/jupyterhub/mybinder.org-deploy/ or http://github.com/berkeley-dsep-infra/datahub/ or http://github.com/berkeley-dsep-infra/data8xhub or http://github.com/yuvipanda/paws :)

Good point, and I totally agree. But I do think we need to come up with a
cohesive solution here suitable for hub, and the other components in this
repo. Having different deployment mechanisms for different parts of this
effort will lead to more confusion. So, as soon as we get to that unified
deployment solution, we can try match upstream as much as possible.
On Dec 7, 2017 11:04 PM, "Yuvi Panda"<EMAIL_ADDRESS>wrote:

To be clear, I'm happy for this to be using KubeSpawner + Vanillay
JupyterHub too! But we've put in a bunch of effort into the helm chart to
provide easy OOTB solutions to things like:

Updating config of hub without disrupting users
Automatic HTTPS
Multiple authenticators
Load tests (http://github.com/yuvipanda/jupyterhub-loadtest
https://github.com/yuvipanda/jupyterhub-loadtest) to validate
performance at scale, etc

That we want to make sure you can re-use all that work, and we can re-use
any improvements you make to the hub deployment without having to re-invent
it. I understand that not everyone wants to use helm, but I do want to try
find a path to not having y'all duplicate all our work there...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/kubeflow/issues/19#issuecomment-350173280, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA3U5xGWAYVDtpnwBkrxBE4XJCpJ23Qqks5s-MNxgaJpZM4Q6Jlz
.

I agree too! The way I'd do it would be to make a Helm Chart for the TF controller and one for the Model Server, and KubeFlow then just configures them together (similar to the http://github.com/jupyterhub/mybinder.org-deploy/ pattern). IMO that's much more user friendly than having users edit kubernets objects directly and then apply them... I agree that using Helm for JupyterHub but something else for the other tools isn't a valid long term strategy.
I'm happy to take an initial shot at doing this if you'd like, which can easily be discarded too without hurting any of my feelings :)

In the meantime I've also created #22 which provides persistent storage for each user with the current setup :)

@jlewi and I also had discussed ksonnet as one potential mechanism here and
I'm not sure where the others stand.
Another thought is that we could be more opinionated here in our config and
possibly have less knobs since we are targeting a very specific ML use-case
as opposed to the upstream project and chart which are aiming at a broader
use-case.
Happy to discuss this more. Perhaps we should have a "how do we manage
config" issue, enumerate the options, pros and cons and go from there.
On Dec 7, 2017 11:12 PM, "Yuvi Panda"<EMAIL_ADDRESS>wrote:

I agree too! The way I'd do it would be to make a Helm Chart for the TF
controller and one for the Model Server, and KubeFlow then just configures
them together (similar to the http://github.com/jupyterhub/
mybinder.org-deploy/ https://github.com/jupyterhub/mybinder.org-deploy/
pattern). IMO that's much more user friendly than having users edit
kubernets objects directly and then apply them... I agree that using Helm
for JupyterHub but something else for the other tools isn't a valid long
term strategy.
I'm happy to take an initial shot at doing this if you'd like, which can
easily be discarded too without hurting any of my feelings :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/kubeflow/issues/19#issuecomment-350174276, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA3U517hkwOEK4zpIAAUvsMST-XwxEBSks5s-MVZgaJpZM4Q6Jlz
.

That sounds like a good idea! ksonnet looks cool too! I am not attached to helm particularly, only against having end users directly edit kubernetes object specifications. 99% of the work that needs to happen to fully support this is in kubespawner anyway, so it's not a very big deal!
+1 on opening another issue. This discussion is already pretty off-topic for this issue :)

@foxish So I modified the JupterHub config jlewi/kubeflow#2 the PVC and PV are created. But it doesn't look tike the pod running Jupyter for me is attaching it as a volume
    volumeMounts:
    - mountPath: /var/run/secrets/kubernetes.io/serviceaccount
      name: no-api-access-please
      readOnly: true


feat(sbb-toggle-check): add new size variant (M)

Closes #1846

Codecov Report

Merging #1849 (fd6d4f2) into master (ab917b0) will decrease coverage by 17.42%.
The diff coverage is 34.45%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##           master    #1849       +/-   ##
===========================================
- Coverage   54.85%   37.44%   -17.42%
===========================================
  Files          49      257      +208
  Lines        1659     9834     +8175
  Branches      406     2119     +1713
===========================================
+ Hits          910     3682     +2772
- Misses        671     5885     +5214
- Partials       78      267      +189

Impacted Files
Coverage Δ

...ts/sbb-accordion-item/sbb-accordion-item.events.ts
100.00% <ø> (ø)

.../sbb-accordion-item/sbb-accordion-item.stories.tsx
0.00% <0.00%> (ø)

...mponents/sbb-accordion-item/sbb-accordion-item.tsx
0.00% <0.00%> (-37.21%)
:arrow_down:

...components/sbb-accordion/sbb-accordion.stories.tsx
0.00% <0.00%> (ø)

...ents/sbb-action-group/sbb-action-group.stories.tsx
0.00% <0.00%> (ø)

...onents/sbb-alert-group/sbb-alert-group.stories.tsx
0.00% <0.00%> (ø)

src/components/sbb-alert/sbb-alert.stories.tsx
0.00% <0.00%> (ø)

...onents/sbb-autocomplete/sbb-autocomplete.events.ts
100.00% <ø> (ø)

...ents/sbb-autocomplete/sbb-autocomplete.stories.tsx
0.00% <0.00%> (ø)

src/components/sbb-button/sbb-button.stories.tsx
0.00% <0.00%> (ø)

... and 58 more

... and 201 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add logic to include animated show images

Leaving this here for reference until properly documented:
ffmpeg -i 4.gif -c vp9 -b:v 0 -crf 41 4.webm
The CRF value could be lowered to save even more space if needed, but as of now this should produce something pretty spot-on with the original GIF at a size smaller than the static image

ffmpeg reference when converting from GIF to MP4. This will need to be referenced when rendering from Premier
ffmpeg -i file.gif -movflags +faststart -pix_fmt yuv420p -vf "scale=trunc(iw/2)*2:trunc(ih/2)*2" file.mp4


crashing on 1.16, had to roll back to 1.14

[x] I have read the FAQ.
[x] I have searched in existing issues.

Environment

OS: Windows 10
scrcpy version: 1.16
installation method: Windows release
device model: Nexus 5
Android version: 6.0.1

Describe the bug
I used 1.14 for a long time without any issue. I noticed there had been several updates so I got the latest version (1.16). After I updated I started getting lots of crashes. I went back to 1.14 and the crashes stopped. Let me know if there is anything specific I can do to get you more information on how to diagnose this.

I need more details.
What do you mean by "crash"? What is the output in the console?

I was using the no-console version. I have switched back to 1.16 and am using the console version now. I will report back once it starts crashing again. Hopefully the console window stays open when it crashes.

whatis' error?
INFO: scrcpy 1.16 https://github.com/Genymobile/scrcpy
/usr/share/scrcpy/scrcpy-server: 1 file pushed, 0 skipped. 61.3 MB/s (33622 bytes in 0.001s)
[server] INFO: Device: Hisilicon L86FA-BO5011A-B (Android 8.0.0)
[server] ERROR: Exception on thread Thread[main,5,main]
android.media.MediaCodec$CodecException: start failed
at android.media.MediaCodec.native_start(Native Method)
at android.media.MediaCodec.start(MediaCodec.java:2043)
at com.genymobile.scrcpy.ScreenEncoder.internalStreamScreen(ScreenEncoder.java:87)
at com.genymobile.scrcpy.ScreenEncoder.streamScreen(ScreenEncoder.java:55)
at com.genymobile.scrcpy.Server.scrcpy(Server.java:76)
at com.genymobile.scrcpy.Server.main(Server.java:225)
at com.android.internal.os.RuntimeInit.nativeFinishInit(Native Method)
at com.android.internal.os.RuntimeInit.main(RuntimeInit.java:284)
ERROR: Could not retrieve device information

@garfieldbanks Did you reproduce with the console open?
@x93008 This is a separate problem, I guess it's not even related to the version (v1.14 or v1.16), just reboot your device.

Still haven't re-encountered the problem. I'll let you know as soon as it happens again.

OK. It crashed but the console didn't show anything except:
WARN: Device Disconnected
Is there anything else I can do to provide more info?

Crashed again. Same warning:
WARN: Device Disconnected

Third crash, same warning.

OK, now it crashes as soon as I run it.

Crashing on 1.14 now as well. But still no error message in the console...
Anything else I can turn on to get more info on what is happening when it crashes?

Oh ok, it's not a crash then, it's just that the device gets disconnected from adb (as if you unplugged it).
If you run adb logcat in parallel, you will notice that it also terminates at the same time.
Try with another cable or USB port (see #281 #283).


Update README wrt instructions

Pull Request Test Coverage Report for Build 55

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

Totals

Change from base Build 54:
-0.3%

Covered Lines:
659

Relevant Lines:
668

💛  - Coveralls

Pull Request Test Coverage Report for Build 55

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

Totals

Change from base Build 54:
-0.3%

Covered Lines:
659

Relevant Lines:
668

💛  - Coveralls


Allow every possible number as a defaultValue for a number option

Search Terms
plugin number declaration option
Problem
Since we allow every possible value for a map option it makes sense to do the same for numbers.
Example usage:
Given a plugin with a number option (minValue = 0 and maxValue = 10). If no value is provided the plugin should completely ignore the option.
Currently defaultValue would default to 0 and therefore there is no way for the plugin to check if the option was provided by the user or not.
Suggested Solution
If we allow every possible number the plugin could define the defaultValue as -1 for example and then proceed accordingly.
Another solution would be not to set the defaultValue to 0 if it is not defined when calling addDeclaration.
What do you think?

This sounds reasonable to me, originally I was going to say that we should instead provide an isSet function, but that complicates the class considerably. This seems like a good compromise.

v0.17.7 released with this :)


Put back default constructor to fix issue with migration on start :

liquibase: Can not use class org.grails.plugins.databasemigration.liquibase.GormDatabase as a Liquibase service because it does not have a no-argument constructor

Problem is still in 3.0.3 with Grails 3.3.5. Please merge.

After a year its still not merged?

Now in 2018 I'm still running into this issue...  Can someone please merge this simple change?

Now we just need a new release! :-)


Add crash report url

Add url to accept crash report from https://github.com/atom/electron/blob/master/docs/api/crash-reporter.md and post it as an issue on the GitHub repo.

This would be awesome.

This would be great


Please provide the cli command to show the NSGs created by az vmss create command in the doc.

What command will list the Network Security Groups created as mentioned by this statement ?
"Creates the virtual machine scale set and connects it to the virtual network, subnet, and network security group. A load balancer is also created to distribute traffic to multiple VM instances. This command also specifies the VM image to be used and administrative credentials."
Assuming this command was run:
Create a scale set
Network resources such as an Azure load balancer are automatically created
az vmss create
--resource-group myResourceGroup
--name myScaleSet
--image UbuntuLTS
--upgrade-policy-mode automatic
--admin-username azureuser
--generate-ssh-keys
thank you,
Maybe we should add :

Create a network security group in the same region as the scale set eg: NSG Name: myexistingnsgeast

Then create the scale set w/ the --nsg parameter eg:

Create a scale set
Network resources such as an Azure load balancer are automatically created
az vmss create
--resource-group myResourceGroup
--name myScaleSetnsg
--image UbuntuLTS
--upgrade-policy-mode automatic
--admin-username azureuser
--generate-ssh-keys
--nsg myexistingnsgeast
[Enter feedback here]

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

ID: f16717fd-4e3a-0fbd-ae02-7911787e5890
Version Independent ID: 90aa598c-a86b-f79c-d005-502b8d2b3763
Content: Azure CLI Samples - Create a virtual machine scale set
Content Source: articles/virtual-machine-scale-sets/scripts/cli-sample-create-simple-scale-set.md
Service: virtual-machine-scale-sets
Sub-service: cli
GitHub Login: @mimckitt
Microsoft Alias: mimckitt

@MikeWilcoxMicrosoft
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@mimckitt
Can you please check and add your comments on this doc update request as applicable.

I updated the sample. Changes should go live sometime this afternoon.


ConfigureNeuropixelsV1e node error messages improvements

Empty filename path error should be “Gain calibration file must be specified.” or “Adc calibration file must be specified.” to match NPV2e.

Wrong calibration file popup does not report probe serial number. Should say “Probe serial number xxx does not match xx calibration file serial number xxx.” to match NPV2e and help the user identify the mistake.

Cryptic error message when no DeviceName specified in NeuropixelsV1eData node. Suggestion: “DeviceName must be specified”.

This has been addressed for 2.0 code, but we still need to fix for 1.0. See this for an example:
https://github.com/open-ephys/onix-bonsai-onix1/blob/7ac700fb8a696d4d9f8d97c9d1b85765b9951ca6/OpenEphys.Onix1/NeuropixelsV2.cs#L120


Add DisplayP3 colorspaces

This PR adds support for the DisplayP3 colorspace, a wider gamut range for computer displays that is standard in the Apple ecosystem and elsewhere. More information can be found in this developer video.
DisplayP3 uses the DCI-P3 gamut with a D65 white point and an sRGB transfer curve. In keeping with discussions with the OCIO group, we've opted to name it in the form <transfer_curve>_<gamut> for consistency.
Hence we add the following two colorspaces:

srgb_displayp3
lin_displayp3

The matrix3x3 values are derived from Nuke

Thanks, @jstone-lucasfilm
HP is working on a matching PR to the OCIO ACES folk and should have that up soon.


iphlpapi.dll

how to inject iphlpapi.dll?

Since that DLL is not dependent only on ntdll.dll, you can achieve that by setting ForceUserApc to FALSE. Note that in this case, it will be loaded as a last DLL, just before WinMain (as written in readme).


Code of Conduct update to replace mediator email with the CNCF committee email

The CNCF recently formed a Code of Conduct Committee to replace the mediator that was listed as the contact point in the CNCF code of conduct.
Since the CNCF no longer works with that mediator, can you update your code of conduct with the new contact details: CNCF Code of Conduct Committee -<EMAIL_ADDRESS>It looks like you need to update it in 2 files:

https://github.com/tikv/tikv/blob/master/CODE_OF_CONDUCT.md
https://github.com/tikv/async-speed-limit/blob/master/CODE_OF_CONDUCT.md

Here is an example from the updated template:
https://github.com/cncf/project-template/blob/main/CODE_OF_CONDUCT.md

Done, thanks!


Allow storing appearances of implicit geometries

In CityGML 3.0, implicit geometries can have one or more appearances. This is not yet reflected in the current 3DCityDB v5 schema.
An implicit geometry property of a feature is stored as row in property. In this row, we could of course also set the val_appearance_id. However, this would only allow to store a single appearance but not multiple ones. The implicit geometry object itself does not go into feature but is stored in implicit_geometry. The implicit_geometry table is not linked to appearance.
Possible solutions:

Each appearance property of the implicit geometry is stored as additional row in property and this row must reference the row storing the implicit geometry property as parent.
We add a implicit_geometry_id column to appearance to link both tables.

Maybe there are more possible solutions.

When thinking more about it, I guess solution 1) is not feasible. It would tightly link the appearances to the feature containing the implicit geometry at import time. But an implicit geometry is independent of a specific feature. So, the appearances must be linked to the implicit geometry instead as done in solution 2).

I think we can store multiple appearances for a single implicit geometry, since all related properties stored as different rows in property table. Similar to gml:name (or class) storage, all we need to increase index_number regarding to appearances for every implicit geometry instances.

Each row in property is related to a feature but the general implicit geometry and its appearances are not. Only the instantiation of an implicit geometry is feature-specific. So I think using the property is not feasible.

I checked again, you have right. Features can share same implicit_geometry instances, that's why we can not use features as a bridge/connector from appearances to implicit_geometries. We need to create a relation from appearance to implicit_geometries, which is shown in UML diagrams. CoreDiagram, AppearanceDiagram

done with done with 8c0111d