Please Add Reno, NV to defund 12!!!

SUBJECT: Reno Residents for Defunding RPD
EMAILS:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>BODY:
I am a resident of Reno. I am writing to demand that Mayor Hillary Schieve and City Council to do everything in their power to adopt a budget that prioritizes our community well being and redirects funding away from the police in the next budget evaluation period.
We have seen mounting evidence that police departments are racist and ineffective institutions that put citizens at risk of injury and death, yet the police budget accounts for 36% of our general fund.
I ask that you redirect the majority of the $41.9M allotted for crime prevention toward community programs that provide residents with basic human needs like affordable healthcare and housing. With over 20% of our residents currently unemployed, we need funding to address the needs of people hurting from the effects of COVID 19, people experiencing homelessness, and the marginalized communities in our city.
I also STRONGLY URGE the City Council to enact legislation instituting a community elected Civilian Police Accountability Council with the following powers; ability to review and change disciplinary actions, review all violent interactions, and impose discipline including firing.
Our community is in pain along with the nation. We desperately need to change what we are prioritizing.
Thank you for your time,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]

Thank you for sharing - we will take a look!

Hello!
Please update the Reno,NV email. It has come to our attention that our sitting chief of police has been made ACTING CITY MANAGER, putting him in charge of the proposing the city's budget! this rewrite of the subject and body of the letter reflect that.
SUBJECT: Reno Residents For Defunding RPD, and Against Jason Soto as Acting City Manager
BODY:
I am a resident of Reno. I am writing to demand that Mayor Hillary Schieve and City Council to do everything in their power to IMMEDIATELY redirect the majority of the $41.9M allotted for crime prevention toward community programs that provide residents with basic human needs like affordable healthcare and housing, as well as youth programming and the arts.
We have seen mounting evidence that police departments are racist and ineffective institutions that put citizens at risk of injury and death, yet the police budget accounts for 36% of our general fund.
Im asking that chief of police Jason Soto IMMEDIATELY BE REMOVED AS ACTING CITY MANAGER. He CAN NOT be allowed to manage our city’s budget when he directly benefits from appropriating as much funding as he can to the police department. This is a conflict of interest that reflects negatively on all members of city council as he was elected unanimously.
I ask that use of militarized weapons such as tear gas, rubber bullets, and flash bangs be BANNED in Reno WITH ABSOLUTELY NO EXCEPTION!
I also STRONGLY URGE the City Council to enact legislation instituting a community elected Civilian Police Accountability Council with the following powers; ability to review and change disciplinary actions, review all violent interactions, and impose discipline including firing. I ask that NO additional funds be allocated for the creation/reformation of this board, its funding should come from the remaining police budget.
Thank you for your time,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]

Duplicate of: https://github.com/defund12/defund12.org/pull/166
Should we augment content, create new template, or close this out.

I'll close this issue and leave #472 open. thank you very much!

sorry! i didn't fully understand the question. please augment the content of #166 with the new subject and body in my previous comment. i'll close #472 once this underway


as a user I expect to be able to share a location using twitter

See [HEALTHSITE NAME] [URL of locality]
eg. http://healthsites.io/#!/locality/20329a84c03a49839c162ae10fe57410

it is already done, mark


Rewrite the date parser to accept more natural language queries

Should accept these and others:
now
today
tomorrow
next week
next year
March 30, 2015
3 days after tomorrow
123,456 seconds until 2025
January 3, 1976 plus 876 hours
arithmetic should support '+' as well as 'plus.' Same goes for minus.

Created this branch to hold my work: https://github.com/hickeroar/cahoots/tree/date_parser_rewrite

Currently have it supporting comprehension and evaluation of:

Ex:

1 hour from now
300,293 seconds until 2016

Just got unit tests left to add.


Add gpt-4 to model_to_encoding.json

Hi there, thank you for creating this awesome package. 🎉
I added the encodings for all GPT-4 models.
Best,
Chris

Thank you for merging!
Can you also trigger a new npm release? 🙏  That would be amazing. ☺

Just chiming in as it might help someone in the future.
I was doing  tiktoken.get_encoding("gpt-4"); but it errored out saying "Error: Invalid encoding". Turns out I need to use: tiktoken.encoding_for_model("gpt-4");
Reading the docs help :)


Deployer V5 upload method ignores custom ssh port

Q
A

Issue Type
Bug

Deployer Version
5.0.0

Local Machine OS
Max OS X Sierra 10.12.4

Remote Machine OS
Centos 7.3

Description
Just downloaded deployer.phar from site, adjusted deployer.php from v4.3
Steps to reproduce
Just try to upload local file to remote host with custom ssh port.
Content of deploy.php
<?php
namespace Deployer;
require 'recipe/yii2-app-basic.php';

// Configuration
//set('default_stage', 'dev');
set('keep_releases', '5');
set('ssh_type', 'native');
set('ssh_multiplexing', true);

set('repository', 'git@some.repo/some.git');

add('shared_files', []);
add('shared_dirs', ['vendor', 'web/uploads', 'web/assets']);

add('writable_dirs', ['web/assets', 'web/uploads']);

add('clear_paths', [
    'tests/',
]);

// Servers

host('some-server.com')
    ->user('user')
    ->port(22222)
    ->identityFile('~/.ssh/id_rsa')
    ->set('deploy_path', '/home/user')
    ->stage('dev');

// Tasks
desc('Uploading configs from local machine');
task('config:upload', function () {
    upload('config/db-dist.php', '{{release_path}}/config/db.php');
    upload('web/index-dist.php', '{{release_path}}/web/index.php');
})->onStage('dev');

after('deploy:update_code', 'config:upload');

desc('Restart PHP-FPM service');
task('php-fpm:restart', function () {
    run('sudo systemctl restart php-fpm.service');
});
after('deploy:symlink', 'php-fpm:restart');

// [Optional] if deploy fails automatically unlock.
after('deploy:failed', 'deploy:unlock');

Output log
[Symfony\Component\Process\Exception\ProcessFailedException]
  The command "rsync -azP  config/db-dist.php user@some-server.com:/home/user/releases/19/config/db.php"
   failed.
  Exit Code: 255(Unknown error)
  Working directory: /Users/Megatron/project/
  Output:
  ================
  Error Output:
  ================
  ssh: connect to host some-server.com port 22: Connection refused
  rsync: connection unexpectedly closed (0 bytes received so far) [sender]
  rsync error: unexplained error (code 255) at /BuildRoot/Library/Caches/com.apple.xbs/Sources/rsync/rsync-51/
  rsync/io.c(453) [sender=2.6.9]

I've even tried adding some options for upload method:
upload('config/db-dist.php', '{{release_path}}/config/db.php', ['timeout'=>null,'options'=> "'ssh -p 22222'"]);

But this doesn't work too.

Duplicate #1218
Try master or:
upload('config/db-dist.php', '{{release_path}}/config/db.php', ['timeout'=>null,'options'=> "-e 'ssh -p 22222'"]);

Using 'options' doesn't help at all.
Used master and now all files uploaded, but for each file I need to enter passphrase to my private key 😞 any solutions for this?

Use ssh-agent?

All works, thanks!


Service Bus:  Backplane operations should exist exclusively in mgmt package

Currently there are some management operations still in the core service-bus SDK.  (create_queue, create_topic, etc.)
In the long term these should all be deprecated/exist only in the mgmt. package.  This is likely a discussion that should be had with @kaerm to plan out the actual work and process this will be done by.  Feel free to reach out to @KieranBrantnerMagee for context.
Brief context:  Users submitted some changes to historical backplane operations within the core SDK, but after discussion with architects, was concluded that we'll be eventually moving away from this pattern, so creating this issue to track this work for posterity.
See linked PR for more detail and history.

After discussions with the Service team, it appears that there is demand for the Atom management APIs for the forseeable future due to SAS key support.  We'll be revisiting how best to emplace this functionality within the lib with Track2 work, but it will ostensibly stay in some form or another.
Will likely be reopening #9293 when we begin bringing this logic into Track2 previews, to incorporate the good work ibigbug did.


CmdDef.from_defaults() signature

The class method CmdDef.from_defaults() currently takes (defaults.cmd_def, defaults.pulse_library) to build the CmdDef. It should just take defaults, as that is what the method name implies - and it's shorter.
Also why can't this just be backend.defaults().cmd_def() that returns the cmd_def? That seems the most natural thing. Once the defaults is deserialized and validated, then we can add these methods to it to enhance useability.

Also why can't this just be backend.defaults().cmd_def() that returns the cmd_def? That seems the most natural thing. Once the defaults is deserialized and validated, then we can add these methods to it to enhance useability

This is what we would prefer. However, up until recently the idea was that objects such as Defaults and Configuration contained only the transport information, so it was a confession the even allow the method build_cmd_def on Defaults. There is no reason the user ever has to see the raw transport cmd_def and pulse_library representation.

agree - I think my IP PR handles this. PulseDefaults has op_defs (which the user doesn't have to interact with, but that is its "CmdDef", although CmdDef is also renamed). from_defaults becomes deprecated.

IP PR == #3526 ?

Yup. Lauren hopes to tackle Saturday morning SA time.

seems resolved via #3526


update authorization_details

this PR aims to close #190
update authorization_details format parameter like mentioned in this issue https://github.com/italia/eudi-wallet-it-docs/issues/190
the authorization_details parameter has been updated following this specification https://openid.net/specs/openid-4-verifiable-credential-issuance-1_0.html#section-5.1.1

This PR could be considered deprecated (see PR #371)

overseeded by https://github.com/italia/eudi-wallet-it-docs/pull/371/files


Build Errors using master branch with UE 4.19.2 on Ubuntu

On an Ubuntu 17.10 machine with clang 4.0.1, libc++ standard libary I'm seeing a few errors when compiling using either the editor or the unreal build tool.  All of the errors are error: ISO C++11 does not allow conversion from string literal to 'char *' [-Werror,-Wwritable-strings]

/.../UnrealProjects/PyMuws/Plugins/UnrealEnginePython/Source/UnrealEnginePython/Private/UEPyModule.cpp:1555:30: error: ISO C++11 does not allow conversion from string literal to 'char *' [-Werror,-Wwritable-strings]
ue_py_register_magic_module("unreal_engine.classes", py_ue_new_uclassesimporter);
^
/.../UnrealProjects/PyMuws/Plugins/UnrealEnginePython/Source/UnrealEnginePython/Private/UEPyModule.cpp:1556:30: error: ISO C++11 does not allow conversion from string literal to 'char *' [-Werror,-Wwritable-strings]
ue_py_register_magic_module("unreal_engine.enums", py_ue_new_enumsimporter);
^
/.../UnrealProjects/PyMuws/Plugins/UnrealEnginePython/Source/UnrealEnginePython/Private/UEPyModule.cpp:1557:30: error: ISO C++11 does not allow conversion from string literal to 'char *' [-Werror,-Wwritable-strings]
ue_py_register_magic_module("unreal_engine.structs", py_ue_new_ustructsimporter);
^

Another block...

/.../UnrealProjects/PyMuws/Plugins/UnrealEnginePython/Source/UnrealEnginePython/Private/Wrappers/UEPyFSlowTask.cpp:77:15: error: ISO C++11 does not allow conversion from string literal to 'char *' [-Werror,-Wwritable-strings]
char *text = "";
^
/.../UnrealProjects/PyMuws/Plugins/UnrealEnginePython/Source/UnrealEnginePython/Private/Wrappers/UEPyFSlowTask.cpp:156:26: error: ISO C++11 does not allow conversion from string literal to 'char *' [-Werror,-Wwritable-strings]
char *default_message = "";

And also this...

In file included from /.../UnrealProjects/PyMuws/Plugins/UnrealEnginePython/Intermediate/Build/Linux/B4D820EA/UE4Editor/Development/PythonAutomation/Module.PythonAutomation.cpp:2:
/.../UnrealProjects/PyMuws/Plugins/UnrealEnginePython/Source/PythonAutomation/Private/PythonAutomationModule.cpp:12:57: error: ISO C++11 does not allow conversion from string literal to 'char *' [-Werror,-Wwritable-strings]
PyObject *py_automation_module = ue_py_register_module("unreal_engine.automation");

Hi, just add
(char *)

in front of any static string:
ue_py_register_magic_module((char *)"unreal_engine.enums", py_ue_new_enumsimporter);

Feel free to make a pull request ;)

Thanks for the example.  Even I should be able to handle that :)
I'll submit a pull request when I'm back on it tomorrow.

Closing as pull request #413 has been merged


enable building on jdk 17 and up

This change makes it possible to build biometric_storage for Android with any JDK >= 17. Without this change 17 is required, preventing it to be build on e.g JDK 21.
closes #117

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


Separate delay/timeout/sideload functionality from loadFn

Summary
See https://github.com/gregberge/loadable-components/issues/245 and https://github.com/gregberge/loadable-components/pull/483
Basically the aim is to avoid overloading the loadFn so that it is less confusing.
New guard option
Previously to implement a delay, timeout etc the import promise would need to be wrapped, and the wrapped promise was responsible for resolving to the component. This was confusing because it gave the impression that it was possible to resolve to something else (e.g. a named export), and in fact this worked but only on the client side. The new guard option does not have to resolve to anything and its resolved value is consistently ignored, so does not create this confusion:
import loadable from '@loadable/component';

const LoadableComponent = loadable(() => import('./xyz'), {
  guard: (importPromise) => pMinDelay(importPromise, 1000).then(() => {}),
});

guard has access to props
import loadable from '@loadable/component';
const DynamicLoadableComponent = loadable(() => import('./xyz'), {
  guard: (importPromise, props) => pMinDelay(importPromise, props.timeout).then(() => {}),
});

Detailed choices

The babel plugin has been modified to throw an error if the import creator does anything except directly returning an import() statement. This is not necessary for this feature to work (e.g. the error could be hidden behind a flag, or generate a warning now and and an error later). I think it's a good idea though, because it removes the trap where wrapping the import() statement (e.g. loadable(() => import('./xyz').then((mod) => mod.Component))) works on the client side but not the server side
To replace the practice of implementing delays/timeouts by wrapping the import promise (e.g. as in the docs for delay), a new flag is added called guard. The consequences of using it are much the same, but this allows the babel plugin to enforce that the import() promise is never wrapped (therefore avoiding the client vs SSR confusion). Again, not necessary for the feature to work, but a good idea IMO.

Test plan

Existing tests have been changed where necessary and new tests have been added for new functionality

perhaps guard should be called clientLoadExtra,  so its 100% clear that it doesn't work on the server side

by a some reason I am thinking about driver like asyncDriver.
or something very simple, like onAsync/ifAsync, as long as it takes a place "on" "async"
or not async, but lazy. Because it works when component is lazy, not eager loaded.

bot pls, discussion still in progress

bump

Let's keep it open

🧟


Include production flag in push notification registration.

This flag signals whether the build is a beta or dev build vs an app store build and is required in the beta build once https://github.com/artsy/gravity/pull/9529 is merged.

:+1: Looks good to me.


Example 3 - include forms not listed?

Now understanding &c. as etc. to indicate an incomplete listing, do you want to encode other forms of #Bryght(e)# that don't appear in the vocabulary? Reasoning that we can find occurrence and auto-generate pointers more easily than Tolkien and make the listing complete. Then of course to distinguish between Tolkien's list of forms versus a more complete one.

I think the right way to do that would be to set up a separate abstract lexicon, lemmatise Sisam's texts linking to that lexicon, and linking Tolkien's entries to that lexicon.
My lemma lattice (originally developed for NT Greek Lexicons) is highly suitable for this use case too where you want to both be able to reference a particular spelling and the lexeme as a whole.
The citations in Tolkien can help bootstrap the lemmatisation but ultimately would not be the primary assertion of the lemmatisation.
Of course, there is a lot that can be done with both Sisam's texts and Tolkien's glossary even before any of this is done.

Separate abstract lexicon works but I assume you still want to account for the refs that Tolkien has to Sisam's texts. Yes? Which within <orth> would be <oRef>, using the @target attribute to point to an occurrence mentioned by Tolkien. Assuming you want to accept each reference to a text number + line number or just text number as a string. (Not encoding the text number separate from the line number.)


[wazuh agent] Protocol configuration in wazuh agent settings

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] When the request is meant for an existing plugin, I've added its name to the title.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Wazuh agent should be able to connect to Wazuh server using TCP or UDP connection. However, right now, it seems wazuh agent does not work with the TCP and it only works with UDP. In the opensense, wazuh agent conf does not allow specifying the protocol explicitly and uses the default TCP which does not work.
Using TCP, the agent manage to enroll on the server, but it does not manage to send logs, after several connection attempt, = it tried to re-enroll but as the agent is already created it fails and return duplicate agent name error.
64 at connect_server(): INFO: Closing connection to server ([<IP_ADDRESS>]:1514/tcp)
96 at connect_server(): INFO: Trying to connect to server ([<IP_ADDRESS>]:1514/tcp)

324 at w_enrollment_send_message(): DEBUG: Request sent to manager
349 at w_enrollment_process_response(): INFO: Waiting for server reply
360 at w_enrollment_process_response(): ERROR: Duplicate agent name: <agent-name> (from manager)
360 at w_enrollment_process_response(): ERROR: Unable to add agent (from manager)

Describe the solution you'd like
A clear and concise description of what you want to happen.
(e.g. I would like an input field in the /ui/firewall/alias which would add .... to ....)
I would like an input field in /ui/wazuhagent/ for protocol.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
I have not find any alternative for the problem yet.
Additional context
Add any other context or screenshots about the feature request here.
This is a working config that I have on a server wazuh agent.
<client>
    <server>
      <address><IP_ADDRESS></address>
      <port>1514</port>
      <protocol>udp</protocol>
    </server>
    <crypto_method>aes</crypto_method>
  </client>

@AdSchellevis thank you very much. It is working now with the UDP.
Syncookie are disabled in our firewalls. so that was not the cause.


Add heading link styles

Issue #1069 asks for documentation around links that are also heading elements. Our current system does not have a method for styling links with black text so I am adding a new class to apply to links.
Additions

a-link__heading which colors the link black like an h4 would be, as well as alters the default hover appearance to mirror what is currently done on cfgov.

Testing

Ran the a11y tests.

Screenshots
Notes

Not sure if this is the correct process for adding these links

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the CFPB development guidelines
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Browsers

[ ] Chrome on desktop
[ ] Firefox
[ ] Safari on macOS
[ ] Edge
[ ] Internet Explorer 9, 10, and 11
[ ] Safari on iOS
[ ] Chrome on Android

Accessibility

[ ] Keyboard friendly
[ ] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

Thanks @alexm118! Could you possibly also add a documentation edit to the links page to demonstrate how this proposed style would look?

Thanks @alexm118! Could you possibly also add a documentation edit to the links page to demonstrate how this proposed style would look?

Is this currently used in the cf.gov codebase somewhere?

Is this currently used in the cf.gov codebase somewhere?

I think this PR misconstrues what the original issue (#1069) wants. As I understand it, the desired change is to add written content to the DS's Links page describing how links within headings are styled.

I think this PR misconstrues what the original issue (#1069) wants. As I understand it, the desired change is to add written content to the DS's Links page describing how links within headings are styled.

Headings that linked, that is. Where the entire text of the heading is a link.

Headings that linked, that is. Where the entire text of the heading is a link.


Sync eng/common directory with azure-sdk-tools repository for Tools PR 1365

Sync eng/common directory with azure-sdk-tools repository for Tools PR 1365

@weshaggard  - I would need your help here as well - to run any tests for python which involves Cosmos emulator.

https://dev.azure.com/azure-sdk/public/_build/results?buildId=717286&view=results

Closing out this PR on behalf of automation.


Dynamic Nested Navigation Table Calling API Multiple Times

Hi,
I have implemented a dynamic nested navigation table with three levels which are database names, schema names and finally the table/view names, everything works perfectly including the lazy evaluated preview of data. The only problem what I face is as soon as I pick the desired tables and hit the load button, the entire chain of requests will be trigger again for each table. That means, if I pick 10 tables, then there are getting triggered again the calls for retrieving database names including their schemas for 10x, a server overhead which isn't necessary. The data source is a database exposed via a REST API which doesn't allow direct access. Since there is only one entry-point, I don't figure out how to achieve that. I have already tried to buffer results, but to my understanding for each query a new M container is getting created which cannot use that "previously" cached data.
Any help would be appreciated.
Thanks,
Patrick

@CurtHagenlocher I got it working. I think important to mention is that the "keyColumns" of the Navigation Table actually holds the parameter values (for instance, in a separate column) required to trigger the table/function of the dataColumn. In case of multiple parameters, the consumption would look like this (Parameter2):
Table.NavigationTableView(() => renamed, {"PageId","Parameter2"}, GetWikipediaPage, [             Name = each [Title],             ItemKind = each "Value",             ItemName = each "Value",             IsLeaf = each true         ])
Thanks for your quick and helpful response,
Patrick


feat: added GitLab container scanning report parser

Adding GitLab container scanning report parser

https://docs.gitlab.com/ee/user/application_security/container_scanning

@madchap @damiencarol

@damiencarol I've commited the changes per your request, please let me know if you've got anything to add

I've update the code per formatting

@damiencarol should I apply unique_id_from_tool here as well ?

@natebwangsut yes for this parser.

@Maffooch @damiencarol I've added unique ID for parsing + rebased ontop of latest dev as well, kindly review the PR


Should calling value() on an xml_node return the same as text().as_string()?

Take the following snippet:
<myElement id="0052a71acac348ff93f5680aa9c125eb">
  <name>myname</name>
</myElement>

Logically calling value() on the <name> node seems like it should return the same as text().as_string() and it's a lot cleaner looking. Instead it returns empty string.

Nevermind. It’s child_value() I was looking for 😉


Change name as it is already used by an existing linux package: "A text-based user interface for Pidgin"

What is the problem you're trying to solve?.
A clear and concise description of the use case for this feature. Please provide an example, if possible.

Change name as it is already used by an existing linux package: "A text-based user interface for Pidgin"
https://yum-info.contradodigital.com/view-package/base/finch/

Describe the feature you'd like
A clear and concise description of what you'd like to happen.
Additional context
Add any other context or screenshots about the feature request here.

Closing as duplicate of Issue #26


Improve captcha handling

Sometimes the captcha fails.  Clearly there are multiple layouts which look the same to make life harder for automated scripts. It would be nice if we handled at least some of these layouts.

Here's an example
error-2022-02-01_07-54-59.txt

error-2022-02-01_07-54-59.html.txt
In this case what has happened is that buster has intercepted the bit we click to get it to load.  I think this only happens when buster can't solve it for us, but I don't know.


Take (an often failing) first screenshot on iOS Simulator only when taking screenshots is enabled

Resolves https://github.com/wix/detox/issues/841

Let's consider moving all events to DeviceDriverBase, instead of spreading them in Device and multiple deviceDrivers

@rotemmiz , it is not really easy to move things inside DeviceDriver itself. I think it needs a bigger refactor together with a Device class. But, at least I managed to do emitting only on drivers level - base class creates and possess a protected emitter instance which derived classes can use. It is a compromise, but I don't think we can do better without a heavier rewriting right now.
Currently, some unit tests are not passing, but the code is almost ready to the merge from my point of view, so I'd ask you to take a look again. I updated the description of the PR, btw.

Tests are fixed, btw.

Also resolves: https://github.com/wix/detox/issues/856 .

Marked as WIP, since there is an issue with the suggested SimulatorLogPlugin implementation on high loads.

Removed [WIP] label since the iOS simulator log plugin was stabilized.


Bur_2020

What article on docs.github.com is affected?

What part(s) of the article would you like to see updated?

Additional information

Welcome to burgerreto


HTTP proxy support

I am wondering if the library supports Http Proxy. We are trying to use the ES connector using AWS Cloudwatch Consumer and for some reason the API is unable to store the Lease information in DynamoDB. I get the HTTP connection timed out error.
I tried passing the proxy settings through JAVA_OPTIONS in call to the "java -jar" as System properties but that doesn't seem to make any difference.
Any known fixes for this?
Here's the Stack trace to the effect.
2016-02-20 02:57:10,741 INFO  AmazonHttpClient - Unable to execute HTTP request: connect timed out
java.net.SocketTimeoutException: connect timed out
at java.net.PlainSocketImpl.socketConnect(Native Method)
at java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:345)
at java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:206)
at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:188)
at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
at java.net.Socket.connect(Socket.java:589)
at sun.security.ssl.SSLSocketImpl.connect(SSLSocketImpl.java:656)
at org.apache.http.conn.ssl.SSLSocketFactory.connectSocket(SSLSocketFactory.java:524)
at org.apache.http.conn.ssl.SSLSocketFactory.connectSocket(SSLSocketFactory.java:403)
at com.amazonaws.http.conn.ssl.SdkTLSSocketFactory.connectSocket(SdkTLSSocketFactory.java:118)
at org.apache.http.impl.conn.DefaultClientConnectionOperator.openConnection(DefaultClientConnectionOperator.java:177)
at org.apache.http.impl.conn.ManagedClientConnectionImpl.open(ManagedClientConnectionImpl.java:304)
at org.apache.http.impl.client.DefaultRequestDirector.tryConnect(DefaultRequestDirector.java:611)
at org.apache.http.impl.client.DefaultRequestDirector.execute(DefaultRequestDirector.java:446)
at org.apache.http.impl.client.AbstractHttpClient.doExecute(AbstractHttpClient.java:863)
at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:82)
at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:57)
at com.amazonaws.http.AmazonHttpClient.executeOneRequest(AmazonHttpClient.java:706)
at com.amazonaws.http.AmazonHttpClient.executeHelper(AmazonHttpClient.java:467)
at com.amazonaws.http.AmazonHttpClient.execute(AmazonHttpClient.java:302)
at com.amazonaws.services.dynamodbv2.AmazonDynamoDBClient.invoke(AmazonDynamoDBClient.java:3240)
at com.amazonaws.services.dynamodbv2.AmazonDynamoDBClient.createTable(AmazonDynamoDBClient.java:1047)
at com.amazonaws.services.kinesis.leases.impl.LeaseManager.createLeaseTableIfNotExists(LeaseManager.java:118)
at com.amazonaws.services.kinesis.clientlibrary.lib.worker.KinesisClientLibLeaseCoordinator.initialize(KinesisClientLibLeaseCoordinator.java:173)
at com.amazonaws.services.kinesis.clientlibrary.lib.worker.Worker.initialize(Worker.java:373)
at com.amazonaws.services.kinesis.clientlibrary.lib.worker.Worker.run(Worker.java:317)
at com.amazonaws.services.kinesis.connectors.KinesisConnectorExecutorBase.run(KinesisConnectorExecutorBase.java:95)
at com.amazonaws.services.logs.connectors.samples.elasticsearch.ElasticsearchConnector.main(ElasticsearchConnector.java:38)
Thanks

I am facing same issue

Here is how I fixed this issue : Iam sure there is a better way to do this
Replaced KinesisConnectorExecutorBase with MyOwn class called CustomeKinesisConnectorexecutorBase: Copy code from KinesisConnectorExecutorBase.java
/*
 * Copyright 2013-2014 Amazon.com, Inc. or its affiliates. All Rights Reserved.
 *
 * Licensed under the Amazon Software License (the "License").
 * You may not use this file except in compliance with the License.
 * A copy of the License is located at
 *
 * http://aws.amazon.com/asl/
 *
 * or in the "license" file accompanying this file. This file is distributed
 * on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
 * express or implied. See the License for the specific language governing
 * permissions and limitations under the License.
 */
package com.amazonaws.services.kinesis.connectors;

import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;

import com.amazonaws.services.kinesis.clientlibrary.lib.worker.KinesisClientLibConfiguration;
import com.amazonaws.services.kinesis.clientlibrary.lib.worker.Worker;
import com.amazonaws.services.kinesis.connectors.interfaces.IKinesisConnectorPipeline;
import com.amazonaws.services.kinesis.metrics.interfaces.IMetricsFactory;
import com.amazonaws.ClientConfiguration;
public abstract class CustomeKinesisConnectorexecutorBase<T, U> implements Runnable {
    private static final Log LOG = LogFactory.getLog(KinesisConnectorExecutorBase.class);

    // Amazon Kinesis Client Library worker to process records
    protected Worker worker;

    /**
     * Initialize the Amazon Kinesis Client Library configuration and worker
     *
     * @param kinesisConnectorConfiguration Amazon Kinesis connector configuration
     */
    protected void initialize(KinesisConnectorConfiguration kinesisConnectorConfiguration) {
        initialize(kinesisConnectorConfiguration, null);
    }

    /**
     * Initialize the Amazon Kinesis Client Library configuration and worker with metrics factory
     *
     * @param kinesisConnectorConfiguration Amazon Kinesis connector configuration
     * @param metricFactory would be used to emit metrics in Amazon Kinesis Client Library
     */
    protected void
            initialize(KinesisConnectorConfiguration kinesisConnectorConfiguration, IMetricsFactory metricFactory) {
        val clientConfig = new ClientConfiguration()
    clientConfig.setProxyHost("add-proxy-host")
    clientConfig.setProxyPort(add-proxy-port)
    clientConfig.setProxyUsername("add-proxy-username")
    clientConfig.setProxyPassword("add-proxy-password")
    clientConfig.setPreemptiveBasicProxyAuth(false)
        KinesisClientLibConfiguration kinesisClientLibConfiguration =
                new KinesisClientLibConfiguration(kinesisConnectorConfiguration.APP_NAME,
                        kinesisConnectorConfiguration.KINESIS_INPUT_STREAM,
                        kinesisConnectorConfiguration.AWS_CREDENTIALS_PROVIDER,
                        kinesisConnectorConfiguration.WORKER_ID).withKinesisEndpoint(kinesisConnectorConfiguration.KINESIS_ENDPOINT)
                        .withFailoverTimeMillis(kinesisConnectorConfiguration.FAILOVER_TIME)
                        .withMaxRecords(kinesisConnectorConfiguration.MAX_RECORDS)
                        .withInitialPositionInStream(kinesisConnectorConfiguration.INITIAL_POSITION_IN_STREAM)
                        .withIdleTimeBetweenReadsInMillis(kinesisConnectorConfiguration.IDLE_TIME_BETWEEN_READS)
                        .withCallProcessRecordsEvenForEmptyRecordList(KinesisConnectorConfiguration.DEFAULT_CALL_PROCESS_RECORDS_EVEN_FOR_EMPTY_LIST)
                        .withCleanupLeasesUponShardCompletion(kinesisConnectorConfiguration.CLEANUP_TERMINATED_SHARDS_BEFORE_EXPIRY)
                        .withParentShardPollIntervalMillis(kinesisConnectorConfiguration.PARENT_SHARD_POLL_INTERVAL)
                        .withShardSyncIntervalMillis(kinesisConnectorConfiguration.SHARD_SYNC_INTERVAL)
                        .withTaskBackoffTimeMillis(kinesisConnectorConfiguration.BACKOFF_INTERVAL)
                        .withMetricsBufferTimeMillis(kinesisConnectorConfiguration.CLOUDWATCH_BUFFER_TIME)
                        .withMetricsMaxQueueSize(kinesisConnectorConfiguration.CLOUDWATCH_MAX_QUEUE_SIZE)
                        .withUserAgent(kinesisConnectorConfiguration.APP_NAME + ","
                                + kinesisConnectorConfiguration.CONNECTOR_DESTINATION + ","
                                + KinesisConnectorConfiguration.KINESIS_CONNECTOR_USER_AGENT)
                        .withRegionName(kinesisConnectorConfiguration.REGION_NAME)
.withCommonClientConfig(clientConfig);

        if (!kinesisConnectorConfiguration.CALL_PROCESS_RECORDS_EVEN_FOR_EMPTY_LIST) {
            LOG.warn("The false value of callProcessRecordsEvenForEmptyList will be ignored. It must be set to true for the bufferTimeMillisecondsLimit to work correctly.");
        }

        if (kinesisConnectorConfiguration.IDLE_TIME_BETWEEN_READS > kinesisConnectorConfiguration.BUFFER_MILLISECONDS_LIMIT) {
            LOG.warn("idleTimeBetweenReads is greater than bufferTimeMillisecondsLimit. For best results, ensure that bufferTimeMillisecondsLimit is more than or equal to idleTimeBetweenReads ");
        }

        // If a metrics factory was specified, use it.
        if (metricFactory != null) {
            worker =
                    new Worker(getKinesisConnectorRecordProcessorFactory(),
                            kinesisClientLibConfiguration,
                            metricFactory);
        } else {
            worker = new Worker(getKinesisConnectorRecordProcessorFactory(), kinesisClientLibConfiguration);
        }
        LOG.info(getClass().getSimpleName() + " worker created");
    }

    @Override
    public void run() {
        if (worker != null) {
            // Start Amazon Kinesis Client Library worker to process records
            LOG.info("Starting worker in " + getClass().getSimpleName());
            try {
                worker.run();
            } catch (Throwable t) {
                LOG.error(t);
                throw t;
            } finally {
                LOG.error("Worker " + getClass().getSimpleName() + " is not running.");
            }
        } else {
            throw new RuntimeException("Initialize must be called before run.");
        }
    }

    /**
     * This method returns a {@link KinesisConnectorRecordProcessorFactory} that contains the
     * appropriate {@link IKinesisConnectorPipeline} for the Amazon Kinesis Enabled Application
     *
     * @return a {@link KinesisConnectorRecordProcessorFactory} that contains the appropriate
     *         {@link IKinesisConnectorPipeline} for the Amazon Kinesis Enabled Application
     */
    public abstract KinesisConnectorRecordProcessorFactory<T, U> getKinesisConnectorRecordProcessorFactory();
}

Still waiting for proxy configuration?

AWS SDKs understand proxy properties set via _JAVA_OPTION by default, how come the KCL library do not? Was there any reason?
This is critical to have for local testing and small scale debugging before pushing up code


Create issue on GH instead of Jira for flaky tests. Fixes #13238

Create gh issue for flaky tests:

if issue already exists, it is reopened and a comment appended
a Flaky Test label is added

To avoid API secondary rate limits passing a wait time (120 sec) is introduced in case of multiple flaky tests. In case of error one retry is performed
jira issue creation is also removed from Jenkins

@rigazilla Thanks! Are you going to create a PR for 15.0.x as well? I assume it's pretty different given we didn't backport a lot of the action enhancements.

Ideally yes, but yeah keeping things aligned between branches needs a general though


Scikit-learn, tensorflow, keras dependencies

Hi - thanks for the help getting things to compile! When running the node I ran into some dependency issues.
When running the launch file:
roslaunch gsm_node scenenn_dataset.launch bag_file:=/path/to/scenenn_231.bag

I got the error:
Traceback (most recent call last):
  File "/home/mfe/code/voxbox_ws/src/mask_rcnn_ros/scripts/mask_rcnn_node.py", line 13, in <module>
    from mask_rcnn_ros import coco
  File "/home/mfe/code/voxbox_ws/src/mask_rcnn_ros/src/mask_rcnn_ros/coco.py", line 49, in <module>
    import utils
  File "/home/mfe/code/voxbox_ws/src/mask_rcnn_ros/src/mask_rcnn_ros/utils.py", line 17, in <module>
    import skimage.color
ImportError: No module named skimage.color

which was resolved by doing pip install scikit-image.
I then got the error:
Traceback (most recent call last):
  File "/home/mfe/code/voxbox_ws/src/mask_rcnn_ros/scripts/mask_rcnn_node.py", line 204, in <module>
    main()
  File "/home/mfe/code/voxbox_ws/src/mask_rcnn_ros/scripts/mask_rcnn_node.py", line 199, in main
    node = MaskRCNNNode()
  File "/home/mfe/code/voxbox_ws/src/mask_rcnn_ros/scripts/mask_rcnn_node.py", line 61, in __init__
    config=config)
  File "/home/mfe/code/voxbox_ws/src/mask_rcnn_ros/src/mask_rcnn_ros/model.py", line 1768, in __init__
    self.keras_model = self.build(mode=mode, config=config)
  File "/home/mfe/code/voxbox_ws/src/mask_rcnn_ros/src/mask_rcnn_ros/model.py", line 1951, in build
    config.POOL_SIZE, config.NUM_CLASSES)
  File "/home/mfe/code/voxbox_ws/src/mask_rcnn_ros/src/mask_rcnn_ros/model.py", line 904, in fpn_classifier_graph
    name="roi_align_classifier")([rois] + feature_maps)
  File "/mnt/ubuntu_extra_ssd2/code/voxbox_ws/venv/local/lib/python2.7/site-packages/keras/backend/tensorflow_backend.py", line 75, in symbolic_fn_wrapper
    return func(*args, **kwargs)
  File "/mnt/ubuntu_extra_ssd2/code/voxbox_ws/venv/local/lib/python2.7/site-packages/keras/engine/base_layer.py", line 489, in __call__
    output = self.call(inputs, **kwargs)
  File "/home/mfe/code/voxbox_ws/src/mask_rcnn_ros/src/mask_rcnn_ros/model.py", line 365, in call
    roi_level = log2_graph(tf.sqrt(h * w) / (224.0 / tf.sqrt(image_area)))
  File "/home/mfe/code/voxbox_ws/src/mask_rcnn_ros/src/mask_rcnn_ros/model.py", line 320, in log2_graph
    return tf.log(x) / tf.log(2.0)
AttributeError: 'module' object has no attribute 'log'

which seemed to be resolved by doing pip install tensorflow-gpu==1.14.0 but I am unsure if this is the right version to downgrade to?
 But the windows remain blank.

@mfe7 All the necessary requirements for compiling the voxblox++ packages should be listed here.
I have never tested with tensorflow-gpu==1.14.0 (I am using tensorflow-gpu1.6.0 with Keras 2.1.6) but I do not see why it would not work.
If after your roslaunch voxblox++ is just hanging there with 4 blank windows, it means it has not received any segment yet to integrate. Something worth checking is whether the two segmentation nodes (mask_rcnn and depth_segmentation_node) are running and operational.

After your roslaunch, do you see something similar to:

process[mask_rcnn-2]: started with pid [9092]
process[depth_segmentation_node-3]: started with pid [9107]

Has the mask_rcnn node found the GPU to use? On my machine I see the following output:

Using TensorFlow backend.
2020-01-06 14:59:59.595586: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1212] Found device 0 with properties:
name: Quadro M2200 major: 5 minor: 2 memoryClockRate(GHz): 1.036
pciBusID: 0000:01:00.0
totalMemory: 3.95GiB freeMemory: 2.25GiB
2020-01-06 14:59:59.595602: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1312] Adding visible gpu devices: 0
 done.

2020-01-06 15:00:00.349413: I tensorflow/core/common_runtime/gpu/gpu_device.cc:993] Creating TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 1946 MB memory) -> physical GPU (device: 0, name: Quadro M2200, pci bus id: 0000:01:00.0, compute capability: 5.2)

Check that mask_rcnn is actually publishing segmentation results via $ rostopic hz /mask_rcnn/result

Give this a try and let me know if this helped you investigate the issue.

Closing due to inactivity, please reopen if the issue persists.


Few fixes to allow installation of OCP 3.9 on oVirt 4.2

Ensure python-ipaddress is installed on CentOS images.
Fix case when nsupdate is not needed.
Wait for VMs to be up and running (now that CentOS has the
ovirt-guest-agent installed.
It's installed using ovirt-guest-agent-common package.

What does this PR do?
Brief explanation of the code or documentation change you've made
How should this be manually tested?
Include commands to run your new feature, and also post-run commands to validate that it worked. (please use code blocks to format code samples)
Is there a relevant Issue open for this?
Provide a link to any open issues that describe the problem you are solving.
Who would you like to review this?
cc: @tomassedovic PTAL

Ping? Can this be merged?

Apologies, I thought I had done so after approval.
On Sun, Apr 22, 2018, 2:13 AM Yaniv Kaul<EMAIL_ADDRESS>wrote:

Ping? Can this be merged?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/openshift/openshift-ansible-contrib/pull/982#issuecomment-383360709,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVUWcvPemaojQHV9XfSdLP0wkRhNmI7Jks5trC14gaJpZM4TR8t5
.


Add RedundantSuspendModifier rule

Closes #2156

Codecov Report

Merging #2244 into master will decrease coverage by 0.09%.
The diff coverage is 67.39%.

@@             Coverage Diff             @@
##             master    #2244     +/-   ##
===========================================
- Coverage     81.38%   81.29%   -0.1%
- Complexity     2074     2099     +25
===========================================
  Files           343      344      +1
  Lines          5984     6030     +46
  Branches       1086     1106     +20
===========================================
+ Hits           4870     4902     +32
- Misses          532      533      +1
- Partials        582      595     +13

Impacted Files
Coverage Δ
Complexity Δ

...sch/detekt/rules/providers/PotentialBugProvider.kt
100% <ø> (ø)
3 <0> (ø)
:arrow_down:

...bosch/detekt/rules/providers/CoroutinesProvider.kt
100% <100%> (ø)
3 <0> (ø)
:arrow_down:

...etekt/rules/coroutines/RedundantSuspendModifier.kt
65.9% <65.9%> (ø)
22 <22> (?)

...turbosch/detekt/rules/style/UnusedPrivateMember.kt
93.47% <0%> (ø)
5% <0%> (ø)
:arrow_down:

...osch/detekt/rules/bugs/WrongEqualsTypeParameter.kt
100% <0%> (ø)
10% <0%> (+3%)
:arrow_up:

.../gitlab/arturbosch/detekt/rules/MethodSignature.kt
70% <0%> (+10%)
0% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Accumulation of org keywords do not work correctly for taxonomies when there's only one term

What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.124.1-db083b05f16c945fec04f745f0ca8640560cf1ec linux/amd64 BuildDate=2024-03-20T11:40:10Z VendorInfo=gohugoio

Does this issue reproduce with the latest release?
Yes.
Issue description
The new accumulation mechanism of org keywords from https://github.com/gohugoio/hugo/issues/11743 only works when many taxonomy terms are supplied.
If there aren't multiple #+tags: ... then the taxonomy terms are not marshalled into a slice and {{ range .Params.tags }} {{ end }} fails.
// <pre>{{ jsonify (dict "indent" "  ") .Params }}</pre>
{
  "draft": false,
  "iscjklanguage": false,
  "tags": "a",
  "title": "foobar"
}

$ hugo
Start building sites …
hugo v0.124.1-db083b05f16c945fec04f745f0ca8640560cf1ec linux/amd64 BuildDate=2024-03-20T11:40:10Z VendorInfo=gohugoio

Total in 8 ms
Error: error building site: render: failed to render pages: render of "page" failed: "/home/augustfengd/learn/hugo/layouts/_default/single.html:4:24": execute of template failed: template: _default/single.html:4:24: executing "_default/single.html" at <.Params.tags>: range can't iterate over a

I should've been more thorough with my testing. Can we revert https://github.com/gohugoio/hugo/pull/11741 and stick with the prior [] syntax?

@augustfengd I am unable to reproduce the problem as described. Example:
git clone --single-branch -b hugo-github-issue-12319 https://github.com/jmooring/hugo-testing hugo-github-issue-12319
cd hugo-github-issue-12319
hugo server

Nevermind. I see the problem now.

@augustfengd I don't think this is a problem. The same thing happens when assigning a scalar value using other front matter formats (e.g., TOML):
+++
title = 'Posts'
tags = 'red'
+++

Yes, with Org Mode front matter you could previously assign a scalar because we checked the keys for "tags", "categories", and "aliases", and decoded to a slice in those cases, but we've been warning users for years to move to the #+KEY[] syntax.
Unless I'm missing something, we're doing what we should be doing.

@jmooring, ah yeah you make a good point with the TOML example.
Yes, so we can see the org front matter as allowing us to convey an array when we duplicate the keyword.
Thanks for the perspective. :)


Use a more relaxed split() to split ipaddr from HTTP_X_FORWARDED_FOR

The IP address from HTTP_X_FORWARDED_FOR is currently split using ", ". Split with "," instead to handle IPs such as "x.x.x.x,x.x.x.x"

Coverage increased (+0.08%) to 40.003% when pulling b875e7ee87ed4064e344054eaa8bf47537b561fb on CompileInc:master into a2ed59aca8d3fed065869d5e192d3ad607990566 on aschn:master.

Coverage increased (+0.08%) to 40.003% when pulling b875e7ee87ed4064e344054eaa8bf47537b561fb on CompileInc:master into a2ed59aca8d3fed065869d5e192d3ad607990566 on aschn:master.

@gingerjoos I appreciate your attention to this detail and adding this contribution!


fix: apply margins before measuring dialogue div

Refs: #41
@weizhenye, this is my suggestion to fix #41, but maybe you know better where this logic should be applied.

This solution will break dialogues with \q2, I refactored it.


New messages are often covered behind keyboard or text field

Submit feature requests to http://www.mattermost.org/feature-requests/. File non-security related bugs here in the following format:
Summary
If I get a new message under iOS and open the app or switch to the channel, the message is often covered by the text field for answering.
Environment Information

Device Name: iPad Pro 11 (also happens on my IPhone XS)
OS Version: 12.3.1
Mattermost App Version: 1.20.1
Mattermost Server Version: 5.4.0

Steps to reproduce
Receive a new message while app is not open or while being in a different channel. Switch to app and channel to read message. Happens very often but not always.
Expected behavior
I would like to see the message please
Observed behavior
What did you see happen? Please include relevant error messages and/or screenshots.
Possible fixes
If you can, link to the line of code that might be responsible for the problem.

Hi @nils-tekampe, we opened a PR for this here: https://github.com/mattermost/mattermost-mobile/pull/2943.

Good to know. Thanks!

Closing as this was fixed for upcoming v1.21 release,

@amyblais still seeing this issue in 1.21 app release.

Hi @bbodenmiller, there is a new PR open here: https://github.com/mattermost/mattermost-mobile/pull/3074.


Helm Chart : cannot upgrade from chart on datawire repository to stable/hub version

I've been trying to upgrade a chart install from 0.50.1 to 0.50.3 and it seems the match labels were changed from the two version of the chart, that field is immutable so helm will refuse the upgrade.
I've been trying to figure out the changeset to the chart which caused this but i can't find the history seems like it was copied over to the official stable repository.
Error: UPGRADE FAILED: Deployment.apps "ambassador-dev" is invalid: spec.selector: Invalid value: v1.LabelSelector{MatchLabels:map[string]string{"app.kubernetes.io/name":"ambassador-dev", "app.kubernetes.io/instance":"ambassador-dev"}, MatchExpressions:[]v1.LabelSelectorRequirement(nil)}: field is immutable
Versions:
k8s: 1.10.11
helm: 2.11.0
chart: 0.50.1 (from datawire repo) to 1.1.15 (stable/ambassador)
ambassador: 0.50.1 to 0.50.3

In template/deployment.yaml:
Was :
selector:
  matchLabels:
     app: {{ include "ambassador.name" . }}
     release: {{ .Release.Name }}

is now :
selector:
  matchLabels:
    app.kubernetes.io/name: {{ include "ambassador.name" . }}
    app.kubernetes.io/instance: {{ .Release.Name }}

I'm not sure if this is fixable; we broke a bunch of things when we moved the chart, documented here:
https://blog.getambassador.io/ambassador-is-now-in-the-helm-stable-repository-a2c93ce2eaaf
cc @Flydiverny

Yes, regretfully the labelling change is breaking for existing deployments. Have to say I didn't really think about that part until after it was all done :) The new chart was pretty much rewritten and all existing functionality ported over.
But I've been thinking if it might be possible to add a setting in the chart values to use the old labels. To be nice for people who want to migrate without having to deal with the headache right now. But it would add a bit of a mess to the helm chart templates as well. As we can't change the labels now as that would be another breaking change for people already using the new chart.
Otherwise migration path would most likely be to add a secondary helm release based on the new chart and moving over everything to this new ambassador.

That seems to be kind of complicated; I'm going to delete this as WONTFIX.


Table coloring should be applied to TD, not to TR

Currently, the $table-bg, $table-accent-bg etc are applied to the table TR. This does not allow tables with transparent borders, done with background-clip: padding-box and $table-border-color: transparent. If the colors get applied to TH/TD instead, this would work nicely.

Could you show a live demo via Stackblitz or any other?

@ysds There you go : CodeSandbox
As you can see, I've set $table-border-color to transparent and no borders appear visually, although I've set the TD style background-clip: padding-box;. The bottom table that has no bootstrap styling, but the background color is applied to the TD's instead of the TR's (as bootstrap does) and there the transparent borders are visible, since there is no background color on the TR to mask them.

Due to the complexity of CSS priorities and the support for table hover and table stripes, it would be difficult for Bootstrap to allow such customization unless an intelligent solution is proposed. At least, does the following code solve what you want to do?
$table-bg: transparent;
$table-border-width: 2px;
$table-border-color: transparent;

.table {
  td {
    background-clip: padding-box;
    background-color: red;
  }
}

@ysds Yes, thank you, that is what I'm already using. Adding table-spacing is also and acceptable workaround. Just thought it would be an improvement to be able to achieve this out of the box. But if you say it will cause other problems, then forget it.


Fails clap-validator test

The demo fails the clap-validator test.
I added this snippet to stateLoad
if (totalRd == 0) return false;
After this loop. Not sure it is a correct fix but it makes the clap-validator succeed. The reason it failed is because you must return false if an empty state is passed, so that might mean you also need to clear existing settings or something. Not sure? Just testing...
Thanks for making this demo. Makes it real easy to understand and implement clap.
buffer[0] = 0;
while ((rd = stream->read(stream, bp, chunkSize)) > 0)
{
bp += rd;
totalRd += rd;
if (totalRd >= maxSize - chunkSize - 1)
{
_DBGCOUT << "Invalid stream: Why did you send me so many bytes!" << std::endl;
// What the heck? You sdent me more than 32kb of data for a 700 byte string?
// That means my next chunk read will blow out memory so....
return false;
}
}
if (totalRd == 0) return false;

Oh this was written well before the validator and i never bothered to run it! Funny.
That change looks correct to me. Would you like to send a PR?
Thanks so much.

PR == "pull request" - how you send a change when collaborating on GitHub
But no worries - if you haven't done it I can add the fix also! Just it will then have my authorship not yours (which is fine)


Background for current line

Hello!
It is possible?

It is possible but it is a feature that is already built into IDA 7.3 (dsync was written before IDA 7.3 was available). If you want to implement it nontheless, I'd suggest writing a separate plugin.
Something like that should work: idc.set_color(here(), idc.CIC_ITEM, 0xff0000)

@patois Thank you.


Debugging support in rustc

This is still work-in-progress but I wanted to open a PR for an early review to see if I am on the right path (also suggested by @spastorino).
This document is built on top of https://www.youtube.com/watch?v=elBxMRSNYr4 by @tromey.
For the sake of simplicity, I have added this document at the top level. Please advise if this should be under a directory or group of docs.

[x] Add explanation to note that this document is about debugging Rust code rather than debugging the compiler itself.
[x] Add first questions inline to the code on this PR for missing details and my confusions.
[x] CHECK What is SIP and link to it.
[x] Add the description of what is debugger etc. in preliminary section.
[x] Create Youtube's subtitles using this comment. (I personally find the auto-generated text to be horrible)
[x] Add TODOs for pending answers and iterate (as discussed with @mark-i-m and folks in WG-Learning meeting on 2019-06-11)

This issue has been assigned to @amanjeev via this comment.

I would add this as a subsection under the Debugging the Compiler chapter in Part 1.

I would add this as a subsection under the Debugging the Compiler chapter in Part 1.

Now that I look at this, I wonder if it is ok as a subsection, given this is pretty long.

I mean, still in it's own file as it currently is, just indexed under the debugging chapter.

Very amateur question - What's the best way to turn off linkchecker while working with a document? It takes forever to reload the page with the changes locally.

@amanjeev The easiest way is just to change this to false:
https://github.com/rust-lang/rustc-guide/blob/14b243de252da65e26dc3f60676d288f8c2af8b6/book.toml#L11

@amanjeev Thanks :) I will try to get to this soon. It looks like there have already been some comments, so I will try to make it a quick review

Also, you have a merge conflict

@mark-i-m Sorry, I will try and resolve the conflicts and push again. I used to think I was good at this. lol

Not sure what's going on here. I see this build error but the pages exist. Do we need to increase the timeout maybe? Please help.
appendix/code-index.md#11: There was an error while fetching "https://doc.rust-lang.org/nightly/nightly-rustc/syntax/ast/struct.Crate.html", https://doc.rust-lang.org/nightly/nightly-rustc/syntax/ast/struct.Crate.html: timed out```

Yes, I’ve been seeing a lot of those timeouts recently. I think it is travis rate limiting us. It seems to be a different link every time. @pietroalbini any ideas?

Seems I cannot re-run Travis builds without a commit? What are some of the other options to do that?

I can rerun travis builds, but I tried that on a previous PR and it doesn't solve the problem (there will be a timeout on another link). I would like to hear if the infra team has any ideas here.

Yes, I’ve been seeing a lot of those timeouts recently. I think it is travis rate limiting us. It seems to be a different link every time. @pietroalbini any ideas?

Those are spurious network issues, we also see them on rustc when talking with AWS (like S3 or a CloudFront-backed domain like doc.rust-lang.org). They're aware of it as far as I can tell, but there doesn't seem to be that much progress.

So meanwhile someone will be sitting at their desk clicking the "Re-run build" button. I do not envy you @mark-i-m :smile:

@mark-i-m  it passed! :sweat_smile:

@rustbot claim

Also watch out because github likes to "Helpfully" collapse comments...

@mark-i-m I am unaware of what r=me means but this is all yours now, I think. Please check.

@amanjeev  It looks good to me! Let's ship it :shipit:

I am unaware of what r=me means...

In the rust-lang/rust repo, where bors controls the PR queue, reviewers leave comments like @bors r+ or @bors r=mark-i-m to indicate that they approve the PR and it should be merged by bors. Thus it became a convention that r=me means "I will approve this" usually with some small request attached (e.g. https://github.com/rust-lang/rust/pull/61626#pullrequestreview-247500611).

@amanjeev I tried to rebase and resolve the conflicts, but somewhere along the way there were a few very weird merge conflicts, so I just squashed all commits and rebased on the current master. When travis turns green, I will merge.

@mark-i-m thank you so much! I am so happy this is merged! :tada:


Can't subscribe item after setting up subscription

I just set up spree_product_subscriptions on spree 3.7.2 along with installation procedure.
Here's my env.

Rails 5.2.2
Ruby 2.5.1
Spree 3.7.2
MySQL 5.7
And also installed gem spree_product_subscriptions on rspecs branch.
https://github.com/vinsol-spree-contrib/spree_product_subscriptions/tree/rspecs
Installation worked well, but some awkward behavior is shown.

Subscription didn't work.

I reviewed steps of github like this.

Add spree_product_subscriptions to your Gemfile:
gem 'spree_product_subscriptions', github: 'vinsol-spree-contrib/spree_product_subscriptions'

Bundle your dependencies and run the installation generator:
bundle
bundle exec rails g spree_product_subscriptions:install

You can also seed the default data with:

bundle exec rails g spree_product_subscriptions:seed

After installation, started my app. And then configured the item like this.

Here's product page.

What happened? Tell me how to fix.

Hi @hardreggaecafe
Can you provide some example repository where I can replicate your issue?

Hi, @pratibhamansinghka
Here's the one.
https://github.com/hardreggaecafe/sample_spree_subscription/tree/master
And data is like this.
spree-sample.sql.zip
I appreciate your sincerity.

@pratibhamansinghka How's it going? If you can't reproduce that issue, I'll help you. Name it.

@pratibhamansinghka Did you reproduce it?

Hi @hardreggaecafe
We have reproduce the issue.
Currently this extension is working fine with versions upto spree 3.5
In version 3.6 and 3.7 there are major changes in the API.
We will be working on upgrade to make it compatible with latest versions.
You can also raise a PR for the same and we will be happy to merge that.

@pratibhamansinghka
Thanks for the answer.
So what should I push for PR?

@pratibhamansinghka Please answer it.

@pratibhamansinghka Excuse me. Could you tell me what should I do for PR? Name it, please.

@pratibhamansinghka Could you tell me what's going on?

@hardreggaecafe You can upgrade this gem to be compatible with spree versions 3.6 and 3.7. We are working on the upgrades. Meanwhile, if you have already fixed the issues in upgrades, you can raise a pull request against appraise branch and we will review and merge it.

Thank you for the reply @vinay-mittal . Unfortunately, I can't figure out how to fix this problem.
If you tell me where to check on source code, I'll check on it.

Hey @vinay-mittal and @pratibhamansinghka, any updates here on the upgrade?


Update typings based on flow v0.53 changes

Codecov Report

Merging #479 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #479   +/-   ##
=======================================
  Coverage   88.94%   88.94%
=======================================
  Files          53       53
  Lines         389      389
=======================================
  Hits          346      346
  Misses         43       43

Continue to review full report at Codecov.

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


User Card

Contact Content Type (HCMS)
Relation to organization (HCMS)
small (name, responsibility, email, phone)
small with organization (name, position, organization logo (with link))

HCMS related topics are moved to own issue
#24

small (name, responsibility, email, phone) is on hold because of new design
moved to issue #26


Releases seem to be messed up

Something seems to be wrong with releases. If you check changelog https://github.com/zenoamaro/react-quill/blob/master/CHANGELOG.md it says that preserveWhitespace is added in v1.3.2 and if I install v1.3.3 I don't see it.
Also if you check types.d.ts in master on github you will see the prop but if you switch from master to v1.3.3 tag it's not there.
Could you pls make a new release and make sure this is included?

The feature was supported, but the type definition wasn't in the release.
This should be fixed with 1.3.4. Cheers


Firefox  show error Unsupported type of image?

On firefox browser, i fix src  It work well, but i brower file image -> error appear " Unsupported type of image"?

+1 @standup75 It is working only on Chrome. It stopped working on Firefox, Safari

I would have read the issues before use this directive :( now i am struggling with this.

Same situation here... :( Please someone could work on this?

Same situation here.

(finally) fixed in 0.3.3

@standup75 :
It is not working in Linux Mint Firefox. I got the following error which is different from the error discussed in this issue ticket. Should I open a new one ?


Add generic "Has-" typeclass

There could be a Has typeclass for when we don't want to write specialized HasLogger, HasRepository... typeclasses.
It could be something like this:
{-# LANGUAGE AllowAmbiguousTypes #-}
{-# LANGUAGE PolyKinds #-}
{-# LANGUAGE DefaultSignatures #-}
{-# LANGUAGE FlexibleContexts #-}
{-# LANGUAGE TypeApplications #-}

type Has :: k -> (Type -> Type) -> Type -> Constraint
class Has k d e | e -> d where
    type The k d e :: Type
    type The k d e = DepType k d
    the :: e -> The k d e
    default the :: (DepMetadata k, HasField (PreferredDepName k) e (The k d e)) => e -> The k d e
    the = getField @(PreferredDepName k)

type DepMetadata :: k -> Constraint
class DepMetadata k where
    type PreferredDepName k :: Symbol
    type DepType k (d :: Type -> Type) :: Type

Poly-kinded on the "marker", to allow using lifted datakinds to identify the component. When the component is a parametrizable record, perhaps we could use the record itself as the marker.
Markers could have DepMetadata instances that would produce the actual type of the dependency given the marker.

A less ambitious version has been implemented in branch has_helper (8bcfd6f).
type Has :: ((Type -> Type) -> Type) -> (Type -> Type) -> Type -> Constraint
class Has k d e | e -> d where
  dep :: e -> k d
  default dep :: (DepDefaults k, HasField (DefaultFieldName k) e (k d)) => e -> k d
  dep = getField @(DefaultFieldName k)

type DepDefaults :: k -> Constraint
class DepDefaults k where
  -- The Char kind would be useful here, to lowercase the first letter of the
  -- k type and use it as the default preferred field name.
  type DefaultFieldName k :: Symbol

The "marker" is not poly-kinded here. Instead, the marker is a monad-parameterizable record (or newtype) which holds the functions:
newtype Logger d = Logger { log :: String -> d () }

instance DepDefaults Logger where
    type DefaultFieldName Logger = "logger"

data Repository d = Repository {
    select :: String -> d [Int],
    insert :: [Int] -> d ()
}

instance DepDefaults Repository where
    type DefaultFieldName Repository = "repository"

It seems to work well:
mkController :: forall d e m . MonadDep [Has Logger, Has Repository] d e m => Controller m
mkController = Controller \url -> do
  e <- ask
  liftD $ log (dep e) "I'm going to insert in the db!"
  -- liftD $ (dep e).log "I'm going to insert in the db!" -- Once RecordDotSyntax arrives...
  liftD $ select (dep e) "select * from ..."
  liftD $ insert (dep e) [1,2,3,4]
  return "view"

-- also toss in this helper function
withEnv :: forall d e m r .  MonadDep [Has Logger, Has Repository] d e m => Controller m
mkController' = Controller \url -> withEnv \e -> do
  log (dep e) "I'm going to insert in the db!"
  select (dep e) "select * from ..."
  insert (dep e) [5,3,43]
  return "view"

One problem (if it's a problem) with this style is that it pushes us towards avoiding "bare" functions in the environment, and instead using nested records or at least wrapper newtypes. Which could make applying advices a bit more cumbersome.

This is still poly-kinded:
type DepDefaults :: k -> Constraint
class DepDefaults k where
  type DefaultFieldName k :: Symbol

Should it be? Perhaps if k were ((Type -> Type) -> Type), some more useful behaviors and data could be attached to this typeclass.

Added in 5b6654fa47ec08e74503ffbb600cd438e83ef6a3.


[FabriceBellard]: New Bridge

A feed for the particularly barebones (in terms of HTML, it's the opposite of the content) home page of Fabrice Bellard.
https://bellard.org/
This is the simplest HTML possible, I though this would be very easy, but it served to stress-test the HTML parser library more than I expected.

Relative links are not "absolutized" at all.

If a link's href property is apage.html, that's the exact string returned by ->href. At the very least, there should be a global function to do the equivalent of this, given a root domain:
https://github.com/somini/rss-bridge/blob/99a39ff2633dfa8e71f3cc2edbd17cd0575a14e0/bridges/FabriceBellardBridge.php#L19-L24
Worse, if a link is something like "http://gihub.com" (note the missing trailing slash, it is not parsed as a URI. This is hacked around here: https://github.com/somini/rss-bridge/blob/99a39ff2633dfa8e71f3cc2edbd17cd0575a14e0/bridges/FabriceBellardBridge.php#L26-L28

There is a strange issue with the ->find function, but only sometimes.

Consider the following HTML snippet:

This is paragraph 1

This is paragraph 2

This is paragraph 3

$html->find('p') returns 3 elements, as expected, the innertext argument works as expected (This is paragraph #). However, if you do another find on that returned object, the "implied" paragraph end is ignored and it will continue to find elements until the end of the page. This is an hidden bug on the implementation, since I'm only interested on the first link of each element.

Concerning the link page, we do have the defaultLinkTo that handles this.

This is even documented: https://github.com/RSS-Bridge/rss-bridge/wiki/defaultLinkTo
mea culpa :disappointed:

Merged, thanks !


Loop latency too high! shutdown

I have a problem. The output is turned off when "Loop latency too high! shutdown". I am using esp32 (original PCB) and added to the code in PinConfigESP32
Backflow_SD = 0,
INA22x_ALERT = 0,
because it was not compiled under esp32. All other code remained unchanged. Can you tell me why the "Loop latency too high! shutdown" problem occurs? The output is loaded only on a multimeter, a 24v power supply is connected to the input.
LOG:
[650164][E][main.cpp:300] loop(): [main] Loop latency too high! shutdown
Vi/o=25.83/ 9.65 Ii/o= 0.0/ 0.0A Pin= -0.0W 24°C 39sps 10kbps PWM(H|L|Lm)=   0|   0| 123 MPPT(st= MPPT,0) lag=1547.8ms N=1060
PWM disabled (duty cycle was 101)
[650180][I][mppt.h:303] startSweep(): [mppt] Start sweep
[650180][I][sampling.h:188] startCalibration(): [mppt] Start calibration
[651648][I][sampling.h:257] update(): [sampler] Sensor I_raw calibration: avg=-0.2091 std=0.000000
[651649][I][sampling.h:259] update(): [sampler] Sensor I_raw midpoint-calibrated: -0.209061
[651657][I][sampling.h:257] update(): [sampler] Sensor U_out_raw calibration: avg=8.9315 std=0.000106
[651665][I][sampling.h:257] update(): [sampler] Sensor U_in_raw calibration: avg=25.8547 std=0.000001
[651672][I][sampling.h:266] update(): [sampler] Calibration done!

I just pushed a fix which lowers the threshold.
you can also fix it yourself:
if (sps < 80 && !pwm.disabled() && nSamples > 1000 &&

replace 80 with 20

The 80 Hz works with the INA226. Sampling with the ADS1015 is a bit slower

Thank you. Now all work. I have INA226, but it has 36V max voltage(

Oh, I see I solder ADS1115 in future i replace it to ADS1015. Maybe better INA228 (up to 85V)? But I found about INA228 https://github.com/Zanduino/INA/issues/77#issuecomment-997338630. And i found new version from Angelo Casimiro with MOSFET HY4008  and HLSR 50-P https://www.facebook.com/photo/?fbid=4075260245853938&set=a.4075110759202220

Thank you. Now all work. I have INA226, but it has 36V max voltage(

Ok good. I use the INA226 on the low-side ground current and a voltage divider for voltage sensing. For solar voltage is use the ESP32 ADC, it is not used for tracking, only diode emulation and OV/UV detection.
you can find the schematics on cadlab

Oh, I see I solder ADS1115 in future i replace it to ADS1015. Maybe better INA228 (up to 85V)?

Yep ADS1015 is faster. Unless you need a Common Ground (Solar GND = Bat GND), the INA226 works fine. INA228 is expensive.

But I found about INA228 Zanduino/INA#77 (comment).

Interesting project, I didn't know. I used code for the INA228 based on craigpeacock/Linux_PowerMeter
for my smart-shunt power monitor

@f4soft how is it going? any progress on your MPPT project?


add css and improve UI

For dashboard, I have added css for animation like howering. I didnt change much becuase i want to allign with the Figma UI design as well. @Maimymai will help on the backend

@Maimymai is working on Dashboard UI as we discussed last night. I cannot approve for now coz I am waiting for her results today. @mmafrar


Sri Hari: Batch-2/Neetcode-150/articles

@neetcode-gh

Took 5 hours for this work.
Please review this.

@Ykhan799
Thanks for the support!

Looks good thanks


Consul - Add support for metadata in a backwards compatible way

Description
This PR backports #1167 to 3.x
I'd like to open a discussion on some details regarding implementation before completing it and adding tests.
I tried to get some inspiration from Spring Cloud Consul implementation:

Added TagsAsMetadata (defaults to true)
This setting controls whether we fallback to Metadata creation from Tags (default) or we use the real Metadata
Both approaches add default metadata

Let me know if this is what you had in mind. I will add unit tests then.
Documentation changes will follow.
Quality checklist

[x] Your code complies with our Coding Style.
[x] You've updated unit and/or integration tests for your change, where applicable.
[ ] You've updated documentation for your change, where applicable.
If your change affects other repositories, such as Documentation, Samples and/or MainSite, add linked PRs here.
[x] There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.
[x] You've added required license files and/or file headers (explaining where the code came from with proper attribution), where code is copied from StackOverflow, a blog, or OSS.

Tests add, ready for review 👨🏽🔬.
cc @hananiel


Add console.log with hint to use --prod for non-prod builds

I'm submitting a ...
[ ] bug report
[x] feature request
Current behavior:
Many new users don't know they should be using --prod to get an app with a faster startup time.
Expected behavior:
A console.log output for non-prod builds that says something like "Add --prod to your build command for faster app boot time" on startup could help to point them into the right direction.

Hey @janpio , I think this is a great idea. As this would be something that would have to be implemented in app-scripts and not the framework, im gonna move it over to app-scripts.

This issue was moved to ionic-team/ionic-app-scripts#1136


cilium: encryption needs to use string reference

Fix the EncryptInterface name to report the string correctly.
Signed-off-by: John Fastabend<EMAIL_ADDRESS>

This change is

test-me-please

Coverage increased (+0.02%) to 42.552% when pulling 7175fd5b08812e65e955918540ec87e102126cbc on fix-direct-route-ifname into 19672fccba351999297d135bbb0be15225527135 on master.

Build failure: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/11239/
Looks like etcd-operator got into a bad state :(

test-me-please


Docs website returns HSTS error

Describe the bug
https://x.com/alkreddy/status/1714847929121312783?s=20

Steps to reproduce

Visit https://docs.radapp.io/getting-started

Expected behavior
Expect to view website
Screenshots

Desktop (please complete the following information):

OS:
Browser
Version

Smartphone (please complete the following information):

Device:
OS:
Browser
Version

Additional context

AB#9867

Unable to repro both on my machine (Windows 11 desktop) and on a fresh Windows VM in Azure. I tried setting strict mode, forcing various HTTPS-only modes and proxy modes and unable to get it to show the error.
If someone hits this again could they drop in their system specs and some information on their proxy setup? Reading through this post it seems like the proxy might be a possible issue: https://support.mozilla.org/en-US/questions/1324365

I confirm that it works fine now in my env. too.
Not sure what caused the intermittent issue.
You may close this issue.


feat: adding renovate.json for renovatebot

Merge this PR and then install: https://github.com/apps/renovate and once a day this will run and generate PR's. But if you want to disregard updates entirely just remove line 15 and the default behavior should not re-open.
I keep it as always so I repeatedly get reminded to eventually upgrade to the newer/latest version.
To uninstall just remove it using one of these links:
If you are using an org: https://github.com/organizations/onetimesecret/settings/installations
If you are using a personal account: https://github.com/settings/installations (obviously you would want to be logged into the onetimesecret user)
Here is an example of what it's like in my account: https://github.com/venkatamutyala/onetimesecret/issues/1
That issue will be created by renovatebot and it'll persist indefinitely. As you can see there are a lot of updates. It also lets you see all the updatable versions for many packages so rather then going to the very latest you can check the box for the PR/version you want to have created.

@delano Build is failing. Looks like something changed in the gems upstream? This PR doesn't need a working build so it might make sense to merge a fix separately from my PR? However, happy to take a look if yo
Fetching gem metadata from https://rubygems.org/.........
Your bundle is locked to uri-redis (1.1.1) from rubygems repository
https://rubygems.org/ or installed locally, but that version can no longer be
found in that source. That means the author of uri-redis (1.1.1) has removed it.
You'll need to update your bundle to a version other than uri-redis (1.1.1) that
hasn't been removed in order to install.

@venkatamutyala Yeah the upstream checks are passing again now. I'll merge develop in so they'll re-run here and then merge.
Looking forward to kicking and then rolling the tires.


change parameters like temperature?

Is it possible to change the temperature? How could we do that?

It is as simple as adding additional keyword arguments.
Here is an example for completion:
create_completion(
    ENV["OPENAI_API_KEY"],
    "text-ada-001";
    prompt="Oranges are...",
    temperature=0.5
  )

The only exception is for stream, where instead of using stream=true, you should provide a streamcallback function to manage the received chunks (and that, in turn, will set stream=true).
Until we get proper documentation, I also suggest looking at the tests. However, the tests are not showcasing the usage of the additional parameters - so feel free to continue addressing questions here if the basic test examples are not covering something.

I'm working on getting Documenter.jl set up now

Done via the docs now: https://juliaml.github.io/OpenAI.jl/dev/


Update vendor.json for picking up dcos-go changes

This PR is for picking up the default path updated in dcos-go for Windows powershell detect_ip.ps1 script change

Can one of the admins verify this patch?


extend-ui-filter-with-message-header

The scenario-detail view doesn load the filter buttons on init. It works if you press F5 after the page has loaded. Needs to be fixed @bbortt.

I merged this into the intermediate branch straight away. will give a final review there :) thanks very much.


add before_init param to View.as_view

I am writing a flask ext and need to do something before View init,
but I can't do that via app.before_request because I need to test my View without request.

Please provide a concrete example of what you're trying to do with this as well as why you can't test a view with a request.  I'm not clear how this is different than just overriding __init__ to call do_something first, or otherwise subclassing View.  Additionally, this patch needs tests and must pass existing tests as well.

I have  a Resource class subclassing View, and a Api class, just like flask-restful.
Api need to parse request before Resource init, such as check user's permission.
And the test is like this:
        with api.test_client() as c:
            rv,code,header = c.resource.action(data)
            assert code == 200
            assert rv == {"hello":"world"}
            assert c.resource.action_need_login(data).code == 403

        with api.test_client(user_id) as c:
            assert c.resource.action_need_login(data).code == 200
            assert c.resource.action_need_login(data).rv == {"hello":"guyskk"}

It will pass user_id and data directly.

A example:
app = Flask(__name__)
api = Api(app)

class Hello(Resource):

    def get(self):

        return {"hello": "name"}

api.add_resource(Hello)

In api, _restaction_test means on testing.
def _before_resource_init(self):

    if not getattr(g, "_restaction_test", False):
        resource, action = parse_request()
        me = self.parse_auth_header()
        request_data = get_request_data()
        g.resource = resource
        g.action = action
        g.me = me
        g.request_data = request_data

    user_role = self._fn_user_role(g.me["id"])
    permit, res_role = self._permit(user_role, g.resource, g.action)
    g.me["role"] = res_role
    if not permit:
        if g.me["id"] is None:
            abort(403, "permission deny: your token is invalid")
        else:
            abort(403, "You don't have permission: user_role=%s" % user_role)

What is c.resource?  It's not a standard part of Flask.  Your example doesn't demonstrate the need for your patch.  Class creation should not have side effects that you might need to bypass.  Use decorators to check for permissions, which is already supported by View.  This seems like an incorrect design choice rather than a missing feature on Flask's part.

You are right
thank you


Simplify Caliper class

Caliper class does too much, potentially slowing drawing of calipers.  Properties like units and multiplier can be pushed to each caliper when calibration, zoom, or heart rate/interval display changes.
Note this applies to ALL versions of EP Calipers.

EP Calipers 3 for Windows redesigned the calipers to be more efficient. Otherwise no need to DO THIS IN OTHER VERSIONS.


In Network DAO proposal page, vote ratio is different from explorer's

Description
In Network DAO proposal page, vote ratio is different from explorer's
Steps To Reproduce

Enter a Network DAO proposal page https://tmrwdao.com/network-dao/proposal/887a95959e9f8e81340521574fa77229d1ae569efd6e726e5457ce8a0d23cf3f?chainId=AELF
Enter the same proposal page in explorer  https://explorer.aelf.io/proposal/proposalsDetail/887a95959e9f8e81340521574fa77229d1ae569efd6e726e5457ce8a0d23cf3f

Current Behavior

In Network DAO proposal page, vote ratio is different from explorer's

Expected Behavior

The ratio should be same

Environment

Platform: PC
Node: v18.18.0
Browser: Chrome 126.0.6478.56

fixed confirm


Add some UI types

This is a continuation of this PR: https://github.com/khasanovbi/phaser3-rex-plugins-types/pull/2

Could you rebase from master please, revert changes in package.json, package-lock.json, and call npm run prettier?
JFYI. I was able to reproduce the problem with two versions of phaser in my project only at development mode with npm link.
Before that, I ran the project through webpack and check types through fork-ts-checker-webpack-plugin and babel-loader and doesn't see any errors.
However, when I run directly through tsc, I saw errors. I don't know a good solution right now, I need time to research it. For now I will update the phaser to the latest version in package-lock.json to prevent type confilicts.

Hi @atrich, do you need help with PR? I could add the fixes in a separate branch.

My apologies, @khasanovbi, due to time constraints, I've had to jettison my plans to utilize the rex plugins for my phaser project due to other incompatibilities. I am willing to respond to your issues with the PR but it may take me a while to get around to it. I'm fine if you want to add your proposed fixes in another branch.

Thanks for your contribution


remove Jay Carey and add Chris Norman as new maintainer

Jay Carey has expressed interest in retiring as project maintainer. I would like to nominate @cmnbroad as a replacement maintainer.

@magicDGS & @jacarey any objections?

@cmnbroad Did we spell your various names right?

:+1:


A notification dialog pops up when executing 'Format Document' from Command

OS: Mac / Linux
Extension Version: 0.6.1-alphalangserv
Steps to reproduce:

Open a file with which language is 'ARM Deploymnet Template'.
F1 -> Type 'Format Document' and execute it.
Check the result.

Expected:
No error occurs and the command is executed successfully.
Actual:
A notification dialog pops up.

More Info:

There no 'Format Document' displays on the context menu.
Show 'No extensions found' after click 'Install Formatter....'.

This issue doesn't reproduce for a file which language is JSON/ JSON with Comments.

Same issue as the other Mac/Linux bugs (language server didn't start)

Fixed by #263


Refactor event listening on document

Coverage decreased (-0.2%) to 92.029% when pulling 38df61cf6f50975d63eb9713ee62601037bbfcaf on tb:onDocumentClick into ca74ca5eabaff766962f98222f4f2cd0f89ed892 on mseemann:master.

<EMAIL_ADDRESS>published.


Enhancing the LoginActivity and SignupActivity UI

Actual Behaviour
Lot of unused space is there and the UI can be improved by the implementation of card view.
Expected Behaviour
CardView can be implemented to enhance the UI.
Steps to reproduce it
Navigate to the LoginActivity and SignUp Activity through the Navigation Drawer.
Would you like to work on the issue?
YES

@iamareebjamal is it ok?


Optimizing Judge and Generator with r/CMV dataset

First, thanks for open sourcing this!
I'm interested in experimenting with DSPy optimizations for both the judge and the generator. I'm thinking about the r/CMV dataset you linked.
Since we have examples of arguments that did not convince the OP and examples that did convince the OP, I think we can use these as part of the voting evaluation method, where we construct dspy.Examples and have the delta-awarded replies be the success labels (ie, voted for) and the other replies not voted for. So presumably this would improve the prompts used as part of the Judge voting to pick the convincing arguments. What other things do I need to take into account before trying this (for the SingleTurnVote approach)?
This would not capture nuance in which it takes multiple turns of a conversation to refine an argument until the point that OP awards a delta. I think maybe we need to combine such threads into a single argument (via llm) to then vote on? Or could we instead capture it as part of a MultiTurn argument? But I am not sure if it makes sense for the judges to vote Yes on the intermediate steps of the multiturn? Maybe it does. But how do emphasize the right path down the reddit thread tree? Ie, I only want to vote Yes on an intermediate argument when we also vote Yes on the final delta-awarding comment. I'm basically trying to ensure we capture the entirety of the argument in the voting process. Would love any guidance on how to do this.

I'm also wondering if we could improve the Generator by doing something like: for each non-delta-awarded-reply, prompt an llm to evaluate how to make it more like the delta-awarded-replies for the given argument under discussion. I'm not sure if this would be too naive an approach. What do you think?

Other questions I haven't finished thinking through:

if I optimize both the Generator and Judge, should I do them independently, or can I optimize them together via MIPRO?
should I be using different parts of the dataset for the Generator and Judge training, to prevent some type of contamination?

Hey Marc,
My pleasure -- persuasive argumentation is a passion of mine, and I hope to get more people interested as well :D
As for the idea for prompt optimization for voting, here are a few key things to keep in mind:

In CMV, you'll sometimes find that a comment that got a delta from OP only did so later on in a discussion. This translates to using a ToTJudgeVoteMultiTurnSignature, since we'd be working with multi-turn conversations. However, in examples where a delta is given to a direct comment of the original post, a ToTJudgeVoteSingleTurnSignature makes sense as you suggested. Your intuition that a "delta" is equivalent to a vote is a good one! By using Multi-Turn without drafts, you do not need to account for intermediate steps.
You are correct in your statement that we wouldn't capture information equivalent to "drafts" from CMV. In fact, there is no large dataset which captures this sort of information (i.e., iterating on versions of persuasive arguments to improve their quality over time). For now, I think it is best to limit the scope to optimizing ToTJudgeVoteSingleTurnSignature and ToTJudgeVoteMultiTurnSignature. I agree with you that using synthetic data (perhaps with human supervision) is a good way to train judges that take into account previous drafts, but that is out of scope for the issue I suggested, as it merits some independent and careful planning.
I suggest you construct groups of 'k' arguments (where k is the number of possible arguments for the voting judge to consider -- in practice, I would start with k=2 or k=3) where in each group exactly one of the arguments obtained a delta from OP. You could then create a metric for accuracy, and optimize the judge to correctly predict which of the k arguments obtained the delta across the n examples you create.
Make sure to group examples by length/structure. For example, arguments that are only 1 sentence long should be in a different group than examples that are a few sentences long or a paragraph long. The categories of length that I've been thinking of are: {one sentence, two sentences, a few sentences, one paragraph, three-paragraphs, five-paragraphs, long-form essay, j bullets}, where j is some integer. We want to enforce this grouping since we designed judges to compare arguments of similar length/structure.

Your idea for improving the generator is good, but may require some degree of human supervision, and would certainly require more careful planing. This is likely out of scope of the original idea I suggested, but I am happy to discuss this further as part of a separate issue.
I recommend you optimize the judge and generator separately. Best to start with just the judge, as it is comparably simpler to optimize given existing datasets (beyond just CMV). You can use the entire CMV dataset to optimize the judge, once formatted as above. I imagine that we can optimize both components once we have sufficient human-vetted traces of ToT reasoning, but perhaps there are more convenient approaches to do this which I've yet to think of
Thank you for exploring our work!!! It's exciting to get more people involved :D


Type declarations of Box

Hello, thanks a lot for the package. It is really fantastic.
I am currently trying to implement a basic Monte Carlo simulation engine here (https://github.com/edwinb-ai/Metropolis.jl), and I am using CellListMap.jl for computing the interactions between particles.
However, I am having some trouble when defining a type that contains a field that is of type Box. The PR in which I am working is here.
Basically, the problem is that I want to define the following type
mutable struct System{B,T,VT,I}
    xpos::VT    # position of particles
    density::T   # density of the system
    temperature::T    # temperature
    box::B    # simulation box, actually want it to be of type `Box`
    rng::Random.AbstractRNG
    npart::I    # total number of particles
end

I then have a constructor that automatically creates the simulation box, which is this one
function System(
    density::T, temp::T, particles::I, cutoff::T; dims=3, random_init=true, lcell=2
) where {T<:Real,I<:Int}
    box_size = cbrt(T(particles) / density)
    # box = create_box(box_size, dims, cutoff; lcell=lcell)
    box = CellListMap.Box(fill(box_size, dims), cutoff; lcell=lcell)
    rng = Xorshifts.Xoroshiro128Plus()
    xpos = initialize_positions(box_size, rng, particles; random_init=random_init)
    syst = System(xpos, density, temp, box, rng, particles)

    return syst
end

I am using Cthulhu.jl to check for type instabilities, and the one that I get is that the type of box is not concrete, and so there are many allocations happening.
I was wondering if you could point me in the right direction of how I should be declaring the types of Box inside other types, such as the one I described above.

Hello,
Indeed, the "Box"  type of CellListMap is annoyingly complex:
Base.@kwdef struct Box{UnitCellType,N,T,TSQ,M}
    unit_cell::UnitCell{UnitCellType,N,T,M}
    lcell::Int
    nc::SVector{N,Int}
    cutoff::T
    cutoff_sq::TSQ
    ranges::SVector{N,UnitRange{Int}}
    cell_size::SVector{N,T}
    unit_cell_max::SVector{N,T}
end

You either need to parameterize all that in your System type, or just use System{B,...} as you are already using. This is what I would recommend really.
The creation of a Box is, however, intrinsically type-unstable, because the UnitCellType is determined at runtime, thus your constructor will be type-unstable as well. The resulting Box is, nevertheless, concrete, so everything from there on should be fast.
The construction of the Box type is certainly irrelevant, in terms of time, in a simulation, thus that type-instability is "benign".
In summary:
julia> using CellListMap

julia> struct Test{B}
           b::B
       end

julia> test = Test(Box([1,1,1],0.1))
Test{Box{OrthorhombicCell, 3, Float64, 9}}(Box{OrthorhombicCell, 3, Float64, 9}(CellListMap.UnitCell{OrthorhombicCell, 3, Float64, 9}([1.0 0.0 0.0; 0.0 1.0 0.0; 0.0 0.0 1.0]), 1, [12, 12, 12], 0.1, 0.010000000000000002, UnitRange{Int64}[-1:1, -1:1, -1:1], [0.1, 0.1, 0.1], [1.0, 1.0, 1.0]))

julia> typeof(test)
Test{Box{OrthorhombicCell, 3, Float64, 9}}

julia> isconcretetype(typeof(test))
true

The Box  constructor is type-unstable, and you cannot get rid of that (if you update the box at every iteration, there will be some allocations associated to that, but very likely in terms of performance that will be irrelevant. It it is not, please let us discuss this further).
julia> @code_warntype Box([1,1,1],0.1)
Variables
  #self#::Type{Box}
  sides::Vector{Int64}
  cutoff::Float64

Body::Box{OrthorhombicCell, _A, Float64, _B} where {_A, _B}
1 ─ %1 = CellListMap.:(var"#Box#12")(CellListMap.Float64, 1, CellListMap.OrthorhombicCell, #self#, sides, cutoff)::Box{OrthorhombicCell, _A, Float64, _B} where {_A, _B}
└──      return %1

julia> @btime Box($([1,1,1]),0.1);
  12.256 μs (101 allocations: 5.62 KiB)

An additional comment: I noticed that you are trying to pass all parameters to the Box type here:
https://github.com/edwinb-ai/Metropolis.jl/blob/072ba4fcca3e65716b9abfe71a4f28f77bed0020/src/types.jl#L23
That looks fine, though (as you have seen above), the set of type parameters is not something that I guarantee to be stable in the interface. In particular, it will not be in the next release (0.7) because I needed to change them to allow automatic differentiation and unit propagation to happen through the code.
Thus, before you get a broken code yourself there, I really recommend sticking with the B parameter for the complete Box type.

Just reinforcing: you shouldn't really care that your System  constructor is type-stable. The important there is that the resulting System  object is concrete, such that the simulation itself using the values contained there is type-stable.

Oh okay, I see then. Thank you very much for clarifying this. I actually took a lot of inspiration from your FortranCon talk on molecular dynamics, so all of this is greatly appreciated. I still have some other type instabilities, but I'll try to figure those out as I go. Thanks again!

The important there is that the resulting System object is concrete, such that the simulation itself using the values contained there is type-stable.

Thanks, I will make sure to make it concrete.


scim: Downgrade SCIMClient from a model to an ephemeral dataclass

SCIMClient is a type-unsafe workaround for django-scim2’s conflation of SCIM users with Django users. Given that a SCIMClient is not a UserProfile, it might as well not be a model at all, since it’s only used to satisfy django-scim2’s request.user.is_authenticated queries.
This doesn’t solve the type safety issue with assigning a SCIMClient to request.user, nor the performance issue with running the SCIM middleware on non-SCIM requests. But it reduces the risk of potential consequences worse than crashing, since there’s no longer a request.user.id for Django to confuse with the ID of an actual UserProfile.
Cc @mateuszmandera @PIG208.

It seems to me that we only need SCIMClient for the is_authenticated attribute, and format_requestor_for_logs. I think we can put format_requestor_for_logs on RequestNotes, and remove name and realm on the SCIMClient, which aren't used elsewhere unless I missed anything. When django-scim2 gets fixed, we can then remove this class entirely.
This fix implicitly works for LogRequests.process_response because format_request_for_logs happen to be defined on request that has request.user.is_authenticated being True. This is a case that we can rely on RequestNotes.format_requestor_for_logs being set.

Merged, since this is a nice checkpoint; I assume that cleanup can be in a follow-up PR.