Flutter module build failed

Hello,
I'm trying to use ssh2 in flutter module and i get some errors when I build it.
I create module flutter create -t module --org com.example flutter_module and add line ssh2: ^2.2.3 in pubspec.yaml and get the following error on build flutter build aar :
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':ssh2:compileDebugJavaWithJavac'.
> Could not resolve all files for configuration ':ssh2:debugCompileClasspath'.
   > Failed to transform bcprov-jdk15on-1.69.jar (org.bouncycastle:bcprov-jdk15on:1.69) to match attributes {artifactType=android-classes-jar, org.gradle.category=library, org.gradle.libraryelements=jar, org.gradle.status=release, org.gradle.usage=java-api}.
      > Execution failed for JetifyTransform: /Users/guillaume/.gradle/caches/modules-2/files-2.1/org.bouncycastle/bcprov-jdk15on/1.69/91e1628251cf3ca90093ce9d0fe67e5b7dab3850/bcprov-jdk15on-1.69.jar.
         > Failed to transform '/Users/guillaume/.gradle/caches/modules-2/files-2.1/org.bouncycastle/bcprov-jdk15on/1.69/91e1628251cf3ca90093ce9d0fe67e5b7dab3850/bcprov-jdk15on-1.69.jar' using Jetifier. Reason: IllegalArgumentException, message: Unsupported class file major version 59. (Run with --stacktrace for more details.)
           Suggestions:
            - Check out existing issues at https://issuetracker.google.com/issues?q=componentid:460323&s=modified_time:desc, it's possible that this issue has already been filed there.
            - If this issue has not been filed, please report it at https://issuetracker.google.com/issues/new?component=460323 (run with --stacktrace and provide a stack trace if possible).
   > Failed to transform jsch-0.1.66.jar (com.github.mwiede:jsch:0.1.66) to match attributes {artifactType=android-classes-jar, org.gradle.category=library, org.gradle.libraryelements=jar, org.gradle.status=release, org.gradle.usage=java-api}.
      > Execution failed for JetifyTransform: /Users/guillaume/.gradle/caches/modules-2/files-2.1/com.github.mwiede/jsch/0.1.66/dace8dd51ea58220b9b5e2a6dd2c453a2925ae93/jsch-0.1.66.jar.
         > Failed to transform '/Users/guillaume/.gradle/caches/modules-2/files-2.1/com.github.mwiede/jsch/0.1.66/dace8dd51ea58220b9b5e2a6dd2c453a2925ae93/jsch-0.1.66.jar' using Jetifier. Reason: IllegalArgumentException, message: Unsupported class file major version 60. (Run with --stacktrace for more details.)
           Suggestions:
            - Check out existing issues at https://issuetracker.google.com/issues?q=componentid:460323&s=modified_time:desc, it's possible that this issue has already been filed there.
            - If this issue has not been filed, please report it at https://issuetracker.google.com/issues/new?component=460323 (run with --stacktrace and provide a stack trace if possible).

I tried to :

update gradle version to 7.0.3 in .android/gradle/wrapper/gradle-wrapper.properties
update Android gradle plugin version to 7.0.2 in .android/build.gradle

but i get another error :
FAILURE: Build failed with an exception.

* Where:
Initialization script '/Users/guillaume/bin/flutter/packages/flutter_tools/gradle/aar_init_script.gradle' line: 9

* What went wrong:
Could not compile initialization script '/Users/guillaume/bin/flutter/packages/flutter_tools/gradle/aar_init_script.gradle'.
> startup failed:
  initialization script '/Users/guillaume/bin/flutter/packages/flutter_tools/gradle/aar_init_script.gradle': 9: unable to resolve class org.gradle.api.artifacts.maven.MavenDeployer
   @ line 9, column 1.
     import org.gradle.api.artifacts.maven.MavenDeployer
     ^

  1 error

Has anyone encountered (and perhaps solved) this problem?
Thanks,

Hello,
I'm afraid I haven't seen this error before. Are you able to build the example app? Maybe you can compare your app the example one in that case.
Best,
Josh

Yes, I me able to build example app.
I also able to build a "fresh" flutter app (flutter create flutter_app) by addingssh2: ^2.2.3 dependency and update gradle and Android gradle plugin versions.
But I'm not able to build flutter module.
Thank you for your reply.

You may want to try updating the Gradle version using Android Studio. I last used version 2020.3 to do this, and it was a simple process.


Add option for PFDs to not follow non-interactive defaults

There are rollup constructions where it is not necessary to to know the commitment to the block data ahead of time. For example a rollup that

uses deferred execution
executes immediately but simply waits to include the header in the next celestia block
only includes block data in celestia and distributes the header via some other means
or
as stated in the specs, one that interacts with the block producer

This would allow for a more efficient use a blockspace, and often a cheaper fee provided submitter's of PFDs end up paying for some portion of padding #431

closing as this issue does not align with the implementation of non-interactive defaults at this time. There doesn't need to be a specific option, as wrapped txs and message inclusion handle this for us.

it also misses the point of non-interactive defaults that I somehow manage to keep forgetting. They have nothing to do with rollup execution, just message inclusion


Sync committee issues

Description
I am seeing some sync committee issues with Teku, with missed duties. Logs aren't too instructive, but there is a "failed to process event queue"
Frequency:
Semi-regularly during this committee, see validator for history
https://beaconcha.in/validator/444933
Logs
eth-main-consensus-1  | 2023-04-30 22:52:40.841 ERROR - Failed to process event queue for client 937039586
eth-main-consensus-1  | java.util.concurrent.CompletionException: java.lang.IllegalStateException: s=OPEN,api=BLOCKED,sc=false,e=null
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture.encodeThrowable(Unknown Source) ~[?:?]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture.completeThrowable(Unknown Source) ~[?:?]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture$UniApply.tryFire(Unknown Source) ~[?:?]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture.postComplete(Unknown Source) ~[?:?]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture.completeExceptionally(Unknown Source) ~[?:?]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.SafeFuture.lambda$propagateResult$3(SafeFuture.java:146) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture.uniWhenComplete(Unknown Source) ~[?:?]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture.uniWhenCompleteStage(Unknown Source) ~[?:?]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture.whenComplete(Unknown Source) ~[?:?]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.SafeFuture.whenComplete(SafeFuture.java:626) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.SafeFuture.whenComplete(SafeFuture.java:33) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.SafeFuture.propagateResult(SafeFuture.java:143) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.SafeFuture.lambda$exceptionallyCompose$34(SafeFuture.java:426) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture.uniWhenComplete(Unknown Source) ~[?:?]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture$UniWhenComplete.tryFire(Unknown Source) ~[?:?]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture.postComplete(Unknown Source) ~[?:?]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture.completeExceptionally(Unknown Source) ~[?:?]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.SafeFuture.lambda$propagateResult$3(SafeFuture.java:146) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture.uniWhenComplete(Unknown Source) ~[?:?]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture.uniWhenCompleteStage(Unknown Source) ~[?:?]
eth-main-consensus-1  | at java.util.concurrent.CompletableFuture.whenComplete(Unknown Source) ~[?:?]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.SafeFuture.whenComplete(SafeFuture.java:626) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.SafeFuture.whenComplete(SafeFuture.java:33) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.SafeFuture.propagateResult(SafeFuture.java:143) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.SafeFuture.propagateTo(SafeFuture.java:320) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.ScheduledExecutorAsyncRunner.lambda$createRunnableForAction$1(ScheduledExecutorAsyncRunner.java:124) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source) [?:?]
eth-main-consensus-1  | at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source) [?:?]
eth-main-consensus-1  | at java.lang.Thread.run(Unknown Source) [?:?]
eth-main-consensus-1  | Caused by: java.lang.IllegalStateException: s=OPEN,api=BLOCKED,sc=false,e=null
eth-main-consensus-1  | at org.eclipse.jetty.server.HttpOutput.write(HttpOutput.java:813) ~[jetty-server-11.0.14.jar:11.0.14]
eth-main-consensus-1  | at org.eclipse.jetty.server.HttpOutput.print(HttpOutput.java:1126) ~[jetty-server-11.0.14.jar:11.0.14]
eth-main-consensus-1  | at org.eclipse.jetty.server.HttpOutput.print(HttpOutput.java:1053) ~[jetty-server-11.0.14.jar:11.0.14]
eth-main-consensus-1  | at io.javalin.http.sse.Emitter.write(Emitter.kt:44) ~[javalin-5.4.2.jar:5.4.2]
eth-main-consensus-1  | at io.javalin.http.sse.Emitter.emit(Emitter.kt:20) ~[javalin-5.4.2.jar:5.4.2]
eth-main-consensus-1  | at io.javalin.http.sse.SseClient.sendEvent(SseClient.kt:65) ~[javalin-5.4.2.jar:5.4.2]
eth-main-consensus-1  | at io.javalin.http.sse.SseClient.sendEvent$default(SseClient.kt:62) ~[javalin-5.4.2.jar:5.4.2]
eth-main-consensus-1  | at io.javalin.http.sse.SseClient.sendEvent(SseClient.kt) ~[javalin-5.4.2.jar:5.4.2]
eth-main-consensus-1  | at tech.pegasys.teku.beaconrestapi.handlers.v1.events.EventSubscriber.lambda$processEventQueue$1(EventSubscriber.java:122) ~[teku-data-beaconrestapi-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.SafeFuture.fromRunnable(SafeFuture.java:155) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.AsyncRunner.lambda$runAsync$0(AsyncRunner.java:23) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | at tech.pegasys.teku.infrastructure.async.SafeFuture.of(SafeFuture.java:74) ~[teku-infrastructure-async-23.4.0.jar:23.4.0]
eth-main-consensus-1  | ... 4 more

Versions (Add all that apply)
v23.4.0
Stepped back down to v23.3.1, but committee for this one is about over, so not sure I get a clear read on whether 23.4 was the issue.

Likely not a Teku version issue, or maybe even not related to Teku. I saw some 502s earlier from Vouch re sync committee response from Teku, that's why the Teku focus. Still missing things on 23.3.1

I'm closing this. The 502 happened and the above stuff happened, but I don't think it was the root cause of my issue.


client EmptyRequestGenerator and EntityRequestGenerator return types

Client's EmptyRequestGenerator and EntityRequestGenerator does have apply methods, that returns F[Request[F]].
However, both methods uses F.pure and there is absolutely no reason to wrap it in such way.
Shouldn't it return just Request[F] instead?

That's a fair question.  Entity encoding used to be effectful, so lifting into F made them consistent with the ones that take a body.  There is probably much less argument for that now, other than inertia.  People using it to pass directly to the client would be unaffected, but it might turn some flatMaps into maps.
I can't think of another reason not to change this.

We should do this, last I heard the only reason not to presently was Rho.

I can't think of a reason that would be bad for rho.  Putting apply directly on the status would be.  This is in client, so I don't think rho is affected at all.  @zarthross, am I missing anything?

Since EmptyRequestGenerator and EntityRequestGenerator are both in the client it will have no impact on Rho since its not even a dependency.
As @rossabaker said, having a apply on status could potentially be problematic.

This would also simplify the Client API (#2483) and could be made simultaneously to achieve a better and more consistent API.

Okay, let's move forward with it for 1.0.  I think it might be tedious to fix the tests, but good practice.  Volunteers?


[dev] child process now close properly.

closes #653
- Summary
Basically when I'd run parcel src/index.html via netlify dev -c, pressing CTRL + C (SIGTERM) would not shut down the process.
- Test plan
Run netlify dev and cancel it over and over again, even if it doesn't start completely. If it doesnt break, it works.
- Description for the changelog
Child processes now exit when SIGTERM command (CTRL + C) is issued.
- A picture of a cute animal (not mandatory but encouraged)
Have some hot diggity dogs.

Thanks for the PR @waynevanson. This was a broader issue and has been addressed in the latest release (https://github.com/netlify/cli/commit/64d6e67a4eb0c39f3dc49e0061a2f3cedb012f32). Please update and provide feedback if it is still a problem.


When option is 0, bulk update fails

When bulk updating the status column in my table which is radio button with 0 and 1, if I choose the option 0, the bulk update will fail.
This is because of what happens at the file path below around Line 36-42
vendor/joy/voyager-bulk-update/src/Http/Traits/BulkUpdateAction.php
At that code section, any 0 value is treated as a false result causing it to fail updating the table

Yes, same if tried to update a dropdown, if you change to some value can't put again value 0

Some update?

no update, was just raising the issue to be known

Thank you for pointing it.
You can raise a PR.


USPTO

This PR adds the code to process the USPTO dataset extracted from Google Patents Public Dataset and uploaded to HF. The dataset covers all US patent applications until Oct 27, 2023 (including historical).
closes #9
Edit: switched over to pandoc as that does the latex formatting pretty well, and also works well with the overall html.

Thanks for the review @blester125! I think i have addressed all your points but let me know if I missed anything! Added the readme as well. The only thing left are the tables. They are mostly fine but the whitespace formatting is quite off. Will bring it up in the meeting!

Thanks! Feel free to merge when ready!

Whenever you're ready!


Create a fresh Slack for production

Do for original wolftutor too!

moved both to https://wolftutor-p.slack.com


Opportunity for lazy initialization in Headers interface

Motivation:
The Headers interface had two member variables (addAllVisitor, setAllVisitor) which are not necessarily always needed but are always instantiated.  This may result in excess memory being used.
Modifications:

addAllVisitor will be accessed via a method addAllVisitor() which will use lazy initialization.
setAllVisitor will be accessed via a method addAllVisitor() which will use lazy initialization.

Result:
Potential memory savings by using lazy initialization.

@normanmaurer - WDYT?  Some additional rational is that the number of Headers objects is expected to be large and these members may not be used at all (or potentially infrequently).

If we have consensus I will backport to 4.1 and 4.0.

@Scottmitch looks good !

Cherry-picked. master (3b1f15e366c36cffe5e638f60dfddd7fb7badfb9), 4.1 (27a2017f7f5db2707b21600c2d976afbc7b57cfd), 4.0 (?).
Humm it looks like the header updates are not in 4.0.  I thought I backported them a while ago...perhaps not because of backwards compatibility?

@trustin - Do you remember what we decided to do for the header modifications on the 4.0 branch?  Looks like we decided not to backport, or was this a miss?

@Scottmitch we can't back port because of api breakage  I think...


Improve messages references instance methods vs static methods

The common convention is to use # to talk about instance methods (replacing .prototype.) and . for static methods (as that's the way they are referenced in JS).
Many rules were already following that convention in their messages, but a few of them were messing that. This PR fixes this.

Thanks for this, although I already cleaned these up in #94 while adding the documentation builder.

Looks like there are still some changes after rebasing, so re-opening


Home Assistant Integration

Hello there, i just realized that the snips integration for home assistant works with rhasspy too (thanks to the hermes protocol). snips.say works out of the box for me.
Sadly snips.say_action only publishes this mqtt message, but nothing happens in rhasspy:
hermes/dialogueManager/startSession {"siteId": "default", "customData": "", "init": {"type": "action", "text": "was", "canBeEnqueued": true, "intentFilter": ["WasGeht"]}}
hermes/dialogueManager/sessionQueued {"sessionId": "193ab2e7-5928-4b11-8d38-c2ac3f596e6a", "siteId": "default", "customData": ""}

https://www.home-assistant.io/integrations/snips/

Another little problem: If "snips:" is defined in HA's configuration.yaml, all tts answers are spoken 2 times

I figured it out. It works fine if you set
can_be_enqueued: false
in your home assistant service call.
It seems like rhasspy is not supporting hermes/dialogueManager/sessionQueued messages yet.
Another Problem: Rhasspy starts listening while it is talking the question, so the question gets recognized again :(

Are you running a separate snips installation? Or just HA and rhasspy? And thus using rhasspy's mqtt broker?

I am running everything in seperate docker containers:

3x Rhasspy Instances
Home Assistant
MQTT Broker

Which mqtt broker are you using?

I am using the latest toke/mosquitto docker image

If you just use it for forwarding mqtt messages you could use rhasspy's broker as well. Just open up the mqtt port on your rhassby base (12183) and let your satellites connect to it.
Will try out the snips: integration as well soon

good to know side effect: if you let home-assistant and rhasspy share the same mqtt broker (and even disable intent handling on rhasspy side - like recommended above) - any example sentences you put into the recognize field on rhasspy's web site, will be handled (even if you do not check the handle box)

I have almost the same setup as Benedikt with the difference that I only have 2 rhasspy's I also get the sound of the text configured in home assistant twice. And yes every test is actually being run for real. Should we use a second mosquitto instance just for rhasspy or just use the internal broker?

I have almost the same setup as Benedikt with the difference that I only have 2 rhasspy's I also get the sound of the text configured in home assistant twice. And yes every test is actually being run for real. Should we use a second mosquitto instance just for rhasspy or just use the internal broker?

Ok I figured out, that you don't need snips at all. At least if you are using mosquitto and intent_script in the configuration. And when you remove everything works and you only have speech once.

Ok I figured out, that you don't need snips at all. At least if you are using mosquitto and intent_script in the configuration. And when you remove everything works and you only have speech once.

@synesthesiam can be closed


export default as doesn't seem to work with Jest.

I got an UIKit/index.js which exports other components out.
export { default as Button } from '../Button';

However, when testing any other components that import components from it
import { Button } from '../UIKit';

works fine in normal environment, but fails in Jest environment which Button is undefined when import.
Anyone knows how to fix it?

Hello!
Which version of Jest and npm are you using? I'm fairly confident that if you are using the latest versions of Jest and babel-jest, this shouldn't be an issue.
If your problem isn't resolved by that, please post more source code and your Jest configuration, ideally even a small repo that I can use to reproduce the issue. If this turns out to be an issue with Jest, I will reopen this issue :)

problem solved! I was previously using
const Button = require('../Button').default;

to avoid heist problem with import statement
When I changed to
import '../auto-mock-off';
import Button from '../Button';

and problem solved.
reference: https://github.com/rauschma/babel-on-node

Thanks for the solution, @AwesomeJerry. But it seems to be a very quirky solution. I hope that underlying issue would be fixed soon and we would be able to get rid of this hack.

jest 0.9.3 (out next week) will contain jest.disableAutomock which will be hoisted above imports. It should resolve this issue.

I'm using jest 21.1.0 and this is still an issue:
export {default as Icon} from './icons/components/icon';

    TypeError: Cannot redefine property: Icon
        at Function.defineProperty (native)

I'm using jest 26.6.3 and this is still an issue:
export { default as Text } from './Text';

TypeError: Cannot read property 'default' of undefined

I have the same problem here:
export { default as DocumentLightBox } from "./DocumentLightBox/DocumentLightBox.vue";

TypeError: Cannot read property 'default' of undefined

This is happening with .ts files to


ESM attributes rename

Currently ESM Scripts attributes are defined as follows
class MyScript extends ScriptType {
    static attributesDefinition = { ... }
}

The static attributesDefinition is both verbose and confusingly names as we already use attributes. This PR uses the existing static attributes property which makes for a cleaner, more intuitive interface, and helps users migrate
class MyScript extends ScriptType {
    static attributes = { ...  }
}

When the script is registered, this is used to populate the ScriptAttributes class and so becomes equivalent to using MyScript.attributes.add(...). Ini addition the attributes` property can already be a plain object, so this doesn't change things significantly, but it improves the overall usability of the class.
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

Any thoughts on this @Maksims too?

How about this:
class MyScript extends ScriptType { }
MyScript.attributes.add('attr', { type: 'text' });

This is a current way of defining attributes and would be the easiest way to migrate from non-ESM scripts.
I do agree that having a static property on a class would make it nice too, but thought of it before and we have this unfortunate collision of names here.

I'm following the decorators proposal currently at stage 3. I'm hoping it goes all the way but we'll have to wait and see. It looks like a great candidate for script attributes if it does land 💪
The MyScript.attributes.add('attr', { type: 'text' }); syntax should still work, but this PR is as just an alternative to the current attributesDefinition which I think works for now. The ordering point is valid, but IIRC it's only numeric keys that get sorted, the rest should retain insertion order.

Then it all sounds great!


Scrolling is clicking in results on web version with using mobile device

I believe that's only on Chrome on Desktop; the attached GIF is on Chrome on my device.
Note however that the scrollview is way too small on small screens.


Novel pose animation

When running the novel pose animation:
python demo/test_ani.py configs/ssdnerf_avatar_uncond_thuman_conv_16bit.py work_dirs/ssdnerf_avatar_uncond_16bit_thuman_conv/ckpt/ssdnerf_avatar_uncond_16bit_thuman_conv/latest.pth --gpu-ids 0
It report error:
~/E3Gen/lib/apis/test.py", line 238, in animate_3d     data['cond_poses'] = data['cond_poses'][:, 31:32] KeyError: 'cond_poses'

You can try changing the key "cond_poses" to "test_poses", the rendered result needs to be flipped horizontally and vertically to get the final animation results. If it still doesn't work, please update the error below, I will check it again. Sorry for the inconvenience.

load your checkpoint error: unexpected key in source state_dict: decoder.init_pcd, decoder.init_rot, decoder.face_mask, decoder.hands_mask, decoder.outside_mask, decoder.deformer.init_spdir, decoder.deformer.init_podir, decoder.deformer.init_faces, decoder.deformer.init_lbsw, decoder.deformer.tfs_inv_t, decoder.deformer.vs_template, decoder.deformer.smpl_faces, decoder.deformer.pose_offset_cano, decoder_ema.init_pcd, decoder_ema.init_rot, decoder_ema.face_mask, decoder_ema.hands_mask, decoder_ema.outside_mask, decoder_ema.deformer.init_spdir, decoder_ema.deformer.init_podir, decoder_ema.deformer.init_faces, decoder_ema.deformer.init_lbsw, decoder_ema.deformer.tfs_inv_t, decoder_ema.deformer.vs_template, decoder_ema.deformer.smpl_faces, decoder_ema.deformer.pose_offset_cano. but it does not influence the code running.
I change the line 238 to 241 in test.py:         data['test_poses'] = data['test_poses'][:, 31:32]      data['test_intrinsics'] = data['test_intrinsics'][:, 31:32]      betas = data['test_smpl_param'][:, 70:80].unsqueeze(1).expand(-1, 50, -1).float()      data['test_smpl_param'] = torch.cat((smplx_param.unsqueeze(0).expand(4, -1, -1)[:, 0::2].to(betas.device).float(), betas), 2). Now it report error     return torch._VF.split_with_sizes(self, split_size, dim) RuntimeError: split_with_sizes expects split_sizes to sum exactly to 175 (input tensor's size at dimension 1), but got split_sizes=[1, 3, 3, 63, 10, 12, 12, 3, 3, 3, 10]

The first warning comes from the self.register_buffer('xxx', xxx, persistent=False) part, which means don't save parameters. So these parameters will not be saved to the last checkpoint and will be reloaded when running inference, which should not affect the final result.
This error occurs because the original test.py uses the SMPLX parameters from the THuman dataset, which uses 12 dimensions to represent gestures, while the SMPLX parameter provided by the AMASS dataset uses 45 dimensions to represent gestures, so the size is incorrect. You need to change split_sizes=[1, 3, 3, 63, 10, 12, 12, 3, 3, 3, 10] to split_sizes=[1, 3, 3, 63, 10, 45, 45, 3, 3, 3, 10].
I will upload cam_36.json tomorrow because the server containing it is under repair and will be back tomorrow.

For inference code, I only tested the test.py, so the other inference codes contain errors, sorry for that. I will update them in this week.

thank u for reply

and also [1, 3, 3, 63, 10, 45, 45, 3, 3, 3, 10] sum is not equal to 175, still the above error

Sorry for the delay, animation code is updated now, and you can pull the latest code.

Great! I can run! But the results don't support facial expression animation.
https://github.com/user-attachments/assets/a2ef7375-3cfb-4de7-9475-2469cda4dd62

I am testing the code with AMASS CMU, which includes common expressions. Therefore, I do not add expression parameters in the code for simplification. To enable expression control, you can modify the animate_3d function in E3Gen/lib/apis/test.py. Append expression parameters to the smplx_param = np.concatenate section and update the prepare_deformer function in E3Gen/lib/models/deformers/smplx_deformer.py. For the check if smpl_params.shape[1] == 175: # smpl_params from amass, adjust the shape size and use torch.split(smpl_params, []) to access the expression parameters. Replace expression = torch.zeros(num_scenes, 10).to(device).float() with this modification. Following these steps, you will gain control over facial expressions.


Segmentation fault with @extend and :global(> .sth)

Before opening an issue:

Read the common workarounds in the TROUBLESHOOTING.md
Search for duplicate or closed issues
Validate that it runs with both Ruby Sass and LibSass
Prepare a reduced test case for any bugs
Read the contributing guidelines

When reporting an bug, you must provide this information:

NPM version (npm -v): 4.5.0
Node version (node -v): 7.9.0
Node Process (node -p process.versions):

{ http_parser: '2.7.0',
  node: '7.9.0',
  v8: '5.5.372.43',
  uv: '1.11.0',
  zlib: '1.2.11',
  ares: '1.10.1-DEV',
  modules: '51',
  openssl: '1.0.2k',
  icu: '58.2',
  unicode: '9.0',
  cldr: '30.0.3',
  tz: '2016j' }

Node Platform (node -p process.platform): darwin
Node architecture (node -p process.arch): x64
node-sass version (node -p "require('node-sass').info"):

node-sass       4.5.2   (Wrapper)       [JavaScript]
libsass         3.5.0.beta.2    (Sass Compiler) [C/C++]

npm node-sass versions (npm ls node-sass):
<EMAIL_ADDRESS>/mydir
└──<EMAIL_ADDRESS>
When encountering a syntax, or compilation issue:

Open an issue on LibSass. You
may link it back here, but any change will be required there, not here

If you delete this text without following it, your issue will be closed.
Here's a content that will fail:
.item {
    display: inline-block;

    :global(> .ext-link) {
        background: #000;
    }
}

.link {
    color: red;
}

.textLink {
    @extend .link;
    padding: 0 10px;
}

Looks like :global(> is very important here. Because without it, it works, or :global(.sth) works too.

Fixed in 4.8.0


xy-output-component: Deserialize for isScatterPlot

Reverse negative if statement.
Rescope yValues/pairs locally.
Deserialize for isScatterPlot.
Add urlFilter to the attach-chrome config.

Each commit message should have their necessary details.

Accidentally closed this PR by mistake; still WIP.

Code looks good. Once the corresponding tsp-typescript-version is available, then you can update this one with the updated yarn.lock file.

Thanks. I removed the tsp-typescript-client@next lines in yarn.lock to force yarn into updating those with the latest. I couldn't get yarn upgrade or other related commands to update that package prior.
This happened to also update another (unrelated) line for json-bigint. However that looks like a trivial update which had to happen anyway it seems.


CRUD for events using postgres

What does this PR do?
Switch from Data structure to Postgressql
implement event CRUD functionalities with Postgres
Description of Task to be completed?
Create migrate file for sqlalchemy
create and get events
get single events update and delete event
How should this be manually tested?
clone the repo and change the branch to challenge-3
create a Postgres database and go to config file in
the TestingConfig file change the SQLALCHEMY_DATABASE_URI
to the one you created
then run
nosetests test
What are the relevant pivotal tracker stories?
#153045245
#153045224

Good implementation of the tests
Shouldn't be having Tokens when logging in a person?
Good implementation of the tables


Page keeps 'loading' after decrypt

I'm using Firefox 56.0a1.
After I decrypt a page (short text string, not HTML), the spinner on the tab keeps going until I hit the escape key.

Yep, it's fixed!  Thanks!


Update Miri invocation in CI

Miri flag passing changed. Fixed #45.

Miri tests failed with:
     Running target/x86_64-unknown-linux-gnu/debug/deps/intaglio-f9786b8e4256ba13
thread 'main' panicked at 'all extern filenames should end in `.rlib`', src/tools/miri/cargo-miri/bin.rs:666:59
stack backtrace:
   0: rust_begin_unwind
             at /rustc/1fd5b9d516c035a898dcb437b2f982bea5d4bc88/library/std/src/panicking.rs:483
   1: core::panicking::panic_fmt
             at /rustc/1fd5b9d516c035a898dcb437b2f982bea5d4bc88/library/core/src/panicking.rs:85
   2: core::option::expect_failed
             at /rustc/1fd5b9d516c035a898dcb437b2f982bea5d4bc88/library/core/src/option.rs:1213
   3: cargo_miri::main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
error: test failed, to rerun pass '--lib'

Tests used this nightly:
  nightly-x86_64-unknown-linux-gnu installed - rustc 1.48.0-nightly (1fd5b9d51 2020-09-20)

@RalfJung did I do something wrong with this upgrade, per #45?

Thanks for the quick reaction and sorry for this. This is a Miri issue that only shows up when certain proc macro crates are in the dependency tree. It is fixed in Miri now, I hope the fix will make it to the next nightly.

Thank you. 🙏
I'll rebuild in a couple of days and merge this.

The new nightly is out, this should work now. :) Sorry for the delay.

Kicked the build. It passes now. 📈 🚀
Thanks @RalfJung.


end sentence, and -ly in ordinals is superfluous

The second part i don't feel strongly about, but it feels out of place to me
as -ly in ordinals is considered superfluous. Especially in this case, as there
is no secondly, etc.

merged in 66f5aa9fa5d53ddd7fbdb7ddac39c429f0c1b4fd, thanks.


chore(merge-back): 2.73.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 119f800eb74025854b46f77b179657690eacda56
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


AttributeError: module 'cv2.cv2' has no attribute 'cudacodec'

Expected behaviour
i success install pip install opencv-contrib-python
but when
cv2.cudacodec.createVideoWriter("tmp", (128, 128), 30)
it  info AttributeError: module 'cv2.cv2' has no attribute 'cudacodec'
why ?
opencv-contrib-python contain cudacodec but why no module

i am in china,can who help  me,my english is not well .but i think i tell the problem exactly


Installation of v20190715.0 fails on aws Ubuntu 18.04

Hi,
I just tried out to install a cluster with v20190715.0 on aws Ubuntu 18.04 (ami-009c174642dba28e4) and the installer failed with this message:

E: Package 'ntp' has no installation candidate
null_resource.flynnlayer0[0] (remote-exec): ===> 14:56:53.010 ERROR: failed to run "apt-get install --yes iptables zfsutils-linux ntp", exit status 100

Here is the complete log of the installer:

===> 14:56:52.327 *** WARNING ***
===> 14:56:52.329 About to stop Flynn and remove all existing data
===> 14:56:52.330 stopping flynn-host daemon
Failed to stop flynn-host.service: Unit flynn-host.service not loaded.
 Done
Building dependency tree... 0%
Building dependency tree... 0%
Building dependency tree... 50%
Building dependency tree... 50%
Building dependency tree
Reading state information... 0%
Reading state information... 0%
Reading state information... Done
Package ntp is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source
E: Package 'ntp' has no installation candidate
===> 14:56:53.010 ERROR: failed to run "apt-get install --yes iptables zfsutils-linux ntp", exit status 100
null_resource.flynnlayer0[0]: Still creating... [50s elapsed]
sudo: flynn-host: command not found
Failed to start flynn-host.service: Unit flynn-host.service not found.

I think the line

/tmp/install-flynn: line 279: zpool: command not found

is also notable.

Hmm, can you try running apt-get update before the install script?
The zpool error is a red herring due to the use of --clean.

That did the trick, thanks.
But now it hangs on

15:50:17.816982 restore-postgres restore
The backup I'm restoring is really small. I once setup an empty cluster with the same domain and took it from there right after the installation, no apps deployed.
I attached the debug information.
restore-debug-information.tar.zip
(had to zip it because GitHub didn't let me upload the tar)

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.


[B03986] Create Field Profile modal improvements

Handle alerts in the model.
Clear the alerts on close.
Provide a cancel button.
Treat the cancel button and X (close) button as the same.
Validation has not been implemented at this time because there appears to be a lot of significant design changes that would be necessary to utilize the weaver validation.
While I would prefer to have the buttons in the button section of the model and the messages across the top, these changes were avoided to ensure consistency in existing design.

Coverage increased (+0.06%) to 80.156% when pulling 22e54ad3c7d41b37a70125c5648821875d7103ee on sprint12-b03986-field_profile_modal into ddd80c94ebcdb1d950fee7d8c01d95e7dd43cb49 on sprint12-staging.


ja: Make docs/reference/_index.md follow v1.18 of the original text

This is a Feature Request

What would you like to be added
Update docs/reference/_index.md to follow v1.18 of the original (English) text.
Why is this needed
content/ja/docs/reference/_index.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/blob/dev-1.18-ja.1/content/ja/docs/reference/_index.md
Original
https://github.com/kubernetes/website/blob/fb6364d/content/en/docs/reference/_index.md
diff between translated and v1.18
https://gist.github.com/cc6b07d7f9a5e824310c2812d3193d04
Note
Currently, we use dev-1.18-ja.1 branch. Please open a PR targeting the branch.
ref
https://kubernetes.io/ja/docs/contribute/localization/
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel.

/milestone 1.18-ja.1

/assign

/close


Formatting on demo files.

Description
Prettier formatting of demo files. It looks like these files didn't get run through prettier when they were initially committed.

Related Issue
None

Motivation and Context
Playing by the rules.

Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] I have signed the Adobe Open Source CLA or I'm an Adobe employee.
[x] I have submitted a documentation pull request or no changes are needed.
[x] I have made any necessary test changes and all tests pass.
[x] I have run the Sandbox successfully.

Nevermind. These files are removed in https://github.com/adobe/alloy/pull/284


Considerable slowdown after updating on 8/28 at 8 am pst

Your question
I've been using this setup with flux D for awhile but after pulling updates this morning I am noticing a considerable slowdown.  Like from a typical generation time of 40 seconds going up to as long as 3 minutes.  No errors or other strange stuff, it's just taking a lot longer to generate.  Was something changed that could account for this?  Some generations are still quick but there is no pattern I can see.  I've had ones using just one character lora take a lot longer than the same setup did yesterday.
Logs
There don't seem to be any errors in the logs, looks the same as before.
Other
The machine has 256g RAM, 24gb 4090 gpu

either (A) your system is just having a time, or (B) it's an issue with a recent comfy update
in the case of B: go to server->backends->edit the comfy backend->set autoupdate to false->then save and shutdown swarm
then open the folder (Swarm)/dlbackend/comfy/ComfyUI, open a terminal in the folder, and run git checkout v0.1.3 or v0.1.2, then relaunch swarm and try that


deps: update strsim to 0.7

https://github.com/docopt/docopt.rs/commit/e105852ccdf4a895114bba9f107d18a331c5de26


Eliminate client certificate for gardenlet and extension controllers

How to categorize this PR?

/area security
/kind enhancement
/merge squash
What this PR does / why we need it:
Similar to #4931, this PR eliminates the client certificate for the gardenlet and extension controllers in favor of a token managed by the TokenRequestor controller in gardener-resource-manager.
Which issue(s) this PR fixes:
Part of #4661
Part of #4878
Special notes for your reviewer:
/invite @BeckerMax
Release note:

`gardenlet` and extension controllers do no longer use a client certificate but an auto-rotated `ServiceAccount` token which is only valid for `12h`.

/assign

I had to rebase, @BeckerMax please check it again.

/lgtm


Curl Backend: fixing unpredictable behavior when empty response

The space allocated via malloc contains garbage and if there is no zero-terminated content in the response from curl, that space stays untouched and garbage is returned to the user.

Thanks! :) That must have been a hard find


Adding RBR

I'm not sure how to add the graphics to the fonts, and it looks like your svg's are much cleaner than what comes out of inkscape. Let me know if I need to do something else. http://tech.ribbit.me

Hi genecyber, it's OK to just commit SVGs, I'll add them to the fonts myself.
Regarding the code of SVGs, I'm exporting them from Illustrator and then manually cleaning up all the redundant meta stuff (it's OK to have just the  part). Inkscape should do the job as well, just be sure to save the graphics in pure SVG (or how it's called), not the Inkscape SVG format.
Also, please try to keep up the consistency with other icons in the set – ICON-alt.svg as a standalone black/white logo and ICON.svg as a logo inside black circle. (However I can post-edit that myself if you'd like.)

ok cool, I'll modify and re PR


Ai generate simulation data

Description
Ai generate simulation data
Added

"mass_flow_scalar" to change the percentage of mass flow available.
LinearDoE

LinearDoe
mass_flow_scalar


[Mention The Feature]: enhancing the resume builder section

Is your feature request related to a problem? Please describe.
The color and various fields are not looking professional
Describe the solution you'd like.
I will add new features and make it more responsive.
Describe alternatives you've considered.
No response
Additional context.
No response
Show us the magic with screenshots
No response
Checklist

[X] I have checked the Existing Issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

sure go ahead


Add meta fields to package.json

I noticed by chance that package.json is missing some meta fields (related to the repository URL, folders, and keywords), so here is a PR to fix this.
In my case, Renovate bot PRs was not adding the project homepage URL to its updates table summary.

Thanks!

Thanks!


Sequence Wallet web and injected connector support

This PR adds support for Sequence Wallet (https://sequence.build), an Ethereum-compatible Web3 contract account wallet. The Sequence Wallet has two modes of integration, either as an on-demand Web wallet, or as a Chrome extension (https://chrome.google.com/webstore/detail/sequence-wallet/ocmccklecaalljlflmclidjeclpcpdim).
In addition to the PR, there is a demo-dapp made which uses this web3modal connector: https://github.com/0xsequence/demo-dapp-web3modal -- however in order to try it you will need to yarn link it until this PR is merged+published.
Here is a video demonstration of the integration as well: https://watch.screencastify.com/v/5bQMPPRklk1RNBbUb4vQ

Btw, if you'd like to try this PR + demo-dapp-web3modal as in the video here are the steps.
1. git clone git@github.com:0xsequence/web3modal.git
2. cd web3modal; yarn; yarn build
3. yarn link

in separate terminal
1. git clone git@github.com:0xsequence/demo-dapp-web3modal.git
2. cd demo-dapp-web3modal; yarn
3. yarn link web3modal
4. yarn dev

It seems like this is just an injected provider so you don't need to add a separate connector

thanks Pedro :D


problem afte update to <IP_ADDRESS>

After update to <IP_ADDRESS>
in end of all page of my project i see this:
An Error occurred while handling another error:
yii\web\HeadersAlreadySentException: Headers already sent in C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\widgets\ContentDecorator.php on line 79. in C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\web\Response.php:366
Stack trace:
#0 C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\web\Response.php(339): yii\web\Response->sendHeaders()
#1 C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\web\ErrorHandler.php(135): yii\web\Response->send()
#2 C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\base\ErrorHandler.php(111): yii\web\ErrorHandler->renderException(Object(yii\web\HeadersAlreadySentException))
#3 [internal function]: yii\base\ErrorHandler->handleException(Object(yii\web\HeadersAlreadySentException))
#4 {main}
Previous exception:
yii\web\HeadersAlreadySentException: Headers already sent in C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\widgets\ContentDecorator.php on line 79. in C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\web\Response.php:366
Stack trace:
#0 C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\web\Response.php(339): yii\web\Response->sendHeaders()
#1 C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\base\Application.php(392): yii\web\Response->send()
#2 C:\xampp\htdocs\myproject\frontend\web\index.php(17): yii\base\Application->run()
#3 {main}
and also in debug panel have one error log
with category:
yii\web\HeadersAlreadySentException
and message:
yii\web\HeadersAlreadySentException: Headers already sent in C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\widgets\ContentDecorator.php on line 79. in C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\web\Response.php:366 Stack trace: #0 C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\web\Response.php(339): yii\web\Response->sendHeaders() #1 C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\base\Application.php(392): yii\web\Response->send() #2 C:\xampp\htdocs\myproject\frontend\web\index.php(17): yii\base\Application->run() #3 {main}
should I do any changes in my project? or this is a bug in framework?

seems related to https://github.com/yiisoft/yii2/issues/15782

Ok I also see http://www.yiiframework.com/news/165/yii-2-0-14-is-released/ Error handling section
But How i can found where i set header by this error trace!

https://stackoverflow.com/a/15844021/3584248

that's right but in Response in framework headers_sent() used and only return probelm is in
view.php of framework files and line 181!
ok if i set a header in my project or other extentions i installed how found that?

Maybe step-by-step debug?

it is not better to change in the error logger to have better log !??
when i comment
//        if (headers_sent($file, $line)) {
//            throw new HeadersAlreadySentException($file, $line);
//        }
in response.php of framework
see this error:
Cannot modify header information - headers already sent by (output started at C:\xampp\htdocs\myproject\vendor\yiisoft\yii2\web\View.php:181)
I am trying to find it for 5 hours step by step commenting all component bootsrap and all thing in view.
but i could not find it yet.

@hooman-pro Are you echoing content somewhere in your application instead of returning it in the controller action?

@schmunk42
No, I did not.
But this error only occurred in frontend , backend has not this error however!
One thing is amazing is in frontend actions if i write $this->render(...) or return $this->render(...) both of them work correctly! but in backend $this->render(...)  dont show any thing!
I guess the problem is related to this.

Maybe you have a file in you project with space or newline before opening PHP tag (<?php) or after closing php tag (?>). This will also lead to Headers Already Sent exceptions

See https://github.com/yiisoft/yii2/issues/15782#issuecomment-368553054 for a quick workaround (enabling output buffering)

Finally problem found. thanks all
I changed
output_buffering=4096
to
output_buffering=On
in php.ini
By changing output_buffering to On in the server, Can this makes problem in future?

The reason for this problem was only one line I forgot to put
beginPage() ?>
in first of my layout before

simply do
 \Yii::$app->response->format = \yii\web\Response::FORMAT_JSON;   \Yii::$app->response->data  =  $data;

In my case problem was when some controller makes "echo".

My project always echo api result (and not return), how can I fix one for all api or I must fix one by one?

Using global find-replace?


⚠ deces.matchid.io search API has degraded performance

In b35f514, deces.matchid.io search API (https://deces.matchid.io/deces/api/v1/search?q=pompidou) experienced degraded performance:

HTTP code: 200
Response time: 2452 ms

Resolved: deces.matchid.io search API performance has improved in b771aab.


notification click error

bro when i add intent for notification builder .. app is too slow downloading

I'm testing it

I don't have this problem here, check the code in your NotificationFactory，or you can post it on here.

It looks like there is no problem... dont know why this happen...

It can cause largest file bro?
I test two 300Mb download links in my app bro


Soft crash

Sometimes, when one map changes to another on my server, clients get stuck at "Downloading sound/weapons/talos/talos_boost_loop.wav." In the console, the message that appears whenever this happens is "CreateFragmentsFromFile: 'sound/weapons/talos/talos_boost_loop.wav' doesn't exist." This happens roughly 40% of the time that the map changes from one to another. Clients can cancel attempting to download the file & reconnect to the server when this happens
Included are several logs from around the times when this has happened
logs.zip

Fairly sure it's not a soft crash; it just takes a while to time out the download.
Thanks for the report though; I might as well drop the custom logic from the builds.


Connection error

Problem with connecting.  Is this the right way to use code?  From the response?  Why is connection failing?
Ref: Issue #31.

bynder_client = BynderClient(
    domain=DOMAIN,
    redirect_uri='https://localhost/callback',
    client_id=CLIENTID,
    client_secret=CLIENTSECRET,
    scopes=['offline',
            'asset:read',
            'meta.assetbank:read'],
)

print(bynder_client.get_authorization_url())
code = input('Code: ')
bynder_client.fetch_token(code)

asset_bank_client = bynder_client.asset_bank_client
media_list = asset_bank_client.media_list({
    'limit': 2,
    'type': 'image'
})

# RESPONSE:
#    ('https://https://[MYDOMAIN]/v6/authentication/
#      oauth2/auth?response_type=code&client_id=[MYCLIENTID]&
#      redirect_uri=https%3A%2F%2Flocalhost%2Fcallback.&scope=offline+asset%3Aread+
#      meta.assetbank%3Aread&state=ElNUPvMv', 'ElNUPvMv')
#
# ENTERED CODE from state above...
#    Code: ElNUPvMv
#
# ConnectionError: HTTPSConnectionPool(host='https', port=443): Max retries
# exceeded with url: //[MYDOMAIN]/v6/authentication/oauth2/token
# (Caused by NewConnectionError(': Failed to establish a new connection: [Errno 11001]
# getaddrinfo failed'))

Hi @tedatkc When looking at the respponse  ('https://https://[MYDOMAIN]/v6/authentication/ it looks like your passing https twice, which is why the connection is failing.

Sorry -- I should have caught that.
Still getting problems.  Here is what I ran.  I ran the two options for the redirect URI and still had problems.  Looks like it is not even getting to the callback?

#DOMAIN='https://digitalassets.kimberly-clark.com'
DOMAIN='digitalassets.kimberly-clark.com'

#REDIRECT_URI='https://localhost/callback'
REDIRECT_URI='localhost/callback'

bynder_client = BynderClient(
    domain=DOMAIN,
    redirect_uri=REDIRECT_URI,
    client_id=CLIENTID,
    client_secret=CLIENTSECRET,
    scopes=['offline',
            'asset:read',
            'meta.assetbank:read'],
)

print(bynder_client.get_authorization_url())
code = input('Code: ')
bynder_client.fetch_token(code)

asset_bank_client = bynder_client.asset_bank_client
media_list = asset_bank_client.media_list({
    'limit': 2,
    'type': 'image'
})

Here is the response (with some replaced text):

('https://digitalassets.[my_DOMAIN].com/v6/authentication/oauth2/auth?response_type=code&client_id=[my_CLIENTID]&redirect_uri=localhost%2Fcallback&scope=offline+asset%3Aread+meta.assetbank%3Aread&state=1IDz48tG', '1IDz48tG')
Code: 1IDz48tG
Traceback (most recent call last):

  File "[my_environment]byndersdk\lib\site-packages\urllib3\connectionpool.py", line 703, in urlopen
    httplib_response = self._make_request(

  File "[my_environment]byndersdk\lib\site-packages\urllib3\connectionpool.py", line 386, in _make_request
    self._validate_conn(conn)

  File "[my_environment]byndersdk\lib\site-packages\urllib3\connectionpool.py", line 1042, in _validate_conn
    conn.connect()

  File "[my_environment]byndersdk\lib\site-packages\urllib3\connection.py", line 414, in connect
    self.sock = ssl_wrap_socket(

  File "[my_environment]byndersdk\lib\site-packages\urllib3\util\ssl_.py", line 449, in ssl_wrap_socket
    ssl_sock = _ssl_wrap_socket_impl(

  File "[my_environment]byndersdk\lib\site-packages\urllib3\util\ssl_.py", line 493, in _ssl_wrap_socket_impl
    return ssl_context.wrap_socket(sock, server_hostname=server_hostname)

  File "[my_environment]byndersdk\lib\ssl.py", line 513, in wrap_socket
    return self.sslsocket_class._create(

  File "[my_environment]byndersdk\lib\ssl.py", line 1071, in _create
    self.do_handshake()

  File "[my_environment]byndersdk\lib\ssl.py", line 1342, in do_handshake
    self._sslobj.do_handshake()

SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:997)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):

  File "[my_environment]byndersdk\lib\site-packages\requests\adapters.py", line 489, in send
    resp = conn.urlopen(

  File "[my_environment]byndersdk\lib\site-packages\urllib3\connectionpool.py", line 787, in urlopen
    retries = retries.increment(

  File "[my_environment]byndersdk\lib\site-packages\urllib3\util\retry.py", line 592, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))

MaxRetryError: HTTPSConnectionPool(host='digitalassets.[my_DOMAIN].com', port=443): Max retries exceeded with url: /v6/authentication/oauth2/token (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:997)')))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):

  File "[my_environment]byndersdk\lib\site-packages\spyder_kernels\py3compat.py", line 356, in compat_exec
    exec(code, globals, locals)

  File "c:\users\[my_UID]\onedrive - [my_DOMAIN]\projects\2022\fia\gre03327-40020\bynder\test\getting_started_ted.py", line 104, in
    bynder_client.fetch_token(code)

  File "[my_environment]byndersdk\lib\site-packages\bynder_sdk\client\bynder_client.py", line 58, in fetch_token
    return self.session.fetch_token(

  File "[my_environment]byndersdk\lib\site-packages\bynder_sdk\oauth2.py", line 34, in fetch_token
    return super().fetch_token(

  File "[my_environment]byndersdk\lib\site-packages\requests_oauthlib\oauth2_session.py", line 341, in fetch_token
    r = self.request(

  File "[my_environment]byndersdk\lib\site-packages\requests_oauthlib\oauth2_session.py", line 521, in request
    return super(OAuth2Session, self).request(

  File "[my_environment]byndersdk\lib\site-packages\requests\sessions.py", line 587, in request
    resp = self.send(prep, **send_kwargs)

  File "[my_environment]byndersdk\lib\site-packages\requests\sessions.py", line 701, in send
    r = adapter.send(request, **kwargs)

  File "[my_environment]byndersdk\lib\site-packages\requests\adapters.py", line 563, in send
    raise SSLError(e, request=request)

SSLError: HTTPSConnectionPool(host='digitalassets.[my_DOMAIN].com', port=443): Max retries exceeded with url: /v6/authentication/oauth2/token (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:997)')))

@tedatkc are you using a vpn or some sort of proxy? This generally leads to ssl errors. We tested the sdk a little while ago and it is working correctly. So if you are still facing the same issue I would recommend doing a clean-up of your python environment,re-importing the package and retrying.
Here is a thread for a similar issue that might be of help https://discuss.python.org/t/ssl-certificate-verify-failed-certificate-verify-failed-self-signed-certificate/14359

Hey @tedatkc we were informed that you reached out to support about this. I want to reiterate on the fact that you please make use of the sdk methods to make calls and not setup your own session with the requests. I do understand you might be running a few checks of your own around ssl for it perhaps with that?
But looks to me that there is something missing with your ssl permissions or vpn tunneling(if you are using one). We do not have a specific ssl certificate that we need the clients to patch during usage. Like the thread above you might find a solution for the ssl issue/tunneling in a stackoverflow thread with some digging around.

My only purpose in using requests was to discover if something was wrong on my end.  I discovered that my SSL certificates were wrong (being a newbie to SSL).  That has now been corrected.  Yay!
I'm only trying to use the most basic example -- nothing exotic, nothing outside of your "getting started" documentation.  I've been trying to make use the example app.py and sdk methods trying to get things to work.  The code earlier in this chain (and issue #31) was right from the documentation.
I now get: MissingTokenError: (missing_token) Missing access token parameter, from the bynder_client.fetch_token(code) call.
print(bynder_client.get_authorization_url())
code = input('Code: ')
bynder_client.fetch_token(code)
By the way, it's not clear from the documentation what I'm supposed to enter in as "Code:".  The get_authorization_url() call returns a 2-tuple with the URL request and a short string.  Is that what is supposed to be entered as the 'Code'?  The short string?
Thank you!

Hi @tedatkc Would you be able to enable the support feature in the portal so that we can have a look at your OAuth2 app setup? The error that you receive is most likely due to the setup of the client in the portal.
https://support.bynder.com/hc/en-us/articles/360013934239-Grant-Bynder-Access-to-Your-Portal
The code that you need to generate a token is returned after you complete the login challenge after running a successful "Authorize Application". Please refer to our API documentation for more information, https://bynder.docs.apiary.io/#reference/oauth-2.0/authorize-endpoint/authorize-application
This error "I now get: MissingTokenError: (missing_token) Missing access token parameter, from the bynder_client.fetch_token(code) call." is returned because the example is trying to to an API requests while you have not setup an Access token yet.
For an OAuth2 APP - https://support.bynder.com/hc/en-us/articles/360013875180-Create-your-OAuth-Apps
You need to make sure you are enabling the proper Scopes that you need, and also setup the grant_type to the type you are running. In this case you need to setup the OAuth2 app to Authorization code + Refresh Token
Once you've the OAuth2 setup correctly you can go ahead and configure the app.py file. Like my example below.

Once you've configured your domain, redirect_URI, client_id, client_secret, scopes you can run the code. Which will return a link that you need to open in your browser. After doing the login challenge you will be redirected to your redirectURI. That URI will have the code that you need. Copy that code in input it / set it up.

After you copy and paste the code and run it, an Access Token will be generated which can be setup at the token on line 16 of the Example app.
All of these steps are standardised OAuth2 protocols, you can always refer to our documentation in regards to OAuth2
https://bynder.docs.apiary.io/#reference/oauth-2.0/
This information should get you on your way. I'm not sure what software you using but Pycharm is an easy way to run Python scripts, you can download it via https://www.jetbrains.com/pycharm/download/#section=mac

We're working on getting it opened up for support.  Can we arrange a webex or the like to do this real time?

Per Savitha:
Let me know when we can look at this collectively. Due to the nature of the data, we cant keep the access open for long.
So we can set an hour or 30 mins and hatch this out.
Let me know your concerns, if any, else me and Ted are available today after noon at 2 pm or wed between 10 -11 CT.
From: Tim Bloembergen @.>
Sent: Monday, December 12, 2022 2:41 AM
To: Bynder/bynder-python-sdk @.>
Cc: Tower, Ted @.>; Mention @.>
Subject: Re: [Bynder/bynder-python-sdk] Connection error (Issue #32)
CAUTION: This email originated from outside the organization.
Hi @tedatkchttps://urldefense.com/v3/__https:/github.com/tedatkc__;!!O6qgkg!zoVVs7_33m_w3BAyLuMj0eq9QErqzxwzBY7-9OspylWCgC6F-fA4P6ts790xg5DNJ0pBfeJ7T4rL8GgDCrKOULc$ Would you be able to enable the support feature in the portal so that we can have a look at your OAuth2 app setup? The error that you receive is most likely due to the setup of the client in the portal.
https://support.bynder.com/hc/en-us/articles/360013934239-Grant-Bynder-Access-to-Your-Portalhttps://urldefense.com/v3/__https:/support.bynder.com/hc/en-us/articles/360013934239-Grant-Bynder-Access-to-Your-Portal__;!!O6qgkg!zoVVs7_33m_w3BAyLuMj0eq9QErqzxwzBY7-9OspylWCgC6F-fA4P6ts790xg5DNJ0pBfeJ7T4rL8GgD9or2Qwk$
The code that you need to generate a token is returned after you complete the login challenge after running a successful "Authorize Application". Please refer to our API documentation for more information, https://bynder.docs.apiary.io/#reference/oauth-2.0/authorize-endpoint/authorize-applicationhttps://urldefense.com/v3/__https:/bynder.docs.apiary.io/*reference/oauth-2.0/authorize-endpoint/authorize-application__;Iw!!O6qgkg!zoVVs7_33m_w3BAyLuMj0eq9QErqzxwzBY7-9OspylWCgC6F-fA4P6ts790xg5DNJ0pBfeJ7T4rL8GgDCRspse8$
This error "I now get: MissingTokenError: (missing_token) Missing access token parameter, from the bynder_client.fetch_token(code) call." is returned because the example is trying to to an API requests while you have not setup an Access token yet.
For an OAuth2 APP - https://support.bynder.com/hc/en-us/articles/360013875180-Create-your-OAuth-Appshttps://urldefense.com/v3/__https:/support.bynder.com/hc/en-us/articles/360013875180-Create-your-OAuth-Apps__;!!O6qgkg!zoVVs7_33m_w3BAyLuMj0eq9QErqzxwzBY7-9OspylWCgC6F-fA4P6ts790xg5DNJ0pBfeJ7T4rL8GgDfJ1D7nQ$
You need to make sure you are enabling the proper Scopes that you need, and also setup the grant_type to the type you are running. In this case you need to setup the OAuth2 app to Authorization code + Refresh Token
Once you've the OAuth2 setup correctly you can go ahead and configure the app.py file. Like my example below.
[Screenshot 2022-12-12 at 09 35 50]https://urldefense.com/v3/__https:/user-images.githubusercontent.com/11890097/206998883-edae4a5c-e0fe-4ca8-b294-572d68531f47.png__;!!O6qgkg!zoVVs7_33m_w3BAyLuMj0eq9QErqzxwzBY7-9OspylWCgC6F-fA4P6ts790xg5DNJ0pBfeJ7T4rL8GgDOtOTIjo$
Once you've configured your domain, redirect_URI, client_id, client_secret, scopes you can run the code. Which will return a link that you need to open in your browser. After doing the login challenge you will be redirected to your redirectURI. That URI will have the code that you need. Copy that code in input it / set it up.
[Screenshot 2022-12-12 at 09 36 13]https://urldefense.com/v3/__https:/user-images.githubusercontent.com/11890097/206999169-80c4ab17-6e68-40fa-bf6e-3eb3d469a6b2.png__;!!O6qgkg!zoVVs7_33m_w3BAyLuMj0eq9QErqzxwzBY7-9OspylWCgC6F-fA4P6ts790xg5DNJ0pBfeJ7T4rL8GgDinifVmE$
After you copy and paste the code and run it, an Access Token will be generated which can be setup at the token on line 16 of the Example app.
All of these steps are standardised OAuth2 protocols, you can always refer to our documentation in regards to OAuth2
https://bynder.docs.apiary.io/#reference/oauth-2.0/https://urldefense.com/v3/__https:/bynder.docs.apiary.io/*reference/oauth-2.0/__;Iw!!O6qgkg!zoVVs7_33m_w3BAyLuMj0eq9QErqzxwzBY7-9OspylWCgC6F-fA4P6ts790xg5DNJ0pBfeJ7T4rL8GgDFXPw6Kg$
This information should get you on your way. I'm not sure what software you using but Pycharm is an easy way to run Python scripts, you can download it via https://www.jetbrains.com/pycharm/download/#section=machttps://urldefense.com/v3/__https:/www.jetbrains.com/pycharm/download/*section=mac__;Iw!!O6qgkg!zoVVs7_33m_w3BAyLuMj0eq9QErqzxwzBY7-9OspylWCgC6F-fA4P6ts790xg5DNJ0pBfeJ7T4rL8GgDSd-7lYg$
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/Bynder/bynder-python-sdk/issues/32*issuecomment-1346096449__;Iw!!O6qgkg!zoVVs7_33m_w3BAyLuMj0eq9QErqzxwzBY7-9OspylWCgC6F-fA4P6ts790xg5DNJ0pBfeJ7T4rL8GgDSuk6Io0$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/A4P6KPAZGC6GXIPSE4WTC5LWM3QLJANCNFSM6AAAAAASULUW7M__;!!O6qgkg!zoVVs7_33m_w3BAyLuMj0eq9QErqzxwzBY7-9OspylWCgC6F-fA4P6ts790xg5DNJ0pBfeJ7T4rL8GgDI3siOVE$.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>K-C Internal Only

Hi @tedatkc Unfortunately that level is support is not something we can provide on our SDKs. We can confirm that the SDK is working and there's nothing broken in it. I think this is just a matter of setup, we don't have the technical resources to get on a call to live troubleshoot.
With the steps I've provided you should get the SDK up and running on your local machine. Deploying and running the SDK on a server environment is not something we could assist with. If you'd download the tool I mentioned, all you'd have to do it open the project, input your client information that can be generated in the portal and put it in there.
Doing those steps outlined above should get you on your way.
What I can do is is that if you sent me your app.py file so that I can have a look and see if it runs on my machine. To do this you can send it to<EMAIL_ADDRESS>Please leave your clientid & secret in there otherwise I won't be able run it.


feat(proto): add go1.23 iterators over ColMap and ColArr

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


Fix incomplete delegate HubConnectionBuilder allocation

Add authenticationChallengeHandler to httpConnectionOptionsCopy, otherwise completion handler is not called when used from client

Thanks for the fix!


ReferenceError: window is not defined

Describe the bug
Getting the following error while trying to use vitest in a monorepo

// packages/client/src/upload-audio/index.test.ts
import { describe, it } from "vitest"

describe("test", () => {
    it("basic test", async () => {
        expect(true).toEqual(true)
    })
})

Reproduction
N/A
System Info
System:
    OS: macOS 12.6
    CPU: (10) arm64 Apple M1 Max
    Memory: 2.19 GB / 64.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.17.0 - ~/.nvm/versions/node/v16.17.0/bin/node
    npm: 8.15.0 - ~/.nvm/versions/node/v16.17.0/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Safari: 15.6.1

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Closing as the issue was in my vite.config.ts. I had the following line defined is:
// https://vitejs.dev/config/
export default defineConfig({
    // ...
    define: { global: "window", "process.env": process.env },
    // ...
})


Autoincrement trove classifier id.

This should allow to not have to bother about getting the id right when
inserting new.
Legacy PyPI does a :
cursor = st.get_cursor()
    cursor.execute("select max(id) from trove_classifiers")
    id = cursor.fetchone()[0]
    if id:
        id = int(id) + 1
    else:
        id = 1

Then insert, which IIUC is subject to race conditions.

That's still imperfect, as for new root-classifiers l2 is supposed to be equal to id, so you would need to :
INSERT into trove_classifiers (classifier, l2, l3, l4, l5) VALUES ('NEW TROVE', something temporary, 0, 0, 0);

Then :
UPDATE trove_classifiers set l2=id where classifier='NEW TROVE';

(which you can skip for non root new trove classifiers, so this is already an improvement)
I'm unsure if you can use something like:
INSERT into trove_classifiers (classifier, l2, l3, l4, l5) VALUES ('NEW TROVE', __autodertermine this from self.id__ , 0, 0, 0);

Which would be great. But anyway that only affect further Pull requests that would add trove classifiers. I don't see any further test to add as inserting without increasing the primary key is already a violation of the uniqueness constraint.

Failure is unrelated (can't found NPM ??) can anyone restart the tests ?


Bench on Linux

Now make bench works on Linux. Let it be?
Please, macOS users, check I didn’t break the macOS bench build accidentally.

Thanks for providing this!


test: Need a test suite for complete coverage of Move2Kube

Description
Move2Kube is actively being developed with new features and bug fixes continuously being added to the code base. We need a concrete test suite to test the entire Move2Kube project and integrate them with the existing CI/CD pipeline.
Related:

https://github.com/konveyor/move2kube/issues/884
https://github.com/konveyor/move2kube/issues/881

Hi @kmehant, I am suraj, Sorry if I am late but I found this project very interesting. I am currently reading Move2Kube documentation, and after getting a good understanding of the project I will apply to it, in the LFX platform. I also have previous experience working as a Google Summer of Code Open Source contributor, I hope we can work together on this project, Thank you.

Sure, I will look into them and get back to you, Thank you for the response 👍

Hi @kmehant , I am interested to contribute to this project under the LFX mentorship program and hence I applied to it now. Earlier, I have been a Google Summer of Code Mentor for a project that was related to end to end tests and CI/CD and hence I think this project somewhat suits my skill sets. Thanks!!

@kmehant sir, I have read the docs and understood as much as I can about the project. I have mailed you my project proposal, Please look into it and give any feedback, especially if you want to change anything in the timeline. I have also uploaded all the required files (Resume, Cover letter, project proposal) to the LFX mentorship website. I will be looking forward to working with you.
Thank you.

@SAEb-ai and @Sai-Suraj-27 Please submit your application material on the portal if you have not done yet. Regarding the timeline, I think submitting an initial version is okay.

@kmehant Sir, I am only able to select the 23rd date in the calendar, I am currently having midsemester exams in my college up to march 1st, can we have the interview on the 3rd of march? So that I will have at least 1 day to brush up on my concepts before the interview. If you want any proof regarding my exams I can show you. But can you please consider any other date? If that is not possible then, even this Sunday (26th) would do fine sir, as I will not have any exam on Sunday.
Sorry for this inconvenience, Thank you.

@Sai-Suraj-27 Please use this thread for discussions specific to this issue. Thanks.

Sorry, sir, I have deleted it. I thought you will not see my mail, so I messaged here. Sorry for it.


Guard against duplicate fragment names in near-operations-file

Right now if you have multiple fragments with the same name, it will not fail, and you may end up with the wrong fragment being used.
Ideally I would prefer that it figured out using the correct fragment based on the imports in the source file, however that could be pretty complicated.
For now, this will at least prevent you from having duplicate fragment names.

That's great! Thank you @mvestergaard !
Can you please rebase? And then I'll merge this.

Done


fix issue with spotify authorization expiration

if the user authorizes spotify priveledge and does nothing for a while, attempting to convert playlist will result in error 500 due to authorization expiration
needs to keep authorization alive

added code to handle authorization expiration, user will now be prompted to reauthorize from spotify


Behaviour of renewBeforePercentage is the reverse of its spec

Describe the bug:

The spec of the renewBeforePercentage field says:

renewBeforePercentage is like renewBefore, except it is a relative percentage
rather than an absolute duration. For example, if a certificate is valid for 60
minutes, and  renewBeforePercentage=25, cert-manager will begin to attempt to
renew the certificate 45 minutes after it was issued (i.e. when there are 15
minutes (25%) remaining until the certificate is no longer valid).

However, the current behaviour and tests expect the reverse of this behaviour — for example for a certificate valid for 60 minutes and renewBeforePercentage=25, cert-manager will begin to attempt to renew the certificate after 15 minutes (i.e. when there are 45 minutes (75%) remaining until the certificate is no longer valid).
For example, the TestRenewBefore tests that for a duration of three hours (180 min) and renewBeforePercentage of 25, RenewBefore returns 135 minutes, instead of 45:

https://github.com/cert-manager/cert-manager/blob/8f71b919189e319627a6fc2b44eb914e10da3b7c/pkg/util/pki/renewaltime_test.go#L116-L119

I presume this has slipped through the tests because the TestRenewalTime tests for the scenario of renewBeforePercentage=50, which doesn't surface this bug:

https://github.com/cert-manager/cert-manager/blob/8f71b919189e319627a6fc2b44eb914e10da3b7c/pkg/util/pki/renewaltime_test.go#L69-L74

If you change this to anything other than 50, e.g. 30:
		"long lived cert, spec.renewBeforePercentage is set to renew 30% before expiry": {
			notBefore:           now,
			notAfter:            now.Add(time.Hour * 730), // 1 month
			renewBeforePct:      ptr.To(int32(30)),
			expectedRenewalTime: &metav1.Time{Time: now.Add(time.Hour * 511)}, // 70% of 1 month
		},

you get an error:
--- FAIL: TestRenewalTime (0.00s)
    --- FAIL: TestRenewalTime/long_lived_cert,_spec.renewBeforePercentage_is_set_to_renew_30%_before_expiry (0.00s)
        /cert-manager/pkg/util/pki/renewaltime_test.go:97:
            	Error Trace:	/cert-manager/pkg/util/pki/renewaltime_test.go:97
            	Error:      	Not equal:
            	            	expected: time.Date(2024, time.December, 6, 14, 55, 10, 0, time.Local)
            	            	actual  : time.Date(2024, time.November, 24, 10, 55, 10, 0, time.Local)

            	            	Diff:
            	            	--- Expected
            	            	+++ Actual
            	            	@@ -3,3 +3,3 @@
            	            	   wall: (uint64) 0,
            	            	-  ext: (int64)<PHONE_NUMBER>0,
            	            	+  ext: (int64)<PHONE_NUMBER>0,
            	            	   loc: (*time.Location)({
            	Test:       	TestRenewalTime/long_lived_cert,_spec.renewBeforePercentage_is_set_to_renew_30%_before_expiry
            	Messages:   	Expected renewal time: 2024-12-06 14:55:10 +0000 GMT got: 2024-11-24 10:55:10 +0000 GMT
FAIL
FAIL	github.com/cert-manager/cert-manager/pkg/util/pki	0.755s
FAIL

Expected behaviour:

I'd expect the behaviour of renewBeforePercentage be congruent with its spec, so that this would pass:
		"long lived cert, spec.renewBeforePercentage is set to renew 30% before expiry": {
			notBefore:           now,
			notAfter:            now.Add(time.Hour * 730), // 1 month
			renewBeforePct:      ptr.To(int32(30)),
			expectedRenewalTime: &metav1.Time{Time: now.Add(time.Hour * 511)}, // 70% of 1 month
		},

This just involves a one-line fix in RenewBefore and aligning the tests with this behaviour change. For all the changes, please see my PR:

https://github.com/cert-manager/cert-manager/pull/7421

Its git diff is:

Diff of the fix
diff --git a/pkg/util/pki/renewaltime.go b/pkg/util/pki/renewaltime.go
index 318c09d13..6e558abca 100644
--- a/pkg/util/pki/renewaltime.go
+++ b/pkg/util/pki/renewaltime.go
@@ -68,7 +68,7 @@ func RenewBefore(actualDuration time.Duration, renewBefore *metav1.Duration, ren
        if renewBefore != nil && renewBefore.Duration > 0 && renewBefore.Duration < actualDuration {
                return renewBefore.Duration
        } else if renewBeforePercentage != nil && *renewBeforePercentage > 0 && *renewBeforePercentage < 100 {
-               return actualDuration * time.Duration(100-*renewBeforePercentage) / 100
+               return actualDuration * time.Duration(*renewBeforePercentage) / 100
        }

        // Otherwise, default to renewing 2/3 through certificate's lifetime.
diff --git a/pkg/util/pki/renewaltime_test.go b/pkg/util/pki/renewaltime_test.go
index c7510087d..7362cd7c6 100644
--- a/pkg/util/pki/renewaltime_test.go
+++ b/pkg/util/pki/renewaltime_test.go
@@ -66,11 +66,11 @@ func TestRenewalTime(t *testing.T) {
                        renewBefore:         &metav1.Duration{Duration: time.Hour * 25},
                        expectedRenewalTime: &metav1.Time{Time: now.Add(time.Hour * 16)},
                },
-               "long lived cert, spec.renewBeforePercentage is set to renew 50% of the way": {
+               "long lived cert, spec.renewBeforePercentage is set to renew 30% before expiry": {
                        notBefore:           now,
                        notAfter:            now.Add(time.Hour * 730), // 1 month
-                       renewBeforePct:      ptr.To(int32(50)),
-                       expectedRenewalTime: &metav1.Time{Time: now.Add(time.Hour * 365)},
+                       renewBeforePct:      ptr.To(int32(30)),
+                       expectedRenewalTime: &metav1.Time{Time: now.Add(time.Hour * 511)}, // 70% of 1 month
                },
                // This test case is here to show the scenario where users set
                // renewBefore to very slightly less than actual duration. This
@@ -115,7 +115,7 @@ func TestRenewBefore(t *testing.T) {
                },
                "spec.renewBeforePercentage is valid": {
                        renewBeforePct:      ptr.To(int32(25)),
-                       expectedRenewBefore: 135 * time.Minute,
+                       expectedRenewBefore: 45 * time.Minute,
                },
                "spec.renewBeforePercentage is too large so default is used": {
                        renewBeforePct:      ptr.To(int32(100)),

Steps to reproduce the bug:

Create a certificate with renewBeforePercentage of anything other than 50%. Observe that the renewal time of the certificate isn't according to the spec of renewBeforePercentage.
Anything else we need to know?:
Thanks a lot for the awesome project!
Environment details::

Kubernetes version:
Cloud-provider/provisioner:
cert-manager version: 1.16.1
Install method: e.g. helm/static manifests

/kind bug

Original PR: https://github.com/cert-manager/cert-manager/pull/6987


SSL cipher to access url

The examples from the '02_ingest' folder are used for a Qwiklabs lab called "Ingesting Data Into The Cloud Using Google Cloud Functions". When running this code in cloud shell, the ingest_flights.py file gives an ssl error: dh key too small. These changes fix the error and make the lab run correctly.
If this change is unwanted for the whole repo, let me know! I can use a forked repo for the lab instead.

@mylenebiddle


WAC Installation problem and URL

Install WAC on Server 2016 with GUI and an existing Web Server certificate. Can't make it work at all.
What is strange that the URL according the documentation should be https://srv.domain.com. On our server, the URL that the admin center installation creates is https://domain.com/
Also, can't manage anything or add a system, not even the local host where Admin Center is installed. Get SMEGateway Events "401 - PSRemotingTransportException: Connecting to remote server sa failed with the following error message : Access is denied. "

It looks like you used a wildcard certificate, this is not supported. you can add your vote to the request here: https://windowsserver.uservoice.com/forums/295071-management-tools/suggestions/33990187--security-certificate-wildcard-certificate


SPSearchServiceApp is Creating a New SA Even with Existing DB

Details of the scenario you tried and the problem that is occurring
Trying to restore the search service application from existing database. For example, in DR recovery scenario or blue-green deployments or stage infrastructure or for v2v upgrade scenarios.
Verbose logs showing the problem
Current version of the SharePointDsc module is using New-SPEnterpriseSearchServiceApplication, whereas when we need to restore a search service application from existing administration database, we need to use Restore-SPEnterpriseSearchServiceApplication commandlet.
Suggested solution to the issue
Check if the administration database already exists in the SQL instance and run either New-SPEnterpriseSearchServiceApplication or Restore-SPEnterpriseSearchServiceApplication depending on the condition.
The DSC configuration that is used to reproduce the issue (as detailed as possible)
# just an example,
        SPSearchServiceApp SearchServiceApp
        {
            DatabaseName                = "Search_Service_Application_DB_b183238874e14d7dad0aed1941f80308"
            ApplicationPool             = "SharePoint Search App Pool"
            ProxyName                   = "Search Service Application"
            Name                        = "Search Service Application"
            DefaultContentAccessAccount = $SPSearchCrawlAccountCredential
            Ensure                      = "Present"
            PsDscRunAsCredential        = $SPInstallAccountCredential
            DependsOn                   = "[Script]SearchServiceApp"
        }

The operating system the target node is running
Version of SharePoint that is used (e.g. SharePoint 2016)
Version and build of PowerShell the target node is running
Version of the DSC module that was used

Good point. SPFarm is using some code to check if the ConfigDB exists. We can reused that code here.

Documenting info:
New-SPEnterpriseSearchServiceApplication
Restore-SPEnterpriseSearchServiceApplication

Hi @shurick81, I have been investigating this change some more. What is the exact scenario you are trying to accomplish?
The documentation for the Restore cmdlet, I see this:

The first set of parameters is for Application Configuration Attach mode and the second set of parameters is for Search Application Attach mode.
Application Configuration Attach mode only restores configuration data that is stored in the administration database. However, Search Application Attach restores configuration, topology and all crawled data.

Which scenario are you looking for?

Let me put it like this. I have a Disaster Recovery farm precreated. Then when necessary, I want to restore the copy of the administration database from production to the SQL instance of the DR farm and then run a DSC configuration that will attach it.
According to my test, using New-SPEnterpriseSearchServiceApplication results in missing all my custom content sources in the service application. So I concluded that entire restored database was replaced with a fresh, empty database.


Fix for empty string class name

classList.add("") throws an exception that is somehow swallowed.

I'd like to speak in favour of merging this commit ASAP.  In my case, I got bit by creating a set of classes to add with code similar to the following:
      def myClasses(cond1: Boolean, cond2: Boolean) =
        cls := Seq(
            if (cond1) "cond1" else "",
            if (cond2) "cond2" else ""
          ).mkString(" ")

Of the four possible ways to call myClasses, one of them fails when the html is rendered:
        p(myClasses(false, true), "my paragraph").toString() ==> "
my paragraph
"

Or, in a simpler form (note the leading space in " cond2"):
        p(cls := " cond2", "my paragraph").toString() ==> "
my paragraph
"

Curiously, a trailing space in the class renders just fine.

I'd like to speak in favour of merging this commit ASAP.  In my case, I got bit by creating a set of classes to add with code similar to the following:
      def myClasses(cond1: Boolean, cond2: Boolean) =
        cls := Seq(
            if (cond1) "cond1" else "",
            if (cond2) "cond2" else ""
          ).mkString(" ")

Of the four possible ways to call myClasses, one of them fails when the html is rendered:
        p(myClasses(false, true), "my paragraph").toString() ==> "
my paragraph
"

Or, in a simpler form (note the leading space in " cond2"):
        p(cls := " cond2", "my paragraph").toString() ==> "
my paragraph
"

Curiously, a trailing space in the class renders just fine.

@bwbecker IDK if this is more readable to you, but you can use Option[AttrValue] and Option[AttrPair]:
https://scastie.scala-lang.org/9BDfZS9xQpq8xgrHcJityA (the last example)
Also, the 2.13's Option.when is really handy for these. 😃


Encounter log message duplication with DD_EXTENSION_VERSION next

Expected Behavior
Logs produced by Lambda function should not be sent to datadog
Actual Behavior
After adding DD_EXTENSION_VERSION: next as lambda function environment variable, I observe 2 log message with the same timestamp, one produced by Lambda function itself and one by the Datadog Log Forwarder
Logs by the Forwarder

Logs by the Lambda function

Datadog CDK construct configuration
const datadog = new Datadog(scope, 'Datadog', {
      apiKeySecretArn: DATADOG_API_KEY_SECRET_ARN,
      captureLambdaPayload: true,
      enableDatadogLogs: false,
      env,
      version: VERSION,
      injectLogContext: true,
      extensionLayerVersion: 65,
      nodeLayerVersion: 115,
      service: 'service-name',
      site: 'datadoghq.eu',
      enableDatadogTracing: !isDevStage,
    });

Specifications

Datadog Lambda Layer version: 64_next, 65_next
Node version: 20 LTS

Hey @smg-kayle - thanks for reaching out! Choosing between the Datadog Lambda Extension and the Datadog Lambda Log forwarder are two distinct options and are not meant to be combined, this is why you're seeing duplicate logs.
The Lambda Extension is a great choice because it allows you to disable cloudwatch logs and save the $.50/gb ingest charge CloudWatch assesses.
You can learn more in the docs.
Hope this helps!

Sure! The Datadog Lambda Extension (including the next-gen Lambda Extension) can send traces, logs, and metrics – so no forwarder is needed at all.
I'd suggest simply unsubscribing your Lambda Log Groups from the Forwarder to resolve this issue. You may still need the Lambda Log Forwarder for other functionality such as VPC flow logs or API Gateway Access Logs, but it's not necessary for Lambda.
Thanks!
AJ

Yes, thanks for your recommendation.
However, my question has not been clarified yet. I would love to know why using next-gen version making logs appear twice in datadog dashboard, but no duplication at all with (let's say) stable version.

Right – I don't have an answer. We'd always expect duplicate logs with the setup you describe, so my guess is there's a setup or deployment or tagging difference between the two.
Either way the fix is the same, we'd recommend following the documentation and choosing the next-gen Lambda extension.
Thanks!

I looked in a bit more and I think we dropped this config at some point in the next gen extension. I'll add it back and it'll be out in the next release, thanks!


Install failure due to full disk not detected

Description
I was attempting to install halide[core,opencl,opengl]:x64-windows, which requires llvm[core,tools,utils]:x64-windows. I did not have sufficient disk space available to install llvm. vcpkg reported errors while installing llvm (see logs below), but continued with the halide build regardless, then failed the halide build because llvm was missing. After freeing disk space vcpkg continues to fail, assuming llvm is installed when it is not. It appears that vcpkg did not detect the errors while installing llvm's files and assumed that it installed correctly.
Environment

OS: Windows 10 Enterprise (10.0.18363 Build 18363)
Compiler: Microsoft Visual Studio 2017, cl.exe reports version 19.16.27031.1 for x64
vcpkg version reports: Vcpkg package management program version 2020.02.04-nohash

To Reproduce
Steps to reproduce the behavior:

Without llvm or halide installed and with less than 77.6 GB of disk space:
./vcpkg install llvm[core,tools,utils]:x64-windows halide[core,tools,utils]:x64-windows
llvm build fails due to lack of disk space, halide build attempted but fails due to missing llvm
free disk space, sufficient for llvm build
./vcpkg install llvm[core,tools,utils]:x64-windows halide[core,tools,utils]:x64-windows
llvm does not reinstall, halide build fails due to missing llvm

Expected behavior
llvm and halide are installed succesfully
Failure logs

Command line session: vcpkg disk-space error logs.txt
buildtrees/llvm/install-x64-windows-rel-err.log
buildtrees/llvm/install-x64-windows-rel-out.log
buildtrees/llvm/stdout-x64-windows.log
buildtrees/halide/config-x64-windows-out.log

Additional context
None.

Try .\vcpkg.exe remove llvm:x64-windows
then: .\vcpkg.exe install llvm[<features>]:x64-windows
If llvmt installs properly, then proceed with .\vcpkg.exe install halide[<features>]:x64-windows
Notes:

llvm take around 85 gb of disk space for a full build with all features installed.
If llvm will not install properly, the vcpkg instance may be corrupted, requiring the deletion and recloning, compiling, linking of vcpkg along with reinstalling all the ports[] wishing to be installed.

@timautry removing and reinstalling llvm was a successful workaround, and thanks for the heads-up about how much space is required!
I also edited my triplets file to do release-only builds (set(VCPKG_BUILD_TYPE release)) which should help with my lack of disk space.
Cheers,
Sam

Thanks for posting this issue. This issue has been fixed. So I am closing this issue for now.


missing libudev on raspbian?

It seems that libudev-dev is not installed by default on raspbian; as a result, node-usb fails to install.
make: Entering directory '/home/pi/gh/jacdac-cli/node_modules/usb/build'
  CC(target) Release/obj.target/libusb/libusb/libusb/core.o
  CC(target) Release/obj.target/libusb/libusb/libusb/descriptor.o
  CC(target) Release/obj.target/libusb/libusb/libusb/hotplug.o
  CC(target) Release/obj.target/libusb/libusb/libusb/io.o
  CC(target) Release/obj.target/libusb/libusb/libusb/strerror.o
  CC(target) Release/obj.target/libusb/libusb/libusb/sync.o
  CC(target) Release/obj.target/libusb/libusb/libusb/os/poll_posix.o
  CC(target) Release/obj.target/libusb/libusb/libusb/os/threads_posix.o
  CC(target) Release/obj.target/libusb/libusb/libusb/os/linux_usbfs.o
  CC(target) Release/obj.target/libusb/libusb/libusb/os/linux_udev.o
../libusb/libusb/os/linux_udev.c:40:10: fatal error: libudev.h: No such file or directory
   40 | #include <libudev.h>
      |          ^~~~~~~~~~~
compilation terminated.
make: *** [libusb.target.mk:154: Release/obj.target/libusb/libusb/libusb/os/linux_udev.o] Error 1
make: Leaving directory '/home/pi/gh/jacdac-cli/node_modules/usb/build'

I installed it manually and run the install again and things worked.
sudo apt install libudev-dev

I wonder if this kind of dependencies can be baked into the package.json.

libudev-dev is a compilation requirement which would be difficult to include.
I'm more concerned why the prebuilt binaries don't work (RPi should use the arm-linux prebuilds)

I've just tested v2.1.2 on rpi and rpi64 with no issue. They use the shipped prebuild binaries.
There should be no need to compile the package, are you trying to rebuild the package with electron-rebuild or similar?

I was doing a yarn install on my package that uses usb v2.1.2.
I did run apt update/upgrade before that. Maybe I need a new image.

From: Rob Moran @.>
Sent: Thursday, February 10, 2022 11:10:04 AM
To: node-usb/node-usb @.>
Cc: Peli de Halleux @.>; Author @.>
Subject: Re: [node-usb/node-usb] missing libudev on raspbian? (Issue #480)
I've just tested v2.1.2 on rpi and rpi64 with no issue. They use the shipped prebuild binaries.
There should be no need to compile the package, are you trying to rebuild the package with electron-rebuild or similar?
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnode-usb%2Fnode-usb%2Fissues%2F480%23issuecomment-1035360740&data=04|01|jhalleux%40microsoft.com|2cdd98e5532f40e7a0b908d9ecc8f1df|72f988bf86f141af91ab2d7cd011db47|1|0|637801170069912414|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=5EKSrGKb6qWSKCT9csaji9Q7hm8c73Mkad3huUD9mEM%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKKQZGSTEZNMJB32YDDU2QEQZANCNFSM5OBJCVVQ&data=04|01|jhalleux%40microsoft.com|2cdd98e5532f40e7a0b908d9ecc8f1df|72f988bf86f141af91ab2d7cd011db47|1|0|637801170069912414|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=TFXVIKgy1w9dkQhvHYX63LzubDAe%2FNRDL8cRzRQ7F9U%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

TBF, this is all testing via a remote CLI, but I doubt running it through raspbian should make a difference.

This issue should be closed, as libudev should never be bundled statically AFAIK.
Instead we should add instructions to the README telling users to install build-essential libudev-dev if the prebuilds don't work.
And yes it is normal for prebuilds not to work, especially on Raspis and other embedded devices, because of libc compatibility issues:
Error: /lib/aarch64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /home/user/app/node_modules/usb/prebuilds/linux-arm64/node.napi.armv8.node)

In that case it will fall back to compilation and you'll need libudev-dev.
If I find time, I'll try to make prebuilds target an older glibc version to make them more portable.

I'll close this issue as I agree libudev-dev is an end user concern.
@mildsunrise , if you want to look at the prebuild images, they are maintained here:
https://github.com/node-usb/docker-images

A little note in the readme would be awesome! Thanks for digging into this.

From: Rob Moran @.>
Sent: Friday, March 25, 2022 8:00:46 AM
To: node-usb/node-usb @.>
Cc: Peli de Halleux @.>; Author @.>
Subject: Re: [node-usb/node-usb] missing libudev on raspbian? (Issue #480)
I'll close this issue as I agree libudev-dev is an end user concern.
@mildsunrisehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmildsunrise&data=04|01|jhalleux%40microsoft.com|d84cc8e0fd494fa784a508da0e703e45|72f988bf86f141af91ab2d7cd011db47|1|0|637838172491831270|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=Z7r5YsQXFzcyBYhNbkPlRCoPWHmJtvf2VifMDgEr5CU%3D&reserved=0 , if you want to look at the prebuild images, they are maintained here:
https://github.com/node-usb/docker-imageshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnode-usb%2Fdocker-images&data=04|01|jhalleux%40microsoft.com|d84cc8e0fd494fa784a508da0e703e45|72f988bf86f141af91ab2d7cd011db47|1|0|637838172491831270|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=%2BGP0tYhakaAAEYc3KqbVOV6FYN6uo8h6E6UEd%2Fw2BRM%3D&reserved=0
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnode-usb%2Fnode-usb%2Fissues%2F480%23issuecomment-1079114952&data=04|01|jhalleux%40microsoft.com|d84cc8e0fd494fa784a508da0e703e45|72f988bf86f141af91ab2d7cd011db47|1|0|637838172491831270|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=VZmPy5GKAy6qPgHowfjMzOua7Jv6fF9y6ke%2FmXVK9ZI%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKPPPO3FQKR2LNNALWLVBXIJ5ANCNFSM5OBJCVVQ&data=04|01|jhalleux%40microsoft.com|d84cc8e0fd494fa784a508da0e703e45|72f988bf86f141af91ab2d7cd011db47|1|0|637838172492431245|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=bHXL%2Be7dik0pVF206XvBKaI94EctIl49NstIm4v%2F6v0%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>


Individual level files not working

First of all, thank you very much for making this plugin. I can see there is a lot of work that went into this! Thanks for that.
I was wondering if the "individual level file" feature should still work, because I can't get it to work in my environment.

When using the project file, everything gets imported correctly and works as expected
When changing the LDtk setting to "Save levels to separate files" and checking those individual ldtkl files in the Unity project view, those do not have the expected game objects under them, just look like this:

I am using

Unity 2021.2.8
LDtkToUnity 2.1.8
LDtk 0.9.3

Am I doing something wrong?

Ah yes, that's the old editor inspector for level files.
The new separate level files are available by using the newest version of the package: 2.1.8 -> 2.2.0
You can update the package from the package manager window:
https://cammin.github.io/LDtkToUnity/documentation/Installation/topic_Update.html
Thanks for trying out the importer!

Ha, I didn't even check if there is an update as I only recently got it and didn't realise you are that active on this, that's awesome.
Update obviously solved it, thanks a lot!