Add tests for weather function, explicit function calls

Also remove FullPath from weather function.

All the tests pass for me locally, so I'm not sure what's going on here.

This originally passed for me locally, but after I removed my local copy of ~/PortalData/ it failed with the same error.
I'm testing in the same way that Travis does, which is to go the directory above your repo (i.e., the directory that the repo directory is in) and run:
R CMD build PortalDataSummaries
R CMD check PortalDataSummaries_<IP_ADDRESS>00.tar.gz

Happy to help with debugging if it's useful at some point.

I fixed the issue with the weather checks, but in the process it rewrote the NAMESPACE file incorrectly. I'm not sure why. (I started changing what we need in the namespace as was mentioned here, but I assume that's still why it's not passing?)

Weird, the tests as a whole were passing for the PR, but now appear to be failing after the merge...

Yeah, something is going on here I don't understand. Everything passes for me and works fine.

I can confirm that we have a test that is failing locally:
> devtools::test('PortalDataSummaries')
Loading PortalDataSummaries
Testing PortalDataSummaries
checks data_processing functions: ............
|===================================================== |100% ~0 s remaining     1W.
checks weather summary output: ..W.....W...

Warnings -----------------------------------------------------------------------
1. data generated by default setting is same (@test-regression.R#7) - joining factor and character vector, coercing into character vector

2. data generated by level=treatment, length=long-term is same (@test-regression.R#15) - joining factor and character vector, coercing into character vector

3. data generated by level=plot, time=newmoon, type=granivore, shape=flat
          is same (@test-regression.R#22) - joining factor and character vector, coercing into character vector

4. data generated by unknowns=T, incomplete=T is same (@test-regression.R#29) - joining character vector and factor, coercing into character vector

5. Monthly option returns 7 columns (@test-weather.R#12) - NAs introduced by coercion

6. Monthly temperatures ok (@test-weather.R#26) - NAs introduced by coercion

Failed -------------------------------------------------------------------------
1. Failure: data generated by level=plot, time=newmoon, type=granivore, shape=flat
          is same (@test-regression.R#25)
digest(data) not identical to "80a297349b34d9eef00da1a6eeb0ec3d".
1/1 mismatches
x[1]: "487361feea3d07b7679c893d8bc0abdb"
y[1]: "80a297349b34d9eef00da1a6eeb0ec3d"

DONE ===========================================================================
Warning messages:
1: character(0)
2: character(0)
3: character(0)
4: character(0)

It's a hash mismatch in the third regression test.
The two most likely options for what is going on are:

A recent change in the data changed something about the output. This could have been the fix to the duplicates that was reported and merged a week or two ago. If that's true then @skmorgane just needs to recheck everything and update the hash if she's happy.
There is an OS difference in the hash for some reason (e.g., line endings) so it's failing on Linux (me and Travis) but not for you.


AnsiColor.fg with normal print does not work in cmd output

I thought to use your class AnsiColor for another purpose and tried this to see just some colored output in the command prompt but it did not do anything.
Any idears?
print('${AnsiColor.fg(208)} Should that lead to a colored text ?');

That is odd. Exactly your code snippet works for me. Does your terminal support coloured output? What terminal are you using?

The terminal inside Android Studio on Windows 10 (cmd prompt)

I confirm that this is an issue in Android Studio (v 3.5.2). The IntelliJ IDEA (on which Android Studio is build) does not support ANSI escape codes. After a long dig this will tell you:
import 'dart:io' as io;
print(io.stdout.supportsAnsiEscapes);

I believe the package should use something like this to detect support and either use or omit the ANSI escape codes.
Long story short; the snippet works on a regular terminal but not in Android Studio/IntelliJ. I also strongly suspect XCode won't work.
I hope this clarifies things. If not please feel free to re-open the issue.


[azure-messaging-eventhubs-cpp] publish version 1.0.0-beta.1

Update vcpkg ports for Azure SDK release. This release may contain multiple ports.

Adding azure-messaging-eventhubs to release
cc: @rickwinter, @ahsonkhan, @antkmsft, @vhvb1989, @gearama, @LarryOsterman

Tested usage successfully by azure-messaging-eventhubs-cpp:x64-windows:
azure-messaging-eventhubs-cpp provides CMake targets:

    # this is heuristically generated, and may not be correct
    find_package(azure-messaging-eventhubs-cpp CONFIG REQUIRED)
    target_link_libraries(main PRIVATE Azure::azure-messaging-eventhubs)


Dealing with Partial Dates

Short Description:
In this project, we will deal with many partial dates.
For now, a full date is required, usually in the format YYYY-MM-dd, but how should we deal with partial dates?
Here are some of the existing scenarios where partial dates will appear:

She is from 92 (no day, or month)
She is 32 years old (year can be inferred, but no month/day
we had a meeting in March last year (the day cannot be inferred)
we share the same birth Day (day can be inferred, but not year and month of birth)

One idea is to have a date range for some of those problems. In this case, we can have a date start, date end, and a flag for precise data. the flag would tell us if the meeting was the entire time, or somewhere in between.


Issue with DynamoDBChatMessageHistory Not Respecting max_token_limit in ConversationTokenBufferMemory

System Info
Langchain Version: langchain 0.0.350, langchain-community 0.0.3, langchain-core 0.1.1
Python Version: 3.10.6
Operating System: macOs
Additional Libraries: boto 2.49.0, boto3 1.34.1
Who can help?
No response
Information

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

Related Components

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

Reproduction
Steps to Reproduce:

Create an instance of DynamoDBChatMessageHistory with a specified table name and session ID.
Initialize ConversationTokenBufferMemory with a max_token_limit.
Attach the memory to a ConversationChain.
Call predict on the ConversationChain with some input.

Code sample:
import boto3
from langchain.llms import Bedrock
from langchain.memory.chat_message_histories import DynamoDBChatMessageHistory
from langchain.memory import ConversationTokenBufferMemory

session = boto3.Session(
    aws_access_key_id=os.environ.get('AWS_ACCESS_KEY_ID'),
    aws_secret_access_key=os.environ.get('AWS_SECRET_ACCESS_KEY'),
    aws_session_token=os.environ.get('AWS_SESSION_TOKEN'),
    region_name='us-east-1'
)

dynamodb = session.resource('dynamodb')
chat_sessions_table = dynamodb.Table('SessionTable')

boto3_bedrock = session.client(service_name="bedrock-runtime")

max_tokens_to_sample = 100
temperature = 0
modelId = "anthropic.claude-instant-v1"
top_k = 250
top_p = 0.999

model_kwargs = {
    "temperature": temperature,
    "max_tokens_to_sample": max_tokens_to_sample,
    "top_k": top_k,
    "top_p": top_p
}

llm = Bedrock(
    client=boto3_bedrock,
    model_id=modelId,
    region_name='us-east-1',
    model_kwargs=model_kwargs,
    streaming=True,callbacks=[StreamingStdOutCallbackHandler()]
    )

message_history = DynamoDBChatMessageHistory(table_name="SessionTable", session_id="10", boto3_session=session)

memory = ConversationTokenBufferMemory(
    llm=llm,  # Use the Bedrock instance
    max_token_limit=100,
    return_messages=True,
    chat_memory=message_history,
    ai_prefix="A",
    human_prefix="H"
)

#add the memory to the Chain
conversation = ConversationChain(
    llm=llm, verbose=True, memory=memory
)

conversation.predict(input="Hello!")

memory.load_memory_variables({})

Expected behavior
Expected Behavior:

The DynamoDBChatMessageHistory should respect the max_token_limit set in ConversationTokenBufferMemory, limiting the token count accordingly.

Actual Behavior:
The DynamoDBChatMessageHistory does not limit the token count as per the max_token_limit set in ConversationTokenBufferMemory and keeps saving all the items in memory on the DynamoDB table.

@samuelbaruffi The BaseChatMessageHistory will store the entire conversation without pruning the Human and AI messages. At runtime, the ConversationTokenBufferMemory will retrieve messages from BaseChatMessageHistory, which, in your case, is DynamoDB. Subsequently, it will prune the messages to the specified token limit in the prompt. Essentially, Memory and Storage are distinct components in Langchain. Memory determines how to construct the past context, while storage actually preserves the entire conversation history. In this scenario, the ConversationTokenBufferMemory prunes the messages obtained from DynamoDB at runtime.

@keenborder786 thanks for the answer. In my tests that does not seem to be the case. I've tested multiple times and my Current conversation: that is the history of the chat keeps increasing without any limit of the tokens I set. In my example I have set max_token_limit=100 and I can see my history being set to the LLM as Current conversation: much bigger than 100 tokens.
Let me know if I could help providing my test scenarios.

https://github.com/langchain-ai/langchain/blob/9e95699277fe0db3bfee1654276a43bfba9ecc64/libs/langchain/langchain/memory/token_buffer.py#L49-L59
The problem happens here:
https://github.com/langchain-ai/langchain/blob/9e95699277fe0db3bfee1654276a43bfba9ecc64/libs/langchain/langchain/memory/token_buffer.py#L58
buffer is just a list map from history storage.
chat_memory : BaseChatMessageHistory isn't implemented the pop method.

Thank you @954-Ivory. So are you saying it's not possible to use max_token_limit with DynamoDB as memory? Could you ellaborate a bit? I'd appreciate.

Thank you @954-Ivory. So are you saying it's not possible to use max_token_limit with DynamoDB as memory? Could you ellaborate a bit? I'd appreciate.

Yes, I think it's a bug.

Thank you! Once you have it working, could you paste the code for the workaround? Appreciate it!

I used Redis to storage history.
You can change it to DynamoDB.
from langchain_community.chat_message_histories import RedisChatMessageHistory
from langchain_core.messages import BaseMessage

class MessageList(list):
    def __new__(cls, *args, history=None, **kwargs):
        cls.history = history
        return super().__new__(cls)

    def pop(self, __index=0):
        if __index == 0:
            self.history.redis_client.rpop(self.history.key)
        elif __index == -1:
            self.history.redis_client.lpop(self.history.key)
        else:
            raise IndexError("Redis doesn't support pop by index.")
        # pop the message from history storage.
        super().pop(__index)

class RedisChatMessageHistoryFixed(RedisChatMessageHistory):

    @property
    def messages(self) -> MessageList[BaseMessage]:  # type: ignore
        messages = MessageList(super().messages, history=self)
        return messages


Get everything going again

Hi there, my name is Kai Salmen and I work at TypeFox since Feb 2021. We have neglected this repository far too long we are committed to contribute here again (see https://github.com/TypeFox/monaco-languageclient/issues/313#issuecomment-1041232925). I will be the first one from TypeFox jumping in again. I have not been involved here before, so I have a learning curve and of course don't know everything, but I will get there. 🙂
I already agreed (https://github.com/TypeFox/monaco-languageclient/issues/313#issuecomment-1041284246) with @CGNonofr to open this uncluttered the dust issue. Let's gather topics here what needs to be done and then we can open other issue when needed:
Already Completed:

Add tags from @codingame releases ✅
Get the release process working again to be able to release npm packages under https://www.npmjs.com/package/monaco-languageclient (#318) ✅
Update CHANGELOG.md (#318) ✅
Fix build issues on Windows (#318) ✅

Required next work (see issue #331 & PR #340):

Some tools/libraries are now quite old (typescript for instance) ✅
The build has unexpected problems (#321) ✅
Do we really need lerna? The only thing being released is the client. Would it make sense to switch to npm workspaces (this options wasn't available in 2018, of course)? ✅
Verify build on every commit: Let's use GitHub Actions. ✅
What about one examples folder. Move the content from ./example to a new sub dir in ./examples. Having two directories is confusing. ✅
#337 ✅
Do we still need the Reconnecting web socket? (https://github.com/TypeFox/monaco-languageclient/issues/313#issuecomment-1041298931) ✅

Examples:
Updates / General topics / Ideas:

A new monaco-editor version was released, we should check the compatibility ✅ (#329)
Introduce semantic release
Add eslint configuration
Improve code where easily possible
Should we make @codingame/monaco-jsonrpc or vscode-ws-jsonrpc a sub-package of this repo?
The examples/browser is confusing people since it doesn't use the whole lib but only the converters

Anything else? Please update the list or leave a comment. Thanks.

Add tags from @CodinGame releases

We need your help on this, @CGNonofr. Once we have the tags, either you or @kaisalmen can publish them to monaco-languageclient (you should both have publishing access now).

done :heavy_check_mark:
There is no v0.17.1 and v0.17.2 because there is no commit on the main branch for it, they were mistakes

@CGNonofr I am currently reconstructing the CHANGELOG and was wondering if we should really re-release your @codingame npm release or if it makes sense to converge on a 0.18.0 and then move onward together? What is your opinion on that?

I'm not sure to really see the point of releasing previous versions, but it's your choice, not mine

I'm not sure to really see the point of releasing previous versions, but it's your choice, not mine

Yes, of course. Just saw that you released 0.17.4. It makes sense to release 0.17.3 as monaco-languageclient) to have the status quo and then release future versions on the main package again (like 0.17.4, 0.17.5 or 0.18.0)?

I released a new version including the currently open MR. I'll let you decide

Ok, I will to a "release test" with a "next" tag first, something like 0.17.4-next.0 and if there are errors or unexpected behaviour release a another next version.
If #317 depends on next monaco-editor release then a new version needs to take this into account.

Ok, we will publish versions 0.14.0 up to 0.17.3, so there is nothing missing. Next new version should then be 0.17.4 including at least the PR 317 and we don't need to wait for next monaco-editor release. 👍
The CHANGELOG should be updated before next release and explain why you needed to have release the npm packages under your namespace and why we released the range of versions later on block to prevent confusion (I started this on the branch referenced above). PR will come, but I can't promise it today. I will start with updating releases first later today.

Versions have been released ✅

@CGNonofr would you favor switching from yarn to npm with workspaces? Today, I would not start a new project with yarn. This strange problem we just observed gives some supportive argument to it as well.

I'm not used to yarn at all, so I would personally use npm as well

@CGNonofr I will release 0.17.4 later today.
I would like to start working towards a 0.18.0 release at the end of the week. Form the above list I see alignment with monaco 0.32.1 / 0.33.0 (should be there soon), fixing build issues (yarn -> npm workspaces), unify example structure as the most important. WDYT?

I'm not sure about monaco 0.33 release soon, what do you mean by alignment?
Do you expect some help? I won't be really available in the next days

I'm not sure about monaco 0.33 release soon, what do you mean by alignment?

Haven't checked/analysed thorougly, yet, but there seem to be API incompatibilities with monaco 0.30.1+.

Do you expect some help? I won't be really available in the next days

I won't be able to do real work before Friday. Don't have a release date in mind, yet, so there is no pressure. If you have time help is welcome. I propose to open a specific next release specific issue where we put the things we really want to do and then we can pick things (if there is time). Sounds good?

Haven't checked/analysed thorougly, yet, but there seem to be API incompatibilities with monaco 0.30.1+.

I'm scared because I'm already using it with monaco 0.32 on production, what are you talking about?

I won't be able to do real work before Friday. Don't have a release date in mind, yet, so there is no pressure. If you have time help is welcome. I propose to open a specific next release specific issue where we put the things we really want to do and then we can pick things (if there is time). Sounds good?

yep :+1:
btw, my goal was always to keep this library usable with the last version of monaco because my company is using it. My employer is not ok with me investing too much time for refactor/structure. I'll do what I can but don't expect big stuff from me

I'm scared because I'm already using it with monaco 0.32 on production, what are you talking about?

Build broke if I remember correctly, but I haven't investigated, yet.

refactor/structure. I'll do what I can but don't expect big stuff from me

I guess, I do the refactor/structure work, but need your reviews/ your opinion on things 🙂

I just tried the 0.19.0-next.0 and everything seem to work (after the vscode-compatibility fix)
Good job :+1:

We introduced version 1.0.0 and 1.0.1 lately. Many tasks from the list above have been completed. I will open new issues for the remaining things and then close this issue.


could not find file license.md

I can't run the sample because this error.
Any sugestion?

Easiest way - copy LICENSE.md from root folder to Camera.MAUI folder.

The file is there but doesn't compile because app don't find it.
I tried set as Resource/Copy if newer but didn't works.
[1452173488405_logo_90x90.png]
Fabio Iko Motta
+5511 99487 6406
AKTTOM Sistemas
www.akttomsistemas.com.brhttp://www.akttomsistemas.com.br

De: CryptKat @.>
Enviado: terça-feira, 26 de março de 2024 04:45
Para: hjam40/Camera.MAUI @.>
Cc: Fabio Iko Motta @.>; Author @.>
Assunto: Re: [hjam40/Camera.MAUI] could not find file license.md (Issue #149)
Easiest way - copy LICENSE.md from root folder to Camera.MAUI folder.
—
Reply to this email directly, view it on GitHubhttps://github.com/hjam40/Camera.MAUI/issues/149#issuecomment-2019599888, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AD3DPPYBYUMISI4KLSV6JH3Y2EKSBAVCNFSM6AAAAABFHMUMWGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJZGU4TSOBYHA.
You are receiving this because you authored the thread.Message ID: @.***>

I've created a pull-request which fixes this problem.
The issue is that some references (inside the .csproj file) are still referencing the file "license.md" but the file has moved and is renamed to "../LICENSE.md"
With a little luck the pull-request is accepted and gets merged soon.


Want to take a photo of yourself directly with your webcam?

when click this link it goes to payment when it must open webcam and let user take photos.
One more issue when user upload full photo we cant see it on website , it must auto crop or let user crop his or her face so it can be visible . how to do these both issue please let me know

I don't understand exactly what you are saying here. Can you be more precise please ?
Are having some problem with the software or is it that you are asking for how to add some specific function ?
You seem to have opened many issues as I see.


Revert "Bundle approve and request in a single action"

Reverts celo-org/celo-monorepo#6431

Not relevant anymore as komenci kit on mainnet was updated to the latest version

Not relevant anymore as komenci kit on mainnet was updated to the latest version


Export from a query with @set commands

Feature request:
Allow user to run a script which consists of just one query; but is preceded by one or more @set commands which set values to variables.
Background: today you can a query and export it automatically; or you can run a script and only view the results in the editor; but you can't run a script and export it automatically.  There's no command to "run script and export to a file."
As Serge pointed out to me, it would be confusing if there was more than one query in the script: which one would you export?  For my purposes, I would be fine with limiting this feature to scripts that only have one executable query.
I see ticket 2855 is sortof related to this request.  But 2855 asks to export multiple queries (presumably into separate files), and I think that's not quite the same thing.

I still think that this is the same as #2855. But #2855 is more general.

Add a new command @export

Export from script works already.
Implementing @export command will requie quite a lot of work. So moving it further.

@serge-rider  you're going to think I'm crazy, but I don't really recall what's the need for this.
As you pointed out at some point, I can click-drag-highlight a list of @set commands and a query, and right-click-execute-and-export the results, today, in current production version.  So, what is the need for a new command?
(Somewhere there is another issue/ticket where we discussed the need to be able to:
Run @set command(s)
Run query
Export
Run more @set command(s)
Run query
Export
etc.
And to execute all of that as a linear script.  Currently dbeaver will execute all of the @set commands before running any queries, and that's not helpful.
I'm not sure if THIS ticket, and a new @execute command, would be intended to solve that problem??)
Given that you say @execute command would be a lot of work... I want to be sure we're all thinking the same things...

Ok.
I think #6041 will covert this ticket as well. Closing then.


Import user defined feature via BIDS

We need to write a script to handle user annotations of features
Example of hand annotations that need to be imported http://studyforrest.org/pages/annot_structure.html

Also add functions to dump database into the same csv format, given that in development especially the database is cleared frequently. stimuli hashes and features should work. if no hash, then its assumed to be a new feature and added to the database. should also be able to ingest 'pure' features not associated with stimuli.

In order to import manually annotated features (e.g. in forrest), we should use pybids and read these features as derivatives in BIDS format. Currently, some of these features may be in dense format which is not yet supported by pybids, thus a script to convert to sparse needs to be written for these features.
Need to add a function to import such features, even when they are not part of the original dataset, and associate them with a dataset already in the db.


Flutter command doesn't run

Flutter doesn't execute commands, it give no output.
I tried deleting the lock file or downloading dart sdk.
If I run command on android studio I get the error: Waiting for another flutter command to release the startup lock.

@blob84
Please check out the official documentation on installing Flutter sdk and confirm you have followed the steps correctly.
waiting for another flutter command means some other flutter command is running in the background process. try to kill all background dart processes that might be running and then start afresh to see if it works.
https://stackoverflow.com/a/53573543

@blob84  Kindly check your Environment Variables Path. Make sure the Path of the Flutter SDK Bin is perfectly set it up.

I reinstalled flutter, now is working.

@blob84 Closing this as resolved based on your comment above.
Thanks.


Linear Search

Submission Title
Linear Search
Submission Type
Algorithm
Submission Description
A linear search or sequential search is a method for finding an element within a list.
Contributors

[x]  C++ -
[x]  C -
[x]  Python -
[x] Java -
[x] JavaScript -
[ ] Golang -

Please assign this task to me. @ahampriyanshu

Please assign this task to me. @ahampriyanshu

What is your preffered language @Pranjal81

Please assign this task to me. @ahampriyanshu

What is your preffered language @Pranjal81

it was C++ but as it is assigned I can do it in C.

#4

Can I work on Javascript on this?
Thanks

Please assign it to me for Python.

Can I go ahead with adding the js file, or should I wait till I get assigned?
I am bit confused
Thanks

@all-contributors please add @Singh-Aishwarya

@all-contributors please add @Singh-Aishwarya  for code

@all-contributors please add @Pranjal81 @Vimlesh-Kumar @Harshi7016 @hemangdtu  for code

@all-contributors please add @hemangdtu for code

@all-contributors please add @Harshi7016 for code

@all-contributors please add @ahampriyanshu  for code

@all-contributors please add @Singh-Aishwarya  for code

@all-contributors please add @Singh-Aishwarya for code

@all-contributors can you assign linear search in go to me?

@ahampriyanshu I have made a PR. Please check.

@ahampriyanshu I have made a PR. Please check.

All conflicts are now resolved.

thanks @everyone :v:


Disable CSRF_COOKIE_HTTPONLY, breaks several OMERO.web apps

Deployed to get web-apps working again, this will need further investigation
Original change: https://github.com/openmicroscopy/prod-playbooks/pull/145/files#diff-21abe72460789362123693324f67520d

Conflicting PR. Removed from build TRAINING-merge#242. See the console output for more details.
Possible conflicts:

PR #153 manics 'cyberessentials: omero-web-patch-settings.yml: SESSION_COOKIE_SECURE = True'

omero/omero-web-patch-settings.yml

--conflicts

Already deployed


[SPARK-44448][SQL] Fix wrong results bug from DenseRankLimitIterator and InferWindowGroupLimit

What changes were proposed in this pull request?
Top-k filters on a dense_rank() window function return wrong results, due to a bug in optimization InferWindowGroupLimit, specifically in the code for DenseRankLimitIterator, introduced in https://issues.apache.org/jira/browse/SPARK-37099.
Why are the changes needed?
Fix wrong results bug.
Does this PR introduce any user-facing change?
Yes, fixes wrong results.
How was this patch tested?
Add sql tests and unit tests.

CC @beliefer @zhengruifeng @cloud-fan

cc @beliefer can you help double check whether there is similar issue in other related Iterator implementations?

Yes, that would be good to double check. In this PR I fixed the iterators for rank and dense_rank. The row_number iterator looks fine (since it just has to increment a counter, doesn't need to do row comparisons.) Those should be the only 3 iterators supported in the InferWindowGroupLimit optimization, but do you know if there may be other iterators elsewhere with a similar problem?

I created a PR with the tests before the fix, so you can see the differences: https://github.com/apache/spark/pull/42042/files#diff-68100e5e6079666747882f5070767601087afc8181f513d997f6e39a948eaa0d

thanks, merging to master/3.5!


Modules Request

Here the outcome of chat with Guillem about modules needed for his project:
Image processing

Web-Cam
OpenCV JS bindings

Sound processing

Pipes
FFT basics

Text-To-Speech
Notifications

SMS
Social outs
Mail
Sound
API

Optional
front-end

Cam-View
Audio (mic)
Johhny-Five controls for servos,...

g

moved to project


Thresholds for FMR=0.1%

Hi, Do you now about the threshold values corresponding to FMR=0.1% for FaceNet and ArcFace models using Euclidean L2 distance?
Thanks,

What FMR is?

I mean the threshold corresponding to false positive rate of 10^-3 on your validation set (e.g., LFW).

Actually, deepface wraps ready-to-use models. I mean that I haven't trained none of them for facial recognition. Those scores can be found in the reference studies or repos cited in the license section of the read me.


Fix for URL obj leak in HTTP2 server push upon calling TSHttpTxnServerPush #2275

Fix for URL obj leak in HTTP2 server push upon calling TSHttpTxnServerPush
https://github.com/apache/trafficserver/issues/2275

[approve ci]

@zwoop @maskit This looks like a back-port candidate for v7.0.0.

Fix #2275

Cherry-picked to 7.1.x


Add metrics for tenants state

Describe your changes
Added a metric that allow to monitor tenants state
Issue ticket number and link
https://github.com/neondatabase/neon/issues/3161
Checklist before requesting a review

[X] I have performed a self-review of my code.
[X] I have added an e2e test for it.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Rust-wise this looks good. Does the created metric allow efficient querying? Should we be worried about removing the metric, say some time after the tenant has been dropped if it hasn't been re-attached? In relation to #3409 and other metric cleaning up tasks.
If we shutdown then there's of course nothing to worry, and the spawned task doesn't really hold up any shutdown either.

This is a gauge that will have for each tenant 1 state, with no histogram so this should be efficient
When a tenant is re-attached it will go through the loading phase


BUG: go schedule main SLSA3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/2241135906
Workflow name: go schedule main SLSA3
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Thu Apr 28 18:16:45 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/2241156449
Workflow name: go schedule main SLSA3
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Thu Apr 28 18:26:22 UTC 2022


Weight label

Can I increase or decrease the weight of the label?
I have not seen this content throughout the doc.

Unfortunately, weighted labels are not supported now. And we don't know how to support it yet.


Error: 'Cannot encrypt() twice with same key + nonce' after Updating to 1.1.0

Since updating from version 1.0.0 to 1.1.0, the following error occurs during aes.encrypt. It was working perfectly before. What could be the cause?
'cannot encrypt() twice with same key + nonce'

You can't reuse key and nonce between two different encryptions. That would break the encryption.
Also you can still do this:
aes(key, nonce).encrypt()
aes(key, nonce).encrypt()

@paulmillr Are you saying that it shouldn't be used like this? It actually causes an error.
const aes = cbc(key, iv);
aes.encrypt('123');
aes.encrypt('456');

Yes, this is invalid.


[Snyk] Security upgrade io.streamnative:pulsar-broker from <IP_ADDRESS> to <IP_ADDRESS>

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

586/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.3
Improper Handling of Length Parameter Inconsistency SNYK-JAVA-ORGECLIPSEJETTY-5902998
io.streamnative:pulsar-broker:  <IP_ADDRESS> -> <IP_ADDRESS>
No
Proof of Concept

496/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 3.5
Arbitrary Code Execution SNYK-JAVA-ORGECLIPSEJETTY-5903003
io.streamnative:pulsar-broker:  <IP_ADDRESS> -> <IP_ADDRESS>
No
Proof of Concept

661/1000  Why? Recently disclosed, Has a fix available, CVSS 7.5
Denial of Service (DoS) SNYK-JAVA-ORGECLIPSEJETTY-5958847
io.streamnative:pulsar-broker:  <IP_ADDRESS> -> <IP_ADDRESS>
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)

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


Build and host docs for each crystal project/library

Host the output of crystal docs for each repository for each tagged release.
This provides an easy way to view the surface of libraries and services that we build.
Ideally, each output should be hosted in a similar fashion to how crystal's API documentation is hosted.
<github pages docs domain>/<repo>/<version>
with a version of latest directing to the most recent tagged version.

Yes! This is something that I've been lusting for. We may be able to solve this a little more generally than a static set of builds. Given this output is very deterministic it is possible to just build these lazily. This way you could hit example.com/<owner>/<repo>/<version> and get what you need.
First load will get hit the latency of the docs build. This could be solved by having a local GitHub Action in each repo run the initial request when a new tag appears so it's not a human experiencing that. We could (ab)use the GH infra further too to both host the output under GitHub pages and run the doc builds themselves. This should just need something like a Cloudflare worker in front of it to trigger a workflow dispatch and return a HTTP 202, or redirect to previously built output.
We can place a filter on the owners / repo this will accept request for, but means others can reuse this too and change the settings for their needs.

I think we should try to solve this for ourselves first in a relatively simple way, which the above is a good path towards.
I'd be hesitant to make such a valuable contribution to crystal tied so heavily to github's infrastructure.
A generic docs service would be the best, docs.rs being the gold standard in my opinion.

This is out of the scope of PlaceOS's platform development.
This however would be an extremely useful thing to develop a basis for - like crates.io


added vec3 class header file

Yashas M Salian

You might also find this article on overloading arithmetic operators using friend functions interesting and relevant.

It would also be convenient to have a unary - operator.

Don't forget to add header guards to your header file!


Please rebase on upstream git

And preserve existing files like AUTHORS etc
Is it a good strategy ?
If yes try to rebase on this branch,
https://github.com/TizenTeam/terminology/tree/sandbox/pcoval/on/upstream/tizen/2.4/mobile
As I reworked the tree to be based on v0.8.0, and then we can try on next version v0.9.0  ?
I will record a short video for @billiob  on Samsung Z2
Related:
https://github.com/voronaam/Terminal3/issues/2

Hi,
I updated to terminology-0.9.1 and reworked tree, please rebase on :
https://github.com/TizenTeam/terminology/tree/sandbox/pcoval/on/latest/tizen
hope this help

It seems you have a bit more time for this project then I do. Would you want to take it on as your project? We'll declare your repo to be the official one and I'll update this one to link there.

As you wish , my time is not unlimited but you have more knowledge that me on the fixes :)
Would you like to share tips in upcoming tizen meeting ?
https://wiki.tizen.org/wiki/Meeting#
Thanks again
Z2


[nrf fromlist] cmake: Fix TINYCBOR in a multi-image context

Fix the TINYCBOR build scripts to work with multi-image.
This patch comes from https://github.com/JuulLabs-OSS/mcuboot/pull/430/files
Signed-off-by: Sebastian Bøe<EMAIL_ADDRESS>

Review needed: @nvlsianpu


Release 0.10.5

Please release 0.10.5 as it will remove a bunch of warnings with Ruby 2.4

Will do later this week :)

Released


Better to kill cita-* process in cita_crosschain.sh for rerunning?

I've seen there's clean data code in cita/tests/integrate_test/cita_crosschain.sh, for rerun test, it would be better to add the kill the cita-* processes, such as:
for pid in cita-forever cita-jsonrpc cita-auth cita-chain cita-network cita-bft trans_evm cita-executor; do
        ps ax |grep ${pid}|awk '{print $1}' |xargs -n 1 -I %  kill -9 % 2>&1 >/dev/null ||true
done

Instead of using the 'for' statement, you can use pkill cita- to kill all processes starting with 'cita-' with less words. And also pkill trans_evm will kill the process of the same name.


Further refactorings and bugfixes

My merge requests contain a lot of code, sorry for that. Currently, I'm trying to resolve all warnings from IntelliJ which are quite a lot.

Is it possible to upgrade the travis-ci build to Java 8? Since it's already a requirement for running DV, I used  some of it's features (Lambdas) and that's the reason the ci build is failing

See the .travis.yml configuration.  Travis is compiling w/Java SE 8.
...
jdk:

oraclejdk8
...

Extract from the Travis log,
...
Switching to Oracle JDK8 (java-8-oracle), JAVA_HOME will be set to /usr/lib/jvm/java-8-oracle
$ java -Xmx32m -version
java version "1.8.0_31"
Java(TM) SE Runtime Environment (build 1.8.0_31-b13)
...

I'm going to close this out for now until you get things fixed up.


Reduce amount of explicit type annotations

⚠ DRAFT PR⚠
By introducing a central type for the buildpack itself, we can leverage associated types to define other types that are valid throughout the buildpack. For example, the type for the buildpack metadata never changes. The same is true about the error type and the target platform. The code has been refactored to reference these associated types instead of requiring explicit values everywhere.

I guess this fixes #68, #70, and may possibly also fix #86?

I think so too! :)


AttributeError: 'Uniswap' object has no attribute 'approve'

Does anyone get this issue `AttributeError: 'Uniswap' object has no attribute 'approve'?
I looked around and it seemed to be here https://github.com/uniswap-python/uniswap-python/blob/master/uniswap/decorators.py#L29 and i don't find any reference to any method with .approve.

Post your code plz

Closing since there's no reply, and no info to go on.


Add implementation of timeseries(subannual) and tests

This PR adds an option for Scenario.timeseries() to return the subannual-column
This will solve the issue reported in https://github.com/iiasa/message_ix/issues/322.
How to review

Review the extended docstring of the timeseries() function and implementation
View changes in the tests and note that the CI checks all pass.
Ensure that changes/additions are self-documenting, i.e. that another
developer like the reviewer will be able to understand what the code does
in the future.

PR checklist

[x] Tests added.
[x] Documentation added.
[ ] Release notes updated.

closes https://github.com/iiasa/message_ix/issues/322

@zikolach @khaeru finally good to be reviewed

Not sure why codecov complains here - all new lines of code are tested. So I guess this is good to be merged anyway @khaeru...

So I guess this is good to be merged anyway @khaeru...

Sorry, I kept coming back and seeing the "Release notes updated" box in the description wasn't checked; so I assumed it was still in progress and didn't look further. But I see now that they actually have been updated; so I've checked the box and will review.

merging as approved by @khaeru & @zikolach


WIP (1.11) Fix dcos-metrics Prometheus producer omitting metrics for some containers

High-level description
TODO
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS-3863 agent metrics prometheus endpoint does not show all containers running on this agent

Related tickets (optional)
Other tickets related to this change:
N/A
Checklist for all PRs

[ ] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label ready for review


Implement CSRF strategy

Because we are storing authentication information in cookies, we need to implement a CSRF strategy for this module.  Any middleware function which is using cookies to authenticate the request should assert that the request is coming from the same domain as the cookie.

Can the access token be used as a csrf token? I came across the concept of double submit cookies. I'm using Angular on the client and its only a simple configuration to have Angular return the access token in the X-XSRF-TOKEN header and then on the server I can compare the cookie and header after the stormpath middleware.
Sounds simple but is it right?

never mind, the access token is rightfully HttpOnly

Hey @robertjd, is this still in the works? I found this during my google search for resources on how CSRF tokens are created and validated in express-stormpath

This is probably going to come in the form of some suggested modules to use, not something that is built into this library.


Incorrect port information after applying uri's assignment operator

After overriding an URI that has port info with an URI that does not have port info, the URI object still tells that it has port info. I guess there is a bug in the implementation of network::uri's assignment operator.
Furthermore, when I pass this re-assigned URI object into the network::uri_builder constructor, the constructed new URI not only still tells it has port info, but also creates a broken URI string when calling network::uri::string(). Checking with Google's AddressSanitizer it reports a heap-use-after-free error. It seems that the string_view that points to the port info is dangling.
Some unit tests for reproduction:
#include <gtest/gtest.h>
#include "network/uri/uri_builder.hpp"

TEST(UriTest, AssignmentOperator) {
  network::uri a("http://a.com:1234");
  ASSERT_TRUE(a.has_port());

  const network::uri b("http://b.com");
  ASSERT_FALSE(b.has_port());

  a = b;
  ASSERT_FALSE(a.has_port()) << a.string();
}

TEST(UriTest, CopyConstructor) {
  network::uri a("http://a.com:1234");
  const network::uri b("http://b.com");
  a = b;

  const network::uri c(a);
  ASSERT_FALSE(c.has_port()) << c.string();
}

TEST(UriBuilderTest, ConstructFromUri) {
  network::uri a("http://a.com:1234");
  const network::uri b("http://b.com");
  a = b;

  network::uri_builder ub(a);  // ASAN reports heap-use-after-free here
  const network::uri c(ub.uri());
  ASSERT_FALSE(c.has_port()) << c.string();
}

int main(int argc, char** argv) {
  ::testing::InitGoogleTest(&argc, argv);
  return RUN_ALL_TESTS();
}

Output
[==========] Running 3 tests from 2 test cases.
[----------] Global test environment set-up.
[----------] 2 tests from UriTest
[ RUN      ] UriTest.AssignmentOperator
src/uri_test.cc:12: Failure
Value of: a.has_port()
  Actual: true
Expected: false
http://b.com
[  FAILED  ] UriTest.AssignmentOperator (0 ms)
[ RUN      ] UriTest.CopyConstructor
src/uri_test.cc:21: Failure
Value of: c.has_port()
  Actual: true
Expected: false
http://b.com
[  FAILED  ] UriTest.CopyConstructor (0 ms)
[----------] 2 tests from UriTest (0 ms total)

[----------] 1 test from UriBuilderTest
[ RUN      ] UriBuilderTest.ConstructFromUri
src/uri_test.cc:31: Failure
Value of: c.has_port()
  Actual: true
Expected: false
http://b.com:1234
[  FAILED  ] UriBuilderTest.ConstructFromUri (0 ms)
[----------] 1 test from UriBuilderTest (0 ms total)

[----------] Global test environment tear-down
[==========] 3 tests from 2 test cases ran. (0 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 3 tests, listed below:
[  FAILED  ] UriTest.AssignmentOperator
[  FAILED  ] UriTest.CopyConstructor
[  FAILED  ] UriBuilderTest.ConstructFromUri

 3 FAILED TESTS

I can also provide ASAN output if needed. My compiler is gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.10) and I use the latest code from the master branch. Thank you for developing cppnetlib, btw -- it's awesome!

Thanks for the report, I will investigate. I also suspect the assignment operator is buggy.

Thanks for the fix. I can confirm my tests and ASan are happy now.


Sprint 2 - Review and Demo

Successes: I successfully made the forearm/claw, even though it took longer than expected. I also helped with the assembly of all the parts together and got started on redoing the intake system
Failures: The forearm/claw took longer than expected and I wasn't sure how to properly build the intake arm with the longer extrustions so my group had to help me
Next steps: Me and Michelle are going to be coding the arm using blocks. Hopefully we could get this done by the end of Sprint 3, but may need some time in Sprint 4.


BUG_1035158

@Santoshtambe,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Updated in wrong place


a few minor edits

The author should do the following, if applicable

[ ] Add tests
[ ] Run tests
[ ] bun run format:fix && bun run lint:fix to format the code
[ ] Add TSDoc/JSDoc to document the code

Hi @sharma-shray, thanks for the suggestion.
It may seem like a change that is obvious to you and doesn't need to be explained, but from the perspective of maintenance, it would be appreciated if the “why” was written in the commit message. Please use the following format as a reference for writing messages in future commits.
https://www.conventionalcommits.org/en/v1.0.0/

Hi @sharma-shray
Thank you for the PR. Looks good. But next time, please follow the @usualoma 's comment.


Pannellum with video.js: Plugin not applied when iOS fullscreen

Although iOS does not support the Fullscreen API, video.js shows an appropriate version of a video when in fullscreen mode. But the Pannellum plugin seems not to be applied in fullscreen and shows the video in its original, distorted, version. Playing the video inline works like a charm, though.
Has this all to do with the lack of Fullscreen API support or is it another issue?

It's my understanding that video fullscreen in iOS uses a native player instead of what's defined in the webpage. Since the video is fullscreen and not the WebGL canvas that Pannellum uses, the projection isn't applied. As far as I know, there's no solution to this. I've never actually been able to get the Video.js plugin to work on iOS at all (I have an iPhone 5c, which Apple dropped support for after iOS 10.3).


No enum constant org.springframework.messaging.simp.stomp.StompCommand.�

We are using stomp-websocket with spring on the server (RabbitMQ) in a react-native project.
Everything was working on ios but on android it was not connecting. We found this related issue with this suggestion to be implemented on the backend: https://github.com/facebook/react-native/issues/12731#issuecomment-330182296
After this suggested patch, it was connected on android but then we started to get this messages both on ios and android:
>>> CONNECT
login:
passcode:
accept-version:1.2,1.1,1.0
heart-beat:10000,10000

<<< CONNECTED
server:RabbitMQ/3.7.4
session:session-lxopMotP6ERQq_z-is_ksw
heart-beat:10000,10000
version:1.2

connected to server RabbitMQ/3.7.4
send PING every 10000ms
check PONG every 10000ms
>>> SUBSCRIBE
id:sub-0
destination:/user/queue/messages

<<< PONG
>>> PING
<<< PONG
 >>> PING
 <<< PONG
 <<< ERROR
message:No enum constant org.springframework.messaging.simp.stomp.StompCommand.�
content-length:0

Unhandled frame:
content-length:84

RROR
message:No enum constant org.springframework.messaging.simp.stomp.StompCommand.
Unhandled frame: content-length:0

any idea what could it be?

I can confirm that strings with NULL must work in order for STOMP to work (https://stomp.github.io/stomp-specification-1.2.html#STOMP_Frames).
I am unclear what all you are running on the back-end and what all on the front-end?

Planning for next version of stompjs is on the way. Your participation will be really appreciated, please join at https://github.com/stomp-js/stompjs/issues/1


error

Hi @Willoverseas ,
May I know what error you encountered?

Sorry to bother you, I have already solved the problem. Thank you for your letter
---- Replied Message ----
| From | @.> |
| Date | 09/21/2023 21:31 |
| To | WindVChen/INR-Harmonization @.> |
| Cc | Li @.>,
Mention @.> |
| Subject | Re: [WindVChen/INR-Harmonization] error (Issue #3) |
Hi @Willoverseas ,
May I know what error you encountered?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


1528 | Shuffle String | C++

Shuffle String
What will change -
this question is not present in C++ folder
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code

Programming Language
Please add/delete options that are not relevant.

[x] C++

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

Can you assign it for me as hacktoberfest contribution :3 @Piyush228

@t3bol90 their are many question in leetcode ,find it which is not present in leetcodeSolution and create an issue then you will definately get assign of new issue you created
if you want new problem then say yes
Happy Coding


WIP: add 闭环

[x] add token about the 闭环 and lexer
[x] add parser about the 闭环 statement
[x] refactor: take reference instead of move on iterating the ast nodes
[ ] add the interpreter logic about the 闭环 statement

BTW, please add some unit tests for this feature.

BTW, please add some unit tests for this feature.

thank your for remind, i've add some test cases about this, please check again 🤝

LGTM


Add @test_approx_eq_sigma_eps for approximate image testing

CC @tbreloff

Cool!

On Sep 26, 2015, at 1:46 PM, Tim Holy<EMAIL_ADDRESS>wrote:
CC @tbreloff
You can view, comment on, or merge this pull request online at:
https://github.com/timholy/Images.jl/pull/367
Commit Summary
Add @test_approx_eq_sigma_eps for approximate image testing
File Changes
M src/Images.jl (3)
M src/algorithms.jl (17)
M test/algorithms.jl (10)
Patch Links:
https://github.com/timholy/Images.jl/pull/367.patch
https://github.com/timholy/Images.jl/pull/367.diff
—
Reply to this email directly or view it on GitHub.

Should I copy/paste this code into Plots for now?  I see the PR is still open, and I'd prefer not to have to track this branch to use it.  Thanks!

Thanks for the reminder, I forgot all about it.

Cool thanks! Now I have to actually write some tests...

On Oct 17, 2015, at 7:16 PM, Tim Holy<EMAIL_ADDRESS>wrote:
Thanks for the reminder, I forgot all about it.
—
Reply to this email directly or view it on GitHub.

I'm just starting to put some test code together... do you have any guidelines as to how I might set sigma/eps?

Nothing special. I'd personally set them to zero until I start seeing test failures, then study the specific failure. Once I understand the reason for the failure (small intensity change? slight change in antialiasing that leads to blurring?) then I'd have a principled choice.
That said, it sounds hard to go wrong blurring by a single pixel. The eps threshold is much harder to guess generically.


Custom Expiry not functioning as expected

I'm submitting a ...  (check one with "x")
[X] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see https://github.com/HackedByChinese/ng2-idle/blob/master/CONTRIBUTING.md#getting-help

Current behavior
Created a cookie based IdleExpiry that stores lastValue as a cookie. Activity in one tab does not prevent timeout in another tab.
I was able to get something close to what I wanted by modifying the toggleState method in Idle and inserting the following at the top of the function:
if (!this.expiry.isExpired()) {           return;         }
This checks the cookie and prevents expiration.
Expected behavior
Activity in one tab prevents timeout in all tabs.
Minimal reproduction of the problem with instructions
Replace SimpleExpiry with the following class
`@Injectable()
export class CookieExpiry extends IdleExpiry {
private lastValue: Date = undefined;
constructor(
private cookieService: CookieService
) {
super();
}
/*

Gets or sets the last expiry date.
@param value - The expiry value to set; omit to only return the value.
@return The current expiry value.
*/
last(value?: Date): Date {
if (value !== void 0 && value !== undefined) {
this.lastValue = value;
let expireDate = new Date(new Date().setFullYear(new Date().getFullYear() + 1));
this.cookieService.put('idle-timeout', this.lastValue.toUTCString(), { expires: expireDate} );
} else {
if (this.cookieService.get('idle-timeout') !== undefined) {
this.lastValue = new Date(this.cookieService.get('idle-timeout'));
} else {
this.lastValue = undefined;
}
}
return this.lastValue;
}

/*

Returns whether or not it is expired.
@return True if expired; otherwise, false.
*/
isExpired(): boolean {
let expiry = this.last();
return expiry !== undefined && expiry <= this.now();
}
}
`
Open app on two tabs, move mouse to trigger activity on one tab while allowing other tab to remain inactive.

What is the motivation / use case for changing the behavior?
Allow activity on one tab to keep other tabs from expiring. Since all tabs are using the same session cookie the "out-of-focus" tab is expiring session for all tabs.
Please tell us about your environment:
Ubuntu 16.04
NPM
Webpack-dev server

@ng-idle version: 2.x
"@ng-idle/core": "^2.0.0-beta.4"

Angular version: 2.x
"@angular/core": "2.1.2"

Browser:
ALL

Language:
ALL

Node: node --version =
v5.12.0

Sorry to bug you but I see responses on other issues, is there something my issue is missing?

@markmaynard Sorry, I've been busy at work and haven't had time to reproduce the issue. If you think you've got a handle on what needs to change to get it working, feel free to submit a pull request. I might be able to take a look this weekend.

I thought I had a fix but I was incorrect. The problem seems to be that activity sets a new timeout time in my cookie, but none of the other tabs actually check the cookie to see that this timeout has been extended. What I think need to happen is that when a timeout occurs on a tab, it first checks the cookie(or other shared storage) to see if is time is greater than the time in the cookie. If not it sets a new timer for the difference between stored_time and now.

Ah, right. So, evergreen browsers (except Edge) support an onChanged event for cookies. That would be ideal (and most analogous to how I handle it via storage in the original ng-idle. However, we'd still need a way to handle browsers that don't support this. It may make sense to create a custom interrupt source that triggers a function at intervals (say, 1 second) to see if the cookie value has changed. If it has, trigger the interrupt. This, in concert with your CookieExpiry should handle activity from other tabs.

If localStorage is an option for you, a new release of ng2-idle has been made last week on tag v2.0.0-beta.6.

I have a cookie storage version that works fine in that it is a centralized store for the timeout value. The problem I ran into is that the idle timeout is set by all the open tabs but the interval in the watch function does not check the the updated values. The change I PRed should fix that.

https://github.com/HackedByChinese/ng2-idle/pull/37 was merged. Should this issue be closed now?

Yes, sorry, closing now.


Investigate Llama Index integration

Investigate Llama Index and determine if there is an integration point with Semantic Kernel that we can demonstrate

LLama.SemanticKernel

Addressed as part of memory investigation.  Can track subsequent work with new issue.


Backwards Compatible Generic Modules

Adds all the backwards compatibility stuff from #566 minus the SPI changes

Did you hit this issue of desiring 10.x to be backwards compatible or did someone else?  Just wondering of the pain level. Also just to say I'm kind of not loving "AvajeModule" as a name ... so I'm hoping that something better might come to me.

Yeah, I noticed that all the plugins and libraries I've made at work ceased to function

Need to resolve why InjectPlugin exists

The maven plugin/generator is unable to get the type array from legacy plugins.

don't like "AvajeModule" as a name

I was thinking of AvajeModule sort of like Guice has GuiceModule.
I got a couple of alternate options (just spitballing here):

BeanModule
ScopeModule
GraphModule
WiringModule
DependencyModule

AvajeModule sort of like Guice has GuiceModule

Yeah, that makes sense. IMO the original "Module" was a poor choice by me. "InjectModule" is taken by the annotation (which seems ok) ... so I'm definately warming to AvajeModule (it's mostly used internally).
I should get to review this again today. Great work by the way !!!


Does it support version 2.4

Does the sdk support 2.4 version of the chain

https://github.com/hyperledger/fabric-gateway is supported for Fabric v2.4 onwards.

Thank you revert。I see that this code does not support contract management member management？

it support fabric version 2.4,I use fabric-sdk-go v1.0.0,and i can  install ,getInstalled,commit approve contract,you can refer my github repo https://github.com/iamlzw/blockchain-benchmark/blob/master/tools/chaincode.go


Add search results download manifest

Summary
We need to retrieve specific data fields from a service to insert into our search results download for Direct Award frameworks. This adds a new manifest which lists the question IDs in the order they should appear in the download file.
I've added the files for historical iterations because the manifest must be available for closed frameworks as well as live ones, and our current standard is to load manifests during start-up rather than on-demand, so the manifests must exist for all versions.
Bump version to 8.15.0.
Required by
https://github.com/alphagov/digitalmarketplace-buyer-frontend/pull/615
Other associated PRs
https://github.com/alphagov/digitalmarketplace-api/pull/656
https://github.com/alphagov/digitalmarketplace-utils/pull/332
https://github.com/alphagov/digitalmarketplace-apiclient/pull/98
https://github.com/alphagov/digitalmarketplace-briefs-frontend/pull/32
Ticket
https://trello.com/c/gs2I3Cj2/676-end-your-search

I think only G-Cloud 6 saved the priceString as part of the JSON.  For G7 and G8 we have :
"priceMax", "priceMin", "priceUnit" and "priceInterval" in the service JSON but the display string was then constructed from these elsewhere.

Left some minor comments, see what you think.  The proof of the pudding is in the eating - if this is all working for you locally (multiquestions, etc.) then this is probably OK to merge.

Addressed comments. The content loader can manage it by iterating over sections and then getting all the fields for that section.

I don't think I follow why we need G6/7/8?

Because of how we/I load the manifests on the front-end (i.e. there's a loop that loads manifests for all G-Cloud iterations).

Got it. Yeah we should have some kind of concept of optional features or something, but not now.


John Lins' Site is Broken

URL: https://www.johnlins.com

cc: @JohnLins

cc: @JohnLins

alright, all good now!

alright, all good now!


refactor tabular tabs out of RAI dashboard into a separate component

Description
Refactor tabular tabs out of RAI dashboard into a separate component.
This is in preparation for adding tabs/components related to text and image scenarios.
Areas changed
npm packages changed:

[ ] responsibleai/causality
[ ] responsibleai/core-ui
[ ] responsibleai/counterfactuals
[ ] responsibleai/dataset-explorer
[ ] responsibleai/fairness
[ ] responsibleai/interpret
[ ] responsibleai/localization
[ ] responsibleai/mlchartlib
[x] responsibleai/model-assessment

Python packages changed:

[x] raiwidgets
[ ] responsibleai
[ ] erroranalysis
[ ] rai_core_flask

Tests

[x] No new tests required.
[ ] New tests for the added feature are part of this PR.
[x] I validated the changes manually.

Screenshots (if appropriate):
Documentation:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Codecov Report

Merging #1256 (d9a47c6) into main (2dd3799) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1256   +/-   ##
=======================================
  Coverage   89.85%   89.85%
=======================================
  Files          26       26
  Lines        1331     1331
=======================================
  Hits         1196     1196
  Misses        135      135

Flag
Coverage Δ

unittests
89.85% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Remove procedures from picostdlib.nim

I placed sleep() and sleepMicroseconds into the time.nim module and moved all of the other proc's inside picostdlib into stdio.nim . Personally, I prefer it this way, and believe it is more consistent across the board.
I was not sure what effects deleting the picostlib.nim file would do to nimble install as a hybrid installation, so I didn't delete it yet.

I tested the following program, and it worked fine:
import picostdlib/[gpio, time, stdio]

stdioInitAll()

let led = 20.Gpio
led.init()
led.setDir(Out)
while true:
  led.put(High)
  sleep(1500)
  led.put(Low)
  print("blink")
  sleep(1500)

Bump the nimble version and update all examples.

would that mean you would just use
import picostdlib

and never have to worry about importing individual submodules?

Indeed, it'd export all sub modules making life easier if wanted.

do you think we want that, or do we want granular control? would there be a difference in size of the .uf2 file? I assume that Nim does not compile/import procs and modules which are never used?

It was just an idea. It does not compile code that's not needed, only code that's used.

I suppose to be idiomatic like the Nim standard library, we should just have one module imported?

I mean you can selectively import if needed, it's not really idiomatic, just an option.

okay. I don't mind import each submodule on it's own. So I guess we will leave it. If I try to delete the picostdlib.nim file however, I get the following error when I run nimble install:
Error: Could not find a file with a .nimble extension inside the specified directory: /home/casey/Desktop/Learning/nim/pico-nim stdlib/picostdlib/docs

I remember running into this problem in another project. The solution is to leave it empty

I plan on making another PR very soon that will completely overhaul the examples folder. Can we have broken examples for a little bit and approve this PR?

Sounds fine for now, if you do not get the examples overhauled in due time I'll fix it myself.

sounds good!


Support values diff for editor apply apis

Signed-off-by: Tamal Saha<EMAIL_ADDRESS>

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


Add console.log stack dumping

While debugging it is very useful to have all possible information in
the rare (cough!) event of an exception.  While both the font-inspector
and the glyph-inspector were notifying the user of exceptions via the
web page, the additional information useful to developers was not seen.
This change adds logging of an exception's stack information to these
two web pages.  Previously little or nothing would be seen in a browser's
debug console window:
(font-inspector)
           A CID Font!

(glyph-inspector)
         A CID Font!
  Uncaught ReferenceError: xxfont is not defined  glyph-inspector.html:381

This change adds logging of the exception along with stack information:
           A CID Font!
  ReferenceError: xxfont is not defined           glyph-inspector.html:380
      at Object.parseCFFTable [as parse] (opentype.js:2271)
      at Object.parseBuffer [as parse] (opentype.js:916)
      at FileReader.reader.onload (glyph-inspector.html:375)
  Uncaught ReferenceError: xxfont is not defined  glyph-inspector.html:381

Thanks! Sorry it took so long.


Move bounds check out of loop in fwd_txfm2d_daala

Provides a very minor performance improvement.
In order to auto-vectorize, more bounds checks
will need to be elided, which are more difficult to remove
without unsafe code.

Coverage decreased (-0.004%) to 75.384% when pulling f9f6e0e0513de4e4ebfb0ff71db9c0d29836d0f1 on shssoichiro:fwd-txfm-bounds-check into 1e0a8ce78a8db533bb813641dbd4e2a17c30d48b on xiph:master.


App2App flow

App2App Flow

Explain  in a few lines the purpose of this pull request

Issue link: N/A
Doc Issue: N/A

Development Checklist

[ ] Build complete solution with pull request in place.
[ ] Ran checkstyle plugin with pull request in place.
[ ] Ran Findbugs plugin with pull request in place.
[ ] Ran FindSecurityBugs plugin and verified report.
[ ] Formatted code according to WSO2 code style.
[ ] Have you verified the PR doesn't commit any keys, passwords, tokens, usernames, or other secrets?
[ ] Migration scripts written (if applicable).
[ ] Have you followed secure coding standards in WSO2 Secure Engineering Guidelines?

Testing Checklist

[ ] Written unit tests.
[ ] Verified tests in multiple database environments (if applicable).
[ ] Tested with BI enabled  (if applicable).

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


Footer section in small screen needs redesigning

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
The footer section looks like a vertical line of texts
And
If you check the screenshot you can see that the last social button can't even be clicked
Expected behavior
Footer should not take full screen height and last link should be visible
Add ScreenShots

On which device are you experiencing this bug?


err

ERROR #11321  PLUGIN
"gatsby-source-hubspot-forms" threw an error while running the sourceNodes lifecycle:
TypeError: Cannot destructure property 'createNode' of 'boundActionCreators' as it is undefined.
52 |     return
53 |   } catch (err) {

54 |     throw new Error(err)
|           ^
55 |   }
56 | }
57 |

File: node_modules\gatsby-source-hubspot-forms\gatsby-node.js:54:11

https://www.gatsbyjs.com/docs/reference/release-notes/migrating-from-v2-to-v3/#removal-of-boundactioncreators


Service containers using volumes_from do not get re-deployed with new target volumes

For a service using volumes_from, it will get the volumes of the target container when it is first deployed:
stack: test/volumes-from
services:
  redis:
    image: redis
  backup:
    image: redis
    volumes_from:
      - redis-%s
    command: sleep 1000

$ docker inspect -f '{{range .Mounts}}{{if eq .Destination "/data"}}{{.|json}}{{end}}{{end}}' volumes-from.redis-1 volumes-from.backup-1
{"Name":"8680965e850f3471872891c4cbf9e53ce4d89fc49197fd1a9064ce3e732cbb60","Source":"/var/lib/docker/volumes/8680965e850f3471872891c4cbf9e53ce4d89fc49197fd1a9064ce3e732cbb60/_data","Destination":"/data","Driver":"local","Mode":"","RW":true,"Propagation":""}
{"Name":"8680965e850f3471872891c4cbf9e53ce4d89fc49197fd1a9064ce3e732cbb60","Source":"/var/lib/docker/volumes/8680965e850f3471872891c4cbf9e53ce4d89fc49197fd1a9064ce3e732cbb60/_data","Destination":"/data","Driver":"local","Mode":"","RW":true,"Propagation":""}

If the target service is re-created:
$ bin/kontena service deploy --force volumes-from/redis
 [done] Deploying service volumes-from/redis
⊛ Deployed instance development/volumes-from/redis-1 to node core-01

The server will also deploy the dependent services:
<IP_ADDRESS> - - [07/Dec/2017:09:55:01 +0000] "POST /v1/services/development/volumes-from/redis/deploy HTTP/1.1" 200 246 0.0853
I, [2017-12-07T09:55:01.736906 #10]  INFO -- GridServiceSchedulerWorker: starting development/volumes-from/redis deploy
...
I, [2017-12-07T09:55:06.434518 #10]  INFO -- GridServiceSchedulerWorker: deploying dependent service development/volumes-from/backup of deployed service development/volumes-from/redis
I, [2017-12-07T09:55:06.464867 #18]  INFO -- GridServiceSchedulerWorker: starting development/volumes-from/backup deploy
...
I, [2017-12-07T09:55:10.228423 #18]  INFO -- GridServiceDeployer: service development/volumes-from/backup has been deployed

But this does not cause the agent to do anything, because it only checks the volumes-from.backup-1 vs the volumes-from/backup service updated_at, not the target volumes-from.redis-1 container:
D, [2017-12-07T09:55:01.780748 #1] DEBUG -- Kontena::RpcServer: rpc request: Kontena::Rpc::ServicePodsApi#notify_update [[]]
D, [2017-12-07T09:55:02.083243 #1] DEBUG -- Kontena::Workers::ServicePodWorker: state of volumes-from.redis-1: running
I, [2017-12-07T09:55:02.123282 #1]  INFO -- Kontena::Workers::ServicePodWorker: service updated at 2017-12-07T09:55:01+00:00 after service container created at 2017-12-07T09:47:45+00:00
I, [2017-12-07T09:55:02.123515 #1]  INFO -- Kontena::Workers::ServicePodWorker: re-creating volumes-from.redis-1
I, [2017-12-07T09:55:02.123593 #1]  INFO -- Kontena::ServicePods::Creator: creating service: volumes-from.redis-1
I, [2017-12-07T09:55:05.005090 #1]  INFO -- Kontena::ServicePods::Creator: removing previous version of service: volumes-from/redis-1
D, [2017-12-07T09:55:05.328960 #1] DEBUG -- Kontena::ServicePods::Creator: creating container: volumes-from.redis-1
D, [2017-12-07T09:55:05.453940 #1] DEBUG -- Kontena::ServicePods::Creator: container created: volumes-from.redis-1
I, [2017-12-07T09:55:05.949603 #1]  INFO -- Kontena::ServicePods::Creator: service started: volumes-from/redis-1
D, [2017-12-07T09:55:06.059840 #1] DEBUG -- Kontena::Workers::ServicePodWorker: sync volumes-from/redis-1 state at 2017-12-07 09:55:01 UTC: {:service_id=>"5a290e3bb59a38000a195f8a", :instance_number=>1, :rev=>"2017-12-07 09:55:01 UTC", :state=>"running", :error=>nil}
D, [2017-12-07T09:55:06.091875 #1] DEBUG -- Kontena::Workers::ServicePodWorker: container event: create
D, [2017-12-07T09:55:06.092788 #1] DEBUG -- Kontena::Workers::ServicePodWorker: container event: start
D, [2017-12-07T09:55:06.092931 #1] DEBUG -- Kontena::Workers::ServicePodWorker: state of volumes-from.redis-1: running
D, [2017-12-07T09:55:06.185412 #1] DEBUG -- Kontena::Workers::ServicePodWorker: state is in-sync: running
D, [2017-12-07T09:55:06.212218 #1] DEBUG -- Kontena::Workers::ServicePodWorker: skip volumes-from/redis-1 state sync at 2017-12-07 09:55:01 UTC: unchanged
D, [2017-12-07T09:55:06.551107 #1] DEBUG -- Kontena::RpcServer: rpc request: Kontena::Rpc::ServicePodsApi#notify_update [[]]
D, [2017-12-07T09:55:06.863288 #1] DEBUG -- Kontena::Workers::ServicePodWorker: state of volumes-from.backup-1: running
D, [2017-12-07T09:55:09.922037 #1] DEBUG -- Kontena::Workers::ServicePodWorker: state is in-sync: running
D, [2017-12-07T09:55:09.941498 #1] DEBUG -- Kontena::Workers::ServicePodWorker: sync volumes-from/backup-1 state at 2017-12-07 09:55:06 UTC: {:service_id=>"5a290e3bb59a38000a195f8e", :instance_number=>1, :rev=>"2017-12-07 09:55:06 UTC", :state=>"running", :error=>nil}

This means that the container using volumes_from will no longer have the same volumes:
$ docker inspect -f '{{range .Mounts}}{{if eq .Destination "/data"}}{{.|json}}{{end}}{{end}}' volumes-from.redis-1 volumes-from.backup-1
{"Name":"3fe9a8354c29012830ec67f4e6ff972dcce7695c049db92e190931d16ee6fd4c","Source":"/var/lib/docker/volumes/3fe9a8354c29012830ec67f4e6ff972dcce7695c049db92e190931d16ee6fd4c/_data","Destination":"/data","Driver":"local","Mode":"","RW":true,"Propagation":""}
{"Name":"8680965e850f3471872891c4cbf9e53ce4d89fc49197fd1a9064ce3e732cbb60","Source":"/var/lib/docker/volumes/8680965e850f3471872891c4cbf9e53ce4d89fc49197fd1a9064ce3e732cbb60/_data","Destination":"/data","Driver":"local","Mode":"","RW":true,"Propagation":""}

A more relevant usecase would be adding a new volume: - test:/test to the target service:
$ docker inspect -f $'{{$name := .Name}}{{range .Mounts}}{{if or (eq .Destination "/data") (eq .Destination "/test")}}{{$name}}: {{.|json}}\n{{end}}{{end}}' volumes-from.redis-1 volumes-from.backup-1
/volumes-from.redis-1: {"Name":"volumes-from.redis.test-instance-1","Source":"/var/lib/docker/volumes/volumes-from.redis.test-instance-1/_data","Destination":"/test","Driver":"local","Mode":"z","RW":true,"Propagation":"rprivate"}
/volumes-from.redis-1: {"Name":"232b63537c62ef57510e06445fc52113f7ed82b862f9e1151d8c040000f5581a","Source":"/var/lib/docker/volumes/232b63537c62ef57510e06445fc52113f7ed82b862f9e1151d8c040000f5581a/_data","Destination":"/data","Driver":"local","Mode":"","RW":true,"Propagation":""}

/volumes-from.backup-1: {"Name":"8680965e850f3471872891c4cbf9e53ce4d89fc49197fd1a9064ce3e732cbb60","Source":"/var/lib/docker/volumes/8680965e850f3471872891c4cbf9e53ce4d89fc49197fd1a9064ce3e732cbb60/_data","Destination":"/data","Driver":"local","Mode":"","RW":true,"Propagation":""}

Workaround is to kontena service deploy --force the service using volumes_from after any changes to the target service volumes.

Per #3100 the services using volumes_from no longer get re-deployed by the server, but it doesn't really matter because the agent wasn't picking the volumes_from changes either.
I suspect that the fix for this issue would be something in the agent to check for changes in any of volumes_from containers, and also re-create the container in that case.


Any reason for Java 18 instead of 17?

Hey!
I finally got around to implementing your version of the api.
Unfortunately, I can not run it, since I am running JDK 17.
Any specific reason you are not using the LTS Version 17, but rather 18?
Kind regards
Dustin

I checked it out, changed to version 17 and tried to build.
Not working :/
Removed maven plugins: sign and javadoc -> building
Using that version -> Caused by: java.lang.ClassNotFoundException: retrofit2.converter.gson.GsonConverterFactory
at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:641)
at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188)


Add Mask subclass

This PR:

Adds the Mask class,
Casts any boolean array passed to Raster.from_array() into a Mask
Makes __array_function__ interface compatible with logical functions, automatically cast into a Mask
Changes output of Vector.create_mask() and rasterize() with certain parameters to Mask,
Changes __eq__ into raster_equal().

Write new tests for all.
Resolves #316

Ready for review, just need to make mypy happy and it's all done.

Closing this, as it had to be developed directly in #351. See there for the final Mask features and tests.


Enhancement: Allow configuration of an application via an AppConfig instance

We already have AppConfig in place, so it would be nice to be able to do something like
from starlite import Starlite, AppConfig

config = AppConfig(...)

app = Starlite.from_config(config)

Cool, why not

How would you recommend this be implemented ?
Should the from_config just call init under the hood
@classmethod
def from_config(cls, config):
    return cls(**config) // this wont work since config is not a dict

Something like this ?

Should the from_config just call init under the hood

Something like calling Starlite(**config.dict(exclude_unset=True)) would certainly work, but then we just turn around and construct another AppConfig object inside __init__ (not that performance is an issue, as it only happens once).
We could add app_config: Optional[AppConfig] = None to the Starlite.__init__() signature, and if it is provided, we use that as the initial config object that is passed to the on_app_init hooks, in lieu of constructing our own. If that was the case, the constructor method wouldn't be necessary as we could simply call Starlite(config=AppConfig(...)), but we'd have to work out precedence rules, e.g., if both route_handlers and config were given to the constructor, then should they be merged, would one take precedence over the other etc etc.
Starlite has a lot of __init__ params, maybe an evolution into 2.0 could be adding the config object to the signature, and then in 2.0 making config of the app only via config object.

maybe an evolution into 2.0 could be adding the config object to the signature, and then in 2.0 making config of the app only via config object.

I'm not sure about that. Feels a bit cumbersome having to construct a config object and pass it immediately to Starlite.
I think for now, the best approach would be to do what you described, but @overload __init__, to only allow either kwargs or a config object, and then just discard all kwargs if a config object is passed.

I think for now, the best approach would be to do what you described, but @overload init, to only allow either kwargs or a config object, and then just discard all kwargs if a config object is passed.

SGTM. We'd need to make route_handlers optional for that, but I can't see why that should be a problem, and also put default values on the AppConfig object

Can we initialize AppConfig from .env's ?

No. It's mostly Python objects, so adding .env-file support doesn't make a lot of sense.

There's not a lot of settings there that would lend to being directly set by environment.
I go pretty heavy on the config-via-environment approach in starlite-saqlalchemy, but mostly that is using environment to configure things like cache config, openapi config, sqlalchemy plugin etc, which are then passed to the app via AppConfig as a on_app_init hook.

@provinzkraut @peterschutt  Thank you for informing, One more question generally we could set defaults in dunder init for AppConfig or create a settings.py like how django does it (Although I think that would over complicate things for this issue)

One more question generally we could set defaults in dunder init for AppConfig

If you take a look at AppConfig, you'll see that it's a Pydantic model. So the correct way to set defaults would be on the class attributes themselves.
I don't think settings.py fits the Starlite way of doing things, and I don't see any benefit of this. Even for Django this makes things more complicated. It's opaque "automagic", which almost never is a good idea.

ok thanks for the help

The one that stands out to me most is the lifecycle hooks., having a LifecycleHookConfig makes sense as a grouping IMO, and would significantly reduce the number of parameters to the application.

Well it would certainly make sense, and it'd be 8 parameters we could remove. On the other hand
app = Starlite(..., on_startup=on_startup)

is way more convenient than
app = Starlite(..., lifecycle_hooks_config=LifecycleHooksConfig(on_startup=on_startup))

@wassafshahzad were you working on this?

@JacobCoffee - you can work on this

@wassafshahzad were you working on this?

Yes will have a pr up by tomorrow


widget.CurrentLayoutIcon cannot find custom icons

Going deeper in this rabbit hole, i'm trying to use custom icons for layout.
 widget.CurrentLayoutIcon(
                       scale = 0.66,
                       custom_icon_paths = "~/.config/qtile/assets/layout",
                       #use_mask = True,
                       foreground=colors[3],
                       ),

The widget does not detect the icons.
My guess is that the ~ is not expanded.

Does it work if you replace it with os.path.expanduser("~/.config/qtile/assets/layout")?

Also, this bug is also in the main repo. Looking at that code, no expanduser call is added to that folder but, more importantly, custom_icon_paths should be a list of folders.

Ok !
So this  custom_icon_paths = ["~/.config/qtile/assets/layout"] didn't work.
But this custom_icon_paths = [os.path.expanduser("~/.config/qtile/assets/layout")] worked

Yes. So there is a bug in the main qtile version of the widget as it never calls os.path.expanduser on those custom paths.

Ok i will close this one and open on qtile.
Although i found something else. It seems the widget can't open .svg files.
That one is qtile-extra ?

No. That's qtile as well. What OS are you on?

Arch linux.
widget.image has not issue to load .svg.
Just the current layout widget is problematic

How i found something interesting.
The widget can work with .svg but only if there is no .png in the same folder


Disable warnings properly in SKDB Workers

Previously, SKDB workers created with createWorker weren't being initialized with the specified disableWarnings option, resulting in warnings being printed during make test-wasm's API tests.

Amazing! I've had this on my todo list for a few weeks and keep failing to get to it. Thanks.


chore: avoid git checkout when visiting git history

We can retrieve all logs at once with git log -10 --pretty=format:///%B///%H/// -- /path/to/crate
with this approach we can retrieve git commits in batches (in this example of 10).
We don't need to do git checkout.
/// is a separator. We can use something more specific, such as @@release@plz@@ to avoid conflicts with git commit messages.
With this, we don't need to copy to a temporary directory when doing release-plz update.

[x] check git history of crates in parallel? Answer: not possible for now

this approach doesn't work. We need to do git checkout because we need to compare the content of the directory


DOC: add open_mfdataset in See Also

Makes life a little easier when navigating the docs

Thank you @raybellwaves ! These are always appreciated.


Rainbow connection

Removed some redundancy in the phase organization by consolidating the sections in the introduction into the first page of each phase section.
Fixed issues with images
General rewording, typo fixing, etc
Added notes on AAA batteries and cases to hardware section
Added a troubleshooting section with common linux commands (still needs work)
started some reformatting of Phase 2, but stopped short (requires further discussion)
Moved document conventions to separate subsection
Split out ToC
Other modifications and fixes I can't quite remember

Not sure what the automatic merge issue is; let me know if I can help alleviate that

Looks pretty good to me. Noticed a couple really minor things I can fix when resolving the merge conflict. Only regression I see is that you got rid of the TOC embedded in the README, so now there's no good way to link to all the content without using the gitbook. We should probably readd that, and add a bunch more links within the new phase sections to each of their component pages. It should be possible for someone who just lands on the repo to read the first page, start clicking to follow links and read things in order, and not miss any content.

Yeah, I went back and forth on the TOC for that same reason—having it on the landing page. The intro had just become so long and I thought having an easily-found TOC (i.e. one that had an explicit sidebar callout) would be good.
I get what you're talking about with the pages being linked to each other, but is there any particular reason for someone not to view it in the gitbook?

Gitbook doesn't work at all for reviewing things in a dev branch, for one thing. We have had discussions about ditching gitbook entirely due to unexplained update delays etc.  And some people just won't notice the gitbook link. All of those point to making the repo self-contained, with gitbooks an optional nice UI.
-Scott
On Tue, Nov 17, 2015 at 9:26 AM -0800, "Chris Hannemann"<EMAIL_ADDRESS>wrote:
Yeah, I went back and forth on the TOC for that same reason—having it on the landing page. The intro had just become so long and I thought having an easily-found TOC (i.e. one that had an explicit sidebar callout) would be good.
I get what you're talking about with the pages being linked to each other, but is there any particular reason for someone not to view it in the gitbook?
—
Reply to this email directly or view it on GitHub.

Finally had a chance to review (all) of this great work! I like it. A few thoughts:
-what about removing or instead embedding the first image of the setup on the Intro page? that takes a lot of space, and it's included elsewhere. I think that would give us more space, and I'd like the TOC on the home page at the bottom since more people are coming to github rather than using gitbooks to nav through this, and then beyond phase 0-4 they can see all the great stuff as a part of these docs.
-Separate TOC is also good to have.
With that, I'll give this another +1. (We'll go ahead and merge #35 and then can talk about how to proceed with these changes so they're compatible).

Closing because #38


First feature (Quotes Rotator) - Create container with basics - mobile version`

[ ] Create container for <img> and <p> text on page

[ ] Container should be centered on page and given percent width
[ ] Include margin-top to position the content away from the top of the page
[ ] Lightgray border-top on container (for now, eventually progress type animation with CF blue)
[ ] within container include a <p> element and an <img> where text is allowed to wrap below

[ ] text within <p> is lightgray sans-serif and relatively large

[ ] include element stacked below the first two tags with the authors name

[ ] text within element is sans-serif, CF blue, and relatively small

#20


[TH2-3133] Put RawMessage.Metadata.properties to MessageToStore.metadata

Please review and help with README and versions

The special build is not stable, details are placed in the TH2-3148 issue


Having trouble importing epub

I encountered a "not exist" error when importing some epub files, while other epub files did not report an error. i don't konw why

Do you have some example epub files that you can share? Ideally, at least 1 success and 1 failure file.

Anki 24.11 (87ccd24e)  (ao)
Python 3.9.18 Qt 6.6.2 PyQt 6.6.1
Platform: Windows-10-10.0.19041
===Add-ons (active)===
(add-on provided name [Add-on folder, installed at, version, is config changed])
Advanced Browser ['874215009', 2024-10-27T22:20, 'None', '']
AnkiConnect ['2055492159', 2024-11-07T09:34, 'None', mod]
Edit Field During Review Cloze ['385888438', 2024-11-16T18:23, '6.21', mod]
Enhanced Cloze for Anki 21 ['1990296174', 2023-09-11T02:48, 'None', '']
FSRS Helper Postpone  Advance  Load Balance  Easy Days  Disperse Siblings ['759844606', 2024-12-07T19:00, 'None', '']
Image Occlusion Enhanced ['1374772155', 2022-04-09T15:15, 'None', '']
Incremental Reading v4119 unofficial clone ['999215520', 2023-11-23T01:54, 'None', '']
===IDs of active AnkiWeb add-ons===
1374772155<PHONE_NUMBER><PHONE_NUMBER> 385888438 759844606 874215009 999215520
===Add-ons (inactive)===
(add-on provided name [Add-on folder, installed at, version, is config changed])
Uploading epub.zip…

Thank you for providing the sample files!
I just tried to import all 3 of them in my Mac machine and they all succeeded. I suspect this might be something Windows specific.
Can you try accessing the file that were reported as "Not exists" and see if it's there?
I also see this part of the file name that doesn't exist in the file you sent me. What if you remove all special characters (e.g. (, ), .) in your epub file?


feat: upgrade hyper to 1.0

Some of the API need redesign.

Continue the work in https://github.com/compio-rs/cyper/pull/6


[Old API] Add Product API

Hi,
is possible to add support for the Product API ?
http://developer.ebay.com/Devzone/product/CallRef/index.html
this API has the benefit to return ePID + ISBN + UPC + EAN the finding API does only return ePID

I'm currently working on adding several of the new APIs that have been announced so can look at supporting the Product API as well. Aiming for a January release.

@davidtsadler thx

just an idea, but a little dev oriented doc (structure, command lines, etc) could give some pointers to others devs on how to fork and add features to your excellent SDK. I ve seen some generated files...

I've been thinking about something along the same lines. It would be something to look at after I've supported the other APIs.

good to know ! keep the good work

thx


[BLAS] Upstreaming portBLAS as GENERIC SYCL BLAS implementation

This PR upstreams the long work done with portBLAS as generic SYCL implementation of BLAS routines.
Everything is added under the onemath/sycl/blas directory and meant to be used as header-only library.
An example of the current usage is the oneMath project where portBLAS is the generic backend for BLAS.

The PR was closed due to a technical GitHub difficulty with the commit history. We were able to push to the main branch instead and preserve the commit history.


django-imgix! 😁

Hey Pancentric folks! I just caught wind of this library. I'm Paul, Developer Relations at imgix. I wanted to stop by and let you know I'm happy to help answer any questions you have, or lend a hand with the library if you'd like. Just let me know how I can help.
If you're interested, we'd also be happy to pull this into the https://github.com/imgix organization, and make it an official library. Of course, you'd retain admin access if we went this route. Either way is fine by us, but I wanted to make sure you had the option.
Anyhow, it'd be great to get this listed as an available library on https://www.imgix.com/docs/libraries and in https://github.com/imgix/imgix-blueprint. If that's good by you, I'll go ahead and get it added to those.

Hi Paul,
Thanks for noticing out module. We would be more than happy to have it listed on your libraries page, however we would like to keep it under our namespace if that's ok with you.
If you have any suggestions or changes in mind please let us know / open an issue!
Cheers
Alex

That's totally okay! Just wanted to make sure I offered. If you ever have any questions etc., feel free to @paulstraw me. We've also got an imgix community Slack set up where you can hang out/ask questions, if that's your cup of tea: slack.imgix.com
I'll get this linked in our docs and then close this issue!
-paul


Inverse impossible to define

Hi Andrej,
I think that upper_bound : {r : Q | upper r} won't let you define the inverse of a real number, because of the sort Type.
Take a positive real number x, but very very small. Then a rational number q > 1/x will be gigantic, and the only way to compute it is to find a rational number 0 < r < x. That suggests to use lower_open x 0, but you cannot because it is in sort Prop.
Anyway why did you use the sort Type for the bounds and the sort Prop everywhere else ? As we already discussed you want the Dedekind reals, not the Cauchy reals, so you probably want Prop everywhere.

At first I tried to use Type and later I realized it should be Prop. I think the upper_bound is just a remnant of that, I forgot to fix it. We should just switch to upper_bound : { r : Q & upper r }. Will you make the fix? (If so, feel free to close the issue as well.)

Do you mean upper_bound : exists r, upper r? Yes, I can do this fix.

This was closed by #15


Reduce size of Docker image

Our Docker image is based on Ubuntu Xenial, making it hundreds of MB in size. Reducing it would improve download times and disk usage.
Other well known distributions that can be used as base for Docker images are Alpine and BusyBox. These may not be as straightforward to use as Ubuntu, with more packages needing to be manually installed.

fixed by #90


Repurpose Repo

@finn-tbd and others. let me know if moving the current test harness into the test-harness directory works

all the  files that come from the tbd template repo should probably remain in the root (CODEOWNERS, CODE_OF_CONDUCT.md, CONTRIBUTING.md, LICENSE, GOVERNANCE.md) as should action.yml (must be in the root to use this repo as a gh action in other repos). Otherwise this seems fine.

@finn-tbd makes sense. done https://github.com/TBD54566975/sdk-development/pull/10/commits/a94cabdf3b518f84be8278800ab1ed3185511224


React 18 peer dependencies @radix-ui hooks

Current Behavior
Can we bump up these 2 hooks from radix-ui to latest versions?
@radix-ui/react-use-escape-keydown
@radix-ui/react-use-previous

Expected Behavior
No response
Tamagui Version
latest

Reproduction
Running the setup with pnpm

System Info
No response

Thanks, will go out in next release 320757d9c338bd5b745346ba21124ce1379db427


Date checking problem on IE (11) with output from toLocaleDateString()

Consider the following test page with two fields populated by Javascript:

Source code follows:
<!DOCTYPE html>

<form id="ValForm" method="get" action="">
<input data-val="true" data-val-date="The field must be a date" id="DateField" name="DateField" type="date" value="" />

<input data-val="true" data-val-date="The field must be a date" id="DateFieldLocale" name="DateFieldLocale" type="date" value="" />

<input type="submit" value="Validate" />
</form>

On Firefox, when the Validate button is pressed, all is well. On IE (at least v11) in the en-US locale, the second field fails validation and the "Please enter a valid date" message is displayed.

After much testing and teeth-gnashing, I figured out it is because the Date method toLocaleDateString is returning a different encoding. Basically I copied and pasted both fields into Notepad++ set to ANSI encoding, and they look like this:

I am willing to believe this is due to IE doing a bad implementation of toLocaleDateString, but still think jQuery validation should handle it gracefully.

I'm sorry for the lack of activity on this issue. Instead of leaving it open any longer, I decided to close old issues without trying to address them, to longer give the false impression that it will get addressed eventually, especially after several years with no activity. It doesn't mean I'm abandoning the project, just that I'm unable to work through 200+ open issues with the little time I can afford to spend on this project.
To the reporter (or anyone else interested in this issue): If you're still affected by the same issue, please consider opening a new issue, with a testpage that demonstrates the issue with a current version of the plugin. Even better, make an attempt to fix the issue yourself, and improve the project by sending a pull request. This may seem daunting at first, but you'll likely learn some useful skills that you can apply elsewhere as well. And you can help keep this project alive. We've documented how to do these things, too. A patch is worth a thousand issues!


Regarding the issue of Tmap generation

I have generated "mstmap.npy" and "bvp.npy" for each sample according to the CVD method. However, I encountered some issues when using create_tmaps_VIPL.py to generate the Tmap. (Additionally, the README mentions a shell script file “create_tmaps_VIPL.sh,” but there is only the create_tmaps_VIPL.py file in the project.)

I’m quite confused about the meaning of temp and mstmap. It seems that filename refers to "mstmap.npy", but mstmap = np.load(filename.replace("temp","mstmap"))[:,:,0:3] makes things even more confusing.
I would like to know if temp and mstmap are both referring to "mstmap.npy".
Thank you!

Additionally, there is a minor issue when generating mstmap in CVD-Physiological-Measurement. CVD sliced the mstmap, then applied smoothing and normalization to the segments. The corresponding MATLAB code is as follows:
for idx = 1:size(final_signal,1)
    for c = 1:channel_num
        temp = final_signal(idx,:,c);
        temp = movmean(temp, 3);
        final_signal1(idx,:,c) = (temp - min(temp)) / (max(temp) - min(temp)) * 255;
    end
end

I noticed that the filt_map function in create_tmaps_VIPL.py implements a similar functionality. So, during the generation of mstmap, is there only one normalization step that occurs in the filt_map function?

Hi, for your first comment, "temp" and "mstmap" are the same. "temp" was just just some legacy code left over from experimenting with different maps in the beginning. So they are the same. For your second comment, compared to CVD matlab code instead of moving average we bandpass filter to smooth the signals, and the normalization step is the exact same. During the generation they are normalised and saved, and also in the dataloader when smaller windows are loaded.

Thank you for your clear and accurate explanation.


Convert for Godot4

SHOULD NOT BE MERGED (yet)
I've made a port of this for myself while I was working on dialogic.
Maybe this is useful to someone, then they can use my fork for now.
And maybe this is usefull for when godot 4 becomes stable or whenever you want to update this.

I'm finally back from a 3-year hiatus. #31 has resolved the Godot 4 update, so I'm closing this. Thank you for your effort nonetheless. I truly appreciate it.