Adds in file ignore to repo sync

This Pull Request adds in code that allows files to be ignored before they are copied over to a built theme repository. In order to make this change, I updated the parts of the code that originally ignored files from deployments so that we are using the same logic everywhere.
When we are seeing if a file is going to be ignored or not, we are actually filtering an array of objects that represent the files rather than reviewing a file system, so we cannot use a standard implementation of glob functionality, i.e. with Globby.
I've opted to allow for use of Regex patterns in the ignore array, so that we can leverage this for needs like ignoring all JSON templates.
A lot of the work here was writing the tests around the actual code changes.
Note: New NPM deployments can be be done via Github Actions so anyone with the right access to the repo can deploy patch, minor and major releases.

@HHAxelG Bumping you here, have a look at this when you have a moment before we add it in and version bump


UNMET PEER DEPENDENCY<EMAIL_ADDRESS>

don't know why npm i -g still complains about peer dependency in my env,

upgraded npm to v3.7.3  (sublime 3 works fine after upgrade though, thanks)
manually checked eslint -v which returns v2.2.0

Strange. Do you have any other plugins installed? I just upgraded my eslint-plugin-mocha to 2.0, it was responsible for that same message for me.

@benmosher sure, currently installed plugins:

babel-eslint
eslint
eslint-config-airbnb
eslint-plugin-import
eslint-plugin-react
jshint
npm
react-native-cli

which eslint? I see that same message for npm list -g when in an nvm version that didn't have ESLint installed at all.

Which is to say: you could try npm i -g eslint or npm up -g eslint and see if it resolves?
or just npm list -g  | grep eslint

(sorry for the timDiff) result of npm list -g | grep eslint is,

+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| `--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>
and I did npm i -g eslint before running npm i -g eslint-plugin-import...
whereas the latter just responsed,

C:\Users\Timothy\AppData\Roaming\npm
+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

Anyway, it's just a WARN in npm 3, looks better than in npm 2 :)) Thanks'

Super weird. I'm not sure why that would happen. I didn't change the peerDependencies spec between 1.0.0 and 1.0.1, so I'm not sure why that would happen.

I am having same exact issue and I've been driving me mad when trying to deploy a vagrant box which installs eslint 2.x and eslint-plugin-import for a Meteor application.
I have run npm clean cache multiple times, sudo, as well as updating NPM to version 3.7.3, also reinstalling eslint and nuking /usr/local/lib
→ npm install -g eslint-plugin-import
/usr/local/lib
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

My best understanding is that npm 3.x no longer manages peerDependencies directly. However that does not explain the fact that I have eslint 2.10.2 installed in the same directory as eslint-plugin-import.
(source)

peerDependencies
peerDependencies no longer cause anything to be implicitly installed. Instead, npm will now warn if a packages peerDependencies are missing, but it's up to the consumer of the module (i.e. you) to ensure the peers get installed / are included in package.json as direct dependencies or devDependencies of your package.
#3803 npm also no longer checks peerDependencies until after it has fully resolved the tree.

So is the solution just to ignore the error message? I'm getting something similar for<EMAIL_ADDRESS>$ npm --version
3.9.5
eslint --version
v3.2.2
$ npm install
...
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>...

While my package.json contains "eslint": "^3.0.1",

My suspicion was that this resulted from #437. What version of eslint-import-resolver-node is installed?

eslint-import-resolver-node is on 0.2.2 and eslint-plugin-import is on 1.12.0:
$ npm list | grep eslint
├─┬<EMAIL_ADDRESS>├─┬ UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ ├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>

ah, got it. yeah, your issue is airbnb's config, looks like the version you've got wants 2.9.0

Yeah, I was thinking this could be a separated issue because the error message is UNMET PEER DEPENDENCY<EMAIL_ADDRESS>which suggest that some package is looking for 3.x that should be already be available.

Ah, yeah, IIRC that message actually means you've got 3.2.2 and it's not meeting a peer's dependency requirements. I see similar things while using React v14 with component libraries looking for v13.

Hum, I'm confused! The part
├─┬<EMAIL_ADDRESS>│ ├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>
would then suggest that eslint-config-airbnb is using eslint 2.9.0 while it needs another version (from here is looks like it needs eslint  3.2.2)
(the wording of
npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>is then a bit confusing :)
And then does the part
$ npm list | grep eslint
├─┬<EMAIL_ADDRESS>├─┬ UNMET PEER DEPENDENCY<EMAIL_ADDRESS>
means that some top packages level is using eslint  3.2.2 and would need some other version?

ah, right. that is bizarre.

I got the same problem

@wangzhen10 does npm ls exit zero?

npm install
npm info "eslint-config-airbnb-base@latest" peerDependencies
npm install -g install-peerdeps
install-peerdeps --dev eslint-config-airbnb-base

Add "extends": "airbnb-base" to .eslintrc file.
Finally, SUCCESS eslint-config-airbnb-base and its peerDeps were installed successfully.


Confirmed to work with b3s

I apologize if I could have gone a different way about confirming this. I was able to successfully print with bluetooth. believe it or not, I cannot find an actual micro usb cable that has data cables inside, I am kind of annnoyed by that honestly. The command was standard except I didn't rotate the image. Full command "sudo poetry run python niimprint -c bluetooth -a mac address -r 0 -i examples/B21_30x15mm_240x120px.png"
I used the default model (b21) This could vary on which distro you use, I am on Manjaro 23.1.4 kde, whenever I want to send a label to print I have to confirm code in the bottom right. Every print it makes the same noise on a successful connection, and then makes the noise again after it prints, idk if this is something in your code, I dabble in python so I may see. If I find an actual data transfer micro usb I'll test it and change this later.

@AndyADD what bluetooth address are you using? Usually there's two different ones for one printer. And did you trust/pair/connect to the printer first using bluetoothctl?

@AndBondStyle Bluetooth address I used starts with an "E1" and the other mac address is the same except of "E1", its "C1" if you want the whole mac address let me know I'll provide it, just was sure if it was necessary. I paired with both mac addresses, and I'll be honest I was making things more complicated than they needed to be I believe.
I use bluetoothctl, however, the bluetooth UI I was able to pair/connect as well and still send labels to print which IDK what stock program manjaro supplies, but it seems to work with bluetoothctl when I installed it.
If you need more info im happy to provide it, I don't use github much, and I kept making guesses on what order I neeed to do to make it work, like I mentioned im probably over complicating it.

@AndyADD there's a note in the readme (recently added):

It seems like B21 and B1 (and maybe other models?) have two bluetooth adresses. They have the same last 3 bytes, but the first 3 are rotated (for example AA:BB:CC:DD:EE:FF and CC:AA:BB:DD:EE:FF). Connection works only if you disconnect from one and connect to the other. After connecting via bluetoothctl you may get org.bluez.Error.NotAvailable br-connection-profile-unavailable error, but printing works fine regardless.

To identify which address is the correct one, run bluetoothctl info on the address you want to check. The incorrect one might list UUID: Generic Access Profile and UUID: Generic Attribute Profile, while the correct one will list UUID: Serial Port.

Could you write both full addresses, and also bluetoothctl info output? My current theories are:

You should disconnect from one of the addresses, and use only the right one (serial)
Maybe printer disconnects / goes to sleep because currently we don't send keepalive messages (and we can't, because the script runs manually, not in a daemon mode like I suggested in #9)
Maybe B3S printer is special in some way I doesn't know (I only have B21)

@AndBondStyle Here is the output of the two, and it appears I only need to connect to the serial port mac address, I am paired to both mac addresses but it seems I dont need to do the connect/disconnect thing, I can connect to the serial port mac address and have it still print.
Device E1:E3:31:09:08:60 (public)
Name: B3S-E331090860
Alias: B3S-E331090860
Class: 0x00040680 (263808)
Icon: printer
Paired: yes
Bonded: yes
Trusted: yes
Blocked: no
Connected: no
LegacyPairing: no
UUID: Service Discovery Serve.. (00001000-0000-1000-8000-00805f9b34fb)
UUID: Serial Port               (00001101-0000-1000-8000-00805f9b34fb)
UUID: PnP Information           (00001200-0000-1000-8000-00805f9b34fb)
Modalias: bluetooth:v005Dp223Bd0100
Device C1:E3:31:09:08:60 (random)
Name: B3S-E331090860
Alias: B3S-E331090860
Paired: yes
Bonded: yes
Trusted: no
Blocked: no
Connected: no
LegacyPairing: no
UUID: Generic Access Profile    (00001800-0000-1000-8000-00805f9b34fb)
UUID: Generic Attribute Profile (00001801-0000-1000-8000-00805f9b34fb)
UUID: Device Information        (0000180a-0000-1000-8000-00805f9b34fb)
UUID: Vendor specific           (0000e0ff-3c17-d293-8e48-14fe2e4da212)
Modalias: bluetooth:v005Dp0000d0100

@AndyADD thanks for the info! To be clear, the only problem you're having right now is that your printer makes noise every time it starts/ends the print?

Well from what I am seeing with the bluetooth icon in the bottom right, it appears to connect -> print -> disconnect. Which is not a big deal might be a quirk with this model.

I have a D11 and it does the same thing AndyADD states. connect -> print -> disconnect
Other than that it worked great! Used the vendor app to calibrate for gaps and offsets tho.
I'm currently working on reversing the NIIMBOT app. Maybe I will find something useful.

@JoshGud777 the official app is, well, an app, meaning that it's a long-running process. And it probably sends keepalive messages in the background to prevent the printer from going to sleep. On the other hand, our script just starts, does the printing and exits, meaning we don't have any way to "keep the connection alive". Except to implement something like described in #9 (daemon/background mode)


View Publishing

I really love this package as it has made the process of displaying maps and coordinates super easy.
Thank you so much for putting in the time to build this!
I do have one request for improving the functionality. Currently on both map types you are essentially hardcoding in the height which is making it a bit more difficult to use with css frameworks such as bootstrap and tailwindcss.
This would be something that I could easily change if the views were published out in the same way that the config file is.
Would you consider changing uncommenting the following lines in LaravelMapsServiceProvider.php
...
// Publishing the views.
$this->publishes([
      __DIR__.'/../resources/views' => resource_path('views/vendor/maps'), // changed Laravel-maps to map since that is the value provided in loadViewsFrom
], 'views');
...

I have done a bit of digging in Laravel's package development documentation and noticed that when using $this->loadViewsFrom it will look in two places. The first of which is the views folder within the package and the second is the views/vendor folder within the resources directory.

When you use the loadViewsFrom method, Laravel actually registers two locations for your views: the application's resources/views/vendor directory and the directory you specify. So, using the courier package as an example, Laravel will first check if a custom version of the view has been placed in the resources/views/vendor/courier directory by the developer.

This would allow any other developers to make minor changes to the view without much effort.
Here is the documentation that I was reading by the way.
https://laravel.com/docs/9.x/packages#overriding-package-views
If you would like I can also test this and submit a PR if you would like, please just let me know in the comments on this issue and I can help as needed.

hi @ryangurn, awesome suggestion!
you can pass in a style or class attribute which will be used for the map, as seen here:
https://github.com/LarsWiegers/laravel-maps/blob/master/resources/views/components/google.blade.php#:~:text=<div id%3D"{{%24mapId,></div>
let me know if that is not enough for your needs.

I have tried to pass style and it works but I want to customize it more than just the styles and its much easier when the views are published out to the resources/vendor/... directory.

@LarsWiegers are you open to publishing the views out?

I dont currently know enough about publishing views and what that would mean for future version of that file to say yes to this right now. I will have to do some research into it.

@LarsWiegers the link in my initial post to this thread acts as a good start. let me know if there is anything you need to understand view publishing!

i added a comment to the pr, if you change that I can merge it

thanks for merging it in!

@ryangurn thanks for the pr for it!


recreate doesn't follow newest config (commit prefix)

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
gradle
Package manager version
7.2 with kotlin
Language version
kotlin gradle script
Manifest location and content before the Dependabot update
/build.gradle.kts
dependabot.yml content
https://github.com/fixrtm/fixRTM/blob/6e9e00837f37b82592750af99555110c2f4bb7d4/.github/dependabot.yml
Updated dependency
see fixrtm/fixRTM#519
What you expected to see, versus what you actually saw
EXPECTED: on @dependabot recreate、dependabot should follow the newest config
ACTUAL: dependabot follow the old (the time PR is created) format.
Native package manager behavior
N/A
Images of the diff or a link to the PR, issue, or logs
see fixrtm/fixRTM#519
Smallest manifest that reproduces the issue
Updating commit prefix of .github/dependabot.yml should reproduce this problem
Maybe relates #3419

👋 pretty sure this is a duplicate of https://github.com/dependabot/dependabot-core/issues/4652... the commit message isn't re-built, so it keeps whatever was on the original PR.


Can you add some screenshots?

Your sample app not working with Swift 3 out of the box - maybe it is time to update it?
Also It will be very helpful if you can add some screenshots to project description.

I agree with this issue. Very important to show screenshots when you deal with UI


Maintain an aspect ratio for content within a window

This code maintains an aspect ratio for an area within a window. It also allows the developer to specify, in pixels, extra space which is not included within the aspect ratio. If you don't specify any extra width or height it will maintain the aspect ratio of the content view.
This was implemented for a video player. Our video player viewing area has an aspect ratio of 16/9 but we also have a 47 pixel control bar at the bottom of the video.
The code was written generically so that you can have extra width, height or both. It will never return a window size that is larger than the one the user specified via their resize action.
CAVEAT: There isn't a Windows implementation. The math used in the windowWillResize:toSize: method of the Mac window should be easily adaptable for use in a Windows implementation.
Related Issue: https://github.com/atom/electron/issues/2252

maintainContentAspectRatio is a bit too long as API name, how about setAspectRatio instead?
And can you also add document in docs/api/browser-window.md with example of usage?

@zcbenz I didn't use setAspectRatio because that is an available API on AppKit's NSWindow which acts somewhat differently than this implementation. If that isn't a problem then I think that setAspectRatio with the three arguments would be a fine name.
I will make the change you suggest and document it tomorrow.

@zcbenz I didn't use setAspectRatio because that is an available API on AppKit's NSWindow which acts somewhat differently than this implementation. If that isn't a problem then I think that setAspectRatio with the three arguments would be a fine name.

I think it is fine to use setAspectRatio as name, it is quite common that different toolkits behave differently in APIs with same names.

Excited for the ongoing work here folks!

This pull request should now be complete.

Thanks!


TerminalProfile: new TerminalProfile vs plain options object?

Testing #127194
The TerminalProfileProvider allows for returning either a raw options dict like:

Or constructing and returning an TerminalProfile object like:

This seems a bit confusing, traditionally when we have exported classes it's because the constructor does some work in managing a large number of field initializations, but here there's only the one, so why not just an interface, or alternatively unpacking the options field into it's constituant fields (via just extending TerminalOptions and TerminalExtensionOptions)

The classes also provide validation which was the main reason here:
https://github.com/Microsoft/vscode/blob/45eea1f87bbb1e98e78ec61268b6b36dcaaa56a3/src/vs/workbench/api/common/extHostTypes.ts#L1726-L1734
We could potentially build that validation out in the future as well. It was originally an interface but changed at Joh's request.


Read BSP connection details from .bsp directory in addition to buildServer.json

SourceKit-LSP currently looks for a BSP server config file in <workspaceRoot>/buildServer.json. BSP specifies that these config files should be located in <workspace-dir>/.bsp (among other locations). We should adopt the .bsp search locations.

Synced to Apple’s issue tracker as rdar://136370012


Issue #884: Ranch listener http terminated (master)

Malformated etag with If-Match and If-None-Match are ignored

This has resulted in a 400 since Cowboy 2.0. I think this is the right way to go since we can't evaluate the precondition and determine if it's true (continue) or false (412).
Closing, thanks!


Make internal console respect user's syntax highlighting theme

Problem Description
A part of #8068 .
Currently, the Internal Console and the Issue Report dialog use the default Spyder syntax highlighting theme, which wasn't as big of a problem in the past. However, now that Spyder is offering an application-wide dark theme, it looks odd to have these panes in a different color as the rest of the UI. As a stopgap, we could have them default to the Spyder Dark scheme if using the dark UI theme, but if we're going that far and it isn't much more effort, it would seem prudent to just use the proper theme to begin with.
e.g.

The error dialog stuff (only the description widget though) was fixed in PR #8081. The traceback part and the internal console will be fixed in another PR.


"Traian Vuia", "Sudriaș", "Jupani", "Săceni", "Surducu Mic" and "Susani" are listed in the drop-down menu

Description:
"Traian Vuia", "Sudriaș", "Jupani", "Săceni", "Surducu Mic" and "Susani" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Traian Vuia" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Traian Vuia, TIMIȘ (Traian Vuia)" and "Comuna Traian Vuia, TIMIȘ".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Sudriaș" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Sudriaș, TIMIȘ (Traian Vuia)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Jupani" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Jupani, TIMIȘ (Traian Vuia)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Type "Săceni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Săceni, TIMIȘ (Traian Vuia)".

Step 8
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 9
Type "Surducu Mic" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Surducu Mic, TIMIȘ (Traian Vuia)".

Step 11
Type "Susani" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Susani, TIMIȘ (Traian Vuia)".


Reuse vcn for Issues #171 and #163

OKE cluster can be created using existing VCN.

Hi, this feature will merged in master? I have checked but for now it's only a branch. Thanks

@hyder can you comment on vcn_reuse branch merge to master?
In my case I also need to run managed databases, but I do not like the idea of hard dependency between OKE VCNs and my database instances.

@hyder can you comment on reuse-vcn branch merge/feature release to master?
In my case I need to run managed databases, but I do not like the idea of hard dependency between OKE VCNs and my database instances.

Hi,
This was what we concluded before: https://github.com/oracle-terraform-modules/terraform-oci-oke/issues/163#issuecomment-638529445
Can you please elaborate on your issue/needs? Maybe we can revisit.

@hyder can you comment on reuse-vcn branch merge/feature release to master?
In my case I need to run managed databases, but I do not like the idea of hard dependency between OKE VCNs and my database instances.

Hi @litewhatever,
This was what we concluded before: #163 (comment)
Can you please elaborate on your issue/needs? Maybe we can revisit.

Our intention was to deploy VCN/network with terraform and then reuse it for OKE, managed databases and other managed services provided by OCI.


Add Type Hints to Modin Source Code

I'll start adding type hints to the source code starting from the re-write (#70)
Here will be my proposed approach:

Add python3 type things gradually: def func(x: int) -> float
Add mypy to CI checks
For python2, we will not use the #type x: int -> float comment workaround, instead when we distribute the package, we will strip away all the type hints using strip-hints so the code is python2 compatible.

Any comments and suggestions welcomed!

This would be great to add. I think the approach makes sense. Will you add it to #70 or will this be a separate commit after that gets merged?


Generate new PIR schema

Close EPF-191, EPF-194

Generate PIR with new schema

The new schema is stored at: here

Reuse component (graphs, nodes, inputs, outputs) names as their IDs

Sounds good
On Fri, Jul 15, 2022, 16:51 Aurick Qiao @.***> wrote:

@.**** commented on this pull request.
In pirlib/pir.py
https://github.com/petuum/pirlib/pull/2#discussion_r922581854:

fields: List[MetaDataSchemaField] = field(default=list)

def init(

   self,

   type: Optional[str] = None,

   name: Optional[str] = None,

   fields: List[MetaDataSchemaField] = []

):

   self.type = type

   self.name = name

   self.fields = fields

   self.fields.append(MetaDataSchemaField())

@.***
+class MetaData:
Got it. In that case schema should probably be an annotation for now.
—
Reply to this email directly, view it on GitHub
https://github.com/petuum/pirlib/pull/2#discussion_r922581854, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHUES6RBOAWTM2T7VKFND63VUH2RNANCNFSM53IS4AVQ
.
You are receiving this because you were assigned.Message ID:
@.***>


Hide caption fix

So, for some reason ccLoadPolicy doesnt work anymore(i mean you cant disable captions with "ccLoadPolicy") .so i found an alternative way to disable captions.
adding player.unloadModule('captions');  to ayp_youtube_player.html does the trick.
Heres a basic example that shows you how to disable captions:
public class CaptionDisableExample extends AppCompatActivity {

    private YouTubePlayerView youTubePlayerView;

    @Override
    protected void onCreate(@Nullable Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        youTubePlayerView = findViewById(R.id.youtube_player_view);

        getLifecycle().addObserver(youTubePlayerView);
        youTubePlayerView.addYouTubePlayerListener(new AbstractYouTubePlayerListener() {
            @Override
            public void onReady(@NonNull YouTubePlayer youTubePlayer) {
                String videoId = "yS1ibDImAYU";
                youTubePlayer.loadVideo(videoId, 0);
            }

            @Override
            public void onStateChange(@NonNull YouTubePlayer youTubePlayer, @NonNull PlayerConstants.PlayerState state) {
                if (state == PlayerConstants.PlayerState.PLAYING) {
                    youTubePlayer.killCC();
                }

            }

        });

    }
}

Hi, what issue are you trying to solve? every video plays with captions? can you show me an example of the problem?

Hi, what issue are you trying to solve? every video plays with captions? can you show me an example of the problem?

well current way to disable caption is using ccLoadPolicy but it doesnt work anymore.so i recommended alternative way

Can you send me a video where the issue happens?

Can you send me a video where the issue happens?

i dont think we need a video.my problem is i cant disable captions with current method.

Can you send me a video where the captions show up? so i can test what's going on?

@PierfrancescoSoffritti Done!

Can you send me a video where the captions show up? so i can test what's going on?

@PierfrancescoSoffritti
So as you see in the video every video i open starts with captions even though i set ccLoadPolicy = 1 to disable captions.Like this:
IFramePlayerOptions build = new IFramePlayerOptions.Builder().controls(0).rel(0).ccLoadPolicy(1).build();

But apperantly setting ccLoadPolicy to 1 doesn't disable captions anymore. Because it still shows captions. But my pull request fixes it
https://user-images.githubusercontent.com/59535990/166449902-e29ff321-74dd-4f86-b23a-495e27d8c6c6.mp4

ok, I understand. Thanks!
I will try everything out in the next few days (over the weekend at the latest) and merge the PR then :)

I'm looking at the IFrame player API, and it seems like the unloadModule method is not documented there. Which makes me think it's not a public API. Which if it's the case is concerning, since it means YouTube can break our library at any time.
How did you find out about the existence of the method?

@PierfrancescoSoffritti
You are right i didnt found this method from docs.
unloadModule is basically javascript object method that allows you to unload spesific js files.
For example player.unloadModule('captions') disables captions.js file.
(i mean whatever you type inside unloadModule it will disable that js file)
So if disabling js files without using docs methods is breaking Youtube terms then sorry for wasting your time.

The problem are not YouTube terms of service. The problem is that it's not good practice for us to rely on this, as it's not a public API and it would break our lib if YouTube changes something. For example if they rename the module to something else (eg: "captions" or "cc").
Here is my suggestion:
Let's replace the method in the interface, from unloadCaptions() to unloadModule(module: String)
But first I would like to see the documentation of the Javascript unloadModule API. Could you add the link to this discussion?


System.UnauthorizedAccessException when inserting

Today I found LiteDB and took some very basic tests:

I've used the Basic example from www.litedb.org
Put it into a 1000 loop.

What happens next will shock you 😉:
After several iterations (sometimes 28, sometimes 101, sometimes ...) I get an exception:

System.UnauthorizedAccessException
Access to MyData-journal.db was denied.

This is the complete source code of my .NET 4.7 console application:
namespace TestNoSQL
{
    using LiteDB;
    using System;
    using System.Diagnostics;
    using System.IO;
    using System.Reflection;

    // Basic example
    public class Customer
    {
        public int Id { get; set; }
        public string Name { get; set; }
        public string[] Phones { get; set; }
        public bool IsActive { get; set; }
    }

    internal static class Program
    {
        private static readonly string _path =
            Path.GetDirectoryName(Assembly.GetEntryAssembly().Location).TrimEnd('\\') +
            @"\MyData.db";

        private static void Main(string[] args)
        {
            const int count = 1000;

            var sw = new Stopwatch();
            sw.Start();
            for (var i = 0; i < count; ++i)
            {
                one();
            }
            sw.Stop();

            Console.WriteLine($@"{count} times '{nameof(one)}' took. {sw.Elapsed}.");
        }

        private static void one()
        {
            // Open database (or create if not exits)
            using (var db = new LiteDatabase(_path))
            {
                // Get customer collection
                var customers = db.GetCollection<Customer>("customers");

                // Create your new customer instance
                var customer = new Customer
                {
                    Name = "John Doe",
                    Phones = new string[] { "8000-0000", "9000-0000" },
                    IsActive = true
                };

                // Insert new customer document (Id will be auto-incremented)
                customers.Insert(customer);

                // Update a document inside a collection
                customer.Name = "Joana Doe";

                customers.Update(customer);
            }
        }
    }
}

My question:
Is this a design limitation or am I doing something significantly wrong?

Hi @UweKeim, try use v4 beta from Nuget/Master branch. You are using v3.x that I found some problems that I'm fixing for next version

It is working as expected with v4. Thanks a lot, David!
Even cross-threading seems to work perfectly:
Multiple connections              : 1000 times 'one' took. 00:00:05.4864547.
One connection                    : 1000 times 'one' took. 00:00:06.6925723.
One connection multi-threaded     : 1000 times 'one' took. 00:00:08.1019311.
Multiple connection multi-threaded: 1000 times 'one' took. 00:00:14.1637235.

FWIW, here is my updated full code:
namespace TestNoSQL
{
    using LiteDB;
    using System;
    using System.Diagnostics;
    using System.IO;
    using System.Reflection;
    using System.Threading;

    // Basic example
    public class Customer
    {
        public int Id { get; set; }
        public string Name { get; set; }
        public string[] Phones { get; set; }
        public bool IsActive { get; set; }
    }

    internal static class Program
    {
        private static readonly string _path =
            Path.GetDirectoryName(Assembly.GetEntryAssembly().Location).TrimEnd('\\') + @"\MyData.db";

        private static void Main(string[] args)
        {
            const int count = 1000;

            var sw = new Stopwatch();
            sw.Start();
            for (var i = 0; i < count; ++i)
            {
                one();
            }
            sw.Stop();

            Console.WriteLine($@"Multiple connections              : {count} times '{nameof(one)}' took. {sw.Elapsed}.");

            // --

            sw.Start();
            // Open database (or create if not exits)
            using (var db = new LiteDatabase(_path))
            {
                for (var i = 0; i < count; ++i)
                {
                    coreOne(db);
                }
            }
            sw.Stop();

            Console.WriteLine($@"One connection                    : {count} times '{nameof(one)}' took. {sw.Elapsed}.");

            // --

            long p = 0;

            sw.Start();
            // Open database (or create if not exits)
            using (var db = new LiteDatabase(_path))
            {
                for (var i = 0; i < count; ++i)
                {
                    var db1 = db;
                    ThreadPool.QueueUserWorkItem(delegate
                    {
                        coreOne(db1);
                        Interlocked.Increment(ref p);
                    });
                }

                while (Interlocked.Read(ref p) < count)
                {
                    Thread.Sleep(0);
                }
            }

            sw.Stop();

            Console.WriteLine($@"One connection multi-threaded     : {count} times '{nameof(one)}' took. {sw.Elapsed}.");

            // --

            p = 0;

            sw.Start();
            for (var i = 0; i < count; ++i)
            {
                ThreadPool.QueueUserWorkItem(delegate
                {
                    one();
                    Interlocked.Increment(ref p);
                });
            }

            while (Interlocked.Read(ref p) < count)
            {
                Thread.Sleep(0);
            }

            sw.Stop();

            Console.WriteLine($@"Multiple connection multi-threaded: {count} times '{nameof(one)}' took. {sw.Elapsed}.");
        }

        private static void one()
        {
            // Open database (or create if not exits)
            using (var db = new LiteDatabase(_path))
            {
                coreOne(db);
            }
        }

        private static void coreOne(LiteDatabase db)
        {
            // Get customer collection
            var customers = db.GetCollection<Customer>("customers");

            // Create your new customer instance
            var customer = new Customer
            {
                Name = "John Doe",
                Phones = new string[] { "8000-0000", "9000-0000" },
                IsActive = true
            };

            // Insert new customer document (Id will be auto-incremented)
            customers.Insert(customer);

            // Update a document inside a collection
            customer.Name = "Joana Doe";

            customers.Update(customer);

            // Index document using a document property
            customers.EnsureIndex(x => x.Name);

            // Use Linq to query documents
            var results = customers.Find(x => x.Name.StartsWith("Jo"));
        }
    }
}

The next question will be about your example in concurrency mode. v3 has some problems with some multi-thread/task but it's fixed in v4 (so far I have not had any problems of this type reported in this new version)

Thanks again, David. I'll stick to V4 with my further tests.
(I'm trying to compare similar operations with ADO.NET with MS Access now to get a feeling about speed)

One last addition, my comparison between MS Access and LiteDB:
Access of 1000 WRITE iterations took 00:00:38.3984094.
LiteDB of 1000 WRITE iterations took 00:00:02.2006439.

Access of 1000 READ iterations took 00:00:40.3999481.
LiteDB of 1000 READ iterations took 00:00:00.0111525.

This is based on the Basic example with the Customer class.

Huge difference between Access and LiteDB 😄
LiteDB is 100% managed .NET code, runs in-process. Read access are optimized to re-use cached block pages, so disk access are reduced too.


Finish Style Guide

As a community member, I want to see the style guide finished and not in it's current DRAFT Version.

currently working on some of the stuff.

will address this in fb-testing

new base colors to be added/changed:
// Colors
$dark: #2c3e50
$gray: #95a5a6
$light: #ecf0f1
// Variables for decoration
$brand-primary: #6cb4e4
$brand-success: #29b764
$brand-warning: #f1c40f
$brand-alert: #eb6f63
In addition, the tints (darken, lighten) are needed.

drop everything above. style-cheat.io is ready and online and should be the source of colors and styling.


ARROW-5504 [R]: move use_threads argument to global option

At this point the function is not exported or documented and threads are always used, users would need to set options(arrow.use_threads) to turn them off.

Codecov Report

Merging #4515 into master will decrease coverage by 11.86%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           master    #4515       +/-   ##
===========================================
- Coverage   88.29%   76.42%   -11.87%
===========================================
  Files         846       52      -794
  Lines      103682     1977   -101705
  Branches     1253        0     -1253
===========================================
- Hits        91541     1511    -90030
+ Misses      11894      466    -11428
+ Partials      247        0      -247

Impacted Files
Coverage Δ

r/R/R6.R
75% <ø> (ø)
:arrow_up:

r/R/ChunkedArray.R
100% <ø> (ø)
:arrow_up:

r/R/array.R
76% <ø> (ø)
:arrow_up:

r/R/read_table.R
100% <ø> (ø)
:arrow_up:

r/R/feather.R
58.33% <100%> (ø)
:arrow_up:

r/R/Table.R
86.66% <100%> (ø)
:arrow_up:

r/R/csv.R
95.45% <100%> (ø)
:arrow_up:

r/R/parquet.R
100% <100%> (ø)
:arrow_up:

r/R/arrow-package.R
100% <100%> (ø)

r/R/RecordBatch.R
96% <100%> (ø)
:arrow_up:

... and 797 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 f3fde09...0ce9c34. Read the comment docs.


chore: move over to Containerfile

Containerfiles are better for this project because it would be easier to contribute to, and having less abstrations makes it so knowing what the hell is in the image a lot easier, and we wouldnt need the jsonnet -> yaml/bluebuild -> containerfile dynamic.

:(

I'm interested to know how would Containerfile look at the end, to see what is the difference in implementation & structure.
I thought about it for doing this for Bluefin, which is possible imo, but Bluefin is not something I use daily, so I scratched that.

I assume the structure would look more like ubluemain or bluefin. IMO this is refactor: though.
Also keep in mind that you can keep using BlueBuild modules for stuff like Flatpaks if you decide to refactor, I can walk you through that if that's desireable.
We're of course also interested in specific frustrations you've had with BB (even if you believe that they can't be fixed) and what you hope to gain from a refactor.
I find this project pretty interesting as a creative myself, so I have taken a good look at the source code and config, and do have some thoughts on how I'd refactor it within the system.

Honestly using bluebuild has been amanzing! The only thing is that there are multiple layers of abstraction and that makes it so that refactoring stuff is a tiny bit confusing Im thinking about just moving everything over to apple pkl (including the modules) so that it might have a bit less layers that we need to work with (jsonnet + yaml -> containerfile vs pkl -> yml -> containerfile)

Honestly using bluebuild has been amanzing! The only thing is that there are multiple layers of abstraction and that makes it so that refactoring stuff is a tiny bit confusing Im thinking about just moving everything over to apple pkl (including the modules) so that it might have a bit less layers that we need to work with (jsonnet + yaml -> containerfile vs pkl -> yml -> containerfile)

If you get pkl working, let me know. I've been trying a bunch of stuff here, but haven't tried pkl yet due to finding the docs kind of hard to read, not finding the info I'm looking for, etc.

Pkl is pretty nice! Implemented stuff with b67f8c6. :+1:

Isn't this "not planned", and not "completed"?

Well, true! I guess this issue was just for moving to something other that raw yaml files


rate-limiting config.policy defaults to 'cluster' on DB-less mode

Summary
rate-limiting config.policy is only allowed to be set to 'local' or 'redis' with DB-less mode,  but policy default value is 'cluster', may be add a validator for policy, will be more user friendly
pseudocode examples:
local function validate_policy(value)
  if kong.db.strategy == "off" and value == "cluster" then
      return nil, "policy is only allowed to be set to 'local' or 'redis' with DB-less mode"
  end

  return true
end

Steps To Reproduce

configure kong

cp /etc/kong/kong.conf.default /etc/kong/kong.conf
sed -i "/^#database/i\database = off" /etc/kong/kong.conf
sed -i "/^#declarative_config/i\declarative_config = /tmp/apis.yaml" /etc/kong/kong.conf

configure routes&services&plugins

cat > /tmp/apis.yaml <<EOF
_format_version: "1.1"
services:
 - name: httpbin-service
   url: http://httpbin.org/
   routes:
   - name: httpbin-route
     paths:
     - /
plugins:
- name: rate-limiting
  config:
    minute: 3
EOF

start Kong
send a request

# curl  http://<IP_ADDRESS>:8000/get
{"message":"An unexpected error occurred"}

view logs/error.log

2020/04/26 09:29:25 [error] 1851#0: *339 lua coroutine: runtime error: ...are/lua/5.1/kong/plugins/rate-limiting/policies/init.lua:113: attempt to index local 'policy' (a nil value)
stack traceback:
coroutine 0:
        ...are/lua/5.1/kong/plugins/rate-limiting/policies/init.lua: in function 'usage'
        ...cal/share/lua/5.1/kong/plugins/rate-limiting/handler.lua:76: in function 'get_usage'
        ...cal/share/lua/5.1/kong/plugins/rate-limiting/handler.lua:125: in function <...cal/share/lua/5.1/kong/plugins/rate-limiting/handler.lua:108>
coroutine 1:
        [C]: in function 'resume'
        coroutine.wrap:21: in function <coroutine.wrap:21>
        /usr/local/share/lua/5.1/kong/init.lua:687: in function 'access'
        access_by_lua(nginx-kong.conf:91):2: in main chunk, client: <IP_ADDRESS>, server: kong, request: "GET /get HTTP/1.1", host: "<IP_ADDRESS>:8000"
2020/04/26 09:29:25 [error] 1851#0: *339 [kong] init.lua:689 [rate-limiting] ...are/lua/5.1/kong/plugins/rate-limiting/policies/init.lua:113: attempt to index local 'policy' (a nil value), client: <IP_ADDRESS>, server: kong, request: "GET /get HTTP/1.1", host: "<IP_ADDRESS>:8000"

Additional Details & Logs

Kong version 2.0.4
CentOS Linux release 7.6.1810 (Core)

This bugs was fixed by @bungle , see https://github.com/Kong/kong/commit/06787f8c07cd7750a636ab764221549a0de802dd .


Update stable version in repo from 0.8.0 to 0.8.1

Think about update stable version to 0.8.1
Version: stable=0.8.0 (50 days old) => latest=0.8.1 (15 days old)
Installs: stable=112 (56.57%), latest=16 (8.08%), total=198
Click to use developer portal
Click to edit
Do not close this issue manually as a new issue will be created if condition for update still exists.
Please drop a comment if any reason exists which blocks updating to version 0.8.1 at this time.
Note: This is an automatically generated message and not personally authored by bluefox!
@mcm1957 for evidence

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 0.8.1 but the current stable version is already 0.8.1.
So this issue should be closed.
@mcm1957 for evidence


neg methods and bugfix

Added __neg__ method to Series, DataFrame and Index.
Fixed a bug in Index.difference and added a relevant test.

There might be a better solution to the Index.difference problem, but nothing I tried seemed to satisfy both test cases other than getting rid of the T1 inside the list.


environment_check.sh: add checks for dmcrypt and dmsetup utilities

Which issue(s) this PR fixes:

Issue Longhorn/longhorn#8217
What this PR does / why we need it:
Special notes for your reviewer:
Additional documentation or context

@mergify backport v1.6.x


Chapter2/hello-buffer.py: incorrectly trimming the output of bpf_get_current_pid_tgid()

Hi, Hope you're having a wonderful day!
This is a quote form bpf-helpers man page:

u64 bpf_get_current_pid_tgid(void)
Return A 64-bit integer containing the current tgid and pid,  created   as   such:   current_task->tgid  <<  32  |  current_task->pid.

As you can see the top 32 bits contain tgid and the bottom 32 bits hold pid of the process.
In the book this is explained the other way around  on page 26. The foot note of page 26 also refers to the lower 32 bits as tgid which is incorrect.
Edit: As I continued reading the book I noticed the same situation with hello-buffer-config.py, hello-ring-buffer-config.py and page 61.

This is pretty confusing but my understanding is that what we typically call "process ID" in user space is actually the thread group ID in the kernel.
From the BCC reference for this helper function:

Returns the process ID in the lower 32 bits (kernel's view of the PID, which in user space is usually presented as the thread ID), and the thread group ID in the upper 32 bits (what user space often thinks of as the PID)

So, the footnote on p26 is wrong, and could explain this better, but (assuming the user is interested in the user space view of process ID) the code is correct. I'll need to do an errata to fix this in the book but for some reason can't log into the O'Reilly author portal right now, so I'll leave this issue to track it. But I think this means the PR in #36 is not needed


First Use Instructions

Hello,
Firstly, thanks for this amazing work of the cycada, as well as having the code shared, so we can start using it on our implementations.
I cloned the repo, and I am starting to use the network for my own problem with my own dataset.
Simple, I want to convert synthetic images to real world data. I have both datasets available, with the labels (semantic segmentation) of the synthetic data (domain A).
My question is, how to start with the CyCada, for a first start, I have to train a semantic network on my synthetic labeled images (domain A), for a semantic segmentation task, and then save this model, and start with the CycleGAN training adapted with the semantic loss (CyCada).
Is there a provided network that can can be trained for a semantic segmentation task ?, or are there provided instructions of how to start cycada with my own dataset ?
Thanks a lot

hello,
I meet the same problem, how can I train my own datasets which are including synthetic data and real world data. Thanks a lot

The same, how can I train my own datasets?

The same question

The same, how can I train my own datasets?
Thx!!!

Same question, I am struggling with the same


Error creating alert rule group

Terraform 1.2.3 and same issue with 1.4.5
on darwin_arm64

provider registry.terraform.io/fgouteroux/mimir v0.1.3

terraform module is working as expected until 04/03/2023
2023-04-17T16:22:40.588-0700 [TRACE] NodeAbstractResouceInstance.writeResourceInstanceState: removing state object for module.alerts_cloudSQL_dev.mimir_rule_group_alerting.CloudSQL_alerts["test_dev_alert"]
2023-04-17T16:22:40.588-0700 [TRACE] statemgr.Filesystem: not making a backup, because the new snapshot is identical to the old
2023-04-17T16:22:40.588-0700 [TRACE] statemgr.Filesystem: no state changes since last snapshot
2023-04-17T16:22:40.588-0700 [TRACE] statemgr.Filesystem: writing snapshot at terraform.tfstate
2023-04-17T16:22:40.594-0700 [ERROR] vertex "module.alerts_cloudSQL_dev.mimir_rule_group_alerting.CloudSQL_alerts["test_dev_alert"]" error: Cannot create alerting rule group 'CloudSqlAlerts_test_dev_alert' - unexpected response code '400': unable to decode rule group
2023-04-17T16:22:40.594-0700 [TRACE] vertex "module.alerts_cloudSQL_dev.mimir_rule_group_alerting.CloudSQL_alerts["test_dev_alert"]": visit complete, with errors
2023-04-17T16:22:40.594-0700 [TRACE] dag/walk: upstream of "module.alerts_cloudSQL_dev (close)" errored, so skipping
2023-04-17T16:22:40.594-0700 [TRACE] dag/walk: upstream of "provider["registry.terraform.io/fgouteroux/mimir"] (close)" errored, so skipping
2023-04-17T16:22:40.594-0700 [TRACE] dag/walk: upstream of "root" errored, so skipping
2023-04-17T16:22:40.594-0700 [TRACE] statemgr.Filesystem: not making a backup, because the new snapshot is identical to the old
2023-04-17T16:22:40.594-0700 [TRACE] statemgr.Filesystem: no state changes since last snapshot
2023-04-17T16:22:40.594-0700 [TRACE] statemgr.Filesystem: writing snapshot at terraform.tfstate
╷
│ Error: Cannot create alerting rule group 'testapp_dev' - unexpected response code '400': unable to decode rule group

I came here to report the same thing. This occurs for alerts with longer expressions and only for fgouteroux/mimir v0.1.3, so downgrade to 0.1.2 will fix it for you temporarily until the issue is fixed for good.
More details:
This will deploy fine for any version of the provider (including 0.1.3):
resource "mimir_rule_group_alerting" "testing1" {
  name      = "testing1"
  rule {
    alert = "testing1"
    expr = "xxxxxxxxxxxxxxxxxxxxxxxxxxxxx > 600"
    for  = "10m"
  }
}

while this will deploy fine for any version of the provider except 0.1.3:
resource "mimir_rule_group_alerting" "testing2" {
  name      = "testing2"
  rule {
    alert = "testing2"
    expr = "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx > 600"
    for  = "10m"
  }
}

╷
│ Error: Cannot create alerting rule group 'testing2' - unexpected response code '400': unable to decode rule group
│
│
│   with module.observability-mimir.module.alerts.mimir_rule_group_alerting.testing2,
│   on ../module-observability-mimir/Alerts/testing2.tf line 1, in resource "mimir_rule_group_alerting" "testing2":
│    1: resource "mimir_rule_group_alerting" "testing2" {
│
╵

I believe the reason is that for longer expressions the newest version of the provider tries to prettify them by breaking them into multi-line, which Mimir is not happy about. Probably some formatting issue, because in general Mimir should accept multi-line alerts (but I might be wrong here).
Anyway here's the difference in plans. Shorter expressions:
Terraform will perform the following actions:

  # module.observability-mimir.module.alerts.mimir_rule_group_alerting.testing1 will be created
  + resource "mimir_rule_group_alerting" "testing1" {
      + id        = (known after apply)
      + name      = "testing1"
      + namespace = "default"

      + rule {
          + alert = "testing1"
          + expr  = "xxxxxxxxxxxxxxxxxxxxxxxxxxxxx > 600"
          + for   = "10m"
        }
    }

Longer expressions:
  # module.observability-mimir.module.alerts.mimir_rule_group_alerting.testing2 will be created
  + resource "mimir_rule_group_alerting" "testing2" {
      + id        = (known after apply)
      + name      = "testing2"
      + namespace = "default"

      + rule {
          + alert = "testing2"
          + expr  = <<-EOT
                  xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
                >
                  600
            EOT
          + for   = "10m"
        }
    }

@rayuduc yes the version 0.1.3 introduce a bug caused by the prettifying promql expression.
@shybbko the begin of your analyse is good, with the prettifying promql expression it add new lines for longer expression, multi-lines strings is well accepted by mimir. The error comes from the spaces added at the begining of multine expression.
When request is yaml marshalled it add the pipe and number of space to preserve for indentation.
Example of http request from debug mode:
2023-04-24T10:46:11.191+0200 [INFO]  provider.terraform-provider-mimir_v0.1.3: 2023/04/24 10:46:11 REQUEST:
POST /prometheus/config/v1/rules/test HTTP/1.1
Host: <IP_ADDRESS>:8080
User-Agent: Go-http-client/1.1
Content-Length: 355
Content-Type: application/yaml
X-Scope-Orgid: test
Accept-Encoding: gzip

name: test3
rules:
    - alert: TestFGX3
      expr: |4-
          up == 0
        unless
          my_very_very_long_useless_metric_that_mean_nothing_but_necessary_to_check_that_test > 600
      for: 1m
      labels:
        priority: SEV-5
        severity: info
      annotations:
        description: testfgx 3
        summary: Summary for TestFGX3: timestamp=2023-04-24T10:46:11.191+0200
2023-04-24T10:46:11.192+0200 [INFO]  provider.terraform-provider-mimir_v0.1.3: 2023/04/24 10:46:11 RESPONSE:
HTTP/1.1 400 Bad Request
Content-Length: 28
Content-Type: text/plain; charset=utf-8
Date: Mon, 24 Apr 2023 08:46:11 GMT
Vary: Accept-Encoding
X-Content-Type-Options: nosniff

unable to decode rule group: timestamp=2023-04-24T10:46:11.192+0200
2023-04-24T10:46:11.194+0200 [ERROR] vertex "module.mimir_alerting_rule.mimir_rule_group_alerting.alert" error: Cannot update alerting rule group 'test3' - unexpected response code '400': unable to decode rule group

That I don't really understand is why there is 4 spaces instead of 2 by default. I made some tests with a custom encoder to force setting indentation to 2, but the code become more complex and could bring other unexpected issues. So I decided to just remove the spaces at the begining of expression.
Release v0.1.4 should fix this issue.

Thanks @fgouteroux !


Css issue with custom repo on mobile

Version of HACS
Latest

Describe the bug
On mobile the dropdown menu is really small and not aligned as well as the input for custom repo
Debug log

Add your logs here.

Fixed in https://github.com/custom-components/hacs/releases/tag/0.13.2


Enable download Git-LFS files on sync

Following an user issue, enable LFS on sync

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

What LFS files need to be downloaded for the build? AFAIK any image assets should be up on https://huggingface.co/datasets/huggingface/documentation-images, and I don't know if there are other binary files that would be part of the hub-docs. Holding off on approving, but otherwise this LGTM!

hi @NimaBoscarino , thanks for the comment. The changes is only for the GitHub Action example when a user wants to sync their GitHub repository with a Space, and the action need to explicitly enable sync LFS files in the repo.
An user, for example, had onnx models in their GitHub repo, and need to LFS push to our hub. It wasn't working without that option  lfs: true .

@radames Oh LOL, I should have looked at the file name 🤦 I thought it was a GitHub action for this repo, hahahaha.

@radames Oh LOL, I should have looked at the file name 🤦 I thought it was a GitHub action for this repo, hahahaha.
haha no worries, I was a bit confused but fun 😂


Tinynumpy not available on PyPI

Wade,
I'd like to use tinynumpy in another project, eppy. It would be nice to be able to import it rather than including the files in our project so that we can keep everything up to date with the least effort.
Let me know if anyone is planning on doing this already, or would be willing to take it up. Otherwise I could take a stab at it, if you're OK with that.
Cheers
Eric

Apologies for taking so long to reply (and to integrate your math changes).  I've never added a package to PyPI before; if you'd like to tackle it that would be wonderful.
I've given you commit access based on the quality of your other changes; feel free to add any manifest files or whatever is required by PyPI.

No worries, Wade. Thanks for your vote of confidence. I would be glad to take on adding tinynumpy to PyPI
Cheers
Eric

Wade,
The project is now available on PyPI @ https://pypi.python.org/pypi/tinynumpy/1.2.0.dev1.

Looks great, thanks for taking care of it!  If the format allows multiple authors, please add Almar Klein as well.

Done!--finally. ;-)

A

If anyone was having trouble installing from pip, try it again. I've made some changes and it should work now. Let me know if it doesn't.

Although the package was uploaded, it was not available to other projects on import. I'll have to give this one another try. If anyone has experience with this, help would be greatly appreciated at this point.
Cheers
Eric

I've successfully re-uploaded to PyPI, but I'm unable to get pip install to work. Can someone else give it a try, and let me know what happens?
Thanks
Eric


Developers should be able to bootstrap env and build agent tools Container image

There is a build-agent workflow that deploys an updated build-agent-tools image to the ACR deployed in bootstrapping. #33 resolves


Set more useful default queries in YASGUI

Load nicer and more useful default queries (with admin graph triples etc.) in the YASGUI tool.

Done with 1227940a7cb24d16495e54b61f9404247a7a3b33.


Enable ch_log() to take variable arguments

I think it is convenient and preferable (if there is no reason to avoid using variable arguments).
Ozaki Kiichi

Codecov Report

Merging #1919 into master will decrease coverage by <.01%.
The diff coverage is 72.72%.

@@            Coverage Diff             @@
##           master    #1919      +/-   ##
==========================================
- Coverage   75.09%   75.09%   -0.01%
==========================================
  Files          76       76
  Lines      125328   125296      -32
==========================================
- Hits        94112    94086      -26
+ Misses      31216    31210       -6

Impacted Files
Coverage Δ

src/message.c
68.35% <100%> (ø)
:arrow_up:

src/channel.c
83.73% <71.69%> (-0.01%)
:arrow_down:

src/version.c
76.76% <0%> (-3.54%)
:arrow_down:

src/gui_beval.c
43.96% <0%> (-0.93%)
:arrow_down:

src/if_py_both.h
76.01% <0%> (-0.01%)
:arrow_down:

src/term.c
52.99% <0%> (+0.05%)
:arrow_up:

src/window.c
81.39% <0%> (+0.06%)
:arrow_up:

src/ex_cmds.c
77.85% <0%> (+0.22%)
: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 e0ab979...56942e2. Read the comment docs.

Ozaki Kiichi wrote:

I think it is convenient and preferable (if there is no reason to
avoid using variable arguments).

Thanks!  I was thinking of doing this for a while.
--
DENNIS:  Listen -- strange women lying in ponds distributing swords is no
basis for a system of government.  Supreme executive power derives
from a mandate from the masses, not from some farcical aquatic
ceremony.
The Quest for the Holy Grail (Monty Python)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Update atmos. Update tests

what

Update atmos
Update tests

why

settings.spacelift.stack_name_pattern allows overriding Spacelift stack names. Supported tokens: {namespace}, {tenant}, {environment}, {stage}, {component}

components:
  terraform:
    "test/test-component-override-2":
      settings:
        spacelift:
          workspace_enabled: true
          # `stack_name_pattern` overrides Spacelift stack names
          # By specifying the following `stack_name_pattern`, the final Spacelift stack name will be
          # `tenant1-ue2-dev-new-component` instead of `tenant1-ue2-dev-test-test-component-override-2`
          # By default (if `stack_name_pattern` is not specified), the stack name is generated using the pattern {tenant}-{environment}-{stage}-{component}
          # Supported tokens: {namespace}, {tenant}, {environment}, {stage}, {component}
          stack_name_pattern: "{tenant}-{environment}-{stage}-new-component"

If the stack name pattern is specified incorrectly or not specific enough, an error like this will be thrown:
Duplicate Spacelift stack name 'tenant2-ue2-new-component' for component 'test/test-component-override-2' in the stack 'tenant2/ue2/prod'.
Check if the component name is correct and the Spacelift stack name pattern 'stack_name_pattern={tenant}-{environment}-new-component' is specific enough.
Did you specify the correct context tokens {namespace}, {tenant}, {environment}, {stage}, {component}?

When a stack name on the command line (e.g. atmos terraform plan test/test-component -s tenant1-ue2-dev) was the same as the name of the YAML config file in a subfolder inside the stacks folder, the stack was incorrectly detected (throwing an error "The stack 'xxx' does not exist") because it was considered as the folder path and not as a logical stack name (which is detected from the context)

references

https://github.com/cloudposse/atmos/pull/130

I don't see a test for the atmos command line bug (stack name matches file name in subfolder), and would like one to catch a regression, but otherwise this looks good.

such a test should be in atmos, I'll add it in the next PR


Update readme to use identifiers.

Fixes #5.

This PR leaves a lot of questions open about what the detailed semantics are (exactly how the static scoping works, and how this interacts with structured clone), but it reflects what I think the direction of this proposal is. Follow-on PRs can address those details.


Make it work with browserify

I had to make some changes to the code:

WebSockets in the browser use the websocket.onmessage = handler syntax (MDN) compared to the ws / EventEmitter syntax websocket.on('message', callback).

ws supports this method out of the box as well as websocket.addEventListener('message', callback) (ws#websocketaddeventlistenermethod-listener). Therefore I changed the ws.on() syntax for the peer and used ws.addEventListener() to make it work in the browser.
I also had to add this method to MessageSocket and used the ws implementation which is simply a wrapper for the EventEmitters ws.on() syntax (ws#addEventListener).

Changed the WebSocket constructor function from

sock = new NodeWebSocket(config.url, {
  protocol: 'jet'
});

to
sock = new WebSocket(config.url, 'jet');

That's because "The WebSocket constructor accepts one required and one optional parameter" (MDN). ws also supports this syntax. The docs are wrong but the code is right.

Rewrote peer/jsonrpc using OOP.

Added a small browserify example to the examples folder.

Added .jshintrc so editors stop complaining about syntax errors like global require function.

And last but no least ... all tests are green :)

Awesome work! :8ball:
Would be good make browserify exclude the daemon stuff, since browsers unfortunately cannot accept incoming connections (via websockets):

lib/daemon.js
lib/daemon/*


ENHANCEMENT Improved slug generation

Previously, the slug would only make 1000 attempts at generating a slug that wasn't already in use and if by then it had failed, it would throw an exception. Now, it'll attempt to find a valid slug indefinitely.

Codecov Report

Merging #3 into master will increase coverage by 0.66%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master       #3      +/-   ##
============================================
+ Coverage     42.97%   43.64%   +0.66%
  Complexity       94       94
============================================
  Files            11       11
  Lines           235      236       +1
============================================
+ Hits            101      103       +2
+ Misses          134      133       -1

Impacted Files
Coverage Δ
Complexity Δ

src/Parsers/Shortcode.php
100% <ø> (ø)
17 <0> (ø)
:arrow_down:

src/Models/CodeTab.php
50.98% <100%> (+2.98%)
21 <4> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Getting error using s3 material that region is not specified

When trying to fetch a material I am seeing this error:
Interaction with plugin with id 's3material' implementing 'package-repository' extension failed while requesting for 'latest-revision'. Reason: [Unable to find a region via the region provider chain. Must provide an explicit region in the builder or setup environment to supply a region.]
Looking on google it seems that the initialization of AmazonS3Client should be done like this now:
AmazonS3 client = AmazonS3Client.builder().withRegion("us-east-1").withPathStyleAccessEnabled(true).build();
Has anyone seen this error or found a solution?

Is the AWS_REGION set as mentioned in the docs?
http://oss.indix.com/gocd-s3-artifacts/installation.html

Thanks for responding.  Yes, indeed, we are setting the AWS_REGION in the environment variables set within the environment itself.  Anything else we can do to fix?

Hey, is this on the server jvm process you mean? Or  environment variables within GoCD?

In the environment variables within GoCD.  If I click on Environments on GoCD Server, and then the settings cog/wheel it will be set/shown along the right side.

That doesn't work for the material fetch since that happens in the server and not the agents. You need to set it in your shell environment / os / startup script etc.


Path/URL troubles with ui-router

We're using ui-router.
When accessing the app via a non-root page, e. g. http://foo.bar/baz instead of just http://foo.bar/, angular-dynamic-locale tries to load the locale file from http://foo.bar/baz/angular/... which obviously results in a 404.
Is there a way to resolve this issue?

Is there a way to customize the path?

tmhDynamicLocaleProvider.localeLocationPattern('http://baz.com/angular/i18n/angular-locale_{{locale}}.js') should do it

Or just tmhDynamicLocaleProvider.localeLocationPattern('/angular/i18n/angular-locale_{{locale}}.js')

Thank you, kind sir!


Ontology reverse id label yaml config

Closes #313
Closes #319
Closes #320

#313 -> switch_label_and_id can now also be configured via the yaml config
#319 -> remove the assignment to new objects (now only one netwokx graph object)
#320 -> improve error messages (log defined node and possible nodes in the ontology)

Good spotted point. I have acted upon it and it should be ready now. @slobentanzer


Update to version 3 from upstream

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 3 committers have signed the CLA.:x: crazy-max:x: tonistiigi:x: temenuzhka-thedeYou have signed the CLA already but the status is still pending? Let us recheck it.


very long waiting time for VOB

def add_VOB(fig):

    start_time = time.time()
    ob_data = smc.vob(df)
    volumized_ob_time = time.time() - start_time
    logger.debug(
        "volumized_ob processing time (for ohlc length={}): {:.2f} seconds",
        len(ob_data),
        volumized_ob_time,
    )

2024-02-23 12:48:40.440 | DEBUG    | __main__:add_VOB:194 - volumized_ob processing time (for ohlc length=1000): 5.01 seconds

pr
https://github.com/joshyattridge/smartmoneyconcepts/pull/9

2024-02-27 23:23:02.913 | DEBUG    | __main__:add_VOB:194 - volumized_ob processing time (for ohlc length=1000): 1.64 seconds


Cannot build index.d.ts file due to an invalid export syntax

Describe the bug
Since the option esModuleInterop: true is not set to the project, the file index.d.ts cannot be compiled. This happens because the module definition of the file uses an invalid export syntax, when it should use the export= syntax type. This is described better on this documentation: Module.d.ts
Note that using export default in your .d.ts files requires [esModuleInterop: true](https://www.typescriptlang.org/tsconfig#esModuleInterop) to work. If you can’t have esModuleInterop: true in your project, such as when you’re submitting a PR to Definitely Typed, you’ll have to use the export= syntax instead

File path: https://github.com/asyncapi/spec-json-schemas/blob/master/index.d.ts
How to Reproduce
npm run build:prod (tested in local environment only)
Link to GitHub repository with project that has issues:
https://github.com/asyncapi/server-api
Expected behavior
After running the command, it should finish without any build errors.

sure @magicmatatjahu. Let me solve it and I will open a PR

https://github.com/asyncapi/spec-json-schemas/pull/288


[Task]: CMD+Click Nested objects

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
Nested objects:-
Allow a user to move directly to a JS function rather than just go to a javascript
Allow a user to move to a widget in the structure of form1.data.  or list1.data.

Duplicate of this issue https://github.com/appsmithorg/appsmith/issues/18636


Improve mobile layout

[x] Make interface responsive
[x] Redo the game grid
[x] Allow inviting friends (eg. in the Play tab)
[x] Make modals fullscreen on mobile
[x] Redo the welcome screen
[ ] Improve Account page

Images:

#282 Fixes the welcome screen, unless you speak about the screen with "play with a friend" and "play with a bot" (which can go in another issue).
#279 Fixes the account page, an issue #283 cover what's left to be done.


fix(treelist): Downgraded angular-tree-component for stability

Downgrading angular-tree-component to 2.7.0 to avoid recent issues with the Angular 4 upgrade.
PRs for platform and runtime:
https://github.com/fabric8io/fabric8-ui/pull/675
https://github.com/fabric8-ui/fabric8-runtime-console/pull/274

@dlabrecq snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1589.int.rdu2c.fabric8.io


aws-sso-cli: 1.14.0 -> 1.14.1

Description of changes
https://github.com/synfinatic/aws-sso-cli/releases/tag/v1.14.1
Switched to buildGo121Module as upstream go.mod requires it.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

@ofborg eval

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2800

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1186

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2889


plugin does not work when using IntelliJ 2021.1.3

Hi all,
gerrit plugin is failed since I upgrade IntelliJ to 2021.1.3
here is my push (review) log.
any ideas?
15:28:26.399: [lightspeed-operation-center] git -c credential.helper= -c core.quotepath=false -c log.showSignature=false push --progress --porcelain gerrit refs/heads/feature/deliveryFee_threshold:refs/for/feature/deliveryFee_threshold%r=rwu
Enumerating objects: 47, done.
Counting objects: 100% (47/47), done.
Delta compression using up to 16 threads
Compressing objects:   4% (1/21)
Compressing objects: 100% (21/21), done.
Writing objects:   4% (1/24)
Writing objects: 100% (24/24)
Writing objects: 100% (24/24), 3.50 KiB | 70.00 KiB/s, done.
Total 24 (delta 17), reused 0 (delta 0), pack-reused 0
remote: Resolving deltas:   5% (1/17)
remote: Resolving deltas: 100% (17/17)
remote:
remote: Processing changes: refs: 1
remote: Processing changes: refs: 1, done
remote: commit f13daeb: warning: subject >50 characters; use shorter first paragraph
error: failed to push some refs to 'ssh://gerrit.coupang.net:29418/lightspeed-operation-center.git'
To ssh://gerrit.coupang.net:29418/lightspeed-operation-center.git
!	refs/heads/feature/deliveryFee_threshold:refs/for/feature/deliveryFee_threshold%r=rwu	[remote rejected] (change http://gerrit.coupang.net/c/lightspeed-operation-center/+/49064 closed)
Done
15:29:03.205: [lightspeed-operation-center] git -c credential.helper= -c core.quotepath=false -c log.showSignature=false add --ignore-errors -A -f -- client/src/utils/messages_en.js
15:29:03.230: [lightspeed-operation-center] git -c credential.helper= -c core.quotepath=false -c log.showSignature=false commit -F /private/var/folders/9w/0991rn111298lwfmzq0pjq6jftfsh6/T/git-commit-msg-.txt --amend --
[feature/deliveryFee_threshold d05b1e59] [TBR][LTSPD-5079]ui to update store-1
 Date: Tue Jan 11 15:01:56 2022 +0800
 8 files changed, 88 insertions(+), 24 deletions(-)
15:29:19.357: [lightspeed-operation-center] git -c credential.helper= -c core.quotepath=false -c log.showSignature=false push --progress --porcelain gerrit refs/heads/feature/deliveryFee_threshold:refs/for/feature/deliveryFee_threshold%r=rwu
Enumerating objects: 47, done.
Counting objects:   2% (1/47)
Counting objects: 100% (47/47), done.
Delta compression using up to 16 threads
Compressing objects:   4% (1/21)
Compressing objects: 100% (21/21), done.
Writing objects: 100% (24/24), 3.48 KiB | 68.00 KiB/s, done.
Total 24 (delta 17), reused 0 (delta 0), pack-reused 0
remote: Resolving deltas: 100% (17/17)
remote:
remote: Processing changes: refs: 1
remote: Processing changes: refs: 1, done
error: failed to push some refs to 'ssh://gerrit.coupang.net:29418/lightspeed-operation-center.git'
To ssh://gerrit.coupang.net:29418/lightspeed-operation-center.git
!	refs/heads/feature/deliveryFee_threshold:refs/for/feature/deliveryFee_threshold%r=rwu	[remote rejected] (change http://gerrit.coupang.net/c/lightspeed-operation-center/+/49064 closed)
Done
15:30:33.020: [lightspeed-operation-center] git -c credential.helper= -c core.quotepath=false -c log.showSignature=false add --ignore-errors -A -f -- client/src/utils/messages_en.js
15:30:33.044: [lightspeed-operation-center] git -c credential.helper= -c core.quotepath=false -c log.showSignature=false commit -F /private/var/folders/9w/0991rn111298lwfmzq0pjq6jftfsh6/T/git-commit-msg-.txt --amend --
[feature/deliveryFee_threshold 6e726dd2] [LTSPD-5079]ui to update store-1
 Date: Tue Jan 11 15:01:56 2022 +0800
 8 files changed, 88 insertions(+), 24 deletions(-)
15:31:01.595: [lightspeed-operation-center] git -c credential.helper= -c core.quotepath=false -c log.showSignature=false push --progress --porcelain gerrit refs/heads/feature/deliveryFee_threshold:refs/for/feature/deliveryFee_threshold%r=rwu
Enumerating objects: 47, done.

remote: Processing changes: refs: 1, done
error: failed to push some refs to 'ssh://gerrit.coupang.net:29418/lightspeed-operation-center.git'
To ssh://gerrit.coupang.net:29418/lightspeed-operation-center.git
!	refs/heads/feature/deliveryFee_threshold:refs/for/feature/deliveryFee_threshold%r=rwu	[remote rejected] (change http://gerrit.coupang.net/c/lightspeed-operation-center/+/49064 closed)
Done

This looks like a usage error, see this error why Gerrit has rejected this change:
remote rejected] (change http://...c/lightspeed-operation-center/+/49064 closed)
Looks like the referenced change has been merged / closed in Gerrit.
In case my assumption is wrong, please re-open this issue.

oh, it's my mistake. thank you!


[Feature-6758][Task] Add limit resource usage for tasks base on cgroup

Purpose of the pull request
close #6758
Brief change log
Add task resource usage limit base on systemd and cgroup.
For more information you can see https://www.freedesktop.org/software/systemd/man/systemd-run.html and https://www.freedesktop.org/software/systemd/man/systemd.resource-control.html#.
Due to CGroup only exists in Linux, the current resource limit only supports Linux. I think this is enough for now since big data scheduling tasks only run in Linux.

Verify this pull request
I've test successfully in Ubuntu 16.04 and Ubuntu 18.04 and Ubuntu 20.04 and Ubuntu 22.04 and Centos7.

Codecov Report

Merging #10373 (cc7d314) into dev (36bba3f) will decrease coverage by 0.01%.
The diff coverage is 32.78%.

@@             Coverage Diff              @@
##                dev   #10373      +/-   ##
============================================
- Coverage     40.57%   40.56%   -0.02%
- Complexity     4741     4752      +11
============================================
  Files           869      870       +1
  Lines         35286    35344      +58
  Branches       3935     3940       +5
============================================
+ Hits          14318    14336      +18
- Misses        19538    19575      +37
- Partials       1430     1433       +3

Impacted Files
Coverage Δ

...apache/dolphinscheduler/common/model/TaskNode.java
24.03% <0.00%> (-1.18%)
:arrow_down:

...dolphinscheduler/dao/entity/TaskDefinitionLog.java
19.04% <0.00%> (-0.96%)
:arrow_down:

...er/server/master/runner/WorkflowExecuteThread.java
7.93% <0.00%> (-0.03%)
:arrow_down:

...duler/plugin/task/api/AbstractCommandExecutor.java
0.00% <0.00%> (ø)

...sk/api/utils/AbstractCommandExecutorConstants.java
0.00% <0.00%> (ø)

...olphinscheduler/plugin/task/api/utils/OSUtils.java
0.00% <0.00%> (ø)

...he/dolphinscheduler/dao/entity/TaskDefinition.java
71.65% <50.00%> (-1.46%)
:arrow_down:

...ache/dolphinscheduler/dao/entity/TaskInstance.java
65.69% <100.00%> (+1.23%)
:arrow_up:

...er/server/builder/TaskExecutionContextBuilder.java
70.21% <100.00%> (+1.32%)
:arrow_up:

...nscheduler/service/process/ProcessServiceImpl.java
30.73% <100.00%> (+0.11%)
:arrow_up:

... and 3 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 36bba3f...cc7d314. Read the comment docs.

merging……


Fix "Add" profile form button position and text box styling

It needs to be align to the right below the text box:

Also change the style for the text box:
width: 444px;
height: 115px;
background-color: #ffffff;
background-image: none;
border: 1px solid #dce4ec;
border-radius: 4px;
-webkit-box-shadow: inset 0 1px 1px rgba(0, 0, 0, 0.075);
box-shadow: inset 0 1px 1px rgba(0, 0, 0, 0.075);
-webkit-transition: border-color ease-in-out 0.15s, box-shadow ease-in-out 0.15s;
transition: border-color ease-in-out 0.15s, box-shadow ease-in-out 0.15s;

Hi Luis,
I worked on changing the color and floating the bottom to the right.
http://www.seattletoursinspanish.com/gigcentral/profiles/add
I was thinking on leaving that right there so that I don't interfered on Turner's re do of the recapcha form. What are you thoughts ? Should I market this like fix and move on to something else ? Or should I get the form to behave responsively on mobile devices ? I want to double check before my first push!


Using require from within jsdom?

I have some code that runs within the jsdom context, specifically I pass it into the src option of jsdom.env(). This code needs to require a module from my node-modules folder, but running it in jsdom errors saying the require function is undefined. Any ideas how I can require modules in this scenario?
PS the module in question is mongoose.

you'd need to load mongoose the same way you would in the browser.  This is can be done by injecting a script tag with the src pointed at file:///path/to/file.js.  Perhaps another option is to use substack's browserify, but I have not tried it in the window context of jsdom (works great on the browser though!)

I am using browserify actually, and it works great within jsdom, but I dont want to put mongoose in it as I only need mongoose on the server side.
The thing i'm trying to do is override the Backbone.sync method on the server only, i.e. get it to talk to my mongoose models directly rather than firing off ajax calls via jQuery within JSDOM!


Clicking a links in an event descriptions doesn't open the link

Nor can I right-click to copy the URL.

This is because links in the event descriptions have a target="_blank" but do NOT include the rel attribute with rel="noopener noreferrer". So the URL for the electron new-window event is effectively about:blank.
You can test this by adding wc.openDevTools() to main.js and adding the rel attribute through dev tools, then clicking the link.
A proposed is to automatically run wc.executeJavaScript(document.querySelectorAll('a[target=_blank]').forEach(node => node.setAttribute('rel', 'noopener noreferrer'))) on DOM change.

Any updates on this issue?


Mojo::Base::with_roles patch

Summary
Add with_roles method to Mojo::Base, per the specs described here:
https://irclog.perlgeek.de/mojo/2017-08-08#i_14984146
Motivation
Allow multiple developers to extend key Mojolicious classes (Test::Mojo, Mojo::UserAgent, etc)
using Role::Tiny (optional dependency)
References
https://irclog.perlgeek.de/mojo/2017-08-08#i_14984146

Don't forget to squash your commits.


Little changes and tried to add bot status(note: status code was NOT made by me)

Btw status code might not work, as I said code is not mine and I don't know if it is compatible with Seb's boilerplate.

NOTE THAT THE FINAL VERSION IS THIS ONE: https://github.com/al5ina5/discord-joie-bot/pull/18/commits/a094b9095652c6908369aaba2c96acba5e83a24b

Perfect! Pulling it. I'll probably remove data/bot-status.js file and stick it into bot.js.
Btw, nothing wrong with reusing code. As most devs say, "no point in reinventing the wheel."


How do I to make 3 cards?

Hi thanks for sharing , how do I to make 3 cards like tinder, ? I tried but did not worked
Thanks

@frank0ch0a can you give me more details on how you implemented 3 cards. Thanks in advance.


Fix app responsiveness

[Finishes #114835413]

Add preview for documents
Update edit profile page
Fix login/signup responsiveness

Current coverage is 84.84%

Merging #12 into master will not affect coverage as of 9199114

@@            master     #12   diff @@
======================================
  Files           22      22
  Stmts          838     838
  Branches        88      88
  Methods          0       0
======================================
  Hit            711     711
  Partial          2       2
  Missed         125     125

Review entire Coverage Diff as of 9199114

Powered by Codecov. Updated on successful CI builds.


No support for password_validation_policy in cloudsql-instance

Describe the bug
There is no current support in the cloudsql-instance module for setting the password_validation_policy, which means the new database shows up in the console with some security warnings out  of the box:

This is supported in the upstream module
https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/sql_database_instance
And reapply the module (after manually making a change) will revert:
  ~ resource "google_sql_database_instance" "primary" {
        id                             = "sjr-db"
        name                           = "sjr-db"
        # (15 unchanged attributes hidden)

      ~ settings {
            # (15 unchanged attributes hidden)
          - password_validation_policy {
              - complexity                  = "COMPLEXITY_DEFAULT" -> null
              - disallow_username_substring = true -> null
              - enable_password_policy      = true -> null
              - min_length                  = 10 -> null
              - reuse_interval              = 0 -> null
            }

Environment
❯ terraform -version
OpenTofu v1.8.5
on darwin_amd64
+ provider registry.opentofu.org/hashicorp/google v6.12.0
+ provider registry.opentofu.org/hashicorp/google-beta v6.12.0
+ provider registry.opentofu.org/hashicorp/random v3.6.3
❯ git rev-parse --short HEAD
6127efb3

To Reproduce
Create a DB using any settings, go to console, see the three warnings above.  (I'm going to try and deal with those next)
Result
Warnings showing up in GCP console

Happy to raise a PR if you can let me know how you'd like it done? Copying a bunch of password_validation_policy from the upstream provider feels like creating a bunch config that's a pain to maintain...
https://github.com/hashicorp/terraform-provider-google/blob/main/google/services/sql/resource_sql_database_instance.go#L650

We have a few pointers in our contributing guide, for something like this we typically

bundle related attributes into a top-level variable
strings that actually behave like enums (complexity?) are enforced via variable validation
optional attributes have a default null value and we leave the provider/API infer a default when needed

If you have the time for a PR don't overthink it too much, we'll review and discuss things together. And thanks of course!

@lyricnz I have added support for password policy settings to the module, but the resource/API documentation is not exhaustive, CloudSQL is not a product I typically use and I don't have the time to bring up a few instances and test this now.
Would you have time to check if

the policy needs to be also be set on replicas
all arguments really are mandatory (just try setting some of them as nulls)
the default_complexity argument works (it's an enum in the API, but the provider docs don't explicitly say it)
the policy generally works as you would expect

Once the above is done we can merge the change. Thanks!

Thanks man! We'll try that this week!

(sorry for the delay).  Yes, applying the configuration applied the changes to the DB, but it's still showing the warning about no password policy. I think this refreshes periodically, so will check again in 24h

Great, thanks for confirming! I'll wait before marking this close then. :)

ach, it's closed already, my bad :) well then reopen please if it turns out we did not fix it


Yap Specialized Routing

Spanish meeting helpline in North Atlanta Area of Georgia Region having several issues.  First, each caller is presented with English and Spanish as choices.  But, when Spanish is chosen, if I am looking in Twilio at call logs and refreshing, Yap dials a mix of English and Spanish speaking volunteers, regardless of the fact that each group is carefully setup just to answer 1 language.  Also, Yap never calls all 6 volunteers on Spanish list but stops after 3-4 and leaves the caller in a perpetual muzak with no option to voicemail even though Cycle Once, Then Voicemail is selected and an email address set.

Have you tagged the volunteers with the correct language?

Yes. J
On Sun, Nov 10, 2019 at 3:04 PM Danny Gershman<EMAIL_ADDRESS>wrote:

Have you tagged the volunteers with the correct language?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bmlt-enabled/yap/issues/325?email_source=notifications&email_token=AF6DMB2XFURC3M5HQGUPOATQTBSNDA5CNFSM4JLJ7RO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDVFNWY#issuecomment-552228571,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF6DMB5KUB2SP2FXKEJL6YDQTBSNDANCNFSM4JLJ7ROQ
.

This is definitely a bug.  Fix will be in yap 3.5.0

The fix for this is in master.

will be in yap 3.5.0


narwhals = Pandas + polars + ...

Use https://github.com/narwhals-dev/narwhals to support pandas and polars!
This seems to be a very cool alternative to support various backends. See for example https://github.com/koaning/scikit-lego/pull/671

Hey, just wanted to stop by and say - thanks for your interest! Feel free to book some time on https://calendly.com/marcogorelli if you'd like to chat about how Narwhals could help PyFixest

Hi both (@MarcoGorelli and @juanitorduz) - I've now thought about it for 15 minutes and I think narwhals might be a great solution for PyFixest! Thanks for offering to chat @MarcoGorelli , I'll book an appointment =)
Just some background on pyfixest and how it works with Data Frames: most of the data manipulation happens via the formulaic library, which requires an input pd.DataFrame. I.e. a usual flow looks like this:
%load_ext autoreload
%autoreload 2

import polars as pl
import pandas as pd
import pyfixest as pf

from formulaic import model_matrix
import narwhals as nw

data = pl.DataFrame(pf.get_data())

def feols(data):

    if isinstance(data, pl.DataFrame):
        data = data.to_pandas()

    # model_matrix requires a pandas DataFrame and returns a pandas DataFrame
    Y, X = model_matrix("Y ~ X1", data = data, output = "pandas")

    # some more pandas manipulations
    Y.dropna(inplace = True)
    X.dropna(inplace = True)

    return Y.to_numpy(), X.to_numpy()

Via narwhals, it could look as
def feols_nw(data, use_polars = False):

    data = nw.from_native(data)

    # model_matrix requires a pandas DataFrame and returns a pandas DataFrame
    Y, X = model_matrix("Y ~ X1", data = data.to_pandas(), output = "pandas")

    if use_polars:
        # another copy? potentially costly?
        Y = nw.from_native(Y)
        X = nw.from_native(X)

    # some more pandas manipulations
    Y.dropna(inplace = True)
    X.dropna(inplace = True)

    return Y.to_numpy(), X.to_numpy()

Hey! Thanks for your explanation - if formulaic requires specifically pandas input/output, and then that might be a good candidate for Narwhalification :) I'll take a look, thanks!

    # another copy? potentially costly?
   Y = nw.from_native(Y)

Just to clarify, from_native just wraps a dataframe in a narwhals.DataFrame - it's a virtually free operation, only takes a few microseconds, and doesn't do any copies - Narwhals only translates syntax

Naive question: It seems formulaic supports pyarrow.Table. Could this be a shortcut for Polars integration? https://docs.pola.rs/api/python/stable/reference/dataframe/api/polars.DataFrame.to_arrow.html

totally!

Ciao! I am not sure if I have the time, but I'd be glad to support this narwhalification (h/t @fbruzzesi too).

That would be amazing @baggiponte! One key step for this would be this PR  to be merged into formulaic, and then we'd mostly have to port code from pandas to narwhals in the model_matrix function =)

There's a nice new vignette  in the narwhals docs on how to handle data frame conversion via narwhals without having to rewrite the entire pandas backend. Maybe we could start by using narwhals to handle the pandas to polars conversion (if users supply polars data frames)? If we additionally rewrite the methods in the FeolsCompressed class from polars to narwhals, we could then drop the polars and pyarrow dependencies.


Readme link update

Updated the "Getting Started" link associated with the "Getting started with MRTK" image
Updated the Release Assets link to point to MRTK 2.4 release assets instead of 2.0 release assets.
Overview
As part of the MRTK Github.io docs changes, there's a new getting started page for MRTK. The URL was updated to reflect this.
Changes
Changed Getting started link to point to https://microsoft.github.io/MixedRealityToolkit-Unity/Documentation/WelcomeToMRTK.html
Release Assets now points to: https://github.com/microsoft/MixedRealityToolkit-Unity/releases/tag/v2.4.0

@polar-kev Should this go into the 2.5 release branch?

@keveleigh yes, these changes should go into the 2.5 prerelease branch as well. Right now the dev branch shows by default on the MRTK github page which is why I pointed these changes there initially.

/azp run mrtk_docs

@polar-kev Looks like a merge conflict :/

@keveleigh resolved

/azp run mrtk_docs


Update 1.04, broken increased animation distance

Hi,
As said on the title, after the latest update (1.04) the arg
--increase-animation-distance
doesn't seem to work.

Hello!
That is strange, to me it seems to be working.
Could you show how you tested it?
Best Regards,
João Pedro Braz

Hi, sure!
Just an example, but after the 1.04 updatea I have seen many cases similar to this one.
Sorry for the scarce quality of the material, anyway...
Map position:

Look at the turtle at the top left (choppy animation) compared to that of the plant at the top center of the image:

Video:
https://youtu.be/EI7sX5lxS_Y
Thanks in advance for any effort and thanks for the awesome mod!

Hello!
I'll look into it, but just wanted to say that I'm not the creator of the mod, just some guy trying to help, thank @gurrgur for his awesome work :)
Best Regards,
João Pedro Braz

You are both my heroes 👌

Hello!
Yeah, just got the problem to "work". Thanks for pointing it out.
Best Regards,
João Pedro Braz


[ZEPPELIN-2257] notification about incompleteness of data

What is this PR for?
Added notification about exceeding the limit in the result.
What type of PR is it?
Improvement
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2257
How should this be tested?

Create table test with more than 2 records (if not exists)
Set parameter common.max_count  = 2
Execute

%jdbc
select *from test

You should see message about exceeds limit
Screenshots (if appropriate)

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

LGTM

Merge to master and branch-0.7 if no further discussions.

Sorry for late reply.

regarding to design, it would be great if we use bootstrap-alert. (warning level would be ok)

For example,

ok.

LGTM

CI green https://travis-ci.org/tinkoff-dwh/zeppelin/builds/211619211

@tinkoff-dwh Then you can make this CI status green by reopening-closing this PR :)

Just saw it, this definitely looks better. I don't have a strong opinion, I just saw there is a small diff, hence, highlighting it.
This is how it gets displayed in spark/livy, with some extra information.

And this is how it will look like if there were multiple queries.

In addition to @prabhjyotsingh mentions, there are another type of message controlled by ZEPPELIN_INTERPRETER_OUTPUT_LIMIT env variable.

All three messages Color, location, Decorations are different.
Basically these three messages are saying the same thing "Not a complete data".
So it make sense to display all three message in similar way, i think.

@Leemoonsoo
what final version, bootstrap?

Message location
Because of existing message Results are limited by X.  and Output exceeds 10240. Truncated. are displayed below the table.
I think showing Attention! Result is incomplete ... below the table helps keep user experience more consistent.
Message style
Bootstrap decoration is pretty. If we apply bootstrap alert to all 3 messages, then that means existing 2 message ux is changed, too.
From taking less height of the screen, receiving less attention,
To taking more height of the screen, receiving more attention.
So which is the way we should go do you guys think? @tinkoff-dwh @1ambda
Message text
All three message are different.
Results are limited by X.
Output exceeds 10240. Truncated.
Attention! Result is incomplete ...
It'll be better use similar form of message. But how they're limited are all different. And it'll be great if we user give some link how they're limited.
So for example,
Output is truncated to X rows. Learn more about common.max_count
Output is truncated to 10240 bytes. learn more about  ZEPPELIN_INTERPRETER_OUTPUT_LIMIT
What do you think?

@Leemoonsoo
location
I think it's better to display above table because  the output can be large and  user will not see message.
style
Bootstreap attracts attention and this is important
text
Output is truncated to {count} rows. Learn more about {variable_name}
sorry for offtopic. @Leemoonsoo can you watch https://github.com/apache/zeppelin/pull/2085

CI green https://travis-ci.org/tinkoff-dwh/zeppelin/builds/213063178

@Leemoonsoo
I think now is fine (data table + message separated from footer), but I can increase the top margin

@tinkoff-dwh thanks!
It's nit, but bottom margin is bit much compare to other margins in paragraph. i tried mark other margins.

Of course there's no rule that all the margin should be the same, but the margin on bottom looks bit inconsistent. Message is already decorated by bootstrap box so it already gives good visual separation.
What do you think?

@Leemoonsoo
possible. This margin within HTML Message maybe there are problems in other messages (without bootstrap). I'll try.

@tinkoff-dwh Ah, i see. i didn't see where this margin comes from.

@AhyoungRyu
if you configure the note and working only in report mode then you will not see when the report is incomplete.I can add the close button

@AhyoungRyu
if you configure the note and working only in report mode then you will not see when the report is incomplete.I can add the close button

I can add the close button

@tinkoff-dwh  Yeah that's better. Having close button will be better for other cases as well.

@tinkoff-dwh Thanks! Looks good :)

@Leemoonsoo
Yes you were right, this margin from alert (bootstreap class). fix it

travis does not work (build was not created for last commit)

ready to review

LGTM

LGTM and merge to master if no further discussions.
Thanks @tinkoff-dwh for the improvement!


Revise definitions of Module, VectorSpace

sm should be Proper: Its impropriety currently prevents the development of proofs that should be trivial from the Module laws (see https://github.com/c-corn/corn/issues/30).

I also redefine vector spaces as modules over fields, so that the relationships between these structures are more clear, and we avoid unnecessary duplication of the laws

Note that the CI build for this PR is failing because it is not based off of my branch for #13, and so doesn't include the required .travis.yml file.

Looks good, modulo the Proper.
Also, we can now test Travis

@spitters Thanks for the feedback, I've done as you suggested and it works with my toy lemma discussed in that issue!


Cleanup filtering

We are no longer filtering messages on get, filtering them on set instead.

I might add a spec for it as well

Travis's green. Merging in


[Docenhancement][mobilebotcommand][1694836]

Updated mobile image.

Docs Build status updates of commit d8eb297:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/Media/mobile-bot-menu-sample.png
:white_check_mark:Succeeded
View

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