Integration does not offer any useful option

Hi,
I have a Sileno City 500, two smart Water Controls and two sensors.
Previously I used openHAB and their Gardena integration. I had a lot of options, e.g. I was able to pause / skip a schedule. For example I was able to create an automation to skip a scheduled irrigation when it is raining or skip a scheduled lawn mowing only for the current day.
With your integration in Home Assistant there a no options. It only shows the current values of the sensor (temperature, etc.) and I'm only able to start/stop with a switch. Is this a bug or why are there no other options? The valves, the schedules, etc.? Due to the fact that the openHAB integration is offering all those options, it should be no problem to receive those values or send those commands from/to the Gardena API?

This is a community driven integration, so feel free to dive in and add
this functionality and submit a pull request for your improvements.
On Sat, 19 Oct 2024 at 08:32, Sascha-L @.***> wrote:

Hi,
I have a Sileno City 500, two smart Water Controls and two sensors.
Previously I used openHAB and their Gardena integration. I had a lot of
options, e.g. I was able to pause / skip a schedule. For example I was able
to create an automation to skip a scheduled irrigation when it is raining
or skip a scheduled lawn mowing only for the current day.
With your integration in Home Assistant there a no options. It only shows
the current values of the sensor (temperature, etc.) and I'm only able to
start/stop with a switch. Is this a bug or why are there no other options?
The valves, the schedules, etc.? Due to the fact that the openHAB
integration is offering all those options, it should be no problem to
receive those values or send those commands from/to the Gardena API?
—
Reply to this email directly, view it on GitHub
https://github.com/py-smart-gardena/hass-gardena-smart-system/issues/232,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQZVQ2GACUMU46OHBICHSC3Z4IDJFAVCNFSM6AAAAABQHHKMRCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGU4TQOBVGUZTGMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Allow specifying a source encoding for the jacoco report tasks

Use the encoding of the compileJava task by default (if specified), or the default encoding if not specified
Fixes #13666
This is a reanimation of #13687 by @jnizet which got abandoned.
I did not make any changes or tested it manually.
I just rebased the old PR and resolved the conflicts.
But if there are any findings, I'll do my best to resolve them.
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[x] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass sanity check: ./gradlew sanityCheck
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

Thank you for your contribution!
This PR has a valid DCO and tests. The relevant team for this area will confirm the design of the implementation choices.

@Vampire please look into the sanityCheck failure reported above.

Can you tell me @cobexer?
What fails is the :architecture-test:checkBinaryCompatibility task: https://scans.gradle.com/s/ppc7t7bix5lx6/failure
I cannot even execute that task here due to #27120.

Ok, got it, it was missing @since and @Incubating.
Fixed now.
The other build failure above is not related to my changes.

Thanks for the fast review @tresat. As I said, I just rebased the work of the other PR and didn't really look into the changes. All you said makes sense from what I've seen, I'll fix the findings.

I went through the changes and cleaned it up in some places to how I think it should be, which then also automatically covered your findings. Please have another look now. :-)

Something about this seems odd. Its likely this is the sort of information you'd want to declare at the source set level instead of the jacoco report task level. The jacoco report should then inherit this info from the source set

@jvandort I would not object, but I don't really want to refactor all that code in Gradle. :-)
There are various other tasks that are like this like JavaCompile, JavaDoc, and others I forgot right now.
This just makes the JacocoReport task in line with those, also providing the possibility to configure an encoding.
Actually, even if that property is added to source sets, the changes here would still be necessary, just the default would be set to the source set encoding then, so this can also easily be done in a second step.

Sorry for the late reply.
The responsible team needs to wrap up their feedback.

This needs to be refreshed for 8.8.  We'll take a look at it then.

Assuming you mean rebasing on master when you say "needs to be refreshed", done. :-)

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

WARN: This is a feature pull request but no changes to release note found.

The merge queue build has started. Click here to see all failures if any.


DS-2040: Fix AIP restore so that it works with cross-mapped Items & add AIP Integration Tests

https://jira.duraspace.org/browse/DS-2040
This PR fixes DS-2040, but also adds AIP Integration Testing (which was integral in actually discovering the underlying bug behind DS-2040). I've tried to keep individual changes as separate commits to make this easy to understand. But here's the overview of what all is in this PR:

Adds a new DSpaceAIPIntegrationTest class which (during Unit Testing) automatically generates a test hierarchy of Communities, Collections and Items, and exports them to AIPs. It then performs a series of test restorations/replacements of test content based on those AIPs, and performs assertions against the final results to ensure restoration/replacement is working properly.
Updates / refactors AbstractPackageIngester and AbstractPackageDisseminator to keep track of all packages processed (whenever their ingestAll() or disseminateAll() methods are called). This is imperative to fixing DS-2040, as the underlying issue is that Mapped Item packages were accidentally being processed multiple times.  See commit https://github.com/tdonohue/DSpace/commit/74241a4c9463998af6e6e9c8e59cc8cf761956c0
Changes the default settings in dspace.cfg for AIP and METS ingester/disseminator plugins. By default, ALL plugin instances get cached in memory. But, in order to fix DS-2040 (as noted in bullet 2 above) for recursive imports/exports, we MUST have instance level global variables which keep track of which packages have already been processed. So, we CANNOT cache these plugin instances, as the global variables would never be reset properly. See commit https://github.com/tdonohue/DSpace/commit/a4d382d46b7a4548fbe25b60d2efeb1ba996b4a2
All the above changes now have Integration Tests which verify that Community/Collection/Items can be restored/replaced properly using AIPs. This includes integration tests which verify that Mapped Items can be restored/replaced properly. Again, see DSpaceAIPIntegrationTest

I'm confident this all works, as it all has tests built against the new code. However, I'd welcome more eyes & testers before we consider merging this for 5.0

Pinging @hardyoyo , just cause I know this bug is of interest to you. If you get a chance to try out this PR, I'd appreciate it.

+1 tested, confirmed that an AIP of a cross-mapped item can now be restored. I used a replacement AIP for item 10673/7, I've uploaded the test AIP I used as an attachment to the DS-2040 ticket (see: https://jira.duraspace.org/secure/attachment/16985/10673-7.zip) if you would like to use it for testing, you should be able to do so on demo.dspace.org, or vagrant-dspace.

+1 tested, confirmed that

I can reproduce the problem on master both with @tdonohue's test (thanks @hardyoyo for the branch with just the test stuff) and with a manually created hierarchy similar to Tim's test structure
Tim's code makes the test pass and a manual import of the hierarchy works too.


How to turn a image into a circle

What are you trying to achieve?
Have you searched for similar questions?
Are you able to provide a minimal, standalone code sample that demonstrates this question?
Are you able to provide a sample image that helps explain the question?

Hi, there's an example of this in the README - please see https://github.com/lovell/sharp/blob/master/README.md#stream

thanks

A "full" example would be:

import fs, { ReadStream } from "fs";
import sharp from "sharp";

(async () => {
    const writeStream = fs.createWriteStream('./output.jpg');
    const readableStream = fs.createReadStream(__dirname+"/img.jpg")
    const roundedCorners = Buffer.from(
        '<svg><rect x="0" y="0" width="50" height="50" rx="50" ry="50"/></svg>'
    );

    const roundedCornerResizer =
        sharp()
            .resize(50, 50)
            .composite([{
                input: roundedCorners,
                blend: 'dest-in'
            }])
            .png();

    readableStream
         .pipe(roundedCornerResizer)
         .pipe(writeStream);
})();

To do the same thing using a buffer you would do:
import fs, { ReadStream } from "fs";
import sharp from "sharp";

// Helper function to change my file read from a readStream to a buffer
export async function streamToBuffer(readable: ReadStream) {
    const chunks: Buffer[] = [];
    for await (const chunk of readable) {
        chunks.push(chunk);
    }
    return Buffer.concat(chunks);
}

(async () => {
    const readableStream = fs.createReadStream(__dirname+"/img.jpg")
    const buff = await streamToBuffer(readableStream);
    const roundedCorners = Buffer.from(
        '<svg><rect x="0" y="0" width="50" height="50" rx="50" ry="50"/></svg>'
    );

    const roundedCornerResizer =
        sharp(buff)
            .resize(50, 50)
            .composite([{
                input: roundedCorners,
                blend: 'dest-in'
            }])
            .png()
            .toFile("./output2.png");;
})();

`toFile()` can be replace by `toBuffer` or whatever else you want.


Optionally, allow parentheses in control flow conditions

This is yet-another-option for the omit_parentheses style for the
Style/MethodCallWithArgsParentheses cop. When set to true, it allows
the presence of parentheses in control flow conditions and the body of
their expression modifier forms.

Hmm, why would we want to treat this specially? Seems pretty weird to me.

This is a request I'm getting from some of my team mates. The following code snippet:
server.create(Reader::PROCESSED_FOLDER) unless server.list('', Reader::PROCESSED_FOLDER)

... is currently being converted to:
server.create Reader::PROCESSED_FOLDER unless server.list '', Reader::PROCESSED_FOLDER

Some people find it more readable like in the first form. To be honest, I also feel those exceptions to the rule are getting a bit too much. If you don't think they are useful to the general public, I can keep them in local cops.

Yeah, honestly this feels excessive. You really need to reconsider your approach to consistent code style. 😉 I certainly see no reason why this code should have parens if you're enforcing the no-parens style. If I were a random observer of such a code style I'd be hard-pressed to figure out it's cohesive...

Cool, closing this one! 🙇♂


LSPosed is not hidden

/dev/block/dm-42 /apex/com.android.art/bin/dex2oat64 ext4 ro,seclabel,noatime,resgid=1065,errors=panic 0 0

The mount above is mounted even if no Xposed modules are injected into the target app.

Fixed in 5c2ba2a

Apparently still doesn't work on some setups.

Fixed in 7cbb41b for all root implementations.


Add mandatory version bump in PRs

Ensures that each PR has bumped the version from the one published on NPM.

Codecov Report

Merging #3 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master     #3   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines         245    245
  Branches       31     31
=====================================
  Hits          245    245

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 379bd11...5dcb7c2. Read the comment docs.


Add unit tests and cleanup code for for-in issues.

Add unit tests and cleanup code for for-in issues.

I'd love to merge. Mind rebasing to make it apply cleanly against master? Also, comments should be sentences that end with a period.

sure thing, will cleanup so it can merge clean and address comment style.

Updated.


ui-grid: fixed reference to the IGridRow type to refer to uiGrid.IGridRow within ui-grid plugins

Some of the plugins for ui-grid define an IGridRow interface to extend the possible options on the row object.  The main IGridRow interface extends all of the plugins' IGridRow interfaces so that you always get the best autocompletion.  But within the plugin modules, referring to the un-qualified IGridRow ends up resolving to the inner interface, with only a couple of options.  Using these plugin methods can therefore cause errors when trying to use other (non-plugin) fields on the row object.
I have qualified references to IGridRow within the plugin modules so the correct IGridRow (uiGrid.IGridRow) is used.

I don't think this is a problem, except for the getSelectedGridRows function which is already fixed in a different PR. However, the functions that takes rowEntity as parameter but uses IGridRow as type instead of any, those should be fixed. See my comments.

@baltie , thanks for taking the time to look at this! I will make changes as you have suggested.

@baltie, @btesser, after changing rowEntity types to be any, I couldn't help but think that we could improve this by using generics, since the gridOptions.data will often be an array of a single type.  This would be a bigger change, as it would involve adding generic types all over the place.  Before I jump in and get started, what do you think?  Do you think there is value in using generics for rowEntity types?

Thanks, @joeskeen!
Changing rowEntity from any to a generic would certainly be a bigger change, meriting its own PR. :) I think it's a good idea and it would improve the ui-grid definitions at least for me, but I think @btesser should have the final say on that one.

Fixed.

LGTM Guys.  Sorry for the delay.  Crazy weekend

Thanks @btesser! :)

@vvakame This PR is ready for merge, we think. :)

@btesser thanks!
@baltie thanks mate! sorry for delay.

@vvakame Thanks! :)
And thanks, @joeskeen! :)


DiskANN CMake Support

issue: https://github.com/milvus-io/knowhere/issues/253

/assign @liliu-z

@matchyc  Need to rebase this PR to support_diskann and solve the conflict first.

@Presburger any thought?

@matchyc
Knowhere support compilation on mac/linux/windows, but DiskANN only support Linux.
Need to consider not to affect Knowhere's compilation on other platforms.

@matchyc Knowhere support compilation on mac/linux/windows, but DiskANN only support Linux. Need to consider not to affect Knowhere's compilation on other platforms.

This pr

MKL is left to users to install, and the build with the MKL will be done by diskann build procedure so we don't need to consider it in cmake for now.

@matchyc Knowhere support compilation on mac/linux/windows, but DiskANN only support Linux. Need to consider not to affect Knowhere's compilation on other platforms.

Add several switches to avoid adding diskann into libknowhere in Mac/Windows. Thanks for pointing it out.

/approve

/run-cpu-e2e

@matchyc Knowhere support compilation on mac/linux/windows, but DiskANN only support Linux. Need to consider not to affect Knowhere's compilation on other platforms.

Plan changed. Adding build option and skip ci in systems except for Ubuntu to avoid build failure in other platforms.


chore(docs): Update host and base url

Flipping the docs site to weave-docs.wandb.ai

I have read the CLA Document and I hereby sign the CLA


Fixing dependencies with bad license in Dapr SDKs

Related to #4158
The objective of this issue is to clean up all dependencies across our SDKs which are not compliant with the CNCF guidelines for licensing - See Allowed third party license policy | cncf/foundation. A similar exercise is on-going for other repos - runtime, components-contrib, kit, dashboard, and go-sdk (see related issue). Per this, we need to repeat the same for all Dapr SDKs.
Part 1: Upload the SDK repo to FOSSA

Download the FOSSA CLI (https://github.com/fossas/fossa-cli#installation)
Clone the main SDK repo (i.e., not any fork) and run fossa analyze --fossa-api-key b88e1f4287c3108c8751bf106fb46db6 --team Dapr (the token is push-only so it's safe to share it here)

This will push the project to FOSSA for scanning.
Part 2: Add a workflow
Use this as a reference to create a new workflow in the SDK repo https://github.com/dapr/components-contrib/pull/1568. Additionally, add a FOSSA badge to the README.
This will ensure that no further bad dependencies are introduced further.
Part 3: Resolve issues
The workflow run will fail if there are any bad dependencies. Please use the report generated in the CI step to see them.
Example:

Report from above.
The licensing issues can be divided into two categories:

false-positives or ones that CNCF have allow-listed: maintainers simply need to resolve this from the FOSSA dashboard
under review by CNCF or needs code change (updating a library/removing it/etc.): these need to have an issue created before it can be resolved temporarily from FOSSA. Once the issue is resolved and taken up by our code, it should be removed from FOSSA's allow-list.

List of exceptions already granted by CNCF: License exceptions
Please comment on your SDK issue with a list of dependencies and justifications that need to be marked as resolved. It needs to be manually resolved by anyone who has the FOSSA account access.

Closing this issue - separate SDK issues will be track this.


Missleading segment in URL

The URL is https://docs.microsoft.com/en-us/azure/load-testing/how-to-update-rerun-test, but the title is "Monitor server-side application metrics by using Azure Load Testing Preview"
I think the last segment in the URL should be corrected.

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

ID: e3c31866-85f2-ea22-704b-ee91e8990a18
Version Independent ID: 06487704-2e0f-fa17-9275-cfbacc1483ae
Content: Monitor server-side application metrics for load testing - Azure Load Testing
Content Source: articles/load-testing/how-to-update-rerun-test.md
Service: load-testing
GitHub Login: @ntrogh
Microsoft Alias: nicktrog

@adymorz Thanks for your feedback! We will investigate and update as appropriate.

@adymorz Thanks for raising this. We're in the process of updating the file name and related links. I'll report back when the updates are published.

@adymorz the updates are live. Thanks again for your feedback!
#please-close


Updated H1 in Readme

So that sample will display properly in MS Samples gallery

Might loop in @jonnords as FYI.

@davidchesnut I made the update. @jonnords let me know if I need to make any more before merging.


No error messaging

I've authenticated with my API key, but no matter what I prompt, I get no response. I assume ChatGPT is returning some kind of error, but nothing is reported in the terminal. Would it be possible to relay errors from ChatGPT?

Same issue here, I'm on the free plan, generated an API key, chatty auth successful, but chatty chat just gives me empty responses to all my queries.

Thanks for sharing the feedback @chrispix @thuss 🙌
I will look into adding some form of error handling, I am guessing there is an issue with the key, have you copied it directly into the terminal?

Thanks for sharing the feedback @chrispix @thuss 🙌
I will look into adding some form of error handling, I am guessing there is an issue with the key, have you copied it directly into the terminal?

Yes. And I checked the dev.polpiella.chatty entry in Keychain Access, looks like it was stored correctly (though I was not prompted to unlock keychain first). I was assuming there might be some kind of error message from OpenAI that you could echo to the terminal.

I discovered the issue in my case was that I didn't have pay-as-you-go set up, and my free tokens had expired. Not sure if the API gives you any indication that there are no API tokens available, but if it does, that would be nice.

Sorry for the late reply but glad to hear that you've found the issue and thank you so much for updating the README ❤
I have created an issue to change SDK providers or roll out my own solution, so I will be looking into that once I have some time: https://github.com/polpielladev/chatty-cli/issues/5

Closing this as this issue will be fixed as part of #5


TouchableHighlight vs TouchableWithoutFeedback

I noticed a huge issue with TouchableHighlight where there is a significant delay with the onPressIn method being called, making this component next to unusable for real-time feedback.
The TouchableWithoutFeedback component, however provided realtime feedback, but being that it doesn't conform to the same  component that TouchableHighlight does, styles break.

Has this been addressed? I'm experiencing the same delay, regardless of whether the TouchableHighlight component is in a ScrollView or just a View.
It's not the biggest deal, since an easy workaround is to just extend the component and override the delay:
class TouchableButton extends TouchableHighlight {
  touchableGetHighlightDelayMS() {
    return 0;
  }
}

However, I am wondering if there could be a better implementation. Currently, the stock TouchableHighlight component is noticeably unresponsive.


Bc compliance

what

BridgeCrew compliance checks fix
readme updated
code clean up
workflows updated

why

To be able to position our modules as standards compliant
Providing explicit note about policy attached directly to user
removed unnecessary providers dependencies
to stay in sync with all repos

references

https://docs.bridgecrew.io/docs/iam_16-iam-policy-privileges-1

/test all

/test all

/rebuild-readme

/rebuild-readme

/test all

/test all


problem importing DbsApi in 4.0.0

Hi @vkuznet  (I can't assing issue to you).
Maybe I am doing something wrong, but it seems that simply importing DbsApi makes it to be also executed, and hence fail[1] since no url is defined yet.  Is thi because I do not have DBS/PycurlClient/.../RestClient in the PYTHONPATH ? Where should I take it from ?
By the way, in a less clean/clear environment, i.e. adding tne DBS to PYTHONPATH before the "usual" stuff, the old RestCLient is found, but import of dbsClient fails same way [2]
I am not sure how to use this at this point, sorry.
belforte@lxplus751/~> cd /tmp/belforte/
belforte@lxplus751/belforte> git clone  https://github.com/dmwm/DBSClient
Cloning into 'DBSClient'...
git remote: Enumerating objects: 590, done.
remote: Counting objects: 100% (590/590), done.
remote: Compressing objects: 100% (355/355), done.
remote: Total 590 (delta 227), reused 485 (delta 170), pack-reused 0
Receiving objects: 100% (590/590), 437.20 KiB | 0 bytes/s, done.
Resolving deltas: 100% (227/227), done.
belforte@lxplus751/belforte> cd DBSClient/
belforte@lxplus751/DBSClient> git checkout 4.0.0
Note: checking out '4.0.0'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b new_branch_name

HEAD is now at 05ec989... Merge pull request #46 from vkuznet/main
belforte@lxplus751/DBSClient> cd src/python/
belforte@lxplus751/python> python
Python 2.7.5 (default, Nov 16 2020, 22:23:17)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-44)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from dbs.apis.dbsClient import DbsApi
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "dbs/apis/dbsClient.py", line 413
    print("The server output is not a valid json, most probably you have a typo in the url.\n%s.\n" % self.url, file=sys.stderr)
                                                                                ^
SyntaxError: invalid syntax
>>>

[2]
Python 2.7.13+ (default, Jan 22 2021, 17:43:09)
[GCC 6.3.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import RestClient
>>> RestClient.__file__
'/data/srv/TaskManager/v3.210920/slc7_amd64_gcc630/cms/dbs3-pycurl-client/3.16.0-comp/lib/python2.7/site-packages/RestClient/__init__.pyc'
>>> import dbs.apis.dbsClient
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/data/repos/DBSClient/src/python/dbs/apis/dbsClient.py", line 413
    print("The server output is not a valid json, most probably you have a typo in the url.\n%s.\n" % self.url, file=sys.stderr)

@belforte , two things:

the new client now is accessible from main branch, so you don't need to do chekout, plus 4.0.0 is a tag not a branch.
you should not use py2.7 and should start using py3

So, you should try the following:

get your py3 environment somehow (use CMSSW, conda, etc.)
either do git clone https://github.com/dmwm/DBSClient or download tar ball from pypi (https://files.pythonhosted.org/packages/2a/cd/afbd8393b7c932be89ec7c687aa69614055d0e7b6e0267669c77d8ac41a6/dbs3-client-4.0.0.tar.gz)
Setup PYTHONPATH to point to DBClient and its dependencies first, like

export DBS3_CLIENT_ROOT=/data/users/vk/dbs/DBS/DBSClient
export PYTHONPATH=$DBS3_CLIENT_ROOT/src/python:$DBS3_CLIENT_ROOT/dbs3-pycurl-3.17.4/src/python

then you can proceed

where do I get dbs3-pycurl from ?
GH seems to only have https://github.com/dmwm/DBS/tree/master/PycurlClient which is old Python2 version (import StringIO)


https://speech.microsoft.com/portal stuck on loading

Hello,
I tried different browser and 2 different Microsoft account but the https://speech.microsoft.com/portal is stuck and won't finish loading (spinner in loop).

Hi @webian, I'm Darren from the Speech Services team. Roughly where are you located in the world? Which browser are you using? I have not heard of any outage in MS Account or the speech portal, and it works for me now from west coast USA. Have you tried clearing your browser cash (deleting all data and cookies)?

Hi @dargilco,
I'm in Italy near Venice.
I tried to clear browser cache and also with a Chrome (v 90.0.4430.212) guest account so clean and no extension.
In the console see these errors: so I generated this log.
OS: Windows 10 Enterprise 20H2 19042.985

Thanks for the quick reply! I contacted the speech portal team and will follow up here when I have more information.

@webian I talk to Richard from the speech portal team and he said that unfortunately there is not enough details in the log you provided. He asked that you follow the instructions in the attached PDF and let us know the result. Thanks! Darren

@dargilco the root cause was that I was missing an Azure subscription even if I could login in Azure.
I then close this and thank you for your help!

@webian Glad to hear your issue was resolved. Good luck with your project!
Darren


[Event Request] Codeunit 5760 - Whse-PostReceipt - OnAfterModifySalesLine

Hi AL-Team,
can you please add the following Publisher?
  local procedure InitSourceDocumentLines(var WhseRcptLine: Record "Warehouse Receipt Line")
   var
       WhseRcptLine2: Record "Warehouse Receipt Line";
       TransLine: Record "Transfer Line";
       SalesLine: Record "Sales Line";
       PurchLine: Record "Purchase Line";
       ModifyLine: Boolean;
   begin
       WhseRcptLine2.Copy(WhseRcptLine);
       with WhseRcptLine2 do begin
           case "Source Type" of
               DATABASE::"Purchase Line":
                   begin
                       PurchLine.SetRange("Document Type", "Source Subtype");
                       PurchLine.SetRange("Document No.", "Source No.");
                       if PurchLine.Find('-') then
                           repeat
                               SetRange("Source Line No.", PurchLine."Line No.");
                               if FindFirst then begin
                                   OnAfterFindWhseRcptLineForPurchLine(WhseRcptLine2, PurchLine);
                                   if "Source Document" = "Source Document"::"Purchase Order" then begin
                                       ModifyLine := PurchLine."Qty. to Receive" <> "Qty. to Receive";
                                       if ModifyLine then
                                           PurchLine.Validate("Qty. to Receive", "Qty. to Receive")
                                   end else begin
                                       ModifyLine := PurchLine."Return Qty. to Ship" <> -"Qty. to Receive";
                                       if ModifyLine then
                                           PurchLine.Validate("Return Qty. to Ship", -"Qty. to Receive");
                                   end;
                                   if PurchLine."Bin Code" <> "Bin Code" then begin
                                       PurchLine."Bin Code" := "Bin Code";
                                       ModifyLine := true;
                                   end;
                               end else
                                   if "Source Document" = "Source Document"::"Purchase Order" then begin
                                       ModifyLine := PurchLine."Qty. to Receive" <> 0;
                                       if ModifyLine then
                                           PurchLine.Validate("Qty. to Receive", 0);
                                   end else begin
                                       ModifyLine := PurchLine."Return Qty. to Ship" <> 0;
                                       if ModifyLine then
                                           PurchLine.Validate("Return Qty. to Ship", 0);
                                   end;
                               OnBeforePurchLineModify(PurchLine, WhseRcptLine2, ModifyLine);
                               if ModifyLine then
                                   PurchLine.Modify;
                           until PurchLine.Next = 0;
                   end;
               DATABASE::"Sales Line": // Return Order
                   begin
                       SalesLine.SetRange("Document Type", "Source Subtype");
                       SalesLine.SetRange("Document No.", "Source No.");
                       if SalesLine.Find('-') then
                           repeat
                               SetRange("Source Line No.", SalesLine."Line No.");
                               if FindFirst then begin
                                   OnAfterFindWhseRcptLineForSalesLine(WhseRcptLine2, SalesLine);
                                   if "Source Document" = "Source Document"::"Sales Order" then begin
                                       ModifyLine := SalesLine."Qty. to Ship" <> -"Qty. to Receive";
                                       if ModifyLine then
                                           SalesLine.Validate("Qty. to Ship", -"Qty. to Receive");
                                   end else begin
                                       ModifyLine := SalesLine."Return Qty. to Receive" <> "Qty. to Receive";
                                       if ModifyLine then
                                           SalesLine.Validate("Return Qty. to Receive", "Qty. to Receive");
                                   end;
                                   if SalesLine."Bin Code" <> "Bin Code" then begin
                                       SalesLine."Bin Code" := "Bin Code";
                                       ModifyLine := true;
                                   end;
                                   // NEW >>>>
                                   OnAfterModifySalesLine(SalesLine, WhseRcptLine2, ModifyLine, PostInvoice)
                                   // NEW <<<<
                               end else
                                   if "Source Document" = "Source Document"::"Sales Order" then begin
                                       ModifyLine := SalesLine."Qty. to Ship" <> 0;
                                       if ModifyLine then
                                           SalesLine.Validate("Qty. to Ship", 0);
                                   end else begin
                                       ModifyLine := SalesLine."Return Qty. to Receive" <> 0;
                                       if ModifyLine then
                                           SalesLine.Validate("Return Qty. to Receive", 0);
                                   end;
                               OnBeforeSalesLineModify(SalesLine, WhseRcptLine2, ModifyLine);
                               if ModifyLine then
                                   SalesLine.Modify;
                           until SalesLine.Next = 0;
                   end;
               DATABASE::"Transfer Line":
                   begin
                       TransLine.SetRange("Document No.", "Source No.");
                       TransLine.SetRange("Derived From Line No.", 0);
                       if TransLine.Find('-') then
                           repeat
                               SetRange("Source Line No.", TransLine."Line No.");
                               if FindFirst then begin
                                   OnAfterFindWhseRcptLineForTransLine(WhseRcptLine2, TransLine);
                                   ModifyLine := TransLine."Qty. to Receive" <> "Qty. to Receive";
                                   if ModifyLine then
                                       TransLine.Validate("Qty. to Receive", "Qty. to Receive");
                                   if TransLine."Transfer-To Bin Code" <> "Bin Code" then begin
                                       TransLine."Transfer-To Bin Code" := "Bin Code";
                                       ModifyLine := true;
                                   end;
                               end else begin
                                   ModifyLine := TransLine."Qty. to Receive" <> 0;
                                   if ModifyLine then
                                       TransLine.Validate("Qty. to Receive", 0);
                               end;
                               OnBeforeTransLineModify(TransLine, WhseRcptLine2, ModifyLine);
                               if ModifyLine then
                                   TransLine.Modify;
                           until TransLine.Next = 0;
                   end;
               else
                   OnInitSourceDocumentLines(WhseRcptLine2);
           end;
           SetRange("Source Line No.");
       end;

       OnAfterInitSourceDocumentLines(WhseRcptLine2);
   end;

[IntegrationEvent(false, false)]
 local procedure OnAfterModifySalesLine(var SalesLine: Record "Sales Line"; WarehouseReceiptLine: Record "Warehouse Receipt Line"; ModifyLine: Boolean; PostInvoice: Boolean)
 begin
 end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Returning syntax error when creating a module in jupyter (Ubuntu 18.04, julia 0.7/1.0)

When trying to create a module in a jupyter notebook the following error message is returned:
syntax: "module" expression not at top level

Thanks, it looks like this is a bug in SoftGlobalScope.jl.     A temporary workaround is to set IJulia.SOFTSCOPE[] = false

Closed by https://github.com/stevengj/SoftGlobalScope.jl/pull/5


Don't test values that are on/near the boundary of an array's dtype

This PR should prevent Hypothesis testing boundary numbers that are unrepresentable for libraries like JAX @jakevdp @rgommers. There might be a performance gain but not sure—we know vectorisation would be the biggest win anywho.
I'll sleep on it but I think its probably fine to just get this in, and a TODO would be to make this optional behaviour and/or check the library we're testing on runtime.
Just to say, this is achieved by setting the min/max value arguments in Hypothesis strategies. Historically there were issue where setting boundaries caused undesirable example generation (https://github.com/HypothesisWorks/hypothesis/issues/2907), but that should be all sorted :thinking:
Relevant discussion from https://github.com/data-apis/array-api-tests/pull/206#pullrequestreview-1731765945

Although I will say it has generally been hard for us to keep up with all the various CI failures in this repo, mostly due to the nature of the test suite that there are expected failures.

I think we should try a little harder to prevent Hypothesis from using floats like 1.5e+308 or ints like 2**63. It results in a lot of spurious failures that are costing us a lot of time to deal with and make using the test suite harder, but are of little interest. Isn't there something like a "use only reasonable ranges" setting?

Yeah we do try avoiding spurious failures, but its been on an adhoc basis when issues have come up with adopters. A blanket "use only reasonable ranges" of Hypothesis would require some re-thinking on how the test suite is architected 🤔
Like 1.5e+308 shouldn't ever be a problem for manipulation functions, so for me it is pretty nice to make sure that is actually the case—on the other hand, yeah we don't really care if some element-wise functions don't play nice with massive floats. And well "nice to have" testing isn't also that important if it shouldn't matter anyway, so I defo see your point.

and

It's very unlikely to fail though, and very unlikely to be real-world relevant. So there's 2 reasons that it's basically a dont-care. I encourage you to think about something like limiting the default range to something like 1e20 for float64, 1e8 for float32, etc. This should be easy to do I'd expect, and a significant improvement in making the test suite more usable/robust, and hence more useful for folks implementing support in their array library.

Seems all good so YOLO

Note to self: remember to set JAX_ENABLE_X64 as 1 when testing JAX!!


ElasticSearch 5.6 for Uwazi v1.4+

See https://github.com/huridocs/uwazi/commit/ab0d5300b4bf300a7460cd6df6f94fdeffaf91c4
Impact on https://github.com/fititnt/uwazi-docker/issues/40

This issue is both for comment Elastic Search 5.6 is a new change (at least on README) and some potential bugs that could happens.

Closing old issues. Using newer ElasticSearch


Cannot resolve symbol 'useNavigate'

when I write import { useNavigate } from '@reach/router'; the webstorm gives me some kind of warning like this.

although this hook works as I expect, I want to know what may cause this issue

"@reach/router": "^1.3.3",
"react": "^16.9.0",
"react-dom": "^16.9.0",
node version is v10.16.3

Ahh, I think just the types are missing. My TS project is failing to compile with Module '".../node_modules/@types/reach__router"' has no exported member 'useNavigate'. If I wasn't just a newbie at TS, I'd try to put up a PR to fix myself!

try with "react": "^17.0.1"

I'm using react 17.0.2 and and react-router-dom "^6.2.1" that warning persists.

I'm using react 18.1.0 and react-router-dom "6.3.0" that warning persists.

Shouldn't it be
import {useNavigate} from "react-router-dom";
Also, I'm getting the same error, however, from the above code.

i also using  "react-router-dom": "^6.3.0", and latest react i tried different versions to

hi，

Any update on this, I'm using:
"react": "^17.0.2",
"react-dom": "^17.0.2",
"react-router-dom": "^6.3.0"
node version: 18.4.0
Still receiving the same warning

use intellij? update tool. I'm used <IntelliJ IDEA 2023.1.2>


Problem with update of server selfhost and change of signal port

Hi, some days ago I installed selfhost server following https://docs.netbird.io/selfhosted/selfhosted-guide
seems done correctly and I also changed coturn range adding "TURN_MAX_PORT=50999" to setup.env and seems was ok
today I must change signal port from 10000 to 11000 and I also tried to update to latest version (from 0.23.0 to 0.23.1)
I not found an howto related to update and I know few things about docker, I tried to update repo, add "NETBIRD_SIGNAL_PORT=11000" to previous setup.env and redone container. netbird seems working and data was maintained but signal port was not changed even if after configure in management.json seems wrote correctly 11000
the exact procedure I did was:
# disable and remove container (FWIK)
docker-compose down
# save setup.env
cp setup.env /root/
# clean repository (I'm not sure if needed and correct)
cd ..
git reset --hard
git clean -f -d -X
# update repository to latest version
git fetch origin
LATEST_TAG=$(basename $(curl -fs -o/dev/null -w %{redirect_url} https://github.com/netbirdio/netbird/releases/latest))
git fetch --tags
git checkout tags/$LATEST_TAG
cd infrastructure_files
# recopy setup file
cp /root/setup.env .
# checked if there was changes to parameters from previous installed version: https://github.com/netbirdio/netbird/commits/v0.23.1/infrastructure_files/setup.env.example
# did configure
./configure.sh
# here I checked management.json and seems signal port is correctly 11000
# generate and start container
docker-compose up -d

now checking netbird is working and data mantained but signal port is still 10000 looking "docker ps"
docker ps
CONTAINER ID   IMAGE                          COMMAND                  CREATED          STATUS          PORTS                                                                      NAMES
57e4b8c384a5   netbirdio/management:latest    "/go/bin/netbird-mgmâ¦"   16 minutes ago   Up 16 minutes   <IP_ADDRESS>:33073->443/tcp, :::33073->443/tcp                                  infrastructure_files_management_1
ee85cc11a930   wiretrustee/dashboard:latest   "/usr/bin/supervisorâ¦"   16 minutes ago   Up 16 minutes   <IP_ADDRESS>:80->80/tcp, :::80->80/tcp, <IP_ADDRESS>:443->443/tcp, :::443->443/tcp   infrastructure_files_dashboard_1
6014132672a5   coturn/coturn                  "docker-entrypoint.sâ¦"   16 minutes ago   Up 16 minutes                                                                              infrastructure_files_coturn_1
dcbc9bb3861a   netbirdio/signal:latest        "/go/bin/netbird-sigâ¦"   16 minutes ago   Up 16 minutes   <IP_ADDRESS>:10000->80/tcp, :::10000->80/tcp                                    infrastructure_files_signal_1

can someone please tell me if the update procedure I did is correct or wrong? if wrong can tell me the correct (and I think is good add to documentation)
about signal port not changed to 11000 in the signal container is related to a bug or something I did wrong?
thanks for any reply and sorry for my bad english

I did a fast compare with implementation of variable of other custom ports working and I found a missed variable in docker-compose.yml.tmpl, I tried to change manually the static port (10000) with $NETBIRD_SIGNAL_PORT and worked
can someone confirm no other things is needed and can someone please tell me if the update procedure I did is correct?

Dear @Fantu
First of all your English is not bad. I understand fully your report :)
The procedure what you did is looks correct at the first look. It was good idea to check the port in the management.json but it is not enough as you recognized. The management.json contains relevant information for the Management server. That server propagate the port information for the peers. The signaling server is a different component and you can manipulate the settings with env variables.
Just to be sure everything is correct, could you share with me your docker-compose.yml file?

@pappz thanks for reply
about the update netbird server to newer version the procedure I did can you please tell me if is correct?

@pappz thanks for reply
I used and know few docker, I saw only after from docker-compose.yml that don't generate the container but download them, so I missed important part of pull the new version of container if present, thanks for the info
about the repository is needed in case of changes in infrastructure_files from what I understand and setup.env should be updated in case of changes from previous version or it risks to cause issue or unexpected cases I think, or I'm wrong?
"docker-compose down" I saw that stop and remove the containers
about the store.json I don't understand full without know how works (from a grep I didn't find operation in infrastructure_files that use it, I suppose manual operations are needed to use/restore it in some cases where needed), I'll do that anyway
if there will be changes to setup.env also ./configure.sh will be needed from what I understand
I suggest to add the full update procedure in the documentation

Yes, I have also discovered this problem and found a solution.
Please first confirm that the port range has been modified correctly in turnserver.com, as follows:
Min port=12000
Max port=13000

Then modify the file mapping directly in the docker compose file:
  coturn:
    image: coturn/coturn
    restart: unless-stopped
    domainname: api.dongjianwei.asia
    volumes:
      - ./turnserver.conf:/etc/coturn/turnserver.conf
    network_mode: host

@13653216371 you mean there is issue also on coturn port range? It seemed to work to me
I setted TURN_MAX_PORT=50999 in setup.env and the generated turnserver.conf have correctly max-port=50999
about docker compose file with network mode host (on coturn container) the port mapping like others containers is not needed and use the host's network directly, so not other things are needed or am I wrong?

There is a problem with the docker-composite. yml file generated using the configure.sh script. Please use the format I wrote above to call turnserver.conf

There are some issues with the configure.sh script. It is recommended to directly modify the conf files of Docker-Compose.yml and other components

the my generated docker compose configuration of coturn container is:
  # Coturn
  coturn:
    image: coturn/coturn
    restart: unless-stopped
    domainname: hidden.domain
    volumes:
      - ./turnserver.conf:/etc/turnserver.conf:ro
    #      - ./privkey.pem:/etc/coturn/private/privkey.pem:ro
    #      - ./cert.pem:/etc/coturn/certs/cert.pem:ro
    network_mode: host
    command:
      - -c /etc/turnserver.conf

the differences I see is remove of ":ro" in the configuration file but I suppose is ok readonly for it and remove of command
looking "docker-compose logs coturn" output coturn seems running correctly
and also enter in container shell and checking its conf:
docker exec -it 8fead6159f65 sh
$ cat /etc/turnserver.conf | grep max-port
max-port=50999

for now I don't see the issue

big thanks @mlsmaycon and @braginin for backup and upgrade informations added to docs in https://github.com/netbirdio/docs/commit/53da0fe16fdd5111d85f54f50c880bc39d38f182
there are still 2 information which I haven't found an answer to and which I think is useful to have in the documentation:

if I move the netbird server from one vm in office to server in cloud (or other cases with new clean install) is possible to the database from backup done with docker compose cp -a management:/var/lib/netbird/ backup/ for restore the users, peers, acl etc.. data (the domain and ports will be the same and only change the record to point to new server)
is possible to change idp (for example from google to zitadel) keeping that netbird data (users, peers, acl etc..)?

Missed things added in this suggestion list, so I think this can be closed


docs: Fix broken imports in documentation

Found via script in https://github.com/langchain-ai/langchain/pull/19611
Only ones remaining are:
File: docs/docs/integrations/tools/google_drive.ipynb
    langchain_community.tools.google_drive.tool.GoogleDriveSearchTool
    langchain_community.utilities.google_drive.GoogleDriveAPIWrapper

But I can't find these objects in the Langchain repo or in https://github.com/langchain-ai/langchain-google/ (cc @efriis)

those google drive imports are supposed to be
langchain_googldrive.tools.google_drive.tool.GoogleDriveSearchTool
langchain_googledrive.utilities.google_drive.GoogleDriveAPIWrapper

but i think at some point we messed them up. see #9999 for original notebook


Make the Counter page increment by 3 instead of 1.

Currently, the Counter page increments by 1 when you click the button. I want it to increment it by 3.

Fixed


Add how-to address issue with new Ruby release

@huwd thanks for the comments :-). In answer to your question about when someone will know this is fixed, it's as simple as "you won't get the error anymore".

Blimey - my PR is getting a lot of love 😃.

After discussing this offline with @huwd, I think we're good to merge...


Add package properties to tv

fixes #269 (again)

tests are implemented now, too, so this should be ready for another review round :)


Auto update common-files when build-tools is built

Its a bit painful to do this manually

Add it as a prow job?

We would either have to wait some period of time, say 45 mins, and then look for an image that was updated in the last 45 mins (and hope it was caused by the same PR that tripped the job). If multiple changes went in near each other, as long as the image build wasn't slow and missed, both automator jobs might pick up the same one, but that might not be too bad. I have some code in my release-builder changes which looks for the last image for a given release so that could be used.

can't we do it in the actual job that builds the image as well? do we don't
need a wait?
On Tue, Nov 3, 2020, 6:15 AM Eric Van Norman<EMAIL_ADDRESS>wrote:

We would either have to wait some period of time, say 45 mins, and then
look for an image that was updated in the last 45 mins (and hope it was
caused by the same PR that tripped the job). If multiple changes went in
near each other, as long as the image build wasn't slow and missed, both
automator jobs might pick up the same one, but that might not be too bad. I
have some code in my release-builder changes which looks for the last image
for a given release so that could be used.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/28436#issuecomment-721141713, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXK2SMNQQZYYLEGVKRDSOAGBXANCNFSM4TFJ2YEA
.

This is a duplicate of https://github.com/istio/test-infra/issues/2469.


Enhance docs to explain differences between fetch from image vs. fetch from imgpkgBundle

The docs on App CR High Level Overview shows all the places where configurations can be fetched from. Enhance the docs to explain what is the differences similarities between fetching from fetch from an image vs. fetch from imgpkgBundle
If the overview section is the wrong place to add this info, then maybe consider adding a page on fetching strategies  with a discussion of the pros/cons of each approach to fetching.

Moving to carvel repo for doc enhancements


Update Pinterest with more regex as not detecting many combinations and correct app and bot sections (currently mixed up)

See github issue: https://github.com/WhichBrowser/Parser-PHP/issues/505

User agent
Version
OS
Hardware Type
Popularity

Pinterest/0.2 (+http://www.pinterest.com/)
0.2
--
Server
Very common

Mozilla/5.0 (compatible; Pinterestbot/1.0; +http://www.pinterest.com/bot.html)

--
Server
Very common

Pinterest/0.2 (+http://www.pinterest.com/bot.html)
0.2
--
Server
Common

Mozilla/5.0 (Linux; Android 6.0.1; Nexus 5X Build/MMB29P) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/41.0.2272.96 Mobile Safari/537.36 (compatible; Pinterestbot/1.0; +http://www.pinterest.com/bot.html)

--
Server
Average

Pinterest/0.1 +http://pinterest.com/
0.1
--
Server
Average

Pinterest/0.2 (http://www.pinterest.com/)
0.2
--
Server
Uncommon

Mozilla/5.0 (compatible; Pinterestbot/1.0; http://www.pinterest.com/bot.html)

--
Server
Uncommon

Pinterest for Android/6.46.4 (g2; 4.4.2)

--
Server
Uncommon

Pinterest/4.5 CFNetwork/711.3.18 Darwin/14.0.0
4.5
--
Server
Uncommon

Pinterest for iOS/6.71 (iPhone9,3; 11.4.1)

--
Server
Uncommon

Pinterest for Android/6.37.0 (j7xelte; 6.0.1)

--
Server
Uncommon

Pinterest for Android Tablet/7.7.7 (SGP511; 4.4.2) 1

--
Mobile - Tablet
Uncommon

Extracting the above data into this repo gives the following:
App section
Pinterest for Android/6.46.4 (g2; 4.4.2)

Pinterest for iOS/6.71 (iPhone9,3; 11.4.1)

Pinterest for Android/6.37.0 (j7xelte; 6.0.1)

Pinterest for Android Tablet/7.7.7 (SGP511; 4.4.2) 1

Bot section
Old UA:
Pinterest/0.2 (+http://www.pinterest.com/)
Pinterest/0.1 +http://pinterest.com/
Pinterest/0.2 (http://www.pinterest.com/)
Pinterest/4.5 CFNetwork/711.3.18 Darwin/14.0.0

New UA:
Pinterest/0.2 (+https://www.pinterest.com/bot.html)

Mozilla/5.0 (Linux; Android 6.0.1; Nexus 5X Build/MMB29P) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/41.0.2272.96 Mobile Safari/537.36 (compatible; Pinterestbot/1.0; +https://www.pinterest.com/bot.html)

Mozilla/5.0 (compatible; Pinterestbot/1.0; http://www.pinterest.com/bot.html)

Link: https://help.pinterest.com/en/business/article/pinterest-crawler

Coverage increased (+0.02%) to 99.97% when pulling cd6e580d602f5461d61e0c1fe57953baf6bd5732 on ayumi-cloud:Pinterest-Bot into 880b9fa797401d14b28956442944c3daa70240ff on WhichBrowser:master.


Multiple fromIds for same model

Thanks for everyone's hard work!
There appears to be an issue where the Prisma 1 datamodel associates two fields with the same node or inherits from itself, e.g.
Prisma 1 model
type DealAddOn {
  id: ID! @id
  createdAt: DateTime! @createdAt
  updatedAt: DateTime! @updatedAt
  persistOnDeal: Boolean @default(value: false)
  dealership: Dealership @relation(link: INLINE)
  onePerDeal: Boolean @default(value: false)
  label: String!
  docs: [Doc]
  customerComponent: String
  description: String
  addToDeal: Boolean @default(value: false)
  isAccepted: Boolean @default(value: false)
  isAcceptedDate: DateTime
  lineItems: [DealLineItem] @relation(name: "DealAddOnLineItems")
  isRequired: Boolean @default(value: false)
  store: Store @relation(link: INLINE)
  inheritsFrom: DealAddOn @relation(link: INLINE)
  isDefault: Boolean @default(value: false)
  deal: Deal @relation(link: INLINE, name: "DealAddOns")
  actions: [DealAction] @relation(name: "DealAddOnActions")
  vehicle: Vehicle @relation(link: INLINE, name: "AddOnVehicle")
}

npx prisma introspect
Result
...
model DealAddOn {
  id                String         @default(cuid()) @id
  createdAt         DateTime
  updatedAt         DateTime
  persistOnDeal     Boolean?
  onePerDeal        Boolean?
  label             String
  customerComponent String?
  description       String?
  addToDeal         Boolean?
  isAccepted        Boolean?
  isAcceptedDate    DateTime?
  isRequired        Boolean?
  isDefault         Boolean?
  store             String?
  dealership        String?
  inheritsFrom      String?
  deal              String?
  vehicle           String?
  Deal              Deal?          @relation(fields: [deal], references: [id])
  Dealership        Dealership?    @relation(fields: [dealership], references: [id])
  DealAddOn         DealAddOn?     @relation("DealAddOnToDealAddOn_inheritsFrom", fields: [inheritsFrom], references: [id])
  Store             Store?         @relation(fields: [store], references: [id])
  Vehicle           Vehicle?       @relation(fields: [vehicle], references: [id])
  DealAction        DealAction[]
  other_DealAddOn   DealAddOn[]    @relation("DealAddOnToDealAddOn_inheritsFrom")
  DealLineItem      DealLineItem[] @relation("DealAddOnLineItems")
  Doc               Doc[]
}
...

npx prisma-upgrade
model DealAddOn {
  id                String         @default(cuid()) @id
  createdAt         DateTime       @default(now())
  updatedAt         DateTime       @updatedAt
  persistOnDeal     Boolean?
  onePerDeal        Boolean?
  label             String
  customerComponent String?
  description       String?
  addToDeal         Boolean?
  isAccepted        Boolean?
  isAcceptedDate    DateTime?
  isRequired        Boolean?
  isDefault         Boolean?
  storeId           String?        @map("store")
  dealershipId      String?        @map("dealership")
  inheritsFromId    String?        @map("inheritsFrom")
  dealId            String?        @map("deal")
  vehicleId         String?        @map("vehicle")
  deal              Deal?          @relation(fields: [deal], references: [id])
  dealership        Dealership?    @relation(fields: [dealership], references: [id])
  inheritsFromId    DealAddOn?     @map("inheritsFrom") @relation("DealAddOnToDealAddOn_inheritsFrom", fields: [inheritsFrom], references: [id])
  store             Store?         @relation(fields: [store], references: [id])
  vehicle           Vehicle?       @relation(fields: [vehicle], references: [id])
  actions           DealAction[]
  inheritsFrom      DealAddOn[]    @relation("DealAddOnToDealAddOn_inheritsFrom")
  lineItems         DealLineItem[] @relation("DealAddOnLineItems")
  docs              Doc[]
}

Note the two inheritsFromId's

Hi @LawJolla, yes unfortunately this can happen in certain cases. Please read more here: https://github.com/prisma/upgrade/pull/63#issue-467388244
It should be straightforward to manually fix by renaming inheritsFromId to deal, etc.


LNK1112

Getting this error "fatal error LNK1112: module machine type 'x64' conflicts with target machine type 'x86'" when running the command "powershell -command "& .\build_dxvk.ps1" Searched for hours for a solutionfor a way to change the target but cant find anything that works. Help :)

@sbartuk thanks for the feedback, can you attach a full build log please so we can take a look and see what's going on?

Closing due to no activity.


Update CONTRIBUTORS.md and fix TYPO

Motivation
I participated in the code4rena audit contest
And fix a little typo in README.md

Solution
README.md:
- You may wish to include a `.env` file that `export`s a specific profile when developing locally.
+ You may wish to include a `.env` file that `exports` a specific profile when developing locally.

Move to #380

Closing this for now as I don't think we've substantially improved on documentation; feel free to reopen with an improvement!


Dual master node context causing old Mac and Windows artifacts to remain

It seems with the 2 master nodes "Jenkins" and "C3jenkins", we are getting previous build artifacts for Windows and Mac, eg:
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk/job/jdk-windows-x64-temurin/93/artifact/workspace/target/
  | OpenJDK-jdk_x64_windows_hotspot_2022-05-28-03-30.msi | May 31, 2022, 5:56:10 PM | 166.44 MB | view
  | OpenJDK-jdk_x64_windows_hotspot_2022-05-28-03-30.msi.json | May 31, 2022, 6:01:13 PM | 27.52 KB | view
  | OpenJDK-jdk_x64_windows_hotspot_2022-05-28-03-30.msi.sha256.txt | May 31, 2022, 6:01:09 PM | 136 B | view
  | OpenJDK-jdk_x64_windows_hotspot_2022-05-28-03-30.zip | May 28, 2022, 7:20:55 AM | 186.78 MB | view
  | OpenJDK-jdk_x64_windows_hotspot_2022-05-28-03-30.zip.json | May 31, 2022, 5:54:46 PM | 27.52 KB | view
  | OpenJDK-jdk_x64_windows_hotspot_2022-05-28-03-30.zip.sha256.txt | May 28, 2022, 7:29:06 AM | 136 B | view
  | OpenJDK-jdk_x64_windows_hotspot_2022-05-31-15-24.msi | May 31, 2022, 5:56:18 PM | 166.44 MB | view
  | OpenJDK-jdk_x64_windows_hotspot_2022-05-31-15-24.msi.json | May 31, 2022, 6:01:16 PM | 27.52 KB | view
  | OpenJDK-jdk_x64_windows_hotspot_2022-05-31-15-24.msi.sha256.txt | May 31, 2022, 6:01:10 PM | 136 B | view
  | OpenJDK-jdk_x64_windows_hotspot_2022-05-31-15-24.zip | May 31, 2022, 5:34:29 PM | 186.79 MB | view
  | OpenJDK-jdk_x64_windows_hotspot_2022-05-31-15-24.zip.json | May 31, 2022, 6:01:12 PM | 27.52 KB | view
  | OpenJDK-jdk_x64_windows_hotspot_2022-05-31-15-24.zip.sha256.txt | May 31, 2022, 5:42:46 PM | 136 B | view

I think this is because previously with just one Jenkins master the "sign_build" stage cleared all of the workspace/target out, prior to the create_installer_windows|mac stages, but now if those run on the other master they are not removed...

Do you know where that clearup is occurring? Sounds like it's potentially occurring on a machine that isn't the one the pipeline is using...
I've dropped the number of executors on the jenkins controller to zero which should stop anything targetted for master (Other than the lightweight jobs that don't use an executor from running on it any more and will re-run the pipeline and see what happens...

@sxa I believe the problem is that there is no clearup happening for buildMac/WindowsInstaller stage that occurs on the builtin|master context

@andrew-m-leonard I've re-enabled the executor on the build-in node (Note that it is NOT labelled as master)
If you need to fiddle around with which of the two have master feel free but I'd recommend logging changes in here as you do it :-)


Add git-pr-backport-at-once command

This command list and cherry-picks PR merge commits in origin/HEAD (usually it is master branch) to current branch.
It opens TODO list without already picked PRs at beginning, like git rebase -i.

Great work! Thank you your contribution!!11!


#11431 Remove load balance access task

As stated on a comment, I propose to simply remove this task.

@pyaillet could you please provide a link to the comment you mentioned?

Sorry !
Here it is: https://github.com/kubernetes/website/issues/11431#issuecomment-443305900

/cc @Bradamant3 @jimangel

See issue #11431 generally. Links provided there for audit going forward (just in case ...)
/lgtm
/approve
Also thanks @pyaillet for your diligence and persistence on this one!


feat: Native HTTP port

closes #27

Hey @fdaciuk,
I think this is good to go, all API tests are passing now.
Maybe we can review this live? It's up to you. I do think that this implementation has space for improvements. 😄

Great idea! Let's review this tomorrow morning at stream! 👯 👯 ✨


Mulligan Indicators Issues (Secondary Monitor)

Hello, Love this program.  I play HS on my second monitor and the new mulligan numbers don't move up to the top of my screen and when I scroll over one of them, the card stays up until I go over to the tracker and mouse over another card (which also stays up now).  Attaching screenshot

Hi, I'm closing this one, both bugs are already known (see #399 and #363)


tools: In ensure_coreutils, add another place to look for Homebrew

It might be found at this path on Macs with Apple Silicon.
Discussion:
https://chat.zulip.org/#narrow/stream/243-mobile-team/topic/Xcode.20on.20new.20laptop/near/1937894

Thanks! Looks good; merging.


Populate BuildPackage.bat with comments

Provide manual on path creation, dlls and documentation copy, batch execution and package upload.

Aggregated with issue https://github.com/iivchenko/ShogunLib/issues/17


twister: coverage: Use GCOVR as default reporting tool

Use GCOVR by default as the more reliable code coverage reporting tool instead of LCOV.
see https://github.com/zephyrproject-rtos/zephyr/pull/66654#issuecomment-1864455941
Additional checks for Twister command line options --coverage-tool and --coverage-formats.

though things will be covered in a different PR, for completeness and depending on which goes first, you need to change .github/workflows/codecov.yaml to deal with a new default tool, i.e. by making sure the tool exists in the environment (pip install gcovr...).

@nashif, m.b. its better to just DNM this PR until #66654 merge ? otherwise too many workflow changes needs to be duplicated here whereas this PR doesn't block it.

removing DNM as #66654 is merged


Does not make main lisp file's name as project name

First, thanks for quickproject! I'm a newbie to CL and it's helpful to generate a skeleton with one command.
What I feel like a bug is that the system.asd file that quickproject generates seems quickproject should generate package.lisp and (#| TMPL_VAR name |#).lisp. However quickproject generates package.lisp and application.lisp, which makes asdf signal an error. Also the comment in application.lisp is ;;;; (#| TMPL_VAR name |#).lisp. I always rename application.lisp to the project name.
Is this expected behavior? I find it little strange to manually change a filename generated by a skeleton.

Are you using CCL? I found this was true for me also, using CCL on linux. I patched the template name generator to pattern match differently
https://github.com/cmstrickland/quickproject/tree/cms/pathname-match-fix-ccl

@cmstrickland Yes, I use CCL. You should make a PR for the patch you made... thanks! :-)
Sent with GitHawk

I made a PR from my changes. I'm sure there's probably a neater way to fix it - #23


CB-12001 Edit network on Gcloud summary does not working. Fixing the …

…id vs name problem on Google.
See detailed description in the commit message.

writing the UT


Docs and blog on Portaljs site

Quick definitions:

Docs - index page where we can consolidate links to individual doc about specific feature. A doc is a blog post.
Blog - list of posts that have published date, author etc. See https://flowershow.app/blog

Acceptance criteria

[x] I can see list of blog posts at https://portaljs.org/blog which is similar to flowershow example. ✅ 2023-04-20 FIXED. See preview: https://site-portaljs-git-feature-blog-datopian1.vercel.app/blog
[x] I have my current guides (each README from each example) added as a blog post. I.e., each example becomes a blog post, see how it's done in https://flowershow.app/blog where a post from /docs/ can be also listed in the blog. Relates to #747 ✅ 2023-04-20 FIXED. See preview: https://site-portaljs-git-feature-blog-datopian1.vercel.app/blog
[x] I have links to those blog posts from the /docs page, see https://flowershow.app/docs (tutorials and guides section). ✅ 2023-04-20 FIXED. See preview https://site-portaljs-git-feature-blog-datopian1.vercel.app/docs#examples

I have my current guides (each README from each example) added as a blog post. I.e., each example becomes a blog post.

@anuveyatsu im a little bit confused here, @rufuspollock had two acceptance criteria in the old discussion>

Get this in examples and have auto-deploy and a README that has screenshot, link to demo, how to use yourself (npx create-next-app path, plus some small info about how it works (e.g. how to configure what is used)

write a super simple blog post about this for portaljs blog! (KISS: motivation, screenshot, link to example, demo link etc)

This made me understand that the blog post and the README would be different things(even tough they seem to require the same information), am i mistaken?

@luccasmmg if you look here - https://flowershow.app/blog - some posts are from /blog/ while others are from /docs/. So we could have comprehensive README in the docs and a super simple post in the /blog/. I think main point is to consolidate READMEs (which are actually docs) into the single place (which is going to be our site).
I've just re-read the acceptance criteria which might be a little bit confusing so I'm changing it a bit. cc @demenech

@luccasmmg if you look here - https://flowershow.app/blog - some posts are from /blog/ while others are from /docs/. So we could have comprehensive README in the docs and a super simple post in the /blog/. I think main point is to consolidate READMEs (which are actually docs) into the single place (which is going to be our site).
I've just re-read the acceptance criteria which might be a little bit confusing so I'm changing it a bit. cc @demenech

Thank you, this was very clarifying.

Updates 2023-04-20
PR: https://github.com/datopian/portaljs/pull/779
Changes

Implemented /blog

Also added the people folder to get the authors
Added the two blog posts about portal examples that @luccasmmg wrote, with links to the respective docs pages
Added blog link to the navbar

Docs

Cleaned up old content
Added the two examples docs

Note: There are 4 examples on the examples folder, but as discussed with Luccas it's unlikely that the ckan-2021 and the frictionless dataset one should be included

Created the tutorials and guide section listing the examples


Setup project

TODO:

[x] Monorepo
[x] Linters
[x] Docs
[ ] CI/CD

Pending CI/CD but you can start using the repository


Added max_item_size to Memcached client

[x] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
While running the memcached index cache in Cortex we've find out that whenever the item size in larger than the max size allowed by memcached (defaults to 1MB, can increase with the -I option) the client will waste resources doing a full round-trip to memcached just to see the Set() operation to fail because the item size is larger than the limit.
In this PR I'm proposing to add a new option to memcached client config to allow to set the max_item_size so that we skip the set operation at all if the item size is greater.
Verification
Unit tests.

Thanks @povilasv and @pstibrany for your review. I should have addressed your comments. May you take a look, please?

Thanks @GiedriusS for your prompt review. I should have addressed the comments.

Don't worry about e2e now, we have flaky tests currently. Fixing now: https://github.com/thanos-io/thanos/pull/2313

Thanks ! :muscle:

Thanks !

Thanks to you!


type narrowing value on "deferred" let assignment does not work when equivalent non-"deferred" assignment does

TypeScript Version:  3.6.3

Search Terms:  let inference inferred union type guard narrow defer deferred
Code
interface Success {
    success: true;
    response: object;
}

interface Error {
    success: false;
    error: object;
}

function request(): Success | Error {
    return null as any;
}

// This does not work:
let r;
r = request();

// This does:
// let r = request();

// The type information give on hover for `r` here is the same in both cases: `Success | Error`.
r;

if (r.success) {
    r.response;
}

Expected behavior: r.response is accessible within the block guarded by r.success.
Actual behavior: r is not type-narrowed, so r.response is not accessible. Changing the declaration to a single line resolves the issue.
Playground Link: https://www.typescriptlang.org/play/index.html#code/JYOwLgpgTgZghgYwgAgMoFcFIM7eQbwChkTltMdsAuZMKdCAbmNKgmwAcB7EbCGrgCMAVhARhmAX0KFQkWIhQBRKFC5QCLEuSztqyeABs+zUsmhqoAkWImFphGOhDjgPZGwCODbGAAUAJQ0GLq4yAA+yCqWmmZsYOhQIMgg6IaGyHB4cCAAnlIyAPSFyAAqABbAeAAmXOwpXGDIAO7qANZUhIYQTVDMGgC8HhDe7P4BzITFZZU1ddRTJd29yENePuOT0xUoYLkcKKAw6gC2cGBuyQDmwABuKO7lXPcaxxoABlDvyOXQh3hgX5kOAnQ7JQSNcrIBBZdg0d4hSgRKKqdTvAB0hD6MmAMGQfig6J0lACsVY6LYnB4JnsQA
Related Issues:

Duplicate #30557

Confirmed as a duplicate: by explicitly annotating the "deferred" declaration
let r: Success | Error;

the type narrowing now works correctly.


Roll engine bc6df5bc34dc..068d6f8a6351 (3 commits)

https://github.com/flutter/engine/compare/bc6df5bc34dc...068d6f8a6351
git log bc6df5bc34dc..068d6f8a6351 --no-merges --oneline
068d6f8a6 Roll src/third_party/skia f40ddfabb7f0..a38ab3b094ba (2 commits) (flutter/engine#6484)
7afbcd974 Add a ratio cap to decoded animated image frames (flutter/engine#6310)
f8d87777a Roll src/third_party/skia 42137de2b2b8..f40ddfabb7f0 (7 commits) (flutter/engine#6482)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

@rmistry @liyuqian : New roll has been created by https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

PullRequest is not longer mergeable. Closing it.

PullRequest is not longer mergeable. Closing it.

Commit queue failed; closing this roll.

PullRequest is not longer mergeable. Closing it.


getaccountaddress No longer compatible with Bitcoin/OpenTrade

After upgrading the Bitcoin wallet  I have noticed that OpenTrade is no longer able to generate a wallet address for Bitcoin only. Other coins are ok.
Further investigation shows that the 'getaccountaddress' has been removed from the Bitcoin wallet and replaced with getnewaddress?
https://github.com/bitcoin-core/bitcoin-devwiki/wiki/0.17.0-Release-notes
Looks like 'move' has been removed too, I think OpenTrade uses move?
I urgently need a fix for this as I am unable to use a previous Bitcoin version as my wallet has been upgraded.
Does anyone have a fix for this?

For what it's worth, you can always revert to a previous bitcoin version.

dump your wallet with the dumpwallet  command
downgrade to previous version, remove wallet.dat (backups etc!)
import the dumped wallet with the importwallet  command
remove the dump filename, which is dumped in clear text and unprotected by any password.

Hope it helps.

if the update 3s3s up command rpc 0.17.0 and still still old command, easy easy add more ethereum and monero, equivalent :)

0.17 can still be used, those rpc calls will be removed from 0.18
see: 'label' and 'account' APIs for wallet
TLDR; start bitcoind with '-deprecatedrpc=accounts'

You can edit your bitcoin.conf and insert deprecatedrpc=accounts
Then you don't have to think about it in case you restart your wallet.

@arvoim @Puual I think you will find although you add deprecatedrpc=accounts it will still fail. But have a go yourself, don't say I didn't warn you.
I beleive the 'move' function was removed, this is something that opentrade heavily relies on.

@daygle
That shocked me for a moment. Tested it with Bitcoin on testnet.

Deposit to user1
exchanged to user2
withdraw from user2 back to the faucet
withdraw from user1 to deposit it at user2

Everything worked as it should be
Might there be a thing that should also be tested?
But I agree, this is something serious. Everyone should test this for himself


RepLoad issue with MiXCR files

Hello,
I just started using Immunarch today but when trying to parse a folder with different .txt clonoset files that I obtained using mixcr it gives me the following error

TCRAs <- repLoad("Users/abolivar1/Documents/TCRseq Project/Bioinformatics/TCRAs/", .format = "mixcr")
== Step 1/3: loading repertoire files... ==
Error in if (file.info(path)$isdir) { :
missing value where TRUE/FALSE needed

I am not sure what would be wrong with the files, as I am using them straight from Mixcr.
Any help would be appreciated
Thanks,
Ana

Hi!
I'm experiencing the same problem. I have no problems loading the data that comes with the package, but can't load my datasets, clonotype files obtained with mixcr:
`> immdata <- repLoad("../data/sample1_folder/sample1.clonotypes.TRA.txt", .format = "mixcr")
== Step 1/3: loading repertoire files... ==
Error in if (file.info(path)$isdir) { :
missing value where TRUE/FALSE needed

`

`> immdata <- repLoad("../data/sample1_folder/sample1.vdjca", .format = "mixcr")
== Step 1/3: loading repertoire files... ==
Error in if (file.info(path)$isdir) { :
missing value where TRUE/FALSE needed

`

I'd love to try out Immunarch, so any help with this problem would be much appreciated!
Best,
Clara

I wonder if this is the same problem I have. When you load a directory, the error message is more detailed:
22 parsing failures.
 row               col           expected               actual
1838 allDHitsWithScore 1/0/T/F/TRUE/FALSE TRDD3*00(70)
1838 allDAlignments    1/0/T/F/TRUE/FALSE 12|26|39|20|34||70.0
1840 allDHitsWithScore 1/0/T/F/TRUE/FALSE TRDD3*00(40)
1840 allDAlignments    1/0/T/F/TRUE/FALSE 14|22|39|23|31||40.0
1842 allDHitsWithScore 1/0/T/F/TRUE/FALSE TRDD3*00(40)

It seems D alignments are not handled correctly. I am using mixcr v3.0.12 and immunarch v0.5.5. Are they not compatible?

@AnaBolivar @clarapereira @hpessa
Hi, thank you for letting us know about the issue and sorry for the very late response!
Can you please send us a small subset of your data to test<EMAIL_ADDRESS>MiXCR files are tricky to work with, and we would greatly appreciate your help here.

Issue update.
After email conversations with researchers who reported the issue, we fixed some of the parsing issues in the latest immunarch version.
To ensure that you are parsing the repertoire files correctly, make sure to:

Apply the repLoad without (!) format specification, i.e., don't pass the .format argument. If repLoad doesn't work without the argument - it won't work at all.
Apply the repLoad to a folder with repertoire files. Singular files work too, but to make the analysis fast make sure to move all your repertoire files into the same folder and create a metadata.txt file with all the meta-information needed for the analysis. You can find more information here: https://immunarch.com/articles/v2_data.html
Apply the repLoad to the text / table files with clonotypes from MiXCR and other tools. *.vdjca and *.report files are suitable. We will make sure they are skipped in the next versions of immunarch. We already skipping .fasta, .fastq and other file formats which comes from single-cell sequencing.

Some of the reported issues were warnings, not errors. Even if you get a long message in your console, please check your data! We are working on removing obscure warning messages from the package to ensure the smooth experience.
I'm closing this issue for now, but please feel free to re-open it if the same problem persists. The whole ImmunoMind team sincerely appreciates you taking your time getting back to us and report this issues. @AnaBolivar , @clarapereira and @hpessa - thank you a lot!


Also Confused: downloadable fonts vs bundled

A bundled font uses multiple megabytes but does not need to download anything. Would that mean that with the downloadable font the app does download multiple megabytes during the first app start? Or how does that exactly work.

Google Play Services downloads the font on behalf of the app. When the app starts it asks for the font, if the font is already downloaded then it is served. Otherwise in a short timeout period, emoji compat init fails.


[Workspace Feeds] Transaction start date in card details page is one day prior to the selected date when assigning a card

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:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @joekaufmanexpensify
Slack conversation (hyperlinked to channel name): quality
Action Performed:
Prerequisite:
Members and company cards added in workspace , 8+ export options for card

Go to staging.new.expensify.com
Choose the workspace from settings
Click company cards >Card Details

Expected Result:
Transaction start date should be same as entered when assigning the card
Actual Result:
Transaction start date in card details page is one day prior to the selected date when assigning a card
Workaround:
unknown
Platforms:

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

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

Screenshots/Videos

Add any screenshot/video evidence
![image - 2024-10-31T153107 487](https://github.com/user-attachments/assets/710fa817-5448-4ce5-94ea-fd9e22472f35)
![image - 2024-10-31T153058 517](https://github.com/user-attachments/assets/6500a64c-2b8f-4d69-a82f-7f8cfc613a40)

View all open jobs on GitHub

BZ team member, Callstack will work on this issue, please wait for them to comment before moving the issue forward.

Handling this as part of the project I have worked on

Hello, Im Artem from Callstack and would like to help with this issue!

I would love to help on this issue with @waterim

Still investigating, couldnt reproduce

I also can't reproduce

Joe is in nee york timezone

@mountiny probably should work with removing new Date converting all the time, cant reproduce
@joekaufmanexpensify feel free to reproduce!

I can still reproduce. Added here.

@waterim This issue only can be reproduced when we set Local time is 8:30 am and EST timezone (GMT -5). The problem comes from the format function

I suggest updating
https://github.com/Expensify/App/blob/b7a9c9b19f81e344e13267882cf8d0b30d98199c/src/pages/workspace/companyCards/WorkspaceCompanyCardDetailsPage.tsx#L162
title={card?.scrapeMinDate ? format(parseISO(card.scrapeMinDate), CONST.DATE.FNS_FORMAT_STRING) : ''}

Thank you, will try tomorrow morning, maybe format from date-fns-tz

@waterim Have you had a chance to come back to this?

BugZero Checklist:

[ ] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[ ] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[ ] 2a. Reported on production
[ ] 2b. Reported on staging (deploy blocker)
[ ] 2c. Reported on both staging and production
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[ ] 3d. QA
[ ] 3z. Other:

[ ] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment:

[ ] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion:

[ ] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal Template

[ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Precondition:

Test:

Do we agree 👍 or 👎

Checklist done, regression test for this specific scenario is not needed I think, it will be covered with the tests for the feature we will add with Joe. Payments are handled separately for the project so closing this one for now


Update manage-access-to-o365-services-for-pcs-managed-by-sccm.md

Corrected "Bitlocker" typo.
Summarize the change in the pull request title
Describe your change, specifically why you think it's needed.
Fixes #Issue_Number (if necessary)

@cliffhobbs : Thanks for your contribution! The author, @aczechowski, has been notified to review your proposed change.

Editorial fix confirmed.


Fragmented MP4

Do you have any examples for fragmented mp4?

Sorry, I missed this.
No, I don't have any examples to hand I'm afraid. ffmpeg supports it though, so it may be sufficient to add -movflags frag_keyframe+empty_moov to FFMPEG_FLAGS. I suspect there are probably some other tweaks needed as well though

Im still actively using the ts method, but also trying to find out how to use fragmented mp4s


Shortcuts.debugLabel is ignored in its FocusNode.debugLabel

I'm setting debugLabel but always prints Shortcuts.
This is the source code of Shortcuts.constructor():
const Shortcuts({
    super.key,
    required Map<ShortcutActivator, Intent> shortcuts,
    required this.child,
    this.debugLabel,
  }) : _shortcuts = shortcuts,
       manager = null;

Here
This is the source code of Shortcuts.build():
@override
  Widget build(BuildContext context) {
    return Focus(
      debugLabel: '$Shortcuts',
      canRequestFocus: false,
      onKey: _handleOnKey,
      child: widget.child,
    );
  }

Here

Hi @busslina, please provide a complete minimal reproducible code sample along with steps to reproduce so that we can investigate this. Please do not provide screenshots of code/logs as they are not searchable.
Please also provide the output of flutter doctor -v
Thank you

@danagbemava-nc I just update it with the code and the links to that code. Flutter Doctor is not need because is an identified code issue.

Hi @busslina,
Please provide a minimal complete reproducible code sample along with steps to reproduce so that we can investigate this issue.
Thank you

@danagbemava-nc
The problem is just that A must be B:
A)
@override
  Widget build(BuildContext context) {
    return Focus(
      debugLabel: '$Shortcuts',
      canRequestFocus: false,
      onKey: _handleOnKey,
      child: widget.child,
    );
  }

B)
@override
  Widget build(BuildContext context) {
    return Focus(
      debugLabel: debugLabel == null ? '$Shortcuts' : '$Shortcuts -- $debugLabel',
      canRequestFocus: false,
      onKey: _handleOnKey,
      child: widget.child,
    );
  }

@busslina When opening an issue, please try to provide all information indicated in the template so that we can address this issue properly (See bug issue template here)
More specifically, please provide the information indicated below:

A completed and minimal reproducible code sample
The output of flutter doctor -v
Reproduction steps
Which environment did this issue happen in? (Android/iOS/Web/MacOS/Windows/Linux)

https://github.com/flutter/flutter/pull/140202


[Dupe] Botania, Mana Lens Force

Version: griefdefender-1.1.0-TEST13-DEBUG3
The following setup on the edge of a claim will duplicate the block the pulse hits, instead of pushing it / nothing.

https://github.com/Vazkii/Botania/blob/fb8ebd8ca7e4dd24a1c93bf2a92ea4f93fee98fe/src/main/java/vazkii/botania/common/item/lens/LensPiston.java#L26

Create new ticket if this issue persists with latest SpongeForge and is caused by GD


[Snyk] Upgrade core-js from 3.36.1 to 3.37.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade core-js from 3.36.1 to 3.37.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 21 days ago, on 2024-04-16.

Release notes

Package name: core-js

3.37.0 - 2024-04-16
Changes v3.36.1...v3.37.0
New Set methods proposal:

Built-ins:

Set.prototype.intersection
Set.prototype.union
Set.prototype.difference
Set.prototype.symmetricDifference
Set.prototype.isSubsetOf
Set.prototype.isSupersetOf
Set.prototype.isDisjointFrom

Moved to stable ES, April 2024 TC39 meeting
Added es. namespace modules, /es/ and /stable/ namespaces entries

Explicit Resource Management stage 3 proposal

Some minor updates like explicit-resource-management/217

Added Math.sumPrecise stage 2.7 proposal:

Built-ins:

Math.sumPrecise

Promise.try proposal:

Built-ins:

Promise.try

Added optional arguments support, promise-try/16
Moved to stage 2.7, April 2024 TC39 meeting

RegExp.escape stage 2 proposal:

Moved to hex-escape semantics, regex-escaping/67

It's not the final change of the way of escaping, waiting for regex-escaping/77 soon

Pattern matching stage 1 proposal:

Built-ins:

Symbol.customMatcher

Once again, the used well-known symbol was renamed
Added new entries for that

Added Extractors stage 1 proposal:

Built-ins:

Symbol.customMatcher

Since the Symbol.customMatcher well-known symbol from the pattern matching proposal is also used in the exactors proposal, added an entry also for this proposal

Added URL.parse, url/825
Engines bugs fixes:

Added a fix of Safari { Object, Map }.groupBy bug that does not support iterable primitives
Added a fix of Safari bug with double call of constructor in Array.fromAsync

Compat data improvements:

URL.parse added and marked as supported from FF 126
URL.parse added and marked as supported from Bun 1.1.4
URL.canParse fixed and marked as supported from Bun 1.1.0
New Set methods fixed in JavaScriptCore and marked as supported from Bun 1.1.1
Added Opera Android 82 compat data mapping

3.36.1 - 2024-03-19
Changes v3.36.0...v3.36.1
Fixed some validation cases in Object.setPrototypeOf, #1329, thanks @ minseok-choe
Fixed the order of validations in Array.from, #1331, thanks @ minseok-choe
Added a fix of Bun queueMicrotask arity
Added a fix of Bun URL.canParse arity
Added a fix of Bun SuppressedError extra arguments support and arity
Compat data improvements:

value argument of URLSearchParams.prototype.{ has, delete } marked as supported from Bun 1.0.31
Added React Native 0.74 Hermes compat data, Array.prototype.{ toSpliced, toReversed, with } and atob marked as supported
Added Deno 1.41.3 compat data mapping
Added Opera Android 81 compat data mapping
Added Samsung Internet 25 compat data mapping
Added Oculus Quest Browser 32 compat data mapping
Updated Electron 30 compat data mapping

    from core-js GitHub release notes

Commit messages

Package name: core-js

598d0b2 3.37.0
88b57ee update the changelog
a7f3a96 `URL.parse` added and marked as supported from Bun 1.1.4
3c2deac fix the place of `Uint8Array` to / from base64 and hex in the readme
1bb8d08 update the changelog
a7f8abf Merge pull request #1343 from zloirock/update-pattern-matching
be019f9 fix a link
c978c8f add an entry for exactors proposal
8957db1 update pattern matching proposal
212ff8a add a link
d393ed4 some simplification
51cb7c8 some simplification
51ce5d0 simplify some constants definition
e7e9d71 update docs
3a54109 add some tests
6912464 update dependencies
9da401f add `Math.sumPrecise`
0ac334d enable some `eslint` rules from `eslint-plugin-import-x`
bec66e7 update dependencies
3f46d78 update some notes
acec5c4 update dependencies
80f1d23 add a fix of Safari `{ Object, Map }.groupBy` bug that does not support iterable primitives
5b908c2 add a fix of Safari bug with double call of constructor in `Array.fromAsync`
92f8618 update dependencies

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Ble noe rot med snyk vs dependabot så fikk merge-konflikter. Lukker denne og så kommer den snart med dependabot


[Question] How to Reference Job Library?

THANK YOU FOR THIS LIBRARY!!
I very much dislike .yml templates and feel at home in C# code. I am loving this library and plan to contribute a NugetInstallerBuilder once I finish these Pipeline tasks.
I have a question about how to reference a Job Library.
I am able to reference Step Libraries without any issue, however, when I try and do the same with a JobLibrary<> I receive errors.

From the documentation, you can reference a StepLibrary like this (This works fine for me):
 new Job("Build")
    {
        Steps =
        {
            Script.Inline("echo 'Hello World'"),

            // I can run this example with no issues
            StepLibrary<ProjectBuildSteps>(),

            Script.Inline("echo 'Goodbye World'"),
        }
    }

My Error
However, when I run the example above when attempting to reference a JobLibrary<> I receive an error:

My class which implements JobLibrary is defined below:

internal class JobBuildDeployPackNugetPackage : JobLibrary
{
    public override List<Conditioned<Job>> Jobs =>
        new()
        {
            new Job("RestoreBuildPackDeployPackage")
            {
                Pool = new HostedPool("$(DefaultBuildAgentWin)"),

                Steps =
                {
                    StepLibrary<StepInstallNet6Sdk>(),

                    StepLibrary<StepNugetToolInstallerV6>(),

                    StepLibrary<StepNugetRestoreSolution>(),

                    Script.Inline("echo 'Hello World'"),

                    Script.Inline("echo 'Goodbye World'")
                }
            }
        };
}

Question
Am I doing something wrong?  Can you please provide a simple example referencing a job library?

Thanks so much again! This is a great library!!! I am surprised that Microsoft has not done this yet. I can see Microsoft adopting this library once it matures!
Thanks so much!

Hi,
Thanks for the feedback and for the bug report! I am very glad Sharpliner makes someone's life easier!
You are not doing anything wrong, there's a bug in Sharpliner. This is a regression from when I was adding DeploymentJob and adding a common ancestor for Job and DeploymentJob which is called JobBase. The problem is that the JobLibrary should be defined as JobBase and not Job.
The fix is super easy, I will have a new version for you in second, please bare with me!

The fix has been published in version 1.3.1 which should appear listed on nuget.org shortly.
Thanks for the extensive report again, it was super easy to determine the cause thanks to this!


Refactor rfid processing

Please merge first https://github.com/esprfid/esp-rfid/pull/490 as this will be easier to review.
Unfortunately this is changing heavily the rfid.esp file. So I understand if you are skeptical about merging this, but this refactor was needed to fix a few bugs, add better pincode support and make it more reliable.
I'm trying to explain what happens now, in particular with the wiegand sensor with pincode.
We have now two state machines, one for the sensor state, another for the processing state.
Everything starts at the rfidLoop(), where now the steps are more separate. The rfidRead() takes care of getting the value of the sensor and setting the state in rfidState. Then, if the pincode is required the following step is going into the pinCodeRead function, which adds the pincode.
Once we have the info in memory and the reading steps are done (tracked in the rfidState), we can proceed with the rfidProcess() function. There we check if the card/pincode combination is valid and if the card has necessary permissions. If so we change the processingState variable (the other "state machine") so that the next step can proceed and we trigger the relays if necessary.
In the last step rfidOutsideMessaging() we send the necessary MQTT messages and signal how things are going with LEDs, so that also the user can immediately see what's going on.
I'm happy to go over this code over a call if you want!
I have a similar important refactor for the WiFi that is coming soon, so that we fix a few outstanding issues there. But this needs to go first to avoid too many changes at once (if these are not enough already!).

Thank you guys. much appreciated.


Change gravity

Hey,
I would like to change the gravity in TSID, to turn it to 0.
In Pinocchio, I would do that with model.gravity = pinocchio.Motion.Zero(). Yet I don't have a direct access to the pinocchio model in the TSID robot wrapper (robot.model() gives me access, I believe, to a copy of the pinocchio model).
What do you think is the best way to change the gravity in TSID? Should I add a setter function in TSID, e.g. setGravity(const Motion & g), and wrap it in python?
Thanks for your advices.

Since there was no update about this for 20 days I consider the issue solved.


Install the MicrosoftTeams module does not work

I m on a MAC Big Sur
I ve installed homebrew.
I ve installed powershell, I ve tested it (pwsh command)
Now trying to install the MicrosoftTeams module through the following command :
Import-Module MicrosoftTeams
$userCredential = Get-Credential
Connect-MicrosoftTeams -Credential $userCredential
Here is the error message  I have :
Import-Module MicrosoftTeams
Import-Module: The specified module 'MicrosoftTeams' was not loaded because no valid module file was found in any module directory.
PS /Users/xxxx> $userCredential = Get-Credential
PowerShell credential request
Enter your credentials.
User<EMAIL_ADDRESS>Password for user<EMAIL_ADDRESS>*********
PS /Users/xxx> Connect-MicrosoftTeams -Credential $userCredential
Connect-MicrosoftTeams: The term 'Connect-MicrosoftTeams' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
[Enter feedback here]

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

ID: 289b637a-ad53-2710-3b6e-7f4ac15916fa
Version Independent ID: e8de4044-38f3-1002-9816-12515d0cdda0
Content: Manage Skype for Business Online with PowerShell - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/manage-skype-for-business-online-with-microsoft-365-powershell.md
Service: o365-administration
GitHub Login: @JoeDavies-MSFT
Microsoft Alias: josephd

@bbollaer Thank you for your feedback. What version of Powershell are you using? There are known issues with PowerShell 7 and Teams PowerShell. For the best experience, we recommend that you use PowerShell 5.1.- For more information about teams module, Visit  https://docs.microsoft.com/en-us/microsoftteams/teams-powershell-install
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @bbollaer Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


use /get instead of /render

see https://github.com/raintank/metrictank/pull/575

this cant be merged until after  raintank/metrictank#575 is merged