Correctly Configure Eureka Client Hostname On Cloud Foundry

I deployed a Eureka Client to cloud foundry, however when it registers with the Eureka server (also running on CF) it does not register it with the a route bound to the application.  (I am actually not really sure where it is getting the hostname it is registering with.  I am sure the hostname the container thinks it has is not the route that the CF router mapped to the application.)  I am wondering if there is a best practice for dealing with this?
I got something working by setting eureka.instance.hostname to ${vcap.application.application_uris[0]}.  However I am not sure how robust of a solution this actually is, especially since I could have multiple routes bound to the application.

I have something similar to this issue,  the difference being this is on AWS.  Right now Eureka is pulling the private hostname ip-192-0-0-1  How would you get the public hostname.  If I set this via the eureka.instance.hostname and Amazon reboots me I get a new hostname but it would be hardwired in the properties.

@ccit-spence netflix uses elastic IP's for eureka, how are client's going to know how to connect to eureka?  CF takes care of that, but on AWS, you'll have a problem.  What are you planning on doing?

@spencergibb I am using elastic IP's for eureka.  Eureka instances are ok since I am using elastic IP's. What I am seeing is that other services are registering with the private hostname instead of the public hostname.

If you use Spring Cloud out of the box it will not use the Netflix EurekaInstanceConfigImpl (which would default to being AWS datacenter aware). You could set up your own bean of that type, or use the Spring one (EurekaInstanceConfigBean) and inject the datacenter metadata in your configuration. I believe the Netflix native datacenter info does it through an API call at runtime (so it needs API credentials).
Is the private IP not a decent default anyway (services in AWS can contact each other directly that way)? I would also assume that the public IP is available through an API call or environment variable (maybe only the latter if you set your EC2 instance up to populate it on startup)?

Hey @ccit-spence did you resolve your issue? I'm currently trying to run some services using spring-boot cloud on aws cloud and facing same problem.

@dsyer Regarding private vs. public ips: The problem here is that we have a mix of private ips and public eips. The discovery client registers the eureka servers with their private ip but the replication is using public eips. I think that's the reason for the registered replicas / available replicas mismatch.
See also: http://stackoverflow.com/q/36284246/1686330
Any hit how i can get eureka to use only private or only public ips?

@DirkLachowski use following snippet:
public EurekaInstanceConfigBean eurekaInstanceConfig() {
    EurekaInstanceConfigBean config = new EurekaInstanceConfigBean();
    AmazonInfo info = AmazonInfo.Builder.newBuilder().autoBuild("eureka");
    config.setDataCenterInfo(info);
    info.getMetadata()
            .put(AmazonInfo.MetaDataKey.publicHostname.getName(), info.get(AmazonInfo.MetaDataKey.publicIpv4));
    config.setHostname(info.get(AmazonInfo.MetaDataKey.publicHostname));
    config.setIpAddress(info.get(AmazonInfo.MetaDataKey.publicIpv4));
    config.setNonSecurePort(port);
    return config;
}

Instead of publicIpv4 you may use local one.

@rozhok thanks for the hint. Still not exactly what i was looking for but the missing hint i needed. I ended up with the following:
@SpringBootApplication
@EnableEurekaServer
@EnableDiscoveryClient
public class EurekaServer {

	@Value("${server.port:8761}")
	private int port;

	public static void main(String[] args) {
		SpringApplication.run(EurekaServer.class, args);
	}

	@Bean
	@Autowired
	public EurekaInstanceConfigBean eurekaInstanceConfig(InetUtils inetUtils) {
	  EurekaInstanceConfigBean config = new EurekaInstanceConfigBean(inetUtils);
	  AmazonInfo info = AmazonInfo.Builder.newBuilder().autoBuild("eureka");
	  config.setHostname(info.get(AmazonInfo.MetaDataKey.publicHostname));
	  config.setIpAddress(info.get(AmazonInfo.MetaDataKey.publicIpv4));
	  config.setNonSecurePort(port);
	  config.setDataCenterInfo(info);
	  return config;
	}
}

@DirkLachowski you don't need @Autowired for bean. Also, why you have @EnableEurekaServer? The snippet I provided is for clients only.

@rozhok First of all, i have no question (at least not more). Your code snipped was the one missing piece for me, many thanks for that (I'm creating a dns based multi zone eureka setup).
Regarding Autowiring: You need an instance of InetUtils for the public ctor of EurekaInstanceConfigBean (and that is autowired, an instance of InetUtils is created by autoconfig). Your code shouldn't compile, the no-args ctor is private (see https://github.com/spring-cloud/spring-cloud-netflix/blob/master/spring-cloud-netflix-eureka-client/src/main/java/org/springframework/cloud/netflix/eureka/EurekaInstanceConfigBean.java#L284 ).
Regarding @EnableEurekaServer: A eureka server is also a eureka client (used for the registered vs available stuff), so i had a eureka client problem but for a eureka server.
Never mind, works perfect now. Thanks again for your help.

@DirkLachowski sorry bud, thought you have issues with my snippet :) Btw, I took my sample from actual code, maybe it compiles because  I use a bit old version of spring-cloud-netflix.

I am having an issue that might be related. I deployed my eureka service and my services that supposed to connect but without look.
Here is the config in my eureka-service:
spring.application.name=eureka-service
eureka.client.register-with-eureka=false
eureka.client.fetch-registry=false
eureka.server.enableSelfPreservation=true
And here is one of my client:
eureka.client.service-url.default-zone = ${EUREKA_URI:https://myapp.cfapps.io/eureka}
eureka.instance.[nonSecurePortEnabled,securePortEnabled] = [false, true]
eureka.instance.hostname=${vcap.application.uris[0]}
I can see the eureka page but not service register. Could someone help me please?

@richardvaldiviesomacias You probably misconfigured eureka ip on the client side.
${EUREKA_URI:https://myapp.cfapps.io/eureka} ?
Is this expression points to actual eureka ip?

@richardvaldiviesomacias please open a new issue for your problem

@rozhok @DirkLachowski : Thanks, Worked for me as well on AWS.
This is my config without "DNS".
eureka:
datacenter: cloud
client:
registerWithEureka: true
fetchRegistry: true
region: eu-west-1
eurekaServerPort: ${server.port}
useDnsForFetchingServiceUrls: false
availabilityZones:
eu-west-1: 'eu-west-1a,eu-west-1b,eu-west-1c'
serviceUrl:
eu-west-1a: http:/-west-1.compute.amazonaws.com:${server.port}/eureka
eu-west-1b: http://-west-1.compute.amazonaws.com:${server.port}/eureka
eu-west-1c: http://-west-1.compute.amazonaws.com:${server.port}/eureka

Works for me (no config needed for hostname). What version of Spring Cloud are you using? Are you adding your own EurekaInstanceConfig bean or just using the default? Here's a eureka on PWS (username:password) that shows it working: http://eureka.cfapps.io/eureka/apps/configserver.

Could you please share, which version is suitable?

@ryanjbaxter I am also using the service registry from PCF marketplace. But i couldn't able to register my client in eureka server. could you please share the yml config file and also share the working project repository if you have

That is something different.  The service registry from the PCF marketplace is our productized version of Spring Cloud called Spring Cloud Services.  Is you are having an issue with that then you should reach out to https://pivotal.io/support

@ryanjbaxter Thanks for your quick response...
Actually i have created the Service Registry from PCF marketplace and bind my application to that service. But still i couldn't see my client application in eureka service registry. Do we have to mention the eureka service details in client application.yml file. Please share what are all the things that needs to be configured in client application.

Please see the appropriate documentation of the commercial project https://docs.pivotal.io/spring-cloud-services/1-5/common/service-registry/index.html

@ryanjbaxter  It is working now.. Eureka client got registered in PCF service Registry. But when i am trying to communicate b/w two client which is registered in service registry, am getting   "com.netflix.client.ClientException: Load balancer does not have available server for client: currexchange"
Note: Am using @FeignClient("https://currexchange"). Please suggest me how to use FeginClient with service registry.
Thanks in advance!!

There are plenty of examples in our documentation, please read the documentation and if you are still having an issue open an issues in Spring Cloud OpenFeign

`@Component
public class PodEnvironmentAware implements EnvironmentAware {
private static final Logger LOGGER = LoggerFactory.getLogger(PodEnvironmentAware.class);
@Override
public void setEnvironment(Environment environment) {
    Properties props = new Properties();
    try {
        props.setProperty("POD_OWN_IP_ADDRESS", InetAddress.getLocalHost().getHostAddress());
        PropertiesPropertySource propertySource = new PropertiesPropertySource("K8S", props);
        if (environment instanceof StandardEnvironment) {
            ((StandardEnvironment) environment).getPropertySources().addFirst(propertySource);
        }
    } catch (UnknownHostException e) {
        LOGGER.warn("Could not get k8s pod ip address.");
    }
}

}`
spring:   cloud:     client:       hostname: ${POD_OWN_IP_ADDRESS:localhost}
:) it's works

I managed to solve this issue by implementing a BeanPostProcessor that intercepted the EurekaInstanceConfigBean during the postProcessAfterInitialization lifecycle hook so I could manually set the instance id, IP address, hostname, status page URL, and health check URL using the host address of the Fargate instance that the task is running on. Below is the implementation I came up with:
`@Component
@Profile("fargate")
public class EurekaInstanceConfigBeanPostProcessor implements BeanPostProcessor {
private static final Logger LOGGER = LoggerFactory.getLogger(EurekaInstanceConfigBeanPostProcessor.class);

@Value("${spring.application.name}")
private String serviceName;

@Value("${server.port}")
private int port;

private String fargateIp;

public EurekaInstanceConfigBeanPostProcessor() {
    try {
        fargateIp = InetAddress.getLocalHost().getHostAddress();
    } catch (UnknownHostException e) {
        LOGGER.warn("Could not get the Fargate instance ip address.");
    }
}

@Override
public Object postProcessAfterInitialization(Object bean, String beanName) {

    if (bean instanceof EurekaInstanceConfigBean) {
        LOGGER.info("EurekaInstanceConfigBean detected. Setting IP address to {}", fargateIp);
        EurekaInstanceConfigBean instanceConfigBean = ((EurekaInstanceConfigBean) bean);
        instanceConfigBean.setInstanceId(fargateIp + ":" + serviceName + ":" + port);
        instanceConfigBean.setIpAddress(fargateIp);
        instanceConfigBean.setHostname(fargateIp);
        instanceConfigBean.setStatusPageUrl("http://" + fargateIp + ":" + port + "/actuator/info");
        instanceConfigBean.setHealthCheckUrl("http://" + fargateIp + ":" + port + "/actuator/info");
    }

    return bean;
}

}`


Update to v1.6.0

Re-render and update to v1.6.0.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


ADRS vignette & template

ADRS Workflow

For PD param see https://github.com/pharmaverse/admiral/issues/1063 from admiral core & see ADRS workflow doc for other dependencies on admiral core

Consider ADEVENT possible set-up that some companies use, and include examples where PD and RESP dates are passed from an intermeidate dataset rather than from already derived ADRS params.

See discussion at https://github.com/pharmaverse/admiralonco/issues/25 around setting adsl vars once at the start and then re-using in all further param function calls, as done in ADEX.


[Question] Accessibility and focus

Hello! Thank you for this library!
Is there a recommended way to handle a11y? Meaning, for each element that is focused, the TV text-to-speech when activated to read a given label.
At the moment I am working on a project, and the usual approach is to place the role and aria-label properties on each element. When the component is focused, then the TV reads the label to the user.
Do you have a standard way of handling this outside of what I described? Do you plan to? Maybe even a way to enforce it, as some TV platforms require this functionality.
Would this work?
import { useFocusable } from '@noriginmedia/norigin-spatial-navigation';

function MovieTile({ title, director }) {
  const { ref, focused } = useFocusable();

  return (
    <div ref={ref} role="" arial-label={`${title}, ${director}`} className={focused ? 'movie-focused' : 'movie'}>

{ title }

{ director }

  );
}

Does aria-current do what you need?

We have implemented it by manually calling the "text to speech" API when interactable elements getting focus. So for us it looks like this, inside the focusable component:
useEffect(() => {
    focused && speak(buttonText);
}, [focused]);

where speak is an imperative call to the "text to speech" API which might be different for different TV models.
I'm not sure if we will handle or enforce it in this library though as we don't use the default browser document.activeElement feature to check the currently focused element. However it may be achieved by applying proper aria- attributes to focused elements only, or to "mirror" the focused state from our library into the DOM .focus() call which will set a browser native focus to the same element. Which in turn should enable a browser TTS engine to speak the element text according to aria- lables.

@dasilvaluis Hello! You might be interested in the new Release that includes the sync of the item focus with the native DOM node .focus(): https://github.com/NoriginMedia/Norigin-Spatial-Navigation/releases/tag/v1.3.0
Now you can simply enable it for all focusable items and use aria tags for accessibility.

That sounds great! thank you!


[AH-52] Add CV PDF download ability

Description

Example: https://www.dejan.works/about/
Note: A11y note: use "download" attribute

closed as duplicate of #117


Format code a la prettier

This PR formats all of the existing code using PrettierJS and adds the following:

adds npm scripts:

npm run format:all will format the entire code base (anything not in .prettierignore)
npm run format:single --file src/some/File.svelte will format a single file

adds a git pre-commit hook that will run ES Lint and Prettier

this is accomplished via lint-staged and husky

After merging this to the main branch make sure to run npm i to grab the new dev dependencies.

 Review configurations for ES Lint & Prettier


critical improvments to reducing loading time!

Precompiling the init() function has a significant improvement to the loading time (user package: 1s+2s -> 1s+0.4s).
We should add this to the guide!

should be mentioned at #25


Update the naming schema in Idp index.

Coverage remained the same at 89.5% when pulling 031812e1c2aa433abc0397d76ca7d246a256391a on akatsoulas:update-index-naming into 4056b38d2c0b6402b6e2a04073812ffbc6f8f86f on mozilla:master.


Add Ubuntu 24.04 build

Hello! Could you please add a build for Ubuntu 24.04 (Noble)?
Thanks!

https://github.com/cachewerk/relay/commit/ad8eb5f7d97c31c7f36d375a0ace2a2b62e05d63

https://github.com/cachewerk/linux-repos/commit/25d105bc9157d9f6616a9c30eff07d457851d63c

Thank you! I'll give the 22.04 binary a shot.


chore: update swagger

Signed-off-by: Gaius<EMAIL_ADDRESS>
Description

Update swagger docs.

Related Issue

Motivation and Context

Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation Update (if none of the other choices apply)

Checklist

[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.

Codecov Report

Merging #1314 (5b0fa35) into main (b580ea6) will decrease coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1314      +/-   ##
==========================================
- Coverage   45.76%   45.55%   -0.22%
==========================================
  Files         140      140
  Lines       11890    11890
==========================================
- Hits         5442     5416      -26
- Misses       5838     5863      +25
- Partials      610      611       +1

Flag
Coverage Δ

Object-compatibility-e2etests
?

e2etests
?

unittests
45.55% <ø> (-0.22%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

client/daemon/peer/peertask_piecetask_poller.go
30.43% <0.00%> (-16.67%)
:arrow_down:

...ent/daemon/peer/peertask_piecetask_synchronizer.go
45.12% <0.00%> (-1.54%)
:arrow_down:

client/daemon/peer/peertask_conductor.go
57.14% <0.00%> (-0.41%)
:arrow_down:

cdn/supervisor/progress/progress.go
96.87% <0.00%> (+6.25%)
:arrow_up:


[CSS] Unrestrict .profile's height

Summary
This PR unrestricts div.profile's height to let all avatars be displayed.
Feedbacks are welcomed.
Changes

Unrestrict .profile's height to let the div grow in height.
Other changes to keep the layout look the same as it was initially.

Caveats
The div.profile might grow too big in height that much scrolling would be required for phone users to get past the profile picture section to see the main content.
Related issues
This resolves #14.
Contribution guidelines
:heavy_check_mark: My Pod Leader knows I'm working on this Pull Request
:heavy_check_mark: I've explained what the Pull Request is adding.
:heavy_check_mark: I've explained why this is important.

@khattakdev I will split it into another issue.

Nevermind, it couldn't be rebased and merge due to earlier conflicts but I can squash it.


Cannot change fontFamily on ThemeData.dark() or ThemeData.light()

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Create a custom theme, based on the dark theme (can be the light one too): ThemeData.dark().copyWith();
Try adding a custom fontFamily: ThemeData.dark().copyWith(fontFamily: 'CustomFont');
You will notice that ThemeData.dark() does not implement the fontFamily attribute (theme ThemeData does, but ThemeData.dark() does not)

Expected results
At the ThemeData.dark() and the ThemeData.light() should be possible to change the font family.
Actual results
The ThemeData.dark() and the ThemeData.light() don't allow me to change the font family.
Code sample
Code sample
void main() {
  runApp(
    MaterialApp(
      theme: ThemeData.dark().copyWith(
        fontFamily: 'MyCustomFont', // this does not work
      ),
      home: const MyApp(),
    );
  );
}

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.5, on macOS 13.4 22F66 darwin-arm64, locale en-BR)
    • Flutter version 3.10.5 on channel stable at /Users/tho/src/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 796c8ef792 (7 days ago), 2023-06-13 15:51:02 -0700
    • Engine revision 45f6e00911
    • Dart version 3.0.5
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at /Users/tho/Library/Android/sdk
    • Platform android-33, build-tools 33.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.79.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.66.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 13.4 22F66 darwin-arm64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 114.0.5735.133

[✓] Network resources
    • All expected network resources are available.

• No issues found!

It's a simple change.
If it is decided that this will be done, I can implement and open a PR adjusting it.

I want to get this issue

Thanks for filing the issue. I see there's an open issue addressing a similar case: https://github.com/flutter/flutter/issues/98534. Even though it didn't mention exactly dark or light but it would be better to centralize on that issue.
Closing this issue in favor of https://github.com/flutter/flutter/issues/98534, so please follow up on it for further updates. You can leave there your comment and react an upvote to the original comment will help to raise the issue. Thank you!


[Error with custom train.csv file] Batch does not contain any data (None)

I am struggling with a custom csv file containing the input and output params. I rebuilt the convert.py script from the tutorial video and created a train.csv with this content, but I get an error:
File "/usr/local/lib/python3.10/dist-packages/autotrain/utils.py", line 280, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/autotrain/trainers/clm/__main__.py", line 440, in train
    trainer.train()
  File "/usr/local/lib/python3.10/dist-packages/transformers/trainer.py", line 1591, in train
    return inner_training_loop(
  File "/usr/local/lib/python3.10/dist-packages/transformers/trainer.py", line 1870, in _inner_training_loop
    for step, inputs in enumerate(epoch_iterator):
  File "/usr/local/lib/python3.10/dist-packages/accelerate/data_loader.py", line 664, in __iter__
    raise ValueError(
ValueError: Batch does not contain any data (`None`). At the end of all iterable data available before expected stop iteration.

If I replace the custom data with one of huggingface like guanaco, it works, so it must be something with the data format
If the content is not shown properly (maybe because of line separator issues), here from my repository:
https://github.com/Koliham/finetuneexample/blob/main/train.csv
instruction,input,output,text
Who is Koliham?,,Koliham is an employee at SuperCorp. He works in the field of Data Science and Backend Devlopment.,"Below is an instruction that describes a task. Write a response that appropriately completes the request.

### Instruction:
Who is Koliham?
### Response:
Koliham is an employee at SuperCorp. He works in the field of Data Science and Backend Devlopment."
What are the Hobbies of Koliham?,,Koliham likes playing video games and programming.,"Below is an instruction that describes a task. Write a response that appropriately completes the request.

### Instruction:
What are the Hobbies of Koliham?
### Response:
Koliham likes playing video games and programming."
What is Kolihams favourite food?,,He usually likes Burger and Curry.,"Below is an instruction that describes a task. Write a response that appropriately completes the request.

### Instruction:
What is Kolihams favourite food?
### Response:
He usually likes Burger and Curry."
Tell me something about Koliham,,Koliham is an employee at SuperCorp. He works in the field of Data Science and Backend Devlopment. He hobbys are playing video games and programming. He likes to eat Burger and Curry.,"Below is an instruction that describes a task. Write a response that appropriately completes the request.

### Instruction:
Tell me something about Koliham
### Response:
Koliham is an employee at SuperCorp. He works in the field of Data Science and Backend Devlopment. He hobbys are playing video games and programming. He likes to eat Burger and Curry."

Question: Can someone provide an example csv file which is compatible with the Mistral Instruct format?

if thats the csv file you linked, you need more samples. try duplicating and have more than 30-40 rows.

I got the same problem here. Your solution is very useful. So I think it is maybe a good idea to write that down in the documentation or somewhere else where it's easy to find.


Preserve player direction even if player stops

Currently, we determine Player.Direction by the player velocity, and if the velocity is zero, then it defaults to Right.
We should preserve the last direction when the player stops.
See TODO[#xxx] in the code.

What about mutable field with previous direction value in Player type?

No mutable fields, the whole game engine should be immutable.


docs: update magento1 and backdrop quickstarts, fixes #6448

The Issue

#6448

How This PR Solves The Issue

Updated the Magento1 quickstart to use a terminal approach like other quickstarts.
Adds ddev config --web-environment-add=MAGE_IS_DEVELOPER_MODE=1 for the Magento1 quickstart as requested.
Adds -f to rm where it was not added.

I tried other quickstarts, and I noticed a problem with backdrop when running it in zsh:
$ mv -f ./backdrop/{.,}* .
mv: cannot stat './backdrop/.editorconfig': No such file or directory
mv: cannot stat './backdrop/.gitignore': No such file or directory
mv: cannot stat './backdrop/.htaccess': No such file or directory

Fixed with it with bash -c mv -f ./backdrop/{.,}* ."

Manual Testing Instructions
Try Magento1 qucikstart.
https://ddev--6449.org.readthedocs.build/en/6449/users/quickstart/#__tabbed_9_2
Automated Testing Overview

Release/Deployment Notes

Rebased.

I updated the quickstarts and tested Magento 1, thank you @sreichel.

A note @sreichel , I see this when I log into the back-end using these instructions:

Is that a problem with OpenMage, the sampledata, or both? Should it be reported and dealt with somewhere? (Would appreciate your followup)

Second question @sreichel - Should we be using a composer install for this, since this is now a composer project? (ddev composer create ?) We wouldn't have to do the funky tag detection then.

Yet another question @sreichel - I see that openmage is trying to add a cron (.ddev/web-build/openmage.cron), but it doesn't have a Dockerfile.openmage to go with it. Do you know anything about that?

Thanks so much for this great work. It's so much improved!

But it seems odd to me that this message is so prominent. I guess it's cared for.

Think it was important enough to make it that prominent, but with >= 20.7.0 installed, it should not pop up. (needs change in OM)

Second question @sreichel - Should we be using a composer install for this, since this is now a composer project?

see OpenMage/magento-lts#3318

Composer installs do not include .ddev.
Btw ... iInstead of using zip-file, why not checkout a git tag?

Yet another question @sreichel - I see that openmage is trying to add a cron

We have only added the config-file. Cronjobs are often not required for loval development, so it is optional.

Yet another question @sreichel - I see that openmage is trying to add a cron
We have only added the config-file. Cronjobs are often not required for loval development, so it is optional.

Yes, but the config file in web-build does nothing at all. Is it intended to do something? If so, it should probably be somewhere other than web-build?


FALCON-2227 Fixing error throws to throw exceptions and also added a check for META-INF/services file to be present in the extensions Jar in the extension build path

+1. Build failure is unrelated. Will merge shortly.


cloudflare_zone_settings_override.browser_cache_ttl default value marker reverts to specific value

Terraform version
v0.13.4
Affected resource(s)
cloudflare_zone_settings_override
Terraform configuration files
resource "cloudflare_zone_settings_override" "resource_name" {
  zone_id = var.cloudflare_zone_id

  settings {
    ...
    browser_cache_ttl        = 0
    ...
}

Debug output

Expected behavior
After applying the changes with browser_cache_ttl = 0, the diff should return the no changes. (or using 0 should not be accepted when applying)
Actual behavior
Diff returns a difference in
~ browser_cache_ttl           = 14400 -> 0

Steps to reproduce

terraform apply
terraform diff

Community note

Please vote on this issue by adding a 👍 reaction
to the original issue to help the community and maintainers prioritize this request
If you are interested in working on this issue or have submitted a pull
request, please leave a comment

It looks like I ran into an issue that is related to this issue in some way. I use Terraform Cloud v.0.14.3. After applying the browser_cache_ttl = 0 option, I found that in the CloudFlare interface the Browser Cache TTL value remained at the default value, which is 4 hours. After that I tried to reproduce the issue several times, but I couldn't, I'm not sure what it could be, but the data in the interface was fresh (I was refreshing the page).

It looks like I ran into an issue that is related to this issue in some way. I use Terraform Cloud v.0.14.3. After applying the browser_cache_ttl = 0 option, I found that in the CloudFlare interface the Browser Cache TTL value remained at the default value, which is 4 hours. After that I tried to reproduce the issue several times, but I couldn't, I'm not sure what it could be, but the data in the interface was fresh (I was refreshing the page).

@viraptor Are you able to provide the TF_LOG=DEBUG output to accompany this change? We have a couple of tests that excercise this (annoying complex) code path which still seem to be passing so I wonder if there is something else happening here
https://github.com/cloudflare/terraform-provider-cloudflare/blob/999d2269765d2a8f40287e090f3526cc93569e27/cloudflare/resource_cloudflare_page_rule_test.go#L346-L361

@viraptor Are you able to provide the TF_LOG=DEBUG output to accompany this change? We have a couple of tests that excercise this (annoying complex) code path which still seem to be passing so I wonder if there is something else happening here
https://github.com/cloudflare/terraform-provider-cloudflare/blob/999d2269765d2a8f40287e090f3526cc93569e27/cloudflare/resource_cloudflare_page_rule_test.go#L346-L361

@viraptor Are you able to provide the TF_LOG=DEBUG output to accompany this change? We have a couple of tests that excercise this (annoying complex) code path which still seem to be passing so I wonder if there is something else happening here
https://github.com/cloudflare/terraform-provider-cloudflare/blob/999d2269765d2a8f40287e090f3526cc93569e27/cloudflare/resource_cloudflare_page_rule_test.go#L346-L361

@viraptor Are you able to provide the TF_LOG=DEBUG output to accompany this change? We have a couple of tests that excercise this (annoying complex) code path which still seem to be passing so I wonder if there is something else happening here
https://github.com/cloudflare/terraform-provider-cloudflare/blob/999d2269765d2a8f40287e090f3526cc93569e27/cloudflare/resource_cloudflare_page_rule_test.go#L346-L361

It looks like I ran into an issue that is related to this issue in some way. I use Terraform Cloud v.0.14.3. After applying the browser_cache_ttl = 0 option, I found that in the CloudFlare interface the Browser Cache TTL value remained at the default value, which is 4 hours. After that I tried to reproduce the issue several times, but I couldn't, I'm not sure what it could be, but the data in the interface was fresh (I was refreshing the page).

I was able to reproduce the issue I mentioned above once again where browser_cache_ttl = 0 was not applied the first time. I created two zones at the same time, the first zone had a value of 0, the second did not, although it should have. On the second commit, when I didn't change anything in the configuration other than the location of the curly braces in the object (just wrapped it to another line), Terraform detected the changes and applied browser_cache_ttl = 0 to the second zone. Now I cannot repeat this again, but I can provide the configuration files and a log of applying changes. Should I create a new issue or is it worth posting all the information here?

It looks like I ran into an issue that is related to this issue in some way. I use Terraform Cloud v.0.14.3. After applying the browser_cache_ttl = 0 option, I found that in the CloudFlare interface the Browser Cache TTL value remained at the default value, which is 4 hours. After that I tried to reproduce the issue several times, but I couldn't, I'm not sure what it could be, but the data in the interface was fresh (I was refreshing the page).

I was able to reproduce the issue I mentioned above once again where browser_cache_ttl = 0 was not applied the first time. I created two zones at the same time, the first zone had a value of 0, the second did not, although it should have. On the second commit, when I didn't change anything in the configuration other than the location of the curly braces in the object (just wrapped it to another line), Terraform detected the changes and applied browser_cache_ttl = 0 to the second zone. Now I cannot repeat this again, but I can provide the configuration files and a log of applying changes. Should I create a new issue or is it worth posting all the information here?

@viraptor @simpsora are you folks still having this issue and able to share the reproduction steps/logs?

Unfortunately not anymore. @alex-feel may be on to something with new zones. Once the current setting is at the explicit14400, reverting it to 0 doesn't show the same issue anymore.

@jacobbednarz I tried again and was able to reproduce the issue, if that helps I can post my config and the debug logs.

Yes please. The information needed can be found at .github/ISSUE_TEMPLATE/bug_report.md

So should I create a new issue or post the info here?

Here is fine if it is the same issue and we can reproduce it.

I believe this is the same issue, but if you think it is not, I can delete this post and create a new issue.
Terraform version
Terraform Cloud 0.14.4
Terraform configuration files
https://gist.github.com/alex-feel/1f23d869d9e6fe6d089203a4e4590504
Debug output
After pushing the config above I got these plan and apply logs:
https://gist.github.com/alex-feel/68471dee9580f5251c711b54cb5688ab
https://gist.github.com/alex-feel/cd500f3a61d935ea33cfcef82ce29807
Here is the screenshots of what I was seeing in the CloudFlare account:
https://ibb.co/x6YMFpC
https://ibb.co/Nt6wXLM
Then I just added a comment to the config, here is a diff:
https://ibb.co/QcPsCFJ
After pushing the config with the comment I got these plan and apply logs:
https://gist.github.com/alex-feel/ac1a827a0ac7da3541626d568d298015
https://gist.github.com/alex-feel/545c9a0ae98b766554afc318e45ef99a
Here is the screenshots of what I was seeing in the CloudFlare account:
https://ibb.co/Hgshktj
Expected behavior
browser_cache_ttl = 0 should have been applied to the testzone.com domain on first run.
Actual behavior
browser_cache_ttl = 0 was applied to the testzone.com domain only on second run without changes.
Steps to reproduce
Look at the Debug output.


Update requirements.txt

closes #23

@mikejgray i believe this is currently only used by HA plugin for the oauth, please verify nothing breaks there

actually, pillow is not even used by the plugin....


chore: revert prettier config

Reverts interledger/testnet#579
This was causing issues on deploy. Needs more investigation.

the issue was not reproduced, most likely the change triggered rebuild of some cached layers that caused the trouble and it is not the source of the problem.


Showing wrong teacher

Gavin, we have an issue with MS Acting 1.  It is showing Bob Baldwin as teacher.  Should be his son Sam Baldwin.  Link to photo of Sam if needed: https://four11.eastsideprep.org/system/teachers/idphotos/000/005/044/medium/sbaldwin.jpg?1611872631
Thanks and hope all is well,
Clarke

Seems to be doing some thing where it takes the first letter of the teacher's last name and uses that for the first name - when i try to click on a teacher it says something like "tried to get username of vviolette" when it should be kviolette. Essentially, it's more broken than we thought, as you can't click on a teacher right now.

Good find. Can you tell which part of the code is doing this?

Yes, just figured it out. The requestAdditionalData function in index.js gets a name that it converts to the username, but somehow the name gets globulated somewhere. I looked above it and it doesn't seem to be the parsing.

Might also have to do with line 458 in index.js, where gavin sets the scheduleObj. Those lines of code are confusing to me because he uses [k] which i don't understand.

Think I found the offending lines - index.js lines 659-660 get the first character from a teacher object and i think gavin expected the teacher object to be like the email, i.e. yielding something like gmein, when really his code gets mmein there.

That analysis sounds right to me, it looks like the code expects "Gunnar Mein" but gets "mein" as you say. Unfortunately, you'll need both first and lastname or the full username, so next step is probably to figure out why item.teacher is missing the full name.
BTW, I agree that
    for (var k = 0; k < schedule["classes"].length; k++) {
is pretty gross, this is definitely a place where ES6 constructs like
 schedule["classes"].length.forEach(class => {
would be much cleaner.

Remind me which four11 endpoint you are hitting and I'll just add username
to the JSON. There are a few that don't fit the pattern i.e dkhedrick jruiz
schinson.
Mr. Briggs
On Sun, Mar 14, 2021, 4:04 PM Justin Uberti @.***>
wrote:

That analysis sounds right to me, it looks like the code expects "Gunnar
Mein" but gets "mein" as you say. Unfortunately, you'll need both first and
lastname or the full username, so next step is probably to figure out why
item.teacher is missing the full name.
BTW, I agree that
for (var k = 0; k < schedule["classes"].length; k++) {
is pretty gross, this is definitely a place where ES6 constructs like
schedule["classes"].forEach(class => {
would be much cleaner.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/guberti/epschedule/issues/412#issuecomment-798996799,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABUPM2Y727YWPSPBRUW6JDTDU6I5ANCNFSM4ZCWKJDA
.

It looks like there is a username already in the JSON, but the code is instead looking at the partial fullname that is created at https://github.com/guberti/epschedule/blob/master/static/js/index.js#L453.

Thanks all!


[Schema Registry Avro Serializer] Minor updates

Making some minor changes:

Switched back the project reference, this has been causing pipeline issues, maybe because Azure.Data.SchemaRegistry isn't GA?
Removed test recordings
A couple small README updates I found when working on the JSON serializer

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


anthropic[patch]: add vertex and bedrock support, streamResponseChunk…

Add support for vertex and bedrock, optimize stream response chunks
This pull request adds support for vertex and bedrock, and optimizes the *_streamResponseChunks method for better efficiency.
Changes:

Added support for vertex and bedrock:

Added necessary configurations and logic to handle vertex and bedrock requests.

Optimized *_streamResponseChunks method:

Simplified the logic for handling response chunks, making it more concise and efficient.

Question:

Should the extractToken method return tool_use related tokens?

Currently, the extractToken method returns tool_use related tokens as new tokens to handleLLMNewToken. This might be problematic because these tokens are not actual language model outputs for human.

After this update (0.2.10), anthropic will stream with bound tools but it will not stream tool chunks.
Related issues:
https://github.com/langchain-ai/langgraphjs/issues/253 (closed but still ongoing issue)
https://github.com/langchain-ai/langchainjs/issues/6238


what to do with previously created Sinopia triples that are no longer used

Now that we don't create triples specifying embedded RTs #1359, and we don't expect those triples when loading a resource #1357, if you try to open a resource that was created before this release, you get unmatched sinopia triples, like for
https://trellis.development.sinopia.io/repository/stanford/e111a712-5a45-4c2a-9201-289b98d7452e
(in search tab search for "fruit" to find this)
Unable to load the entire resource. The unused triples are:
_:c14n0 http://sinopia.io/vocabulary/hasResourceTemplate "ld4p:RT:bf2:SeriesInformation" .
_:c14n1 http://sinopia.io/vocabulary/hasResourceTemplate "ld4p:RT:bf2:InstanceTitle" .
_:c14n2 http://sinopia.io/vocabulary/hasResourceTemplate "ld4p:RT:bf2:ParallelTitle" .
_:c14n3 http://sinopia.io/vocabulary/hasResourceTemplate "ld4p:RT:bflc:TranscribedTitle" .
...
QUESTION:
--delete these triples from resources saved in Production?
--ignore these when showing unmatched triples?

Or ignore: When a user re-saves, they will be dropped.


add debounce for secret watcher

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ X] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ X] Does not have any changes that may affect Istio users.

@ramaraochavali: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
e71133bfffc04822656eaab44f54ac6694b75567
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
e71133bfffc04822656eaab44f54ac6694b75567
link
/test unit-tests_istio

lint_istio
e71133bfffc04822656eaab44f54ac6694b75567
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
e71133bfffc04822656eaab44f54ac6694b75567
link
/test lint_istio

unit-tests_istio
f984f67d6b174ec047fed43e5cb7b43bac43ec67
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests_istio

@ramaraochavali: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
2c4b4734876e70f217d4e5e6544089b45283ec5b
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
2c4b4734876e70f217d4e5e6544089b45283ec5b
link
/test unit-tests_istio

integ-pilot-k8s-tests_istio
2c4b4734876e70f217d4e5e6544089b45283ec5b
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
21c31f3a5210399f4105e0ea3970b80f1c5a533e
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
678901471f7626d02a24edfcc67849baf397608c
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-pilot-k8s-tests_istio


type '_GrowableList' is not a subtype of type 'Map<String, dynamic>'

Fixed in https://github.com/Mause/ChooseFood/commit/dd66df2180948d968f013a58f9991df54bca9652


faulty link

http://ricostacruz.com/nprogress/nprogress.css is a faulty link

Thanks for noting! The new link is<EMAIL_ADDRESS>I updated the book content.


04/07/2018 17:57 : Create Pull Request for branch master -> prTesting1_2018040717

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Prevent Focus-loss if tapped on specific Button / GestureDetector

How do i still consume e.g. onTap event on another widget (e.g. GestureDetector) without stealing the Focus from e.g. a TextFormField?
I explicitly do not want to lose my focus, I don't want to get the focus and request the focus from the FocusNode that lost the focus.

Hi @Eerey, Thanks for filing the issue. I think it is not possible to have two widgets focused at the same time. There will always be one widget that will have the primary focus. You will have to manage the focus of the widget yourself, I am quite not sure what it means to have two widgets focused at the same time or How should two widgets respond simultaneously considering flutter is single-threaded. In case you want help with switching focus between widgets please try asking for support at https://flutter.dev/community
Closing as this is neither a bug nor a feature request, Incase you disagree feel free to write in the comments and we will reopen it.
Thank you.

Hey @maheshmnj ,
I don't want to have two widgets being focused at the same time.
I want to keep my focus on the TextFormField, even if I press on a specific widget.
For example, think about an on-screen-keyboard - you press on it, yet the focus remains the same.

If possible, I request to re-open the ticket.

I want to keep my focus on TextFormField, even if I press on a specific widget.

Can you share a minimal code sample for us to take a look?


[5.5] Add followingRedirects() method

This adds a test helper method called followingRedirects() that will instruct the the test case to follow any redirect responses automatically. It can be chained prior to making a HTTP request and it's named to read more sentence-like.
$response = $this->followingRedirects()->post('/login', ['email' => 'john@example.com']);

$response->assertSee('Welcome back, John.');

From memory a similar test helper used to exist in an earlier iteration of Laravel's test suite, so this is just returning the functionality. It was prompted by #18016.

Are you thinking of the old laravel/browser-kit-testing which was default in 5.3? It has a followRedirects() method.

That's the one. Happy to match the naming too if preferred.

I don't really like how if you call this in one test method it's not going to apply to all test methods that run after that in that class. How would I disable it? Shouldn't it be disabled after each test?

I had tested for that and it appeared to reset between tests in the same class for me. However, I've now explicitly set followRedirects back to false after each request made. Let me know if that feels better to you or if there's another approach you'd rather me take.

Clean and sweet.

So there is no possibility to make all test methods follow redirects automatically right now?

Is it normal that this method persists an app state between calls? Ex when I put dump into main middleware It will be fired X times where X is numbers of redirects. Now when I set something in this middleware for example Menu (by library) declaration it is called twice and the second call sees already defined menu. It is easy to check also by declaring function - PHPUnit will return error " Cannot redeclare App\Http\Middleware\s3223() (previously declared in \app\Http\Middleware\MenuGen
erator.php:56)                "


Screen reader reads extension's . at top of extension settings page

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.63.2
OS Version: Win11 22000.376

Steps to Reproduce:

Open VSCode.
Open the Settings page and expand the "Extensions" item.
Turn on your screen reader (I'm using Windows Narrator).
Select an extension from the tree.
Hit tab to move focus to the extension's settings page.
Focus will move to a top-level element (which isn't really visible). Note that this is distinct from name of the extension at the top of the settings page.
Narrator will announce the extension's ".".

Expected result:
I think ultimately the element shouldn't be focusable as it causes the screen reader to announce some text that isn't actually there.

GithubTags:#A11yMAS;#a11y-partner;


Mesh plot errors.

Hi, Thank you for releasing Pytorch version. I was hoping to have some fun but ran into some plotting issues.

I was wondering if you can add the code to output it as obj to actually preview the mesh in 3D packages ?

Yes, thanks for the repo, this looks very promising :)
I have the same issue while trying to plot the result actually.

The preview seems to works in Jupyter only, so if you run the notebook (reconstruct_pipeline.ipynb) then you can test the detection, reconstruction and preview.
k3d is a Jupyter notebook extension for 3D visualization so it won't work out of the box in a .py file. The example code in the readme should run in a jupyter notebook I assume.
Nevertheless it would be real nice to know how to export / save the mesh and texture 😼

As explained by @gu-ma, the interactive visualization works only inside a notebook, you can either try the non-interactive matplotlib one or use the newly added option to export the result as an STL file
p2v.save2stl(result['Z_surface'], 'res.stl')


Handle watchman hangs

Sometimes, watchman hangs when we're trying to talk to it; we should use the appropriate socket settings so that we timeout when this happens rather than hanging forever.

We did this.


Motor Setup Issue

Hey! I've replaced a Dynamixel for my turtlebot..
Strangely, it is not recognized by the robot, even if checked the setup:

firmware 41
ID 1
Baud 1Mb

Anyway, when I try to run the "setup motor" sketch, I got this error:

1. setup left  motor
2. setup right motor
3. test  left  motor
4. test  right motor
>> Do you really want to setup ? y/n : yes
setup.... left
Find Motor...
    setbaud : 57600

This function is deprecated. Use 'Serial.print()' and 'getRxPacketError()' instead
[TxRxResult] There is no status packet!
    setbaud : 1000000

This function is deprecated. Use 'Serial.print()' and 'getRxPacketError()' instead
[TxRxResult] There is no status packet!

1. setup left  motor
2. setup right motor
3. test  left  motor
4. test  right motor

Any help?

Hello :)
You need to check
First, you have to power on to the Dynamixel
Second, OpenCR is connected with single Dynamixel
Thanks
Darby

Hi :)
I tested both one or two dynamixel at the same time..
In my understanding, the dynamixel powered by the OpenCR board right? I've connected it with the turtlebot3 battery and the red led on the dynamixel blinks when I turn it on! So it should be powered on.

Then it might be baudrate of yours is not 57600 or 1000000.
I recommend to use monitor.ino in dynamixel-workbench example for check baudrate.
(File -> Examples -> OpenCR -> DynamixelWorkbench -> Monitor)
Thanks
Darby

This issue will be closed since there were no actions for a while.
You can reopen this issue to show this issue to the users whenever.
Thanks. :)


Port names

It is useful to know port's kind (for complex nodes)

this issue got obsolete because in new version there is no "Port kind" entity


--version or -V doesn't tell the version

I installed dua yesterday and tried to get the version number for another (now closed) issue, but found that it wouldn't tell me:
mburr@mint-19:~$ ~/.cargo/bin/dua --version
dua
mburr@mint-19:~$ ~/.cargo/bin/dua -V
dua

I installed dua like so:
curl -LSfs https://raw.githubusercontent.com/byron/dua-cli/master/ci/install.sh |     sh -s -- --git byron/dua-cli --target x86_64-unknown-linux-musl --crate dua

Running on Linux Mint 19.3

Thanks for letting me know - fixed with 2.10.10.

Thanks for letting me know - fixed with 2.10.10.


Username and/or password are incorrect

I have tried all versions from presto 0.80 to 0.100 i can't authenticate against a cassandra db  2.1.4 nor 2.0.14 . If i use no password everything works fine
my presto connector config is
connector.name=cassandra
cassandra.contact-points=<IP_ADDRESS>
cassandra.username=cassandra
cassandra.password=cassandra

the error i get is
presto:default> describe cassandra.system_auth.users;
Query 20150409_144516_00001_96dtc failed: Authentication error on host /<IP_ADDRESS>:9042: Username and/or password are incorrect

Remove the quotes in the properties file. Those are treated as literal quotes.

On Apr 9, 2015, at 7:46 AM, Vali Malinoiu<EMAIL_ADDRESS>wrote:
I have tried all versions from presto 0.80 to 0.100 i can't authenticate against a cassandra db 2.1.4 nor 2.0.14 . If i use no password everything works fine
my presto connector config is
connector.name=cassandra
cassandra.contact-points=<IP_ADDRESS>
cassandra.username="cassandra"
cassandra.password="cassandra"
the error i get is
presto:default> describe cassandra.system_auth.users;
Query 20150409_144516_00001_96dtc failed: Authentication error on host /<IP_ADDRESS>:9042: Username and/or password are incorrect
—
Reply to this email directly or view it on GitHub.

I removed them :), my code is the same as in the initial comment code block
...

Did that fix the issue for you?


Avoid synchronized block or methods

According to https://docs.oracle.com/en/java/javase/21/core/virtual-threads.html#GUID-04C03FFC-066D-4857-85B9-E5A27A875AF9:

A current limitation of the implementation of virtual threads is that performing a blocking operation while inside a synchronized block or method causes the JDK's virtual thread scheduler to block a precious OS thread, whereas it wouldn't if the blocking operation were done outside of a synchronized block or method. We call that situation "pinning".

IMHO, all synchronizd blocks or methods should be replaced with ReentrantLock usage.

Not all, only io bound I think.

This is a temporary limitation of Virtual Threads and applies only to blocking operations inside synchronized blocks. I went through all cases of synchronized, and only deprecated-for-removal Lazy can run blocking code inside it, and that's the only change needed here I believe

Hi @pivovarit,  I would be happy to contribute. Please let me know if I can pick this up if not assigned to someone already?

Sure, it’s yours :)

@pivovarit I have created a PR for review - https://github.com/vavr-io/vavr/pull/2842

Looks done to me :)


Review OUSD Uniswap V3 strategy for the launch

The Uniswap V3 strategy code is in PR https://github.com/OriginProtocol/origin-dollar/pull/1258

I've done an initial pass of the code and raised PR https://github.com/OriginProtocol/origin-dollar/pull/1652 for some changes.
A detailed review of the logic is still to be done.


Using C# 7 pattern matching features

Some trivial changes to use more C# 7.0 features.

@onovotny @shiftkey - There seem to be build issues:
[00:01:59]      2>SourceLinkCreate:
[00:01:59]          unable to find repository origin with: dotnet sourcelink-git origin
[00:01:59]      2>Done Building Project "C:\projects\rx-net-la3iv\Rx.NET\Source\src\System.Reactive\System.Reactive.csproj" (default targets) -- FAILED.

Any thoughts what's off?

@bartdesmet looks like something with the sourcelink setup

I just bumped sourcelink to version 2.1, lets see if that helps. Otherwise, we can loop in @ctaggart to help.

I've worked around this in dev by disabling sourcelink until @ctaggart can investigate further. Builds are green now.


Removed dead links for ruby courses

What does this PR do?
Remove resource(s)
For resources
Description
I removed some dead links for ruby courses.
Why is this valuable (or not)?
This will save the user searching time.
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[ ] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!

Hi eshllman. Please revert and merge this PR again for it to get accepted by Haktoberfest.


Add support for Immax NEO Smart / Visionic Door Window Sensors + Tuyatec temp and humidity

Hi @artanisdesign, thanks for the PR.
I will not merge it as it's just manufacturer names and product Id's.
You are welcome to just file an issue with this data next time :)
I found the units online so I understand what id goes with what manufacturer, it helps with documentation if you mention this is future requests :)


Wall joint

First draft of wall-to-wall joints.
next step after https://github.com/gramaziokohler/compas_timber/pull/320

What type of change is this?

[ ] Bug fix in a backwards-compatible manner.
[ ] New feature in a backwards-compatible manner.
[ ] Breaking change: bug fix or new feature that involve incompatible API changes.
[ ] Other (e.g. doc update, configuration, etc)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I added a line to the CHANGELOG.md file in the Unreleased section under the most fitting heading (e.g. Added, Changed, Removed).
[ ] I ran all tests on my computer and it's all green (i.e. invoke test).
[ ] I ran lint on my computer and there are no errors (i.e. invoke lint).
[ ] I added new functions/classes and made them available on a second-level import, e.g. compas_timber.datastructures.Beam.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added necessary documentation (if appropriate)

included and will continue in https://github.com/gramaziokohler/compas_timber/pull/336


What device?

Wondering what device you are using for this?
I bought one of these recently https://www.ebay.com.au/itm/304645792686
It has the more powerful CH32V307VCT6

It is the CH32V003, not the more powerful one.  The tooling should be very similar so you could use this as a basic guide.  But my focus with this project is the CH32V003.  I've updated the readme with more info.


Localize all timestamps

Following

#21410

We are now able to localize all timestamps. Some examples:
short-date format, French, user profile page:

date-time format, Portuguese, mirror repository settings page:

Signed-off-by: Yarden Shoham<EMAIL_ADDRESS>

We may be able to do this on backend with https://github.com/go-gitea/gitea/pull/21443. I think it's certainly better to do it there to avoid content flashes. If we want to keep date formatting in frontend, there is the option of https://github.com/go-gitea/gitea/pull/21429 which I devised.

I read both PRs you linked. About the following comment:
    `requestAnimationFrame` is the proper way to mutate DOM without flicker. It can and should be avoided. It's just sloppiness on GitHub's side that they still have these flickers.

    I will also move all stuff in [formatting.js](https://github.com/yardenshoham/gitea/blob/main/web_src/js/features/formatting.js) into this script as well. These are all critical replacements.

Originally posted by @silverwind in https://github.com/go-gitea/gitea/issues/21429#issuecomment-1277119954
If you are still doing that then it's worth merging this as it's simply building on

#21410

Any solution of a different implementation would still have to include this JS so I suggest merging this for now
#21410 did this for the activity page, this just does it for all other timestamps, easy win

If we decide for the JS solution, we can certainly go ahead with this already.

I'm hoping to merge this, I believe having better datetime localization is a big plus even if it comes with a lightweight flicker. The work on #21429 can keep going, this doesn't hinder it at all

make L-G-T-M work


multiple xpansion launch

change lp_namer and study_updater log path to output directory for multiple xpansion launch
closes #99
There is still an issue if the same study is launched at the same time.
In this case we can't define which antares-solver output directory is the current directory and an assert is raised :
new_output = os.listdir(self.antares_output())
assert len(old_output) + 1 == len(new_output)
diff = list(set(new_output) - set(old_output))

We might need antares output for debugging purposes. So it is not desirable to redirect it to /dev/null.

Even if redirection is done to /dev/null we still have antares solver log in antares output directory


Mark StructureManager::structureList return value as const

To prevent accidental modification of the original structure lists, the return value should have const added, as below:
const StructureList& structureList(Structure::StructureClass structureClass);

You know, I was looking at this and thinking "These lists should probably be const since we don't want to modify the list even if we're modifying the objects pointed at by the lists".
Thanks for fixing!


feat: add Node 22 patch

Adds the patch for Node 22(.8) - consider this experimental for now
Patch:

The new V8 version moved things around a bit, but the patch is mainly concered with "no source" handling on Script objects, seems to work
On the Node side, parsing/stating package.json content has been moved to Node internals, and I had to restore parts of the original code using the patched "fs" module otherwise it doesn't know about the snapshot fs
I had to rewrite some code in V8 which uses C++20 features but the musl cross-platform tooling seems not to support that (the muslcc docker image and project seem to be abandoned, maybe its just some missing CC flag)

Build env:

I was able to get all builds to work except macos-arm64, because it times out or crashes with out of memory on the GH actions workers
The linux images work on a hello-world app, both with and without bytecode
Bumped the alpine and linuxstatic images to Ubuntu 22.04 and Python inside the Oracle linux build container to 3.12
Bumped the windows build image to vs-2022

This still requires extensive testing on real-world apps, but the basics should be in place.

Hi @faulpeltz and thanks a lot for this PR!

I was able to get all builds to work except macos-arm64, because it times out or crashes with out of memory on the GH actions workers

I was having the same issue in #40, I fixed it by reducing job counts to 2, try using 1 maybe?

@faulpeltz New majors usually require some work on pkg side too, would you mind to open a PR there and reference this?

Hi @faulpeltz and thanks a lot for this PR!

I was able to get all builds to work except macos-arm64, because it times out or crashes with out of memory on the GH actions workers

I was having the same issue in #40, I fixed it by reducing job counts to 2, try using 1 maybe?

I tried setting it to 1, but then it runs into the 6h build time limit

@faulpeltz About macos-arm64 you could try to use macos-14 runner as it's ARM64, you can then remove the compiler env vars (also rememver to change the arch check step at start to check for arm64 instead.

no luck with the macos-14 runner, setting jobs to 1 helps but it seems to run out of disk space every time

Thanks for giving it a try, I'm sincerly out of ideas about arm64, I have lost a week tring to make it work for latest images, I think the problem is that each new major version requires more resources so the error happens. If you are aware of any build flag and/or env that could help mitigate the issue let me know. For what I saw x64 cross build still preforms better then the native arm64 in terms of RAM so I would restore it for now and work on that.
The alternative could be to use the -large runners but I don't think them are free to use

@faulpeltz thanks so much for all the explainations 🙏🏼 Do you think this can be merged and released?

@faulpeltz I would disable arm64 build on nodejs 22 for now as we need it to end successgully otherwise the build all job will not end successfully and I cannot do a bump

@faulpeltz thanks so much for all the explainations 🙏🏼 Do you think this can be merged and released?

I think we can merge it, but maybe make some sort of prerelease or release Node 22 as experimental because I think this should be tested by the community.
I'll just clean up history and remove the arm64 macos build for now

I think we can merge it, but maybe make some sort of prerelease or release Node 22 as experimental because I think this should be tested by the community.

I would make a normal minor release of pkg and tell on changelog that node 22 support is experimental

Let's see how this goes: https://github.com/yao-pkg/pkg-fetch/actions/runs/10789654031

Let's see how this goes: https://github.com/yao-pkg/pkg-fetch/actions/runs/10789654031

I just saw this issue in the main NodeJS repo:
https://github.com/nodejs/build/issues/3878
We could try this for pkg fetch builds as well

@faulpeltz do you mean:
https://github.com/nodejs/node/pull/54658/files ?
May be worth a try, yeah! :)


Add "exclude" option

I added -x, --exclude option, which is sometimes useful, if you don't want your chore, refactor or style commits to appear in the changelog.
I also fixed the annoying 97.78% coverage on branches issue

I fixed all lint issues and applied the changes you've asked for.


Support building Hoodie with Hadoop 2.8.x

This unblocks Hudi from building against Hadoop 2.8.x (when specified as an argument during the maven build).  The following command should now work to support building Hoodie against newer versions of Hadoop.
mvn clean install -Dhadoop.version=2.8.5

@tokaaron this looks good in general. do you think we should also depend explicitly on hadoop-common (just like hadoop-hdfs)?

It seems like the hadoop-common tests package is explicitly needed.  Removing it from the pom.xml will result in errors:
Results :

Tests in error:
com.uber.hoodie.common.table.log.HoodieLogFormatAppendFailureTest.com.uber.hoodie.common.table.log.HoodieLogFormatAppendFailureTest
  Run 1: HoodieLogFormatAppendFailureTest.setUpClass:75 » NoClassDefFound org/apache/ha...
  Run 2: HoodieLogFormatAppendFailureTest.tearDownClass:80 NullPointer

  HoodieLogFormatTest.setUpClass:100 » NoClassDefFound org/apache/hadoop/net/Sta...
com.uber.hoodie.common.util.TestDFSPropertiesConfiguration.com.uber.hoodie.common.util.TestDFSPropertiesConfiguration
  Run 1: TestDFSPropertiesConfiguration.initClass:48 » NoClassDefFound org/apache/hadoo...
  Run 2: TestDFSPropertiesConfiguration.cleanupClass:80 » NullPointer

yes.. we cleaned up a bunch of thigns in the pom here.  https://github.com/apache/incubator-hudi/tree/hackathon-0619
do you mind generating the PR against that branch? I am planning to test that (after I redo the bundles also) against different hadoop/spark/hive combos

Generated a PR against that branch:
https://github.com/apache/incubator-hudi/pull/762

Closing in favor of #762


core.coffee / javascript?

Hi,
i just stumbled over the following line in core.coffee@734:
f1 = func
`func = function() { var __ = f1.apply(this, arguments); return '' + (__ || (__ == null?'':__)) }`

It should be the same to write:
f1 = func
func = ->
    __ = f1.apply this, arguments
    "" + (__ or (if __ is null then '' else __))

Please correct me if i am wrong.

if __ is null then '' else __
we need __ == null, not __ === null, because null==undefined

Yeah, i thought already about that.
But then you could simplify the code:
f1 = func
func = ->
    __ = f1.apply this, arguments
    "" + (__ or "")

Then we miss number 0
On Mar 22, 2015 3:15 AM, "dev-rke"<EMAIL_ADDRESS>wrote:

Yeah, i thought already about that.
But then you could simplify the code:
f1 = funcfunc = ->
__ = f1.apply this, arguments
"" + (__ or "")
—
Reply to this email directly or view it on GitHub
https://github.com/lega911/angular-light/issues/38#issuecomment-84461867
.

f1 = func
func = ->
    __ = f1.apply this, arguments
    "" + (__ or (if __ is 0 then __ else ""))

Is it relevant to catch all possible cases, e.g. if it is false etc.?

Yes, true and false give us text "true, false"
On Mar 22, 2015 3:27 AM, "dev-rke"<EMAIL_ADDRESS>wrote:

f1 = funcfunc = ->
__ = f1.apply this, arguments
"" + (__ or (if __ is 0 then __ else ""))
Is it relevant to catch all possible cases, e.g. if it is false etc.?
—
Reply to this email directly or view it on GitHub
https://github.com/lega911/angular-light/issues/38#issuecomment-84462729
.

Ok, just crawled through the coffeescript docs, we could use the existential operator, @see http://coffeescript.org/#operators and scroll down to existential operator.
f1 = func
func = ->
    __ = f1.apply this, arguments
    "" + (__ or __ ?= '')

Any problems left? :-)

Wohoo, every iteration gives better results and my knowledge about CS improves. ;-)
f1 = func
func = ->
    __ = f1.apply this, arguments
    "" + (__ or (__ ? ''))

compiles to
f1 = func;
func = function() {
  var __;
  __ = f1.apply(this, arguments);
  return "" + (__ || (__ != null ? __ : ''));
};

Yes, it gives right result. fixed.


ShapeManager.addTextureToVtkPolyData() should accept an image format

Looking at a file name extension results in brittle code, and the method should accept an image format rather than looking at a file name extension.  If absolutely necessary, the method signature can set a default of None for the format, in which case the file system can be looked at if it is not passed, but this is still not optimal code.

Hi Greg,
I disagree. I think the suffix determines the reader. The same principle is
used on Mac when you type the command "open ". Windows uses a similar
approach.
I could raise an exception...
On Tue, Dec 8, 2015 at 10:50 AM, Greg Von Kuster<EMAIL_ADDRESS>wrote:

Looking at a file name extension results in brittle code, and the method
should accept an image format rather than looking at a file name extension.
If absolutely necessary, the method signature can set a default of None for
the format, in which case the file system can be looked at if it is not
passed, but this is still not optimal code.
—
Reply to this email directly or view it on GitHub
https://github.com/pletzer/icqsol/issues/42.

--
Alexander Pletzer
+1<PHONE_NUMBER> cell
<EMAIL_ADDRESS>

Hmmm....I guess we'll disagree on this one.  My opinion : OO programming allows for much more sophisticated coding practices than is generally available with OS commands, so just because OS X or Windows provides commands that inspect file extensions doesn't mean that that is the optimal approach..   Just a few examples with case sensitivity show problems - what happens if the file extension is JPEG, JPG or PNG or any combination of case?  Handling all cases of how a user may name a file creates non-optimal code with higher maintenance overhead.  The optimal solution is for a caller to be able to pass the file type to the receive, especially when the caller can discover the file format by actually inspecting it's contents like Galaxy does.  Then there is no question because there is no dependency on how a particular user chooses to name a file.  Why do you feel that it would be wrong to enhance the ShapeHanlder's method signature to accept a file type with the default set to None, something like format=None.  And only when the format is None is the file extension inspected?

Done


Error HTTP 413 curl 22 during large file push

It seems related to the upload limit for nginx, how can i  add in nginx.conf the "client_max_body_size 50m;" line?

This fork uses the node-git-http-serverinstead of Nginx, see: https://github.com/catks/gitserver-http/pull/1
If you are facing a problem with a large file, it will probably require a fix for node-git-http-server


Read-only super role

This issue is imported from pivotal
For support-engineers, it would be very helpful if they could impersonate all users, without the ability to change anything.

Relevant if student support will become active for invite.


add transform option

opts.transform gives the user the option to alter the destination path
Say that you transform markdown files into html files for a static site generator. To have nice slugs, you not only want to change the extension type (.md to .html) but probably also create a folder with the original filename, containing a index.html file. See the README for an example
I did nog include a test for this new option. I was quite confused with the test.js file. I could use some guidance to write a proper test.

Thanks for the feedback, will update tonight

I did nog include a test for this new option. I was quite confused with the test.js file. I could use some guidance to write a proper test.

You could do something similar to https://github.com/sindresorhus/gulp-changed/blob/b2aa51fecd244de09881b90076f0ccf244f0a1cc/test.js#L100-L108

Updated according to your comments, except for the question about the fallback to an empty string. Do you prefer to handle that part differently?

Also note: https://github.com/sindresorhus/gulp-changed/pull/60#discussion_r101571358

👍

@sindresorhus, @SamVerschueren thank you for your patience and feedback


Add channel id

Mempool.space lightning explorer link format uses the full channel id, so wanted to add this to be able to link to the lightning explorer page for a channel. I think it would need changes to bitcoin-s for c-lightning though.

This works but we already have this info in the shortChannelId variable. I updated the json writer to include it in #342


Create ROS2.gitignore

Pull Request
Thank you for contributing to @dvcs/gitignore and https://www.gitignore.io.
New or update
Select the appropriate check box for this pull request. This helps when merging to ensure there are no conflicts with other templates or misunderstandings of how thee template list works.
New

[x] Template - New .gitignore template
[ ] Composition - Template made from smaller templates
[ ] Inheritance - Template similar to an existing template
[ ] Patch - Template extending functionality of existing template

Update

[ ] Template - Update existing .gitignore template

Details
replace this section with links and/or info about the proposed request

This was added to Github's gitignore and will be synced back to our repo very soon.
https://github.com/github/gitignore/pull/3662

Thank you @iwishiwasaneagle
This file was added to Github's gitignore and will be synced back to us soon https://github.com/github/gitignore/pull/3662


[Feature Request]Add support for regex flags for BigQuery

Is your feature request related to a problem? Please describe.
For consistency sake since we already support flags for Postgres and Snowflake; and since the project description states it supports BigQuery, I believe it makes sense to add support for re2 flags in BigQuery regex matches .
Describe the solution you'd like
Although BigQuery doesn't support flags as regex function parameters, it uses re2 and re2 supports flags by in-lining the flags (prepending the regex with (?flag)).
Additional context
This is a follow-up on issue #245 that added support for flags in Postgres, Snowflake, and Redshift. The idea was briefly introduced here

Having flags as parameters for BigQuery instead of having them in the test pattern has a few advantages such as

Easier flag validation
Simpler adapter switching

However, it adds a few lines of code so it's going to be more work to maintain.
Note In the PR, adding flags both as parameters and in the test pattern doesn't cause execution errors. The flags in the pattern take precedence over the ones in the flag (which is expected flag behavior).
e.g.:
regex: (?i)[A-Z] (case-insensitive letter match)
flags:  -i (case-sensitive match)
will be translated to  (?-i)(?i)[A-Z] and match any  lower- or upper-case character.

Closed via #253


Add integration tests for resource types with parameters

This adds integration tests for https://github.com/concourse/atc/pull/220, which was in response to https://github.com/concourse/concourse/issues/1690

The easiest way to verify this would be to have a resource type with a
rootfs stored in s3. Setting
params: {unpack: true} would allow it to work yes?
On Mon, Nov 13, 2017 at 16:24 Alex Suraci<EMAIL_ADDRESS>wrote:

@vito requested changes on this pull request.
In pipelines/fixtures/resource-types-with-params.yml
https://github.com/concourse/testflight/pull/3#discussion_r150669270:

@@ -0,0 +1,93 @@
+---
+resource_types:
+- name: custom-resource-type

type: git
privileged: ((privileged))
source:
uri: {{origin-git-server}}
branch: master
params:
foo: bar

Is this actually observed/verified in the tests somehow? In other words,
would these tests fail without the implementation, or would this just be
silently ignored?
This may be difficult to verify, to be honest; the git resource params
https://github.com/concourse/git-resource#parameters only has a few
params that we could really detect being used, and even then, this only
applies to the resource type fetching...
Maybe we could do something wacky like point it to a repo containing some
of the resource type's code as a submodule? But it even defaults to
fetching them by default, so we could only really turn it off and see that
it fails when the params are respected. Weird stuff.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/concourse/testflight/pull/3#pullrequestreview-76254344,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABv9GCzNeS_KvNrhNsMvBXhBvqK5Jg43ks5s2LOggaJpZM4QaIzn
.

--
Josh Ghiloni
<EMAIL_ADDRESS>

I know we talked about using the time-resource but I couldn't get it to effectively run in test-flight as it wouldn't recognize new versions. I switched to semver-resource so that it would work.

@jghiloni That's weird. If configured with interval it should return a version right off the bat. What happened?

IIRC, it was returning that there were versions available, but the job wasn't recognizing them. The semver works nicely though, and isn't TOO much larger of an image (50MB vs 20MB)

Not sure what you mean by not recognizing them. Did the TriggerJob just sit there and cause the test to hang?
I just want to make sure to understand this since to me using semver for this seems a bit overkill. Not really worried about the size, just the semantics are a bit confusing for the test.
Could you also include scripts that set up the bucket that it references? Then maybe I can try it out myself. I want to block the PR on that anyway just to make sure everything is reproducible.
Thanks!

I switched back to time after resolving the issues that were there (combination of PEBKAC and existing issues around radar scanning of resource types). As for the script that created the bucket, I don't have one but I can put one together real quick.

Ok @vito this should take care of everything you've asked for. I switched to using ((foo)) instead of {{foo}}, and added a script to upload a rootfs to a new s3 bucket (with a concourse task to generate the rootfs from a docker image, if necessary)

Hi @vito et al, is there a chance that this and https://github.com/concourse/atc/pull/220 could get an up or down vote today? It'll make our client's life a lot easier if we have it.

Takin' a look at this now. We had some work in flight in the radar that might remove the need for the sleep. I'll remove it and merge if so.

merged manually (and applied the sleep fix myself)!


async filters

I have the following code, which works:
alight.filters.validateGroup = class X
	watchMode: 'array'
	onChange: (oldGroups = []) =>
		setTimeout =>
			@setValue (group for group in oldGroups when group.counter > 0)

I'd like to know why i definitively need to use setTimeout.
When i remove the setTimeout function, it won't work anymore.
I understand that the filter is called "async", but i do not understand, why i need to handle this manually.
Also i thought the filter would be applied by watching the result of al-repeat, and if it changes, the onChange method is executed.
Can you explain, why the async part is necessary? It would be great to put this also to the docs.

try to use a not-fat arrow (->) for class, => can give you wrong "this"
alight.filters.validateGroup = class X
    watchMode: 'array'
    onChange: (oldGroups = []) ->
        @setValue (group for group in oldGroups when group.counter > 0)
        return

You inject the setValue method as prototype into my class, am i right?
Then the scope should be correct with =>.
Your approach with -> does not work, i need the setTimeout.
Some more information, about what i am doing:
I render groups into multiple select elements:
<option data-al-repeat="group in filteredGroups | validateGroup" data-al-option="group">
  {{group.title}} ({{group.counter}})
</option>

filteredGroups gets updated by a filtering method, when other input elements are used.
I do not use the filter "filter", because it is too limited for my use cases.
So as i understand the process works as follows:

filteredGroups gets modified
The watcher of al-repeat triggers change event of the value
The watcher pipes the changed result to the validateGroup filter
The validateGroup filter sets the new result, which will be rendered by al-repeat

Is my understanding correct?
Do you have some ideas how i can better debug this?

Ok, i found out, that my validateGroup filter runs, after this the filteredGroups scope variable gets updated.
When i execute the filtering manually, the filter is initiated correctly, and then it works.
Crazy stuff....

Omg, got the source of the issue.
My code did not "initialize" the filtering process on startup, i had to do a major change (complete new layout, other filtering technique) and before the initial filtering was executed by $watch methods.
A question to the $watch method of a scope: is it possible to exclude properties by name?
I need to track an array of objects, which have hierarchical relations (children and parent property) and therefore it ends in too much recursion.
Before i initalized the tracking of these variables by mapping a $watch method for each array item.
But this is not very performant, because the watcher automatically fires at initialization and the filtering runs five times on startup. I think this could be done better.
How do i use the function instead of a expression as $watch paramter, as mentioned here:
http://angular-light.readthedocs.org/en/latest/change_detector.html
?

Another idea: is it currently possible to watch only a defined depth?
Currently the option deep:true watches all levels.
I only need to watch the first two levels.

I made a simple example:
http://jsfiddle.net/gqtguovs/
I want the watcher to handle my text when i press the button "change title", not when i press "add watcher". The watcher callback should be only executed when the value was changed.
The other approach is to force the watcher only to watch to depth levels.
The third approach is, that i do not have further objects which have children or parent elements.
What do you think would be best?

wow, so much, do you still have a problem with your filter?

I want the watcher to handle my text when i press the button "change title", not when i press "add watcher".

the best solution is using a handler (function) instead of expression:
al-click="title = 'test after'"
change to
al-click="handler()"
But you can do something like this: http://jsfiddle.net/lega911/7vfpfdet/

or http://jsfiddle.net/lega911/p8qeu6q2/
or deferred: http://jsfiddle.net/lega911/g5fmuboo/

it is based on a text directive: https://github.com/lega911/angular-light/blob/master/src/text/textWatch.coffee

Maybe you want this? http://jsfiddle.net/lega911/o8erofa1/

http://jsfiddle.net/lega911/u3y3fy5q/
Am I right understand your issue?

http://jsfiddle.net/lega911/u3y3fy5q/
Am I right understand your issue?

This last example exactly matches my requirements (i also thought about this), but i have to implement this for 8 different watch handler (propably more in future). This is a bit "ugly" to have so much conditions doing the same.
Yes, i could abstract this into a method, but i think its still ugly.
The problem is, that i use different directives and can't use handler scope methods in all of them (e.g. al-select does not allow using a method as far as i understood).
So i try to use scope variables and change them, because this can be done in all cases.
I have links, select boxes, text input and propably soon some checkboxes/radio buttons.
If i modify the code to avoid the recursion due to hierarchy in the variable, i need minimum double amount of code for the same (estimated).
I handle around 500 products in the browser and when executing the filter 4 times initially there are 2000 iterations (it's still very very fast, but when i have 3000 products there will be 12000 iterations and so on...)
The rest of the code is optimized for performance, this is the only bottleneck i stumbled over. :D
If you have any further idea, please let me know. :-)

Another idea: is it currently possible to watch only a defined depth?

you can watch a function: http://jsfiddle.net/lega911/q7mh48wb/

when i have 3000 products there will be 12000 iterations and so on...

The fastest way is making result array by hand.
Angular Light can serve 10M+ watchers per second, and usually a bottleneck is DOM and rendering, so if you have a lot of DOM it is first reason of a slow application.

Angular Light can serve 10M+ watchers per second

I know that JavaScript can be fast and also handle huge of memory.

and usually a bottleneck is DOM and rendering, so if you have a lot of DOM it is first reason of a slow application.

No. Our Bottleneck are older mobile devices, which have less memory and slow processing power.
Currently my DOM is small, the page loads fast and is fast rendered.
I just want to avoid unnecessary iterations due to framework handling. That's all. :-)
Would it be possible to define instead of the option deep: true the depth of levels the watcher should look for, e.g. deep: 2 to watch exact 2 levels?
The advantage is, that this also would increase performance for normal watching structures.
Thank you for supporting me. :-)

look for, e.g. deep: 2 to watch exact 2 levels?

yes, why not
but anyway, custom-watch (watch function) can give you better performance

e.g. deep: 2 to watch exact 2 levels?

done, http://jsfiddle.net/lega911/o1qvt257/

This is really great!
Works as expected. :-)


Adjusts snippets for incomes to match current income labels

Ok, folks, this has been hanging out here too long.

@KevinSJ : If you're up for adjusting the Chinese snippets to match these, we'd really appreciate it. We also hope to have some input from case managers soon to provide clearer and more specific explanations for the various incomes, so we'll ping you when we get some of those in place. Thanks for your help so far!


[Task]: Adopt Design components to use rebranding colors

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
Components should change

Checkbox
Date Range Picker
Dropdown - multiselect
Tabs
Rectangular switch
Showcase carousal
Statusbar
Toggle
Toast undo redo

This has been verified in release.


switch dart2js build to depfile, remove Source.function

Description
Lookup the inputs to the dart2js build using the dependencies file it outputs, which contains a newline separated list of file uris. Updates Depfile class to parse this, and the dart2js rule to use it.
Removes Source.function and associated code, now that it is no longer used.

I'm not able to reproduce the windows failures on a local windows machine. I'll push an update to hopefully make them non-fatal...

Fixed issue where missing file would crash build :(


Google Maps element requires API key

In step 3, Google Maps requires API key, else it throws a warning. Should be mentioned in the documentation somewhere regarding the same, and a link to get API key.

Not really sure its worth mentioning something like that in the tutorial, as it could needlessly complicate the tutorial for novice developers.

Totally forgot to update this issue. It was not working (no graph being shown) on my system due to other issues in code (forgot to add dependency). But this works fine even without an API key for maps.
Closing this. 👍  thanks for update.


Mark secret parameters of forward plugins as secret

@repeatedly Could you check please?
As far as I seen, secret parameters of out_forward are already marked as secret: https://github.com/fluent/fluentd/blob/master/lib/fluent/plugin/out_forward.rb#L87

LGTM


require genMockFromModule

Is it possible to use genMockFromModule outside of a Jest test env? I'd like to use it in another testing framework.
Have you guys thought of moving the module mocker into another project?

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.

This'll be done soon as part of #161.


Allow setting border color from details window

Allow setting border color from the Person Details and Sub-Org Details windows.

Add a Border tab to the dialog, with a color picker…