Add AmpYear from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of Jplrepo, to add AmpYear to CKAN.
Mod details:
name = /mod/144/AmpYear
author = Jplrepo
abstract = The AmpYear Power Manager makes electricity more manageable. Including some new ION RCS parts.
license = GPLv3
Homepage = http://forum.kerbalspaceprogram.com/index.php?/topic/114991-105-ampyear-main-reserve-power-manager-ion-rcs-traffic-light-icons-edition-v1200-28-feb-2016/
description = [[http://imgur.com/a/4Nj4u#0]]
Features:

Power Manager and Readout displays.
Reserve Power.
Emergency Shutdown Processing Computer (and restart)
Dark-side EC power usage calculations
Realistic EC power production simulation in the VAB/SPH
Part by part list of Power Consumption and Production at your fingertips.
This is the new Full Release Thread of the old abandoned AmpYear by SodiumEyes. SodiumEyes gave permissions a while back for anyone to continue and do what they will with his old version.

Power Manager

The manager function is attached to all probes and command pods through the power of Module Manager.
The Power Manager must be ON for all subsystems to activate/work, this includes RCS & SAS.
The AmpYear menu can be switched on/off via integration with Stock Toolbar or Toolbar.
Emergency Shutdown procedure - when All power is critical will activate solar panels (if attached and not in atmosphere) and shutdown non-essential parts using electricity.
Saves all settings and window positions per vehicle to persistent save file.
Gives %age indicators of Current Battery capacities, Electric Charge usage and generation. You can transfer charge from reserve to main batteries. Parts that produce electric charge will fill Main batteries first and then reserve batteries.
In the Editor and in Flight you can get indications of total power production and consumption of your vessel. You can also get a full list of all power production and consumption parts.
Includes Dark-Side calculations for celestial bodies providing EC required to transit teh dark-side of a selected body.
Provides Estimated Time Remaining readouts for Main and Reserve batteries.
Functions using stock toolbar or Blizzy's Toolbar (can be changed via Settings menu from the Space center).
Includes correct power calculations if you have the following Mods installed (but not required for mod to function): Near Future Electrical/Solar, KAS, RemoteTech, ScanSat, Telemachus, TAC LS, AntennaRange.

Emergency Shutdown Processor

In the settings menu you can set a Power Percentage Shutdown level for High Medium and Low.
You can also set default priorities for supported modules in the settings menu.
Every part module in a vessel can be assigned a priority (High,Medium,Low), to override the default values.
You can select on an individual module level which parts you want included in ESP processing or not.
By default when power is re-gained within the vessel and the ESP was previously triggered to deactivate, then parts that were previously active will be automatically re-activated by the system progressively by priority as power returns to the vessel.
You can Execute ESP processing manually (manual override button in the GUI) which will cause all parts to be shutdown regardless of their priority setting and the current EC status.
ESP still only processes for the active vessel in flight mode. Planned future background processing is in the works.

TurnBooster
Allows you to use electrical power to harness SAS/Reaction Wheel modules for additional turning power. Boosting them by 25%.
Reserve Power

Attach special batteries to your ship to save power for emergencies. Ideal for keeping your probes from dying.
Power can be transferred to and from main and reserve power resources.
Electrical power generators (solar panels, etc) will re-charge main power followed by reserve power when a threshold is reached.
Reserve Power Battery Parts included in several sizes.

**Dark-Side Calculations. **

This allows you to select a Celestial Body, Input a Target Orbit height and AmpYear will calculate the dark-side period in Hours:Mins:Secs and also show you the amount of EC you would require to run all vessel parts that consume EC for that period.

Auto-Hibernate

All AmpYear functions will now continue to run during timewarp up to timestep 7x. Then AmpYear will hibernate. This is due to a bug in electriccharge generation in KSP at high timewarp.
A Warning Pop-up will appear and stop warp if the ElectricCharge falls below a set percentage (configurable via the spacecenter window).

Show Crew

This to show the crew and their roles rather than having to switch to the map view all the time.

KabinKraziness
An additional (optional) add-on to AmpYear. Go here for the details.
Supported Mods:
Ampyear Recognises and supports Electrical Usage by parts distributed by other mods in this list below. If you would like to see your favourite mod supported drop me a line:

Aviation Lights
KAS
Near Future Electrical
Near Future Solar
Remote Tech
SCANSat
TAC LS
Telemachus
Antenna Range
ToggleFuelCell

This mod includes version checking using MiniAVC.
If you opt-in, it will use the internet to check whether there is a new version available.
Data is only read from the internet and no personal information is sent.
For a more comprehensive version checking experience, please download the KSP-AVC Plugin
Install Instructions
Dependency: You must have installed (INCLUDED in this package). Module Manager
Toolbar - is OPTIONAL.
Completely delete AmpYear folder from gamedata before updating from a previous version.
If you also want KabinKraziness go here.
If you are looking for the ION RCS parts that used to be part of this mod go here.
Please direct questions about this pull request to Jplrepo.

We already index this mod from the exact same $kref.


Misc Fuse.Controls.Native fixes

This PR contains:

[ ] Changelog
[ ] Documentation
[ ] Tests

@mortoray bump

@nekronos Looks good, but your editor changed a lot of whitespace in https://github.com/fusetools/fuselibs-public/pull/437/commits/2e967d4a4e9933660998920c87b903e2c9945a84
Bad setting somewhere?
Shall I accept anyway (I reviewed the actual change, which is fine)

@nekronos Did you still want to get this merged?  It's just the whitespace changes (lots of them) that are an issue.

@mortoray: In general, I don't think the white-space changes are a problem on master. I generally object to changes like that on release-branches, due to the needless divergence and risk of merge-conflicts.
Ideally they should come in a separate commit, though.


Browsing history for Markdown Preview, and opening files in the same tab

VS Code has awesome feature of opening such location in source file, that was DBL clicked in the Preview. That feature could allow to use VSC as a full and clean PKM tool.
But when source file is opened for editing, thats added to history which is used by Go Back command, effectively blocking the history even if separate window is used for all the browsing.
Please:

Add a per-tab history for files which were opened in Markdown Preview, that could, for example, include addition of Back and maybe Forward buttons to the toolbar on right of tab bar.
Make so LMB in Preview Editor which can be set to have files open in it by default, would open links into the same tab.

Current issues

 When Markdown Preview is set as default editor, and user follows a link in it (by using LMB), it opens a new tab as if the user opened the link by using MMB.

https://github.com/user-attachments/assets/dd3102b2-3ab8-48c6-9df0-d0b64ec1a69a

 With use of Go Back/`workbench.action.navigateBack`

https://github.com/user-attachments/assets/fe8ca2c7-177a-435a-a79c-0c93c11a4558
https://github.com/user-attachments/assets/467e4958-38db-4d83-8541-99b54d4f6a29

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

@mjbvz can you provide context about implementation of this? I didn't see new features in markdown-language-features in a long time.


Bugfix issue 80 - Add CLI _process_args method for issue 80

Here is my first pass at trying to fix #80
Expand the "spoiler" section below for explanations of the issue, the approach I took, and why it's not perfect.

Expand for summary of Problem
When using the CLI command such as:
`youtube_transcript_api -tiUDjn7-as --languages en --exclude-generated --json`
The -tiUDjn7 leading - video ID case gets interpreted by argparse.parse_args() as an option which removes it as an argument passed in making the CLI raise an error youtube_transcript_api: error: too few arguments.
The CLI can accept 1 or more video ids and those could all potentially meet this same case. So this means we have to iterate over the args passed into the YouTubeTranscriptCLI. The incoming args for the command above looks like this:
['-tiUDjn7-as', '--languages', 'en', '--exclude-generated', '--json'] and again there can be any number of video IDs. So my thought was to iterate over all of it parsing just the specific leading dash case and leave the rest of the -- and other args in-tact.
Also, we just cannot accurately determine if we are dealing with an argument versus an option if we run into a video ID that leads with --. It just breaks the CLI argparse expectations since youtube accepts these characters. I believe there is an alternative way that we could fix this to avoid this solution entirely but it would break the existing way commands have to be written using the CLI which would be even less ideal.

Expand for summary of solution
Using this new `def _process_args` method we can use it to temporarily mutate our video_ids.

I _process_args using the incoming args:
['-tiUDjn7-as', '--languages', 'en', '--exclude-generated', '--json']
_process_args iterates and only mutates the video ID strings in this arg list to something like:
['#tiUDjn7-as', '--languages', 'en', '--exclude-generated', '--json']
We allow argparse.parse_args() to run, now without error. Which returns a Namespace object:

Namespace(list_transcripts=False, video_ids=['#tiUDjn7-as'], languages=['en'], exclude_generated=True, exclude_manually_created=False, json=True, translate='', http_proxy='', https_proxy='', cookies=None)

Now we need to revert the video ID strings back to their original state which we have to do on the stored .video_ids attribute that was returned back from argparse.parse_args() BEFORE we try to move forward with transcript retrieval. This is done with the same method self._process_args([...], revert=False) but passing revert=False swaps # with -.

Namespace(list_transcripts=False, video_ids=['-tiUDjn7-as'], languages=['en'], exclude_generated=True, exclude_manually_created=False, json=True, translate='', http_proxy='', https_proxy='', cookies=None)

Now we avoided collisions with argparse and the transcript should be able to be retrieved.

@jdepoix Let me know your thoughts when you have time. We should definitely discuss more around this. I dont think the solution is perfect. It could even be implemented as a utility function instead. But I thought this was a good initial pass and it works for the single leading dash case. 🙂
NOTE: I will get test cases in for coverage issue after we discuss this approach more.

Hi @crhowell,
once again, thanks for taking the time!
I thought about this a bit and came up with another solution. Check it out in #103. In this PR I enabled the CLI to handle masking hyphens in the arguments like: youtube_transcript_api "\-abc123".
I think I like this solution better as it seems less invasive and using a \ to mask characters is common practice. What do you think about this?

@jdepoix Yes, I do like your idea of removing an added prefixed character instead of trying to find and mutate the original arg value to a new value and then back like I was doing.
Though when I run:
youtube_transcript_api -tiUDjn7-as --languages en --exclude-generated --json
I do still end up with the original issue.
If I am reading your comment above correctly. The idea is that I would when manually running a CLI command I would just need to know to append a leading \ before the video id?
My other thought (I totally dont know how common use-case this is): Would this create a problem since it is a CLI command? I imagine some users may have main program scripts that potentially run a YouTube ID scraping process and then feeds those video_id values into a call to a subprocess that runs this CLI command. So they would need to alter their script to prefix that \ to the CLI command before it gets passed into a subprocess call?

Exactly, you'll have to run: youtube_transcript_api \-tiUDjn7-as --languages en --exclude-generated --json
Regarding your described use-case: in general I would say using this module for more sophisticated/automated tasks probably requires using it as part of a python script anyways, so I am not sure if that problem would occur often. But even if it does, the user could implemented replacing all - with \- in the video id before passing it to the CLI.

@jdepoix Fair and I don't want to be too worried about a problem no one has. That is what Git issues are for in any case! 🙂
I imagine it will be fine as long as we document the README  for this interesting - edge case and the need for \ it should be fine. The tests pass and look like it should even work for the -- case as well. 👍

I am going to go ahead and close this PR, #103 does seem like the better approach to take. 🙂

Nonetheless, thank you for your efforts @crhowell ! 👍


fixing eco parser test

@kramasamy @nwangtw This PR fixes the broken EcoParserTest.

@kramasamy  Once master build is fixed I will merge in those changes

Looks like integration test failed. Need to retrigger travis.

@nwangtw Didn't realize I had the permissions to restart build.  I don't think I did when Heron was in Twitter repo.   Just restarted it.  Thanks for bringing this up.


Changelog

Could you please update the Changelog.md?

@dotmagic could you help me out? 🙏

@yowainwright Mmmm? It's just adding the changes from git history. I think you're the one who knows best what happened between the releases?

@dotmagic so you don't want to help out? I assume you can look at history and commit comments.

Haha, I just did and its slightly confusing. It seems the only changes between 2.0.6 -> 2.0.13 was:

2.0.7 Add es-check
2.0.13 Fix markdown issue in README.

The other releases just bump the version!? Am I missing something? You really want me to create a pull request for that?

Yes please!
I'm not trying to sound rude. I would just like help.
I would like to automate the changelog.md. If you would like to do that = event better!!!
~Thanks again

@dotmagic SORRY! I will update this changelog. If you would like to PR to automate a changelog that would be awesome.

@yowainwright Sorry, I cannot help here, but for automating the changelog generation you can use https://www.npmjs.com/package/release-it which does this among other things. It does not look difficult to setup, but I guess one needs npmjs and github permissions to actually do it.


[Enhancement] Add some parameter checks in DeltaWriterBuild

Fixes #31932
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:heart_eyes: pass : 25 / 25 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/storage/lake/schema_change.cpp
1
1
100.00%
[]

:large_blue_circle:
src/runtime/lake_tablets_channel.cpp
1
1
100.00%
[]

:large_blue_circle:
src/storage/lake/delta_writer.cpp
17
17
100.00%
[]

:large_blue_circle:
src/storage/lake/async_delta_writer.cpp
6
6
100.00%
[]

@Mergifyio backport branch-3.2


No manifest.json file found 'manifest.json'

Hi @nyroDev ,
Thanks for this promising card.
Unfortunatly, I've try to install through HACS and got this error message:
"No manifest.json file found 'manifest.json'"
If you can have a look, it will be great.
Thanks a lot et bonne journée à Besançon 😄

Hi @urbatecte,
I updated the doc to be way more verbose.
When you add the repository into HACS, be sure to choose "Lovelace" in the category field and it will work.

Perfect !
Many thanks


bgpv2: correct service reconciler initialization

Service Reconciler Init should initialize the diff store, not clean up.

/test


Edited metadata for Settings LRs for global LR page

Reviewed the metadata tags for the learning resources on the Settings > Learning resources page.
Added and edited a few tags as appropriate.
Mostly, broadened the scope of the Getting Started Guide so it will appear in any product family.
Edit: After discussing with notifications/integrations PM, we determined it would help users to have these cards appear when filtering for each product family - added more product families.

Hi @florkbr @ryelo  - this is ready to review and merge when you can. Thanks so much!


Session manager improvements

Intent

Improve SessionManager performance by returning hot session right away instead of waiting for new session to be created.
Improve error handling.
Refactoring

Implementation

src/SessionManager.ts has been refactored.
Added additional unit tests to src/test/SessionManager.spec.ts.
Fixed typo in src/test/RequestClient.spec.ts.

Checks
No PR (that involves a non-trivial code change) should be merged, unless all items below are confirmed!  If an urgent fix is needed - use a tar file.

[x] All sasjs-cli unit tests are passing (npm test).
(CI Runs this) All sasjs-tests are passing. If you want to run it manually (instructions available here).
[ ] Data Controller builds and is functional on both SAS 9 and Viya

@allanbowe, please review this changes


Default framework images such as rails, django don't work with Kitematic.

I'm trying to run the official Rails image and the container immediately shuts down with "Switch to inspect mode" in logs. No further logs are available. The same is happening with the official Ubuntu image, consistently on two different OS X machines.

Thanks for the issue! Sorry you hit a dead end trying to run the official Rails image with Kitematic.
From what I understand, the official rails image is meant to be used as a base image for your own Dockerfiles. It's page on DockerHub might help: https://registry.hub.docker.com/_/rails/
In this case running it in Kitematic won't work since it effectively runs nothing.

I see, thanks! That would not explain the same thing for the Ubuntu image though, no?

Hi @marcinwyszynski, Ubuntu image is a base OS image which is used in other images, but doesn't technically "run" as a prolonged process.

Are there any official OS images that are not only intended as base images but actually "run" as prolonged processes? I tried Ubuntu and CentOS and both stay stopped in the UI (and no process shown using docker ps).

Good to know! Base image won't start! Noted!

Same issue here with rails.

Closing this issue due to lack of update/activity - Feel free to re-open if appropriate.


Implement new Pipeline add/remove error-message-handler services

Implement new services for client error-message handling
/**
 * @brief Callback typedef for a client handler function. Once added to a Pipeline,
 * the function will be called when an error message is received by the Pipeline's bus-watcher.
 * The callback is called from the mainloop context allowing clients to change the state of, and
 * components within, the Pipeline.
 * @param[in] source name of the element or component that is the source of the message
 * @param[in] message error parsed from the message data
 * @param[in] user_data opaque pointer to client's data
 */
typedef void (*dsl_error_message_handler_cb)(const wchar_t* source, const wchar_t* message, void* user_data);

/**
 * @brief Adds a callback to be notified on the event of an error message received by
 * the Pipeline's bus-watcher. The callback is called from the mainloop context allowing
 * clients to change the state of, and components within, the Pipeline
 * @param[in] pipeline name of the pipeline to update
 * @param[in] handler pointer to the client's callback function to add
 * @param[in] client_data opaque pointer to client data passed back to the handler function.
 * @return DSL_RESULT_SUCCESS on success, DSL_RESULT_PIPELINE_RESULT on failure.
 */
DslReturnType dsl_pipeline_error_message_handler_add(const wchar_t* pipeline,
    dsl_error_message_handler_cb handler, void* client_data);

/**
 * @brief Removes a callback previously added with dsl_pipeline_error_message_handler_add
 * @param[in] pipeline name of the pipeline to update
 * @param[in] handler pointer to the client's callback function to remove
 * @return DSL_RESULT_SUCCESS on success, DSL_RESULT_PIPELINE_RESULT on failure.
 */
DslReturnType dsl_pipeline_error_message_handler_remove(const wchar_t* pipeline,
    dsl_error_message_handler_cb handler);

/**
 * @brief Gets the last error message received by the Pipeline's bus watcher
 * @param[in] pipeline name of the pipeline to query
 * @param[out] source name of the GST object that was the source of the error message
 * @param[out] message error message sent from the source object
 * @return DSL_RESULT_SUCCESS on success, DSL_RESULT_PIPELINE_RESULT on failure.
 */
DslReturnType dsl_pipeline_error_message_last_get(const wchar_t* pipeline,
    const wchar_t** source, const wchar_t** message);

Example python callback..
##
# Function to be called on New-Error-Message received by the Pipeline bus-watcher
##
def error_message_handler(source, message, client_data):
    print('Error: source = ', source, ' message = ', message)
    dsl_main_loop_quit()

// add the callback to our Pipeline
retval = dsl_pipeline_error_message_handler_add('pipeline', error_message_handler, None)
if retval != DSL_RETURN_SUCCESS:
    break

this is complete and checked into the rtsp_stream_mgt branch


[follow-up] Codeunit 90 "Purch.-Post" - event OnBeforeCheckItemQuantityPurchCredit

Hello,
In https://github.com/microsoft/ALAppExtensions/issues/22434 we requested an event.
It is still not implemented. Can you please let us know when it will be implemented? Thank you.

This is implemented in v.23
    IsHandled := false;
    OnBeforeCheckItemQuantityPurchCredit(PurchHeader, PurchLine, IsHandled);
    if IsHandled then
        exit;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: .


Added support for custom jwtid value

Description
This PR adds support for a custom JTI value being provided in the generateToken method. This can be used by customers to use their own provided JTI guid value. One use case would be using Entra-ID to generate tokens where customers want to provide their own JTI value.
Breaking Changes
Adds a new optional arg - jti - in generateToken.
Reviewer Guidance
Do I need to update this method definition in all packages to support custom JTI values?

Is it intentional to not also add this param to the services-client function of the same name?

I am not sure if I do need to add this update to other packages as well. I can update the implementation if needed. However, if this feature is only to be used for Entra-ID access, we need not update the services-client package. Lemme know what you think. Maybe @yunho-microsoft  can also add some feedback on this.


Update upgrades.md

Changed "Node:" to "Note:" because it was a note :)
Also a few minor changes for clarification.

NOTE: After opening the PR, please un-check and re-check the "Allow edits from maintainers" box so that maintainers can work on your patch and speed up the review process. This is a temporary workaround to address a known issue with GitHub.>
Please delete this note before submitting the pull request.

This change is

/assign @heckj

/approve
/lgtm


Add 'rank mode' to practice menu

Practice menu should let you try out the 17-or-so rank challenges.
At the start, only ranks 7Q, 6Q, 5Q and 4Q will be shown. If you clear those, all 17 ranks will be revealed: 7 kyu, 6 kyu... 1 kyu, 1 dan, 2 dan... 10 dan, master. Most of them will be locked, and the unlock criteria will be convoluted. For example, to unlock 1 dan, you will need to either get an S- on 1 kyu, or an S on 2 kyu,  or an S+ on 3 kyu, etc. etc.... Ranks 5 dan and above cannot be skipped.
The UI can be done with a dynamic slider and a single button. The slider has a range from 7 kyu up to your highest playable rank. As you change it, the button tells you the mode, score and grade.
We want this early so it's there when we solicit feedback from expert level players. We want this for several reasons:

We want people to be able to show off how good they are
We want a specific challenge people can't beat, so they're motivated to improve
We want to know if the rank requirements are completely out of line such that nobody could ever complete them
We want to know where average, above-average, and expert players will plateau so we can balance story mode around it. Average players should finish the game in about 10 hours. Expert players should have a shortcut to blitz through the boring story stuff -- so if expert players can get $1,000 in 80 seconds, that's where we want that shortcut lever.

This work has been started on the rank-mode-scenarios branch.

Version numbers need to be added to these new scenarios.


Edit the command with prompt_toolkit instead of readline

The readline on some Python versions is based on libedit, which doesn't support either set_startup_hook or insert_text - it just sits there waiting for input. IPython uses prompt_toolkit for editing, which feels like a more modern solution. It can even highlight the shell command using Pygment's BashLexer.
Fixes #3

I love the prompt toolkit thing!
I'm not sold on Pyments as a default - mainly because on my terminal it actually makes things harder to read (red on black):

So not sure it's worth the dependency weight.
Maybe there's a way to make it optional.

https://pypi.org/project/prompt-toolkit/ looks like a clean new dependency - BSD licensed, widely used.
Actively maintained on https://github.com/prompt-toolkit/python-prompt-toolkit for 10+ years, and good documentation: https://python-prompt-toolkit.readthedocs.io/en/stable/

I'm going to land this as-is and then decide what to do with pygments.

Suggestion for pygments handling in #7


Redirect URI should end with /

Per this issue, ADAL should probably help users out by adding a trailing / or error out when one is not provided in the redirect URI.

@MarkHerhold  The issue that you pointed out is for ADAL. Are you seeing the same issue in MSAL?

@nehaagrawal The issue exists will both ADAL and MSAL.

@MarkHerhold  The issue that you reference was for safari browser where cookie was disabled and our library can't work if cookie is disabled. That issue was closed. I saw your comment and it seems your issue is related to reply url not ending with slash.
Can you please send me the exact error that you are getting. Also if it's related to slash, it should be reproducible in all browsers. I am not clear why it's reproducible only in safari.

@MarkHerhold I just did a generic test on my mac using the latest Safari in Sierra.  I used our VanillaJSTestApp in the repo.  Have you tried that also...just wondering if there is a specific setup I can do to repro this.

See comment in referenced issue. Closing.


New line character ignored for flow parameters

I have two flow files:
# reusable_flow.yaml
appId: app.id
---
- stopApp
- openLink: ${DEEPLINK_URL}
- assertVisible:
    text: ${EXPECTED_TEXT}

# specific_flow.yaml
appId: app.id
---
- runFlow:
    file: deeplink_flow.yaml
    env:
      DEEPLINK_URL: https://google.com
      EXPECTED_TEXT: |-
        Lorem ipsum dolor sit amet, consectetur adipiscing elit.
        Mauris ut molestie eros.

If I run the specific_flow.yaml, it tries to assert the "Lorem ipsum dolor sit amet, consectetur adipiscing elit.Mauris ut molestie eros." string instead of "Lorem ipsum dolor sit amet, consectetur adipiscing elit.\nMauris ut molestie eros.".
I've also tried:
# specific_flow.yaml
appId: app.id
---
- runFlow:
    file: deeplink_flow.yaml
    env:
      DEEPLINK_URL: https://google.com
      EXPECTED_TEXT: "Lorem ipsum dolor sit amet, consectetur adipiscing elit.\nMauris ut molestie eros."

But the \n character was ignored as well.
And if I don't use the env vars, everything works as expected:
# reusable_flow.yaml
appId: app.id
---
- stopApp
- openLink: ${DEEPLINK_URL}
- assertVisible:
    text: |-
        Lorem ipsum dolor sit amet, consectetur adipiscing elit.
        Mauris ut molestie eros.

Hi @mugbug, thanks for reporting this. We'll take a look!

@mugbug for now you can pass your multiline string with the \n character present and it should work
# specific_flow.yaml
appId: app.id
---
- runFlow:
    file: deeplink_flow.yaml
    env:
      DEEPLINK_URL: https://google.com
      EXPECTED_TEXT: Lorem ipsum dolor sit amet, consectetur adipiscing elit.\nMauris ut molestie eros.

There is indeed a bug passing multiline strings as a parameter but I don't foresee this being prioritized very soon due to bandwidth so please use this workaround for now. Let me know if this works for you!


Running process_specificity() gives deprecation warning, use process_seqtol() instead

In analysis/README.rst, change process_specificity() to process_seqtol()

This deprecation warning has been present for years. I will change this since it is confusing. The solution is for the user to specify all the options i.e. make the process_seqtol function call that process_specificity makes with the default arguments.

The default arguments are the same between the two functions.

Yep, I just noticed - this is fixed now in the repo.


Ajr56 builtin circuit breaker

Thanks for these changes. I haven't tested them but everything looks good. Will you update the readme with new documentation for your changes?
Thanks.

Yes. Do I do that on the branch? Will I have to make a new pull request?

Documentation has been added.


Sorting of properties

As mentioned here, it would be really helpful and a time-saver to sort the properties always the same way, so random sorting won't pop up in any source diff. WDYT?
PropertyComparator.java is used for sorting.
We are mostly interested in YAML but would propose to sort also in JSON and XML.

From looking at the code, it looks like the properties should always be sorted.
Any object properties not listed in the arrays will get sorted alphabetically in this line:
https://github.com/spdx/spdx-java-jackson-store/blob/d9673e80c7566335b242213ceba553b40747a943/src/main/java/org/spdx/jacksonstore/PropertyComparator.java#L85
I did notice that the arrays are not sorted and the packages and files may be in a different order.
I'm going to close this for now.
@MarkusTe if you disagree with the above analysis, please let me know where I went wrong or provide an example and I'll re-open.
Also, if you feel it would help to sort the arrays, please open another issue and I'll take a look at it on the next release.


About 'node' in prompt of contextual_precision

❗BEFORE YOU BEGIN❗
Are you on discord? 🤗 We'd love to have you asking questions on discord instead: https://discord.com/invite/a3K9c8GRGt
Describe the bug
I noticed that the contextual_precision prompt mentioned 'The retrieval contexts is a list of JSON with three keys: verdict, reason (reason for the verdict) and node. ' However, when I actually used the contextual_precision, I did not observe the generated prompt containing the 'node' key. Why is that?
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@Ishiki-Iroha Must be a bug we forgot to remove in the prompts, thanks for bringing it up!


No error message is shown when input name contains brackets

We have this input:

and this corresponding validation rule:
$rules = [
'customer.name' => 'required',
];
If empty input is submitted, we should have an error, but no error or message was set.


[Feature Request] When workers detect a gym changes hands, send notification.

I know gym data is already stored with info about the defenders, so encounters are already happening and being saved in the database. It would be helpful if, when a gym changes hands, information about the change were sent to notifications and/or webhooks. Similar scripts like RocketMaps already have this feature, and PokeAlarm is already set up to receive them.

Yes please and a report on each gym!

Im looking forward to this one, it'll make my keeping track of gym changes a whole lot easier :) I might give it a go, but it would most likely be a hack since Im not a python coder, but I see in the web.py console that it displays the new gym icon, so there is definitely a trigger of some sort when a change occurs.

Where are you seeing this gym data? I just checked the database, it is not in there.

@DavePlater Im just seeing when /gym_data is called that when a gym changes color then a new fort is loaded. So whenever a new fort image is loaded then it seems like that would be a good time to send a notification...

Oh I thought maybe you had found the data on all the defenders and I was looking for it.


Documentation for Queue

Fixes https://github.com/scalaz/scalaz-zio/issues/186
Note: this PR describes the full features of ZIO Queue as developed in https://github.com/scalaz/scalaz-ioqueue . It is intended to be merged only after those features have been merged back to ZIO. That explains why Tut is failing now.

Looking good!

@jdegoes @wi101 this PR is now ready to be merged


rendering problem of potatoe field example

https://pyriemann.readthedocs.io/en/latest/auto_examples/artifacts/plot_detect_riemannian_potato_field_EEG.html
see figure with traces. the second axes is empty

Last commit has completed the legend of figure.
@agramfort , I let you check documentation display and close the issue.


release-2.2 mutual tls config key and peer property key not matching #260

I have done the changes as follows:

change property names from tlsClient* to client* in NetworkConfig.java; add unit test and associated fixture.
change from tlsClient* to client* in HFCAClient.java; add unit test and associated fixtures.
Refactor clientCert* and clientKey* to constant in Endpoint.java and apply the constants in Channel.java.

Closes #260

@Mergifyio backport release-2.2


change grad_cam initialization from np.ones to np.zeros

Per discussion with @nalinimsingh here; this should be np.zeros (it won't change the visualization qualitatively, but the numerical values with be shifted by -1 now and properly centered at 0).

@nsthorat @dsmilkov -- When you have a chance, could one of you review this PR?
Once you do, I'm happy to update the GradCAM test to reflect this change in another PR that follows (I'm unable to edit this one). Otherwise, if someone else is willing, the change could just be made in this PR.

Ups, we missed this PR. Sorry! Thanks for fixing this!!!


all-packages: add static exe alias for haskell 'shake' package

This executable exposes a ninja-compatible build system named shake, which is useful in its own right for people who use tools like CMake, etc.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS

@GrahamcOfBorg build shake


toDefineExpr for Either type is missing the TypeVarMap

Given a simple record type which auto derives Atomable and Tupleable, if it contains an Either type field, toDefineExpr returns an invalid expression:
data TestType = TestType { eTextInt :: Either Text Int }
  deriving (Generic, Show, Eq, NFData, Atomable, Tupleable)
  deriving W.Serialise via W.WineryVariant TestType

In ghci:
> toDefineExpr (Proxy :: Proxy TestType) "testType"
Define "testType" [NakedAttributeExpr (Attribute "eTextInt" (ConstructedAtomType "Either" (fromList [])))]

What is expected:
> toDefineExpr (Proxy :: Proxy TestType) "testType"
Define "testType" [NakedAttributeExpr (Attribute "eTextInt" (ConstructedAtomType "Either" (fromList [("a", TextAtomType), ("b", IntAtomType)])))]

Good catch- we were just missing a function in Atomable.


fix(tests): Distinguish between deploy and up args in integration tests

spin deploy does not support the same set of args as spin up. This helps distinguish that in the tests, right now most apparently for --sqlite

Hi Kate,
I am a little reluctant for this change as well. with this now user will need to be aware of what should go into "up_args", vs what should go into "deploy_args".
In past, if there was a flag that is supported on spin up but not with spin deploy, we would add that as a separate test scenario (e.g. header_dynamic_env_works which made use of --env) and disable it with spin deploy.
Similarly if there is a test which is going to be available only on spin deploy, it should be added to cloud tests repo only.
I hope this explaination makes sense.

It sounds like we want to keep this as is. I think we can close this and resurface it if other contributors find a separation of arguments useful


Log view with Socket Listener has sizeable lag

When using a log view with the Socket Listener, sometimes I have as long as 4-5 seconds between the time the log is sent from the application (logback) and displays in OTROS.
The socket type is "Logback - socket", port is 50505.
I cross-checked it with LogMX and that displays the messages near instantly, so the delay is not on the application/logback side.
NOTE: I don't seem to find any setting for the delay on the log table/view or in the Preferences, so it is probably hard-coded.
OS: Linux Mint 19.3 x64
OTROS: 1.4.13
JRE: openjdk version "11.0.8"
Memory: -Xmx1024m (unchanged)

BTW: Thanks for all the hard work on OTROS, it has been a great help for me over the years.

Hi.
OtrosLogViewer is parsing logs and adding them in batches to UI. It uses https://github.com/otros-systems/otroslogviewer/blob/master/OtrosLogViewer-app/src/main/java/pl/otros/logview/BufferingLogDataCollectorProxy.java
As you can see here: https://github.com/otros-systems/otroslogviewer/blob/master/OtrosLogViewer-app/src/main/java/pl/otros/logview/gui/actions/StartSocketListener.java#L72 logs are added to UI every 4000ms. It is not configurable.
As a workaround you can change this value to 100ms and build application with ./gradlew dist
Additionally if you are parsing input from socket OLV can't tell if buffered input is complete log line or more data will be send. Due to this last log event is not parsed and send to UI.

@otrebski Many thanks


'payload too large' error when adding 262kb json document

this is printed in the console in chrome:
http://<IP_ADDRESS>:1234/my/myDB/person/insert_doc Failed to load resource: the server responded with a status of 413 (Payload Too Large)
not sure if it's the same bug, but when i press the 'edit' button on a json document of the same size in this collection, i get 'Error - Document not found' and a link to 'take me home', although nothing is printed in the console.

This is an issue with Express bodyparser defaulting to 100kb. I will make a change to up the limit to a few megabytes.

OK, Mongodb document size limit is 16Mb currently according to the doco.

Fixed in 6932ead0d6e799d1bc18082b9e7ce26569401ba7. I set the value to 16MB. As you say the largest BSON document size is 16mb but I'm reading that BSON documents are generally larger than raw JSON due to additional metadata. The limit should cover off 99.99% of instances.


change default desynchronized drawing context attribute to false

fixes #1908

Coverage remained the same at 80.578% when pulling cd6dfe5380203136df5afa53fc6d5a8465f211a6 on desynchronized-1908 into fb8f358fa0245a7446c9d59cc17e214aa32525e5 on master.


Display Bug Fixes

Updating bugs in the display, including numbers without enough decimals and market naming.
Description

Create futures long/short names
Implement new names on market dropdown
Implement new names on position card
Add decimals to position card values for funding and small percentage changes
Remove hard-coded percentages

Related issue
Issue
Motivation and Context
Squashing
Screenshots (if appropriate):
New market copy:

New position copy:

@asherism I'm also going to add "Position Side" instead of "Position" for short/long


rename IstioCluster to MutableCluster

I felt MutableCluster is a better name compared to IstioCluster
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[X] Does not have any changes that may affect Istio users.

@howardjohn @hzxuzhonghu PTAL if this rename makes sense

feel free to remove the hold once typo is fixed


Some drops are missing in the table

This wasn't a bug, but admittedly the information on how the FM APworld implementation works is so bad that I didn't realize until I stepped back and looked at all what is going on.
Cards that are always out of logic, but still obtainable, still grant checks, but the checks are guaranteed to be useless items. The tracker is knowledgeable of this, so it hides those cards. The built-in AP multiworld tracker on the website is not, and doesn't.
Usually this is reported with examples of these "never in logic" cards. I'm closing this until I can find a case with in-logic cards.


Suggesstions on README

Hi, i'm a new learner. While following the README to get start with addon, i found some problems in Deploy the helloworld addon.

the command make image needs to call https://github.com/openshift/imagebuilder to be installed & configured first.
this will be better:

$ go get<EMAIL_ADDRESS>$ export PATH=$PATH:$(go env GOPATH)/bin
$ make images

the exported environment variable EXAMPLE_IMAGE_NAME is not used.
kind load command can be modified:

kind load docker-image  $EXAMPLE_IMAGE_NAME --name <your-kind-cluster-name> # kind load docker-image  $EXAMPLE_IMAGE_NAME --name cluster1

thanks, would you like to send a PR to fix it :)

ok，i will do this asap

/assign ycyaoxdu


Add ability to set annotations on the Deployment

Overview
This PR add ability to set annotations on the deployment.
What this PR does / why we need it
Sometimes you need to add annotations on your Deployment.
For example I need it to automatically restart Dex when the TLS secret got updated.
Special notes for your reviewer
Checklist

[ ] Change log updated in Chart.yaml (see the contributing guide for details)
[ ] Chart version bumped in Chart.yaml (see the contributing guide for details)
[ ] Documentation regenerated by running make docs

Any updates on this? This looks like also implemented in #105 but I like this wording more...

I have rebased it.

If i can do something else tell me @sebastiangaiser  :)

At least I'm no developer of this project 😅

Thank you @Krast76 ! I'm going to accept #105

At least I'm no developer of this project 😅 But I would like to see this PR merged.

That's why you asked for updates on it 😅

Thank you @Krast76 ! I'm going to accept https://github.com/dexidp/helm-charts/pull/105

I close this one then.


Alignment error with -march=native, fixed with c++17

Hi
I am trying to get more performance by compiling with -march=native to get Eigen vectorization.
I am compiling with clang for now:
 CC=clang CXX=clang++ CXXFLAGS="-march=native" cmake .. -DCMAKE_PREFIX_PATH=/home/pal/install -DCMAKE_BUILD_TYPE=Debug -DCMAKE_INSTALL_PREFIX=/home/pal/install -DBUILD_PYTHON_INTERFACE=OFF
=> the test tsid-formulation fails

~/tsid/build > tests/tsid-formulation
Running 4 test cases...

*** test_invdyn_formulation_acc_force_remove_contact ***
q: 0 0 0 0 0 0 1 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0
tsid-formulation: /usr/include/eigen3/Eigen/src/Core/DenseStorage.h:128: Eigen::internal::plain_array<double, 36, 0, 32>::plain_array() [T = double, Size = 36, MatrixOrArrayOptions = 0, Alignment = 32]: Assertion `(internal::UIntPtr(array) & (31)) == 0 && "this assertion is explained here: " "http://eigen.tuxfamily.org/dox-devel/group__TopicUnalignedArrayAssert.html" " **** READ THIS WEB PAGE !!! ****"' failed.
unknown location(0): fatal error: in "tsid_formulationTest/test_invdyn_formulation_acc_force_remove_contact": signal: SIGABRT (application abort requested)
/home/pal/tsid/tests/tsid-formulation.cpp(128): last checkpoint

*** test_invdyn_formulation_acc_force ***
q: 0 0 0 0 0 0 1 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0
tsid-formulation: /usr/include/eigen3/Eigen/src/Core/DenseStorage.h:128: Eigen::internal::plain_array<double, 36, 0, 32>::plain_array() [T = double, Size = 36, MatrixOrArrayOptions = 0, Alignment = 32]: Assertion `(internal::UIntPtr(array) & (31)) == 0 && "this assertion is explained here: " "http://eigen.tuxfamily.org/dox-devel/group__TopicUnalignedArrayAssert.html" " **** READ THIS WEB PAGE !!! ****"' failed.
unknown location(0): fatal error: in "tsid_formulationTest/test_invdyn_formulation_acc_force": signal: SIGABRT (application abort requested)
/home/pal/tsid/tests/tsid-formulation.cpp(128): last checkpoint

*** test_contact_point_invdyn_formulation_acc_force ***
tsid-formulation: /usr/include/eigen3/Eigen/src/Core/DenseStorage.h:128: Eigen::internal::plain_array<double, 36, 0, 32>::plain_array() [T = double, Size = 36, MatrixOrArrayOptions = 0, Alignment = 32]: Assertion `(internal::UIntPtr(array) & (31)) == 0 && "this assertion is explained here: " "http://eigen.tuxfamily.org/dox-devel/group__TopicUnalignedArrayAssert.html" " **** READ THIS WEB PAGE !!! ****"' failed.
unknown location(0): fatal error: in "tsid_formulationTest/test_contact_point_invdyn_formulation_acc_force": signal: SIGABRT (application abort requested)
/home/pal/tsid/tests/tsid-formulation.cpp(517): last checkpoint

If we follow the link (http://eigen.tuxfamily.org/dox-devel/group__TopicUnalignedArrayAssert.html), one solution is to switch to c++-17, which seems to work. In CMakeList.txt:
CHECK_MINIMAL_CXX_STANDARD(17 ENFORCE)
Level 1
 - contact_rfoot_force_reg_task: w=1e-05, equality, 6x61
 - contact_lfoot_force_reg_task: w=1e-05, equality, 6x61
 - task-com: w=1, equality, 3x61
 - task-posture: w=0.01, equality, 31x61

Time 0
  contact_rfoot err: 0  contact_lfoot err: 0    task-com err: 0.1       task-posture err: 0     v=0      dv=0
dv(37x1): [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]

### TEST FINISHED ###
v(37x1): [ 0.0001,    0.03,  -6e-06,   1e-09,  0.0002,   0.004,  -0.004,   -0.05, -0.0006, -0.0001,  0.0005,    0.05,  -0.004,   -0.05,  0.0005, -0.0001, -0.0006,    0.05,   0.007, -0.0009,   0.003,   1e-05,  0.0009,   4e-06,  0.0001,  -1e-05,  0.0002,   1e-05,  -8e-06,  -3e-05,  0.0006,   0.003,   8e-06,  0.0009,   4e-06,  0.0001,   4e-06]
Final   CoM position:   0.0219545 0.000104108    0.665915
Desired CoM position: 0.0219541       0.1  0.665915
[...]

Otherwise, maybe there is a EIGEN_MAKE_ALIGNED_OPERATOR_NEW that is missing, but I did not find it (yet).
=> should TSID switch to c++-17?

From my point of view, the oldest compiler we currently support at LAAS is GCC 7.5 (default compiler of ubuntu 18.04) which is fully compliant with C++17.
Therefore, I wouldn't mind upgrading that CHECK_MINIMAL_CXX_STANDARD().
If anybody objects, we'll have to spend some time on that webpage. This is not an issue if anybody claim anything, but if nobody care I'd prefer to raise the standard.
Meanwhile, @jbmouret, while you're at it, could you make a quick check with -DCMAKE_CXX_STANDARD=17 and see if the unit test is happy ?

Yes, compiling with -DCMAKE_CXX_STANDARD=17 and clang works (for both my test and my code).
We are also using Ubuntu 18.04 (official distribution for Talos), although our Tiago is actually in 16.04 (but we will need to update it), which might cause some issues...
I will try to check with the g++ of ubuntu 18.04 later today.


PHP7 compatibility

Since PHP7, Null class name is not allowed anymore.
Cache\Null should be refactored with another class name.

Duplicate of #175

Right, I haven't noticed. Closing it.


I want to return TArray,

I want to return TArray, but that's the wrong way to write it. Can anyone tell me the correct way to write it? Thank you

There was a small bug with the array. The bug has been fixed, thanks for reporting.


There are several issues in the WebID swticher that need to be enhanced/fixed

There are several issues in the WebID swticher/dropdown that need to be enhanced/fixed:
When the user is logged in, the item Authorise to display your WebIds is a link which even logs an error when clicked (Uncaught TypeError: n is not a function). That item should be just a label and the text should be improved as it's not clear what it means.
When the connecton to the network is slow, this brings some confusion to the user. The WebIDs are not displayed immediately when opening the dropdown, the Authorise to display your WebIDs link is shown for a few secs, then for another secs No WebIds found and Updating WebIDs items are shown, then after another few secs the webids are displayed, and this may even fail suposely due to some connection problems. We need to enhance this to have some visual indication of what's happenign in the back, perhaps a "loading" message/icon
All these issues were taken from the discussion taking place in the forum starting in this post: https://safenetforum.org/t/safe-browser-v0-11-0-release/26792/56

Is this related to this issue reported on the forum?
https://safenetforum.org/t/webids-not-showing-in-browser/27499

I think so @calumcraig , if I understand correctly what's being reported there.
I also think I found a pattern for this issue, e.g I seem to be able to reproduce it always like this:

launch the browser for prod
log in with the authenticator
enable experimental apps
try to get the list of WebIDs form the dropdown

If I otherwise skip step 2 and log in the authenticator only after step 3, then step 4 seems to work fine, well...with its current UX limitations ofc, which we should enhance, but I get the list of WebIDs.

Hmm, I'm getting the issue in both situations - whether I log in before toggling experiments or toggling then logging in...
https://docs.google.com/document/d/1xraXXC0jT_Q2qENemcgknna1uk058WMFo7-GoceVjZ4/edit
This is prod browser I have built today.

ah, ok, I was trying that with v0.11.1

The forum guy who raised it said he was on that version too.

Yes, and just to clarify, this was reported when we released v0.11.0, as you can see in the description, so it's not a change that broke it but an earlier known issue. I belive that after the very next release where we tackle some higher priority bugs we can put this in the top of the list for next release, assuming we can have the UX/UI fully defined for it.

Closing this issue as at the moment, there are no API's for creating/fetching webIds in the pweb browser.


contracts: Charge rent for code storage

Fixes #6596  <- Check this out for an introduction to this topic.
@jacogr  This replaces the put_code dispatchable with instantiate_with_code and therefore requires changes to the off-chain environment.
This PR is separated into two commits which implemented the linked issue:
Implement refcounting for wasm code
Here we implement a PrefabWasmModule::refcounter for the CodeStorage storage item that holds the instrumented code for every contract including some metadata like the said counter. Every time a contract is instantiated using a specific code the counter is incremented and when the contract is wilfully terminated or evicted the counter is decremented. When it reaches zero the code is removed from storage.
As a consequence of this we can no longer allow to store code to the chain without deploying a contract at the same time (the refcounter for this code would be zero). This is why we also remove the put_code dispatchable and replace it with the instantiate_with_code dispatchable.
This commit is so big because almost every test needed to be adapted and because some substantial refactoring was necessary for decoupling the code instrumentation from the code storage step.
Also charge rent for code storage
This will be a small PR because all the refactoring work was done in the previous commit.  It merely changes the rent formula to also account for the storage that is occupied by the contracts code. These costs are shared between all contracts that use the same code.
The bulk of the commit is number tuning of the tests to account for the greatly increased rent incurred during testing.
Bonus commit: Fix dispatchables erroneously refunding base costs
When unifying weight and gas an error slipped through which caused dispatchables to always refund their base costs. This means only the gas costs where charged but not the overhead of the extrinsic itself. I noticed the error during test refactoring and fixed it.

/benchmark runtime pallet pallet_contracts

/benchmark runtime pallet pallet_contracts

/benchmark runtime pallet pallet_contracts

/benchmark runtime pallet pallet_contracts

/benchmark runtime pallet pallet_contracts

/benchmark runtime pallet pallet_contracts

apart from above comments, it looks good to me

all good to me now

bot merge


*.obj files only support less than ~65000 triangles.

Same solution for *.stl files needs to be added.

fixed in version 0.3


Pause on all exceptions button is not intuitive while seeing this guide.

Hi
Pause on all exceptions button is not intuitive while seeing this guide.
I was first looking for 'Pause on all exceptions' menu or option without realizing the button.
Guide should highlight or read it as button with optional text 'Pause on all exceptions'.
I referred other wiki to realize this.
https://umaar.com/dev-tips/32-pause-on-exception/
https://developers.google.com/web/updates/2015/05/automatically-pause-on-any-exception
Welcome! Please use this template for reporting documentation issues on https://developers.google.com/web/.
For questions about Chrome DevTools, checkout the
google-chrome-devtools tag on StackOverflow.
For questions about Chrome, check the google-chrome
tag. If you think you've found a Chrome bug, check out our
feedback page for more details on how to file it.
Page Affected: https://developers.google.com/web/TODO
What needs to be done?
Please provide description of what's needs to be done here. Please be sure to include the link to the original page
and any supporting links/data.

[ ] Appropriate type-something label has been added.
[ ] Appropriate section-something label has been added.

What guide are you referring to?

https://umaar.com/dev-tips/32-pause-on-exception/
I referred this above link right after google page. while referring to
google page it was not clear to me where i can find that 'pause on all
exceptions option' but it was button with said icon.
I believe this would be the same impression with any newbies to chrome dev
tools.
On Thu, Nov 15, 2018 at 6:08 AM Kayce Basques<EMAIL_ADDRESS>wrote:

What guide are you referring to?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/WebFundamentals/issues/6828#issuecomment-438871853,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEsfmD_S296kaqUBiJZYquiVuV1hG-zWks5uvLehgaJpZM4YbFCa
.

--
thanks
Sundar

Sorry, what I meant was, what Google page are you referring to? In other words, what doc do we need to update?
Do you mean this one? https://developers.google.com/web/updates/2015/05/automatically-pause-on-any-exception

I mean this link
https://developers.google.com/web/tools/chrome-devtools/javascript/add-breakpoints#exceptions
On Thu, Nov 15, 2018 at 11:28 PM Kayce Basques<EMAIL_ADDRESS>wrote:

Sorry, what I meant was, what Google page are you referring to? In other
words, what doc do we need to update?
Do you mean this one?
https://developers.google.com/web/updates/2015/05/automatically-pause-on-any-exception
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/WebFundamentals/issues/6828#issuecomment-439133387,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEsfmGAXeay6JeHEYAZBWscDJ6UgFaSoks5uvathgaJpZM4YbFCa
.

--
thanks
Sundar

Thanks. I'm adding this screenshot to the section:


how to convert sparql/cypher to s-expression or inverse ?

is there any tools like this https://github.com/Flitternie/GraphQ_Trans?

Hi,
You can find our implementation for converting S-expressions to SPARQL queries via https://github.com/dki-lab/ArcaneQA/blob/8c6df7f0dc47d78f6e2df02a5570454f39d2ced4/source/utils/logical_form_util.py#L382
Also, you may find our code for converting graph query to S-expression via
https://github.com/dki-lab/ArcaneQA/blob/06122c727af4f841187074550b6020bf29b29a09/source/utils/logical_form_util.py#L232
Our implementation is a bit ad-hoc and specific to GrailQA or WebQSP. It should be good enough if you just want to work with these popular benchmarks. The tool shared by you may represent a more systematic effort (I really appreciate the contribution by the authors of GraphQ_Trans). Unfortunately, we don't have a such dedicated and generic tool so far.
Feel free to let me know if you have more questions!

Thanks !


custom note thing

Describe your problem here.
basically i ported shape notes from vs dave 3.0 into lua, but there is one problem...
how do i make them able to hit during the time that player holds space (just to make it closely to the original)
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
no

ok thx

i tried to add some code, but it shows me the error if i load a song with specific notes in a chart
the error:

code itself:
function onCreate()
	--Iterate over all notes
	for i = 0, getProperty('unspawnNotes.length')-1 do
		--Check if the note is an Instakill Note
		if getPropertyFromGroup('unspawnNotes', i, 'noteType') == 'Shape Notes' then
			setPropertyFromGroup('unspawnNotes', i, 'texture', 'NOTE_assets_Shape'); --Change texture

			if getPropertyFromGroup('unspawnNotes', i, 'mustPress') then --Doesn't let Dad/Opponent notes get ignored
				setPropertyFromGroup('unspawnNotes', i, 'ignoreNote', false); --Miss has no penalties
			end
		end
	end
	--debugPrint('Script started!')
end

local blockHit = true;

if keyPressed("space") then
    setPropertyFromGroup("local", 0, "blockHit", false);
	else
	setPropertyFromGroup("local", 0, "blockHit", true);
end

if blockHit then
	setPropertyFromGroup('unspawnNotes', i, 'hitCausesMiss', true);
else
	setPropertyFromGroup('unspawnNotes', i, 'hitCausesMiss', false);
end

The value you called the keyPressed doesn't exist.
On line 18, say "if keyPressed("SPACE") then". Should fix your problem.
If you keep running into errors like these, take a look at the HaxeFlixel documentation for FlxKeyList here.

i tried and it still gave me an annoying error
code:
function onCreate()
	--Iterate over all notes
	for i = 0, getProperty('unspawnNotes.length')-1 do
		--Check if the note is an Instakill Note
		if getPropertyFromGroup('unspawnNotes', i, 'noteType') == 'Shape Notes' then
			setPropertyFromGroup('unspawnNotes', i, 'texture', 'NOTE_assets_Shape'); --Change texture

			if getPropertyFromGroup('unspawnNotes', i, 'mustPress') then --Doesn't let Dad/Opponent notes get ignored
				setPropertyFromGroup('unspawnNotes', i, 'ignoreNote', false); --Miss has no penalties
			end
		end
	end
	--debugPrint('Script started!')
end

local blockHit = true;

if keyPressed("SPACE:true") then
    blockHit = false;
end

if keyReleased("SPACE:true") then
    blockHit = true;
end

if blockHit then
	setPropertyFromGroup('unspawnNotes', i, 'hitCausesMiss', true);
else
	setPropertyFromGroup('unspawnNotes', i, 'hitCausesMiss', false);
end

i changed it a bit, looks incorrect bc this is my first time using flxinput stuff

Pretty sure it should be if keyPressed("SPACE") == true instead of if keyPressed("SPACE:true"). Same applies to keyReleased.


Separate server for task and model

My problem is that my server can not run docker. So I rent a server without gpu to run the task. My question is if there is a way to run model and task separately. For example, load the model in the server with a gpu and when testing, it send a request to the task server and the task sever return an output.

Hi, @Reason-Wang . Yes, actually that's how we conducted the evaluation. You may host your model as a server with fastchat and write a configuration file for it. Please see configs/agents/fs_agent.yaml for reference.

Great! Thanks for your answer.


[Task] Use LZ4 for Compression

Task Description
This should help with reducing the amount of data sent to Redis, and will help with thr performance generally (network wise)

https://docs.redis.com/latest/ri/memory-optimizations/
Will instead use a better serializer than orjson


Use gcc crate in native build script example

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @huonw (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Fixed the executable bit + squashed.

@bors: r+ 64049b2d7c48f144e83e83f3bedac3dd8c46f146

:hourglass: Testing commit 64049b2 with merge 4e5fdcd...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-64


Add a mvnd.rawStreams property

Alternative proposal to #712.
The only drawback is that you can't specify -Dmvnd.rawStreams=false but only -Dmvnd.rawStreams which sets to value to true.

Hi @gnodet I think this pr works fine, treating -Dmvnd.rawStreams as a zero parameter switch like --raw-streams for now, and i would close #712.
In the mean while, should we leave another issue to record some properties like mvnd.rawStreams or mvnd.serial can not accept false value.

Hi @gnodet I think this pr works fine, treating -Dmvnd.rawStreams as a zero parameter switch like --raw-streams for now, and i would close #712.

Good, I just need to fix the failing tests.

In the mean while, should we leave another issue to record some properties like mvnd.rawStreams or mvnd.serial can not accept false value.

Yes, please do.


Delete explore.ipynb

Coverage remained the same at 99.275% when pulling ffda01b20f6f01de40b15cf0bffb2c648383ee2e on duplicate-explore-notebook into 2b19db5559e292f3ce1f30e3fa8c64ab23c5fc84 on master.


add zlib to allowed licenses

The CI on #1298 is failing due to https://github.com/shotover/shotover-proxy/actions/runs/6180379036/job/16776751233?pr=1298#step:4:11
A dependency uses the zlib license which we have not allowed in our deny.toml.
Zlib is described as a permissive free software license and has been approved by the Free Software Foundation as a free software license and by the Open Source Initiative as an open source license. It is compatible with the GNU General Public License. refs
I also removed pktparse from our exceptions as we no longer use it as a dependency.

I cant find anything that clearly says the zlib license is compatible with apache license.
It could be fine, but I dont really know enough to say.

https://www.apache.org/legal/resolved.html#category-a

Category A: Licenses in Category A may be included in Apache Software Foundation products. They are said to be "Apache-like".

zlib is included in that list


Passing batches of input values to a lateral subquery

tl;dr
A proposed new batch operator:
batch 2 => (
  collect(this) |  // do something with 2-element array
)

Details
Repro is with super commit 0af34e0.
A community user recently asked about this in the same Slack thread that spawned this comment. In their original question they were attempting to operate on just two back-to-back input values, but then widened their inquiry to the more general case of needing to handle an indefinite number of back-to-back input vales of a similar nature.
Let's use the example of 6 incrementing numbers.
$ seq 6
1
2
3
4
5
6

Using what's in the language currently, much like the response to the user's original question, it seems this would require some initial aggregation of the input pairs into a complex value and then picking apart the complex values, potentially using over and a lateral subquery. For instance if we wanted to add each pairing of numbers together:
$ super -version
Version: v1.18.0-35-g0af34e0f

$ seq 6 | zq -z '
yield {i:(count()+1)/2,v:this}
| values:=collect(v) by i
| sort i
| over values => (
  sum(this)
)' -

3
7
11

However, @mccanne pointed out that for a use case like this a purpose-built operator like batch proposed above would provide a more direct and intuitive way to get the values into the lateral subquery.

For comparison, here's a SQL approach to get the same result. I confess I had ChatGPT come up with this, so maybe there's a better way. I also had difficulty getting DuckDB to take the seq input from stdin, so I put the input in its own file for now.
$ cat seq.csv
1
2
3
4
5
6

$ duckdb --version
v1.1.2 f680b7d08f

$ cat sum_pairs.sql
WITH NumberedRows AS (
    SELECT
        column0 AS value,
        ROW_NUMBER() OVER () AS rn
    FROM
        read_csv('seq.csv')
)
SELECT
    (n1.value + n2.value) AS sum_value
FROM
    NumberedRows n1
JOIN
    NumberedRows n2 ON n1.rn = n2.rn - 1
WHERE
    n1.rn % 2 = 1;

$ duckdb < sum_pairs.sql
┌───────────┐
│ sum_value │
│   int64   │
├───────────┤
│         3 │
│         7 │
│        11 │
└───────────┘


Hard crash on launch iOS at [BugsnagBreadcrumb objectValue] (BugsnagBreadcrumb.m:105:37)

Describe the bug
My app crashes immediately after launch on iOS, presumably when the first breadcrumb is added with
NSInvalidArgumentException: -[__NSArrayI objectForKeyedSubscript:]: unrecognized selector sent to instance 0x283eb3900

0  CoreFoundation          ___exceptionPreprocess
1  libobjc.A.dylib         _objc_exception_throw
2  CoreFoundation          -[NSObject(NSObject) doesNotRecognizeSelector:]
3  CoreFoundation          ____forwarding___
4  CoreFoundation          ___forwarding_prep_0___
5  MyApp                -[BugsnagBreadcrumb objectValue] (BugsnagBreadcrumb.m:105:37)
6  MyApp                -[BugsnagBreadcrumbs dataForBreadcrumb:] (BugsnagBreadcrumbs.m:217:21)
7  MyApp                -[BugsnagBreadcrumbs addBreadcrumbWithBlock:] (BugsnagBreadcrumbs.m:113:20)
8  MyApp                -[BugsnagClient addBreadcrumbWithBlock:] (BugsnagClient.m:861:5)
9  MyApp                -[BugsnagClient leaveBreadcrumbWithMessage:metadata:andType:] (BugsnagClient.m:594:5)
10 MyApp                +[Bugsnag leaveBreadcrumbWithMessage:metadata:andType:] (Bugsnag.m:183:9)
11 MyApp                -[BugsnagReactNative leaveBreadcrumb:] (BugsnagReactNative.m:82:9)
12 CoreFoundation          ___invoking___
13 CoreFoundation          -[NSInvocation invoke]
14 CoreFoundation          -[NSInvocation invokeWithTarget:]
15 MyApp                -[RCTModuleMethod invokeWithBridge:module:arguments:] (RCTModuleMethod.mm:584:3)
16 MyApp                facebook::react::invokeInner(RCTBridge*, RCTModuleData*, unsigned int, folly::dynamic const&, int, (anonymous namespace)::SchedulingContext) (RCTNativeModule.mm:183:17)
17 MyApp                facebook::react::RCTNativeModule::invoke(unsigned int, folly::dynamic&&, int)::$_0::operator()() const (RCTNativeModule.mm:104:7)

This may have happened after a minor version bump from bugsnag. It began when I created a new iOS build.
Steps to reproduce

Launch iOS app, immediate crash
See crash from bugsnag in bugsnag

Environment

Bugsnag version: "@bugsnag/react-native": "^7.14.1",
Browser framework version (if any):

Vue:
React:
Angular:

Server framework version (if any):

Koa:
Express:
Restify:

Browser version (e.g. chrome, safari):
Device (e.g. iphonex): iPhone 13 mini iOS 15.1.1

Error messages:
see above

Hi @AndrewMorsillo, thanks for the report.

My app crashes immediately after launch on iOS, presumably when the first breadcrumb is added

Are you making any calls to Bugsnag.leaveBreadcrumb or console.log prior to the app crashing? If so, can you share the kind of data are you attempting to add as breadcrumb metadata? (by default Bugsnag will automatically capture log messages as breadcrumbs)
Are these crashes being reported to Bugsnag? If so, please can you share a link to the error in your Bugsnag Dashboard so that we can take a look? You can drop a link here or write in to us at<EMAIL_ADDRESS>

Closing this out as we've not heard anything further, but do let us know if you've got some more details behind the crash.


[FFI] Add new containers and Implementations

Description
This is the follow up PR for RFC #19672. Map container is added and more data types are supported by new FFI, like dictionary, list of strings.

[x] Make ADT container and MAP container support NDArray type.
[x] Adopt PackedFunc based FFI on CachedOp.

Some CachedOp functions are implemented: create, free, invoke, get_optimized_symbol

Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[ ] Changes are complete (i.e.  I have pasted the average improvements as follows. (The latency is the average number with different batch_size, sequence_length as input)

model
training latency withou this PR (s)
training latency with this PR (s)
Improvement (s)

google_en_uncased_bert_base
0.09161326
0.09133351
0.00027974

google_en_uncased_bert_base
0.3565172
0.35624171
0.000275489

google_en_uncased_bert_large
0.91762223
0.9173615
0.000260731

google_albert_base_v2
0.38036531
0.38022336
0.00014195

google_albert_large_v2
0.74285129
0.74271887
0.000132424

google_albert_xlarge_v2
1.53808278
1.53795535
0.000127428

google_albert_xxlarge_v2
2.49918614
2.49904376
0.000142379

google_electra_small
0.07791454
0.07770361
0.000210933

google_electra_base
0.35639018
0.35617552
0.000214658

google_electra_large
0.91575478
0.9154471
0.000307674

google_uncased_mobilebert
0.1725719
0.17218696
0.000384942

fairseq_bart_base
0.43927581
0.43899117
0.00028464

fairseq_bart_large
0.70489126
0.70455636
0.0003349

Also, I have compared the Training and inferencing time with the real workloads:
Running google_electra_small model on SQuAD dataset and will get the following results.

Training/Inferencing
Latency without this PR
Latency with this PR
Throughput without thie PR (samples/s)
Throughput with this PR (samples/s)

Training
1.59179 h
1.48754 h
70
75

Inferencing
55.566 s
55.41125 s
216.35
216.96

Environment

python_version
3.6.9

instance
g4dn.2x

system
Linux

cpu
x86_64

architecture
64bit

fp16
FALSE

cpu_ram_mb
63622

use_gpu
TRUE

num_gpus
1

gpu
Tesla T4

gpu_ram_mb
15079

gpu_power_watts
70

gpu_performance_state
0

@mxnet-bot run ci [windows-cpu, unix-gpu]

I have replaced the backend APIs(MXInvokeCachedOp, MXNET_REGISTER_GLOBAL("cached_op.invoke")) with simple or dummy implementation so that we can fully expose the overhead of the API call with/without this PR by removing the computational costs. The results is shown as follows:
For CachedOp invocation call without this PR, it takes around 7.22 us and the most of the overhead is in making cython/python args; For CachedOp invocation call with this PR, it takes around 4.041 us and the most of the overhead is in type translation/checking in packedfunc system.


[idea] Community reviews

Currently, the analyzer looks to help with determining the "relevance" of the module on the basis of Github activity. One step ahead would be to add a community driven review system for the modules.
Essentially, "amazon-like" star based reviews and comments. The main aim is to incorporate this into search results so that good modules show up higher than crappier ones. What do you guys think about this?

@MrDHat Ty for the suggestion but this is out of the scope of npms, at least for now.


state route rule: Fix verify_route_rule()

When comparing two state with route rules to 2+ route tables,
the verify_route_rule() might fail due to the unsorted keys of
indexed route rules.
Unit test case added.

Coverage decreased (-4.09%) to 61.684% when pulling 02853f76ffb3fe8fa8012110303eaf826c481d66 on cathay4t:fix_route_rule into 48ed8dcc465e920c66b102d6025389084ece76b7 on nmstate:master.

Coverage decreased (-4.09%) to 61.684% when pulling 02853f76ffb3fe8fa8012110303eaf826c481d66 on cathay4t:fix_route_rule into 48ed8dcc465e920c66b102d6025389084ece76b7 on nmstate:master.


Allow update/delete of tenanted document from session with no TenantId

Use case:
The need to delete a tenanted document from within an "un-tenanted" session, similar to how the call to .Store(tenantid, object) works.
Suggestions

Allow the tenant id to be changed/set from with an already created IDocumentSession
Potential methods as follows:

_session.Update<T>(string tenantId, T object)
_session.Delete<T>(string tenantId, T object)

Sounds reasonable.
Shall we also add the following to IDocumentSession? It does make the API rather noisy
void Delete<T>(string tenantId, T entity);
void Delete<T>(string tenandId, int id);
void Delete<T>(string tenantId, long id);
void Delete<T>(string tenantId, Guid id);
void Delete<T>(string tenandId, string id);
void Update<T>(string tenantId, params T[] entities);

@oskardudycz @jeremydmiller Currently this is a pain point for us, the work around is to create a new session just to do the delete. Do you think this is something that could be addressed as part of 4.0? I am happy to help with a PR, but recall this not being as straight forward as it seems.
I have seen @Miggleness comment on adding various methods and i agree it does add a lot of noise to the API. Would my suggestion be feasible? Any other ideas or suggestions?

Finally updating this a bit. In the V4 branch so far, you can do:

IDocumentSession.DeleteByIdInTenant<T>(all the id types)
IDocumentSession.Store(tenant Id, documents)

We don't yet have anything for doing that with Insert() or Update() or Delete(document) operations. At this point after the V4 internals work, that's actually pretty low hanging fruit. Would you have any interest in doing the PR to finish this off?
If you start by looking at those two methods above, I think the code to finish that off will be self-explanatory and it should mostly be test coverage on the new methods.

Sure, will have a stab at it 👍

If you go after that, it might be beneficial to make the method names consistent. I'm good with the InTenant suffix on everything rather than having yet more overloads.

@tonykaralis This ended up a little differently than we'd talked about in this issue. More in line w/ #1460, you can do all possible operations against a separate tenant within a single session by using syntax like session.ForTenant("tenant id name").Store(doc). You effectively get a "nested" tenant session that still writes to the same transaction. Made the internals a lot simpler.


Split raiwidgets/e2e-notebooks test pipeline in two

Description
The existing pipeline .github\workflows\Ci-raiwigets-python-typescript.yml serves two distinct purposes:

running raiwidgets tests (pytest)
running e2e notebook tests
They are in one pipeline since they both require yarn install, yarn buildall, and the subsequent installation of raiwidgets. However, this pipeline has become the most time-consuming of all of our pipelines and it will only get worse in the future as we add more notebooks and more test cases. For that reason, this PR splits them into two and prunes unnecessary steps (e.g., installing torch for the pytest tests). The e2e-notebook pipeline isn't run on macos-latest which also removed a lot of custom parts. So far, this was just handled via if statements.

Beyond that, the e2e-notebook pipeline is also broken down into each individual notebook using the matrix strategy. This required extending e2e-widget.js to take additional input params -n for the notebook name and --skipgen to skip notebook generation which considerably speeds up the local test iterations. In this new setup, the notebook names need to match the test folders so I renamed some of the test folders to match the notebook names.
Potential future work: Do yarn install and yarn buildall in a single workflow, store the output as an artifact and install raiwidgets using this artifact in the individual workflows. This would mean we're running the yarn buildall part only once instead of n times. I'll reevaluate whether this is needed depending on the execution time.
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #1478 (2618e3e) into main (e79a0a9) will decrease coverage by 11.57%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #1478       +/-   ##
===========================================
- Coverage   87.15%   75.58%   -11.58%
===========================================
  Files         104       10       -94
  Lines        4897      258     -4639
===========================================
- Hits         4268      195     -4073
+ Misses        629       63      -566

Flag
Coverage Δ

unittests
75.58% <ø> (-11.58%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

responsibleai/responsibleai/_config/__init__.py

responsibleai/responsibleai/exceptions.py

erroranalysis/erroranalysis/_internal/metrics.py

erroranalysis/erroranalysis/version.py

raiwidgets/raiwidgets/model_analysis_dashboard.py

raiwidgets/raiwidgets/__init__.py

...idgets/raiwidgets/responsibleai_dashboard_input.py

...i/responsibleai/managers/error_analysis_manager.py

raiutils/raiutils/models/model_utils.py

...bleai/responsibleai/_tools/causal/causal_config.py

... and 84 more

Continue to review full report at Codecov.

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


Graphical bug in integrated terminal

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.56.2 (user setup)
OS Version: Windows_NT x64 10.0.19042

Steps to Reproduce:

Use the integrated terminal

This is the issue i'm getting. A visual bug where lines appear in the middle of text.

Did you change window.zoomLevel value?

Yes, i put zoom level 0 and now it's fine. Thanks, i didn't think of that

Probably duplicate of #106981
Upvote and track that issue to be notified when it's fixed.
Also there were couple of suggestions.


The display is incomplete，when direct_mode = 1

LVGL version
v8.3.11
What happened?
Conditions:

Using the lv_port_win_codeblocks project, the lvgl version is V8.3.11.
32-bit Color depth
set full_refresh=0  for partial refresh
set direct_mode=1
set background transparency
use the following code in the main function  to repeat the problem

lv_obj_t *scr = lv_scr_act();
lv_obj_set_style_bg_opa(scr, LV_OPA_TRANSP, 0); //Full transparent background
lv_example_anim_2();
lv_example_png_1();

Background transparency needs to be set, otherwise the problem cannot be reproduced！
Normal

Abnormal

How to reproduce?
int APIENTRY WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR szCmdLine, int nCmdShow)
{
    /*Initialize LittlevGL*/
    lv_init();

    /*Initialize the HAL for LittlevGL*/
    lv_win32_init(hInstance, SW_SHOWNORMAL, 854, 480, NULL);

    /*Output prompt information to the console, you can also use printf() to print directly*/
    LV_LOG_USER("LVGL initialization completed!");

/** add my test **/
    lv_obj_t *scr = lv_scr_act();
    lv_obj_set_style_bg_opa(scr, LV_OPA_TRANSP, 0); //Full transparent background
    lv_example_anim_2();
    lv_example_png_1();
/*****************/

    while(!lv_win32_quit_signal) {
        /* Periodically call the lv_task handler.
         * It could be done in a timer interrupt or an OS task too.*/
        lv_task_handler();
        usleep(10000);       /*Just to let the system breath*/
    }
    return 0;
}

LVGL v9 is not what I want, I am using v8.3.11. And this problem doesn't just occur under the codeblocks project, it also occurs on my embedded platform.
Can you help me fix this on v8.3.11?

LVGL v9 is not what I want, I am using v8.3.11. And this problem doesn't just occur under the codeblocks project, it also occurs on my embedded platform.
Can you help me fix this on v8.3.11?

Hey,
I don't know immediately what the problem could be, but I would rather update the CodeBlocks repo to LVGL v9. Is it an option for you?
FYI, I transferred the issue to the Codeblocks repo.

I see and I could reproduce the issue in Eclipse with SDL too. I can fix only later next week, but I can show you where the issue is.
Here LVGL clears the whole buffer, but in direct mode only the area where the rendering happens needs to be cleared.
So I would try this:

Add an if(!direct_mode) condition here.
In direct mode clear the area to be rendered here. It should result in a "hole" in the middle of the screen sized buffer.

Thank you for your help. I tried your method, but it still didn't work out。Can you take care of this for me？


[bug]: LCM LoRA not probed successfully.

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
24
What version did you experience this issue on?
LCM Branch
What happened?
On importing latent-consistency/lcm-lora-sdxl received "Import Failed" error.
Screenshots
No response
Additional context
No response
Contact Details
No response

This may just be because it's a Repo. The file itself on download imports fine

Fixed in PR#5068


I need help with the project

Thanks for this great project, you need help since I don't know much about programming, I am doing a project to help the community of a game to update it and I would like very much if someone can help me with this project, I am trying to change the progressbar for an image so that it fits well with the visual theme I'm doing, can someone help me please

wtf r u talking about???


Update naming of console type to be consistent with vscode

For #4321

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[x] Title summarizes what is changing
[x] Has a news entry file (remember to thank yourself!)
[x] ~Has sufficient logging.~
[x] ~Has telemetry for enhancements.~
[x] ~Unit tests & system/integration tests are added/updated~
[x] ~Test plan is updated as appropriate~
[x] ~package-lock.json has been regenerated by running npm install (if dependencies have changed)~
[x] ~The wiki is updated with any design decisions/details.~

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

Thanks for the change @thomasgassmann ! Although this will currently break the launch.json of users containing:
"console": "none"

We need to notify the users that "console": "none" is to be replaced with "console": "internalConsole". We already do this for notifying similar things in file invalidDebuggerType.ts. Unless you want to really do it yourself, I can do that for you in a separate PR.

I have updated your branch so you can see the updated invalidDebuggerType.ts.

@karrtikr Ok, thanks! I will take this over.

@karrtikr Ok, thanks! I will take this over.

FYI Tests will also needed to be added for that. It may take time so it will be great if you let me do this entire bit, as I am very much familiar. But if you really want to, then go ahead :)

Ok, sure, if you're already familiar with it, then you can take this over as well :)

Ok, sure, if you're already familiar with it, then you can take this over as well :)

Alrighty, you can have a look at the changes. You can go ahead and merge it now.

Ok, sure, if you're already familiar with it, then you can take this over as well :)

Alrighty, have a look at the changes. You can go ahead and merge this now.

Ok, sure, if you're already familiar with it, then you can take this over as well :)

Alrighty, please have a look at the changes. You can go ahead and merge this now.


fix(show-value-menu): hid the show value menu on home page

:tada: This PR is included in version 1.185.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Can I create a service principal with certificate and password based authentication at the same time?

I want to automate service principal provisioning - currently I'm using --create-cert --cert option, but then in output I see "Null" next to password parameter. My organisation is using both authentication methods so I would like to be able to have password as well as certificate stored in Key Vault. Is it possible to do it now in one step?

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

ID: a3f84bdc-9ed0-eb85-f3af-9711e8083a95
Version Independent ID: 03a47d06-5fbf-4764-0882-99cd8bd41110
Content: Use Azure service principals with Azure CLI
Content Source: docs-ref-conceptual/create-an-azure-service-principal-azure-cli.md
Technology: azure-cli
GitHub Login: @sptramer
Microsoft Alias: sttramer

@krysickij1 Thank you for reaching out to us, we will further look into this and get back to you at the earliest.

@krysickij1 Apologies for the delayed response on this. Currently you can only store the certificate in the key vault, the password automatically gets stored in to Azure Active Directory. To answer your specific query "No" we cannot store both password and certificate to Azure Key vault.
As a work around you can write a script to get the password when using pass-based authentication mechanism and read the output and password parameter out of it and create a key/secret with in Azure Key vault.
However you can always share your feedback using az feedback , which would give our product team an opportunity to further evaluate your ask.
Currently we don't see need for any further edits or updates to existing documentation. Will go ahead and close this issue for now however if you have any further queries please feel free to tag us in your reply will be more than happy to continue the conversation. Thank you


bug: slow data load

https://user-images.githubusercontent.com/38668796/146326938-4e500867-a224-4fa6-adce-b7fc878d5e14.mp4

@productdevbook
Thank you for reporting, I fixed this bug.
Please update si-grenoble to v0.6.0.
demo: https://si-newtech.github.io/si-grenoble/

good and no good :/
because very network connection in page.
maybe used lazy loading etc.
a better solution is needed.

example -> https://icones.js.org/collection/twemoji
maybe this will help you.

and -> https://icones.js.org/collections/twemoji-meta.json your all.json change. and very becatiful search

@koyasaeki you have to keep improving :)


Expression type refactor

We current have 3 function related expression types:
SCALAR_FUNCTION
AGGREGATE_FUNCTION
MACRO

We should consider have a single function expression type since those three types are a type of FUNCTION expression.

So ExpressionType::MACRO is no longer in use.
I do need a way to differentiate FUNCTION & AGGREGATE_FUNCTION. So it either happens at ExpressionType level or at FunctionType level. I don't perceive a major difference so I'm closing this issue.


not support the argument that using action='store_true'

This is supported but seems not well documented.
Simply specify the argument in the bracket, [bool_param] will be converted into --bool_param if bool_param=True, else it will be eliminated.
Also see the example param.yaml:
template:
  dtype: >
    echo command {param1} [param2] "{'param3': {param3}}" [param4]
...
---
param4: [ True ] # will be converted into flag "--param4"

I will add detailed use cases in README.md.

Thanks!


Span IndexOf overload that takes multiple byte values has a bug

I was writing some tests to validate correctness for IndexOf overloads and noticed this bug.
https://github.com/dotnet/corefxlab/blob/master/src/System.Buffers.Primitives/System/SpanExtensions.cs#L156
     [Fact]
        public static void MakeSureNoChecksGoOutOfRangeTwo_Byte()
        {
            byte[] a = new byte[42];
            a[0] = 99;
            a[1] = 99;
            a[40] = 99;
            a[41] = 99;
            Span<byte> span = new Span<byte>(a, 1, 40);
            int index = span.IndexOf(99, 98);
            Assert.Equal(-1, index);
        }

Message: Assert.Equal() Failure
Expected: -1
Actual:   0
The sequence of bytes 99, 98 do not exists anywhere in the span and hence the index should be -1.
cc @benaadams, @davidfowl, @KrzysztofCwalina

Shouldn't this be a BitwiseAnd?
https://github.com/dotnet/corefxlab/blob/master/src/System.Buffers.Primitives/System/SpanExtensions.cs#L188
Also, we are using the same vData to compare to both values0 and values1?
https://github.com/dotnet/corefxlab/blob/master/src/System.Buffers.Primitives/System/SpanExtensions.cs#L187
cc @shiftylogic, thoughts?

Shouldn't this be a BitwiseAnd?

Can't be two bytes at the same time

Also, we are using the same vData to compare to both values0 and values1?

Want to find one or the other; then return the first

Test is wrong, you are adding 2 bytes then starting at second one; should be?
Span<byte> span = new Span<byte>(a, 2, 38);

Test is wrong, you are adding 2 bytes then starting at second one; should be?

I think I misunderstood what these IndexOf overloads do:
byte[] a = new byte[10];
a[0] = 99;
a[9] = 98;
Span<byte> span = new Span<byte>(a);
int index = span.IndexOf(99, 98);

a = "99, 0, 0, 0, 0, 0, 0, 0, 0, 98"
What should index be? Should it be 0? Shouldn't it be -1 since "99, 98" is not found anywhere?
I thought IndexOf(99, 98) looks for where 99 and 98 appear together.
Is it IndexOf either 99 or 98 whichever is first?

Maybe it should be called IndexOfAny 😆
Will make PR


calling modifications on Pacbio Revio platform

Dear authors,
thanks for your great tool 'ccsmeth'. I try calling modifications on a hifi.bam generated from Pacbio Revio platform. However, I only get a empty bam files.
I have noticed the given models are trained using PacBio Sequel II reads, I wonder whether differences between the two platforms leads to my results.
I look forward to your reply!

Hi @eric-hy , sorry for the delay. To call methylation using ccsmeth, the input data (hifi.bam) must contain the kinetics signals (i.e., containing TAGs such as fi, fp,ri, rp in each item of the bam file). Also, please add --no_decode in the call_mods command. For more information, pleash check issue #40 .
Best,
Peng


only want dynamic Imports in production environment

like:https://github.com/webpack/webpack/issues/4636
l have a big project. It‘s has about 200 pages, , l used the dynamic Imports every page.
but the hmr spend too long time

when l not use dynamic Imports, It reduced the advanced module optimization time to milliseconds.

So l want only in production use lazy-loading by dynamic Imports.
But if l write a function like
function _import(file) {
 return<EMAIL_ADDRESS>}

according to

Every module that could potentially be requested on an import() call is included. For example, import(./locale/${language}.json) will cause every .json file in the ./locale directory to be bundled into the new chunk. At run time, when the variable language has been computed, any file like english.json or german.json will be available for consumption.

So it will  require every .vue in @/views and build this .vue although l not use it.
//@/views/login
import socialsignin from './socialsignin'

//@/views/socialsignin

it will build like this.
static/js/22.41970889bdf8bc55e772.js (2.84 KB) is build,but will never be used

It looks like you just deleted/ignored our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please open a new issue with filled issue template. Also make sure your issue is not a question. Questions should be posted on Stack Overflow.


Add {d,p,q,r}logis recycing rule implementation

Includes tests for each version in test-coreR.R.

@paciorek What do you think about this.  @jpdunc23 noticed that dlogis etc. lacked compiled recycling rule behavior.  Do you want to add this for 0.8.0 or hold off since there is a lot going in?

@perrydv I'm fine with adding it.


Image bug

I get an error when the script is started.  That is something I will fix in version 2.0 #576
Regarding your specific issue, I couldn't reproduce it. see this fiddle, it's working fine
Also, it works fine in the demo

This issue #585 reports the same issue as you. It should be fixed now in master.

Donate
As you know, ShowdownJS is a free library and it will remain free forever. However, maintaining and improving the library costs time and money.
If you like our work and find our library useful, please donate through Pledgie or directly through paypal!! Your contribution will be greatly appreciated and help us continue to develop this awesome library.


[ng-core] [WrappedFormControlSuperclass] emitOutgoingValue should not be called implicitly

I understand that the utility class attempts to reduce the boilerplate as much as possible and in doing so, it assumes that the sub class  will want to call emitOutgoingValue any time when this.formControl.valueChanges emits a new value. However, this just may not be true for certain components.
Imagine creating a autocomplete component, which only allows values from the provided options. It should only emit a new value if the value typed into the control matches one of the options.
I think lines 56-58 need to be removed.
Unfortunately, that would be a breaking change.

@DmitryEfimenko If you happen to be available right now, I'm looking at this on my twitch stream. If you get this message very soon, please drop in and say hi so we can discuss! https://www.twitch.tv/ersimont

Too slow. It was right at the end of my stream. :)
If you don't mind, would you check out the video? Then you can respond to my thoughts here, or on Discord. You'll want to skip to around the 1:49:00 mark: https://www.twitch.tv/videos/978077129

I did. It was great to watch your though process. It's a shame I missed the twitch invite :)
To answer your question "What is the control value when the user typed in an invalid value - something that does not match available options"?
The autocomplete component allows to configure this. This is configurable. All three scenarios are possible:

allow any typed in value to be emitted
emit null if the input does not match any options
do not emit new value (keep the previous valid value) if user input does not match any of the options

As you mentioned, the first and second scenarios are perfectly supported by the WrappedFormControlSuperclass. However, the third option - the one that is most commonly used in our application does not work.
I can also see your point when you were speculating whether the WrappedFormControlSuperclass is suitable at all for my use-case. But then you were saying that I might find useful the other 95% of the class - which is exactly the case! It would indeed be a shame if I could not use it.
I wonder if the functionality in question could be made optional and enabled by default. This way everybody would be happy and it would not be a breaking change.
On the flip side I love how small and clean the class is and I'd hate to bloat it. Although maybe it would not be too much.
If we were to add such customization, I think it would have to be in a form of a new method that would enable/disable this functionality.
Also, adding distinctUntilChanged() to the valueChanges totally makes sense to me. I don't think I've heard anything where Angular team decided to emit duplicates on purpose. I'd love to learn more about it.
Finally, I did join the Discord. Perhaps we could chat some time tomorrow.

What about overriding innerToOuter() to keep returning the latest valid value for as long as the current one is invalid? Would that do anything for you?

definitely sounds like a possibility. But now we're getting into this situation of emitting duplicate values. I think the following change would make my use-case work:
const valuesToEmit$ = this.formControl.valueChanges.pipe(
  map(value => this.innerToOuter(value)),
  distinctUntilChanged()
);

this.subscribeTo(valuesToEmit$, (value) => {
  this.emitOutgoingValue(value);
});


azure-cosmos Release 2024-02-20 API review fixes

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-cosmos