Custom configmap for lifecycle image results in controller not booting

Follow up of https://github.com/pivotal/kpack/pull/596
Context
We are adding kpack to our stack and are using kustomize to make some tweaks to the yaml file provided with each release due to how we structure things in our cluster. Our kustomize.yaml looks like this:
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
namespace: custom-namespace
namePrefix: kpack-
resources:
- release-0.2.2.yaml

Root cause
I saw this on logs:
$ kubectl logs -f deploy/kpack-controller controller

W0323 22:03:16.464147       1 client_config.go:543] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
{"level":"info","ts":"2021-03-23T22:03:16.560Z","caller":"logging/config.go:110","msg":"Successfully created the logger."}
{"level":"info","ts":"2021-03-23T22:03:16.560Z","caller":"logging/config.go:111","msg":"Logging level set to info"}
{"level":"info","ts":"2021-03-23T22:03:16.561Z","logger":"controller","caller":"profiling/server.go:59","msg":"Profiling enabled: false","commit":"d4de54b"}
E0323 22:03:22.820315       1 aws_credentials.go:77] while getting AWS credentials NoCredentialProviders: no valid providers in chain. Deprecated.
        For verbose messaging see aws.Config.CredentialsChainVerboseErrors
{"level":"info","ts":"2021-03-23T22:03:24.060Z","logger":"controller.Stacks","caller":"controller/controller.go:398","msg":"Starting controller and workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.060Z","logger":"controller.Stacks","caller":"controller/controller.go:408","msg":"Started workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.060Z","logger":"controller.Images","caller":"controller/controller.go:398","msg":"Starting controller and workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.060Z","logger":"controller.Images","caller":"controller/controller.go:408","msg":"Started workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.060Z","logger":"controller.Builds","caller":"controller/controller.go:398","msg":"Starting controller and workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.060Z","logger":"controller.Builds","caller":"controller/controller.go:408","msg":"Started workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.060Z","logger":"controller.Builders","caller":"controller/controller.go:398","msg":"Starting controller and workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.060Z","logger":"controller.Builders","caller":"controller/controller.go:408","msg":"Started workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.060Z","logger":"controller.ClusterBuilders","caller":"controller/controller.go:398","msg":"Starting controller and workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.060Z","logger":"controller.ClusterBuilders","caller":"controller/controller.go:408","msg":"Started workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.060Z","logger":"controller.ClusterStores","caller":"controller/controller.go:398","msg":"Starting controller and workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.061Z","logger":"controller.ClusterStores","caller":"controller/controller.go:408","msg":"Started workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.061Z","logger":"controller.SourceResolvers","caller":"controller/controller.go:398","msg":"Starting controller and workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.061Z","logger":"controller.SourceResolvers","caller":"controller/controller.go:408","msg":"Started workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.161Z","logger":"controller.SourceResolvers","caller":"controller/controller.go:410","msg":"Shutting down workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.161Z","logger":"controller.ClusterBuilders","caller":"controller/controller.go:410","msg":"Shutting down workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.161Z","logger":"controller.ClusterStores","caller":"controller/controller.go:410","msg":"Shutting down workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.161Z","logger":"controller.Stacks","caller":"controller/controller.go:410","msg":"Shutting down workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.161Z","logger":"controller.Builds","caller":"controller/controller.go:410","msg":"Shutting down workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.161Z","logger":"controller.Images","caller":"controller/controller.go:410","msg":"Shutting down workers","commit":"d4de54b"}
{"level":"info","ts":"2021-03-23T22:03:24.161Z","logger":"controller.Builders","caller":"controller/controller.go:410","msg":"Shutting down workers","commit":"d4de54b"}
{"level":"fatal","ts":"2021-03-23T22:03:24.662Z","logger":"controller","caller":"controller/main.go:215","msg":"Error running controller","commit":"d4de54b","error":"configmap \"lifecycle-image\" not found","stacktrace":"main.main\n\t/workspace/cmd/controller/main.go:215\nruntime.main\n\t/layers/tanzu-buildpacks_go-dist/go/src/runtime/proc.go:204"}

So I believe the issue is related to this hard coded lifecycle-image configmap name introduced here which makes this watcher fail, resulting in the controller not booting:
https://github.com/pivotal/kpack/blob/84a6273dc9e68ba6057adff2a066cb231cb434c8/pkg/config/lifecycle_provider.go#L12-L15
Workaround
For now I've just duplicated the configmap, stripping the prefix so that the controller can boot up
:thought_balloon: :thought_balloon: :thought_balloon: :thought_balloon:
I know there's no easy way to customize kpack installation (like potentially using a helm chart) but I suppose that eventually there will be ways to configure things and this is probably going to become a blocker

Hi @fgrehm, It looks like this problem could be resolved by making the lifecycle image configmap configureable. Although, that would break external tooling like the kpack-cli which use the exact name of configmaps.
Out of curiosity, why do you want to add a name-prefix to components inside the kpack namespace?

Out of curiosity, why do you want to add a name-prefix to components inside the kpack namespace?

We're installing kpack to a different namespace, using the kustomize config mentioned in the issue description which not only changes the namespace but also adds a kpack- prefix to all resources that don't have it :smile:  (ex: service accounts and configmaps)

Out of curiosity, why do you want to add a name-prefix to components inside the kpack namespace?

We're installing kpack to a different namespace, using the kustomize config mentioned in the issue description which not only changes the namespace but also adds a kpack- prefix to all resources that don't have it :smile:  (ex: service accounts and configmaps)
The namespace we're installing has other things so we want to be sure kpack's resources won't collide with other things in the target namespace.


Week 1

Name
Sasja Koning
Class
Tech-2
Repo link
https://github.com/sasjakoning/blok-tech-2022/tree/development
Summary
This week I started work on creating a matching app. I decided to go with something that involves gamification. The specific feature I'll be doing probably wont use it but it would be a cool thing to add if I manage to have spare time.
I made job stories:

Thought out a concept:

And did some research:

I also made several sketches and came up with a wireframe(ish):

I created a seperate branch to start work on the base html/css and made some branches to mess around with Node.js, npm, Express and mongoDB. Also did some reading about AJAX and such.
Thats about all the tryhard stuff I'll be doing this week.
For Backend
Played around with node.js and npm. Installed Nodemon and Express. I had some trouble understanding what I was doing so I removed my development branch and started over. I read some docs on npm and node.js which made things a little easier. I'll have to actually make stuff with it to fully understand it, though.
Resources used
https://docs.npmjs.com/
https://nodejs.org/en/docs/guides/
https://github.com/cmda-bt/be-course-21-22/
Any thoughts?
(for Backend)
The assignments were a little confusing. Right at the first step I was already thinking "uhh... what?" even though I'm very excited to start working on backend. I can understand the idea of letting students figure it out themselves but doing so during the very first lesson is perhaps a little overboard.
In the future I'm excited to start working on Express and getting a website to actually run on it.

Hi Sasja, dank voor je lekker uitgebreide update.  Hopelijk staat dit ook allemaal in je wiki (die ga ik straks nog bekijken) en dan zou je in je issue ook mogen volstaan met een verwijzing daar naartoe. Scheelt je hopelijk weer wat werk :)


[tool_crash] StateError: Bad state: The client closed with pending request "streamListen".

Command
flutter --no-color run --machine --track-widget-creation --device-id=emulator-5554 --start-paused --dart-define=flutter.inspector.structuredErrors=true lib\main.dart

Steps to Reproduce

...
...
...

Logs
StateError: Bad state: The client closed with pending request "streamListen".
#0      new Client.withoutJson.<anonymous closure> (package:json_rpc_2/src/client.dart:70:24)
#1      _rootRun (dart:async/zone.dart:1182:47)
#2      _CustomZone.run (dart:async/zone.dart:1093:19)
#3      _FutureListener.handleWhenComplete (dart:async/future_impl.dart:167:18)
#4      Future._propagateToListeners.handleWhenCompleteCallback (dart:async/future_impl.dart:666:39)
#5      Future._propagateToListeners (dart:async/future_impl.dart:722:37)
#6      Future._propagateToListeners (dart:async/future_impl.dart:621:9)
#7      Future._completeWithValue (dart:async/future_impl.dart:529:5)
#8      Future._asyncCompleteWithValue.<anonymous closure> (dart:async/future_impl.dart:567:7)
#9      _rootRun (dart:async/zone.dart:1190:13)
#10     _CustomZone.run (dart:async/zone.dart:1093:19)
#11     _CustomZone.runGuarded (dart:async/zone.dart:997:7)
#12     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1037:23)
#13     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#14     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#15     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#16     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:169:5)

[✓] Flutter (Channel master, 1.22.0-10.0.pre.264, on Microsoft Windows [Version 10.0.19041.508], locale en-US)
    • Flutter version 1.22.0-10.0.pre.264 at E:\Application\android-sdk-windows\flutter
    • Framework revision 3e6863ab2b (9 hours ago), 2020-09-18 21:05:27 -0700
    • Engine revision 2abe69c608
    • Dart version 2.10.0 (build 2.10.0-136.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at E:/Application/android-sdk-windows
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = E:/Application/android-sdk-windows
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[✓] Android Studio (version 4.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 49.0.2
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] Connected device (1 available)
    • Android SDK built for x86 64 (mobile) • emulator-5554 • android-x64 • Android 10 (API 29) (emulator)

• No issues found!

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: beta
Creation framework version: fabeb2a16f1d008ab8230f450c49141d35669798

same problem here. trying to integrate flutter in existing project. after such issues no way gonna consider flutter for development. too much time getting wasted on resolving issues of cross platform frameworks .

same program when I ran "flutter run", even the application was just created

Possibly related to https://github.com/flutter/flutter/issues/61274

@jonahwilliams said https://github.com/flutter/flutter/issues/61274 was fixed by reverting a "dds" change, though I'm not sure which one.
\cc @bkonyi
@LichKing255 there's been some work in this area since you filed it, would you mind updating on the master channel and seeing if you can reproduce it?

Sorry for the forward duplicate, there's already conversation happening at https://github.com/flutter/flutter/issues/66514
Duplicate of https://github.com/flutter/flutter/issues/66514.


Update templates based on other packages

[x] pysatCDAAC
[x] pysatMadrigal
[x] pysatMissions
[x] pysatModels
[x] pysatNASA
[ ] pysatSeasons
[x] pysatSpaceWeather
[x] pysat

@jklenzing pysatSeasons is not going to be "ahead" of this repository, should we consider this task completed?


Roll Engine from 2663be837081 to 368fe27d23dc (6 revisions)

https://github.com/flutter/engine/compare/2663be837081...368fe27d23dc
2020-05-27<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from 5yPfm... to Id94G... (flutter/engine#18630)
2020-05-27<EMAIL_ADDRESS>Roll Dart SDK from 253022eb1b27 to 6489a0c68df8 (4 revisions) (flutter/engine#18626)
2020-05-27<EMAIL_ADDRESS>Fix copy/paste in the Linux embedding (flutter/engine#18624)
2020-05-27<EMAIL_ADDRESS>Roll Skia from fd9745eee98f to 3f479f3da7f8 (1 revision) (flutter/engine#18623)
2020-05-27<EMAIL_ADDRESS>created a linter that runs clang-tidy (flutter/engine#18607)
2020-05-27<EMAIL_ADDRESS>Roll Dart SDK from e007545c42fd to 253022eb1b27 (5 revisions) (flutter/engine#18622)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from 5yPfmohgVONA to Id94GHHR_eiq
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


fix merging null multi value in partial upsert

bugfix
When persisting merged result of previous and new row in PartialUpsertHandler, if a null multivalue field is populated using _defaultNullValue value it is persisted as a primitive and fails due to typecasting when updating dictionary and adding new record. Null multivalue fields should be persisted as null.
Observed errors:
java.lang.ClassCastException: class java.lang.String cannot be cast to class [Ljava.lang.Object; (java.lang.String and [Ljava.lang.Object; are in module java.base of loader 'bootstrap')
at org.apache.pinot.segment.local.indexsegment.mutable.MutableSegmentImpl.updateDictionary(MutableSegmentImpl.java:590)
at org.apache.pinot.segment.local.indexsegment.mutable.MutableSegmentImpl.index(MutableSegmentImpl.java:492)
at org.apache.pinot.core.data.manager.realtime.LLRealtimeSegmentDataManager.processStreamEvents(LLRealtimeSegmentDataManager.java:592)
at org.apache.pinot.core.data.manager.realtime.LLRealtimeSegmentDataManager.consumeLoop(LLRealtimeSegmentDataManager.java:443)
at org.apache.pinot.core.data.manager.realtime.LLRealtimeSegmentDataManager$PartitionConsumer.run(LLRealtimeSegmentDataManager.java:679)
at java.base/java.lang.Thread.run(Thread.java:829)

java.lang.ClassCastException: class java.lang.Double cannot be cast to class [Ljava.lang.Object; (java.lang.Double and [Ljava.lang.Object; are in module java.base of loader 'bootstrap')
at org.apache.pinot.segment.local.indexsegment.mutable.MutableSegmentImpl.addNewRow(MutableSegmentImpl.java:720)
at org.apache.pinot.segment.local.indexsegment.mutable.MutableSegmentImpl.index(MutableSegmentImpl.java:493)
at org.apache.pinot.core.data.manager.realtime.LLRealtimeSegmentDataManager.processStreamEvents(LLRealtimeSegmentDataManager.java:592)
at org.apache.pinot.core.data.manager.realtime.LLRealtimeSegmentDataManager.consumeLoop(LLRealtimeSegmentDataManager.java:443)
at org.apache.pinot.core.data.manager.realtime.LLRealtimeSegmentDataManager$PartitionConsumer.run(LLRealtimeSegmentDataManager.java:679)
at java.base/java.lang.Thread.run(Thread.java:829)

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

Project coverage is 62.20%. Comparing base (59551e4) to head (2b1a750).
Report is 382 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #13031      +/-   ##
============================================
+ Coverage     61.75%   62.20%   +0.44%
+ Complexity      207      198       -9
============================================
  Files          2436     2502      +66
  Lines        133233   136672    +3439
  Branches      20636    21165     +529
============================================
+ Hits          82274    85010    +2736
- Misses        44911    45377     +466
- Partials       6048     6285     +237

Flag
Coverage Δ

custom-integration1
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration1
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration2
0.00% <0.00%> (ø)

java-11
62.13% <100.00%> (+0.42%)
:arrow_up:

java-21
62.08% <100.00%> (+0.45%)
:arrow_up:

skip-bytebuffers-false
62.15% <100.00%> (+0.41%)
:arrow_up:

skip-bytebuffers-true
62.05% <100.00%> (+34.33%)
:arrow_up:

temurin
62.20% <100.00%> (+0.44%)
:arrow_up:

unittests
62.19% <100.00%> (+0.44%)
:arrow_up:

unittests1
46.76% <0.00%> (-0.13%)
:arrow_down:

unittests2
27.94% <100.00%> (+0.21%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


When the "Data2Seq" module dataset and code will be available?

Hi, this is a great work. When the "Data2Seq" module dataset and ode will be available? Do you have a timetable now？
Thanks so much!

We're trying to release it by this weekend. Please stay tuned.


⚠ GitLab Server has degraded performance

In 6e228b3, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6839 ms

Resolved: GitLab Server performance has improved in 2c388df.


my issue

why my issue has been deleted, i can't see my issue????

I closed the issue.
You can find it here: https://github.com/artisaninweb/laravel-soap/issues/145


ADS update installer gets into a fight with Panda Endpoint Protection

Azure Data Studio Version: 1.8.0

On Windows 10 Pro with Panda Endpoint Protection installed, whenever an update (initiated from within the application) is initiated from Azure Data Studio, the Panda service seems to max out a CPU core. The ADS installer then doesn't progress unless I kill or suspend all Panda processes. It's happened every time I've updated ADS, but didn't happen on the initial install.
I know this isn't a traditional bug report as it's a software interaction thing, but hopefully having it highlighted and on a backlog will be useful to other users experiencing the same issues as uptake of ADS increases. I'm emailing Panda's technical support with a link to this issue as I imagine it might need some collab to resolve.

Thanks for the report @logicalextreme. Given that as you implied there's not much we can do if external processes are affecting performance I'll be closing this - although if Panda comes back with information on something ADS is doing that is wrong or unexpected we can look into it.


Child components are being rendered even when not active

I was testing case with 30 tabs. Each has a non-trivial content (e.g. component with SVG content, connected to Redux store). Manipulation with content of active tab was very slow, because all 30 components are actually live even when they aren't visible. This makes the usage of these tabs for anything but simple text/image somewhat slow. The (original) 'react-tabs' do not have this behavior.

Hi @petrdamborsky,
Thanks for your suggestion!
I totally agree with you. We shouldn't render inactive tabs. I'll work on this feature on the weekend.

That would be great @maslianok. I've been looking for an alternative to Material-UI Tabs, simply because MUI doesn't lazily render tabs. With this feature, my team could adopt this library and benefit from your excellent responsive design work. :+1:

Starting from v.0.7.0 you can use getContent instead of content to render only active tab.
Before:
{
  title: 'Title',
  content: 'Content',
}

After:
{
  title: 'Title',
  getContent: () => 'Content',
}

Sorry for such a long answer :(

Sweet great work!

Hey @maslianok,  If I lazily load components as my tab content, will those components be unmounted when the current tab is switched? I want to know if switch tabs preserves component state. This is important for nested tabs.
I.e.
2 levels of tabs, a parent and nested.
Parent Tabs = A | B | C where A is selected
Nested Tabs = A1 | A2 | A3
Are the following assertions true?

When A is rendered, if A1 is the default tab, it will be rendered
If A2 is selected, it will be lazily rendered.
If the user then selects B, then goes back to A to re-visit A1 and A2, as A1 and A2 have already been lazily rendered, they will still have the same component state as before

Thanks

No, the component unmounts from the DOM tree when it becomes inactive. So the state gets lost.
Actually, we can either render all tab panels or render only active panel and remove others :)
I see two options for you to avoid this:

Use content for stateful panel with nested tabs:

{
  title: 'Statefull panel with nested tabs',
  content: 'This panel always present in the DOM',
},
{
  title: 'Stateless panel',
  getContent: () => 'This panel renders only when active',
},

Keep state in the parent component

{
  title: 'Statefull panel with nested tabs',
  getContent: () => <Compnent
    onChangeNestedTab={this.onChangeNestedTab}
    activeTab={this.state.activeTab}
  >,
},

Actually, the more I think about it, the current design is most correct. +1
I think option 2 makes the most sense for me. It is good design to wrap the component in a stateful container.
Thanks @maslianok


Implement low frequency adversarial perturbation strategy

This is a black-box attack search strategy allowing to reduce the number of queries to a model by 2-4x. It is based on discrete cosine transform (DCT). This selection strategy can be combined with existing black-box attacks, e.g., the paper proves effectiveness with NES and boundary attack. This should be implemented as a wrapper if possible, to allow the usage with existing black-box attacks in ART.
Paper link: https://arxiv.org/abs/1809.08758

@beat-buesser Hello there, can you kindly assign this issue to me? I'll get started on it.

@rish-16 Thank you! And let us know here if you have questions or need support.

Will do. Thank you

Hi @rish-16 Are you still working on this issue?

@ririnicolae Our implementation in ART I forked may be useful.
https://github.com/kztakemoto/SimBA-Keras

@kztakemoto Would you like to pull request to ART your implementation of this attack?

@ririnicolae thanks. I pull requested. https://github.com/IBM/adversarial-robustness-toolbox/pull/469


Add another IOI variant to the ppips

Maybe it is worth to add also the BRKH_INT at this point, as I suppose those are ppips as well.

Maybe it is worth to add also the BRKH_INT at this point, as I suppose those are ppips as well.

Agreed.

Make format


Fix tc hack

This set of commits fixes Issue #318.
The code depends on some commits that are in upstream Aya, but not in our fork.  So we either need to pull those commits into our fork (https://github.com/redhat-et/aya/pull/3) or move to upstream Aya, and update the main Cargo.toml before it can be merged.

To see the specific tasks where the Asana app for GitHub is being used, see below:

https://app.asana.com/0/0/1203910656683151

/lgtm


chore: use better conversion technique to go from TransactionContext to TransactionInfo

Follows: https://github.com/paradigmxyz/reth/pull/11114
need to add block number here can do it two ways:

ignore block number
add a with_block_number helper

I think you can bump revm-inspectors to make this pass?

yeah i think it was the inverse direction that would be possible!
sorry got busy otherwise would have pinged here with that comment


'WebClient' object has no attribute 'assistant_threads_setStatus'

Hi Team,
Here are the Slack packages we've been using so far:

slack-bolt==1.18.1
slack_sdk==3.27.2

This week, we learned about Agents & Assistants, which is quite new to us. However, the documentation has been very helpful in getting us started.
We’re currently exploring the assistant.threads.setStatus feature. We're trying to implement the following code:
client = WebClient(token=bot_token)
client.assistant_threads_setStatus ---> Throwing a 'no attribute' error.

Are we missing something here?

Hi @MadhabaPatra, thanks for writing in! The method is available since v3.33.0. Once you upgrade slack-sdk to the version or newer, you can use it. Also, checking this template https://github.com/slack-samples/bolt-python-assistant-template should be helpful for you!

Thanks @seratch , I was focused on tracking the Bolt library package and missed the slack-sdk version update. I'll upgrade to the recommended version and check out the template you've shared. Appreciate the help!


Move Workflows directory

Troubleshooting why Workflows weren't integrated into PR check ... hopefully moving it out of the ISSUES_TEMPLATE directory and directly into the .githubs directory addresses this issue.

 I see the msg "Some checks were not successful". That looks like workflow is integrated with PR. So  if I pull this code after merged, then whenever try to publish PR, I will see the same thing in the PR.
And it's okay the test is failed, or is it required to pass the test then be able to merge?

 I see the msg "Some checks were not successful". That looks like workflow is integrated with PR. So if I pull this code after merged, then whenever try to publish PR, I will see the same thing in the PR.
And it's okay the test is failed, or is it required to pass the test then be able to merge?

 It looks like the workflow is integrated with this PR; so yes, after this PR gets merged into main, the same check will be run on every PR. It it is my philosophy that any checks included as part of the PR process ought to be passed before merging in. Although I don't think it's an active requirement now, I think I can actually enforce that within the repo settings. For now, as long as a PR has an approval, it should be mergable.


replace grunt-autoprefixer with gruntpostcss + autoprefixer

yes thanks, i see. Done.

Any change in dist files?

dist files are unchanged (as expected)

Then it should be OK.

Modulo the indentation, LGTM.
We'll need to reshrinkwrap after merging this.

@bassjobsen: can you fetch and rebase and also take care of @cvrebert's comments?

i will try it this weekend. I'm not sure if i do understand the rebase thing.

Well i tried / try to do the rebase.
i'm not sure i did it right.
I also found some diffs in dist now:
    4893,4894c4893
    <        -o-transition:         transform .3s ease-out, -o-transform .3s ease-out;
    <           transition: -webkit-transform .3s ease-out;
    ---
    >        -o-transition:      -o-transform .3s ease-out;
    4896d4894
    <           transition:         transform .3s ease-out, -webkit-transform .3s ease-out, -o-transform .3s ease-out;
    5289,5290c5287
    <          -o-transition:         transform .6s ease-in-out, -o-transform .6s ease-in-out;
    <             transition: -webkit-transform .6s ease-in-out;
    ---
    >          -o-transition:      -o-transform .6s ease-in-out;
    5292d5288
    <             transition:         transform .6s ease-in-out, -webkit-transform .6s ease-in-out, -o-transform .6s ease-in-out;

i do not understand why 31 files are change now? @XhmikosR should i better create a new PR now?

You didn't do the rebase properly, which would account for the extra unwanted commits and file changes.
I would make a new one, it's easier and faster.

I'd agree that it's easier and faster to make a new one @bassjobsen. I'd recommend this guide on how to rebase https://help.github.com/articles/about-git-rebase/ if you aren't familiar with it / don't do it often

git rebase -i upstream/v4-dev

@XhmikosR for the sake of being complete, git fetch upstream/v4-dev && git rebase -i upstream/v4-dev

okay thanks, i will create a new PR soon

That is not in order to create a new PR...

@XhmikosR yes i understand, i will repeat my changes on the latest version and add it to a new PR

@XhmikosR maybe it is rebased now?

@bassjobsen: yeah it's good now. But you should squash some of the patches while rebasing.

commit 17f7b5c4788aedaa161d9cbfd79cc12f02b835e1 includes unwanted changes (from improper rebase)

Good catch, indeed it does.

@wolfy1339 yes i see, but https://github.com/bassjobsen/bootstrap/commit/52b5bd5ce99e2c125b16939ea6e0101381756b55 repairs this?

 You would need to rewrite the git history of your branch to only have your changes.
When you do a rebase, you need to do a force push by using git push -f, so the git history is replaced and nothing unwanted happens.
Since you aren't comfortable with rebase, I would recommend that you create a new pull request and not that it superseded this one.

Leave this open. I'm rebasing and merging this myself.

@XhmikosR This is why I always recommend qualifying requests for rebases with "if you are comfortable+familiar with rebasing..."

Rebased. Running Travis build: https://travis-ci.org/twbs/bootstrap/builds/90641415

Build failed. Retrying now that #17652 has been merged.
https://travis-ci.org/twbs/bootstrap/builds/90642551

There seems to be a problem installing<EMAIL_ADDRESS>under Node.js v4.2.2 &<EMAIL_ADDRESS>

Forcing npm@3 fixes that problem, but now some unrelated random RVM problem has raised its head.

Filed https://github.com/travis-ci/travis-ci/issues/5092. Travis is missing curl for some reason, which causes the RVM problem.

Why did you need to upgrade Ruby anyway? I use 2.2 in another project of mine without any issues.

env:
  matrix:
    - RUBY_VERSION=2.2

before_install:
  - rvm install $RUBY_VERSION
  - rvm use $RUBY_VERSION --fuzzy
  - export GEMDIR=$(rvm gemdir)

@cvrebert: try this, it should work.

Also, we shouldn't force npm@3 either. I mean, I'm using 2.x locally just fine. Did you try to clear Travis cache just in case?

@cvrebert: this seems to pass https://github.com/twbs/bootstrap/commit/9b50133387b688a058844693ce13c9a0d0ed08ad
https://travis-ci.org/twbs/bootstrap/builds/90675687

Didn't really need to upgrade Ruby. I was just grasping at straws when debugging the RVM issue.

this seems to pass 9b50133
https://travis-ci.org/twbs/bootstrap/builds/90675687

@XhmikosR Looks good, you should open a PR for that.

No, haven't tried clearing the Travis cache.

Shouldn't I just merge that then?

Nicely done: 7b19dfc06fc9c4f5d964c0e9a596843e9a4b4f33

For whatever reason, the RVM error is no longer occurring.

Purged the caches and building one more time to verify whether npm@3 is truly necessary:
https://travis-ci.org/twbs/bootstrap/builds/90687640

Build failed. @XhmikosR Yup, npm@3 is needed to install from an npm@3 shrinkwrap, which is reasonable. I'm guessing you don't see it locally since you're probably not installing via shrinkwrap.

@cvrebert: we don't use a v3.x shrinkwrap, do we?

@XhmikosR npm v3 is the latest stable major version of npm and seems to the preinstalled version for Node.js v5. And it's compatible with Node.js v4. Since the shrinkwrap is only used internally, I don't see much reason to use an old version.
(My branch switches to npm v3 shrinkwrap since I have Node.js v5 on my local box.)

Rebased and merged as 50c43bc55a3576636cf83eae2a918fa6dd231b75. Thanks @bassjobsen!
@XhmikosR Feel free to argue about 3236a4df6971836111b768302b7d464fb457a0e0 tomorrow :smile:

The thing is, I don't want to use v5.0.0 yet... This broke my branch and I will need to use npm 3.x to rebase it.


[Feature]: Add support to run in Azure Playwright Testing

🚀 Feature Request
Run dotnet tests using Azure Playwright Testing
Example
Run dotnet tests using Azure Playwright Testing
Motivation
Better than using DevOps Agents

Folding into https://github.com/microsoft/playwright-testing-service/issues/16. There are examples for .NET:

https://github.com/microsoft/playwright-testing-service/tree/main/samples/.NET


unbound-control forward fails in captive portal with "failed to prime trust anchor -- could not fetch DNSKEY rrset . DNSKEY IN"

Describe the bug
I'm on a network with a captive portal whose DHCP server has handed out a resolver, say <IP_ADDRESS>.  This network's resolver server returns NXDOMAIN for . DNSKEY queries.
When I try to ask Unbound to temporarily forward all queries to <IP_ADDRESS>, in order to get through the captive portal, it fails all queries with SERVFAIL and logs:
Feb  2 16:40:42 singbulli unbound: [15648:0] info: generate keytag query _ta-4f66. NULL IN
Feb  2 16:40:42 singbulli unbound: [15648:0] info: failed to prime trust anchor -- could not fetch DNSKEY rrset . DNSKEY IN

To reproduce
Steps to reproduce the behavior:

connect to a network with a captive portal that returns NXDOMAIN for . DNSKEY queries
configure unbound with any combination of:

unbound-control forward <IP_ADDRESS>
unbound-control forward_add +i . <IP_ADDRESS>
unbound-control stub_add +i . <IP_ADDRESS>
[edited 2024-02-05T21:08Z to correctly reflect the commands I tried; without . the forward_and and stub_add commands obviously fail immediately]

unbound-control flush_zone . and query unbound for example.com A

Expected behavior
Unbound returns the answer given by <IP_ADDRESS>.  I don't care if it's true according to DNSSEC; I care about getting access to the internet.
System:

Unbound version: 1.16.3
OS: NetBSD 10.0_RC3
unbound -V output:

Version 1.16.3

Configure line:
Linked libs: pluggable-libevent 2.1.12-stable (it uses kqueue), OpenSSL 3.0.12 24 Oct 2023
Linked modules: dns64 respip validator iterator

BSD licensed, see LICENSE in source package for details.
Report bugs to<EMAIL_ADDRESS>or https://github.com/NLnetLabs/unbound/issues

Additional information
I understand dnssec-trigger is supposed to handle this, but (a) my past experience with dnssec-trigger on Ubuntu was that it never once worked for me, and (b) by reading the code, I can't figure out what else it might be doing besides essentially what I already tried with forward_add +i . <captiveportalresolver>: https://github.com/NLnetLabs/dnssec-trigger/blob/c8493ef4daaee7a8bed57f6722f92ff39f838937/dnssec-trigger-script.in#L279-L301.  If dnssec-trigger does work in this scenario after all, I would like to know what magic it is doing to convince Unbound to answer these queries without SERVFAIL or REFUSED.

The correct command is to add a forward zone, since it is a resolver that you are asking unbound to send to. Since you want dnssec to be ignored, the '+i' option is nice. That would mean the forward_add line. For both the forward_add line and stub_add line you need to add the zone that is modified, '.', the root zone is what you want here.
But since you already have a trust anchor for the root, the insecure option, as I see reading the code that tries to add it, does nothing. It notices that already a trust anchor exists for the name, and then does not modify the existing entry. So it did not turn off DNSSEC. Because the root trust anchor is configured in config; the '+i' option would work for domain names that fall underneath the configured trust anchor, and thus have no entry yet.
So the forward command is good to have it forward to that resolver, if the network was not captive it would be nicer to go to the authorities on its own without that forward, and then dnssec would work, but since it is captive that is also what dnssec-trigger does.
What dnssec-trigger does for forced insecure, is disable unbound's resolution, temporarily, and set the system to talk to the upstream resolver straight up. What could work for you, is to disable the root trust anchor, by commenting out the 'auto-trust-anchor-file' statement in the config of unbound. That removes the DNSSEC validation from Unbound, and then setting unbound to forward to <IP_ADDRESS> would likely work.

Perhaps put the trust anchor line in a separate text file that is included from the main config. Then change that one line and reload unbound. That would make a sort-of quick change.
Another option could be to turn on permissive mode, but since that does actually perform validation and especially a lot of DNSSEC failure retries because that is not working, it would likely be slow if it works.
With the file change the change would be persistent. With reload_keep_cache the cache is kept while reloading.

The magic in dnssec-trigger is to avoid contacting broken resolvers. Instead use different DNSSEC-aware resolvers. If you need to pass captive portal, it might help to forward only captive portal domain to local resolver. forward_add +i onboard-wifi.com <captiveportalresolver>, where onboard-wifi.com is domain used for captive portal. If it responds with NXDOMAIN to . domain, then it is definitely broken.
If you can send just basic queries to captive portal and other queries to more decent public resolver, it should work better. You would have to otherwise disable validator module in whole unbound to be able to interact with such server. Better to use it only for bare minimum you can. +i or domain-insecure: are your friends.


easyXDM RPC, Cannot use 'in' operator to search for 'method' in..

Hi!
I inject easyxdm to the page and use it on various blogging software on the web for my widget purposes. It's strange but on some platforms I get the following error:
Cannot use 'in' operator to search for 'method' in [{ url:..., method:..., data:...  }]
so I wonder why and probably you know where to start to determine the problem.

Log (pak is consumer hostname):
easyXDM.debug.js:935 pak - 23:51:52.053: easyXDM present on 'http://pak/mt/MTOS-5.1b2-en/mt.cgi?__mode=view&_type=entry&blog_id=2&id=1
easyXDM.debug.js:980 pak - 23:51:52.059:{Private}: Settings namespace to 'Example.transport'
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.059:easyXDM.Rpc: constructor
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.059:{Private}: preparing transport stack
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.060:{Private}: resolved url 'http://api.example.com/easyxdm/cors/index.html'
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.060:{Private}: selecting protocol: 1
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.061:easyXDM.stack.PostMessageTransport: constructor
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.061:easyXDM.stack.QueueBehavior: constructor
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.061:easyXDM.stack.RpcBehavior: init
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.062:easyXDM.stack.RpcBehavior: creating stubs
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.062:easyXDM.stack.RpcBehavior: creating method request
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.062:easyXDM.stack.PostMessageTransport: init
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.063:{Private}: adding listener message
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.063:{Private}: creating frame: http://api.example.com/easyxdm/cors/index.html?xdm_e=http%3A%2F%2Fpak&xdm_c=default4163&xdm_p=1
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.064:{Private}: HAS_NAME_PROPERTY_BUG: false
easyXDM.debug.js:935 api.example.com - 23:51:52.482: easyXDM present on 'http://api.example.com/easyxdm/cors/index.html?xdm_e=http%3A%2F%2Fpak&xdm_c=default4163&xdm_p=1
easyXDM.debug.js:935 api.example.com - 23:51:52.490: native JSON found
easyXDM.debug.js:935 api.example.com - 23:51:52.492: easyXDM.Rpc: constructor
easyXDM.debug.js:935 api.example.com - 23:51:52.492: {Private}: preparing transport stack
easyXDM.debug.js:935 api.example.com - 23:51:52.493: {Private}: using parameters from query
easyXDM.debug.js:935 api.example.com - 23:51:52.493: easyXDM.stack.PostMessageTransport: constructor
easyXDM.debug.js:935 api.example.com - 23:51:52.493: easyXDM.stack.QueueBehavior: constructor
easyXDM.debug.js:935 api.example.com - 23:51:52.494: easyXDM.stack.RpcBehavior: init
easyXDM.debug.js:980 api.example.com - 23:51:52.494:{Private}: firing dom_onReady
easyXDM.debug.js:980 api.example.com - 23:51:52.495:... deferred messages ...
easyXDM.debug.js:980 api.example.com - 23:51:52.495:easyXDM.Rpc: constructor
easyXDM.debug.js:980 api.example.com - 23:51:52.495:{Private}: preparing transport stack
easyXDM.debug.js:980 api.example.com - 23:51:52.495:{Private}: using parameters from query
easyXDM.debug.js:980 api.example.com - 23:51:52.495:easyXDM.stack.PostMessageTransport: constructor
easyXDM.debug.js:980 api.example.com - 23:51:52.496:easyXDM.stack.QueueBehavior: constructor
easyXDM.debug.js:980 api.example.com - 23:51:52.496:easyXDM.stack.RpcBehavior: init
easyXDM.debug.js:980 api.example.com - 23:51:52.496:... end of deferred messages ...
easyXDM.debug.js:980 api.example.com - 23:51:52.496:easyXDM.stack.PostMessageTransport: init
easyXDM.debug.js:980 api.example.com - 23:51:52.497:{Private}: adding listener message
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.499:easyXDM.stack.PostMessageTransport: firing onReady
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.500:{Private}: removing listener message
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.500:{Private}: adding listener message
easyXDM.debug.js:980 api.example.com - 23:51:52.500:easyXDM.stack.QueueBehavior: removing myself from the stack
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.502:easyXDM.stack.QueueBehavior: removing myself from the stack
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.635:easyXDM.stack.RpcBehavior: executing method request
easyXDM.debug.js:980 api.example.com - 23:51:52.645:easyXDM.stack.PostMessageTransport: received message 'default4163 {"method":"request","params":"[{\"url\":\"http://api.example.com/services/rest/0.0/\",\"method\":\"POST\",\"data\":{\"method\":\"preferences\",\"api_key\":\"1234\",\"format\":\"json\",\"browser\":\"safari\",\"interface\":\"movabletype51\",\"deployment\":\"easyXDM\",\"viewportWidth\":1417,\"viewportHeight\":175},\"timeout\":5000}]","id":1,"jsonrpc":"2.0"}' from http://pak
easyXDM.debug.js:980 api.example.com - 23:51:52.646:easyXDM.stack.RpcBehavior: received request to execute method request using callback id 1
easyXDM.debug.js:980 api.example.com - 23:51:52.646:easyXDM.stack.RpcBehavior: requested to execute procedure request
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.659:easyXDM.stack.PostMessageTransport: received message 'default4163 {"id":1,"error":{"code":-32099,"message":"Cannot use 'in' operator to search for 'method' in [{\"url\":\"http://api.example.com/services/rest/0.0/\",\"method\":\"POST\",\"data\":{\"method\":\"preferences\",\"api_key\":\"1234\",\"format\":\"json\",\"browser\":\"safari\",\"interface\":\"movabletype51\",\"deployment\":\"easyXDM\",\"viewportWidth\":1417,\"viewportHeight\":175},\"timeout\":5000}]"},"jsonrpc":"2.0"}' from http://api.example.com
easyXDM.debug.js:980 pak:Example.transport - 23:51:52.659:easyXDM.stack.RpcBehavior: received return value destined to callback with id 1

Please use the debug version and note the offending line number - this should be in the RpcBehavior.

Simple test on WordPress:
window.JSON.stringify({data:[{hello:"world"}]});
"{"data":[{"hello":"world"}]}"
on MovableType:
window.JSON.stringify({data:[{hello:"world"}]});
"{"data":"[{"hello":"world"}]"}"
Meanwhile:
window.JSON
JSON
parse: function parse() { [native code] }
stringify: function stringify() { [native code] }
proto: Object
Seems like native.. How's that possible...

Problem found with help of following topic: http://stackoverflow.com/questions/710586/json-stringify-bizarreness
delete Array.prototype.toJSON helped to solve the problem. Seems like some 3rd party library defined those method or something like that happend.

However probably it would be a good idea to pass an object not within array, AFAIK easyXDM rewraps the object to array anyway, thus we may prevent any problems with older Prototype (library) based code.

When you first said that what should have been an array was a string, I
knew the problem :) This happens when the JSON implementation is broken, as
it is when you are using older Prototype version (<1.7?).
easyXDM does ist's best to verify that the JSON implementation is working -
normally it should also work with the broken Prototype, but this failed for
some reason.
I'm not going to change easyXDM for this, we need this to be an array in
order to pass multiple values :)
yvind Sean Kinsey
<EMAIL_ADDRESS>http://kinsey.no/blog/index.php/about/
2012/2/2 Andrei Mikhailov <
<EMAIL_ADDRESS>

However probably it would be a good idea to pass an object not within
array, AFAIK easyXDM rewraps the object to array anyway, thus we may
prevent any problems with older Prototype (library) based code.

Reply to this email directly or view it on GitHub:
https://github.com/oyvindkinsey/easyXDM/issues/150#issuecomment-3774787

Yeah makes sense if passing multiple values, so #wontfix. I didn't know that toJSON used by JSON implementation if this method available that's why I was quite frustrated and created an issue.


chore: test case of isKanji

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.55 :tada:

Comparison is base (a2c8901) 14.58% compared to head (533e4ae) 15.14%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
+ Coverage   14.58%   15.14%   +0.55%
==========================================
  Files           5        5
  Lines         912      918       +6
==========================================
+ Hits          133      139       +6
  Misses        777      777
  Partials        2        2

Impacted Files
Coverage Δ

qr_segment_advanced.go
7.92% <100.00%> (+2.81%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add badges and connect with us and license section

What type of PR is this?:
enhancement
What this PR does / why we need it:
This PR enhances the repository by adding badges to showcase important information, connecting with us for further inquiries or collaborations, and including a license section for clarity on permitted use.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation:

How about add discord-md-badge at the top of README.md?


Can I set Showdown to recognize internal anchors links?

Hi,
My markdown has some local links, but showdown shows them as plain text like:
[Declare options](#Declare current operation options)
I can't find a mention about this in the documentation, only basic links are discussed :|

Immediately resolute

Immediately resolute

Fragments should be permissible:
CommonMark allows them: https://spec.commonmark.org/0.30/#example-500
Github Markdown allows them: https://github.github.com/gfm/#example-509
DF is ambiguous: https://daringfireball.net/projects/markdown/syntax#link

However, a space in the fragment name is not permissible. To get your example above to work the markdown needs to change from this:
[Declare options](#Declare current operation options)

to this:
[Declare options](#Declare%20current%20operation%20options)
# or
[Declare options](<#Declare current operation options>)


ApplicationHealthStatus and ApplicationSyncStatus are assuming that an empty health/sync string means it is healthy/synced

What does this PR do / why we need it:
ApplicationHealthStatus and ApplicationSyncStatus are assuming that an empty health/sync string means it is healthy/synced.
These functions should only return nil if the application is Healty and Synced, respectively.

/test all

FYI @amitkrout

/retest

/retest

/retest

/retest

/approve
/lgtm

@jgwest Test fails. PTAL
 === RUN   TestGitOpsService/Validate_machine_config_updates
I0630 16:01:05.832997    8680 request.go:621] Throttling request took 8.542981442s, request: GET:https://api.ci-op-w74ssqdz-02c74.origin-ci-int-aws.dev.rhcloud.com:6443/apis/project.openshift.io/v1?timeout=32s
    gitops_service_test.go:310: application 'image' health is ''
    gitops_service_test.go:310: application 'image' health is ''
    gitops_service_test.go:310: application 'image' health is ''
    gitops_service_test.go:310: application 'image' health is ''

Test Validate_machine_config_updates does not skip in CI.

/lgtm


Revert "Add tests. (#5)"

This reverts commit 130dabe05c9880341cfeb6a1222422301f29a15e.

The tests fail on Windows, and there is currently no good way to let the surrounding system about it.
Once that is in place I will reland it.
TBR.

ok


Nail down format of the spreadsheet

So we know what to ask for

Proposed columns:

id
artist
title
song_art
media_url
rdio
spotify
itunes
asin

Placeholder spreadsheet with SWL14 data: https://docs.google.com/spreadsheets/d/1zvJ-RLAOd8UujpwN_1v6QyvZDassZ1UcqigmlPl2nxA/edit#gid=0

download_url
facebook_link?
bandcamp_link?
twitter_link?
other_link?
sxsw_link?

Final format: https://docs.google.com/spreadsheets/d/1eJ4qkOXaLUvPaRvjjeplgWD7hZCnxW_By3FzrfF3wss/edit#gid=0


MudTextField  Date format Chinese language error , MudDataGrid Unable to set Chinese language

Bug type
Component
Component name
MudTextField  MudDataGrid
What happened?
1：MudTextField  Date format Chinese language error
2：MudDataGrid Unable to set Chinese language

Expected behavior
I hope to set up the Chinese language correctly
Reproduction link
https://try.mudblazor.com/snippet/mkwdaZuhPkjItnxY
Reproduction steps
https://try.mudblazor.com/snippet/mkwdaZuhPkjItnxY
Relevant log output
No response
Version (bug)
6.9.0
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

MudDataGridPager has a RowsPerPageString property, you can use it.

MudDataGridPager has a RowsPerPageString property, you can use it.

Thank you. May I ask how to set the filter for cultural information

Currently I don't see any official solution other than making your own filter with 😣 (I wish FilterOperator was not 'constant' or could support string conversion better.)

MudDataGridPager has a RowsPerPageString property, you can use it.

Thank you. May I ask how to set the filter for cultural information

But FilterOperator is not constant, you can make your own localization for it
https://mudblazor.com/features/localization#translation-keys

Also forgot to add, that the Culture property in PropertyColumn here is only for tostring formatting, to localization, if you want to change the content of filters, please follow the guide from the link above


Missing requirements.txt

No requirements.txt found :(

Good call.
Added the requirements document in commit  d83d53cdc016441c9a66b3ba8e39e49daa1497c6.


Fill in the blank with placeholder test

Challenge Fill in the Blank with Placeholder Text has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.133 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

Hello World

CatPhotoApp

Kitty ispum dolor sit amet

@pallujain
Thank you for reporting this issue.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "Help" button on the challenge on freeCodeCamp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification. Thank you, and happy coding.


fix(dedicated): override width for large modals

ref: MANAGER-3751

Discussed altogether, will refactor and reopen 😉


Copy short URL to clipboard

I would love to have a way to get short URLs onto my clipboard through Alfred. I volunteer as a social media host for our streaming services, and I need to quickly give our congregation links to verses in the Bible app. I have made some small changes in my copy of your workflow to make this work, but it's a bit of a hack. You might be interested in making it work in a more rigorous, less buggy way.

I added a Copy to Clipboard node with the content "{var:ref_url}" that works when the alt modifier is added.

I also changed the URL template at yvs/core.py:25 to use a shorter version of the URL that still dependably works.
# The template used to build the URL for a Bible reference
# REF_URL_TEMPLATE = 'https://www.bible.com/bible/{ref}'
REF_URL_TEMPLATE = 'bible.com/{ref}' # Added by Alex Mercado for shorter URLs on 20 Dec 2020

Of course, my URL template is surely less stable than yours, but it helps so much when I have character limits. Perhaps you'd want to make this a switch that users can set. I'd love to hear your thoughts. Thank you for your excellent contribution to the Kingdom!
Aside: I use a different keyword "bible", instead of your standard "yv", just because my brain has a hard time invoking "yv" when I need it.

@merchako
I can't make any promises, but I will consider adding a preference for enabling a shorter reference URL in the future! For the time being, though, I see no issue with your solution from a stability perspective. I've actually helped a handful of my users tweak the workflow to fit their needs.

Well that didn't take long to break! The short REF_URL_TEMPLATE = 'bible.com/{ref}' I used doesn't work anymore because it's missing the protocol https://. When I add that back,  resulting in https://bible.com/{ref}), it works fine.
What caused the change? I'm not sure. I upgraded to Big Sur recently. Maybe it was that, or maybe it was a consequent update to Alfred.
I would love to have the even shorter version back because those few characters make a different in social media posts. Let me know if you come across a workaround.

Well that didn't take long to break! The short REF_URL_TEMPLATE = 'bible.com/{ref}' I used doesn't work anymore because it's missing the protocol https://. When I add that back,  resulting in https://bible.com/{ref}), it works fine.
What caused the change? I'm not sure. I upgraded to Big Sur recently. Maybe it was that, or maybe it was a consequent update to Alfred.
I would love to have the even shorter version back because those few characters make a different in social media posts. Let me know if you come across a workaround.

@merchako You might be able to "cheat" a little by prepending the protocol in the Open URL object that's part of the workflow. See my attached screenshot.

@merchako You might be able to "cheat" a little by prepending the protocol in the Open URL object that's part of the workflow. See my attached screenshot.

Thanks for the tip! This helped me open it in the browser correctly, but to make Copy Content to Clipboard work, I also had to change a line in copy_ref.py.
# Retrieves HTML for of the chapter to which the reference belongs
def get_chapter_html(ref):

    chapter_uid = get_ref_chapter_uid(ref)
    # url = core.get_ref_url(ref_uid=chapter_uid)
    # Updated by Alex Mercado on 6 Jan 2020
    url = "https://" + core.get_ref_url(ref_uid=chapter_uid)

Thanks for the tip! This helped me open it in the browser correctly, but to make Copy Content to Clipboard work, I also had to change a line in copy_ref.py.
# Retrieves HTML for of the chapter to which the reference belongs
def get_chapter_html(ref):

    chapter_uid = get_ref_chapter_uid(ref)
    # url = core.get_ref_url(ref_uid=chapter_uid)
    # Updated by Alex Mercado on 6 Jan 2020
    url = "https://" + core.get_ref_url(ref_uid=chapter_uid)

@merchako That's good to know, and glad it's working for you! I will be closing this issue since you found a suitable workaround.

@merchako That's good to know, and glad it's working for you! I will be closing this issue since you found a suitable workaround.


Question: Using git version for subprojects

What is the best way to add git versioning if I have multiple subprojects?
Say I have a library module and a sample module and they both should have the same version code and name. Currently I'm adding the plugin and settings to both app/build.gradle and library/build.gradle but I'm wondering if it could be done in the root gradle through sub projects.

Unfortunately, at present, I don't see a lot of help for this use case in https://docs.gradle.org/current/userguide/plugins.html#sec:subprojects_plugins_dsl.
I typically version libraries and apps differently using prefix and onlyIn so that a library I'm distributing doesn't move versions when the app changes. So in the case I would have different androidGitVersion blocks anyway. And if you're not distributing the library itself, then it's somewhat likely that its version code is not very important -- it's the versionCode/versionName of the app that will be seen by users.
But if you come up with anything that works better for your use case, let me know!


Improve result collection & logging

A few minor improvements to the log generator routines. If anything needs changing, let me know.

Refactors result collection to a Collector class
Adds errored, timeouts, killed keys to JSON log file

It may be necessary to discuss whether JSON logs should incude all error output by default, as it leads to a big increase in the log files (on my test project with errors, size went roughly from 65K to 650K due to the stack traces included in stderr property of each mutant data).

@aztech-dev Errors could likely be left out of the JSON output. Technically, they should always mean that a mutation was detected (by taking the nuclear option of breaking stuff!). They are in the text output as a means of tracking whether Humbug hits any snag and itself has an error while running mutations through the grinder.

Allright, that's what I ended up doing, keeping only the first line of stderr to keep a smaller logfile (300K instead of 650K now).

@aztech-dev All of them is fine for JSON as a log format. They are more curtailed for text inspection which is the readable target. Let us know how your webapp turns out!

Sure will ! Still a pretty rough beta, but should go live pretty soon :)

No other comments - will merge when ready :).

Thanks @aztech-dev !

You're welcome :) Got another one coming up actually...


Add README steps to upgrade Timesketch DB

This PR adds a section in the Timesketch charts readme file explaining how to perform Database upgrades for the kubernetes deployment when required by the new Timesketch release.

Failing test:
Deleting outdated charts
Linting chart "timesketch => (version: \"1.0.5\", path: \"charts/timesketch\")"
Checking chart "timesketch => (version: \"1.0.5\", path: \"charts/timesketch\")" for a version bump...
Old chart version: 1.0.5
Error: failed linting charts: failed processing charts
New chart version: 1.0.5

@wajihyassine Given I only updated documentation and not the charts, should this still be bumped?

Failing test:
Deleting outdated charts
Linting chart "timesketch => (version: \"1.0.5\", path: \"charts/timesketch\")"
Checking chart "timesketch => (version: \"1.0.5\", path: \"charts/timesketch\")" for a version bump...
Old chart version: 1.0.5
Error: failed linting charts: failed processing charts
New chart version: 1.0.5

@wajihyassine Given I only updated documentation and not the charts, should this still be bumped?

So I thought that was the case, but issue template needs to be updated it doesn't look like the chart-testing knows how to differentiate certain files while including it into the packaged chart still https://github.com/helm/chart-testing/issues/530. Need to look into either updating the issue template or finding a way to update exclude *.md files
For this case can you bump the version number and then run helm dependency update and then should be good to merge.


Relative paths sometimes not included as links in integrated terminal

VSCode Version:  1.10.2
OS Version: Mac OSX (10.11.6)

Steps to Reproduce:

Open VS Code integrated terminal
Echo a relative path (eg. echo "src/app.js")

Description:
Let's say you have a build system that outputs a stack trace on a build error or failed test. If this was run from the integrated terminal, the file paths in the output should be included as clickable links which can be clicked to open the file in VSCode.
However, this only works if the path is either an absolute path (e.g. ~/work/project/src/app.js) or a relative path that begins with "./".
Many build systems and tools output file paths relative to the directory the command was run, without the leading "./", Karma for example. So the output would be something like: src/app.js, not ./src/app.js. The first is not matched by VSCode as an openable path, but the second is.
I believe that what is determined to be a path comes from the Regex in "terminalLinkHandler.ts"
Is there a reason why relative paths of this form (e.g. no leading "./") are not included?

The reason was because until 1.11 links aren't validated so there would be heaps of false positives. Performance was also a factor but it shouldn't impact that much really now that I think about it.
Now that links are validated maybe we should be more lenient when linkifying.

I'll assign this to April but if someone wants to do a PR before then you're welcome. You will need to:

Modify the regex to allow paths that are not qualified with ~/, ./, etc.
Make sure that the new path matcher does not conflict with previously matched links and does not create links with adjacent characters. Eg. we want to avoid this: ~/link

Nice. I'll write a PR...

That would be awesome @markwpearce :+1:

Fixed in https://github.com/Microsoft/vscode/pull/22602


add e2e tests

Signed-off-by: anandrkskd<EMAIL_ADDRESS>/kind tests
What does this PR do / why we need it:
This PR adds e2e tests for odo.
Which issue(s) this PR fixes:
Fixes #5752
Fixes #5750
PR acceptance criteria:

[ ] Unit test

[x] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

/restest

/approve

/lgtm

/retest

/override prow/v4.10-integration-e2e
IBM tests pass.


[SPIKE / Audit Version] - Persist record when the commit page is submitted

Context
https://trello.com/c/VI0mtlSP/1019-3-days-persist-record-when-the-commit-page-is-submitted
Changes proposed in this pull request
Same proof of concept done here https://github.com/DFE-Digital/register-trainee-teachers/pull/534 (with some caveats).
Notes
This PR demonstrates a proof of concept for enabling an "undo" functionality allowing providers to undo their form changes if they decide to cancel.
Two approaches were considered:

Stashing form changes into Redis - https://github.com/DFE-Digital/register-trainee-teachers/pull/534
Using the audit gem to try and "rollback" changes (this PR)

Review

Head to a trainee's personal details page. They should have a state which allows their information to still be changed.
Attempt to change anything, confirm the new changes are shown on the confirm page. Click cancel and assert your original changes are shown on the record page
Do the above but commit the changes and assert the changes are now saved

What needs to change
Compared to the Redis approach in the PR above, we don't need to change as much but there are more UX cases to deal with using the audited gem.
Since we track any change to a model, we have to keep track of what the last set of audited changes where when a trainee is edited via one of the forms (along with the action).
If the trainee edits are cancelled then we need to search through the trainee's audits and find the last stashed changes in order to "undo" them, a functionality provided by the audit gem. This effectively "rolls back" the object to its previous revision.
Caveats
Where to stash the changes?
As we need to keep track of the last set of audited changes, we need to put these somewhere. At the moment, they're stored in the session. This quickly blows up with a cookie size limit since some audited changes (particularly the first initial record) would track all the properties of the model. We would need to move sessions to the db to combat this.
Associated data
There's a situation where if you try to add an associated data (like a nationality), and undo it - it works fine the first time around and also saves fine if you commit the changes.
If you remove that nationality (after saving it) but then try to undo the deletion, it won't work. This could be because the audits aren't configured properly but it seems that at this point there's no audit of the deletion of that record in the audit trail so we can't undo it.

 [3 days] Persist record when the commit page is submitted


Facebook Android SDK Update

Bindings for the Facebook SDK for Android is several versions behind the latest version. Concern from developers that the requirement to resubmit applications for review may cause issues with using an older version of the SDK. Already had a discussion with Jonathan on getting these updated.
Current version: 4.26.0
https://www.nuget.org/packages/Xamarin.Facebook.Android/
Latest Facebook SDK: 4.32
https://developers.facebook.com/docs/android/

You may want to consider Xamarin.Essentials as it includes Filesystem helper and Secure storage apis
https://github.com/xamarin/Essentials/wiki/Feature-Support

@ivzhang  We are working on this ATM and should have a PR request to implement the update shortly

@newky2k we are blocked due to this.  Please provide an update

We are working on a branch here: https://github.com/xamarin/FacebookComponents/tree/android-updates?files=1

@caseyniemann it looks this has been merged now from this PR in the facebookcomponents https://github.com/xamarin/FacebookComponents/pull/73


cancel button moves

i see this on a regular basis

here's how it should look (the cancel button not the word "submitting")

https://github.com/twosigma/beaker-notebook/pull/1781


Update msgpack-c to 1.4.2

Update to latest msgpack-c release v1.4.2
Run by command git subtree pull --prefix=MessagePack/Resources/C-Sources/msgpack-c https://github.com/msgpack/msgpack-c cpp-1.4.2 --squash
Close #27.

Any reason not to merge?

Merge or not?

I changed to merge 2.0.0 instead (calling a cmake project from the autotools since this is the major change in that version)


fixes #131

Please don't merge without passing through a reviewer first

@creativepsyco I thought that this was a minor PR, so I merged it. Will wait for reviews from the next PR

#202


use of internal package not allowed

Hi
I used this command to install cameradr but got an error:
Command:
go get github.com/ullaakut/cameradar
Error:
go-projects/src/github.com/ullaakut/nmap/nmap.go:13:2: use of internal package not allowed
GO Version:
go version go1.10.4 linux/amd64

My solution was to update go.
https://github.com/golang/go/wiki/Ubuntu
You have to add a ppa if you're running an older version of Ubuntu.

My solution was to update go.
https://github.com/golang/go/wiki/Ubuntu
You have to add a ppa if you're running an older version of Ubuntu.

I updated go to go1.13 and now I get this error:
go: extracting github.com/ullaakut/cameradar v3.0.1+incompatible
go: finding github.com/andelf/go-curl latest
go: finding github.com/Ullaakut/nmap latest
go: extracting github.com/Ullaakut/nmap v0.0.0-20190907125850-63196347c2d4
# github.com/andelf/go-curl
go/pkg/mod/github.com/andelf/go-curl@v0.0.0-20180313164100-9d81ad32de98/easy.go:408:22: identifier "_Ctype_struct_curl_slist" may conflict with identifiers generated by cgo`

Hi @majidmc2 !
How are you building Cameradar? What command are you running to get this output? Thanks.

I'm new to the go ecosystem so this all may be obvious but I just tried on a new machine and ran into both above issues.

Not following the directions correctly and leaving GO111MODULE empty:

$ export GO111MODULE=
$ go get -u -v github.com/ullaakut/cameradar
github.com/ullaakut/cameradar (download)
created GOPATH=/home/jed/go; see 'go help gopath'
github.com/ullaakut/disgo (download)
github.com/fatih/color (download)
github.com/ullaakut/go-curl (download)
github.com/ullaakut/nmap (download)
github.com/Ullaakut/nmap (download)
go/src/github.com/ullaakut/nmap/nmap.go:13:2: use of internal package github.com/Ullaakut/nmap/internal/slices not allowed

I noticed that there were 2 separate [U|u]llaakut repos:
I edited to go/src/github.com/ullaakut/nmap/nmap.go to use the lowercase nmap slices package and it worked. Wildly speculating it's trying to use Ullaakut as 'internal' but since Linux is case sensitive it works out to be a different path and not "Internal".
This fixed it:
sed -i "s/Ullaakut/ullaakut/g" `go env GOPATH`/src/github.com/ullaakut/nmap/nmap.go

$ go get -u -v github.com/ullaakut/cameradar
github.com/ullaakut/cameradar (download)
github.com/ullaakut/disgo (download)
github.com/fatih/color (download)
github.com/ullaakut/go-curl (download)
github.com/ullaakut/nmap (download)
github.com/Ullaakut/nmap (download)
github.com/ullaakut/cameradar

Then this works fine:
$ cd `go env GOPATH`/src/github.com/ullaakut/cameradar/cmd/cameradar
$ go install

With export GO111MODULE=on I get the text that @majidmc2 got ending in:
go/pkg/mod/github.com/andelf/go-curl@v0.0.0-20180313164100-9d81ad32de98/easy.go:408:22: identifier "_Ctype_struct_curl_slist" may conflict with identifiers generated by cgo

However I don't know enough about go to understand what folder structure it setup:
$ ls `go env GOPATH`
pkg

There is no src directory and annoyingly it sets the permissions to 444:
$ rm ~/go
....
rm: cannot remove<EMAIL_ADDRESS>Permission denied
$ ls -lah<EMAIL_ADDRESS>
-r--r--r-- 1 jed jed 3.3K Sep 26 18:49<EMAIL_ADDRESS>

The issue is that go-curl no longer compiles on go1.13 apparently, will need to follow that issue.

Alright, as it seems that the maintainer of go-curl is no longer maintaining the project, I will fix this issue in my fork and make a PR to update Cameradar accordingly.
Thanks for your patience.

For me everything works fine when testing it on my machine, so I tested it on a blank environment as well by doing the following:
docker run -it -v (pwd)/cameradar:/cameradar golang:1.13.1-alpine3.10 sh
This puts cameradar into an empty docker container that has golang 1.13.1 (with go modules enabled by default, on an alpine OS).
I then ran cd /cameradar/cmd/cameradar && go install and it correctly imported go: downloading github.com/ullaakut/go-curl v0.0.0-20190525093431-597e157bbffd and not<EMAIL_ADDRESS>So, not sure why for you guys the module that go chose to import was wrong 🤔 @jed-frey , what version of go did you use? Thanks

Closing for inactivity, feel free to reopen @jed-frey if you are back on this :)


*ngIf on a mat-step breaks the stepper if the expression becomes false from a later step

Bug, feature request, or proposal:
bug
What is the expected behavior?
I can remove a step at any time without the stepper crashing
What is the current behavior?
If an earlier step is removed while a later step is active, the current step disappears. If you manually click a still visible previous step, we get an error Error: Cannot set property 'interacted' of undefined
What are the steps to reproduce?
https://stackblitz.com/edit/angular-material2-issue-fm21qe

toggling step 2 from step 1 works fine.
toggling step 2 from step 3 breaks step 3
after toggling from step 3, navigation to step 1 emits an error

What is the use-case or motivation for changing an existing behavior?
We can remove later steps using *ngIf without issue, so one would presume they could remove a step at any time. If it's decided that removing a previous step doesn't make sense, this should at least fail gracefully.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
6.2.0
Is there anything else we should know?

@crisbeto is this resolved by https://github.com/angular/material2/pull/9132? Or is it a different issue?

There's #11813 which handles this one.


BUG: Inconsistent results in signal.firls between aarch & x86

Describe your issue.
The following snippet shows different results when executed in ARM & X86 for wheels downloaded from PyPi, building SciPy from a conda installation works as expected.
Some of the very small values are different between both architectures. The small magnitude of the differing results (e-14) doesn't really indicate a problem per se in this specific case, but having the conda installation in aarch64 matching exactly x86 results while pypi doesn't may indicate that some build flags regarding FP ops precision are different for PyPi wheels and can cause problems in other routines. As a side note, CuPy results match x86 results 1:1 too.
(Thanks to @leofang for figuring this out by comparing conda & PyPi installs!)
Reproducing Code Example
import scipy as scp

N = 11  # number of taps in the filter
a = 0.1  # width of the transition band
# design a halfband symmetric low-pass filter
h = scp.signal.firls(N, [0, a, 0.5-a, 0.5], [1, 1, 0, 0], fs=1.0)
print(h)

Error message
aarch64 output
[ 9.20115457e-03  2.13215371e-14 -5.71460483e-02 -6.05624404e-14
  2.98061198e-01  5.00000000e-01  2.98061198e-01 -6.05624404e-14
 -5.71460483e-02  2.13215371e-14  9.20115457e-03]

x86 output:
[ 9.20115457e-03  1.81577277e-14 -5.71460483e-02 -5.15707625e-14
  2.98061198e-01  5.00000000e-01  2.98061198e-01 -5.15707625e-14
 -5.71460483e-02  1.81577277e-14  9.20115457e-03]

SciPy/NumPy/Python version and system information
aarch64
1.12.0 1.26.4 sys.version_info(major=3, minor=10, micro=11, releaselevel='final', serial=0)
Build Dependencies:
  blas:
    detection method: pkgconfig
    found: true
    include directory: /usr/local/include
    lib directory: /usr/local/lib
    name: openblas
    openblas configuration: USE_64BITINT= DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP= MAX_THREADS=80
    pc file directory: /usr/local/lib/pkgconfig
    version: 0.3.21.dev
  lapack:
    detection method: pkgconfig
    found: true
    include directory: /usr/local/include
    lib directory: /usr/local/lib
    name: openblas
    openblas configuration: USE_64BITINT= DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP= MAX_THREADS=80
    pc file directory: /usr/local/lib/pkgconfig
    version: 0.3.21.dev
  pybind11:
    detection method: config-tool
    include directory: unknown
    name: pybind11
    version: 2.11.1
Compilers:
  c:
    commands: cc
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  c++:
    commands: c++
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  cython:
    commands: cython
    linker: cython
    name: cython
    version: 3.0.8
  fortran:
    commands: gfortran
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  pythran:
    include directory: ../../tmp/pip-build-env-otvjb90d/overlay/lib/python3.10/site-packages/pythran
    version: 0.15.0
Machine Information:
  build:
    cpu: aarch64
    endian: little
    family: aarch64
    system: linux
  cross-compiled: false
  host:
    cpu: aarch64
    endian: little
    family: aarch64
    system: linux
Python Information:
  path: /opt/python/cp310-cp310/bin/python
  version: '3.10'

x86
1.12.0 1.26.4 sys.version_info(major=3, minor=10, micro=11, releaselevel='final', serial=0)
Build Dependencies:
  blas:
    detection method: pkgconfig
    found: true
    include directory: /usr/local/include
    lib directory: /usr/local/lib
    name: openblas
    openblas configuration: USE_64BITINT= DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP= HASWELL MAX_THREADS=2
    pc file directory: /usr/local/lib/pkgconfig
    version: 0.3.21.dev
  lapack:
    detection method: pkgconfig
    found: true
    include directory: /usr/local/include
    lib directory: /usr/local/lib
    name: openblas
    openblas configuration: USE_64BITINT= DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP= HASWELL MAX_THREADS=2
    pc file directory: /usr/local/lib/pkgconfig
    version: 0.3.21.dev
  pybind11:
    detection method: config-tool
    include directory: unknown
    name: pybind11
    version: 2.11.1
Compilers:
  c:
    commands: cc
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  c++:
    commands: c++
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  cython:
    commands: cython
    linker: cython
    name: cython
    version: 3.0.8
  fortran:
    commands: gfortran
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  pythran:
    include directory: ../../tmp/pip-build-env-oj5r1eje/overlay/lib/python3.10/site-packages/pythran
    version: 0.15.0
Machine Information:
  build:
    cpu: x86_64
    endian: little
    family: x86_64
    system: linux
  cross-compiled: false
  host:
    cpu: x86_64
    endian: little
    family: x86_64
    system: linux
Python Information:
  path: /opt/python/cp310-cp310/bin/python
  version: '3.10'

I wonder if this is related to https://github.com/scipy/scipy/issues/18533
Could you check if switching the lapack driver matters here?


added new style for button start hacking

hacktoberfest like button style added

Hey! could you fix the conflicts so i can merge :) Otherwise great addition

Hey! could you fix the conflicts so i can merge :) Otherwise great addition

conflicts resolved

@richie-south conflicts have been . Fixed .
Review nd merge:smile:


Position of percentage (%) character is wrong

Hi,
Impressive software! One remark: I noticed that in several places the percentage character (%) is placed before the number. This should be placed behind the number.

Hi, Impressive software! One remark: I noticed that in several places the percentage character (%) is placed before the number. This should be placed behind the number.

Hello,
First of all, thank you for your thoughtfulness.
I will examine the relevant parts.
You can be sure that I will make a correction in the next patch.
Best regards.

👍Sent from my iPadOn 5 Jun 2024, at 21:25, Eray @.***> wrote:
This issue has been resolved with version v24.09.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


gitlab: project list not loading if other webhooks exist

When at least one of the projects of the user contains a webhook which is not related to the REANA instance the project list doesn't show up, the cause: an error 500 thrown by /api/gitlab/projects (most probably happening here).
How to reproduce?
Create a random webhook on one of your projects in GitLab going to https://gitlab.cern.ch/<username>/<project-name>/hooks/:

Even if you have already finished the synchronisation with GitLab you won't see the list of projects and you will see the "Connect" button again, if you open the network traffic in the browser you will see a 500 on the call to /api/gitlab/projects.

Closing manually as closes keyword doesn't work on branches different than master, see docs.


[Tests] fix skip usage

Conditionally skipping tests doesn't work this way, I think?

I just rebased to tweak the commit messages; two tests are failing.

I just rebased to tweak the commit messages;

Got them, thanks!

two tests are failing.

Hopefully this does the trick: We downgrade eslint-import-resolver-typescript on Node.js < 8: https://github.com/import-js/eslint-plugin-import/blob/fd85369fc219c40862d2afcfb11ad053387ba967/tests/dep-time-travel.sh#L32-L35
1.0.2 doesn't resolve .js extensions (I presume) and this test needs a resolved import: https://github.com/import-js/eslint-plugin-import/blob/fd85369fc219c40862d2afcfb11ad053387ba967/src/rules/no-extraneous-dependencies.js#L176


PuTTY Default Parameters documentation

I couldn't find any reference to the PuTTY Default Parameters in the documentation.  Is that covered somewhere else?  Is this where you can set specific Putty settings for your sessions?  i.e. logging, appearance, colours, etc...

+1
And I cannot use a GNS3 with a -wt "Windows Title" !

Documentation for logging of superputty is no win the wiki, for anything putty specific you should check putty's documentation. If you configure a session and save any preferences, you can load those preferences by selecting them in the SuperPutty sessions editor where you can select the PuTTY session profile.
PuTTY documentation is here: https://www.chiark.greenend.org.uk/~sgtatham/putty/docs.html


Isolated declarations: support optional class methods

Example class
export class A {
  m1?(): void;
  m2(): void {}
}

Before the fix:
export declare class A {
  m1(): void;
}

with the fix:
export declare class A {
  m1?(): void;
  m2(): void;
}

@Dunqing one more ;)


The %all% tag does not include "_" in tags and only uses spaces

Bug description
When using the %all% tag, tags that normally use an underscore to utilize multiple words in one tag instead user white space. This is a problem because multiple tags are also seperated by white space.
Steps to reproduce
Open grabber and put the %all% tag in your file name.
Add some links to the downloader
Download the files
The files will not utilize and underscores int heir file name
Expected behavior
Tags which use an underscore to separate words should also do so in file names. Most (All?) file systems support underscores, so this shouldn't pose a problem.
System information
Windows 10
Grabber 7.1.1 (x64)

Hello, have you tried disabling the "replace spaces by underscores" setting?

You could also use the underscores modifier in your filename (%all:underscores%).

I updated the docs to mention this modifier:
https://bionus.github.io/imgbrd-grabber/docs/filename.html#underscores-bool

Good god I can't believe I missed that. Thanks for pointing it out and HUGE thanks for updating the docs.


feat: Food item page

To see the UI for this, can test by using this URL: http://localhost:3000/foodItem?barcode=749826126487
Currently hardcoded but if the response follows the structure of macros variable then it should display correctly (needs to be tested obviously)

Users can go into negative numbers for number of servings. This should not be possible


TS3.7 regression: crash checking JS assigning properties in array literal

TypeScript Version:  3.7.0-beta
Code
function fun() {
  var arr = [];
  var mean = [
    arr[0].prop[0] = 1,
  ];
}

Expected behavior:
Pass.
Actual behavior:
Crash:
node_modules/typescript/lib/tsc.js:1808
            throw e;
            ^

Error: Debug Failure. Invalid cast. The supplied value [object Object] did not pass the test 'isBindableStaticAccessExpression'.
    at Object.cast (node_modules/typescript/lib/tsc.js:1268:25)
    at bindSpecialPropertyAssignment (node_modules/typescript/lib/tsc.js:25709:53)
    at bindWorker (node_modules/typescript/lib/tsc.js:25329:29)
    at bind (node_modules/typescript/lib/tsc.js:25221:13)
    at bindEach (node_modules/typescript/lib/tsc.js:23997:21)
    at visitNodes (node_modules/typescript/lib/tsc.js:14079:24)
    at Object.forEachChild (node_modules/typescript/lib/tsc.js:14237:24)
    at bindEachChild (node_modules/typescript/lib/tsc.js:24005:16)
    at bindChildrenWorker (node_modules/typescript/lib/tsc.js:24102:21)
    at bindChildren (node_modules/typescript/lib/tsc.js:23975:17)

(also repros in 3.8 nightly)

Even smaller:
var arr = [];
arr[0].prop[0] = 1;

I think it's a mismatch between getAssignmentDeclarationKind, the initial classifier for assignments-as-declarations, and subsequent code. I think it is too-permissively classifying arr[0].prop[0] as a potential declaration.

Yep. getAssignmentDeclarationKind proves that arr[0].prop isBindableStaticAccessExpression but bindSpecialPropertyAssignment assumes that arr[0].prop[0] isBindableStaticAccessExpression .


Website/server down

I've tried pinging your website and it's giving me a 404 error

@thehoppa sorry for the inconvenience. I updated the domain to https://api.pinyin.pepe.asia.
I just added the old domain so now it should work like before.

Hey thanks. Now that the new domain is up I've updated my stuff so it pings
the new domain. Thanks for keeping it going!
On Sat, Aug 10, 2019 at 1:58 PM Pepe Becker<EMAIL_ADDRESS>wrote:

Closed #6 https://github.com/pepebecker/pinyin-rest/issues/6.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pepebecker/pinyin-rest/issues/6?email_source=notifications&email_token=AH26DLYBWKZGCYMWACEAH7LQD4MWTA5CNFSM4IKXR7GKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOS7SFQTA#event-2548324428,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH26DL76V6JQR2QOBXJD55LQD4MWTANCNFSM4IKXR7GA
.


instance type 'c5ad.12xlarge' not supported

Environment:

AWS ParallelCluster 2.9.1

Bug description and how to reproduce:
$ pcluster create -c /home/centos/.parallelcluster/config-2.9.1-v1 HR-11
Beginning cluster creation for cluster: HR-11
ERROR: The configuration parameter 'instance_type' generated the following errors:
The instance type 'c5ad.12xlarge' used for the 'instance_type' parameter is not supported by AWS ParallelCluster.
Why are c5ad.12xlarge instances not supported?
Kind regards,
Richard

Hi @rems0 thanks for opening this.
The message means that we didn't tested yet ParallelCluster with this instance type.
In any case it should work, if you are using slurm.
You could try it by setting sanity_check = false in the [global] section.
I'm marking it as feature enhancement, to officially add support for the c5ad.* family.

This instance is going to be enabled in the upcoming release

Instance is now enabled on AWS ParallelCluster version 2.10.0


[WIP] UI: Initial 'hover' component

This is an initial, basic hover component that will follow the cursor. For now, we can show the following:

Syntax tokens (per token provider)

And then extend it with other information, as we build out support

Errors / diagnostics
QuickInfo
Links

Is there a difference between this and a potential Tooltip component?

Good question!

Edit: I guess a tooltip would typically be anchored to some other UI component.

Right, the <Hover /> will be based on the cursor position in the buffer (and some logic to determine whether it needs to be above/below depending on the position)
One other difference is I think there are a few types of data we'll be showing in the hover:

QuickInfo (text / markdown)
Errors / diagnostics
Definition location

I believe a <ToolTip /> will actually be simpler - just a string to show as the tooltip with the border. The interesting part is detecting the hover and deciding when to show it. It's possible we might be able to reuse some pieces.
I also need to check the theme colors - if VSCode themes specify different colors for tooltips & quickinfo (for borders, background, etc)


no configure script in smtpf.git

Cloned https://github.com/SirWumpus/smtpf.git per instructions, but there doesn't appear to be a confgure script in the repository.

Workaround:

install autoconf
autoconf -f
./configure ...


BUG-1: Fixed the bug to make the table look pretty.

Fixed the bug that I was asked to fix.

Since you're already working on this code, please capitalize the second "t" in the Time tracking column heading.


Somethings about llama-index evaluation

Could you tell me which dataset in your llama-index evaluation or how to do this, thanks

The dataset is the paper 《Llama 2: Open Foundation and Fine-Tuned Chat Models》.
If you  want to evaluate, you need to modify the "SentenceTransformerRerank" class in llama_index [https://www.llamaindex.ai/blog/boosting-rag-picking-the-best-embedding-reranker-models-42d079022e83?ref=jina-ai-gmbh.ghost.io], you can use "LLMReranker" class in FlagEmbedding [https://github.com/FlagOpen/FlagEmbedding/blob/master/FlagEmbedding/flag_reranker.py] to modify it.

Another question: Do you use Anthropic LLM to generate Question-Context Pairs?

We don't use Anthropic LLM to generate Question-Context Pairs.

So, how do you generate Question-Context Pairs?

We use GPT-3.5 to generate Question-Context Pairs.

How many Question-Context pairs did you generate?

Following llama_index, we use Pages from start to 36 for the experiment which excludes table of contents, references, and appendix.


server goes off and does not stay 24/7

Hi when ever i host i can't make server stay online if im online...plz help

use this commands frm next
"sudo apt update
apt list --upgradable
apt list -a
sudo apt install software-properties-common -y
sudo add-apt-repository ppa:deadsnakes/ppa
sudo apt install python3-pip python3.10-dev python3.10-venv
git clone https://github.com/imayushsaini/Bombsquad-Ballistica-Modded-Server
cd Bombsquad-Ballistica-Modded-Server
chmod 777 bombsquad_server
chmod 777 dist/bombsquad_headless
tmux
./bombsquad_server"
to turn off a server use "pkill tmux"

Thank you very much it really worked and i apologise for the trouble and time you took to assist me. i am truely honored

From: YOYO @.>
Sent: 16 October 2023 02:25
To: imayushsaini/Bombsquad-Ballistica-Modded-Server @.>
Cc: FANAIRL @.>; Author @.>
Subject: Re: [imayushsaini/Bombsquad-Ballistica-Modded-Server] server goes off and does not stay 24/7 (Issue #88)
use this commands frm next
"sudo apt update
apt list --upgradable
apt list -a
sudo apt install software-properties-common -y
sudo add-apt-repository ppa:deadsnakes/ppa
sudo apt install python3-pip python3.10-dev python3.10-venv
git clone https://github.com/imayushsaini/Bombsquad-Ballistica-Modded-Server
cd Bombsquad-Ballistica-Modded-Server
chmod 777 bombsquad_server
chmod 777 dist/bombsquad_headless
tmux
./bombsquad_server"
to turn off a server use "pkill tmux"
—
Reply to this email directly, view it on GitHubhttps://github.com/imayushsaini/Bombsquad-Ballistica-Modded-Server/issues/88#issuecomment-1763554344, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A6QXSFSZLQIKRJVBSOIRYP3X7R5IPAVCNFSM6AAAAAA42IQG32VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRTGU2TIMZUGQ.
You are receiving this because you authored the thread.Message ID: @.***>

Np you're welcome


Improve MAVEN build Performance

Parallel builds in Maven 3 Maven 3.x has the capability to perform parallel builds.
According to Maven parallel test, we can run tests in parallel.
=====================
If there are any inappropriate modifications in this PR, please give me a reply and I will change them.

Let me try just using parallel builds....

For better developer experience instead of Maven Parallel builds I'd recommend using mvnd https://github.com/apache/maven-mvnd instead.

Closing due to lack of activity over the past year.  You're free to update the pull request to the current master branch and re-submit if you'd like for us to take another look at this change.


WIP Update for breaking huge class

Fixes #33
I have broken down the single class in greenberry.py and included 8 more files with a single class each.
I have run some tests to check the implementation at by my end it is working OK. Let me know if you receive any issue or you think the code needs to be updated.

All seems fine on first look. I'll test it a bit at the weekend to see if everything works as expected.

@hassaanseeker can you follow up with a contributor tutorial? it'll be nice!

@hassaanseeker can you follow up with a contributor tutorial? it'll be nice!

Yes sure will work on that.


feat: Add directions to access the UI from within HA

This edit adds directions to add the UI to HA using Lovelace, so that the UI is accessible from within HA itself.
Result:

@blhoward2 I would add those instructions under a new section ##

@blhoward2 I would add those instructions under a new section ##

Ok

Ok

@robertsLando How's that?

@robertsLando How's that?

@blhoward2 LGTM, could you fix lint issues? npm run lint-fix

@blhoward2 LGTM, could you fix lint issues? npm run lint-fix

I can tonight. I was doing this through the GitHub website so I need to pull it down to do so.

I can tonight. I was doing this through the GitHub website so I need to pull it down to do so.

@zwave-js-bot fix lint

@zwave-js-bot fix lint

@AlCalzone Arggg

Permission to blhoward2/zwavejs2mqtt.git denied to zwave-js-bot

:cry:

@AlCalzone Arggg

Permission to blhoward2/zwavejs2mqtt.git denied to zwave-js-bot

:cry:

@zwave-js-bot fix lint

@zwave-js-bot fix lint

@AlCalzone Arggg

Permission to blhoward2/zwavejs2mqtt.git denied to zwave-js-bot

😢

Is there a way for me to give it permission for the future?

@AlCalzone Arggg

Permission to blhoward2/zwavejs2mqtt.git denied to zwave-js-bot

😢

Is there a way for me to give it permission for the future?

Interesting... Did you check the option "allow maintainers to make changes"?

Interesting... Did you check the option "allow maintainers to make changes"?

Think so. I’ve never unchecked it.

Think so. I’ve never unchecked it.

Hmm... think I'll have to investigate. I haven't actually been able to test this on a foreign branch yet.

Hmm... think I'll have to investigate. I haven't actually been able to test this on a foreign branch yet.

@AlCalzone I just forgot to add the bot to my repo. Now it has done something but still not working

@AlCalzone I just forgot to add the bot to my repo. Now it has done something but still not working

Well... it can only fix things that are auto-fixable. Could probably check that and not offer the fix if there isn't anything to do.

Well... it can only fix things that are auto-fixable. Could probably check that and not offer the fix if there isn't anything to do.

Should be fine now. At least I have checked that the bot is working. Thanks @AlCalzone and thanks @blhoward2 for the PR :)

Should be fine now. At least I have checked that the bot is working. Thanks @AlCalzone and thanks @blhoward2 for the PR :)

Want me to enable the fix lint hint comments for this repo too?

Want me to enable the fix lint hint comments for this repo too?

@zwave-js-bot fix lint

@zwave-js-bot fix lint

@AlCalzone Yep would be awesome :)

@AlCalzone Yep would be awesome :)

Ok, since you're using different linters than me, what's the exact check name and which part of the log indicates that there's a lint error?

Ok, since you're using different linters than me, what's the exact check name and which part of the log indicates that there's a lint error?

Pull Request Test Coverage Report for Build 478089113

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

Totals

Change from base Build 478065690:
0.0%

Covered Lines:
1938

Relevant Lines:
7996

💛  - Coveralls

Pull Request Test Coverage Report for Build 478089113

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

Totals

Change from base Build 478065690:
0.0%

Covered Lines:
1938

Relevant Lines:
7996

💛  - Coveralls

@robertsLando @AlCalzone Can anyone invoke the bot or just you guys? For little things I finding pulling/pushing tedious versus just editing in the website and having it automatically create a PR.

@robertsLando @AlCalzone Can anyone invoke the bot or just you guys? For little things I finding pulling/pushing tedious versus just editing in the website and having it automatically create a PR.

We both and the respective PR author can do that.

We both and the respective PR author can do that.

We both and the respective PR author can do that. So you should be able to.

Are there directions on how to use it?

We both and the respective PR author can do that. So you should be able to.

Are there directions on how to use it?

You comment the following if there is a lint error:
@zwave-js-bot fix lint
A comment explaining that should also appear now (if auto fixing is possible and the script does not fail).

You comment the following if there is a lint error:
@zwave-js-bot fix lint
A comment explaining that should also appear now (if auto fixing is possible and the script does not fail).

OK, so maybe I'm just a noob, but it took me a while to figure out how this works. It might be useful to include a link to https://www.home-assistant.io/lovelace/dashboards-and-views/#panel and describe that you need to create a new Lovelace Dashboard, and then go to its raw editor. Also, the instructions say to use the ip address but make no mention of the protocol, http or https. I was putting in just the ip address and it just kept loading hass in an infinite loop. The instructions should say either:
url: 'http://[PUT IP ADDRESS HERE]:8091/'

or
url: 'PUT THE FULL URL WITH THE PORT HERE'

or even better, with an example:
url: 'http://<IP_ADDRESS>:8091'  # replace with the url to your zwavejs2mqtt

OK, so maybe I'm just a noob, but it took me a while to figure out how this works. It might be useful to include a link to https://www.home-assistant.io/lovelace/dashboards-and-views/#panel and describe that you need to create a new Lovelace Dashboard, and then go to its raw editor. Also, the instructions say to use the ip address but make no mention of the protocol, http or https. I was putting in just the ip address and it just kept loading hass in an infinite loop. The instructions should say either:
url: 'http://[PUT IP ADDRESS HERE]:8091/'

or
url: 'PUT THE FULL URL WITH THE PORT HERE'

or even better, with an example:
url: 'http://<IP_ADDRESS>:8091'  # replace with the url to your zwavejs2mqtt

Ok, I can make those improvements.

Ok, I can make those improvements.


Create pagination for images MVP

Type

✨ Feature

Allow results per page to be changed by the user?

Add to #1002 when complete.

UI Specification

Refer to #1002
Need to follow up with Jenn to see if we can update the pagination pattern to improve usability.

Visual Design

Refer to Cardigan

@jennpb what was the conclusion please?