Create aolink.md

Grant Application Checklist

[x] The application-template.md has been copied, renamed ( "project_name.md") and updated.
[x] A BTC or Ethereum (DAI) address for the payment of the milestones is provided inside the application.
[x] The software of the project will be released under the Apache license version 2.0 as specified in the terms and conditions.
[x] The total funding amount of the project is below USD $30k for initial grant applications and $100k for follow-up grants.
[x] The initial PR contains only one commit (squash if needed before submitting your PR).
[x] The grant will only be announced once we've successfully delivered the first milestone.

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

add aolink

Hi @jacklcz, thanks for your application. A few things:

All contributors to your application need to sign the CLA.
Can you please provide a roadmap with one or more milestones - with one or more deliverables - each as described in our application template?
Costs need to be designated in US$ and can only be paid out in BTC or DAI. If you require payment in fiat, you have to apply through the General Grants program.

Hi @jacklcz, thanks for your application. A few things:

All contributors to your application need to sign the CLA.
Can you please provide a roadmap with one or more milestones - with one or more deliverables - each as described in our application template?
Costs need to be designated in US$ and can only be paid out in BTC or DAI. If you require payment in fiat, you have to apply through the General Grants program.

Thanks for your reply, I will continue to add

Hi @jacklcz,

the grant amounts need to be denominated in USD,
we do not fund company infrastructure (Purchase Amazon server),
the project being funded must be open source, so we would not fund a deliverable Open source and promote,
you have to provide more technical specifications. For example, Use polkadot js to complete the interaction between native and dapp is too vague and hardly measurable,
there are already a number of dapp browsers and wallets in existence (cf. our tech stack). To fund one of these, we would need a convincing reason, such as unique functionality.
every milestone needs documentation and testing guidelines,
I would like to see any kind of open source development and maintenance by the team. The repositories you provided contain the Substrate node template with no visible changes and both Android repos under Ao* organisations have no commits.

Closing due to inactivity. Happy to reopen when the questions are answered.


Keycloak support

For a great open source commitment, add support for Keycloak (oidc).

+1
Well, not necessarily keycloack but I suppose the ability to use custom oauth/oidc providers would make me install this in my homelab right now

I have removed the default social login method and implemented OIDC in my fork. To enable this, the following environment variables should be set:

SOCIAL_AUTH_OIDC_OIDC_ENDPOINT: URL of your oidc provider. Please note that the other config is auto-detected, so be sure that <SOCIAL_AUTH_OIDC_OIDC_ENDPOINT>/.well-known/openid-configuration presents and contains the necessary information.
SOCIAL_AUTH_OIDC_KEY: Client ID
SOCIAL_AUTH_OIDC_SECRET: Client secrets

And the redirection URL will be: https://<Tabby>/api/1/auth/social/complete/oidc/

Nice! why not PR it?

I can only double the comment from @nlahmi, why not PR it here @kaaass?

It's mainly because my patch removed other default social login methods (Github, Microsoft, ...) for convenience. It's okay for self-hosting, but from README, since this repository "is the exact code that runs at https://tabby.sh/," removing them might not be a good idea. I could re-add them again, but that means I need to implement something to enable/disable the login method (tabby.sh doesn't need OIDC), including the frontend stuff. Since I'm unfamiliar with angular (the frontend framework tabby-web use), I just hardcoded the OIDC login button in my fork, which I think might not be a good idea to PR this.


Cursor line number background

@rakr what do you think about set same backgrounds for cursor line and cursor line number?

Very good idea, thanks for pointing this out.

Your commit set cursor line background to all line numbers (LineNr)

But I mean set background to cursor line number only (CursorLineNr). Look at the screenshot in first comment. Did you do this intentionally?

The color changes to LineNr lead to a wrong background color of vim-gitgutter signs:

vim-one links GitGutterAdd to SignifySignAdd which does not have a defined background color. Therefore the default background color is used for gitgutter signs.

Last update solves these two issues.

Now that the colorscheme is stable enough I'm now thinking about enabling some settings to allow user to tweak one to better suit their needs. I'm currently thinking about:

LineNr
Pmenu

if you have other ideas, feel free to open an issue.

@rakr thanks, my issue was fixed by last update.


Add filter_horizontal for for allowed and disallowed

Fix #34

So good :+1:

@jazzband/members anyone up to review this?

@yakky simple update, looks good to me. Thanks!


Add 'before_bulk_action' hook to disable delete

Adds Wagtail hook to prevent bulk delete of objects.
How to test this PR

Create a few pages
Attempt to delete multiple pages using bulk actions

Screenshots

Added some cheap tests just to have some coverage. However, these tests do not call the hook, but the function directly. These tests, ideally, would call the actual Wagtail hook and let wagtail run the function.

Added proper hook tests via wagtail recommended documentation.


No option to cache web-based credentials

I feel like this is not the correct place to post this, but I'm not sure where else this should go. My apologies if that is indeed the case, and I'd appreciate any pointers on whose attention this should be directed to instead. Additionally, it is unclear if this project is officially affiliated with / developed by Microsoft or not.
In Visual Studio with the Sourcelink option enabled, for each file that Visual Studio wishes to obtain the source code for a new instance of TF.exe is launched which spans a web view wherein the credentials prompt is displayed. The problem is that this webview does not have cookie support (and does not share a cookie jar with devenv.exe) and thus, VS will prompt for credentials (username, then password, then two-factor code!) each and every time a SourceLink operation is attempted.
Screenshot:

@mqudsi Yes, this is not the right spot for this. May be try stackoverflow.
My understanding is that tf.exe is used for source indexing, not source linking. I try to explain the different in the readme.


Clarify the meaning of 'const entries'

This PR is intended only to clarify the existing practice of P4 compilers and run times regarding their behavior when a table's entries are declared using const entries in the source code.  I am not aware of any P4 implementations that differ from the description in these changes.

You can paste that item in the agenda for next week.

@mbudiu-vmw "You can paste that item in the agenda for next week." Sorry for my ignorance, but I don't know where to go to do such pasting.  If you do not mind adding it to the agenda, that would be much appreciated.

Here:
https://docs.google.com/document/d/1XSgdXeG1UuF1FM_XAqxDrHeN4dHZWBnJPKVS6SnGNwM/edit
On Wed, Sep 28, 2022 at 3:30 PM Andy Fingerhut @.***>
wrote:

@mbudiu-vmw https://github.com/mbudiu-vmw "You can paste that item in
the agenda for next week." Sorry for my ignorance, but I don't know where
to go to do such pasting. If you do not mind adding it to the agenda, that
would be much appreciated.
—
Reply to this email directly, view it on GitHub
https://github.com/p4lang/p4-spec/pull/1155#issuecomment-1261376536, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOCFUX5JEW3JEZ776F4GSDWASMG5ANCNFSM6AAAAAAQXID5V4
.
You are receiving this because your review was requested.Message ID:
@.***>

This link is on the front page of the spec repository.

To add a changelog entry


Cluster-autoscaler runaway scale-up and daemonsets

How do you make this work with the cluster-autoscaler so it will not just keep spinning up nodes in the event of a daemonset failure? For example, if we set taints based on daemonsets A and B, and then there's an issue where daemonset B is broken and crashlooping, the autoscaler will just keep spinning up tainted nodes because it sees that there are pending pods. Is there a way to prevent this from happening?

In most cases you would probably want this behaviour.
e.g: you try to schedule a new pod, autoscaler creates new node, that node fails for some strange reason, it's critical daemonsets aren't working, no pods are scheduled on it, autoscaler creates another new node, it come up successfully, yay.
The scenario you're describing where the critical Daemonsets are broken cluster wide, would indicate you're having a some kind of serious cluster problem, in which case you would probably want someone to manually intervene, turn off autoscaler and fix whatever is broken with your cluster.

I actually found the latest version of the autoscaler has a --ignore-taint flag that does exactly this. But you do have a good point. We will see how this works in our cluster. I'm closing this issue because I figured out what I need, and I don't think it's the responsibility of Nidhogg to handle this case anyway.


Implement patternplate-transform-react-docgen

Rationale
patternplate projects tend to be implemented on a React-based stack. Component definitions therefore often contain valuable information in the form of React.PropTypes. We should strive for a solutions where we can derive additional information from component definitions
Features
Things to implement for first release

[ ] Generate markdown documentation from JS files
[ ] Show automatically generated documentation as code tab in patternplate-client
[ ] Handle non-React JS gracefully (without failing)
[ ] Test all functionality via ava
[ ] Check internal APIs via flow-type / typescript
[ ] Perform tests on CI and make sure win32 support is given via AppVeyor against Node 4 / Node 6

Non-Features
Things to ignore for first release

Configurable templates for generated markdown
Data API for react-docgen JSON output

Further reading

https://github.com/reactjs/react-docgen

Obsolete as patternplate will move towards a dev/design tool without own transform chain.


Image loading indicator does not work

Hi,
I observed that image loading indicator des not work.
It always appears and disappears properly but never animate.

I found that MWZoomingScrollView does not contain any image in photo property during fetching image from web. So it can not satisfy condition above:
- (void)setProgressFromNotification:(NSNotification *)notification {
    dispatch_async(dispatch_get_main_queue(), ^{
        NSDictionary *dict = [notification object];
        id <MWPhoto> photoWithProgress = [dict objectForKey:@"photo"];
        if (photoWithProgress == self.photo) {
            float progress = [[dict valueForKey:@"progress"] floatValue];
            _loadingIndicator.progress = MAX(MIN(1, progress), 0);
        }
    });
}

I'm experiencing the same issue.

I'm experiencing the same issue.

I'm experiencing the same issue.
How  fix it?

I'm experiencing the same issue.
How fix it?

Me too, please help us 👍

@listicos @JaydeepPatel @DevChen @rafalkitta
Check your code for something that cancels your web operations on view disappearance. And by view I mean a controller that presents your MWPhotoBrowser. Because that 'bug' happens only when an image downloading operation is rapidly canceled right after beginning.

I have the same issue and there isn't  anything that cancels downloading operation!

Put this methods in your view controller, it will solve your issue.
#pragma mark - MWPhotoBrowserDelegate

(NSUInteger)numberOfPhotosInPhotoBrowser:(MWPhotoBrowser *)photoBrowser {
return _photos.count;
}

(id )photoBrowser:(MWPhotoBrowser *)photoBrowser photoAtIndex:(NSUInteger)indexed {
if (indexed < _photos.count)
return [_photos objectAtIndex:indexed];
return nil;
}

(id )photoBrowser:(MWPhotoBrowser *)photoBrowser thumbPhotoAtIndex:(NSUInteger)indexed {
if (indexed < _thumbs.count)
return [_thumbs objectAtIndex:indexed];
return nil;
}

//- (MWCaptionView *)photoBrowser:(MWPhotoBrowser *)photoBrowser captionViewForPhotoAtIndex:(NSUInteger)index {
//    MWPhoto *photo = [self.photos objectAtIndex:index];
//    MWCaptionView *captionView = [[MWCaptionView alloc] initWithPhoto:photo];
//    return [captionView autorelease];
//}
//- (void)photoBrowser:(MWPhotoBrowser *)photoBrowser actionButtonPressedForPhotoAtIndex:(NSUInteger)index {
//    NSLog(@"ACTION!");
//}

(void)photoBrowser:(MWPhotoBrowser *)photoBrowser didDisplayPhotoAtIndex:(NSUInteger)indexed {
NSLog(@"Did start viewing photo at index %lu", (unsigned long)indexed);
}

//- (NSString *)photoBrowser:(MWPhotoBrowser *)photoBrowser titleForPhotoAtIndex:(NSUInteger)index {
//    return [NSString stringWithFormat:@"Photo %lu", (unsigned long)index+1];
//}

(void)photoBrowserDidFinishModalPresentation:(MWPhotoBrowser *)photoBrowser {
// If we subscribe to this method we must dismiss the view controller ourselves
NSLog(@"Did finish modal presentation");
[self dismissViewControllerAnimated:YES completion:nil];
}


Test Run

Test all the scripts with actual data.

Check all the attributes are scrapped
All the cleaners are cleaning
Data is written in the correct format

Documentation done in PR13 and test run in PR12


Discord invite link expired

Description
A clear and concise description of how the documentation at https://gpt-engineer.readthedocs.io/en/latest/ is providing wrong/insufficient information.

Thanks for letting us know. Currently, the docs don't even compile. We are working on it.

 Somehow the issue was not with the discord link, but the way I was personally logged into discord.


[DiscogsBridge] URL output containing two slashes leading to 404

Currently, the bridge generates a URL like https://www.discogs.com//release/14314167 (notice the two slashes).
self::URI already contains a trailing /
This PR fixes that.
Cheers
Thomas

@TwizzyDizzy Can you please take a look at this?


API-Login from Mikrotik has changed with Version 6.43

Home Assistant release with the issue:
Home Assistant 0.77.3

Last working Home Assistant release (if known):
Home Assistant 0.77.3
Operating environment (Hass.io/Docker/Windows/etc.):
Docker on Synology NAS

Component/platform:

device_tracker/mikrotik.py
Description of problem:
API-Login from Mikrotik has changed with Version 6.43
https://wiki.mikrotik.com/wiki/Manual:API#Initial_login
Traceback (if applicable):
Error doing job: Task exception was never retrieved

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/librouteros/connections.py", line 181, in write
    self.sock.sendall(string)
BrokenPipeError: [Errno 32] Broken pipe

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/app/homeassistant/components/device_tracker/__init__.py", line 692, in async_device_tracker_scan
    found_devices = await scanner.async_scan_devices()
  File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/app/homeassistant/components/device_tracker/mikrotik.py", line 137, in scan_devices
    self._update_info()
  File "/usr/src/app/homeassistant/components/device_tracker/mikrotik.py", line 159, in _update_info
    device_names = self.client(cmd='/ip/dhcp-server/lease/getall')
  File "/usr/local/lib/python3.6/site-packages/librouteros/api.py", line 79, in __call__
    self.protocol.writeSentence(cmd, *words)
  File "/usr/local/lib/python3.6/site-packages/librouteros/connections.py", line 139, in writeSentence
    self.transport.write(encoded)
  File "/usr/local/lib/python3.6/site-packages/librouteros/connections.py", line 185, in write
    raise ConnectionError('Failed to write to socket. ' + str(error))
librouteros.exceptions.ConnectionError: Failed to write to socket. [Errno 32] Broken pipe

Additional information:

Library is updated from 2.1 to 2.1.1 but the Error persists

I did some digging on this tonight.  I manually update librouteros with pip to version 2.1.1.  Verified that it was correctly installed in the virtualenv.  After starting up hass, I checked the environment again and it was once again running 2.1.0.  Looking at the device tracker code for this it has
REQUIREMENTS = ['librouteros==2.1.0']
Looks like it is running as designed.  Just need a pull request to update the version of librouteros.  It appears the new library works with the new version of firmware.  I don't have a device to check backwards compatibility to earlier versions.


Tasking: add pod failures to Task.Error.

The task manager needs to propagate task pod errors/failure to Task.Error.
Task.Error is cleared between retries.

Done.


Maximum call stack size exceeded

I first thought it was due to #436 , but I removed that similar code where I call a function to resolve image url and I still get Maximum call stack exceeded on a diffObjectRemovals. I see it's a recursive function, so I guess the condition to break out of the recursion is not good.
I don't know how to see what is calling this function and how it came to that.

I had a similar issue. Do you $watch a reactive car? Do you use transform function?
Check any $meteor.autorun code as well.

@MilosStanic can you please share what is different in your own code?

I think #467 should solve this issue.

great, new version coming soon


Ionic Appflow build breaks with cordova-res generated assets

When generating assets with cordova-res --skip-config --copy which is mentioned in the official ionic docs and your
readme my build throws an error. I am using ionic appflow in the paid version.
I dont know if I have to add the command to my build process or doit once. I did it both and in both situations the build breaks.
It seems that the environment gets confused by the generate folder "android" in my root project folder.
Here is my project structure:

Thats the error message from appflow.
/builds/project-0/fastlane/actions/build_android_capacitor.rb:58:in `chmod': No such file or directory @ apply2files - android/gradlew (Errno::ENOENT)

Does someone know what todo? The appflow support says it is a issue with my setup and is not really helpful at all.

@nhh did you ever resolve this? Running into an identical issue.

@nhh did you ever resolve this? Running into an identical issue.

I canceled ionic ci/cd system and using fastlane.tools manually. Works pretty good so far 👍


added release notes and pr template

I couldn't find a .stickler.yml file in this repository. I can make one for you, or you can create one by following the documentation.


move from suffix to prefix for filedocuments

This is part of a long migration. We can now safely let the runtime
write filedocuments to prefix based objects in the s3 bucket.
The way we do this is hacky, but we need to support previous runtime
versions too. In the future the runtime can make a prefix option and
then we can have two code paths, one with the hacky way of setting the
prefix, one with the proper if runtime version is newer.

LGTM!

Failing tests are due to timeouts, nothing to do with these changes. Verified the functionality manually in test environment. Will merge now.


[incubator-kie-issues-916] PIM for kogito workflow / remove unused kie api / kie internal api

Closes https://github.com/apache/incubator-kie-issues/issues/861#issue-2101810651

@elguardian can you please double check if the failures in kogito-examples are in any way related with this PR? (I don't think so)

@mariofusco they are related to the PR in kogito runtimes


[Macros] Don't allow types to shadow freestanding macros.

When performing name lookup for freestanding macros (e.g., after the #), don't allow types to shadow macros from imported libraries.
Fixes rdar://110429368.

@swift-ci please smoke test

@swift-ci please smoke test


No X11 DISPLAY variable was set, but this program performed an operation which requires it.

Hello All,
      Am trying creating docker container for JMETER, but am getting error while creating docker container.

Error,
No X11 DISPLAY variable was set, but this program performed an operation which requires it.
Below the screenshot.

hi ramana,
follow this link Click Here
if you are resolve the issue please close the issue.


:memo: v4 logo on main readme

also cc @kodadot/design-squad

yes, let me create a nice banner for you sir

you have the right to choose @vikiival

https://github.com/kodadot/.github/commit/d9b8f498bfe47a0ae7cf1bca417759153550c5b8


Unable to execute with any TF/Keras combination (wt/wo GPU)

Hello,
I've tried running this model with TF 1.6 /1.4 /1.2 (with and without GPU) and Keras 2.0 and am getting the stack trace below [ubuntu 16.04, python3.6, data folder with examples] when i execute the train.py script
Unable to load model from file.
Loading data
Training
Traceback (most recent call last):
File "train.py", line 32, in
main()
File "train.py", line 16, in main
train(models)
File "train.py", line 29, in train
models[0].fit(train_data, train_labels, epochs=1000, callbacks=cbs, batch_size=BATCH_SIZE)
File "/usr/local/lib/python3.6/dist-packages/keras/engine/training.py", line 1405, in fit
batch_size=batch_size)
File "/usr/local/lib/python3.6/dist-packages/keras/engine/training.py", line 1295, in _standardize_user_data
exception_prefix='model input')
File "/usr/local/lib/python3.6/dist-packages/keras/engine/training.py", line 121, in _standardize_input_data
str(array.shape))
ValueError: Error when checking model input: expected input_1 to have 4 dimensions, but got array with shape (0, 1)
I tried downgrading Keras to 1.xx but the code uses Keras 2.0 syntax.
can you please look into it?

Closed issue, was not using master branch....

I didn't understand how did you solve it?Could you please explain it?

i have the same problem, how you solve this?


Iphone (ios) not working

not working on Smartphones iphone, black frame without video and sound:

Device:  iPhone6, 7 not working
iOS 13 and above
Browser: safari

Additional context (here is my code):
`<!doctype html>

      <!--
            <video  src="/files/other/161c15a9fe0152c.mp4"
               preload="auto" id="vid" response-type="arraybuffer"  playsinline autoplay loop muted crossorigin    >
            </video>
        -->

        </a-assets>

<a-marker       smooth="true"
                smoothCount="10"
                smoothTolerance=".01"
                smoothThreshold="5"
                  type='barcode' value='15'        videohandler >
    <a-video
        scale="0.003 0.003 0.003"
        src="#vid"
        rotation='-90 0 0'
        position='0 0 0'
        width='600px'
        height='300px'
    >
    </a-video>

</a-marker>
<a-entity camera></a-entity>

`
Link for test:  https://artup.ge/webar/15
pattern: https://artup.ge/AR_MATRIX_CODE_4x4/0015.png

Please, open issue in the new repository, this one is not maintained anymore. You can find references on the readme.


Adjust hero styling on devices

This update tweaks spacing on devices for the Hero header on micheal.dev.

This now looks much better on the MBP!
Before:

After:


Export variable not shown in inspector when using external script editor

Godot version: 3.2.rc1.mono
OS/device including version: Linux 5.4.12-1-MANJARO
Issue description:
I wrote a GDscript, not C#.
When I save the exported variables aren't shown in the inspector.
I only can see them after I restart Godot.
Steps to reproduce:

Open a project with the mono version of Godot and create a scene
Create a node and a GDscript script
Create a second GDscript where you put the export variable
Inherit the second script in the script attached to the node
No exported variable is shown in inspector
Build the project
Restart Godot and the exported variable is shown

Minimal reproduction project:
Create any Node, add two GDscript / one attached to the node, make an export variable in the script without a node and inherit the script with the variable in the script attached to the node

This is an issue I can confirm this on 3.2
Add this line from Visual Studio Code into a script and go back to Godot Editor
export (NodePath) var path2
You will see the script update but the export option does not show up in the UI

Open the Script in the built in editor and save again. All the sudden the export DOES show up.

A test project here isn't going to show the issue. It updates when you open a new project, which fixes the problem. It will only update when you either
1.) Save from Godot's text editor, or
2.) Open a different project, then reopen your first project

Can confirm this, only soft reloading the script from the built-in editor or reloading the entire project fixes it.

Linking issues as they seem to be connected:
https://github.com/godotengine/godot-vscode-plugin/issues/136

Looking for a workaround and so have been poking around in the Godot source. Unfortunately I am no CPP expert, but what I understood is this:
The the reload function is only called from the script editor drop down file menu if the the script is open in editor or via (hardcoded?) shortcut.
The shortcut and the filemenu entry are only available if the script is open in the internal editor. This makes it impossible to reload if the scripts are only opened in the external editor.
Question: Is it somehow possible to open scripts in external AND internal editor at the same time? Then one could simply hit cmd + shift + R after switching back to Godot window, as a workaround...
To fix this we need either

a way to reload from external
or make sure Godot reloads whenever the file is touched or changed.
or a global reload shortcut.

I hope I got that right 😅.
@reduz  which one would be the preferred fix?

Hi,
I had this issue, but following the "Your first game" Tutorial I saw the comment below:

I did it and it works. re(Build) project to show the variables in the inspector.

I'm facing the same issue!
I use Vscode for external scripting and sees that export has no effect on the Godot Ui!
@teknnovator is right !
Some way of global reloading is really helpful!

Hi,
I had this issue, but following the "Your first game" Tutorial I saw the comment below:

I did it and it works. re(Build) project to show the variables in the inspector.

Thanks, this saved me today! that did the trick for me ^^

Not just external editor, but the internal one as well: https://imgur.com/a/EYgNAeT
What's odd is that it seems to work and stop working at random. (Godot version for that clip is 3.3.4 win64, using Windows 10)

Exactly this is happening to me too. The exported variables appear only when I modify the script and save it. It happens sometimes.

I am experiencing similar random behavior with Godot 4.0.beta.3 on Windows 10.
Sometimes, the properties exported from the script disappear completely.
Reopening the scene seems to help. Although, when I first tried, that resulted in the script's exported properties not being loaded at all.
https://user-images.githubusercontent.com/2036806/198893698-34fe9c1e-5773-4faa-b6bb-99eb62ff8839.mp4

I got the same. I found out, that in my case it happens because I added a _ready() function in the super class. Then it stopped working. When I remove and readd, it is working until I restart Godot. Very weird.

After Debugging the code more, I found the problem.
When starting Godot, he is for some reason not parsing the script. The parser returns an error (ERR_PARSE_ERROR).
After saving the script again manually, everything works as now he does not have a parse error for some reason.

this also does not work for me in 3.5, using NeoVim. I have to manually save the script inside Godot's editor

This is happening for me in 4.2.2. If I instantiate a scene as a child of my main scene it doesn't show it's export variables until I save the script that's attached to it.

After creating this:
@export var speed: float = 5.0
saving...
now is not visible in the inspector panel.

This happened for me too with latest stable verison


Don't trust X-Forwarded-For and related headers by default

X-Forwarded-For/X-Real-IP/CF-Connecting-IP/etc should not be used as a source of IPs if CTI is not actually behind a reverse proxy.
(Maybe it should be a config option of some sort, along with a way to configure which IPs such headers can be accepted from)
Example:

Someone seems to have abused this already:

I have to check the logic again, because I do have to consider some other means of getting an IPv4 while being behind some level of proxy/gateway.
The service is currently running on fly.io, so there are some layers in front of it and in the specific setup I cannot always rely on the client IP, as for some requesters this would be a fly.io internal IP (people coming from IPv6 are quite the usual suspects, but there might be other reasons why the client IP is just an internal one).
So it's quite tricky to provide both a fair game and prevent too easy hacking/cheating.

So, since I plan to continue using fly.io and they actually do provide me with a client IP, which just may or may not be IPv4, I decided to remove all other means of IP extraction. IPv6 is not supported as intended and thus also overriding/faking it with forward headers is not allowed anymore.
Thanks for reminding me about this issue. ❤


The Label issues

Hello, yesterday I've updated libgdx from 1.5.2 to 1.7.0 and have noticed some issues with scene2d.ui Labels.
Issue 1
Maybe it's intended, maybe not. In 1.7.0 Labels don't have a little space before the first character (actually the part of this character) and after the last one.
Please, see this example. It's a Label added to a table cell. Compare its left and right boundaries as was in the version 1.5.2 and as is in the version 1.7.0.

Issue 2 - really bizarre one
When you add a Label to a table cell and set the width of the latter, some characters may become broken...
Here is an example, where I have a table and add a lot of labels into its cells (like table.add(label).width(columnWidth)). Please, notice some strange artifacts in some characters (especially 'F', '5', '6'). The same thing without setting the width is below -- as you can see, all characters there are rendered correctly. Btw the font is 32px size and pregenerated with FreeTypeFontGenerator.
Please, click on the image to see it in full size.

Another example is with a TextButton and its Label. When simply added to a table, the Label is rendered correctly, but if added followed by fillX(), the artifact appears:

The font is Lato 2.015 Light (Lato-Light.ttf):
http://www.latofonts.com

The previous examples were from Desktop.
Here are some from Android. Please, click on the images to see them in full size.

I've checked different gdx versions. Both issues were introduced in libgdx 1.6.0.

probably the result of https://github.com/libgdx/libgdx/issues/3074 , http://www.badlogicgames.com/wordpress/?p=3678 - check your fonts first, it's probable they have some semi-strange values in them.

Hi @FyiurAmron, thank you for your reply!
I have just tried to simply use the mentioned fixing function this way:
includePaddingInMetrics(mLight.getData());
Where mLight is one of my pre-generated fonts. I hope this was correct... But both issues still remain.
I will have more time tomorrow to see into these solutions. And also I will check other fonts. However, I'm not sure this is something appropriate for the 2nd issue -- a label is rendered perfectly till it's placed into a resized cell. So, I've done a simple workaround for this issue -- to place my Labels into additional Containers.
Anyway, thank you for your recommendation!

Well, I've tested with these fonts:

Arial
Verdana
Times New Roman
Georgia
Roboto-Thin

Both issues appear for all of them. So, I don't think they are related to a specific font.

@serhii-k to be more precise: by "check your fonts" I meant you should check the fonts (i.e. the resulting data structures) generated by your calls to FreeTypeFontGenerator. If there's a discrepancy in font rendering engine, tracking the font's data structs all the way from the beginning in FTFG is usually the best way to go.
Using libGDX's reference Arial font (in the bitmap version) is a good way to start debugging it. IMO if you can repro this bug with reference libGDX's Arial - we've a bug in font rendering. If not, we've a bug in FTFGen. BTW always try to reduce the dependency on external assets while reporting bugs - by minimizing the amount & complexity of data needed to repro the bug, the amount of code which needs to be analyzed to track the bug gets reduced.

I think I bumped into this issue too, at least the artifacts were similar with libgdx 1.7.1, what I did was different:
Generate a BitmapFont with FreeTypeFontGenerator (some fonts I used: LiberationSans-bold and Windows 10 system font arialbd) with size 48.
Render some text with the generated font, for example: "Game Over", "512", "256", "8"
Some of the characters are cut off and seems like they are misplaced a few pixels (like the top of G is cut off, and it's higher then the rest "ame").
This problem didn't occurred when using size 64, and for size 32 some fonts looked good while some fonts had errors.
With libgdx 1.5.2, I haven't seen any error with these generated fonts.
I'll fiddle with it a bit and I'll post an example code later today or tomorrow.

Ok, I found what my problem was, which seems that not directly related to this issue.
FreeTypeFontGenerator sets the integer boolean of the generated BitmapFont to false. I don't know what is the reason for that. When creating the BitmapFont directly it is set to true (as it is documented at usesIntegerPositions).
This solved my issues with my artifacts, see example code where it is reproduced here: https://github.com/qzole/gdx-font-test
Could it be that the same is happening here? This problem of course only happens if you try to render at a non integer position. I could imagine that without setting the width it could just so happen that everything is put on an integer position?
Tested my problem with 1.5.2, and if I try to put text on a non integer position, I can reproduce my issue (in the previous comment, when I changed version I changed where I rendered my text because GlyphLayout was not available.
So it seems that the two issues are not related, but it could be a (better?) workaround to simply set the font's setUseIntegerPosition to true.

BitmapFont constructs default the integer parameter to true when omitted. You're right, FreeTypeFontGenerator doesn't. I don't think integer even belongs as a constructor parameter. It isn't needed when loading the font and just blows up the number of convenience constructors needed. I think we should remove the parameter and default integer to true. Anyone disagree? It would be a breaking change for those passing integer as false, but pretty obvious to fix (call the setter using false).
FreeTypeFontGenerator would then return a font with integer true.

FreeTypeFontGenerator has been updated to use integer=true by default.
Your other issue (1) is by design, fonts now have tighter bounds.

Guys, thank you very much for resolving this issue!
I have just checked and the fonts are now rendered perfectly!
Thank you!!!!!


Error while switching Greenhill scene

While we try to switch greenhill scene to some other scene, It Throws this warning and the other scene doesn't loads.

https://user-images.githubusercontent.com/35926530/168116136-db9f6243-0e6e-4d86-b61b-9ed7235693db.mp4

fixed. Could you please test it again?

Fixed


Implement an alarm clock component

I posted about this in the forum: https://community.home-assistant.io/t/alarm-clock-component/34250 and there was some interest.
On top of the basic functionality (enable/disable the alarm; pick the time; trigger a light, a group or a scene when the alarm goes off) it supports an “additional” timer that can go off before / after the main one.
A typical use case: I want my alarm to play at 7am; but 15 minutes I want my Hue lights to start a “sunrise” transition that slowly turns them on. I could do that by setting up an extra automation at 6:45am, but then what if tomorrow I need to wake up 20 minutes earlier? I’d need to change both.
To give you an example, this is what I currently have in my config:
input_boolean:
  alarm_clock:
    name: Enable Alarm
    icon: mdi:calendar

input_select:
  wake_up_time:
    name: Alarm time
    options:
    - "06:00"
    - "06:15"
    - "06:30"
    - "06:45"
    - "07:00"
    - "07:15"
    - "07:30"
    - "07:45"
    - "08:00"

alarm_clock:
  time: input_select.wake_up_time
  master_control: input_boolean.alarm_clock
  entity_id: scene.wakeup
  additional:
    time: -900
    entity_id: scene.sunrise

In my opinion this is already possible just by using the script component:
script:
  # Turns on the bedroom lights and then the living room lights 1 minute later
  wakeup:
    alias: Wake Up
    sequence:
      # This is Home Assistant Script Syntax
      - event: LOGBOOK_ENTRY
        event_data:
          name: Paulus
          message: is waking up
          entity_id: device_tracker.paulus
          domain: light
      - alias: Bedroom lights on
        service: light.turn_on
        data:
          entity_id: group.bedroom
          brightness: 100
      - delay:
          # supports seconds, milliseconds, minutes, hours
          minutes: 1
      - alias: Living room lights on
        service: light.turn_on
        data:
          entity_id: group.living_room

Issues are for bug reports, this looks more like a (premature) announcement? Just file a PR when you are ready to contribute your work :)

@amelchio   ah fair enough, I’m used to other projects / tools where you always create a task before sending a diff. I’ll close.
@tringler sure it’s possible, but now your alert is no longer at 7am but some minutes earlier—you have to do the math yourself.
Similar with other (future) ideas I and others mentioned in the forum: sure the building blocks are all there, it’s only convenience; but nothing wrong in convenience.


Validator Features

NO STAKING FROM APP
some suggestions from community on adding validator-based features to the app:

[ ] follow a validator
[ ] see earnings on account
[ ] see earnings on hotspot page (if you own the validator)
[ ] know when it's in consensus

tracked as part of #768


Feature Request: Channel logos

Would be really cool to show logos above channel names on the left side! Just like in Hypnotix :) Thank you!

yay! thank you so freakin' much!!!! I can't wait for the Snap update, this will be fantastic :)


LinkedIn videos not supported? [Site Request]

Checklist

[x] I'm reporting a new site support request
[x] I've verified that I'm running yt-dlp version 2021.09.25
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that none of provided URLs violate any copyrights
[x] The provided URLs do not contain any DRM to the best of my knowledge
[x] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Single video: https://www.linkedin.com/posts/mishalkhawaja_sendinblueviews-toronto-digitalmarketing-ugcPost-6850898786781339649-mM20

Description

Is LinkedIn not supported? The reason I ask is that I am able to download LinkedIn Learning videos using a cookie file but not LinkedIn feed videos, like the one I have referenced above.
Here's the verbose log:
yt-dlp -J https://www.linkedin.com/posts/fordnation_shubnavratri-hindu-navatri-ugcPost-6852030793657794561-4UPb --verbose
[debug] Command-line config: ['-J', 'https://www.linkedin.com/posts/fordnation_shubnavratri-hindu-navatri-ugcPost-6852030793657794561-4UPb', '--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] yt-dlp version 2021.09.25
[debug] Python version 3.7.5 (CPython 64bit) - Linux-4.15.0-135-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: none
[debug] Optional libraries: Crypto, keyring, mutagen, sqlite, websockets
[debug] Proxy map: {}
[debug] [generic] Extracting URL: https://www.linkedin.com/posts/fordnation_shubnavratri-hindu-navatri-ugcPost-6852030793657794561-4UPb
[generic] fordnation_shubnavratri-hindu-navatri-ugcPost-6852030793657794561-4UPb: Requesting header
WARNING: [generic] Falling back on generic information extractor.
[generic] fordnation_shubnavratri-hindu-navatri-ugcPost-6852030793657794561-4UPb: Downloading webpage
[generic] fordnation_shubnavratri-hindu-navatri-ugcPost-6852030793657794561-4UPb: Extracting information
[debug] [generic] Extracting URL: https://www.linkedin.com/embed/feed/update/urn:li:activity:6852030803648622592?compact=true
[generic] urn:li:activity:6852030803648622592?compact=true: Requesting header
WARNING: [generic] Falling back on generic information extractor.
[generic] urn:li:activity:6852030803648622592?compact=true: Downloading webpage
[generic] urn:li:activity:6852030803648622592?compact=true: Extracting information
ERROR: [generic] Unsupported URL: https://www.linkedin.com/embed/feed/update/urn:li:activity:6852030803648622592?compact=true
Traceback (most recent call last):
  File "/home/bd7/.local/lib/python3.7/site-packages/yt_dlp/extractor/common.py", line 585, in extract
    ie_result = self._real_extract(url)
  File "/home/bd7/.local/lib/python3.7/site-packages/yt_dlp/extractor/generic.py", line 3712, in _real_extract
    raise UnsupportedError(url)
yt_dlp.utils.UnsupportedError: Unsupported URL: https://www.linkedin.com/embed/feed/update/urn:li:activity:6852030803648622592?compact=true
Traceback (most recent call last):
  File "/home/bd7/.local/lib/python3.7/site-packages/yt_dlp/extractor/common.py", line 585, in extract
    ie_result = self._real_extract(url)
  File "/home/bd7/.local/lib/python3.7/site-packages/yt_dlp/extractor/generic.py", line 3712, in _real_extract
    raise UnsupportedError(url)
yt_dlp.utils.UnsupportedError: Unsupported URL: https://www.linkedin.com/embed/feed/update/urn:li:activity:6852030803648622592?compact=true

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/bd7/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1227, in wrapper
    return func(self, *args, **kwargs)
  File "/home/bd7/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1252, in __extract_info
    ie_result = ie.extract(url)
  File "/home/bd7/.local/lib/python3.7/site-packages/yt_dlp/extractor/common.py", line 602, in extract
    e.msg, video_id=video_id, ie=self.IE_NAME, tb=e.traceback, expected=e.expected, cause=e.cause)
yt_dlp.utils.ExtractorError: [generic] Unsupported URL: https://www.linkedin.com/embed/feed/update/urn:li:activity:6852030803648622592?compact=true

I got some linkedin videos to download using this command: yt-dlp --cookies "C:\Users\username\Downloads\linkedin.com_cookies.txt" -o "C:\Temp\Training Videos\LinkedIn Learning\Powershell\%(playlist_title)s\Lesson %(chapter_number)02d - %(chapter)s\%(playlist_index)s - %(title)s.%(ext)s.mp4" https://www.linkedin.com/learning/powershell-7-essential-training.
Here is something that was in my notes from a previous script:
https://github.com/ytdl-org/youtube-dl/issues/20326

Use Chrome to login to linkedin:learning
    Play a learning video e.g. on this page.
    Use the cookies plugin to export the file cookies.txt

How to get the "x-li-identity” header value
    In Chrome press F12 to open the Network Monitor (development console).
    Reload the linkedin:learning page with the video.
    In the Network Monitor, search for “/learning-api”, and click on one of the lines with the HTTP requests.
    On the right side, scroll down to the request header line “x-li-identity: ...”, and copy
    the value to your command line option: --add-header "x-li-identity:..."

:start
yt-dlp --cookies "linkedin.com_cookies.txt" --add-header "x-li-identity:dXJuOmxpOmVudGVycHJpc2VQcm9maWxlOih1cm46bGk6ZW50ZXJwcmlzZUFjY291bnQ6MjEzMzg0OSwxMzcwNDk0NjAp" -o "C:\Temp\Training Videos\LinkedIn Learning\%(playlist_title)s\Lesson %(chapter_number)02d - %(chapter)s\%(playlist_index)s - %(title)s.%(ext)s.mp4" https://www.linkedin.com/learning/programming-foundations-databases

I'm not sure if the "x-li-identity:" is still needed, as I didn't need it this morning. Hope this helps.

That video is from a LinkedIn Learning course. The issue that I've described is for Linkedin videos that appear in the feed. Those videos cannot be downloaded by yt-dlp at the moment, even when using cookies.

@hanimourra I figured out how to download LinkedIn Live videos.
I used the stream-detector Chrome plugin while browsing the LinkedIn live page, and it gave me yt-dlp commands.
I then copied and executed the last of the commands (ending in livemanifest.ism/manifest(format=m3u8-aapl)"). and that got me the full video file.
For whatever reason for me the video/audio are slightly out of sync so the lip movement doesn't match the audio but I can live with that for now.


improve task-step labels

Resolves issue #135 where I couldn't make sense of how loop_tasks's progress messages were matching up with actual applier actions. Now instead of just reporting Building [task], it says "Building [task-step]".
loop_tasks continues to interact awkwardly with messages within applier functions, but for full transparency of actions, here's what I'm seeing in my test case (from ds-pipelines-3-course repo), where the task names are state codes (AL, CT, etc.) and the step names are tally and plot:
### Starting loop attempt 2 of 30 with 9 of 28 tasks remaining:
  Building AL-tally [========================================================>---------------------------]  68%
  Tallying data for AL-02465000
  Building CT-tally [===========================================================>------------------------]  71%
  Retrieving data for CT-01199000
* Error in get_site_data("1_fetch/tmp/inventory_CT.tsv", parameter): Ugh, the internet data transfer failed! T...
  Building CT-plot [============================================================>------------------------]  71%
  Retrieving data for CT-01199000
* Error in get_site_data("1_fetch/tmp/inventory_CT.tsv", parameter): Ugh, the internet data transfer failed! T...
  Building DE-tally [===========================================================>------------------------]  71%
  Tallying data for DE-01479000
  Building DC-tally [==============================================================>---------------------]  75%
  Retrieving data for DC-01648000
* Error in get_site_data("1_fetch/tmp/inventory_DC.tsv", parameter): Ugh, the internet data transfer failed! T...
  Building DC-plot [===============================================================>---------------------]  75%
  Retrieving data for DC-01648000
Plotting data for DC-01648000
  Building ID-tally [=================================================================>------------------]  79%
  Tallying data for ID-13077000
  Building IL-tally [====================================================================>---------------]  82%
  Tallying data for IL-05572000
  Building IN-tally [=======================================================================>------------]  86%
  Tallying data for IN-03373500
### Starting loop attempt 3 of 30 with 3 of 28 tasks remaining:
  Building CT-tally [==========================================================================>---------]  89%
  Retrieving data for CT-01199000
Tallying data for CT-01199000
  Building CT-plot [==============================================================================>------]  93%
  Plotting data for CT-01199000
  Building DC-tally [================================================================================>---]  96%
  Tallying data for DC-01648000
All tasks complete [=====================================================================================] 100%

If I take out the within-applier messages, I see the task-step labels overwriting themselves more like a typical progress bar. here are some screenshots:

...huh. I asked Jordan to hold up on reviewing this PR because I was getting this error and doing all kinds of debugging to understand it:
[    OK ] parameter
[    OK ] AL_data
[    OK ] 3_visualize/out/timeseries_AL.png
[    OK ] AL_tally
[    OK ] AZ_data
[    OK ] 3_visualize/out/timeseries_AZ.png
[    OK ] AZ_tally
[    OK ] AR_data
[    OK ] 3_visualize/out/timeseries_AR.png
[    OK ] AR_tally
[    OK ] CA_data
[    OK ] 3_visualize/out/timeseries_CA.png
[    OK ] CA_tally
[    OK ] CO_data
[    OK ] 3_visualize/out/timeseries_CO.png
[    OK ] CO_tally
[    OK ] CT_data
[    OK ] 3_visualize/out/timeseries_CT.png
[    OK ] CT_tally
[    OK ] DE_data
[    OK ] 3_visualize/out/timeseries_DE.png
[    OK ] DE_tally
[    OK ] DC_data
[    OK ] 3_visualize/out/timeseries_DC.png
[    OK ] DC_tally
[    OK ] FL_data
[    OK ] 3_visualize/out/timeseries_FL.png
[    OK ] FL_tally
[    OK ] GA_data
[    OK ] 3_visualize/out/timeseries_GA.png
[    OK ] GA_tally
[    OK ] ID_data
[    OK ] 3_visualize/out/timeseries_ID.png
[    OK ] ID_tally
[    OK ] IL_data
[    OK ] 3_visualize/out/timeseries_IL.png
[    OK ] IL_tally
[    OK ] IN_data
[    OK ] 3_visualize/out/timeseries_IN.png
[    OK ] IN_tally
[    OK ] IA_data
[    OK ] 3_visualize/out/timeseries_IA.png
[    OK ] IA_tally
[    OK ] timeseries_plots.yml_promise
Error: hash 'f9e884084b84794d762a535f3facec85' not found

but then I manually modified 123_state_tasks.yml from something like
  timeseries_plots.yml_promise:
    command: summarize_timeseries_plots(I('3_visualize/out/timeseries_plots.yml'),
      `AL_tally`,
      '3_visualize/out/timeseries_AL.png',
      `AZ_tally`,
      '3_visualize/out/timeseries_AZ.png',
      `AR_tally`,
      '3_visualize/out/timeseries_AR.png',
      `CA_tally`,
      '3_visualize/out/timeseries_CA.png')

to just one state:
  timeseries_plots.yml_promise:
    command: summarize_timeseries_plots(I('3_visualize/out/timeseries_plots.yml'),
      `AL_tally`,
      '3_visualize/out/timeseries_AL.png')

and then it built just fine and now i'm having trouble getting the error back.
So I don't know whether I broke something with this PR or not. I guess I'm going to go with having corrupted something temporarily, because I just deleted .remake and 3_visualize/out/* and reran scmake() without error:
> scmake()
Starting build at 2020-06-02 13:41:43
<  MAKE > main
[ BUILD ] states                              |  states <- c(c("AL", "AZ", "AR", "C...
[  READ ]                                     |  # loading packages
[ BUILD ] parameter                           |  parameter <- c("00060")
[ BUILD ] oldest_active_sites                 |  oldest_active_sites <- find_oldest...
Inventorying sites in AL
Inventorying sites in AZ
Inventorying sites in AR
Inventorying sites in CA
[ BUILD ] state_combiners                     |  state_combiners <- do_state_tasks(...
v0.0.17: tickquote_combinee_objects is deprecated and will effectively be always TRUE in future versions
Run all tasks and finalizers with:
obs_tallies:
  command: scmake(I('obs_tallies_promise'), remake_file='/Users/aappling/Documents/Code/Pipeline/ds-pipelines-3/123_state_tasks.yml')
3_visualize/out/timeseries_plots.yml:
  command: scmake(I('timeseries_plots.yml_promise'), remake_file='/Users/aappling/Documents/Code/Pipeline/ds-pipelines-3/123_state_tasks.yml')
### Starting loop attempt 1 of 30 with 8 of 8 tasks remaining:
* Error in get_site_data("1_fetch/tmp/inventory_AZ.tsv", parameter): Ugh, the internet d...
* Error in get_site_data("1_fetch/tmp/inventory_CA.tsv", parameter): Ugh, the internet d...
* Error in get_site_data("1_fetch/tmp/inventory_CA.tsv", parameter): Ugh, the internet d...
### Starting loop attempt 2 of 30 with 3 of 8 tasks remaining:
* Error in get_site_data("1_fetch/tmp/inventory_CA.tsv", parameter): Ugh, the internet d...
### Starting loop attempt 3 of 30 with 1 of 8 tasks remaining:
All tasks complete [===============================================================] 100%

### Final check for completeness of all targets
Starting build at 2020-06-02 13:42:28
<  MAKE > 123_state_tasks
[    OK ] parameter
[    OK ] AL_data
[    OK ] 3_visualize/out/timeseries_AL.png
[    OK ] AL_tally
[    OK ] AZ_data
[    OK ] 3_visualize/out/timeseries_AZ.png
[    OK ] AZ_tally
[    OK ] AR_data
[    OK ] 3_visualize/out/timeseries_AR.png
[    OK ] AR_tally
[    OK ] CA_data
[    OK ] 3_visualize/out/timeseries_CA.png
[    OK ] CA_tally
[ BUILD ] obs_tallies_promise                 |  obs_tallies_promise <- combine_obs...
[ BUILD ] timeseries_plots.yml_promise        |  timeseries_plots.yml_promise <- su...
[ ----- ] 123_state_tasks
Finished build at 2020-06-02 13:42:28
Build completed in 0.00 minutes
Starting build at 2020-06-02 13:42:29
<  MAKE > obs_tallies_promise
[    OK ] parameter
[    OK ] AL_data
[    OK ] 3_visualize/out/timeseries_AL.png
[    OK ] AL_tally
[    OK ] AZ_data
[    OK ] 3_visualize/out/timeseries_AZ.png
[    OK ] AZ_tally
[    OK ] AR_data
[    OK ] 3_visualize/out/timeseries_AR.png
[    OK ] AR_tally
[    OK ] CA_data
[    OK ] 3_visualize/out/timeseries_CA.png
[    OK ] CA_tally
[    OK ] obs_tallies_promise
Finished build at 2020-06-02 13:42:29
Build completed in 0.00 minutes
Starting build at 2020-06-02 13:42:30
<  MAKE > timeseries_plots.yml_promise
[    OK ] parameter
[    OK ] AL_data
[    OK ] 3_visualize/out/timeseries_AL.png
[    OK ] AL_tally
[    OK ] AZ_data
[    OK ] 3_visualize/out/timeseries_AZ.png
[    OK ] AZ_tally
[    OK ] AR_data
[    OK ] 3_visualize/out/timeseries_AR.png
[    OK ] AR_tally
[    OK ] CA_data
[    OK ] 3_visualize/out/timeseries_CA.png
[    OK ] CA_tally
[    OK ] timeseries_plots.yml_promise
Finished build at 2020-06-02 13:42:30
Build completed in 0.00 minutes
[ BUILD ] obs_tallies                         |  obs_tallies <- pluck(state_combine...
[ BUILD ] timeseries_plots_info               |  timeseries_plots_info <- pluck(sta...
[ BUILD ] 3_visualize/out/site_map.png        |  map_sites(oldest_active_sites, "3_...
[ BUILD ] 3_visualize/out/data_coverage.png   |  plot_data_coverage(out_file = "3_v...
[ BUILD ] 3_visualize/out/timeseries_map.html |  map_timeseries(out_file = "3_visua...
[ ----- ] main
Finished build at 2020-06-02 13:42:32
Build completed in 0.82 minutes

...so I guess it's fine?


[Website Component] Icon card & icon search/filter bar

Final icon card design, spec and skeleton state info
Sketchfile: https://ibm.box.com/s/o5qhcg064uzwedl63yimxh1hzx37kcn7

Will add empty state design soon

Empty state!

@shixiedesign @jeanservaas
I think the spacing between Categories is too much. 80 seems excessive. The Dl site seems good or it can be slightly more open maybe.

@mjabbink okay. I will make an update today!

Icon page spacing update

make sure to address search bar buggy behavior described here https://github.com/carbon-design-system/carbon-website/issues/1132

Some developer notes that could be helpful for that use-case/user archetype:
Importing an icon
The primary way a developer will include an icon is through one of our icon packages. These include:

@carbon/icons
@carbon/icons-react
@carbon/icons-angular
@carbon/icons-vue

Each project contains all of the icons available in Carbon Elements with a consistent name and import path. This means that if you want to import an icon, say AddFilled16, one would write the following in each framework:
import AddFilled16 from '@carbon/icons/es/add--filled/16';
import AddFilled16 from '@carbon/icons-react/es/add--filled/16';
import AddFilled16 from '@carbon/icons-angular/es/add--filled/16';
import AddFilled16 from '@carbon/icons-vue/es/add--filled/16';

However, the module name and the path to import it can be confusing or hard to discover. For example, how would one know that AddFilled16 translates to add--filled/16. It would be helpful if this information was presented in a way such that a developer could understand how to include an icon resource in their corresponding tool chain 👍
With respect to the paths themselves, the folder name es could be one of the following:

es used for ECMAScript modules (ESM), latest and greatest syntax for code in JavaScript, requires specific settings to use
lib used for CommonJS modules, commonly used as an interop layer. Works in most use-cases
umd used for Universal Module Definitions, used as an interop layer, works in all use cases but suffers from code size/bloat as a result

As a result, the following would be options depending on the developer and their setup:
import AddFilled16 from '@carbon/icons/es/add--filled/16';
import AddFilled16 from '@carbon/icons/lib/add--filled/16';
import AddFilled16 from '@carbon/icons/umd/add--filled/16';

Current design for icon expansion panel:
Spec & sketchfile coming soon.

[ ] A few more design update coming in via Mike
[ ] Mobile experience

from Ryan

the work i did on iconography page is on the branch iconography-functionality

note, the first 3 icons are the only ones that work because i didn't want to write out 700 pieces of dummy data
AddFilled, AddOutline, Add work. when the drawer is open, you can click the related icons, and click the related search items (edited)

@alisonjoseph @jeanservaas  Let’s remove cards from Icon Library page as per Jeannie’s comment  in #1284.

@mjabbink @shixiedesign @alisonjoseph @jeanservaas This spec as delivered represents a wedding cake; we still need to see a plan for the cake that lies somewhere between this and where we are right now.

(commenting here bc #1284 is closed.)
@mjabbink @jeanservaas @shixiedesign please keep in mind a couple of important points:

Just to re-iterate, we ~do not want to~ cannot present people a frictionless path in the Library page to download individual, or groups of individual, icons either for their Sketch design documents or their code for their products. Downloading individual SVGs for use in design documents or god forbid hosting in a product's endpoint orphans them, so they will never get any asset updates as we make them.

The most likely desired/supported reason to download an asset's SVG directly is to edit it or use it as the basis for the creation of a new assets (hopefully for submission). So people can still go to the linked repo and get them there - it's just not how we will instruct or directly enable from the Icon Library page.

The reason the asset cards are on the Library page is that the most common reason to go there is not to figure out what icons we have according to synonym or size, but to get them for use. The sources we provide for that are the sketch library that contains them or, for development, the way to use them from the carbon elements package.

If the cards themselves don't work for folks at the top of the page, please let's figure out an resource download embodiment that does, perhaps a half height card for example? Simply moving the cards to the end of the page is probably the same as removing them, given the length of that page.

@chrisconnors-ibm i totally get what you're saying, but what about people who don't have Sketch?

Also not bottom of long page but the results of filtering or search. It would make the cards right under the icon/s one searched for.
But to your point. Why did we add this to library if you can download the whole master Ai file from the repo? I know Conrad is working on a final master. This eliminates the need for this pattern.
Orinally we had ‘copy’ as the only functionality per icon. Can we not go back to that?
@jeannie I think the non Sketch users (there are many) can download the master and get them all!

An Illustrator Master is generally not used  in service of design/spec docs, but in any case should be generated out of what is in carbon-elements Icon package programmatically, as our carbon-elements sketch library will be soon.
The problem with assets (illustrator, sketch, adobe XD etc.) created by hand is they have to be maintained and kept in sync by hand, which is already probably impossible given that our 32x32 SVG asset count alone is 645 individual svg files.
But the thing we need to keep front of mind is that when @Conrad and I or @mjabbink and I and @jeanservaas decide some specific icon is janky, we only need to change it in its repo, and then the sketch library updates, and its symbol is updated, and design docs using that symbol are updated, and built applications already shipped that use the package path to that asset get updated, etc.

Can we generate Ai file with the whole lot programmatically? That would be awesome. I’d be cool with this but there are a lot of users not using Sketch that may need the icons and one by one downloads won’t be great. Can the repo download the master folder with all svgs in it?
Also the Ai master is nice for anyone also creating some new ones.
Hmm

I don't know if generating an Adobe Illustrator master file based on repo contents is possible, but that's what we're investigating on the Sketch kit side. If we could generate a master .ai file from those contents, it gets us closer to the single source of truth we're generally striving for.
the 'creating new ones' is interesting in that it doesn't have the same requirement to pick up future changes behaviors that we want in our Sketch doc creation or our production environments.

re-opening -- need clarity around what functionality should be limited on IDL site


[feature request] NodePool Governance Capability: pool-coordinator controller need to deal with a sence that nodepool disconnect with cloud

What would you like to be added:
pool-coordinator controller need to deal a sence when nodepool disconnect with cloud, the node in nodepool can be autonomy, the pod in node cannot be evict.

node actually down, close the node autonomy, the pod binding controller cannot use, the pod in node can be evict.
nodepool disconnect cloud, controller can add node autonomy for nodepool, then the pod binding controller will working, k8s cannot evict the pod.

Why is this needed:
On the current situation, when nodepool disconnect with cloud, if node does not open autonomy, the pod in node wil be evict. In another way, if the node open autonomy, when node actually down, the pod binding controller will not evict the pod, it`s different from nodepool governance.
others
/kind feature

@JameKeal Thank you for raising issue. yes, we need to consider how to handle pods eviction when the network between cloud and all nodes in nodePool disconnected. I think we need to keep the pods un-evicted at that time.

@gnunu @LindaYu17 do you have any comments?

Any progress? Should nodelifecycle be considered since zone disruption can make the minimal change to handle this case. (refer to partialDisruption and FullDisruption in kube node lifecycle controller)

Any progress? Should nodelifecycle be considered since zone disruption can make the minimal change to handle this case. (refer to partialDisruption and FullDisruption in kube node lifecycle controller)

@AndyEWang Do you have any suggestions how to integrate original partialDisruption and FullDisruption into NodePool in OpenYurt?

Remember we talked about this issue during our earlier discussion about pool coordinator design, then we decided to let apiserver decide what to do when whole pool is lost.
However it's natural that pods should not be evicted on pool disconnection occurring. Anyway pool should be in automate mode if pods are pool binding. So the implementation should take care of this requirement.
How about we add this capability during our migration to yurt-controller?

Personally, OpenYurt can take charge of node lifecycle like old versions before v1.2. If not, we must find out many workarounds against kube-controller-manager. Should OpenYurt give an option to let users decide which node lifecycle to use?

Personally, OpenYurt can take charge of node lifecycle like old versions before v1.2. If not, we must find out many workarounds against kube-controller-manager. Should OpenYurt give an option to let users decide which node lifecycle to use?

@AndyEWang I think It's not a good idea to maintain two selections for nodelifecycle contorller because it will be very difficult to maintain OpenYurt.
by the way, the original solution also can not satisfy the requirements when nodepool disconnects with cloud kube-apiserver, and we need to add webhooks and controllers to handle nodepool network disconnection too.

Remember we talked about this issue during our earlier discussion about pool coordinator design, then we decided to let apiserver decide what to do when whole pool is lost.
However it's natural that pods should not be evicted on pool disconnection occurring. Anyway pool should be in automate mode if pods are pool binding. So the implementation should take care of this requirement.
How about we add this capability during our migration to yurt-controller?

@gnunu @JameKeal is migrating pull request #1234 which handle this case into yurt-manager.


Weird suggestions

I installed this plugin today and it has been working fine until I jumped into another project.
Getting the following suggestions, and I don't know where they're coming from:

Seems like I should add some additional filtering for long strings with numbers in them.
You might be able to fix it by creating a .ignore file (to make TabNine skip the node_modules while keeping them checked in to Git).

I have the following .gitignore:
node_modules/
dist/


Re-export esp-idf-part

Library users may need to parse partition tables and it is surprising that the partition table is in a separate crate.
This PR re-exports the esp-idf-part crate for better developer experience. This way the users don't need to add the dependency and keep it in sync in the future.

I had considered doing this previously, however this would mean that any semver-incompatible changes in esp-idf-part will force us to bump the major version here. Given that esp-idf-part is currently on 0.1.x I'm a little hesitant to do this, but I will think on it some more.

any semver-incompatible changes in esp-idf-part will force us to bump the major version here

Fair point, cargo-semver-checks to the rescue :P Jokes aside, pinning the dependency version would probably allow a breakage like this to be fixed.
The alternative is exposing the user to a weird error: if espflash depends on some version of esp-idf-part and the user manually adds another version, it's possible to end up with a compile error that goes something like "expected type X but found type X" which is a real head-scratcher for newcomers. But I don't expect many newcomers to start integrating espflash into their own projects, so this may not be an issue. 🤷♂
Up to you. While I would find this a bit more convenient, semver concerns probably hold more weight.

In our community meeting this week there were some changes requested for esp-idf-part, so I will be holding off on re-exporting it at least until this work is done.


Add new UV api

http://openweathermap.org/api_uv

Hi, are you already working on this? Otherwise I'd be glad to do it and send in a pull request.

Hi Felix, I have not started working on it and probably won't for the next
weeks. I'd be more than happy to receive a pull request though!
Felix Rauch<EMAIL_ADDRESS>schrieb am Mi., 18. Nov. 2015 um
10:31 Uhr:

Hi, are you already working on this? Otherwise I'd be glad to do it and
send in a pull request.
—
Reply to this email directly or view it on GitHub
https://github.com/cmfcmf/OpenWeatherMap-PHP-Api/issues/48#issuecomment-157655839
.

UV api is still a beta feature now.
The example call in http://openweathermap.org/api_uv is not available. I also get the 502 bad gateway.
And I also found another url: http://openweathermap.org/api/uvi.
It seems the new version of uv api now.

I implement this API feature now.
Please check this.
Thanks.


Execute only changed commands

This makes commands only be executed if they changed.
Fixes #114

About extracting the "repasing", I think there is generally a lot that can be refactored do avoid duplicate code. E.g. it seems that a message is parsed once in each handler, while it could be parsed only once and the result of that passed to each handler.

I agree -- initially the duplication happened due to a lack of knowledge about what the right abstraction is, and I think that's still (to some extent) true -- not all of our handlers have the same handling. But I do think in this case it's pretty clear that the code is essentially identical between all the places, so deduplicating it makes sense.

I can try to extract common code to a common place, but I'd rather do so in another PR.
Btw are multiple command in the same comment supported ?

I forget (would need to look at code). It's not an explicit goal, though would of course be a nice to have.
I think this looks good now, though, so merging (and it will automatically deploy soon thereafter).


Minimum python version documentation

Added a sentence to mention the minimum Python version, fixes #996.
My first PR! Any feedback welcome ^_^

Thanks! :duck:


[internal] Apply xref links for southworks/add/jsdoc/botbuilder-skills

PR 2818 in MS, #143 in SW
Feedback applied to use xref to link to other methods.
note that the links will not work in visual studio, these links are meant to be used to build the web documentation.
searchable here https://docs.microsoft.com/en-us/javascript/api/
This PR doesn't have conflicts, I didn't update the branch with main here to don't bring irrelevant changes.

Corrected, thanks @ceciliaavila!


fix(ci): Clean lxc instances on bundle integration tests

More details on the issue.
Fix #602
Testing
This is essentially verified on the PR's CI run. In order to verify that it doesn't affect running locally (without --clean-lxc-instances), you can try triggering a run without it. Note that the CI can fail due to #601.

After discussions with @NohaIhab , we decided to drop this PR in favor of #616 for a more deterministic and less error prone approach, given that the script did some assumptions (e.g. the startswith("charmcraft") part)
    lxc_instances_charmcraft = jq.compile('.[] | select(.name | startswith("charmcraft-")) | .name').input_text(lxc_instances).all()


feat: add compose run command

Signed-off-by: Min Uk Lee<EMAIL_ADDRESS>close #270
Changes (for more details, see each item below):

remove -d from default RunArgs of serviceparser.Container
add apt install expect into docker-compatibility test
add Base.ComposeCmdWithHelper() into testutil
add compose run command (no details below)
add compose run docs into README.md (no details below)

Unimplmented Flags:

--no-TTY, --use-aliases

Not tested Flags:
- volumes, entrypoint
(I'll add additional test commit as soon as possible. I'm trying to figure out how to test it properly.) (Add test cases)
Details
remove -d from default RunArgs of serviceparser.Container

Previously, composer.upServiceContinaer() uses -d flag to find cid.

-d, --detach                      Run container in background and print container ID

Because of it, cannot support --interactive flag of compose run and StdinOpen option of compose-spec
Remove -d from default RunArgs and explicitly add property of container. Containers are now possible to determine whether or not to detach when they really start to run.
Alternatively, use --cid-file flag to find it.

add apt install expect into docker-compatibility test

Currently, docker-compatibility test does not contain unbuffer command to emulate tty.

I found unbuffer command from a test for run command.

And even some tests are skipped because unbuffer is not installed.

TestIssue108
TestExecTTY
TestRunTTY

add Base.ComposeCmdWithHelper() into testutil

As I mentioned, unbuffer needs to emulate tty to test compose run.
Basically, I referenced Base.CmdWithHelper()

I'm finding the reason why docker compatibility test are failed
As soon as possible, I'll fix it and push again.

I added some test cases for --entrypoint, --volume.
Could anyone review, please?

Found a minor incompatibility: docker compose run randomize container name (compose-wordpress_db_run_cca27158caa2 ) but nerdctl compose run does not (`compose-wordpress_db_1).

LGTM

@AkihiroSuda could you plz ask for squashing commits before merging the PR in the future :) !

@AkihiroSuda could you plz ask for squashing commits before merging the PR in the future :) !

In this case having separate commits is fine, as each of the commits has a separate topic.
Add tests and docs for compose run command  could be meld into Add compose run command template though.


Add capability to run openmp with K matrix.

It isn't super hard. You just need to remove some NULL() pointers to nullify. Make a patch to CRTM, and changes to interface to make it all work.

fixed with merge #23.


Add support for AWS bedrock claude3 opus model support

Add support for AWS bedrock claude3 opus model support

@wmz7year , can you point me to a reference suggesting that AWS bedrock already supports Anthropic Claude3  Opus ? and for which reason?

@tzolov Check this out: https://aws.amazon.com/about-aws/whats-new/2024/04/anthropics-claude-3-opus-amazon-bedrock/
But currently, it's only available in the US West (Oregon) AWS Region.

Thanks for the information @wmz7year and thank you for the contribution.


Roll Plugins from 0892a2d292c2 to e52725a5f8f5 (1 revision)

https://github.com/flutter/plugins/compare/0892a2d292c2...e52725a5f8f5
2021-04-19<EMAIL_ADDRESS>Replace path_provider_linux widget tests with simple unit tests (flutter/plugins#3812)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


How to change port for two model

I use docker-compose to establish my label studio ml. I have two model，one is yolov8，anthor is sam.
In yolov8，I use port 9090：9090.  In sam，I use port 9091：9091 （both in docker-compose.yml and Dockerfile）
But it comes to an error ：sam

it seems to listen to 9090. In docker-compose.yml and Dockerfile change does not work. How can I edit port for sam.

I can go into image  and use
label-studio-ml start my_ml_backend -p 9091   to change port.
but this is not the best way  I think.


Scrot: Automatically detect tools

Description
This PR adds a new function called scrot_program() which automatically detects screenshot tools on your system. Users no longer have to specify which tool to use in their config file.
This only supports systems with an X server, we can add support for macOS and Windows if we can figure out how to take screenshots on these machines from the commandline.
One of the tools in the scrot_program() function is most likely already installed so for majority of people this will be plug and play.
The scrot_cmd config option will still exist and still work if users want more control over which program/flags to use.
Features

Automatically use whatever screenshot tool is available.
Added screenshot support to macOS
Added screenshot support to Haiku

Issues

This function supports every OS but Windows. There's no builtin way on Windows to take screenshots from the command line. CYGWIN + an X server will still work on Windows.

Works on macOS!


Use pixi instead of poetry

Current dependencies on/for this PR:

main

PR #8   👈

This stack of pull requests is managed by Graphite.


dry-run seems to ignore the -n flag

Describe the bug
I want to be able to dry-run my vela application so I'm able to deliver a "pure K8S" manifest.
This app plays around two custom traits, which use the "context.namespace" namespace (so, no hardcoded namespace there)
I did manage to run it on both online and offline modes, and the manifest itself runs fine.
On the other hand, it seems the -n flag (to directly set a namespace in the manifest instead of using the default namespace) is ignored.
To Reproduce
vela dry-run --offline -n cis -d ../kubevela-conf/definitions/ -f ../kubevela-conf/first-hotstandby-app.yaml
Expected behavior
All fields pertaining to a namespace are set with "cis" instead of "default"
KubeVela Version
$ vela version
CLI Version: 1.8.2
Core Version: 1.8.2
GitRevision: git-360f69be
GolangVersion: go1.19.9

I think it is fixed in CLI Version: 1.9.6, please upgrade the cli and try again.

Uninstalled 1.8.2, downloaded 1.9.6 and installed it.
Running the same command yields the expected result : The -n flag is taken into account.
Thanks ! :D


Caching weather results

Currently the weather is refreshed for every page load with this script. Is there a way to cache the weather results for X minutes/hours?

Currently it doesn't cache the results. It is something I just haven't gotten to yet.


Added hidelinks flag to hyperref usage to links are hidden

Some PDF readers make links make your paper look awful with these big ugly boxes around them. When I had my paper formatting reviewed, they told me to remove the usage of yellow and green and blue coloring from my paper which I was unaware of, and this was the cause.
I enabled the hidelinks flag in the hyperref package in order to stop this from happening (link here).
Here is what my paper looked like before and after in one offending pdf reader:
Before:

After:

Does this disable links altogether? Or just remove the boxes?

It just removes the horrifically ugly boxes, the links still work. Documentation here: http://mirrors.concertpass.com/tex-archive/macros/latex/contrib/hyperref/doc/manual.pdf

@Jdban cool, can you rebase and I'll merge?

Sorry for the delay, one more time?

It is really annoying that Github doesn't notify when a rebase is done. No wonder I missed it the first time.

Oh that's annoying. Good to know.


Compile-time checking of string literal arguments based on type

Often a method will take as a string the name of a property of some object that it's working on. A common example would be http://backbonejs.org/#Model-get
It would be ideal if TypeScript could provide compile-time checking on these calls to make sure that the string is actually a valid property of the underlying model. As a possible proposal:
interface Person {
	firstName: string
	lastName: string
	phoneNumber: string
}

function get(property: memberof Person) {}

get('firstName') // compiles
get('middleName') // would not compile

In the context of the get function, "memberof Person" would be equivalent to "string".

I am currently using Baobab for a project, it is an immutable data tree / cursor library. When using it I have to use something like this (non-working simplified example):
type Foo = {
   a: number;
};
get<T>(property: string): T;

// elsewhere
get<number>('a');

This for one does not check that the property actually exist and also forces me to cast the get to the correct type every time, which can lead to error for example if I change a in Foo to string my get will still be casting it to number.
This suggestion would therefore be really useful to help keep my code as type checked as possible.
I like this proposal from @jbrantly:
get(property: memberof T is A): A;
set(property: memberof T is A, value: A);

My proposal would be something like this:
get(property: A memberof T): A;
set(property: A memberof T, value: A);

The righthand would need to be an object type ({...}, interface, Class) or it would throw an error.
The lefthand side would be optional and only used if you need a reference to the type.
If you try to pass something that doesn't resolve to a constant string to a memberof type it would throw an error.
Or a way to reference a subtype using the parameter name:
get(property: memberof T): T[property];
set(property: memberof T, value: T[property]);

the proposal in #1295 should cover the scenarios outlined in this issue.

Looks like this is largely solved by string literal types: https://github.com/Microsoft/TypeScript/wiki/What's-new-in-TypeScript#string-literal-types ?

Not really, but I think at least it is a prerequisite to solve this issue. #10425 will do.


[IOTDB-5514] Fix race condition in MemoryPool without synchronized block

See JIRA: https://issues.apache.org/jira/browse/IOTDB-5514

@lancelly @JackieTien97 PTAL

LGTM~


fix(Stepper): always show active Step's StepLabel

FX-3885
EXP-2638
Description
This PR fixes the issue where the StepLabel of an active Step is hidden on mobile resolutions
How to test

visit https://picasso.toptal.net/EXP-2638-always-show-active-stepper-label-on-mobile/?path=/story/components-stepper--stepper
resize the page to a width below 576px
verify that only the labels for non-active step are hidden

Screenshots

Before.
After.

Development checks

[x] Add changeset according to guidelines (if needed)
[x] Read CONTRIBUTING.md and Component API principles
[x] Make sure that additions and changes on design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[n/a] Annotate all props in component with documentation
[n/a] Create examples for component
[ ] Ensure that deployed demo has expected results and good examples
[n/a] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[ ] Covered with tests

Breaking change

[n/a] codemod is created and showcased in the changeset
[n/a] test alpha package of Picasso in StaffPortal

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers


Allow package name to be different from repository name

It is not uncommon for importable packages to have different names than the repositories (or distributions) that hold them, so {{cookiecutter.repo_name}}/{{cookiecutter.repo_name}} should use two distinct variables.
project_name is not a drop-in for either, since it can include spaces and capital letters.
It seems like maybe there should be an additional option declared between repo_name and first_module_name.
Also, it seems a bit odd to default to creating the first module as projectname/projectname.py. Would it be reasonable to change the default to the proposal from the workshop material? (i.e. functions?)
{
    "project_name": "ProjectName",
    "repo_name": "{{ cookiecutter.project_name.lower().replace(' ', '_') }}",
    "package_name": "{{ cookiecutter.repo_name.lower().replace(' ', '_') }}",
    "first_module_name": "functions",
...

Then, the template directory structure can be updated to
{{cookiecutter.repo_name}}/{{cookiecutter.package_name}}

For those with access, see also https://molssiedu.slack.com/archives/C03Q6GCCZ1B/p1659622847027609


'Sort on save' not working with css files

Environment
Info

WindiCSS IntelliSense
0.18.3

vscode
1.56.2

System
macos 11.4

Browser
google chrome

issue: 'Sort on save' not working.

test/test-ghe-restore.sh

please provide more information of your project and setup

dzimidrol5:main

should be fixed with newer version of the extension


Add option to ignore columns in equality test

resolves #734
This is a:

[ ] documentation update
[ ] bug fix with no breaking changes
[x] new functionality
[ ] a breaking change

All pull requests from community contributors should target the main branch (default).
Description & motivation
This is to include an option to ignore columns in the equality test. Currently the only way to do this is to use the compare_columns option and include all columns we want to compare, leaving out any that we want to exclude. In certain cases this might result in large yml files, difficult to maintain. The ignore_columns option fixes that.
Checklist

[x] This code is associated with an Issue which has been triaged and accepted for development.
[x] I have verified that these changes work locally on the following warehouses (Note: it's okay if you do not have access to all warehouses, this helps us understand what has been covered)

[x] BigQuery
[x] Postgres
[ ] Redshift
[x] Snowflake

[ ] I followed guidelines to ensure that my changes will work on "non-core" adapters by:

[ ] dispatching any new macro(s) so non-core adapters can also use them (e.g. the star() source)
[ ] using the limit_zero() macro in place of the literal string: limit 0
[ ] using dbt.type_* macros instead of explicit datatypes (e.g. dbt.type_timestamp() instead of TIMESTAMP

[x] I have updated the README.md (if applicable)
[x] I have added tests & descriptions to my models (and macros if applicable)
[x] I have added an entry to CHANGELOG.md

Thanks @brunocostalopes - seems like something odd is happening with the CI, I'll have to look into that! Code itself looks good though 🎉

Hello! Nice, would love this feature! Just a suggestion:
don't you think you could remove the elements from compare_columns instead of creating a second list ?
the loop would be:
for column in ignore_columns:
compare_columns.remove(column)
So it would use only existing objects and loop only on ignore_columns elements (which schould be fewer than compare_columns ones) .
I do not really use Jinja a lot but the remove syntax is a Python one so it is maybe possible.

@moreaupascal56 thanks for the tip. I think you are right. If we remove instead of append it makes the code neater and I've pushed those changes.
I've also tried your other suggestion of looping through the elements in the ignore list first but that would make the code a bit more complex when removing from the original compare list so I've decided against it as I don't think the potential performance gains would be that relevant here.
@joellabes seems like the checks are passing again, thank you for looking into it. I think the PR is now ready for your review.

Excited to see this come through - have been running into a need for this during legacy code conversion.

@joellabes @deanna-minnick Can this PR get reviewed and approved?  I'd love to have this feature available.

I've just realised that the latest change I've pushed has a bug which I can't explain.
This is the code with the issue:
    {%- for column in compare_columns -%}
        {%- if column | lower in ignore_columns -%}
            {% do compare_columns.remove(column) %}
        {%- endif %}
    {%- endfor %}

whenever a match is found a column is removed from compare_columns, the next column in the list is ignored and it is not tested against the ignore_columns list.
I can't understand why this is happening. If anyone has a clue please let me know, I would be happy to fix. In the meantime, I'm going to revert back to the previous version which, even thought it is not the tidiest implementation, it seems to work well.

I've just realised that the latest change I've pushed has a bug which I can't explain.
This is the code with the issue:
    {%- for column in compare_columns -%}
        {%- if column | lower in ignore_columns -%}
            {% do compare_columns.remove(column) %}
        {%- endif %}
    {%- endfor %}

whenever a match is found and a column is removed from compare_columns, the next column in the list is ignored and it is not tested against the ignore_columns list.
I can't understand why this is happening. If anyone has a clue please let me know, I would be happy to fix. In the meantime, I'm going to revert back to the previous version which, even thought it is not the tidiest implementation, it seems to work well.

I believe @moreaupascal56's suggestion was that you wouldn't loop over compare_columns anyway.  Why not try it like this?  Should avoid the bug you're seeing and be a faster loop anyway.
    {%- for column in ignore_columns -%}
        {%- if column | lower in compare_columns -%}
            {% do compare_columns.remove(column) %}
        {%- endif %}
    {%- endfor %}


ESP-IDF v4.4.4 was used to evaluate m5stack_sample (mros2-esp32 v0.5.0 sample code)

I am trying mros2-esp32 v0.5.0.
An error occurs when building M5Unified v0.1.7 with the ESP-IDF v5.2.
Using ESP-IDF v4.4.4 can avoid M5Unified build errors.
@s-hosoai : Thank you for your cooperation.
Pub/sub worked with M5Stack Core2 for AWS and Seeed Studio XIAO ESP32C3.

Pub/sub worked with M5Stack Core2 for AWS and Seeed Studio XIAO ESP32C3.

Pub/sub worked with M5Stack Core2 for AWS and Seeed Studio XIAO ESP32C3.

@yamati-kz thank you again and again!!
I think the information for the version is essential and important. So I added it on READM by c3526ab

@takasehideki
I have seen the README. I appreciate it very much.


DO NOT MERGE: remove shared redis config option

refs travis-pro/team-teal#2064

Going to keep the two separate ones for now, as it is also more friendly to enterprise.


#2 Using new fields in lucene index to return more known locations

Algorithm is as per description in -
https://github.com/chrismattmann/lucene-geo-gazetteer/issues/2
Weights are calculated based on occurrence in “name” and “alternate
name“. Priority is given to results ranking higher as per sort applied
in algorithm

@smadha looking great, please see my comments above!

great work! merged


Ability to override ID operations

Description of your changes
Allows users to override the default TF State ID -> External Name and External Name -> TF State ID flows. Note that we already have SetIdentifierArgumentFn endpoint but that's for overriding External Name -> Main TF File. Details about why we're doing this are in the issue.
Depends on https://github.com/crossplane-contrib/terrajet/pull/128 . See the last 3 commits for actual diff.
Fixes https://github.com/crossplane-contrib/terrajet/issues/119
I have:

[x] Read and followed Crossplane's [contribution process].
[x] Run make reviewable to ensure this PR is ready for review.
[x] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested
Unit tests. I'll add a draft PR in one of TF provider using this.

@muvaf would this mean, users no longer reliably use Terraform import id as an external name? I was liking that what to use as an external name is documented for all resources, especially for import cases.
If we chose to still use Terraform import ids as external names, I believe we could cover all cases with two functions as below:

CR (metadata.name + spec) => identifier attributes (e.g. attributes["id"] and attributes["bucket-name"] set)
attributes["id"] -> CR (external-name and spec)

I think this would mean us not marking some (required) fields as not required so that they can be late initialized during import.

Synced with @turkenh about his proposal and we decided not to proceed with it because it'd mean to make external name same as ID which is not acceptable in some cases especially in composition where you may not have some parts of the information stored in the ID, such as subscriptionID in Azure.
@ulucinar @turkenh I added your suggestion about the provider config content and opened https://github.com/crossplane-contrib/provider-tf-azure/pull/84 to show how it all would look like.

@ulucinar @turkenh I think I have responded to all comments and made changes, ready for another look 🙂


If the setUser function is used in a react hook dependency array, it causes an infinite loop

Hello there 👋
I believe that the setUser function that is returned from the useBaselimeRum hook needs to be wrapped with useCallback inside the library as currently it can result in an infinite loop when used inside useEffect
You can reproduce this easily with some code like this:
const { setUser } = useBaselimeRum();

useEffect(() => {
  console.log("in effect");
  setUser('fake-user-id');
}, [setUser])

The workaround for now is to ignore the linting warning about adding setUser to the dependency array, but it would be nice if we didn't need to have this workaround.
I would be happy to look into opening a PR for this fix!

Nice, Thanks so much for raising this! :) Taking a look at the PR now


Lack of 'Refresh' item for storage accounts

Platform: Mac/Windows/Linux
Repro Steps:
Right click one storage account and check the context menu.
Expect:
Display 'Refresh'.
Actual:
No 'Refresh' item displays.

More Info:
The 'Refresh' refreshes the applied subscriptions actually.

Sorry for logging this issue in a wrong area path, this is a Azure Storage issue :( So we close it.


change location of .pkg-cache

how can i tell pkg to look somewhere else for fetched-v8.11.3-win-x64 in the C:\Users%user%.pkg-cache\v2.5 directory? Im trying to user pkg on a jenkins job with no internet and no access to user directories.

Use the environment variable PKG_CACHE_PATH when calling pkg. For Windows:
set PKG_CACHE_PATH=C:\myPkgCachePath
or mac/linux:
PKG_CACHE_PATH=/myPkgPath pkg ...


Merge code from pyxis branch

The pyxis branch contains code for LightBlue decommission, we're ready to merge the code.

This PR contains the code from pyxis branch (which is rebased), with one addition commit of "Decrease the number of default page size for Pyxis queries" (changed default Pyxis query page size from 250 to 200).


[Rebranch] Disable storage3 check in NewStringAppending test

The other instances had been disabled in commit 5236224eb16a.
FileCheck was recently updated so that it complains about undefined
variables, which is why no one noticed until now. storage3 wasn't
defined because the place where it got defined was disabled. Disabling
that instance now.
rdar://82052117

This needs to go in to unblock work. Merging. If there are issues, please file a new pull request.


Added OpenAI cli checker

Adds a new cli checker that allows using an OpenAI prompt to check source strings against.
The checker sends a request to a Mojito endpoint, which forwards the request to the configured OpenAI endpoint.

One last small note: non OpenAI-specific files should probably moved from the openai folder into a separate one or a level up, e.g.: webapp/src/main/java/com/box/l10n/mojito/service/ai/openai/AIPromptContextMessageRepository.java (there's a set of these)

Done!


question: how to handle request response status

About .request, maybe I'm missing something, then and catch are working correctly but I never see err populated, always empty, and while I get the response object from then I can't check the server status which may return an application error (eg: 422 data validation failed).
Currently I'm using a workaround saving xhr from config: function(xhr) but I guess this is not how is supposed to work, what I'm missing?

I can see the convenience in having it trigger the then/catch block... thinking of adding this as either, 1. 400+ codes pass the xhr object directly to the then handler or 2. pass it to the catch handler.

will open a way to do the following

dio.request('404').then(function (data) {
	if (data && data.status >= 400) {
		throw data.status;
	} else {

	}
}).catch(function (err) {
	console.log(err);
});

and 2.
dio.request('404').then(function (data) {
	// silence
}).catch(function (err) {
	console.log(err.status);
});

I like both but solution 2. is more compact and more semantically correct,  errors are handled by catch (something users would expect without reading any docutmentation I guess).
I'm not sure .status is enough or always populated on network errors, for instance if there isn't a service listening (net::ERR_CONNECTION_REFUSED) xhr.status is 0.

Yes if the server hangs and doesn't send a response status might be 0 so maybe  xhr.status !== 0 || xhr.status >= 400 would be better.

Option 2 is the correct way

added this, works like option 2 - changelog for more details.

Notice that currently status code like 421 422 (used in some API for application level error) are sent through then which wouldn't have access to .status in 2. implementation.

anything >= 400 should find itself in the catch handler, the catch handler can then determine what to do with the response and status in the xhr object, or maybe i'm not getting your point?

= 400 should do, gonna give a try today, thanks!

ps: do you plan to support typescript with a .d.ts?

ps: do you plan to support typescript with a .d.ts?

@xenogenesi now that you mention it, yes. backing issue tracker

.request seems working as expected, nice! the npm minified version though seems still 5.0.3

now that you mention it, yes

Good to know, looking forward to it
The issue can be closed for me, thanks again.

@xenogenesi Thanks, published 5.0.5 that fixes that.


Fix a tiny typo in the CLI help text.

Issue Number
None
Overview

[x] I have fixed a tiny typo in the CLI help text.

bors r+