Improvements to connector list page

Hook up Delete buttons
Re-think task status

Sure, we can rework the status.  I'm still unclear on the best approach there, so let me know if you have any ideas.  We talked about combining overall/task status into a single status - does that make the most sense?

It kinda would for Postgres, but we'd lock ourselves into a corner
considering MongoDB does support multiple tasks. How about having the
same RUNNING or FAILED badge as for the connector for each of the task(s),
and if you hover over it, you'd e.g. get the task id?

@gunnarmorling  I changed it so each indicate "Task x: status" - is that any better?  I think the big confusion point for me is how can the 'overall' state be 'RUNNING' when the only task is 'FAILED' - does that make sense?

Yes, that's definitely a good improvement.
Re the overall status, yeah, I see where you're coming from. Maybe consider
the connector as some kind of runtime environment, which is in charge of
spinning up task(s). E.g. it could have multiple tasks in theory (not in
the case of most DBZ connectors), out of which some might be failing and
some running. In that light the RUNNING connector status seems reasonable,
e.g. it could try and restart a failed task.

Would an overall status of "AVAILABLE" or something like that make sense - instead of RUNNING?

No, let's better not do that, as it'd be confusing for folks familiar with
the terminology actually used in Kafka Connect.

Changed title, since the tasks display issue has been addressed.  Hooking up delete button is dependent upon Issue #151

@mdrillin @indraraj @uidoyen the delete connector endpoint is in master now. See #151with PR #193
The request is a HTTP DELETE request to /connectors/{cluster}/{connector-name}.
The request just proxies the returned value for Kafka Connect, which means 404 when the connector name is wrong, and as before 500 and 503 when there were errors with Kafka Connect URI config or connection to Kafka Connect.

Raised PR https://github.com/debezium/debezium-ui-poc/pull/194

pr merged


Bot gets stuck in Powder Snow

Steps to Reproduce
Spawn in a snowy mountain and the bot may fall into powder snow.
Expected Behavior
Bot should escape by going upwards to the surface.
Actual Behavior
Bot reproduces the same type of jump as if the bot was on water, resulting in constant death if human inputs arent pressed.

Probably need to merge upstream baritone

@adrisj7 As I recall from a conversation with wagyourtail, the snow thing is a baritone problem that they were working on. Do you know if this was finished? Did we ever merge this version of baritone into Altoclef?

Fixed


NPM homepage github link is a 404

Currently:

The Github link leads to a 404. Manually adding "-nodejs" to the URL let me find the repo, but the link is currently borked. Your package.json has the right link, so you might just need to update with NPM.

:+1:

Hey @TessMyers @MetaMemoryT
I spoke to Shane, and he seems underwater at the minute, but gave me access to create a new npm release of bcrypt.
Do you (or anyone else) want to rally some peeps together to get a new release out? Maybe something incremental? I don't want to break anyone, but it would be nice to start pushing out some things.

publishing with a bugfix version bump would be nice, and only takes a moment.
any other development necessary?

I'd say releasing a fix to point npm to correct github repo is important; people can't find the source otherwise (took me some time to find this repo).

+1

This is still broken and needs to be fixed.
The link on npm is https://github.com/shaneGirish/bcryptJS
when it needs to be https://www.npmjs.com/package/bcrypt-nodejs

@shaneGirish it looks like the repo link is still out of date

@joshball @shaneGirish ping


Add 'get_status' to datalake file client

This implements get_status for the datalake file client, as mentioned in #496.

Ping...  Any thoughts yet on this PR?  It's really pretty lightweight/simple.  Most of the files in this PR are due to the new test (didn't exist before) and all the transaction files needed to pass CI checks.


testing mermaid in rm file

mermaid layout test
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

auto merge - testing mermaid for readme


Control center settings tab

New settings tab in control center data Hub administration page and default discoverability status setting

Thanks @BananiMendix is this in production already? Is it GA?

Thanks @BananiMendix is this in production already? Is it GA?

Not in production yet. We are planning to deploy it this week.

@BananiMendix when this week, and will there be release notes for this?

@BananiMendix when this week, and will there be release notes for this?

@Adam-Dupaski - HI Adam , we will postpone this deployment to next week(2 November,2021) , yes there will be release note

Superseded by https://github.com/mendix/docs/pull/3927


Light controlled via mqtt and switch at same time

Marvin, thank you very much for your great work. I have a noob question: I am trying to make a light controlled via mqtt (home-assistant.io) and a push button/switch at the same time (I am using a sonoff). The code below works alright, the only thing is that when I use the switch to turn on/off the light that was previously operated via the mqtt control, nothing happens and I need to push it again to make it work, and viceversa if I use the mqtt interface to operate the light previously turned on/off with the switch.
I am definitely making something wrong but cannot see what.
Thanks in advance for your help.
#include <Homie.h>

const int PIN_RELAY = 12;
const int PIN_LED = 13;
const int PIN_BUTTON = 0;

Bounce debouncerButton = Bounce();
int lastButtonValue = -1;

HomieNode switchNode("switch", "switch");

void buttonPressHandler(){
  int buttonValue = debouncerButton.read();
  int relayValue = digitalRead(PIN_RELAY);

  if (buttonValue != lastButtonValue){ //if button state has changed
    lastButtonValue = buttonValue;

    if (buttonValue == HIGH){ //only toggle relay if the new button state is HIGH
      digitalWrite(PIN_RELAY, !relayValue);
      Homie.setNodeProperty(switchNode, "on", relayValue ? "true" : "false");
    }
  }
}

bool switchOnHandler(String value) {
  if (value == "true") {
    digitalWrite(PIN_RELAY, HIGH);
    Homie.setNodeProperty(switchNode, "on", "true");
    Serial.println("Switch is on");
   } else if (value == "false") {
    digitalWrite(PIN_RELAY, LOW);
    Homie.setNodeProperty(switchNode, "on", "false");
    Serial.println("Switch is off");
   } else {
    return false;
   }
   return true;
}

void setup() {
  pinMode(PIN_RELAY, OUTPUT);
  digitalWrite(PIN_RELAY, LOW);
  pinMode(PIN_BUTTON, INPUT);
  digitalWrite(PIN_BUTTON, HIGH);
  debouncerButton.attach(PIN_BUTTON);
  debouncerButton.interval(50);

  Homie.setFirmware("itead-sonoff", "1.0.0");
  Homie.setLedPin(PIN_LED, LOW);
  Homie.setResetTrigger(PIN_BUTTON, LOW, 10000);
  switchNode.subscribe("on", switchOnHandler);
  Homie.registerNode(switchNode);
  Homie.setLoopFunction(buttonPressHandler);
  Homie.setup();
}

void loop() {
  Homie.loop();
  debouncerButton.update();
}

To be honest, I don't see anything wrong in this code... I'll test when I'll have time to. :)

Thanks Marvin, that is much appreciated. I will keep debugging and will post an update if I can get it to work as expected.
Keep up the good work. This framework is awesome!

Marvin,
after some debugging I have sorted things out: when the button is pressed, the latest value of variable holding relay state is reversed. The correct code looks like this:
if (buttonValue == HIGH){ //only toggle relay if the new button state is HIGH
      digitalWrite(PIN_RELAY, !relayValue);
      Homie.setNodeProperty(switchNode, "on", !relayValue ? "true" : "false");
    }

I still have one issue to fix: I was expecting the push button/switch to work even with no internet connection and/or without connection to the mqtt broker. However the node must be connected to the broker for the button to toggle the relay. Am I missing something?
Thanks

@plosi Great!
The loop handler is only called when the connection to Wi-Fi/MQTT is up. But in your case you have a button so indeed, the loop handler is not the best place to do this.
Cut/paste the code from the loop handler to the main Arduino loop(), which is called every time.

@plosi can you post your completed code once you have it working nicely?

I will for sure but I'm off on vacation and will return in a couple of weeks. Will post it as soon as I am back.

@TommySharpNZ
I just uploaded a version based on the work of @plosi
On my version the button is also working offline.
https://github.com/yves-ledermann/arduino_esp8266_homie_sonoff
maybe there is a better aproach to do this... lets see how @plosi code looks..

@TommySharpNZ  sorry for the delay. I just followed @marvinroger 's advice and moved the button function from the loop handler to the main Arduino loop(). The code updated looks as follows and it works nicely:
#include <Arduino.h>
#include <Homie.h>

const int PIN_RELAY = 12;
const int PIN_LED = 13;
const int PIN_BUTTON = 0;

Bounce debouncerButton = Bounce();
int lastButtonValue = -1;

HomieNode switchNode("switch", "switch");

#define FW_NAME "itead-sonoff"
#define FW_VERSION "1.0.1"

/* Magic sequence for Autodetectable Binary Upload */
const char *__FLAGGED_FW_NAME = "\xbf\x84\xe4\x13\x54" FW_NAME "\x93\x44\x6b\xa7\x75";
const char *__FLAGGED_FW_VERSION = "\x6a\x3f\x3e\x0e\xe1" FW_VERSION "\xb0\x30\x48\xd4\x1a";
/* End of magic sequence for Autodetectable Binary Upload */

void buttonPressHandler(){
  int buttonValue = debouncerButton.read();
  int relayValue = digitalRead(PIN_RELAY);

  if (buttonValue != lastButtonValue){ //if button state has changed
    lastButtonValue = buttonValue;

    if (buttonValue == HIGH){ //only toggle relay if the new button state is HIGH
      digitalWrite(PIN_RELAY, !relayValue);
      Homie.setNodeProperty(switchNode, "on", !relayValue ? "true" : "false", true);
    }
  }
}

bool switchOnHandler(String value) {
  if (value == "true") {
    digitalWrite(PIN_RELAY, HIGH);
    Homie.setNodeProperty(switchNode, "on", "true");
    Serial.println("Switch is on");
   } else if (value == "false") {
    digitalWrite(PIN_RELAY, LOW);
    Homie.setNodeProperty(switchNode, "on", "false");
    Serial.println("Switch is off");
   } else {
    return false;
   }
   return true;
}

void setup() {
  pinMode(PIN_RELAY, OUTPUT);
  digitalWrite(PIN_RELAY, LOW); //relay is off when pin is low
  pinMode(PIN_BUTTON, INPUT);
  digitalWrite(PIN_BUTTON, HIGH);
  debouncerButton.attach(PIN_BUTTON);
  debouncerButton.interval(50);

  Homie.setFirmware(FW_NAME, FW_VERSION);
  Homie.setLedPin(PIN_LED, LOW);
  Homie.setResetTrigger(PIN_BUTTON, LOW, 5000);
  switchNode.subscribe("on", switchOnHandler);
  Homie.registerNode(switchNode);
  //Homie.setLoopFunction(buttonPressHandler);
  Homie.setup();
}

void loop() {
  Homie.loop();
  buttonPressHandler();
  debouncerButton.update();
}

the solution proposed  by @yves-ledermann is very neat and nice indeed, I might give it a try. In the meantime, I would like to install the sonoff within a standard electrical wall box and see how it behaves. My only concern would be if it get too warm.
@marvinroger: I consider the issue/question resolved. Please feel free to tag it as closed or leave it open for further discussion on the matter. Thank you very much!

Awesome, thanks! Let's close this issue. Feel free to continue the discussion if you wish to, but this is not an actual issue anymore. :wink:


ignore packaging test fixtures

no idea why this didn't show up sooner, but git clean -xdf && pnpm i && pnpm build creates a bunch of untracked files that should be ignored

actually that's not true, it was only happening on #4710 because i borked up the rimraf implementation. carry on


Rewrite of the program

This is an near complete rewrite of the original dl-nhltv project by cmaxwe with
the main goal being improving maintainability, stability and fixing a number of bugs.
I found myself debugging a lot of issues in the original project but even after a few refactoring sessions the codebase was difficult to test
and maintain so I decided to do this rewrite reusing only some of the core logic.
Biggest differences are:

Upgraded to python 3, where the original was python 2
Unit test code coverage >90% compared to the previous ~30%
Added a linter (flake8) that now reports no issues
Added CI and automated Docker image builds

New features:

Added obfuscation of the video length, so that the final minutes in the 3rd period stay interesting even if you pause or otherwise glance at the remaining time of the video
Added progress bars to a number of long running tasks such as downloading and decoding
Added the ability to choose which stream to prefer (not forcing the local stream for the team, however that is still the default)

Major fixes:

Added error handling for a lot of cases that previously would pass silently and result in broken or no video
Fixed a few bugs responsible for a lot of intermittent download failures eliminating the need for retry functionality
Fixed a video corruption bug when there were streams with only audio, corrupting the entire video

Removed features:

Retry functionality is removed in favour of retrying with aria2, after removing alternate urls due to issues they caused - this seems to solve most issues
Mobile video recoding was removed, I'm not planning to add it back as I don't see much of a use for it but PRs are of course welcome

Features not implemented yet (that I will eventually still want to implement):

Housekeeping
Run with multiple teams

Also PRs are open and welcome, Github Actions is set up to run the same test and lint suite as is defined
in the Makefile - so you can use make locally to ensure your changes can be accepted.

Reimplement checkinterval


fix(52664): Assertion failure on completions for derived class with computed base property name

Fixes #52664

I'm not sure if this PR is still in progress, but @a-tarasyuk could you add some tests? There are a few things in this PR I couldn't fully understand, and I think tests could help me review it. Thanks.

I think the lack of tests comes from a merge conflict or something. There was a test previously at https://github.com/microsoft/TypeScript/pull/52673/commits/d0c5db8cebbe3bd36fe6712e75ca0076b8a63658#diff-8bc13ef2c947f0e85e6a6d61129663eb774aa9f40a694b2cf8f85a4757a75c4b

I think the lack of tests comes from a merge conflict or something. There was a test previously at d0c5db8#diff-8bc13ef2c947f0e85e6a6d61129663eb774aa9f40a694b2cf8f85a4757a75c4b

Ah, that explains it. @a-tarasyuk can you re-add the test then?


Add tkFileDialog to future.moves.tkinter.filedialog

TLDR: future.moves.tkinter.filedialog needs to import tkFileDialog

With the current version of future (0.18.2) installed, this works
from tkinter import filedialog
filedialog.askopenfilename()

However, this will fail
from future.moves.tkinter import filedialog
filedialog.askopenfilename()

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'module' object has no attribute 'askopenfilename'

The problem was patched in addressing #233 (commit https://github.com/PythonCharmers/python-future/commit/a6ed514a4b17cf03a46bcdb49f85f30f9eb42bcc), but the future.moves.tkinter.filedialog wasn't updated. In Python2, askopenfilename is part of tkFileDialog. Both FileDialog and tkFileDialog were combined to create tkinter.filedialog in Python3.

Thanks for contributing!


String Normalization

This change is

src/tokenizers.js, line 31 at r1 (raw file):
Previously, klappy (Christopher Klapp) wrote…
I feel like we need to optionally normalize. For example, Whitespace is sometimes required in the output to not be squashed, especially when needing to preserve line breaks/etc...
Big picture we will have both safe and destructive normalization have added characters or even strings added.
It will likely become some sort of map of what characters or regexes will map to what output.
In theory it will become more like the parsers option where it can be overridden and multiple defaults provided like safe and unsafe/destructive.

export const normalizations = [
  [ [/(\u200B)/g], '' ],
  [ [/\s+/g], ' ' ],
];

/**
 * Tokenize a string into an array of words
 * @param {Object} params - string to be tokenized
 * @return {Array} - array of tokenized words/strings
 */
export const tokenize = ({
  text='',
  includeWords=true,
  includeNumbers=true,
  includePunctuation=false,
  includeWhitespace=false,
  greedy=false,
  verbose=false,
  occurrences=false,
  parsers={word, whitespace, punctuation, number},
  normalize=false,
  normalizations=normalizations,
}) => {


Gift cards admin: create gift card(s)

Should be able to create gift cards

Set amount
Give email (optionnal)
Description / custom message (optionnal)
Explicit select of graphqPaymentMethod
Default currency = collective currency

On result page, show the created gift cards with links
Send notification to given email (new template)
Store email in PaymentMethod, data field
Should be limited to OpenSource hosts per default

To go in the way of https://github.com/opencollective/opencollective/issues/1451, the new component should take care of making the mutation and updating its own state.

@cuiki @znarf @xdamman The sketch for "Create gift card(s)"

Implemented by https://github.com/opencollective/opencollective-frontend/pull/1165


[Icon] - Icons in 2.4 are bolder than in Font Awesome 5

Steps
I am aware that the icon set was upgraded to Font Awesome 5 in latest releases, which made my application looks worse. I though it was due to the Font Awesome 5 redesign but then I tested both sets side by side and it seems that for some reason icons in Semantic UI (default theme) look quite different (uglier).
Expected Result
UI Semantic icons should look identical to Font Awesome 5 icons using the default theme.
Actual Result
Icons looks bolder
Version
2.4.2
Testcase
https://jsfiddle.net/tk17os4n/6

Can anyone please provide a link to the corresponding issue in Fomantic UI, in order one can fix the big in her local sUI setup manually?

@srissel This is the latest PR for FA 5.7 upgrade in Fomantic-UI
https://github.com/fomantic/Fomantic-UI/pull/446

Many thanks, @lubber-de !
So it seems it wasn't an issue with Semantic UI, but rather with FontAwesome 5.6?
For our current release, unfortunately we cannot upgrade to Fomantic UI yet. Therefore we need to incorporate this fix manually into our current sUI integration.

@srissel You can use our icon-script to help update your SUI builds 😉


Systemtest suite to ginkgo

We should not have a yaml file copy of a test in two different directories based on networkname. There should be only one copy of each test, e.g. the 8MB test input file. Tests could be grouped together if appropriate in a directory that is NOT based on networkname, or put in separate directories. The tester should be able to pick and choose to run whatever tests he wants, without looking in a networkname subdirectory. Can we run ginkgo while specifying a single or subset of test names?
Also: I may have missed it, but I did not see how to enable/disable the GinkgoWriter.

Suppose we are running the 8MB test in two different networks(one with tls and other with no tls). How can we use only one file for two different network configurations?

@suryalnvs @dongmingh Can you please review this?


moved sign code into separate directory

@TalhaAli00 I glanced over but gonna have next round when:

Conflicts are resolved
Just to avoid spam in comments, let's remove all android boilerplate code that is not necessarily used like all test files, icons etc.
Have a look at the ReadMe comment

@TalhaAli00 PR merged

Closed and started new branch. PR: https://github.com/WalletConnect/WalletConnectKotlinV2/pull/210


Running Ollama on mac but accessing through SSH only?

Can I run the app on an apple silicon based mac accessible via SSH only? After copying the installer out there, something like:
unzip Ollama-darwin.zip
mv Ollama.app /Applications/.
cd /Applications/.
chmod +x Ollama.app
open -n Ollama.app

but this gives no indication of changes, and when i subsequently run ollama list I get "zsh: command not found: ollama" (even with new shell, or login/out).
Is there a way to run it in this manner? Thanks!!

When starting Ollama.app, it prompts to create a symlink, but you can do that manually
sudo ln -s /Applications/Ollama.app/Contents/Resources/ollama /usr/local/bin/ollama

ollama list etc should work afterwards
Hope this helps!

Thank you. I had to sudo mkdir -p /usr/local/bin, but then that command worked perfectly. I'm up and running.
Is it possible for me to copy in and load models without an internet connection? If this is already documented, could you point me to it please? TIA.
Love the project, keep up the great work!

vim .zshrc
#ollama
export PATH=/usr/local/bin:$PATH
source .zshrc
enjoy 🤣


Adding new lemmas dite_some_none_eq_some and dite_some_none_eq_none

There are ite_some_none_eq_none and  dite_some_none_eq_some in Init.ByCases. But I would like to add the 'dite' versions to the same file, as proved below:
theorem dite_some_none_eq_none {P : Prop} [Decidable P] {α : Type*} {x : P → α} :
    (if h : P then some (x h) else none) = none ↔ ¬P := by
  simp only [dite_eq_right_iff]

@[simp] theorem dite_some_none_eq_some {P : Prop} [Decidable P] {α : Type*} {x : P → α} {y : α} :
    (if h : P then some (x h) else none) = some y ↔ ∃ h : P, x h = y := by
  split <;> simp_all

I've never contributed directly to this repository, so I thought I'd open an issue here to make sure this is the correct place to add the lemmas.

It seems reasonable to me to add these lemma if we have their counterparts for ite. Please do create a PR!

@nomeata Great! Do I create a branch on this repo (which I don't have access for) or will a fork also work?

@wupr Forks work (and are usually preferred).

Sorry, I missed your question!

No worries!


Added remaining entity type ids

Not that crazy. I wrote a quick lil program to generate them for me from the registry.json. So that I could copy paste them over. Maybe you're interested in that. We should probably use a proper registry for that stuff in the future tho.

Looks good. But sure there are that few entities ?. I tought mc had like 1000 of them. I also would like to have the generator somewhere in case of an new update

No, no, 129 sounds about right in terms of entities, not counting block entities or entity variants (e.g. cats, wolfs, llamas etc.). And there are 129 entries in the entity_type entries list as of now.
About the generator: It's a separate Rust project with about 20 lines of code. Didn't want to include it in the commit tho. If you want I can send it as a gist or whatever.

No, no, 129 sounds about right in terms of entities, not counting block entities or entity variants (e.g. cats, wolfs, llamas etc.). And there are 129 entries in the entity_type entries list as of now.
About the generator: It's a separate Rust project with about 20 lines of code. Didn't want to include it in the commit tho. If you want I can send it as a gist or whatever.

If would like to have this tool included in pumpkin somewhere (if you want)

Hey, Until CI is failing nothing will be merged. This is also the case for your other open PR


[#167617997] Add Redis Plans

What
This adds the following Redis plans so we have more HA options available to tenants.

tiny-ha-3.2 -> cache.t2.small
small-ha-3.2 -> cache.t2.medium

How to review

Code Review (confirm prices are correct)
Deploy to your dev env
Create, bind and delete a service using the new plans

Who can review
Not me

Deployed to my dev env and LGTM


DateRange picker - Total days selected

Feature request type
Enhance component
Component name
MudDateRangePicker
Is your feature request related to a problem?
No, this feature is not related to a problem.
Describe the solution you'd like
I want to be able and see the selected amount of days after selecting a date range.
Including a default set of periods is bonus!
Small preview image:

In this snippet, you can experience an example of what I would like to create:
Here is the example code in full (if the snippet does not work):
@page "/Test"
<MudDateRangePicker
	@ref="_picker"
	Color="Color.Success"
	Label="Start and end date"
	Required RequiredError="Provide a start and end date."
	DateRangeChanged="@DateRangeChanged"/>

		<MudChip Color="Color.Success" OnClick="@(() => SelectDays(10))"<EMAIL_ADDRESS>Disabled="@(_selectedDaysCount == 10)">10</MudChip>
		<MudChip Color="Color.Success" OnClick="@(() => SelectDays(30))"<EMAIL_ADDRESS>Disabled="@(_selectedDaysCount == 30)">30</MudChip>

		<MudChip Color="Color.Primary" CloseIcon="@Icons.Material.Filled.Today">@_selectedDaysCount days selected</MudChip>

@code{
	protected DateRange NewDateRange = new();
	private MudDateRangePicker _picker;
	private int _selectedDaysCount;

	public void SelectDays(int days)
	{
		_picker.Clear();
		_picker.DateRange = new DateRange(DateTime.Now, DateTime.Now.AddDays(days));
	}

	private void DateRangeChanged(DateRange range)
        => _selectedDaysCount = (int)(range?.End ?? DateTime.Now).Subtract(range?.Start ?? DateTime.Now).TotalDays;
}

I have not implemented this, but an option of boolean to turn this feature on/off would be nice also.
Have you seen this feature anywhere else?
I have not seen this anywhere else. It's an idea that would be quite useful for me and others as well...
Describe alternatives you've considered
No response
Pull Request

[X] I would like to do a Pull Request

Code of Conduct

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

@henon IMO This is a very specific requirement that can be achieved through composition or inheritance as shown in the issue.  I would suggest not to add API for this.

@GerritVanMol Showing this information below the picker is already possible as your snippet proves. I don't see why adding this to the date range picker would be necessary. Or are you asking for a way to add custom user interface into the picker when it is open?

It would be nice if it was built in, but if you think it is not necessary you can leave it for now.


Minor rendering issues

Thanks for this interesting plugin!  I am @fralau,  the maintainer of the mermaid2 plugin. I believe that the idea of using markdown to represent mindmaps is sensible, and that the concept appears more mature than Mermaid minmaps!
Since it worked out of the box 👍 , I threw the whole demo diagram from markmap.js.org at it.  It also worked immediately, so 👍  again!
Just a minor issue, the inline code is printed vertically:
```
- `inline code`
```

(I also appreciated that it works directly with pymdownx.superfences, out of the box).

As a side note, I tested the same diagram with Material, including with dark mode, and it doesn't give the same issue.

The code block is slightly squashed, but this is also a minor issue.
It also works perfectly with Mermaid2, which is a side bonus.
site_name: My Docs

theme:
  # name: readthedocs
  name: material
  language: en
  palette:
    scheme: slate
    primary: red
    accent: pink

markdown_extensions:
  - pymdownx.superfences:
        # make exceptions to highlighting of code:
      custom_fences:
        - name: mermaid
          class: mermaid
          format: !!python/name:mermaid2.fence_mermaid_custom

plugins:
  - search
  - markmap
  - mermaid2:
      version: '10.1.0'

It also works perfectly with Mermaid2, which is a side bonus.
site_name: My Docs

theme:
  # name: readthedocs
  name: material
  language: en
  palette:
    scheme: slate
    primary: red
    accent: pink

markdown_extensions:
  - pymdownx.superfences:
        # make exceptions to highlighting of code:
      custom_fences:
        - name: mermaid
          class: mermaid
          format: !!python/name:mermaid2.fence_mermaid_custom

plugins:
  - search
  - markmap
  - mermaid2:
      version: '10.1.0'

Here is the Mermaid code:
```mermaid
graph LR
    hello --> world
    world --> again
    again --> hello
```


Update README.md

Make sure set the target branch to develop
What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #8 (8964d55) into develop (259df11) will decrease coverage by 0.02%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@              Coverage Diff              @@
##             develop       #8      +/-   ##
=============================================
- Coverage      43.37%   43.36%   -0.02%
+ Complexity      8846     8840       -6
=============================================
  Files           1093     1093
  Lines          76843    76843
  Branches       10013    10013
=============================================
- Hits           33331    33322       -9
- Misses         39349    39356       +7
- Partials        4163     4165       +2

Impacted Files
Coverage Δ

...va/org/apache/rocketmq/store/FlushDiskWatcher.java
81.25% <0.00%> (-9.38%)
:arrow_down:

...org/apache/rocketmq/store/ha/WaitNotifyObject.java
80.35% <0.00%> (-5.36%)
:arrow_down:

...tmq/remoting/protocol/body/ConsumerConnection.java
95.83% <0.00%> (-4.17%)
:arrow_down:

...controller/impl/DefaultBrokerHeartbeatManager.java
82.27% <0.00%> (-3.80%)
:arrow_down:

.../org/apache/rocketmq/store/ha/DefaultHAClient.java
58.46% <0.00%> (-3.08%)
:arrow_down:

...he/rocketmq/client/trace/AsyncTraceDispatcher.java
82.21% <0.00%> (-1.45%)
:arrow_down:

...rocketmq/client/impl/factory/MQClientInstance.java
48.00% <0.00%> (-0.60%)
:arrow_down:

...main/java/org/apache/rocketmq/store/CommitLog.java
65.03% <0.00%> (-0.11%)
:arrow_down:

.../org/apache/rocketmq/proxy/config/ProxyConfig.java
44.65% <0.00%> (ø)

...e/rocketmq/namesrv/routeinfo/RouteInfoManager.java
63.75% <0.00%> (+0.32%)
:arrow_up:

... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Small logic change for goron mines

Add bomb arrows in logic and move ball and chain to nich stuff

I will close it until I get some feedback to add in logic


Fix baseUri Deprecation Error in monitor/opentelemetry

Packages impacted by this PR
@monitor/monitor-opentelemetry-exporter
Describe the problem that is addressed by this PR
Setting the baseUri value is throwing errors due to the deprecation of the value in the core-utils package.
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[x] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


How to build Clibsodium.framework as Universal framework?

Could you elaborate a bit on what you want to achieve and what you have tried?


Correct the implementation of ZHEN

The implementation of current ZHEN has a little mistake. This patch fixes the issue.

@crazyboycjr Does this affect CI? If not I can approve and merge

@zhisbug No, it does not affect CI.

@crazyboycjr cheers! thanks for the first contribution

Thanks @zhisbug! The changes are quite minor, @yf225 can also take a quick look.

LGTM, thanks @crazyboycjr!

@crazyboycjr  I cannot run test_zhen.py after this PR
======================================================================
ERROR: test_zhen_homogeneous (__main__.TorchZHENTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test_zhen.py", line 476, in test_zhen_homogeneous
    loss_func, optim_gen, parallel_method)
  File "/home/ubuntu/efs/alpa/alpa/torch/trainer.py", line 104, in train_torch_module
    state, loss_value = train_step(state, pt_batch)
  File "/home/ubuntu/efs/alpa/alpa/torch/trainer.py", line 61, in train_step
    targets)
  File "/home/ubuntu/efs/alpa/alpa/torch/__init__.py", line 87, in wrapper
    output = func(*args, **kwargs)
  File "/home/ubuntu/efs/alpa/alpa/torch/trainer.py", line 52, in compute_loss
    bufs, out = module_func(params, bufs, inputs)
  File "/home/ubuntu/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1150, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/ubuntu/efs/alpa/alpa/torch/nn/__init__.py", line 323, in forward
    return buffers, self.stateless_model(*args, **kwargs)
  File "/home/ubuntu/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1150, in _call_impl
    return forward_call(*input, **kwargs)
  File "test_zhen.py", line 424, in forward
    input, skip_connection = layer(skip_connection, input)
  File "/home/ubuntu/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1150, in _call_impl
    return forward_call(*input, **kwargs)
  File "test_zhen.py", line 362, in forward
    bias=self.layer_norm_b,
  File "/home/ubuntu/.local/lib/python3.7/site-packages/torch/nn/functional.py", line 2501, in layer_norm
    return torch.layer_norm(input, normalized_shape, weight, bias, eps, torch.backends.cudnn.enabled)
RuntimeError: Expected weight to be of same shape as normalized_shape, but got weight of shape [64] and normalized_shape = [240]


Segmenter fails if lung carnification  / dense

In some severe COVID-19 cases of a recent analysis, the lung density was very high in collapsed lung areas and Lung CT Segmenter failed to produce lung masks covering this area.
Workaround:
Painting in segment editor and "Fill between slices"

Placing additional markers just add the basal spots as shown here

Placing additional markers just add the basal spots as shown here

You can use Grow from seeds even if there is no intensity difference between the segments at all. The trick is to always put lung and other sample points in pairs, on two sides of the boundary.
If this turns out to be too tedious then we could improve Grow from seeds to accept additional boundary curve segments as inputs.

You can use Grow from seeds even if there is no intensity difference between the segments at all. The trick is to always put lung and other sample points in pairs, on two sides of the boundary.
If this turns out to be too tedious then we could improve Grow from seeds to accept additional boundary curve segments as inputs.

Placing additional markers in the Lung CT Segmenter just add the basal spots as shown here

This is probably because that the intensity range that you specified is too narrow.

Placing additional markers in the Lung CT Segmenter just add the basal spots as shown here

This is probably because that the intensity range that you specified is too narrow.

I tried to push the intensity range, but that resulted in leaks from right and left lung.
Placing a few more pairs of fiducials did not help at all, the preview just remains unchanged.
I pressed "Apply", went to the segment editor and painted a few slices in axial view:

This works quite well as a workaround. Of course, I would prefer the fiducials within CT Segmenter because they are much easier to place.
The HU of the dorsal collapsed area goes up to up to +200 ...

I tried to push the intensity range, but that resulted in leaks from right and left lung.
Placing a few more pairs of fiducials did not help at all, the preview just remains unchanged.
I pressed "Apply", went to the segment editor and painted a few slices in axial view:

This works quite well as a workaround. Of course, I would prefer the fiducials within CT Segmenter because they are much easier to place.
The HU of the dorsal collapsed area goes up to up to +200 ...

I tried to push the intensity range, but that resulted in leaks from right and left lung.

When there is little intensity difference between the regions you want to separate then you need to place seeds in pairs, on both sides of the boundary. However, if there is no intensity difference at all (it is comparable to the image noise), then grow from seeds may become too tedious to use. You can try to reduce image noise using image filtering before starting segmentation, but in extreme cases (such as the image above) I agree that switching to a more manual method, such as Fill between slices is a better option.

I tried to push the intensity range, but that resulted in leaks from right and left lung.

When there is little intensity difference between the regions you want to separate then you need to place seeds in pairs, on both sides of the boundary. However, if there is no intensity difference at all (it is comparable to the image noise), then grow from seeds may become too tedious to use. You can try to reduce image noise using image filtering before starting segmentation, but in extreme cases (such as the image above) I agree that switching to a more manual method, such as Fill between slices is a better option.


refactor(transformer): remove boilerplate code around decorators to reduce noise

#2991  👈
main

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

Join @Boshen and the rest of your teammates on  Graphite


[9.4] ISPN-6994 - Fix Embedded Configuration docs

ISPN-6994 - Fix Embedded Configuration docs

This is fixed in https://github.com/infinispan/infinispan/pull/6715


[new locale] add en-in: English (India) locale

Closes #4012

Coverage increased (+0.3%) to 94.763% when pulling 0799c0e4fd445f57e67b5bd5fdff024a8d97ece6 on kumarharsh:develop into 3062ca13cd1cfbcef1c4a0cd5543f5fcd329c1d9 on moment:develop.

Coverage increased (+0.3%) to 94.763% when pulling 0799c0e4fd445f57e67b5bd5fdff024a8d97ece6 on kumarharsh:develop into 3062ca13cd1cfbcef1c4a0cd5543f5fcd329c1d9 on moment:develop.

@marwahaha ping
I also see a bunch of other languages (esp Indian) pending PR due to some extra work which needs to be done - can I clean 'em up and submit a PR?

Hi @kumarharsh - Thanks for this. If you'd like to work on other Indian languages, that sounds great.

Updated some comments in the en-in file which were confusing. It's ready for merge I guess.

@marwahaha ping?

en-in was merged in d9913b2640a0c2b34b191640aeaa9d40ea379cc3
To be fair it differs from this one in both the AM/PM and dow/doy values... You might submit a PR with the changes, but make sure to include some proof of these claims from official sources.


emeditor: Update to version 22.2.0

Closes #10340

[x] I have read the Contributing Guide.

/verify


In-fly shortcut reference links

I need to support [q.b.f] links for my language doc comments that refer to lexically-visible items.
I think that modifying LinkInlineParser.ProcessLinkReference should do it, passing some processor function to the pipeline that returns an URL string or null in case none was found. Could you add something like this to Markdig?

You should probably instead use a post processing pass, like described here via markdownObject.Descendants<LinkInline>(). That won't require to modify Markdig and you can get this up and running with just a few lines of code.


Setting cookies with splash:init_cookies(cookies) does not work

I'm trying to initialize cookies before opening url.
Tried example from documentation https://splash.readthedocs.io/en/latest/scripting-ref.html#splash-init-cookies
function main(splash)
    splash.resource_timeout = 60
    --splash.images_enabled = false
    splash:set_viewport_size(1920, 1080)
    splash:set_custom_headers(splash.args.splash_headers)

    splash:init_cookies({
        {name="baz", value="egg"},
        {name="foo", value="bar"},
    })

    assert(splash:go("http://example.com"))

    return {
        cookies = splash:get_cookies(),
        png = splash:png(),
    }

end

Using local proxy I can see that cookies are not added to request

Same with different domains.
Tested on Splash version: 3.5

Sorry, figured it out.
You must specify the domain like
splash:init_cookies({
        { name = "spam", value = "egg", domain = "example.com" },
    })

Maybe example without domain should be removed from the documentation...


Improve travis configuration

Finally, sorry for previous PRs noise.
Fixes https://github.com/jbittel/django-mama-cas/issues/51

tests did not really fail, s3 download did, please retry that test.


New version: PkgPage v0.3.3

Registering package: PkgPage
Repository: https://github.com/tlienart/PkgPage.jl
Created by: @tlienart
Version: v0.3.3
Commit: 12ce69f3b00ba877ca38397e29a59c05d502a59b
Reviewed by: @tlienart
Reference: https://github.com/tlienart/PkgPage.jl/commit/12ce69f3b00ba877ca38397e29a59c05d502a59b#commitcomment-42655152

[noblock]
@tlienart Did you intend to skip a version?

🙈 no sorry... this can probably be closed and I'll fix it


Strongly Typed SDKs

Describe the proposal
Hey folks!  Saw this stuff was released recently and am super excited about it!
One thing I noticed while looking through these SDKs is that they seem to just be exposing the generated gRPC code of the desired language.  I was wondering if it's worth wrapping the gRPC logic in some thin, language specific wrappers.  This could allow language specific patterns to be exposed in each of the SDKs rather than expecting all consumers to understand gRPC right out of the gate.
I threw a branch together of something that it could look like, but wanted to get some feedback on the idea from the maintainers before I go down a rabbit hole.
https://github.com/dapr/go-sdk/compare/master...bign8:language-specific-api
Looking forward to collaborating!
(Also, sorry if this comes across and edgy or short, I have yet to have my morning coffee ☕)

@bign8 that's great.
Generated sdks are just the beginning, we plan to have strongly typed sdks. Looking forward to collaboration.

@bign8 Perfect, we can start with your PR.
cc @yaron2 as well.

PTAL at the "bespoke client proposal" PR #18

Closing this since #18 landed in master


Is there a way to Paste Excel Data into new table?

That would be cool, if there isn't.
-cdj

Hi @seedjay1 . Can you clarify what you mean by pasting Excel Data into a new table?
Tabular Editor does not deal with any kind of actual data within the Tabular Model tables. It only deals with the model metadata (that is, tables, column, measures, hierarchies, etc.).
If you're thinking of using Power Query to automatically import a table from Excel to a Tabular model, that is unfortunately only possible through Visual Studio / SSDT. In Tabular Editor, you would have to create a Table object and populate it with Data Columns manually (remember to set the Data Type and Source Column property of each column). Finally, you would need to provide the Power Query (M) expression to extract the data from the Excel file, for the table partition.
Let me know if you have any further questions.

I meant the copy (excel cells -> edit menu in VS -> paste) functionality. :)
No biggie - there's also the handy DATATABLE function that i can use within your tool!
Thanks!

I see - I was actually unaware of this feature in VS. Will consider for a future release.

Will not be supported. Use DATATABLE or the DAX table constructor syntax instead.


remove direct install links

Signed-off-by: Evan Hazlett<EMAIL_ADDRESS>(cherry picked from commit 09a73a237358125faffd0b21fcb346d0cbd13f1e)

LGTM


Store password for IMAP server

I have followed every trace of a guide on the internet to be able to save my password in hopes that i dont have to type it everytime i want to go in my Inbox or send an email as my password is very long.Am i missing something or can someone please help me achieve this?Thank you

This particular fork of alpine isn't active any more, I think you'll want to ask whoever is providing alpine to you. (The one I run seems to support passwords in .netrc files.)


when will the output_created_cb get called?

Hi,
I ran into a issue that I found my output_created_cb get called for the same output handle every time I switch among Linux sessions with Ctrl-Alt-Fn, is it normal?

I found that every time you switch to another Linux session and switch back, the output_created_callback get invoked.


reference to writing good scenarios - best practices

http://itsadeliverything.com/declarative-vs-imperative-gherkin-scenarios-for-cucumber

Will moved to upcoming a knowledgebase section on the repo.


Documenter could not fetch from remote error

Having an issue with Travis and documenter for this package:
https://travis-ci.org/Mattriks/CoupledFields.
The Linux builds give the same error: "could not fetch from remote", when Travis tries to push the documentation to gh-pages (I have a DOCUMENTER_DEBUG key set).
I can build the docs locally, and I've added the SSH deploy keys as in the Documenter documentation. The settings/Deploy keys tab in my github repo indicates the documenter ssh key has been used. So what is the issue here?
My versioninfo is:
Julia Version 0.5.0
Commit 3c9d753 (2016-09-19 18:14 UTC)
Platform Info:
  System: NT (x86_64-w64-mingw32)
  CPU: Intel(R) Core(TM) i7-3632QM CPU @ 2.20GHz
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Sandybridge)
  LAPACK: libopenblas64_
  LIBM: libopenlibm
  LLVM: libLLVM-3.7.1 (ORCJIT, ivybridge)

and I'm using Documenter 0.8.5.
@MichaelHatherly

I'd guess it's that repo should match actual repo name: https://github.com/Mattriks/CoupledFields/blob/9777e2a15f68f0de830771fa674c359c3590aaba/docs/make.jl#L9, note the .jl suffix.

Success! :relieved: & thanks! - Documenter is a really nice package!

Great, glad it's working now. (Note for any future issues, best to just open an issue over at Documenter so that other future users who also run into the same problem are more likely to find the solution.)


Allow using imported variables in entrypoint options

Describe the bug
In a content script entrypoint, if you import a variable, lets call it MATCHES, and try to use that variable to for the matches option, the build fails with the following error message:
import { MATCHES } from "@/data/matches";
import { sayHello } from "@/helpers/dev";

export default defineContentScript({
  matches: MATCHES.allUrl,
  async main() {
    sayHello("_document-idle.content");
  },
});

ERROR: MATCHES is not defined

But import { sayHello } from "@/helpers/dev" is okey.
To Reproduce
https://github.com/yunsii/wxt-reproduction
Steps to reproduce the bug using the reproduction:

pnpm i --ignore-scripts
pnpm build --debug

Here's the output:
><EMAIL_ADDRESS>build /Users/aklinker1/Development/local/wxt-entrypoint-imports-repro
> wxt build "--debug"

WXT 0.13.5                                                                                11:12:19 AM
ℹ Building chrome-mv3 for production with Vite 5.0.11                                                              11:12:20 AM
[11:12:20 AM] ⚙ Loading file metadata: /Users/aklinker1/Development/local/wxt-entrypoint-imports-repro/src/entrypoints/_document-idle.content/index.ts
[11:12:20 AM] ⚙ Loading file metadata: /Users/aklinker1/Development/local/wxt-entrypoint-imports-repro/src/entrypoints/background.ts
⚙ Text:                                                                                11:12:20 AM
export default defineBackground(() => {
  console.log('Hello background!', { id: browser.runtime.id });
});

No imports:
import { defineUnlistedScript, defineContentScript, defineBackground } from 'wxt/sandbox';

export default defineBackground(() => {
  console.log('Hello background!', { id: browser.runtime.id });
});

Code:
import { browser } from 'wxt/browser';
import { defineUnlistedScript, defineContentScript, defineBackground } from 'wxt/sandbox';

export default defineBackground(() => {
  console.log('Hello background!', { id: browser.runtime.id });
});

[jiti] [transpile] /Users/aklinker1/Development/local/wxt-entrypoint-imports-repro/src/entrypoints/background.ts (40.18ms)
[jiti] [transpile] [esm]<EMAIL_ADDRESS>(0.85ms)
[jiti] [transpile] [esm]<EMAIL_ADDRESS>(0.791ms)
[jiti] [transpile] [esm]<EMAIL_ADDRESS>(1.293ms)
[jiti] [native]<EMAIL_ADDRESS>[jiti] [transpile] [esm]<EMAIL_ADDRESS>(0.935ms)
[jiti] [native] /Users/aklinker1/Development/local/wxt-entrypoint-imports-repro/node_modules/.pnpm/@webext-core+match-patterns<EMAIL_ADDRESS>⚙ Text:                                                                                11:12:20 AM
import { MATCHES } from "@/data/matches";
import { sayHello } from "@/helpers/dev";

export default defineContentScript({
  matches: MATCHES.allUrl,
  // matches: ["<all-url>"],
  async main() {
    sayHello("_document-idle.content");
  },
});

No imports:
import { defineUnlistedScript, defineContentScript, defineBackground } from 'wxt/sandbox';

export default defineContentScript({
  matches: MATCHES.allUrl,
  // matches: ["<all-url>"],
  async main() {
    sayHello("_document-idle.content");
  },
});

Code:
import { defineUnlistedScript, defineContentScript, defineBackground } from 'wxt/sandbox';

export default defineContentScript({
  matches: MATCHES.allUrl,
  // matches: ["<all-url>"],
  async main() {
    sayHello("_document-idle.content");
  },
});

[jiti] [transpile] /Users/aklinker1/Development/local/wxt-entrypoint-imports-repro/src/entrypoints/_document-idle.content/index.ts (1.392ms)

 ERROR  MATCHES is not defined                                                                                11:12:20 AM

  at src/entrypoints/_document-idle.content/index.ts:25:12
  at evalModule (node_modules/.pnpm/jiti@1.21.0/node_modules/jiti/dist/jiti.js:1:256443)
  at jiti (node_modules/.pnpm/jiti@1.21.0/node_modules/jiti/dist/jiti.js:1:254371)
  at importEntrypointFile (node_modules/.pnpm/wxt@0.13.5/node_modules/wxt/dist/cli.js:1786:23)
  at async getContentScriptEntrypoint (node_modules/.pnpm/wxt@0.13.5/node_modules/wxt/dist/cli.js:451:35)
  at async node_modules/.pnpm/wxt@0.13.5/node_modules/wxt/dist/cli.js:230:18
  at async Promise.all (index 0)
  at async findEntrypoints (node_modules/.pnpm/wxt@0.13.5/node_modules/wxt/dist/cli.js:218:23)
  at async internalBuild (node_modules/.pnpm/wxt@0.13.5/node_modules/wxt/dist/cli.js:2503:23)
  at async build (node_modules/.pnpm/wxt@0.13.5/node_modules/wxt/dist/cli.js:2538:10)

✖ Command failed after 367 ms                                                                                11:12:20 AM

 ERROR  MATCHES is not defined                                                                                11:12:20 AM

  at src/entrypoints/_document-idle.content/index.ts:25:12
  at evalModule (node_modules/.pnpm/jiti@1.21.0/node_modules/jiti/dist/jiti.js:1:256443)
  at jiti (node_modules/.pnpm/jiti@1.21.0/node_modules/jiti/dist/jiti.js:1:254371)
  at importEntrypointFile (node_modules/.pnpm/wxt@0.13.5/node_modules/wxt/dist/cli.js:1786:23)
  at async getContentScriptEntrypoint (node_modules/.pnpm/wxt@0.13.5/node_modules/wxt/dist/cli.js:451:35)
  at async node_modules/.pnpm/wxt@0.13.5/node_modules/wxt/dist/cli.js:230:18
  at async Promise.all (index 0)
  at async findEntrypoints (node_modules/.pnpm/wxt@0.13.5/node_modules/wxt/dist/cli.js:218:23)
  at async internalBuild (node_modules/.pnpm/wxt@0.13.5/node_modules/wxt/dist/cli.js:2503:23)
  at async build (node_modules/.pnpm/wxt@0.13.5/node_modules/wxt/dist/cli.js:2538:10)

 ELIFECYCLE  Command failed with exit code 1.

Expected behavior
Variable matches works with defineContentScript intuitively
Environment
  System:
    OS: Linux 5.15 Ubuntu 20.04.6 LTS (Focal Fossa)
    CPU: (16) x64 AMD Ryzen 7 5700G with Radeon Graphics
    Memory: 27.67 GB / 31.35 GB
    Container: Yes
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 18.4.0 - ~/.nvm/versions/node/v18.4.0/bin/node
    npm: 8.12.1 - ~/.nvm/versions/node/v18.4.0/bin/npm
    pnpm: 8.14.0 - ~/.nvm/versions/node/v18.4.0/bin/pnpm
  Browsers:
    Chrome: 115.0.5790.98
  npmPackages:
    wxt: ^0.13.5 => 0.13.5

I have to use relative path import instead path alias import for now.

Yes, you must use literal values when defining options for entrypoints.
Why? When importing your entrypoints to get it's options during the build process, the file is imported in a node environment. It doesn't have access to window or other DOM globals. It also doesn't have access to the chrome or browser globals an extension normally has access to.
So to get around this, wxt strips all imports from your entrypoints, and then import the file.
This makes sure any module with side-effects doesn't cause a build failure when accessing one of these global variables.
The only solution is to setup a full browser environment while loading the entrypoints files, which I decided would be too error prone. So imports are removed instead.
I would like to allow imports for setting up options, but haven't had the time to enable that yet. Would be open to contributions for this.

It is inconvient to arrange matches with logic, and is it significant impact of build entrypoints if setup a full browser environment?

It is inconvient to arrange matches with logic, and is it significant impact of build entrypoints if setup a full browser environment?

Sorry, I don't understand the question.

For future reference, here is the file responsible for importing the entrypoints during the build process to grab their options.
https://github.com/wxt-dev/wxt/blob/8c834835c453dcb49f8b44fa432203d83b902a81/src/core/utils/building/import-entrypoint.ts

Variable matches can be used in business logic

Since WXT doesn't run the main function during the build, it only grabs the object passed into defineContentScript, any imported variables used inside the main function doesn't throw any errors because that code is never executed.
It's the same reason why this code runs without throwing an error. The example function is never called.
function example() {
  console.log(someVariableThatDoesNotExist);
}

console.log("Hello world");

developers have to maintain two copies of matches data sometime

Yes. That's why I'd accept a PR to fix this. Eventually, I'll get to this, but not until after v1.0 is out.

I'm gonna close this issue, it will be fixed in #506, follow that one for updates.

For future reference: working-repro.zip
Removing the experimental.entrypointImporter option will cause the original error to occur.


Documentation doesn’t mention template fields can accept functions too

I was delighted to find out that I can pass functions instead of template strings for fields like scaleLabel and tooltipTemplate, but I found it by looking at the source for for the functions parsing those fields. Mentioning this feature in the documentation probably saves some time for other new users. It can also assure users like me that they’re not using some private API that may change in future.

Hello, today I am closing all issues that are only affecting version 1 of Chart.js as WONTFIX.
If this issue does affect version 2 as well, I apologize for the error. Please create a test case against Chart.js 2 using on of the below websites and we will be happy to reopen the issue and update its classification:

http://jsbin.com/
http://jsfiddle.net/
http://codepen.io/pen/
Premade template: http://codepen.io/pen?template=JXVYzq


Should Class present their ToWritableProperties fo ToStyleSheetTheme

Hello,
A Style Sheet is composed of a selector(type, stamps, and combinaison between them), a set of properties (backgroud, border, animation, ...) and values for the properties.
Currently the properties are defined in the method: ToStyleSheet class >> #defaultWritablePropertyList.
But it would make more sens if the Toplo Widget contains their own stylable properties.
This properties could be the standard API of the Widget.
For example, we have the property (ToFeatureProperty new name: #innerImage)., this property only makes sens on a Widget with a #innerImage method.
Why do we need the #defaultWritablePropertyList ?

Why do we need at the same time ToSkinEventAction and ToPropertyWriter to modify the properties of an object when we could only use ToSkinEventAction that are way simplier than ToPropertyWriter ?

Answer: If two styleRules want to change the same property, ToStyleSheet will only apply the last installed one. Therefor we need a system to identify the properties, and it is difficult to identify the property if it is use directly in a block.

Interesting questions! the need for ToStyleSheet class >> #defaultWritablePropertyList is not clearly answered.
@Nyan11 it would be cool to add your answer in the toplo book

It is already present in: https://github.com/SquareBracketAssociates/Booklet-Graphics/blob/main/Chapters/toplo/stylesheet.md

By default, only the last writter per editable property and look event is installed for a given graphic element. You can force a rule to be applied using the supplements variable.


Show total data count next to legend labels

What problem does this feature solve?
You currently have to manually add numbers to see what the total for a given data series is across a whole chart, it would be great if there was a way to append calulated totals to legend labels.
I could try and do this programatically but if I add the count as part of the label the label goes away because it no longer matches the series name.
What does the proposed API look like?
I'd propose adding something like the following to the legend part of the API:
showCount: true

If adding an option showCount: true and make built-in implement, it become a "common feature" and need to define how it look link and how to configure its style. It's not a easy work I am afraid.
I think it's probably better for user to make the count of data displayed in like text component or series label or custom series, which is more flexible.

Can we add custom data to bar and line chart legend irrespective of series object?

I'd like to do this aswell.


Update Elasticsearch plugin to v4.0.3

Because the previous ES plugin versions contain non-preserving scheme
issue.
elasticsearch-ruby treats scheme information without option.http.scheme
as ephemeral information.
So, https scheme users suffered suddenly fallback to http issue.
Elasticsearch plugin v4.0.3 fixes this issue.
See: https://github.com/uken/fluent-plugin-elasticsearch/pull/713
Signed-off-by: Hiroshi Hatake<EMAIL_ADDRESS>

@repeatedly ?

You don't need to wait my review for such small changes ;)

Thanks. But I don't have push permission in this repository. :confounded:

Ah, I updated the team. Could you check?

Thanks. Merge button got appeared!


Problems with opening up the page on clicking

You can't open any item in the search page unless you actually click on the thumbnail. If you're hovering over the item's text and then try to click, you'll receive a console error.

unsure if this is fixed??


ci using extism cli

@zshipko, if you get a chance, can you take a look at the ci branch & the CI workflow in particular.. I can't seem to get the CLI installed in such a way that it uses the installed lib/.h and doesn't attempt to fetch git resources.
See: https://github.com/extism/assemblyscript-pdk/runs/8218063319?check_suite_focus=true

I pushed a commit to extism/cli to use the http remote and pushed a commit to ci to install cffi and now ci is passing!

Awesome! I tried to switch to http remote in the CLI earlier but when I used it locally GitHub wanted to auth me with username and password which is no longer supported... do you happen to hit the same issue when you use it locally now?

(This was when attempting to extism install git)

I think it's because there was a typo in the url but extism install git is working for me now

Haha wow I didn't notice that 🤦 - good catch!


Ability to move or hide the button adjacent to micro menu

Hi!
Would it be possible to add an option to move or hide the button adjacent to the micro menu? I currently have my micro bar as mouseover so it it strange to have the button always displayed.
Thanks!

Hi @Oddezt,
I don't plan to make it moveable, but I can determine if it's possible to make it respect the visibility condition of the micro menu. What addon do you use to hide it except on hover?

Thank you! I use ElvUI and choose 'mouseover' from there.

This has been addressed in 64ab9c8. See below.

Hi, sorry to bother you again! I can't seem to get this to work on my end. The micro menu button is still always visible. Thanks!

This won't be available until the release of 3.0.5. The hotfix build that went out today is not a content update, and therefore has none of my work scheduled for August 27.

@Oddezt
I've reopened the issue and will investigate.
Would you be able to send me a copy of the ElvUI saved variables for your main character? (The one you checked if the mouseover setting is working.)
My initial guess is HelpMePlay can't determine whether it's enabled or not, and therefore it keeps the icon shown.

@Saaappi
Of course! What is the best way for me to send it across to you?

You should be able to attach a file directly to a comment.
I need the ElvUI.lua file from C:\Program Files (x86)\World of Warcraft\_retail_\WTF\Account\{GUID}\SavedVariables, where {GUID} will be your account's unique identifier.

You can't attach LUA files to comments unfortunately.

Compress the file to .zip format and upload that instead.

ElvUI.zip
Done!

e491fd8 should have this resolved. Extract the file below and replace the MainMenuBar.lua file at the root of the addon folder. If you're not asked "Are you sure you want to replace this file", or something similar, then you pasted it in the wrong folder.
Let me know if you have any additional issues.
MainMenuBar.zip

Thank you! That has fixed it!


Add support for ItemOption dependencies

This could be useful (for example) if an event had a some elective optional things that are only available if you have a pass for the day they're on. So for example, a competition might only be available if you have a pass that lets you attend the dance during which the competition takes place.
I can foresee a need at least for the option to have a "one of these" dependency (you can do the comp if you have a full pass, saturday pass, or entrance just to this dance) but I suppose it would hypothetically also be useful to have an "all of these" dependency. Maybe that could be a feature that's added later? But if we want to support it, it would be nice to structure things accordingly from the outset.

I know this will probably seem preposterous, but what if embedded a scripting language for power-users that would allow them to write their own logic for analyzing a cart and determining if it's valid for checkout. It could use attendees, forms, housing, and orders as inputs and return a list of messages (to be displayed by brambling) as well as a "ready for checkout" or "not ready for checkout" boolean. This could be a powerful system and a big selling point for events that have complex checkout requirements.
Even if we didn't get a lot of users using this system, it would allow us to quickly customize events to user's needs on a per-event basis and we could offer some basic templates like one-to-one pass-to-attendee correspondence.
The embedded language would probably need to be a language that is designed to be safe (in terms of injection and cpu consumption), so not python, unless there's an embedding-safe version of python?

That seems like a cool feature idea, but way out of scope for this ticket? But if we did end up going that route, Lua is designed as an embedded language and is pretty simple.

Lua was my first thought too. Yeah. I guess I'm just having trouble imagining a good UI for this ticket, so I thought that embedded logic might actually be easier than trying to come up with that (and would also obviate this ticket) but I guess a simple FK would be easy enough.


Pinch zoom at last zoom level displays grey

Zooming with double taps works fine when you hit the last zoom level, and try to zoom further ; but when already at the last zoom level, a zoom attemps with a pinch gets the display all grey.
We have found that undefining maxZoom in the layer options removes the issue.
Example
http://jsfiddle.net/tJCLw/6/
Environment

Leaflet version: 0.7.2
OS/Platform (with version): iOS

Can someone confirm the bug is still present in master?

Hello,
I can confirm that this behaviour still occurs in Leaflet 1.0.2 with Chrome for Android 54.
If I set maxZoom in the TileLayer options, pinching in on the last zoom level causes a full grey display for a second. If I set maxZoom via map.setMaxZoom(zoom); it works fine.
Thanks

Hello,
Since this issue is very old and therefore doesn't get much attention anymore, I'm mentioning @IvanSanchez - could you please take a look?
Thanks

@PascalReintjens could you provide running examples (like on http://playground-leaflet.rhcloud.com/) of the problem, or update the fiddles linked above to lateast Leaflet?
The bug title says iOS, but you're referring to Chrome on Android, which makes me confused... you mean this can be reproduced on Android as well?

Updated the jsfiddle, with maxZoom gives me a grey map on initial load on Safari 7 on iPhone 5:

With maxZoom
Without maxZoom

Updated with @latest Leaflet versions:

With maxZoom
Without maxZoom

@nicomgd, @Starefossen, @PascalReintjens Are you able to reproduce the bug using the @latest version?

Updated the jsfiddle, with maxZoom gives me a grey map on initial load on Safari 7 on iPhone 5:

With maxZoom
Without maxZoom

@Starefossen I know this is an old issue, but I can't tell if you're describing the same problem as @nicomgd. Below are updates of your fiddles (because the original fiddle by @nicomgd seems to break using later versions of Leaflet), using @latest (currently v1.9.2):

With maxZoom
Without maxZoom

The tiles aren't grey on initial load, so something's changed, but I also don't have an iOS device to test pinch zoom.


Prevent conflicting system resource parameters

Currently, a system is allowed to have multiple resource parameters to the same resource type. No current checks prevent multiple mutable borrows aliasing the same resource. Surprisingly, this applies to multiple local resources too in a slightly different way.

examples
#[derive(Default)]
pub struct Buffer(Vec<u8>);

// builds without an error and goes uncaught by runtime checks.
fn system(a: Res<Buffer>, mut b: ResMut<Buffer>, mut c: ResMut<Buffer>) {
    // heck, two aliasing mutable borrows + a shared borrowed
    let x: &Vec<u8> = &a.0;
    let y: &mut Vec<u8> = &mut b.0;
    let z: &mut Vec<u8> = &mut c.0;
    dbg!(x.len());
    y.push(42);
    z.push(42);
    dbg!(x.len());
}

pub fn system(mut a: Local<Buffer>, mut b: Local<Buffer>) {
    let a = &mut a.0;
    let b = &mut b.0;
    assert_eq!(a.len(), 0);
    assert_eq!(b.len(), 0);
    a.push(42);
    assert_eq!(a.len(), 1);'

     // this assert will fail because the two locals _are_ the same resource
    assert_eq!(b.len(), 0);
}

This PR adds access checks to the SystemParam::init implementation of Res, ResMut, and ChangedRes. The current place was chosen since it is minimally invasive and will allow failing early, following the lead of how conflicting queries are handled. Systems are now only allowed at most one read or write access to a specific resource type.
While multiple read-only parameters to the same type would be harmless, it seems like fn(Res<Foo>, Res<Foo>) would be at best be user-error and would be nice to catch early.
Something I'm unsure about is the removal of system_state.resource_access.add_write(TypeId::of::<T>()); for Local resources. Without doing so, a Local<Foo> and Res<Foo> would not be able to co-exist without leading to a panic, but it's possible it's currently needed in the parallel scheduler's checks (ideally it shouldn't be be since the access to that resource would only occur by the system with the local or a accessed by ID by a thread local system).

Heh, testing on more examples, one place that the Local resource else branch currently does not work with is a scenario like
https://github.com/bevyengine/bevy/blob/43aac1a78480e196f9b8b29339a34d78bc44b663/crates/bevy_render/src/render_graph/nodes/camera_node.rs#L46-L60
I'll look back into this later in the afternoon.

This should prevent the issues pointed out in #598 (except for the issue title).
If splitting up the TypeAccess tracking for local resources is the correct course of action too, then it potentially resolves #483 as well.

I think it can be useful to have multiple copies of the same Res<T> in a SystemParam. It enables encapsulation for sub-objects.
#[derive(SystemParam)]
struct Outer<'a> {
    inner: Inner<'a>,
    foo: Res<'a, Foo>,
}

impl<'a> Outer<'a> {
    pub fn use_inner(&self) { self.inner.use_foo() }
}

mod inner {
    #[derive(SystemParam)]
    pub struct Inner<'a> {
        foo: Res<'a, Foo>,
    }

    impl<'a> Inner<'a> {
        pub fn use_foo(&self) {}
    }
}

@bonsairobo do you have a fuller example? Inner is repeating the fields thar Outer already has access to.

I just mean that Inner might be some reusable SystemParam object, so you can build larger objects out of it. But sometimes both the inner and outer object need to access the same global resource. It shouldn't be a problem if it's Res, so disallowing this is limiting from a design perspective.

Makes sense ^^ it’s a small change to make the Res checks look for only write accesses rather than reads/writes. I’ll push a change later today.

@memoryruins Thanks for hearing me out. I only spoke up because I've run into this issue before with specs, and I found it annoying that it wouldn't let me read the same thing twice from the same system.

Very good call! Thanks :smile:

If splitting up the TypeAccess tracking for local resources is the correct course of action too, then it potentially resolves #483 as well

Yeah I think this is a good call. Right now its fine to just ignore local resource access in the scheduler because there are only two ways to access Local resources:

exclusive &mut World access
via the Local SystemParam, which is restricted to accessing the current system's local resource.

If we ever add the ability to access arbitrary Local resources in query systems, then we will need to take that in to account.


Scaffolding for pluggable models

A placeholder issue for implementing system for pluggable classifiers.

One place where we can add simple scaffolding is in database.py around lines 1347-1390. @jeremykubica suggested a simple registration system with a parent class and then assorted subclasses this makes a lot of sense in this case.
@AmandaWasserman pointed out that down the line, we'll probably want to let people bring their own search strategy (or other classes) - we'll keep that in mind and make sure the registration system can handle that.

Also checkout database.py lines:

268-365 load_features_from_file These are the hardcoded feature extractor names.
949-971 classify These are the different classifiers
1240-1289 make_query_budget Similar to the second comment in this issue - basically everything that is going on in query_budget_strategies.py.

After chatting with Amanda, the three classes (or will likely become classes) that should be made pluggable are:

classifiers.py
metrics/query strategy in query_strategies.py
feature extractor resspect/feature_extractors/[bazin|bump|light_curve|malanchev].py

Pluggable classifiers has been mostly taken care of with PR #45. There are a couple of follow up issues that should be addressed with #48 and #49.
After those, we should move over to making the feature extractors pluggable.


distinguish msg from docker logs

Just a small change to distinguish building/testing message from docker logs, which looks like:
hack/test.sh rhel7
---> Building nodejs-0.10-rhel7
Sending build context to Docker daemon 28.67 kB
Sending build context to Docker daemon
Step 0 : FROM rhel7
 ---> 8a06c82684c8

@soltysh thoughts ? :)

Yep, that makes sense. LGTM.


[stable/redis-ha] define prometheus exporter port in service

This enables prometheus-operator to work with this chart almost out-of-the box. Only a ServiceMonitor definition is necessary.

What this PR does / why we need it:
This adds an optional port definition for the exporter-port to the redis-ha Service. The port was already defined in the Deployment, but is not visible in the Service.
This enables this chart to work with prometheus-operator basically out-of-the box - only an additional definition of a ServiceMonitor is needed. I did not include this ServiceMonitor, because I want to leave this decision up to other maintainers.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

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

/ok-to-test

/lgtm


An app name has non ASCII characters throws an exception.

HTTP Error: Unexpected char 0xb79c at 20 in header value: Digits/<IP_ADDRESS> ( 테스트; Android 6.0.1), API Error: 0, User Message: 다시 시도

I looked up the Digits source code and it builds its user agent with a current app name from a package manager. The above is the error I'm getting if an app name contains a non ASCII character. JakeWharton's comment mentioned that OkHttp validates header values so using a current app name in user agent needs to be changed to support non-english apps.

@njovy Thank you for the issue report. This is a known issue. I believe this will be updated in the next release.

This issue has been resolved: https://github.com/twitter/digits-android/commit/0600a305efbec8ab46a6578145e39cd7f981a295
Please update to latest version.


Update problem.md

Verify is looking for the number 45. However, the direction say to set the limit to 10.

Not quite :) Please read through the instructions again - it says to add them all up.

Hi @SomeoneWeird,
Thanks for the note. It's very possible I am missing something. :) In any case, I am still scratching my head. Here are my thoughts on the problem/directions:
In that file define a variable named total and make it equal the number 0:
var total = 0;
Define a second variable named limit and make it equal the number 10:
var limit = 0;
Create a for loop with a variable i starting at 0 and increasing by 1 each time through the loop. The loop should run as long as i is less than limit. On each iteration of the loop, add the number i to the total variable. After the for loop, use console.log() to print the total variable to the terminal:
var total = 0;
var limit = 10;

for (var i=0; i<limit; i++) {
    total += 1;
}

console.log(total);

Is the code above correct?
The validator fails it because it is looking for a total equal to 45. However, if we add by 1 on each loop and stop at 10 we never reach 45.

Nearly! You need to add the value of i to total each time, instead of 1.

Wow! I am a fool! :)


Add Skypath-logo

hello, there is an issue with this PR. Please resubmit with the below instructions:
Submit pull request by adding your token logo 120X120 PNG. (Folder name needs to be named with the token address)
-Please name it logo.png

Hey! I sent them to you on telegram as requested. Will that be ok to
continue?
On Sun, Feb 4, 2024, 7:19 PM unserhtain @.***> wrote:

hello, there is an issue with this PR. Please resubmit with the below
instructions:
Submit pull request by adding your token logo 120X120 PNG. (Folder name
needs to be named with the token address)
-Please name it logo.png
—
Reply to this email directly, view it on GitHub
https://github.com/traderjoe-xyz/joe-tokenlists/pull/1146#issuecomment-1926162531,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZHPSY3OPCDT43FVZPQHD3DYSBFTDAVCNFSM6AAAAABCYHWIEOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRWGE3DENJTGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Copy sqldatabase fix withreplace - fixes #601

Can someone test this by tomorrow afternoon CET? I'd like to include it in the release tomorrow. I've tested it a bit with creating a db with a non default path on sql2016/vnext then replacing it.  Also, this param may be named improperly.
Copy-SqlDatabase -Source sql2008 -Destination sql2012 -BackupRestore -NetworkShare \\dc\sql  -AllDatabases -WithReplace

Copy-SqlDatabase -Source sql2012 -Destination sql2016  -BackupRestore -NetworkShare \\dc\sql  -AllDatabases


Error packaging image assets

Attempting to add a newly added image breaks the build:
Unable to resolve module image!Fail Icon from /Users/aschachter/Development/LetsDoThis-iOS/ReactComponents/NetworkErrorView.js: Invalid directory /Users/node_modules/image!Fail Icon
Sounds a lot like https://github.com/facebook/react-native/issues/4968
which node:
/Users/aschachter/.nvm/versions/node/v5.5.0/bin/node
react-native --version:
react-native-cli: 0.1.10
react-native: 0.19.0

Workaround for this issue has been to just specify the filename as the uri within React Native.


 The spec valid usage text states 'The VkCommandPool that commandBuffer was allocated from must support graphics operations' (https://www.khronos.org/registry/vulkan/specs/1.0/html/vkspec.html#VUID-vkCmdSetViewport-commandBuffer-cmdpool).
GPU: AMD RX 480 8GB, OS: GNU/Linux Antergos, Driver: Mesa 17.3.3-2.

I can't immediately see what could cause this. See what values does the queueFamilyIndex get.
pool_create_info.queueFamilyIndex	= r._graphics_family_index;
Then see in the Renderer.cpp lines
 bool found = false; for( uint32_t i=0; i < family_count; ++i ) { 	if( family_property_list[ i ].queueFlags & VK_QUEUE_GRAPHICS_BIT ) { 		found = true; 		_graphics_family_index = i; 	} }
See what value does the _graphics_family_index have. compare it to the family_property_list flags with the same index. If you're sure that the queueFlags contains the VK_QUEUE_GRAPHICS_BIT then I think it's the validation layer bug.

Hi GarethH96,
I'm also an Radv user.
You might be interested in this article: https://www.phoronix.com/scan.php?page=news_item&px=Mesa-17.3-Remains-Buggy

@hodasemi That's interesting. @GarethH96 Can you test this with AMD closed drivers to verify it's a problem with Mesa?

I'm also using Mesa (17.3.5 by now) and Archlinux. Tutorial 4 doesn't make any problems on my side (RX 470).


Fix issue #63: invalid reads after write

Per our discussion on #62, this makes all writes atomic.

PTAL. I've overhauled the way we handle atomic write temp files by jamming them into a (hidden) subdirectory. The name of this directory is configurable; anything within that directory will be ignored when listing keys.

Are writes not atomic if you set TempDir to the same physical disk as the diskv path?

They are. The problem is that if you don't set TempDir, ReadStream is broken. You will read inconsistent data if someone sneaks in a Write before you finish reading. So the current situation is basically "Set both BaseDir and TempDir unless you want diskv to be broken, but we won't warn you if you disregard TempDir".
I've put quite a few hours into fixing this now. If you're not interested in the fix, feel free to close the issue and we will continue to maintain and use my fork and not bother you with further PRs.

Unless I fundamentally misunderstand the semantics of modern filesystems — which is possible! — an open os.File should be a snapshot of its contents when opened, and should not change if, while open, a different process or thread modifies the file on disk. Is that not true?

Copy TestIssue63 from the branch into the same file in master. It should fail--it did on my Linux box, anyway. It demonstrates that what is read from an already-opened file will indeed change if the file on disk is changed.

Ah, yes, I see, it's a consequence of the TempDir decisionmaking and the mode flags we pass.
Solution here is to enforce a default TempDir in the diskv path if none is provided, rather than a separate option altogether, I think.


ColVis' stateChange callback function only triggers when

ColVis' stateChange callback function only triggers when an user clicks within a list item's label's content. The callback function doesn't trigger if an user clicks anywhere else in the list item.
http://jsfiddle.net/fL17ev1t/

Thanks for the test case. This is just the oddest thing - if you click on the span element, two click events are triggered, causing two callbacks. This commit fixes that, but causes the error you are seeing.
I can't figure out what is causing the double event - it just doesn't appear to make any sense!
What I would suggest at the moment is to forego the use of the stateChange callback and instead use DataTables' column-visibility event.

I think I managed to replicate the issue here: http://jsfiddle.net/5m9os05y/
Clicking the span of the list item gets you two alerts instead of one like you'd expect. But using change() instead of click() apparently fixes the issue and I'm not sure why.
All I have to back me up is this SO comment here (though the page itself may help you to elucidate the issue): http://stackoverflow.com/questions/8238599/jquery-click-fires-twice-when-clicking-on-label#comment30702808_13756844

Fixed in #35


Front Camera Not Working

Model: gt-n8013
App give me 2 option but both of those are the back camera

I had the same issue. Using a first generation nvidia shield tablet.
If I update the config to use the 2nd camera and reboot the device it works (ie its using the front camera)

Thanks. i will try tonight.

Installed version .6 and now im getting "0: Error" "1: Error" under the camera selections

I also get 0:Error and 1:Error on my moto G4 plus phone.. Is there any debugging I can do?

Same for me on my nexus 5. I use the latest version. (6.1)

Test release: https://github.com/WallPanel-Project/wallpanel-android/releases/tag/0.7b1


Add tests to the Slider component

From #7 there are many components to test.
This issue is to track testing of the Slider component.

Closed by https://github.com/vczb/sagu-ui/pull/11


Document integer scaling functionality and limitation

Fixes #81024.
On a related topic, noticed that setting all of integer scaling, stretch, and aspect > 1 results in something broken... These settings probably need a useless combinations check.

Thanks!


BLOB/TEXT column 'id' used in key specification without a key length

I am getting an error when running the command npx better-auth migrate on MySQL Database. This is an example of the auth.ts I am using.
export const auth = betterAuth({
  database: {
    provider: 'mysql',
    url: 'mysql://root:@<IP_ADDRESS>:3306/superOceanDB',
  },
  emailAndPassword: {
    enabled: true,
  },
  user: {
    modelName: 'users',
  },
  account: {
    modelName: 'accounts',
  },
  session: {
    modelName: 'sessions',
  },
});

And this is the error that appears in the terminal window:
...at async CreateTableBuilder.execute (file:///Users/yusuf/Documents/sandbox/next-js/super-ocean/node_modules/kysely/dist/esm/schema/create-table-builder.js:350:9) {
  code: 'ER_BLOB_KEY_WITHOUT_LENGTH',
  errno: 1170,
  sqlState: '42000',
  sqlMessage: "BLOB/TEXT column 'id' used in key specification without a key length",
  sql: 'create table `users` (`id` text primary key, `name` text not null, `email` text not null, `emailVerified` boolean not null, `image` text, `createdAt` date not null, `updatedAt` date not null)'
}

Node.js v20.14.0

The error does not occur when running with an SQLite database.
When I debug the generated query,
CREATE TABLE `users` (
	`id` text PRIMARY KEY,
	`name` text NOT NULL,
	`email` text NOT NULL,
	`emailVerified` boolean NOT NULL,
	`image` text,
	`createdAt` date NOT NULL,
	`updatedAt` date NOT NULL)

I get the same error
Query 1 ERROR at Line 1: : BLOB/TEXT column 'id' used in key specification without a key length
I suspect this is due to the table schema generator.

I believe the query should be
CREATE TABLE `users` (
	`id` text PRIMARY KEY NOT NULL,
	`name` text NOT NULL,
	`email` text NOT NULL,
	`emailVerified` boolean NOT NULL,
	`image` text,
	`createdAt` date NOT NULL,
	`updatedAt` date NOT NULL
)

@Bekacru Bro, what have you done, my exam was tomorrow and for contribution I neglected it and you merged another pull request. And you can also see that I made a commit earlier to you.


after select driver program`s windows is clear

Hi, again the question ... I started the program, went into Action -> Select drivers/devices....  selected Device/KeyboardClass0 and clicked ok. The window is empty ....

Did you checked the Hook item in the context menu when selecting the device?
Did you checked Capture events in the Monitoring menu?
How many keyboards do you have connected to your machine? For example, laptop keyboards are usually referenced as PS/2 and represented by the KeyboardClass0 device, additional keyboards are KeyboardClass1 etc. (USB ones). In other words, you need to use the right keyboard to see the events (num lock, caps lock and scroll lock are exceptions).

seems to figure it out ... Great tool, but how do you know which irp is sent to the driver, and which irp sends the driver in response? Thanks

Basically, my driver just hooks IRP handlers of the target driver. Hence, it gets control when an IRP is sent to the device. For each IRP, the driver also registers a completion routine, so it gets notified when the IRP gets completed.

super, but how can I see what the driver returned?

Look into the IOSB Status column of the IRP completion. Or into the Result column of the IRP (if it is STATUS_PENDING (0x103), you still need to look at the IOSB Status of the corresponding IRP completion event).

mean all the packages STATUS_PENDING(0x103) from the driver? no others?

Well, the keyboard events (key press and release) are processed in the following manner:

an IRP_MJ_READ is sent to each KeyboardClassX device,
kbdclass.sys pends the IRP (thus, STATUS_PENDING) and stores it in its internal queue,
when a keyboard event occurrs, one read IRP is removed from that internal queue and completed (IOSB Status and IOSB Information is set).

Caps lock, num lock and scroll lock should produce also an IOCTL setting the keyboard LEDs which AFAIK is not pended by the driver.
So, look at IOSB Status and IOSB Information columns in rows representing IRP completions.

Yes, I already experimented with the keyboard. Is Status_pending set after processing by any driver? If for example, is it a software driver of some kind of antivirus?

Is Status_pending set after processing by any driver?

No. If the driver processes the IRP synchronously (e.g. directly in its dispatch handler, no queuing), it must

set IOSB Status and Information to report the result,
return the NTSTATUS value set to IOSB Status.

Many drivers do process IRPs synchronously. But all depends on the driver author. Also, there are some IRP types that AFAIK must be processed synchronously (the driver MUST NOT return STATUS_PENDING). IRP_MJ_CREATE is one of them.

I have fields where IOSB is 0x00000000 and there is where IOSB is 0x0000000000002 or 0x000000000012. IOSB status everywhere is STATUC_SUCCESS

IOSB is a shortcut for I/O Status Block which contains the following information

a status code reporting whether the IRP was completed successfully or an error occurred,
anInformation field meaning ofwhich depends on the IRP type – for example, drivers use it to signal number of bytes read.

So,you must be careful to distinguish between these two fields. As far as I know, the keyboard class driver usually completes its IRPs with STATUS_SUCCESS (0).
Driver responses are reported as events of type IRP Comp.
I have also improved (quite greatly I think) my tutorial about keyboard monitoring. Hope you find it helpful.
https://github.com/MartinDrab/IRPMon/wiki/KeyboardMonitoring
The Wiki also contains some additional information about Windows drivers in general which may also provide some insight.
I am not sure whether I answered your question. If not, please try asking again.

thank you for help . I work with you program, this is great program.


Fix recursion type validation

Coverage decreased (-0.2%) to 95.071% when pulling e9aac0b1773bb6f9cca9ebbb3549094c12ac38a3 on fix-recursion-type-validation into 53910728e54e49941b855266b7d7259a858bce35 on master.


Add ESLint configuration

Add an ESLint configuration using eslint-config-airbnb-base. For the moment this is only linting the specs.

Looks like this would need either:

Moving away from build-package.sh
Decaffeinating the codebase

Since the second option is already planned, it looks like this PR will have to wait for that.


Not allowed perform sensitive operations via gremlin

Implement #145
Change-Id: I9a590fe40d3b5a808b569ed0af8fd83214a2941a

only check in the gremlin threads, also consider asynchronous gremlin job


Not able to install a package in Ubuntu 16.04

Hi,
I have been using Ubuntu 16.04. and I tried to install from packages using, 'sudo dpkg -i inbucket_VER_linux_amd64.deb'. But it always throwing an error saying "cannot access archive: No such file or directory".
Is there any other way to install inbucket from packages .?

Hi, so you need to first download one of those packages (they aren't part of Debian or Ubuntu's repositories) from here: https://github.com/jhillyerd/inbucket/releases
When running the command, you will need to replace VER with the version you downloaded:
sudo dpkg -i inbucket_2.0.0_linux_amd64.deb


Autorouter with "auto-cloud" is only routing a single trace

CC @imrishabh18, my guess is this is in the registry-api, maybe we're not waiting for the job to finish? Needs investigation
Here's the reproduction:
https://tscircuit.com/editor?&snippet_type=board#data:application/gzip;base64,H4sIAEUCQWcAA62SvQ7CMAyEd57CygQDkAIj6YLEAzDBGNKURtA6Sl2BhHh3Qkn4lYCBLd/JlztL1geLjiDTuWx2BN0eiBS6HYDpGqXLYG8yKgRLeFkyKLTZFBRJNoQOG9JOsMu7r3bYZCz1Zm93uja1H2gJ4IqyUtrbtyyoOSJZZyr/J5/wUZQrWfqxRRK5VsVSHMengFat7zj8X96NP+W1uHqLV9JKZe75gUMBzrmd/1Ri9rJ0/6VF5JB7DScnlYbcYSnYYJFACgNrqoQBoRdmD0Ks++wYfXNMh+01pJ3eGWy5+44wAgAA

@seveibar  it is routing two traces in schematic but only one in PCB what is you thought.

and it is routing two traces if not the same port.

@seveibar Will take a look at this

@seveibar This is something related to the freerouting, as you can see in the freerouting for this circuit, we can see the white source trace connecting both of them to the C1 pin, but autoroutes only one.   have tried editing this and changing the dsn file format, but freerouting is routing only one trace, when one pin is connected to two different pins.
https://github.com/user-attachments/assets/35b2818e-c135-4a25-ace4-ac0b525b41f6
Claude's response:

@seveibar Another thing, I have noticed is that if I am changing just the X position of R2 on the circuit you have linked above. The freerouting is producing both the traces.
https://tscircuit.com/editor?&snippet_type=board#data:application/gzip;base64,H4sIALA4RGcAA62RvQ7CIBSF9z7FDZMOKlVH6WLiAzjpiJRaou0l9DaaGN9dVPCvgw5ufCf38HGDPlp0BLkuZLsn6PVBZNBLAGYblC6Hg8mpFCzlVcWg1GZbUiTZEjpsSTvBrueB2mObs8yXfd3pxjR+4EYAd5S10r6+YyEtEMk6U/s7+ZSPY1zLyo8t08iNKlfiNDkHtGrzxNH/fA/u+tYdn5JWKvMUBg5Gzrld/GSdf2w5+FgzcvDe5eSk0lA4rAQbLlPIYGhNnTIg9MH8JYjPfW+MvzVmo9v3Z0k/uQBoZeeYIgIAAA==


Add support for SO_REUSEPORT to uv_tcp_t?

Evaluate if we want to do this. It's now supported on Linux >= 3.9 and RedHat seems to have backported it.
This would add a new flag to uv_tcp_bind:
...
r = uv_tcp_bind(handle, addr, UV_TCP_REUSEPORT);
...

Note that we already take flags in uv_tcp_bind.
This would help create a multi-threaded server in which the Kernel does the load balancing.
 The nice-load-balancing-thingy is a Linux-ism. I'll keep this open for a bit, food for thought.
https://lists.freebsd.org/pipermail/freebsd-net/2013-July/036131.html

For reference, I explicitly disabled it in 3d2c820 ("linux: don't turn on SO_REUSEPORT socket option".)

I'm +1 on this.
I have already benchmarked SO_REUSEPORT and cluster using this branch : https://github.com/yosuke-furukawa/libuv/tree/feature_turn_on_reuseport
Simple Compare:
SO_REUSEPORT 2113.25 req/sec
cluster 1431.47 req/sec
47% faster.

The detail is here. https://gist.github.com/yosuke-furukawa/e3d593db8f8963518b19
I know SO_REUSEPORT has the different behavior on BSD and linux. http://stackoverflow.com/questions/14388706/socket-options-so-reuseaddr-and-so-reuseport-how-do-they-differ-do-they-mean-t
But I want to use SO_REUSEPORT on the linux kernel >= 3.9 only.
If you can accept the behavior, I want to send a pull request.

I'm not sure, to be honest. The problem is that this behavior cannot be amulated on platforms which don't support it (yet?) so this would exclusively work on Linux >= 3.9 and only there. That's not cross-platform enough, by eny measure.
Now, it shouldn't be hard to use uv_fileno, extract the fd and set the option manualy, BUT we create the sockets late, which means that we currently don't have a way to get the fd before calling uv_tcp_bind.
I've started to experiment with all of this as a libuv + http_parser thingy (hopefully to be published soon) and the only way to do it currently is to create the socket manually, set SO_REUSE{ADDR,PORT}, use uv_tcp_open and then use the tcp handle normally.
This would be much easier if we created the sockets early, oh well! :-)
So, I myself am -0.1 at the moment. I hope that experiment I'm working on might give me some more thoughts on what to do here.
PS: In my experiment I'm using multiple threads not processes, FWIW.

Yes. That is not cross-platform enough. I guess it is hard to explain the specification for libuv. And that is hard to emulate.
I don't need to turn on SO_REUSEPORT by default. It is no problem to add a flag like UV_TCP_REUSEPORT. :-)

So, I myself am -0.1 at the moment. I hope that the experiment I'm working on might give me some more thoughts on what to do here.

Yes, I agree. We need more experiments on this.

I'd like to use this as well (+on UDP as well). I understand the motivation behind 3d2c820 so I guess it's best if there's a UV_{TCP|UDP}_REUSEPORT flag (that is ignored if the platform doesn't support it).
It's somewhat meh going the uv_{udp|tcp}_open() way.

After thinking about this some more for the past few days, I came to the conclusion that it's not a good idea to provide an API for this at the moment, since the platform differences are irreconciliable. It's not the fact that some platforms have SO_REUSEPORT and other don't, it's the fact that even if it's called the same it can mean different things.
So, I think the right thing to do here is to make it easy to set if from the outside. In order to achieve this we need to create the sockets early, because right now they are created too late (when binding). This was planned for a while: https://github.com/libuv/leps/pull/5, so I started implementing it here: https://github.com/libuv/libuv/pull/400 (look at that beautiful round number!)
Usage (pseudocode-ish) example:
uv_tcp_init_ex(loop, handle, AF_INET);
uv_fileno(handle, &fd);
setsockopt(fd, SOL_SOCKET, SO_REUSEPORT, &on, sizeof(on));
uv_tcp_bind(...)
/* profit! */

+1
In NodeConf, I talked about SO_REUSEPORT with libuv contributors.
SO_REUSEPORT is really confusing feature for cross platform environment.
libuv does not support the SO_REUSEPORT. Sounds reasonable.
And creating sockets earlier is good idea.

(look at that beautiful round number!)

cool!!

That would work for me as well.

I'm going to close this then, since we seem to ha A Plan :-) Let's continue over at #400.


Support contextmenu event

Hello again !
If I am not mistaken, right click event (contextmenu) is not yet supported on layers.
Could you please add it ?
Thank you

Released in v0.3.5


Update flow-bin version to 0.36.0

Hey - sorry for taking a while to get back to you. Thanks for doing this.
Would using tilde(~) instead of carat(^) make more sense here while flow is pre 1.0?


Hello

I tried many times but get stuck
Can I have your telegram please? Thank you

@fadyOne you have a version of this repository with a lot of upgrades in https://hackedwalletrecovery.com/
we have moved this repo to the BuidlGuidl, try it there. also to increase the chance to success, try to do it when the ethereum network fees are low


Can't create file on "advert" folder

Hi,
I use Codiad v2.7.4. When I create a folder named "advert", I can't create any file on this folder.
I had the same strange bug on Live Demo.
Thank you!

Do you have an adBlocker installed?


Use updated Application CRD ( apiextensions/v1beta1 to v1 )

after https://github.com/argoproj/argo-cd/pull/5516 is merged.
Creating this issue as an FYI though I wouldn't expect any change in how the Golang dependency is consumed.

Thanks @sbose78, ApplicationSet controller switched to non-beta v1 version with https://github.com/argoproj-labs/applicationset/pull/128, and I've confirmed that Argo CD's switch does not affect our functionality.
I've swapped the integrated Argo CD + ApplicationSet controller install YAML to Argo CD's master branch (eg what will become v1.9.0, as there isn't a v1.9 tag/branch yet), and everything works as expected there. (It will officially swap on release of ApplicationSet controller v0.1.0.)
So we should be good to go, absent any additional issues found during testing.

Thank you!