update mfem url link in build_visit

to: http://goo.gl/Is2XJq
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 2334
Status: Resolved
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: update mfem url link in build_visit
Assigned to: Cyrus Harrison
Category: -
Target version: 2.10
Author: Cyrus Harrison
Start: 07/08/2015
Due date:
% Done: 0%
Estimated time:
Created: 07/08/2015 12:48 pm
Updated: 09/22/2015 10:48 pm
Likelihood: 3 - Occasional
Severity: 2 - Minor Irritation
Found in version: 2.8.2
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
to: http://goo.gl/Is2XJq
Comments:
Hi Everyone,Updated the mfem url to reflect the new mfem website and move to github.Note: I used http://mfem.github.io/releases, instead of their google shortened url (http://goo.gl/Is2XJq), since build_visit doesnt support downloads that dont include the filename.Sending svn_bin/bv_support/bv_mfem.shTransmitting file data .Committed revision r27232.-Cyrus

2339_smime.p7s.zip


Add option to generate empty migration files even when changes are detected

Problem
tldr; i'd like prisma migrate to generate an empty migration where i can add the sql statements i wish to have in that migration.
longer: i'd like to use prisma to handle migrations, however, my schema and database are not perfectly mapped.  this leads to prisma generated migrations containing a lot of dangerous sql that i don't want to use (mainly dropping things).  what i do instead is rip out most of the sql and then either fill in my own migrations or remove the ones prisma generated which i don't want to keep.
Suggested solution
i'd like a simple command prisma migrate new which just produces a blank migration with today's timestamp.  it should not use prisma's feature to compare the schema against the database (and generate migration sql which is not valid in my case).
Alternatives

i can manually create a migration directory with the timestamp and create a migration.sql file.
run prisma migrate dev and then replace/edit the  migration.sql file

Additional context
this is by no means a big deal or show stopper, but this seems like a pretty easy feature to add in the 'nice to have' category.  until prisma schema and database are perfectly in sync, hand written migrations will likely be necessary in many contexts.  saving me some boilerplate copy paste would be nice.

Hi @carchrae
It's actually already possible with prisma migrate dev --create-only if no change is detected an empty migration file will still be created and you will be able to edit it, put any SQL you like then run prisma migrate dev to execute it locally to verify it is valid.
After applying once you can still edit it and rerun prisma migrate dev as many times as needed, if Migrate detects that the computed checksum of the file changed it will try to rerun it.

thanks for the quick reply @Jolg42
the problem i have is that prisma migrate dev --create-only and prisma migrate dev will always think there are changes.  for example, i have had to comment out relations due to issues like this https://github.com/prisma/prisma/issues/5289
i know in the perfect world, my schema would be mapped correctly, but when taking an existing database with all its lovely warts, i would still like to harness most of the migrate functionality (and the rest of prisma) but disable prisma generating the actual migration sql for me, since the schema isn't mapped perfectly.
i know i can still use it and delete the generated sql, hence this isn't a massive problem for me.
i'll update the issue above to make that more clear.

just to clarify, re the title change - i don't even want prisma to check for changes and start guessing about what needs to be changed!  i just want to generate a blank migration and ideally without wiping my dev database.
much like how rails does it (in the very basic case with an empty template): https://guides.rubyonrails.org/active_record_migrations.html#creating-a-migration

any progress on this?  There are many reasons to want to apply a manually written migration file including on database features that prisma does not support.

This would be great


Deal with duplicated oem*.inf files somehow

Nope.
Reading all OEM drivers... done
Reading oem*.inf files...
Traceback (most recent call last):
  File "<string>", line 287, in <module>
  File "<string>", line 237, in main
Exception: oem39.inf is duplicate of oem37.inf

oem39.inf.txt


Discussion: We don't need INotifyDataErrorInfo for validation

Discussion: What's the practical point of supporting that interface?
Perhaps, in the times of first WPF releases the idea of highlighting textbox with red when validation fails was innovative and fresh, but today it is not enough. Validation should allow convenient customization of error messages with bindings, specialized controls and templates, but INotifyDataErrorInfo offers just event and a method to get information about errors. It would be much more convenient if it would provide bindable collection instead and would not be coupled with ViewModel.
In fact, most of the problems with validation can be solved quite simply if you would try to change the approach and instead of repeating old mistakes, start making new ones. For example, this is what I came up with after playing a couple of hours in the Visual Studio:
    <Grid>
        <Grid HorizontalAlignment="Center" VerticalAlignment="Center">
            <!-- There should be new templatable control for validation warnings instead of Popup -->
            <Popup IsOpen="{x:Bind viewModel.FirstNameValidator.HasIssues, Mode=OneWay}" Closed="{x:Bind viewModel.FirstNameValidator.Reset}" IsLightDismissEnabled="True" VerticalOffset="-80">
                <Border BorderThickness="1" BorderBrush="{StaticResource SystemControlDisabledAccentBrush}" Background="{StaticResource ApplicationPageBackgroundThemeBrush}" CornerRadius="4" Padding="16,20,16,20">
                    <winui:ItemsRepeater ItemsSource="{x:Bind viewModel.FirstNameValidator.Issues, Mode=OneWay}" />
                </Border>
            </Popup>
            <TextBox Width="200" Text="{x:Bind viewModel.FirstName, Mode=TwoWay}" PlaceholderText="First Name" />
        </Grid>
    </Grid>

    public class MainViewModel : INotifyPropertyChanged
    {
        public RulesValidator<string> FirstNameValidator { get; } = new RulesValidator<string>(
            FrameworkValidationRule.StringIsNotEmpty(),
            FrameworkValidationRule.StringMinLength(2));

        private string _firstName;

        public string FirstName
        {
            get
            {
                return _firstName;
            }
            set
            {
                if (FirstNameValidator.Validate(value))
                {
                    _firstName = value;

                    OnPropertyChanged();
                }
            }
        }
    }

What in this code is better than in the ObservableValidator samples:

ViewModel does not have to implement additional interface or inherit from base class.
It is clear where the validation data is and which validation object is responsible for which area.
You can make whatever you want with that data, it can be used with new specialized control or with custom layouts - usage is as simple as binding.
Bindings don't have to support anything new, both {x:Bind } and {Binding } can bind to Validator.
It is not required to update TextBox and other controls templates. We can add a property IsValid and new visual state to some controls where it would be appropriate to let developers bind that property to Validator.HasIssues, but once we will ship ValidationPopup, it will not be a blocking task.
There is no reflection.

Below is the rest of the code required to better understand or compile this sample. It's all a draft, but it demonstrates that validation without INotifyDataErrorInfo can be simple and flexible, so I encourage you to consider whether WinUI should really support the old interface or consider other options. I am sure many people here can suggest better options.
    public interface IValidator<T> : INotifyPropertyChanged // Common interface to use binding in the new ValidationPopup control
    {
        bool HasIssues { get; }

        IEnumerable Issues { get; } // Error message strings or complex objects for binding within custom templates in the new control

        bool Validate(T value);

        void Reset();
    }

    public interface IValidationRule<T> // Validation rule predefined in MVVM framework or created by developer
    {
        object Validate(T value);
    }

    public class RulesValidator<T> : IValidator<T> // Validator for validation rules
    {
        private IValidationRule<T>[] _rules;

        public RulesValidator(params IValidationRule<T>[] rules)
        {
            _rules = rules;
        }

        public bool HasIssues { get; private set; }

        public IEnumerable Issues { get; private set; }

        public event PropertyChangedEventHandler PropertyChanged;

        public bool Validate(T value)
        {
            var result = true;

            var issues = new List<object>(_rules.Length);

            foreach (var rule in _rules)
            {
                var validationResult = rule.Validate(value);

                if (validationResult != null)
                {
                    result = false;

                    issues.Add(validationResult);
                }
            }

            HasIssues = !result;

            Issues = issues;

            FirePropertiesChanged();

            return result;
        }

        public void Reset()
        {
            HasIssues = false;

            Issues = null;

            FirePropertiesChanged();
        }

        private void FirePropertiesChanged()
        {
            PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(nameof(HasIssues)));

            PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(nameof(Issues)));
        }
    }

    public static class FrameworkValidationRule // Static class with predefined validation rules
    {
        private static StringIsNotEmptyValidationRule _stringIsNotEmpty = new StringIsNotEmptyValidationRule();

        public static IValidationRule<string> StringIsNotEmpty()
        {
            return _stringIsNotEmpty;
        }
    }

    public class StringIsNotEmptyValidationRule : IValidationRule<string> // Sample validation rule
    {
        public object Validate(string value)
        {
            return string.IsNullOrWhiteSpace(value) ? "This field should not be empty!" : null;
        }
    }

    public class DataAnnotationsValidator<T> : IValidator<T> // Validator for DataAnnotation attributes
    {
        ...
    }

    public class MyValidator : IValidator<T> // Developer can make custom validators for complex cases
    {
        ...
    }

Related Links
#179 Add Input Validation Support to WinUI (UWP and Win32 Desktop) with INotifyDataErrorInfo
ObservableValidator from Community Toolkit
INotifyDataErrorInfo

@ranjeshj and @MikeHillberg

Duplicate/Related to #5859 and #4671, alternative to #179.


feat: Update openapi-generator to v7

Big bump for openapi-generator from v4.3 to v7. Tried to keep the generated code as close as possible, by modifying the templates which have previously been dumped to openapi-generator/templates folder.
For comparing results, I was checking out another copy of this project, generating the libraries, and then running a recursive folder comparison tool.

@jablan great we could keep the existing interfaces with the latest generator 🔥 do you think it would make sense to add those tests first before we merge this to check that they pass before and after?

@theSoenke that'd be great, if can be done in this sprint.


Detecting events outside of the kivymd application screen!

Code_link: https://github.com/ramayanbindas/ClassConnect
You could try to run the ClassConnect.py file.
What I am doing?
-> While I am trying to develop the application, at first I have developing it in a windows machine, this code runs smoothly and fine. But after switching the OS to the federo version 36. And try to set up all my code. I found the issue of it detecting touches/inputs of mouse clicks outside of the boundary of the kivymd application screen, even if the screen is not get focused.
Besides all the code suddenly begun to act wired it doesn't act like in the window system. As my knowledge I have doesn't make
error my code bases. Because I have run and test this codes in window 10 OS multiple times.
What I have tried?

I had tried to test is there may be problem in my code. So I develop a test prototype like my code base just added multiple inputs fields inside the Boxlayout with in the same script file. Loaded by the :method Builder.load_string:. But it acts the same.

I have see or search the bug it was reported previously by some one in the Ubuntu machine link (https://github.com/kivy/kivy/issues/7960). But he had figure out the issue somehow but I can't understand well.
If some one give a time to solve my issue than I well be very much appreciate it.

What I am expecting?
-> I am expecting that while opening the application it can detect the button, events, and everything as other application does.
If any more information needed than fell free to give me a message.

Attach the minimum code that reproduces the bug otherwise your issue will be closed.


WhiteSource Bolt Project on Azure DevOps Demo

AB#3890

Error while creating build definition: The pipeline is not valid. A task is missing. The pipeline references a task called '521d1e00-f5fb-4b73-a93b-b2fe88a9af91'. This usually indicates the task isn't installed, and you may be able to install it from the Marketplace: https://marketplace.visualstudio.com. (Task version , job 'Phase_1', step 'WhiteSourceBolt3'.)

Kindly install the whitesource-bolt extension from Azure DevOps Marketplace and try to provision the project,


FIX+ADD+ENH: fix SPM permissions issue + add default entrypoint + support minimized freesurfer + other updates

fixes #61 #62

free permissions to /opt for non-root users.
add the directory /neurodocker with a default entrypoint bash script startup.sh.

add the appropriate source ... command to /neurodocker/startup.sh for freesurfer and fsl.

add support for minimized freesurfer.
add mention of fsl's non-free license.
make non-free sources the default for neurodebian.

Codecov Report

Merging #63 into master will decrease coverage by 5.78%.
The diff coverage is 65.38%.

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   82.85%   77.07%   -5.79%
==========================================
  Files          32       32
  Lines        1289     1304      +15
  Branches      112      113       +1
==========================================
- Hits         1068     1005      -63
- Misses        187      254      +67
- Partials       34       45      +11

Impacted Files
Coverage Δ

neurodocker/interfaces/tests/test_mrtrix.py
72.22% <ø> (-27.78%)
:arrow_down:

neurodocker/interfaces/tests/test_freesurfer.py
76.19% <ø> (ø)
:arrow_up:

neurodocker/interfaces/tests/test_afni.py
64.28% <ø> (ø)
:arrow_up:

neurodocker/interfaces/tests/test_miniconda.py
46.15% <ø> (-53.85%)
:arrow_down:

neurodocker/interfaces/tests/test_ants.py
76.19% <ø> (-23.81%)
:arrow_down:

neurodocker/interfaces/tests/test_neurodebian.py
50% <ø> (-50%)
:arrow_down:

neurodocker/interfaces/tests/test_spm.py
35.71% <ø> (-64.29%)
:arrow_down:

neurodocker/interfaces/afni.py
50% <0%> (-5.27%)
:arrow_down:

neurodocker/interfaces/neurodebian.py
68.29% <100%> (-14.64%)
:arrow_down:

neurodocker/dockerfile.py
76.66% <100%> (-2.33%)
:arrow_down:

... and 18 more

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 236d4c0...0e16116. Read the comment docs.

adds --workdir option


Update the virtual clock in isolation mode to step forward with around the same speed as the host system.

Before this, the 1s sleep test took around 4 minutes on my machine.

r=me with "some sample programs" made a bit less vague.

@bors r=RalfJung

:pushpin: Commit 16eb6980e3ad3001fde2f4601522953f34b67d3a has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 16eb6980e3ad3001fde2f4601522953f34b67d3a with merge d20acaf002261844aa2cf6fed96abe19d26b7a57...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing d20acaf002261844aa2cf6fed96abe19d26b7a57 to master...


Provide a way of customizing DOCKER_PRUNE_UNTIL, DISK_MIN_AVAILABLE, and DISK_MIN_INODES for the cron job

Right now, there is no way of customizing those values for the hourly docker-gc cron job. The defaults are not good enough for us since our docker images are quite big and the disk gets fill up very quick. We also run a RAM drive for the runners, which requires different clean up params from the builders.
I imagine and approach could be to add those variables to the cloud formation template parameters, and generate the cron job on provision time, rather than image building time. Another option is to generate a file on provision time that the cron job would source when it runs. Thoughts?

Yeah, I think this would be worth adding a stack parameter for.

This would definitely be nice to have. Would happily accept a PR.

We have builds start failing when there's a significant amount of disk (~2GB), because our images are large; a large chunk of these images can be automatically cleaned with the docker system prune (even with the until=1h filter), except the DISK_MIN_AVAILABLE  threshold is too low, so the script doesn't end up running the prune.
With the current 4.3 stacks, the limit seems to be 1GB, but on master it seems to be 5GB (https://github.com/buildkite/elastic-ci-stack-for-aws/commit/e97921cc74e8a69fccbe98ebcc6ee0f50a9ccbcf). That would be good-enough as a work-around for us. Is there a reason it isn't in the 4.3 series?


Image Loader works for production but not tests

Note: for support questions, please use one of these channels: Chat: AngularClass.slack or Twitter: @AngularClass

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

What is the current behavior?

Components with images work great in dev/production mode, but when it comes to testing, they can't be bundled, e.g.

Error: Module parse failed: C:\myProj\client\assets\img\background.jpg Unexpected character '�' (1:0)
You may need an appropriate loader to handle this file type.
(Source code omitted for this binary file)
at config/spec-bundle.js:72122

What is the expected behavior?

For the test webpack config to more closely reflect the dev configuration

What is the motivation / use case for changing the behavior?

A more unified test/develop experience

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

@georgeedwards - The test config isn't merged with the common config like dev or prod is.
You will need to copy the relevant loader configs into webpack.test.js
See this portion of webpack.common.js
https://github.com/AngularClass/angular2-webpack-starter/blob/master/config/webpack.common.js#L140-L161

@d3viant0ne Awesome - thanks, that has fixed it. Although not working for .svg? I tried changing to:
        /* File loader for supporting images, for example, in CSS files.
         */
        {
          test: /\.(jpg|png|gif|svg)$/,
          loader: 'file'
        },

but that didn't work. How shoudl I fix that?

@georgeedwards - Short version SVG != Image
For SVG's and also fonts, you would want to do something like this.
        {
          test: /\.(ttf|eot|svg)(\?v=.+)?$/, loader: 'file-loader'
        },

Resolution exists in issue, resolving.


Add an assert_not_impl macro

This is the polar opposite of the assert_impl macro and abuses name
resolution with trait.
A trait with type parameter A provides a simple method. The trait is
implemented on the checked type T potentially twice: with type
parameter unit A=() for all types T, and with another parameter type
-abitrarily a new Invalid- when the T implements the trait bounds.
We then try to select T as an instance of the trait, using turbo-fish
to try and access one of its items, without providing the trait type
parameter. When only the first instance is implemented the type checker
can infer A=() but when the second is implemented as well it can not.
No other ambiguity can occur, only the trait's type parameter is
ambiguous. Thus, type inference and compilation succeeds if the trait
bound is not satisified by the input type.

I needed to modify the design slightly, as the method call was possibly not completely unambiguous. Now the ambiguity has moved to the type parameter of a single locally defined trait which allows us to restrict the set of possibly applicable methods to that trait alone using turbo-fish syntax. By leaving out the trait's type parameter it can only be inferred when one impl is applicable.

Out of curiosity, why do Check<T> and the type parameter on the trait exist at all?
To check that a type S doesn't implement a trait T, one can do:
trait ShouldNotImplT {}
impl ShouldNotImplT for U where U: T {}
impl ShouldNotImplT for S {}

@nox No particular reason, it turns out, only that a previous version of the implementation had a self parameter instead of the turbofish for trait resolution. Thanks for the simplification suggestion.

Awesome, thank you very much!

This closes https://github.com/nvzqz/static-assertions-rs/issues/8.

Coming back to this a few years later, the self parameter can be handy if you have a value of a type you can't name, e.g. as happening with futures.
async fn some_lib_feature() {
    // Do some async work. To future proof this, and not cause unintentional semver breaks, we want to assert this
}
// We can use a very similar trait to the one in the PR
trait AmbiguousIfImpl<A> { fn some_item(&self) {} }
impl<T: ?Sized> AmbiguousIfImpl<()> for T {}
struct Invalid;
impl<T: ?Sized + Send> AmbiguousIfImpl<Invalid> for T {}
// And use it to test the future is not Send
let fut = some_lib_feature(); // We can construct a value of the type, but can't name it
let () = <_ as AmbiguousIfImpl<_>>::some_item(&fut);
//        ^ type is deduced to the type of the future, although we can not name it. *

* This comment might become a bit outdated with the stabilization of type_alias_impl_trait in rust-lang/rust#63063.


Unify doc/comment line width.

Maintaining the line width in the code is sometimes hard, and we have a CI check for it so it never goes above 120.
Yet, maintaining it for the documents in the code is trivial, and we don't have any rule for it.
As far as I know our preferred line width is 100 for document/comments. Yet, I often see lines aligned to 60 or 80.
Both aligning this comments, and enforcing it potentially in the CI should be easy to do, and a good small step toward making the have a higher standard.

Here's a good (random -- all due respect to whoever the author of the code is :D) example
https://github.com/paritytech/substrate/blob/2743ed4b26b786efdcb5d1123a277f21da0c29c0/frame/staking/src/lib.rs#L867-L870
Comparing this with the adjacent lines is not easy on the eyes IMO.

@TriplEight could this ever be a CI job that rewraps all comment lines to exact 100?

@kianenigma you mean some script/bot that would fix and commit this automatically? Pretty sure there are tools for that, but we will have to research what's their i.e. licensing of the produced code.


@conda-forge-admin, please add bot automerge

Additional comment:
@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/de-bundesrat-feedstock#2.


Certain unicode characters are smaller than they should be.

Screenshot:

Out of curiosity, have you tried installing the font? Oddly I've never had issues where someone needed to install a font to resolve an issue, but it's just another minor thing you could try if you haven't. The font's located in budhud/resource/fonts/font-lato-semibold.ttf (it may also be called Lato-Semibold.ttf)
I'm asking around to see if anyone knows of any solutions

closing this for now, feel free to re-open if more information surfaces


Add recent The Register article

Adding at least one recent notable press article mentioning OWA

Oh, just noticed that it is in the https://github.com/OpenWebAdvocacy/website/pull/108 along with many more articles...


Create 2024-10-09-september-release-notes

Hey @kevglass! Since this is the first one I'd love a quick glance over before I do anything wrong 😅

The file name for the blog needs to have an ".md" extension otherwise the blog won't pick up.
Otherwise 💯

@kevglass Did this do it correctly? i don't see the change in the top

But I do see where I made the change


[Do not merge] Test vendor guava 32.1.2

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

https://github.com/google/guava/commit/b1350a6a99b9164a6cdc15f177cbda50fa1baf3d makes the value returned from ImmutableMap nullable, breaking checkerframeworks in various places

Run Java_IOs_Direct PreCommit

Run Java_PVR_Flink_Batch PreCommit

artifact released, closing this PR


Fixed destructiveness of Array.prototype.randomize()

Hi there @ondras ! Fixed the problem where using .randomize() on an array destroyed the array's value. I'm not sure if I need to compile rot.js as well? Let me know if you need me to tweak anything or if you think this will cause any problems...
By the way, thank you so much for this tool! I have stumbled across roguelikes within the last few years, and as a web developer I was thrilled to find such a useful framework that uses my 'bread and butter' language: Javascript!

Hi @jakofranko,
thanks for your code and sorry for my late reply (just returned from vacation).
As far as the cloning goes, you can prehaps simplify the whole operation to
var clone = this.slice();

?

Good idea, I didn't realize you could use slice to make shallow copies like that. Handy! I'll update and re-push.

Cool, thanks!

Thanks for the framework! Glad I could contribute.


Remove polling for Enketo IDs

The most complicated part of the FormShow component is polling Backend for Enketo IDs for a form or form draft that's missing them. With getodk/central#385 in place, it should be the case that Backend returns Enketo IDs immediately in the vast majority of cases, making that polling much less useful. If that proves to be true in experience, I think we should strongly consider removing that polling from Frontend as a source of complexity that's outlived its usefulness. That polling is also noisy during local development if you don't have Enketo set up.
EnketoPreview and EnketoFill already recommend refreshing, so I don't think any text would need to change:
https://github.com/getodk/central-frontend/blob/972062f259293877ca2f718aefc83ad80979f2ca/src/components/enketo/preview.vue#L57
https://github.com/getodk/central-frontend/blob/972062f259293877ca2f718aefc83ad80979f2ca/src/components/enketo/fill.vue#L55

If that proves to be true in experience

@lognaturel, how have things been going in terms of Backend being able to fetch Enketo IDs synchronously instead of async? If it's now rare that Enketo IDs are fetched async, I'd love to remove this polling from Frontend.

Can you please remind me what action I was supposed to take? 🫣 It's my believe that Enketo's response has always been immediate and synchronous. Maybe I should verify in the Enketo source?

Mostly I'm curious about what you've seen in practice. When you create a form or create a form draft or replace an existing form draft or publish a form draft, are the buttons in Frontend that link to Enketo enabled right away? I guess if Enketo's response is immediate, that should mean that the buttons in Frontend are enabled right away. I don't think you necessarily need to verify that in the source; mostly I'm interested in your observations in practice. (Assuming that enough time has passed since v2023.4 to make that determination.) I'm eager to remove the polling described above from Frontend, but first I want to confirm that it is useful only rarely if ever.

I use all of the all the time and have never seen it disabled since the 2023.4 changes!


[BUG] Invalid line number

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior

Every time I import classes or do something wrong in syntax writing, a crash will happen, and this error output comes up. But, this will only happen when I pressed O instead of Enter in the end of the line.
Expected Behavior
It should have behaved normally and wouldn't crash.
Steps To Reproduce

Open up a project
Import classes, for this example write "import 'blabla.dart'"
Enter the class and then create a new line with O instead of Enter
???

Environment
- OS: Arch Linux
- Flutter version: 3.16.9
- Is flutter in $PATH: yes
- neovim version: v0.10.0

Anything else?
I know I need to search deeply about how to debug this on neovim -> lsp level deep, but unfortunately I don't have much time left lately cause of my uni and job, so I reported this here instead. Please give me any links or references if this issue has any blame on my configuration fault or something :)

This is still an issue for me too whenever I paste text into any file at all. LSP crashes and have to restart neovim each time.


Recognition on x32

Environment

Tesseract Version: 4.0.0-beta4
Platform: Ubuntu 16.04 x32

Current Behavior:
Bad text recognition on x32-bit server ;C
Expected Behavior:
Good recognition.
Suggested Fix:
Additional building flags ?

Support for x32 might indeed be interesting, because it should be significantly faster than 32 bit code and could even be slightly faster than 64 bit code because of better cache usage. I expect that it will require some work on the Tesseract code base to fix all portability issues. See https://wiki.debian.org/X32Port for a few first hints.

@qb1t0, is the recognition on other installations better? Could you provide an example for the bad text recognition (image, command line, results)?

With the test image from https://github.com/tesseract-ocr/tessdata/issues/108, I get the same results for x32 and for x86_64 when I add --psm 6.
There is a difference when running tesseract test.tif -: x32 shows the well know assertion contains_unichar_id(unichar_id):Error:Assert failed:in file src/ccutil/unicharset.h, line 509

@zdenop Please label as Feature Request.

I think it can be closed, as x32 and x86_64 deliver the same results.


Issue with deployment - No route matches URL "/"

Fresh first deployment, startup logs dont show anything of concern, but when I browse to http://:3000 I receive a 404 error and the following logs are seen:
dock02_headplane  | Error: No route matches URL "/"
dock02_headplane  |     at getInternalRouterError (/app/node_modules/.pnpm/@remix-run+router@1.17.1/node_modules/@remix-run/router/router.ts:5400:5)
dock02_headplane  |     at Object.query (/app/node_modules/.pnpm/@remix-run+router@1.17.1/node_modules/@remix-run/router/router.ts:3509:19)
dock02_headplane  |     at handleDocumentRequest (/app/node_modules/.pnpm/@remix-run+server-runtime@2.10.2_typescript@5.5.3/node_modules/@remix-run/server-runtime/dist/server.js:268:35)
dock02_headplane  |     at requestHandler (/app/node_modules/.pnpm/@remix-run+server-runtime@2.10.2_typescript@5.5.3/node_modules/@remix-run/server-runtime/dist/server.js:160:24)
dock02_headplane  |     at /app/node_modules/.pnpm/@remix-run+express@2.10.2_express@4.19.2_typescript@5.5.3/node_modules/@remix-run/express/dist/server.js:41:28
dock02_headplane  | GET / 404 - - 209.756 ms

Relevant config:
  headplane:
    container_name: dock02_headplane
    image: ghcr.io/tale/headplane:0.3.2
    restart: unless-stopped
    volumes:
      - /opt/stacks/headscale/data:/var/lib/headscale
      - /opt/stacks/headscale/config:/etc/headscale
      - /var/run/docker.sock:/var/run/docker.sock:ro
    networks:
      - npm-backend
    ports:
      - 3000:3000
    environment:
      # This is always required for Headplane to work
      COOKIE_SECRET: 'xxxx'

      HEADSCALE_INTEGRATION: 'docker'
      HEADSCALE_CONTAINER: 'dock02_headscale'
      DISABLE_API_KEY_LOGIN: 'false'
      HOST: '<IP_ADDRESS>'
      PORT: '3000'
      DEBUG: 'true'
      # Only set this to false if you aren't behind a reverse proxy
      COOKIE_SECURE: 'false'
      ROOT_API_KEY: 'xxxx'

It expects it to be on /admin

Do you use a reverse proxy?
im using caddy and it works perfect!
use this for your compose.yml
  caddy:
    image: caddy
    restart: unless-stopped
    ports:
      - "80:80"
      - "443:443"
      - "443:443/udp"
    volumes:
      - ./caddy/config:/config
      - ./caddy/data:/data
      - ./caddy/Caddyfile:/etc/caddy/Caddyfile

and this is your Caddyfile, create this before starting first time the container
https://sub.domain.com {
    reverse_proxy /admin* http://headplane:3000
    reverse_proxy * http://headscale:8080
}


Debbuging project

Hi. I've downloaded project. Downloaded npm and nodeJs on my computer. I'm using VSCode. To check app in Browser, I used apache-tomcat. Also I noticed, that I can start app as desktop application.
I have two problem - debugging and dynamic JS updating . Could you help me please, how you do it more effectively (without continuesly project rebuilding)? Also, I noticed, that to build application - uses Ant without webpack.

Hi guys, why this ticket was closed? I am also wondering how to run JS in DEV mode?
One of the option I found it to concatenate JS in build.xml instead of minifying it. But I would live to hear authors' optinion and how do you guys do JS debugging

It's closed because it's too broad to be an issue and discussions didn't exist at the time. If you want to create a discussion on this, please feel free. But note, if the post is not of the standard of a professional software engineer, I'll just ignore it.


ProxyModel List/Set of Enum/DataObject

Marked as todo in ProxyModel.

I use java.util.Collection in my API. Maybe implementing a Collection Proxy Model would be enough to cover all kind of collection type ?

@vietj Is this issue still open?

that is still open and contributable, I think this issue should be moved to vertx-service-proxy has now it holds the logic for this

Okay Thanks Julien.
On Mon, 10 Sep 2018, 18:35 Julien Viet<EMAIL_ADDRESS>wrote:

that is still open and contributable, I think this issue should be moved
to vertx-service-proxy has now it holds the logic for this
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/vert-x3/vertx-codegen/issues/70#issuecomment-419905070,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APInBA2M8HEjtAdR6NwWEHqcT2IrKxUtks5uZmO2gaJpZM4HXcDi
.

Thank you @vietj


Add Missing Content in CSS

Hey @arpit456jain @ankitapuri
Please assign under SWOC

@LegendSumeet Please assign to me

Hey @arpit456jain Should i make separate PR for separate topics in Css?

No make 1 PR only


First Pass on Docs Redesign

Hello Peli!
This is the first pass of the re-design of Docs. Please let me know if there is something you would like in a different way.

We use the primary color as accent color to be in sync with the editor.
Added some html markup elements to match de design (mainly app header)
Modify the docs.less file
We added the "Lato" fonts. Would you like this to be placed locally in a folder "docs/static/fonts" or is it ok to reference it thru an url
We would like to add new interaction features to the sidebar but it is created dynamically by docsrender.ts. We would like to include icon markup and other classes.
Currently we are working more on the mobile experience and localization switch

Looks good overall.

could you add screenshots of the current changes? For pxt docs, Microbit docs and adafruit docs.
do not enable google fonts. If we need a new font, we should go through the regular OSS process to host on our back end.
avoid calling the header “app something” as it is confusing wrt to our Windows/Web apps . Let’s call it “docs-header”

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

Please add screenshots.

Here are some screenshots. One thing we noticed until going to the microbit target, was the nested accordions. We are currently styling those. Is it ok if we add those changes to this PR? Thanks.
We use @primaryColor as the base accent color for Docs as well.

Yep accordions need a bit of love
Get Outlook for iOShttps://aka.ms/o0ukef

From: Alejandro Toledo Martinez<EMAIL_ADDRESS>Sent: Wednesday, March 6, 2019 9:24:18 AM
To: Microsoft/pxt
Cc: Peli de Halleux; Comment
Subject: Re: [Microsoft/pxt] First Pass on Docs Redesign (#5313)
Here are some screenshots. One thing we noticed until going to the microbit target, was the nested accordions. We are currently styling those. Is it ok if we add those changes to this PR? Thanks.
We use @primarycolorhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fprimarycolor&data=02|01|jhalleux%40microsoft.com|9d17d2f6d0c74c5bab4108d6a25890d7|72f988bf86f141af91ab2d7cd011db47|1|0|636874898601664982&sdata=5pjawMNzkRGKMH5hr7QDCT7jIvWLHN0SN%2BWcBmfRL6M%3D&reserved=0 as the base accent color for Docs as well.
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F704682%2F53900353-00db6000-4002-11e9-97ee-c3953b4a2aec.png&data=02|01|jhalleux%40microsoft.com|9d17d2f6d0c74c5bab4108d6a25890d7|72f988bf86f141af91ab2d7cd011db47|1|0|636874898601664982&sdata=jWMeZzuHRgCCzQzj%2BtmB2oAoXRi8XAKVVvHRQgOiJLQ%3D&reserved=0
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F5313%23issuecomment-470197831&data=02|01|jhalleux%40microsoft.com|9d17d2f6d0c74c5bab4108d6a25890d7|72f988bf86f141af91ab2d7cd011db47|1|0|636874898601664982&sdata=qQtwct8IwOJrx1HSPBPYZnwmmoUd%2FdSj78kIlKbxe8s%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KUX8ytnZQX8cJT2jExheUogh8NHSks5vT_nCgaJpZM4be8sH&data=02|01|jhalleux%40microsoft.com|9d17d2f6d0c74c5bab4108d6a25890d7|72f988bf86f141af91ab2d7cd011db47|1|0|636874898601821229&sdata=16r117hKiHuqobfhV2xe%2BRyhysNHN%2Ff3QPGxcsqbaJQ%3D&reserved=0.

Looks pretty slick @toledoal

New screenshot of Arcade:

We have the following To Do items:

Set media queries width size for sidebar (Small phones)
Place the Hamburger menu to the right and leave "logo" fixed to the left

@toledoal use "- [ ] blabla" syntax in the workitem description and github will track todos for you.

github has this neat "resolve conversation" when you address a comment. it help keep track of things.

You need to test the editor as well. The Arcade banner is now purple too.

pxt docs looking very slick

We still have some heavy dependency on jquery for docs rendering. Try to avoid it but don’t try to remove it completely.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Alejandro Toledo Martinez<EMAIL_ADDRESS>Sent: Thursday, March 28, 2019 4:54:25 PM
To: Microsoft/pxt
Cc: Peli de Halleux; Comment
Subject: Re: [Microsoft/pxt] First Pass on Docs Redesign (#5313)
@toledoal commented on this pull request.

In docfiles/docs.jshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F5313%23discussion_r270241070&data=02|01|jhalleux%40microsoft.com|a8d6f4d9b6414f085f9e08d6b3d8b585|72f988bf86f141af91ab2d7cd011db47|1|0|636894140675209934&sdata=2hphwGnJIbON1Ke9kkCIoTPg9OMs4ObiKS8SbAWIXhQ%3D&reserved=0:

@@ -257,8 +258,19 @@ function renderSnippets() {

 });

}
+function responsiveResize(){

// Add Responsive Adjustments depending on screen size

$( window ).resize(function() {

I was checking that Semantic UI depends on jquery 😟
I found this https://github.com/morganbarrett/Vanilla-Semantic-UIhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmorganbarrett%2FVanilla-Semantic-UI&data=02|01|jhalleux%40microsoft.com|a8d6f4d9b6414f085f9e08d6b3d8b585|72f988bf86f141af91ab2d7cd011db47|1|0|636894140675219942&sdata=j3nN7ZL7Z3S5%2BdoIm3M89uKcYean1oefocWzcLRWcpM%3D&reserved=0 that tries to remove that dependency.
I think in this case (docs) we would need to talk more about this. What do you think?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F5313%23discussion_r270241070&data=02|01|jhalleux%40microsoft.com|a8d6f4d9b6414f085f9e08d6b3d8b585|72f988bf86f141af91ab2d7cd011db47|1|0|636894140675219942&sdata=yLO9aJzQg%2FGunOJ9TJF5rit8JwLyqLpuGwLnlyr3F5k%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KZYbrCJxqvkdrIJIHViMc8ciinHpks5vbVYxgaJpZM4be8sH&data=02|01|jhalleux%40microsoft.com|a8d6f4d9b6414f085f9e08d6b3d8b585|72f988bf86f141af91ab2d7cd011db47|1|0|636894140675229951&sdata=92JOhdAfHSBDx3A3BDSa64kN9dEN4CFqT4B0%2BYKs2FE%3D&reserved=0.

Aside for IE issues, this PR looks good to me.

I'll be adding changes to the Icons of the docs Blocks content. Please check it out. And question should I continue using this PR or should I open a new one. Thanks! @pelikhan @abchatra @riknoll

Screenshots?

We are basically moving the icons to the top right.

And adding some hover effect

Adding the accordion in root level

I removed the changes to docsrenderer.ts and added the styles for the "things to do" elements.
We have now a Hero macro for main or hub pages of Courses, Projects, Lessons.
The Docs now display the docMenu we have in pxtarget.json to navigate across categories of content

Animation and responsiveness added.

why do we have multiple versions of python/javascript icons? why do we need new icons?
are the python/jvaascript icon used in the editor as well?

[run] node C:\gh\pxt\node_modules\less\bin\lessc theme/style.less built/web/semantic.css --include-path=node_modules/semantic-ui-less:node_modules/pxt-core/theme:theme/foo/bar --no-ie-compat
Skipped data-uri embedding of ../docs/static/logo/icons/python-v2.svg because file not found
Skipped data-uri embedding of ../docs/static/logo/icons/python-v2.svg because file not found
Skipped data-uri embedding of ../docs/static/logo/icons/python-v2.svg because file not found
Skipped data-uri embedding of ../docs/static/logo/icons/python-v1.svg because file not found

I'm getting a double scroll bar.

when clicking on the links in the menu (i.e. "JavaScript"), it jumps to that page but scrolls to the bottom.

Yes, I'm missing the python logo. I'll add those svg's. These new logos (js, python) are used in the block box here (the python when using python of course):

Print CSS needs attention. Lots of blank area in cards since background-image does not get printed:

We already have a python logo in the semantic icon font.

Looks pretty slick otherwise

For the Print CSS issues. Is it ok if I move the changes from the other PR here? And remove the docsrender.ts change

We can fix the print later yes

Capitalizing the API names makes them less readable -- it looks like a rendering bug.

I think they are too big. These are the JavaScript names and they are not localized.

For the API Name capitalization (got it). It has to do with the Styles applied to the ui.card. I'll add a different style when we are using "blocklyPreview" elements. I guess this is when we use this type of content title?

Pending Items (June 24) for Mobile

[ ] Correct Placement of search input on Mobile
[ ] Correct Placement and interaction of Content Menu in Mobile
[ ] Use overlay with sidebar when in Mobile
[ ] Sidebar height in iPhone 100%
[ ] Search box Width in Mobile

Pending Items (June 24) for Desktop

[ ] Hide Sticky list content when Sidebar is opened
[ ] Fix content-width in iPad width
[ ] Remove resize windows method (no need anymore)
[ ] Clean up code docs.js
[ ] Clean up code style.css (move to docs.less)

I added mobile behaviors for the sidebar and the sticky content. Basically we are hiding/showing buttons and displaying different styles.

@riknoll @abchatra this looks good to go for me.

I reverted as this didn't work


Added email notification option to receive all own messages (#20179)

Backport #20179

But 1.17 has not been released yet, I hope this feature can be included from the old version update, what do other people think?

Thank you for your PR and backport.
According Gitea's release plan, the stable and rc releases only get bug fixes, but not features or enhancements.
https://github.com/go-gitea/gitea/blob/main/CONTRIBUTING.md#release-cycle
All the feature pull requests should be merged before feature freeze.
And, during the frozen period, a corresponding release branch is open for fixes backported from main branch.
Release candidates are made during this period for user testing to obtain a final version that is maintained in this branch.
A release is maintained by issuing patch releases to only correct critical problems such as crashes or security issues.

sorry enhancements where backported in some edgecases, but features wont

😭


improv: use @ts-check in the CLI's setup command

Migrating everything to TS may be too much, but It's probably worth at least using the @ts-check directive. Enabling it here indicated two things: 1) maybe we could simplify the array we're passing to prompts (I'm not sure what the filter for name == null was for exactly?) 2) fs.copy doesn't take a recursive option: https://github.com/jprichardson/node-fs-extra/blob/cc7b3b22a984de5124131c7897574091c9df00e1/docs/copy.md

@tjoar yeah im not too clued up on TS, but TS support is one of the first thing i want to look at after v1 release


Fix borderpatrole example module and weave core and auth module together

Fixes #78

Current coverage is 92.70%

Merging #79 into master will increase coverage by +14.15% as of 14a7c43

@@            master    #79   diff @@
=====================================
  Files           30     25     -5
  Stmts          373    329    -44
  Branches        16     12     -4
  Methods          0      0
=====================================
+ Hit            293    305    +12
  Partial          0      0
+ Missed          80     24    -56

Review entire Coverage Diff as of 14a7c43

Powered by Codecov. Updated on successful CI builds.

@trane please note that coverage went up, as I excluded the "example" from the code coverage

@trane I have taken care of all comments, except delete/store.set


updates and automated IDE refactoring

more cleanup and sweeping IDE refactoring changes.  some minor bug fixes emerged involving string manipulations whose result was ignored, but they didnt seem to affect tests either way.
all unit tests pass except for 5: gnuplot and db integration tests.
gnuplot-related regression "Learning" could be replaced with https://github.com/Hipparchus-Math/hipparchus/tree/master/hipparchus-fitting/src/main/java/org/hipparchus/fitting  (hipparchus is a fork of apache commons math), and/or something more powerful.  this is an interesting part of the API that could be abstracted beyond numeric-only regression to include any type of data predictors and imputers of arbitrary combinations of data types

Examples in pdatalog were removed; I consider those a kind of benchmark for evaluation. There really is an interest in streamlining this codebase, parameterizing all the collections (I found a few errors in the code), and turning it into  a powerful library. But, I use those pdatalog examples to validate builds - not unit tests.

We need to coordinate this effort. There are some known bugs in the code (actual coding mistakes) and the collections need to be parameterized.


Build fails on both server and db images

The stable branch of the database now requries mysql 5.6 unless you change the insert statements in the SQL files.
Also the server use gcc7 instead of 5 now. So I edited the server docker file to be:
apt-get install -y g++-7 &&
update-alternatives --install /usr/bin/g++ g++ /usr/bin/g++-7 90 &&
apt-get install -y libx32gcc-7-dev && \
Instead of
apt-get install -y g++-5 &&
cd /usr/bin &&
rm gcc g++ cpp &&
ln -s gcc-5 gcc &&
ln -s g++-5 g++ &&
ln -s cpp-5 cpp && \
This resolved the issues for me and now the comtainers both build and start up :)

Thank you, feel free to make a PR and I'll test/merge, otherwise, I'll make the changes in the next week or two. I appreciate you posting the fix either way :) I mostly play FFXIV when I have time these days.

Submitted the PR, just thought id share as a had about of nostalga for the game and wanted to get it running love docker so this was my goto option. Hopefully it will help some people. Just need a private server for FFXIV now :D


[Coder] Tag appended to package name when creating new class in new package and is not set on class

When attempting to create a new class in Coder if a package that doesn't exist is specified along with a tag, the result is that the package name has the tag value appended to it and this value is also a tag under this package, but selecting the newly created class doesn't show it having any tag.
GT v0.8.184

Specified package and tag are now respected during class creation:


Ubuntu 16.10 Tray Icon Issues - Square with X & Unread Message Behavior

When switching between conversations, the tray icon flashes to a black box with a red "X" in it, and then back to the normal grey icon.
Additionally, the tray icon only illuminates to the green (you have an unread message) color after selecting that unread conversation, which of course in a couple of seconds returns to the normal grey icon after the application determines that the message has now been read.
Ubuntu 16.10 x64
Kernel 4.9.0-040900
YakYak v1.4.2 (latest)

I confirm this. It is an electron issue where changing the trayicon image (even to the same) will have a transitional stage where the black box with the red 'X' appears.
I believe I minimized the number of times this happens in the #567 pull request (by only changing the icon if the state changes)
Things to test before submitting to electron/electron:

[x] Using a native image instead of a path (in case the blank box is shown while image is being created)
[ ] Calling tray from the main process instead of the renderer

Created issue in electron/electron https://github.com/electron/electron/issues/8582


x/net/http2: Transport should handle 421

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
What operating system and processor architecture are you using (go env)?
What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
What did you expect to see?
What did you see instead?

Triage, @bradfitz?

Thanks. Yeah, we could still do this sometime. It's just lowish priority, since I've never seen one in the wild and the spec only says we "MAY" retry the request.
http://httpwg.org/specs/rfc7540.html#rfc.section.9.1.2
/cc @tombergan

@bradfitz
I don't see how the transport code could handle a retry. According to the RoundTripper, "RoundTrip should not attempt to handle higher-level protocol details such as redirects, authentication, or cookies." Though a redirect is not the same as a retry, they are at a similar abstraction. I.e., callers of transport's round tripper would probably not expect that there was an intermediate 421 response before an additional request. I know there is a kind of precedence here with acting on 100 responses. Maybe you're saying that despite 4xx being client errors, 421 should be on the level of 1xx in terms of lack of client fault.
I'm new here so please let me know if I'm missing something.

There is more detail in RFC 7838. I believe what can happen is:

Client is informed that foo.cdn.com is an Alt-Svc for foo.com
Client sends http://foo.com/request to foo.cdn.com
foo.cdn.com replies with 421 (I don't fully understand why the server would do this in practice, but I guess it could happen if foo.com is removed from the CDN and the client's Alt-Svc cache has not been updated)

RFC 7838 Section 6 says that the Alt-Svc cache entry MUST be cleared, then the request MAY be retried:

Clients receiving 421 (Misdirected Request) from an alternative service MUST remove the corresponding entry from its alternative service cache (see Section 2.2) for that origin.  Regardless of the idempotency of the request method, they MAY retry the request, either  at another alternative server, or at the origin.

Since this is a MAY, I tend to agree with @meirf that RoundTrip should not automatically retry. Instead, the implementation can remove foo.cdn.com from the Alt-Svc cache to guarantee that a subsequent retry will use foo.com.
However, RFC 7540 Section 9.1.2 says:

[Receiving 421] is possible if a connection is reused (Section 9.1.1)

I really don't understand why this would happen -- isn't that what GOAWAY is for? In any case, I think the http2 client library could handle this by treating 421 as GOAWAY(NOERROR).

I really don't understand why this would happen

Here's an example of 421 because of connection reuse.

isn't that what GOAWAY is for?

I think GOAWAY is too extreme for reuse-caused 421s. Specifically, 6.8 says:

The GOAWAY frame (type=0x7) is used to initiate shutdown of a connection or to signal serious error conditions...Receivers of a GOAWAY frame MUST NOT open additional streams on the connection, although a new connection can be established for new streams...The GOAWAY frame applies to the connection, not a specific stream.

GOAWAY prevents any additional steams, but 421 (at least for connection reuse) indicates that the connection can be used again - just not with the problematic host.

I think the http2 client library could handle this by treating 421 as GOAWAY(NOERROR)

Indeed, we want the request to be retried on a different connection just as we do for GOAWAY(NOERROR). We agree that http.Transport should not be retrying as discussed above. So http.Client might have a boolean field saying retry on 421, which defaults to false. The client has a Transport field which is of type RoundTripper. A 421 is received and the client knows it should tell the transport to try the request on a new/different connection. But the RoundTripper interface in Client only takes a Request - no other options. Changing the API is probably not what we want. There is a RoundTripOpt in x/net/http2 which takes options and is bundled into h2_bundle.go. But RoundTrip calls into RoundTripOpt not vice versa.
I'd love to submit some code for this, but I don't see how it would work. I'm open to any ideas/corrections on what I've written here or above.

@meirf, thanks for the virtual hosts example. I knew about 421 with Alt-Svc, but not virtual hosts. Before diving into a solution, I want to make sure we completely understand the problem. Specifically, I want to understand all the ways a client may decide to send a request to a server that will return 421. I believe there are (at least) three ways this can happen:

Via Alt-Svc (RFC 7838), the client previously discovered that foo.com is also available at cdn.com. The client happens to have an open connection to cdn.com, so it sends requests to foo.com on that connection rather than opening a new connection.

The client has a connection open to www.foo.com. The TLS cert for that connection names multiple hosts (perhaps using a wildcard), including www1.foo.com. The client recognizes this, so it sends requests for www1.foo.com over the same connection.

foo.com is served by three distinct IPs, each of which are virtual hosts. One of those IPs is misconfigured and will return 421 for all foo.com requests. The client sees all three IPs in the DNS record for foo.com. If the client unluckily selects the wrong IP, it will get a 421.

I believe 1 and 2 cannot happen in the current Go library because Go does not implement Alt-Svc or connection sharing when advertised by TLS certs (@bradfitz, correct me if I'm wrong). The third case can happen, but unfortunately, this is trickier to fix because we'd need to blacklist specific IPs in the dialer so we don't end up with the same IP when redialing a new connection.
Does that sound right?

Thanks for bringing up the status quo: "I believe... connection."
I think it's worthwhile to go back to 9.1.2. The (unfortunately vague) gist is

the request was directed at a server that is not able to produce a response

It then lists 3 categories of cases when this might happen.
Case A

server that is not configured to produce responses for the combination of scheme and authority that are included in the request URI

Case B

a connection is reused (Section 9.1.1)

Case C

an alternative service is selected [ALT-SVC].

Cases B & C are the only cases referred to in the context of retrying. (For A, we would never want to retry the request because it would certainly fail.) For examples of B, the spec describes one with a middlebox. As another example, I referenced one above with virtual hosts that had different ssl configurations. For C, 7838 says that this happens when "an alternative service is not authoritative". One way to be authoritative is to present a TLS certificate that’s valid for the origin . 7838 also says "they MAY retry the request, either at another alternative server, or at the origin."
Going back to the 3 cases you listed.

I believe this conflates Case B and Case C together. Either the alternate service aspect or the connection reuse aspect is enough to get a 421 in some scenarios.
This is a vanilla case of when an http2 + ssl connection can be reused: "host having resolved to the same IP address" and "a certificate that is valid for the host in the URI". (7540)
This is just an example I gave of Case B. I wouldn't put it into a separate case/category.

Go does not implement Alt-Svc.

If it does not implement Alt-Svc/there is no plan to, then we can rule out Case C.

Go does not implement ... connection sharing when advertised by TLS certs.

I think this is correct - connection sharing is done only on the basis of authority (host:port). Though there is a currently a "TODO: add support for sharing conns based on cert names". The spec's only example of 421 in  case B and the only cases I've seen involve different authorities that are covered by a single cert. Because of that, I assume there are currently no cases of 421 occurring in Case B with the current implementation.

The third case can happen

I don't see how the third case could happen. The vhosts have different authorities. Additionally, it requires connection reuse based on cert because it's https.
If Cases A, B and C are exhaustive and my assumption ("...there are currently no cases...") is correct, then it looks like retrying would accomplish nothing right now. It would only be valuable once alt-svc  or sharing conns based on cert names was implemented. Even at that point, we would still have to address the issue I brought up way earlier - "retry responsibility".

Having just experienced a problem related to this in real life - I do think Go should handle it in the transport.
However, it specifically should handle it when:

a request is made on an existing shared connection
you receive a 421

And then the request should then be made on a new connection - not a shared one.
If the connection was received when sent on its own connection, pass it up to the client.  The author of the server may have been creatively using http codes that the client may know how to handle.


allow configuration of executor_signal_escalation_timeout

This is for use in Executor when processes really need more time to handle shutdown before KILL is sent.

@ooitschristina : what is the ticket #? Want to see the UT. thanks.


Update Italian translations

I've added some missing translations, as well as fixed some ugly translations, and used always the same translation terms for some original translation terms (eg "handle" is now always "nome utente").

Hi @mlocati, thanks for your Italian translation work! To streamline the localization process and avoid conflicts, I’ve merged my recent PR #6477 with another one #6457, but there is also open the #6347.
Let's coordinate our efforts and merge all Italian translation PRs into a single one.
Please let me know if you’re open to this.
cc @Lucaterzo

What about transifex or something like that?

This PR seems great to me and the most complete of the open ones, and it has many of the things I wanted to add after my PR that was already merged the other day.
I would say to keep this one, what do you think? maybe if you want @mlocati  try to see if among the strings of the other two open PRs there is something that you missed, so we can close them and leave only this one as the next update.
I don't know what you say? @lucaterzo @arestifop

I've merged into this #6487 the changes from #6477 and #6347

Great! Thanks @mlocati!
I just noticed a typo in my PR:
#: src/components/ProgressGuide/Toast.tsx:156 msgid "Tap to dismiss" msgstr "**Togga** per ignorare"
Could you please make a quick fix? I apologize for the oversight. 🙈

#: src/components/ProgressGuide/Toast.tsx:156 msgid "Tap to dismiss" msgstr "**Togga** per ignorare"
Could you please make a quick fix?

Quickly fixed :wink:

Great thanks! 💪

@marcomaroni-github PR updated - see e7846742a92b5aed9fc4487985517b21395d9d68

@marcomaroni-github @crazycoder1999 @arestifop @Lucaterzo what about switching to Transifex to maintain the Italian translations of Bluesky? This way we could collaborate on the translation without me having to pick other people's translations 😉

@mlocati are you thinking of become the mantainer of the italian language? :D
PersonaIly I don't know how much this work will go forward in next months
Let's try to reach a point where this PR is integrated in a release than we should have a Meeting on Zoom or similar and discuss some point with the one who want to continue contributing

Meanwhile I gave Transifex a try:

I created this Transifex project (If anyone wants to try it just let me know)
this repo extracts translatable strings from the bluesky repo
Transifex is set to automatically update its strings starting from that repo of mine

I've added commit c2be829d8a612146984138f39f8ebd153f949ef2

@mlocati I agree that using a community translation platform like Transifex would make managing translations much easier. It would reduce the number of PRs and make it simpler to keep track of changes. I can join the project.

I can join the project.

@arestifop what's your Transifex username?

@mlocati just joined, username "arestifop"

username "arestifop"

@arestifop invited!

@marcomaroni-github I accepted your Transifex join request too (sorry for the delay: Transifex has always had problems sending email notifications).

I've added 2b2a46956f20921352c92b05aad68cf4e6df9b89 with the latest translations from Transifex.


Possible author comment on StackOverflow?

https://stackoverflow.com/questions/56082707/textcsv-supported-encodings
StackOverflow question from 2019 on Raku's Text::CSV file encodings could use a comment from the module's author (especially now that the issue appears resolved).

I don't know if I'm one of the authors; I certainly have privs here. And my comment is there. Is there anything I can help with?

Maybe it's me who is mis-understanding. If a Repo is under "Raku Community Modules" then I would expect a Community Member's answer to suffice. If a Repo is under "Tux" then some readers might presume the issue still persists until "Tux" chimes in to say it was resolved ( in version X.Y.Z, etc.).

hmmm... I'm not sure Tux does StackOverflow @Tux ??

/me does not actively "do" StackOverflow. I /think/ I do have an acoount, but I rarely visit SO, just when people point me there.
Feel free to answer there on behalf of me: you know more about Raku than I do :)


change behavior style (data panel)

If you click on the style's link, should go to the style panel (not to the widget).
Also some things:

Maybe we could add another button to customize the widget. If we add this one, the arrow back should be back to data panel

I don't get that, why do we need to go to the style panel? For me what @nobuti has done makes sense, probably the name of the button is not the correct one.
cc @saleiva @urbanphes

yes if we change the name it has sense, but if we use "style" should go to style panel, maybe both, one for style layer and other one for edit widget

+1
On Jun 25, 2016 11:40 AM, "piensaenpixel"<EMAIL_ADDRESS>wrote:

yes if we change the name it has sense, but if we use "style" should go to
style panel, maybe both, one for style layer and other one for edit widget
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/8183#issuecomment-228526631,
or mute the thread
https://github.com/notifications/unsubscribe/AAIENZMxRFAXLkFFEOEfOP6vpP-vevKXks5qPPeEgaJpZM4I9xI_
.

So? Two buttons then? One button with different label?

I'd say yes. What about? CUSTOMIZE and STYLE?
cc/ @urbanphes

But it makes no sense to do that. Have a button (custom) without a selection previous, why¿? If I don't click the checkbox why the interface show it me the custom button¿? I don´t understand this behaviour. At the begining this module was thinking: Checkbox selected add like a widget; Style link, to make style by this column. IMO we must think we really want to do with this component.


How to Setup?

Hello, I'm trying to implement this into my company's app and I'm not seeing any installation instructions for this plugin. When I try to do cordova plugin add cordova-plugin-twiliovoicesdk I get an error like this
Failed to install 'cordova-plugin-twiliovoicesdk': Error: pod: Command failed with exit code 1 Error output:
Ignoring ffi-1.14.2 because its extensions are not built. Try: gem pristine ffi --version 1.14.2
    at ChildProcess.whenDone (/Users/hvcs/Documents/GitHub/Apps/iOS/node_modules/cordova-common/src/superspawn.js:136:25)
    at ChildProcess.emit (events.js:315:20)
    at maybeClose (internal/child_process.js:1048:16)
    at Socket.<anonymous> (internal/child_process.js:439:11)
    at Socket.emit (events.js:315:20)
    at Pipe.<anonymous> (net.js:673:12)
pod: Command failed with exit code 1 Error output:
Ignoring ffi-1.14.2 because its extensions are not built. Try: gem pristine ffi --version 1.14.2

Any ideas on why I might be getting this error?

PS. when I try to use gem pristine ffi --version 1.14.2 I get this...
ERROR:  While executing gem ... (Gem::Exception)
    Failed to find gems ["ffi"] = 1.14.2

Hi! That's actually a problem with Ruby, Cocoapods, and M1 Macs (the new ones).
Here's a workaround to use the Intel architecture for Cocoapods, for the time being:
https://medium.com/p-society/cocoapods-on-apple-silicon-m1-computers-86e05aa10d3e
You may need something similar when you run cordova.

Thanks for the reply! I figured out the issue. I had an outdated version of something don't remember what it was but I updated and now it works.


Add an explicit error to InvokeHostFunctionResult.

This is to be used for more useful error reporting for the failed contract invocations.

@graydon, should we revive this? It still seems like a useful thing to have.


Add backend_url option to client initializer

To use a custom backend URL, for example to be able to use the EU Novu API, you can pass a custom backend_url when initializing the client.
client = Novu::Client.new(
  access_token: '<your-novu-api_key>',
  backend_url: 'https://eu.api.novu.co/v1'
)

Thanks for submitting this PR @ludwigbacklund
@Eazybright Please can you help review this PR?

I also have this issue. Would be great if we can have this merged 🙏

@ludwigbacklund Tests are failing. https://github.com/novuhq/novu-ruby/actions/runs/8188109840/job/22390018410#step:6:1294
Please can you send a PR to test this and ensure all the tests are passing. cc @Eazybright

@unicodeveloper https://github.com/novuhq/novu-ruby/pull/74

@ludwigbacklund @Augustalex New release is live. Update your packages to v2.0.0


Ignore null in generic lambdas parameters

This is a WIP and needs to be double-checked before merge.
Related but not exact reasons why this happens can be found in JDK's
issue tracker: https://bugs.openjdk.java.net/browse/JDK-8178523?jql=text ~ "lambda generic type"
We ignore nulls that are returned by getActualTypeArguments.
It would be good to file a ticket in JIRA before merge.
Fixes #389.

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

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

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

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

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

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

Things that I need to do before merge:

[ ]: Update commit message and PR with description of fix and trade-offs.
[ ]: Open ticket in JDK's issue tracker to keep track of this one.

This slipped through the rocks. Handling tomorrow.

It looks like I cannot sign in into JDK's issue tracker, so I won't be filing a ticket. I'm updating the commit message with the reason behind this problem.

This PR has been updated with a better commit message.

So this should target 1.0.x branch, right?

Closing this in favor of https://github.com/sbt/zinc/pull/446.
Note that 1.0.x is slightly ahead of 1.x now.

Thanks for the rebase. I missed this notification.


Block setting spec.ConfigSource for Kubernetes 1.24+

What this PR does / why we need it:
We already removed the feature flag in kubelet, but in my opinion we should also block this from being set in a MachineSpec, as it mirrors to the Node object and I would expect that the NodeSpec fields are going to go away eventually as well.
Which issue(s) this PR fixes (optional, in fixes #<issue number> format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Optional Release Note:

Machine specs for Kubernetes 1.24 or higher are not allowed to set spec.ConfigSource (as the dynamic kubelet configuration feature has been removed)

Signed-off-by: Marvin Beckers<EMAIL_ADDRESS>

/retest

/retest

/unhold


Analysis tab style tweaks

To fix the test, put this on the line 37 of templates_test, run it once, remove:
new File('$goldenDir/$fileName').writeAsStringSync(content);

PTAL


Add Gradle configuration for Kotlin DSL files without plugins

Thanks - i'll patch this in to test and if everything seems okay I'll merge (after you update the versioning.)

To help reduce on conflicts, could this change just add the Kotlin tasks and not change the formatting of the README.md?


Flash target details

Getting started page with minimal setup
Embedding resources
Using Flash libraries
Flash target metatags

@Merelleya: Can you comma-check this one and https://github.com/HaxeFoundation/HaxeManual/pull/89 and https://github.com/HaxeFoundation/HaxeManual/pull/92


Cannot read property 'customStyle' of undefined

Using version 0.7.0 when using the code from the example in Readme I get this error, which relates to Day.js:33

When using 0.6.4 the styles seem a bit messed up:

@chrise86 hi, can you show your code in order to help you?

I had to give up and try something else, so don't have the code anymore. However, I pretty much just copied the example code in the readme into a new (es6) component and as soon as I navigated to that component it crashed. If I find some time I'll try and replicate it and share the code.

@chrise86 I had the problem of style in 0.6.4 but I don't remember how I solved it.
I could probably help you if you decide to try it one more time.

I'm had the dayCircleError as well. I'm just running the example. I'm a bit lost on how each of these styles are supposed to work. What does a filler expect for instance?

Shouldn't be a problem in the latest version.
Please re-open if you see the issue come up.


Publish 11/11/2021, 10:30 AM

Docs Build status updates of commit 4c62618:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-DlpCompliancePolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Support for 1.20.6

Could you add support for version 1.20.6? I'd appreciate it, thanks!

you can use 1.20.4 version on 1.20.6


"wget https://get.docker.com/..." failed while building the container whisk/dispatcher

I run "ant build" and meet with an error while building the container whisk/dispatcher.
 [exec] Executing 'docker --host <IP_ADDRESS>:4243 build -t whisk/dispatcher /home/daisy/workspace/openwhisk/core/dispatcher'
.......
 [exec] Step 1 : FROM whisk/scala
 [exec]  ---> ebdaf0547678
 [exec] Step 2 : RUN wget -O /usr/bin/docker "https://get.docker.com/builds/Linux/x86_64/docker-1.9.0" && chmod +x /usr/bin/docker
 [exec]  ---> Running in a5dcc5836e59
 [exec] --2016-05-10 09:21:16--  https://get.docker.com/builds/Linux/x86_64/docker-1.9.0
 [exec] Resolving get.docker.com (get.docker.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
 [exec] Connecting to get.docker.com (get.docker.com)|<IP_ADDRESS>|:443... connected.
 [exec] Unable to establish SSL connection.
 [exec] The command '/bin/sh -c wget -O /usr/bin/docker "https://get.docker.com/builds/Linux/x86_64/docker-1.9.0" && chmod +x /usr/bin/docker' returned a non-zero code: 4
 [exec]
 [exec] Command 'docker --host <IP_ADDRESS>:4243 build -t whisk/dispatcher /home/daisy/workspace/openwhisk/core/dispatcher' failed with exitCode 1, 2 retries left, retrying...

But in my host server, "wget -O /usr/bin/docker "https://get.docker.com/..." works fine. My host server is Ubuntu 14.04 and wget is 1.15. I don't know the wget version in container whisk/dispatcher.
And I find this: https://github.com/docker/docker/issues/14650 and a patch to change wget to curl in the official docker document: https://github.com/ankushagarwal/docker/commit/508ab7adceba1f6b4cea1f74ce5c24eccb717222.
I think it might be better to use "curl" here. I'd like to dig more on the root cause of this issue.
Daisy

I don't reproduce it anymore.
I will close it.


Flat modifiers to AC not working with predicates

The recent 4.12.0 update seems to have broken any flat modifiers to AC that use a toggle predicate. i.e. Warding Aggression, Nimble Dodge. Non-predicated bonuses seem to affect AC just fine though.

Fixed by #7950

Hello! I'd actually like to reopen this, as the bug is still applying for me? Warding Aggression is a status bonus and the toggle -is- checked on, but still not applying. Or do I make a new ticket?

OK, we'll take a look!


Highlight Importance of setting correct baseName

The guidance should be updated to highlighted importance of baseName parameter as most resource names will include this text including resources like keyvault and storage account which need to be globally unique so provide this value needs to be provided accordingly.

This has been addressed in #20


why i can't  find the instructions on how to load and display the training data.

the link is empty and shows nothing

Do you mean this link: https://github.com/openai/baselines/blob/master/docs/viz/viz.ipynb?
It works for me.


SnarkyJS parties uber test: Parties transaction through SnarkyJS/MinaSigner/Mina daemon.

Explain your changes:
This PR adds a new integration test snarkyjs.ml along with a new buildkite job "build-js-tests" to support it. The test works as follows:

From the test executive, we call out to a nodejs script which can generate two transactions, one for deploying a zkapp and one for updating it with a proof
The nodejs script uses snarkyjs to generate the bulk of the transactions and mina-signer to add a feePayer and sign it. So we're testing the full "end-user" flow of interacting with zkapps
We send both of those transactions to the network and wait for them to be applied
We check that the new zkapp's balance and on-chain zkapp state were updated as expected

In addition to the above, there are some changes to the ocaml side of snarkyjs:

Expose a Sponge to JS which can absorb and squeeze field elements. This was added a while ago in support of https://github.com/o1-labs/snarkyjs/pull/99, but we didn't have snarkyjs in develop back then, so it's merged here
Enable the JS Pickles interface to compile SmartContracts with > 1 method. Closes https://github.com/MinaProtocol/mina/issues/10724
Expose methods to compute and sign the transaction statement (in support of making the intg test work)
In the mock Mina interface, check signatures before calling apply_parties_unchecked (in support of making the intg test work)

Explain how you tested your changes:

By checking that the new integration test works in CI

Closes #10438
Closes #10749
Closes https://github.com/o1-labs/snarkyjs/issues/110
Closes #10724

!ci-build-me

I think we're good to go :D


Uncaught ReferenceError: $ is not defined in TOTP registration page

Describe the issue:
When progressive enrolment is enabled for TOTP, QR for the TOTP enrolment is not displayed in the authentication flow due to the error in $subject.
How to reproduce:

Create an application.
Add TOTP in the authentication flow. (As a MFA option)
Create an user from console.
Try login to the application with the user.
TOTP is not getting prompted in the expected step.

Expected behavior:

TOTP should be prompted in the relevant step, if the previous steps are successfully authenticated.

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

NOTE: The issue is not reproducing in the latest IS product.


Change the colour of the highlighted section

Checked other resources

[x] added a very descriptive title to this issue.
[x] searched the greatbuilder documentation with the integrated search.
[x] used the GitHub search to find a similar question and didn't find it.

Issue Image (if any)

Issue in existing code (if any)
Find the code to fix it
Description
Chage the color of the section in the blue circle to greyish white
System Info
Windows 11
npm 10.2.3
node 20.10.0
Related Component

[ ] Deployment
[x] Nextjs
[ ] LLMs
[ ] Langchain
[ ] Rust
[ ] Python

Maybe should change the colors of Get GreatBuilder for free button too to match.
Is bg-slate-200 good or a darker one would be better ?


When using OpenAI chatGPT. How can i customize system_prompt.

Hi, I wish to add some customize content in system_prompt. Hence I don't want to hard code the system_prompt. Is there any supported functions in deep chat that I can use? Thanks!

Sorry for the late reply. What do you mean by customize the system_prompt content.
The OpenAI Chat documentation details on how you can change the system_prompt property content to your liking. E.g:
<deep-chat
  directConnection='{
    "openAI": {
      "key": "your key",
      "chat": {"system_prompt": "You are a helpful assistant that can answer anything and everything"}
    }
  }'
></deep-chat>

Closing this issue due to no further communication.


Run adb reverse when platform is 'all'

adb reverse is only run when platform is android, it should be run if platform is all also.

@zamotany can you please cherry-pick this onto next branch?

@thymikee I think there is no need for that


Tooltip placement menu link

Subject
Tooltip
Description
The link for "Placement" in the left menu is broken: https://chakra-ui.com/docs/overlay/tooltip#placement

It works now

Can confirm it's working for me too.

@romseguy what do you mean by link? The link in the Table of Contents on the right?

Closing this because of inactivity. The behaviour explained cannot be reproduced.


[BUG] My bot is not answering me when I mention it on a channel

Describe the bug
I've created a Discord bot and added it to my server, but when I mention it in a channel, it doesn't answer, although my local code is running. Also, I can't DM it.

Same thing here

This bot is not meant to respond when you mention it in a channel. You also cannot DM it. You need to create a chat with /chat to use the bot.


Problems with downloading games from watch.nba.com

Hi
Great project - really useful for watching videos off-line.
Though I cannot get the latest binary to download anything from my NBA League Pass subscription.
Tried all flavour of the url, i.e.: https://watch.nba.com/game/20170407/ATLCLE?s=i
and getting:

ERROR: Unable to extract video id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

When in verbose I'm getting this:

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'https://watch.nba.com/game/20170407/ATLCLE?s=i', u'--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.04.03
[debug] Python version 2.7.10 - Darwin-15.6.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg N-85274-g9fe73b0-tessus
[debug] Proxy map: {}
[NBA] ATLCLE: Downloading webpage
ERROR: Unable to extract video id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 761, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 429, in extract
ie_result = self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/nba.py", line 137, in _real_extract
path = remove_start(self._search_regex(r'data-videoid="([^"]+)"', webpage, 'video id'), '/')
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 778, in _search_regex
raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract video id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Does anyone have the same issue or know other way of downloading games from nba?
Would appreciate your comments / views
Cheers
m

It looks like the site requires authentication.  I'm not able to test it because I do not have an account.  Have you tried using the --username and --password options?
youtube-dl --username YOUR_USERNAME --password YOUR_PASSWORD https://watch.nba.com/game/20170407/ATLCLE?s=i

Hello

Hi masonelmore
Thanks for your comment.
Yeap, tried the -u and -p but unfortunately no joy - I get the same error... :(
m

Same problem, --username and --password options doesn't solve it, can provide active NBA League Pass account credentials to developers, NBA website seems to be supported on the youtube-dl list

I'm also seeing this issue using cookies.txt from watch.nba.com


Adds various laws to the haddocks

I added most of the laws to the haddocks. I had trouble coming up with descriptions for about half of them.

I added a bunch more descriptions and updated as per your recommendations.

These all seem good to me (just need full stops). @Lysxia - thoughts or suggestions before I merge?

LGTM
On 9/29/2020 1:23 PM, Koz Ross wrote:

These all seem good to me (just need full stops). @Lysxia
https://github.com/Lysxia - thoughts or suggestions before I merge?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kozross/test-monad-laws/pull/1#issuecomment-700861545,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATGCALLM3AW3SJ6YM6LWLDSIIJYJANCNFSM4R42CHXQ.

By full stops, do you mean periods to end sentences?

@ssbothwell I meant exactly periods to end sentences - sorry that I was unclear.

Sorry about the delay, i just pushed a change with all the periods added.

I wouldn't want to let this linger if you can't get to these last changes. @ssbothwell Would you prefer we merge this PR and keep polishing separately/on another PR?

I just corrected the typos, but I didn't change (f . ('fmap' . 'fmap') 'absurd') to be pointful. I skipped it because I'm having trouble building the project currently due to quickcheck-higherorder not being in the stable channel for nix. 🤷

No problem, thanks for the help!

Happy to help!

If @Lysxia is good with merging this, I'll merge.

:+1: @kozross Do you plan to make more changes soon, or should I merge the original PR after this?

@Lysxia Probably fine to merge - I don't think I could add anything to the work @ssbothwell already did here.

@ssbothwell Have you addressed all of @Lysxia's comments to their satisfaction? If so, I'll merge.

I think this is good to merge.
On 10/7/2020 5:13 PM, Koz Ross wrote:

@ssbothwell https://github.com/ssbothwell Have you addressed all of
@Lysxia https://github.com/Lysxia's comments to their satisfaction? If
so, I'll merge.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kozross/test-monad-laws/pull/1#issuecomment-705196930,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATGCAK6RTTG6RSF6YA6EALSJTKXVANCNFSM4R42CHXQ.

And merged! @ssbothwell - thank you so much for doing this!

I didn't have any more changes. Thanks for merging!
On Wed, Oct 7, 2020, 7:39 PM Koz Ross<EMAIL_ADDRESS>wrote:

Merged #1 https://github.com/kozross/test-monad-laws/pull/1 into master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kozross/test-monad-laws/pull/1#event-3853233790, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAF3ZWO5P344KV2FLOEVKTSJUQ7ZANCNFSM4R42CHXQ
.


Add a ZIO-style Service type for idiomaticity points

There's usually (when adhering to the ZIO module pattern) a Service trait which defines interface (send and openWebsocket here).
In lieu of that refactoring, it could be nice to expose a Service type alias such that it could be idiomatically accessed when relying upon SttpClient in ones own ZLayers.
Then it would be trivial to refactor from
  def live(token: String): ZLayer[Console with SttpClient, Nothing, SlackClient] =
    ZLayer
      .fromServices[Console.Service, SttpBackend[Task, Stream[Throwable, Byte], WebSocketHandler], SlackClient.Service](
        (console: Console.Service, sttp: SttpBackend[Task, Stream[Throwable, Byte], WebSocketHandler]) =>
          LiveSlackClient(console, sttp, token)
      )

to
  def live(token: String): ZLayer[Console with SttpClient, Nothing, SlackClient] =
    ZLayer
      .fromServices[Console.Service, SttpClient.Service, SlackClient.Service](
        (console: Console.Service, sttp: SttpClient.Service) =>
          LiveSlackClient(console, sttp, token)
      )

Ahh, much better ;)
I'd also be happy to fully refactor this to use the trait Service pattern if that's something you'd be into, my dear sttp-eople.
Take care and thx for all the lovely software!

Released in 2.2.5, thanks!


Alert types for targeting and tracking

@ldodds This is an after party task that creates the new alert types for targeting and tracking that Philip is introducing - they're already on test, but this needs to be merged to master (and then can be part of the release tomorrow).

@ldodds This is an after party task that creates the new alert types for targeting and tracking that Philip is introducing - they're already on test, but this needs to be merged to master (and then can be part of the release tomorrow).

Looks OK to me :+1:

Looks OK to me :+1:


Error: Unsupported block type; Blocks of type "contact" are not expected here.

Describe the bug
A clear and concise description of what the bug is.
Terraform plan return code: 0
Terraform returned errors:

Error: Unsupported block type

  on /home/vscode/.terraform.cache/modules/launchpad/modules/security/keyvault/keyvault.tf line 50, in resource "azurerm_key_vault" "keyvault":
  50:   dynamic "contact" {

Blocks of type "contact" are not expected here.

Error on or near line 446: Error running terraform plan; exiting with status 2000

To Reproduce
Steps to reproduce the behavior:

git clone https://github.com/Azure/caf-terraform-landingzones-starter.git
Reopen in container
git clone --branch 2010.0.0 https://github.com/Azure/caf-terraform-landingzones.git /tf/caf/public
rover login
rover -lz /tf/caf/public/landingzones/caf_launchpad -launchpad -var-folder /tf/caf/configuration/demo/level0/launchpad -a apply
See error

Expected behavior
Expecting the launchpad to deploy.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: Windows 10 with WSL2 Ubuntu 20.04
Rover Version: aztfmod/rover:2010.2808

Additional context
Following along with the getting started video here: https://www.youtube.com/watch?v=M5BXm30IpdY

I was able to spend a few cycles on this.
In /tf/caf/public/caf_launchpad/main.tf I changed the hashicorp/azurerm version from 2.33.0 to version 2.37.0.
Once I updated the version, this worked as expected, so instead of pulling from the tag mentioned in the docs, I pulled from master.

thanks for the PR @EntityAdam


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Modify for 3DT for downstream compatibility

Minimial refactor so that 3DT is compatible
with SSL enabled clusters

@darkonie PTAL
@cmaloney @spahl This is ready for review. The refactor is by no means a great example of how this should be done, but the goal is to have ~700 lines less of code to consider when overhauling the cluster test API

@cmaloney Comments addressed assuming you are OK with my position on setting via environment variable

Responded inline to you're comment, I'm fine with not doing env vars for now, but definitely need a TODO comment then.

Should be good once the code-quality is fixed

:ship:


Speed up wiki scraping

This is different from https://github.com/luttje/glua-api-snippets/pull/31, this PR aims to speed up the "initial" wiki scrape. It is way too slow right now.
Also wipe the output folder by default when running npm run scrape-wiki, so that files do not have duplicated data.

Hi there, thanks a bunch for contributing to this little project. I really appreciate you taking time form working on Garry's Mod to help out here.
IIRC me not wiping the output folder by default was so I could separately test the packing functionality on preexisting output (without accidentally wiping my "test data"). However since everything seem to work alright, there's no reason not to wipe now.
Thanks again!

@all-contributors add @robotboy655 for code


Array type in struct raises MethodError

I'm in the process of updating some old code that was working with ModelParameters.jl v0.3.0.
I am updating the code to work with Julia v1.11.1 and ModelParameters v0.4.4
The model structs define a mix of Param fields and non-Param fields, but the error seems to be raised in the presence of arrays:
using ModelParameters

# Works
Base.@kwdef struct OkayStruct
    a = Param(0.5, bounds=(0.2, 0.9))
end

Model(OkayStruct())

# Works
Base.@kwdef struct MixedStruct1
    a = Param(0.5, bounds=(0.2, 0.9))
    n = "test"
end

Model(MixedStruct1())

# Does not work
Base.@kwdef struct MixedStruct2
    a = Param(0.5, bounds=(0.2, 0.9))
    b = []
end

Model(MixedStruct2())

# Does not work
Base.@kwdef struct MixedStruct3
    a = Param(0.5, bounds=(0.2, 0.9))
    b = ["test"]
end

Model(MixedStruct3())

# Works fine
Base.@kwdef struct MixedStruct4
    a = Param(0.5, bounds=(0.2, 0.9))
    t = ("test", )
end

Model(MixedStruct4())

The error raised is always the same and the stacktrace is:
ERROR: MethodError: no method matching GenericMemory(::Int64, ::Ptr{Nothing})
The type `GenericMemory` exists, but no method is defined for this combination of argument types when trying to construct it.
 Use `show(err)` to see complete types.

Yes this is because of the changes to Array in new Julia versions.
We need to fix it, maybe even in ConstructionBase.jl, but I haven't had time to look.

I haven't tried it, but I think one quick workaround would be to include AbstractArray in the ignored types passed to 'Flatten`.


Library not found

java.lang.UnsatisfiedLinkError: dlopen failed: library "libswresample.so" not found

Is a duplicate of another issue.

Is a duplicate of another issue.


feat(test): add pascals triangle exercise (resolves #156)

Gave this a difficulty rating of '3'. It's either a '3' or '4'.
Bit annoyed that I had to use clone in the next_row() function.

I'm always passing the last row of the triangle to next_row. Tried making the row mutable but the prepending & appending of '0' polluted that row, as expected. So I have to call clone.
As an aside, I tried the following in the playground
fn foo(mut a &[]int) {
	a << 2
}

fn main() {
	mut a := []int{}
	foo(mut &a)
	println(a)
}

And got the following error
code.v:1:15: error: V arrays are already references behind the scenes,
there is no need to use a reference to an array (e.g. use `[]string` instead of `&[]string`).
If you need to modify an array in a function, use a mutable argument instead: `fn foo(mut s []string) {}`.
    1 | fn foo(mut a &[]int) {
      |               ^
    2 |     a << 2
    3 | }

I did not realise that but, it makes sense as the alternative is to make a copy of the array each time foo was called. This could be expensive.
While we are on the subject of references: In the following code reveal has a reference to the structure, show does not. Is there any difference between these two methods?
struct Value {
	v int
}

fn (a Value) show() int {
	return a.v
}

fn (a &Value) reveal() int {
	return a.v
}

fn main() {
	v := Value{3}
	println(v.show())
	println(v.reveal())
}

Honestly, I don't know enough about when V passes by ref vs by val to know if there's a difference between the two methods. You could always try mutating some stuff to try to see if there's any difference in how they're treated, but I don't know off the top of my head


merge package template from Kronos-Tools/npm-package-template

Updated standard to latest version

Codecov Report

Merging #149 into master will increase coverage by 0.39%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
+ Coverage   90.14%   90.54%   +0.39%
==========================================
  Files           1        1
  Lines          71       74       +3
==========================================
+ Hits           64       67       +3
  Misses          7        7

Impacted Files
Coverage Δ

src/module.js
90.54% <0%> (+0.39%)
:arrow_up:

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 5b2decf...07983b4. Read the comment docs.


Remove Google Cloud logger dependency?

I'm wondering if having the Google Cloud logger as a dependency is really neccessary?  It's adding a few Mb to my lambda bundle which is a bit large for a lightweight slack nestjs wrapper.  Is it possible to remove that with the default nestjs logger?  I feel like that would make this a bit more nestjs friendly.

Yes, I think that is doable.
We probably rid ourselves of some of the types used there or copy them over. The dependency can be moved as a peer dependency or potentially dropped in favour of writing it ourselves. Especially considering that we are not using any of the Google Cloud-logger request bits (meaning where we send logs directly to Google Cloud Logger). We're just using the form and types.

I was unable to run the tests unfortunately... node-fetch / esm vs commonjs issues?  But here we go!
https://github.com/bjerkio/nestjs-slack/pull/326 @simenandre

nvmd, fixed tests


ColQwen2 example

Demonstrates using the new model checkpoint based on QWen2 https://huggingface.co/vidore/colqwen2-v0.1
I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Awesome with notebook with better and faster model! 🚀
Linkcheck will pass as the docs are updated.
PR is open to fix Notebooks except cloud.
A few points to consider:

Rename application to visionrag6 to avoid conflict in naming (will fail if run in CI in parallel)
Add tqdm to pip installs (may be depended on by one of the others, so possibly not necessary)
Add to TOC