New operator state interfaces

This PR contains the proposed changes for the streaming operator state interfaces as described in
https://docs.google.com/document/d/1nTn4Tpafsnt-TCT6L1vlHtGGgRevU90yRsUQEmkRMjk/edit?usp=sharing
Highlights:

Added OperatorState interface accessible from RuntimeContext
System managed checkpointing/restore of the states (no required implementations for Serializable states)
confirmCheckpoint(..) method in the CheckpointCommitter interface has been changed to return the StateHandle for the checkpointed state
Removed Checkpointed/AsynchronouslyCheckpointed interfaces
Updated Persistent Kafka source to work on the new interfaces (does not work properly yet :/ )
Updated Tests and IT cases

Generally looks good. I was wondering whether we could use Flink's serialization instead of having Serializable as a bound - do we have always Flink code on every side of the serialization?

release-0.9 is forked off, this pull request is not blocked by that any more. Did you manage to fix the Kafka source or is some help needed there?

I could not fix the kafka test yet. Also I need to fix some tests hardcoded for the previous interfaces that have been added lately :p
This will probably happen later this week. At some point i might ask for help for getting kafka to work.

Some issues that we need to fix before merge:
Committing the checkpoint doesnt work properly. We need to commit the states one by one and also pass the names for the state.
Statehandles are not properly discarded. I need to add a wrapper statehandle that will discard the ones wrapped as well.

Also we need to add the partitioning setting to the operators as it is currently not exposed through the API

Alright I think I fixed the issues, now the only thing remains is to add partitioning setting to the API.
State partitioning should be a property of the operator therefore it should be set afterwards like parallelism.
For example: stream.map(Mapper).setStatePartitioner(...)
This is quite tricky however as the state partitioner should affect the partitioning scheme of the input streams (otherwise it makes no sense). I see two approaches here: 1. simply overwrite the partitioning without warning
2. Only overwrite in case it is not defined (forward), otherwise throw exception stating that partitioning cannot be different from statePartitioning

I vote for the second option, it is more clean and more inline with the current behavior of the API.

Let's keep the current interfaces Checkpointed and AsynchronouslyCheckpointed to not fully break current programs. They are used actually in examples that have been published.

Thats's a good point Stephan, I fixed it.

I still get these random KafkaITCase failures on travis. This might be a timeout issue of some sort, @rmetzger do you have any tips on debugging that?

Mh. Is it always the same test failing with the same message?
What is the failure?

Its always the same error in the KafkaITcase:
org.apache.flink.streaming.connectors.kafka.KafkaITCase
testPersistentSourceWithOffsetUpdates(org.apache.flink.streaming.connectors.kafka.KafkaITCase)  Time elapsed: 21.356 sec  <<< ERROR!
org.I0Itec.zkclient.exception.ZkNoNodeException: org.apache.zookeeper.KeeperException$NoNodeException: KeeperErrorCode = NoNode for /consumers/flink-tests/offsets/testOffsetHacking/1
at org.apache.zookeeper.KeeperException.create(KeeperException.java:111)
at org.apache.zookeeper.KeeperException.create(KeeperException.java:51)
at org.apache.zookeeper.ZooKeeper.getData(ZooKeeper.java:1155)
at org.apache.zookeeper.ZooKeeper.getData(ZooKeeper.java:1184)
at org.I0Itec.zkclient.ZkConnection.readData(ZkConnection.java:103)
at org.I0Itec.zkclient.ZkClient$9.call(ZkClient.java:770)
at org.I0Itec.zkclient.ZkClient$9.call(ZkClient.java:766)
at org.I0Itec.zkclient.ZkClient.retryUntilConnected(ZkClient.java:675)
at org.I0Itec.zkclient.ZkClient.readData(ZkClient.java:766)
at org.I0Itec.zkclient.ZkClient.readData(ZkClient.java:761)
at kafka.utils.ZkUtils$.readData(ZkUtils.scala:449)
at kafka.utils.ZkUtils.readData(ZkUtils.scala)
at org.apache.flink.streaming.connectors.kafka.api.persistent.PersistentKafkaSource.getOffset(PersistentKafkaSource.java:282)
at org.apache.flink.streaming.connectors.kafka.KafkaITCase.testPersistentSourceWithOffsetUpdates(KafkaITCase.java:258)

I like this very much, Kudos Gyula and Paris, this is good work. This text should definitely go into the Wiki, for reference.
I have a few thoughts and ideas on how we may be able to push this even one step further. Let's figure out how many of those actually mean changes, and which ones can simply be added after this is merged.
Current State Interface
I would like we keep the current interface in parallel to the new one. People seem to like it quite a bit (from the demos in the past days) and I think the additional code paths to maintain for that are not too many. The current interface also comes in handy for states that cannot be maintained per key, for example the probabilistic counters and aggregates that explicitly do not maintain an entry per key, but only bitmaps across all keys.
I think it would be an okay initial restriction if they are non-re-partitionable.
State Backing / Checkpointing & Restoring
This design makes the assumption that the state is held in the Flink operators and is checkpointed to an outside storage, via the StateCheckpointer.
How about we generalize this a bit, to make it pluggable where the state is actually stored? From recent feedback, I heard that people find it useful to store the state actually in an external key/value store (habe, cassandra, ...) and have Flink' checkpointing algorithm coordinate the group commits.
This would mean:

Having the state in Flink is one implementation of the state store. The current state backend becomes something that is used when the state is held in Flink.

Having the state in a key/value store would mean that we group commit changes into that key/value store (upon shanpshots) and that we keep inside Flink only caches of recent keys (for faster read access). That way, we solve out-of-core state and this also subsumes the lazy state fetching, as an operator would start with an empty local cache and pull key state as needed. No special code required.

In the long run, we will probably want to have a lightweight key-value store that we start with the system in streaming mode.

API integration
My feeling here is that we should not have special constructs like "setStatePartitioner()", but introduce something like a "KeyedDataStream", and key-state operators work only on that type of data stream. GroupBy data streams are a special case (stronger) of the KeyedDataStream, so groupBy/window and mapWindow, or reduce, have always the ability to hold key state. For map/flatMap, one would need to write stream.keyBy("userid").map(...)
Asynchronous state checkpoints
This interface is a good basis for asynchronous checkpoints, because we can do a copy-on-write per key if it is changed while the checkpoint writing is still in progress. Since we have multiple concurrent checkpoints (possibly), we need to design this carefully to allow us to have a sequence of deltas against also other deltas, not only against the full state.
This is only relevant when the state is in Flink. When the state is backed externally (key/value store), the checkpoint conceptually only group commits and starts a new bulk transaction, which does not involve any data movement.
Incremental state checkpoints
Again, this is only relevant when the state is in Flink. Here, this interface is also a good basis, since we can persist the state of the keys that changes. We cannot go incremental per key with that, which remains to be seen whether it would be relevant to do that. In the case of long windows, we certainly want incremental snapshots of the individual  window buffers, but it is unclear whether we need to expose such a thing to the user.
(Very vague so far) Shared replicated state (via CRDTs)
I have been thinking a bit whether it would make sense to allow parallel instances to have shared state
that is updated asynchronously. This would not make sense for general state/data types, but where the types can be implemented by CRDTs (conflict-free replicated distributed datatypes), this may be very feasible and quite useful. Parallel operators could maintain shared replicated counters or histograms that way.

Thanks for the nice feedback Stephan it makes a lot of sense what you have written :+1:
Let me quickly react to these points separately:
Current State Interface
I am okay with keeping the current Checkpointed interfaces, but I wonder if they are equivalent with the simple non-partitioned state abstractions.
Anyways, I want to do some refactor to make partitioned/non-partitioned states more transparent on the runtime and allow mixing them. This will probably mean that I will add a boolean flag to the getOperatorState(...) method to get either partitioned or non-partitioned state (partitioned would throw exception if it is not a keyed stream).
State Backing / Checkpointing & Restoring
This is a very good idea, I like it :) I think a slight refactor would allow us to specify any action to be taken during a snapshot, which would either return the handles and store it right now, or return null after doing the proper interaction with the external store. I  think it is safe to say that this could be an extension do be added later.
I see some immediate issues with the group commit logic to the external store. I wonder when we would actually do the commit, because the fact that the operator takes a snapshot doesnt mean that the checkpoint will be completed so we might need to roll-back. This assumes that we can roll back on the external store as well, and that is a strong assumption. Another way would be to commit the changes with the checkpoint committer interface (like in the KafkaSource) but then if we fail before committing we end up with a corrupted snapshot (the system thinks its complete but in reality the external storage doesnt contain the changes so no way to restore).
API integration
I agree completely. We need to allow mixing partitioned/nonpartitioned states as I said in the first point.
Asynchronous state checkpoints
Agreed.
Incremental state checkpoints
Agreed, and I even think maybe we can support per-key incremental snapshots if that is absolutely necessary (such as windowbuffers). This could be an interface which incrementally computes the delta after each update(...) call. This might be tricky though but should be possible.
Shared replicated state
This would be very cool indeed :)
I will start doing some refactoring to make it possible to incorporate the suggested changes.

Thanks for the design outline guys, it looks great. Two minor comments in terms of implementation:

Let us make some more emphasis on the architecture of discarding or compaction old checkpoint data. This seems a minor issue, but we have seen during the recent release testing that it has implications. [1] In the current version the JobManager discards old state, so it needs access to it. If I understand it correctly that behavior is intact after this PR.
API: KeyedDataStream should be feasible if we really make it a more general version of data stream. The interplay with windowed and connected streams is interesting. The reason why e.g. a windowed and grouping fits nicely is that the windowed groups contain only elements from one group. You propose the same design for keyed streams as well?

[1] https://issues.apache.org/jira/browse/FLINK-2221

@rmetzger : I think I found the cause of the random kafka issues, it was swallowed nul-lpointer exceptions which somehow led to this error, good to know
@StephanEwen : I implemented some of the points that you have raised, now one can mix partitioned, non-partitioned and "Checkpointed state". (Although state partitioning is not accessible from the API without the KeyedDataStream yet)

We have added a KeyedDataStream type to the API as you can see in the last commit. It is basically a supertype of GroupDataStream and represents a DataStream with partitioned operator state by a given key. To create a KeyedDataStream one has to call keyBy(...) on a DataStream. Mind that no further repartitioning is allowed on a  KeyedDataStream so all shuffle, groupBy, forward...methods will throw an exception when a user calls them.

@senorcarbone You mentioned that the KeyedDataStream disallows calls to shuffle()/repartition()/groupBy() and so on. But KeyedDataStream doesn't seem to have this functionality, unless I'm mistaken.

The setConnectionType method is overwritten which will actually make this happen

Ahh I see, my bad. :sweat_smile:

Should we merge this?

I think we have no real blocker here. I would prefer the exception issue could be addressed (message for wrapping exception).
Everything else will probably show best when we implement sample jobs and sample backends for this new functionality.

Okay I will fix the exceptions and will merge it afterwards

Please create JIRAs for changes in the future.


Disable main section scrolling when scroll on left navigation

is there any option to enable / disable the scrolling thing separately -

Disable main section scrolling when  scroll on left navigation
but keep scrolling to left navigation when scroll on main section.

 For instance checkout this example https://mrin9.github.io/RapiDoc/examples/read-mode.html

Make sure that you are properly defining the height/width of the div under which you are inserting <rapi-doc> element

Ya ....its happening when we add Header and Footer of our website.

I cannot comment how the parent HTML could be effecting Rapidoc scrolling. but another option would be to insert HTML inside RapiDoc instead of embedding RapiDoc inside an HTML
check this out for more info
https://mrin9.github.io/RapiDoc/list-html-content.html


add c.har.li

This PR is:

[x] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[x] I used the uncompressed size of the site
[x] I have included a link to the GTMetrix report
[x] The domain is in the correct alphabetical order
[x] This site is not a ultra lightweight site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: c.har.li
  url: https://c.har.li/e/
  size: 148
  last_checked: 2023-03-05

GTMetrix Report: https://gtmetrix.com/reports/c.har.li/4RfvNGzV/

Oh, interesting -- I implicitly had LANG=en_US.UTF-8 which caused the scripts/check_alphabetical_sorting.sh to fail on a different site and the behavior of sort to be entirely different! I've updated it with LANG=C set instead. That satisfied the sorting in the script.

Hah, that's good to know. Thanks!


Add i18n support

The web application should support internationalization.
I also suggest we create a project to localize it in Transifex.
That should help users of other languages to browse for public bodies in their native language.

+1


Semantic versioning for the library

The library should be versioned using tags. Would be nice to have a constant defined so it can be accessed programatically.

https://github.com/apcera/libretto/pull/23

Version const has been added, will be tagging master with 0.9.0.


fix: retry 420

SDR was retrying these for deploy polling, probably should do everywhere
@W-16119908@

@mshanemc I used 420 in the retry UTs, you'll need to update those:
https://github.com/jsforce/jsforce/blob/0184e2113962ec9f3193a9a8b41db3ccdfcd44a7/test/http-api.test.ts#L63
npx jest test/http-api.test.ts to run them locally, don't need org creds.

plugin-org NUT failure is unrelated, merging...


ffm_standby_on_float not always working

I'm actually not sure if this is an issue ffm_standby_on_float by itself, or some timing with windows that are marked as float but ffm_standby_on_float isn't applied in time or something, but I can reproduce this quite frequently, at least a couple of times a day (I have no idea why it's more prominent lately)
Basically a float window is opened and focused, but ffm_standby_on_float doesn't work and other windows are focused. Happens all the time with System Preferences. I don't have particular reproduction steps...

There are actually 2 separate issues:
1- floating window doesn't get ffm_standby_on_float when focused (when user moves mouse other windows are focused)
2- floating window has ffm_standby_on_float but not as soon as opened:
a- window A is tiled, mouse is over window A
b- window B is opened and gets focused, it's a floating window (not set by rule usually but because chunkwm decides so), mouse still over window A
c- move mouse, the movement is inside window A, window A is focused even if window B is a floating window
d- focus window B, now  ffm_standby_on_float works correctly

I have the same issue.

Bump,  ffm_standby_on_float does not work at all for me.


[chef/4294] group: split #members/#excluded_members on commas

Is there some backstory on this? Nothing else in Chef behaves like this.

Given that we can't remove the array coercion altogether without breaking backwards compat, :+1: on this.  However, I do think we should consider being more strict about user-inputs in the future.

Is there some backstory on this?

The related bug report has some more detail it seems.  A key point for me is that the comma-separated list doesn't cause a problem on create but then causes the resource to unnecessarily update on subsequent runs.
@nvwls It would be good if your commit message had some of the context from the bug report about what this change accomplishes.

Nothing else in Chef behaves like this.

Many resources have ad-hoc coercion rules.  The mount resources options property splits on both , and spaces.  The most tragic I know about is the remote_file's source property:
      # source can take any of the following as arguments
      # - A single string argument
      # - Multiple string arguments
      # - An array or strings
      # - A delayed evaluator that evaluates to a string
      #   or array of strings
      # All strings must be parsable as URIs.
      # source returns an array of strings.
      def source(*args)
        arg = parse_source_args(args)
        ret = set_or_return(:source,
                            arg,
                            { :callbacks => {
                                :validate_source => method(:validate_source)
                              }})
        if ret.is_a? String
          Array(ret)
        else
          ret
        end
      end

:+1:

I'm not sure how to resolve the test failure:
ERROR: You authenticated successfully to https://api.chef.io/organizations/chef_proxy_tests as client-creator but you are not authorized for this action

This seems unrelated to my change.

Not sure about that last commit, could you just rebase on master to eliminate the test failure and then force push instead of merging master back into here?

Spec would be nice

:+1: needs spec

@nvwls if you wanna write a spec that'd be cool, otherwise we should write one


Is the plugin compatible with react leaflet ?

I'm having a hard time using the plugins with react-leaflet, I believe they are not compatible with each other. But I might have missed something.

Hi,
Sorry I do not use React Leaflet, so I am not sure about its exact implications.
But as is, it should be just a wrapper around Leaflet objects, so most things should remain compatible, even though you might need to make a wrapper yourself to have a similar usage as in React Leaflet, if you do not want to manage it "the JS way".
Without seeing a reproduction example of how you tried to make them work together, it is hard to help any further.
Please feel free to open a question on Stack Overflow if you need help on using Leaflet and this plugin, if not already done. Make sure to share enough code and a reproduction example of what you have tried.


Add Toulouse Lautrec Institute Domain

Add support for talento.tls.edu.pe domain

@joelibaceta First stage of verification passed, official domain is toulouselautrec.edu.pe as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://www.toulouselautrec.edu.pe/sites/default/files/2023-02/23-1_Lineamientos_de_pago_TLS.pdf

@philipto It is correct, as you can see in Title 13 of the document, it is indicated that the domain assigned to the student's email is<EMAIL_ADDRESS>The literal translation from Spanish says:
"All undergraduate students have an active institutional email account. This account is composed of the student's code followed by the extension<EMAIL_ADDRESS>The access password is the same one provided by Toulouse Lautrec to access the academic system."

Sure @philipto, I'll share the link to the video game design career where programming topic is taught in the second and third years. Additionally, there are often short complementary courses offered, such as Python, data science + IA, and mobile application development.

@joelibaceta Thank you for the clarification and your patience while waiting for a review and processing your request. Pull request merged.


pull request for e2e test at 01/11/2017 19:26:12

:white_check_mark: Validation status: passed
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.


fatal error LNK1112: module machine type 'X86' conflicts with target machine type 'x64'

I seem to be hitting this error consistently now, whether I'm building for windows 32 or 64bit per the build instructions.
[882/961] LINK_EMBED(DLL) node.dll
FAILED: c:\Python27\python.exe gyp-win-tool link-with-manifests environment.x64 True node.dll "c:\Python27\python.exe gy
p-win-tool link-wrapper environment.x64 False link.exe /nologo /IMPLIB:node.dll.lib /DLL /OUT:node.dll @node.dll.rsp" 2
mt.exe rc.exe "obj\vendor\node\node.node.dll.intermediate.manifest" obj\vendor\node\node.node.dll.generated.manifest
v8_base.lib(v8_base.api.obj) : fatal error LNK1112: module machine type 'X86' conflicts with target machine type 'x64'
Traceback (most recent call last):
  File "gyp-win-tool", line 316, in <module>
    sys.exit(main(sys.argv[1:]))
  File "gyp-win-tool", line 29, in main
    exit_code = executor.Dispatch(args)
  File "gyp-win-tool", line 71, in Dispatch
    return getattr(self, method)(*args[1:])
  File "gyp-win-tool", line 169, in ExecLinkWithManifests
    subprocess.check_call(ldcmd + add_to_ld)
  File "c:\Python27\lib\subprocess.py", line 540, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command 'c:\Python27\python.exe gyp-win-tool link-wrapper environment.x64 False link.exe
/nologo /IMPLIB:node.dll.lib /DLL /OUT:node.dll @node.dll.rsp node.dll.manifest.res' returned non-zero exit status 1112

I gather it has something to do with a compilation issue... but I don't know what to do about it exactly.
I'm running Windows 8.1 64bit with Visual Studio 2013 (Update 4) -- executing the commands from a variety of terminals gives me the same error message.
I had the 32bit electron build working earlier, but then I must have done something to anger it because it no longer works. The only thing I can think of that changed was that I used nvmw to change the version of Node.js

I completely uninstalled Node, then reinstalled just the 32bit version (via Node installer). Still hitting the error... it's like something is forcing my environment to use a 64bit build, but I don't know what.

I just rebased by branch (previously off 0.25.2 tag) to the latest (0.26.0) and here is the most recent error / stack trace from running build.py:
[956/979] LINK_EMBED(DLL) node.dll
FAILED: c:\Python27\python.exe gyp-win-tool link-with-manifests environment.x64 True node.dll "c:\Py
thon27\python.exe gyp-win-tool link-wrapper environment.x64 False link.exe /nologo /IMPLIB:node.dll.
lib /DLL /OUT:node.dll @node.dll.rsp" 2 mt.exe rc.exe "obj\vendor\node\node.node.dll.intermediate.ma
nifest" obj\vendor\node\node.node.dll.generated.manifest
MSVCRT.lib(MSVCR120.dll) : error LNK2005: memmove already defined in LIBCMT.lib(memcpy.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: "public: bool __cdecl type_info::operator!=(class type_inf
o const &)const " (??9type_info@@QEBA_NAEBV0@@Z) already defined in LIBCMT.lib(typinfo.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: "public: bool __cdecl type_info::operator==(class type_inf
o const &)const " (??8type_info@@QEBA_NAEBV0@@Z) already defined in LIBCMT.lib(typinfo.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: strncpy already defined in LIBCMT.lib(strncpy.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: strchr already defined in LIBCMT.lib(strchr.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: strncmp already defined in LIBCMT.lib(strncmp.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: sprintf already defined in LIBCMT.lib(sprintf.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: strtod already defined in LIBCMT.lib(strtod.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: qsort already defined in LIBCMT.lib(qsort.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: _vsnwprintf already defined in LIBCMT.lib(vsnwprnt.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: atoi already defined in LIBCMT.lib(atox.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: isdigit already defined in LIBCMT.lib(_ctype.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: strrchr already defined in LIBCMT.lib(strrchr.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: free already defined in LIBCMT.lib(free.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: malloc already defined in LIBCMT.lib(malloc.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: realloc already defined in LIBCMT.lib(realloc.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: strstr already defined in LIBCMT.lib(strstr.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: isspace already defined in LIBCMT.lib(_ctype.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: strtoul already defined in LIBCMT.lib(strtol.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: atol already defined in LIBCMT.lib(atox.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: getenv already defined in LIBCMT.lib(getenv.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: __timezone already defined in LIBCMT.lib(timeset.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: __tzname already defined in LIBCMT.lib(timeset.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: _tzset already defined in LIBCMT.lib(tzset.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: strtol already defined in LIBCMT.lib(strtol.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: wcstombs already defined in LIBCMT.lib(wcstombs.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: _aligned_free already defined in LIBCMT.lib(align.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: _aligned_malloc already defined in LIBCMT.lib(align.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: exit already defined in LIBCMT.lib(crt0dat.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: __iob_func already defined in LIBCMT.lib(_file.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: fclose already defined in LIBCMT.lib(fclose.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: fwrite already defined in LIBCMT.lib(fwrite.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: abort already defined in LIBCMT.lib(abort.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: fflush already defined in LIBCMT.lib(fflush.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: ungetc already defined in LIBCMT.lib(ungetc.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: _lock_file already defined in LIBCMT.lib(_file.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: _unlock_file already defined in LIBCMT.lib(_file.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: memcpy_s already defined in LIBCMT.lib(memcpy_s.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: printf already defined in LIBCMT.lib(printf.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: fopen already defined in LIBCMT.lib(fopen.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: fread already defined in LIBCMT.lib(fread.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: fseek already defined in LIBCMT.lib(fseek.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: ftell already defined in LIBCMT.lib(ftell.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: feof already defined in LIBCMT.lib(feoferr.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: ferror already defined in LIBCMT.lib(feoferr.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: fgets already defined in LIBCMT.lib(fgets.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: fprintf already defined in LIBCMT.lib(fprintf.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: sprintf_s already defined in LIBCMT.lib(sprintf.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: memchr already defined in LIBCMT.lib(memchr.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: strncpy_s already defined in LIBCMT.lib(strncpy_s.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: raise already defined in LIBCMT.lib(winsig.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: _localtime64_s already defined in LIBCMT.lib(loctim64.obj)

MSVCRT.lib(MSVCR120.dll) : error LNK2005: fopen_s already defined in LIBCMT.lib(fopen.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: vfprintf already defined in LIBCMT.lib(vfprintf.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: _vsnprintf_s already defined in LIBCMT.lib(vsnprnc.obj)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: _beginthreadex already defined in LIBCMT.lib(threadex.obj)

MSVCRT.lib(MSVCR120.dll) : error LNK2005: rand_s already defined in LIBCMT.lib(rand_s.obj)
MSVCRT.lib(ti_inst.obj) : error LNK2005: "private: __cdecl type_info::type_info(class type_info cons
t &)" (??0type_info@@AEAA@AEBV0@@Z) already defined in LIBCMT.lib(typinfo.obj)
MSVCRT.lib(ti_inst.obj) : error LNK2005: "private: class type_info & __cdecl type_info::operator=(cl
ass type_info const &)" (??4type_info@@AEAAAEAV0@AEBV0@@Z) already defined in LIBCMT.lib(typinfo.obj
)
MSVCRT.lib(MSVCR120.dll) : error LNK2005: "public: char const * __cdecl type_info::_name_internal_me
thod(struct __type_info_node *)const " (?_name_internal_method@type_info@@QEBAPEBDPEAU__type_info_no
de@@@Z) already defined in LIBCMT.lib(typinfo.obj)
LINK : warning LNK4098: defaultlib 'MSVCRT' conflicts with use of other libs; use /NODEFAULTLIB:libr
ary
node.dll : fatal error LNK1169: one or more multiply defined symbols found
Traceback (most recent call last):
  File "gyp-win-tool", line 316, in <module>
    sys.exit(main(sys.argv[1:]))
  File "gyp-win-tool", line 29, in main
    exit_code = executor.Dispatch(args)
  File "gyp-win-tool", line 71, in Dispatch
    return getattr(self, method)(*args[1:])
  File "gyp-win-tool", line 169, in ExecLinkWithManifests
    subprocess.check_call(ldcmd + add_to_ld)
  File "c:\Python27\lib\subprocess.py", line 540, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command 'c:\Python27\python.exe gyp-win-tool link-wrapper environment
.x64 False link.exe /nologo /IMPLIB:node.dll.lib /DLL /OUT:node.dll @node.dll.rsp node.dll.manifest.
res' returned non-zero exit status 1169
ninja: build stopped: subcommand failed.

After deleting my local directory, re-cloning from GitHub, and clean/bootstrap/build things magically work again for a 32bit build.
I did notice that the default --target_arch is ia32 so I'm wondering if something got cached when I originally set that to x64 (or if x64 was simply the wrong target architecture for my WIndows 8.1 machine). In any case, I'll close this ticket as I am no longer bothered by the original problem.

https://github.com/atom/electron/issues/3654
mybe help you.


add server.forward-headers-strategy=framework (FINERACT-914)

FINERACT-914

awaiting #800, then rebase (will also get just merged #793)

Ready for review - do you want to merge this @awasum or any other active committers?

I am not very familiar with this feature. How is server.forward-headers-strategy=framework different from server.forward-headers-strategy=native?
Is the server.forward-headers-strategy=framework going to affect good old fashion tomcat deployments?
If these are irrelevant questions, you can ignore them and self merge as I am not familiar with the problem. @vorburger

How is server.forward-headers-strategy=framework different from server.forward-headers-strategy=native

@awasum E.g. https://github.com/spring-projects/spring-boot/issues/18667 explains this a bit, linking to https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-2.2-Release-Notes#deprecations-in-spring-boot-22, note, quote:

(...) previously server.use-forward-headers=true would use the web server native support. You can now achieve the same with server.forward-headers-strategy=native. Because each server has a specific behavior, we now offer an alternative which relies on Spring’s ForwardedHeaderFilter: server.forward-headers-strategy=framework. Developers should use the option that fits best their use case.

It seems to me that for us to use a Spring Framework Filter is better than to rely on Tomcat's for this, because this means it will work the same if we ever switched e.g. from Tomcat to Jetty, or if someone were to be crazy enough to deploy the Fineract WAR in something like WAS or WLS... :smiling_imp: Also, frameworks actually does work and solves the problem... :smile: see details in FINERACT-914.

Is the server.forward-headers-strategy=framework going to affect good old fashion tomcat deployments?

No, it should not. This thing basically just checks for certain HTTP Headers related to SSL proxies and relevant for "serious" deployments, such as https://www.fineract.dev. In our "traditional" deployment environments, such as Tomcat running locally, or inside a Docker Container for Docker Compose, those HTTP Headers will not be present, and this PR should be a NOOP for those.
Thanks for the questions and doing a real review!

@awasum OK for you to merge, or LGTM for me to self merge?

@awasum was the explanation above helpful for you to merge this one?
I'll self merge it let's on Sunday (day after tomorrow, 1 week) if I don't hear back, hope fair.

I am sorry...I forget about this because I saw your reply on mobile but never returned to merge this. Your explanation was good enough and I dont have any more questions. We can merge this now. Sorry for delay.


Features

Added role buy feature and added some ticket names + added manage server permission for add money

this feature i already have a plan to add in auction command

this feature i already have a plan to add in auction command

oh okok

about server auction

about server auction role

i should close it for now then


Hotfix CommandAPI throwing errors because of Paper's Brigadier command API

Some of this is not pretty but doing this the VarHandle-less way also got rid of the exception in the console which I didn't seem to do with VarHandles.
Nevertheless, this should make it possible to at least register commands using Paper 1.20.6 build 65 and onwards.
I plan to handle all this properly in #517

I gave this PR a quick spin with my plugin and it seems to work without any issues


Update CircleCI cache strategy

Update the caching strategy in CircleCI to prevent cross-contamination between branches.
Also moves the apm clean step after apm install to prevent a related issue.

:tada: This PR is included in version 5.2.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fix: disable warning when iterating nullable array elements

Disable warning when accessing nullable elements
Description
As suggested added null supression to NullDelegateMapping when accessing array elements.
Fixes #416
Checklist

[x] The existing code style is followed
[x] The commit message follows our guidelines
[x] Performed a self-review of my code
[x] Hard-to-understand areas of my code are commented
[x] Unit tests are added/updated

Code coverage bot generated a weird message after rebating. Not sure how to rerun the ci pipeline


Can't find the file: dino_vitbase16_pretrain.pth

Hi, thanks for sharing.
But I met a problem that is I can't the file: dino_vitbase16_pretrain.pth, from the project folder. Could you please upload this file?
Thanks a lot.

I think you can find the DINO backbone in the original DINO repository (here)!

Thanks! I've downloaded the file from the link.


Form: Radios and checks are not vertically aligned

In all the SIII browsers

Please review it.

Cannot reproduce. Closed as outdated.


Implemented NthOrDefault & Nth

Implemented NthOrDefault and Nth, similar to FirstOrDefault and First, respectively, but you pass the "position" of the desired element as argument.
Currently missing to add method reference in "project.json" and "README.md" files.
I pretend to do it latter to avoid conflicts.

Apart from the overload accepting a predicate and using a positional indexing (i.e. 1-based), how are these different from ElementAt or ElementAtOrDefault?

Nothing. The difference is exactly the items you mentioned. Also, conceptually ElementAtOrDefault and NthOrDefault are differents, because when you pass the predicate you are not caring about positional indexing itself, but about the ordinalty of an specific element of those who satisfy the predicate..


[yew-macro] Improve error message

[X] missing property value as in <input autofocus />
[X] missing identifier after with as in <Button with>

Fixes https://github.com/yewstack/yew/issues/1179

So much better, thanks!


[Feature] [1.17] Terra stalactites/stalagmites should use pointed dripstone

Pre-Request Checklist

[x] I have checked that I am on the latest version of Terra.
[x] I have searched github for similar features requests, including closed ones, and found none.
[x] I believe this is within the scope of Terra.
[x] This feature request is for all of Terra, and isn't something that should be implemented by a pack or addon.

Feature Description
In 1.17, Terra's stalactites and stalagmites should use vanilla pointed dripstone instead of walls and iron bars.
What Problem Does This Solve?
It feels strange to see stalactites and stalagmites made out of walls and iron bars when pointed dripstone has been added to the game.
A Solution You'd Like
Replace walls and iron bars with pointed dripstone. Additionally, add dripstone blocks around them to help them blend in.
Alternative Solutions

Replace only walls and iron bars with pointed dripstone, without adding dripstone blocks around them.

@dfsek I don't have perms to transfer to the default pack repo


Fix for Parallax Mirroring regression introduced by pull request #12709

Ensures that parallax layers are mirrored while still keeping parallax zoom fixes of previous pull request. Discussed in pull request #12709. Similar fix to regression fix used in 3.0 at TheSofox/godot@3551fc6 used to fix #13167.

Perfect timing, I was about to make a 2.1.5-beta build :D

I rarely get accused of "Perfect timing" but I'll gladly accept it on this occasion!


vsphere cloud config should create a network definition

vsphere cloud config should create a network definition
currently the cloud config generated does not have any networks defined at all, ever.

seems like this is due to some bug in using ENV variables instead of actual flag vars...
this only seems to affect the network-name-1 flag... needs a look


Add native autodetect schema feature

Description
What is the current behavior?
Currently for inferring the schema we are always using pandas.
For more information, check the issue below..
closes: #709
What is the new behavior?
Add native autodetect schema feature for snowflake

add snowflake file format data class
add option to use native autodetect schema for databases
implement snowflake specific autodetect schema

Does this introduce a breaking change?
No
Checklist

[ ] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

The failed checks are unrelated.

I have merged this PR, but we should take care of the double-loading in the next PR


Improved Documentation for PL Bolts Transforms

Good morning! I love PL Bolts and I'm super grateful you've made these models available to the community. However, I'm a bit confused by the transforms and I'd like to request better documentation. For two quick examples, suppose you want to use SwAV. The documentation says to do the following:
            from pl_bolts.models.self_supervised.swav.transforms import SwAVTrainDataTransform, SwAVEvalDataTransform
            from pl_bolts.transforms.dataset_normalizations import stl10_normalization
            train_transform = SwAVTrainDataTransform(
                normalize=stl10_normalization())
            eval_transform = SwAVEvalDataTransform(
                normalize=stl10_normalization())

But now several questions arise. The documentation suggests SwAV was pretrained on ImageNet, but here, we're using STL-10 normalization - why is that? Are these transforms appropriate only for ImageNet and STL-10? If so, what should I use for COCO or another dataset? What happens if I don't include the STL-10 normalization?
If someone who knows the library better could improve the documentation, that would be massively helpful :)
Thanks in advance!
cc @borda

Hey @RylanSchaeffer,
I believe this issue should be open on the Bolts repo. Mind doing so?

Happy to. I thought Bolts was part of Lightning?


Add form, presenter

Oh! Like a dullard, I broke CI with my last commit. I wonder what needs to be done to make the tests pass again? Ideally we'd have tests for these new methods. If it's not a ton of work, maybe you want to fix whatever's broken with the build while you're in there?

Ok -- I rewrote some of the form and presenter logic and added a comment about one of those statements. I'm thinking that the next step would be to completely remove all of the form stuff from this repo and move it all to a new fern-form, similar to fern-parameters. Thoughts?


Information

I've tried to install requirements.txt with "bash pip install -r requirements.txt" but didn't work. I've found another code on python website and I've tried with "py -m pip install -r requirements.txt" and then worked.  I hope that this info can help somebody :)
P.S. I'm on Windows 10, 64-bit

Thank you!


Update README.md

Introducing the Wildcard Round: At a predetermined point in the game, players are allowed to play two white cards instead of one, introducing a dynamic element of strategy. This round allows for the potential to earn double points if both cards are selected by the Card Czar, encouraging deeper thought in card selection and synergy.
Activation: The Wildcard Round is introduced at a specific, pre-announced point in the game, such as after every third round, to inject a refreshing twist.
Dual Play: During this round, players are permitted to select two white cards from their hand instead of the standard single card. This challenges players to think strategically about card combinations that will resonate with the prompt and with each other.
Double Points Opportunity: If the Card Czar selects a player’s combination as the most compelling, that player earns double points for the round. This raises the stakes and encourages players to engage more deeply with both the prompt and their cards.

For coherence sake, you'd also need to change main.tex and add the rules there as well. Besides, you'd have to specify a wildcard that can be upload to the PlayingCards.io


xpra: 4.3.2 -> 4.3.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for xpra is: "Persistent remote applications for X"
meta.homepage for xpra is: "https://xpra.org/"
meta.changelog for xpra is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.3.3 with grep in /nix/store/4dkzs5sgbclg5g3k81hn5v82181g73pg-xpra-4.3.3
found 4.3.3 in filename of file in /nix/store/4dkzs5sgbclg5g3k81hn5v82181g73pg-xpra-4.3.3

Rebuild report (if merged into master) (click to expand)

5 total rebuild path(s)

4 package rebuild(s)

4 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
ib-controller
run-scaled
xpra
xpraWithNvenc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4dkzs5sgbclg5g3k81hn5v82181g73pg-xpra-4.3.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A xpra https://github.com/r-ryantm/nixpkgs/archive/69fbabf4df8b9f22c4e23bfa3ed93c205f413c5b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4dkzs5sgbclg5g3k81hn5v82181g73pg-xpra-4.3.3
ls -la /nix/store/4dkzs5sgbclg5g3k81hn5v82181g73pg-xpra-4.3.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package failed to build:

ib-controller

3 packages built:

run-scaled
xpra
xpraWithNvenc

Maintainer pings
cc @offlinehacker @numinit @mvnetbiz for testing.

This is broken. I'll take a look at the problem.

Succeeded by #172682


Error on docknet restore: Access to the Path is Denied

After implementing the workaround in #2, I'm getting an error on docknet restore: Access to the path is denied.  Here is my output:
~/Source/docknet-test $ ./docknet new
Created new C# project in /docknet-test.
~/Source/docknet-test $ ./docknet restore
log  : Restoring packages for /docknet-test/project.json...
error: Access to the path is denied.

Do you have an idea what I might be doing wrong here? I'm a Linux newbie, so pardon if this is an obvious faux pas on my part. ;)

I am having the same issue. However in my case it is not only the files created. When I run the script in any way it changers the owner of my home directories and files to printadmin.
Here are the results of running ls -l after executing the script:
-rw-r--r-- 1 1000 printadmin  446 May 28 22:49 Dockerfile -rwxr-xr-x 1 1000 printadmin  406 May 28 22:49 docknet -rwxr-xr-x 1 1000 printadmin  352 May 28 22:49 dotnetproxy -rw-r--r-- 1 1000 printadmin 1078 May 28 22:49 LICENSE -rwx------ 1 1000 printadmin  386 May 28 22:58 NuGet.Config -rwx------ 1 1000 printadmin  202 May 28 22:58 Program.cs -rwx------ 1 1000 printadmin  227 May 28 22:58 project.json -rw-r--r-- 1 1000 printadmin 1304 May 28 22:49 README.md


[CXF-9066] Fix MaskSensitiveHelper masking of multi-line content

Thanks you @trustlydaniel !

Seems like MaskSensitiveHelperTest needs some conflict resolution ...

@reta conflicts resolved


Fix cosine of solar zenith angle glitches

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #1110

[x] Tests for the changes have been added (for bug fixes / features)

[x] (If applicable) Documentation has been added / updated (for bug fixes / features)

[x] CHANGES.rst has been updated (with summary of main changes)

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added

What kind of change does this PR introduce?
Refactor of the function so we can use a ufunc, making it easier to implement  the different cases.
Earlier version was not handling the cyclical nature of angles correctly and had issues in the polar regions.
Does this PR introduce a breaking change?
Yes, the output has changed. I think it is now correct. The signature has also changed.
Other information:
PyWGBT was used as a comparison.

Before:

After:

The whole left half of the "before" was wrong because of mishandled radians. And the arctic circle was put to 0 because the "sunset" time was NaN.

Merge when ready!

This is really great work. Thank you for fixing those glitches.


showsUserLocation does not work well with Google Maps

When I try to use ShowsUserLocation with GoogleMaps, it gives me a white box and doesn't animate properly.

The same code without provider={"google"} works perfectly.

Does anyone have any ideas?

If I am not wrong this only happens on the Simulator, on a real device it's fine.

At least for me, on a real device it does not have the white box background but it does not "pulsate". Not sure if others are having the same issue

yep it works on real devices, the pulsate is not implemented i think. At some point i might do it and give a pull request.


Compilation problem: Boost libraries not found

Original post by @ntb13230
Hello,
I've found your files and I'm trying to use them with my Basler acA1920-25um camera. Following the fact that you have developed this for pylon4 that's what I've installed and also I got boost 1.55 After a while of struggling I got to the point where your make.m file finally finds pylon files properly, however i think there is a problem with the boost.
First of all in your make file , in libraries paths, the boost one has addition of 'stage\lib', but there is no such a path in this folder. I tried to change it for 'libs' but this fails as well. Where should this point to?
what i'm getting now while running the make file is this (sorry - very long):
>> make
=> Creating Libraries
Building with 'Microsoft Windows SDK 7.1 (C++)'.
MEX completed successfully.
Building with 'Microsoft Windows SDK 7.1 (C++)'.
MEX completed successfully.
=> Creating Functions
Building with 'Microsoft Windows SDK 7.1 (C++)'.
MEX completed successfully.
Building with 'Microsoft Windows SDK 7.1 (C++)'.
MEX completed successfully.
Building with 'Microsoft Windows SDK 7.1 (C++)'.
Error using mex
baslerGetParameter.cpp
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(13)
: error C2332: 'enum' : missing tag name
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(13)
: error C2236: unexpected 'class' 'ParamType'. Did you forget a ';'?
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(13)
: error C3381: 'ParamType' : assembly access specifiers are only available in code compiled
with a /clr option
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(14)
: error C4430: missing type specifier - int assumed. Note: C++ does not support default-int
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(15)
: error C4430: missing type specifier - int assumed. Note: C++ does not support default-int
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(16)
: error C4430: missing type specifier - int assumed. Note: C++ does not support default-int
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(18)
: error C2143: syntax error : missing ';' before '}'
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(18)
: error C4430: missing type specifier - int assumed. Note: C++ does not support default-int
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(40)
: error C2039: 'Float' : is not a member of 'ParamType'
        C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(13)
        : see declaration of 'ParamType'
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(40)
: error C2065: 'Float' : undeclared identifier
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(41)
: error C2039: 'Int' : is not a member of 'ParamType'
        C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(13)
        : see declaration of 'ParamType'
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(41)
: error C2065: 'Int' : undeclared identifier
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(42)
: error C2039: 'Bool' : is not a member of 'ParamType'
        C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(13)
        : see declaration of 'ParamType'
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(42)
: error C2065: 'Bool' : undeclared identifier
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(43)
: error C2039: 'String' : is not a member of 'ParamType'
        C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(13)
        : see declaration of 'ParamType'
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(43)
: error C2065: 'String' : undeclared identifier
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(86)
: error C2450: switch expression of type 'ParamType' is illegal
        No user-defined-conversion operator available that can perform this conversion, or the
        operator cannot be called
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(88)
: error C2039: 'Float' : is not a member of 'ParamType'
        C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(13)
        : see declaration of 'ParamType'
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(88)
: error C2065: 'Float' : undeclared identifier
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(88)
: error C2051: case expression not constant
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(95)
: error C2039: 'Int' : is not a member of 'ParamType'
        C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(13)
        : see declaration of 'ParamType'
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(95)
: error C2065: 'Int' : undeclared identifier
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(95)
: error C2051: case expression not constant
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(104)
: error C2039: 'Bool' : is not a member of 'ParamType'
        C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(13)
        : see declaration of 'ParamType'
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(104)
: error C2065: 'Bool' : undeclared identifier
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(104)
: error C2051: case expression not constant
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(111)
: error C2039: 'String' : is not a member of 'ParamType'
        C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(13)
        : see declaration of 'ParamType'
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(111)
: error C2065: 'String' : undeclared identifier
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(111)
: error C2051: case expression not constant
C:\Users\ntb13230\Documents\MATLAB\Cameras\HBadertscher-Matlab_BaslerCamDriver-419332a\baslerGetParameter.cpp(124)
: warning C4065: switch statement contains 'default' but no 'case' labels
Error in make (line 64)
            mex(flags{:},ipaths,lpaths,libraryObjects{:},drivers{k})

I don't know if that's something I'm doing wrong or... could you please have a look?
(btw - I'm using Matlab 2015a, 64 bit)
Thank you!

Hi @ntb13230
I copied your comment to a new issue, as this is unrelated to the problem the other issue (which is about getting it to run with Pylon 5 instead of Pylon 4).
Getting it to run on Windows 7 seems to be a bit tricky. Luckily, Simon Lansbergen has created a walk-through tutorial which should solve the problem you are having, by installing a pre-compiled Boost library and changing some of my code to be compatible with the Windows SDK 7.1. Can you try to follow these instructions and see if that solves your problem?
PS: sorry that this is a bit tricky.. I wrote this code on a developer machine with self-compiled Boost and a Visual Studio compiler with newer C++ features than SDK 7.1, which is why a few things have to be changed.

Hi!
Thank you very much for so fast reply! There is nothing to apologise - I'm just extremely grateful you wrote these files before, as I would be in big trouble now.
I had SDK 7.1 before and indeed the Boost libraries were the problem in my case - getting pre-compiled libraries solved the problem. Everything run smoothly this time!  Now you know that your code was verified also for Win 8.1 64-bit with Matlab 2015a :)
Thank you again!!


Ensures CertificateId is unique when calling CreateCertificate

currently the CertificateId only uses the current date and time for CertificateId field. If multiple certificates are requested at the same time this can lead to errors as the CertificateId needs to be unique.
to solve this appending a GUID at the end. The total number of characters is 55 which is less than the maximum allowed by CA Service of 63 characters.

Hi m8rmclaren@, I'm an eng from GCP CA Service. can you PTAL, we have a costumer facing this issue.

@dgaley Can you take a look at this? We would need to consider the delayed issuance use cases that require a stable certificate id.


French translation implementation

Summary | Résumé
Implemented French language translations for the whole app. All the messages and text should be translated and you can switch languages in the menu. Sorry for the massive PR!
Closes #50

Works really well! Nice work! I added four valid URL callbacks to Google OAuth to account for the new language namespacing, ex http://localhost:8000/en/accounts/google/login/callback/
The only other recommendation I would have is to capitalize the F in français at the top of the page

Ha! The updated callback URI should solve my first issue! 🎉  Testing right away!

Confirming that the updated callback URIs have solved the Google SSO issue for me!
Not sure if the auto navigation to Home on language toggle should happen or not.

Works really well! Nice work! I added four valid URL callbacks to Google OAuth to account for the new language namespacing, ex http://localhost:8000/en/accounts/google/login/callback/
The only other recommendation I would have is to capitalize the F in français at the top of the page

@maxneuvians - I have changed the code so that now it appears as Français

Confirming that the updated callback URIs have solved the Google SSO issue for me!
Not sure if the auto navigation to Home on language toggle should happen or not.

@gcharest - I have changed the code so that now when switching languages, you always stay on the same page that you are currently at.


Lack of arrows, can not scroll by one row

this is significant when many rows returned and one scroll event scroll many rows instead of one.
Linux Mint 18.3

You can use row scroll buttons (bottom toolbar), e.g. ctrlk+alt+right

You can use row scroll buttons (bottom toolbar), e.g. ctrl+alt+right

this hotkey is used by system to switch between workplaces

You can redefine it in preferences->keys


Building connectedhomeip/examples/bridge-app: error => Failed to resolve component 'mdns' (IDFGH-10335)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
5.1-rc1
Operating System used.
Linux
How did you build your project?
Command line with Make
If you are using Windows, please specify command line type.
None
What is the expected behavior?
Building connectedhomeip bridge-app examples without error.
What is the actual behavior?
error => Failed to resolve component 'mdns'
Steps to reproduce.
git clone https://github.com/project-chip/connectedhomeip.git
cd ~/connectedhomeip
source scripts/bootstrap.sh
source scripts/activate.sh

cd ~/connectedhomeip/examples/bridge-app/esp32
gn gen out/debug
ninja -C out/debug
idf.py set-target esp32c3
idf.py build

Build or installation Logs.
Building connectedhomeip/examples/bridge-app: error => Failed to resolve component 'mdns'

user@matter-vm:~/connectedhomeip/examples/bridge-app/esp32$ idf.py build
Executing action: all (aliases: build)
Running cmake in directory /home/user/connectedhomeip/examples/bridge-app/esp32/build
Executing "cmake -G Ninja -DPYTHON_DEPS_CHECKED=1 -DPYTHON=/home/user/connectedhomeip/.environment/pigweed-venv/bin/python -DESP_PLATFORM=1 -DCCACHE_ENABLE=0 /home/user/connectedhomeip/examples/bridge-app/esp32"...
-- IDF_TARGET is not set, guessed 'esp32c3' from sdkconfig '/home/user/connectedhomeip/examples/bridge-app/esp32/sdkconfig'
-- Found Git: /usr/bin/git (found version "2.34.1")
-- The C compiler identification is GNU 12.2.0
-- The CXX compiler identification is GNU 12.2.0
-- The ASM compiler identification is GNU
-- Found assembler: /home/user/.espressif/tools/riscv32-esp-elf/esp-12.2.0_20230208/riscv32-esp-elf/bin/riscv32-esp-elf-gcc
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /home/user/.espressif/tools/riscv32-esp-elf/esp-12.2.0_20230208/riscv32-esp-elf/bin/riscv32-esp-elf-gcc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /home/user/.espressif/tools/riscv32-esp-elf/esp-12.2.0_20230208/riscv32-esp-elf/bin/riscv32-esp-elf-g++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Building ESP-IDF components for target esp32c3
CMake Warning at /home/user/esp-idf-v5.1-rc1/tools/cmake/build.cmake:565 (message):
  "idf_component.yml" file was found for components:

        /home/user/connectedhomeip/examples/bridge-app/esp32/third_party/connectedhomeip/config/esp32/components/chip

  However, the component manager is not enabled.
Call Stack (most recent call first):
  /home/user/esp-idf-v5.1-rc1/tools/cmake/project.cmake:464 (idf_build_process)
  CMakeLists.txt:31 (project)

-- Found Python3: /home/user/connectedhomeip/.environment/pigweed-venv/bin/python3.10 (found version "3.10.6") found components: Interpreter
CMake Error at /home/user/esp-idf-v5.1-rc1/tools/cmake/build.cmake:266 (message):
  Failed to resolve component 'mdns'.
Call Stack (most recent call first):
  /home/user/esp-idf-v5.1-rc1/tools/cmake/build.cmake:308 (__build_resolve_and_add_req)
  /home/user/esp-idf-v5.1-rc1/tools/cmake/build.cmake:309 (__build_expand_requirements)
  /home/user/esp-idf-v5.1-rc1/tools/cmake/build.cmake:595 (__build_expand_requirements)
  /home/user/esp-idf-v5.1-rc1/tools/cmake/project.cmake:464 (idf_build_process)
  CMakeLists.txt:31 (project)

-- Configuring incomplete, errors occurred!
See also "/home/user/connectedhomeip/examples/bridge-app/esp32/build/CMakeFiles/CMakeOutput.log".
HINT: The component mdns could not be found. This could be because: component name was misspelled, the component was not added to the build, the component has been moved to the IDF component manager or has been removed and refactored into some other component.
Please look out for component in 'https://components.espressif.com' and add using 'idf.py add-dependency' command.
Refer to the migration guide for more details about moved components.
Refer to the build-system guide for more details about how components are found and included in the build.
cmake failed with exit code 1, output of the command is in the /home/user/connectedhomeip/examples/bridge-app/esp32/build/log/idf_py_stderr_output_51215 and /home/user/connectedhomeip/examples/bridge-app/esp32/build/log/idf_py_stdout_output_51215

Could you hep me to fix that issue?

### More Information.

_No response_

It could be caused by that the python package idf_component_manager is not installed in the matter python environment. You can install it and try again.

@wqx6
Isn't it possible to add this dependency in the install script to avoid this error?


Mast: Watch ausweiswebsite.net

Mast requests the watch of the watch_keyword ausweiswebsite\.net. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

ausweiswebsite\.net has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Machavity in Charcoal HQ


Promotion from null breaks type parameters

Here's a fun one that may have the same root cause as #38900. I'm not sure, so filing a separate bug. Given:
class Foo<T> {
  checkNull(T t) {
    if (t == null) throw "!";
    T t2 = t;
  }

  noCheck(T t) {
    T t2 = t;
  }
}

Run:
dartanalyzer --enable-experiment=non-nullable temp.dart

I get:
error • A value of type 'T' can't be assigned to a variable of type 'T' at temp.dart:4:12 • invalid_assignment

It looks like any operation that subtracts null from a type that happens to be a type parameter produces a new type that is considered distinct from the underlying type parameter.

I believe the promoted type should be T & Object (or T & nonNullable(bound) if T has a bound), which should be assignable to T.

Yes, I fixed this as part of https://github.com/dart-lang/sdk/issues/38900.


HBASE-26977 HMaster's ShutdownHook does not take effect, if tablesOnMaster is false

https://issues.apache.org/jira/browse/HBASE-26977

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 46s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 22s
branch-2 passed

+1 :green_heart:
compile
2m 52s
branch-2 passed

+1 :green_heart:
checkstyle
0m 35s
branch-2 passed

+1 :green_heart:
spotbugs
1m 50s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 54s
the patch passed

+1 :green_heart:
compile
2m 43s
the patch passed

+1 :green_heart:
javac
2m 43s
the patch passed

+1 :green_heart:
checkstyle
0m 35s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
8m 55s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
1m 47s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s
The patch does not generate ASF License warnings.

32m 11s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4397/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4397

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux 8431ae572fc3 5.4.0-96-generic #109-Ubuntu SMP Wed Jan 12 16:49:16 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / be8a9a5920

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
64 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4397/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 26s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
2m 48s
branch-2 passed

+1 :green_heart:
compile
0m 46s
branch-2 passed

+1 :green_heart:
shadedjars
3m 49s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 28s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 30s
the patch passed

+1 :green_heart:
compile
0m 46s
the patch passed

+1 :green_heart:
javac
0m 46s
the patch passed

+1 :green_heart:
shadedjars
3m 46s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 26s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
177m 18s
hbase-server in the patch passed.

194m 40s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4397/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4397

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux d0a0d81d81aa 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / be8a9a5920

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4397/1/testReport/

Max. process+thread count
2845 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4397/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 13s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
2m 53s
branch-2 passed

+1 :green_heart:
compile
0m 46s
branch-2 passed

+1 :green_heart:
shadedjars
4m 54s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 29s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 24s
the patch passed

+1 :green_heart:
compile
0m 47s
the patch passed

+1 :green_heart:
javac
0m 47s
the patch passed

+1 :green_heart:
shadedjars
4m 42s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 31s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
200m 26s
hbase-server in the patch passed.

220m 40s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4397/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4397

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 94f8a46a5c1c 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 11 12:03:04 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / be8a9a5920

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4397/1/testReport/

Max. process+thread count
2379 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4397/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

PTAL sir @carp84 , thanks.


Beginning of column modification logic/tests

This is the start of the column modification logic.
I suspect we'll want to add further validation rules about what can and cannot be changed somewhere. Validation rules returned from generate could be an interesting extension. I'm thinking it could return a vector of "Violations" which the publish profile could then "filter" before potentially erroring out if more exist.
This PR ignores changing table constraints at a column level at the moment, e.g. adding UNIQUE or removing UNIQUE at a column level. These are anonymous table level constraints so we'll need to likely extend the AST to support these types as well as loop through them as part of #30.
One other change in this PR is the merging of delta.rs and profiles.rs into a single publish.rs. I've been struggling remembering what each file is responsible for; this is perhaps a step towards association each file with the operation it's closely tied to.
Anyway, keen to hear your thoughts on

So, the delta represents the diff of 2 states, publish_profile is the options used to generate it.
delta was a separate file because it's used for all the operations (script is generating a delta and then turning that into SQL, report is generating the delta and rendering that as a report etc)
Not sure the 2 files should be merged, but not super against it either.

Yeah, I see your point re breaking out delta.rs. Perhaps the logical separation then is:

delta.rs contains the diff logic
sql.rs contains the impl for generating sql (i.e. this may get more complex)
xyz.rs contains the impl for generating xyz

In terms of publish profiles - since they define delta generation they could be kept in the same file? I actually made this all part of a separate commit so it's easy to roll back if need be :)

@justinlatimer I reverted back to keeping the files separate. We can have a chat about the strategy we want to take over slack later on.


Add functionality for flavor aware sentry properties in gradle

Fixes https://github.com/getsentry/react-native-sentry/issues/157
If you pass -Pflavoraware=true to gradle e.g.:
./gradlew -Pflavoraware=true assembleDebug --stacktrace
it produces following output and uses these file in the env.
**********************************
* Flavor aware sentry properties *
**********************************
For demoDebug using: /Users/haza/Projects/react-native-sentry/examples/react-native/AwesomeProject/android/sentry-demoDebug.properties
For demoRelease using: /Users/haza/Projects/react-native-sentry/examples/react-native/AwesomeProject/android/sentry-demoRelease.properties
For fullDebug using: /Users/haza/Projects/react-native-sentry/examples/react-native/AwesomeProject/android/sentry-fullDebug.properties
For fullRelease using: /Users/haza/Projects/react-native-sentry/examples/react-native/AwesomeProject/android/sentry-fullRelease.properties

Sidenote @mitsuhiko
it's a bit weird that sentry-cli expects --url and --auth-token to be before the command and --org and --project to be after the command.

@HazAT that's because not all commands take the same stuff. For instance releases accept more than one project and some other commands only one. Likewise org is optional for quite a few.

@HazAT one more thing that i just realized is that we should make the flavour aware setting a config on that sentry ext instead of command line. Then you can flip it in the gradle file similar to how you can turn on/off the upload.

Switch to ext parameters:
project.ext.sentry = [
        logLevel: "debug",
        flavorAware: true
]

should be before:
apply from: "../../node_modules/react-native-sentry/sentry.gradle"

The PR makes sense to me.
I didn't have any specific ideas for different locations for these files, did anyone else?

i assume that we also want to make the sentry client itself flavour aware.

Would this mean that users would define multiple configuration sets in their single sentry.properties and then choose which set to use with some switch?


Interactive Window at the bottom of the screen

Hi,
Thanks for creating this awesome coding environment!
One feature that I would like to be able to use is placing the Interactive Window at the bottom of the screen. For example, that it is a tab besides the [terminal, problems, output, debug console] options. This would be of value to me since sometimes I have to code on my laptop screen alone, and placing the python file and the interactive window screens side-to-side is not great for readability. This is especially the case if the Explorer view for files is also open, or when reading material on the a webpage that would be convenient to read side-by-side the vscode application.
Kind regards
Stefan

@isidorn has similar suggestion for this too https://github.com/microsoft/vscode/issues/126415#issuecomment-870721935 ❤ .

/cc @greazer

Having this suggestion as well

any updates on this?

hello, is there any update on this?


Should there be a GET end point for getting the data?

It would be easier to access from curl, etc.

e.g. http://<IP_ADDRESS>:8000/reverse_geocode?latitude=1&longitude=2


build: Cannot locate specified Dockerfile: Dockerfile

This is the error I get after updating docker this morning from master and trying to build the Dockerfile in the repo.
michael|~/docker > docker build -t docker .
WARN[0000] Tar: Can't archive a file with includes
Sending build context to Docker daemon 1.536 kB
Error response from daemon: Cannot locate specified Dockerfile: Dockerfile
michael|~/docker > docker version
Client:
 Version:      1.8.0-dev
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   f0f753d
 Built:        Mon Jul 27 16:53:02 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.8.0-dev
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   f0f753d
 Built:        Mon Jul 27 16:53:02 UTC 2015
 OS/Arch:      linux/amd64
michael|~/docker >

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

My vote goes to https://github.com/dmcgowan/docker/commit/578b1521df85eae8a6205118131751c631323ba5

I know the root cause.  It's because my cwd is a symlink ~/docker.  If I do a docker build here it's broken.  If I resolve my cwd to the full path ~/development/gocode/src/github.com/docker/docker docker build works again.

@crosbymichael is this a dup of https://github.com/docker/docker/issues/12842 (and https://github.com/docker/docker/issues/7804)? (Haven't checked thoroughly, but remembered that one)


Upgrade to Next.js 10

The new version of Next.js ships the Image component that will help us to load images on the website

Forgot to create a PR
Done on https://github.com/LauraBeatris/floripamais/commit/7fe7ec59d3864d6c20ac1d5b7e63e006502e4ac8


Deleting key from redis doesn't recreate it on next run

I deleted a key of waiting type from redis via redis-cli to reset the queue, and now mosquito doesn't recreate it automatically and I can't queue any jobs of that type

@greenbigfrog I don’t understand. When a task is moved into Waiting, the redis list is created. If some outside influences deleted the queue, mosquito will lose track of all the tasks it was holding — but there is nothing that can be done about that.
Can you be more specific about the way to reproduce?

mosquito will lose track of all the tasks it was holding — but there is nothing that can be done about that.
That was my intention. I'll describe the way to reproduce soonish


Removed bower.json, .bowerrc, and removed ember-data from package.json

No longer need bower.json or .bowerrc as we're not testing a project. Removed ember-data as a devDep, as I think it would actually override the blueprints in ember-cli-legacy-blueprints, as the ember-data package has priority.

@Turbo87 r+ ?

Appveyor failure was npm cache issue. This should be good to go.

@homu r+

:pushpin: Commit 1c07e15 has been approved by Turbo87

:zap: Test exempted - status


When saving image, create directory if it doesn't exist

Resubmit of #905, that has been closed by OP.
Full props @LucienCorreia for finding out that this very helpful code was missing.

Any updates? 👀

:shipit:


Hamburger spin animation fix

Enables the spin animation of hamburger to arrow and vice versa.
TL;DR :wink:
Watch the video
Merge-able @bguertin @1zaman ?

:+1:

Did we change our minds? Looks like we intentionally disabled it.
We'll also end up disabling it again once we implement the menu correctly (according to Material design, the drawer overlap the toolbar, and the button should not animate)
Anyway, this is fine for now if we want it :+1:

Did we change our minds? Looks like we intentionally disabled it.
I think it had something to do with the irregular behaviour of the previous custom hamburger icon. But i'll dig it and update this thread,

We'll also end up disabling it again once we implement the menu correctly (according to Material design, the drawer should overlap the toolbar, and the button should not animate)
I kinda hate why Material design first introduced and then deprecated it, like its real cool thing to have in an app, improves the outlook at first sight imo, I also vote we keep it as is and dont do the overlapping drawer thing.

I think it had something to do with the irregular behaviour of the previous custom hamburger icon

Oh yeah, that makes sense.

I also vote we keep it as is and dont do the overlapping drawer thing

Well we should commit to one or the other. As it is, our implementation doesn't follow either Material or Holo guidelines for the drawer. With Holo, when opening the drawer, the titlebar is supposed change to the app name and any action buttons should be hidden. (With Material, since the nav appears over the toolbar there's no reason to do that)


Autocomplete bypasses read-only mode

Description
You can insert something from autocomplete menu when viewing document in read-only mode
Reproduction URL
https://typst.app/project/rCaczXS9aP_6THRRXwdBw9
Browsers
Chrome (Vivaldi)
OS
Linux

Fixed.


Add verify imports alias and static check in github ci

Signed-off-by: wuyingjun<EMAIL_ADDRESS>

@Iceber


npm watch bug?

ruthmoog@Ruths-Mac-mini bee % npm run watch

> watch
> npm-watch

No task specified. Will go through all possible tasks
[build] [nodemon] 2.0.22
[build]
src/main.js → static/bundle.js...
[build] created static/bundle.js in 36ms
[build] node:events:489
      throw er; // Unhandled 'error' event
      ^

Error: write EPIPE
    at afterWriteDispatched (node:internal/stream_base_commons:160:15)
    at writeGeneric (node:internal/stream_base_commons:151:3)
    at Socket._writeGeneric (node:net:946:11)
    at Socket._write (node:net:958:8)
    at writeOrBuffer (node:internal/streams/writable:399:12)
    at _write (node:internal/streams/writable:340:10)
    at Writable.write (node:internal/streams/writable:344:10)
    at console.value (node:internal/console/constructor:299:16)
    at console.log (node:internal/console/constructor:376:26)
    at log (/Users/ruthmoog/Computer/bee/node_modules/nodemon/lib/utils/log.js:34:15)
Emitted 'error' event on Socket instance at:
    at emitErrorNT (node:internal/streams/destroy:151:8)
    at emitErrorCloseNT (node:internal/streams/destroy:116:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  errno: -32,
  code: 'EPIPE',
  syscall: 'write'
}

Node.js v20.1.0

Ah, node. How I've missed you.

First step to debugging this is to do some error handling in the builder

ah you can't because it's a black box. BAH.

This command now runs ok, assume it's fixed by @quii 's changes running via the package.json file


[FEATURE] Add 'UNDO' functionality

**Ability To UNDO **
It might be difficult to store all previous actions, but -at least just the previous action- should be able to be undone.
Sometimes a complex node is deleted by accident, the ability to recover that node is super valuable.

Dupe #2290


Provide Message Based Cloud Event Emitter (maybe Kafka)

The Cloud Event Router should provide an extension to be able to receive Cloud Events using different transports. Kafka might be a first good candidate.

Apache Pulsar and GCP PubSub would be more interesting to my case but Im sure its a minority position compared to Kafka.

I would like to take this up

@tomdavidson @masterskipper sounds good to me.. I am wondering how the integration will work out.. as we should be able to support all of them if we have a plan for how the extension will work. I am happy if you guys want to work together on that.

@masterskipper I would recommend listing the steps that you are planning to do before spending time in coding or sending PRs, so we can review it together.

@salaboy  From the description, the problem statement is not quite clear to me. Looks like event routing needs to be made flexible ( not sure what are high level requirements ). any spec you can point to?

can you confirm the requirement?

Event Router should listen as a consumer to a message queue and listen for the events.
There can be different producers but we need to have some particular contract across the two.

@masterskipper let's define the requirements here for clarity:

The Event Router exposes an HTTP endpoint for consuming Cloud Events
The Event Router emits Cloud Events via HTTP

The scope for this issue is to provide a Cloud Event Message-Based emitter that can send Cloud Events via Kafka, using probably Spring Cloud Streams to achieve that. This needs to work with the Knative Broker as the HTTP version is doing.
We cannot add Kafka dependencies straight into this project, which means that we will need to create a separate project (maven module) for the integration that can be added optionally to the cloud event router fat jar.
My suggestion will be to start by adding those dependencies to this project and then we split it up when it is working.
Hopefully, that makes sense, if not please feel free to add questions.


cpp building error on arm64/ubuntu/gcc

Because stdatomic.h is not compatible with cpp on g++, please change the related testcase from cpp to c.
arch: aarch64
system: aws t4g.large
os: Ubuntu 18.04.6 LTS
gcc: 7.5.0
command: ./cppbuild/cppbuild

[ 53%] Building CXX object aeron-client/src/test/c/CMakeFiles/spsc_rb_test.dir/concurrent/aeron_spsc_rb_test.cpp.o
In file included from aeron/aeron-client/src/main/c/concurrent/aeron_atomic64_c11.h:22,
                 from aeron/aeron-client/src/main/c/concurrent/aeron_atomic.h:25,
                 from aeron/aeron-client/src/main/c/concurrent/aeron_rb.h:22,
                 from aeron/aeron-client/src/main/c/concurrent/aeron_spsc_rb.h:20,
                 from aeron/aeron-client/src/test/c/concurrent/aeron_spsc_rb_test.cpp:26:
/usr/lib/gcc/aarch64-linux-gnu/9/include/stdatomic.h:40:9: error: ‘_Atomic’ does not name a type
   40 | typedef _Atomic _Bool atomic_bool;
      |         ^~~~~~~
/usr/lib/gcc/aarch64-linux-gnu/9/include/stdatomic.h:41:9: error: ‘_Atomic’ does not name a type
   41 | typedef _Atomic char atomic_char;
      |         ^~~~~~~
/usr/lib/gcc/aarch64-linux-gnu/9/include/stdatomic.h:42:9: error: ‘_Atomic’ does not name a type
   42 | typedef _Atomic signed char atomic_schar;
      |         ^~~~~~~

Following comments on a GCC bug report, I was able to workaround this compilation problem by building with clang/clang++. It appears clang is either more permissive wrt _Atomic C keyword, more aware of C++20 stdatomic.h standardization?
https://github.com/real-logic/aeron/pull/1291 fixes ARM clang link.
cmake -DCMAKE_C_COMPILER=clang-14 -DCMAKE_CXX_COMPILER=clang++-14 ../..
Jakub Jelinek 2021-08-11 09:58:13 UTC

Note, stdatomic.h is now in C++20.

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60932#c14


Compatibility with Home Assistant 2022.12

Hi,
After installing beta release of Home Assistant 2022.12 pyopensprinkler is not installing automatically.
I have installed it manually, but in logs I see:
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
homeassistant 2022.12.0b0 requires aiohttp==3.8.3, but you have aiohttp 3.8.1 which is incompatible.
I think that aiohttp requirements should be bumped to 3.8.3

same here.

fixed in #70

will do a component release once 2022.12 is released


Investigate why I could add twice the same tp on the same site

after I set the tp numbers the public dids are no longer available in the didsList against the entered did is tested


Pretzel 0.7.0 uses white text for output irregardless of console background

Pretzel 0.7.0 seems to override the console text color to white, without setting a contrasting console background color. See this image:

This seems to be just cosmetic, running "pretzel taste" works, but it's still annoying if you prefer to work in a console with white background.

Sorry, I hadn't think of that possibility.
I will propose a contrasting theme or at least a color visible in white background.

Sorry, no PR; I'm still getting used to GitHub. This change should provide readable colors regardless of console color settings.
internal static class ConsoleTrace
{
    private static Dictionary<TraceLevel, ConsoleColor> Colors = new Dictionary<TraceLevel, ConsoleColor>
         {
             { TraceLevel.Error, ConsoleColor.DarkRed },
             { TraceLevel.Warning, ConsoleColor.DarkYellow },
             { TraceLevel.Info, ConsoleColor.White },
             { TraceLevel.Debug, ConsoleColor.Gray }
         };

    private static readonly HashSet<ConsoleColor> brightConsoleColors = new HashSet<ConsoleColor>
        {
            ConsoleColor.Blue, ConsoleColor.Cyan, ConsoleColor.Gray, ConsoleColor.Green,
            ConsoleColor.Magenta, ConsoleColor.Red, ConsoleColor.White, ConsoleColor.Yellow
        };

    static ConsoleTrace()
    {
        // If console uses bright background, adjust text colors.
        if (brightConsoleColors.Contains(Console.BackgroundColor))
        {
            Colors[TraceLevel.Info] = ConsoleColor.Black;
            Colors[TraceLevel.Debug] = ConsoleColor.DarkGray;
        }
    }

// ...

}

Thanks, I will propose a new version with this.

V0.7.1 with the fix have been pushed to GitHub releases and Chocolatey.


Timer with arbitrary number of events in mark

Is there a reason not to expose the amount of events in the Timer#update() method?
Something like:
   /**
     * Adds a recorded duration.
     *
     * @param duration the length of the duration
     * @param unit     the scale unit of {@code duration}
     * @param n        the number of events
     */
    public void update(long duration, TimeUnit unit, long n) {
        update(unit.toNanos(duration), n);
    }

    }

    private void update(long duration, long n) {
        if (duration >= 0) {
            histogram.update(duration);
            meter.mark(n);
        }
    }

I'm guessing this is just a rare use case so not sure it makes sense on the standard API, maybe better fit for a custom utility method that just calls update N amount of times. Are you looking to only update the meter though but not the histogram N amount of times? The percentiles would be off then because the duration would only be counted once but maybe that's what you're looking for, what is your use case?

I have a method that run every x minutes. The method moves around objects and I would like to measure the throughput, but relative to the time it executed. Does that make sense?

I see what you mean, but @ryanrupp is correct that this doesn't appear to be a sufficiently common use case as to justify inclusion in the API.
Could I recommend maintaining both a Timer and a Meter? That way you'll be recording the count and rate of the method invocation, the method's execution time, and the count and rate of the events/objects processed.

I will try out using my own Meter objects yes. Thank you.

The use case is quite common when you are doing batch processing with the variable size of a batch and you want to see if there is a gain in making batch smaller or bigger (latency can be measured only as a whole and then needs to be divided). Well just need to write me own timer.


Elasticsearch ELSER Timeout

System Info
Python 3.10.12
Google Colab
Elasticsearch Cloud 8.9.2
Langchain - latest
Who can help?
@hwchase17 @agola11
Information

[X] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[X] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Steps:

Load list of documents
Setup ElasticsearchStore of Langchain, with appropriate ES cloud credentials
Successfully create index with custom embedding model (HF embedding model, deployed on colab)
Deploy ELSER model and run it (with default model id).
Try creating index with SparseVectorRetrievalStrategy (ELSER) over the same list of documents.
Tried to change timeout, but didn't effect the outcome.
NOTE: It does start uploading docs and docs count is increasing, but it stops after about 10 sec. I tried to run the ELSER model on 3 nodes, but nothing changed.

Expected behavior
WARNING:elastic_transport.node_pool:Node <Urllib3HttpNode(https://-------.us-central1.gcp.cloud.es.io:443)> has failed for 1 times in a row, putting on 1 second timeout
ConnectionTimeout                         Traceback (most recent call last)
 in <cell line: 1>()
----> 1 elastic_elser_search = ElasticsearchStore.from_documents(
2     documents=split_texts,
3     es_cloud_id="cloudid",
4     index_name="search-tmd-elser",
5     es_user="elastic",
10 frames
/usr/local/lib/python3.10/dist-packages/elastic_transport/_node/_http_urllib3.py in perform_request(self, method, target, body, headers, request_timeout)
197                 exception=err,
198             )
--> 199             raise err from None
200
201         meta = ApiResponseMeta(
ConnectionTimeout: Connection timed out

Thanks for report. ELSER is taking a while to transform the text so the bulk import is timing out. I will add a feature to allow the developer to change the default "chunk" batch size from 500 so that it reduces hitting the timeout.

Thanks for the response. I am looking forward to using this feature.

@DrSebastianK within 303, you can now pass in bulk args when ingesting documents, to reduce hitting the timeout. See documentation on how to do this 😀


Reset Layout not working latest version 0.0.55

const CollapseLeftSide = () => {
  const panelGroup = refhorizontal.current;
  const currentLayout = panelGroup.getLayout("percentages");
  panelGroup.setLayout([0, 100], "percentages");
};

const CollapseRightSide = () => {
  const panelGroup = refhorizontal.current;
  const currentLayout = panelGroup.getLayout("percentages");
  panelGroup.setLayout([100, 0], "percentages");
};

<PanelGroup
  direction="horizontal"
  ref={refhorizontal}
  disablePointerEventsDuringResize={true}
  units="percentages"
>
  <Panel ref={ref2} collapsible={false} className="Panel-panel" minSize={50}>

      <LeftsidePanel />

  </Panel>
  <ResizeHandle
    CollapseLeftSide={CollapseLeftSide}
    CollapseRightSide={CollapseRightSide}
  />
  <Panel ref={ref3} collapsible={false} className="Panel-panel" minSize={20}>

      <RightsidePanel />

  </Panel>
</PanelGroup>

I am getting error console log "Invalid size (0) specified for Panel ":r1:" given the panel's min/max size constraints"

Edited by @bvaughn to fix formatting.

Both panels are collapsible={false} and minSize={50} so 0 is an invalid size.

I don't know what the issue is. I'm just pointing out that you can't set a panel's size to 0 if you've explicitly said the minimum size is 50 and the panel isn't collapsible. That won't work.

Initially I used Sync Fusion Splitter plugin, but I used its paid version, so I switched to resizable panel. Earlier it was working but when I update latest version 0.0.55 , Its not working
When I click right or left icon, the panel should collapse, this is my problem, it doesn't collapse in latest version.
https://github.com/bvaughn/react-resizable-panels/assets/56765556/48a0c659-467c-4456-ad4f-46560fb5bc81 - version 0.0.54
https://github.com/bvaughn/react-resizable-panels/assets/56765556/740fb47f-7e6f-4f94-bef8-bdc5d1873dcf  - latest version 0.0.055
Kindly give solution

I think you might have better luck asking a question like this on Stack Overflow or somewhere like reddit.com/r/react.
I've tried to point out what's wrong above but it seems like we're talking past each other somehow.
If you want a panel to be collapsible (so its size can be 0) then set collapsible={true} instead of collapsible={false}.