Suggestion: add a "touch all" function to Form controls

I have a situation where I reuse a form within a Bootstrap's modal declared once in a page. This page has a list of items and the user can select one to edit its data in the form or click at a button to pass empty data to form to save a new item.

Each time I open modal I call $form.reset() to clean validations and read fresh data, either from selected item or new blank item. PS.: without leaving page and reusing declared form at modal.
However, if I launch a modal to fill new item data and ignore fields filling (not "touching" them and hitting submit button), validation fails and let form submit proceed as valid. I needed to add the following code before submit to proper handle validation errors:
Object
   .keys($form)
   .forEach(field => $form[field].touched = true);

Am I doing something wrong? Or the form submit is working as expected and missing a "touch all fields" function?
BTW, congrats for the amazing component.

Regarding touchAll()
I like the idea :)
I'll let you know when it's implemented. (probably today or tomorrow)
About your specific use case. Could you create a svelte REPL for it? If not I'll try to recreate it myself

Thanks for the detailed explanation!
Actually there already is a touch all functionality. $form.touched = true will set every form control to touched.
The main problem that you're facing isn't about the controls not being touched or validated. It's about how you set the new values of the input.
At the moment you're binding the person.nome property to the nome input and when the value changes, it updates the input correctly but not the internal logic of svelte-use-form. This is due to the fact, that setting the value of an input doesn't start any events that notify svelte-use-form of the change.
Clarifications

You shouldn't change input values directly by setting the value attribute at runtime. Although you can use the value attribute to set the initial value statically.
reset() sets the value of the input to the value it had when it was rendered (initial value).

Solutions for the internal state not being updated
1. Key Solution
Wrap the form with {#key person}...{/key} and set nomes value to the desired new value. This will recreate the form every time person is changed and therefore display the new initial value.
2. Updating it manually
After updating the value of the input by binding or setting it programatically, update the svelte-use-form state of the control by running e.g. $form.nome.value = person.nome
This only works if you aren't running $form.reset() before hand.
export const open = () => {
    showDialog = true;
    $form.touched = false;
    $form.nome.value = person.nome;
    console.log("person:", person);
  };

3. Set the value by resetting
Remove the binding on the input and change the value of the input by using the reset method on the control: $name.nome.reset({ value: person.name })
This will update the input value and also its internal state.
4. Change the value (coming next update)
Remove the binding on the input and set the value by running the change method $form.nome.change(person.nome)
This will do the same thing as $form.nome.reset({ value: person.nome }), except that it won't set the input to untouched if it was touched.
Problem with your specific example
You're updating the values of the form like this:
const editPerson = async (person) => {
    selectedPerson = { ...person };
    dialog.open();
  };

The problem is that even though you're changing the selectedPerson before opening it, the component "SupposedDialog.svelte" will still have the same reference of person since svelte only updates it after the current tick has ended. Solution is to await tick() Docs
Solution
const editPerson = async (person) => {
    selectedPerson = { ...person };
    await tick();
    dialog.open();
  };

Thanks @noahsalvi and sorry for the late response.
The key solution didn't solved the problem, because each time user type some info on the inputs, Svelte refreshes components and looses focus, transforming it in a poor user experience.
The await tick() solution solved some rendering problems.
In the end, updating form values manually (individual input resetting) was the definitive solution.


Fix broken arm multiplication timings

Why does it pass running out of RAM but not ROM? 2 cycles wrong? Implies that I need to switch back to non-SEQ access somewhere I think

The technical reference documentation says that the first cycle of this instruction should be a bus read with non-seq. That's not really possible in my implementation so I make the next read non-sequential . That's another opcode read so I can't come up with a time it would matter


Update use-sp-pnp-js-with-spfx-web-parts.md

Category

[x] Content fix
[ ] New article

Related issues

fixes #9002

What's in this Pull Request?
Replicating an update to the getSP method made in the pnpjs docs below. The current implementation causes an 'Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'web')' error when navigating to the SharePoint page from another page.
https://github.com/pnp/pnpjs/blob/version-3/docs/concepts/project-preset.md
Additional context is available in the below issue
https://github.com/pnp/pnpjs/issues/2371

@shajumohamed please accept the CLA before we can merge your submission... thanks!

@microsoft-github-policy-service agree


Extension source issue

Source information
Kimoitv V14.1
Source language
English
Steps to reproduce

Open Kimoitv source
2.Cloudfare Pass through by adding chrome user agent in settings
Open a certain movie title
Unable to fetch video when clicked on play button

Expected behavior
Video should be fetched directly from website and start playing
Actual behavior

Aniyomi version
Animiru fork
Android version
Android 11
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Kimoitv is not a source we manage

Kimoitv is not a source we manage

Then why is there a source in the app if you don't
FYI I use Animiru app , a fork of Aniyomi which has anime section only and since all the extension got took down, I used extensions repo source code to download source. If you don't manage it, tell me who does 😠

If you don't manage it, tell me who does 😠

No, we don't associate with or support any third party repo/extension.

Ok. But pls look forward for implementing it for future updates since that third party repo /extension has almost every media source you could possibly think of.
It has Anime,Tv shows, Hollywood movies, Korean & Chinese shows, animated Movies and manga too
Just that, it has issues while passing cloudfare and downloading it

The app already has support for adding third party extensions

Yes I have issues for that particular extension

Pls resolve it or look upon yourself

Pls resolve it or look upon yourself

I already told you, we don't support this extension anymore and we won't give any support to third party extensions either. If you have any problems with third party extension, you should contact whoever is in charge of the extension repo, not us.


[Bug] Responsive sizes do not return different sizes

Bug Report
Describe the bug
Currently the sizes property has no effect on the image URLs, repeating only the same original URL for each size

I'm still getting this behavior in 4.22.1 with Next 13.5.4


[Feature Request] Add support for separate input/output schemas

New Feature - add an ability to maintain separate schemas for request and response models, similar to FastAPI's implementation.
Current Behavior
Currently, parse_parameters overwrites component schemas (APIView.components_schemas) that were previously extracted by get_responses.
Usually get_responses returns different schemas due tomode="serialization" and we're loosing any additional information (such as @computed_fields)
Proposed Solution
I could add a new configuration option separate_input_output_schemas:

Defaults to false to maintain backward compatibility
When enabled, maintains distinct schemas for:

Request bodies and parameters -Input suffix
Response models -Output suffix

@luolingchun What are your thoughts on this? If you feel this feature would be valuable, I can begin implementation next week. Please share any context or feedback that would help inform the work.

Good idea, thank you in advance for the work you did on this.

@luolingchun I've raised draft PR. Once the implementation approach is confirmed, I plan to:

Add more unit tests
Include usage examples
Update the documentation

Please let me know if the current implementation direction works, and I'll proceed with these additional improvements.


Compatibility with X

Since Twitter's name change to X, the extension doesn't work anymore and I have to manually put the links on the Upload page again.

Fixed in #11 (v3.3.4).


Add Dashcam Viewer v2.1.6

Adding a new cask

[x] Checked there aren’t open pull requests for the same cask.
[x] Checked there aren’t closed issues where that cask was already refused.
[x] When naming the cask, followed the token reference.
[x] Commit message includes cask’s name.
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.

Everything looks good. Thank you for the contribution!


enter-claim bypass using bikes from pixelmon

If you have enter-claim blocked on any it is possible to skip the event entirely using any of the two bikes from pixelmon.
Basically for a while if you ride a pokemon into a claim that denies entry it can glitch tp to the other side of the claim.
Bikes on the other hand occasionally glitch straight into the claim, this can be easily reproduced by riding into the side of a claim and tapping forward. Lets you straight in.
Sponge Version: 7.2.3
GD Version: 1.4.4
LP Version: 5.1.75

Just a small update. This also works with boats and I'm assuming other similar land mounts

Thanks! I wrote a workaround in case this happens to bypass. GD will check current claim to validate you are able to enter and if not cancel it. The fix will be included with next release 1.4.8

Actually, looking at this some more, the proper fix would be to have sponge fire events for vehicles with no player. However, the pixelmon bike issue is odd.  It seems it glitches through when you stop then try again. I'll have to play with this some more. My original workaround is fine but it would add more performance strain which wouldn't be good so I'll work on a better solution =)

Did this occur on older SF versions? This seems like a SF bug as cancelling the event outright still produces the same issue.

I've now tried going back as far as Sponge 7.1.6 and it was still happening

Fixed in Sponge.
https://github.com/SpongePowered/Sponge/commit/e9dde5ab4076eacad13f61c3028259774c5143d6

Fixed in Sponge.
https://github.com/SpongePowered/Sponge/commit/e9dde5ab4076eacad13f61c3028259774c5143d6


Add test cases to Sum of Multiples

Adding test cases to Sum of Multiples as per exercism/exercism.io#2341.

Cheers!


Issue with “Agent stopped due to iteration limit or time limit” in CrewAI with AgentOps Integration

Description:
I am encountering an issue with the CrewAI integration when using AgentOps. Specifically, when integrating AgentOps into my CrewAI project, the agents consistently stop with the message "Agent stopped due to iteration limit or time limit," even though the tasks should complete without hitting these limits.
This issue does not occur when I run the same code without AgentOps integration. The tasks execute successfully, and the agents complete their interactions as expected.
Steps to Reproduce:

Set up a simple CrewAI script with AgentOps integration.
Initialize a session using agentops.init.
Define simple agents and tasks using CrewAI.
Run the script.
Observe the issue where agents stop due to an iteration or time limit.

Sample Code:
import os
import json
import agentops
from crewai import Agent, Task, Crew, Process
from langchain_openai import ChatOpenAI
from dotenv import load_dotenv

# Load environment variables
load_dotenv()
openai_api_key = os.getenv('OPENAI_API_KEY')
agentops_api_key = os.getenv('AGENTOPS_API_KEY')

if not openai_api_key:
    raise EnvironmentError("OPENAI_API_KEY not found in environment variables.")
if not agentops_api_key:
    raise EnvironmentError("AGENTOPS_API_KEY not found in environment variables.")

# Initialize AgentOps session
agentops.init(api_key=agentops_api_key, default_tags=["crewai-simple"])

# Define the Simple Generator Agent
simple_generator = Agent(
    role='Simple Generator',
    goal='Generate a simple message based on input.',
    allow_delegation=False,  # No delegation to ensure single interaction
    backstory='You are a simple agent for generating messages.',
    llm=ChatOpenAI(model_name='gpt-4o-mini', openai_api_key=openai_api_key),
    max_iter=1  # Limit to one iteration
)

# Define the Simple Checker Agent
simple_checker = Agent(
    role='Simple Checker',
    goal='Check the generated message for any issues.',
    allow_delegation=False,  # No delegation to ensure single interaction
    backstory='You are a simple agent for checking messages.',
    llm=ChatOpenAI(model_name='gpt-4o-mini', openai_api_key=openai_api_key),
    max_iter=1  # Limit to one iteration
)

# Define the Simple Tasks
generate_message_task = Task(
    description="Generate a simple message based on input.",
    expected_output="A simple message generated by the agent.",
    agent=simple_generator
)

check_message_task = Task(
    description="Check the generated message for issues.",
    expected_output="If no issues, return 'Pass'. Otherwise, describe the issue.",
    agent=simple_checker,
    context=[generate_message_task]
)

# Function to save the final message, recorded with AgentOps
@agentops.record_action('save_final_message function execution')
def save_final_message(task_output):
    output_path = 'data/output/message.txt'
    output_data = json.loads(task_output.raw)
    if 'final_message' in output_data:
        with open(output_path, 'w') as file:
            file.write(output_data['final_message'])
        print(f"Final message has been written to {output_path}.")

# Create and execute the crew
crew = Crew(
    agents=[simple_generator, simple_checker],
    tasks=[generate_message_task, check_message_task],
    manager_llm=simple_generator.llm,
    process=Process.sequential,  # Ensure tasks are executed in sequence
    verbose=True
)

crew.kickoff(
    inputs={'message': 'Hello, this is a test message!'}
)

# End the AgentOps session
agentops.end_session("Success")

Expected Behavior:
Agents should complete their tasks without prematurely stopping due to iteration or time limits.
Actual Behavior:
Agents stop with the message "Agent stopped due to iteration limit or time limit," even though tasks are not expected to reach these limits.
Additional Notes:

The issue occurs only when AgentOps is enabled.
When AgentOps is disabled, the script runs correctly, and tasks complete successfully.

Please let me know if further information is needed to assist with debugging this issue.
Thank you!

Hey @nickzren -- We're taking a look. Can you verify:

Which version of AgentOps you're using?
Which version of CrewAI you're using?

Thanks
AgentOps: 0.3.9
CrewAI: 0.51.1

Thanks
AgentOps: 0.3.9 CrewAI: 0.51.1

Can you upgrade to 0.3.10? It's working for me in Jupyter notebook

This is odd, I have upgraded AgentOps to 0.3.10, but still getting the same error.

This is odd, I have upgraded AgentOps to 0.3.10, but still getting the same error.

Seems to be a LangChain error. No clear answer, but it may have to do something with your OpenAI rate limits.
Can you try swapping models?

Hey @nickzren -- any updates on this from you? Are you running into this issue with the latest version of Crew + AgentOps?


Cannot load theme.

Loading a theme by pointing powerline to it in the config.json file does not work. Even loading the default theme file default.py fails to work leading powerline-shell to throw the following error:
Traceback (most recent call last):
 File "/usr/bin/powerline-shell", line 11, in <module>
    sys.exit(main())
 File "/usr/lib/python3.6/site-packages/powerline_shell/__init__.py", line 214, in main
    theme = read_theme(config)
 File "/usr/lib/python3.6/site-packages/powerline_shell/__init__.py", line 177, in read_theme
    mod = importlib.import_module("powerline_shell.themes." + theme_name)
TypeError: must be str, not list
Inserting any data where the path for the theme file is supposed to go within the config.json file, throws the same error. I have installed and reinstalled powerline-shell multiple times as well as test for the issue on different machines only to recieve the same error.
My config.json follows:
{
 "segments": [
    "hostname",
   "username",
    "ssh",
   "cwd",
    "read_only",
   "root"
  ]
,"theme" : [
    "~/.config/powerline-shell/default.py"
 ]
}

Hey @Eel52 - the theme should not be a list, it should just be a string. Your config should be:
{
"segments": [
"hostname",
"username",
"ssh",
"cwd",
"read_only",
"root"
]
,"theme" : "~/.config/powerline-shell/default.py"
}

Make sure the path points to a real theme. The default configuration will not be at ~/.config/powerline-shell/default.py.


Screen flickers while transitioning

i have multiple transitions in my app and all of them flickers since im using the library.
the image is too big i'm sorry but look couple of times please..there is a black screen that flickers just in the start of the transition.
https://i.imgur.com/gMdw9lY.mp4

Same issue here on Hero 0.3.6 and iOS 11. It works correctly on iOS 10

Found a solution to this problem is other issue here.


[ACCSEC-33208] - Size Artifact iOS Framework

Ticket

[33208]

Github Issue

Description

Add Size Artifact for iOS Framework

Commit message

Screenshot

Testing

[ ] Added unit tests
[ ] Ran unit tests successfully
[ ] Added documentation for public APIs and/or Wiki

          1 Warning

:warning:
The Pull Request title does not follow the convention [00000] PR Title text

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Size impact

ABI
APK Size Impact

x86
2MB

x86_64
2MB

armeabi-v7a
2MB

arm64-v8a
2MB

universal
2MB

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
extracted/Payload/iosApp.app/Frameworks/TwilioPasskeys.framework: 5752 KB

  Generated by :no_entry_sign: Danger

🧛 shared module Code Coverage: 96.16%
Coverage of Modified Files:

File
Coverage

Modified Files Not Found In Coverage Report:
Appfile
Fastfile
Gemfile
Gemfile.lock
Matchfile
README.md
config.yml
dangerfile_ios_sdk_size_report
project.pbxproj
report.xml

Codebase cunningly covered by count Shroud 🧛

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
The Pull Request title does not follow the convention [00000] PR Title text

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Size impact

ABI
APK Size Impact

x86
2MB

x86_64
2MB

armeabi-v7a
2MB

arm64-v8a
2MB

universal
2MB

  Generated by :no_entry_sign: Danger

🧛 shared module Code Coverage: 96.16%
Coverage of Modified Files:

File
Coverage

Modified Files Not Found In Coverage Report:
Appfile
Fastfile
Gemfile
Gemfile.lock
Matchfile
README.md
config.yml
dangerfile_ios_sdk_size_report
project.pbxproj
report.xml

Codebase cunningly covered by count Shroud 🧛

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
The Pull Request title does not follow the convention [00000] PR Title text

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Size impact

ABI
APK Size Impact

x86
2MB

x86_64
2MB

armeabi-v7a
2MB

arm64-v8a
2MB

universal
2MB

  Generated by :no_entry_sign: Danger

🧛 shared module Code Coverage: 96.16%
Coverage of Modified Files:

File
Coverage

Modified Files Not Found In Coverage Report:
Appfile
Fastfile
Gemfile
Gemfile.lock
Matchfile
Pluginfile
README.md
config.yml
dangerfile_ios_sdk_size_report
project.pbxproj
report.xml

Codebase cunningly covered by count Shroud 🧛

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
The Pull Request title does not follow the convention [00000] PR Title text

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Size impact

ABI
APK Size Impact

x86
2MB

x86_64
2MB

armeabi-v7a
2MB

arm64-v8a
2MB

universal
2MB

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
extracted/Payload/iosApp.app/Frameworks/TwilioPasskeys.framework: 5752 KB

  Generated by :no_entry_sign: Danger

🧛 shared module Code Coverage: 96.16%
Coverage of Modified Files:

File
Coverage

Modified Files Not Found In Coverage Report:
Appfile
Fastfile
Gemfile
Gemfile.lock
Matchfile
Pluginfile
README.md
config.yml
dangerfile_ios_sdk_size_report
project.pbxproj
report.xml

Codebase cunningly covered by count Shroud 🧛

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
The Pull Request title does not follow the convention [00000] PR Title text

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Size impact

ABI
APK Size Impact

x86
2MB

x86_64
2MB

armeabi-v7a
2MB

arm64-v8a
2MB

universal
2MB

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
App size: 6.7 MB
TwilioPasskeys.framework: 5.6 MB, Impact: 84.4%

  Generated by :no_entry_sign: Danger

🧛 shared module Code Coverage: 96.16%
Coverage of Modified Files:

File
Coverage

Modified Files Not Found In Coverage Report:
Appfile
Fastfile
Gemfile
Gemfile.lock
Matchfile
Pluginfile
README.md
config.yml
dangerfile_ios_sdk_size_report
project.pbxproj
report.xml

Codebase cunningly covered by count Shroud 🧛

  Generated by :no_entry_sign: Danger

@AlejandroOrozco For verify push, there are two values: compressed and uncompressed, what is the difference with the value reported here?

Basically, compressed refers to when it is in its IPA state. All IPA files are just compressed binaries of the applications. Uncompressed refers to what we are using for this. We can report both values without any issue, but I think the uncompressed size is more precise.

          1 Warning

:warning:
The Pull Request title does not follow the convention [00000] PR Title text

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Size impact

ABI
APK Size Impact

x86
2MB

x86_64
2MB

armeabi-v7a
2MB

arm64-v8a
2MB

universal
2MB

  Generated by :no_entry_sign: Danger

🧛 shared module Code Coverage: 96.16%
Coverage of Modified Files:

File
Coverage

Modified Files Not Found In Coverage Report:
Appfile
Fastfile
Gemfile
Gemfile.lock
Matchfile
Pluginfile
README.md
config.yml
dangerfile_ios_sdk_size_report
project.pbxproj

Codebase cunningly covered by count Shroud 🧛

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
IPA size: 1917.7 MB
App size: 6.7 MB
TwilioPasskeys.framework: 5.6 MB, Impact: 84.4%

  Generated by :no_entry_sign: Danger

Can you change to dev as the base branch?

@AlejandroOrozco For verify push, there are two values: compressed and uncompressed, what is the difference with the value reported here?

Basically, compressed refers to when it is in its IPA state. All IPA files are just compressed binaries of the applications. Uncompressed refers to what we are using for this. We can report both values without any issue, but I think the uncompressed size is more precise.

So, if I understood correctly, compressed will be the impact to the IPA (download from appstore?) and uncompressed will be the impact in the device, correct? I can think of times were customers were worried about the download size (IPA, compressed?) and also the installation size (uncompressed?), so it could be helpful to have both. Also, it could be interesting to know the impact for compressed and the percentage, to see if the zip process is reducing the percentage compared to uncompressed

@AlejandroOrozco For verify push, there are two values: compressed and uncompressed, what is the difference with the value reported here?

Basically, compressed refers to when it is in its IPA state. All IPA files are just compressed binaries of the applications. Uncompressed refers to what we are using for this. We can report both values without any issue, but I think the uncompressed size is more precise.

So, if I understood correctly, compressed will be the impact to the IPA (download from appstore?) and uncompressed will be the impact in the device, correct? I can think of times were customers were worried about the download size (IPA, compressed?) and also the installation size (uncompressed?), so it could be helpful to have both. Also, it could be interesting to know the impact for compressed and the percentage, to see if the zip process is reducing the percentage compared to uncompressed

To be improved in a future task

Can you change to dev as the base branch?

Ping!

          1 Warning

:warning:
The Pull Request title does not follow the convention [00000] PR Title text

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Size impact

ABI
APK Size Impact

x86
2MB

x86_64
2MB

armeabi-v7a
2MB

arm64-v8a
2MB

universal
2MB

  Generated by :no_entry_sign: Danger

🧛 shared module Code Coverage: 96.16%
Coverage of Modified Files:

File
Coverage

Modified Files Not Found In Coverage Report:
Appfile
AuthenticationManager.swift
Fastfile
Gemfile
Gemfile.lock
Matchfile
Pluginfile
README.md
config.yml
dangerfile_ios_sdk_size_report
iosApp.entitlements
project.pbxproj

Codebase cunningly covered by count Shroud 🧛

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
IPA size: 1.9 MB
App size: 6.6 MB
TwilioPasskeys.framework: 5.5 MB, Impact: 84.2%

  Generated by :no_entry_sign: Danger


Bump dependency versions

Bumps dependency versions:

vite -> v5
svelte -> v4
sveltekit -> v2

Something is still broken atm, but I'm not even sure how to fix it but hopefully I will soon™
css-tree is a dependency of Svelte, but somehow after the upgrades it is getting included in the houdini-plugin-svelte-global-stores output, which causes this issue.
Ideas on how to fix this are more than welcome 🙃
To help everyone out, please make sure your PR does the following:

[ ] Update the first line to point to the ticket that this PR fixes
[ ] Add a message that clearly describes the fix
[ ] If applicable, add a test that would fail without this fix
[ ] Make sure the unit and integration tests pass locally with pnpm run tests and cd integration && pnpm run tests
[ ] Includes a changeset if your fix affects the user with pnpm changeset

Ok I'm going to close this as it would be too much of a hassle to get this working correctly. I think the only fix for the css-tree problem is upgrading to svelte 5, but that's already being solved in #1300


master: add MemoryUsageMetricSet and GarbageCollectorMetricSet

It would be nice to keep these stats and have them emitted via the
metrics reporter. This will help for tracking heap size (of different
generations), time spent in GC, etc etc.

@rohansingh @dflemstr @negz @davidxia

Not sure if rearranging all those imports was entirely necessary but 👍


Add additional mods

Add the following additional mods:
https://modrinth.com/mod/memory-usage-screen
https://modrinth.com/mod/controlling/
https://modrinth.com/mod/dimensionaldoors
https://modrinth.com/mod/fix-keyboard-on-linux
https://modrinth.com/mod/krypton
https://modrinth.com/mod/timeoutout
https://modrinth.com/mod/inventory-profiles-next
https://modrinth.com/mod/memoryleakfix
https://modrinth.com/mod/morechathistory
Zoomify
https://modrinth.com/mod/language-reload
https://modrinth.com/mod/fastquit
https://modrinth.com/plugin/chunky
https://modrinth.com/mod/wthit
https://modrinth.com/mod/sound-physics-remastered
https://modrinth.com/mod/htm
https://modrinth.com/mod/shulkerboxtooltip
https://modrinth.com/mod/trinkets
https://modrinth.com/mod/travelersbackpack/
https://modrinth.com/mod/clear-void
https://modrinth.com/mod/mouse-tweaks
https://modrinth.com/mod/universal-graves

Added missing mod badpackes as well. Whoopsie!


Tooltip variable should be printed in the listing.

bug found by @brittsikora
Test notes

The tooltip_col variable should appear in the details listing.

Tested and confirmed that the Details column (which is the default tooltip_col variable) now appears in the details listing by default.


Don't check for ConfigurationOptions equality during ConfigurationNode equality check

I was expecting there to be more to this, but it doesn't appear as though there are any changes required elsewhere to fix this.
Closes #114

That whole method can probably be replaced by
return Objects.hashCode(key) ^ value.hashCode();

@lucko Implemented the requested change. Silly mistake.

@kashike Also fulfilled your request.


Master to Live

Docs Build status updates of commit 0d4899d:
:clock10: Full build: the average full build time is 17 min(s) 36 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 0d4899d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/app-only-auth-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-online-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-ClientAccessRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-ClientAccessRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Start-HistoricalSearch.md
:white_check_mark:Succeeded
View (exchange-ps)

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOTenant.md
:white_check_mark:Succeeded
View (sharepoint-ps)

teams/teams-ps/teams/Get-CsBatchPolicyAssignmentOperation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-TeamChannel.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/New-CsGroupPolicyAssignment.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Not showing any answer with Mistral model but still showing sources (RAG/Docker Setup)

I am running on an H100 with a docker setup. I keep running into this issue when I use mistral 7b that an answer will not show up, but the sources will still show as if it knows I'm asking a question about my document. So I know the answer is generating, but I just don't see anything. I was doing some digging in the github and @pseudotensor recommended to use mistral 7b for rag and not llama 2 7b. I have this long, repetitive document that llama 2 7b can't decipher because I assume of it's limited context window. I have tried changing --max_seq_len to see if that was the problem, but I know I have enough vram to probably run it at the maximum sequence length. Any help would be appreciated. Added pictures of the background activity as well.

use latest docker image tag and set prompt mode to mistral instead of llama2

Switched prompt mode to mistral and still getting no output. And I updated my docker image last week is there a new one already?

@dakotal4  You are using a foundational model rather than an instruct model, so the prompt is "plain" by default (i.e. no prompting).  If you really intend this, you can try using "one_shot" for prompt type, but I assume you are just making a mistake in using mistralai/Mistral-7B-v0.1.
I think you instead meant to use mistralai/Mistral-7B-Instruct-v0.1 and we'll detect that name as "mistral" prompt_type or you can pass it.


Build a docker image for openchatkit

Is your feature request related to a problem? Please describe.
A docker image might be easier for people to use.
Describe the solution you'd like
We could add a /docker folder or a simple dockerfile to the repo, so people could build the image by themselves. And maybe we could push the image to dockerhub so they could just pull and test.

Thanks for the feature request. This is a great idea. Will put it on the roadmap.

Hey @Jonuknownothingsnow  i am new to open source . and it would be very happy to work on this idea under your guidance

Dockerfile
# Base image
FROM ubuntu:20.04

# Set working directory
WORKDIR /app

# Update and install required packages
RUN apt-get update && \
    apt-get install -y git-lfs wget && \
    rm -rf /var/lib/apt/lists/*

# Download and install Miniconda
RUN wget https://repo.anaconda.com/miniconda/Miniconda3-latest-Linux-x86_64.sh && \
    bash Miniconda3-latest-Linux-x86_64.sh -b -p /opt/conda && \
    rm Miniconda3-latest-Linux-x86_64.sh

# Set conda to automatically activate base environment on login
RUN echo ". /opt/conda/etc/profile.d/conda.sh" >> ~/.bashrc && \
    echo "conda activate base" >> ~/.bashrc

# Create OpenChatKit environment
COPY environment.yml .
RUN conda env create -f environment.yml

# Install Git LFS
RUN git lfs install

# Copy OpenChatKit code
COPY . .

# Prepare GPT-NeoX-20B model
RUN python pretrained/GPT-NeoX-20B/prepare.py

# Set entrypoint to bash shell
ENTRYPOINT ["/bin/bash"]

Build the Docker image using the following command:
docker build -t openchatkit .

Run the Docker container using the following command:
docker run -it openchatkit

This will start a new bash shell in the container.
Activate the OpenChatKit environment by running the following command:
conda activate OpenChatKit

You should now be able to use the OpenChatKit code and run the prepare.py script.

As I mentioned in the PR, both the pretrained model and datasets can be quite large.
$ du -sh data/* pretrained/GPT-NeoX-20B/
172G    data/OIG
238M    data/OIG-moderation
38G     data/wikipedia-3sentence-level-retrieval-index
39G     pretrained/GPT-NeoX-20B/

The Dockerfile above bakes the 39GB pretrained model into the image. In my opinion, it would be better to download the pretrained model into a bind mount when the container starts. The image would be much smaller and the bind mount persists the model across container restarts.

As I mentioned in the PR, both the pretrained model and datasets can be quite large.
$ du -sh data/* pretrained/GPT-NeoX-20B/
172G    data/OIG
238M    data/OIG-moderation
38G     data/wikipedia-3sentence-level-retrieval-index
39G     pretrained/GPT-NeoX-20B/

The Dockerfile above bakes the 39GB pretrained model into the image. In my opinion, it would be better to download the pretrained model into a bind mount when the container starts. The image would be much smaller and the bind mount persists the model across container restarts.

Sure i will try to do it

Hello,
I'm just starting with OpenChat, I was checking to use Docker and I found your Dockerfile.
One question, for training the model, I understand cuda + nvidia are used if available?
If that yes, yesterday I found this, maybe useful https://blog.roboflow.com/nvidia-docker-vscode-pytorch/
Looks like there are docker-nvidia accelerated containers nvidia/cuda:11.0.3-base-ubuntu20.04 ready to be used.
(Note: Because I use docker-compose I had to update to a 1.28.0+ version to configure of '--gpus all' parameter)
Thank you

Hello there,
Just mention that I had issues building the environment with package netifaces, that I solved updating environment.yml file from netifaces===0.11.0 to netifaces2==0.0.16
Thank you

Hello there,
Just mention that I had issues building the environment with package netifaces, that I solved updating environment.yml file from netifaces===0.11.0 to netifaces2==0.0.16
Thank you

This can be fixed by installing gcc. On Ubuntu, you'd run sudo apt install gcc. That should fix your error!

If that yes, yesterday I found this, maybe useful https://blog.roboflow.com/nvidia-docker-vscode-pytorch/ Looks like there are docker-nvidia accelerated containers nvidia/cuda:11.0.3-base-ubuntu20.04 ready to be used. (Note: Because I use docker-compose I had to update to a 1.28.0+ version to configure of '--gpus all' parameter)

Thanks for the great resource @xsanz! I was able to get the model loaded onto the GPU in docker using those instructions

the conda binary was not found during my docker build. If anyone runs into this issue, I had set the PATH correctly before trying to run conda
ENV PATH=/opt/conda/bin/:$PATH


Remove version: BBC.iPlayer version 2.13.13

installers return following HTTP status code: 404
Microsoft Reviewers: Open in CodeFlow

URL: https://static.files.bbci.co.uk/iplayer-pc-download-app/releases/win32-x64/BBCiPlayerDownloadsSetup-2.13.13.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 920)


implement a kubernetes runtime for linux executor

Description
What is your idea?
Value
Why is this important? Who does it impact? Will this make something better, faster, etc?
Definition of Done
What is the end goal of this story?
Effort  (Optional)
Estimated effort to complete this story? (Best Guess e.g. 1-2 days)
Impacted Personas (Optional)
Which personas will benefit from completing this story? (Users, Administrators, etc)

@kneal and I are going to begin by writing the ability to create volumes in Kubernetes.
It appears the gitRepo volume type is deprecated:

Warning: The gitRepo volume type is deprecated. To provision a container with a git repo, mount an EmptyDir into an InitContainer that clones the repo using git, then mount the EmptyDir into the Pod’s container.

We will follow the recommended guidelines to use emptyDir instead 👍

After some trial and error, we've come to a roadblock 😞
Unfortunately, it would appear that Kubernetes prevents the ability to patch specific fields in a Pod. Specifically, it appears that we're unable to patch some Container fields which is required to maintain a "1 pod per build" strategy.
The initial plan was to create a single pod per build, and then to patch that pod with each container (step) from the pipeline.
The error message we receive:
spec: Forbidden: pod updates may not change fields other than `spec.containers[*].image`, `spec.initContainers[*].image`, `spec.activeDeadlineSeconds` or `spec.tolerations` (only additions to existing tolerations)

Issue with more information:
https://github.com/kubernetes/kubernetes/issues/9043

Create separate runtime package:
https://github.com/go-vela/pkg-runtime/pull/1

Store Kubernetes pod in client struct:
https://github.com/go-vela/pkg-runtime/pull/2

Use the go-vela/compiler for our runtime CLI:
https://github.com/go-vela/pkg-runtime/pull/3

Create a working runtime CLI:
https://github.com/go-vela/pkg-runtime/pull/4

Add container wait logic for Kubernetes:
https://github.com/go-vela/pkg-runtime/pull/5

Add container tail logic for Kubernetes:
https://github.com/go-vela/pkg-runtime/pull/6

Use default (strategic merge) patch for Kubernetes pods:
https://github.com/go-vela/pkg-runtime/pull/7

Add logic to remove Kubernetes resources:
https://github.com/go-vela/pkg-runtime/pull/8

Tune Kubernetes tail backoff for large pipelines
https://github.com/go-vela/pkg-runtime/pull/9

Implement the inspect process for Kubernetes
https://github.com/go-vela/pkg-runtime/pull/10

Make the namespace configurable
https://github.com/go-vela/pkg-runtime/pull/11

Add function to create new runtime:
https://github.com/go-vela/pkg-runtime/pull/12

Add validate logic for runtime:
https://github.com/go-vela/pkg-runtime/pull/13

Add actions pipeline:
https://github.com/go-vela/pkg-runtime/pull/14


Consider what to do with Avro aliases

Currently we don't support Avro aliases, since Lancaster doesn't. I've created a ticket at the Lancaster repo, and there's an ongoing discussion regarding that feature and our use case:
https://github.com/deercreeklabs/lancaster/issues/16
Part of completing this issue is to also document the mapping in the wiki. Please let me review the wiki changes.

Aliases seem to primarily serve the purpose of faciliating schema evolution. I think we should consider dropping this entire mapping, and leave it to users to extend the generated schema such that they maintain compatibility with schemas of different versions.
From the spec:

Named types and fields may have aliases. An implementation may optionally use aliases to map a writer's schema to the reader's. This faciliates both schema evolution as well as processing disparate datasets.
Aliases function by re-writing the writer's schema using aliases from the reader's schema. For example, if the writer's schema was named "Foo" and the reader's schema is named "Bar" and has an alias of "Foo", then the implementation would act as though "Foo" were named "Bar" when reading. Similarly, if data was written as a record with a field named "x" and is read as a record with a field named "y" with alias "x", then the implementation would act as though "x" were named "y" when reading.
A type alias may be specified either as a fully namespace-qualified, or relative to the namespace of the name it is an alias for. For example, if a type named "a.b" has aliases of "c" and "x.y", then the fully qualified names of its aliases are "a.c" and "x.y".

Apparently, Joep didn't have any plans to use this field. AFAIC this issue can be closed (and so can the one at the Lancaster repo).


Get rid of system tests from after-merge workflow

Is your feature request related to a problem? Please describe.
We have ci_pipeline.yml, which is triggered every after merge.
However, this result is always "red", because we have unstable system tests. Now we don't check or fix the failure. For me, it makes no sense to keep such tests.
Describe the solution you'd like
My proposal is to disable the jobs below from ci_pipeline.yml.

system-test-offsite-matrix
system-test-postprocessing

Describe alternatives you've considered
How to disable the jobs? => Either comment them out or completely remove these lines. I'd prefer commenting them out as we can keep some information how to use GitHub Secrets...
Additional context
I would like to remark, other jobs in ci_pipeline.yml works as expected.

Once we decide to deprecate our pysys tests, we can close the following issues.

https://github.com/thin-edge/thin-edge.io/issues/1089
https://github.com/thin-edge/thin-edge.io/issues/1088
https://github.com/thin-edge/thin-edge.io/issues/1083
https://github.com/thin-edge/thin-edge.io/issues/1081
https://github.com/thin-edge/thin-edge.io/issues/1012
https://github.com/thin-edge/thin-edge.io/issues/804
https://github.com/thin-edge/thin-edge.io/issues/788
https://github.com/thin-edge/thin-edge.io/issues/752

Our pysys tests won't be removed by this task. Pysys tests will be replaced by the framework proposed by https://github.com/thin-edge/thin-edge.io/issues/808.

How to disable the jobs? => Either comment them out or completely remove these lines. I'd prefer commenting them out as we can keep some information how to use GitHub Secrets...

Hi! Why not removing them? They are still in the git history and the "Remove xyz test" patches can easily be git-reverted! Outcommented code is just dept.

How to disable the jobs? => Either comment them out or completely remove these lines. I'd prefer commenting them out as we can keep some information how to use GitHub Secrets...

Hi! Why not removing them? They are still in the git history and the "Remove xyz test" patches can easily be git-reverted! Outcommented code is just dept.

Going to remove :D Don't worry.

We can't close these two issues.

https://github.com/thin-edge/thin-edge.io/issues/1089
We still run smoke test to c8y from the after-merge workflow. The test uploads the certificate to Cumulocity.
https://github.com/thin-edge/thin-edge.io/issues/792
It looks we have some issue in batching. Need further investigation.

Two jobs below from ci_pipeline.yml is already disabled by #1300.

system-test-offsite-matrix
system-test-postprocessing

Also, close some issues no longer applicable. So, I would close the issue.


Remove safety parser and db

At the request of pyup.io, we had to remove the parser for Safety scans.

You beat me by 3 minutes 😂 I like yours better though. Will approve after tests pass.


Added OSX to Travis CI for external-storage

Added .travis.yml changes for testing on OSX in travis-ci. Similarly, added logic in test.sh for building and unit testing on OSX.
After this PR is merged, if someone attempts to checin something that doesn't work on OSX, then the CI will fail and they'll see the errors.
Certain duplicate test are skipped, to keep CI fast and save travis-ci resources. For example, skip linux tests on osx and vice versa.
Note: travis-ci on osx does not support docker, yet. Therefore, a docker alias is created to enable docker commands to execute successfully for OSX build and test.
You can see the successful run of these travis-ci changes here: https://travis-ci.org/ianchakeres/external-storage/builds/356325691

Here is a PR adding OSX to the external-storage travis-ci. If you have any questions, please let me know.
/assign @wongma7 @childsb

@wongma7 @childsb - Can you please take a look at this PR and let me know if you have any questions or comments?

/lgtm thank you

this PR Is so old you changed your place of employment & presumably git email since, merging


Rich Presence not showing in Discord

The program runs fine without any errors but the rich presence doesn't seem to be showing in discord. am I missing a step in the tutorial?

havent opened the rich presence in a while since i dont play as much now but will see if mine works still and troubleshoot your issue

also when opening the rich presence, make sure you dont close the cmd console thing thats what makes the rich presence happen o/

ah right i also removed the release for the latest one now that i see
should have that but the latest doesnt, so think that would be why

anyway ill troubleshoot it now, i was downloading latest and fixing other things

ill make a new release in a bit

alright thank you so much

alright it works now thank you


Question: Can the PolicyRecordingBot be run locally?

Please excuse my ignorance here but I'm new to Bot Framework and Microsoft Graph.  I have downloaded this sample and working with my Tenant Admin (who did all the 'setup' type activities in the Azure Portal side) I have updated the code to contain the necessary values for the cert thumbprint, appID, etc.  After overcoming initial build issues around having a CNG cert and not a CSP one we can now get the project to build and run without throwing an exception.   However, the problem I have is that I cannot seem to access the web/api interface (i.e. http://localhost:9441/logs) so that I can step into the controllers within the CRFrontEnd projectjust so I can have a nosey and how this thing works before i start looking into the areas triggered by the call events.
When i access the NGrok URIs they return a 502 Bad Gateway due to the above mentioned endpoint(s) not being there.
I do have some MVC and WebAPI experience and to sanity check myself I have even created a simplistic 'do-nothing' bot from the VS2019 template and i can spin that up and access it's web interface (granted it's hosted in IISExpress) and step into the controller code fine....................so, what am i missing here......been puzzled for a couple of days now :(

+1 please, I want to know the same thing here.

What fixed it for me was editing file https://github.com/microsoftgraph/microsoft-graph-comms-samples/blob/master/Samples/V1.0Samples/LocalMediaSamples/PolicyRecordingBot/FrontEnd/Http/HttpConfigurationInitializer.cs to have
httpConfig.DependencyResolver.BeginScope(); right before the httpConfig.EnsureInitialized();.  After doing that i was able to at least talk with the Localhost endpoint(s)

@andir78 so with that modification you can run the entire sample on your local enviroment? or there is another issue?
thanks

@ovninosa apologies for the tardy response.  To answer your question, I honestly cannot remember as i feel like i've had to modify and rewrite so much to get to where i am.  It's pretty hard to get any meaningful support from anywhere too as it seems so 'bleeding edge'.
Closing now though, as i forgot to do so in July


return type of query not as array if simple property

When having someting like this:

      "value" : [ {
        "id" : 2,
        "name" : "alf"
      }]

and I do the query
<EMAIL_ADDRESS>== 2)]
then the returned object is a JSONArray
[2]
instead of a Integer
2
Please return is as simple type and not within the corresponding array.

Okay, I guess to return the array is the right thing, because there can be more elements returned by this query.


Revert "✨ add SVG support for logos"

Reverts nunocoracao/blowfish#1646

Rationale:
1. This does not handle the "secondary" logo at all ie. breaks the parameter.
2. The raw SVG logo data will be injected into every HTML file, therefore will not be cached!

:warning: Please do not merge yet, this is WIP :warning:

Question: What do the width and height params on the original img.logo DOM element even do that isn't already done in CSS?

@ldericher I am closing this PR as it is not ready to merge. Please work off your branch and open the PR once it's ready to merge. If you need help let's use the respective issue


spark_apply not working with sdf_repartition

From @kevinykuo:
iris_tbl <- copy_to(sc, iris) %>%
  sdf_repartition(2L)

spark_apply(iris_tbl, function(data) {
  data[1:4] + rgamma(1,2)
})

 Error: org.apache.spark.SparkException: Job aborted due to stage failure: Task 1 in stage 8.0 failed 1 times, most recent failure: Lost task 1.0 in stage 8.0 (TID 8, localhost, executor driver): java.lang.Exception: sparklyr worker rscript failure, check worker logs for details.
	at sparklyr.Rscript.init(rscript.scala:51)
	at sparklyr.WorkerRDD$$anon$2.run(rdd.scala:79)

This is now working for some reason (also works with memory = FALSE). Let's leave open until we have a test?

Looks fixed by https://github.com/rstudio/sparklyr/pull/883, can't repro this anymore.


Better to have a sample on Geographical Distributed wide API Management

Description:
Geographical Distributed wide API Management is widely used use case.
It's better if we have a sample on this area.
Suggested Labels:
Type/Docs
Type/Improvement
Priority/Normal
Affected/2.2.0
Affected Product Version:
WSO2 API Manager 2.2.0

We will not be fixing this as this issue is based on an older version of the product.


flag-list.scss compiler errors

I'm getting multiple compiler errors in this file:
OS: Windows 10
Compiler<EMAIL_ADDRESS>
Line 23:  @if $ioc = null...
assignment operator ( = ) is used in stead of evaluation operator ( == ). This holds for other @if/else conditionals
Line 53: @include flag(and, ad, 20, and, and);
Line 276: @include flag(tza, tz, 834, tan, tan);
arguments aren't quoted, resulting in errors on the mixin - reserved words and and tan seem to be throwing the error.

Have you managed to resolve this? We have the same compile errors.

I don't have Windows. Could you PR?

To submit a PR I would need to first find the cause and implement a solution?

Yeah, I resolved. The issue was with the naming convention used - as per my OP, the string concatenation threw errors when the compiler did an eval and ran into the reserved words. I think I ended up quoting or back ticking or something I'll take a look at it today...

Thanks that would be really helpful.

@ProNotion sorry about the long wait, please see pull request added above.

Thanks @murraybiscuit !

@murraybiscuit Thanks that's great. @7kfpun will there be a Bower package update with this fix?
Thanks guys!

Yes, soon!


Feature idea: screenshots

Several different potential use cases revolving around screenshots:

Generating screenshots endpoint. Maybe it allows you to specify some viewport options, so you can get a good size view
An option to have screenshots generated and injected into response as the open graph/image sharing tags.

<meta property="og:image" content="generated-screenshot.png" />

Screenshots implem


fix uppercases and spaces

This solves https://segment.atlassian.net/browse/INT-655
Hubspot doesn't accept any trait keys that include uppercases or spaces. So we're lowercasing any trait keys if uppercase is detected, replacing any spaces with underscores if spaces are detected.

[ ] server side
[ ] docs
@myclamm @f2prateek

Current coverage is 100% (diff: 100%)

Merging #10 into master will not change coverage

@@           master   #10   diff @@
===================================
  Files           1     1
  Lines          23    37    +14
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
+ Hits           23    37    +14
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update ba63c00...06ee296

LGTM


biological_validation_with_iLINCS_portal

The iLINCS portal can be used to search for documented perturbations connected to a certain gene.
The idea is to integrate the iLINCS API at the end of our benchmarking pipeline for biological validation. The necessary script should check (for a gene X where Digital KO was performed) which other genes will be perturbed according to iLINCS and compare the predicted results with the iLINCS database results. (For example, how many documented Genes were predicted by the tool in %).
Primarily the idea emerged during the GRN-models meeting, but it might be useful for other tools...
iLINCS: https://www.ilincs.org/ilincs/
API: https://www.ilincs.org/ilincs/APIinfo

Contribution Guidelines

Initial Setup

Clone the repository: git clone <repository_url>

Update Local Content

Ensure you’re up to date with the development branch:git checkout devel
git pull origin devel

Create a New Branch

Create a descriptive branch for your work:git checkout -b <branch_name>

Commit & Push Changes

Stage and commit your changes with clear messages:git add <file_name>
git commit -m "Brief description of changes"

Push to GitHub:git push origin <branch_name>

Create a Pull Request

Target the devel branch for merging.
Follow the PR template, review, and submit.

Final Merge & Update

Once approved, merge your PR and delete your branch if desired.
Update your local repository:git checkout devel
git pull origin devel

By following these steps, you contribute effectively and collaboratively.


updating index and api page

This updates the index page, and also adds an API page. I'm not sure how to document a traitlets API automatically...anyone have an idea for how to do this with sphinx? @yuvipanda or @minrk or @willingc ?
(the samples.rst is also moved from CRLF to LF, hides head in shame)

@choldgraf Are you looking to do something similar to this section of the jupyterhub docs?
If so, you would do something similar to this source for the doc page. You would also add https://github.com/jupyterhub/jupyterhub/blob/master/docs/source/conf.py#L23 to conf.py and add the sphinxext directory too.

ah perfect, yes I think autodoc_traits is what I was looking for :-) I'll give it a shot

hmmm, OK latest push has autodoc_traits installed and building, though I can't get the docs quite how I want them. I'm hoping to list the arguments that are defined in get_argparser for the Repo2Docker class...but it isn't showing up with autoconfigurable. Any idea how to get that working?
Here's what it looks like now:
http://predictablynoisy.com/repo2docker/api.html

I'm gonna close this one in favor of #165 which covers the same ground and is less stale. We can figure out API stuff in a separate PR but it seems like it'll be non-trivial for me to tack on


copyright relabel... from c* to   © ...

change from c* to   © ..
Note: don't forget the hidden copyright line in the login page... also change it to ©

change all of C* to ©


⚠ Bofrost.com has degraded performance

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

HTTP code: 200
Response time: 9754 ms

Resolved: Bofrost.com performance has improved in c250625 after 6 minutes.


Region detection algorithm not always working properly

Great Britain allows two different valid time formats:

The 24-hour time format:  14:34
The 12-hour time format: 2:34 pm

Intl uses the 24-hour time format for locale en-GB.
If you want to use the 12-hour time format for GB then you have to use locale en-GB-u-hc-h12.
Please find here a litte demo.
So now assume you are using locale en-GB-u-hc-h12. Then this line
https://github.com/shoelace-style/localize/blob/421744e6be069d92cf65b4e52e05a979a72cb34a/src/index.ts#L120
will fail to detect the proper code (which should be "en-gb").
Please find here a little demo that shows one case where that bug leads to wrong i18n behavior.
It might be a good idea to  use new Intl.Locale('en-GB-u-hc-h12') (see here and here).

Just a heads up — 3.0.2 detects the region correctly but fails to prepend the language code to it. 🤦🏻♂
Please use 3.0.3 instead, which works as intended.


Missing option to hide the row id column

For the end user, it's often not necessary to see the row id column, wasting valuable screen space. An option to hide this column would make this software even better!
Thanks!

Are you referring to these numbers on the left?

Yes, not sure, if this is the ID-column, sorry for the confusion, but a hiding option would be great!

I agree with adding an option to remove the ID column / numbering (column most left) from showing. For my use case I have no need to see the ID and in my case it's both visually unpleasing and cluttering the screen. Hope this option can be added, thank you :)


Master not discovered

when  I am trying to use this command   curl -XGET --insecure http://localhost:9200/_cat/indices -u admin:admin  It gives me this error!
{"error":{"root_cause":[{"type":"master_not_discovered_exception","reason":null}],"type":"master_not_discovered_exception","reason":null},"status":503}

@bittu664 please provide more context on this issue - about your setup and which plugin?

Closing for now. @bittu664 if you're still looking for help, feel free to post a question to the Open Distro forum.


can't I use viewModel and DestinationsNavigator  at the same time?

how can I  use both of viewModel which use hilt and navController or DestinationsNavigator as my composable function parameters, by the way I use navArgsDelegate argument  parameter of Destination annotation to clarify arguments passed by navigate function. I am asking this question because I can't provide two of them at the same time, it gave me an error for DestinationsNavigator which was ,  the library have been wanted DestinationsNavigator from me  but as I understood it should have been given by library.

You most definitely can :)
Can you copy paste the error you get when using both?

yes you are 100% right after I realize the gradlew kspDebugKotlin command doesn't work for me to build new or changed function names or any code changes  related to it so I had to rebuild the project and after that it worked. did I misuse command kspDebugKotlin? by the way what does it mean Debug between ksp and Kotlin for this command?

You can just run the app and it will generate the classes, no need for specific command. But Debug should be a combination of flavour and buildType. Sometimes it’s useful if you need to use a file that was not yet generated.
I’ll close then, let me know below if there is any other issue.

thank you very much by the way your library is amazing, I hope android team see your library and select it as default in next major release of navigation compose.

Never gonna happen ahah 😝 thanks !

This is not an issue, you need to pass the VM explicitly in some way.
Check here:
https://composedestinations.rafaelcosta.xyz/common-use-cases/providing-viewmodels

This is not an issue, you need to pass the VM explicitly in some way. Check here: https://composedestinations.rafaelcosta.xyz/common-use-cases/providing-viewmodels

Thank you very much!
Now everything is in place. Link is super usefull and explain everything i needed.

Great! Glad it helped

Sir @raamcosta, I follow the above link, the viewmodel is shared but data is not coming with viewmodel. Can you please help me in this.


how can the lstm learn from request label?

1.how can the lstm learn from the request label given next timestep?

I found something that could be wrong

the reward code:
a_t = tf.cast(a_t, tf.float32)
label_t = tf.cast(label_t, tf.float32)
rewards_t = label_t*params.reward_correct + (1-label_t)*params.reward_incorrect # (batch_size, num_labels), tf.float32
rewards_t = tf.pad(rewards_t, [[0,0],[0,1]], constant_values=params.reward_request) # (batch_size, num_labels+1), tf.float32
r_t = tf.reduce_sum(rewards_t*a_t, axis=1) # (batch_size), tf.float32
is the code means   the agent will get a reward no matter whether it requests the label or not,and the reward would guide agent to choose the right choice?

We have not done experiments to see how the lstm is learning. The assumption is that it has learned to store a representation of the example in the hidden state, and on the subsequent step store the label for that example. Then, in future steps, compare the new example with the stored representations for previous examples.

rewards_t stores rewards for all possible actions (i.e. n_labels-1 reward_incorrect, 1 reward_correct, 1 reward_request. a_t is a one-hot vector for the chosen action. The multiplication selects out the reward for the chosen action.

but  in the view of my comprehension, it seems have some problem if every action have a direct reward.
For a random policy at the beginning,the agent can learn to make decision correctly only from incorrect or correct reward,without the request.
or I have some misunderstand?

Let me try to give an example of possible learning path.

The agent randomly (e-greedy) chooses the "request label" action on the
first time step of an episode, and receives -0.05 reward
The label is supplied on the next step
later in the episode an example of the same class is provided as input
The agent randomly (e-greedy) predicts the correct label, and receives +1

back-propagation, in reducing sum of the Q-function errors over the entire
episode, makes the connection that requesting the label, incurring a small
penalty, storing the label, will then allow it to predict correctly later
and receive a larger reward.
At a high level, understanding the supervised situation first, might help
in understanding my paper: https://arxiv.org/abs/1605.06065
On Fri, Jul 6, 2018 at 10:34 AM, flazerain<EMAIL_ADDRESS>wrote:

but in the view of my comprehension, it seems have some problem if every
action have a direct reward.
For a random policy at the beginning,the agent can learn to make decision
correctly only from incorrect or correct reward,without the request.
or I have some misunderstand?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/markpwoodward/active_osl/issues/1#issuecomment-403100020,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGgTpdgjH4O8UR0vlayiz6hJVZBPApxxks5uD5-wgaJpZM4VDafr
.

but if it randomly chooses a correct action for classification,it will get 1 reward. it seems like a supervise signal for the unlabel data.

The lstm never sees the rewards. The rewards are used in the loss after all
actions have been chosen for that episode.
On Fri, Jul 6, 2018 at 11:02 AM, flazerain<EMAIL_ADDRESS>wrote:

but if it randomly chooses a correct action for classification,it will get
1 reward. it seems like a supervise signal for the unlabel data.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/markpwoodward/active_osl/issues/1#issuecomment-403106767,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGgTpTuI4M_Cl3norK9n-9QMJFv2jS5nks5uD6YngaJpZM4VDafr
.


Graceful handling for PulsarClient.CloseAsync

Hi,
Current version of the client appears to rely on socket connection dispose call to break out of read operation when PulsarClient.CloseAsync is called. This leads to warnings in the log and prints out an exception like this:
System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'SslStream'.
...
These exceptions show up a few seconds after the PulsarClient.CloseAsync call was made, so don't easily show up on integration tests.
The proposed change supplies a CancellationToken to the PipeReader.ReadAsync method. And then triggers token cancellation on Dispose method call in ClientCnx.
Here is an example test (C#) to observe the fix, assuming we have an ILogger implementation that collects log statements as a string:
    [Fact]
    public async Task Client_Close_Graceful()
    {
        // Arrange
        var logTextBuilder = new StringBuilder();
        var logger = new StringLogger(logTextBuilder, LogLevel.Information);

        // Act
        var client = await GetClientAsync(logger);
        var topic = "public/default/test_graceful_client_close";
        var producer = await client.NewProducer()
            .Topic(topic)
            .CreateAsync();
        await producer.DisposeAsync();
        await client.CloseAsync();

        // Wait for socket timeout
        await Task.Delay(TimeSpan.FromSeconds(60));

        // Assert
        var logOutput = logger.Output.ToString();
        Assert.DoesNotContain("Exception", logOutput);
    }

Let me know if you need any additional information.
Regards,
Leo.

Published 2.15.1

Thank you!


Expecting close brace } or a comma,

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
ftp 密码中包含特殊符号 导致校验失败  加了双引号还是密码错误
SeaTunnel Version
2.2 beta
SeaTunnel Config
"env" : {
        "execution.parallelism" : 1,
        "job.mode" : "BATCH"
    },
    "source" : [
        {
            "schema" : {
                "id" : "int",
                "name" : "string",
                "age" : "int"
            },
            "path" : "/data/tianxy",
            "password" : "P*=u^[fU7U28",
            "port" : 22,
            "delimiter" : ",",
            "host" : "<IP_ADDRESS>",
            "type" : "csv",
            "plugin_name" : "FtpFile",
            "user" : "root"
        }
    ],
    "transform" : [],
    "sink" : [
        {
            "plugin_name" : "Console"
        }
    ]
}

Running Command
bin/start-seatunnel-spark-connector-v2.sh --master local[4] --deploy-mode client --config config/ftp_hdfs.conf

Error Exception
Exception in thread "main" org.apache.seatunnel.shade.com.typesafe.config.ConfigException$Parse: config/ftp_hdfs.conf: 13: Expecting close brace } or a comma, got '=' (if you intended '=' to be part of a key or string value, try enclosing the key or value in double quotes, or you may be able to rename the file .properties rather than .conf)

org.apache.seatunnel.api.common.PrepareFailException: PluginName: FtpFile, PluginType: source, Message: Check file path fail.

Flink or Spark Version
No response
Java or Scala Version
No response
Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

wrong usage


fix #576

Ⅰ. Describe what this PR did
Fix #576
Ⅱ. Does this pull request fix one issue?
fixes #576
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
test on fescar-samples

enable <aop:aspectj-autoproxy />

create Aspect
@Aspect
public class MyAspect {

    @Before("execution(* com.alibaba.fescar.samples.dubbo.service.impl.BusinessServiceImpl.purchase(..))")
    public void before(){
    }
}

GlobalTransactionScanner can correctly detect @GlobalTransactional
Ⅴ. Special notes for reviews

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #584 into develop will decrease coverage by 0.01%.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             develop     #584      +/-   ##
=============================================
- Coverage      31.93%   31.91%   -0.02%
  Complexity       770      770
=============================================
  Files            200      200
  Lines           7917     7920       +3
  Branches         938      938
=============================================
  Hits            2528     2528
- Misses          5129     5132       +3
  Partials         260      260

Impacted Files
Coverage Δ
Complexity Δ

...ing/annotation/GlobalTransactionalInterceptor.java
12.06% <0%> (-0.66%)
2 <0> (ø)

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 34bef93...f3b44bd. Read the comment docs.


Chart Generation Taking a Long Time (or not happening at all)

Describe the bug
Generating charts for my inbox seem to take a very long time (I have yet to see the charts actually display). Have waited for charts to appear, assuming it is because of my large inbox. But have waited for likely an hour with the tab open at this point and still no charts. The spinner is spinning so I assume something is happening, but I'm not sure if I should continue to wait or if it's a bug,
To Reproduce

I click on the button to display the stats dashboard.
The top-line stats display, such as: Mails total, Mails unread, Mails received, etc
None of the charts display information. Just black where the charts should show up, the titles however are there.

Expected behavior
Expected the charts to appear and update as more information was processed, or the charts to show up faster.
Screenshots

System (please complete the following information):

OS: elementaryOS 5.1.7
Thunderbird Version 78.3.1 (64-bit)

This definitely is a bug. I have an inbox with ~30k emails which takes 13s to generate the charts. This is still quite long, so performance should definitely be improved, but never finishing chart generation is a bug.
Do you have multiple email accounts? Does the popup loading spinner finish (before clicking your account and opening the stats page)?

Just one account.
The spinner does not seem to stop if I let it sit before opening the page (in the pop over menu).

Ok, thank you for your feedback! I'll try to reproduce this and come back as soon as I have a solution.

I can look at the console or anywhere you think there might be useful information.

I set up a virtual environment with elementaryOS 5.1.7 and Thunderbird Version 78.3.1 (64-bit) and wasn't able to reproduce your issue. So I assume it must be something with your emails or folder structure.
Can you please do the following:

open the console (add-ons > gear > debug add-ons > scroll to ThirdStats > Inspect)
clear console (small trash icon on the upper left)
open (or right click > reload if its already open) the Stats tab
make a screenshot or copy the (hopefully existing) output and paste it here

Thank you!

Plus:
I just release v0.4.0 with improved loading indication (you shouldn't see empty charts anymore now). Maybe this changes anything for you, @ryanleesipes?

Here is the screenshot you requested.

I do like the new indicator:

I created a possible fix. To test it, please do the following:

Clone this repo git clone https://github.com/devmount/third-stats
Change into the created directory cd third-stats
Switch branch git checkout fix-infinite-loading
Open Thunderbird > main menu > add-ons > gear menu > debug add-ons > load temporary add-on > choose the manifest file in the cloned repo's directory
Check, if charts are loaded now. If not, please create a screenshot of the console again.

I'm not sure if you have to uninstall ThirdStats first before taking the steps above (maybe its better). Thank you very much @ryanleesipes  for taking the time to test and debug this issue.

So two things to note (one might be the issue).
I installed Thunderbird on this machine via Flatpak so I wonder if that is causing issues. When attempting to load the temporary add-on I get the following message:

Extension is invalid
Loading locale file _locales/en/messages.json: Error: Error while loading 'file:///run/user/1000/doc/c28881a6/_locales/en/messages.json' (NS_ERROR_FILE_NOT_FOUND)

This could be because Flatpaks have their own filesystem, I'm troubleshooting.

Interesting, is that file actually existing on that location? Looks like the root directory of ThirdStats was not properly recognized... I don't have any experience with Flatpak, so thank you for troubleshooting!

I'm seeing the same issue on:

Windows: 10 Pro x64 (2004, 19041.572)
Thunderbird: 82.0b3 (64-bit)
Third-Stats: 0.5.0

Additional symptoms to report:

The processing hangs after the same number of messages are parsed. In my case: 82,562 (now 82,564 after 2 new messages are received)
I see the same errors in debug addons console output as @ryanleesipes

However, when I view the standard console output (not via debug addons), I see some additional errors that dont appear to show up during an 'addon debug' context. This is with the full-page third-stats loading, not just stats via the menu icon.
console-export-2020-10-20_21-33-0.txt
Additional information. Unsure if relevent or related:

The processing (count) seems to restart at zero each time the toolbar icon is opened -> closed -> opened (may be intended)

Thank you @koobs for this detailed description and error analysis! 👏🏻 This definitely helps. Have you checked the possible fix I created? Does it change anything?
Thanks to your error log I can do some additional test and see if I can find what's causing this issue.

The processing (count) seems to restart at zero each time the toolbar icon is opened -> closed -> opened (may be intended)

A click of the toolbar icon always reloads the popup content, so the counts are processed again - I can see if it's possible to implement a storage for already processed email/folder counts. If this is a feature you would like, you can create a feature request for that.

I use TB 78.4 on MacOS Catalina 10.15.7 … and experiencing the exact same behaviors. You’re seeing this on 3 OS’s , so it does not seem OS-related.

Thank you @tonewarper for confirming this issue. Unfortunately I cannot reproduce this on my own machines, so I'm waiting for someone to test the possible fix. I can provide the fix as XPI file, if that helps testing it.

Next version of ThirdStats contains a possible (!) fix, let's see if this issue persists.

@tonewarper @koobs @ryanleesipes The possible fix is now public in the Thunderbird add-on repository. Please update ThirdStats to v0.7.0 and let me know, if you still have this issue. Thank you all for testing and improving this add-on!

@devmount Success! Thank you 👍 Now we can focus on overwhelming you with feature requests ;D
Which commit was the likely fix for the issue?

Great! I'm very happy that it works for you now 👏🏻

Now we can focus on overwhelming you with feature requests ;D

Go ahead, I'm looking forward to your ideas 💡👍🏻😅
Yes, it was 97506847a23cd20c9091068f124af0046ada24e7

I am the bearer of unpleasant news… on my end, this very-same behavior still persists on v0.70 , even after restarting TB twice. Perhaps it worked for @koobs on Windows while I’m on MacOS? Or maybe there is another variable at play. Most of my mails are fetched via IMAP on Gmail servers.  It’s peculiar that Thunderbird Stats is able to produce a count of the mails but doesn’t generate the charting. The counter counts up while calculating, and at some point the counting stops while the wheel keeps spinning and the ‘Fetching’ loader remains where the charts should be… forever.

@tonewarper Sorry that it didn't work for you. Can you share your console output (as described here)?

So for @koobs is the fix working, for @tonewarper not. How about you @ryanleesipes? Did 0.7.0 fix this issue for you?
@tonewarper Did you have some time to look into the console?

Hi,
Same issue here:
TB 78.4.3 (64b) / W10
It works great with my local folder...
but fail with my main account:

I hope it helps...

Hi @6Sim9, thank you for these valuable additional information. It still looks like an issue with the folder processing, so I implemented an error handler to provide more information and gracefully continue processing in case of failure (d3203fd09fe6d5fea76d4fd9088eceed741641db). It will be part of the next release v0.8.0 - let's hope that it works 🙏🏻 (I still can't reproduce this issue on any of my machines)

ThirdStats v0.8.0 is public. @6Sim9 @tonewarper: Please upgrade your ThirdStats add-on and check if the problem still persists. Have a nice weekend.

Updated and restarted, bug still alive. There are several CallResult for closed conduit Child errors when I switch to the add-on but I think here is when — yes — I also experience folder processing issues:

Sorry to hear that. I'll further investigate this issue and will let you know when I have a solution.

Hi!
Same here :'(

I've think about something... Maybe it's a clue.
My mails are stored on a different drive letter(D:) than TB (C:).
Have a nice week :)

Thank you for confirming this issue. Before I make another guess of a solution, we need to debug this issue on your end, to really find the cause of the problem.
@6Sim9 and @tonewarper Can you please do the following:

Open Thunderbird
Open the ThirdStats stats page and wait ca. 1min
Open the console (main menu > add-ons > gear > debug add-ons > scroll to ThirdStats > Inspect)
Clear console (small trash icon on the upper left)
Past the following snippet and hit the Enter key:let accounts = await messenger.accounts.list(); let allfolders = [];
accounts.map(a => { function traverse(folders) { if (!folders) return; for (let f of folders) { allfolders.push(f); traverse(f.subFolders); }} traverse(a.folders);})
console.log(allfolders);

This should print a list of all folders of all accounts or - most probably in your case - a hopefully helpful error message.

Hi!
222 folders found but no error.

Interesting. If you expand the folder list, are all folder entries valid or are there folders without a name or without a path?

All seems to be OK.
Here is an exemple:

Ok, now that we know, that the folder list is correct, let's take it a step further and count all messages of all folders of all accounts. Please enter the following, if you have the console still open:
let count = 0;
await Promise.all(allfolders.map(async f => { let page = await messenger.messages.list(f); page.messages.map(m => count++); while (page.id) { page = await messenger.messages.continueList(page.id); page.messages.map(m => count++); } })).then(() => { console.log(count); });

If you have a new console, here is the full code snippet:
let accounts = await messenger.accounts.list(); let allfolders = [];
accounts.map(a => { function traverse(folders) { if (!folders) return; for (let f of folders) { allfolders.push(f); traverse(f.subFolders); }} traverse(a.folders);})
let count = 0;
await Promise.all(allfolders.map(async f => { let page = await messenger.messages.list(f); page.messages.map(m => count++); while (page.id) { page = await messenger.messages.continueList(page.id); page.messages.map(m => count++); } })).then(() => { console.log(count); });

The output can take some time, according to the number of messages you have. Please let me know, if you now have any error messages.

I followed the same routine you asked of @6Sim9 . Message counts not working -
folder is null ext-messages.js:189     list chrome://messenger/content/parent/ext-messages.js:189     list self-hosted:844     result resource://gre/modules/ExtensionParent.jsm:996     withPendingBrowser resource://gre/modules/ExtensionParent.jsm:602     result resource://gre/modules/ExtensionParent.jsm:996     callAndLog resource://gre/modules/ExtensionParent.jsm:958     recvAPICall resource://gre/modules/ExtensionParent.jsm:995     AsyncFunctionNext self-hosted:693 Uncaught (in promise) Error: An unexpected error occurred
Folder name/path fetching worked without any errors, same as @6Sim9 . Out of 65 folders for me, all look quite normal except for one: ( "?", path: "/", … } ) which is how it appears in TB, perhaps a TB bug in fetching that foldername from Gmail with IMAP or not. The rest look quite ordinary, just some have [ ] or spaces or @ characters which I don’t think is a problem since they’re inside the “ “. .

Thank you @tonewarper for these information. The code snippet above basically uses the same method given as example in the docs: https://thunderbird-webextensions.readthedocs.io/en/78/how-to/messageLists.html
Due to resource management message lists are splitted into page which you have to iterate over to get all messages of a folder. So seeing how this error appears, I assume that it's caused by a bad combination of OS file handling, Gmail folders and the way, Thunderbird handles that. As I don't see a way to solve this in ThirdStats, I issued a corresponding topic at Thunderbirds topicbox.
Let's hope, that somebody knows a solution 🙏🏻

Hi!
Here are my results:

I don't have any IMAP nor Gmail account.

Thank you @6Sim9 for your results and these additional information. So when you're not using IMAP, you use good ol' POP3? I see if someone at Thunderbirds topicbox can help us here.

Hi there, I'm currently not able to reproduce it (yet), but I have found the problem in the Thunderbird source code where the error occurs and will investigate further. As far as I can see there's no bug ticket filed on Bugzilla, so I collect more information and file one.
The main problem so far is to find a reliable scenario so that one is able to test and reproduce the error and to ensure that it definitively was fixed. To find this scenario it would be helpful to have some more information at hand, e.g. what type of accounts are set up, what folders are available in this account and maybe some more details about the server side like if there any shared IMAP folders, and the profile itself, e.g. are there any disabled accounts or lately deleted folders.
@6Sim9 & @tonewarper are you available for giving more insights about your setup and/or executing some more scripts and share the information with me? It might collect personal information like account and folder names, so this thread is may not be the perfect match to share this sensitive information.
I'll keep you updated.

Thank you @arndissler for investigating this issue further 👏🏻

It looks like it could be caused by this bug:
https://bugzilla.mozilla.org/show_bug.cgi?id=1529791
@6Sim9 & @tonewarper can you somehow determine if you use virtual folders?

@devmount For what its worth, I'm running (successfully, without errors) ThirdStats on a Gmail backed account too and not running any special settings or folder subscribe settings. I mention this as "[Gmail]" is mentioned in the the bug mentioned in the previous comment. See screenshots below:

You can also see a "MFH Pending" folder (a 'Saved Search'), which might also be classed a virtual folder (i'm not sure)

@koobs thank you. Another user just confirmed, that he is successfully using ThirdStats with virtual folders, so that might not be the problem here. I'll take a look what other folder types exist and if there is any type currently not supported by the webextension API.

Is anyone who's facing this issue here up for testing a "daily build" of Thunderbird? The testing procedure will be as follows:

clone (duplicate) your Thunderbird profile where the error occurs
download a specific Daily build of Thunderbird
extract the ZIP (Windows) or open the DMG (macOS)
start the Thunderbird from the ZIP/DMG via use the cloned profile from step 1., so Thunderbird should be started from the command line with the "profile" option set
open the Debugger as described in this comment, step 1. to 4.
run the scripts which are provided by devmount in this comment
there should be still errors in the console, but they should mention the exact folder, e.g. Error listing messages in folder 'your-foldername-here' in account 'account-name'

The Daily build containing the patch are available for Windows 32bit, Windows 64bit and macOS
To start Thunderbird Daily with the cloned profile, run Thunderbird.exe --profile C:\path\to\profile on windows or /Volumes/Thunderbird\ Daily/Thunderbird\ Daily.app/Contents/MacOS/thunderbird --profile /path/to/profile on macOS, ensure to modify the pathnames according to your local system.

@arndissler Probably not the best place for this, but relevent nonetheless given the request. If I (and users) had control over Telemetry (the pref is locked True on beta/nightlies), we'd use nightly (and I use to). I'd be more than happy and in fact, I'd prefer, to run nightly again, particularly to help support extension developers ramp up after the mass webextensions exodus

I’m not sure if I use virtual folders, but what I do know for sure is that I’ve configured my IMAP folders in TB to sync all Gmails and store on my hard drive too for faster search-speed, so I have a massive profile size (I still use PST, not Mbox, if that makes a difference). I can also now confirm that the “?” folder path I mentioned earlier was indeed not a TB bug implicated in the null result… it was relabelled that way inside the Gmail interface.
While I’d love to dedicate to troubleshooting and deep-testing, what I’m able to commit to is occasional quick validation/verification over the long haul. I’ll keep checking in on this project! I’m happy that at least you’ve gotten to some degree of pinpointing the problem for some users like me, and hope that in time more users experiencing it will chime in here to get us over to the promised land.

Four your information: it seems like the fix for this behaviour will be available with Thunderbird 85.

Thank you Arnd 👏🏻

Hi!
I don't know if it's help.
I copied my email folder from d: to c: and created a new account with this local directory.
And it works!

Initial account still doesn't work...

I copied my email folder from d: to c: and created a new account with this local directory.

This confirms, that it's a problem with IMAP folders. I guess that converting the IMAP account to a local account creates "real" folders only which the WebExtensions API can handle properly. We will wait for Thunderbird 85 to be released to test this issue again.

OK ... I'm sorry ... I realize I didn't respond when you asked me about IMAP or POP.
My server setting is "POP mail server".

No problem. It seems to be the same for POP mail server. There are special folder types on your mail server Thunderbird can't handle properly. Let's hope, that TB85 gets released soon.

Hello everyone,
I am happy to announce that Thunderbird 91 is now available as a beta (direct download). It will be only a month now, that Thunderbird 91 becomes stable and gets released.
So if anyone still affected by this (@ryanleesipes @tonewarper @6Sim9 ?) already wants to test it, please go ahead. Otherwise I hope you can confirm the fix in a month at latest, when Thunderbird 91 gets released, so we can close this issue. Finally! 😅

It worked! And it works on 78 now, great job @devmount!

Great news! Glad to hear that it now works on 78 too 👏🏻

Now that Thunderbird 91 got released, I can finally close this 👏🏻  If some of you still have issues with the generation of the stats data, please open a new issue.


Provide abstraction interface to save/load arbitrary files as an Item

Possibly as an encoded data: field.
The encoding probably depends on the file type.
Encoding should also be recorded alongside the payload.
Something like:
id: "... item id..."
data:
  type: file
  encoding: "..."  # none | base32 | base64 | ...
  payload: |
    ...
status: valid
dateAccepted: ...

Not really needed.  Closing.


Denormalized tables types and type_names in .chiseld.db

The database .chiseld.db contains two tables, types and type_names, that contain a row for every user-defined ChiselEntity:
CREATE TABLE `types` (
    `type_id` integer PRIMARY KEY AUTOINCREMENT,
    `backing_table` text UNIQUE,
    `api_version` text UNIQUE
);
CREATE TABLE `type_names` (
    `type_id` integer,
    `name` text UNIQUE,
    FOREIGN KEY (`type_id`) REFERENCES `types` (`type_id`) ON DELETE CASCADE
);

with data that look like:
sqlite> select * from types;
1|ty_Book_DB9DFA65892B4E02805E2FBC012FD911|
2|ty_Book_90AB39185CE34CE99891366B8AD5F4D7|
3|ty_Author_49FB41771CE04E908A6E1EE353066980|
4|ty_Author_83BE57E2356141F0AA021A3FE5D5A86D|
5|ty_Human_D665CCAFB4BD4260972AA6CE2C7730BD|

sqlite> select * from type_names;
1|dev.Book
2|main.Book
3|dev.Author
4|main.Author
5|dev.Human

We should probably normalize this into one table by dropping type_names and adding columns name and api_version to types. We should not be parsing strings from type_names.name to get the name and the api version.
This is related to #1601

the reason we parse strings is that we want combinations of columns, and not columns, to be unique.
Field names have to be unique within a model, but not globally unique. sqlite doesn't support a group of columns being unique, so we resorted to that.

I think that SQLite does support unique index with multiple columns:
https://www.sqlite.org/lang_createindex.html
sqlite> create table types (name text, api_version text);
sqlite> create unique index i on types (name, api_version);
sqlite> insert into types (name, api_version) values ('Book', 'dev');
sqlite> insert into types (name, api_version) values ('Book', 'dev');
Error: UNIQUE constraint failed: types.name, types.api_version

Or we could simply use (api_version, name) as a primary key in the types table, so that we don't have to keep track of type ids in chiseld.


[v9.0.0-rc.3] TypeScript syntax error when bundling for React Native

Hi, the syntax for typescript incorrect in<EMAIL_ADDRESS>at ReactNative environment.
The error report is below:

if I change to
export { AnimationResult } from './AnimationResult'
export { UnknownProps } from './types/common'

It works.
What should i do or how can i resolve it?
Originally posted by @moonrailgun in https://github.com/react-spring/react-spring/issues/985#issuecomment-634048743

Maybe the problem is old typescript version? This syntax is introduced in one of the latest versions, right?

The TypeScript compiler in Metro is outdated. It needs this commit for export type syntax to work, which was released in<EMAIL_ADDRESS>https://github.com/facebook/metro/blob/master/packages/metro-react-native-babel-preset/package.json#L55
Please open an issue/PR over there. 👍

Actually, if you regenerate your lockfile, the new @babel/plugin-transform-typescript should be installed.

This problem still alive.
Although i install @babel/plugin-transform-typescript@latest by myself .

Checked node_modules folder and ensure its @babel/plugin-transform-typescript is 7.9.6.
Checked node_modules/metro-react-native-babel-preset to ensure its have not own node_modules
Even print<EMAIL_ADDRESS>in node_modules\metro-react-native-babel-preset\src\configs\main.js ensure require right module and check the version of it is correct.

What else I forget to do?
Can you help me to resolve it? Thanks for a lot.

Try --reset-cache when starting the react-native dev server.

Oh i resolve it. Its looks like alse dependent on other core package. Just upgrade @babel/core and @babel/runtime. And everything looks well
https://github.com/facebook/metro/issues/537#issuecomment-630190299


Add documentation on how to create identifiers

closes #54
@opoudjis is it enough or you need more details on every or specific attributes?

Not adequate; this is after all meant to be used by not just me. I need:

an indication of every attribute whose type is not a string; I am assuming a few of these are arrays and can take multiple values; joint_document is presumably boolean
an explanation of copublisher and when to use it instead of publisher
why are amendments, corrigendums etc plural? Are they expected to be an array? I have trouble seeing why they would be
what is the distinction between supplement and supplements?
for every attribute with a controlled vocabulary, I need to know the legal values: I cannot trust that you and I understand the same enums, and they need to be documented in any case
what is dirtype?

@ronaldtse @opoudjis what do you think about this? Is something missing?

Ok, that's a lot cleaner, thank you. I'd have preferred prose, but I can use this.

Thank you @mico @opoudjis !


LUCENE-10285: try to force ordering of internal tasks, in spite of ma…

@rmuir Can you try on your machine? I've ran
gradlew regenerate --rerun-tasks
on a linux box but I suspect the graph ordering may depend on the number of cores, wind patterns, ectoplasm surrounding the CPU and maybe other factors.

@dweiss running now. I'll run regenerate twice and see if i encounter similar behavior.

I can already see the difference in execution order. in the main branch i was able to trivially reproduce the problem. with this patch it runs the TLD stuff first:
> Task :lucene:analysis:common:generateTldsInternal
Found 1489 TLDs in IANA TLD Database at https://data.iana.org/TLD/tlds-alpha-by-domain.txt
                      ASCIITLD: 1370 TLDs
    ASCIITLDprefix_1CharSuffix:  108 TLDs
    ASCIITLDprefix_2CharSuffix:   11 TLDs
                         Total: 1489 TLDs
Generated TLD content identical as before, not updating.

> Task :lucene:analysis:common:generateUAX29URLEmailTokenizerInternal
...

I do believe it should be all right, although there may be other quirks lurking in there. The thing is: when you have this task graph:
A.dependsOn B
C.dependsOn D
then when you say:
A.dependsOn C
this only means C will run prior to A but says nothing about relative order of B and D. The ordering can be executed like this: B, D, C, A and it's valid.


Add cloudwatchevents service prow jobs and labels

Add cloudwatchevents service prow jobs and labels
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Cross compile the AVS Device SDK for iOS

Briefly summarize your issue:
Cross compile the AVS Device SDK for iOS，I've compiled it，But I don't know how to use it，Because it doesn't have some .h files.
Tell us about your OS (Type & version):

[ ] Linux
[x] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ √] Other - tell us more:

this is my cross compiled files:
https://pan.baidu.com/s/1f7DxR2MzL3mwZGnHNDXAVg

Hi @kclchan  I don't how to use this Lib file, because add that file to Xcode project ，I can't find lib head file.

Hi @179992096, sorry for not replying sooner.  The header files are in the folder where you put the SDK source code.  I think the first header file you should look at is:
ApplicationUtilities/DefaultClient/include/DefaultClient/DefaultClient.h
Also, you can use "make install" to install the headers and libraries files to your specified location.  Please rerun CMAKE with this option to specify your location:
-DCMAKE_INSTALL_PREFIX=<location to where you want to install the files>
Hope this helps!
Thanks,
Kenneth


Simplify code paths within PARSER_DUMP_BYTE_CODE

Also fix an uncovered issue in the reporting of switching to strict
mode.
JerryScript-DCO-1.0-Signed-off-by: Akos Kiss<EMAIL_ADDRESS>

PR rebased and updated according to the review.

PR updated.

Code rewritten to follow the suggested style for ?:. PR rebased and updated.

LGTM


[lit-starter-js] [lit-starter-ts] Update starter template READMEs

Change github pages branch name from "master" to "main"
Change npm run docs:watch to npm run docs:gen:watch

@augustjk Done - that look ok?

Awesome, thanks!


[xh]Qdrant integration for data loader and exporter

Description

Add new data loader template and explorter template for Qdrant.
Add io/qdrant.py script to support the load and exporter function.

How Has This Been Tested?
Mage UI Loader:

Exporter, number of reords written by exporter:

With auto generation vector list to query:

Checklist

[X] The PR is tagged with proper labels (bug, enhancement, feature, documentation)
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] If new documentation has been added, relative paths have been added to the appropriate section of docs/mint.json

cc:
@wangxiaoyou1993

@wangxiaoyou1993 I followed our discussion to add SentenceTransformer 'all-MiniLM-L6-v2' as default embedding function in both loader and exporter template so customer just can query with the text. PTAL. Thank you!

Addressed the comments. Let me know your thoughts! @wangxiaoyou1993

Also send to David @dy46 for review.

Thank you for the review David! Addressed comments. PTAL.