Using LocalTracer across threads

Is the local tracer designed to be used across threads? If I run the following test I don't get the results I expect.
import java.net.InetAddress;
import java.net.UnknownHostException;
import java.nio.ByteBuffer;
import java.util.concurrent.CountDownLatch;
import java.util.concurrent.Executors;

import com.github.kristofa.brave.Brave;
import com.github.kristofa.brave.BraveExecutorService;
import com.github.kristofa.brave.InheritableServerClientAndLocalSpanState;
import com.github.kristofa.brave.LoggingSpanCollector;
import com.twitter.zipkin.gen.Endpoint;

public class Test2 {

    public static void main(String[] args) throws UnknownHostException {

        final int ip = ByteBuffer.wrap(InetAddress.getLocalHost().getAddress()).getInt();
        Brave brave = new Brave.Builder(new InheritableServerClientAndLocalSpanState(Endpoint.create("localhost", ip)))
                        .spanCollector(new LoggingSpanCollector()).build();

        CountDownLatch testLatch = new CountDownLatch(1);

        BraveExecutorService threads1 = new BraveExecutorService(Executors.newFixedThreadPool(3),
                brave.serverSpanThreadBinder());
        BraveExecutorService threads2 = new BraveExecutorService(Executors.newFixedThreadPool(3),
                brave.serverSpanThreadBinder());
        BraveExecutorService threads3 = new BraveExecutorService(Executors.newFixedThreadPool(3),
                brave.serverSpanThreadBinder());

        brave.serverTracer().setStateCurrentTrace(1000L, 1001L, null, "server-received-span");
        brave.serverTracer().setServerReceived();

        threads1.execute(new Runnable() {
            @Override
            public void run() {
                brave.localTracer().startNewSpan("thread-pool-1 local span", "first local span");
                try {
                    threads2.execute(new Runnable() {
                        @Override
                        public void run() {
                            brave.localTracer().startNewSpan("thread-pool-2 local span", "second local span");
                            try {
                                threads3.execute(new Runnable() {
                                    @Override
                                    public void run() {
                                        brave.localTracer().startNewSpan("thread-pool-3 local span", "third local span");
                                        try {
                                            brave.localTracer().startNewSpan("thread-pool-3 nested local span", "nested local span");
                                            try {
                                                testLatch.countDown();
                                            } finally {
                                                brave.localTracer().finishSpan();
                                            }
                                        } finally {
                                            brave.localTracer().finishSpan();
                                        }
                                    }
                                });
                            } finally {
                                brave.localTracer().finishSpan();
                            }
                        }
                    });
                } finally {
                    brave.localTracer().finishSpan();
                }
            }
        });

        try {
            testLatch.await();
            brave.serverTracer().setServerSend();
            threads1.shutdown();;
            threads2.shutdown();
            threads3.shutdown();
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
    }
}

I see the following
Oct 03, 2016 6:21:46 PM com.github.kristofa.brave.LoggingSpanCollector collect
INFO: {"traceId":"00000000000003e8","id":"00000000000003e9","name":"server-received-span","timestamp":1475544106367000,"duration":6000,"annotations":[{"timestamp":1475544106367000,"value":"sr","endpoint":{"serviceName":"localhost","ipv4":"<IP_ADDRESS>"}},{"timestamp":1475544106373000,"value":"ss","endpoint":{"serviceName":"localhost","ipv4":"<IP_ADDRESS>"}}]}
Oct 03, 2016 6:21:46 PM com.github.kristofa.brave.LoggingSpanCollector collect
INFO: {"traceId":"00000000000003e8","id":"045410f5ebdc31fe","name":"first local span","parentId":"00000000000003e9","timestamp":1475544106369000,"duration":504,"binaryAnnotations":[{"key":"lc","value":"thread-pool-1 local span","endpoint":{"serviceName":"localhost","ipv4":"<IP_ADDRESS>"}}]}
Oct 03, 2016 6:21:46 PM com.github.kristofa.brave.LoggingSpanCollector collect
INFO: {"traceId":"00000000000003e8","id":"7dd34f6e8b2a1e24","name":"second local span","parentId":"045410f5ebdc31fe","timestamp":1475544106372000,"duration":576,"binaryAnnotations":[{"key":"lc","value":"thread-pool-2 local span","endpoint":{"serviceName":"localhost","ipv4":"<IP_ADDRESS>"}}]}
Oct 03, 2016 6:21:46 PM com.github.kristofa.brave.LoggingSpanCollector collect
INFO: {"traceId":"00000000000003e8","id":"2ac208a84ef1832d","name":"nested local span","parentId":"824c98abf3e42c18","timestamp":1475544106372000,"duration":27,"binaryAnnotations":[{"key":"lc","value":"thread-pool-3 nested local span","endpoint":{"serviceName":"localhost","ipv4":"<IP_ADDRESS>"}}]}
Oct 03, 2016 6:21:46 PM com.github.kristofa.brave.LoggingSpanCollector collect
INFO: {"traceId":"00000000000003e8","id":"045410f5ebdc31fe","name":"first local span","parentId":"00000000000003e9","timestamp":1475544106369000,"duration":52072,"binaryAnnotations":[{"key":"lc","value":"thread-pool-1 local span","endpoint":{"serviceName":"localhost","ipv4":"<IP_ADDRESS>"}}]}

I would expect an entry for 'third local span'

What you are seeing is that BraveExecutorService was written when the
main concern was passing a server span. That's why it asks for
"serverSpanThreadBinder". So, it isn't really so much that LocalTracer
wasn't designed for threads, it was that BraveExecutorService wasn't
designed for local spans. Ex. localSpanThreadBinder is what you need
for what you are doing.
does this make sense?

Hey thanks for the reply. I think I understand each context server/local/client tracks it's own thread local.
Should something like this work?
import java.net.InetAddress;
import java.net.UnknownHostException;
import java.nio.ByteBuffer;
import java.util.concurrent.CountDownLatch;
import java.util.concurrent.Executors;

import com.github.kristofa.brave.Brave;
import com.github.kristofa.brave.BraveExecutorService;
import com.github.kristofa.brave.InheritableServerClientAndLocalSpanState;
import com.github.kristofa.brave.LoggingSpanCollector;
import com.twitter.zipkin.gen.Endpoint;
import com.twitter.zipkin.gen.Span;

public class Test2 {

    public static void main(String[] args) throws UnknownHostException {

        final int ip = ByteBuffer.wrap(InetAddress.getLocalHost().getAddress()).getInt();
        Brave brave = new Brave.Builder(new InheritableServerClientAndLocalSpanState(Endpoint.create("localhost", ip)))
                        .spanCollector(new LoggingSpanCollector()).build();

        CountDownLatch testLatch = new CountDownLatch(1);

        BraveExecutorService threads1 = new BraveExecutorService(Executors.newFixedThreadPool(3),
                brave.serverSpanThreadBinder());
        BraveExecutorService threads2 = new BraveExecutorService(Executors.newFixedThreadPool(3),
                brave.serverSpanThreadBinder());
        BraveExecutorService threads3 = new BraveExecutorService(Executors.newFixedThreadPool(3),
                brave.serverSpanThreadBinder());

        brave.serverTracer().setStateCurrentTrace(1000L, 1001L, null, "server-received-span");
        brave.serverTracer().setServerReceived();

        Span currentLocalSpan1 = brave.localSpanThreadBinder().getCurrentLocalSpan();
        threads1.execute(new Runnable() {
            @Override
            public void run() {
                brave.localSpanThreadBinder().setCurrentSpan(currentLocalSpan1);
                brave.localTracer().startNewSpan("thread-pool-1 local span", "first local span");
                try {
                    Span currentLocalSpan2 = brave.localSpanThreadBinder().getCurrentLocalSpan();
                    threads2.execute(new Runnable() {
                        @Override
                        public void run() {
                            brave.localSpanThreadBinder().setCurrentSpan(currentLocalSpan2);
                            brave.localTracer().startNewSpan("thread-pool-2 local span", "second local span");
                            try {
                                Span currentLocalSpan3 = brave.localSpanThreadBinder().getCurrentLocalSpan();
                                threads3.execute(new Runnable() {
                                    @Override
                                    public void run() {
                                        brave.localSpanThreadBinder().setCurrentSpan(currentLocalSpan3);
                                        brave.localTracer().startNewSpan("thread-pool-3 local span", "third local span");
                                        try {
                                            brave.localTracer().startNewSpan("thread-pool-3 nested local span", "nested local span");
                                            try {
                                                testLatch.countDown();
                                            } finally {
                                                brave.localTracer().finishSpan();
                                            }
                                        } finally {
                                            brave.localTracer().finishSpan();
                                        }
                                    }
                                });
                            } finally {
                                brave.localTracer().finishSpan();
                            }
                        }
                    });
                } finally {
                    brave.localTracer().finishSpan();
                }
            }
        });

        try {
            testLatch.await();
            brave.serverTracer().setServerSend();
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
    }
}

I still see unexpected results
Oct 04, 2016 10:29:36 AM com.github.kristofa.brave.LoggingSpanCollector collect
INFO: {"traceId":"00000000000003e8","id":"00000000000003e9","name":"server-received-span","timestamp":1475602176356000,"duration":6000,"annotations":[{"timestamp":1475602176356000,"value":"sr","endpoint":{"serviceName":"localhost","ipv4":"<IP_ADDRESS>"}},{"timestamp":1475602176362000,"value":"ss","endpoint":{"serviceName":"localhost","ipv4":"<IP_ADDRESS>"}}]}
Oct 04, 2016 10:29:36 AM com.github.kristofa.brave.LoggingSpanCollector collect
INFO: {"traceId":"00000000000003e8","id":"616d2f69eb5775ab","name":"second local span","parentId":"6ddc248b1cdbbcca","timestamp":1475602176361000,"duration":407,"binaryAnnotations":[{"key":"lc","value":"thread-pool-2 local span","endpoint":{"serviceName":"localhost","ipv4":"<IP_ADDRESS>"}}]}
Oct 04, 2016 10:29:36 AM com.github.kristofa.brave.LoggingSpanCollector collect
INFO: {"traceId":"00000000000003e8","id":"80e79baf39a9f834","name":"nested local span","parentId":"ac060c4ee47ccfca","timestamp":1475602176362000,"duration":54,"binaryAnnotations":[{"key":"lc","value":"thread-pool-3 nested local span","endpoint":{"serviceName":"localhost","ipv4":"<IP_ADDRESS>"}}]}
Oct 04, 2016 10:29:36 AM com.github.kristofa.brave.LoggingSpanCollector collect
INFO: {"traceId":"00000000000003e8","id":"6ddc248b1cdbbcca","name":"first local span","parentId":"00000000000003e9","timestamp":1475602176358000,"duration":517,"binaryAnnotations":[{"key":"lc","value":"thread-pool-1 local span","endpoint":{"serviceName":"localhost","ipv4":"<IP_ADDRESS>"}}]}
Oct 04, 2016 10:29:36 AM com.github.kristofa.brave.LoggingSpanCollector collect
INFO: {"traceId":"00000000000003e8","id":"616d2f69eb5775ab","name":"second local span","parentId":"6ddc248b1cdbbcca","timestamp":1475602176361000,"duration":46324,"binaryAnnotations":[{"key":"lc","value":"thread-pool-2 local span","endpoint":{"serviceName":"localhost","ipv4":"<IP_ADDRESS>"}}]}

I'll spend a little more time today trying to debug it.

I think you'll want to look at our learning test:
https://github.com/openzipkin/brave/blob/master/brave-core/src/test/java/com/github/kristofa/brave/LocalTracingInheritenceTest.java
Maybe some of these tests need to have javadoc describing them further. If
you find some notes of insight, please raise a PR with any doc or comment
updates!

Thanks for the reply. I think my initial problem was I was using an ExecutorService which was reusing threads.
I tried outputting the LocalTracingInheritenceTest into Zipkin and found the hierarchy was still incorrect.
I'm still looking into the issue but I think the mutable Deque stored in the InheritableThreadLocal are being modified which will reflect across threads.
I haven't used InheritableThreadLocal before but it looks like the child thread inherits the spans via reference to the parent which means changes in the parent thread will reflect in the child thread.
I will continue investigating tomorrow and post my findings.

I did a quick test and confirmed my above assumption. If you implement
 protected Deque<Span> childValue(Deque<Span> parentValue) {
 }

and clone the Deque the trace hierarchy looks correct.

Thanks for digging into this!
Can you add this and/or make a failing test as a pull request? That way it
is easier to discuss with folks who may not be reading this issue.
In the javadoc of the test, please explain what's going on.

Sure I was going to work on it tonight. I'll submit a pr tomorrow.

I've added a pull request with tests that fail. I will look at a solution later tonight. Wanted to get the tests in so others can provide any feedback.

CurrentTraceContext in Brave 4.3 handles all of this now


fix: PinInput: Add background-color to fix dark mode

🔗 Linked issue
#894

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Adds the bg-background class to the Pin-input input fields, which styles it similar to regular Input components, and which fixes the background color in dark mode.
Stackblitz demo without this fix applied
Resolves #894
📸 Screenshots (if appropriate)
Before:

After:

📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

PIN Input demo is working fine in the documentation because of
html.dark {
    color-scheme: dark;
}

The main shadcn-ui Input OTP also has transparent background for the input element
If you have dark-mode feature in your app you must use color-scheme property to indicate color schemes of your app

Ah; thank you for that insight, that indeed works as it should.
Btw: I've setup the dark mode according to the description in the shadcn docs and I did not spot this info there.


zone.js:355 Unhandled Promise rejection: Template parse errors

Hi,
I am using modal code as per the DOC, but getting below exception.

'modal-content' is not a known element:

If 'modal-content' is an Angular component, then verify that it is part of this module.
If 'modal-content' is a Web Component then add "CUSTOM_ELEMENTS_SCHEMA" to the<EMAIL_ADDRESS>of this component to suppress this message. ("

<sm-modal title="MyForm" #MyForm>
[ERROR ->]
<form class="ui large form" (ngSubmit)="onSubmit()" #loginForm="ngForm" nova"): BannerHeaderComponent@26:4 ; Zone:  ; Task: Promise.then ; Value: Error: Template parse errors:(…) Error: Template parse errors:
'modal-content' is not a known element:

If 'modal-content' is an Angular component, then verify that it is part of this module.
If 'modal-content' is a Web Component then add "CUSTOM_ELEMENTS_SCHEMA" to the<EMAIL_ADDRESS>of this component to suppress this message. ("

+1

Same here, is it something new with angular?

Yeah, Dont know. Nothing mentioed in DOC.

Anyway the CUSTOM_ELEMENTS_SCHEMA fix works, but probably it's just a way to ignore that warning

And also how do I change the modal size? I needed small login modal.

Probably with a class on the modal

Can you please provide any sample block code?

<sm-modal title="Hello from Modal" class="small large" #myModal>

Thanks, the small is working.
Is anyone knows how to open the modal from angular conponent method. I mean I can provide a method name in click event but what code I need to write to open any modal ?

In your template
<sm-modal #editSupplierModal>

in your component
import { SemanticModalComponent } from 'ng-semantic';

@ViewChild('editSupplierModal') editSupplierModal: SemanticModalComponent;

...

this.editSupplierModal.show();

Thanks, Its working.

How to pass data in modal?
I want to show dynamic data in modal. Is there any way to pass variables while showing the data.

@anil1712 the modal is in the same scope as the main template, so you can use the same values you use in the component that includes the modal, otherwise, if you create a custom component for the modal, you pass data the same way you do with any other component

Hi @alex88 ,
I have created new component for modal. So I need to share data from one component to another. I am using @angular/common": "~2.1.0. So can you please share me any link for send me any block of code to pass data from one component to another. Actually I am pretty new for angular2. Please help me to out  of it.
Thanks,

Me too, so I don't really know what's the best way, maybe using input/outputs https://angular.io/docs/ts/latest/cookbook/component-communication.html


Tuple to seq

I believe it will be useful (and it already useful in jackson module)

Hi Ruslan,
we should find a general way to do it. I re-opened #692 and will work on it.
Please keep this PR open until we see in which direction #692 evolves.
Greets
Daniel

I fiddled around for hours just to learn (again) that there is no good way to let extend Tuple the Iterable interface. Also inheriting other methods is overkill.
I committed an impl of the Tuple.toSeq method (which I regret now because of the merge conflicts). Could you please commit your tests for that method? I think the simplest way is to copy your tuples test of the Generator.scala to a fresh copy of master...
Thanks for your patience with me!

done

thx!!


opt format long in explain

Proposed changes
in explain, "1500000" will be printed as "1,500,000"
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.89 seconds
load time: 450 seconds
storage size:<PHONE_NUMBER>5 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221116124116_clickbench_pr_47201.html


Feature request: Sort by column

Hi,
Awesome tool, thanks for developing it!
I would love to see an option to reorder columns in the html flexibly by a column. I have multiple numeric columns with thousands of rows, and in this case without sorting datavzrd becomes significantly less useful.

Hi @stefanpeidli ,
Thanks for reaching out, and thanks for the kind words! We're glad you find datavzrd useful.
Regarding your suggestion to reorder columns in the HTML output flexibly, we understand the convenience this would provide, especially when dealing with large datasets. However, at the moment, we don't have plans to implement such a feature. The functionality would require a significant amount of work, and we're focused on other priorities for now.
As an alternative, we recommend using existing command-line tools to reorder columns beforehand. You might find xsv or csvkit helpful for this purpose. These tools allow you to manipulate CSV data in various ways, including sorting, reordering, and more.
We appreciate your feedback and encourage you to keep suggesting features and improvements. If you have any other questions or need further assistance, please let us know.

Understandable, thanks for the explanation and the pointers!


Update Go, Bazel, rules-go, and gazelle to the latest versions

All of the packages mentioned in the title are outdated. Especially Bazel, which is now available in version 3.0.0 instead of 1.2.0. Updating these packages fixes a number of issues, especially regarding compilation of SCION for Android smartphones.

What exactly is Bazel 3.0 vs 1.2 fixing for you?

I had several issues with Android compilation which were due to the use of Bazel 0.26.1 in SCION's previous versions, where upgrading to 3.0 helped a lot (but caused other issues in turn, e.g. nogo crashing, which had to be addressed separately). So, upgrading 1.2 to 3.0 does not benefit me as such, but maybe others in the future.
Here are some of my issues, which are partly due to old versions:
bazelbuild/rules_go#195, bazelbuild/rules_go#2089, bazelbuild/rules_go#2172,
bazelbuild/rules_go#2371, scionproto/scion#3325, Catfriend1/syncthing-android#370, golang/go#29674. For example, one problem was a misaligned TLS segment in Go's compilation output, preventing Android to start the executable. This is due to Go 1.11 and fixed in SCION v0.5.0 by upgrading to Go 1.13. Keeping all versions up-to-date helps prevent such issues in the first place.

With https://github.com/scionproto/scion/pull/3782 merged, we are at least on Bazel on a very recent version, other stuff you mentioned is also quite recent. We constantly monitor updates in libs around, sometimes they go a bit out of date but we try to keep up to date with them. Just blindly updating has bitten us in the past, so if there is no (strong) reason to update we will keep current versions for a while even if there are newer version available.


Add git precommit with clang-format

This follows up to

https://github.com/skupperproject/skupper-router/pull/504

There are two possibilities, either write the hook from scratch, or use one of the hook frameworks

https://github.com/sds/overcommit
https://github.com/pre-commit/pre-commit

https://github.com/pre-commit/pre-commit only works on the file-level, that is, check succeeds only if the whole file is properly formatted. What we want (for now) is to only check formatting of the changed lines.


Axios bump to 1.2.1  Throws an error : "FATAL  Request to site https://example.com threw an unhandled exception. Please check the URL is valid. unexpected end of file" on some sites

Describe the bug
The latest 0.4.4 release throw an error on some valid sites (not all).
› Testing Site https://www.example.com/ is valid.                                                                                unlighthouse 15:36:01

  at Function.AxiosError.from (node_modules/unlighthouse/node_modules/axios/lib/core/AxiosError.js:89:14)
  at BrotliDecompress.handleStreamError (node_modules/unlighthouse/node_modules/axios/lib/adapters/http.js:489:29)
  at BrotliDecompress.emit (events.js:412:35)
  at emitErrorNT (internal/streams/destroy.js:106:8)
  at emitErrorCloseNT (internal/streams/destroy.js:74:3)
  at processTicksAndRejections (internal/process/task_queues.js:82:21)

 FATAL  Request to site https://www.example.com/ threw an unhandled exception. Please check the URL is valid. unexpected end of file

This is related to bumping the axios dependency for the core "axios": "^1.1.3", which defaults to the latest v1.2.1. The error has also been documented here https://github.com/axios/axios/issues/5346.
Adding encoding headers to the function as suggested inhttps://github.com/axios/axios/issues/5346#issuecomment-1340241163 fixes this. I think also downgrading to axios 1.1.3 as suggested on the thread would fix it.
async function fetchUrlRaw(url) {
  try {
    const response = await axios.get(url, {
      headers: { "Accept-Encoding": "gzip,deflate,compress" },
      httpsAgent: new https.Agent({
        rejectUnauthorized: false
      })
    });
    const redirected = response.request.res.responseUrl && response.request.res.responseUrl !== url;
    const redirectUrl = response.request.res.responseUrl;
    if (response.status < 200 || response.status >= 300 && !redirected) {
      return {
        valid: false,
        redirected,
        response,
        redirectUrl
      };
    }
    return {
      valid: true,
      redirected,
      response,
      redirectUrl
    };
  } catch (e) {
    return {
      error: e,
      valid: false
    };
  }
}

Version 0.4.2 and below are working fine.
Reproduction
No response
System Info
System:
  OS: macOS 12.2.1
  CPU: (10) x64 Apple M1 Pro
  Memory: 230.56 MB / 32.00 GB
  Shell: 5.8 - /bin/zsh
Binaries:
  Node: 14.17.6 - ~/.asdf/installs/nodejs/14.17.6/bin/node
  npm: 6.14.15 - ~/.asdf/plugins/nodejs/shims/npm
Browsers:
  Chrome: 108.0.5359.124
  Safari: 15.3

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Hey @MamlukiSn
Thanks for the issue and the research behind it. Have pushed up a fix, just going to lock axios, for now, to make life easier.

Thanks for the update @harlan-zw .


Get the thumbnail full path

Hi guys,
I'm using sorl-thumbnail in my project and i'm generating a PDF file from my website, but i have this problem: https://github.com/fdemmer/django-weasyprint/issues/14
I solved it partially adding the full path, but i need load it from the framework and not manually from my hand.
What can i do?

Have you set MEDIA_URL in Django settings?  I believe the URL is just MEDIA_URL added to the relative path stored in the database field.  If you need a full URL (such as when the media is hosted on a separate server) then you can make MEDIA_URL a full URL.

Yes, i set MEDIA_URL, do i need a full URL in this case?

If the issue is just with that PDF generation and not the rest of the site then maybe you don't and you just need to deal with the PDF creation.  I don't know the specifics of your setup.  This is definitely not an issue with sorl-thumbnail, though.
Maybe you need to add a <base> tag to the document so the PDF generator can figure out the full paths?  The issue seems to be with the PDF generator not being able to handle relative paths.

I think yes, it is the problem, with full paths I haven't errors, so, how can I handle it?


Change Custom Hooks

use functional updates and destructure the event values before the functional update

Hi there!
Does the old code not work?

works fine but i think it would be better. and thank you for creating this guide.

Ah I see. I need to change it in multiple places, and I'll do it on the next update. Let's leave this open for now.


Support for CentOS stream 9

Adding support for CentOS Stream 9 as SEAPATH Hosts.
This is meant to share code with the Debian variation.
For CentOS Stream installations you can use my reference Kickstart file:
https://github.com/yairpod/SEAPATH_CentOS_Stream_Kickstart

Hi @yairpod. Thanks for your pull request.
So I guess that we would need to add https://github.com/yairpod/SEAPATH_CentOS_Stream_Kickstart as a repo in LFEnergy SEAPATH.
As you know, we have an intensive CI approach. I would like to have a CI for this SEAPATH flavor so that we can monitor the all the tests succeed.
Do you think you could provide a running CI @yairpod ?
By the way did you check that all the tests are green with your distro ?
Also we should consider an other name for "debian-main". We would have 3 different OS of SEAPATH: Yocto, Debian and centos.
The best would be to merge all those flavors in a common Ansible branch. As a first step, we would need to rename debian-main into precompiled-main for instance.
@insatomcat @dupremathieu what is your point of view ?
Best,
Eloi

I'd prefer "debiancentos" over "precompiled".
And yes, this is great work but we would need some form of engagement that there would be someone to maintain centos compatibility long term, including maintaining the infrastructure to run CI testing.
Seapath is quite a dynamic project, it will require work to make sure all changes work on centos and this is something noone on the current team has time for. Is it something you or Redhat is ready to do?
If yes I think we need to discuss this during next TSC.


Best pattern for chaining dependent actions (Question)

react: 0.14.3
react-router: 1.0.0
react-redux: 4.0.0
redux: 3.0.4
redux-thunk: 1.0.0
redux-promise-middleware: 2.2.4
I have everything working together great, but I'm reaching a point in development where it would be beneficial to start chaining a series of actions together.  I know this has been covered in the README, #8, #24, and lots of other places, but I simply can't get any of them to work, so I must be going about it completely wrong.
Here's an example use case:

Create - creates a new record via async service call, store in {record: {}} state
Start - wait for the record to be created and then update the record in state to start a process
Redirect - redirect to a new page beginning the process

The Create and Start are individual actions that I'd like to use in a single new action if the user knows that they want to start the process when they create the record.
actions.jsx
export function create(name) {
    return {
        type: CREATE,
        payload: {
            promise: axios.post(`${apiRoot}/create`, {name: name})
        }
    }
}

export function start() {
    return {
        type: START,
        payload: {
            promise: Promise.resolve()
        }
    }
}

reducer.jsx
        case `${CREATE}_PENDING`:
        ...
        case `${CREATE}_SUCCESS`:
        ...
        case `${CREATE}_ERROR`:
        ...
        case `${START}_PENDING`:
        ...
        case `${START}_SUCCESS`:
        ...
        case `${START}_ERROR`:

Individually, both actions (action creators) do exactly what I want them to do.  However, I never seem to be able to get the second action to dispatch after the first one has finished.
I won't list all of the things I've tried, but here's an example of a failed attempt from #24.
export function createAndStart(name) {

    return (dispatch) => {
        dispatch({
            type: CREATE_AND_START,
            payload: {
                promise: new Promise((resolve, reject) => {
                    dispatch(create(name))
                        .then(() => {
                            dispatch(start());
                        })
                        .then(() => {
                            dispatch(push('/start'));
                        })
                })
            }
        });
    };
}

In this example, I see the createInspection action dispatched.  I see it flow through each reducer state _PENDING then _SUCCESS, but I never see anything fall into the next .then()
I'm certain that I'm doing something stupid, but nothing has worked for me.  The example in the README fails for me initially because dispatch isn't the first argument I see, action is.  But even if I change that, the second action never gets dispatched.
bad
Promise.resolve((dispatch, getState) =>

good
Promise.resolve((action, dispatch, getState) =>

I appreciate any help that folks can offer.  Thank you!

I'm busy for the rest of the week. I'll get back to you this weekend when I have some more time to read through your question to give you a worthwhile answer!

Thanks!  Sorry to be a bother, I really want to learn the/a proper way to do things.  I'm grateful for the willingness to help.

I was looking for a way to do this type of thing as well. This works, but was wondering if there was a better way:
return ({fetch, settings: {firebaseUrl}}) => {
    return ({
      type: 'LOAD_PROFILE_USER',
      payload: {
        promise: fetch(`${firebaseUrl}/users/${slug}.json`)
          .then(response => response.json())
          .then(response => (users.viewer && users.viewer.slug === response.slug) ?
            Promise.resolve(loadProfileSettingsForm(response)) :
            Promise.resolve(true))
          .catch(err => {
            throw err;
          }),
      }
    });
  };

Also, if there is some way to do this with aysnc/await?

Maybe this?
export function createAndStart(name) {
  return (dispatch) => {
    dispatch({
      type: CREATE_AND_START,
      payload: {
        promise: new Promise(async (resolve, reject) => {
          await dispatch(create(name)).payload.promise
          await dispatch(start()).payload.promise
          dispatch(push('/start'))
        })
      }
    });
  };
}

Have a look at our tests to get more of an idea of the return from dispatch calls when using a promise-middleware-action

@tomatau Thank you!!  That worked perfectly.   I truly appreciate the help, and I'll be sure to dig into the tests to learn more.
@rkoberg Hopefully this example using async/await helps you too...

Sure, but I just moved it to a separate async function (example below). Using Este, it takes care of dispatch for actions:
https://github.com/este/este/blob/master/src/common/configureStore.js#L23
https://github.com/este/este/blob/master/src/common/lib/injectDependencies.js
However, it just uses the dispatch function (not always a promise). What is the benefit of appending .payload.promise? I want access to the data to transform for the reducer. And I am not clear what you are doing with resolve(push('/start')). What is that for?
Here is what I am doing:
async function _login(firebase, fields) {
  try {
    const authData = await firebase.authWithPassword(fields.toJS());
    const userData = await getUserFromAuthData(firebase, authData);
    return {authData, userData};
  } catch (err) { //eslint-disable-line brace-style
    throw err;
  }
}

@rkoberg the resolve(push('/start')) bit was just for my specific example.  I needed to redirect upon completion of the second action.

@rkoberg

What is the benefit of appending .payload.promise?

The return from dispatch is an action object with the promise available at payload.promise, this allows you to await for the promise to be resolved, following @baronnoraz question and code.

OK. I have setup the firebase functions to return a promise. authWithPassword resolves into authData first. Then, async getUserFromAuthData has an await inside of it (to another Promisified firebase function). It resolves and all the data is sent in the return to the reducer. In other words, I don't seem to need .payload.promise -- wondering if I am somehow doing something wrong that currently results in the correct result.

The example I posted was using actionCreator functions inside an actionCreator and awaiting on dispatch calls, check the OP for the API of the functions used in the example code I posted.

@rkoberg I am going to close this since we resolved the OP's question. Would you mind creating a new issue? I'm happy to help.

I think we are good.

One more example:
/*
 * redux-thunk + redux-promise-middleware + redux-actions
 */
const _createAndStart = createAction(CREATE_AND_START,
  async (name, { dispatch }) => {
    await dispatch(create(name));
    await dispatch(start());
    await dispatch(push('/start'));
  });

export const createAndStart = (name) =>
  (dispatch) => dispatch(_createAndStart(name, { dispatch }));


spin_wait::spin_one() is incorrectly yielding to os first and then spinning

https://github.com/lewissbaker/cppcoro/blob/99bb7f80b3e8156ba568e6a113415dce7a38ae08/lib/spin_wait.cpp#L48
There is a bug in this conditional. It should be reversed so that it spins first and then yields to the OS after a small number of spins.
Thanks to @loopunit for finding this one.
https://github.com/loopunit/cppcoro/commit/242db2b0f6d2c485e3f869b284ff1962071b1eba

Fixed by 7ac4be2c0eb825ce4ea92f6496255065ca6fc42f


Nonce is not added to CSP header in rails 5

Steps to reproduce

On a brand new rails <IP_ADDRESS> project add the secure_headers gem.
Create an initializer including SecureHeaders::Configuration.default
Create an action and view including:

Hello

<%= nonced_javascript_tag do %>
  alert("world!");
<% end %>

Expected outcome
The script tag is rendered including a nonce attribute and a nonce-X origin is added to the Content-Security-Policy header.
Actual outcome
The script tag correctly includes a nonce attribute, but there's no nonce origin in the Content-Security-Policy HTTP header, so the browser refuses to execute the inline script.

Ouch. I'll take a look at this shortly.

@afcapel can you provide your SecureHeaders::Configuration.default block? I can only reproduce this if I do not have a script-src value set. If that's the case, this bug isn't as awful as I had thought :smile:
While it's always better to supply the script-src value rather than falling back to default-src, the library is supposed to handle this gracefully and copy over the default-src and include the nonce.

Fix released in 3.4.1


[BitwiseCopyable] Require Copyable and Escapable.

Update the protocol to extend Copyable/Escapable, depending on the flags the stdlib is built with.
Emit targeted diagnostics to prevent a type from conforming to _BitwiseCopyable while being noncopyable/nonescapable for now.

@swift-ci please test

@swift-ci please test


[Auctions2] Add  component

Addresses https://artsyproduct.atlassian.net/browse/GRO-246
This finishes up the various MyBids UI. Will follow-up with a PR that contains tests.

Looks good @damassi! I've got some questions up, but nothing blocking. I'll go ahead and merge.


Job Detail refactor

This PR refactors Job Detail, migrates styling to two-column flex-based adaptable layout, uses Cards to section page content, adds links to output files
Preview:

Refactor completed, ready for review. Please see latest preview in the PR description


@flowmap.gl/data latest vertion not match

Since I run pnpm upgrade to the latest @flowmap 8.0.0-alpha27, the error comes:
deck: initialization of CompositeLayer({id: 'flowmap-layer'}): dataProvider.updateLayersData is not a function TypeError: dataProvider.updateLayersData is not a function

basemap render sucessfully, but the flows hidden.
So I check the lock file, I found that @flowmap.gl/layers version is up to 27, but the @flowmap.gl/data version is 8.0.0-y.14. Downgraded to version 26 and everything works fine. So I guess this is the problem here 👀?

If the versions didn't match in the lock file, that's probably because you had installed different versions. Try to make sure that both versions you install are the same:
pnpm add<EMAIL_ADDRESS><EMAIL_ADDRESS>

When I try pnpm add<EMAIL_ADDRESS><EMAIL_ADDRESS>I got:
 ERR_PNPM_NO_MATCHING_VERSION  No matching version found for<EMAIL_ADDRESS>
This error happened while installing a direct dependency of /Users/naigai/Documents/code/flow_vis

The latest release of @flowmap.gl/data is "8.0.0-alpha.0".

Other releases are:
  * next: 8.0.0-alpha.27

If you need the full list of all 29 published versions run "$ pnpm view @flowmap.gl/data versions".

Then I run: pnpm view @flowmap.gl/data versions:
[
  '8.0.0-alpha.0',  '8.0.0-alpha.1',
  '8.0.0-alpha.2',  '8.0.0-alpha.3',
  '8.0.0-alpha.4',  '8.0.0-alpha.5',
  '8.0.0-alpha.6',  '8.0.0-alpha.7',
  '8.0.0-alpha.8',  '8.0.0-alpha.9',
  '8.0.0-alpha.10', '8.0.0-alpha.11',
  '8.0.0-alpha.12', '8.0.0-alpha.13',
  '8.0.0-alpha.14', '8.0.0-alpha.15',
  '8.0.0-alpha.16', '8.0.0-alpha.17',
  '8.0.0-alpha.18', '8.0.0-alpha.19',
  '8.0.0-alpha.20', '8.0.0-alpha.21',
  '8.0.0-alpha.22', '8.0.0-alpha.23',
  '8.0.0-alpha.24', '8.0.0-alpha.25',
  '8.0.0-alpha.26', '8.0.0-alpha.27',
  '8.0.0-y.14'
]

After I edit the dependencies in package.json
  "dependencies": {
<EMAIL_ADDRESS>"8.0.0-alpha.27",
<EMAIL_ADDRESS>"8.0.0-alpha.27",
  },

and pnpm install, works fine.
The issue I mentioned should only occur when using pnpm upgrade to upgrade all dependencies to the latest version.

Ah I see, it considers<EMAIL_ADDRESS>as the latest, that's why. I will try to publish the next version as 8.0.1. Hopefully, that will solve it. Thanks for letting me know!

Should be solved now


India: Diwali and Holi holidays update

Coverage remained the same at 100.0% when pulling b4d3be473360bdc086028b43b0ccb799d402b6f1 on KJhellico:upd-india into 755401177bcb6bc3d0b80469ddaebbdd017fa147 on dr-prodigy:beta.

👍


Refactor the read and write functions in Unix and SSL NetVC

There are redundant wrapper functions in UnixNetVC and the design is inconstant between read and write, and also between UnixNetVC and SSLNetVC.
This PR is a first step for further refactoring/abstraction. It removes those redundant functions and aligns the design. I also removed a few parameters that don't need to be passed around. There should be no change in the flow.
With this change, we should be able to move the TLS related code into SSLNetVC from UnixNetVC like load_buffer_and_write.
Before
UnixNetVConnection
UnixNetVConnection::net_read_io -> ::read_from_net
UnixNetVConnection::net_write_io -> ::write_to_net -> ::write_to_net_io -> UnixNetVConnection::load_buffer_and_write
UnixNetVConnection::netActivity -> ::net_activity

SSLNetVConnection
SSLNetVConnection::net_read_io
UnixNetVConnection::net_write_io -> ::write_to_net -> ::write_to_net_io -> SSLNetVConnection::load_buffer_and_write

After
UnixNetVConnection
UnixNetVConnection::net_read_io
UnixNetVConnection::net_write_io -> UnixNetVConnection::load_buffer_and_write
UnixNetVConnection::netActivity

SSLNetVConnection
SSLNetVConnection::net_read_io
UnixNetVConnection::net_write_io -> SSLNetVConnection::load_buffer_and_write

stale_response failed.

[approve ci autest]

Jenkins failed to fetch git repo.

[approve ci autest]


S6: updated to 2.1

Notes to self
https://github.com/just-containers/s6-overlay/releases/tag/v<IP_ADDRESS> (May 13, 2020) to https://github.com/just-containers/s6-overlay/releases/tag/v<IP_ADDRESS> (Oct 19, 2020)
$ git log v<IP_ADDRESS>..v<IP_ADDRESS> --oneline
8b8d679 (tag: v<IP_ADDRESS>) bump version in README
e70ed90 version bumps
ff2b608 Merge pull request #305 from just-containers/no-root
13e844c (tag: v<IP_ADDRESS>, origin/no-root) use pre-init v1.0.2 (only creates /var/run/s6 dir)
d91bd15 add some support for USER directive
8fa9443 no-catchall: create supervise dir at runtime

This is interesting: https://github.com/just-containers/s6-overlay/commit/d91bd15 .. need to check whether this has downstream impact to what we do today.

@adobejmong its definitely great to see the official/unofficial support of running in this mode. More than we have had to hack around.

I was able to read through the s6-overlay changes and I think this looks good and I'd like to try it out so :shipit: @bryanlatten


Only Time Played not working properly

When having only time played it looks glitched, but when I activate something else that adds something in the same place (for example, last played) it looks correct.

Fixed in master


Refactor architecture specific configurations

In local, it works fine for testing command in travis.yml:
make config ARCH=arm
make VERBOSE=1
make check
make clean
make config ARCH=riscv
make VERBOSE=1
make check
sh .ci/check-format.sh

I'm not sure if it's bug of Travis-CI's or mine.

I'm not sure if it's bug of Travis-CI's or mine.

CI reported:
out/shecc-stage1.elf -o out/shecc-stage2.elf src/main.c
out/shecc-stage1.elf: 1: Syntax error: word unexpected (expecting ")")

It seems that file out/shecc-stage1.elf was executed without QEMU. Can you check the environment variable(s)?

I'm not sure if it's bug of Travis-CI's or mine.

CI reported:
out/shecc-stage1.elf -o out/shecc-stage2.elf src/main.c
out/shecc-stage1.elf: 1: Syntax error: word unexpected (expecting ")")

It seems that file out/shecc-stage1.elf was executed without QEMU. Can you check the environment variable(s)?

Thanks, I'll figure out later.


From PCORNet Common Data Model for "About Me Survey"

01=American Indian or Alaska Native
02=Asian
03=Black or African American
04=Native Hawaiian or Other Pacific Islander
05=White
06=Multiple race
07=Refuse to answer
NI=No information
UN=Unknown
OT=Other

This will split Race and Ethnicity into two questions for this survey.

@krand7 let me know when this update happens, since I will have to update answer migration mappings.

@pmanko sounds good! All questions have been approved by @ekontos, and are waiting on Susan's approval now. Should be ASAP

@pmanko The new ethnicity question appeared when I ran this on my localhost, but the answer options did not appear. This had happened before and I think you resolved it -- do you know off-hand what causes it?

@krand7 I hopefully solved this issue with commit 3c39a7817139b9b2ea2789329cd6287eadea5de5 - it had to do with naming conflicts between the fixtures and survey files.

Reloaded my database and it worked fine on my computer!


how do you fix "source/psychlua/HScript.hx:317: characters 89-96 : Type not found : TeaCall"

Describe your problem here.

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

Replace all instances of TeaCall with Tea

haxelib install SScript 8.1.6

none of those worked


[Feature] removeAllColumns

When I load another excel file, I need to clear the whole table.
now there is stateManager.removeAllRows();, but no removeAllColumns.

@sinkcup I would looking at https://github.com/doonfrs/pluto_grid_plus for a maintained pluto_grid_version

@sinkcup Please create this issue in pluto_grid_plus. and tag me i will create the feature for yo


Syncing from upstream zalando/patroni (master)

bt_gitbot

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.008%) to 99.852%

Totals

Change from base Build<PHONE_NUMBER>:
0.008%

Covered Lines:
12107

Relevant Lines:
12125

💛  - Coveralls


inv_depth variable

What does inv_depth variable present in the constructor of NeWCRFs indicate and where is it being used?

In some cases, someone may want to try inv_depth output to see if the results would be better. Then you can output the inv_depth by setting inv_depth to True.


Include permission validation step to programmatically cloned repos

Reference Issues or PRs
fixes #2218

What does this implement/fix?
Put a x in the boxes that apply

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[x] Did you test the pull request locally?
[ ] Did you add new tests?

Any other comments?

Before the new permission function check: (in this example, we are looking for the pre-initialized examples/test dir)

After this PR changes: (triggered by removing .firstrun)

Ps. I also tested removing the folder, all working (user had the Developer role to avoid all rights conflict given to Admin)

The code "could" be heavily simplified for posteriority if we use su -u 1000 or similar for the mkdir and git clone calls. Though this might cause permission errors with the rest of the container, so we would need to add some control to the container execution mode as well -- I vaguely remember that I had encountered a blocker while trying to do so with the init container specs, but this can be revisited later


deps(certifi): bumped to 2024.07.04 for fixing Trivy CVE scan.

Issue link
Relates: #104, #105
Changes in this PR

upgraded certifi from current to 2024.07.04 to fix Trivy CVE scanning in the PR above.
added deps prefix for further PR of dependencies update

Changes not included in this PR
N/A

Since Semantic PR requires to check semantic.yml in branch main to apply changes, we can ignore CI error on Semantic PR for now.


Step 6 & 7 need to be updated

Azure AD Connect now shows Device Systems first, and then SCP.

All of the screenshots should likely be updated with this as well.

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

ID: 816f0138-d336-3905-9f4f-e60c04da9326
Version Independent ID: ce5166e4-5d15-ef1f-1090-576a69e601e8
Content: Configure hybrid Azure Active Directory join for managed domains
Content Source: articles/active-directory/devices/hybrid-azuread-join-managed-domains.md
Service: active-directory
Sub-service: devices
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@daduckMSFT
Thanks for your feedback! We will investigate and update as appropriate.

@daduckMSFT
Thanks for your post! I'll assign this issue to the author to investigate and update the docs as necessary based off your findings.
Current steps/screenshot:

If you have any other questions, please let me know.
Thank you for your time and patience throughout this issue.


which cameras are vulnerable?

hey thanks for this amazing script but do you have a list for which cameras are vulnerable.

Hi,
Please see page 6: https://media.blackhat.com/us-13/US-13-Heffner-Exploiting-Network-Surveillance-Cameras-Like-A-Hollywood-Hacker-WP.pdf
And possibly other models and vendors too.

Thank you.


Docbook 5.1 conversion using docbook-xslTNG-2.2.0.jar ignore  elements inside

I have a Docbook 5.1 XML file called "product_manual.xml", and I'm using docbook-xslTNG-2.2.0.jar to convert it to HTML.
I want a company logo to appear above the title of my document, but the jar file seems to be ignoring it.
The beginning of product_manual.xml looks like this:
<?xml version="1.0" encoding="utf-8"?>
<!-- DocBook V4.x converted to DocBook V5.x by 10-db4to5.xsl version 1.0 -->
<book
  xmlns="http://docbook.org/ns/docbook"
  xmlns:xlink="http://www.w3.org/1999/xlink"
  version="5.1"
  xml:lang="en"
>

  <info>
    <mediaobject role="logo">
      <imageobject>
        <imagedata fileref="pictures/company_logo.jpg" format="JPG" />
      </imageobject>
    </mediaobject>

    <title><trademark class='registered'>Product Series</trademark>: Product Name</title>
  </info>

  [...body...]

</book>

I convert with the command:
java -jar <path to xslTNG>/libs/docbook-xslTNG-2.2.0.jar product_manual.xml -o:product_manual.html
The result is an info section with a title and border, but no image. The image is totally ignored.
If I move the mediaobject outside of my info block, in the HTML output the image appears after both the info section and the table of contents.
Why is my mediaobject being ignored?
How can I customize my command line (jar options) or CSS (docbook.css) to print mediaobject elements within ?

The stylesheets don't attempt to output everything that appears in the info element. You can put a lot of metadata in there, and may systems do. The user guide should really cover this case, thank you for raising the issue. I'll try to update the guide in the next day or two.

Thank you for your quick reply. Glad to know I'm not seeing unexpected behavior.
Is there a workaround to achieve my goal of an image on the title/cover page?
I notice that both the DocBook 5.1 Reference and the xslTNG Reference have title pages structured with an image on the left, and what appears to be title and info on the right. Am I to assume that these pages were not built with xslTNG or specifically docbook-xslTNG-2.2.0.jar?

They were both produced with these stylesheets. You just need a small customization layer to specify what and how you want titlepage elements displayed (if, as is the case here, the defaults aren't satisfactory). It's a very common use case, so common, that I've had to check twice that it isn't discussed in the documentation. A surprising oversight!
I'll try to get to it this weekend.

Great, thank you for addressing this!

Here's my first attempt: https://xsltng.docbook.org/guide/2.2.0a/ch-customizing#titlepages
(Version "2.2.0a" of the guide isn't the default yet, that will be in 2.2.1 though.)

Thank you, this is very helpful! I was able to customize my titlepage as needed.

Marking this complete.


A litte change in ch02 ex2.27 about reference

In fifth edition section 2.41, it says

Technically speaking, there are no const references. A reference is not an object, so we cannot make a reference itself const.

My test code shows the error message

ch2_constRef2.cpp: 7:19: error: 'const' qualifiers cannot be applied to 'const int&'
const int & const r1 = i;
^

It's may be not about initialization of a reference.

@zbjdonald  Agree. Thx your contribution. :+1:

@zbjdonald Thx.
I reproduced this error on gcc. But on VS2013 it compiled...
Anyway pretty sure you are right. All reference are consts or as said in cp5 "we cannot make a reference itself const."
Please make more contribution for us.

I found MS guys like to do such funny stuff..


Update codegen service port and update e2e test

Description
Update codegen service port and update e2e test
Issues
fixes issue #76
Type of change
List the type of change like below. Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would break existing design and interface)

Dependencies
n/a
Tests
n/a

I would suggest split this changes into 2 different PR. One is for CI actions, it should succeed with any port settings in the helm-chart and/or manifest. Another is for populating MEGA_SERVICE_PORT as environment into k8s deployment defintion.

I would suggest split this changes into 2 different PR. One is for CI actions, it should succeed with any port settings in the helm-chart and/or manifest. Another is for populating MEGA_SERVICE_PORT as environment into k8s deployment defintion.

The MEGA_SERVICE_PORT should not be exposed to users. User should care only the service.port, whose default is equal to MEGA_SERVICE_PORT but can be customized.
Let's first merge this fix to make sure we're using the correct port, and do more cleanups in another PR.

I would suggest split this changes into 2 different PR. One is for CI actions, it should succeed with any port settings in the helm-chart and/or manifest. Another is for populating MEGA_SERVICE_PORT as environment into k8s deployment defintion.

I would suggest split this changes into 2 different PR. One is for CI actions, it should succeed with any port settings in the helm-chart and/or manifest. Another is for populating MEGA_SERVICE_PORT as environment into k8s deployment defintion.

Will do.


feature_request: selectable markdown cells

Make text selectable in markdown cells in jupyter notebook.
Sometimes you can found code examples in markdown cells, and when you want to copy that, you need to start editing cell, and search for that part.

Power try to avoid creating duplicate


Removing KML features fails in WorldWind

When KML is deleted from the overlay the graphics still persist on the map until the user zooms or pans the map.

KML graphics should now disappear immediately on removal

Reintegrate


rbpnet train command fails: ValueError: No validation loss ('val_loss') found in logs.

I'm not sure I'm running the commands appropriately. I created different BED files for training and validation, made corresponding yml files for each bed file with the same transcriptome-wide bigwigs, and ran rbpnet tfrecord on each yml.
Then I run rbpnet train with the validation tfrecord and the training tfrecord
rbpnet train --validation-data RPB.val.tfrecord -c configs/model.config.no-penalty.gin -d RBP.train.dataspec.yml -o RBP.h5 RBP.train.tfrecord
It starts epoch 1 but then throws the following error:

ValueError: No validation loss ('val_loss') found in logs.
In call to configurable 'train' (<function train at 0x2b1be0b4a040>)

Any suggestions?

Hi,
I could not reproduce your error. I've created a develop branch and added some test data. Could you do the following and let me know whether the error persists?

Activate your rbpnet conda environment
Clone develop branch: git clone -b develop https://github.com/mhorlacher/rbpnet.git
Change into repo and install cd rbpnet; pip install .
Change into example directory and run training: cd examples; rbpnet train --validation-data data/PTBP1_HepG2.data.tfrecord -c config.gin -d data/PTBP1_HepG2.dataspec.yml -o out/ data/PTBP1_HepG2.data.tfrecord

Thanks!

Hi thanks for your help. The example ran, and I was able to do all the modeling I needed!
I have a question though. You used the same tfrecord for training and validation input: is the validation data meant to be a subset of the training set? I'm still not sure I totally understand how to use those commands.

Hi @augustboyle - This was just for debugging purposes. In practice the training and validation sets should not intersect, otherwise one overestimates the model's performance and the model will likely overfit. I'll update the README in the next days and add a more verbose description of the command line arguments, please let me know if you encounter any other issues.


OCR only works for admins

Not sure why, but OCR context menu message only works for admins! Im guessing its something with the publish configuration...

yep close

^ Basically, we needed to update  the integration settings in discord settings, it wasn't coding related


If you havent done so already make a backup of the wallet.dat file and keep it safe (while it may be broken, need to have something to come back to if needs be.
To resolve the issue, the best thing to do is to attempt to recover the private keys in the current wallet.dat  file and import them into a new wallet.
There are a number of different ways to do this.

use the dump/import wallet call in reddcoin-qt
use an external tool to attempt to read the file.

for option 1, using the wallet

shutdown Reddcoin and make a backup of wallet.dat, move that backup somewhere safe
start Reddcoin
disable staking (Help > Debug Window > Console > setgenerate false)
unlock wallet (Help > Debug Window > Console > walletpassphrase   )
dump the wallet (Help > Debug Window > Console > dumpwallet "path_to_wallet_dump.dat")
shutdown Reddcoin
rename the wallet.dat > wallet.original (there should be no wallet.dat in the data directory)
start Reddcoin (which will generate a new empty wallet.dat file)
import the wallet (Help > Debug Window > Console > importwallet "path_to_wallet_dump.dat")

depending on the size, it could take a while to import as it will scan the blockchain to recover all known tx's for all known addresses
if the original wallet.dat is not too damaged, this proceedure should recover all reachable private keys.
But please check wallet balance matches.
Else, will need to get into more details with you

Hmm Strange i have had no issues for years, I already have a backup  of my wallet i will try using that and see if that fixes the issue.

Please let me know if everything is ok after restoring wallet.
In the past, usually found antivirus or power failures to be the main culprit.
I will close the ticket, but feel free to reopen if the issue is not resolved

Fixed the issue thanks :)


Handling config changes

I am trying uamp sample, whenever orientation changes MediaBrowserFragment recyclerview state is lost(its scroll position is not maintained).
My problem is how to maintain scroll position in uamp sample.
I know I can save scroll position and restore it later but since recyclerView itself maintains scroll state I don't want that hacks.
I tried pushing this simple fragment instead of MediaBrowserFragment and its maintaining its scroll
public class TestFragment extends Fragment {

    @Override
    public View onCreateView(@NonNull LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        View mRootView = inflater.inflate(R.layout.fragment_test, container, false);
        RecyclerView recyclerView = (RecyclerView) mRootView.findViewById(R.id.recyclerView);
        recyclerView.setLayoutManager(new LinearLayoutManager(getActivity()));
        recyclerView.setAdapter(new RecAdapter());
        return mRootView;
    }

    private class RecAdapter extends RecyclerView.Adapter<RecyclerView.ViewHolder> {

        @NonNull
        @Override
        public RecyclerView.ViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
            View itemView = LayoutInflater.from(parent.getContext()).inflate(R.layout.layout_text, null);
            return new ItemViewHolder(itemView);
        }

        @Override
        public void onBindViewHolder(@NonNull RecyclerView.ViewHolder holder, int position) {
            ((ItemViewHolder) holder).title.setText(""+ position);
        }

        @Override
        public int getItemCount() {
            return 200;
        }
    }

    private class ItemViewHolder extends RecyclerView.ViewHolder {
        TextView title;

        ItemViewHolder(View itemLayoutView) {
            super(itemLayoutView);
            title = (TextView) itemLayoutView.findViewById(R.id.title);
        }
    }
}

This should be better in the rewrite, but there might be some work to do, which would require more work than is probably appropriate for this sample.
If you disagree and want to create a PR to address this I'd be more than happy to review and include it.

Can you please provide more inputs in this case, how should I proceed, will be happy to contribute on this matter.

I'm not sure. I use the app with the default catalog, which isn't many songs, so it doesn't create a problem for me. :)

Then in this case, please reopen the issue, as If this is not a problem for you doesn't  mean the problem doesn't exists :) , also not everyone is using the default catalog of 10 or 12 songs.


allow @ character in lexer

fixes https://github.com/argoproj/argo-workflows/issues/8734

Foo[“bar@“]

@antonmedv<EMAIL_ADDRESS>does not work, the @ is needed for gjson lib

Don’t know what it means.

case r == '@':
	l.backup()
	return identifier

I think adding that in https://github.com/antonmedv/expr/blob/baae791a9321e8570785ea18814d0697057b2a00/parser/lexer/state.go#L47 will fix it

I don’t see a problem. Why expr should support it?


Question: Generate C# dto classes with same casing of the definition

I've searched but I was not able to find any option to generate C# client with DTO with the same casing of the API
As an example, I have an api with standard camelCase name, if I generate typescript client evertying is ok.

export interface AceReducedDto {
    id?: string | undefined;
    name?: string | undefined;
    enabled: boolean;
}

In typescript client, correctly all properties starts with lowercase letter. If you look at the snippet below, C# client uses only PascalName properties with the correct Newtonsoft attributes that contains properties with the correct casing.
I have a scenario where I need the real C# property to have the exact casing as I specified in swagger file.
    [System.CodeDom.Compiler.GeneratedCode("NJsonSchema", "<IP_ADDRESS> (Newtonsoft.Json v<IP_ADDRESS>)")]
    public partial class AceReducedDto
    {
        [Newtonsoft.Json.JsonProperty("id", Required = Newtonsoft.Json.Required.Default, NullValueHandling = Newtonsoft.Json.NullValueHandling.Ignore)]
        public string Id { get; set; }

I'd like the above C# client is generated like this
    [System.CodeDom.Compiler.GeneratedCode("NJsonSchema", "<IP_ADDRESS> (Newtonsoft.Json v<IP_ADDRESS>)")]
    public partial class AceReducedDto
    {
        [Newtonsoft.Json.JsonProperty("id", Required = Newtonsoft.Json.Required.Default, NullValueHandling = Newtonsoft.Json.NullValueHandling.Ignore)]
        public string id { get; set; }

I've searched a little bit around but I was not able to find any simple solution.
What am I missing?
Thanks for any help anyone can give me.

@alkampfergit, what you are asking for does not conform with the Framework design guidelines, which every tool in the .NET space aims to conform with.
Why would you want that?

I know that the request is strange. I use reflection from generated C# client in a software that uses jint to let the user write JavaScript scripts.
It would be useful to have C# classes that are identical in shape with JavaScript, they would not be idiomatic, but this is really an edge situation. The library we are using generates intelligense for the user based on C# class. With a C# client generated in this way we can work seamlessly between C# and Js.

Jint by default ignores the first character casing and then is case-sensitive for the rest of them. You could probably just generate the intellisense helpers with first character lower-cased.
You could also probably just replace the default property name generator: https://github.com/RicoSuter/NJsonSchema/blob/63ee6152e9819596e9c7158a9ac634a96f73b2e9/src/NJsonSchema.CodeGeneration.CSharp/CSharpGeneratorSettings.cs#L46

Thank y for the suggestion!

Or, since you're already using reflection, use the annotation attributes to extract the JSON names.


[Test] vitest migration blockers

This issue documents obstacles in migrating to vitest in our repo.
Node
Browser

We use an envProcessor plugin in Karma tests to inject environment variables to browser tests
util dependency causes problem in tests.

the main tsc problem seems to be chai globals conflicting with the bundled version of chai that vitest provides

for environment variables I think we can use vite config for this: https://vitejs.dev/config/#using-environment-variables-in-config

for environment variables I think we can use vite config for this: https://vitejs.dev/config/#using-environment-variables-in-config

yes, looking into that

/cc @maorleger


SnackbarOptions: Add IconColor and IconSize properties

Description
SnackbarOptions: Add IconColor and IconSize properties
-Added property: SnackbarOptions.IconColor
-Added property: SnackbarOptions.IconSize
-Add unit-test: CustomIconTest
-Updated example: SnackbarCustomIconExample.razor
Resolves #4427
How Has This Been Tested?
Unit-test, visually.
Types of changes

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

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

@henon, ready for your review

Thanks!


Korean translation of chapter 03

Hello! This is translation for Korean of chapter 03-using-generative-ai-responsibly.

@mreraser please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree


Fix metadata file path

Fixed path resolution for the metadata file and added error handling in case it's missing

Hi, @erademacher! Thanks for opening this PR!

In the file tdvt/tdvt/tdvt_core.py:

Line 83:51: F405 'TestMetadata' may be undefined, or defined from star imports: .resources, .test_results

For reference: Error Code Reference, PEP8, Python code style

Could you run the unit tests to see if they pass unchanged after this? If they do we probably need to add a test involving a metadata file.

Sorry, I just don't have time to write a test this sprint. I'm going to push it through (with a version bump) and put something in TFS for test debt.


Feature/sticky header

Add support for sticky header column

Coverage increased (+0.05%) to 77.988% when pulling 88f9cd8f4b494730483751d7c0b4b63634dad661 on feature/sticky-header into e872978a1ca6664ab109257fed7e5d6ddc1fcc62 on master.

Can you confirm that adding a background color of white is causing unexpected behavior with the rows selected shadow (no longer visible) and the background color of the checkboxes for each row (is also white instead of inherited)?

The fixed header option does not seem to be doing anything for me either.

Any plans to implement this feature?

@jlei523 - it was working again for me after a recent update

This feature only appears to work when the responsive mode is set to 'stacked'.
Has anybody found a work around to get the sticky headers without enabling 'stacked' responsive mode? I've been toying around with using stacked and overriding the rendering to be identical to scroll.

With some further research, this now makes sense as to why sticky doesn't work in scroll mode.
Via Stacked a comment on this overflow accepted answer:
"Worth noting that this won't work if the table is contained inside an element with overflow: auto or overflow-x: auto."
https://stackoverflow.com/questions/44001954/css-only-sticky-table-headers-in-chrome
Here is where overflow is set to auto.
https://github.com/gregnb/mui-datatables/blob/529307099c96ec5d983f6e4cf0f58a010ebe1c2c/src/MUIDataTable.js#L26
I've been able to resurrect the sticky positioning via customizing the MUI Theme at the cost of needing to find a solution around the overflow of content.

For anybody interested I've been able to make both the Toolbar and the table headers sticky with this:
    MUIDataTable: {
      responsiveScroll: {
        maxHeight: 'unset',
        overflowX: 'unset',
        overflowY: 'unset',
      },
    },
    MuiToolbar: {
      root: {
        top: 0,
        position: 'sticky',
        background: 'white',
        zIndex: '100',
      },
    },
    MUIDataTableHeadCell: {
      fixedHeader: {
        top: '62px',
      },
    },


[FLINK-22298][table-planner-blink] The ExecNode's id should always start from 1 in json plan test

What is the purpose of the change
The ExecNode's id should always start from 1 in json plan test, which could make the test more stable
Brief change log

Reset the id counter before generating json in in test

Verifying this change
This change is already covered by existing tests*.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

The e2e test in My private azure is successful, https://dev.azure.com/godfreyhe/godfreyhe-flink/_build/results?buildId=401&view=results. I will merge this pr


Constant register fix

This PR enhance GSNDevProvider logic by checking if the relayer already registered. If so, there is no reason to make it again - tabookey relay does not do that before every tx, right? Moreover, such behavior violates the default assumption that calling a contract method is an atomic action, which causes problems with unit tests.

@nventuro @ylv-io

This isn't completely addressed in this PR, since the relayer will register itself on the first relay request. Is this causing issues on your setup? Should we make it register before that happens?

Yeah, it may cause the issue. To prevent that, we register the relayer before a test. See an example here

@nventuro we would be really appreciate if OpenZeppelin could publish something from their twitter regarding our help from PepperSec.com

Right! I'm on it @rstormsf !


⚠ Samsung has degraded performance

In 5a49f8b, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2309 ms

Resolved: Samsung performance has improved in bf71ec2.


weird error message: "(0,-1): Error: a postcondition"

Failing code
too much to paste (down to 6K lines)

(0,-1): Error: a postcondition could not be proved on this return path

Steps to reproduce the issue

Dafny version: <IP_ADDRESS>
Dafny VSCode extension version: 3.3.1

Expected behavior
(line number, column number)
Actual behavior
(0,-1)

If you figure out which piece of code the error is about, please add it to this issue.

If I can isolate the code I will,
otherwise it's not a super-high priority
(although you could put a trap in for
an error report on line 0 or column -1)
James

On 15 Aug 2024, at 21:36, Remy Willems @.***> wrote:
If you figure out which piece of code the error is about, please add it to this issue.

Dunno if this helps  but theIDE just gave me this popup also with a zero line number.


Added HTK format support to sox_io's save & info

Cherrypicking #1276 & #1291 from the master branch

Note that the failing test ci/circleci: unittest_linux_cpu_py3.9 is due to the fact that CPU PyTorch is getting installed even though the code is requesting CPU-only. So you can ignore it.

@imaginary-person Thanks!


Export failure and success actions

Coverage remained the same at 97.015% when pulling b3bfb4c59f6967e8a303180b6c378f4d1d06609f on export-actions into 8570a43e915c77acec25ec3cf0a604fe59df0a74 on master.


load_model() Error: "Unknown entries in loss dictionary" in 2.0

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
TensorFlow installed from (source or binary):binary
TensorFlow version (use command below): 2.0.0.dev20190417
Python version: 3.6.5
Describe the current behavior
Tried very simple transfer learning on Inception V3. Training finished, prediction works fine. Saved the model with model.save_model(). When trying to load it with model = load_model(), I get this error:
ValueError: Unknown entries in loss dictionary: ['class_name', 'config']. Only expected following keys: ['dense_73']
Describe the expected behavior
Code to reproduce the issue
base_model = tf.keras.applications.inception_v3.InceptionV3(weights='imagenet', include_top=False)
x = base_model.output
y = tf.keras.layers.GlobalAveragePooling2D()(x)
z = tf.keras.layers.Dense(1024, activation='relu')(y)
predictions = tf.keras.layers.Dense(4, activation='softmax')(z)
model = tf.keras.Model(inputs=base_model.input, outputs=predictions)
for layer in base_model.layers:
layer.trainable = False
model.compile(optimizer='rmsprop', loss=tf.keras.losses.CategoricalCrossentropy())
model.fit(trainData, validation_data=(valData), shuffle=True)
model.save(myPath)
outValues = model.predict(testData)
model = tf.keras.models.load_model(myPath) #error here
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

I now see that the same problem was reported in #25938, so closing.


Logo/Icon Proposal

Hi good day! I would like to ask for your permission to propose a new designs of logo in your application/project. I would like to help projects in open-source .I would just like to propose new designs. Hoping for your positive response Thank You!

Hey @jbeguna04. You can work on it, but i can not guarantee you that it will be merged.

@jbeguna04 just be sure to notice, the current logo is the one at https://steem.show.


[BUG] change the PATH in post_proc_sp.bash into the right one

make PATH command line option

Most likely a reference to this line


Unhandled Promise Rejection Error

Describe the bug
Unhandled Promise Rejection:ERROR: spawn UNKNOWN
Udacimak Log
udacimak render C:\Users\Hanson\Android -t C:\Users\Hanson\Desktop
2019-10-24T07:35:12.289Z [info]: Image already exists. Skip downloading C:\Users\Hanson\Desktop\Android\img\android-basics-large.jpg
2019-10-24T07:35:12.313Z [info]: Completed rendering Nanodegree summary file C:\Users\Hanson\Desktop\Android/index.html
2019-10-24T07:35:12.314Z [info]: ____________________

2019-10-24T07:35:12.337Z [info]: Completed rendering lesson summary file C:\Users\Hanson\Desktop\Android\Part 01-Module 01-Lesson 01_课程1A：打造布局\index.html
2019-10-24T07:35:12.338Z [info]: ____________________

- Getting Youtube video id zO9Bu0IwFTM information2019-10-24T07:35:12.358Z [error]: Unhandled Promise Rejection:ERROR: spawn UNKNOWN
CODE: UNKNOWN
STACK: Error: spawn UNKNOWN
    at ChildProcess.spawn (internal/child_process.js:366:11)
    at spawn (child_process.js:551:9)
    at execFile (child_process.js:221:15)
    at youtubeDl (C:\Users\Hanson\AppData\Roaming\npm\node_modules\udacimak\node_modules\youtube-dl\lib\youtube-dl.js:25:3)
    at call (C:\Users\Hanson\AppData\Roaming\npm\node_modules\udacimak\node_modules\youtube-dl\lib\youtube-dl.js:153:10)
    at Function.getInfo (C:\Users\Hanson\AppData\Roaming\npm\node_modules\udacimak\node_modules\youtube-dl\lib\youtube-dl.js:238:3)
    at module.exports (C:\Users\Hanson\AppData\Roaming\npm\node_modules\udacimak\node_modules\youtube-dl\lib\youtube-dl.js:100:8)
    at Promise (C:\Users\Hanson\AppData\Roaming\npm\node_modules\udacimak\lib\commands\utils\downloadYoutube.js:89:42)
    at process._tickCallback (internal/process/next_tick.js:68:7)

Desktop (please complete the following information):

OS: [Windows 10 1903 ]
Node version: (10.16.3)
npm version: (6.12.0)
Udacimak Version: (1.4.5)
Browser [chrome]

Course Information

Nanodegree or Course name: [Android Basic Nanodegree]
Version: [v1.0.0]
Locale: [zh-cn]

Additional context
I can watch the youtube video via the id zO9Bu0IwFTM , not sure where goes wrong :)

@Hanson7300 The error is thrown by youtube-dl.
Could you try rendering in a different VPN network?

@udacimak I tried ShadowsocksR and V2ray, both works when downloading youtube videos.
Render still doesn't work.

@Hanson7300 that's weird. Actually it's an error from youtube-dl so I'm afraid I can't provide much help. It's working on my end so I have to assume that the CLI is working fine.
Are you able to try on another network?

@udacimak I re-install node.js,npm,udacimak, and got the error unable to locate youtube-dl.I tried to put youtube-dl.exe into the path but doesn't work...

Error: ERROR: unable to locate youtube-dl at C:\Users\Hanson\AppData\Roaming\npm\node_modules\udacimak\node_modules\youtube-dl\bin
at module.exports (C:\Users\Hanson\AppData\Roaming\npm\node_modules\udacimak\node_modules\youtube-dl\lib\get-binary.js:11:11)
at Object. (C:\Users\Hanson\AppData\Roaming\npm\node_modules\udacimak\node_modules\youtube-dl\lib\youtube-dl.js:18:41)
at Module._compile (internal/modules/cjs/loader.js:971:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1011:10)
at Module.load (internal/modules/cjs/loader.js:822:32)
at Function.Module._load (internal/modules/cjs/loader.js:730:14)
at Module.require (internal/modules/cjs/loader.js:864:19)
at require (internal/modules/cjs/helpers.js:74:18)
at Object. (C:\Users\Hanson\AppData\Roaming\npm\node_modules\udacimak\lib\commands\utils\downloadYoutube.js:18:41)
at Module._compile (internal/modules/cjs/loader.js:971:30)

@Hanson7300
youtube-dl can't be run directly as such. It is installed and used internally by udacimak

I have the same issue @Hanson7300
/usr/local/lib/node_modules/udacimak/node_modules/youtube-dl/lib/get-binary.js:11
throw new Error('ERROR: unable to locate youtube-dl at ' + binPath)
^
Error: ERROR: unable to locate youtube-dl at /usr/local/lib/node_modules/udacimak/node_modules/youtube-dl/bin
at module.exports (/usr/local/lib/node_modules/udacimak/node_modules/youtube-dl/lib/get-binary.js:11:11)
at Object. (/usr/local/lib/node_modules/udacimak/node_modules/youtube-dl/lib/youtube-dl.js:18:41)
at Module._compile (internal/modules/cjs/loader.js:701:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:712:10)
at Module.load (internal/modules/cjs/loader.js:600:32)
at tryModuleLoad (internal/modules/cjs/loader.js:539:12)
at Function.Module._load (internal/modules/cjs/loader.js:531:3)
at Module.require (internal/modules/cjs/loader.js:637:17)
at require (internal/modules/cjs/helpers.js:22:18)
at Object. (/usr/local/lib/node_modules/udacimak/lib/commands/utils/downloadYoutube.js:18:41)

@Hanson7300
@udacimak suggest me to use udacimak docker version, and it's working, so you can try that out.

@vsay01
Hi
By Docker version r u able to download full course and how to use udacimak renderdir command

@vsay01
Hi
Plz check https://github.com/udacimak/udacimak/issues/67
I download and install docker toolbox but show some error, can u suggest me what can I do.
Thanks

@vsay01
Hi
Ur udacimak Docker version is working so can u download Intro to Deep Learning with PyTorch and share with via google drive.
Thanks

@vsay01 I tried many ways to apply my VPN network on docker container, but all failed.

@Hanson7300 Could you share the log you have ?

Udacimak not working, plz try other methods.
Even I try all Udacimak method and Dockers but no result
On Mon, Feb 17, 2020 at 1:42 PM udacimak<EMAIL_ADDRESS>wrote:

Closed #59 https://github.com/udacimak/udacimak/issues/59.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/udacimak/udacimak/issues/59?email_source=notifications&email_token=ABZK3KQJHBO4VLCSB2MI2N3RDJBHXA5CNFSM4JEQHYM2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOWVNZT5I#event-3042679285,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZK3KQL5O6Q6Y3LTSF5ACTRDJBHXANCNFSM4JEQHYMQ
.


fix: conversation mapping bug

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use ksp when it supports js(IR)

See also: https://github.com/google/ksp/issues/33
For now, maybe just hack things by parsing "manually" with like a regex or something?

Here's some code that can used as a starting point. Thanks @DVDAndroid for the initial work!

https://github.com/DVDAndroid/kobweb/tree/dvd/ksp

So I finally got a chance to look into this, nearly one year later :)
With the way Kobweb has evolved, it's possible that KSP may not be the approach we want to use after all. Let me document my thoughts here for now and I'll sleep on them.

Background
Currently, codegen works through a Gradle plugin which, when applied, adds a bunch of tasks, including kobwebGenSiteSource (main.kt for frontend), kobwebGenApi (ApisFactoryImpl.kt for backend), kobwebGenSiteIndex (index.html), kobwebExport, and kobwebStart.
The project parsing is handled by the kotlin-compiler-embeddable artifact.
There's also a markdown plugin which adds kobwebxMarkdownConvert, which needs to run before kobwebGenSiteSource, as it may generate files that feed into it as inputs. Currently, this is easy to do because with Gradle, you just register one task as a dependency of another.

If we transition to KSP, the Kobweb application plugin doesn't go away! You still need kobwebStart, etc.
The only tasks that might go away are kobwebGenSiteSource and kobwebGenApi. So now we'd have this weird complicated thing where a Kobweb project needs to both specify a KSP processor AND a gradle plugin, which both need the other active to be effective.
Another wrinkle -- with kobwebGenSiteSource, we output different source based on whether we're targeting debug or release. I don't see a way to control that behavior with KSP very easily. Even if there is a way, we have to make sure we keep the Gradle and KSP logic in sync (since currently Gradle parses a property from the command line and turns that into a build release value).
And finally, we have markdown -> Kotlin (from the markdown plugin), which definitely isn't related to KSP (KSP doesn't care about markdown). So now we'd have some gen source tasks handled by KSP and some by Gradle.
All in all, I'm starting to think that KSP isn't the way forward here. I think it makes a Kobweb project...

more complicated to understand (what KSP does and what Gradle does may be hard to follow)
less flexible due to lack of debug / release config control
more fragile because someone might go into their build.gradle file, remove a KSP dependency they don't understand, and blammo, things just silently stop working.

The only reason I can think that we'd want to use KSP over Gradle is performance. But I'm not sure that the Kotlin parsing we're currently doing is that slow. Maybe I need to time stuff, but unless we're talking about 5-10s slimming down to sub-second, I'm not sure it's worth the complexity!

I'm going to close this for now. I think having the Gradle plugin continue to do the parsing is the easiest path forward.
Otherwise, I'd need to ask users to both apply a Gradle plugin (e.g. com.varabyte.kobweb.library) and a ksp plugin + ksp dependency.
In the future, if we find out that compile time additions from the Kobweb plugin are prohibitive, then we can revisit this decision.
As an aside, thanks to my notes from the above comment, I did significantly refactor my Gradle code, so that the code parsing / processing step is simultaneously cleaner and it also caches intermediate outputs which means that 1) subsequent task runs which do expensive parsing might not need to happen and 2) doing an export will only run the parsing step once (before, it was running it twice in a row from different tasks. whoops!)

One thing that might make this worth revisiting is KSP probably has better support for resolving values?
Like, if I have
val SomeVariant = SomeStyle.addVariant("...")

and I want to know the fully qualified path of SomeStyle, I believe KSP lets me resolve that (even though it's generally discouraged as an expensive operation)
With the embedded kotlin compiler artifiact, I'm not 100% sure I can do that. So I end up with some checks I can't do at compile time...


TypeError: arg 5 (closure) must be None or tuple

Hi @pierluigiferrari
After you updated HDF5 in data generator, the SSD7 training model can't load model successfully, and it has error below:

Could you have a look at please?
Thank you,

The data generator and the Keras model have nothing to do with each other. In other words, changes in the data generator cannot affect loading a model.
I just tried loading a model and it works fine for me.
In general, if you open an issue, please bear the following things in mind:

Never post screenshots of text. Just copy and paste the text instead. Posting screenshots of text makes no sense and has two disadvantages: First, I cannot copy lines of text from a screenshot. Second, screenshots of text are a tremendous waste of storage space. Imagine how many terabytes of unnecessary screenshots like the one you posted are on GitHub's servers.
Always post the full stack trace of an error. You posted two screenshots above, but even both together don't contain the full stack trace.

FYI, @pierluigiferrari and whoever else runs into this, I also ran into OP's issue and the resolution was to do what was outlined in https://github.com/keras-team/keras/issues/6211.
Specifically, manually reinstate the code in this commit to keras\utils\generic_utils.py https://github.com/soaxelbrooke/keras/commit/aaada44607e8f45a13732347f70a2b372ac729b2
Cheers


Show sweet logos on build badges

Makes the build badges on top of the README display nice logos of Travis CI and AppVeyor
Removes the build badge for Shippable (it's too clumsy to use)

He's back! :)

We should get all the automatic building of PythonTurtle for Windows and macOS working, and push a release out with Python 3 before the end of this year. Also, a release on PyPI would be nice (to allow Python nerds to install PythonTurtle via pip install pythonturtle).
If we don't do this all the effort of automatic builds and the migration to Python 3 had only minimal value.

Sounds good.


kie-issues#114: JavaCodeCompletion module fails to compile because of API change

Finally closes https://github.com/apache/incubator-kie-issues/issues/114
One year ago the JDT.LS API suddenly changed breaking the Java class "Fetching" mechanism we had in place.
We that change, it was no longer possible to retrieve a class FQCN, which is a required property for the "Fetching" mechanism to properly work.
In the latest release, the API now supports the TypeHierarchy command: given a class name, that command can retrieve the class hierarchy and the first items represent the class itself in FQCN format.
To restore the "Fetching" feature, we can then rely on this new command.
For example, if we use that command on a String type variable, this will return:

The first item contains exactly what we need.
https://github.com/apache/incubator-kie-tools/assets/16005046/15f0c4bc-f772-4b9f-93af-6a427edf25c4

DO NOT MERGE - I need to double-check check it working with JDT.LS 1.30.x

It works with version 1.30.1! @tiagobento Ready to be reviewed and eventually merged.

@jomarko Yep, the ones you select are put at the top of the list

@tiagobento Can you please review it?

I have some difficulties rebasing this... opening a new PR


Fix the base directory issue for the fetcher and update deps & manifests

/test pull-knative-operator-serving-upgrade-tests

/test pull-knative-operator-serving-upgrade-tests


Add support to perform a vulnerability scan on container images

Is your feature request related to a problem? Please describe.
BotKube should perform a vulnerability scan on the container images when pod is created and report CVE report to the user
Describe the solution you'd like

We can use Clair to perform vulnerability static analysis container images
BotKube should have ability to disable/enable this feature through config flags

I would like to add some points here.

Implementing this feature as an "on demand vulnerability scan" using botkube commands, would be nice.
Running scans on all pods by default, would lead to repetive scaning of images, for instance during scaling of replias, and we might need a mechanism to know if the image was scanned already.

@aananthraj totally agree with you. This will also add the overhead of deploying Clair as a part for BotKube deployment, we need to search for the better option if there is any

Yes @PrasadG193 , I guess we could create/excute a kubernetes job for this instead of creating a permanent clair deployment of clair. The same holds good for #129. Let me know your opinion..?

Yeah, that is also one way


Feature/replace underscore with lodash

Leave Underscore as path key in requirejs config

Coverage remained the same at 100.0% when pulling fa2fe8de53c120674a12f9fbc17e1d2b16c356fc on feature/replace-underscore-with-lodash into 2d4fbacc8e21e657b2333a0e0e30d104939c65b5 on master.


Cannot get property 'BASE_PATH_GRAPHICS' on null object.

I get this when being on http://localhost:8080/streama/#/admin/shows .
I click 'Create new show', enter 'The IT Crowd', select the first result and click 'Save show'. Then nothing happens in the browser and a NullPointerException follows in the console. :/
Console log: http://pastebin.com/SaUCEDLH

Yep! Fixed that 2 commits ago :) please pull & try again

Oh lol, I just cloned about half an hour ago ;)

Okay works, thanks :+1: btw awesome program!

And how can you add video files to an TV show? I added the episodes from season 1 (automatically) , but when I click "Browse for File" I get a "File Browser"... How can I add files there...?

Drag and drop em in :) The browser is only for already added files

Where to Drag&Drop?

It just downloads the video files. Using Ubuntu 15.04, Chrome 45.0.2454.15 dev (64-bit)

from your file system onto the individual episode-boxes. you should see in the gif in my readme.

Wow lol, didn't know that was even possible... :o :+1:  to you ! xD

you can upload 7 at a time, all the others get queued.

Can I also link existing files as it copies the files to /data/streama ?

That is not yet implemented, but I will add that asap :)

Okay thanks.
java.io.IOException: No space left on device
lol

oh dang... is that a bug or is it really a storage issue on your end?

no, I made the /boot partition smaller, as I wanted to maybe install more than one Linux version on this hard drive and made the rest big. Apparently /data is on /boot :D

you can change the directory though, as long as you update the path in config.groovy

Can I then just copy the /data/streama folder to the new location?

yep. just make sure the application has read/write permissions to that new folder. By default, the app uses the path in the config.groovy (so /data/streama) and adds another folder inside of it, called upload. All files in it have hashed names, and those are the ones it looks for. So if you change the path to /bla in the config, then one of the files should be at /bla/upload/XYZ.mp4

Jeah, made it thanks! Manual symbolic links should work already, don't they?

Yep, don't see a reason why they shouldnt. Unless there are permission restrictions to the linked files.

Aaannd I have another error. Posting it in a second.

aww :(

While starting a video/stream

And you should create a startscript like start-streama.sh because I forget the command to start all the time :crying_cat_face:

http://pastebin.com/RvfTcv7r

This is the folder I specified in the Config.groovy:
drwxrwxrwx  3 tomcat7 tomcat7    4096 Jul 30 22:33 streama
I navigated to http://localhost:8080/streama/ and clicked the play icon on the show I added previously
Player says duration is "null"

I know the error in the middle. but it looks like it is restarting also ... is that true or doesnt it auto-restart due to the error?

I don't exactly know what you mean with "auto restart" but I guess no^^

The video files have the permission
-rw-rw-r-- 1 tomcat7 tomcat7 182181888 Jul 30 22:45 ffa39f42e6f79f2ee499c9b800ff0cab7e6a75f3d1cd36748fd0a06c316c3aa7.avi

ok, if it doesnt auto-restart then all this is, is a cosmetic error that I couldnt yet get rid off. The reason why your file isnt playing seems to be of a different nature: the file may not be html5 compatible. Could you try the following for me: could you open a new browser tab (empty is fine) and just drag the file in? if a video-player shows up, the file is html5-compatible. if the file gehts downloaded, its not. Please report back.

Format is .avi and it downloads :/

That means nearly all my video files are not compatible?

yeah..
avi often uses an incompatible codec :( mp4 & mkv is the way to go ... In the future I will add an auto-conversion, so that when you add files it converts to the right formats, but i havent added it yet ... :( Might do that over the weekend now.

That would be awesome!

Btw, are you using chrome or firefox?

Chrome 45.0.2454.15 dev (64-bit)

kk, good. more formats are compatible with chrome as they are with firefox :)

Video:
Dimensions: 608 x 336
Codec: MPEG-4 Video
Framerate: 25 frames per second
Bitrate: 509 kbps

here is an overview of codecs & browsers https://en.wikipedia.org/wiki/HTML5_video#Browser_support
I'll add that to the readme too

But actually the video file IS MPEG-4 video...

"The adaptive bitrate streaming standard MPEG-DASH can be used in Web browsers via the HTML5 Media Source Extensions (MSE)[16] and JavaScript-based DASH players." I have not added those extensions. But I will look into it. The best format in my experience is h.264 (most mp4 and mkv files)

And do you know if there is already support for x265 video files in Chrome 45? Or can I test this?

This is one of my files for example:
http://grab.by/JftM
I can see that the "kind" says MPEG-4 movie, but the important part is the codec further below.

Ubuntu says at the .mkv x265 files "Unknown" everywhere^^

x265 is pretty new afaik, not even sure if ffmpeg can convert it properly (which I would use to convert). I dont think its supported by chrome yet.

I just read the following: "Both of these codecs are still a ways off from any sort of widespread adoption. The specs aren't even finalized, as far as I know."

http://www.libde265.org/blog/2014/04/10/libde265-js-javascript-hevch-265-bitstream-decoder/
maybe?

Thats pretty awesome, I'll take a look if I can implement that without breaking regular viewing :P

And it works with ~60fps at my pc http://strukturag.github.io/libde265.js/

Okay .mkv x265 encoded KINDA works. Audio works, video is black... Okay :100: / :100:


Add Support for Seed Vault

Simple support for Seed Vaults

Can't believe I missed that, added. I just did a amend so please review the change.


Reduce data copying in storage::client::DownloadToFile

Assuming we have implemented the changes in #3393 we can eliminate one data copy in DownloadToFile(): instead of reading the data into a buffer and then write the buffer to a file, we can use the callback from libcurl and write the buffer directly to the file.

It seems like DownloadToFile is not used very often. We can reopen if we find there is enough demand.


Update GeminiGoogleClientService.kt

fix:INVALID_ARGUMENT (code 400) Unable to submit request because it has a topK value of 64 but the supported range is from 1 (inclusive) to 41 (exclusive). Update the value and try again.

Add to main via https://github.com/Blarc/ai-commits-intellij-plugin/commit/bfa3f7a78318984c7b878627491b1119479845f4.