RefreshControl on initial render will not beginRefresh when refreshing state is false

There's an edge case in the RefreshControl which causes it to show as refreshing when the state is set to false. When the component is initialized with refreshing set to true on initial render but set to false before layoutSubviews is called, it will call beginRefresh and ignore its state. That's because layoutSubviews never checks if _currentRefreshingState is in fact still true, it merely assumes it is.
This is fixed by simply doing a check for _currentRefreshingState before entering beginRefresh from layoutSubviews.

By analyzing the blame information on this pull request, we identified @janicduplessis and @skv-headless to be potential reviewers.

Thanks @aforty !
@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


[FLINK-12697] [State Backends] Support on-disk state storage for spill-able heap backend

What is the purpose of the change
This PR aims at introducing a compacted, on disk format for state storage. The design thoughts are described in the FLIP-50 document.
Brief change log
Mainly introducing the CopyOnWriteSkipListStateMap which is a state map based on a compacted skip list data structure and supporting copy on write. Also introduces the Allocator and Chunk interfaces for the off-heap space allocation and maintenance.
Verifying this change
This change added tests and can be verified with the newly added tests in the org.apache.flink.runtime.state.heap package.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not documented yet)

There is some extra Unsafe Helper logic in this PR. Could we consolidate this with the MemoryUtils class in Flink?
For when we need to do the Unsafe / VarHandle migration for Java 12+, it would be great if that logic was as consolidated as possible.

Thanks for the review @StephanEwen @tillrohrmann ! I've addressed most of the comments and left some questions open.
For the Unsafe helper, since merging it into MemoryUtils now will cause quite some code refactor, how about we move the UnsafeHelp and ByteBufferUtils into flink-core module under org.apache.flink.core.memory package first, and then align all Unsafe/VarHandle in another PR? Thanks.

Thanks for the continuous efforts on review @tillrohrmann ! Before addressing detailed comments, let me answer the general ones first:

About thread safety of the implementation: the testConcurrentSnapshots, testCopyOnWriteContracts and testSnapshotPruneValues cases in CopyOnWriteSkipListStateMapTest are designed to cover the possible concurrency with asynchronous checkpoint. There might be one or two fields could be volatile but not, but checking the logic those are all nice to improve but won't affect the correctness of logic even if keep them as is. Will talk more when addressing the specific comments.
About the design on translation from node pointer to ByteBuffer and offset, yes it's on purpose for performance consideration. If we construct a "Node" object (for example) every time, the cost of constructing object is relatively big comparing to "address"-only operations (and note that those address operations cannot be saved for off-heap data structure even if we have an abstraction of Node object). What's more, we don't need all fields of a Node for each operation, such as locating which Node a given key belongs to, we only need to compare keys until finding the target, all other "fields" of a Node is not needed during the process.

Thanks for the general answers @carp84.

Ok, then I guess that CopyOnWriteSkipListStateMap.stateSnapshot is only being called under the checkpoint lock? If this is the case, then it should be ok. Maybe a clarifying comment could help here.

That's what I was suspecting. I don't wanna necessarily change it but do we know how much performance we gain by this? I think there would have been also other solutions to the problem. For example, one could have a reusable object which has a mutable nodeAddress field. Then one would not have to create a new object every time. Ideally one would hide this behind a factory method which can decide how to handle it. Also the access to the field could have been encapsulated by the object and if needed could have happened lazily. The benefit of this approach would be that it is easier to maintain and test because the API would tell you what such an object can do and we would have a dedicated type instead of longs you have to pass around.

That's what I was suspecting. I don't wanna necessarily change it but do we know how much performance we gain by this? I think there would have been also other solutions to the problem. For example, one could have a reusable object which has a mutable nodeAddress field. Then one would not have to create a new object every time. Ideally one would hide this behind a factory method which can decide how to handle it. Also the access to the field could have been encapsulated by the object and if needed could have happened lazily. The benefit of this approach would be that it is easier to maintain and test because the API would tell you what such an object can do and we would have a dedicated type instead of longs you have to pass around.

About the performance impact, lease refer to the analysis of JDK CSLM implementation and a compacted data structure we introduced for HBase to reduce GC pressure. Search for Key space schema and Value space schema in SkipListUtils and we could find a similar design here.
About reusable object, it will add a lot of efforts/complexity making sure to prevent concurrent manipulation on it.

About the performance impact, lease refer to the analysis of JDK CSLM implementation and a compacted data structure we introduced for HBase to reduce GC pressure. Search for Key space schema and Value space schema in SkipListUtils and we could find a similar design here.

Where exactly do I see the performance comparison?

About reusable object, it will add a lot of efforts/complexity making sure to prevent concurrent manipulation on it.

Why would this be the case? The only accessing threads should be the Task's main thread and the asynchronous checkpointing, right? Couldn't we say that the asynchronous checkpointing creates one single instance and reuses this instance for the whole checkpointing procedure? One could make it even a thread local variable if one wants to have an easy solution. So I'm not sure where the argument comes from that using a thin wrapping object around a pointer will necessarily decrease performance.

Where exactly do I see the performance comparison?
Allow me to quote the conclusion: "we need 62.5% extra objects and 100% extra memory when using CSLM", and more detailed analysis please refer to the google doc.

Why would this be the case? The only accessing threads should be the Task's main thread and the asynchronous checkpointing, right? Couldn't we say that the asynchronous checkpointing creates one single instance and reuses this instance for the whole checkpointing procedure? One could make it even a thread local variable if one wants to have an easy solution. So I'm not sure where the argument comes from that using a thin wrapping object around a pointer will necessarily decrease performance.

When talking about wrapping a Node object and reuse it in every node manipulation, it's no more single thread accessed, because there might be multiple checkpoints ongoing in parallel. Considering we're using the copy-on-write mechanism, there might be multiple versions (or say a value chain) for a single key, which will make reusing a wrapper really tricky. I'm afraid it will not only increase the complexity of implementation but also even worsen readability, on the contrary to the intuition. As we all know, the killer is not the 99% cases, but the 1% corner case (smile).

bq. Some of the tests seem to be a bit convoluted with a lot of indirections which are not easy to understand.
Could you give me some example about such "convoluted with indirections"? Is it possible that the original CopyOnWriteSkipListStateMapTest is better regardless of the test case split up part?
I've addressed all comments on CopyOnWriteSkipListStateMap and will get back on CopyOnWriteSkipListStateMapTest after getting some response. Thanks.

Thanks for the review @StephanEwen !
Checking the code I agree that there're similar goals between ByteBufferUtils and MemorySegment, will give it a more careful check about whether we could merge them.
I admit it's my bad to directly borrow the classes (UnsafeAccess and ByteBufferUtils) from hbase and lack of knowledge about whether there's already similar functionalities in Flink. I checked MemoryUtils and didn't find enough util methods so goes the "easy" way. And the test was also following the hbase way (TestByteBufferUtils), that covering all logic through ByteBufferUtilsTest instead of introducing an UnsafeHelpTest.
Since the unsafe tool and unaligned logic in hbase has been used for a long time in different environments, I believe it should work fine, while I agree that it's better to have a unit test to cover the correctness checking.
Will get back soon with updates after more thinking.
PS. I'm still learning to switch between HBase and Flink code convention and hopefully this is not causing big problem, and my apology if already any (pray).

@StephanEwen here comes some more detailed response for the review questions:
About the 2nd question about unaligned and its testing, from the jdk source code and os.name/os.arch mapping we could know most of our tests actually fall into the if (UNSAFE_UNALIGNED) clause (also confirmed through test coverage tools), and the else clause calls ByteBuffer.getXXX method which JVM itself will assure the correctness, so I think we don't need to worry about the "dead path"
About the 4th question, checking the source code of DirectByteBuffer.getLong we could know that it checks the unaligned and big/little endian again, so yes in our implementation way calling UNSAFE.getLong(ByteBufferUtils.getBufferAddress(buf) + offset); is really faster than DirectByteBuffer.getLong(offset);

@carp84 did you have time to investigate how much effort it would be to replace the ByteBuffer usage with MemorySegments in order to unify the unsafe utils? Based on that we can make the decision how to continue with this open question.
What we should also do before going forward with this PR is to address Stephan's comments concerning the serializers. Let me know once the last open questions have been resolved.

Sorry for the late response @tillrohrmann .
By comparing the current implementation of ByteBufferUtils and MemorySegment, I think most fundamental methods (putPrimitive, getPrimitive, compareTo, copyTo, etc.) are already included in MemorySegment, and I agree it's no big deal not to support operating systems that don't support unaligned memory access (thus we don't need those branches for Unaligned case in ByteBufferUtils), so overall it's definitely workable to replace ByteBuffer usage with MemorySegment.
Regarding the efforts of the merge, there're mainly two aspects from my point of view:

We may still need some modifications on MemorySegment. This is to some extent conflict with our original intention of making spill-able backend a standalone module to prevent impact on existing components, so I'm a little bit concerned.

In the current design we map the file segment into memory (in format of MappedByteBuffer) and use it as the base of a "Chunk", furthermore splitting it into "Buckets" for finer-grained usage, so everything in-flight is ByteBuffer. Replacing ByteBuffer with MemorySegment requires many interface changes in CopyOnWriteSkipListStateMap and SpaceAllocator.

I have made a new commit to address @StephanEwen 's comments on value serializer. I also tried to modify key serializer with DataOutputSerializer and DataInputDeserializer but found it may not be as efficient due to two additional System.arrayCopy w/o changing the existing design of position manipulation in DataOutputSerializer (currently we can only set the position after data ingestion and don't allow "holes"). Attached is the patch I made for PoC, please check it and let me know your thoughts. Thanks.
use_data_input_output_serializer_in_skiplist_key_serializer.patch.txt

I'm wondering whether it would be easier to do it now as we are opening the PRs and can update them bit by bit or after everything has been merged. I fear that the inertia might be a bit higher if everything is in place and a much bigger pile of code would needed to be touched. But I'm also ok to do it later if you say that this is not a problem.

Thanks for sharing the thoughts @tillrohrmann , I think the concern makes sense.
I have been preparing the commit of replacing ByteBuffer with MemorySegment for some time and have just completed the work. It turns out only one new method required for MemorySegment and no modifications for the existing one (we could open another PR if would like to add the hbase-like padding method for operating systems w/o unaligned memory access support, which is not that urgent), so the impact on old components is within control. Please check the new commit for more details.
I also checked the integration with the not-committed-yet SpaceAllocator using MemorySegment and confirmed everything works fine (please check the PoC branch for details), will update the SpaceAllocator PR soon if the changes here look good.

Closing PR since it's already merged through commit 525f3c9c45.
Thanks for reviewing and merging the PR @StephanEwen , and thanks @tillrohrmann for the review!


Summa Smart Contract deals with time stamps

Add timestamp which indicates the moment in which the merkle sum tree of liabilities was created
Add block_hashes which indicates the moment in which the proof of assets is taken. It should be the next available block hash at timestamp

Normally, this is not done on blockchain because this is metadata available for any blockchain indexer. I.e., when frontends/backends work with the emitted contract events, they can easily read their timestamps (or block hashes). Do we need this info in the contract for future computations or for other contracts to read?

Let me rephrase it since the issue was poorly explained.
The origin of the issue is that we need to make sure that the snapshot for assets and liabilities is taken at the same time.
The way it works right now is that the liabilities are snapshotted at the moment in which we create the csv file containing the list of users and their balances. The assets instead are fetched by the contract at the current blockhash. So this creates a misalignement. An attendee of my presentation in Paris also recognized this issue.
Ideally, to solve this we can add to the π of Solvency a UNIX liabilities_timestamp that indicates the time in which the csv file for the liabilities was snapshotted.
When the proof gets sent for verification to the Summa Smart Contract the various balances are not fetched at the current block but at the very next block available at liabilities_timestamp.
https://github.com/summa-dev/summa-solvency/blob/ded7155c585eabef653bfc84ec26ca10a6acd632/contracts/src/Summa.sol#L96-L97

Let me translate it into a new discussed architecture:
/**
 * @dev Interface for asset balance retriever contracts
 */
interface IBalanceRetriever {
    /**
     * @dev Get the balance of an address for a given asset type. Gets address balance for ETH, balanceOf() for ERC20, or calls an oracle for non-native assets
     * @param _address The address to check the balance of
     * @param args Additional arguments needed to get the balance (e.g., in case of ERC20, the address of the ERC20 contract)
     * @return The balance of the address for the given asset type
     */
    function getAddressBalance(
        bytes memory _address,
        bytes memory args
    ) external view returns (uint256);
}

We need to modify this to have the timestamp as an argument (e.g., for future implementation using Axiom):
interface IBalanceRetriever {
    function getAddressBalance(
        bytes memory _address,
        bytes memory args,
        uint256 timestamp
    ) external view returns (uint256);
}

Is that correct?

Yep that's correct!

Implemented in #131.


Code snippet for get subscriber is not shown

Hi @andrewgolovanov,
I have added this code snippet:
https://github.com/novuhq/api-docs/blob/main/content/code-examples/subscribers/get.md
But it does not show on the api docs... Can you please look into why this is not working?

Hi @davidsoderberg !
I think I figured out what the reason is :sparkles:
We get links as - /v1/subscribers/{subscriberId} on input from Swagger and use as an identifier to filter examples, instead of /v1/subscribers/:subscriberId - which we show on pages
I added a fix that allows you to use your variant when specifying an id


Reference to commits in issues

Hello !
The more I use Gogs, the more I like it.
I just found a missing feature (as of Version <IP_ADDRESS>0 Beta) : seeing a commit in an issue.
Let me explain : I open an issue, that have the number '#num'.
Then I mention the issue number in a git commit message, like "this commit corrects issue #num".
When I go to the "commits" view, I see the commit message and the issue number is a link to the correct issue page :

But on that issue page, there is no reference to the commit.
Like in github :

Subsidiary question : is it possible to close an issue with the presence of certain keywords in the commit message ? like "this commit closes #num", with the word "closes" set in configuration to close an issue if used in commit message followed by #num
Thanks in advance.

It is somehow implemented a while ago...


Excess curly braces

Description
There's an excess curly brace on line 9 in the charts/haproxy/templates/clusterrole.yaml
There's an excess curly brace on line 32,34,35 in the charts/reticulum/templates/persistentvolume.yaml
This issue may lead to syntax errors in the creation of the clusterrole and pvs
example in clusterrole:
  verbs:
  - get
  - list
  - watch}

which removes the last verb.

Thanks!

merged #9


not compatible with mlflow v1.28.0

when run workflow:
qrun ALSTM_workflow_config_alstm_Alpha158.yaml

mlflow v1.27.0 work fine,but failed when with mlflow v1.28.0:
File "miniconda3/envs/qlibdev/lib/python3.8/site-packages/pyqlib-<IP_ADDRESS>-py3.8-linux-x86_64.egg/qlib/workflow/recorder.py", line 441, in log_params
    self.client.log_param(self.id, name, data)
  File "miniconda3/envs/qlibdev/lib/python3.8/site-packages/mlflow-1.28.0-py3.8.egg/mlflow/tracking/client.py", line 852, in log_param
    self._tracking_client.log_param(run_id, key, value)
  File "miniconda3/envs/qlibdev/lib/python3.8/site-packages/mlflow-1.28.0-py3.8.egg/mlflow/tracking/_tracking_service/client.py", line 305, in log_param
    raise MlflowException(msg, INVALID_PARAMETER_VALUE)
mlflow.exceptions.MlflowException: Param value '[{'class': 'SignalRecord', 'module_path': 'qlib.workflow.record_temp', 'kwargs': {'model': '<MODEL>', 'dataset': '<DATASET>'}}, {'class': 'SigAnaRecord', 'module_path': 'qlib.workflow.record_temp', 'kwargs': {'ana_long_short': False, 'ann_scaler': 25' had length 778, which exceeded length limit of 500

i think the new mflow feature cause this bug.mlflow limit param valu lengh to 500,by read code ,it can not be overwrite.
maybe relate with this issue

Can this PR solve your problem?


Wiki pages fix

Fixed #2 by making readme selector for wiki pages more restrictive.
Passed all tests

@arthurhammer Have you had a chance to look at this yet?

Thanks for this!


Linguistic Analysis API is outdated

Linguistic Analysis API is no longer available in v2. Please correct me if wrong. If I am correct, please stop listing it on a page that was updated in 2018.

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

ID: a5ead4a3-3e74-b543-c686-73cf3c899bfb
Version Independent ID: 17980270-d395-92b6-36c6-837ce9576f90
Content: What are Azure Cognitive Services?
Content Source: articles/cognitive-services/Welcome.md
Service: cognitive-services
GitHub Login: @nitinme
Microsoft Alias: nitinme

@goluhaque Please keep eyes on #15161 Thanks. We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.


2017B CORS support added for easier configuration

According to https://github.com/apollographql/apollo-client/issues/653

@ojczeo Thank you.
Do you have any other problems or ideas for the faker?
I'm very interested to know how people use this tool.

Added CORS for less integration problems with custom apps in development.

Do you use it to develop frontend app?
What client lib(Apollo, React) do you use?
You fake entire API or use extend mode to do add fields into existing objects?

@IvanGoncharov GraphQL faker is great tool! It very speeds up development of web apps with distributed services (if GraphQL api not working or is sitll in development).
I've used it as a fake api for my app, and it works very well.
One disadvantage - if mutations could return consistient data (based on mutation input) it would be incredible useful, or if it save mutation somewhere...
I'm using Apollo.

One disadvantage - if mutations could return consistient data (based on mutation input) it would be incredible useful, or if it save mutation somewhere...

@ojczeo I started to work on this and would try to commit today.
Just to be sure, do you mean something like this?
mutation {
  testMutation(input: {testStr: "Hello"}) {
    testStr
    anotherStr
  }
}

And result would be:
{
   "testStr": "Hello",
   "anotherStr": "string"
}

My assumption is that you are using Relay style mutation where data passed through single input object named "input". Is it correct?

@IvanGoncharov Yes, I've meant it exacly like that!
And yes, I'm using single input as mutation input.
It's very nice! Looking forward for update!

@ojczeo I already pushed changes to the repo.
The only issue I see so far is aliases:
mutation {
  test(input:{testStr: "Hello"}) {
    testAlias: testStr
    anotherStr
  }
}

Produce:
{
  "test": {
    "testStr": "Hello",
    "anotherStr": "string"
  }
}

Not sure if there are other issues, so it would be great if you can test this changes before we publish a new version.

@ojczeo I published new release with your changes and mutation trick 🎉


Try libtmt as an alternative to libtsm

libtmt has a similar purpose to libtsm, but it is smaller and simpler.
Some current issues such as wide character support (#17) might be resolved by switching to libtmt.
But there are some limitations such as lack of mouse support (#4).
In any case, I think it is worth plugging in and trying.

The libtmt library is ok, but the libtsm is performing much better now after d2f073d7aed9a9f5de790b1463c563b05f0707ac and the upgrades to libtsm. And libtmt lacks too many features in comparison with libtsm (such as true color support). So I will stick with libtsm.
The libtmt code is quite small though (just over 500 lines of C, including comments and whitespace). So one idea might be to port libtmt to gdscript as a non GDNative fallback.

The libtmt library is ok, but the libtsm is performing much better now after d2f073d and the upgrades to libtsm. And libtmt lacks too many features in comparison with libtsm (such as true color support). So I will stick with libtsm.
The libtmt code is quite small though (just over 500 lines of C, including comments and whitespace). So one idea might be to port libtmt to gdscript as a non GDNative fallback.

If this port hasn't been made already, I'd love to help (or even port it myself).

If this port hasn't been made already, I'd love to help (or even port it myself).

@supremestdoggo Yes, feel free to have a go at porting it.
I attempted a port previously, but don't have the files anymore sorry. The main problem I encountered was libmt's heavy use of macros which I found difficult to translate into GDScript. I hope you might have more success!

The (currently incomplete) port's here. Also, I managed to find a macro preprocessor that works with any language (AnyMacro), so the port does use macros.


❓ [addon] Wger ssl support

Which addon?
Wger

Addon or app?
Addon

Describe the solution you'd like
I would like to use my ssl certificat from home assistant to the addon, but unfortaly i didn't see any information how to do

Describe alternatives you've considered
None

Additional context
None

Hi, the new way of working proposed by homeassistant is that you install the nginx proxy manager according to this very short guide on adding ssl to the addons : https://github.com/alexbelgium/hassio-addons/wiki/Add‐ons-feature-:-ssl
I won't add anymore ssl to individual addons as it is just less optimal

Thanks for reply I will do it
Do you have any idea why I'm always being logged-off and getting this in log :
WARNING 2023-11-29 20:28:43,153 log The request's session was deleted before the request completed. The user may have logged out in a concurrent request, for example.: /api/v2/exercisebaseinfo/
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/django/contrib/sessions/middleware.py", line 59, in process_response
request.session.save()
File "/usr/local/lib/python3.10/dist-packages/django/contrib/sessions/backends/cache.py", line 62, in save
raise UpdateError
django.contrib.sessions.backends.base.UpdateError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/django/core/handlers/exception.py", line 55, in inner
response = get_response(request)
File "/usr/local/lib/python3.10/dist-packages/django/utils/deprecation.py", line 136, in call
response = self.process_response(request, response)
File "/usr/local/lib/python3.10/dist-packages/django/contrib/sessions/middleware.py", line 61, in process_response
raise SessionInterrupted(
django.contrib.sessions.exceptions.SessionInterrupted: The request's session was deleted before the request completed. The user may have logged out in a concurrent request, for example.


Cars pass on red light

Hello! I found a bug that is: Cars pass on red lights on timed traffic lights. Here is the link for the video: https://www.youtube.com/watch?v=0K86-TKUoq8

Segments are too short (doesn't matter how big node you make). Lower speed limit


Possible to show more than one carousel item at a time?

Looking through the docs it doesn't seem possible to have more than one item visible at a time.
However, the "buffered ngRepeat with custom transition" demo does show the previous and following items, so it seems possible in theory.
Is it possible to to show more? And if not, consider this a feature request!
I'm looking to migrate away from jQuery where I use the Slick carousel with multiple items:
http://kenwheeler.github.io/slick/

if you arrange your images by "page" you can do it. in this example i was displaying multiples items at once : https://github.com/revolunet/angular-carousel/blob/master/demo/flickr-full/index.html#L85-L95


Transcoder Reward Call Alerting App

What's the problem? (required)
Transcoders have to make sure they call reward for every round in the protocol.  This logic is built into the go-livepeer codebase so it should be done automatically.  However, a few different reasons can stop the automated calling behavior in the Livepeer node.
What's the current behavior? (required)
Nothing happens if a transcoder misses a reward call.
Describe a possible solution you've considered (optional)
There should be a monitoring tool that sends out alerts if transcoders miss reward calls.  This tool can start with the following feature:

Allow anyone to opt into an email alert for when any transcoder has not called reward for the current round
This email alert can be scheduled for a specific time everyday.

Additional context (optional)
The livepeerjs sdk gives you an easy way to query the information that lives on the blockchain.  There are examples in the explorer package (which uses the graphql sdk to interact with the blockchain and load data).
Note for bounty hunter
The tool can live as a stand-alone app, using APIs like Mailgun to send out emails.

I am working  on this issue yet.  for resolve this issue, I need to review all functions. So it will take some time.

@iSasFTW I'd certainly be interested in using a productized version of this. One example that we've seen so far is this simple SMS alerting app from Bison Trails: https://gist.github.com/aaronhenshaw/d3c206a07fd86e286da3974da22979de
But there's probably a different take on the productization of this - how and when are alerts delivered, how do I opt in/out, etc. I'd rather sign up for a service that manages the reliability of this than run a script for example (and I'd be willing to pay for that!)

Hey @bleushark1988, Ryan from Gitcoin here. Are you still working on this bounty? Please let @dob know if so! 👍 🔨

@dob That sounds good. I was mostly thinking about just adding a new package to livepeerjs, that monitors the chain and sends out emails, like the original issue described. Do you think there's still a need for something like that?

I suspect the value is in the service that performs this for you moreso
than code that's available to setup and run yourself. So for example a PR
into livepeer.js might be a good starting point, but until someone actually
runs the service reliably 24/7, I think transcoders will still miss reward
calls without realizing.
On Thu, Oct 25, 2018 at 4:06 PM Saska Kolehmainen<EMAIL_ADDRESS>wrote:

@dob https://github.com/dob That sounds good. I was mostly thinking
about just adding a new package to livepeerjs, that monitors the chain and
sends out emails, like the original issue described. Do you think there's
still a need for something like that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/livepeer/livepeerjs/issues/158#issuecomment-433064801,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAolah3cMuEdVZRliNj4wypQViYa8zvks5uocVhgaJpZM4VlFZf
.

Sorry guys.  I can't know issues yet.

Hey @dob, Ryan from Gitcoin here. Looks like @bleushark1988 isn't working on this one -- do you want to approve @iSasFTW to work on it?

Sure, we can remove @bleushark1988, but it looks like as the funder maybe only @ericxtang can do it?

@ericxtang what's happening with this issue?

I think we can close this issue.  There is a grant for this now and someone is working on it.


Update Statistics link on homepage

Update statistics link on homepage to point to the shiny new finder search/research-and-statistics
https://trello.com/c/EEJupWMu/516-replace-stats-link-on-homepage

@barrucadu I've got the brakeman error you had earlier https://github.com/alphagov/specialist-publisher/pull/1471. Shall I follow the same steps to ignore it?

I don't think there are any cookies here we care about breaking, but yes, best to be safe.


Add support for GitHub Enterprise

Fixes #315
New domains can be specified in GH's options view, which can also help #193, should that ever materialize.

To test this without access to GHE:

remove all the permissions to github.com in manifest.json
disable and re-enable the extension (don't just reload it)
Verify the permissions, github.com should not be there
Add https://github.com via the form

@bfred-it this is gold.
@sindresorhus could we get some eyes on this? I'd love to have it merged as soon as possible.

Should there be an option to see currently added domains and remove them if no longer needed?

@busches I don't think it's a big deal, worst thing that can happen is that a domain you no longer use takes 20 bytes on disk, I think.

Alternative style:

@bfred-it Very nice work! Tested this on GHE and the actual permission stuff worked great. All of the css changes worked without issue.
However the javascript doesn't fire at all because this is loading after the page loaded and the javascript changes are done using a listen on the DOMContentLoaded event and not taking into account if it's loaded after that event has triggered. Should be easy to fix.  content.js should probably be executed last (right now there are a few after it and just calling the content loaded function errors because of this)
It also looks like right now you're guaranteeing execution order - the javascript is noticeably slower to load on the GHE sites as well and this may be why. We should be able to load all of them async except for content.js and load that after, right?

Thanks for the info! I'll apply the two fixes (domready and content.js loaded last) but the speed will automatically be improved once #404 is in and only a file is loaded.

I don't use GHE, so this PR is out of my reach I think. @hkdobrev maybe? Or @jgierer12?

Applied the 3 suggested changes and rebased.

Nope, I don't use GHE either

@bfred-it Got it
Confirming this is working for me on GHE. Also note you can see what sites you gave permission to in the settings for the extension.

Thanks @derimagia! Hopefully this can be merged/squashed soon :)

This works well for the most part. There are still a lot of places that have github.com hardcoded, such as branch names, reaction avatars, show names, etc. These links do not work when you're on another domain as they incorrectly go to the wrong URL. This is a larger undertaking, so I can see it being moved out to another PR, but is core to supporting other domains.

@busches do you see value in merging this now to get at least some features working while further and specific issues can be reported and fixed later?

@bfred-it yes. We'd just need to create an issue for what I mentioned above.

Cool. There's a chance that some features won't ever be available or compatible with GHE, but custom domain support at least gives a basis for other to find and fix issues too.

@bfred-it I'd be happy to work with you as needed, what features aren't compatible with it would you say?

@derimagia anything that has github.com hardcoded, instead of using something like location.origin:

Linkify Branch References
Add Delete Fork Link
isGist function detection
Reaction Avatar images
Showing Real Name next to username
Linkify issues

There could be more, but from a quick glance at the code and attempting them, they don't work.

Sorry I mean't this part of it: "Some features won't ever be available or compatible with GHE" - is it that they can't be done or that we would need to fix the hard code locations?

I don't have GHE so I', only guessing, considering that GHE is not 1:1 with GH.com

The URLs probably only need to be changed from https://github.com/something to /something. But that can come in a different PR and further testing

@busches Can you open an issue about further improvements to GHE support?

@bfred-it Can you mention GHE support in the readme?

Just a heads-up, this doesn't work on Firefox since it doesn't allow changing permissions dynamically. I don't think there's much we can do about it currently, so maybe mention this in the readme too @bfred-it

Pinging some people who showed interest in Refined GitHub Enterprise:

@nobso in #318
@mwakerman, @thefotios in #315
@zafarella in #81

Would you mind giving it a go and reporting new issues?

Thank you so much for adding the support. I was adding the GHE URL to manifest file as workaround.
Works for me but only after installing it from store. When loading locally, it does not work.
Extension was this red box
There were warnings when trying to install this extension:
Unrecognized manifest key 'applications'.

I pulled latest master and:
npm  install
npm run build

You don't need to change the manifest anymore, you can just set your domain in the options:


AlexNet & VGG network implementation.

Addition: AlexNet 7.1 & VGG 7.2 Implementation.

Can you add notebook or folder to CI test if they did not require a long time to run on CPU?


css-gather: Update README

This just updates the README to the new usage using docker.
It's as simple as building an image and executing the correct
script. We just pass the url to the run.rb script in the
container.
Closes: https://github.com/iFixit/ifixit/issues/46407

QA 🟢
LGTM


Bug with wrong decimal separator

I have very simple model:
[Document(StorageType = StorageType.Hash, Prefixes = new[] { nameof(TestModel) })]
public class TestModel
{
    [Indexed]
    [RedisIdField]
    public Guid Id { get; set; }

    // other string properties

    public decimal SessionIndex { get; set; }
}

I have a char , in my Thread.CurrentThread.CurrentCulture.NumberFormat.NumberDecimalSeparator.
I can save data to Redis without nay issues. But this data will be saved with my comma separator.
And when I try to read the same model I have an exception:
System.AggregateException
  HResult=0x80131500
  Message=One or more errors occurred. ('5' is an invalid start of a property name. Expected a '"'. Path: $ | LineNumber: 0 | BytePositionInLine: 386.)
  Source=System.Private.CoreLib
  StackTrace:
   at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
   at System.Threading.Tasks.Task`1.GetResultCore(Boolean waitCompletionNotification)
   at System.Threading.Tasks.Task`1.get_Result()
   at Test.CachePOC.Test.Integration.BaseTest.Configure() in D:\Projects\Test\CachePOC\Test.CachePOC\Test.CachePOC.Test.Integration\BaseTest.cs:line 66

  This exception was originally thrown at this call stack:
    [External Code]

Inner Exception 1:
JsonException: '5' is an invalid start of a property name. Expected a '"'. Path: $ | LineNumber: 0 | BytePositionInLine: 386.

Inner Exception 2:
JsonReaderException: '5' is an invalid start of a property name. Expected a '"'. LineNumber: 0 | BytePositionInLine: 386.

It seems when Redis.OM serialize/deserialize values then need to use InvariantCulture.
It is possible to provide what Culture need to use inside the Redis.OM library?

@AlexPulsesAi - do you have the same issue if you change the storage type to JSON? (we've had these issues with geo, think we might have fixed them there.)

Thanks! It works fine with JSON storage type.


DeviceInfo.getDeviceLocale not working on emulator

Currently I'm testing DeviceInfo on an Android emulator using API level 24 and I'm always getting "und" as a response.
Is DeviceInfo capable of working with a virtual device?
Note: I haven't tested it on a real device just yet.
Thanks! Nice library BTW!

So, I've tested on a real device and this works just fine.

@Eldelshell I know this isn't a fix/helpful for this lib, but react-native-i18n gives the locale on emulator if you really need it.

Thanks Rebecca, I can live with this without having to add another dep.


QA issues that are fixed in v0.6

[x] #420
[x] #439
[x] #426

We also have https://github.com/masa-finance/masa-oracle/issues/426 which is part of this

Closing this one as we have already split cards


Hello, I can not correspond the phrase with its coarse category.

Hello:
I am a master student and am trying to do some jobs about this excellent job.  I failed to use your plc-c code to compute the HGLMM features. I spent several days fixing it but it still said some of the words do not exist in the vocabulary. So I decided to use your uploaded text features to do the experiments. But in the ***_imfeats.h5 files, there are no features like phrase_id or  phrase_type. I really do not know how to correspond the phrases their coarse category like "people, scene". Could you please tell me how do you do it?
Thank you very much for your precious time！

Not all words will exist in a vocabulary, this is expected behavior.  An alternative is to use word2vec or fasttext features instead, which should work roughly as well.  As far as how to match phrases to the course category, these comes from the original Flickr30K Entities annotations, and can be obtained from there.


Add configurable global limit on number of nodes/machines.

Description
What problem are you trying to solve?
due to the limited IP range, our clusters can only scale up to a certain number of nodes.
on Cluster Autoscaler we used the flag --max-nodes-total to enforce the limit
Similar configuration for karpenter will be handy
This will also help mitigate the risk here
The suggested mitigations (resource limit on CPU and memory) are not ideal because:

it is per provisioner, not cluster-wide
it is on the number of CPU and memory, while the limitation we are facing is on the number of nodes

How important is this feature to you?
it is not a blocker for us right now, but we hope to resolve this risk before Cyber Week

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments; they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Closing in favor of https://github.com/aws/karpenter/issues/2474. Feel free to re-open if you disagree.

Thank you, @njtran, for the quick response. However, I feel this is slightly different.
the limitation we have is on the node count (limited IP range available for the nodes) so it does not matter what global CPU and memory limit we define.
assuming the limit is 10 nodes
we can not enforce this using resource limits like CPU unless we are very strict about the instance-types.
in fact we would like not to limit the overall capacity of the cluster in terms of CPU and memory allowing karpenter to replace smaller nodes with larger ones creating more CPU and memory without breaking the node limit

@njtran I do not have the right permissions to re-open the issue

Hey @myaser, commented on the other issue. I think we can aim to track it there, let me know if you disagree.


Unable to run flutter build ipa  Command PhaseScriptExecution failed with a nonzero exit code

Greetings,
If possible, and hoping I'm in the right space to mount it. I want to show you my "flutter build ipa --verbose", hoping you can provide me with a possible help or solution.
Very grateful for any help you can provide. Thank you.
Log File:
output.txt
Moved to:
Originally posted by @gmediard in https://github.com/flutter/flutter/issues/91142#issuecomment-966447122

@gmediard
From your log, it's failing due to below errors:
`../../../flutter/.pub-cache/hosted/pub.dartlang.org/flutter_math_fork-0.3.3+1/lib/src/widgets/selectable.dart:407:7: Error: The non-abstract class 'InternalSelectableMathState' is missing implementations for these members:
               [   +8 ms]  - TextSelectionDelegate.copySelection
               [        ]  - TextSelectionDelegate.cutSelection
               [        ]  - TextSelectionDelegate.pasteText
               [        ]  - TextSelectionDelegate.selectAll
               [        ] Try to either
               [        ]  - provide an implementation,
               [        ]  - inherit an implementation from a superclass or mixin,
               [        ]  - mark the class as abstract, or
               [        ]  - provide a 'noSuchMethod' implementation.
               [        ] class InternalSelectableMathState extends State<InternalSelectableMath>
               [        ]       ^^^^^^^^^^^^^^^^^^^^^^^^^^^
               [        ] ../../../flutter/packages/flutter/lib/src/services/text_input.dart:1034:8: Context: 'TextSelectionDelegate.copySelection' is defined here.
               [        ]   void copySelection(SelectionChangedCause cause);
               [        ]        ^^^^^^^^^^^^^
               [        ] ../../../flutter/packages/flutter/lib/src/services/text_input.dart:1014:8: Context: 'TextSelectionDelegate.cutSelection' is defined here.
               [        ]   void cutSelection(SelectionChangedCause cause);
               [        ]        ^^^^^^^^^^^^
               [        ] ../../../flutter/packages/flutter/lib/src/services/text_input.dart:1022:16: Context: 'TextSelectionDelegate.pasteText' is defined here.
               [        ]   Future<void> pasteText(SelectionChangedCause cause);
               [        ]                ^^^^^^^^^
               [        ] ../../../flutter/packages/flutter/lib/src/services/text_input.dart:1028:8: Context: 'TextSelectionDelegate.selectAll' is defined here.
               [        ]   void selectAll(SelectionChangedCause cause);
               [        ]        ^^^^^^^^^
               [+5597 ms] ../../../flutter/.pub-cache/hosted/pub.dartlang.org/photo_view-0.11.1/lib/src/core/photo_view_gesture_detector.dart:106:29: Error: The argument type 'PointerEvent' can't be assigned to the parameter type 'PointerDownEvent'.
               [        ]  - 'PointerEvent' is from 'package:flutter/src/gestures/events.dart' ('../../../flutter/packages/flutter/lib/src/gestures/events.dart').
               [        ]  - 'PointerDownEvent' is from 'package:flutter/src/gestures/events.dart' ('../../../flutter/packages/flutter/lib/src/gestures/events.dart').
               [        ]     super.addAllowedPointer(event);
               [        ]                             ^
               [   +5 ms] Target kernel_snapshot failed: Exception
                          #0      KernelSnapshot.build (package:flutter_tools/src/build_system/targets/common.dart:241:7)
                          <asynchronous suspension>
                          #1      _BuildInstance._invokeInternal (package:flutter_tools/src/build_system/build_system.dart:839:9)
                          <asynchronous suspension>
                          #2      Future.wait.<anonymous closure> (dart:async/future.dart)
                          <asynchronous suspension>
                          #3      _BuildInstance.invokeTarget (package:flutter_tools/src/build_system/build_system.dart:777:32)
                          <asynchronous suspension>
                          #4      Future.wait.<anonymous closure> (dart:async/future.dart)
                          <asynchronous suspension>
                          #5      _BuildInstance.invokeTarget (package:flutter_tools/src/build_system/build_system.dart:777:32)
                          <asynchronous suspension>
                          #6      FlutterBuildSystem.build (package:flutter_tools/src/build_system/build_system.dart:606:16)
                          <asynchronous suspension>
                          #7      AssembleCommand.runCommand (package:flutter_tools/src/commands/assemble.dart:317:32)
                          <asynchronous suspension>
                          #8      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1150:27)
                          <asynchronous suspension>
                          #9      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
                          <asynchronous suspension>
                          #10     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
                          <asynchronous suspension>
                          #11     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
                          <asynchronous suspension>
                          #12     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
                          <asynchronous suspension>
                          #13     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
                          <asynchronous suspension>
                          #14     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
                          <asynchronous suspension>
                          #15     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
                          <asynchronous suspension>
                          #16     main (package:flutter_tools/executable.dart:93:3)
                          <asynchronous suspension>

               [   +5 ms]
                          #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
                          #1      AssembleCommand.runCommand (package:flutter_tools/src/commands/assemble.dart:334:7)
                          <asynchronous suspension>
                          #2      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1150:27)
                          <asynchronous suspension>
                          #3      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
                          <asynchronous suspension>
                          #4      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
                          <asynchronous suspension>
                          #5      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
                          <asynchronous suspension>
                          #6      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
                          <asynchronous suspension>
                          #7      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
                          <asynchronous suspension>
                          #8      run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
                          <asynchronous suspension>
                          #9      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
                          <asynchronous suspension>
                          #10     main (package:flutter_tools/executable.dart:93:3)
                          <asynchronous suspension>
               Failed to package /Users/jorgemartinez/Documents/app/cirilla.
               Command PhaseScriptExecution failed with a nonzero exit code`

pointing to a 3rd party plugin flutter_math_fork and probably conflicting with framework's updates. You'll need to upgrade that plugin to latest and try again to see if it works.

Greetings!
Thank you, I did indeed have a number of 3rd Party bugs that were fixed once they were updated.
Thank you very much!

Closing this as resolved based on above comment.

@darshankawar I have the same issue but in android
I don't using flutter_math_fork
[+12401 ms] [+12388 ms] Persisting file store
[        ] [  +14 ms] Done persisting file store
[        ] [  +19 ms] "flutter assemble" took 27,308ms.
[   +2 ms] [  +17 ms] Target debug_android_application failed: FileSystemException: Creation failed, path = '/Users/imac/development/projects/boraevolta-flutter/android/app/build/intermediates/flutter/debug/flutter_assets/android/app/build/intermediates/merged_assets/debug/out/flutter_assets/android/app/build/intermediates/flutter/debug/flutter_assets/android/app/build/intermediates/merged_assets/debug/out/flutter_assets/android/app/build/intermediates/merged_assets/debug/out/flutter_assets/android/app/build/intermediates/merged_assets/debug/out/flutter_assets/android/app/build/intermediates/merged_assets/debug/out/flutter_assets/android/app/build/intermediates/merged_assets/debug/out/flutter_assets/android/app/build/intermediates/merged_assets/debug/out/flutter_assets/android/app/build/intermediates/merged_assets/debug/out/flutter_assets/android/app/build/intermediates/merged_assets/debug/out/flutter_assets/android/app/build/intermediates/merged_assets/debug/out/flutter_assets/android/app/build/intermediates/merged_assets/debug/out/flutter_assets/android/app/build/intermediates/merged_assets/debug/out/flutter_assets' (OS Error: File name too long, errno = 63)
[   +1 ms]            #0      _Directory.createSync (dart:io/directory_impl.dart:133:7)
[        ]            #1      ForwardingDirectory.createSync (package:file/src/forwarding/forwarding_directory.dart:20:16)
[        ]            #2      ErrorHandlingDirectory.createSync.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:433:22)
[        ]            #3      _runSync (package:flutter_tools/src/base/error_handling_io.dart:597:14)
[        ]            #4      ErrorHandlingDirectory.createSync (package:flutter_tools/src/base/error_handling_io.dart:432:12)
[        ]            #5      copyAssets.<anonymous closure> (package:flutter_tools/src/build_system/targets/assets.dart:99:21)
[        ]            <asynchronous suspension>
[        ]            #6      Future.wait.<anonymous closure> (dart:async/future.dart)
[        ]            <asynchronous suspension>
[        ]            #7      copyAssets (package:flutter_tools/src/build_system/targets/assets.dart:87:3)
[        ]            <asynchronous suspension>
[        ]            #8      AndroidAssetBundle.build (package:flutter_tools/src/build_system/targets/android.dart:65:34)
[        ]            <asynchronous suspension>
[        ]            #9      _BuildInstance._invokeInternal (package:flutter_tools/src/build_system/build_system.dart:836:9)
[        ]            <asynchronous suspension>
[        ]            #10     FlutterBuildSystem.build (package:flutter_tools/src/build_system/build_system.dart:603:16)
[        ]            <asynchronous suspension>
[        ]            #11     AssembleCommand.runCommand (package:flutter_tools/src/commands/assemble.dart:314:32)
[        ]            <asynchronous suspension>
[        ]            #12     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1125:27)
[        ]            <asynchronous suspension>
[        ]            #13     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
[        ]            <asynchronous suspension>
[        ]            #14     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
[        ]            <asynchronous suspension>
[        ]            #15     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:288:9)
[        ]            <asynchronous suspension>
[        ]            #16     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
[        ]            <asynchronous suspension>
[        ]            #17     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:236:5)
[        ]            <asynchronous suspension>
[        ]            #18     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
[        ]            <asynchronous suspension>
[        ]            #19     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
[        ]            <asynchronous suspension>
[        ]            #20     main (package:flutter_tools/executable.dart:92:3)
[        ]            <asynchronous suspension>
[        ] [   +7 ms]
[        ]            #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
[        ]            #1      AssembleCommand.runCommand (package:flutter_tools/src/commands/assemble.dart:331:7)
[        ]            <asynchronous suspension>
[        ]            #2      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1125:27)
[        ]            <asynchronous suspension>
[        ]            #3      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
[        ]            <asynchronous suspension>
[        ]            #4      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
[        ]            <asynchronous suspension>
[        ]            #5      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:288:9)
[        ]            <asynchronous suspension>
[        ]            #6      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
[        ]            <asynchronous suspension>
[        ]            #7      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:236:5)
[        ]            <asynchronous suspension>
[        ]            #8      run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
[        ]            <asynchronous suspension>
[        ]            #9      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
[        ]            <asynchronous suspension>
[        ]            #10     main (package:flutter_tools/executable.dart:92:3)
[        ]            <asynchronous suspension>
[ +287 ms] [ +261 ms] ensureAnalyticsSent: 260ms
[        ] [   +3 ms] Running shutdown hooks
[        ] [        ] Shutdown hooks complete
[        ] [        ] exiting with code 1
[+6101 ms] > Task :app:compileFlutterBuildDebug FAILED
[ +110 ms] FAILURE: Build failed with an exception.
[        ] * Where:
[        ] Script '/Users/imac/development/flutter/packages/flutter_tools/gradle/flutter.gradle' line: 1005
[        ] * What went wrong:
[  +87 ms] Execution failed for task ':app:compileFlutterBuildDebug'.
[        ] > Process 'command '/Users/imac/development/flutter/bin/flutter'' finished with non-zero exit value 1
[        ] * Try:
[        ] Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
[        ] * Get more help at https://help.gradle.org
[        ] BUILD FAILED in 42s
[        ] Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
[        ] Use '--warning-mode all' to show the individual deprecation warnings.
[        ] See https://docs.gradle.org/6.7/userguide/command_line_interface.html#sec:command_line_warnings
[        ] 26 actionable tasks: 2 executed, 24 up-to-date
[{"event":"app.progress","params":{"appId":"ec178082-86d1-4d98-889a-fdc290166595","id":"0","progressId":null,"finished":true}}]
[+4360 ms] Exception: Gradle task assembleDebug failed with exit code 1
[   +4 ms]
           #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
           #1      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:571:9)
           <asynchronous suspension>
           #2      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1125:27)
           <asynchronous suspension>
           #3      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #4      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
           <asynchronous suspension>
           #5      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:288:9)
           <asynchronous suspension>
           #6      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #7      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:236:5)
           <asynchronous suspension>
           #8      run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
           <asynchronous suspension>
           #9      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #10     main (package:flutter_tools/executable.dart:92:3)
           <asynchronous suspension>


Chainmail: implement save/unsave message #855

Implements/fulfills #855

@cool-ant There is an unhandled TODO in mail-list.tsx. There's commented out UI there for displaying a pin on a message in the message in the list if it's saved. For that, we need to know from the message's metadata whether it is saved or not. Can you add a saved property?


Automatic Service Discovery

Is there a method/property in the Nameko MethodProxy / ServiceProxy API that allows for the retrieval of the available RPC methods on a given service?

Not out of the box, but it's trivial to write a service that can introspect itself:
from collections import defaultdict

from nameko.extensions import DependencyProvider
from nameko.rpc import rpc

class EntrypointList(DependencyProvider):

    def get_dependency(self, worker_ctx):
        # all Extensions have access to self.container
        return list(self.container.entrypoints)

class Service:
    name = "discovery"

    entrypoint_list = EntrypointList()

    @rpc
    def introspect(self):
        entrypoints = defaultdict(list)
        for ep in self.entrypoint_list:
            entrypoints[ep.method_name].append(type(ep).__name__)
        return entrypoints

$ nameko shell
Nameko Python 3.5.3 (default, Jun 30 2017, 18:28:54)
[GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-8<IP_ADDRESS>)] shell on darwin
Broker: pyamqp://guest:guest@localhost
>>> n.rpc.discovery.introspect()
{'introspect': ['Rpc']}
>>>

Hi Matt,
Thanks for the speedy reply - that's much more elegant than what I'd bodged together in the meantime - thanks!

This issue was moved to https://discourse.nameko.io/t/automatic-service-discovery/283


(feature request) handle long lines better

If you set max-width too low, rustfmt bails on long lines. If you set it too high, it squashes multiline function calls or struct constructions into single lines. There's no middle ground.

Could you give some examples please?

So I have these extern functions, for example
    fn DMR_GetDevice(pHDev: *mut HDEV, searchMode: DeviceSearchMode, pSearchString: *const c_char, devNr: c_uint, wildcard: c_char) -> TDMR_ERROR;

For whatever reason, rustfmt doesn't want to deal with this (i.e. putting in line breaks between the function arguments). It just says the line is too long.
So I set max-width = 500. But then it reformats
Writer { handle: rx.recv().unwrap(),
         dst: Destination::Name,
         other_field: "stuff",
         _ghost: PhantomData
}

to
Writer { handle: rx.recv().unwrap(), dst: Destination::Name, other_field: "stuff", _ghost: PhantomData }

which isn't so great.

So, not spreading the function signature across multiple lines is very bad - we should do better there, the only thing I can think of is perhaps it being in an extern block means we are skipping it? Should be an easy fix if that is the case.
The struct literal on one line is expected behaviour. There is an option you can set to always use multi-line formatting for struct literals. There is another issue filed some where for setting a maximum number of fields on one line, which should improve things.

Yeah, if I just paste the function signature into a file and replace the ; with {} then it splits the args. But in an extern block it won't touch it.


LightOccluder in Tilemaps loosing their lightmask

When using LightOccluders on different light masks in a tileset scene, the light masks will be reset to 1 after convert to TileSet.
After a quick view in the generated tileset.xml it seems to me, that the light mask info isn't stored there.
Converting it to a .res file instead of .xml has the same issue.

tilemap draws everything as a batch, including occluders so any per-tile information except material is lost by design. Maybe what we could do is add a property so we can set the occluder mask for the entire tilemap?

an alternative for what you desire may be using more than one tilemap with different occluder masks

In my case, there was only one occluder in the tilemap. It was on mask 2, but then reset to mask 1. Sorry, my description was a bit unclear in that point.

oh ok, now you can change occluder mask in tilemap settings
On Sat, Dec 12, 2015 at 10:50 AM, Sven Lange<EMAIL_ADDRESS>wrote:

In my case, there was only one occluder in the tilemap. It was on mask 2,
but then reset to mask 1. Sorry, my description was a bit unclear in that
point.
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/3025#issuecomment-164153100.

great, thx! i'll check it with the next build :)


Deployment with secure parameter that references compiletime imported udt, System.InvalidOperationException is thrown

Bicep version
Bicep CLI version 0.24.24 (5646341)
Describe the bug
When deploying a template with a secure parameter that references a compile time imported udt, compilation throws:
System.InvalidOperationException: Sequence contains no elements
To Reproduce
I've included a project as attachment to reproduce the situation.
deploy-fails.ps1 --> main-fails.bicep --> throws the error
deploy-works.ps1 --> main-works.bicep --> completes as expected
Additional context
If secure decorator is removed, it works as expected. If the UDT is replaced with generic object, also works as expected.
See attachment:
bicep-udt-compiletimeimport-secureparam-error.zip

Thanks for reporting! The @secure() decorator can't be used on statements whose type clause is a reference to a user-defined type, but the check responsible for enforcing that rule was never updated to handle imported types.


flutter_crashlytics issue: MissingPluginException

Steps to Reproduce

I've been trying to use fluter_crashlytics as explained at https://pub.dartlang.org/packages/flutter_crashlytics
But when it tried to initialize with FlutterCrashlytics().initialize(), it returns Dart Error:
Unhandled exception:
MissingPluginException(No implementation found for method initialize on channel flutter_crashlytics).

Which is coming from platfom_channel.dart
    final dynamic result = await BinaryMessages.send(
      name,
      codec.encodeMethodCall(MethodCall(method, arguments)),
    );
    if (result == null)
      throw MissingPluginException('No implementation found for method $method on channel $name');
    return codec.decodeEnvelope(result);

That means should I implement "flutter_crashlytics" channel?
The page didn't mention about that.
So I thought the plugin should provide that implementation.
Or Maybe I am missing a big basic thing here.
Thanks.
No issue from Doctor.
I am using version 1.0.0
[✓] Flutter (Channel beta, v1.0.0, on Mac OS X 10.14 18A391, locale en-US)

Logs

Dart Error: Unhandled exception:
MissingPluginException(No implementation found for method initialize on channel flutter_crashlytics)
#0      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:291:7)
<asynchronous suspension>
#1      FlutterCrashlytics.initialize (package:flutter_crashlytics/flutter_crashlytics.dart:18:53)
<asynchronous suspension>
#2      main (package:my_package/main.dart:24:30)
<asynchronous suspension>
#3      _startIsolate.<anonymous closure> (dart:isolate/runtime/libisolate_patch.dart:289:19)
#4      _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

Try with 2.1, should be ok now

Closing this, if not fix please provide a small example project to reproduce the problem and I'll check this.


0.10.0 cannot be installed with IJ 2022.3 EAP

The ktlint plugin 0.10.0 cannot be installed on IntelliJ 2022.3 EAP. It would be nice if you could support the new IJ release.

Thanks for the heads up! It seems I break my IDE update automation every cycle right before a new version drops. Fixing now!

Fixed script, #235 opened 😁

Same problem with IJ 2022.2.1, it installs, then lingers in disabled state. The enable checkbox is greyed out. I hope the same fix applies.


Cleaning lost nodes job

Because we are creating multiple nodes sometimes we have few of them left on GCP..
I was not able to find reason for it but maybe some connection issues. (delete is not checking if it was able to really delete node)
Should fix:

https://issues.jenkins-ci.org/browse/JENKINS-52736

@rachely3n @craigatgoogle Please review.. Im testing on our prod if it will do it job as intended..

That's a good question, @evandbrown what do you think re: preemptive instances?

Another question about this approach, what is the expected behavior when the GCP project is not dedicated to the Jenkins installation? For example, what if this project also houses VMs for other non-related services?

Another question about this approach, what is the expected behavior when the GCP project is not dedicated to the Jenkins installation? For example, what if this project also houses VMs for other non-related services?

It will work like charm.. (we dont have project just for slaves)
We are only checking our own instances.. (marked with specific label) https://github.com/jenkinsci/google-compute-engine-plugin/pull/17/files#diff-b233b70c4a5e6ae6fcb1d458beb7e46dR119
We can even have few clouds in one GCP project and each own should check its own instances.


React Navigation detect tab bar press and re-rendering screen help

I am in need of a solution to a problem I am having with react navigation. I will start off by listing what I want to get done and if it is possible with react navigation:

currently, I have 2 tabs on my app screen. One tab displays weather information to the user, and the second tab controls what the user wants to see in his/her weather forecast in the first tab. So the second tab is essentially a settings screen while the first screen is where the GUI takes place. Now, In my second tab, once the user changes his/her options, I want to make sure they saved their preferences by clicking a button I have on the screen. Now, here is where my issue with react navigation comes into place, I want the second tab to work in such a way that when the user clicks back on the first tab, an Alert will pop up asking them, "Are you sure you want to leave this page without saving? Yes/no" If they click no, I want them to stay in this tab, if they click yes, I want them to go back to the first tab.

Here is my code:
export const Tabs = TabNavigator({
  WeatherDisplay: {
    screen: WeatherDisplay,
    navigationOptions: {
        header: null,
        tabBarLabel: 'Weather',
        tabBarIcon: ({ tintColor }) => <Icon name="weather" size={24} color={tintColor} />
    },
},

  WeatherOptions: {
    screen: WeatherOptions,
    navigationOptions: {
      header: null,
        tabBarLabel: 'Weather Options',
        tabBarIcon: ({ tintColor }) => <Icon2 name="settings" size={23} color={tintColor} />
    },
  },
},
{
  animationEnabled:true,
  swipeEnabled:false,
  gesturesEnabled:false,
  tabBarOptions: {
    inactiveTintColor: 'gray',
    showIcon:true,
    activeTintColor:'#D088EB',
    showLabel: false,
    indicatorStyle: {
      borderBottomColor: '#D088EB',
      borderBottomWidth: 2,
    },
    style: {
    backgroundColor: '#F8F8F8',
  },
  onTabPress: (tab) => {
    if (tab.index == 0) {
	    	Alert.alert("Save","Dont forget to save your weather!");
      }
	    }

},
}
);

Thus far, I have done extensive research and reading online and have been banging my head against the react-navigation doc's but I can't seem to find a function related to onTabPress. In my code above, the onTabPress works only and android and not on iOS, meaning the onTabPress() function gets called when a user changes tabs, but this still doesn't address my problem because I only want the tab to change if a user clicks 'yes' when an alert popups. Currently, the code just sends an Alert but still automatically changes the screen, and it only works on android.
I appreciate any help! thank you for reading this!

There's not currently a way that's built in to do this in React Navigation but, by overriding the tabBarComponent, you can customize it in a variety of ways.
#1335 has an example of how you could do it - I actually implemented something similar on a project this morning! 😄

Hey @spencercarli, sorry I am new to react native and GitHub and all of these things. In that #1335 link do I just go to the "commits" section and copy the changes he made to specific files then I can have the onTabPress method and then make it so that I can add an alert with option yes/no and if they click yes it will change screens, if they click no then it will stay in the same screen?

@shlomi84 you can use that code as a reference point for building your own tabbar. Try to understand what the code is doing vs. just copying it.

Hi! In an effort to get the hundreds of issues on this repo under control I'm closing issues tagged as questions. Please don't take this personally - it's simply something we need to do to get the issues to a manageable point. If you still have a question I encourage you to re-read the docs, ask on StackOverflow, or ask on the #react-navigation Reactiflux channel. Thank you!


can't install  codegpt extension because it is not compatible with the current version of vscode 1.73.0

v

The next version will have support for VSCode 1.73.0.
I will update it during the week

The next version will have support for VSCode 1.73.0.
I will update it during the week

Thanks Daniel


Configure pre-commit hooks

Description
This PR adds a configuration file for the pre-commit tool, which is "useful for identifying simple issues before submission to code review". Pre-commit is recommended by both learn.scientific-python.org and pyopensci.org.
Besides the black, black-doc, isort, codespell, and cython-lint tools we already use, this PR adds autoflake, pyupgrade, prettier, and other useful tools.
I have not enabled flake8 and ruff since their output is currently very noisy and it will be tedious to configure correctly. Mypy is also not enabled since it takes longer to run.
Pre-commit can be run manually or registered with git. Once we are confident about these tools, pre-commit can be enforced in CI.
Release note
Summarize the changes in the code block below to be included in the
release notes:
Configure pre-commit hooks

Checklist

[ ] The pull request title, summary, and description are concise.
[ ] Related issues are linked in the description.
[ ] New dependencies are explained.
[ ] The source code and documentation can be distributed under the MIT license.
[ ] The source code adheres to code standards.
[ ] New classes, functions, and features are thoroughly tested.
[ ] New, user-facing classes, functions, and features are documented.
[ ] New features are covered in tutorials.
[ ] No files other than source code, documentation, and project settings are added to the repository.

we can now just run pre-commit and mypy to apply all code standards required by the library, right?

Yes.

flake8 and ruff, since they are too verbose and will add too much noise.

I would still like to switch to ruff though. It's fast, replaces several other tools, and many of the warnings are legit. It will take some effort to correctly configure and fix remaining warnings.


Allow execution of Submap2D::ToResponseProto without a grid

related to https://github.com/googlecartographer/cartographer_ros/issues/819

@wally-the-cartographer merge

Merge requested by authorized user pifon2a. Merge queue now has a length of 1.


Transaction amount not shown in transaction list

Im seeing this on dcrios as well, which is built with dcrlibwallet v1.0

Fixed in #361


Composer updates

@Jaland I merged my PR for the cluster setup.
I'm pretty sure I have everything you included here, but would appreciate a second set of eyes!

@Jaland I merged my PR for the cluster setup.
I'm pretty sure I have everything you included here, but would appreciate a second set of eyes!

Taking a look now


chore(main): release 1.0.7

:robot: I have created a release beep boop
1.0.7 (2024-07-25)
Bug Fixes

token (8662f10)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.7
:sunflower:


Try experimental patch for OSx fix

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[X] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

Just an experimental build to test patch proposed on https://github.com/doxygen/doxygen/issues/9557

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


No module named"requests" on chapter 37

After I copied the code to my VS and added subscription key, I execute the code, however, it showed the mistake.
Traceback (most recent call last):
File "c:/Autumn_Courses_2019/Python/API/API_json.py", line 6, in
import requests
ModuleNotFoundError: No module named 'requests'
Why it triggered and how to solve it ? Thanks

This issue is more of a local problem, you are not able to run the python file because you have not installed requests module in your local machine.
That triggers the issue, I will open another issue that will solve any other similar issues

This can be solved by adding requirements.txt file in the Packages folder
We can additionally add documentation in the readme file in Packages folder on how to use use this file

One of the problems could be that in VS the python interpreter is still selected from the previous exercise (virtual environment). check in the lower right corner if you get something like Python 3.7.....-bit('venv':venv).


Terminal unresponsive while running ctop on remote server

Environment
Windows build number: Microsoft Windows NT 10.0.18363.0
Windows Terminal version (if applicable): 1.0.1811.0

Any other software?
ctop and some running containers on a remote Ubuntu server

Steps to reproduce
SSH into a linux server with containers running and ctop installed
Expected behavior
Being able to scroll up/down and to the sides for more information
Actual behavior
Terminal is unresponsive until ctop is closed with q
Reference:
ctop: Top-like interface for container metrics

Which ssh client are you using? ssh.exe, one that ships with WSL, some other one?
How are you trying to scroll in ctop? With the arrow keys? With the mouse wheel?

I'm using the ssh client included with Git Bash. I'm trying to scroll with the arrow keys.

Hokay, so this is a long shot, but this might actually be #6737

So, weird question ... if you use the SSH client included with Windows, does it suddenly work? You can run it directly from C:\Windows\System32\OpenSSH\ssh.exe.

Interesting, it seems to work fine using Windows SSH client. I'd like to note that it used to work using the Git Bash SSH client.

This issue seems to also happen in htop

Thanks for the confirmation. That definitely makes it a /dup of #6737, which should be addressed by #6859


can not test queryParameters case

Description
when I add queryParameters in Dio, in that case URL is added unexpected "GET/null/{test: true}/{}"
DioError [DioErrorType.DEFAULT]: Assertion failed: "Could not find mocked route matching request for https://example.com/GET/null/{test: true}/{}"

Steps to reproduce
below one is my test code.
    test("query parameters API test", () async {
      final dio = Dio();
      final dioAdapter = DioAdapter();

      dio.httpClientAdapter = dioAdapter;

      const path = 'https://example.com';
        dioAdapter
            .onGet(
              path,
            (request) => request.reply(200,
                {'message': 'Successfully mocked GET!'}));

        // Making dio.get request on the path an expecting mocked response
        final getResponse = await dio.get(path, queryParameters: {"test": true});
        expect({'message': 'Successfully mocked GET!'},
            getResponse.data);
    });

Expected behavior
expected URL is just below
https://example.com/

Hi man! First of all thanks for response.
Here I can't see any bug, probably it's a mechanical problem.
Yeah you was given queryParameters: {"test": true} into dio.get but you wasn't into dioAdapter function, so that is issue.
Here is fixed test code:
test("query parameters API test", () async {
    final dio = Dio();
    final dioAdapter = DioAdapter();

    dio.httpClientAdapter = dioAdapter;

    const path = 'https://example.com';
    const queryParameters = {"test": true};

    dioAdapter.onGet(
      path,
      (request) => request.reply(
        200,
        {'message': 'Successfully mocked GET!'},
      ),
      // You should give queryParameters also here,
      queryParameters: queryParameters,
    );

    // Making dio.get request on the path an expecting mocked response
    final getResponse = await dio.get(path, queryParameters: queryParameters);
    expect({'message': 'Successfully mocked GET!'}, getResponse.data);
  });

I will close the issue as @theiskaa has already pointed out the solution for the problem which is more of a user error than a bug.
If you don't deem the issue to be fixed, please feel free to re-open the issue!


[WIP] Adding encryption decorator

Question
Answer

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Tests pass?
no

Fixed tickets
#8

License
MIT

Doc PR

Description
Adding a encryption decorator for psr6
TODO

[x] Add tests
[ ] Add documentation
[ ] Updated Changelog.md
[x] Rename package from encrypted to encryption

@Nyholm any idea why i cant get testSaveDeferredWhenChangingValues to pass?

Ill see.

The testSaveDeferredWhenChangingValues test makes sure that the raw value has not changed on the deferred queue before we do a save.
Since you encrypt the value you break the test. It is the test that is flawed though. I'll fix it.

Maybe i can find a workaround

@Nyholm should i ignore this https://scrutinizer-ci.com/g/php-cache/cache/inspections/a5e25a31-409b-4228-876e-f715b2b126b5/issues/files/src/Encryption/EncryptedItemDecorator.php?status=new&orderField=path&order=asc&honorSelectedPaths=0 ?

I think we can ignore them for now but this sure is an issue.. let's fix that before making our first tag.

Current coverage is 83.56% (diff: 89.61%)

Merging #107 into master will increase coverage by 0.46%

@@             master       #107   diff @@
==========================================
  Files            22         24     +2
  Lines          1006       1083    +77
  Methods         218        241    +23
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            836        905    +69
- Misses          170        178     +8
  Partials          0          0

Powered by Codecov. Last update 55d7af5...27bc164

Really like this!

So the last thing to do is add documentation

test are all green, only need to add docs for it


Fix / Some updates to the README.md + CONTRIBUTING.md

Before submitting this PR, please make sure:

[X] Your code builds clean without any errors or warnings
[X] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:

Biggest update: replace GREEN, YELLOW, RED icons with emojis supported by markdown
Some updates to URLs
CoinAlpha/hummingbot -> hummingbot/hummingbot

Tests performed by the developer:
Did my best to proof read everything, testing URLs etc.
Tips for QA testing:
N/A

Hi @GuybrushX, please resolve conflicts in this PR. Thanks!

Thanks @GuybrushX so much for this PR - it looks great! Let me know if you want to fix the merge conflicts or if you want me to resolve!

Hello @GuybrushX, can you please resolve the conflicts in this PR so we can merge this easily? thanks!

@JeremyKono lets just do it ourselves. Can you give it a shot?

Sure thing @mifeng

Sorry for my delay. Glad to see you resolved the conflict and merged it already - LGTM :-)


Logs obfuscation

We should research a solution to obfuscate the sensitive information from cloudwatch logs

The Ansible default solution to prevent leaking any password is to disable the logs of the tasks: https://docs.ansible.com/ansible/latest/reference_appendices/logging.html
Example:
- name: secret stuff
  command: "echo {{secret_root_password}} | sudo su -"
  no_log: true

We could review all our Ansible roles and use disable logs on the tasks that use passwords.


Windows authentication across untrusted domains

I'm developing a web service based on ASP.NET Core 2.2 + IdentityServer4, supporting both credentials authentication (i.e. username+password) and Windows authentication. I am not able to login using a domain user from an untrusted domain.
Steps to reproduce the behavior:

Using this version of ASP.NET Core '2.2'
Write this code HttpContext.AuthenticateAsync(Microsoft.AspNetCore.Server.IISIntegration.IISDefaults.AuthenticationScheme) or Challenge(Microsoft.AspNetCore.Server.IISIntegration.IISDefaults.AuthenticationScheme)
Try to access the resource and when prompted for Windows credentials (i.e. Challenge), type untrusted domain credentials.

Expected behavior
Domain user can log in. (Given I could specify the untrusted domain somewhere).
Notes
The two untrusted domains have DNS stubs so they can 'know' each others.
The final goal is to use Http.Sys (instead of IIS) or even Kestrel 3.0, but for now I was thinking that using IIS would be the easier thing.

This is more a windows configuration issue, but with no trust between the two domains I don't believe its going to work, the server isn't going to be able to validate the untrusted domain due to the lack of trust.

Hi @blowdart ,
thanks for your help. I was hoping that from the application I could specify the untrusted domain we want to authenticate against, but it may open up security concerns if the two domain aren't trusted each other.
Looking forward to an answer re: windows configuration though.
Thanks,

That's beyond us, it's a Windows AD configuration question and not one I'd feel comfortable even trying to answer.

@blowdart Any idea who can I ask it?
I tried on SO here but still no luck
Thanks again for your help, much appreciated!

Honestly no. I can't even find AD documentation under the Windows server site.

Ok so you start feeling my pain! 😄

Talked to some windows friends.
https://docs.microsoft.com/en-us/windows-server/identity/ad-ds/plan/forest-design-models is the overview, which you should read the forest trust sections to understand what you're going to do, then https://docs.microsoft.com/en-us/previous-versions/windows/it-pro/windows-server-2008-R2-and-2008/cc754626(v=ws.11) are the actual steps.
Please understand the security implications of forest trusts before implementing them :)

Closing as answered, and also really a windows issue.


Translated to Japanese

I've translated your great Git Style Guide to Japanese.

Sorry.  I've done misoperation.  Please ignore this PR :bow:

That's great to hear @objectx! Please see https://github.com/agis-/git-style-guide/pull/21#issuecomment-93992907 for info on how translations are added.


feat: enable focus reporting

supersedes #1079

We could also add tea commands to enable this feature tea.EnableReportFocue & tea.DisableReportFocus

good idea, done!


Create launch presentation

Create a launch presentation for MindTrek. Optionally, organise a brainstorming about  presentation content.
Definition of done

[x] Create an outline of launch presentation.
[x] Get feedback on outline.
[x] Find or create images for presentation.
[x] Write the key points.
[ ] Organise a review.
[x] Store the presentation at least on OwnClould, but preferably also on Github under outreach materials.

Launc presentation is available in OwnCloud > Shared > API manager.

Presentation available as pdf and ppt in Github in the documentation repository (Organization > outreach)


[Feature Request] ARM version of the container

Have you read the Important information text above

[X] Yes i did

Describe the feature
Just wanted to know if it's possible to also support ARM architecture. Your containers are by far the best for hosting game servers.
Additional information

[X] Would you be willing to help implement this feature?

Final checks

[X] I understand that this is a feature request function. I also unterstand that if i use this to report a bug report, that my issue might be un-answered closed.

Wait, i see there was already one about this:
https://github.com/jammsen/docker-palworld-dedicated-server/issues/6
Closing feature request.

Search-function the issues 👍


The annotated dataset of ships and planes

Can the dataset used by you for detecting ships and planes be provided in the public domain. It would be really helpful.
Thanks @avanetten

Unfortunately, we cannot share the raw dataset at this time.


[Feature]

When I tried to start it by running "-e PROXY_URL="http://<IP_ADDRESS>.:7890"" directly after enabling system proxy with Clash, the startup failed.

启动 log 发一下


Update cartservice Dockerfile to support ARM64

Use multiarch runtime-deps image, and configure the runtime identifier based on the images architecture using uname -m.

Is the cart service the only one that is problematic with ARM ?

So far the rest build fine on arm64 without changes. That said, the currently released images aren't all built for arm64 yet IIRC.

that's correct @chancez - The currently released images are build to ARM. We have a bug for ARM images not being build   .


Rate Limit Error

Which API Provider are you using?
Anthropic
Which Model are you using?
Claude 3.5 Sonnet
What happened?

Issue Description:
When using the Roo-Cline package, I encountered the following error after reaching the rate limit:
429 {"type":"error","error":{"type":"rate_limit_error","message":"This request would exceed your organization’s rate limit of 80,000 input tokens per minute. For details, refer to: https://docs.anthropic.com/en/api/rate-limits; see the response headers for current usage. Please reduce the prompt length or the maximum tokens requested, or try again later. You may also contact sales at https://www.anthropic.com/contact-sales to discuss your options for a rate limit increase."}}
The error itself is expected, but the current behavior requires manually clicking the "Retry" button every time.
Desired Behavior:
I would like to see an automated retry mechanism that waits for 1 minute after receiving this error and then retries the request without user intervention.
Steps to reproduce

Relevant API REQUEST output
No response
Additional context
No response

Good idea. This seems similar to https://github.com/RooVetGit/Roo-Cline/pull/116... I just don't love the code and haven't really tested it. If you're experiencing this frequently would you want to give a custom VSIX with this change a shot to see if it helps?

Sure, I can try it on weekend.


Web - Workspace - Emoji is misaligned in the name Workspace

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

Action Performed:

Go to URL https://staging.new.expensify.com/
on a mobile device
Login with any account on a mobile device
Create a workspace and rename it with emoji on a mobile device
Check in Web the same account and the rooms of this Workspace

Expected Result:
Emoji is in the middle in the window with the names of room this Workspace.
Actual Result:
Emoji is misaligned in Web.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] Web / Chrome
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Triggered auto assignment to @miljakljajic (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@miljakljajic Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

This one doesn't strike me as a required improvement - going to close.


test commit #2

Details
Testing somthing #2
https://github.com/Expensify/Expensify.cash/issues/3787
https://github.com/Expensify/Expensify.cash/issues/3787
Fixed Issues
$ https://github.com/Expensify/Expensify.cash/issues/3787
Tests
N/A

@tgolen apologies for the pings, this was necessary in order to QA this issue: https://github.com/Expensify/Web-Expensify/pull/31276


Various improvements to database manager

Looks good :smile: We should utilise the rollback more.

(The devils of optimization are striking ;-) )
For the record, I tried to optimize further by wrapping  into a single transaction the creation of all new articles of a feed. I didn't notice any significant improvements.
Probably because the time spent waiting for the queue to become available is counterbalancing the reduction of SQLite transactions.


oidc does not exist on Request type

Issue
import { Request } from 'express'

...

export async function getServerSideProps(context: GetServerSidePropsContext & { req: Request }) {
  const variable = context.req?.oidc?.user['parameter'];
}

Tried

Upgrading to 4.18.x
Upgrading to beta
Restarting ts server
re-yarn-ing

are you using next.js? if yes you cannot mix express Request type with it

@CalebJamesStevens if you want to have oidc in the req object, you will need to modify the GetServerSidePropsContext type.
Try doing this:

add a new next.d.ts file in your project and add the following code in it.
aim here is to add a new oidc property in it.

import type { GetServerSidePropsContext as OriginalGetServerSidePropsContext } from "next/types";

declare module "next" {
	export type GetServerSidePropsContext<
		Q extends ParsedUrlQuery = ParsedUrlQuery,
		D extends PreviewData = PreviewData,
	> = OriginalGetServerSidePropsContext<Q, D> & {
		req: {
			oidc: string; // use the correct type here
		};
	};

	export type GetServerSideProps<
		P extends { [key: string]: any } = { [key: string]: any },
		Q extends ParsedUrlQuery = ParsedUrlQuery,
		D extends PreviewData = PreviewData,
	> = (context: GetServerSidePropsContext<Q, D>) => Promise<GetServerSidePropsResult<P>>;
}

export {};


typo in "Pair Program on Bonfires" waypoint

look for
"Your pair will have their own cursor, and will be able to type text on his or her and keyboard."
Remove the "and" after "his or her".

You should fork and fix this, then submit a pull request for the correction.

Fixed, opened a pull request - https://github.com/FreeCodeCamp/freecodecamp/pull/1115


importing scrollzzz from 'scrollzzz' results in empty object returned

Hey there, I'm trying to import it as a module but all I get returned is {}
It works fine while loading it as a script, but not when importing it via es6 module syntax

importing 'scrollzzz/dist/scrollzzz.esm' works

Hi, thanks for the report! Are u using webpack? If so, this seems related to #5756

Not sure :) most other libs I use including scrollama work as intended when using import

I think it's due to the fact your main points to the browser iife rather than pointing to the main one
An example in scrollama:
https://github.com/russellgoldenberg/scrollama/blob/master/package.json

based on this
https://medium.com/free-code-camp/anatomy-of-js-module-systems-and-building-libraries-fadcd8dbd0e
you'd need to export the UMD as main in package.json

I tested using Parcel bundler and this bug is still present.
Scrollama export only one bundle (umd) which works on both client and server side.
I export three different bundles: one for browser (iife), one for nodejs (cjs), and one using the new standard (es6). Then I link them in package.json file using pkg.main, pkg.browser, pkg.module fields.
It should be the bundler job to choose the correct build based on its configuration.
I don't understand why this is not working.
Anyway, I can try to export an umd version and point it to the pkg.main and pkg.browser fields, and maintain  the pkg.module field as it is now.
Thanks for the article!