Bio Weekly - Correct Table 22a/22b Footnote

Please remove the second "Note:" in Table 22a/22b footnote.

Requestor(s): Domonique
Table or Figure title:

Table 22a. Time (in Days) from Verification to Baseline Clinical Collection for Participants Verified Before 02/01/2023
Table 22b. Time (in Days) from Verification to Baseline Clinical Collection for Participants Verified on or After 02/01/2023

Metrics to be updated or created: N/A
Variables needed (include variable name and Concept ID): N/A
Specify numerator and denominator by variable name where applicable: N/A
Mock-up table if applicable: N/A
Notes: N/A
Approved by:

@HopkinsDC

@KELSEYDOWLING7 Thank you!

Update successfully implemented in 11/20 weekly biospecimen metrics report.


fix: Maximum Temperature for OpenAI endpoint set to 1 instead of 2 (which would result in gibberish output above a temperature 1)

Summary
fix: I changed the maximum Temperature for OpenAI endpoint to 1 instead of 2 , because anything above 1 would result in gibberish output. See the OpenAI playground for example. There OpenAI sets the maximum temperature also to 1.
Unexperienced users can set it to too high values without this fix resulting in unnecessary problems.
(Second idea would be to decrease steps from 0.01 to 0.1, since 0.01 steps is way to granular)
Change Type

[x] Bug fix (non-breaking change which fixes an issue)

Testing
I build the docker container new and tested if the changes work well in front-end and correctly work with OpenAI endpoint by testing some messages. Its better now this way.
Test Configuration:
Checklist

[x] My code adheres to this project's style guidelines
[x] I have performed a self-review of my own code
[x] I have commented in any complex areas of my code
[x] I have made pertinent documentation changes
[x] My changes do not introduce new warnings
[x] I have written tests demonstrating that my changes are effective or that my feature works
[x] Local unit tests pass with my changes
[x] Any changes dependent on mine have been merged and published in downstream modules.

It's better to implement it as a toggleable default option, hidden in the settings menu.

Thanks for the PR, but I'm not sure this is necessary. The API allows up to 2 and I would not like to intentionally limit it: https://platform.openai.com/docs/api-reference/chat/create#chat-create-temperature

yeah okay, maybe there could be beginner and advanced settings in future in the ui

now it's 1😢😢


Dynamic routes

Is there a way to change routes object after the store is created?

Not yet, although it's coming up soon. I'm hoping to release #189 soon, which allows Fragments to ignore static routes entirely, making them code-splittable 😄

@SCLeoX what's the use case for changing route objects after the store is created?

@faceyspacey The fragments are not all loaded at beginning. At beginning, we do not want to send over information about our whole site as some of them should be only accessible by an admin.

And this isn't solved by simply making the content empty on the URLs without proper with? This is just about preventing hackers from seeing URLs correct?

Yes. It prevents hackers from understanding our admin panel's structure.

Implemented a replaceRoutes action creator in v14.2.0! You'll want to look at the advanced docs for usage.
If you're just using Fragment and not relying on the static routes for their result key, you're already set, since we decoupled Fragment from the static routes in v14.


build JSON file, use timestamp in seconds since 1970 instead of full date format

In the [build_number].json it seems like currently we store "2018-01-30 16:24:23 -0500" for time stamps, however I think we should use<PHONE_NUMBER> instead 👍

Are PR's on some issues welcomed here?

 Check out our contributing doc: https://github.com/fastlane/ci/blob/master/CONTRIBUTING.md#before-you-start

Yep, un-assigned myself, feel free to grab this one 👍

I've been digging into this issue, and I can't find where you're stablishing that format when converting to json the Time object of a build, referencing the docs:

to_json(*args) public
Stores class name (Time) with number of seconds since epoch and number of microseconds for Time as JSON string

I haven't looked into the code base yet for this one, but from what I remember, in JSONConvertible we iterate over all available attributes, and just directly access the instance_variable_value. What that means is that we basically just convert the Time object to a String. We don't have support for conversions of formats on saving/loading

So I'm thinking of adding an argument to the convertible, a hash in a key => proc form in order to customize the output of the different conversions. Should I open a PR covering this?😊 (You can assign it to me)

So I'm not too familiar with JSONConvertible - I think a proc might work, but maybe we could do something alone the lines of defining a mapping for each attribute? So that a class can define, this attr_accessor is of type Time, but here's a method to convert it to/from it

But I'm that case the json_convertible would know about the methods in the model itself and then they're coupled? Or you're talking about overriding the setters and getters in the class with the custom if seconds from epoch convert to Time, else set it normally, and the getter is always the seconds from epoch? Or just I'm missing something about your point. 😅

Ahh, good point, I got it wrong, ignore my last comment. Maybe let's wait for @taquitos to chime in on this one 👍

Yeah, I was thinking that we could add some extra methods to JSONConvertible:
self.attribute_key_name_map : A map of Ruby attribute names to json key names for times when a key in your model should be named differently in the resultant JSON.
And, more importantly, I think @minuscorp and I are on the same page with this:
self.attribute_name_to_json_proc_map : a map of ruby attribute names to proc, so that when you are going to transcode the value into json, it grabs a proc that can do custom processing.
So, each of these methods would have a basic implementation on JSONConvertible but then, when you need to use them you'd update the corresponding mixin, using the adaptor pattern so that we don't need to change the underlying model class.
For example, in json_build_data_source.rb you can update the mixin to implement these methods (actually, pulling out the mixin to it's own file would be ideal at this point, since it's sufficiently large now that we're adding more to it)
So I was thinking something like:
class Build # this is the mixin as seen in `json_build_data_source.rb`
  include FastlaneCI::JSONConvertible

  # class method
  def self.attribute_name_to_json_proc_map
    timestamp_to_json_proc = proc { |timestamp|
      return timestamp.strftime('%Q')
    }
    return { "timestamp" => timestamp_to_json_proc }
  end
end

then in JSONConvertible, you'd need to update to_object_dictionary(ignore_instance_variables: []) to look into this mapping and then call the proc with the proper attribute value.
If that sounds like what you were thinking 🎈:goat:

I'll take a look to this tomorrow and Monday and try to setup a PR for this upcoming week 👨🏻💻

So if def self.attribute_name_to_json_proc_map is needed, then def self.attribute_json_to_name_proc_map is mandatory, didn't it? Without that method, the JSONConvertible wouldn't know how to map back to the ruby value from the json.

#118 🚀


[Feature] Add basic PR checks as Github actions

Lint is run as part of the current acceptance test task, however this isn't run automatically for community sourced PR's which unnecessarily lengthens the feedback cycle for those PR's. Basic CI checks should be automatically triggered for all PRs via GH actions, they should include:

Lint + Compilation
Unit tests
That there are no uncommitted docs changes

I would love to have CI on github actions to let community contributors to work easily on the project.
Could we also move acceptance tests to Github actions?


Fix Append Files

We should call MessageType_KtoU_NotifyFilePreConvertToFull on KAUTH_VNODE_APPEND_DATA
Revert Previous method of relying on FILEOP operations

Other than some suggestions for the unit tests this is looking good!

/azp run GitHub VFSForGit Mac Functional Tests


 and testing framework updates

I moved my pull request from a pass-through on myrho's fork to my own direct fork. This fixes four things:
<foo/> now parses correctly. Before it needed attributes or some space before the closing />.
Updated to the latest elm-test, which no longer requires Main.elm or the rtfeldman/node-test-runner package.
Updated to elm-verify-examples and its new doc test syntax.
Added sysconfcpus to .travis.xml for epic Elm compile time improvement, minutes become seconds.
Trimmed the giant 1000-element S3 directory string to 100 elements, to prevent timeouts on Travis.

Published as 2.2.3

Yay! How do you like the 2 minute and 6 second Linux testing?
I don't know what's up with Travis macOS testing. It's been very slow to start of late.

I'm well aware of the libsysconfig trick and have looked into the causes :)

Not surprising. I'm a lot newer at Elm than you, so it was a revelation to me a few days ago.

Glad to hear it. Have I said recently that Elm rocks! Well, Elm rocks! In spades!
The crypto code that I've written recently is the most beautiful code I've ever written in my 40 years of writing software for a living. And Elm is largely responsible for that.


Support python3

I fixed sources to support python3.

Fix print
Fix import syntax
Use six for iterxxx, CStringIO, copyreg, xrange, queue, assertCountEqual ...
Use // operator instead of / operator for int
Fix heapq
Use __truediv__

fixes #22

LGTM!


rest of the wallet balance is gone after sending a portion of btc

I need help admins.
Sent a portion of my btc now the rest of my balance isnt showing up

Hello @jan8888
Your issue is related to a temporary synchronization delay we were experiencing and should be fine now.
Kind regards.

@jan8888
did you get back your missing balance
if not contact me I can restore it for you<EMAIL_ADDRESS>


Update go to 1.22 and errkit to latest tag

Update go to 1.22.7
Update errkit to latest tag v0.0.3

These two changes will fix some vulnerabilities seen on Snyk scanning.

LGTM!


alacritty 0.3.2:  URL parser drops trailing /

Which operating system does the issue occur on?  Linux
If on linux, are you using X11 or Wayland?  xorg-server 1.20.4
If a URL ends with a trailing /, alacritty's URL parser in 0.3.2 is (sometimes?  always?  LOOKS like always, in all of my test cases so far) dropping the trailing slash, which in most cases causes the link to fail.
I'm fairly sure this is a regression from 0.2.x.

which in most cases causes the link to fail

That should not be the case. The trailing slash is dropped intentionally, but it should not cause the link to fail.
Trailing slashes should usually be optional, could you share an example on when this will break a link?

https://www.leaf.tv/articles/difference-between-virginia-black-forest-ham/
Without the trailing /, this link does not resolve.

(I should probably have said "in most of my test cases".)

In my experience this is more than extremely unusual. Since usually pages end in .html and the .html is truncated, leaving just the name.
With a trailing slash at the end, that would make a rewrite from site.html to site/ which doesn't make much sense to me.
But trying to keep context whenever possible makes sense, the reason trailing slashes are stripped is primarily to be able to strip other content.
So the text Testing (my website is alacritty.io/testing)/ should currently resolve properly to the alacritty.io/testing URL, but if we do not allow to truncate / at the end it will resolve to alacritty.io/testing)/.
So I think there is some benefit in truncating slashes from the end, but the question is probably how common this scenario is. I think removing the truncation just for / should be okay, but either way will likely introduce some issues. That's just the nature of the problem with having to parse URLs inside plain text.

Hmm.  Yeah, not a simple problem, and I've never yet seen a text parser that someone couldn't construct a crafted example to break.
I suspect the problem mainly arises when URLs point to dynamically generated content.
Normally, http://foo......bar/ is parsed as http://foo......bar/index.html or whatever the designated default index is.  I've been doing further testing on a wider variety of URLs and found that most of the time it does work, but I found a significant number of cases where it didn't.
I thought that () were illegal characters in URLs which should cause your above example to be correctly parsed anyway, but looking it up, I see that () are not only allowed in URLs but considered "safe".
Could stripping trailing / perhaps be made an option?

In my experience this is more than extremely unusual. Since usually pages end in .html and the .html is truncated, leaving just the name.

In certain web frameworks it is indeed a difference between URLs with/without trailing slash, e.g.: https://stackoverflow.com/questions/1596552/django-urls-without-a-trailing-slash-do-not-redirect. I do not recommend stripping it.

I thought that () were illegal characters in URLs which should cause your above example to be correctly parsed anyway, but looking it up, I see that () are not only allowed in URLs but considered "safe".

Parenthesis are perfectly legal in URLs and wikipedia frequently uses them:
https://en.wikipedia.org/wiki/Rust_(programming_language)
Just as a general opinion, I don't think a slash at the end of a URL makes sense at all as a resource identifier (it's a resource, not a directory!), but we don't have any other choice here as to follow what other people think is best, since we just parse the URLs.
So I'm up to change this behavior and stop dropping trailing /.

Yeah, if a server could possibly care if a trailing slash exists, we shouldn't mess with it, I think

This has been fixed by #2449 but didn't close automatically (no "Fix" in PR message)

Thanks for letting me know!


fix: Grammatical error

Fix Grammatical error in README

LGTM
Thanks 👍 for the PR

Any tips on how to get started with this project ?

You can refer the Readme,
This is like a hack project,so not completely tested but still core functionality works well.
Start with creating a database and populate the creds in the script and clone the repo and you can experiment according to the your use cases mentioned in readme.
Any further questions we can have a quick chat on this.


Renaming column names

when I define column as tablename_somecolumnname in java class, sugar ORM internally creates column  as tablenamesomecolumnname, because of this I am facing so much issues, and there is no proper documentation of why it does like this and what are all conditions where it changes column name. So I am shifting to normal android database implementation of creating DB.

It's not entirely a major issue. Camel case SugarRecords are also renamed to ex:
PushMmsMessage ---> PUSH_MMS_MESSAGE
I has trouble updating my databases to add columns, had no idea what the database name was :-P Had to dump the Sqlite databases from an emulator and view it on my comp.

Read the last lines from http://satyan.github.io/sugar/query.html :)

@shreenivas-ch @nseidm1 you guys can use NamingHelper.toSQLName(Model.class) to get the name of the TABLE
you can also use @Column(name="columnName") to define a custom column name for a property

how to use a custom column name for the existing database ??


Badges for WalletConnect and notifications

This PR adds badges to the WalletConnect and notifications tab.

I don't think these are necessary and don't really see how they benefit the user in these cases.

We could use badges as a CTA for Pillar owners to indicate x number of projects/phases are waiting for votes. The badge would be added to the AZ tab.

Some suggestions have been made by community members to use badges for the Transfer tab to indicate that the selected address has pending transactions.
This feature would only apply when the auto-receive function is disabled.


Add Some Parallelism to AGiXT

Feature/Improvement Description
I wonder would it be possible to basically set a looping chain step until it gets the necessary required information to complete the step and the prompt can set a variable when it thinks the chain should be finished so an example research until you figure out this aspect of the task is the chain and then it starts the prompt after it has gotten information, then you execute a prompt and the prompt analyzes the info it grabbed and checks it to see if it makes sense and then say it can set a variable false to stop looping back to the chain if it has enough information to proceed with the next step of the chain.
kind of like a chain to chain operation but instead within itself but using different prompts and chains to do that
Proposed Solution
I asked Sonnet to expand on it
Yes, it is possible to create a looping process where an initial prompt or chain gathers information and research on a topic, and then it passes that information to another prompt or chain for analysis and evaluation. This second prompt or chain can then determine if the gathered information is sufficient to proceed to the next step, or if it needs to loop back and gather more information from the initial prompt or chain.
This can be achieved using a technique called "re-ranking" or "recursive re-ranking," where the output of one prompt or chain is fed back into the initial prompt or chain as additional context or instruction. The initial prompt or chain can then use this additional context to refine its search and gather more relevant information.
To implement this, you would typically use a combination of prompts and chains, with a "controller" prompt or chain orchestrating the flow between the different components. Here's a high-level outline of how this might work:
Initial Prompt or Chain: This prompt or chain is responsible for gathering information and research on the given topic. It can be a simple search query, a more complex research prompt, or a chain of prompts and tools designed for information gathering.
Analysis Prompt or Chain: This prompt or chain receives the output from the initial prompt or chain and analyzes the information to determine if it is sufficient to proceed to the next step or if more information is needed.
Controller Prompt or Chain: This prompt or chain manages the flow between the initial prompt or chain and the analysis prompt or chain. It initializes the loop, passes the output from the initial prompt or chain to the analysis prompt or chain, and based on the analysis output, determines whether to loop back to the initial prompt or chain for more information or to proceed to the next step.
Within this structure, you can set variables or flags to control the flow and determine when the loop should terminate. For example, the analysis prompt or chain could set a variable (e.g., is_information_sufficient = True/False) that the controller prompt or chain can use to decide whether to loop back or proceed.
This approach allows for a flexible and adaptive information gathering and analysis process, where the system can iteratively refine its understanding and gather more information as needed before moving forward.
It's important to note that implementing this type of recursive re-ranking process can be complex and may require significant engineering effort, particularly when dealing with large amounts of data or complex tasks. However, it can be a powerful technique for building intelligent systems that can adaptively gather and analyze information.
Acknowledgements

[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for everyone to understand why this feature request is needed in AGiXT.

Of course the feature and description also Solution is just scratching the surface that's not all to the ideas I have for parallelism in AGiXT.

Example of self-solving chain step parallelism, use how you please:
import random

class Step:
    def __init__(self, id, name, dependencies=[]):
        self.id = id
        self.name = name
        self.dependencies = dependencies

    async def execute(self):
        # Only await tasks that have not yet finished
        results = await asyncio.gather(
            *(dep for dep in self.dependencies if not dep.done())
        )
        print(
            f"Executing step {self.id}: {self.name}. Dependencies resolved as {results}."
        )
        delay = random.randint(3, 10)
        await asyncio.sleep(delay)
        self.resolved = True
        toReturn = random.randint(1, 100)
        print(f"Completed step {self.id}: {self.name} - returned {toReturn}.")
        return toReturn

async def resolve():
    step1 = asyncio.create_task(Step(1, "Step 1").execute())
    step2 = asyncio.create_task(Step(2, "Step 2", [step1]).execute())
    step3 = asyncio.create_task(Step(3, "Step 3", [step1]).execute())
    step4 = asyncio.create_task(Step(4, "Step 4", [step2, step3]).execute())
    step5 = asyncio.create_task(Step(5, "Step 5", [step1, step2]).execute())
    step6 = asyncio.create_task(Step(6, "Step 6", [step4, step5]).execute())
    step7 = asyncio.create_task(Step(7, "Step 7", [step6]).execute())
    step8 = asyncio.create_task(Step(8, "Step 8", [step1]).execute())
    step9 = await asyncio.create_task(Step(9, "Step 9", [step7, step8]).execute())

asyncio.run(resolve())

Completed step 1: Step 1 - returned 22.
Executing step 2: Step 2. Dependencies resolved as [22].
Executing step 3: Step 3. Dependencies resolved as [22].
Executing step 8: Step 8. Dependencies resolved as [22].
Completed step 2: Step 2 - returned 60.
Executing step 5: Step 5. Dependencies resolved as [22, 60].
Completed step 8: Step 8 - returned 87.
Completed step 3: Step 3 - returned 92.
Executing step 4: Step 4. Dependencies resolved as [60, 92].
Completed step 5: Step 5 - returned 83.
Completed step 4: Step 4 - returned 32.
Executing step 6: Step 6. Dependencies resolved as [32, 83].
Completed step 6: Step 6 - returned 88.
Executing step 7: Step 7. Dependencies resolved as [88].
Completed step 7: Step 7 - returned 31.
Executing step 9: Step 9. Dependencies resolved as [31, 87].
Completed step 9: Step 9 - returned 43.```

Example of self-solving chain step parallelism, use how you please:
import random

class Step:
    def __init__(self, id, name, dependencies=[]):
        self.id = id
        self.name = name
        self.dependencies = dependencies

    async def execute(self):
        # Only await tasks that have not yet finished
        results = await asyncio.gather(
            *(dep for dep in self.dependencies if not dep.done())
        )
        print(
            f"Executing step {self.id}: {self.name}. Dependencies resolved as {results}."
        )
        delay = random.randint(3, 10)
        await asyncio.sleep(delay)
        self.resolved = True
        toReturn = random.randint(1, 100)
        print(f"Completed step {self.id}: {self.name} - returned {toReturn}.")
        return toReturn

async def resolve():
    step1 = asyncio.create_task(Step(1, "Step 1").execute())
    step2 = asyncio.create_task(Step(2, "Step 2", [step1]).execute())
    step3 = asyncio.create_task(Step(3, "Step 3", [step1]).execute())
    step4 = asyncio.create_task(Step(4, "Step 4", [step2, step3]).execute())
    step5 = asyncio.create_task(Step(5, "Step 5", [step1, step2]).execute())
    step6 = asyncio.create_task(Step(6, "Step 6", [step4, step5]).execute())
    step7 = asyncio.create_task(Step(7, "Step 7", [step6]).execute())
    step8 = asyncio.create_task(Step(8, "Step 8", [step1]).execute())
    step9 = await asyncio.create_task(Step(9, "Step 9", [step7, step8]).execute())

asyncio.run(resolve())

Completed step 1: Step 1 - returned 22.
Executing step 2: Step 2. Dependencies resolved as [22].
Executing step 3: Step 3. Dependencies resolved as [22].
Executing step 8: Step 8. Dependencies resolved as [22].
Completed step 2: Step 2 - returned 60.
Executing step 5: Step 5. Dependencies resolved as [22, 60].
Completed step 8: Step 8 - returned 87.
Completed step 3: Step 3 - returned 92.
Executing step 4: Step 4. Dependencies resolved as [60, 92].
Completed step 5: Step 5 - returned 83.
Completed step 4: Step 4 - returned 32.
Executing step 6: Step 6. Dependencies resolved as [32, 83].
Completed step 6: Step 6 - returned 88.
Executing step 7: Step 7. Dependencies resolved as [88].
Completed step 7: Step 7 - returned 31.
Executing step 9: Step 9. Dependencies resolved as [31, 87].
Completed step 9: Step 9 - returned 43.```

Added in PR #1201


getLog("browser") returns an empty array

What are you trying to achieve? (Expected behavior)

When calling getLog("browser") I expect the webdriver to return an array containing all of the browser logs.
What do you get instead? (Actual behavior)

The browser returns an empty array. I am able to call getLog("server") and get a nonempty array, but getLog("browser") is always empty no matter how many entries are in the console log
How could the issue be reproduced? (Steps to reproduce)

I am using the code below:
$logs = $this->webDriver->manage()->getLog("browser");
var_dump($logs);

Details

Php-webdriver version: 1.6.0
PHP version: PHP 7.2.8
Selenium server version: Selenium 3.13.0
Operating system: MacOS High Sierra 10.13.4
Browser used + version: Google chrome Version 67.0.3396.99

please try it:
$caps->setCapability( 'loggingPrefs', ['browser' => 'ALL']);
var_dump($driver->manage()->getLog('driver'));

Is it possible to listen to this logs ?

This does not seems like an issue of php-webdriver.


GEN-20: Revise User Interaction and Guest Users

What's up?
As mentioned in #162, revise the User Intereaction and Guest Users subtitles under the User Requirements and come up with improvements.
To Do

[x] Revise the User Interaction related User Requirements.
[x] Revise the Guest Users related User Requirements.
[x] Share your ideas with other team members in the next meeting on 16.10.22 @18.00.

Deadline
16.10.2022 @18.00
Additional Information
No response
Reviewers
No response

Reviewed 1.1.2 Guest Users, no actions are taken. All requirements are clearly explained and seem necessary to me.

Reviwed 1.1.4 User Interaction. Following points are noted for discussion at the next meeting:

Comment editing feature: (<IP_ADDRESS>. Registered users shall be able to edit their comments. Same functionality applies for replies.)
Chatting feature: (<IP_ADDRESS>. Registered users shall be able to chat with other users. They shall be able to send and receive private messages.) We may want to elaborate on the chatting feature.

Other than these, I think all the requirements related to 1.1.4. User Interaction are necessary and sufficient.

I ve also reviewed the requirements, @kostanya you have pointed goof points. I aggree all of you mentioned except the chatting feature. It should stay, I think.
Plus,

Comment replying feature: (<IP_ADDRESS>. Registered users shall be able to reply to a comment. Reply is also considered as a comment.)

we can discuss it in our weekly meeting and take the necessary action

Removed the private visibility feature mentioned in the definition of the follower.
Removed <IP_ADDRESS>.2. (t was rielated to setting the art item visibility while uploading an image)

Thanks for your effort.


cross posting, tunneling TURN thru other protocols, in the browser

linking to here in the hope there might be some interest
https://github.com/w3c/webtransport/issues/270

This doesn't have much to do with quiche, closing.

@LPardue Correct, but it could lead to revenue opportunities for cloudflare.  webrtc traffic is a big deal these days.  Perhaps a product owner would be interested?

Github isn't the appropriate venue to discuss this. Our team participate in standards bodies such as the IETF and W3C, which is the best place to develop new ideas that require technological changes across industry. We've seen your threads in those venues.
There's lots of options for carrying TURN traffic over a QUIC substrate - native application mapping, over HTTP/3, over MASQUE, over WebTransport. Quiche is a low layer library for the QUIC protocol, it will likely support any type of substrate that people wish to use. But unless there is a concrete issues with, or suggestion for, the library, then creating issues here fragments discussion.
My advice would be to try and sketch out what substrate you'd use for TURN and to test out the concept. If you find implementation issues using quiche to build the concept, that is entirely appropriate to bring to this GitHub repo.


Add invariant example

https://github.com/zertosh/invariant

(:+1:)

The problem I have with that library is that there is no way to separate development assertions from production assertions.

What is production validation? You mean object validations? For that I am using this simple https://gist.github.com/steida/d54a2cd2296ec7c164d1

Asserts has different purpose than validations.

No, I mean that there are assertion you only want in development "mode" due to the performance implications.

They are removed. For example check compiling React with and without webpack.DefinePlugin for process.env.NODE_ENV. Or your code. Dead code removal works.

My point is, that some invariants you want to remove and some you want to stay.

I think this is over the scope of invariants. They are defined as dev time asserts and being used in production is misusement. Your require(isAdmin)? What's that? I woulduse if statement.

If statement it is. Close this.

invariant (asserts) is recommended pattern, just have to added an example into Este.

Let's just write an invariant for direct accessing the pending property of an action?
Although accessing pending property of <actionName> in <componentName> works, it may cause hard to debug issues. It is advisable to keep your components pure and use pendingActions instead

Pending prop is gone :)
I think we don't need invariant example in Este since it's app. Invariant checks belong to libraries.

since it's app

Yet...


Error with stock Select component

Describe the bug
I get this error when just using the basic example on RN 0.62
To Reproduce
It appears fine on the screen at first. The error only appears once the component is clicked/selected.
Relevant code (copied from the docs)
<View style={[styles.content, { backgroundColor: theme.secondary, borderColor: theme.accent }]}>
        <Select
          label={'Select'}
          menuItems={[
            { id: 1, name: 'one' },
            { id: 2, name: 'two' }
          ]}
          onSelect={value => setSelectedItem({ selectedItem: value.name })}
          selectedItem={selectedItem}
          textFieldProps={{
            leadingIcon: (
              <Icon name={'date-range'} size={24} color={'#6e6e6e'} />
            ),
          }}
        />
        <Select
          label={'Select'}
          type={'filled'}
          menuItems={[
            { id: 1, name: 'one' },
            { id: 2, name: 'two' }
          ]}
          onSelect={value => setSelectedItem({ selectedItemTwo: value.name })}
          selectedItem={selectedItemTwo}
          textFieldProps={{
            leadingIcon: (
              <Icon name={'date-range'} size={24} color={'#6e6e6e'} />
            ),
          }}
        />
        <Select
          label={'Select'}
          type={'outlined'}
          menuItems={[
            { id: 1, name: 'one' },
            { id: 2, name: 'two' }
          ]}
          onSelect={value => setSelectedItem({ selectedItemThree: value.name })}
          selectedItem={selectedItemThree}
          textFieldProps={{
            leadingIcon: (
              <Icon name={'date-range'} size={24} color={'#6e6e6e'} />
            ),
          }}
        />
      </View>

Error: Style property 'fontSize' is not supported by native animated module

This error is located at:
    in TextFieldLabel (at withTheme.js:11)
    in _class (at TextFieldFlat.js:174)
    in RCTView (at TextFieldFlat.js:167)
    in TextFieldFlat (at withTheme.js:11)
    in _class (at TextField.js:64)
    in TextField (at withTheme.js:11)
    in _class (at Select.js:112)
    in RCTView (at Select.js:111)
    in RCTView (at TouchableHighlight.js:297)
    in TouchableHighlight (at TouchableHighlight.js:381)
    in ForwardRef (at Select.js:103)
    in RCTView (at Menu.js:223)
    in RCTView (at Menu.js:222)
    in Menu (at withTheme.js:11)
    in _class (at Select.js:90)
    in Select (at withTheme.js:11)
    in _class (at SettingsScreen.js:32)
    in RCTView (at SettingsScreen.js:31)
    in RNCSafeAreaView (at SettingsScreen.js:20)
    in SettingsScreen (at SceneView.tsx:122)
    in StaticContainer
    in StaticContainer (at SceneView.tsx:115)
    in EnsureSingleNavigator (at SceneView.tsx:114)
    in SceneView (at useDescriptors.tsx:150)
    in RCTView (at CardContainer.tsx:219)
    in RCTView (at CardContainer.tsx:218)
    in RCTView (at CardSheet.tsx:33)
    in ForwardRef(CardSheet) (at Card.tsx:562)
    in RCTView (at createAnimatedComponent.js:144)
    in AnimatedComponent (at createAnimatedComponent.js:194)
    in ForwardRef(AnimatedComponentWrapper) (at Card.tsx:544)
    in PanGestureHandler (at GestureHandler.native.tsx:13)
    in PanGestureHandler (at Card.tsx:538)
    in RCTView (at createAnimatedComponent.js:144)
    in AnimatedComponent (at createAnimatedComponent.js:194)
    in ForwardRef(AnimatedComponentWrapper) (at Card.tsx:534)
    in RCTView (at Card.tsx:528)
    in Card (at CardContainer.tsx:187)
    in CardContainer (at CardStack.tsx:558)
    in RCTView (at Screens.tsx:70)
    in MaybeScreen (at CardStack.tsx:551)
    in RCTView (at Screens.tsx:48)
    in MaybeScreenContainer (at CardStack.tsx:461)
    in CardStack (at StackView.tsx:458)
    in KeyboardManager (at StackView.tsx:456)
    in SafeAreaProviderCompat (at StackView.tsx:453)
    in RCTView (at StackView.tsx:452)
    in StackView (at createStackNavigator.tsx:82)
    in StackNavigator (at App.js:35)
    in EnsureSingleNavigator (at BaseNavigationContainer.tsx:390)
    in ForwardRef(BaseNavigationContainer) (at NavigationContainer.tsx:91)
    in ThemeProvider (at NavigationContainer.tsx:90)
    in ForwardRef(NavigationContainer) (at App.js:34)
    in Root (at App.js:58)
    in Provider (at App.js:57)
    in Provider (at App.js:56)
    in Provider (at App.js:55)
    in RNCSafeAreaProvider (at SafeAreaContext.tsx:74)
    in SafeAreaProvider (at App.js:54)
    in App (at renderApplication.js:45)
    in RCTView (at AppContainer.js:109)
    in RCTView (at AppContainer.js:135)
    in AppContainer (at renderApplication.js:39)

validateStyles
    NativeAnimatedHelper.js:248:12
__getNativeConfig
    AnimatedStyle.js:119:4
__getNativeTag
    AnimatedNode.js:167:8
__getNativeConfig
    AnimatedProps.js:166:31
__getNativeTag
    AnimatedNode.js:167:8
__connectAnimatedView
    AnimatedProps.js:130:6
__makeNative
    AnimatedProps.js:105:8
__makeNative
    AnimatedWithChildren.js:28:8
__makeNative
    AnimatedStyle.js:105:4
__makeNative
    AnimatedWithChildren.js:28:8
__makeNative
    AnimatedInterpolation.js:330:4
__makeNative
    AnimatedWithChildren.js:28:8
__makeNative
    AnimatedInterpolation.js:329:4
__makeNative
    AnimatedStyle.js:102:8
__makeNative
    AnimatedWithChildren.js:28:8
__makeNative
    AnimatedTransform.js:34:4
__makeNative
    AnimatedWithChildren.js:28:8
__startNativeAnimation
    Animation.js:62:4
start
    TimingAnimation.js:115:10
start
    TimingAnimation.js:126:11
animate
    AnimatedValue.js:206:4
start
    AnimatedImplementation.js:210:6
start
    AnimatedImplementation.js:217:13
animations.forEach$argument_0
    AnimatedImplementation.js:383:10
forEach
    [native code]:0
result.start
    AnimatedImplementation.js:365:6
_handleLabelAnimation
    TextFieldLabel.js:1:4024
componentDidUpdate
    TextFieldLabel.js:1:2938
commitLifeCycles
    ReactNativeRenderer-dev.js:18169:10
commitLayoutEffects
    ReactNativeRenderer-dev.js:21528:22
invokeGuardedCallbackImpl
    ReactNativeRenderer-dev.js:286:4
invokeGuardedCallback
    ReactNativeRenderer-dev.js:497:2
commitRootImpl
    ReactNativeRenderer-dev.js:21267:29
commitRootImpl
    [native code]:0
unstable_runWithPriority
    scheduler.development.js:818:23
commitRoot
    ReactNativeRenderer-dev.js:21109:17
finishSyncRender
    ReactNativeRenderer-dev.js:20514:12
performSyncWorkOnRoot
    ReactNativeRenderer-dev.js:20493:24
performSyncWorkOnRoot
    [native code]:0
runWithPriority$argument_1
    ReactNativeRenderer-dev.js:5703:31
unstable_runWithPriority
    scheduler.development.js:818:23
flushSyncCallbackQueueImpl
    ReactNativeRenderer-dev.js:5698:21
flushSyncCallbackQueue
    ReactNativeRenderer-dev.js:5686:28
batchedUpdates$1
    ReactNativeRenderer-dev.js:20575:28
batchedUpdates
    ReactNativeRenderer-dev.js:2731:29
_receiveRootNodeIDEvent
    ReactNativeRenderer-dev.js:2834:16
receiveTouches
    ReactNativeRenderer-dev.js:2911:27
__callFunction
    MessageQueue.js:425:19
__guard$argument_0
    MessageQueue.js:112:6
__guard
    MessageQueue.js:373:10
callFunctionReturnFlushedQueue
    MessageQueue.js:111:4
callFunctionReturnFlushedQueue
    [native code]:0

Screenshots

And since this is probably related, I'm getting the same thing with the Menu component, except the error is

@JakeHadley What version are you using?

^0.2.10

@codypearce and forgot to mention, this is happening on both android and ios. I was thinking it had to do with the useNativeDriver thing again or whatever. I'm not terribly familiar with the animation side of RN, trying to get more acquainted with it.

Fixed in #423

@codypearce I've installed off master and still getting this error.
Is there a good way I can try and experiment and fix this stuff and submit a PR? I don't really know how to do work with these types of modules, what process you go through to actually run and test these. I'd love to help out instead of bugging you with any and all problems I encounter.

@JakeHadley The issue probably still exists on master because the package points to the dist folder, you can see this in the  package.json's main property https://github.com/codypearce/material-bread/blob/master/package.json#L79. I pushed a new commit that includes the changes for the dist folder now if you want to try it.

Is there a good way I can try and experiment and fix this stuff and submit a PR? I don't really know how to do work with these types of modules, what process you go through to actually run and test these.

I wrote a guide on how to get started contributing if you're interested https://material-bread.org/contributing/library
Essentially you need to clone the project, run npm i, then run one of the start scripts to run the storybook environment on the platform you want to test on
Ios: react-native run-ios
android: react-native run-android
web: npm run storybook-web
This will open up a simulator or a web server with all the component stories

All the component code can be found in src/Components/[Component]. For example,

Appbar.js contains the component itself
Appbar.stories.js contains the example components, variations that you see in the simulator
Appbar.styles.js contains all styles for the component
Appbar.test.js simple tests
index.js just exports the component


Update B+ Tree Layer.md

fixed error in the documentation in BPlusTree::bPlusCreate,it was creating a new leaf block since it was updating in the attribute catalog it can't insert index

Thanks for raising a PR! Just confirming, does this fix [this issue[(https://groups.google.com/g/nitcbase-users/c/TfjH_sWVHPw).


bloq.call_graph() should preserve ordering of successors across calls / build_call_graph should return a Dict instead of Set

Every time you call bloq.call_graph() you can get a different nx.DiGraph where the ordering of successors of a node is different. This happens because build_call_graph returns a Set instead of a Dict so iterating over a set to get the call graph destroys ordering.
I encountered this when writing tests for https://github.com/quantumlib/Qualtran/pull/732 where the output of get_flame_graph_data is different across calls because the DFS ordering of nodes in bloq.call_graph is different.
@mpharrigan What's the reason to destroy ordering? Is this by design? Can we preserve ordering? I agree that most properties of the call graph analysis shouldn't depend upon the ordering but I think it's still nice to preserve the order across calls.

Are sets not ordered in Python? either a list of 2-tuples or a dict would be fine. I think the point of the "set" was to encode that order doesn't matter. Perhaps a more important property is that each bloq appears only once, which isn't captured by set.
It's possible that networkx will shuffle things later anyways, so if you want determinism I suggest throwing in some sorteds.

Are sets not ordered in Python?

Nope. Dicts are, but sets aren't. I vote for a dict instead of a list of 2-tuples.

Perhaps a more important property is that each bloq appears only once, which isn't captured by set.

That's true, and a dict would capture that as well.

It's possible that networkx will shuffle things later anyways

I think it's less likely if we just traverse the graph and not modify it. But we can deal with any potential indeterminism due to networkx when we encounter it. AFAIK, this shouldn't be an issue.

@mpharrigan should the bloq author ensure that each unique bloq in the BloqCountT occurs only once? i.e. is something like this valid:
counts = {}
if condition_1: counts.add((XGate(), 1))
if condition_2: counts.add((XGate(), 2))
return counts

This luckily works as long as the frequencies are different, but fails if they are equal.

^^^ I've been caught out by this before

@anurudhp I think that's what this issue is about. If we change it to use dictionaries, you'd have to do something like
counts = defaultdict(lambda: 0)
if condition_1: counts[XGate()] += 1
if condition_2: counts[XGate()] += 2
return counts

My latest thoughts: I think this is still a good idea. IMO, the execution of this change should go through a mini deprecation cycle. Since build_call_graph is not supposed to be called by users directly, we can change its definition to return either a set or a dict; and change the internal uses of the function to turn sets into dictionaries; emit a deprecation warning if a set is encountered; and change all the existing implementations at our leisure to use dictionaries; eventually remove the type annotation for set; and even more eventually, remove the defensive coding within the consumers of build_call_graph.

This is done thanks to @dstrain115


[WIP] zfs: 0.7.12 -> 0.7.13

(cherry picked from commit b57080d34136fd8dcc49632696ceb5ed1ff73326)
Motivation for this change
cc @clefru regarding your question, personally I found rebasing a patch with
git easier then having a patch file in nixpkgs itself.
However we could also move the zfs fork to a more official place, what do you think?
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg test zfs

Mhm. now that I run the tests I get a build error.

Ah, spl also needs an upgrade.

duplicate https://github.com/NixOS/nixpkgs/pull/56882?

Yeah. Also I still prefer git because it is easier to update.

Tests run on my machine.


Transaction rejected as dust

Have a user who is having trouble making transactions most of the time because it either fails due to a dust warning or high fee if he messes around with bids.  The rules for selecting utxos must have a bug in some edge case. We had filed this on lbrycrd previously, but they said to make sure the client doesn't 0 value tx (https://github.com/lbryio/lbrycrd/issues/186#issuecomment-474944579).
2019-07-21 14:46:31,056 WARNING  torba.client.basenetwork:33: Wallet server returned an error. Code: 1 Message: the transaction was rejected by network rules.

dust (code 65)
[010000000140506c0788199a132c21555f8fb1fddd0c3b2616fed15d4f47e1e45236348f54010000006b483045022100c5591a496eb21bb40d409588f033ba38dcc28d3c1bc771824875f42a4666d4000220605f6ee84006753220ba1b0f8a2f509c96d2c8ddaf9bc9c7231b7a038a056c670121028486943fede99ddab35ec7e508a7482694792786329ec28321a4f530bb59ea96ffffffff020a00000000000000fd9401b51274686174732d7072657474792d67726561744d62010163221b027f3f8772cb6cbd68de5f5df0bd349bea326d86c1adfa57fd4b002f15df02fae287834f17253c5a7e573861bc376f45d469858c39e5e5281aa15d5352a2165e12b3f1eb87dbef636990a4728ba8b46e240acb010ab7010a309236fd71ecb5b715dacdaac40d06c6191850282523c59e680ac6e8b6645f2970efade7416a3362772bb45dab32c1e21912416d616b652d6d652d707265676e616e742d6f722d696d2d74656c6c696e672d6d6f6d2d796f752d6675636b65642d6865722d626573742d667269656e642e6d703418f9b6b02c2209766964656f2f6d703432308b6b1d94bb77c839a659ede99aac71b5740805bfd66ba4394e86391c184f6d58e99a244d3c3fc516026784d57880b9821a044e6f6e655a0908800a10d00518ca08420754657374696e6752302a2e68747470733a2f2f737065652e63682f632f736a4142396847355972715956584c7435644a644a4c376b2e676966620208016d7576a9147fcb0783224560b540684c48b187b078eaeb8b7888ac7abf8329000000001976a914dad487e759d31e6adb1e86746b5cd747598e54a588ac00000000]
2019-07-21 14:46:31,213 ERROR    lbry.extras.daemon.Daemon:586: error handling api request
Traceback (most recent call last):
  File "lbry\extras\daemon\Daemon.py", line 580, in _process_rpc_call
  File "lbry\extras\daemon\Daemon.py", line 2339, in jsonrpc_publish
  File "lbry\extras\daemon\Daemon.py", line 2479, in jsonrpc_stream_create
  File "lbry\extras\daemon\Daemon.py", line 3502, in broadcast_or_release
  File "torba\client\basenetwork.py", line 34, in send_request
  File "torba\client\basenetwork.py", line 31, in send_request
  File "asyncio\tasks.py", line 442, in wait_for
  File "torba\rpc\session.py", line 481, in send_request
torba.rpc.jsonrpc.RPCError: (1, 'the transaction was rejected by network rules.\n\ndust (code 65)\n[010000000140506c0788199a132c21555f8fb1fddd0c3b2616fed15d4f47e1e45236348f54010000006b483045022100c5591a496eb21bb40d409588f033ba38dcc28d3c1bc771824875f42a4666d4000220605f6ee84006753220ba1b0f8a2f509c96d2c8ddaf9bc9c7231b7a038a056c670121028486943fede99ddab35ec7e508a7482694792786329ec28321a4f530bb59ea96ffffffff020a00000000000000fd9401b51274686174732d7072657474792d67726561744d62010163221b027f3f8772cb6cbd68de5f5df0bd349bea326d86c1adfa57fd4b002f15df02fae287834f17253c5a7e573861bc376f45d469858c39e5e5281aa15d5352a2165e12b3f1eb87dbef636990a4728ba8b46e240acb010ab7010a309236fd71ecb5b715dacdaac40d06c6191850282523c59e680ac6e8b6645f2970efade7416a3362772bb45dab32c1e21912416d616b652d6d652d707265676e616e742d6f722d696d2d74656c6c696e672d6d6f6d2d796f752d6675636b65642d6865722d626573742d667269656e642e6d703418f9b6b02c2209766964656f2f6d703432308b6b1d94bb77c839a659ede99aac71b5740805bfd66ba4394e86391c184f6d58e99a244d3c3fc516026784d57880b9821a044e6f6e655a0908800a10d00518ca08420754657374696e6752302a2e68747470733a2f2f737065652e63682f632f736a4142396847355972715956584c7435644a644a4c376b2e676966620208016d7576a9147fcb0783224560b540684c48b187b078eaeb8b7888ac7abf8329000000001976a914dad487e759d31e6adb1e86746b5cd747598e54a588ac00000000]')
2019-07-21 14:46:31,932 INFO     lbry.stream.reflector.client:97: Sent reflector descriptor 8b6b1d94
2019-07-21 14:46:31,932 INFO     lbry.stream.reflector.client:102: Reflector needs 45 blobs for 8b6b1d94
2019-07-21 14:46:35,380 INFO     lbry.stream.reflector.client:119: Sent reflector blob 5fbe6f50

high fee:

Multiple users have been running into this recently - I think something is up with our fee calculation. For dust ones, we need to add more LBC into the tx so it's  not dust.

closed by #2359

This is not the same problem - this error occurred during stream creation:
  File "lbry\extras\daemon\Daemon.py", line 580, in _process_rpc_call
  File "lbry\extras\daemon\Daemon.py", line 2339, in jsonrpc_publish
  File "lbry\extras\daemon\Daemon.py", line 2479, in jsonrpc_stream_create
  File "lbry\extras\daemon\Daemon.py", line 3502, in broadcast_or_release
  File "torba\client\basenetwork.py", line 34, in send_request
  File "torba\client\basenetwork.py", line 31, in send_request
  File "asyncio\tasks.py", line 442, in wait_for
  File "torba\rpc\session.py", line 481, in send_request```

Even more concerning is that the it was still reflected - I thought we fixed that bug on tx error. Will try to repro.

This is probably due to new lbrycrd settings and not related to the SDK. Will track down.

This was an issue on lbrycrd - we set the minrelayrate to 0.0000001, closing.


How to perform a selection

Is there any interface like this:
didSelectItemAtIndexPath
Thanks!

Hi @Insfgg99x
you could just attach a tap action to your grid views.
i.e.
WaterfallGrid(rectangles) { rectangle in
  RectangleView(rectangle: rectangle)
    .onTapGesture {
      print("Tap action")
    }
}

or
WaterfallGrid((0..<cards.count), id: \.self) { index in
  CardView(card: self.cards[index])
    .onTapGesture {
      print("Did selected item at index \(index)")
    }
}

Hope this helps
Thanks!


Computed fields with fallback value that is unicode results in OperationalError

Environment

Python version: *
Nautobot version: 1.1.0b1

Steps to Reproduce

Create a computed field
Enter a unicode character like 💀 into the "fallback value" field
💥

Expected Behavior
It should accept unicode.
I suspect we might have to deliberately create a MySQL table w/ utf8 encoding, which I personally assumed to be the default in MySQL 8.x. (See: https://stackoverflow.com/a/20349552/194311)

Observed Behavior
django.db.utils.OperationalError: (1366, "Incorrect string value: '\\xF0\\x9F\\x92\\x80' for column 'fallback_value' at row 1")

@jfach 💀

I am able to alleviate this as indicated in the linked SO answer. If I add the following to DATABASES:
        "OPTIONS": {"charset": "utf8mb4"},

I am able to successfully create a computed field on MySQL 8.x:

The final DATABASES config looks like this:
DATABASES = {
    "default": {
        "NAME": os.getenv("NAUTOBOT_DB_NAME", "nautobot"),  # Database name
        "USER": os.getenv("NAUTOBOT_DB_USER", "nautobot"),  # Database username
        "PASSWORD": os.getenv("NAUTOBOT_DB_PASSWORD", "insecure_password"),  # Database password
        "HOST": os.getenv("NAUTOBOT_DB_HOST", "localhost"),  # Database server
        "PORT": os.getenv("NAUTOBOT_DB_PORT", ""),  # Database port (leave blank for default)
        "CONN_MAX_AGE": int(os.getenv("NAUTOBOT_DB_TIMEOUT", 300)),  # Database timeout
        "ENGINE": os.getenv(
            "NAUTOBOT_DB_ENGINE", "django.db.backends.mysql"
        ),  # Database driver ("mysql" or "postgresql")
        "OPTIONS": {"charset": "utf8mb4"},
    }
}

Further context here: https://mathiasbynens.be/notes/mysql-utf8mb4


release: 2.1.2

:robot: I have created a release beep boop
2.1.2 (2022-10-09)
Performance Improvements

perfect use go 1.19 (87090e0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/starudream/clash-speedtest/releases/tag/v2.1.2 :sunflower:


fix(eval): support setting hard timeout per evaluation instance

End-user friendly description of the problem this fixes or functionality that this introduces

[ ] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

Give a summary of what the PR does, explaining any non-trivial design decisions
Seems this https://github.com/All-Hands-AI/OpenHands/commit/f31adac07a64cfa0f9f11199ae027d00a5bc83aa commit got lost somehow, and @mamoodi run into this issue: https://openhands-ai.slack.com/archives/C07L41Y6BDX/p1731962756272799

Link of any specific issues this addresses

To run this PR locally, use the following command:
docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:42afc43-nikolaik   --name openhands-app-42afc43   docker.all-hands.dev/all-hands-ai/openhands:42afc43

@xingyaoww I just tried this branch and I got:
16:48:40 - openhands:ERROR: shared.py:361 - ----------
Error in instance [astropy__astropy-12907]: SandboxConfig.__init__() got an unexpected keyword argument 'keep_remote_runtime_alive'. Stacktrace:
Traceback (most recent call last):
  File "/Users/mahmoudwork/Desktop/Main/OpenHands/evaluation/utils/shared.py", line 313, in _process_instance_wrapper
    result = process_instance_func(instance, metadata, use_mp)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/mahmoudwork/Desktop/Main/OpenHands/evaluation/swe_bench/run_infer.py", line 387, in process_instance
    config = get_config(instance, metadata)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/mahmoudwork/Desktop/Main/OpenHands/evaluation/swe_bench/run_infer.py", line 138, in get_config
    sandbox=SandboxConfig(
            ^^^^^^^^^^^^^^
TypeError: SandboxConfig.__init__() got an unexpected keyword argument 'keep_remote_runtime_alive'

@mamoodi fixed in most recent commit


⚠ Bofrost.at has degraded performance

In a80f199, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1630 ms

Resolved: Bofrost.at performance has improved in 9dba616 after 41 minutes.


rtmp

Recommended to support RTMP push command
`ffmpeg -re -ss 100 -i  test.mp4 -c:v copy  -c:a copy -y -f flv  "rtmp:"

How should we do that? The script waits for the response of ffmpeg before it gives the control back. But feel free to do a pr for this.

If you could also provide any link to the related documentation, so we could at least understand what this is supposed to do, it would be great.

When the live broadcast, the video through the RTMP protocol pushed to the live server
Publishing video to the server @Romain

I also would like to read rtsp and send them to a rtmp server. For hobby projects i would like to stream birdhouses and stuff and manage this trough a web interface built with php.
I don't know how it all works but if i could contribute i certain will contribute.

Thanks for this documentation @BPing.
I've got absolutely no experience with live stream, if someone wants to dig into this...
However, I'll tag this as a feature request.

how about this feature ?


add: send errors of the node to sentry

Since we already have a sentry connection and want to capture errors,
let's also capture contrib-node errors there for debugging and reporting
purposes.

@w4tsn, Thanks for contributing,
It's a good option to have these errors sent to Sentry, But I guess this is not required for all users.
I would suggest adding a checkbox option for the Sentry node (aka. config) to handle this optionally.
You may name it "Send internal node errors to Sentry." or any name you think is better clear.
let me know when you do this to merge the request.


Get Feedback form on Android does not send screenshot to feedback website

Hello,
On Android/MIUI devices after you upload a photo with camera or from gallery or even the screenshot taken with loadFeedbackFormWithCurrentViewScreenshot and Submit the form, the sent photo won't appear on the get feedback website for the respective feedback data, while the other stuff (text, rating, etc.) will appear.
This issue is happening only on Android/MIUI phones, while on iOS is working as expected.
Versions used:
"react": "18.2.0",
"react-native": "0.71.3",
"usabilla-react-native": "^1.0.4"

Thank you for your issue submission.
On December 1, we will be are closing out the issues tabs on the Usabilla GitHub repos. We will continue supporting the apps/SDKs via our Help Center and support team. Information on how to contact support will be provided in the readme sections of the repos. By consolidating our contact channels, our team can monitor and reply more efficiently.
If the issue you have reached out about persists, please send us an email at<EMAIL_ADDRESS>Please include all details provided here in your email, along with any other details that may be helpful.
-GetFeedback Digital Team


Increment version for containerregistry releases

Increment package version after release of Azure.Containers.ContainerRegistry

API change check
API changes are not detected in this pull request.


Make curvebot.fi mobile responsive

90% completed.
Header menus are removed. we still need hamburger menu in mobile ?
Regarding the Connect wallet (metamask & paloma) and Wallet address (metamask & paloma), how to show them on the mobile layout? ( We don't need to care about chain selector. it will be removed) cc: @Yoya
Note that we will need the chain selector in the future. Curve Momentum bot should be on ethereum and polygon


Docs: Passing an array to concat needs clarification

Documentation Related To Component:

concat
Please check those that apply

[ ] typo
[ ] documentation doesn't exist
[x] documentation needs clarification
[x] error(s) in example
[ ] needs example

Description Of The Issue

It says in the description that

You can pass either an array of Observables, or put them directly as arguments.

The way I understand this is literally passing an array of Observables. There's also an example illustrating this - Concatenate an array of 3 Observables.
In reality, however, it seems like passing an array doesn't work (example on Stackblitz). Instead of getting the values, I get back Observables. It only works if I pass an array of Observables and use the spread operator.
So is this a problem in the documentation or the implementation?

You have to use the spread syntax because an array is itself an ObservableInput. That is, an array is considered an observable source.

Thanks for the clarification. In that case the docs should be updated to reflect that.

Fixed in #4713


Migrate to use v0.14 API

I've tried to migrate to use v0.14 Input Plugin API.
This PR contains major update change.
Could you bump up major version if releasing new version of gem?
And please refer http://docs.fluentd.org/v0.14/articles/plugin-update-from-v12 before release new version.
For future work, timer plugin helper might be usable this plugin:
http://www.clear-code.com/blog/2016/8/30.html
Thanks in advance.

Thank you for you PR 😃
I will fix it with another PR


Can't build for WebGL2

Hi, thanks for this awesome library!
But I can't build WebGL2 and it seems to me that there are some files that are missing in this library.
Here are the logs:
[572/572] link path_fiddle.js
FAILED: path_fiddle.js
em++ -o path_fiddle.js obj/default/path_fiddle/fiddle_context_d3d.o obj/default/path_fiddle/fiddle_context_dawn.o obj/default/path_fiddle/fiddle_context_gl.o obj/default/path_fiddle/fiddle_context_vulkan.o obj/default/path_fiddle/path_fiddle.o librive.a librive_pls_renderer.a librive_decoders.a liblibpng.a libzlib.a liblibjpeg.a librive_harfbuzz.a librive_sheenbidi.a librive_yoga.a -flto -sALLOW_MEMORY_GROWTH=1 -sDYNAMIC_EXECUTION=0 -sWASM=1 -sUSE_GLFW=3 -sMIN_WEBGL_VERSION=2 -sMAX_WEBGL_VERSION=2 --preload-file /home/gold/rivs@/
error: /home/gold/rivs@/ does not exist
em++: error: '/home/circleci/project/submodules/rive-cpp/build/dependencies/emsdk/upstream/emscripten/tools/file_packager path_fiddle.data --from-emcc --preload /home/gold/rivs@/' failed (returned 1)
ninja: build stopped: subcommand failed.

Exited with code exit status 1

Code that I think related to the error:
https://github.com/rive-app/rive-renderer/blob/a7407663c3e4425703373d92bdf34037b9906d7d/premake5.lua#L136-L146
You can view the full logs from here: https://app.circleci.com/pipelines/github/double-pi/rive-renderer/8/workflows/1a8d2399-3cf9-4e67-9924-47da5bb028d2/jobs/9

I had a similar issue and had to clone the cpp package to the folder inside submodules for the renderer as well as the WASM submodules folder.


Try bumping coveralls and checkout actions

Trying to see if we can fix the coveralls reporting issues and unblock some of the outstanding PRs.

Coverage remained the same at 94.188% when pulling 00ad3adae14f9a2bf235e3c555a2db973b64f587 on fix-coveralls into 9b94828ba5b122e17eb1b7dfffdd30aa66ddf666 on master.

Seems like it works!


Update defining blocks page and playground samples

Replaced all '%' parameter annotations with '$' (closes #9211)
Removed 'eg:' from JSDoc comments in favor of setting the default with PARAM.defl. The eg notation automatically sets the first value listed as the default value for the parameter. This indirect way of setting the default may cause confusion, so we are no longer advertising it.
Added a sample for fixed instances of a class
Added a sample for default array values
Updated the classes sample to include an example with blockCombine
Removed all the information on Blockly mutators. The mutator blocks do not fit with the look and feel of our other blocks, so we no longer want to advertise it.

Also closes #9175 because the corresponding section was removed as unnecessary.

Nice set up changes.


Azure OpenAI API

How to config with Azure OpenAI?

OK,I fixed it
llm_api_key:
llm_base_url: https://test.openai.azure.com/
llm_custom_provider: azure
llm_model: TESTmodel
and go to "llm_client.py"
response = completion(
base_url=self.base_url,
api_key=self.api_key,
model=self.model,
messages=messages,
api_version='2023-07-01-preview',                <---add this one
temperature=1.0,
custom_llm_provider=self.custom_provider,

Thanks for reporting. I updated the readme. It doesn't say exactly what to do but it points to litellm which has good docs for azure.


Feature request: Support for Noise_XK_secp256k1_ChaChaPoly_SHA256

Having noiseprotocol support Noise_XK_secp256k1_ChaChaPoly_SHA256 (the Lightning variant of Noise) would be very useful for developers looking to build Lightning Network related projects in Python.
See BOLT #8: Encrypted and Authenticated Transport for details.
As far as I know there are no Noise protocol frameworks in Python with support for Noise_XK_secp256k1_ChaChaPoly_SHA256.

Scheduled for 0.3 release :)

@plizonczyk Very glad to hear that! Is the Noise_XK_secp256k1_ChaChaPoly_SHA256 development taking place in a specific branch/fork or is that job yet to be started? :-)

@practicalswift Took a while... 😞 noiseprotocol v 0.3.0 is now released and supports non-default backends.


Locale.toLanguageTag() uses the wrong cached version when switching separators

(This issue is in https://github.com/flutter/engine but since it has no issue tracker I assume this repo is fine.)
Package: dart:ui
When calling Locale.toLanguageTag(), it first checks if this is the same as the locale for a cached string, but does not check if the separator is the same as was used in the cached string:
https://github.com/flutter/engine/blob/b14d9719607a876778b27149ec37ea170187bb97/lib/ui/window.dart#L497-L503
If you're using this method with two different separators, or using code like flutter_localizations which uses .toString() and then calling .toLanguageTag() in your own code, one of the two methods is going to return the string in an unexpected format.
This is currently causing a crash at the previously linked line in flutter_localizations but only when our code calls .toLanguageTag() first and has not called it for another locale in the meantime, so it was a bit difficult to pin down.
If caching is desired for performance then the choice of separator should be cached as well so switching separators invalidates the cache.

The fact that flutter_localizations uses toString() when the method's own documentation discourages use except for debugging purposes is concerning as well but that's probably a separate discussion.

Fixed by https://github.com/flutter/engine/commit/8d28215c8b5dfbbb3d3d1e36e8f6492667c2a23a


Replace ambiguous python shebangs

ROS 2 exclusively uses Python 3. It would be best to specifically use a Python 3 interpreter to avoid getting invoked with an unsupported Python 2 interpreter.
Should resolve RPM build errors on RHEL like this one: https://build.ros2.org/view/Rbin_rhel_el864/job/Rbin_rhel_el864__rqt_image_overlay__rhel_8_x86_64__binary/41/

I was confused about that builfarm error, but this explains it. Thanks for your fix @cottsay


java.lang.RuntimeException: A cast to int has gone wrong. Please contact the mp4parser discussion group

I am getting below exception on line m = MovieCreator.build(videopath); ..my video path is correct..what could be the reason of this exception?
Fatal Exception: java.lang.RuntimeException: A cast to int has gone wrong. Please contact the mp4parser discussion group<PHONE_NUMBER>)
       at com.googlecode.mp4parser.util.CastUtils.l2i(CastUtils.java)
       at com.googlecode.mp4parser.AbstractBox.parse(AbstractBox.java)
       at com.coremedia.iso.AbstractBoxParser.parseBox(AbstractBoxParser.java)
       at com.googlecode.mp4parser.BasicContainer.next(BasicContainer.java)
       at com.googlecode.mp4parser.BasicContainer.hasNext(BasicContainer.java)
       at com.googlecode.mp4parser.util.LazyList.blowup(LazyList.java)
       at com.googlecode.mp4parser.util.LazyList.size(LazyList.java)
       at com.googlecode.mp4parser.BasicContainer.getBoxes(BasicContainer.java)
       at com.googlecode.mp4parser.authoring.samples.DefaultMp4SampleList.<init>(DefaultMp4SampleList.java)
       at com.coremedia.iso.boxes.mdat.SampleList.<init>(SampleList.java)
       at com.googlecode.mp4parser.authoring.Mp4TrackImpl.<init>(Mp4TrackImpl.java)
       at com.googlecode.mp4parser.authoring.container.mp4.MovieCreator.build(MovieCreator.java)
       at com.googlecode.mp4parser.authoring.container.mp4.MovieCreator.build(MovieCreator.java)
    at android.os.Handler.handleCallback(Handler.java)
       at android.os.Handler.dispatchMessage(Handler.java)
       at android.os.Looper.loop(Looper.java)
       at android.app.ActivityThread.main(ActivityThread.java)
       at java.lang.reflect.Method.invoke(Method.java)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java)

It looks like the parser encountered a box in the file larger than 2GB. Typically only the mdat box is larger than 2GB but the mdat box is prepared to deal with this situation. So my best guess is a corrupted file. Could you perhaps make this file available to me so I can check what's going on?

I have sent u files on email..My file is only around 20 mb.

Hi,
the mergedvideo file is corrupted. Your 18MB file only contains a data section (mdat or MediaDataBox) at 4225 which reports a size of 0xF3AD67 = 15969639. 15969639 + 4225 = 15973864 but your file is 18827037 bytes. The last 2.9MB do not belong to any 'box'.
Grüße,
Sebastian

That file i created by appending videos using this library.What i noticed is that replacing audio in video works fine for normal video but gives above exception if  i try to replace audio of appended video..Below code I am using for appending video-
      File mediaStorageDir = new  File(Environment.getExternalStoragePublicDirectory(
            Environment.DIRECTORY_PICTURES), "Videos");
    File f = new File(mediaStorageDir.getPath());
    File file[] = f.listFiles();
    String paths[] = new String[file.length];
    Movie[] inMovies = new Movie[file.length];
    for (int i = 0; i < file.length; i++) {
        paths[i] = file[i].getPath();
        inMovies[i] = MovieCreator.build(paths[i]);
    }
    List<Track> videoTracks = new LinkedList<Track>();
    List<Track> audioTracks = new LinkedList<Track>();
    for (Movie m : inMovies) {
        for (Track t : m.getTracks()) {
            if (t.getHandler().equals("soun")) {
                audioTracks.add(t);
            }
            if (t.getHandler().equals("vide")) {
                videoTracks.add(t);
            }
        }
    }
    Movie result = new Movie();
    if (audioTracks.size() > 0) {
        result.addTrack(new AppendTrack(audioTracks
                .toArray(new Track[audioTracks.size()])));
    }
    if (videoTracks.size() > 0) {
        result.addTrack(new AppendTrack(videoTracks
                .toArray(new Track[videoTracks.size()])));
    }
    BasicContainer out = (BasicContainer) new DefaultMp4Builder()
            .build(result);
    @SuppressWarnings("resource")
    FileChannel fc = new RandomAccessFile(String.format(Environment
            .getExternalStorageDirectory() + "/mergedvideo.mp4"),
            "rw").getChannel();
    out.writeContainer(fc);
    fc.close();
} catch (FileNotFoundException e) {
    e.printStackTrace();
    Log.e("Appending", e.toString());
} catch (IOException e) {
    e.printStackTrace();
    Log.e("Appending", e.toString());
}
String mFileName = Environment.getExternalStorageDirectory()
        .getAbsolutePath();
mFileName += "/mergedvideo.mp4";
return mFileName;

I recorded audios from MediaRecorder Android in .aac formats . I am getting the same error.

why this issue is closed
is there anyone who solved this


Propose site design

Joanna will propose an initial site design that the team will provide feedback about, for further revision and iteration in #286.
I'll actually close this task once we have the interactive XD URLs (EOD 8/12).

Here's the interactive XD prototype: https://xd.adobe.com/view/082065b6-ada4-454e-a03c-80defb22f2e7-5d73/
🎉


Fix ambiguous class resolution warning

Fixes Composer ambiguous class resolution warning:
Warning: Ambiguous class resolution, "Radic\BladeExtensions\Facades\Markdown"
was found in both  "/var/www/vendor/radic/blade-extensions/src/Facades/BladeString.php" and
"/var/www/vendor/radic/blade-extensions/src/Facades/Markdown.php", the first will be used.

Which also resulted in Markdown facades not working correctly.

Woops.. Thanks


Correct metadata-enrichment annotation prefix

Description

metrics-enrichment is not a thing, its metadata-enrichment 😅
How can this be tested?

play around with the annotations

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

Project coverage is 58.77%. Comparing base (3e41510) to head (d224969).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3273   +/-   ##
=======================================
  Coverage   58.77%   58.77%
=======================================
  Files         325      325
  Lines       18153    18153
=======================================
  Hits        10669    10669
  Misses       6285     6285
  Partials     1199     1199

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


Disabled feature

Hi! Thanks for you package!
Please, can you add "disabled" feature in new versions?

Good call
Rob Rogers
<EMAIL_ADDRESS>
On Mar 23, 2019, at 11:55 AM, Roman Fandeev<EMAIL_ADDRESS>wrote:
Hi! Thanks for you package!
Please, can you add "disabled" feature in new versions?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I also encountered a similar situation, I hope to add a function that prohibits writing.

@mrFANRA @KattemChen
done


Support multiple private container registries

Currently, Krane only supports one set off registry credentials at a given time.
They get configured when setting up Krane and are used for every request
that pulls from a container registry.
This has the unfortunate behavior that using multiple container
registries is currently not supported.

Say your using GitHub container registry, and you have private packages, so
you set your credentials. When Krane attempt to pull it's only dependent container
krane-proxy (from DockerHub not GCR) it runs into authorization errors

unauthorized: incorrect username or password

OPTION 1
Toggle the use of registry credentials.
This still means Krane uses a single global set of registry
credentials but can be turned off per-deployment
deployment.json
{
    "name: "postgres",
    "image: "postgres",
    "use_registry_credentials": false
}

OPTION2
Config driven approach for setting up registry auth.

We can remove the notion of "global" registry credentials
We are now multi-registry capable
Per deployment registry auth configuration

GitHub Container Registry
deployment.json
{
    "name: "postgres",
    "image: "postgres",
    "registry": {
        "url": "ghcr.io",
        "username": "@GITHUB_USERNAME",
        "password": "@GITHUB_TOKEN"
    }
}

Docker Container Registry
deployment.json
{
    "name: "postgres",
    "image: "postgres",
    "registry": {
        "url": "docker.io",
        "username": "@DOCKER_USERNAME",
        "password": "@DOCKER_PASSWORD"
    }
}

Nothing provided
When no registry details are provided container resources are pulled via a Anonymous user
deployment.json
{
    "name: "postgres",
    "image: "postgres",
}

Storing registry credentials
Krane offers a deployment secrets store. This means you can configure
per-deployment registry secrets and reference them in your deployment.json.
More about secrets
Here's an example of setting registry secrets
$ krane secrets add <deployment> -k DOCKER_USERNAME -v <value>
$ krane secrets add <deployment> -k DOCKER_PASSWORD -v <value>

And referencing them in your config
deployment.json
 {
  ...
   "registry": {
        "url": "docker.io",
        "username": "@DOCKER_USERNAME",
        "password": "@DOCKER_PASSWORD"
    }
  ...
}

OPTION 2 implemented in PR https://github.com/krane/krane/pull/30
Closing.


Excel XSSWorkbook takes more than 4 to 5sec to load from Stream (Bulk data)

NPOI Version
2.6.2
File Type

[] XLSX
[ ] XLS
[ ] DOCX
[ ] XLSM
[ ] OTHER

Upload the Excel File
         -

Reproduce Steps
       The excel should have bulk data with multiple sheet.

Issue Description
While reading the bulk data (2500KB) file taken almost 4 to 5 sec just create a XSSWorkBook instance.
Attempt 1:
               using (FileStream stream = new FileStream(filePath, FileMode.Open, FileAccess.Read))
               {
                         XSSFWorkbook xssfWorkbook = new XSSFWorkbook(stream);  == > 4 to 5 sec
               }

Attempt 2:
             IWorkbook xssfWorkbook;
             using (FileStream stream = new FileStream(filePath, FileMode.Open, FileAccess.Read))
             {
                     xssfWorkbook = WorkbookFactory.Create(stream);  == > 4 to 5 sec
              }

Both Attempt 1 & 2 has same behavior.

Without a sample Excel file I would just suggest buying a faster machine.

@tonyqus  I have attached the sample (.xlsx) file for your reference.
Configuration1 - Copy.xlsx
@lahma  FYI , I using a dev system with Intel 11th Generation Processors with 16GB RAM.

Looking at the file there just seems to be a lot data, like Sheet7 having 25k rows. With NPOI release build it takes 2 secs to initialize the workbook on my machine (AMD Ryzen 9 5950X). One option could be to lazy-initialize some structures, but if you need to access the data anyway it would then be loaded making the lazy-loading moot.

@utthungagokul Have you tried ExcelDataReader of ClosedXML?  I'd like to know their response time on this case.

I've tested with ClosedXML and here is result:
Elapsed NPOI: 9,0083647
Elapsed ClosedXML: 13,5467912

tested on core i3 6100u, code:
string filePath = "Configuration1.-.Copy (1).xlsx";

Stopwatch sw = new Stopwatch();
sw.Start();

using (FileStream stream = new FileStream(filePath, FileMode.Open, FileAccess.Read))
{
    using (XSSFWorkbook xssfWorkbook = new XSSFWorkbook(stream)) { }
}

sw.Stop();
Console.WriteLine("Elapsed NPOI: " + sw.Elapsed.TotalSeconds);

sw.Reset();
sw.Start();

using (FileStream stream = new FileStream(filePath, FileMode.Open, FileAccess.Read))
{
    using (var workbook = new XLWorkbook(stream)) { }
}

sw.Stop();
Console.WriteLine("Elapsed ClosedXML: " + sw.Elapsed.TotalSeconds);


Implement Coinbase

Is your feature request related to a problem? Please describe.
Coinbase trading platform is not available yet.
Describe the solution you'd like
Add a view to manage Coinbase accounts. Let the user choosing between Binance and Coinbase creds on Bot Create View.
Describe alternatives you've considered
None.
Additional context
None.

I'm new to the cryptocurrency world, but having explored the subject in detail, I understood that Coinbase is one of the safest options out there. It's the world's largest Bitcoin exchange and broker and is backed by trusted investors. I've already found the coinbase phone number out, so I can ask questions in case I face issues. Of course, when it comes to cryptocurrency, safety is never 100% guaranteed, but the fact that Coinbase is located in San Francisco (a highly regulated location) is another welcome bonus.


[locale] az: fix a word

@topchiyev
In Azeri, the word 'bir neçə' should be written separately. Here are some references:
// Wiktionary
https://en.wiktionary.org/wiki/bir_neçə
// Obastan
https://obastan.com/bir neçə/1010367/?l=az
// Azleks
https://www.azleks.az/online-dictionary/bir+neçə?s=undefined

Coverage decreased (-0.07%) to 88.436% when pulling 2d353392fdcdf2c262ce0cd08fac75a2fa3cdf4f on emiraslan:develop into 2c0b063b3bf95a285f9b38c173e262b6416c2e7f on moment:develop.

Good work emiraslan. I suggest to accept the merge request.


[docs] Use immediate export in MenuAppBar.js

[ ] I have followed (at least) the PR section of the contributing guide.

Interestingly, VSCode doesn't find this unless I actually open the file.
Please update the typescript version as well

Is this okay?

@aditya1906 Thanks


[jvm-packages] xgboost-jvm metapackage is not suffixed with Scala version

In past versions, the top-level metapackage xgboost-jvm was always suffixed with Scala version.
For example, from https://repo1.maven.org/maven2/ml/dmlc/xgboost-jvm_2.12/ :

However, the latest XGBoost (2.0.1) does not have the suffix:

As a result, downstream projects may not be able to correctly locate artifacts for a given Scala version. Now that we want to distribute XGBoost JARs for both Scala 2.12 and Scala 2.13, the issue is quite critical.
I am preparing a new patch release 2.0.2 to fix this issue.

XGBoost 2.0.2 is now available on Maven Central:
https://repo1.maven.org/maven2/ml/dmlc/xgboost-jvm_2.12/2.0.2/
https://repo1.maven.org/maven2/ml/dmlc/xgboost-jvm_2.13/2.0.2/
https://repo1.maven.org/maven2/ml/dmlc/xgboost4j_2.12/2.0.2/
https://repo1.maven.org/maven2/ml/dmlc/xgboost4j_2.13/2.0.2/


Step 12 seems to be missing parts 3 & 4

In the Angular app, change the font and boldness of new messages based on their sentiment.
Also in the Angular app, display a list of recently discussed topics as new messages are added.

@jmdobry ptal

The functionality to do the steps you mentioned is implemented for you, as the Cloud Functions Angular tutorial focuses on the Cloud Functions. I updated the tutorial to call this out.


Replace mongojs by mongoose

This MR replaces Mongojs by Mongoose. This is a really nice improvement as it keeps the model and the controller separate. It also really simplifies all of the controllers. Finally, the structure of the database can now be clearly understood by simply looking at the files in the /models/ directory.

did you get the chance to look at this @NikolasHadjipanayi ?

@mircealungu  Not yet am checking them chronologically.


Move out all shared code

Move all shared code into its own gem:

[x] Init Poltergeist
[x] iTunes Connect Login
[x] Apple Dev Portal Login and Team Selection
[x] Update Manager
[x] Available languages (deliver and snapshot)
[x] Helper class

Started: https://github.com/KrauseFx/fastlane_core

Integrate with tools

[ ] deliver
[ ] snapshot
[ ] frameit
[ ] PEM
[ ] sigh
[ ] produce
[ ] cert


Update actions versions

Actions were only set on major releases, but there are several released versions for the action in the minor and patch versions. This brings all actions to the latest.
Changes

Update all actions to latest releases.

How to test this PR

PR checks should pass.

Dunno why coverage is failing. Doesn't seem applicable.


Encoded values are not decoded?

I have a program set up as follows:
require 'rlp'

class Transaction
  include RLP::Sedes::Serializable

  set_serializable_fields(
    to: RLP::Sedes::Binary.fixed_length(20, allow_empty: true)
  )

  def initialize(*args)
    fields = parse_field_args(args)
    fields[:to] = [fields[:to]].pack('H*')

    serializable_initialize(fields)
  end

  def encoded
    RLP.encode(self)
  end

  def self.decode(data_in)
    deserialize(RLP.decode(data_in))
  end
end

recipient = "6ba381ce15b19c7e44b8603ad7e698059c09a39b"
tx = Transaction.new(recipient)

puts "Should decode to #{recipient}"
puts "Actually decodes to #{Transaction.decode(tx.encoded).to.unpack('H*').first}"

As you can see, it decodes a different value than it encoded. Does anyone know why this is the case? It should be decoding the same value.

Because binary data in 'to' will be incorrectly decode again in overridden initialize when you try to decode transaction.


'"tsd reinstall && tsd rebundle"' is not recognized as an internal or external command

When generating an app with the options listed below, the tsd step fails. However, it does work from the command line outside the script
Angular Version – 1.3
Structure – Feature based
Markup – HTML
Language – TypeScript
Controller as – Yes
Test Language – TypeScript
UT – Jasmine
E2E – Jasmine
Style – CSS
Polymer – No
Framework – None
ngRoute – No
Additional – None

Thank you. This is now fixed in 80170ec113185ca512a2feba5eaee14b1f2825ca.


Window 2022 Black screen

Description
We use GitHub actions to run automation tests that test our desktop application UI. FlaUI locator does not find the window, while it does in Windows 2019 image.  The video made in Windows 2019 has all of these.
Platforms affected

[ ] Azure DevOps
[X] GitHub Actions - Standard Runners
[X] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] Windows Server 2019
[X] Windows Server 2022

Image version and build link
20230219.1
Is it regression?

Expected behavior
Explorer and windows persist on the video
Actual behavior
No explorer and windows on the video
Repro steps

@11v1 hello! could you please provide minimal repro steps?

@ddobranic I'm sorry for the delay, had no chance to do the required. I have returned to this task again. My company granted me access to large runners with Windows 2019 meanwhile and I rechecked. So the results for now:

Windows 2019 default - works well
Windows 2019 large runner (any CPU number) - no desktop (black screen)
Windows 2022 default  - works well
Windows 2022 large runner (any CPU number) - no desktop (black screen)

I have created a public repo with workflow (just need to change the runner). It uploads the screen video to the artifacts: https://github.com/11v1/TestGithubActions

hello, @11v1! thank you for the info. this is known bug in larger runners. it will be fixed, but i cannot tell you exact time frame right now.

we've deployed fix. should work now.
I'm closing this issue, feel free to reopen if the problem still exists


Used pcall to load gitsigns if installed

Won't do anything if gitsigns package isn't installed, but if it is then binds h (for "hunk") to jumping git hunks.

Hi, thanks for this, but as for now, I would prefer to leave things like this to users. In my mind, plugins should do one thing well and allow users to combine them.
It is not a strong opinion, though. Maybe I will reconsider "supporting" some plugins in the future, but it would be starting with treesitter.
And by the way, you can also do this if you want the preview window to open:
require("nap").nap('h',
    function() require("gitsigns").next_hunk({ preview = true }) end,
    function() require("gitsigns").prev_hunk({preview = true }) end,
    "Next diff", "Previous diff")


rename @threepointone/react-css to glamor

This should work, but I'm having a problem where running npm run build is generating the below error -
ERROR in ./button.js
Module parse failed: /Users/threepointone/code/css-in-js/glamor/button.js Unexpected token (15:2)
You may need an appropriate loader to handle this file type.
SyntaxError: Unexpected token (15:2)
    at Parser.pp$4.raise (/Users/threepointone/code/css-in-js/glamor/node_modules/acorn/dist/acorn.js:2221:15)
    at Parser.pp.unexpected (/Users/threepointone/code/css-in-js/glamor/node_modules/acorn/dist/acorn.js:603:10)
    at Parser.pp$3.parseExprAtom (/Users/threepointone/code/css-in-js/glamor/node_modules/acorn/dist/acorn.js:1822:12)
    at Parser.pp$3.parseExprSubscripts (/Users/threepointone/code/css-in-js/glamor/node_modules/acorn/dist/acorn.js:1715:21)
    at Parser.pp$3.parseMaybeUnary (/Users/threepointone/code/css-in-js/glamor/node_modules/acorn/dist/acorn.js:1692:19)
    at Parser.pp$3.parseExprOps (/Users/threepointone/code/css-in-js/glamor/node_modules/acorn/dist/acorn.js:1637:21)
    at Parser.pp$3.parseMaybeConditional (/Users/threepointone/code/css-in-js/glamor/node_modules/acorn/dist/acorn.js:1620:21)
    at Parser.pp$3.parseMaybeAssign (/Users/threepointone/code/css-in-js/glamor/node_modules/acorn/dist/acorn.js:1597:21)
    at Parser.pp$3.parseParenAndDistinguishExpression (/Users/threepointone/code/css-in-js/glamor/node_modules/acorn/dist/acorn.js:1861:32)
    at Parser.pp$3.parseExprAtom (/Users/threepointone/code/css-in-js/glamor/node_modules/acorn/dist/acorn.js:1796:19)
    at Parser.pp$3

<snip...>

the line/char it's pointing to is a jsx <, quite odd it doesn't work. Not sure how to proceed :(

thanks @threepointone, much appreciated.
not sure why but the content of the webpack.config.js is missing the module.exports part.
changing this:
module: {
  loaders: [
    {
      test: /\.js$/,
      exclude: /(node_modules|bower_components)/,
      loader: 'babel',
      query: {
        presets: ['es2015', 'react']
      }
    }
  ]
}

to this:
module.exports = {
  module: {
    loaders: [
      {
        test: /\.js$/,
        exclude: /(node_modules|bower_components)/,
        loader: 'babel',
        query: {
          presets: ['es2015', 'react']
        }
      }
    ]
  }
}

fixes the issue.
if you don't mind, I would ask you to rename threepointone-react-css to glamor in the package.json (name and description) as well.

good catch! not sure how I missed that. made the suggested changes too. thanks!

awesome, thanks!


Add converter between UInt/UInt64/Int/Int64/Float/Double and Bytes.

Pull Request Test Coverage Report for Build 4242
Details

0 of 24 (0.0%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 80.949%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

double/double.mbt
0
2
0.0%

float/float.mbt
0
2
0.0%

int/int.mbt
0
2
0.0%

int64/int64.mbt
0
2
0.0%

uint/uint.mbt
0
2
0.0%

uint64/uint64.mbt
0
2
0.0%

bytes/bytes.mbt
0
12
0.0%

Totals

Change from base Build 4239:
-0.3%

Covered Lines:
4538

Relevant Lines:
5606

💛  - Coveralls

Maybe it would be better to have conversions with BytesView (after we add BytesView of course)
graph TD;

Data --> Bytes;
Bytes --> BytesView;
BytesView --> Data;

Maybe it would be better to have conversions with BytesView (after we add BytesView of course)
graph TD;

Data --> Bytes;
Bytes --> BytesView;
BytesView --> Data;

OK. Should I close this PR for now?

It will come soon, so we can leave this PR as it is now. Or we can merge the data -> bytes part first.


Bug: no-callback-literal

Environment
Node version: v20.10.0
npm version: 10.2.3
ESLint version: v8.56.0
eslint-plugin-n version: 16.6.1
Operating System: macOS 14
What rule do you want to report?
no-callback-literal
Link to Minimal Reproducible Example
N/A
What did you expect to happen?
let result: string

try {
  result = renderToString(component)

  if (options.template) {
    result = templateRenderer.renderSync(result, context)
  }
  cb(null, result)
} catch (err) {
  cb(err as Error) // `Unexpected literal in error position of callback.` error here
}

Participation

[X] I am willing to submit a pull request for this issue.

Additional comments
No response

Mmm, I am just taking a quick look on my phone and I can't see anything that is immediately obvious.

After taking a look at the ast, maybe we should add TSAsExpression to the couldBeError function 🤔
https://github.com/eslint-community/eslint-plugin-n/blob/master/lib/rules/no-callback-literal.js#L56
It may be worth giving that a try, but I can also try look in a day or two if that does not work

we could just ignore unknown nodes - to avoid possible false positives. will make a PR later.


Update package dependencies

This PR makes FakeBundle work with Swift 5 by updating the package dependencies (which now work with Swift 5).

Thanks! 😊


Flatten elements with nested arrays

First thanks for "mithril-query". I usually use MSX. When MSX transform, nested arrays occur as a consequence of iteration using array.
fix #20

@StephanHoyer I created the PR for #20. Could you check it?

thanks! Just published v1.0.1 with your fix.


Is it still work in 2023?

I just found this repo, is it still working at the moment?

Yes, but I'm too lazy to update fileID to corresponding files on Google Drive 🗿

Ok. I've downloaded the SEB, a long time wait but I got it 😅
Thank you.

Hi man, the SEB has just updated to a new version... and your tool does not work now 😥


ICP method does not write OD files

The ICP workflow does not currently calculate or write the Optical Density values.
This would be useful for:

downstream analysis and validation
clarity with modules and how to separate modules

My current plan is to add this code to "icp_wf.point_registration".  It will include the following operations:

Gather centroid locations at end of ICP-WF and place "blocks" at the centroid locations.
Compute background estimation.
Compute BG and Spot regionprops on centroid "blocks" from 1
compute ODs

This development exists in branch "robustspotsdetection" and in "bryant_OD_calc_in_ICP".  Will submit a PR from one of those.

Addressed in PR #34 and #38


Update README.md

CamelCase DevOps
Make buttons stand out more with back-ticks

Messed up that last line.


fix dpo_trainer bug for LLMs without bos_token in config

In dpo_trainer.py, the bos_token will be  automatically added if the first token in current tokenized sentence is not equal to  bos_token. Currently this bos_token is read from tokenizer config file, but in some LLMs, like Qwen2, bos_token is leaving to None in config, which results in None is added to the input_ids tensor , as shown below:

and then the followiing trace will be raised when running dpo:
Traceback (most recent call last):
  File "examples/scripts/dpo.py", line 185, in <module>
    trainer.train()
  File "/usr/lib/python3.10/site-packages/transformers/trainer.py", line 1859, in train
    return inner_training_loop(
  File "/usr/lib/python3.10/site-packages/transformers/trainer.py", line 2165, in _inner_training_loop
    for step, inputs in enumerate(epoch_iterator):
  File "/usr/lib/python3.10/site-packages/accelerate/data_loader.py", line 452, in __iter__
    current_batch = next(dataloader_iter)
  File "/usr/lib/python3.10/site-packages/torch/utils/data/dataloader.py", line 631, in __next__
    data = self._next_data()
  File "/usr/lib/python3.10/site-packages/torch/utils/data/dataloader.py", line 675, in _next_data
    data = self._dataset_fetcher.fetch(index)  # may raise StopIteration
  File "/usr/lib/python3.10/site-packages/torch/utils/data/_utils/fetch.py", line 54, in fetch
    return self.collate_fn(data)
  File "/usr/lib/python3.10/site-packages/trl/trainer/utils.py", line 461, in __call__
    torch.tensor(ex[k], dtype=dtype)
TypeError: 'NoneType' object cannot be interpreted as an integer
  0%|          | 0/25 [00:00<?, ?it/s]

Also, in LLMs like Qwen2, bos_token(<|endoftext|>) is not equal to the first template token(<|im_start|>). Automatically add this bos_token without user awareness will cause unexpected behavior when using the trained dpo model to inference. Actually the bos_token should only be added when this value in tokenizer config is not None.
After this PR, the data is running normally like this:

and the model can be trained normally:

Test command is:
python examples/scripts/dpo.py
  --dataset_name trl-internal-testing/hh-rlhf-helpful-base-trl-style \
  --model_name_or_path Qwen2/Qwen2-0.5B-Instruct \
  --per_device_train_batch_size 1 \
  --learning_rate 1e-3 \
  --gradient_accumulation_steps 1 \
  --logging_steps 10 \
  --eval_steps 500 \
  --output_dir ./output \
  --warmup_steps 150 \
  --report_to tensorboard \
  --bf16 \
  --logging_first_step \
  --max_length 4096 \
  --sanity_check True

@DZ9 since this snippet is used in a number of other trainers, would it be better to add it as a helper function and then use it in DPO, CPO, ORPO trainers?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@kashif Sure. I've changed the adding of bos and eos token function to utils.py, and applied them in dpo, orpo and cpo trainer.

thanks @DZ9 can you also run pre-commit run --all-files in the root of TRL folder to fix formatting issues

@kashif Absolutely. Done with running the formatting command and commited.


Very High Memory  与vscode 版本有关

vscode 1.31.1 版本内存正常释放.  常驻内存:400mb
vscode 1.32 版本内存不能正常释放. 常驻内存:1.5G+
在vscode 早期版本也出现过 .
这不是一个插件存在这个问题.应该是vscode内部处理内存释放有关!

Vscode 1.31.1 version memory is released normally.
Resident Memory: 400mb Vscode version 1.32 memory does not release properly.

Resident Memory: 1.5G+
Earlier versions of Vscode also appeared. This is not a plug-in that has this problem. should be vscode internal processing memory release related!

Does this reproduce in the latest VS Code insiders build with all extensions disabled?

同样的插件代码,为什么在不同版本内存会这么大的不同.  插件中添加了很多的 CompletionItem 如果api有修改在哪可以找到对应的文档?

The same plug-in code, why the memory in different versions will be so different.
A lot of CompletionItem have been added to the plug-in if the API has been modified where can I find the corresponding document?

Can you please test using the insiders build. If this only shows up when extension are enabled, please file an issue against that extension

insiders build 同样存在此问题.

我明天提供一个工程 复现这个问题吧！

是否和 Electron version  有关呢!
https://github.com/electron/electron/issues/17213

Can you please follow these instructions to collect performance information. Share the output of code-insiders --status

Closing due to lack of additional information.


Retry webhook EOF errors

This test sometimes fails due to [1].
[1] https://github.com/knative/pkg/issues/1509
Signed-off-by: Pierangelo Di Pilato<EMAIL_ADDRESS>

/retest

/test integration-tests_eventing-kafka-broker_main