[fmt] Add a way to disable FMT_UNICODE when building fmt

Is your feature request related to a problem? Please describe.
Hi, I have a small test project built on Windows with MSVC, with fmt 11.0.2#1 installed through vcpkg.
My test project uses the /source-charset:windows-1252 compile option for its own sources:
cmake_minimum_required(VERSION 3.29.0 FATAL_ERROR)

set(CMAKE_TOOLCHAIN_FILE "${VCPKG_ROOT}/scripts/buildsystems/vcpkg.cmake" CACHE PATH "")

project(Test CXX)

find_package(fmt CONFIG REQUIRED)

add_executable(main
    main.cpp
)

target_compile_options(main PRIVATE
    /source-charset:windows-1252
)

target_link_libraries(main PRIVATE
    fmt::fmt
)

However, when try to build it I get the following error:
C:\PROGRA~1\MIB055~1\2022\PROFES~1\VC\Tools\MSVC\1441~1.341\bin\Hostx64\x64\cl.exe  /nologo /TP -DFMT_SHARED -external:IC:\Users\martin\fmt_test\build\vcpkg_installed\x64-windows\include -external:W0 /DWIN32 /D_WINDOWS /EHsc /Ob0 /Od /RTC1 -MDd -Zi /source-charset:windows-1252 /utf-8 /showIncludes /FoCMakeFiles\main.dir\main.cpp.obj /FdCMakeFiles\main.dir\ /FS -c C:\Users\martin\fmt_test\main.cpp
cl : Command line error D8016 : '/source-charset:windows-1252' and '/utf-8' command-line options are incompatible

I believe the reason for this is the /utf-8 option being set as PUBLIC for the fmt target, here:
https://github.com/fmtlib/fmt/blob/0c9fce2ffefecfdce794e1859584e25877b7b592/CMakeLists.txt#L362
which results on the following code being generated by vcpkg's CMake machinery (on fmt-targets.cmake):
set_target_properties(fmt::fmt PROPERTIES
  INTERFACE_COMPILE_DEFINITIONS "FMT_SHARED"
  INTERFACE_COMPILE_FEATURES "cxx_std_11"
  INTERFACE_COMPILE_OPTIONS "\$<\$<AND:\$<COMPILE_LANGUAGE:CXX>,\$<CXX_COMPILER_ID:MSVC>>:/utf-8>"
  INTERFACE_INCLUDE_DIRECTORIES "${_IMPORT_PREFIX}/include"
)

I believe this ends up propagating the /utf-8 option to my own sources, and thus breaking the build.
Proposed solution
I reported this to fmt (see https://github.com/fmtlib/fmt/issues/4221) and the maintainer indicated that /utf-8 can be disabled through FMT_UNICODE, and it's an issue on the vcpkg side. Perhaps there could be done with a port feature?

Create a custom triplet and add something like that:
if("$PORT" STREQUAL "fmt")
    list(APPEND VCPKG_CMAKE_CONFIGURE_OPTIONS "-DFMT_UNICODE=OFF")
endif()

https://learn.microsoft.com/en-us/vcpkg/users/triplets#vcpkg_cmake_configure_options


[ENH]:  file.rename could change . to pt in the case of spec.entities.xxx = num2str(1.5)

Is there an existing issue for this?

[X] I have searched the existing issues

New feature
Hi there,
thanks for this useful tool.
I was wondering if there is any option to get file.rename deal with eg spec.entities.xxx = num2str(1.5) so that in the renamed file name the output is eg ..._xxx-1pt5_... and not ..._xxx-15_...
Unclear documentation
No response

yeah I think this is something we could implement


fix: Fixes for the nightly test run against Sepolia

This PR makes some fixes to the nightly Sepolia test run

Benchmark results
Metrics with a significant change:

proof_construction_time_poseidon_hash_100_ms (16): 672 (-25%)
proof_construction_time_poseidon_hash_30_ms (16): 202 (-27%)
proof_construction_time_sha256_30_ms (16): 1,375 (-20%)
protocol_circuit_simulation_time_in_ms (private-kernel-tail-to-public): 669 (-52%)
avm_simulation_time_ms (Token:mint_public): 44.9 (-35%)
avm_simulation_time_ms (Token:assert_minter_and_mint): 66.5 (+114%)
avm_simulation_time_ms (Token:transfer_public): 32.1 (+31%)
l2_block_processing_time_in_ms (4): 295 (+16%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_ms
5,751 (-1%)
1,560
710
759 (-1%)
776

proof_construction_time_sha256_30_ms
11,468 (-1%)
3,074 (-2%)
:warning: 1,375 (-20%)
1,426 (-12%)
1,461 (-7%)

proof_construction_time_sha256_100_ms
44,093
11,836
5,439 (-1%)
5,571 (+2%)
5,689 (-4%)

proof_construction_time_poseidon_hash_ms
78.0 (-1%)
34.0
35.0 (+3%)
57.0 (-2%)
88.0 (-1%)

proof_construction_time_poseidon_hash_30_ms
1,529
421
:warning: 202 (-27%)
227 (-14%)
267 (-7%)

proof_construction_time_poseidon_hash_100_ms
5,637
1,513
:warning: 672 (-25%)
727 (-11%)
748 (-6%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs
16 txs

l1_rollup_calldata_size_in_bytes
4,356
7,876
14,884

l1_rollup_calldata_gas
50,220
92,958
178,084

l1_rollup_execution_gas
842,880
1,576,514
3,361,696

l2_block_processing_time_in_ms
:warning: 295 (+16%)
473 (+5%)
843 (+1%)

l2_block_building_time_in_ms
11,400
22,301 (-4%)
44,574

l2_block_rollup_simulation_time_in_ms
11,400
22,301 (-4%)
44,574

l2_block_public_tx_process_time_in_ms
9,685 (+1%)
20,492 (-4%)
42,729

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
3,027 (+2%)
4,007 (+3%)

node_database_size_in_bytes
12,619,856
16,740,432

pxe_database_size_in_bytes
16,254
26,813

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Circuit
simulation_time_in_ms
witness_generation_time_in_ms
input_size_in_bytes
output_size_in_bytes
proving_time_in_ms

private-kernel-init
92.3 (-8%)
394 (-1%)
21,735
44,860
N/A

private-kernel-inner
178 (-1%)
700 (+3%)
72,544
45,007
N/A

private-kernel-reset-tiny
315 (-9%)
711 (+1%)
65,593
44,846
N/A

private-kernel-tail
168 (-7%)
136 (+4%)
50,644
52,257
N/A

base-parity
5.64 (+2%)
N/A
160
96.0
N/A

root-parity
35.7 (+1%)
N/A
73,948
96.0
N/A

base-rollup
3,446 (-2%)
N/A
189,136
664
N/A

block-root-rollup
41.4 (+1%)
N/A
58,205
2,448
N/A

public-kernel-setup
85.0
N/A
105,085
71,222
N/A

public-kernel-app-logic
97.7 (-3%)
N/A
104,911
71,222
N/A

public-kernel-tail
859 (-2%)
N/A
390,582
16,414
N/A

private-kernel-reset-small
309 (+1%)
N/A
66,341
45,629
N/A

private-kernel-tail-to-public
:warning: 669 (-52%)
613 (+4%)
455,400
1,825
N/A

public-kernel-teardown
84.0
N/A
105,349
71,222
N/A

merge-rollup
20.4 (-2%)
N/A
38,174
664
N/A

undefined
N/A
N/A
N/A
N/A
78,433 (-7%)

Stats on running time collected for app circuits

Function
input_size_in_bytes
output_size_in_bytes
witness_generation_time_in_ms

ContractClassRegisterer:register
1,344
11,731
347 (+1%)

ContractInstanceDeployer:deploy
1,408
11,731
18.3 (+1%)

MultiCallEntrypoint:entrypoint
1,920
11,731
406 (+1%)

FeeJuice:deploy
1,376
11,731
388

SchnorrAccount:constructor
1,312
11,731
74.3 (+2%)

SchnorrAccount:entrypoint
2,304
11,731
398 (+2%)

Token:privately_mint_private_note
1,280
11,731
103 (+6%)

FPC:fee_entrypoint_public
1,344
11,731
25.7 (-7%)

Token:transfer
1,312
11,731
224

Benchmarking:create_note
1,344
11,731
86.3

SchnorrAccount:verify_private_authwit
1,280
11,731
27.8 (+1%)

Token:unshield
1,376
11,731
522

FPC:fee_entrypoint_private
1,376
11,731
699 (+1%)

AVM Simulation
Time to simulate various public functions in the AVM.

Function
time_ms
bytecode_size_in_bytes

FeeJuice:_increase_public_balance
58.6 (+2%)
7,101

FeeJuice:set_portal
13.3 (+17%)
2,128

Token:constructor
89.6 (-5%)
25,285

FPC:constructor
59.9 (+11%)
17,853

FeeJuice:mint_public
39.4 (+2%)
5,415

Token:mint_public
:warning: 44.9 (-35%)
10,101

Token:assert_minter_and_mint
:warning: 66.5 (+114%)
6,844

AuthRegistry:set_authorized
38.1 (+2%)
3,969

FPC:prepare_fee
230
6,747

Token:transfer_public
:warning: 32.1 (+31%)
38,126

FPC:pay_refund
53.4 (-2%)
9,398

Benchmarking:increment_balance
1,201
6,179

Token:_increase_public_balance
39.2 (-1%)
7,705

FPC:pay_refund_with_shielded_rebate
62.3 (-1%)
9,881

Public DB Access
Time to access various public DBs.

Function
time_ms

get-nullifier-index
0.159 (-2%)

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
64,779
668,997

Transaction size based on fee payment method
| Metric |  |
| - |  |


CSUB-559: Add bootnodes & chainspecs info to README

Description of proposed changes
Add instructions to connect to different Creditcoin networks using different bootnodes and chainspecs.

Practical tips for PR review & merge:

[ ] All GitHub Actions report PASS
[ ] Newly added code/functions have unit tests

[ ] Coverage tools report all newly added lines as covered
[ ] The positive scenario is exercised
[ ] Negative scenarios are exercised, e.g. assert on all possible errors
[ ] Assert on events triggered if applicable
[ ] Assert on changes made to storage if applicable

[ ] Modified behavior/functions - try to make sure above test items are covered
[ ] Integration tests are added if applicable/needed

Codecov Report

Merging #1138 (9251ffa) into dev (c310af9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #1138   +/-   ##
=======================================
  Coverage   74.62%   74.62%
=======================================
  Files          74       74
  Lines       10686    10686
=======================================
  Hits         7974     7974
  Misses       2712     2712

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


chore: release v0.0.17

:robot: I have created a release beep boop
0.0.17 (2024-01-15)
What's Changed

fix(ci): ensure bash is used when building for Windows by @coffeebeats in https://github.com/coffeebeats/gdpack/pull/100

Full Changelog: https://github.com/coffeebeats/gdpack/compare/v0.0.16...v0.0.17

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/coffeebeats/gdpack/releases/tag/v0.0.17 :sunflower:


Use Datadog Gradle Plugin 1.14.0

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

Project coverage is 78.23%. Comparing base (f55c64b) to head (8c82375).
Report is 3 commits behind head on develop.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop     #271      +/-   ##
=============================================
- Coverage      78.34%   78.23%   -0.11%
+ Complexity       254      251       -3
=============================================
  Files             31       31
  Lines            914      914
  Branches         114      114
=============================================
- Hits             716      715       -1
  Misses           147      147
- Partials          51       52       +1

see 3 files with indirect coverage changes


[E2 - MS1] - Dae

Post your solution in the comments 😄
ps. remember to assign the amount of story points in the dedicated label.

https://gist.githubusercontent.com/gurrpi/6be1e6db3a29e92b08dc2ab20aefddc8/raw/65834c408700d4e84cbc74ac6c3344e04c089bd1/main.circom


Improve support for soft deletes

Elixir version
All
Database and Version
PostgreSQL 14+
Ecto Versions
All
Database Adapter and Versions (postgrex, myxql, etc)
Postgrex
Current behavior
I am trying to enable soft-delete behaviour in an application, following the rules in this article: https://evilmartians.com/chronicles/soft-deletion-with-postgresql-but-with-logic-on-the-database
Assuming a "item" table with the following field:
  add :_deleted_at, :utc_datetime

I can trivially add soft deletes like this:
    execute """
            CREATE OR REPLACE RULE "phx_sync_soft_deletion" AS ON DELETE TO "items"
            DO INSTEAD UPDATE items SET _deleted_at = NOW() WHERE id = OLD.id RETURNING OLD.*;
            """,
            """
            DROP RULE IF EXISTS phx_sync_soft_deletion ON items;
            """

As well as a view that hides all deleted rows:
    execute "CREATE OR REPLACE VIEW visible_items AS SELECT * FROM items WHERE _deleted_at IS NULL",
            "DROP VIEW IF EXISTS visible_items"

And then I change my Ecto schema to the following:
  schema "visible_items" do

This means I only see the non soft-deleted rows. But if I want to see them all, I can do Repo.all({"items", Item}).
However, this has one issue: because the delete never happen, the underlying Repo.delete(item) operation returns zero rows. Ecto then assumes this is happening because the record does not exist, and return an Ecto.StaleEntryError. We can workaround this by calling {:error, _changeset} = Repo.delete(item, stale_error_field: :id) but it still returns an error tuple.
Expected behavior
Perhaps we should allow a allow_stale: true option? Or allow_no_return: true?
I thought about enabling this by default, because a delete should be idempotent, but perhaps that's not the best idea. Some applications may want to detect when two deletes race each other?

:allow_stale seems better to me because the count of 0 is still technically a return value. and also you can still use returning.
i thought maybe postgres has a way to send the number of affected rows if a rule is changing the delete. but i haven't been able to find anything to do that

Enabling by default might be confusing. Assuming we will still return {:ok, struct}.
My reason for thinking this is because I see the same kind of confusion when insert/update are used without returning. It is relatively common for new people to be confused why the returned struct is not what they expect to be in the database. So this is kind of like another way to be confused.

Since zero rows are returned, returning has no effect, unfortunately.

Implementation is trivial. I'm against it being default because:

It's a breaking change
It can be a foot-gun / confusion point. Being opt-in is more straightforward IMO

Oh if both the rule and the delete operation specify returning I think you can still get data. At least it worked when I tested on my psql

Oh if both the rule and the delete operation specify returning I think you can still get data. At least it worked when I tested on my psql

So psql is doing something special. If I do this on the sample app I have:
Sync.Repo.query!("DELETE FROM items WHERE id = $1 RETURNING id", [Ecto.UUID.dump!(id)])
|> IO.inspect()

It returns this:
%Postgrex.Result{
  command: :delete,
  columns: nil,
  rows: nil,
  num_rows: 0,
  connection_id: 1286,
  messages: []
}

I would love if it worked :(

Yeah it seems it works with simple queries but not the binary protocol. I tried testing in Ecto as well and when it didn't work I tried in Postgrex directly to see what was going on. And I could get it to work on a simple connection but not a "normal" postgrex connection.
I'm curious what's going on so I'll probably still keep looking into it. But don't know if I'll find anything useful so probably shouldn't hold up this issue.

Beside the obvious benefits that comes with creating rules in the the database, then Ecto already points in the directions of implementation in the application layer:  https://hexdocs.pm/ecto/Ecto.Repo.html#c:prepare_query/3 and to me, this is where Ecto/Elixir application becomes powerful.
So maybe we should be able to modify a repos insert/delete/update even more.

We can close this, no?

I reopened because while the suggestion was implemented, there could be further discussion. But yeah, I think it can be closed :)


chore(main): release 3.9.0

:robot: I have created a release beep boop
3.9.0 (2024-09-11)
Features

added support for specifying kubelet identity (#257) (4584874)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v3.9.0
:sunflower:


Remove unused stuff from Godeps

We don't need it any more, but it never got removed.  Yay!

Clean up ok for 1.0. Assigning reviewer

LGTM. Thanks.


Create a CONTRIBUTING.md file

Expected Behavior
Devs can see how to contribute
Current Behavior
No contributing docs

Hello @cmcewen ,
Is this not the replacement of the CONTRIBUTING.md?


Additional tests for Utf8 writer

Ensure that the exception here has been fixed and unit tested https://github.com/aspnet/SignalR/issues/2187#issuecomment-386871789

@JamesNK Can this be closed?


Case statements

Only works for variants - and requires something for all different variants.
Maybe something like this:
case Ab.A 1 do
	Ab.A x do
		ret x + 1
	end
	else do
		ret 2
	end
end

?

the double do feels a bit much to me but i realize that that's how our scopes look.
what about an additional case expression? where the case "block" has to be just a single expression and thus doesn't need a do/end.
// statement
case Ab.A 1 do
  Ab.A x do
    ret x + 1
  end
  else do
    ret 2
  end
end

// expression
ret case Ab.A 1 do // possibly something other than `do` so the parser knows the difference between statement and expression
  Ab.A x -> x + 1
  else -> 2
end


The parent entity of a joint should be the child link

Environment

OS Version: Ubuntu 18.04
Source or binary build? source, https://github.com/ignitionrobotics/ign-gazebo/commit/ae1a048d5e04b49ef532562cbdce59706a3b0679

Description

Expected behavior: The pose component of a joint is relative to its child link, so the parent entity of joint should also be the child link
Actual behavior: The parent entity of a joint is currently its model. This causes issues when computing poses of joints or any of it's child XML elements using worldPose.

The parent entity of a joint should be the child link

I don't think it's that straight forward. The joint has technically 2 parents:

The entity that the joint belongs to, which is the model
The default frame of reference for the joint pose, which is the child link

components::ParentEntity currently holds the model entity, and I think that's correct. The question is how can we resolve its pose correctly. Some ideas:

Have yet another component, i.e. components::PoseFrame (I think this can become too complicated to maintain, and will probably duplicate some frame semantics functionality already in SDF, so I think it's not a good idea)
Handle joints as a special case within worldPose (not ideal, but not too bad)
Recommend against using worldPose for joints, and instead handle this in the new Joint class that we need to create eventually (#325 )

Expected behavior: The pose component of a joint is relative to its child link, so the parent entity of joint should also be the child link

Currently the JointPositionController GUI plugin lists the joints in a given model by checking for joints whose ParentEntity component matches a specified model entity. If we did decide to change the ParentEntity of a joint from its model to its child link, I think we'd still want to support this functionality in another way.

components::ParentEntity currently holds the model entity, and I think that's correct.

I also favor keeping a joint's ParentEntity as its containing model

The question is how can we resolve its pose correctly. Some ideas:

Handle joints as a special case within worldPose (not ideal, but not too bad)

I think this would be the easiest thing to implement. We are already working around this in the ForceTorque system.


Not deselecting item after scroll to different month

I have a bug where sometimes a day is not deselecting while selecting a new day. This is after I select a date in a next or previous month and the calendar scrolls to a different month.
How to reproduce:

Show month calendar
Select a day in the next month
The calendar scrolls to the next month
Select another day in that same month (which is now being displayed).
Now 2 items are selected, while only one should be selected.

I made a screengrab so the problem is more clear.
https://youtu.be/YZNH-OZaVMg
I use the master branch source code in this app.

Similar with #131
Fixed in branch 1.6.1.

Tell me if the problem is still there.
BTW: The design looks nice.


Allow specifying per-task storage limits in manifest.yaml

Similar to how we specify CPU and RAM limits. This would override TASK_ENVIRONMENT_STORAGE_GB.

Duplicate of #141


Configure Azure PipelinesBuild pipeline

This configuration builds the application and produces builds for both Debug and Release configurations.

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


custom field partials

I'm using rails admin on several projects, and I made heavy use of the feature of using my own partials for a field just by specifying on config.
Now I started a new project with rails 3.1 and the new rails admin (849898ff5bb1f4b2f3774320d7590b8c54946c31), but I found that rails admin doesn't reach to my partials. I followed the docs and tried several ways to do this, but I couldn't get it to work.
I see in the logs that rails admin renders its own '_form_field.html.haml' partial. When I put my own '_form_field.html.haml' on rails_admin/main, then I see it renders my form_field partial. OK so far. But it never renders my custom '_my_partial.html.haml' (or '_my_partial.html.erb' either) when I put it on rails_admin/main or any other place.
I tried this way:
  config.model MyModel do
    edit do
      field :my_field do
        partial 'my_partial'
      end
    end
  end

But also this way:
  config.model MyModel do
    edit do
      field :my_field do
        render do
          bindings[:view].render :partial => 'my_partial', :locals => {:field => self, :form => bindings[:form]}
        end
      end
    end
  end

Also tried specifying the full pathname, and using haml and erb partials. None worked. Rails admin always goes for  _form_field.html.haml, either its own or my local one. Also I didn't get any errors that could give some clues.
On the list section, 'pretty_value' doesn't seem to work either, but that's maybe unrelated.
What am I doing wrong?

partials must be in your app/views/rails_admin/main folder.
Name it _my_partial.html.haml
and reference it like:
  config.model MyModel do
    edit do
      field :my_field do
        partial :my_partial
      end
    end
  end

It should work...

It doesn't work for me. It seems like it's ignoring the code block given for the field.
I'll dive into your code to figure why, if I have the time. If I don't get the time, I'll simply do the job with some previous version.
Will tell you if I find anything.

Hello, I had the same issue and was able to get it somewhat functional by defining the partial as you suggested.
This works:
partial :flight_reservation_details
However, I cannot pass locals to it as with the previous implemtation. This breaks it:
partial :flight_reservation_details, :locals => { :flight => bindings[:object], :field => self }

OK, I have an update which is a bit of a hack but i think does the job.
1.) Edit the fields base class located in "libs/rails_admin/config/fields/base.rb" and add locals to the instance's accessible attributes like this
attr_accessor :defined, :order, :locals
This will essentially add the locals property to all fields, you can use this as your own little storage variable to pass data to the fields.
2.) Define the property in the config with whatever data you want, like this
RailsAdmin.config do |config|
config.model SomeModelName do
edit do
field :whatever do
self.locals = {:object => bindings[:object], :some_variable => "something", :something_else => "sure, why not..."}
partial :my_custom_partial
end
end
end
end
3.) Create the partial name like this
"app/views/rails_admin/main/_my_custom_partial.html.erb"
4.) Use your locals property from the partial like this
<%= "some variable = " + locals.some_variable + ", something else = " + locals.something_else %>
This should print on the page this
some variable = something, something else = sure, why not...

Now, what do you guys think about adding locals to the base attr_accessor list? Let me know if you like the concept and I'll add a new branch and submit the pull request.

Here's another workaround - to access your object inside partial you can use that:
field.bindings[:form].options[:parent_builder].object

@Rustik form.object should work in any partial.

Same issue here
do I have to apply  jorgeramos  patch ?
does it change anything if rails admin is installed as a plugin or a gem...I've been trying everythings

I have a similar situation, i call the partial successfully in edit section which applies for new and edit records, my partial renders a collection_select with values from one table in my database for users to choose.
rails_admin.rb:
config.model Tc::TcAgencia do
edit do
field :tc_tipo_agencia do
partial :tc_tipos_agencias_partial
end
field :tc_region do
partial :tc_regiones_partial
end
end
end
_tc_tipos_agencias_partial.html.erb:
<%= collection_select(:tc_tc_agencia, :tc_tipo_agencia_codigo, Tc::TcTipoAgencia.all, :codigo, :nombre, {:include_blank => true}, {:placeholder=>"Buscar", :style=>"display:none;"}) %>
When the user create a new record everything works as expected, but in edit view the collection_select doesn't mark the value saved for that field as selected, it renders the same as the new view.
I know that using form.object.[name_of_my_field] is possible to get the saved value but i don't know how to show it as selected in collection_select, it suppose to be taken by the object and rendered as selected without doing anything else.
Thanks.

@ejmm320 Use the form.select(see Rails doc)
I tested the :partial option, no worries whatsoever.
render was not configurable, made it configurable again, adding a spec.

@bbenezech Thanks a lot!! I fixed the problem using form.select like you said and now the partial behaves the way we expected.

This one NOT working with rails_admin-1.1.1.

the solution mentioned does not work with list command, (we're trying to implement inline edit)


Statistics included from local development?

I'm uncertain if Shynet also includes my site visits and sessions while developing locally. I tried reading the source code but didn't find any exclusions of HTTP_REFERERS or ORIGIN matching localhost or <IP_ADDRESS>.
Do I need to block requests to my Shynet host manually or does Shynet automatically ignore all requests coming from localhost?

By default Shynet don't ignore any requests.
There is an option to add ignored IP's but You have to do it for every service separately.
I personally exclude Shynet script during development (Simple if statement + DEV env variable)

If you go to Manage >  Advanced
There is a field Allowed origins in which you can set the origins for which you want to record statistics


Added package.json.

This fiyes https://github.com/gskinner/console-polyfill/issues/1.

:-1:

Sorry about that. For some reason this never made it onto my radar. I didn't notice it until your comment showed up in my mailbox.


The draco-gltf show error

1558405142978IfcWallStandardCase.zip
This zip is my original file with gltf2.0. This file is perfect. Having said that, thank you very much for your other tool that is obj2gltf.
Now I try to convert this gltf to draco-gltf. Not only using it as a command-line tool but also using it as a library is no problem.But the draco-gltf can not work normally. That means the tool works fine，but the result is not good. I don't know why.
I first guessed that this is result of draco's parameter configuration problem. I use gltf-pipeline as a library with different draco's parameter, but the result is the same as before. I try to analyze the binary part of the draco-gltf file, but I have no idea about this.
I hope that you can help me. Thank you very much.

@walsrlsraw What draco options are you using and what viewer are you loading the glTF in?
I converted it to draco with default settings and it loads fine in ThreeJS and CesiumJS.


Added Gemini in exchanges folder and exchanges.js, it's not showing up.

Am I mssing something? Where do I go to expose the new object

Awesome! I'd love to see a PR for this if you like to share (you don't have to as per license).
The docs for exchange addition were written before the UI, if you want the exchange to be visible there you need to recompile the UI (single page web app). How to do that is explained here:
https://gekko.wizb.it/docs/internals/gekko_ui.html#Developing-for-the-Gekko-UI-frontend

Hi @askmike
Did you ever receive a PR for this?
I think Gemini's API is almost exactly the same as Bitfinex (https://docs.gemini.com/rest-api/#introduction). It might be as simple as updating names, comments, and the capabilities.
I plan to give it a try if I have a minute. How do you recommend I test it out before sending the PR? Just run a high-frequency paper trader with a funded test account, and check logs to make sure everything looks good?

Hey!
No, I've never received a PR for gemini.

How do you recommend I test it out before sending the PR?

I usually use these strategies to force buy and sell actions (first one does it once, second one keep toggling):  https://gist.github.com/askmike/3031ea7aa587b61e20e38238cdcff770

Or is there some JS equivalent of a Junit or Mockito test you are using?

I am using mochajs for unit tests, however there are currently only unit tests for some core components. We are currently not unit testing exchange integrations since it's a lot of work ton keep them up to date (and if an exchange API changes the integration breaks anyway).


feat(ui): chips

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 15.78947% with 16 lines in your changes missing coverage. Please review.

Project coverage is 46.23%. Comparing base (4a1cde2) to head (05de4f0).
Report is 17 commits behind head on main.

Files with missing lines
Patch %
Lines

apps/ui/lib/components/Chip/Chip.tsx
15.78%
16 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #71      +/-   ##
==========================================
- Coverage   48.35%   46.23%   -2.12%
==========================================
  Files          18       19       +1
  Lines         273      292      +19
  Branches       57       70      +13
==========================================
+ Hits          132      135       +3
- Misses        141      157      +16

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


Check if device supports Bluetooth

Even if AndroidManifest contains uses-feature requirement for Bluetooth it is still possible to manually install apk to device without Bluetooth. In my case I install app to Android emulator which doesn't have Bt.
Code without this fix causes infinitive loop:

MainActivity.onResume calls reconnect() if Bl is not connected
BluetoothUtilImpl.reconnect() try to start Bl enabling intent.
Intent fails and MainActivity.onResume is called. Loop begins again

Not relevant anymore


Add item OpenCV in User Guide

Motivation and context

This PR contains adding item OpenCV and images.
Also updated item controls sidebar and shortcuts
The following measures were taken to solve problem #2765
How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file
[x] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

Coverage decreased (-0.04%) to 69.738% when pulling a609e346b9a90612e12ff408aa2cd3e930139d2c on TOsmanov:UG-add-item-OpenCV-tools into f49eec7fba96f830cfc02effad9b357eab254ec4 on openvinotoolkit:develop.


python38Packages.pysdl2: 0.9.8 -> 0.9.9

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.pysdl2 is: "A wrapper around the SDL2 library and as such similar to the discontinued PySDL project"
meta.homepage for python38Packages.pysdl2 is: "https://github.com/marcusva/py-sdl2"
meta.changelog for python38Packages.pysdl2 is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.9.9 with grep in /nix/store/m9jx35rx47wz7hnyg84piwbm66x5363j-python3.8-PySDL2-0.9.9

found 0.9.9 in filename of file in /nix/store/m9jx35rx47wz7hnyg84piwbm66x5363j-python3.8-PySDL2-0.9.9

directory tree listing: https://gist.github.com/ded2555adc6d5e2d677044d9a2e1b514

du listing: https://gist.github.com/586f07c709890cddef9b4e0b0b44185d

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

19 total rebuild path(s)

5 package rebuild(s)

5 x86_64-linux rebuild(s)
5 i686-linux rebuild(s)
4 x86_64-darwin rebuild(s)
5 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python38Packages.pysdl2
python38Packages.pytmx
python39Packages.pysdl2
python39Packages.pytmx
tauon

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/m9jx35rx47wz7hnyg84piwbm66x5363j-python3.8-PySDL2-0.9.9 \
  --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 python38Packages.pysdl2 https://github.com/r-ryantm/nixpkgs/archive/fff6978a1f8f4895f6f3adeddfcf1238c75faccf.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/m9jx35rx47wz7hnyg84piwbm66x5363j-python3.8-PySDL2-0.9.9
ls -la /nix/store/m9jx35rx47wz7hnyg84piwbm66x5363j-python3.8-PySDL2-0.9.9/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

5 packages built:

python38Packages.pysdl2
python38Packages.pytmx
python39Packages.pysdl2
python39Packages.pytmx
tauon

Maintainer pings
cc @pmiddend for testing.

Result of nixpkgs-review pr 139086 at fff6978a run on aarch64-linux 1

5 packages built successfully:

python38Packages.pysdl2
python38Packages.pytmx
python39Packages.pysdl2
python39Packages.pytmx
tauon

1 suggestion:

warning: python-include-tests
Consider adding a checkPhase for tests, or if not feasible, pythonImportsCheck.
Near pkgs/development/python-modules/pysdl2/default.nix:35:0:
   |
35 |     description = "A wrapper around the SDL2 library and as such similar to the discontinued PySDL project";
   | ^


Legends pixel aggregation

This PR fixes #10224 and fixes #10247

We look at quantification in order to generate color legends.

It implements a custom choropleth legend in builder side. The form associated is the same as a regular choropleth, keeping the prefix and suffix fields. Could you confirm @saleiva?
Also, I've observed some race conditions when changing the map aggregation carousel. In order to minimize them, a buffer strategy is implemented between the saving in the backend and operating with the legend model.
CR @xavijam

@jatorre please take a look at the backend part 🙏

I've got to ask for some tests in spec/requests/carto/api/legends_controller_spec.rb (guid0 made me promise I'll take care of his tests :laughing:). It should be easy enough, just follow the example of any legend payload (e.g: custom_legend_payload).
It's basically just adding an example JSON, and then adding it to a couple of places, but the machinery to test the result is already there.
After that, :green_apple:


Change hydrogen on hydrogen mapping default to True

Following the review of the default filters in #59, this PR changes the default value for map_hydrogens_on_hydrogens_only to True this should have no change on the performance when using the hybrid topology protocol from OpenFE as these mappings were removed silently as they involve bond constraints.

@IAlibay, you mentioned changing the defaults in openfe as well, but openfe exports the mapper again.


Door does not generate

The locked_door element sometimes will not generate, possibly because something is spawning on top of it, or it is trying to spawn on something.

This is possibly because the script which checks for the door location doesn't work, or it regenerates onto the door. This can be fixed by reiterating through the loop until it is on a space where another element isn't, but I'm unsure on how to  implement this.

This is a pretty rare occurrence, but It's happened to me a few times.

Fixed with new used_slots array checking


Doesn't work for gzipped items

If AWS_IS_GZIPPED is set to True, then this will check the hash of the un-gzipped item against the etag of the gzipped item, which will obviously not match (though once upon a time it used to, but amazon fixed that).
This means all js and css files on a set-up using gzipping will be copied on every collect.

I was wondering if there have been any updates on this, or if anyone has worked on a PR to fix it? I've been encountering the same issue.


Clean up in code from #539

just ignore that little linting issue. vs code forgot to autoformat for some reason.


python310Packages.pytenable: 1.4.10 -> 1.4.11

Diff: https://github.com/tenable/pyTenable/compare/refs/tags/1.4.10...1.4.11
Changelog: https://github.com/tenable/pyTenable/releases/tag/1.4.11
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 206567 run on x86_64-linux 1

2 packages built:

python310Packages.pytenable
python39Packages.pytenable


[SPARK-45207][SQL][CONNECT] Implement Error Enrichment for Scala Client

What changes were proposed in this pull request?

Implemented the reconstruction of the complete exception (un-truncated error messages, cause exceptions, server-side stacktrace) based on the responses of FetchErrorDetails RPC.

Why are the changes needed?

Cause exceptions play an important role in the current control flow, such as in StreamingQueryException. They are also valuable for debugging.
Un-truncated error message is useful for debugging
Providing server-side stack traces aids in effectively diagnosing server-related issues.

Does this PR introduce any user-facing change?

No
How was this patch tested?

build/sbt "connect-client-jvm/testOnly *ClientE2ETestSuite"
build/sbt "connect-client-jvm/testOnly *ClientStreamingQuerySuite"

Was this patch authored or co-authored using generative AI tooling?

No

@juliuszsompolski Please take a look

Merged to master.


Add flashMode propos & flashlight feature support

This is a good feature!

@calvin0x00 Please resolve the conflict. I hope this feature will be released asap.

@daviddang91 I fixed it.

Can this feature be merged?

@daviddang91 @shilu-stha you can use the property cameraProps to extends the RNCamera properties, for now.
For example:
<QRCodeScanner
  cameraProps={{
    flashMode: RNCamera.Constants.FlashMode[torchEnabled ? 'torch' : 'off']
  }}
/>

Hello everyone,
Any news on this PR? we would love to implement this on our app.
Thanks!

Hi all,
Any update on getting this PR merged into master? I'd love to implement this feature into my app. Thanks!

For the love of gawd, let's get this thing merged.

Thanks for adding this, apologies for the delay, releasing this as part of the next release (today)


Bug: error: no registered implementations of type exported.ClientState

Overview
When trying to fetch Ibc_ClientState with querier.Ibc_ClientState(clientId), there is an error (more details to reproduce the error below)
no registered implementations of type exported.ClientState

Please correct me if I am not using lens as intended
Potential Issue
exported.ClientState is an interface, and we would need to use RegisterInterface for all the clients, so the UnpackClientState function is able to work as expected.
Potential Fix

Maybe lens can expose the interface for a codec to the users, users can register interfaces as needed
Initialze the codec in the Querier

How to reproduce the bug
Here is a simple go file, when one runs it with go run main.go should give you the same error
// cat main.go

package main

import (
	"os"

	lens "github.com/strangelove-ventures/lens/client"
	"github.com/strangelove-ventures/lens/client/query"
	"go.uber.org/zap"
)

func NewClient(logger *zap.Logger) *lens.ChainClient {
	ccc := &lens.ChainClientConfig{
		ChainID:        "osmosis-4",
		RPCAddr:        "https://rpc.osmosis.zone:443",
		KeyringBackend: "test",
		Debug:          true,
		Timeout:        "30s",
		SignModeStr:    "direct",
	}
	client, err := lens.NewChainClient(logger, ccc, os.Getenv("HOME"), os.Stdin, os.Stdout)
	if err != nil {
		panic(err)
	}

	return client
}

func main() {
	// Setup logger, client and querier
	logger := zap.NewExample()
	client := NewClient(logger)
	querier := query.Query{Client: client, Options: query.DefaultOptions()}

	// Works well
	logger.Info("fetch particular channel")
	channel, err := querier.Ibc_Channel("channel-0", "transfer")
	if err != nil {
		panic(err)
	}
	logger.Info("channels fetched", zap.Any("channel", channel))

	// Does not work
        // .
        //
	clientId := "07-tendermint-1"
	logger.Info("going to make request for client state", zap.String("client-id", clientId))
	state, err := querier.Ibc_ClientState(clientId)
	if err != nil {
		panic(err)
	}
	logger.Info("client state retrived", zap.Any("state", state))
}

Detailed logs
{"level":"info","msg":"fetch particular channel"}
{"level":"info","msg":"channels fetched","clientstates":"channel:<state:STATE_OPEN ordering:ORDER_UNORDERED counterparty:<port_id:\"transfer\" channel_id:\"channel-141\" > connection_hops:\"connection-1\" version:\"ics20-1\" > proof_height:<revision_number:1 revision_height:8957707 > "}
{"level":"info","msg":"going to make request for client state","client-id":"07-tendermint-1"}
panic: no registered implementations of type exported.ClientState

goroutine 1 [running]:
main.main()

As you can see, the channel information is retrvied properly, but client state is not.

Looking more into this. I think i figured out the issues:

In my code I was not setting ChainClientConfig.Modules, issue still persisted even when i set ChainClientConfig.Modules = lens.ModuleBasics (due to following reason)
chainClient.Codec needs to register tmlightclient.RegisterInterfaces to be able to parse ClientState any proto (this is currently not done)

Fixes
Inorder to fix this, we need to RegisterInterfaces in the MakeCodec function.
There are two ways to go about this:

Just add the following snippet in MakeCodec function

	tmlightclient.RegisterInterfaces(encodingConfig.InterfaceRegistry)
	solomachine.RegisterInterfaces(encodingConfig.InterfaceRegistry)

Or create a generic AppModule and add it to the ModuleBasics, go-relayer has done this: https://github.com/cosmos/relayer/blob/main/relayer/chains/cosmos/module/app_module.go#L26
Let users pass ChainClientConfig.Modules, which will require no changes on the lens side, it will move the burden to the caller of the API then

I am fine with either ways. Would love to know what people think here.

@KyleMoser will take a look and advise on approach.

Any update on this?

@Anmol1696 I agree with your assessment, I think you would typically import the app module that contains the missing interfaces as you said (with the relayer example). If there isn't one you can directly call RegisterInterfaces but I think there are additional codecs that would miss - the legacy amino. RegisterLegacyAminoCodec
Here's an alternate reference of how Defiant Labs did this btw.
https://github.com/DefiantLabs/lens/tree/v0.0.15-dl/extra-codecs/osmosis
If you want to assemble a similar PR we can review it?


feat: customize measurement stroke color

Ref: https://github.com/zakodium-oss/analysis-ui-components/issues/313

Codecov Report
Base: 88.01% // Head: 88.01% // No change to project coverage :thumbsup:

Coverage data is based on head (95503a4) compared to base (7738172).
Patch has no changes to coverable lines.

:exclamation: Current head 95503a4 differs from pull request most recent head 7434abb. Consider uploading reports for the commit 7434abb to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #358   +/-   ##
=======================================
  Coverage   88.01%   88.01%
=======================================
  Files          12       12
  Lines         868      868
  Branches      133      133
=======================================
  Hits          764      764
  Misses        104      104

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@stropitek Please tell me if you're still working on it, otherwise I'll take it over in 30mins

Not working on it anymore


Honors /usr/lib/os-release when /etc/os-release is not available

Hey over here :wave:
So, there is a bug an unexpected behavior with distro in very "lite" environments, when for example even /etc/os-release is not available.

One-liner to reproduce : docker run archlinux:latest /bin/bash -c 'pacman --noconfirm -Sy python-distro && python3 -m distro'

It actually appears that we should be dealing with /usr/lib/os-release in such cases (reference).
This patch implements the expected behavior, and adds a very specific test case to verify it.
Bye :bow:

Note : the patch seems back-portable to the python2.6-support branch.

Rebased against new master, fixing conflicts to include changes brought by #247.
(cc @nir0s)

Up @nir0s please :roll_eyes:

Up... again @nir0s (cc @hartwork maybe ?) 😕

One more "up" for 2020.

Up... again @nir0s (cc @hartwork maybe ?) confused

I don't have capacity for proper review here atm, I'll leave this to @nir0s for now.

Fellas, I truly apologize, but I indeed hardly have any capacity to deal with issues nowadays.
I'd appreciate adding collaborators. Any volunteers?

Well, thanks for your answer. I can't go full-time on it of course, but as I now maintain a project completely based on yours, I'm indirectly interesting in distro's "health condition".
So I'm okay for sharing work with other(s), if it's really needed.
Bye, take care.

Fellas, I truly apologize, but I indeed hardly have any capacity to deal with issues nowadays.

I think it's good and fair to be realistic and open about it, thanks for that. And thanks for your work on distro in general :pray: At the same time I think it's a bit sad also e.g. because of the 9000+ projects dependening on distro today (on GitHub alone) including important players like SaltStack, Let's Encrypt Certbot, docker-compose and package manager conan. So the 154 stars up here are only the tip of the iceberg.
If I may ask, are your capacity constraints temporary or expected long-term? If long-term, maybe it makes sense to turn this project into a new GitHub org then. It's rather easy to do technically — create the org, then do a repo transfer (which moves all issues and PRs just like that, then add people) — and it may communicate better that the project has become the effort of a new team if you're saying goodbye long-term. At least one other trusted person will also need PyPI access for a full transition. I hope I'm being respectful here, I didn't intend otherwise. What do you think?

I'd appreciate adding collaborators. Any volunteers?

Speaking for myself, I will not be able to shoulder much load here but if there are other active contributors, I may be able to join and help a bit myself too (after my current sprint on https://github.com/git-big-picture/git-big-picture has cooled down more).

@HorlogeSkynet looks good to me now, thanks for your contribution and your patience! 🙏

You're very welcome, sorry for this little leftover that didn't catch my attention sooner.
Bye 🙇

@HorlogeSkynet looks good to me now, thanks for your contribution and your patience! 🙏

You're very welcome, sorry for this little leftover that didn't catch my attention sooner.
Bye 🙇

@nir0s this PR is ready and waiting for your review by now. Do you have a minute?

@nir0s this PR is ready and waiting for your review by now. Do you have a minute?

@nir0s any news?

@nir0s any news?

@nir0s how can we move forward?

So sad only pull requests "brandishing" the fork totem are able to get things move forward... 🙁

@nir0s this PR is ready and waiting for your approval by now. Do you have a minute?

@nir0s please please

There are conflicts now, though. @HorlogeSkynet.

There are conflicts now, though. @HorlogeSkynet.

🙄 I'll try to rebase soon.

roll_eyes I'll try to rebase soon.

@HorlogeSkynet let me know if you need help

@nir0s I went for a manual merge, all good now ✅


Documentation Improvement: wasmtime crate readme page

The wasmtime crate's readme is a bit sparse and isn't as thorough as the Rust - Wasmtime documentation page. Although it doesn't need to be (since they serve different purposes), it can and probably should be enhanced. By improving this page wasmtime can make itself more appealing and accessible to users looking at WebAssembly projects on crates.io.
One additional point of comparison is to the wasmer crate readme, which includes some features that can be borrowed.

Using links for anything that has an associated page (e.g. on "wasmtime Webassembly runtime")
Offering a "usage" section with an example and links to where you can find more.
Offering a "features" section that describes any high-level features of the crate.
The inclusion of CI, crates.io version, and license badges.

While a bit more nebulous, the wasmer crate also has a much warmer feeling to it and makes a lot of appeals to users like

mentioning that it is "the most popular WebAssembly runtime",
emphasizing that "It's designed to be safe and secure, and runnable in any kind of environment",
saying that wasmer is "Made with ❤ by the Wasmer team, for the community".

While it doesn't make sense for wasmtime to try to match these various appeals 1-to-1, it might make sense to think about what messages about wasmtime the crates page should try to convey to users about the tool, it's properties, and it's creators. One appeal that might make sense is to refer to wasmtime's status as part of the Bytecode Alliance.

I recently noticed that the wat crate, and a number of other wasm-tools crates, actually do the sorts of things I'm suggesting. It might make sense to extract out the structure of the more refined READMEs into a template for Bytecode Alliance crates.


Add a MQTTException as a Base Class for all MQTTnet Exceptions

Describe the feature request
I would like to have the ability to catch all MQTTnet exceptions in only one try catch statement.
Which project is your feature request related to?

Client
ManagedClient

Describe the solution you'd like
Create
public class MqttException : Exception
{
}
public class MQTTnetException : MqttException
{
}
Additional context
I have a feature running which fetches data, computes something and sends the data via mqtt, a solution like above would help to create an easier solution to differentiate between mqtt errros and compute errors.

Hi,
in my opinion this is already done. There is the exception "MqttCommunicationException". The timeout exception already derives from that exception. Is there an exception which is not covered here?
Best regards
Christian

My bad, I looked into all the classes and saw that  MqttProtocolViolationException and MqttCommunicationException derive from exception and didnt saw the connectiontion in TimeOut and UnexpectedDisconnect.
Sorry my bad.


Publish chart & images to Rancher Prime registry

We need to publish the chart and image to the Rancher Prime registry.

Working with EIO to get access: https://github.com/rancherlabs/eio/issues/1814

We have the username / password added to our repo. Waiting on the actual repo address.

The EIO ticket above contains details of the secrets that i have been added to our repo.


Remove extraneous comma from word arrays

Context: https://github.com/GoogleCloudPlatform/google-cloud-ruby/issues/1837

Codecov Report

Merging #1695 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1695   +/-   ##
=========================================
  Coverage     84.98%   84.98%
  Complexity     4383     4383
=========================================
  Files           425      425
  Lines         17452    17452
  Branches       1822     1822
=========================================
  Hits          14831    14831
  Misses         2004     2004
  Partials        617      617

Continue to review full report at Codecov.

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


TypeError: Object # has no method 'parse'

Bumping to protractor 2.5.0 this afternoon has caused our Travis builds to fail.
$ ./node_modules/.bin/protractor
/home/travis/build/rackerlabs/encore-ui/node_modules/protractor/node_modules/q/q.js:126
                    throw e;
                          ^
TypeError: Object #<Object> has no method 'parse'
    at Function.ConfigParser.resolveFilePatterns (/home/travis/build/rackerlabs/encore-ui/node_modules/protractor/lib/configParser.js:102:29)
    at new TaskScheduler (/home/travis/build/rackerlabs/encore-ui/node_modules/protractor/lib/taskScheduler.js:31:31)
    at /home/travis/build/rackerlabs/encore-ui/node_modules/protractor/lib/launcher.js:180:21
    at _fulfilled (/home/travis/build/rackerlabs/encore-ui/node_modules/protractor/node_modules/q/q.js:797:54)
    at self.promiseDispatch.done (/home/travis/build/rackerlabs/encore-ui/node_modules/protractor/node_modules/q/q.js:826:30)
    at Promise.promise.promiseDispatch (/home/travis/build/rackerlabs/encore-ui/node_modules/protractor/node_modules/q/q.js:759:13)
    at /home/travis/build/rackerlabs/encore-ui/node_modules/protractor/node_modules/q/q.js:573:44
    at flush (/home/travis/build/rackerlabs/encore-ui/node_modules/protractor/node_modules/q/q.js:108:17)
    at process._tickCallback (node.js:448:13)
    at Function.Module.runMain (module.js:499:11)

These run locally though! Very strange. We have Jenkins tests that run for internal projects that are experiencing the same failure. It may be something to do with Linux? I'm not quite sure as this is a very recent development.

Are you sharding in Jenkins but not locally? That was our issue; still haven't figured out why.

We run one instance of webdriver in the link to Travis above. This is also the case in our Jenkins tests.

Are you using at least node 0.12? According to issue #2588, that is what is causing this error.

Ah, thanks. I will close this issue then.


feat(telegram): Support Telegram Channel

Hi
This PR adds support for handling telegram channel message, I also formatted some code while developing.
To use it, simply add bot to your channel and add telegram-channel-name in user_setting table and it's done.
However, currently the bot will also reply a message under the channel name (as pic below), which may be annoying, can obviously can not be used in public channel.
At first I thought it could be fixed by disabling the bot's post_message privilege in channel, but this cause an error since the bot first sends a "request pending message" then edit the message, if no message was sent in the first place, it won't be able to edit the message which cause errors.
This can be fixed by deleting the bot's message after the message was sent, or completely refactor the send message logic.

Also, the channel name is not a unique identifier (you can have multiple channel with the same name), but the channel id can not be directly obtained from telegram app, it must go through the api. This is not intuitive so I tend to use the channel name as identifier :D

cc @athurg

fiexed CI

Channel is used for publishing message in Telegram, and only the admins (almost just one person or a bot) send messages, other subscriber just receive without sending anything. So in Telegram's API, there's no sender in channel update, which could used for identified the sender.
So I think it's may not satisfied for sending Memos in Channel, maybe Group is better than that.

Hi
I 'm afraid that I don't get your idea, what't the point of getting sender?
Also, IMHO supporting Group is weird since I don't think memos is a place for logging group chat message?
Initially I thought this could serve as a "sync" between channel and memos, even though it's a bit off my original intention while developing (mostly the bot reply under channel's name part), I think it can still help some channel operator who happens to be a memos user? Support for multi telegram user is more like a side effect.

Telegram bot is designed as a convenient way to send/save personal memos.
If you‘d like to sync the history of a group or channel to Memos, maybe there's a better way to do this, the exists bot maybe not the one.
For example, the bot require assign the Telegram user and Memos user first. But for a group/channel message sync, maybe it'll be better to auto create a Memo user from Telegram which is not exists.
And as you say, the bot should not reply a confirm message while receive a message from group/channel, and should set the message's visibility to Protected as default instead of Private.

close: https://github.com/usememos/memos/issues/1906 and
related: https://github.com/qazxcdswe123/telegramMemoBot/issues/16

Telegram bot is designed as a convenient way to send/save personal memos.

Yes I totally agree, but some channels are also run by one man so that count as "personal memos that I would also like to share with others"?

If you‘d like to sync the history of a group or channel to Memos, maybe there's a better way to do this, the exists bot maybe not the one.

Probably not history but new messages from channels, beside that I agree with this part.

For example, the bot require assign the Telegram user and Memos user first. But for a group/channel message sync, maybe it'll be better to auto create a Memo user from Telegram which is not exists.

They can probably share a same memos account? Given that they are using the same memos instance, I don't think a new user for everyone in group/channel is really that necessary. Also, maybe one have multiple telegram account?

And as you say, the bot should not reply a confirm message while receive a message from group/channel, and should set the message's visibility to Protected as default instead of Private.

Yeah I agree with you, this is a bit off my original thought...

If you still think it's better to support channel&group message, and share Memos user with more than one Telegram user, I think there is a walk around here:

If message is from a private chat ( means send to the bot directly), save it to Memos user  bind with the sender's id, which you can got from result.message.from.id or result.message.chat.id from Telegram's JSON request.
If message is from a group chat (mentions and/or not?), save it to Memos user bind to the group's id, which you can got from result.message.sender_chat.id or result.message.chat.id from Telegram's JSON request.
If message is from a channel, save it to Memos user bind to the channel's id, which you can got from result.channel_post.sender_chat.id or result.channel_post.chat.id from Telegram's JSON request.

For better compatible, maybe chat.id and chat.type should always be used together.
By the way, the Telegram user id looks like always be positive, with the channel/group id be negative, but I didn't found any document to describe it.

Thank you for your suggestion. You are right, this is a less-than-ideal solution :D. The complete solution is too complicated for me so I decided to close this PR :)


Fixed the golang error in remote-local-setup and made improvements

How to categorize this PR?

/area dev-productivity testing
/kind bug
What this PR does / why we need it:
This PR,

Fixed the Error loading shared library libresolv.so error in remote local setup by installing go using apk.
Installed k9s for debugging purposes.
Added a startupProbe to wait until all packages are installed.
Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:
Release note:

NONE

/retest


Can't install sdkman when using curl -O

Bug report
Install command on homepage doesn't work when curl is aliased to curl -O:
$ curl -s "https://get.sdkman.io" | bash

Due to -s, it returns silently with no error. Disabling alias works. This is the error when running without -s:
➜ curl -O "https://get.sdkman.io" | bash
curl: Remote file name has no length
curl: (23) Failed writing received data to disk/application

I disabled the alias for install and re-installed it after install, but it looks like sdk itself can't run either:
➜ sdk
==== INTERNET NOT REACHABLE! ===================================================

 Some functionality is disabled or only partially available.
 If this persists, please enable the offline mode:

   $ sdk offline

================================================================================

To reproduce

alias curl="curl -OJ"
Attempt to install or run sdkman

System info
macOS
zsh 5.9 (x86_64-apple-darwin23.0)
curl 8.6.0 (x86_64-apple-darwin23.0) libcurl/8.6.0 (SecureTransport) LibreSSL/3.3.6 zlib/1.2.12 nghttp2/1.61.0

This could be fixed by adding unalias -a to the appropriate scripts.
But I strongly suggest you do not alias fundamental Unix utilities—thereby possibly breaking a lot of scripts.
alias curlo="curl -OJ" would be one option on your side.
Depending on your use case you might want to edit the default curl config file.

This is not a bug. We assume the underlying commands to be unaltered. If you want an alias for this behaviour, I suggest having it as a different name, perhaps something like curlo or similar.


[Menu][ClickAwayListener] Broken touch scroll on nested menu items

Problem description
When you have menu with nested menu using menuItems and you will try to scroll nested menu using touch gestures it will close menu when you finish scrolling.

Link to minimal working code that reproduces the issue
Working example of this situation: https://physiotherapist-florence-87015.netlify.com/
First option With menu items have nested menu.
It can be reproduced on mobile device or Chrome with touch emulation.
Reason for this is ClickAwayListener that Menu is using for click away detection to close menu.
ClickAwayListener register handleClickAway callback for touchend event.
I already found solution for it by adding listeners for touchstart and touchmove in ClickAwayListener to detect it touch was moved after start and don't trigger handleClickAway for touchend if so. https://github.com/adam187/material-ui-custom-build/commit/bb59509ae6e5cbb6fa66e993b3318f7c04f4f87d
Here is working example with fix https://dancer-wills-87087.netlify.com/
I can prepare PR if this appropriate approach for this issue.
Versions

Material-UI: 0.18.7
React: 15.6.1
Browser: Safari Mobile

We have been porting the component on the v1-beta branch. We reimplemented it from the ground-up. While we haven't tested it, I think that the issue is most likely fixed on that branch. Hence, I'm closing it.
Still, we will accept PR fixes until v1-beta takes over the master branch.


Docs 404 page not working

When you call up a non-existing page for which we don't have a redirect in place, we no longer show our custom 404 page, e.g., this file for 19.1 docs: https://github.com/cockroachdb/docs/blob/master/v19.1/404.md.
Example of when you should get that 404 page (client-driver.html instead of the correct client-drivers-html): https://www.cockroachlabs.com/docs/stable/install-client-driver.html.

Solution: https://www.netlify.com/docs/redirects/#custom-404


Add Generated SBOM to release as artifacts

Describe the feature here
The idea is having something like:
https://repo1.maven.org/maven2/org/apache/commons/commons-dbcp2/2.10.0/
So people can watch/run/check any kind of analysis directly from an immutable SBOM. We just need to add a specific maven  plugin configuration in the release profile, like we did in https://github.com/apache/camel-kamelets/pull/1631
Thoughts on this?

Sounds ok to me.
I think in the quarkus-platform it's planned to generate SBOMs for each participant. So we'd be covered there also.


Best practice to have key attribute with v-for and avoid v-if with v-for

Describe the bug
In some components v-for is used with v-if which as per Vue documentation is not a good practice. Also key is missing in some components.
https://vuejs.org/v2/guide/list.html#v-for-with-v-if
Expected behavior
Best practice should be followed as per documentation.
Components that needs to be fixed:

VueBreadcumb
VueCalendar
VueDataTable
VueDataTableHeader
VueDonutChart
VueSelect

will be in the next release


MOJ Frontend issues

https://design-patterns.service.justice.gov.uk/get-started/prototyping/

These instructions are slightly wrong:

add window.MOJFrontend.initAll() to app/assets/javascripts/application.js below the line doing the same for GOVUKFrontend

(there is no line for GOVUKFrontend)

It requires jQuery

npm install jquery

It would be better as a plugin - and this would avoid having to do the instructions in number 1 at all

I wonder if we can work with them to improve?

Thanks Joe, we'll take a look into how we can integrate with the latest version of the kit better. Sounds like we probably need to extend our plugin config file a bit :)

oh sorry if you have a config then it is a plugin already, I didn't realise. It might need an update to avoid the init but I'll check with the rest of the team. Otherwise you just need to tell people to install jquery - its no longer in the kit by default in v13. You can just run npm install jquery and we do the rest.

We believe this should be fixed in v1.6.6 of the MOJ Pattern Library where we're now using a packaged file that includes its own jQuery distro, and also an init.js script to automatically initialise the components.


Configure ruff and black for linting

This PR configures linting with the Ruff linter (and the Black formatter that's already in use in the repository) to be easily runnable with pre-commit. It also adds a GitHub Actions workflow to run those checks for every pushed commit.
It then fixes the issues that the default set of configuration for Ruff (with small changes, see pyproject.toml) notes.

@benjaminaubin rebased

@benjaminaubin Rebased again 😁 I also enabled import sorting via Ruff (ref. #59).

Rebased post #57 / #59.

Rebased once again.

Rebased 🍂

Rebased on 059d8e9cd9c55aea1ef2ece39abf605efb8b7cc9.


[Feature Request] Entity-Query: give index into filter function

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[X ] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
The filter function filterBy provide only the current Element.
export interface SelectOptions<E> extends SortByOptions<E> {   asObject?: boolean;   filterBy?: ((entity: E) => boolean) | undefined;   limitTo?: number; }
Expected behavior
Expect to have the index number.
export interface SelectOptions<E> extends SortByOptions<E> {   asObject?: boolean;   filterBy?: ((entity: E, index?: number) => boolean) | undefined;   limitTo?: number; }
What is the motivation / use case for changing the behavior?
Permit to have in filter function the index. Can be usefull, for exemple, to test, if odd or even row, to work with position in the list, or filter to get 1 element every x element :
for exemple, to display list in 3 column, we can get 3 selectAll with modulo.
const colum1 = this.entityQuery.selectAll({filterBy: (entity: E, index: number) => index % 3 === 0 }); const colum2 = this.entityQuery.selectAll({filterBy: (entity: E, index: number) => index % 3 === 1 }); const colum3 = this.entityQuery.selectAll({filterBy: (entity: E, index: number) => index % 3 === 2 });
Implement it
It is easy, because, we have the information on each loop.
`for (let i = 0; i < ids.length; i++) {
const id = ids[i];
if (!entityExists(id, entities)) {
  continue;
}

if (!filterBy) {
  arr.push(entities[id]);
  continue;
}

if (filterBy(entities[id], i)) {
  arr.push(entities[id]);
}

}`

Create a PR, please :)

Create a PR, please :)

Yes, I am woking on it ! See PR.


Missing event groups/categories/actions

Missing event groups/categories/actions in current data set

Good point thx for reporting

Solved in #33, let me close this issue


Not working on safari browser

it is not working on safari browser although I am getting allow mic and later the audio spoke on micro phone is not recorded all settings are enabled for app still the issue

have you found any solution @venkatasaive ?

I understood the issue. When you run a streamlit app it runs on http. iOS gives mic permissions only to https sites.
You can switch to https using ngrok


Products going out of stock

Update OMS backend settings to avoid products going out of stock because of orders placed by e2e tests

Backend PR https://github.tools.sap/cx-commerce/spartacussampledataaddon/pull/74

works (changes in localhost)
old

new


Black screen youtube video in firefox 38.1

In page http://codefirefox.com/video/marketing-yourself

Hi.
Are you sure it's not a HTML5 player? Flash version looks different.

I was confuse, it's html5, but now everything it's ok (i don't know why suddenly works) thanks for the response


[Communication]filtering generated classes for code coverage

Hi @paolamvhz.  There hasn't been recent engagement on this PR.  Would you please be so kind as to let us know if this is still an active work stream by removing the no-recent-activity label?  Otherwise, we'll close this out in 7 days.

Since there hasn't been recent engagement, this is being closed out.  Feel free to reopen if you'd like to continue working on these changes.  Please be sure to remove the no-recent-activity label otherwise, this will be closed again in the next cleanup pass.


UK localisation...

I'm using this for a client, and now they've filled out tons of recipes they've raised the issue that the output is not what you expect in Britain:
British recipes use mixed units; metric and imperial (e.g., we use gram and teaspoon and ml - all at once). Ingredient lists do not look right when output as only one unit type. No one here would use 9.9mls (2 teaspoons) in a listing, but would at the same time use 1.5Kg beef and 2 teaspoons Thyme.
Is there a way to output the unit type per ingredient as input, rather than forcing a conversion?
We do not use "gs" to mean Gram. We use g. 400g. Never 400 gs. At the moment I'm planning to use a replace to get " gs" to "g" and " mls" to "ml".
Here's an example of how this is done in the UK: https://www.bbc.co.uk/food/recipes/nigel_slaters_lancashire_48954

As it turns out, the way the plugin is already built, you are able to pass any value other than null, imperial, or metric and Unit Conversion will be skipped, spitting out the units as originally entered. So... this works if you want British style unit display, without any tweaks to the plugin:
{% set ingredients = entry.recipe.getIngredients('original') %} {% for ingredient in ingredients %} 	{{ ingredient | replace({" gs": "g", " ml": "ml" }) }} {% endfor %}

Just like I planned it :)


is:issue function log_predictive_density_sampling ImportError

It import function logsumexp from scipy.misc, which has moved into scipy.special in version 1.3.0.
Please fix this in some new version.

It would be great if you could do this fix and submit a pull requeest!
Neil
Professor Neil Lawrence
http://inverseprobability.com

From: kae_mihara<EMAIL_ADDRESS>Sent: Sunday, March 8, 2020 2:41:33 PM
To: SheffieldML/GPy<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [SheffieldML/GPy] is:issue function log_predictive_density_sampling ImportError (#816)
It import function logsumexp from scipy.misc, which has moved into scipy.special in version 1.3.0.
Please fix this in some new version.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/SheffieldML/GPy/issues/816?email_source=notifications&email_token=AABAUWWZNK6M2R6KSAGJUBTRGOVB3A5CNFSM4LDZ7TEKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ITMD2HQ, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AABAUWXEM3I3HVKTQX42KWTRGOVB3ANCNFSM4LDZ7TEA.

glad to do this.


Change the non-zero output to zero when the detection is empty

What does this PR do?

Change the return code to 0 when no packages are detected, as empty repositories could want to start the scan right away without blocking everything

Nit: I think it would be interesting to report in the output that there are warnings in the output. 'No issues found' seems that everything is ok and I think a warning is something that should be reviewed


Issue #31: Avoid createSpiceInterface bug

Description
Remove an unused flag from createSpiceInterface that causes quadratic/exponential slowdown on reset
How should this pull request be reviewed?

[ ] By commit
[X] All changes at once

Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.

[X] Unit tests (General Environment only) pytest --cov bsk_rl/envs/general_satellite_tasking --cov-report term-missing tests/unittest
[X] Integrated tests (General Environment only) pytest --cov bsk_rl/envs/general_satellite_tasking --cov-report term-missing tests/integration

Test Configuration

Python: 3.11
Basilisk: 2.2.1b0
Platform: Mac

Checklist:

[X] My code follows the style guidelines of this project (passes Black, ruff, and isort)
[X] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] Commit messages are atomic, are in the form Issue #XXX: Message and have a useful message
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes

See https://github.com/AVSLab/basilisk/issues/548 for more details.


separate positioning for each monster or align left, right and meddle

enhancement

That would require me to rewrite some of the code for the monster widget, add separate configs and it's not something I wanna do anytime soon

even to just make it load from the right rather then the middle? haha ill try not let my OCD get to me


Fix storehost shutdown

Looks like we are closing a nil channel here, which
will cause panics. This patch simply removes the unnecessary close, since it is now within RPM.

Coverage increased (+0.2%) to 54.532% when pulling 13d53895ef1ff7cb20268efcb311e9068e0868f3 on fix_shutdown into e689d8e4ebc88e249e072050d3708dde7df678f6 on master.


Cant start the Game with this Addon / Mod "requires any version of fabric-api, which is missing!"

I tryed starting it with different versions of fabric and only together with Meteorclient, nothing worked, the game wont start, fabric and its API are installed
Error Copyed:
Incompatible mod set!
net.fabricmc.loader.impl.FormattedException: Mod resolution encountered an incompatible mod set!
A potential solution has been determined:
- Install fabric-api, any version.
Unmet dependency listing:
- Mod 'MeteorAdditions' (meteor-additions) 1.2.0 requires any version of fabric-api, which is missing!
at net.fabricmc.loader.impl.FormattedException.ofLocalized(FormattedException.java:51)
at net.fabricmc.loader.impl.FabricLoaderImpl.load(FabricLoaderImpl.java:192)
at net.fabricmc.loader.impl.launch.knot.Knot.init(Knot.java:148)
at net.fabricmc.loader.impl.launch.knot.Knot.launch(Knot.java:68)
at net.fabricmc.loader.impl.launch.knot.KnotClient.main(KnotClient.java:23)

When i try starting it without Meteor client i get the missing Meteor client in the Error message which means it checks the versions and wont start and its maybe not an hard error like dividing by "0". So maybe the Error is in the "Fabric identification Algorithm".
Maybe it could be fixed with just initializing the start anyway even if it thinks the Version is wrong, or just skipping the check.

Current versions of Fabric API use the fabric-api mod id, while fabric was used previously. Make sure you have installed an up-to-date version of Fabric API, which is required for this addon. If you have one already, please provide a full log including fabric-loaders mod tree.

You mean this with fabric API?
https://www.curseforge.com/minecraft/mc-mods/fabric-api/files/4757925

For logs, see this wiki page.
A quick local test confirms that this is not an issue with MeteorAdditions, since a simple test instance with fabric-loader-0.14.22-1.20.1, Fabric API with 0bc5ff2d5df97e2c654789bd5e325d89f9ff883e, Meteor Client with de66315531988cbdddbed2bc66533cd9589eb91b and MeteorAdditions with 309a4f32c689808b2df2efc83c1460b612c10cca works just fine.
Make sure you are really loading a current fabric api and that your checksums match, as something going wrong with your download is one possible source for issues.

latest.log

It does not. Try replacing meteoradditions with some other mod that requires, but does not bundle (!) fapi and see if that works. As I said, it works on my machine™, so this is something you will probably have to solve on your own and not an issue with this project.

Other Mods are working,  what do you mean with bundle fabricAPI?

A mod like appleskin.
Some mods only depend on a subset of fapi and can JiJ that subset, preventing users from having to download the full FAPI at the cost of a good bit of storage for every single mod doing so.
I do not bundle FAPI.


The hooks use a hard-coded value for GRID P40-6Q.

There is a single hard-coded value on line 804 of vgpu_unlock_hooks.c that makes the resulting lookup table match the GRID P40-6Q device:
pci_info[3] = 0x11ec;

This value is printed next to the PCI device ID in the list of supported vGPUs printed by nvidia-vgpud:
nvidia-vgpud[674]: Supported VGPU 0x32: max 4
nvidia-vgpud[674]: VGPU Type 0x32: GRID P40-6Q Class: Quadro
nvidia-vgpud[674]: DevId: 0x10de / 0x1b38 / 0x10de / 0x11ec
nvidia-vgpud[674]: Framebuffer: 0x164000000
nvidia-vgpud[674]: Mappable video size: 0x400000
nvidia-vgpud[674]: Framebuffer reservation: 0x1c000000
nvidia-vgpud[674]: FRL configuration: 0x3c
nvidia-vgpud[674]: CUDA enabled: 0x1
nvidia-vgpud[674]: ECC supported: 0x1
nvidia-vgpud[674]: Multi vGPU supported: 0x0
nvidia-vgpud[674]: Encoder Capacity: 0x64
nvidia-vgpud[674]: BAR1 Length: 0x100
nvidia-vgpud[674]: Frame Rate Limiter enabled: 0x1
nvidia-vgpud[674]: Number of Displays: 4
nvidia-vgpud[674]: Max pixels: 58982400
nvidia-vgpud[674]: Display: width 7680, height 4320
nvidia-vgpud[674]: License: Quadro-Virtual-DWS,5.0;GRID-Virtual-WS,2.0;GRID-Virtual-WS-Ext,2.0

As a result vgpu_unlock will only support the GRID P40-6Q device. We will need to have a lookup table of all these device IDs per GPU so that they can all be inserted into the lookup tables in the kernel module.
A workaround is to edit the hard-coded value and then remove and reinstall the kernel module using dkms.

This is perfect! Assuming that only one profile is currently working, the GRID P40-6Q device, it's still great progress and we can explore getting the others to work. Would creating a script to unload the kernel module, modify the necessary values, and then load it back in be a feasible solution for this bug?

This is perfect! Assuming that only one profile is currently working, the GRID P40-6Q device, it's still great progress and we can explore getting the others to work. Would creating a script to unload the kernel module, modify the necessary values, and then load it back in be a feasible solution for this bug?

That could possibly work, but it seems a lot more complicated than just adding all the IDs into vgpu_unlock_hooks.c. And with that solution you would only be able to use a single profile at a time.

Edit: I noticed that modifying any of those values like the ones you listed above, such as "License:" or "ECC supported:" seems to disable the specified VGPU configuration, making it not possible to use that profile in the hypervisor. Tested on Red Hat and GTX 1080 spoofed to Tesla P4.

Could you clarify? What did you modify and what step failed when you tried to use the profile?

That could possibly work, but it seems a lot more complicated than just adding all the IDs into vgpu_unlock_hooks.c. And with that solution you would only be able to use a single profile at a time.

If I understand the vGPU documentation correctly, you only actually can use one profile at a time on a card. For example, 6 2GB Quadro profiles on one GPU.

Could you clarify? What did you modify and what step failed when you tried to use the profile?

I modified the values nvidia-vgpud[674]: ECC supported: 0x1 and nvidia-vgpud[674]: License: Quadro-Virtual-DWS and what I found is that modifying any of these values seems to make the profile inaccessible. In other words, the GUI for the hypervisor provides a convenient drop-down menu for the profiles on the Tesla P4. For example, I modified a value on the P4-4Q profile and that profile now no longer shows up in the menu.
Some more digging reveals that these profiles are backed by signatures for each of their values. I don't know what's the workaround for that just yet, but one thing I can say is that if the end user of this script wants to use a particular profile like P40-6Q, there could be a way for the user to modify the script for that one particular profile, even if it doesn't let them use a different profile. For example, someone with a 12GB GP102 card could run two of those 6Q profiles, but not one 6Q and 3 2Q profiles at the same time without vGPU rejecting the configuration in the first place.

You are correct. Combining different profiles is not working.
I have updated the code to include support for all GRID-P40, GRID-P4 and GRID-T4 profiles, as such the hard-coded value is gone and I will close this issue.


Support for running tests continuously

We more often want to run the same test or set of tests over and over again. The scenario looks something like:
LOOP (upto N times) {
  Run Test
  Validate Test
  [Optional] Reset System State
  Goto LOOP
}

We can also extend this continuous behavior across multiple tests, although I think this will bring up the question of maintaining ordering across the tests.
LOOP (upto N times) {
  Run test1
  Validate test1
  [Optional] Reset System State
  Run test2
  Validate test2
  ...
  Goto LOOP
}

By adding continuous testing support, we can, in future, extend support to run the Zopkio tests via Hudson or other CI tool.

These features should be supported in the latest version of zopkio (0.1.14). I have also added a section at the end of the README on how to run few examples that allow running multiple iterations for each test and how to enforce order across multiple tests in case we are looping over entire test suite.


docs(express): Documented way to override express configuration provider

This aims to help integrator override express configuration by providing an easy copy-pastable example in their code.

Preview


normalize scale -> deploy.replicas, defaults to 1

set Scale=1 by default before loading the service config from yaml
normalize scale into deploy.replicas to conform to the compose specification
serialize deploy.replicas even if set to 0 (to fix https://github.com/docker/compose/issues/8571)

close https://github.com/docker/compose/issues/8571

/generate-artifacts


iOS: Full screen mode toolbar appears and disappears

There are a few oddities with the toolbar in full screen mode that I'd like to address.
Can we attach the toolbar to the top of the story and have it scroll both in and out of view? It seems like it scrolls out of view but is then detached. If it were attached the entire time, it wouldn't require any changes to the scroll bar or scrolling speed. I expect the toolbar to come back when I scroll back up but instead I'm seeing some wonky behavior with it appearing and disappearing.
What I'm trying to avoid is any bounce to the story content itself. As I scroll up and down and the toolbar disappears and reappears, it's causing scrolling to bounce around instead of directly staying with my finger.

I don't think there's anything I can do to improve that behavior. It is now using the standard iOS API to hide and show the navigation bar on scrolling, so the way it works when doing that is up to Apple.
I've actually spent quite a bit of time beyond the monthly amount on getting this to work as well as possible; making the feed bar and status bar background behave in a fairly reasonable way was very difficult.
If you can provide a screen recording of a specific behavior that you don't like, I could try to improve it next month, though no promises.

Thanks for going extra on this. I'm hoping to get the scroll dynamics of NewsBlur on par with the other big competitors (like Reeder) and it's these small details that I want to get right. Happy to go over for the month if you have the time to spare.
Here's a video showing two bugs. One is the missing feed bar, and second is the snap back.
https://www.dropbox.com/s/he2qjjx96x4i5jz/RPReplay_Final1593474142.mov?dl=0

The feed bar scrolls away with the nav bar, and I re-show it once you stop scrolling. I tried for some time to constrain it to the visible area, without success. I have to wait for scrolling to stop, otherwise it appears too low, as the view offset changes when starting hiding the bar, even though it is still partially visible — there is a disconnect between the theoretical position and the actual position. There may be a way to hook into that interactive animation, but I couldn't find it.
The snap might be a result of redrawing it, so I might be able to improve that, though not sure.
I don't have any more time available this month (or brain cells to melt on this), so I'll come back to it next month, if you don't mind.

It isn’t perfect, but I’ve spent a lot of time trying to get it tolerable in the various paging directions and fullscreen options. I’ll probably want to revisit this when doing the three-column layout changes (#817), since that'll be an opportunity to modernize the paging mechanism.
I'll leave this open for now.

The work for the Catalyst edition will improve this; see #817 for more info. So I'll close this.


Update landing page to add more detail and links to other services

User Story 184174: Build: Search page iteration
Update the landing page to add more information about what users can find in FIAT and links to other services to get more related information.
Screenshots of UI changes
Before

After

Checklist

[x] Pull request attached to the appropriate user story in Azure DevOps
[x] ADR decision log updated (if needed)
[x] Release notes added to CHANGELOG.md
[x] Testing complete - all manual and automated tests pass

Tested by @dneed-nimble


random 500 internal server error when doing queries

Request failed for https://api.meetup.com/2/events?group_urlname=LosAngeles&key=XXXXXXXXXXXXXXXXXXXXXXXXXX returned code 500. Truncated server response: internal server error (use muteHttpExceptions option to examine full response) (line 44, file "Code")
key removed for obvious reasons, but it seems that randomly the response given to any query is 500 internal server error, and this is wreaking havoc with my script... it seems kind of random cause I have several calls in my script and the error will be thrown all over the place in the script, it might be in line 44 on one run, line 128 in another, line 95 in another.
All of these errors come from UrlFetchApp.fetch()

While this version shouldn't fail could you try the new newer version 3 api for requesting group events  http://www.meetup.com/meetup_api/docs/:urlname/events/#list we have been steadily replacing version 2 apis with version 3 counterparts.

We are getting this more and more frequently too.
https://gist.github.com/dcce3a334e6cd380b2b4ff0d18b6ba72

Version 3 doesn't seem to be failing but I wasn't able to update all my code, some of the code returns a 404 when updated to v3... basically anything that is a push vs a get isn't working:
var aaurl = "https://api.meetup.com/" + ArrayGroupname[p+1] + "/events/" + ArrayID[p]; var aaoptions = { "method": "post", "payload": { "key": "XXXXXXXXXXXXXXXXXXXXX", "description": ArrayMnewdescription[p], //"description": ArrayMdescription[p] + rsvpdescription, } }; var response = UrlFetchApp.fetch(aaurl, aaoptions);
The above is the converted version of my old code which i will paste below, the above code will not work for some reason, when i paste it into browser, it says the authentication isn't working, but I've included the key, so I don't know why auth is failing in browser.
  var aaurl = "https://api.meetup.com/2/event/" + ArrayID[p]; var aaoptions = { "method": "post", "payload": { "key": "XXXXXXXXXXXXXXXXXXXXXXXXXXXX", "description": ArrayMnewdescription[p], //"description": ArrayMdescription[p] + rsvpdescription, } }

When editing events in the v3 events api, you'll want to use an HTTP patch instead of a post https://www.meetup.com/meetup_api/docs/:urlname/events/:id/#edit it looks like you are using a post which would explain the 404.
Browser requests from one domain to another will not work without CORS authentication which is only supported for oauth credentials https://www.meetup.com/meetup_api/docs/#cors

@jamescarr could you try out the v3 version of group member profiles amd let us know if you observe the same issues? https://www.meetup.com/meetup_api/docs/:urlname/members/#list

Is this still happening @jamescarr ?


Switch ccache to using compiler content instead of its modified time.

This is necessary because the compiler gets rebuilt with each new toolchain build.

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[ ] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[ ] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[ ] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[ ] All package sources are available
[ ] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[ ] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[ ] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[ ] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary
CCache hashes a number of attributes of each compilation operation including attributes of the compiler binary, the compiler switches, and the source code.
CCache is by default configured to record the modified time of the compiler binary as one of those attributes. This causes a problem because with each of our full builds, we recompiler the compiler as part of the toolchain and then use it to build the different packages. This means that with every full build, the entire ccache cache will get invalidated. Partial builds that just re-builds the rpms are okay since they use the same toolchain packages.
The fix is configure ccache to use the content of the compiler binary instead of its modified time. This will eliminate the confusion from the rebuild of the compiler. The downside is that it's a tiny bit slower since the compiler content has to be hashed as opposed to just reading the modified time stamp.
Change Log

Change
Change
Change

Does this affect the toolchain?

YES/NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-YYYY-XXXX

Test Methodology

Pipeline build id: xxxx

Please update the toolchain manifests for the new mariner-rpm-macros release number. This should resolve the "Check Manifests" PR issue.
From the toolkit directory, running the following should fix this:
sed -i "s/-2.0-23.cm2/-2.0-24.cm2/g" resources/manifests/package/*.txt


server: allocate replicas return ResourceExhausted

This PR closes #896.

The issue in CI will be fixed in PR #907.


ci: Display performance measurement results as custom metrics

The author should do the following, if applicable

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

What's this all about?
This will be an improvement in type checking performance monitoring added by the following Pull Request.
https://github.com/honojs/hono/pull/3406
In conjunction with octocov, type checking performance will be displayed as Pull Request comments and CI summary as follows

Pull Request  comments

note: The bundle size is not shown as a diff since it is the first time, but it will be shown after the main branch is updated.

CI summary

Hi @k2tzumi
Sorry for the late reply. This is great! But please wait a bit for the discussion in PR https://github.com/honojs/hono/pull/3443 to be finished.

@yusukebe
There is a discussion going on for #3443, is there anything we should address in our PR here?

If this PR is taken in, additional action may be needed.

Hi @k2tzumi
Thanks for nice work,
as you said, if https://github.com/honojs/hono/pull/3661 is merged, I think we should add custom metrics.

@EdamAme-x
Thanks for the comment! Once #3661 is merged, I’d like to add the bundle size to the custom metrics!

Thank you all, these are nice improvements! Type performance is the only thing I mildly warn about when recommending hono to everyone! I love seeing how hono and its ecosystem is improving every week and hoping to contribute back with our own OS projects. Keep up the great work.

@k2tzumi
I merged #3661!

Hi @k2tzumi
I realized that the codes in perf-measures are not formatted. I've created the PR and merged it into the main:
https://github.com/honojs/hono/pull/3683
Can you apply this change?

@k2tzumi
Sorry! I forgot to add a linter for the perf-measures directory. I've added it to this PR: https://github.com/honojs/hono/pull/3686
Could you update this PR?

Hi @EdamAme-x
Also, can you review this?

I was not able to verify the execution of jobs in perf-measures-type-and-bundle-check-on-main, so I tried it and found several problems, so I fixed them.
I've reflected the results in the Issue comment, so please check it out.

@k2tzumi @EdamAme-x
Thank you! Let's land it!

Wooow, super cool!
https://github.com/honojs/hono/pull/3666#issuecomment-2484697912


Error: Port XXX not bound after 10000ms  docker-compose -rabbitmq

Hello,  I am trying to create a docker-compose file with rabbitmq service but got the following error:
Error: Port 15691 not bound after 10000ms
evreytime with diffrent port
my docker -compose
version: "3"

services:
  rabbitmq:
    image: rabbitmq:3
    ports:
      - "1234:5672"

my code
  environment = await new DockerComposeEnvironment('./test/', composeFile)
      .withStartupTimeout(new Duration(10, TemporalUnit.SECONDS))
      .up();

thanks

Rabbit doesn't bind to all its exposed ports by default, meaning testcontainer's default wait strategy will timeout waiting for these ports to be bound. Use a log wait strategy instead:
.withWaitStrategy("rabbitmq_1", Wait.forLogMessage("Server startup complete"))

Duplicate of https://github.com/testcontainers/testcontainers-node/issues/133. Would be a good idea to create a module for rabbit to stop people running into this.