Latest esp-idf master breaks working camera code that uses the ledc submodule

By reverting esp-idf back to earlier commit, the error does not happen.
The earlier commit where the code was working is :
$ git show
commit 7d13e3fa6af8c500ac207b2734c2611cf2f24e34 (HEAD)
Merge: 22ae5adb 37b98f73
Author: Angus Gratton<EMAIL_ADDRESS>Date:   Wed Feb 14 09:45:09 2018 +0800
Bringing esp-idf up to date with:
Git checkout master
Git pull
Git submodule update --init
$ git status
On branch master
Your branch is up to date with 'origin/master'.
nothing to commit, working tree clean
$ git show
commit b2ff235bd003bf3dcb114ff7efe3e0c605c4dd9f (HEAD -> master, origin/master, origin/HEAD)
Merge: 8a4c9d7d a8504b96
Author: Angus Gratton<EMAIL_ADDRESS>Date:   Tue Jun 5 11:51:35 2018 +0800
Now flashing code with latest esp-idf causes the following errors:  (The code uses igrr’s camera code, and appears to be breaking when using the ledc code to create the XCLK PWM output.
Here is info from monitoring the output :
Flashing binaries to serial port COM3 (app at offset 0x10000)...
esptool.py v2.3.1
Connecting....
Chip is ESP32D0WDQ6 (revision 1)
Features: WiFi, BT, Dual Core
…….
D (1202) intr_alloc: Connected src 22 to int 13 (cpu 0)
D (1202) camera: Enabling XCLK output
E (1202) ledc: ledc_set_duty_with_hpoint(383): hpoint argument is invalid
D (1207) ledc: LEDC_PWM CHANNEL 0|GPIO 19|Duty 0001|Time 0
D (1213) camera: Initializing SSCB
D (1217) camera: Resetting camera
D (1240) camera: Searching for camera address
E (1376) test.c: Camera probe failed with error 0x20001

This issue was moved to igrr/esp32-cam-demo#80


Game Name variable in filename variables.

Would it be too much work to have the game name as a variable in the file name been offered?
eg {gamename} or so.

From the 2.0.2 change log:

Filename wildcards {game} and {fps} have been removed (not available in new API)


fix: SafelistRequestStatus undefined fix

@thebrianchen The previous fix handled the undefined value but the comparison was wrong.
rawNftContract.openSeaMetadata?.safelistRequestStatus !== null will return 'true' if the value is undefined.

@thebrianchen Kindly check this.
It has been a blocker for a long time now 🙏

@thebrianchen the bug isn't fixed! please merge soon!

@thebrianchen Kindly check this.
It has been a blocker for a long time now 🙏

@thebrianchen Kindly check this. It has been a blocker for a long time now 🙏

Since the team is not working on fixing this, I used direct API call as below:
{baseURL}/getNFTMetadata?contractAddress=${contract}&tokenId=${tokenId}&tokenType=${tokenType}
This worked for me 👍

@thebrianchen Kindly check this. It has been a blocker for a long time now 🙏Please check this. It’s been blocked for a long time🙏

Since the team is not working on fixing this, I used direct API call as below: Since the team is not working on fixing this, I used direct API call as below: {baseURL}/getNFTMetadata?contractAddress=$ {contract}&tokenId=${tokenId}&tokenType=${tokenType}
This worked for me 👍This worked for me👍

Can you post the complete code that calls the function?


More descriptive error messages than 'couldn't send'

When I use the Emulator, instead of the error message simply saying "couldn't send," it should be a lot more descriptive. Describe in more detail, such as, "Failed to establish a TCP IP connection. Did you remember to start your emulator?"

I am referring to the former: the message under the failed message's chat bubble. "Couldn't send." Sorry for the ambiguity.

No worries, I will move this over to the WebChat repository then because that UI is embedded in the WebChat control and out of the control of the Emulator.

@compulim Tagging because this would need to be a Webchat change.

Wouldn't the message in the log panel provide sufficient information about the error? The message under the chat bubble is not meant to be comprehensive and probably should not be due to the limited  screen real estate.

Hello!
This is going to be addressed in the Offline UI set of stories in the WebChat project here


Add option to move plus and minus buttons to the bottom

I have the problem you can see in the screenshot, if i have 2 versatile thermostats in a horizontal stack this happens.
I don't want to hide the plus/minus buttons, it would be cool if we could have a switch to move them below the other buttons, or next to the heat/off buttons

Hello @0verEngineer,
I'm really not a specialist of css. In the next release you will have the option to remove the circle.
I hope this helps.

Kind of duplicate of #26

@jmcollin78 Which cycle do you mean?

This one: https://github.com/jmcollin78/versatile-thermostat-ui-card/releases/tag/0.6.2

Hmm i don't really know if this will fix the issue, and i quite like the circle 😅

If you are an expert in html/css, I would love to have a PR.  I just copy this from Best Thermostat UI and adapt but the result is not fabulous.

I don't know how to set up the dev env, i tried following things:

start:hass script

installed HACS and Versatile Thermostat in it
connected to my MQTT server to have real thermostats
But now i have no idea how i get the ui card in it

start script
Here i get a url in the terminal but the page shows following:

To set-up the env dev, I do the following:

have a VScode with Versatile Thermostat,
have another VSCode with Versatile Thermostat UI. Run npm run build to build the dist directory,
in the first VScode, you have in the devcontainer.json, I mount the dist directory of the UI into config/www/community "source=${localEnv:HOME}/SugarSync/Projets/home-assistant/versatile-thermostat-ui-card/dist,target=/workspaces/versatile_thermostat/config/www/community/versatile-thermostat-ui-card,type=bind,consistency=cached" . You will have adapt this line to mount the dist directory of your second VSCode.
Run Versatile Thermostat in the first VScode and create a VTherm and create a UI card.


C++ test coverage

This PR:

Demonstrates a basic unit test infrastructure, including test coverage.
Demonstrates interfacing with coveralls.io for coverage reports.
Fixes #2

Changes Unknown when pulling f18c1c7e119a38e64cec3b2d1201d4a88970e12c on cpp-test-coverage into ** on master**.

Ah, just noticed- I think in the Makefile, you want the target to be "install", not "instal"...

Heh, good catch.  Also I need to update the change log & some other documents, so not quite ready for merging.

Changes Unknown when pulling 37ed0460f512632f0cb86b18a62b23ce7ccbe4eb on cpp-test-coverage into ** on master**.

Changes Unknown when pulling 7887435e66e85b5b349bfd1482eca23d714d619d on cpp-test-coverage into ** on master**.

Changes Unknown when pulling cecc68a93ae37cdd1f6ae73f209ae66bc04193f5 on cpp-test-coverage into ** on master**.


Wrong metadata.component content in CycloneDX

Picking up the question from https://github.com/anchore/syft/pull/2131#issuecomment-1719529908:
The CycloneDX contains this metadata when using it on a conan lockfile.
"metadata": {
   "component": {
      "bom-ref": "7ea68ac679dc44fd",
      "type": "file",
      "name": "conan.lock",
      "version": "sha256:sha256:421e3aca902b6310bc89875e13c135843d574a0d01bc399c895fde46ecc16068"
    }
}

This was the comment from @kzantow:

This is the "source" you scanned: image, directory, file, etc.. However, this does not really have dependencies to the other components. Creating a graph in CycloneDX is much more limited than in SPDX -- in SPDX we add CONTAINS relationships for this, but the only option in CycloneDX is dependencies, which isn't accurate and we try very hard not to misuse the formats. I'm curious if you add additional dependency entries from the first entry to the other top-level components, does it look to work properly in DependencyTrack?

Now I looked at the CycloneDX webpage:
https://cyclonedx.org/specification/overview/#bom-metadata

BOM metadata includes the supplier, manufacturer, and target component for which the BOM describes. It also includes the tools used to create the BOM, and license information for the BOM document itself.

Based on this, I understand it in the way that metadata.component is not the source from where the Bom was generated, but actually the target for which it is generated.
To me it would be more logical to have here specifically a component which is referenced in the components array further down. This is also how https://github.com/CycloneDX/cyclonedx-conan is using it.
And also how it is shown in the examples here:
https://github.com/CycloneDX/bom-examples/blob/master/SBOM/dropwizard-1.3.15/bom.json#L47
My Question:
Could it be that syft is wrongly setting the metadata.component entry to the conanfile, instead of the specific component?

I see the basic point: if I'm generating an sbom for a single lib that has multiple dependencies from a directory scan, its more semantically useful to have the root component be the single lib that was discovered, not the file it was discovered from (which has non semantic meaning). I think changing the default behavior makes sense, however, there will be times when more than one component would be discovered so choosing one package over another might not make sense (in which case the behavior we have today is alright).
Additionally, adding a configurable for this to allow folks to opt out of this "promote the root package" behavior being described also makes sense to me.
Lastly, I think this strategy should be considered for SPDX too, even if we decide to not implement it for SPDX.

@wagoodman for a conan lock file or conanfile.txt, there is always only one root component for which this lockfile is generated.
I do not see a good example where such a file may describe multiple root components, and therefore the metadata.component should be something different.
So IMHO, there is no need to have this additional "promote the root package" option, but simply always use the root package in the metadata.component.


this code make it does not work

This change is

have you tried the newest interpreter?

i act as a fool @shia86


feat(beacon-network): add content validation

What was wrong?
We don't validate the beacon content type before storing and gossiping.
How was it fixed?

Add validations for the beacon content types.
Extract a test_utils module for common usage in tests.

To-Do

[ ] Clean up commit history and use conventional commits.

Reading through the validation code here, I wanted to ask what the status of our hive tests is for validating these objects.  It seems like with as many validation checks as there are it would be easy for a client to miss or incorrectly implement the validation logic.  If we don't have comprehensive tests vectors in place for both valid and invalid payloads, that seems like a place where effort should go.  At minimum, getting issues written in the specs repo that detail the tests that need to be written.

Reading through the validation code here, I wanted to ask what the status of our hive tests is for validating these objects. It seems like with as many validation checks as there are it would be easy for a client to miss or incorrectly implement the validation logic. If we don't have comprehensive tests vectors in place for both valid and invalid payloads, that seems like a place where effort should go. At minimum, getting issues written in the specs repo that detail the tests that need to be written.

Currently there is no negatives, and we don't really test for positives either

Reading through the validation code here, I wanted to ask what the status of our hive tests is for validating these objects. It seems like with as many validation checks as there are it would be easy for a client to miss or incorrectly implement the validation logic. If we don't have comprehensive tests vectors in place for both valid and invalid payloads, that seems like a place where effort should go. At minimum, getting issues written in the specs repo that detail the tests that need to be written.

Yeah, we will work on adding more tests for testing content validations for history and beacon. We are currently trying to fix the failing beacon hive tests for Trin.


Add Swift Concurrency support

Add Swift 5.5 and implement Swift Concurrency and Actors.

Fixed in v5.0.0


Entity copy

I'm trying to figure out how to copy an entity and its components, and came across this snippet in the docs:
registry.visit(entity, [&](const auto info) {
    auto storage = registry.storage(info);
    storage->emplace(registry, other, storage->get(entity));
});

It seems this is broken right now. I tried v3.7.1, master, and experimental. In master/experimental, emplace is a member of storage, but doesn't exist at all in 3.7.1. storage->get(entity) doesn't exist in all three. Hopefully I'm just misunderstanding something though.
registry.visit(entity, [&](const auto info) {
    entt::poly<entt::Storage<entt::entity>>& storage = registry.storage(info);
    storage.emplace(registry, other, storage->get(entity));
});

As a nice to have, I was thinking a simple copy/clone abstraction method would be better anyway. Something like so:
entt::registry registry;
entt::registry otherRegistry;

entt::entity entity = registry.create();
// numerous components are added to entity

entt::entity other = registry.copy(entity);
// or from one registry to another
entt::entity other = otherRegistry.copy(entity, registry);

Probably you're interested in this discussion (it also contains a link to a working example).
The copy function isn't something that I feel like I want to add to the registry. You can easily define your utility like:
my_namespace::my_copy_function(entt::registry &registry, entt::entity from, entt::entity to);

As for the why there isn't a built-in mechanism, it's quickly explained. Consider two types, T and U. What should copy do for them? Emplace the source into the destination? Replace it? Emplace or replace? What if one of them is such that a copy boils down to a custom behavior like porting only some data members? What if you want to copy T but ignore U when you copy an entity? And so on.
This happened more often than not in the past and a one-fits-all solution that works for all users doesn't exist apparently.
The poly storage is there exactly for this purpose. You can look at it as an empty class that you can extend for all types or on a per-type or per-traits basis so as to customize the copy functionality (as well as any other functionality) the way you like. Moreover, there is also the nice-to-have side effect that I won't pay the cost of instantiating erased functions to copy all types if I don't need to.

Thanks! That discussion link was enough (I'll remember to check the discussions tab next time). You can close this issue if you want.
To answer this question:

As for the why there isn't a built-in mechanism, it's quickly explained. Consider two types, T and U. What should copy do for them? Emplace the source into the destination? Replace it? Emplace or replace? What if one of them is such that a copy boils down to a custom behavior like porting only some data members? What if you want to copy T but ignore U when you copy an entity? And so on.

I can see why it gets complicated. Ultimately, in this use case, I want "other" to have the same data as "entity" in the end, just like I'm copying a class/struct. But I guess that logic would be: If the component type T or U exists on "other", replace, otherwise emplace, and if there are other components in "other" that don't exist on "entity", remove them. Maybe that boils down to naming: clone_entity or something. "copy" might be slightly more ambiguous.


Drop host dnsmasq setup on Linux

While https://github.com/code-ready/crc/pull/264 makes the dnsmasq setup simpler already, it'd be nice to not have to do any dnsmasq setup on the host machine at all.

WIP branch here: https://github.com/code-ready/crc/pull/590


Multiple Language support in the Editor UI?

Requesting a feature *

Now that multiple languages are supported, are there any plans to have the ability to enter the language translations for each supported string (question title, choice text, matrix column headers, etc.) into the survey editor UI, rather than just in the raw JSON?

@SlowburnAZ It is already done.
Change survey.locale to the language you need and start to edit the titles, choices text and so on.
After you are done, change to another survey.locale and repeat.
You may see the result in the JSON.
Thank you,
Andrew

Ah, okay.  I guess I didn't put 2 and 2 together because it still showed my English wording when I changed the locale:

The UI doesn't seem update the choices when you change the locale.  For me, to get the editor to update the text for the choices, I've had to select the question, then bring up the choice editor, then click OK.  Not a huge deal, but it might confuse other people as it did me.
In any case, is Spanish support coming at some point?

@SlowburnAZ Choices have to be updated. It works before and even unit tests are still there, but after markdown support, we start to use another rendering mechanism for strings. I forgot to check the localization. I will work on the fix.
About Spanish localization. As soon as somebody add the Spanish localization here: https://github.com/surveyjs/surveyjs/tree/master/src/localization, we will have it. It can be you :-)
Thank you,
Andrew

@andrewtelnov
I'd be happy to provide the Spanish translations.  Do you have the list of the original English strings for each?

It would be great!
Localized strings in English:
https://github.com/surveyjs/surveyjs/blob/master/src/surveyStrings.ts

@SlowburnAZ The bug was fixed.
FYI: @tsv2013 is working on examples with custom widgets + multiple language support. At the current moment custom widget doesn't re-render itself on locale change. We will change it.
Thank you,
Andrew

@andrewtelnov Here's the Spanish localization code:
import {surveyLocalization} from "../surveyStrings";

export var spanishSurveyStrings = {
    pagePrevText: "Anterior"
    pageNextText: "Siguiente",
    completeText: "Completo",
    otherItemText: "Otro (describa)",
    progressText: "Pagina {0} de {1}",
    emptySurvey: "No hay pagina visible o pregunta en la encuesta.",
    completingSurvey: "Gracias por completar la encuesta!",
    loadingSurvey: "La encuesta se esta cargando...",
    optionsCaption: "Seleccione...",
    requiredError: "Por favor conteste la pregunta.",
    requiredInAllRowsError: "Por favor conteste las preguntas en cada hilera.",
    numericError: "La estimacion debe ser numerica.",
    textMinLength: "Por favor entre por lo menos {0} symbolos.",
    textMaxLength: "Por favor entre menos de {0} symbolos.",
    textMinMaxLength: "Por favor entre mas de {0} y menos de {1} symbolos.",
    minRowCountError: "Por favor llene por lo menos {0} hileras.",
    minSelectError: "Por favor seleccione por lo menos {0} variantes.",
    maxSelectError: "Por favor seleccione no mas de {0} variantes.",
    numericMinMax: "El '{0}' debe de ser igual o mas de {1} y igual o menos de {2}",
    numericMin: "El '{0}' debe ser igual o mas de {1}",
    numericMax: "El '{0}' debe ser igual o menos de {1}",
    invalidEmail: "Por favor agrege un correo electonico valido.",
    urlRequestError: "La solicitud regreso error '{0}'. {1}",
    urlGetChoicesError: "La solicitud regreso vacio de data o la propiedad 'trayectoria' no es correcta",
    exceedMaxSize: "El tamaño der archivo no debe de exceder {0}.",
    otherRequiredError: "Por favor agrege la otra estimacion.",
    uploadingFile: "Su archivo se esta subiendo. Por favor espere unos segundos y intente de nuevo.",
    addRow: "Agrege hilera",
    removeRow: "Retire",
    choices_firstItem: "primer articulo",
    choices_secondItem: "segundo articulo",
    choices_thirdItem: "tercer articulo",
    matrix_column: "Columna",
    matrix_row: "Hilera"
};

surveyLocalization.locales["es"] = spanishSurveyStrings;

@SlowburnAZ Great, thank you!
We will release new version on Monday. Spanish translation will be included.

v0.12.11 is released


fix wrong parameter name

q08.sql uses "q08_seconds_before_purchase' parameter for the streaming script, however, this is not defined in queryParameters.sql and fails this query.

Sorry i forgot to push the changed queryParameters.sql in the last commit.
Just fixed that and updated the 0.5 release version (we cant have a not working release, can we )
https://github.com/intel-hadoop/Big-Data-Benchmark-for-Big-Bench/releases/tag/0.5


Add pixi project version {major,minor,patch} commands

Problem description
Blocked by https://github.com/mamba-org/rattler/issues/449

This is a follow up of https://github.com/prefix-dev/pixi/issues/575 and https://github.com/prefix-dev/pixi/pull/579 but we don't want to block the ongoing PR.

@ruben-arts happy to take the lead on this one once https://github.com/mamba-org/rattler/issues/449 is merged and a new rattler release is cut.
I tried to compile pixi with rattler from https://github.com/mamba-org/rattler/issues/449, but I have a few compilation errors not related that should be fixed first.

@hadim yeah you are right, there are some differences between rattler main and the latest release that don't work in pixi.
The best experience for you would be to wait for a new release.

Ok if I wrap up this one, given the latest 0.15 rattler release?

I gave it a try at https://github.com/prefix-dev/pixi/pull/633


C5 with spot instance support in Ningxia region?

C5 instance type with Nitro hypervisor is available in Ningxia and Beijing regions and it has very great cost-price ratio.
Let's make a note on how to build kops-cn in Ningxia with C5 spot instance to save the cost up to +60%.

assigning to myself

just specify maxPrice when you kops edit ig --name=cluster.zhy.k8s.local nodes


HypervisorPresent bug

Not using this project, just run across this copy/paste bug and thought I'd give you a heads up:
https://github.com/sondernextdoor/WindowsHardwareInfo/blob/main/hwid.h#L637
		QueryWMI(L"Win32_ComputerSystem", L"Model", IsHypervisorPresent);

should be:
		QueryWMI(L"Win32_ComputerSystem", L"HypervisorPresent", IsHypervisorPresent);

Good catch! Fixed.
Thank you :)


When performing manual depreciation, the system is fetching the session's default cost center in the accumulated depreciation account while creating depreciation journal entry

Information about bug
When performing manual depreciation, the system is fetching the session's default cost center in the accumulated depreciation account while creating depreciation journal entry

Module
assets
Version
Assets: v0.0.1 (pre-prod)
ERPNext: v15.34.0 (pre-prod)
Frappe Framework: v15.41.0 (pre-prod)
Installation method
None
Relevant log output / Stack trace / Full Error Message.
No response

Done https://github.com/8848digital/Assets/pull/67

Functionality Working. Issue Resolved.


Ajax-requests and X-No-Composition-header responses are now cached

Coverage increased (+0.08%) to 92.873% when pulling c3f741160f9f5fe06c547464b073b3741a5930da on ajax-and-x-no-composition-caching into 83573b9915901f590f8c5c700cafccb4fea22fcb on master.

Coverage increased (+0.1%) to 92.883% when pulling f0d05ba7e16741ef3216f3eae053628860fa5bcd on ajax-and-x-no-composition-caching into 83573b9915901f590f8c5c700cafccb4fea22fcb on master.

Coverage decreased (-0.1%) to 92.663% when pulling 6011bb7c1ab03b2410e3309a023eb2e31648bef9 on ajax-and-x-no-composition-caching into 83573b9915901f590f8c5c700cafccb4fea22fcb on master.

Coverage decreased (-0.1%) to 92.663% when pulling 8b3a7107e695c90493bc0baad26644d6a30357d0 on ajax-and-x-no-composition-caching into 83573b9915901f590f8c5c700cafccb4fea22fcb on master.

+1

Coverage increased (+0.08%) to 92.868% when pulling bae4484d609d65971f6c52d8471b1d93a2c46080 on ajax-and-x-no-composition-caching into 83573b9915901f590f8c5c700cafccb4fea22fcb on master.


[macOS] Hide dock Icon - keep menubar

Can the Dock icon be hidden in macOS and keep only the Menubar icon?
PS: with this command is possible /usr/libexec/PlistBuddy -c "Add :LSUIElement bool true" "/Applications/clipshare.app/Contents/Info.plist"
But I would like to know if it is a "standard" way.

The command above makes the Camera stop working.
Opening a dialog with "Camera permission is required to use this feature." - even if it is already enabled.


Do you use top-k trick in the code?

You mentioned in this answer that you applied top-k trick to the baseline in all ablation studies. (https://github.com/hkchengrex/STCN/issues/25#issuecomment-881896022)
But top-k trick doesn't seem to be applied in your code in get_affinity()  (in network.py). Is it because I misunderstood your code or the code you provided does not use top-k trick？

It's in evaluation only: https://github.com/hkchengrex/STCN/blob/e03c1585fd852624204b5c4e8d8e828ddb2d406f/inference_memory_bank.py#L37


0 gets exported as empty cell.

[x] Able to reproduce the behaviour outside of your code, the problem is isolated to Laravel Excel.
[x] Checked that your issue isn't already filed.
[x] Checked if no PR was submitted that fixes this problem.
Versions
PHP version: 7.1
Laravel version: 5.6
Package version: ~2.1.0
When the value 0 is exported, it is exported as an empty cell.
I tried setting the 'strictNullComparison', this works for the first columns with 0's. The second column with 0's contains empty cells again.
Any fix for this?

Thanks for submitting the ticket. Unfortunately the information you provided is incomplete. Before we can pick it up, please check (https://github.com/Maatwebsite/Laravel-Excel/blob/3.0/.github/ISSUE_TEMPLATE.md) and add the missing information.
To make processing of this ticket a lot easier, please make sure to check (https://laravel-excel.maatwebsite.nl/docs/3.0/getting-started/contributing) and double-check if you have filled in the issue template correctly. This will allow us to pick up your ticket more efficiently. Issues that follow the guidelines correctly will get priority over other issues.

@Sander00 part of the issue template is to show your code. Without seeing your code, it's very tricky to help. Can you please add more information based on the issue template.

Im sorry. Here is my code:
`Excel::create('clients', function ($excel) {
$excel->sheet('Klanten', function ($sheet) {
$clients = Client::select([
'client_id',
'email',
'company',
'city',
'firstname',
'lastname',
'token',
'active',
])->get();
            $sheet->fromModel($clients, '0', 'A1', true);
        });
    })->download('csv');`

@Sander00 perhaps try the config setting here: https://github.com/Maatwebsite/Laravel-Excel/blob/2.1/src/config/excel.php#L256

@patrickbrouwers That does not fix the problem.

You could have a look at custom value binders: https://laravel-excel.maatwebsite.nl/docs/2.1/import/formatting#docs That's most likely the most reliable way to make sure the value set in the cell is the one you want.

hello @patrickbrouwers  , I read this
https://github.com/Maatwebsite/Laravel-Excel/blob/2.1/src/config/excel.php#L256
but that config is Laravel-Excel version 2.1
how to config  'strictNullComparison' in Laravel-Excel version 3.0?

Hello @patrickbrouwers , @nanfeng
I had the same issue with the 3.0 version and after reading the comments here I searched the code for "strictNullComparison".  I found it in the phpoffice package, in the class PhpOffice\PhpSpreadsheet\Worksheet\Worksheet at the function fromArray (line 2479), where it has the default value false.  I changed it to true, to test it out and that fixed the issue for me.
May be the solution would be to have a config value for that?  I hope this helps in anyway.

Closing due to inactivity on the 2.1 question and the 3.0 issue being solved.

$sheet->cell('B1', function ($cell)
{
$cell->setValue(0);
});


Deprecation removal Symfony 4.2

ContainerAwareCommand is deprecated since Symfony 4.2. I just replaced it by Command service and then use dependency injection in command controller to make use of ChunkManager or OrphanageManager instead of getting it by container get method.
Fix #354

Looks good. Can we please merge this?

Sure, 2.1.2 is just released! Thanks @Wait4Code 👍

@bytehead, please take a look

Oops, I'll recheck!

Indeed, my pull-request breaks theses 2 commands. My bad. I've done needed changes locally. I can provide a new pull request if needed.

No worries, I'm just finishing the fix. 🙂

2.1.3 should fix it! thanks @neoantox for reporting!


Add destination information to ramp types

Where a destination is provided, it should be used in place of name for ramps.
/cc @freenerd

This should already be the case, with destination taking precendence over name strings in general https://github.com/Project-OSRM/osrm-text-instructions/pull/27/files#diff-168726dbe96b3ce427e7fedce31bb0bcR93

Fixed in #27


Using MySql instead of MS Sql Server

If I try to use MySql in this DDD environment should I install the Connector and the necessary packages in all projects or just in DATA and Identity?

already figured out, is not that difficult, but has a lot of hand work


Update travis-ci environment and fix some build issues

The build is currently failing with ghc 8.2.2, because that job includes checks with hlint that complain:
src/Data/Morphism/Cata.hs:91:1: Suggestion: Use newtype instead of data
Found:
  data CataOptions = CataOptions{cataName :: String}
Why not:
  newtype CataOptions = CataOptions{cataName :: String}
Note: decreases laziness

src/Data/Morphism/Cata.hs:180:19: Suggestion: Redundant bracket
Found:
  (FunD funName . (: [])) <$> funImpl cons
Why not:
  FunD funName . (: []) <$> funImpl cons

src/Data/Morphism/Cata.hs:212:23: Warning: Use <$>
Found:
  do argNames <- replicateM (length . conArgTypes $ c)
                   (VarP <$> newName "a")
     return (ConP (conName c) argNames)
Why not:
  do ConP (conName c) <$>
       replicateM (length . conArgTypes $ c) (VarP <$> newName "a")

Wow, this PR seems to do a whole lot more than just adding support for GHC 8.4...
In particular, it's a pretty huge change to .travis.yml! Is that really needed - how does this new .yml file work?

That script is auto-generated by https://github.com/haskell-CI/haskell-ci/. I find it super helpful and I use it all the time for my projects.

I'm not sure what to make of "too much" in the context of CI testing and verification. I reckon that "as much as possible" would be the desired category here ...

@peti I meant 'too much' as in 'too much of a change'; to verify that building with GHC 8.4.2 works, it appears that just two lines of diff are needed (cf. #9 ). This is because I'm trying to get away with the documented defaults as much as possible. :-)
The haskell-ci stuff looks comprehensive, but it also does more than I need right now. I just read that the GitHub page explains

At time of writing Travis-CI has support for building Haskell packages but only for a single GHC configuration (i.e. Haskell Platform 2<IP_ADDRESS> with GHC 7.4.1).

...maybe the need for something like haskell-ci is less pressing than it used to do, given that Travis CI does support building with many GHC configurations in a matrix?
In any case, there's another patch which caught my eye, but I'm not sure what it's needed for: in
8bf7228 you're introducing hspec-discover as build-tools dependency of the tests. Why is that needed?

A follow-up on the question about hspec-discover: of course it is a dependency, what I meant to ask is why it's needed because there seems to be something else which pulls that in already, right? I guess it does not harm to explicitly define that dependency.

@peti I found a related comment which explains

hspec-discover is a transitive dependency of hspec. In the past (with cabal sandboxes and stack) it just worked to depend on hspec and hspec-discover would be on the path when the test suite is build.

I.e. it appears that simply depending on hspec is/was enough. However, with cabal new-build, this is not the case anymore. Did you maybe need to add it because you're using new-build?

@peti More tests sound good - but I'd really like to only include stuff which I actually fully understand (and can maintain). I think I'll take this slow and extend the set of tests over time - thanks for bringing haskell-ci to my attention!

The haskell-ci script is using new-build, among other things, and its tests failed without the proper dependency declaration.

I'd really like to only include stuff which I actually fully understand (and can maintain)

That strikes me as a rather irrational position. You are using ghc to compile your code even though I'm pretty sure that you don't fully understand every single detail of the compiler and you could probably not maintain it by yourself. So how is that different from using a straight-forward Haskell program to generate a CI script for Travis-CI? The whole point of re-using that tool is that you don't have to understand every single bit of it and you don't have to maintain it personally, yet you can still benefit from the intelligence and effort other people have put into that tool. Consciously deciding to stick with a less sophisticated tool for no apparent reason does not sound like a very smart thing to do ...
Anyway, it's your project and you're entitled to run it any way that pleases you. Just my 2 cents.

Please don't get me wrong, I see your point - however, contrary to e.g. ghc, the operating system or whatnot, the .travis.yml file which is generated by haskell-ci gets checked into my code repository, i.e. it is included in the source code (unlike ghc) repository and I (need to) maintain it. I need to understand the effects of that script (i.e. what does it make Travis CI do?). If I happen to use a new version of haskell-ci, I will still need to review (and maybe adjust?) the output it generates before I can commit it.
I think your PR gave a few good examples: suddenly, there was an (to me inexplicable) need to add hspec-discover as a dependency of the tests. I don't want to just add that dependency without having the full picture, i.e. what is it that introduces this dependency and why wasn't this dependency required so far.
Furthermore, the GHC 8.2 build suddenly stopped working since apparently the Travis script generated by haskell-ci also enables hlint (and it does so in a way which causes hlint diagnostics to make the build fail). I'm a big fan of hlint in some cases (not so much in others), but in any case it's a separate topic which is independent of adding support for a new GHC release.
So I very much appreciate the haskell-ci-based Travis script as 'food for thought' but I'd prefer to take multiple small steps instead of taking a big leap and going all-in with haskell-ci. :-)

The .travis.yml file which is generated by haskell-ci gets checked into my code repository, i.e. it is included in the source code (unlike ghc) repository and I (need to) maintain it.

I honestly don't follow. What is there to "maintain"? Why would you even care how that script works? I don't see a reason to. You just let the script do whatever it does, and every could of months, if you feel like it, you re-generate a new copy with the latest version of the generator to benefit from new features and/or new compiler versions. That .travis.yml is not a part of your product and it's not distributed as part of your Hackage tarball. I can't see what kind of maintenance work you would possibly have to do about it.

I think your PR gave a few good examples: suddenly, there was an (to me inexplicable) need to add hspec-discover as a dependency of the tests.

To me, this sounds like cause and effect are reversed. The need for that tool has always been there. It's not a consequence of the new script. On the contrary, the new script detected that issue, which has existed for a long time, you just never noticed because the CI environment you've been using does not test "cabal new-build".

I don't want to just add that dependency without having the full picture, i.e. what is it that introduces this dependency and why wasn't this dependency required so far.

I completely agree, but I don't see how that logically connects with a policy of "I won't use a superior test environment because if I would then it could potentially discover issues in my code that I'd need to understand and fix". Yes, if you have superior tests that discover issues the old one didn't, then you'll have to invest effort into fixing those. But now you seem to be saying that -- under these circumstances -- you'd rather not run those superior tests! That seems odd.

The GHC 8.2 build suddenly stopped working since apparently the Travis script generated by haskell-ci also enables hlint.

You can do one of two things about this:

Configure hlint to ignore those issues that you feel are illegitimate
warnings and therefore you'd rather not try to change your code to make the
tool happy.

Call the make-travis-yml generator without the --hlint option to disable the test entirely.

I read up on hlint and integrating it with Travis CI and it appears that it's quite easy these days, so I recently merged #11 which adds hlint checking. I'm quite new to Travis (this is the first project where I'm using it, which is also why I'm keen on understanding precisely what happens in .travis.yml :-)), I decided to run hlint in before_install -- such that in case it finds an issue, the build fails early (before installing dependencies or anything else which takes time).
I suppose the two other checks to consider would be doctest (though that has no effect right now, it seems - probably because the example(s) are not written in a format which doctest recognises?) and cabal check (this should be trivial and just another call right where hlint is running, if I understood things correctly). Sounds good?

It seems that doctest is not usable with this package right now: this is because the examples generate declarations (i.e. values of type Q [Dec]). However, ghci (which appears to what doctest feeds all the example code to) can only evaluate expression (i.e. Exp) splices at this point (cf. #7331 at this point). I.e. something like this fails (and this is what happens when running doctest):
λ: :set -XTemplateHaskell
λ: import Data.Morphism.Cata
λ: $(makeCata defaultOptions ''Bool)

<interactive>:4:3: error:
    • Couldn't match type ‘[Dec]’ with ‘Exp’
      Expected type: ExpQ
        Actual type: Q [Dec]
    • In the expression: makeCata defaultOptions ''Bool
      In the untyped splice: $(makeCata defaultOptions ''Bool)


Dear platformer.

Hello, you must know me as TGC OFFICIAL, but I have one concern. You never asked me for you take my CSS and games and name it yours, if you keep this up I will contact higher ups. We can talk this out over via Discord "saturn#0020"

saturn

Hello, My friend Smartfolio contacted you and he said you agreed to help with our site. Is that true?


🚨 error: Could not find package s.

Hi all,
I've just installed emojicode and written the 'Greetings' program from this doc. When I go to compile I get the following:
$ emojicodec greetings.emojic
🚨 error: Could not find package s.

I can't find instructions about installing s separately, but I assume it would come by default.
The steps I have gone through to get here are the following:

MacBook Pro (2014) running macOS High Sierra (10.13.6).
I installed via the 'magic installation' found here, with settings of:

version 0.6.3
macOS
curl

The installation seemed to run fine, ending with:

✅  Emojicode was successfully installed.

Can you please advise?
Thanks!

Thanks for reporting.
I just realised this myself yesterday. The build is actually broken (misconfigured). Please rebuild yourself using the instructions here or wait until I have time to replace the build.

I have just released 0.6.4 with a properly configured build: https://github.com/emojicode/emojicode/releases/tag/v0.6.4 😁

@thbwd Thanks so much! I've just installed it and everything is perfect 🙌


allowed netlify drawer to show up

What kind of PR is this?:

/kind chore

Codecov Report

Merging #174 (866287d) into main (42d035c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #174   +/-   ##
=======================================
  Coverage   77.62%   77.62%
=======================================
  Files          19       19
  Lines         219      219
  Branches       52       52
=======================================
  Hits          170      170
  Misses         45       45
  Partials        4        4

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 42d035c...866287d. Read the comment docs.


Roll Flutter Engine from ef760d6e1f13 to f98cefb7c40a (2 revisions)

https://github.com/flutter/engine/compare/ef760d6e1f13...f98cefb7c40a
2024-11-13<EMAIL_ADDRESS>[engine] force semantics action to post a task. (flutter/engine#56514)
2024-11-13<EMAIL_ADDRESS>[Impeller] Wrap provided FBO instead of defaulting to FBO0 and cleanup/document the texture API. (flutter/engine#56543)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


CLOUD-51757 Update to Docker 1.9.1 and overlay fs

@keyki

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/2007/


Closed invoice notification

Add Closed invoice notification

@Draal Was this intended to be posted here? Not sure if you meant to merge into your fork, but all of the imports are changed to github.com/autopilot3/recurly.
BTW if you add your fork as a remote to this repo, you can work on it inside of your GOPATH under blacklightcms/recurly but not have to change any of the imports. e.g. git remote add autopilot3 git@github.com:autopilot3/recurly.git and git push autopilot3 <branch>.


Make modules in file_transfer_protocol importable

#21

I don't disagree. I just didn't want to change their code at all. But I think you're right.

Actually, this branch doesn't appear to pass linting. I'll have to fix this

OK I think this is good now


Wrong DROP function DDL script generated from UI

System information:

Operating system (distribution) and version : Arch Linux 5.16.14-arch1-1
DBeaver version : <IP_ADDRESS>203212220
Additional extensions

Connection specification:

Database name and version: PostgreSQL 13.6
Driver name : PostgreSQL JDBC Driver
Do you use tunnels or proxies (SSH, SOCKS, etc)? Yes

Describe the problem you're observing:
The generated DROP DDL for function with return type TABLE is wrong. Return type is being included as parameter in SQL.
Steps to reproduce, if exist:
Here is simple function i created
CREATE OR REPLACE FUNCTION public.addition(d1 integer, d2 integer)
 RETURNS TABLE(id integer)
 LANGUAGE plpgsql
AS $function$
DECLARE
BEGIN
	return query Select d1 + d2;
END;
$function$
;

Try to delete it from UI, it should give following error
ERROR: function public.addition(integer, integer, integer) does not exist

View Script option is giving following SQL for that
DROP FUNCTION public.addition(in int4, in int4, int4);

But it should be
DROP FUNCTION public.addition(in int4, in int4);

Include any warning/errors/backtraces from the logs

Hello @samtech09
Thanks for the bug report.
This issue was fixed: https://github.com/dbeaver/dbeaver/issues/15563
Fix will be available in the next DBeaver release 22.0.2 at 04-04-2022


Add new Lint/UnneededWithIndex cop

Feature
This cop checks for unneeded with_index.
% cat /tmp/example.rb
# frozen_string_literal: true

ary.each_with_index do |v|
  v
end

ary.each.with_index do |v|
  v
end

% bundle exec rubocop /tmp/example.rb
Inspecting 1 file
W

Offenses:

/tmp/example.rb:3:5: W: Use each instead of each_with_index.
ary.each_with_index do |v|
    ^^^^^^^^^^^^^^^
/tmp/example.rb:7:10: W: Remove unneeded with_index.
ary.each.with_index do |v|
         ^^^^^^^^^^

1 file inspected, 2 offenses detected

Target Problem
The main aim is to remove unneeded with_index and clear the code. This will reduce unneeded code, thus improving performance.
The following is a benchmark script.
% cat /tmp/bench.rb
require 'benchmark/ips'
require 'uri'

Benchmark.ips do |x|
  x.report('each')            { %i[a b].each { |v| v } }
  x.report('each_with_index') { %i[a b].each_with_index { |v| v } }
  x.report('each.with_index') { %i[a b].each.with_index { |v| v } }
  x.compare!
end

It is the result of running the above benchmark script.
% ruby /tmp/bench.rb
Warming up --------------------------------------
                each   207.982k i/100ms
     each_with_index   147.822k i/100ms
     each.with_index   102.341k i/100ms
Calculating -------------------------------------
                each      4.123M (± 3.6%) i/s -     20.590M in   5.000595s
     each_with_index      2.380M (± 3.6%) i/s -     11.974M in   5.038652s
     each.with_index      1.374M (± 5.0%) i/s -      6.857M in   5.005021s

Comparison:
                each:  4123458.5 i/s
     each_with_index:  2379638.7 i/s - 1.73x  slower
     each.with_index:  1373928.0 i/s - 3.00x  slower

The benchmark is for reference, the main aim is to make the code correct. That is why I chose this cop as Lint department.
Other Information
This topic is about JRuby. JRuby has the following Issue related to with_index. And this Issue seems to regress in the latest JRuby <IP_ADDRESS>.
jruby/jruby#4610
RuboCop had one place using unneeded with_index. Perhaps it is the cause of a build error of JRuby <IP_ADDRESS> at #4703.
https://travis-ci.org/bbatsov/rubocop/jobs/272483544
This PR removes unneeded with_index, so I think that it will solve the above build error.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests(rake spec) are passing.
[x] The new code doesn't generate RuboCop offenses that are checked by rake internal_investigation.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

I think there could make this cop more generic and apply the same concept to each_with_object.
This isn't entirely a safe assumption to make about the code based on the number of arguments that are passed to the block. You can still access the index from the single argument.
[3] pry(main)> [1, 2, 3].each_with_index { |foo| puts "#{foo[0]}: #{foo[1]}" }
1: 0
2: 1
3: 2

The opposite of this is used to give meaning to the key and value when iterating over a hash
[6] pry(main)> {a: 1, b: 2, c: 3}.each { |hash| puts "#{hash[0]}: #{hash[1]}" }
a: 1
b: 2
c: 3
[7] pry(main)> {a: 1, b: 2, c: 3}.each { |key, value| puts "#{key}: #{value}" }
a: 1
b: 2
c: 3

Thanks for your comment. I also thought about integrating each_with_object. However, since we have not encountered a concrete use case that wrong block argument with with_object, this PR focused on with_index.

[3] pry(main)> [1, 2, 3].each_with_index { |foo| puts "#{foo[0]}: #{foo[1]}" }
1: 0
2: 1
3: 2

This behavior is JRuby's regression reported in jruby/jruby#4610. The original behavior of each_with_index is as follows, and index is not handed over.
% ruby -e '[1, 2, 3].each_with_index { |foo| puts "#{foo[0]}: #{foo[1]}" }'
1: 0
0: 1
1: 1

And this regression was fixed in JRuby <IP_ADDRESS> released several hours ago (!) .
http://jruby.org/2017/09/06/jruby-9-1-13-0
% rbenv shell jruby-<IP_ADDRESS> && ruby -e '[1, 2, 3].each_with_index { |foo| puts "#{foo[0]}: #{foo[1]}" }'
1: 0
2: 1
3: 2
% rbenv shell jruby-<IP_ADDRESS> && ruby -e '[1, 2, 3].each_with_index { |foo| puts "#{foo[0]}: #{foo[1]}" }'
1: 0
0: 1
1: 1

If it depends on the regression behavior, it will break when its regression is fixed. So every code should not depends on the regression behavior.
And I thought that the original each_with_index behavior can be autocorrected.

The example that I provided was using Ruby 2.4.1. This isn't a difference between Ruby and jRuby.
I do see that jRuby was not handling each_with_index with a single block argument properly.
% rbenv shell jruby-<IP_ADDRESS> && ruby -e '[1, 2, 3].each_with_index { |foo| puts "#{foo[0]}: #{foo[1]}" }'
1: 0
2: 1
3: 2
% rbenv shell jruby-<IP_ADDRESS> && ruby -e '[1, 2, 3].each_with_index { |foo| puts "#{foo[0]}: #{foo[1]}" }'
1: 0
0: 1
1: 1

Based on this example, it looks like there is still a bug in jRuby with how a single block argument is handled. I assume that jRuby is working to fix this so jRuby and Ruby function the same way. Assuming they do function in the same way, a single block argument to each_with_index is technically valid. Simply checking for a single block argument is not enough to determine if each or each_with_index was intended.
What would you think about changing this slightly to recommend passing multiple arguments to block methods that yield more than one item. each_with_index should have 2 arguments. This could be a little tricky because of the chained methods such as with_index.
[].each_with_object([]).with_index { |(element, object), index| something }

I see!
I'm wrong. I seem to have misunderstood this problem 💦

[].each_with_object([]).with_index { |(element, object), index| something }

I think about this use case, please wait for a while. Thank you for pointing out.

What's the status here?

I'm thinking about redesigning this cop considering the pointed out. I will update it soon. Please just a wait.

I examined the case of using only one block argument with ary.each_with_object.with_index.
In conclusion, in cases where only one block argument is used, index seems not to be passed.
So, there is no problem with the current method of checking by the number of block arguments.
In some cases I confirmed. It is exemplified by an Array object, but a Hash object is similar.
Receive index argument
This is a case that does not offense in this cop.
> %i[foo bar].each_with_object([]).with_index { |(v, ret), index| puts "v = #{v}, index = #{index}" }
v = foo, index = 0
v = bar, index = 1
=> []

This also applies to the following cases pointed out. And in this case is no offense.

[].each_with_object([]).with_index { |(element, object), index| something }

The following case is also no offense.
> %i[foo bar].each_with_object([]).with_index { |v, index| puts "v = #{v}, index = #{index}" }
v = [:foo, []], index = 0
v = [:bar, []], index = 1
=> []

with_index is used even though the block argument index is not used
It is a problem case with this cop. It used with_index, but there is no index in the block argument.
> %i[foo bar].each_with_object([]).with_index { |v| puts "v = #{v}" }
v = [:foo, []]
v = [:bar, []]
=> []

Below is also a offense case.
> %i[foo bar].each_with_object([]).with_index { |(v, ret)| puts "v = #{v}, ret = #{ret}" }
v = foo, ret = []
v = bar, ret = []
=> []

From these, we can judge whether it is offense by looking at the number of block arguments.
I thought that redesign was necessary, but it was not necessary from the above confirmation.
In addition, I added some of these use cases to test cases.

📝 I changed this cop name from Lint/UnneededWithIndex to Lint/RedundantWithIndex. I'd like to change the commit message to the following when do squash.
Add new Lint/RedundantWithIndex cop


Testsuite improvements

This is ready to merge too, the failures are LVM only and so unrelated to this branch (will be resolved by the other branch getting merged).

Pushed a few more fixes in there, looks like Jenkins is happy with it, so ready to merge.


Extract state keys from component views

Is your feature request related to a problem? Please describe.
Currently, we isolate all of the keys used to query the state into the implementations of the View traits.  This is good because it isolates all of the keys into one place (the read and write methods of the view traits), but it's still difficult to see a comprehensive list of all of the keys we use to store data in the state.
Describe the solution you'd like
Each component crate should have a (public) crate::state_key module.  This module should contain a bunch of bare functions, all of which return a KeyHash, and which take whatever arguments they need to produce the key, for instance:
fn token_supply(asset_id: &asset::Id) -> KeyHash {
    format!("shielded_pool/assets/{}/token_supply", asset_id).into()
}

fn known_assets() -> KeyHash {
    "shielded_pool/known_assets".into()
}

Then the rest of the component crate can access these keys as state_key::token_supply(asset_id), state_key::known_assets(), etc:
async fn token_supply(&self, asset_id: &asset::Id) -> Result<Option<u64>> {
    self.get_proto(state_key::token_supply(asset_id)).await
}

This is a totally mechanical refactoring, so it should be easy, if a bit tedious (rg format! would be helpful to catch any stray keys), but the upshot is that at the end, we'll have one file in each component that has a comprehensive list of keys we use to query the state.

Interested...can assign please

Interested...can assign please

@borngraced sure! I assigned you just now. One thing though is that this might end up on our critical path, so it'd be preferable if you could make a draft PR as soon as you start working on it, rather than when you're finished.  That way, if we discover that some later tasks need this in a time-critical way, and that we need to pick it up and carry it over the finish line to avoid blocking work, we'll be able to use the work you did.

Ok sure will do that..👍


[Event Request] Codeunit 5987 Serv-Posting Journals Mgt.: OnBeforePostCustomerEntry - Add Parameters

Hey,
we would need an additional parameter on the OnBeforePostCustomerEntry of type Codeunit "Gen. Jnl.-Post Line" that is passed on by reference.
Ideally both the OnBeforePostCustomerEntry and OnAfterPostCustomerEntry in the "Serv-Posting Journals Mgt." Codeunit would behave exactly the same to their Sales and Purchase posting counterparts:
Sales:
 [IntegrationEvent(false, false)]
local procedure OnBeforePostCustomerEntry(var GenJnlLine: Record "Gen. Journal Line"; var SalesHeader: Record "Sales Header"; var TotalSalesLine: Record "Sales Line"; var TotalSalesLineLCY: Record "Sales Line"; CommitIsSuppressed: Boolean; PreviewMode: Boolean; var GenJnlPostLine: Codeunit "Gen. Jnl.-Post Line")
begin
end;

[IntegrationEvent(false, false)]
local procedure OnAfterPostCustomerEntry(var GenJnlLine: Record "Gen. Journal Line"; var SalesHeader: Record "Sales Header"; var TotalSalesLine: Record "Sales Line"; var TotalSalesLineLCY: Record "Sales Line"; CommitIsSuppressed: Boolean; var GenJnlPostLine: Codeunit "Gen. Jnl.-Post Line")
begin
end;

Thank you.
Regards,
Tine

@tinestaric , what did you mean saying "Codeunit would behave exactly the same to their Sales and Purchase posting counterparts:"? I didn't find similar events for purchase side.

@RomanZanizdra In the Codeunit 90 Purch.-Post we have the same set of events OnBeforePostVendorEntry and OnAfterPostVendorEntry that have the same set of Parameters (with the Header and Lines of course pointing to the Purchase records) as the events on the Sales side.
The request is about having the same logic on the Service posting codeunit.
Purchase side Events
[IntegrationEvent(false, false)]
local procedure OnBeforePostVendorEntry(var GenJnlLine: Record "Gen. Journal Line"; var PurchHeader: Record "Purchase Header"; var TotalPurchLine: Record "Purchase Line"; var TotalPurchLineLCY: Record "Purchase Line"; PreviewMode: Boolean; CommitIsSupressed: Boolean; var GenJnlPostLine: Codeunit "Gen. Jnl.-Post Line")
begin
end;

[IntegrationEvent(false, false)]
local procedure OnAfterPostVendorEntry(var GenJnlLine: Record "Gen. Journal Line"; var PurchHeader: Record "Purchase Header"; var TotalPurchLine: Record "Purchase Line"; var TotalPurchLineLCY: Record "Purchase Line"; CommitIsSupressed: Boolean; var GenJnlPostLine: Codeunit "Gen. Jnl.-Post Line")
begin
end;

here are the Service OnBeforePostCustomerEntry and OnAfterPostCustomerEntry
As seen, they each lack certain parameters that are present on the Sales and Purchase equivalents:
[IntegrationEvent(false, false)]
local procedure OnBeforePostCustomerEntry(var GenJournalLine: Record "Gen. Journal Line"; ServiceHeader: Record "Service Header"; var TotalServiceLine: Record "Service Line")
begin
end;

local procedure OnAfterPostCustomerEntry(var GenJournalLine: Record "Gen. Journal Line"; var ServiceHeader: Record "Service Header"; var GenJnlPostLine: Codeunit "Gen. Jnl.-Post Line")
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. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Add smooth context sharing across threads for GPUs

Right now,kernel executions fail on multiple threads due to sharing contexts across threads. Allow for smoother sharing of contexts across threads.

Fixed with most recent release.


OAuth call issues "400 Bad Request"

When running google-drive-ocamlfuse, after clicking "Allow" on the page which reads "The application gdfuse OAuth2 Endpoint is requesting permission to access your Google Account", the next page says:

Error: Bad Request

Your client has issued a malformed or illegal request.

The title on the page is "400 Bad Request".
The client hangs at that point.

running with -debug simply outputs
Starting application setup (label=default, base_dir=/home/user/.gdfuse).
Opening log file: /home/user/.gdfuse/default/gdfuse.log

/home/user/.gdfuse/default/gdfuse.log says:
Saving application state in /home/drj/.gdfuse/default/state...done
Current version: 0.5.21
Setting up cache db...done
Setting up CURL...done
Saving application state in /home/drj/.gdfuse/default/state...done
Starting web browser with command: xdg-open "https://accounts.google.com/o/oauth2/auth?client_id=564921029129.apps.googleusercontent.com&redirect_uri=https%3A%2F%2Fgd-ocaml-auth.appspot.com%2Foauth2callback&scope=https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fdrive&response_type=code&access_type=offline&approval_prompt=force&state=uzG4BoGpQxNjRXcmujO0cO%2FfVqnkmnUSclz5PqzILa8"...done
Getting tokens from GAE proxy...not found, retrying
Getting tokens from GAE proxy...not found, retrying
Getting tokens from GAE proxy...not found, retrying

Please try removing the directory .gdfuse (rm -r /home/drj/.gdfuse) and start the authorization procedure again. If the problem persists let me know.

I've done that several times, with no change.
I've got version 0.5.21-0ubuntu1~ubuntu15.04.1.

OK, I see. Can you please try the alternative authorization method? The problem is that the error you obtain is caused by a page on Google servers that are not under my control, so I don't have logs and it's difficult to understand what went wrong.


package.json build and start issue

I trying to build this project in local :
"scripts": {
"start": "node --es-module-specifier-resolution=node --enable-source-maps .",
"build": "tsc -p tsconfig.json",
but below build script getting failed . Also above "start" value correct?

<EMAIL_ADDRESS>clean
rimraf coverage build tmp

$ tsc -p tsconfig.json
node_modules/discord.js/typings/index.d.ts:3799:14 - error TS2536: Type 'T' cannot be used to index type 'MappedChannelCategoryTypes'.
3799   ): Promise<MappedChannelCategoryTypes[T]>;
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
node_modules/discord.js/typings/index.d.ts:3846:14 - error TS2536: Type 'T' cannot be used to index type 'MappedGuildChannelTypes'.
3846   ): Promise<MappedGuildChannelTypes[T]>;
~~~~~~~~~~~~~~~~~~~~~~~~~~
Found 2 errors in the same file, starting at: node_modules/discord.js/typings/index.d.ts:3799
Can you please provide the correct script ? also please add build & start instruction.

Currently master branch is in WIP state, and I didn't suppose that someone will got happy to use this bot. You could take version tagged as 0.1 and would have try to run it. This version is a state before reworking (I'm lazy and didn't created dev branch...)
https://github.com/ZirionNeft/chatgpt-discord-daily-news-bot/releases/tag/0.1


Idea: Not only

Hi,
this is a feature idea. Ever thought of using your project to not only produce formatted text but also formatted input fields?
Cheers, Sven

I'm curious what you have in mind. Are you referring specifically to text inputs or all sorts? FormattedMessage or the message formatter itself could be used pretty easily to handle the input labels.

Hi Josh,
I reckon that all components under /lib/components return  elements in their render methods. I think that a lot of users would have advantages if they could use your great library for more than just text elements.
I see the reference on how to extend/ make use of the message formatter in own/separate components in the readme file.
So do you see your library more broadly e.g. you would be happy seeing lib/components grow or do you want others better use your library for building their own components?

If there are use cases that warrant additional components, I'm all for it. But given that it's pretty easy already to just use the getFormatter functions in your own components, I'd say that additional components need to have some functionality that you can't accomplish in a couple of lines already.

Ok. Thanks. I see that. Feel free to close this issue.


Proposed docs changes for 0.0.2 release

Docs changes and a changelog entry for 0.0.2

Great. It would be nice to be able to reference issues in future ;-)


BUG: "Deployed by" image blinks.

Go to the Previews page
Start new preview deployment

Result: "Deployed by" image blinks when a request with project deployments is posted.

MR: https://gitlab.com/dualbootpartners/idyl/uffizzi_app/-/merge_requests/1058/diffs

tested - OK

verified in production


uploading haddocks now takes several hours

With the switch from S3 to CF storage for stackage haddock documentation,
uploads with the awscli to the CloudFlare via their S3 compatibility is really taking too long.
Nightly currently takes about 3 hours and LTS around 4 hours.
In the past with S3 it just too minutes to do those uploads.
The long delay basically prevents curators continuing to do work while the upload is happening.

LTS-22.22:
Tue May 14 05:38:42 PM UTC 2024
:
Uploading docs to S3
Creating hoogle/orig.tar
Shelling out to AWS CLI to upload docs
Uploading snapshot definition to Github
:
real    248m5.723s
user    0m0.781s
sys     0m0.218s
Tue May 14 09:46:48 PM UTC 2024

Nightly:
Wed May 15 01:18:19 AM UTC 2024
:
real    188m39.820s                                                                                user    0m0.680s
sys     0m0.158s
Wed May 15 04:26:59 AM UTC 2024

Thanks for opening a ticket! From earlier discussion had been under the impression that this wasn't a blocker. I'll ping cloudflare to see if they've made any progress. It should not be happening.

Btw what's the actual evidence that the docs upload is the slow part of the process? I was unable to reproduce slow uploads, myself.
(That's not to say I don't believe it's slow. I'm just not sure how to reproduce it.)

It looks like the actual command run by curator upload-docs is aws s3 cp --only-show-errors --recursive --acl public-read --cache-control maxage=31536000.
I didn't try --recursive -- maybe that has an effect.

Currently nightly is uploading while creating the debug-logs using the new curator version - but as it seems CF R2 is slow - upload speed around 1.3 MiB/s (and we upload several GB's). Will update this comment once the upload finishes...

Interesting. I wonder if the problem is a fixed cost per file. I tried uploading one big file and it was fast. But curator is uploading a large number of small files. If there's a significant startup cost for each file, that could explain the issue.

That makes sense. From previous interactions with GCS, I observed a similar behavior. It's faster to archive all files into a single large upload than uploading individually.

Last log entry from Nightly-Upload - uploaded 9.8 GB by 1.0 MiB/s => ~ 2,75 h; one way to maybe improve upload time would be parallelization (e.g. there are tools that support parallel uploads - e.g. https://rclone.org/ and others). Maybe something you want to try @chreekat? 😉

My guess was wrong - also awscli uses multiple threads (https://docs.aws.amazon.com/cli/latest/topic/s3-config.html#max-concurrent-requests). So maybe we could tweak the setting to use e.g. 50 parallel connections?

@alaendle I just heard from CF and they have essentially recommended the same action. They recommended rclone as a better alternative since apparently it "gives more control over increasing concurrency/number of transfers".

I do wonder if there was some global aws config in the old system that implicitly increased the concurrency? I have a hard time believing that a sequential upload of all the files would be fast in any circumstance.

I am adding rclone to the nightly image now - though I dunno how to use it

Thanks @alaendle !  Great suggestion (specially also since I am not sure how to map aws to rclone)
So Bryan ran:
$ nix-shell -p awscli
[nix-shell:~]$ aws configure set default.s3.max_concurrent_requests 20

creating ~/.aws/config now, though we might need to move it to the correct location for the container.

The time output looks different format today: 703.10user 78.78system 1:45:09elapsed 12%CPU (0avgtext+0avgdata 166412maxresident)k
But I saw average upload speed was ~2.5MiB, so indeed it seems the above configuration change actually helped!!

Anyway I suggest this problem is now solved, thank you - makes a Big difference!

Awesome! Thanks @juhp !


[Bug]: 4.Snake Game

Give a brief about the bug ✍

The home icon and the heading are not properly aligned in the navbar
What is the expected behavior? 🤔
Proper alignment of the two elements
Provide step by step information reproduce the bug 📄
I will align them in the same line for a good look
Please assign me this issue @kunjgit
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @suma30 , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#3059

Hey @suma30 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Add Error Handling, Logging, and Security Improvements to Database Script

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Hi @owner
I'm a participant in GSSoC and I'm interested in working on this issue. Could you please assign it to me?
After reviewing the script, I've identified a few areas that could benefit from improvement:

Error Handling: It seems that error handling is currently minimal. Adding try-catch blocks around database operations could help catch and handle errors more gracefully.

Environment Variables: Sensitive information like the database URL and password is hard-coded in the script. It's safer to retrieve this information from environment variables for security reasons.

Logging: While there are some console.log statements, more informative logging could be added to track the progress and status of database operations, making debugging easier.

Code Organization: The script could be refactored into smaller, reusable functions to improve readability and maintainability.

Testing: Unit tests could be written to ensure that individual functions behave as expected and catch bugs early on.

Optimizations: Consider optimizing database queries for better performance, especially for large datasets.

Security: Implementing best practices for handling sensitive data, such as password hashing and input validation, can improve security.

I'm eager to tackle these challenges and contribute to the project. Please assign the issue to me, and I'll get started right away!
Thank you!
Ashutosh Singh
(GSSOC Participant)
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I think , Foodies Web follows almost 5 rules for the give 7 points , there's no need right now to make any changes !


cmd: tikv-ctl upgrade to futures 0.3

What problem does this PR solve?
Problem Summary:
What is changed and how it works?
Change tikv-ctl.rs to use futures 0.3
Check List
Tests

No code

/test

/run-all-tests

LGTM

/merge

/rebuild

/run-all-tests

/run-all-tests

/lgtm

/merge

/run-all-tests

/merge

/merge

/merge

/merge


Valid TrustThresholdFraction and further simplify tests code

ref: https://github.com/interchainio/tendermint-rs/pull/141#issuecomment-581477550 and https://github.com/interchainio/tendermint-rs/pull/132#discussion_r373813746

[ ] Referenced an issue explaining the need for the change
[ ] Updated all relevant documentation in docs
[ ] Updated all code comments where relevant
[ ] Wrote tests
[ ] Updated CHANGES.md

Codecov Report

Merging #142 into master will decrease coverage by 0.05%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #142      +/-   ##
==========================================
- Coverage   36.08%   36.02%   -0.06%
==========================================
  Files          90       90
  Lines        3140     3145       +5
  Branches      490      495       +5
==========================================
  Hits         1133     1133
  Misses       1751     1751
- Partials      256      261       +5

Impacted Files
Coverage Δ

tendermint/src/lite/types.rs
52.77% <0%> (-1.51%)
:arrow_down:

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 0ca3e8c...b86b69e. Read the comment docs.

@ebuchman when can I start adding @Shivani912 as a reviewer? :-)

I'll merge in master shortly. @Shivani912 do you feel comfortable to take over the remaining test related issues?

Actually, I think we should merge #151 first, as it touches upon error handling and then resolve merge conflicts.

test related issues

resolved all merge conflicts


Unable to complete pytorch tests using recommended installation guide.

Hello, I'm trying to set up an environment for training some models using pytorch using my AMD GPU.
OS: ubuntu 20.04
gpu: rx 570
cpu / mobo: MSI B450 + Ryzen 5 1600
I'm simply attempting to set up using the official guide. I'm able to install ROCm itself to see the device in clinfo, however, I next try to follow: https://rocmdocs.amd.com/en/latest/Deep_learning/Deep-learning.html#recommended-install-using-published-pytorch-rocm-docker-image
I get the the step of running the pytorch tests:
PYTORCH_TEST_WITH_ROCM=1 python3.6 test/run_test.py –-verbose
This runs for a long while before getting to a few similar errors like the following:
ERROR: test_context_manager_test (test_jit_cuda_fuser.TestPassManagerCudaFuser)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/root/.local/lib/python3.6/site-packages/torch/testing/_internal/common_utils.py", line 815, in wrapper
    method(*args, **kwargs)
  File "/var/lib/jenkins/pytorch/test/test_jit_cuda_fuser.py", line 518, in test_context_manager_test
    t_jit(x, y)
  File "/root/.local/lib/python3.6/site-packages/torch/testing/_internal/common_utils.py", line 118, in prof_func_call
    return prof_callable(func_call, *args, **kwargs)
  File "/root/.local/lib/python3.6/site-packages/torch/testing/_internal/common_utils.py", line 115, in prof_callable
    return callable(*args, **kwargs)
RuntimeError: Running the CUDA fuser requires a CUDA build.

----------------------------------------------------------------------
Ran 18 tests in 1.112s

FAILED (errors=11, skipped=6)
Traceback (most recent call last):
  File "test/run_test.py", line 718, in <module>
    main()
  File "test/run_test.py", line 707, in main
    raise RuntimeError(err)
RuntimeError: test_jit_cuda_fuser_legacy failed!

I've not been able to find any more information about this error. Can anyone provide some pointers so I could start to look into solving it?

Thank you for reaching out and for your comments about the Installation guide for Deep Learning. We are checking with the QA team about the errors, and we will get back to you as soon as we hear from them.
AMD ROCm Documentation Team

Hi @sona1111
Thanks for reaching us.
We are not officially supporting RX570 and request you to check our Documentation for the supported list of GPUs @ https://github.com/RadeonOpenCompute/ROCm#supported-gpus
I am pasting for your reference too.
Supported GPUs
Because the ROCm Platform has a focus on particular computational domains, we offer official support for a selection of AMD GPUs that are designed to offer good performance and price in these domains.
Note: The integrated GPUs of Ryzen are not officially supported targets for ROCm.
ROCm officially supports AMD GPUs that use following chips:
GFX9 GPUs

    "Vega 10" chips, such as on the AMD Radeon RX Vega 64 and Radeon Instinct MI25

    "Vega 7nm" chips, such as on the Radeon Instinct MI50, Radeon Instinct MI60 or AMD Radeon VII,

CDNA GPUs
    MI100 chips such as on the AMD Instinct™ MI100


[oxlint] result inconsistencies for rule no-cycle

There are inconsistencies in the result of oxlint using --threads N and in the results compared to eslint.
There are two types of problems:

inconsistencies with the --threads option. When I use --threads 1 I get 2 results and when I use --threads 10 I get 5 results. The expected is 7 cycles; what eslint is reporting.
Eslint reports 7 cycles while oxlint reports only 5 of them. The 7 cycles are explained in the test repository.

Reproductible repo: https://github.com/folkvir/oxc_no-cycle_test
The cycles are explained in the README

Regarding different number of cycles comparing to eslint - I think both results are correct because a cycle is formed, the difference seems to come from a depth first or a breath first search.
I'll look into --threads next.

I can't fix the mismatch for the time being, it is a bit difficult figure out what's wrong with --threads=1.
Given that no-cycles are still being detected for all the entry files, I'm going to close this as won't fix for now.


Clean up demos

Remove:

Don't use http://www.sciencekids.co.nz resources in the demos (the flags). Use emoji for example.

I already thought on this when adding the demo, so images are intentional to demonstrate that we could use any arbitrary html content as selected item but the selected value is the text content. Actually they are flags but in my first version was pictures of faces.
So if I add flags as part of the texContent like <vaadin-item>🇪🇸 Spain</vaadin-item> final dropdown.value == "🇪🇸 Spain"
One solution could be the following, but needs to set labels which is something that I wanted to avoid at the beginning.
 <vaadin-item label="Spain">
  <span style="margin-right: 10px">🇪🇸 </span> Spain
 </vaadin-item>

We already depend on external images in other components demos

About the noise in index.html was fixed in https://github.com/vaadin/vaadin-dropdown-menu/commit/b95f8d0bdf5bfab8149faaf3fb03ea517591341a

Use the label, that's what it's for.

label was already demoed in another demo. Leaving images so as the user can focuses in  the feature we want to demonstrate. Images now are aligned with the used in other examples


Config map is not created on OSO

Steps to reproduce:

Login to OSIO.
Create a space and add a codebase from quickstarts - Vertx with Config map
Wait until stage deployment is completed.

io.fabric8.kubernetes.client.KubernetesClientException: Failure executing: GET at: https://<IP_ADDRESS>/api/v1/namespaces/yivantso-stage/configmaps/app-config. Message: Forbidden!Configured service account doesn't have access. Service account may have been revoked..

This is because app-config map is never created. As a result, the app is broken

As indicated in the doc, you need to give the "read" permission on your project:
oc policy add-role-to-user view -n $(oc project -q) -z default

Also, ensure that you have created the config map explicitly:
oc create configmap app-config --from-file=app-config.yml

Check https://appdev.openshift.io/docs/vertx-runtime.html#mission-configmap-vertx to get more details.


Template fix for keycloak v12

Fix for templates so that the plugin works with keycloak v12.
The 'displayWide' param is not longer supported in templates it seems.

@mschielmann  Thanks for your contribution.


Support for serving static files

If you are building a serverless website, Static files like Javascript, HTML or PNG are usually uploaded to S3 in a separate build step. This works when you are deploying and running code in Lambda. But when using SAM Local, you need these resources served by SAM Local CLI
Ex:
localhost:3000 -> serves the index.html
localhost:3000/myfile.js -> should serve Javascript
localhost:3000/static/image.png -> should serve images
Today, the only endpoint exposed via SAM Local are the APIs configured in your SAM file. This is not sufficient. We need to support serving static files from the directory where their Lambda function is mounted.
Proposal:
If a route is not found, before returning 404, have SAM Local check pwd or Lambda function's mount directory for static files matching the name/path. If found, return the file. Otherwise return 404. This will give you static file capabilities right out of the box, without any extra work.
We can expose --public-dir parameter to specify path to search for static files.

Should be quick to add.
https://github.com/gorilla/mux#static-files

I think it makes more sense to default the public/ for the --public-dir.
Two reasons:

The AWS CodeStar Node.JS Serverless project uses public/, so this would work out of the box - great for new customers and demos.

I don't think there would be many instances of people wanting to serve up both their Serverless app (java/node.js source) as well as their static assets, so I don't think that matches what people would deploy to AWS in their Lambda deployment package.

Thoughts?

Agree with 1.
Don't agree with 2. If I am building a serverless website, I need to serve static content.
Can we call it --static-dir instead of public-dir? It makes the intent more clear. Also can we allow multiple --static-dir? Frameworks like expressjs allow this. If a file with same name exists in both places, the directory specified first will take precedence. All of them will mount at / of the HTTP listener.

Will make the param --static-dir, but I don't think the CLI framework we're using allows multiple, so will have to leave that for now.
PR incoming.

Sounds good.

Sounds good.

Hi Sanath ,
In serverless there are a few custom plugins which help you in uploading a set of files/folders to a s3 bucket as part of the serverless deploy step.
Is there any alternative like that ?
I want to avoid the step of uploading the files manually .
I want to simply create a bucket as part of my SAM template and then upload files from a specific folder in my local directory to the created bucket.
Is this possible?


Sebastian wesolowski/issue18

:tada: This PR is included in version 3.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Advice on conditional slug uniqueness

My Doumentation model has a column parent_id with a default value of 0.
A model instance can be either a 'Documentation' or 'Documentation Page' by setting the parent_id.
Now my actual issue:
I only want slugs to be unique where slugs in my model are unique within the same parent_id to later on get a specific documentation and its page from the url.
The indented behaviour would look like this:
Docs A - Slug: page-a
Page - Slug: page
Page - Slug: page-2
Docs B - Slug: page-b
Page - Slug: page
Page - Slug: page-2
Docs B - Slug: page-b-2
Page - Slug: page
Page - Slug: page-2

If I understand correctly, this should be what you want:
https://github.com/cviebrock/eloquent-sluggable?tab=readme-ov-file#scopewithuniqueslugconstraints

Thanks for the advice! I must've missed that in the docs. I was able to achieve the desired result with:
    * Scope to apply unique constraints for slugs based on assigned parent.
    */
    public function scopeWithUniqueSlugConstraints($query)
    {
        if ($this->parent_id == 0) {
            return $query->where('parent_id', 0);
        }
        return $query->where('parent_id', $this->parent_id);
    }```

Thanks for the advice! I must've missed that in the docs. I was able to achieve the desired result with:
    * Scope to apply unique constraints for slugs based on assigned parent.
    */
    public function scopeWithUniqueSlugConstraints($query)
    {
        if ($this->parent_id == 0) {
            return $query->where('parent_id', 0);
        }
        return $query->where('parent_id', $this->parent_id);
    }```

You could also just remove that if {…} block entirely and get the same result. 😉On Jun 8, 2024, at 2:56 AM, Stanley Richter @.***> wrote:
Thanks for the advice! I must've missed that in the docs. I was able to achieve the desired result with:
* Scope to apply unique constraints for slugs based on assigned parent.
*/
public function scopeWithUniqueSlugConstraints($query)
{
if ($this->parent_id == 0) {
return $query->where('parent_id', 0);
}
return $query->where('parent_id', $this->parent_id);
}```
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

You could also just remove that if {…} block entirely and get the same result. 😉
public function scopeWithUniqueSlugConstraints($query)
{
    return $query->where('parent_id', $this->parent_id);
}


Update meeting-settings-in-teams.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/6246
Added note.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@LanaChin PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@LanaChin @SerdarSoysal

This request will be superseded by planned content changes and is being rejected as a result, because it will conflict with other upcoming changes to the article.


feat: toast

Try out this version of Leather — Extension build, Test report

This PR implements a new Toast using a Radix primitive and a reference implementation from them to manage toast notifications: example codesandbox
I am proposing here we abandon react-hot-toast bc the library is outdated, ref their Github repo here. There are lots of outstanding issues and only one commit in 2024.
The Radix implementation, using React Context, allowed me to simply swap out our current implementation with a new context hook, toast = useToast(). As an example, our implementation is still, toast.success('Message'). The context also includes: info, error, and promise. The implementation here allows multiple toasts to occur, but we can customize that if we want. There is a lot we can do with this in the future.
The only place I wasn't able to use the hook was in the debug code, but I don't think it is necessary to show a toast there bc a user is already in the console. I switched that one to a logger message.
Stories included...

@fabric-8 @mica000 I'm curious if we have (or should create) design guidance for whether to show the toast always at the top or bottom of the viewport? We seem to show in both places without consistently, though I suspect we should decide just one so the user can rely on it there.

@markmhendrickson It's actually supposed to alway appear on top

Thanks, got it. @fbwoolf could we do a general audit of placement as part of this work stream or a new separate issue?

Thanks, got it. @fbwoolf could we do a general audit of placement as part of this work stream or a new separate issue?

The changes here position it only at the top for all instances. It was refactored based on the design spec which only positioned the toast now at the top center.

Not a blocker, but maybe s.th. we can look into later: I think the tick animation for the success toast was a nice touch, would be nice if we can bring it back.

Do you mean with the old checkmark icon? This is the old one to ref: https://github.com/timolins/react-hot-toast/blob/main/src/components/checkmark.tsx


Get-Team fetches Team by -GroupId in 5 secs but takes 3 to 5 minutes with -MailNickName

Hi! I would like to know if its normal for this command to take only 4 secs to fetch a Team from server when GroupId parameter is used, and about 3 to 5 minutes (!) to fetch it when -MailNickName parameter is used. I´m working with clients with massive records of Microsoft Teams and they can provide me only display name or mailNickName. Causing a really slow script that takes more time than a human to complete its tasks. Does the search algorithim searches for teams with this -MailNickName in ALL the teams of the world? Or it only searches in the teams within the organization where the autenthicated user belongs?

@officedocsbot assign @yogkumgit

Hello @FedeFigueredo Thanks for your feedback.
This issue system is specifically to report issues currently present in documentation, for feature requets, bug reports or in this case, support (that could lead to a bug report) I need to kindly redirect you to creating a support ticket directly on your tenant using the Office 365 admin panel.
Hope this information gets you in the right direction to answer your query.
Thank you

@get-itips Thank you very much for the contribution and sharing this explanation. @FedeFigueredo
Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution.
We will be very happy if you can share the outcome or resolution with us. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also
Link: https://docs.microsoft.com/office365/admin/contact-support-for-business-products

@officedocsbot close

Thanks for your reply! I will consult with my Tenant. Greetings


Missing Googlebot keyword in middleware botUserAgents list.

These days I am very interested in polymer's pwa starter kit which using prpl-server which using rendertron.
So, when I try using Google's fetch as Google service for testing SEO, I got blank pages.

I fixed that with adding userAgentPattern option in prpl-server's cli.ts (not configurable from CLI !!, have to fix hard coded)
I wonder if there is a reason for not including Googlebot in this list?
Thanks.

Hi,
Yes, this is intentional. Please review https://medium.com/dev-channel/polymer-2-and-googlebot-2ad50c5727dd.
Thanks,
Sam


Code Question

First of all, thank you for your paper and codes, it really helps me a lot.
Actually, I have a question about your code. This is the code of your model.
class G2_GNN(nn.Module):
    def __init__(self, nfeat, nhid, nclass, nlayers, conv_type='GCN', p=2., drop_in=0, drop=0, use_gg_conv=True):
        super(G2_GNN, self).__init__()
        self.conv_type = conv_type
        self.enc = nn.Linear(nfeat, nhid)
        self.dec = nn.Linear(nhid, nclass)
        self.drop_in = drop_in
        self.drop = drop
        self.nlayers = nlayers
        if conv_type == 'GCN':
            self.conv = GCNConv(nhid, nhid)
            if use_gg_conv == True:
                self.conv_gg = GCNConv(nhid, nhid)
        elif conv_type == 'GAT':
            self.conv = GATConv(nhid,nhid,heads=4,concat=True)
            if use_gg_conv == True:
                self.conv_gg = GATConv(nhid,nhid,heads=4,concat=True)
        else:
            print('specified graph conv not implemented')

        if use_gg_conv == True:
            self.G2 = G2(self.conv_gg,p,conv_type,activation=nn.ReLU())
        else:
            self.G2 = G2(self.conv,p,conv_type,activation=nn.ReLU())

    def forward(self, data):
        X = data.x
        n_nodes = X.size(0)
        edge_index = data.edge_index
        X = F.dropout(X, self.drop_in, training=self.training)
        X = torch.relu(self.enc(X))

        for i in range(self.nlayers):
            if self.conv_type == 'GAT':
                X_ = F.elu(self.conv(X, edge_index)).view(n_nodes, -1, 4).mean(dim=-1)
            else:
                X_ = torch.relu(self.conv(X, edge_index))
            tau = self.G2(X, edge_index)
            X = (1 - tau) * X + tau * X_
        X = F.dropout(X, self.drop, training=self.training)

        return self.dec(X)

I thought n-layers was the number of layers. But when I looked at the code, I realized that it means that one layer is used as nlayers. I think this means that whether I insert the number of layer such as 16 or 32, the number of layer is always one.
May I ask why you implement the code like this? or Did I misunderstand?
And I also want to ask, if I want to check the model performance, what is the order of the Model, do I just pile the layer G2 up?

Thanks for reaching out. It is a multi-layer GNN, however, in our case we share the same parameters among the different layers. That's why we do the for-loop over the number of layers, but calling the same GNN each time. The reason for that is:

it gives the same and sometimes better performance than using different weights for each layer
it corresponds to a graph-dynamical system modeled by a differential equation (please look at our paper for that)

I don't understand your second question, i.e., about checking the model performance.

Thank you so much for your answer. I really appreciate it. I was only thinking of layers with different parameters.
And the second question mean is, I just want to run this code on the condition of different parameters among the different layers.

@Hamsss Did you have any success in adapting it to non-shared parameters.
@tk-rusch Is this a requirement for the Gradient Gating framework to work? Sorry if this is a novice question.
Thanks you both in advance.

No, it's absolutely not! You can extend it to using different parameters among different layers by simply:
self.convs = nn.ModuleList()
for i in range(nlayers):
  self.convs.append(GCNConv(nhid, nhid))

and then in forward():
for i in range(self.nlayers):
  X_ = torch.relu(self.convs[i](X, edge_index))

You can do the same for the GG layers.

@tk-rusch Thank you for the swift reply and example.

Just to be clear, I can have multiple layers that do not share weights, and I can apply a gradient gate to each of these layers. Is that correctly understood? This does not break anything on the theoretical side, correct? Thank you in advance!


Content-Security-Policy prevents resource loading?

When trying to load http://github.com , I get the following warning for several resources:
[info] [remote] Refused to load style from 'https://a248.e.akamai.net/assets.github.com/assets/github-fdebe8d3f60746fb87c763a59741ff520ae3d8e8.css' because of Content-Security-Policy.
It seems to be related to the CSP header in the server's response:
Content-Security-Policy:default-src *; script-src https://github.com https://a248.e.akamai.net https://jobs.github.com https://ssl.google-analytics.com https://secure.gaug.es https://collector.githubapp.com https://gist.github.com; style-src https://github.com https://a248.e.akamai.net 'unsafe-inline'; object-src https://github.com https://a248.e.akamai.net
Interestedly, the CSP header should allow loading resources from the domains specified in the CSP header, but for some reason, they can't be loaded.
https://github.com/blog/1477-content-security-policy
I'm using phantomjs 1.9. Also, setting --web-security=no does not help as well.
Should this be the normal behavior?
Is there any way to remove/filter a response header (i.e. remove the CSP header) and then reload (without making an actual http request) the page, or any other advice to mitigate this?

I'm experiencing the same problem.  The tests work fine in Chrome and Firefox but fail in PhantomJS.  Curiously, the PhantomJS tests run fine on OSX but fail on Windows.  The Content-Security-Policy is preventing PhantomJS from retrieving css and javascript resources so my automated tests are failing.  Any chance we can get this fixed?  Please let me know if you need further details.

I was having the same issue with phantomjs version 1.9.0; updated to version 2.0.0 and it works perfectly

If I remember correctly I did experiment with 2.0.0 and it did not help in my case.
Upvoting this issue: +1:

Still having this problem with PhantomJS 2.0.0 , run with a CasperJS 1.1.0-beta3 script. Upvoting as well.

Having this issue too. Any workarounds?

I believe this should be closed as a duplicate of #13114.

Anything new here ?


Stale injected branch protection info

Expected Behavior
When changing the GH brach protection rules to run macos-13 instead of the (unsupported) macos-12, mergify should update the injected branch protection rules.
Actual Behavior
I merged https://github.com/dfinity/motoko/pull/4791 a few days back. Our admins changed the branch protections, now they don't contain macos-12 references any more.
Instead the injected rules (even for fresh PRs!) won't get updated, and stay stale. This renders the PRs non-merged until I manually intervene. See below:

Here is a live example: https://github.com/dfinity/motoko/pull/4802
Steps to Reproduce the Problem

Change one target OS to a bumped one in the build matrix (macos-12 -> macos-13) and update branch protections for the main (default) branch.
Re-run checks
Check Mergify summary: Injected rules are stale

Running @mergifyio refresh won't help.
Specifications

Pull Request URL: https://github.com/dfinity/motoko/pull/4802
Mergify Config URL: https://github.com/dfinity/motoko/pull/4802/checks?check_run_id=33986327476

Recent PRs have the correct check in that place. Closing as not reproducible.


Environment Variable "MONGODB_URI" references Secret "mongodb_uri", which does not exist.

<EMAIL_ADDRESS>So, I'm new to this, but I keep getting this error on vercel. What should I do?
I've ran it on my computer at localhost:3000 fine, without mongodb, but when I try to use vercel, it seems like mongodb is missing. I keep getting this error: "Environment Variable "MONGODB_URI" references Secret "mongodb_uri", which does not exist."
What should I do?

Looks like there was a typo in the README. The secrets you'll need to add are MONGODB_URI and MONGODB_COLLECTION.
You can do this from the cli or Vercel's admin page

Sorry, I was too quick here.
Have you added the secret ´mongodb_uri´ to Vercel?
From the cli vc secrets add mongodb_uri <your-connection-string> or admin page

I tried, but I get an error using this command:
Vercel CLI 20.1.0
Error! The specified scope does not exist
Learn More: https://err.sh/now/scope-not-existent
I've already tried going to this URL and searching it on google. Couldn't find an answer

Try to remove the scope setting i now.json https://github.com/rubynor/bigfive-web/blob/master/now.json#L3
I'm not sure if that DB-URL will work.
We use something like mongodb://user:password@host/db

@maccyber sorry, I'm completely new to all this... I know user and password because that's pretty straightforward, but I don't know what's my db nor host, couldn't find any of it on mongodb

Seems that you also have to change this: https://github.com/rubynor/bigfive-web/blob/master/nuxt.config.js#L194 to the URL you deployed it to + /api

Changed and commited to repo. How do I know if it is changed at vercel?
Also, is there a way to test this without going all over the test?

Vercel gives you an unique URL for every deploy, but it should also provide you with an URL for the project (for me it's bigfive-web.rubynor.vercel.app) or you can add your own custom domain. Read the vercel docs for how this is done.


Updated preview information and language.

Included some key information such as.

This is only for Enterprise Agreement customers
Customers must still create the initial Mail Enabled User object themselves

I noticed a typo in the PR, it says 'deocrate' instead of 'decorate'


exploitdb: 2023-10-10 -> 2023-10-14

Diff: https://gitlab.com/exploit-database/exploitdb/-/compare/refs/tags/2023-10-10...2023-10-14
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 261265 run on x86_64-linux 1

1 package built:

exploitdb


Compiling Error

I added this sdk but its giving error on other pods
created new Project added sdk getting error

I am using M1 chip

its fixed now


arch: Review get_gpu_info

@FabioLuporini , please check review comments. We could also improve another points (e.g.,  get physical id also from lspci and nvidia-smi)

Thanks, merging


Unknown Error:1

Sometimes face detect successfully and sometimes it shows Unknown Error:1 Why?
Please help!

You've been blocked from  participating, posting the same thing 10+ times is not how you should behave and you know that. You've had plenty of warnings.