If the tag has a space, formatTags not working

update
There is no issue here. I figure it out.
Hello again, I found another bug in this plugin.
for example:
Here is my tag(all from backend data)
#hello world #helloword

when I call .formatTags(), only the second one can be formatted.

Here is another issue I found it.
For example, here is my text

if the cursor is in the middle of hello and word, click any emoji, always add to the end.

Here is another issue I found it.
For example, here is my text

if the cursor is in the middle of hello and word, click any emoji, and always add to the end.
but I don’t think it's the bug, my solution is that find the offset and insert it. however  the len of formattedText and text is different, I am still trying figure it out how to insert the correct position.
 final int cursorPosition = _contentTextController.selection.baseOffset;
 String formattedText = _contentTextController.formattedText;
 String newText ='${formattedText.substring(0, cursorPosition)}#${formattedText.substring(cursorPosition)}';

@Amber916Young what is the correct way to work with spaces in a tag? I am having the same problem.

you can change it in the _buildTextSpan and _formattedText function and reformat the splitter text @flodaniel @Amber916Young

Both are private functions. Does the package just not offer formatting of values with special characters in between without having to dive deeper into the implementation/overriding the existing implementation?

Hi folks. I’ve been following this conversation via email notifications. I’m currently swamped with my 9-5 but I thought to pop in and drop this.
I’m going to respond to everyone in a few hours.

Hi folks. I’ve been following this conversation via email notifications. I’m currently swamped with my 9-5 but I thought to pop in and drop this. I’m going to respond to everyone in a few hours.

Appreciate you taking the time! I am looking forward to seeing your approach :)

Apologies folks, a few hours turned into days.

@flodaniel currently, it's impossible to get tags with whitespaces to work.
You can currently update the search regex to allow for whitespaces to be a part of the search query. But the added tag will not be treated correctly because the current implementation splits the text by whitespace before trying to format the tags.
That was an assumption I made that tags wouldn't have whitespace. I have an idea to fix it. It'll involve doing an autocomplete suggestive search with the trie which would allow for whitespaces and other kinds of characters, really.
That may take me some time to implement. If you want to take a stab at this and submit a PR, you're more than welcome.
You can check out an implementation of prefix autocompletion with the trie here

@Amber916Young I'm thinking I can introduce a cursorPosition getter which will be corresponding position of the cursor in the formatted text and with that you can do something like:
 onTap: (emoji) {
      final cursorPosition = controller.cursorPosition;
      final substring =
          controller.formattedText.substring(0, cursorPosition);
      final newText = substring +
          emoji +
          controller.formattedText.substring(cursorPosition);

      controller.text = newText;
      controller.selection = TextSelection.fromPosition(
        TextPosition(offset: controller.text.length),
      );
      controller.formatTags();
  },

That should fix the issue for you. Let me know if you think otherwise.

@Amber916Young I have introduced cursorPosition in v2.2.1 which tells the position of the cursor in the formatted text.
I've also updated the emoji onTap function in the example.
onTap: (emoji) {
    final baseOffset = controller.selection.baseOffset;
    final cursorPosition = controller.cursorPosition;
    final substring = controller.formattedText
        .substring(0, cursorPosition);
    final newText = substring +
        emoji +
        controller.formattedText.substring(cursorPosition);

    controller.text = newText;
    controller.formatTags();

    controller.selection = TextSelection.fromPosition(
      TextPosition(offset: baseOffset + emoji.length),
    );
  },

It's important to set the TextSelection relative to the base offset of the previous selection.

@flodaniel currently, it's impossible to get tags with whitespaces to work. You can currently update the search regex to allow for whitespaces to be a part of the search query. But the added tag will not be treated correctly because the current implementation splits the text by whitespace before trying to format the tags.
That was an assumption I made that tags wouldn't have whitespace. I have an idea to fix it. It'll involve doing an autocomplete suggestive search with the trie which would allow for whitespaces and other kinds of characters, really. That may take me some time to implement. If you want to take a stab at this and submit a PR, you're more than welcome.
You can check out an implementation of prefix autocompletion with the trie here

Hi @Crazelu, thanks for your reply and working on this.
In the meantime I managed to resolve it replacing all whitespace with a non-whitespace, but looks like whitespace character:
 final bugfixedText = displayedText.replaceAll(' ', '\u00A0');

@Amber916Young I have introduced cursorPosition in v2.2.1 which tells the position of the cursor in the formatted text.
I've also updated the emoji onTap function in the example.
onTap: (emoji) {
    final baseOffset = controller.selection.baseOffset;
    final cursorPosition = controller.cursorPosition;
    final substring = controller.formattedText
        .substring(0, cursorPosition);
    final newText = substring +
        emoji +
        controller.formattedText.substring(cursorPosition);

    controller.text = newText;
    controller.formatTags();

    controller.selection = TextSelection.fromPosition(
      TextPosition(offset: baseOffset + emoji.length),
    );
  },

It's important to set the TextSelection relative to the base offset of the previous selection.

Thanks for your reply, I am testing the plugin

@flodaniel currently, it's impossible to get tags with whitespaces to work. You can currently update the search regex to allow for whitespaces to be a part of the search query. But the added tag will not be treated correctly because the current implementation splits the text by whitespace before trying to format the tags.
That was an assumption I made that tags wouldn't have whitespace. I have an idea to fix it. It'll involve doing an autocomplete suggestive search with the trie which would allow for whitespaces and other kinds of characters, really. That may take me some time to implement. If you want to take a stab at this and submit a PR, you're more than welcome.
You can check out an implementation of prefix autocompletion with the trie here

Hi @Crazelu, thanks for your reply and working on this. In the meantime I managed to resolve it replacing all whitespace with a non-whitespace, but looks like whitespace character:
          final bugfixedText = displayedText.replaceAll(' ', '\u00A0');

          tagController.addTag(id: connectedUser.username, name: bugfixedText);

This is a clever hack!

@Amber916Young I have introduced cursorPosition in v2.2.1 which tells the position of the cursor in the formatted text.
I've also updated the emoji onTap function in the example.
onTap: (emoji) {
    final baseOffset = controller.selection.baseOffset;
    final cursorPosition = controller.cursorPosition;
    final substring = controller.formattedText
        .substring(0, cursorPosition);
    final newText = substring +
        emoji +
        controller.formattedText.substring(cursorPosition);

    controller.text = newText;
    controller.formatTags();

    controller.selection = TextSelection.fromPosition(
      TextPosition(offset: baseOffset + emoji.length),
    );
  },

It's important to set the TextSelection relative to the base offset of the previous selection.

Thanks for your reply, I am testing the plugin

Great!
I’ll be closing this now. You can reopen with a comment if there’s still an issue here.


Data license

Hi,
under what license is the data published? is it available for commercial usage?

@eran-deci  Hi, thanks for your attention on this work. This dataset can only be used for research purposes only.


Azure SQL Server state store changes order of transaction

Similar to #1209
Expected Behavior
For the following transaction:
upsert key1 value1
delete key1
upsert key2 value2

Expectation is that key1 should be deleted, key2 should have a value of value2.
Actual Behavior
key1:value1 is present, along with key2: value2.
Steps to Reproduce the Problem
Please use the JS snippet from the referenced issue (1209)
Details
In the transaction snippet for this state store, the delete operations are always executed before upsert operations. This results in failure with message Deletion is always attempted before upserts, and fails with message delete affected only 0 rows, expected 1 whenever the above tx is attempted.
Release Note

RELEASE NOTE: FIX Bug with ordering of operations in Azure SQL Server state store transactions.

/assign


Pydian partials: have nicer ways to work with lists

Problem

Working on a case where the result of a get is a list[tuple] where I want to do something to each tuple in the list while keeping the list structure, so the apply= parameter is kind of useless because it forces me to apply to the list first (which makes sense)
Can get around this
Requested feature

Maybe a foreach function? E.g.
from typing import Any
from pydian import get
import pydian.partials as p

def swap_tup_order(t: tuple[Any, Any]) -> tuple:
    return (t[1], t[0])

data = { 'list_tups': [(1, 2), (3, 4), (5, 6)] }
print(get(data, 'list_tups[::-1]', apply=p.foreach(swap_tup_order))) # Expect: `[(6, 5), (4, 3), (2, 1)]`

Alternatives considered

Maybe just have people use list comprehensions (more "pythonic"), though could get hard for nested cases

Additional context

From working with OpenAI Whisper model -- audio chunks are passed as tuple of timestamps and corresponding text

Another partial thing: for p.keep provide more intuitive handling for indices. E.g. I want to keep the last two -- isn't a way to do this now (in Python: some_list[-2:])


[add-resource] New Plugin -- Server-side syntax highlighing with highlight.php

Hi, Folks --
I would like to add my new plugin, which adds server-side syntax highlighting courtesy of the highlight.php library and the ShortCode Core plugin, to the Grav Repository.
The project lives here: iainsgillis/grav-plugin-highlight-php
FWIW, I wrote a series of blog posts about conceiving, designing, and releasing this plugin on my own site, starting with https://www.iainsgillis.com/essays/build-a-grav-plugin-1

Many thanks for this review -- I appreciate the suggestions, and the rationale.
I'll make some revisions in the coming days.
On Mon, Mar 21, 2022, at 07:54, pamtbaau wrote:

@iainsgillis https://github.com/iainsgillis, I've taken the liberty to create a review of your plugin.
Suggestions:

HiglightPhpShortcode.php

Use shortcode with parameters
`[phphighlight=js]
class Rectangle {
constructor(height, width) {
this.height = height;
this.width = width;
}
}
[/phphighlight]

[phphighlight=js code="console.log('hello')"]
`
For three reasons:

Only load shortcode for requested language(s), instead of 349. It saves a bit of performance.
No pollution of the shortcodes namespace with 349 extra names.
No need to search for 349 languages in Markdown while they are not used anyway.
Use a fixed location for custom styles. I don't see a real added value for a customisable location.
My suggestion would be to use folder: /user/data/hightlight-php/. I prefer the user/data/ folder to not pollute the user folder. Drop the folder property from the config.
Use two config properties for styles: style for build-in styles and customStyle for a custom style. Add a None value to both.

User can now

Select a build-in style or None
Select a custom style or None
Select both build-in and custom style to override the build-in style.

Load each style (if value !== 'None') as separate asset with customStyle as last.

Add a property in a page to enable/disable the plugin.
I presume a page using snippets will be the exceptions instead of the rule. Test for it during onPageInitialized.

—
Reply to this email directly, view it on GitHub https://github.com/getgrav/grav/issues/3565#issuecomment-1073926614, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACR666B2MJTTT3G5V5VFLC3VBB5TDANCNFSM5RGDWE7Q.
You are receiving this because you were mentioned.Message ID: @.***>

@iainsgillis,  Forgot to mention that the following condition will never be true, because the plugin will not be called when not enabled. It can be removed.
if (!$this->config->get('plugins.highlight-php.enabled')) {
    return;
}

@pamtbaau Many thanks for the review and the suggestions! Really appreciate your time.

Sorry missed this one, added.


Passing through fastlane command line options to actions

How do you pass through fastlane command line options to actions?
For example, how would you specify the device on the command line for scan?
These do not work:
$ fastlane test device:"iPhone 6s"
$ fastlane test "device:iPhone 6s"
$ fastlane test device="iPhone 6s"
$ fastlane test --device "iPhone 6s"
Thanks in advance.

Can you share your test lane? You'll need to pass parameters like this: https://github.com/fastlane/fastlane/blob/master/docs/Advanced.md#passing-parameters and then pass the device to the scan action

Hi - Thanks Felix! Yes, sure, it's just the basics right now:
desc "Runs all the tests"
lane :test do
  scan(scheme: "Scheme")
end

Based on your comment, I changed to this (and it works):
desc "Runs all the tests"
lane :test do |options|
  scan(scheme: "Scheme", device: options[:device])
end

So that's awesome!
But what about a general way to do this for all options? Meaning without having to pass them through this way in code. It would be nice to be able to pass action specific options when running fastlane, no? Am I missing something?
Thanks

@chris-fuller that's the way to do it, looks good! :+1:

Interesting, so fastlane should try to automatically apply the given parameters to the actions if the name matches? I wonder if that causes troubles if there are actions with the same parameter name, but different meaning..

Thanks @KrauseFx! Appreciate it.
Yes, I suppose that was my thinking. I do not know enough about Fastlane to know if this is possible or problematic. It just seemed like being able to passthrough options from the command line was useful to me. I will accept the fact that this is by design, if that's the case.
Thanks again.

revising issue title to be more apropos for this conversation

I'll mark this issue as a feature for now, it's a good idea, however it might result in a some side effects.

Agreed. What about an advanced CLI that accepts 'namespaced' params? Something like:
$ fastlane test scan:key:value
Just a thought. No idea if this makes sense or not.

I'll close this issue for now, as we're not actively working on it :+1:

@KrauseFx this link is broken: https://github.com/fastlane/fastlane/blob/master/docs/Advanced.md#passing-parameters
What's the proper syntax for passing through a parameter?

Ah never mind, I got it:
fastlane test device:"iPhone 6s"

👍

Fixed link: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Advanced.md#passing-parameters

I like @chris-fuller 's proposal of namespaced options without having to add the manual options passing in code.  For example fastlane mylane deliver:skip_screenshots:true or more generally fastlane <lane> <action>:<options>:<value>
I could be wrong, but it seems like this avoids any ambiguity in terms of passing the right options/value pairs to the right actions. For example if there's not deliver action in the lane mylane it does nothing, and likewise if there is no skip_screenshots options in the deliver action, it also does nothing.


Disable/Enable Services

When service is disabled, the applications under all these service should all stop being authed.
Service can be enabled back. Apps start authorizing again.
Only public API endpoints should be affected by service state. Internal API should work regardless service state.

@eguzki I think this can be closed now?


Remove std.xml, std.digest.digest, and all references to those deprecated modules

The next major release will be 2.101, these are flagged to be removed in that release.

Blocked by dlang/tools#450.

Unblocked.


add autoscaler for queriers

What this PR does:
Adds a HorizontalPodAutoscaler for the queriers.
https://cortexmetrics.io/docs/architecture/#querier

Queriers are stateless and can be scaled up and down as needed.

Checklist

[x] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

It'd also be nice if we could add HPA for every component that can safely be autoscaled.

It'd also be nice if we could add HPA for every component that can safely be autoscaled.

I've been adding them one at a time as a learn/recognize that they are safe to autoscale, and by what metric. If someone with more operational experience could share appropriate scaling metrics for each component, I could add the autoscalers.


Add windows support through windows scheduled tasks

I needed to support chef-client updates on windows and I get from IRC channel for chef that in last chef summit the discussion pointed that this should be done through scheduled tasks.
I have gave it a try.
One thing I have not been able to workaround is the windows dependency on the metadata, as if you are using omnibus updater for linux, this has no sense, but I cannot find how to do conditional dependencies.
Modified the cookbook to only upgrade windows (msi installer does not support downgrade, although it could be achieved deleting current version and installing new one)

Please target the Develop branch. We do not merge PRs directly to master. Thank you.

PR #101 adds Windows support. We should be merging that for a new releases shortly. Please review that PR to see if there's additional functionality you require.


TNO-327: Pager styling

Disabled button was working prior, background color was the same when the page wasn't properly expanding (which has been fixed in a separate PR already merged), top margin removed.
I had spent some time on trying to figure out that local bug, but have stopped investing time in it for now (last index not working locally).
We could make the disabled colour more obvious if needed.

@Fosol, updated. Guessing the colour will need a revise but that is easy enough.

Not the prettiest blue, but it'll do for now.


//sw/device/tests:chip_power_sleep_load_sim_verilator hangs

This PR affects a verilator test that times out.
Test appears to stop making progress after printing:
I00006 chip_power_sleep_load.c:338] PWM active

Maybe the OTP isn't properly configured for the verilator test and otp_ctrl is hanging?
Or maybe it's getting configured and servicing the checks is preventing the test from making progress?
Originally posted by @drewmacrae in https://github.com/lowRISC/opentitan/issues/16541#issuecomment-1343165772

I think it's always timed out without making progress in verilator, so I'm inclined to mark it as broken.

I need to take a closer look to see what is happening.
@yehuda-bejar @moidx, can this test operate without the associated vseq, or does it rely on it to make progress? I think it can operate without, but just wanted to double check.

The fpga flavors work without the vseq and pass.

OK, then we may have to tweak the timeouts (the clock ratios are different).

It seems to stop in the middle of processing the LOG_INFO for OTP. Would we have gotten an alert somehow? Might need --verilator-args=--trace to find out what's happening.

Ahh, so it turns out the gpiodpi module hangs up simulation. Once the PWM turns on, gpiodpi gets busy writing all the changes it sees to the gpio-read FIFO. And once that FIFO fills up... the simulation blocks on the write() syscall, waiting for FIFO space.

Triage: verilator only issue. Looks like something to do with GPIO.


win32: Fix test log

On Win32, &t_md and &t_me are specific to Vim. Writing them to the test
log doesn't make sense.
Use normal ANSI escape sequences. They can be shown correctly on the
recent Windows 10.

Now the log is shown correctly also on AppVeyor.


Add manpages, manpages-posix to C++ and kitchen sink, possibly other images

We've had requests to add a few more manpages into these images.

manpages and manpages-dev we should just be able to add to common-debian.sh
However, manpages-posix and manpages-posix-dev are not in the "free" sources for Debian (though they are available by default in Ubuntu). We'd also need to add non-free sources to include them. On the surface this does not bother me for a general purpose dev image, but we'd likely want to have it as a script option for those using it themselves.

This adds 6mb to the uncompressed image. The zipped image that is downloaded will be much smaller so it shouldn't significantly impact aquisition time.
//cc @2percentsilk @joshspicer @jkeech

https://github.com/microsoft/vscode-dev-containers/blob/master/script-library/common-debian.sh


Fixed typo

PACPAC => BACPAC

@OskarKlintrot : Thanks for your contribution to the Azure documentation! The author, @janeng, has been notified to review your proposed change.

#sign-off
@OskarKlintrot thank you


Request to update dictionary to fuzz on smart values

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (f625d0f 2024-04-02T00:16:42.810315000Z)
What command(s) is the bug in?
forge test
Operating System
None
Describe the bug
We recently manually caught an outdated fuzz test that should have thrown had a correct input been chosen. See this PR for reference/repro
https://github.com/Uniswap/v4-core/pull/529/files

#7552 should decrease number of runs this tests needs to fail, however, it will still be very high.
This is happening due to test having several branches, and most of the runs are not hitting the one which should fail, thus decreasing its chance to be executed with failing input.
I'd recommend rewriting this test into several separate tests to ensure that all scenarios are covered by high number of runs

cc @grandizzy do you see any actionable item for this ticket?

cc @grandizzy do you see any actionable item for this ticket that we can improve upon?

I think that's more suitable for symbolic testing which we track to implement with https://github.com/foundry-rs/foundry/issues/15


Adds child_spec to Lapin.Supervisor

Supervisor.Spec.supervisor/2 is deprecated in favor of child_spec.

Looks good, thanks!


Update Rails 5.2 gemfile to use 5.2.3

In order to keep the CI running with latest versions in the tested gemfiles, this PR updates the Rails 5.2 version to the latest patch version of Rails.

Cool, is this ready to merge?


Check start, end date behaviour on get_* functions

xarray/core/indexes.py:97: FutureWarning:

Indexing a timezone-naive DatetimeIndex with a timezone-aware datetime
is deprecated and will raise KeyError in a future version.  Use a timezone-naive object instead.

Is this warning still being printed? If I run the tests on the command line (with -s) I can't see it in there but was there another way to produce this?

Running tests in tests/analyse/test_scenario.py results in these warnings.

@gareth-j - what version of xarray are you running with when you get these warnings?


[AIRFLOW-590] Set module parameter in OracleHook

Dear Airflow Maintainers,
Please accept this PR that addresses the following issues:

https://issues.apache.org/jira/browse/AIRFLOW-590

Please let me know if 'master' is the right target branch or you prefer this targeted elsewhere.
Testing Done:

Tested by hand on Oracle XE.

Also solved 2 flake8 issues.

lgtm.. though we dont use oracle so can't comment on the impact of the change..

+1

+1


format

Add functionality to handle the output. For instance, to fit a telescope operator needs:

14:45:53.58	+15:31:57.50 ---> 144553.58	153157.50 [deadline: may]

Check the way data is formatted before being saved. At the moment the code repeats the formatting to save on files with simple and detailed information about visible satellites. Finally, check how to create a list [or equivalent] so once a satellite is visible, the code appends the visibility of the satellite to it. The idea is to use this list as shared memory, rather than returning all via the pool of workers: visible_satellites = mp.pool(...)

Redefine variable names and logic behind output to comply with flake8 and pylint. Split issue in smaller issues


Update Jedis example to use latest release

The Jedis example is currently on a snapshot version of Jedis, update it to use the latest release when out (needed a command that wasn't released yet).

Update will be needed here https://github.com/redislabs-training/ru204/blob/740447636bd8dd390d110d341959377d0ca95765/src/java/jedis_example/pom.xml#L31

https://github.com/redis/jedis/releases/tag/v4.3.0-m1

Move to 4.3.0 when it is fully released... latest is still a pre-release... https://github.com/redis/jedis/releases/tag/v4.3.0-m1

Looks like a full 4.3 release is now out https://github.com/redis/jedis/releases/tag/v4.3.0


chore: explicit error messages from put sync

Checklist

[ ] I have read the coding guide.
[ ] My change requires a documentation update, and I have done it.
[ ] I have added tests to cover my changes.
[ ] I have filled out the description and linked the related issues.

Description

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

This change is

Thank you for picking up the depthmonitor promotion.  I'm not a git expert and was considering a whole new PR just to change the comment.  Google didn't help as apparently you cannot rename a comment text directly on github.  One of these days I'll remember to do the proper format commit the first time!   (And I'll take the time to clean up my fork to be able to use it locally more effectively!)

@ldeffenb You can try git commit --amend "new commit msg" and git push -f
to your branch which will update the branch on github.
Anyhow, thanks for the commit!


Suggest a new tag/release

The current tag/release is quite old now. I suggest creating a new release as the current version is much better then the old one.

https://github.com/mjibson/esc/releases/tag/v0.2.0


Hide Execution Time

Hi All,
I am trying to find the option for hiding or disabling the execution time but unable to get the option. I tried searching on google but no luck. Can someone please help me with this issue?

Thanks

You can hide the whole status bar with the following setting in your user settings.json:
    "notebook.showCellStatusBar": "hidden",

@joyceerhl it worked. Thanks for the help. Can you also share the documentation for such settings?

Here's the list of all notebook layout settings: https://code.visualstudio.com/updates/v1_57#_notebook-layout-customization


tuple return values should be namedtuple instances

In most cases a tuple return value would be better represented as a namedtuple instance instead.  Especially for functions such as GetNamedPipeHandleState which have several return values that you'd otherwise have to read the docs for.  This also makes it easier to specify the return type for a given function too.

Closing, this is already the case in the non core/dev functions.


Use headless Chrome instead of Capybara-Webkit

This repo has been using capybara-webkit for testing, but that gem depends on an old version of qt.  Then, when I want to do anything else, I have to undo it all. This has cost me hours of work.
This PR migrates the tests to Selenium with headless Chrome to avoid unnecessary dependencies on software that cannot be installed via the Gemfile. This will make it much easier for me to run tests, and should do the same for others as well, at least for Mac users who rely on Homebrew.
By default, Selenium uses Firefox, but that has some other non-gem dependencies that led me to choose Chrome instead. I also tried Apparition without success before using Chrome.
I've left the Capybara-Webkit-specific code in case we ever go back to it once it receives updates. It is wrapped in conditional statements anyway, so I didn't have to modify or comment it out except to remove the line require 'capybara/webkit'.
While I was at it, I also:

removed duplicate settings from the RSpec config
removed a workaround for a previous version of bundler from .travis.yml
added a build for Ruby 2.6 and Rails 6.

Unfortunately, a database issue is still preventing some jobs from passing. That's a pre-existing issue, though, and nothing to do with this PR. All the jobs that passed before this PR still pass. Running tests locally with newer Ruby/Rails fails when running the whole suite, but passes when running one directory at a time.

Update: I removed transactional-capybara. Tests are green.

Thanks so much!


call load_aws_credentials() in a docker container doesn't load credentials

What happens?
When running duckdb inside a docker container, the function call load_aws_credentials(); gives empty rows. Running it outside the container works.
To Reproduce
Setup 1 (Local machine)

Install duckdb with brew install duckdb
Configure aws cli (in my case sso)
check with aws s3 ls that you can list the buckets in the chosen profile
List files in the chosen bucket
Open duckdb
In the the duckdb console run call load_aws_credentials();
It should display the secrets
Run select * from read_csv(s3://bucket_path)
It works
Sample output of call load_aws_credentials(); locally

┌──────────────────────┬──────────────────────────┬──────────────────────┬───────────────┐
│ loaded_access_key_id │ loaded_secret_access_key │ loaded_session_token │ loaded_region │
│       varchar        │         varchar          │       varchar        │    varchar    │
├──────────────────────┼──────────────────────────┼──────────────────────┼───────────────┤
│   access_key_id      │  secret key              │       session token  │ region        │
└──────────────────────┴──────────────────────────┴──────────────────────┴───────────────

Step 2: Docker container

Build a docker image
Install duckdb, boto3, awscli, also the duckdb binary
Run the docker container with your local .aws directory mounted to the one in the container
Inside the docker container, run aws s3 ls you might need to do an export AWS_PROFILE=profilename
If you can see the buckets listed, it means that credentials sharing between the container and host works fine
run duckdb with ./duckdb
Run call load_aws_credentials();
It doesn't show anything, even after specifying the name of the profile with call load_aws_profile('profilename')
Running it from python also does't work

Dockerfile
FROM python:3.11-slim
RUN pip install poetry --no-cache-dir
RUN apt update && apt install awscli wget unzip -y
RUN wget https://github.com/duckdb/duckdb/releases/download/v0.10.1/duckdb_cli-linux-amd64.zip
RUN mkdir /app
RUN unzip duckdb_cli-linux-amd64.zip -d /app
RUN rm duckdb_cli-linux-amd64.zip
WORKDIR /app

Build the container
docker build -t duckdb .

Command to run the container locally
docker run -it -v $(pwd):/app -v /Users/home/.aws:/root/.aws   duckdb /bin/bash

once inside the container, run duckdb with
./duckdb and then run call load_aws_credentials()
They're empty as it is shown here
As it shows here, running the command inside the container creates an empty table like this
┌──────────────────────┬──────────────────────────┬──────────────────────┬───────────────┐
│ loaded_access_key_id │ loaded_secret_access_key │ loaded_session_token │ loaded_region │
│       varchar        │         varchar          │       varchar        │    varchar    │
├──────────────────────┼──────────────────────────┼──────────────────────┼───────────────┤
│                      │                          │                      │ region        │
└──────────────────────┴──────────────────────────┴──────────────────────┴───────────────

Since the .aws directory is shared, the credentials are there and with the awscli installed in the container, you can check if they're valid with
aws s3 ls
What could it be?
I think it might've something to do with permissions on the files in .aws, on how these credentials are accessed. Unfortunately, I don't know how to debug or trace what happens when the function is called for.
OS:
x64 docker, Mac OS aarch64 host
DuckDB Version:
v0.10.1 4a89d97db8
DuckDB Client:
Bash, Python
Full Name:
Pratyay Modi
Affiliation:
None
Have you tried this on the latest nightly build?
I have tested with a release build (and could not test with a nightly build)
Have you tried the steps to reproduce? Do they include all relevant data and configuration? Does the issue you report still appear there?

[X] Yes, I have

Hi @pratio,  load_aws_credentials() function is now deprecated, could you try out the secret based flow and retry?

@samansmink I'll try

@samansmink It works, Thank you


Appreciation

This is by far the best single purpose bot in the slash era for adding polls to your discord guild.
Long may it live.
I went looking for alternatives when my full-featured pre-slash bot stopped working.
Was amazed by how many of the most popular "poll bots" are glorified embed creators with a few reactions attached.
There's a few decent ones but they either lack features (no anonymized polls, no timed close, no multiple selection, no aggregates) or advertise features that do not actually work.
Instant-poll is a breath of fresh air.
Good job.

Thank you for the kind words!


It'd be nice if a new pane started in the same folder as the one you split from

Environment
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]
Windows Terminal version (if applicable):

Any other software?

Steps to reproduce

Expected behavior

Actual behavior

Thanks for the suggestion! Please make sure to search for duplicates when filing new issues. This is already being tracked by another issue on our repo - please refer to #3158 for more discussion.

/dup #3158 "Open new terminal tab in same directory as existing tab (OSC 7?)"
/dup #7850 "split pane "splitMode:duplicate" not working."
/dup #7734 "Easy way to duplicate tab and preserving current working directory"
/dup #7544 "duplicate tab: working directory not duplicated"
/dup #7049 "🔥 [wt +Powerline ] duplicateTab not working"
/dup #7022 "Open another profile in same directory"
/dup #1437 "Feature Request: Option to open new tabs in the same directory as the current tab"
/dup #4259 "Open a new pane with the environment vars of the parent"
/dup #4970 "Feature Request: duplicate current path with panes"


perf improvement on target with avx2 and windows OS

Description
Remove unnecessary mixed xmm/ymm/zmm instructions, which prevent ooo instructions to retire.
Ticket
54930

@dmitry-gorokhov , could you please review?


Plotting the convergence parameters to analyse DDP solver - [merged]

In GitLab by @cmastalli on Nov 15, 2018, 17:46
Merges solver -> devel
This PR allows us to record the solver performance through the iterates inside the solverParams. This feature is important to  compare the actual DDP solver against previous one (see #6), and it tackles the task #27.

In GitLab by @cmastalli on Nov 16, 2018, 08:48
@rbudhira I guess you're not available. I would merge this request since it's not affecting any fundamental function of the DDP software.
If you find a problem here, please create an issue

In GitLab by @cmastalli on Nov 16, 2018, 08:48
merged

In GitLab by @cmastalli on Nov 16, 2018, 08:48
mentioned in commit 850efe1f5110ca5df26129f12e0b9c3caa1bebea


Key Guardians: Recover your Account (2/2)

See Ticket #794 for Part 1.
Describe the solution you'd like
Once the user has confirmed their username and wants to continue the recovery process, they will arrive to a screen that generates a link that they can send to their Key Guardians. This Link can be sent to the user's Key Guardians outside of the app.

Also on this screen, the user will see the 3 (or more) Key Guardians that they have already previously selected to help them in the event that they need to recover their account.
The Key Guardians will get a notification and an option to give the requesting user access to their account again. This will create a Green Check to appear next to the Key Guardian's name. This is to ensure the user requesting access to their account can know which Key Guardians have accepted their request and which have not.
Once all 3 Key Guardians have accepted the user's request, they will be taken to the screen where they see a countdown timer of 24 hours until the request is processed.

See #784 for how the User can cancel this process in the event that the event that this request is fraudulent.

Update: There will be an inactive "Go Home" button that activates when the timer reaches 00:00:00.


create new blank map and table properly named for 2023

current placeholder map doesn't have shading for the Monument
current placeholder map and table do not have the names they will have when all is operating

@ybarkley New 'dailyMap_2023_leg01_OES.png' and 'summaryTable_2023_leg01_OES.png' are in the outputs folder and ready to be moved to the website.
The table now has zeros for Leg 1 but I think that is ok for now...

Hey @sfregosi,
Be prepared for the uber nit-picky (sorry).
Is it possible to swap the colors for the table. I think the table headers should be the blue and then totals should be purple. Then keep the purple lines at the top and bottom of the table looks good. Last, can you change the table contents (now white) to a slight gray color: #D0D0D0.
It's kinda stupid, but the hierarchy of colors in above sections seem to be teal, purple, grayish-white, then green.
thanks for considering it,
kym

@sfregosi
I also noticed that the pre-determined transects on the blank map are gray instead of black (grey0) like the placeholder map @badgerjj sent before. There isn't a legend for them either for some reason. Could you check that as well please?

Sorry this is taking longer than expected. I couldn't find the reimaged HRP from April, so I'm redoing it now and will upload it via the network at the IRC tomorrow. I know @sfregosi you'll be on leave, but I can attempt to generate the xwavs and see if the same errors arise. Let me know.

@kmyano Working on this now. What color would you like the 'intermediate' border line above the totals to be? Also purple since the totals text will now be purple?
I'll go with that (cuz I wanna see if it pushes automagicallly to the website) but let me know if you have another idea 😁
@ybarkley Good catch... I need to dig into why the legend didn't work on my 'blank' map. I suspect it will work fine once there is data, but I'm basically forcing it to make the blank one without any data and so that could be why it isn't giving me a legend. Standby on that one.

To test for auto updating, I'm replacing the current 'placeholder' table with the table named as it will be in Leg 1.
Steps:

clicked on three dots over image and said 'replace image'
chose link and pasted in https://github.com/PIFSC-Protected-Species-Division/cruise-maps-live/blob/main/outputs/summaryTable_2023_leg01_OES.png?raw=true which I got by going to that image on github, right clicking, and selecting 'copy image link'
updated attribution and Alt Text

When you go to replace an image within the StoryMap it of course removes your description and alt text so copying here for safe keeping:

Attribution: NOAA Fisheries
Alt text: A table showing preliminary survey data from HICEAS 2023. This figure will be updated to include recent survey effort, cetacean sightings, and acoustic detections.

As of now the table is still the 'wrong' colors (totals are blue).
When I 'push' the fixed colors, want to see if it updates on own.

@sfregosi I also noticed that the pre-determined transects on the blank map are gray instead of black (grey0) like the placeholder map @badgerjj sent before. There isn't a legend for them either for some reason. Could you check that as well please?

This had to do with how I was plotting 'empty' data. I could NOT for the life of me get all the colors to work right with a empty input data frame so I just plotted one tiny dot on Pearl Harbor. It now includes the pink lines in the legend but I'm hoping that is ok/workable?
You can check it out here: https://github.com/PIFSC-Protected-Species-Division/cruise-maps-live/blob/main/outputs/dailyMap_2023_leg01_OES.png
If that is ok, I'll also change that out on the web (With the correct name for Leg 1 moving forward)

@kmyano @ybarkley Auto updating works!!!

I mapped the link to the image on the web to the generic name for this leg/ship
I ran the table code (with new colors!)
I pushed that to GitHub
An immediate refresh of the HICEAS website didn't do anything, but I opened a new browser, navigated to it, and the new colors (blue title) were there!

This same workflow will work for the map so I will replace the current placeholder map with the correct map with the leg/ship name that is generated by the code.

@sfregosi
Oh SOOOO BEAUTIFUL! :)
I love that it updates without us needing to publish it.  :D Thanks for looking into that

Thanks 😊
I'm going to close this but feel free to reopen this or a new one with any new ideas!!


Compatibility with Spring Security 6.0

The last release is not compatible with Sprint Security 6.0
Is there an update soon to fix this compatibility ?

@Snyfir thanks for reaching out. This repository has been deprecated and migrated to https://github.com/Azure/azure-sdk-for-java. So could you close this one and raise your issue to https://github.com/Azure/azure-sdk-for-java/issues? Besides, please help to provide the information of the library and version you are using, thanks.

repository deprecated

Hi@Snyfir , thanks for closing it, could you create another one in https://github.com/Azure/azure-sdk-for-java to track your issue? Also we need to know which library and version you are using to furthur invesitgate on it.


Client Dependency Reference is BLOCKING any local development

The ClientDependency.Core reference in the DNN_Platform.sln is breaking and blocking any local development on the platform. This was originally setup as a git submodule but has been causing lots of issues for several developers.
I have been speaking with the team in DNN TAG and we have a proposed solution:

Add a nuget on the DNN MyGet for the Client Dependency project https://github.com/dnnsoftware/ClientDependency
Update the platform code to reference via NuGet

I created a fork that is compiling right now with the ClientDependency.Core submodule removed. Once we get the forked repo building and producing a nuget file I will update the branch I have and submit a Pull Request

@tpluscode Yes, I just need to be given access to the DNN Company on github and access to that repository so I can create the necessary builds in AppVeyor.
This isn't really a complicated setup process at all, it is just a matter of getting access and then I can setup the builds and submit a PR

What I usually do when setting up AppVeyor on someone else's repo is to set it up for my fork, and then create a PR for the final appveyor.yml file, which they can then merge in.  The turning on AppVeyor on their end is just clicking one button.
I think it would be good for a handful of folks from the DNN TAG to have more access in the dnnsoftware organization on GitHub, but I also don't want to wait for that to get decided and acted upon before this can get moving.

It's enough for me to add ClientDependency to Appveyor and set it up for running pull requests which is pretty much the default behaviour. I will do it as soon as @ashishpd adds the admin team to the repo. Currently it doesn't show up on AppVeyor for me. When it's done you will be able to submit a PR with the appveyor.yml definition, @ahoefling.
Once it builds fine I will merge and set up environment deployment to NuGet/MyGet.

I think it would be good for a handful of folks from the DNN TAG to have more access in the dnnsoftware organization on GitHub

I do agree, @bdukes. As per the above, this itself is not a blocker to proceed with the issue at hand though. Let's discuss adding more people on GH at the TAG meeting?

@tpluscode I just created a Pull Request for the dnnsoftware fork of ClientDependency and I linked all the commits/PRs back to this parent issue. There were some issues with the default build script in our dnn fork so I commented out stuff that we are not using in the Dnn Platform.
I don't know if we got a clear direction in our NuGet feeds so the build currently produces the .nukpg and it should be dropping those packages into the appveyor project feed at https://ci.appveyor.com/nuget/clientdependency-dnn
Once we get this changed merged into the dnnsoftware fork we can address how we want to deliver this to the platform project, we have lots of options.

This issue should NOT be closed until we have a pull request merged into Dnn platform that resolves the local development scenario. I just opened a new linked issue in the CD project regarding the MyGet server
dnnsoftware/ClientDependency#6

@tpluscode once we finish reviewing #2065 and merge that PR we can close this issue

Ofc. It was closed via your commit, mind you ;)

Why is this closed? I can't build DNN Platform because Dnn.ClientDependency can't be found..

@chrisvesco This was closed due to the merged PR back in May of 2018.  Be sure to get the latest, as a local compile is successful.

I downloaded the source from https://www.dnnsoftware.com/community/download just now, unzip it try to build, and I get the Dnn.ClientDependency is missing.
FYI I went to your site and wanted to check out the DotNetNuke performance presentation and the link is broken.

We don’t have easy control on those links. If you can download the source from the Releases tab here on GitHub that should work.
Thanks on the other link ill address tomorrow

Yep thanks Mitchell if I get the 9.3.2 release from GitHub it works fine. I'm also starting to understand how MVC fits into DNN by checking out the code

Off topic: Say Mitchel where do I get your Module Professional Development book?

Looks like it is still available on Amazon. Sadly though it is a bit out of date at this point as MVC etc wasn’t around yet.
Professional DotNetNuke Module Programming https://www.amazon.com/dp/0470171162/ref=cm_sw_r_cp_api_i_O1FhDbDE7D42R


Local Domains with example-.local cause "Invalid ID given" error

Q
A

Bug report?
yes

Feature request?
no

RFC?
no

How used?
Yii2

Swiftmailer version
x.y.z

PHP version
7

Observed behaviour
My local domains have a -.  http://example-.local.
This causes an error in swiftmail.
Invalid ID given<EMAIL_ADDRESS>Expected behaviour
mime_deps.php should not allow dash.  Line 16    ->asValue(!empty($SERVER['SERVER_NAME']) && preg_replace('/(?:^[)?[a-zA-Z0-9:]]+.?/', '', $_SERVER['SERVER_NAME']) === '' ? $_SERVER['SERVER_NAME'] : 'swift.generated')
Example
SERVER_NAME = example-.local and try sending an email.

Given RFC 1035 section #2.3.1, it would seem that a label (or more specifically in this case, a subdomain of local) should not end with an hyphen.

The labels must follow the rules for ARPANET host names.  They must start with a letter, end with a letter or digit, and have as interior characters only letters, digits, and hyphen.  There are also some restrictions on the length.  Labels must be 63 characters or less.

If domain is invalid it should return "swift.generated" ?


N1-canards variant

Update simulation.ork
added canards to n1 with a stability of 1

Closing this as N1 didn't have canards


Feature: lifecycle management for zero downtime distributed hosting

When running at scale, gateway instances must report their health state indicating whether an instance is willing or not to receive new traffic. See e.g. Azure Load Balancer health probes, which would call the Gateway periodically.
It makes sense for the Reverse Proxy core to support this since this requires careful coordination of startup / teardown activities, and it also influences runtime gateway behavior. E.g during graceful shutdown, Gateway must remain serving new requests, while reporting its upcoming teardown. It must also advise clients to close existing connections by sending a Connection: close response header when applicable during graceful shutdown, while delaying teardown until (a) all requests are drained; *and (b) Load Balancer has detected the instance is not eligible for new traffic.
We have implemented this in Island Gateway and I can share more details if desired.

It seems like this has 3 parts to it:
a) Supporting graceful shutdown where clients are advised to connect to something else
b) Draining connections before actually shutting down - probably with a timeout
c) Eventing & signal model so that activating shutdown can tell the router to stop traffic, and when shutdown is complete, and the API to activate shutdown when told to by the router - assuming that app code will be listening to signals from the router.

The "Hosted" .NET app model (i.e. using Microsoft.Extensions.Hosting) on which ASP.NET Core is based has a graceful shutdown model and Kestrel participates in it already I believe. When shutdown is requested, Kestrel stops accepting new connections and drains existing ones (I'm not sure if it puts Connection: close in but that could be added). If a (configurable) timeout elapses, the server terminates more forcefully (yields back to Program.Main which will likely exit and terminate the process).
So I think we have (b) already and adding (a) is relatively straightforward.
For (c), we have the necessary event hooks (IApplicationLifetime.ApplicationStopping). I'm not sure there's a clear enough "pattern" that we could just implement in YARP though. I think custom code would be the primary way users would interact with that.

Triage: We do also have (a) it turns out.
Sounds like we need at least a sample (and maybe even a feature) for signalling health state based on IApplicationLifetime.

See the following as a follow on from c)
#125 - Having an API for querying and understanding proxy status
#124 - REST / gRPC endpoint to expose the data from the proxy
#123 - Dashboard showing that data

We achieve the first part (not ready state when starting/stopping/stopped) of c) in our services with a health check on IHostApplicationLifetime that tracks the status (starting/started/stopping/stopped) and use that in the readiness check.
The second part, removing from the upstream Load Balancer (LB) before rejecting new connections, is a bit more tricky. It either needs an active call from the service (and response from the LB) to remove the instance from rotation on the LB OR some knowledge about the configuration of health checks on the LB (ex. 3 failed checks with 10 sec. check interval) to know how long we need to report as unhealthy to the LB so it will have us removed before we proceed with Stopping and rejecting new connections.

Seems to be working for Island Gateway with a 3min shutdown for Kestrel (long duration due to gRPC streaming).
Should look at how to gracefully shutdown inner protocols such as grpc streaming. Eg https://github.com/grpc/grpc-go/issues/2888

Triage: We should be able to just see it working on Island Gateway prototype.
Sample may be useful for everyone

Triage: Scenario which didn't need any work, unclear what to do in sample. Closing.

Triage: Scenario which didn't need any work, unclear what to do in sample. Closing.


Failed to match alliance whitelist

Image
Failed to login

Same problem here, it called "alliance whitelist" for persons who in ally, and a "corporation whitelist" for those who just in corp (even npc corp)


"yield" is not highlighting

From @IgorNovozhilov on December 27, 2016 8:53

VSCode Version:

OS Version: Linux Mint 18

Steps to Reproduce:

write code:

/* jshint esnext:true */
let it = require('./it');
it.execute(function* builder(options) {
   let metaData = yield it.read(option.metaFile);
   it.meta = yield it.parse(metaData, {
      ICON: yield it.read(options.icon),
      ICON64: (yield it.read(options.icon64)).base64()
   });
});

key word not highlighted in all cases

Copied from original issue: Microsoft/vscode#17844

From @dbaeumer on December 27, 2016 9:19
Looks like a TS/JS grammar problem.


Fix uninitialized vboId in GenMeshHeightmap

mesh.vboId is  not uninitialized in GenMeshHeightmap, which cause segfault in the models_heightmap example.

@oraoto thanks for the fix! :)


Missing /dev/full

I'm trying to make flatpak/bubblewrap work with WSL, and one of the issues I found is that /dev/full is missing.
I have a workaround for this in the wip/WSL bubblewrap branch here:
https://github.com/projectatomic/bubblewrap/commit/37cba1851b8420b1fc73e089643ad9c49a1d8f25
But it would be nice if it was just there.

Thank you, Alex for your work. It's nice to see you treat WSL as a first class "Linux" platform.

Thanks for submitting this issue. We're currently working on a large body of work for WSL that will provide a fix for this. Will update this here when that fix hits Insiders Builds


Enrolling Admin failed.

I followed all the steps to get the marbles network deployed, from the readme. I encountered no errors while setting up the network. However after the login screen, Enrolling the Admin failed with the following error.

The Logs:

I meet the same issue. All the previous go smoothly until gulp marbles_local, the log is the same as post

That worked. Thanks!


create a improved installer

make some easiest installer for another distros

make install should work on all distros?
I think you mean create a package for other distros?
My programs are available on the AUR (Packaged by me) since i use Artix linux, Some packages are available on NIX (Packaged by random individuals)
I can't maintain packages on other distros nor can i be bothered to package them myself.
I have no issue with people packaging my software though so if you're interested feel free to do so!
I am however down to help any issues that prevent my software from being installed on other distros.


RE: Spindle

Hello...would I use 0 - 10v or another section of the Spindle headers for a spindle that can handle 10,000 RPM?
Seth
P.S. T41U5XBB is the board I am using with the Teensy4.1. I was thinking that the Spindle needs a different set up since I did not populate the trimpot for the Spindle 0 - 10v header. Please send guidance.

For instance, would I just use PWM and GND?

Or...if using PWM and GND only, would I need a driver or can I run the bare spindle wires directly from the T41U5XBB?

The trim pot is needed to set the 0-10V.  But, if your VFD supports 5V PWM, you can use that.  You will need to set up the VFD to take external input and 5V PWM. Connect to Gnd on the board to Gnd on the VFD (ACM or DCM, consult your VFD manual) You will also need to use a relay to connect FOR to ground (DCM, probably).

@phil-barrett ...thank you but I only have a cheap one for now. The two wires of the spindle are just that now, i.e. two wire interface.
The older controller plugged right in. I have no clue as to how it was wired, i.e. not open source stuff.
Seth
P.S. So, the 12v supposedly of this spindle can be directly attached to PWR of my PSU and then use GNDs all around? I will research more so I can understand a bit more.

Okay and A-Okay,
I was reading some. VFD or variable speed drives come in 110v and 220v. I will be purchasing one to handle 110v input while outputting some voltage and amperage. Please look here if you think my spindle would get along well with it:
https://filecenter.deltaww.com/Products/download/06/060101/Catalogue/DELTA_IA-MDS_VFD-EL_C_EN_20210420.pdf
That link is a .pdf file online from digikey's datasheets on their products they sell and handle.
In the meantime, I can test the RPM with a photo tachometer to find out if indeed it is producing 7000 to 10000 RPM when the ioSender states it is using that speed.
Seth
P.S. Thank you first off for replying. Secondly, I appreciate you taking time out of your schedule to help out.

What spindle do you have?  Can you post a link to it?  Be careful because spindles come in 110 and 220 and need to be matched to the VFD for both voltage and power.
By the way, Delta VFDs are very good.  Pricey.  I've never used one but the manual is reputed to be very good.

Thank you for understanding so far. I usually mess with PWM lasers and other cheap machines so far.
I am still workin' my way up the Totem pole in this onslaught on technology and electronics. I will confer to the manual. Bear with me.
Seth

@phil-barrett , hello again. I contacted the company that produced the machine. It is a small machine that does not handle a lot of torque or current. So, in hindsight, I should have broke down and got the machine that is better suited for my taste.
I just posted on their Q & A "forum" to find out about the spindle in question.
Seth
P.S. I will report back soon.

Could you please give me a link to the machine?

https://www.monoprice.com/product?p_id=44220
It is an inexpensive machine but I may get another set up soon. It seems the machine does not quite have the specifications one would need to handle nice parts.
Seth
P.S. Thank you.

OK, that is a DC spindle.  You don't need a VFD.  It runs off 12 or 24V DC - not sure which, maybe either. Did the machine come with a controller?

Yes...the machine came with a controller.

Could take a photo of it?

Sure. Bear with me. I need to find it, dig it out of hibernation, and post the photo.
Seth

@phil-barrett ,
Do you have any news that may be helpful?
Seth

OK, looking at the board, it appears to be a woodpecker board.  That has a mosfet for driving a DC spindle or laser.  My boards don't have that.  To drive the spindle, you will need a separate driver for the spindle.  Something like this might work. I have ordered one and will give it a try.  I'll be out for a couple days. Hopefully I can try it out by monday.  I am still looking to see if there are better modules to use.

Thank you, sir. I may have a driver here I can test but I will need to check.

Also sir, I have been looking at M and G-codes for setting parameters or furthest extents.
Is the GRBL g-code still 101 and 110 for XY? Off to research more. I keep getting other g and m-codes on different sites stating something different. I will go directly to GRBL.
Seth

https://www.diymachining.com/wp-content/uploads/2015/10/grbl-Settings-steps-per-mm-math-example.png
and...
https://www.diymachining.com/downloads/GRBL_Settings_Pocket_Guide_Rev_B.pdf
And A-Okay but...

Once I devise the math and set up the data to be understood by the controller and drivers, then what exactly?
a. Can I set up my cartesian plane within your settings on the ioSender source?
b. How would I set up the XYZ furthest extents to let my machine know not to go any further?

Seth

Here's a nice grbl settings pocket guide.  It doesn't have all the grblHAL ones but these are the basics.
As to GCode, I use the LinuxCNC documentation.  It covers a lot more than grblHAL.
As to setting max travel - $130, $131 and $132 will do that for X, Y and Z.  You will need to turn on soft limits.

Thank you sir. XY is cartesian...I thought. I am using Z also, i.e. so I guess just XY extents are not relevant.
Seth
P.S. Thank you for the link. I will research it and thank you for making me understand the commands for max travel. I just did not type in max travel and was typing in google furthest extents GRBL. Oops. I got some odd results. Yes!

Okay. I found the max travel in the ioSender source...
I set up my Max Travel to X270 Y270 Z25. I have drivers that should handle the motors in question but I cannot get motion from my motors so far.
I read in the Wiki you guys/gals put together that the app.config file can be setup to handle ideas also.
I am not sure where to start so far...
At first my motors turned but not well. Now, the motors do not move at all. The motors are not hot like I would expect them to be hot if over powering them. I am using a 12v PSU with GeckoDrive G251 drivers and a cheaper Leadshine driver clone.
Seth
P.S. At first, I think the reason is that I do not know how to wire the Leadshine driver clone to the T41U5XBB and Teensy4.1.
any help would be greatly appreciated...

Stuck at best right now...

You probably need to invert the stepper enable pin setting.  $4=7 will do it.

Okay. I will try it... Thank you.

@phil-barrett ,
Seth here once more. Do you think I should install the wires for the Disable Pins on the G251 drivers and the Enable Pin on the DM332T Leadshine Clone driver?
And then, do you think the inverting of the enable pin on the DM332T will do it?
Seth
P.S. I removed my laser project from the T41U5XBB, e.g. the machine was too cheap. The build was for a laser engraver/cutter but the lack of sturdy parts made the build irrelevant for now. Please send guidance. I can test the wiring but I am a bit mistook on what to do so far. Here is the G251 photo of connections and the DM332T connections:

and...

Also, if using the OPTO pin on the DM332T driver, which I think is needed, where should I attach it? GND or another header connector on the T41U5XBB?

OK, the DM332T has open collector interfaces.  What you will need to do is find a source of +5V.  If you don't have a +5V PSU, you can use the +5V terminal on the T41U5XBB.  Connect +5V to the opto input.  Connect the Dir, Step and Enable signals to DIR, PUL and ENA signals. No need to use Gnd if sourcing the +5V from the T41U5XBB.  You will need to invert Dir, Step and Enable in grblHAL settings ($2, $3 and $4) for the axes driven by the DM332T.
For the axis driven by the G251,  you can connect DIRECTION, STEP, ENABLE and SIGNAL GND to the screw terminal pins on the T41U5XBB.  You may have to invert the enable signal for that Axis - try it both ways.

Thank you, sir...
I will do just that and get back to you. I really appreciate you making me understand possibilities with these drivers.
Seth

No problem.  Happy to help.

Hey sir...you were right. I got most of the drivers working XZ so far. Y is giving me issues. I will post a photo. If you see anything out of the ordinary, please let me know.
here:

Does the highlighted XYZ and other Red highlighted parts look okay?
Seth

OK, looking at the board, it appears to be a woodpecker board. That has a mosfet for driving a DC spindle or laser. My boards don't have that. To drive the spindle, you will need a separate driver for the spindle. Something like this might work. I have ordered one and will give it a try. I'll be out for a couple days. Hopefully I can try it out by monday. I am still looking to see if there are better modules to use.

Sir,
Did you get around to the testing yet? Also, would I need a medium between the circuit and board or circuit and DC spindle?
Something like a controller to handle changing the speed of the device or do you think the source can handle that aspect of it all?
Seth


Initialize the injector before super.onCreate in components

Fixes #50

I wonder if a more correct fix for this issue would be to have Fragment injection happen in onActivityCreated.

The only issue with that is that it occurs after the View is created, and users most likely would require some of their dependencies before then.

Stumbled across this in the Dagger 2 docs:

When members injection is required (as discussed above), prefer to inject as early as possible. For this reason, DaggerActivity calls AndroidInjection.inject() immediately in onCreate(), before calling super.onCreate(), and DaggerFragment does the same in onAttach(), which also prevents inconsistencies if the Fragment is reattached.

https://google.github.io/dagger/android.html#when-to-inject
Sounds like Kodein should follow suite.

@coreydowning nice find!
@SalomonBrys seems like this is ready to be merged

Thanks a lot :)


plist should be all lowercase

After reading the TS for the xcode build i found that the docs say to use exportOptions: Plist for a plist file but the code checks for all lowercase plist. Updating to match code.

@aleccaputo : Thanks for your contribution! The author(s) have been notified to review your proposed change.


[core] Remove all ...All() methods from sensors and motors

The data holders (SensorSharedDataHolder_t ...) should be refactored to be actual classes managing set of sensors, and as such, being friend of the corresponding abstract class (AbstractSensorBase ...). It should provide methods operating on all sensors in the holder at once (get, setOptions ...) and being renamed accordingly (SensorManager ...).

Ideally, it should be possible to instantiate standalone sensors without shared storage, then update and fetch sensor measurement in isolation. The manager should only be helper that provides a bunch of optimised routines to process a ground a consistent sensors at once. It should not have to rely on private methods. It would be nice if the said manager could be completely independent from the shared storage but it actually comes together since it only makes sense if shared storage is enabled in the first place.


Remove direct interactions with contracts from hop-node and rely strictly on the SDK to interface with the contracts

The hop-node should not interact directly with the contracts. All interactions should occur via the sdk.

Closed with https://github.com/hop-protocol/hop/pull/613


Added rosdeps for glfw3.

This adds a rosdep key for glfw3.

debian: https://packages.debian.org/jessie/libglfw3-dev
ubuntu: https://packages.ubuntu.com/xenial/libdevel/libglfw3-dev

I am using this dependency to compile librealsense as a catkin package. Tested on Ubuntu 16.04.
I am not sure whether the key shall be glfw or glfw3. On Ubuntu 16.04, the following packages are available:

libglfw2
libglfw3
libglfw3-dev
libglfw-dev (installs libglfw2)

For compiling librealsense, I need libglfw3-dev, therefore glfw3 would more specific. However, on some the distributions I googled (e.g. gentoo: https://packages.gentoo.org/packages/media-libs/glfw), it seems that it is not possible to select the version to install and you just get the latest.

I am not sure whether the key shall be glfw or glfw3

The key should match the ubuntu / debian package name. If you want libglfw3-dev that's the name of the key we should pick.
Unfortunately libglfw3(-dev) does not exist on Ubuntu Trusty, that is one of our targeted active Ubuntu distribution.
We will need to add a special case for trusty to say that this key cannot be satisfied on that platform.
Something like:
libglfw3-dev:
  debian: [libglfw3-dev]
  ubuntu:
    '*': [libglfw3-dev]
    trusty: null

Thank you for your comment.
I updated the PR accordingly. As I haven't seen this syntax somewhere else in the file, can rosdep handle this?

(e.g. gentoo: https://packages.gentoo.org/packages/media-libs/glfw), it seems that it is not possible to select the version to install and you just get the latest.

Yes Gentoo sometimes provides only the latest. It's fine as gentoo is a rolling release. Please add it to the list of rules as, currently, media-libs/glfw points to version 3.

As I haven't seen this syntax somewhere else in the file, can rosdep handle this?

Both the wildcard and null syntax have been added "recently" https://github.com/ros-infrastructure/rep/pull/153. While we used the wildcard in a couple of places, I think this is the first time we use the "null". I copied the example from https://github.com/ros-infrastructure/rep/blob/master/rep-0111.rst#allow-package_argument-to-be-explicitly-null . According to rosdep's changelog, support for both has been added in 0.12.0: https://github.com/ros-infrastructure/rosdep/blob/master/CHANGELOG.rst#0120-2018-02-07

That's great, thanks. I added the Gentoo package.

thanks @remod for iterating.
While rosdep supports the new syntax, the rosdistro tests don't know about it ATM.
I opened https://github.com/ros/rosdistro/pull/18113 to add support for it.
I tested this new key in a trusty container and I get what I think is the expected behavior:
root@a468953e6c30:/# rosdep resolve glfw3

ERROR: [glfw3] defined as "not available" for OS version [trusty]

[glfw3] defined as "not available" for OS version [trusty]
	rosdep key : glfw3
	OS name    : ubuntu
	OS version : trusty
	Data: debian:
- libglfw3-dev
ubuntu:
  '*':
  - libglfw3-dev
  trusty: null

This change is breaking rosdep db on Ub 14.04.
ERROR: Rosdep experienced an error: rosdep OS definition for [libglfw3-dev:ubuntu] must be a dictionary, string, or list: None
IMO we don't need to make trusty as a special case, that if any package requires libglfw3-dev in Ub 14.04, just let the build fail.

@bluryi You likely haven't installed the latest version of the python-rosdep package. Please update that package and the current rosdep database should be readable without a problem.

@dirk-thomas  thanks!


moblie button issue

If you have a question, please ask it on StackOverflow and use signature-pad tag instead of creating an issue.
Do you want to request a feature or report a bug?
report a bug
What is the current behavior?
When I turned on the https://stackoverflow.com/questions/ask?tags=javascript+signature-pad site on mobile and tested it, the click event didn't work at first, but next time I pressed it again, I found a behavior that works and I'm expecting to use the touch event as a solution, but I want to find an exact solution.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://jsfiddle.net or similar (template: https://jsfiddle.net/szimek/h80phxyr/).
@click event -> @touchend event
What is the expected behavior?
Which versions of SignaturePad, and which browser / device are affected by this issue? Did this work in previous versions of SignaturePad?
There is a problem with the mobile device.

Can you provide code so we can reproduce this?

I try this site
https://codesandbox.io/s/n5qjp3oqv4?resolutionWidth=320&resolutionHeight=675

What is the problem? It seems to work perfectly for me

It looks like you are using vue-signature-pad which is using an old version of signature_pad if you need help you should create an issue at https://github.com/neighborhood999/vue-signature-pad

@gaetanlgce I noticed when you press the clear button the first couple of times the button changed color (slightly darker), but it didn't change after that. My guess is that it is your browser that is causing the debounce to prevent accidental clicks. If signature_pad doesn't actually receive a click event from the browser we can't know that the user clicked the button.
Maybe there is a browser or phone setting that can help you.


No registered metadata for type "class org.surfnet.oaaas.model.Client".

I deployed the apis-authorization-server in the tomcat. But have the exception:
org.springframework.orm.jpa.JpaSystemException: No registered metadata for type "class org.surfnet.oaaas.model.Client".
I can't found the reason, anyone could help me?

Same here :-(

+1  I compiled with Intellij IDEA

我用 Intellij IDEA 编译运行 也是同样的错误，楼主有解决吗？


Upon doing research on the fiscal code. I have found that in Identify…

…ing short code of first/last name that contains less than 3 characters ("OM" (2) in this case).You are not handling the case of placing consonant before the vowel as per fiscal code generation algorithm. In this case M should come before O.
PRKMOX65D01Z222C --->Actual Code for a case
PRKOMX65D01Z222L --->Fiscal Generated By Current Code

Can one of the admins verify this patch?

I'll look into it asap! thanks for the contribution


heroku/updater.rb error

Win7 x64
C:\Program Files\Heroku>heroku
C:/Program Files (x86)/Heroku/lib/heroku/updater.rb:164:in spawn': No such file  or directory - open (Errno::ENOENT)         from C:/Program Files (x86)/Heroku/lib/heroku/updater.rb:164:in backgro
und_update!'
from C:/Program Files (x86)/Heroku/lib/heroku/updater.rb:144:in inject_ libpath'         from C:/Program Files (x86)/Heroku/bin/heroku:24:in '

@dickeyxxx Well, since you invited further comment, I will. Here is my error:
←[0KRunning... 0.00BC:/Users/.heroku/client/lib/heroku/cli.rb:45:in ``': No such file or directory - stty ica
non echo (Errno::ENOENT)
from C:/Users/.heroku/client/lib/heroku/cli.rb:45:in rescue in start'         from C:/Users/.heroku/client/lib/heroku/cli.rb:35:in start'
from C:/Program Files/Heroku/bin/heroku:29:in <main>' Terminate batch job (Y/N)? N (green) PS G:\blue\BLUE> heroku pg:backups === Backups  !    No backups. Capture one with heroku pg:backups capture`.
(green) PS G:\blue\BLUE>
This happens when I am trying to import a database. I saw some of these issues had to do with using, or not using, a latin username. I don't think that's my issue. I am on Windows 8.1.
This guy  https://github.com/heroku/heroku/issues/1093  seems to have the same error but a very different fix. I look forward to your assistance.

Would you mind trying again after running heroku update? I put a small fix around this error.
On Thu, Mar 26, 2015 at 12:53 PM, MalikRumi<EMAIL_ADDRESS>wrote:

@dickeyxxx Well, since you invited further comment, I will. Here is my error:
←[0KRunning... 0.00BC:/Users/.heroku/client/lib/heroku/cli.rb:45:in ``': No such file or directory - stty ica
non echo (Errno::ENOENT)
from C:/Users/.heroku/client/lib/heroku/cli.rb:45:in rescue in start'         from C:/Users/.heroku/client/lib/heroku/cli.rb:35:in start'
from C:/Program Files/Heroku/bin/heroku:29:in <main>' Terminate batch job (Y/N)? N (green) PS G:\blue\BLUE> heroku pg:backups === Backups  !    No backups. Capture one with heroku pg:backups capture`.
(green) PS G:\blue\BLUE>
This happens when I am trying to import a database. I saw some of these issues had to do with using, or not using, a latin username. I don't think that's my issue. I am on Windows 8.1.
This guy  https://github.com/heroku/heroku/issues/1093  seems to have the same error but a very different fix. I look forward to your assistance.
Reply to this email directly or view it on GitHub:
https://github.com/heroku/heroku/issues/814#issuecomment-86689483

Ok, it says it updated, but now I am back to my old database errors.
On Thu, Apr 2, 2015 at 6:18 PM, Jeff Dickey<EMAIL_ADDRESS>wrote:

Would you mind trying again after running heroku update? I put a small
fix around this error.
On Thu, Mar 26, 2015 at 12:53 PM, MalikRumi<EMAIL_ADDRESS>wrote:

@dickeyxxx Well, since you invited further comment, I will. Here is my
error:
←[0KRunning... 0.00BC:/Users/.heroku/client/lib/heroku/cli.rb:45:in ``':
No such file or directory - stty ica
non echo (Errno::ENOENT)
from C:/Users/.heroku/client/lib/heroku/cli.rb:45:in rescue in start' from C:/Users/.heroku/client/lib/heroku/cli.rb:35:in start'
from C:/Program Files/Heroku/bin/heroku:29:in <main>' Terminate batch job (Y/N)? N (green) PS G:\blue\BLUE> heroku pg:backups === Backups ! No backups. Capture one with heroku pg:backups capture`.
(green) PS G:\blue\BLUE>
This happens when I am trying to import a database. I saw some of these
issues had to do with using, or not using, a latin username. I don't think
that's my issue. I am on Windows 8.1.
This guy https://github.com/heroku/heroku/issues/1093 seems to have the
same error but a very different fix. I look forward to your assistance.
Reply to this email directly or view it on GitHub:
https://github.com/heroku/heroku/issues/814#issuecomment-86689483

—
Reply to this email directly or view it on GitHub
https://github.com/heroku/heroku/issues/814#issuecomment-89075603.


Sporadic exception on DnsNameResolver

Resolves the issues consistently without issues
Periodically throws the following exception:
io.netty.handler.codec.DecoderException: java.lang.IndexOutOfBoundsException: readerIndex(512) + length(2) exceeds writerIndex(512): PooledUnsafeDirectByteBuf(ridx: 512, widx: 512, cap: 4096)
	at io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:98) ~[netty-codec-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:374) ~[netty-transport-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:360) ~[netty-transport-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:352) ~[netty-transport-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.channel.DefaultChannelPipeline$HeadContext.channelRead(DefaultChannelPipeline.java:1408) ~[netty-transport-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:374) ~[netty-transport-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:360) ~[netty-transport-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.channel.DefaultChannelPipeline.fireChannelRead(DefaultChannelPipeline.java:930) ~[netty-transport-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.channel.nio.AbstractNioMessageChannel$NioMessageUnsafe.read(AbstractNioMessageChannel.java:93) ~[netty-transport-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.channel.nio.NioEventLoop.processSelectedKey(NioEventLoop.java:682) ~[netty-transport-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.channel.nio.NioEventLoop.processSelectedKeysOptimized(NioEventLoop.java:617) ~[netty-transport-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.channel.nio.NioEventLoop.processSelectedKeys(NioEventLoop.java:534) ~[netty-transport-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:496) ~[netty-transport-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:906) ~[netty-common-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.util.internal.ThreadExecutorMap$2.run(ThreadExecutorMap.java:74) ~[netty-common-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30) ~[netty-common-4.1.36.Final.jar:4.1.36.Final]
	at java.base/java.lang.Thread.run(Thread.java:834) ~[na:na]
Caused by: java.lang.IndexOutOfBoundsException: readerIndex(512) + length(2) exceeds writerIndex(512): PooledUnsafeDirectByteBuf(ridx: 512, widx: 512, cap: 4096)
	at io.netty.buffer.AbstractByteBuf.checkReadableBytes0(AbstractByteBuf.java:1428) ~[netty-buffer-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.buffer.AbstractByteBuf.readShort(AbstractByteBuf.java:742) ~[netty-buffer-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.buffer.AbstractByteBuf.readUnsignedShort(AbstractByteBuf.java:758) ~[netty-buffer-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.handler.codec.dns.DefaultDnsRecordDecoder.decodeRecord(DefaultDnsRecordDecoder.java:61) ~[netty-all-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.handler.codec.dns.DatagramDnsResponseDecoder.decodeRecords(DatagramDnsResponseDecoder.java:110) ~[netty-all-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.handler.codec.dns.DatagramDnsResponseDecoder.decode(DatagramDnsResponseDecoder.java:68) ~[netty-all-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.handler.codec.dns.DatagramDnsResponseDecoder.decode(DatagramDnsResponseDecoder.java:33) ~[netty-all-4.1.36.Final.jar:4.1.36.Final]
	at io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:88) ~[netty-codec-4.1.36.Final.jar:4.1.36.Final]
	... 16 common frames omitted

Steps to reproduce
Minimal yet complete reproducer code (or URL to code)
public class DnsClient{

	private static EventLoopGroup eventLoop = new NioEventLoopGroup(1);
	private CompletableFuture<String> completableFuture;
	private DnsNameResolver resolver;
	private static DefaultDnsCache cache = new DefaultDnsCache();
	public DnsClient() {
//		eventLoop = new NioEventLoopGroup();
	}

	public CompletableFuture<String> call() throws Exception {
		completableFuture = new CompletableFuture<String>();
		try {
			resolver = new DnsNameResolverBuilder(eventLoop.next()).queryTimeoutMillis(10000).resolveCache(cache).maxQueriesPerResolve(5).channelType(NioDatagramChannel.class).build();
			DefaultDnsQuestion defaultDnsQuestion = new DefaultDnsQuestion("gmail.com", DnsRecordType.MX);
			Future<AddressedEnvelope<DnsResponse, InetSocketAddress>> future = resolver.query(defaultDnsQuestion);
//			AddressEnvelopeListener listener = new AddressEnvelopeListener();
			future.addListener(new GenericFutureListener<Future<AddressedEnvelope<DnsResponse, InetSocketAddress>>>() {

				@Override
				public void operationComplete(Future<AddressedEnvelope<DnsResponse, InetSocketAddress>> future)
						throws Exception {
					handleOperationComplete(future);
				}
			});
		} catch (Exception e) {
			System.out.println(e);
		}
		return completableFuture;
	}

	protected synchronized void handleOperationComplete(
			Future<AddressedEnvelope<DnsResponse, InetSocketAddress>> future) {
		System.out.println("operation complete");
		List<MxDnsRecord> dnsRecords;
		if (future.isSuccess()) {
			try {
				dnsRecords = new ArrayList<>();
				AddressedEnvelope<DnsResponse, InetSocketAddress> answer = future.get();

				DnsResponse content = answer.content();
				final int count = content.count(DnsSection.ANSWER);
				final List<MxDnsRecord> mxList = new ArrayList(count);
				for (int i = 0; i < count; i++) {
					final DnsRecord record = content.recordAt(DnsSection.ANSWER, i);
					if (record.type() == DnsRecordType.MX) {
						mxList.add(new MxDnsRecord(record));
					}
				}
				answer.release();
				sortRecordsByPriority(mxList);
				StringBuilder stringBuilder = new StringBuilder();
				mxList.forEach(m -> stringBuilder.append(m.toString()));
				completableFuture.complete(stringBuilder.toString());

			} catch (InterruptedException e) {
				e.printStackTrace();
			} catch (ExecutionException e) {
				e.printStackTrace();
			}
		}
		else {
			try {
				AddressedEnvelope<DnsResponse, InetSocketAddress> answer = future.get();
			} catch (InterruptedException | ExecutionException e1) {
				e1.printStackTrace();
			}
		}
		resolver.close();
	}

	private List<MxDnsRecord> sortRecordsByPriority(List<MxDnsRecord> dnsRecords) {
		if (dnsRecords != null) {
			Collections.sort(dnsRecords, new Comparator<MxDnsRecord>() {
				@Override
				public int compare(MxDnsRecord o1, MxDnsRecord o2) {
					return ((Integer) o1.getPriority()).compareTo(o2.getPriority());
				}
			});
		}
		return dnsRecords;
	}
}

And here is the MxDnsRecordClass:
public class MxDnsRecord implements DnsRecord {

	private String name;
	private DnsRecordType recordType;
	private int dnsClass;
	private long timeToLive;
	private int priority;
	private String inetHost;

	public MxDnsRecord(DnsRecord dnsRecord) {
		this.name = dnsRecord.name();
		this.recordType = dnsRecord.type();
		this.dnsClass = dnsRecord.dnsClass();
		this.timeToLive = dnsRecord.timeToLive();
		ByteBuf recordContent = ((ByteBufHolder) dnsRecord).content();
		this.priority = recordContent.readUnsignedShort();
		this.inetHost = DefaultDnsRecordDecoder.decodeName(recordContent);
	}

	@Override
	public String name() {
		return name;
	}

	@Override
	public DnsRecordType type() {
		return recordType;
	}

	@Override
	public int dnsClass() {
		return dnsClass;
	}

	@Override
	public long timeToLive() {
		return timeToLive;
	}

	public int getPriority() {
		return priority;
	}

	public String getInetHost() {
		return inetHost;
	}

	public String toString() {
		StringBuilder stringBuilder = new StringBuilder();
		stringBuilder.append(StringUtil.NEWLINE);
		stringBuilder.append('\t');
		stringBuilder.append(getName());
		stringBuilder.append(' ');
		stringBuilder.append(getRecordType().name());
		stringBuilder.append(' ');
		stringBuilder.append(getPriority());
		stringBuilder.append(' ');
		stringBuilder.append(getInetHost());
		return inetHost;
	}
}

Netty version
4.1.36
JVM version (e.g. java -version)
11
OS version (e.g. uname -a)
Windows 10

I was able to reproduce this with simpler code as well:
EventLoopGroup eventLoop = new NioEventLoopGroup();
			DnsNameResolver resolver = null;
			DnsResponse content = null;
			AddressedEnvelope<DnsResponse, InetSocketAddress> answer = null;
			String result = "nothing to show";
			try {
				resolver = new DnsNameResolverBuilder(eventLoop.next()).channelType(NioDatagramChannel.class).build();
				DefaultDnsQuestion defaultDnsQuestion = new DefaultDnsQuestion("gmail.com", DnsRecordType.MX);
				Future<AddressedEnvelope<DnsResponse, InetSocketAddress>> future = resolver.query(defaultDnsQuestion);

				answer = future.syncUninterruptibly().getNow();
				content = answer.content();
	            final int count = content.count(DnsSection.ANSWER);
	            final List<DnsRecord> mxList = new ArrayList(count);
	            for (int i = 0; i < count; i ++) {
                    final DnsRecord r = content.recordAt(DnsSection.ANSWER, i);
                    if (r.type() == DnsRecordType.MX) {
                        mxList.add(r);
                    }
                }
	            StringBuilder buf = new StringBuilder();
                for (DnsRecord r: mxList) {
                    ByteBuf recordContent = ((ByteBufHolder) r).content();

                    buf.append(StringUtil.NEWLINE);
                    buf.append('\t');
                    buf.append(r.name());
                    buf.append(' ');
                    buf.append(r.type().name());
                    buf.append(' ');
                    buf.append(recordContent.readUnsignedShort());
                    buf.append(' ');
                    buf.append(DefaultDnsRecordDecoder.decodeName(recordContent));
                }
                System.out.println(buf.toString());
	            result = buf.toString();
			}
			catch (Exception e) {
				System.out.println(e);
			}
			finally {
				if (resolver != null) {
					resolver.close();
					answer.release();
				}
				eventLoop.shutdownGracefully();
			}
			System.out.println(result);
			return result;

Will have a look over the next days.

I wanted to give a bit more information. This has been very difficult to debug, since it is completely unpredictable; however, for what its worth, I have not been able to replicate it since setting the following 2 settings:
.optResourceEnabled(false)
.resolvedAddressTypes(ResolvedAddressTypes.IPV4_PREFERRED)

This does not mean that it is decisively fixed yet, since the issue does not manifest itself all the time, but it is a good sign.

@lechiefe I tried many times locally, but I couldn't reproduce the problem.

@lechiefe I tried many times locally, but I couldn't reproduce the problem.

I believe it, that is why I pointed out that the issue is sporadic and is hard to reproduce. Because of its nature, I believe that it is cause by a certain response from a remote dns server. Like I mentioned above, it seems to go away if I apply those settings, the first one specifically mentions that setting it to true might cause some dns providers to send a broken response. If that is correct, and that is the case(I cannot decisively confirm that) then perhaps there should be better handling of such an error, for example, if the retry setting was set on the resolver, perhaps it could retry with the setting set to false. As of right now its just a mysterious, cryptic error.

Would you please tell me what DNS server address you are using? I tried @<IP_ADDRESS> and @ns1.google.com, @<IP_ADDRESS>, I can't reproduce this problem.  By the way, the code I tested is here : https://gist.github.com/qeesung/7b5d5a42e7d8b97597d2baf55a83b43b

@lechiefe I suspect your problem is that you use use the "opt record" which tells the remote peer that you support datagram packets up to 4096 bytes (by default) and you sometimes get truncated datagrams. You can try to use a different maxPayloadSize to set a smaller limit if you encounter problems.

@normanmaurer My apologies for the delay, it may be the case, you obviously understand the infrastructure better than myself. I also suspected that that was the case, however while stepping though the code, on a rare occasion that I was able to catch an erroneous request in execution, the isTruncated flag was not set. Is my understanding correct that the current release does not support truncated packets?

@qeesung Yes, I can, I will have some time to look at the issue again this weekend, I will update this when I have the info. Thank you for looking into this, you guys do amazing work!

@lechiefe no it will have no truncated flag set as you told it via the OPT record that you can read 4096 bytes. It will set the truncated flag if the response would be bigger then 4096 bytes in this case

This will be fixed by https://github.com/netty/netty/pull/9468


0073 - Set Matrix Zeroes

Description of the Problem
Given an m x n matrix. If an element is 0, set its entire row and column to 0. Do it in-place.
Follow up:
A straight forward solution using O(mn) space is probably a bad idea.
A simple improvement uses O(m + n) space, but still not the best solution.
Could you devise a constant space solution?
Link To The LeetCode Problem
LeetCode

linked a pull request that will close https://github.com/vJechsmayr/PythonAlgorithms/pull/255#issue-497467740

I've linked the complete solution in two ways to solve this, please tag this as hacktoberfest.
Thank you.


Fix cutout depth bug

@rosenhauer,
Can you please look at this?
Is he doing something wrong?
Regards


Fixes #245

Fixes #245 - _1 in layer names

Assigning to @Dbarnes1 to verify - this update is required for the Pro Symbol Editor which does not have a fully working version of the lpkx ("spatial index invalid" error in prior version of lpkx, and _1 in current version)

@Dbarnes1 - quick check if you want to test before auto-merging

@csmoore I'll check- I thought I had already checked this but let me add it in to Pro real quick

@csmoore looks good, I'll go ahead and merge


No attachment points shown for micromolecule

Steps to Reproduce

Load from file: No Attachment points.zip
OR
Create molecule as shown on the screen:

Switch to Macro
Hover mouse over micromolecule

Actual behavior
No attachment points shown

Expected behavior
Eight attachment points shown

Versions [e.g. v2.4.2].

Indigo Toolkit Version 1.20.0-dev.1.32-gf28bad61-wasm32-wasm-clang-12.0.0
Ketcher Version 2.21.0-rc.2 Build at 2024-05-13; 05:32:02
Chrome Version 124.0.6367.201 (Official Build) (64-bit)
Windows 10

Issue found while testing https://github.com/epam/ketcher/issues/4530

Not a bug - works as intended


Feat/swiglu

Pull Request Template
Checklist

[x] Confirmed that run-checks all script has been executed.
[x] Made sure the book is up to date with changes in this PR.

Related Issues/PRs
#1504
Changes
Implement SwiGLU activation module to be used in LLM models

@antimora Added tests. Let me know if it looks good to merge.


[FEATURE REQ] Computer Vision - v3 Read API Suggestions

Library or service name.
What library or service is this request related to? Cognitive Services - Computer Vision
Is your feature request related to a problem? Please describe.
What feature would you like to get added? What problem is it solving?
The current v6.0.0-preview.1 SDK and documentation proposes customers to begin migrating off the soon deprecated v2 endpoints. As part of attempting to integrate the new v3 APIs via the SDK we have noticed the following areas which might make lives of other developers in this area easier.

Make available SDK method to extract the operationId when performing a ReadAsync call. Users will commonly have to reuse the operationId in order to obtain the actual result later, currently this is only exposed out as part of the ReadHeaders object which only has an OperationLocation value which is the full URI. Users are then left to perform string manipulation in order to extract the operationId for future operations

Amend or add overload to GetReadResultAsync(Guid operationId) to accept a string operationId instead. Given users can only receive this as a string from the previous above extraction method, having to then pass it in with Guid.Parse(operationId) is just another hoop to jump through in order to use the API

Provide supported language enum for the Read API, similar to OcrLanguages. It appears that the OcrLanguages enum is the list of supported languages for the legacy OCR endpoint, it would be great to have an equivalent for the v3 version

Thank you for your feedback.  Tagging and routing to the team best able to assist.

bump

Hi @wahyuen, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.