[HDL 9-849]

Hi,
Thanks for the repository.
I got this error in Vivado 2020.1:
[HDL 9-849] Syntax error : file ended before end of clause. ["../Cores-SweRV-EL2/snapshots/default/el2_param.vh":156]
Seems like the parameter instance is not parsed correctly.
Also, what is the highest frequency I can expect on FPGA?
Thanks

do you see compilation problems with other tools verilator/vcs/xrun?
FPGA implementation should run at 50-100MHz ...

I haven't tried with Vivado 2020.1 but it builds fine in vivado 2019.1 when using FuseSoC. If you install FuseSoC, you can then add SweRV EL2 as a library with fusesoc library add el2 https://github.com/chipsalliance/Cores-SweRV-EL2 and run 'fusesoc run --target=synth chipsalliance.org:cores:SweRV_EL2` to have it building for vivado.

hmm.. I just noticed that #3 isn't applied yet, so the FuseSoC builds are currently broken

Eventually it was a config issue, closing.
Thanks

Can you describe what was the issue and how was it resolved?
Thank you


Qrels and Run query ids do not match

Describe the bug
I am evaluating my test set against my algorithm recommendations, but it gives the error
Qrels and Run query ids do not match
To Reproduce
Steps to reproduce the behavior:
metrics = ["recall@10", "mrr@10","ndcg@10"]
person_date_indexs = df_recs_train_top10['person_date_index'].unique()
run = tranform_recs_to_ranx(df_recs_train_top10, person_date_indexs, "person_date_index", "gym_id", "rank")
qrels = tranform_test_to_ranx(df_test_checkins_new_col_renamed, df_recs_train_top10)
evaluate(qrels, run, metrics)
Expected behavior
print the recall, mrr and ndcg
Screenshots

Desktop (please complete the following information):
Ubuntu 22.04
Additional context
It worked fine with another test set. The only difference is that in this test set I removed items that the user already interacted in train.

Does your run contains all the elements on the qrel (and vice-versa)? I had this problem many times working with a dataset that had many queries that did not have relevance labels. It can also happen in the opposite sense (although rarer), if you have relevance data for a query that is not evaluated by your system it will also crash (reading your additional context it could be this case if you have the same qrel file for train and test)

Hi @EmotionFever and thank @cadurosar,
As @cadurosar told you, this is not a bug, it is a sanity check to let you know your qrels and run query ids to not match, thus you have results for queries without relevance judgements or queries without results, or both.
Maybe, you should just remove the queries you removed from qrels from the run too.
Please let me know if this solve your issue and, if you like it, consider giving ranx a star.
Thank you,
Elias

Closing for inactivity. Feel free to re-open.


Feature request: Forwarded port returned via built-in HTTP server

What's the feature?
Add ability to return currently forwarded port via the built-in HTTP server.

Why do you need this feature?
I am trying to automate the update of port forwarded in other apps/containers and a simple value returned via a http source is a lot easier to integrate into other solutions. Often direct file access in the current solution is not suitable.

(Hope this is a fair request, and thank you for such an awesome solution and all the work put in.)

That's a great idea, I'll do it soon! Quick question out of the topic, how are you using port forwarding? I can't figure out how to test it actually (trying with netcat, nmap, http servers nothing seem to work).

Hello, you now have an http route where you can for example:
wget -qO- http://<your-docker-host-ip>:8000/openvpn/portforwarded | jq .port

To get the port forwarded only. This requires jq installed (i.e. apk add jq on Alpine).
If you can also answer the question above about how you use port forwarding please? I'm trying to test it but it doesn't seem to work so far for me (no use for it, but it would help to test things for users). Thanks


model: Sort projects by type and then by definition file

Instead of sorting projects by their full id, only sort by type, and
then by definition file depth / path. This causes root projects to
appear first in the result file, which makes using the id of the first
entry as part of the Jenkins build description (see 0366b45) more
meaningful.
Signed-off-by: Sebastian Schuberth<EMAIL_ADDRESS>

/cc @mlmilan

Can you add an example how that change would change one of the expected results files?

I was planning to wait for failing tests for that purpose (and thus converted the PR to a draft again).

What other places would be affected by changing the comparator, in addition to the list of projects in the analyzer result?

As the new version of this PR shows, reports might also be affected.

I'm not so happy with how this change affects the static HTML report. Before the identifiers for the projects at the bottom were at least alphabetically sorted in the index of the report:
Gradle:org.ossreviewtoolkit:a1:1.0
Gradle:org.ossreviewtoolkit:a2:1.0
Gradle:org.ossreviewtoolkit:a:1.0
Gradle:org.ossreviewtoolkit:b1:1.0
Gradle:org.ossreviewtoolkit:b2:1.0
Gradle:org.ossreviewtoolkit:b:1.0
Gradle:org.ossreviewtoolkit:c:1.0
Gradle:org.ossreviewtoolkit:d:1.0
Gradle:org.ossreviewtoolkit:root:1.0

This is not ideal, because it hides the hierarchy of the projects, but at least it has an easy to understand order.
Now the order seems to be random (in a real project they would likely not have hierarchical names like I used in this example, so it would seem even more random):
Gradle:org.ossreviewtoolkit:root:1.0
Gradle:org.ossreviewtoolkit:a:1.0
Gradle:org.ossreviewtoolkit:b:1.0
Gradle:org.ossreviewtoolkit:c:1.0
Gradle:org.ossreviewtoolkit:d:1.0
Gradle:org.ossreviewtoolkit:a1:1.0
Gradle:org.ossreviewtoolkit:a2:1.0
Gradle:org.ossreviewtoolkit:b1:1.0
Gradle:org.ossreviewtoolkit:b2:1.0

If we change sorting of projects to look at the hierarchy, it should show nested projects next to each other, like this:
Gradle:org.ossreviewtoolkit:root:1.0
Gradle:org.ossreviewtoolkit:a:1.0
Gradle:org.ossreviewtoolkit:a1:1.0
Gradle:org.ossreviewtoolkit:a2:1.0
Gradle:org.ossreviewtoolkit:b:1.0
Gradle:org.ossreviewtoolkit:b1:1.0
Gradle:org.ossreviewtoolkit:b2:1.0
Gradle:org.ossreviewtoolkit:c:1.0
Gradle:org.ossreviewtoolkit:d:1.0

An the hierarchy should also be visualized by adding indentation.
I'm also not so happy with the approach of changing the project sorting for the whole of ORT only for convenience in the Jenkinsfile, I think if we change the sorting it should be in a way that makes sense when listing the projects and not only look at what we want to be first in the list.
What do you think of those alternatives?

Implement finding the root project in the Jenkinsfile, finding the project with the least amount of "/" in the definition file path should be easy.
Only apply the new sorting to the ORT result file, but not globally.

    val projects = sortedSetOf(
        Project.EMPTY.copy(
            id = Identifier("Gradle:org.ossreviewtoolkit:root:1.0"),
            definitionFilePath = "build.gradle.kts"
        ),
        Project.EMPTY.copy(
            id = Identifier("Gradle:org.ossreviewtoolkit:a:1.0"),
            definitionFilePath = "a/build.gradle.kts"
        ),
        Project.EMPTY.copy(
            id = Identifier("Gradle:org.ossreviewtoolkit:a1:1.0"),
            definitionFilePath = "a/1/build.gradle.kts"
        ),
        Project.EMPTY.copy(
            id = Identifier("Gradle:org.ossreviewtoolkit:a2:1.0"),
            definitionFilePath = "a/2/build.gradle.kts"
        ),
        Project.EMPTY.copy(
            id = Identifier("Gradle:org.ossreviewtoolkit:b:1.0"),
            definitionFilePath = "b/build.gradle.kts"
        ),
        Project.EMPTY.copy(
            id = Identifier("Gradle:org.ossreviewtoolkit:b1:1.0"),
            definitionFilePath = "b/1/build.gradle.kts"
        ),
        Project.EMPTY.copy(
            id = Identifier("Gradle:org.ossreviewtoolkit:b2:1.0"),
            definitionFilePath = "b/2/build.gradle.kts"
        ),
        Project.EMPTY.copy(
            id = Identifier("Gradle:org.ossreviewtoolkit:c:1.0"),
            definitionFilePath = "c/build.gradle.kts"
        ),
        Project.EMPTY.copy(
            id = Identifier("Gradle:org.ossreviewtoolkit:d:1.0"),
            definitionFilePath = "d/build.gradle.kts"
        )
    )

* Implement finding the root project in the Jenkinsfile, finding the project with the least amount of "/" in the definition file path should be easy.

I went for that approach: https://github.com/oss-review-toolkit/ort/pull/2583


As a devops, I want to deploy a separate PostgreSQL instance for Fertiscan so we can have full control over the database environment

Description
Context
Currently, Fertiscan's database shares the same PostgreSQL deployment as Nachet, which is in a deployment we do not control.
Problem Statement
We need to have a dedicated PostgreSQL instance for Fertiscan deployed in our own resource group. This would give us full control over database performance, scalability, security and costs monitoring while eliminating external dependencies on Nachet's infrastructure.
TODO

[ ] Create 2 Database server (Test/Prod)
[ ] Update the secrets to represent the new architecture (Nachet & FertiScan)
[ ] Deploy the live schema (Nachet & FertiScan)
[ ] Implement alerts and docs to manage our Database instances

Acceptance Criteria

A dedicated PostgreSQL instance is deployed in our own resource group.
Fertiscan is fully migrated to the new PostgreSQL instance without data loss.
All existing connections to the database are updated to point to the new instance.
All previous tests are passing
Documentation is created to describe this decision
Ensure minimal downtime

Moving this issue to the howard repo


MSPSDS-690 and MSPSDS-691 record phone and meeting

Add functionality to investigations for recording phone calls and meetings. Approach will be commonised in a separate PR.

Pull Request Test Coverage Report for Build 1443

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+3.6%) to 96.786%

Totals

Change from base Build 1438:
3.6%

Covered Lines:
3824

Relevant Lines:
3951

💛  - Coveralls

@broder changes to keycloak were not intentional.


Code snippets show v2.0 endpoint but that cannot work with resources

[Enter feedback here]
The code snippet where the login.onmicrosoft.com url is build uses the 2.0 endpoint together with resources. This delivers error as 2.0 endpoint does not support resources but requires scopes.

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

ID: 3b1dcd0d-471b-2d33-54b5-39a93dfba2a8
Version Independent ID: 5bd44149-777e-b449-bd2e-f20b9bd04038
Content: Authentication for tabs using Azure Active Directory - Teams
Content Source: msteams-platform/tabs/how-to/authentication/auth-tab-aad.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

@svandenhoven ,
I have tried with
URL
https://login.microsoftonline.com/common/oauth2/v2.0/authorize?
body
client_id=xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
&resource=https%3A%2F%2Fgraph.microsoft.com%2FUser.Read%20openid
&redirect_uri=https%3A%2F%2fac21ab06.ngrok.io%2Ftab%2Fsilent-end
&grant_type=id_token token
&client_secret=xxxxxxxxxxxxxxxxxxxxxxxx
&response_mode=fragment

And it did work for me. Are you getting any error ?

Please re-open if you still have any queries here.


Accessibility Issues - Listbox role + tabIndex

Documenting three issues that should be discussed / actioned on:
1. The carousel should likely not be using a "listbox" role
This issue has already been discussed at length here, but I do not see a clear resolution. I understand there is an argument that the screen reader cannot read the contents of the carousel correctly without the 'listbox' role, and therefore it's better to leave it in. We should challenge this type of thinking when it comes to accessibility -- roles have a very specific meaning that is analogous to semantic HTML. Using a role is like a contract in that we are telling the person using the screen reader that they can expect certain functionality & a certain interaction.
In this case, we are communicating that there is essentially a list of selectable items (that can contain images, etc...) - which is very rarely the actual functionality of a carousel. Not only that, we are communicating that we have implemented a keyboard control pattern for navigating through this interaction (look here for the expected keyboard interaction). Neither of these are the case, so therefore it's better to leave the role empty and allow developers to implement these features themselves. This would mean that 'role' and all 'aria' attributes are available on Slider & Slide.
This guide demonstrates how to correctly + accessibly create a carousel.
2. Remove tabIndex = 0 on Slider & Slide
This article states it best, "there is a myth that applying tabindex="0" to non-interactive elements (paragraphs, headings, lists, etc.) helps “improve” accessibility by providing a way for a person who uses a screen reader to focus on all the content present in a webpage or web app. Unfortunately, this well-intentioned idea actually does not create a good assistive technology experience". Instead, interactive elements (such as forms & links) will be included in the tab order by default, and should be used instead.
3. DotGroup considerations
The DotGroup is comprised of buttons which will automatically be included in the tab order. Developers do not have the ability to easily control whether or not this is the case (ie. control tabIndex on the carousel__dot buttons).
Also, this article demonstrates how the DotGroup could be used to provide intuitive and accessible navigation by populating the buttons with visually hidden titles + aria attributes.
Both of these necessitate manual control over the button array. I believe this feature technically already exists with the 'renderDots' prop, however there may be a more developer friendly way to go about it.
Suggested implementation:

Remove "listbox" role from Slider
Remove "option" role from Slide
Ensure 'role' and other aria html attributes are available to developers on these components
Remove tabIndex = 0 from Slider & Slide

Useful Resources

https://www.w3.org/WAI/tutorials/carousels/functionality/
https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/listbox_role

Notes:
Thanks for all the hard work y'all have done on this project! I will likely continue to add to + edit this post as I continue to research the problem.

Our implementation uses an anchor on each slide of the carousel to allow the user to click through for more information - presumably not an unusual way to do it.
Testing accessibility with cypress and axe-core is currently giving a fail, throwing this error:
https://dequeuniversity.com/rules/axe/4.3/nested-interactive?application=axeAPI
I am assuming at this point that this is due to the above mentioned listbox-option roles


shelley-spec-ledger: MonadFail issue in 8.8.x and beyond (fix suggestion)

Hello,
When trying to build the code on ghc 8.8.3 and 8.10.1, I get the following error:
Configuring shelley-spec-ledger-<IP_ADDRESS>...
Preprocessing library for shelley-spec-ledger-<IP_ADDRESS>..
Building library for shelley-spec-ledger-<IP_ADDRESS>..
[ 5 of 50] Compiling Shelley.Spec.Ledger.BaseTypes ( src/Shelley/Spec/Ledger/BaseTypes.hs, dist/build/Shelley/Spec/Ledger/BaseTypes.dyn_o )

src/Shelley/Spec/Ledger/BaseTypes.hs:214:3: error:
    ‘fail’ is not a (visible) method of class ‘Monad’
    |
214 |   fail _ = SNothing
    |   ^^^^
==> ERROR: A failure occurred in build().
    Aborting...

I read the MonadFail Proposal, but not sure what's the best approach to address this. When I delete the offending line  fail _ = SNothing, shelley-spec-ledger builds successfully on 8.8.3.

Note that we only support GHC 8.6.5 at the moment.

Should be addressed now!


Combine: owner and access columns

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.

I'm not able to import ./infra-legacy/example-configuration.sh so it is a bit hard to see changes in UI

On master ? What are you seeing ?

On master ? What are you seeing ?

Yes, import fail

But why ?

will report to you tommorow or after 17th oct


Nested dropdown style changes for mobile devices

Hey @arpansaha13 ,
Styles changes for mobile devices have been pushed. Kindly review.

Thank you for the contributions


Hosting 0.12 docs after 1.0 release

The English version will be hosted at 012.vuejs.org. For Japanese version, please create a repo named 012-jp.vuejs.org and host the old versions there :)

All right! :)

Done!
Please check :)
http://012-jp.vuejs.org

Cool! One small thing: the home page github star button needs to change its url from yyx990803/vue to vuejs/vue

Thanks for your checking!!
I fixed tiny mistake. :)


names from parent modules are not accessible without an explicit import

Setup
Versions

Rust:1.29.0-nightly (254f8796b 2018-07-13)
Diesel:1.3.2
Database:sqlite
Operating System:win10

Feature Flags

diesel:

Problem Description
I got a lot of warns after rustup and cargo update, like follow:
warning: cannot find type `ThingDefine` in this scope
  --> src\data\thing\mod.rs:71:41
   |
71 | #[derive(Serialize, Deserialize, Debug, Queryable, Clone, PartialOrd, PartialEq)]
   |                                         ^^^^^^^^^ names from parent modules are not accessible without an explicit import
   |
   = note: #[warn(proc_macro_derive_resolution_fallback)] on by default
   = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
   = note: for more information, see issue #50504 <https://github.com/rust-lang/rust/issues/50504>

warning: cannot find type `NaiveDateTime` in this scope
  --> src\data\thing\mod.rs:71:41
   |
71 | #[derive(Serialize, Deserialize, Debug, Queryable, Clone, PartialOrd, PartialEq)]
   |                                         ^^^^^^^^^ names from parent modules are not accessible without an explicit import
   |
   = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
   = note: for more information, see issue #50504 <https://github.com/rust-lang/rust/issues/50504>

What are you trying to accomplish?
I try to add #[allow(proc_macro_derive_resolution_fallback)] on the structure used for query to disable the errors, but the warning exists yet.
What is the expected output?
I want to remove these warnings

Thanks, but we don't accept errors that are specific to nightly versions of Rust. There's an ongoing discussion about this and it's not clear what the final direction will be, Diesel will not be acting on it until it's clear what we actually will need to do for stable.

This broke semver's CI as well.

I had mistook this for a different issue, this one does look like one that we can address

Following the tutorial here I keep getting names from parent modules are not accessible without an explicit import for Queryable and Insertable too, but #[macro_use] extern crate diesel; is present in main.rs? Am I missing something? Did read the merge report and tried changing to extern crate diesel; but unsure if this is correct as I got a bunch of other errors?

@Deedasmi Ah ok, thanks. I'll ignore for now

@WillSquire you may want to mute that warning. No big deal, AFAICS.
Shows up also with latest Rust nightly (1.30.0 as I write)

Here is a good way to mute the warning on a per-file basis:
#![allow(proc_macro_derive_resolution_fallback)]

Now that 1.29 is out is there a plan to cut a release for this fix?

I'm also wondering when this is going to get released. Hit it just now as well.

The fix may be included in 1.4, which will be released in the next week or two -- But that's only assuming https://github.com/rust-lang/rust/issues/52545 is fixed. If our docs still fail to build, I'll be reverting the fix before releasing 1.4.
This is a warning, not an error -- and one that you can easily silence, so it's not something we're going to cut a release over.

You can silence specifically this error with #![allow(proc_macro_derive_resolution_fallback)]. Unless you're writing a ton of custom derives, it's unlikely you'll be getting new instances of this warning.

I know this is a closed issue but I still get this warning after addding  #[allow(proc_macro_derive_resolution_fallback)] in src/schema.rs and models.

Just ‘#[allow’ or ‘#![allow’? They do different things (this definition vs module level)
Also, I think this was also fixed on 1.4? Could be wrong.


uart: add a new "line ctrl changed remotely" callback

The BOSSA bootloader is used on Arduino compatible SAM boards like the Arduino MKR Zero, Adafruit Trinket M0, and Seeedstudio XIAO.  These boards use the 'Arduino hack' of setting the USB CDC baud rate to 1200 baud to signal the application to restart into the bootloader.
Add a new callback to the UART API so the application can listen for baud rate changes.  Add a SAM0 specific module that listens for these changes and restarts into the bootloader if found.
With these changes, a "west flash" will reset the board and flash the new version.

@pabigot please revisit.

Hi @jfischer-phytec-iot I see you changed the title to be UART API specific.  Should I split out the BOSSA changes then?

Hi @jfischer-phytec-iot @finikorg I see you're the USB subsystem owners.  What do you think of the API change?  If required, I'm happy to do a follow up PR that changes the CDC ACM implementation to register callbacks in a thread safe way.  See @pabigot 's comment for more.

Hi @jfischer-phytec-iot I see you changed the title to be UART API specific. Should I split out the BOSSA changes then?

I think not or not yet, my intention was to get more attention.

What do you think of the API change?

+1, but I think we should discuss it in the API meeting. @carlescufi

Notes from 2020-06-30 API telecon.  Edit if I missed something.
JF: Arduino BOSSA uses changes of baud rate to determine whether system should trigger boot loader.  Do we need this?
KC: Seems limited
JF: Likely that other signals than baud rate could be used.
PB: Not sure hardware would support notification of change in baud rate even if autobaud supported.  Similar for low-level control signals.
Summary: Need more input to determine whether this is useful, what would be useful, and whether any of it is implementable on anything other than CDC-ACM.  If not this may be suitable as a device-specific extension for CDC-ACM somehow.

Hi Peter.  There's two PRs for adding BOSSA / Arduino reset support so far - this and #25422
#25422 directly modifies cdc_acm and the feedback was that it was too tightly coupled and should be a generic API.  This PR is a generic API, but the feedback is that there's not enough use cases to add it.
We're getting conflicting feedback.  How should we proceed?

If not this may be suitable as a device-specific extension for CDC-ACM somehow.

We're getting conflicting feedback. How should we proceed?

I guess something was misunderstood. From my side, only this proposal makes sense, not the hack in #25422

We're getting conflicting feedback. How should we proceed?

Sorry; like many things in Zephyr what at first seems simple gets complicated.
My position is that if we're adding new API to the generic UART driver we need to know the requirements for the API.  uart_line_ctrl_callback_set() needs justification, more clear definition, and confirmation that it can be implemented on multiple platforms.
I understand that the goal is:

Detect an Arduino-style upgrade request and implement the required response

The Zephyr implementation should respect these distinct phases.  I agree with @jfischer-phytec-iot that the approach in #25422 is not acceptable: it only works with one platform on one driver, and puts target-specific upgrade logic in a serial driver.  If somebody has a non-SAM0 Arduino-style board it won't help them, and maintaining the serial driver becomes complex.
So my first question is: Can any UART driver other than CDC-ACM detect this event?  If so, what other line-control events can be detected, and how must the subsystem uart_something_changed() API be defined to support them?
In answering that I've now learned about BOSSA which is a red-herring, and  about how original Arduino hardware tied DTR to the MCU RESETn signal to force a reboot, and then how for USB where DTR is not a physical signal this is implemented by opening the serial port at 1200 baud and then closing it.
So it appears that this functionality is only ever useful for CDC_ACM UARTs, and for wired RS-232 there's no software needed because the reset is done in hardware.
So: IMO the line state change detection API doesn't belong in the UART API.
In that case, I would propose to add an an option CONFIG_CDC_ACM_ENABLE_ARDUINO_RESET flag, which puts the detection logic into the cdc_acm driver and defines a non-syscall cdc_acm_arduino_reset_callback(dev, cb) function that installs a callback to be used when the event occurs.  You'll need to add a small header in install/drivers/uart/cdc_acm.h to provide the definition.
Reading the links above I'm not convinced the response to the event needs to erase the first word of the application: it seems it's just supposed to be a soft reset, which starts the bootloader, which may or may not perform an update.  I may be wrong in that interpretation.
But in any case the most flexible approach is just detecting the situation with API like the one two paragraphs up, and letting the application determine what "reset" means.  (If necessary I suppose a lib/arduino could be added with arduino_reset_to_upgrade() implemented for shared use, but that should be a separate PR from adding the ability to detect the request.)
@jfischer-phytec-iot Does this plan make sense to you?

Sorry; like many things in Zephyr what at first seems simple gets complicated.

It's all good.  Thanks for the detailed reply.

My position is that if we're adding new API to the generic UART driver we need to know the requirements for the API. uart_line_ctrl_callback_set() needs justification, more clear definition, and confirmation that it can be implemented on multiple platforms.
I understand that the goal is:

Detect an Arduino-style upgrade request and implement the required response

The Zephyr implementation should respect these distinct phases. I agree with @jfischer-phytec-iot that the approach in #25422 is not acceptable: it only works with one platform on one driver, and puts target-specific upgrade logic in a serial driver. If somebody has a non-SAM0 Arduino-style board it won't help them, and maintaining the serial driver becomes complex.
So my first question is: Can any UART driver other than CDC-ACM detect this event? If so, what other line-control events can be detected, and how must the subsystem uart_something_changed() API be defined to support them?

Unfortunately I don't have other concrete use cases.  Other serial emulation protocols such as Bluetooth SPP or RFC2217 have similar features where the line control and configuration is set remotely.
I can imagine using this for a Zephyr based USB -> Serial adapter, but that's a hypothetical.

In answering that I've now learned about BOSSA which is a red-herring, and about how original Arduino hardware tied DTR to the MCU RESETn signal to force a reboot, and then how for USB where DTR is not a physical signal this is implemented by opening the serial port at 1200 baud and then closing it.
So it appears that this functionality is only ever useful for CDC_ACM UARTs, and for wired RS-232 there's no software needed because the reset is done in hardware.

Correct.  We can ignore the RS-232 case as well as no modern boards use it.

So: IMO the line state change detection API doesn't belong in the UART API.
In that case, I would propose to add an an option CONFIG_CDC_ACM_ENABLE_ARDUINO_RESET flag, which puts the detection logic into the cdc_acm driver and defines a non-syscall cdc_acm_arduino_reset_callback(dev, cb) function that installs a callback to be used when the event occurs. You'll need to add a small header in include/drivers/uart/cdc_acm.h to provide the ~definition~ declaration.

SGTM.
BTW, another alternative is adding a SYS_REBOOT_INTO_BOOTLOADER in power/reboot.h, but then cdc_acm has specific knowledge about the Arduino bootloader.

Reading the links above I'm not convinced the response to the event needs to erase the first word of the application: it seems it's just supposed to be a soft reset, which starts the bootloader, which may or may not perform an update. I may be wrong in that interpretation.

I agree.  Recent bootloaders will stay in the bootloader if there's a magic value at the top of RAM, and this PR uses that method.

But in any case the most flexible approach is just detecting the situation with API like the one two paragraphs up, and letting the application determine what "reset" means. (If necessary I suppose a lib/arduino could be added with arduino_reset_to_upgrade() implemented for shared use, but that should be a separate PR from adding the ability to detect the request.)

The reset code is very SOC specific and also tiny, so I'd duplicate it for other non-SAM0 SOCs if it's ever needed.

@jfischer-phytec-iot Does this plan make sense to you?

Changed to a CDC ACM specific callback.  PTAL.

@nzmichaelh Can you edit your PR title?

@nzmichaelh Can you edit your PR title?

Good catch; the previous text was confusing given the current content.  I also updated the summary slightly to clarify that this is no longer proposing a change to the UART subsystem API, just extending CDC-ACM UART device.

Thanks for all the comments everyone.  The remaining issue is identifying the CDC ACM device name, but otherwise any objections?

Thanks for all the comments everyone. The remaining issue is identifying the CDC ACM device name, but otherwise any objections?

That's correct, thanks!

Hi folks.  The CDC ACM device name issue has been addressed and 1 defers the decision on this landing to others.
What's the next step here?

@jfischer-phytec-iot can you please approve?


Add Polish translations

I've added also Localizable.stringsdict file, because of specificity of Polish language. In some cases I recommend You to use *.stringsdict file instead of *.strings. You can find more at https://developer.apple.com/library/content/documentation/MacOSX/Conceptual/BPInternational/StringsdictFileFormat/StringsdictFileFormat.html

@malcommac why you revert polish language? somothing was wrong?

I'm also curious why polish was reverted.


App header

Added the logic with login state.

As said, it'd be nicer if you can enhance the existing unit test to check: the new view of the header should be on screen as expected after logging in .
With that, we assure the new feature with an automated test  😊


Allow use of timezone names

Allow use of "America/Chicago" based time zone definitions in the lookup file instead of just static offsets like -5. (From @FusionFC)

I think this is possible with the toLocaleString function.

Even better... https://docs.cribl.io/logstream/cribl-reference/#time
C.Time.adjustTZ(epochTime: number, tzTo: string, tzFrom?: string): number
e.g. C.Time.adjustTZ(_time, 'UTC', 'America/Chicago')


Run t_server_null.sh tests with release/2.5 and release/2.6 clients

This has been implemented as a special buildbot worker coupled with a customized t_server_null.rc file. That special buildbot worker container image should be rebuilt periodically to ensure we're not using outdated client code.

The choice of Debian 12 the "legacy" buildbot worker was purely arbitrary.
Also: I'll look into a reasonable mechanism for rebuilding the image periodically. I'm thinking of using systemd timers as they allow building after boot and then in regular intervals. That said, a cronjob would be good enough as well.

This systemd timer and service combination seems to do the trick:
$ cat /etc/systemd/system/rebuild-buildbot-worker-debian-12-legacy.timer
[Unit]
Description=Rebuild buildbot-worker-debian-12-legacy Docker container image

[Timer]
OnBootSec=1m
OnUnitActiveSec=1w

[Install]
WantedBy=timers.target

$ cat /etc/systemd/system/rebuild-buildbot-worker-debian-12-legacy.service
[Unit]
Description=Rebuild buildbot-worker-debian-12-legacy

[Service]
Type=oneshot
RemainAfterExit=no
ExecStart=/bin/bash -c "cd /root/openvpn-buildbot/buildbot-host && ./rebuild.sh buildbot-worker-debian-12-openvpn-legacy"

@flichtenheld looks ok to you as well? This should be easily testable on the community buildbot despite the Ubuntu version differences.


The current card says Feb 17th?

@ccpowers is the current card (served by Netlify, huzza) says Feb 17th - is this correct?

I see the new card now, all good

@bahmutov Yeah sorry! I was just really delayed in posting it. Next priority is finding a way for other people to be able to upload the cards so it's not reliant on me doing it.


Can't pause the recording after Refreshing SF page while the call is being recorded

The agent will be no longer able to pause the recording if the SF page is refreshed while the call is being recorded. Also, an error will appear saying “We can’t resume the recording. Try again.”
Findings
We found out that after refreshing the SF page when the user clicks on the toggle recording button, it sends an aura request which gets failed due to a missing conversation ID.
Attachement

dlouvton sfyz
This is actually one of colleagues who asked this question, and it still seems to be an issue with the latest release
Is this a known issue on your side? Is there a fix coming down the pipe in a future release?
Thanks
@dlouvton @sfyz

@zabbasingenius Eh Zaheer is still an issue? If not can we close it?


Add support for external person honorifics and affiliations

Plato metadata contains honorific prefixes and affiliation strings for non UGent people, enable storing this data.

[x] Add HonorificPrefix and Affiliation fields to ExternalPerson
[ ] Adapt controllers and ui to support this

see #1289
see #1288

people service also has this attribute honoric prefix


WIP: Experiment Versioning model changes

Putting this here for feedback on the model changes so long, since I am busy updating the application logic. Note that I rebased this PR on Steph's branch with the UI changes that doesn't have the latest main branch's changes, which is why the first few commits are from "old" commits.
You can follow the commits when going through the PR. There are only minor changes (like naming, refactors etc) that happen in later commits, but generally it should be OK to go through this PR commit-wise.
Description
These are the model changes.
Things I still want to do (my list so far)

[ ] Deleting experiments according to the spec (so deprecate those with versions, hard delete those without etc)
[ ] More testing by doing random stuff on the platform and writing more tests.
[ ] Routes: Filter an experiment's own versions (or family of versions) from the list of allowed children
[ ] Update experiment sessions' experiment whenever we update the default version
[ ] Store which version generated the response in the ChatMessage metadata

This is also a good time to establish some nomenclature. I went with:

"working version" -> the draft version
version family -> refers to the set of version (including the working version) of a specific experiment.

Do let me know if ya'll are fine with this or not

Deleting experiments according to the spec (so deprecate those with versions, hard delete those without etc)
noting here we actually want to just move the working_exp is_archived = True whether there are versions or not, but if it does have versions then all of those will be archived as well

Deleting experiments according to the spec (so deprecate those with versions, hard delete those without etc)
noting here we actually want to just move the working_exp is_archived = True whether there are versions or not, but if it does have versions then all of those will be archived as well

I was thinking to maybe keep this change for a next PR. I am still in the process of referencing the versioned experiment

Closing this PR in favor of this one. The model changes are in, so it's just the UI and code changes that's left


Is it possible to truncate BPTT to less than max_time in dynamic_rnn?

I am wondering whether it is possible to feed, say, sequences of length 10000 while letting the gradients only flow back over a maximum of 100 time steps. This seem to me a pretty common thing to do in sequential learning tasks, so I am wondering whether there is support for that in dynamic_rnn and if not, whether it is planned.

We have some internal code that does this, am planning to open source it someone this summer.

The code is now opensourced on master branch.  See tf.contrib.training.batch_sequences_with_states (sorry; for now the only examples are in the unit tests).  This was added in 955efc90026ee08205ad84551dd4a308439fd741.


[Wa] alphin

Name :alphin
Link :

Check is a Wallet provider and/or for payments

[ ] Wallet Provider
[ ] Wallet for payments

Hello @alphingj ! Do you have a link about this wallet ?

Hello, no more information about your request. I close that.


JCK_GIT_REPO not being set for Temurin JCK jobs

Not sure why but JCK_GIT_REPO is not being set, thus Temurin JCK jobs are failing with:
15:56:08  BUILD FAILED
15:56:08  /home/jenkins/workspace/Test_openjdk21_hs_sanity.jck_x86-64_linux/aqa-tests/TKG/scripts/build_test.xml:95: The following error occurred while executing this line:
15:56:08  /home/jenkins/workspace/Test_openjdk21_hs_sanity.jck_x86-64_linux/aqa-tests/jck/build.xml:294: env.JCK_GIT_REPO: ${env.JCK_GIT_REPO} was not correctly set for running JCK tests. If you do not want to compile JCK tests,       please use BUILD_LIST to include test folders you want to test.

Last successful run looks like 2 days ago ?

Might relate with https://github.com/adoptium/aqa-tests/pull/5048

Those jobs in Eclispe Temurin were generated with default value and triggered with default value. Now it seemed were triggered with empty string.

No, it's not set in  AQA Test Pipeline job

Yes, https://github.com/adoptium/aqa-tests/pull/5048 loop through all params in the job and hence trigger the child job setting JCK_GIT_REPO="".

Can add JCK_GIT_REPO same as those params?  @llxia

the value of JCK_GIT_REPO is "" in  AQA Test Pipeline job

You are setting JCK_GIT_REPO to empty. This value got passed down to the child test build.
If you have the JCK_GIT_REPO default value in the child test job and would like to use it, then you just need to remove JCK_GIT_REPO parameter in AQA Test Pipeline.

the value of JCK_GIT_REPO is "" in  AQA Test Pipeline job

You are setting JCK_GIT_REPO to empty in AQA Test Pipeline job. This value got passed down to the child test build. If you have the JCK_GIT_REPO default value in the child test job and would like to use it, then you just need to remove JCK_GIT_REPO parameter in AQA Test Pipeline.

Ah thanks @llxia i've removed the blank JCK_GIT_REPO from our CI compliance Jenkins AQA_Test_Pipeline job configuration, and will see if that resolves the issue in the overnight builds. As you say the default value in the Test_openjdk21_hs_sanity.jck_x86-64_linux child jobs should then win through... thanks
@sophia-guo

I'm not sure if there is any specific reason we added parameter JCK_GIT_REPO. Looking at history it was added July 18 2023. Seems preparing for  the 2023 July release.

Think it was added when the remote trigger failed to work and we needed to launch AQA_Test_Pipeline manually on temurin-compliance server.

Close this as the issue was fix https://github.com/adoptium/aqa-tests/issues/5062#issuecomment-1944159313 .  If we need to set JCK_GIT_REPO in special cases that we might need https://github.com/adoptium/aqa-tests/pull/5064.


More Test Data

Items in this update:

Added additional *.raw test files.
Added *.txt files used to generate the test files
Documented usage of the Protoscope tool.  See the README in directory generate-test-data for details.
Updated protobuf.spicy to produce output in a more human readable format.

Note: Updates may have broken the ./tools/build.sh -t functionality
Suggested next parsing challenge:

Negative numbers.  See ./test-data/protobuf_neg_150.raw and ./generate-test-data/protobuf_neg_150.txt

Nice! The protoscope stuff is going to come in handy big time for testing as we flesh this out.


Add README example using Vuex store modules

Hi there!
Thanks for this helpful package.
I thought it might help to have an example showing how to set up shared mutations using a store module.
What do you think? 😄

@xanf please merge

Did this actually work for you? I tried to do this like your PR is documenting but my application just loses scope of the mutation.

Hey @tance77!
That's right, it worked for me.  Hrm. Is it possible your store modules aren't namespaced? I noticed that mine are.

They definitely are namespaced.
auth.js
export default {
    namespaced: true,

I wonder if it has to do with me dynamically importing my mutations as follows:
const predicates = [];
Object.keys(modules).forEach(module => {
    Object.keys(modules[module].mutations).forEach(mutation => {
        predicates.push(module + "/" + mutation);
    });
});

Sample console log of the predicates array

I figured it out. Your solution is correct. Thanks for the quick replies.
Had do do with something I commented out to accommodate  this library.

Oh hey, awesome! Nice one 👍
Glad it worked! It had been a while so I was digging into my code. :)


Change enums and slots in previously harmonized data to match text, not title in LinkML

Also, make sure Trio+ in HTP matches whatever is in model

closing bc dupe of #161


Almost embarrassingly parallel workload piles up on a single worker

https://github.com/dask/distributed/issues/6570

Looks like this is not necessarily a thing we need a big integration test for but rather should fix it on unit test level


[FEAT] Allow batch remove / add room member

Allow add and remove mutiple contacts in room can reduce low-lever IO requests, and it's also a feature supported by many IMs. e.g.:
whatsapp

We can make room.add and room.remove support both ContactInterface and ContactInterface[] to make minium impact on current codes.

https://github.com/wechaty/puppet/pull/215


Update Invoke-AnalyzerNicSettings.ps1

Change format of IP4 address.
Issue:
Output currently created by the tool looks like this:
Address: <IP_ADDRESS>\24 Gateway: <IP_ADDRESS>
Reason:
This subnet prefix formatting is backwards to any existing standard. Microsoft's own documentation here for example uses forward slashes:
https://learn.microsoft.com/en-us/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide#exchange-online
The Wikipedia page describes the format as containing a slash, and not a backslash:
https://en.wikipedia.org/wiki/Subnet
Fix:
Change the slash direction
Validation:
Not much to validate on a single character change

Closing/Re-opening as pipeline does not run

@technion will approve once you can also change the pester testing files to make them pass as well. We can't make changes to your fork to update this.

@lusassl-msft @dpaulson45 Thanks for looking at this. I've updated the Pester tests and ensured they run cleanly. I've just sent a new push.

@microsoft-github-policy-service agree


Exception layout renderer 'format=@' generates invalid json

NLog version: 4.6.6
Platform:  .NET Core 2.2
Current NLog config (appsettings.json format)
{
  "NLog": {
    "autoReload": true,
    "throwConfigException": true,
    "internalLogLevel": "warn",
    "internalLogToConsole": true,

    "extensions": {
      "NLog.Extensions.Logging": {
        "assembly": "NLog.Extensions.Logging"
      },
      "NLog.Web.AspNetCore": {
        "assembly": "NLog.Web.AspNetCore"
      }
    },

    "default-wrapper": {
      "type": "AsyncWrapper",
      "overflowAction": "Block"
    },

    "targets": {
      "console": {
        "type": "Console",
        "layout": {
          "type": "JsonLayout",
          "renderEmptyObject": false,
          "attributes": [
            { "name": "timestamp", "layout": "${date:format=o}" },
            { "name": "level", "layout": "${uppercase:${level}}" },
            { "name": "logger", "layout": "${logger}" },
            { "name": "callsite", "layout": "${callsite}" },
            { "name": "message", "layout": "${message}" },
            {
              "name": "exception",
              "encode": false,
              "layout": "${exception:format=@:innerFormat=@:maxInnerExceptionLevel=5}"
            },
            {
              "name": "request",
              "encode": false,
              "layout": {
                "type": "JsonLayout",
                "renderEmptyObject": false,
                "attributes": [
                  {
                    "name": "id",
                    "layout": "${aspnet-traceidentifier}"
                  },
                  {
                    "name": "method",
                    "layout": "${aspnet-request-method}"
                  },
                  {
                    "name": "url",
                    "layout": "${aspnet-request-url:IncludeHost=false:IncludeScheme=false}"
                  },
                  {
                    "name": "queryString",
                    "layout": "${aspnet-request-querystring:outputFormat=Json}"
                  }
                ]
              }
            }
          ]
        },
        "detectConsoleAvailable": true
      }
    },

    "rules": {
      "100_IgnoreMicrosoft": {
        "logger": "Microsoft.*",
        "maxLevel": "Info",
        "final": true
      },

      "101_IgnoreSystem": {
        "logger": "System.*",
        "maxLevel": "Info",
        "final": true
      },

      "102_IgnoreSwashbuckleFluentValidation": {
        "logger": "MicroElements.Swashbuckle.FluentValidation.*",
        "maxLevel": "Info",
        "final": true
      },

      "200_WriteToConsole": {
        "logger": "*",
        "minLevel": "Info",
        "writeTo": "console"
      }
    }
  }
}

Here's an example log output with an inner exception (formatted and stack traces removed for readability).  Notice how the inner exception data seems to be duplicated after the exception property?
{
  "timestamp": "2019-08-12T15:12:10.7470354-04:00",
  "level": "WARN",
  "logger": "Storable.Payments.Application.Infrastructure.TokenService",
  "callsite": "Storable.Payments.Application.Infrastructure.TokenService.GetClaimsFromExpiredToken",
  "message": "Unexpected error while validating token",
  "exception": {
    "Type":"System.ArgumentException",
    "Message":"IDX12723: Unable to decode the payload '[PII is hidden]' as Base64Url encoded string. jwtEncodedString: '[PII is hidden]'.",
    "Data":{},
    "InnerException": {
      "Type":"Newtonsoft.Json.JsonReaderException",
      "LineNumber":6,
      "LinePosition":5,
      "Path":"Storable_User_Id",
      "Message":"Bad JSON escape sequence: \\?. Path 'Storable_User_Id', line 6, position 5.",
      "Data":{}, "TargetSite":"Void ReadStringIntoBuffer(Char)",
      "StackTrace":"<snip>",
      "Source":"Newtonsoft.Json",
      "HResult":-2146233088
    },
    "TargetSite":"Void DecodeJws(System.String[])",
    "StackTrace":"<snip>",
    "Source":"System.IdentityModel.Tokens.Jwt",
    "HResult":-2147024809
  }
{ // this object seems to be repeated from the inner exception
  "Type":"Newtonsoft.Json.JsonReaderException",
  "LineNumber":6,
  "LinePosition":5,
  "Path":"Storable_User_Id",
  "Message":"Bad JSON escape sequence: \\?. Path 'Storable_User_Id', line 6, position 5.",
  "Data":{},
  "TargetSite":"Void ReadStringIntoBuffer(Char)",
  "StackTrace":"<snip>",
  "Source":"Newtonsoft.Json",
  "HResult":-2146233088
  },
  "request": {
    "id": "0HLOVB3SN4M8L:00000001",
    "method": "POST",
    "url": "\/api\/v1\/authentication\/refresh"
  }
}```

Pretty sure this is not supported to generate valid Json:
"${exception:format=@:innerFormat=@:maxInnerExceptionLevel=5}
Think you should stay with this:
"${exception:format=@}
The exception-layout-renderer knows nothing about the Json-format. So just concats the strings without thought.

@snakefoot Thanks for the suggestion.  I actually get the same result output after removing the inner format specifier.

@mbcrawfo Pretty sure you are doing something wrong. Maybe you modification of NLog.config has not been deployed properly. But please create sample application that reproduces invalid JSON when just using "${exception:format=@}"

@snakefoot Here you go.  It took a bit to narrow down the cause but the problem occurs when combining format=@ with maxInnerExceptionLevel.  Along the way I also confirmed that the renderer does not (appear to) care if you supply format=<EMAIL_ADDRESS>using System;
using NLog;
using NLog.Config;
using NLog.Layouts;
using NLog.Targets;

namespace nlog_json
{
    class Program
    {
        static void Main(string[] args)
        {
            var config = new LoggingConfiguration();
            var target = new ConsoleTarget
            {
                Name = "console",
                Layout = new SimpleLayout("${exception:format=@:maxInnerExceptionLevel=5}")
            };
            config.AddTarget(target);
            config.AddRule(LogLevel.Trace, LogLevel.Fatal, target);
            LogManager.Configuration = config;

            var log = LogManager.GetCurrentClassLogger();
            log.Info(new Exception("Test message", new Exception("Inner exception", new Exception("Inner exception 2"))));
        }
    }
}

In a .NET Core 2.2 console project with the same NLog versions mentioned previously that program produces:
{
    "Type":"System.Exception",
    "Message":"Test message",
    "Data":{},
    "InnerException": {
        "Type":"System.Exception",
        "Message":"Inner exception",
        "Data":{},
        "InnerException": {
            "Type":"System.Exception",
            "Message":"Inner exception 2",
            "Data":{},
            "HResult":-2146233088
        },
        "HResult":-2146233088
    }, "HResult":-2146233088
}
{
    "Type":"System.Exception",
    "Message":"Inner exception",
    "Data":{},
    "InnerException": {
        "Type":"System.Exception",
        "Message":"Inner exception 2",
        "Data":{},
        "HResult":-2146233088
    },
    "HResult":-2146233088
}
{
    "Type":"System.Exception",
    "Message":"Inner exception 2",
    "Data":{},
    "HResult":-2146233088
}

Looking at the source of ExceptionLayoutRenderer.cs, I'm not sure this is an easily fixable problem.  You had the right idea earlier about the json of the inner exceptions being blindly concatenated together, and it seems like these two options simply aren't compatible.  Honestly it feels like format=@ should have been a separate layout renderer, since enabling it does not seem to be compatible with most (any?) of the other options on the exception layout renderer, and that's surprising/unexpected behavior.  But I suppose that change would have to wait for NLog 5. 😄
Obviously there should be some warnings in the documentation about this, and I'm happy to edit the wiki tomorrow.  I don't know how flexible the NLog config validation system is, but would it be possible to add some validation rules/warnings that trigger if someone sets format=@ as well as other incompatible properties?

"${exception:format=@}" <> "${exception:format=@:maxInnerExceptionLevel=5}"
Exception-Layout-Renderer knows nothing about JSON. I just concats the strings you ask for without regard to JSON-rules.

I assume your question has been answered, if not, please let us know!


VLMO text only data, "wikibk.{index}.txt"

Model I am using (VLMO), I found that the text-onlt data is loaded from "wikibk.{index}.txt" where index=0,1,...,49，I want to ask I can I get the .txt files?

Hi @NieShenRuc, you can follow BERT/UniLM/RoBERTa to process the text-only data (EN Wikipedia and Bookcorpus), and then split it into several small files. Or you can directly use our stage-2 model which is pretrained on text-only data.


Add descriptions / prop documentation method

Add a print option to print out the description of a object and which properties can be accepted to create the object

:octocat: cibot: Thank you for posting issue #751. The person in charge will reply soon.


finish styling

Complete final styling for the page.

Noted! Thanks @levy002

@Felix45 I think you used the wrong reviewing template. But thanks for approving!

Hi @Bifabig, your project met all the requirements, I meant to approve it. I've updated the template.


Cannot use type narrowing with a generic against a type map

🔎 Search Terms
mapped type, map narrowing, generic mapped type, generic narrowing, generic return
🕗 Version & Regression Information

This changed between versions 3.3.3333 (where it worked) and 3.5.1 (where I see the same errors as on the most recent TS version) based on the playground

⏯ Playground Link
https://www.typescriptlang.org/play/?ts=5.6.2#code/KYOwrgtgBAyg9hYAVAngB2AZygbwFBSFQCGANAUQEblFQDGeAvnngC7rBSoZJwBqxADZhgAWWJooAXlwVCAbXiJuWAHTEAugC4o4CJWAAnGkUUJkHTKsraolOHEHBiIEwqUWMVurcytDAJYgAOZMLHRwIH5QaMSGmMACwpwyADxIUMAAHqygACbYHiqYAHwAFABuQiI6foEhpFDsGDpIAJStHLxJIuJo8kga0iWytAEAZlBlzSlSMkWW6m2jtESGwKxghiBQAHKQBoaV1cBtANxyUMyXE1Mz0nOw5sXWy-iraxtbOwBCDk4uY7Jc6Xa5jSbTDgPebPRZ0N6XWjrTbbKBVZIXWhgoisAAWhjgAHddMBiQBRQwEo4AcgAkiB0QE8k0ONSQdcIlFHMBVII4MEyhRYvFEicytSAIzUxoLLxLEzChI9YDiqUy2FyyhtBVxJViyXSp7KOFtPBtIA
💻 Code
enum SomeTypes {
    a,
    b,
    c
}

type TypeToValueMap = {
    [SomeTypes.a]: number,
    [SomeTypes.b]: boolean,
    [SomeTypes.c]: string
}

const parseValue = <T extends SomeTypes>(value: string, type: T): TypeToValueMap[T] => {
    if (type === SomeTypes.a) {
        return Number(value);
    }

    if (type === SomeTypes.b) {
        return Boolean(value);
    }

    if (type === SomeTypes.c) {
        return value;
    }

    throw new Error('Invalid type');
}

console.log(
  parseValue('1', SomeTypes.a),
  parseValue('1', SomeTypes.b),
  parseValue('1', SomeTypes.c)
)

🙁 Actual behavior
All return statements in the parseValue function result in an error that looks something like:
Type 'number' is not assignable to type 'TypeToValueMap[T]'.
  Type 'number' is not assignable to type 'never'.(2322)

This is incorrect because there is no value of T (which must be a member of SomeTypes) that has no mapping into TypeToValueMap, which means that there is no case where narrowing the type should result in never in TypeToValueMap[T].
🙂 Expected behavior
The code as written should have no errors because I am narrowing type to a specific value in SomeTypes, which should provide the TS compiler enough info to map into TypeToValueMap[T].
Additional information about the issue
No response

Duplicate of #43922.

Ah, looks like it is, thanks. Followed that to the original issue: #33014 . Looks like it is potentially desired but not currently planned, and that thread has some workarounds.
Will close this one.


”Holboca”, ”Cristești”, ”Dancu”, ”Orzeni” are listed in the drop-down menu

Description:
When user writes ”Holboca”, ”Cristești”, ”Dancu”, ”Orzeni” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Holboca"

Expected results
The location appears in the drop-down menu as "Sat Holboca, IAȘI (Holboca)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Cristești"

Expected results
The location appears in the drop-down menu as "Sat Cristești, IAȘI (Holboca)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Dancu"

Expected results
The location appears in the drop-down menu as "Sat Dancu, IAȘI (Holboca)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Orzeni"

Expected results
The location appears in the drop-down menu as "Sat Orzeni, IAȘI (Holboca)".


Add "Atom and XDebug with Drupal VM" section

From PR https://github.com/geerlingguy/drupal-vm/pull/1631.

Nice and simple, thanks!


Update for guild war functionality

For every active guild/guild that has called warlist at least once, the bot should automatically update the values every 24 hours. When this is done, the daily/or more than daily values depending on if the members manually call it should be made into a time/kills graph, with 2 line graphs displaying each guilds kills over time. An example is here: https://media.discordapp.net/attachments/786948013888634920/857254681479086151/line-graph_66.png?width=518&height=337  to get a graph, command ~war (guild ID) should be called; own gnild should be in blue and enemy in red.

Ill take a look at it, would have loved it if all the wars were sorted so that the guild you're looking up showed up on the same side.

Updated with new endpoints, wars are now sorted on one side and listed in order of completion.


Tag version 2.0.6

This update backports a few fixes and a lot of improvements to the tests.
The only important fix is the one that fixes potential hangs on macOS.
The buffer overwrite in minideflate is not as severe because the minideflate tool is primarily used for testing, and likely not widely used by others.
Changes since 2.0.5:

Fix hangs on macOS #1031
Fix minideflate write buffers being overwritten #1060
Fix build problems when building outside of source dir #1049
Fix build problems on arm2-7 #1030
Fixed some compile warnings #1020 #1036 #1037 #1048
Improved posix memalign support #888
Improvements to testing #637 #1026 #1035 #1051 #1056 #1063 #1067
Improvements for integration into other projects #1022 #1042
Code style fixes #637 #1040 #1050

x86-64 benchmarks
2.0.5 Stable
   text    data     bss     dec     hex filename
 111628    1296      32  112956   1b93c libz-ng.so.2

 Tool: minigzip   Levels: 1-9
 Runs: 70         Trim worst: 40

 Level   Comp   Comptime min/avg/max/stddev  Decomptime min/avg/max/stddev  Compressed size
 1     57.780%      1.212/1.224/1.229/0.005        0.846/0.855/0.859/0.004       90,924,650
 2     43.876%      1.439/1.449/1.454/0.003        0.556/0.566/0.571/0.003       41,427,218
 3     42.490%      1.533/1.542/1.546/0.004        0.460/0.471/0.474/0.003       33,432,016
 4     41.467%      1.449/1.463/1.468/0.004        0.354/0.361/0.364/0.003       26,101,482
 5     41.212%      1.588/1.594/1.598/0.003        0.353/0.363/0.367/0.003       25,940,920
 6     41.039%      1.439/1.448/1.451/0.003        0.267/0.272/0.274/0.002       19,374,021
 7     40.781%      1.295/1.300/1.303/0.002        0.176/0.181/0.183/0.002       12,834,980
 8     40.706%      1.661/1.669/1.672/0.003        0.171/0.180/0.182/0.003       12,811,166
 9     40.695%      1.834/1.839/1.842/0.002        0.173/0.180/0.183/0.003       12,807,941

 avg1  43.339%                        1.503                          0.381
 tot                                405.865                        102.840      275,654,394

 Level   Comp   Comptime min/avg/max/stddev  Decomptime min/avg/max/stddev  Compressed size
 1     57.780%      1.214/1.224/1.230/0.005        0.841/0.854/0.860/0.005       90,924,650
 2     43.876%      1.435/1.449/1.455/0.005        0.558/0.565/0.569/0.003       41,427,218
 3     42.490%      1.530/1.541/1.545/0.003        0.460/0.469/0.473/0.004       33,432,016
 4     41.467%      1.453/1.463/1.468/0.004        0.353/0.360/0.364/0.003       26,101,482
 5     41.212%      1.587/1.594/1.599/0.003        0.353/0.361/0.365/0.003       25,940,920
 6     41.039%      1.439/1.446/1.449/0.003        0.268/0.272/0.275/0.002       19,374,021
 7     40.781%      1.291/1.300/1.303/0.003        0.174/0.180/0.182/0.002       12,834,980
 8     40.706%      1.662/1.669/1.671/0.002        0.171/0.179/0.181/0.002       12,811,166
 9     40.695%      1.832/1.840/1.843/0.003        0.175/0.180/0.182/0.002       12,807,941

 avg1  43.339%                        1.503                          0.380
 tot                                405.746                        102.628      275,654,394

2.0.6 pre-release
   text    data     bss     dec     hex filename
 111838    1296      32  113166   1ba0e libz-ng.so.2

 Tool: minigzip   Levels: 1-9
 Runs: 70         Trim worst: 40

 Level   Comp   Comptime min/avg/max/stddev  Decomptime min/avg/max/stddev  Compressed size
 1     57.780%      1.215/1.232/1.237/0.005        0.854/0.863/0.868/0.004       90,924,650
 2     43.876%      1.445/1.452/1.455/0.002        0.556/0.565/0.570/0.004       41,427,218
 3     42.490%      1.536/1.544/1.548/0.004        0.462/0.471/0.474/0.003       33,432,016
 4     41.467%      1.450/1.465/1.471/0.005        0.354/0.361/0.364/0.003       26,101,482
 5     41.212%      1.590/1.598/1.603/0.004        0.358/0.363/0.365/0.002       25,940,920
 6     41.039%      1.442/1.450/1.455/0.004        0.265/0.272/0.275/0.003       19,374,021
 7     40.781%      1.296/1.301/1.304/0.003        0.175/0.181/0.183/0.002       12,834,980
 8     40.706%      1.664/1.670/1.673/0.002        0.174/0.179/0.182/0.002       12,811,166
 9     40.695%      1.835/1.840/1.844/0.003        0.174/0.180/0.182/0.002       12,807,941

 avg1  43.339%                        1.506                          0.382
 tot                                406.594                        103.033      275,654,394

 Level   Comp   Comptime min/avg/max/stddev  Decomptime min/avg/max/stddev  Compressed size
 1     57.780%      1.219/1.231/1.238/0.005        0.850/0.862/0.867/0.004       90,924,650
 2     43.876%      1.440/1.449/1.454/0.003        0.558/0.567/0.570/0.002       41,427,218
 3     42.490%      1.533/1.544/1.550/0.004        0.462/0.470/0.475/0.003       33,432,016
 4     41.467%      1.459/1.466/1.471/0.004        0.351/0.361/0.365/0.003       26,101,482
 5     41.212%      1.591/1.598/1.602/0.004        0.353/0.363/0.366/0.003       25,940,920
 6     41.039%      1.442/1.449/1.453/0.004        0.264/0.273/0.275/0.003       19,374,021
 7     40.781%      1.294/1.301/1.303/0.002        0.175/0.181/0.183/0.002       12,834,980
 8     40.706%      1.662/1.669/1.672/0.002        0.173/0.180/0.183/0.003       12,811,166
 9     40.695%      1.833/1.842/1.845/0.003        0.178/0.181/0.184/0.002       12,807,941

 avg1  43.339%                        1.505                          0.382
 tot                                406.463                        103.186      275,654,394

Slightly slower, but very close, I am not sure what caused it.

Call me party pooper (again), but I think we have at around 99% fixes I would want for next release... We're getting close, but not quite 100%...

@mtl1979 which fix do you think is missing?

@nmoinvaz I'm not sure which ones, but looking at the CI logs, there seems to be buffer overflow in deflate_quick_bi_valid.c.

Regarding the "Improvements to testing" part - can we also pick c762db1366541fe56ec95d64dad49947b27dfcf3? I'd love to see the stable version automatically tested on a real IBM Z.

@iii-i I agree, we can pull that commit and #1071

@Dead2 I was checking Windows builds like I do once in a while... Apparently Visual C++ adds implicit zero byte to string literals which breaks raw binary written using escape codes... To fix this, the lines 24-59 need to be converted as array of bytes.
I haven't seen that warning in earlier Visual C++ versions than 2019. For the sake of catching these kind of issues, we should make sure that specific warning is treated as an error.

#1075

@iii-i It seems that DFLTCC tests are failing now that we do testing on the real IBM Z.
Seems like something is different in stable vs develop.
Configure-based testing fails with:
make -C test
make[1]: Entering directory '/home/actions-runner/_work/zlib-ng/zlib-ng/test'
bad dictionary length
hello world
make[1]: *** [Makefile:49: teststatic] Error 1

CMake tests fail too, but I did not investigate whether those are any different or not.

@Dead2 This should be fixed by 4b4b8a7777b0bd7a18184cd3969683e40a395069.

@iii-i Thanks, I missed that and a couple other commits from the same PR. 👍


[Bug] “Open App When Last Option” feature has problem with an app's name with more than one word

Current Behavior
When an app's name is more than one word, it won't open it up until you type and get to the second word, even if there's no other matching name.
Example: If I want to open “Aurora Store” app I have to type auroras then it will open
Expected Behavior
In my case, it should open “Aurora Store” after I typed aur because there isn't any other app that matches.

This is a feature, not a bug. I understand the desire, but requiring that the name be an exact match prevents a significant number of other issues.


Improve granularity in resiliency metrics

In what area?

/area runtime
Describe the feature

As a Dapr user, I desire additional granularity for resiliency metrics. It can be difficult to monitor if a resiliency policy is working as expected with the metrics that are provided today. Today I can only see the following (as per docs):

dapr_runtime_resiliency_loaded: The number of resiliency policies loaded.
dapr_runtime_resiliency_count: The number of times a resiliency policy has been executed.

I would like to have:

the target the resiliency policy is being executed on, for component whether this is inbound or outbound
the circuit breaker state: open, half-open, closed
the number of times a default resiliency policy has been executed

Release Note

RELEASE NOTE: UPDATE Improved granularity on resiliency metrics

in regards to the default resiliency policy, couldn't we already filtered them by using the names or even prefixes?:

built-in DaprBuiltInServiceRetries, DaprBuiltInActorRetries, DaprBuiltInActorReminderRetries and DaprBuiltInInitializationRetries
Global defaults: DefaultRetryPolicy, DefaultTimeoutPolicy and DefaultCircuitBreakerPolicy

Just trying to see if we really need another label

/assign


Yii 2.0.13 fix, use yii\base\BaseObject instead of yii\base\Object

@c006 Please merge this. Without this change the extension does not work under PHP7, and Yii 2.1 will require PHP7

Thank you, Eseperio.


Load release list only when needed

It was something we discussed some time ago. Instead of loading the release list all the time, with this PR it's only loaded when the menu is opened. It should greatly reduce the page size on crates with a lot of releases.
Original issue: https://github.com/rust-lang/docs.rs/issues/1999
cc @Nemo157 @jsha

I finished porting the code so now JS generates the same thing as jinja used to. The problem now is that some tests relied on the DOM content to check some things, which isn't possible anymore. We could however have the failing tests ported to some "front-end testing"like what https://github.com/rust-lang/docs.rs/pull/1698 proposes.

That would indeed make it much simpler to maintain and we could directly re-use the existing template. :+1:

I applied suggestions: this path now returns HTML which is directly inserted into the DOM, allowing to greatly simplify the code and also minimize the impact on tests.

@Nemo157: Anything else that needs to be done here?

Applied suggestions.


Problems with setup

Hey guys,
Was having trouble making this work in my project, so I decided to create a brand new 2.14 project to see if I can get it to work... and I still cannot.
What step am I missing?
I basically did:
ember new dummy
cd dummy
ember install ember-burger-menu
ember g template index
then I went into application template and pasted the code from the documentations (just changed the links to index):
{{#burger-menu as |burger|}}
  {{#burger.menu itemTagName="li" as |menu|}}

      {{#menu.item}}
        {{link-to 'index' 'index'}}
      {{/menu.item}}

      {{#menu.item}}
        {{link-to 'index' 'index'}}
      {{/menu.item}}

      {{#menu.item}}
        {{link-to 'index' 'index'}}
      {{/menu.item}}

  {{/burger.menu}}

  {{#burger.outlet}}

    {{outlet}}
  {{/burger.outlet}}
{{/burger-menu}}

and went into index and pasted
Please show up hi from index
when I navigate to localhost:4200 I see a blank page with my sentence;
Bit confused on what am I doing wrong... I really would love to get this to work.
No error on console when I inspect
Thank you

@eduardo-hyodo
I had a similar experience. I ended up grabbing the css used in the demo to make it show and just used what I needed.
CSS

Closing as this issue has been abandoned. Feel free to reopen if needed.


Incorrect value set to Events.EVENT_TIMEZONE (Android)

I have been investigating some timezone related issues and found this in the plugin code:
https://github.com/builttoroam/flutter_plugins/blob/develop/device_calendar/android/src/main/kotlin/com/builttoroam/devicecalendar/CalendarDelegate.kt#L390
I am quite sure displayName is not the correct value to put there, you should use ID instead (as told e.g. here).
So why does it seem to work then without issues? I think this is because the value (DateTime.millisecondsSinceEpoch) passed from Dart is UTC time. Setting invalid value to EVENT_TIMEZONE is probably ignored and actual timezone is reverted to UTC.
So I think it is also a bug even trying to put the current device timezone to EVENT_TIMEZONE, because the Event.start is UTC.
Fix options:

Save in EVENT_TIMEZONE the UTC timezone.
Convert Event start/end to local time and save in EVENT_TIMEZONE the device timezone.
Remove setting EVENT_TIMEZONE at all? Maybe the default is UTC? Should be tested.

Or have I misunderstood something?

I haven't seen any issue with using displayName except it uses wrong (or null) timezone when you use your local calendar in Samsung phones. Just tried out with using id and it seems to be working.
Thanks for the point and I'll put this in so it's available in the next release.

All TimeZone variables are now using id in v3.1.0, closing this ticket.


[stable/cluster-autoscaler] Fix default image on README

Is this a new chart
No
What this PR does / why we need it:
The default image tag on README differs from values.yaml.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

Thanks! Please bump the patch version in Chart.yaml.

/ok-to-test
/assign
/lgtm


upgrade to 1.4.6

doesn't run the introSteps function

nicolas will make a change to the current version to remove a dependence on game in the intro steps, and put that check behind the unmanagedGame short-circuit
use unmanaged game flag in empirica context
check for round and stage in the game object before rendering its children.


Read Only Not Throwing Expected Error

When opening a snapshot/version using the open_read_only() method, it is possible to write and commit changes to a version without a runtime error. The test case is currently failing, and needs a fix that will throw a Privateer specific error regarding read only protection.

This has been fixed in the v0.1-uffd branch in my fork of Privateer


can't pickle dict_keys objects.

I chose to use multiple GPU for training,I found a problem.The question:type:  can not pickle dict_keys objects.
I don't know how to solve it，After checking, this problem only occurs on Windows system, but I use ubuntu system
Could you tell me how to solve it?，Thank you

Hello,
I also encountered the same issue and it was resolved by replacing this line character.py 56 with self.labels = list(self.vocab_dict.keys())


Option to flag certain unused imports as exceptions

I have a Django app whose traditional models.py file is actually a models package. To keep it compatible with other Django functions, the package's __init__.py contains:
from .base import *

When I ran autoflake --in-place --remove-all-unused-imports src it stripped out this line, breaking my entire app, and the cause wasn't immediately obvious.
Still, autoflake correctly removed a ton of other unused imports. Is there any option to flag certain lines as exceptions, so you can run autoflake on an entire codebase but ensure that it won't incorrectly remove a line that's an exception to the rule?
Pylint supports this feature with a syntax like:
some_line # pylint: disable=rule-name

Does autoflake support anything like that?

I think it will leave the import there if you have any comment inline.
from .base import *  # Hello.


[QUERY] Integration with InfluxDB

Hello!
I have seen that there is the possibility of adding other databases, such as InfluxDB, and we wanted to know if you have implemented the integration with that database or in what state that possible implementation is.
A greeting.

Hi,
During the first months of the project we did some tests with CrateDB, InfluxDB and RethinkDB, just as a proof of concept we developed a oversimplified translator for each. You can still find some remaining in the translators folder. The Influx translator deals with entities a bit differently, as it has to work with the concept of measurements.
It'd be possible to work on Influx's translator to catch up with the development done for the Crate one, but it'd take some effort which is not in our roadmap for now.

The rational to invest on CrateDB are as follows:

InfluxDB does not support natively geo-data-type, which means that to cover queries related to that, we will have to introduce a layer in the API, to process all entities there.
InfluxDB (in the open source version) does not support HA configuration, limiting its usage in a production environment (unless you buy the enterprise license, which would not be anymore compatible with the ideals of FIWARE community)

Thank you very much for your quick response and the information.
I'm glad to know that the decision to choose CrateDB was after a study of the rest.
Greetings.

Hi,
Now I have a problem with crate-datasource plugin for grafana. This plugin has removed from the official grafana plugins list. How can install this plugin manually?
Thanks

Hi @jmsbernabeu , please don't hesitate to create new issues to discuss different problems. Otherwise this will not be visible to others.
Back to your question, we suggest you try the postgresql datasource plugin instead (docs updated recently) because crate has deprecated the crate datasource plugin. We'll be migrating our usecases to postgres datasource plugin as well. If you still need to use the old plugin in the mean time, the overall idea is that you'll have to clone datasource code and place it in /var/lib/grafana/plugins and probably activate it in the config file. Have a look at the grafana plugin installation docs for the details.


Add an additional amountHuman to currency fields that uses the "regular" representation

When adding grafana dashboards, it's quite difficult to deal with scaling the currency amounts stored in the satoshi/planck representations (0 decimals).
Key issues:

Each query needs to have a manual transformation that scales the currency. However, since currencies have different decimals (Dot 10, KSM 12, USDT 6, iBTC 8, ...), queries need to be broken up to transform values correctly.
For foreign assets, the problem is even more severe. For example, foreign asset 2 is LKSM on Kintsugi and USDT on Interlay. This requires different handling and more logic that is not easily representable in grafana.

Instead of handling the decimal representation in the representation layer, squid should all a amountHuman field that returns the typical representation, e.g., in BTC, DOT, KSM, iBTC, ...

Thinking about this, maybe we want to stick to a naming scheme appending Human to the original value. So, e.g., on the issue model, we'd have griefingCollateral and griefingCollateralHuman as well as backingPayment and backingPaymentHuman. Otherwise, it might be difficult to track what amountHuman would refer to.
On the oracle specifically, I think it worthwhile to add updateValueHuman but this one is a bit involved. Maybe we can reuse some function from the lib here: https://github.com/interlay/interbtc-api/blob/master/src/parachain/oracle.ts#L112
@bvotteler @peterslany what do you guys think?

Thinking about this, maybe we want to stick to a naming scheme appending Human to the original value.

I agree, it makes sense to do it in this explicit way :+1:


Adjust config values

Purpose

For reviewers

Related Github issues

Changes

Tests

Deployment Considerations
Version Notes
{ major | minor | patch }

NOTE: Include the version notes in the squash commit message

prnet cleaned-up successfully


Pricing - price extra inactivity

pricing of the extra inactivity

billing not done on STOP transaction anymore

The Transaction.END event is now used:

To price the extra inactivity
To trigger the invoice generation

I have some doubts about this PR. The changes are very sensitive to the connector status and to the corresponding OCPP status notifications.
What happens if we restart some server processes while transactions are in progress?
Billing changes are the ones below:

the billing module does not trigger the billing task on a STOP transation anymore.
It now simply puts the transaction.billingData.stop.status to Billing.PENDING!
the actual billing async task is now only created when the extra inactivity is computed.
The billing module expects to be called with a Transaction.END action (and not Transaction.STOP).
Checks have been added to make sure we do not trigger this twice
billing data thresholds are only checked during the STOP transaction

Pricing changes:

A final call to the PricingEngine is made when computing the extra inactivity.
Method: OCPPUtils.checkAndUpdateLastCompletedTransaction()
PricingEngine will create chunks to properly price the final consumption but this is part of another PR.

What happens if we restart some server processes while transactions are in progress?

The charger should resume the meter values which it was not able to send.

I have some doubts about this PR. The changes are very sensitive to the connector status and to the corresponding OCPP status notifications.

We also need to add a Job when no Status Notification have been sent. Like the CDR.

I have some doubts about this PR. The changes are very sensitive to the connector status and to the corresponding OCPP status notifications.

We also need to add a Job when no Status Notification have been sent. Like the CDR.

right

All tests are green!
Please review again ;) ... thanks

Some Transaction tests are failing because of the chunk logic used when pricing.
Exemple below: 2 sessions of one hour, consuming only 1 kWh.
This generates 60 chunks with a very low consumption.
The accumulated amount (priced at 1 euro per kWh) loses some precision.
Here is the result: totalPrice is not the expected one!

I'm not sure we can fix that one!

I'm not sure we can fix that one!

Oups!
Is this a real use case?

I'm not sure we can fix that one!

Oups! Is this a real use case?

Well ... real use-case or not, the root cause is the fact that we do not preserve the precision when storing the data in the DB.
c.f.: below an example reproducing the root cause:

totalPriceAsDecimal is correct
totalPrice is wrong and corresponds to what we do (since we convert to number the persisted data)

I'm investigation on how to persist the accumulated amount with its 'full' precision!

I'm investigating on how to persist the accumulated amount with its 'full' precision!

Okay.

I'm not sure we can fix that one!

Oups! Is this a real use case?

Well ... real use-case or not, the root cause is the fact that we do not preserve the precision when storing the data in the DB. I'm investigating on how to persist the accumulated amount with its 'full' precision!
c.f.: below an example reproducing the root cause:

totalPriceAsDecimal is correct
totalPrice is wrong and corresponds to what we do (since we convert to number the persisted data)

Problem solved by serializing the Decimal object instead of converting it to a number.
The Decimal object can be serialized and de-serialized thanks to some undocumented capabilities of the Decimal.js module.


Only ONE voting key is shown

Multiple voting keys can be linked to the same account, but explorer only shows one.
Also, it would be nice if the key's period of validity (Start and End Epoch) would be shown along with the linked key.
Extra points awarded if the period of validity is highlighted in red when it is about to expire :)

do you have an account with a linked multiple voting key?

This account has 3 linked voting keys:
http://explorer.testnet.symboldev.network/accounts/TAZBGUACETXZHAQEQZACFHTLE7OYG2FMGJHZ2SQ
And several Link/Unlink transactions in its history.

After I checked on the API, it doesn't return multiple Voting key records.
http://api-01.ap-northeast-1.testnet.symboldev.network:3000/accounts/TAZBGUACETXZHAQEQZACFHTLE7OYG2FMGJHZ2SQ
@fboucquez Do we have any endpoint return list of voting key?

I see an array there...
			"voting": {
				"publicKeys": [{
					"publicKey": "45FBCF2F0EA36EFA7923C9BC923D6503169651F7FA4EFC46A8EAF5AE09057EBD",
					"startEpoch": 100,
					"endEpoch": 1000
				}]
			}

I believe this issue had resolved in the main branch.
https://symbol.fyi/accounts/NCQG34DBGQAXA7XD2Y3EGFQOASPVRCYMOT6PLIY


Changes not being sent to sharepoint

Note: for support questions, please use stackoverflow and tag the post with #spgo.

I'm submitting a ...

[ *] bug report
[ ] feature request

What is the current behavior?
I am trying to use the SPGO tool to backup a sharepoint site and move to a different tenant after a spliting off of a part of the company.

SPGO: Populate local workspace works but editing then saving or adding files are not saved to sharepoint.  Tried with multiple sites and not working.

If the current behavior is a bug, please

Your spgo.json file (You may replace corporate PII with "xxx")

"sourceDirectory": "src",

"sharePointSiteUrl": "https://axxxs.sharepoint.com/sites/Team2",
"workspaceRoot": "c:\Users\xxx\Documents\SPGO\xxx",
"publishWorkspaceOptions": {
"destinationFolder": "/",
"globPattern": "c:\Users\xxx\Documents\SPGO\xxx\src\\.",
"localRoot": "c:\Users\xxx\Documents\SPGO\xxx\src"
},
"publishingScope": "SaveOnly",
"authenticationType": "Digest",
"remoteFolders": [
"/SitePages//",
"/SiteAssets/**/",
"/_catalogs/wp/",
"/_catalogs/master
]
}
Steps to reproduce
modify a file and save
SPGO: Publish local workspace (enter credentials)

What is the expected behavior?
Updates to be copied to share point and be visible in site pages.

Please tell us about your environment:

OS and Version Windows 10 20H2
SharePoint Location (Online vs. On Premise) Online (office 365)
Authentication Type (e.g. Digest, NTFS, ADFS) Digest (albe to populate local so know working)

More behaviour is that the source file is checked out but is never updated or republished.
Opening the edited page shows the page open for editing but in original state.
No error in VSCode

Hello @SaintsMan76 - in your spgo.json file, what is the current value of the publishingScope property?
SPGo supports three 4 different values for this:

None - No server changes when you save locally.
SaveOnly - only attempts to save the file to the server, but does not check the file in or publish the file.
Minor - Publishes a minor version of the file
Major - Publishes a major version of the file

You can read more at the documentation page here
Finally, what happens when you manually publish a file by left-clicking the file and selecting the SPGO > Publish <Major|Minor> Version option?

Hi Chris i found the issue.
I had to modify a setting on my sharepoint site
•	Open Sharepoint online management shell
•	Connect-SPOService -Url "https://xxx-admin.sharepoint.com" -Credential $userCredential
•	Log in with user with sharepoint admin role should allow MFA support
•	Run
o	set-sposite "https://xxx.sharepoint.com/sites/MySite" -denyaddandcustomizepages $false
I was then able publish and update changes.  Apologies but I couldn't find this in setup documentation.

Hey folks,
Had the same issue as @SaintsMan76 and was able to fix it by setting DenyAddAndCustomizePages to $false on the site in question.

@roschinsky good catch! Thank you for posting this potential solution!
@SaintsMan76 - did the suggestion above resolve the issue?

@chrishasz: actually @SaintsMan76 brought up that workaround. just wanted to confirm the approach. ;)

Ack! You are correct. So much for my morning reading comprehension.


[day1] Loading\unloading durations

The problem statement doesn't specify whether loading and unloading takes some time or it's an instant operation.

Loading is an instant operation within the first exercise.
Thanks for bringing up the question. I will add the clarification to the exercise text.


Add to RAM Cache on every request

This came up in the spring 2017 ATS summit.  With the LRU strategy, objects are only put on RAM cache on second request.  There is no reason not to put objects into the RAM cache on first request.

As a tunable?

You mean as a setting to adjust the behavior?  I am just transcribing my notes.  As I recall @zwoop and @solidwallofcode were discussing this.

Yeh it seems useful to make it a tunable, but to change the default I would like a lot more information, including benchmarks.

The discussion was actually around the "fast-fill" feature of the CLFUS. Basically, we should allow things to get promoted to the LRU RAM cache immediately, until it gets filled. Then we can go back to normal behavior (which I think has many benefits).

This issue will pollute ram cache, if client failed.

Yes, it is a good idea to fast fill on LRU.

Hi, I want to work on this issue. How to get started?

@alokrahul513 We had a discussion and took off the "good first issue".  This would be more complex.

Closing for now if anyone is interested in it please reopen.


Can'i install Messer on Raspberry Pi

I'm running Raspbian Stretch on my Raspberry Pi with Node.js 10.15.3 and npm 6.4.1. When i try to install Messer with
npm install -g messer
i have the error
npm WARN checkPermissions Missing write access to /usr/lib/node_modules
npm ERR! path /usr/lib/node_modules
npm ERR! code EACCES
npm ERR! errno -13
npm ERR! syscall access
npm ERR! Error: EACCES: permission denied, access '/usr/lib/node_modules'
npm ERR!  { [Error: EACCES: permission denied, access '/usr/lib/node_modules']
npm ERR!   stack:
npm ERR!    'Error: EACCES: permission denied, access \'/usr/lib/node_modules\'',
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'access',
npm ERR!   path: '/usr/lib/node_modules' }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator (though this is not recommended).

npm ERR!  Command failed: /usr/bin/git submodule update -q --init --recursive
npm ERR! fatal: Could not change back to '/root/.npm/_cacache/tmp/git-clone-8143ea22': Keine Berechtigung
npm ERR!

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2019-04-19T08_59_39_801Z-debug.log

It seems there are some problems with write permissions to some folders but i don't understand why root can write here.

Hey 👋 Messer 0.5.1 has just been published with a bunch of bug fixes - install it and give it a whirl 🤘. If you’re still experiencing this issue, please comment below with your symptoms and we'll keep the ticket open. Thanks for supporting Messer 💬

I dont think this is a Problem with Messer, i think it a Problem with npm. Problem still exists.

Yeah okay. In this case, I would just clone the repository directly and run npm start :)

ok, i've cloned it with 'git clone  and cd into ~/Messer. Then i run 'npm start' and get this error:
npm start
><EMAIL_ADDRESS>start /home/pi/Messer
> node index.js
internal/modules/cjs/loader.js:584
    throw err;
    ^
Error: Cannot find module 'minimist'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:582:15)
    at Function.Module._load (internal/modules/cjs/loader.js:508:25)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:22:18)
    at Object.<anonymous> (/home/pi/Messer/index.js:2:19)
    at Module._compile (internal/modules/cjs/loader.js:701:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:712:10)
    at Module.load (internal/modules/cjs/loader.js:600:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:539:12)
    at Function.Module._load (internal/modules/cjs/loader.js:531:3)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `node index.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?

npm ERR!  Is there still an other package.json?

did you npm install ?

Seems this works. I can start messer with npm start when i'm in the folder. But how can i start messer in a shell script?
npm start /path/to/messer doesn't work.

try node path/to/messer/index.js instead

Yes, this works. Thanks. Also non-interactive mode works

Seems there are some problems with the login because facebook has disabled the account. when i try to login with my browser i have to enter my mobile number - what i don't want to do. i want to try it with a new facebook acount.
messer cleanup should be enough to delete all login information that are saved in my home dir? can i check this? where are the files with the login information?


Kurenai no Homura: Sanada Ninpou Chou for Switch

https://vndb.org/v19853

Hey @Kalleo1 - seems like you're getting a lot of switch games hooked - I'm wondering if this looks like one you'd be able find the hook for?


No cuda support for NFSP algorithm

First of all, thanks for the repo!
I am trying to use NFSP to run some simple imperfect-information games implemented in the repo, however, I found that the networks and losses in NFSP/DQN didn't define on cuda(gpu) devices. Are there any methods to run NFSP on gpu device?
Thanks a lot~

Hmm, should be done in the usual way..? You need to install tensorflow-gpu and then set the device to the GPU.
Have you used Tensorflow with a GPU before? If not, there are many excellent resources online on how to do this. See https://www.tensorflow.org/install/gpu for installations and some tutorials, e.g. https://dzone.com/articles/using-gpu-in-tensorflow-model.
We don't have any explicit examples for this because we didn't want to depend on tensorflow-gpu because not everyone has a GPU and we figured everyone would customize their training setup anyway. If you get it working, maybe you could add an example that uses the GPU as a demonstration, that would be great.

thanks, maybe I have to set the device when building the graph. I see that other examples, such as AlphaZero, have provided the option of whether to build the graph on the GPU. I will add methods similar to that


0.20.0 upgrade guide

Removals:

:x: R.commute (see R.traverse and R.sequence) #1625
:x: R.commuteMap (see R.traverse and R.sequence) #1625

Deprecations:

:floppy_disk: R.allUniq #1626

Changes:

R.zipObj now will truncate to shorter list #1669
R.partition now works with other filterables #1655
R.equals now compares Promises by reference #1619
R.add and R.subtract coerce arguments to Number #1580, #1573
Correct arity in R.juxt, R.converge, R.cond functions #1571, #1587
R.either and R.both work with any fantasy-land compatible applicative functor #1566
Performance improvements:

R.merge and R.mergeAll#1660
Transducer implementation #1657
R.intersection #1597
R.uniqBy #1639

Additions:

:new: R.until #1692
:new: R.reduceBy #1598
:new: R.applySpec #1592
:new: R.tryCatch #1581
:new: R.clamp #1564

Minor correction: R.converge always reported correct arity. The only thing that changed is that it no longer throws for empty-function-list (the combining-function is nullary in that case).


add previous and next page which is available or not

In case if prev or next page is not available you will receive None as a value.

but  im nt getting anything like prev or next

@clinto_router.get('/paginateMedicines', response_model=Page[GET_Medicine])
async def paginate_mediciens(request: Request) -> any:
medicines = await paginate(Medicine.all())
im using fastapi and tortoise-orm


category information

The paper says that there are 3 categories of videos in the QVHighlights dataset. But, I am unable to find any category information in the annotation or elsewhere. Could you share the category information of the videos?

I would also like to know the category information from the videos please.

Hi @keflanagan and @zahid58, I am looking for the feature files for the QV highlights data. It seems the link is not working anymore. If either of you have the feature files, I would be extremely grateful if you could please share it.
Thanks in advance for your help.


mistral_model_forward_4_36() got an unexpected keyword argument 'cache_position'

Running mistral with transformers==4.42.3 will have the following error present and unable to run.
mistral_model_forward_4_36() got an unexpected keyword argument 'cache_position'

transformers==4.42.3 was not fully tested, our test only covered 4.38.
You can downgrade 4.36.2 for mistral now.  We will inform you when we support 4.42.

transformer 4.36.2 have another issue with the mistral model.
I had tested the transformers v4.40 can works with the mistral model.