I don't get any log output from LogWriter

Im trying to enable the debug output from NetSparkle however I don't get any output to the Visual Studio Console:
My Application is a .Net 5.0 WPF UI Application.
This is my Code :
       private void Button_Click_1(object sender, RoutedEventArgs e)
        {
            _sparkle = new SparkleUpdater("https://somurl.com/appcast.xaml", new Ed25519Checker(SecurityMode.Unsafe, "base_64_public_key"))
            {
                UIFactory = new NetSparkleUpdater.UI.WPF.UIFactory(null),
                LogWriter = new LogWriter(true),
                ShowsUIOnMainThread = true,

            };
            _sparkle.StartLoop(true, true);
        }

If I hit this button I get the popup with my custom changelog and the download, skip  and so on buttons.
If I hit "download / install" button the downloading window appears but never makes any progress.
Nevertheless NetSparkle works up to the downloading windows so I should get some LogOutput but the only thing that is printed to the VisualStudio Console is :
"UserForge.exe" (CoreCLR: clrhost): "C:\Users\lkohl\source\repos\UserForge\UserForge\bin\Debug\net5.0-windows10.0.18362.0\NetSparkle.dll" geladen.
"UserForge.exe" (CoreCLR: clrhost): "C:\Users\lkohl\source\repos\UserForge\UserForge\bin\Debug\net5.0-windows10.0.18362.0\NetSparkle.UI.WPF.dll" geladen.

I also tried LogWriter = new LogWriter(false), but this didn't change anything.
What could I try to get any log output?

Hi @Byolock, I'm sorry for the slow reply.
I'm not sure why you are not seeing any log output. When using the WPF sample (adjusted to run on .NET 5), I tried the following code:
private void ManualUpdateCheck_Click(object sender, RoutedEventArgs e)
        {
            _sparkle = new SparkleUpdater("https://netsparkleupdater.github.io/NetSparkle/files/sample-app/appcast.xml",
                new Ed25519Checker(SecurityMode.Unsafe, "base_64_public_key"))
            {
                UIFactory = new NetSparkleUpdater.UI.WPF.UIFactory(null),
                LogWriter = new LogWriter(true),
                ShowsUIOnMainThread = true,
            };
            _sparkle.StartLoop(true, true);
            //await _sparkle.CheckForUpdatesAtUserRequest();
        }

I was able to view the console output. It also worked with new LogWriter(false).

Are you sure your console/debug output isn't being redirected anywhere? You could also try implementing your own class that implements the ILogger interface and pass that to SparkleUpdater instead to verify that things are being passed to the PrintMessage method.
As far as the not downloading goes, the log will be helpful. You can also try setting ShowsUIOnMainThread to false as a test.

@Byolock I can replicate your issue, using the steps you provided. Thank you!
I think the issue is a bit complex:

When debugging your own application, the VS window shows Debug.WriteLine, not Console.WriteLine, so new LogWriter(true) won't work as that uses Console.WriteLine
When using NuGet, you're getting a release build, so the Debug.WriteLine calls from NetSparkleUpdater.LogWriter do nothing because it's a release build.
Because of this, LogWriter essentially becomes useless when debugging your own app. Whoops.

The fix is for LogWriter to use Trace.WriteLine instead of Debug.WriteLine, which should enable viewing the debug lines from other apps.
Fix for this is available in 2.0.0-preview20210314001.

Reopening because Trace.WriteLine didn't fix it somehow. Needs more investigation.

Bother. I didn't check to make sure it was enabled.

OK, I'm getting console output in VS with LogWriter = new LogWriter(false) now. Fixed.


Modification to add function

Updates:

removed the prefix for expense and income
combined expense and income
expense is denoted with a - and income is denoted with a +
Expense and Income inherits from the super class MoneyFlow
All Junit tests verified

thanks for the add function

Should be working

Thanks Vincent


Fixing approver bugs

Fixes bugs with radio buttons and being able to overwrite your own gatorlink

Fixed old tests that were erroring. Put in place holder failing tests that will need to be filled in later.


[BUG] 0.0.5 Port conflicts with Chia

After installing 0.0.5 can't connect to wallet.
It seems to conflict with chia client port 9256.
I would like to suggest changing the used port 9256 -> 10056

It seems work properly in 0.0.9


Dashboard view is broken

Please ensure:

[X] The bug is caused by Refined GitHub. It doesn't happen if I disable the extension.

Include in this issue:

[X] Screenshots/video/gif demonstrating the bug, if it’s visual
[ ] Console errors, if any

Description

Error with Permissions-Policy header: Unrecognized feature: 'interest-cohort'.
DevTools failed to load source map: Could not load content for chrome-extension://hlepfoohegkhhmjieoechaddaejaokhf/build/refined-github.js.map: HTTP error: status code 404, net::ERR_UNKNOWN_URL_SCHEME
DevTools failed to load source map: Could not load content for chrome-extension://jbmgjfpepenggaeblnmpdbjapdhkjeep/common/browser-polyfill.min.js.map: HTTP error: status code 404, net::ERR_UNKNOWN_URL_SCHEME
DevTools failed to load source map: Could not load content for chrome-extension://jbmgjfpepenggaeblnmpdbjapdhkjeep/common/purify.js.map: HTTP error: status code 404, net::ERR_UNKNOWN_URL_SCHEME

Clicking on "More" yields no results
I also have an organization. When I switch the dashboard context to it, I'm seeing the activity for it.
When switching back, it's the same problem again (no results).
Disabling refined-gh + F5 and the results are visible. Re-Enabling: error like in the screenshot.
Example URL
https://github.com/
Browser(s) used
Chrome  Version 93.0.4577.82 (Official Build) (64-bit)

Is it broken or is it empty? It might be empty because every event is noise: hide-noisy-newsfeed-events

Yup, that was it. It's not "noise" though - at least not from my perception.
The entries that are hidden by this feature are messages about commits and releases from others on repos I'm working on (not my own).
I had been using the refined extension for a long time and re-installed these days after short a break (no more than a few months). The problem hadn't existed earlier (exact same setup).
It's ok for me, know that I know how to switch it off, just wondering whether the noise-conditions make sense as a default behavior.
Thanks a lot for your assistance,
sw

Hm, then the only possible explanation would be that I might have encountered this years ago already (my environment and work habits haven't changed much and it was always the same machine which I have replaced recently), changed that option and forgot about it.. 😉
Thanks again, I could have gone through the options a hundred times without finding the culprit!
Also thanks for the extension, I think it takes an extra amount of passion to develop and maintain this, as you probably often don't get the appreciation that it deserves, because:

user installs plugin and plays a bit
user forgets about the installed plugin
as time goes by, user discovers refined-gh features

one after another
user thinks: "oh nice, GitHub has introduced a nice new UI feature"

when accessing GH from another computer or browser

user thinks: "something's wrong with GitHub today, they must have a technical problem"
(happened to me at some point..)

It seems to be a somewhat cursed situation: the only way to get a full understanding of all the little benefits that refined-gh provides, is to uninstall it after using it for weeks or even months..
Only then, you'll be really ready to say: Good Job!


GCC 5.2 ISL error

How can i fix this?:
https://gist.github.com/anonymous/2c083cb0300cf631c528

From your brew doctor:
Warning: Xcode 7 lacks the 10.10 SDK which can cause some builds to fail.
We recommend installing the Command Line Tools with:
  xcode-select --install
to resolve this issue.

thks, It worked!


feat: mark form template as archived instead of deleting them right away

Summary | Résumé
closes https://github.com/cds-snc/platform-forms-client/issues/1166
linked to https://github.com/cds-snc/forms-terraform/pull/271

Changed the form template delete API function in order to mark them as archived instead of deleting them

Test instructions | Instructions pour tester la modification

Delete a form template using the admin UI
It should disappear from the list of templates in the UI but will stay in the database (yarn prisma:studio)

Pull Request Checklist
Please complete the following items in the checklist before you request a review:

[ ] Have you completely tested the functionality of change introduced in this PR? Is the PR solving the problem it's meant to solve within the scope of the related issue?
[ ] The PR does not introduce any new issues such as failed tests, console warnings or new bugs.
[ ] If this PR adds a package have you ensured its licensed correctly and does not add additional security issues?
[ ] Is the code clean, readable and maintainable? Is it easy to understand and comprehend.
[ ] Does your code have adequate comprehensible comments? Do new functions have docstrings?
[ ] Have you modified the change log and updated any relevant documentation?
[ ] Is there adequate test coverage? Both unit tests and end-to-end tests where applicable?
[ ] If your PR is touching any UI is it accessible? Have you tested it with a screen reader? Have you tested it with automated testing tools such as axe?

Will wait for @bryan-robitaille to merge feature/permissions since it would add more conflits to resolve for him.


Add Fred Hollows Foundation to charities

SS from localhost

Awesome, thanks for adding this new charity @rougee!


Mod05 - Traffic Manager Lab Could Not Be Completed

Module: 05
Lab/Demo: 02
Task: All
Step: 01
Description of issue
DNS Errors encountered by all students when doing the Traffic Manager section of the lab. Instructor encountered same issues. Lab could not be completed.
Repro steps:

There are issues with the Azure Polices that are restricting the Cloud Slice Subscription from allowing the required task steps to be undertaken by the Students / Instructor.

The DNS configuration steps are certainly wrong. We spent time trying to find a resolution but were unsuccessful.

MOC Labs Review did not experience this issue.


Trajectories should be deque, not lists (while generating)

When looking over #1066 I noticed that when the engine is running backwards we are adding to the trajectory via:
trajectory.insert(0, snapshot.reversed)

This is slow, and becomes slower when the list is longer. As we are only ever appending to the trajectory on the ends, the appropriate data structure should be deque instead.

Access by index becomes slower (especially in the middle) so that might need to be handled. Ideally a trajectory would be a deque while running/generating and converting them to a list when finalizing.

Here are some runs for the 1000-10_000 range (more typical for trajectories)
1000

10_000

True, but is there any reason to believe that this is a performance bottleneck in real-world use? I think that we have enough overhead in other places that we'd get better performance improvements elsewhere.
An update for this would be welcome, but I don't think it's urgently required (so I probably won't implement it, personally).
One note: some aspects around the DynamicsEngine.generate and DynamicsEngine.iter_generate are likely to be significantly changed in 2.0. First part of that change in #1042, but I'm also don't particularly like the way DynamicsEngine.generate does this for trajectory in it: pass thing -- it seems wasteful to create an entire new trajetory object that we just throw away over and over again (overhead of UUIDs, etc). I'm open to significant changes in that for OPS 2.0, such as returning an iterator of snapshots instead of trajectories and only wrapping it in a trajectory in the generate method.

True, but is there any reason to believe that this is a performance bottleneck in real-world use? I think that we have enough overhead in other places that we'd get better performance improvements elsewhere.

I would guess so as well (outside of strange toy-engines that are both fast and generate long trajectories). This was just a thing I noticed when looking over that PR and I made this issue to get it off my (mental) list. I agree that this should not have any priority.


Experimental highlighting of subsets, with custom shader

See email... (will send soon)

That's really efficient and elegant! Thanks a lot @astrofrog !

It's great ! Just kind of blur now. But we could implement the OpenGL part later.

Replaced by https://github.com/glue-viz/glue-3d-viewer/pull/55


Using this plugin with bootstrap/django validators

const FILE_SELECTOR_IDS = ['#id_photo', '#id_file'];
let submitData = {};

function sleep(ms) {
    return new Promise(resolve => setTimeout(resolve, ms));
}

function getErrorString(errors) {
    let errorStr = '';
    $.each(errors, function (index, error) {
        errorStr += '<p>' + error + '</p>';
    });
    return errorStr;
}

let currentImageLength = $('.image-list > img').length - 1;

function addThumbnails(data) {
    let files = data.originalFiles;

    if (FileReader && files && files.length) {
        files.forEach(function (value) {
            let FR = new FileReader();
            FR.onloadend = function () {
                console.log('wow')
                if (currentImageLength < 5) {
                    let blankImage = $('#image-list').children().last();
                    blankImage.parent().append(blankImage.clone());
                    blankImage.attr('src', FR.result);
                    currentImageLength = $('#image-list > img').length - 1;
                    if (currentImageLength === parseInt($('#id_addimage').data('max')))
                        $('#image-list > img:last').remove()
                    $('#id_imageloader').prop('disabled', true)
                }
            }
            FR.readAsDataURL(value);
        })
    }
}

function overrideFiles(data) {
    let fileBuffer = [];
    if (submitData.files)
        fileBuffer = submitData.files;
    submitData = data;
    console.log(submitData.files)
    console.log(fileBuffer)
    submitData.files = $.merge(fileBuffer, submitData.files);
    return submitData;
}

$(document).ready(function () {
    FILE_SELECTOR_IDS.forEach(function (entry) {
        if ($(entry).length) {
            $(entry).fileupload({
                dataType: 'html',
                dropZone: $(entry).parent(),
                sequentialUploads: false,
                autoUpload: false,
                singleFileUploads: false,
                start: function (e) {
                    $('#modal-progress').modal('show');
                },
                stop: function (e) {
                    sleep(500).then(() => {
                        $("#modal-progress").modal("hide");
                    });
                },
                add: function (e, data) {
                    // submitData = overrideFiles(data);
                    $('input[type=submit]').off('click').on('click', function (e) {
                        e.preventDefault()
                        let form = $(this).parents('form:first');
                        if(form[0].checkValidity()) form[0].submit();
                        else form[0].reportValidity();
                    });
                    addThumbnails(data);
                },
            });
        }
    })
});

This is my code which works as expected IF I don't do form[0].submit() but submitData.submit().
Ok, that's is nice, but MVP validators like in Django need to send HttpResponse() after form.submit(), so what I expect is NOT reload from website using javascript, but handle HttpResponse (or HttpResponseRedirect) to get working form.errors properly into MVP (Django, or Laravel) template.
This what returns form_invalid in Django:
    def form_invalid(self, form):
        """If the form is invalid, render the invalid form."""
        return self.render_to_response(self.get_context_data(form=form))

Yeah I can load the exact HTML into site using $('html').(response); but that's not what I want. Any ideas?

What I want, to have working $(form) or $(form[0]) submit functionality with page reload to get errors in HttpResponse and I'm not sure how can do that. I tried also adding files from submitData into form type='file' but was not able too, since files from submitData are array and HTMLFileInput expect them to be FileList.
I also though about disabling XHR or messing with fileInput and fileReplaceInput class options...
Also I tried to read documentation and search stack overflow for basic form functionality using this plugin and was not able to implement it.

Hi @HoshiYamazaki,
you might wanna check out the various Django integration examples on the Wiki:
https://github.com/blueimp/jQuery-File-Upload/wiki#python
Please also have a look at #2190

Sorry, but the question was not Django-only specific but is also correct to various MVP frameworks and I used it only as example. I think it's very helpful to close every thread writing "look at #2190" where there is NO ANSWER at all.
I think you miss some kind of functionality if there is no possibility to send form without AJAX call.
Please also have a look at #2190 ;)

https://stackoverflow.com/questions/41036208/add-files-from-dropzone-to-form
This is what I meant, ALL I want is to just add files to form or send them in standard way without overriden $(form).submit() (without unnecessary ajax call). I want do it because I don't have saved object in db and I want to add photos and create object instance in one request call.
Examples in the FAQ are not helpful AT ALL.

Ok, now I get it, this can't be done also in dropzone.js due to security reasons (but in chrome adding files to FileList is possible, not sure if with other browsers too). This explains everything, have a nice day.

No worries @HoshiYamazaki, I can understand that it's very frustrating when a maintainer closes your request without you having received the help you needed.
The problem is that many open source maintainers cannot actually afford to provide extensive support for their projects.
I used to do that for jQuery File Upload and I spent so much time triaging issues, answering questions and still continuing to develop the project further that it became a second full-time job.
This was not healthy nor sustainable and I actually closed the issue tracker for a couple of years because of that.
I've since reopened it, mainly so people can still report bugs, but I don't offer any support for this project.
So yeah, I hope you can find a solution to your problem, but I also hope you can understand my situation and that of other maintainers.


Rename "Fez"

The term "Fez" has been culturally deprecated. At least one person brought up the functionality name in 2023.

In private discussions, we elected to update the public URLs in the site UI to /lfg in place of /fez (we already have /seamail for its usage). This achieves minimum viability.


Host the app somewhere

Try to host the app somewhere so that during the workshop, we can just all look at the hosted app instead of having to run the end app locally. Just less things that can go wrong.
Will need the "database" to live somewhere. Look into Firebase: https://css-tricks.com/intro-firebase-react/

Closed by #47


hangs when disconnecting device using pulseaudio backend

This is a known issue, and may be a problem with miniaudio, but I'll leave this issue here in case anyone else has the same problem.
When playing out to a device using the pulseaudio backend, and the device is disconnected, the render continues as if the device was operating normally. What should happen is that the render is aborted, and an error message is displayed to the user. Since we do not get an error, the device is still used and results in a deadlock (hang) when trying to use it again. Miniaudio does not report an error when stopping the device, so there is no way to know if the device disconnected.
A possible workaround would be to implement a watchdog in the render function, such that if the buffer remains full for more than 1
second or so, consider the device inoperable and abort the render.
I have posted an issue in the miniaudio repo, will update when I hear back from the maintainer.

So apparently the device is "suspended" when this happens and pulseaudio does stream routing, so miniaudio considers the device still running. A recent commit fixes this issue, by invoking the stopCallback when suspended but it requires MA_DEBUG to be defined. For now, we'll stick with the watchdog workaround, but I'll leave this issue open until a better solution is found.
The dev plans on adding an event system for things like starting, stopping and switching, so hopefully that system will work better for us once it's implemented.


No vetiver_pin_read method

Hi - I don't seem to be able to import the vetiver_pin_read function from the latest development version of this package.
Describe the bug
When importing vetiver_pin_read, an error message is displayed:
AttributeError: module 'vetiver' has no attribute 'vetiver_pin_read'

Does vetiver_pin_read need to be added to __init__.py ?
Thanks!

Hey there, thanks for the report @josho88! The function vetiver_pin_read was deprecated and then removed in favor of a more pythonic VetiverModel.from_pin(board, pin_name) as a clearer description that the results of the function are a VetiverModel object created from that pin. Here's an example of a round trip pin. Let me know if you have any questions about using this method!

Ah I see, thanks that makes sense!


Outdated WebRTC Version breaks build

Describe the bug
Hey folks,
checking out this repository i ran into some issues with the build scripts. Running with the Android Studio 2020 version the app can be built with the outdated gradle and android build tools versions. However the google-webrtc dependency introduced the lastest  PR is not outdated and not maintained. Additionally Jcenter shutdown and redirects the requests to Maven Central (see https://jfrog.com/blog/jcenter-sunset/) which doesn't host the package.
Could not find org.webrtc:google-webrtc:1.0.32006.
Required by:
    project: app

I'd suggest to use a different webrtc dependency. e.g. https://github.com/GetStream/webrtc-android?tab=readme-ov-file or the one from Twilio
Steps To Reproduce
No response
Screenshots
No response
Drone Model
N/A

Smartphone Model
- Model:
- Android version:

Additional context
No response

Thanks.
@angel-ayala can you take a look please?

Quoting From the JCenter sunset note:

What should a user do if a package they need is not available in Maven Central?
If a specific package version you need is not available on Maven Central, you have the following options:

Contact the package owner and request that they make it available on Maven Central
Check for a more recent version of the package on Maven Central
Remove your dependency on the package
Copy the package to your organization’s internal repository
If you are a JFrog user, and none of these options work for you, please reach out to<EMAIL_ADDRESS>

For webrtc its seems we can use this package:
dependencies {
    implementation("io.getstream:stream-webrtc-android:1.1.3")
}

But also i found that the mapbox-android-sdk:9.2.0 links to jcenter packages which are not available anymore. I guess you still have a local copy in the package cache if you had a successful build before Aug 15th
Could not find com.mapbox.mapboxsdk:mapbox-android-accounts:0.7.0.
Required by:
    project :app > com.mapbox.mapboxsdk:mapbox-android-sdk:9.2.0

https://github.com/mapbox/mapbox-gl-native-android/issues/721 Mapbox sdk is archived and there seem to be no plans to release those sdks on mavencentral. I guess the dependency need to be updated  to make it work again for new users of rosettadrone
Found the related issue created by @angel-ayala at the DJI MSDK https://github.com/dji-sdk/Mobile-UXSDK-Android/issues/154

There is another explanation, but I don't want to mix different issues here.

This issue was for @angel-ayala (the webrtc issue), but since you are mixing things here and the title is not clear anymore, I'm closing this one.

Are you sure you provided a Google Maps API key as explained in the README?


chore: move "plugin:typescript-sort-keys/recommended" to override scope

PR Checklist

[x] Addresses an existing open issue: fixes #69
[x] That issue was marked as accepting prs
[x] Steps in CONTRIBUTING.md were taken

Overview
Moves TypeScript interface & type key sorting in ESLint to an override scope so that it only applies to TypeScript files.

I don't know why Compliance / compliance (pull_request) is failing. We can ignore that. 🤷 if it keeps happening I can look more into it.

@allcontributors add @NazCodeland for tooling
(sorry, I'd forgotten I'd set this bot up!)


Review corner rounding of elements

Review corner rounding of elements

Button radius set to 10, however in Bootstrap 4 the common border-radius: .25rem;

Right now is set to .25rem => 2.5px. is that the correct setting?

It’s 4px
Kind regards,
Anna Laeder
Electronic Knowledge Interchange
33 W Monroe St. Suite 1050
Chicago, Illinois
alaeder<EMAIL_ADDRESS>847-337-2288(cell )
<EMAIL_ADDRESS>
From: Christian Calderon<EMAIL_ADDRESS>Sent: Friday, December 6, 2019 9:38:03 AM
To: Chicago/design-cds-bootstrap<EMAIL_ADDRESS>Cc: Anna Laeder<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [Chicago/design-cds-bootstrap] Review corner rounding of elements (#42)
Right now is set to .25rem => 2.5px. is that the correct setting?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/Chicago/design-cds-bootstrap/issues/42?email_source=notifications&email_token=ANWKKU6LXU2O264WCEQ55KLQXJWVXA5CNFSM4JTYTNYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGEOXTI#issuecomment-562621389, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANWKKU5MXLEHCGLVYZJ2UU3QXJWVXANCNFSM4JTYTNYA.


document what each script and each dotfile do

https://github.com/joeybaker/generator-iojs/commit/97cb421503e9a62c6041b457692ce50c0f6866d1 done


results taken from camera

Your library is great man! however i encountered a problem when photo taken from the camera, it doesn't give result.

Please provide more information of this problem

Camera doesn't start when choosed.
Tested on Android 8.1.0 API 27.

It could be a permission issue, could you try out the latest release

Hi @maayyaannkk  , I too faced issue with Camera @mermaidshop  mentioned? Any solution!!

Hi @maayyaannkk , Not getting camera result , In case of Fragment Activity.

No camera result for intents with only camera.
Intent intent = new Intent(getContext(), ImageSelectActivity.class);
intent.putExtra(ImageSelectActivity.FLAG_CAMERA, true);
intent.putExtra(ImageSelectActivity.FLAG_GALLERY, false);
startActivityForResult(intent, TAKE_IMAGE_ID);

String filePath = data.getStringExtra(ImageSelectActivity.RESULT_FILE_PATH); Bitmap bitmap = BitmapFactory.decodeFile(filePath); //returns null


Explore GLI

http://naildrivin5.com/gli/

It's really slow and bloated. Sticking with plain old Ruby and optparse


Update calicoctl command reference for v3

Description

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Awesome thanks for the PR @gunjan5

If possible, can you squash your commits? May be some broken links, as well.

@emanic just did :)


Changed binary annotations type from binary to string

The thrift Annotation Type doesn't conform to the actual type it encodes. Notice the  TraceScope.addKVAnnotation function signature:
public void addKVAnnotation(String key, String value)

Using StandardCharsets doesn't throw the UnsupportedEncodingException

@adriancole I wanted to but I think the difference only happens after passing through the whole thrift encoding/decoding process and I couldn't find a place this is done.

The key thing here is testing what you're asserting (ex that you fixed the bug as before we weren't encoding strings properly).
Here's a test that should cover it.

  @Test
  public void testKVSerializesAsUTF8String() throws IOException, TException {
    Span span = new MilliSpan.Builder().
        description(ROOT_SPAN_DESC).
        spanId(new SpanId(100, 100)).
        tracerId("test").
        begin(1).
        build();
    span.addKVAnnotation("http.path", "/foo/⻩");

    com.twitter.zipkin.gen.Span zs =
        new HTraceToZipkinConverter(127<<24|1, (short) 80).convert(span);

    assertEquals(1, zs.getBinary_annotations().size());
    BinaryAnnotation path = zs.getBinary_annotations().get(0);
    assertEquals(AnnotationType.STRING, path.getAnnotation_type());
    assertEquals("/foo/⻩", new String(path.getValue(), UTF_8));
  }

@adriancole The test does cover it and I wrote a test a different test before sending the PR but it wouldn't fail.
I'm pretty sure the problem lies in https://github.com/openzipkin/zipkin/blob/master/zipkin/src/main/java/zipkin/internal/ThriftCodec.java#L244
Where the type should be inferred as String but was inferred as Bytes
 @Test
    public void testBinaryAnnotationEncoding() throws IOException, InterruptedException {
        FakeZipkinTransport transport = new FakeZipkinTransport();
        Tracer tracer = newTracer(transport);
        Span rootSpan = new MilliSpan.Builder().
                description("root").
                spanId(new SpanId(100, 100)).
                tracerId("test").
                begin(System.currentTimeMillis()).
                build();
        TraceScope rootScope = tracer.newScope("root");
        rootScope.addKVAnnotation("foo", "bar");
        rootScope.close();
        com.twitter.zipkin.gen.BinaryAnnotation binaryAnnotations = transport.nextMessageAsSpan().getBinary_annotations().get(0);
        Assert.assertTrue(binaryAnnotations.getKey().equals("foo"));
        Assert.assertTrue(new String(binaryAnnotations.getValue()).equals("bar"));
        tracer.close();
    }

Or do we place the test there to enforce this change?

I'd recommend using the test I wrote as it would fail before this change, particularly as the wrong type was used. merging code with no tests isn't a good plan, especially when you are handed one!

@adriancole I totally agree, I just wanted to make sure I am testing for the right things.

@raam86 yeah I would consider this backfilling a unit test vs adding an end-to-end test (which this isn't). It is still valuable.
If you want an end-to-end test, you could add a dependency on zipkin-collector-scribe and use that directly. something like this https://github.com/openzipkin/zipkin-reporter-java/blob/master/libthrift/src/test/java/zipkin/reporter/libthrift/LibthriftSenderTest.java

Thank you for the suggestion, @adriancole. I've added the test but I still can't make it fail when changing encoding type.
In the zipkin frontend this change definitely fixes the reported bug but I can't find a way to reproduce it in a test.

I Hope it's just the beginning 👯


Not recognizing organization members

I'm using GHPRB for the first time on a GitHub Enterprise instance.  I have an organization owned repo fork set to trigger GHPRB.  I set the name of the organization (like foo-org) under List of organizations. Their members will be whitelisted..  Although I didn't think it would be necessary, I also set Allow members of whitelisted organizations as admins.  I am a member of the organization, and the bot user has been added as a collaborator.
However, when I create a pull request, I get told that an admin needs to approve the pull request; it doesn't seem to automatic whitelist me as an organization member.  It's only when I actually add myself as an admin explicitly that pull requests will trigger the job.
Do I have something configured incorrectly?  If not, how I can debug this?  There's not a whole lot in the Jenkins log to reveal details about checking the organization members, etc.

I'm facing this issue as well.  It looks like maybe GH's API isn't returning members.

I'm also seeing this (on a GitHub enterprise 2.8.3 system).  Here's the tail end of my log file:
[George Hartzell] Added comment: retest this please
Dec 11, 2016 1:05:33 AM FINEST org.jenkinsci.plugins.ghprb.GhprbPullRequest
Retest phrase
Dec 11, 2016 1:05:33 AM FINE org.jenkinsci.plugins.ghprb.GhprbGitHub
org.hasMember(member)? user:hartzelg org: ci-template == no

After a bit of poking around, I've come to the conclusion that:

if this user is a private member of the org, then the token you're using must have read:org (Read org and team, under admin:org) or admin:org for this query to return 'yes'; or
that the user must be a public member of the org.

@DavidTanner -- perhaps the documentation could be touched up?  I tried running the Create API Token process, but no matter what I put in the Username temp and Password temp fields the response is a bright red "No username/password credentials provided".
This set of scopes seems to let me do everything I need to do (I'm only working on public repos, so far):
admin:repo_hook, public_repo, read:org, repo:status


Update constants.py

Reference issue

What does this implement/fix?

Add missing SI prefix for yocto (1e-24, https://physics.nist.gov/cuu/Units/prefixes.html)
Additional information

CI failure is unrelated. I will merge. Thanks @mefuller 😃

Thank you @tupui


[hep] Alternative to koa-controller?

Installed nodejs and koa to give it a try since Laravel libs for RethinkDB isn't good enough.
I found koa-controller to have views and controllers but I got a message saying it's going to be deprecated soon.
Any good alternatives? :)
Thanks!

Found routing-controllers.
thanks anyway


TypeError: jwt_decode__WEBPACK_IMPORTED_MODULE_5__ is not a function

When using jwt-decode I receive this error: "TypeError: jwt_decode__WEBPACK_IMPORTED_MODULE_5__ is not a function"
I have been using jwt-decode for many years and am currently using jwt-decode version 3.0.0-beta.2 and recently upgraded to Angular 10 (v10.1.3) using the angular update tool in Angular CLI
import * as jwt_decode from 'jwt-decode';

export class AuthService {
...
  decode(token: string): any {
    return jwt_decode(token);
  }
...
}

Not sure if this actually has anything to do with webpack as the error suggests, but I am on versions
"webpack": "^4.44.2",
"webpack-cli": "^3.3.12"
I also tryed installing the type defs for jwt-decode, but they haven't fixed anything
"@types/jwt-decode": "^2.2.1",

This might be a duplicate of #96
Try importing the library as import jwt_decode from 'jwt-decode';

That seems to fix the issue. Is this issue understood?
As, I said I've been using jwt-decode for many years and never had any issues with this. Maybe something in another dependency?

jwt_decode has always been a CommonJS module which generally are imported as const jwt_decode = require('jwt-decode');, it's what Node.js uses.
The way to import CommonJS libraries with a JS import statement is import * as library-name from 'library-name;.
Modern frameworks like Angular 10 throw a warning when using packages with the CommonJS format, because they generally speaking can't be tree-shaked.
Version 3 (now in beta) of this library includes a more modern ESM build, which is what JS import statements are meant for, so importing the modern ESM package can be done the regular way using import jwt_decode from 'jwt-decode';. This is a breaking change, that's why I've created a new major version 3.
We still have a CommonJS build, but by default most modern build systems for the web (not node) will use the ESM build.
if you checkout the package.json file when installing the lib, you'll notice both builds in there.
{
    ...
    "main": "build/jwt-decode.cjs.js",
    "module": "build/jwt-decode.esm.js",
    ...
}

I hope this helps :)

I'm updating the docs soon, to indicate this change when importing.

Thank you for the explanation. I see now that I updated to the most recent version of jwt-decode without realizing.
Thanks for keeping the project maintained! :)

If you are in react, you can import it like this:
import { jwtDecode } from 'jwt-decode';
``` `

instead of:
```typescript
import jwtDecode from 'jwt-decode';
``` `


TypeError: src1 data type = 17 is not supported

I got this error at
dog_images_in_octave.append(subtract(second_image, first_image)) of generateDoGImages(). How should I solve it? Many thanks!

Hi @KurumiKakako , can you please send me the images you are using? Are you just using the images included in this repo? Also, can you please list your python, numpy and cv2 versions?

Hi @voldenuit-yanagi , I haven't heard from you, so I will close this issue. If you can supply me with your images you're using, feel free to reopen this issue.


Revert "Re-enabling testing against dev"

Reverts dart-lang/dart-services#402
Turns out dev isn't happy, still.

For reference: https://travis-ci.org/dart-lang/dart-services/builds/527535343

lgtm
(for a revert to unbreak things, you can generally just land TBR)


DO NOT MERGE: run testsuite with tiered compaction enabled

Running testsuite with the new compaction from #6830.
DO NOT MERGE
part of #6768

PR's purpose is fulfilled. The list is quite long, wow, report link


Feature Request:  json output

Is it possible to have GAM output commands like user info and print users print their results as json?  I'm guessing that's how the data comes into GAM from the Google API, so would it be possible to pass it back out in the same format?

Many GAM7 commands have a formatjson argument now. Upgrade and try that.


Support for 2.7 ?

So https://github.com/pybee/Python-iOS-template/tree/2.7  points that 2.7 is supported, however briefcase does not.  Does Briefcase support 2.7 or not?

Briefcase does not support Python 2 (at least, not any more).
At one point it did, but we dropped that support a while back, due to the impending EOL of Python 2.

Thank you @freakboy3742


Add solver to admin metadata for numerical test cases

The modelling approach for test cases can be "Numerical" or "Physical".
For physical we need the facility in the admin metadata (which we already have) but for numerical test cases we need solver instead.
So:
For numerical test cases: add solver to admin metadata sidebar.
For both cases add text ("for  modelling Test Cases") next to the word "Facility" or "Solver"

It's more complicated to add a solver than first thought.
It doesn't appear to be possible to add a solver to the schema and admin page without a warning message appearing on the admin page: "key "null" not found in collection "software". (It does this for Project in the Test Case schema too but it's less obvious because that is set up as an array to allow multiple projects (in error) - see #122). It's not a problem, per se, it just is an unnecessary warning message which might confuse the user.
The best semi-workaround I can think of is to set it up as an array like project is, and set the max items to 1 so it's an array which can only have 1 solver in it. This means the warning message is hidden from the initial view, and only seen if a solver is going to be added. It also has the advantage that you can delete the solver if you added it by mistake (see #1 )

Another work around required - need to add a "help" message to the contact page saying to add Solver "If Modelling Approach is numeric". But for arrays, the help message doesn't appear. To work around this, I've added it to the label section of solver as, because it's an array, we already have the label Solver at the higher level (i.e. the help label replaces what would otherwise be "foreignkey".

Just need to add a little more padding between "for Numerical Test Cases" text and the solver as there's less space with pictures than text(?!) i.e. it's not so bad between "for Physical Test Cases" and the facility name...

Closing this for now despite minor styling requirement left.


[Mention The Feature]: add a google translator feature

Is your feature request related to a problem? Please describe.
add a google translator feature for those who cant understand english
Describe the solution you'd like.
I can add a translator to this web page
Describe alternatives you've considered.
No response
Additional context.
No response
Show us the magic with screenshots
No response
Checklist

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

@HS202022 on which page you want to add this feature.

i thought of placing it in the nav bar user might get language difficulty in any of the pages
so its better to put it in the nav bar

Okay, @HS202022 assigning this issue to you.
Make sure to complete this within 2 days.
And please don't make any changes in navbar. just add the feature in it.

Okay, @HS202022 assigning this issue to you.
Make sure to complete this within 2 days.
And please don't make any changes in navbar. just add the feature in it.

for sure

@HS202022 After this issue, make sure to contribute to other projects also.


Remove local alloc/free uses of malloc

https://github.com/bespoke-silicon-group/bsg_f1/blob/12617602a16b781418463f639d31abeb1adae9a0/cl_manycore/libraries/bsg_manycore_cuda.cpp#L758
My possibly flawed opinion is that it's faster (no call to glibc), and less error prone, to stack allocate.
@max-ruttenberg, your thoughts?

Stack allocation typically makes for cleaner programming because

You don't have to free it - one less potential bug.
Syntactically (I think) it's easier to read. The above line would be re-written like this to stack allocate:

hb_mc_coordinate_t tile_list[num_tiles];

I would imagine that it would be faster for a couple of reasons. No function call to glibc, stack allocation is typically as trivial as adding a value to your stack pointer, better data locality, etc.

@drichmond is correct that in the above example there is really no reason why tile_list needs to be heap allocated -- It's freed anyway before it would go out of scope. There's a misconception that C requires dynamically sized arrays to be allocated with malloc and friends. It's a nasty misconception that's probably left over from the old days of C. You can use the line shown in (2) to allocate your array on the stack (and you should).
There ARE times to use malloc and friends;

When you want the memory that you're allocating to persist after your function returns.
You have a global pointer that you want initialized to point to a dynamically sized buffer.

While this line of code is not a good example of when to use malloc, there are plenty of appropriate uses of malloc in bsg_manycore_cuda.cpp so certainly don't try to get rid of ALL of them.

Cool. I've updated the title to be more specific

Thanks for the Stack-Overflow like explanation, Max!

All of this makes complete sense to me, and thanks a lot for the detailed explanation @max-ruttenberg ! I will iterate the code and remove the unnecessary mallocs like this one.

Addressed in PR #275 .


Check if manifest exists before downloading

Issue #, if available: https://github.com/aws/eks-anywhere/issues/1753
Description of changes:
CAPC/CAPT providers are not GA and not a part of the bundle-release.yaml corresponding to releases 0.7.x and 0.8.x
But all providers are part of the dev bundle-release.yaml when they get added to the manifest list to download for download artifacts. This PR adds a check to see if the provider's artifacts exist in the bundle-release.yaml
This will ensure download artifacts runs against any bundle-release.yaml irrespective of which providers are added in a release.
Testing (if applicable):
Tested against release-0.8 branch
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherry-pick release-0.8


Investigate windows resizing/moving on restart

Witnessed in Spell Timers, Map & Mob Info.
Mob Info makes sense to resize since it shrinks/expands.
Look into not shrinking.

Moving on restart = Not remembering where they were.

The overlay window works for me and stays where I put it.  but all of the other windows reload in default positions and sizes when I open the app.  I use 2 monitors on my laptop, not sure if that has anything to do with it.  But the windows will load on whichever monitor I open the app from (opened from taskbar shortcut)

But the windows will load on whichever monitor I open the app from (opened from taskbar shortcut)

That sounds like it's a monitor issue.
Does it happen if you open the app, move the windows around, close them (one by one), close the app entirely, then immediately reopen it?

Issue still persists for me in [Release 11 - v1.0.9].  I have opened the app, positioned windows and closed individually and they did not retain their size or position on my 2nd monitor.  Its like they revert to default if I place them on the 2nd monitor.  They do retain their positions on the primary monitor though.  Thanks for the work, great project.  I'll dig around and see if I can see why some apps work and retain on an additional monitor and some do not.
Thanks you


how to print accurate in this code?

Thank you very much for the excellent work you do
I successfully ran your code, but I noticed that the output contains these three data:
training/train_loss_mean
training/train_loss_std
training/action_error
but not the accuracy data, what should I do if I want to print the accuracy of this code?

Do you know the answer? I am also troubled by this question

troubled by this question, too.


ref is not a prop. Trying to access it will result in undefined being returned.

Warning: InfiniteScroll: `ref` is not a prop. Trying to access it will result in `undefined` being returned. If you need to access the same value within the child component, you should pass it as a different prop. (https://fb.me/react-special-props)
It would be great to change the name of 'ref' prop to something other, because it is reserved in React for special props.

A code example would be nice. I don't see the need for accessing the ref prop on any component. You should be passing function refs instead of strings, yeah? Then there's no need to call on InfiniteScroller.props.ref.
<InfiniteScroll ref={c => this._scroll = c}>
    {items}
</InfiniteScroll>

****Hi. Any updates here?

Hi.Is There  no final solution？

Is this lib abandoned?

Hello,
I try to delete all ref props, it work fine, without error in my project, but the test 'should attach scroll listeners' fail in the plugin...
Has anyone found a solution ?

Any solution to start at bottom?

For those who are looking for a solution. I created an PR here https://github.com/danbovey/react-infinite-scroller/pull/268, hopefully this repo maintainer can merge it and publish a new version.
Before it merged, you guys can temporarily use this package alternatively "@meikidd/react-infinite-scroller"

For those who are looking for a solution. I created an PR here https://github.com/danbovey/react-infinite-scroller/pull/268, hopefully this repo maintainer can merge it and publish a new version.
Before it merged, you guys can temporarily use this package alternatively "@meikidd/react-infinite-scroller"

I needed to reset the scroller when using a search input and instead of using ref I fixed it another way by increasing the key when needed.
const [scrollKey, setScrollKey] = useState(1);
<input onChange={() => setScrollKey(scrollKey + 1)} />
<InfiniteScroll key={scrollKey} />

Not as performant I suppose, but it works.


Bad assertion on TextFormField

Flutter told me to file this issue as shown below:
I/flutter (11570): ══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
I/flutter (11570): The following assertion was thrown building MyForm(dirty):
I/flutter (11570): type '(String) => dynamic' is not a subtype of type '(String) => String'
I/flutter (11570):
I/flutter (11570): Either the assertion indicates an error in the framework itself, or we should provide substantially
I/flutter (11570): more information in this error message to help you determine and fix the underlying cause.
I/flutter (11570): In either case, please report this assertion by filing a bug on GitHub:
I/flutter (11570):   https://github.com/flutter/flutter/issues/new

I've made a demo repo to show why this is a problem (explanation is in the README).

@jonahwilliams ah, that fixes the IDE error.
As for the error received, should I close this issue, or leave it open? Flutter said to report it in either way.

I can close it, glad to help!


shouldFail &c. in WordSpec

Please consider adding shouldFail and the various shouldThrow[...] functions to WordSpec as infix String functions, as should currently is. The presence of String.should in WordSpec suggests the existence of the others. Also, their lack leads to unwieldy DSL expressions:
import io.kotlintest.shouldBe
import io.kotlintest.specs.WordSpec

class MathTest: WordSpec({
    "wrong assertions" should {
        shouldFail {
            1 shouldBe 2
        }
    }
})

I tried adding them myself, but couldn't figure out how to have them call the existing functions, given both the current project structure (AbstractWordSpec in kotlintest-core and the functions in question in kotlintest-assertions) and Kotlin's lack of support for extending inner extensions (i.e. multiple extension receivers, e.g. infix fun AbstractWordSpec.String.shouldFail(...).

Should is defined there so that it doesn't clash with the keyword of the DSL.
Otherwise you can't do this
"my test" should {
  a should exist()
}

So you want something like
"my test" shouldFail {
 ...
}

But that is conflating the should of the spec DSL with should as a matcher.

I hadn't examined the source when creating this issue, and hadn't noticed the inner description strings (WordSpec.WordScope.String.invoke()), which addresses the unwieldy expression aspect.
This issue originally came about when creating tests for matchers. Using inner description strings, information gets duplicated between them and tests that should fail or throw, which has development costs. The cost is small per test, but with a large number of tests gains significance. Would a spec (named e.g. PassFailSpec) with DSL terms supporting  be of use to others? If so, I could take a shot at it. Use would be:
"something" should {
    succeed("for some things") { ... }
    fail("for others") { "a".shouldBe("b") /* note: no `shouldFail` */ }
    abort<IndexOutOfBoundsException>("when exceeding its capacity") { ... }
    "do something more complex in certain circumstances" { ... }
}

throw could be added as an alias for abort (I went with abort as it's not a keyword). It could also potentially support non-nested terms:
"something" should succeed("for some things") { ... }
"something else" should fail("all the time") { ... }
"" should abort<NoSuchElementException>("when ") { ... }

Implementing the DSL methods is straight-forward; what's not so clear is dealing with the cross-project nature (as specs are in core and the exception handling matchers are in assertions). Should I pursue designing this further?

I see what you're saying. You want to combine naming the test with the fail / pass / exception semantics.
I guess when testing the framework itself it would be handy, but when acting as a user of the framework, instead of saying
"string length" should {
  shouldThrow {
    "string".shouldHaveLength(3)
  }
}

You would say
"string length" should {
  "string.shouldNotHaveLength(3)
}

Also in this case, the usage of should is just particular to WordSpec. The other specs have their own nomenclature, eg for StringSpec:
"string length" {
  shouldThrow {
    "string".shouldHaveLength(3)
  }
}

I am in favour of anything that improves the experience for the user, but you do have the issue of the multiple project modules (assertions vs core, and there are plenty of people who only use one or the other).
What might be nice is finding some generic way to do this, so that
"my test" shouldThrow {
  "string".shouldHaveLength(3)
}

could be used in any spec, and then you could also expand this idea out to other things like table driven tests, or property tests, so they don't have to be nested inside test cases anymore, but can themselves form test cases.

I may think about this one more. What inspired this (and what is more useful to me and, I suspect, in general) is supporting matchers on Sequence. I'll finish that up first (simply putting the useful extension methods in the test class).

This discussion was never followed up by the original author, and we didn't really see a big improvement with this.
Should this need rise again in the future, we may discuss it again. For now, I'll close this.


[Feature] 求添加一个符号链接

Checklist

[X] I have already read docs/Tips.md, but my feature is not implemented.
[X] My suggested feature was not mentioned by others, and it is not a duplicate feature.

Summary

求大佬再添加一个符号链接的功能这样，更新的时候一并更新了。
我使用命令行自己搞了一个符号链接可以用，但是手动搞的话也很麻烦，需要命令行创建
大佬，我已经赞助了一下下，求满足
Solution or Design
求大佬再添加一个符号链接的功能

感谢支持，但是 https://github.com/Scighost/Starward/issues/749#issuecomment-2258226089


Problems with fractional zoom levels on leaflet-master branch

When using fractional zoom levels markerclusters are not visible on the map for me. I couldn't manage to reproduce the same error that I had but a very similar one.
As you can see in this jsfiddle http://jsfiddle.net/Eschon/m1cemf5c/3/ the cluster does not appear at first. If you zoom in until the markers appear and zoom out again the cluster shows up, but the markers don't disappear.
If you change back to integer values for the zoom everything works as usual.

Yeah that'll probably be broken with fractional zoom.
Should be able to fix it if we round the zoom level

Ok I quickly looked over the code and was able to fix it. I'll create a pull request so that you can check it out.

:+1:


footer and header

Footer and header for the new website

I'd second @peduarte regarding BEM vs SuitCSS. I personally prefer Suit too. Up to you though as long as we stick to one.


feat(build): enable auto build for arm64 binaries

auto build and push arm64 and amd64 binaries
make tests dependent on the platform

shellcheck, unit-tests and integration tests will be run only on amd64, along with the build. On all other platforms the binary will be built and image will be pushed.

Codecov Report

Merging #371 into master will decrease coverage by 0.49%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #371     +/-   ##
=======================================
- Coverage   51.49%    51%   -0.5%
=======================================
  Files          55     55
  Lines        2637   2637
=======================================
- Hits         1358   1345     -13
- Misses       1172   1182     +10
- Partials      107    110      +3

Impacted Files
Coverage Δ

cmd/ndm_daemonset/probe/eventhandler.go
46.05% <0%> (-7.9%)
:arrow_down:

cmd/ndm_daemonset/probe/udevprobe.go
60.3% <0%> (-5.35%)
:arrow_down:

Continue to review full report at Codecov.

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


Black screen

I have android 13 oneplus 9 mobile. I have installed ubunutu using this command . In vnc viewer I can only see the black screen with small square dot in black screen area.

That problem is with termux itself, as termux is not stable from android 12


Unexpected Behavior with setPeriodic

I have a job that i want to run every 6hours...scheduled as specified below
    public void scheduleNewsPollerJob() {
        Log.d(TAG, "scheduleNewsPollerJob() called with: " + "");
        // .setPeriodic(21_600_000L)
       int jobId =  new JobRequest.Builder(NewsPollerJob.TAG)
               .setPeriodic(21_600_000L)
               .setRequiredNetworkType(JobRequest.NetworkType.CONNECTED)
                .setPersisted(true)
               .build()
                .schedule();
        sessionManager.setNewsPollerJobId(jobId);
    }

the main job is suppose to get latest news from a server and sync it with the database content, also shows a notification if there are new news.
However i noticed a new news notification pop up at less than 6hours intervals since installation on my test device. Subsequent notifications are also not at 6hours intervals, at times i get two notifications coming within 5minutes. Since a notification is the possible end result of a job, this actually means the jobs are not actually run at 6hours intervals.

Which Android version are you using? If it's 5+, then this behavior is expected, because the JobScheduler runs the job once during the 6 hours whenever it's a good time. The timing isn't fixed nor exact. If you want to have an exact job (what I wouldn't recommend), you need to rely on the one-off task.

Yeah, am running 6.0, but it should only run one job within the 6hours interval, so i should only get one possible notification within 6hours interval, i noticed more than one notification within the interval, does it mean JobScheduler  runs job more than once within the specified interval? or is it possible an unsuccessful job is postponed till the next 6hours interval?

I'm not sure right now. Have you made sure that only one job is scheduled?

Just discovered a method silently rescheduling job scheduled already. Thanks

i have this problem...please help me


[$500] IOU - Request details link with /edit/receipt opens Not here page with Receipt header

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.26-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to workspace chat
Create a manual expense without receipt
Navigate to request details page
Add /edit/receipt to the end of URL and navigate to it

Expected Result:
The app should either open receipt upload page, or not here page without the Receipt header
Actual Result:
The app opens not here page with Receipt header
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/ed8bfc3b-2bf8-444b-a524-71cdd6923a8a
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~011b2f947836ddc62f
Upwork Job ID:<PHONE_NUMBER>092935168
Last Price Increase: 2024-01-17

@strepanier03 This can be closed based on https://github.com/Expensify/App/issues/34659#issuecomment-1896222499.

Thanks @akinwale - I was just coming to close this anyway lol.


Add gradle wrapper to the project

allows the "gradlew/gradlew.bat" script to be used to ensure
gradle doesn't have to be installed prior to building the project,
and that all developers share a single version.
The gradle-wrapper.jar, gradle-wrapper.properties, gradlew, and gradlew.bat scripts were generated by running gradle wrapper.
I deleted the existing contents of .gradle because it's a local cache and shouldn't be checked in. It already existed in .gitignore.
[fast-serialization]$ ./gradlew jar
[-XDignore.symbol.file, -nowarn]:compileJava UP-TO-DATE
:processResources NO-SOURCE
:classes UP-TO-DATE
:jar UP-TO-DATE

BUILD SUCCESSFUL

Total time: 0.619 secs

Thanks, note that I do not use gradle build anymore so, I do not review request's regarding gradle ..


Is there a way to export/reload trained model for reusing?

Hi, usually I work with the following steps:
1.prepare data for training->2.train a model->3.evaluation->4.export model->5.reload model->6.predict
So is there an easy way to export/reload trained models? Didn't got it from the document.
Thanks~

If you're under python, you can easily serialize your forecaster instance using pickle.
Do as follows:
import pickle

with open("your_path_to_pickle_the_forecaster", "w") as f:
     pickle.dump(my_forecaster, f)

To reload your forecaster, you can simply do:
with open("your_path_where_forecaster_pickled", "w") as f:
   my_loaded_forecaster = pickle.load(f)

Hope it helps!
Don't hesitate to come back to me if it does not work 😉

Thanks a lot~

write() argument must be str, not bytes
hmm， may you could try open file with "wb" flag rather than "w"

thanks , actually I searched it and found the same solution


EditContext: Change "focus" to "Activate" to avoid confusion with the focus in the activeElement

EditContext's focus doesn't steal the focus away from DOM. This creates confusion so consider renaming it to something else (Activate?)

@gked and I agree! @snianu can you make this update?

Tracking now with this issue in the w3c/editing repo.  Closing this one.


Remove the need for credentials when downloading FNL data

Description
Removes credentials when downloading from RDA.
Since we download GRIBs without any reformatting/subsetting we don't require any credentials.
Checklist
Please confirm that this pull request has done the following:

[ ] Tests added
[x] Documentation added (where applicable)

Notes
We can remove the repo secrets for ORCID and RDA_TOKEN once this is merged

Secrets removed


document "cargo init"

The commit documents the 'cargo init' sub-command in the usage string and in the manpage.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @wycats (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+ 81d1ab240339d5b3b2bc1b8dc41ef45e63bdea0f
Thanks!

:hourglass: Testing commit 81d1ab2 with merge 1a22370...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


doc: implement metacontroller docs site

As a starting point - github pages ? Do you have any thought which static site generator to use ?

I like the way this has been built including the style, font, theme etc.
https://github.com/tinkerbell/tinkerbell.org
So its HUGO & other stuff.

does it allow to generate diagrams (like dot), which can be handy in documentation ? I see it is based on .md sources which is good (as they are pretty simple and powerful)

So the approach would be to have separate repository which will be hosted site ?

Let the site content be in the same repo as metacontroller

@AmitKumarDas what do you think of initially build existing docs and host them, and in second step - change to this hugo ?

@grzesuav you are right. Second step can be delayed.

Actuall done, docs are on https://metacontroller.github.io/metacontroller/ , @AmitKumarDas if we are good with this issue can be closed


can't disconnect/quit

Hello I'm trying to quit/disconnect but proccess still running

class RedisCache extends Redis {
    constructor() {
        super();
    }
}
module.exports = new RedisCache();

and if i just import that file the process would keep running indefinitely , quit, disconnect doesn't work
i tried many things nothing works
ioredis v5.2.5
node 19.4.0

@luin can you check this please

after investigation i found that it did disconnect just what i need was process.exit()


UP board adc updates

Some minor updates for the UP board support, and also a proposed change to the pin matching scheme for AIO channels.

Merged the first part, going to test the second part quickly. I may just add an aio property channel in the mraa_board_t def.

Any news?

Any updates? Will this ever be merged?

@cstatz it looks like the first commit here "update UP board hardware details and enable ADC" was merged after a few minor changes. However, the second commit wasn't merged. @Propanu @pylbert could either of you please check/approve this commit real quick, I will merge it then.

In theory this looks good and should work OK. I remember reviewing it when it was submitted and I liked the idea of not forcing ADC pins past GPIOs, which, as Dan mentioned, is a big limitation for many boards that have ADC pins spread around a multi-function low speed I/O header. Unfortunately, I don't have any of the legacy boards handy to test this further.

There's a conflict applying db4911381d1918ab31d9350c0e10ee20af99b68b.  Can anyone shed a bit of light on which lines to keep (or provide an updated PR).


build a library or web component throw error when enable pwa

Version
3.0.0-beta.15
Reproduction link
https://github.com/anhulife/vue-cli-issue-reproduction
Steps to reproduce
# install dependencies
npm install

# build a web component will throw error
./node_modules/.bin/vue-cli-service build --target wc --name hello-world src/components/HelloWorld.vue

What is expected?
vue-cli-service will produce a single JavaScript file (and its minified version) with everything inlined.
What is actually happening?
this command exit with a error.
Building for production as web component... ERROR  TypeError: Cannot read property 'tapAsync' of undefined
TypeError: Cannot read property 'tapAsync' of undefined
    at compiler.hooks.compilation.tap.compilation (/Users/******/Sources/vue-cli-demo/node_modules/@vue/cli-plugin-pwa/lib/HtmlPwaPlugin.js:18:63)
    at SyncHook.eval [as call] (eval at create (/Users/******/Sources/vue-cli-demo/node_modules/tapable/lib/HookCodeFactory.js:17:12), <anonymous>:15:1)
    at SyncHook.lazyCompileHook [as _call] (/Users/******/Sources/vue-cli-demo/node_modules/tapable/lib/Hook.js:35:21)
    at Compiler.newCompilation (/Users/******/Sources/vue-cli-demo/node_modules/webpack/lib/Compiler.js:451:26)
    at hooks.beforeCompile.callAsync.err (/Users/******/Sources/vue-cli-demo/node_modules/webpack/lib/Compiler.js:487:29)
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/Users/******/Sources/vue-cli-demo/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:6:1)
    at AsyncSeriesHook.lazyCompileHook [as _callAsync] (/Users/******/Sources/vue-cli-demo/node_modules/tapable/lib/Hook.js:35:21)
    at Compiler.compile (/Users/******/Sources/vue-cli-demo/node_modules/webpack/lib/Compiler.js:482:28)
    at readRecords.err (/Users/******/Sources/vue-cli-demo/node_modules/webpack/lib/Compiler.js:221:11)
    at Compiler.readRecords (/Users/******/Sources/vue-cli-demo/node_modules/webpack/lib/Compiler.js:349:11)
    at hooks.run.callAsync.err (/Users/******/Sources/vue-cli-demo/node_modules/webpack/lib/Compiler.js:218:10)
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/Users/******/Sources/vue-cli-demo/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:6:1)
    at AsyncSeriesHook.lazyCompileHook [as _callAsync] (/Users/******/Sources/vue-cli-demo/node_modules/tapable/lib/Hook.js:35:21)
    at hooks.beforeRun.callAsync.err (/Users/******/Sources/vue-cli-demo/node_modules/webpack/lib/Compiler.js:215:19)
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/Users/******/Sources/vue-cli-demo/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:15:1)
    at AsyncSeriesHook.lazyCompileHook [as _callAsync] (/Users/******/Sources/vue-cli-demo/node_modules/tapable/lib/Hook.js:35:21)

In case you didn't notice... it was fixed by the commit https://github.com/vuejs/vue-cli/commit/85e6e5ea6c34ac56ab54127d72b86d25271a31fa - which is why your previous issue was closed.


loader: explicitly manage policy programs in the Cilium agent

Fixes: #29331
This PR puts the agent in charge of picking out specific programs from a Collection and inserting them into a policy map at a defined, predictable point during the loading process. All but one remaining (cilium_calls_) bpf_elf_map has been converted to BTF-style map definitions. The last step in sunsetting the iproute2 legacy is porting over cilium_calls, which is now well within reach.

/test

/test

/test

/test

/test

/test

/test

/test

/test


fix the exception occurred when publish/rollback namespaces with grayrelease

What's the purpose of this PR
fix the exception occurred when publish/rollback namespaces with grayrelease
Which issue(s) this PR fixes:
Fixes #4563
Brief changelog

revert the OPERATION_CONTEXT_TYPE_REFERENCE to new TypeToken<Map<String, Object>>() { }.getType()

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Read the Contributing Guide before making this pull request.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit tests to verify the code.
[x] Run mvn clean test to make sure this pull request doesn't break anything.
[x] Update the CHANGES log.

Codecov Report

Merging #4564 (f4c7d35) into master (e690e65) will increase coverage by 0.02%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master    #4564      +/-   ##
============================================
+ Coverage     53.30%   53.32%   +0.02%
- Complexity     2710     2712       +2
============================================
  Files           495      495
  Lines         15438    15437       -1
  Branches       1599     1599
============================================
+ Hits           8229     8232       +3
+ Misses         6648     6646       -2
+ Partials        561      559       -2

Impacted Files
Coverage Δ

...p/framework/apollo/biz/service/ReleaseService.java
71.87% <33.33%> (+0.27%)
:arrow_up:

...ework/apollo/internals/RemoteConfigRepository.java
88.34% <0.00%> (+0.61%)
:arrow_up:

.../apollo/internals/RemoteConfigLongPollService.java
78.44% <0.00%> (+1.19%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


not running Get ".txt": unsupported protocol scheme ""

any solution?

......

Hi Sir,
Good day
Facing similar issue can you please help
[*] 🦊 Start scan [SID:0][0/1][0.00%%] / URL: .txt
[E] not running Get ".txt": unsupported protocol scheme ""
using 64 bit Kali verified all dependencies still issue persist. Thanks in advance have a wonderful day :)

Hey @pavanreddi @hassankhan98 ,
Sorry for the issues. I am working on a fix , Will release it soon.

Hey @pavanreddi @hassankhan98
I have fixed the issue, Can you reclone & Check once again?

Thanks for your time. runs perfectly without any issues :)

Thanks Bro
working now .....

Hey,
I'm working on a fix.


Minimum mesos version?

What's the minimum mesos version that Metronome can run on? I tried looking in the docs available thus far, but there was no information about this.

Metronom is based on Marathon so I bet you should follow recommended version for Marathon. The current Marathon version could be found here. And then you should take a look at Marathon Release page where recommended version should be defined.
Java wrapper require native lib to be available and this should be at least 1.0.0 but Mesos itself could be in other version.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-1636. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


[Feature] Control xenorchestra_vm power state

It would be great to have the option to create a vm in a powered off state. The use case would be for changes to any unexposed options prior to starting the VM for first boot.
Likewise an option to set an existing imported vms powerstate to halted or running, would be beneficial for inter-dependencies between services.

@ravager-dk thanks for opening the PR for boot after create and suggesting this idea for the VM resource.
I actually have a work in progress change that implements the creation part of this (commit) and includes a test that exercises creation and updates to the power_state argument.
I see adding support for power_state as a more complete solution than boot after create. The former would work for creation and updates, so I see the boot after create option as a temporary solution. In addition to that since it requires adding an argument to the VM resource, that means that a later breaking change would need to be introduced.
I'd prefer to go the full way and provide complete control over the power_state setting. I really appreciate you taking the time for opening #276, but I think it would be best if we pursue that "full fledged support" I described above.

@ddelnano My initial thought was that this would help the use case for xenstoredata #261.
The flow for this would be

create VM powered off
set xenstoredata
power on VM

Exposing the bootAfterCreate param was such an easy one, since it obviously needs handling in any case. Reading your coments I totally get what you are saying. It would be better if a power state argument handled both bootAfterCreate and power on/off.
I have yet to look at your branch, but I would suggest setting bootAfterCreate to false in vmcreate and then after creation if power should be on, call vm.start. This would make it easier to implement xenstoredata or any other setting that might need to be run after creation but before startup.
I am already doing some initial work on xenstoredata #261. I have it working for resource creation, so I just need to get updates supported. I should probably hold off on anything more until power state is implemented.
Writing and running tests in Terraform provider development is very much unknown to me. I will need to look more into how to write and run them and what sort of coverage Terraform Registry requires.

#278 could be merged this week. I wasn't sure if there would be complexity in the VM update path, but it was a small change to the existing code.

This would make it easier to implement xenstoredata or any other setting that might need to be run after creation but before startup.

Sorry for missing your earlier comments, but it does seem like we will need to perform creation across vm.create and vm.set calls.

I will need to look more into how to write and run them and what sort of coverage Terraform Registry requires.

Terraform registry doesn't require any amount of test coverage. Unless it would require a significant investment to write tests for a given feature, I just believe it's a necessary for this project. The VM resource is complex and verifying if a given change breaks any of its functionality broke done very early in the project.

@ravager-dk https://github.com/terra-farm/terraform-provider-xenorchestra/pull/281 implements separating the VM creation into two steps (create followed by start). It's not possible to do that for all VM create requests because destroyCloudConfigVdiAfterBoot relies on boot after create.
I need to revamp #278 to include that change. I want to see that through before merging #281, but hoping that should happen soon.

@ravager-dk this is complete with #281 and #278 merged. This will be available in the next terraform provider release (v0.26.0), which should be released today.

v0.26.0 is available now.


Setting up PIXL in my local host device and adding few recommendations with a fresh perspective

I am new to PIXL so the first idea that comes to mind is to have an on-boarding template listing steps on how to setup PIXL locally and then remotely.

[x] clone repo, create python env, pip install PIXL and pass tests
[x] add templates for issues (bugs, new features, something is not working), PRs, onboarding, etc
[x] Priorities for issues (might be an overkill, as it is already in Project dashboard labels )
[x] AOB

Hi @stefpiatek @p-j-smith
I managed to install dev deps using pip install -e ../pixl_core -e ../pytest-pixl -e .[test] -e .[dev] but I get some failures with test_docker_commands.py

Just updating here but it seems like its the --dry-run flag not being available for Miguel's docker, he's looking at updating this and documenting the updates

Docker version looks the same and dry-run is in the docs so should be available in linux.
Edit: Oh I think its the docker compose version that you'll need to update
Docker Compose version v2.28.1-desktop.1

some notes to be added in relevant issues

Do we want to remove versions of deps in https://github.com/UCLH-Foundry/PIXL/blob/main/cli/pyproject.toml (core does not have a version)?
Duplicated requirements for test and dev in https://github.com/UCLH-Foundry/PIXL/blob/main/cli/pyproject.toml and https://github.com/UCLH-Foundry/PIXL/blob/main/pixl_core/pyproject.toml and other pixl_* packages
Duplication of setups instructions:
https://github.com/UCLH-Foundry/PIXL/tree/main?tab=readme-ov-file#development
https://github.com/UCLH-Foundry/PIXL/tree/main/cli#installation
https://github.com/UCLH-Foundry/PIXL/blob/main/docs/setup/developer.md

Thanks!. I'm pretty fine with duplication of documentation, so you can stumble across information as an when you need it. Will make issues for the other ones


Migration guide

New guide for helping users to Export and Import their account data.

Nice job @Gaboris your first PR!


Don't check preconnect links

Preconnect links are used to establish a server connection without loading a specific resource yet.
Not always do these links point to a URL that should return a 200, and they are not user-facing, i.e. they don't show up in the final rendered version of a page.
Therefore, I think we should exclude them at all; not even in --include-verbatim
mode, as they might not point to a valid resource.
Fixes #897

I'm thinking I need to improve the attribute handling in general to make the tests pass.
I would have to cache the attributes and only extract links once I'm fully done with reading all attributes for a single tag.
Quite frankly, driving the HTML parsing state-machine is a bit finicky with html5gum. Is there a way to change the parsing such that I can get a fully parsed set of attributes for a tag? It would probably come with a performance penalty, but it would reduce the complexity of the implementation.
(That's the one reason why I haven't removed html5ever support yet. I like how simple the parsing is, although it's slower of course.)

instead of writing your own emitter you can always just read through the DefaultEmitter (like how tags are being processed in the README), though it is a lot slower and I think also not competitive with html5ever

@untitaker, the way I would model it (without switching to the DefaultEmitter) would be to create a HashMap with key being the attribute and value being a list/set of links, e.g. { key: "href", "value": [https://example.com] }. When I reach the end of a tag, I check the attributes and emit the list of links if there are no excluded attributes (like rel=prefetch).
Does that make sense? Is there an easier way in html5gum (without DefaultEmitter)? It feels like the perf could suffer from the additional bookkeeping.

yeah that totally makes sense. currently there's no simpler way in html5gum. I would like to have more kinds of emitters that take some of that work off of you, but when I tried to build them in the past they didn't feel general purpose enough.
On Thu, Aug 24, 2023, at 16:44, Matthias Endler wrote:

@untitaker https://github.com/untitaker, the way I would model it (without switching to the DefaultEmitter) would be to create a HashMap with key being the attribute and value being a list/set of links, e.g. { key: "href", "value": [https://example.com] }. When I reach the end of a tag, I check the attributes and emit the list of links if there are no excluded attributes (like rel=prefetch).
Does that make sense? Is there an easier way in html5gum (without DefaultEmitter)? It feels like the perf could suffer from the additional bookkeeping.
—
Reply to this email directly, view it on GitHub https://github.com/lycheeverse/lychee/pull/1187#issuecomment-1691821423, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGMPROI3GASJ5FLPDYNPMLXW5SDJANCNFSM6AAAAAA24YQC7A.
You are receiving this because you were mentioned.Message ID: @.***>

This commit doesn't exclude all types of preconnect links e.g. dns-prefetch is not excluded even though it should be.

After more than a year, I finally got around to cleaning up this PR and fixing the remaining attribute ordering issues. Along the way, I refactored (and hopefully improved) the html5gum parser.
The new verbatim_stack also handles nested verbatim elements now.
Other changes include refactoring the HTML link extractor for improved performance and maintainability, extending the documentation. I also removed unsafe code where possible, using String::from_utf8_lossy. (We can switch back if it becomes a performance bottleneck, but I doubt it will be the case.)
Thanks, @untitaker, for the review.


Scale data in Visium tutorial? #question

Couldn't find anything in exisitng/closed issue so I asked here.
Thank you for implementing those concise yet comprehensive tutorials. In Visium data processing tutorial, no scaling was done before PCA is conducted. Is there any specific reason for skipping this step?
Thanks in advance for the answer!

The best place for questions is probably https://discourse.scverse.org/
That being said, I think scaling isn’t done commonly anymore because it densifies sparse data (which is heavy on the memory), and many analysis tools these days don’t need scaled data to work.