A nasty bug possibly introduced by some commit after 1.3.1

Hello guys, thanks again for the help with issue #836! Your support here is really top-notch.
I've stumbled upon a pretty interesting finding.
A bit of story for context first, though:
After getting past that problem by building from source the version of j2obc at this point in history and using it to translate my project, I realized that, unfortunately, I still got problems with the translation process.
This is the issue I got, pretty nasty IMHO as it was pretty hard to track down what caused it:
$ ./j2objc_translate
Obtaining Java sources list...
Preparing to translate to iOS-ready Objective-C sources...
Translating using classpath: lib/guava-14.0.1-sources.jar:lib/slf4j-api-1.7.13-sources.jar:src.

Exception in thread "main" com.sun.tools.javac.util.Abort
	at com.sun.tools.javac.comp.Check.completionError(Check.java:287)
	at com.sun.tools.javac.comp.Attr.attribTree(Attr.java:584)
	at com.sun.tools.javac.comp.Attr.attribType(Attr.java:638)
	at com.sun.tools.javac.comp.Attr.attribType(Attr.java:631)
	at com.sun.tools.javac.comp.Attr.visitTypeApply(Attr.java:3911)
	at com.sun.tools.javac.tree.JCTree$JCTypeApply.accept(JCTree.java:2135)
	at com.sun.tools.javac.comp.Attr.attribTree(Attr.java:576)
	at com.sun.tools.javac.comp.Attr.attribType(Attr.java:638)
	at com.sun.tools.javac.comp.Attr.attribType(Attr.java:631)
	at com.sun.tools.javac.comp.Attr.attribBase(Attr.java:786)
	at com.sun.tools.javac.comp.MemberEnter.complete(MemberEnter.java:1057)
	at com.sun.tools.javac.code.Symbol.complete(Symbol.java:574)
	at com.sun.tools.javac.code.Symbol$ClassSymbol.complete(Symbol.java:1037)
	at com.sun.tools.javac.comp.Enter.complete(Enter.java:493)
	at com.sun.tools.javac.comp.Enter.main(Enter.java:471)
	at com.sun.tools.javac.main.JavaCompiler.enterTrees(JavaCompiler.java:982)
	at com.sun.tools.javac.api.JavacTaskImpl.enter(JavacTaskImpl.java:316)
	at com.sun.tools.javac.api.JavacTaskImpl.analyze(JavacTaskImpl.java:359)
	at com.sun.tools.javac.api.JavacTaskImpl.analyze(JavacTaskImpl.java:344)
	at com.google.devtools.j2objc.javac.JavacParser.parseFiles(JavacParser.java:160)
	at com.google.devtools.j2objc.pipeline.FileProcessor.processBatch(FileProcessor.java:141)
	at com.google.devtools.j2objc.pipeline.FileProcessor.processInputs(FileProcessor.java:70)
	at com.google.devtools.j2objc.pipeline.TranslationProcessor.processInputs(TranslationProcessor.java:77)
	at com.google.devtools.j2objc.J2ObjC.run(J2ObjC.java:128)
	at com.google.devtools.j2objc.J2ObjC.main(J2ObjC.java:178)

The stacktrace gave no clue about which portion of my project that exactly caused it, as opposed to issue #836's situation.
Googling wouldn't help me much. The closest I got to finding someone facing the exact same problem was this thread. I don't even know if they really had the same problem, I didn't  bother looking to much into their case as it seems to be a completely unrelated project.
Eventually I could narrow down the problem a bit more... noticed that it would happen when I tried to include guava-sources.jar version 19.0 into my -sourcepath. So I thought that maybe it was a problem with the version I was using, and I got evidence to support my hyphothesis when I found this j2objc-related project including a subtly customized version of guava.
But using it still wouldn't help me. The problem apparently still happened while j2objc attempted at translating the guava sources (or something along those lines).
So my interesting finding is this: when I decided to give it a try at using the distributed j2objc-1.3.1 version to translate the exact same project, it actually worked!! So probably some commit after the distributed 1.3.1 version is causing this issue.
You guys can use this repository for verifying the issue on your own. If you decide to test it using the j2objc_translate script I provided you might want to have your $PATH point to the latest version of j2objc on the master branch and also edit the file to point the proper absolute path of your j2objc-1.3.1 tool.
Thanks again for all the help!
PS: Something I might try at some point is a git-bisect for finding which commit seems to be the guilty party. I would be a fun occasion to actually learn how it works in practice :)
Cheers!

Before we jump into this too deeply, there are a few things you could try:
Since 1.3.1 we have switched our default parsing front end from JDT to Javac (here). You can toggle between the two using -Xuse-jdt and -Xuse-javac or by setting an environment variable J2OBJC_FRONT_END=JDT or =JAVAC. Switching back to JDT will probably unblock you for now, but we plan to get rid of JDT eventually so we should make sure Javac works for you.
Since the error is coming from Javac, it's very likely that you would get the same error when using javac to compile the same code. Try switching "j2objc" with "javac" in your command line and see if you get the same error. (When building locally, j2objc will be using the same version of javac that is on your machine, taken from $JAVA_HOME/lib/tools.jar)
I believe that Guava 19 is the last version written in Java7. Try toggling between -source 7 and -source 8 and see if either of those help.

guava-14.0.1-sources.jar? Please consider upgrading, as this release is
almost 4 years old. As Keith said, it's likely that the version of javac on
your system cannot compile it (though do try the -source 1.7 suggestion
first).
On Thu, Feb 23, 2017 at 7:22 AM Keith Stanger<EMAIL_ADDRESS>wrote:

Before we jump into this too deeply, there are a few things you could try:
Since 1.3.1 we have switched our default parsing front end from JDT to
Javac (here
https://github.com/google/j2objc/commit/02db36ef3f23a38e9487705951a06e0ad4311999).
You can toggle between the two using -Xuse-jdt and -Xuse-javac or by
setting an environment variable J2OBJC_FRONT_END=JDT or =JAVAC. Switching
back to JDT will probably unblock you for now, but we plan to get rid of
JDT eventually so we should make sure Javac works for you.
Since the error is coming from Javac, it's very likely that you would get
the same error when using javac to compile the same code. Try switching
"j2objc" with "javac" in your command line and see if you get the same
error. (When building locally, j2objc will be using the same version of
javac that is on your machine, taken from $JAVA_HOME/lib/tools.jar)
I believe that Guava 19 is the last version written in Java7. Try toggling
between -source 7 and -source 8 and see if either of those help.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/j2objc/issues/838#issuecomment-282021663, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AC2kryghpYntmKXi7tOdDYvj_U9po-XOks5rfaQYgaJpZM4MKD5m
.

Thank you all for the quick responses. So, I did as you suggested, got back to using Guava 19, and attempted at translating with the -source flags set first -- it yields the same results.
(Weirdly enough, I didn't get quite the exact same errors using javac directly, but it didn't work nonetheless.)
But Keith was spot on in his suggestion of swapping the front end. Using JDT really did work as expected! So I'll go ahead and experiment trying out the next steps. Please et me know if you need anything from me regarding this issue.

You might consider updating your java installation if it is older than Guava 19. I'd be interested to see the errors you get from javac.

I do have a up-to-date javac version, though.. let me know if I'm doing something wrong. When I try running it:
#!/bin/zsh

CLASSPATH="$(for i in lib/*.jar ; do echo -n $i: ; done)src"

echo "Obtaining Java sources list..."
find . \! -name ".java" | grep \\.java > java_sources.list
echo "Preparing to translate to iOS-ready Objective-C sources..."
mkdir -p j2objc_gen/src
echo "Translating using classpath: `echo $CLASSPATH`."
javac -d j2objc_gen/src -sourcepath "$CLASSPATH" @java_sources.list
echo "Done translating."
rm java_sources.list

The following errors pop up everywhere (I tried to filter repeated occurrences of the same kinds of errors):
Obtaining Java sources list...
Preparing to translate to iOS-ready Objective-C sources...
Translating using classpath: lib/guava-19.0-sources.jar:lib/jsr305-1.3.9.jar:lib/slf4j-api-1.7.13-sources.jar:src.
lib/guava-19.0-sources.jar(com/google/common/eventbus/SubscriberRegistry.java):38: error: package com.google.j2objc.annotations does not exist
import com.google.j2objc.annotations.Weak;
                                    ^
lib/guava-19.0-sources.jar(com/google/common/eventbus/Subscriber.java):22: error: package com.google.j2objc.annotations does not exist
import com.google.j2objc.annotations.Weak;
                                    ^

[...]
lib/guava-19.0-sources.jar(com/google/common/collect/AbstractMapBasedMultimap.java):354: error: cannot find symbol
  @WeakOuter
   ^
  symbol:   class WeakOuter
  location: class AbstractMapBasedMultimap<K,V>
  where K,V are type-variables:
    K extends Object declared in class AbstractMapBasedMultimap
    V extends Object declared in class AbstractMapBasedMultimap
lib/guava-19.0-sources.jar(com/google/common/collect/Maps.java):2800: error: cannot find symbol
    @WeakOuter
     ^
  symbol:   class WeakOuter
  location: class FilteredEntryMap<K,V>
  where K,V are type-variables:
    K extends Object declared in class FilteredEntryMap
    V extends Object declared in class FilteredEntryMap
lib/guava-19.0-sources.jar(com/google/common/collect/Maps.java):2834: error: cannot find symbol
    @WeakOuter
     ^
  symbol:   class WeakOuter
  location: class FilteredEntryMap<K,V>
  where K,V are type-variables:
    K extends Object declared in class FilteredEntryMap
    V extends Object declared in class FilteredEntryMap
lib/guava-19.0-sources.jar(com/google/common/collect/Maps.java):2910: error: cannot find symbol
    @WeakOuter
     ^
  symbol:   class WeakOuter
  location: class FilteredEntrySortedMap<K,V>
  where K,V are type-variables:
    K extends Object declared in class FilteredEntrySortedMap
    V extends Object declared in class FilteredEntrySortedMap
lib/guava-19.0-sources.jar(com/google/common/collect/ImmutableMap.java):536: error: cannot find symbol
  @WeakOuter
   ^

[...]
  symbol:   class WeakOuter
  location: class ImmutableMultiset<E>
  where E is a type-variable:
    E extends Object declared in class ImmutableMultiset
lib/guava-19.0-sources.jar(com/google/common/cache/LocalCache.java):4495: error: cannot find symbol
  @WeakOuter
   ^

[...]
  symbol:   class WeakOuter
  location: class LocalCache<K,V>
  where K,V are type-variables:
    K extends Object declared in class LocalCache
    V extends Object declared in class LocalCache
lib/guava-19.0-sources.jar(com/google/common/util/concurrent/CollectionFuture.java):26: error: package com.google.j2objc.annotations does not exist
import com.google.j2objc.annotations.WeakOuter;
                                    ^
lib/guava-19.0-sources.jar(com/google/common/util/concurrent/AggregateFuture.java):27: error: package com.google.j2objc.annotations does not exist
import com.google.j2objc.annotations.WeakOuter;
                                    ^
lib/guava-19.0-sources.jar(com/google/common/util/concurrent/AggregateFuture.java):90: error: cannot find symbol
  @WeakOuter
   ^
  symbol:   class WeakOuter
  location: class AggregateFuture<InputT,OutputT>
  where InputT,OutputT are type-variables:
    InputT extends Object declared in class AggregateFuture
    OutputT extends Object declared in class AggregateFuture
lib/guava-19.0-sources.jar(com/google/common/util/concurrent/CollectionFuture.java):38: error: cannot find symbol
  @WeakOuter
   ^
  symbol:   class WeakOuter
  location: class CollectionFuture<V,C>
  where V,C are type-variables:
    V extends Object declared in class CollectionFuture
    C extends Object declared in class CollectionFuture
lib/guava-19.0-sources.jar(com/google/common/collect/AbstractMapBasedMultimap.java):614: error: cannot find symbol
  @WeakOuter
   ^

[...]
  symbol:   class WeakOuter
  location: class AbstractMapBasedMultimap<K,V>
  where K,V are type-variables:
    K extends Object declared in class AbstractMapBasedMultimap
    V extends Object declared in class AbstractMapBasedMultimap
lib/guava-19.0-sources.jar(com/google/common/base/AbstractIterator.java):31: error: duplicate class: com.google.common.base.AbstractIterator
abstract class AbstractIterator<T> implements Iterator<T> {
         ^
lib/guava-19.0-sources.jar(com/google/common/collect/Collections2.java):490: error: cannot access AbstractIterator
  private static final class OrderedPermutationIterator<E> extends AbstractIterator<List<E>> {
                                                                   ^
  bad source file: lib/guava-19.0-sources.jar(com/google/common/collect/AbstractIterator.java)
    file does not contain class com.google.common.collect.AbstractIterator
    Please remove or make sure it appears in the correct subdirectory of the sourcepath.
lib/guava-19.0-sources.jar(com/google/common/eventbus/SubscriberRegistry.java):71: error: cannot find symbol
  @Weak private final EventBus bus;
   ^
  symbol:   class Weak
  location: class SubscriberRegistry
lib/guava-19.0-sources.jar(com/google/common/eventbus/Subscriber.java):51: error: cannot find symbol
  @Weak private EventBus bus;
   ^
  symbol:   class Weak
  location: class Subscriber
lib/guava-19.0-sources.jar(com/google/common/cache/LocalCache.java):2014: error: cannot find symbol
    @Weak final LocalCache<K, V> map;
     ^
  symbol:   class Weak
  location: class Segment<K,V>
  where K,V are type-variables:
    K extends Object declared in class Segment
    V extends Object declared in class Segment
lib/guava-19.0-sources.jar(com/google/common/collect/Maps.java):3794: error: cannot find symbol
    @Weak final Map<K, V> map;
     ^
  symbol:   class Weak
  location: class Values<K,V>
  where K,V are type-variables:
    K extends Object declared in class Values
    V extends Object declared in class Values
lib/guava-19.0-sources.jar(com/google/common/collect/Maps.java):3614: error: cannot find symbol
    @Weak final Map<K, V> map;
     ^
  symbol:   class Weak
  location: class KeySet<K,V>
  where K,V are type-variables:
    K extends Object declared in class KeySet
    V extends Object declared in class KeySet
lib/guava-19.0-sources.jar(com/google/common/collect/ImmutableSetMultimap.java):433: error: cannot find symbol
    @Weak private final transient ImmutableSetMultimap<K, V> multimap;
     ^
  symbol:   class Weak
  location: class EntrySet<K,V>
  where K,V are type-variables:
    K extends Object declared in class EntrySet
    V extends Object declared in class EntrySet
lib/guava-19.0-sources.jar(com/google/common/collect/ImmutableMultimap.java):517: error: cannot find symbol
    @Weak final ImmutableMultimap<K, V> multimap;
     ^
  symbol:   class Weak
  location: class EntryCollection<K,V>
  where K,V are type-variables:
    K extends Object declared in class EntryCollection
    V extends Object declared in class EntryCollection
lib/guava-19.0-sources.jar(com/google/common/collect/ImmutableMultimap.java):660: error: cannot find symbol
    @Weak private final transient ImmutableMultimap<K, V> multimap;
     ^
  symbol:   class Weak
  location: class Values<K,V>
  where K,V are type-variables:
    K extends Object declared in class Values
    V extends Object declared in class Values
lib/guava-19.0-sources.jar(com/google/common/cache/LocalCache.java):4453: error: cannot find symbol
    @Weak final ConcurrentMap<?, ?> map;
     ^
  symbol:   class Weak
  location: class LocalCache<K,V>.AbstractCacheSet<T>
  where T,K,V are type-variables:
    T extends Object declared in class LocalCache.AbstractCacheSet
    K extends Object declared in class LocalCache
    V extends Object declared in class LocalCache
Done translating.

It does feel like I'm missing out on something simple.

The current version of Guava depends on j2objc-annotations, so add a path to the j2objc_annotations.jar file in your j2objc distribution. The good news is that this dependency means that Guava leaks much less on iOS than in earlier versions.

One problem I see with your script is that it appears to confuse -sourcepath with -classpath. The jar files in guava/lib contain class files (*.class) and so should be put on the class path. Only files or directories that contain Java source files belong on the source path. The j2objc_annotations.jar file should be included in the classpath.
I tried compiling guava-14.0.1 with javac and it failed with a Java 8 JVM, whether -source 1.7 or 1.8 is specified, while it does build with a Java 7 JDK. Since j2objc's jre_emul library is based on Java 8 now, that means that version of Guava cannot be translated by j2objc 1.2 (the first release supporting Java 8).
guava-19.0 (and higher) does build with Java 8, and with j2objc. To get the correct jars, I ran "mvn generate-resources" and copied the classpath jars to a local "jars" directory):
$ javac -d build \
    -classpath jars/jsr305.jar:jars/javax-inject.jar:jars/j2objc_annotations.jar:jars/animal-sniffer-annotations-1.14.jar \
    -sourcepath guava/src `find guava/src -name '*.java'`
$ j2objc -d build \
    -classpath jars/jsr305.jar:jars/javax-inject.jar:jars/j2objc_annotations.jar:jars/animal-sniffer-annotations-1.14.jar \
    -sourcepath guava/src `find guava/src -name '*.java'`


feat: account ID refactor additional methods

This PR adds two small quality of life methods to AccountIdPrefix and BlockHeader. Both stemming from comments in this review.

@tomyrd - could you also add a changelog entry?


Add field and button to do a manual shoutout

If a name does not get said it might be useful to pass a differently spelled name manually or to replay a name that did not get parsed properly.

This is now shipped.


Keep ratio and center avatars

fix bug 725308

looks

whoops forgot to r+

I don't have the rights to merge this. Can you do it?

sure thing


No cited_by data?

Cyril, Gaétan
Great job, I am just looking into it. Have two questions:

Do I understand correctly that the present beta version of dataset contains only bibliographic info on NPL, and no patent identifier? Because I can find none (correct me if I am wrong). That is, it is about what's cited, but not who's citing
Do NPL citations come only from search reports, or do they also include in-text citations? From your e-mail it is not entirely clear
Thanks and congratulations again
Francesco

Dear @leflix311,
thanks for your kind words!

Do I understand correctly that the present beta version of dataset contains only bibliographic info on NPL, and no patent identifier? Because I can find none (correct me if I am wrong). That is, it is about what's cited, but not who's citing

Yes, you can get back to the citing document.
The npl_publn_id in our database is the NPL_PUBLN_ID reported in the Patstat.tls214 table. Using this key, you can get back to the citing patents (aka PAT_PUBLN_ID) using the CITED_NPL_PUBLN_ID in the Patstat.tls212 table.
That being said, this is definitely the kind of data-gym that we want to avoid to the end-user. The database should be self-sufficient. Actually, as this is a 1:n relationship and that we would like to avoid the database to expand dramatically, we are still looking at an efficient and user-friendly way to do it. So, definitely, would you have any suggestion, we will consider it carefully.

Do NPL citations come only from search reports, or do they also include in-text citations? From your e-mail it is not entirely clear

The first release include only front page npl citations, not in-text citations as they are not reported in the DOCDB database. It seems that this is gradually evolving but there is no plan for backfiles. Anyway, from the NPL_PUBLN_ID we can get the CITN_ORIGIN (Patstat.tls212 table) which gives you the origin of the citation (APP, SEA, etc). You are right, this information should also be reported in the final dataset.
That being said, we are also working on in-text patent (AND patent) citations using the same pipeline techno/pipeline for the US. We are currently processing US patent full-texts to extract, parse and consolidate in-text patent and NPL citations. We are also planning to do it for other major patent offices. Would you have any suggestion on the top priorities, let us know.
Hope it helps,
Cyril

Hello @leflix311 ,
I have just updated the beta.
You will now find two additional (nested) columns:

cited_by.origin: origin of the citation (SEA, APP, OPP, etc) where the code refers to PatStat CITN_ORIGIN. I report the codes description below.

APP: citations introduced by the applicant
SEA: citations introduced during search (from Search Report)
ISR: citations from the International Search Report
SUP: citations from the Supplementary Search Report
PRS: "PRe-Search" citations (available before official publication)
EXA: citations introduced during examination
OPP: citations introduced during opposition (citations by opponent published with a European Patent Specification (EP-B2))
APL: citations introduced when filed for appeal by applicant / proprietor / patentee
FOP citations introduced when filed opposition by any third party after the publication of a European Patent Specification (EP- B1)
TPO citations introduced because of Third Party Observations (Art 115 EPC) CH2 citations introduced during the Chapter 2 phase of the PCT

cited_by.publication_number: publication number of citing patents (e.g. EP-0768779-A3)

Does it help?
Cyril

Hello,
This is now streamlined.
You will find the variables defined above with the following schema in all V0+ releases:

Field name
Type
Mode
Description

cited_by
RECORD
REPEATED

cited_by. origin
STRING
REPEATED
Origin of the citation (e.g. APPlicant, SEArch report, etc)

cited_by. publication_number
STRING
REPEATED
DOCDB publication number of citing patent(s)

I close this issue, feel free to re-open it if needed
Cheers


Multilanguage support for wallet selector Modal-UI

Currently the language presented in the present modal for wallet selector is English, this is a limitation for people who use wallet selector and don't speak the language.
They could generate multilanguage support in the modal-ui to be able to approach more users within the different platforms.
Perhaps within the modal configuration or globally you could decide which language to display.
Add support for new languages like Spanish, French or others

Team, we are asking for our spanish-oriented platform called nativonft.app, if you can add support to extra langs we can help you to translate this into spanish!
Spanish is a language with 550 million speakers!
You can reach me on TG if required help on this :) @alan_network

Team, we are asking for our spanish-oriented platform called nativonft.app, if you can add support to extra langs we can help you to translate this into spanish!
Spanish is a language with 550 million speakers!
You can reach me on TG if required help on this :) @alan_network

Hi @cloudmex-alan we are currently working on this, and we will reach out regarding extra help. Thanks!

We're in the final stage so I've reached out on Telegram regarding the help with Spanish translations.

Thanks for the support team!


WIldcard.all support for Tuple

Closes #1870

Instead of providing support for mapping Wildcard.all into tuple we could directly support a Map<String, Object> mapping?
Projections.tuple(Wildcard.all)

sorry, closed accidently

Wildcard.all in QTuple should still binding all the columns of the result set into a single object array. The test should be changed to assert a single a tuple of size with a value of Object[] instead.


feat(testing-mongoose): add replicaSet option to use ReplicaSet mode

Closes: #2001
Information

Type
Breaking change

Feature
No

Usage example
A ReplicaSet can be easily started with:
import {PlatformTest} from "@tsed/common";
import {PlatformExpress} from "@tsed/platform-express";
import {TestMongooseContext} from "@tsed/testing-mongoose";
import {expect} from "chai";
import * as SuperTest from "supertest";
import {Server} from "../Server";

describe("Rest", () => {
  // bootstrap your Server to load all endpoints before run your test
  let request: SuperTest.SuperTest<SuperTest.Test>;

  before(
    TestMongooseContext.bootstrap(Server, {
      platform: PlatformExpress,
      mongod: {
        replicaSet: true
      }
    })
  ); // Create a server with mocked database
  before((done) => {
    request = SuperTest(PlatformTest.callback());
    done();
  });

  after(TestMongooseContext.reset); // reset database and injector

  describe("GET /rest/calendars", () => {
    it("should do something", async () => {
      const response = await request.get("/rest/calendars").expect(200);

      expect(response.body).to.be.an("array");
    });
  });
});

Todos

[ ] Tests
[ ] Coverage
[x] Example
[x] Documentation

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 93.634%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2328

Relevant Lines:
2425

💛  - Coveralls

:tada: This PR is included in version 6.121.0 :tada:
The release is available on:

v6.121.0
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.0.0-beta.13 :tada:
The release is available on:

v7.0.0-beta.13
GitHub release

Your semantic-release bot :package::rocket:


Inaccurate floating point arithmetics produce invalid Geometries (non-closed LinearRings)

How to reproduce
Input GeoJSON file created with ogr2ogr  (polygon.geojson)
{
"type": "FeatureCollection",
"features": [
{ "type": "Feature", "properties": { }, "geometry": { "type": "Polygon", "coordinates": [ [ [ 5425435.733081569895148, 2012689.63544030720368 ], [ 5425333.066045090556145, 2012658.8061882276088 ], [ 5425324.357915714383125, 2012693.518385621719062 ], [ 5425426.5193927353248, 2012720.238697179593146 ], [ 5425435.733081569895148, 2012689.63544030720368 ] ] ] } }
]
}

1.
json2geobuf data/polygon.geojson > data/polygon.geobuf

2.
geobuf2json data/polygon.geobuf

Output:
{"type":"FeatureCollection","features":[{"type":"Feature","geometry":{"type":"Polygon","coordinates":[[[5425435.733082,2012689.63544],[5425333.066046,2012658.806188],[5425324.357917,2012693.518385],[5425426.519394,2012720.238697],[5425435.733083,2012689.63544]]]},"properties":{}}]}

Note, first and last vertex are not equal
re: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Math/round

Can we solve this simply by throwing out the closing point in each polygon ring? It's assumed to always be equal the first point so we can reconstruct the ring when decoding.

@mourner - yes, I think ensuring first and last point of polygon ring are 'equal' would be an easy workaround.
Better still, lets see if we can get a consensus on enforcing geographic WGS-84 for geobuf as you suggested ( ref : http://www.ietf.org/id/draft-butler-geojson-05.txt)
@springmeyer @sgillies @tmcw - any thoughts ?

:+1: Yep, I'm in favor of dropping the last coord and resuscitating it on conversion, as well as going WGS84-onlye.

Yep, I'm in favor of dropping the last coord and resuscitating it on conversion, as well as going WGS84-only.

Same, :+1:

:+1:  on the last coordinates. By enforcing CRS84 do you mean throwing an error when there's a non-CRS84 CRS object in the GeoJSON?

By enforcing CRS84 do you mean throwing an error when there's a non-CRS84 CRS object in the GeoJSON?

Yes, and maybe geojson2geobuf converters could also try to detect and throw on coordinates outside of -360->360/-90->90?

@springmeyer - did you mean -180/180 -90/90 ?
Looks like we all on the same page re:drop last coordinate from linear rings.
Also, it's so tempting to just enforce Lon/Lat WGS84 and avoid CRS tags and undetermined rounding errors in geobuf. Instead of throwing we can handle  -> WGS84 translation in our tools ? /cc @mourner @sgillies

Instead of throwing we can handle -> WGS84 translation in our tools?

Yes, that's what I had in mind.

@springmeyer - did you mean -180/180 -90/90 ?

Well, I said 360 as I understand this may be important as per https://github.com/mapbox/tilelive.js/issues/113.

Yeah, there are lots of sources that go beyond -180/180 to solve date line problems in regions like NZ.

@mourner is there anything you could use a hand with before publishing another version with this fix?

@tschaub the biggest showstopper currently is redesigning the format to support efficient streaming, see https://github.com/mapbox/geobuf/pull/51 and #37 for several opinions. I'd love more hands on this — I think this is the last thing we want to solve before settling on a stable major version.


Project Structure Is Inconsistent

There are several inconsistencies that make this project feel a bit messy:

The examples folder only contains one of the five examples files
There is a sub directory for test images, but not one for test videos. Test videos are all kept in the root directory for some reason?
The project is requesting students write images back into the test image source directory, instead of a destination directory. This will give students the extra task of keeping track of which images are which if they run their pipeline on images more than once (which it seems like they should).

I would be happy to clean this up in a PR (including update the ipython notebook) if you guys are comfortable with it.

Thanks for the feedback David!  That would be great if you want to go ahead and clean it up to your liking and submit a PR :)

Thanks @ryan-keenan, I submitted a PR


Feature: Implement X so users can do Y (REPLACE THIS)

Disclaimer:
Please go through existing feature requests 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.

Description:
A clear and concise description of what the feature is and should the developer implement it.
User flow & interface:
A clear and concise description of how the user would interact with the proposed change (User interface, guidance and understanding of the feature)

Hi, this looks like an empty template, so I am closing it now, please open a new issue if you have a feature request.


ignore rabbitmq self signed ssl

Provide a description of what has been changed
Checklist

[ ] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[ ] Tests have been added
[ ] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[ ] A PR is opened to update the documentation on (repo) (if applicable)
[ ] Changelog has been updated and is aligned with our changelog requirements

Fixes #

Hi @pretystar
Could you open an issue explaining the problem that you have?
From the code I can understand that you'd like to connect to a rabbit using amqps with a self-signed cert, but the change that you have done skips tls verification always (only in case of amqp protocol).
IMO, we shouldn't do that as default and should be the user who splicitly set that ignore in due security reasons.

Stale, closing


Connection to USB Freezing

Hi, just wanted to mention really appreciate all the work you guys are doing.
I am running Fedora 16, and having incredible issues getting stm32f4discovery board working consistently.
I was able to get ./st-util working and was able to load the code, but very quickly the microcontroller stops responding. This is the output on the st-util:
[!] send_recv
libusb_submit_transfer(-6)
[!] send_recv
libusb_submit_transfer(-6)
[!] send_recv
libusb_submit_transfer(-6)
[!] send_recv
libusb_submit_transfer(-6)
This occurs continuously, I can't stop the st-util process at all, no matter what I do. The continue command in gdb always hangs the st-util. It is an uninterruptable process. And the only way to fix it is to restart the computer. I couldn't find any way to the get it working without restarting the entire computer. GDB consistently hangs. I seem to get flash working relatively well, it doesn't hang to to much, but it happens after a while. I am having a bit of a tough time figuring everything out, but this is making things far more difficult.
If I disconnect the board and reconnect it the LED close to USB blinks RED. A lot of the issues I can't seem to find info on anywhere, it doesn't looks like its a typical problem. Any help is really appreciated.
Also I noticed the makefiles for all the libraries on the recent update. I really appreciate this as well, for the last couple days I was working on getting the makefiles for the libraries, I had just finished gotten it working, but I can just use yours now.
Again thanks for the help.
TheLegace

CTRL-C might break out of the server.
Try reading and writing to flash with flash.exe.
Have you run the test_usb.exe ? Any problems?
Does the stlink build all build without any error messages?

Everything works at least once, before it get's stuck.
I will give test_usb a shot, but it's not like nothing works, it just stops working after a while. Flash is working for some time, and stlink builds perfectly.

I don't know if this helps, but once the libusb freezes, and I disconnect the device and do lsusb, the  Bus 003 Device 003: ID 0483:3748 SGS Thomson Microelectronics doesn't disappear, which is totally weird.
Here is some more info if you would like:
Bus 003 Device 003: ID 0483:3748 SGS Thomson Microelectronics
Device Descriptor:
bLength                18
bDescriptorType         1
bcdUSB               2.00
bDeviceClass            0 (Defined at Interface level)
bDeviceSubClass         0
bDeviceProtocol         0
bMaxPacketSize0        64
idVendor           0x0483 SGS Thomson Microelectronics
idProduct          0x3748
bcdDevice            1.00
iManufacturer           1 STMicroelectronics
iProduct                2 STM32 STLink
iSerial                 3 Q�kH�IS'�
bNumConfigurations      1
Configuration Descriptor:
bLength                 9
bDescriptorType         2
wTotalLength           39
bNumInterfaces          1
bConfigurationValue     1
iConfiguration          0
bmAttributes         0x80
(Bus Powered)
MaxPower              100mA
Interface Descriptor:
bLength                 9
bDescriptorType         4
bInterfaceNumber        0
bAlternateSetting       0
bNumEndpoints           3
bInterfaceClass       255 Vendor Specific Class
bInterfaceSubClass    255 Vendor Specific Subclass
bInterfaceProtocol    255 Vendor Specific Protocol
iInterface              4 ST Link
Endpoint Descriptor:
bLength                 7
bDescriptorType         5
bEndpointAddress     0x81  EP 1 IN
bmAttributes            2
Transfer Type            Bulk
Synch Type               None
Usage Type               Data
wMaxPacketSize     0x0040  1x 64 bytes
bInterval               0
Endpoint Descriptor:
bLength                 7
bDescriptorType         5
bEndpointAddress     0x02  EP 2 OUT
bmAttributes            2
Transfer Type            Bulk
Synch Type               None
Usage Type               Data
wMaxPacketSize     0x0040  1x 64 bytes
bInterval               0
Endpoint Descriptor:
bLength                 7
bDescriptorType         5
bEndpointAddress     0x83  EP 3 IN
bmAttributes            2
Transfer Type            Bulk
Synch Type               None
Usage Type               Data
wMaxPacketSize     0x0040  1x 64 bytes
bInterval               0
Device Status:     0x0000
(Bus Powered)
I think both flash, and st-util end up becoming a zombie process. I will test with other distro, but I'm sure it's a kernel issue, but no one else seems to be having trouble. I am assuming your not.
Thanks for everything.

After going through distro after distro, I manged to figure out that this looks like a either a Fedora 16 problem, or a newer kernel issue.
I was able to successfully use your program without any hiccups or problems in Ubuntu 11.04. I don't remember the specifics, but if you want me to do any testing feel free to ask.
Thanks.

I bet this is the same as https://github.com/texane/stlink/issues/31
Looks like the USB isn't getting cleaned up properly in certain exit cases, and restarting the app doesn't clean it up either.  Having a bit of a look at a proper fix

When I get some time after my exams I will try and help out with the project and fix the bug (not a fan of ubuntu). Are you running latest kernel? We could also try and compare libusb versions.
On the working Ubuntu I have is 1.0.8-2.
Oh also, if you ever get a chance and update the write program could add a command that resets the microcontroller. I don't know if it's normal, but after writing I have to power cycle the stm32f4 board and my program starts running.
Thanks appreciate it.

Please test against v1.2.0 or master, because this issue references a very old version of the stlink tools. Feel free to open a new issue if the problem still remains.

I know this references a very old libusb-version. However we may give it another try with our v1.3.0-patch  branch (the oldest one with a working compilation) with some real hardware (any 0x2ba01477). If this does not produce the described output, we would be able to close this with a solution and reference this ticket correctly . Can somebody help out?

@Vascom with your Fedora system maybe?

I think this issue should be closed :)

No, I don't simply close tickets just because they are old. There should be either a verification or another solved ticket that addresses the same issue. Here we have the chance to verify with Fedora (whatever version available) and v1.3.0-patch , so we should do it. This is also one reason why the patch-branches are still there.

@Vascom: Could you test anyway?

I can't test because I have not hardware for it. I am only maintain package.
Anyway old Fedora releases not suppurted and should not be used. All bugs on unsupported releases should be closed and user must try new release.

I can't test because I have not hardware for it. I am only maintain package.

I understand, so we would need somebody else for this task.

Anyway old Fedora releases not suppurted and should not be used. All bugs on unsupported releases should be closed and user must try new release.

I know, that's why I wrote "whatever version available" ...

Are there any other fedora users that can help to verify this with real hardware?

Are there any other fedora users that can help to verify this with real hardware?

Fedora 16 used libusb 1.0.9.


JIT/PAM Support?

Is there any documentation or methods to implement Nebula for JIT access? Basically having Nebula act as a PAM versus having to rely on PAM solutions such as CyberArk (shudders).
Use case:
I want to allow Bob the user to access one or more servers via SSH, but only for a certain period of time (let's say 24 hours).
Additional notes:
I understand there would have be some lifting on my end - such as creating a service/web portal to request access. After that point, how quickly could Nebula revoke access and update its configuration to blocklist the existing cert? Is there a way to potentially tweak how long a certificate would last before expiring? And could a user request a new cert to re-allow access?

Apologies for the late response here.
The Nebula project is focused on providing highly performant and security tunneling between your devices, with simplicity as a design goal. In order to ensure a focus on security and performance it's important to keep the surface area of Nebula low. The project's stance is that key management and signing oracles are outside of the scope of Nebula and should be implemented as separate projects.


[update] Fixed shortcuts

It seems that on my Mac, Option + Click works fine, but Command + Click is invalid.

Great, thank you!


run travis tests on node 4,6,8

Followup to https://github.com/octokit/node-github/pull/1#discussion_r133840317

@dang, wrong repo


Question: Requirements

Not an issue, just a question- what are the VRAM/RAM requirements for running this method? I have an 8GB GPU/16GB RAM macbook and it said it attempted to allocate 15.8GB. I added attention slicing to the pipeline but it didn't seem to make any difference. Is there any potential for lowering the requirements with this method?

hmm.. have you tried with a smaller batch size?

I haven't- where's that set in the code? I see it set in the forward function on line 80, but it never seems to get used

Nevermind, I missed the number of samples in the interface- thanks!


Adding a new workbook on Azure Service Health

Adding a new workbook on Azure Service Health for Azure Service Health based on Azure Activity logs.

Hi @samikroy,
Please resolve branch conflicts. Thanks

Hi @samikroy, Please resolve branch conflicts. Thanks

@v-atulyadav - Thank you for sharing. Done.


Record/add fades using python?

I noticed that Audaspace supports input(record?) of audio(as stated on the first page), but looking through the api, I don't see any obvious commands for doing this? https://docs.blender.org/api/current/aud.html
If it is possible to record audio with audaspace through python, is there a snippet showing how to do it somewhere?
There is this Linux-only project, which uses ffmpeg to record the audio, but it would be great if using Audaspace instead could make it cross platform: https://github.com/britalmeida/blender_addon_push_to_talk
Another thing I noticed when looking through the API is that it includes operators for doing fades. The current VSE built-in(python start up script) way to add fades is by adding keyframes, but that will make a mess out of existing keyframes.
Those audaspace fades are they added "on top" of the f-curve animation of the volume?
Can these fades be added through python to VSE strips?

Audio recording with audaspace is very limited and currently only supported through the C++ API. Recorded samples have to be handled directly and thus this feature is not available in the Python bindings. There are currently no plans to extend functionality in this area.
Regarding adding fade in/out effects of audaspace to VSE strips in Blender: this is not possible, the VSE cannot store python objects. This would have to be implemented within Blender as  separate effect strips.

Ok. Thank you. And adding filters to vse strips through python is also out of the question right?

Yes, that's what I was trying to say with the second paragraph

Ok. I don't know if you have seen this new project, which is a fork of Blender: https://github.com/ResonaBlender/resona
The project lead's main ambition is apparently to improve the audio side of the the VSE and nodes.

On audio recording with ffmpeg on Windows in this project: https://github.com/britalmeida/blender_addon_push_to_talk
There is some slow startup time, which makes it impossible to use it as thought, so external python libs will have to be used.
On a different note, I don't know if this is audio or video related: https://developer.blender.org/T79231


pass --advertise-host flag for conf.single-node: true

In order to have host match the hosts in the certificate when combined with tls.enabled: true.
Fixes https://github.com/cockroachdb/helm-charts/issues/57

Hi @JuanLeon1 @DuskEagle any chance this can be merged and released in the next version?


Rename machinecontroller to machine-controller.

This makes the naming more consistent with other components.

This change is


clr-tooltip over progress-bar inside progress group

Select one ...  (check one with "x")
[x] bug
[ ] feature request
[x] enhancement

Expected behavior

I'd like to add a tooltip over one progress bar inside a progress group to display the percentage inside.
Actual behavior

Unfortunately, the progress bar won't show, if I hover over it where it should display, the tooltip shows though.
plunker: https://plnkr.co/edit/3PtHo7?p=preview
Am I doing something wrong or is this something that was not intended to do?

@ninodinatale: We don't recommend this pattern. Our UX team recommends that tooltips should only be shown on actionable icons (icons when clicked perform an action, for example direct you to a new page). We recommend that if you had to show the value, you show it next to the progress bar we do in the documentation.

But if this is feature absolutely required by your app, you need to make sure that your tooltip is assigned a width: 100%. Right now the width is 0 and thats why its not showing the tooltip.


Fix access to buildkite-agent owned git repositories

Fixes https://buildkite.com/gusto/payroll-building-blocks/builds/22494#018f874e-1aa8-4454-9b08-b849c6b59645/217-219, which was likely introduced in https://github.com/Gusto/buildkite-builder/pull/113 with the git upgrade from 2.30.2 to 2.39.2.
The container is running at root (id: 1), but the repositories are owned by buildkite-agent (id: 9999), which is tripping git. I believe /workdir is a Buildkite Docker plugin concept.

This is an app-specific problem. We'll address it there.


no progress report when pushing huge files

Although there is a progress bar, but it seems there are only
two states: 0% and 100%, which is quite confusing. I had to
strace the process to make sure it's actually uploading the
file, not stuck.

Thank you for reporting this. I too have found this to be undesireable behaviour. Right now it is because of the way the progress counter is used. Basically progress is incremented per completed file transfer and not per byte transferred. Watch this space.

This is the same as #23 FYI.

Please take a look at PR https://github.com/odeke-em/drive/pull/147

Fixed by https://github.com/odeke-em/drive/commit/b2659cfeaf3b689223aee3da1e26d75db7d874f6.


[Bug]: Github pybop.CMAES tests are failing due to np.inf cost return

Python Version
3.8-3.11
Describe the bug
Failing tests here: https://github.com/pybop-team/PyBOP/actions/runs/6895515377/job/18759625531
Seperate from #98. I can't repeat the test fails locally (Intel Mac, AMD Ubuntu), but I suspect it's from the new optimiser options here:https://github.com/pybop-team/PyBOP/blob/2c7e74495425ba801aeb2b09670b55fa405465f3/tests/unit/test_parameterisations.py#L119.
Steps to reproduce the behaviour
n/a
Relevant log output
No response

To solve the random failure of CMAES, I think we need to pass a representation of pints.RectangularBoundaries to CMAES. See here.


Update request: tpnote 1.24.2 → 1.24.3

Package name: tpnote
Latest released version: 1.24.3
Current version on the unstable channel: 1.24.2
Current version on the stable/release channel: 1.24.2
Change log:
Enhance the HTML clipboard configuration options
This release extends the html_to_markup filter by adding a default=
parameter in case the filter fails, its result is the empty string, or if
the filter is disabled for a certain markup language with the configuration
variable extensions.1.
The clipboard variable is replaced by the variables txt_clipboard and
html_clipbaord. They both represent the clipboard's text content. The
difference is, that the html_clipboard carries HTML formatting along the
text, while txt_clipboard holds the plain text only.

From my quick test this can not be built until rustc is upgraded to 1.79, which will likely be a massively breaking change.

I downgraded the mentioned dependencies and release v1.24.4. This compiles with rustc 1.77.2 and above.

@Frontear Could you check again?

Compilation succeeds, and software from an extremely basic level works as well. I'll submit a PR up

Sorry, forgot to push. Its done now.

Will look into pushing later tonight, assuming someone doesnt beat me to it :)


LR Finder fails when using multi-node training

Bug description
When running lr_finder with a multi-node cluster (for e.g. 2 nodes each having 8 GPUs), this line tries to restore the checkpoint on all the nodes. This leads to an error as the checkpoint file is only saved on the 0-th node and is not present on other nodes  (as described here). The code to restore the checkpoints in lr_finder should first check if the global_rank is zero. It seems like this was indeed the case earlier (see here) but was changed in later versions.
What version are you seeing the problem on?
master
How to reproduce the bug
No response
Error messages and logs
# Error messages and logs here please

Environment

Current environment
#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0):
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 2.0):
#- Python version (e.g., 3.9):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):
#- Running environment of LightningApp (e.g. local, cloud):

More info
No response

@praritagarwal Lightning saves checkpoints on (global) rank 0. It assumes that the checkpoint path resides on a shared filesystem (e.g. a network mount in a cluster). For loading, each rank needs to load the file to restore its weights, and thus again we assume that the checkpoint is visible to all nodes (through the shared filesystem). This is how it is currently. It is not specific to the LR-finder, but all saving/loading of checkpoints.


[FEATURE] Custom request data in CLI

Problem
Currently, Schemathesis CLI doesn't allow the user to directly set specific values for different elements in generated API requests such as query parameters, cookies, and body. Only headers are possible directly (via the -H CLI option) and the rest are only possible with Python hooks.
Solution
Add a set of new CLI options, --set-<location> to allow users to set fixed values in API requests. The location could be header, query, cookie, or body.
Examples for header, query, and cookie:
st run --set-header 'Authorization:Bearer 12345' ...
st run --set-query 'foo:bar' ...
st run --set-cookie 'session_id:bar' ...

The syntax above is what -H uses now for headers, so I believe it could be a simpler transition that provided a unified way to set values there.
{name}:{value}
Multiple options could be specified too!
NOTE: Implementing this feature effectively deprecates the -H option which is inherited from cURL and users are familiar with it. But I think it is more important to have a separate namespace for customizing the generated data via CLI which will unify all possible request elements.
For body it is a bit different - no need to specify the name part:
schemathesis run --set-body '{"first_name": "John", "last_name": "Doe"}' ...

Future extensions
I think it could be a minimal version for now, just set the whole body. In the future we might have something like this to set specific values within e.g. a JSON value:
schemathesis run --set-body-at 'foo[3].bar:42' ...

and it will take the foo field, then its 4th element, and set the bar field of this element to 42. However, such a feature requires filling some gaps within the value when e.g. there are not enough elements or they don't have such fields, or maybe they even have different types! These cases are important to resolve upfront before implementing this extension. Another aspect of it is that the "light" version of this feature could work by modifying the schema with some "const" keywords, but this extended version can not in the general case (as the generation of complex structures could be affected by many different keywords).
To discuss

Should a space be allowed after :? What if the user wants to pass a space as a part of the value?
Setting values for specific endpoints. Probably it could be nice to have, but I think that having a config file is better for such a scenario, otherwise CLI could be quite overloaded and hard to manage.
Add --set-form-data for Open API 2 with the same syntax as for query / header / cookie?
What about path parameters?
Usage in the pytest integration? It feels less relevant as usually users just set desired values directly on the generated Case instance. This is basically what #8 suggests.

Implementation details

Extend CLI with new parameters and collect them into a separate entity
Pass this entity through the runner and when it comes to the Case generation via the get_case_strategy modify the schemas, relevant to these components.
Open API: It could be adding a new property with a const schema inside properties and extending the required list. For negative cases, it should come after mutation.
GraphQL: Just modify the generated case (or skip the generation step if body is provided)
Add deprecation notice for -H

Related issues

#1639
#8

Being able to provide certain values to always be used in your tests seems like an excellent idea to me.
I personally use schemathesis for Bi-Directional contract testing where I need to verify that an APIs request/response pairs function as is documented in an OpenAPI specification. Due to the complexity of most of the APIs, I work with it is quite difficult to get test cases generated for some of these responses with schemathesis.
Having the option to hardcode some of the values probably will assist in getting some better testcases for my situation.

Hey @Stranger6667 here are my answers to your questions.
Which specific types of values do you need to be able to hardcode? E.g. headers / query parameters / etc
Currently Schemathesis already allows the "hardcoding" of specific headers in the CLI. I would like a similar function for queries (I made a feature request for that #1639). I have a use case where a query value decides from which environment a request is performed and I would like to provide the value for this in the CLI command instead of providing it with a enum value.
Perhaps having the option for providing path parameters in the CLI might also be useful for testing. I have had some use cases where there are specific items in a database which are for testing purposes (other items are added/removed depending on the time of day) and to interact with these item I had to provide specific path parameters using enum.
How do you currently handle such use cases, if at all?
By providing these values in an enum in my OAS and in some cases this enum is dynamically updated using a SED command prior to the Schemathesis test.
Do you anticipate any potential downsides, such as increased maintenance or difficulty in updating hardcoded values?
For my own use cases I expect no downsides or extra difficulty.
Do you think you'd need to dynamically change these hardcoded values during a test run? or e.g. provide N possible options to choose randomly
Yes, there are some use cases where I will need to dynamically change the values prior to the test run.
Yes, I think there could eventually be some use cases where I have to provide multiple hardcoded values to test various example situations (where I do not want to provide them in the OAS for security reasons). Perhaps the option for providing a file containing multiple values for a header, query or path parameter in the CLI might be a viable solution for this.


OSError: Could not find lib geos_c or load any of its variants ['libgeos_c.so.1', 'libgeos_c.so'].

I decided to install this on my linux server instead of my computer (OS X) but now Im getting another error which I try to "import db"
lots of traceback and then the subject error
OSError: Could not find lib geos_c or load any of its variants ['libgeos_c.so.1', 'libgeos_c.so'].
I installed geos, but that didn't work

@baskettcase  try   sudo apt-get install libgeos-dev

Is there a way to do this on a shared environment with pip in a venv?

Im having this problem in OS X also, so OS X and CentOS 6

I installed as root on the server with yum, so that seems to have fixed things. Thanks!


try with resources decompilation is not accurate

A jdk8 class file with "try with resources" code doesnt decompile correctly
Here's the source

And here's what the tool outputs ( im using v0.5.4) in my default config

I couldn't find past issues around this area.
Is this addressed in any of the recent builds?
thanks in advance!

@sadaaithal
This is not Luyten issue, it's up to decompiler engine - procyon
I'd recommend to open up such an issue over here
https://github.com/mstrobel/procyon/issues

@sadaaithal This is not Luyten issue, it's up to decompiler engine - procyon I'd recommend to open up such an issue over here https://github.com/mstrobel/procyon/issues

thanks @greenozon , ive opened an issue under the procyon project


nth:0.2.0

I am submitting

[ ] a new package
[x] an update for a package

I rewrote this package to make it simpler, and fixed a critical bug in version 0.1.0. Thanks to @jeffa5 for pointing out the issue!
This version uses the fixed ends-with function and is only compatible with Typst compiler 0.8.0 and above.

@fnoaman @jeffa5 are you both happy to be included as authors for this version in typst.toml?
https://github.com/typst/packages/blob/f62a235427c83a59ee41bfd4d301d21ebbdb8e67/packages/preview/nth/0.2.0/typst.toml#L5

@fnoaman @jeffa5 you have both contributed to this package, do you agree to be included as authors for this version in typst.toml?
https://github.com/typst/packages/blob/f62a235427c83a59ee41bfd4d301d21ebbdb8e67/packages/preview/nth/0.2.0/typst.toml#L5

Sure!

Tell me when it's ready to merge!

Ok yes, I'll leave @fnoaman as an author in typst.toml and this to good to merge now. 👍


Way to circle back rapid review to authors

Something that is not really COVID-19 specific but that would help a lot with making the content useful fast, is to have a way to email the corresponding author of the preprint with an alert that a rapid review has been submitted on their preprint.
Ideas on how to do this? On bioRxiv/medRxiv preprints there's always a corresponding author's email available in the metadata, as far as I know.

@murkatr @dasaderi ToDO: We need to talk with the OASPA Initiative more about a workflow for notifying authors that a review has been submitted. What are journals workflow for this? Are there any meta-data on the preprint about journal submission?


Improve release workflow

Description

Enhances the user-friendliness of Makefile and improves the way we manage releases.
Related Issues
It depends on #1030, which must be merged first.
It closes #1022 and #802.
Checklist

[ ] Is this code covered by new or existing unit tests or integration tests?
[ ] Did you run make test successfully?
[ ] Do new classes, functions, methods and parameters all have docstrings?
[ ] Were existing docstrings updated, if necessary?
[ ] Was external documentation updated, if necessary?

Additional Notes or Comments

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (a4a75f1) 78.27% compared to head (c3966f6) 78.25%.
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1038      +/-   ##
==========================================
- Coverage   78.27%   78.25%   -0.03%
==========================================
  Files         101      101
  Lines        6351     6345       -6
==========================================
- Hits         4971     4965       -6
  Misses       1380     1380

see 7 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Group not change state with setAutoCollapseOnExpand(true)

First of all, incredible work bro :D.
I have a lot of groups (expandable section) and i set the adapter with param setAutoCollapseOnExpand(true). Each group have 2 background color states: gray on expanded and white when collapsed. First i click on one group to expand it then i click to another group to expand it too. At that time the first group should collapse (this work nicely) but the state not change. It mean that the background color not change its state.
I fixed this by add notifyItemChanged(position) in expand and collapse method of FlexibleAdapter class. Just want you to know.
Again, incredible work bro. Sorry for my bad English.

Thanks for the message @rainboy93
.
I think in this case, the notifyItemChanged(position) is specific for your use case, while others don't need this feature and because you can reach this behavior by simply extending the Adapter and override the appropriate methods. It seems more a customization than a general feature.
However, this can be better obtained in the expand/collapse of the ExpandableViewHolder, without calling the binding method.


V 0.4.0

Codecov Report

Merging #30 into master will increase coverage by 2.15%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master    #30      +/-   ##
========================================
+ Coverage   97.84%   100%   +2.15%
========================================
  Files           7      8       +1
  Lines          93    126      +33
========================================
+ Hits           91    126      +35
+ Misses          2      0       -2

Impacted Files
Coverage Δ

src/can.js
100% <100%> (ø)
:arrow_up:

src/core.js
100% <100%> (ø)
:arrow_up:

src/filter.js
100% <100%> (ø)

src/test/simple/data/posts.js
100% <100%> (ø)
:arrow_up:

src/test/simple/data/roles.js
100% <100%> (+22.22%)
: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 f14ee6f...56e85fc. Read the comment docs.


Show/Hide Operations based on the User

I have tried using the delgate handler and the operation and document filter for protecting the methods that is not supposed to be displayed for the user (as mentioned in issue #601 ).
But, how can we configure the Swagger UI to protect it and ask for the Login as soon as the user try to access the swagger ui page.
We use OAuth 2.0
Any help is really appreciated

I figured out the way to do this. Use the latest swashbuckle version and add the below div tag in the injected index.html

This will show an Authorize button in the swagger UI which can be used for authentication and once Authenticated, for all the requests to the API, the JWT token will be passed from the swagger UI


json file from server not working...

I am trying to access music json file from my own server but its not working only assest file json working. Please give me brief details about this feature or fix this issue thank you!

Sorry but I didn't get you can you some send some more details. Could you please provide audio file server URL by which you are playing the song. For Audio Streaming we need a simple audio URL like 'http://storage.googleapis.com/automotive-media/Jazz_In_Paris.mp3' it does not depend on assets or server. I have used as assets for only demo purpose.

i want to say .... i will put json list of music on anywhere online and then access directly inside the application.
Means, when I update online file of json music list like new music or replace any link inside file online also update list of music in application. For now if i want too add any new music in json every time need to update app. So please add online json file support like other apps.
I hope you understand what I want too say........
Thanking you for quick replay!

Hello MohdShamim, have you been able to find a solution fetching your data from your own remote server?


Favicon location.

Currently it seems one can only display the icon in the lower right corner. Does it make sense to add an option to allow the dev to specify the location? Like: location: "top-right".
We desire this because we use tinycon in our web app, and our app places notification badges on the top-right corner, like iOS, and being able to move the favicon to the top-right corner would make it look more consistent.
Thanks!

I'd be open to such an option being added 😄


Issue using export() with convert_to_jpeg

Describe the bug
I am unable to use the convert_to_jpeg flag:
>>> photos_to_process[0].export(os.getcwd(), filename="test.jpg",overwrite=True,convert_to_jpeg=False)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: export() got an unexpected keyword argument 'convert_to_jpeg'

export() got an unexpected keyword argument 'convert_to_jpeg'

I am running the latest version, I installed with pip:
Name: osxphotos
Version: 0.55.7
Summary: Export photos from Apple's macOS Photos app and query the Photos library database to access metadata about images.
Home-page: https://github.com/RhetTbull/
Author: Rhet Turnbull
Author-email<EMAIL_ADDRESS>License: License :: OSI Approved :: MIT License
Location: /Users/eecue/mambaforge/lib/python3.9/site-packages
Requires: bitmath, bpylist2, Click, Mako, more-itertools, objexplore, osxmetadata, packaging, pathvalidate, photoscript, ptpython, pyobjc-core, pyobjc-framework-AppleScriptKit, pyobjc-framework-AppleScriptObjC, pyobjc-framework-AVFoundation, pyobjc-framework-Cocoa, pyobjc-framework-CoreServices, pyobjc-framework-Metal, pyobjc-framework-Photos, pyobjc-framework-Quartz, pyobjc-framework-Vision, pytimeparse2, PyYAML, requests, rich, rich-theme-manager, shortuuid, strpdatetime, tenacity, textx, toml, wrapt, wurlitzer, xdg
Required-by:

convert_to_jpeg needs to be passed as an ExportOptions argument to PhotoExporter.export().  There are two ways to export a photo:

Using PhotoInfo.export() which is a "legacy" method with a limited number of arguments.
Using PhotoExporter.export() which provides a much more robust way to export photos. In this method, you must use an ExportOptions instance to pass the parameters including convert_to_jpeg.

Thanks!


index.d.ts in base folder is causing issue with TypeScript

I installed latest ng-redux module and put it into my project using browserify + tsify (I'm using TypeScript). I've also included the type definitions from the DT repo.
When I include this lib in my project via import 'ng-redux', I get the following error:
Error TS2306: File `node_modules/ng-redux/index.d.ts` is not a module

If I move index.d.ts outside the folder, it will compile properly and include the lib in my project.
Is it possible to move this file into dist folder or another location?
BTW thanks for making this lib, it's very helpful when using redux with ng1!

Also trying to solve this. I believe it is related to babel 6+ changes, as referenced here.
Unfortunately, I haven't been able to figure out the proper structure yet. I'll issue a pull request if I manage to figure out the syntax that TypeScript wants here.

I'm not sure with babel 6, I was using just TypeScript 1.6.
I see that the type definitions were submitted to DT but were closed a few days ago
I would think the best solution would be to have the type definitions separate from the actual repository if possible. Removing the index.d.ts file from the base directory is what solved the problem for me.
cc @Droritos

See PR #34 by @sstorie:

I'm trying to confirm this, but I thought at the recent AngularConnect conference there was a typescript talk saying that newer versions of TS will automatically look in npm packages for d.ts files. Including this file in the package would let typescript automatically find the definition file.

If that's causing problems I don't mind moving the typing to DT.

I get the same error with 1.7. I'm not sure about the auto-discovery of .d.ts files, that would be nice though.
Either way, this should probably be renamed to something other than index.d.ts? I think it's tripping up the compiler which is trying to treat it as the entrance to this module. Similar to this discussion.

The root issue here is the index.d.ts file needs to be updated to act like a external Typescript module, and not like an internal module. I ran into the same problem with some definitions I added to the redux-ui-router library, and required a simple fix to get things working without errors: https://github.com/neilff/redux-ui-router/pull/30.
As for the DT typings, I suggest that once we correct this index.d.ts file to have the right format, you remove the DT typings from your code base. The use of index.d.ts was added to let lib authors include the typings directly with a package, and not force users to get them from somewhere else.
I'm happy to submit a PR that fixes the issue.

I just tested declaring the typings as external modules as @sstorie  said and it works grea, so this is the way to go.
Feel free to submnit a PR, otherwise i'll do it later.
Thanks

Released on npm @3.1.2.

Thank you @wbuchwalter


Regarding the functionality of test_process_align

Hey @penincillin , Really appreciate the work and the effort you put behind this work. I had a few queries.
What exactly is the functionality of the binary file 'test_process_align'? What is it supposed to do and what is the output supposed to be? I don't need exact codes as I understand that they belong to sensetime, I only need a general understanding of what goes in and what comes out, so that I can figure out the process on my own.
Thanks

Hi ic-gcp,
The first functionality of test_process_align is to generated aligned the face images as shown in here. The second functionality is to predict yaw angles which is used in training DREAM block.
I suggest you to try any convenient tools that can achieve these two functionalities.
Hope this can help you.

@penincillin yes that was helpful, however we downloaded the subset of the MSCeleb dataset mentioned in the repo. In the labels txt file there are two values, one I assume is the image class and the other the yaw value to be added. When comparing the yaw values included with the images they dont make any sense. Which leads to the assumption that those values aren't exactly yaw values. Can you please help me understand what the values represent?
I've added the test labels and list file to help you understand the question better.
Thanks
test_list.txt
test_label.txt

@ic-gcp
Yes you are right, the values are original yaw angle (in angle representation) mapped with this function

Just wondering if you can share the source code for test_process_align binary.
Thanks


Feature/installer/exit codes

To be merged after https://github.com/canonical/multipass-private/pull/259 is.
This scraps the use of the registry for leaving callbacks to the MicroK8s installer and replaces it with exit codes.
Tested with the built Multipass branch.

Looks ok to me. But was not able to verify the affected paths.


feat: turn on community templates

Flips template flag to be on to match cloud behavior.

[x] CHANGELOG.md updated with a link to the PR (not the Issue)
[x] Well-formatted commit messages
[x] Rebased/mergeable
[x] Tests pass

why aren't we just removing the flag?

why aren't we just removing the flag?

@drdelambre it's only been flagged on a day, we want to reserve the right to revert it if we discover a catastrophe


Added proper field types to consumer info to match upstream kafka

@serejja It is good
But it would not  support  {"Version": "1", "Timestamp": "12313123", "Subscription": {"test": "1"}}. The Subscriptions would be empty.
I will check later who put to zookeeper string instead of int.

@miry yes please, do so, because it feels to me like this should be fixed in the library that puts that kind of data and not should we be able to handle all cases

@miry any updates? can we merge this now or maybe there's something else that should be added? thanks

@miry I'm merging this now, let me know if we need some additional changes to this

@serejja it is all ok. thanks.


[HOLD for payment 2022-12-20] [$1000]  (Dark Theme) Tab highlight color is not correct.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the app on web.
Click on the search icon on LHN.
Use Escape key to close the page.
Press Tab multiple times until first item on the LHN is highlighted.

Expected Result:
The Highlight outline should be #5AB0FF, and ensure we use this regardless of dark or light mode.
Actual Result:
The highlight outline is blue which is part of light theme and should match the dark theme
Workaround:
unknown
Platform:

Where is this issue occurring?

Web
Desktop App

Version Number: 1.2.35-0
Reproducible in staging?: y
Reproducible in production?: y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

https://user-images.githubusercontent.com/43996225/205190497-0d39c332-6daf-4571-b8be-e6d128c2c281.mp4
Expensify/Expensify Issue URL:
Issue reported by: @parasharrajat
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1669646862849829
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~015b64bc84249fb93c
Upwork Job ID:<PHONE_NUMBER>521079296

Proposal

We need to change the color on the below file to focus-visible & focus[data-focusvisible-polyfill] property box-shadow to greenHover: '#00C271',

This greenHover color looks neat & clean and but might we need to confirm the color with the design team

https://github.com/Expensify/App/blob/a8e3cb5ecab2ed91528dbe3fb559cc215ac71349/web/index.html#L35-L42
Result

picking this up myself

I would suggest discussing this first with Design team to decide if this is really needed. I had doubt on this as I posted on slack.

Note: Please discuss this issue if we really need to change the outline color.

@Expensify/design any input here?

Let's make sure the Expected Result section of the GH is more specific, as right now it's open ended as-is:

Expected Result:
The Highlight outline should match the dark theme.

What would matching the dark theme actually mean?

This is where the design team kicks in.

Understood. I'm not going to export this job to Upwork until we define/agree on the Expected Result.
https://expensify.slack.com/archives/C049HHMV9SM/p1669986622516479?thread_ts=1669646862.849829&cid=C049HHMV9SM

Cool, commenting over in Slack.

seem like @shawnborton has decided on a blue color, the one from the links on the app, which I think it's still different from this one. So this GH still makes some sense

Can we delineate which blue we have now, vs the blue we want to use? I want to make sure the expected solution is accurate before we work on this further.
Also - @sketchydroide do you agree this is still external.

While this is externally resolvable, @sketchydroide is electing to work on it and self assigned for that reason.  I don't see it in my preferences/settings. I don't miss it yet 😂 but just asking for sake up updating GH

Thanks! I missed that while I was OOO

I created the PR

Should we follow up with a PR to round the corners, and particularly, match the roundness of buttons?

the PR has been merged

Noice

maybe it will go in the next release 🤞🏼

kewl

nice work moving fast :)

this is in prod,  Ithink we can close it, unless we need to pay something, @michaelhaxhiu

Issue reported by: @parasharrajat

$250 for rajat catching & reporting this

Accepted, waiting for you to confirm next. Drop me a comment here when you do.

@aimane-chnaif yep, don't worry! :) I will dispatch you payment in 2 days via https://www.upwork.com/jobs/~012a0e934343a83aee
Invited you to that job ^

Cool, hired!

I don't think there is an offending PR here, it's jus that the UI changes made this apparent, maybe it was forgotten, IDK

Agreed, this isn't a bug or error in our process. Checking the final box.


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=363592302
https://github.com/Azure/azure-cli-extensions/commit/a0b24e846c3a9d74a5deddac5289753d5187869b

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-extensions-list.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

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


[WebView] Support saving and loading state and custom layout params

Enables the ability to save and restore WebView state when leaving and reentering composition.
In order to restore state you must use the new rememberSaveableWebViewState() API that doesn't take an initial URL.
Then you can check if any view state exists before loading a URL in the WebView. This also allows for state to not be restored if required.
val webViewState = rememberSaveableWebViewState()
    val navigator = rememberWebViewNavigator()

    LaunchedEffect(navigator) {
        val bundle = webViewState.viewState
        if (bundle == null) {
            // This is the first time load, so load the home page.
            navigator.loadUrl("https://bbc.com")
        }
    }

    WebView(
        state = webViewState,
        navigator = navigator,
        modifier = Modifier.fillMaxSize()
    )

As part of this refactor, a new API was introduced that allows custom LayoutParams to be passed into the WebView composable. There are now two entry points, one that attempts to size itself based on Modifier constraints and one where you must pass in the appropriate LayoutParams to size the WebView correctly.
Fixes #1178
Fixes #1407
Fixes #1435
Fixes #1563
Fixes #1573
Fixes #1564

It will be in the next release in approximately two weeks time

This has just been released to maven in 0.31.1-alpha

Is it somehow possible to hoist the webViewState and the navigator above the NavHost? So for example in the WebViewSaveStateSample.kt to call the rememberSaveableWebViewState() and the rememberWebViewNavigator() just right after the rememberNavController() call and then pass down the webViewState and the navigator to the Home composable via arguments?
So something like:
val navController = rememberNavController()
val webViewState = rememberSaveableWebViewState()
val navigator = rememberWebViewNavigator()

LaunchedEffect(navigator) {
   val bundle = webViewState.viewState
   if (bundle == null) {
      // This is the first time load, so load the home page.
      navigator.loadUrl("https://bbc.com")
   }
}

NavHost(navController = navController, startDestination = "home") {
   composable("home") {
      Home(webViewState, navigator)
    }
   composable("detail") {
       Detail()
    }
}

Because for me this doesn't work. With this approach the webview is not keeping the state after navigating. Any idea how to make this work?

Is there a way to remember WebView state which is in a LazyColumn item? I'm testing the rememberSaveableWebViewState but when I scroll away from that item and go back to it - WebView refreshes the page.

I put a webview into a ScrollableControlView,  Then I click one button, and the page navigates to annother page. Then, I back to the webview page, the webview refreshes the page. It loss its scroll position.
Box(Modifier.fillMaxSize().verticalScroll(rememberScrollState())){
Button(Modifier.click(gotoAnnotherPage)){  Text('NEXT PAGE') }
WebView(Modifier.fillMaxWidth().wrapContentHeight())
Text('Hello , this is a test!')
}
Is there any way to remember the scroll position? Thank you!

Thank you @bentrengrove for adding this functionality, I think it's a great step in making webviews easier to manage with compose. I wanted to second the use case that @rafakob mentioned above. I have a small webview element being displayed in a lazy column that I can't revert to a non-recycling column due to performance concerns, and the reloads are quite jarring from a UX perspective. Will be following this thread to see if there are any updates/plans to help support this use case in the future.


Initializer lists

Description
Allows C++11 style initializer list syntax.
Uses type information available at compile time to fill in proper types for variables, function arguments, and return values.
Makes ASTcompound_initializer a subclass of ASTtype_constructor so it can use it's codegen and typecheck methods.
Tests
testsuite/initlist
testsuite/oslc-err-initlist-args
testsuite/oslc-err-initlist-return
testsuite/array-aassign
Checklist:

[x] I have read the contribution guidelines.
[x] I have previously submitted a Contributor License Agreement.
[x] I have updated the documentation, if applicable.
[x] I have ensured that the change is tested somewhere in the testsuite (adding new test cases if necessary).
[x] My code follows the prevailing code style of this project.

Gonna resolve this and rebase to validate on Windows.

So a new class of ambiguity now arrises.

struct A{ float a, b; };
struct B{ float b, c; };

float ambig(A a);
float ambig(B b);

ambig({0,1});  // ambig(A) or ambig(B); ?

I would think this is too ambiguous and erroring might be wise?

struct C { float r,g,b; };

float noise(C c);

noise({0,1,2});  // noise(color), noise(vector), or noise(C); ?

Consider ambiguity via init-list an error or put user structs at the end of the rules for selection, so noise(color) is chosen?

Eew, too confusing, looks like an invitation for bugs. Let's issue an error for this kind of ambiguity.

Are you thinking erring for all three ambiguities presented?
I.e. should noise({0,1,2}); be allowed but warn & use precedence unless a function with a user struct is in competition (which then errs)?
Is there a test case/snippet for the changes to ASTtype_constructor::typecheck you pushed in?

I'm tempted to consider it an error (we can't "break" anything, since we didn't support {} constructors before at all). Then if people tell us that it's too strict, we can think about rank+warn like we do for function arguments.
Re: type_constructor... are you asking for an explanation of what I changed, or an OSL test case that shows the case that it applies to?

Shame noise({0,1,2}); does seem rather convenient, esp. considering a color4 variant: noise({{4,3,2}, 1}); vs noise(color4(color(4,3,2), 1))
I think I get what changed, but would like an explicit example beyond (vector)foo()) vs vector(foo()) or know if a test case I'm not seeing already covers that.
Just to verify mucking around in ASTtype_constructor::typecheck hasn't regressed.

I'll add a testsuite case that verifies the type constructor fix.
Noise is an interesting case.  It is tempting to want noise({x,y,z}) to work, but it's only because we know that noise(point) and noise(vector) are identical implementations. So it works for noise, but in the general case where you had func(structA) and func(structB) that were different, but A and B could each have the same initializer list form, it seems like it would be ripe for errors.
I'm not entirely convinced that noise{{3,2,1},0) is really a lot better than noise(point(3,2,1),0), or at least not enough to be worth grappling with a general solution to ambiguous cases. But if people really really didn't like having to write vector, you could always just define
float noise(x,y,z) { return noise(vector(x,y,z)); }

(and all its varieties)
Dunno if that's solving any real problems for people, or if it's just rearranging deck chairs.

I'm not entirely convinced that noise{{3,2,1},0) is really a lot better than noise(point(3,2,1),0)

Agree, but in more complicated cases (like color4 or worse) avoiding nested constructor calls is nice.

Part of resolving this has shown two important cases overlooked in #844 that are currently present in master.
struct A { float a; };
struct B { float b; };

A ambig() { printf("A\n"); A a = { 1 }; return a; }
B ambig() { printf("B\n"); B b = { 2 }; return b; }
void ambig() {}

void subfunc(A a) { printf("AA\n"); }
void subfunc(B b) { printf("BB\n"); }

shader test()
{
    // Both are still undefined behavior, based on which function defined last.
    ambig();
    subfunc(ambig());
}

To me it seems return candidates precedence should be either:
float, int, color, vector, point, normal, matrix, string, struct, void
struct, float, int, color, vector, point, normal, matrix, string, void
When ambiguity is being resolved via return type, always error if more than one overload returns a user struct.

I prefer your first precedence list -- structs should be lower priority than built-in types, I believe.
And I agree that it should be an error for multiple possible overloads are user structs. I don't see any way to definitively rank user structs except by declaration order, which is what we're striving to avoid.

Do a separate PR for this or just a single commit in this one that can be cherry-picked?
Hopefully the last un-defined case is closures, do they have a higher or lower priority than the type they wrap?

FWIW, the only reason I saw of structs being highest is so explicit casting an overloaded builtin isn't necessary.
struct A {...};
A noise(point p);
color mix(color a, color b, A v);

// Believe a cast is necessary to ever invoke the mix above.
// But the intent is far clearer that way anyway.
mix(a, b, (A)noise(P));

Separate PR -- either way is ok. Since the initializer notation is new, and it'll take at least several days after merging before even the people using master regularly are using it in earnest, I think it's ok to get the 90% solution in first, and then have a follow-up PR in the next few days that handles the tricky edge cases.
closures -- gee, I have no idea.  Make your best guess. I don't have a preference at this time. I suspect that there will be very few practical cases where this turns up. You can't typecast a non-closure to a closure, so it won't be an issue of function argument ambiguity. It can only be an issue if there are polymorphic functions that can return either closures or non-closures for the same argument list. Seems like an extreme edge case, and if we see that the strategy we choose is tripping people up, we can change it. (Remember, 'master' is subject to change in the rules until it becomes a release branch).

Updated & includes #858 for my sanity later.

I think LGTM and passes all my tests. There are some stylistic fixups I'm tempted to make, but I think I'll just do that myself as a separate PR.


MatDialog afterClosed before beforeClose with NoopAnimationsModule

Bug, feature request, or proposal:
Bug
What is the expected behavior?
beforeClose should be called before afterClosed when a dialog closes, even with NoopAnimationsModule being used.
What is the current behavior?
afterClosed is called before beforeClose when NoopAnimationsModule is being used.
Does not happen if using BrowserAnimationsModule
What are the steps to reproduce?
https://stackblitz.com/edit/angular-material2-issue-ehetcr
What is the use-case or motivation for changing an existing behavior?
Incorrect behavior.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
master

I strongly suspect it to belinked to #6736 & #6719
The pull request fixing it has been available for more than a month but still not been merged alas...

What is the delay? I will potentially have to downgrade 6 separate projects after tediously upgrading them all, only to find out I can't open a dialog over dialog in IE.

Closing as this is being addressed in #8051


Portfolio view. Create portfolio view

Toggle for Chain-centric and Token-centric views
Implement Token-centric view itself

Status verification: ⏳
What has been verified:

Token-centric view
https://github.com/novasamatech/nova-spektr/issues/1834

Status verification: ✅


glitched artifacts when cursor is at specific/random positions of the screen

https://user-images.githubusercontent.com/47404953/212354401-cf0078f5-e19b-4ddf-8bb6-d734c230cd7f.mp4

I committed a bunch of changes accumulated in the last weeks.
It may work now.

Yes.

No.

https://user-images.githubusercontent.com/47404953/212524583-22274426-7121-43f0-967c-6754101c8a5f.mp4

https://user-images.githubusercontent.com/47404953/212525468-08304444-16ff-43ae-ac44-3e924bfdf861.mp4
this time, it is very consistent; in context mode.

Alas! this is actually a shod bug. please move to shod.

I'm not entirely sure if this is actually a shod bug. I saw a window that had the same effect under shod. But yet again the original video of this issue was taken on dwm, so I'm not exactly sure.

Have you changed .triangle_width or .triangle_height on config.c?
Which changes have you done in config.c?

It seems to be the case.
An entry that triggers a submenu has a triangle at the right of it (just like the hierarchical menus of GTK+ and QT toolkits).
The size of the triangle is configured on config.c.
In your videos, there are no triangles, and the glitch only happens on a menu that has such submenu-triggering entries; and only next to such entries.  I tried to set .triangle_width and .triangle_height to high values and the glitch happens here.

I'm removing those options from config.c for being set at compile time; they are now computed at runtime from the font size.

the issue still occurs

https://user-images.githubusercontent.com/47404953/212610594-38be031b-c7fa-488c-b6c7-d116a9e7b7bf.mp4

That's weird. The error seems to be caused by the drawing of the triangle.
Which changes have you made to the code (like, in config.c)?
Does the error still occur out of the box (with no change at al)?

Which changes have you made to the code (like, in config.c)?

None. i'm building straight from git respository - as seen in the screenshot.

Does the error still occur out of the box (with no change at al)?

What do you mean

What do you mean

I wanted to know whether the bug still occur with no changes in the code at all.
Does the error still occur on the last commit?

Does the error still occur on the last commit?

09cbf47 (HEAD), Yes; but the resulting artifacts are slightly different.
https://user-images.githubusercontent.com/47404953/212663442-197f6ebb-5566-41c5-afa0-ac136f339029.mp4
ctrlmenu in the monospace font is the orignal, which was before 09cbf47 (HEAD). ctrlmenu in the sans font is the latest HEAD.


Accept negative floats in the zql grammar

Would be nice to have a test for this.

Exercised in #338


Expand the Functionality of MergeMarkups with other markup types.

Currently MergeMarkups only merges either multiple curves, or fixed points with semiLM points (created by SlicerMorph tools). As seen on forum, this is restrictive for some people. https://discourse.slicer.org/t/merging-landmarks-and-semi-landmarks-with-slicermorph/39977
So, expand the MergeMarkup tab to allow other types of Markups to be listed. Onus will be on the user to choose the correct markup type for each set. Also allow more then two pairs to be merged.

Thanks a lot for completing this! How do I use the new features? Just update SlicerMorph in Slicer?

It is done yet. We have discovered a few pesky bugs that we are sorting out. We will announce when it is fully done.


Where Is "Purchase"?

Page says:
"In this quickstart you learned how to: Purchase a phone number"
That's false. The page does NOT explain how to purchase a number.

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

ID: 4c2f1ea8-0092-93ff-e522-41d6f5f3b26a
Version Independent ID: 9fced436-f282-880f-308c-ca9a890a6eed
Content: Quickstart - Get and manage phone numbers using Azure Communication Services
Content Source: articles/communication-services/quickstarts/telephony/get-phone-number.md
Service: azure-communication-services
Sub-service: pstn
GitHub Login: @Prakulka
Microsoft Alias: prakulka

@johnaweiss
Thanks for your feedback! We will investigate and update as appropriate.


Fix missing max_position_embeddings in model config in run_clm.py

What does this PR do?

See https://github.com/huggingface/transformers/pull/27499.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Netlify Deploy error

Hi! :)
If Deploy with Netlify, error messages, and no deploy Site.
My repo: https://github.com/tiborbokor/hallo-hugo
Howto install this theme? What is problem?
Error messages:
https://pastebin.com/wySPSJ3q
Thanks!

Hi @tiborbokor, personally I've never used Netlify, so I can't really help troubleshoot. There is a clue in the logfile:
5:43:03 PM: $ hugo server -t hallo
5:43:05 PM: Error: Unable to find theme Directory: /opt/build/repo/themes/hallo

I guess the theme hasn't been installed correctly for usage on Netlify. That's the closest guess I can make without any knowledge about Netlify. Hope it helps nevertheless!

Thanks for the help. :)
What should I use to work well?

My guess is that something in the Netlify configuration needs to be changed so that the theme is loaded correctly. I guess that you'd get the same error with a different theme. Unfortunately, as I said, I don't know anything about Netlify. The best place to ask for help in this matter is the Hugo forum. Good luck! Sorry I can't help any further in this matter...
If it turns out to be a bug in the theme after all, feel free to let me know and I can have a look!

Thank you! 👍


Replacing in game gui breaks every other mod that uses events

Forge added events to the in game gui so replacing it is not necessary. Replacing it prevents all other mods that properly hook the gui from working.
https://github.com/maxpowa/AdvancedHUD/blame/master/advancedhud/TickHandler.java#L24
The RenderGameOverlay event has an eventtype variable included, so you can cancel the elements you need to, and leave the rest.

rich1051414 does make a point. Since 1.7 its been increasingly easier to make mods compatible with others. I.e. using dynamic ids.
If you would like help I'd be glad to. I have my own mod that doesn't do a super huge amount of stuff but I do understand the basics of events.

The way I see it there are three types of HUD Items (4 if you include the cross-hairs):
1. Bar (Boss, Experience)
2. Icon (Health (vanilla), Hunger (vanilla), Armor (vanilla), Air (vanilla))
3. Text (i.e. Chat, ToolTip, Record, PlayerList, Debug (this should be left alone))
In order to set this up, the code would have to recognize which type each HUDItem was and, if you wanted out-of-the-box compatibility recognize other new bars, and recognize that a resource packs can change the type.


Report on on status

Hi,
I set up this integration with Xgimi halo. I only need it to report state of projector. I mean on or off. So in case I turn it on, it will change state to on. In case I turn it off, it will change state to off. Would it be possible to add it? I only see it now as off all time.
Thank you for your help.
Filip

I actually implemented the on/off status fetch, see here.
I am using XGIMI Z6X for testing, and it reponses to asyncio.open_connection request with port=554.
So for you, the remote command works but no on/off status response, am I right?
If that is the case, it seems that the port 554 is not universal for all XGIMI device, I will try to figure this out and update the codes.


Any way to change the style of the columns and the board itsself?

Is there anyway to change the background color/ the columns color and style?
Great package by the way!

You can completely customize the look and feel of each card:
http://localhost:9002/?selectedKind=react-trello&selectedStory=Custom Card styling&full=0&down=0&left=1&panelRight=0
https://github.com/rcdexta/react-trello/blob/master/stories/CustomCard.story.js
Let me know if that works for you...

I just ended up writing it all by myself!

cool..


Login throws TypeError Key must be a buffer - Hmac

When I login I get this error in the output from npm run api i'm trying to update some packages but still seeing it
TypeError: Key must be a buffer
at new Hmac (crypto.js:117:16)
at Object.createHmac (crypto.js:643:10)
at sign (/Users/......./auth-flow-spa-node-react-master/node_modules/keygrip/index.js:22:8)
at Keygrip.sign (/Users/......./auth-flow-spa-node-react-master/node_modules/keygrip/index.js:29:38)
at Cookies.set (/Users/......./auth-flow-spa-node-react-master/node_modules/cookies/index.js:109:30)
at ContextSession.save (/Users/......./auth-flow-spa-node-react-master/node_modules/koa-session/lib/context.js:322:26)
at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188:7)

This happens when using the test password

Had to set KOA_SESSION_SECRET


Handle unblockable signals.

Handle signals which cannot be blocked (without causing undefined behaviour):

SIGBUS
SIGFPE
SIGILL
SIGSEGV

This issue was moved to seec-team/seec#14


raylib new release publish checklist

This is the list with required steps I usually follow to publish a new raylib release, categorized and organized by priority. It's a slow and mostly manual process that really takes long time and effort.
raylib code update checklist

[x] Update ALL raylib/src/external libraries: dependencies list
[x] Compile raylib/src for ALL supported platforms -> CI: GitHub Actions, Travis, AppVeyor
[x] Update and test raylib/examples
[x] Update and test raylib/templates
[x] Update and test raylib/games

raylib webpage update checklist

[x] Update raylib.com/cheatsheet.html - Add/Review new functionality
[x] Review raylib.com/examples.html - Add new examples -> Port examples for web compilation
[x] Update raylib.com/games.html - Add new games and recompile for web
[x] Review raylib.com/index.html - Add new version features
[x] Update raylib ARCHITECTURE image if required

raylib GitHub update checklist

[x] Review README - Any addition/modification required?
[x] Review HISTORY - Add new version evolution comments
[x] Review CHANGELOG - Check, categorize and format all changes for the release
[x] Review CONTRIBUTING - Add new contributors
[x] Review ROADMAP - Review current implementations and add future plans

raylib package setup checklist

[x] Update Notepad++ to latest version -> 7.8.4
[x] Update Notepad++ raylib functions list xml
[x] Review Notepad++ compiling scripts
[x] Test Notepad++ compiling scripts
[x] Update MinGW to latest version -> GCC 8.1
[x] Update TCC to latest version

raylib installer package checklist

[x] Setup package information (version, publisher, …)
[x] Setup package files (MinGW, Notepad++, raylib)
[x] Compile package: raylib_installer_v[x.x].[migw | tcc].exe
[x] Test raylib generated package

Please, feel free to comment and give some feedback!

ok, it seems everything is ready... :)