TS error with v7.0.0 - Type error: Module '"../../../node_modules/markdown-to-jsx/dist"' has no exported member 'MarkdownOptions'. Did you mean to use 'import MarkdownOptions from "../../../node_modules/markdown-to-jsx/dist"' instead?

The TS types haven't been published to definitelyTyped @types/markdown-to-jsx and upgrading to v7.0.0 crashes because of some incompatibility.
I didn't encounter this issue locally, but I did encounter it when deploying to Vercel:

I'll rollback to 6.11.2 until newer TS types are added.
"markdown-to-jsx": "7.0.0",
"@types/markdown-to-jsx": "6.11.2",

Do you happen to have third party types for this library installed?
I'm shipping them with the library itself now. Options is the proper name of the type I wrote.

Ah, didn't think about that, I was still using DefinitelyTyped, so they must have clashed together. I'll give it another try!

Okay, so indeed removing types from DefinitelyTyped helped about resolving the proper error locally.

But I can't use the types though, because they're not exported in the .ts file.
/**
 * markdown-to-jsx@6 is a fork of [simple-markdown v0.2.2](https://github.com/Khan/simple-markdown)
 * from Khan Academy. Thank you Khan devs for making such an awesome and extensible
 * parsing infra... without it, half of the optimizations here wouldn't be feasible. 🙏🏼
 */
import React from 'react';
declare type CreateElement = typeof React.createElement;
declare type HTMLTags = keyof JSX.IntrinsicElements;
/**
 * RequireAtLeastOne<{ ... }> <- only requires at least one key
 */
declare type RequireAtLeastOne<T, Keys extends keyof T = keyof T> = Pick<T, Exclude<keyof T, Keys>> & {
    [K in Keys]-?: Required<Pick<T, K>> & Partial<Pick<T, Exclude<Keys, K>>>;
}[Keys];
declare type Override = RequireAtLeastOne<{
    component: React.ComponentType<any>;
    props: Object;
}> | React.ComponentType<any>;
declare type Overrides = {
    [tag in HTMLTags]?: Override;
} & {
    [customComponent: string]: Override;
};
declare type Options = Partial<{
    /**
     * Ultimate control over the output of all rendered JSX.
     */
    createElement: (tag: Parameters<CreateElement>[0], props: React.Props<any>, ...children: React.ReactChild[]) => JSX.Element;
    /**
     * Disable the compiler's best-effort transcription of provided raw HTML
     * into JSX-equivalent. This is the functionality that prevents the need to
     * use `dangerouslySetInnerHTML` in React.
     */
    disableParsingRawHTML: boolean;
    /**
     * Forces the compiler to always output content with a block-level wrapper
     * (`<p>` or any block-level syntax your markdown already contains.)
     */
    forceBlock: boolean;
    /**
     * Forces the compiler to always output content with an inline wrapper (`<span>`)
     */
    forceInline: boolean;
    /**
     * Supply additional HTML entity: unicode replacement mappings.
     *
     * Pass only the inner part of the entity as the key,
     * e.g. `≤` -> `{ "le": "\u2264" }`
     *
     * By default
     * the following entites are replaced with their unicode equivalents:
     *
     * ```
     * &
     * '
     * >
     * <
     *
     * "
     * ```
     */
    namedCodesToUnicode: {
        [key: string]: string;
    };
    /**
     * Selectively control the output of particular HTML tags as they would be
     * emitted by the compiler.
     */
    overrides: Overrides;
    /**
     * Override normalization of non-URI-safe characters for use in generating
     * HTML IDs for anchor linking purposes.
     */
    slugify: (source: string) => string;
}>;
export declare function compiler(markdown: string, options?: Options): JSX.Element;
/**
 * A simple HOC for easy React use. Feed the markdown content as a direct child
 * and the rest is taken care of automatically.
 */
declare const Markdown: React.FC<{
    [key: string]: any;
    children: string;
    options?: Options;
}>;
export default Markdown;

Only compiler is found, because it's indeed exported. But you should use export declare everywhere, instead of just declare, so that we may import them.

Also, the type Override seems too restrictive:
declare type Override = RequireAtLeastOne<{
    component: React.ComponentType<any>;
    props: Object;
}> | React.ComponentType<any>;

I was using a component: 'a', and it worked well, but now I have a TS error because 'a' isn't assignable to ComponentType.

Full code:
https://github.com/UnlyEd/next-right-now/blob/286ea44c870bf7da1a5de2b5fd33fd7825d7a867/src/components/utils/Markdown.tsx#L29

a renders as an a, you don’t need to supply component unless you’re actually overriding the rendered output
Sent from my iPhone

On Sep 8, 2020, at 9:51 AM, Vadorequest<EMAIL_ADDRESS>wrote:

Also, the type Override seems too restrictive:
declare type Override = RequireAtLeastOne<{
component: React.ComponentType;
props: Object;
}> | React.ComponentType;
I was using a component: 'a', and it worked well, but now I have a TS error because 'a' isn't assignable to ComponentType.
Full code:
https://github.com/UnlyEd/next-right-now/blob/286ea44c870bf7da1a5de2b5fd33fd7825d7a867/src/components/utils/Markdown.tsx#L29
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Workaround for Options not being exported:
type FunctionalComponentPropsType<T> = T extends React.FC<infer X> ? X : never;
type Options = FunctionalComponentPropsType<typeof ReactMarkdown>['options'];

Fixed in 7.0.1 https://github.com/probablyup/markdown-to-jsx/releases/tag/7.0.1

Thank you, here is what I did to migrate to v7 with new TS types.
https://github.com/UnlyEd/next-right-now/commit/36105610541e48094c50933c0bffa50f95708d6b

Thank you, here is what I did to migrate to v7 with new TS types.
https://github.com/UnlyEd/next-right-now/commit/36105610541e48094c50933c0bffa50f95708d6b


Support namespace pings

Proposed change
Support the ability to "probe" a nexus (cluster of nex nodes) for a given namespace. So instead of pinging and getting a reply from all machines, instead get a reply from each workload in a given namespace.
The ping and ping filtering system could use a bit of a refactor anyway, especially to accommodate another filter like this.
Use case
Probe for "everything running in this namespace".
Contribution
No response

I think the best approach here might to change the response type of the WPING operation to be a set of matching workloads, and then allow the workload Id to be left off of the ping topic, which would essentially result in a namespace-wide workload ping.

It might still be useful to allow for pinging the running agents in a namespace, in addition to this proposed WPING, or ping for running workloads.

the WPING already exists to ping a specific workload ID or name. This issue only recommends supporting not specifying a workload filter on that wping.
This is a discovery feature, pinging the running agents is a health check feature


[MB-2416]  Copy changes - SM Create profile / Current duty station page

Description
Continued copy changes on SM create profile/Current duty station page; lowercase d and s in "Current duty station" heading
Setup
Add any steps or code to run in this section to help others prepare to run your code:
make server_run
make client_run

Local login, click Create a New milmove User, progress through pages Create your profile>Name>Your contact info> to the page with the heading Current duty station
Code Review Verification Steps

[ ] Code follows the guidelines for Logging
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Screenshots
Fixed copy

Previously on this episode of Current duty station....

Failed on build_storybook_app  -- rerunning from failed, fingers crossed

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

  Generated by :no_entry_sign: dangerJS against 64e74ecff66bf6f4984f78fd18534565a6b68c81


Best practice for using on working configuration?

Hello
I have a big wg0.conf file with 30+ peers.
Is there any good way to use WireRest with current configuration and dont broke anything?
Also, I dont use pre-shared key. How to disable it via any flag or variable?
Is there any variable to listen only specific IP?
Thank you.

Hi!
WireRest has been tested on configurations with over 10,000 peers, 30 is a breeze :)
There is no flag to disable PSK generation, but you can simply send an empty value as in the example below. PSK will not be generated
{
  "publicKey": "PUBKEY",
  "presharedKey": "",
  "privateKey": "PRIVATE_KEY",
 ...other data
}

WireRest does not have the feature of listening to a specific IP address. You can instead specify a subnet by passing --server.address=<IP_ADDRESS> in the run arguments. Replace <IP_ADDRESS> with the IP address available on your server. Alternatively, you can configure some kind of firewall (such as UFW) to configure access rules.

Well. Yes. I tried to play and yes. I have to send only to auto generate peer data.
{
  "presharedKey": ""
}

But this is super weird :D
Thank you for server IP setting :)

Okay. Here is mine 2 coins about backup after any changes in wg0.conf:
pupa@vpn-1:~$ cat /etc/systemd/system/wireguard-backup.path
[Unit]
Description=Watch for changes in config of WireGuard VPN

[Path]
Unit=wireguard-backup.service
PathChanged=/etc/wireguard/wg0.conf

[Install]
WantedBy=multi-user.target

pupa@vpn-1:~$ cat /etc/systemd/system/wireguard-backup.service
[Unit]
Description=Upload backup for config file of WireGuard VPN
After=network.target

[Service]
Type=oneshot
ExecStartPre=/usr/bin/sleep 10
ExecStart=/usr/bin/bash -c "/usr/bin/gcloud storage cp /etc/wireguard/wg0.conf gs://my-vpn/$(date +"%%Y%%m%%d%%H%%M%%S")_wg0.conf"

[Install]
WantedBy=multi-user.target

sudo systemctl enable wireguard-backup.{path,service}
sudo systemctl start wireguard-backup.{path,service}


[FEATURE] Edit Project

Problem

need a way to edit project

User Story(s):
As a project owner, I want to be able to edit my project, so that I can update it with new information
Acceptance Criteria

edit projects. like #7 and accessible from #18
save as a transaction
only can edit, check if admin of project
pulls in information from that project from chain
pass in project idinto url

Limitations

dependent on designs #8

Resources

outline links and relevant resources, references, implementations

UI Done

https://github.com/PotLock/grantpicks/pull/40


Added support for multi-qubit gates in qcircuit

Mypy isn't smart enough to see that the relevant attributes are not None at the point in the code that they're referenced. Is there an annotation to help it out?

You can use cast from typing to assert that something is a type. E.g. cast(float, x).

Casting doesn't seem to work, even if I define an intermediate variable. Not sure why.
Are the assertions not okay?

The assertions are fine.


Application enforced restrictions section need an update

[Hello, please update that app enforced restrictions are supported not only for SharePoint Online and Exchange Online, but also Office 365 suite. Thanks!]

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

ID: 609a0eda-f113-8b77-67ee-c5e8bd94cbac
Version Independent ID: 7468d74e-9a7e-2a11-ba35-da46103a1b80
Content: Session controls in Conditional Access policy - Azure Active Directory
Content Source: articles/active-directory/conditional-access/concept-conditional-access-session.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@ChristianJBergstrom Thanks for the feedback! We are currently investigating and will update you shortly.

@ChristianJBergstrom
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@JamesTran-MSFT Why closed without any update?

@ChristianJBergstrom
Thank you for following up on this!
The issue was auto closed because a PR merged within the hour, which would update the documentation.

Please allow some time for the changes to reflect and if you don't see the doc update, please let me know.
Thank you again for your time and patience throughout this issue.

@JamesTran-MSFT Ah.. my bad. Don't really get all this to be honest. I'm simply just adding feedback when I notice that something isn't right in the Docs. Thank you for the very prompt reply.


Fix "method_complexity" issue in pyi18n/loaders.py

Function load has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
https://codeclimate.com/github/sectasy0/pyi18n/pyi18n/loaders.py#issue_630125cae66b0d000100001a

fixed here https://github.com/sectasy0/pyi18n/pull/8


Add a command for bootstrapping minimal working project

ffbt init, for example
it creates config.js, app.ts and index.ejs
relatives to #6

https://github.com/fluix/ffbt-init


Kafka channel controller won't start

Describe the bug
Gets the following error and restarts "Error reading/parsing logging configuration"
Expected behavior
channel controller to start
To Reproduce
Install kafka channel controller
Knative release version
0.17.2 serving, 0.17.3 eventing, 0.17.1 contrib
Additional context
here is the config-logging
apiVersion: v1
data:
  loglevel.controller: debug
  loglevel.webhook: debug
  zap-logger-config: |
    {
      "level": "info",
      "development": false,
      "outputPaths": ["stdout"],
      "errorOutputPaths": ["stderr"],
      "encoding": "json",
      "encoderConfig": {
        "timeKey": "ts",
        "levelKey": "level",
        "nameKey": "logger",
        "callerKey": "caller",
        "messageKey": "msg",
        "stacktraceKey": "stacktrace",
        "lineEnding": "",
        "levelEncoder": "",
        "timeEncoder": "iso8601",
        "durationEncoder": "",
        "callerEncoder": ""
      }
    }
kind: ConfigMap
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"v1","data":{"loglevel.controller":"debug","loglevel.webhook":"debug","zap-logger-config":"{\n  \"level\": \"info\",\n  \"development\": false,\n  \"outputPaths\": [\"stdout\"],\n  \"errorOutputPaths\": [\"stderr\"],\n  \"encoding\": \"json\",\n  \"encoderConfig\": {\n    \"timeKey\": \"ts\",\n    \"levelKey\": \"level\",\n    \"nameKey\": \"logger\",\n    \"callerKey\": \"caller\",\n    \"messageKey\": \"msg\",\n    \"stacktraceKey\": \"stacktrace\",\n    \"lineEnding\": \"\",\n    \"levelEncoder\": \"\",\n    \"timeEncoder\": \"iso8601\",\n    \"durationEncoder\": \"\",\n    \"callerEncoder\": \"\"\n  }\n}\n"},"kind":"ConfigMap","metadata":{"labels":{"eventing.knative.dev/release":"v0.17.3","knative.dev/config-category":"eventing","knative.dev/config-propagation":"original"},"name":"config-logging","namespace":"knative-eventing","ownerReferences":[{"apiVersion":"operator.knative.dev/v1alpha1","blockOwnerDeletion":true,"controller":true,"kind":"KnativeEventing","name":"knative-eventing","uid":"acbf8e84-2c2e-449e-8b95-acd422a643c9"}]}}
  creationTimestamp: "2020-09-10T05:23:22Z"
  labels:
    eventing.knative.dev/release: v0.17.3
    knative.dev/config-category: eventing
    knative.dev/config-propagation: original
  name: config-logging
  namespace: knative-eventing
  ownerReferences:
  - apiVersion: operator.knative.dev/v1alpha1
    blockOwnerDeletion: true
    controller: true
    kind: KnativeEventing
    name: knative-eventing
    uid: acbf8e84-2c2e-449e-8b95-acd422a643c9
  resourceVersion: "112153294"
  selfLink: /api/v1/namespaces/knative-eventing/configmaps/config-logging
  uid: d5859ea7-77b2-48d7-8bf1-2266c9f3d291

The config map sounds fine, can you show the log you see?

I found the issue, I missed a --- in my yaml for cluster roles so it was an RBAC issue.


Still issues when lots of files are changing

Open Brackets to the brackets source
Run grunt build in the Brackets folder and watch the console (open localhost:9234 as the internal DevTools may crash)

Result:
Many error messages saying Unexpected error refreshing file tree for directory foo: NotFound undefined

Eventually, Brackets becomes unresponsive.
Windows 8.1, latest master, Release 1.1 Brackets Shell

@MarcelGerber, what are the scenarios that you see issues after removing the brackets-todo extension?

As I said above, there are still errors showing up when running grunt build on the Brackets project, but way less (6 this time):

@MarcelGerber is brackets still unresponsive when you only get the 6 errors?

No, it's responsive.
cc @dangoor btw


Compiler error

Compiler message:
lib/presentation/screen/calendar/calendar_screen.dart:184:7: Error: No named parameter with the name 'onFormatChanged'.
onFormatChanged: _onCalendarFormatChanged,
^^^^^^^^^^^^^^^
file:///Users/bsturza/Desktop/Development/flutter/.pub-cache/hosted/pub.dartlang.org/table_calendar-1.1.4/lib/table_calendar.dart:127:3: Context: Found this candidate, but the arguments don't match.
TableCalendar({

it cannot find anything in the TableCalendar. If i comment it out, the app builds

I assume you are on master branch. If you check out to dev branch, it should all work correctly.
Just run git checkout dev in the project directory.
Right now I am working on a big update - I will merge branches when it is ready.

@sturza
You can now just update from master, since #34 fixes all breaking changes.


Redisson not set-up correctly when using JCache API and custom config file

I have a Spring Boot application and a Bean that implements org.springframework.boot.autoconfigure.cache.JCacheManagerCustomizer, i.e. there is a method that takes a javax.cache.CacheManager, and then follows the instructions provided here. Specifically, I have implemented a mixture of the code snippets for:

"Using config file with custom location"
"Using Redisson's config object:"

Essentially, I have the following that does create caches as I would expect on my local Redis instance:
@Override
public void customize(javax.cache.CacheManager manager) {
    MutableConfiguration<String, String> jcacheConfig = new MutableConfiguration<>();
    Config redissonCfg = getClass().getResource(expectedRedissonCacheConfigLocation);
    Configuration<String, String> config = RedissonConfiguration.fromConfig(redissonCfg, jcacheConfig);
    Cache<String, String> cache = manager.createCache("namedCache", config);
}

However, when I run tests or start/stop the Spring boot application, I always receive the following Log message and the Redis caches are not cleared.  Note that the caches are reset if multiple tests are run in sequence but the data cached in the last test is still present in the Redis instance after the test has been run.
WARN 10556 --- [           main] o.s.b.f.support.DisposableBeanAdapter    : Invocation of destroy method 'close' failed on bean with name 'jCacheCacheManager': java.lang.NullPointerException
After debugging, it seems that there is no org.redisson.Redisson object created. This appears to be caused by the checks for either a redisson-jcache.json and redisson-jcache.yaml being on the classpath in org.redisson.jcache.JCachingProvider#loadConfig(URI uri). If these files are not found, the config returned is null which means that org.redisson.jcache.JCachingProvider#getCacheManager(URI uri, ClassLoader classLoader, Properties properties) does not invoke the code inside the if conditional below:
Redisson redisson = null;
if (config != null) {
    redisson = (Redisson) Redisson.create(config);
}
manager = new JCacheManager(redisson, classLoader, this, properties, uri);

I do not have either a redisson-jcache.json and redisson-jcache.yaml file on the classpath since my team are trying to develop a way of loading different Redisson configuration files based upon the Spring Boot profile that is used to launch a Spring Boot application.  Configuration files are instead searched for in a specified location with a specified file name (the name of a Spring Boot profile).

To destroy cache in case of absence any config file you should invoke JCacheManager.destroyCache method

Hi @mrniko! I've worked around this issue by defining a redisson-jcache.yaml file that contains the default configuration that should be used if a specific application profile YAML file is not defined in a service that uses Redisson. This works well, but my main issue is that the documentation implies that a custom YAML file can be used to set-up Redisson in the same way as it would be set-up if a redisson-jcache.yaml file were present on the classpath. This does not seem to be the case, however!
Thank you for responding and all your work on this library though, it has been a tremendous help to our team :+1:

@mlk5060
You shared code for starting jcache instance in issue description, but I can't find code for shutdown. Could you share it?

The library I've been building (that makes use of Redisson) doesn't declare any shutdown code. After taking the steps I described above, the Log message I originally posted no longer appears. So, I'm assuming the Redisson object created by virtue of including redisson-jcache.yaml on the classpath contains shutdown code?

It's interesting to see that instance got through org.redisson.jcache.JCachingProvider#getCacheManager for shutdown. Whereas javax.cache.Cache implements java.io.Closeable and should be shutdown through close method. Could you debug it to understand why org.redisson.jcache.JCachingProvider#getCacheManager is invoked ?


Laravel 6.0 support

It seem due to old dependencies, you can't use it on Laravel/Lumen 6.0
Can you raise them to 6.0?

i could try, however not sure if it is 100% compatible, we will see it on tests

I wish I read this issue before trying to install. Old dependencies prevent it from working on Laravel 7 too.


docs: start the requirements debate

This PR is intended to start a requirements specification document to follow in our research and development journey. Currently, this document is not sealed and requests your contribution. Feel free to contribute with comments or suggestion where it's possible to apply them.

I suggest adding in the communication section the ability to send location, images & video as they maybe crucial during blackouts.


Gradle Deployment

Switching to Gradle needs a way to create artifacts, sign the artifacts, and upload them to Nexus.

Especially with the last round of contribs from Justin.  We should look at a release.
Also figure out which of the checkers are enabled by default. (all of them?).

I have a few more things to contribute on my list tat's work in progress but it can go into this or the next release.
Will need to update the documentation (perhaps we should consider auto generate via java docs?)

I'm looking for a simple way to either release a snapshot or an actual release with a CI system. When I figure that all out, I'll be releasing a snapshot for testing, then do an actual release. Thanks for all the updates Justin. This release will be awesome!

Javadocs sound like a good idea, see #62
On Tue, Oct 6, 2015 at 7:49 PM, Justin Mclean<EMAIL_ADDRESS>wrote:

I have a few more things to contribute on my list tat's work in progress
but it can go into this or the next release.
Will need to update the documentation (perhaps we should consider auto
generate via java docs?)
—
Reply to this email directly or view it on GitHub
https://github.com/cflint/CFLint/issues/80#issuecomment-146036299.

@jjames967 Thanks for looking into a release
On Tue, Oct 6, 2015 at 8:27 PM, Ryan Eberly<EMAIL_ADDRESS>wrote:

Javadocs sound like a good idea, see #62
On Tue, Oct 6, 2015 at 7:49 PM, Justin Mclean<EMAIL_ADDRESS>wrote:

I have a few more things to contribute on my list tat's work in progress
but it can go into this or the next release.
Will need to update the documentation (perhaps we should consider auto
generate via java docs?)
—
Reply to this email directly or view it on GitHub
https://github.com/cflint/CFLint/issues/80#issuecomment-146036299.

Javadocs was #61. How the Gradle deployment will work, it will automatically compile javadocs into a jar and upload them to the Nexus Repository. Just need the html information inside the source.


No scheme for monorepo

In one project I work on we have two apps on the same repo and the extension cannot find the schemes because the response from xcodebuild -list -json has a different structure:
{
  "workspace" : {
    "name" : string,
    "schemes" : string[]
}

where inside a project it is
{
  "project" : {
    "configurations" : string[],
    "name" : string,
    "schemes" : string[],
    "targets" : string[]
  }
}

It would be nice to support this configuration

Hello! Could you please describe your case in detail? Are they completely separate programs? When you launch an application, do you launch only one app at a time, or do you launch multiple applications in parallel?

@rafaelpedretti-toast Hello! I've added basic support for workspaces in the recently released version.
My first attempt to implement this feature was to parse Xcode configuration files manually, but I failed. I couldn't properly determine the schema list for the projects. Finally, I've decided to utilize the xcodebuild -list output, similar to single project workspaces. It's not ideal, but should work for some workspaces

@hyzyla would be possible to add workspace configuration to specify a workspace name or a project name? Apparently, I have a similar problem where the directory contains multiple projects at the same time and in that case, xcodebuild can't provide correct output for listing schemes.

@milanhorvatovic, Yeah, I think it's not a problem to add some configuration to specify a workspace or a project. I just need more information about how your project looks like to reproduce it locally. Could you provide the structure of your project and workspace? It can be the output of a comman find . \( -name "*.xcworkspace" -o -name "*.xcodeproj" \):
➜  ~/Developer/sweetpad-examples/sweetpad-demo-workspace find . \( -name "*.xcworkspace" -o -name "*.xcodeproj" \)

./sweetpad-demo-app1/sweetpad-demo-app1.xcodeproj
./sweetpad-demo-app2/sweetpad-demo-app2.xcodeproj
./sweetpad-demo-workspace.xcworkspace
./sweetpad-demo-app4/sweetpad-demo-app4.xcodeproj
./sweetpad-demo-app3/sweetpad-demo-app3.xcodeproj

Maybe the better would be to add an option for developers to specific path to the workspace or project. In that case, you don't need to worry about the structure itself because it would suit almost every needs. What do you think?
Many developers have mono repositories or different directory organization for project structure and so on.
It's not ideal, but you can get the idea from this:
./TestCombineExt/TestCombineExt.xcodeproj
./TestCombineExt/TestCombineExt.xcodeproj/project.xcworkspace
./Test.xcodeproj
./Test.xcodeproj/project.xcworkspace
./TestFoundation/TestFoundation.xcodeproj
./TestFoundation/TestFoundation.xcodeproj/project.xcworkspace
./TestSDK.xcodeproj
./TestSDK.xcodeproj/project.xcworkspace
./Test.xcworkspace
./Pods/Pods.xcodeproj
./TestUI.xcodeproj
./Playground.playground/playground.xcworkspace
./SampleApp.xcodeproj
./SampleApp.xcodeproj/project.xcworkspace

@milanhorvatovic I've added a new configuration for setting the Xcode workspace path in VSCode's settings.json Set Xcode workspace path. Let me know if it works for you 🙏

@hyzyla sorry about my timing, but it works nicely, thanks again.


build(examples): move Angular CLI versions file to another dir

Issue or need
Seems Renovate failed to update to pnpm v9 for the ngx-meta/examples directory
A reason could be it has "two package.json files"
One is actually a package.json , the other is added to Renovate config as one via config

Proposed changes
Move Angular CLI versions file to another directory to avoid misunderstandings

Quick reminders

🤝 I will follow Code of Conduct
✅ No existing pull request already does almost same changes
👁 Contributing docs are something I've taken a look at
📝 Commit messages convention has been followed
💬 TSDoc comments have been added or updated indicating API visibility if API surface has changed.
🧪 Tests have been added if needed. For instance, if adding new features or fixing a bug. Or removed if removing features.
⚙ API Report has been updated if API surface is altered.

#546  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite


radeon-crimson: download failed. curl: option -: is unknown

Issue description
building '/nix/store/w5nxdk461d9z3245qw77a7arjvy57s98-radeon-crimson-15.12-15.302-151217a-297685e.zip.drv'...
building '/nix/store/9h53aw49mhhymh4hmmhhzrz9nv76qz63-xorg-server-1.17.4.drv'...
building '/nix/store/38cyl426hfvzrc5il3bhxd5nxrjj2v7m-nixos-manual-combined.drv'...
building '/nix/store/wxjdipjn9z2k3p83hjsh6d8lz7cqn272-xwayland-1.17.4.drv'...
waiting for locks or build slots...
error checking the existence of http://tarballs.nixos.org/sha256/0n0ynqmjkjp5dl5q07as7ps3rlyyn63hq4mlwgd7c7v82ky2skvh:
curl: (7) Failed to connect to 0 port 80: Connection refused
curl: (22) The requested URL returned error: 404 Not Found

trying https://www2.ati.com/drivers/linux/radeon-crimson-15.12-15.302-151217a-297685e.zip
curl: option -: is unknown
curl: try 'curl --help' for more information
error: cannot download radeon-crimson-15.12-15.302-151217a-297685e.zip from any mirror
builder for '/nix/store/w5nxdk461d9z3245qw77a7arjvy57s98-radeon-crimson-15.12-15.302-151217a-297685e.zip.drv' failed with exit code 1
cannot build derivation '/nix/store/yka3lsw47il6ykvdik06hd6sw5y70s90-ati-drivers-15.12-4.14.78.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/lazl5jgzz3d74ckikd1mw1i6asfyj06k-etc.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/4w2kn7x2rwdy3m2n8an227fxqwz0g2fl-kernel-modules.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/lcf8fgz18j7k3d5gd201kx2wqbd0avis-opengl-drivers.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/f6rkslb8mxk7v44z1bcgizmyd4l12axi-system-path.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/30wgfc83ih8n4x9kmhakrkp4ygbrrflp-nixos-system-imac-18.09.1023.06fb0253afa.drv': 1 dependencies couldn't be built
error: build of '/nix/store/30wgfc83ih8n4x9kmhakrkp4ygbrrflp-nixos-system-imac-18.09.1023.06fb0253afa.drv' failed

Maybe it's a problem with the curlOpts? https://github.com/NixOS/nixpkgs/blob/2c2f1e37d4374ea61caefd9389927ea03df4ce31/pkgs/os-specific/linux/ati-drivers/default.nix#L65
Don't seem so. I can also not download it manually with curl:
curl -O --referer http://support.amd.com/en-us/download/desktop?os=Linux+x86_64 https://www2.ati.com/drivers/linux/radeon-crimson-15.12-15.302-151217a-297685e.zip

(i get a 0 byte file)
But i can download it using a browser from the website. But the hash is also different:
[root@imac:~]# sha256sum radeon-crimson-15.12-15.302-151217a-297685e.zip
704f2dfc14681f76dae3b4120c87b1ded33cf43d5a1d800b6de5ca292bb61e58  radeon-crimson-15.12-15.302-151217a-297685e.zip

So they prevent us from downloading the file using something else than their website. Maybe the referer is not enough now?
Workaround (failed)

Go to https://www.amd.com/de/support
select some GPU, for example 7990

scroll down to Linux x86_64
and download AMD Radeon Software Crimson Edition 15.12 Proprietary Linux x86 Display Driver
then add the file to nix store manually:

[root@imac:~]# nix-store --add radeon-crimson-15.12-15.302-151217a-297685e.zip
/nix/store/j7chyjymgs1kic42byqnhfqn6gj3k495-radeon-crimson-15.12-15.302-151217a-297685e.zip

Now you can rebuild. But the file don't match because the hash is different!
Steps to reproduce
  services.xserver.videoDrivers = [ "ati_unfree" ];
  hardware.opengl.driSupport32Bit = true;
  nixpkgs.config.allowUnfree = true;

Technical details

system: "x86_64-linux"
host os: Linux 4.14.78, NixOS, 18.09.1023.06fb0253afa (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.1
channels(root): "nixos-18.09.1023.06fb0253afa, nixos-hardware, nixos-unstable-19.03pre156996.c70ad805d21"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

I can fix the download error with this patch, but build fails now.
diff --git a/pkgs/os-specific/linux/ati-drivers/default.nix b/pkgs/os-specific/linux/ati-drivers/default.nix
index d5d42faa913..cda7e03cd06 100644
--- a/pkgs/os-specific/linux/ati-drivers/default.nix
+++ b/pkgs/os-specific/linux/ati-drivers/default.nix
@@ -61,8 +61,8 @@ stdenv.mkDerivation rec {
   src = fetchurl {
     url =
     "https://www2.ati.com/drivers/linux/radeon-crimson-15.12-15.302-151217a-297685e.zip";
-    sha256 = "0n0ynqmjkjp5dl5q07as7ps3rlyyn63hq4mlwgd7c7v82ky2skvh";
-    curlOpts = "--referer http://support.amd.com/en-us/download/desktop?os=Linux+x86_64";
+    sha256 = "704f2dfc14681f76dae3b4120c87b1ded33cf43d5a1d800b6de5ca292bb61e58";
+    curlOpts = "--referer https://www.amd.com/en/support";
   };

   hardeningDisable = [ "pic" "format" ];

Maybe they have changed the content of the file, so also the hash has changed.
checking if gcc supports -Wformat=2... yes
+ for pelib1 in fglrx_dri.so dri/fglrx_dri.so
+ patchelf --remove-needed libX11.so.6 /nix/store/bzvxw3xn7061ny9w5z13cdb6nvpcwl97-ati-drivers-15.12/lib/fglrx_dri.so
checking if gcc supports -Wdeclaration-after-statement... yes
checking if gcc supports -Wstrict-prototypes... yes
+ for pelib1 in fglrx_dri.so dri/fglrx_dri.so
+ patchelf --remove-needed libX11.so.6 /nix/store/bzvxw3xn7061ny9w5z13cdb6nvpcwl97-ati-drivers-15.12/lib/dri/fglrx_dri.so
checking if gcc supports -Wunused... yes
+ for pelib2 in libatiadlxx.so xorg/modules/glesx.so dri/fglrx_dri.so fglrx_dri.so libaticaldd.so
+ patchelf --set-rpath /nix/store/a0d6q18gxggcvb5x24nrdg6nvhbnipnb-glibc-2.27/lib/:/nix/store/jzlj3z1z8wf0qjdjd4460br66xkr17gm-gcc-7.3.0-lib/lib/ /nix/store/bzvxw3xn7061ny9w5z13cdb6nvpcwl97-ati-drivers-15.12/lib/libatiadlxx.so
+ for pelib2 in libatiadlxx.so xorg/modules/glesx.so dri/fglrx_dri.so fglrx_dri.so libaticaldd.so
+ patchelf --set-rpath /nix/store/a0d6q18gxggcvb5x24nrdg6nvhbnipnb-glibc-2.27/lib/:/nix/store/jzlj3z1z8wf0qjdjd4460br66xkr17gm-gcc-7.3.0-lib/lib/ /nix/store/bzvxw3xn7061ny9w5z13cdb6nvpcwl97-ati-drivers-15.12/lib/xorg/modules/glesx.so
+ for pelib2 in libatiadlxx.so xorg/modules/glesx.so dri/fglrx_dri.so fglrx_dri.so libaticaldd.so
+ patchelf --set-rpath /nix/store/a0d6q18gxggcvb5x24nrdg6nvhbnipnb-glibc-2.27/lib/:/nix/store/jzlj3z1z8wf0qjdjd4460br66xkr17gm-gcc-7.3.0-lib/lib/ /nix/store/bzvxw3xn7061ny9w5z13cdb6nvpcwl97-ati-drivers-15.12/lib/dri/fglrx_dri.so
checking if gcc supports -Wmissing-prototypes... yes
checking if gcc supports -Wuninitialized... yes
/nix/store/13r1dybm9f1xz0bl2s33qqi36cypbxzd-linux-4.14.78-dev/lib/modules/4.14.78/source/Makefile:953: *** "Cannot generate ORC metadata for CONFIG_UNWINDER_ORC=y, please install libelf-dev, libelf-devel or elfutils-libelf-devel".  Stop.
make[2]: *** [Makefile:146: sub-make] Error 2
make[1]: *** [Makefile:24: __sub-make] Error 2
make[1]: Leaving directory '/nix/store/13r1dybm9f1xz0bl2s33qqi36cypbxzd-linux-4.14.78-dev/lib/modules/4.14.78/build'
make: *** [Makefile:88: kmod_build] Error 2
+ exitHandler
+ exitCode=2
+ set +e
+ '[' -n '' ']'
+ ((  2 != 0  ))
+ runHook failureHook
++ shopt -po nounset
+ local 'oldOpts=set +o nounset'
+ set -u
+ local hookName=failureHook
+ shift
+ local 'hooksSlice=failureHooks[@]'
+ local hook
+ for hook in "_callImplicitHook 0 $hookName" ${!hooksSlice+"${!hooksSlice}"}
+ _eval '_callImplicitHook 0 failureHook'
++ type -t '_callImplicitHook 0 failureHook'
+ '[' '' = function ']'
+ set +u
+ eval '_callImplicitHook 0 failureHook'
++ _callImplicitHook 0 failureHook
++ set -u
++ local def=0
++ local hookName=failureHook
++ case "$(type -t "$hookName")" in
+++ type -t failureHook
++ '[' -z '' ']'
++ return 0
+ set -u
+ eval 'set +o nounset'
++ set +o nounset
+ return 0
+ '[' -n '' ']'
+ exit 2
builder for '/nix/store/z2rhgqkfvg6brw2lq1fsblvj31x1s0qr-ati-drivers-15.12-4.14.78.drv' failed with exit code 2

I would like to also fix the build before creating a PR, but i have no idea how to fix this.


GH-44555: [C++][Compute] Allow casting struct to bigger nullable struct

Rationale for this change
Sometimes its useful to add a column full of nulls in a cast. Currently this is supported in top level columns but not inside structs. Example where this is important: https://github.com/delta-io/delta-rs/issues/1610
What changes are included in this PR?
Add support for filling in columns full of null for nullable struct fields.
I've gone for a fairly minimal change that achieves what I needed but I wonder if there should be a more significant change so that this casting is done by field name and ignore the field order.
Are these changes tested?
Yes. The expected behaviour in several existing tests has been altered and a couple of new tests have been added.
I also rolled out a custom build with this change internally because it suddenly became a critical problem.

Are there any user-facing changes?
Yes. There are scenarios that previously failed with struct fields don't match or are in the wrong order but now succeed after filling in nulls.

GitHub Issue: #44555

I'm not confident that I've taken the right approach here but I think its ready for review.

Ok, it looks like there is at least one more tests I need to update. I can probably sort that tomorrow.

Sorry for the direct ping but @lidavidm please could you review when you get a chance. I assume you are the right person to review this?

Ok, I don't think that one CI failure is related here. Merging now. Thanks @Tom-Newton, sorry for the delays!

Ok, I don't think that one CI failure is related here. Merging now. Thanks @Tom-Newton, sorry for the delays!

Thanks for reviewing


Before 10k server requests are sent, reset our connection.

https://kolmafia.us/threads/ioexception-goaway-received.27397/#post-167957

Regarding FileUtilities, I would be surprised if it was a non-issue there. But at the same time, as you said it's also pretty lenient there.
It should be only new installations of mafia that could hit that limit.
In regards to GenericRequest. I'm not sure moving it to its own class is needed. Although it could be prettier if this code is used elsewhere; Eg FileUtilities.
If you'd like to make the changes for moving it to its own class, you're more than welcome.

It should be only new installations of mafia that could hit that limit.

Maybe I'm missing something but since there is a user initiated cache clearing command it seems like the limit would also be approached after the cache was cleared in an old installation.


Mental Blast rolled 4 damage but dealt 3 damage.

Likely the same as #508. The root cause of that is not correctly treating roll.terms when it has more than 1 term (i.e. code is generally using roll.terms[0]).


Can't close the markdown preview, when it's untitled and the security setting has been changed

Open an untitled editor and set the language mode as markdown
Add the cat markdown from #30879
Open preview
Change the security setting
Click the 'x' or cmd+w to close the preview
No response 😱

Debugged out of curiosity and seems like this check in editorPart.doCloseEditor fails: if (group.activeEditor && group.activeEditor.matches(input)) { because in areHtmlInputOptionsEqual, left.allowSvgs !== right.allowSvgs.

Good find.
The x doesn't work for you? I can repo the issue with cmd+w but can still close the editor when I press x on the tab

I have tabs disabled, different code path?

I don't follow how the change fixes it, because it looks like the same code from matches is basically inlined in the if? But I tried it in master and it works.

Yes, not a great fix but I wanted to keep the fix scoped this late in the ship cycle.
The logic: we want editorInput.matches(otherInput) to return true for the htmlPreview even if the otherInput's security options do not match the current input's. Removing the custom match from HtmlInput fixes this bug. However when we update the input of the htmlPreview, we want to rerender the contents if the security options have changed. That's where the inlining comes in

Ah now I understand, I was thinking the line in setInput was the closeEditor line I was looking at last night.


Add a reason attribute to filePUT and fileGET

This adds a new reason property to filePUT and fileGET that could give a little bit more information why certain file types are so popular over others. Most specifically, for the fileGET case we will be able to filter out those calls that did not originate from a normal editor open. To give an example, when you open a TS file, we also seem to resolve a tsconfig.json file from the TS extension. This file is not opened as editor but still resolved and we count a fileGET for this call. This seems to happen only once though.

I love this idea and the approach (adding a property so we can check the delta) - is there a way to indicate an open based on a folder open that resulted in past windows that resulted in files opening again?
e.g. I closed 15 windows last week (some JSON, Some JS, some Java) and opened that folder this week again (and they were restored)?
You may have accounted for this already but asking anyhow.
Sean

@seanmcbreen no, it would all look like a normal editor open. Let's wait to see what the current changes tell us and then tweak this further if we have to.

LGTM


Get rid of github.com/geertjanvdk/xkit

We should use the packages found in golistic organization instead.

This is also a generic clean up. We replace most with golistic/xgo.

We also upgrade to Go 1.21.


Template /Server_Hardware/RAID_Cards/template_adaptec_microsemi_using_arcconf/ is broken

It was missing config file and scripts scripts (powershell and bash) to get the value of the variables

They are still missing, were can we find them?
BC


Embedded

Hi there,
just quick question. What are my options when I need to run DICOM Viewer embedded in my web app, but want to host image archiver locally and connect my DICOM Viewer to it?
Thank you

In latest master branch your options is to use iframe https://docs.ohif.org/deployment/cors/#embedding-ohif-in-an-iframe


Partly remove crossplane-runtime dependency

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Getting away from the crossplane-runtime dependency should make controller-runtime updates easier in the future.
Which issue(s) this PR fixes:

None
Does this PR have test?
None

Special notes for your reviewer:
We still have a dependency to the condition API: https://github.com/crossplane/crossplane-runtime/blob/master/apis/common/v1/condition.go
Does this PR introduce a user-facing change?

None

/retest

/test pull-security-profiles-operator-test-e2e

PTAL @jhrozek @JAORMX @pjbgf


Fix Coursier cache location for windows

This time I verified that the coursier directory was actually cached on windows.

To validate this pull request against other sbt modules, please visit this link.


Make the bot PRETTIER!!!

The Issue
Bots these days have to have PRETTY OUTPUT!! We also don't want to spam users with 100000+ files if we find that many (even 7 images gets overwhelming...)
Questions

Can we use discord Embeds to clean up how people use the bot?
Can we have a preview picture?
What about a scroll bar or pagination using reaction buttons?

Let's try to combine embeds and discord buttons


2017-2018 Honda CR-V -- Lateral control + Stock ACC

I think the only thing missing here is that I have a radar error commented out. Was going to add civic hatch and accord but going to wait on those, they require some more testing and possibly development.
Would like to get this upstreamed so we have a good base for bosch ports.

I drive an 2018 CRV Touring; The 17 is the same... The Accell pedal does not cancel the ACC, it will just override throttle control and the ACC will resume throttle control when the pedal is released. It is possible to accel right into car in front if you really wanted to. The brake pedal, cancel switch, low traction event or over speed ( above 145KMH) will cancel the ACC operation. Hope the Giraffe will come out of this soon for the CRV /Bosch camera!

What else do I need to get going on my '17 CR-V /w sensing? I made a neo and 12p cable, and I just ordered a white panda.
Are any other accessories required for lateral control? Without the panda, the EOS seems unable to calibrate or prevent lkas fault codes. Is this expected?

Cmiller did you have any luck? I'm looking to get my 17 crv going but none had written it up. Someone put up a diagram for a 12p giraffe in the other thread and said it would be for sale 'soon'. I don't understand enough about all of this to get hacking yet but if you got yours working, would you mind writing up a guide?

You need an EON + Panda + Giraffe for this code, it requires 3 canbuses, NEO only has two

Is there a 12p giraffe available, or is everyone fabricating their own based on the diagram in the other thread?

That's a negative. There's two people with proof of concept videos on
YouTube, but I haven't gotten around to emailing them yet.
Please copy me if you learn anything
Regards.
On Mar 8, 2018 11:30 AM, "willitblend"<EMAIL_ADDRESS>wrote:

Cmiller did you have any luck? I'm looking to get my 17 crv going but none
had written it up. Someone put up a diagram for a 12p giraffe in the other
thread and said it would be for sale 'soon'. I don't understand enough
about all of this to get hacking yet but if you got yours working, would
you mind writing up a guide?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/commaai/openpilot/pull/175#issuecomment-371541384,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Af3puW-N_yJTLL9cYBqDkOBKFfpY4aQLks5tcVybgaJpZM4Ribiw
.

Just purchased an EON + Panda + Giraffe for my 2018 Honda CRV EX-L with sensing. Any one near San Francisco that could help me set up the Giraffe connector? I've read the previous post but don't have any experience with hacking cars. Not confident enough to do it myself. Will compensate generously for your time.

@achen415 Not compatible with comma giraffe

Since a giraffe isn't available for this vehicle yet, here is what you need to build to run this code.  Jumper wires soldered to an OBDII connector work great (male jumpers for wire harness side, female jumpers for camera side)
Panda CAN 2 → ACC CAN (camera 6/12)
Panda CAN 3 → ACC CAN (harness 6/12)
F-CAN B (harness 5/11) → Panda CAN1 → F-CAN B (camera 5/11)
IGN/GND (harness 1/2) → Panda IGN/GND → IGN/GND (camera 1/2)
LWD SW/IND (harness 7/8) → LWD SW/IND (camera 7/8)

Question how is it there's so much info on newer cars and not much on 2016
Honda Accord touring.  I am new to this and don't know anything but no one
seems to know about this car
On Thu, Mar 29, 2018, 6:09 PM Greg Hogan<EMAIL_ADDRESS>wrote:

Since a giraffe isn't available for this vehicle yet, here is what you
need to build to run this code. Jumper wires soldered to an OBDII connector
work great (male jumpers for wire harness side, female jumpers for camera
side)
Panda CAN 2 → ACC CAN (camera 6/12)
Panda CAN 3 → ACC CAN (harness 6/12)
F-CAN B (harness 5/11) → Panda CAN1 → F-CAN B (camera 5/11)
IGN/GND (harness 1/2) → Panda IGN/GND → IGN/GND (camera 1/2)
LWD SW/IND (harness 7/8) → LWD SW/IND (camera 7/8)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/commaai/openpilot/pull/175#issuecomment-377387752,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjApxiQP0qWESSPHmwLUyfDhaA4AHy_Kks5tjVu2gaJpZM4Ribiw
.

@billy2kool If you have questions about a particular car, please post on slack, not off-topic on other car threads.

@Geohot already threatened him with a ban because of this 1-2 weeks ago. He’s gone and done it again.

I have reservations about the stock acc as the sole forward control. I often don't have the heart to let the car continue breaking when the free way stops or when the driver in front of me is reacting slowly to a red light. Do those of you using this set a really high follow distance?

@willitblend Radar does not detect stopped cars well for technical reasons, this is common in most adaptive cruise control systems, including openpilot but if the system has detected the car in front of you it should be ok to stop safely.

@energee did we make the same comment at the same time? :)
Anyway, not clear why this happens but should be easy to find out. Let me know if you get stuck

Haha, i realized you asked same thing right after i published. It looks like its trying to put an 8 into the counter singal on 0x296. I don't know too much about the test scripts but trying to catch myself up.

Yeah, I would read the stopped state from the cruise state msg instead of
the hud.
On Wed, Jun 6, 2018, 8:42 AM Ted Slesinski<EMAIL_ADDRESS>wrote:

@energee commented on this pull request.
In selfdrive/car/honda/carstate.py
https://github.com/commaai/openpilot/pull/175#discussion_r193459181:

 self.brake_switch = cp.vl["POWERTRAIN_DATA"]['BRAKE_SWITCH']

self.brake_pressed = cp.vl["POWERTRAIN_DATA"]['BRAKE_PRESSED'] or \

if self.CP.safetyModel == car.CarParams.SafetyModels.hondaBosch:

 self.stopped = cp.vl["ACC_HUD"]['CRUISE_SPEED'] == 252.

Good point, will talk with @rbiasini https://github.com/rbiasini about
this, naming matches nidec hondas but i think this message matches them
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/commaai/openpilot/pull/175#discussion_r193459181, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQ4FkibmFVn7DyDXMde5ado54oGuWgWrks5t5_hugaJpZM4Ribiw
.

Closing since it was merged internally by @rbiasini


nested user components may not refresh as expected

Storing a elem.Italic()  in a struct as a vecty.ComponentOrHTML seems to interfere with the diffing algorithm for that struct.
https://play.jsgo.io/3e44bb6e46a5c866c79da52fe30219a72c119507
What you'll see is two fontawesome sync-alt that when you move your mouse over them, they change color.
The first one is broken (weirdly it changes once but no more). The last one works as I would have expected.
The difference between them is that the last one has the elem.Italic() directly stored as a vecty.ComponentOrHTML instead of in a struct.
This occured while using
github.com/gopherjs/vecty v0.0.0-20191227222824-6a0a25ee5a96

I took a very brief look at this, the problem is with the way we keep a single persistent component around.
For example, if you have a component Foo that each time Render is called produced a new &Bar we guarantee that we only keep one *Bar around even though there are two (the old and the new) that way you can trust that private fields on Bar can be mutated to maintain state.
The problem with this code is that (*Div).Render() returns components of the same type *FontTemp and so Vecty's internals say "Understood, we'll only keep the last one and will just copy the prop values over from the new one to the old one" and then suddenly both of your component instances:
		&Div{
			Normal:   &FontTemp{On: false},
			Hovering: &FontTemp{On: true},
		},

Have an On value of true, and thus all subsequent rerenders (even though they are correctly rerendering) are rerendering with the unexpected value of On: true.
How to address this is something I haven't pondered yet.

It's also worth noting that if you use vecty.If this does work as you'd expect because Vecty is able to differentiate between the two FontTemp instances in this case: https://play.jsgo.io/620688b80910cddfebaad8c500273dce4e2cf765

I'm still wrapping my head around what you've said. BTW thanks for looking into this!
Ok, I get why the old component pointer is the one you keep, and that the compare checks the types and says, "we'll just copy the prop values over from the new one to the old one".
How/why does the same thing not happen with elem.Div() or any of the other elem.*?
I was looking in dom.go but I realize I'm REALLY not sure where this check is happening.
That said, I feel like this isn't necessarily a bug. It's undesirable but not a bug. It also means I have other code that I'm not sure how it's working correctly as I thought it was doing this same thing.
Based on what I think I understand, you're getting performance gains by not calling the render() function and diffing the nodes; and I don't see how it could be changed without causing a performance penalty.

How/why does the same thing not happen with elem.Div() or any of the other elem.*?
I was looking in dom.go but I realize I'm REALLY not sure where this check is happening.

It is primarily happening here: https://github.com/gopherjs/vecty/blob/master/dom.go#L999-L1009
The behavior is the same regardless of which elem.Foo type you're using. If you are seeing otherwise that would be extremely surprising.

That said, I feel like this isn't necessarily a bug. It's undesirable but not a bug. It also means I have other code that I'm not sure how it's working correctly as I thought it was doing this same thing.

If you can share more cases like this that would be helpful.

Based on what I think I understand, you're getting performance gains by not calling the render() function and diffing the nodes; and I don't see how it could be changed without causing a performance penalty.

No, this has nothing to do with performance, actually. Here is a complete explanation:
func (*PageView) Render() vecty.ComponentOrHTML {
  return &SubView{}
}

type Subview struct {
    vecty.Core
    i int
}

func (s *Subview) Render() vecty.ComponentOrHTML {
    s.i++
    return vecty.Text(fmt.Sprint(s.i))
}

If we were to continuously ask vecty.Rerender(pageView) then looking at the code above you would expect i to continuously increase with each render. But, actually, we have the following:

After the first render, a *Subview with i == 1
After the second render, an entirely new *Subview instance with i == 1
After the third render, yet an entirely new *Subview instance with i == 1

What Vecty can do is:

Copy over fields using reflection, but this wouldn't solve the issue because i is not exported.
Copy over the entire struct from the prior render to the new render, but this means that if e.g. a method of Subview spawns a goroutine to increment i it would then be operating on the wrong instance. -- also confusing!
Copy over fields using reflection from the new render into the old render.

We chose the third option above, since it means we only keep around one instance of the component at a time and things generally just work as you'd expect!
However, we have since learned through this issue that there are still cases where this can be pretty confusing behavior (and I largely agree).
I'm going to keep this issue open to think about whether we can improve this some more, and if not at least find a way to clearly document this behavior.

Hello. Think I have the same problem but source code of the first message is not available.
My example with global variables:
https://gist.github.com/daniil388/f3689983006466572659f8aa12633d33
In console log printed correct global list each Render method.
In browser I see the same list but shorter.
What is wrong with this code? except global variables :)

@daniil388 almost certainly the problem with your code here is the usage of those global variables. You must create component instances in the Render method, generally speaking, because Vecty expects functional programming basically. Most likely we just don’t panic on your code as we should


Passing Complex Objects in Navigation

Back in the 8.0 days we would only pass strings in the navigation methods.
I am successfully passing complex objects with child collections without problem.....well at least I hope there is not a problem I am unaware of.
So has this issue been solved for a vanilla T10 project, or are there some gotcha's I don't know about that will end up causing issues after apps get in user's hands?
Thanks.

https://github.com/Windows-XAML/Template10/wiki/Page-Parameters

Wow, I did not see that while researching.  Thanks for the link.

Because the Template 10 NavigationService serializes all parameters, not all types can be successfully passed to a page

So what types can't be serialized?

Can I assume that if the object model I am passing now works, that it is not a fluke and that it will work going forward?

Good work @AmitBhatnagar24


Job queue in k8s. Mutliple pods == duplicate jobs?

If I have 5 pods running for an application in k8s. If a new job is created in the bull job queue in pod 1, will bull only trigger that job to be executed on 1 of those pods or will all 5 then run?

I tested this by...

Create a very simple bull job that just logs to the console.
Run my app in minikube with 3 replicas.
Add job to be executed. See in console how many times the console ran which indicates how many apps ran the job.

The results: Only 1 pod ran the job. This issue is not an issue.


[Blazor] Disposal of JavaScript module should also release eventhandlers, if necessary

Hey @guardrex, here's another important corner case.
The blazor docs  cover js module disposal here.
However if javascript event handlers are registered, they should (probably?!) be released, else they're liable to cause hard to debug problems.
An interesting solution can be found on SO. In summary:
MyComponent.razor.js
export function dispose()
{
  _myElement.removeEventListener("mousedown", handleMouseDown);
}

MyComponent.razor
@implements IAsyncDisposable
@code {
  private IJSObjectReference? _module;
  public async ValueTask DisposeAsync()
  {
    if (_module != null)
    {
      await _module.InvokeVoidAsync("dispose");        // <---------------
      await _module.DisposeAsync();
    }
  }
}

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

ID: 2b7ea38a-649b-303d-bebf-c701e216ba25
Version Independent ID: 9e197a8b-2616-8363-0a15-38cac802e82e
Content: Call JavaScript functions from .NET methods in ASP.NET Core Blazor
Content Source: aspnetcore/blazor/javascript-interoperability/call-javascript-from-dotnet.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

AFAIK per https://github.com/dotnet/AspNetCore.Docs/pull/27439, which Steve reviewed, that can't be called if a circuit is being disposed.
https://learn.microsoft.com/en-us/aspnet/core/blazor/javascript-interoperability/call-javascript-from-dotnet?view=aspnetcore-6.0#javascript-interop-calls-without-a-circuit
The alternative to JS interop during disposal is to use the MutationObserver pattern in client code.
If you need additional support on these scenarios, you can open a product unit issue at ...
https://github.com/dotnet/aspnetcore/issues
If you inquire on it, please add the following to your opening comment ...
Please add ...
cc: @guardrex https://github.com/dotnet/AspNetCore.Docs/issues/27536

... to the bottom of your opening comment so that I can follow along. I might re-open this for doc work depending on what they say.

Yes, that should be fine. Actually, I think I'll work on this in the near-term, so let's let this issue track it. I'll probably be able to reach this within a few weeks.

NOTE TO SELF: Double-check cross-linking because the JS interop approach for placing JS code in an app doesn't look like it received a cross-link to the new guidance for Blazor Server where it speaks to disposal without a circuit, and this new bit for Blazor WASM will also need cross-linking from the same location.

@lonix1 ... Moving back over here now that Javier provided information for this.
Just for completeness here ... as a bit more NOTE TO SELF so that I don't forget ...
On the Call JS side, disposal of IJSObjectReference appears in the example at ...
https://learn.microsoft.com/en-us/aspnet/core/blazor/javascript-interoperability/call-javascript-from-dotnet?view=aspnetcore-7.0#javascript-isolation-in-javascript-modules
... BUT, the disposal example that uses a module's FN doesn't appear until the reader reaches Steve's Mapbox example at ...
https://learn.microsoft.com/en-us/aspnet/core/blazor/javascript-interoperability/call-javascript-from-dotnet?view=aspnetcore-7.0#javascript-libraries-that-render-ui
... along with the new guidance on disposal without a circuit at ...
https://learn.microsoft.com/en-us/aspnet/core/blazor/javascript-interoperability/call-javascript-from-dotnet?view=aspnetcore-7.0#javascript-interop-calls-without-a-circuit
JS-side disposal isn't currently covered in the Call JS article, so I'll see about working that in.
Overall, I think dedicated sections with disposal examples on each side (JS and .NET) SxS will clear all of this up.
I have at least a week of .NET 7 release work to do, so I hope to reach this by the middle of next week. I'll ping u on the PR when it goes up. If I run into problems/concerns, I'll ask Javier to review the PR. If I can base it directly on prior approved guidance and his latest remarks, I won't need his review. This will mostly end up being a re-organization and enhancement of coverage based on what he just said ... no need to bug him further if all goes well.

During a release, it's pretty much .......
⛰⛏😅
... followed by a lot of ...
🏃🏃🏃🏃🏃🏃🏃🏃🏃🏃😅
... and then things will calm down in a week ... or THREE!!! 😆
I'll try to get back to this by the end of next week; but if you don't hear from me by then, it will slip one more week. I definitely want it resolved by 🦃 Day tho. I'd like to get a restful 🦃 weekend break before a big security docs push through EOY. I'll ping u on the PR later.

I almost made it back to this 😅. It's the next item on the regular workload list.
I'm kind'a 🧠 fried at the moment to take it on the last day prior to the 🦃 weekend holiday. Then, I have a feeling that given the high priority Blazor Security node work that needs to be addressed in the 🦃 to New Years 🎆 period that this will get pushed back to early January.
I would've preferred to square this away prior to the Security node work, but there was a large amount of work to do for .NET 7's release and a mop-up of a bunch of issues lying around prior to reaching the Security node. I literally just got caught up to reach this issue this morning. I will get to this one as soon as I can, and I think it will be early January. It's a top priority after the Security node work is complete.


Plist config Support

Hi
I am without access to the company's MDM, but want to deploy the app to a small group of clients I manage.
Without MDM access, I try to use a plist. I have created a plist with keys from the sample profile nl.root3.support.plist.txt and placed it in /Library/Preferences, but the app doesn't seem to read the settings.
Something is happening when the plist is placed, the app's CustomColor setting change from blue to green (same color as in the sample profile)
Hope this is maybe a bug and is something that can be fixed?
If not, thanks for the app anyway, it looks great :)

Hi @AndersTao,
We did not really focus on a non-MDM deployment but I was able to run some tests. Because the app is sandboxed, the ideal location should be /Users/USERNAME/Library/Containers/nl.root3.support/Data/Library/Preferences/nl.root3.support.plist in the user's home folder, but we found /Library/Preferences/nl.root3.support.plist also works. Other ways is installing a configuration file locally, but this cannot be automated on Big Sur anymore.
Could you try removing the nl.root3.support.plist file in all Libraries, try one of the above and restart the app? Hope it works for you as well!

Thanks for looking into this @jordywitteman
Removing the "default" nl.root3.support.plist from ~/Library/Containers/nl.root3.support/Data/Library/Preferences solved the problem, and the plist in /Library/Preferences are now read by the app 👍🏼


Add notification on ítem with maximum rolls

a suggestion for the module that fetch prices on ítem components, for the components that dont trade on merchant,  add a notification if the ítem has one or more maximum rolls on mods son its easier and quicker to identify valuables

this ítem has 2 max roll mods and the dmg mod is not max(min?) roll

Unless mods have obscure max stats, this would just add chaff to the tooltip rather than new info

the idea proposed, is intended for both New and old players.
New players dont know the mayority of mods that exists or their values, this game only value perfect mods, a 20% enchant is soight for, a 19% enchant is not.
for veteran players , it adds a really nice and Quick way of selecting loot for salvanging/ vendoring / stashing
referring to the pictured attached, that weapon does no have any modifier that has vendo rvale, therefore, theres no extra clutter on the UI.
the same ares used for displaying rune and insignia values, could be used to indicaste if one of your mods has a máx value roll, in the case of the picture attached, it is 20/20 sundering and also 20% enchant.


i18n(ja): Update ja/guides/cms/contentful.mdx

Description (required)

Update ja/guides/cms/contentful.mdx

https://github.com/withastro/docs/commit/0bad0edf9ee997a9be8ff4a4af3e4107a498a3e1#diff-7f4550f0c31345b1ef85a70b5f4f2b11ab0fc191dda80a3af7afa6e77ab527ac
https://discord.com/channels/830184174198718474/972429103821111326/1260957795818278922

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ja
guides/cms/contentful.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Fix the height of bar charts when a bar is focussed.

before.webm
after.webm
In before file, you can notice that the smaller section on the 4th bar becomes even smaller on hover.
In after file, this is fixed so both of them proportionally grow bigger while anchoring to the same origin point.
This fix is applied to both BarChart and  BarStackedChart

@ankur2136 Great fix! Thanks!

Quality gate passed: https://github.com/dautovicharis/Charts/pull/137


[enh] add get_oid_value_all() to http/certs/js/x509 example.

This PR Implements get_oid_value_all() returning a list of all the matching fields with the specified 'oid', and resolves #14 .

Committed, thanks.


Tighten mnist tutorial tests

Given the atol_fac, the tests had become too loose. the new values allow 1.5% change in the train/test clean/adv accuracies except for adversarial training accuracy that is allowed to change 6%.

@npapernot, thanks for the merge. Is there a reason for having different atol_fac for different TF versions? Is there a difference in the underlying seeding mechanism? The change was introduced in 7ab2a7fa4a07f3a52a49e5197b347ba78c7cb8ed. I have a commit that fails the test for TF<1.1.0.

The reason is that some tests were previously failing for one of the two TF
versions we're testing. These tests are not ideal to begin with, but before
we solve the reproducibility issue #253, it's going to remain a problem.
On Fri, Nov 3, 2017, 7:45 PM Fartash Faghri<EMAIL_ADDRESS>wrote:

@npapernot https://github.com/npapernot, thanks for the merge. Is there
a reason for having different atol_fac for different TF versions? Is there
a difference in the underlying seeding mechanism? The change was introduced
in 7ab2a7f
https://github.com/tensorflow/cleverhans/commit/7ab2a7fa4a07f3a52a49e5197b347ba78c7cb8ed.
I have a commit that fails the test for TF<1.1.0.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/cleverhans/pull/308#issuecomment-341866452,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIbS9uUUWfcQcmAA2ZHF5JOFa0wFwDWcks5sy8-wgaJpZM4QRs6O
.

Ah that is possible, I will try to spend some time on this Friday afternoon. This has been a problem for a long time and at this point we cannot rely on these tests for what it's worth.

In the meanwhile, feel free to adjust the atol_fac if needed, I trust your judgment.

This causes the tests to fail on CPU on tf 1.0.1 on my MacBook Pro now.
File "/Users/goodfellow/cleverhans/tests_tf/test_mnist_tutorial_tf.py", line 46, in test_mnist_tutorial_tf
atol=atol_fac * 5e-3)
File "/Users/goodfellow/cleverhans/cleverhans/devtools/checks.py", line 29, in assertClose
assert np.allclose(x, y, *args, **kwargs), (x, y)
AssertionError: (0.94540000000000002, 0.96060000000000001)
-------------------- >> begin captured stdout << ---------------------

As a temporary fix, I made a new PR #310 to loosen tests for TF<1.1.0.


Naming convention in GPIO

All classes defined in gpio_cxx uses the following naming convention:
GPIO
but Output and OpenDrain use
GPIO_
As the project follows esp-idf coding style, these classes should be GPIOOutput and GPIOOpenDrain

@CyrilB1531 We appreciate your sense for detail and noticing the diversion from the IDF style-guide. In this case, however, the diversion has been a conscious decision that we made to increase the readability of the name. GPIO_Output is more readable than GPIOOutput in our opinion.
Among the goals of the style-guide are readability and consistency. In rare cases like this, these two goals collide and a decision has to be made what is more important.

Ok,
i will close the issue then

@CyrilB1531 We made this explicit in the IDF code style guide. The change should be visible when the IDF master branch is updated the next time.


Add MultiControlX() bloq with functional classical simulation

Adds MultiControlX() bloq which is similar to MultiControlPauli(), but it is a bloq, has a classical simulation, and does not use any And() gates. It allows for 0-controlled operations. It also decomposes into the XGate(), CNOT(), and Toffoli() when the number of controls matches those bloqs.

run check/format-incremental --apply to format the code and check/pylint to check pylint


WIP: Cassie Debugging CI

Description
Testing all kinds of changes bc this issue doesn't repro locally.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

this hung: @cicoyle
try rm-ing: DaprClientGrpcTelemetryTest, DaprClientGrpcTest, DaprPrev…
421c7ae
…iewClientGrpcTest
this proceeded tho by additionally rm-ing: DaprClientHttpTest, DaprPreviewClientHttpTest, GrpcChanne…
7beb235
…lFacadeTest

add back in DaprPreviewClientHttpTest does NOT hang - it passes the unit test step.
Try adding another one back

Just as a sanity test, Im going to re-trigger the prior comment build to ensure it still doesnt hang

oh thank goodness, that was repeatable (doesn't hang)
Time to add another one back

still magically hangs, sigh. ok. so it has something to do with`GrpcChannelFacadeTest


Update dependencies

Pull Request Test Coverage Report for Build 754

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

Totals

Change from base Build 732:
0.0%

Covered Lines:
1482

Relevant Lines:
1816

💛  - Coveralls


Add hard limits to processing

Add some basic hard limits to processing, limiting execution time, for example.

This was actually done in @4d337fc.


Make bootstrap happen faster when the node first starts up. Instead of waiting for the next block

We should be able to request to other nodes what the latest block is for quicker bootstrapping

Closing out this one, since it's a dupe of #3028


ImportError: No module named 'config_parser'

After installing when I am trying to run ngxtop its giving me error
Traceback (most recent call last):
  File "/usr/bin/ngxtop", line 7, in <module>
    from ngxtop.ngxtop import main
  File "/usr/lib64/python3.4/site-packages/ngxtop/ngxtop.py", line 77, in <module>
    from config_parser import detect_log_config, detect_config_path, extract_variables, build_pattern
ImportError: No module named 'config_parser'

@wonderer007 I ran into the same issue, and after fixing it realized that the fixes are actually already present inside github, the issue is really that the pypi version of this predates those fixes. If you clone this repo and install from there, it's working just fine. The version needs to get bumped to 0.0.3 and uploaded to pypi so that the pypi version actually works on install.

Same issue.  If it is fixed, when is it going to get bumped to 0.0.3?  The last version was in 2018 in march.  This is a critical error since it doesn't allow the user to run the app.  It immediately crashes.  Not everyone is patient enough to fork a branch and run it, or even know how to do that.

The year 2020, past 2 years. The issue is still there.

The year 2020, past 2 years. The issue is still there. #2

still here. 2020

still here 2020

i hate 2020

what?

Reply to this email directly, view it on GitHub
https://github.com/lebinh/ngxtop/issues/90#issuecomment-741487851, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACR5FZ6YWXYWWRMHRSRRMJ3ST3PJBANCNFSM4EX73WQQ
.

I also don't like 2020, so here the new version: https://pypi.org/project/ngxtop/0.0.3/


Update up-tck C++ test agent for 1.5.8 API

The C++ test agent for up-tck will need to be ported to the updated up-cpp API once it is released.
While porting to the 1.5.8 API, the implementation should be refactored some to make it easier to maintain. For example, pulling json manipulation out into common functions.

I would like to work on this.

I will take this one as David is working on socket transport

PR for this issues moved to different PR due to RC1 changes. Please change linked PR
https://github.com/eclipse-uprotocol/up-tck/pull/90


Adding invalidation on form

When someone enter wrong information in input box it should show some warning

Okay. Add form validation but only give warning. Don't stop the user to going to the next step as dummy input is consistently required. Although we can test using automation also, but nevertheless only display warning.


Add docs for operator : error handling - catch

Add docs for operator catch under group error handling
Below is minimum details needed

[ ] name
[ ] operatorType
[ ] signature
[ ] parameters
[ ] short description
[ ] Technical description
[ ] Visual example  (Marble diagram)
[ ] Real world example (Non-framework)
[ ] Related Operators

I can pick this up if no one's started on it?

@rarmatei Thanks for your support. But we have recently changed the repository to the rxjs repo https://github.com/ReactiveX/rxjs. I would really appreciate a pr on this, but please do it in that repository. If you have any questions on that, don't hesitate to ask me. Thank you

@JWO719 Oh, I see, thanks for pointing that out!
I'm definitely missing something but I can't seem to find where the operator docs are under the repo you linked to?
I'll submit a PR to this repo today if that's okay, since I have some free time - and I can transfer it to  the new one later in the week?


Delete CONTRIBUTING.md

going with a different contributing format go forward

@swift-ci test

@swift-ci test Windows

I suppose you can just force-merge it as it can't have caused the CI failure.


[WIP][CARBONDATA-152]Double min max difference compression issue.

1.While compressing maxdifference typecast with proper DataType
2.While writting we follow like
a. diff with max value , say d = max - x
b. multiply with 10^decimal i.e w = d*10^decimal
3.Similarly while reading we should follow like
a. divide written value 10^decimal i.e d = w/10^decimal
b. diff with max i.e x = max -d
4. There is problem in java with double subtraction
i.e double value = 3.141818 - 0.000610
java will result 3.1412080000000002
To avoid this we are converting to BigDecimal and then doing subtraction
5.Changed logic to calculate no of integer after decimal.

Could you please provide a test case? Especially the values MAX - MIN has little delta.

Retest this please

@Zhangshunyu  added test case

retest this please

LGTM

closing this issue as it is merged in incubator


Add Fedora (rpm) packaging

Following #17, I create a dedicated issue for fedora/rpm packaging.

[x] Initiate .spec file
[x] Building srpm and rpm
[x] Add completions files => Location: /usr/share/kubectx
[x] Create repo for sharing packages : https://copr.fedorainfracloud.org/coprs/cerenit/kubectx/
[x] Mirror gitlab code on github : https://github.com/nsteinmetz/kubectx-rpm

To enable repo:
(sudo) dnf copr enable cerenit/kubectx
(sudo) dnf install kubectx

Work in progress is available here and soon on Github too.

Upgraded for kubectx v0.7.0 just now

Now that you have a spec file, how hard would it be to build for EPEL?

By EPEL, you mean CentOS/RHEL ? I can add them as target on COPR and see what happens. I will try later today with the update to 0.7.1
It should work I think due to the minimal dependencies.

It worked seamlessly with EPEL8 and failed with EPEL7. So I only kept EPEL8.
As I don't use CentOS/RHEL, I'll not investigate further EPEL7. If you have some patch to make it work also on EPEL7, I'd be happy to integrate it.
Files for shell completions are in /usr/share/kubectx.

@nsteinmetz still not working? What's the error message?

Just tested here, my suggestion patch to fix the issue in EL7:
$ git diff
diff --git a/kubectx.spec b/kubectx.spec
index b667e32..04bf640 100644
--- a/kubectx.spec
+++ b/kubectx.spec
@@ -1,6 +1,6 @@
Name:       kubectx
Version:    0.7.1
-Release:    1%{?dist}
+Release:    2%{?dist}
Summary:    kubectx and kubens ease managing kubernetes contexts / name spaces
License:    ASL 2.0
@@ -9,6 +9,9 @@ Source0:    %{url}/archive/v%{version}.tar.gz
BuildArch:  noarch
+%prep
+%setup -q -n %{name}-%{version}
+
%description
kubectx is a utility to manage and switch between kubectl contexts.
kubens is a utility to switch between Kubernetes name spaces.
@@ -17,12 +20,12 @@ Requires: bash
%install
mkdir -p %{buildroot}/%{_bindir}
-mkdir -p %{buildroot}/%{_datadir}/licenses/%{name}
+mkdir -p %{buildroot}/%{_datadir}/licenses/%{name}-%{version}
mkdir -p %{buildroot}/%{_datadir}/%{name}
tar xzf %{_sourcedir}/v%{version}.tar.gz
install -p -m 755 %{name}-%{version}/kubectx %{buildroot}/%{_bindir}
install -p -m 755 %{name}-%{version}/kubens %{buildroot}/%{_bindir}
-install -p -m 644 %{name}-%{version}/LICENSE %{buildroot}/%{_datadir}/licenses/%{name}/
+install -p -m 644 %{name}-%{version}/LICENSE %{buildroot}/%{_datadir}/licenses/%{name}-%{version}/
install -p -m 644 %{name}-%{version}/completion/kubectx.bash %{buildroot}/%{_datadir}/%{name}/
install -p -m 644 %{name}-%{version}/completion/kubectx.fish %{buildroot}/%{_datadir}/%{name}/
install -p -m 644 %{name}-%{version}/completion/kubectx.zsh %{buildroot}/%{_datadir}/%{name}/
@@ -42,6 +45,9 @@ install -p -m 644 %{name}-%{version}/completion/kubens.zsh %{buildroot}/%{_datad
%{_datadir}/%{name}/kubens.zsh
%changelog
+* Tue Nov 26 2019 Douglas Schilling Landgraf<EMAIL_ADDRESS>- 0.7.1-2
+- Fix build for EL7
+

Mon Nov 11 2019 Nicolas Steinmetz<EMAIL_ADDRESS>- 0.7.1_1

Upgrade to 0.7.1
Add shell complets to /usr/share/kubectx/

$ cat /etc/redhat-release
CentOS Linux release 7.7.1908 (Core)
[douglas@nuc01 kubectx-rpm]$ rpmbuild -ba ./kubectx.spec
Executing(%prep): /bin/sh -e /var/tmp/rpm-tmp.yT6Wua

umask 022
cd /home/douglas/rpmbuild/BUILD
cd /home/douglas/rpmbuild/BUILD
rm -rf kubectx-0.7.1
/usr/bin/gzip -dc /home/douglas/rpmbuild/SOURCES/v0.7.1.tar.gz
/usr/bin/tar -xf -
STATUS=0
'[' 0 -ne 0 ']'
cd kubectx-0.7.1
/usr/bin/chmod -Rf a+rX,u+w,g-w,o-w .
exit 0
Executing(%install): /bin/sh -e /var/tmp/rpm-tmp.3hoHmq
umask 022
cd /home/douglas/rpmbuild/BUILD
'[' /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64 '!=' / ']'
rm -rf /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64
++ dirname /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64
mkdir -p /home/douglas/rpmbuild/BUILDROOT
mkdir /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64
cd kubectx-0.7.1
mkdir -p /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64//usr/bin
mkdir -p /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64//usr/share/licenses/kubectx-0.7.1
mkdir -p /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64//usr/share/kubectx
tar xzf /home/douglas/rpmbuild/SOURCES/v0.7.1.tar.gz
install -p -m 755 kubectx-0.7.1/kubectx /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64//usr/bin
install -p -m 755 kubectx-0.7.1/kubens /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64//usr/bin
install -p -m 644 kubectx-0.7.1/LICENSE /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64//usr/share/licenses/kubectx-0.7.1/
install -p -m 644 kubectx-0.7.1/completion/kubectx.bash /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64//usr/share/kubectx/
install -p -m 644 kubectx-0.7.1/completion/kubectx.fish /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64//usr/share/kubectx/
install -p -m 644 kubectx-0.7.1/completion/kubectx.zsh /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64//usr/share/kubectx/
install -p -m 644 kubectx-0.7.1/completion/kubens.bash /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64//usr/share/kubectx/
install -p -m 644 kubectx-0.7.1/completion/kubens.fish /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64//usr/share/kubectx/
install -p -m 644 kubectx-0.7.1/completion/kubens.zsh /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64//usr/share/kubectx/
/usr/lib/rpm/find-debuginfo.sh --strict-build-id -m --run-dwz --dwz-low-mem-die-limit 10000000 --dwz-max-die-limit 110000000 /home/douglas/rpmbuild/BUILD/kubectx-0.7.1
/usr/lib/rpm/sepdebugcrcfix: Updated 0 CRC32s, 0 CRC32s did match.
/usr/lib/rpm/check-buildroot
/usr/lib/rpm/redhat/brp-compress
/usr/lib/rpm/redhat/brp-strip-static-archive /usr/bin/strip
/usr/lib/rpm/brp-python-bytecompile /usr/bin/python 1
/usr/lib/rpm/redhat/brp-python-hardlink
/usr/lib/rpm/redhat/brp-java-repack-jars
Processing files: kubectx-0.7.1-2.el7.noarch
Executing(%license): /bin/sh -e /var/tmp/rpm-tmp.Iub8zG
umask 022
cd /home/douglas/rpmbuild/BUILD
cd kubectx-0.7.1
LICENSEDIR=/home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64/usr/share/licenses/kubectx-0.7.1
export LICENSEDIR
/usr/bin/mkdir -p /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64/usr/share/licenses/kubectx-0.7.1
cp -pr LICENSE /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64/usr/share/licenses/kubectx-0.7.1
exit 0
Provides: kubectx = 0.7.1-2.el7
Requires(rpmlib): rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(FileDigests) <= 4.6.0-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
Requires: /usr/bin/env
Checking for unpackaged file(s): /usr/lib/rpm/check-files /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64
Wrote: /home/douglas/rpmbuild/SRPMS/kubectx-0.7.1-2.el7.src.rpm
Wrote: /home/douglas/rpmbuild/RPMS/noarch/kubectx-0.7.1-2.el7.noarch.rpm
Executing(%clean): /bin/sh -e /var/tmp/rpm-tmp.JWfKTt
umask 022
cd /home/douglas/rpmbuild/BUILD
cd kubectx-0.7.1
/usr/bin/rm -rf /home/douglas/rpmbuild/BUILDROOT/kubectx-0.7.1-2.el7.x86_64
exit 0

@dougsland I was typing my answer when I saw your patch - thanks I'll add it in my package and let you know if copr package works :)

On fedora, I needed to add the whole line %{_datadir}/licenses/%{name}-%{version}/LICENSE  for licence file ; otherwise it claimed for an installed file which was not in the package at the end
%files
%license LICENSE
%{_bindir}/kubectx
%{_bindir}/kubens
%{_datadir}/licenses/%{name}-%{version}/LICENSE
%{_datadir}/%{name}/kubectx.bash
%{_datadir}/%{name}/kubectx.fish
%{_datadir}/%{name}/kubectx.zsh
%{_datadir}/%{name}/kubens.bash
%{_datadir}/%{name}/kubens.fish
%{_datadir}/%{name}/kubens.zsh

On fedora, I needed to add the whole line %{_datadir}/licenses/%{name}-%{version}/LICENSE  for licence file ; otherwise it claimed for an installed file which was not in the package at the end
%files
%license LICENSE
%{_bindir}/kubectx
%{_bindir}/kubens
%{_datadir}/licenses/%{name}-%{version}/LICENSE
%{_datadir}/%{name}/kubectx.bash
%{_datadir}/%{name}/kubectx.fish
%{_datadir}/%{name}/kubectx.zsh
%{_datadir}/%{name}/kubens.bash
%{_datadir}/%{name}/kubens.fish
%{_datadir}/%{name}/kubens.zsh

If it works, it works ;-) Glad to help. @nsteinmetz

@dougsland sucess ! :tada: Thanks a lot for your contribution !
@TJM if you're interested by EPEL7, it works from now thanks to @dougsland patch. EPEL8 was already working.

@nsteinmetz Thanks! Do we need to do anything to get these packages into the EPEL repos, or am I just too early, maybe they need to propagate?

hey @nsteinmetz let's bring the spec into the main repo, so more people is aware of the spec file. :)  (Just my 2 cents)

@TJM it won"t be in the official EPEL repo for now - you have to add the repo from here https://copr.fedorainfracloud.org/coprs/cerenit/kubectx/ - pick the one you expect to use to have the related repo file.
@dougsland you mean the main repo here or the main EPEL repo ? For the kubectx repo, @ahmetb said that he's not very interested in package management or expect a general solution to be implemeted (fpm based ?). At least, my repo could be mentionned as for other repo in the README section.
For the official EPEL repo, I don't know the process and I don't have a lot of freetime to invest in it nor a lot of interest to be honest. I use fedora only as laptop, for servers I'm more debian based. I use kubectx only on my laptop so...
But if you want to make the process, feel free to do so and take ownership on this.

@dougsland I mirrored repo on github => https://github.com/nsteinmetz/kubectx-rpm - I can add you as collaborator on it or you can fork it if you prefer.

@dougsland you mean the main repo here or the main EPEL repo ? For the kubectx repo,
@ahmetb said that he's not very interested in package management or expect a general solution to
be implemeted (fpm based ?). At least, my repo could be mentionned as for other repo in the
README section.

Okay, if @ahmetb is not interested in spec files, yeah we should have it in other github fork like you did. However, I would expect @ahmetb re-think about this topic, not only to get better support to RPM based distro in a single git tree but also more visibility to the project.

For the official EPEL repo, I don't know the process and I don't have a lot of freetime to invest in it
nor a lot of interest to be honest. I use fedora only as laptop, for servers I'm more debian based. I
use kubectx only on my laptop so...
But if you want to make the process, feel free to do so and take ownership on this.

I am not looking for ownership but I can contribute.

Cf https://github.com/ahmetb/kubectx/issues/184#issuecomment-552694846

For the record : I no longer maintain the rpm version - I switched to the krew plugin - if someone is interesed, repo is in archived mode and builts rpm will remain as long as possible but new builds will not be proposed.


Error with babel

I'm using Factory Girl with babel and jest and get this error when attempting to use this module.
TypeError: Adapter is not a constructor
  at Object.<anonymous> (node_modules/factory-girl-bookshelf/index.js:5:30)

The bookshelf adapter is included in factory-girl V4. No need for a
separate module.
On Sun., 2 Apr. 2017, 10:07 am Adrian<EMAIL_ADDRESS>wrote:

TypeError: Adapter is not a constructor
at Object. (node_modules/factory-girl-bookshelf/index.js:5:30)
I'm using Factory Girl with babel and jest and get this error when
attempting to use this module.
My code:
require('factory-girl')
require('factory-girl-bookshelf')()
// ...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aexmachina/factory-girl-bookshelf/issues/7, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAP-rO_6qYT6-WhsdmLFSxL7jRD9Law5ks5rrua_gaJpZM4Mwpzt
.

--
Simon Wade - Chief Technical Officer
jtribe - love your apps
13/243 Collins Street, Melbourne VIC 3000
<EMAIL_ADDRESS>| +61 422 777 262 <+61422777262> | http://jtribe.com.au

Okay, that's nice! I used this package as I got an error signifying that Bookshelf was trying to perform an update query.
factory.define('module', Module, buildOptions => {
  return {
    id: faker.random.uuid()
  }
})

await factory.create('module')

CustomError: No Rows Updated

Yes, that's because Bookshelf will perform an UPDATE if an id is present in
the model. Simply remove the id from your model and it should work.

Thanks!


[sql] Add support for transactions

Here's the method I use to execute a bunch of queries within a transaction:
    async executeTransaction(transaction: Transaction): Promise<void> {
        let wasError: boolean = false;
        try {
            this._db.execute('BEGIN;');
            await transaction();
        } catch (err) {
            wasError = true;
            this._db.execute('ROLLBACK;');
            throw err;
        } finally {
            if (!wasError) {
                this._db.execute('COMMIT;');
            }
        }
    }

Example:
        await db.executeTransaction(async () => {
            // A bunch of await db.execute() statements
        });

When I do this, and throw an error, rollback is executed, but rollback doesn't take place.

Hi, thanks for reaching out. Could you also include the output of the tauri info command and a bit more info about your database? sqlite, mysql, psql, etc?
Also, do you think you could come up with a runable minimal reproduction example?

Sure thing!
[✔] Environment
    - OS: Windows 10.0.22631 X64
    ✔ WebView2: 120.0.2210.121
    ✔ MSVC: Visual Studio Build Tools 2022
    ✔ rustc: 1.75.0 (82e1608df 2023-12-21)
    ✔ cargo: 1.75.0 (1d8b05cdd 2023-11-20)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 20.10.0
    - pnpm: 8.14.1
    - npm: 10.2.3

[-] Packages
    - tauri [RUST]: 2.0.0-alpha.20
    - tauri-build [RUST]: 2.0.0-alpha.13
    - wry [RUST]: 0.35.1
    - tao [RUST]: 0.24.0
    - @tauri-apps/api [NPM]: 2.0.0-alpha.13
    - @tauri-apps/cli [NPM]: 2.0.0-alpha.20

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: ../dist
    - devPath: http://localhost:1420/
    - framework: Svelte
    - bundler: Vite

I'm using Sqlite. I think the problem is that because I'm issuing multiple commands, they're being executed on different members of the connection pool.
I dug in to this a little, and I think sqlx has a transaction abstraction that's not available through the current tauri-plugin-sql APIs. I think that might be the issue.
I'll try to get a repro to you, but I'm kind of slammed at the moment.

If there is no mutex on the native side it is not safe to execute commands even when wrapped in a transaction. You should implement a mutex in your wrapper class. Here is an example of how I did it for RN but the idea should work here as well.

@Sahasrara looking at the Rust code and SQLX documentation I dont see implementation of transactions logic: https://docs.rs/sqlx/latest/sqlx/struct.Transaction.html
There is nothing like this implemented in the tauri sql plugin: https://github.com/tauri-apps/tauri-plugin-sql/blob/v1/src/plugin.rs#L202
@FabianLars is this something thats supported at all?

@FabianLars is this something thats supported at all?

Probably not, I don't know.
I was once told that it worked for someone without explicit support in the plugin but I doubt it tbh.

could we change the title of this issue and turn it into a feature request?

Any news here?

@FabianLars
I read that the plugin is based on sqlx.
I found the following blog. It says that you have to create a transaction with a special method in sqlx to get a transaction object and then you have to do your sql executions with this new transaction object.
https://jmoiron.github.io/sqlx/#transactions
It looks like that this method and the transaction object is not available in the plugin.
Can you please add it?

That would be a nice feature!

In simple cases, it's possible to perform the transaction in one call, e.g. db.execute("BEGIN; INSERT INTO ...; INSERT INTO ...; COMMIT;"). However, this doesn't work when not all commands are known in advance.
It seems to be relatively simple to expose sqlx::Transaction to JavaScript – I have done a small proof of concept at https://gist.github.com/RunasSudo/8d30798e7cd7bbddaaba0348c8ea8f58 for SQLite.
However, I suspect (though don't know how to verify) that this would lead to resource leaks if a transaction is begun in JavaScript but not committed or rolled back, e.g. due to an error in JavaScript. Currently I am using a FinalizationRegistry in JS to account for this but unsure if that is the best method. Ideally Tauri would be able to take care of all this behind the scenes.


feature: remove sendForList method and use Type class instead

Using a Type argument, rather then a separate sendForList method makes the client more flexible and reduces duplication within client implementations
Also changing headers type from Map<String, List<String>> to Map<String, Collection<String>> for better compatibility

is it better to add this on top of this generated PR?

I was actually just about to send RM Phellippe a message about this 😄
My plan was to first merge #84 (the auto-generated PR before that one) and then merge #86, which will then only have the auto-generated changes related to this PR, into here. Assuming this newly combined PR builds nicely then I'll merge it too.
I think that will keep the commit history cleaner, because it separates the 'normal' OAS changes from these changes which are changing the core interface. Wdyt @brumarqu-te @phpinhei-te?

is it better to add this on top of this generated PR?

I was actually just about to send RM Phellippe a message about this 😄
My plan was to first merge #84 (the auto-generated PR before that one) and then merge #86, which will then only have the auto-generated changes related to this PR, into here. Assuming this newly combined PR builds nicely then I'll merge it too.
I think that will keep the commit history cleaner, because it separates the 'normal' OAS changes from these changes which are changing the core interface. Wdyt @brumarqu-te @phpinhei-te?

fine by me, i think it's safe enough. was just trying to not merge untested code into main 😃


Merge release/2.6.0 into master

What does this PR do?
This PR does the merge of release/2.6.0 branch into master branch.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1863 (35d855b) into master (9350a50) will decrease coverage by 0.04%.
The diff coverage is 86.13%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1863      +/-   ##
==========================================
- Coverage   83.55%   83.51%   -0.04%
==========================================
  Files         467      470       +3
  Lines       16472    16568      +96
  Branches     2482     2485       +3
==========================================
+ Hits        13763    13836      +73
- Misses       2033     2051      +18
- Partials      676      681       +5

Files
Coverage Δ

...ore/src/main/kotlin/com/datadog/android/Datadog.kt
76.25% <ø> (ø)

...com/datadog/android/core/feature/event/JvmCrash.kt
100.00% <100.00%> (ø)

...m/datadog/android/core/feature/event/ThreadDump.kt
100.00% <100.00%> (ø)

...n/com/datadog/android/core/internal/CoreFeature.kt
88.35% <100.00%> (+0.09%)
:arrow_up:

...id/core/internal/metrics/BatchMetricsDispatcher.kt
98.61% <100.00%> (+0.08%)
:arrow_up:

...in/kotlin/com/datadog/android/log/LogAttributes.kt
0.00% <ø> (ø)

...dog/android/ndk/internal/DatadogNdkCrashHandler.kt
81.36% <100.00%> (+0.54%)
:arrow_up:

...in/com/datadog/android/log/internal/LogsFeature.kt
85.38% <100.00%> (+1.10%)
:arrow_up:

...android/log/internal/domain/DatadogLogGenerator.kt
97.92% <100.00%> (+0.16%)
:arrow_up:

...atadog/android/log/internal/domain/LogGenerator.kt
81.82% <100.00%> (+1.82%)
:arrow_up:

... and 21 more

... and 14 files with indirect coverage changes


Refactor webhooks

Coverage decreased (-0.12%) to 54.96% when pulling 499e48879a491fbd5cd345d7cbf1dd4f924f6b97 on refactor-webhooks into 939365b225493c98f7a1f22d0fe19c3298df4c45 on master.

test