ElasticsearchLogShipper has the wrong success criteria for Bulk operations

Moved from: https://github.com/serilog/serilog/issues/599

When reading from a durable file and uploading bulk actions to the elastic search server, the log shipper checks the HTTP response for a Success Code and if true it writes the bookmarks and moves on.

Due to our monotonic bookmarking in the durable shipper, as noted in the linked issue, it is difficult to know what to do when a single document in the middle of a bulk index fails.
We can use the _id field to stop us inserting duplicates here.  If we generate a unique _id when we save the files to the durable file, we can use this in each of our index commands to allow ElasticSearch to either: reject duplicate _ids that are already indexed using the create operation or "upsert" (updating any existing document with the same _id or create if nonexistent) using the index operation.  Only if the entire batch passes do we bookmark and continue.  Perhaps we could retry a batch a certain number of times before bookmarking it anyway?

What's the status on this issue ?
@konste stated on https://github.com/serilog/serilog/issues/599 that a PR was made with NEST 2.0, but I can't seem te find it.
@mivano was working on a feature branch  https://github.com/serilog/serilog-sinks-elasticsearch/tree/handlefailures , but this was never finished or merged to dev.
It would be nice to just parse the bulk response and log any failures. Retries are less important to me, as most failures will most likely be the result of malformed requests.
Are there still any plans to continute this work, or will it be discarded?

This is custom modification of the official Serilog ES Sink, done for two reasons:

To be able to apply modified ES Json Formatter;
To modify EmitBatch method so that it records possible server side failures to internal log;

ElasticsearchSinkMod.cs.txt

Thanks @konste . I'm using the DurableElasticsearchSink though, the parsing of the response happens in the ElastsearchLogShipper. Which means I'ld have to copy a lot more code.  I'll first try fixing the problem in my fork. I see you are using Nest to parse the response to BulkResponse. Something I already tried and which wasn't 100% corrent. So I'll look into that. Problem is the overlap with @mivano 's work, of which I wonder why it was discontinued.

#110

It just lost interest but we had indeed some ideas how to act on failures. So at least check the result from ES  by parsing the return data. If there is a failure in there (can even be one of the 50 in the batch) we allowed to either send it to selflog or another sink.
If interest is there, I can port it to the current version.


build: add node 16 and 17 to github ci

Solves https://github.com/addaleax/synchronous-worker/issues/2, and probably breaks node 15 or 16. Let's see.

Since my C++ is rusty, I don't have an answer for the incompability issues. It seems InitializeContext returns a boolean for node 15 and returns v8::maybe for node 16+.

Released in 📦<EMAIL_ADDRESS>thanks for the PR! :)


Add Forno API Key to ODIS

Forno recently added an API key for rate limiting requests. We need to add API keys for the Combiner and Signers to prevent them from getting rate limited.
Release TODO:

[ ] Test Combiner in staging
[ ] Test Signer in staging
[ ] Test Combiner in Alfajores
[ ] Test Signers in Alfajores
[ ] Release Combiner to mainnet
[ ] Release to mainnet cLabs signers
[ ] Release to mainnet partner signers (Note: Let's try to lump this with another change to limit overhead for partner signers, as they just upgraded to 1.1.9)

https://github.com/celo-org/celo-monorepo/pull/9225

This PR has been merged and just needs to be released

note to self: Would make sense to deploy to staging with a bigger change (e.g. account recovery)


allreduce_always_fp16

The same logic as allreduce_always_fp32 but using fp16 data type. It can be used when we have a very large model and PCIe bandwidth is insufficient.
Also changed new_zeros to new_empty in sparse_all_gather because we do not need zeros arrays there.

The same logic as allreduce_always_fp32 but using fp16 data type. It can be used when we have a very large model and PCIe bandwidth is insufficient.

Thanks for the PR. As I mentioned elsewhere, can you please share more motivation for this PR? I am also curious as to how PCIe bandwidth affects gradient reduction.

@Dipet, I am seeing your frequent updates on this PR. Can you please ping me when you think it is ready to review? Thanks so much for this massive contribution to DeepSpeed.

@tjruwase I fixed errors. And replaced  allreduce_always_fp16 and allreduce_always_fp32 by single argument - communication_data_type


Function call return type resolution based on function args

simplifies the mechanism that used to be part of Signature (Signature.return_type could be a Callable) and moves it into Function.call_return_type()
adding call type resolution to various function modules

This change is


vue/vue-server-renderer  vue-ssr-client-manifest.json  The initial property is an empty array times the bug

What problem does this feature solve?

When the initial attribute of the vUE - SSr-client-manifest.json file is an empty array

Start the Node service and access the IP in the browser

Build.dev. js file in the vue-server-renderer plugin reported a bug in line 9074sha

TypeError: Cannot read property 'file' of undefined
at /Users/binbinwang/workspace/x-fe/vue-ssr/node_modules/vue-server-renderer/build.dev.js:9074:24
What does the proposed API look like?
vue-ssr-client-manifest.json
{
  "publicPath": "/dist/",
  "all": [
    "app.0178268b84efde12045b.js"
  ],
  "initial": [],
  "async": [
    "app.0178268b84efde12045b.js"
  ],
  "modules": {
    "239adfd4": [
      0
    ],
    "6fbc4fdc": [
      0
    ]
  }
}

I solved it
return needed.map(function (ref) {
      if(!ref) return   // Add this line
      var file = ref.file;

      return ("<script src=\""   (this$1.publicPath)   file   "\" defer></script>")
    }).join('')

fixed in https://github.com/vuejs/vue/pull/11609


net: tcp: Correctly determine when the TCP transmit window is full

In the stack both unacked_len and send_data_total track the amount
of data for retransmission. send_data_total actually accounts the
total bytes in the buffer, where unacked_len is used to control the
retransmission progress.
Using unacked_len is sometimes reset to 0, this can lead to more data
being allowd in the send_data buffer. In worse case this can cause
depletion of the net buffers, causing a stall and crash of the connection.
The value send_data_total actually accounts the total amount of data in
the send_data buffer, so it is the proper value to used in the
tcp_window_full function.
Signed-off-by: Sjors Hettinga<EMAIL_ADDRESS>

Thanks for your comments, I applied your feedback

@jukkar, Could you have a look at this PR?


Switch using sticky sessions and hash director in varnish, resolves #71.

Why is this done?

When we have more than 1 backend web servers, round_robin just randomly throws all the request between servers. This ensures all requests within a session end up in the same server so that for example batch processes and file uploads work correctly.

Ok, great. Thanks for the info!


Scrollbar needed for pages overview in admin

The new touch-enabled interface doesn't work well for me on a desktop.
In the pages overview, it needs zooming out to see all languages (I'd really like to know which pages have been translated) - and then the text becomes so small, that I can barely read it.
A simple scroll bar that is available if on a not-touch-enabled, stationary device would be of great help.
(we might want to add additional languages)
Screenshots attached (Standard zoom, zoomed out to fit contents)

Django 1.8.7., Django-cms 3.2.0

@vxsx was this fixed on 3.3?

I have to check @czpython
It's definitely scrollable horizontally with djangocms-admin-style, will have to check what happens without it

Hello @Moini,
Thank you very much for your contribution to the django CMS project. The django CMS project has recently joined forces under the banner of the newly founded django CMS Association to revive the development of django CMS. For this reason we have analyzed all pull requests and issues in detail and decided which ones fit the product vision we want to go with django CMS.
Due to the age of this issue we are unable to safely accept the contribution. If you feel this issue is still required please inform a member of our technical committee<EMAIL_ADDRESS>Nevertheless we thank you for your participation and would like to emphasize that we do not want to discourage you in your contribution to the django CMS project. On the contrary, we encourage everyone who is interested in the success of django CMS to join us!
If you want to learn more about the work of the technical committee, our product vision and how you can become a member of the dCA, you can find all information here:

The Technical Committee of the django CMS Association
Technical Roadmap and Product Vision for django CMS
How to become a member of the django CMS Association

We are looking forward to every companion who would like to shape the future of django CMS together with us!
If you have any questions, feel free to reach out to Nicolai, our Community Manager<EMAIL_ADDRESS>Sign up for the Association now!


fix(OsvServiceFunTest): Update expected test results

Merging despite the BabelFunTest failure, which seems to be flakyness as the test passes locally for me.


SelectInput width not resizing

While using shinybootstrap2 in Shiny 0.11, The selectinput components are not resizing in wellpanel even though i set width as "100%". Is this a known issue or am i doing something wrong?

Could you supply a simple reproducible example?

shinybootstrap2::withBootstrap2({
  shinyApp(
	ui = fluidPage(
	  selectInput("n", "resize", c(1, 5, 10),width = "100%")
	),
	server = function(input, output) {
	}
  )
})

this is a simple example where the select input box should have been full size of the page.

shinyApp(
  ui = fluidPage(
	tags$div(
	selectInput("n", "resize", c(1, 5, 10),width = "100%")
	)
  ),
  server = function(input, output) {
  }
)

I just noticed enclosing selectInput in it's own div makes it work fine


Update linein to 2.3.0

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Not sure if this should be moved to caskroom/homebrew-versions.

This implies that it is discontinued.
  caveats do
    discontinued
  end

I think there was another app by RogueAmoeba that you updated with the legacy link. Could you add the caveat there as well? Thanks!

I think there was another app by RogueAmoeba that you updated with the legacy link. Could you add the caveat there as well? Thanks!


Pivot table: is it possible to apply a filter value?

In the example below (Linux) I created a pivot table with a PivotFilter on the field 'City'. Can I instruct ImportExcel to apply the filter value 'Dallas' on startup of Excel?
$data = ConvertFrom-Csv @"
Region,State,Units,Price,City
West,Texas,927,923.71,Dallas
West,Texas,927,923.71,Dallas
West,Texas,927,923.71,Houston
North,Tennessee,466,770.67,Nashville
East,Florida,520,458.68,Miami
East,Maine,828,661.24,Bangor
West,Virginia,465,053.58,Richmond
North,Missouri,436,235.67,St Louis
South,Kansas,214,992.47,Wichita
North,North Dakota,789,640.72,Bismarck
South,Delaware,712,508.55,Georgetown
"@

if (Test-Path ~/salesData.xlsx) {
    Remove-Item ~/salesData.xlsx
}

$data | Export-Excel ~/salesData.xlsx -Autosize -AutoNameRange -PivotRows State -PivotColumns Region -PivotData @{'Units'='sum'} -PivotFilter City -Activate -Show

My dev environment:
PSVersion                      7.3.2
PSEdition                      Core
GitCommitId                    7.3.2
OS                             Linux 5.19.0-32-generic #33~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Mon Jan 30 17:03:34 UTC 2
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0
ImportExcel version:  7.8.4 (Linux)
Regards,
Daniel

No. That is only doable after Excel opens the doc. Some have gone the lengths of creating VBA and enabling that to run when Excel opens. I don't have examples of that.


Incorrect memory usage stats [Container statistics]

Before you start please confirm the following.

[X] Yes, I've searched similar issues on GitHub.
[X] Yes, I've checked whether this issue is covered in the Portainer documentation or knowledge base.

Problem Description
Container statistics page displays wrong memory usage stats.
Example for Jellyfin:

Where htop shows:

And docker stats:

Expected Behavior
Portainer should display correct information about memory usage.
~600MiB in this case
Actual Behavior
Shows that 10GB is used
Steps to Reproduce

Go to Container
Click on 'Stats'
See memory usage

Portainer logs or screenshots
No response
Portainer version
2.21.0
Portainer Edition
Community Edition (CE)
Platform and Version
Docker 27.1.2
OS and Architecture
Ubuntu 24.04
Browser
No response
What command did you use to deploy Portainer?
docker run -d -p 8000:8000 -p 9443:9443 --name portainer --restart=always -v /var/run/docker.sock:/var/run/docker.sock -v portainer_data:/data portainer/portainer-ce:latest

Additional Information
No response

Closing this as this seems to be Jellyfin memory leak problem:
https://github.com/jellyfin/jellyfin/issues/11551

Fwiw, I'm having the same observation. The memory usage reported using docker stats differs from the one reported in Portainer-ce (v2.21.3).
Moreover, I don't uderstand how a Jellyfin memory leak could influence this, since the leak would cause the memory usage of the container to rise?
I suggest to re-open this issue.

Reopening since as mentioned, portainer should display values correctly

Seems similar to https://github.com/portainer/portainer/issues/1961

I found the following information in the Docker documentation:
On Linux, the Docker CLI reports memory usage by subtracting cache usage from the total memory usage. The API does not perform such a calculation but rather provides the total memory usage and the amount from the cache so that clients can use the data as needed. The cache usage is defined as the value of total_inactive_file field in the memory.stat file on cgroup v1 hosts.
On Docker 19.03 and older, the cache usage was defined as the value of cache field. On cgroup v2 hosts, the cache usage is defined as the value of inactive_file field.
https://docs.docker.com/reference/cli/docker/container/stats/
I think it should be consistent with the results of docker stats to reduce misunderstandings.


Add user via cmd line creates owner

Description:
It was no longer possible to create an owner via the local authentication script. Thanks to @jeradM for catching it. This fixes it.
Checklist:

[x] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Oops that was an accidental merge, didn't see the comments. I will address them.


feat: shrink to minimum value first

Current dependencies on/for this PR:

main

PR #102   👈

This comment was auto-generated by Graphite.


Is the custom property for richtext type still working?

According to the documentation: https://adminbro.com/tutorial-cms.html we can pass a property named custom with parameters that are then serialized and passed to the frontend. But after several attempts, I cannot get Adminbro to pass the data, and after looking in the source https://github.com/SoftwareBrothers/admin-bro/blob/master/src/frontend/components/property-type/richtext/edit.tsx It doesn't seem that something is passed, in fact, the options for Quill are static (https://github.com/SoftwareBrothers/admin-bro/blob/c68e88be4414ede929899c90b82c9ebadf908cb4/src/frontend/components/property-type/richtext/edit.tsx#L78-L94)
Am I missing something or this is really broken?

we are in the process of rewriting quill because it is buggu. New version will be released next week


Default avatar color is wrong on busy-img

For users not having avatar there is a colored default avatar. It seem that since the recent change the default color depend on the first letter of the username, while it should depend on the full username. Having a different color help user to not confuse blocktrades with wrong spelling username like blocktarde

Should be good now.

Indeed problem solved, closing this issue.


Fix Zenodo configuration

Pull Request Checklist:

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

This PR fixes #xyz

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Documentation has been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated (with summary of main changes)

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

[ ] bumpversion patch has been called on this branch
[ ] The relevant author information has been added to .zenodo.json

What kind of change does this PR introduce?

Adds a missing curly bracket that prevented version being published on Zenodo

Does this PR introduce a breaking change?
Other information:

Coverage remained the same at 90.773% when pulling d43f82337d57a9c2a815b937770b748208e65cee on zenodo_fix into b7856bf40fe1e054396a085a4486973991aa90a3 on master.


CB-13170 crn region followup - enforce correct region usage

most of Crn Builder methods and constructor are package private
added regionaware generator to mock
added test crn generator util to integration-test module
moved crn related class to proper package

See detailed description in the commit message.

retest this please


Missing require and assert built-in functions

Just addressed this in a newly maintained fork https://github.com/thesis/vim-solidity


using nebula.field() on same page as sn-table makes table rendering unreliable

If I add 4 nebula field lists to the same page as my sn-table, table doesn't render and I see this in console:

Hi @chadmoore,
Try to update all @nebula.js related packages to the latest version.
Please let me know if it does not help.

Unfortunately it seems this issue persists with v1.9.0

Unfortunately it seems this issue persists with v1.9.0

Thank you for letting me know. Which version of the Qlik sense and the @nebula.js/stardust do you use?

Qlik Sense Enterprise Aug 2021 - 14.28.3
@nebula.js/stardust - v2.6.1

The issue persists w/ QSE Aug 2021 & stardust v2.8.0

The issue persists w/ QSE Aug 2021 & stardust v2.8.0

Thanks. We gonna look into this and reply back to you soon.

Hey @chadmoore, I had a test with Aug 2021 QSEfW and cannot reproduce your issue. Please have a look at my test code https://github.com/qlik-oss/sn-table/tree/mashup/QSEfW/mashup-example/using-build-tool/qlik-sense-enterprise and feel free to do changes based on my branch.

Hi @chadmoore. Since we can't reproduce this bug, we will close it unless additional information is provided


Added FormNX in Web application

Added FormNX

Hi there, is this free software? Could you get the license link?


[DX] Research on Firebase features

Description
Get ready to migrate to Firebase by doing some research - in case we actually decide to go with it. Related: #243
[ ] How to do migrations (within Firebase)?
[ ] How to get a schema and create a new environment based on it?
[ ] How to protect an API path and use admin-level access in it
[ ] How to make a local backup
[ ] What's the pricing model?
[ ] Any other potential issues?

working on it will submit shortly

@adilwahla thank you. Perhaps I should've clarified this earlier, but what I meant by migrations is migrations within a Firebase project, not a migration from Supabase to Firebase.

[ ] How to do migrations (within Firebase)?
https://firebase.google.com/docs/firestore/manage-data/move-data
[ ] How to get a schema and create a new environment based on it?
iam not sure what the question is
[ ] How to protect an API path and use admin-level access in it
https://firebase.google.com/docs/projects/iam/roles
[ ] How to make a local backup
https://firebase.google.com/docs/database/backups
[ ] What's the pricing model?
https://firebase.google.com/pricing
[ ] Any other potential issues?
when scale migrating out of firebase is problematic
cost when scale is high
don't use it when you have heavy relational data and need to enforce ACID


Add support for #if clauses for code generation

When you use weaver to declare an object, the imports of that file get added to the Generated/Weaver.swift file. If I have something like
import Foundation
#if os(watchOS)
import WatchKit
#endif

The generated Weaver file will have this at the top:
import Foundation
import WatchKit

Implemented in #148


IJsselmeer not visible on map

We could search for another source file for the basemap.

Finally found IJselmeer in Natural Earth lakes. Those shapes have been substracted from the countries, creating a countries-minus-lakes.shp, which is used as the basis for the topojson.


Memory and database not matching interface available_tags values

While creating 160 EVCs (80 request at a time), the database did not have the same available_tags as what I had in memory.
From memory:
kytos $> controller.switches['00:00:00:00:00:00:00:01'].interfaces[4].available_tags
Out[2]: {'vlan': [[161, 4095]]}

From database
    _id: '00:00:00:00:00:00:00:01:4',
    available_tags: { vlan: [ [ 160, 4095 ] ] },
    id: '00:00:00:00:00:00:00:01:4',
    inserted_at: ISODate('2024-08-05T15:52:37.062Z'),
    special_available_tags: { vlan: [ 'untagged', 'any' ] },
    special_tags: { vlan: [ 'untagged', 'any' ] },
    tag_ranges: { vlan: [ [ 1, 4095 ] ] },
    updated_at: ISODate('2024-08-05T15:53:01.050Z')

I  have only seen this issue happen once. It was found when only EVC creation was performed. Another modification for VLANs would hide this issue.
This interface was part of failover_path for all EVCs and it was an NNI (from RingTopo).
Edit:
I am working with a modified mef_eline which could be the cause of the problem. Although at second glance these changes should not affect the process of reporting/using VLANs

Good finding @Alopalao, I've temporarily set the target 2024.1 just in case to show up in our issues filter. If it's confirmed that it was just the modified local changes then it's OK and can get closed, otherwise we can reassess if it needs to be fixed.

I got this error again. Seems to be a problem with MongoDB. I added a log in between the lines, log.info(f"From mongo {id_}: {updated['available_tags']}"). It should be printing the last mongo updated dictionary.
2024-08-06 17:26:57,096 - INFO [kytos.napps.kytos/topology] (thread_pool_app_153) From mongo 00:00:00:00:00:00:00:02:3: {'vlan': [[158, 4095]]}
2024-08-06 17:26:57,125 - INFO [kytos.napps.kytos/topology] (thread_pool_app_191) From mongo 00:00:00:00:00:00:00:02:3: {'vlan': [[156, 4095]]}
2024-08-06 17:26:57,229 - INFO [kytos.napps.kytos/topology] (thread_pool_app_173) From mongo 00:00:00:00:00:00:00:02:3: {'vlan': [[160, 4095]]}
2024-08-06 17:26:57,269 - INFO [kytos.napps.kytos/topology] (thread_pool_app_58) From mongo 00:00:00:00:00:00:00:02:3: {'vlan': [[2, 2], [160, 4095]]}
2024-08-06 17:26:57,328 - INFO [kytos.napps.kytos/topology] (thread_pool_app_22) From mongo 00:00:00:00:00:00:00:02:3: {'vlan': [[160, 4095]]}
2024-08-06 17:26:57,340 - INFO [kytos.napps.kytos/topology] (thread_pool_app_62) From mongo 00:00:00:00:00:00:00:02:3: {'vlan': [[161, 4095]]}

The last log has the VLAN list [[161, 4095]] but the database was not updated:
  {
    _id: '00:00:00:00:00:00:00:02:3',
    available_tags: { vlan: [ [ 160, 4095 ] ] },
    id: '00:00:00:00:00:00:00:02:3',
    inserted_at: ISODate('2024-08-06T21:26:29.048Z'),
    special_available_tags: { vlan: [ 'untagged', 'any' ] },
    special_tags: { vlan: [ 'untagged', 'any' ] },
    tag_ranges: { vlan: [ [ 1, 4095 ] ] },
    updated_at: ISODate('2024-08-06T21:26:57.095Z')
  },

It may be worth to stress test the sending of kytos/core.interface_tags which triggers the upsert of interface_details.

Aldo, yes, it needs further investigation. Is it still reproducing without the local changes too, or you don't know yet?
I was also reviewing, we do have event out of order protection there too, but that only works if the events themselves are guaranteed to have being produced in order too, since it's based on its timestamp creation attribute. If you're not seeing the write happen in the DB, add a log to see if it's discarding. If it's discarding what's not supposed to, then we'll need to fix/review and ensure that the producer of this event on kytos core to ensure it produces in order too.
It's still possible that a thread get preempted right on this line here too, although MongoDB ensures thread safety for find_one_and_update, but let's go by parts here and see if anything is getting discarded first. In the worst case we can also extend the scope of the lock.

I tried in local changes but it did not appeared. Trying to figure out how my changes can make this issue occurred at the moment.

~I tried in local changes but it did not appeared~. Trying to figure out how my changes can make this issue occurred at the moment.
Edit: It finally did appeared in master branch. This time was with interface 01:1 which was a UNI. I did not check if it was rejected.

OK. If it's happening on master it's a bug then. I've set the bug label, and priority_major. We'll discuss about it in the next meeting. If you can find out if it was rejected or discarded the event that's very helpful.

Something extra I noticed is that when this issue appeared, failover_path of a deployed EVC was set two times by consistency check. These two issues seemed not to be related since mongo logs showed a mismatch with the end values in the DB.
The second deployment of the failover_path occurs in circuit.try_setup_failover_path() from execute_consistency()

To duplicate this issue with a high rate of success is to add this method inside the class Interface:
    def testing3(self, controller, n):
        for i in range(n):
            self.available_tags = {"vlan": [[i, 4095]]}
            self._notify_interface_tags(controller)

And this can be called from kytos console preferably with a high value number:
controller.switches['00:00:00:00:00:00:00:01'].interfaces[1].testing3(controller, 200)

Also, only works after a kytos shutdown.
Moving this line to be inside the lock solves the issue.


Consider warning on Livy connections

Something along the lines of "you shouldn't do this if you can help it ;)"
@edgararuiz @javierluraschi

Hi why not ?

@aschmu it's much slower than running yarn-client or yarn-cluster mode and doesn't support extensions.

Thanks, so if I understand correctly this isn't inherent to sparklyr per se but to Livy, right ? I suspect that it's mainly because moving around large datasets through HTTP is inefficient, are there other causes of inefficiency?

The way Livy is implemented in sparklyr each invoke() call is a separate request to the server. In some functions invoke() is used liberally so the overhead adds up. I'm not too familiar with how the other APIs talk to Livy. There's also some relevant discussion at https://github.com/rstudio/sparklyr/issues/741.


Saml, Saml2, JsonWebToken, JwtSecurityToken shouldn't share error codes.

These handlers are sharing codes from Microsoft.IdentityModel.Tokens.Messages.cs.
They should be partitioned so it is immediately obvious where the exception was thrown.

not going to worry about this.


⚠ Thumbnails API Endpoint has degraded performance

In 9b82d68, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1069 ms

Resolved: Thumbnails API Endpoint performance has improved in 8b30038 after 16 minutes.


Allow to set and get cookies on XHR

It should be possible to set and get cookies that are handled by XHR.

An implementation has been added for Android but not for iOS yet.

We need to clarify the technical details on ios.

Research will be gathered in https://github.com/eclipsesource/tabris-js/issues/857.


Doesn't work in newer Unity/URP Versions

Really interested in getting this to work, but getting a bunch of errors when importing in newer version;
Unity 2020.2.5f1
URP 10.3.2
Universal Render Pipeline/Iridescence Lit': 'EnvironmentBRDF': cannot implicitly convert from 'struct BRDFDataAdvanced' to 'struct BRDFData' at Models/Iridescence/ShaderLibrary/IridescenceLighting.hlsl(326)

Yeah there's also "InputDataAdvanced" that is broken.


USPS (add more details in response)

https://www.usps.com/business/web-tools-apis/track-and-confirm-api.pdf


remove (unnecessary) python dependency

Currently there is a single line depending on python. It would be nice to remove  this unnecessary dependency entirely or make it optional.

Is this hypothetical or is this preventing blackbox from running on a particular OS?

I does not prevent me from running blackbox but it makes the installation more difficult (because I need to install python or modify the line). The python requirement is also not mentioned in readme so I found out about it by running blackbox and receiving error which is not very friendly.

Thank you!


JobsPrePaymentEvent always gives null job

*Description of issue:
JobsPrePaymentEvent always fires null job.
Here is my code:
@EventHandler(ignoreCancelled = true)
    public void onPayment(JobsPrePaymentEvent event) {
	if (event.getPlayer().getPlayer() == null) return;
	Player player = event.getPlayer().getPlayer();

	if (player == null || event.getJob() == null) return;
        Bukkit.broadcastMessage(player.getName() + " isimli oyuncuya " + event.getJob().getName() + " mesleğinden ödeme geldi.");

	double amount = getPayment(player, event.getJob(), event.getAmount());
	event.setAmount(amount);
	event.setPoints(amount);
    }

It always returns because event.getJob() is always null. I tried it with different jobs but result didn't change.

https://github.com/Zrips/Jobs/commit/6cd52625e2be91ab3a6cea828561a4c5b7db2ec9

Thanks for the fast fix!


⚠ AVS (kty) has degraded performance

In 45050e8, AVS (kty) (https://avus.com.br) experienced degraded performance:

HTTP code: 200
Response time: 9729 ms

Resolved: AVS (kty) performance has improved in 6581559.


chore!: update to ACVM 0.13.0

Description
Problem
Updates to track ACVM 0.13.0
Summary*
This PR tracks the changes made in https://github.com/noir-lang/acvm-backend-barretenberg/pull/205.
Getting a strange error where rust can't find acvm-backend-barretenberg for some reason.
Documentation

[x] This PR requires documentation updates when merged.

[x] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

compute_merkle_root is no longer implemented as a black box function (but still exists in stdlib)

Additional Context

PR Checklist*

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

@TomAFrench thanks for helping with documenting this PR!

compute_merkle_root is no longer implemented as a black box function (but still exists in stdlib)

We don't currently mention is compute_merkle_root a black box function or written in Noir in our docs:
https://noir-lang.org/dev/standard_library/merkle_trees
Anything that you'd like to add to the docs in this case?


Add more info to manifest and to publishing

following (and more) can be added to manifest and publishing can also take advantage of more information
            "Specification-Title"   : "ProximaX Sirius Blockchain Java SDK"
            "Implementation-Vendor" : "ProximaX"

PR created


Adding my solution

FizzBuzz wasting some CPU cycles... 👯

Thanks for your PR! Unfortunately a solution like that has already been submitted :/


[Snyk] Security upgrade express from 4.17.3 to 4.19.2

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

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

courses/developingapps/v1.3/nodejs/datastore/end/package.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

519/1000  Why? Has a fix available, CVSS 6.1
Open Redirect SNYK-JS-EXPRESS-6474509
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: express
    The new version differs by 136 commits.

b28db2c 4.19.2
0b74695 Improved fix for open redirect allow list bypass
4f0f6cc 4.19.1
a003cfa Allow passing non-strings to res.location with new encoding handling checks fixes #5554 #5555
a1fa90f fixed un-edited version in history.md for 4.19.0
11f2b1d build: fix build due to inconsistent supertest behavior in older versions
084e365 4.19.0
0867302 Prevent open redirect allow list bypass due to encodeurl
567c9c6 Add note on how to update docs for new release (#5541)
69a4cf2 deps<EMAIL_ADDRESS>4ee853e docs: loosen TC activity rules
414854b docs: nominating @ wesleytodd to be project captian
06c6b88 docs: update release date
1b51eda 4.18.3
b625132 build: pin Node 21.x to minor
e3eca80 build: pin Node 21.x to minor
23b44b3 build: support Node.js 21.6.2
b9fea12 build: support Node.js 21.x in appveyor
c259c34 build: support Node.js 21.x
fdeb1d3 build: support Node.js 20.x in appveyor
734b281 build: support Node.js 20.x
0e3ab6e examples: improve view count in cookie-sessions
59af63a build<EMAIL_ADDRESS>e720c5a docs: add documentation for benchmarks

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

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

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Open Redirect

The merge request fixes a medium severity vulnerability.


Fix syntax for image

Fix invalid syntax for image rendering

@pascalberger : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Verified the update.


GodotHingeJoint3D is ignoring the softness attribute m_limitSoftness

Godot version
latest on master (543750a1b3f5696f9ba8e91cb49dc7db05d2ae62)
System information
Not relevant
Issue description
I think this line:
real_t amplitude = ((angVelB - angVelA).dot(axisA) * m_relaxationFactor + m_correction * (real_t(1.) / p_step) * m_biasFactor) * m_limitSign;

should read:
real_t amplitude = m_limitSoftness * ((angVelB - angVelA).dot(axisA) * m_relaxationFactor + m_correction * (real_t(1.) / p_step) * m_biasFactor) * m_limitSign;

The m_limitSoftness member is not used at all by the hinge. This would make it work in the same way as the slider joint.
As an additional note, I also have the feeling that this line:
velrelOrthog *= (real_t(1.) / denom) * m_relaxationFactor;

was maybe a typo and should have been:
velrelOrthog *= (real_t(1.) / denom) * relaxation;

Just like in the next code block. It seems strange that the relaxation for the joint limit is also used to fix the error in one but not the other axis orthogonal to the hinge axis.
Steps to reproduce
None
Minimal reproduction project
None, I'm just browsing through the code for the godot-jolt plugin.

Don't think this is relevant anymore.


New repository for TypeScript grammar

The Microsoft TypeScript grammar for Sublime Text is now hosted at Microsoft/TypeScript-TmLanguage. See Microsoft/TypeScript-Sublime-Plugin#610 for details.

I had to whitelist the MIT license the new repository uses.
I don't understand the new error... @Alhadis Any idea?

Which new error? This?

The command "script/licensed verify" exited with 1.
nothing changed, not updating cache
Done. Your build exited with 1.

Yes. I'm guessing it has to do with the warning above:
Warnings:

vendor/licenses/grammar/TypeScript-TmLanguage.txt:

  - license needs reviewed: other.

287 dependencies checked, 1 warnings found.

Oh, I've seen that numerous times... I thought you kicked up something else.
No clue, sir. :( The license-recognition stuff is well outside my field of understanding.

This is going to be the same issue we saw in https://github.com/github/linguist/pull/3782 and should also be resolved when I update licensed (once the pending PR is merged).
:+1: to manually setting the license to MIT in the license file.

:+1: to manually setting the license to MIT in the license file.

Done.


update download url for openjkd

It seems google have changed where they store openJKD.
This has broken the download in configure.js
Here is a fix.
Thanks!

Merged with #51


pkg/repro: repro succeeded but failed

Looking at this bug:
https://syzkaller.appspot.com/bug?extid=3c249314730382052089
It does not have a reproducer.
But looking at the first failed repro log:
https://syzkaller.appspot.com/text?tag=ReproLog&x=137576e8580000
It actually succeeded with the right crash:
final repro crashed as (corrupted=false):
------------[ cut here ]------------
VFS: brelse: Trying to free free buffer
WARNING: CPU: 1 PID: 19 at fs/buffer.c:1229 __brelse fs/buffer.c:1229 [inline]
WARNING: CPU: 1 PID: 19 at fs/buffer.c:1229 brelse include/linux/buffer_head.h:324 [inline]
WARNING: CPU: 1 PID: 19 at fs/buffer.c:1229 __invalidate_bh_lrus fs/buffer.c:1498 [inline]
WARNING: CPU: 1 PID: 19 at fs/buffer.c:1229 invalidate_bh_lru+0x102/0x1b0 fs/buffer.c:1511
Modules linked in:
CPU: 1 UID: 0 PID: 19 Comm: rcu_exp_gp_kthr Not tainted 6.12.0-next-20241122-syzkaller #0
Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 09/13/2024
RIP: 0010:__brelse fs/buffer.c:1229 [inline]
RIP: 0010:brelse include/linux/buffer_head.h:324 [inline]
RIP: 0010:__invalidate_bh_lrus fs/buffer.c:1498 [inline]
RIP: 0010:invalidate_bh_lru+0x102/0x1b0 fs/buffer.c:1511

Yet the repro is considered failed and the bug does not have a repro.

In the end, it crashed as
program crashed: WARNING in invalidate_bh_lru

which is different from WARNING in invalidate_bh_lrus_cpu
So the reproducer went to https://syzkaller.appspot.com/bug?extid=9743a41f74f00e50fc77


Tab is shown even when it's disabled

when I click to a disabled tab it shouldn't be shown, however it shows.

@yodanielo What version of Bootstrap are you using? What browser are you using?

Google Chrome V47.0
Windows 7 Professional
Bootstrap v3.3.5

@yodanielo Please upgrade to v3.3.6 and re-test.

Hey there!
We're automatically closing this issue since the original poster (or another commenter) hasn't yet responded to the question or request made to them 14 days ago. We therefore assume that the user has lost interest or resolved the problem on their own. Closed issues that remain inactive for a long period may get automatically locked.
Don't worry though; if this is in error, let us know with a comment and we'll be happy to reopen the issue.
Thanks!
(Please note that this is an automated comment.)


CORS problems

I just tried to update from imgCrop to this newer version but the resulting images cause CORS problems when trying to upload them to my API which has CORS enabled for all endpoints and works flawlessly with result images from the old library.
Not sure what the origin of this could be, but I want to migrate to this newer version since it solves a small problem that I have with the old one where sometimes it generates 1px wide black bar on the right sight on the resulting image.
Any ideas?

Yes, I'm experiencing the same problem. My server-side CORS middleware works perfectly as well, but when loading the image placeholder into the cropper it just gives a CORS error. Very rare.

and previous version don't have this issue?

CORS is not a problem of the cropper.
It's a problem of the server serving the image. The cropper is hosted on a certain domain that is not trusted or similar as the resource where the image is hosted on.
We can't help this what so ever.


Syscall decoder

For issue #209

I added some changes. Let me know what you think.

It seems fine to me.


Rework the readme to make it clear it is for contributors.

r: @addyosmani @surma
Fixes #773

lgtm

Thanks! I'll merge this now beacause we need to get the readme in a better stat. If there are further comments I'll handle them in a speparate PR.


Identify feature gaps with vanilla Dependency-Track

We refactored and restructured a lot of things in the API server to make it work with Hyades.
It is possible that by doing this we removed or broke certain features (currently: policy evaluation is not happening).
This macro issue is for identifying more of those gaps and documenting them.
Areas:

Vulnerability analysis after BOM upload
Policy evaluation after BOM upload
Recurring daily vulnerability analysis
Recurring policy evaluation
Metrics calculation
Repository meta analysis
Vulnerability mirroring
Notification publishing

Test with actual external services (MS Teams)

...

Known Gaps

[ ] NEW_VULNERABLE_DEPENDENCY notification doesn't work

Must be evaluated after vulnerability scan after BOM upload completed

[ ] Policy evaluation is not triggered

Will be addressed with #341

[ ] Completion of repository meta analysis can not be tracked

DT 4.8.0 will include various policy conditions that make this necessary in order to work

[ ] Mirroring and ad-hoc vulnerability analysis with VulnDB is not implemented

See #286

[ ] Mirroring of GitHub Advisories is not implemented

The library we use for NVD mirroring supports GHSA, too


A new action, debugging, and improved logging

Added an action to add a AWS IAM user to a group. Use this in combination with an AWS IAM deny policy to effectively 'quarantine' users that don't match your Dome9 compliance and governance rule conditions. The new action uses more descriptive logging for busy environments by identifying which function was called in the log (good for multifunction operations). A new debugging action helps identify calls & AWS entity information to aid in building your own actions.

Patrick - I suggest adding somewhere (in docs / README / JS comment ) the expected IAM policy of the 'quarantine group'.
Still, I'm merging this one


make RunicArgs handle nested parameters

Nested object parameters don't get runified with RunicArgs:
type Foo = { a?: number, b?: { c?: { d?: number } } }
// RunicArgs<X, Foo> becomes
{
  a:  number | Rune<number | undefined, X> | undefined,
  b?: { c?: { d?: number } } | Rune<{ c?: { d?: number } }, X> | undefined
}

This makes it harder both for the consumer of a Runified function and of the creator:
As a user, I have to add a pyramid of Rune.rec if I want to feed a Rune value to nested parameters
Nfts
  .create({
-   config: Rune.rec({
      settings: DefaultCollectionSetting.allOff,
-     mintSettings: Rune.rec({
        mintType: MintType.Issuer(), // Rune<MintType>
        defaultItemSettings: DefaultItemSetting.allOn,
      }),
    }),
    admin: alice.address,
  })

As the function creator, it's hard to deal with possible undefined values in nested parameters
const { b }  = RunicArgs.resolve(props)
const d = b?.access("c").unhandle(undefined).access("d") // error: Type 'string' is not assignable to type 'Rune<never, never>' (Type "c" is not found on undefined)

The Rune.rec thing is also confusing for new users who don't know why they need to runify their already rune parameters to consume a pallet.
We could create an optional RunicArgs.rec to handle nested object params, and it should be used by default in pallet libraries.

unrelated question... should we be naming our parameter types as Props while we pass them through RunicArgs? Should it be RunicProps or Args? :)

@tjjfvi @harrysolovay I can implement this, just need some thoughts on how it should look (is it a good idea? is there something I'm missing?)

I don't think we want to take this on considering that most users will use factories from codegen (in which case, this shouldn't pose any difficulty).


Syntax error in > 1.2.0

I get a syntax error when building my code.
SyntaxError: Cannot declare a parameter named 'request' in strict mode
  at /bower_components/jquery.terminal/js/jquery.terminal.js:3310

The same error is thrown with the minified file as well.

Fixed in version 1.4.1 with npm I'll wait until you test.
Where do you see this error, (what browser/os)? I don't see it in chrome and lint didn't report it. Maybe this should reported to eslint (or maybe I don't have rule for that).

yep it is fixed.
I use jshint and jscs. But this error is caught by PhantomJs. I use PhantomJs 2.1.1.


Ghost (Handlebars) template support/fix

I'm submitting a bug report and/or submitting a feature request
I have a development environment utilizing Gulp and Webpack together. The sole purpose of using Gulp is to serve up a Ghost server for theme development using ReactJS (which I have working). The purpose for Webpack is to compile and serve my dependencies for my project. As of right now, I cannot load the bundled .js file within my test theme. I'm curious if this is a Webpack issue by itself, that it either can't load .hbs files or something within my configuration is messed up.
Webpack version:
1.13.0
Please tell us about your environment:
OSX El Capitan
Current behavior:
Ghost server loads fine, but when I switch to the theme I'm developing, the page errors out saying it can't find my bundled .js file with a 404. I'm thinking:

I don't have Webpack configured to serve .hbs files correctly.
I do have Webpack configured to serve .hbs files correctly, but my configuration in general is messed up.
This is an impossibility for Webpack at the moment (hence the added feature request 😉)

Expected/desired behavior:
Load bundled .js within served .hbs file for a Ghost theme.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem along with a gist/jsbin of your webpack configuration.

I've provided my project for an example. Just follow the super simple setup and you're good to go.
Project link
My-Project
|--/gulp
   |--/dev-tasks              //individual dev gulp tasks
      |-ghost.js              //Initializes Ghost config, run's sequence of tasks for webpack-dev-server
      |-init.js               //triggers symlinking of Ghost theme
      |-sequence.js           //maps out sequence of Gulp tasks
      |-styles.js             //style compiling
      |-symlink.js            //links local theme with Ghost configuration in /node_modules/
      |-task-runner.js        //runs Gulp tasks
      |-webpack-build-dev.js  // Loads the dev compiler
      |-webpack-build-prod.js // Not worrying about this one...
      |-webpack-dev-server.js // Runs the devServer
   |-ghost-dev-config.js      //configures Ghost environment
|--/public
   |-default.hbs
   |-index.hbs
   |-package.json
|--/src
   |--/containers
      |-App.js
      |-Home.js
      |-StyleGuide.js
      |-Uploader.js
   |--/styles
      |-main.sass
   |-router.js
|-gulpfile.js
|-package.json
|-webpack.dev.config.js
|-webpack.prod.config.js

What is the motivation / use case for changing the behavior?
To support the loading of bundled .js file within .hbs files for use of templates. I'm aware of the HTMLWebpackPlugin for trying to load template files, then have Webpack spit out a desired file name and path that is specified, but I don't think this is doing me any justice.
Language: [ES6/7, ReactJS, Handlebars]

Any luck with this? webpack can't load Handlebars without some extra configuration.

@bebraw I haven't touched on this project for a while because my issue still stands.


Page reduction

Guys, we need to work on page reduction and "STATS" wording optimization, cuz this professional report ACTUALLY targets "non-statistician".

(page 4 on Projectintro-1.pdf)
We can work on transform that after tomorrow's crazy dues, no big deal. @andrewkroening @echodpp
@poojakabber7

Thanks @Yer1k , it looks like we're good to go!


vaultwarden-mysql: 1.25.2 -> 1.26.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/dani-garcia/vaultwarden/releases.
meta.description for vaultwarden-mysql is: "Unofficial Bitwarden compatible server written in Rust"
meta.homepage for vaultwarden-mysql is: "https://github.com/dani-garcia/vaultwarden"
meta.changelog for vaultwarden-mysql is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.26.0 with grep in /nix/store/p6q8v7v16xhsdznv2pkigv0w2g743v7a-vaultwarden-1.26.0
found 1.26.0 in filename of file in /nix/store/p6q8v7v16xhsdznv2pkigv0w2g743v7a-vaultwarden-1.26.0

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

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
nixos-install-tools
tests.nixos-functions.nixos-test
vaultwarden
vaultwarden-mysql
vaultwarden-postgresql
vaultwarden-sqlite

Instructions to test this update (click to expand)

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

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

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

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

4 packages built:

vaultwarden (bitwarden_rs)
vaultwarden-mysql (bitwarden_rs-mysql)
vaultwarden-postgresql (bitwarden_rs-postgresql)
nixos-install-tools

Maintainer pings
cc @msteen @ivan for testing.

duplicate of https://github.com/NixOS/nixpkgs/pull/196536


[ERROR] MIPROv2 + multi hop module results in IndexError

MIPROv2 with multiple hop module results in error IndexError: list index out of range.
Using the latest version: dspy-ai==2.4.13
Error details:
IndexError                                Traceback (most recent call last)
Cell In[185], line 29
     19 lm = llm_compile
     20 optimizer_MIPROv2 = MIPROv2(
     21     prompt_model=lm,
     22     task_model=lm,
   (...)
     27     track_stats=False,
     28 )
---> 29 optimized_program_MIPROv2 = optimizer_MIPROv2.compile(
     30     ContextVerdict(),
     31     trainset=train_HotPotQA,
     32     valset=validation_HotPotQA,
     33     num_batches=batches,
     34     max_bootstrapped_demos=2,
     35     max_labeled_demos=2,
     36     requires_permission_to_run=False,
     37     eval_kwargs=eval_kwargs,
     38 )

File /opt/conda/lib/python3.11/site-packages/dspy/teleprompt/mipro_optimizer_v2.py:291, in MIPROv2.compile(self, student, trainset, valset, num_batches, max_bootstrapped_demos, max_labeled_demos, eval_kwargs, seed, minibatch, program_aware_proposer, requires_permission_to_run)
    289 proposer.use_instruct_history = False
    290 proposer.set_history_randomly = False
--> 291 instruction_candidates = proposer.propose_instructions_for_program(
    292     trainset=trainset,
    293     program=program,
    294     demo_candidates=demo_candidates,
    295     N=self.n,
    296     prompt_model=self.prompt_model,
    297     T=self.init_temperature,
    298     trial_logs={},
    299 )
    300 for i, pred in enumerate(program.predictors()):
    301     instruction_candidates[i][0] = get_signature(pred).instructions

File /opt/conda/lib/python3.11/site-packages/dspy/propose/grounded_proposer.py:302, in GroundedProposer.propose_instructions_for_program(self, trainset, program, demo_candidates, prompt_model, trial_logs, N, T, tip)
    299         if pred_i not in proposed_instructions:
    300             proposed_instructions[pred_i] = []
    301         proposed_instructions[pred_i].append(
--> 302             self.propose_instruction_for_predictor(
    303                 program=program,
    304                 predictor=predictor,
    305                 pred_i=pred_i,
    306                 prompt_model=prompt_model,
    307                 T=T,
    308                 demo_candidates=demo_candidates,
    309                 demo_set_i=demo_set_i,
    310                 trial_logs=trial_logs,
    311                 tip=selected_tip,
    312             ),
    313         )
    314 return proposed_instructions

File /opt/conda/lib/python3.11/site-packages/dspy/propose/grounded_proposer.py:349, in GroundedProposer.propose_instruction_for_predictor(self, program, predictor, pred_i, prompt_model, T, demo_candidates, demo_set_i, trial_logs, tip)
    347 with dspy.settings.context(lm=prompt_model):
    348     prompt_model.kwargs["temperature"] = T
--> 349     proposed_instruction = instruction_generator.forward(
    350         demo_candidates=demo_candidates,
    351         pred_i=pred_i,
    352         demo_set_i=demo_set_i,
    353         program=program,
    354         data_summary=self.data_summary,
    355         previous_instructions=instruction_history,
    356         tip=tip,
    357     ).proposed_instruction
    358 prompt_model.kwargs["temperature"] = original_temp
    360 # Log the trace used to generate the new instruction, along with the new instruction itself

File /opt/conda/lib/python3.11/site-packages/dspy/propose/grounded_proposer.py:201, in GenerateModuleInstruction.forward(self, demo_candidates, pred_i, demo_set_i, program, previous_instructions, data_summary, max_demos, tip)
    199     matches = re.findall(pattern, init_content, re.MULTILINE)
    200     modules = [match[0].strip() for match in matches]
--> 201     module_code = modules[pred_i]
    203 module_description = self.describe_module(
    204     program_code=self.program_code_string,
    205     program_description=program_description,
   (...)
    208     max_depth=10,
    209 ).module_description
    211 # Generate an instruction for our chosen module

IndexError: list index out of range

Script:
#!/usr/bin/env python
# coding: utf-8

import os
import dspy
from dspy.evaluate import Evaluate

os.environ["OPENAI_API_KEY"] = "abc"

colbertv2 = dspy.ColBERTv2(url='http://<IP_ADDRESS>:2017/wiki17_abstracts')
llm_compile = dspy.OpenAI(model='google/gemma-2-9b-it', api_base="http://<IP_ADDRESS>:8000/v1/", max_tokens=100, stop='\n\n')

dspy.settings.configure(rm=colbertv2, lm=llm_compile)

# based on: https://github.com/stanfordnlp/dspy/blob/main/skycamp2023.ipynb

train_dataset = [
    ('Kevin Greutert directed the 2009 movie featuring Peter Outerbridge as William Easton.', "True"),
    ('The heir to the Rockefeller family fortune sponsored the Foxcatcher wrestling team.', "False"),
    ('Audie Murphy, the star of To Hell and Back, was born in 1925.', "True"),
    ('The first book of Gary Zukav received the Pulitzer Prize.', "False"),
    ('The Killing Season, a documentary about the Gilgo Beach Killer, debuted on Netflix.', "False"),
    ('John Braine, an English author, wrote "Room at the Top".', "True"),
    ('Butch Vig produced the album that included a re-recording of "Lithium".', "True")
]

dev_dataset = [
    ('E. L. Doctorow has a broader scope of profession than Julia Peterkin.', "True"),
    ('Right Back At It Again contains lyrics co-written by the singer born in Gainesville, Florida.', "True"),
    ('The party of the winner of the 1971 San Francisco mayoral election was founded in 1854.', "False"),
    ('Anthony Dirrell is the brother of Andre Dirrell, a super middleweight title holder.', "True"),
    ('The sports nutrition business established by Oliver Cookson is based in Kent, UK.', "False"),
    ('The actor who played roles in First Wives Club and Searching for the Elephant was born on February 13, 1980.', "True"),
    ('Kyle Moran was born in the town on the Thames River.', "False"),
    ('The actress who played the niece in the Priest film was born in Surrey, England.', "True"),
    ('The movie in which the daughter of Noel Harrison plays Violet Trefusis is called Portrait of a Marriage.', "True"),
    ('The father of the Princes in the Tower was born in 1445.', "False"),
    ('The River Tyne is near the Crichton Collegiate Church.', "True"),
    ('Renault purchased the team Michael Schumacher raced for in the 1995 Monaco Grand Prix in 2000.', "True"),
    ('André Zucca was a French photographer who worked with a German propaganda magazine published by the Luftwaffe.', "False")
]

train = [dspy.Example(statement=statement, answer=answer).with_inputs('statement') for statement, answer in train_dataset]
dev = [dspy.Example(statement=statement, answer=answer).with_inputs('statement') for statement, answer in dev_dataset]

from dsp.utils import deduplicate

class CheckStatementFaithfulness(dspy.Signature):
    """Verify that the statement is based on the provided context."""

    context = dspy.InputField(desc="facts here are assumed to be true")
    statement = dspy.InputField()
    verdict = dspy.OutputField(desc="True/False/Irrelevant indicating if statement is faithful to context")

class GenerateSearchQuery(dspy.Signature):
    """Write a simple search query that will help retrieve info related to the statement."""
    context = dspy.InputField(desc="may contain relevant facts")
    statement = dspy.InputField()
    query = dspy.OutputField()

class ContextVerdict(dspy.Module):
    def __init__(self, passages_per_hop=3, count=3):
        super().__init__()
        self.generate_query = [dspy.ChainOfThought(GenerateSearchQuery) for _ in range(count)]
        self.retrieve = dspy.Retrieve(k=passages_per_hop)
        self.generate_verdict = dspy.ChainOfThought(CheckStatementFaithfulness)
        self.count = count

    def forward(self, statement):
        context = []
        for hop in range(self.count):
            query = self.generate_query[hop](context=context, statement=statement).query
            passages = self.retrieve(query).passages
            context = deduplicate(context + passages)

        verdict = self.generate_verdict(context=context, statement=statement)
        pred = dspy.Prediction(answer=verdict.verdict, rationale=verdict.rationale, context=context)
        return pred

# reference: https://github.com/stanfordnlp/dspy/blob/main/examples/multi-input-output/beginner-multi-input-output.ipynb

from dspy.teleprompt import MIPROv2

NUM_THREADS = 24
metric_EM = dspy.evaluate.answer_exact_match
eval_kwargs = {"num_threads": NUM_THREADS, "display_progress": False, "display_table": 0}

evaluate_MIPROv2 = Evaluate(
    devset=dev,
    metric=metric_EM,
    **eval_kwargs
)

n = 1 #10  # The number of instructions and fewshot examples that we will generate and optimize over
batches = 1 #30  # The number of optimization trials to be run (we will test out a new combination of instructions and fewshot examples in each trial)
temperature = 1  # The temperature configured for generating new instructions

lm = llm_compile
optimizer_MIPROv2 = MIPROv2(
    prompt_model=lm,
    task_model=lm,
    metric=metric_EM,
    num_candidates=n,
    init_temperature=temperature,
    verbose=False,
    track_stats=False,
)
optimized_program_MIPROv2 = optimizer_MIPROv2.compile(
    ContextVerdict(),
    trainset=train,
    valset=dev,
    num_batches=batches,
    max_bootstrapped_demos=2,
    max_labeled_demos=2,
    requires_permission_to_run=False,
    eval_kwargs=eval_kwargs,
)

Tried both local model google/gemma-2-9b-it and gpt-4o-mini from OpenAI, same error.
I'm able to avoid the error by adding one line inside the module ContextVerdict:
# self.generate_query = dspy.ChainOfThought(GenerateSearchQuery)  # IMPORTANT: solves error `list index out of range`
self.generate_query = [dspy.ChainOfThought(GenerateSearchQuery) for _ in range(max_hops)]

Ah thanks for the note, tagging @XenonMolecule

Hi @etwk,
Try dspy.experimental - dspy.settings.configure(rm=colbertv2, lm=llm_compile, experimental=True).
This error happens as MIPRO is summarizing DSPy code and is dependent on certain module structure parsing to retrieve the right code. As certain new releases point towards the experimental setup (which will be comprehensively updated in future releases), I would advise checking if enabling experimental mode fixes this error.

Getting the same error here with GPT-4o. @etwk Have you found a workaround or a solution yet?

Getting the same error here with GPT-4o. @etwk Have you found a workaround or a solution yet?

Yes, please find it at the end of the first post.
Add one comment line has bypassed this issue for me:
# self.generate_query = dspy.ChainOfThought(GenerateSearchQuery)  # IMPORTANT: solves error `list index out of range`

+1

I also encountered the same error when using MIPRO V2. Is there any new progress?

Ill fix this today or tomorrow @okhat

I think this is now fixed


[a11y] Add semantics: button to bottom navigation bar items and dropdown menu items

issue: https://github.com/flutter/flutter/issues/117997
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] I followed the breaking change policy and added Data Driven Fixes where supported.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Can we get this landed?

@hangyujin FYI it looks like there are some Google testing failures here

I'm adding a g3 fix for Google testing failures


fix: :fire: Allow automation to run from manual triggers.

Removing check `github.event_name == 'push'

@whizzzkid  feel free to merge when you have time to eyeball if it works as expected :)


Added canRotate and aspectRatio attribute

Two attributes have been added to control whether rotation is allowed, and shapes that support a fixed aspect ratio

Reformat

review

There still seem to be formatting issues. Did you run dart format on the dir?

There still seem to be formatting issues. Did you run dart format on the dir?
When I ran dart format, the format of the old code was also changed. It seems that our formatting rules are inconsistent.

I have added travis with extra pr checks. Can you fix the conflicts & trigger another build (new commit)

I have added travis with extra pr checks. Can you fix the conflicts & trigger another build (new commit)

I'm sorry, but I recently changed jobs. So no time to fix these issues, I'll fix it later.

Any progress on this pr? @demonskp

@demonskp any progress on this pr?

Any update @demonskp ??


Add a getter for the inner tracing-subscriber's Layer

Before this commit, the only way to effectively consume the layer
created with this crate is to use the subscriber() method that wraps
the layer into a Registry. This might not always be desired by end
users of this crate, so this commit adds a new way to retrieve the inner
tracing's Layer so one can add it into an existing Registry, for
instance.

Thanks for the review!

I'm also honestly tempted to remove the wrapper type altogether and just provide the MakeWriter and helper methods to create a sensible fmt::Layer

I don't have a lot of perspective about how other tracing implementations are implemented, but one idea that comes to mind would be to use a builder pattern, and have the build() function would return the fmt::Layer? This is pretty much what the crate is already doing, and it's probably a matter of just renaming things (inner() -> build(), Layer -> LayerBuilder or just Builder, etc.).


Weights File (Weights.pb.gz)

I'm trying to add this weights file (https://www.patreon.com/posts/leelenstein-15-0-38164065) to the file lc0goorm
My issue is that is says "Yowza that's a big file.  Try something smaller the 25mb."  Does that mean I can't use any weights file that are  over 25mb?

Its the Leelenstein  (20 blocks x 256 filters)

Its the Leelenstein  (20 blocks x 256 filters)

Another question.  If i put another weights file would it automatically overwrite the default weights file (weights.pb.gz)

Another question.  If i put another weights file would it automatically overwrite the default weights file (weights.pb.gz)

Then how do I delete the old one?

Then how do I delete the old one?


Add new features to AVL; Add new syntax for analysis-specific options in geometry stack

This has been successfully merged into develop! Closing PR

The way you simplified analysis_specific_options is very elegant! I didn't know about the dict.update() method. But there is no longer any check for incorrect user input keys. Not sure if this was intended, but I think it would still be a nice check to prevent the user from getting the name of an analysis-specific option key wrong and thinking the option has been applied when it has really been discarded.
The way you simplified the control surfaces to one deflection using individual control surface gains is also very clever. However, it does eliminate a couple functionalities:

The user cannot manipulate the control surfaces individually at runtime within AVL, so the output geometry files cannot stand on their own. The user would have to manually change the gains of each control surface in the geometry file or change the deflections in Python.
AVL's trimmed flight calculations will not work if multiple control surfaces are present because they cannot be individually manipulated to trim different moments.
AVL will not give any information on the derivatives of aero coefficients w.r.t. individual control surface deflections.

I see why you simplified the control surface handling so that you don't have to keep track of control surface information across both the geometry and keystroke files, but do you think there's a reasonable way to retain these functionalities?

Hey @adamwass!
Validation of analysis_specific_options keys is now added and working with asb.AVL, as seen in the 4 latest commits on develop (validation code here: 16bb419fcd2dc24d96ce109338be3d5596bb362e, docs here: e20f76c07264459d853bbf3ad8d5621cb42590ab, 7732dd50f1845cebc21c2d94547e297aae098d00, edits to AVL.py here: 0f69ddb5beb4caa24682cda1ab9abb17bbaef887).
Re: control deflections in AVL:
That's a good point... it's a bit of a tricky issue, because AeroSandbox doesn't track groups of control surfaces together as AVL does - and as a conceptual aircraft design tool, that seems perhaps outside of scope?
As an alternative to the all_deflections implementation (where the "deflection" of all surfaces is 1, and the gain is their actual deflection), we could perhaps completely separate all control surfaces by procedurally generating unique names:
control_surface_name_in_avl = f"wing{wing_index}xsec{xsec_index}surf{control_surface_index}"

Does that seem like a more useful implementation perhaps?

Thanks for feedback @peterdsharpe!
Perhaps the finer points of control surfaces are beyond the scope of a conceptual design tool. However, if you do want to go down this rabbit hole, I have a couple ideas :). And in fact, these ideas could be generalized to the handling, identifying, and grouping of wings and fuselages as well for surface or fuselage-wise force reports, grouping multiple surfaces into one virtual surface, etc.
Identification Schemes in AVL
AVL has 3 different identification schemes for aircraft components:

Lists - wings (surfaces in AVL terminology) and fuselages (bodies in AVL terminology) are identified using lists, sequenced by the order in which the surfaces/bodies were defined in the geometry file. Although each wing/fuselage also has a user-defined name, this name does not have to be uniquely identifying. For example, if I make an AVL geometry file with two surfaces, both called "Wing", the surface force report would look like this:

 Surface Forces (referred to Sref,Cref,Bref about Xref,Yref,Zref)
 Standard axis orientation,  X fwd, Z down

     Sref =   6.000       Cref =    1.0000   Bref =    6.0000
     Xref =      0.0000   Yref =    0.0000   Zref =    0.0000

 n      Area      CL      CD      Cm      CY      Cn      Cl     CDi     CDv
 1     3.000  0.1947  0.0040 -0.0421  0.0000 -0.0027 -0.0428  0.0040  0.0000   Wing
 2     3.000  0.1947  0.0040 -0.0421  0.0000  0.0027  0.0428  0.0040  0.0000   Wing (YDUP)
 3     0.500  0.0164  0.0008 -0.0672  0.0000 -0.0000 -0.0012  0.0008  0.0000   Wing
 4     0.500  0.0164  0.0008 -0.0672  0.0000  0.0000  0.0012  0.0008  0.0000   Wing (YDUP)

where each "Wing" surface is listed in the order defined in the geometry file.

Note: this scheme does not allow grouping.

Unique ID number - although surfaces are identified using lists, they can be grouped together using the COMPONENT keyword and associated component ID number. All surfaces with the same component ID number will be treated as one composite virtual surface for flow solving.

Unique ID string - control surfaces each have a user-defined ID string, and all control surfaces with the same ID string are treated as one control surface (y-duplicated control surfaces are also considered to be part of the same control surface), allowing control surfaces to span multiple sections or even wings. Like the unique ID number, this scheme allows for grouping.

Identification Schemes in AeroSandbox
Any of these methods could be used to identify and track control surfaces, wings, or fuselages in AeroSandbox.
The current implementation of the ControlSurface class is probably closest to a list, as each instance is identified uniquely only by its order within the Airplane>Wing>XSec hierarchy and its memory address, and they cannot be combined. You translated this into AVL using the unique ID string scheme (all_deflections) to create one virtual control surface. However, any of these ID schemes could be used to modify the desired behavior of and user interactions with control surfaces.
For example, here is a simple example for a unique ID number scheme implemented using a class attribute of the ControlSurface class:
class ControlSurface(AeroSandboxObject):
    id = 0 # class attribute
    def __init__(self):
        self.id = __class__.id
        __class__.id += 1

...
from aerosandbox.geometry.wing import ControlSurface

class Airplane(AeroSandboxObject):
    def __init__(self):
        ControlSurface.id = 0 # reset ControlSurface id class attribute every time a new Airplane is initialized.

Note that this scheme would not allow for grouping multiple control surfaces into one.
Anyway, I'm just brainstorming and throwing out ideas. Certainly, any identification scheme would add additional complexity tot he code. Let me know if you think any of these would be worth pursuing.

These are some interesting things to think about - thank you for writing this up!
Re: the proposed scheme:
I'm a bit sketched out by the idea of tracking id as a class attribute of ControlSurface in the manner shown - it seems easily broken, and difficult to debug when it does break. For example, here's a illustrative failing example:
import aerosandbox as asb
import aerosandbox.numpy as np

aileron = asb.ControlSurface(
    ...
)  # aileron.id == 0

wing_root = asb.WingXSec(
    ...,
    control_surfaces=[
        aileron
    ]
)

wing_tip = asb.WingXSec(...)

wing = asb.Wing(
    ...,
    name="Main Wing",
    xsecs=[wing_root, wing_tip]
)

airplane = asb.Airplane(
    ...,
    wings=[wing]
)

### Oh wait, I forgot to add the tail, let me do that
airplane.wings.append(
    asb.Wing(
        name="Horizontal Stabilizer",
        xsecs=[
            asb.WingXSec(
                control_surfaces=[
                    asb.ControlSurface(
                        ...,
                        name="Elevator",
                    )  # elevator.id == 0
                ]
            ),
            asb.WingXSec(...)
        ]
    )
)

So, now you have instances of a class (ControlSurface) whose attributes depend on when they were defined with respect to initialization of the separate Airplane class.
And, in this example, both the aileron and elevator would have id == 0, unintentionally losing uniqueness.

Perhaps a different way to group surfaces would be to check for equality between surfaces (either using a is b, i.e. same object in memory, or by overloading a __eq__() method in ControlSurface that checks each attribute), and then group surfaces in AVL iff they are "equal" in some sense. So, you could define a grouped surface (here, two aileron sections that move together) like so:
import aerosandbox as asb
import aerosandbox.numpy as np

aileron = asb.ControlSurface(
    name="Aileron",
    ...,
)

airplane = asb.Airplane(
    wings=[
        asb.Wing(
            xsecs=[
                asb.WingXSec( # Root
                    ...,
                    control_surfaces=[aileron]
                ),
                asb.WingXSec( # Mid
                    ...,
                    control_surfaces=[aileron]
                ),
                asb.WingXSec( # Tip
                    ...,
                )
            ]
        )
    ]
)


Main content area (task result, creation) should use the whole space available

Please adjust the main content area (e.g. task result), so that it uses whole width and height of the screen. See below, it looks ugly and causes not all value are visible (would be even worse for csv files with more column).

I dare to object it doesn't look ugly and it was an intention (originally I created the template for another website), but I guess everybody has his/her own preferences.
Nevertheless I corrected it. The problem is the footer area, which is not always positioned at the bottom (specifically, when the page occupies less than 100% of the screen height). It seems that either the bootstrap framework has no implicit support for it, or its solution simply doesn't work on our template. Solving this would require non-trivial rewriting of certain css/less scripts + many html changes. So I propose we leave it as it is now, or remove the footer completely.

I dare to object it doesn't look ugly and it was an intention (originally I created the template for another website), but I guess everybody has his/her own preferences.

I am sure Tomáš meant no harm. :-)

The problem is the footer area, which is not always positioned at the bottom (specifically, when the page occupies less than 100% of the screen height). It seems that either the bootstrap framework has no implicit support for it, or its solution simply doesn't work on our template. Solving this would require non-trivial rewriting of certain css/less scripts + many html changes. So I propose we leave it as it is now, or remove the footer completely.

I see. It does not bother me that much. I will assign the issue to @tomas-knap and let him decide if it is acceptable, or it can be done later, or kept without change as it is.

Testing that in Safari, the footer seems to work in a way that it appears directly below the main content part (if it doe not occupy whole screen, it is in the middle of the screen). This is OK. No need to invest further time on that.

So it is solved.


Solve Timeout behaviour

Timeout behaviour
Since the Notification Component fires the timer when receive props and verify the (if) condition, it is necessary a clearTimeout to avoid not desired behaviour with different timers fired.
Add an extra condition to only apply if has same onDismiss so don't brake the Notification Stack behaviour.
See the images below for a visual example:
wrong

right

Example onClick function
Add the missed function to show its behaviour
Commet
Remove a duplicated comment that described a functional Component in a extended Class Component.

@elrumordelaluz Thanks for the PR.  I just scanned the changes quickly and everything looks good. I will take a closer look this evening and merge! :smile:

This has been released on npm as<EMAIL_ADDRESS>Thanks again for the PR!

no problem @pburtchaell, glad to help improving a little.


Exclude<T, U> type not working like Diff<T, U>

TypeScript Version:  2.8.0-dev.20180213
Search Terms:

Exclude
Diff

Code
// Old 'Diff' type, superseded since #21847 by new 'Exclude' builtin type
type Diff<T extends string, U extends string>
    = ({[P in T]: P } & {[P in U]: never } & { [x: string]: never })[T];

type NonFooKeys1<T extends object> = Diff<keyof T, 'foo'>;      // remove 'foo' using Diff
type NonFooKeys2<T extends object> = Exclude<keyof T, 'foo'>;   // remove 'foo' using Exclude

type Test1 = NonFooKeys1<{foo: 1, bar: 2, baz: 3}>; // Test1 is "bar" | "baz"
type Test2 = NonFooKeys2<{foo: 1, bar: 2, baz: 3}>; // Test2 is "foo" | "bar" | "baz"

Expected behavior:
Test2 should be "bar" | "baz", the same as Test1.
Actual behavior:
The key "foo" was not removed from keyof T when using the Exclude builtin type.
Related Issues:

Predefined conditional types #21847
Add Omit<T,K> and Diff<T,U> to lib.d.ts #19569

Of note, I'm getting some strange quick info. When defined as
type NonFooKeys<T> = Exclude<keyof T, 'foo'>;

I get
type NonFooKeys<T> = keyof T

Is this related to #21664 ? Exclude works when used directly for me.
type BarBaz = Exclude<keyof { foo: 1, bar: 2, baz: 3 }, 'foo'> // BarBaz = "bar" | "baz"

related #21862.

Duplicate of #21862.

The other issue is marked fixed now.


Ability to change unit type from Imperial to Metric

It would be nice to have the ability to change the unit type. My client use metric units for all his product.
There's only 4 files containing the keyword "imperial" so it should be an easy trick. I might try to do the change and make a pull request out of it.
Thanks for the plugin

This is a very good suggestion.

Hi makeMagik,
After reviewing the code related to this issue, I am not sure what exactly the need is. The plugin takes the dimensions of products. If the product dimensions are in metric, they will be converted to imperial and sent to the Flagship Shipping API. So metric dimensions still work.


Configurate alpha-pooling for ResNet ?

Dear @cvjena,
I am trying to replace average pooing with alpha-pooling of ResNet. I plan to use this modified ResNet to train ImageNet-1000. I notice that your modified VGG-16 has some "SpatialTransformer" stuff, which is out of my best knowledge. Here is my question:

Is it necessary  to use "SpatialTransformer" and "alpha-pooling"  at the same time?
alpha-pooling = SignedPower + CompactBilinear ? or alpha-pooling = SignedPower + CompactBilinear + SignedPower?

I am really appreciated if you have other guide
Thanks

Hi and thanks for your interest in the work.
The spatial transformer layer is only used to resize the input images during multi scale training. You can omit it by either training only single scale or by implementing resizing by other operations like average pooling layer.
For your second question: CompactBilinear has two inputs. The first one is the output of signed power and the second one is the unmodified feature. The output of CompactBilinear is passed to another signed power layer to calculate the signed square root as in previous work. The exponent is not learned but fixed to 0.5 in our experiments.
I am also preparing the script to work with resnet as well. It will be released soon.

If you use our ResNet 50 from https://github.com/cvjena/cnn-models/tree/master/ResNet_preact/ResNet50_cvgj  then you can append the following snipped after the last_relu layer instead of waiting for the script. However, please be awar that training from scratch is fairly difficult due to the L2 normalization.
layer {
  name: "alpha_power"
  type: "SignedPower"
  bottom: "layer_512_3_sum"
  top: "alpha_power"
  param {
    name: "alpha_power"
    lr_mult: 10
    decay_mult: 0
  }
  power_param {
    power: 1.0
  }
}
layer {
  name: "outer_product"
  type: "CompactBilinear"
  bottom: "layer_512_3_sum"
  bottom: "alpha_power"
  top: "outer_product"
  compact_bilinear_param {
    num_output: 8192
  }
}
layer {
  name: "root"
  type: "SignedPower"
  bottom: "outer_product"
  top: "root"
  param {
    lr_mult: 0
    decay_mult: 0
  }
  power_param {
    power: 0.5
  }
}
layer {
  name: "l2"
  type: "L2Normalize"
  bottom: "root"
  top: "l2"
}
layer {
  name: "fc8_ft"
  type: "InnerProduct"
  bottom: "l2"
  top: "fc8_ft"
  param {
    lr_mult: 1
    decay_mult: 1
  }
  param {
    lr_mult: 2
    decay_mult: 2
  }
  inner_product_param {
    num_output: 201
  }
}
layer {
  name: "loss"
  type: "SoftmaxWithLoss"
  bottom: "fc8_ft"
  bottom: "label"
  top: "loss"
}
layer {
  name: "Accuracy"
  type: "Accuracy"
  bottom: "fc8_ft"
  bottom: "label"
  top: "Accuracy"
  include {
    phase: TEST
  }
}```

Thank you @MarcelSimon . I will try this and get back to you. No problem to finetune since I have a norm softmax loss caffemodel already. But I think the CompactBilinear should have two input ? The above snipped has only one

I'm not quite sure what you mean :) CompactBilinear has two different inputs in the snippet, please check again
  ...
layer {
  name: "outer_product"
  type: "CompactBilinear"
  bottom: "layer_512_3_sum"
  bottom: "alpha_power"
...

Cool. I am misleaded by network visual software "netscope".
Ignore me :）

Yeah, it's missing in the netscope because the layer layer_512_3_sum is not defined in the snippet. No problem :)


Incremental model mocking fails with include_missing_columns set to true

We're running into problems when we try to mock our incremental models and have the include_missing_columns flag set to true. We get an "Object does not exist or not authorized error" (Snowflake).
It seems that when this flag is enabled the sql builder macro omits the {{ this }} reference, and does not mock the initial state of the incremental model.
This issue can be replicated with your own incremental model example in the README and enabling the above option.

Hi @strictness, could you please tell me which version are you using? I believe this happens in v0.2.8 but it's fixed in v0.2.9. Can you verify this?
Thank you!

Hi @psousa50, we are using v0.2.9.

Hi @psousa50 ,
It's never too late :) Yes, the inital issue was resolved.
However, we do get 001042 (XX000): SQL compilation error: Compilation memory exhausted. now since (I'm guessing) our models are rather complex.


Add collapse buttons to groups on demand

This addresses issue #15 (collapse layer groups)

I wonder if the Group property should be fold with values open or close. A Group with either no fold property or fold: false would not be foldable.

Yes good point!
On 21.10.18 18:59, Matt Walker wrote:

I wonder if the Group property should be |fold| with values |open| or
|close|. A Group with either no |fold| property or |fold: false| would
not be foldable.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/walkermatt/ol-layerswitcher/pull/106#issuecomment-431685295,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ40NF-Y1MMAJjgbB8b-U3-GFCCiBaoMks5unKgKgaJpZM4XcoRe.

I have incorporated the changes as proposed by @walkermatt but I can't get it looking how it's supposed to do. Maybe some CSS wizard out there who could have a look?

@swalterfub Didn't have a chance to test myself, but this would work nicely I think: https://www.w3schools.com/howto/howto_css_arrows.asp.

Nice @umbe1987 ! There is a new solution in my repo... no dependencies. (i still think the font-awsome solutions is a bit more fancy, but I can totally live with this one. Minimal-Style... Also now you can click on the layer label to (un-)collapse...

@swalterfub that is sweet! Thanks for providing this functionality ;)

@swalterfub this is looking good. I've a few changes I'd like to make, can you add me as a collaborator on your fork so I can commit/ push to your branch?

Current WIP can be tested here: swalterfub/ol-layerswitcher/master/examples/collapse-groups.html

@swalterfub, @umbe1987 I think I'm happy to merge/ release, anyone spot anything outstanding?

@swalterfub, @umbe1987 I think I'm happy to merge/ release, anyone spot anything outstanding?

@walkermatt I think everything is super nice!
@swalterfub @walkermatt thank you both for this neat introduction :collision: :collision: :collision:

Fine from my side as well! Thanks!

The classes close and open conflicts with classes in Bootstrap. If you change class names, i.e. closed and opened, it works well even when using bootstrap.

@kgiil thanks for the heads up, I've just tested the collapse-groups example with Bootstrap and the close class does mess our styles up :man_shrugging:
To avoid clashes we're going to have to be more specific, I'll give it some thought.

@walkermatt I saw your commit and I agree it makes perfect sense to prefix layer-switcher classes with a identifiable/"less-clashing" name. Any chance this PR will get merged anytime soon? Cheers.

Merged in a1061301dae8864d1a3ba058e7a8ffed32fd9c19
@swalterfub thanks for your work on this and thanks to @umbe1987 and @kgiil for your input :tada:

And released to npm as v3.2.0


test(extension): use split e2e for daily regression

Checklist

[ ] JIRA - <link>
[ ] Proper tests implemented
[ ] Screenshots added.

Proposed solution
Explain how does this PR solves the problem stated in JIRA ticket.
You can also enumerate different alternatives considered while approaching this task.
Testing
Describe here, how the new implementation can be tested.
Provide link or briefly describe User Acceptance Criteria/Tests that need to be met
Screenshots
Attach screenshots here if implementation involves some UI changes

Allure Report
allure-report-publisher generated test report!

processReports: ✅ test report for 12f0db40

passed
failed
skipped
flaky
total
result

Total
32
0
0
0
32
✅


No way to focus camera

There needs to be a way to programmatically focus the camera

Added in 3ae9f6d8c9fbede57cde5022aa69697806be005a.


feat: force audio mute support

@Gongreg added the query param https://github.com/rune/rune-games-sdk/pull/65/commits/ae5ef1c588d8de59ce3647d6201ab5d1c6373053


SceneGraph example

Magnum is a library similar in spirit to bgfx: https://github.com/mosra/magnum
They include a scene graph in the engine, so, i imagine it would be nice to have this kind of feature in bgfx as well.

I think of bgfx as a little more low level than that. Scene graphs are really implementation dependent and it'd be a little annoying to impose one on every user of the library.

@mosra should use just bgfx for rendering in Magnum and I can then point you to Magnum as scene graph example. :)
+1 what @kyle-emmerich said. Scene graph, Engine, etc. is out of scope of this library.

Heh :)


In autocomplete using renderOptions scroll position getting resets when new options are added

Steps to reproduce
CodesandBox link: https://codesandbox.io/p/sandbox/modest-fire-vh8534
Current behavior
When adding new options when render options prop is present the scroll position gets reset.
Expected behavior
When adding new options when render options prop is present the scroll position should not get reset.
Context
No response
Your environment
No response
Search keywords: In autocomplete using renderOptions scroll position getting resets when new options are added

Hey @kalaiy21, thanks for the report. I don't understand the issue's description, can you provide further details?

When adding new options when render options prop is present the scroll position gets reset.

How are you adding new options?
A video of the behavior would be helpful to understand as well.

Hi @DiegoAndai
I am adding the new options using listboxprops onScroll method when the scroll position reaches bottom.
CodeSandBox Link: https://codesandbox.io/p/sandbox/modest-fire-vh8534
Video Link : https://drive.google.com/file/d/1z6P2xvKLuKgH-cX6G3W0ZTbHlORL0S3y/view?usp=sharing

Thanks for the clarification @kalaiy21, I see the problem now. This is a duplicate of https://github.com/mui/material-ui/issues/40250, so I'm closing this one. I recommend adding an upvote (👍🏼) to that issue.
If what you're looking for is virtualization, have you looked into our virtualization example? https://mui.com/material-ui/react-autocomplete/#virtualization


chore: Remove KTX from basic map demo

This pull request reverts https://github.com/googlemaps/android-samples/pull/193 and https://github.com/googlemaps/android-samples/pull/188 to remove KTX from the basic map demo.
Per further discussion we're going to have these samples be pure Kotlin implementations without KTX.

Hmmm...some imports are missing from the revert. Will fix...

Ok, should be fixed in https://github.com/googlemaps/android-samples/pull/196/commits/f37137f15e625c543a78aed6744620d36fed5f35. Should be ready for review.


Fix startLogin being called on every configuration change with only one provider enabled

#549

LGTM, but can you quickly explain how to reproduce the intial bug?  I didn't really understand the report in #549

@samtstern Of course!
Using a single provider with or without smart lock eventually calls redirectToIdpSignIn -> an instance of IdpSignInContainer is created with setRetainInstance(false) (by default) -> on every screen rotation, startLogin gets called. The reason we don't see this bug anywhere else in FirebaseUI is because stuff like AuthMethodPickerActivity doesn't use IdpSignInContainer. IdpSignInContainer's only usage is for SignInDelegate.

@samtstern I just realized you might have been asking for a non-technical explanation for which the answer would be: start login with a single provider -> get to the provider's login section (eg not the smart lock thing) -> rotate the screen.

@SUPERCILEX yep the second one was what I wanted, confirmed the bug.

@SUPERCILEX https://travis-ci.org/firebase/FirebaseUI-Android looks like the build is failing with a lint error here but I don't get any on my machine.  Can you reproduce?

@samtstern Travis says it's because support lib 25.1.1 came out https://developer.android.com/topic/libraries/support-library/revisions.html#25-1-1
I'll create an update PR later today since it's just bug fixes (and I'll also disable GradleDependency to make our builds less flaky like in #542).


Content-Encoding: gzip

I wonder if it would make sense to support compressed requests, esp. for /rerank, where the query and document list could be many 1k or 2k chunks of text?  The incoming request could easily exceed 20 or 30k.  The http server does not appear to handle gzipped request bodies, if present.

Have you considered grcp protocol? If you fork the project and start building, thats something I potetntially would consider to pull in.
Questions:

I have never heard of gzip-requests - how does validation of requests (error 422 handling work?)
What kind of issues are you experienced when sending e.g. 2k requests? Why is this feature needed?
is sending 20-30k a good paragdim? When do you need it? Even with gpu you can encode around 200-1000 texts per second? I think this encourages a bad workload?

Does your FastAPI server accept gRPC?  I am using your docker container, behind nginx terminating TLS as a reverse proxy.  Nginx apparently can proxy gRPC.

content-encoding: gzip is pretty common.  All browsers will try to compress their request bodies if the server accepts.
I am not experiencing any issues with large requests.  They can be slower is all.
My RAG text chunks are about 1k.  My prompt, coming from Continue in vscode can be quite large (like an entire file.js).  I fetch like 20ish chunks from my vector database, then I want to re-rank.  Think this is too much data?

Here is an example of decompression middleware for FastAPI:
from fastapi import FastAPI, Request
from starlette.middleware.base import BaseHTTPMiddleware
import gzip

class GZipRequestMiddleware(BaseHTTPMiddleware):
    async def dispatch(self, request: Request, call_next):
        if 'content-encoding' in request.headers and request.headers['content-encoding'] == 'gzip':
            # Decompress the request body
            body = await request.body()
            decompressed_body = gzip.decompress(body)

            # Create a new request with the decompressed body
            scope = request.scope
            scope['body'] = decompressed_body
            request = Request(scope)

        response = await call_next(request)
        return response

app = FastAPI()

# Add the middleware to the app
app.add_middleware(GZipRequestMiddleware)

After that, request.body is used just as before.
I'll look into gRPC.  I need speed.

@peebles Thanks for the extensive example. https://stackoverflow.com/questions/43628605/does-the-zlib-module-release-the-global-interpreter-lock-gil-in-python-3 -> I assume this will not affect the GIL or performance. decompressed_body = gzip.decompress(body)
starlette integration seems elegant and without any extra dependencies at first glance!
Thoughts:

Could you do routing based on the json content?
Are you sure that the performance bottleneck is in sending/receiving the request? I think validation, tokenization, and especially forward pass of model will be much more compute heavy.
The response (embedding) should be all unique floats, with little pattern - json is kind of lossy, but I would consider adding a grcp server to be more elegant, and has more traction in the embedding community (https://github.com/huggingface/text-embeddings-inference?tab=readme-ov-file#grpc) grcp is not supported by fastapi

"routing based on json content" sounds intriguing but I am not sure I understand ...
I am not sure the performance bottleneck is network transfer, although the machine hosting Infinity is a big iron monster and executes reranking almost instantaneously.  Network transfer is probably my longer term concern.
Typically, client compression is not turned on unless the request payload goes above some threshold, like 1k or so, where the cost of transfer becomes greater than the cost of compression/decompression.

I am doing /rerank, where the input (to you) is a potentially large amount of text, and the output is a very small summary ... no floats, all text.  In /rerank, it may make sense to compress the input but not the output ... the output is too small.
As for "I assume this will not affect the GIL or performance. decompressed_body = gzip.decompress(body)", I don't know.  I come from more of a NodeJS background where everything is async.

What is the difference between Infinity and https://github.com/huggingface/text-embeddings-inference?

@peebles the most similar project out there. here are a couple of key differences.

Benchmarks: https://michaelfeil.eu/infinity/main/benchmarking/
more supported architectures (e.g. Mistral, due to Rust integration) + hardware
License: You can use infinity if you are working for-profit companies.


Added mpexecutive vehicle hashes

Vehicle hashes for mpexecutive update (1.0.757.2).

These were already added in #499 ...

@crosire Oh, I'm sorry. Didn't see that even though I checked all PRs.

No worries. It's a bit confusing right now, since we have two development branches (dev_v2 and dev_v3).

Btw, I'm going to change my name in 6-12 months so you only have one reason left to hate me :) :+1:


KryptonRibbon.RibbonAppButton.AppButtonMenuItems has error

Thanks for update.
I tried click 'Add' button on 'AppButtonMenuItems collection editor' in design mode.
When i was clicked, shown error message is 'Cannot create an instance of ComponentFactory.Krypton.Toolkit.KryptonContextMenuItemBase because it is an abstract class.'
[ ScreenShot ]

My develop setting is
OS: Windows 10 x64
Tool: VisualStudio 2017
Krypton Version: <IP_ADDRESS>

I have the same problem

This issue was written 1 month.
This repository was stopped?

I also have the same problem

vip00112, can you post your code where the problem occurs?

Here are the steps to reproduce the problem.

Start Visual Studio (VS2017 VS 15.9.8) and create a "Windows Forms App (.NET Framework)

Add the Krypton DLL to the project. (Projects, Add Reference)
ComponentFactory.Krypton.Docking.dll
ComponentFactory.Krypton.Navigator.dll
ComponentFactory.Krypton.Ribbon.dll
ComponentFactory.Krypton.Toolkit.dll
ComponentFactory.Krypton.Workspace.dll

Note: If needed, also add the Kryton DLL to the Toolbox in Visual Studio.  (To be able to do drag-drop of the KryptonRibbon to the GUI)

Add a KryptonRibbon to the GUI. (drag drop from the toolbox)

Try to add an AppButtonMenuItem. An error will occur after pressing the Add button.

Here is a screnshot of the Design when the error occurs:

Try making the form Form1 inherit from the KryptonForm class, rather than Form.
Try adding the using statements as well
using ComponentFactory.Krypton.Ribbon;
using ComponentFactory.Krypton.Toolkit;

In your form.
You can do that if you need to.
            kryptonRibbon1.RibbonAppButton.AppButtonMenuItems.AddRange(new ComponentFactory.Krypton.Toolkit.KryptonContextMenuItemBase[] {
            kryptonContextMenuItem1,
            kryptonContextMenuItem2});
            //Need to define and instantiate KryptonContextMenuItems separately.

I am getting the issue too and am not sure what causes it.

OK, thanks

It's an error with the editor, it's trying to add an instance of the base class (which is abstract) instead of the actual implementation that can be instantiated. Try creating the item programatically, then adding it to the collection, don't use the visual editor. Like so: `KryptonContextMenuItem item = new KryptonContextMenuItem();
kryptonRibbon1.RibbonAppButton.AppButtonMenuItems.AddRange(new ComponentFactory.Krypton.Toolkit.KryptonContextMenuItemBase[] {
item });

No solution?

FYI:
This is being investigated in latest alpha of Krypton-Suite:
https://github.com/Krypton-Suite/Standard-Toolkit/issues/34

Important change for whomever has code access displayed here:
https://github.com/Krypton-Suite/Standard-Toolkit/pull/550