update adapters

Before submitting a new pull request, please make sure:

[ ] Test cases have been added/updated/passed for the code you will submit.
[ ] Documentation has added or updated.
[ ] Commit message is following the Convention Commit guideline with maximum 72 characters.
[ ] Squash the repeat code commits, short patches are welcome.

adapter interfaces recommitted

收到！

2024年3月4日 17:28，zealotchen @.***> 写道：
@zealotchen0 commented on this pull request.
In framework/ohos/src/main/ets/hippy_framework/adapter/http/DefaultHttpAdapter.ets https://github.com/sohotz/Hippy/pull/21#discussion_r1510853986:

import { http } from @.';
-import { call } from @.';

const TAG = 'DefaultHttpAdapter'
这里 TAG 可以考虑加个前缀，方便搜索定位
—
Reply to this email directly, view it on GitHub https://github.com/sohotz/Hippy/pull/21#pullrequestreview-1913823832, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADIVTWKRV55VDTWL3Q3434LYWQ5FBAVCNFSM6AAAAABEER3MWKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSMJTHAZDGOBTGI.
You are receiving this because you modified the open/close state.


@conda-forge-admin, please add bot automerge

[ ] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/databroker-pack-feedstock#2.


Flags passed to a yarn script with -- are considered as file globs by jest

What version of Turborepo are you using?
1.3.1
What package manager are you using / does the bug impact?
Yarn v2/v3 (node_modules linker only)
What operating system are you using?
Mac
Describe the Bug
When we pass the --colors flag thru turbo, to a test:unit npm script that runs jest, that flag ends up being interpreted as a file glob (cf Pattern --colors - 0 matches below):
▶ yarn turbo run test:unit -- --colors
[...]
@choose/types-helpers:test:unit: No tests found, exiting with code 1
@choose/types-helpers:test:unit: Run with `--passWithNoTests` to exit with code 0
@choose/types-helpers:test:unit: In /Users/adrienjoly/dev/choose/backend/packages/types-helpers
@choose/types-helpers:test:unit:   10 files checked.
@choose/types-helpers:test:unit:   testMatch: /Users/adrienjoly/dev/choose/backend/packages/types-helpers/src/**/*.test.ts - 1 match
@choose/types-helpers:test:unit:   testPathIgnorePatterns: /node_modules/ - 10 matches
@choose/types-helpers:test:unit:   testRegex:  - 0 matches
@choose/types-helpers:test:unit: Pattern: --colors - 0 matches
@choose/types-helpers:test:unit: ERROR: command finished with error: command (packages/types-helpers) yarn run test:unit -- --colors exited (1)
command (packages/types-helpers) yarn run test:unit -- --colors exited (1)

Expected Behavior
The behavior should be the same as when we had not added test:unit in our turbo.json file, or as when we run the tests using yarn directly in the package's directory (instead of going thru turborepo):
▶ (cd packages/types-helpers/ && yarn run test:unit --colors)
 PASS  src/index.test.ts
  types-helpers
    isErrorLike()
      ✓ returns true for an empty Error object

Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        0.181 s, estimated 1 s
Ran all test suites.

To Reproduce
/package.json:
{
  "name": "@choose/backend",
  "version": "8.10.5",
  "private": true,
  "workspaces": [
    "configs/*",
    "packages/*",
    "servers/*"
  ],
  "scripts": {
    "test:unit": "turbo run test:unit --parallel",
  },
  "engines": {
    "npm": ">=7.0.0",
    "node": ">=16.0.0"
  },
  "packageManager"<EMAIL_ADDRESS>  "devDependencies": {
    "jest": "^28.1.1",
    "turbo": "^1.3.1"
  }
}

/turbo.json:
{
    "pipeline": {
        "test:unit": {},
    }
}

/packages/legacy-helpers/package.json:
{
  "name": "@choose/types-helpers",
  "version": "0.0.0",
  "packageManager"<EMAIL_ADDRESS>  "private": true,
  "main": "./dist/index.js",
  "types": "./dist/index.d.ts",
  "scripts": {
    "test:unit": "$(npm bin)/jest --config=jest.config.js"
  }
}

I have same problem. And it may be related with #1477

I believe that this has been fixed. Can you please retest to confirm?

This is still broken for us.

@m7kvqbe1 please file a new issue with detailed reproduction steps.

If you're getting failures for this we believe it is likely a separate issue in configuration for your application. We have tests ensuring that this behaves as expected.
I'm going to close this issue, if you are encountering it please file a new issue with detailed reproduction steps.


Filter by severity

Provide a way to filter by severity so it's easy to choose to show only severe errors.

@swrobel I'll add this today.

@dbalatero Did you get to it yet? :)

@buschtoens @swrobel released 0.2.1 with this feature.


[Workspace Feeds] Expensify Card - Expensify Card page reopens when clicking Back from workspace editor

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.11-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Launch New Expensify app
Go to Workspace settings > Expensify Card
Click Issue card
Click back button to return to Expensify Card main page
Click back button to return to workspace editor
Click back button again

Expected Result:
App will return to workspace list
Actual Result:
App returns to Expensify Card main page again
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/9c344a94-058d-429d-8474-0c3843fa8e70
View all open jobs on GitHub

We think that this bug might be related to #wave-collect - Release 2

This is not a blocker as this feature is behind beta

I can take this one

I cannot recreate this in version 9.0.11-2
https://github.com/user-attachments/assets/9a42846c-6877-41e7-addf-b38b285cf56b
this was fixed in https://github.com/Expensify/App/pull/45690/files#diff-dd029be30c66f2c0c9f9087ee89e752d8771153991c2f2cf9c4ceec89d64e9af

Thanks

Issue is still reproducible on the 9.0.11-2
https://github.com/user-attachments/assets/fe84960d-768c-45c0-8c73-96c133c9b951

cc @koko57

@lanitochka17 can you please retest on production version 9.0.11-5 or 9.012-0? The line I mentioned earlier fixes the issue for sure, it was deployed to staging in version  9.0.11-2, to production 9.0.11-5 or 9.0.12-0. So if you were retesting production version of 9.0.11-2 the change wasn't there.
I cannot reproduce it locally in v 9.0.13-0 Develop
https://github.com/user-attachments/assets/aa649f6c-dc01-46e4-9e2a-6c78cc65671d

Issue is not reproducible
https://github.com/user-attachments/assets/5b6a8740-8b8b-4c8a-9be9-9d46dcbabb11

nice, thanks


xue hua

/assign

/help

uhh

/wallet 0x0000000000000000000000000000000000000000

what version is this bot 💀

qa bot is working?

/assign

mr qa?

you there?

uhhh

/help

its working now?

/assign

umm wha?

lets see if everything works 💀

/autopay

/assign

/unassign

/unassign


Add support for bridging to individual steam chats.

Added this because I wanted to be able to bridge to individual people, let me know what you think!

Looks good, it's small enough change to be merged.
You've tested it ? :)
How do you find the userid? https://steamidfinder.com/ and steamID64 ?

I tested it with a config that created two gateways, one bridging matrix and a steam group chat, another bridging matrix and a steam individual chat, both worked for me.
I found the user id by pasting their profile URL into steamrep.com (but your website works too) this is only required if they have a custom profile URL. most people just have the standard URL which is their 64bit userId.

Thanks!


OHLC chart data values are not matching with the API pulled values

Hi first of all thank you very much for your work!
I have tried to experiment a little with the library but I cannot seem to have the same values that are shown by Degiro's website on the charts. i have tried it on several stocks but can't seem to get the same values.
Here is the code I used to get the OHLC data for AAPL:

request = Chart.Request()
request.culture = "fr-FR"
request.period = Chart.Interval.P1D
request.requestid = "1"
request.resolution = Chart.Interval.PT1H
request.series.append("ohlc:vwdkey:AAPL.BATS,E")
request.tz = "EST"
request.override["resolution"] = "P1D"
request.override["period"] = "P1W"

chart = quotecast_api.get_chart(
    request=request,
    raw=False)

ChartHelper.format_chart(chart=chart, copy=False)
chart_df = ChartHelper.serie_to_df(serie=chart.series[0])
chart_df["timestamp"] = pd.to_datetime(chart_df["timestamp"]+3600, unit="s")

Here are the results from the API and a screenshot of the data on the website:
timestamp        open        high         low       close
0 2022-02-01  173.891456  174.370854  172.103703  173.142397
1 2022-02-02  174.570603  175.329649  173.122423  174.810301
2 2022-02-03  174.285960  176.008795  173.641770  175.050000
3 2022-02-04  171.690000  172.900000  170.690000  172.520000

I would really appreciate it if anyone knows the reason for this discrepency.
Thank you very much!

Hello @hamoudeh90,
I don't know why but this seems to be not the case when you are using issueid.
request.series.append("ohlc:issueid:350015372")

For some reason when the vwdkey is used it doesn't display the same result than the website.
Hope that helps

Closing this issue.
Feel free to open a new one if you have other questions.


Bulb Profile Definition - Flange Width attribute

In the 2.8.6 version, the FlangeWidth attribute is mandatory:

For the time being, there are different interpretations : some CAD solutions export the OCX model with FlangeWidth equal to 0 or generate an OCX file without this FlangeWidth attribute (not conform with the schema).
Proposal: Update the format removing the FlangeWidth attribute for bulb profile.

Instead of removing the FlangeWidth attribute, can we make it optional?

Yes - I'm Ok to propose this attribute as optional.

Thanks, this is how it's currently implemented.
Ole Chr.
From: odegrand @.>
Sent: tirsdag 14. mars 2023 18:18
To: OCXStandard/OCX_Schema @.>
Cc: Astrup, Ole Christian @.>; State change @.>
Subject: Re: [OCXStandard/OCX_Schema] Bulb Profile Definition - Flange Width attribute (Issue #7)
Yes - I'm Ok to propose this attribute as optional.

Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOCXStandard%2FOCX_Schema%2Fissues%2F7%23issuecomment-1468511703&data=05|01|ole.christian.astrup%40dnv.com|06ca958a6c244dfb6d3308db24b00c21|adf10e2bb6e941d6be2fc12bb566019c|0|0|638144110763384831|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=ep%2FkfOUaf5oL7jRBk8IoCOpsQaSyfa88zMwHhvPAc1c%3D&reserved=0, or unsubscribehttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKSW7DA25I427R3FI3T5BCLW4CR4BANCNFSM6AAAAAAUDDKOEI&data=05|01|ole.christian.astrup%40dnv.com|06ca958a6c244dfb6d3308db24b00c21|adf10e2bb6e941d6be2fc12bb566019c|0|0|638144110763384831|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=1NZ5xgSSiqI8V5KaFG2PvRVlrjJpgbQscDcASxWvGzY%3D&reserved=0.
You are receiving this because you modified the open/close state.Message ID<EMAIL_ADDRESS>
This e-mail and any attachments thereto may contain confidential information and/or information protected by intellectual property rights for the exclusive attention of the intended addressees named above. If you have received this transmission in error, please immediately notify the sender by return e-mail and delete this message and its attachments. Unauthorized use, copying or further full or partial distribution of this e-mail or its contents is prohibited.

@ocastrup -> thanks.
When you say " this is how it's currently implemented". In which version of format (the official one and the next released one)?


Add tags for mods

Fixes #398, could fix #269

First look is a solid implementation, Going to cross-reference and make tweaks with our newly planned API spec to allow for better compatibility down the line. http://docs.solder.apiary.io/#reference/modpack/build/view-a-builds-details


Added tests that show that get_query_set patching is broken

See issue https://github.com/arteria/django-compat/issues/29

Thanks Luke for providing this! get_query_set is a serious one!!


[WIP] Improved HH example

making sure all text in the notebook is adapted to sbi
moving the external input, the simulator, and the summary statistics to a separate file

Is this still WIP @michaeldeistler ?

It will still need a rebase which I will take care of later today.


Fix perpetual state update in task form

https://app.zenhub.com/workspaces/epictrack-63891ea941d309001fa292cf/issues/gh/bcgov/epic.track/1724
Details:

remove unnecessary logic and refs
refactor how number of days and end date are updated

Could you please handle this

 Could you please handle this

is this happening with the changes in this PR or this is what is happening currently before this merge.
Because I handled this issue in this PR, is it still happening despite these changes?

 Could you please handle this

is this happening with the changes in this PR or this is what is happening currently before this merge.
Because I handled this issue in this PR, is it still happening despite these changes?

I had checked out this PR yesterday in local and saw this happening

I rechecked and it's not happening on my local. There's a chance you might of tested my code before I committed that fix. I will merge this


Question on masking in transformer encoder

Hello Mr. Hirofumi, thanks for opening source this excellent repository. Here I have some questions about the masking mechanism in transformer encoder:
In the code here
https://github.com/hirofumi0810/neural_sp/blob/3be0bac8a1b009ee36f10ca901f4c64160a5ce45/neural_sp/models/seq2seq/encoders/transformer.py#L394
there is a note says: # NOTE: no mask to avoid masking all frames in a chunk. If you would be so kind, could you explain a little bit the reason why avoiding masking all frames in a chunk? Isn't it ok if all frames in a chunk are masked? In which case, I think both the masks and encoder output could be correct after reshape from chunkwise shapes to their normal shapes.
Another question is how do you deal with overlapped chunks when streaming_type=='mask'?

All frames in the tail chunk in some utterances might be completely masked out. That's why I did set xx_mask to None. But I'm trying to implement the strict masking in the tail chunk now.
Regarding 'mask' option, the context size will be accumulated according to the depth.
'reshpae' option avoids this by trading memory consumption.

Thank you. Will it make any differences on parameter updating or final performance result if all frames of some padded tail chunks are masked out?

@t13m I checked it and found that the explicit tail masking did not change the performance.

Hi @hirofumi0810
I have found that transformer_enc_pe_type 'add' is used with a lc_type 'reshape' in the mma example 'lc_transformer_mma_hie_subsample8_ma4H_ca4H_w16_from4L_64_64_32.yaml'.
In this config, different chunks should have same position encoding. Don't you think it will degrade the performance?

Hi @hirofumi0810
I have found that transformer_enc_pe_type 'add' is used with a lc_type 'reshape' in the mma example 'lc_transformer_mma_hie_subsample8_ma4H_ca4H_w16_from4L_64_64_32.yaml'.
In this config, different chunks should have same position encoding. Don't you think it will degrade the performance?

@SoonSYJ I remember I tried different indices in each chunk, but it was not helpful.
So I simply reuse the same indices in each chunk.
Note that such positional encoding is still helpful on AISHELL-1.

@SoonSYJ I remember I tried different indices in each chunk, but it was not helpful.
So I simply reuse the same indices in each chunk.
Note that such positional encoding is still helpful on AISHELL-1.


Fix deadcode warnings

Signed-off-by: Dhammika Pathirana<EMAIL_ADDRESS>

It seems to be introduced by #1290. I think we can fix this warning by actually using conf. In persist function, we should skip calls to fsync() when conf.no_sync is set.

Thanks, I'll check it.
Not sure why this would break e2e tests though. I didn't see that crate used elsewhere.

Not sure why this would break e2e tests though.

It shouldn't, it looks more like flaky tests or some other issue.


chore(package): update @types/node to version 8.0.0

Closes #19

Coverage remained the same at 95.918% when pulling a261309fe6c22e36888499c53edd57e2be23472f on<EMAIL_ADDRESS>into c100ca283ba34cd7b48f531b11c406106a4f6961 on master.


Missing translation for Hindi

❌ Actual phrase(s)
at line 35 of scripts/installer-lang/hi.ps1
PodcatsSelect   = "Do you want to disable podcasts, episodes and audiobooks from the main page? [Y/N]" #
✔ Expected phrase(s)
PodcatsSelect   = "क्या आप मुख्य पृष्ठ से पॉडकास्ट, एपिसोड और ऑडियोबुक को निष्क्रिय करना चाहते हैं? [Y/N]"
ℹ Why is the current translation wrong
The translation was missing, I think the translation framework was updated and thus this field was not added back.

Thanks


remove unnecessary qualifications

While compiling with rustc version 1.78 i got these errors related to unnecessary qualifications, hence i removed these qualifications.

Oh, nice. Thanks


NPE in RecordSerde.java::write() when records have null values

Noticed this when doing (probably terrible) things to some streams in my kafka instance - I haven't dug in too deeply, could be wrong, but it seems like null is a placeholder value for deletion?  The ::write() function crashes but ::read() seems happy enough with null values.
Easy fix is to guard .value() with a null-check and write zero-length, though I don't have great testing up on restore yet to ensure this doesn't break things in a way I didn't expect.

Hi Greg,
This makes sense. Yes null messages are meant for deleting messages. I have
not thought about that. Thanks for pointing it out!
Your proposition for the fix would be my first idea. Feel free to open a PR
otherwise I will try to fix it soon.
Greg Studer<EMAIL_ADDRESS>schrieb am Mi. 5. Feb. 2020 um 10:29:

Noticed this when doing (probably terrible) things to some streams in my
kafka instance - I haven't dug in too deeply, could be wrong, but it seems
like null is a placeholder value for deletion? The ::write() function
crashes but ::read() seems happy enough with null values.
Easy fix is to guard .value() with a null-check and write zero-length,
though I don't have great testing up on restore yet to ensure this doesn't
break things in a way I didn't expect.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/itadventurer/kafka-backup/issues/14?email_source=notifications&email_token=AADWQNOWIS5FRCYD25IRRMDRBKBGDA5CNFSM4KQHUPOKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ILEUTQA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADWQNPYOT4JBXAMOEZ7DC3RBKBGDANCNFSM4KQHUPOA
.

--
Viele Grüße,
Anatoly Zelenin

Would be great if you can fix the issue. If you have an empty String entry in a partition and write it, you will end up with length == 0. This is still fine, until you restore it. Then the entry is interpreted as null and not an empty String ! When you now try to backup the entry again you get the Nullpointer exception. Either you write a length of -1 to indicate a null entry, which I would prefer, or you put a null check in the write function. Would be good to have a distinction between null and empty String, because null might cause errors in the application receiving the data !

@LBmaster001 Yup good point - didn't catch that.  A patch here probably needs testing around that, since it's subtle...

See #16 (Just to get the GitHub Link)
Do you have a good idea how to do the system tests (what to use for?) – You can find some tests in system_tests – but they need to be executed and checked manually :( – In the long run this is obviously not a good idea

@itadventurer yeah, it's a little tricky.  Off the top of my head TravisCI + Docker + https://github.com/wurstmeister/kafka-docker could do what we want here?  I've set up something similar in the past.  To get confidence to use this in a real setup we may need to do something equivalent anyway...

Yeah, I also thought about something like this ;) How did you do test automation afterwards in the past? Using Bash scripts?

Nope - python - the closest equivalent in Java world probably is groovy?  Though the bash isn't really an issue for me - probably better in a "real" language but all my deployments are linux-y, so...
Maybe getting TravisCI or something running with a docker-compose'd kafka would work basically with what's there right now, aside from some "docker exec" prefixes?  Then you can refactor the bash utils into whatever format you like (JUnit? or whatever the cool kids are using nowadays - I've been out of the JVM for a few years).
(Using "you" in the generic sense here :-) happy to help if I manage the time - got an open testing ticket for this on my side, so I'll see...)

Moved discussion about testing to #20

Fixed by #21


Scrolling throws off Color Picker

Scrolling a page throws off the color picker widget completely.
http://codepen.io/matthewlein/pen/emJDF/
Scroll over to the right then try to pick a color.

The issue still exists and is even reproducible on http://workshop.chromeexperiments.com/examples/gui/#4--Color-Controllers .
In the color controller there 2 functions, setSV and setH and both of them have something like:
var v = 1 - (e.clientY - o.top + document.body.scrollTop) / w;

If you remove the + document.body.scrollTop and scrollLeft the bug is fixed on chrome and firefox.
But I'm not sure it will work on all browsers and it will also possibly make the color doesn't work if the gui has a custom position.
I can happily make a pull request if this seems to be the right solution.

Hi, your modification is not related to Selector position.
I wonder why it works on your side??

It is still very bad on chrome with color picker when user scroll

fixed in 6f0fcb64f5b3d3eee2c49204819d382a4e684f3a

Hey I'm sorry but this issue still persists - using Chrome Version 52.0.2743.116 (64-bit) and I tried with the current dat.gui version (master 0.6.1) and also the develop branch - after scrolling, the color controls are not accessible - this issue should be re-opened

Hmm.. there were two scrolling issues.

One is when the GUI is longer than the page, the GUI itself has a scrollbar. This is still broken, and is described in #105
The scrolling issue I fixed (or thought I did), was when the HTML page is super long or wide. In this case, when scrolling the page down or right, the color picker and slider were breaking.

What issue are you seeing?

Hi,
This one I think:
The scrolling issue I fixed (or thought I did), was when the HTML page is super long or wide. In this case, when scrolling the page down or right, the color picker and slider were breaking
Only on Chrome. Firefox  works well with a ugly scrollbar :D

Hey the problem i'm having is that the GUI is longer than the page - the GUI has a scrollbar, and I scroll down to the color picker but i cannot select a color, since it appears at an offset much further away from the field - and so disappears the moment I try to select a color... I guess similar to #105

I have the same problem with @cybafelo ...
Who can solve this? i even can't figure out how the color selector visible or invisible.
the selector need calculate the gui's scroll offset.

ok so now i'm really needing this to work and its not - I tried several things - like setting the dg.selector position attribute to 'relative' - in which case the entire color control is not visible but appears to be at the correct place - and various other things - none of which seem to work -
very frustrating indeed


Get Test Case assigned to user info

Hi, thank you for the API, its very useful.
But I need to get assign information, the information we like is similar to the table when user click "Test cases assigned to me" label on Testlink home.
We tried to get all test cases and then get the lastest execute status, but there are no assigned date. Can you show me a way to get there such information?
The image to illustrate the table I want to get:

Thanks for reporting it and for the kind words.
It depends first on the PHP API availability. We are a simple facade for the PHP API. And then it depends on me finding spare time to update the code and release it, or someone else submitting some code.
Pull requests welcome :)

Thanks for replying.
I understand now, thank you.


Chore/setup docs

:tada: This PR is included in version 2.0.1-beta.85 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Allow users to specify instance root volume encryption key

Added a --kms-key-id <string> CLI flag that allows the user to specify which KMS key should be used when encrypting the root volume of ONV-created instances. This option has also been added as a parameter (kmsKeyID) to the ValidateEgress() API function. Not specifying this key simply defaults back to the cloud account default KMS key.
Implementation-wise, adding this option as yet-another parameter to createEC2Instance() would have made that function's signature unreadably long, so I instead refactored the function such that it now takes in a param struct of type createEC2InstanceInput.
This addresses OSD-11084, where verifier users requested the ability to specify which KMS key is used for encryption.

/lgtm

/method squash

/label tide/merge-method-squash


Incomplete tree-node validation

It seems to be possible to add new tree-nodes with a empty-string label. This should be avoided.

/remove-lifecycle stale
/lifecycle frozen


hack: fix taint value

getting error
Error: invalid argument "node-role.kubernetes.io/master:NoSchedule" for --register-with-taints=node-role.kubernetes.io/master:NoSchedule: invalid taint spec: node-role.kubernetes.io/master:NoSchedule

I found that the taint was missing the taint value. It needed a = so that the value is set to empty string "".
Fixes @Quentin-M's issues in https://github.com/kubernetes-incubator/bootkube/pull/434.

LGTM! Thanks!

Is any reason why this isn't also in the hack/quickstart service files? If not lets put it there too for consistency with PR testing. (tests source their service files from quickstart)

Good call, @pbx0. I just pushed that change.


ImageMagick/GraphicsMagick is not installed (MiniMagick::Error)

I got this error though I already installed ImageMagick on Ubuntu / DigitalOcean.

This is fixed in now in 4.3.0 (https://github.com/minimagick/minimagick/commit/f444d2887563f8f130d5358f90a51b2da3cfa07f).


Fabric8 Discovery client clean-up 2

@ryanjbaxter I'll have more small PRs like these to clean-up things. I want to do a few smaller and easier to review ones. thank you for looking into it


License problem due to using "colorize" gem that is licensed under GPL

Thanks for building this amazing gem!
I was planning to use it in one of the apps, however, the license finder reported a severe issue: the gem is dependent on colorize which is licensed under GPL-2 (https://github.com/fazibear/colorize/blob/master/LICENSE) which creates a massive issue since it's a copyleft license. Also that would imply that langchainrb itself would need to use GPL-compatible license and MIT would not be an option.
Would you consider merging a PR that replaces colorize with something that has a friendlier license, e.g. rainbow gem? I would be happy to submit it.

@Azdaroth Thank you for flagging, I was definitely unaware of this. I'd be happy accept a PR switching to rainbow if you'd like to put it together. Thank you!


Fix organizer::vertex::prepare not creating component pools

Fixes #1013

Oh, I'm fine with this PR but the changes to the test are waaaaay easier 🙂 this should be enough actually:
TEST(Organizer, Prepare) {
    entt::organizer organizer;
    entt::registry registry;
    clazz instance;

    organizer.emplace<&ro_int_double>();
    organizer.emplace<&clazz::rw_int_char>(instance);

    const auto graph = organizer.graph();

    ASSERT_EQ(std::as_const(registry).storage<int>(), nullptr);
    ASSERT_EQ(std::as_const(registry).storage<double>(), nullptr);
    ASSERT_EQ(std::as_const(registry).storage<char>(), nullptr);

    ASSERT_FALSE(registry.ctx().contains<int>());
    ASSERT_FALSE(registry.ctx().contains<char>());

    for(auto &&vertex: graph) {
        vertex.prepare(registry);
    }

    ASSERT_NE(std::as_const(registry).storage<int>(), nullptr);
    ASSERT_NE(std::as_const(registry).storage<char>(), nullptr);

    ASSERT_FALSE(registry.ctx().contains<int>());
    ASSERT_FALSE(registry.ctx().contains<char>());
    ASSERT_TRUE(registry.ctx().contains<double>());
}

Checking that the storage exists after invoking prepare only takes a couple of lines. 👍

That would be a lot cleaner, but storage() doesn't return a pointer, it just returns a static placeholder pool when the requested pool doesn't exist, and since both the placeholder and the actual pool should be empty, there's no easy way to check that result. Iterating over the existing pools was the only way I could think of that works around that, but maybe the registry API could be extended with a separate function that checks whether a pool has been created or not?

it just returns a static placeholder pool when the requested pool doesn't exist

Oh, that's because the PR points to master. Wait. I'm switching it to wip which is the upcoming master.

Ok! The test is updated now.


Cannot use lime with grayscale images

Hello Marco!
I have an issue with grayscale images.
If I understand correctly, when using grayscale what I gotta do is:
1 - Convert image (1 x img_width x img_height x 1) to (img_width x img_height)
image = x_img[0, :, :, 0]
2 - Define a function to convert image to (img_width x img_heigth) back to (1 x img_width x img_heigth x 1)
def new_predict_fn(img)
      img = img[np.newaxis, ..., np.newaxis]
      return model.predict(img)
3 - Run lime
explainer = lime_image.LimeImageExplainer()
explanation = explainer.explain_instance(image, new_predict_fn(image), top_labels=5, hide_color=0, num_samples=250)
However, when I do this it starts to run but it stops at 9/250 and I got:
TypeError                                 Traceback (most recent call last)
~\AppData\Local\Temp/ipykernel_6332/722228459.py in
9 print('image shape: ', image.shape)
10
---> 11 explanation = explainer.explain_instance(image,
12                                          new_predict_fn(image), top_labels=5,
13                                          hide_color=0, num_samples=250)
~\anaconda3\lib\site-packages\lime\lime_image.py in explain_instance(self, image, classifier_fn, labels, hide_color, top_labels, num_features, num_samples, batch_size, segmentation_fn, distance_metric, model_regressor, random_seed)
196         top = labels
197
--> 198         data, labels = self.data_labels(image, fudged_image, segments,
199                                         classifier_fn, num_samples,
200                                         batch_size=batch_size)
~\anaconda3\lib\site-packages\lime\lime_image.py in data_labels(self, image, fudged_image, segments, classifier_fn, num_samples, batch_size)
259             imgs.append(temp)
260             if len(imgs) == batch_size:
--> 261                 preds = classifier_fn(np.array(imgs))
262                 labels.extend(preds)
263                 imgs = []
TypeError: 'numpy.ndarray' object is not callable

Hi, I had a similar issue and it was because my model.predict(img) returned an array within an array, so I just did model.predict(img)[0] instead. Don't know if this is your issue though, but it might help to check the output shape of the predict call.

LIME does not explain the grayscale images. You have to convert your grayscale images into RGB images.

Hi, I had the same problem some hours ago. Do not know if you still need this but this is my workaround
`
def predict4lime(img2):
return model.predict(img2[:, :, :, 0])
exp = explainer.explain_instance(img[0, :, :, 0], predict4lime, top_labels=1, hide_color=0, num_samples=1000)
`
In my script img is a (1, 256, 256, 1). in explain_instance i'm giving it as a (256, 256).
The predict4lime is the normal prediction but only on the first channel. Inside explain_instance they call gray2rgb of skimage. This function simply take the gray image and copy it on three channel so you know that each channel of the new rgb image is simply your original gray image.
You might be wondering why am i giving a  (256, 256) and then accessing four indexes instead of three in predict4lime. Well the fourth dimension is created by explain_instance after the gray2rgb call. The first dimensio is also generated by explain_instance. You can chesk this by uncommenting the print inside predict4lime.


Provide CODEOWNERS

Provide CODEOWNERS configuration so that PRs can get automatically assigned, such as https://github.com/Azure/API-Portal/pull/53

@azaslonov Any thoughts on this?


Added Spell Check Workflow

Added a spellcheck workflow that runs when .md files are changed. Tested and fixed invalid spellings using it.

Thanks for the changes!


ZEPPELIN-2221 Show Jobs page when jobId is missing

What is this PR for?
Because of yarn's bug, Spark's jobId is not passed. This causes some Spark UI link looks broken. In this kind of case, showing Jobs page looks reasonable. Yarn's bug is already fixed with the latest version only, so we need to handle it in Zeppelin side.
What type of PR is it?
[Bug Fix]
Todos

[x] - Return Jobs page when Job page is not available

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-2221

How should this be tested?

Install outdated yarn
Run script
Click Spark UI to show Jobs page instead of 404

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

Changed it and added related tests

Merging it if there's no further discussion


Read tenant from file

Make it available to read tenant url from file or environment variables in delete option and read token from file in upload function

Out of scope


install error for pytorch3D

I try to install this,but found this error,Could anyone help me?

I have encountered the same problem. Maybe you should try a different cuda version to avoid this issue.

I have encountered the same problem. Maybe you should try a different cuda version to avoid this issue.

I’ve changed the cuda version to 11.7,while still has the same problem.Could you please tell me your cuda version? And all the module I installed is the same as the requirements.txt.

I have encountered the same problem too. But I successed on 3090 gpu with the same torch and cuda version. Now I moved my code on A800 GPU servers, and the docker image can not work. So I have to rebuild the environments, and the error occured when building on pytorch3d. And my cuda version is 11.7 on A800 servers and 3090 servers.  But the error just appeared on A800 servers.


Error while using Null values in Tableau

Hi! We found some error, while using Null values in Tableau.
We get this error in release https://github.com/ClickHouse/clickhouse-odbc/releases/tag/v<IP_ADDRESS>01226, but
in release https://github.com/ClickHouse/clickhouse-odbc/releases/tag/v<IP_ADDRESS>91108 everything works just fine.
An error occurred while communicating with the Other Databases (ODBC) data source 'oroboros_test_123 (cubes)'
Bad Connection: Tableau could not connect to the data source.
Error Code: B19090E0
unknown type
SELECT SUM({fn CONVERT(NULL, SQL_BIGINT)}) AS sum_Calculation_3063644052176396288_ok
FROM oroboros_test_123
HAVING (COUNT(1) > 0)
This error occurs while creating Calculation Filed like IF FALSE THEN 1 END
How can we fix this?
UPD:
I changed some code here https://github.com/ClickHouse/clickhouse-odbc/blob/3c37dbd1e4e75cfba8dfa5300c52fcfd0d94a008/driver/utils/type_info.h#L51, by replacing this line to throw std::runtime_error("unknown type. Got type " + type);
And got next message
An error occurred while communicating with the Other Databases (ODBC) data source 'cubes_clickhouse__fact_ue_partitioned_distributed (cubes)'
Bad Connection: Tableau could not connect to the data source.
Error Code: B19090E0
unknown type. Got type Nothing
SELECT SUM({fn CONVERT(NULL, SQL_BIGINT)}) AS sum_Calculation_3485082461064859648_ok
FROM cubes_clickhouse__fact_ue_partitioned_distributed
HAVING (COUNT(1) > 0)
As far as I can see, there is no type Nothing here https://github.com/ClickHouse/clickhouse-odbc/blob/3c37dbd1e4e75cfba8dfa5300c52fcfd0d94a008/driver/utils/type_info.cpp#L7,
but there is type Nothing in Clickhouse doc https://clickhouse.tech/docs/ru/sql-reference/data-types/special-data-types/nothing/

Can anyone help with this?

Hi Sergei - you need to add mapping to TypeInfo with mapping to some  SQL Type and number of bytes required to store it


Importing data into matrix field doesn't create new blocks

I'm having an issue when I try to import some fields from my XML feed into a matrix field made up of 3 date fields (1 date and 2 time). Here is a simplified version:
...
<item>

  <eventDate>
    <date>22/12/2015</date>
    <startTime>12:00</startTime>
    <endTime>15:00</endTime>
  </eventDate>

  <eventDate>
    <date>22/12/2015</date>
    <startTime>12:00</startTime>
    <endTime>15:00</endTime>
  </eventDate>

</item>

<item>

  <eventDate>
    <date>22/12/2015</date>
    <startTime>12:00</startTime>
    <endTime>15:00</endTime>
  </eventDate>

</item>
....

e.g.
<date> -> matrixField.date
<startTime> -> matrixField.startTime
<endTime> -> matrixField.endTime
The problem is that it doesn't create a new block when there are multiple eventDate's ... It works if I only have one field in the matrix, but the second there are multiple it just replaces the contents of the first block with the next. I have also tried this with a table field and have had the same results.
Note: I tried to send you a support request through the dashboard but I'm on localhost so I don't think it worked.
Any suggestions? Cheers.

Okay, a bit more testing and I have realised that if the first item in my XML only has 1 eventDate node then all of the other items can only ever have 1 eventDate block/row. So, to test I added two eventDate nodes to my first item and it then allowed the plugin to import multiple rows (all of them, not just 2)!
But I then realised that it now refused to import items that only have 1 eventDate... Is this a problem with the way my XML is structured or the way the plugin handles blocks/rows?

Hey @Lighty46 - Sorry for the delay, but just needed some time to test this issue properly. Its indeed an issue with the plugin mapping, and is to do with irregular nested nodes, as per your example. This has now been fixed in the 1.4.4 release.
I've also ensured this works using the following data (using item as the primary node), but please let me know if you still have issues!
<?xml version="1.0" encoding="UTF-8"?>
<rss>
  <channel>
    <item>
      <title>Title1</title>

      <eventDate>
        <date>22/12/2015</date>
        <startTime>12:00</startTime>
        <endTime>15:00</endTime>
      </eventDate>

      <eventDate>
        <date>23/12/2015</date>
        <startTime>13:00</startTime>
        <endTime>16:00</endTime>
      </eventDate>
    </item>
    <item>
      <title>Title2</title>

      <eventDate>
        <date>24/12/2015</date>
        <startTime>14:00</startTime>
        <endTime>17:00</endTime>
      </eventDate>
    </item>
  </channel>
</rss>

Hi Josh,
Thanks for taking a look at this. Don't worry about the delay, it was
Christmas!
Just tested and I'm still having the issue - I used your example and it
worked, but then I swapped the two items around and it started limiting all
blocks to 1 row. In the end I decided to add a permanent "Dummy Item" to
the top of my XML feed that had multiple eventDates and that's fixed it.
It looks like the update you put out did fix the issue I was having where
items with 1 date weren't being imported if the first item had more than
one though!
Have a good New Year and thanks again!
Dan.
On Mon, Dec 28, 2015 at 2:18 AM, Josh Crawford<EMAIL_ADDRESS>wrote:

Hey @Lighty46 https://github.com/Lighty46 - Sorry for the delay, but
just needed some time to test this issue properly. Its indeed an issue with
the plugin mapping, and is to do with irregular nested nodes, as per your
example. This has now been fixed in the 1.4.4
https://github.com/engram-design/FeedMe/releases/tag/1.4.4 release.
I've also ensured this works using the following data (using item as the
primary node), but please let me know if you still have issues!

Title1
  <eventDate>
    <date>22/12/2015</date>
    <startTime>12:00</startTime>
    <endTime>15:00</endTime>
  </eventDate>

  <eventDate>
    <date>23/12/2015</date>
    <startTime>13:00</startTime>
    <endTime>16:00</endTime>
  </eventDate>
</item>
<item>
  <title>Title2</title>

  <eventDate>
    <date>24/12/2015</date>
    <startTime>14:00</startTime>
    <endTime>17:00</endTime>
  </eventDate>
</item>

—
Reply to this email directly or view it on GitHub
https://github.com/engram-design/FeedMe/issues/24#issuecomment-167461043
.

For the record, this is the recommended way to fix this sort of issue. Its a little bit of a pain, but a requirement to ensure Feed Me gets consistent, structured data so it knows what to do with it.

I am having a similar issue here with JSON, though for myself it only updates the first, then gives up:
example JSON:
[
  {
    "title": "entry title",
    "matrixData": [
      {
        "title": "mtitle1",
        "description": "mdesc1",
        "link": "mlink1"
      },
      {
        "title": "mtitle2",
        "description": "mdesc2",
        "link": "mlink2"
      },
      {
        "title": "mtitle3",
        "description": "mdesc3",
        "link": "mlink3"
      },
      {
        "title": "mtitle4",
        "description": "mdesc4",
        "link": "mlink4"
      },
      {
        "title": "mtitle5",
        "description": "mdesc5",
        "link": "mlink5"
      }
    ]
  }
]

@lexbi If you can send through a help request, I can double-check your settings to make sure its all in order.

Nice one cheers, will do!


How to get file absolute path ?

Hi, thank you for your library, it help me a lot !!
But one more thing, that my project would like to load mp3 file with MediaMetaDataRetriever that need Uri to file.
Are there any method that I can get the path of particular file ?

@magnusja
How could we go with the storageprovider or httpmodule for that ? Since i have applied required part from readme file.
My concern is how usbfile directly be open using Intent.ActionView ？ How could we get Mount path of connected Usb Pendrive and how can we open any of the image , video , audio directly from their instead if copying it to the internal storage first. I don't want to copy file to internal storage when USB OTG pendrive gets connected what are the way we can open image , video , audio directly.
Please help me for that Sir.

There is no way of getting the mount path because it is not mounted under Android.
Please use the http module to stream videos, audio, and images to another app.

@magnusja
So any workaround to open UsbFile directly through Intent.ActionView ? I tried with it but it every time showing media not found on click of UsbFile adapter item. I tried to view an image.
So i need your help in that regards. I also added httpmodule but i doesn't get idea how can i start with View image , video , audio etc..
So any help will be greatly appreciated in this.
If you need to look at  code snippet that i am trying i will provide that also sir.

Please have a look into the example application, it is all implemented there.

@magnusja
I tried with  whatever implemented in example application. But when inside example application itself going to open an image it is asking to open in any browser and then something like localhost:8000/Image.png loading and it's get stuck showing nothing in example application itself.
What need to do in that case ?

Yeah for images the standard external application is usually the browser.
In the example application, I do not show anything I am only opening third-party apps. If you want to show that in your application you need to pass the URL from the http module to an ImageView or video player widget.


update the latest pinecone version to the latest version with support for Serverless indexes (the current only option for free pinecone accounts)

In the new version of pinecone, the capacity mode of an index can be serverless or pod-base (before it was only pod-base). The free account of pinecone currently only allows the serverless mode.
In the serverless mode, there is no an environtment parameter; the PINECONE_ANVIRONMENT parameter does not make sense anymore.
The pinecone-python lib needs to be updated to the latest version.
Ref: I asked Pinecone about it:
https://community.pinecone.io/t/how-to-find-out-what-is-my-environment-name-in-a-free-pinecone-account/6253
where they mention that:

It looks like that code is using the older connection method which is not supported with Serverless indexes.

gpt4-pdf-chatbot-langchain needs to be updated to use the latest version of the NodeJS SDK.

im getting the same issue. Does this basically mean can't be used until it is updated? :(

This app was extremely useful to understand complex topics on different kinds of pdf documents. Perhaps we can change the connection method to reflect the new modality, if possible to keep it free.

I was able to fixe the issue by using the individual Langchain packages
import { PineconeStore } from "@langchain/pinecone";
import { Document } from "@langchain/core/documents";
import { OpenAIEmbeddings } from "@langchain/openai";
import { Pinecone } from "@pinecone-database/pinecone";

Check out a sample implementation https://github.com/arpeiks/gpt-langchain

I was able to fixe the issue by using the individual Langchain packages
import { PineconeStore } from "@langchain/pinecone";
import { Document } from "@langchain/core/documents";
import { OpenAIEmbeddings } from "@langchain/openai";
import { Pinecone } from "@pinecone-database/pinecone";

Check out a sample implementation https://github.com/arpeiks/gpt-langchain

Thanks @arpeiks, I went into your fork for this project. I thinking, shall I clone this files and replace them into this project?

im getting the same issue


Commits & Pull request flow

@Idrinth  Could you add your comments on flow of commits/closing tickets/pull requests in general.

commit with "Fixes #[ticket]" so the commits are automatically listed in the ticket
test the changes yourself and fix any bugs you might encounter in your branch
create a pull-request and (if possible) let someone have a look
when accepting the pull-request all related tickets are closed

maybe we should add a developer-readme with that kind of information?

Good idea, could you add your notes to developer-readme and close this task.


Folder restructure

I have restructured the folder by taking the test folder out of hexgame and putting in root, as by convention. Also removed useles src subfolder.
Added also pyproject.toml with build instruction and used this to build and publish on  pypi test. This allows to install hexgame using pip , see: here
Finally updated readme (still bare and bad though) and added a main file so that we can call the main game by just doing python -m hexgame

I think if you put the folder with the rust implementation i imagine it will be ignored automatically when packaging since it will look for python models and packages that have the init.
It does feel a bit dirty to be mixing up python and Rust but I guess I am okay with it.


"vania serve" error on new project

It's related to pointycastle package
What is your Dart SDK version

Dart SDK version: 3.3.4 (stable)

It's related to pointycastle package
please do this

go to this path \AppData\Local\Pub\Cache\hosted\pub.dev find pointycastle-3.9.0 and delete it

put this in your pubspec

dependency_overrides:
  pointycastle: 3.7.3

run this command

dart pub get

and wait to update pointycastle to the new version

It's worked! Congratulations for your project
🤍


mtenv setup fails when a symlink to another install already exists

I tried to set up mtenv on an Ubuntu machine, and ran into a problem:
I cloned the repo, compiled it with shards build, and moved the binary onto my $PATH, which all worked fine.
When I ran mtenv setup though, I got this error:

It looks like you already have a myst symlink in /usr/local/bin that's pointing to some other Myst installation that isn't managed by mtenv (and also apparently doesn't exist, which is why it's red).
If you remove that symlink (rm /usr/local/bin/myst), then run mtenv setup again, it should succeed.
I actually specifically remember when I was writing the setup command that I thought "no one will ever hit this corner-case", so I didn't handle it........woops. It checks to see if the File is a regular File or doesn't exist at all, but doesn't handle where the File exists as a symlink.
To make sure people don't hit this in the future, the setup command should check to make sure that if a myst filex already exists as a symlink, it points to the location that mtenv expects (e.g. ~/.mtenv/shims/myst). If it points somewhere else, it should complain and tell the user to remove the existing symlink. I don't want to forceably remove the file without the user knowing what happened.


check_o_url is ignored by update_parameter

update_parameter() does not use the check_o_url argument and thus complain about missing orchestrator even with this flag set to false.

Fixed by 48fc21


Get rid of Babel/Polyfill

We should keep the dependencies small and use only a promise and async/await polyfill.

Closed via 21a295daf401ab26477b7eac1eb14781b84d9317


Live-reloading gettext translations from other apps in an umbrella

Is it possible to live-reload translation changes from other apps within an umbrella?
I'm exploring two different setups:

Each app has it's own Gettext module. Example repo: https://github.com/wojtekmach/gettext_umbrella/tree/master/apps
There's just one Translator app that holds all translation *.po, and other apps depend on it. At work, we're leaning towards this setup, but any recommendations are very welcome. Example repo: https://github.com/wojtekmach/gettext_umbrella/tree/translator/apps

Live-reloading doesn't work in either setup.
Might be related to #40

Fixed when #40 was fixed. :)

Thank you! ❤


Supervised-pretrained ViT-L results

Hi, do you have any plans to release the code for reimplementation of the supervised-pretrained ViT-L results presented in your paper (Fig. 1 (c) and Fig. 7)?

We can add this within the next few days.

Thanks for your great work!


BSOD

Get bsod

Parameters are null and can be set in kdmapper with the values that you want, this isn't a normal driver loading, is a manual mapping then there is no driver object


Syntax changes

What type of PR is this? (check all applicable)

[ ] ♻ Refactor
[ ] ✨ Feature
[x] 🐛 Bug Fix
[ ] 👷 Optimization
[ ] 📝 Documentation Update
[ ] 🚩 Other

Description
Fixes issue #3
Related Tickets & Documents
Issue #3
Mobile & Desktop Screenshots/Recordings (if there are UI changes)
Added to documentation?

[ ] 📜 readme
[ ] 🙅 no documentation needed

[optional] What gif best describes this PR or how it makes you feel?

Thanks a lot for that! Changes look great! 🙌


Add option to do per-operation compensation

This is a simplified and less accurate version of per-moment compensation. It might be useful where the per-moment compensation is too expensive to run.

I'm a little confused at how this compares and contrasts with the other method(s). Can you expand a bit? I see the input is still a circuit, so where does the differing behavior come into effect?

This new method doesn't care if two operations in a compensated moment were characterized together or in a different moments. This is not the case for the other method.
I'll add more docstrings on this tomorrow.

I think I get it now: You could have done calibration any number of ways (either fully-grid-parallel, according to the circuit's moment structure, or completely randomly if you're feelin' wild) and this new function will take the results no matter what and match them up by pair whereas the previous method needs the "mapping" between circuit moment and calibration result. Is this right?

That is correct, I described this in a docstring now.


Issue with Gmail layout is completely wrong since a few days

How can we reproduce this bug?
Send the basic template to a Gmail account and use the webmail. Basic template from official docs.
What did you expect to happen?
Email is displayed correctly.
What happened instead?
Email layout is completely wrong.

Hello Gremo!
Can you, please, give more info to try to reproduce your bug? Or can you post the flu html of your basic.html from the dist folder to see if it is any different? I have just compiled a test project from 0 using foundation cli and there is nothing wrong with the template - it displays perfectly fine both in gmail and inbox.

Hi @nikitasol, thanks for the reply.
I was having display issues with Gmail and emails sent (using a PHP script) with templates by Foundation for Emails.
So, to really test if it was my fault, I downloaded the starter kit. I sent a test email from Thunderbind (copy and paste from the basic.html template) to my Gmail account. Same identical issue.
No display issue so far with Thunderbird, Outlook and web-based email clients.

Hello Gremo,
The problem you are having is due to not inlining the css. Follow this link: https://foundation.zurb.com/emails/inliner-v2.html
There you can insert you html and css separately and it will inline it. You can try a different one, both litmus and mailchimp have one.
Without inlining you will always have different clients stripping css from head and/or body.

You are right, I know sorry... totally forgot! But I'm pretty sure that I'm inlining my css using PHP. I'll check tomorrow and report back, thank you.

I have tried inlining with litmus and it works as intended so there must be some issue with your php script. Try comparing between the two versions.

Was a fault in library that I used to send mails. Thanks @nikitasol, closing this.


Nec port

Okay I’ve just about finished writing tests now for the nec display driver which passes on both the ruby driver and the crystal driver. I’ll open a PR so you can review this @KimBurgess  but this driver has been quite a headache to write just because of the way the protocol has been implemented and how difficult it is to write tests for the spec.
Do let me know if anything can be cleaned up or done better though.
https://github.com/acaprojects/ruby-engine-drivers/commit/ee1ff475aed1852cdabcac050d26dc434a6b2383

Hey @KimBurgess , I've been working through your feedback but I feel that using enums to store values makes the code a lot less readable mainly because the commands can be built using ascii characters which is easier to read like in the original ruby implementation. Let me know if you still think differently but I've made these 2 commits which aren't as readable imo.
https://github.com/PlaceOS/drivers/pull/50/commits/4301a0109be5e31628a0b0ef03246eddb6def428
https://github.com/PlaceOS/drivers/pull/50/commits/e0363f99a60fa1fcf7a8fcfd01f2932cf2c03d86

Actually just realised I converted those ascii incorrectly to the equivalent hex numbers so it should look more like this.
https://github.com/PlaceOS/drivers/pull/50/commits/a2819a0c6ea9ba077d00038525d3081fdcbd3146#

What a horrendously inefficient protocol! It has all the overhead of something easily typed by a human with the string encoding, but none of the readability.
One thing that will really help out is explicitly specifying a base when you're converting between string and numeric representations. By default, when you use #to_s and #to_i this assumes base 10, but you can still use these to map between different forms too.
0xBEEF # => 48879

48879.to_s(16) # => "beef"

"beef".to_i(16) # => 48879

This should let you model some of the message structure in enums still, while keeping some sanity.

Ah damn it, I think messed up the last I tried to rebase this branch from master and now there's 333 commits in this PR but at least now the PR only contains those 2 nec files.

Don't stress the number / structure of commits here. We'll squash this down to a single commit for the merge - just important to ensure that squashed commit will only include the intended changes.
Let me know when you're happy with the new rebased state and I'll take a final look.

Popped a quick refactor inline for the response parsing - this is just to seperate those weird command reply packets and remove some of the cruft from previous iterations.
Aside from that, can you drop all of the -- NEC LED,... prefixes from log messages. These will always appear with the module / driver details as context so that'll remove some noise.
Tests looks good. Once the above two items are resolved, good for merge.

Just made this commit as well to handle the save response which wasn't been handled in the refactor https://github.com/PlaceOS/drivers/pull/50/commits/2ec673e83a02cdccfae6e94132b957d8ba6ef228 and this commit to cleanup the debugging messages/not used code https://github.com/PlaceOS/drivers/pull/50/commits/124d6edfc5312f74ca449e6c4ab98141cd2905ef

Popped a quick refactor inline for the response parsing - this is just to seperate those weird command reply packets and remove some of the cruft from previous iterations.
Aside from that, can you drop all of the -- NEC LED,... prefixes from log messages. These will always appear with the module / driver details as context so that'll remove some noise.
Tests looks good. Once the above two items are resolved, good for merge.

Just made this commit as well to handle the save response which wasn't been handled in the refactor https://github.com/PlaceOS/drivers/pull/50/commits/2ec673e83a02cdccfae6e94132b957d8ba6ef228 and this commit to cleanup the debugging messages/not used code https://github.com/PlaceOS/drivers/pull/50/commits/124d6edfc5312f74ca449e6c4ab98141cd2905ef


Exmaple

What version of Go are you using (go version)?

$ go version
go version go1.18.3 windows/amd64

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\Toyly\AppData\Local\go-build
set GOENV=C:\Users\Toyly\AppData\Roaming\go\env
set GOEXE=.exe
set GOEXPERIMENT=
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\Toyly\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\Toyly\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=c:\go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=c:\go\pkg\tool\windows_amd64
set GOVCS=
set GOVERSION=go1.18.3
set GCCGO=gccgo
set GOAMD64=v1
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=C:\Users\Toyly\Desktop\Go_Learn\examples1\go.mod
set GOWORK=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\Toyly\AppData\Local\Temp\go-build3541603774=/tmp/go-build -gno-record-gcc-switches
GOROOT/bin/go version: go version go1.18.3 windows/amd64
GOROOT/bin/go tool compile -V: compile version go1.18.3

What did you do?

What did you expect to see?
What did you see instead?

go1.18.3 windows/amd64


rte: remove more dead code

use RTE's middleware to replace code previously found in rte/pkg, and remove no longer needed support code.
No expected change in behavior.

/cc @Tal-or

/hold cancel
looking good
Thanks


questions about the ML_decoder

Hi, awesome work. I have a few questions about the ML_decoder.
I noticed that during the initialization, the pre-trained teacher network is loaded from state_dict, and then the fc is discarded and replaced by a randomly initialized ML_decoder, in the following lines.
https://github.com/penghui-yang/L2D/blob/d305c3d24c46749fe97d4e5d2afd8110f552c133/models/tools.py#L19
Does that mean the ML_decoder part of the teacher network is not pre-trained, since the teacher network is fixed, will it produce random results for distillation?

Good question.
If I understand correctly, what you talk about are the following statements in several files in the folder ./models :
if pretrained:
    net.load_state_dict(torch.load(pretrained_model_path["xxx"]))

What we load into the models here are the weights of backbones pretrained on ImageNet by unsupervised learning. Those pretrained weights can be easily downloaded from the Internet, and they can be seen as a special initialization of the untrained backbones. Such techniques are widely used in today's SOTA MLL methods.
The real weights of teacher models are saved and loaded by the following statements in the file main.py:
if not cfg.teacher_pretrained:
    # omit the training code for teacher models
    torch.save(model_teacher.state_dict(), "pretrained_models/model_teacher_%s_%s_%d.pth" % (cfg.model_t, cfg.dataset, cfg.img_size))
else:
    model_teacher.load_state_dict(torch.load("pretrained_models/model_teacher_%s_%s_%d.pth" % (cfg.model_t, cfg.dataset, cfg.img_size)))

The weights of the module ml-decoder are also included when saving and loading the weights of teachers.
Hope that answers your question.

Thanks a lot, that addresses my confusion.


Feature/ds connection

Description
DS connection feature that makes connection between frontend and DS; it takes mentee/mentor data from frontend and sends it to DS.
Video
DS Connection

[x] New feature

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have removed unnecessary comments/console logs from my code
[x] I have made corresponding changes to the documentation if necessary (optional)
[x] My changes generate no new warnings
[x] I have checked my code and corrected any misspellings
[x] No duplicate code left within changed files
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

This approach could definitely work, however I think it would be easier to send the data to DS at the end of the application endpoint we already use. All we would need to do is plug in the endpoint into the cacheSignUpData middleware and send the data at that point. What are your thoughts?

This is a great thought - let's find time to discuss this on Monday!

process.env.DS_API_URL/:role/create

Code is perfect, just need to remove empty spaces

remove it

DS API endpoint should be process.env.DS_API_URL/:role/create

resolved it

I believe it should be process.env.DS_API_URL/:role/create for the api calls.

resolved


How to add custom rules?

Laravel Mix Version: #.#.# (npm list --depth=0) latest
Node Version (node -v): v8.1.4
NPM Version (npm -v): 5.3.0
OS: Windows 10

Description:
Some scripts do not have the correct preparation for modular assembly. It is necessary to introduce here such fix, for the pace-progress package, how i can to do?
{
    test: require.resolve("pace-progress"),
    loader: "imports?define=>false"
}

I tried to do so, but got an error
mix
    .js  ('resources/assets/js/app.js',              'public/js')
    .sass('resources/assets/sass/app.scss',          'public/css')

    .webpackConfig({
        module: {
            rules: [
                {
                    test: require.resolve("pace-progress"),
                    loader: "imports?define=>false"
                }
            ]
        }
    })

Hi, @fearrr ! Try use null-loader

did it work?


Fl nnunet

PR Type
[Feature | Fix | Documentation | Other ]
Short Description
Changelog

Added a fully functional nnUNetClient for federated learning with nnunet models
Added the TransformsMetric class to metrics.py to allow users to apply transforms prior to calculating a metric. This allows the transforms to vary depending on the metric since different metrics might require different inputs.
Added the update_metric_manager method to BasicClient so that users coud override the function and make changes to preds and targets that apply to all metrics
Changed the typing of preds and targets for all clients to TorchPredType and TorchTargetType respectively. TorchPredType remains the same as before (Dict[str, torch.Tensor]). TorchTargetType extends the typing for targets to allow dictionaries of tensors, similar to TorchInputType. Created typing.py in utils to keep track of types across the entire package
Changed the _move_input_data_to_device() method of BasicClient to _move_data_to_device(). Changes the call to this function so that it is used for both the inputs and the targets
Added empty_cache() method to BasicClient

Notes

Although I changed the typing on all the clients in order to satisfy mypy checks, I left the original type hints for preds and targets in the method docstrings. This is intentional as it communicated to the user that the given method has not necessarily been updated to support all possible types of TorchTargetType or TorchPredType
The nnUNetClient contains code to support the ignore_label and has_regions functionality of nnunet. It's important to note however that the client has not been tested with nnunet models that use these features as I do not have the proper types of datasets to test them.

Known Issues
Update
I've figured this out and fixed it in two lines. Essentially one of the api's being used deep under the hood (batchgenerators) was doing logging in a naive way and added a stream handler to the root logger. This resulted in all the logs being printed to console once by the flwr logger and a second time by the root logger. Quick fix was to just clear the handlers on the root logger after getting the dataloaders which now that I think of it i'm surprised that worked and the stream handler wasn't just created over and over again everytime the next method was called but anyways it works. Raised an issue and might do a quick pr request for them.
Original issue
There currently exists an issue where when using the nnUNetClient, logs are printed to the console twice every time the flwr.common.logging.log() method is called. I have narrowed this issue down to the nnunet dataloaders. calls to log() before line 237 of nnunet_client:
train_loader, val_loader = self.nnunet_trainer.get_dataloaders()
result in the message only being printed to the console once. calls to log() after this line result in the same message being printed twice. There is most likely an additional log handler being created under the hood in either the nnunet api or the batch generators api. Since this issue does not affect the functionality of the client, and only affects the nnUNetClient, I have decided to include it in this PR. Also I currently have no idea how to fix this and figure it's a problem best left for another day.

I run and check why it is failing the smoke tests and pytests tomorrow

The logging fix has been commited, the tests are still going to fail.
The original test failures after 58114e3 don't make sense to me. The pre-commit checks passed on my machine but failed on github. Furthermore there were two failed pytests that also seemed unrelated to my code, they had something to do with the tests/utils/resources folder not being present. It was defenitely present on my machine so maybe the tests are being run out of the wrong working directory? Again this seems like an issue that is a result of changes made to the tests.
Additionally after merging with main in 532059a a bunch of mypy errors are now introduced that were not present before. I checked out and pulled the main branch and ran the precommit checks, and there are a bunch of mypy failures. Not sure how these were introduced

After fixing my dependencies, all pre-commit checks and pytests pass locally on my machine. Not sure why that is not the case on github

hold on the dependency updates might have broken the nnunet import

Ok I purposefully pushed an update that needs to be reformatted to pass the precommit checks. Will fix them monday. The upgrade to flwr 1.9.0 broke the nnunet client a bit because its dataloaders use multiprocessing and flwr redefined the SIGTERM and SIGINT signal handlers to raise an error any time a signal is interrupted or terminated. Therefore closing out the program caused errors when the main process went to close its subprocesses. I don't know why they did this but anyways I circumvented it by changing the signal handlers of the child processes

pre-commit checks and pytests all pass on my machine


Error passing named arguments to macros when using named imports

Given the following macro saved in the file forms.html:
{% macro input(type="text") %}
    <input type="{{ type }}"  />
{% endmacro %}

If you import forms.html without assigning the import to a variable then using named arguments works fine, e.g:
{% import "forms" %}
{{ input(type="text") }}

But if you import forms.html and set it as a variable then passing named arguments throws a ParserException with the message "Can not use named arguments when calling a bean method", e.g.
{% import "forms" as forms %}
{{ forms.input(type="text") }}

current implementation does not support named arguments , you can work around like this：
···
{% import "forms" as forms %}
{{ forms.input("text") }}
···


Origins launchpad - no sale on

The code is ready for review, the only question is if we will have email integration ready by the launch as this wait for backend. That's why there is the email input with handler that does nothing, it will be either commented-out or finished but that does not block the review.

Since we want to launch it today and the development branch is not yet ready to be merged to master, I squashed all of the commits into one and cherry-picked it to a new branch that has master as a base.
I am going to close this PR and https://github.com/DistributedCollective/Sovryn-frontend/pull/1228 is the new Origins launchpad PR.


Invalid user

Can you fix the bug that causes epb to display "The faggot known as @invalid-user has left :("??

hi sorry, can you describe it more specifically?
i added this function for a single server, so if you are running the scrip yourself, you might need to set the server id. it might be because its also displaying the leave message from other servers so it shows theyre invalid


Redesign how we present troubleshooting information

Currently, troubleshooting information is a bit fragmented across different pages:

https://keda.sh/docs/2.12/troubleshooting/
https://keda.sh/docs/2.12/faq/
Scaler pages

Although all of them look as versioned, only the last one (scaler pages) is versioned, so we have different paged for serving the troubleshooting versions, some of them versioned, other not versioned...
I think that we should rethink this section to make it more usable and user friendly. IMHO it's a high value change because if the FAQ/troubleshooting information is easily discoverable, we can reduce the overhead answering to user questions and users will be more happy because they can solve the issues by themself.
My proposal is generating a troubleshooting section with all this information available and and in page search to look for info by keywords. This section can contain some subsections/pages:

FAQ
General
Scalers

Scaler that requires info
...

Auths

Auth that requires info
....

@tomkerkhove , we could include this as part of next round of work from CNCF?

Goal is to have it yet, project is associated

I didn't catch that this had been closed. Is this work still needed?

It was closed by inactivity but it's still required


Resolves same-origin-policy violation when ADS web is running in a container

This PR fixes #17172
The change is to resolve the same-origin policy violation that only occurs when ADS web is running in a container and the themes or language syntax settings are changed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 41.48%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
26893

Relevant Lines:
58801

💛  - Coveralls


Remove unnecessary flavors

Current dependencies on/for this PR:

main

PR #87   👈

This comment was auto-generated by Graphite.


Mcdan/jdk11 containers

Description

Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[ ] I signed an Apache CLA.
[ ] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #8 into master will increase coverage by 0.01%.
The diff coverage is 79.37%.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
+ Coverage   84.45%   84.47%   +0.01%
==========================================
  Files         183      196      +13
  Lines        8305     8753     +448
  Branches      574      594      +20
==========================================
+ Hits         7014     7394     +380
- Misses       1291     1359      +68

Impacted Files
Coverage Δ

...rg/apache/openwhisk/core/database/StoreUtils.scala
100% <ø> (ø)
:arrow_up:

...isk/core/controller/actions/PrimitiveActions.scala
87.24% <ø> (ø)
:arrow_up:

...pache/openwhisk/core/entitlement/Entitlement.scala
88.42% <ø> (ø)
:arrow_up:

...rg/apache/openwhisk/core/controller/ApiUtils.scala
73.56% <ø> (ø)
:arrow_up:

.../org/apache/openwhisk/core/entity/WhiskStore.scala
96.96% <ø> (ø)
:arrow_up:

...enwhisk/connector/kafka/KamonMetricsReporter.scala
0% <0%> (ø)
:arrow_up:

...pool/docker/StandaloneDockerContainerFactory.scala
0% <0%> (ø)
:arrow_up:

...isk/core/database/memory/NoopActivationStore.scala
0% <0%> (ø)

...pache/openwhisk/core/monitoring/metrics/Main.scala
0% <0%> (ø)

...erpool/kubernetes/KubernetesContainerFactory.scala
0% <0%> (ø)
:arrow_up:

... and 58 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2036548...b4b6604. Read the comment docs.


Change config files names

config_bridge.toml -> bridge.cfg
config_compliance.toml -> compliance.cfg

Another option is #45.


Support for Verilog language

Description
Add support for the Verilog hardware description language.
Existing third party linters
There exists an open source and configurable linter/beautifier here: https://github.com/thomasrussellmurphy/istyle-verilog-formatter

Published to v0.32.0

wow thanks!


Fix capture mode respawn time

I totally support the change of the respawn time in capture and regen capture to 10 seconds instead of the current 5s for the named reasons:
http://cubeengine.com/forum.php?action=display_thread&thread_id=2666http://cubeengine.com/forum.php?action=display_thread&thread_id=2666
What do you all think about that?

I'd say we should not change this for now until we have #339 and then we can try it experimentally

We will have #339 soon, any more thoughts?
I would be in favor, since capture was one of the most popular modes and seemed to have suicided after that change.

It was decided and implemented on the hackathon. Go for it into master branch.

https://github.com/inexorgame/inexor-core/commit/de412cea0902d54c4a3b58d76dfbbb20f62aa3de

Seems like writing (close #64) within a PR title doesn't work any longer.
Needs be written in the pr body now.
Fixed by #556


What the villager doing?

If you mark a bed that a villager is sleeping in then this happens

They keep the sleeping pose and walk at the same time can that get fixed?

It seems to be related to marking a bed when a villager is already sleeping in it after loading into a world. Can you confirm this on your end? Breaking the bed resets their pose as a temporary fix. Currently investigating the cause.

I've got a fix coming out for both 1.18.2 and 1.19.2 that should solve it, but you will still have to break the bed the villager was in to correct their pose. It had to do with the several issues combining:

The villager AI running its path to bed at night behavior even when already in bed.
This mod clearing the bed location if the bed is marked when the AI tries to path to the bed.
The villager AI not running its sleep behavior when the game is reloaded if it was sleeping the last time the game was shutdown. This behavior is typically what kicks the villager out of bed when you mark it.

Thanks for the report. If you find any other issues or it is still occurring after you break and replace the offending bed, please be sure to let me know.


Improve operating system and architecture types in uv-toolchain

Prompted in https://github.com/astral-sh/uv/pull/2842#discussion_r1555640300
Updates the toolchain Os and Arch types to be a closer match to those in platform-tags.
They don't match exactly, e.g. we track Libc separately instead of using Manylinux and Musllinux and we don't need release versions. This makes me think we might not actually want to consolidate to central shared types.

Merging into #2842.


Error when creating MOF files

I am running windows 10 pro and currently have Hyper V installed using it for some test VM's already. I have followed the instructions to get Auto Lab working but regardless of which config I  use, when I run the Unattend-Lab I get as far as trying to create the MOF files and then it fails and goes no further
PS C:\AutoLab\Configurations\devops-powershell-fundamentals> Unattend-Lab
   This runs Setup-Lab, Run-Lab, and validate-Lab

Starting the lab environment
    This is the Setup-Lab script. This script will perform the following:
    * Run the configs to generate the required .mof files
    Note! - If there is an error creating the .mofs, the setup will fail

    * Run the lab setup
    Note! If this is the first time you have run this, it can take several
    hours to download the .ISO's and resources.
    This only occurs the first time.

    **Possible problem, if the downloads finish but the script doesn't continue (pauses)
        Hit the return key once and it will continue

    *You will be able to wipe and rebuild this lab without needing to perform
    the downloads again.

Installing required DSCResource modules from PSGallery
You may need to say "yes" to a Nuget Provider
Build the .Mof files from the configs
Write-NodeMOFFile : Invalid MOF definition for node 'DC1': Exception calling "ValidateInstanceText" with "1"
argument(s): "Convert property 'IPAddress' value from type 'STRING[]' to type 'STRING' failed
At line:20, char:2
Buffer:
onName = "AutoLab";
};^
insta
"
At
C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:2369
char:21

...             Write-NodeMOFFile $Name $mofNode $Script:NodeInstanceAlia ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [Write-Error], InvalidOperationException
FullyQualifiedErrorId : InvalidMOFDefinition,Write-NodeMOFFile

Directory: C:\AutoLab\Configurations\devops-powershell-fundamentals

Mode                LastWriteTime         Length Name

-a----       16/10/2018   7:51 PM         526374 DC1.mof.error
Write-NodeMOFFile : Invalid MOF definition for node 'S1': Exception calling "ValidateInstanceText" with "1"
argument(s): "Convert property 'IPAddress' value from type 'STRING[]' to type 'STRING' failed
At line:20, char:2
Buffer:
onName = "AutoLab";
};^
insta
"
At
C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:2369
char:21

...             Write-NodeMOFFile $Name $mofNode $Script:NodeInstanceAlia ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [Write-Error], InvalidOperationException
FullyQualifiedErrorId : InvalidMOFDefinition,Write-NodeMOFFile
-a----       16/10/2018   7:51 PM           8406 S1.mof.error
Write-NodeMOFFile : Invalid MOF definition for node 'S2': Exception calling "ValidateInstanceText" with "1"
argument(s): "Convert property 'IPAddress' value from type 'STRING[]' to type 'STRING' failed
At line:20, char:2
Buffer:
onName = "AutoLab";
};^
insta
"
At
C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:2369
char:21

...             Write-NodeMOFFile $Name $mofNode $Script:NodeInstanceAlia ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [Write-Error], InvalidOperationException
FullyQualifiedErrorId : InvalidMOFDefinition,Write-NodeMOFFile
-a----       16/10/2018   7:51 PM           8406 S2.mof.error
Write-NodeMOFFile : Invalid MOF definition for node 'Cli1': Exception calling "ValidateInstanceText" with "1"
argument(s): "Convert property 'IPAddress' value from type 'STRING[]' to type 'STRING' failed
At line:20, char:2
Buffer:
onName = "AutoLab";
};^
insta
"
At
C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:2369
char:21

...              Please review the errors reported in error
stream and modify your configuration code appropriately.
At
C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:3917
char:5

throw $ErrorRecord

~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (AutoLab:String) [], InvalidOperationException
FullyQualifiedErrorId : FailToProcessConfiguration

PS C:\AutoLab\Configurations\devops-powershell-fundamentals>
Any guiders or assistance would be greatly appreciated.
BTW I am running the latest 1803 release of windows 10 as well

I got the same thing last night when I tired.

same issues as reported above when attempting to create the .mof files. smh Jason.

Try prefixing the variable with the datatype.  in this case [string].  This worked in my case

Same issue for me.  Attempted to prefix the variable, but still not working for some reason.
The error I received when running Setup-Lab for POC-Multirole
`Build the .Mof files from the configs
Write-NodeMOFFile : Invalid MOF definition for node 'DC1': Exception calling "ValidateInstanceText" with "1"
argument(s): "Convert property 'IPAddress' value from type 'STRING[]' to type 'STRING' failed
At line:20, char:2
Buffer:
onName = "AutoLab";
};^
insta
"
At
C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:2369
char:21

...             Write-NodeMOFFile $Name $mofNode $Script:NodeInstanceAlia ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [Write-Error], InvalidOperationException
FullyQualifiedErrorId : InvalidMOFDefinition,Write-NodeMOFFile

Directory: C:\AutoLab\Configurations\POC-MultiRole

Mode                LastWriteTime         Length Name

-a----        3/23/2019  11:20 PM         519848 DC1.mof.error
Write-NodeMOFFile : Invalid MOF definition for node 'S1': Exception calling "ValidateInstanceText" with "1"
argument(s): "Convert property 'IPAddress' value from type 'STRING[]' to type 'STRING' failed
At line:20, char:2
Buffer:
onName = "AutoLab";
};^
insta
"
At
C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:2369
char:21

...             Write-NodeMOFFile $Name $mofNode $Script:NodeInstanceAlia ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [Write-Error], InvalidOperationException
FullyQualifiedErrorId : InvalidMOFDefinition,Write-NodeMOFFile
-a----        3/23/2019  11:20 PM           8930 S1.mof.error
Write-NodeMOFFile : Invalid MOF definition for node 'N1': Exception calling "ValidateInstanceText" with "1"
argument(s): "Convert property 'IPAddress' value from type 'STRING[]' to type 'STRING' failed
At line:20, char:2
Buffer:
onName = "AutoLab";
};^
insta
"
At
C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:2369
char:21

...             Write-NodeMOFFile $Name $mofNode $Script:NodeInstanceAlia ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [Write-Error], InvalidOperationException
FullyQualifiedErrorId : InvalidMOFDefinition,Write-NodeMOFFile
-a----        3/23/2019  11:20 PM           5834 N1.mof.error
Write-NodeMOFFile : Invalid MOF definition for node 'Cli1': Exception calling "ValidateInstanceText" with "1"
argument(s): "Convert property 'IPAddress' value from type 'STRING[]' to type 'STRING' failed
At line:20, char:2
Buffer:
onName = "AutoLab";
};^
insta
"
At
C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:2369
char:21

...             Write-NodeMOFFile $Name $mofNode $Script:NodeInstanceAlia ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [Write-Error], InvalidOperationException
FullyQualifiedErrorId : InvalidMOFDefinition,Write-NodeMOFFile
-a----        3/23/2019  11:20 PM          13274 Cli1.mof.error
Compilation errors occurred while processing configuration 'AutoLab'. Please review the errors reported in error
stream and modify your configuration code appropriately.
At
C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:3917
char:5

throw $ErrorRecord

~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (AutoLab:String) [], InvalidOperationException
FullyQualifiedErrorId : FailToProcessConfiguration`

The Variable I attempted to prefix was the IPAddress variable.

This project is now under new ownership. The 3.x version line is obsolete. Version 4.0.0 has been published to the PowerShell Gallery with new updates, configuration, and documentation. It is expected that this version will resolve most if not all of the problems you were experiencing. Read Updating for more information.
No further action will be taken on this issue and it will most likely be closed at some point. If you still have the same issue with the new version, then go ahead and open a new issue.

For future reference, a number of configurations have been archived and are no longer supported.


fix(jQueryMixin): clear ToastMessage.

Because not trigger re-setTimeout on mouseleave when ToastMessage isHiding.

Any updates on this?


Data

There are some issues with<EMAIL_ADDRESS>and loading data. To handle the situations that I see from the site.yaml file, I added a loadData function that can be called. It does the following...

load regular data files
load root data files (site.yaml, package.json)
namespace package.json data as pkg
process templates found in data values... <%= pkg.author %>

@doowb  Excellent Brian! I felt like it was me at first, but I knew something had to be up. I owe you a really nice write up on all this very soon :+1:


Can Kap be used to get screen recording of test run on Circle CI

Hi, I'm trying to get the screen recording of the test run in the $CIRCLE_ARTIFACTS folder in Circle CI.
Below is the test section of the build where I would like to start capturing
test:
  override:
    - >
      if [ -n "${RUN_APPIUM_TESTS}" ]; then
        # Install java jdk8
        brew update
        brew cask install java
        brew update && brew cask install kap
        # Install maven
        brew install maven
        npm install -g<EMAIL_ADDRESS>        appium &
        pod install
        xcodebuild CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= PROVISIONING_PROFILE= -scheme "Mumbo Jumbo" -workspace 'MumboJumbo.xcworkspace' -sdk 'iphonesimulator' -destination 'platform=iOS Simulator,name=iPhone 6 Plus,OS=11.2' | xcpretty
        #cd to Code directory & run tests!
        pushd "appium-selenium/MumboiOSTests"
        mvn clean test;
        popd
      fi

How do I start kap from the command line, start recording and store the recoding in $CIRCLE_ARTIFACTS folder. Thanks in advance for the help!

Kap can unfortunately not be controlled from the command-line.

You can however use our recording library (macOS only) https://github.com/wulkano/aperture to archive this @aleenadavy90 💨


Add setReleaseStage to config.yml

Allo release stage to be set in config.yml in case one wants to override kernel.environment.

Thanks for getting in touch, The recommended way to set this is programmatically at the moment:
https://docs.bugsnag.com/platforms/php/symfony/configuration-options/#release-stage

Yeah - we should allow you to override in config.yml unless there is a reason not to?

Yeh, I agree. I can't remember why this wasn't implemented in both the Laravel and Symfony notifiers. I guess in Laravel it's quite common to perform "configuration" programmatically in service providers.

I'll have a go at writing this feature in both notifiers, and see if anything jogs my memory as to why it wasn't done before.

Symfony PR implemented at https://github.com/bugsnag/bugsnag-symfony/pull/39, and Laravel PR implemented at https://github.com/bugsnag/bugsnag-laravel/pull/228.

The semantics are, if the "release_stage" is set in config, use that, otherwise, fallback to using the application/kernel environment name, as before.

And for silex: https://github.com/bugsnag/bugsnag-silex/pull/20.


Testing/Validation metadata

Enforce certain fields on all content:

Title
Description
Test description lenght?
Tags
Capitalization
Enforce non-unique tags? Warning?
Hero

https://www.mapbox.com/blog/unit-test-blog/

https://www.mapbox.com/blog/retext-mapbox-standard/
https://github.com/mapbox/retext-mapbox-standard

https://github.com/amperser/proselint

Added basic testing:

Automatically title case tags
Ensure hero image is present and includes name, alt, and caption


added proper support text

I'm also concerned that writing "LMAO" implies a level of effort I'm not willing to put in. After all, if I've got time to use the shift key, why don't I have time to help debug your issue, huh?

I'm also concerned that writing "LMAO" implies a level of effort I'm not willing to put in. After all, if I've got time to use the shift key, why don't I have time to help debug your issue, huh?

💐 😅

Getting watch notifications on this silly topic, I think this pull request should be closed. @moyix

Gotcha, don't want to annoy people! :)


neo4j jar files should be put in a separate gem

This will make it easier to upgrade to newer gems.
Does also make it faster to push the neo4j.rb gem to gemcutter since the size of the gem will be reduced a lot.
See http://github.com/andreasronge/neo4j-jars

Do you reckon we can close or still need more testing?

We have to either update the docs about the online backup (http://neo4j.rubyforge.org/guides/configuration.html#backup)
or fix so that it works. Have to test the examples/ha-cluster so that it still work. And then we should use it in a real application, using both community and advanced and enterprise. Afterthat I guess we can close it

Ok. Unfortunately I don't think I'll be able to properly test it yet. We need ask somebody to I guess.


Refresh dialoge

Hi,
I notice the save and apply button on a dialog refreshes the content in the dialog.  How can I do this after a custom button save?
Thanks

If inside you dialog class:

this.reloadById();

Thanks this worked


Cannot create Pivot model with fillable attribute

Laravel Version: 5.6.5
PHP Version: 7.2
Database Driver & Version: mysql innodb 5.7.21

Description:
I am trying to create a test for testing a relation of a pivot table/model.
However, I cannot seem to be able to create the pivot model even though I have the $fillable attribute defined on it.
Steps To Reproduce:
I have a model extending from Pivot
Some class extends Pivot
{
     protected $table = 'some';

     protected $fillable = ['field_1', 'field_2'];

}

I have the following factory:
$factory->define(\App\Some::class, function (Faker $faker) {
    return [
        'field_1' => function () {
            return factory(\App\Another::class)->create()->id;
        },
        'field_2' => $faker->randomDigit(),
    ];
});

When I try to create the pivot model in tinker:
$test = factory('App\Some')->create();
I get the error
Field 'field_1' doesn't have a default value (SQL: insert into `somes` (`updated_at`, `created_at`) values (2018-03-01 15:59:55, 2018-03-01 15:59:55))'

I can see that the call in the factory to create Another is executed and see the row in the database.

My mistake...I was overriding the pivot constructor but did not call parent..


[Proposal] Build standalone JS class to convert fastlane output from generic format to prettified HTML

So I'm thinking a bit more about how we render the real-time log of fastlane.ci in the user's browser. We haven't decided on front-end frameworks and stuff yet, so things are still in the open
but having HTML based output for fastlane output is something that would be interesting outside of fastlane.ci also
What I mean, imagine you can run fastlane beta --html (a regular lane) and it would additionally generate a HTML file with colored output, which can then be used for other CI systems, etc.
so the way we could architect is could be that it's a single .js file that converts JSON data to beautiful HTML based log files. The JSON data could look something like this
[
  {
      "type": "message",
      "text": "fastlane is great"
  },
  {
      "type": "header",
      "text": "Step: ensure_git_status"
  },
  {
      "type": "error",
      "text": "Ugh, git isn't clean"
  }
]

generating HTML + CSS to something like this (very simplified of course)

fastlane is great

Step: ensure_git_status

Ugh, git isn't clean

by having a single .js file, we could use it for existing fastlane setups by just running that js file after running a full build
and then we could use the same .js file for real-time output rendering for fastlane.ci

So in the case of fastlane it would run on the user's machine locally using the local JS engine that's pre-installed on the Mac
In the case of fastlane.ci we'd run it on the user's browser, piping over the JSON metadata through a web socket, updating the HTML dom for the user in real-time

Firstly, I don't think there is a JS engine pre-installed in macOS.
And secondly, where would the CSS be? Or we require the user to provide one, being able to customize it how they want?

The CSS would be at 2 locations:

for fastlane.ci it would be part of fastlane.ci
for fastlane it would be bundled in the HTML file, probably a more basic style sheet

I'm gonna have to look into more detail on the JavaScript side of things, but it looks like it's pre-installed on every Mac
/System/Library/Frameworks/JavaScriptCore.framework/Versions/A/Resources/jsc

Oh, ok, didn't know about that

I'm of course also open for other suggestion. I guess the best alternative would be a Ruby based implementation:

A ruby method that takes a message object (see sample JSON above) and returns an HTML formatted string
This way it can be used directly within fastlane without a dependency to JS
This way it can be easily be used by fastlane.ci by piping over the direct HTML via a socket connection

Yah I think the Ruby implementation makes more sense, will do that. Thanks @revolter 👍

So, this takes a fair amount of thinking about, especially due to memory usage on browsers, and handling of both long, and wide logs.
The way that many CI providers handle this is by streaming logs from the worker to the log processor (in this case probably fastlane ci main process, but this will be a memory issue). The log processor will then do two things: build a buffer of the entire log (in something like redis), and stream log lines to streaming browsers.
When the build is completed, the full log buffer is then moved to persistent storage, and served as a file for future display in builds and downloading.
If the log is above y chars, then save a truncated browser version, and a full downloadable version.
So there are actually two ways that you load logs: The live log from streaming, and the full log when viewing an old build.
You can also pre-render the log as html, as a browser optimisation.
Many browsers have issues with rendering large logs, and will completely lock up under many circumstances. (which is where truncation comes into play)

Update: We decided to not generate HTML on the server, but instead do it using JavaScript

cc @taquitos

Follow-up work via https://github.com/fastlane/ci/issues/213

Let's close this, as it's covered by other issues