Road data script troubleshooting and fixes (FY25 Q2)

Benefit
As road data comes in from the counties, occasionally they change their schema or add values to the data that is outside the expected domain. This causes our scripts to fail. Troubleshooting these error messages and fixing the issues will speed up the monthly road update process.
Acceptance Criteria
Getting the scripts to run after they throw an error message. This will require either altering the scripts a bit or the data.
Notes
Notes on troubleshooting script errors
The majority of the time, when a script throws an error it's because the county changed the schema. This could be a change in expected domain values or a field name change. Our county ETL scripts are hard-coded to expect certain values and when they change, the script throws an error.
The troubleshooting part is determining what changed and how to proceed. Here are some steps to take when troubleshooting and then fixing the issue:
Check the error message. It will often inform you of the unexpected value (domain change) or a missing or renamed field name.
Based on the error message, determine if it's easier to fix/update the script or the county data. Oftentimes, it's quicker to alter the county's data (fixing the wonky field values or the missing field). Other times, it's easier to alter the script to accommodate their new schema. It's a case-by-case situation.
When updating scripts, if the issue occurs in Step 1 or Step 3 you can update the script directly. If the issue occurs in Step 2, feel free to work with Greg the first couple times as these scripts are a slightly more complex, but not out of scope. Essentially, if it occurs in Step 2, you'll need to look for the county's name in UtransETL_FieldMappingFunctions.py and make the change there.
Risks
No response
Issue Reference
refs #

The November 2024 data push was the first run without Greg, and it presented some learning opportunities.
It took several days to determine that after pushing the data, during the re-establishment of editor tracking I mistakenly chose to enable tracking at the dataset level.  This should have been performed at the feature class level for the roads feature class.  Setting editor tracking at the dataset level resulted in the 4 tracking fields being added to all tables within the dataset.  This then broke the append that runs at the end of the second data preparation script.
To resolve, I ended up removing the 4 fields added to the DFC_RESULT table, which allowed the append to complete.  (Schema mismatch.)  Re-enabled editor tracking on the Roads feature class.


Invalid value should not be written to the vm

Suppose we have a input with v-model, and we set a maxlength=10 validation to make sure the length of value should be at most 10.
Now when the user input 11 characters, the validation will fail as expected.
But the error value is still written to the vm.
My expectation is if the validation failed, the corresponding value should not be written to the vm.

I think it may be difficult if we don't change v-model code. the v-model must not update the model if the validation failed

My expectation is if the validation failed, the corresponding value should not be written to the vm.
I think it may be difficult if we don't change v-model code.

Yeah,
I think so, too.
If we need to support it, maybe I think it develop the v2.0 release later, but we will need to help from @yyx990803

I'll try to implement at v3.0.


Merging std::nothrow changes from ip-gpu

Merging @ip-gpu changes to one request

@ip-gpu - if you wanna get credit for this, feel free to submit your own pull request with these changes. Else in a few days I'll likely commit this and reject your other pull requests.
If there's some alternate means for merging your changes that would be better, let me know.


README: Include limitations of jsdom with axe.

https://twitter.com/marcysutton/status/963569635340636160
maybe could add a puppeteer example

Here's an example if you need it, which is setup with Jest's Puppeteer guide, in particular jest-environment-node:
import { toHaveNoViolations } from 'jest-axe';

expect.extend(toHaveNoViolations);

jest.setTimeout(10000);

describe('/', () => {
    let page;
    beforeAll(async () => {
        page = await global.BROWSER.newPage();
        await page.goto(global.ROOT);

        await page.addScriptTag({ path: require.resolve('axe-core') });
    });

    it('loads', async () => {
        const text = await page.evaluate(() => document.body.textContent);
        expect(text).toContain('draftail');
    });

    it('a11y', async () => {
        const text = await page.evaluate(() => {
            return new Promise(resolve => {
                window.axe.run((err, results) => {
                    if (err) resolve(err);
                    resolve(results);
                });
            });
        });
        expect(text).toHaveNoViolations();
    });
});

The full setup is available here: https://github.com/springload/draftail/tree/master/tests/integration.

Thanks so much :) I'll try and get this added to the docs when I get a chance.

I've done a spike into adding puppeteer support into the library, if anyone is interested upvote this comment:
https://github.com/nickcolley/jest-axe/compare/spike-puppeteer?expand=1

I think this spike would have been useful when upgrading to Jest v24?
https://github.com/alphagov/govuk-frontend/pull/1277
I noticed access to global.__BROWSER__ (and other globals) was lost due to:

Note: Any global variables that are defined through globalSetup can only be read in globalTeardown. You cannot retrieve globals defined here in your test suites.

So each tests needs to specify if it wants the puppeteer or jsdom @jest-environment to run correctly.

Hey @nickcolley are there any plans for getting it working with puppeteer? That would be super awesome!

Heya, I don't have time to work on this feature right now and it's fairly involved sorry @donifer, I agree it would be cool though.
The spike I proposed above allows you to pass through an instance of puppeteer which means that jest-axe doesn't have to bundle puppeteer itself which would be too big of a dependency.
If enough people are interested and I get some time in the future I'll consider exploring this more, thank you for your patience.


Watching tests not working with IntelliJ's Jest Run Configuration

We recently switched from react-scripts v1 to v2. And with that, we now also use craco to rewire some configurations. (Thanks btw. for the awesome package. We used react-app-rewired and the configuration for craco is much nicer and easier to understand!)
Everything works fine so far, except the watch mode of tests in IntelliJ's Jest Run Configuration does not work properly.
The first run passes without any problem, but as soon as a file changes and a new test run is therefor triggered, the src/setupTests.ts file is not loaded again. Meaning that enzyme throws an error since no enzyme adapter is configured any more. (Other packages also throw errors since they are not initialized properly)
When I run the test in the regular terminal with the same configuration that I apply in IntelliJ (e.g. yarn run craco test --env=jsdom --watch), watching works normally and does not throw any errors.
Does anyone know how to fix this? I really do not want to loose the IntelliJ test result reporting since it makes working with tests so much easier.
I'm using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>if that makes any difference.

Hi @Lukas-Kullmann
I am not an IntelliJ user so it's hard for me to support it.
react-app-rewired have a fix for IntelliJ support: https://github.com/timarney/react-app-rewired/blob/master/packages/react-app-rewired/bin/jest.js
If you have time, you could try to add it to craco and send a PR! :)
Thank you

Hi @Lukas-Kullmann
I pushed a fix for IntelliJ in the 3.2.0 release.
Can you try it?
Thank you

Does not work, unfortunately. :/

I will close this issue, @Lukas-Kullmann feel free to open it again once you have news from IntelliJ


feat: handle errors better in web client

Summary

Closes https://github.com/MobilityData/gtfs-validator/issues/1370

Refactor upload methods
Handle errors better

Expected behavior

Unified upload process should work at least as well as it did before for both URL- and file-based uploads
The following failures should now show feedback in the UI:

Failed to get a job ID from the server

Failed to PUT the file

Failed to check for a report (other than expected 404 while waiting)

Failed to fetch local RULES.md

Checklist

[ ] Run the unit tests with gradle test to make sure you didn't break anything
[x] Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
[x] Linked all relevant issues
[x] Include screenshot(s) showing how this pull request works and fixes the issue(s)

Notes
~This branch is based on https://github.com/MobilityData/gtfs-validator/tree/master which used to be the feat/web branch in the Jarvus fork, hence all the extra commits.~

closed in favor of https://github.com/MobilityData/gtfs-validator/pull/1383


Remove md, add rst

This PR removes README.md and pansharpening_methods.md uses README.rst and pansharpening_methods.rst instead. Fixes: https://github.com/mapbox/rio-pansharpen/issues/15.

Current coverage is 96.77% (diff: 100%)

Merging #28 into master will not change coverage

@@             master        #28   diff @@
==========================================
  Files             5          5
  Lines           124        124
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            120        120
  Misses            4          4
  Partials          0          0

Powered by Codecov. Last update f9525c4...4188830


Fix: smooth 30 fps output

Fixed frames delay in gif generation. The ideal duration of each frame is (animationSeconds * 1000) / animationFPS. Also need BufferSource instead of ImageData to be able to set the delay param.
Before:

After:

Since the delay is the same for all frames, it could (and should) be extracted as a const.


Capture Screenshot as Mobile

Dear TagUI,
I just want to say thanks for your lovely tool!
How would I be able to capture the screen shot as mobile?

Thanks @lyonz! This is not supported but you can try editing browser display size in tagui_config.txt. It may work on some websites that check the browser resolution to decide which version to show. Another more troublesome way is to experiment with setting browser user agent, but it is not fool-proof either.
sample flow - mobile
https://www.nrdc.org
snap page

output of - tagui mobile
(desktop vs mobile)


Broken image in sea level rise tutorial

The first image on this tutorial is broken:
https://earthaccess.readthedocs.io/en/latest/tutorials/SSL/

@mfisher87 I can't seem to do it... could you add the label good first issue to this one?

Label added and sent an invite to this repo with triage access!


reflex\testing.py is using platform.system, causing pre-commit error for Windows

Describe the bug
The output of sys.platform and os.name are determined at compile time. platform.system() determines the system type at run time. https://stackoverflow.com/questions/4553129/when-to-use-os-name-sys-platform-or-platform-system
Using platform.system for platform-specified call will not let the pre-commit check ignore unix calls in Windows, thus causing pre-commit error "getpgid" is not a known member of module "os"
Let's change it to sys.platform https://docs.python.org/3.11/library/sys.html#sys.platform
To Reproduce
Run pre-commit in Windows
Expected behavior
pre-commit check pass
Screenshots

Specifics (please complete the following information):

Python Version: 3.11
Reflex Version: latest develop
OS: Windows
Browser (Optional):

Additional context
Good first issue

Can u assign this to me

I can't assign task. Tagging some maintainer @masenf @ElijahAhianyo

I can't assign task. Tagging some maintainer @masenf @ElijahAhianyo

Sure


Implement NSFW Toggle

As discussed in the Accrescent Matrix room, if 18+/NSFW content is allowed to be published to Accrescent (there doesn't currently seem to be a policy regarding this), implement a toggle to enable such content for download by users. Such content may include, but is not limited to, violence (usually found in video games) and sexual content (video games, video/streaming apps etc).
The toggle should be disabled by default to protect users who do not wish to view this type of content.
Developers should be required to mark their apps as 18+/NSFW if they contain such content, perhaps via a checkbox in the developer console.

implement a toggle to enable such content for download by users. Such content may include, but is not limited to, violence (usually found in video games) and sexual content (video games, video/streaming apps etc).

To be clear, the toggle should prevent viewing of such content, not just downloads. The application's very icon and name may cause discomfort for some audiences.
Furthermore this should be split up into more tags than an overbroad "NSFW." Arbitrary tag inclusion/exclusion listing should be permitted if applications are tagged with more than just sexual content or violence. This is useful for searching application stores that contain hundreds or thousands of applications.


long attribute category names in the legend color rectangles

What can be done about long attribute names that get cutoff in the legend?

Maybe just let them run out of the legend box?


double-beep: Blacklist zooksoftware.com

double-beep requests the blacklist of the website zooksoftware\.com. See the MS search here and the Stack Exchange search here.

zooksoftware\.com has been seen in 5 true positives, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


Do not export utils.

They're almost all gone and this will allow refactoring them with breaking backwards compatability.

Thanks!


the version 0.26.0 can not work well, init failed

Describe the bug

To Reproduce

Environment

Expected behavior

Screenshots

Additional context

when i enter app with arr version 0.26.0
errorMessage=degradeToH5|createInstance fail|wx_create_instance_error isJSFrameworkInit==false reInitCount == 1
i suggest developers user 0.24.0 to develop


add dependabot

Fixes Issue
Closes #37
Changes proposed
Screenshots
Note to reviewers

Thank you so much for your contribution :)


Ability to control blank lines between groups

I would like to reproduce my intellij config as closely as possible. In .idea/codeStyles/Project.xml I have this:
      <option name="importLayout">
        <array>
          <option value="java" />
          <option value="_______ blank line _______" />
          <option value="scala" />
          <option value="_______ blank line _______" />
          <option value="org.scalajs" />
          <option value="play" />
          <option value="net.liftweb" />
          <option value="reactive" />
          <option value="slick" />
          <option value="japgolly" />
          <option value="io.github.nafg" />
          <option value="_______ blank line _______" />
          <option value="all other imports" />
          <option value="_______ blank line _______" />
          <option value="chesednow" />
        </array>
      </option>

So for example, imports from japgolly should always go before imports from io.github.nafg, however I don't want a blank line between them.
Is there any way to achieve this, or could there be?

@nafg, that's a nice feature request. It cannot be done today but can be down relatively easily. I'd add one configuration named blankLines and a special group - representing the blank line.
When blankLines is set to Auto, which is the default value, OrganizeImports respects the current behavior, i.e., a blank line is automatically inserted between adjacent groups. When blankLines is set to Manual, you need to add the - group to insert blank lines manually.
With this, your sample configuration can be translated into:
OrganizeImports {
  blankLines = "Manual"
  groups = [
    "re:javax?\\."
    "-"
    "scala."
    "-"
    "org.scalajs."
    "play."
    "net.liftweb."
    "reactive."
    "slick."
    "japgolly."
    "io.github.nafg."
    "-"
    "*"
    "-"
    "chesednow"
  ]
}

WDYT?

Maybe use --- instead of - to make the blank line groups easier to locate visually.

That sounds like a great strategy.
On Mon, Dec 14, 2020 at 8:59 PM Cheng Lian<EMAIL_ADDRESS>wrote:

Maybe use --- instead of - to make the blank line groups easier to locate
visually.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/liancheng/scalafix-organize-imports/issues/140#issuecomment-745000904,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAYAUDUQ5GPESUKXZ4PDF3SU27IVANCNFSM4UZYH2YQ
.

@nafg I had a draft implementation for this feature in PR #142. It turned out to be much hairier than I initially expected. I'll try to see whether things can be refactored and simplified. On the other hand, here's a workaround that largely satisfies your requirement. You can configure the OrganizeImports as following:
OrganizeImports {
  groups = [
    "re:javax?\\."
    "scala."
    "re:(org\\.scalajs|play|net\\.liftweb|reactive|slick|japgolly|io\\.github\\.nafg)\\."
    "*"
    "chesednow"
  ]
}

The only thing is that the 3rd group will sort all the imports in that group according to the importsOrder configuration, so it does not strictly reflect the original order you wanted. However, I think it makes more sense to order them alphabetically instead of using an ad-hoc order? Are there any reasons why you prefer the specific order you listed in the PR description?

My main reason is that being able to use it without making changes across
the codebase. I primarily use intellij to organize imports. If I could run
a fast command line tool across the codebase that would not rock the boat
too much it would be nice.
On Wed, Dec 16, 2020, 2:47 PM Cheng Lian<EMAIL_ADDRESS>wrote:

@nafg https://github.com/nafg I had a draft implementation for this
feature in PR #142
https://github.com/liancheng/scalafix-organize-imports/pull/142. It
turned out to be much hairier than I initially expected. I'll try to see
whether things can be refactored and simplified. On the other hand, here's
a workaround that largely satisfies your requirement. You can configure the
OrganizeImports as following:
OrganizeImports {
groups = [
"re:javax?\."
"scala."
"re:(org\.scalajs|play|net\.liftweb|reactive|slick|japgolly|io\.github\.nafg)\."
"*"
"chesednow"
]
}
The only thing is that the 3rd group will sort all the imports in that
group according to the importsOrder configuration, so it does not
strictly reflect the original order you wanted. However, I think it makes
more sense to order them alphabetically instead of using an ad-hoc order?
Are there any reasons why you prefer the specific order you listed in the
PR description?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/liancheng/scalafix-organize-imports/issues/140#issuecomment-746906467,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAYAUFI3J2OOHJB3JTRMBLSVEFD5ANCNFSM4UZYH2YQ
.

@nafg, thanks for the context. So it's for migrating existing projects using IntelliJ import optimizer to OrganizeImports. That totally makes sense. Let me try to find some time to refactor #142 and add tests.


support flit for pyproject.toml

Add flit support. I know someone wants to use it with Deadpendency so let's do it.

Release as part of <IP_ADDRESS>


Coherent naming convention for SOA and OSB clusters

WKO uses in create-domain-inputs.yaml

a default static suffix "osb_cluster" when it creates the the OSB cluster
a changable suffix "soa_cluster" when it creates the the SOA cluster
if eg type soaessosb has been chosen.

See
Cluster name
clusterName: soa-cluster
This should be replaced by eg
osbClusterName: osb_cluster
soaClusterName: soa_cluster
to let users be able to maintain a coherent naming convention for cluster created by one declaration and generation from WKO.
A sample of what could happen is if I choose to set
domainUID: soainfra
clusterName: soa1_cluster
WKO would generate:
soainfra-osb-cluster
soainfra-soa1-cluster
not giving me the opportunity to maintain a naming convention for osb to match soa; ie ...osb1....

I can only deduce you are requesting the ability to configure the osb and soa cluster name when the domainType is set to soaosb?

Yes, for any of the domain types:
soa: Deploys a SOA domain
osb: Deploys an OSB (Oracle Service Bus) domain
soaess: Deploys a SOA domain with Enterprise Scheduler (ESS)
soaosb: Deploys a domain with SOA and OSB
soaessosb: Deploys a domain with SOA, OSB, and ESS
Especially for soaosb and soaessosb since two clusters are created for these.

I'm assuming the issue is resolved by now, feel free to re-open the issue if I'm incorrect.


Windows and Office 365 deployment lab kit

Download Links are not working anymore

@SmuSmu Please indicate which document you are referring to. Thanks

@SmuSmu We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if you see an improvement is required in the specific area of this documentation. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Run tests and acceptance tests in parallel

Splits the tests and acceptance tests in two jobs so they'll run in parallel. This should speed up the Checks workflow by a couple of minutes.

Rerunning the acceptance tests on docker flake (HACBS-1354) is now a bit quicker as well


Provide install number and update numbers separately

On VS Code Marketplace the number of "installs" is in fact the number of installations + updates.
it would be nice to provide the numbers of installation and updates separately. The advantages will be to better compare popularity of plugins between each other (a plugin releasing twice more often will have an higher number currently even if there is the same number of users)
it will allow to have a better insight on how many people are still using the extensions.

I´ve forwarded this to the Marketplace team. I´m closing the issue since it is nothing the VS Code team can act on, but the issue can still be used to communicate between you and the market place team.

Thanks Erich. We (marketplace) are working on this. Expect to get this enabled sooner.


Azure pipeline failure, how to resolve

Comment:
@sugatoray  linux linux_64_ job failed because "nothing provides requested ipyvizzu".
Is this because ipyvizzu and iypvizzu-story recipes were created in the same PR19838 (and that point ipyvizzu is not available yet)?
If the answer is yes, can we fix it by sending a "please restart ci" command to conda forge admin?

Typically this is fixed with sending an empty commit to a new PR (no changes to the recipe).


python311Packages.sqlalchemy-utils: 0.40.0 -> 0.41.1

Changelog: https://github.com/kvesteri/sqlalchemy-utils/releases/tag/0.41.1
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Update is part of #251878.


Lib size

Lib size is 20mb is it correct?

Unfortunately this library takes up too much space. Apk file was 7.5 mb. After the library it went up to 28.5mb. I am searching another library to show pdf. It is larger than the whole application. This is unacceptable.

If you have a link of pdf you can use this: http://docs.google.com/viewer?url=PDF_URL


Fixes #16 - Do not add getters and setters when there is no emit method.

@andyhall does this look good to you?

Looks great, thanks! I ran it in my app, and no problems.

Thanks!


docs: Update deprecated label section

Fixes #N/A
Description
Update docs for deprecated labels.These labels should only be used for persistent volumes.
https://github.com/aws/karpenter-provider-aws/issues/6440
How was this change tested?
Does this change impact docs?

[x] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/amifamily/ami.go
1
93.33%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
5967

Relevant Lines:
7551

💛  - Coveralls

Looks like we had intended to merge this in but it got closed-out by the stale-bot


make `make buildrpm' work fine on CentOS 5 and 6

This fixes BrighcoveOS/Diamond#12

I wish the .conf of each collector ended up in /etc/diamond/collectors instead. Currently they're along with their respective class files in /usr/share/diamond/collectors/

I wanted to put default config files with the collectors and then allow overrides in /etc/diamond/collectors. I don't expect people to edit the config files in /usr/share/diamond/collectors/.
This way, if we add a new default and someone updates and doesn't add it to their etc config, it still runs correctly vs bailing out due to a missing config option.
Thoughts?


Fixes #330: Running cloud task locally

Fixes #330
Changes proposed in this pull request:

Running cloud task locally

Coverage increased (+0.2%) to 91.94% when pulling f70c5f740f693fe4350233fea44c11c7114aa0eb on Esjob-Cloud-DevOps:2.1.3-SNAPSHOT into 88d0e824aece1c17105e47f583efb8519b25a509 on dangdangdotcom:2.1.3-SNAPSHOT.

shardingItemExpression to elastic job native config way.

Coverage increased (+0.1%) to 91.909% when pulling fc90361c1faf40e615b8c42c5d4c669b411b2efa on Esjob-Cloud-DevOps:2.1.3-SNAPSHOT into 88d0e824aece1c17105e47f583efb8519b25a509 on dangdangdotcom:2.1.3-SNAPSHOT.

Coverage increased (+0.1%) to 91.909% when pulling fc90361c1faf40e615b8c42c5d4c669b411b2efa on Esjob-Cloud-DevOps:2.1.3-SNAPSHOT into 88d0e824aece1c17105e47f583efb8519b25a509 on dangdangdotcom:2.1.3-SNAPSHOT.

Coverage increased (+0.1%) to 91.904% when pulling 27e90da89a73b8bf4855f51ca4dc05c15f129d20 on Esjob-Cloud-DevOps:2.1.3-SNAPSHOT into 88d0e824aece1c17105e47f583efb8519b25a509 on dangdangdotcom:2.1.3-SNAPSHOT.


Component definitions without rules have unexpected markdown editing behavior

Describe the bug
Trestle does not support OSCAL-compliant Component Definitions and require Rule properties for correct processing. When using OSCAL Component Definitions without rule properties, the markdown files are generated without the source implementation information. If the markdown is subsequently edited, the information is not added to the JSON.
Additional discussion and reproduction information is located on #1631
To Reproduce
Steps to reproduce the behavior:
trestle init
trestle import -f https://raw.githubusercontent.com/CivicActions/oscal-component-definitions/main/aws/oscal/aws.json -o aws
trestle author component-generate -o md_comp/aws --name aws
trestle author component-assemble --markdown md_comp/aws/ -o aws

Expected behavior
At minimum, an error should be thrown for unsupported OSCAL Component Definitions or control implementations.
From the discussion: trestle author component-generate is run for a component definition with no rule information, the resulting markdown files should contain the original information and allow editing of the OSCAL schema compliant implementation details (e.g. implementation description)
Screenshots / Logs.
Additional information - Adding rules manually to the JSON fixed the behavior
            "implemented-requirements": [
              {
                "uuid": "c9cbefe4-5c62-4a14-bbb6-91cb5658dfc8",
                "control-id": "ac-2",
                "description": "The system partially inherits this control from the FedRAMP Provisional ATO granted to the AWS Cloud dated 1 May 2013 for the following: AWS account management and other things.",
                "props": [
                  {
                    "name": "Rule_Id",
                    "ns": "http://ibm.github.io/compliance-trestle/schemas/oscal/cd",
                    "value": "Test-rule_001"
                  },
                  {
                    "name": "implementation-status",
                    "value": "implemented"
                  }
                ]
              }

Environment

OS: [e.g. iOS] Fedora
Python version: 3.11
Installed packages: trestle 3.3.0

Support Component Definition without rules
Determine the workflow with the SSP and make sure it is consistent


Use https://crates.io/crates/proc-macro-error

It would be good to use https://crates.io/crates/proc-macro-error to improve error reporting for proc macros.
Basically we should look everywhere we panic() in our proc macros and add span info.

Just wondering, how is that different from syn::Error?

Ah I see it mentioned in the readme further down. Nvm

https://github.com/graphql-rust/juniper/pull/441 lays the ground work for this. Once that is merged I’ll update the rest of code gen.

This is done.


Fix godeps for mangen

This fixes godep restore for the project.
Fixes https://github.com/GoogleCloudPlatform/kubernetes/issues/10195
/cc @eparis

See https://github.com/GoogleCloudPlatform/kubernetes/pull/10154 where we "fixed" this the first time. @timstclair broke it again (by mistake) in 66763316

https://github.com/GoogleCloudPlatform/kubernetes/pull/12153 seems like a better long term solution


Motivation for creating this library / fighting with the REST API

First of all, thank you very much for creating this tool, @Voyz. I've been using it since you fixed #225 and it's working great. Well, as great as it can, because it still has to restart the Client Portal gateway when it freezes, which means there will be downtime, guaranteed, of a few minutes at a time, several times a day in practice.
I was interested in IBeam because I was hoping the IB REST API would be easier to use than the stream-oriented TWS API, which was so weird that I had to use wrappers right away: the FIELDS OF an object, say an account summary, were STREAMED (WTF?). But anyway, with various wrappers, the TWS API worked and it was stable, requiring only one 2FA login per week, during the weekend. I can deal with that, and pretty much all ATSs can.
But as I actually looked into the REST API, what others said was confirmed to be true: it was a nightmare. Not just because of the restart requirements, but look at how confusing and inconsistent the endpoints are. Just to get contract details, you have at least three endpoints:

Contract information by Contract ID
Search SecDef information by conid
Search the security definition by Contract ID

Though some contract details are also returned by the Security Future by Symbol endpoint (conId, expiration dates) and the Live Market Data Snapshot endpoint among the Market Data Fields: SecType, Company name, Industry etc. - these are not market data like the bid/ask/volume, greeks, indicators and so on. And by the way, these market data field IDs are incompatible with the TWS API tick types.
So 5 (five) endpoints to get contract details. And not one of them gets ALL the details (exercise for the reader: get the multiplier of a contract). And they break compatibility with the established TWS API.
This is crazy.
Keep in mind that IB has been working on this REST API since 2018. And it's still the most disorganized API I've ever seen. Look at how many names there are for the underlying conid field: undConid, underlyingConid, underlying_con_id, underlying_conid. They can't even decide on one name for the option right field, it's right or putOrCall. Or the most basic of all fields, symbol, is sometimes returned as ticker .
Who wrote this crap?
So it seems to me that with a wrapper, the stream-oriented old TWS API is the lesser evil, as another dev said. A good wrapper can abstract the streaming nature when it doesn't make sense. And the field names are consistent.
But with the REST API, there's just no way to avoid the sudden disconnection problem, which can be a show stopper for many unattended trading scenarios.
So I'm honestly curious,

what was the motivation for writing IBeam
what is the hope for still working on it?
when is IBeam still useful given that connecting directly to api.ibkr.com via OAuth is apparently possible for regular users? (I haven't yet fully audited or tried that client)

Hey @Bomper thanks for sharing your thoughts. Let me address your questions:

Who wrote this crap?

As much as I sympathise with your frustration, it's I'd suggest to refrain from this kind of language. IBKR aren't all bad, there surely are developers there who try to provide some reasonable service to us, but likely are restrained by management or resources. We know next to nothing about the types of scenarios, clients and legacy compatibility they need to support, which may well explain the reasons behind such a convoluted system.

what was the motivation for writing IBeam

To automate the authentication with their new REST API. At the time it was originally written, around 2019, it wasn't as obvious as it is today that the REST API has so many issues. And they do improve it over time.

what is the hope for still working on it?

I don't understand this question - can you elaborate? Do you mean: why am I still working on it? Because it works, despite its issues, for many users including myself. I've spoken to people who've been using it for years, and as you know the alternative TWS API is not without flaws itself.

when is IBeam still useful given that connecting directly to api.ibkr.com via OAuth is apparently possible for regular users?

To be honest this sounds like a subjective question. If IBeam and CP Web API with the Gateway are not useful to someone, then that's fine. Currently OAuth is not fully supported and the Gateway has not yet been disabled. Until that happens and there are people using it, IBeam will remain useful.


Long links in description tend to overflow the <details> container

If there is a long link included in the description of an acmCSUF event on the acmcsuf.com/events page, the link will overflow the <details> container of the event item and does not look pretty. We need to force long words to somehow wrap instead of overflow out of the container.
Screenshot

Reopened after noticing that #331 broke all words no matter what, which is not our desired look for this component:


CentOS (6, 7, 8) builder + CLI refinements + new docker builder image

What type of PR is this?
/kind bug
/kind feature
Any specific area of the project related to this PR?
/area build
/area cmd
/area pkg
/area docs
What this PR does / why we need it:

Adds target "centos"
Adds builder for various CentOS versions: 6, 7, 8
Fixes the invocation of CLI without subcommands
Fixes the validation of persistent flags for subcommands
Introduces a new docker builder image containing gcc-4.8, gcc-5, gcc-6 to support a broader range of kernel/target combinations

Which issue(s) this PR fixes:
Fixes #5
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
new: centos target
new: centos6 builder
new: centos7 builder
new: centos8 builder
fix: CLI invocation without subcommands
fix: CLI persistent flags validation for subcommands
new: docker builder base image with various GCC

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: add38066d0e4608f09e60227d53da387514fb92a

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add an event for when a ProvisioningRequest is created for a Workload

What type of PR is this?
/kind feature

What this PR does / why we need it:
An event associated to a Workload, for when the ProvisioningRequest is created, including it's name. Useful for debugging problems.
Which issue(s) this PR fixes:

Fixes #1393
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/assign

/unhold

/lgtm


3dct - Seq nodes VRG output: incremented filenames- rather than series name

In pr #75

Can we allow vrgs to inherit their series name from the sequence node?

Just to confirm rather than outputting the Radiographs as <sequence_name>-<frame #>.tif you just want <frame #>.tif

If there are meaningful series descriptors in the sequences
generation (flex, adduct, grasp etc) we want to retain. If only sequential
(4dct frame #) indices are fine
On Mon, Mar 11, 2024 at 10:58 AM Anthony Lombardi @.***>
wrote:

Just to confirm rather than outputting the Radiographs as <sequence_name>-<frame
#>.tif you just want <frame #>.tif
—
Reply to this email directly, view it on GitHub
https://github.com/BrownBiomechanics/SlicerAutoscoperM/issues/84#issuecomment-1988642485,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APUUPFVHA76EANOZHZKJO3DYXXIBHAVCNFSM6AAAAABEB5AOFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYGY2DENBYGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Amy Morton, MSc
Sr. Research Engineer, Bioengineering Lab, Department of Orthopaedic
Research
The Warren Alpert Medical School of Brown University and Rhode Island
Hospital
1 Hoppin Street, CORO West, Ste. 404, Providence, RI 02903

Added new option to the advanced options tab, if checked only the index will be used in the file name if not the name parameter from the sequence will be used.


[Task][Code]: Create Modal Size Variant

Migrated task to appsmith repo https://github.com/appsmithorg/appsmith/issues/27296


I have a question about your code "DJSCC-for-Wireless-Image-Transmisson"

I use your code to train the model successfully. But the MSE(mean square error) is so high. For example, when SNR=20dB and compression ratio= 0.09, the MSE is about 0.009. But in paper named 'Deep Joint Source-Channel Coding for
Wireless Image Transmission", the MSE is about 0.001  when SNR=20dB and compression ratio= 0.09, as fig3 shows in paper. That's a big gap. I want to know that when you train the model, could you reach the performance the same as paper shows?

I tried to reproduce the results but I didn't succeed

I tried to reproduce the results but I didn't succeed

0.2356

0.2356

I have found a problem in your code. In the first layer and second layer of conv layers, you set the padding = 'VALID'. Now I change it to padding = 'SAME' and train the model again. The MSE decreased a lot. You can have a try.

I have found a problem in your code. In the first layer and second layer of conv layers, you set the padding = 'VALID'. Now I change it to padding = 'SAME' and train the model again. The MSE decreased a lot. You can have a try.

And you could set the learning rate = 1e-3, it will be better.

And you could set the learning rate = 1e-3, it will be better.

@xwk111 can you please send

@xwk111 can you please send

I want to ask you a question. How to reproduce the image transmission system(JPEG + LDPC + QAM)? Are there any python libs for JPEG algorithm with variable bit per pixel?

I want to ask you a question. How to reproduce the image transmission system(JPEG + LDPC + QAM)? Are there any python libs for JPEG algorithm with variable bit per pixel?


did anyone configured the gitlab pipeline plugin with the new backend api?

A bit of context can be found here: https://github.com/veecode-platform/platform-backstage-plugins/issues/25
However I am wondering if the new backend API was used to configure the new plugin version? If so, can someone point to me where can I see a tutorial only for the new backend?

The only way I could make it work with Backstage 1.28 was by adding the following line to the proxy configuration
proxy:
  endpoints:
    '/gitlab/api':
      target: https://gitlab.com/api/v4
+     credentials: dangerously-allow-unauthenticated
      allowedHeaders: ['Authorization', 'Content-Type']
      headers:
        Accept: application/json
        Content-Type: application/json

well well well, that did the trick, thanks for that!

adding the credentials to allow fixed my issue, however, is this the final solution?

I hope not. I'll try to do more debugging later to see if I can come up with the right solution, hopefully using credentials: require

yep, I tried to understand the auth flow but at some point in time I got lost with the proxy, but I see that there is a exchange between the token we use and gitlab api. I will post any updates I might find in here as well.


Meeting (Project Management) -  Wednesday, 07th August, 2024

Welcome
SingularityNET-Archive contains a history of the activities of the SingularityNET Ambassador Program.
All content is provided under an Open Source Apache 2 Licence.
Contributions will be licensed under a Creative Commons licence.
Video Recording
https://youtu.be/xX3wvH1iWdY
Our goal
Our goal is to provide a collaborative platform for the SingularityNET-Archive community to share knowledge, ideas, and best practices. We welcome contributions from anyone interested in SingularityNET, whether it's a new project, code example, or documentation.
Our Process
Please review the following at the beginning of each meeting.
And add any new Action or Decision items that arise from issues here. The assignee will update the issue, speak to the work and update with any comments
Action and Decision Items
Mark Action Items in each issue as follows -
https://archive-oracle.netlify.app
Previous meeting

https://github.com/SingularityNET-Archive/SingularityNET-Archive/issues/171

Links:

Project Board
Archival Workgroup proposal
Archive GitBook

Present:

[x] Andre
[ ] Stephen
[x] Vani
[ ] Peter
[x] Onyeka
[x] Colleen
[x] Kateri
[x] Photogee
[x] Effiom
[x] Onize
[x] Sucre
[x] LadyTempestt
[ ] Duke
[x] Jenn
[x] Gorga
[x] Clement
[ ] Ekemini
[ ] Ayo
[ ] Martin

Archive

[x] Add to meeting summary channel

[ ] Close meeting issue (to ensure charge in correct month)

[x] Meeting uploaded to YouTube https://youtu.be/6WczGZ5u6hQ

[ ] Meeting recorded in Archive GitBook

Previous in series: #192

Next in series: #197


Add word "[Jannock]

Add all the definitions of [Jannock] in a file named [Jannock].json in folder [J]
Refer CONTRIBUTING.md for more details on how to contribute.
Comment the words you'll be adding if you're are working on this issue to avoid conflicts.
Add reference to this issue i.e Issue # in your pull request description.

Jannock


Unable to import Earth on multiple platforms

I believe there is some sort of compatibility issue with the entire module that prevents me from importing Earth from the pyearth library. This same issue is happening on cloud computing platforms, such as Google Colab or AWS EMR, as well as locally, on both Windows and MAC OS.
Here is a recent post where the user ran into the same problem. This is my own post outlining another instance of this issue.
Google Colab example
!pip install pyearth
from pyearth import Earth

ImportError: cannot import name 'Earth' from 'pyearth' (/usr/local/lib/python3.7/dist-packages/pyearth/init.py)

Windows example
from pyearth import Earth

ImportError                               Traceback (most recent call last)
 in
----> 1 from pyearth import Earth
ImportError: cannot import name 'Earth' from 'pyearth' (D:\Users\XXXXXXX\Anaconda3\lib\site-packages\pyearth_init_.py)

Oddly enough, import pyearth runs smoothly on both platforms.

@ArturoSbr I would recommend building pyearth directly from git instead of using pip install and see if the problem persists. That is clone the repo to your machine and run
python setup.py install
Depending on your python version, you may have to instead clone branch issue 191 and run
python setup.py install --cythonize

pips listing of pyearth is an earth-science package, not this one:  https://pypi.org/project/pyearth/
To use pip to install, specify sklearn-contrib-py-earth:  https://pypi.org/project/sklearn-contrib-py-earth/
(That said, I agree with @kevin-dietz, and would suggest also considering installing from the v0.2dev branch.  You can do this directly with pip, e.g. pip install<EMAIL_ADDRESS>

@bmreiniger Thank you! This worked. I am confused though. What is the difference between pyearth and sklearn-contrib-py-earth?

@ArturoSbr there's absolutely no connection besides an unfortunate naming collision.  pyearth on PyPI is, as you can read from the link I gave, an earth science package; this py-earth package is for spline regression, specifically the MARS algorithm (but the name is trademarked, and so this and other packages have taken to calling their implementations "earth").

That is super unfortunate. I will close the issue. Thanks for your help!


add csharp sample code for public api 2023-12-01-preview

Purpose
add csharp sample code for public api 2023-12-01-preview
Pull Request Type
What kind of change does this Pull Request introduce?
[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

/azp run

/azp run

/azp run

/azp run


Is this project dead?

There are now two forks on Github that have on-going maintenance while there's not really any activity here ... should I switch to one of the maintained projects?  As you've seen from my previous PRs, I'm not afraid to help with this project but you'd need to be around to at least merge (or reject) the PRs.

I've started using https://github.com/gowebapi/webapi instead which is generated from the Web IDL provided by Firefox.  I'm not a contributor to those projects and will be moving the test server from this project over there and making it a bit more configurable so that it's compatible with Skaffold and Ko.

https://github.com/dominikh/go-js-dom
is in better state, at least it supports (v2) latest Go.

https://github.com/dominikh/go-js-dom
is in better state, at least it supports (v2) latest Go.

Yeah, sorry, this project is slightly dead. I don't have time to maintain it at the moment :(

Yeah, sorry, this project is slightly dead. I don't have time to maintain it at the moment :(

If anyone is interested in taking it further, I can add a contributor or transfer an ownership.

If anyone is interested in taking it further, I can add a contributor or transfer an ownership.


Registry authentication service token verification

Hi
I'm doing authentication service with Django for Docker registry.

I've specified realm config parameter to get Token from authentication service, and it works fine with Docker client, but I can't find any documented part how make a validation of Token sent from authentication service to Docker client.

As documented here https://docs.docker.com/registry/spec/auth/token/ Authentication service responding to Docker client (daemon) with Bearer Token using JSON response. Then client making HTTP request with Authorization: Bearer header, but where is the validation of Token in Registry part ?

Thanks

@tigranbs : The registry can be configured with an auth type of token:
https://docs.docker.com/registry/configuration/#auth
When this is configured the registry will validate requests with the given token.

@RichardScothern thanks for response, after all day playing with certificates finally I made token authentication which is described here https://docs.docker.com/registry/spec/auth/jwt/
Closing this issue


Regression of -C module.type=commonjs

I got a report from a korean community.
v.1.2.177 works, while v1.2.178 fails
https://github.com/swc-project/swc/blob/main/CHANGELOG.md#12178---2022-05-07

Found the cause, import had . inside the import path
So this is duplicate of issues I was going to fix today or tomorrow, but I'll keep this as open and close all at once with a PR


Runelite Lagging

Hello everyone. I'm having an odd issue with the RuneLite client. Whenever it starts up it makes my screen flicker until it loads the login screen. After that, everything is fine until I try and use the sidebar features. It then makes my fps drop drastically and sometimes shuts my screen off for a second or two.
I've also followed the accelerator guide in turning it off, it fixed an initial issue at first but was no help for this particular one.
If anyone has any idea why this could be happening please let me know.

The issue tracker is not a support forum. Please join the Discord for questions of support.


Repo Linter

This is certainly worth checking out with respect to the Risk category:
https://github.com/todogroup/repolinter

I've been doing some of my own investigation into this too, and I've also found this repo: https://github.com/mntnr/dashboard

Thanks @BenLloydPearson. I'll keep this issue open as I think we're going to be pushing the Risk metric sometime in June. I really do think that these plug-in well with what we should be capturing.

@sgoggins, do you think that this has anything relevance in Risk?

@germonprez : I think both are tools that could be evaluated for their utility for risk, yes. Since tools like these are relatively declarative (it has a thing? no?) I think what they do could prove useful. @BenLloydPearson : do you have a need for these tools in your work, or could you anticipate one?


Let intersects() geometry algorithm be commutative

Related to #122 #584

We probably shouldn't merge this because it would fall apart if we promote {ArborX::Detail:: -> ArborX::}intersects


resync with luminous

Signed-off-by: Sébastien Han<EMAIL_ADDRESS>

jenkins test ceph_ansible-jewel-centos7-dedicated_journal

current failures need https://github.com/ceph/ceph-ansible/pull/1939

jenkins test this please

jenkins test this please

jenkins test ceph_ansible-jewel-xenial-dedicated_journal


Option to restart worker based on metrics (i.e. memory consumption / tasks processed)

It would be very convenient to have the option to restart a worker process once it hits a certain threshold, such as memory consumption or number of tasks processed. In long-running deployments, this can help with things such as memory leaks that are out of control of the application.

Celery allows this with their worker_max_tasks_per_child and worker_max_memory_per_child, which I default all my celery deployments to. Migrating to saq, this has been a bit of a holdup.
If this is something you'd consider adding, I'd be happy to contribute a PR :)

sure, i'm open to it, feel free to make a PR. thank you.
closing as not planned since core team won't work on it. but will work with you to get it in.


Set up sphinx-stack for the first time $@100000

Install and run sphinx-stack locally and provide feedback:

any errors or confusing points
your hardware/OS

first time installation worked for me flawless and without errors or confusion…

@ckoys can you add some notes from the install process, otherwise I'll go ahead and pay out the bounty? I'll ping you on sphinx

I've tried on two separate (Linux/ubuntu) boxes: initially on my desktop and then on a dev box I have running. The same error occurred on both, (see attached) on my desktop ./clearall.sh was enough to resolve it on my desktop, but not on the dev box.
sphinx-relay-setup.log

Looks like stack might be using an older version of the relay image cause its trying to get the transport token but that endpoint might not be available would you be able to check to see if that image is upto date with what we have here
https://hub.docker.com/r/sphinxlightning/sphinx-relay/tags

sure,
user@host:~/code/sphinx-relay$ docker logs  carol.sphinx
[info] 03-28-22T14:43:32 [MISC] => env development
[info] 03-28-22T14:43:32 [DB] => [db] starting setup
[info] 03-28-22T14:43:35 [DB] => [db] sync now
[info] 03-28-22T14:46:26 [DB] => [db] done syncing
[info] 03-28-22T14:47:13 [DB] adding lsat table
[info] 03-28-22T14:47:18 [DB] adding requestsTransportTokens table
[info] 03-28-22T14:47:18 [DB] => [db] setup done
Mon, 28 Mar 2022 14:47:18 GMT body-parser deprecated undefined extended: provide extended option at dist/app.js:81:25
[info] 03-28-22T14:47:18 [MISC] => 2022-03-28 14:47:18 [lightning] reconnecting... attempt #1
(node:1) DeprecationWarning: grpc.load: Use the @grpc/proto-loader module with grpc.loadPackageDefinition instead
[info] 03-28-22T14:47:18 [MISC] Node listening on 3003.
[info] 03-28-22T14:47:18 [MISC] => [lightning] connected! 2022-03-28 14:47:18
[info] 03-28-22T14:47:18 [MISC] >>> FINISH SETUP
[info] 03-28-22T14:47:34 [MISC] [db] created node owner contact, id: 1
[info] 03-28-22T14:47:34 [MISC] => Relay version: , commit:
[info] 03-28-22T14:47:34 [MISC] >> aXA6Omh0dHA6Ly8xMjcuMC4wLjE6MzAwMzo6ODI3MWM2OTNlMTg0NDNhYjJmZGRhMjRkYzJjMmVlMjY=
[info] 03-28-22T14:47:34 [MISC] Scan this QR in Sphinx app:
[info] 03-28-22T14:47:34 [MISC]

[info] 03-28-22T14:47:39 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883
[info] 03-28-22T14:47:45 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883
[info] 03-28-22T14:47:50 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883
[info] 03-28-22T14:48:04 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883
[info] 03-28-22T14:48:13 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883
[info] 03-28-22T14:48:22 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883
[info] 03-28-22T14:48:31 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883
[info] 03-28-22T14:48:42 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883
[info] 03-28-22T14:48:51 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883
[info] 03-28-22T14:49:01 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883
[info] 03-28-22T14:49:14 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883
[info] 03-28-22T14:49:24 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883
[info] 03-28-22T14:49:30 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883
-> 2022-03-28 14:49:33: GET /is_setup 200 19ms
-> 2022-03-28 14:49:34: GET /request_transport_key 401 1ms
-> 2022-03-28 14:49:34: GET /request_transport_key 401 1ms
[info] 03-28-22T14:49:37 [TRIBES] [tribes] try to connect: tcp://host.docker.internal:1883

That is super weird because request_transport_key should be totally public...

@Lombitron
I just saw this on my stack would you be able to try and pull in the latest code for sphinx-stack and try it and if that doesnt work try going to the latest version of relay and run docker build -t sphinxlightning/sphinx-relay .

sphinx-stack works on my desktop fine (has always worked ok here, I did have to manually build the image for relay 1st though)
I cant use that dev box for sphinx-stack anymore (used for another project atm)
I'll spin up on another box when I get a chance - I got as far as working out part of the problem was due to running a DNS resolver locally on that box (it messes up Docker internal naming and the host.docker.internal domain gets sent to the resolver to be not resolved)
my fix was to just not use that box ... i didn't look into it further

I noticed that issue aswell with host.docker.internal when trying to get sphinx-stack running on popOS yesterday when working with another dev, I might take a look into trying to find a more elegant solution to that instead of using host.docker.internal


Need a list of all the UAI

**Need a list of UAI with below action **

[ ] Heading

[ ] Message

[ ] Priority

[ ] CTAs

[ ] Trigger

[ ] Check on CTA

[ ] Action/ screen on CTA

Verified this issue on keeper app v 1.1.12(303)


Need to transmit diagnostic data

The blaster needs to be able to transmit diagnostic data to the iOS companion application for monitoring battery health, current output, etc.
The following data points are required from Motor_Driver_1:

M1CS
M2CS

The following data points are required from Motor_Driver_2:

CS

More information may be added after issue #10 has been closed and something is monitoring the battery health via the balance connector.

This should now be enabled by #42 and exposing GATT characteristics with no write permission and read only with notification on the client

The GATT notification characteristics worked as expected. The value from the flywheel M1 and M2 sensors along with the belt M1 sensor has been exposed as characteristics with a read notification with no write privileges.
As the blaster is active it will transmit notifications much more frequently to the connected device.

Need to connect the blaster up to the battery pack and see what data comes across to the connected device.


FunctionTypeVar hasSelf not correctly set for some methods

Example:
type Object<P> = {
    New: (...any) -> (Object<P>)
} & P

type Account = Object<{
    Balance: number,

    Deposit: (self: Account, Amount: number) -> (),
}>

local Account = {}

function Account.New()
    local self = setmetatable({}, Account)
    self.Balance = 10

    return self
end

function Account:Deposit(Amount: number) -- hasSelf is set to false on the FunctionTypeVar produced for this method
    self.Balance += Amount
end

local a: Account = Account.New()

a:Deposit(10)

The impact later is that for the call a:Deposit(10), since hasSelf is false, trying to hover to see the function definitions gets stringified as:
function Account:Deposit(self: {| Balance: number, Deposit: (self: <CYCLE>, Amount: number) -> (), New: (...any) -> <CYCLE> |}, Amount: number): ()

when calling Luau::toStringNamedFunction with opts.hideFunctionSelfArgument enabled.
Ideally it would be stringified to the (much nicer)
function Account:Deposit(Amount: number): ()

Just noticed in the example that the type for Deposit is being specified in the type alias at the top, which is most likely why the hasSelf is not inferred correctly.

This works when using the more common OOP pattern:
local Account = {}
Account.__index = Account

function Account.New(): Account
    local self = setmetatable({}, Account)
    self.Balance = 10

    return self
end

function Account:Deposit(Amount: number)
    self.Balance += Amount
end

type Account = typeof(Account.New())

local a = Account.New()
a:Deposit(10)

Yeah I'm not sure if this is by design or not. Maybe there's argument that the function definition should override hasSelf in the type annotation but I'm not sure that's right. @alexmccord @vegorov-rbx thoughts?

I just wanted to point out that this will be even more problematic with the change in the recommended way to do OOP in https://github.com/Roblox/luau/pull/578
Is it reasonable to assume "hasSelf" if the first argument is named as "self" and annotated with the same type as the one being assigned?

I can no longer reproduce this issue.
And I also can't tell when the chance that fixed this happened.
There was some interaction with 'LuauSharedSelf' flag that has now become disabled by default (as 'DebugLuauSharedSelf'). But even then the hasSelf parameter was set for all functions defined with ':'.


Use --pure-lockfile

per https://github.com/newhavenio/newhavenio.github.io/pull/94#discussion_r186862364

I left a comment in #94 but I'll add it here too. I don't think this is necessary in recent versions of yarn.

@jnimety You've convinced me that it is not needed.


Unable to download package nerdfonts

Describe the bug
When I do home-manager switch it is blocked.
Image: https://zupimages.net/up/19/50/72tl.jpg
To Reproduce
Steps to reproduce the behavior:

Add nerdfonts to home.packages in home.nix. Screenshot: https://zupimages.net/up/19/50/ls6k.jpg
Save and perform home-manager switch

Expected behavior
It will try to download nerdfonts from github but it'll never end.
Screenshots
Image: https://zupimages.net/up/19/50/72tl.jpg
Metadata

system: "x86_64-linux"
host os: Linux 5.4.2, NixOS, 19.09.1549.45ea6092203 (Loris)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3
channels(thomas): ""
channels(root): "home-manager, nixos-19.09.1549.45ea6092203"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
  https://github.com/NixOS/nixpkgs/tree/master/pkgs/data/fonts/nerdfonts

Out of curiosity how long did you wait? The source archive of this package is quite sizeable at ~ 2GB; depending on your connection it could take a good while to complete unless otherwise it downloads quickly outside of a Nix build.

Out of curiosity how long did you wait? The source archive of this package is quite sizeable at ~ 2GB; depending on your connection it could take a good while to complete unless otherwise it downloads quickly outside of a Nix build.

One hour and I got that at the end: https://zupimages.net/up/19/50/go50.jpg

It means the connection dropped during download and resuming it failed
Simply trying again could fix it

It means the connection dropped during download and resuming it failed
Simply trying again could fix it

Okay but then why does the percentage never changes? Also is there a way to download only one font? I'ld like to download only fura code.

I'm currently downloading the entire tar.gz file using aria2c, how could I tell to nixpkgs to use this file instead of redownloading it?

nix-prefetch-url file:///path/to/your/file --unpack should do it AFAIK

nix-prefetch-url file:///path/to/your/file --unpack should do it AFAIK

I think nixos isn't able to recognize that this is the same file: https://zupimages.net/up/19/50/eamz.jpg

Hrm. Possibly wrong arguments given. Try without --unpack. But really, I have no clue

Hrm. Possibly wrong arguments given. Try without --unpack. But really, I have no clue

it still tries to download it :/

It's still quite annoying. They do have release archives for the individual fonts, but that would be a bit more effort to fetch since we couldn't just use fetchFromGitHub anymore. It would be nice to be able to just get a single font!

I’ve also encountered issues when trying to fetch nerdfonts (via nixos-rebuild)
Download errors seem to occur the most at my workplace where queue trees are enabled.
My best workaround so far has been to... use another network 😁
Haven’t seen it happen it quite some time, though.

The command
$ nix-prefetch-url file:///path/to/your/file --unpack --name source

may work, I haven't tried.

Fixed in last version.


ISSUE #1655 solved Added AI powered doddle classifier

This pull request introduces an AI-powered doodle classifier using the Canvas API and TensorFlow.js. Users can draw sketches directly on the canvas, and the model predicts their doodles in real-time, providing an engaging and interactive experience.

fix #1655


CLI fails to start on macOS

Tried running cargo run in the psst-cli folder, and got this:
thread 'main' panicked at 'Expected <track_id> in the first parameter', psst-cli/src/main.rs:20:10

Nevermind, I just realises this is be design 🤦
Closing


Streaming Module?

Hi, I've been trying to run the workflow on Galaxy installed on MacOS Big Sur and ran into this error:
Can't locate Streaming.pm in @INC (you may need to install the Streaming module) (@INC contains: /Users/phemarajata/galaxy/database/dependencies/_conda/bin /Users/phemarajata/galaxy/database/dependencies/_conda/bin/../lib/5.30.3/darwin-thread-multi-2level /Users/phemarajata/galaxy/database/dependencies/_conda/bin/../lib/5.30.3 /Users/phemarajata/galaxy/database/dependencies/_conda/bin/../lib /usr/local/Library/Perl/5.28 /Users/phemarajata/galaxy/database/dependencies/_conda/lib/site_perl/5.30.3/darwin-thread-multi-2level /Users/phemarajata/galaxy/database/dependencies/_conda/lib/site_perl/5.30.3 /Users/phemarajata/galaxy/database/dependencies/_conda/lib/5.30.3/darwin-thread-multi-2level /Users/phemarajata/galaxy/database/dependencies/_conda/lib/5.30.3 .) at /Users/phemarajata/galaxy/database/dependencies/_conda/bin/vcf2snv_alignment.pl line 16.
BEGIN failed--compilation aborted at /Users/phemarajata/galaxy/database/dependencies/_conda/bin/vcf2snv_alignment.pl line 16.
I can't seem to find Streaming module anywhere. How do I install it? Thank you!

I thought I fixed this but I didn't. @apetkau could you please help? Thank you so much!

As per my response in https://github.com/phac-nml/snvphyl-tools/issues/11 the SNVPhyl pipeline is only available for Linux right now. It may be possible to get some of the dependencies installed and working on MacOS, but I cannot make any guarantees.

Also note that you can try running the Galaxy Docker image on a Mac and then installing the SNVPhyl pipeline. There's currently a version with the SNVPhyl tools already installed (among other tools) on Docker hub which you can run like:
docker run -d -p 48888:80 phacnml/galaxy-irida-20.05

Then go to http://localhost:48888 and login with username<EMAIL_ADDRESS>and password password. This https://github.com/phac-nml/irida/blob/development/src/main/resources/ca/corefacility/bioinformatics/irida/model/workflow/analysis/type/workflows/SNVPhyl/1.2.2/irida_workflow_structure.ga would be the latest SNVPhyl workflow.

I'm trying the Docker container now. Am I allowed to install new tools inside this container if I'm running Docker with an export option? I tried to install Trimmomatic and when I started the job I got an error saying "This tool was disabled before the job completed.  Please contact your Galaxy administrator."
Thank you!

You can install new tools. For any errors though you may need to look at the Galaxy logs (run docker logs -f [container id]) to see what's going on.

@apetkau I think per this log somehow the tool got removed. It installed properly and was listed in the menu.
==> /home/galaxy/logs/handler0.log <==
galaxy.jobs.handler WARNING 2020-09-17 15:24:23,595 (8) Tool 'toolshed.g2.bx.psu.edu/repos/pjbriggs/trimmomatic/trimmomatic/0.38.1' removed from tool config, unable to run job
galaxy.jobs DEBUG 2020-09-17 15:24:23,599 (8) fail(): Job destination raised JobMappingException('Can't map job to destination, tool 'toolshed.g2.bx.psu.edu/repos/pjbriggs/trimmomatic/trimmomatic/0.38.1' is unavailable'), caching fake 'fail' destination for completion of fail method
galaxy.tools.error_reports DEBUG 2020-09-17 15:24:23,784 Bug report plugin <galaxy.tools.error_reports.plugins.sentry.SentryPlugin object at 0x7f79e0300f98> generated response None
galaxy.tools.error_reports DEBUG 2020-09-17 15:24:23,785 Bug report plugin <galaxy.tools.error_reports.plugins.sentry.SentryPlugin object at 0x7f79e0300f98> generated response None
galaxy.tools.error_reports DEBUG 2020-09-17 15:24:23,785 Bug report plugin <galaxy.tools.error_reports.plugins.sentry.SentryPlugin object at 0x7f79e0300f98> generated response None
galaxy.tools.error_reports DEBUG 2020-09-17 15:24:23,785 Bug report plugin <galaxy.tools.error_reports.plugins.sentry.SentryPlugin object at 0x7f79e0300f98> generated response None

If you still have docker running you can try restarting Galaxy in Docker (without restarting the Docker container) like:
docker exec -it [container id] supervisorctl restart galaxy:

Beyond that though I may have to redirect you to the Galaxy Docker project https://github.com/bgruening/docker-galaxy-stable for more details on how to customize (e.g., install new tools) in the Galaxy Docker container.


x/tools/go/ssa: add support for range-over-int

Per https://github.com/golang/go/issues/61405#issuecomment-1727898328, it is likely that range-over-int will land for 1.22.
This issue tracks supporting this new range form in x/tools/go/ssa. Marking as a release blocker because we endeavor to keep our tools in sync with language changes.

I think the change to the builder is actually very small: https://go.dev/cl/532580
The hard part is plumbing the version through.


SyntaxError: Unexpected token '<' when running Jest

Describe the bug
I'm using ignite to create an expo react native app. I'm using this guide https://ignitecookbook.com/docs/recipes/GeneratorComponentTests to create component test files
Here is the first test file

import React from "react"
import { fireEvent, render, screen } from "@testing-library/react-native"
import { Profile } from "../Profile"

describe("Profile", () => {
  it("renders", () => {
    render(<Profile />)
    expect(screen.getByText("Hello")).toBeTruthy()
  })
})

I followed https://reactnativetesting.io/component/setup/ in order to install @testing-library/react-native but now I'm getting
 ● Test suite failed to run

    Jest encountered an unexpected token

    Jest failed to parse a file. This happens e.g. when your code or its dependencies use non-standard JavaScript syntax, or when Jest is not configured to support such syntax.

    Out of the box Jest supports Babel, which will be used to transform your files into valid JS based on your Babel configuration.

    By default "node_modules" folder is ignored by transformers.

    Here's what you can do:
     • If you are trying to use ECMAScript Modules, see https://jestjs.io/docs/ecmascript-modules for how to enable it.
     • If you are trying to use TypeScript, see https://jestjs.io/docs/getting-started#using-typescript
     • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
     • If you need a custom transformation specify a "transform" option in your config.
     • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

    You'll find more details and examples of these config options in the docs:
    https://jestjs.io/docs/configuration
    For information about custom transformations, see:
    https://jestjs.io/docs/code-transformation

    Details:

    /Users/giahuy/PizzaApp/app/components/specs/Profile.spec.tsx:12
            (0, react_native_1.render)(<Profile_1.Profile />);
                                       ^

    SyntaxError: Unexpected token '<'

Here my jest.config.js
const { defaults: tsjPreset } = require("ts-jest/presets")

/** @type {import('@jest/types').Config.ProjectConfig} */
module.exports = {
  ...tsjPreset,
  preset: "react-native",
  transformIgnorePatterns: [
    "<rootDir>/node_modules/(react-clone-referenced-element|@react-native-community|react-navigation|@react-navigation/.*|@unimodules/.*|native-base|react-native-code-push)",
    "jest-runner",
  ],
  testPathIgnorePatterns: ["<rootDir>/node_modules/", "<rootDir>/.maestro/", "@react-native"],
  testEnvironment: "jsdom",
  setupFiles: ["<rootDir>/test/setup.ts"],
  transform: {
    "^.+\\.spec.tsx?$": [
      "ts-jest",
      {
        tsconfig: "<rootDir>/test/test-tsconfig.json",
      },
    ],
  },
}

how can  I fix it ??
Ignite version
9.3.1
Additional info
System
  platform           darwin
  arch               arm64
  cpu                8 cores      Apple M2
  directory          PizzaApp     /Users/giahuy/PizzaApp

JavaScript (and globally-installed packages)
  node               18.18.2      /Users/giahuy/.nvm/versions/node/v18.18.2/bin/node
  npm                9.8.1        /Users/giahuy/.nvm/versions/node/v18.18.2/bin/npm
    corepack         0.19.0
    npm              9.8.1
  yarn               1.22.21      /opt/homebrew/bin/yarn
    create-jest      29.7.0
  pnpm               -            not installed
  bun                -            not installed
  expo               49.0.21      managed

Ignite
  ignite-cli         9.3.1        /Users/giahuy/.npm/_npx/e31027f3785124a8/node_modules/.bin/ignite
  ignite src         build        /Users/giahuy/.npm/_npx/e31027f3785124a8/node_modules/ignite-cli/build

Android
  java               17.0.9       /usr/bin/java
  android home       -            /Users/giahuy/Library/Android/sdk

iOS
  xcode              15.1
  cocoapods          1.14.3       /opt/homebrew/bin/pod

Tools
  git                git version 2.39.3 (Apple Git-145)   /usr/bin/git

This also might happen if you file is named .js or .ts instead of .jsx/.tsx. The "X" in the file name indicated that the compiler should expect XML in the javascript

@frankcalise
Here is Profile component
import * as React from "react"
import { StyleProp, TextStyle, View, ViewStyle } from "react-native"
import { observer } from "mobx-react-lite"
import { colors, typography } from "app/theme"
import { Text } from "app/components/Text"

export interface ProfileProps {
  /**
   * An optional style override useful for padding & margin.
   */
  style?: StyleProp<ViewStyle>
}

/**
 * Describe your component here
 */
export const Profile = observer(function Profile(props: ProfileProps) {
  const { style } = props
  const $styles = [$container, style]

  return (
    <View style={$styles}>
      <Text style={$text}>Hello</Text>
    </View>
  )
})

const $container: ViewStyle = {
  justifyContent: "center",
}

const $text: TextStyle = {
  fontFamily: typography.primary.normal,
  fontSize: 14,
  color: colors.palette.primary500,
}

I also removed  testEnvironment: "jsdom" like you said but it doesn't work .Did i miss something?

@markrickert  My component file and test file are both tsx files.

@GiaHuy216 - I just got my tests to work again, had to remove: testEnvironment: "jsdom", as suggested by @frankcalise and then replaced the transform, here is what I have now as my jest.config.js (including commented out code):
const { defaults: tsjPreset } = require("ts-jest/presets")

/** @type {import('@jest/types').Config.ProjectConfig} */
module.exports = {
  ...tsjPreset,
  preset: "jest-expo",
  transformIgnorePatterns: [
    "<rootDir>/node_modules/(react-clone-referenced-element|@react-native-community|react-navigation|@react-navigation/.*|@unimodules/.*|native-base|react-native-code-push)",
    "jest-runner",
  ],
  testPathIgnorePatterns: ["<rootDir>/node_modules/", "<rootDir>/.maestro/", "@react-native", "<rootDir>/test/i18n.test.ts"],
  // testEnvironment: "jsdom",
  setupFiles: ["<rootDir>/test/setup.ts"],
  "setupFilesAfterEnv": [
    "@testing-library/jest-native/extend-expect"
  ],
  coverageDirectory: "<rootDir>/test/coverage",
  collectCoverageFrom: [
    "**/*.{js,jsx,ts,tsx}",
    "!**/node_modules/**",
    "!**/vendor/**",
    "!**/test/**",
    "!**/coverage/**",
    "!**/.expo/**",
    "!**/android/**",
    "!**/ios/**",
  ],
  // moduleNameMapper: {
  //   "app/(.*)": "<rootDir>/app/$1",
  // },
  transform:{
    // '^.+\\.test.tsx?$': ['ts-jest', {
      // tsconfig: '<rootDir>/test/test-tsconfig.json'
    // }],
    "^.+\\.[jt]sx?$": "babel-jest",
  },
  moduleFileExtensions: ["ts", "tsx", "js", "jsx", "json", "node"],
  "globals": {
    "ts-jest": {
      "diagnostics": false
    }
  }
}

Let me know if this works, or causes other issues I haven't come across yet

@3210jr thanks for confirming it's working. it also aligns with what I found here on RNTL's issue board: https://github.com/callstack/react-native-testing-library/issues/896#issuecomment-1165384492
Will make that update to the boilerplate

I can't reproduce this in the latest Ignite, @GiaHuy216 are you still experiencing issues?

@3210jr After I copied your jest.config.js file and it worked perfectly,

:tada: This issue has been resolved in version 9.6.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


SetScript second param can be nil

When using nil as second param to SetScript you currently get a warning, looks like in https://github.com/Ketho/vscode-wow-api/blob/1fea0434899666e194c8a18eeb2cbff3bde4a713/EmmyLua/API/Widget/Abstract/ScriptObject.lua#L25 you defined that though.

Not sure what's up

Yeah, I haven't pushed a release yet with the new emmylua changes. It has been a while 😓

yay for a new release


Release 0.6.0

I suggest we upgrade to a minor version because this version will  include https://github.com/containers/oci-spec-rs/issues/115

Codecov Report

Merging #120 (33f53f7) into main (07c2a12) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #120   +/-   ##
=======================================
  Coverage   24.38%   24.38%
=======================================
  Files          21       21
  Lines        1636     1636
  Branches      804      804
=======================================
  Hits          399      399
  Misses        547      547
  Partials      690      690

@saschagrunert May I ask you to review it?

@saschagrunert Thanks a lot :heart_eyes:


Add a Gitter chat badge to README.markdown

symphonycms/symphony-2 now has a Chat Room on Gitter
@brendo has just created a chat room. You can visit it here: https://gitter.im/symphonycms/symphony-2.
This pull-request adds this badge to your README.markdown:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

:rocket:

:rocket: :guitar:
@brendo Do you want me to manually merge it into integration ?

Ya sure!
On Wed, May 20, 2015 at 9:26 AM, Nicolas Brassard<EMAIL_ADDRESS>wrote:

[image: :rocket:] [image: :guitar:]
@brendo https://github.com/brendo Do you want me to manually merge it
into integration ?
—
Reply to this email directly or view it on GitHub
https://github.com/symphonycms/symphony-2/pull/2431#issuecomment-103696801
.

I've merged it into 2.6.x so it will be featured in the README sooner ;)


request new university for free license

@bomlinh I am sorry, the domain seoultech.ac.kr is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


MathUtils trigonometry improvements

This includes a bug fix and some additions, as well as minor doc fixes in 3 files. The bug fix is for MathUtils.atanUnchecked() and MathUtils.atan(), which affects MathUtils.atan2(); when given a point on the positive x-axis, they should return 0f, but didn't before. The additions are degree versions of asin, acos, atan, and atan2, plus the new atan2Deg360(), which won't return a negative degree value (its range is 0-360). The behavior of Math.atan2() where it can produce negative results often makes me do extra work with a modulus or conditional to get it back to a positive range, so it might be nice to have a version of atan2() that doesn't require that. I'm totally fine if the degree methods are removed from this commit, I just thought they helped round out the degree-based code in MathUtils. The typos seem to be mostly copy-paste-related: PixmapPacker called itself ImagePacker, an untypeable ligature got into the name of a distance field paper in DistanceFieldGenerator, and I had copied some docs from another library that still referred to libGDX as an external library (in MathUtils).

Good idea. I also just wrote a tan() approximation, and I should test that too. Not sure if I should add tan() to the PR.

Good idea. I also just wrote a tan() approximation, and I should test that too. Not sure if I should add tan() to the PR.
EDIT: The reason a separate tan() could be needed is because with the resolution our sine table has, just doing sin(x)/cos(x) has severely aliased output (limited to specific multiples).

Yeah, I think it's worth adding it if it's also faster than Math.tan and is more precise than sin/cos.

I spent a decent part of the day writing and refining a test just for tan(), and I haven't even added it to the PR yet. It's hard to measure the error for tan() because tan(MathUtils.PI * 0.5f) should either be infinite or very close to infinite by the mathematical definition (and depending on floating-point imprecision), so any results that should be approaching infinity but aren't approaching it quickly enough wind up off by millions (out of a correct result in the billions). I think normally when people are calling a method like tan(), they worry more about small absolute deviations from correctness where the relative deviation is large, than large absolute deviations where the relative deviation is small. That is, getting 2 when you expected 1 is extremely bad, but getting 1000002 when you expected 1000001 isn't bad at all.

I ran some benchmarks and the new tan() is slower than the old tan(), but not by much on recent Java, and both are quite a bit faster than Math.tan().
Measuring tan() on Java 18:
Benchmark                       Mode  Cnt   Score   Error  Units
MathBenchmark.measureDivideTan  avgt    5   5.217 ± 0.566  ns/op
MathBenchmark.measureMathTan    avgt    5  17.659 ± 1.176  ns/op
MathBenchmark.measureSoontsTan  avgt    5   7.824 ± 0.646  ns/op

And on Java 8:
Benchmark                       Mode  Cnt   Score   Error  Units
MathBenchmark.measureDivideTan  avgt    5   4.605 ± 0.219  ns/op
MathBenchmark.measureMathTan    avgt    5  51.494 ± 2.440  ns/op
MathBenchmark.measureSoontsTan  avgt    5  13.219 ± 0.858  ns/op

If the higher-speed DivideTan seems better, it might not after you see the graph of a statistical distribution using old tan() vs. new:
Old:

New:

Changing the implementation of tan() is the only change here; both also used double precision and the old one still encountered the strange "flat lines with hairs."

Sorry I'm late, had a literal hurricane disaster to deal with.
It looks great to me. Super thorough, as always!


[ENG-2974] Save failed message

Ticket: ENG-2974
Feature flag: n/a

Purpose

Make the save-failed message go away when appropriate

Summary of Changes

Update the lastSaveFailed to keep track of the correct task

Side Effects

NA

QA Notes

The save failed error can currently be triggered by choosing No license and then setting the Year field to be empty. Before this fix, changing the license would not clear the save failed message. After this fix, changing the license should get rid of the message.

Thanks for the QA notes!


Add support for ISTIO sidecar

What was changed

Add support for ISTIO deployment
Why?

Istio run with each pod due to which one time job/pod are not successfully terminal, istio is not required here as db schema setup and update are one time job, hence skipping istio for jobs.
Checklist

Closes
#https://github.com/temporalio/helm-charts/issues/353

How was this tested:

Testing on AWS EKS with istio.

Any docs updates needed?

no

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

Please make these optional via extraAnnotations value.
As an example, see https://github.com/temporalio/helm-charts/blob/master/templates/serviceaccount.yaml#L16.

@mindaugasrukas : I have checked this example, it does not seems correct solution.
I am facing this issue with DB migration job due to istio, rest service are working fine with istio, so I do not want remove istio for entire application, but as per this example this will skip istio for other temporal services.
There are 2 problem I have identified with DB migration:
1: DB migration is one time job, hence db migration container quit but istio container is log running so it showing error,
2: DB migration started before istio networking hence it stop DB migration container to connect with SQL port.

My point is that this change will add environment-specific fields to the generic template. We don't want to add every environment-specific setting to the template but have the ability to inject user-specific configs. So, I'd like to ask you to make the template change, allowing you or anyone else to inject their needs.

Any update on this?


Wildcards not working in postgres engine

Hello,
I have a postgres database that I connected to in jupyter, and when I query using LIKE '%1' like this:
SELECT topic_title
FROM engagement
WHERE url LIKE '%/1' AND is_pm = 'No';

I get: <sql_magic.exceptions.EmptyResult at 0x8dc35b0>
Now here's the thing. I know that this query will get be several results because I queried it successfully in pgadmin. But I would love to have it working it in jupyter. I tried restarting the kernel and running the cells again, and I tried it on a difference database, and LIKE % wildcards just don't work.
Any idea how to resolve this?

Okay, so I tried it again using the ipython-sql jupyter magic, and the query works successfully. I'm trying to figure out how to resolve this, but I unfortunately I don't have the skills to go deep and fix any possible bug.
Also, I tested this query on an sqlite db using a different engine connection, and wildcards work.
It's just this postgres engine that doesn't seem to run wildcards successfully.

Having trouble reproducing this. Can you try below and see if it's working? Which API are you using? SQLAlchemy? or psycopg2?
%load_ext sql_magic
import pandas.io.sql as psql
import psycopg2
connect_credentials = {'database': 'postgres',
                       'host': 'localhost',
                       'password': '',
                       'user': 'postgres'}

# connect to postgres connection object
conn = psycopg2.connect(**connect_credentials)
conn.autocommit = True
schema_name = 'template'
psql.execute('SET search_path TO {}'.format(schema_name), conn)
%config SQL.conn_name='conn'

%%read_sql -n
DROP TABLE IF EXISTS my_table;
CREATE TABLE my_table (
  column1 INT,
  column2 text
);

INSERT INTO my_table
VALUES  (1,'hello');

SELECT *
FROM my_table
WHERE column2 like '%ello';

returns:
column1	column2
1	        hello

Hi @crawles, I'm using sqlalchemy as recommended by the main sql_magic repo page here. This is my current set up:
#Libraries used
from sqlalchemy import create_engine
import matplotlib.pyplot as plt
from pylab import *

#Initialize sqlalchemy engine
postgres_engine = create_engine('postgresql://user:pass@host:port/database')

#Jupyter magics used
%load_ext sql_magic
%config SQL.conn_name = 'postgres_engine'
%matplotlib inline

Querying anything works fine, except for using WHERE LIKE 'W%', which returns an <sql_magic.exceptions.EmptyResult at 0x8dc35b0>. I will be trying your approach now and will let you know what happens.

@crawles , your approach works well, thank you, good sir. Do you think the main repo page should mention this as a default approach rather than sqlalchemy? Were you able to reproduce the problem with sqlalchemy?
Also, I just had to include port in the credentials dictionary, as the default is set to 5432.
connect_credentials = {'database': 'postgres',
                       'host': 'localhost',
                       'password': '',
                       'port': '',
                       'user': 'postgres'}

@WakeMeAtThree glad to hear it! Will try to reproduce it.

I hit the same issue, I think you will find your wildcard will work with SQLAlchemy if you enter %% rather than just %
so WHERE p.package_type like 'LOAD%%' rather than WHERE p.package_type like 'LOAD%'

Nice observation, @maurice1408! I just recreated this in my original db while using sqlalchemy, and 'Wal%%' does seem to work. But I prefer to use psycopg2 , I don't want to pick up a bad habit of writing double %% in queries. Thanks for mentioning this.


mips fixes

I'm working on getting lxd running on mips and this WIP PR fixes some low hanging fruit, there are some other issues like Rdev uint32/64 mismatches which may affect other platforms so they're not going to be included in this PR.

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

Hey, thanks for the patch. That's great. We only require that you sign-off your commits. Could you do that please? :)

Yup I always forget! There are a few more things I need to fix before this is ready however


(1) Creating submit article button

(Create) the rudimentary button to navigate to a submit page as the submit form.

Frontend folder structure organized and it follows the react fundametnal standards.


[proposal] koord-scheduler needs to support that Pod can only use resources of Reservation

What is your proposal:
The current implementation of koord-scheduler is that Pod allocates resources on nodes with Reservation as much as possible (assuming there are available Reservations), but sometimes it is possible to use other nodes that do not have reserved resources. But there are also some cases where the Pod should only use the resources of the Reservation.
Why is this needed:
The cluster administrator will reserve resources for some applications or users in advance, and the remaining resources will continue to be reserved for other users. The purpose of this is to manage resources more finely.
Is there a suggested solution, if so, please add it:

Maybe we should define a new API for Reservation to help the scheduler decide whether to filter nodes without Reservations.
In PreFilter, check whether the Pod has a Reservation, if not, abort scheduling.
In Filter, if the node has no Reservation, skip it.

cc @hormes @FillZpp @saintube @ZiMengSheng

@eahydra Is this an attribute of the reservation or of the pod?


unknown runtime specified

I know i need to follow the nvidia instructions to get this, but does this apply to windows docker installations as well? Or can i only get nvidia gpu support on linux ?
Their instructions look like they are geared towards non windows docker installations

https://www.docker.com/blog/wsl-2-gpu-support-for-docker-desktop-on-nvidia-gpus/
I think Windows should support GPU now as well.


docs: gRPC Guide Serves 404

I navigated to the gRPC guide and the page serves a 404 at the moment.

@jon-whit Thanks for creating this issue, I will fix it.
/assign


Is "Version compatibility" wrong?

We found that the 9.0 solution package based on the online version cannot be imported into the on-premise of the 9.0 version.

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

ID: 1ee79678-c9e1-0e4a-bd78-b9ff63d4d3ab
Version Independent ID: fbf9a7d9-0330-07ed-ea6b-152a570f8906
Content: Introduction to solutions (Developer Guide for Dynamics 365 Customer Engagement (on-premises))
Content Source: ce/customerengagement/on-premises/developer/introduction-solutions.md
Service: crm-online
GitHub Login: @JimDaly
Microsoft Alias: jdaly

@KumarVivek  Can you find another owner for this topic.
Recommend @shmcarth


I use Target=Database , async =true, Try to set connection string in code, get exception Unable to cast object of type 'NLog.Targets.Wrappers.AsyncTargetWrapper' to type 'NLog.Targets.DatabaseTarget'

Hi!
If you have a question that the logging isn't working, check the troubleshooting guide first before posting.
Almost every case is solveble with the internal log
Are you searching for a layout or layout renderer? You could find them here
Stackoverflow is recommend for questions. You could expect a faster response there (but do include all the info below) and the search is better.
For the stackoverfolow the following is recommend:

Use the tag "nlog"
Include versions of NLog/NLog.AspNetCore etc.
Include the platform (.NET 4.5, ASP.NET Core 3 etc)
Include the current config (xml or code)
Explain current result
Explain desired result
Please post full exception details (message, stacktrace, inner exceptions) in case of an error

You have nicely skipped all the requirements from the issue-template, and provided a random error-message.
Please include your NLog.config if you expect to get any help.

This is Nlog.config:

This is My Code:
static ErrorLog()
{
NLog.GlobalDiagnosticsContext.Set(LOGPATH, AppDomain.CurrentDomain.BaseDirectory);
var config = LogManager.Configuration;
var dbTarget = (DatabaseTarget)config.FindTargetByName("LogDB");
dbTarget.ConnectionString = ApplicationDbContext.GetConnectionString;
LogManager.ReconfigExistingLoggers();
}
This line throw the exception:
var dbTarget = (DatabaseTarget)config.FindTargetByName("LogDB");
I would appreciate your help

For some reason the config was deleted,
Nlog.config:
nlog autoReload="true"
globalThreshold="Debug"
xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
targets
target xsi:type="Database"
name="LogDB"
dbProvider="sqlserver"
commandText="INSERT INTO [Log_Entries] ([CallSite], [Date], [Exception], [Level], [Logger], [MachineName], [Message], [StackTrace], [Thread], [Username]) VALUES (@CallSite, @Date, @Exception, @Level, @Logger, @MachineName, @Message, @StackTrace, @Thread, @Username);"
<parameter name="@CallSite" layout="${callsite:filename=true}"
<parameter name="@Date" layout="${longdate}"
<parameter name="@Exception" layout="${exception}"
<parameter name="@Level" layout="${level}"
<parameter name="@Logger" layout="${logger}"
<parameter name="@MachineName" layout="${machinename}"
<parameter name="@Message" layout="${message}"
<parameter name="@StackTrace" layout="${stacktrace}"

<parameter name="@Username" layout="${windows-identity:domain=true}"
/target
/targets
rules
logger name="LogData"  minlevel="Debug"  writeTo="LogDB"
/rules
/nlog

When using <targets async="true"> then it actually becomes:
<targets>
    <target type="AsyncWrapper" name="LogDB">
                <target type="Database" name="LogDB_wrapped">
                </target>
    </target>
</targets>

You can do this as a workarounde:
           foreach (var dbTarget in config.AllTargets..OfType<DatabaseTarget>())
                dbTarget.ConnectionString = ApplicationDbContext.GetConnectionString;
           LogManager.ReconfigExistingLoggers();

Planning to improve NLog 5.0 a little so one can do this:
            var dbTarget = config.FindTargetByName<DatabaseTarget>("LogDB");
            dbTarget.ConnectionString = ApplicationDbContext.GetConnectionString;

Thank you very much, working properly


Day63

Resolves #136 .

Pull Request Test Coverage Report for Build 285

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 283:
0.0%

Covered Lines:
24

Relevant Lines:
24

💛  - Coveralls


Use a web service instead connect database in SSH commands

I proposed add a group URI in web command in "/internal/" sub URL. All other SSH sub commands should request the URL to do the operations but not read or write the database directly.
Especially when the database is sqlite and tidb, it's useful.

This has been resolved.


Autorest Regen Preview to 3.0.0-alpha.20240313.1 by Alexander Sher from refs/pull/4366/head

Triggered from https://github.com/Azure/autorest.csharp/pull/4366

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