"non-standard" bit fields

I have a library for integer types that don't align with bytes. It's called ux. The goal is to conform as close as possible to how the ordinary rust types work (u8, u16, u32, u64, i8, i16, i32, i64).
If BitField was implemented on these types, it would be easy to use types of length 2-63 as bit fields. Is this something that would be interesting?

Ux looks really useful! Implementing BitField would be great. We could put that implementation either in this crate or in ux. Either way, one crate would need to depend on the other.

I thought about putting the implementation inside ux, i think it makes more sense to keep it in this crate for two reasons.

uX will conform to the ordinary rust types very closely, so it's very small chance of it needing to break anything. If a dependency on bit_field was added, breakage would be required whenever compatibility was broken for bit_field.
Since a compatibility breakage in bit_field is the most probable reason for requiring a impl update it will be easier (quicker) to update the impl if it's inside bit_field.
The only thing holding back stabilization (v1.0.0) is that I want to find out if custom literals are something that will be implemented in rust in the foreseeable future. This could remove the need for the new constructor. I know this would be useful for some ffi stuff as well. But I'm not updated on the official stance. Adding bit_field as a dep will create a stabilization blocker.

Do you agree with my reasoning?

I understand your points, especially since this library is still in flux. On the other hand, this crate only defines two generic traits (and impls for the standard library types). Adding dependencies on all crates that want to implement one of these traits is not practical. The normal procedure would be that each of these crates add a dependency on the trait and provides the implementations for its types. But this only works if the trait is stable enough, otherwise you get problems when different versions of the trait are used.
I think I'm in favor of putting the implementations in this crate under an optional ux feature flag (so that no dependency is added if you don't use ux), because the ux types seem to be more standard-like at the moment. I'm happy to merge a PR that implements this.

Great!
The implementation will be pretty trivial since bit shift operators are already implemented for ux. I will write up a PR one of the next days unless someone else shows say they want to do it first.


config to allow enums in separate entrypoint

hi - love the new enum stuff <3
one thought - would it be possible to allow a config option to let us generate them into a separate file?
defualt can stay as is - but i would like the option to say i want the runtime stuff in a separate entrypoint than the types :)

Hey @juliusmarminge, great to see you here! I think this is would be a good improvement. I've left a few comments on your PR, otherwise it's looking good. 🤘
Thank you for all the good work on tRPC by the way!


Fix workspaces parse

Testing out multiple workspaces for aws-golden-gate, found a bug.
var wsNames []string
- yaml.Unmarshal([]byte(githubactions.getInput("workspaces")), wsNames)
+ yaml.Unmarshal([]byte(githubactions.getInput("workspaces")), &wsNames)

^^ this line should have dereferenced the variable so that a pointer was passed
I moved the logic to a function and added a test for both single and multi workspaces

Btw, have something cooking for this:
https://github.com/bendrucker/go-githubactions-inputs
The primitive handling is done, still working on structs. Once done you should be able to drop this in and decode all the inputs into a tagged struct.


model name is not consistent across endpoints

Feature request
Add a --served-model-name option to control the model name.
Motivation
I ran:
docker run -p 8080:8080 michaelf34/infinity:latest --model-name-or-path BAAI/bge-m3 --port 8080

Query the models endpoint:
$ curl -s http://<IP_ADDRESS>:8080/models | jq
{
  "data": [
    {
      "id": "BAAI/bge-m3",
      "stats": {
        "queue_fraction": 0,
        "queue_absolute": 0,
        "results_pending": 0,
        "batch_size": 32
      },
      "object": "model",
      "owned_by": "infinity",
      "created":<PHONE_NUMBER>,
      "backend": "torch"
    }
  ],
  "object": "list"
}

Query the embeddeings endpoint:
$ curl -s -X 'POST'   'http://<IP_ADDRESS>:8080/embeddings'   -H 'accept: application/json'   -H 'Content-Type: application/json'   -d '{
  "input": [
    "string"
  ]}' | jq | grep model
  "model": "BAAIbge-m3",

Via embeddings endpoint the model is BAAIbge-m3 for the model endpoint it is BAAI/bge-m3, Somehow would be cool to control the name.
vLLM is doing this e.g. with the following options:

--served-model-name:  The model name used in the API. If not specified, the model name will be the same as the huggingface name.
--model:  name or path of the huggingface model to use

Your contribution
I can create a PR  for this

Sounds useful to me, would be great to PR it.


Navbar overflowing on about page

On small screens, the top navbar is overflowing to multiple lines, obscuring the page content

Also happening on Pachyderm page and contact page

I've started re-writing the nav on pages/about.html such that it doesn't require bootstrap, which I think is the cause here. It it works on dev, I'll merge.


Trimming str before parsing to int and float

Change Summary
Add trim() before parsing str to both int and float.
trim() is called before length check since Python behave similarly as below. Thought to match the behavior.
long_str = ' ' + '1'*4300 + ' '
assert len(long_str) == 4302
int(long_str) # valid, no errors

invalid_str = '1'*4301
int(invalid_str) # ValueError raised

Decimal isn't included since python does handle the same situation already(Python behavior).
assert Decimal(' 123 ') == Decimal('123')

It's my first contribution, please let me know if there are any I've missed or I should follow.

Related issue number
https://github.com/pydantic/pydantic/issues/8419

Checklist

[x] Unit tests for the changes exist
[ ] Documentation reflects the changes where applicable
[x] Pydantic tests pass with this pydantic-core (except for expected changes)
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review
An additional trim downgrades performance, wonder if this trade off is acceptable.


Create Splash Screen

We need to create the splash screen. It should have the original Kovuworks logo and waves overlay for now.

Screenshots of the original splash screen for reference:


Test cases last newline issue

I found out an unintended bug, if the last Test has an endline character, it will fail with
TYPE-ERROR: The value
              NIL
            is not of type
              REAL

Make it clear in the document that all tests are last-endline expected, and the test file has no empty lines at the bottom.
I also think that we should ignore the last endline of the output, it is often not important to fail a test case.

Thank you for reporting. Should be fixed in a few days.

Fixed at https://github.com/fukamachi/getac/commit/512f9b4a15219b150e56383577919dfdd26de808.


Initial commit

I added my name "Anthony Oyakhilome Justice" to the README.md file.

Thank you Eddie.


create a pull request for HomeBrew Cask

u can copy a snippet from:

https://github.com/zeph/goad/blob/master/Casks/goad.rb

we do have automated builds also in goadapp organization at github.com/goadapp/goad/releases

This is already done. We could safely delete the hombrew recipe from our local repository and close this now.

@cwaltken-edrans sure, please proceed
i'll close this


Catalogs that are not valid in the current Rancher version are not correctly looking up available versions

Rancher Version:
v1.1.4 upgraded to v1.2.0-pre4-rc10
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
k8s
Steps to Reproduce:

Launch a kubernetes environment in Ranchr 1.1.4. Current default template is 1.2.4-rancher10

.catalog:
  name: Kubernetes
  version: v1.2.4-rancher10
  description: Rancher Kubernetes service
  minimum_rancher_version: v1.1.4
  maximum_rancher_version: v1.1.99

Upgrade Rancher server to 1.2.0-pre4-rc10. Upgrade the k8s environment

Results:
Kubernetes stack doesn't show upgrade available.
Expected:
Kuberenetes shows upgrade available for 1.4.6 template.
Note: If I set maximum_rancher_version so that it would be valid for v1.2.0-pre4-rc10, upgrade available appears.

Tested with upgrade from 1.1.4 to 1.2.0-pre4-rc12. After a Kubernetes env upgrade, "Upgrade available" option was shown. Upgrade to Kubernetes 1.4.6 was successful.


Serialization of Name component may not be working correctly

Bevy version
0.13
What you did
Modified the scene example to include Name::new("TEST").
Weirdly, I also had to .register_type::<Name>() in the example's App, despite it being registered in bevy_core.

Diff

diff --git a/examples/scene/scene.rs b/examples/scene/scene.rs
index dac36e96d..3acb19f11 100644
--- a/examples/scene/scene.rs
+++ b/examples/scene/scene.rs
@@ -8,6 +8,7 @@ fn main() {
         .register_type::<ComponentA>()
         .register_type::<ComponentB>()
         .register_type::<ResourceA>()
+        .register_type::<Name>()
         .add_systems(
             Startup,
             (save_scene_system, load_scene_system, infotext_system),
@@ -114,6 +115,7 @@ fn save_scene_system(world: &mut World) {
         component_b,
         ComponentA { x: 1.0, y: 2.0 },
         Transform::IDENTITY,
+        Name::new("TEST"),
     ));
     scene_world.spawn(ComponentA { x: 3.0, y: 4.0 });
     scene_world.insert_resource(ResourceA { score: 1 });

And observe the serialized scene in the logged output.
What went wrong
Name is serialized as:
"bevy_core::name::Name": (
  hash:<PHONE_NUMBER>44375361,
  name: "TEST",
),

Which is contrary to this comment on name:
https://github.com/bevyengine/bevy/blob/abb8c353f49a6fe9e039e82adbe1040488ad910a/crates/bevy_core/src/name.rs#L23-L29
And the stated goal of #11447.
And the migration guide text which suggests that Name will be serialized as a plain string.
Additional information
Scenes will properly deserialize without hash, but only as a struct with a name field. Not as a plain string.
See also: #1037 where we removed the migration guide entry related to #11447.

@coreh, any clues on what happened here?

Do you have the serialize feature enabled? I wonder if it's causing issues if it's not enabled

However, deserialization only works with
"bevy_core::name::Name": "TEST"
if the serialize feature is included!

Sorry for missing this, and thanks @soqb for the investigation/solution!


Uncaught TypeError: s.renderWebGL is not a function

Version

Browser: Chromium 114.0.5735.198

Description

When attempting to apply a mask to an arcade image object, the console gives the following error:
Uncaught TypeError: s.renderWebGL is not a function
    at initialize.preRenderWebGL (phaser.min.js:1:492042)
    at initialize.render (phaser.min.js:1:521596)
    at initialize.render (phaser.min.js:1:773418)
    at initialize.render (phaser.min.js:1:189563)
    at initialize.render (phaser.min.js:1:388712)
    at initialize.step (phaser.min.js:1:733432)
    at initialize.step (phaser.min.js:1:448455)
    at e (phaser.min.js:1:445807)

Example Test Code
	const obj = this.physics.add.image(50, 50, imageName);

	obj.setCircle(48, obj.body.Width, obj.body.Height);

	obj.setMask(new Phaser.Display.Masks.GeometryMask(this, mask));

Additional Information
I could be using the engine wrong somehow since this is my first time using phaser, but I don't think it should be failing internally regardless, right?

I used it wrong


UserChangeEvent.user.updated should be typed as a number

Description
The attribute UserChangeEvent.user.updated here:
https://github.com/slackapi/bolt-js/blob/main/src/types/events/base-events.ts#L796
should have the type number, as it is in web-api definition:
https://github.com/slackapi/node-slack-sdk/blob/main/packages/web-api/src/response/UsersListResponse.ts#L41
(Also, as mentioned by a TODO, UserChangeEvent.user definition should probably be shared with TeamJoin.user, short of pulling the definition from web-api. But I guess this is a larger change).
What type of issue is this? (place an x in one of the [ ])

[x] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Hey @pmezard, thanks for creating this issue 🙇🏻
I've labelled this as a bug and we can keep it open to track updating the type. A question that we'll want to answer is whether this will cause problems for existing developers expecting a string type.

Talking it over with the maintainers, we don't see any major side effects and changing this type should be fine.


Iconv-lite warning: decode()-ing strings is deprecated.

Installed mercury-parser from NPM v2.0.0
Works great, thanks!
Every execution however throws an "Iconv-lite warning: decode()-ing strings is deprecated." error.
We are passing in strings per the documentation. This is fixable by converting the string to a buffer before passing it to Mercury using something along the lines of Buffer.from(htmlSource, 'utf8');
Expected behavior would be to either disallow sending strings to Mercury, to convert strings automagically to buffers, or to override the iconv error with iconv.skipDecodeWarning = true; per the iconv-lite author's recommendation

Passing html string to the parser as per the documentation  causes incorrect parsing.Some text are not properly decoded.Using Buffer with utf-8 encoding fixes the issue

I discovered the same issue. Thanks to recommendation by @arlogilbert, I was able to get my code to work using the following format:
Mercury.parse(urlToParse, { html: Buffer.from(htmlText, 'utf8') });
It also appears that the word_count result is incorrect when parsing a string of html. I noticed that the result was always 1 no matter how long the text.


Links not working inExtend Connected Field Service solutions page

There is a hyperlink in the article below
https://docs.microsoft.com/en-us/dynamics365/customer-engagement/field-service/developer/extend-connected-field-service-solutions
which refers to
https://www.microsoft.com/dynamics/crm-customer-center/create-connections-to-view-relationships-between-records.aspx
This link does not exist and does not work. This requires an update.

@veeyeskay Thanks for the feedback. The docs are updated and pushed to live. Closing the issue.


Fixed syntactic truthy semantics for 0n

fixes https://github.com/microsoft/TypeScript/issues/60320

There's also #60324, which seems like it handles more cases?


feature request: sort by due date

Those pages /servers, /shared, and so on, currently is sorted by the random ID.
It would be better those pages can be sorted by the due date so we can know which server is going to expired.

In progress: https://github.com/cp6/my-idlers/pull/70

I found this which can sort table in front-end
https://www.kryogenix.org/code/browser/sorttable/
just FYI

Thanks for that, ill stick with the backend sorting for now as a front-end library may likely struggle when the dates are formatted to human readable strings

https://www.kryogenix.org/code/browser/sorttable/#customkeys

I tried the latest version, the server order seems still fixed to me no matter how I change the order setting.

Was an issue with the price relationship have got this working https://github.com/cp6/my-idlers/commit/beaf9af041c56701f3f12c1517485a75ca4020b4

The Server page work as expected after change the order setting,
but it seems you forget to clear the cache for the Domain page.

you forget to clear the cache for the Domain page.

Fixed here https://github.com/cp6/my-idlers/commit/a20ce989dcf4940de262ff9bb89cc58252f8a46e

LGTM


Documentation is incorrect for Certificate.Issuer

The documentation states that Certificate.Issuer
Gets the name of the certificate issuer.
This is incorrect. Certificate.Issuer only returns the Common Name (CN) of the certificate issuer and not the full X509 Issuer Distinguished Name.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 808a8db4-e570-3c65-af3f-0af71c435885
Version Independent ID: 4105d558-2347-5e2f-fd59-6a733ed17e77
Content: Certificate.Issuer Property (Windows.Security.Cryptography.Certificates) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.Security.Cryptography.Certificates/Certificate.xml
Product: uwp
Technology: winrt-reference
GitHub Login: @lastnameholiu
Microsoft Alias: alholiu

Thanks for submitting your feedback and we apologize for the delayed response. I see that updates have not been made since the creation of this Issue, so I went ahead and completed an internal work item to incorporate your suggestion. The changes should go live shortly.
Though the Issue will be marked as Closed, we will monitor any replies that come afterwards so treat this like an open Issue. Many thanks!


Console spam when looking at a barrel with The One Probe

in #7b3cc90 you accidentally commited a debug-console output to master. It spams my console with "test" whenever I look at a barrel.
src/main/java/exnihiloadscensio/blocks/BlockBarrel.java:161
Method addProbeInfo
Also, for some reason the TOP integration seems to not work that well with Ex Nihilo at all, as almost all blocks display an error when I look at them, but the main issue is the console spam.
Screenshot of the error displayed in game. (It's currently of a sieve, but it doesn't matter, it's the same for every block from Ex Nihilo Adscensio)

Line in console output: [exnihiloadscensio.blocks.BlockBarrel:addProbeInfo:161]: test

Apparently there was a PR for this... #110

So the console spam is fixed, but I'm curious about the error.
I've been testing this out and have yet to get one block to error. Can you confirm your TOP version?

The One Probe 1.1x-1.4.4
If you want, I can give you the exported zip from Cursed Client which will recreate the exact environment I have on your end.
UrmitFactory-1.10.2-1.6.zip
It's roughly 200 mods in that thing and I'm surprised it even starts up :P

Just FYI, this is waiting upon https://github.com/McJty/compatlayer/issues/9


Missing models

I want test the project but in this link i get 404(NOT FOUND)
https://qurator-data.de/sbb_ned/models.tar.gz
regards.

Link has been fixed.

Link has been fixed.


Update to 1.20.5 and 1.20.6

This mod seems to be simply necessary for the game.  Correct me if this bug was fixed in the vanilla game, but I haven't heard about it.

Afaik the vanilla issue is not a bug, but rather a design flaw.
I am still considering whether to work on 1.20.5 and 1.20.6 or purely wait for 1.21, I tend to much prefer working on major releases than sub versions (also because Mojang has decided to make all their breaking changes in minor versions, which makes it a real pain to follow through)

Yes. I only have other priorities on the modding department, but it will come, possibly pretty soon even.

now when 🗿

Have actually been working on it, but I have to work out something else beforehand

Done and released on Modrinth


drivers/flash: Add flash_get_erase_val call to flash API

Add function that will return flash cell erase value set via driver, or default if user decides
to make such selection via Kconfig.
Resolves #22063

I like how code looks like now.
Is it possible to make erased_val configurable for flash simulator via DTS?
Is it then possible to add test-case with either 0xff and 0x00 erase_value for nvs test (https://github.com/zephyrproject-rtos/zephyr/blob/master/tests/subsys/fs/nvs/testcase.yaml)?

Yes. Now the sim flash will have erase-value param that allows selection of flash erase value.

@pabigot

Should the implementation support flash devices where we don't know at compile time what the erase value is for a specific device, e.g. determining it from SFDP parameters.

Driver implementation might recognize erase-value in the runtime.

Driver implementation might recognize erase-value in the runtime.

Yeah, but then the value can't be stored in the API structure, which is const and the same for all devices used by the API.  In that case we need it to be accessed through a function, not by reading the common structure.

@pabigot ^^understood, right

Maybe the idea of extending the api with a get_flash_erased_value() is not the best approach. Maybe it is better/easier to extend the api with a function flash_iserased(dev, addr, size) that would do the compare inside the flash driver.

Maybe it is better/easier to extend the api with a function flash_iserased(dev, addr, size) that would do the compare inside the flash driver.

It's true that the motivation for this API change has not been documented as clealy as it could be.
There is a use for initializing a buffer with the intent that any elements not specifically assigned otherwise will not mutate the content of the flash, essentially preserving the unerased parts.  That's currently handled in one situation by first reading the region, modifying parts, and writing the whole thing back.
Is that an approach that would solve #22063 without changing the flash API?
Even in that case knowing the erase value would be necessary to verify that the technique won't break (e.g. flash already has 0xAA instead of 0xFF and user wants to write 0xF3 to set bits 0..3, but the result would be 0xA2 which has the wrong value in bits 0..3).

@pabigot, I think this case is different, nvs is searching for the end of the fs by searching for a unwritten value. The read and rewrite to update the flash data will not help.
@pabigot, regarding the read and rewrite for mcuboot support, it might be a bad idea to support this in general, the result might be that flash wear is a lot faster than expected or the method is not supported at all by the hardware. Mcuboot afaik always states that they do not need byte access write, if mcuboot does need read and write back it would be best to add this in mcuboot.

@Laczen @nvlsianpu @pabigot I have search for information on flash and eeproms to find if there is device that does erase to non-1 value but I have failed to identify any.
There are FRAM (FeRAM) devices and MRAM, but they behave like RAM generally which means that they do not require erase prior to write.
Maybe this PR should be dropped. I can not find an example of non-1 erase value.
OK, so NVS would not work with some other erase values: yes but we do not have example of such.
In case of FRAM and MRAM, the NVS will not work on them either because they do not have "erase value", support for them  could be emulated via flash API if we add erase functions for them that would default the RAM to all 1 (which will cost us some time).

Mcuboot afaik always states that they do not need byte access write, if mcuboot does need read and write back it would be best to add this in mcuboot.

Mcuboot doesn't and won't use any read-modify-write to flash. It just need to know what the erased value is for working properly on SoC of 0x00 and 0xFF erased values

@de-nordic +1 for introducing this API.
cc @utzig

@de-nordic +1 for introducing this API.
cc @utzig

@nvlsianpu  I still voice my concern regarding other technologies of NV type memories, but I am modifying the PR to implement @pabigot suggestions regarding the callback instead of data member.  Additionally I will change it from erase specific callback to flash_get_capabilities, flash_get_params or similar, that could be used in more general way to obtain flash parameters that could be defined during compilation or run-time.
I am also looking into NVS driver to figure out how would it be possible to make it work with non-volatile memories that does not require erase prior to write.

@pabigot can you re-review please?

Maybe for memories which have not defined erase-value (or erase is not an operation for them) API can return -ENOSUP etc.

@pabigot can you re-review please?

There are no changes yet. I am still working o it. @pabigot I will let you know when I will have something to present, although I would like to hear your point of view on current support of NV type memories in Zephyr and whether flash/eeprom api shouldn't be merged under one NV API that would also cover capabilities of other type of NV memories.
Actually I would also encourage @nvlsianpu  and @Laczen, to voice their opinions on the subject as they are more experienced, than me, with the Zephyr API and NV memory usage within the system.
Thanks in advance!

I think it's something to consider but @henrikbrixandersen initiated that API and should weigh in on whether its operations can be supported with the flash API. I assume in that situation we'd need to define the expected behavior of erase: should it do nothing, or should it write a constant to remove any information present (I would propose the latter).

We had the discussion of whether to use the flash API for EEPROM devices (and whether to add an erase API function) back when the EEPROM API was introduced. Please see #19972 for details.

@de-nordic, it seems that the flash api extension is getting defined.
IMO the best way to use this extension in nvs is to extend the nvs fs structure with a pointer to the flash parameters and to set this during nvs_init. If any of the required flash parameters is missing raise a cfg error and keep the nvs flash_device = null. In the nvs routines use the nvs pointer to flash parameters  (so the get_flash_param() is only used in nvs_init and not inside any other nvs routines).

IMO the best way to use this extension in nvs is to extend the nvs fs structure with a pointer to the flash parameters and to set this during nvs_init. If any of the required flash parameters is missing raise a cfg error and keep the nvs flash_device = null. In the nvs routines use the nvs pointer to flash parameters (so the get_flash_param() is only used in nvs_init and not inside any other nvs routines).

I will look into this.

@Laczen does the changes to NVS follow your idea?

@Laczen does the changes to NVS follow your idea?

Yes, this is completely what I was thinking. I don't know if it is possible to use a const struct *flash_parameters, it might be easier to use when it's not const and gets assigned during init (and a check is made on the write_block_size).

Yes, this is completely what I was thinking. I don't know if it is possible to use a const struct *flash_parameters.

@Laczen  How do you want to use it? I do not encourage anyone to modify contents of flash_parameters outside of drivers.

Yes, this is completely what I was thinking. I don't know if it is possible to use a const struct *flash_parameters.

That should be easy to do: you return a pointer to const, but keep the structure in RAM and initialize it when the device is initialized.
I agree the API should not provide a pointer to non-const.  I'd like to document that the caller has the choice of caching the pointer, or copying the information desired to a different data structure with a guarantee it will never change.

Yes, this is completely what I was thinking. I don't know if it is possible to use a const struct *flash_parameters.

@Laczen How do you want to use it? I do not encourage anyone to modify contents of flash_parameters outside of drivers.

Exactly how you are using it. Probably my remark is a result of limited c knowledge, can you assign the const struct *flash_parameters in nvs_init() ?

Yes, this is completely what I was thinking. I don't know if it is possible to use a const struct *flash_parameters.

@Laczen How do you want to use it? I do not encourage anyone to modify contents of flash_parameters outside of drivers.

Exactly how you are using it. Probably my remark is a result of limited c knowledge, can you assign the const struct *flash_parameters in nvs_init() ?

Yes, const qualifier 'protects' here the structure as it is in the part that declares pointed type. If you want to protect pointer, you need to move const:
struct *const flash_parameters
or you can have both:
const struct *const flash_parameters

@pabigot

(...) I'd like to document that the caller has the choice of caching the pointer, or copying the information desired to a different data structure with a guarantee it will never change.

You want this in the doxygen of the flash_get_parameters() ?

(...) I'd like to document that the caller has the choice of caching the pointer, or copying the information desired to a different data structure with a guarantee it will never change.

You want this in the doxygen of the flash_get_parameters() ?

I think so, yes, assuming you and others think this is worth making clear.  I'd like to have this API raised during the API telecon today for visibility.  But also we should confirm that nobody has a use case for the values changing for a given device while the system is running.

I think so, yes, assuming you and others think this is worth making clear. I'd like to have this API raised during the API telecon today for visibility. But also we should confirm that nobody has a use case for the values changing for a given device while the system is running.

I do not think there is a case for the contents of flash_parameters to get modified at runtime. I will add the note to documentation, but we will need to wait until it gets clarified.

Yes, this is completely what I was thinking. I don't know if it is possible to use a const struct *flash_parameters.

@Laczen How do you want to use it? I do not encourage anyone to modify contents of flash_parameters outside of drivers.

Exactly how you are using it. Probably my remark is a result of limited c knowledge, can you assign the const struct *flash_parameters in nvs_init() ?

Yes, const qualifier 'protects' here the structure as it is in the part that declares pointed type. If you want to protect pointer, you need to move const:
struct *const flash_parameters
or you can have both:
const struct *const flash_parameters

OK, thanks for the clarification

NVS changes looks ok

@pabigot @nvlsianpu could you take another look please?

@Laczen @pabigot Thanks for reviewing.


daemon: create directory with correct permissions in prepareEndpointDirs

Pass permissions as octal instead of decimal value.

This change is

test-me-please

Coverage remained the same at 45.555% when pulling 9a87833c46633cc510a5ceb4821a21ef5cc79980 on pr/tklauser/daemon-policy-test-perm into 220604167c3f8795980ecdee8a7b022a43b7b9d5 on master.

@tklauser I've not seen any real CI issues which could be traced to this so far. But since the change is fairly small and might prevent potential CI issues, I'd still suggest to backport it.


Map escape to jk without mapping to jj

Is there a way to map escape to jk without first mapping jj? I understand that it's not the end of the world, but it seems odd to have to first impliment a mapping that I will never use.

Currently not possible, but can be supported

Duplicate of #563.


source maps with non-empty 'sourceroot' entries don't resolve to the correct source path

Source maps of the form:
"sourceroot": ".../../src",
"sources": [ "node/nodeDebug.ts" ]

do not resolve to the correct source.

to verify use this project.

open project in original Insiders release
launch debug session and step through the code

Observe: stepping occurs in .js file not .ts
In the stable release after a first step in .js you should step through the .ts file


Add JSX, TSX and TSConfig file templates to Node projects

Add TS, JSX and tsconfig templates for Node projects

Hi @anubmat, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Anubha Mathur). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


Bug? with "never" type

TypeScript Version:  <IP_ADDRESS>
class C
{
	public f1() {
		return 1;
	}

	public f2()
	{
		if (this instanceof Sub)
			return 2;

		return this.f1(); // Does not compile: error "f1 does not exist on type never"
	}
}

class Sub extends C
{

}

Expected behavior:
Compile without error
Actual behavior:
Error "f1 does not exist on type never"

instance of operator is out of order in TypeScript.
It uses shape comparison.
You can bypass it by adding a fake property:
class C
{
	public f1() {
		return 1;
	}

	public f2()
	{
		if (this instanceof Sub)
			return 2;

		return this.f1(); // ok
	}
}

class Sub extends C
{
    private _prop: string;
}

This issue is similar to : #11664


add left eye and right eye attributes

Script currently doesn't work without these attributes defined.

Thanks for the MR - this was made as a fun project. I think we should add some documentation of which version of the package is compatible with which DeepFace version and also some tests that will verify that it's still working as expected.


i18n(it): Updates guides/customization.mdx

Description
Updates guides/customization.mdx

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

it
guides/customization.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Insufficent instructions on ARM templating

This doc assumes the user has a measure of knowledge on how ARM templates and deployments work. If they do not, it is very easy to get lost during a deployment, especially when errors are as short as 'Expecting value: line 1 column 1 (char 0)'. I recommend that we include a link near the top to "https://docs.microsoft.com/en-us/azure/azure-resource-manager/resource-group-overview" that explains how ARM templates and az CLI work under the hood.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 096d934c-2856-e868-efeb-6622e2a5eafb
Version Independent ID: 8bf74926-6a35-5753-0a1a-6663ba5965b2
Content: Deploy your bot - Bot Service
Content Source: articles/bot-builder-deploy-az-cli.md
Service: bot-service
Sub-service: abs
GitHub Login: @ivorb
Microsoft Alias: v-ivorb

Thanks for your feedback Jessica. We've added an Azure resource manager link into the article prereqs.
You should see this change the next time our documentation updates go live.


03/28/2018 14:50 : Create Pull Request for branch prTesting_2018032814 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


TETP-75 youth can list postings

Description :sparkles:
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #865 (fe43cd0) into develop (ee80233) will decrease coverage by 62.31%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop    #865       +/-   ##
===========================================
- Coverage    62.31%   0.00%   -62.32%
===========================================
  Files          861      63      -798
  Lines        22967     946    -22021
  Branches      2344     137     -2207
===========================================
- Hits         14311       0    -14311
+ Misses        8267     916     -7351
+ Partials       389      30      -359

Impacted Files
Coverage Δ

...end/tet/admin/src/backend-api/linked-events-api.ts
0.00% <ø> (ø)

...ntend/tet/admin/src/components/editor/Combobox.tsx
0.00% <ø> (ø)

...tend/tet/admin/src/components/editor/DateInput.tsx
0.00% <ø> (ø)

...ntend/tet/admin/src/components/editor/Dropdown.tsx
0.00% <ø> (ø)

...ntend/tet/admin/src/components/editor/EditById.tsx
0.00% <0.00%> (ø)

...rontend/tet/admin/src/components/editor/Editor.tsx
0.00% <0.00%> (ø)

.../src/components/editor/EditorErrorNotification.tsx
0.00% <ø> (ø)

.../tet/admin/src/components/editor/HiddenIdInput.tsx
0.00% <ø> (ø)

...nd/tet/admin/src/components/editor/NumberInput.tsx
0.00% <ø> (ø)

...end/tet/admin/src/components/editor/PhoneInput.tsx
0.00% <ø> (ø)

... and 824 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 ee80233...fe43cd0. Read the comment docs.


Compile error

Hello, I've followed all instructions to install the wallet but receive the below error msg at compile time:
/home/trenaz/bowser-bitcoin-hardware-wallet/bowser/bowser.ino:423:11: warning: integer constant is too large for its type
     delay(99999999999999999999999);
           ^
/home/trenaz/bowser-bitcoin-hardware-wallet/bowser/bowser.ino: In function 'void seedChecker()':
/home/trenaz/bowser-bitcoin-hardware-wallet/bowser/bowser.ino:423:34: warning: large integer implicitly truncated to unsigned type [-Woverflow]
     delay(99999999999999999999999);
                                  ^
At global scope:
cc1plus: warning: unrecognized command line option '-Wno-frame-address'
In file included from /home/trenaz/Arduino/libraries/M5Stack/src/M5Display.cpp:198:0:
/home/trenaz/.arduino15/packages/esp32/hardware/esp32/2.0.0-alpha1/tools/sdk/esp32/include/esp32/include/rom/tjpgd.h:1:2: warning: #warning rom/tjpgd.h is deprecated, please use esp32/rom/tjpgd.h instead [-Wcpp]
 #warning rom/tjpgd.h is deprecated, please use esp32/rom/tjpgd.h instead
  ^
cc1plus: warning: unrecognized command line option '-Wno-frame-address'
In file included from /home/trenaz/Arduino/libraries/M5Stack/src/utility/pngle.c:31:0:
/home/trenaz/.arduino15/packages/esp32/hardware/esp32/2.0.0-alpha1/tools/sdk/esp32/include/esp32/include/rom/miniz.h:1:2: warning: #warning rom/miniz.h is deprecated, please use esp32/rom/miniz.h instead [-Wcpp]
 #warning rom/miniz.h is deprecated, please use esp32/rom/miniz.h instead
  ^
cc1: warning: unrecognized command line option '-Wno-frame-address'
In file included from /home/trenaz/Arduino/libraries/M5Stack/src/utility/Power.cpp:10:0:
/home/trenaz/.arduino15/packages/esp32/hardware/esp32/2.0.0-alpha1/tools/sdk/esp32/include/esp32/include/rom/rtc.h:1:2: warning: #warning rom/rtc.h is deprecated, please use esp32/rom/rtc.h instead [-Wcpp]
 #warning rom/rtc.h is deprecated, please use esp32/rom/rtc.h instead
  ^
cc1plus: warning: unrecognized command line option '-Wno-frame-address'
/home/trenaz/.arduino15/packages/esp32/tools/xtensa-esp32-elf-gcc/1.22.0-97-gc752ad5-5.2.0/bin/../lib/gcc/xtensa-esp32-elf/5.2.0/../../../../xtensa-esp32-elf/bin/ld: cannot find crt1-sim.o: No such file or directory
/home/trenaz/.arduino15/packages/esp32/tools/xtensa-esp32-elf-gcc/1.22.0-97-gc752ad5-5.2.0/bin/../lib/gcc/xtensa-esp32-elf/5.2.0/../../../../xtensa-esp32-elf/bin/ld: cannot find _vectors.o: No such file or directory
/home/trenaz/.arduino15/packages/esp32/hardware/esp32/2.0.0-alpha1/tools/sdk/esp32/lib/libwpa_supplicant.a(sha256.c.obj): In function hmac_sha256': /home/runner/work/esp32-arduino-lib-builder/esp32-arduino-lib-builder/build/../esp-idf/components/wpa_supplicant/src/crypto/sha256.c:122: multiple definition of hmac_sha256'
libraries/uBitcoin/utility/trezor/hmac.c.o:/home/trenaz/Arduino/libraries/uBitcoin/src/utility/trezor/hmac.c:67: first defined here
collect2: error: ld returned 1 exit status
Multiple libraries were found for "Electrum.h"
 Used: /home/trenaz/Arduino/libraries/uBitcoin
 Not used: /home/trenaz/Arduino/libraries/uBitcoin-0.1.1
Multiple libraries were found for "WiFiClient.h"
 Used: /home/trenaz/.arduino15/packages/esp32/hardware/esp32/2.0.0-alpha1/libraries/WiFi
 Not used: /home/trenaz/Downloads/arduino-1.8.13/libraries/WiFi
Multiple libraries were found for "SD.h"
 Used: /home/trenaz/.arduino15/packages/esp32/hardware/esp32/2.0.0-alpha1/libraries/SD
 Not used: /home/trenaz/Downloads/arduino-1.8.13/libraries/SD
exit status 1
Error compiling for board M5Stack-Core-ESP32.

I reinstalled everything on a fresh testing environment and everything worked out smoothly.
Thanks btw for the amazing software


Dark theme does not work with Nuxt Color Mode

Installed Nuxt Color Mode and configured the color switch button
const colorMode = useColorMode()
const isDark = computed({
  get() {
    return colorMode.value === 'dark'
  },
  set() {
    colorMode.preference = colorMode.value === 'dark' ? 'light' : 'dark'
  },
})
</script>

<template>
  <div>
    <UIButton variant="ghost" size="icon" @click="isDark = !isDark">
      <Icon :name="isDark ? 'i-heroicons-moon-20-solid' : 'i-heroicons-sun-20-solid'" color="black" />
    </UIButton>
  </div>
</template>

but the color doesn't change. How do I configure shadcn-vue to make the dark theme work?

In short, sorry, I looked more closely and discovered that I forgot to add to nuxt.config.ts
colorMode: {
    classSuffix: ""
  },

Someone must land us some help on the dark-mode documentation in Vite and Nuxt :'(

Although this is closed, this should still be added to the documentation.
I get the IDE warning of:
Object literal may only specify known properties, and 'colorMode' does not exist in type 'InputConfig<NuxtConfig, ConfigLayerMeta>'.ts(2353) when adding this option though


Remote debugging is so slow

I run my RoR app using foreman with Puma as web server. When I activate remote degugging:
  require 'byebug'
  Byebug.wait_connection = true
  Byebug.start_server('localhost', 33333)

The server slows down, it gets a request per second like if it was monothreading. Is this the expected behaviour? Can I improve the performance?
Thank you

There's already an opened issue to improve performance. Thanks.

Sorry, I can't find any other open (or closed) issue about performance.
Can you give any pointers why it happens / what it would take to improve?
cc @borod108 who is experiencing this.

Ping @deivid-rodriguez if you can shed any light/pointers on why it's slow, I'll try to give this some love.


Problem running with hibernate5:6.1.4

@betoesquivel wrote:
I managed to get this to work on grails 3 using your example, I just had to downgrade to the version 6.1.3 of hibernate5 since if I set the 6.1.4 it was trying to get the mongodb dependency 6.1.4 (which doesn't exist). I needed mongodb to work because some of my domains are mapped there. However, the ones that I want to search on are not on mongo.

I managed to make it work on hibernate5:6.1.4 with org.grails.plugins:mongodb:6.1.2
I just had to force the version in the build.gradle therefore I might miss something.
I created a branch in the Grails 3 quick start project so you could check the difference with your project:
https://github.com/lgrignon/grails3-quick-start/blob/mongotest/build.gradle#L73
Hope this helped.
Please close issue if you think it's solved

This worked, the issue I was having was that I had to clean the dependency cache. All the build/ folders and the .gradle folders. A grails clean didn't  cut it.
I am marking as solved.

I assume I cannot close the issue without a commit, can I?

Yes, you can close issues without commit but maybe you can't because you didn't create it.
I close this issue and answer to the other


Query bug

http://browserl.ist/?q=>1%25
> 1% => edge 16: 0.75% is returned

The browserl.ist website passes your query directly to browserslist, so I'd expect this to be an issue for that project or possibly the underlying caniuse data. 🤔

Live website

Local: master + freshly installed node_modules

Huh! 🤔 @jonrohan, may want to try deploying a fresh install?

Site is updated and I hope is more automatic on keeping up to date now.


Add the az login --scope parameter

Is your feature request related to a problem? Please describe.
We are using the Windows Task Scheduled to automate the execution of some powershell scripts that use the az login command.  Sometimes, we only require read-only access so we login using a read-only account.  Sometimes, write access are necessary.  The issue is that since the session scope is the same on any powershell sessions, if the two tasks run one after the other, it makes the first script crash.
Describe the solution you'd like
Having the possibility to specify the scope of the az login command similar to the Connect-AzAccount -Scope command.  It would then be able to run the two Powershell sessions with two different user context at the same time without having any impact on each of the scripts.
Describe alternatives you've considered
At the moment, we plan our scheduled tasks in order for them to never be executed at the same time if the context of the two scripts are different are might impact one another.

login

Azure CLI is only a native python.exe application, so Connect-AzAccount -Scope which is backed by PowerShell is not a possible solution in Azure CLI.
There is a similar environment variable AZURE_CONFIG_DIR you can use for concurrent builds. You may set it for different Azure CLI scripts and these scripts will  run with their own "context".

Azure CLI is only a native python.exe application, so Connect-AzAccount -Scope which is backed by PowerShell is not a possible solution in Azure CLI.
There is a similar environment variable AZURE_CONFIG_DIR you can use for concurrent builds. You may set it for different Azure CLI scripts and these scripts will  run with their own "context".


Update capabilities.md

Supported call type update

@elavarasidc
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Court72 I approve this PR. I am the original author and author of this PR.
Can you please help merge

Due to the upcoming conference, all merging and publishing are on hold until the morning of 5/23.
This PR is ready for merge on 5/23.
#label: "event-triaged-to-10AM"
#label: "missed-content-complete"

#sign-off


Proposal: Create a new schema with the types based on the describe value

Is this possible or not?
seems the types are not showing here
export const Schema= z.object({
  __typename: z.literal('ProjectMember'),
  id: z.string(),
  attributeField: z.number().optional().nullable().describe('[attribute]'),
});
const SchemaWithAttributeOnly = Schema.pick(
  mapValues(
    Schema.shape,
    item => item.description?.startsWith('[attribute]') ? true : undefined,
  ),
);

I think I may need the same thing, although I'm not entirely understanding your code. In my case my schema has a "translations" property in which I want to allow all STRING, and only string, properties of that same model. Currently I'm doing a hardcoded pick, which is not only prone to mistakes or being outdated, you also can't  "deep-pick" anything, so it's also incomplete.


Fix documentation of NStepGRUBase

This PR fixes documentation of chainer.links.NStepGRUBase.
NStepRNN and NStepBiRNN here should be NStepGRU and NStepBiGRU respectively.

pfnCI, test this please.

Jenkins CI test (for commit 85ae07ba0351a439b6b1aed33fb9899860f87076, target branch master) succeeded!

LGTM! Test failures are not related to this PR.

Jenkins CI test (for commit 85ae07ba0351a439b6b1aed33fb9899860f87076, target branch master) succeeded!

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?


Updated IntelliJ Idea CE to 2016.2.1

Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

What's going on?

Nothing wrong with the cask. The test suite has recently been changed in the main repo, but not yet in the taps, so all pull requests are currently failing.


Address field background color in user profile not match other field and it very hard to see.

The latest version has resolved this bug. Thank you for your notice!


fix: We shouldn't show error message in the end of a proposal

We should trigger the request in background without showing the error message, we can show the result directly. We should also have a way to not trigger calculate multiple time if few people request it in same time.

You mean the results ase calculating message?

Actually, with https://github.com/snapshot-labs/snapshot-sequencer/pull/273 we still show an error message at the end of proposal, we need a background task, either on sequencer of webhook to trigger score calculation on proposal end

Should already be a webhook task to trigger the computation on proposal end, to sequencer.
This pr is in case the first computation fail

Should already be a webhook task to trigger the computation on proposal end, to sequencer.
This pr is in case the first computation fail

Correct 😮  i completely forgot about the get webhook we had


RFE: is it possible to start making github releases?🤔

gh release can contain additional comments (li changelog) or additional assets like release tar balls (by default it contains only assets from git tag) however all those part are not obligatory.
In simplest variant gh release can be empty because subiekt of the sent email contains git tag name.
I'm asking because my automation process uses those email notifications by trying to make preliminary automated upgrades of building packages, which allows saving some time on maintaining packaging procedures.
Probably other people may be interested to be instantly informed about release new version as well.
Documentation and examples of generate gh releases:
https://docs.github.com/en/repositories/releasing-projects-on-github/managing-releases-in-a-repository
https://cli.github.com/manual/gh_release_upload/
https://github.com/jbms/sphinx-immaterial/pull/282
https://github.com/marketplace/actions/github-release
https://pgjones.dev/blog/trusted-plublishing-2023/
https://github.com/jbms/sphinx-immaterial/issues/281#issuecomment-1700933026
tox target to publish on pypi and make gh release https://github.com/jaraco/skeleton/blob/928e9a86d61d3a660948bcba7689f90216cc8243/tox.ini#L42-L58

All because assets are generated on top of existing git tag.
Additionally you can make tag assets verifiable.
https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-tags
Other thing is thar use of those assets as input source tar ball works perfectly fine.
If you don't care about spreading notifications about new rerelease (because create gh release only does this) that is perfectly fine as well ..

The problem is that the git tag and the generated assets from this tag are not what is released.
This repository holds the portability bits to create release tarballs (by using make distcheck) and this pulls in
the needed OpenBSD source files from a 2nd repo. So people using this repository may not get what is in the
release tarball.
This is why I dislike using the github release feature. It signals to people to use the github source
which is actually incorrect, people should use the tarballs from the official website and mirrors.

The problem is that the git tag and the generated assets from this tag are not what is released.

I don't see single change in files which are common part of VCS and dist tar ball.
distcheck after regenerate GNU auto files forms dist tar ball and unpacks it to perform build with check target execution.
As you are using autoconf 2.69 (Which is EVEVEN years old version) which do not support --rundir swich I must on build regenerate everything using autoreconf -fiv.
Because tar ball taken from tag is shorter and it guarantees that I would be possible to apply aby patch generated from commits after release I've choose to use it .. and everything is working).
Nevertheless my RFE was about spreading email notification about new release using github infrastructure.

You realize that the autoconf used to build the configure file includes --runstatedir support?
Also you could just ask us to switch to 2.71 -- which is something I wanted to do for a while.
Also be aware that a lot of files are copied into this git repository by update.sh. This script always copies the latest source from OpenBSD so you will get different builds at different times. So you need to be ultra careful if you want to build from this repo.


RHEL 8.5: tar installer

This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] create a file in news/unreleased directory if this change should be mentioned in the release news
[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

Implements a tar-installer: an installer image that embeds an OS tarball (same as the tar image type) that's installed when the ISO is booted.
I had this implemented and ready for a while based on #1351.  Rebased now on #1391.  Opening now for early review but keeping as a draft until we #1391 is merged and to add tests.

Rebased on main and added an integration test that builds an installer, installs it and checks for installed packages (I based it on ostree.sh).

@atodorov or @ondrejbudai I haven't enabled the integration test in the .gitlab-ci.yml file yet.  Is the 8.5 runner ready?

Really nice @achilleas-k !
@jikortus is this something you could use/extend as part of your work?

Rebased and resolved conflicts.

Had to update the test manifests for the edge image types introduced in #1351.  The core stages that are shared between all image types didn't include the fix-bls stage before the installer type was added.

Please, don't merge PRs that are clearly untested by the CI. For this PR, none of the tests were actually run because the CI actually failed on the RPM build step. I hope this won't break main.

Just a note why it failed on rpms: This was caused by the outage of RHSM servers.

My bad, sorry. I didn't inspect the GitLab results this time and relied on the last time I saw things were looking good.


Relax version pins

See https://github.com/conda-forge/numba-stats-feedstock/pull/1#issuecomment-782576672

The CI failure seems to be unrelated and the same problem exists for the most recent commit to main.

Yes, will fix this now and then merge. Thank you for the help!


Having trouble installing on MacOS

Hello, I know this is probably not a bug with the code but I was hoping someone here would know.
I intend to run this on a M1 Mac and I cannot get past the conda env create -f UFold.yml
A few things stand out for me. One is that it seems like conda cannot resolve the packages in that yml, but I also don't know much of Conda, so I might be mis-interpreting what this does.
Second is that I see CUDA references, but I don't have an NVDIA card. Will this still run on CPU or Apple GPU perhaps?
If so, should I remove the CUDA references or that is a show stopper for me?
I've been working on a project to slice motifs from RNA and I was hoping I could tailor my output to fit UFold and script it so it goes out of our tool and into UFold and make it a breeze for users to input a sequence and get all the folded structures.
Just in case, this is what I'm getting from the command:
conda env create -f UFold.yml
Collecting package metadata (repodata.json): done
Solving environment: failed
ResolvePackageNotFound:

cudatoolkit==10.1.243=h036e899_8
readline==7.0=hb321a52_4
ninja==1.10.2=hff7bd54_1
libopenblas==0.3.13=h4367d64_0
libffi==3.2.1=hf484d3e_1007
_openmp_mutex==4.5=1_gnu
libgomp==9.3.0=h5101ec6_17
libgfortran4==7.5.0=ha8ba4b0_17
sqlite==3.26.0=h7b6447c_0
setuptools==52.0.0=py36h06a4308_0
python==3.6.6=hd21baee_1003
tk==8.6.10=hbc83047_0
pip==21.2.2=py36h06a4308_0
zlib==1.2.11=h7b6447c_3
scipy==1.3.1=py36he2b7bc3_0
ca-certificates==2021.7.5=h06a4308_1
libgcc-ng==9.3.0=h5101ec6_17
libstdcxx-ng==11.1.0=h56837e0_8
pandas==0.25.0=py36he6710b0_0
pytorch==1.4.0=py3.6_cuda10.1.243_cudnn7.6.3_0
xz==5.2.5=h7b6447c_0
libgfortran-ng==7.5.0=ha8ba4b0_17
mkl==2021.3.0=h06a4308_520
openssl==1.0.2u=h7b6447c_0
certifi==2021.5.30=py36h06a4308_0
intel-openmp==2021.3.0=h06a4308_3350

Hello nlabrad, I am a MacUser and I encounntered the same problem as yours. But I ran it successfully on Linux system, I think this is a software intended for Linux only.  You may try other RNA secondary structure predictor(for instance, RNAfold in ViennaRNA,I installed it successfully on my mac)


Specifing specific value for metrics in melt push command.

Description
Adding a new property to MELT YAML metric definition called "value" to allow users to specify a precise value for generated telemetry packets.
Type of Change

[ ] Bug Fix
[x] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

[x] I have read the contributing guidelines
[ ] Existing issues have been referenced (where applicable)
[x] I have verified this change is not present in other open pull requests
[x] Functionality is documented
[x] All code style checks pass
[ ] New code contribution is covered by automated tests
[ ] All new and existing tests pass

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

Comparison is base (a7110df) 27.06% compared to head (0893b1e) 27.06%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #250   +/-   ##
=======================================
  Coverage   27.06%   27.06%
=======================================
  Files          43       43
  Lines        4533     4533
=======================================
  Hits         1227     1227
  Misses       3212     3212
  Partials       94       94

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


Get contributors list with additions, deletions, and commit counts

Adding support for this API call https://developer.github.com/v3/repos/statistics/#code-frequency

Tests for "Repo Deleted" failed but this is not concern of this PR

Howdy! I restarted the build and the tests check out. Must be a race condition or something with the repo deleted test, will have to investigate.
At any rate, have tested and reviewed and LGTM; please add documentation to README.md and rebase, will then merge.
Thanks for your work!

Hi @aendrew, sorry for delay.
I've added the documentation and rebased my PR.

@eugef No worries for the delay! Thanks for doing that! Merging now! :boom: :+1: :shipit:


National countries of a wrong culture

There may be cases of CK2 conversions where a national country would be of a wrong culture, e.g. France becoming an Italian country.
This issue is here to remind me I need to create events that could fix this madness.

Thing is, in my recent example, france had italian king in ck2 and this persisted as primary_culture=italian in eu4 throughout the entire game, resulting in italian colonies later going italo-americano, etc. I don't think there's anything you can do in vic2 that would reverse the damage done by ck2 converter previously. If eu4 AI isn't bright enough to say "hey, i have zero italian provinces but 50 french ones, maybe I should switch primary culture", there's no helping this.


Update README with installation

Update README with feedback from here: https://github.com/opensearch-project/opensearch-py/pull/76

Since this was first opened, this information has been added to GETTING_STARTED.md. There are mentions of both pip install and git but a little more tersely than originally proposed. Unless there are any objections, I would propose we close this as solved.


Code Analysis page does not match screenshot

I'm trying to migrate from the legacy FxCop analyzers to the new ones, but my Code Analysis page does not match the one provided in the screenshot. Here's instead what I see for projects targeting .NET Framework 4.7.1:

And for .NET Standard 2.0 & .NET Framework 4.6.1:

I'm using Visual Studio 2019 16.8 Preview 3.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: eee67c1d-5856-93e0-e2d3-e9551436aabc
Version Independent ID: b756aa96-bbb1-40e8-34ca-97f9939bf7e9
Content: Install FxCop analyzers - Visual Studio
Content Source: docs/code-quality/install-fxcop-analyzers.md
Product: visual-studio-windows
Technology: vs-ide-code-analysis
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

In general, because of the screenshot discrepancy, I'm not actually sure how to switch to the new analyzers and disable the legacy ones.

@50Wliu -- Winston, thank you for your feedback.  Please consider these resources:

Visual Studio on Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@Mikejo5000 -- Mike, please look into this issue.

Page has been updated for 16.8 and .NET5, so the screenshots have been removed.
The page also has links to two migration docs, which might be relevant:

Migrate from legacy analysis
Migrate from FxCop analyzers to .NET analyzers


Add contracts to files

Hi again:) I'd like to ask if you'd be ok with adding contracts to the package.json, so that I can depend on your repository with my contracts - I want to leverage the code for my project, but I don't want to fork it, just to build a wrapper simplifying the API fit for out use case.
This would allow me to simply add your repo to package.json like with for example Openzeppelin contracts

Hi again:) I'd like to ask if you'd be ok with adding contracts to the package.json, so that I can depend on your repository with my contracts - I want to leverage the code for my project, but I don't want to fork it, just to build a wrapper simplifying the API fit for out use case.
This would allow me to simply add your repo to package.json like with for example Openzeppelin contracts

Sure. thanks for the PR


Devices api

Should this change to the REST API make it into the owgw schema in openapi/owgw.yaml?

I thought we were changing the parameter to includeProvisioned=true/false rather than an empty param?


How to remove the background so it uses the configuration from the terminal?

Description of the problem or steps to reproduce
I like the color scheme, but I would prefer to be able to use just the background set by my terminal. Is there any way to do so?
I mean, how can I use the default scheme but without background? I can't find the themes folder (~/.config/micro/colorschemes doesn't exist), and setting up the simple color scheme is an option, but I prefer the default without the background much better.
Specifications
Version: 2.0.13
Commit hash: 68d88b57
Compiled on October 22, 2023

Done! I've followed these steps:

Create ~/.config/micro/colorschemes
Download the default.micro colorscheme there as custom.micro
Replace all ,#282828 to nothing
set colorscheme as "custom"

Sorry for opening an issue and closing it after 30 min.

https://github.com/zyedidia/micro/blob/master/runtime/help/colors.md


Please check this part of the documentation of scrapyd https://scrapyd.readthedocs.io/en/stable/overview.html#scheduling-a-spider-run


remove - on julia version in REQUIRE

that refers to prereleases of julia 0.3, and has not been necessary for nearly a year

I have removed it on the latest pull request.

On 10 Aug 2015, at 22:59, Tony Kelman<EMAIL_ADDRESS>wrote:
that refers to prereleases of julia 0.3, and has not been necessary for nearly a year
You can view, comment on, or merge this pull request online at:
https://github.com/kzahedi/Shannon.jl/pull/3 https://github.com/kzahedi/Shannon.jl/pull/3
Commit Summary
remove - on julia version in REQUIRE
File Changes
M REQUIRE https://github.com/kzahedi/Shannon.jl/pull/3/files#diff-0 (2)
Patch Links:
https://github.com/kzahedi/Shannon.jl/pull/3.patch https://github.com/kzahedi/Shannon.jl/pull/3.patch
https://github.com/kzahedi/Shannon.jl/pull/3.diff https://github.com/kzahedi/Shannon.jl/pull/3.diff
—
Reply to this email directly or view it on GitHub https://github.com/kzahedi/Shannon.jl/pull/3.

No, you do want a julia 0.3 line in your REQUIRE file otherwise you're claiming to support Julia versions 0.2 and earlier. I'm assuming you are not testing the package against those early versions.

Without the -.

Ah, ok. I misunderstood your email. I added it back into the REQUIRE file. Thanks!

On 10 Aug 2015, at 23:48, Tony Kelman<EMAIL_ADDRESS>wrote:
No, you do want a julia 0.3 line in your REQUIRE file otherwise you're claiming to support Julia versions 0.2 and earlier. I'm assuming you are not testing the package against those early versions.
—
Reply to this email directly or view it on GitHub https://github.com/kzahedi/Shannon.jl/pull/3#issuecomment-129621946.

It's still not right. This pull request was merely deleting the trailing -.
julia 0.3- means "require a prerelease of Julia 0.3 or newer."
julia 0.3 means "require a release of Julia 0.3 or newer."
All this patch was suggesting is to change from 0.3- to 0.3.

I tried to read up on how to do it right. Now I know :)
I thought that 0.3- means any version 0.3.x

On 11 Aug 2015, at 00:06, Tony Kelman<EMAIL_ADDRESS>wrote:
It's still not right. This pull request was merely deleting the trailing -.
julia 0.3- means "require a prerelease of Julia 0.3 or newer."
julia 0.3 means "require a release of Julia 0.3 or newer."
All this patch was suggesting is to change from 0.3- to 0.3.
—
Reply to this email directly or view it on GitHub https://github.com/kzahedi/Shannon.jl/pull/3#issuecomment-129629553.

It's a really unfortunate and unintuitive syntax. I agree that at first glance 0.3- looks like it should mean "0.3 or newer," but it doesn't. So I'm trying to set things straight and make sure people understand it better. Thanks, fb2f7dc9c2948d8c0678b48c07d4305284a0b944 looks correct to me.

Shannon is still listed as version 0.1.2 on pkg.julialang.org and it's description does not match the current repository description. Have I forgotten something?


Fix: recursive/glob support

Changes:
-r is now a flag, not an argument
file argument changed to path, a string that can be either a file path, directory, or glob pattern
Logic to support the above in a natural way
This isn't quite ready for merging. Needs some testing

@calum-bird
https://chat.openai.com/share/902cd378-3ebc-4e35-99ed-e63c7150c6ad


[SPARK-45059][CONNECT][FOLLOWUP] Remove try_reflect problem filter rule in connect

What changes were proposed in this pull request?
This is a followup PR for #42783 , to remove  try_reflect problem filter rule in spark connect.

Why are the changes needed?
make sure the try_reflect check work.

Does this PR introduce any user-facing change?
No

How was this patch tested?
exist test.

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

Merged to master for Apache Spark 4.0.0.


Addressing strange memory leak

https://github.com/Code-Hex/go-generics-cache/blob/2601bb0d3c67fc00cdbd5763cc2361d2a363f0ee/cache.go#L277-L279
After some pprof job, I believe this line make memory leaks as this would not checking lru cache capacity before doing heap push.

@gonejack Thanks!
Could you give me a minimal reproduction code to cause it?
So I can fix this problem🙏


v11, long term. Compatibility with future versions

Thank you for great extension ⭐
This is more a question rather than an issue.
As it seems, this extension was created as a follow-up to this discussion. The current major version of Directus is 11 tho as per documentation tested with Directus 10.10.4.
I wonder if this extension has been battle-tested with v11 and if it is safe to rely on that going forward, as it seems there is still no native functionality like that.

I havent personally tested the extension with v11 yet however it should work 🤔 None of the functionalities this extension relies on have been changed since 10.10 afaik

@br41nslug yeah I see, what do you think, are there any chances similar functionality will land in directus sometime in the future?

I somewhat doubt it considering the "best" (most performant) way to implement this is on the SQL level like (in postgres):
SELECT *
FROM my_collection
ORDER BY RANDOM();

But I am not sure this is consistently supported for all database providers supported by Directus.

@br41nslug does directus support running queries like that natively?

You can from within backend extensions like this, but when using raw SQL you'll have to be sure of the underlying database compatibility if you'd want to distribute that extension

just dig a bit deeper https://github.com/directus/directus/discussions/12452 tho this is the best pointer I've found so far https://github.com/directus/directus/discussions/3312


Omg, is this really back?

TileStudio is my favorite tile editor ever, but I long ago switched to Linux and it's never been the same without it. I can't wait to hear more about Tile Studio II!

Hi bweir, sorry, Tile Studio II is Windows only (.NET).
This is the old Tile Studio which I'm now porting to Lazarus, so you can compile it on Linux.

Any news when Tile Studio II will be available? I can't wait to use it on a new game that I'm developing (I'm still using the old one)
Thanks for your hard work!


HIde when showing notifications option will never make the widget(s) appear

Installed version is 1.1.2
The hide when showing notifications option, even with notification listener active, shows no widget when there are no notifications.
If notification listener looks for every type of notifications, consider switching to watching just non-swipeable notifications. Switching to non-swipeable and non-silent notifications (Android 8+) would be even better.
My phone is Oneplus 6T running EvolutionX (Custom ROM) based on Android 10.0

It currently checks for notifications above minimum priority. Are there no notifications posted at all?

I have 2 ongoing notifications, with minimum priority, and still widgets are not shown

This should be fixed in 1.2.0


Add support for RWS 2.0

Hello everyone,
a support for RWS 2.0 would be very nice to communicate with the new OmniCore controllers. I would like to work with my students on an implementation. Could you give me a small overview of the places where changes would be necessary?

Thanks for the offer, that's always appreciated.
We have some internal work already towards RWS 2 compatibility, but cannot release that yet.
In the meantime, the NoMagicAi/abb_librws fork already has RWS 2 compatibility, in addition to some other nice improvements. I would suggest to collaborate with them.
We'll look into upstreaming those changes in the future (provided NoMagicAi is interested in that of course).

It seems NoMagicAi has disabled the Issues tab on their fork (or haven't enabled it).
Perhaps @mkatliar could comment here?

We'll look into upstreaming those changes in the future (provided NoMagicAi is interested in that of course).
@gavanderhoorn of course we are interested!

It seems NoMagicAi has disabled the Issues tab on their fork (or haven't enabled it).
Perhaps @mkatliar could comment here?

I have enabled issues on https://github.com/NoMagicAi/monomagic

I have enabled issues on https://github.com/NoMagicAi/monomagic

is this a copy-pasta? That repository is probably private, and I assume it's not the fork we're discussing here :)

Sorry, wrong link. On https://github.com/NoMagicAi/abb_librws, of course.

Hey thanks for the feedback. I just pulled the version of @mkatliar but it seems that their is some problem with the tests.
I can't build the version on the master branch. Gives me the following error:
/home/developer/workspace/build/abb_librws/abb_librws-test: symbol lookup error: /home/developer/workspace/build/abb_librws/abb_librws-test: undefined symbol: _ZN3abb3rws26extractDelimitedSubstringsERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEEccc
CMake Error at /usr/share/cmake-3.16/Modules/GoogleTestAddTests.cmake:40 (message):
  Error running test executable.

    Path: '/home/developer/workspace/build/abb_librws/abb_librws-test'
    Result: 127
    Output:

It just build it with catkin build abb_librws. Do you have any advice @mkatliar?

Nevermind.. should have cleaned the build directory beforehand..

@herrvonregen we encounter the same problem from time to time, so if you find why it happens and how to fix it please submit a PR.

It seems that also https://github.com/JOiiNT-LAB/abb_wrapper contains a fork of abb_librws with support for OmniCore, even if it is not easy to say as the history of the abb_librws repo was not preserved.

Hm. It would have been nice if those were just forks ..
@gialen: what was your rationale for making disconnected copies? Collaboration is the main way we make progress, and disconnecting things like this goes against the OSS spirit.

After a few quick tests, it looks to me like the version of JOiiNT is built on an outdated version of this repository. Unfortunately, this makes it almost impossible to simply replace/use it.

Hi guys,
we kept developing the repo from JOiiNT lab and it is now usable. We currently support Gofa and Yumi Single Arm with its SmartGrippers.
I basically managed to create a ros_control pkg to control the robot and link it to MoveIt!. The repo has been tested with both robots.
I leave the link hereafter: https://github.com/MerlinLaboratory/abb_wrapper
Hoping this repo can help someone,
Cheers

Hi all, I would like to know if anyone is currently working on this RWS 2.0  porting.
If not, I would like to ask @gavanderhoorn:

How much effort is needed to do so?
such porting should be done here or in a different repo?

Cheers

The fork mentioned in https://github.com/ros-industrial/abb_librws/issues/147#issuecomment-972779173 adds RWS 2.0 support. A number of users have been able to use it with the rest of the driver stack. I believe @herrvonregen fi.


How to get string values out of a cell array?

I'd like to get the string values out of a cell array, but I cannot see how.
Here's my code and I'm attaching a screenshot of the command prompt output.
using MatFileHandler;
namespace MatLabFiles.Header
{
class Program2
{
private const string FilePath = "C:\Data\ImportFiles\CMI\Test_20190225\nn01_Bin1_cnnClass_Elephant_Multi_16Aug18_V7.mat";
    static void Main(string[] args)
    {
        IMatFile matFile;
        using (var fileStream = new System.IO.FileStream(FilePath, System.IO.FileMode.Open))
        {
            var reader = new MatFileReader(fileStream);
            matFile = reader.Read();
        }

        IVariable var2 = matFile["clab"];
        Console.WriteLine(var2.Name.ToString());
        Console.WriteLine(var2.GetType());

        IArray arr = var2.Value;

        Console.WriteLine(arr.Count.ToString());
        Console.WriteLine(arr.ToString());
        Console.WriteLine(arr.Dimensions);
        Console.ReadKey();
    }
}

}

Continuing your example, you can get values that are inside your cell array by casting to the ICellArray interface and using the indexer:
ICellArray cellArray = arr as ICellArray;
IArray value1 = arr[0];
IArray value2 = arr[1];

(you already know there are two of them by looking at arr.Count).
If you know that both of these values are strings (char arrays in MATLABspeak), you can cast them to ICharArray and extract the strings:
string string1 = (value1 as ICharArray).String;
string string2 = (value2 as ICharArray).String;

You can find more examples along the same lines in MatFileHandler.Tests/MatFileReaderTests.cs.

Thanks, Mahalex! I really appreciate your help, but please bear with me as I am not very familiar with Matlab array types.
I tried what you suggested, but I had to change from this:
IArray value1 = arr[0];
To this: IArray value1 = cellArray[0];
I couldn't get your version to compile. The error I got was: "Cannot apply indexing with [] to type of IArray."
I was told that the values in the "clab" variable were strings, but when I try to cast them as such, I get a System.NullReferenceException. I am interpreting this to mean that the cast is not successful and so is returning a NULL.
Here's my code so far:
`using System;
using MatFileHandler;
namespace MatLabFiles.Header
{
class Program2
{
private const string FilePath = "C:\Data\ImportFiles\CMI\Test_20190225\nn01_Bin1_cnnClass_Elephant_Multi_16Aug18_V7.mat";
    static void Main(string[] args)
    {
        IMatFile matFile;
        using (var fileStream = new System.IO.FileStream(FilePath, System.IO.FileMode.Open))
        {
            var reader = new MatFileReader(fileStream);
            matFile = reader.Read();
        }

        IVariable var2 = matFile["clab"];

        IArray arr = var2.Value;

        Console.WriteLine(arr.Count.ToString());
        Console.WriteLine(arr.ToString());

        ICellArray cellArray = arr as ICellArray;

        IArray value1 = cellArray[0];
        IArray value2 = cellArray[1];

        Console.WriteLine(value1.Count);
        Console.WriteLine(value2.Count);

        string string1 = (value1 as ICharArray).String;
        string string2 = (value2 as ICharArray).String;

        Console.WriteLine(value1.ToString());
        Console.WriteLine(value2.ToString());

        Console.ReadKey();
    }
}

}
`
Error at Console.WriteLine(value1.ToString()); System.NullReferenceException
HResult=0x80004003
Message=Object reference not set to an instance of an object.

Yes, you are right about cellArray instead of arr.
From what I see, it looks like the values in your cell array are not strings (at least not in the traditional MATLAB sense of char arrays). Can you look at cellArray[0].GetType()? This would give a hint about what kind of object is there. If you want, you can send me the file that you are inspecting, and I'll take a look at it myself.

Thanks! When I get the type of cellArray[0] it returns MatFileHandler.MatCellArray. So it must be a cell array, inside a cell array. I'm guessing the string values are in the inner cell array. Is that possible? I'll check with the creator of the file.

Yes, cell arrays may contain nested cell arrays; you can treat them in a similar way to access their elements and inspect their types. Standard MATLAB strings should look like MatCharArrayOf<byte> or MatCharArrayOf<ushort>.

Thank you! I can now see the MatCharArrayOf1[System.Byte]. I still need to get the actual strings that are inside of that. Here's my code so far:         static void Main(string[] args)
{
IMatFile matFile;
using (var fileStream = new System.IO.FileStream(FilePath, System.IO.FileMode.Open))
{
var reader = new MatFileReader(fileStream);
matFile = reader.Read();
}
        IVariable var2 = matFile["clab"];

        IArray arr = var2.Value;

        Console.WriteLine(arr.Count.ToString());
        Console.WriteLine(arr.ToString());

        ICellArray cellArray = arr as ICellArray;

        IArray value1 = cellArray[0];
        IArray value2 = cellArray[1];

        Console.WriteLine(cellArray[0].GetType());

        ICellArray cellArray1 = value1 as ICellArray;
        ICellArray cellArray2 = value2 as ICellArray;

        IArray valA = cellArray1[0];
        IArray valB = cellArray2[1];

        Console.WriteLine(value1.Count);
        Console.WriteLine(value2.Count);

        Console.WriteLine(valA.Count);
        Console.WriteLine(valB.Count);

        string string1 = (valA as ICharArray).String;
        string string2 = (valB as ICharArray).String;

        Console.WriteLine(valA.ToString());
        Console.WriteLine(valB.ToString());

        Console.ReadKey();
    }`

And this is what it returns:


Do not fail getLockWithTimeout before timeout

When a node leaves the cluster, the thread invoking tryAcquire may be interrupted before timeout has expired. So we should check if there is some time remaining and try again.

This PR follows up on eclipse/vert.x#1609
Do not merge before the core PR is merged.

👍


Make it work on Mac/Linux

from https://code.google.com/p/lessmsi/issues/detail?id=5

is is possible to use lessmsi on linux.

Not yet, but the cab extraction code does all work on Linux now. So we're half way there. The part that need some work is the MSI code which is a combination of OLE Compound Files (aka Structured Storage) and a special MSI-specific database. I think the most reasonable thing to do is to use the WINE libraries (or their source?) to get the last part done.
Always glad to help you along if you're interested in contributing!

On Feb 27, 2015, at 01:12, john550<EMAIL_ADDRESS>wrote:
is is possible to use lessmsi on linux.
—
Reply to this email directly or view it on GitHub.

I have a very specific use case in which I need to get the MSI ProductVersion from the Property table of an MSI file using the command line.
I still have not figured out how to do this using the command line and lessmsi on OS X, but I do have the lessmsi GUI working with WineBottler.  http://winebottler.kronenberg.org/
You can download the zip compressed .app I made here:  https://dl.dropboxusercontent.com/u/640753/lessmsi.zip
You can make it yourself using the following screenshot & winebottler:

Just want to throw it out there that I've been able to use Less MSIerables in linux with wine after doing winetricks dotnet40. It's probable that wine's version of mono works as well - I can get a GUI to come up in the linux-native version but can't open MSI files due to the windows-specific code.
I think it's quite probable that it could be compiled with libwine, but don't have the hard disk space on this laptop to verify.

I am able to use it on a Mac, and it doesn't have any windows-specific code outside of whatever is in Wine.

well as wine consists of alternative API's for windows-only code, you're saying the wine team has done it's job well - not that it's actually a portable program. Right?

Yes, Wine's code is windows specific, just not written by Microsoft.
LessMSI is not a portable program, but it works through Wine on the Mac, which doesn't have anything to help it run other than Wine and anything installed with Winetricks.

On my side I would be interested to have command-line only native (no wine required) way to build & run on Linux and Mac in addition to Windows, to add support for a better MSI extraction in https://github.com/nexB/scancode-toolkit beside the 7z-based extraction I have today.

I haven't explored this option, but since lessmsi appears to be written in C# then it might be able to run on Mono or .NET core which would make it portable.

Has anyone tried compiling with winelib?  That should fill in all of the Win-specific functions.

@activescott wrote:

The part that need some work is the MSI code which is a combination of OLE Compound Files (aka Structured Storage)

Gnome msitools uses GNOME/libgsf which is partially influenced by Wine.

and a special MSI-specific database. I think the most reasonable thing to do is to use the WINE libraries (or their source?) to get the last part done.

GNOME/msitools used the same approach, IIUC.
Both libs are under LGPL-2+. For MIT-licensed project it would be hard to reuse GPL code directly.

I've actually switched to using https://github.com/GNOME/msitools for my non-windows use cases for this. That said, I wish I had the time to experiment with lessmsi to see how easy it would be to get it to work with winelib or mono.

I've actually switched to using https://github.com/GNOME/msitools for my non-windows use cases for this. That said, I wish I had the time to experiment with lessmsi to see how easy it would be to get it to work with winelib and/or mono.

@jgstew thanks for your share. can you show me how to deploy msitools on linux?


fix: renderToString mode is always ignored and returns an empty page

Overview
What is it?

[ ] Feature / enhancement
[x] Bug
[ ] Docs / tests / types / typos

Description
After changing the entry.ssr.tsx to:
export default function (opts: RenderToStringOptions) {
    const result = renderToString(<Root />, {

the application always returns an empty page when served from express server.
Checklist:

[ x] My code follows the developer guidelines of this project
[x ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

LGTM!


Fix "Service Unavailable" for Viewer

Hitting viewer.shabados.com returns a 503 Service Unavailable, whereas viewer.next.shabados.com works fine.
The kubernetes deployment states ImagePullBackOff

Fixed in https://github.com/shabados/manifests/commit/b69a610786ff5f0e976e79e8b00f649146ff50e6

Fixed in https://github.com/shabados/manifests/commit/b69a610786ff5f0e976e79e8b00f649146ff50e6


Registry: Remove table drop in case of type change

This piece of code was dropping the registry tables iff the type of the registry
was changed between two runs. There's apparently no technical reason to do so,
as all types use the same table format.
This code created problems because it consistently prints 3 error messages when a node
starts, which can be startling to users. This could be avoided by checking for the
table existence before actually doing the SELECT, however, when considering the logic,
it stood out as no other node has that behavior (e.g. switching between full node and validator).
In this case, I think it is better to simply not do this and let the operator
wipe the table clean (by removing cacheDB) if that behavior is desired.

I think we might end up with some "metadata" table on the node to account for those situations: For example, to allow nodes to auto-upgrade their database when restarted with a new release.


[bazel] Enable C warning flags in .bazelrc

Fixes #12553

Thanks @dmcardle! It would be great if we could minimize new OT_UNREACHABLE()s. Also, it would be a good idea to rebase this PR as you suggested.

Done! Managed to get rid of the new OT_UNREACHABLEs by zero-initializing local variables.

@msfschaffner I'm sorry if this is causing headaches. Feel free to revert if it's not feasible to fix-forward. This was the last wide-reaching change that modified C warning flags.
What could we do to prevent this from happening again? Can we add a test to CI?
I'm also not 100% sure which changes you're asking to be notified about — changes to C flags?

Let's leave it in for now - we'll attempt to fix this since we should minimize differences between the open and closed DV environments. The simplest way to check this in CI is likely to add this and similar global config files to the CI blockfile.
I know it's not always obvious (I ran into this issue myself) - but we should be mindful about changes that are known to have a global impact on the DV environment. The closed-source side is using the same infrastructure and adds additional C tests on top that are not exposed in the open-source.

@dmcardle, @msfschaffner
I am little late for this thread. because I was fixing compile errors in the closed source.
Changes added to .bazelrc cause about 12 regression failures in closed source regression.
I fixed all closed source files for now, but at least for the next a few weeks,
we need a stable rule check and focus our time to fix run_time test failure.


qutebrowser-qt6: unstable-2023-03-19 -> unstable-2023-04-18

Description of changes
Closes #227275.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

2 packages built:

qutebrowser-qt6
qutebrowser-qt6.dist


issues with module loading  and dynamic init

Using script provided from readme
//1. bootstrap.jsx
import {AppInsights} from "applicationinsights-js"

/* Call downloadAndSetup to download full ApplicationInsights script from CDN and initialize it with instrumentation key */
AppInsights.downloadAndSetup({ instrumentationKey: "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxx" });
..

// 2. app.jsx
import {AppInsights} from 'applicationinsights-js';
AppInsights.trackPageView(..);

App is called after bootstrap.
AppInsights.trackPageView can crush as undefined since was not init yet.

@dlebedynskyi do you mean, that if trackPageView is called before downloadAndSetup for any reason, than it would crash, while you would expect it merely to put item in the queue?

@AlexBulankou yes you got it correct.

This is fixed, so the next release on npm will have the fix


[zh] Synchronize the README file

/language zh

Weird. Let me try rebase.

/retest

@jimangel Looks like changes to README requires a higher privilege to approve.

/assign @zacharysarah

/lgtm
/approve


add unsigned rotates

Thanks, I guess these were left out in the migration. Sorry about that!

@TheIronBorn oh crap, wrong repo. Care to re-submit to rust-lang-nursery/packed_simd ?

oh nice!


update electron to get better linux appindicator support

Dose this upgrade need so many changes ?

Great! It is necessary to use a instead of b to implement singleton mode. Currently, multiple Instances will occur in the Archlinux Gnome WayLand environment.
And Should we integrate into the notification panel? Then Remove the tray icon completely.


eslint-module-utils from npm is to new

Gitea Version
1.15.5 (also 1.15.4 ...)
Git Version
No response
Operating System
linux
How are you running Gitea?
not relevant for this bug.
Database
No response
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Description
No build with current nodejs packages.
Since about 1 week version 2.7.x of eslint-module-utils is available.
If you install the nodejs packages with npm install you can't build gitea afterwards.
It comes the error message:
error exported declaration 'default' not used within other modules import/no-unused-modules
With the last version 2.6.2 of eslint-module-utils this error does not occur.
Screenshots
No response

Oudated