Some skills not converted to new sheet format

Hello,
thanks for the new sheet template !
However, lots of skills do not appear on the new sheet for the characters in our campaign.
Notably: repeating_language***, repeating_folkspell***, repeating_path***, repeating_practicedtalent***, meditation etc. are all missing.
I did some debug and it seems the function upgradeCharacter3Dot0 is not called. Specifically, in versioning, Current sheet version = 2.7 but sheet_type is undefined, so sheet conversion is not done, and goes directly to setting Current sheet version = 3.0.
Sorry for the limited details, I hope it makes sense.

Hi, I'm working on a v3.1 update now and will see if I can address this.  I'll also re-add sections to the sheet with historical data so people can copy/paste anything that was missed.

Thanks 👍

My pull request with v3.1 and hopefully some patches for this issue was
just merged a little easier today.  Please let me know if the Redo V2-v3
Conversion button seems to work for you.
Thanks,
On Thu, Jun 1, 2023 at 1:48 PM r3mi @.***> wrote:

Thanks 👍
—
Reply to this email directly, view it on GitHub
https://github.com/DigitalPagan/Mythras-Roll20-Sheet/issues/69#issuecomment-1572524344,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZBQQH2UD76UY5CNYYVSSTXJDIXJANCNFSM6AAAAAAYROC4UM
.
You are receiving this because you commented.Message ID:
@.***>

--
Matthew Carpenter

Had a number of people tell me this is working now so I'll close this one out.  Please let me know if there is anything esle.


Verify linear growth of intermediate cells

Probably this is a Moduro-CC3D issue.

Problem seems to be in CC3D.


Unreported range restriction violation

The following program should be rejected due to range restriction, but it runs fine. Thanks @enetsee for reporting this.
p(Y) :- q(_).
?- p(Y).

Fixed by bfb3819f9ccf1da38e67696ad44ff3503424fb17.


WIP: Canal fix

@lewismarshall take a look at this: https://kubernetes.io/docs/concepts/cluster-administration/sysctl-cluster/

Unfortunately this is only for namespaced items:

Sysctls which are not namespaced are called node-level and must be set manually by the cluster admin, either by means of the underlying Linux distribution of the nodes (e.g. via /etc/sysctls.conf) or using a DaemonSet with privileged containers.

So I guess we either deploy bash or update the userdata with the "correct value" and test. It all comes down to if smiladon requires 'non-strict' filters or not...

or update the userdata with the "correct value"

I prefer this, but for now this PR looks good. I am going to work on hardening host kernel with various MAC modules, etc and plan to stick all of that in userdata, but that's later.
Let's go with your changes first!


⚠ TorrentLeech has degraded performance

In 90d5c5e, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 4212 ms

Resolved: TorrentLeech performance has improved in 963fed6 after 7 minutes.


Registry entry in installation docs did not match up for me

I have 64-bit Windows and 64-bit SQL Server 2012 installed on my system. The installation doc says:
* If 32-bit Windows: `HKLM\SOFTWARE\Red Gate\SIPFramework\Plugins`
* If 64-bit Windows: `HKLM\SOFTWARE\Wow6432Node\Red Gate\SIPFramework\Plugins`

To get it to work, the reg entry on my system was the 32-bit key listed above, even though I am 64/64 on Win and SQL.
On my system, SSMS is installed in Program Files (x86).  I don't think there is a 64-bit version of SSMS. I checked a Windows 2012 64-bit system here and SSMS is also in Program Files (x86) there also.

@jdenicola67 I've released Alpha 12 which has a chocolatey package.  Would you be willing to try to test an install from that?  It should pass moderation in just a few days and I'll post the instructions after that.

I'm going to close this issue for now.  I've just released beta 1; if this is still an issue in that version (using Chocolatey to install) please let me know and reopen this issue.


DRAFT: Construct PolygonArray from an Array of Exterior Coordinates

Closes #122
Overview

Introduces the PolygonArray.from_exterior_array() class method, which takes in an array of exterior coordinates equivalent to the following that is computed from a Shapely Polygon
https://github.com/holoviz/spatialpandas/blob/dd941f2f5f6994c6f09996e9b8af315e16d00051/spatialpandas/geometry/polygon.py#L40-L42

Usage
In UXarray, we use SpatialPandas paired with Datashader to visualize geoscience data that resides on unstructured grids as Polygons. Our unstructured grid is represented as a set of connectivity and coordinate arrays, which we would like to avoid needing to convert to Shapely, because we can derive the exterior of each Polygon quickly using the connectivity and coordinate information.
Performance Comparison
Initial benchmarks, comparing across n_polygon & n_vertices
All benchmarks are run on an NCAR Casper HPC Node, with 4 CPU Threads and 128gb of Memory
Test 1
(40,000,000 Closed Polygons with 7 total verticies)

Method / # of Polygons
400,000
4,000,000
40,000,000

Construct with Exterior Coord Array
1.02s
9.74s
97.0s

Construct with Shapely Polygons
16.2s
159s
pending

Depending on any initial feedback from the developers here, I'd be willing to continue implementing this to support Polygons with holes, MultiPolygons, etc.
While this PR focusses on Polygon's specifically, I believe other geometries could benefit from this type of construction support.

Thank you for the PR! :+1:


Missing zone options

I would like to set several options inside zones that are currenty not supported by named.zone.erb. Would you like to make me a merge request with a bunch of new dns::zone parameters or with an additional_options parameter?

A PR with that would be welcome. It depends on the number of options and how well defined they are, but I think a bunch of parameters is where I'd start.

A zone definition supports ~75 different parameters. Do I need to be exhaustive in the PR? I am currently only interested in about five of them.

No need for an exhaustive list, just the ones you care about. It's how I always do PRs :)

With both PRs merged, is there still something to do here?

With both PRs merged, is there still something to do here?

Nope, we are done, thanks. :)


Add Savage Worlds Spanish character sheet

This is a Savage Worlds character sheet in spanish, simplified, and that includes auto buttons to calculate rises in fight and damage rolls (after comparing with parry and toughness).

I'm sorry, but we no longer accept new sheets that are just an alternate translation to an existing sheet. If you would like to add a new translation for this sheet, please update the original sheet to use Roll20's sheet translation tools.

I didn't understand, this was not just a translation, it included new functionalities that the english savage worlds sheet doesn't include. If I prepare this in english and then translate it to spanish will it be valid? Thanks

Sorry, since this is brand new I can't see "changes" only that the whole file is new.
You are correct. If you want to make these updates, and add the Spanish translation, you'll need to make the updates to the original sheet and hook it up to the translation tools to add the new translation. We are trying to eliminate the need for users to use separate sheets for different translation, unifying everything into a single sheet that is the "same" for everyone except that it will use the user's selected language.

Hi,
Is a complete rework. Is so different that it calculates the rises on skill rolls and injuries on damage rolls (no other existing sheet does that). Also, it includes an attacks section, that uses token targets to calculate successful attacks. What I will do then is to include the translation tools to have it in English and Spanish, and allow further translations, and create another pull request.
Thanks for your help

Perfect, thank you for your work in upgrading AND translating this sheet!


unsupported operand type(s) for +: 'datetime.datetime' and 'int'

AuthToken.objects.create function tries to add expires as integer to datetime resulting in the above exception.  It should be wrapped as a timedelta instead.
http://james1345.github.io/django-rest-knox/settings/
Here in the doc you do mention that TOKEN_TTL should be timedelta but then the example settings show it to be an integer. Either the doc should be fixed or you should support integer as hours as also mentioned in the doc.
  REST_KNOX = {
    'SECURE_HASH_ALGORITHM': 'cryptography.hazmat.primitives.hashes.SHA512',
    'AUTH_TOKEN_CHARACTER_LENGTH': 64,
    'TOKEN_TTL': 10,
    'USER_SERIALIZER': 'knox.serializers.UserSerializer',
 }

You are correct, the example is wrong. Please use timedelta, the doc shall be corrected.

Please update the documentation.

this was fixed in c928368c


mobile_settings not effective on iOS app

below config works fine on the entity_settings, but unfortunately I can not get the mobile settings to delete the overflow and
menu button on the iOS companion app
It does work on a browser window on mobile, and the desktop app, with minimal width the threshold
# https://github.com/maykar/kiosk-mode#config-options
mobile_settings:
  hide_menubutton: true
  hide_overflow: true
  ignore_entity_settings: true

entity_settings:
#Hides both the header and sidebar.
  - entity:
      input_select.mode: 'Kiosk'
    kiosk: true
# Hides only the header
  - entity:
      input_boolean.hide_header: 'on'
    hide_header: true
# Hides only the sidebar. Disables swipe to open.
  - entity:
      input_boolean.hide_sidebar: 'on'
    hide_sidebar: true
# Hides only the sidebar menu icon. Does not disable swipe to open.
  - entity:
      input_boolean.hide_menubutton: 'on'
    hide_menubutton: true
# Hides the top right menu
  - entity:
      input_boolean.hide_overflow: 'on'
    hide_overflow: true

things started working in the app !
might I ask if it is normal to see a rather large empty space at the bottom of a view? seems. much bigger then before (without Kiosk-mode)
secondly: can I add some helpers for mobile_settings too, as under entity_settings. (use case: default behavior would be without the menu button and overflow on mobile, but for that one moment one needs to 'develop' on the go, a boolean could be flipped to get the overflow )

Kiosk mode has no template capabilities and I don't think I'll add any. I'll think on a solution for single settings having the ability to be overridden or not.


fix: prevent multiple parent focus events on click

Closes #4260
📝 Description

caused by #4220
affects Checkbox, Radio, and Switch components
occurs when parent container is focusable (see sandbox)
need to call preventDefault to prevent parent focus from firing multiple times

⛳ Current behavior (updates)
https://github.com/user-attachments/assets/11655fd6-a81c-4e53-b7ba-dda01fb51d62
🚀 New behavior
https://github.com/user-attachments/assets/8ab62565-d06a-4a62-976c-80bd9a6e3f46
💣 Is this a breaking change (Yes/No):
No
📝 Additional Information

Summary by CodeRabbit

New Features

Improved focus behavior for checkbox, radio, and switch components to enhance user experience by preventing unnecessary focus events.

Bug Fixes

Resolved an issue where focus was incorrectly triggered multiple times on parent elements when interacting with checkbox, radio, and switch components.

Tests

Added new test cases to verify correct focus behavior for checkbox, radio, and switch components upon interaction.

Hey @Peterl561, thanks a lot fixing the issue.
The PR looks good to me.

Handling this issue in #4306

closing - fixed in https://github.com/nextui-org/nextui/pull/4311


feat: 🎸 upgrade tensorflow

fix security vulnerability

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Where is the label for "type: "row"" supposed to show up?

Bug Report
At least inside an array it doesn't show up.
Other Details
If it can't be displayed, disable it in the typescript type

Seeing as it is a presentation only field, I have found use-cases where having a label for the row field is desirable. Is there an alternative, seeing as it has been removed? Thanks!


this.store is not defined on the RowSelectionWrapper

The this.store is not defined. It seems the store is part of the properties of the RowSelectionWrapper, not the object itself.
https://github.com/react-bootstrap-table/react-bootstrap-table2/blob/4011cae18e8e8bf632671647ad950e8b3209e2c1/packages/react-bootstrap-table2/src/row-selection/wrapper.js#L31-L38

Hi @grogi
Exactly. It's the root cause of #246. Thanks for your information and we will fix it ASAP.

Duplicated with #246

@grogi  It's very sorry that we don't fix this critical issue in time, right now we fixed on<EMAIL_ADDRESS>please let me know if this issue still remain, thank


Cannot read properties of null (reading 'files')

Hello there !
I tried for the first time to launch server and connection to  to with my laptop. When I launch the server before I try to connect , this error spawn (crash).
But when I launch the server after launch keppler on my laptop not problem.

Just add the ? before the .files in the path specified.
Example:   project?.files.create(data.path, data.content)

Same issue here, if I launch the server first, it crashes while the client connects :
/usr/local/lib/node_modules/keppler/lib/index.js:267
                project.files.create(data.path, data.content)
                        ^

TypeError: Cannot read properties of null (reading 'files')
    at Socket.<anonymous> (/usr/local/lib/node_modules/keppler/lib/index.js:267:25)
    at Socket.emit (node:events:513:28)
    at /usr/local/lib/node_modules/keppler/node_modules/socket.io/lib/socket.js:531:14
    at process.processTicksAndRejections (node:internal/process/task_queues:77:11)

Node.js v18.16.0

amantank's solution fails as otherwise it just crashes on line 268.
Yet launching the server after the client's launch works fine. Even though the first file sync fails and files wait to be modified for a proper sync.


Should multiple fields with the same name be allowed in record construction?

This compiles without error which is pretty bad (at least in how I think it may interact with the current typechecker and vm which assume fields are unique).
#[test]
fn double_field_declared() {
    let _ = ::env_logger::init();
    let text = r#"
{ x = 1, x = "" }
"#;
    let result = support::typecheck(text);
    assert!(result.is_err());
}
```

As mentioned in #132 we should either disallow declaring two or more fields with the same name in record constructors or choose the last field as the field used for actual construction. (Might be some other way of handling this as well).

7bb8f0b fixes the immediate problem by making it an error. We might want to allow this instead and simply shadow earlier defined fields with those defined later.
{ x = 1, x = "" } // -> { x = "" } : { x : String }


[Question] ESP32 All Clusters App BLE Advertising but Not Visible on Phone?

Hi,
I've been testing out Matter in an ESP32 with the All Clusters App. I've been able to build and flash properly as well as commission through the chip-tool and change a light state. However, when the ESP32 says it's advertising, I cannot see that possible connection in anything. I've tried with the Google Home app and nRF Connect BLE sniffer (on Android) to view with no success.
`I (3029) chip[DL]: Configuring CHIPoBLE advertising (interval 25 ms, connectable, device name MATTER-3840)
I (3039) chip[DL]: Device already advertising, stop active advertisement and restart
I (3049) NimBLE: GAP procedure initiated: stop advertising.
I (3059) NimBLE: GAP procedure initiated: advertise;
I (3059) NimBLE: disc_mode=2
I (3059) NimBLE:  adv_channel_map=0 own_addr_type=1 adv_filter_policy=0 adv_itvl_min=40 adv_itvl_max=40
I (3069) NimBLE:
I (3079) chip[SVR]: Cannot load binding table: a0
I (32949) chip[DL]: Configuring CHIPoBLE advertising (interval 500 ms, connectable, device name MATTER-3840)
I (32949) chip[DL]: Device already advertising, stop active advertisement and restart
I (32959) NimBLE: GAP procedure initiated: stop advertising.
I (32959) NimBLE: GAP procedure initiated: advertise;
I (32979) NimBLE: disc_mode=2
I (32979) NimBLE:  adv_channel_map=0 own_addr_type=1 adv_filter_policy=0 adv_itvl_min=800 adv_itvl_max=800
I (32989) NimBLE: `
I'm confused on why that would be, any thoughts based on the log above? Does it have to do with the Phone's OS?
Thanks!

@loganbenda BLE advertisement is active before commissioning and hence the chip-tool can commission the ESP32. Once commissioning is complete, BLE (and hence the advertisement) is turned off in the default case.
Is the above log before commissioning? Do you mean that before commissioning too the BLE scanner app cannot see it? How do you try to find the device in the BLE scanner app? I notice a bug in the print statement which might cause the confusion -

device name MATTER-3840

The BLE advertisement does not use name at all. So, may be, there is an ESP32 with MAC address instead of the above name and that is your device. You can open the details and verify if the following are the contents of raw advertisement data for the default all-clusters-app: 0x0201060B16F6FF00000FF1FF008000
We will fix the print statement. Also, the Matter specification does not mention the name to be mandatory in the BLE advertisement and hence it is not added.

Once you commission a board you have to full-erase the ESP flash before the BLE device will appear again.

@loganbenda BLE advertisement is active before commissioning and hence the chip-tool can commission the ESP32. Once commissioning is complete, BLE (and hence the advertisement) is turned off in the default case.
Is the above log before commissioning? Do you mean that before commissioning too the BLE scanner app cannot see it? How do you try to find the device in the BLE scanner app? I notice a bug in the print statement which might cause the confusion -

@dhrishi Thanks for the comment. Yes, this is a log before commissioning. I was looking for MATTER-3840, so that was part of the confusion. I think I see the device now, if the UUID is specified as 0xFFF6? Do you know how we could enable the name as part of the advertising data to make it easier to search for with a phone app?
@jonsmirl I was thinking that was the case after commissioning was complete when reading through the example code. Thanks!

When you deploy your products Apple, Google and Amazon are providing commissioning libraries.
https://developers.home.google.com/mobile-sdk
Those libraries will take care of all of the QRCode and BLE stuff. Once Apple, Google and Amazon have commissioned the device onto wifi there is an option in their Home apps to open a commissioning window. Opening the commissioning window allows your app access. In most cases you won't bother with directly commissioning the device.

@jonsmirl After commissioning, is the intent to run all of the matter devices' data through one of the respective hubs (Apple, Google, Amazon) to the cloud, or could it also be through a custom application to the cloud without necessarily relying on a hub? I've been trying to find details on how AWS might integrate the cloud portion, but haven't found a ton of info to date.

Apple, Google and Amazon apps are going to provide remote control of standard devices, you don't need a cloud piece if you have a standard device.  If your device is non-standard and you want to use the cloud, then you have to do everything yourself like before Matter.

I think I see the device now, if the UUID is specified as 0xFFF6?

@loganbenda That's right

Do you know how we could enable the name as part of the advertising data to make it easier to search for with a phone app?

You will have to add local name into the BLE advertisement on ESP32 here
Feel free to close the issue if you think your questions are answered.

That answers my questions on this, thanks!


[Core] Migrate @azure/ms-rest-azure-js as @azure/core-lro

This change migrates @azure/ms-rest-azure-js as @azure/core-lro as this library is needed by AutoRest-generated clients in @azure/keyvault-keys and other SDK client libraries.
~@bterlson is this package name appropriate or should we use a different name?~ -- Brian and I discussed this offline and decided to name it @azure/core-lro since most of what this library contains is long-running operation (LRO) implementations.

This looks good to me code-wise, but it's quite a bit different from what we landed on in our arch board meetings. So I'm now a little concerned about the name given that the we may want two pollers - this one, left as is, to support the management plane libraries that are already using it, and another one used by the data plane libraries. If we go with that approach, maybe call this one core-mgmt-lro and leave core-lro for the data plane poller? Open to suggestions.

Luckily I was able to avoid this with some targeted changes to autorest.typescript so I'll close this for now until it becomes necessary.


Using the model without ReusableEmbedding and TiedOutputEmbedding

Hi, I just want to ask how to use the model without using these embedding layers. I tried to write it like this and got an exception.
x = np.array([[1, 1, 1]] * 1000)
y = np.array([[1]] * 1000)

input_layer = Input(shape=x.shape[1:], dtype='int32', name="input_layer")

transformer_layer = TransformerBlock(name="transformer", num_heads=2, use_masking=True, vanilla_wiring=False,
                                     residual_dropout=0.1, attention_dropout=0.1)

coordinate_embedding_layer = TransformerCoordinateEmbedding(1, name='coordinate_embedding')

transformer_act_layer = TransformerACT(name='adaptive_computation_time')

output_avg = Lambda(lambda in_x: K.mean(in_x, axis=1), name='avg_layer')
output_softmax_layer = Softmax(name='output_softmax')

next_step_input = input_layer
next_step_input = coordinate_embedding_layer(next_step_input, step=0)
next_step_input = transformer_layer(next_step_input)
next_step_input, act_output = transformer_act_layer(next_step_input)

transformer_act_layer.finalize()

final_output = output_softmax_layer(inputs=output_avg(act_output))

model = Model(inputs=[input_layer], outputs=[final_output])

optimizer = Adam(
    lr=0.1, beta_1=0.6, beta_2=0.999)

model.compile(
    optimizer,
    loss=sparse_categorical_crossentropy,
    metrics=["accuracy"])

I got exception like this
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/tensorflow/python/framework/tensor_util.py", line 558, in make_tensor_proto
    str_values = [compat.as_bytes(x) for x in proto_values]
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/tensorflow/python/framework/tensor_util.py", line 558, in <listcomp>
    str_values = [compat.as_bytes(x) for x in proto_values]
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/tensorflow/python/util/compat.py", line 61, in as_bytes
    (bytes_or_text,))
TypeError: Expected binary or unicode string, got None

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "try.py", line 64, in <module>
    main()
  File "try.py", line 35, in main
    next_step_input = coordinate_embedding_layer(next_step_input, step=0)
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/keras/engine/base_layer.py", line 431, in __call__
    self.build(unpack_singleton(input_shapes))
  File "/home/gregory112/go/src/keras-transformer/keras_transformer/position.py", line 113, in build
    trainable=True)
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/keras/legacy/interfaces.py", line 91, in wrapper
    return func(*args, **kwargs)
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/keras/engine/base_layer.py", line 249, in add_weight
    weight = K.variable(initializer(shape),
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/keras/initializers.py", line 112, in __call__
    dtype=dtype, seed=self.seed)
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/keras/backend/tensorflow_backend.py", line 4139, in random_uniform
    dtype=dtype, seed=seed)
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/tensorflow/python/ops/random_ops.py", line 239, in random_uniform
    shape = _ShapeTensor(shape)
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/tensorflow/python/ops/random_ops.py", line 44, in _ShapeTensor
    return ops.convert_to_tensor(shape, dtype=dtype, name="shape")
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1039, in convert_to_tensor
    return convert_to_tensor_v2(value, dtype, preferred_dtype, name)
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1097, in convert_to_tensor_v2
    as_ref=False)
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1175, in internal_convert_to_tensor
    ret = conversion_func(value, dtype=dtype, name=name, as_ref=as_ref)
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/tensorflow/python/framework/constant_op.py", line 304, in _constant_tensor_conversion_function
    return constant(v, dtype=dtype, name=name)
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/tensorflow/python/framework/constant_op.py", line 245, in constant
    allow_broadcast=True)
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/tensorflow/python/framework/constant_op.py", line 283, in _constant_impl
    allow_broadcast=allow_broadcast))
  File "/home/gregory112/go/src/keras-transformer/venv/lib/python3.6/site-packages/tensorflow/python/framework/tensor_util.py", line 562, in make_tensor_proto
    "supported type." % (type(values), values))
TypeError: Failed to convert object of type <class 'tuple'> to Tensor. Contents: (None, 3). Consider casting elements to a supported type.

In my case, those ones in the x (the input dataset) will all be one-hot encoded so I think I can skip the ReusableEmbedding layer part. How can I do this?

I changed the dtype to float32 and reshaped the input data and now it's working.


scroll to top is confusing on long pages

Hiya,
We now have the workaround for the unwanted scrolling down to the tab (pr #11), however I think we've created a different problem as a result.
The scroll to top is happening even when you already are on the page and just switching tabs. I don't think it should scroll at all when you already have the page open (easier said than done I know).
To see what I mean:

up the back-to-top test page
make your browser small enough that the filler text at the top pushes the tabs off the bottom of the window
scroll down to the tabs
click another tab
notice that the page is scrolled to the top and you can no longer see what you clicked on

this is in firefox.

perhaps @jabbtech could pile in on this one?

Hey @timabell!
Having the scroll to top always happen can be useful in instances where there is a desire to switch tabs while at the same time keeping a sites primary navigation at the top of a page in view. It wouldn't be too difficult to expand the backToTop option though, we could address the issue you described by passing the desired behavior like:
backToTop: always | onload | none


Make Julia 1.10 type ids stable

Julia 1.10 changes the string representation of NamedTuple types, and we use these string representations to generate stable hash values for types:
What was once:
Base.NamedTuple{(:a,:b),Tuple{Int64,NamedTuple{(:x,:y),Tuple{Int64,Int64}}}}

Is now:
@NamedTuple{a::Int64, b::@NamedTuple{x::Int64, y::Int64}}

As I see it we have two options for how to handle things moving forward with 1.10: we could start to create a function that generates a string (or some other stable representation) from the internal representation of a type OR we can parse the string generated in Julia 1.10 to generate the string of a type from Julia 1.9 and lower. The former is a long-term solution, but it changes existing hashes (and I have a sketch of a plan for that below in the comments). I think the latter needs to happen so we can avoid breaking existing hashes.
So in this PR I am going with the latter option, using PikaParser to parse out the bits of syntax I need to re-arrange named tuple strings into the pre-1.10 string representation.
I haven't run benchmarks yet, but they should be relatively unchanged, since this is all stuff that gets computed at compile time.
Benchmarks
Author, please:
Run the benchmarks under test/benchmarks.jl locally, and update test/benchmark_records.md by inserting a new table at the end of the document with the updated numbers
printed at the end of the benchmark script. Place the table as it occurs before
and after the application of this PR.
Before
After

Having thought about this more, I think that long-term, a more viable solution to computing a stable hash of types would be to rely on a single internal parameter of DataType: parameters. We can generate e.g. Array{1, Int} with something like:
function stable_type_string(::Type{T}) where T
    result = string(parentmodule(T))*"."*string(nameof(T))
    if !isempty(T.parameters)
        result *= "{"*join(stable_type_string.(T.parameters), ",")*"}"
    end
   return result
end
stable_type_string(x) = sprint(show, x)

btw I think only DataTypes have a parameters fields not Types in general. https://github.com/JuliaDebug/Cthulhu.jl/pull/493

btw I think only DataTypes have a parameters fields not Types in general. https://github.com/JuliaDebug/Cthulhu.jl/pull/493

Thanks for this tip @Zentrik! If I understand correctly then, for the long-term fix, I will need to check for T isa DataType and otherwise print nameof(T) without the parameters. Or do you know if that is going to still run into problems? An alternative would be to fallback to string(T) if !(T isa DataType), and run the risk of running into problems if the printing of non-DataType  types changes.

the PikaParser stuff is a lot ....I don't think I have time to really understand what's going on there yet... I can come to back to have another look but some unit tests would help me understand what it's doing and also give me better confidence that it works as intended

Fair enough. This is an unfortunately ugly / bulky effort. I think it is probably best to do in the short-term to make sure that existing hashes can be leveraged on Julia v1.10. That is part of why I have started implementing #46 which should be a much more sustainable long-term approach.
I'll cook up some unit tests to verify the basic assumptions of the parser and hopefully improve your review of it.

btw I think only DataTypes have a parameters fields not Types in general. JuliaDebug/Cthulhu.jl#493

Thanks for this tip @Zentrik! If I understand correctly then, for the long-term fix, I will need to check for T isa DataType and otherwise print nameof(T) without the parameters. Or do you know if that is going to still run into problems? An alternative would be to fallback to string(T) if !(T isa DataType), and run the risk of running into problems if the printing of non-DataType types changes.

I have no idea about the stability of these things over time/ Julia versions. I'm not really the right person to ask.


Add back compatibility with torch

Adds back compatibility with torch, invoked by passing the USE_TORCH env var.
Fixes #1897

Looks great!
Suggestion: add a file .example.env with USE_TORCH= and add a comment to indicate what have it set to 0 or 1 does.


No dimension set for key window

Is this a bug report?
(write your answer here)

Have you read the Bugs section of the How to Contribute guide?
(Write your answer here.)

Environment

react-native -v:
node -v:
npm -v:
yarn --version:

Then, specify:

Target Platform:

Development Operating System:

Build tools:

Steps to Reproduce

(Write your steps here:)

Expected Behavior

(Write what you thought would happen.)
Actual Behavior

(Write what happened. Add screenshots!)
Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

Debug is working，but ‘No dimension set for key window’


Small changes on /version and /imagine support messages

What is the reason for {config.imagine or False}?

This is how my /version looks like
...

AI information:
- model: gpt-3.5-turbo
- history depth: 10
- imagine: None
- shortcuts: bugfix, proofread, summarize, translate, eng

I think it's better to have imagine: False or something like imagine: disabled

What is your version and what is the value of the imagine property in the config.yml?

Version: 125
I do not have property imagine

I don't think so. More likely is that you have imagine: null in your config.yml. Please check this.

I was wrong.  Currently, I can't reproduce that behavior.
Default value is True, which is surprising

The default value for imagine is True. That's correct.

You can revert the imagine changes, and I'll merge the PR.

Thanks!


[KeyVault] CHANGELOG and package.json updates after recent fixes

CHANGELOG update after releasing the recent hotfixes, and making sure the upcoming versions have mention of the hotfix that will be baked in.
I'm making this as a draft PR since I am not sure if I should be adding something else besides these changes, or if these changes should be made differently. First time releasing a hotfix!

My only comment here is that we might want to sort by version. Do you know how other changelogs are sorting?

@ramya-rao-a hi! any thoughts on this regard? https://github.com/Azure/azure-sdk-for-js/pull/8892/#issuecomment-628149645 I am partial to the way I did it, but I can change it.


EKF: pull in miscellaneous bug fixes

Incorporates the following fixes.
https://github.com/PX4/ecl/pull/289
https://github.com/PX4/ecl/pull/291
https://github.com/PX4/ecl/pull/292
https://github.com/PX4/ecl/pull/283
https://github.com/PX4/ecl/pull/288

@PX4TestFlights Please test

couple flights on a pixracer (V4)
http://logs.px4.io/plot_app?log=d7460b44-78e5-47e4-9419-9e03d2676237
http://logs.px4.io/plot_app?log=c50d804d-7bd8-440c-81d6-c665c5f492a9
overall good flights; slight variations in vertical position when in hover

few flight on pixhawk mini (v3)
http://logs.px4.io/plot_app?log=d1a6a137-3ca2-4561-937e-a61f2fdcd971
http://logs.px4.io/plot_app?log=8c4af454-a3c6-49d4-9e4f-10bfb233f494
http://logs.px4.io/plot_app?log=c05d17f7-a1df-4e76-b4bd-086d263684ef
overall good flights; no issues

Testing results look good. Summary plots for the test population here:
population_data.pdf


fix: ensure we back off for longer when out of slots

backoff time was not being set since exception was not yet wrapped

2022-10-24 11:21:37.931 DEBUG (MainThread) [bleak_retry_connector] Eve Room 0881 [EB:4A:D4:93:68:EF] (id=6C:D6:F8:C8:96:56) - EB:4A:D4:93:68:EF: Connecting (attempt: 2, last rssi: -57)
2022-10-24 11:21:37.931 DEBUG (MainThread) [bleak_retry_connector] Eve Room 0881 [EB:4A:D4:93:68:EF] (id=6C:D6:F8:C8:96:56) - EB:4A:D4:93:68:EF: Failed to connect: No backend with an available connection slot that can reach address EB:4A:D4:93:68:EF was found, backing off: 1.5 (attempt: 2, last rssi: -57)
2022-10-24 11:21:37.934 DEBUG (MainThread) [bleak_retry_connector] Eve Room 0881 [EB:4A:D4:93:68:EF] (id=6C:D6:F8:C8:96:56) - EB:4A:D4:93:68:EF: Connecting (attempt: 3, last rssi: -57)
2022-10-24 11:21:37.934 DEBUG (MainThread) [bleak_retry_connector] Eve Room 0881 [EB:4A:D4:93:68:EF] (id=6C:D6:F8:C8:96:56) - EB:4A:D4:93:68:EF: Failed to connect: No backend with an available connection slot that can reach address EB:4A:D4:93:68:EF was found, backing off: 1.5 (attempt: 3, last rssi: -57)
2022-10-24 11:21:37.937 DEBUG (MainThread) [bleak_retry_connector] Eve Room 0881 [EB:4A:D4:93:68:EF] (id=6C:D6:F8:C8:96:56) - EB:4A:D4:93:68:EF: Connecting (attempt: 4, last rssi: -57)
2022-10-24 11:21:37.937 DEBUG (MainThread) [bleak_retry_connector] Eve Room 0881 [EB:4A:D4:93:68:EF] (id=6C:D6:F8:C8:96:56) - EB:4A:D4:93:68:EF: Failed to connect: No backend with an available connection slot that can reach address EB:4A:D4:93:68:EF was found, backing off: 1.5 (attempt: 4, last rssi: -57)
2022-10-24 11:21:37.939 DEBUG (MainThread) [bleak_retry_connector] Eve Room 0881 [EB:4A:D4:93:68:EF] (id=6C:D6:F8:C8:96:56) - EB:4A:D4:93:68:EF: Connecting (attempt: 5, last rssi: -57)
2022-10-24 11:21:37.940 DEBUG (MainThread) [bleak_retry_connector] Eve Room 0881 [EB:4A:D4:93:68:EF] (id=6C:D6:F8:C8:96:56) - EB:4A:D4:93:68:EF: Failed to connect: No backend with an available connection slot that can reach address EB:4A:D4:93:68:EF was found, backing off: 1.5 (attempt: 5, last rssi: -57)

Looks like we aren't actually backing off though
It is calculating it right now though


Add energy_distance

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

Project coverage is 100.00%. Comparing base (0ea8c67) to head (448d6e9).
Report is 13 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #440   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          100       101    +1
  Lines         2549      2567   +18
  Branches       156       157    +1
=========================================
+ Hits          2549      2567   +18

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


Pagination should have  semantics since it represents an act of navigation

Description of the bug
The HTML of the Pagination-component uses <button> to represent each clickable page that the user can navigate to.
The general rule of semantics for buttons and links is that:

 initiates navigation
<button> initiates an action on the page

I suggest that the pagination component is changed to use  instead of (or in addition to) <button>.
Note that if an  the user should be able to interact with the link in every way they are used to, like opening it in a new tab.
Steps To Reproduce

Open the Pagination page in Storybook
Inspect on a link and see that it is a <button>

Additional Information
Image of the markup next to the preview of the component:

Thanks for the issue! :sparkles:
As said on Slack, I agree. However, we have to make sure the user can choose what element is rendered, some might need a, Button, NextLink, etc.
I think we need to make some changes to the component API, to make sure this is possible.
Something like this is what I am thinking, but we would probably need to investigate a bit
<Pagination>
  {/* with anchor */}
  <PaginationPrevious asChild>

      Forrige

  </PaginationPrevious>

  <PaginationItem>
    <PaginationLink asChild>

        1

    </PaginationLink>
  </PaginationItem>

  <PaginationEllipsis />

  {/* with button */}
  <PaginationItem>
    <PaginationLink href="#" isActive asChild>
      <button onClick={() => {}}>
        3
      </button>
    </PaginationLink>
  </PaginationItem>

  <PaginationNext asChild>
    <button href="#">
      Neste
    </button>
  </PaginationNext>
</Pagination>

Meeting 12.02:
Keep old Pagination, add new components .Root?, .Item, .Next/Previous, .Ellipsis etc.
Expose a new usePagination hook to aid in using pagination.
All components supports asChild, so users can change the rendered element.
<Pagination.Root> // nav
  <Pagination.Content> // ul

    <Pagination.Item> // li
      <Pagination.Previous> // button
        Forrige
      </Pagination.Previous>
    </Pagination.Item>

    <Pagination.Item> // li
      <Pagination.Link> // button
        1
      </Pagination.Link>
    </Pagination.Item>

    <Pagination.Item> // li
      <Pagination.Ellipsis /> // p
    </Pagination.Item>

  </Pagination.Content>
</Pagination.Root>


chore(hm-instances): Add page about available HM instances

chore(hm-instances): Add page about available HM instances
Changelog: None
Ticket: MEN-5970
Signed-off-by: Maciej Tomczuk<EMAIL_ADDRESS>External Contributor Checklist

🚨 Please review the guidelines for contributing to this repository.

[ ] Make sure that all commits follow the conventional commit specification for the Mender project.

The majority of our contributions are fixes, which means your commit should have
the form below:
fix: <SHORT DESCRIPTION OF FIX>

<OPTIONAL LONGER DESCRIPTION>

Changelog: <USER-FRIENDLY-CHANGE-DESCRIPTION> or <None>
Ticket: <TICKET NUMBER> or <None>

[ ] Make sure that all commits are signed with git --signoff. Also note that the signoff author must match the author of the commit.

Description
Please describe your pull request.
Thank you!

@MaciejTe there are several other places in the docs where hosted.mender.io is mentioned; please add the note about the different regions everywhere.


Next.js middleware instrumentation only supported on >=12.2.0 <=13.4.12, got 13.5.6

Are you working on increasing compatibility?
Is there any reason why they have this version limitation or could I make a fork and adapt it?

@DiegoRodriguez93 middleware now runs in worker threads. Our agent doesn't propagate async context in and out of worker threads.


I can not boot "./mtg run "

First I run "./mtg generate-secret -c google.com tls"
And then run "./mtg run ee852380f362a09343efb4690c4e17862e676f6f676c652e636f6d
Result:
fatal error: cannot initialize stats server: listen tcp <IP_ADDRESS>:3129: bind: cannot assign requested address

You have something working on 3129 port. You can try to reassign stats endpoint somewhere else.


Keyed children diff optimization

Summary of the change

Determine if the new keyed child matches the next old child (thanks to preact for the inspiration here).
If the current old child is keyed, leave it alone.
If the current old child is non-keyed, remove it.  This keeps the "mixed keyed/nonkeyed" test happy.
Increment the old children index and go back to the start of the while loop.

Benefits

Closes #660: Keyed <input> stays focused if siblings before it are removed (Proof on Codepen).  I tried writing test cases for this, but the <input> never fired a "blur" event like it did in-browser.

Better keyed list modification performance.  Results in fewer "insertBefore" calls when a keyed item is removed but other items are in the same order.  Open the "Details" below to see a screenshot from JS Framework Benchmark.  Row removal is 2x faster.  Row swap is 5x faster.

Bytes
Minified file size changes from 3.45KB to 3.51KB.
Gzipped file size changes from 1.54KB to 1.56KB (using default options in linux gzip command).

Edit:  Added benchmark screenshot.

I've done a little more experimenting to see how flexible this change is.
As the code implies, this is an optimization for single item removals.   It also handles cases like this, where single items are removed in multiple parts of the list.  Here's another demonstration which is automated:  https://codepen.io/SkaterDad/pen/WzZPxr
[ 1, 2, 3 ] ==> [ 2, 3 ]
[ 1, 2, 3, 4, 5] ==> [ 2, 4, 5]

I believe these are common operations in real apps, and are worth optimizing for.  The benchmark results are a nice bonus.
This change does not handle multiple consecutive items being removed, or if a non-keyed item is mixed in.  The diffing will go back to its current behavior and do a bunch of "insertBefore" DOM operations.
Those cases are where a more complicated diffing algorithm like the one in ivi can make better decisions, but for now I'm happy to write my views in a way that avoids those scenarios when possible.

@SkaterDad If the current old child is non-keyed, remove it. This keeps the "mixed keyed/nonkeyed" test happy.

Do we want to keep those tests happy though? 🤔

@jorgebucaran Do we want to keep those tests happy though? thinking

If you don't remove the non-keyed child at that point in the code, it stays in the DOM since cleanup doesn't happen until the end and assumes the only children you need to remove are at the end of the oldChildren array.
I think mixed keyed/non-keyed is a nice feature to support.  I do find myself only keying specific children sometimes, and it behaves intuitively.

@cjh9 ...having non-keyed elements together with keyed elements in the same parent...

This hasn't changed.

@SkaterDad I published 1.2.6 by the way. Sorry for the incredible delay. 🙇

Haha, no worries!
On Wed, May 30, 2018, 7:46 PM Jorge Bucaran<EMAIL_ADDRESS>wrote:

@SkaterDad https://github.com/SkaterDad I published 1.2.6 by the way.
Sorry for the incredible delay. 🙇
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hyperapp/hyperapp/pull/663#issuecomment-393365245,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMpDr6SMDxqEXqwdgwV_AyE_JB88uDuPks5t3z1bgaJpZM4S6N7Z
.


fix: reject message (de)serialization

BIP-61 defines response-to-msg (Reject::message in rust-bitcoin; the message that triggered the reject) to be a var_str. However, by using the CommandString it was (de)serialized as 12 byte string. This is showcased in the failing test added in the first commit using two reject messages received from an older Bitcoin Core peer. The second commit changes message to be (de)serialized as var_str.
Reject message sending has been removed from Bitcoin Core, I'm still receiving them from older peers from time to time.
gh-ref: https://github.com/rust-bitcoin/rust-bitcoin/pull/323

I was surprised that this PR passed CI, I thought we had GitHub configured to ensure every patch passed but the first patch adds a failing test?

The CI tests doesn't test every commit if you push it together.
But the policy is to have every commit passing to facilitate bisecting if needed, so @0xB10C could you please swap the commits?

Oh, awesome. Thanks for the explanation.

Wasn't aware of the bisecting policy, but makes sense. Swaped the commits (fix, then test) and edited the commit description and the OP.

Thanks @0xB10C, this all looks good to me. Don't be surprised if this doesn't merge for a few days while 0.28 is being released.

Oops, we could've gotten this into 0.28, but I missed the second ACK, sorry. Too many github notifications..

@0xB10C how important is this bugfix to you? Should we try to push out a minor rev quickly?
(Right now that's possible but might not be in a few hours, depending what we merge.)

not important, but thanks for asking! I'm using a custom branch with this and a few other changes atm anyway.


Complete FieldTrip neighbors descriptions

We've added lots of new FieldTrip neighbors (adjacency matrices) definitions and corresponding descriptions in #10749
However, for a number of the neighbors, I was unable to find a proper description, and the respective field remained blank (empty string). We should aim to get them filled.
Maybe this could be part of our upcoming sprint? @drammock I'll leave it to you to judge :)

where would folks look up the description if it's not obvious to them what it ought to be?

I took the existing descriptions from the FT website, maybe I overlooked some. Hence, my approach would be the following (in that order):

For each built-in FT neighbors file, check if it exists in the FT repository
If it does, check if there's any description on https://www.fieldtriptoolbox.org/template/neighbours/ that I maybe simply overlooked
If there isn't one, probably ask on the FT issue tracker (?) as to how those "unknown" neighbor definitions were created
If the file does not exist on the FT repository, ask the MNE developers if they know where that file comes from; potentially use the Git history feature to figure out who added the file to MNE, and when


Implement getTransactionHash()

Also transaction hashes are now deterministic with StarkNet Alpha3 (#29) lets implement a method to calculate the transaction hash given the transaction details
Reference in python:
https://github.com/starkware-libs/cairo-lang/blob/master/src/starkware/starknet/services/api/gateway/transaction_hash.py
(not as important, and maybe depending on #32

@janek26
I already did it in our bridge (StarkWare).
TAL here:
https://github.com/starkware-libs/starknet-bridge/blob/dev/src/utils/hash.js#L6
The only thing that is actually missing is to calculate dynamically the tx hash prefixes from the chain ID and the transaction ID.
In python it's been done by b function and then from_bytes function. From my searches, I didn't find something equal in Javascript that gave me the same outputs.
For now, I'm saving it hard-coded here:
https://github.com/starkware-libs/starknet-bridge/blob/dev/src/enums/ChainType.js#L9
https://github.com/starkware-libs/starknet-bridge/blob/dev/src/enums/ChainType.js#L17
https://github.com/starkware-libs/starknet-bridge/blob/dev/src/constants.js#L5

Hey Dan,
I think https://github.com/seanjameshan/starknet.js/blob/main/src/utils/shortString.ts is what you are looking for. It returns the same as b from python afaik

solved in #166


OSError: Error no file named diffusion_pytorch_model.bin found in directory /content/drive/MyDrive/sd-ma/MagicAnimate/appearance_encoder.

the file provided in the magicanimate repo is a safetensors file, how do I make it look for that instead of a bin file?

Files had not downloaded completely


frontend_restructured

Frontend folder restructured for better environment.
i have restructured the frontend folder for better environment. And also installed common dependencies for the application, that will enhance the project UI on development.
Changes Made

[ ] change directory into projectspark-frontend >>>>
[ ] use the cd command and navigate into projectspark_dashboard-main which in the main directory.

Pls you can now take up with the frontend file structure and layout

[ ]  app directory is important for file base routing,
[ ] implement components with route within app folder

Additional Information
I will be happy  to get feedback, in any issues or suggestion.

Hey @OLINYA-PAUL . I see you've made the requested changes.
Can you update/customise the frontend readme file so it briefly explains the updated structure, and then I'll merge the PR

the README was updated before pushing back to branch. pls check them, other new folders where added e.g constants and others where modified.

the README was updated before pushing back to branch. pls check them, other new folders where added e.g constants and others where modified.

Okay, please can you help me confirm what the Dashboard folder is for?
I'm guessing it's supposed to be under the components


How to bundle and make a static asset linkable/downloadable (docs theme)

Hi, I have my pages/something directory with index.mdx and _meta.json in it and I dropped in another file, a JSON OpenAPI spec file.
Now I am trying to make that file linkable/downloadable using this Markdown: [Download](./file.json). This generates a localhost:3000/file.json URL (I would expect it to be /test/file.json) and when accessing that URL, I get a 404.
I tried to figure out the right path by looking at what happens when I include a MarkDown image but since that is processed by next/image I get some special URL applicable only to images so it is not going to work for arbitrary files.
I also tried importing the file like an ES module to see if it maybe imports it as a string: import file from './file.json' as well as using import assertions: import file from './file.json' assert { type: 'json' } but neither worked and both fail at runtime.
Next I figured since Nextra is built on Next I could follow the advice on https://nextjs.org/docs/app/building-your-application/optimizing/static-assets. Finally, this clicked! I guess I could have thought of this sooner but I think maybe more people will get confused by this so I wonder if this is worth mentioning in the docs.
Would you accept a contributing to the docs showing how to do this?

put your json file in public folder and change URL accordingly


⚠ Bofrost.com has degraded performance

In f93b918, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 1211 ms

Resolved: Bofrost.com performance has improved in a37257c after 9 minutes.


Conflicting with GSAP's CSSPlugin

Hey @BrentonCozby, when I load this plugin alongside with CSSPlugin from GSAP it throws a 'classList' of undefined error.

Hey @virajsoni06 , thanks for the comment. I'll look into it this week.

@virajsoni06 I'm not familiar with GSAP, but I've included the CDN in a script tag and the error comes from "TweenLite" (GSAP), not dom-slider, so I am unable to figure out what is causing the error.
Please give me more information about the error you see.

Hey @BrentonCozby, yeah the error is coming from TweenLite. More specifically it happens when clearProps property is used.
I spent some time looking into dom-slider's code and haven't able to figure out exactly whats wrong. But it's likely that it is related to Object.prototype.slideDown function
Will give it a go again sometime next week and see if I can figure out the issue

Hey, I'm refactor dom-slider to be an independent module. Its method are no longer built on the Object prototype chain. These are breaking changes (different API).


Fix showing error when type passed into refcount is wrong.

Motivation and Context
There is a segmentation fault when argument passed to the refcount function is string or int.
Approach
The change was to convert the argument to a string version, like it's expected by the %S format flag.
How Has This Been Tested?
There is a new test which fails without the change.

This looks good. Pls fix the linting error and squash to one change and I'll merge. Thx.


Question: Keeping Spells Specific to a Wand

I promise I searched first, so I apologize if this is a stupid question.
We've created a wand (BrewingGuide) that serves as a recipe book for our brewing plugin. On use the wand only casts one spell, recipeselector, which branches into categories and eventually spells that bring up the recipe in a book GUI (facilitated by another plugin). Most of the recipes are locked and can be purchased with game currency within the selector, from NPCs, or won in quests. All of that: the custom wand itself, the unlocking, the casting, works exactly like we want it to.
The problem we can't figure out is that every time a new recipe spell is unlocked for the custom wand, it is also added to the default wand inventory. They can't cast the spell from their default wand because the spell requires them to be holding the BrewingGuide, but it's there taking up inventory space.
We did create a separate path for it. I don't really understand the classes thing, though, so I'm guessing it's something I'm missing there? I tried experimenting with that but all it did was automatically add all of the recipe spells to the default wand! All help is much appreciated!

Ah yes sorry you would probably need to make separate classes for the two wands. Classes are basically the things that "remember" what spells you've unlocked, and how much mana you have.
So if two wands are using the same class, they will share the same spells.
However it may be hard to change that at this point since players already have a mix of both types of spells on their base class (probably the builtin mage class?)
So another approach, since you had the foresight to create separate paths, is to add this tag to your base wand:
wand:
  limit_spells_to_path: true

That will hide any spells the player knows that aren't part of the basic wand path.
Let me know if that helps!

(Also really sorry for how long it took me to respond to this one, I somehow missed it completely...)

Ah, cool, yeah that makes sense. I will try this, and thank you!

So limit_spells_to_path: true did work, but then we ran into the problem of not being able to add other random spells, e.g. those earned in events, so for any future searchers, as a workaround we moved all of the "extra_spells" in the paths to the main spells within the path, added our event spells to the extra_spells, then set show_extra_spells: false in the spellshop. This successfully hid any event spells we'd want to use while still allowing them on the wand.
Seems like classes would have been the simplest way to go, but this seems to have worked out. Thanks again!


Invalid Wallet Provider [BUG]

### Describe the bug
Invalid wallet provider
at new PrimeSdk
Expected behavior
Im trying to inistantiate the prive sdk with "npm run 01-get-address"
`
import { PrimeSdk } from '../src';
import * as dotenv from 'dotenv';
dotenv.config();
async function main() {
// initializating sdk...
const primeSdk = new PrimeSdk({ privateKey: process.env.WALLET_PRIVATE_KEY }, { chainId: Number(process.env.CHAIN_ID), projectKey: 'public-prime-testnet-key' })
// get EtherspotWallet address...
const address: string = await primeSdk.getCounterFactualAddress();
console.log('\x1b[33m%s\x1b[0m', EtherspotWallet address: ${address});
}
main()
.catch(console.error)
.finally(() => process.exit());
`
Steps to reproduce
No response
Additional context
No response
Operating system
macOS
Prime SDK version or commit hash
1.3.13

 ### Describe the bug
Invalid wallet provider at new PrimeSdk
Expected behavior
Im trying to inistantiate the prive sdk with "npm run 01-get-address" ` import { PrimeSdk } from '../src'; import * as dotenv from 'dotenv';
dotenv.config();
async function main() { // initializating sdk... const primeSdk = new PrimeSdk({ privateKey: process.env.WALLET_PRIVATE_KEY }, { chainId: Number(process.env.CHAIN_ID), projectKey: 'public-prime-testnet-key' })
// get EtherspotWallet address... const address: string = await primeSdk.getCounterFactualAddress(); console.log('\x1b[33m%s\x1b[0m', EtherspotWallet address: ${address}); }
main() .catch(console.error) .finally(() => process.exit());
`
Steps to reproduce
No response
Additional context
No response
Operating system
macOS
Prime SDK version or commit hash
1.3.13

hello there,
thanks for creating issue on GitHub.
Have you set WALLET_PRIVATE_KEY in .env and CHAIN_ID ? Keep in mind priv key should start with 0x...

 ### Describe the bug
Invalid wallet provider at new PrimeSdk
Expected behavior
Im trying to inistantiate the prive sdk with "npm run 01-get-address" import { PrimeSdk } from '../src'; import * as dotenv from 'dotenv'; dotenv.config(); async function main() { // initializating sdk... const primeSdk = new PrimeSdk({ privateKey: process.env.WALLET_PRIVATE_KEY }, { chainId: Number(process.env.CHAIN_ID), projectKey: 'public-prime-testnet-key' }) // get EtherspotWallet address... const address: string = await primeSdk.getCounterFactualAddress(); console.log('\x1b[33m%s\x1b[0m',EtherspotWallet address: ${address}); } main() .catch(console.error) .finally(() => process.exit());
Steps to reproduce
No response
Additional context
No response
Operating system
macOS
Prime SDK version or commit hash
1.3.13

hello there, thanks for creating issue on GitHub. Have you set WALLET_PRIVATE_KEY in .env and CHAIN_ID ? Keep in mind priv key should start with 0x...

It's working now, but it should mentioned in that prime sdk docs..

 ### Describe the bug
Invalid wallet provider at new PrimeSdk
Expected behavior
Im trying to inistantiate the prive sdk with "npm run 01-get-address" import { PrimeSdk } from '../src'; import * as dotenv from 'dotenv'; dotenv.config(); async function main() { // initializating sdk... const primeSdk = new PrimeSdk({ privateKey: process.env.WALLET_PRIVATE_KEY }, { chainId: Number(process.env.CHAIN_ID), projectKey: 'public-prime-testnet-key' }) // get EtherspotWallet address... const address: string = await primeSdk.getCounterFactualAddress(); console.log('\x1b[33m%s\x1b[0m',EtherspotWallet address: ${address}); } main() .catch(console.error) .finally(() => process.exit());
Steps to reproduce
No response
Additional context
No response
Operating system
macOS
Prime SDK version or commit hash
1.3.13

hello there, thanks for creating issue on GitHub. Have you set WALLET_PRIVATE_KEY in .env and CHAIN_ID ? Keep in mind priv key should start with 0x...

It's working now, but it should mentioned in that prime sdk docs..

good point! we gonna improve the docs. thank you!


Basic transformation

Codecov Report

Merging #5 into dev will increase coverage by 41.33%.
The diff coverage is 87.14%.

@@              Coverage Diff              @@
##                dev       #5       +/-   ##
=============================================
+ Coverage     34.61%   75.95%   +41.33%
- Complexity       18      125      +107
=============================================
  Files            12       22       +10
  Lines           104      420      +316
  Branches          4       38       +34
=============================================
+ Hits             36      319      +283
- Misses           67       93       +26
- Partials          1        8        +7

Impacted Files
Coverage Δ
Complexity Δ

...oprogramming/lab5/exception/NoCameraException.java
100% <100%> (ø)
2 <2> (?)

...labs/introtoprogramming/lab5/geometry/Vector3.java
100% <100%> (+100%)
19 <18> (+19)
:arrow_up:

...oprogramming/lab5/scene/BasicRaytracingRender.java
100% <100%> (ø)
10 <10> (?)

...abs/introtoprogramming/lab5/geometry/Matrix44.java
100% <100%> (ø)
18 <18> (?)

...ava/labs/introtoprogramming/lab5/geometry/Ray.java
100% <100%> (ø)
9 <9> (?)

...java/labs/introtoprogramming/lab5/object/Disk.java
100% <100%> (ø)
5 <5> (?)

.../labs/introtoprogramming/lab5/scene/Transform.java
44.44% <44.44%> (ø)
5 <5> (?)

...ava/labs/introtoprogramming/lab5/scene/Camera.java
50% <50%> (+50%)
3 <3> (+3)
:arrow_up:

.../java/labs/introtoprogramming/lab5/scene/Mesh.java
75% <75%> (ø)
3 <3> (?)

...labs/introtoprogramming/lab5/scene/BasicScene.java
75% <75%> (ø)
7 <7> (?)

... and 14 more

Continue to review full report at Codecov.

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

 PR#5 (Basic transformation)


JBPM-5144: code dealing with MetaDataType extracted into separate met…

…hods
Cherry pick of #283

Prepared for review.

+1


adding docstrings to apigw.py

adding docstring examples to apigw module

i assume these are tested?, or lifted from the extant unit tests?

All of the examples have all been vetted and function correctly. I'll go back through and correct the policy names & look into the formatting...

tested/validated the alternate method used here works exactly like previous for the patch values


need to add UI controllers using drag and drop

i need to convert this demo in form builder http://kelp404.github.io/angular-form-builder/  like we have in this demo.
But in this demo we don't have feature of container. so please suggest me how i can convert this demo to form builder with container and item form.
In your demo i am not able to add more UI controller instead of list only.

That's a very broad question, I'll probably not gonna be able to help you with that. You'll need to understand how the directive works from studying the demos, and then think about how to apply it to what you want to do...


Context.sess should be public ?

Hi,
I'm implementing an SMPP server, and my server is waiting for ESME connections.
So I wrote the following handler :
	sessConf := smpp.SessionConf{
		SystemID: systemID,
		Handler: smpp.HandlerFunc(func(ctx *smpp.Context) {
			switch ctx.CommandID() {
			case pdu.BindTransceiverID:
				// store ctx.Sess to reuse it for sending Delivery Receipt and MO
			case pdu.SubmitSmID:
				// manage MT
			}
		}),
	}

In the case of BindTransceiverID PDU, I would like to be able to store context session, in order to reuse it later (when sending back Delivery Receipt for example). To do that, I should have a way to access to the session of the context. I can create a Pull Request to make it public if you'd like.
Maybe I missed something, but I didn't find a way to send my DeliverSM PDU.
Thanks,
Emmanuel

@goten4

@goten4 It's solved

Hi @goten4 , Do you have any way to solve this problem？

Hi @freedom9527, I'm using this library in a fake SMPP server, so I forked and made the Context.Sess variable public.
I'd prefer a better approach :)

I also make a fake SMPP server, It can send DeliverSM PDU to the client after fixed #10 ,Thanks @goten4

Hi Freedom9527,
Could you please help how you are able to trigger DeliverSM PDU to client

Hi Freedom9527,
Could you please help how you are able to trigger DeliverSM PDU to client

When client bind transceiver, made the  Context.Sess variable public.


Tooltip improvements & copy change

Summary
This PR closes #179, it adds support for adding  tags to tooltips and prevents the tooltip from being deactivated when you move your cursor to the tooltip itself.

I couldn't find a good neutral page that explains what a block explorer is so I left the original url to blockstream in place. Feel free to modify this in a separate PR if you have a better alternative.

@GBKS, @satsie or @ecurrencyhodler please confirm whether the copy for the tooltip is okay, feel free to suggest better alternatives.

Let's try this copy and url:
"A block explorer is a useful tool to quickly look up information about bitcoin transactions."
Also does your code also resolve this issue mobile tooltips? https://github.com/saving-satoshi/saving-satoshi/issues/220

A block explorer is a useful tool to quickly look up information about bitcoin transactions.

Made the requested change. No it does not fix #220, I can address that issue in a separate PR.

Noticed an issue with the tabs in the Topbar, converting this to a draft until issue has been resolved.

Should be ready to review now

Tested. Sizing of the expanded tooltip looks great.
I think the pr does address #220. I am now able to click on the tooltip for mobile. Then when I tap on "block explorer" in the tip it takes me to the website.

Tested. Sizing of the expanded tooltip looks great.
I think the pr does address #220. I am now able to click on the tooltip for mobile. Then when I tap on "block explorer" in the tip it takes me to the website.

Oh great, then I think I may have fixed the issue by accident 😆

LOLL our boy kevin here is so efficient he does it by accident!
Okay let's get one code review and then we can merge.

Sweet, that looks great.
A small addition would be to find a solution for accessibility. Requires a bit of mark-up and potentially that the tool tip is always visible in the HTML (not 100% on that one). Some more info here.

Sweet, that looks great.
A small addition would be to find a solution for accessibility. Requires a bit of mark-up and potentially that the tool tip is always visible in the HTML (not 100% on that one). Some more info here.

Thanks @GBKS, what markup would be required to make this more accessible?
This is what our tooltip currently looks like:

  A block explorer is a useful tool to quickly look up information about bitcoin transactions.

This markup is always visible in the html btw, so that should cover the second part of your remark. 😁


fix for multiple Location: headers in response

Summary
Fix/workaround for servers that return multiple Location headers.
Motivation
Make Mojo::UA work with servers that return multiple Location headers.
References
When a server returns multiple Location headers (which it shouldn't, but some do), Mojo::UserAgent redirects to a URL that is made of the multiple URLs in the headers concatenated. Example:
MOJO_USERAGENT_DEBUG=1 perl -MMojo::UserAgent -e 'Mojo::UserAgent->new->max_redirects(2)->get("http://www.lvz.de/Leipzig/Polizeiticker/Polizeiticker-Leipzig/Amokdrohungen-gegen-vier-Leipziger-Schulen-Polizei-sichert-Gebaeude")'
This fix simply uses the first URL for the redirect.

Thanks for the PR.
For this PR to be considered, there must at least be a test, and preferably some documentation.

Thanks, fixed. https://github.com/kraih/mojo/commit/01a86c9f2bace0a92c71f4b1c8cf0da8272176e3


Debug issue with github not using cache for building docker images

The docker builds actions are not using cache for building the images, the actions tab shows the cache was accessed, but the builds are ignoring it.
Tasks:

Investigate cache issues
modify the workflows to resolve the issues
test them out on a fork

@rithviknishad Hey, I would like to work on this issue.

@sainak Hi, I would like to work on this issue.

Hey @himanshu-sharmav are you working on this issue ? If not can i try ?

I'm working on this one but down with fever.

@sainak Can I work on this issue?


Migrate website/documentation into main git repo

We currently have a separate git repository for the website and documentation. This makes it somewhat difficult to keep track of whether or not new features have been properly documented, which issues changes to the website/documentation belong to, etc.
I'd like to propose that we move the rdf4j-doc contents into the main git repository, so that code changes and accompanying documentation changes can be solved in the same pull request if we want. It will also make updating the published javadoc easier.

One thing I haven't fully figured out yet is how we handle (continuous) deployment of the website in a single repo. Currently, the website is deployed straight from the master branch (the website host server runs a git pull and a site build on a regular schedule), but if we start handling things in a single repo, we don't (always) want documentation updates for things that haven't yet been released.
Though perhaps, if master is always used for bug fixes only (including "bug fixes" in the website/documentation) this isn't a big deal.

Scheduled to start on this after the 3.3.0 release, to minimize the number of parallel branches I will have to transport.

Main branch merged. Website server reconfigured to use the main rdf4j repo to generate the site. We're migrated.


Add SavePlotLogs setting

In reference to #17 and #32
I have added a SavePlotLogs setting that writes to ~/.chia/mainnet/plotter/plotng_log_ID.txt. This should be cross platform as it uses os specific separators when creating the file.

Thanks for the PR.
I do have a suggestion, if we have config "SavePlotLogDir" in the json file.  If the user put a valid directory path there then the log files is created there.  The default should be empty string which means no log files is created.

Just for clarification, do you mean replace SavePlotLogs with SavePlotLogDir?

Yes, I do.  I think it is the simplest and most flexible solution.

Should be all set. I'm going to test overnight and will report in the morning.


Move ss58-registry into its own crate

The ss58-registry currently exists as the following json file: https://github.com/paritytech/substrate/blob/master/ss58-registry.json and once in code here: https://github.com/paritytech/substrate/blob/master/primitives/core/src/crypto.rs#L486
As it is generally a bad idea that we have two places for the same information, we should merge them. This was the idea from the beginning of the json file. So the following needs to be done:

Create a new crate in a separate repo (the name of the repo and crate needs to be decided on).
The repo contains the json file and this crate.
The crate parses at compile time the json file and generates the proper Ss58AddressFormat enum.
Release and integrate this crate into substrate.

Great idea.
On Sat, 4 Sep 2021 at 20:23, Bastian Köcher @.***>
wrote:

Assigned #9696 https://github.com/paritytech/substrate/issues/9696 to
@gilescope https://github.com/gilescope.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/paritytech/substrate/issues/9696#event-5256772031,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGEJCBQPPWN6RBF2SID2DDUAJW43ANCNFSM5DNSMD3Q
.


Feature request: support temporary AWS credentials

Some of our accounts use SSO to login into AWS. Would be great if basti can support this flow.
This should be possible with a aws sso login command.

How would you implement it in terms of UX?
My idea was to add a new flag to CLI commands --login-sso <profile-name> which when used will start SSO session before anything else.
What do you think?

We could try wrapping the AWS CLI SSO functionality with Basti, but it isn't a trivial task. Instead, we could leave credentials management outside of Basti's area of responsibility and continue using existing credentials from the user's environment.
The user can set up his or her environment separately using the aws sso login command. So it will be entirely up to the user, what credentials to use:

AWS CLI default or named profile
AWS_KEY_ID and AWS_SECRET_KEY environment variables
Temporary credentials obtained using aws sso login

Okay, let me try to set up sso in our org to see how it works and how difficult it would be to use.
I would vote for wrapping it into the basti though to make end-users UX better.
I believe it may be used by different audience, e.g. I would like our support team to use it and they may not be as technical.
Up to you to decide though 😄

I've just quickly set up SSO on my personal account. Here're the steps users will have to take to set up their environment with SSO credentials:

Install AWS CLI

Run aws configure sso [--profile <profile-name>] and enter couple of configuration values:

Use Basti: [AWS_PROFILE=profile-name] basti connect

In case temporary credentials expire, refresh them with aws sso login

The profile-related options wrapped in square brackets ([]) are entirely optional if the user doesn't want to maintain multiple profiles on his machine.

Gotcha. Then it's not too bad 👍

Please, share your experience after establishing SSO and Basti for your organization!

This seems fine. I don't think we need anything special if you implement https://github.com/BohdanPetryshyn/basti/issues/12


Batch configuration elements by type

Configure multiple element settings selected by element type

This should be able to achieve by a simple loop, or list comprehension.


README.md: outdated install instructions, GOROOT setting is too prominent

CL c497c9ea4bbe90833ee0bb8660fe91b8f17adee7 updated the installation instructions in doc/install.html to clarify that GOROOT should only be set when using a custom install path. This fixed #6613 (doc: clarify that GOROOT is only necessary for non-standard install locations), but the README.md file (the one you see when you visit github.com/golang/go) was not updated, and the Binary distribution section still starts like this:

If you have just untarred a binary Go distribution, you need to set the environment variable $GOROOT

which is not ideal.

What if we make cmd/go use the new os.Executable API to find GOROOT
automatically? Then we can remove mention of GOROOT in all user facing
documents.

@minux Only if you require the go exec to live in the GOROOT, which is different then today.

Let's discuss the GOROOT auto-detection proposal in #18678.


README fix with the objective of fixing release pipeline ephemeral deployment

Description
README now mentions how to start the service properly.
But the real goal of this PR is to get our rule release pipeline to pass (see https://gitlab.cee.redhat.com/ccx/ccx-rules-releaser/-/blob/master/scripts/common.sh#L41). Right now, the pipeline is trying to run the tests with the latest commit with message "Bumped ccx-rules-ocp version xxx", which was a version that could not be started properly due to the issue fixed in #506.
Fixes rule release ephemeral tests deployment
Type of change

Documentation update

Testing steps
N/A
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report
Patch has no changes to coverable lines.
:loudspeaker: Thoughts on this report? Let us know!.


Separate terms and conditions for user and service

Not a fan of how this is done...
Can't we create a new url for legal stuff in case its a service?

Also a user shouldn't be able to see terms for a service

That's the point, there's separate terms of use for services and users
We just don't have the real document yet

Well they aren't seperated... DOCS contains all items

LGTM


Add interaction_element_ids to Component trait.

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#631  👈
#582 : 1 other dependent PR (#589 )
#573 : 1 other dependent PR (#574 )
#572
#562
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alonh5 and the rest of your teammates on  Graphite

Codecov Report
Attention: Patch coverage is 0% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 94.41%. Comparing base (c2d33be) to head (a8fa06c).

Files
Patch %
Lines

crates/prover/src/core/prover/mod.rs
0.00%
3 Missing :warning:

Additional details and impacted files
@@                                 Coverage Diff                                 @@
##           04-17-remove_wide_fibonacci_boundary_constraint     #631      +/-   ##
===================================================================================
- Coverage                                            94.44%   94.41%   -0.04%
===================================================================================
  Files                                                   69       69
  Lines                                                 8765     8768       +3
  Branches                                              8765     8768       +3
===================================================================================
  Hits                                                  8278     8278
- Misses                                                 417      420       +3
  Partials                                                70       70

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


Titan-core fails when an error is encountered (e.g. feature not available in this edition/license)

Thanks for your work here!
I'm testing Titan in an Snowflake "Trial Account".
When executing: python -m titan export --all --out=test.yml, titan fails not too gracefully due to some features that are not available (e.g. compute pools)
─❯ python -m titan export --all --out=test.yml
[OWNER:ACCOUNTADMIN] > SHOW ALERTS IN ACCOUNT    (0 rows, 0.34s)
[OWNER:ACCOUNTADMIN] > SHOW API INTEGRATIONS    (0 rows, 0.08s)
[OWNER:ACCOUNTADMIN] > SHOW AUTHENTICATION POLICIES IN ACCOUNT    (0 rows, 0.34s)
[OWNER:ACCOUNTADMIN] > SHOW CATALOG INTEGRATIONS    (0 rows, 0.07s)
[OWNER:ACCOUNTADMIN] > SHOW COMPUTE POOLS    (err 395054, 0.22s)
Traceback (most recent call last):
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/titan/client.py", line 77, in execute
    result = cur.execute(sql_text).fetchall()
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/snowflake/connector/cursor.py", line 1080, in execute
    Error.errorhandler_wrapper(self.connection, self, error_class, errvalue)
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/snowflake/connector/errors.py", line 290, in errorhandler_wrapper
    handed_over = Error.hand_to_other_handler(
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/snowflake/connector/errors.py", line 345, in hand_to_other_handler
    cursor.errorhandler(connection, cursor, error_class, error_value)
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/snowflake/connector/errors.py", line 221, in default_errorhandler
    raise error_class(
snowflake.connector.errors.ProgrammingError: 395054 (22023): Snowpark Container Services feature COMPUTE POOL not available for trial accounts

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File<EMAIL_ADDRESS>line 87, in _run_code
    exec(code, run_globals)
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/titan/__main__.py", line 5, in <module>
    titan_cli()
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/titan/cli.py", line 130, in export
    resource_config = export_resources(exclude=_parse_resources(exclude))
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/titan/operations/export.py", line 23, in export_resources
    config.update(export_resource(session, resource_type))
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/titan/operations/export.py", line 31, in export_resource
    resource_names = list_resource(session, resource_label)
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/titan/data_provider.py", line 1858, in list_resource
    return getattr(__this__, f"list_{pluralize(resource_label)}")(session)
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/titan/data_provider.py", line 1903, in list_compute_pools
    show_result = execute(session, "SHOW COMPUTE POOLS")
  File "/PATH_TO_PROJECT/titan/.env/lib/python3.9/site-packages/titan/client.py", line 87, in execute
    raise ProgrammingError(f"failed to execute sql, [{sql_text}]", errno=err.errno) from err
snowflake.connector.errors.ProgrammingError: 395054: failed to execute sql, [SHOW COMPUTE POOLS]

I found something similar with a database created on a share (which is fixed in this newest version).
In my mind, it could be useful to have a flag that could be set to enable it to continue after errors (maybe just summarizing the errors at the end). At least for the "export" part, I understand for the "apply" we may not be so lenient with errors.

Snowflake annoyingly doesn't provide a direct way to check account edition or trial status. This one is tricky because COMPUTE POOLS are available in standard edition just not during trial mode. And I can't automate trial mode checks without manually creating throwaway emails and accounts.
There's an undocumented system function SYSTEM$GET_TRIAL_USAGE() but I just tried it and it seems to fail.
I can write a quick fix for now.

That's a good one.
In the trial account the function select SYSTEM$GET_TRIAL_USAGE(); works correctly and returns:
{"organization_id":"1028_f3889eb6-64c0-4ac9-963a-2069224233c9","remaining_free_usage_amount":39999,"remaining_free_usage_currency":"USD","remaining_free_usage_date":1721952000,"total_allocated_free_usage_amount":40000,"total_allocated_free_usage_currency":"USD","contract_end_date":1724630400,"updated_on":1722028506}

but in a regular account (running as accountadmin) I get an error:
Object '1003_73565840-b7c6-4773-8f46-734606322507' does not exist or not authorized.

Fixed in v0.8.8. I didn't run into any other resource types that throw an error in trial accounts.


DSD-585: Add AudioPlayer Component

Fixes JIRA ticket DSD-585
This PR does the following:

Add AudioPlayer Component

How has this been tested?

Unit test and storybook

Accessibility concerns or updates

N/A

Checklist:

[x] I have updated the Storybook documentation accordingly.
[x] I have added relevant accessibility documentation for this pull request.
[x] All new and existing tests passed.

Front End Review:

[ ] Review the Vercel preview deployment once it is ready.

@EdwinGuzman @bigfishdesign13 This PR is ready to be reviewed again.
Should I create the ticket for the parseIframeEmbedCode update and potentially remove the Box wrapper on the VideoPlayer component? or maybe @bigfishdesign13 can? I am not familiar with how the process works. But either way, I am happy to work on it in future sprints.

@Toxiapo Is this PR now ready for a final review?

@bigfishdesign13 yes it is!


mac-8,28,31 bots are down

Type of Request
bug
Infrastructure Environment
devicelab
What is happening?
the bots have been dead for about 13 hours.
Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

salt states applied, all bots are back up.


can I create CODE_OF_CONDUCT.md

Sure

Resolved the ISSUE#18
PR: https://github.com/TusharAMD/Runn/pull/19/#issue-1157494983


Projects Gallery - markdown files

In order to make each of these projects more attractive, we need to find photos, screenshots or logos of them to display. Ideally these photos would all have the same aspect ratio so we don't have to get clever with cropping and art direction via CSS.
We should also include more descriptive information about each project. I copy/pasted the README.md content from the Instabus project for _projects/instabus.md. It's not well formatted yet but you can use that file as a template.
TODO

[ ] add a thumb: to the front-matter of each .md file in _projects. example
[ ] save a thumbnail for each project to assets/images. Can we try for 250x200px resolution? (open to debate)
[ ] add other details to the front-matter, like the current CfA project stage. example
[ ] add content to each project file underneath the front-matter. example

In case you care...
The way Jekyll knows about our projects is via the _project folder which is a Jekyll "collection".
Ben Balter has a nice blog post explaining how collections work.

Here's some inspiration on what a cooler more interactive project gallery could be:
http://codefordc.org/projects/
They use some javascript for filtering projects by language. Also seems to be sorted by recent github activity. They are using a lot of Github data via the API.

overtaken by events


Reenable eslint on build command

Description
On build now with NEXT.js 11, eslint runs as part of the build. Currently it's erroring out with the following lint errors. Reenable linting on builds and fix the lint errors.
In next.config.js remove
-  eslint: {
-    // Warning: Dangerously allow production builds to successfully complete even if
-    // your project has ESLint errors.
-    ignoreDuringBuilds: true,
-  },

protege on  feature/upgrade-to-nextjs-11 [$!] via  v14.17.3 took 18s
❯ y build
yarn run v1.22.10
$ next build
info  - Loaded env from /Users/yolo/dev/protege/.env.local
info  - Using webpack 5. Reason: Enabled by default https://nextjs.org/docs/messages/webpack5
info  - Checking validity of types

Failed to compile.

./pages/_app.js
58:7  Warning: Unexpected console statement.  no-console

./pages/account-select.js
1:10  Error: 'useState' is defined but never used.  no-unused-vars
1:20  Error: 'useEffect' is defined but never used.  no-unused-vars
2:8  Error: 'Link' is defined but never used.  no-unused-vars
33:11  Error: Missing an explicit type attribute for button  react/button-has-type
47:11  Error: Missing an explicit type attribute for button  react/button-has-type
59:11  Error: Missing an explicit type attribute for button  react/button-has-type

./pages/candidate/[displayName]/dashboard.js
14:42  Error: Expected to return a value at the end of arrow function.  consistent-return

./pages/candidate/[displayName]/edit-profile.js
25:44  Error: Expected to return a value at the end of arrow function.  consistent-return

./pages/candidate/[displayName]/index.js
4:42  Error: Expected to return a value at the end of arrow function.  consistent-return

./pages/company/[displayName]/[jobId]/[applicationId].js
15:44  Error: Expected to return a value at the end of arrow function.  consistent-return

./pages/company/[displayName]/[jobId]/edit.js
24:28  Error: Expected to return a value at the end of arrow function.  consistent-return

./pages/company/[displayName]/[jobId]/index.js
18:35  Error: Expected to return a value at the end of arrow function.  consistent-return
170:17  Warning: Dangerous property 'dangerouslySetInnerHTML' found  react/no-danger
186:17  Warning: Dangerous property 'dangerouslySetInnerHTML' found  react/no-danger

./pages/company/[displayName]/edit-profile.js
25:42  Error: Expected to return a value at the end of arrow function.  consistent-return

./pages/company/[displayName]/post-a-job.js
27:32  Error: Expected to return a value at the end of arrow function.  consistent-return

./pages/company/[displayName]/thanks.js
10:30  Error: Expected to return a value at the end of arrow function.  consistent-return

./pages/forgot-password.js
15:9  Error: 'router' is assigned a value but never used.  no-unused-vars
17:10  Error: 'loading' is assigned a value but never used.  no-unused-vars
40:14  Error: 'error' is already declared in the upper scope on line 19 column 10.  no-shadow

./pages/get-in-touch.js
19:10  Error: 'inputs' is assigned a value but never used.  no-unused-vars
73:14  Error: 'response' is defined but never used. Allowed unused args must match /^_/u.  no-unused-vars

./pages/job-board/index.js
9:10  Error: 'activeJobs' is assigned a value but never used.  no-unused-vars
9:22  Error: 'setActiveJobs' is assigned a value but never used.  no-unused-vars

./pages/learning-resources.js
55:3  Error: propType "learningResources" is not required, but has no corresponding defaultProps declaration.  react/require-default-props

./pages/sign-in.js
16:11  Error: 'currentUser' is assigned a value but never used.  no-unused-vars
17:10  Error: 'loading' is assigned a value but never used.  no-unused-vars
34:35  Error: 'control' is assigned a value but never used.  no-unused-vars
45:14  Error: 'error' is already declared in the upper scope on line 18 column 10.  no-shadow
51:9  Error: 'handleSignInWithGithub' is assigned a value but never used.  no-unused-vars
51:41  Error: 'data' is defined but never used. Allowed unused args must match /^_/u.  no-unused-vars
55:14  Error: 'error' is already declared in the upper scope on line 18 column 10.  no-shadow

./pages/sign-up.js
19:10  Error: 'loading' is assigned a value but never used.  no-unused-vars
57:9  Error: 'handleSignInWithGithub' is assigned a value but never used.  no-unused-vars

./components/contributors/IndividualContributor.js
29:7  Error: Empty components are self-closing  react/self-closing-comp

./components/form/RadioInput.js
2:26  Error: Unable to resolve path to module 'formik'.  import/no-unresolved
4:24  Error: 'id' is missing in props validation  react/prop-types
4:28  Error: 'label' is missing in props validation  react/prop-types

./components/form/StatusBar.js
4:23  Error: Must use destructuring status assignment  react/destructuring-assignment

./components/form/TierSelectCard.js
26:7  Error: Empty components are self-closing  react/self-closing-comp

./components/global/FormCard.js
8:7  Error: Empty components are self-closing  react/self-closing-comp

./components/global/GlobalHeader.js
39:7  Warning: Unexpected console statement.  no-console
345:23  Error: Visible, non-interactive elements with click handlers must have at least one keyboard listener.  jsx-a11y/click-events-have-key-events
345:23  Error: Static HTML elements with event handlers require a role.  jsx-a11y/no-static-element-interactions
345:23  Error: Empty components are self-closing  react/self-closing-comp

./components/job/JobList.js
7:9  Error: 'jobs' is assigned a value but never used.  no-unused-vars
10:55  Error: 'useState' is not defined.  no-undef
12:3  Error: 'useEffect' is not defined.  no-undef
13:20  Error: 'fetchApplications' is defined but never used.  no-unused-vars
59:25  Error: 'JobItem' is not defined.  react/jsx-no-undef
66:14  Error: 'CompanyDashboardEmpty' is not defined.  react/jsx-no-undef
98:12  Error: 'archivedListings' is not defined.  no-undef
108:11  Error: 'archivedListings' is not defined.  no-undef

./components/user/ProfileMenu.js
20:16  Error: 'setLogo' is assigned a value but never used.  no-unused-vars

Need to disable some ESLint rules? Learn more here: https://nextjs.org/docs/basic-features/eslint#disabling-rules

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Originally posted by @nickytonline in https://github.com/drewclem/protege/issues/324#issuecomment-881900859
Related Issues
Depends on #324 being merged .

@secondl1ght, yes ut still needs to be done. Assigning it to you! 😎

Alrighty thanks, fixed with PR #339!

Oops yes you are right I missed that part, for the unused variables do you want them commented out or deleted outright?

If they are unused variables, remove them. No point on keeping dead code around.

Ok and I think I should probably try building again and getting an updated output. Because I am sure things have changed to the codebase since this lint report was generated.


Model Window Registry in the context of Data Ontology

TODO: ttlize

Data Address [the lookup key]

Windows Registry Key may-contain Windows Registry Key
Windows Registry Key addresses Windows Registry Value
Windows Registry Hive rdfs:subClassOf Windows Registry Key
Windows Sub Key rdfs:subClassOf Windows Registry Key

Data Value [data returned upon lookup]

Windows Registry Value Data queried-by Windows Registry Value Name
Windows Registry Value Data identified-by Windows Registry Value Name

Data Structure has Data Type

Complex Data Structure may-contain Data Structure

Windows Registry Value has Windows Registry Value Data

Simple Data Structure

Data Type

String
Integer
Float
Complex Data Structure

Our proposal:

Data Structure has Data Type

Simple Data Structure  - defined as having only one value

Windows Registry Value Data

Windows Registry Value Data is a Simple Data Type

Windows Registry Value Name

Windows Registry Value Name is a Simple Data Type

Complex Data Structure - defined as having multiple values

Windows Registry Key
Windows Registry Value

Windows Registry Value contains Windows Registry Value Data
Windows Registry Value contains Windows Registry Value Name
Windows Registry Value is a Complex Data Type (specifically of type VALENTW which is a struct)

Data Type

Simple Data Type - primitive types of data as defined in C

char
int
long
float
double

Complex Data Type - other types of data

Arrays
Structures
Unions
Enums
Pointers?

These ought to have multiple parents.

Does Windows Registry Key and Windows Registry Value still belong under System Configuration Database Record?

These ought to have multiple parents.

Do we need the Hive + Subkey objects under Registry Key?

Based on our prior discussion, I believe yes but we need to double check (probably the code).

Peter - Need tuples, dictionaries, sets. Java people say hash map right (gross)?


ASP.NET Core

Hi Steve,
Is this safe to use with ASP.NET Core?  I read about issues with ThreadStatic and to use AsyncLocal<T> instead.  Do you have any experience of this?
Thanks

I haven't run this particular sample's code style in a production app, and I'm not sure what you're referring to regarding ThreadStatic issues, so I guess my answer is "I don't know" ?

It seems to behave very strangely with ASP.NET Core.  I've just tried and sometimes it works fine.  If I refresh my page to keep calling task.MarkComplete() sometimes _actions grows to have multiple handlers registered, sometimes it's empty and the handler in Project doesn't fire.  I don't think I understand threading enough to use this pattern which is a shame as its very clean.

Did you try having it use AsyncLocal<T>? I haven't worked with that and your issue here was the first I've heard of it, so if you have any experience to share I'd appreciate it.
It's probably also worth updating this sample to use something else to do the message orchestration. For example, MediatR. That way we're not trying to figure out the threading and other issues, since that library's already well-proven.

Yes I think MediatR might be a better option, plus I've tried a version which uses ObservableCollection and regular events/handlers https://github.com/pauldbentley/AggregateEvents

@ardalis I was curious about this too. From what I've read, the issue is that the ThreadStatic attribute makes a static variable unique to each thread, but not to each web request, because each request can jump between different threads during its lifetime.
So the AggregateEvents.AggregateEvents._actions variable maybe isn't reliable for registering and dispatching domain events? Seems like the only reliable way to make sure static variables are unique to each request is by using Microsoft.AspNetCore.Http.HttpContext.Items.
https://stackoverflow.com/questions/4791208/threadstaticattribute-in-asp-net
https://github.com/dotnet/aspnetcore/issues/1371

Thanks. I’ll investigate. Might be that a scoped service managed by DI could work.


Cargo release removes prerelease version

It looks like it will attempt to use the wrong version here.
I'm using --no-dev-version because I'm managing the version number myself, don't want cargo release to change and make a commit. Does this make sense?
Version in Cargo.toml is:
version = "0.9.0-alpha.1"

command is: cargo release --dry-run --no-dev-version
output:
cd .
git commit  -am (cargo-release) version 0.9.0
cd -
cargo publish
git tag -a 0.9.0 -m (cargo-release)  version 0.9.0
git push origin --follow-tags

version: cargo-release-release 0.7.1

By default, cargo release increase patch version. If you want to release another alpha version, you need to specify --level alpha.

Thanks, I had missed that. I usually use it so that cargo release never makes any commits or version bumps at all, just using the currently committed version.

Maybe I can add another level option to disable version bumps.

If you think it is a reasonable thing to support. I seem to not be using the tool as intended.