NPE with httpclient 4.4

Caused by: java.lang.NullPointerException
    at java.io.FilterInputStream.read(FilterInputStream.java:133)
    at com.fasterxml.jackson.core.json.ByteSourceJsonBootstrapper.ensureLoaded(ByteSourceJsonBootstrapper.java:489)
    at com.fasterxml.jackson.core.json.ByteSourceJsonBootstrapper.detectEncoding(ByteSourceJsonBootstrapper.java:126)
    at com.fasterxml.jackson.core.json.ByteSourceJsonBootstrapper.constructParser(ByteSourceJsonBootstrapper.java:215)
    at com.fasterxml.jackson.core.JsonFactory._createParser(JsonFactory.java:1191)
    at com.fasterxml.jackson.core.JsonFactory.createParser(JsonFactory.java:753)
    at com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:2207)
    at org.ektorp.impl.StdCouchDbConnector$5.success(StdCouchDbConnector.java:250)
    at org.ektorp.http.RestTemplate.handleResponse(RestTemplate.java:122)
    at org.ektorp.http.RestTemplate.get(RestTemplate.java:22)
    at org.ektorp.impl.StdCouchDbConnector.get(StdCouchDbConnector.java:246)
    at org.ektorp.impl.StdCouchDbConnector.get(StdCouchDbConnector.java:237)
    at org.ektorp.support.StdDesignDocumentFactory.getFromDatabase(StdDesignDocumentFactory.java:57)
    at org.ektorp.support.CouchDbRepositorySupport.initDesignDocInternal(CouchDbRepositorySupport.java:280)
    at org.ektorp.support.CouchDbRepositorySupport.initStandardDesignDocument(CouchDbRepositorySupport.java:274)

This exception is thrown after upgrading httpclient from 4.3.6 to 4.4 (using Java8u31 and ektorp 1.4.2).
I reverted for now but wanted to inform you about this problem.

I upgraded my apps from 4.3.6 to 4.4 and did not found any issue.
Maybe you should upgrade ektorp to latest version (1.4.3) ?

But 1.4.2 is the latest released version...

I suppose @helun did not release 1.4.3 after all?
https://github.com/helun/Ektorp/commit/720ca25c0d89592887946d368447714d70c533b3
I'm using unreleased version myself, along with non-default configuration.
Maybe you should build a snapshot with the latest revision from master in order to test if you still get the error.

Just built the current HEAD and run our tests with it. Similar error, just on a slightly different line:
java.lang.NullPointerException: null
	at java.io.FilterInputStream.read(FilterInputStream.java:133)
	at com.fasterxml.jackson.core.json.ByteSourceJsonBootstrapper.ensureLoaded(ByteSourceJsonBootstrapper.java:489)
	at com.fasterxml.jackson.core.json.ByteSourceJsonBootstrapper.detectEncoding(ByteSourceJsonBootstrapper.java:126)
	at com.fasterxml.jackson.core.json.ByteSourceJsonBootstrapper.constructParser(ByteSourceJsonBootstrapper.java:215)
	at com.fasterxml.jackson.core.JsonFactory._createParser(JsonFactory.java:1191)
	at com.fasterxml.jackson.core.JsonFactory.createParser(JsonFactory.java:753)
	at com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:2207)
	at org.ektorp.impl.StdCouchDbConnector$5.success(StdCouchDbConnector.java:244)
	at org.ektorp.http.RestTemplate.handleResponse(RestTemplate.java:126)
	at org.ektorp.http.RestTemplate.get(RestTemplate.java:22)
	at org.ektorp.impl.StdCouchDbConnector.get(StdCouchDbConnector.java:240)
	at org.ektorp.impl.StdCouchDbConnector.get(StdCouchDbConnector.java:231)
	at org.ektorp.support.CouchDbRepositorySupport.get(CouchDbRepositorySupport.java:133)

What is the server HTTP Status and Response content? Are you sure they are valid?
It seems the HttpEntity is returning a null InputStream on getContent().
Would you like to build https://github.com/YannRobert/Ektorp/commit/e26580fed7f0327946131c5c6787d97339190359 and quick test again? I did a lot of changes back at a time for a better support of httpclient 4 that I didn't reported upstream.
If it still does not work it would help that you provide a test.

I hacked some output into StdHttpResponse.getContent() and ran my tests with httpclient 4.3.6 and httpcore 4.3.3 (working) as well as httpclient 4.4 and httpcore 4.4 (broken).
The first difference in the output (ignoring object identifiers of toString()) looks like this:
working
Entity: org.apache.http.impl.client.cache.CacheEntity@41d426b5
Entity.getContentEncoding(): null
Entity.getContentLength(): 105
Entity.getContentType(): Content-Type: text/plain; charset=utf-8
Entity.isChunked(): false
Entity.isRepeatable(): true
Entity.isStreaming(): false
Entity.getContent(): java.io.ByteArrayInputStream@8dbfffb

broken
Entity: org.ektorp.http.StdHttpResponse$NullEntity@4612b856
Entity.getContentEncoding(): Content-Encoding: UTF-8
Entity.getContentLength(): 0
Entity.getContentType(): Content-Type: null
Entity.isChunked(): false
Entity.isRepeatable(): true
Entity.isStreaming(): false
Entity.getContent(): null

I'll give your code a try in a few minutes.

I can't build your integration branch.
[INFO] Fork Value is true
     [java] The following errors occurred during analysis:
     [java] Feb 05, 2015 7:49:04 PM edu.umd.cs.findbugs.TextUIBugReporter reportAnalysisError
     [java] SEVERE: Unable to get XClass for java/util/function/Consumer
     [java] java.lang.ArrayIndexOutOfBoundsException: 27695
     [java] 	at org.objectweb.asm.ClassReader.readClass(Unknown Source)
     [java] 	at org.objectweb.asm.ClassReader.accept(Unknown Source)
     [java] 	at edu.umd.cs.findbugs.asm.FBClassReader.accept(FBClassReader.java:44)
     [java] 	at org.objectweb.asm.ClassReader.accept(Unknown Source)
     [java] 	at edu.umd.cs.findbugs.classfile.engine.ClassParserUsingASM.parse(ClassParserUsingASM.java:110)
     [java] 	at edu.umd.cs.findbugs.classfile.engine.ClassParserUsingASM.parse(ClassParserUsingASM.java:565)
     [java] 	at edu.umd.cs.findbugs.classfile.engine.ClassInfoAnalysisEngine.analyze(ClassInfoAnalysisEngine.java:76)
     [java] 	at edu.umd.cs.findbugs.classfile.engine.ClassInfoAnalysisEngine.analyze(ClassInfoAnalysisEngine.java:38)
     [java] 	at edu.umd.cs.findbugs.classfile.impl.AnalysisCache.getClassAnalysis(AnalysisCache.java:266)
     [java] 	at edu.umd.cs.findbugs.ba.XFactory.getXClass(XFactory.java:652)
     [java] 	at edu.umd.cs.findbugs.ba.AnalysisCacheToAnalysisContextAdapter.setAppClassList(AnalysisCacheToAnalysisContextAdapter.java:391)
     [java] 	at edu.umd.cs.findbugs.FindBugs2.setAppClassList(FindBugs2.java:981)
     [java] 	at edu.umd.cs.findbugs.FindBugs2.execute(FindBugs2.java:224)
     [java] 	at edu.umd.cs.findbugs.FindBugs.runMain(FindBugs.java:391)
     [java] 	at edu.umd.cs.findbugs.FindBugs2.main(FindBugs2.java:1298)

Probably caused by Java 8?

Managed to build your integration branch with Java 7. Doesn't work either but gives a different exception:
org.ektorp.DbAccessException: com.fasterxml.jackson.databind.JsonMappingException: No content to map due to end-of-input
 at [Source: java.io.InputStreamReader@6c45ee6e; line: 1, column: 1]
	at org.ektorp.util.Exceptions.propagate(Exceptions.java:19)
	at org.ektorp.http.RestTemplate.handleResponse(RestTemplate.java:132)
	at org.ektorp.http.RestTemplate.get(RestTemplate.java:22)
	at org.ektorp.impl.StdCouchDbConnector.get(StdCouchDbConnector.java:247)
	at org.ektorp.impl.StdCouchDbConnector.get(StdCouchDbConnector.java:237)
	at org.ektorp.support.CouchDbRepositorySupport.get(CouchDbRepositorySupport.java:133)

I forgot to mention that I'm running the tests against CouchDB 1.6.0 - if that makes a difference...

This is the way I create the StdCouchDbInstance in my tests:
StdHttpClient.Builder builder = new StdHttpClient.Builder()
		.host("localhost")
		.port(5984);

Properties props =  new Properties();
File propFile = new File(System.getProperty("user.home"), "couchDbCredentials.properties");
InputStream is = null;
try {
	is = new FileInputStream(propFile);
	props.load(is);
}
catch(IOException ex) {
	props = null;
}
finally
{
	if(is != null)
	{
		try {
			is.close();
		} catch (IOException e) {
			// ignore
		}
	}
}

if(props!=null) {
	builder.username(props.getProperty("couch.user")).password(props.getProperty("couch.password"));
}

dbInstance = new StdCouchDbInstance(builder.build());

dbInstance is then used to create a StdCouchDbConnector
StdCouchDbConnector db = new StdCouchDbConnector(dbName, couchDbInstance, new SomeObjectMapperFactory());

which is in turn used as a c'tor argument for a type-specific CouchDbRepositorySupport class. This class has several custom views
@Views({
		@View(name = "all", file="/couch/foo/allView.json"),
		@View(name = "null_tags", file="/couch/foo/fooWithoutTags.json"),
		@View(name = "tag_counts", file="/couch/foo/fooTagCounts.json")
})

and calls initStandardDesignDocument(); in the c'tor after calling super(Foo.class, db).
That's all the info I can think of right now...

@huxi when using ApacheHC 4.x you need to change the way you instantiate the Apache HttpClient.
I have a working Spring XML Context file that does that
https://github.com/YannRobert/Ektorp/blob/a8b6d3c53f75cc6dd1d2644ed8ea88a427f4ff68/org.ektorp.spring/src/test/resources/ektorp-httpclient-config-4.xml
For the moment it depends on code that is not released in Ektorp.
The following branch https://github.com/YannRobert/Ektorp/tree/SupportingNewestVersionsOfApacheHttpComponent should contain all that is needed (but it contains more than what is needed)

But... it still works with 4.3.6.
I'll have to wait for a released ektorp version to use this in our production system so I just stay at 4.3.6 for now.
Configuration looks quite a bit more complex. Wouldn't it be possible to prevent some of it by using sensible defaults? Or is this already the case and your config example simple configures everything that is possible?

Getting the same issue with 4.5.2. It would be nice to see a release update.
@helun what are your plan regarding a new release?

The issue still remains with httpclient-4.5.3. Please fix it or at least provide a workaround. Thanks.

I'm getting the same issue with a fresh build of Ektorp 1.4.5-SNAPSHOT and httpclient-4.5.1. Did anyone find a workaround for this ? Are there any commits from Yann Robert's SupportingNewestVersionsOfApacheHttpComponent branch that might help and that have not been merged upstream yet ?

I merged the SupportingNewestVersionsOfApacheHttpComponent branch of Yann Robert's fork into helun/Ektorp master branch (1.4.5-SNAPSHOT) and it seems to be working fine with httpclient-4.5.1 (see https://github.com/amischler/Ektorp).
Should I create a PR ?
Note : the configuration of the http client is a bit more complex now :
RequestConfig requestConfig = RequestConfig.custom().build();
ConfigurableHttpRequestRetryHandler retryHandler = new ConfigurableHttpRequestRetryHandler();
retryHandler.setIncludeExceptionInLogs(true);
CredentialsProvider credentialsProvider = new CredentialsProviderConfigurer.DefaultCredentialsProvider();
CredentialsProviderConfigurer credentialsProviderConfigurer = new CredentialsProviderConfigurer();
credentialsProviderConfigurer.setHost("localhost");
credentialsProviderConfigurer.setPort(5984);
credentialsProviderConfigurer.setUsername(couchDbConfiguration.getUserName());
credentialsProviderConfigurer.setPassword(couchDbConfiguration.getPassword());
credentialsProviderConfigurer.setCredentialsProvider(credentialsProvider);
credentialsProviderConfigurer.configure();
PoolingHttpClientConnectionManager connectionManager = new PoolingHttpClientConnectionManager();
CouchDbDefaultRoutePlanner routePlanner = new CouchDbDefaultRoutePlanner();
routePlanner.setServerHostName("localhost");
routePlanner.setServerHostPort(5984);
HttpClientBuilder httpClientBuilder = HttpClientBuilder.create()
  .setKeepAliveStrategy(new ConfigurableConnectionKeepAliveStrategy())
  .setRetryHandler(retryHandler)
  .setConnectionReuseStrategy(new CustomConnectionReuseStrategy())
  .setRoutePlanner(routePlanner)
  .setDefaultRequestConfig(requestConfig)
  .setDefaultCredentialsProvider(credentialsProvider)
  .setConnectionManager(connectionManager);
AuthenticatedCloseableHttpClientRequestExecutor requestExecutor = new AuthenticatedCloseableHttpClientRequestExecutor();
requestExecutor.setServerHostName("localhost");
requestExecutor.setServerHostPort(5984);
requestExecutor.setBackendBuilder(httpClientBuilder);
requestExecutor.setClientBuilder(httpClientBuilder);
requestExecutor.init();
httpClient = new StdHttpClient(requestExecutor);

Hi @amischler
Great to see the SupportingNewestVersionsOfApacheHttpComponent branch is working for you.
I can confirm that this code is currently in use on one of my project for several years now.
I worked on this a long time ago, so I am not sure what it contains exactly.
I would have to review it. It contains many commits so it's difficult.
One concern would be to check it does not break things for users of older versions of Apache HC.
Any feedback would be appreciated and would help in getting it merged.

Just got hit by the this issue - triggered by the Amazon S3 API which won't play with the 4.3.6 HTTP Client version.
@YannRobert and @amischler
Given you efforts - would it be an option for you to release a 2.0.0 version with a breaking change notification? The time required to verify (and maintain) backwards compatibility with the 2014 release of HC might be of better use in other areas.

Hi @robertgartman. Thank you for the comment.
Given that this issue hits more and more people, we will sure need to address this.
I'll try to work on that by next week.

Trying with the version by @amischler landed in the No content to map....
Caused by: com.fasterxml.jackson.databind.JsonMappingException: No content to map due to end-of-input
 at [Source: java.io.InputStreamReader@af29cfa; line: 1, column: 0]
	at com.fasterxml.jackson.databind.JsonMappingException.from(JsonMappingException.java:270) ~[jackson-databind-2.8.8.jar:2.8.8]
	at com.fasterxml.jackson.databind.ObjectMapper._initForReading(ObjectMapper.java:3838) ~[jackson-databind-2.8.8.jar:2.8.8]
	at com.fasterxml.jackson.databind.ObjectMapper._readMapAndClose(ObjectMapper.java:3783) ~[jackson-databind-2.8.8.jar:2.8.8]
	at com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:2887) ~[jackson-databind-2.8.8.jar:2.8.8]
	at org.ektorp.http.ClassInstanceResponseHandler.success(ClassInstanceResponseHandler.java:46) ~[org.ektorp-1.4.5-SNAPSHOT.jar:?]
	at org.ektorp.http.RestTemplate.handleResponse(RestTemplate.java:130) ~[org.ektorp-1.4.5-SNAPSHOT.jar:?]
	at org.ektorp.http.RestTemplate.get(RestTemplate.java:22) ~[org.ektorp-1.4.5-SNAPSHOT.jar:?]
	at org.ektorp.impl.StdCouchDbConnector.get(StdCouchDbConnector.java:247) ~[org.ektorp-1.4.5-SNAPSHOT.jar:?]
	at org.ektorp.impl.StdCouchDbConnector.get(StdCouchDbConnector.java:237) ~[org.ektorp-1.4.5-SNAPSHOT.jar:?]

I did not experience this issue with the merged version. Note that we are not making an intensive use of Ektorp, we are just evaluating the Ektorp + CouchDB solution for the moment.
Maybe something wen't wrong when I solved some conflicts during merging ? In this case a unit test for the case you are experiencing might be missing ;)

Any news on this?

Setting StdHttpClient.caching to false helped me

As of HttpClient-Cache 4.4+ HEAD requests will be cached. This seems to break in cases where the HEAD request comes first before any subsequent GET requests on the same target. In other words: if you can use httpclient-cache:4.3.x in your setup, you should be able to work around that issue. Details/demo see https://github.com/gesellix/httpclient-cache-demo. I've already started a discussion about that issue at the httpclient-users mailing list<EMAIL_ADDRESS>- feel free to join.


Update gujarat.json

Description of PR
Type of PR

[x] Bugfix
[ ] New feature

Relevant Issues
Fixes #...
Checklist

[ ] Compiles and passes lint tests
[ ] Properly formatted
[ ] Tested on desktop
[ ] Tested on phone

Screenshots

The officail name of district is Dahod not Dohad, local people somtimes called it as Dohad as district share its border with 2 states

@murtuza1234 Thanks for your first PR. This modifications are already available in PR - #807
cc - @shuklaayush

Census 2011 officially called it Dohad http://censusindia.gov.in/2011census/dchb/GujaratA.html we are sticking to this for now


Experimental Control Styles in VS 17.9.0 (Preview)

Visual Studio version 17.9.0 (Preview 1) has introduced a new "Experimental control styles" setting for enabling the upcoming/new UI styling, which includes changes to/new built-in themes.

The UI Refresh appears to have broken the Catppuccin Theme, for example Macchiato is now a flash-bang:

naturally, it is broken - it is in preview and isn't supported yet. Labeling this as enhancement.

closing this for now - the preview feature doesn't seem to be present in recent VS versions.


Can't get unattended backup to write to log

I am using cron to schedule backups and it's working fine.  What I am unclear on is how to get it to write to a log (or, indeed, how the log file is designated).
I've tried using -q and -u options, but I must be missing something.
Could anyone tell me please:
a) where is the log file defined (or is it just by redirecting output)?
b) what command should I use in my crontab to run unattended but also write out the log to this file?

1 0 * * * /usr/local/sbin/rpi-clone -f -U >> /root/rpi-clone.log 2>&1

The help output (from running rpi-clone with no parameters) says:

A line logging a rpi-clone run is written to /var/log/rpi-clone.log.

The help output (from running rpi-clone with no parameters) says:

A line logging a rpi-clone run is written to /var/log/rpi-clone.log.

Thank you. Don't know how I missed that!


Handle exposing (..) in NoRedundantlyQualifiedType

Adds support for handling exposing (..) in the new NoRedundantlyQualifiedType rule.
@lydell if you could review this that would be great!
Checking I haven't missed any tests may be the most important part (but any other comment are welcome too)

This looks great!
I ran it at work, and it found a couple of more things to simplify compared to before this PR, and it caused no compilation errors.
The code and tests look good (I learned a lot from reading it). The only thing I noticed not being tested, is import ... exposing (..) from an installed package (dependency). Or did I just miss it?

Good catch! I've added some tests with dependencies.

You may notice that in the new test code with dependencies, there is code like Project.Project (from import Review.Project as Project). Was this purposefully not handled @lydell? If it is, then I think we should document that. If it isn't, then maybe we should add this handling when the type alias is also the same as the type name. What do you think?

I’m not sure if I follow. Can you show an example?
I did notice one thing in the new tests, though (but it doesn’t sound like what you are describing?). For OtherSetTypeAlias:
test "should not report an error if an import from a dependency exposes a type alias we want to shorten (exposing all)" <|
            \() ->
                """module A exposing (..)
import Set
import OtherSetTypeAlias exposing (..)
a : Set.Set a
a = Set.empty
"""

I guess it would be safe to shorten Set.Set here, if the type alias in OtherSetTypeAlias is actually an alias for the real Set? But for me, it doesn’t matter if we support this or not, it feels like too much of an edge case.

I thought we already did that, and already had a test in my PR?

Maybe I broke that functionality, but I don't think there was such a test. I at least don't see the as keyword in any test (and I haven't removed one in this PR).
But good to know. I'll try to re-add it.

Nevermind, I had enabled it in the wrong ReviewConfig.elm file (the example/ one) instead of the project's :facepalm:
I still added a test for the import alias. I think this is now ready to be merged, and therefore to be published!
Do you see anything I'm missing?

Awesome! Then I think this is good to go. I can’t think of anything else! Thank you for finishing the part of the rule that was difficult for me.

Awesome! Thank you for the help!
I'll publish it now.


ci: Fix implicit octal values in main.yml

Enhancement:
Reason: Forbidden implicit octal values "0644" were found on lines 118, 131, and additional occurrences.
Result: Updated to use explicit octal format for better readability and to adhere to linting standards.
Issue Tracker Tickets (Jira or BZ if any):na

@HVSharma12 thanks!


Uncaught NotFoundError in utils.py line 8071

Aider version: 0.58.0
Python version: 3.12.3
Platform: Linux-6.8.0-45-generic-x86_64-with-glibc2.39
Python implementation: CPython
Virtual environment: Yes
OS: Linux 6.8.0-45-generic (64bit)
Git version: git version 2.43.0
An uncaught exception occurred:
Traceback (most recent call last):
  File "openai.py", line 907, in completion
    raise e
  File "openai.py", line 825, in completion
    self.make_sync_openai_chat_completion_request(
  File "openai.py", line 683, in make_sync_openai_chat_completion_request
    raise e
  File "openai.py", line 672, in make_sync_openai_chat_completion_request
    raw_response = openai_client.chat.completions.with_raw_response.create(
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "_legacy_response.py", line 353, in wrapped
    return cast(LegacyAPIResponse[R], func(*args, **kwargs))
                                      ^^^^^^^^^^^^^^^^^^^^^
  File "_utils.py", line 274, in wrapper
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "completions.py", line 704, in create
    return self._post(
           ^^^^^^^^^^^
  File "_base_client.py", line 1268, in post
    return cast(ResponseT, self.request(cast_to, opts, stream=stream, stream_cls=stream_cls))
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "_base_client.py", line 945, in request
    return self._request(
           ^^^^^^^^^^^^^^
  File "_base_client.py", line 1049, in _request
    raise self._make_status_error_from_response(err.response) from None
openai.NotFoundError: Error code: 404 - {'error': {'message': 'The model `o1-mini` does not exist or you do not have access to it.', 'type': 'invalid_request_error', 'param': None, 'code': 'model_not_found'}}

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "main.py", line 1419, in completion
    raise e
  File "main.py", line 1372, in completion
    response = openai_o1_chat_completions.completion(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "o1_handler.py", line 58, in completion
    response = super().completion(
               ^^^^^^^^^^^^^^^^^^^
  File "openai.py", line 914, in completion
    raise OpenAIError(
litellm.llms.OpenAI.openai.OpenAIError: Error code: 404 - {'error': {'message': 'The model `o1-mini` does not exist or you do not have access to it.', 'type': 'invalid_request_error', 'param': None, 'code': 'model_not_found'}}

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "aider", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "main.py", line 727, in main
    coder.run()
  File "base_coder.py", line 730, in run
    self.run_one(user_message, preproc)
  File "base_coder.py", line 773, in run_one
    list(self.send_message(message))
  File "base_coder.py", line 1208, in send_message
    saved_message = self.auto_commit(edited)
                    ^^^^^^^^^^^^^^^^^^^^^^^^
  File "base_coder.py", line 1891, in auto_commit
    res = self.repo.commit(fnames=edited, context=context, aider_edits=True)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "repo.py", line 110, in commit
    commit_message = self.get_commit_message(diffs, context)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "repo.py", line 195, in get_commit_message
    commit_message = simple_send_with_retries(
                     ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "sendchat.py", line 44, in wrapper
    return decorated_func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "_sync.py", line 105, in retry
    ret = target(*args, **kwargs)
          ^^^^^^^^^^^^^^^^^^^^^^^
  File "sendchat.py", line 102, in simple_send_with_retries
    _hash, response = send_completion(**kwargs)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "sendchat.py", line 83, in send_completion
    res = litellm.completion(**kwargs)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "utils.py", line 1086, in wrapper
    raise e
  File "utils.py", line 974, in wrapper
    result = original_function(*args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "main.py", line 2847, in completion
    raise exception_type(
          ^^^^^^^^^^^^^^^
  File "utils.py", line 8194, in exception_type
    raise e
  File "utils.py", line 8071, in exception_type
    raise NotFoundError(
litellm.exceptions.NotFoundError: litellm.NotFoundError: NotFoundError: OpenrouterException - Error code: 404 - {'error': {'message': 'The model `o1-mini` does not exist or you do not have access to it.', 'type': 'invalid_request_error', 'param': None, 'code': 'model_not_found'}}

Thank you for filing this issue.
The o1 models on the OpenAI API still require an account with Tier 4, you can use OpenRouter instead to access those models without this requirement.

thanks yes thats what i did, its set on default openrouter/openai/o1-mini now and works well

As this issue appears to be resolved, I'm closing it.
If any new related concerns arise, please feel free to comment, and I'll reopen the issue.

🙂 np thank you
-------- Original message --------From: fry69 @.> Date: 9/30/24  08:52  (GMT+02:00) To: paul-gauthier/aider @.> Cc: Rudyz - CTO and head of AI R&D @.>, Author @.> Subject: Re: [paul-gauthier/aider] Uncaught NotFoundError in utils.py line
8071 (Issue #1834)
As this issue appears to be resolved, I'm closing it.
If any new related concerns arise, please feel free to comment, and I'll reopen the issue.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Small refactoring and code cleanup

Small refactoring and code cleanup

@confluentinc It looks like @ralmsdeveloper just signed our Contributor License Agreement. :+1:
Always at your service,
clabot

thanks. most of these changes are moving away from (implicit) coding standards throughout the repo though ...

@mhowlett
Do you think about taking it forward?


Tried to find Eigen3::Eigen_interface_link_libraries-NOTFOUND

I have a pure cmake package that I am leveraging in a ros package. I switched the pure cmake package to using the target Eigen3::Eigen when linking but I am now getting the following build error from catkin tried to find library Eigen3::Eigen_interface_link_libraries-NOTFOUND. I believe issue is when we check if it is an interface library shown below it calls get_target_property but it is not checking if it returns NOTFOUND so it is adding an incorrect item to list.
https://github.com/ros/catkin/blob/f9a573fcdc9e85f5f5ef99e9f9b04374096d404b/cmake/catkin_libraries.cmake#L138-L142

To verify that this is the case added the following to my ros cmake package:
find_package(Eigen3 REQUIRED)
get_target_property(test_type Eigen3::Eigen TYPE)
message(${test_type})
get_target_property(test_imported Eigen3::Eigen IMPORTED)
message(${test_imported})

if(test_type STREQUAL "INTERFACE_LIBRARY")
    get_target_property(test_interface_link_libraries Eigen3::Eigen INTERFACE_LINK_LIBRARIES)
    catkin_replace_imported_library_targets(test_resolved_libs ${test_interface_link_libraries})
    message(${test_resolved_libs})
endif()

Terminal output:
Warnings << tesseract_rosutils:check /home/larmstrong/catkin_ws/trajopt_only_ws/logs/tesseract_rosutils/build.check.019.log
INTERFACE_LIBRARY
TRUE
test_interface_link_libraries-NOTFOUND

Please consider to provide a pull request for this issue.

@dirk-thomas I just submitted a PR that address this issue.

Addressed by #1013.


Modem pipe add TRANSMIT_IDLE event

This PR adds the notification TRANSMIT_IDLE which will be sent by modem backends once they have sent all data passed through the modem_pipe_transmit API. This will allow users of the modem_pipe to synchronize the modem_pipe_transmit calls more efficiently.
Currently, we are stuck in a loop like this while transmitting data through a modem_pipe until all data has been transmitted:

Transmit data using modem_pipe_transmit() until it returns 0 (its internal buffer is full)
Yield for "some time"
Go to step 1

To underline how inefficient the current approach is, while performing the transmit throughput test which is part of the cellular_modem sample, it completely starves the logging thread until all bytes have been transmitted...
This blind and continuous call to modem_pipe_transmit() before it is ready to accept new data has no utility, and is therefore wasted. The only current way to diminish the wasted calls is to yield for longer between each call to modem_pipe_transmit(), but this hurts throughput.
With the TRANSMIT_IDLE event, we can synchronize the transmit with the modem backend by waiting once the modem_pipe_transmit() returns 0:

Transmit data using modem_pipe_transmit() until it returns 0 (its internal buffer is full)
Wait for TRANSMIT_IDLE event
Go to step 1

This results in only one potentially "wasted" call to modem_pipe_transmit() if its internal buffer is completely filled, which we can't avoid in any case.
This PR also adds a test suite for the modem_pipe module, and implementations of the TRANSMIT_IDLE events for the existing modem backends; UART and TTY.
After extending the CMUX and PPP modem modules to utilize the TRANSMIT_IDLE event (future PR), the max CPU utilization of the system work queue thread (which all modem modules run in) went down to 5% from 36%, while reducing the throughput by 12%. The logging is now unblocked while running the throughput test as well :)

ping @fabiobaltieri @ycsin @rerickson1 I created this PR in the middle of the holidays so I think it went unnoticed :)


[Loc] changes for 11/16/2022

This PR fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.978%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
28754

Relevant Lines:
63637

💛  - Coveralls


ForestDB storage cause native crush during push replication performance test with 200 docs

By fixed #750, this issue is no longer reproducible.


Extending of Models not allowed. Constructor is private.

I would like to extend the model Artist because it's preferable in my business logic.
Currently there is no way to extend the artist object because the constructor access is private. I presume this is done because of the Builder.
But this creates a problem for me. Is there any way to get this working?
I'm on version 2.0.0-RC1

Model objects are not meant to be extendable, because they follow a specification.

What exactly is the problem you are speaking of?
What exactly do you want to change in you extending class?
What prevents you from writing your own custom class?

Maybe an option would be to make the constructors public and deprecated. But I'd first like to know answers to the questions above.

Putting this on hold until someone gives a conrete example.


Eliminating conversions to unscoped enum type

Figured I'd open an issue for this, since it was discussed in multiple places (#22, #23).
I have another suggestion besides making "strict conversions" the default: eliminate implicit conversions to enums completely. IIRC they are only needed for switch, for which we could provide a function with a name like _switch:
switch(my_enum._switch()) {
    case Foo::A: break;
    case Foo::B: break;
}

We could also make this operator +, if you can bear all the operator +s :)
switch(+my_enum) { ...

The conversion this would perform would be to an unscoped enum type, so we could use the Foo::A syntax in cases without having to prepend the existing operator +.
cc @cheparukhin @mbevin @rcdailey

Re: @rcdailey's suggestion of enumerators from #23:
I considered enumerators, but IIRC the preprocessor+template hack I use to get mixed initialization syntax ({A, B, C = 4, D}) working doesn't work very well with them, and I had convinced myself that there wasn't a variant that does. I may be wrong, as it has been a while, but for background on that part of Better Enums, you can see this answer on SO.

Re: @rcdailey on breaking compatibility with C++03/98:
I should repost from #22:

The guideline I am using is that the enum.h in master should be immediately compatible with recent versions of GCC, Clang, and MSVC. [snip]
Since MSVC has had enum class for a while now, we can switch to that, and drop support for [converting to] the old-style internal enum completely. After doing that, we can just add a note to the README telling users of older MSVC versions which release of Better Enums they should download.

The same guideline would probably preclude switching to C++11 completely (i.e. constexpr), due to the infamous #9.

If you want to know the status of that, do an amend or add a dummy commit in the vc2015-constexpr branch to trigger a build, and see what happens in AppVeyor. You might have to look up the most recent version of MSVC, and check that the AppVeyor image we are using has that.

So it's not so much the age of C++11 that constrained me, as how long it has been since MSVC added support for each particular feature. Have you heard anything about MS potentially switching to Clang recently?
Also, the approach I suggested when opening this issue won't cause Better Enums to stop supporting C++03/98, though we might drop it anyway on another count. But either approach is fine.

Concerning the internal type:
I think it actually doesn't matter that much what that is, whether enumerator, scoped enum, or unscoped enum, as long as a Better Enum can be introduced naturally (Foo::A) and used naturally, or close to that, in switch. The latter seems to imply that there must be some kind of conversion to some kind of enum type, as otherwise you lose exhaustiveness checking. The current interface is, as we all agree, really leaky, since it has an implicit conversion to an unscoped enum. AFAIK it is the only way to get fully natural switch syntax, but I think a lot of people are willing to forgo that. What are your opinions?

I don't like the discrepancy between external and internal enum types because of all these conversion issues, so I'd rather try to have just a single type.
Some crude ideas from the top of my head:

Use enum class for enum type and augment it with free functions in the macro:
enum class MyEnum: uint8_t { ONE, TWO, THREE };
MyEnum toMyEnum(const std::string&);
std::string toString(MyEnum);

This will obviously break API and it means that almost the whole library should be rewritten, so I guess it's not viable.

Define enum values as static constants inside the enum type, using the same type:
class MyEnum {
public:
    static const MyEnum ONE;
    static const MyEnum TWO;

private:
    constexpr MyEnum(uint8_t value): m_value(value) {}
    uint8_t m_value;
};

const MyEnum MyEnum::ONE = 1;
const MyEnum MyEnum::TWO = 2;

What bothers me is that it's not possible to use constexpr in this case as MyEnum would be incomplete at the point of these definitions.

@aantron How do you reckon?

Use enum class for enum type and augment it with free functions in the macro:

Let me paraphrase to make sure I understood you correctly: you want the macro to generate a normal, non-wrapped enum, and then also generate free functions that do the various conversions and whatnot that Better Enums provides. If so, this is a good idea, and it would eliminate a ton of problems.
I was calling this "enum traits," because in this mode, Better Enums is generating something like a traits struct over a regular enum. This is opposed to the current approach, where a struct wraps an enum, and you use objects of this struct type.
There is an old implementation of traits in a branch. I decided not to pursue it at the time for various reasons. It was discussed multiple times on Reddit and in email. I later decided that it's worth maintaining side-by-side with the existing "object" enum.h (see #17). But, then I got burned out, so that didn't go anywhere.
I wouldn't worry about breaking the API. It can be implemented in a branch, or in a separate file enum_traits.h, or whatever. The separate file is probably the better. We can let people know, and if people definitively prefer traits, Better Enums can drop the object approach. It is extra maintenance work, but the experiment is probably worth it.

Define enum values as static member constants of the same type:

I'm not sure this can work. Consider some of the constraints:

One of the strengths of Better Enums is that it uses the already-familiar (and pretty decent) enum declaration syntax, i.e. A, B, C = 4, D.

Each of A, B, C = 4, D becomes a macro paramter.

There is no way, AFAIK, to separate the name C in the macro parameter from the = 4 initializer. This already seems to preclude
 static const MyEnum TWO;

since you have to paste in both name and initializer, if the latter is present. A further problem is you don't really know what values to assign to ONE, TWO. Using my example, how would you know that D should be 5?
I've considered fancier schemes with enumerators, like defining an enum and then referring to it when declaring enumerators:
    enum Internal {A, B, C = 4, D};
    // ...
    static const Whatever A = Internal::A
    static const Whatever B = Internal::B
    static const Whatever C = 4 = Internal::C = 4   // wat

As you can see, that breaks down. In the last line, casting to dummy types with overridden operator = can eliminate all the problems, except, AFAIK, for the problem of assigning to 4 (because, AFAIK, you cannot get a cast on either the C or the 4, due to this being declaration syntax, and C = 4 coming from a macro argument; you cannot have free overloads of operator =).
There are several other variations with enumerators, but all that I considered seem to suffer from variations of this problem.

If you want to work on the "object"-style enums, the best advice I can give is to start with the code in the SO answer I linked. It's pretty short (<200 lines including whitespace), and includes only the core of Better Enums, with only two functions implemented on top of that. You can use it as a small example to test modifications, or as a reference for writing an approach from scratch. It should quickly filter out approaches that don't pass the (weird) constraints of C++.

After reading more into the discussion, I agree with @cheparukhin's first idea. I think it would make sense to keep the scoped enum outside and just build a series of free functions to operate on them. This is especially easy to do with operators such as operator<< and operator>>, which are arguably all you really need. STL and Boost use external stream operators on types for conversion to/from string format. So you could easily do:
BETTER_ENUM(TestEnum, int, One, Two, Three);

std::string one_str = boost::lexical_cast<std::string>(TestEnum::One);

Naturally, switch will already work and you get type-safety guarantees.
Are there any current requirements that would be difficult or impossible to support using this approach? Honestly the only reason I really use better enums in production code is for the convenient string to enum mapping.
I think this functionality will be built into the language by C++17 or C++20, though.

Indeed, that's what is in the traits branch now. I just didn't keep that branch up-to-date with subsequent improvements to the internals of master.
See:

basic usage
switch
all the samples

Are there any current requirements that would be difficult or impossible to support using this approach?

Besides what's in the linked FAQ, I think not. You lose:

reflective enums in namespaces other than the global namespace, unless we can come up with a workaround,
a small degree of conciseness in the syntax,
the ability to prevent default construction, for people who care about that.

This is properly issue #17, but basically, getting the traits branch updated would involve going through the history since May 17 or so, and applying anything that makes sense. For example:

Inlining the map macro (analog of 460bb806a38f9a583882fdbb4484b3ce8a954160), for a single-header library.
Renaming the main macro (#11).
Various internal improvements, such as removing weak symbols.

Regarding C++17, it will greatly simplify the internals of the library. I am not sure what, at this point, exactly is planned for C++17, or whether it will have to wait until C++20. But, the last I checked, the interfaces proposed were pretty basic and difficult to use directly, so it may still be worthwhile to have Better Enums around. Otherwise, one of those standards will retire this library :)


configurable max attempts for docker pull in custom executor

@jonathanwgoodwin , still need to test this out, but should do what we need

This has been working fine in stable, merging


Bump CONTROLLER_TOOLS_VERSION to 0.14.0

bump controller tools to the version which bumps k8s deps to v0.29.0, which aligns with the version in the operators.

/retest

recheck

/approve


PerWorldInventoryAPI#getGroup is always null

Describe the bug
the PerWorldInventoryAPI#getGroup will always result in null
To Reproduce
Steps to reproduce the behavior:

use the getGroup method how you normally would, (api.getGroup("GroupName"))
view error

Expected behavior
Id imagine it would return a valid group where i can add worlds to via api.
Plugin version
Using latest version on spigot, 2.3.1
Server version
Paper-637, Minecraft 1.13.2
Additional context and logs
Logs:
https://paste.ubuntu.com/p/HRpRgp8zJD/
I did add the plugin as a dependancy in my plugin.yml, as well as i tried using a group i had created in game.
Edit: wanted to add, i know that this is null from trial and error.
Edit 2:
Workaround
You can use PerWorldInventoryAPI#getGroupFromWorld to create a group on the fly and add to that group with the expected result.

It's possible that when you're calling that method, the groups haven't been loaded yet. As a temporary workaround, you could try waiting at least 2 ticks. I'll see if I can make things happen a bit faster.


An update

Please build new images to use new php versions. One of projects I work on requires php version 7.4.7 and above becase yield from is bronken in 7.4.6.
Thank you in advance!

Re-run actions on a1e1ac5b9e9a6c3d79eda20c594e58cd24e4f2b8 commit

Thank you very much!


Add CI workflow

Run black, flake8 and mypy tests on creating a pull request.
This does not include running unit tests yet

🥳


bot crashes when userargs is cpol

message that caused the error: ;osu cpol

Error message:
time = new Date(user.rank_highest.updated_at).getTime() / 1000
^
TypeError: Cannot read properties of null (reading 'updated_at')
at getUserFirstPage (D:\miaosu\src\commands\osu_profiles\osu.js:220:41)
Original Message by @yoru#9267

the issue was caused by the rank of the user not being defined, so the bot couldn't get peak rank


Enable vue-cli-service inspect to export target- or command-specific config.

What problem does this feature solve?
The current implementation of the inspect command only returns the generic config for an app. We can set --mode to switch i.e. between development and production configurations, but we can't get the actual, final config that is used when we i.e. run build with --target lib or --target wc
But these flags result in changes to the base config, tailoring the config to i.e. bundle as a commonjs library or a web component, and these changes are invisible to the user.
So someone attempting to customize their lib bundle configuration might have a hard time customizing some things, as some plugins are missing (i.e.copy) , while other options are changed (i.e. entry)
What does the proposed API look like?
Add additional flags to inspect that allow the user to get the config for all of the different scnearios we support, i.e.
vue-cli-service inspect --mode production --target lib ---formats commonjs

vue-cli-service inspect --mode production --target wc

Open questions

The example above is not a complete list as I'm not even sure: What are all the scenarios that we should support?
What about test config (mocha-webpack)?
What about custom commands from 3rd party plugins (i.e. build:ssr)? We can't really hook cleanly into config changes applied by those commands, can we?

+1


support viewBox

hello, we need to support viewBox property for responsive webpage
plz handle it on pull requests

Is possible to fix this issue? we need use this SVG in a mobile device.
thanks

I am very sorry. I had fixed it in v1.3.12. I've been busy designing v3.x to solve the problems thoroughly but more time need. The new release will solve a series of problems and provide more graceful APIs.
Keep in touch if you have any questions. Thanks for your support.


Pokedex: Dashboard not working

In Pokedex example it states you can monitor indexing using Jina Dashboard. However opening https://dashboard.jina.ai it says 'could not connect to Jina instance'. I guess Pokedex doesn't have Dashboard port enabled?
Fixes
Either

Open the port by default
Remove this reference from the readme

@alexcg1 is this about the part where it says you can view the logs?

@aga11313 this bit: https://github.com/jina-ai/examples/tree/master/pokedex-with-bit#monitor-progress


Opt out from extending Active Record

Currently it's impossible to not load AR extensions in Rails apps.
Would you consider a change to extend AR just by including explicit some geocoder module?
Something like:
class User < ActiveRecord::Base
  include Geocoder::ActiveRecord
end

If no, maybe option in configuration to opt out from it would be possible to add?

Yes, that seems reasonable. I would consider a pull request.

Which version do you prefer?
The first one looks more clear, but it would be backward incompatible.
I believe this also apply to mongoid and mongomapper extensions

Definitely don't want to introduce anything that would break anyone's app. Maybe configuration is the way to go.

I tried with adding config, but looks like it's impossible. Geocoder.configure is usually placed in initializer, where the rails application is already loaded, thus it's after the gem configuration done in Railtie, where we would like to opt out based on user set config.
I'm also not sure what would be the best way of deprecating it, if we decide to switch to explicit extend:
class User < ActiveRecord::Base
  extend Geocoder::Model::ActiveRecord
end

While I acknowledge the benefits of this, I don't like the idea of requiring an explicit extend. First priority is not to break apps (which includes deprecating things). But like I said, this is desirable functionality and I remain open to a pull request.
Closing for now.


Lab 08: Creation of bucket fails

In terraform/Lab_08_Infrastructure_Security_Network, terraform apply fails with:
│ Error: 409-BucketAlreadyExists, Either the bucket 'IAD-NP-LAB08-BUCKET-98927591-lab.user01' in namespace 'ocuocictrng7' already exists or you are not authorized to create it
│ Suggestion: The resource is in a conflicted state. Please retry again or contact support for help with service: Object Storage Bucket
│ Documentation: https://registry.terraform.io/providers/oracle/oci/latest/docs/resources/object_storage_bucket
│ API Reference: https://docs.oracle.com/iaas/api/#/en/objectstorage/20160918/Bucket/CreateBucket
│ Request Target: POST https://objectstorage.us-ashburn-1.oraclecloud.com/n/ocuocictrng7/b
│ Provider version: 6.18.0, released on 2024-11-13. This provider is 2 Update(s) behind to current.
│ Service: Object Storage Bucket
│ Operation Name: CreateBucket
│ OPC request ID: iad-1:IcjPkU6TBvhsrVI2UIL5U78IlrreS6oYo5CYtUjmOGD_k0uBOGDIuUWiyhfhAj3K
│
│
│   with oci_objectstorage_bucket.IAD-NP-LAB08-BUCKET,
│   on ashburn_bucket.tf line 43, in resource "oci_objectstorage_bucket" "IAD-NP-LAB08-BUCKET":
│   43: resource "oci_objectstorage_bucket" "IAD-NP-LAB08-BUCKET" {

Use of random number in bucket name did not fix the problem:
│ Error: 409-BucketAlreadyExists, Either the bucket 'IAD-NP-LAB08-BUCKET-98927591-lab.user01-729' in namespace 'ocuocictrng7' already exists or you are not authorized to create it
│ Suggestion: The resource is in a conflicted state. Please retry again or contact support for help with service: Object Storage Bucket
│ Documentation: https://registry.terraform.io/providers/oracle/oci/latest/docs/resources/object_storage_bucket
│ API Reference: https://docs.oracle.com/iaas/api/#/en/objectstorage/20160918/Bucket/CreateBucket
│ Request Target: POST https://objectstorage.us-ashburn-1.oraclecloud.com/n/ocuocictrng7/b
│ Provider version: 6.18.0, released on 2024-11-13. This provider is 2 Update(s) behind to current.
│ Service: Object Storage Bucket
│ Operation Name: CreateBucket
│ OPC request ID: iad-1:WjPBjF-SjctUaNkN5DZ-tHjleGpjwN4yzQD-SOuoaK1Y7a0vFOMreGQ_TMw5pkMj
│
│
│   with oci_objectstorage_bucket.IAD-NP-LAB08-BUCKET,
│   on ashburn_bucket.tf line 56, in resource "oci_objectstorage_bucket" "IAD-NP-LAB08-BUCKET":
│   56: resource "oci_objectstorage_bucket" "IAD-NP-LAB08-BUCKET" {

MyLearn has marked the lab for this course as complete. I am unable to work on this issue further.

Lab for course has been reopened.


Add gas price to appendable block

CHANGELOG:

Add new field current_gas_price to the appendable block
Add new field current_gas_price to the executable and finalized block
Added field current_gas_price to the block validator which checks the gas price in the proposed block and ensures the value matches, raises an invalid block if the price does not match

good enough to retest

bors r+

bors r+


Question: How can I use RATreeView with a storyboard?

Is there any way to integrate with storyboards? For example, if there was some integration with tableviewcontroller then we could create segues more easy.

Just like the examples, you need a class that implements UIViewController.  In storyboard you can create a blank UIViewController instance.  You can add toolbars, etc; and set up your segues.  In the class implementation, manually add your RATreeview as a subview in ViewDidLoad (just like the example).

Thank you for your reply :)
I have already integrated with simple view controller storyboard object following the example. But, I think that the example is more focused on a programming solution.

When you touch a row, the children of the node unfolds. What I would prefer is to find out how I can create a segue/link to a next view controller to see details for the specific node. I think that the functionality to unfold on touch of a row is somehow buildin. Please correct me...
Can I use a tableviewcontroller storyboard object instead of simple view controller and how the programmatic functionality of RATreeView can be integrated with TableViewController cells etc.?

@kmav I don't think you can really use RATreeView as you'd like with storyboards, because it doesn't subclass UITableView, but UIView.
So you need to take you UIView reference and then set everything up programmatically in viewDidLoad. Found a (much less popular) alternative library that subclasses UITableView, going to give using it a shot instead.

@AndrewSB Thank you for your feedback and the information for alternative library. In RATreeView library there is an RATableView class that inherits from UITableView. I wonder if I can use that to establish the integration.
Also, regarding segues, i.e. opening a detail view when the user touches on a tree row, what I have noticed is that when you touch a tree row the children of that row are expanding/unfold. Can I change the functionality of touching a row to e.g. open a detail view and use some other control for expanding the children, or is this not permitted at all (this is how it is designed)?

I apologize for missing your response to my initial comment.
When using a navigation controller you must use the "embed in" command in storyboard for any child view controllers to automatically add the toolbar and prevent overlap over sub views.
Regarding the expand/collapse functionality - implement the "should expand cell" and "should collapse cell" delegate methods to always return No. That will turn off the automatic expand/collapse action.
In my programs I use an addition style button that is only displayed by cells with children.  Tapping that button calls "expand" or "collapse" on the cell, and reloads the tree view.
capture the tapping action like a normal tableview cell with "didselect"
Joshua Jackson, PhD
Senior Research Engineer
(800) 604-1822
(256) 648-5109
<EMAIL_ADDRESS>www.nSide.io
4031 Parkway Dr, Suite B, Florence, AL 35630

On Jan 16, 2016, at 04:32, Konstantinos Mavroudakis<EMAIL_ADDRESS>wrote:
@AndrewSB Thank you for your feedback and the information for alternative library. In RATreeView library there is an RATableView class that inherits from UITableView. I wonder if I can use that to establish the integration.
Also, regarding segues, i.e. opening a detail view when the user touches on a tree row, what I have noticed is that when you touch a tree row the children of that row are expanding/unfold. Can I change the functionality of touching a row to e.g. open a detail view and use some other control for expanding the children, or is this not permitted at all (this is how it is designed)?
—
Reply to this email directly or view it on GitHub.

@kmav I don't think it's possible to use the private internal class to make a table. I ended up just rolling my own solution for this, since I only needed one level of collapsibility it wasn't too bad.
Can share my solution if it would be of any help with you

@joshjack and @AndrewSB thank you so much for your kind assistance and explanations that show me the direction to follow.
I will experiment a while and get back to you.

@joshjack
After inactivating default shouldExpand/Collapse, what do you call to reproduce their behaviour with the '+' style button?

@kmav it's a tree view command, something like [_mytreeview expandrowforcell:cell];
Because this is done as a code block using weak self inside the cell creation method (just like the demos), you have access to both the cell and the parent tree view.

@joshjack I have implemented most of what you suggested related to collapse/expand with an addition symbol and it works!
performseguewithidentifier crashes, because, I think, as @AndrewSB pointed out, RATreeView cannot be used as a UITableView, so I cannot create in storyboards a TableView object and connect it to my RATreeView. So, I cannot have a cell view to start ctrl-drag from it to my next detail scene.
What I did for the moment, is a programmatic solution in didSelectRowForItem as below.
- (void)treeView:(RATreeView *)treeView didSelectRowForItem:(id)item {

    TeamTableViewController *detailViewController = [self.storyboard instantiateViewControllerWithIdentifier:@"UnitTeam"];
    if (detailViewController) {
        detailViewController.activeUser = self.activeUser;
        OrgUnitsChildren *dataObject = [self.treeView itemForSelectedRow];
<EMAIL_ADDRESS>with unitID %d", (int)dataObject.unitID);
        [detailViewController setActiveUnit:[[Unit alloc] initWithName:dataObject.name unitID:[NSString stringWithFormat:@"%f",dataObject.unitID] lat:[NSString stringWithFormat:@"%f",dataObject.latitude] lng:[NSString stringWithFormat:@"%f",dataObject.longitude]]];
    }
    [self.navigationController pushViewController: detailViewController animated:YES];

}

@kmav here's an alternate approach:
#1 - create a segue in storyboard that is connected not from a cell, but from the parent viewcontroller of the RATreeView.  This is done by CTRL-CLICK dragging from the viewcontroller icon (yellow icon) at the top of the canvas to the next view.  You will be prompted to choose what type of seque to use.
#2 - name the segue like, "segueToTeamTBVC"
#3 -
- (void)treeView:(RATreeView *)treeView didSelectRowForItem:(id)item {
            [self performSegueWithIdentifier:@"segueToTeamTBVC" sender:self];
}

You'll still have to pass the "activeUser" somehow (I use a singleton class for that).
*Last note - this line:
OrgUnitsChildren *dataObject = [self.treeView itemForSelectedRow];
should be simplified to:
OrgUnitsChildren *dataObject = item;

@joshjack
Yes. You have right! I failed yesterday with that ctrl-click just due to bad arrangement of my scenes!!!
Thank you very much for your help.


Function return register

Register for function returns so you don't have to write so much to memory
Possible syntax
RET

fixed 4ee7d2c


Bug: Can't adjust volume

Disclaimer:
Please go through existing issues to avoid creating duplicates. If you found a similar issue please post a comment there. If you are uncertain your issue is related to others then create the issue. Obvious ignorance of this rule might get you blocked from posting new issues. Please tick the box below to show that you have followed the guideline.

[x] I have checked for a similar issue and sure it hasn't been reported before.

Describe the bug
Cannot adjust volume with mac keys
Steps to Reproduce
Steps to reproduce the behaviour (feel free to change the placeholder as you need):

Connect BT headphones
Adjust volume up on mac
Notice that the Mac visualizations shows the volume adjust input but the volume doesn't go up

Expected behaviour
Volume adjusts
Setup information:

Audio device used for playback: [eg AirPods or Macbook speaker or Bose QC32]
Audio transmission interface: [eq HDMI or Bluetooth or 3.5mm Jack]
macOS Version: 12.0.1
eqMac Version 1.2.0
eqMac UI Version 2.1.1

Screenshots or Console.app logs
In the video, I'm only pushing up on the volume
https://user-images.githubusercontent.com/29850965/142478642-8bba7e81-432c-493e-8d33-d374439949de.mov
Additional information
Add any other context about the problem here.

Duplicate


refactor(core): refactor the sending and support spoiler

[!WARNING]
waiting for updates from upstream

On the Telegram platform, the combination of <spl><image/></spl> is not supported, and this method is only applicable to text. However, the image API does support the has_spoiler option, so we can push for the standardization of this method in Satori Element: <image spoiler/>.
Update: Koishi RFC: https://github.com/koishijs/koishi/issues/1258

Is there a link for the upstream updates, an issue or a pending PR? Please include it into the description so we could track it from here.

TODO:

Add a comment at the line 267 that you're checking the current platform, explaining that why we should do this.
Mark this option as an experimental feature.
Add a comprehensive document for the option correspondingly, mostly why and when to use it.

3. Add a comprehensive document for the option correspondingly, mostly why and when to use it.

The document appears to be lagging behind many configurations. It is recommended that we open another pull request to address this.


Spyder 3.0.0b2 revives old copy/paste bug #2710

An issue described in #2710 has cropped up again in Spyder 3.0.0b2
Description
What steps will reproduce the problem?
What steps will reproduce the problem?
Copy text from Spyder iPython console using keyboard shortcut or context menu.
Paste into editor or another program.
Note the last character is missing.
What is the expected output? What do you see instead?
I would expect all the text to be copied.
Please provide any additional information below
Version and main components

Spyder Version: 3.0.0b2
Python Version: 2.7.11
Qt Versions: 4.8.6, PyQt4 (API v2) 4.10.4 on Darwin

Dependencies
pyflakes >=0.5.0   :  0.8.1 (OK)
pep8 >=0.6         :  1.5.6 (OK)
pygments >=1.6     :  1.6 (OK)
IPython >=3.0      :  4.1.2 (OK)
zmq >=13.0.0       :  14.3.0 (OK)
qtconsole >=4.0    :  4.0.1 (OK)
pandas >=0.13.1    :  0.14.0 (OK)
numpy >=1.7        :  1.8.1 (OK)
sphinx >=0.6.6     :  1.2.2 (OK)
psutil >=0.3       :  2.1.1 (OK)
rope >=0.9.4       :  0.9.4 (OK)
jedi >=0.8.1;<0.9.0:  0.8.1 (OK)
matplotlib >=1.0   :  1.3.1 (OK)
sympy >=0.7.3      :  0.7.5 (OK)
pylint >=0.25      :  1.5.4 (OK)

This is not a Spyder bug, but a bug in qtconsole (the package that we integrate in Spyder to provide the IPython console).
Please update qtconsole to version 4.2 (you're currently using 4.0.1). That will solve the problem :-)


Add Swift to OpenStackControlPlane

This allows deploying the Swift operator and also updates the default configuration samples to deploy a single replica Swift storage backend.

@cschwede: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name 	Commit 	Details 	Required 	Rerun command
ci/prow/openstack-operator-build-deploy 	2ed31fb link 	true 	/test openstack-operator-build-deploy
Full PR test history. Your PR dashboard.

It looks like the deployment timed-out because the Swift resources never become Ready:
    - lastTransitionTime: "2023-06-20T10:55:35Z"
      message: OpenStackControlPlane Swift in progress
      reason: Requested
      severity: Info
      status: "False"
      type: OpenStackControlPlaneSwiftReady

(https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openstack-k8s-operators_openstack-operator/352/pull-ci-openstack-k8s-operators-openstack-operator-main-openstack-operator-build-deploy/1671098647317581824/artifacts/openstack-operator-build-deploy/openstack-k8s-operators-gather/artifacts/api-resources/openstackcontrolplanes.yaml)
Looking at the Swift CR itself, you can see that its Ready condition is indeed not True:
    - lastTransitionTime: "2023-06-20T10:55:35Z"
      message: Setup started
      reason: Requested
      status: Unknown
      type: Ready

(https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openstack-k8s-operators_openstack-operator/352/pull-ci-openstack-k8s-operators-openstack-operator-main-openstack-operator-build-deploy/1671098647317581824/artifacts/openstack-operator-build-deploy/openstack-k8s-operators-gather/artifacts/api-resources/swifts.yaml)
I looked at the Swift operator code itself and noticed that it does not follow the typical reconcile pattern we have used in the other Podified operators.  But the particular issue here is that it never sets the Ready condition other than to Unknown during initialization.  We'll need to update the operator to at least set this to True when appropriate and then perhaps follow-up beyond that to make the operator conform to the Podified reconcile pattern in general.

The Swift resource should be fine now, according to https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openstack-k8s-operators_openstack-operator/352/pull-ci-openstack-k8s-operators-openstack-operator-main-openstack-operator-build-deploy/1671204011975380992/artifacts/openstack-operator-build-deploy/openstack-k8s-operators-gather/artifacts/api-resources/swifts.yaml
However, it looks like Cinder is now failing: https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openstack-k8s-operators_openstack-operator/352/pull-ci-openstack-k8s-operators-openstack-operator-main-openstack-operator-build-deploy/1671204011975380992/artifacts/openstack-operator-build-deploy/openstack-k8s-operators-gather/artifacts/api-resources/cinders.yaml
The openstack-operator-crc-podified-edpm-deployment because SwiftStorage tries to setup a PVC using the storageClass "local-storage" and it looks like only crc-csi-hostpath-provisioner is available.

/retest-required

/retest

/retest

recheck

recheck


Feat: limit / specify number of rows to display

Checks

[X] I have checked that this enhancement has not already been requested

How would you categorize this request.  You can select multiple if not sure
Display (is this related to visual display of a value)
Enhancement Description
As far as I understand, currently Data Frames are displayed in their entirety up to 10k rows, after which they are sampled to 10k rows and displayed.
This request is looking for argument to DFViewer, or wherever makes the most sense, to limit the number of rows displayed to some n, where 10k > n >1.
While I understand that its possible to call DFViewer(df.head(10)) to only display 10 rows, this also only provides summary stats over those 10 rows. This request is looking for some behavior like below:
DFViewer(df, max_rows=10)  # only displays 10 rows, show summary stats over entire / sampled df

If this is already possible my apologies.
Appreciative of this great tool!
Pseudo Code Implementation
NA
Prior Art
NA

Other than the ellipsis row this should do what you want.   I'd need to think a bit about how to accommodate  an ellipsis row.  You could just do values, but really you want a row with different styling, which requires a separate release for frontend mods.

How has this solution been working for you?

Closing because of no further comment


Reference to Kimball 2nd Edition -> 3rd Edition

I think the reference to Kimballs book should note that there is a 3rd edition (2013) unless there is a good reason to refer to the older one.
Suggestion: The Data Warehouse Toolkit: The Definitive Guide to Dimensional Modeling (3rd Edition 2013) by Ralph Kimball and Margy Ross

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 470ca67d-ed62-fbe0-85b2-6b1604f9f946
Version Independent ID: e8c82d66-ad63-3efa-11b2-060b9b16d4ad
Content: Understand star schema and the importance for Power BI - Power BI
Content Source: powerbi-docs/guidance/star-schema.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @peter-myers
Microsoft Alias: v-pemyer

Thanks for your comment, @kaysauter . We’ve forwarded your feedback to the writer, who should respond to this thread shortly. Thanks!

Hi @kaysauter - thanks for pointing out the older book reference. I've updated the article today. #please-close


"can't detect preamble" error with React plugin

[x] I have tried upgrading by running bundle update vite_ruby.
[x] I have read the troubleshooting section before opening an issue.

Description 📖
Hello! I'm playing with Vite Rails, trying to create a Vite Rails + React demo app. The problem is that I'm getting the following error:
Uncaught Error: @vitejs/plugin-react can't detect preamble. Something is wrong. See https://github.com/vitejs/vite-plugin-react/pull/11#discussion_r430879201

The error is reported in the readme of VIte Rails plugin, suggesting that:

In middleware mode, you should make sure your entry index.html file is transformed by Vite

But frankly I know nothing about Vite so my debugging stopped here :D
Reproduction 🐞

Clone the folowing repository: https://github.com/mdesantis/rails-7-demo-vite
bin/setup
foreman start -f Procfile.dev
visit http://localhost:5100/hello_react

Vite Ruby Info
Run bin/rake vite:info and provide the output:
bin/vite present?: true
vite_ruby: 3.0.3
vite_rails: 3.0.1
rails: 7.0.0.rc1
node: v14.17.5
npm: 8.0.0
yarn: 1.22.15
pnpm:
ruby: ruby 3.0.2p107 (2021-07-07 revision 0db68f0233) [x86_64-linux]

installed packages:
rails-7-demo-vite@ /home/maurizio/Sviluppo/rails-7/rails-7-demo-vite
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
└──<EMAIL_ADDRESS>

Screenshots 📷

As mentioned in the Troubleshooting section, when using Vite Ruby the entrypoints are JS files and stylesheets instead of an index.html file, it's necessary to add the preamble manually when using @vitejs/plugin-react.
To make the integration simpler, Vite Ruby provides a vite_react_refresh_tag to inject the preamble, which you should place in your layout.
I might asd

@ElMassimo got it, I didn't notice that section of the guide. Thanks!


Add support for internationalized blog-posts

Currently it is a bit unclear how to implement internationalized blog-posts from markdown-files in the starter. Would it be possible to add it to the starter or add in the readme how to implement it?

Well, I have never personaly tried to created any blog using md,but I'm guessing it is the same as when you created normal pages but you use graphql to query data from md files. Not sure if it is the scope of the starter, it was meant to be a basic building block for any internationalization, not spefic to blog etc.

But will take a look at it and consider adding :)


feat: add new configuration "sync-interval" which controls the HTTP polling interval

…, in minutes, bet. each HTTP_sync calls
EDIT by @toddbaert : I have modified this PR. I've done a couple things differently than @OsamaNabih :

interval is in seconds (I think it's likely some people will want sub-minute behavior, and the existing behavior was seconds
using sources for this, as @james-milligan suggested:

for example: ./bin/flagd start --sources='[{"uri":"https://raw.githubusercontent.com/open-feature/playground/main/config/flagd/flags.json","provider":"http","interval":1}]' --debug will poll every 1s

defaults to 5s to maintain current behavior
added tests

Hi @OsamaNabih, thanks for the PR!
I reckon configuration may be a good contender to be passed under the sync-provider-args flag. The sync provider args are passed to all sync providers and are a slice of key value pairs separated by =, e.g. []string{"foo=bar","http-sync-interval-minutes=5"}. My reasoning for this is to prevent us bloating the number of start command flags, wrapping all configuration for the sync providers under a single flag.
Let me know what you think

Thanks @OsamaNabih ! This is good work and definitely something we want done.
I've converted this to a draft for now, mainly because we need to figure out how we want to pass individual arguments to sync providers in flagd (see issue).

I will handling rebasing this PR and getting the idea of it into main, it's a good thing to have.

@toddbaert  I think this is way outdated and we will need a revisit 😞

Hey everyone, are there any updates on this topic?

@OsamaNabih @MilosBabicN26 @Kavindu-Dodan oof yes. This got way out of date waiting for related changes.
I will take a look at this today to see what can be done to import this change.

I have modified this PR. I've done a couple things differently than @OsamaNabih :

interval is in seconds (I think it's likely some people will want sub-minute behavior, and the existing behavior was seconds
using sources for this, as @james-milligan suggested:

for example: ./bin/flagd start --sources='[{"uri":"https://raw.githubusercontent.com/open-feature/playground/main/config/flagd/flags.json","provider":"http","interval":1}]' --debug will poll evety 1s

defaults to 5s to maintain current behavior
added tests

Thank you @toddbaert!

Release with https://github.com/open-feature/flagd/pull/780


Incremental java compilation fails when renaming classname with $ character

Issue occurs since Gradle 7.3. Tested with OpenJDK 11 and 13. Tiny reproduction sample is here: https://github.com/horseunnamed/Gradle-change-detection-issue
Expected Behavior
Incremental java compilation is successful when classname with $ is renamed (as it is with Gradle 7.2 and earlier versions)
Current Behavior
After changing java class name with $ character and running incremental build we get the error:
> Task :compileJava FAILED
Caching disabled for task ':compileJava' because:
  Build cache is disabled
Task ':compileJava' is not up-to-date because:
  Input property 'stableSources' file .../src/main/java/issue/Class$Name.java has been removed.
  Input property 'stableSources' file .../src/main/java/issue/Main.java has changed.
  Input property 'stableSources' file .../src/main/java/issue/Class$NameChanged.java has been added.
Created classpath snapshot for incremental compilation in 0.0 secs.
Compiling with toolchain '/Library/Java/JavaVirtualMachines/openjdk-11.0.2.jdk/Contents/Home'.
Compiling with JDK Java compiler API.
.../src/main/java/issue/Main.java:5: error: cannot find symbol
        new Class$NameChanged();
            ^
  symbol:   class Class$NameChanged
  location: class Main
1 error
Incremental compilation of 3 classes completed in 0.012 secs.
:compileJava (Thread[Execution worker for ':',5,main]) completed. Took 0.02 secs.

FAILURE: Build failed with an exception.

Context
On our real world Android-project we had a very harmful impact from this issue when upgrading Gradle version to 7.3.1.
Incremental annotation processor in our project generates some files with dollar sign and this issue silently led us to ClassNotFoundException at runtime in some cases.
It's very serious blocker for us to upgrade from 7.2.
Steps to Reproduce
Please see reproduction sample: https://github.com/horseunnamed/Gradle-change-detection-issue

Run build task to compile java project with classname that conatins dollar sign (e.g. Class$Name.java) and reference to it somewhere else (from Main.java in my reproduction sample)
Change classname and reference to it with something that still contains dollar sign
Rerun build task
Get mentioned incremental compilation error

Your Environment
Build scan of failed incremental build from reproduction sample: https://scans.gradle.com/s/b4qlo2umn7dms

@horseunnamed PR with the fix will be merged soon. Can you maybe share the name of the annotation processor? Is this some popular annotation processor in the Android world?
We would like to find out how many users might be affected by this issue.

@asodja, thanks for fixing this issue so quickly!
On real project we faced this issue with annotation processor from Toothpick library.
I also have a more complex reproducer with annotation processor from Toothpick. It's very close to our real project issue. See README from this sample for details.

Thank you for that info. Note that the fix is already on the latest nightly. Can you maybe try it and confirm that that fixes the issue? Here is the info how you can update the wrapper: https://gradle.org/nightly/

I've tried to reproduce this issue on our real project with latest nightly build and can approve that issue was fixed :)
Thanks again!


[question][Documents] RunValidators is set by default?

Hello, this is not an issue, just a question about README. Seem that I don't understand it correctly. Could some guys can explain it, please?
In README, there's an paragraph:
params.mongoose
When making a service method call, params can contain a mongoose property which allows you to modify the options used to run the Mongoose query. Normally, this will be set in a before hook:
app.service('messages').hooks({
  before: {
    patch(context) {
      // Set some additional Mongoose options
      // The adapter tries to use sane defaults
      // but they can always be changed here
      context.params.mongoose = {
        runValidators: true,
        setDefaultsOnInsert: true
      }
    }
  }
});

And here is my concern. With this docs, does it mean runValidators is set to true by default?

runValidators is set to true by default. It was set here:
https://github.com/feathersjs-ecosystem/feathers-mongoose/pull/87/files#diff-480db08609ef6a51bf20f049cc9bda48R143
and you can view the defaults used in the update operation here:
https://github.com/feathersjs-ecosystem/feathers-mongoose/blob/5d956bc52eea93b57cb9d57bf2cbecd1fb6f4f1f/lib/service.js#L202
and here it is in patch:
https://github.com/feathersjs-ecosystem/feathers-mongoose/blob/5d956bc52eea93b57cb9d57bf2cbecd1fb6f4f1f/lib/service.js#L252
Hope this helps!

@jamesjnadeau Thank you very much. In source code, it 's set by default, I just concern about documents.
When I read the documents, I think it's not set by default. This option is disable from [mongoose](runValidators is off by default from mongoose, so I'm very surprise while seeing this option in query.
Do you think documents should mention runValidators: true is default for update/patch? It helps someone has bad English like me to understand clearly

Maybe? From my understanding the current functionality works like other feathers adapter. I personally think having runValidators set to true is a good choice.
Feel free to submit a pull request with the updated docs.

@jamesjnadeau I submit this PR.
Thanks for helping!


@custom-media

Hi,
I cannot get the @custom-media variables to get recognized.
Here is my implementation:
http://<IP_ADDRESS>/
It works when @media queries set literal dimensions, like this:
@media (min-width: 768px)

@custom-media is not yet supported in all browsers. The Solved by Flexbox site uses PostCSS to process these into currently usable CSS.


Better handle corrupted videos

Fixes https://github.com/pytorch/vision/issues/1271 and https://github.com/pytorch/vision/issues/1327
This PR adds a try: except guards to the locations where we access the file. This should make it more robust to corrupted videos, as before we were only handling errors during seek, and now he handle it both during container creation (i.e., if the header is corrupted, as in https://github.com/pytorch/vision/issues/1271) or when parts of the file are corrupted (as in https://github.com/pytorch/vision/issues/1327).
cc @bjuncek @stephenyan1231

Codecov Report

Merging #1463 into master will increase coverage by 0.06%.
The diff coverage is 78.04%.

@@            Coverage Diff             @@
##           master    #1463      +/-   ##
==========================================
+ Coverage   64.08%   64.14%   +0.06%
==========================================
  Files          80       80
  Lines        6328     6342      +14
  Branches      973      975       +2
==========================================
+ Hits         4055     4068      +13
  Misses       1986     1986
- Partials      287      288       +1

Impacted Files
Coverage Δ

torchvision/io/video.py
75.32% <78.04%> (+1.75%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for adding for exception handling logic. Hopefully the pyav video decoder is most robust.


NPM stack level too deep (SystemStackError)

To reproduce, create a package.json with this in it:
{
  "dependencies": {
    "eslint": "2.13.1"
  }
}

Then run npm install and license_finder
You get the following output:
$ license_finder
................./usr/local/lib/ruby/gems/2.4.0/gems/license_finder-3.0.2/lib/license_finder/package_managers/npm_package.rb:41:in `flattened_dependencies': stack level too deep (SystemStackError)
	from /usr/local/lib/ruby/gems/2.4.0/gems/license_finder-3.0.2/lib/license_finder/package_managers/npm_package.rb:41:in `map'
	from /usr/local/lib/ruby/gems/2.4.0/gems/license_finder-3.0.2/lib/license_finder/package_managers/npm_package.rb:41:in `flattened_dependencies'
	from /usr/local/lib/ruby/gems/2.4.0/gems/license_finder-3.0.2/lib/license_finder/package_managers/npm_package.rb:41:in `map'
	from /usr/local/lib/ruby/gems/2.4.0/gems/license_finder-3.0.2/lib/license_finder/package_managers/npm_package.rb:41:in `flattened_dependencies'
	from /usr/local/lib/ruby/gems/2.4.0/gems/license_finder-3.0.2/lib/license_finder/package_managers/npm_package.rb:41:in `map'
	from /usr/local/lib/ruby/gems/2.4.0/gems/license_finder-3.0.2/lib/license_finder/package_managers/npm_package.rb:41:in `flattened_dependencies'
	from /usr/local/lib/ruby/gems/2.4.0/gems/license_finder-3.0.2/lib/license_finder/package_managers/npm_package.rb:41:in `map'
	from /usr/local/lib/ruby/gems/2.4.0/gems/license_finder-3.0.2/lib/license_finder/package_managers/npm_package.rb:41:in `flattened_dependencies'
	 ... 8961 levels...
	from /usr/local/lib/ruby/gems/2.4.0/gems/thor-0.19.4/lib/thor/base.rb:444:in `start'
	from /usr/local/lib/ruby/gems/2.4.0/gems/license_finder-3.0.2/bin/license_finder:5:in `<top (required)>'
	from /usr/local/bin/license_finder:22:in `load'
	from /usr/local/bin/license_finder:22:in `<main>'

This is using the latest version of LicenseFinder (3.0.2). It looks like this issue could have been introduced with 3214741.

Verified this is working for me as of commit '2565e6a'


Parsing crashed on GD file from Breseq 0.26.1

I've just testing genomediff parsing on a GD file that was created using Breseq 0.26.1.
I've executed genomediff on a GD file that we're currently parsing and received the following error:
Python 3.4.0 (default, Jun 19 2015, 14:20:21)
[GCC 4.8.2] on linux
Type "help", "copyright", "credits" or "license" for more information.

from genomediff import *
document = GenomeDiff.read(open('2-1.gd', 'r', encoding='utf-8'))
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python3.4/dist-packages/genomediff/init.py", line 19, in read
for record in GenomeDiffParser(document=gd, fsock=fsock):
File "/usr/local/lib/python3.4/dist-packages/genomediff/parser.py", line 85, in iter
raise Exception('Could not parse line #{}: {}'.format(i, line))
Exception: Could not parse line #12: RA	6	.	NC_000913	4541	0	A	T	bias_e_value=64.2575	bias_p_value=1.38437e-05	fisher_strand_p_value=9.29839e-07	frequency=2.083e-01	genotype_quality=260.6	ks_quality_p_value=1	log10_base_likelihood=-17.0334	log10_qual_likelihood_position_model=19.0126	log10_strand_likelihood_position_model=6.23447	new_cov=20/0	polymorphism_quality=11.7	quality=11.7	quality_position_model=Inf	ref_cov=32/44	reject=POLYMORPHISM_STRAND,FISHER_STRAND_P_VALUE	tot_cov=52/44

This data was generated from FASTQs from the CfB; I'll email you the data shortly.

Hi Patric,
I didn't get a file with your email?
The issue was the '.' for the missing parent ids. We tested with files that had a blank field instead. I released 0.2.1 with a fix. Let me know if you encounter more issues. If you send me a file I will make sure that it works.


ldeploy import issue

Hi Team, I am working on ldeploy POC for my organization and I am pretty excited about this tool. When this tool is productionized, it will definitely add lot of values and benefits for our team.
I am able to successfully export the contents from one instance but when I try to import this to other instance, I keep getting the following error and the dashboard is being created in the new instance with only filters but I don't see any Tiles. Am I missing something here?
ERROR: The resource you're looking for could not be found
Error creating query({
"view": "customer_demo",
"fields": [

I figured out the issue. I was importing the Dashboard to new environment so the model configuration and connection set-up was not there. Once I set-up those, I was able to successfully import the Dashboards into new instance.


amam/gatsby incremental POC

Description
Summary of the change and/or issue that is fixed.
Changes:

change build command on build:inc

Type of change

[ ] Bug fix
[x] New feature
[ ] Refactor code
[ ] Update translation text
[ ] Breaking change
[ ] Dependency update
[ ] This change requires a documentation update
[ ] Release

closing this as this is no longer relevant.


Processing experiments

additional HSDS access methods and example notebooks

@vedina
I suggest changing h5module parameter of .cha related functions to h5_backed: typing.Literal['h5py', 'h5pyd'] = 'h5py'

@vedina I suggest changing h5module parameter of .cha related functions to h5_backed: typing.Literal['h5py', 'h5pyd'] = 'h5py'

no, for the two new functions h5py makes no sense (Folders exist only in h5pyd )
we are not using only h5py and h5pyd - in https://github.com/h2020charisma/charisma-api a custom class is used as h5module parameter, which makes use of api_key parameter (login through keycloak)


Following the tutorial gives me error

This is the tutorial I'm following: https://github.com/prisma/prisma2/blob/master/docs/tutorial.md
I'm using a Heroku server with a postgres db add on. It wouldn't let me create a new schema under the name "hello-prisma2" so I just used the public one.
I can get to the part 2.6. Explore next steps before it fails. This is what I get in my console:
Preparing your demo script (TypeScript) ...

✔  Downloading the demo script from GitHub ...
✔  Extracting content to hello-prisma2 ...
⠧  Installing dependencies with: `npm install` ...
4  Preparing your database ...
 ERROR  Error during command execution

(node:4582) UnhandledPromiseRejectionWarning: Error: ENOENT: no such file or directory, mkdir
(node:4582) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing in
side of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejec
tion id: 1)
(node:4582) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejectio
ns that are not handled will terminate the Node.js process with a non-zero exit code.
Error: Get config Error: Command failed with exit code 13 (EACCES): /Users/joshua/WebstormProjects/my-app/db2/hello-prisma2/node_modules/prisma2/query-engine-darwin cli --get_config {"datamodel":"generator photon {\
n  provider = \"photonjs\"\n}\n\ndatasource db {\n  provider = \"postgresql\"\n  url      =
<EMAIL_ADDRESS>ute-1.amazonaws.com/dc4s65qj9jci71?schema=hello-prisma2&sslmode=prefer\"\n}\n\nmodel User {\n  id    String  @default(
cuid()) @id\n  email String  @unique\n  name  String?\n  posts Post[]\n}\n\nmodel Post {\n  id        String   @defaul
t(cuid()) @id\n  createdAt DateTime @default(now())\n  updatedAt DateTime @updatedAt\n  published Boolean  @default(tr
ue)\n  title     String\n  content   String?\n  author    User?\n}"}

spawn /Users/joshua/WebstormProjects/my-app/db2/hello-prisma2/node_modules/prisma2/query-engine-d
arwin EACCES
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! script@ postinstall: `prisma2 generate`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the script@ postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/joshua/.npm/_logs/2019-10-16T10_49_16_495Z-debug.lo

What is going on here and why am I encountering obstacles in the tutorial? Thanks

Hi, seem related to #668

Yes, looks like a duplicate of #668.
@joshuarobs Can you please tell me how did you install prisma2? What exact command you run and in which OS you are using Prisma.
Maybe comment this on #668 and close this one as a duplicate.

I'm running on MacOS 10.15 Catalina and Node 10.16.3. I tried all sorts of things, in https://github.com/prisma/prisma2/issues/668, including installing with sudo and --unsafe-perm. I even tried what lvauvillier tried, with the link he provided, in zsh, and still it didn't work. I guess there's nothing I can do to make prisma2 work unfortunately.


Dragging and dropping of the flexibility options doesn't work in [IE9]

The library I use is a little to fancy.

I switched to jQuery ui and used the sortable module.
It still contains the same problem. IE9 doesn't finish the Ajax request.
https://github.com/quintel/etmodel/blob/master/app/assets/javascripts/lib/views/base_chart_view.coffee#L108
Doesn't it have something to do with the fact that we use api_url always instead of the proxy_url ?

@antw Doesn't it have something to do with the fact that we use api_url always instead of the proxy_url ?

Yes. I found that you should use App.api.path instead; it will return the correct path depending on what the browser supports:
App.api.path('hello')
// => "http://www.et-engine.com/api/v3/hello"

Fixed by: https://github.com/quintel/etmodel/commit/32e4f593cd35378aa77865c3437aaee86c17530e


Show email, phone, and recovery options for MFA verification

What does this PR do?
Show the appropriate text and buttons depending on what factors are
enabled and what challenge has been created.
Test Plan
Only authenticator

Only email

Only phone

Authenticator and email
Starting

After clicking on Email verification

After clicking on Use recovery code

Related PRs and Issues

https://github.com/appwrite/appwrite/issues/7845

Have you read the Contributing Guidelines on issues?
Yes

Converting to draft because recovery codes are no longer just 6 digits...

Updates after design feedback:
Desktop

Mobile

Fixed mobile view:


New LA categories and AmlComputeClusterNodeEvent

Added more Log Analytics categories with definitions and info about AmlComputeClusterNodeEvent deprecation

@eldarku : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@eldarku : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

@Blackmist  Can you please review it?
Thank you!

#sign-off

@Blackmist: I'm sorry - only the author of this article, @aashishb, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Fix transparent viewport backgrounds

This fixes #79778. This code change zeroes out the alpha component of the clear color. This basically just ports a change made in #61109 from the Forward+ renderer to the Mobile renderer, as well as an equivalent change to the Compatibility renderer.

I went ahead and made the change to the PR for the Compatibility renderer. I tested it, and it is working, but with a caveat: The viewport's transparency doesn't seem to update until you reload the scene. I'm not sure why, but I'll look into it.

For the Compatibility renderer, I changed if (render_data.transparent_bg) to if (rt->is_transparent), which resolved the issue of the background transparency not updating until the scene was reloaded. I don't really understand why the two are different, but the change worked.
I also made another change to force consistency between the renderers. Right now, setting an alpha on the custom clear color is ignored by the Forward+ and Mobile renderers. However, the alpha of the clear color was used when clearing the viewport in the Compatibility renderer, even with the viewport set to use an opaque background. I decided to force the alpha of the clear color to be 1 when the render target is not transparent to make the Compatibility renderer's behavior match the other renderers.

From what I can tell, RenderSceneBuffersGLES3::is_transparent is redundant, since it's just duplicating GLES3::RenderTarget::is_transparent, and trying to keep them in sync looks to be a problem. I added a commit (not an amended commit since it might want to be approved / declined separately, and wanted to keep an easy way to undo it) that removes RenderSceneBuffersGLES3::is_transparent entirely, which turned out to be fairly simple. This directly addresses the issue I had when making this work with the Compatibility renderer, as well as addressing any other part of the Compatibility renderer that depended on render_data.transparent_bg being accurate.

From what I can tell, RenderSceneBuffersGLES3::is_transparent is redundant, since it's just duplicating GLES3::RenderTarget::is_transparent, and trying to keep them in sync looks to be a problem. I added a commit (not an amended commit since it might want to be approved / declined separately, and wanted to keep an easy way to undo it) that removes RenderSceneBuffersGLES3::is_transparent entirely, which turned out to be fairly simple. This directly addresses the issue I had when making this work with the Compatibility renderer, as well as addressing any other parts of the Compatibility renderer that depended on render_data.transparent_bg being accurate.

Thanks for looking into that more deeply. Indeed it makes sense to remove is_transparent from render buffers. That was an artifact from before we merged RT and RB.
Go ahead and squash the commits together, then we can merge this!

Also needs a rebase now :)

Thanks!

Cherry-picked for 4.1.4.


[in_app_purchase] Is possible to check active subscription on iOS

When it comes to Android, we can query PastPurchases and Get the Active Subscription based on Past Purchases is empty or not, but on IOS is there any way to get the active and valid subsctiption?

Hi @AathifMahir
I see there's an open issue addressing the case you described.
Please follow up on that issue, Closing as duplicate of https://github.com/flutter/flutter/issues/61339
If you disagree, please write in the comments and I will reopen it.
Thank you


Bug with simple fix after using code from PyPi site

I was trying to use the code you put on your PyPI page and I got an error: AttributeError: 'DataFrame' object has no attribute 'append'. Did you mean: '_append'?
the error was leading to code line 176 in HistoricalData.py and I found that if you change the line to:
data = pd.concat([dataset])
it solves the error. maybe you should look into this error a little bit and if possible fix it because I'm sure many people would try your library, see it doesn't work, and give up and look for another library. Thank You in advance

The correct solution is to replace line 176 with this:
data = pd.concat([data,dataset])
If you use the previous suggested solution above, only the LAST chuck of received data will be saved to the dataframe (each chuck will just over-write the dataframe). Hope this helps!


Command to advance a player to at least a certain rank on a track

I'm looking for something sort of like parent settrack so that I can advance players to a specific group on a track. However, I don't want it to blindly set their rank because some players will already have a higher one.
Thanks.

Check this and this

Example usecase?

Specifically, it would work nicely with automatic rank ups from various achievements. Specifically, we use AutoRank to advance players to a member rank after they've played for a few hours. However, that player may have already achieved a higher rank by voting, donating, or whatever other reason.

That sounds like a good excuse to have several ladders.
One for voting, one for donating and one for playing.

I considered something like that, but the groups on each track would just end up inheriting the actual rank anyway because I don't care how they got to that rank. A player would end up inheriting a playtime group that inherits member as well as a voting group that inherits member+, which in turn inherits member. Instead of having players inherit both voter and playtime groups, I may as well just use parent add with the single existing track.

You can use the scoreboard system.  It can track everything you see in the achievements list, then figure out how to run a command when the score for a certain task reaches a specified number.
If you must use a plugin, here is one  for spigot and bukkit, or here is one if you're on sponge


Incorrect Descriptions of URI Types

The descriptions are incorrect:
0 = User URI
1 = Phone URI
Kindly check and let me know if I am correct. I can submit a PR if confirmed.
--

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 404edb8b-aaf7-23af-ad56-f5e1aeb80121
Version Independent ID: 2cd376ca-cb29-0604-cd8c-2a96f714545d
Content: UriTypes table
Content Source: Skype/SfbServer/schema-reference/call-detail-recording-cdr-database-schema/uritypes.md
Service: unspecified
Product: skype-for-business-itpro
GitHub Login: @SerdarSoysal
Microsoft Alias: serdars

Verified the necessary changes, publishing  later today

Thanks. :)

fixed as part of
https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness-pr/pull/1046#issuecomment-394461962


No SUI option to clear "recentCommands"

Windows Terminal version (or Windows build number)
Windows Terminal Preview Version: 1.12.2922.0
Other Software
Windows 10 version 10.0.19043.0
Steps to reproduce

open command palette
backspace '>' then enter commands
close WT
open command palette

Expected Behavior

option to delete recent commands

Actual Behavior

no SUI option to clear recent commands
the only option is to edit or delete 'state.json'

Sample screenshot:

This brings up a bigger point - we can't really clear out any app state in the SUI yet. You'd have to manually delete the entries from state.json.
I suppose the things we're storing right now (and possibly in the future):

recent commands in command palette
window state (can be cleared by turning the setting on/off
dynamic profiles that we've seen before?
allowed commandlines (https://github.com/microsoft/terminal/projects/5#card-67023791)
"Find" history

So there probably should be a separate page for these things? This may need some thought as to the right way to handle.


modify welcome message

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign ruwbdy
You can assign the PR to them by writing /assign @ruwbdy in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[jx-info] Hi, we've detected that the pipelines in this repository are using a syntax that will soon be deprecated.
We'll continue to update you through PRs as we progress. Please check #8589 for further information.

:star: PR built and available in a preview jx-lfs268-ruwbdy-node-lab-02-pr-1 here

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Ruwbdy
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Ruwbdy]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/approve