text:
thermostat_over_climate: What do the external temperature sensor and the room temperature probe do?

The documentation states that for the thermostat_over_climate mode three things are required: an underlying climate entity, a room temperature sensor and an external temperature sensor.
What do the external temperature sensor and the room temperature probe actually do? How do they influence the control of a TRV (thermostatic radiator valve)? How does versatile thermostat calibrate the target_temperature based on the measured indoor temperature?
I've browsed the code and it seems like both these values are only used in the thermostat_over_switch mode but are irrelevant in the thermostat_over_climate mode. But maybe I'm missing something, my Python/Jinja knowledge is very small.

https://github.com/jmcollin78/versatile_thermostat#parameters-synthesis
Both values are only used by thermostat_over_switch .
But what I don't get is what the external sensor is for. There is no hint in the documentation except that it is crucial.
"Better Thermostat" also needs one, but is also lacking an explanation in the docs.

Better thermostat only uses the external temperature to turn on/off the heating when the average temperature of the past three days rises above or falls below a certain level. Or it might be similar to that, I didn't look it up, but it does not alter the heating power based on the outdoor temperature.
If Versatile Thermostat neither uses the external temperature nor the room temperature when using the thermostat_over_climatemode, it would be good to adjust the documentation, as that specifically says, these two sensors are the minimum requirement even in this mode.
Still, it would be sad if Versatile Thermostat really does not do anything with the indoor room temperature. After these two days of heating with my new setup (Aqara Zigbee room temperature sensors in every room and Eve Thermo TRVs in every room) I can see that the TRVs would certainly benefit of calibration. Some TRVs keep the valve open at 20-30% although the room temperature is more than 1°C above the target temperature. This kind of calibration is a key feature of Better Thermostat and I hoped that VTherm does this calibration too.

The documentation states that for the thermostat_over_climate mode three things are required: an underlying climate entity, a room temperature sensor and an external temperature sensor.
What do the external temperature sensor and the room temperature probe actually do? How do they influence the control of a TRV (thermostatic radiator valve)? How does versatile thermostat calibrate the target_temperature based on the measured indoor temperature?
I've browsed the code and it seems like both these values are only used in the thermostat_over_switch mode but are irrelevant in the thermostat_over_climate mode. But maybe I'm missing something, my Python/Jinja knowledge is very small.

You are right @maia , temperature and external temperature are now not used for thermostat over climate. It was the case in previous release but has been removed. Previously, it was to manage the security mode (when temperature sensors are not available). But, because in over climate mode, the regulation is done by the underlying climate itself, the security mode have been deactivated for over climate thermostat. I will fix the documentation.
As said by @masi , this have been added into the https://github.com/jmcollin78/versatile_thermostat#parameters-synthesis table.

@jmcollin78 Do you consider the possibility of adding a calibration feature similar to the one used by Better Thermostat? (see https://better-thermostat.org/configuration specifically "Calibration Type" and "Calibration Mode")

@jmcollin78 Do you consider the possibility of adding a calibration feature similar to the one used by Better Thermostat? (see https://better-thermostat.org/configuration specifically "Calibration Type" and "Calibration Mode")

I didn't know this thermostat and i'm discovering this thermostat. I will have a close look at it. The approach is quiet similar (thermostat over climate) but the calibration with 3 modes is interesting.

Some TRVs keep the valve open at 20-30% although the room temperature is more than 1°C above the target temperature. This kind of calibration is a key feature of Better Thermostat and I hoped that VTherm does this calibration too.

I don't think that force activation of Thermostat using the external temperature is a good idea. Turning on and off thermostat should stay a manual operation i think.

I agree. The only reason for integrating an outdoor temperature sensor is to adjust heating parameters based on the outdoor temperature. For example some central heating systems with radiators adjust the flow temperature based on the outdoor temperature. In the case of VTherm the outdoor temperature sensor could have an effect on the offset/calibration (see below).

Let's try to define what is a calibration and when it is needed:

a calibration is an offset added to a target temperature,
a calibration should be used when the target temperature cannot be reach after the current temperature is stable

In my setup the issue is that while the room temperature already is above the target temperature, the radiator keeps heating, as its internal temperature sensor has a different result. So what the calibration could do is:
Initial offset calculation (calibration phase)

For each climate entity, create a offset value with default 0.
Whenever the room temperature is more than 0,5°C above or below the target temperature, compare the room temperature with the TRV internal temperature sensor.
Adjust the offset value by adding/subtracting a fraction of the difference calculated in the step above. This could be done as a simple exponential moving average or (more complex but maybe interesting too) as bayesian model averaging in a time series model

Usage of offset (regular usage phase)

Once the calculated offset is stable, one could manually activate it to alter the underlying target temperature. For example if my room usually overheats by about 1°C (measured by the room temperature) and if my target temperature is 21°C, the VTherm entity will ask the underlying climate entity to heat to 20°C.
The offset should be adjusted based on additional calculations as described above, but with smaller adjustments.

We can see, the current temperature is never stable. Have you something different ? I fear it will be difficult to obtain something usefull.

In my case the TRVs adjusts the valve percentage constantly based on the differences between current temp and target temp. The fluctuations are tiny, as the TRV acts preemptively. Here's an example, you can see when I opened the windows, when the target temperature changed or when the door to the next room (which is slightly warmer) was open. And this room has the target temp set at 21°C most of the time, so you can see that it overheats constantly.

In the case of VTherm the outdoor temperature sensor could have an effect on the offset/calibration (see below).

Exactly. I found some interesting algorithm to achieve this.

Usage of offset (regular usage phase)
Once the calculated offset is stable, one could manually activate it to alter the underlying target temperature. For example if my room usually overheats by about 1°C (measured by the room temperature) and if my target temperature is 21°C, the VTherm entity will ask the underlying climate entity to heat to 20°C.
The offset should be adjusted based on additional calculations as described above, but with smaller adjustments.

Indeed ! If you are a "specialist" I can share with you the algorithm I found and I intend to test. It is really near what you are saying. It is based on PI (Proprotionnal and Integral methods).

I close this issue and keep the #129 instead.

You are right @maia , temperature and external temperature are now not used for thermostat over climate. It was the case in previous release but has been removed.

In this case, is there a reason to keep “Outdoor temperature sensor entity ID” as a mandatory field?

external temp is used for self-regulation algorithm now. If you don't have it, give it the meteo info

authors: jmcollin78 maia masi ovizii


text:
"Presence Management" do not make use of the Person entity

Hi there, I'm new to this integration but I'm liking it a lot so far, using the "valve" proportional control is giving me a far better regulation paired with underfloor radiant heating compared to the stock one.
Is your feature request related to a problem? Please describe.
I'm noticing that the presence management feature do not make use of the person entity.
Describe the solution you'd like
It would be useful to have the option to use the person "home" and "away" state as a trigger for occupancy instead of exposing another boolean sensor. It would also be great if the occupancy could switch between eco, comfort and boost mode instead of applying a different set point.
Thank you all, great work!

As for the presets, I‘m personally also a bit overwhelmed with eco/comfort/boost all having an _away preset too (plus a motion/activity preset). If I‘d design something just for myself, I‘d have 4 presets:

vacation (away longer than a day)
eco (when at work and for some rooms at night)
comfort (default when at home and awake)
boost (when sick)

…but from what I see having „only“ 4 presets isn‘t enough for others. :)

It would be useful to have the option to use the person "home" and "away" state as a trigger for occupancy instead of exposing another boolean sensor.

This should definitively work out the box. The only limitation is that groups of person don't work because a group of person is not a person in HA (bug ?).  So if you give one person, this should work (this is my test case so I think it is working) and you have more than one person you should do a template binary_sensor with occupancy

As for the presets, I‘m personally also a bit overwhelmed with eco/comfort/boost all having an _away preset too (plus a motion/activity preset). If I‘d design something just for myself, I‘d have 4 presets:

Hello @maia, I didn't understand why this response to this question. Is this related ?

As for the presets, I‘m personally also a bit overwhelmed with eco/comfort/boost all having an _away preset too (plus a motion/activity preset). If I‘d design something just for myself, I‘d have 4 presets:

Hello @maia, I didn't understand why this response to this question. Is this related ?

I was just a reply to „ It would also be great if the occupancy could switch between eco, comfort and boost mode instead of applying a different set point.“ from @albertofralbe above, thought he is referring to the _away presets, but maybe I misunderstood him?

... from @albertofralbe above, thought he is referring to the _away presets, but maybe ...

Ok. I don't understand he need another preset but more instead of using another set a temperature for the same preset, he would prefer to switch preset depending of occupancy (like it is done for Motion and Activity preset). That is my understanding.

Hello @albertofralbe ,

I'm new to this integration but I'm liking it a lot so far, using the "valve" proportional control is giving me a far better regulation paired with underfloor radiant heating compared to the stock one.

Great news !

It would be useful to have the option to use the person "home" and "away" state as a trigger for occupancy instead of exposing another boolean sensor.

This should definitively work out the box. The only limitation is that groups of person don't work because a group of person is not a person in HA (bug ?). So if you give one person, this should work (this is my test case so I think it is working) and you have more than one person you should do a template binary_sensor with occupancy

I encountered the same issue.
I think in PresenceBinarySensor, you should not just check for STATE_ON / STATE_OFF but also for STATE_HOME / STATE_NOT_HOME.
from homeassistant.const import STATE_ON, STATE_OFF,STATE_HOME, STATE_NOT_HOME
...
    async def async_my_climate_changed(self, event: Event = None):
        """Called when my climate have change"""

        _LOGGER.debug("%s - climate state change", self._attr_unique_id)
        old_state = self._attr_is_on
        # Issue 120 - only take defined presence value
        if self.my_climate.presence_state in [STATE_ON, STATE_OFF, STATE_HOME, STATE_NOT_HOME]:
            self._attr_is_on = self.my_climate.presence_state == STATE_ON || STATE_HOME
            if old_state != self._attr_is_on:
                self.async_write_ha_state()
        return

Hello @ben-ty ,
Thanks for your suggestion but the presence sensor management is in base_thermostat.py here:
async def _async_update_presence(self, new_state):
        _LOGGER.info("%s - Updating presence. New state is %s", self, new_state)
        self._presence_state = new_state
        if self._attr_preset_mode in HIDDEN_PRESETS or self._presence_on is False:
            _LOGGER.info(
                "%s - Ignoring presence change cause in Power or Security preset or presence not configured",
                self,
            )
            return
        if new_state is None or new_state not in (
            STATE_OFF,
            STATE_ON,
            STATE_HOME,
            STATE_NOT_HOME,
        ):
            return
        if self._attr_preset_mode not in [PRESET_BOOST, PRESET_COMFORT, PRESET_ECO]:
            return

        new_temp = self.find_preset_temp(self.preset_mode)
        if new_temp is not None:
            _LOGGER.debug(
                "%s - presence change in temperature mode new_temp will be: %.2f",
                self,
                new_temp,
            )
            await self._async_internal_set_temperature(new_temp)
            self.recalculate()

And it takes STATE_OFF,  STATE_ON,  STATE_HOME,  STATE_NOT_HOME as you can see. This is why it works for a single person in the presence sensor configuration.
The code you are showing in not the right one. This piece of code only listen to the presence_state of the VTherm itself which can only be 'on' or 'off'.
I hope it is clear.
You are not the first to ask this, but this doesn't work for groups of person, and I don't remember exactly why (i believe that a group of person is not an occupancy sensor but i'm not sure).

@jmcollin78 Sorry but presence_state of the VTherm itself is not on or off when a person (we don't speak about a group of persons) entity is selected for presence detection... it's home / not_home ;)
presence_state = self.hass.states.get(self._presence_sensor_entity_id)
--> Take a look at https://github.com/home-assistant/core/blob/dev/homeassistant/components/person/init.py
The code in base_thermostat.py is ok (the right preset is applied) BUT the code in binary_sensor.py need to be modified to handle home / not_home state so the sensor on the VTherm device display the right status.

Ah ok @ben-ty, I got your point. I though we were talking about group of persons. I will do a quick fix today.

In release 4.1.0. I cannot totaly test it in my environment. Please have a test in yours and keep me in touch.
Don't hesitate to reopen if not fixed.
Thanks for your report.

source: gharchive/issue

authors: albertofralbe ben-ty jmcollin78 maia

license: MIT

license_type: permissive

license_source: github-api


text:
com.google.inject.ProvisionException: Unable to provision

API
Java
Operating System
Windows 11 - WSL Ubuntu
It works on other platforms that are non-wsl.
JDK
1.8
Expected Behavior
Listening to broadcasts and connect to them
Actual Behavior
Fatal error during startup, aborted
com.google.inject.ProvisionException: Unable to provision, see the following errors:
while locating XXX
while locating XXX annotated with @com.google.inject.internal.Element(setName=,uniqueId=19, type=MULTIBINDER, keyType=)
Error injecting constructor, java.net.SocketException: Invalid argument
Caused by: java.net.SocketException: Invalid argument
at java.net.PlainDatagramSocketImpl.join(Native Method)
at java.net.AbstractPlainDatagramSocketImpl.join(AbstractPlainDatagramSocketImpl.java:178)
at java.net.MulticastSocket.joinGroup(MulticastSocket.java:323)
at javax.jmdns.impl.JmDNSImpl.openMulticastSocket(JmDNSImpl.java:463)
at javax.jmdns.impl.JmDNSImpl.(JmDNSImpl.java:420)
at javax.jmdns.JmDNS.create(JmDNS.java:81)

@nc-jeni i am running on WSL2 and Ubutnu without issues running 3.5.12.
Distributor ID: Ubuntu
Description:    Ubuntu 22.04.3 LTS
Release:        22.04
Codename:       jammy

Can this be closed?

@kaikreuzer no repsonse from OP. Safe to close

authors: melloware nc-jeni

repo: jmdns/jmdns

license_source: bigquery


text:
Allow async implementation of cache interface

I tried to connect angular-cache with ocombe/angular-localForage (in general - CacheFactory and mozilla/localForage) but it turns out that async storge implementation is not supported by CacheFactory . For example index.js#L417-L420:
$$storage().removeItem(this.$$prefix + '.data.' + key);
var keysJson = $$storage().getItem(this.$$prefix + '.keys');
var keys = keysJson ? utils.fromJson(keysJson) : [];
var index = keys.indexOf(key);

this clearly does not allow removeItem and getItem to return promises.
@jmdobry are you interested in implementing async storage in CacheFactory? Do you see any obvious cons for this idea?

See https://github.com/jmdobry/angular-cache/pull/146#issuecomment-72510034 and https://github.com/jmdobry/angular-cache/issues/108#issuecomment-45746483
I recommend js-data & js-data-localforage instead.

Ok

authors: jmdobry jrencz

repo: jmdobry/CacheFactory

license: mit


text:
[Source Request] <f1livegp.me>

Source website

Name: f1livegp.me
Link: https://f1livegp.me/f1/live3.html
Language(s): English

Other details
This website live streams F1 races. Is it possible to add this as an extension to aniyomi ? @jmir1

it is possible, but this type of extension will not be in this repository, as the main purpose of the app is anime and manga :)


text:
Issue with running Hugo using NPM scripts

Hi Joe. First of all, thank you for creating this great tool.
I am using HVM for a Hugo project.
I also have the following in package.json:
  "scripts": {
    "dev": "hugo server --navigateToChanged --gc"
  }

When I run npm run dev, I get the following error:
> hugo server --navigateToChanged --gc

sh: 1: hugo: not found

However, if I run hugo server --navigateToChanged --gc directly in the terminal, it works perfectly fine.
It would be great if HVM worked with npm run *, since I will not have to remember lengthy options (like --navigateToChanged).

~/.bashrc

I suspect npm run foo uses the Bourne shell not Bash, so it doesn't see the aliases. See:
https://docs.npmjs.com/cli/v9/using-npm/config#script-shell

Upon further review and research, this may not be possible without creating a separate script that npm can call directly, and that's not something I'm going to spend any time on. Moving this to discussions.

authors: hugo-sid jmooring

repo: jmooring/hvm


text:
Fixed Runtime.xcodeproj path for apple targets

Describe the issue (if no issue has been made)
The project file Runtime.xcodeproj does not exist at the root of the repository after running premake5, contrary to what the README.md file suggests.
Proposed Solution
This pull request adds Runtime/ to the beginning of the Runtime.xcodeproj path in the README.md sections for Apple targets to accurately reflect the file structure of the project.
Additional Notes
We might want to consider updating the README.md to include an example of compilation for Apple targets using Lumos.xcodeproj instead of Runtime.xcodeproj to maintain consistency with Windows and Linux targets.

Thanks!

source: gharchive/pull-request


text:
[WIP] Rails 5: Rack 2 breaks capybara's click_button method

Appears that rack 2 (for Rails 5) has broken capybara's click_button method.
Now getting "undefined method `normalize_params' for Rack::Utils:Module"
error when we using capybara's click_button method.

After porting to Rails 5, I get this error with at least 3 repos:

   Failure/Error: click_button "Create Task"
     NoMethodError:
       undefined method `normalize_params' for Rack::Utils:Module

To Reproduce, you can do the following:

Example repo (I remove all but 1 test) with error:

git clone https://github.com/jasnow/tasks5lite.git
After set up, run "rake" and you will get this error:

        Failure/Error: click_button "Create Task"
        NoMethodError:
          undefined method `normalize_params' for Rack::Utils:Module

After tracking down the trigger line, I found this line

Rack::Utils.normalize_params(params, key, value)

in this method:

merge_param!(params, key, value)

in this file:

capybara-2.5.0/lib/capybara/rack_test/form.rb

Also found these references interesting:

http://apidock.com/rails/Rack/Utils/normalize_params

Where I found this sentence:

This method is deprecated or moved on the latest stable version. The last existing version (v2.3.2) is shown here.

http://openwall.com/lists/oss-security/2015/06/16/14

Mentioned security-related changes to normailze_params inside rack.

https://thewebfellas.com/blog/rails-2-3-8-rack-1-1-and-the-curious-case-of-the-missing-quotes

Unclear if this is helpful.

This should already be fixed in the master branch, please try using that

You are right - Sorry for the issue.

When is the next release of capybara gem?

No current dare set - use the master branch

OK

I've faced this same issue and upgrading to a newer version fixed it for me. In my case, I've used:
gem 'capybara', '~> 2.8'

authors: embs jasnow twalpole

repo: jnicklas/capybara


text:
Avoid double null check

Since the pointers have already been confirmed to be non-null by the assert!() calls, there is no need to use .as_ref() and the raw pointers can be dereferenced directly. See the Null-unchecked version section of the standard library documentation for the pointer type's as_ref method.

LGTM, thanks! :)

authors: agraven jnqnfe

repo: jnqnfe/pulse-binding-rust


text:
🛑 UTN Aula Virtual is down

In cd03557, UTN Aula Virtual (https://aulavirtual.frbb.utn.edu.ar/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: UTN Aula Virtual is back up in a278379 after 57 minutes.

repo: joacoserra/statusPage


text:
🛑 Laravel Enso Docs is down

In b1b5c38, Laravel Enso Docs (https://docs.laravel-enso.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Laravel Enso Docs is back up in 612e352.


text:
🛑 Security & Telecom Systems is down

In 41be7c5, Security & Telecom Systems (https://security-telecom.ro) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Security & Telecom Systems is back up in d6c8533 after 5 minutes.


text:
Naive Questions:

Hello,
Can someone answer some naive questions:

Why agents have "backstory" and Task have "Description". Why do not we have only one? In the task, we provide agent then cann't it use Agent's backstory?
Why we need to provide both agents and tasks to the Crew? We are providing an agent to the task then why don't we use it internally? Can it be potentially dangerous if the agents list and task list do not match?  Say in the example posted on the web, the first task is task1 in the tasks array and by mistake, in the agents in write [writer, researcher] then what could happen?

Thanks

don't forget we have https://chat.openai.com/g/g-qqTuUWsBY-crewai-assistant to help and also the discord community https://discord.com/invite/X4JWnZnxPb

The backstory is the system prompt , while the description is the user's prompt ,so you need both of them to help the llm understand what to do exactly  .

2.The reason is that when you define the agents in the task  you just telling it what agents it should use for this exact task , while in the Crew you are telling it to load all the agents that it will need for the entire thing , so the Crew loads them and the task choses which one it needs from them .

repo: joaomdmoura/crewAI


text:
Update readme

The steps specified in the readme is stale and should be reviewed and updated.

Hi, Joe! I just wanted to register my interest in trying this out. For some reason, I'm getting the error ModuleNotFoundError: No module named 'bragi.metric_generator'; 'bragi' is not a package; that's after installing the package (or trying) using this git URL. If you find the time to tinker with the startup instructions, I'll be eager to try them out.

Hi, @robinsloan! I missed this, but am glad you're interested in taking bragi for a spin. I'll find some time this week to figure out what went wrong.

authors: joehoover robinsloan

repo: joehoover/bragi


text:
Rails Integration

Opening this issue as a reminder to consider interoperability with Rails partials — i.e. rendering an ERB partial inside a component and making it possible to render Phlex components in ERB templates.
Why?

Interoperability with Rails ERB templates might make it easier to migrate a project to Phlex step-by-step.

[x] Render Rails partials in a Phlex component
[x] Render Phlex pages from Rails controller
[x] Rails component helper to render components in Rails views
[x] #75
[ ] #57
[ ] #120

I don't find this epic issue particularly helpful, so I’m closing it. There's still work to be done on Rails helpers and documentation. I don't know if we could ever call that "finished" so let's not try.

repo: joeldrapper/phlex


text:
Generate the wrong code

The library couldn't generate the correct code with below schema, it only generate the 'feed summary''s code.
{
  "$schema": "http://json-schema.org/draft-04/schema#",
  "title": "home feeds list",
  "description": "home feeds list",
  "type": "array",
  "items": {
    "title": "feed summary",
    "type": "object",
    "properties": {
      "name": {"type": "string"},
      "name1": {"type": "string"}
    }
  }
}

The generated java code :
package com.example;

import java.util.HashMap;
import java.util.Map;
import javax.annotation.Generated;
import com.fasterxml.jackson.annotation.JsonAnyGetter;
import com.fasterxml.jackson.annotation.JsonAnySetter;
import com.fasterxml.jackson.annotation.JsonIgnore;
import com.fasterxml.jackson.annotation.JsonInclude;
import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.annotation.JsonPropertyOrder;

/**
* feed summary
* <p>
*
*
*/
@JsonInclude(JsonInclude.Include.NON_NULL)
@Generated("org.jsonschema2pojo")
@JsonPropertyOrder({
"name",
"name1"
})
public class Example {

@JsonProperty("name")
private String name;
@JsonProperty("name1")
private String name1;
@JsonIgnore
private Map<String, Object> additionalProperties = new HashMap<String, Object>();

/**
*
* @return
* The name
*/
@JsonProperty("name")
public String getName() {
return name;
}

/**
*
* @param name
* The name
*/
@JsonProperty("name")
public void setName(String name) {
this.name = name;
}

/**
*
* @return
* The name1
*/
@JsonProperty("name1")
public String getName1() {
return name1;
}

/**
*
* @param name1
* The name1
*/
@JsonProperty("name1")
public void setName1(String name1) {
this.name1 = name1;
}

@JsonAnyGetter
public Map<String, Object> getAdditionalProperties() {
return this.additionalProperties;
}

@JsonAnySetter
public void setAdditionalProperty(String name, Object value) {
this.additionalProperties.put(name, value);
}

}

Feed summary is the only type that needs to be generated, the parent schema
is just an array of feed summary which is simply a List in Java.
On 4 May 2015 23:24, "Sim Sun" notifications@github.com wrote:

The library couldn't generate the correct code with below schema, it only
generate the 'feed summary''s code.
{
"$schema": "http://json-schema.org/draft-04/schema#",
"title": "home feeds list",
"description": "home feeds list",
"type": "array",
"items": {
"title": "feed summary",
"type": "object",
"properties": {
"name": {"type": "string"},
"name1": {"type": "string"}
}
}
}
—
Reply to this email directly or view it on GitHub
https://github.com/joelittlejohn/jsonschema2pojo/issues/346.

@joelittlejohn Thanks for your quick feedback. It works for me.

authors: joelittlejohn simpleton


text:
added private modifier to configuration options where missing

The fields includeRequiredPropertiesConstructor, includeAllPropertiesConstructor, and includeCopyConstructor in the maven plugin cannot be configured in the pom. They are ignored by maven because they are missing the private modifier. This pull request adds this modifier.

Thanks all! :+1:

authors: MCSafety joelittlejohn


text:
Fix tailwind postcss setup #709

The --postcss parameter tells the Tailwind CLI to use our custom Postcss configuration.
The new @config directive doesn't work for some reason. However, specifying the config file directly as a CLI parameter seems to work.
I've also added the missing text-sm to the redesign.config.js, it's used by Pagy.
Fixes #709

Ah! Great catch. I removed the --postcss option to get @config working. Not realizing that we actually have a custom config.

authors: jkostolansky joemasilotti


text:
Index in a PDF

Thanks for this nice Plugin, I got a question, is it possible to make the index and its links clickable in a exported PDF?
It (the links) didn't work for me in the PDF.

Hi!
The plugin generates Obsidian-style internal links under the hood; a title:
# Some title

will generate the corresponding link:
[[#Some title]]

It seems these links aren't supported yet in PDFs: PDF export: Make links within same document functional
I can't find a workaround, so we'll probably have to wait for an official Obsidian update (or a dedicated plugin; reading the thread linked above, it seems some are in the work)

Hi! The plugin generates Obsidian-style internal links under the hood; a title:
# Some title

will generate the corresponding link:
[[#Some title]]

It seems these links aren't supported yet in PDFs: PDF export: Make links within same document functional I can't find a workaround, so we'll probably have to wait for an official Obsidian update (or a dedicated plugin; reading the thread linked above, it seems some are in the work)

Thank you very much for the information!

authors: johansatge mikajed


text:
Window Size is Too Small that Impossible to use

And I Can't extend size so I Can't use This Tool. Please fix this bug. Thanks.

Wow, this is weird.
This must be a unique case. Have you enabled windows display scaling or something similar?
Unfortunately, I can't fix it, I don't think?
I've moved past this project and I no longer even have a C# development toolchain installed😬😬
I should have frozen this project a while back. sorry.
(turns out I did update the Readme a while back, in the last lines there)
However,

Try right clicking the exe and disabling the DPI scaling from there.

try this tool that makes all windows resizable: http://www.digitallis.co.uk/pc/ResizeEnable/index.html
you should be able to resize the cdcrush after running this

You can try the nodeJS version https://github.com/john32b/cdcrush
it even has more features like batch converting.

Personally I still use the nodeJS version for everything.
Tell me how it goes


text:
Fix default value of "leases" attribute

It fixes an error:
           ================================================================================
           Error executing action `read` on resource 'vault_secret[secret/secretpath]'
           ================================================================================

           TypeError
           ---------
           no implicit conversion of String into Integer

           Cookbook Trace:
           ---------------
           /tmp/kitchen/cache/cookbooks/hashicorp-vault/libraries/vault_secret.rb:66:in `[]'
           /tmp/kitchen/cache/cookbooks/hashicorp-vault/libraries/vault_secret.rb:66:in `block (2 levels) in <class:VaultSecret>'
           /tmp/kitchen/cache/cookbooks/poise/files/halite_gem/poise/helpers/subcontext_block.rb:54:in `instance_eval'
           /tmp/kitchen/cache/cookbooks/poise/files/halite_gem/poise/helpers/subcontext_block.rb:54:in `subcontext_block'
           /tmp/kitchen/cache/cookbooks/poise/files/halite_gem/poise/helpers/notifying_block.rb:67:in `notifying_block'
           /tmp/kitchen/cache/cookbooks/hashicorp-vault/libraries/vault_secret.rb:60:in `block in <class:VaultSecret>'

That was my bad, introduced by https://github.com/johnbellone/vault-cookbook/pull/56. I'm sorry 😞

👍

repo: johnbellone/vault-cookbook


text:
Plunder allocator interface for ideas

In particular, the numeric_traits::make function resembles rebind::other.

deprecated


text:
Breaking component import. "... is declared but its value is never read"

Hello!
Tell me when you fix this problem, and will you? :)
I don’t know if this error was introduced .. or how to get around it, but there is no such problem with the functions.
Only when importing components. Thank you!

Found a similar problem with functions

Hmm, in this form it is not possible to use the extension (I returned to Vetur and turned off the script validation setting + installed the usual extension of EsLinter. Maybe it will help someone:

I discovered that I by adding "jsx": "preserve", in my jsconfig compilerOptions, it solved this issue.
Looks something like this:
{
  "compilerOptions": {
    "jsx": "preserve",
    // other stuff
  },
}

Fixed in https://github.com/johnsoncodehk/volar/commit/6f79f8fe77efe90ead624d6731de414cbb029331.

authors: Ailis7 esbeto johnsoncodehk

repo: johnsoncodehk/volar


text:
Vue.vscode-typescript-vue-plugin not working in a Cypress component test

I can't seem to get Vue.vscode-typescript-vue-plugin to recognize a vue SFC import in a Cypress component test in this repo https://github.com/segevfiner/vue-component-test, see cypress/component/HelloWorld.cy.ts, it doesn't seem to recognize the .vue import in that file. I'm not sure if I configured something incorrectly for it to do so.
[{
	"resource": "/Users/segevfiner/junk/vue-component-test/cypress/component/HelloWorld.cy.ts",
	"owner": "typescript",
	"code": "2307",
	"severity": 8,
	"message": "Cannot find module '../../src/components/HelloWorld.vue' or its corresponding type declarations.",
	"source": "ts",
	"startLineNumber": 1,
	"startColumn": 24,
	"endLineNumber": 1,
	"endColumn": 61
}]

This is a bug of TS plugin and not easy to fix, as a workaround you can consider to use takeover mode for now.

Figured as much. But it does feel like something worth fixing at some point.

Seems to have been fixed by https://github.com/johnsoncodehk/volar/commit/062895a2eba72ea659b98e23fcc5bbc443af7e7d (Released when 1.0.16)

authors: johnsoncodehk segevfiner


text:
Document styling the component

Purpose
As a developer I want to know what and how I can style the component.
Proposal
Customisation is currently done using CSS custom properties. We should document this.
Additionally, it might be helpful to explain how the children should be styled. In particular, the children should take up the parents' dimensions and so should have at least height: 100%;.
Acceptance criteria

[ ] As a developer I can read about at least one CSS custom variable I can override in the README

Issue #31 supersedes this one.

repo: johnwalley/allotment


text:
TypeScript type exports are not working for modern environments

Hi!
Issue description
When importing anything from allotment, my tsc gives me the following error:
Could not find a declaration file for module 'allotment'. '<(Project path omitted)>/node_modules/allotment/dist/modern.mjs' implicitly has an 'any' type.
  There are types at '<(Project path omitted)>/node_modules/allotment/dist/types/src/index.d.ts', but this result could not be resolved when respecting package.json "exports". The 'allotment' library may need to update its package.json or typings.ts(7016)

Possible context
This seems to be related to the issue reported in TS: https://github.com/microsoft/TypeScript/issues/52363
Environment
allotment 1.19.0
typescript 5.1.6
vite 4.4.4
NodeJS 18.13.0 (Mac M1)
I'm using "module": "esnext", "moduleResolution": "bundler", in my tsconfig. No other library so far has given the same error for me.
Possible workaround
While it is a hack, the workaround that worked for me so far was to create a typings file like the following:
declare module "allotment" {
  export * from "allotment/dist/types/src/index.d.ts";
}

I encountered this issue in version 1.19.3 on vite
could you please add the following content to package.json 'exports':
{
  "exports": {
    ".": {
      "require": "./dist/modern.mjs",
      "import": "./dist/module.js",
      "types": "./dist/types/src/index.d.ts"
    }
  },
}

indeed, even the ts error points to @pangxiaoli solution:
Could not find a declaration file for module 'allotment'. '/node_modules/allotment/dist/modern.mjs' implicitly has an 'any' type.
  There are types at '/node_modules/allotment/dist/types/src/index.d.ts',
  but this result could not be resolved when respecting package.json "exports".
  The 'allotment' library may need to update its package.json or typings.

authors: betafcc pangxiaoli spaguette


text:
Grammar and Visual Appeal

It may be a tiny thing but it should be like Ignore Warnings and Try Sending?, try to send seems grammatically incorrect.
The color white looks weird and on hover turns black that looks good,

My proposal for point 2 is => Make it black by default and change its border change on hover.

@theborakompanioni @editwentyone let me know what you guys think?

https://www.w3schools.com/howto/tryit.asp?filename=tryhow_css_buttons_animate1
Take a look at this button

repo: joinmarket-webui/jam


text:
Ram usage

It ate like 12gb of my ram in a minute of farming

Electron app moment 💀 (I'll look into this)

Gonna mark this as resolved by https://github.com/jojobii-arks/ngs-log-observer/commit/a0d1b910c71d14a25786cab37a9deb4dc4465aaf
I'll keep a look out if there's anything unusual. If it's still hogging 12GB, lmk.

repo: jojobii-arks/ngs-log-observer


text:
红米Note2 安卓5.0.2权限有问题

吧所有权限设置成询问之后，所有权限都提示成功，但是全都不成功。依然还是询问状态

是这样的，需要开启 5.0 权限申请，请添加 .requestUnderM(true)

authors: JackiePenghe jokermonn


text:
fix: accept None in model_from_dict to allow setting model fields…

… to NULL
This change allows for sending a dict with None as values; the model will then be updated to NULL for those fields. The existing handling in model_from_dict will drop dict keys if they're set to None.
Pull Request Checklist

[x] New code has 100% test coverage
[ ] (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
[ ] (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR
[x] Pre-Commit Checks were ran and passed
[ ] Tests were ran and passed

Description

Allows for sending a dict with None as values to model_from_dict; the model will then be updated to NULL for those fields. This changes the existing behavior of model_from_dict, which will drop dict keys if they're set to None, causing the record's field to be unchanged.

Today we needed that! If you need an hand you can ping us :) @ysnbyzli and me

Today we needed that! If you need a hand you can ping us :) @ysnbyzli and me

Will do! Sorry for the delay on this. I've been super sidetracked. I'm hoping I can get to it some time before the end of the weekend.

Today we needed that! If you need a hand you can ping us :) @ysnbyzli and me

Will do! Sorry for the delay on this. I've been super sidetracked. I'm hoping I can get to it some time before the end of the weekend.

I've actually got a working implementation that is non-breaking.

To get the new behavior, you pass exclude_unset as False into the any of the service calls that perform an update.

To get the new behavior, you pass exclude_unset as False into the any of the service calls that perform an update.

I haven't been able to test this myself yet as I have to run till later tonight, but just curious, does this change require you to explicitly set False if you are sending one or more values as None for it to work? What happens when you pass one field as None but others that aren't None in either scenario (True and False)? I do plan to do my own testing, but wanted to throw it out there. The "unset" nomenclature is a bit confusing to me since explicitly passing None, to me, is "set", it's just set to a null value. Is there perhaps a clean way to handle this without having to pass a flag for better DX?

To get the new behavior, you pass exclude_unset as False into the any of the service calls that perform an update.

I haven't been able to test this myself yet as I have to run till later tonight, but just curious, does this change require you to explicitly set False if you are sending one or more values as None for it to work? What happens when you pass one field as None but others that aren't None in either scenario (True and False)? I do plan to do my own testing, but wanted to throw it out there. The "unset" nomenclature is a bit confusing to me since explicitly passing None, to me, is "set", it's just set to a null value. Is there perhaps a clean way to handle this without having to pass a flag for better DX?

Hold off on testing.  I don't think the change I've pushed are what you want.  Give me a bit to test a few things out, and I'll push some additional changes shortly.

I was able to get the original issue with the mock repository resolved bit hit a new issue with the test_lazy_load in the repository mocks.  This looks to be unrelated, so for now, I set the lazy load test to skip the mock repository.  I'll address that issues in a followup PR.

I was able to get the original issue with the mock repository resolved bit hit a new issue with the test_lazy_load in the repository mocks.  This looks to be unrelated, so for now, I set the lazy load test to skip the mock repository.  I'll address that issues in a followup PR.

Beautiful, thanks for your help, @cofin!

authors: cemrehancavdar cofin rseeley

repo: jolt-org/advanced-alchemy


text:
[BB Model & Texture ONLY] Eidan 9000 (Prototype/Unefurbished)

"Tozai Line 15000 or 9000" was in the Patron Backlog, but I made it because I was planning to make Eidan 9000.
(As an aside, 9000 has no record of entering the Tozai Line.)
It is currently in temporary operation as a custom train at LPS.
However, just like when I made 10300, I can't code Java, so I only created models and textures this time.
I'll leave it up to Jonathan to change the specs from here.

Thank you! This looks great.

authors: ShimajiroOTS jonafanho

repo: jonafanho/Minecraft-Transit-Railway


text:
🛑 Shoppen is down

In a16fc72, Shoppen (https://shoppenin.mechelen.be/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Shoppen is back up in e50f62d.

repo: jonassalen/uptime-mechelen


text:
use 'Get.put(App())' and 'Get.put(App())' will generate two different instance

ATTENTION: DO NOT USE THIS FIELD TO ASK SUPPORT QUESTIONS. USE THE PLATFORM CHANNELS FOR THIS. THIS SPACE IS DEDICATED ONLY FOR BUGS DESCRIPTION.
Fill in the template. Issues that do not respect the model will be closed.
Describe the bug
use 'Get.put(App())' and 'Get.put(App())' will register two different instance
'Get.put(HomeController())' the key is  'HomeController?'
'Get.put(HomeController())'  the key is 'HomeController'
bugs: when using GetBuilder(init: Get.put(HomeController)) will generate two  instance. One is 'HomeController?' , another instance generates by 'GetBuilderState#isRegistered is false '.
Because if isRegistered is false, it use this way to registers GetInstance().put<T>(controller!, tag: widget.tag);
Reproduction code
NOTE: THIS IS MANDATORY, IF YOUR ISSUE DOES NOT CONTAIN IT, IT WILL BE CLOSED PRELIMINARY)
example:
import 'package:flutter/material.dart';
import 'package:get/get.dart';
import 'package:get/get_rx/get_rx.dart';

void main() => runApp(MaterialApp(home: Home()));

class Home extends StatelessWidget {
  final count = 0.obs;
  @override
  Widget build(context) => Scaffold(
      appBar: AppBar(title: const Text("counter")),
      body: GetBuilder(
        init: Get.put<HomeController>(HomeController()),
        builder: (HomeController controller) {
          return Center(
            child: Obx(() => Text("$count")),
          );
        },
      ),
      floatingActionButton: FloatingActionButton(
        child: const Icon(Icons.add),
        onPressed: () => count.value++,
      ));
}

class HomeController extends GetxController {}

To Reproduce
Steps to reproduce the behavior:

just run

Expected behavior
Get.put(HomeController()) should generate the key is 'HomeController'
Screenshots
If applicable, add screenshots to help explain your problem.
Flutter Version:
3.3.10
Getx Version:
4.6.5
Describe on which device you found the bug:
ex: Moto z2 - Android.
Minimal reproduce code
Provide a minimum reproduction code for the problem
Discussion
guess the Dart Type something error. We should unity the 'put' way

@jonataslaw  Hi, could you see this ?

I think the init of the GetBuilder is meant to be used with just the Controller Initialiser without Get.put().
This should stop creating multiple controllers:
class Home extends StatelessWidget {
  final count = 0.obs;
  @override
  Widget build(context) => Scaffold(
      appBar: AppBar(title: const Text("counter")),
      body: GetBuilder(
        init: HomeController(),
        builder: (HomeController controller) {
          return Center(
            child: Obx(() => Text("$count")),
          );
        },
      ),
      floatingActionButton: FloatingActionButton(
        child: const Icon(Icons.add),
        onPressed: () => count.value++,
      ));
}

If you want to use the HomeController outside of the GetBuilder you can initialise it somewhere else and then do:
GetBuilder<HomeController>(
          builder: (s) => Text('${s.counter}'),

See https://github.com/jonataslaw/getx/blob/master/documentation/en_US/state_management.md#usage

@Incro Yes, I know the right usage. But I think the Get.put<HomeController>(HomeController()) and Get.put(HomeController()) should keep same.

to retrieve an already created instance you can just use
final myController = Get.find<YourController>() ?? Get.create(Controller...) ou Get.put(Controller...);

authors: Incro chejdj kauemurakami

repo: jonataslaw/getx


text:
Non-monospace font used in code blocks in docs

The code blocks in the docs use variants of Lucida Sans on Windows, which is not a monospace font.
This makes the ASCII art a little confusing:

You probably want Lucida Console instead.

Thanks for letting me know! I'll make sure we're using a monospace font for Windows too

Should be good now, thanks!

authors: LachlanArthur jondot

repo: jondot/hygen


text:
sneakers_rails integration

Suggestion
I made a gem to integrate sneakers with rails and automate the process in https://github.com/jondot/sneakers/wiki/How-To:-Rails-Background-Jobs
Interested to hear feedback and maybe include the gem into the docs here
https://rubygems.org/gems/sneakers_rails

Thanks. It's at 0.0.3, perhaps it can mature a bit before we begin recommending it.

authors: essamhassan michaelklishin

repo: jondot/sneakers


text:
[BUG] Install on Mac Big Sur 11.2 fails to find neuron dependency (NEURON all caps)

Stack trace:
(mnedev) Alexs-MBP:hnn-core alexrockhill$ sudo -H python setup.py develop
Password:
running develop
running egg_info
writing hnn_core.egg-info/PKG-INFO
writing dependency_links to hnn_core.egg-info/dependency_links.txt
writing requirements to hnn_core.egg-info/requires.txt
writing top-level names to hnn_core.egg-info/top_level.txt
reading manifest file 'hnn_core.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
warning: no files found matching '*' under directory 'hnn_core/mod/x86_64'
warning: no files found matching '*' under directory 'hnn_core/mod/x86_64/.libs'
warning: no previously-included files found matching 'Makefile'
warning: no previously-included files found matching 'pytest.ini'
warning: no previously-included files found matching '.circleci'
warning: no previously-included files matching '*.pyc' found under directory 'hnn_core'
warning: no previously-included files matching '*' found under directory 'doc'
writing manifest file 'hnn_core.egg-info/SOURCES.txt'
running build_ext
Creating /Users/alexrockhill/software/anaconda3/envs/mnedev/lib/python3.9/site-packages/hnn-core.egg-link (link to .)
Adding hnn-core 0.2.dev0 to easy-install.pth file

Installed /Users/alexrockhill/projects/hnn-core
Processing dependencies for hnn-core==0.2.dev0
Searching for NEURON>=7.7
Reading https://pypi.org/simple/NEURON/
No local packages or working download links found for NEURON>=7.7
error: Could not find suitable distribution for Requirement.parse('NEURON>=7.7')
(mnedev) Alexs-MBP:hnn-core alexrockhill$ pip install neuron -U
Collecting neuron
  Downloading NEURON-7.8.2-cp39-cp39-macosx_10_9_x86_64.whl (11.5 MB)
     |████████████████████████████████| 11.5 MB 3.8 MB/s
Requirement already satisfied: numpy>=1.9.3 in /Users/alexrockhill/software/anaconda3/envs/mnedev/lib/python3.9/site-packages (from neuron) (1.20.1)
Installing collected packages: neuron
Successfully installed neuron-7.8.2

It works when I pip install manually though.

it's a problem with the develop install that i've noticed too. cc @cjayb does the .pyproject.toml trick not work for the develop install?

Not sure, I think it pyproject.toml might be in play when using pip+setuptools. I think this will work for a development-install (in the folder containing setup.py):
pip install -e .

Ah, yes, looking at the trace more closely, this is exactly what pyproject.toml is doing: making sure NEURON is installed before trying to build_mod in the setup of hnn_core.

authors: alexrockhill cjayb jasmainak

repo: jonescompneurolab/hnn-core


text:
installation error

Installation led to following error; haven't yet looked into this.

ok, fixed. i knew I'd seen this before. There were a two problems with your generator but I've added a PR which should fix these.

Fixed by #2

authors: jonesetc ksnyde

repo: jonesetc/ember-cli-horizon


text:
🛑 AL - Albania is down

In f24a808, AL - Albania (https://www.skyshowtime.com/al/help/) was down:

HTTP code: 404
Response time: 838 ms

Resolved: AL - Albania is back up in 85e573e after 9 minutes.

repo: jonesyriffic/gsp-sst


text:
🛑 AL - Albania is down

In bed36e4, AL - Albania (https://www.skyshowtime.com/al/help/) was down:

HTTP code: 404
Response time: 806 ms

Resolved: AL - Albania is back up in c7c010a after 10 minutes.


text:
Upgrade to Bytes 0.0.6

Quick update for evmap to use latest version of bytes
cargo test makes this look simple enough

This change is

Thank you! Sadly, this is a breaking change, so I'll have to hold off on merging this until the next major release. I think that'll come in not too long though :)

All good I'd just left the version as version = "11.0.0-alpha.1" based on master, but yeah its def a major version change.
I ended up moving my other deps back to bytes=0.5 anyway (:

Heads up that this will be fixed by #83, since it no longer uses the ShallowCopy trait, and thus we don't need to implement anything for Bytes at all!

authors: benbromhead jonhoo

repo: jonhoo/rust-evmap


text:
Sidemenu overlays on top if the view as a new view

New Issue Checklist

I have read the guidelines for contributing and I understand:

[ ] My issue is happening in the latest version of SideMenu.
[x] My issue was not solved in the README.
[x] My issue can not be answered on stackoverflow.com.
[x] My issue is not a request for new functionality that I am unwilling to build and contribute with a pull request.
[x] My issue is reproducible in the demo project.

Issue Description

On click left side menu or the right side menu button, controller is displayed on top of the home controller. Swipe gesture works fine. Having this issue after updating xcode to 11 - ios 13. Worked fine for mu previous projects.
Swift version - 4.2

Same problem happening with me... @AbdullahFaiz, Please label this issue as Bug

Hi @AbdullahFaiz, I have just tested this in the latest version of SideMenu using the demo project and everything works as expected. Please make sure you're following the README carefully, or provide more details using the demo project to reproduce this problem and reopen this issue.

authors: AbdullahFaiz Sinan-Noureddine jonkykong

repo: jonkykong/SideMenu


text:
Move App Id to secrets

As titled.

Codecov Report

Merging #6 (a280d3f) into main (c9ed54e) will decrease coverage by 51.27%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##              main       #6       +/-   ##
============================================
- Coverage   100.00%   48.73%   -51.27%
============================================
  Files           11       11
  Lines          513      513
  Branches        37       15       -22
============================================
- Hits           513      250      -263
- Misses           0      263      +263

see 6 files with indirect coverage changes

authors: codecov-commenter jonnydgreen

repo: jonnydgreen/nifty-lil-tricks-testing


text:
Make overlays a attr set

apparently overlays should be an attr set of overlays

Fixed upstream

repo: jonringer/nix-template


text:
Error thrown when input object does not inherit from Object.prototype

The rename function in index.js uses obj.hasOwnProperty, which is not defined when the object does not inherit from Object.prototype. In the most basic example, this would fail with TypeError: obj.hasOwnProperty is not a function:
const input = Object.create(null);
input.a = 0;
require('rename-keys')(input, key => key);

A more realistic example would be trying to rename the keys of an object returned from querystring, which in Node v6 stopped inheriting from Object.prototype.
const querystring = require('querystring'),
    rename = require('rename-keys');

rename(querystring.parse('a=1'), key => key);

(Live example of the same)

@jonschlinkert I will prepare a PR for this now.

repo: jonschlinkert/rename-keys


text:
Automatically set Rightarrow

At the moment, nfold=1 is not permitted. This means nfold should only be used when Rightarrow is used (and, indeed, there are graphical issues if it is not used). Therefore, it would make sense for the nfold parameter to automatically set this option.

Thank you for the suggestion. tikz-nfold can be used on arbitrary TikZ paths and is thus not restricted to tikz-cd, while Rightarrow is a tikz-cd feature. What would be your expected behaviour here? One idea that comes to my mind is for nfold to set Rightarrow depending on whether we are in a tikzcd environment (if that is possible).

Ah, I had overlooked that, it's a good point. I think that overloading the behaviour in a tikz-cd context specifically would be surprising behaviour. This behaviour would need to be implemented in tikz-cd itself (making use of tikz-nfold), so that would be the appropriate place to report it; I'll close this issue.
(I suppose that if #3 is implemented, then double distance will automatically be set, which would mean no package error. A user would still have to use Rightarrow to get the correct arrowhead, but it would be an improvement.)

authors: jonschz varkor

repo: jonschz/tikz-nfold


text:
Usage issue

I've got this running on heroku. But in Acceptable Use Policy line 21 of Prohibited Actions , they say we are not allowed to use such tools on heroku for scraping websites. Am i missing something?
Prohibited Actions:
21. Use the Service to access a third party web property for the purposes of web scraping, web crawling, web monitoring, or other similar activity through a web client that does not take commercially reasonable efforts to:
identify itself via a unique User Agent string describing the purpose of the web client; and
obey the robots exclusion standard (also known as the robots.txt standard), including the crawl-delay directive;

Puppeteer is used for more than just scraping websites, its a tool, and its up to you how you use it. Also from the quoted text:

that does not take commercially reasonable efforts to:
identify itself via a unique User Agent string describing the purpose of the web client; and obey the robots exclusion standard (also known as the robots.txt standard), including the crawl-delay directive;

So if that is how you want to use it, thats what you need to do to follow Heroku's rules.
Also this repo is just the buildpack for getting puppeteer running on Heroku and nothing more, so questions about Heroku's acceptable use might be better directed towards them.

authors: jontewks lynxionxs

repo: jontewks/puppeteer-heroku-buildpack


text:
SyntaxError: Unexpected token { in JSON at position 30     at JSON.parse ()     at parseBody (E:\gitlab\postman-to-openapi-test\node_modules\postman-to-openapi\lib\index.js:114:29)

SyntaxError: Unexpected token { in JSON at position 30
at JSON.parse ()
at parseBody (E:\gitlab\postman-to-openapi-test\node_modules\postman-to-openapi\lib\index.js:114:29)
at postmanToOpenApi (E:\gitlab\postman-to-openapi-test\node_modules\postman-to-openapi\lib\index.js:38:10)

when i add " "

Hi @ZhouQQQ,
This is a known issue, as we don´t have any way now to replace values the thing is that a body without replacing the variable values is not a valid JSON, that why the error is thrown.  I'm already has task for that here #76 that will solve this problem.
Best Regards.

authors: ZhouQQQ joolfe

repo: joolfe/postman-to-openapi


text:
Where is vim-airline color scheme?

Docs state: "VimBox includes vim-airline and Spacegray color scheme."
But there is no such color scheme. If I use vim-airline, the default color scheme is used.

Can you screenshot please?
The plugins (including vim color schemes) are all downloaded into ~/.vim/bundle/.
The spacegray color scheme is defined in the VimCleanColors bundle in that directory, including the corresponding airline theme.
The color scheme spacegray is activated here.

Are you using MacVim? (VimBox is tested on MacVim).

Yes, I use MacVim.
This is what I see in the VimCleanColors directory:
lightBright.vim lightBrightMuted.vim solarizedLight.vim spacegray.vim
and in the autoload directory:
lightBright.vim solarizedLight.vim
Could it be that a reference to vim-airline is wrong? How is the theme named?

Oh, I misunderstood. So, spacegray and many other colorschemes will use the special airline colorscheme that dynamically pulls colors from the colorscheme. I submitted a PR to airline itself to make it look better with spacegray, but it should be working out of the box dynamically. It's confusing because as you said, the actual airline colorscheme isn't called spacegray, it's called base16, and that loads the colors dynamically from a variety of colorschemes.

Please reopen if you have more questions.

authors: Robert-M-Muench jordwalke

repo: jordwalke/VimBox


text:
parquet -> arrow utf8-8 column conversion yields too many values

We're seeing an issue decoding parquet row groups containing UTF-8 string columns with many null values. Occassionally these conversions yield more values than specified in the parquet file metadata.  I traced this as far as arrow2::io::parquet::read::binary::iter_to_array with the problematic data set. After modifying that function to add some sanity check asserts, multiple test cases fail in the arrow2 test suite.
diff --git a/src/io/parquet/read/binary/mod.rs b/src/io/parquet/read/binary/mod.rs
index a0422f1a9..67812e2b1 100644
--- a/src/io/parquet/read/binary/mod.rs
+++ b/src/io/parquet/read/binary/mod.rs
@@ -53,6 +53,8 @@ where
             )?
         }
     }
+    debug_assert_eq!(validity.len(), capacity);
+    debug_assert_eq!(values.len(), capacity);
     Ok(utils::finish_array(data_type, values, validity))
 }

yields
---- io::parquet::write::list_large_binary_optional_v1 stdout ----
thread 'io::parquet::write::list_large_binary_optional_v1' panicked at 'assertion failed: `(left == right)`
  left: `12`,
 right: `15`', /home/ec2-user/src/rust/arrow2/src/io/parquet/read/binary/mod.rs:56:5

---- io::parquet::write::list_large_binary_optional_v2 stdout ----
thread 'io::parquet::write::list_large_binary_optional_v2' panicked at 'assertion failed: `(left == right)`
  left: `12`,
 right: `15`', /home/ec2-user/src/rust/arrow2/src/io/parquet/read/binary/mod.rs:56:5

---- io::parquet::write::list_utf8_optional_v1 stdout ----
thread 'io::parquet::write::list_utf8_optional_v1' panicked at 'assertion failed: `(left == right)`
  left: `12`,
 right: `15`', /home/ec2-user/src/rust/arrow2/src/io/parquet/read/binary/mod.rs:56:5

---- io::parquet::write::list_utf8_optional_v2 stdout ----
thread 'io::parquet::write::list_utf8_optional_v2' panicked at 'assertion failed: `(left == right)`
  left: `12`,
 right: `15`', /home/ec2-user/src/rust/arrow2/src/io/parquet/read/binary/mod.rs:56:5

---- io::parquet::write::utf8_required_v1 stdout ----
thread 'io::parquet::write::utf8_required_v1' panicked at 'assertion failed: `(left == right)`
  left: `0`,
 right: `10`', /home/ec2-user/src/rust/arrow2/src/io/parquet/read/binary/mod.rs:56:5

---- io::parquet::write::utf8_required_v2_compressed stdout ----
thread 'io::parquet::write::utf8_required_v2_compressed' panicked at 'assertion failed: `(left == right)`
  left: `0`,
 right: `10`', /home/ec2-user/src/rust/arrow2/src/io/parquet/read/binary/mod.rs:56:5

---- io::parquet::write::utf8_required_v2 stdout ----
thread 'io::parquet::write::utf8_required_v2' panicked at 'assertion failed: `(left == right)`
  left: `0`,
 right: `10`', /home/ec2-user/src/rust/arrow2/src/io/parquet/read/binary/mod.rs:56:5

I'm not entirely sure these debug asserts are correct, but I suspect they are.  I'm going to keep digging and see if I can isolate the issue further, but figured I'd throw up a bug report sooner than later.   Thanks!

Although perhaps this isn't the same root cause as we're seeing with the problematic data set; we see too many values, whereas those tests are failing with too few.  Here's the output from our data:
thread ... panicked at 'assertion failed: `(left == right)`
  left: `41795`,
 right: `41784`', /home/ec2-user/src/rust/arrow2/src/io/parquet/read/binary/mod.rs:56:5

I'm narrowing in on a fix, I've identified some bugs in the nullable dictionary read code.  Iterating on a fix here: https://github.com/danburkert/arrow2/commit/8d7cb88a29c7b6c9db12da6a91b0bbb00db4e691

Closed by #791. Thanks a lot for the report and the fix, @danburkert !

authors: danburkert jorgecarleitao


text:
Initial Changes for Upcoming API Update

Bot refactoring for updated API.

[ ] Rewrite README for clarity and completeness
[ ] Establish documentation setup
[ ] Implement API endpoints in the bot
[ ] Integrate API error handling into the bot

repo: josephistired/Discord-Pi-hole-Bot


text:
Lifecycle Callbacks?

Is there any way to set before_reflex etc type callbacks in my ViewComponentReflex::Component? It's been extremely painful without this capability.
Looking further, I think one root cause is that the library inherits off of StimulusReflex::Reflex, whereas many codebases use the base ApplicationReflex pattern... if one could configure a base reflex class, that would be helpful too

Lifecycle methods should now work.
You can now specify the reflex base class with reflex_base_class(new_class) in your component.

authors: joshleblanc suan


text:
unsupported graphic type spice

Hi Josh
LOVE the project I can see many many uses for it future please keep up the great work!
I am trying to get vdiclient on a linux box to connect to a win10 system I have the drivers installed from spice-space.org and also tried seeing if proxmox itself will let me switch the client from noVNC to spice but it always defaults back to novnc
DIsplay driver under dev mgr says red hat QXL
Can you please point me in the right direction here?

virt-viewer -V
virt-viewer version 11.0-1.el9 (OS ID: rhel9)

this is for connecting VM and containers running under the proxmox hypervisor. it doesn't directly connect to the machines.
you also have to remember RHEL has deprecated and removed spice in EL9

Correct I am connecting to the proxy main Hv ip I am using win11 as a client as wellBut I think I have a working conceptThanksSent from mobileOn Aug 17, 2024, at 1:15 PM, Damo Dye @.***> wrote:
this is for connecting VM and containers running under the proxmox hypervisor. it doesn't directly connect to the machines.
you also have to remember RHEL has deprecated and removed spice in EL9
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

repo: joshpatten/PVE-VDIClient


text:
Doesn't work if no internet connection

Package requires internet connection to read data from AMF_CVs. Could develop an option to download all the required data on package install (when an internet connection is required anyway), include that data within this package (for "from source" builds), or an caching option to save files as it downloads them so they don't need to be downloaded again.

Next thought, package could come with (or download on install?) data for latest version of standard, and then if future versions (or past I suppose) are required, downloads them then?

where would this data be downloaded to and saved on the computer? Linux/Windows/Mac differences?

Added option to use locally stored tsv files in https://github.com/joshua-hampton/ncas_amof_netcdf_template/commit/3baa6d5268a29f5cd5d39fbcd60b857354c33a34
User will need to download tsv files manually - instructions on how to will be added to documentation.

Forgot to add checking for using local files to the product_dict function in tsv2dict, so that a url request was still being made when trying to use local files. Fixed in https://github.com/joshua-hampton/ncas_amof_netcdf_template/commit/03d850e707b2db60a5a572eed0b0a06e38984889


text:
Support multiple Tilesets

What problem does this solve or what need does it fill?
It is ideal not to just use a single tileset but to actually define which tileset to use per tile. They MUST be identified by a integer and not a Handle to reduce cost of bytes per tile.
Describe the solution would you like?
This is mostly a Tile implementation level feature. There should be logic that has a HashMap at the Tile Map level of all handles and then the Tiles simply reference the index location.
Describe the alternative(s) you've considered?
n/a
Additional context
https://www.reddit.com/r/rust/comments/jntjp8/with_the_launch_of_bevy_game_engine_03_im/gb3u25f?utm_source=share&utm_medium=web2x&context=3

This is mostly closed with this update: https://github.com/joshuajbouw/bevy_tilemap/commit/5956f451094159ddede1486d123a8c546f0a37fb
Now you can just use multiple tile maps if you want to use multiple sprite sheets.


text:
Add config menu for adding file matchers

I wanted to customize the behavior of this awesome plugin to match an additional range of file types (styles.less, __init__.py, etc.). This PR adds a config menu option for adding additional file matching regex pieces.
The only caveat right now is that any changes in the config menu are not immediately reflected in the tabs - you have to either open a new tab or close an open tab for the changes to be displayed

This is awesome!
Thanks :)

Published v1.0.3

authors: cannoneyed joshwcomeau

repo: joshwcomeau/nice-index


text:
Perhaps a non-commonJS dist?

For those of use who work on projects that aren't using commonJS yet. Tnx.. Great project bdw!

Thanks, good suggestion. Im not a good person to answer this since I'm using commonjs for everything :) What would be the most convenient dist for you and your projects?

I'm also not sure what the convention behind this is.. but for now i was able to get by with just a few changes.. https://github.com/digitalmaster/react-visibility-sensor/commit/b32702079179000f5fb0b9c2bc20a9a8aebb82c4

@digitalmaster we're now publishing a UMD version to npmcdn, you can see an example here: https://github.com/joshwnj/react-visibility-sensor/blob/master/example-umd/index.html#L82

authors: digitalmaster joshwnj

repo: joshwnj/react-visibility-sensor


text:
Alternatives, for use with Redux

Reapop
react-hot-toast (i think)

Great, I'll look into that. Thanks.

@jossmac , are there any plans to have Redux integration in this package as well? In my Next.js app, I wrapped my main component around ToastProvider and all of this went inside Redux's Provider, but problem is I keep getting this error:

@shehi No, there's no plans to integrate with Redux.
The findDOMNode warning is unrelated, see #82

authors: geekdaniels jossmac shehi

repo: jossmac/react-toast-notifications


text:
UII - /vapor/proposals?status=active - List of active proposals

Create a page to display active proposals. The proposals are paginated. Present up to 20 proposals per page.
URL: http://localhost:3000/vapor/proposals?status=active
The smart contract is not yet deployed, but for the moment define something like this in the code and comment it:
/*
const { result: activeProposals } = await contract.functions.get_active_proposals({
  start: 0, // first page
  limit: 20, // proposals per page
  direction: 0,
});
*/

Use the following dummy data as the response of the call:
const activeProposals = [
  {
    id: 12,
    title: "Marketing campaign in Twitter and Youtube 🚀" ,
    summary: "Contact with influencers in twitter and youtube to show the advantages of koinos. Contact with influencers in twitter and youtube to show the advantages of koinos. Contact with influencers in twitter and youtube to show the advantages of koinos. Contact with influencers in twitter and youtube to show the advantages of koinos. Contact with influencers in twitter and youtube to show the advantages of koinos. Contact with influencers in twitter and youtube to show the advantages of koinos. Contact with influencers in twitter and youtube to show the advantages of koinos.",
    url: "https://koinos.io",
    monthly_payment: "100000000000", // 1000 koin
    starting_date: "1698796800000", // 2023-11-01
    ending_date: "1730419200000", // 2024-11-01
    beneficiary: "1B2Us2ynMLjCEGNJEewTXSzymQSQSeDUer",
    date_last_payment: "0", // no last payment,
    total_paid: "0",
    vapor: "500000000000", // 5000 vapor
    contributors: 12,
    total_shares: "500000000000",
  }, {
    id: 17,
    title: "Ledger hardware wallet support 🔐" ,
    summary: "Develop the interface to interact with Ledger in order to support hardware wallets in Koinos.",
    url: "https://medium.com/ledger/ledger-hardware-wallet-support-koinos-blockchain",
    monthly_payment: "100000000000", // 1000 koin
    starting_date: "1688169600000", // 2023-07-01
    ending_date: "1730419200000", // 2024-11-01
    beneficiary: "1B2Us2ynMLjCEGNJEewTXSzymQSQSeDUer",
    date_last_payment: "1698796800000", // 2023-11-01
    total_paid: "500000000000", // 5000 koin
    vapor: "7200000000000", // 72000 vapor
    contributors: 33,
    total_shares: "7200000000000",
  }, {
    id: 18,
    title: "Apply security patch in the Koin contract" ,
    summary: "Apply a security patch in the Koin contract. Apply a security patch in the Koin contract.Apply a security patch in the Koin contract.Apply a security patch in the Koin contract.Apply a security patch in the Koin contract.Apply a security patch in the Koin contract.Apply a security patch in the Koin contract.",
    url: "https://medium.com/security/apply-security-patch-in-the-koin-contract-123kekro456",
    monthly_payment: "120000000000", // 1200 koin
    starting_date: "1688169600000", // 2023-07-01
    ending_date: "1730419200000", // 2024-11-01
    beneficiary: "1B2Us2ynMLjCEGNJEewTXSzymQSQSeDUer",
    date_last_payment: "1698796800000", // 2023-11-01
    total_paid: "500000000000", // 5000 koin
    vapor: "7200000000000", // 72000 vapor
    contributors: 33,
    total_shares: "7200000000000",
  },
]

Do not display the VAPOR support. Instead of that, convert VAPOR into KOIN for a better readability: koin = (Number(proposal.vapor)/6e8).toString() + " KOIN".
For each proposal display how many months are covered with the current funding.
Add a button (or link) on each proposal to redirect the user to http://localhost:3000/vapor/proposals/{proposal_id}
At the top of the page add a link to the upcoming proposals:
http://localhost:3000/vapor/proposals?status=upcoming
Reference page to copy things related to koilib, env vars, header, etc: https://github.com/joticajulian/kondor-website/blob/main/website/src/views/nicknames/index.vue
BONUS: Load nicknames contract and call nicknames.functions.get_main_token({ account: "1MdqwaSBy6rbasPJ9vmg2pZFJSVZ29GFpZ" }) to get the nickname of the benefiary. If the nickname exists display it instead of the benefiary address.

Updated response:
const activeProposals = [
  {
    id: 12,
    title: "Marketing campaign in Twitter and Youtube 🚀" ,
    summary: "Contact with influencers in twitter and youtube to show the advantages of koinos. Contact with influencers in twitter and youtube to show the advantages of koinos. Contact with influencers in twitter and youtube to show the advantages of koinos. Contact with influencers in twitter and youtube to show the advantages of koinos. Contact with influencers in twitter and youtube to show the advantages of koinos. Contact with influencers in twitter and youtube to show the advantages of koinos. Contact with influencers in twitter and youtube to show the advantages of koinos.",
    url: "https://koinos.io",
    token_id: "15DJN4a8SgrbGhhGksSBASiSYjGnMU8dGL", // ID of token to receive payments in = KOIN
    payment_amount: "100000000000", // 1000 koin
    payment_interval: "monthly",
    starts_at: "1698796800000", // 2023-11-01
    ends_at: "1730419200000", // 2024-11-01
    beneficiary: "1B2Us2ynMLjCEGNJEewTXSzymQSQSeDUer",
    last_paid_at: "0", // no last payment,
    total_paid: "0",
    contributor_count: 12,
    allocated_amount: "500000000000", // 5000 vapor
    total_shares: "500000000000",
    fee: "2000000000", // 20 koin
  }
];

authors: frank-weijers joticajulian

repo: joticajulian/kondor-website


text:
URLs with 0-length host, port are considered valid

What it says.
We could fix this by adding s_req_host_start and s_req_port_start  states which require a single valid character before transitioning to the existing s_req_host and s_req_port states.

Fixed in 8da60bc.

authors: bnoordhuis pgriess

repo: joyent/http-parser


text:
关于每个线程一个epoll去监听listenfd

您好，我发现代码里面是每个线程一个epoll去监听客户端的连接，这样当新的链接到来时，每个线程都会得到可读事件然后调用accept，在我理解 这里应该只有第一个线程会接收到连接，其他线程都会accept失败（监听socket设置了非阻塞），这样会不会有性能损失？

这个确实是一个可以改进的点，也就是存在所谓的惊群效应。不过考虑到使用redis大多数场景是长连接，频繁发起连接并不是一种好的使用方式。因此当前的处理方式倒也不会有什么问题。

authors: fortrue rootzc

repo: joyieldInc/predixy


text:
#26: Adding an "overwrite" flag to read_dotenv, False by default.

PR for issue #26.
Allows for an optional "overwrite" parameter, False by default (so nothing existing breaks). Documented as well.
This would especially be useful when running tests, so test cases can still rely on a .env file and ensure it will override pre-existing environment variables that may not be suitable to use when running tests.

@jpadilla All tests passing. What's your release schedule look like for this? We would love to be able to use this in our project, but this issue is causing breakage in our tests—wondering if we can rely on this soon being available on PyPi or if we should make our own dependency?

@alexdlaird thanks you for working on this! I'll publish a new version today.

@alexdlaird done, thanks again
https://github.com/jpadilla/django-dotenv/releases/tag/1.4.2

authors: alexdlaird jpadilla

repo: jpadilla/django-dotenv


text:
Winning Items But Amazon Says page does not Exist

I''ve won a few giveaways with the script. but when trying to go to the page, it says it does not exist, or can not find the giveaway. Any idea how to ge

t around this?

When you tried to go back to those pages, were you logged in to the same account as the one you used to run the giveaway-grabber? Because if you are not logged in it won't find the page.
oh that's stinks! How long did you wait to go to those pages? I wonder if they expire after awhile

Yes sir. I was logged into the account. What's strange is I lost 3 giveaways because of that issue.  But had the same ending on the url on other giveaway and they seem to work okay

I think the issue is related with the time frame to redeem the winning entry. I just won an entry and i was lucky enough to see the notification while i was using the computer. I was wondering if i receive email notification and saw that email got stock on the spam folder. I saw that there was another entry i won 2 days ago when i tried to check the product to redeem i got the "I couldn't find what you were looking for!" message. Amazon gives a time frame to redeem so i believe it is the issue.

authors: hungstick jpchip vahapkalkan

repo: jpchip/giveaway-grabber


text:
Read time-out

Hi,
While trying to connect the unit 100-CPND004 with the dongle in a rpi3 (deban 10) I am getting a time-out in modpoll. I set up the thermostat in Int mode 49. Am I missing anything? Should I use another int mode number?

It works perfectly. Issue was a bad wiring on the device


text:
🛑 May Flower Seafood is down

In ae65768, May Flower Seafood (https://mayflowerseafoodrm.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: May Flower Seafood is back up in e4ec901 after 1 hour, 9 minutes.


text:
🛑 Launch Morrisville is down

In 45187c4, Launch Morrisville (https://LaunchMorrisville.org) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Launch Morrisville is back up in 33c6151 after 5 minutes.


text:
Release v0.3.0

resolves #95
first release for jlab 3.x!

The browser check CI seems to be flaky at the moment, failing on the command
jlpm add puppeteer@^4

But I manually checked the release and it seems to be good, so :heavy_check_mark:

repo: jpmorganchase/jupyter-fs


text:
Card

### Tasks
- [x] Development
- [x] Testing
- [x] Documentation
- [ ] https://github.com/jpmorganchase/salt-ds/issues/1360
- [ ] https://github.com/jpmorganchase/salt-ds/issues/1118
- [ ] https://github.com/jpmorganchase/salt-ds/issues/1321

Figma: https://www.figma.com/file/gbHcDLjP1M976avIlRDWl7/Card?t=KA1urnH7kky95OTC-0

2/6 - requires design review; pr up to move to core

31/3 waiting on dev review @joshwooding

April 14 - considering ADA adjustments and awaiting reply from @maj-stella on ada ticket

repo: jpmorganchase/salt-ds


text:
[WIP] Update inconsistent line heights

Created to see the impact of #4224

@dplsek the biggest change I notice is the fact that so many line-heights are no longer even so you get odd heights where components can grow in height.

You get better results if you bump the size if you +1 when round(size * 1.3) results in an odd number


text:
fix(dnd): add correct dragOverFromOutside behavior in week view

Hey,
This is an attempt to fix https://github.com/jquense/react-big-calendar/issues/2383.
The 2 issues I noticed were:

The selectors were listening to the wrong event ( dragOver instead of dragOverFromOutside )
The old dragOver code called the handleDrop which is not exactly true !

It's not breaking, as the old "dragOver" event was never emitted, so the code was never called (from my understanding)!
I'd love to have your feedback, and I'd be happy to add tests or better documentation if deemed necessary !
This is how it looks on the storybooks
https://github.com/jquense/react-big-calendar/assets/63306573/d8fbb4a2-80d9-4424-b344-950a6ec4c163

@achrafl0 I have to pull this code down and run it myself, but a quick look at you changes looks good. Give me a day or two to make sure there are no adverse effects.

@achrafl0 The issue I'm seeing right now occurs within that same Storybook example. There is an item there which you should not be able to drop on the calendar ("Draggable but not for calendar"). In current master it respects that in the week and day views. Attempting to drop on the timegrid does nothing. With your code it actually shows the slot preview and creates an event on drop.

Is there an update on this issue? I also need this feature. Thank you very much.

@achrafl0 If you could please pull in latest changes so that I can review this again

repo: jquense/react-big-calendar


text:
ES2017: Async function and await expression

async for function declarations/expressions, methods, and arrow expressions
A new await expression

Fixes #1079

There are still corner cases not tackled here, related to the cover grammar, invalid await as binding identifiers, etc. We also still need to align the error messages to follow V8. I will file some follow-up issues to keep track of those tasks.
And yet, I think the bulk of the implementation is at the feature complete stage and thus let's try to have this giant pull request landed first.

Current coverage is 100% (diff: 100%)

Merging #1597 into master will not change coverage

@@           master   #1597   diff @@
=====================================
  Files           1       1
  Lines        3775    3907   +132
  Methods         0       0
  Messages        0       0
  Branches      634     664    +30
=====================================
+ Hits         3775    3907   +132
  Misses          0       0
  Partials        0       0

Powered by Codecov. Last update 92bccda...7691383

I plan to land this one soon so we can continue with its further improvements.

:+1:

authors: ariya codecov-io mikesherov

repo: jquery/esprima


text:
update the plain-javascript README.md

Hi!
I'm just investigating the migration story from 0.x to 1.x and as a first step I thought I'd give the plain JavaScript example a go.  I got there after a little fiddling and so I thought I'd try and clarify the steps required in the README.md.  I hope that's okay?
Best,
John

Hi @rxaviers / @jzaefferer,
I've tried to submit a docs change (just a README.md change) and have already fallen foul of the Continuous Integration build.  The following slightly cryptic messages have been thrown up:

First line (subject) must be no longer than 72 characters
First line (subject) must indicate the component

Could you tell me what these errors refer to please?  I'd be happy to fix them up if I knew!

That's about the commit message, you can just ignore that. Or do a git commit --amend to try to address it.

Thanks @jzaefferer - I'll ignore it as advised!

@johnnyreilly thank you very much for your clarifications. I have left a couple of comments above. Just let me know on any questions.
Thanks

@johnnyreilly just wondering if you had a chance to look at my comments? Thanks

Hi @rxaviers,
I've made changes following your comments.  I've followed all your suggestions to the letter apart from the last.  I think the 2 separate examples are valuable as they are - I hope that's okay?  I've slightly rejigged it for clarity though.

@johnnyreilly for some reason your last commit uses a different author name ("johnnyreilly" instead of "John Reilly"), that's why it no longer passes our CLA check. You could update that with `git --amend --author=... or squash all commits into one, as long as the remaining commit has the right author information. Or I guess @rxaviers could do that for you, when landing this.

Tried and failed @jzaefferer :cry: Hopefully @rxaviers can sort it for me?

Hi @johnnyreilly, I can squash your commits and keep "John Reilly" as your correct one.

Great - go for it!

I've made changes following your comments. I've followed all your suggestions to the letter apart from the last. I think the 2 separate examples are valuable as they are - I hope that's okay? I've slightly rejigged it for clarity though.

Looks good to me.
I have left two comments though in other sentences. Sorry for the ping pong, which delays landing your PR. But, hopefully the last one.

Thank you @johnnyreilly

authors: johnnyreilly jzaefferer rxaviers

repo: jquery/globalize


text:
Long loading on huge string

Need to add some max buffer size or something

Solved with e57aa30ad5cbdd3c32dd167700d297bd65fa5ec9

repo: jraska/Console


text:
Saving a file from a port's work directory?

I've been having a problem building a certain port, and the maintainer has asked for a certain file from its work directory (/construction/xports/lang/gnat12/work/.build/config.log). It seems to me like /construction is something that Synth automatically makes and automatically destroys? In any case, I have no such directory on my machine. Is there a way to save that file so that I can access it after Synth finishes up? Thanks.

Break into the build interactively with the ENTERAFTER environment variable and the "test" mode.
e.g.
env ENTERAFTER=build synth test gnat12

when it stops and give you the prompt, you have two options:

from another terminal, access the files you want from the builder directory.  I can't remember what that is, something like "/var/synth/builders/SL09" maybe.

from the prompt, copy the file you want to say "/distfiles".  When you hit "exit" the file will be inside the distfiles directory where you can move it somewhere else.

Thank you!

repo: jrmarino/synth

license: isc


text:
FACIAL_LANDMARKS_68_IDXS indices incorrect?

Hello,
Perhaps I'm wrong.  But it appears as though the points defined in FACIAL_LANDMARKS_68_IDXS at https://github.com/jrosebr1/imutils/blob/master/imutils/face_utils/helpers.py are incorrect when cross checking them with https://www.pyimagesearch.com/wp-content/uploads/2017/04/facial_landmarks_68markup.jpg ?
When looking at the graphic in that last link, I'm not sure why FACIAL_LANDMARKS_68_IDXS is defined as:
FACIAL_LANDMARKS_68_IDXS = OrderedDict([
	("mouth", (48, 68)),
	("right_eyebrow", (17, 22)),
	("left_eyebrow", (22, 27)),
	("right_eye", (36, 42)),
	("left_eye", (42, 48)),
	("nose", (27, 36)),
	("jaw", (0, 17))
])

Correcting for a zero bounded index, shouldn't it be as below?
FACIAL_LANDMARKS_IDXS = OrderedDict([
    ("mouth", (48, 67)),
    ("right_eyebrow", (17, 21)),
    ("left_eyebrow", (22, 26)),
    ("right_eye", (36, 41)),
    ("left_eye", (42, 47)),
    ("nose", (27, 35)),
    ("jaw", (0, 16))
])

QA

A good way to debug this question would be to visualize each of the facial landmarks. If you swap out the original FACIAL_LANDMARKS_IDXS for the one you defined, how does the visualization change? Are you seeing more/less facial landmarks being drawn?

Pythons range indexing does not include the last element which is referenced. so list[0:4] gives the first three elements.
Further intel: https://stackoverflow.com/questions/509211/understanding-pythons-slice-notation

authors: itsMeBrice jrosebr1 qacollective


text:
2.7 tag no longer available on docker hub

My company uses docker for developer VMs, and one of these extends FROM jrottenberg/ffmpeg:2.7. Today I noticed that the 2.7 tag is no longer available on docker hub. Do you know why that is? Was it removed by you, or does docker hub have an automatic expiration on old tags?
Thanks!

Oh sorry about that, definitely not intended :
https://hub.docker.com/r/jrottenberg/ffmpeg/tags/
the 2.7 seems unavailable for some reason, I'll check with docker hub team as why.
Worst case I can branh from that git tag https://github.com/jrottenberg/ffmpeg/releases/tag/2.7 and have them rebuild.

Ok you should have a 2.7 tag back online, 2.7.7 to precise (the latest from the 2.7 series)
I think I know what happened, the dockerfile for that image had some dependancies (libvpx) not pinned, it was a simple git clone , make & make install. Of course webm/libvpx evolved to a point that broke 2.7.
If you look at the fix, it's mostly a pinned version of libvpx to 1.3

authors: jfairley jrottenberg

repo: jrottenberg/ffmpeg


text:
Motor Example: Updates for driving position and reactivating body

Show resetting position Motors and iterating through locations.
Add more activity to the sphere ball. Bounce it off wall and windmill multiple times (sometimes)

Thanks again!

authors: PhoenixIllusion jrouwe


text:
Enhancement: Provide Ruby binding impls

java_import Java::javafx.beans.binding.StringBinding

class RubyStringBinding < StringBinding
  def initialize(property:, method:)
    super()
    @property, @method = property, method
    bind @property
  end

  def computeValue
    property_value = @property.get
    return unless property_value
    property_value.__send__ @method
  end
end

The Java bindings provided from Bindings assumes Java reflective lookup so they cannot see our Ruby methods...

I should also point out that StringBinding is just some convenience methods on a type-erased type.
abstract StringBinding implements Binding<String>

so I think we can make a single binding to rule them all but we might not be able to leverage one of their abstract classes (unsure about that).

repo: jruby/jrubyfx


text:
By definition

По определению оператора &&.

Ага, когда читал лекцию понял, что я там бред написал


text:
🛑 Lychee is down

In b68db1d, Lychee ($SERVER_BASE/lychee/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Lychee is back up in d361447.

repo: jscmidt/upptime


text:
🛑 Tandoor is down

In d0660e2, Tandoor ($SERVER_BASE/recipes/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Tandoor is back up in 657bf1e after 10 minutes.


text:
🛑 Startpage is down

In bb100cf, Startpage ($SERVER_BASE) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Startpage is back up in 17735f6 after 1 hour, 2 minutes.


text:
Move triple lined comment support to be 1st class?

Hi there 👋
I know about https://github.com/jsdoc3/jsdoc/issues/1237 which was mainly about the existence of supporting /// docs - I'd like to go a step further and make a request that /// be considered a first class citizen in JSDoc.
This desire comes mainly because I'd like my dev-tools like VS Code to support this, and to make it less effort to do single line comments which should be attached to expressions for docs. Code which I think would benefit from switching to /// tends to be long sections of code with lots of small contextual comments (TypeScript here is documenting the params etc, here's the Swift equivalent)
Happy to explore the grunt work for this, if someone offers a bit of advice what it'd take to make it work 👍

I don't think the demand for this feature is large enough to justify making the change. Also, VS Code doesn't actually use any code from JSDoc, so there's no guarantee that a JSDoc change would result in a VS Code change.
I'd be happy to reconsider if more demand emerges, but for now, please use the commentConvert plugin.

Connecting this issue to the TypeScript PR which would bring support for /// inside vscode: https://github.com/microsoft/TypeScript/pull/39930

authors: hegemonic orta

repo: jsdoc/jsdoc


text:
dont warn using optional semicolons in classes

I was reading http://hacks.mozilla.org/2015/07/es6-in-depth-classes/ and it occurred to me we perhaps shouldn't  warn about unnecessary semicolons in classes e. g.
class a {
  b() {} ;
}

Does JSHint ignore some other places where the semicolon is unnecessary?

I don't think so. I was just surprised seeing a mozilla dev advocating a semi-colon style for classes. I haven't seen anyone advocating semi-colons for function declarations though.

Warning W032 describes this usage perfectly, so it is definitely appropriate.
It may not be tenable for users to disable W032 by name for just this case
(since that directive would also apply the bodies of class methods), but we
have no indication beyond that blog post that this is a common usage.
If anyone would prefer to use semicolons in this way, please comment here. With
enough interest (and preferably some rationale put in terms of code safety), we
could justify a change here--either removing the warning or the introduction of
a dedicated linting option.

authors: jugglinmike lukeapage nicolo-ribaudo

repo: jshint/jshint


text:
remove redundant CSS

Removing leftovers of #691 - that elements are already hidden by hidden attribute.
Not sure how the fix in JS are related here (but it was a bug anyway), but without these changes tests are breaking.

Thanks again for your contributions!! 😄 💯

authors: jshjohnson tinovyatkin

repo: jshjohnson/Choices


text:
ClassCastException when deleting missing path in specific circumstances

Summary
When deleting a missing path from a DocumentContext coming from a Configuration with the Option.SUPPRESS_EXCEPTIONS and the GsonJsonProvider, JacksonJsonNodeJsonProvider, JettisonProvider, JsonOrgJsonProvider or TapestryJsonProvider, a ClassCastException is thrown. The other modification methods, set, add, renameKey and put, might be affected too.
Steps to reproduce
Configuration configuration = Configuration.builder()
        .jsonProvider(new JacksonJsonNodeJsonProvider())
        .options(Option.SUPPRESS_EXCEPTIONS)
        .build();

DocumentContext documentContext = JsonPath.parse("{}", configuration);

documentContext.delete("$..this..path..is..missing");  // this throws a ClassCastException

I saw this regression too when trying to upgrade from 2.6

hitting this now, can we get this merged ?

authors: ben-manes binokkio joseph-neeraj

repo: json-path/JsonPath